AliRana30 opened a new pull request, #806:
URL: https://github.com/apache/wayang/pull/806

   ### Summary of Changes
   - **Configurable ID Encoding**: Replaced the hardcoded `final boolean 
encodeIds = false;` in `TreeEncoder` with `wayang.ml.encoding.encode-ids` 
(defaulting to `false`, with shorthand `wayang.ml.encode-ids` supported).
   - **TreeEncoder Enhancements**: Added `TreeEncoder(mappings, configuration)` 
constructor, getter/setter for configuration, and overloaded `encode` methods 
that resolve `encodeIds` from `OptimizationContext` or local configuration.
   - **Default Properties**: Loaded `wayang-ml-defaults.properties` in 
`MachineLearning.plugin().setProperties(Configuration)` and declared 
`wayang.ml.encoding.encode-ids = false`.
   - **Unit Tests**: Added test cases in `TreeEncoderTest` verifying default 
behavior, enabled ID encoding via configuration, and plugin properties loading.
   
   Closes #776.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to