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

fix cuda build with openfst 1.8.3 #4936

Merged
merged 1 commit into from
Sep 12, 2024
Merged

Conversation

jtrmal
Copy link
Contributor

@jtrmal jtrmal commented Sep 12, 2024

the openfst version define must be propagated to CUDA compilers, too (due to including base/kaldi-types.h)

@jtrmal
Copy link
Contributor Author

jtrmal commented Sep 12, 2024

I'm merging it as it shouldn't damage anything and I actually need it

@jtrmal jtrmal merged commit 5bfb431 into kaldi-asr:master Sep 12, 2024
1 of 2 checks passed
rkjaran added a commit to tiro-is/kaldi that referenced this pull request Sep 27, 2024
* upstream/master: (76 commits)
  [CI] Automatically refresh docker images (kaldi-asr#4942)
  make sure nvtx3 is used (kaldi-asr#4937)
  [CI] Upgrade checkout action (kaldi-asr#4940)
  Upload logs after build failure (kaldi-asr#4939)
  upload the error logs to artifact repository (kaldi-asr#4938)
  fix cuda build with openfst 1.8.3 (kaldi-asr#4936)
  fix the non-compiling tests, thx to @csukuangfj
  add support for later openfst versions
  make nonconst catches const (kaldi-asr#4926)
  Support openfst-1.8.2
  Support openfst-1.8.1
  Support openfst-1.8.0
  Support openfst-1.7.6
  make codefactor happier
  improve compatibility with C++ standard, esp. C++20
  do a full cleanup on apple silicon
  disable warning about unused flags msse and msse on Apple Silicon
  fix tests and address comments
  catch exception by reference so that compiler does not complain
  Fix reported issues w.r.t python2.7 and some apple silicone quirks
  ...
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

Successfully merging this pull request may close these issues.

1 participant