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

In the sar-c_safe reader, add platform_name to the attribute. #996

Closed
TAlonglong opened this issue Dec 4, 2019 · 1 comment · Fixed by #998
Closed

In the sar-c_safe reader, add platform_name to the attribute. #996

TAlonglong opened this issue Dec 4, 2019 · 1 comment · Fixed by #998
Labels
component:readers enhancement code enhancements, features, improvements

Comments

@TAlonglong
Copy link
Collaborator

I would like to have the platform_name in the attributes in this reader.

The filename_info has the mission_id. This can be translated to platform_name like:

In the init of the SAFGRD:
self._mission_id = filename_info['mission_id']

and in get_dataset something like:
data.attrs.update({'platform_name': self._mission_id})

@mraspaud
Copy link
Member

mraspaud commented Dec 4, 2019

Good idea! PR welcome :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:readers enhancement code enhancements, features, improvements
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants