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

block one test that regression problem in the backend PR1253 #18

Merged
merged 2 commits into from
May 4, 2023

Conversation

acdo2002
Copy link
Collaborator

@acdo2002 acdo2002 commented May 2, 2023

#17 Because the carta-backend PR#1253 has a regression issue#1259. Thus I temporary block the comparison test between PV generated images from fits and hdf5.

After the issue#1259 has been solved, the blocked test lines can be re-open again.

Please merge this PR after carta-backend PR#1253 merging into the dev.

@acdo2002 acdo2002 requested a review from ajm-ska May 2, 2023 09:14
@acdo2002
Copy link
Collaborator Author

acdo2002 commented May 3, 2023

The pv preview frontend PR#2100 has updated the BackendService.ts.

This PR also update the corresponding MessageController.ts, adding the PV preview related codes.

Copy link
Contributor

@ajm-ska ajm-ska left a comment

Choose a reason for hiding this comment

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

OK. PR#1253 is now merged into dev.

@ajm-ska ajm-ska merged commit 10f2143 into dev May 4, 2023
@acdo2002 acdo2002 deleted the mylin/block_pv_hdf5_fits_compart_part_test branch May 30, 2023 15:31
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.

2 participants