"MPEGMode is not defined"

Question

"MPEGMode is not defined"

DonLifeScore opened this issue 3 years ago · 21 comments

Hi.

I'm hitting an error in the new update (1.2.1 vs. 1.2.0):

Uncaught (in promise) ReferenceError: MPEGMode is not defined
at Lame.js:135
at e.lame_init (Lame.js:229)
at new e.exports.Mp3Encoder (index.js:100)
at export-audio.js:80

Where I'm call this:

const mp3_bitRateKbps = 256;
const mp3_sampleRate = 44100;
const mp3encoder = new lamejs.Mp3Encoder(2, mp3_sampleRate, mp3_bitRateKbps);

-- Don Tillman

Selithrarion commented 3 years ago

still

Answer 1 · 2021-12-13T13:56:10.000Z

Hi Don, same here! The previous fix unfortunately caused some issues with modules relying on implicitly & improperly loaded submodules. The following changes fix at least my pipeline, I hope yours as well. Until there is a new version out, you need to edit the respective files in your node_modules/lamejs folder. After the filename I give the line number to insert the requirement.

Lame.js:29
Encoder.js:112
PsyModel.js:162
all need var MPEGMode = require('./MPEGMode.js');

BitStream.js:33
Presets.js:18
both need var Lame = require('./Lame.js');

QuantizePVT.js:54
needs var BitStream = require('./BitStream.js');

Answer 2 · 2021-12-19T22:06:08.000Z

Also having the same issue with sveltekit, waiting for PR to be merged and new version deployed

Answer 3 · 2022-01-06T01:22:26.000Z

Any update on this conversation guys ? I have the same issue

Answer 4 · 2022-03-01T10:08:03.000Z

I'm still having to make these manual changes despite using 1.2.1.

Answer 5 · 2022-03-03T21:20:29.000Z

same issue too

Answer 6 · 2022-03-10T22:27:03.000Z

I was using this approach to linking with unreleased deps for a while and it was working, but at the moment for me neither that approach nor direct linking to 1.2.1 works. This issue shouldn't have been closed yet; we need a working release thanks!

(BTW, I found that unlinking on both ends and re-linking did the trick, but a new release would be so much easier)

Answer 7 · 2022-04-03T18:25:58.000Z

I have the same issue still. Why is it closed?

Answer 8 · 2022-04-08T14:02:07.000Z

Try audio-encoder npm . works.

Answer 9 · 2022-04-09T08:20:50.000Z

Try audio-encoder npm . works.

it seems audio-encoder uses lamejs and i have the same err ;(
just copied their example to my app
Selithrarion/vue_audio-tools@391fd2f

Answer 10 · 2022-04-09T12:05:04.000Z

Ok sorry. I must have done the manual fix - which is pretty easy to do. Tom Bancroft +447974 983701 Sent from my iPhone

…

On 9 Apr 2022, at 09:21, Sergey ***@***.***> wrote: Try audio-encoder npm . works. it seems audio-encoder uses lamejs and i have the same err ;( just copied their example to my app ***@***.*** — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.

Answer 11 · 2022-06-04T10:52:36.000Z

It's pretty dirty, but this repo fixed it for me if you don't want to manually each time:
https://www.npmjs.com/package/lamejstmp

It's dirty as hell of course, I hope this gets resolved.

Also, this repo might do what most people want with less boilerplate, it has lamejs bundled into it and doesn't suffer from this issue:
https://github.com/closeio/mic-recorder-to-mp3

Answer 12 · 2022-12-28T15:26:05.000Z

Is there something I'm missing about this? The issue has been closed for a while, but I am still experiencing the same problem. This fix works, when can it be expected to be included in production?

Hi Don, same here! The previous fix unfortunately caused some issues with modules relying on implicitly & improperly loaded submodules. The following changes fix at least my pipeline, I hope yours as well. Until there is a new version out, you need to edit the respective files in your node_modules/lamejs folder. After the filename I give the line number to insert the requirement.

Lame.js:29 Encoder.js:112 PsyModel.js:162 all need var MPEGMode = require('./MPEGMode.js');

BitStream.js:33 Presets.js:18 both need var Lame = require('./Lame.js');

QuantizePVT.js:54 needs var BitStream = require('./BitStream.js');

Answer 13 · 2022-12-30T21:07:06.000Z

The issue is fixed in the latest release on Github, just not on NPM apparently. You can use this as a workaround in your package.json:

"dependencies": {
    "lamejs": "github:zhuker/lamejs"
}

Answer 14 · 2023-01-09T12:16:45.000Z

The issue is fixed in the latest release on Github, just not on NPM apparently. You can use this as a workaround in your package.json:
"dependencies": {
    "lamejs": "github:zhuker/lamejs"
}

Great! Worked for me.

Answer 15 · 2023-01-10T17:24:32.000Z

@viniciusgusmao it doesn't work for me :(

Answer 16 · 2023-01-10T22:57:21.000Z

@viniciusgusmao it doesn't work for me :(

could be you're using mic-recorder like i was and this one carries a 3 year old version of lamejs which you will have to manually override, even with this solve from @jgentes
at least i did, although it somewhat contradicts my understanding of it having lamejs as a dependency, here's sharing hope you can use it to solve your issue too

Answer 17 · 2023-01-11T01:45:24.000Z

@TVNielsen Im using lamejs directly

Answer 18 · 2023-01-11T07:49:50.000Z

i was too, but also using another module which also was and it was that module that didn't pick up the direct github reference or in any effect ended up having an internal version of lamejs which was at least not the one with the updated code for the example above direct. So in my case it was the same file which would error in the lamejs version without the direct github link or well at least another than this, and finding that folder and replacing it with the suggested one pulled from git also, did solve the problem. Seems it is some change to parent scope variables which was impacting the library. Den ons. 11. jan. 2023 kl. 02.45 skrev Khoa Le ***@***.***>:

…

@TVNielsen <https://github.com/TVNielsen> Im using lamejs directly — Reply to this email directly, view it on GitHub <#86 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABUHDKGCZ56BPKWEEIUUSJLWRYGD5ANCNFSM5JSNTU4A> . You are receiving this because you were mentioned.Message ID: ***@***.***>

-- Med venlig hilsen, *Thomas V. Nielsen* email: ***@***.*** Telefon: 24 60 88 56

Answer 19 · 2023-07-10T15:28:43.000Z

Declaring global variables can help:

import MPEGMode from 'lamejs/src/js/MPEGMode';
import Lame from 'lamejs/src/js/Lame';
import BitStream from 'lamejs/src/js/BitStream';


window.MPEGMode = MPEGMode;
window.Lame = Lame;
window.BitStream = BitStream;

Answer 20 · 2023-11-07T23:46:54.000Z

@viniciusgusmao it doesn't work for me :(

have you done npm install after changing package.json ?