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.

Sign up for GitHub

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

Jump to bottom

arpack wrappers should use Int32 for boolean #167

Closed
JeffBezanson opened this issue Aug 4, 2011 · 2 comments
Closed

arpack wrappers should use Int32 for boolean #167

JeffBezanson opened this issue Aug 4, 2011 · 2 comments
Assignees
@ViralBShah
Labels
bug Indicates an unexpected problem or unintended behavior

Comments

@JeffBezanson
Copy link
Member

The native code expects bools to be 32 bits wide, and it writes to these arrays, so we are getting buffer overruns. Prevents tests from completing, unless you are "lucky".
@ghost ghost assigned ViralBShah Aug 4, 2011
@ViralBShah
Copy link
Member

That's odd, because I never had the tests fail for this reason. Will fix.

@JeffBezanson
Copy link
Member Author

Yes, to clarify, tests.j doesn't fail but doing things after like loading perf.j would segfault.

StefanKarpinski pushed a commit that referenced this issue Feb 8, 2018
@tkelman
Merge pull request #167 from MichaelHatherly/mh/delete-duplicates
a6184ae 
Remove duplicated definitions
KristofferC added a commit that referenced this issue Mar 6, 2018
@KristofferC
allow package to load themselves when building themselves... yeah (#167)
f62de07 
(cherry picked from commit 13ff205413c6e34222120c28502bc5e693715254)
cmcaine pushed a commit to cmcaine/julia that referenced this issue Sep 24, 2020
@SaschaMann
Merge pull request JuliaLang#167 from exercism/ci-updates
cc74a0f 
Upgrade CI setup
LilithHafner pushed a commit to LilithHafner/julia that referenced this issue Oct 11, 2021
@andreasnoack
Merge pull request JuliaLang#167 from JuliaStats/anj/05
6c8a667 
Use String instead of ByteString
Keno pushed a commit that referenced this issue Oct 9, 2023
@davidanthoff @KristofferC
Update README (#167)
ad91dd9 
* Update README

* remove some more stuff
ViralBShah pushed a commit that referenced this issue Dec 16, 2024
@DilumAluthgeBot @DilumAluthge
🤖 [master] Bump the Statistics stdlib from 68869af to d49c2bf (#56831)
dafaa61 
Stdlib: Statistics
URL: https://github.com/JuliaStats/Statistics.jl.git
Stdlib branch: master
Julia branch: master
Old commit: 68869af
New commit: d49c2bf
Julia version: 1.12.0-DEV
Statistics version: 1.11.2(Does not match)
Bump invoked by: @DilumAluthge
Powered by:
[BumpStdlibs.jl](https://github.com/JuliaLang/BumpStdlibs.jl)

Diff:
JuliaStats/Statistics.jl@68869af...d49c2bf

```
$ git log --oneline 68869af..d49c2bf
d49c2bf Merge pull request #178 from JuliaStats/dw/ci
d10d6a3 Update Project.toml
1b67c17 Merge pull request #168 from JuliaStats/andreasnoack-patch-2
c3721ed Add a coverage badge
8086523 Test earliest supported Julia version and prereleases
12a1976 Update codecov in ci.yml
2caf0eb Merge pull request #177 from JuliaStats/ViralBShah-patch-1
33e6e8b Update ci.yml to use julia-actions/cache
a399c19 Merge pull request #176 from JuliaStats/dependabot/github_actions/julia-actions/setup-julia-2
6b8d58a Merge branch 'master' into dependabot/github_actions/julia-actions/setup-julia-2
c2fb201 Merge pull request #175 from JuliaStats/dependabot/github_actions/actions/cache-4
8f808e4 Merge pull request #174 from JuliaStats/dependabot/github_actions/codecov/codecov-action-4
7f82133 Merge pull request #173 from JuliaStats/dependabot/github_actions/actions/checkout-4
046fb6f Update ci.yml
c0fc336 Bump julia-actions/setup-julia from 1 to 2
a95a57a Bump actions/cache from 1 to 4
b675501 Bump codecov/codecov-action from 1 to 4
0088c49 Bump actions/checkout from 2 to 4
ad95c08 Create dependabot.yml
40275e2 Merge pull request #167 from JuliaStats/andreasnoack-patch-1
fa5592a Merge pull request #170 from mbauman/patch-1
cf57562 Add more tests of mean and median of ranges
128dc11 Merge pull request #169 from stevengj/patch-1
48d7a02 docfix: abs2, not ^2
2ac5bec correct std docs: sqrt is elementwise
39f6332 Merge pull request #96 from josemanuel22/mean_may_return_incorrect_results
db3682b Merge branch 'master' into mean_may_return_incorrect_results
9e96507 Update src/Statistics.jl
58e5986 Test prereleases
6e76739 Implement one-argument cov2cor!
b8fee00 Stop testing on nightly
9addbb8 Merge pull request #162 from caleb-allen/patch-1
6e3d223 Merge pull request #164 from aplavin/patch-1
71ebe28 Merge pull request #166 from JuliaStats/dw/cov_cor_optimization
517afa6 add tests
aa0f549 Optimize `cov` and `cor` with identical arguments
cc11ea9 propagate NaN value in median
cf7040f Use non-mobile Wikipedia urls
547bf4d adding docu to mean! explain target should not alias with the source
296650a adding docu to mean! explain target should not alias with the source
```

Co-authored-by: Dilum Aluthge <dilum@aluthge.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ViralBShah ViralBShah

Labels
bug Indicates an unexpected problem or unintended behavior
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ViralBShah @JeffBezanson