Skip to content
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

Fixes in ACTS dependent analyzers #259

Merged
merged 4 commits into from
Mar 28, 2023
Merged

Fixes in ACTS dependent analyzers #259

merged 4 commits into from
Mar 28, 2023

Conversation

kjvbrt
Copy link
Contributor

@kjvbrt kjvbrt commented Mar 22, 2023

Also fixing issues when compiling with the latest version of the ACTS.

@kjvbrt kjvbrt requested a review from BrieucF March 22, 2023 10:31
@BrieucF
Copy link
Contributor

BrieucF commented Mar 27, 2023

@EmanuelPerez could you confirm that it is ok to switch ACTS off by default?

@EmanuelPerez
Copy link
Contributor

Hi Brieuc, I have just posted a message to the software-analysis e-group, asking if anyone is using it.

@kjvbrt kjvbrt changed the title Turning ACTS OFF by default Fixes in ACTS dependent analyzers Mar 28, 2023
@kjvbrt
Copy link
Contributor Author

kjvbrt commented Mar 28, 2023

For now, lets keep the AUTO option

@BrieucF BrieucF merged commit d88f938 into HEP-FCC:master Mar 28, 2023
@kjvbrt kjvbrt deleted the acts branch March 30, 2023 13:35
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.

None yet

3 participants