Skip to content
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

PnToneの初期化パラメータを変更したほうが良い。 #18

Open
nyatla opened this issue Jan 21, 2024 · 1 comment
Open

Comments

@nyatla
Copy link
Owner

nyatla commented Jan 21, 2024

PnToneはintervalとBaseToneを引数にもつが、intervalを引数とする場合にBaseToneが固定されている。
https://github.com/nyatla/TBSKmodem/blob/master/tbskmodem/kokolink/protocol/tbsk/toneblock.py

固定する場合のBaseToneは矩形波として、トーン長とintervalを指定する形式にしたほうが良い。

Platform Apply Status Commit
Python yes f3d19d5
C# yes
C++ yes
Java yes e181ce081cdf48d1635a088b9a6f7f76da3cdb31
Processing ?
Javascript ?
@nyatla
Copy link
Owner Author

nyatla commented Mar 17, 2024

以下の3種類のAPIに統一する。

@classmethod
@overload
def createPn(cls,seed:int,interval:int,base_tone:TraitTone)->PnTone:
    ...
@classmethod
@overload
def createPn(cls,seed:int,interval:int,length:int)->PnTone:
    """ seedのPN符号を使用して、interval単位でPN符号を適応した長さlengthのトーン信号を得る。
    """
    ...
@classmethod
@overload
def createPn(cls,seed:int,length:int)->PnTone:
    """ seedのPN符号を使用して、1tickごとにPN符号を適応した長さlengthのトーン信号を得る。
    """

seedの初期値は299とする。

nyatla added a commit that referenced this issue Mar 17, 2024
PnToneパラメータの変更
nyatla added a commit to nyatla/TBSKmodemJava that referenced this issue Mar 26, 2024
nyatla/TBSKmodem#19 に関する修正。 →IterableとIteratorをkokolink配下から除外。
・nyatla/TBSKmodem#18 に関する修正
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

No branches or pull requests

1 participant