openvdb icon indicating copy to clipboard operation
openvdb copied to clipboard

WIP: fVDB CI setup

Open swahtz opened this issue 1 year ago • 3 comments

swahtz avatar Sep 18 '24 09:09 swahtz

@danrbailey, I implemented the updates since the last attempt (merge master->feature/fvdb and feature/fvdb->fvdb_ci_setup) but I'm still triggering all the other tests. I manually cancelled them. Perhaps it is the changes I make here to .github/workflows?

swahtz avatar Sep 18 '24 19:09 swahtz

@danrbailey, I implemented the updates since the last attempt (merge master->feature/fvdb and feature/fvdb->fvdb_ci_setup) but I'm still triggering all the other tests. I manually cancelled them. Perhaps it is the changes I make here to .github/workflows?

I wondered that too, I can't seem to find any reference docs on this to confirm this. Either way, I think now that you've merged your changes to these .github/workflow files to master, I don't think you need to modify either of these yaml files? From the diff, it looks like you're adding the fvdb/** line when it already exists?

danrbailey avatar Sep 18 '24 21:09 danrbailey

@danrbailey, I implemented the updates since the last attempt (merge master->feature/fvdb and feature/fvdb->fvdb_ci_setup) but I'm still triggering all the other tests. I manually cancelled them. Perhaps it is the changes I make here to .github/workflows?

I wondered that too, I can't seem to find any reference docs on this to confirm this. Either way, I think now that you've merged your changes to these .github/workflow files to master, I don't think you need to modify either of these yaml files? From the diff, it looks like you're adding the fvdb/** line when it already exists?

Oh yes, that is a mistake... a slight mismatch in the changes I was making. I'll make sure to fix that up.

swahtz avatar Sep 19 '24 02:09 swahtz