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.

Sign up for GitHub

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

Jump to bottom

Versions above 2.5 broken SI - #15

Closed
sawjam opened this issue Jan 21, 2021 · 2 comments · Fixed by #32
Closed

Versions above 2.5 broken SI - #15

sawjam opened this issue Jan 21, 2021 · 2 comments · Fixed by #32

Comments

@sawjam
Copy link

sawjam commented Jan 21, 2021

versions above 2.5 cause the following errors when trying to plot charts in SI in JupyterLab downgrading to 2.5 resolve issue:
C:\ProgramData\Anaconda3\lib\site-packages\psychrochart\chartdata.py:356: RuntimeWarning: divide by zero encountered in GetRelHumFromTWetBulb
GetSatVapPres(db_t) * GetRelHumFromTWetBulb(db_t, wb_t, p_atm),

Great package really appreciate the time - note downgrading package solves the issue for me which is fine.
@jmagnuso13
Copy link

Thanks for identifying a short term solution. @azogue will this be resolved in the future

@rossboccia
Copy link

Any Update?

azogue added a commit that referenced this issue Jun 7, 2023
@azogue
🐛 Fix #15 (avoid calls to GetRelHumFromTWetBulb if wb_t > db_t)
58bf5fa
@azogue azogue mentioned this issue Jun 7, 2023
Merged
@azogue azogue closed this as completed in #32 Jun 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

✨ Chart config auto-refresh + bugfixes azogue/psychrochart
3 participants
@sawjam @jmagnuso13 @rossboccia