If an error causes the temporary file in the json backend to not get copied properly, it would previously remain in the directory. It has not been needed so it will now be removed if left behind.
In addition, because this file is no longer needed, the datetime and username are no longer needed in the file name. It is now just the unique hash to prevent collisions. Imports previously used to add time and username are now removed from the file.
How Has This Been Tested?
Interactively, and a new test in test-backends.py
Where Has This Been Documented?
Issues #303, #304
Comments within function itself and test.
Pre-merge checklist
[x] Code works interactively
[x] Code contains descriptive docstrings, including context and API
[x] New/changed functions and methods are covered in the test suite where possible
[x] Test suite passes locally
[x] Test suite passes on GitHub Actions
[x] Ran docs/pre-release-notes.sh and created a pre-release documentation page
[x] Pre-release docs include context, functional descriptions, and contributors as appropriate
Description
Motivation and Context
Closes #304
If an error causes the temporary file in the json backend to not get copied properly, it would previously remain in the directory. It has not been needed so it will now be removed if left behind.
In addition, because this file is no longer needed, the datetime and username are no longer needed in the file name. It is now just the unique hash to prevent collisions. Imports previously used to add time and username are now removed from the file.
How Has This Been Tested?
Interactively, and a new test in test-backends.py
Where Has This Been Documented?
Issues #303, #304
Comments within function itself and test.
Pre-merge checklist
docs/pre-release-notes.shand created a pre-release documentation page