Skip to content

Conversation

@AckslD
Copy link
Member

@AckslD AckslD commented Apr 22, 2019

See corresponding pull request for simulaqron with more info at: SoftwareQuTech/SimulaQron#175

Axel Dahlberg added 3 commits March 29, 2019 11:40
@AckslD AckslD requested review from LeonWubben and Wojtek242 April 22, 2019 16:14
@AckslD AckslD self-assigned this Apr 22, 2019
Copy link
Contributor

@LeonWubben LeonWubben left a comment

Choose a reason for hiding this comment

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

make: *** No rule to make target '_verified', needed by 'verify'. Stop.
I know it's copied over code, and not part of this pr. Still good to let you know

I don't know what "make build" does, but if fails as well

$ make build
/usr/lib/python3.6/distutils/dist.py:261: UserWarning: Unknown distribution option: 'long_description_content_type'
  warnings.warn(msg)
/usr/lib/python3/dist-packages/setuptools/dist.py:294: UserWarning: The version specified ("'3.0.0'") is an invalid version, this may not work as expected with newer versions of setuptools, pip, and PyPI. Please see PEP 440 for more details.
  "details." % self.metadata.version
usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
   or: setup.py --help [cmd1 cmd2 ...]
   or: setup.py --help-commands
   or: setup.py cmd --help

error: invalid command 'bdist_wheel'
Makefile:28: recipe for target '_build' failed
make: *** [_build] Error 1

Should be fine codewise. Can't finish the pr before finishing the simulaqron one though

self._cqcNet = networkConfig(self.cqcFile)
if socket_address is None or use_classical_communication:
if backend is None or backend == "simulaqron":
if _simulaqron_major < 3:
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't know if this is the best way to go about this. But this should probably be in simulaqron itself as requirement. V2 simulaqron should have cqc <3.0.0

Copy link
Member Author

Choose a reason for hiding this comment

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

@LeonWubben Why should it be in SimulaQron? Why should simlaqron v2 have cqc<3.0.0, simulaqron v2 works fine with cqc 3 it's the pythonlibrary of cqc 3 which does not work with simulaqron 2.

@AckslD
Copy link
Member Author

AckslD commented Apr 24, 2019

@LeonWubben What's your version of setuptools? You also need wheel installed.

@AckslD AckslD merged commit 43a06b9 into Develop Apr 27, 2019
@AckslD AckslD deleted the 151-settings branch April 27, 2019 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants