Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add sense field to Instance #28

Merged
merged 6 commits into from
Apr 23, 2024
Merged

Add sense field to Instance #28

merged 6 commits into from
Apr 23, 2024

Conversation

NY57
Copy link
Member

@NY57 NY57 commented Apr 23, 2024

変更点

OMMX::Instanceが意味する最適化問題が最小化問題であるか最大化問題であるかが判別できないという課題があった為、senseを追加しました。

確認事項

こちらの変更後、Pythonパッケージをv0.1.2として、Rustクレートをv0.1.1としてリリースしても良いでしょうか?(バージョニングは適切でしょうか?)

@NY57 NY57 requested review from termoshtt and Shitaro April 23, 2024 05:43
NY57 and others added 4 commits April 23, 2024 15:45
Co-authored-by: Toshiki Teramura <toshiki.teramura@gmail.com>
Co-authored-by: Toshiki Teramura <toshiki.teramura@gmail.com>
Co-authored-by: Toshiki Teramura <toshiki.teramura@gmail.com>
Copy link
Member

@termoshtt termoshtt left a comment

Choose a reason for hiding this comment

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

LGTM

@NY57 NY57 merged commit e0f5114 into main Apr 23, 2024
24 checks passed
@NY57 NY57 deleted the add_sense branch April 23, 2024 06:53
@termoshtt termoshtt added the proto Changes in protobuf schema label Apr 26, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
proto Changes in protobuf schema
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants