Skip to content

Removing RMLFNMLTC0001-CSV and Updating Cases 102, 103, and 104 - #74

Merged
dachafra merged 2 commits into
kg-construct:mainfrom
eiglesias34:main
Mar 13, 2026
Merged

dachafra merged 2 commits into
kg-construct:mainfrom
eiglesias34:main

Conversation

@eiglesias34

Copy link
Copy Markdown
Contributor

No description provided.

@dachafra dachafra changed the title Removing RMLFNMLTC0001-CSV Removing RMLFNMLTC0001-CSV and Updating Cases 102, 103, and 104 Mar 13, 2026
@dachafra
dachafra merged commit 634888c into kg-construct:main Mar 13, 2026
1 check passed
@bjdmeest

Copy link
Copy Markdown
Member

Why should RNMLTC0001 be removed? This removes the use case of using functions without parameters. I do not agree with that removal

@eiglesias34

Copy link
Copy Markdown
Contributor Author

Hello @bjdmeest,

The main reason this test case was removed was that the function's result was always random, making it impossible to consider it "passed" or "failed". For the last two editions of the challenge, that was always the main complaint about RML-FMNL. Therefore, the test case was removed.

With that being said, you do bring up a good point about having a function with no parameters. I propose adding RNMLTC0001 again, but this time with a different function that produces a much more consistent result. If you have any ideas, please share them with me.

Sincerely,
Enrique

@bjdmeest

Copy link
Copy Markdown
Member

Easiest path forward would be to create a dummy function alwaysReturnsABC() that... always returns the string "ABC" :D

@eiglesias34

Copy link
Copy Markdown
Contributor Author

Hello @bjdmeest,

I made a new PR with the new RMLFNMLTC0001-CSV.

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

Successfully merging this pull request may close these issues.

3 participants