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

4.3.6.1 is not recording data #1005

Closed
Ezward opened this issue Apr 3, 2022 · 7 comments
Closed

4.3.6.1 is not recording data #1005

Ezward opened this issue Apr 3, 2022 · 7 comments

Comments

@Ezward
Copy link
Contributor

Ezward commented Apr 3, 2022

version: tag/4.3.6.1
RaspberryPi 4
Relevant config:

  • CAMERA_TYPE = "PICAM"
  • DRIVE_TRAIN_TYPE = "PWM_STEERING_THROTTLE"
  • CONTROLLER_TYPE = 'xbox'
  • JOYSTICK_MAX_THROTTLE = 1
  • AUTO_RECORD_ON_THROTTLE = True
  • JOYSTICK_DEADZONE = 0.1
  • RECORD_DURING_AI = False
  • AUTO_CREATE_NEW_TUB = False

Ran with a fresh install of 4.3.6.1. I had auto-save with throttle on; the configuration in myconfig.py that causes recording whenever the throttle is non-zero: AUTO_RECORD_ON_THROTTLE = True. I drove a number of laps to create new data. during that time I used the 'x' button to delete mistakes. The results was that it created the catalog and images folder, but collected no data. The catalog was zero bytes and the images folder was empty.

@Ezward Ezward changed the title 4.3.5.1 is not recording data 4.3.6.1 is not recording data Apr 3, 2022
@zlite
Copy link
Contributor

zlite commented Apr 4, 2022

I'm running 4.3.8 and it's saving the data correctly for me

@DocGarbanzo
Copy link
Contributor

DocGarbanzo commented Apr 4, 2022

Thanks @zlite - @Ezward, this is your #989 which fixes it then. That's the only 'real' difference between the two versions (i.e. HEAD of main and 4.3.6.1). I have now patched #989 into the release_4_3_6 branch. Can one of you check that this is working as expected? That would be great. I will then publish a new bug fix release 4.3.6.2 which will hopefully be stable for the next months.

@Ezward
Copy link
Contributor Author

Ezward commented Apr 5, 2022

This is actually a duplicate of another issue and that issue has a proposed fix #996 (comment)

@Ezward
Copy link
Contributor Author

Ezward commented Apr 5, 2022

I'm running 4.3.8 and it's saving the data correctly for me

@zlite What controller are you using when you test with 4.3.8?

@DocGarbanzo
Copy link
Contributor

DocGarbanzo commented Apr 5, 2022

@Ezward - do you suggest that I should use PR #1007 in the 4.3.6 branch instead of PR #989? I can do that.

@Ezward
Copy link
Contributor Author

Ezward commented Apr 7, 2022

The problem is addressed by this pull request #1007 which is now merged.

@Ezward Ezward closed this as completed Apr 7, 2022
@Ezward Ezward reopened this Apr 10, 2022
@Ezward
Copy link
Contributor Author

Ezward commented Apr 10, 2022

I'm reopening this issue. I've tested with 4.6.10 after a user complained on discord that this was still broken. They are correct, AUTO_RECORD_ON_THROTTLE is broken; it does not record when that is True and the game controller is used to drive; it creates the metadata files, but they are zero length and it creates the images folder, but is remains empty. Also toggling manual recording with the game controller when AUTO_RECORD_ON_THROTTLE is on is broken. I am able to toggle recording using the webui. I am unable to toggle recording using the game controller. It does get the button press, but it will not toggle the mode. someone had tested the fix and said it worked, but apparently that is not true. In addition, if I set AUTO_RECORD_ON_THROTTLE = False, the the game controller button says it turned on recording, but it can never toggle it off. Also in that condition, the game controller never records any data, but the webui will if you use the virtual joypad. So it is more than toggling the mode that is broken.

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

No branches or pull requests

3 participants