Jack Gerrits

For example if using bagging, then internally VW maintains several models. Each occupying a different hash space. You need to know the number of bags in order to decipher what...

+1 to this, would really like to be able to use a custom header

Can you explain the date.h changes? Is this a new version or did you make the changes?

When there is no model for CB, it does not use the safe_vw factory. It instead uses an explicit explore only mode. See here: https://github.com/VowpalWabbit/reinforcement_learning/blob/bf4d7bb54b79703831b46af74edbd7047d4fa5af/rlclientlib/live_model_impl.cc#L64

I would say the best option is to upgrade to VS2017 since the C# project files are only compatible with VS2017+

The solution has been removed so this is no longer an issue.

We actually currently have this and it is how I generated the relevant exceptions in Python and error constants in C++. See this file: https://github.com/VowpalWabbit/reinforcement_learning/blob/master/include/errors_data.h

@rajan-chari thoughts?

I wonder if it is possible to not bundle libssl. We should most likely be depending on the system copy right?

Looks like a typo in the erase call https://github.com/VowpalWabbit/vowpal_wabbit/blob/5129996594be56359495c6d9f78703495030d6c0/vowpalwabbit/model_merger/src/main.cc#L50 Can you replace this line with: ```c++ newline_stripped_message.end()); ``` Let me know if that fixes the issue for you