-
Notifications
You must be signed in to change notification settings - Fork 84
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
MOSES example moses-ann-pole1 segmentation fault #26
Comments
Here's the line where it occurs: |
The program says that it should be invoked as |
If I try running it with 6 command line arguments, then I receive an error
https://github.com/opencog/moses/blob/master/moses/comboreduct/combo/convert_ann_combo.h#L117 Any suggestions? |
I can't replicate it now (not in my computer right now), but let me try On Wed, Nov 11, 2015 at 1:48 AM, Cosmo Harrigan notifications@github.com
|
For this second, could you put the values that you tried?
|
Thanks @ArleyRistar. For example, I tried
After executing it, it freezes; and then, if you press enter, it displays that stack trace and terminates. |
It should have worked, these are basically default values. On Wed, Nov 11, 2015 at 2:09 AM, Cosmo Harrigan notifications@github.com
|
Sorry for the delay, my crap hotel internet isn't helping. On Wed, Nov 11, 2015 at 2:18 AM, Arley Ristar arley@ristar.me wrote:
|
If you guys start using the NN part, please by all means, add a unit test for it. The lack of unit test is the reason this code has gotten broken over time. |
@ngeiswei Indeed; perhaps it would make sense to use one of the existing NN examples as a unit test? |
@ArleyRistar The unit tests all pass. It would be great if you can try this out on your machine when you get back. |
Yes indeed there is some ANN unit test. Once you understand the problem you should complete the unit test to cover that case, or as @cosmoharrigan suggests create a unit test based on the example. |
Do these examples even make sense? It wasn't clear to me that the ANN examples were any good. Note also: now that the POS (particle-swarm-optimization) code is integrated, doesn't that change the motivation to use ANN's in moses? The examples should include some description as to why they are worthy. |
@linas In what sense? From an abstract point of view, the pole balancing/inverted pendulum problem is a classic benchmark in control theory. But, I don't know about the specifics of how these examples were implemented. |
I meant "do they use moses in a way that makes sense; i.e. do they actually --linas On Thu, Nov 12, 2015 at 12:58 AM, Cosmo Harrigan notifications@github.com
|
So, I replicated the "error" here. So, Linas or Nil, you can close this issue, and tell Cosmo an example of On Thu, Nov 12, 2015 at 11:23 AM, Linas Vepštas notifications@github.com
|
Thanks @ArleyRistar . Although before closing I think it would be better to fix this example first so that it detects the missing argument and return an informative error message. |
Issue #26: Update ANN examples to give a informative error message.
I receive a segmentation fault when I run the MOSES example program moses-ann-pole1 on Ubuntu 14.04.1 with GCC 4.8.4.
Preparation:
mkdir build
cd build ; cmake -DCMAKE_BUILD_TYPE=Debug .. ; make
make examples
cd ../examples/example-progs
Start gdb:
gdb moses-ann-pole1
Run the program:
(gdb) run 100 1
(gdb) backtrace
The text was updated successfully, but these errors were encountered: