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

[jsk_image_capture] add jsk_image_capture pacakge #1705

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

sktometometo
Copy link
Contributor

@sktometometo sktometometo commented Nov 9, 2021

Add jsk_image_capture package.

TODO

  • Add README.md
  • Add eusliep client example
  • Add Install section.

@k-okada
Copy link
Member

k-okada commented Nov 9, 2021

@sktometometo
Copy link
Contributor Author

@k-okada I know that node. But I need an interface to save an image with a specified file name. That node does not provide it.

@k-okada
Copy link
Member

k-okada commented Nov 9, 2021 via email

@knorth55
Copy link
Member

knorth55 commented Nov 9, 2021

i think the problem of image_saver is that it cannot dynamically change the filename.
the rosparameter is only loaded at the beginning.
I don't know which we should improve image_saver or make a new node is better.

@k-okada
Copy link
Member

k-okada commented Nov 9, 2021

If so, please consider adding that feature to https://github.com/jsk-ros-pkg/jsk_common/tree/master/jsk_ros_patch/image_view2
adding new feature to existing packages have many advantages;

  • existing maintainer will also take care of your new feature
  • your feature already have uses/system, so you have more chance to propagate your idea
    Thus, even if you loose interest on that feature, someone will keep maintain it and will use them in their system. So you can put that results within your thesis and you can claim that your developed system is so useful/strong that people keep using them without original developers help.

@sktometometo
Copy link
Contributor Author

Moved scripts and service files to image_view2.

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

Successfully merging this pull request may close these issues.

3 participants