search

gmlc-dispatches / dispatches

Primary repository for distributed dispatches software tools
https://dispatches.readthedocs.io/
Other
12 stars 34 forks source link

Fix RE Notebook Check #181

Closed dguittet closed 1 year ago

dguittet commented 1 year ago

Addresses issue:

176

Legal Acknowledgement

By contributing to this software project, I agree to the following terms and conditions for my contribution:

  1. I agree my contributions are submitted under the license terms described in the LICENSE.md and COPYRIGHT.md file at the top level of this directory.
  2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.
dguittet commented 1 year ago

@ksbeattie @lbianchi-lbl There are failing tests still but the notebook completes in time

lbianchi-lbl commented 1 year ago

For reference, these seem to be the relevant updates to the source code cells:

image

codecov[bot] commented 1 year ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (3cad6e1) 93.74% compared to head (006633e) 93.74%.

:exclamation: Current head 006633e differs from pull request most recent head ec37b24. Consider uploading reports for the commit ec37b24 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #181 +/- ## ======================================= Coverage 93.74% 93.74% ======================================= Files 59 59 Lines 7130 7130 ======================================= Hits 6684 6684 Misses 446 446 ``` Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=gmlc-dispatches). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=gmlc-dispatches)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

lbianchi-lbl commented 1 year ago

Closing this in favor of #190 which is on a non-main branch.