No session, please create a session first with Neo4j::Session.open(:server_db) or :embedded_db

I tried looking everywhere but couldn't find the solution.
When I run,
config.around do |example|
  Neo4j::Transaction.run do |tx|
    example.run
      tx.mark_failed
    end
  end

it throws No session, please create a session first with Neo4j::Session.open(:server_db) or :embedded_db error.
Additional information which could be helpful if relevant to your issue:
Code example (inline, gist, or repo)
Runtime information:
Neo4j database version: 3.1.4
neo4j gem version: 8.0.17
neo4j-core gem version: 7.1.2

If you're using the neo4j gem, you need to setup the session (docs for: without Rails and with Rails).  I'm guessing that's the problem.  That error message should probably be updated to reflect the new API...

I've included
config.neo4j.session.type = :http # or :bolt
config.neo4j.session.url = 'http://localhost:7475'

in all the environments' Rails.application.configure block, but it's not working. If I go into rails console test and do Neo4j::Session.open(:server_db) but the response I get the following error:
ETHON: performed EASY effective_url=http://localhost:7474/ response_code=0 return_code=couldnt_connect total_time=0.005008
Faraday::ConnectionFailed: Couldn't connect to server

How come it's still connecting to port 7474, although I configured it to 7475?

in the console,  Neo4j::Session.open(:server_db, 'http://localhost:7475 works.
ETHON: Libcurl initialized
ETHON: performed EASY effective_url=http://localhost:7475/ response_code=200 return_code=ok total_time=0.006738
ETHON: performed EASY effective_url=http://localhost:7475/db/data/ response_code=200 return_code=ok total_time=0.000775
=> Neo4j::Server::CypherSession url: 'http://localhost:7475/db/data/' version: '3.1.4'

So I guess it's a configuration issue?

If you're connecting via HTTPS you may need to do:
config.neo4j.session.type = :http
config.neo4j.session.url = 'https://localhost:7475'

Also, Neo4j::Session is from the old, deprecated API in neo4j-core.  For the new API you would do:
http_adaptor = Neo4j::Core::CypherSession::Adaptors::HTTP.new('https://localhost:7475')
neo4j_session = Neo4j::Core::CypherSession.new(http_adaptor)

You should also check to see if you have a config/neo4j.yml.  That might have defaults which are overwriting the application.rb (though application.rb should have priority, I think)

As far as I can tell, config/neo4j.yml seems to be configured right.
development:
  type: http
  url: http://localhost:7100

test:
  type: http
  url: http://localhost:7475

production:
  type: http
  url: http://neo4j:password@localhost:7000

Also, I already have config.neo4j.session.type = :http and config.neo4j.session.url = 'https://localhost:7475' in the application.rb. I don't know where that ETHON thing is coming from, and why it's still trying to connect to http://localhost:7474.

I wrote about config.neo4j.session.url because in your code you had http instead of https
ETHON comes from the typhoeus gem which is the driver which the neo4j-core gem uses for Faraday.
I'm not really sure why you're having the issue, unfortunately.  The gem should probably have better debugging output to say where it's getting the URL from.  The only other thing that I can think of is that you're looking at the local_env.yml file.  Maybe that has a NEO4J_URL environment variable specified?

Thank you for helping me out. Switching to https gave me http.rb:923:in connect': too many connection resets`. The reason for switching to 8.0.x was to be able to make indexed spatial queries in neo4jrb_spatial plugin, because they removed the spatial_index from the newer version. I'll revert back to 7.1.0 for now, and keep researching. I will let you know if things work out! :) Thank you so much for your help.

@kroyagis, is it possible this issue is related to https://github.com/neo4jrb/neo4j/issues/1357 ?

@kroyagis & @cheerfulstoic , while running into another issue with Neo4j, I decided to troubleshoot by creating a new blank project with
rails new issue-test -m http://neo4jrb.io/neo4j/neo4j.rb -B
rake neo4j:install[community-3.1.4]

and suddenly I ran into this problem.
By adding an initializer to rails with the following code, the problem was solved.
require 'neo4j/core/cypher_session/adaptors/http'
neo4j_adaptor = Neo4j::Core::CypherSession::Adaptors::HTTP.new('http://localhost:7474')
Neo4j::ActiveBase.on_establish_session { Neo4j::Core::CypherSession.new(neo4j_adaptor) }

Supposedly, this initialization should be handled automatically by a railtie, but for whatever reason I needed to manually do it. It's not a perfect solution, because opening up rails c brings up several lines of initialization code
ETHON: Libcurl initialized
ETHON: performed EASY effective_url=http://localhost:7474/db/data/schema/constraint response_code=200 return_code=ok total_time=0.071625
ETHON: performed EASY effective_url=http://localhost:7474/db/data/schema/index response_code=200 return_code=ok total_time=0.098502
ETHON: performed EASY effective_url=http://localhost:7474/db/data/transaction/commit response_code=200 return_code=ok total_time=0.7863

Now, while writing up this comment I realized that, when generating my project, I failed to omit ActiveRecord with the -O flag. Neo4jrb is suppose to work fine alongside ActiveRecord according to the docs, but I decided to test generating the same project except without ActiveRecord using
rails new issue-test -m http://neo4jrb.io/neo4j/neo4j.rb -B -O
rake neo4j:install[community-3.1.4]

In this scenerio, everything works normally. I imagine its not super common for someone to use Neo4j alongside ActiveRecord, so maybe this is a regression that was introduced at some point? Regardless, @kroyagis, does your project have ActiveRecord in it? If so, do you need it? You might be able to easily resolve the issue by removing ActiveRecord. To do so edit your application.rb file so that instead of require rails/rails it has
require "rails"
# Pick the frameworks you want:
require "active_model/railtie"
require "active_job/railtie"
# require "active_record/railtie"
require 'neo4j/railtie'
require "action_controller/railtie"
require "action_mailer/railtie"
require "action_view/railtie"
require "action_cable/engine"
require "sprockets/railtie"
require "rails/test_unit/railtie"

@cheerfulstoic, this test was created with the following Vagrantfile. This test works if the project is generated with rails new issue-test -m http://neo4jrb.io/neo4j/neo4j.rb -B -O. It does not work if the project was generated with rails new issue-test -m http://neo4jrb.io/neo4j/neo4j.rb -B.
# Project Generated with
# rails new issue-test -m http://neo4jrb.io/neo4j/neo4j.rb -B -O

# -*- mode: ruby -*-
# vi: set ft=ruby :

# All Vagrant configuration is done below. The "2" in Vagrant.configure
# configures the configuration version (we support older styles for
# backwards compatibility). Please don't change it unless you know what
# you're doing.
Vagrant.configure("2") do |config|
  # The most common configuration options are documented and commented below.
  # For a complete reference, please see the online documentation at
  # https://docs.vagrantup.com.

  # Every Vagrant development environment requires a box. You can search for
  # boxes at https://atlas.hashicorp.com/search.
  config.vm.box = "ubuntu/xenial64"

  # Create a forwarded port mapping which allows access to a specific port
  # within the machine from a port on the host machine and only allow access
  # via <IP_ADDRESS> to disable public access
  config.vm.network "forwarded_port", guest: 3000, host: 3800, host_ip: "<IP_ADDRESS>"
  config.vm.network "forwarded_port", guest: 7474, host: 7474, host_ip: "<IP_ADDRESS>"
  config.vm.network "forwarded_port", guest: 7473, host: 7473, host_ip: "<IP_ADDRESS>"
  config.vm.network "forwarded_port", guest: 7472, host: 7472, host_ip: "<IP_ADDRESS>"
  config.vm.network "forwarded_port", guest: 7574, host: 7574, host_ip: "<IP_ADDRESS>"
  config.vm.network "forwarded_port", guest: 7573, host: 7573, host_ip: "<IP_ADDRESS>"
  config.vm.network "forwarded_port", guest: 7572, host: 7572, host_ip: "<IP_ADDRESS>"

  # Share an additional folder to the guest VM or configure . The first argument is
  # the path on the host to the actual folder. The second argument is
  # the path on the guest to mount the folder. And the optional third
  # argument is a set of non-required options.

  project_name = "issue-test"
  raise if project_name.nil? || project_name == ""

  config.vm.synced_folder ".", "/home/ubuntu/#{project_name}"

  # Provider-specific configuration so you can fine-tune various
  # backing providers for Vagrant. These expose provider-specific options.
  # Example for VirtualBox:
  #
  # config.vm.provider "virtualbox" do |vb|
  #   # Display the VirtualBox GUI when booting the machine
  #   vb.gui = true
  #
  #   # Customize the amount of memory on the VM:
  #   vb.memory = "1024"
  # end
  #
  # View the documentation for the provider you are using for more
  # information on available options.

  # Disable notify-forwarder plugin, if present, because it was messing with Neo4j and causing a database panic
  # https://discuss.elastic.co/t/es-cluster-in-docker-containers-alreadyclosedexception-underlying-file-changed-by-an-external-force/48874/8
  if Vagrant.has_plugin?("vagrant-notify-forwarder")
    config.notify_forwarder.enable = false
  end

  # Enable provisioning with a shell script. Additional provisioners such as
  # Puppet, Chef, Ansible, Salt, and Docker are also available. Please see the
  # documentation for more information about their specific syntax and use.

  config.vm.provision "bootstrapping", type: "shell", privileged: false,
    keep_color: true, inline: <<-SHELL
      PURPLE='\033[1;35m'; GREEN='\033[1;32m'; NC='\033[0;0m'
      echo "${PURPLE}BEGINNING BOOTSTRAPPING PROCESS...Grab a drink. This could take a while.${NC}"
      echo ' '
      echo "${PURPLE}UPDATING .BASHRC FILE${NC}"
      echo "export ROOT_URL='http://localhost:3000'" >> /home/ubuntu/.bashrc
      echo "cd /home/ubuntu/#{project_name}" >> /home/ubuntu/.bashrc
      cd ~/#{project_name}
      echo "${GREEN}COMPLETED UPDATING .BASHRC FILE${NC}"
      echo ' '
      echo "${PURPLE}INSTALLING JAVA${NC}"
      # Updates apt-get to point to updated java8 package
      yes | sudo add-apt-repository ppa:webupd8team/java
      # Updates apt-get
      sudo apt-get update
      # Set debconf config file to mark java8 license prompts as completed
      # https://askubuntu.com/questions/190582/installing-java-automatically-with-silent-option#
      # Install java8 without prompts
      echo debconf shared/accepted-oracle-license-v1-1 select true | sudo debconf-set-selections
      echo debconf shared/accepted-oracle-license-v1-1 seen true | sudo debconf-set-selections
      yes | sudo apt-get install oracle-java8-installer
      echo "${GREEN}COMPLETED INSTALLING JAVA${NC}"
      echo ' '
      echo "${PURPLE}INSTALLING RVM & RUBY${NC}"
      yes | sudo apt-get install gnupg2
      gpg2 --keyserver hkp://keys.gnupg.net --recv-keys 409B6B1796C275462A1703113804BB82D39DC0E3 7D2BAF1CF37B13E2069D6956105BD0E739499BDB
      # Current issue with rvm: https://github.com/rvm/rvm/issues/4068
      # use this patch until fixed
      curl -sSL https://raw.githubusercontent.com/wayneeseguin/rvm/stable/binscripts/rvm-installer | bash -s stable --ruby=2.4
      # curl -sSL https://get.rvm.io | bash -s stable --ruby=2.4
      source /home/ubuntu/.rvm/scripts/rvm
      echo "${GREEN}COMPLETED INSTALLING RVM & RUBY${NC}"
      echo ' '
      echo "${PURPLE}INSTALLING GEMS${NC}"
      gem install bundler --no-rdoc --no-ri
      # also install libcurl3, libcurl3-gnutls, & libcurl4-openssl-dev
      # which are needed by the "curb" gem, used in some 3rd party rake tasks
      yes | sudo apt-get install libcurl3 libcurl4-openssl-dev #libcurl3-gnutls
      bundle install
      echo "${GREEN}COMPLETED INSTALLING GEMS${NC}"
  SHELL

  config.vm.provision "bootstrapping complete", type: "shell", privileged: false,
    keep_color: true, run: "always", inline: <<-SHELL
      GREEN="\033[1;32m"; BLUE='\033[1;34m'; NC="\033[0;0m"
      echo "${GREEN}BOOTSTRAPPING COMPLETE!${NC}"
      echo "${BLUE}Database and app server not started${NC}"
      echo ' '
      echo "${BLUE}To connect to the app server in your browser visit:${NC}"
      echo "${BLUE}- HTTP: http://localhost:3000${NC}"
      echo It may take a minute or two for the servers to boot up...
    SHELL
