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

When output folder doesn't exist, error shows in UI #846

Closed
yezr opened this issue Jun 14, 2024 · 1 comment · Fixed by #847
Closed

When output folder doesn't exist, error shows in UI #846

yezr opened this issue Jun 14, 2024 · 1 comment · Fixed by #847

Comments

@yezr
Copy link
Collaborator

yezr commented Jun 14, 2024

RCTab Version: 1.3.999

What Happened

I was doing some testing and set the output folder to a folder that didn't yet exist ~/Downloads/vmwgoutput-with_one_undervote. Usually, RCTab creates the output folder for me if it doesn't exist. During this round of testing, however, when calculating ballot numbers I saw an error in the log when writing the rctab_cvr saying that the file didn't exist (I think the real issue was the folder didn't exist). I was able to continue on and tabulate. Tabulate worked successfully, correctly created the new folder, and successfully wrote the rctab_cvr.
image

I confirmed later by configuring to a folder that does exist that the rctab_cvr is being written twice if the configured output folder exists: once during ballot counting and once during tabulation.

What I Would Expect

I would expect RCTab not to attempt to write a rctab_cvr during the ballot counting step. If it does, I would expect the same logic that creates the folder during tabulation to create the folder during ballot counts.

@yezr yezr added this to RCTab v2.0 Jun 14, 2024
@yezr yezr moved this to Todo in RCTab v2.0 Jun 14, 2024
@artoonie
Copy link
Collaborator

I will update to not write an rctab_cvr. In the process, I've noted that logs that get ouput to the GUI window during "Check Ballot Counts" are not written to an audit log. Should they be? Perhaps in a separate file? I think I prefer that they are.

Here's a demo: the audit log is separated into "check ballot counts" and "tabulate" sections.

auditlogdemo.mov

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants