Skip to content
Snippets Groups Projects

Publish oop

Merged
Mayr, Hannesrequested to merge
publish-oop into dev
1 open thread

OOP can still be embraced in the code. But for now the basics of publish are implemented in OOP-style.

Merge request reports

Pipeline #764923 passed

Pipeline passed for fc5adde9 on publish-oop

Approved by

Merged by Richter, ManuelaRichter, Manuela Jul 27, 2022 (Jul 27, 2022 7:10am UTC)

Merge details

  • Changes merged into dev with ed0e2fea.
  • Deleted the source branch.

Pipeline #768773 passed

Pipeline passed for ed0e2fea on dev

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
78 Attributes
79 ----------
80 src_datapath : string
81 Path to data that should be published.
82 dst_path : string
83 Path to the destination directory.
84 figure : figure object
85 Figure that was tagged and now should be saved as picture.
86 plot_name: string
87 Name for the exported plot.
88 data_storage: string
89 Method how the data should be stored. Available options:
90 centralized: The raw data will copied only once. All other plots
91 will reference this data via sym link.
92 individual: The complete raw data will be copied to a folder for
93 every plot, respectively. This is the default value.
  • [...] every plot respectively.

    I believe? We could also talk about "data file" instead of "raw data" (It might be stored with compression, or with metadata, which makes it "less raw" :D)

  • Please register or sign in to reply
  • I believe the comma is correctly used: https://www.advancedenglishediting.com/blog/respectively-how-to-use-it-in-your-writing
    But the word "respectively" makes no sense here, so I leave it out and also adjust the term of "raw data" in the next commit.

  • Mayr, Hannes added 1 commit

    added 1 commit

    • eff0b1af - Small documentation changes.

    Compare with previous version

  • Mayr, Hannes added 7 commits

    added 7 commits

    Compare with previous version

  • Richter, Manuela approved this merge request

    approved this merge request

  • mentioned in commit ed0e2fea

  • mentioned in issue #55 (closed)

  • Please register or sign in to reply
    Loading