-
Notifications
You must be signed in to change notification settings - Fork 0
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
Make to CTS #22
Make to CTS #22
Conversation
…s.py file bc of conflict
import logging | ||
from pathlib import Path | ||
|
||
import pandas as pd |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this is added to the poetry file as a requirement yet...can you also make sure that you choose a version of pandas that match what is included with SEED?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hm, I added it to the toml and its in the lock, am I missing something? seed doesn't seem to specific a pandas version, it requires https://github.com/SEED-platform/buildingid, which itself requires pandas, but it neither specifics or locks a pandas versions.
I can tell you I required this new version of BAE in seed locally and it worked. 🤷🏻
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚚
to document: we will lock the version of pandas to v2.2.2 to match SEED. Once that's done this will be ready to merge and release! thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🏆
The entry point is
building_sync_to_cts
. ItAgency Designated Covered Facility ID
to make facilities