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

gradle: is the mary depenency missing? #12

Open
MiniXC opened this issue Jul 7, 2020 · 8 comments
Open

gradle: is the mary depenency missing? #12

MiniXC opened this issue Jul 7, 2020 · 8 comments
Labels

Comments

@MiniXC
Copy link

MiniXC commented Jul 7, 2020

I was only able to get the gradle branch to work after adding the following to gradle.build

implementation group: 'de.dfki.mary', name: 'marytts-builder', version: '5.2', {
        exclude group: '*', module: 'groovy-all'
        exclude group: '*', module: 'mwdumper'
        exclude group: '*', module: 'sgt'
}

Did I do something wrong on my part or is this missing from the example?

@seblemaguer
Copy link
Member

Hello,

it works for me without using these dependencies. Have you modified anything from the build.gradle before having the problem?

@MiniXC
Copy link
Author

MiniXC commented Jul 7, 2020

No I didn't, but I only checked out this repository and didn't have marytts installed on my system.

@seblemaguer
Copy link
Member

hum weird. I am going to check with a docker image a little bit later and keep you informed

@seblemaguer
Copy link
Member

I tried with the following docker build and it works on my side

FROM ubuntu

###################################################################################
### Installation of the system source
###################################################################################
# Prepare the system
RUN apt-get update
RUN apt-get upgrade -y
RUN apt-get install -y build-essential locales
RUN echo "en_IE.UTF-8 UTF-8" > /etc/locale.gen
RUN echo "en_GB.UTF-8 UTF-8" >> /etc/locale.gen
RUN locale-gen

ENV TZ=Europe/Dublin
RUN ln -snf /usr/share/zoneinfo/$TZ /etc/localtime && echo $TZ > /etc/timezone

###################################################################################
### Environment packages
###################################################################################
# Install necessary tools
RUN apt-get install -y git wget default-jdk

###################################################################################
### Testing part
###################################################################################

RUN git clone https://github.com/marytts/marytts-txt2wav.git /marytts-txt2wav
RUN cd /marytts-txt2wav && git checkout gradle && ./gradlew shadowJar && java -jar build/libs/txt2wav-1.0-SNAPSHOT.jar -o "test.wav" "ok let's go"

@MiniXC
Copy link
Author

MiniXC commented Jul 9, 2020

Ok, must be something going on on my system then. Thank you for taking the time to look into this.

@MiniXC
Copy link
Author

MiniXC commented Jul 14, 2020

Figured it out, seems like at least some other voices cannot be used without voice-cmu-slt-hsmm, is that correct?
With the first line commented out, the build will fail.

//compile group: 'de.dfki.mary', name: 'voice-cmu-slt-hsmm', version: '5.2'
compile group: 'de.dfki.mary', name: 'voice-bits3-hsmm', version: '5.2'
compile group: 'de.dfki.mary', name: 'voice-bits1-hsmm', version: '5.2'

@seblemaguer
Copy link
Member

Thanks for the report. I will try that. It is possible that the voices do not include the transitive dependencies but I would find that weird. I keep you updated

@seblemaguer
Copy link
Member

Yes it is confirmed. It is likely to be corrected only when we will update the voices with the new version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants