search

mmcdermott / MEDS-DEV

The MEDS Decentralized Extensible Validation (MEDS-DEV) Benchmark: Establishing Reproducibility and Comparability in ML for Health
MIT License
11 stars 7 forks source link

Aces and MEDS evaluation dependency issue #32

Closed Simonlee711 closed 2 weeks ago

Simonlee711 commented 3 weeks ago

Have a dependency issue between MEDS eval and Aces: es-aces 0.5.0 requires polars==1.5.*, but you have polars 1.1.0 which is incompatible. and meds-evaluation 0.1.dev57+g5793cb7 requires polars~=1.1.0, but you have polars 1.5.0 which is incompatible..

Simonlee711 commented 3 weeks ago

Wondering if there are work arounds that can be documented in the README.md or we should also just try to fix?

mmcdermott commented 3 weeks ago

I think there was a bug with Polaris 1.3 or 1 4 that mandated the 1.1 dependency, so we can likely update to 1.5+

On Tue, Nov 5, 2024, 12:05 AM Simon Lee @.***> wrote:

Wondering if there are work arounds that can be documented in the README.md or we should also just try to fix?

— Reply to this email directly, view it on GitHub https://github.com/mmcdermott/MEDS-DEV/issues/32#issuecomment-2456243336, or unsubscribe https://github.com/notifications/unsubscribe-auth/AADS5XYXZLF7MILLTPSG4D3Z7A7YRAVCNFSM6AAAAABRFWJLQWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJWGI2DGMZTGY . You are receiving this because you were assigned.Message ID: @.***>

mmcdermott commented 2 weeks ago

@Simonlee711 -- have you seen any issues using polars 1.5 and/or any other updates for this? Or does this still need investigation from the MEDS side?

Simonlee711 commented 2 weeks ago

@mmcdermott we can close the issue. We modified meds-evaluation to now be compatible with polars==1.5. Everything should be resolved now