Skip to content
This repository has been archived by the owner on Jul 12, 2022. It is now read-only.

Update co2mpas requirements #4

Open
vinci1it2000 opened this issue Oct 12, 2019 · 8 comments
Open

Update co2mpas requirements #4

vinci1it2000 opened this issue Oct 12, 2019 · 8 comments
Assignees

Comments

@vinci1it2000
Copy link
Member

continuation of https://github.com/JRCSTU/co2mpas/issues/526

@stefanocorsi
Copy link
Contributor

@ankostis and @vinci1it2000 I've been assigned this but it's not clear to me what I have to do... is it pinning xgboost<=0.82 in co2mpas forge recipe?

@vinci1it2000
Copy link
Member Author

vinci1it2000 commented Oct 12, 2019

Yes. There are also some other limitations like wltp and pandalone.

@ankostis
Copy link
Member

@stefanocorsi did you apply this version-limit in co2mpas recipe, needed by Vinz?
https://github.com/vinci1it2000/co2mpas/blob/fb708c08e2564b2f861c208d296f54ed099b3d5f/setup.py#L175

@vinci1it2000
Copy link
Member Author

@stefanocorsi don't forget wltp and pandalone.

@stefanocorsi
Copy link
Contributor

Pandalone is a custom version that I'm including in the distribution directly from @ankostis's code, what about wltp? What limitation should be included?

@vinci1it2000
Copy link
Member Author

wltp<1. But this is incompatible with pandalone >= 0.3.

@stefanocorsi
Copy link
Contributor

Life is difficult.

@vinci1it2000
Copy link
Member Author

@stefanocorsi @ankostis you should update also syncing>=1.0.4.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants