Skip to content

DM-50298: Write configs when initializing new RUN collections.#117

Open
TallJimbo wants to merge 1 commit intomainfrom
tickets/DM-50298
Open

DM-50298: Write configs when initializing new RUN collections.#117
TallJimbo wants to merge 1 commit intomainfrom
tickets/DM-50298

Conversation

@TallJimbo
Copy link
Copy Markdown
Contributor

No description provided.

task = taskFactory.makeTask(taskNode, butler, inputRefs)

for writeEdge in taskNode.init.outputs.values():
for writeEdge in taskNode.init.iter_all_outputs():
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Only just noticed this, and just gave it a go, and it failed with errors like

Traceback (most recent call last):
  File "/sdf/home/m/mfl/lsst/rubintv_production/tests/ci/test_rapid_analysis.py", line 719, in _exec_script
    exec(script_content, exec_globals)
  File "<string>", line 41, in <module>
  File "/sdf/home/m/mfl/lsst/rubintv_production/python/lsst/rubintv/production/processingControl.py", line 632, in __init__
    self.outputRun = self.getLatestRunAndPrep(forceNewRun=forceNewRun)
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/sdf/home/m/mfl/lsst/rubintv_production/python/lsst/rubintv/production/processingControl.py", line 666, in getLatestRunAndPrep
    latestRun = ensureRunCollection(
                ^^^^^^^^^^^^^^^^^^^^
  File "/sdf/home/m/mfl/lsst/rubintv_production/python/lsst/rubintv/production/processingControl.py", line 167, in ensureRunCollection
    getattr(task, writeEdge.connection_name),
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'IsrTaskLSST' object has no attribute '_config'. Did you mean: 'config'?

Any idea what's wrong there? I've not investigated, as I figured it'd likely be obvious to you.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Just a bad first attempt by me. I've replaced that commit with a new one that I think should work.

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.

2 participants