search

cxbrooks / test

Second test for bugzilla to git
0 stars 0 forks source link

Datagram actors fail their test #51

Open cxbrooks opened 17 years ago

cxbrooks commented 17 years ago

Note: the issue was created automatically with bugzilla2github tool

Original bug ID: BZ#54 From: @cxbrooks Reported version: 6.0.1 CC: pt-dev@chess.eecs.berkeley.edu

cxbrooks commented 17 years ago

Christopher Brooks wrote:

It seems like the datagram actors do not work very well.

At some point, I modified the test suite so that it would report an error if the Test.fire() was never called.

The datagram test in actor/lib/net/test/auto/DatagramReader.xml failed in this manner.

I then managed to forget about this problem. The tests in actor/lib/net/test were not running for awhile. Then in December, I reenabled the tests.

After some poking around, it seems like this test sometimes works in Ptolemy II 4.0.1, but other times I get no output.

The Datagram actors use threads and I've never really trusted that the implementation was any good.

Should we just remove these actors?

They do try to solve a problem we've never solved, that of asynchronous data read/write. The audio actors have a similar problem with codegen.

Edward A. Lee wrote:

I have no opinion, except that indeed these actors are representative of a broad unsolved problem. I would hate the lose them completely...

At this point, I'll leave the Datagram actors in place. I'll update the actors to mention that they might have problems.

There seems to be a good 199 project here, or possibly part of a masters that involves asynchrounous data read/write.