end

Oh ho!  Very good catch!  I tried this and found that the problem is that require 'neo4j/railtie' isn't being added to the top of the config/application.rb.  That is the thing that triggers the railtie.
The code to make this happen is in the gh-pages branch of the neo4j gem:
https://github.com/neo4jrb/neo4j/blob/gh-pages/neo4j.rb
That's what allows us to have the http://neo4jrb.io/neo4j/neo4j.rb URL for the -m flag.
That script adds require 'neo4j/railtie' after require "active_record/railtie", but when you don't use the -O flag you end up with require 'rails/all' in the application.rb.  With the -O flag you get:
require "rails"
# Pick the frameworks you want:
require "active_model/railtie"
require "active_job/railtie"
# require "active_record/railtie"
require 'neo4j/railtie'
require "action_controller/railtie"
require "action_mailer/railtie"
require "action_view/railtie"
require "action_cable/engine"
require "sprockets/railtie"
require "rails/test_unit/railtie"

So I've just added the following line to the setup script which seems to fix it:
inject_into_file 'config/application.rb', "\nrequire 'neo4j/railtie'", after: "require 'rails/all'"

@cheerfulstoic nice 👍 !

Just tested, and I can confirm that this fixes the issue I was experiencing.

I'm running into this issue after having upgraded from neo4j 7.1.2 to 8.2.3
calling code is in a Rails rake task:
neo4j_transaction = Neo4j::Transaction.new
This is based on the documentation at https://github.com/neo4jrb/neo4j-core/wiki/Transaction
I know that the Neo session has already been established by looking at my logs, up to this point the rake task has successfully been reading records from the Neo4j database via the ActiveNode models.
It looks like the method being called has a default argument of Session.current! per the rubydoc at: http://www.rubydoc.info/github/neo4jrb/neo4j-core/Neo4j%2FTransaction:new
The upgrade guid does mention a change to how to access the current session: http://neo4jrb.readthedocs.io/en/8.2.x/UpgradeGuide.html#sessions
So following that pattern, I was able to fix the issue by passing in the current session as:
neo4j_transaction = Neo4j::Transaction.new(Neo4j::ActiveBase.current_session)
Transactions are not mentioned in the upgrade guide at: http://neo4jrb.readthedocs.io/en/8.2.x/UpgradeGuide.html
Transaction handling is also not mentioned (except in the context of Migrations) in the new docs at http://neo4jrb.readthedocs.io/en/8.2.x/
It seems like this is either a bug in the default parameter of Neo4j::Transaction:new() or a documentation issue, I'm not sure which.

Hey @benjaminbradley !
Good point, I think there were some things that changed that weren't documented.  Another thing that you might try is using:
Neo4j::ActiveBase.run_transaction do
end
# or
tx = Neo4j::ActiveBase.new_transaction

That should do transactions and use the current ActiveBase session in the process.
Somebody pointed out recently that ActiveBase isn't documented, and it's become a place where some central things live.  So if that works for you we can make documenting that a priority.

also
Neo4j::ActiveBase.run_transaction do |tx|
  tx.mark_failed
end

i was having the above session missing issue with Neo4j::Transaction (and other bare metal Neo4j commands) even though the "normal" Neo4j::ActiveNode functionality (ie: Item.create()) was working/configured correctly.
i switched to the ActiveBase style transactions (per @thefliik and @cheerfulstoic), and that cleared up the session error message, but transactions don't seem to rollback.
for ref: https://github.com/neo4jrb/neo4j-core/wiki/Transaction

Hey @whatbird !  Thanks for letting us know about this.  Could you perhaps provide some simple example code that we can test?


Add config for endpoint

Provides a way to specify the endpoint in a config file.
If the endpoint is not set the default will use the unix socket.
cat /etc/fullerite/conf.d/DockerStats.conf
{}

If it's false, it fails...
bash-4.3# cat /etc/fullerite/conf.d/DockerStats.conf                          f
{
   "dockerEndPoint": "unix:///var/run/docker.FALSE"
}
bash-4.3# /fullerite/build/usr/bin/fullerite -c /etc/fullerite/fullerite.conf
time="04 Mar 16 19:37 UTC" level=info msg="Starting fullerite..." app=fullerite
time="04 Mar 16 19:37 UTC" level=info msg="Reading configuration file at /etc/fullerite/fullerite.conf" app=fullerite pkg=config
time="04 Mar 16 19:37 UTC" level=info msg="Starting collectors..." app=fullerite
time="04 Mar 16 19:37 UTC" level=info msg="Reading collector configuration file at /etc/fullerite/conf.d/DockerStats.conf" app=fullerite pkg=config
time="04 Mar 16 19:37 UTC" level=info msg="Starting handlers..." app=fullerite
time="04 Mar 16 19:37 UTC" level=info msg="Starting handler Graphite" app=fullerite
time="04 Mar 16 19:37 UTC" level=info msg="Starting to run internal metrics server on port 29090 on path /metrics" app=fullerite pkg=internalserver
time="04 Mar 16 19:37 UTC" level=info msg="Running DockerStatsCollector" app=fullerite
time="04 Mar 16 19:37 UTC" level=error msg="ListContainers() failed: Get http://unix.sock/containers/json: dial unix /var/run/docker.FALSE: connect: no such file or directory" app=fullerite collector=DockerStats pkg=collector
bash-4.3#

correct leads to success... :)
bash-4.3# cat /etc/fullerite/conf.d/DockerStats.conf                          f
{
   "dockerEndPoint": "unix:///var/run/docker.sock"
}
bash-4.3# /fullerite/build/usr/bin/fullerite -c /etc/fullerite/fullerite.conf
time="04 Mar 16 19:37 UTC" level=info msg="Starting fullerite..." app=fullerite
time="04 Mar 16 19:37 UTC" level=info msg="Reading configuration file at /etc/fullerite/fullerite.conf" app=fullerite pkg=config
time="04 Mar 16 19:37 UTC" level=info msg="Starting collectors..." app=fullerite
time="04 Mar 16 19:37 UTC" level=info msg="Reading collector configuration file at /etc/fullerite/conf.d/DockerStats.conf" app=fullerite pkg=config
time="04 Mar 16 19:37 UTC" level=info msg="Starting handlers..." app=fullerite
time="04 Mar 16 19:37 UTC" level=info msg="Starting handler Graphite" app=fullerite
time="04 Mar 16 19:37 UTC" level=info msg="Starting to run internal metrics server on port 29090 on path /metrics" app=fullerite pkg=internalserver
time="04 Mar 16 19:37 UTC" level=info msg="Running DockerStatsCollector" app=fullerite
time="04 Mar 16 19:37 UTC" level=info msg="Starting to emit 18 metrics" app=fullerite handler=Graphite pkg=handler
time="04 Mar 16 19:37 UTC" level=info msg="POST of 18 metrics to Graphite took 0.011235 seconds" app=fullerite handler=Graphite pkg=handler
bash-4.3#

+1 lgtm

+1 please fix test and we can merge this.

Hey @gnufied, @tsheasha,
I am afraid I am confused with the way how to test the case. Could you guys give me a hand in the test?
Cheers
Christian

