Skip to content

Merge two decompress! for star coloring#129

Merged
gdalle merged 1 commit intomainfrom
am/star_decompression
Sep 29, 2024
Merged

Merge two decompress! for star coloring#129
gdalle merged 1 commit intomainfrom
am/star_decompression

Conversation

@amontoison
Copy link
Copy Markdown
Collaborator

@amontoison amontoison commented Sep 28, 2024

We also want the optimized version of decompress! for star coloring when the user specifies uplo=:F and not only when uplo is not specified.

@amontoison amontoison requested a review from gdalle September 28, 2024 02:27
@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (96830d4) to head (3cefc33).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #129   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           11        11           
  Lines          855       851    -4     
=========================================
- Hits           855       851    -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@amontoison amontoison added the benchmark Run benchmarks on PR label Sep 28, 2024
@JuliaDiff JuliaDiff deleted a comment from github-actions Bot Sep 28, 2024
@amontoison amontoison removed the benchmark Run benchmarks on PR label Sep 28, 2024
Comment thread src/decompression.jl Outdated
Comment thread src/decompression.jl Outdated
Comment thread src/decompression.jl
@amontoison amontoison requested a review from gdalle September 28, 2024 19:45
@amontoison amontoison force-pushed the am/star_decompression branch from 7af83fb to 3cefc33 Compare September 28, 2024 22:52
@amontoison
Copy link
Copy Markdown
Collaborator Author

@gdalle It should be good now :)

@gdalle gdalle merged commit a03a455 into main Sep 29, 2024
@gdalle gdalle deleted the am/star_decompression branch September 29, 2024 06: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.

2 participants