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

Change mode to perm in analysis view #742

Merged
merged 8 commits into from
Jan 29, 2023

Conversation

egbicker
Copy link
Contributor

@egbicker egbicker commented Jan 17, 2023

Change the 'mode' column to 'perm' in the analysis view.

Also includes updates to the analysis backend to generate the perm string from the logged event.

Closes #701

@egbicker egbicker linked an issue Jan 17, 2023 that may be closed by this pull request
@jw3
Copy link
Member

jw3 commented Jan 23, 2023

@egbicker I wired up the backend

image

@jw3
Copy link
Member

jw3 commented Jan 23, 2023

Ignore the RPM failures, #745

@egbicker
Copy link
Contributor Author

@jw3 Cleaned up the pytests. Do you want the single word displayed or all the options with the selected one marked up?

@jw3
Copy link
Member

jw3 commented Jan 23, 2023

Single word is good.

@jw3 jw3 changed the title Analysis object mode to perm update Change mode to perm in analysis view Jan 29, 2023
@jw3 jw3 merged commit 86818a7 into ctc-oss:master Jan 29, 2023
This was referenced Mar 16, 2023
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.

Review the mode column in the analysis view
2 participants