ml-energy / zeus

Deep Learning Energy Measurement and Optimization
https://ml.energy/zeus
Apache License 2.0
179 stars 24 forks source link

doc: fix typo #69

Closed Sunt-ing closed 1 month ago

Sunt-ing commented 1 month ago

Maybe we can also change the example in https://ml.energy/zeus/reference/monitor/energy/#zeus.monitor.energy.ZeusMonitor to 1) avoid using a global ZeusMonitor variable, 2) provide a dummy training function (e.g., implemented by sleep), so that users can run the demo easily?

jaywonchung commented 1 month ago

Yeah, that makes sense. I'd be happy to review the PR if you're willing to implement the change.

Sunt-ing commented 1 month ago

https://github.com/ml-energy/zeus/pull/70