Problems with Constants in Traj

automl / ParameterImportance

Parameter Importance Analysis Tool

http://www.ml4aad.org/

BSD 3-Clause "New" or "Revised" License

76 stars 19 forks source link

Closed mlindauer closed 4 years ago

mlindauer commented 5 years ago

https://github.com/automl/ParameterImportance/blob/development/pimp/importance/importance.py#L267 does not handle Constants (and Ordinal). I would recommend a simple else case to pass all other parameter types without any casting.

shukon commented 4 years ago

Should be fixed in next release with #108