@ajharvie can you return to this PR, because this shouldn't be approved.
First, the build files should not be included in the main repository - this is your local test only. On gitignore, you can add something like */build/ to prevent any build folder from being added and not only the ones in the main directory.
Second, you shouldn't remove the sessions about parallel/GPU. I understand that the delivery did not include all the material, but I don't see any reason for preventing people from having the material. We get questions related to speeding up the code that would be useful to reply to by pointing to the material, and I can not do this at the moment, as you removed all the parallel/GPU parts of it. One thing is the delivery, one thing is the material available.
So, for actions, can you
Originally posted by @patricia-ternes in #24 (comment)
@ajharvie can you return to this PR, because this shouldn't be approved.
First, the build files should not be included in the main repository - this is your local test only. On gitignore, you can add something like
*/build/to prevent any build folder from being added and not only the ones in the main directory.Second, you shouldn't remove the sessions about parallel/GPU. I understand that the delivery did not include all the material, but I don't see any reason for preventing people from having the material. We get questions related to speeding up the code that would be useful to reply to by pointing to the material, and I can not do this at the moment, as you removed all the parallel/GPU parts of it. One thing is the delivery, one thing is the material available.
So, for actions, can you
Originally posted by @patricia-ternes in #24 (comment)