Skip to content

Commit 731e8d0

Browse files
sawenzelclaude
andcommitted
Say that a converted geometry's payload paths are absolute
This corrects the README on whether a converted folder can be moved. - The macro was documented as resolving its payloads relative to its own location. - It names them by absolute path, so the folder cannot be moved or handed on as it stands. - The sed that rewrites the prefix is given instead. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
1 parent dd0a7fa commit 731e8d0

1 file changed

Lines changed: 7 additions & 1 deletion

File tree

‎Detectors/CADSupport/README.md‎

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,13 @@ The output folder holds:
7878
- `brep_*.brep` (with `--dump-brep`);
7979
- `surface_report.json` (with `--surface-report PATH`).
8080

81-
The macro loads its payloads relative to its own location, so move the folder as a whole.
81+
The macro names its payloads by absolute path, so a converted folder cannot be moved or handed
82+
on as it stands. Rewrite the prefix before shipping one, and run from the directory the payload
83+
folder sits in:
84+
85+
```bash
86+
sed -i "s|<absolute conversion dir>/|payloads/|g" geom.C
87+
```
8288

8389
`geom.C` exports `get_builder_hook_unchecked()`, which `o2-sim` calls, and
8490
`build_and_export(const char* out_root = "geom.root", bool check = true, bool checkOverlaps = false)`

0 commit comments

Comments
 (0)