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

Missing imports in documentation #828

Open
stavros11 opened this issue Mar 6, 2024 · 1 comment
Open

Missing imports in documentation #828

stavros11 opened this issue Mar 6, 2024 · 1 comment
Labels
documentation Improvements or additions to documentation

Comments

@stavros11
Copy link
Member 

          Maybe we should take the time to investigate this, even if probably not immediately. It could be confusing for users (as it was in #801), and maybe there is a configuration to change this behavior.

Originally posted by @alecandido in #789 (comment)

We should investigate why CI is not failing even if some doctest cells, such as

qibolab/doc/source/main-documentation/qibolab.rst

Line 392 in 5953517

sequence = PulseSequence()

and

qibolab/doc/source/main-documentation/qibolab.rst

Line 512 in 5953517

sequence = PulseSequence()

are missing imports.
@stavros11 stavros11 added the documentation Improvements or additions to documentation label Mar 6, 2024
@stavros11 stavros11 mentioned this issue Mar 6, 2024
Merged
16 tasks
@rscircus
Copy link
Contributor

Ran into the same thing, hence, decided to use literal includes. See also:

@rscircus rscircus mentioned this issue Apr 21, 2024
Open
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rscircus @stavros11