-
Notifications
You must be signed in to change notification settings - Fork 32
Wipe cycstub repo #419
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
Wipe cycstub repo #419
Changes from 13 commits
364d292
5741731
81ef584
9473224
27cf707
2cc538a
7ef8007
f43aaf6
a2356c5
c41a394
6ad43cf
e1238b7
46417e5
573d153
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,4 @@ | ||
| USE_CYCLUS("helloworld" "tutorial_facility") | ||
| USE_CYCLUS("helloworld" "tutorial_inst") | ||
| USE_CYCLUS("helloworld" "tutorial_region") | ||
| INSTALL_CYCLUS_MODULE("helloworld" "tutorial") | ||
| USE_CYCLUS("TutorialLibrary" "tutorial_facility") | ||
| USE_CYCLUS("TutorialLibrary" "tutorial_inst") | ||
| USE_CYCLUS("TutorialLibrary" "tutorial_region") | ||
| INSTALL_CYCLUS_MODULE("TutorialLibrary" "") |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,3 @@ | ||
| INSTALL_CYCLUS_STANDALONE("TutorialFacility" "tutorial_facility" "tutorial") | ||
| INSTALL_CYCLUS_STANDALONE("TutorialInst" "tutorial_inst" "tutorial") | ||
| INSTALL_CYCLUS_STANDALONE("TutorialRegion" "tutorial_region" "tutorial") | ||
| INSTALL_CYCLUS_STANDALONE("TutorialFacility" "tutorial_facility" "" ) | ||
| INSTALL_CYCLUS_STANDALONE("TutorialInstitution" "tutorial_inst" "") | ||
| INSTALL_CYCLUS_STANDALONE("TutorialRegion" "tutorial_region" "") |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,15 +2,17 @@ A Tour of Cycstub | |
| ================= | ||
|
|
||
| Since |Cyclus| version 1.3.1 the Cycstub CLI is included in a standard |Cyclus| installation. | ||
| If you have an older version of |Cyclus| please refer to the `Cycstub repository <https://github.com/cyclus/cycstub>`_ | ||
| for installation instructions. | ||
|
|
||
| This section will walk through the source files of the stub :term:`archetypes | ||
| <archetype>` in Cycstub. Cycstub provides | ||
| three stub archetypes: | ||
| <archetype>` generated when a user invokes the ``cycstub`` utility. ``Cycstub`` | ||
| can generate stubs for each of the agent types. The standard usage to generate the | ||
| stubs for a Facility, Institution or Region, respectively, is: | ||
|
|
||
| .. code-block:: console | ||
|
|
||
| * ``StubFacility`` | ||
| * ``StubInstitution`` | ||
| * ``StubRegion`` | ||
| cycstub --type facility :stublibrary:StubFacility | ||
| cycstub --type institution :stublibrary:StubInstitution | ||
| cycstub --type region :stublibrary:StubRegion | ||
|
Comment on lines
+13
to
+15
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do we want these to have CamelCase in the library name?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I did it this way before I committed to fixing the CamelCase problem. In the fixed version, all combinations will work successfully, so these don't need to change. |
||
|
|
||
| We will walk through ``StubFacility``\ 's source specifically because its the | ||
| most complicated of the three. | ||
|
|
@@ -34,26 +36,32 @@ which includes most :term:`cyclus kernel` headers as a convenience for new | |
| include the specific kernel headers you need if you require a smaller | ||
| executable. | ||
|
|
||
| Moving on in the header file we come to the class declaration | ||
| Moving on in the header file we come to the class declaration inside the namespace | ||
| of this specific module library, `stublibrary`, with standard Doxygen documentation | ||
| strings. | ||
|
|
||
| .. code-block:: cpp | ||
|
|
||
| namespace stublibrary { | ||
|
|
||
| /// @class StubFacility | ||
| /// ... | ||
|
|
||
| class StubFacility : public cyclus::Facility { | ||
|
|
||
| which simply states that the ``StubFacility`` inherits from ``cyclus::Facility``. | ||
|
|
||
| We then come to the constructor declaration | ||
| We then come to the constructor declaration in the header file | ||
|
|
||
| .. code-block:: cpp | ||
|
|
||
| explicit StubFacility(cyclus::Context* ctx); | ||
|
|
||
| and implementation | ||
| and definition in the implementation file | ||
|
|
||
| .. code-block:: cpp | ||
|
|
||
| StubFacility::StubFacility(cyclus::Context* ctx) | ||
| : cyclus::Facility(ctx) {}; | ||
| StubFacility::StubFacility(cyclus::Context* ctx) : cyclus::Facility(ctx) {}; | ||
|
|
||
| The constructor takes a single ``cyclus::Context`` argument. The :term:`context` | ||
| is the mechanism by which :term:`agents <agent>` can query and otherwise | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure what our minimum version is these days????
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My guess would be python 3.9, but I can look around.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Going off of the dependencies for installing cyclus via conda, it looks like 3.8 is the lowest version allowed. I can build cyclus with 3.8 locally, but the unit tests won't pass unless you have python 3.11+, as of commit 4d1eeef5