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

Godot patches #93

Merged
merged 1 commit into from
Apr 21, 2022
Merged

Godot patches #93

merged 1 commit into from
Apr 21, 2022

Conversation

fire
Copy link
Contributor

@fire fire commented Apr 19, 2022

Here are the changes I did. Pushing for cicd testing.

test/data/Csaszar.ply Outdated Show resolved Hide resolved
test/polygon_test.cpp Outdated Show resolved Hide resolved
tools/perf_test.cpp Outdated Show resolved Hide resolved
@fire fire force-pushed the godot-patches branch 3 times, most recently from d5c6a5a to 0cf5739 Compare April 19, 2022 13:03
@fire
Copy link
Contributor Author

fire commented Apr 19, 2022

@elalish 1 workflow awaiting approval Can you approve the workflow?

tools/perf_test.cpp Outdated Show resolved Hide resolved
third_party/graphlite/include/graph.h Outdated Show resolved Hide resolved
manifold/src/impl.cu Outdated Show resolved Hide resolved
@elalish
Copy link
Owner

elalish commented Apr 19, 2022

CUDA compilation work on my machine, so i may just need to update the version of CUDA on the CI.

@fire
Copy link
Contributor Author

fire commented Apr 20, 2022

Rebased.

Needs cicd approval.

Also, do geometry tests run on github actions?

Edited:

I have some things in the queue, so I will pause this until the weekend.

@elalish
Copy link
Owner

elalish commented Apr 20, 2022

No prob, I should have the graph stuff merged by then, which hopefully will supersede this PR. Yes, all the tests, most of which are geometry, run on the CI.

@elalish
Copy link
Owner

elalish commented Apr 21, 2022

Is this needed anymore now that #92 is merged?

@fire
Copy link
Contributor Author

fire commented Apr 21, 2022

Let me check.

@fire
Copy link
Contributor Author

fire commented Apr 21, 2022

@elalish Cicd approval required.

@elalish elalish merged commit 0453418 into elalish:master Apr 21, 2022
@fire fire deleted the godot-patches branch April 21, 2022 17:32
cartesian-theatrics pushed a commit to SovereignShop/manifold that referenced this pull request Mar 11, 2024
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.

None yet

2 participants