@ChristianKniep this is what we meant - https://github.com/Yelp/fullerite/commit/0c73f6e84ea62bc74ac9fc09df5eb292a83d78fc

OK, this shoudl do it then...
bash-4.3# make
Cleaning fullerite...
Getting dependencies...
Linting fullerite sources...
Building fullerite...
Building beatit...
...............
----------------------------------------------------------------------
Ran 15 tests in 0.012s

OK
Testing fullerite
ok  	beatit	0.008s	coverage: 22.2% of statements
ok  	fullerite	3.015s	coverage: 39.4% of statements
ok  	fullerite/collector	7.519s	coverage: 80.8% of statements
ok  	fullerite/config	0.008s	coverage: 78.1% of statements
ok  	fullerite/handler	3.023s	coverage: 56.9% of statements
ok  	fullerite/internalserver	0.455s	coverage: 88.1% of statements
ok  	fullerite/metric	0.004s	coverage: 81.0% of statements
ok  	fullerite/util	0.038s	coverage: 93.2% of statements
bash-4.3#

I have cleaned up the commits and merged them into master. Closing this PR. @ChristianKniep  thank you for contributing!


conversation-activity-filter: undefined is not an object (evaluating 'container.classList')

Description
The feature is broken 😰
How to replicate the issue + URL
https://github.com/refined-github/refined-github/issues/6000
Extension version
23.11.3
Browser(s) used
Safari

Apparently only broken on some pages, since it works here.

My guess is that the "hide as default" on that page causes a race condition:
https://github.com/refined-github/refined-github/blob/4d78a9f164479fddd82dc632f6af2bbaf3c416de/source/features/conversation-activity-filter.tsx#L224-L226
If so, this might just be fixed via if in that array, await dowloaded


[WIP] [Enhance] tune default configuration

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

@Mergifyio rebase


Add a little padding on top of the allergen alert

Hello, I would like to add padding on this to get accustomed to contributing.
Thank you.

@Orlandov87 Go ahead


Regression in AES-GCM on Alpha and Arm platforms after the GCM refactoring

The on-push cross-compile builds in CI for Alpha and Arm platforms fail since the GCM refactoring was merged.
See: https://github.com/openssl/openssl/runs/7468446782?check_suite_focus=true https://github.com/openssl/openssl/runs/7468446918?check_suite_focus=true
Perhaps something like https://github.com/openssl/openssl/commit/48e35b99bd0071207cfe39da22eb2502db5c09dc will be needed?

Despite the face that the PR CIs all passed... :(

Thanks @t8m for the PR.

The PR CIs only compile the source code for each platform.
The on commit CIs also do this but then run the tests using qemu.
The reason is that running the tests is glacially slow for multiple emulated platforms.


Analytics documentation and breaking up readme

Trello: https://trello.com/c/FM3MCuAl/255-investigate-email-alert-api-database-size-and-push-email-archives-to-s3
This creates an an analytics documentation page which explains how to find data with AWS athena and then breaks up the readme so that it's easier to bring parts of this into developer-docs and make the readme less of a monster.

@boffbowsh no worries, I've just updated this now. Thanks for reviewing.


Added appveyor CI config.

The appveyor.yml contains all information which needs to be configured
for appveyor builds. No need to configure appveyor beside connecting
appveyor with github.
Appveyor offers a badge for the build status. Example @ https://github.com/th3ragex/Nitra
NOTE: The PowerShell script will try to download and install the nemerle
setup with the configured  $nemerleVersion .

What is it?

Looks like spam.

I tried to add a build server support to this project. Appveyor seems to be the only service which offers building of open source .NET projects for free. I know the PowerShell script looks like garbage...

Looks like spam.
And no, i am not associated in any way with that service. It just seemed to be the only working option.

Please give it a try, connect Github with appveyor and add Nitra as project. The .yml file contains the whole build configuration, it should start building without any further adjustments.

New repo is: https://github.com/rsdn/nitra


Confution when you read this document and you are trying to enable this feature

[Enter feedback here]
In section "How to use trickle feed-based posting" the step 2 is referring to preview feature and if someone is trying to enable it, when you go to License parameters in 10.0.9 and above, it will not found this option and it is not necessary to do it, that's why this is causing confusion.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f53eb353-455f-37e8-646b-500d9a5a6402
Version Independent ID: 81c4d76f-ec29-877d-cb51-374c05741909
Content: Trickle feed-based order creation for retail store transactions - Commerce | Dynamics 365
Content Source: articles/commerce/trickle-feed.md
Service: dynamics-365-retail
Product: ****
Technology: ****
GitHub Login: @josaw1
Microsoft Alias: josaw

@clugarte, thank you for posting this feedback.
@josaw1, please review and consider updating the content.

Bug 466223 has been logged to track this issue.

@anpurush, please review and update the topic as needed. Thanks.

This content has been updated and will be published later today. Thanks!


Automate Data Ingestion

From @kenjinp on October 18, 2018 13:59
Specifically for morphology release, at least, to start.
Can be enough to create a job that handles this, but ideally would read the stream of events that Nexus produces.
connects #212
Copied from original issue: BlueBrain/nexus-search-webapp#111

After some trials using making a kafka stream consumer to listen to events, I think that's not the way to go. This is because we don't have a nexus service account for the consumer, so it cannot (easily) read the entities from nexus once a new event has been emitted.
So we could

Make a user interface to re-sync search, using the user's token
Re-assemble the graph from the JSON payloads from kafka's history.

I think the best strat here is making a "Sync Search" interface.

For v1 we can create a service account to support this workflow. I would very much recommend using an incremental approach to migrate data over a user triggered one.

Sounds good to me! ;)

This is currently only implemented for Morphology Releases, with other data types coming soon!


can not log in to virtualbox gui, but vagrant ssh login works

Hi,
Installed ubuntu desctop-x64 from atlas repo
and can not log in to virtualbox gui by default pwd vagrant - virtualbox redirects me to login screen
vagrant ssh login works
i have tried to login to guest session
but virtualbox redirects me to login screen again

win 8.1 64 bit

ok,solved
it was not a box related issue at all but virtualbox
there was the 3d accelerate enabled, which uncompatible with my vga settings


Update VMImage to windows-2019

/azp run net - client - ci

/azp run net - client - ci

I assume you plan to similarly update the vmImage in the other language repos?

Yes, am working on that.


Refactor code a bit

Remove self invoking anonymous function
Skip native functions
Split up render into smaller functions

You can review the changes with this link: https://github.com/Timer/create-react-app/pull/5/files?w=1

Is c9982e2bb3b4f047bb5418e05523a5c21d6a38a9 desirable?

@Timer This is what is looks like without it:

The Chrome stack trace in the console also skips it:

Fair enough.


[Question] About using AWS lambda  case

Hello! Thanks for maintaining the library.
I use linebot by boybundit for my private Line App.  But I found this forked version, then i started this version's library(linebot2) because this ver's is currently maintained.
I am building a Line bot server in Lambda. So, in the example directory, can I submit a pull request for a use case for using this library in Lambda?

Yes. You are welcome to PR. Nowadays, serverless services are very popular and it is convenient for developers to concentrate on bot development, so your contribution will help. I'm waiting for it. Thanks. :)


Add Hyperterm-blink to packages.

https://github.com/amio/hyperterm-blink
Thanks!

Looks good - merged!


AWS DNS zone are not unique

AWS zones are merged by name, however, many different AWS accounts can have an AWS zone with the same name. Need to merge on ZoneId instead.

Because the other queries match on zoneid the graph will be missing a lot of data.

https://docs.aws.amazon.com/Route53/latest/APIReference/API_HostedZone.html
It doesn't look like there is an ARN defined on hosted zones.  Are IDs globally unique?  If not, we'll need to concatenate together an ID (and I dislike doing this even though we've been forced to do this in many other places ☹)

There is a unique zoneid in AWS. However cartography is merging on name

Are we sure that is AWS-globally unique?

Yes


Logistics Pipes will come back ?! =D

Hello, I found this project after looking for options to transport items in minecraft, and it is amazing that you are updating this mod that I most played in modpack. I wanted to leave my congratulations and if I can help in any way, 3D model or texture, I will be glad to have helped this mod to come back to life.

If you want more information, read the other issue posts and feel free to join the Discord to discuss things and see the progress: https://discord.gg/JEsaj9M

I am glad you like the project and Any contribution is MORE than welcome!
I only have limited knowledge in 3D modelling and honestly the pipes could use a rework. I am currently thinking about writing our own rendering for the pipes because frankly, OBJ support from forge SUX for multi part models!!!

I'm happy to help, I can do some sketches on paper or you can give the idea and I try to execute it.


Is this compatible with Vim?

I'm trying to find a Vim plugin for sqlfluff. Does the one you created require Nvim + coc? I'm not too familiar with those, so I apologize in advance if this is a dumb question.

A simple explanation of coc.nvim is that it is a "VSCode-like" plugin for Vim8/Neovim. coc-sqlfluff is an extension (like an extra plugin) for coc.nvim.
coc.nvim has .nvim in the plugin name, but it also works with Vim8.

Got it thanks. I will download coc.nvim then and go from there.


KeyError: 'edit-email'

Hi @taylormitchell, firstly thanks for building the module :) I ran into an issue yesterday, executing ankify_roam add <json_path> in Terminal - attaching screenshot of the traceback.
Running Python 3.8.5 on 2017-Macbook Air model that runs MacOS Big Sur 11.1 - Both Anki and AnkiConnect are up-to-date. Do share your thoughts on how I can rectify this...

Hey @karthajee
Can you try installing the latest ankify_roam with the following command?
pip install git+https://github.com/taylormitchell/ankify_roam.git

I'm pretty sure this issue is resolved on the latest version on github. Did you original install it using pip install ankify_roam? I think the version on https://pypi.org/ is behind a couple versions...
One thing to be aware of: in the latest version, the html generated by ankify_roam may be slightly different. I'd actually suggest making a back up of your cards before running the new ankify_roam version just in case. If all goes smoothly, all you might need to do is update the css on your Roam Basic and Roam Cloze cards. This section of the README should be able to help you out there.
I hope that does it! Let me know if the issue persists, or if you run into anything else after upgrading the version.

