Skip to content

Updating build to work on generic linux (e.g. arch)#28

Draft
thomas-gale wants to merge 6 commits intomainfrom
tgale/generic-linux
Draft

Updating build to work on generic linux (e.g. arch)#28
thomas-gale wants to merge 6 commits intomainfrom
tgale/generic-linux

Conversation

@thomas-gale
Copy link
Owner

  • Will need to rethink package installation stuff - maybe just state pre-reqs and let users install manually.
  • Updating max to latest nightly
  • Fixing a bunch of errors/warning - inout deprecated to mut etc.

alias dim = 3

fn random_float(mut rng : Rng, min: Scalar[T] = 0, max: Scalar[T] = 1.0) -> Scalar[T]:
fn random_Float64(mut rng : Rng, min: Scalar[T] = 0, max: Scalar[T] = 1.0) -> Scalar[T]:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was this picked up in a autoconvert?

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, this looks wrong, good catch.

@thomas-gale
Copy link
Owner Author

Does testing appear broken to you @robertgoss? I'm thinking latest nightly is bad.

from testing import assert_true

fn test_pass() raises:
	assert_true(True)

Error:

fatal error: test execution generated an unexpected number of results

(Getting this error for all tests)

@robertgoss
Copy link
Collaborator

Does testing appear broken to you @robertgoss? I'm thinking latest nightly is bad.

I will have a go at doing a local build now

@robertgoss
Copy link
Collaborator

Does testing appear broken to you @robertgoss? I'm thinking latest nightly is bad.

I will have a go at doing a local build now

So on my computer I get:

Testing Time: 1.292s

Total Discovered Tests: 46

Passed : 46 (100.00%)

I did need to update magic with magic self-update before ti would build though.

@thomas-gale
Copy link
Owner Author

I did need to update magic with magic self-update before ti would build though.

Ok confusing! Thanks for sanity check, I'll try again on my arch box, maybe I'm missing some package.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants