Configurable target property name

g3rv4 / RandomForest

A tiny library to run in C# random forests created in R

MIT License

12 stars 2 forks source link

Configurable target property name #4

Closed donpratt closed 5 years ago

donpratt commented 5 years ago

Currently, this looks for a data field named "_target". Are you open to making this configurable? I have a pmml file that a coworker created, but he used a more descriptive name for his output data field. I already made the change locally if you're open to a pull request (not that it's at all difficult).

g3rv4 commented 5 years ago

sure! go for it! just keep the default to the current behavior ;)

donpratt commented 5 years ago

Thanks. I just opened a pull request for this. Pretty basic, but let me know if you have any questions or suggestions.

g3rv4 commented 5 years ago

Merged! I’ll let you know once it’s on nuget

donpratt commented 5 years ago

Thanks.

g3rv4 commented 5 years ago

Done! version 0.2.3 has been uploaded to nuget... it may take a couple hours for it to be indexed. Thanks a ton!