Use same requirements in all SNEWS tools: SNEWPY, snewps-pt, snews-cs, snewpdag

SNEWS2 / snewpy

A Python package for working with supernova neutrinos

https://snewpy.readthedocs.io

BSD 3-Clause "New" or "Revised" License

26 stars 19 forks source link

Use same requirements in all SNEWS tools: SNEWPY, snewps-pt, snews-cs, snewpdag #199

Closed mcolomermolla closed 1 year ago

mcolomermolla commented 2 years ago

As an example, now: snews-pt 1.0.2.dev4 requires pandas~=1.4.2, while SNEWPY requires pandas 1.3.4, which is incompatible.

JostMigenda commented 2 years ago

@Sheshuk, you added the pandas==1.3.4 requirement in https://github.com/SNEWS2/snewpy/commit/d126da84a4fb1d79ee1ff6da0f124feb32c5ece2#diff-4d7c51b1efe9043e44439a949dfd92e5827321b34082903477fd04876edb7552—was there any reason for this strict requirement or can we relax that?

Sheshuk commented 1 year ago

Sorry, missed this question. There is no reason for strict requirement, we can replace = to >=1.3.4 or just remove the version. 1.3.4 was the version I used at that moment, in principle there are no restrictions.