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.

Sign up for GitHub

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

Jump to bottom

FieldsBuilder raise runtime error on cc backend #5143

Open
erizmr opened this issue Jun 13, 2022 · 3 comments
Open

FieldsBuilder raise runtime error on cc backend #5143

erizmr opened this issue Jun 13, 2022 · 3 comments
Labels
potential bug Something that looks like a bug but not yet confirmed

Comments

@erizmr
Copy link
Contributor

erizmr commented Jun 13, 2022

Describe the bug
SNode writer raise runtime error with FieldsBuilder on cc backend.

To Reproduce

import taichi as ti
ti.init(arch=ti.cc, log_level=ti.TRACE)

dual_root = ti.FieldsBuilder()
x_dual = ti.field(float)
dual_root.dense(ti.i, 5).place(x_dual)
dual_root.finalize()

Log/Screenshots
image
image

Additional comments
The script works fine on cpu, cuda, vulkan, opengl.
@erizmr erizmr added the potential bug Something that looks like a bug but not yet confirmed label Jun 13, 2022
@taichi-ci-bot taichi-ci-bot moved this to Untriaged in Taichi Lang Jun 13, 2022
erizmr added a commit to erizmr/taichi that referenced this issue Jun 13, 2022
@erizmr
[autodiff] Support basic operations for forward mode autodiff
2f2fa17 
Support cpu and gpu backends. The cc backend has an issue on FieldBuilder ref to taichi-dev#5143.
erizmr added a commit that referenced this issue Jun 13, 2022
@erizmr
[autodiff] Support basic operations for forward mode autodiff
b6c30d1 
Support cpu and gpu backends. The cc backend has an issue on FieldBuilder ref to #5143.

[ghstack-poisoned]
@erizmr erizmr mentioned this issue Jun 13, 2022
Closed
erizmr added a commit that referenced this issue Jun 13, 2022
@erizmr
[autodiff] Support basic operations for forward mode autodiff
970401d 
Support cpu and gpu backends. The cc backend has an issue on FieldBuilder ref to #5143.

ghstack-source-id: 5ff22fe6dfa0720fc3b2ceafb27c6c8051cec542
Pull Request resolved: #5145
@k-ye
Copy link
Member

k-ye commented Jun 13, 2022

Is there any specific reason that you want to use the CC backend..?

erizmr added a commit to erizmr/taichi that referenced this issue Jun 13, 2022
@erizmr
[autodiff] Support basic operations for forward mode autodiff
33dbcc2 
Support cpu and gpu backends. The cc backend has an issue on FieldBuilder ref to taichi-dev#5143.
erizmr added a commit that referenced this issue Jun 13, 2022
@erizmr
[autodiff] Support basic operations for forward mode autodiff
872fceb 
Support cpu and gpu backends. The cc backend has an issue on FieldBuilder ref to #5143.

[ghstack-poisoned]
@erizmr erizmr mentioned this issue Jun 13, 2022
Closed
erizmr added a commit that referenced this issue Jun 13, 2022
@erizmr
[autodiff] Support basic operations for forward mode autodiff
8427526 
Support cpu and gpu backends. The cc backend has an issue on FieldBuilder ref to #5143.

ghstack-source-id: 3d94eff5173c5113be434c19d46bc4fec6b0032e
Pull Request resolved: #5147
@erizmr
Copy link
Contributor Author

erizmr commented Jun 13, 2022

Is there any specific reason that you want to use the CC backend..?

There are test cases based on the cc backends.. ioi

erizmr added a commit that referenced this issue Jun 13, 2022
@erizmr
Update on "[autodiff] Support basic operations for forward mode autod…
22687f7 
…iff"

Support cpu and gpu backends. The cc backend has an issue on FieldBuilder ref to #5143.

[ghstack-poisoned]
erizmr added a commit that referenced this issue Jun 13, 2022
@erizmr
[autodiff] Support basic operations for forward mode autodiff
81ba712 
Support cpu and gpu backends. The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently only supports materializing one snode tree (See OpenglProgramImpl::compile_snode_tree_types in opengl_program.cpp), thus FieldBuilder is not supported.

ghstack-source-id: 92df76a148c759671838ec864b328c7a56dfea5b
Pull Request resolved: #5147
erizmr added a commit that referenced this issue Jun 13, 2022
@erizmr
Update on "[autodiff] Support basic operations for forward mode autod…
a0d7ad0 
…iff"


Support cpu and gpu backends. The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently does not support materializing multiple snode trees (see [OpenglProgramImpl::compile_snode_tree_types](https://github.com/taichi-dev/taichi/blob/master/taichi/backends/opengl/opengl_program.cpp#L45)), thus `FieldBuilder` is not supported.

[ghstack-poisoned]
erizmr added a commit that referenced this issue Jun 13, 2022
@erizmr
[autodiff] Support basic operations for forward mode autodiff
b9c92cf 
Support cpu and gpu backends. The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently only supports materializing one snode tree (See OpenglProgramImpl::compile_snode_tree_types in opengl_program.cpp), thus FieldBuilder is not supported.

ghstack-source-id: af3cc46c5cea62dddf942013d7ac86a408a041ed
Pull Request resolved: #5147
erizmr added a commit that referenced this issue Jun 13, 2022
@erizmr
Update base for Update on "[autodiff] Support basic operations for fo…
cb74979 
…rward mode autodiff"


Support cpu and gpu backends. The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently does not support materializing multiple snode trees (see [OpenglProgramImpl::compile_snode_tree_types](https://github.com/taichi-dev/taichi/blob/master/taichi/backends/opengl/opengl_program.cpp#L45)), thus `FieldBuilder` is not supported.

[ghstack-poisoned]
erizmr added a commit that referenced this issue Jun 13, 2022
@erizmr
Update on "[autodiff] Support basic operations for forward mode autod…
8abe2f3 
…iff"


Support cpu and gpu backends. The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently does not support materializing multiple snode trees (see [OpenglProgramImpl::compile_snode_tree_types](https://github.com/taichi-dev/taichi/blob/master/taichi/backends/opengl/opengl_program.cpp#L45)), thus `FieldBuilder` is not supported.

[ghstack-poisoned]
erizmr added a commit that referenced this issue Jun 13, 2022
@erizmr
[autodiff] Support basic operations for forward mode autodiff
0d666d8 
Support cpu and gpu backends. The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently only supports materializing one snode tree (See OpenglProgramImpl::compile_snode_tree_types in opengl_program.cpp), thus FieldBuilder is not supported.

ghstack-source-id: b1d6529c9e14e318ac84193e6bb591ab4028ff7a
Pull Request resolved: #5147
erizmr added a commit that referenced this issue Jun 14, 2022
@erizmr
[autodiff] Support basic operations for forward mode autodiff
66d84cc 
Support cpu and gpu backends.
The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently does not support materializing multiple snode trees (see OpenglProgramImpl::compile_snode_tree_types), thus FieldBuilder is not supported.

[ghstack-poisoned]
@erizmr erizmr mentioned this issue Jun 14, 2022
Merged
erizmr added a commit that referenced this issue Jun 14, 2022
@erizmr
[autodiff] Support basic operations for forward mode autodiff
0f760c8 
Support cpu and gpu backends.
The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently does not support materializing multiple snode trees (see OpenglProgramImpl::compile_snode_tree_types), thus FieldBuilder is not supported.

ghstack-source-id: b6a758af374b78d151a19297dd3351ef33bd85e7
Pull Request resolved: #5160
erizmr added a commit that referenced this issue Jun 14, 2022
@erizmr
Update base for Update on "[autodiff] Support basic operations for fo…
e1fba49 
…rward mode autodiff"

Support cpu and gpu backends.
The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently does not support materializing multiple snode trees (see OpenglProgramImpl::compile_snode_tree_types), thus FieldBuilder is not supported.

[ghstack-poisoned]
erizmr added a commit that referenced this issue Jun 14, 2022
@erizmr
Update on "[autodiff] Support basic operations for forward mode autod…
4aef890 
…iff"

Support cpu and gpu backends.
The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently does not support materializing multiple snode trees (see OpenglProgramImpl::compile_snode_tree_types), thus FieldBuilder is not supported.

[ghstack-poisoned]
erizmr added a commit that referenced this issue Jun 14, 2022
@erizmr
[autodiff] Support basic operations for forward mode autodiff
905a1ab 
Support cpu and gpu backends.
The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently does not support materializing multiple snode trees (see OpenglProgramImpl::compile_snode_tree_types), thus FieldBuilder is not supported.

ghstack-source-id: 97cafab313a596b79295050d6edbfd4625917a97
Pull Request resolved: #5160
erizmr added a commit that referenced this issue Jun 14, 2022
@erizmr
Update base for Update on "[autodiff] Support basic operations for fo…
01ffaac 
…rward mode autodiff"

Support cpu and gpu backends.
The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently does not support materializing multiple snode trees (see OpenglProgramImpl::compile_snode_tree_types), thus FieldBuilder is not supported.

[ghstack-poisoned]
erizmr added a commit that referenced this issue Jun 14, 2022
@erizmr
Update on "[autodiff] Support basic operations for forward mode autod…
494b5ec 
…iff"

Support cpu and gpu backends.
The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently does not support materializing multiple snode trees (see OpenglProgramImpl::compile_snode_tree_types), thus FieldBuilder is not supported.

[ghstack-poisoned]
erizmr added a commit that referenced this issue Jun 14, 2022
@erizmr
[autodiff] Support basic operations for forward mode autodiff
ee2dcea 
Support cpu and gpu backends.
The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently does not support materializing multiple snode trees (see OpenglProgramImpl::compile_snode_tree_types), thus FieldBuilder is not supported.

ghstack-source-id: 3148a68ab7d4146586ac2a272b7aa6808c821c51
Pull Request resolved: #5160
@erizmr erizmr moved this from Untriaged to Backlog in Taichi Lang Jun 14, 2022
erizmr added a commit that referenced this issue Jun 14, 2022
@erizmr
Update base for Update on "[autodiff] Support basic operations for fo…
7db8389 
…rward mode autodiff"


Support cpu and gpu backends.
The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently does not support materializing multiple snode trees (see OpenglProgramImpl::compile_snode_tree_types), thus FieldBuilder is not supported.

Related #5055 

[ghstack-poisoned]
erizmr added a commit that referenced this issue Jun 14, 2022
@erizmr
Update on "[autodiff] Support basic operations for forward mode autod…
ce01e36 
…iff"


Support cpu and gpu backends.
The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently does not support materializing multiple snode trees (see OpenglProgramImpl::compile_snode_tree_types), thus FieldBuilder is not supported.

Related #5055 

[ghstack-poisoned]
erizmr added a commit that referenced this issue Jun 14, 2022
@erizmr
[autodiff] Support basic operations for forward mode autodiff
3b0bba5 
Support cpu and gpu backends.
The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently does not support materializing multiple snode trees (see OpenglProgramImpl::compile_snode_tree_types), thus FieldBuilder is not supported.

ghstack-source-id: 6b8ea7fff130cea9ef9c78933291b6f719cab9de
Pull Request resolved: #5160
erizmr added a commit that referenced this issue Jun 14, 2022
@erizmr
Update base for Update on "[autodiff] Support basic operations for fo…
ba19461 
…rward mode autodiff"


Support cpu and gpu backends.
The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently does not support materializing multiple snode trees (see OpenglProgramImpl::compile_snode_tree_types), thus FieldBuilder is not supported.

Related #5055 

[ghstack-poisoned]
erizmr added a commit that referenced this issue Jun 14, 2022
@erizmr
Update on "[autodiff] Support basic operations for forward mode autod…
937c317 
…iff"


Support cpu and gpu backends.
The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently does not support materializing multiple snode trees (see OpenglProgramImpl::compile_snode_tree_types), thus FieldBuilder is not supported.

Related #5055 

[ghstack-poisoned]
erizmr added a commit that referenced this issue Jun 14, 2022
@erizmr
[autodiff] Support basic operations for forward mode autodiff
92708c7 
Support cpu and gpu backends.
The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently does not support materializing multiple snode trees (see OpenglProgramImpl::compile_snode_tree_types), thus FieldBuilder is not supported.

ghstack-source-id: 28c6d5f219d93aa8688b910e9ce84ec77d195728
Pull Request resolved: #5160
ailzhang pushed a commit that referenced this issue Jun 14, 2022
@erizmr
[autodiff] Support basic operations for forward mode autodiff
5fda516 
Support cpu and gpu backends.
The cc backend has an issue on FieldBuilder ref to #5143.
The opengl backend currently does not support materializing multiple snode trees (see OpenglProgramImpl::compile_snode_tree_types), thus FieldBuilder is not supported.

ghstack-source-id: 28c6d5f219d93aa8688b910e9ce84ec77d195728
Pull Request resolved: #5160
@k-ye
Copy link
Member

k-ye commented Jun 15, 2022

OK... If no-one is using it, we should just drop it later. It's not worth the maintenance cost.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
potential bug Something that looks like a bug but not yet confirmed
Projects
Taichi Lang
Status: Backlog
Milestone
No milestone
Development

No branches or pull requests
2 participants
@k-ye @erizmr