aimhubio / aim

Aim 💫 — An easy-to-use & supercharged open-source experiment tracker.
https://aimstack.io
Apache License 2.0
5.18k stars 317 forks source link

How to track system logs manually & legitimately, when cli convert implemented? #2402

Open hjoonjang opened 1 year ago

hjoonjang commented 1 year ago

❓Question

Following #2348, I'd like to implement several features more on cli convert wandb. But for now, I have some pain points to implement those ideas, which, I think, the maintainer might have the keys to solve. The followings are describing each of my implementation goal and its related questions.

(Assuming I'm on the way implementing cli convert wandb specifically.)

alberttorosyan commented 1 year ago

Hi @hjoonjang. Seems the reason of other system metrics not being shown is the fact that we use the pre-defined list of names in UI. Here's for example the part where the metric names are mapped to more user-friendly texts: https://github.com/aimhubio/aim/blob/cb5472f9cc43317a8e4b0ef013f03cec127a5baf/aim/web/ui/src/config/systemMetrics/systemMetrics.ts#L4 The quick fix would be to map metric names defined in wandb to the ones expected by Aim UI. For more robust solution, @roubkar, can you please take a look?

hjoonjang commented 1 year ago

Hi @hjoonjang. Seems the reason of other system metrics not being shown is the fact that we use the pre-defined list of names in UI. Here's for example the part where the metric names are mapped to more user-friendly texts:

https://github.com/aimhubio/aim/blob/cb5472f9cc43317a8e4b0ef013f03cec127a5baf/aim/web/ui/src/config/systemMetrics/systemMetrics.ts#L4

The quick fix would be to map metric names defined in wandb to the ones expected by Aim UI. For more robust solution, @roubkar, can you please take a look?

@alberttorosyan, thanks for your help! I've applied your 'quick fix' suggestion to my implementation (#2403), and of course, there were remaining unpaired names which I think are not perfectly robust but inevitable in lossy conversion.

If there is another way to make this better, please let me know again. Thanks 👍🏻