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

update to latest hessio library #51

Open
kosack opened this issue Oct 20, 2017 · 5 comments
Open

update to latest hessio library #51

kosack opened this issue Oct 20, 2017 · 5 comments

Comments

@kosack
Copy link
Contributor

kosack commented Oct 20, 2017

The latest hessioxxx library is needed for reading the latest sims, so we should update the version inside pyhessio.

@jacquemier
Copy link
Contributor

@kosack Could you please provide one of these lasted sim file that need the new hessio library for test purpose ?

@kosack
Copy link
Contributor Author

kosack commented Mar 8, 2018

That's a good question: I'll ask if Gernot can provide one - I think the prod3b ones are still readable, but the next production may need the updated library to access the new information stored in them (the current info should still be readable, since it is backward compatible).

At least you should test with a prod3b file from the grid. I plan on uploading on to our new shared webserver soon, but you can get one from the CTA-Grid web interface as well.

@GernotMaier
Copy link
Contributor

I am actually listening :-)

Do you just need a random file for testing? If yes, then I would try to find a small one. Do you have access to the GRID?

@jacquemier
Copy link
Contributor

Yes, I have access to the GRID.

@cbigo68
Copy link

cbigo68 commented Dec 27, 2018

I believe that hessio library should be updated once again. I'm trying to read a recently produced sim_telarray file with pyhessio and I'm getting the following error message:

Unsupported camera definition version: 5.
Skipping telescope event sub-item of type 2032 for telescope 1
Skipping telescope event sub-item of type 2032 for telescope 1
...

I traced this back to the io_hess.c routine at https://github.com/cta-observatory/pyhessio/blob/master/hessioxxx/src/io_hess.c

...
item_header.type = IO_TYPE_HESS_CAMSETTINGS; /* Data type */
if ( (rc = get_item_begin(iobuf,&item_header)) < 0 )
return rc;
if ( item_header.version > 4 )
{
fprintf(stderr,"Unsupported camera definition version: %d.\n",
item_header.version);
get_item_end(iobuf,&item_header);
return -1;
}
...

In a recent version of hessioxxx I found instead

item_header.type = IO_TYPE_HESS_CAMSETTINGS; /* Data type */
if ( (rc = get_item_begin(iobuf,&item_header)) < 0 )
return rc;
if ( item_header.version > 5 )
{
fprintf(stderr,"Unsupported camera definition version: %d.\n",
item_header.version);
get_item_end(iobuf,&item_header);
return -1;
}

I attached a file for testing purposes.

proton_20.0_180.0_alt1740.0_run010000.simtel.gz

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

4 participants