Skip to content

Conversation

@A-Wanderer
Copy link
Contributor

No description provided.

@A-Wanderer A-Wanderer force-pushed the parquet branch 3 times, most recently from 4d7c6ad to e3acc98 Compare June 19, 2022 22:09
@A-Wanderer
Copy link
Contributor Author

背景:
cuda:11.4.2
cudnn8
ubuntu18.04
os.environ['ARROW_NUM_THREADS'] = '2' os.environ['CUDA_VISIBLE_DEVICES'] = ''

image

@liutongxuan liutongxuan added the enhancement New feature or request label Jun 24, 2022
@2sin18 2sin18 self-requested a review June 24, 2022 10:05
"""sed -i.bak 's/type_traits/std::max<int16_t>(sizeof(int16_t), type_traits/g' cpp/src/parquet/column_reader.cc""",
"""sed -i.bak 's/value_byte_size/value_byte_size)/g' cpp/src/parquet/column_reader.cc""",
],
sha256 = "a27971e2a71c412ae43d998b7b6d06201c7a3da382c804dcdc4a8126ccbabe67",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Attention: Arrow 4.x is incompatible with Apsara libraries. Compile DeepRec with Apsara should disable the capabilities.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Can I use Arrow 5.x to avoid this?

Copy link
Member

Choose a reason for hiding this comment

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

开源版本暂时不考虑飞天,咱们内部版本会有修改。

Copy link
Collaborator

Choose a reason for hiding this comment

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

@A-Wanderer No, you should add a compiler macro to disable the library if DeepRec must be compiled with apsara libraries.

@A-Wanderer A-Wanderer force-pushed the parquet branch 2 times, most recently from da64864 to 84344f5 Compare June 24, 2022 18:26
@liutongxuan liutongxuan merged commit b5a74e7 into DeepRec-AI:main Jul 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants