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

EPW blank file creation error incorrect dates. #513

Closed
sawjam opened this issue Jun 14, 2021 · 4 comments · Fixed by #515
Closed

EPW blank file creation error incorrect dates. #513

sawjam opened this issue Jun 14, 2021 · 4 comments · Fixed by #515
Assignees
Labels
bug Something isn't working

Comments

@sawjam
Copy link

sawjam commented Jun 14, 2021

I believe this a bug and not the same as #490 enhancement by chriswmackey.
I've been trying to create an epw file from scratch, but climate consultant 6 has been telling me the file is invalid.
If you do just:

from ladybug.epw import EPW
new_epw = EPW.from_missing_values()

You get an epw file with incorrectly ordered dates, and when you insert data it's incorrect.
See screengrab of a file, this occurs on everyday end.

image

@mostaphaRoudsari mostaphaRoudsari added the bug Something isn't working label Jun 14, 2021
@sawjam
Copy link
Author

sawjam commented Jun 14, 2021

For further information, I've tested the file generated in Grasshopper with a number sequence in ladybug and the "rendering" the values from the generated epw, do retain the correct date order in analysis within ladybug.

@chriswmackey
Copy link
Member

Sorry it took me a while to respond, @sawjam . This looks like it is, indeed, a bug and probably one that's pretty easy to fix. I'll try to push a fix now.

@chriswmackey
Copy link
Member

Thanks for reporting, @sawjam . I just pushed a fix and it should be available throughout the core libraries in an hour or so.

@sawjam
Copy link
Author

sawjam commented Jun 22, 2021

Thank you appreciate the work, and loving pollination :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants