Skip to content
This repository has been archived by the owner on Feb 15, 2022. It is now read-only.

Fix docker builds for forex.analytics #424

Merged
merged 10 commits into from
Jul 31, 2017
Merged

Fix docker builds for forex.analytics #424

merged 10 commits into from
Jul 31, 2017

Conversation

cmroche
Copy link
Contributor

@cmroche cmroche commented Jul 29, 2017

The forex.analytics documentation says to install this package globally, I am unable to test if it work locally since forex.analytics itself does not build with docker-compose build

@cmroche cmroche changed the title Fix missing node-gyp needed by forex.analytics Fix docker builds for forex.analytics Jul 29, 2017
@cmroche
Copy link
Contributor Author

cmroche commented Jul 29, 2017

I was able to fully fix the docker build, npm now needs the --unsafe-perm option passed to it as forex.analytic needs this to properly call it's post-install scripts.

@DeviaVir DeviaVir merged commit 1e8714e into DeviaVir:master Jul 31, 2017
christian452 pushed a commit to christian452/zenbot that referenced this pull request Aug 4, 2017
…os8f-master

* 'master' of https://github.com/carlos8f/zenbot:
  Improve command help for train command (DeviaVir#436)
  Support for missing win/loss and error rate from output (DeviaVir#426)
  Fix docker builds for forex.analytics (DeviaVir#424)
  Minor fixes for Quadriga trading API (DeviaVir#421)
  xmpp for trading alarms (DeviaVir#333)
  Strategies: TA (ema+macd) and Trust/Distrust (DeviaVir#285)
  Fixed docker build (DeviaVir#349)
  updated c.default_selector to c.selector in README.md (DeviaVir#418)
  added check for message before doing anything with it (DeviaVir#412)
  FIX: Properly check for unknown indicators in forex_analytics (DeviaVir#408)
  Added support for the BTCe exchange (DeviaVir#388)
  Add strategy: forex.analytics, an genetic optimization algorithm for TA-lib stats (DeviaVir#389)

# Conflicts:
#	.gitignore
#	commands/sim.js
#	extensions/exchanges/bittrex/exchange.js
#	extensions/exchanges/quadriga/exchange.js
#	lib/engine.js
#	scripts/auto_backtester/backtester.js
@cmroche cmroche deleted the fa_fixes branch August 7, 2017 12:26
supersabbath pushed a commit to supersabbath/zenbot that referenced this pull request Oct 2, 2017
* Add missing node-gyp for forex.analytics

* Fix docker build of forex.analytics

Needs --unsafe-perm option to properly build dependant ta-lib libraries since docker run the container build with root permissions
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants