Skip to content

Fix some code lints found with ruff #6545

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

Merged
merged 7 commits into from
Feb 25, 2023
Merged

Conversation

Armavica
Copy link
Member

I used the command ruff pymc --extend-ignore E501 --extend-exclude __init__.py and fixed some of the most prominent lints.

Checklist

Major / Breaking Changes

  • ...

New features

  • ...

Bugfixes

  • ...

Documentation

  • ...

Maintenance

  • Fix some code lints found with ruff

@codecov
Copy link

codecov bot commented Feb 24, 2023

Codecov Report

Merging #6545 (96b7160) into main (e1d36ca) will increase coverage by 0.00%.
The diff coverage is 96.87%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #6545   +/-   ##
=======================================
  Coverage   91.95%   91.96%           
=======================================
  Files          89       89           
  Lines       14933    14932    -1     
=======================================
  Hits        13732    13732           
+ Misses       1201     1200    -1     
Impacted Files Coverage Δ
pymc/step_methods/hmc/integration.py 81.66% <0.00%> (ø)
pymc/step_methods/hmc/nuts.py 97.27% <ø> (-0.02%) ⬇️
pymc/distributions/dist_math.py 87.42% <100.00%> (ø)
pymc/distributions/distribution.py 96.61% <100.00%> (+0.29%) ⬆️
pymc/gp/util.py 96.55% <100.00%> (ø)
pymc/logprob/transforms.py 96.13% <100.00%> (ø)
pymc/model.py 89.90% <100.00%> (ø)
pymc/model_graph.py 80.11% <100.00%> (ø)
pymc/sampling/jax.py 98.26% <100.00%> (-0.01%) ⬇️
pymc/variational/approximations.py 90.49% <100.00%> (+0.04%) ⬆️
... and 1 more

@michaelosthege michaelosthege merged commit b9a3dcc into pymc-devs:main Feb 25, 2023
@Armavica Armavica deleted the lints branch October 7, 2024 14:58
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.

2 participants