quran / quran_android

a quran reading application for android
http://android.quran.com
GNU General Public License v3.0
2.01k stars 890 forks source link

[Surah Al-Aḥzāb: 32] AbdulMuhsim al Qasim Recitation Incorrectly mapped to verse #2247

Open slimatic opened 1 year ago

slimatic commented 1 year ago

ٱلنَّبِیِّ لَسۡتُنَّ كَأَحَدࣲ مِّنَ ٱلنِّسَاۤءِ إِنِ ٱتَّقَیۡتُنَّۚ فَلَا تَخۡضَعۡنَ بِٱلۡقَوۡلِ فَیَطۡمَعَ ٱلَّذِی فِی قَلۡبِهِۦ مَرَضࣱ وَقُلۡنَ قَوۡلࣰا مَّعۡرُوفࣰا } Surah Al-Aḥzāb: 32

For AbdulMuhsim al Qasim (Gapless), the audio recitation is incorrectly mapped to verses 28 through 32 instead of simply mapping to verse 32.

Note: I'll be happy to help implement this change if someone is willing to point me to the code that handles this and if its an easy enough fix.

ahmedre commented 1 year ago

jazakumAllah khairan - I guess in this case the sheikh is repeating the set of ayat, so I think this is actually part of the mp3 in addition to the database.

you can download the database from here.

ahmedre commented 1 year ago

the audio file is here.

slimatic commented 1 year ago

Wa jazakallahu khairan,

So I took a look at the database and found the timings and saw something like this:

33|1|4474 33|2|16343 33|3|24985 33|4|30366 33|5|56458 ... 33|32|512391 33|33|603015 33|34|629199 33|35|642250 33|36|686216 33|37|709485

The audio file is about 22:27 long. Any idea what format is used to mark the timings?

Ideally, I want to analyze the file to see the feasibility of removing that extended repeated portion and accurately re-organizing the timings.

Let me know if this is worth looking into.

On Sat, Apr 15, 2023 at 7:20 AM Ahmed El-Helw @.***> wrote:

the audio file is here https://download.quranicaudio.com/quran/abdul_muhsin_alqasim/024.mp3.

— Reply to this email directly, view it on GitHub https://github.com/quran/quran_android/issues/2247#issuecomment-1509735273, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACN27XSYAX2K6O5NVNO373XBKAAVANCNFSM6AAAAAAW7K4GYM . You are receiving this because you authored the thread.Message ID: @.***>

slimatic commented 1 year ago

I figured out the timings. I have the correct audio file here https://download.quranicaudio.com/quran/abdul_muhsin_alqasim/033.mp3 and it maps to the timestamp accurately. This will involve editing the audio file and adjusting the timestamps from verse 32 to 73.

On Sat, Apr 15, 2023 at 4:48 PM Salim Ibrahim @.***> wrote:

Wa jazakallahu khairan,

So I took a look at the database and found the timings and saw something like this:

33|1|4474 33|2|16343 33|3|24985 33|4|30366 33|5|56458 ... 33|32|512391 33|33|603015 33|34|629199 33|35|642250 33|36|686216 33|37|709485

The audio file is about 22:27 long. Any idea what format is used to mark the timings?

Ideally, I want to analyze the file to see the feasibility of removing that extended repeated portion and accurately re-organizing the timings.

Let me know if this is worth looking into.

On Sat, Apr 15, 2023 at 7:20 AM Ahmed El-Helw @.***> wrote:

the audio file is here https://download.quranicaudio.com/quran/abdul_muhsin_alqasim/024.mp3.

— Reply to this email directly, view it on GitHub https://github.com/quran/quran_android/issues/2247#issuecomment-1509735273, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACN27XSYAX2K6O5NVNO373XBKAAVANCNFSM6AAAAAAW7K4GYM . You are receiving this because you authored the thread.Message ID: @.***>

ahmedre commented 1 year ago

updated the link - yeah we have to remove the duplicate piece in the audio file and then update the timings.