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.

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

Test script needs to test file writing #27

Closed
kibaekkim opened this issue Apr 18, 2020 · 3 comments
Closed

Test script needs to test file writing #27

kibaekkim opened this issue Apr 18, 2020 · 3 comments
Assignees
Labels
bug Something isn't working enhancement New feature or request

Comments

@kibaekkim
Copy link
Collaborator

kibaekkim commented Apr 18, 2020

The test script should test writing files fro CBF and SDPA formats. The current file write functions are not working, although the commit comment says test passed.
Related to #26

@kibaekkim kibaekkim added bug Something isn't working enhancement New feature or request labels Apr 18, 2020
@kibaekkim
Copy link
Collaborator Author

kibaekkim commented Apr 22, 2020

I have reported bugs for both CBF and SDPA writers in MOI.

But, integer variables are not supported in SDPA format. I threw the issue to MOI: jump-dev/MathOptInterface.jl#1074

@bcdandurand
Copy link
Collaborator

CBF format is tested in test/runtests.jl as of last push to master.

I haven't looked at what has been done with SDPA format yet, and these are not in test yet, so I'm keeping the issue open.

@kibaekkim
Copy link
Collaborator Author

I have pushed the updated test scripts. I decided to use JuMP functions for writing files. So we don't need to test it in our package.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants