SciML / JumpProcesses.jl

Build and simulate jump equations like Gillespie simulations and jump diffusions with constant and state-dependent rates and mix with differential equations and scientific machine learning (SciML)
https://docs.sciml.ai/JumpProcesses/stable/
Other
135 stars 35 forks source link

format with JuliaFormatter attempt 87 br #383

Closed isaacsas closed 3 months ago

isaacsas commented 6 months ago

@domluna I tested your pr https://github.com/domluna/JuliaFormatter.jl/pull/792

There still seem to be some issues with moving args that fit on one line across many lines, and with sub-lines of function signatures not getting indented. I'll tag them below.

coveralls commented 6 months ago

Pull Request Test Coverage Report for Build 7261713574


Changes Missing Coverage Covered Lines Changed/Added Lines %
src/aggregators/directcr.jl 0 1 0.0%
src/aggregators/nrm.jl 1 2 50.0%
src/aggregators/rdirect.jl 1 2 50.0%
src/aggregators/sortingdirect.jl 1 2 50.0%
src/solve.jl 0 1 0.0%
src/spatial/spatial_massaction_jump.jl 6 7 85.71%
src/spatial/utils.jl 1 2 50.0%
src/aggregators/rssa.jl 0 2 0.0%
src/aggregators/rssacr.jl 2 4 50.0%
src/aggregators/ssajump.jl 8 10 80.0%
<!-- Total: 107 128 83.59% -->
Files with Coverage Reduction New Missed Lines %
ext/JumpProcessFastBroadcastExt.jl 1 50.0%
src/problem.jl 1 65.28%
src/spatial/spatial_massaction_jump.jl 2 90.74%
<!-- Total: 4 -->
Totals Coverage Status
Change from base Build 7210509298: 0.008%
Covered Lines: 2301
Relevant Lines: 2558

💛 - Coveralls
domluna commented 6 months ago

@isaacsas i think most of the issues are solved now - aside from some off by one by 1 errors

isaacsas commented 6 months ago

Ok, cool thanks for letting me know. If I have a chance I’ll try to check it out tomorrow morning before I head on holiday for a while and give you an update.