search

uhlmanngroup / bioimage_embed

3 stars 3 forks source link

add `wandb` to master for monitoring purposes #33

Closed afoix closed 10 months ago

afoix commented 10 months ago

Add wandb to master for monitoring purposes. It does not only affect the shape_embeded.py file so, I need @ctr26 to review it

ctr26 commented 10 months ago

Ideally you'd get wandb out of the lightning modules because the tests don't instantiate it

ctr26 commented 10 months ago

Maybe rebase on logger_improvements and then merge any conflicts. Ideally I'd like the core code not to be dependent on wandb and for users to chose their favourite loggers.

afoix commented 10 months ago

I am closing this pull request @ctr26 because is overwritten by yours.

ctr26 commented 10 months ago

Does it work for you? @afoix