Skip to content

Add a test for $SystemMemory, MemoryAvailable[]...#1351

Merged
rocky merged 1 commit intomasterfrom
add-memory-test
May 3, 2021
Merged

Add a test for $SystemMemory, MemoryAvailable[]...#1351
rocky merged 1 commit intomasterfrom
add-memory-test

Conversation

@rocky
Copy link
Copy Markdown
Member

@rocky rocky commented May 3, 2021

and MemoryInUse[]

P.S. I think MemoryInUse is not quite right.

and MemoryInUse[]

P.S. I think MemoryInUse is not quite right.
@rocky rocky requested a review from mmatera May 3, 2021 11:47
Comment thread mathics/builtin/system.py

The relationship between $SystemMemory, MemoryAvailable, and MemoryInUse:
>> $SystemMemory > MemoryAvailable[] > MemoryInUse[]
= True
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I suppose they could be equal if all are 0, but that is unthinkable.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I like this test!

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Great - then let's ship.

@rocky rocky merged commit 1145dd1 into master May 3, 2021
@rocky rocky deleted the add-memory-test branch May 3, 2021 12:12
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

Successfully merging this pull request may close these issues.

2 participants