-
Notifications
You must be signed in to change notification settings - Fork 163
Feat dotnet p invoke #424
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Feat dotnet p invoke #424
Conversation
- Created GoRules.Zen.Tests project for unit testing the Zen Rules Engine bindings. - Implemented TemplateTests and UnaryExpressionTests to validate expression rendering and evaluation. - Updated GoRules.Zen project to target .NET 10.0 and include necessary references. - Added solution file for easier project management in Visual Studio. - Enhanced README.md with detailed usage instructions and examples. - Introduced USAGE.md for comprehensive integration guidance in .NET applications. - Added build.sh script to automate the building of the Rust library and .NET bindings. - Updated ZenEngine.cs to clarify default values for evaluation options.
- Introduced a dynamic pricing decision model with multiple adjustment factors based on market demand, time of day, competitor pricing, and customer segments. - Implemented a real-time quotation decision model that calculates insurance premiums based on general liability, commercial property, and professional indemnity coverage. - Each model includes input nodes, decision tables, and output nodes to facilitate complex pricing calculations.
bindings/c/build-all-platforms.ps1
Outdated
| @@ -0,0 +1,38 @@ | |||
| # build-all-platforms.ps1 | |||
| # Windows PowerShell 脚本 - 使用 cross 或原生工具链构建 | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, can you remove non english language and translate it to english
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
deleted, the file is useless, just had a try for cross compiling, it does not work as smooth as GoLang, so moved to the github workflows build-native.yml
bindings/c/build-all-platforms.sh
Outdated
| @@ -0,0 +1,36 @@ | |||
| #!/bin/bash | |||
| # build-all-platforms.sh | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above, english only
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above, deleted.
| CopyToOutputDirectory="PreserveNewest" | ||
| Condition="Exists('runtimes/win-x64/native/zen_ffi.dll')" /> | ||
|
|
||
| <!-- Linux x64 --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we able to add linux arm?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
windows arm64 and linux arm64 added. but I do not have real machine to test them, they should work.
| target: x86_64-pc-windows-msvc | ||
| lib_name: zen_ffi.dll | ||
| runtime: win-x64 | ||
| - os: ubuntu-latest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you be able to add unknown-linux arm?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, added. wish you guys a happy holiday!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, we have couple more days of holidays and we will soon be back - we will take a look as soon as we are back
|
Hi, thank you for contribution! This is really valuable as our goal is to add dotNet as well! Please note, most of the team is off due to holidays and we will take a look at this PR when we get back. |
No description provided.