Devis Lucato
Devis Lucato
The current planner is designed for GPT4 (and probably we should document that). Are you using GPT3 or 3.5 by any chance? Those models are not powerful enough for this...
the gain in performance has a pretty big impact on code readability. Is this approach common when using ILogger?
To be revisited later. Need to look at options with extension methods, code generation, measure overhead.
I think we should change FunctionsView to protect the internal data, allowing only to add and list, not remove, and potentially handle conflicts. Plus needs testing and clear specs about...
@awharrison-28 FYI
Todo: for ISKFunction and Plan it would be nice having the cancellation token as an optional parameter, without ever forcing the user to pass one. There are a lot of...
@dluc make class internal. For more recent .NET version we should use PriorityQueue
Agreed, once we have a chance to revisit the unit tests approach, we'll start using this, removing the bespoke version
thanks @chuanzhubin -- sorry about the disruption, we're releasing a new version and updating the docs today, pretty soon everything will be aligned and work as usual
@awharrison-28 we updated the README too soon, e.g. before releasing the pip package. When introducing breaking changes like this one, the documentation should be synchronized with the release. In other...