-
Notifications
You must be signed in to change notification settings - Fork 4
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
PDL::NetCDF #6
Comments
PDL does indeed support |
@mohawk2 the version that I am seeing on NetCDF.pm is 4.20. Ah, but the PDL version is 2.020. |
@cbrownstevenson That suggests your installed version of PDL doesn't have |
Hi all: Yes, we have been living with this limitation for a long time
and have worked around it. Ours is a large production organization with
many dozens of compute clusters. It takes a long while to upgrade, and
there is little motivation to spend lots of programmer and sysadmin time
to make a change which may result in down time for our customers.
We will upgrade (I would say) within the next year or so.
Regards,
Doug
NB: cosmicops: There is desire to force an upgrade in PDL to allow
PDL::NetCDF to have a signed byte type, something which our version of
PDL does not support,
but more recent versions do.
…On 10/8/22 10:47 AM, mohawk2 wrote:
@cbrownstevenson <https://github.com/cbrownstevenson> That suggests
your installed version of PDL doesn't have |sbyte|, which means you'd
have to upgrade your PDL to get the benefit of such a change?
—
Reply to this email directly, view it on GitHub
<#6 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABVLHCWEYHVHLSQTBQ7FAW3WCGQQJANCNFSM6AAAAAAQ7O3C6U>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
If it helps, PDL 2.080 looks very solid on CPAN Testers (https://www.cpantesters.org/distro/P/PDL.html?oncpan=1&distmat=1&version=2.080&grade=2) with the exception of BSD, which has older and buggy versions of clang, which PDL now triggers those bugs and crashes the compiler. With GCC, or more recent clang, it seems to work well. Therefore, it appears to me (subject to proper testing for your application), that upgrading to PDL 2.080 should not cause failures or downtime. |
@doughunt What's the situation with this? |
In the PDL::NetCDF module, in typemap3. Unsigned PDL::byte are being mapped to NC_BYTE () - signed. Shouldn't be NC_UBYTE().
PDL's signed byte type is sbyte
NetCDF.pm line 930
The text was updated successfully, but these errors were encountered: