Skip to content

[Draft] Replace level_map and deprecate reference_point parameter#92

Draft
cfs-data wants to merge 24 commits intomainfrom
replace-map-level
Draft

[Draft] Replace level_map and deprecate reference_point parameter#92
cfs-data wants to merge 24 commits intomainfrom
replace-map-level

Conversation

@cfs-data
Copy link
Copy Markdown
Collaborator

@cfs-data cfs-data commented Jan 21, 2026

The intention for now is not to merge this PR, but to use it for testing. If no problems are encountered over time, this could be considered a replacement of the current implementation.

Changes:

  1. Our implementation of level_map() is replaced by a built-in function from surfalize
  2. It seems reference_point has no influence on end-to-end tests so this parameter has been removed in this branch.

@cfs-data cfs-data changed the title Replace level_map with built-in function [Draft] Replace level_map with built-in function Jan 21, 2026
@cfs-data cfs-data marked this pull request as draft January 21, 2026 13:43
@cfs-data cfs-data changed the title [Draft] Replace level_map with built-in function [Draft] Replace level_map with built-in function and depecrate reference_point parameter Jan 21, 2026
@cfs-data cfs-data changed the title [Draft] Replace level_map with built-in function and depecrate reference_point parameter [Draft] Replace level_map and depecrate reference_point parameter Jan 21, 2026
@cfs-data cfs-data changed the title [Draft] Replace level_map and depecrate reference_point parameter [Draft] Replace level_map and deprecrate reference_point parameter Jan 21, 2026
@cfs-data cfs-data changed the title [Draft] Replace level_map and deprecrate reference_point parameter [Draft] Replace level_map and deprecate reference_point parameter Jan 21, 2026
@snregales
Copy link
Copy Markdown
Collaborator

@Raytesnel will PR impact us?

@Raytesnel
Copy link
Copy Markdown
Collaborator

@Raytesnel will PR impact us?

ja

@cfs-data cfs-data requested a review from SimoneAriens February 9, 2026 15:29
@cfs-data cfs-data changed the title [Draft] Replace level_map and deprecate reference_point parameter Replace level_map and deprecate reference_point parameter Feb 16, 2026
@snregales
Copy link
Copy Markdown
Collaborator

Is this PR still applicable?

@cfs-data cfs-data changed the title Replace level_map and deprecate reference_point parameter [Draft] Replace level_map and deprecate reference_point parameter Mar 3, 2026
@cfs-data cfs-data changed the title [Draft] Replace level_map and deprecate reference_point parameter [Draft] Replace level_map and deprecate reference_point and Mark.center parameter Mar 16, 2026
@cfs-data cfs-data changed the title [Draft] Replace level_map and deprecate reference_point and Mark.center parameter [Draft] Replace level_map and deprecate reference_point parameter Mar 16, 2026
@cfs-data cfs-data changed the title [Draft] Replace level_map and deprecate reference_point parameter Replace level_map and deprecate reference_point parameter Mar 17, 2026
@cfs-data cfs-data changed the title Replace level_map and deprecate reference_point parameter [Draft] Replace level_map and deprecate reference_point parameter Mar 19, 2026
@github-actions
Copy link
Copy Markdown

Diff Coverage

Diff: origin/main..HEAD, staged and unstaged changes

  • packages/scratch-core/src/conversion/leveling/core.py (100%)
  • packages/scratch-core/src/conversion/leveling/data_types.py (100%)
  • packages/scratch-core/src/conversion/preprocess_impression/preprocess_impression.py (100%)
  • packages/scratch-core/src/mutations/filter.py (100%)

Summary

  • Total: 36 lines
  • Missing: 0 lines
  • Coverage: 100%

@github-actions
Copy link
Copy Markdown

Code Coverage

Package Line Rate Branch Rate Health
. 96% 92%
comparators 100% 100%
computations 95% 80%
container_models 99% 100%
conversion 96% 89%
conversion.export 100% 100%
conversion.filter 97% 89%
conversion.leveling 100% 100%
conversion.plots 99% 88%
conversion.preprocess_impression 99% 91%
conversion.preprocess_striation 90% 62%
conversion.profile_correlator 96% 82%
conversion.surface_comparison 99% 89%
conversion.surface_comparison.cell_registration 100% 90%
extractors 99% 75%
mutations 100% 100%
parsers 97% 50%
parsers.patches 89% 60%
preprocessors 99% 91%
processors 100% 83%
renders 98% 67%
utils 71% 100%
Summary 98% (3168 / 3242) 87% (341 / 394)

Minimum allowed line rate is 50%

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.

3 participants