Skip to content

Allow nnz=1 in the destriper#191

Merged
ziotom78 merged 14 commits intomasterfrom
nopol_destriper
Sep 7, 2022
Merged

Allow nnz=1 in the destriper#191
ziotom78 merged 14 commits intomasterfrom
nopol_destriper

Conversation

@ziotom78
Copy link
Copy Markdown
Member

@ziotom78 ziotom78 commented Sep 1, 2022

This small PR lets the destriper run if one specifies nnz=1 (i.e., only solve for intensity I).

@ziotom78 ziotom78 changed the title Allow nnz=1 in the destriper Allow nnz=1 in the destriper and fix dependencies Sep 2, 2022
@ziotom78
Copy link
Copy Markdown
Member Author

ziotom78 commented Sep 5, 2022

I had to avoid using poetry in CI builds because of this bug, so the YAML recipe exports a plain requirements.txt file and uses it to set up the environment.

@ziotom78 ziotom78 mentioned this pull request Sep 7, 2022
3 tasks
@ziotom78 ziotom78 changed the title Allow nnz=1 in the destriper and fix dependencies Allow nnz=1 in the destriper Sep 7, 2022
@ziotom78
Copy link
Copy Markdown
Member Author

ziotom78 commented Sep 7, 2022

Now the part modifying the dependencies has been incorporated in #192, so that the history for this PR will be cleaner when merged.

@ziotom78 ziotom78 merged commit b839050 into master Sep 7, 2022
@ziotom78 ziotom78 deleted the nopol_destriper branch September 7, 2022 06:11
@ziotom78 ziotom78 mentioned this pull request Aug 4, 2023
13 tasks
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.

1 participant