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

upgraded cmakelist and package.xml, and setup.py for noetic #70

Merged

Conversation

jmastrangelo-cpr
Copy link
Contributor

I have tested this on ROS Noetic, for the following cameras:

  1. AXIS M3057-PLVE Fixed Dome Network Camera
  2. AXIS P5514-E Network Camera
  3. AXIS P5514-E PTZ Dome Network Camera (seemed to have a different web UI than the one above)

@civerachb-cpr civerachb-cpr merged commit 9d3208d into ros-drivers:noetic-devel Nov 26, 2021
@Timple
Copy link

Timple commented Jan 11, 2022

noetic-devel and master seem out of sync.

Isn't master supposed to be the latest and greatest?

@civerachb-cpr
Copy link
Collaborator

No, it isn't in this case. Each ROS distro has its own distinct *-devel branch. Because multiple ROS distros are active simultaneously there is no single "latest and greatest" -- melodic and kinetic are both currently active.

@Timple
Copy link

Timple commented Jan 11, 2022

Ah, alright.

I interpreted the README as such, noetic being the last release 🙂

The master branch normally contains code being tested for the next ROS release


Shameless plug because I'm not sure anyone is watching release repositories: ros-drivers-gbp/axis_camera-release#3 🙂

civerachb-cpr added a commit that referenced this pull request Mar 17, 2023
* Update the python files to be python-3 compliant.  Fix some bugs in the image data parsing needed as part of this update

* Revert "Remove README.rst; no point in maintaining 2 READMEs"

This reverts commit fd18a57.

* upgraded cmakelist and package.xml, and setup.py for noetic (#70)

* Update changelog ahead of release

* 0.4.0

* Fix the #! lines to use python3, decode the utf8 bytes into a string to suppress a warning when parsing the camera position

* Update the changelog ahead of release

* 0.4.1

* Explicitly use Python3 in shebang for teleop nodes; remove unnecessary shebang in setup.py (#72)

* Update changelog ahead of release

* 0.4.2

* Add frames-per-second (fps) as a configurable option

* Reverting package change as it happens automatically

* Update changelog ahead of release

* 0.4.3

* Start adding support for the Q6215 IR mode and wiper

* Fix a couple of copy/paste errors in the launch file, set the internal IR/wiper state

* IR on/off is now working with the demo unit on my desk

* Implement the wiper service. The wiper requires a fixed duration; it can't operate continuously. Run the wiper for 10s intervals

* Add support for the Q6215's defogger. Ensure that wiper, defog, IR modes are all disabled on startup

* Add a launch argument to expose the encrypted password option

* Add a timeout to the axis-cgi calls, make sure the HTTP response is 200 (otherwise raise an exception)

* Expand the readme with details on usage, supported devices, available topics & services

* Small code cleanup, remove the default password. Add support for basic auth if the encrypted password flag isn't set

* Add support for authentication to the PTZ node

* Update the readme

* Enable password encryption in view_axis.launch

* Don't bother loading the custom calibration in the view_axis test launch

* Add python3-requests as a dependendency

* Fix the pan direction in the readme

* Fix a bug where the PTZ brightness wasn't being reported correctly. Add support for the autoiris feature available on some cameras. Update the list of tested camera series.

---------

Co-authored-by: jmastrangelo-cpr <68664094+jmastrangelo-cpr@users.noreply.github.com>
Co-authored-by: Joey Yang <jyang@clearpathrobotics.com>
Co-authored-by: Jason Higgins <jhiggins@clearpathrobotics.com>
Co-authored-by: Tony Baltovski <tbaltovski@clearpathrobotics.com>
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