Thanks a bunch @taylormitchell  - the issue is resolved!


Improve multithreading testing with shrinking

Idea from John Hughes talk: https://youtu.be/1LNEWF8s1hI?t=1603
Create an initial state, parallel operations, then compare to all the possible linearized versions. Final state must equal at least one of them. This can be shrunk to a minimal threading bug. Quite powerful.
public static void SampleConcurrent<S>(Gen<S> initial, Func<S, S, bool> equal, params Gen<Action<S>>[] operations)

Sequential model-based testing would actually have a similar method signature so needs better function names:
public static void SampleSequential<S, M>(Gen<(S, M)> initial, Func<S, M, bool> equal, params Gen<Action<S, M>>[] operations)

Where S and M are the state and model.
Example testing ConcurrentBag model-based and parallel:
[Fact]
public void ConcurrentBag_Sequential()
{
    Gen.Int.List.Select(l => (new ConcurrentBag<int>(l), l))
    .SampleSequential(
        equal: (bag, list) => bag.OrderBy(i => i).SequenceEqual(list.OrderBy(i => i)),
        Gen.Int.Select<int, Action<ConcurrentBag<int>, List<int>>>(i => (bag, list) =>
        {
            bag.Add(i);
            list.Add(i);
        }),
        Gen.Const<Action<ConcurrentBag<int>, List<int>>>((bag, list) =>
        {
            Assert.Equal(bag.TryTake(out var i), list.Remove(i));
        })
    );
}

[Fact]
public void ConcurrentBag_Concurrent()
{
    Gen.Int.List.Select(l => new ConcurrentBag<int>(l))
    .SampleConcurrent(
        equal: (bag1, bag2) => bag1.OrderBy(i => i).SequenceEqual(bag2.OrderBy(i => i)),
        Gen.Int.Select<int, Action<ConcurrentBag<int>>>(i => bag =>
        {
            bag.Add(i);
        }),
        Gen.Const<Action<ConcurrentBag<int>>>(bag =>
        {
            bag.TryTake(out var i);
        })
    );
}

After reading this paper Finding Race Conditions I think there can be a good implementation:

Given N operations these are parallel.for processed on T threads recording in an array which thread each was on.
The number of permutations of the linearized versions since we know the order on each thread are far fewer than the N!
We have a big advantage in the shrinking as the path shrinking methods have to repeat each run many times because they may not always fail and they need to follow a failing path, this is quite brittle.  Our monte-carlo shrinker doesn't have to do this at all.

Writing the model-based sample function was shockingly easy. The hardest part of the concurrent sample function looks to be the generation of the permutations.

2.0.0 released


HTML url to inject for standalone components?

Hi! I'm trying to find the HTML url to inject in my index.html so that React-Hot-Toast is available to all my components (as opposed to importing it in each component).
Where can I find the url? I currently am using angularJS but inject individual react components.
Again, my goal is to avoid importing {Toaster} into each component and just doing it once and do it everywhere.

I'm not sure if I understand. You don't have to import {Toaster} in each component, just once in your container (this is where all toasts will be placed). Then, you can import {toast} to dispatch new toasts.

Thanks for following up, I ended up figuring this out. The issue was how to inject the singular Toaster in HTML (app rendered in Angular JS, but we inject React components individually in angular JS html files).
We ended up wrapping  in a standalone react component and injecting it into our index.html using ng-react.


[Communication] - Phone Number Management - Added support for AAD auth

PR looks good.

PR looks good.


Yatin: Watch thenewyorkpharmacy.com

Yatin requests the watch of the watch_keyword thenewyorkpharmacy\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

thenewyorkpharmacy\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by double-beep in Charcoal HQ


Update Decryption class so that it matches specification

Python 3.6.5 (default, Apr 14 2018, 13:17:30)
[GCC 7.3.1 20180406] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from lantern.modules import shift
>>> from lantern import fitness
>>> ciphertext = "iodj{EuxwhIrufhLvEhvwIrufh}"
>>> decryptions = shift.crack(ciphertext, fitness.english.quadgrams)
>>> print(decryptions[0])
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: __str__ returned non-string (type list)

I think it makes sense to have the plaintext stored as a list. If it's to be kept as a list, the __str__ method should be updated to join the list prior to returning the plaintext. The documentation would need to be updated accordingly as well. For now, I've updated the Decryption class so that self.plaintext is a string as it meets the spec.

Coverage remained the same at 99.569% when pulling e5123197fdeaa733dfce4ae1d6ddc29790c96444 on jtalowell:master into 235e163e96bf0719d49c54204ee576b2ca93abb6 on CameronLonsdale:master.

This is on the current master branch.
Python 3.6.5 (default, Apr 14 2018, 13:17:30)
[GCC 7.3.1 20180406] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from lantern.modules import shift
>>> from lantern import fitness
>>> ciphertext = "iodj{EuxwhIrufhLvEhvwIrufh}"
>>> decryptions = shift.crack(ciphertext, fitness.english.quadgrams)
>>> print(decryptions[0])
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: __str__ returned non-string (type list)
>>> from lantern.modules import simplesubstitution
>>> ciphertext = "iodj{EuxwhIrufhLvEhvwIrufh}"
>>> decryptions = simplesubstitution.crack(ciphertext, fitness.english.quadgrams)
>>> print(decryptions[0])
oldi{NtureOftheSaNearOfthe}

The inconsistency here probably needs a better fix than just joining the string (this pull request).

Thanks for this PR, but for now the str representation will be removed in favor of manually joining decryptions


Highlight matching text in record display text search

Highlight (such as with yellow background) the matching text within the record display.
Some resources:

http://www.smashingmagazine.com/2010/09/5-coding-techniques-for-more-dynamic-websites/
http://james.padolsey.com/snippets/highlighting-text-with-javascript/
http://www.dotnetbull.com/2011/12/javascript-for-highlighting-text-inside.html
http://jsfiddle.net/FutureWebDev/HfS7e/
http://jsfiddle.net/UPs3V/291/
http://www.the-art-of-web.com/javascript/search-highlight/?hilite=search+engine+keywords

Currently the text search and the record display summaries can have different content.
Merge the two into one.

@adilyalcin I can only advise against using the above named plugins. They may work, but are poorly implemented. A few reasons:

All of them are using either jQuery's .html() method ore the native innerHTML. This will destroy all events in the highlighting context and also triggers generation of the DOM over and over again which is bad for performance
They all aren't hosted on a professional hosting platform like GitHub with issues. Users may not have the chance to submit their problems, so you don't have a guarantee that others didn't encountered problems
They all aren't built upon unit tests. There are dozens of situations which need to be tested in order to guarantee functionality.
They don't escape search terms. Entering * for example may end in unexpected behavior
They don't merge multiple blanks (see e.g. https://github.com/julmot/mark.js/issues/33)
They may not ignore <script> and <style> tags or tag names in general. Type in e.g. script here and you will see that the HTML <script> tag will be highlighted
They may not contain a license information
They may not be maintained
... (I think you get an idea ;-))

Instead, have a look at mark.js which is developed implemenented in several dozen cross-browser unit tests and brings you the benefit to have further options.

Very nice! Thanks for the information @julmot . I will consider using mark.js for highlighting searched text.


windowmaker dockapps

two dockapps for windowmaker : a clock and a systemtray

Squash commits into one, please.

Check travis.
error: undefined variable ‘libXfixes’

Why did you close? You can do changes, rebase and force push.

I'm just starting with git and was following this guide
I will have to study git a bit so at the moment it was easier for me to start all over.
When I have more experience I will certainly follow your advice.
Thank you for beying so kind to a newbie.
I have filed a new pull request 6443 so please close this one.


⚠ ClamAV has degraded performance

In bf20c76, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 64 ms

Resolved: ClamAV performance has improved in d191b39.


Timestamp retail package for code signing purposes?

Is there any ability to utilize code timestamp functionality (via a timestamp URL) when building the Retail Deployable package?  Without time stamping, packages would become invalid for install once the code signing certificate reaches it's expiration date.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c268f5fb-7fbc-b82c-5f6f-b3640344f4a4
Version Independent ID: 559b9e5f-7514-f3d1-8b73-bf9248cbac80
Content: Create retail deployable packages - Retail | Dynamics 365
Content Source: articles/retail/dev-itpro/retail-sdk/retail-sdk-packaging.md
Service: dynamics-365-retail
GitHub Login: @mugunthanm
Microsoft Alias: sijoshi

Unfortunately this is not supported.  Certificate management has to be managed separately form this.


Server Issues Each Agent's Damage

Entry:

Missions Server
https://github.com/Bannerlord-Coop-Team/BannerlordCoop/issues/264 Completed

Exit:

Disable damage callback from engine to module
Send callback damage data to server
Server issues damage to all clients based on from <AgentUniqueID,> to , damage amount.

This will now be client based, so when a controlled agent does damage. The damage is broadcasted from that client to all others.


feat(icon-button): create component

fixes https://github.com/material-components/material-components-web-react/issues/143

Codecov Report

Merging #154 into master will decrease coverage by 0.16%.
The diff coverage is 92.68%.

@@            Coverage Diff             @@
##           master     #154      +/-   ##
==========================================
- Coverage   95.78%   95.61%   -0.17%
==========================================
  Files          21       22       +1
  Lines         712      753      +41
  Branches       63       64       +1
==========================================
+ Hits          682      720      +38
- Misses         30       33       +3

Impacted Files
Coverage Δ

packages/icon-button/index.js
92.68% <92.68%> (ø)

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 3ef1123...fd26ee1. Read the comment docs.

