Skip to content
This repository was archived by the owner on Mar 24, 2026. It is now read-only.

some tuning and add setting on prerequisites.sh for real-time priority#1630

Merged
msmith-techempower merged 2 commits intoTechEmpower:masterfrom
stefanocasazza:master
Jun 3, 2015
Merged

some tuning and add setting on prerequisites.sh for real-time priority#1630
msmith-techempower merged 2 commits intoTechEmpower:masterfrom
stefanocasazza:master

Conversation

@stefanocasazza
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why add an entire new test for json when it was already included in the default?

EDIT: I see below with the setups; you are messing around with the values of UMEMPOOL per test type. This is really fine-tuned tweaking, but I guess that's fine.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hi,

in this way I am able to differentiate the execution context (plaintext has a
greater need for resources than json) so I can tuning better...

Greetings

2015-05-28 19:48 GMT+02:00 Mike Smith notifications@github.com:

In frameworks/C++/ULib/benchmark_config.json
#1630 (comment)
:

@@ -20,6 +19,24 @@
"notes": "",
"versus": ""
},

  • "json": {

Why add an entire new test for json when it was already included in the
default?


Reply to this email directly or view it on GitHub
https://github.com/TechEmpower/FrameworkBenchmarks/pull/1630/files#r31259137
.

@msmith-techempower
Copy link
Copy Markdown
Member

No one is responding to this >_< I will have to test this on a vanilla environment before merging.

@msmith-techempower
Copy link
Copy Markdown
Member

Since I have gotten no feedback and I tested this locally and it worked, I'm inclined to merge. LGTM!

msmith-techempower added a commit that referenced this pull request Jun 3, 2015
 some tuning and add setting on prerequisites.sh for real-time priority
@msmith-techempower msmith-techempower merged commit 919723d into TechEmpower:master Jun 3, 2015
@stefanocasazza
Copy link
Copy Markdown
Contributor Author

Hi,

thank you very much!

Greetings

2015-06-03 19:24 GMT+02:00 Mike Smith notifications@github.com:

Merged #1630
#1630.


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

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants