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

Fix for missing filters #1898

Merged
merged 1 commit into from
Dec 12, 2024
Merged

Fix for missing filters #1898

merged 1 commit into from
Dec 12, 2024

Conversation

SzymanskiBartlomiej
Copy link
Contributor

@SzymanskiBartlomiej SzymanskiBartlomiej commented Dec 12, 2024

This pull request includes changes to the src/services/ShSimulatorService.tsx file to introduce a new function for updating estimator references and integrate it into the existing workflow. The most important changes include the addition of the recreateRefToScoringManagerOutputs function and its usage within the recreateRefsInResults function.

New function addition:

  • src/services/ShSimulatorService.tsx: Added the recreateRefToScoringManagerOutputs function to update the scoringOutputJsonRef property of each estimator with the corresponding output from the ScoringManagerJSON.

Integration into existing workflow:

  • src/services/ShSimulatorService.tsx: Modified the recreateRefsInResults function to use the newly added recreateRefToScoringManagerOutputs function before calling recreateRefToFilters, ensuring that the estimators have updated scoring manager outputs.

@SzymanskiBartlomiej SzymanskiBartlomiej self-assigned this Dec 12, 2024
@SzymanskiBartlomiej SzymanskiBartlomiej linked an issue Dec 12, 2024 that may be closed by this pull request
@grzanka grzanka added this pull request to the merge queue Dec 12, 2024
Merged via the queue into master with commit 8bcf666 Dec 12, 2024
8 checks passed
@grzanka grzanka deleted the 1897-filters-not-being-displayed branch December 12, 2024 19:05
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.

Filters not being displayed
2 participants