search

microsoft / causica

MIT License
430 stars 56 forks source link

DECI Examples #21

Closed moshe-salhov closed 1 year ago

moshe-salhov commented 2 years ago

Hello, Really amazing work!!. Small issue - is it possible to fix the notebooks under the DECI examples so it will be possible to run them? Currently there are several issues that prevent such run:

  1. metrics_logger not defined under the run_context
  2. model run_train doesnt accept run_context=run_context
  3. mode_f_sem is not supported under model_config when running DECI()
  4. RunContext has no attribute get (I suspect that since "evaluation_pipline.aml_run_context" is not available there is a mismatch in RunContext functionality )

Many thanks

mry-tvk commented 2 years ago

Adding another point to this: 5- since the name of the repo is causica maybe the imports should be changed as below: "from open_source.causica.models.deci.deci import DECI" -->"from causica.causica.models.deci.deci import DECI"

confoundry commented 1 year ago

Closing as the file no longer exists