search

basf / MolPipeline

MIT License
89 stars 4 forks source link

utils: add own _print_elapsed_time #14

Closed JochenSiegWork closed 3 months ago

JochenSiegWork commented 3 months ago 
- sklearn changed location of private _print_elapsed_time
  function in sklearn 1.5, which is breaking our code.
- To stay consistent but robust against such changes we
  adapted the print functions and implement them ourselves.
  This stops future code breaks due to location changes
  and we stick to a similar logging style as sklearn.
- If sklearn is doing major changes to their logging style
  we might should adapt in the future. However, that's
  rather unlikely.