spoke offline with @lynnjepsen about remove the toggle feature of the icon-button component, since icon button will be rewritten: see https://github.com/material-components/material-components-web/issues/3107. Looking at MDC Web foundation logic, its difficult to turn off without losing a11y support. See:
    this.on_ = this.adapter_.getAttr(ARIA_PRESSED) === 'true';

The only line we need to remove is the #adapter.getAttr line.

created 2 new issues:
https://github.com/material-components/material-components-web-react/issues/160
https://github.com/material-components/material-components-web/issues/3162

@bonniezhou which handleFocus method are you talking about?

The handleFocus method in RippledComponent. Filed #265


ui: repositories#show content reorganization

Repositories#show page had its content finally reorganized. Sections
were organized into tabs. We had 'Tags' and 'Comments' in the same page
that turned into tabs and we've created a new tab named 'Overview'.
The 'Overview' tab now contains a description that is editable. This is
a new feature that will help users to provide a more general information
about the repository if necessary.
The workflow stays the same for the rest.
Signed-off-by: Vítor Avelino<EMAIL_ADDRESS>Fixes #2074

Which is the tab that is shown by default? I'd say the "Tags" one :thinking:

Which is the tab that is shown by default? I'd say the "Tags" one

Tags.


[enhancement] The setter in useLocalStorage is now stable across renders

Typically, state setters are stable across renders in React. This means, among other things, that you don't need to include them as a hook dependency (eg. to useEffect or useMemo).

React guarantees that setState function identity is stable and won’t change on re-renders. This is why it’s safe to omit from the useEffect or useCallback dependency list. – https://reactjs.org/docs/hooks-reference.html#usestate

I got into a little trouble in #216 because I assumed this to be the case for the state setter returned by our custom  useLocalStorage hook. This was not the case; it changed every time that the value or key changed.
In this PR, we refactor useLocalStorage so that the state setter is stable across renders. This should make things more efficient overall, and prevent folks from making the same mistake I made above.
# Tests still pass after the refactor.
(cd packages/core/react; yarn test)

cc/ @jordansexton


Add MakeMKV settings screenshots to README

Unfortunately the help for MakeMKV is lacking at best. It would be extremely helpful to see not just read an example of your settings for MakeMKV when you rip a Blu-Ray.

@agrajag9 I would certainly agree that the documentation for MakeMKV is sparse, at best. However, the focus for my tools, and their README file, is on transcoding and not ripping. I've even considered removing the whole "Preparing your media for transcoding" section in the "Guide" since I think it's distracting. So I'm very hesitant to expand it at this point.
Properly explaining MakeMKV -- or even my particular settings -- could easily double the size of the existing README file. Especially when it comes to handling subtitles. That's because every disc is different. So my settings change all the time.
Is there a particular use case that you need help with? I can try to answer your questions here.

@agrajag9  take a look at this MakeMKV documentation I generated. It may provide some guidance with setup for a Blu-Ray rip.

@bmhayward excellent writeup! This answers most of my questions.
The only outstanding questions are about subtitles. I don't think I actually understand the "forced" subtitles thing. What's the difference between the following:

Selecting the parent but not the "forced only" child
Selecting the "forced only" child but not the parent
Selecting both the parent and "forced only" child

@agrajag9 all great questions. I have no idea what any of them do.
I use Plex exclusively for my media content distribution and rely on its built-in subtitle support.
Maybe @donmelton or someone else reading this thread has an idea of what these do.

@bmhayward @agrajag9 OK, but you might not want to read this. :)
Subtitles are tricky. And it's impossible to know ahead of time exactly which subtitles contain embedded forced subtitles. This situation is further complicated because many discs are mastered incorrectly or, perhaps worse, mastered inconsistently in a boxed set—that can really make you crazy when you're ripping TV shows.
However, I have a very simple solution to this problem. But you may not like it …
Here's what I do, step by step:

Rip every subtitle track on the disc (or at least those tracks in your target languages). That means both the parent and child subtitles are checked in MakeMKV. Both of them, dammit! I'm serious.
Use a player like VLC or a tool like mediainfo to figure out which new subtitle tracks have been created in the output. These will (usually) be the embedded forced tracks.
Check (with VLC or mediainfo) to see if there is a  subtitle track that had its "forced" flag set by MakeMKV. You probably want to keep that track. But not always. This could be due to bad mastering.
Decide which subtitle tracks to keep. Sometimes newly created child tracks are simply duplicated versions of the parent track. This is due to bad mastering. Just shoot me. And you probably don't want to keep those duplicates.
Remux the ripped MKV file with mkvmerge from MKVToolNix to exclude the subtitle tracks you don't want (and to set or reset the forced flag as needed).

Remuxing (not "remixing," dammit!) is not as hard as it sounds. And there is no other way to get the subtitle tracks you want without re-ripping the disc. Not only does that take a shitload longer, it's very prone to error.
So, that's it. Let me know if I can clarify anything for you.

@donmelton thanks!

@agrajag9 With the cool documentation from @bmhayward and my cranky answer, I think we've answered the original question. So I'm going to close this issue for now. But feel free to keep commenting here. Thanks.

This is a great rundown of answers I needed as well! Thanks y'all!
@bmhayward, maybe you could add subtitle steps @donmelton clarified to your documentation? It'd be nice to have it all in one place.

@hisaac let me see what I can do. I was thinking about potentially adding some of Don's suggestions to my workflow solution. Look for something in the next couple of weeks.


Entity Recognition does not work properly.

Rasa NLU version: 0.13.1
Operating system (ubuntu 17.10.):
Content of model configuration file:
language: "en"

pipeline:
- name: "nlp_spacy"
  model: "en"
- name: "tokenizer_whitespace"
- name: "tokenizer_spacy"
- name: "ner_synonyms"
- name: "ner_crf"
- name: "intent_entity_featurizer_regex"
  BILOU_flag: true
  features:
    # features for word before token
    - ["low"]
    # features of token itself
    - ["bias", "low", "prefix5", "prefix2", "suffix3", "suffix2", "digit", "pattern"]
    # features for word after the token we want to tag
    - ["low"]
  max_iterations: 50
  L1_c: 0.1
  L2_c: 0.1
- name: "intent_featurizer_spacy"
- name: "intent_featurizer_count_vectors"
- name: "intent_classifier_tensorflow_embedding"
- name: "intent_classifier_sklearn"

Issue:
The entities are not recognized in all test sentences.
Some it recognize the entity, some time it does not ?
How much training instances do I need to train to recognize properly ?

@azizullah2017 this looks like a 'how to' question, so I would recommend you post it in the rasa community forum together with a bit of your training data and some examples which don't work. That will make it easier to understand what's happening

I will close this issue as it's been moved to the forum


workload/management cluster renaming in dashboards

Towards: giantswarm/giantswarm#15028
Renaming tenant cluster and control plane respectively to workload cluster and management cluster.

This is a dupe of https://github.com/giantswarm/dashboards/pull/13/files


Scale controller logging needs sample query

The new Scale Controller logging capability is great, but it'd be better to have a sample query showing to pull out some of the information logged by it. Either on this page or the Analyze Logs page.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 13672b67-22ea-3dda-8b6b-3332c24e6030
Version Independent ID: fdc60aeb-9971-1d9f-a6ba-b3ad818cf0d5
Content: Monitor Azure Functions
Content Source: articles/azure-functions/functions-monitoring.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Hello @brandonh-msft,  Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

Hi, is it possible to get an example query for scale controller logs please?

Hi, is it possible to get an example query for scale controller logs please?

We now have this content available here. Thanks for your patience. #please-close


fix: do filter in advance

if target element is 'input' or 'textarea' ,  _downKeys should not record the eventKey

Codecov Report

Merging #76 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #76   +/-   ##
=======================================
  Coverage   84.11%   84.11%
=======================================
  Files           1        1
  Lines         170      170
=======================================
  Hits          143      143
  Misses         27       27

Impacted Files
Coverage Δ

dist/hotkeys.js
84.11% <100%> (ø)
: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 2c9814e...63498c5. Read the comment docs.

Pull Request Test Coverage Report for Build 133

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 70.822%

Totals

Change from base Build 132:
0.0%

Covered Lines:
143

Relevant Lines:
170

💛  - Coveralls

not press 'w', in ,press 'del' ,key '8' will be recorded in _downKeys

sorry, this is not hotkey-js's problem, but adjust the order of 'filter' may be better

@yyjjqq94 I tested no problem.


4.4 beta 6 - withRouter() causes subsequent -Renderings to fail.

Version

react-router: 4.4.0-beta.6
react-router-dom: 4.4.0-beta.6
react-dom: 16.7.0
react: 16.7.0

Test Case
https://codesandbox.io/s/k2ll7nroqv
Steps to reproduce
Wrapping any component that renders a <Route/> with withRouter() will cause an invariant violation:

Invariant failed: You should not use  outside a

Minimal example:
const export SubRoute() => { // using this export works
    return <Route component={DummyComponent} />
}

export default withRouter(SubRoute); // using this export fails.

Expected Behavior
The <Route/> should work regardless of whether or not it's parent is wrapped with withRouter.
This is sometimes needed where access to the current match object is needed for proper routing - like this: <Route to={`${this.props.match.url}\new} />.
Actual Behavior
Route throws an invariant violation.

It seems like I missread the error message and stacktrace. From what I can tell, withRouter() itself fails, not the subsequent .
After poking around in the code, I found the cause:  Importing withRouter from "react-router" instead of "react-router-dom".  That is quite unfortunate, as the documentation / examples for 4.3 indicate one should import withRouter from the "react-router" package. As seen here:
https://reacttraining.com/react-router/web/api/withRouter
Meaning one has to alias the existing imports or change all occurances. I'd consider this a breaking change.

There are two ways I was able to get your sandbox working:

Move the withRouter import:

import { withRouter } from "react-router-dom";

Remove the react-router dependency

It feels like a bug in CodeSandbox's module loading (cc @CompuIves). The reason for the error is that your application has two instances of React Router's context: one from the react-router dependency and one from react-router-dom's react-router dependency. Those should be the same thing (if I download and run it locally, there are no issues), which is why I think that it is a bug.

@pshrmn This is not exclusive to codesandbox. I stumbled accross this issue in a normal application too.
Using npm 6.5.0, webpack 4.29.0, and Babel 7.
Did you use yarn to try the reproduction locally?

I've tried installing its deps with both yarn and npm and both worked.

Can confirm that there's a duplicate version. Looking into it now!

Fixed with https://github.com/codesandbox/dependency-packager/commit/20033fd31afb7e93ec5dd49a4c404eeaa34ef4b8. It may take a while to propagate since we have pretty strong caching on the dependencies.

Nice. Thanks Ives!.


Duplicate received, balance showing incorrectly

In transaction, I received two duplicated transactions and now the balance is showing incorrectly.

all ok after leaving it on and close/open the app.


Generate an assignment with a config file that uses a given "startup" script or command

User Story:
As a... GatorGradle user, I want to...
generate an assignment with a config file that uses a given "startup" script or command.
Proposed Plan:
Research scripts and command implementation to gradle.properties file.

@kailaniwoodard The "startup" script user story refers specifically to configuring the startup header parameter in the config/gatorgrader.yml configuration file. See the GatorGradle documentation for an example. gradle.properties is probably not relevant for this user story.

Kind of important, but can be done at the very end if we have time


Is it meant to be used in dependency injection or not?

Hello there, I was just wondering if it makes sense to create one Translator and use dependency injection to get it to the classes it's needed in or if there should be a new Translator each time it is needed?
Best regards,
tasutafat

Hello,
on a technical level: The overhead to create multiple Translators itself is pretty low, the only thing is that multiple Translator objects would create multiple HTTP sessions (which is a bit of overhead).
I think the main question here is just an architectural decision. By default I would lean towards creating a single Translator which you can expose to your Application as a singleton or something similar.

Thank you very much for your response! I will add it to my Service collection!

@daniel-jones-deepl, @tasutafat,  would you evaluate the possibility to add some service collection extensions? I wrote a small POC in this PR #28.

Hey @antoniovalentini
Is this what you had in mind?
https://github.com/DeepLcom/deepl-dotnet/blob/issue-26%2B29/src/DeepL.Service/DeepLService.cs

Hey @DeeJayTC, yes that's what I had in mind. Since you're integrating this into the development branch, I think my PR is not useful anymore. Thanks.

Yea, i alread had it sitting here unfinished. Your PR reminded me to get it done! 😇


DNM: CI test

https://github.com/apple/swift-package-manager/pull/3569
@swift-ci please test

18:09:46 /home/buildnode/jenkins/workspace/pr-swift-tools-support-core-linux/branch-main/swiftpm failed (ret=128): ['git', 'fetch', 'origin', 'pull/3569/merge:ci_pr_3569', '--tags']
18:09:46 fatal: Couldn't find remote ref pull/3569/merge

🤔

Maybe an issue with the SwiftPM PR being a draft?

apple/swift-package-manager#3569
@swift-ci please test

Seems like it was a temporary issue, there's now the expected failure:

/home/buildnode/jenkins/workspace/pr-swift-tools-support-core-linux/branch-main/swift-tools-support-core/Tests/TSCBasicTests/miscTests.swift:18: error: miscTests.testExpectedFailure : failed - test that we see this failure

So we are indeed running the TSC tests twice right now and can merge the SwiftPM PR mentioned here.


nixos/kanata: init

Description of changes
I test simple commands like echo with enableCmd = true with these hardening options, and it works.

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[X] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

cc @jtroo You may be interested in those hardening options if you plan to provide a systemd service.

A friendly ping. Could we merge it?

since there seem to be no further comments, sure :)


Updates term assignment status for SemanticAssignment

Hi, Chris! I have tested this, the SemanticAssignment relationship is propagated correctly. Thanks for update!


[WIP] Optimize data imports (Fix #125)

Trying to fix #125, importing reimbursements and irregularities at once. Just fixed the problems shown in pull request #147 . Give me more feedback so I can start working on tests and so on. Thanks a lot!

Changes Unknown when pulling f67225bcc1e4ea0adcc763bbfe8901fed070d74d on Guilhermeslucas:guilhermeslucas-optimize-data-imports into ** on datasciencebr:master**.

Changes Unknown when pulling da0c88999a45e733b9eefde22ba99037f7d30ed3 on Guilhermeslucas:guilhermeslucas-optimize-data-imports into ** on datasciencebr:master**.

The tests are running ok. Shouldn't it break when running the pip install?
( I really don't know it)
Em 10/02/2017 15:56, "Eduardo Cuducos"<EMAIL_ADDRESS>escreveu:

@cuducos commented on this pull request.
In requirements.txt https://github.com/datasciencebr/jarbas/pull/148:

@@ -6,6 +6,7 @@ django-cors-middleware==1.3.1
django-test-without-migrations==0.4
djangorestframework==3.5.3
gunicorn==19.6.0
+pandas==0.19.2

The problem is that pip install -r requirements.txt would fail. And
requiring a Anaconda is not common for most webdevs — including the way
conda conflicts with virtualenv…
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/datasciencebr/jarbas/pull/148, or mute the thread
https://github.com/notifications/unsubscribe-auth/AJO89M-l_WM4Y0LNYAyFiGIBiHN8RAdVks5rbKTogaJpZM4L8lJu
.

The tests are running ok. Shouldn't it break when running the pip install?
( I really don't know it)

Tests passing is a good thing, for sure. But if pip install pandas fails and we have it by default probably we'll see issues being opened here. In my macOS it worked pretty fine, but was because I already had numpy and a couple of compilers installed. What OS do you use? Can you try to install numpy and then pandas via pip? Also maybe for some Linux users sudo apt-get install python-pandas or yum install python-pandas.

I can't make it work without anaconda, have no idea why. I'm running Ubuntu 14.04. That's really awkward, cause python manage.py test runs without any problem. I'm trying to run it, from the beginning in a virtual machine!

It works on a VM running Ubuntu 14.04. What should we do?

[…] What should we do?

Write tests. It shouldn't pass the test if it's not working ; )
I'm really sorry I'm not able to pair or something at the moment. I'm focused in other parts of the project, and, at the end of the day, this is working (I mean, this PR is a enhancement in terms os speed)… but hopefully you get my point. I don't want push pandas to the master branch and see the Issues panel flooded with It's not working, pip fails etc…

Closing this issue:
With @RosieDaSerenata Jarbas cannot afford to have its database wiped out each time we update reimbursement and suspicions. So this entire strategy to feed our database should be revised. Until now doing it quickly was a priority but from now on doing it without needing to --drop-all is the new black ; )
Probably a strategy with bulk update as in the irregularities.py will have a better fit.


feat: Adds support for authenticating via service account

Description
Fixes #151
Supports passing a path to a google service account file which is used for authentication
Type of Change

[x ] ✨ feat -- New feature (non-breaking change which adds functionality)
[ ] 🛠 fix -- Bug fix (non-breaking change which fixes an issue)
[ ] ❌ ! -- Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 refactor -- Code refactor
[ ] ✅ ci -- Build configuration change
[ ] 📝 docs -- Documentation
[ ] 🗑 chore -- Chore

@russellwheatley Are you able to approve the workflow? Does this PR sound like something you'd be interested in merging?

Hwllo guys. I appreciate the work you guys do on this CLI, butI am also curious to know what the ETA is on this as we have some CI workloads that would benefit from this feature?
The current state of the CLI causes the program to sometimes fail due to authentication. I have a service account that would probably allow me to solve the authentication issue by using it in the Flutter fire CLI.
Thanks

Hey @josh-burton, the next release isn't for at least a couple of weeks, so I will investigate if CI failure is occurring as a result of PR which I suspect it is before merging.

Hey @josh-burton, the next release isn't for at least a couple of weeks, so I will investigate if CI failure is occurring as a result of PR which I suspect it is before merging.

Thank you!

@josh-burton - do you mind fixing merge conflict and merging with latest on main? I think the tests ought to pass now 👍. Thanks!

@josh-burton - do you mind fixing merge conflict and merging with latest on main? I think the tests ought to pass now 👍. Thanks!

Sure, rebased now.

@josh-burton - I was trying to figure out why this was failing CI 🤔 . I realised secrets are not available for forked pull requests. I've ran locally and it works fine. Thanks once again 🎉

@josh-burton - I was trying to figure out why this was failing CI 🤔 . I realised secrets are not available for forked pull requests. I've ran locally and it works fine. Thanks once again 🎉

thanks for merging :)

Thanks for this feature! It would be good to add any notes on this to docs. For example, what is the minimal required role for service account for configure command?


fix(shared): remove the keydown listener once the thyEnter is destroyed

The reason why the keydown event was not unsubscribed is that the bound element is not a global, when the element is destroyed, the event will never be executed.
Are there any scenes that must be destroyed？

Including many other PR submitted by you, it is similar, as long as the event is not global, it seems unnecessary to remove it when it was destroyed.

Hey @why520crazy . I'm not sure I understand what means "event is global". The rule of thumb is always remove event listeners.
Lemme show you the following scenario:

given this template <button thyEnter></button>
the thyEnter adds a keydown event listener
zone.js stores the ZoneTask within the document.querySelector('button').__zone_symbol__keydownfalse with the callback (which is ThyEnterDirective.prototype.onKeydown)
the onKeydown captures this
the button is removed from DOM
the Blink does re-layout (removes the button from the screen) and marks the HTMLButtonElement as detached (to be GCed in the future)
the garbage collector never runs immediately, it then goes through the detached DOM tree to remove objects that are not referenced (dangling objects)
the button element references the component through the __zone_symbol__keydownfalse and the component references the button through the elementRef
at the end we have a circular reference

I have a small article that explains that mechanism here: https://ng-guru.io/guide/garbage-collection/
Cheers 🙌

I see, thank you for your explanation.

@arturovt to verify whether there is a memory leak when do not remove event listener, I wrote an example：
https://stackblitz.com/edit/angular-ivy-eewmnf
create and destroy app-child component every 500ms, bind click event listener in app-child component, and create big data assignment to this.data, it is found that there is no memory leak through the performance tool of Chrome Browser.

If print data through the console.log(this.data) in the event, memory leaks do occur.
do you know the specific reason or is there a vulnerability in this example？

I'm not very good at english. I hope you can understand, thanks！

Hey @why520crazy
To simplify the example I removed the setInterval from constructor and uncommented the button in app.component.html, so my template looks as follows:
<button (click)="showChild = !showChild">Destroy</button>
<app-child *ngIf="showChild"></app-child>

Then I clicked the button and made snapshot:

The HTMLElement is still retained and cannot be garbage collected. We see the address is 0x1ff2d24cd380, it's an HTMLElement which is aggregated within the objects list, we can search it through objects:

The HTMLElement is captures by __zone_symbol__clickfalse (which I showed you in the scenario).

Is this analysis display generated by your own tool?

I use Firefox heap analyzer and some addons written in Rust by Mozilla for querying snapshots.
I'm only curious what is the reason of not removing event listeners on destroy? Since this has been a best practice of not introducing memory leaks for decades starting from Angular 1.x (and other frameworks too):

Listeners registered to scopes and elements are automatically cleaned up when they are destroyed, but if you registered a listener on a service, or registered a listener on a DOM node that isn't being deleted, you'll have to clean it up yourself or you risk introducing a memory leak. (https://docs.angularjs.org/guide/directive)

This is an example of the commit to Material where they remove event listeners that don't capture this: https://github.com/angular/components/commit/764030d04f0097fe8215ce96cc32610778bf3ace
Garbage collectors behave differently in all engines, but they use the same algorithm for determining if an object can be removed or not (reference counting). But we're not responsible for forcing GC for specific objects.

I have no oppose to removing event listeners on destroy, I admit that this is a best practice, I'm just exploring how to use it to cause memory leaks.


ajour: 1.3.0 -> 1.3.1

Motivation for this change
https://github.com/ajour/ajour/releases/tag/1.3.1
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 137857 run on x86_64-linux 1

1 package built:

ajour


gmp: fails to build with --c++11

The gmp formula includes a --c++11 option, but it doesn't work on Linux:
brew install gmp --c++11
==> Downloading https://gmplib.org/download/gmp/gmp-6.1.0.tar.xz
Already downloaded: /home/ben/.cache/Homebrew/gmp-6.1.0.tar.xz
Error: The selected compiler doesn't support C++11: gcc

I don't really understand the error since I'm on a Fedora 23 system with gcc 5.3.1, which does support C++11 (maybe it's trying to compile C code rather than C++ with -std=c++11 and is getting confused?) Unfortunately I'm installing a formula which needs both C++11 and gmp, so brew is automatically trying to recompile gmp with --c++11 for me.
FWIW, I can make things "work" by mangling the gmp formula and removing the option :cxx11. But I suspect this only works because the other formula doesn't use any C++ code from gmp.

This is an anachronism that we've inherited from Homebrew. /usr/bin/gcc is assumed to be GCC 4.2 as it is on old Mac OS distributions, and I haven't fixed this in Linuxbrew. The workaround is to creating a symlink from /usr/bin/gcc to $HOMEBREW_PREFIX/bin/gcc-5, g++ and gfortran as well. See
https://github.com/Linuxbrew/linuxbrew/wiki/Standalone-Installation#create-symlinks-for-gcc


New way of referencing a value

Currently, we are using the dot-notation to refer to a particular value in a document. This notation is limited for several reasons:

it doesn't feel natural: accessing array indexes using dots instead of square brackets feels weird
it doesn't allow specifying ranges

This PR changes that by using a different notation, that is closer to what is used in Go, Python, Javascript, etc.
Here are some examples:
// simple field
a

// field with backquotes
`   some "field" `

// nested fields
a.b
a.` b `.c

// array indexes
a[0]
a.b[100][10].c

This will allow us to introduce interesting features in the future:
// slicing
a[1:3]
a[1:]

// accessing last element
a[-1]
// accessing second last element
a[-2]

// expressions
a[1 + 3]
a[ARRAY_LENGTH(a) - 4]

I believe this notation feels more natural since it's the same as in mainstream programming languages

When the field using backquotes with spaces like this "   some \"field\" ", the spaces must be deleted or not?
input:  \"   some "field" `=> output:" some "field" ",`
Is the expected behaviour?

@tzzed Please open an issue if you see something odd, discussing in closed pull requests is not ideal


Redirects in infinite loop if a body parser is not set up for express

Minor, but should be documented: when defining the callback route:
let app = express()
app.get('/login', passport.authenticate('saml', { failureRedirect: '/' }), (req, res) => res.redirect('/'))
app.post('/callback', passport.authenticate('saml', { failureRedirect: '/' }), (req, res) => res.redirect('/'))

There needs to be middleware that supports body parsing, otherwise (in my case) it starts redirecting between my IdP and my app indefinitely.

Can you submit a PR with the proposed documentation change?  Thanks!

I seem to be having this issue, but i'm using the node package body-parser, what body parser did you end up using to resolve this issue?

I feel like adding the body-parser middleware has caused functionality in Rocky to break.  When trying to reverse-proxy connections after ensuring authentication, the middleware fails with:

I also had the same issue. Adding the bodyParser finally resolved it. 😅
const bodyParser = require('body-parser');
app.use(bodyParser.urlencoded({ extended: false }));

Wasted days chasing this infinite loop issue. Many thanks to OP for creating this issue. There is no mention of it in documentation. :disappointed:

@mrchief I created a PR for this a long time ago, but it's had very little activity :( #151

Spent a couple hours this morning setting this up on a new app, and encountered the same problem.  It would be nice to have this in the documentation.

Its kinda in the documentation, in its own convoluted way: :wink:

This is closed, but I have spend too much time to find out the problem is in body-parser.
This should be as mandatory requirement on the mine README in bold. I don't get it why is so hard to mention this. Thanks for awesome library anyway.

Hello!  If anyone is using express 5, since body parser isn't needed, simply make sure the app.use(express.urlencoded({ extended: false })); before the route calls.
Many thanks to kashiif and makstos above for leading me down that path.  Immense Thanks!

@DORRITO Please submit a doc-patch for the Express 5 example syntax.

another victim who wastes days here. Thanks for sharing this and saving my life @jrop


[backport] Use search index

Backport of #6881

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 3870f2bbef3d7eb0a9a0e91676c5a0419368239d, target branch v6) failed with status FAILURE.

The Jenkins failure has been reported in #6905 and it is unrelated to the PR.

LGTM!


How to use setCustomerUserId function?

I need to add user id for CVV, how can i solve this?

Here CUID has not appearing on CVV too, any solution?

@15110011 I found the solution. It's necessary set CUID before initSdk.
@amit-kremer93 Why is it necessary? Is there a way to improve it?


Add latest and v0 tag

Can you add a 'latest' tag and 'v0' tag so that we can reference the latest version rather than a specific version eg:
uses: microsoft/powerplatform-actions/import-solution@latest

partially resolved with manually setting both the v0 and latest tags as aliases to the current v0.1.7 release
Will leave it open until automation is added


Add TeslaMate

Add TeslaMate

Thank you for taking over the maintenance of the TeslaMate integration. I have seen that you have also included the PostgreSQL addon in your repository. Can't you also officially reference the PostgeSQL in https://github.com/alexbelgium/hassio-addons?
In that case, the DB is always up to date. I have my TeslaMate running with this version without any problems.

Hi lildude. Is there a time line to add Teslamate add-on to this repository? I added your add-on repository, but Teslamate is not available in it. Thank you.

No time line. I plan to merge as soon as I've had a chance to properly test it... I've been really short of free time recently but things should hopefully ease up over the next week or so.

No time line. I plan to merge as soon as I've had a chance to properly test it... I've been really short of free time recently but things should hopefully ease up over the next week or so.

Thank you for your quick reply.

Testing confirms all working well.

Amazing. Just downloaded and everything working well. Thank you again for your hard work on this.


Config.unobserve is deprecated.

Config::unobserve no longer does anything. Call .dispose() on the object returned by Config::observe instead.
Config.unobserve (/Applications/Atom.app/Contents/Resources/app/src/config.js:374:19)
LinterPep8.destroy (/Users/ashrayjain/.atom/packages/linter-pep8/lib/linter-pep8.coffee:28:16)

https://github.com/AtomLinter/linter-pep8/pull/14


Add animation completion block

I need to know when the animation is complete. It would be wonderful if you could add another method that supports completion: setValue:animated:completion. I'm looking into it but it seems more difficult with implicit animation.

I wrote this. Will submit a PR tomorrow.

Bump


Add shading plugin

This PR adds an extra SBT plugin that allows to shade dependencies.
Use like
lazy val myProj = project
  .enablePlugins(coursier.ShadingPlugin)
  .settings(
    shadingNamespace := "myproj.shaded",
    libraryDependencies += "com" %% "foo" % "0.2.3" % "shaded"
  )

sbt myProj/shading:publish then publishes myProj with dependency towards com::foo shaded .
Once published, coursier would use it to shade fastparse, to prevent conflicts with other versions of it (like in https://github.com/alexarchambault/coursier/issues/395), and not to have coursier 1.0 final depend on a 0.x dependency.
It would be cool to have sbt-ensime use it to shade coursier itself, to prevent conflicts like https://github.com/alexarchambault/coursier/issues/417. (ping @fommil)

+1


Add autoheal container to CLI

Describe the Solution
We don't have an Autoheal container in the CLI.
Why?
We need this to start working on healthcheck measures

Congratulations for this great job!
It is a brilliant step in terms of control of autonomous agents (containers) and all the maintenance of their healthy status.
I appreciate the efforts that you are all doing here.


[Ghost tables] Visualization backup revamp

Deprecate the current visualization backups a.k.a visualization cemetery and create a new one that stores the deleted layer/maps .carto JSON in the database

I've been reviewing this part and I have some questions @javitonino:

We have two VisualizationBackup services: 1 and 2. Is safe to revamp/remove version 1 and adapt to our current requirements?
If I'm not mistaken we should use version 2 to get the visualization's JSON and store it in the VisualizationBackup table. Is that right?

Yes and yes :rocket:
VisualizationsExportService is the old one that creates a VisualizationBackup model with the vizjson.
VisualizationsExportService2 is the new one (.carto) that returns a JSON with all metadata (akin to mapcaps).
In fact, you can look at mapcaps since this implementation will probably be very similar to this new backup system (could we reuse some code? not sure).

Cool thank you!

After looking at this issue, the doc and the code the final objective is to save a visualization with VisualizationsExportService2Importer without dataset into visualizations_backup table before ghost table deletes a visualization table.
Right now, we are saving an export_vizjson in the visualizations_backup table. Should we stop this? should we keep these records?
A solution could be to change the current visualization backup hooked before_destroy in the Visualization model, using VisualizationsExportService2Importer. Is not it?

The vizjson is almost worthless as a backup since it can't be recovered via automated means. Feel free to get rid of it and all existing data. We have never recovered one of those because it's basically impossible.

A solution could be to change the current visualization backup hooked before_destroy in the Visualization model, using VisualizationsExportService2Importer. Is not it?

I think that should be possible, yes.

These are the task I am going to implement. If you see something wrong or forgotten, ping me:

[ ] Stop current backups disabling FF visualizations_backup
[ ] Remove FF visualizations_backup and the code behind it
[ ] Drops visualizations_backup & create again (changing the export_vizjson to a new one more generic like export). Or maybe removing data and altering the column name.
[ ] Use VisualizationsExportService2Importer in before_destroy of the Visualization model
[ ] Make something similar to the previous step with the Layer model.
[ ] Deploy

:rocket:


Error: Cannot read property getSupportInfo of null - on coc-prettier startup

On Vim startup, as coc-nvim initializes, I'm getting the following error:

I'm using vim-plug to manage coc-nvim plugins. The error is happening in a project (my dotfiles) where I don't have a prettier config file, but I do have some random JSON files (like coc-settings.json 😄). I also don't have an .eslintrc or .eslintignore file in the project.
Interestingly, the error doesn't happen on startup in a javascript project where there are .eslintrc and prettier config files.
Huge props for making such an amazing tool for Vim! I really love it and the plugin system is amazing and perfect.
Here's the output of CocInfo:
## versions

vim version: NVIM v0.3.8
node version: v12.9.1
coc.nvim version: 0.0.74
term: iTerm.app
platform: darwin

## Messages
coc.nvim initialized!
[coc.nvim] Error on active extension coc-prettier: TypeError: Cannot read property 'getSupportInfo' of null
## Output channel: watchman

[Info] watchman watching project: <project>
[Info] subscribing "**/.eslintr{c.js,c.yaml,c.yml,c,c.json}" in <project>
[Info] subscribing "**/.eslintignore" in <project>
[Info] subscribing "**/package.json" in <project>

Oh wait, I read through some coc-prettier code and I think I know what's going on:
I don't have prettier installed globally, only locally in specific projects. So here, in the initialization code, it's not finding any installed version, and that's probably what's causing the prettier instance to be null https://github.com/neoclide/coc-prettier/blob/b82606312da9baa85f1ff5b601d6217f4b86b67f/src/utils.ts#L27

Can't reproduce.


unable to open /cnb/buildpacks/paketo-buildpacks_opentelemetry/0.2.0/bin/helper

I am trying to use this buildpack with spring's bootBuildImage task.
I am using the default builder (paketobuildpacks/builder:base) along with the following buildpacks:
buildpacks = ['paketo-buildpacks/java', 'gcr.io/paketo-buildpacks/opentelemetry']
and the following env vars:
environment = ["BP_OPENTELEMETRY_ENABLED" : "true"]
I am getting this error from bootBulldImage:
[creator]     Running build for buildpack<EMAIL_ADDRESS>    [creator]     Looking up buildpack
    [creator]     Finding plan
    [creator]     Creating plan directory
    [creator]     Preparing paths
    [creator]     Running build command
    [creator]
    [creator]     Paketo OpenTelemetry Buildpack 0.2.0
    [creator]       https://github.com/paketo-buildpacks/opentelemetry
    [creator]       OpenTelemetry Java Agent 1.19.1: Contributing to layer
    [creator]         Downloading from https://github.com/open-telemetry/opentelemetry-java-instrumentation/releases/download/v1.19.1/opentelemetry-javaagent.jar
    [creator]         Verifying checksum
    [creator]         Copying to /layers/paketo-buildpacks_opentelemetry/opentelemetry-java
    [creator]         Writing env.launch/JAVA_TOOL_OPTIONS.append
    [creator]         Writing env.launch/JAVA_TOOL_OPTIONS.delim
    [creator]         Writing env.launch/OTEL_JAVAAGENT_ENABLED.default
    [creator]         Writing env.launch/OTEL_METRICS_EXPORTER.default
    [creator]       Launch Helper: Contributing to layer
    [creator]     unable to invoke layer creator
    [creator]     unable to open /cnb/buildpacks/paketo-buildpacks_opentelemetry/0.2.0/bin/helper
    [creator]     open /cnb/buildpacks/paketo-buildpacks_opentelemetry/0.2.0/bin/helper: no such file or directory
    [creator]     ERROR: failed to build: exit status 1

And the task fails without building the image.
Expected Behavior
File should be found and image should be created.
Current Behavior
as listed above.
Steps to Reproduce
bootBuildImage {
    imageName = ".."
    runImage = ".."
    environment = ["BP_OPENTELEMETRY_ENABLED" : "true"]
    buildpacks = ['paketo-buildpacks/java', 'gcr.io/paketo-buildpacks/opentelemetry']
    verboseLogging = true
}

Motivations
This buildpack fits perfectly with my needs to add otel java agent to my images.
I would love to use it and I can't figure out what's wrong.
Any help will be appreciated!

Apologizes. I'm not quite sure how that has slipped through testing. The helper binary was not being packaged with the buildpack. I've got a PR open to fix this. When that's merged, I'll cut another beta release.
Thanks for testing & for the feedback!

@talberadt  v0.2.1 is publishing now. Should be ready in a few mins. If you could give that a try, I think it should get you past this error. Thanks

@dmikusa Works like a charm.
Thanks for your work and quick response!


Internal Server Error

Internal Server Error (500)
This Serverless Function has crashed.
This happens when you try to go any route /:player, /:player/characters
sometimes works or sometimes dont work at all (error 500)

Fixed: Moving Axios from devDependencies to dependencies.


Add calibration-to-noise pipeline

This PR reinstates the cirq_google utility for converting Calibration objects to noise properties. To create a simulator with behavior that mimics some Calibration "cal", the expected usage is:
noise_props = cg.noise_properties_from_calibration(cal)
noise_model = cg.NoiseModelFromGoogleNoiseProperties(noise_props)
simulator = cirq.Simulator(noise=noise_model)  # will mimic noise from `calibration`

This is part 4 of #4666; part 3 was #5082. Remaining steps include:

Convenience methods for modifying noise properties
Reinstatement of compare_generated_noise_to_metrics using new types

Removed the approval mostly to prevent myself from absentmindedly merging this before the circular dependencies are resolved - I'll ping reviewers when this is ready for re-review.

Okay, I've moved these file to engine and removed circular imports. Imports are working, verified in another PR (which will go out separately from this and #5188).
This is ready for re-review.


Is there a plan to support Pie charts ?

Is there a plan to support Pie charts ?

UP?

I doubt it will get added. The reasons:

drawing a pie chart is 100% different codewise than drawing line graphs.
sparklines are definitely not pie charts


Duplication of columns on each table

Love this tool! Using it for a MySQL instance hosted on AWS RDS. I'm running into an issue causing 95% (not all) of the columns in each table to duplicate twice (for a total of three columns with the same name/type). The process runs and completes successfully with the duplication issue and it also doesn't generate any constraints even though they're set up from the MySQL side.
Has this been fully tested on a MySQL db? Just want to make sure I'm doing everything right. Here's an example of the yaml file I'm using to run the process:
# Connection properties
connectionString: "mysql://username:password@tcp(database.identifier.us-region-1.rds.amazonaws.com:3306)/default-schema"

# Define what tables should be used
useAllTables: true

# Additional flags
showAllConstraints: true
encloseWithMermaidBackticks: true
outputFileName: "default-schema-mermerd.md"
debug: true
omitConstraintLabels: true

There is a hyphen "-" in our schema name. Could that be causing a problem? The only odd thing I've noticed is that (because it's MySQL) we're specifying the schema in the connection string and then the CLI asks again for the schema once it connects to the db instance. That might just be a MySQL thing because I know Postgres is set up differently to have a DB and then have Schemas under each DB whereas MySQL considers schemas like Postgres does a database.
Thanks!

Glad you find the tool useful :)
All supported databases are automatically tested via the integration tests (but with a defined set of tables, so not all edge cases are covered)
Could you provide the ddl statements for the tables that caused the issue? Then I can easily reproduce/analyze the issue and add a testcase to prevent the issue for future changes.

Closing this issue due to inactivity. If the issue is still present feel free to reopen it.


feat: using initial date before now date

Goal of this PR : Solves https://github.com/aeecleclair/Titan/issues/235

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (31fe974) 55.96% compared to head (1952f7c) 55.96%.

Files
Patch %
Lines

lib/tools/functions.dart
0.00%
3 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #244   +/-   ##
=======================================
  Coverage   55.96%   55.96%
=======================================
  Files         166      166
  Lines        3581     3581
=======================================
  Hits         2004     2004
  Misses       1577     1577

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.