discuss-gnuradio
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Discuss-gnuradio] Swig Trouble


From: Marcus Müller
Subject: Re: [Discuss-gnuradio] Swig Trouble
Date: Sat, 02 Jan 2016 18:01:33 +0100
User-agent: K-9 Mail for Android

Hi Jan,

Heh, I only spotted that only after scratching my head quite a lot, then git grepping through all the non-.cc, non-_impl, non-doc, non-example files for every line containing cc_decoder (but not pccc_decoder), then "sed '/s/.../viterbi_decoder/g' replacing cc_ with viterbi_ in those lines, and then looking for exact matches for every one of these modified strings. At some point I really might want to learn doing Perl correctly, because there's only so much sanity you can lose

Cheers,
Marcus

Am 2. Januar 2016 16:59:47 MEZ, schrieb "Jan Krämer" <address@hidden>:
Thanks Marcus,

I knew it was something stupid like that. Probably would have taken ages for me to spot that!

Cheers,
Jan

2016-01-02 15:25 GMT+01:00 Marcus Müller <address@hidden>:
Hey Jan,

> I need help with the swig magic.
Too bad, full moon just passed and we ran all out of black cats to
sacrifice...
> I have a new block inside gr-fec called
> viterbi_decoder which is basically just a copy of the cc_decoder class.
> And a impl class viterbi_decoder_impl class, which is basically a copy
> of the cc_decoder_impl class. They compile fine, and I can see the
> symbols exported in the libgnuradio-fec.so.
>
> I also put the viterbi_decoder.h into the fec_swig.i. Still I get the
> dreaded
> AttributeError: 'module' object has no attribute 'viterbi_decoder'
>
> Did I miss something?
> Code can be found at
> https://github.com/SpectreJan/gnuradio/tree/viterbi_decoder
>
> new files are
> include/viterbi_decoder.h
> lib/viterbi_decoder_impl.cc
> lib/viterbi_decoder_impl.h
update the copyright year ;)

Regarding the problem: I saw the same, but if you just change L85 in
swig/fec_swig.i:

 %include "gnuradio/fec/cc_encoder.h"
-#include "gnuradio/fec/viterbi_decoder.h"
+%include "gnuradio/fec/viterbi_decoder.h"
 %include "gnuradio/fec/ccsds_encoder.h"

you should be one step further.

Cheers,
Marcus


PS: no cats were harmed during the making of this FOSS project, afaik.

_______________________________________________
Discuss-gnuradio mailing list
address@hidden
https://lists.gnu.org/mailman/listinfo/discuss-gnuradio


--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
reply via email to

[Prev in Thread] Current Thread [Next in Thread]