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

BB template & vertex outliers #333

Merged
merged 17 commits into from
Dec 17, 2024
Merged

BB template & vertex outliers #333

merged 17 commits into from
Dec 17, 2024

Conversation

jordandekraker
Copy link
Collaborator

No description provided.

@jordandekraker
Copy link
Collaborator Author

Merging dev into BBtemplate I began to remove the redundant get_final_surfaces(), but this will likely take some tweaking still

@jordandekraker jordandekraker merged commit 48ebd46 into dev-v2.0.0 Dec 17, 2024
4 checks passed
@jordandekraker jordandekraker deleted the BBtemplate branch December 17, 2024 19:02

# get unfolded grid (from 0 to 1, not world coords), using meshgrid:
# note: indexing='ij' to swap the ordering of x and y
epsilon = snakemake.params.epsilon
Copy link
Member

Choose a reason for hiding this comment

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

I know this PR is merged already, but easier to comment here than anew -- these changes break the rule because epsilon is not a parameter in the rule. Also, unfold_gx/unfold_gy/unfold_gz get create here, but don't actually get used .. so it seems we can just revert the create_warps changes to fix the problem, unless you were trying to do something else here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

oh yeah, good point. Looks like we were never actually using the epsilon that we'd previously defined? I'll make a small PR to patch this & 1 other small bug i found

Also yeah points and unfold_gx somehow became redundant. We can cut the latter.

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