Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adding nonhomologous_grid.py #2121

Merged
merged 11 commits into from
Apr 20, 2023
Merged

Conversation

sonachitchyan
Copy link
Member

📝 Description

Type: 🪲 bugfix | 🚀 feature | ☣️ breaking change | 🚦 testing | 📝 documentation | 🎢 infrastructure

This PR adds nonhomologous_grid.py to montecarlo/montecarlo_numba for further use in nonhomologous expansion implementation.

Also, link issues affected by this pull request by using the keywords: close, closes, closed, fix, fixes, fixed, resolve, resolves or resolved.

📌 Resources

Examples, notebooks, and links to useful references.

🚦 Testing

How did you test these changes?

  • Testing pipeline
  • Other method (describe)
  • My changes can't be tested (explain why)

☑️ Checklist

  • I requested two reviewers for this pull request
  • I updated the documentation according to my changes
  • I built the documentation by applying the build_docs label

Note: If you are not allowed to perform any of these actions, ping (@) a contributor.

@codecov
Copy link

codecov bot commented Aug 23, 2022

Codecov Report

Merging #2121 (13cb264) into master (a8034b2) will decrease coverage by 0.03%.
The diff coverage is 60.60%.

❗ Current head 13cb264 differs from pull request most recent head c591723. Consider uploading reports for the commit c591723 to get more accurate results

@@            Coverage Diff             @@
##           master    #2121      +/-   ##
==========================================
- Coverage   71.81%   71.78%   -0.03%     
==========================================
  Files         133      135       +2     
  Lines       12355    12388      +33     
==========================================
+ Hits         8873     8893      +20     
- Misses       3482     3495      +13     
Impacted Files Coverage Δ
.../montecarlo/montecarlo_numba/nonhomologous_grid.py 48.00% <48.00%> (ø)
tardis/montecarlo/tests/test_nonhomologous.py 100.00% <100.00%> (ø)

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

@andrewfullard andrewfullard self-requested a review September 19, 2022 14:29
@andrewfullard
Copy link
Contributor

Looks good, needs tests

@andrewfullard andrewfullard marked this pull request as draft February 20, 2023 15:09
@sonachitchyan sonachitchyan marked this pull request as ready for review March 15, 2023 19:04
Copy link
Contributor

@andrewfullard andrewfullard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks much nicer with np.roots :)

@sonachitchyan sonachitchyan requested a review from chvogl March 29, 2023 15:13
return factor


# @njit(**njit_dict_no_parallel)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this here?

@Rodot- Rodot- merged commit eb75df9 into tardis-sn:master Apr 20, 2023
light2802 pushed a commit to light2802/tardis that referenced this pull request May 27, 2023
* adding nonhomologous_grid.py

* got rid of unnecessary comments

* wiki link

* returning dv/dr from velocity function

* changing the name of roots to quartic_roots

* corresponding change to dv/dr usage

* adding tau_sobolev factor equation

* renamed function velocity to velocity_dvdr

* rewrote the quartic root finding

* added more cases to the tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants