Skip to content
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

FEEL instead of SFEEL #20

Open
rmerkx opened this issue Aug 18, 2023 · 3 comments
Open

FEEL instead of SFEEL #20

rmerkx opened this issue Aug 18, 2023 · 3 comments

Comments

@rmerkx
Copy link

rmerkx commented Aug 18, 2023

Hi Radec,

I am very happy with this open source implementation and am heavily using it as a generic back end service able to execute DM's from a central repository. Only thing I am missing is being able to use the complete FEEL expression language constructs. Are you considering adding this in the future? In my optinion that would make this implementation a valuable product. How much effort would you need to add this? Would you be interested to work on this (with me)?

Kind regards,
Radoslaw

@adamecr
Copy link
Owner

adamecr commented Mar 9, 2024

Hi @rmerkx , sorry for the late response, I'm quite busy in my day-job and didn't have much time for the hobby coding for a while.
I decided to use S-FEEL as it was quite easy to implement it's syntax within the Dynamic Expresso used as main scripting platform for DMN.Engine.

Full FEEL implementation will mean to implement the complete language grammar and somehow extend the Dynamic Expresso to use it. I'm not saying it's not possible, just didn't want to spend that much effort.

@michaelglasson
Copy link

I wonder if it might be possible simply to allow use of Dynamic Expresso to evaluate C# expressions in DMNEngine. If we could do that, we might be able to avoid the large labour of implementing FEEL fully. I am exploring this with someone who is better at .Net and C# than I am. If we come up with something, we will submit an update proposal.

@adamecr
Copy link
Owner

adamecr commented Mar 13, 2024

Hi @michaelglasson , you can use dynamic Expresso. I’ll give you more details in the thread related to arrays #23 within a few days whenever I’ll have a bit of time to prepare some samples

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants