-
Notifications
You must be signed in to change notification settings - Fork 46
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.
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
Issues building any of the stable models #189
Comments
Hi Ahmed, yes, As for There might be a couple more issues with ROSS-Models, it was archived after all right at the time when the changes above were being implemented. There are a couple of models that work out of the box with the current version of ROSS, if you wanna try them. They are meant to be very simple models too: https://github.com/helq/highlife-ross/ and https://github.com/sanjaychari/ROSS-coffeeshop. I hope that helps, don't forget to check out the FAQ |
Many thanks for this.
My interest in the torus model in particular, is that I'm researching
characteristics of this topology, and ROSS looked promising for a model
like that, out of the box - looking to fully understand latency and
throughput characteristics, versus network diameter, size and routing
protocol.
It still wouldn't compile after I made the changes you suggested: now
complaining about an implicit declaration of tw_getpe (arning: implicit
declaration of function ‘tw_getpe’; did you mean ‘tw_getkp’?
[-Wimplicit-function-declaration]
1056 | pe = tw_getpe(kpid % 1);)
The only models which did build successfully are phold, phold_test and
coffeeshop.
Thanks again,
Ahmed.
…On Fri, 15 Mar 2024 at 18:46, helq ***@***.***> wrote:
Hi Ahmed, yes, g_tw_npe was removed a while ago. Once upon a time, ROSS
supported multithreaded execution and MPI. Nowadays, it only supports
parallelization via MPI. That variable indicated how many PEs were spawn in
a specific MPI rank. That value is now 1. So, you can replace it for 1
anywhere you find it.
As for g_tw_pe, it was a pointer to an array of PEs. It is now just a
pointer. So, the brackets in tw_kp_onpe(kpid, g_tw_pe[0]) should be
removed: tw_kp_onpe(kpid, g_tw_pe).
There might be a couple more issues with ROSS-Models, it was archived
after all right at the time when the changes above were being implemented.
There are a couple of models that work out of the box with the current
version of ROSS, if you wanna try them. They are meant to be very simple
models too: https://github.com/helq/highlife-ross/ and
https://github.com/sanjaychari/ROSS-coffeeshop.
I hope that helps, don't forget to check out the FAQ
<https://ross-org.github.io/faq/usage-faq.html>
—
Reply to this email directly, view it on GitHub
<#189 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA47TA5ONHHPIG7LSLEN6UDYYM6ZZAVCNFSM6AAAAABEWONLNWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBQGI2TGNRRGM>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
If you wanna try a more complete, although unmaintained, torus network model, check out CODES. I suggest you run the script I hope that helps |
Hi,
Having issues building any of the published stable models. Tried the airport and torus models: same error (output of make attached). Seems that g_tw_npe has either been moved, redefined ...etc., since this error is consistent across all models.
Any help gratefully received.
Ahmed.
outfile.txt
The text was updated successfully, but these errors were encountered: