Replies: 4 comments 1 reply
-
Have you compiled it with Clang? That makes a difference compared to GCC. Also -O3 and -Ofast makes a difference since, if I understand it correctly, it vectorizes more instructions. |
Beta Was this translation helpful? Give feedback.
-
I have -O3 and -ofast. And the code all vectorizes properly. Clang is
usually slightly slower than GCC; but that could account for the
difference. That would certainly explain why the profiling results look
weirdly different even though both plugins are running substantially the
same code.I'll give that a try. Thanks.
…On Tue, Sep 24, 2024, 03:17 38github ***@***.***> wrote:
TooB Neural Amp Modeler runs 5% slower than Mike Oliphant's Neural Amp
Modeler, despite running virtually identical code from Steven Atkins'
NeuralAmpModelerCore project. Investigation continues.
Have you compiled it with Clang? That makes a difference compared to GCC.
Also -O3 and -Ofast makes a difference since, if I understand it correctly,
it vectorizes more instructions.
—
Reply to this email directly, view it on GitHub
<#200 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACXK2DGUUS5XYGCZV5U4BS3ZYEGX7AVCNFSM6AAAAABOT55WNGVHI2DSMVQWIX3LMV43URDJONRXK43TNFXW4Q3PNVWWK3TUHMYTANZTGUYTCMQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
Was the 8% CPU gain from using Clang? If I remember correctly the LSTM models will gain even larger performance by compiling with -fopenmp. Also a tip if you use neural-amp-modeler-lv2 code is to edit the dcblock part because its high-pass effect is way to much. I change the 220.0 value to just 1.0 on row 291 in neural-amp-modeler-lv2/tree/main/src/nam_plugin.cpp |
Beta Was this translation helpful? Give feedback.
-
Clang didn't actually didn't give me a boost. It ended up being a very
subtle code tweak that I don't honestly fully understand.
Toob NAM uses a different DC blocker. I can't remember what the cutoff is,
but it's low. 10hz? (Note to self to check that it is). And the matrices
involved are probably not big enough to benefit from openmp.
Soooo close to being able to use two instances in a single patch on a Pi 4
:p
…On Wed, Sep 25, 2024, 14:29 38github ***@***.***> wrote:
Was the 8% CPU gain from using Clang? If I remember correctly the LSTM
models will gain even larger performance by compiling with -fopenmp. Also a
tip if you use neural-amp-modeler-lv2 code is to edit the dcblock part
because its high-pass effect is way to much. I change the 220.0 value to
just 1.0
—
Reply to this email directly, view it on GitHub
<#200 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACXK2DCAC46FLSK5C4XFY7LZYL6GZAVCNFSM6AAAAABOT55WNGVHI2DSMVQWIX3LMV43URDJONRXK43TNFXW4Q3PNVWWK3TUHMYTANZVGQ4TONQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
Click here to download PiPedal v1.2.51 Release
Important notice: Changes to the Auto-update code prevent previous versions from automatically updating to this version. You will have to install this version by downloading the debian package and installing it manually.
This release replaces Wi-Fi Direct connections with Wi-Fi hotspots. Support for Wi-Fi Direct on Linux and Android has been fragile for some time. An update to Raspberry Pi OS in early September broke Wi-Fi Direct support completely. As it turns out, Auto-hotspots work much better.
The new Auto-Hostpot feature in PiPedal allows you to configure your Raspberry Pi so it automatically starts a Wi-Fi hotspot whenever you are away from home. The updated Android PiPedal Client will automatically detect and connect to your Raspberry PI whenever it is visible on the current Wi-Fi network. An updated Android PiPedal Remote app has been posted on Google Play. Make sure you are using the updated version.
New MIDI system bindings allow you to enable or disable the Wi-Fi hotspot, and to shut down or reboot your Raspberry Pi using MIDI-triggered events.
NOTICE: PiPedal 1.2.47 fixed a significant and dangerous defect that may cause loss of presets, banks or configuration data if you remove power from your Raspberry Pi within up to five minutes of last saving data instead of performing an orderly shutdown or reboot. Users of PiPedal should upgrade to at least version 1.2.47 immediately, if they have not already done so.
Bug fixes:
Known issues:
Beta Was this translation helpful? Give feedback.
All reactions