Skip to content

fix: typescript-build-issues related to DepthPicking and Linocut #175

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

Merged
merged 4 commits into from
Jan 28, 2025

Conversation

alvarosabu
Copy link
Member

No description provided.

@alvarosabu alvarosabu added bug Something isn't working types labels Jan 28, 2025
}
const props = defineProps<DepthPickingPassPmndrsProps>()
const composer = inject(effectComposerInjectionKey)
const pass = new DepthPickingPass(props)
const pass = shallowRef(new DepthPickingPass(props))
Copy link
Collaborator

Choose a reason for hiding this comment

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

We discussed this a while ago with @Tinoooo about the responsiveness of the pass. #164 (comment)

Copy link
Collaborator

@damienmontastier damienmontastier left a comment

Choose a reason for hiding this comment

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

I don't see anything problematic apart from shallowRef that we can discuss...

Copy link

pkg-pr-new bot commented Jan 28, 2025

Open in Stackblitz

npm i https://pkg.pr.new/@tresjs/post-processing@175

commit: 1e3284b

@alvarosabu alvarosabu merged commit a2d90b0 into main Jan 28, 2025
4 checks passed
damienmontastier added a commit that referenced this pull request Jan 30, 2025
…cut (#175)"

This reverts commit a2d90b0.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working types
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants