Open charlesbaynham opened 3 months ago
This breaks abstraction layers, HasEnvironment does not otherwise know about kernels. Note that NAC3 does not use kernel_invariants.
This breaks abstraction layers, HasEnvironment does not otherwise know about kernels. Note that NAC3 does not use kernel_invariants.
True. Is there a better place for it you can think of?
ARTIQ Pull Request
Description of Changes
A simple usability tweak. Ensures that
kernel_invariants
always exists, so that user code doesn't need to add the usual boilerplate ofself.kernel_invariants = getattr(self, "kernel_invariants", set())
before each addition.Related Issue
Type of Changes
Steps (Choose relevant, delete irrelevant before submitting)
All Pull Requests
git commit --signoff
, see copyright).Code Changes
flake8
to check code style (follow PEP-8 style).flake8
has issues with parsing Migen/gateware code, ignore as necessary.Documentation Changes
cd doc/manual/; make html
) to ensure no errors.Git Logistics
git rebase --interactive
). Merge/squash/fixup commits that just fix or amend previous commits. Remove unintended changes & cleanup. See tutorial.[ ] Write short & meaningful commit messages. Review each commit for messages (
git show
). Format:Licensing
See copyright & licensing for more info. ARTIQ files that do not contain a license header are copyrighted by M-Labs Limited and are licensed under LGPLv3+.