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

Updating tools/flexynesis from version 0.2.10 to 0.2.12 #1563

Merged
merged 3 commits into from
Jan 8, 2025

Conversation

gxydevbot
Copy link
Contributor

Hello! This is an automated update of the following tool: tools/flexynesis. I created this PR because I think the tool's main dependency is out of date, i.e. there is a newer version available through conda.

I have updated tools/flexynesis from version 0.2.10 to 0.2.12.

Project home page: https://github.com/BIMSBbioinfo/flexynesis/tree/main/releases

For any comments, queries or criticism about the bot, not related to the tool being updated in this PR, please create an issue here.

If you want to skip this change, close this PR without deleting the branch. It will be reopened if another change is detected.

Any commit from another author than 'planemo-autoupdate' will prevent more auto-updates.

To ignore manual changes and allow autoupdates, delete the branch.

@bgruening
Copy link
Owner

@nilchia is that ok?

@nilchia
Copy link
Collaborator

nilchia commented Jan 7, 2025

@bgruening No I have to add XGBoost and fix some tests. Working on it.
Thanks!

@nilchia
Copy link
Collaborator

nilchia commented Jan 8, 2025

@bgruening Can you please give me access to this repo so I can push my changes?
Thanks!

@bgruening
Copy link
Owner

Done. But please note, that you can always push to your repo and make a PR against the bot repo or a new PR.

@nilchia
Copy link
Collaborator

nilchia commented Jan 8, 2025

Thanks!
Is this PR correct then? planemo-autoupdate#4

@bgruening
Copy link
Owner

Yes it is!

<has_text_matching expression="DirectPred,Erlotinib,numerical,mse,"/>
<has_text_matching expression="DirectPred,Erlotinib,numerical,r2,"/>
<has_text_matching expression="DirectPred,Erlotinib,numerical,pearson_corr,"/>
<has_text_matching expression="GNN,Erlotinib,numerical,mse,"/>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did it change from DirectPred to GNN?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one is correct actually. In the previous test <param name="model_class" value="GNN"/> was not enclosed within the <conditional name="model_class"> and </conditional>, and instead of GNN, the default model (DirectPred) was called.

The other tests are correct.

@bgruening bgruening merged commit 4a6a7a2 into bgruening:master Jan 8, 2025
11 checks passed
@bgruening bgruening deleted the tools/flexynesis branch January 8, 2025 17:04
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