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

import bambi as bmb in tests #662

Merged
merged 3 commits into from
Mar 30, 2023

Conversation

GStechschulte
Copy link
Collaborator

@GStechschulte GStechschulte commented Mar 29, 2023

This draft PR resolves #653 to avoid importing specific objects only in the test files of the test/ directory . For directories treated as packages in the init.py I use import bambi as bmb. Directories not treated as packages, I import the submodule individually. For example, from bambi.terms import CommonTerm

Tests are completed and passed locally:

  • Make sure all test pass.
  • Make sure your code passes black.
  • Make sure your code passes pylint.

GStechschulte and others added 2 commits March 29, 2023 22:25
* documentation on custom prior

* changes as suggeted by Tomás

* minor changes
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@GStechschulte GStechschulte marked this pull request as ready for review March 30, 2023 17:12
@codecov-commenter
Copy link

codecov-commenter commented Mar 30, 2023

Codecov Report

Merging #662 (aa19dac) into main (afcabb6) will increase coverage by 0.06%.
The diff coverage is 95.45%.

@@            Coverage Diff             @@
##             main     #662      +/-   ##
==========================================
+ Coverage   87.56%   87.62%   +0.06%     
==========================================
  Files          40       40              
  Lines        2629     2650      +21     
==========================================
+ Hits         2302     2322      +20     
- Misses        327      328       +1     
Impacted Files Coverage Δ
bambi/defaults/distributions.py 100.00% <ø> (ø)
bambi/defaults/families.py 87.50% <ø> (ø)
bambi/defaults/utils.py 87.50% <ø> (ø)
bambi/families/likelihood.py 85.36% <ø> (ø)
bambi/families/univariate.py 90.37% <95.00%> (+0.80%) ⬆️
bambi/families/family.py 93.25% <100.00%> (ø)

... and 1 file with indirect coverage changes

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

@tomicapretto
Copy link
Collaborator

Thanks Gabriel! :D

@tomicapretto tomicapretto merged commit d8444ee into bambinos:main Mar 30, 2023
@GStechschulte GStechschulte deleted the bambi-as-bmb-tests branch March 31, 2023 05:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use import bambi as bmb in tests
4 participants