Skip to content

Update README.md about eedE - #198

Merged
michmx merged 9 commits into
HEP-FCC:mainfrom
apausa:document-eede
Sep 15, 2026
Merged

michmx merged 9 commits into
HEP-FCC:mainfrom
apausa:document-eede

Conversation

@apausa

@apausa apausa commented Aug 31, 2026 •

Copy link
Copy Markdown
Member
  • Update README.md with information about the new features and functionalities.
  • Replace images with the current interface.
  • Add image showing filtered out particles.

@apausa
apausa marked this pull request as draft August 31, 2026 14:57
@apausa apausa changed the title Update README.md for eedE Update README.md about eedE Sep 1, 2026
Comment thread 2-gen-and-fastsim/2-4-eedE/README.md Outdated
Comment thread 2-gen-and-fastsim/2-4-eedE/README.md Outdated
Comment thread 2-gen-and-fastsim/2-4-eedE/README.md Outdated
Comment thread 2-gen-and-fastsim/2-4-eedE/README.md Outdated
@apausa
apausa marked this pull request as ready for review September 1, 2026 13:38

@michmx michmx left a comment •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks a lot for this update! I could follow it easily.

I am just leaving a couple of suggestions. And let's wait for @kjvbrt to give the green light after addressing his comments.

Comment thread 2-gen-and-fastsim/2-4-eedE/README.md Outdated
@michmx
michmx requested a review from kjvbrt September 3, 2026 19:58
Comment thread 2-gen-and-fastsim/2-4-eedE/README.md
@apausa

apausa commented Sep 9, 2026

Copy link
Copy Markdown
Member Author

Hi @michmx, the tests are not being ran, can you please take a look at the GitHub Action? I believe the nightly stack should be sourced with --spack. Thank you

@michmx

michmx commented Sep 15, 2026

Copy link
Copy Markdown
Collaborator

Hi @apausa . Sorry for the late reply. Addresing the issue at #202.

(It is related to key4hep/key4hep-images#32)

@apausa

apausa commented Sep 15, 2026

Copy link
Copy Markdown
Member Author

Hi @michmx, just force-pushed. This can be merged, as well as #201. Thank you!

@michmx
michmx merged commit f3c819b into HEP-FCC:main Sep 15, 2026
2 checks passed
@apausa
apausa deleted the document-eede branch September 21, 2026 10:20
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.

3 participants