aesara-devs / aehmc

An HMC/NUTS implementation in Aesara
MIT License
33 stars 6 forks source link

Updates to copyright years, cast policy, and pre-commit #86

Closed brandonwillard closed 1 year ago

brandonwillard commented 1 year ago

I've update the step size dtypes to int64, but I imagine it would be better to perform that conversion somewhere within AeHMC; however, I don't know the best place for that at the moment.

@rlouf, if you get a minute, tell me where you think an astype could be performed so that users can specify int32 and avoid those Scan casting issues.

codecov[bot] commented 1 year ago

Codecov Report

Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (70f839b) compared to base (a459753). Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #86 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 12 13 +1 Lines 542 544 +2 Branches 32 31 -1 ========================================= + Hits 542 544 +2 ``` | [Impacted Files](https://codecov.io/gh/aesara-devs/aehmc/pull/86?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aesara-devs) | Coverage Δ | | |---|---|---| | [aehmc/trajectory.py](https://codecov.io/gh/aesara-devs/aehmc/pull/86/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aesara-devs#diff-YWVobWMvdHJhamVjdG9yeS5weQ==) | `100.00% <ø> (ø)` | | | [aehmc/\_\_init\_\_.py](https://codecov.io/gh/aesara-devs/aehmc/pull/86/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aesara-devs#diff-YWVobWMvX19pbml0X18ucHk=) | `100.00% <100.00%> (ø)` | | | [aehmc/algorithms.py](https://codecov.io/gh/aesara-devs/aehmc/pull/86/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aesara-devs#diff-YWVobWMvYWxnb3JpdGhtcy5weQ==) | `100.00% <100.00%> (ø)` | | 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=aesara-devs). 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=aesara-devs)

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