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

Exception during raster import with miss formated GEN files #17

Open
ghost opened this issue Jan 27, 2016 · 5 comments
Open

Exception during raster import with miss formated GEN files #17

ghost opened this issue Jan 27, 2016 · 5 comments

Comments

@ghost
Copy link

ghost commented Jan 27, 2016

Sometimes, in GEN files linked with USR/ASRP files, the data size is not defined. Furthermore, the last field into the header (TXT=) can be absent. The results are some exception due to a size of 0 and due the absence of the TXT filed.
gen_file_bug_report

@ghost
Copy link
Author

ghost commented Jan 27, 2016

A new pull request had been done.

@dfdietrick
Copy link
Contributor

Hi Tioben,

Thanks for the pull request, I have your changes and I’m in the process of checking them out. I appreciate you taking the time to send in a update.

Thanks again,

Don

On Jan 27, 2016, at 9:48 AM, Tioben Neenot [email protected] wrote:

A new pull request had been done.


Reply to this email directly or view it on GitHub #17 (comment).

@ghost
Copy link
Author

ghost commented Feb 1, 2016

Hi Don,
OK about the update, but how can I do that ? I tried  the last week and I've got some errors like "no sufficient rights" !
Le 1 févr. 2016 15:58, Don Dietrick [email protected] a écrit :Hi Tioben,

Thanks for the pull request, I have your changes and I’m in the process of checking them out. I appreciate you taking the time to send in a update.

Thanks again,

Don

On Jan 27, 2016, at 9:48 AM, Tioben Neenot [email protected] wrote:

A new pull request had been done.


Reply to this email directly or view it on GitHub #17 (comment).

—Reply to this email directly or view it on GitHub.

@dfdietrick
Copy link
Contributor

I just downloaded the linked .zip file you attached to the pull request. There seemed to be a bunch of other changes in the pull request that were previous updates to OpenMap, I ignored those. In the future, you can specify which changes to include in the pull request and git will handle them for me (and you).

But that’s OK, I can work with almost anything, patch files, zipped changes, etc.

Thanks,

Don

On Feb 1, 2016, at 10:11 AM, Tioben Neenot [email protected] wrote:

Hi Don,
OK about the update, but how can I do that ? I tried the last week and I've got some errors like "no sufficient rights" !
Le 1 févr. 2016 15:58, Don Dietrick [email protected] a écrit :Hi Tioben,

Thanks for the pull request, I have your changes and I’m in the process of checking them out. I appreciate you taking the time to send in a update.

Thanks again,

Don

On Jan 27, 2016, at 9:48 AM, Tioben Neenot [email protected] wrote:

A new pull request had been done.


Reply to this email directly or view it on GitHub #17 (comment).

—Reply to this email directly or view it on GitHub.

Reply to this email directly or view it on GitHub #17 (comment).

@FredPraca
Copy link
Contributor

Hello Don and Tneenot,
I just created a pull request containing only the patch from Tneenot which solves the problem.
I think it will be easier to include :)

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

2 participants