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.

Sign up for GitHub

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

Jump to bottom

BioCelerateCrossStudyApp: Certain Selection Combinations Cause Crashes #33

Open
sbutler5 opened this issue Nov 15, 2023 · 3 comments
Open

BioCelerateCrossStudyApp: Certain Selection Combinations Cause Crashes #33

sbutler5 opened this issue Nov 15, 2023 · 3 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@sbutler5
Copy link
Collaborator

Nikolai has found that: When selecting both sexes (and all repro organs) it is possible to generate the visual, but if you then try to include the non-selected kidney or liver MI or OM parameters (eg Ureter, Bladder, Gall Bladder) the app crashes

Testing for combinations that cause crashes and better selection handling is needed.
@sbutler5 sbutler5 added bug Something isn't working help wanted Extra attention is needed labels Nov 15, 2023
@sbutler5 sbutler5 changed the title BioCelerateApp: Certain Selection Combinations Cause Issues BioCelerateCrossStudyApp: Certain Selection Combinations Cause Issues Nov 15, 2023
@sbutler5 sbutler5 changed the title BioCelerateCrossStudyApp: Certain Selection Combinations Cause Issues BioCelerateCrossStudyApp: Certain Selection Combinations Cause Crashes Nov 15, 2023
@Yousuf28
Copy link
Collaborator

Yousuf28 commented Nov 15, 2023 •
edited
Loading

If you @sbutler5 can give little details how I can run the app, I can look into the crash issue.

@Yousuf28
Copy link
Collaborator

The issue is probably in code_link
line 895. So Incid Table is empty and assigning name to a column that does not exist generate error.
Now question is how you want to handle the empty table?

@sbutler5
Copy link
Collaborator Author

sbutler5 commented Nov 18, 2023 •
edited
Loading

@Yousuf28 Thank you for looking this over. If you have an idea on how to handle empty tables, I welcome it. Any ideas?

@sbutler5 sbutler5 reopened this Nov 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Yousuf28 @sbutler5