search

comet-ml / comet-sdk-extensions

Open source contributions for integrations with comet_ml
MIT License
7 stars 1 forks source link

Added MPM smoke test #6

Closed jverre closed 2 months ago

dsblank commented 2 months ago

Thank you for this PR! I've refactored it a bit with the following changes:

  1. removed need for pandas and numpy (comet_mpm does require pandas, but didn't want to add it here as a dependency); easier ways to do their functions
  2. Good idea about having an "all" idea; refactored to allow specifying what to include, and what to exclude (the list is going to get very long)
  3. refactored to not have test data in memory at once

I'll close this PR, and push a refined one.

Thanks again!