Skip to content

version 0.0.1 is uploaded#1

Open
ma5oud180 wants to merge 11 commits intobig-data-lab-team:masterfrom
ma5oud180:master
Open

version 0.0.1 is uploaded#1
ma5oud180 wants to merge 11 commits intobig-data-lab-team:masterfrom
ma5oud180:master

Conversation

@ma5oud180
Copy link

No description provided.

Copy link
Contributor

@glatard glatard left a comment

Choose a reason for hiding this comment

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

Nice to see this getting started! See my comments below. This can't be called "version 0.0.1" until the code is released.

## Prerequisites

* [SimGrid 3.17](http://simgrid.gforge.inria.fr)
* C
Copy link
Contributor

Choose a reason for hiding this comment

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

C is not really a pre-requisite.

using similar processes as described in (Iturria-Medina et al. 2014). We leverage the SimGrid
(Casanova et al. 2013) toolkit, the most advanced simulator for distributed systems, to simulate
such a system. We compare our discrete-event simulation to the simulator in (Iturria-Medina et
al. 2014).
Copy link
Contributor

Choose a reason for hiding this comment

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

Add links to the papers.


* You could also use the SimGrid docker container and get your code into the container, by typing the following in a terminal:

`sudo docker run --rm -it -v /path/to/local/directory:/SimBrain path/to/container/directory /bin/bash`
Copy link
Contributor

Choose a reason for hiding this comment

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

Container name (bigdatalabteam/simgriddocker) is missing here.


* Under Debian, Ubuntu and similar systems, you can simply get the latest version of SimGrid from your distribution. For example, simply type the following in a terminal (as root):

`apt-get install simgrid`
Copy link
Contributor

Choose a reason for hiding this comment

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

This won't work when SimGrid 3.18 is released. I suggest to support only the container-based installation for now.


1. List files (ls)
2. Change directory to (cd) SimBrain
3. List files. Then, you will see the following files inside of it: `simbrain.c simbrain.xml simbrain_d.xml`
Copy link
Contributor

Choose a reason for hiding this comment

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

Step 1, 2 and 3 are not needed. Just say, in the SimBrain directory, compile the code as follows:


XBT_INFO("Task received : %s", in_task->name);
xbt_free(in_task->data);
MSG_task_destroy(in_task);
Copy link
Contributor

Choose a reason for hiding this comment

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

the previous 3 lines are not well indented.


XBT_LOG_NEW_DEFAULT_CATEGORY(SimBrain, "Messages specific for this msg example");

static int output(int argc, char *argv[])
Copy link
Contributor

Choose a reason for hiding this comment

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

add a comment to explain what this function does.

xbt_free(in_task->data);
MSG_task_destroy(in_task);

sleep(1);
Copy link
Contributor

Choose a reason for hiding this comment

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

Same comment on sleep as before.

MSG_init(&argc, argv);

xbt_assert(argc > 2, "Usage: %s platform_file deployment_file\n"
"\tExample: %s ../../platforms/small_platform.xml app-pingpong_d.xml\n", argv[0], argv[0]);
Copy link
Contributor

Choose a reason for hiding this comment

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

update example paths.

<platform version="4.1">
<zone id="zone0" routing="Full">
<host id="P0" speed="98.095Mf"/>
<host id="H0" speed="76.296Mf"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

We could use the speed variable as the production/clearance rate. That is, model production/clearance as computing tasks (let's discuss this).

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