Skip to content
New issue

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

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

Already on GitHub? # to your account

Read model info to attrs with from_pystan #1353

Merged
merged 16 commits into from
Aug 16, 2020
Merged

Conversation

ahartikainen
Copy link
Contributor

@ahartikainen ahartikainen commented Aug 16, 2020

Description

Add PyStan metadata info.

Checklist

  • Follows official PR format
  • Includes a sample plot to visually illustrate the changes (only for plot-related functions)
  • Includes new or updated tests to cover the new feature
  • Code style correct (follows pylint and black guidelines)
  • Changes are listed in changelog

@codecov
Copy link

codecov bot commented Aug 16, 2020

Codecov Report

Merging #1353 into master will decrease coverage by 0.05%.
The diff coverage is 83.82%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1353      +/-   ##
==========================================
- Coverage   91.80%   91.74%   -0.06%     
==========================================
  Files         101      101              
  Lines       10636    10697      +61     
==========================================
+ Hits         9764     9814      +50     
- Misses        872      883      +11     
Impacted Files Coverage Δ
arviz/data/io_pystan.py 96.69% <83.82%> (-1.98%) ⬇️
arviz/plots/backends/matplotlib/pairplot.py 92.02% <0.00%> (-0.05%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6411cde...fb4c902. Read the comment docs.

@ahartikainen ahartikainen force-pushed the feature/pystan_metadata branch from ff0e8f7 to fb4c902 Compare August 16, 2020 18:30
Copy link
Member

@ColCarroll ColCarroll left a comment

Choose a reason for hiding this comment

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

took a light look, and code looks good.

flags=re.IGNORECASE,
).group(1)
)
except AttributeError:
Copy link
Member

Choose a reason for hiding this comment

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

is attribute error what happens when re.search returns None, which has no .group attribute?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, my lazy solution. "comment" field should still have the raw value.

@ahartikainen ahartikainen merged commit f124e64 into master Aug 16, 2020
@OriolAbril OriolAbril deleted the feature/pystan_metadata branch August 16, 2020 22:20
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants