Bug report: cannot install firefox

Description of issue
Error with language trying to install Firefox.
Same results when specifying the language or not: brew cask install firefox --language=en
I am based in Brazil, my locale is set to en_CA though, not sure where pt_BR is coming from.

Output of `brew cask install firefox --verbose`
Error: 'pt_BR' cannot be parsed to a Locale while loading 'firefox' from '/usr/local/Homebrew/Library/Taps/caskroom/homebrew-cask/Casks/firefox.rb'
Follow the instructions here:
  https://github.com/caskroom/homebrew-cask/blob/master/doc/reporting_bugs/pre_bug_report.md

If this doesn’t fix the problem, please report this bug:
  https://github.com/caskroom/homebrew-cask#reporting-bugs

/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:23:in `rescue in load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:19:in `load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:11:in `load_from_file'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/source/path_base.rb:20:in `load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/source/tapped.rb:15:in `load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc.rb:57:in `load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:21:in `block in install_casks'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:19:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:19:in `install_casks'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:10:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:110:in `run_command'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:149:in `process'
/usr/local/Homebrew/Library/Homebrew/cask/cmd/brew-cask.rb:8:in `<top (required)>'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/usr/local/Homebrew/Library/Homebrew/brew.rb:24:in `require?'
/usr/local/Homebrew/Library/Homebrew/brew.rb:100:in `<main>'
Error: Kernel.exit

Output of `brew doctor`
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry and just ignore them. Thanks!

Warning: Your XQuartz (2.7.7) is outdated.
Please install XQuartz 2.7.9 (or delete the current version).
XQuartz can be updated using Homebrew-Cask by running
  brew cask reinstall xquartz

Warning: You have unlinked kegs in your Cellar
Leaving kegs unlinked can lead to build-trouble and cause brews that depend on
those kegs to fail to run properly once built. Run `brew link` on these:
  boot2docker
  docker
  docker-machine
  nspr

Warning: Some directories in your path end in a slash.
Directories in your path should not end in a slash. This can break other
doctor checks. The following directories should be edited:
  /usr/local/Cellar/android-sdk/24.4.1_1//tools/
  /usr/local/Cellar/android-sdk/24.4.1_1//tools/
  /usr/local/Cellar/android-sdk/24.4.1_1//tools/
  /opt/homebrew-cask/Caskroom/genymotion/2.2.2/Genymotion.app/Contents/MacOS/
  /Users/filipe/Applications/Firefox.app/Contents/MacOS/
  /opt/homebrew-cask/Caskroom/genymotion/2.2.2/Genymotion.app/Contents/MacOS/
  /Users/filipe/Applications/Firefox.app/Contents/MacOS/
  /opt/homebrew-cask/Caskroom/genymotion/2.2.2/Genymotion.app/Contents/MacOS/
  /Users/filipe/Applications/Firefox.app/Contents/MacOS/

Warning: Broken symlinks were found. Remove them with `brew prune`:
  /usr/local/bin/aapt
  /usr/local/etc/bash_completion.d/brew_bash_completion.sh

Warning: Some installed formula are missing dependencies.
You should `brew install` the missing dependencies:
  brew install gd gmp@4 mpfr@2 <EMAIL_ADDRESS>webp

Run `brew missing` for more details.

Output of `brew cask doctor`
==> macOS Release:
10.11.6
==> Hardware Architecture:
intel-64
==> Ruby Version:
2.0.0-p648
==> Ruby Path:
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby
==> Homebrew Version:
Homebrew 1.1.6-25-g1c710ad
Homebrew/homebrew-core (git revision 74063; last commit 2017-01-06)
==> Homebrew Executable Path:
/usr/local/bin/brew
==> Homebrew Cellar Path:
/usr/local/Cellar
==> Homebrew Repository Path:
/usr/local/Homebrew
==> Homebrew Origin:
https://github.com/Homebrew/brew.git
==> Homebrew-Cask Version:
Homebrew-Cask 1.1.6-25-g1c710ad
caskroom/homebrew-cask (git revision ee78; last commit 2017-01-07)
==> Homebrew-Cask Install Location:
<NONE>
==> Homebrew-Cask Staging Location:
/opt/homebrew-cask/Caskroom
==> Homebrew-Cask Cached Downloads:
/Users/filipe/Library/Caches/Homebrew/Cask
0 files, 0B
==> Homebrew-Cask Default Tap Path:
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-cask
==> Homebrew-Cask Alternate Cask Taps:
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-fonts
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-versions
==> Homebrew-Cask Default Tap Cask Count:
3501
==> Contents of $LOAD_PATH:
/usr/local/Homebrew/Library/Homebrew/cask/lib
/usr/local/Homebrew/Library/Homebrew
/Library/Ruby/Site/2.0.0
/Library/Ruby/Site/2.0.0/x86_64-darwin15
/Library/Ruby/Site/2.0.0/universal-darwin15
/Library/Ruby/Site
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0/x86_64-darwin15
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0/universal-darwin15
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/x86_64-darwin15
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/universal-darwin15
==> Contents of $RUBYLIB Environment Variable:
<NONE>
==> Contents of $RUBYOPT Environment Variable:
RUBYOPT=""
==> Contents of $RUBYPATH Environment Variable:
<NONE>
==> Contents of $RBENV_VERSION Environment Variable:
<NONE>
==> Contents of $CHRUBY_VERSION Environment Variable:
<NONE>
==> Contents of $GEM_HOME Environment Variable:
<NONE>
==> Contents of $GEM_PATH Environment Variable:
<NONE>
==> Contents of $BUNDLE_PATH Environment Variable:
<NONE>
==> Contents of $PATH Environment Variable:
PATH="/Users/filipe/.gem/ruby/2.3.0/bin:/Users/filipe/.rubies/ruby-2.3.0/lib/ruby/gems/2.3.0/bin:/Users/filipe/.rubies/ruby-2.3.0/bin:/usr/local/opt/coreutils:/usr/local/bin:/usr/local/sbin:/sbin:/bin:/usr/sbin:/usr/bin:./node_modules/.bin:/Users/filipe/node_modules/.bin:/Users/filipe/Dropbox/bin:/Users/filipe/bin:/Users/filipe/.dots/bin:/usr/local/Cellar/android-sdk/24.4.1_1//bin:/usr/local/Cellar/android-sdk/24.4.1_1//tools/:/usr/local/Cellar/android-sdk/24.4.1_1//platform-tools:/usr/local/opt/coreutils:/usr/local/bin:/usr/local/sbin:/sbin:/bin:/usr/sbin:/usr/bin:./node_modules/.bin:/Users/filipe/node_modules/.bin:/Users/filipe/Dropbox/bin:/Users/filipe/bin:/Users/filipe/.dots/bin:/usr/local/Cellar/android-sdk/24.4.1_1//bin:/usr/local/Cellar/android-sdk/24.4.1_1//tools/:/usr/local/Cellar/android-sdk/24.4.1_1//platform-tools:/usr/local/opt/coreutils/libexec/gnubin):/usr/local/bin:/usr/local/sbin:/sbin:/bin:/usr/sbin:/usr/bin:./node_modules/.bin:/Users/filipe/node_modules/.bin:/Users/filipe/Dropbox/bin:/Users/filipe/bin:/Users/filipe/.dots/bin:/usr/local/Cellar/android-sdk/24.4.1_1//bin:/usr/local/Cellar/android-sdk/24.4.1_1//tools/:/usr/local/Cellar/android-sdk/24.4.1_1//platform-tools:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin:/opt/homebrew-cask/Caskroom/genymotion/2.2.2/Genymotion.app/Contents/MacOS/:/Users/filipe/Applications/Firefox.app/Contents/MacOS/:/opt/homebrew-cask/Caskroom/genymotion/2.2.2/Genymotion.app/Contents/MacOS/:/Users/filipe/Applications/Firefox.app/Contents/MacOS/:/opt/homebrew-cask/Caskroom/genymotion/2.2.2/Genymotion.app/Contents/MacOS/:/Users/filipe/Applications/Firefox.app/Contents/MacOS/:/usr/local/Homebrew/Library/Taps/homebrew/homebrew-boneyard/cmd:/usr/local/Homebrew/Library/Homebrew/cask/cmd:/usr/local/Homebrew/Library/Homebrew/shims/scm"
==> Contents of $SHELL Environment Variable:
SHELL="/bin/bash"
==> Contents of Locale Environment Variables:
LANG="en_CA.UTF-8"

Cannot reproduce:
-bash-4.4.5- /Users/raesy [> brew cask install firefox --language=pt
==> Downloading https://ftp.mozilla.org/pub/firefox/releases/50.1.0/mac/pt-PT/Firefox%2050.1.0.dmg
######################################################################## 100.0%
==> Verifying checksum for Cask firefox
==> Moving App 'Firefox.app' to '/Applications/Firefox.app'
🍺  firefox was successfully installed!

The only significant difference in your setup and mine is:
==> Homebrew-Cask Staging Location:
/opt/homebrew-cask/Caskroom

@filipesperandio What’s the output of echo $HOMEBREW_CASK_OPTS?

@miccal Isn't his case a little different though? He chose a locale that is not listed in the Cask (en_CA) and pt_BR, which also isn't listed, is probably brought into the mix by his Timezone or Language & Region settings in macOS. I am not familiar with the inner workings of brew cask install, so can't chime in on why the default locale en is not chosen in his case.

@neutric the OP said he ran brew cask install firefox --language=en (which is equivalent to brew cask install firefox)  and it failed - I used pt just as an example.
@filipesperandio what language specifically are you trying to install?

@vitorgalvao
$ echo $HOMEBREW_CASK_OPTS
--caskroom=/opt/homebrew-cask/Caskroom

@miccal: @neutric is kinda right... If you see my brew doctor output, you will see LANG=en_CA.UTF-8.
I expect brew cask to pick this one if I don't specify anything, but the result is the error pointing to pt_BR... and even if I especify --language=en the error is still the same one.
For what it is worth, I want to install an english version

brew cask does not choose a language based on LANG. What happens if you run brew cask install --force Firefox?

How is the language selected though?
Despite of the arguments I pass (--language=en, --force), I always get:
$ brew cask install --force firefox
Error: 'pt_BR' cannot be parsed to a Locale while loading 'firefox' from '/usr/local/Homebrew/Library/Taps/caskroom/homebrew-cask/Casks/firefox.rb'
Follow the instructions here:
  https://github.com/caskroom/homebrew-cask/blob/master/doc/reporting_bugs/pre_bug_report.md

If this doesn’t fix the problem, please report this bug:
  https://github.com/caskroom/homebrew-cask#reporting-bugs

/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:23:in `rescue in load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:19:in `load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:11:in `load_from_file'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/source/path_base.rb:20:in `load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/source/tapped.rb:15:in `load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc.rb:57:in `load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:21:in `block in install_casks'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:19:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:19:in `install_casks'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:10:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:110:in `run_command'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:149:in `process'
/usr/local/Homebrew/Library/Homebrew/cask/cmd/brew-cask.rb:8:in `<top (required)>'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/usr/local/Homebrew/Library/Homebrew/brew.rb:24:in `require?'
/usr/local/Homebrew/Library/Homebrew/brew.rb:100:in `<main>'

@filipesperandio the language is selected with the flag --language= for Casks that have the language stanza.
This is weird since pt_BR is not a part of firefox.rb.
Does this happen when you install anything else via brew cask? Try running brew cask install operadriver (because it is just a small binary which is completely removed by brew cask uninstall operadriver) and report back if it works.

I understand --language, my question is more about how is it selected when I don't pass anything. Should it always fallback to the default language if not specified? Or does brew cask try to select what it thinks is best for the user?
Operadriver goes fine, but I notice it does not have anything on language on its formula.
$ brew cask install operadriver
==> Downloading https://github.com/operasoftware/operachromiumdriver/releases/download/v0.2.2/operadriver_mac64.zip
######################################################################## 100.0%
==> Verifying checksum for Cask operadriver
==> Symlinking Binary 'operadriver' to '/usr/local/bin/operadriver'
🍺  operadriver was successfully installed!

... and it is not related exclusively to firefox, I think it is to everything that has language specs:
$ brew cask install world-of-tanks
Error: 'pt_BR' cannot be parsed to a Locale while loading 'world-of-tanks' from '/usr/local/Homebrew/Library/Taps/caskroom/homebrew-cask/Casks/world-of-tanks.rb'
Follow the instructions here:
  https://github.com/caskroom/homebrew-cask/blob/master/doc/reporting_bugs/pre_bug_report.md

If this doesn’t fix the problem, please report this bug:
  https://github.com/caskroom/homebrew-cask#reporting-bugs

/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:23:in `rescue in load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:19:in `load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cask_loader.rb:11:in `load_from_file'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/source/path_base.rb:20:in `load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/source/tapped.rb:15:in `load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc.rb:57:in `load'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:21:in `block in install_casks'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:19:in `each'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:19:in `install_casks'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli/install.rb:10:in `run'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:110:in `run_command'
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:149:in `process'
/usr/local/Homebrew/Library/Homebrew/cask/cmd/brew-cask.rb:8:in `<top (required)>'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/usr/local/Homebrew/Library/Homebrew/brew.rb:24:in `require?'
/usr/local/Homebrew/Library/Homebrew/brew.rb:100:in `<main>'

I've also removed all language entries on firefox formula, made some adjustments and I could install it with success.
Maybe I you help me understand how language works I could have some insights about this problem.

Link to the language stanza documentation.

Closing due to lack of reproducibility.


fixing linking error

there was a .md in the url, removing to fix linking.

:+1:

:tada: This has been pushed live to https://developers.google.com/web/


Reset priorities after transfer of leadership

https://github.com/Alluxio/alluxio/issues/14228
What changes are proposed in this pull request?
The quorum elect command now resets priorities after a transfer of leadership, regardless of transfer success.
Why are the changes needed?
Not resetting causes issues with cluster behavior down the line (during elections, if masters go down and come back up, leave and rejoin the cluster).
Does this PR introduce any user facing changes?
No

Codecov Report

Merging #14273 (2c1e423) into master (d148ad0) will decrease coverage by 25.86%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #14273       +/-   ##
=============================================
- Coverage     43.22%   17.35%   -25.87%
+ Complexity     9320     2707     -6613
=============================================
  Files          1437     1443        +6
  Lines         83863    84042      +179
  Branches      10148    10167       +19
=============================================
- Hits          36247    14583    -21664
- Misses        44626    68190    +23564
+ Partials       2990     1269     -1721

Impacted Files
Coverage Δ

...ient/journal/RetryHandlingJournalMasterClient.java
0.00% <0.00%> (ø)

...a/alluxio/master/journal/DefaultJournalMaster.java
21.73% <0.00%> (-3.27%)
:arrow_down:

...ter/journal/JournalMasterClientServiceHandler.java
23.80% <0.00%> (-2.51%)
:arrow_down:

...alluxio/master/journal/raft/RaftJournalSystem.java
0.00% <0.00%> (-55.98%)
:arrow_down:

...lluxio/cli/fsadmin/journal/QuorumElectCommand.java
0.00% <0.00%> (ø)

...io/stress/common/AbstractMaxThroughputSummary.java
0.00% <0.00%> (ø)

...ava/alluxio/stress/common/GeneralBenchSummary.java
0.00% <0.00%> (ø)

.../java/alluxio/stress/common/GeneralParameters.java
0.00% <0.00%> (ø)

...o/stress/jobservice/JobServiceBenchParameters.java
0.00% <0.00%> (ø)

...uxio/stress/jobservice/JobServiceBenchSummary.java
0.00% <ø> (ø)

... and 694 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bddb70e...2c1e423. Read the comment docs.


e2e add one test as a sample

Signed-off-by: akihikokuroda<EMAIL_ADDRESS>This PR adds one e2e test that can be used a sample of real tests.

@timflannagan @tylerslaton Thanks!  I don't have power to merge in this repo.  Would you merge this?


Handle file system case sensitivity.

This adds ability to check for file systems case sensitivity. Middlewares can now call middleware.CaseSensitiveFS to know if the file system is case sensitive or not and handle it as required.
This also caters for FastCGI middleware.

Hmm, case sensitivity should probably be something that is configurable... I like the default of "case sensitive" like we have it now, but look at this for instance: http://stackoverflow.com/questions/14814419/how-do-i-make-urls-case-insensitive-in-linux-server
What do you think?

So, where do we place the configuration ? I like the idea too.

Let's consider an env variable. Default is case-sensitive like we have now, but CASE_INSENSITIVE_PATH=1 to make it case-insensitive, maybe?

I think that this should be a configuration option (as mentioned before), along with the etag config option (option to disable if it causes issues).  As part of an option to a filesystem directive of some sort, the default could be "preserving" for OSX, possibly "sensitive" and "insensitive" as well.

As part of an option to a filesystem directive of some sort, the default could be "preserving" for OSX, possibly "sensitive" and "insensitive" as well.

True, although this can be somewhat tricky to detect, and further, I'm not sure it's safe to assume that a certain OS uses a certain file system...

Looking good now! Squash and we'll merge this in :+1:


Generalized Search

Generalized the search to work with any table
with some basic classes and some updated
JavaScript, we can now search for colors and emojis.
I also felt the previous styling was whack

@khalidabuhakmeh Awesome, thank you very much! 👍 💯


Hexadecimal Optimization

We often use GUID and other unique identifiers in code generated files,   This is a good thing, but can be a detriment to memory use in Blitz Search.  a really good optimization to play with is to ignore words that are all in the Hexadecimal range when storing the "Unique Word List".  We also need to short circuit the check on the file if the user is actually pasting in some GUID or otherwise.
There's potential for a negative performance impact but how many English words can you think of that fit in "ABCDEF" range.
bed fed abaccadaba fad bad dad, are examples ( I should play more word games! ) of words that would end up triggering a slower, all the files search.

Little better, The common words cache will help even more:

I also don't store the underscores either.. performance is still fast.


Odata AsyncQueryExecutor

Pull Request
📖 Description
Implementation of an Odata IAsyncQueryExecutor.
📑 Test Plan
I've created a test sample using TripPin sample OData service available here TripPin Service
✅ Checklist
General

[ ] I have added tests for my changes.
[x] I have tested my changes.
[ ] I have updated the project documentation to reflect my changes.
[x] I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

[x] I have added a new component
[ ] I have added Unit Tests for my new component
[ ] I have modified an existing component
[ ] I have validated the Unit Tests for an existing component

Hi,
Bit late to the party but can you change the project folder and name to Microsoft.FluentUI.AspNetCore.Components.DataGrid.ODataAdapter so we stay in line with the EF naming scheme. Also, I think adding .Client could be a bit misleading on what it is and does.

@vnbaaij renaming done!


chore(userspace/libsisnp): addresses is_pod deprecation in C++20

Signed-off-by: Andrea Bonanno<EMAIL_ADDRESS>
What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area libsinsp

What this PR does / why we need it:
Addresses is_pod deprecation in C++20
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: andreabonanno
To complete the pull request process, please assign fntlnz after the PR has been reviewed.
You can assign the PR to them by writing /assign @fntlnz in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @andreabonanno. Thanks for your PR.
I'm waiting for a falcosecurity member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Hi! Thanks for this PR!
/ok-to-test

LGTM label has been added.  Git tree hash: 45c42b7d608f852da359b916b66a5ae632b3ef8a

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: andreabonanno, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: andreabonanno, FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


fix!: change exports map

This PR changes the export map, as there were a minor inconsistencies in it.

#24  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @luxass and the rest of your teammates on  Graphite

Merge activity

Jul 12, 3:53 AM EDT: @luxass merged this pull request with Graphite.


Add Tracing support for Storage

[ ] Files
[ ] Blobs
[ ] Queues

Completed adding tracing support for storage


Update js-templates.cson (fix deprecated promise usage)

As per angular docs:

The $http legacy promise methods success and error have been deprecated. Use the standard then method instead. If $httpProvider.useLegacyPromiseExtensions is set to false then these methods will throw $http/legacy error.

Very sorry for super late. It looks good.


Make TextTransform.exe invocation platform safe. Needs windows implem…

…entation

https://github.com/mono/Embeddinator-4000/issues/337

Thanks for looking into this.
Only issue I can see is that this still forces users to install Xamarin Studio to build the project though. It's an issue for Windows and for macOS Travis CI build. Only alternative I've found was to use https://www.nuget.org/packages/Mono.TextTransform/ to generate .tt file.
Another alternative would be to just commit the generated subscript.cs file, it should not change very often.

I missed that part. Let me look into the nuget. Should be trivial to swap.

Ok, that at least uses the nuget instead of XS.
It still doesn't address Windows, and I don't even have a windows dev machine to test it.
@tritao - Could you look into what path bits I'd need to run nuget / the nuget on Windows and I can add that.
If this becomes too much of a mess we can check in output, but I hate to for fear of staleness.

Looks good. @realvictorprm is working on a PR with Windows fixes, he should be able to help out there.

Build and tests passed.

Anyway regarding the Windows bits, I think it should be a matter of adding the same call to the generator from the nuget, except we do not call via mono, but call the .exe directly.

Build and tests passed.

Excellent, this is exactly what I  planned to fixed also because I alteady ran into the bug thats it's not dowmloaded for window's (workaround was nuget install :wink:).
Anyway, merging the fix is requirement for my PR.

@chamons Should I merge the fix into my PR? Without it people cannot try it out instantly because of the missing dep.


Weights file

Hi @WenqiJiang
I want to ask weights file which you use "VGG_C_implementation/main.c". I can't see these files in version2 or version1 folder.
Can you share this weights with us? It helps a lot.
Thank you.

Hi,Sorry for late reply. The data file lost. So you may need to create it by yourself.BestsPORSCHE DESIGN HUAWEI Mate 30 RS-------- 原始邮件 --------发件人： kirmans @.>日期： 2021年4月4日周日 21:38收件人： WenqiJiang/VGG16_FPGA_Accelerator @.>抄送： Subscribed @.***>主    题： [WenqiJiang/VGG16_FPGA_Accelerator] Weights file (#1)
Hi @WenqiJiang
I want to ask weights file which you use "VGG_C_implementation/main.c". I can't see these files in version2 or version1 folder.
Can you share this weights with us? It helps a lot.
Thank you.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.


Several issues on 'update'

Say I have a 'messages' service with 3 existing messages. I query those via .find() and subscribe:
{
  "total": 3,
  "limit": 5,
  "skip": 0,
  "data": [
    {
      "text": "read me!",
      "id": 0
    },
    {
      "text": "read me!",
      "id": 1
    },
    {
      "text": "read me!",
      "id": 2
    }
  ]
}

I now change one of the messages (id 1) using cURL. The Observable emits the following object:
{
  "total": 0,
  "limit": 5,
  "skip": 0,
  "data": [
    {
      "text": "read me!",
      "id": 0
    },
    {
      "text": "read me!",
      "id": 1
    },
    {
      "text": "I changed",
      "id": "1"
    },
    {
      "text": "read me!",
      "id": 2
    }
  ]
}

Several things are unexpected:

the outdated object isn't removed.
total changed to 0. Expected 3 (or 4 in case of not removed item). not sure about this, couldn't reproduce
id of the new object is of type string. Expected number.

What does your setup look like? I think that most of the weirdness is happening because the id is converted to a string. You may have to parseInt it in a before hook for update and patch.

I checked and on the wire 'id' is a string already. 42["messages updated",{"text":"I changed another time","id":"1"}] So it appears to be a server-side problem. Sorry for the confusion.
I created the server using the CLI. 'generate', 'generate service'. No auth, memoryDB.
However, this is totally a bug. Want me to file it somewhere else?

Oh, I wasn't saying this isn't a bug, I'm just trying to figure out how to reproduce it. From what I got:

Simple setup with in-memory DB
Make plain query via .find()
POST to the REST endpoint (works?)
PATCH an existing entry via REST (Curl) (doesn't work?)

Never had the feeling you were!
Yep, POSTing works fine, the problem occurs after PUTting a new value:

Simple setup with in-memory DB
Make plain query via .find()
POST to the REST endpoint - works
PUT an existing entry via REST (Curl) - updating the value works, but 'id' changes from type int to type string

$ curl 'http://localhost:3030/messages' -H 'Content-Type: application/json' --data-binary '{ "text": "read me!" }'
{"text":"read me!","id":0}                                                                                $ curl 'http://localhost:3030/messages' -H 'Content-Type: application/json' --data-binary '{ "text": "read me!" }'
{"text":"read me!","id":1}                                                                                $ curl 'http://localhost:3030/messages/1' --request PUT  -H 'Content-Type: application/json' --data-binary '{ "text": "I changed" }'
{"text":"I changed","id":"1"}

This was a bug in feathers-memory that has been fixed with https://github.com/feathersjs/feathers-memory/pull/26. I think most other database adapters should not have that problem. Thanks for reporting!


Give this poor guy some tests

Every other repo is sleeping soundly at night in the knowledge that there is a good test suite to prevent mayhem. Only our poor federation API is currently dancing without a harness. We should rectify this and start gradually adding a test suite.

https://github.com/neurobagel/federation-api/tree/main/tests 👀
Would definitely benefit from a test workflow too though 😅

Oh wow, how did I miss that? I was looking for them after I saw that we had no runners on the last PR. Ok, let's tone down that issue title maybe ...

:rocket: Issue was released in v0.1.0 :rocket:


On Linux, ~/.fonts/** are missing .Name and .Family

But ones installed system-wide /usr/share/fonts/** work fine for me.
I'd like to use the Name and Family for a little font-inspector with a web frontend.
I'd like to use the .Match(name) func to autocomplete the font-family CSS attribute to take the frustration out of trying to nail the whitespace and little variations one has to do to get the web-compatible names of fonts.
Any ideas?

Might've found it:
https://stackoverflow.com/questions/70892173/how-can-i-get-the-font-family-name-from-a-ttf-file-in-golang
So I'll .List() all the fonts to get the paths,
parse those without a .Name by .Filename with the truetype package,
and modify the .Name and .Family accordingly.


Update notebook images to include kserve instead of kfserving. Also update repo name.

Description of your changes:
Update notebook images to include kserve instead of kfserving. Also update repo name.
Tests have passed
Testing:

[ ] Unit tests pass
[ ] e2e tests pass
Details about new tests (If this PR adds a new feature)
Details about any manual tests performed

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Also, will update all refs as part of this https://github.com/awslabs/kubeflow-manifests/search?p=1&q=c9e4w0g3

Merging with one approval and pending final test run - will comment here once tests pass.

Merging with one approval and pending final test run - will comment here once tests pass.

Tests Passed.


User - Partner Logic

Users should be able to be associated with more than one partner and partners should be able to be associated with multiple users.
Currently i'm running into problems with this in multiple ways:

Registering a partner with the same email address is not possible - see screenshot:

From the admin console, I cannot add a users to a partner if the user is already associated with another partner.

@Keristiena  @robynlloos  Per our discussion, please review for examples.

@kalpanajha @EdemD QA team please test the 1st screenshot of this ticket. The 2nd screenshot is not an issue and we solved using another ticket. This ticket would only be the 1st screenshot.
C'c ed - @jerrodanzalone


API for creating custom presets

Hey! I know this issue not for current missions, but I want to discuss future of steiger and possibility to extend them for FSD-like architecture. I have already created an issue about possibility to write self rules (https://github.com/feature-sliced/steiger/issues/35). By now, I understand, this feature doesn't solve a more general problem. How I cant lint my architecture using steiger, If I have a more complex architecture which build on FSD.
I think devs who work on any big project build on FSD introduces local architectural abstractions and its own rules that are specific to the project at some point in time. And its logical growth, since the methodology does not cover absolutely all cases.
For example, we use cross-imports on entity layer (yes, I know about @x, but we decide not to use it). Or we use a few new layers as abstract-widgets (abstract widgets which used in other widgets) or page-drawers (we have addition level of navigation into drawer on top of regular pages. In fact, these are the same pages, but we decided to separate them semantically). Current package steiger-plugin-fsd is strictly made for the basic set of layers (has high coupling with @feature-sliced/filesystem) and I don't see any chance to extend it for our project. Now we can't use steiger, because I can't extend it (without a complete rewrite @feature-sliced/filesystem and steiger-plugin-fsd).
So, I would like to see about future plan of steiger. What a development strategy of this library? And I don't think this will be applicable to most projects based on FSD, if steiger will be developed as only set of strict rules for a methodology without the possibility of extension.

Hi! Thanks for this perspective, it's very valuable. Sometimes I get a few perspectives like this and then go through an existential crisis of sorts and then come out with very interesting realizations. Something that's possibly intended for FSD 2.2. I'll be keeping this in mind.
The future plan of Steiger? There's a big feature coming up very soon, the new config format. After that we will explore further DX improvements for writing rules and auto-fixing. Then we will also tackle performance, CLI experience, editor extensions, etc. That's the near-term roadmap.
As for extensibility, I think that the new config format will go a long way towards achieving that, but the one big hurdle that is currently very difficult to overcome is additional layers. As I said, maybe a future version of FSD will relax the layer standardization and allow adding more layers, but with the current spec, that's how it is. I'd be interested to hearing your experience with the current spec of FSD, your challenges and pain points, also about the cross-imports.

And I don't think this will be applicable to most projects based on FSD

Why so?


Support TS moduleResolution, NodeNext

Is your feature request related to a problem? Please describe.
I get the following TS error when trying to use @hookform/resolver/zod with TS moduleResolution, NodeNext:
src/pages/UserInfo.tsx:2:29 - error TS7016: Could not find a declaration file for module '@hookform/resolvers/zod'. '/Users/stickb/Code/niwc-pac/cosmos-ui/node_modules/.pnpm/@hookform+resolvers@2.9.10_react-hook-form<EMAIL_ADDRESS>implicitly has an 'any' type.
  Try `npm i --save-dev @types/hookform__resolvers` if it exists or add a new declaration (.d.ts) file containing `declare module '@hookform/resolvers/zod';`

2 import { zodResolver } from "@hookform/resolvers/zod";

Describe the solution you'd like
The following pnpm patch fixes this issue:
diff --git a/package.json b/package.json
index b46f9e11d988e11c89fbaf580e6bbc74502c6e7f..5bb7790760e298507138675b2f780c0cc2ec7e69 100644
--- a/package.json
+++ b/package.json
@@ -15,6 +15,7 @@
       "require": "./dist/resolvers.js"
     },
     "./zod": {
+      "types": "./zod/dist/index.d.ts",
       "umd": "./zod/dist/zod.umd.js",
       "import": "./zod/dist/zod.mjs",
       "require": "./zod/dist/zod.js"

Describe alternatives you've considered
I think alternatively, if zod.d.ts contained the same content as index.d.ts, then this would work automatically without the change to the package.json
Additional context
None

See TS docs:

The new support works similarly with import conditions. By default, TypeScript overlays the same rules with import conditions - if you write an import from an ES module, it will look up the import field, and from a CommonJS module, it will look at the require field. If it finds them, it will look for a colocated declaration file. If you need to point to a different location for your type declarations, you can add a "types" import condition.

https://www.typescriptlang.org/docs/handbook/esm-node.html#packagejson-exports-imports-and-self-referencing

Note: your patch is actually wrong
The correct patch should be:
diff --git a/package.json b/package.json
index b46f9e11d988e11c89fbaf580e6bbc74502c6e7f..143135bd80304d991581643016799b0a0964c83b 100644
--- a/package.json
+++ b/package.json
@@ -16,8 +16,14 @@
     },
     "./zod": {
       "umd": "./zod/dist/zod.umd.js",
-      "import": "./zod/dist/zod.mjs",
-      "require": "./zod/dist/zod.js"
+      "import": {
+        "types": "./zod/dist/index.d.ts",
+        "default": "./zod/dist/zod.mjs"
+      },
+      "require": {
+        "types": "./zod/dist/index.d.ts",
+        "default": "./zod/dist/zod.js"
+      }
     },
     "./yup": {
       "umd": "./yup/dist/yup.umd.js",
diff --git a/zod/dist/index.d.ts b/zod/dist/index.d.ts
index 18dffb9fef1bf09e83d9ff5e3aec325cf562c780..b55eff2beb1311c50274f7040dd88102e5858faf 100644
--- a/zod/dist/index.d.ts
+++ b/zod/dist/index.d.ts
@@ -1,2 +1,2 @@
-export * from './zod';
-export * from './types';
+export * from './zod.js';
+export * from './types.js';
diff --git a/zod/dist/zod.d.ts b/zod/dist/zod.d.ts
index 839632e3d8495c00933dada6dd9f7a7d4003311a..3ad42563f00f8fb0a6d09c7234b66653b84561fe 100644
--- a/zod/dist/zod.d.ts
+++ b/zod/dist/zod.d.ts
@@ -1,2 +1,2 @@
-import type { Resolver } from './types';
+import type { Resolver } from './types.js';
 export declare const zodResolver: Resolver;

@Jack-Works, it works for me, but your suggestion looks more robust. Thank you!

Can you provide a repro example that I can clone?

@jorisre, https://github.com/bestickley/rhf-zod-resolver-node-next-issue

This error also shows up with TypeScript 5.0's bundler module resolution.

Ran into it this morning when we tried to switch to bundler.

@jorisre, https://github.com/bestickley/rhf-zod-resolver-node-next-issue See error in editor within src/App.tsx on line 3 (import { zodResolver } from '@hookform/resolvers/zod';)

Thank you! I tried with a NextJs project as well. With(out) moduleResolution: "NodeNext" and it works with the @Jack-Works 's solution.
Now, I've to check the standard in order to open a PR on https://github.com/marvinhagemeister/check-export-map.
If someone have link that document this pattern in package.json?


Resolve relative paths on configuration loading

This moves the concern of resolving the files relative to the config
file into the configuration loading itself.
It also fixes #921 which did not load the cert and token files relatively.
@jimmidyson I missed the relative loading in my review, you could hardly know. Just be aware that this might break your setup.
@juliusv

@fabxc Thanks - will keep an eye on it when we next upgrade our fork. We normally use absolute paths anyway - on kubernetes the relevant files (ca certs, bearer token files, mainly) are mounted at known locations.

:+1:

@juliusv added commit to validate existence of cert and token files via promtool check-config

Nice - thanks @juliusv.

ping @grobie

:+1:


[#365] Leave space in buffer for null-terminating byte when using std::string::copy

This change is safe as it just reduces the number writable bytes to the destination buffer by 1 as defined by the CheckAuthCredentialsInput input struct.
See https://docs.irods.org/4.3.3/doxygen/structCheckAuthCredentialsInput.html.

was this found by inspection?  or did a tool point you to this?   do we need a comment in the code?

It was upon inspection. No comment needed as this is just how C/C++ string processing works.

gotcha.

And because of similar code in the iRODS 5 PR.

Tested against iRODS 5 PR and auth still works, so we're good.
Pounded.


Enderchest per inv (or chared between inventories)

I believe it would be nice to be able to specify seperate enderchests per PJI inv, to where my creative worlds don't need to block all use of echest, but instead just have a seperate echest.

Its a planned feature

Added in latest release

Is there any documentation on this feature??


[Windows] The result of subtracting two dates in Windows is different from the result calculated on other platforms。

Steps to reproduce
Running code separately on Android, Web, and Windows
Expected results
Get the same result
Actual results
android:<PHONE_NUMBER>
windows:<PHONE_NUMBER>
Code sample
Code sample
final a = DateTime.now();
final b = DateTime(1912, 2, 18);
days = a.difference(b).inDays;
developer.log('gzdate diff: $days');

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

@desert1210
Since DateTime is a dart core method, this issue probably need to be reported in dedicated repo: https://github.com/dart-lang/sdk/issues
Closing from here.


Default CarPlay on an Iphone 7 IOS 14.4 jailbroken with OdysseyRa1n

When i connect my phone to my car only the default CarPlay appears.I can't use 3d touch on default apps because it does nothing.
To try to fix the problem i used Libhooker configurator and put CarplayEnable in the substrate compatibility mode but it did nothing more.
Edit: Forgot to mention, the tweak appears in settings and I can choose which apps are supposed to appear on CarPlay but even if all the apps are selected to show up none of them actually shows up on my CarPlay screen

this is a libhooker issue. I can't work around it as I dont have an OdysseyRa1n device (and the libhooker developer does not make the package available to other jailbreaks)


Support AAPM low-dose CT helical reconstruction

Although Low Dose CT Grand Challenge is 2016 dataset. Nowadays still many paper use it. But it seems not a clear code to guide how to reconstruct to CT image from the raw projection data.
If LEAP can do that, I can be use to the down stream task(sparse-view & limited angle).

It's been a long time since I've looked at the data in this challenge.  I think there is some flying focal spot (FFS) rebinning, but don't remember what else is necessary.  Can you remind me?  LEAP does not have FFS rebinning, but if that is all that is missing, shouldn't be too hard to get an implementation of this completed.

I'm not familar with that, All I can found is this two project(odl and cuda-based) focus to slove this reconstruction problem but they both too old to be run.

Thanks for the links, but unfortunately, they don't help me determine what preprocessing is needed without doing a deep dive into the code.  It would be more helpful to have a verbal explanation of what is needed.
I would like to provide support for this dataset in LEAP, but I don't have much time to work on this right now.  If you can provide a list (or link to a list) of the necessary algorithms for me, then I can try to get this working.

@kylechampley Hi, Kyle, I wanna to know if the LEAP supports flying focal spot (FFS) rebinning in helical reconstruction now, Thx!

Unfortunately, it does not.


Fix metrics tests and ClassNotFoundException when calling stats API

Signed-off-by: Joshua Li<EMAIL_ADDRESS>Description
Calling GET /_plugins/_reports/_local/stats would crash OpenSearch with ClassNotFoundException.
[2022-11-29T00:53:06,489][ERROR][o.o.b.OpenSearchUncaughtExceptionHandler] [dev-dsk-lijshu-2b-daaa4402.us-west-2.amazon.com] fatal error in thread [Thread-3], exiting
java.lang.NoClassDefFoundError: com/eclipsesource/json/JsonValue
        at org.opensearch.reportsscheduler.metrics.Metrics.collectToFlattenedJSON(Metrics.java:184) ~[?:?]
        at org.opensearch.reportsscheduler.resthandler.ReportStatsRestHandler.prepareRequest$lambda-0(ReportStatsRestHandler.kt:78) ~[?:?]

This is because for some reason OpenSearch plugins don't resolve indirect dependencies of com.github.wnameless.json:json-flattener, I had to manually add them to build.gradle:
  implementation group: 'com.github.wnameless.json', name: 'json-base', version: '2.2.1'
  implementation "com.fasterxml.jackson.core:jackson-databind:2.13.4"
  implementation "com.fasterxml.jackson.core:jackson-annotations:2.13.4"

This PR also addresses tests issues (they were not running and were not passing previously).
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[x] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #546 (4c21b71) into 2.x (28ea7a0) will decrease coverage by 1.51%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                2.x     #546      +/-   ##
============================================
- Coverage     74.27%   72.75%   -1.52%
- Complexity      383      391       +8
============================================
  Files            94       64      -30
  Lines          4050     2103    -1947
  Branches        649      229     -420
============================================
- Hits           3008     1530    -1478
+ Misses          908      443     -465
+ Partials        134      130       -4

Flag
Coverage Δ

dashboards-reports
?

reports-scheduler
72.75% <ø> (+0.57%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

dashboards-reports/server/model/index.ts

...ic/components/main/loading_modal/loading_modal.tsx

...s-reports/server/routes/utils/dataReportHelpers.ts

...ions/report_settings/report_settings_constants.tsx

...itions/report_trigger/report_trigger_constants.tsx

...eports/public/components/utils/settings_service.ts

...main/__tests__/__utils__/main_utils_test_utils.tsx

dashboards-reports/test/httpMockClient.js

...ards-reports/public/components/main/main_utils.tsx

...itions/report_settings/report_settings_helpers.tsx

... and 22 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


website: show TypeScript information for packages that have DefinitelyTyped type definitions

Summary
Packages may not have bundled TypeScript type information, but someone may have done the hard work of creating the type declarations and publishing them to Definitely Typed .
For packages with DefinitelyTyped type definitions, we will decorate the package on the website to indicate that it has TypeScript type declarations available in DefinitelyTyped, with a link to the package.
Intended Outcome

Users get a link to the DefinitelyTyped package that provides type declarations

This feature is live!!!
https://github.blog/changelog/2021-01-25-npm-displays-links-to-definitelytyped-packages/

This feature is live!!!
https://github.blog/changelog/2021-01-25-npm-displays-links-to-definitelytyped-packages/


Suggestion about is_valid function of LineBuffer struct

A little suggestion. It seems like first check and second check do the same check implicitly.
Is there a case where the first check is false and the second true ? What do you think ?
pub fn is_valid(&self) -> bool {
        self.lines.is_char_boundary(self.insertion_point())  // first check
            && (self
                .lines
                .grapheme_indices(true)
                .any(|(i, _)| i == self.insertion_point())
                || self.insertion_point() == self.lines.len())    // second check
            && std::str::from_utf8(self.lines.as_bytes()).is_ok()
    }

You are correct that if the second test (grapheme cluster boundary) is true the first one (UTF-8 char boundary) will always be true.
The first check is local and only needs to check the byte at the index (or length), while the follow-on second check requires us to traverse the string. Without benchmarking with real data (that is most likely valid), there could be a questionable argument about short-circuiting saving us the work for the second check. Given that we expect valid data anyways, we could probably elide the first check.


bugfix: missing classtransformer upgrade and proper model types

Issue ARCH-1058
Description of changes:
Updating the template for the model to conform to the new type coming from class transformer as well as the codegen and package.json templates to match the new versions.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I think #130 should fix it.

I think #130 should fix it.

oh hahaha I just did the same thing

@HeatherFlux there is one more fix you will need for the macOS builds in that PR.  Then I can close my PR

Should be all good now

@kddejong any update on this getting merged?


Extra double quote in Prom read/write page

URL for relevant page?
https://docs.influxdata.com/influxdb/v1.6/supported_protocols/prometheus
Note: The Prometheus API Stability Guarantees states that remote read and remote write endpoints are features listed as experimental or subject to change, and thus considered unstable for 2.x”. Any breaking changes will be included in the InfluxDB release notes.

Fixed in master -- change to appear in next build.


AssertionError: Exception encountered when calling layer "gradient_boosted_trees_model" (type GradientBoostedTreesModel).

When trying to get a prediction I am getting the error in the title, it also gives the following:
in user code:

    File "/home/laner107/.local/lib/python3.8/site-packages/tensorflow_decision_forests/keras/core.py", line 791, in call  *
        normalized_inputs = self._build_normalized_inputs(inputs)
    File "/home/laner107/.local/lib/python3.8/site-packages/tensorflow_decision_forests/keras/core.py", line 747, in _build_normalized_inputs  *
        assert len(self._semantics) == 1

    AssertionError:

Call arguments received:
  • inputs=tf.Tensor(shape=(14,), dtype=float32)
  • training=False

The following is where I call the prediction:
  def predict_to_data(self):
        testing_data = test_preprocess()
        testing_data = np.array(testing_data)
        predicitions = self.model(testing_data[0])

here is what the data im passing in looks like:
[0.484375   0.83007665 0.56508876 0.46099291 0.52793453 0.75438596
 0.52066116 0.7826087  0.         0.65852121 0.40425532 0.58974359
 0.69047619 0.37058824]

and here is the architecture for the model:
def create_single_model(self):
        input_features = tf.keras.Input(shape=(self.num_features,))

        # bootstrap_size_ratio: Number of examples used to train each trees; expressed as a ratio of the training dataset size. Default: 1.0.
        rf_model_1 = tfdf.keras.GradientBoostedTreesModel(
            verbose=0,
            task=tfdf.keras.Task.CLASSIFICATION,
            hyperparameter_template="benchmark_rank1@v1",
            num_trees=self.num_of_trees,
        )

        model = tf.keras.models.Model(input_features, rf_model_1(input_features))

        return model

For now I was just going to generate one prediction to see what the output was, eventually though I plan on doing it in batches just have to figure that out, any idea why this error is occurring?
All of this is done after the GradientBoostedTreesModel is trained(fit) and evaluated using validation data.

@laneciar can you please tell why the training parameter is set false in your code, also can you try using model.evaluate() or model.predict() to see the test set reults?

@Cheril311 I tried using .predict() as well and same issue happened, the training parameter is disabled because the model is no longer needing to be trained, this is after the model is trained and evaluated based of valuation data, I am now wanting to plot out the models predications to a file based off of more data the model has never seen.

@laneciar can you please share your entire training and testing data so that I can try reproducing the issue?

Here is 100 rows of training data, theres to much to send the full thing, just take off the last 25 or so rows for predicting, you will also want to cut the last column of those 25 since thats the label and we are wanting it to predict a label.
[
0,0.4769230769230769,0.4520547945205479,0.5110942704569856,0.37681159420289856,0.5118040925303997,0.2903225806451613,0.4552845528455285,0.4565217391304348,0.488095238095238,0.5247706422018349,0.7608695652173914,0.5,0.5301204819277108,0.3727810650887574,3.0
1,0.7384615384615384,0.7625570776255708,0.5710156676829771,0.8695652173913043,0.5952399824784605,0.7258064516129032,0.7560975609756098,0.5869565217391304,0.619047619047619,0.7394495412844038,0.4782608695652173,0.8289473684210527,0.8554216867469878,0.8284023668639053,1.0
2,0.5230769230769231,0.4277016742770167,0.5146190585291028,0.32608695652173914,0.5101353747314386,0.46774193548387094,0.4552845528455285,0.4130434782608694,0.5238095238095238,0.6293577981651377,0.7173913043478262,0.3815789473684211,0.48192771084337344,0.4615384615384615,4.0
3,0.3384615384615385,0.3318112633181126,0.4405985090146428,0.32608695652173914,0.4533989695667572,0.38709677419354843,0.3577235772357724,0.3913043478260869,0.38095238095238093,0.3504587155963303,0.7608695652173914,0.368421052631579,0.32530120481927705,0.3372781065088757,5.0
4,0.3384615384615385,0.4140030441400304,0.5040446943127513,0.2681159420289855,0.5034605035355937,0.2419354838709677,0.38211382113821146,0.34782608695652173,0.3333333333333333,0.21284403669724772,0.673913043478261,0.4210526315789474,0.3855421686746987,0.3136094674556213,5.0
5,1.0,0.6788432267884322,0.9058705345341056,0.9855072463768115,1.0,0.870967741935484,0.9268292682926831,1.0,0.9761904761904762,0.8477064220183487,0.0,1.0,0.6867469879518071,0.9349112426035503,1.0
6,0.29230769230769227,0.2998477929984779,0.48642075395216566,0.3550724637681159,0.5017917857366324,0.2419354838709677,0.3252032520325204,0.17391304347826075,0.4285714285714285,0.3559633027522936,0.4130434782608696,0.2894736842105263,0.2650602409638554,0.43786982248520706,2.0
7,0.6615384615384615,0.4672754946727549,0.5428173631060399,0.5217391304347826,0.545178448509624,0.596774193548387,0.6097560975609757,0.7826086956521738,0.6428571428571428,0.7192660550458716,0.5434782608695652,0.4736842105263158,0.6626506024096385,0.47337278106508873,4.0
8,0.36923076923076925,0.4748858447488584,0.47937117780793137,0.30434782608695654,0.5017917857366324,0.629032258064516,0.2682926829268293,0.5434782608695652,0.40476190476190477,0.4073394495412844,0.6521739130434783,0.21052631578947367,0.32530120481927705,0.2781065088757396,3.0
9,0.6,0.6438356164383561,0.5992139722599142,0.5869565217391304,0.5977430591769023,0.596774193548387,0.6422764227642277,0.7608695652173911,0.5833333333333333,0.5412844036697249,0.4565217391304348,0.7894736842105263,0.6626506024096385,0.5680473372781065,3.0
10,0.8615384615384615,0.7777777777777777,0.6415114291253199,0.8188405797101449,0.6311174151561266,0.9193548387096775,0.910569105691057,0.7826086956521738,0.7619047619047619,0.7174311926605506,0.21739130434782605,0.8947368421052633,0.8915662650602408,0.7928994082840237,4.0
11,0.6307692307692309,0.7656012176560121,0.5674908796108599,0.5797101449275363,0.5643687031976781,0.6774193548387097,0.6747967479674797,0.7608695652173911,0.6071428571428571,0.6422018348623854,0.3695652173913043,0.6842105263157895,0.7108433734939759,0.6568047337278107,3.0
12,0.7076923076923077,0.4809741248097412,0.5251934227454542,0.5869565217391304,0.5476815252080659,0.6451612903225807,0.5934959349593496,0.6956521739130435,0.7023809523809523,0.8275229357798166,0.6956521739130435,0.5657894736842106,0.6024096385542168,0.6035502958579881,1.0
13,0.6923076923076924,0.5585996955859969,0.620362700692617,0.7971014492753623,0.6227738261613206,0.6129032258064515,0.8048780487804879,0.34782608695652173,0.6785714285714285,0.6201834862385321,0.4347826086956522,0.7631578947368421,0.7108433734939759,0.9408284023668639,3.0
14,0.5692307692307693,0.669710806697108,0.5428173631060399,0.5942028985507246,0.6019148536743054,0.43548387096774194,0.5447154471544716,0.5434782608695652,0.5595238095238095,0.5486238532110093,0.826086956521739,0.5657894736842106,0.46987951807228906,0.5976331360946745,4.0
15,0.6615384615384615,0.578386605783866,0.5851148199714457,0.7101449275362319,0.6035835714732666,0.49999999999999994,0.6260162601626017,0.6304347826086956,0.6428571428571428,0.655045871559633,0.6304347826086956,0.4868421052631579,0.5301204819277108,0.7337278106508875,4.0
16,0.5384615384615384,0.3987823439878234,0.521668634673337,0.5144927536231884,0.5435097307106628,0.564516129032258,0.5121951219512195,0.34782608695652173,0.5238095238095238,0.5743119266055046,0.7826086956521738,0.5263157894736843,0.4457831325301204,0.46745562130177515,3.0
17,0.6461538461538462,0.6894977168949771,0.5851148199714457,0.6014492753623188,0.5702092154940422,0.564516129032258,0.7154471544715448,0.6739130434782608,0.619047619047619,0.5944954128440367,0.4347826086956522,0.7631578947368421,0.6987951807228915,0.6153846153846154,3.0
18,0.5846153846153846,0.8082191780821917,0.5674908796108599,0.6666666666666666,0.6019148536743054,0.49999999999999994,0.6097560975609757,0.4782608695652173,0.5714285714285714,0.5724770642201835,0.7391304347826086,0.5131578947368421,0.5783132530120481,0.680473372781065,3.0
19,0.49230769230769234,0.6453576864535768,0.5005199062406342,0.572463768115942,0.5702092154940422,0.596774193548387,0.47154471544715454,0.5869565217391304,0.5119047619047619,0.46788990825688076,0.8913043478260869,0.5263157894736843,0.4457831325301204,0.43786982248520706,3.0
20,0.24615384615384617,0.4079147640791476,0.4899455420242828,0.2971014492753623,0.47842973655117543,0.3548387096774193,0.38211382113821146,0.34782608695652173,0.13095238095238093,0.06605504587155964,0.5434782608695652,0.5131578947368421,0.49397590361445776,0.3727810650887574,1.0
21,0.4,0.3211567732115677,0.5181438466012199,0.3333333333333333,0.5168102459272834,0.41935483870967744,0.40650406504065045,0.13043478260869557,0.39285714285714285,0.29908256880733947,0.6086956521739131,0.3026315789473685,0.33734939759036137,0.35502958579881655,2.0
22,0.6615384615384615,0.5601217656012176,0.59216439611568,0.5869565217391304,0.5560251142028719,0.6612903225806452,0.7154471544715448,0.4130434782608694,0.5833333333333333,0.708256880733945,0.9782608695652173,0.5526315789473685,0.5903614457831324,0.6863905325443787,3.0
23,0.44615384615384623,0.5996955859969558,0.49699511816851705,0.41304347826086957,0.5226507582236476,0.532258064516129,0.41463414634146345,0.6956521739130435,0.45238095238095233,0.526605504587156,0.8043478260869565,0.4473684210526316,0.5180722891566264,0.3431952662721893,3.0
24,0.676923076923077,0.5966514459665144,0.6097883364762656,0.7101449275362319,0.5785528044888484,0.7419354838709677,0.8048780487804879,0.5434782608695652,0.619047619047619,0.5798165137614679,0.5,0.7631578947368421,0.7469879518072288,0.7810650887573964,2.0
25,0.23076923076923073,0.4094368340943683,0.48642075395216566,0.18840579710144928,0.44171794497402866,0.3225806451612903,0.3414634146341464,0.26086956521739124,0.17857142857142855,0.0,0.3695652173913043,0.3026315789473685,0.36144578313253006,0.12426035502958577,5.0
26,0.41538461538461535,0.2161339421613394,0.5181438466012199,0.5072463768115942,0.5418410129117016,0.3225806451612903,0.46341463414634154,0.3695652173913043,0.4285714285714285,0.363302752293578,1.0,0.39473684210526316,0.4216867469879517,0.4970414201183432,2.0
27,0.4769230769230769,0.3987823439878234,0.5357677869618056,0.3333333333333333,0.5059635802340354,0.40322580645161293,0.5203252032520326,0.4347826086956521,0.488095238095238,0.5302752293577981,0.826086956521739,0.4078947368421053,0.5301204819277108,0.42603550295857984,2.0
28,0.3076923076923077,0.3515981735159817,0.47937117780793137,0.28985507246376807,0.5034605035355937,0.41935483870967744,0.27642276422764234,0.23913043478260865,0.3571428571428571,0.28990825688073396,0.6086956521739131,0.27631578947368424,0.21686746987951802,0.22485207100591714,2.0
29,0.5538461538461539,0.4079147640791476,0.46879681359157993,0.5434782608695652,0.5685404976950811,0.629032258064516,0.41463414634146345,0.5434782608695652,0.6428571428571428,0.7376146788990826,0.5,0.4736842105263158,0.43373493975903604,0.5739644970414202,1.0
30,0.46153846153846156,0.4429223744292237,0.49699511816851705,0.39855072463768115,0.5201476815252059,0.3709677419354838,0.3658536585365854,0.6304347826086956,0.47619047619047616,0.3577981651376147,0.673913043478261,0.3815789473684211,0.32530120481927705,0.21893491124260356,3.0
31,0.4769230769230769,0.5616438356164384,0.49699511816851705,0.36231884057971014,0.5101353747314386,0.532258064516129,0.39837398373983746,0.30434782608695643,0.47619047619047616,0.5412844036697249,1.0,0.4473684210526316,0.33734939759036137,0.2662721893491124,4.0
32,0.6615384615384615,0.6773211567732116,0.6168379126204999,0.6956521739130435,0.6202707494628787,0.596774193548387,0.7073170731707318,0.6521739130434783,0.6309523809523809,0.5431192660550459,0.6086956521739131,0.6447368421052632,0.5542168674698794,0.757396449704142,3.0
33,0.2615384615384616,0.3866057838660578,0.4828959658800485,0.3695652173913043,0.49511691454078754,0.2741935483870968,0.3739837398373984,0.26086956521739124,0.17857142857142855,0.04220183486238532,0.5869565217391304,0.4078947368421053,0.3975903614457831,0.47337278106508873,5.0
34,0.5538461538461539,0.3013698630136986,0.5110942704569856,0.5289855072463768,0.5418410129117016,0.40322580645161293,0.4552845528455285,0.5869565217391304,0.5595238095238095,0.6477064220183486,0.8043478260869565,0.35526315789473684,0.45783132530120474,0.5857988165680473,4.0
35,0.5846153846153846,0.6514459665144596,0.5569165153945085,0.6739130434782609,0.6144302371665145,0.41935483870967744,0.5853658536585367,0.3695652173913043,0.5357142857142857,0.5743119266055046,0.4130434782608696,0.605263157894737,0.542168674698795,0.6449704142011834,3.0
36,0.3538461538461538,0.4642313546423135,0.49347033009639996,0.43478260869565216,0.5226507582236476,0.2741935483870968,0.3902439024390244,0.23913043478260865,0.4285714285714285,0.3100917431192661,0.3913043478260869,0.4342105263157895,0.2771084337349397,0.5207100591715976,2.0
37,0.8153846153846155,0.9299847792998477,0.6274122768368513,0.7463768115942029,0.6369579274524909,0.6451612903225807,0.7479674796747968,1.0,0.9047619047619048,0.8935779816513763,0.26086956521739135,0.868421052631579,0.7108433734939759,0.7869822485207101,1.0
38,0.8923076923076924,0.8249619482496194,0.5780652438272114,0.9130434782608696,0.5868963934836544,0.967741935483871,0.943089430894309,0.7608695652173911,0.6904761904761905,0.6844036697247706,0.3913043478260869,0.8947368421052633,0.8674698795180722,1.0,4.0
39,0.4307692307692308,0.6879756468797564,0.5005199062406342,0.4492753623188406,0.5184789637262446,0.48387096774193544,0.4471544715447155,0.4347826086956521,0.4285714285714285,0.29541284403669726,0.5652173913043479,0.5526315789473685,0.5301204819277108,0.3431952662721893,2.0
40,0.4,0.2678843226788432,0.41240020443770564,0.2971014492753623,0.4266994847833777,0.1935483870967742,0.2682926829268293,0.5217391304347825,0.4285714285714285,0.3339449541284404,0.4130434782608696,0.1842105263157895,0.253012048192771,0.30177514792899407,5.0
41,0.5384615384615384,0.5296803652968036,0.5498669392502742,0.4492753623188406,0.5309943472184537,0.532258064516129,0.5691056910569107,0.6086956521739131,0.5595238095238095,0.5211009174311927,0.9782608695652173,0.5394736842105263,0.5542168674698794,0.514792899408284,3.0
42,0.7230769230769231,0.7503805175038052,0.6133131245483827,0.6666666666666666,0.6102584426691114,0.6612903225806452,0.7154471544715448,0.6086956521739131,0.6666666666666666,0.6844036697247706,0.3913043478260869,0.6578947368421053,0.7349397590361444,0.5266272189349113,4.0
43,0.7846153846153846,0.8614916286149162,0.5992139722599142,0.717391304347826,0.5952399824784605,1.0,0.7723577235772359,0.9130434782608694,0.6785714285714285,0.7155963302752294,0.3695652173913043,0.8026315789473685,0.7469879518072288,0.7692307692307692,4.0
44,0.7846153846153846,0.7397260273972602,0.6379866410532028,0.7318840579710144,0.6252769028597625,0.7419354838709677,0.7886178861788619,0.6086956521739131,0.6071428571428571,0.6623853211009174,0.26086956521739135,0.8289473684210527,0.8554216867469878,0.7751479289940828,4.0
45,0.8307692307692308,0.9497716894977168,0.6027387603320313,0.8478260869565217,0.6119271604680727,0.6612903225806452,0.845528455284553,0.3913043478260869,0.7142857142857143,0.634862385321101,0.19565217391304346,0.868421052631579,0.6987951807228915,0.8757396449704142,4.0
46,0.8461538461538461,0.817351598173516,0.6062635484041485,0.8840579710144927,0.6227738261613206,0.7741935483870968,0.853658536585366,0.3695652173913043,0.7738095238095237,0.7834862385321102,0.5217391304347827,0.8289473684210527,0.8915662650602408,0.9112426035502958,4.0
47,0.6,0.6742770167427701,0.5463421511781571,0.6884057971014492,0.5618656264992362,0.5806451612903225,0.6341463414634148,0.5434782608695652,0.5833333333333333,0.6110091743119266,0.5434782608695652,0.6842105263157895,0.6987951807228915,0.5976331360946745,1.0
48,0.6153846153846154,0.639269406392694,0.5815900318993285,0.7246376811594203,0.6269456206587236,0.7903225806451613,0.6341463414634148,0.4565217391304348,0.5952380952380952,0.7431192660550459,1.0,0.6710526315789473,0.6385542168674698,0.7159763313609467,1.0
49,0.36923076923076925,0.3668188736681887,0.4899455420242828,0.40579710144927533,0.5226507582236476,0.3064516129032258,0.3577235772357724,0.4565217391304348,0.41666666666666663,0.29357798165137616,1.0,0.4078947368421053,0.19277108433734935,0.43786982248520706,2.0
50,0.5692307692307693,0.5829528158295281,0.6062635484041485,0.5652173913043478,0.5810558811872901,0.46774193548387094,0.6422764227642277,0.4782608695652173,0.5595238095238095,0.5908256880733945,0.6956521739130435,0.6315789473684211,0.5662650602409638,0.6627218934911243,1.0
51,0.5384615384615384,0.4444444444444444,0.5428173631060399,0.4855072463768116,0.5418410129117016,0.2741935483870968,0.5284552845528456,0.3695652173913043,0.5238095238095238,0.5229357798165137,0.5869565217391304,0.4736842105263158,0.49397590361445776,0.5088757396449703,3.0
52,0.5384615384615384,0.2846270928462709,0.5533917273223914,0.5289855072463768,0.5518533197054689,0.564516129032258,0.5447154471544716,0.13043478260869557,0.5595238095238095,0.42201834862385323,0.673913043478261,0.4736842105263158,0.36144578313253006,0.6331360946745562,3.0
53,0.36923076923076925,0.4490106544901065,0.4758463897358142,0.3695652173913043,0.5034605035355937,0.5483870967741935,0.40650406504065045,0.5434782608695652,0.40476190476190477,0.5486238532110093,0.9782608695652173,0.5394736842105263,0.5542168674698794,0.4497041420118343,5.0
54,0.7538461538461539,0.5068493150684932,0.6168379126204999,0.7753623188405797,0.6144302371665145,0.7258064516129032,0.8048780487804879,0.7826086956521738,0.7142857142857143,0.5743119266055046,0.7391304347826086,0.5789473684210527,0.6144578313253011,0.7278106508875739,3.0
55,0.4307692307692308,0.4307458143074581,0.5146190585291028,0.36231884057971014,0.5143071692288416,0.3387096774193548,0.46341463414634154,0.19565217391304346,0.45238095238095233,0.30642201834862387,0.4782608695652173,0.4868421052631579,0.43373493975903604,0.5976331360946745,2.0
56,0.7846153846153846,0.7199391171993912,0.6415114291253199,0.8188405797101449,0.6369579274524909,0.8387096774193548,0.8617886178861789,0.6304347826086956,0.7142857142857143,0.726605504587156,0.34782608695652173,0.75,0.7710843373493974,0.7159763313609467,4.0
57,0.46153846153846156,0.5585996955859969,0.49699511816851705,0.3913043478260869,0.5143071692288416,0.7419354838709677,0.3739837398373984,0.4347826086956521,0.47619047619047616,0.42568807339449544,0.5,0.34210526315789475,0.4096385542168674,0.24260355029585795,3.0
58,0.46153846153846156,0.4094368340943683,0.4652720255194628,0.3188405797101449,0.47008614755636935,0.48387096774193544,0.39837398373983746,0.7391304347826086,0.4642857142857143,0.6220183486238533,0.7391304347826086,0.4210526315789474,0.3855421686746987,0.4556213017751479,4.0
59,0.5538461538461539,0.7488584474885844,0.5745404557550942,0.5434782608695652,0.5643687031976781,0.3387096774193548,0.6260162601626017,0.6956521739130435,0.5238095238095238,0.4954128440366973,0.4782608695652173,0.6710526315789473,0.5542168674698794,0.5266272189349113,3.0
60,0.7230769230769231,0.6529680365296804,0.6838088859907256,0.717391304347826,0.660319976637948,0.3548387096774193,0.7317073170731708,0.6956521739130435,0.6785714285714285,0.6036697247706422,0.4130434782608696,0.7894736842105263,0.7469879518072288,0.6686390532544378,4.0
61,0.8153846153846155,0.730593607305936,0.620362700692617,0.9057971014492754,0.6227738261613206,0.7903225806451613,0.910569105691057,0.5434782608695652,0.75,0.5724770642201835,0.17391304347826086,0.9473684210526316,0.7590361445783131,0.8402366863905325,2.0
62,0.8307692307692308,0.7047184170471841,0.6767593098464914,0.9710144927536232,0.6770071546275601,0.9193548387096775,1.0,0.3695652173913043,0.7738095238095237,0.5431192660550459,0.32608695652173914,0.855263157894737,0.7710843373493974,0.9467455621301775,5.0
63,0.32307692307692315,0.2191780821917808,0.40535062829347135,0.2608695652173913,0.41168102459272676,0.46774193548387094,0.27642276422764234,0.30434782608695643,0.40476190476190477,0.44036697247706424,0.23913043478260865,0.14473684210526316,0.32530120481927705,0.2781065088757396,3.0
64,0.9076923076923077,0.7427701674277016,0.6274122768368513,0.9420289855072463,0.6269456206587236,0.8064516129032258,1.0,0.6956521739130435,0.7976190476190476,0.691743119266055,0.26086956521739135,0.9210526315789475,0.9999999999999999,0.9171597633136095,4.0
65,0.49230769230769234,0.4611872146118721,0.4899455420242828,0.4855072463768116,0.5418410129117016,0.3225806451612903,0.40650406504065045,0.17391304347826075,0.5,0.3779816513761468,0.7173913043478262,0.39473684210526316,0.2650602409638554,0.6272189349112426,5.0
66,0.15384615384615385,0.1887366818873668,0.4828959658800485,0.22463768115942026,0.4934481967418264,0.2258064516129032,0.21138211382113825,0.28260869565217384,0.23809523809523808,0.26422018348623855,0.30434782608695654,0.21052631578947367,0.15662650602409633,0.35502958579881655,5.0
67,0.5692307692307693,0.7077625570776256,0.5392925750339228,0.5072463768115942,0.5435097307106628,0.6612903225806452,0.5853658536585367,0.3913043478260869,0.5357142857142857,0.5889908256880734,0.4347826086956522,0.868421052631579,0.6265060240963854,0.47337278106508873,3.0
68,0.23076923076923073,0.2389649923896499,0.4405985090146428,0.08695652173913043,0.4333743559792226,0.3225806451612903,0.1951219512195122,0.23913043478260865,0.2976190476190476,0.42201834862385323,0.5652173913043479,0.11842105263157893,0.253012048192771,0.17751479289940825,2.0
69,0.32307692307692315,0.5585996955859969,0.5146190585291028,0.3550724637681159,0.5084666569324773,0.3709677419354838,0.47154471544715454,0.4130434782608694,0.23809523809523808,0.28623853211009176,0.673913043478261,0.5394736842105263,0.49397590361445776,0.46745562130177515,1.0
70,0.3076923076923077,0.2998477929984779,0.5075694823848685,0.2971014492753623,0.4917794789428651,0.41935483870967744,0.40650406504065045,0.23913043478260865,0.22619047619047616,0.08440366972477065,0.5869565217391304,0.34210526315789475,0.3493975903614457,0.40828402366863903,5.0
71,0.13846153846153847,0.2800608828006088,0.37715232371653423,0.1159420289855072,0.4141841012911686,0.14516129032258063,0.12195121951219512,0.19565217391304346,0.2738095238095238,0.3504587155963303,0.23913043478260865,0.15789473684210525,0.21686746987951802,0.14792899408284022,2.0
72,0.6461538461538462,0.4657534246575342,0.5428173631060399,0.7608695652173914,0.5685404976950811,0.596774193548387,0.6585365853658538,0.5652173913043477,0.619047619047619,0.5651376146788991,0.7826086956521738,0.6315789473684211,0.5301204819277108,0.7928994082840237,5.0
73,0.6923076923076924,0.8538812785388128,0.5710156676829771,0.717391304347826,0.6060866481717084,0.6935483870967742,0.6747967479674797,0.6304347826086956,0.6309523809523809,0.6880733944954129,0.6304347826086956,0.6710526315789473,0.6867469879518071,0.6863905325443787,4.0
74,0.5230769230769231,0.4916286149162861,0.5251934227454542,0.43478260869565216,0.5284912705200119,0.43548387096774194,0.5203252032520326,0.4565217391304348,0.5238095238095238,0.5889908256880734,0.7826086956521738,0.5394736842105263,0.5542168674698794,0.4437869822485207,3.0
75,0.6307692307692309,0.578386605783866,0.5357677869618056,0.572463768115942,0.5868963934836544,0.43548387096774194,0.5691056910569107,0.4782608695652173,0.5476190476190476,0.6660550458715597,0.4565217391304348,0.5657894736842106,0.7108433734939759,0.591715976331361,4.0
76,0.8461538461538461,0.7092846270928462,0.5992139722599142,0.8768115942028986,0.6186020316639176,0.8225806451612903,0.8048780487804879,0.7391304347826086,0.75,0.726605504587156,0.4130434782608696,0.8421052631578948,0.8192771084337348,0.8284023668639053,1.0
77,0.5076923076923078,0.5083713850837138,0.5287182108175713,0.5434782608695652,0.5585281909013138,0.43548387096774194,0.49593495934959353,0.3913043478260869,0.5119047619047619,0.41467889908256883,0.5869565217391304,0.4342105263157895,0.4216867469879517,0.5739644970414202,3.0
78,0.46153846153846156,0.5555555555555556,0.5287182108175713,0.6884057971014492,0.5727122921924841,0.48387096774193544,0.5365853658536586,0.13043478260869557,0.488095238095238,0.47522935779816516,0.4565217391304348,0.5526315789473685,0.46987951807228906,0.6568047337278107,2.0
79,0.7538461538461539,0.3866057838660578,0.6591353694859057,0.8768115942028986,0.660319976637948,0.6774193548387097,0.853658536585366,0.4782608695652173,0.738095238095238,0.4935779816513762,0.8478260869565217,0.5657894736842106,0.5301204819277108,0.9349112426035503,5.0
80,0.41538461538461535,0.2009132420091324,0.47937117780793137,0.3695652173913043,0.5101353747314386,0.41935483870967744,0.3008130081300813,0.30434782608695643,0.44047619047619047,0.41467889908256883,0.28260869565217395,0.1842105263157895,0.253012048192771,0.272189349112426,3.0
81,0.5538461538461539,0.4094368340943683,0.5463421511781571,0.5869565217391304,0.5727122921924841,0.6129032258064515,0.5365853658536586,0.4782608695652173,0.5476190476190476,0.42385321100917434,0.6086956521739131,0.4342105263157895,0.43373493975903604,0.5266272189349113,3.0
82,0.5230769230769231,0.5555555555555556,0.5075694823848685,0.41304347826086957,0.5001230679376713,0.6935483870967742,0.4552845528455285,0.5217391304347825,0.5119047619047619,0.5394495412844037,0.5217391304347827,0.39473684210526316,0.46987951807228906,0.378698224852071,4.0
83,0.5230769230769231,0.4764079147640791,0.5322429988896885,0.5217391304347826,0.545178448509624,0.49999999999999994,0.5121951219512195,0.5434782608695652,0.5357142857142857,0.5376146788990825,0.8478260869565217,0.46052631578947373,0.48192771084337344,0.272189349112426,3.0
84,0.6923076923076924,0.715372907153729,0.620362700692617,0.8623188405797102,0.6770071546275601,0.8064516129032258,0.7479674796747968,0.5652173913043477,0.5833333333333333,0.6293577981651377,0.4130434782608696,0.6973684210526316,0.7710843373493974,1.0,3.0
85,0.7538461538461539,0.776255707762557,0.5604413034666257,0.6739130434782609,0.5685404976950811,0.870967741935484,0.6910569105691058,0.5869565217391304,0.7142857142857143,0.7651376146788991,0.6086956521739131,0.6710526315789473,0.7349397590361444,0.591715976331361,1.0
86,0.5692307692307693,0.5083713850837138,0.5287182108175713,0.5289855072463768,0.5476815252080659,0.49999999999999994,0.5203252032520326,0.30434782608695643,0.5595238095238095,0.5633027522935781,0.5217391304347827,0.5394736842105263,0.45783132530120474,0.6449704142011834,3.0
87,0.6,0.4764079147640791,0.5498669392502742,0.5434782608695652,0.5535220375044301,0.6774193548387097,0.5853658536585367,0.6086956521739131,0.5833333333333333,0.6146788990825689,0.7173913043478262,0.5657894736842106,0.48192771084337344,0.5266272189349113,1.0
88,0.5076923076923078,0.6727549467275494,0.6133131245483827,0.6159420289855072,0.5893994701820963,0.7096774193548387,0.7398373983739839,0.4130434782608694,0.44047619047619047,0.42752293577981654,0.5652173913043479,0.7236842105263158,0.6867469879518071,0.5857988165680473,1.0
89,0.32307692307692315,0.2161339421613394,0.5110942704569856,0.28985507246376807,0.5184789637262446,0.43548387096774194,0.33333333333333337,0.326086956521739,0.488095238095238,0.23669724770642203,0.4130434782608696,0.27631578947368424,0.15662650602409633,0.30177514792899407,2.0
90,0.4,0.5205479452054794,0.48642075395216566,0.3550724637681159,0.5059635802340354,0.3709677419354838,0.34959349593495936,0.4565217391304348,0.44047619047619047,0.3743119266055046,0.6304347826086956,0.39473684210526316,0.28915662650602403,0.30769230769230765,3.0
91,0.6615384615384615,0.5357686453576864,0.6133131245483827,0.7318840579710144,0.6019148536743054,0.6451612903225807,0.7642276422764229,0.6304347826086956,0.619047619047619,0.5467889908256881,0.3695652173913043,0.6710526315789473,0.6024096385542168,0.7869822485207101,2.0
92,0.5076923076923078,0.4322678843226788,0.521668634673337,0.572463768115942,0.5535220375044301,0.43548387096774194,0.49593495934959353,0.6521739130434783,0.5119047619047619,0.42385321100917434,0.7826086956521738,0.5131578947368421,0.4216867469879517,0.5621301775147929,3.0
93,0.6153846153846154,0.6331811263318112,0.5428173631060399,0.5072463768115942,0.5435097307106628,0.629032258064516,0.5609756097560976,0.5652173913043477,0.5833333333333333,0.671559633027523,0.7173913043478262,0.46052631578947373,0.5301204819277108,0.4023668639053254,4.0
94,0.5230769230769231,0.4748858447488584,0.5287182108175713,0.43478260869565216,0.5268225527210507,0.49999999999999994,0.5121951219512195,0.5,0.5119047619047619,0.41284403669724773,0.8478260869565217,0.5,0.45783132530120474,0.514792899408284,3.0
95,0.44615384615384623,0.3911719939117199,0.37715232371653423,0.30434782608695654,0.4166871779896104,0.46774193548387094,0.28455284552845533,0.30434782608695643,0.5,0.5376146788990825,0.5,0.32894736842105265,0.3012048192771084,0.3727810650887574,4.0
96,0.41538461538461535,0.5068493150684932,0.5110942704569856,0.427536231884058,0.5226507582236476,0.38709677419354843,0.4227642276422765,0.3695652173913043,0.4285714285714285,0.24220183486238533,0.8043478260869565,0.3815789473684211,0.3855421686746987,0.4497041420118343,5.0
97,0.6923076923076924,0.5677321156773212,0.5498669392502742,0.7101449275362319,0.5585281909013138,0.6935483870967742,0.7235772357723578,0.7173913043478259,0.5952380952380952,0.6422018348623854,0.8043478260869565,0.605263157894737,0.7831325301204819,0.9289940828402367,3.0
98,0.7076923076923077,0.700152207001522,0.5851148199714457,0.681159420289855,0.5852276756846932,0.7258064516129032,0.6910569105691058,0.6739130434782608,0.6666666666666666,0.6201834862385321,0.9130434782608696,0.5657894736842106,0.5180722891566264,0.5443786982248521,4.0
]

The full error code when using a tf data set of np.float32:
Traceback (most recent call last):
  File "Train.py", line 132, in <module>
    new_model.predict_to_data()
  File "Train.py", line 88, in predict_to_data
    predicitions = self.single_model(data_tf, training=False)
  File "/home/laner107/.local/lib/python3.8/site-packages/keras/utils/traceback_utils.py", line 67, in error_handler
    raise e.with_traceback(filtered_tb) from None
  File "/home/laner107/.local/lib/python3.8/site-packages/tensorflow/python/framework/func_graph.py", line 1147, in autograph_handler
    raise e.ag_error_metadata.to_exception(e)
AssertionError: Exception encountered when calling layer "gradient_boosted_trees_model" (type GradientBoostedTreesModel).

in user code:

    File "/home/laner107/.local/lib/python3.8/site-packages/tensorflow_decision_forests/keras/core.py", line 791, in call  *
        normalized_inputs = self._build_normalized_inputs(inputs)
    File "/home/laner107/.local/lib/python3.8/site-packages/tensorflow_decision_forests/keras/core.py", line 747, in _build_normalized_inputs  *
        assert len(self._semantics) == 1

    AssertionError:

Call arguments received:
  • inputs=tf.Tensor(shape=(27165, 14), dtype=float32)
  • training=False

facing the exact same problem

@laneciar @aioneersPC I am able to train the model and get the test prediction outputs this way, sorry for the late reply I was busy with my exams. Note that I have used just 6 examples for training thus the accuracy is 0

@laneciar now I get your error, if I don't have test labels, the model throws an error like you mentioned, a simple workaround can be setting up a dummy label column in the testing data and then predicting it, however this does seem to be a bug @achoum

Could you put up an example for a regression task as well.
Also what about the case when we don't have an actual. Is there no way now besides setting a dummy variable for the label?

@aioneersPC sure, will post it

@aioneersPC you can see the examples for regression and predicting on a test set without labels here

yes, thanks to the sample code provided by @Cheril184.
works like a charm

Btw sorry for posting from 2 different accounts I had forgotten my earlier password (for Cheril311). Telling this as I do not know if it is okay for posting through different accounts.

@laneciar please close the issue if you're satisfied with the answer


TextMetrics.wordWrap() customization of splitting method

Description of change
As requested in #6201, made it so that the method used to split a token into characters in TextMetrics.wordWrap() can be customized.
Pre-Merge Checklist

[X] Lint process passed (npm run lint)
[X] Tests passed (npm run test)

Codecov Report

Merging #6251 into dev will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev    #6251      +/-   ##
==========================================
+ Coverage   76.71%   76.71%   +<.01%
==========================================
  Files         202      202
  Lines       10238    10240       +2
==========================================
+ Hits         7854     7856       +2
  Misses       2384     2384

Impacted Files
Coverage Δ

packages/text/src/TextMetrics.js
95.21% <100%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 52df29c...0739886. Read the comment docs.

Thanks for pulling this together.
I don't have a problem with this functionally, but wondered if we could do something a little simpler for the API.
PIXI.TextMetrics.configuration.wordWrapTokenSplit is rather verbose. Also we don't have other examples in the API that use a configuration object. My preference would be to do something like TextMetrics.wordWrapSplit and specify in the docs that it can be overridden to support emojis (for example). An example of how to support emojis would also be excellent thing to document, since that's the main use-case.

For reference, canBreakWords and canBreakChars are other examples of code split out into functions that can be overridden; in those cases to help with custom CJK language support.

@themoonrat I am happy to change things to better fit the rest of the code :).
If I understand correctly, the way you'd like this to be used would be in essence:
import * as PIXI from 'pixi.js`;

// If this is not done then the default implementation is used
PIXI.TextMetrics.wordWrapSplit = <put your own custom implementation here>;

That sounds fair. My personal preference is a configuration variable, just like the way I implemented it, but I am not here to impose my preferences on others :D. I'll update the PR soonish!

@themoonrat I've got one question though: you mentioned canBreakWords and canBreakChars but those are marked as @private and thus do not appear in the documentation (at least not when I run npm docs, though I don't see them in the official documentation either). Is it intended or an oversight? I feel like it'd make more sense to expose them in the docs.
And how about changing the line "This method exists to be easily overriden" to something like "Overridable helper method used internally by TextMetrics, exposed to allow customizing the class's behavior"

Yeah, I'd say it's a mistake that they are marked as private! Your changes would be appreciated :)

Pushed the changes - npm run lint and npm run test both pass (with the exceptions of the tests that fail on Windows)

Do you want to join PixiJS Slack? I can sent an invite to your public email on your site.

Sure! I was hoping to contribute a bit more, as long as life allows, so that might be helpful.

<insert a generic excuse about sleep deprivation/doing fast too quickly/something here>
Fixed those!

You’re on a serious roll this release @EvidentlyCube!

Let's see how much more I can fit before the next release 🙃. And if it's around the corner - the one after that!


Comments wrongly converted in Definition List and Paragraphs

Within a definition list (<DL></DL>) in a markdown document, Hugo converts comments (<!-- -->) to <!– –>. Outside the list scope the comments are treated correctly and left as comments (<!-- -->). This is a bug.
Correction: within any list, including markdown lists demarcated with - or *, not just HTML tagged lists, Hugo renders the the comments incorrectly, turning them into displayed characters.

This should now be fixed in Blackfriday -- and will be part of the next Hugo release.


Implementation advice

Howdy,
I'd like to implement this paper: http://arxiv.org/pdf/1504.03410v1.pdf
The inputs are triplets of images, so my plan is to use Graph to create three identical networks and then tie their weights/params together for every layer, like Francois describes in #783. So first question is: is that the most efficient way to create a shared pipeline that 3 different images will go through?
If so, the final layer would be a merge (concatenate) layer that would emit the three final encodings.
Now, the loss function doesn't need a Y - it uses the information inherent in the ordering of the 3 concatenated outputs. The training data is ordered such that the first of the 3 images should be closer to the second than the third.
So let's say the output is 3 rows of embeddings/hashes as follows:
[embedding_1,
embedding_2,
embedding_3]

The loss function would then be just like eq (2) in the paper:
T.maximum(0 , ||embedding_1 - embedding_2||**2 - ||embedding_1 - embedding_2||**2 + 1)
Interestingly, there is no need for a Y. Is there a natural way to make a loss function that doesn't need a Y, or will Keras always require some Y, even if it is a dummy one?
Thanks.

Similar topic #894

Thanks for the pointer! Sounds like my Graph idea will work - just have to figure out how to do the loss function without Y's.

To make this more concrete, here is the loss function I'd like to use:
import theano.tensor as T

def triplet_loss(y_true, y_pred):
    mse1 = T.sqr(y_pred[0] - y_pred[1]).mean(axis=-1)
    mse2 = T.sqr(y_pred[0] - y_pred[2]).mean(axis=-1)
    return T.maximum(0, mse1 - mse2 + 1)

Note that it doesn't actually make use of y_true, but if you just do that, you get the following Theano error:
UnusedInputError: theano.function was asked to create a function computing outputs given certain inputs, but the provided input variable at index 3 is not part of the computational graph needed to compute the outputs: Elemwise{second,no_inplace}.0.
To make this error into a warning, you can pass the parameter on_unused_input='warn' to theano.function. To disable it completely, use on_unused_input='ignore'.

I'm not sure where to pass the parameter on_unused_input='ignore'. Any ideas?

I'm having a delightful conversation with myself =)
Here's a supremely ugly hack:
def triplet_loss(y_true, y_pred):
    #stuff
    return T.maximum(0, mse1 - mse2 + 1) - y_true[0]*0

Look! y_true is being used!

Another approach was discussed for Siamese networks in this ticket: https://github.com/fchollet/keras/issues/242#issuecomment-114996378. You can probably use a similar implementation for a triplet loss. If you use that approach, be sure to modify the fit function to preserve the triplets.

Thanks @mmmikael !

@sergeyf Hey, may I ask how could you implement the semi-hard choice of negative pairs? It seems that it is not very difficult to implement the triplet loss function but how to choice "good" positive and negative pairs ONLINE bothers me a lot. Could you share your implementation? Thanks!

@sergeyf Is this way work?


Update specification.md

Adding in some notes to clarify/answer comments from GA4GH PRC process

how about now?

Closer, but I don't feel that's quite the right approach.
New components intended to be used by everyone should have input from the group as much as possible, which should be the primary method for new standard components.
The concept of allowing non-standard components is to allow those that wish to express more the ability to do so without waiting for a consensus, and to demonstrate utility.

Hey Ben, cool, please feel free to edit directly, add what you said to PR,
Best,
H.
Sent from my iPhone
On Sep 20, 2018, at 10:40 AM, Ben Hutton<EMAIL_ADDRESS>wrote:
Closer, but I don't feel that's quite the right approach.
New components intended to be used by everyone should have input from the group as much as possible, which should be the primary method for new standard components.
The concept of allowing non-standard components is to allow those that wish to express more the ability to do so without waiting for a consensus, and to demonstrate utility.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@harindra-a I've made a change.
If that reads OK to you, please merge!

oh yeah, that wording is better. merging


replace grub2-install

This is a proof-of-concept showing how we would like grub2-install to behave.

Force pushed to drop the changed timeout, and also to make a small change suggested by Javier by email: we can specify the prefix explicitly, and drop the searching for fs load.cfg.

Thanks for the fixes Lars! I squashed them into the main commit, moved things around and cleaned up a few things I had added in desperation ;) Fingers crossed that it all still works ;)


BeginProduce vs ProduceAsync

I am getting confused when I shall use "ProduceAsync" or "BeginProduce".
If I use "await ProduceAsync(oneMessage)", the method is very slow. It will wait a duration of linger.ms, then only produce one message. It doest't work well with batching
the "BeginProduce" method is fast. However, if kafka broker is down. no exception is throwed. I don't know whether the operation is failed.

[x] Confluent.Kafka nuget version: 1.0-beta2

The ProduceAsync call is only slow because you await it - you don't need to. Note that you might typically want to use this in the context of a web app, where each call may take a few milliseconds, but you can have many of them active simultaneously.
The DeliveryReportResult object provided in the BeginProduce callback has an Error property - you can check that for errors.

I found a bug. When I enable idempotence and set max.inflight to 5, it will throw exception. Then I can only set it to 4.

Moreover, the consume result after seek() sometimes is not what I seek.

what is the exception? note that the idempotence feature is beta quality in this version, there's been quite a lot of work done on this in librdkafka since (look out for beta3 which will reference a later version of librdkafka). I wasn't aware of this issue, but will check on it.
you may be assuming the operation of seek is different from what it is. when you consume, you may get a message from any of the subscribed partitions - not necessarily the one you just called Seek with. this is a common miss-understanding and one we need to document better.

@mhowlett I cannot remember what exception. I think it is because you type "< 5" rather than "<= 5"

You can use Task.WhenAll or Task.WaitAll to await multiple requests at once.

all is only slow because you await it - you don't need to. Note that you might typically want to use this in the context of a web app, where each call may take a few milliseconds, but you can have many of them active simultaneously.

i noticed that ProduceAsync is awaiting the result of the TaskCompletionSource instead of just returning the task.
https://github.com/confluentinc/confluent-kafka-dotnet/blob/9617c9e8ecd30189d01fa47e266ca65952e4cdfb/src/Confluent.Kafka/Producer.cs#L679
I believe this will also stop batching.

await is needed here because this needs to execute sequentially with other async calls. it won't prevent batching.

Nw thank you for the response

Closing as the original query was answered


Revert back to function2

any_invocable that is used right now does not optimize to zero overhead, but function2 does. Reverting back to function2 would remove some overhead that shouldn't be there.

Nope, it works.


Add testing docs

Add some testing docs /c @brynary

@brynary This parser should help https://github.com/asenchi/scrolls/blob/master/test/test_parser.rb

@mrb @brynary What's the next step for this PR?

@mrb @brynary I'm inclined to close this PR until we reprioritize the work needed to push this forward. Is that okay with you two?

Closing this PR for now.


Separate backward_seek out from MvccReader

What have you changed? (mandatory)
This PR moves backward seek from MvccReader to a new module named BackwardScanner. This is a part of #3344 .
There are also some small fixes in this PR....
What are the type of the changes? (mandatory)

Improvement (non-breaking change which is an improvement to an existing feature)

How has this PR been tested? (mandatory)
By unit tests
Does this PR affect documentation (docs/docs-cn) update? (mandatory)
No
Does this PR affect tidb-ansible update? (mandatory)
No

Ouch our CI is not triggered

There might be other approach to do the same thing. But I want to do it in
future PRs
Connor<EMAIL_ADDRESS>于 2018年8月8日周三 17:04写道：

@Connor1996 commented on this pull request.
In src/coprocessor/dag/executor/scanner.rs
https://github.com/tikv/tikv/pull/3405#discussion_r208503929:

@@ -97,11 +97,7 @@ impl<S: Snapshot> Scanner {
return Ok(None);
}

   let kv = match self.scan_mode {

       ScanMode::Backward => self.scanner.reverse_seek(Key::from_raw(&self.seek_key))?,

       ScanMode::Forward => self.scanner.next()?,

       _ => unreachable!(),

   };

   let kv = self.scanner.next()?;

is that seek_key useless now?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tikv/tikv/pull/3405#pullrequestreview-144323571, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJfNBtfMRKzNISb7C9iMc6mlsy13MePmks5uOqmRgaJpZM4Vxq04
.


Authenticated Proxy Fails

Standard requests are not using the proxy username/password.
Was able to fix this as a workaround by modifying utils.py get_proxy as follows:
First, import quote from urllib.parse:
from urllib.parse import quote
Second, update get_proxy when proxy_type is "requests" to add the username/password and format the proxy url as http://:@:.
        if proxy_type.lower()=="requests":
            user_pass = ""
            if proxy["proxy_username"] and proxy["proxy_password"]:
                username = quote(proxy["proxy_username"], safe="")
                password = quote(proxy["proxy_password"], safe="")
                user_pass = f"{username}:{password}@"

            proxy_url = "%s%s%s:%s" % ("http://", user_pass, proxy["proxy_url"], proxy["proxy_port"])
            proxies = {
                    "http" : proxy_url,
                    "https" : proxy_url
                }

Not clear why the UI has the username/password but it is not in use by the code.

I had the same problem and your code fixed it !
Thanks a lot @tomworster
I hope it'll be fixed in the next version cc @JasonConger


Python script says module is missing but when run in terminal its fine

So I had to reinstall boto at one point in time and it seems to have broken bitbar but even after restarts and refreshes it doesn't work in bitbar but when I tell it to run from terminal its fine.
Is there some kind of caching I don't understand going on?
here is a screenshot: http://i.imgur.com/j8PfBYe.png
The terminal version was initiated via "Run in terminal" from bitbar.

Any chance you are using Homebrew or Macports?  I'm betting it's a $PATH issue...  Bitbar is getting system path/system python, terminal is getting your set path/python.

Yeah it was Homebrew. Guess it was dumb luck that it was fine before the reinstall of boto. I pointed the script to /usr/local/bin/python which is symlinked to ../Cellar/python/2.7.11/bin/python and it fixed it.
I was using env to determine the python location. Thanks lol


Problem when setting colors

Hello,
Thanks for this great plugin, very complete !
However I experience a little problem when setting colors on the emojis.
For example:
color: "green",
My emojis still stay the same ... the color doesn't change.
I have this problem on Chrome.
Do you know how can I resolve this ?
Thanks in advance.

Hello @cdric0701  thanks for your feedback!
In order to set a color you should use the CSS code, but this feature will only work depending on the OS you are using since in mobile or Mac the emojis does not appear as unicode characters but as the image itself.
for example using Mac Os I see them like this (yancinerio.com/emotion-ratings)

Okay thanks ! 👍

Yes, there is one:
If you include the font called Symbola  and add this font-family to your emojis container, you will be able to see the unicode figures and not the emojis, so you will be able to add the color you want!