Remove discontinuity handling in the callback

[ChrisRackauckas]

since it's just for monitoring, we should disable the extra saving that is occurring with it. By default it does a pre and post save at the callback points in the assumption that it's required for accurately handling the resolution of the discontinuity. But here we don't want that.

[ChrisRackauckas]

@joshday is there a way to test the behavior with the callback? None of the test cases have this callback, so I'm not entirely sure where it enters the web service. That said, I am 100% sure this handles the issue.

[joshday]

The tests that send requests to the running server will hit it.

[joshday]

You'll see this while running the tests. The callback is calling publish_to_rabbitmq to send data back to the client.

┌ Warning: RabbitMQ disabled - `publish_to_rabbitmq` with content {"iter":1,"params":[0.9999952915229029,5.919145546989561e-7,3.774131629178174e-6,2.0931364205487055e-7,8.067936912130783e-8,6.747733045656112e-8,1.6241788384319312e-9,3.0555809387394606e-12],"id":"sciml-f4162b51-a673-4472-bc4f-55a014c0298f","abserr":1.1498277159670557e-6,"retcode":"Success","time":0.4851521399160408}
└ @ SimulationService ~/github/simulation-service/src/SimulationService.jl:267

[codecov-commenter]

Codecov Report

Merging #94 (90f74c7) into main (09a3a7d) will decrease coverage by 0.91%.
The diff coverage is 100.00%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##             main      #94      +/-   ##
==========================================
- Coverage   80.64%   79.74%   -0.91%     
==========================================
  Files           2        2              
  Lines         310      311       +1     
==========================================
- Hits          250      248       -2     
- Misses         60       63       +3     

Impacted Files	Coverage Δ
src/operations.jl	89.13% <100.00%> (+0.07%)	⬆️

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more