feat(firestore): Partition samples

Related python PR,  java PR.
Not sure about collectionGroupID record.
How do you think will be better?

collectionGroupID := collectionID + "-subcollection"
collectionGroupID := tc.ProjectID + "-subcollection"?

And collectionGroupID or maybe rename to subcollection?

@AlisskaPie,

How do you think will be better?
collectionGroupID := collectionID + "-subcollection"
collectionGroupID := tc.ProjectID + "-subcollection"?

collectionID already include tc.ProjectID, so the first variant looks better.

Maybe this way will be easier to read:
parent := fmt.Sprintf("projects/%v/databases/(default)/documents", projectID)?

Yes


update snappy import path to new repository location

use "github.com/golang/snappy" instead of "code.google.com/p/snappy-go/snappy"

Fixed in #455... I wish go had proper dependency management :(


NodePort/LoadBalancer defined ports are not reachable from host

Hi,
I am running a service on bare-metal and EKS cluster and it is working but when I run it on microk8s (version v1.16.15) I am not able to connect to the NodePort that I defined but I am able to access it if I use the random port.
$ kubectl get svc
NAME               TYPE           CLUSTER-IP       EXTERNAL-IP   PORT(S)          AGE
kubernetes         ClusterIP      <IP_ADDRESS>     <none>        443/TCP          125d
log-gen            ClusterIP      <IP_ADDRESS>    <none>        8080/TCP         18d
log-gen-publicip   NodePort       <IP_ADDRESS>   <none>        8080:31555/TCP   18d
wwslogs            ClusterIP      <IP_ADDRESS>   <none>        9080/TCP         46m
wwslogs-publicip   LoadBalancer   <IP_ADDRESS>     <pending>     9080:30097/TCP   46m

And when I try to access them:
$ curl http://localhost:8080/
curl: (7) Failed to connect to localhost port 8080: Connection refused

# But the random port works
$ curl http://localhost:31555/
{"Hello":"World"}

# It is the same for LoadBalancer
$ curl http://localhost:9080/version
curl: (7) Failed to connect to localhost port 9080: Connection refused

$ curl http://localhost:30097/version
{"version":"0.0.3\n"}

I also tried <IP_ADDRESS> and <IP_ADDRESS> but the results were the same.
Microk8s info:
$ microk8s.status
microk8s is running
addons:
cilium: disabled
dashboard: disabled
dns: enabled
fluentd: disabled
gpu: disabled
helm: enabled
ingress: disabled
istio: disabled
jaeger: disabled
knative: disabled
linkerd: disabled
metrics-server: enabled
prometheus: disabled
rbac: disabled
registry: enabled
storage: enabled

$ microk8s.inspect
Inspecting services
  Service snap.microk8s.daemon-cluster-agent is running
  Service snap.microk8s.daemon-flanneld is running
  Service snap.microk8s.daemon-containerd is running
  Service snap.microk8s.daemon-apiserver is running
  Service snap.microk8s.daemon-apiserver-kicker is running
  Service snap.microk8s.daemon-proxy is running
  Service snap.microk8s.daemon-kubelet is running
  Service snap.microk8s.daemon-scheduler is running
  Service snap.microk8s.daemon-controller-manager is running
  Service snap.microk8s.daemon-etcd is running
  Copy service arguments to the final report tarball
Inspecting AppArmor configuration
Gathering system information
  Copy processes list to the final report tarball
  Copy snap list to the final report tarball
  Copy VM name (or none) to the final report tarball
  Copy disk usage information to the final report tarball
  Copy memory usage information to the final report tarball
  Copy server uptime to the final report tarball
  Copy current linux distribution to the final report tarball
  Copy openSSL information to the final report tarball
  Copy network configuration to the final report tarball
Inspecting kubernetes cluster
  Inspect kubernetes cluster

WARNING:  Docker is installed.
Add the following lines to /etc/docker/daemon.json:
{
    "insecure-registries" : ["localhost:32000"]
}
and then restart docker with: sudo systemctl restart docker
Building the report tarball
  Report tarball is at /var/snap/microk8s/1671/inspection-report-20210319_114427.tar.gz

inspection-report-20210319_114427.tar.gz

Hi @avarf
This behavior looks right to me.
Looking at line:
log-gen-publicip   NodePort       <IP_ADDRESS>   <none>        8080:31555/TCP   18d

The port 8080 of the deployment is exposed on node's port 31555.
For the LoadBalancer:
wwslogs-publicip   LoadBalancer   <IP_ADDRESS>     <pending>     9080:30097/TCP   46m

Internal port 9080 is exposed on the port 30097 of the host but there is no LoadBalancer enabled in MicroK8s so the ExternalIP is marked as <pending>.

But this is different from running on a bare-metal or AWS EKS cluster, on those clusters the port that you define in the yaml/helm will be accessible (in my case 8080 and 9080).
This behavior of Microk8s makes it very hard to work with because it means we need to change the port in the scripts and clients every time. Is there any workaround for this?

For example Minikube is behaving exactly similar to a full K8s cluster: https://minikube.sigs.k8s.io/docs/handbook/accessing/#check-external-ip

My bad, I had to add and configure the externalIP


Evaluate and add tmux

want to have several terminals in the container
do not want to enter for each terminal with docker exec
think tmux is one waz to do that
tried but then the lunarvim does not look good

I looked around and found a related issue in the LunarVim repository: https://github.com/LunarVim/LunarVim/issues/4057
In case you want to have it in the image, please open a pr. Otherwise pass the custom config when launching the container.


Add additional parameter to FAILED_TESTS token to enable yaml output generation

FAILED_TESTS token is often used to generate emails of build failures.
The output is currently a plain text file. It is very hard to iterate through this text file to read the failures case by case. Hence, a format easily readable and iterable would be beneficial.
YAML is a human readable as well as machine readable format.

Please resolve conflicts now that your other PR has been merged.

@haridsv Addressed your comments. Please have a look.

Hi @slide
We are waiting on this feature. When is the next release cycle for this plugin ?


Create animals table

In this pull request, I used a relational database to create the initial data structure for a vet clinic. I created a table to store animals' information, insert some data into it, and query it.

Activities involved are:

Created database vet_clinic.
Created a table named animals in the created database.
Inserted data into the table.
Queried the table for different data information.

[ ] Screnshots for queries

Thank you @TuyishimireEric


adding new line in cells?

i'm trying to add a new line to a cell and i can't get it to work. Not sure if it's a bug or i'm doing something wrong.
pseudo code:
const rt = new RichText()
rt.add('hello\nworld')
sheet.cell('A1').value(rt).style({wrapText: true})

It sets the wrap text property on the cell but my \n is ignored. I would expect there to be a line break in the cell.

Did you figure it out?

@papb hi yes, it was my own fault. i did not realize i had code that explicitly removed certain tokens which was causing the issue:
value.replace(/[^\x20-\x7E]/g, '')

While i have you...is there a setting to expand cells to fit their data? like the opposite of shrinkToFit?

While i have you...is there a setting to expand cells to fit their data? like the opposite of shrinkToFit?

Like this?

I ended up developing my own library (on top of exceljs) to achieve exactly this: https://github.com/papb/json-excel/


Python: use buildEnv for application, remove PYTHONPATH wrapper

Motivation for this change
I got the following idea to make Python applications work, without wrapping all scripts with a wrapper that sets PYTHONPATH. In this PR I introduce a function that builds a python.buildEnv for the chosen Python application, and then another derivation that symlinks only the scripts of the application, but then via the buildEnv. This works because python.buildEnv sets PYTHONHOME.
Eventually, I will want to put this function in the passthru of the interpreter.
Things done

[x] Tested using sandboxing
(nix.useChroot on NixOS,
or option build-use-chroot in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

@RonnyPfannschmidt this might be of interest to you.

hmm, should have tested this better.


feat: copy package.json to dist during build if it exists

Close #87

Nice work! The only issue is that I think package.json should be copied over if it exists (regardless of any generator options), since that's the behavior of other Nx library build executors.
So I removed the plugin from the generators and added it to the build executor. It seems to work!

Ah, I also removed the migrations file. The reason is that when upgrading Nx versions, I like to review any breaking changes manually and see if any modifications are actually needed in our repo. In this case, the migration is just about the nx wrapper script, which is only necessary in non TS/JS projects.

Merged, but leaving the issue open in case we want to add some more advanced package.json handling later.


Content-Length with obsolete line folding is accepted with invalid input

According to https://tools.ietf.org/html/rfc7230#section-3.2.4
requests/responses with obsolete line folding can be rejected or obs-fold can be treated as one or more spaces.
obs-fold       = CRLF 1*( SP / HTAB )
Currently any input after an obsolete line folding in Content-Length field would be accepted.
Example request:
"POST / HTTP/1.1\r\n"
"Content-Length:  42\r\n"
" ANY TEXT\r\n"  // would be accepted and reported as part of Content-Length header field value
"\r\n"
...

There is no test yet, and it was reporting error before the commit https://github.com/nodejs/http-parser/commit/01da95feade5e5612499f5374498e7968c1a4a82 was introduced.
The issue happens because after obsolete line folding is discovered the parser goes back to s_header_value_start. But as new content length states h_content_length_num, h_content_length_ws are not handled there it goes to default case and h_general state.

Thank you for checking it. Your analysis of the code path is absolutely correct! Did I understand it right, that after 01da95f the bug is no longer reproducible?

No, vice versa, the bug appears after
https://github.com/nodejs/http-parser/commit/01da95feade5e5612499f5374498e7968c1a4a82

@obatysh would you care to clarify if this error is actually reproducible and you experienced it?

Nevermind, I can reproduce it now.


Result about the NYUD-v2 test data on S2RDepthNet

Hi, I'm evaluating the trained S2RDepthNet on NYUD-v2 test data by the command you provided, but I can see nothing from the "pred_depth_np" when I use "cv2.imwrite()" to save it,  when I print the "pred_depth_np",  all data are between 1 and 5, do you know where is the problem ?
Look forward to your reply, thank you!

cv2.imwrite()默认应该使用了uint8保存，也就是像素值范围为0~255，在这个范围内显示0~5的值肯定是看不到的，整个图片应该是全黑。如果你想看到可视化效果，建议可以将0~5等比缩放到0~255，然后再保存


run wrong

After injection, mouse click has no response

Did you add junk code? Maybe try opening and closing the menu?

Did you add junk code? Maybe try opening and closing the menu?


Add editor to command context

I have a couple of Commands which alter the state of the document editor, rather than the document itself (for switch editing modes). I wrote a base command class for that whose execute method needs to be able to access the editor:
  this.execute = function(props, context) {
    // Toggle active switch
    this.active = !this.active;
    // Since this command does not trigger a document update
    // we need to trigger an update to command states BEFORE
    // extending the state of, and thus rerending, the editor
    context.editor.commandManager.updateCommandStates();
    // Update the editor state
    var state = {};
    state[this.getStateName()] = this.active;
    context.editor.extendState(state);
    return true;
  }

The best way I could see to do that was to add the editor to the command context - hence the single line commit in this PR. But please let me know if there is a better approach.

You should not do this as you tangle lower level components to the top-level editors API. The preferred way for communicating with parent components is to to send an action (with the minimal information needed) that gets handled by the top level component.
// In the child component
this.onClick = function() {
  var newContextId = ...
  this.send('switch-context', newContextId);
};

// In the top-level component's constructor
function MyEditor() {
  MyEditor.super.apply(this, arguments);
  this.handleAction({
    'switch-context': this._switchContext
  });
}

Thanks a lot @michael, that is SO much better! I was really on the wrong track - for some reason was assuming that Tools always needed to have an associated Command.


Renaming stream causes problem in changing stream type

This is similar to #1540.
Reproducer: Rename a stream. Once successfully renamed, click on "Make stream public/private" button. It shows error message which says "Error making stream private: Unknown stream {{old_stream_name}}".

Cool, to confirm, can you reproduce this on master?  I would have thought the fix for #1540 would have worked here as well.

Yes, this is on master.

Cool, definitely worth debugging then

I fixed this a couple days ago, thanks @krtkmj for the bug report!


undefined method `links' for nil:NilClass (NoMethodError)

I've just updated my gems to the latest Middleman (I was already on v4) and now I'm getting the following when trying to use activate :sprockets
/Users/olliekav/.rvm/gems/ruby-2.3.0@capra/bundler/gems/middleman-sprockets-9515bcee4c9e/lib/middleman-sprockets/extension.rb:165:in `process_candidate_sprockets_resource': undefined method `links' for nil:NilClass (NoMethodError)
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/bundler/gems/middleman-sprockets-9515bcee4c9e/lib/middleman-sprockets/extension.rb:73:in `block in manipulate_resource_list'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/bundler/gems/middleman-sprockets-9515bcee4c9e/lib/middleman-sprockets/extension.rb:72:in `map'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/bundler/gems/middleman-sprockets-9515bcee4c9e/lib/middleman-sprockets/extension.rb:72:in `manipulate_resource_list'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/middleman-core-4.1.3/lib/middleman-core/sitemap/store.rb:234:in `block (4 levels) in ensure_resource_list_updated!'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/activesupport-4.2.6/lib/active_support/notifications.rb:166:in `instrument'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/middleman-core-4.1.3/lib/middleman-core/util.rb:21:in `instrument'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/middleman-core-4.1.3/lib/middleman-core/sitemap/store.rb:232:in `block (3 levels) in ensure_resource_list_updated!'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/hamster-3.0.0/lib/hamster/vector.rb:1316:in `each'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/hamster-3.0.0/lib/hamster/vector.rb:1316:in `traverse_depth_first'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/hamster-3.0.0/lib/hamster/vector.rb:431:in `each'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/middleman-core-4.1.3/lib/middleman-core/sitemap/store.rb:231:in `block (2 levels) in ensure_resource_list_updated!'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/activesupport-4.2.6/lib/active_support/notifications.rb:166:in `instrument'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/middleman-core-4.1.3/lib/middleman-core/util.rb:21:in `instrument'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/middleman-core-4.1.3/lib/middleman-core/sitemap/store.rb:224:in `block in ensure_resource_list_updated!'
	from /Users/olliekav/.rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/monitor.rb:214:in `mon_synchronize'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/middleman-core-4.1.3/lib/middleman-core/sitemap/store.rb:221:in `ensure_resource_list_updated!'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/middleman-core-4.1.3/lib/middleman-core/sitemap/extensions/on_disk.rb:21:in `ready'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/middleman-core-4.1.3/lib/middleman-core/callback_manager.rb:57:in `instance_exec'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/middleman-core-4.1.3/lib/middleman-core/callback_manager.rb:57:in `block in execute'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/hamster-3.0.0/lib/hamster/vector.rb:1316:in `each'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/hamster-3.0.0/lib/hamster/vector.rb:1316:in `traverse_depth_first'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/hamster-3.0.0/lib/hamster/vector.rb:431:in `each'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/middleman-core-4.1.3/lib/middleman-core/callback_manager.rb:57:in `execute'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/contracts-0.13.0/lib/contracts/method_reference.rb:43:in `send_to'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/contracts-0.13.0/lib/contracts/call_with.rb:76:in `call_with'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/contracts-0.13.0/lib/contracts/method_handler.rb:138:in `block in redefine_method'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/middleman-core-4.1.3/lib/middleman-core/callback_manager.rb:28:in `block in install_methods!'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/middleman-core-4.1.3/lib/middleman-core/application.rb:297:in `initialize'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/middleman-core-4.1.3/lib/middleman-core/preview_server.rb:142:in `new'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/middleman-core-4.1.3/lib/middleman-core/preview_server.rb:142:in `initialize_new_app'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/middleman-core-4.1.3/lib/middleman-core/preview_server.rb:34:in `start'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/middleman-cli-4.1.3/lib/middleman-cli/server.rb:53:in `server'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/thor-0.19.1/lib/thor/command.rb:27:in `run'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/thor-0.19.1/lib/thor/invocation.rb:126:in `invoke_command'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/thor-0.19.1/lib/thor/invocation.rb:133:in `block in invoke_all'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/thor-0.19.1/lib/thor/invocation.rb:133:in `each'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/thor-0.19.1/lib/thor/invocation.rb:133:in `map'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/thor-0.19.1/lib/thor/invocation.rb:133:in `invoke_all'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/thor-0.19.1/lib/thor/group.rb:232:in `dispatch'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/thor-0.19.1/lib/thor/invocation.rb:115:in `invoke'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/thor-0.19.1/lib/thor.rb:40:in `block in register'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/thor-0.19.1/lib/thor/command.rb:27:in `run'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/thor-0.19.1/lib/thor/invocation.rb:126:in `invoke_command'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/thor-0.19.1/lib/thor.rb:359:in `dispatch'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/thor-0.19.1/lib/thor/base.rb:440:in `start'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/gems/middleman-cli-4.1.3/bin/middleman:66:in `<top (required)>'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/bin/middleman:23:in `load'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/bin/middleman:23:in `<main>'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/bin/ruby_executable_hooks:15:in `eval'
	from /Users/olliekav/.rvm/gems/ruby-2.3.0@capra/bin/ruby_executable_hooks:15:in `<main>'

Gemfile
# If you do not have OpenSSL installed, change
# the following line to use 'http://'
source 'https://rubygems.org'

gem 'middleman', '~> 4.1', '>= 4.1.3'
gem "turbolinks", github: 'rails/turbolinks', branch: 'master', :require => false
gem "middleman-sprockets", github: "middleman/middleman-sprockets"
gem 'middleman-livereload'
gem "middleman-syntax", github: 'thisiscapra/middleman-syntax', branch: 'master'
gem "contentful_middleman", github: 'contentful/contentful_middleman', branch: 'dl/upgrade-to-v4'
gem "nokogiri"
gem "redcarpet"
gem 'therubyracer'
gem 'oj'

Any ideas? Is it something thats changed in the latest Middleman Core or Middleman Sprockets?

Middleman Sprockets has been evolving quickly these last few weeks. I'll let @stevenosloan chime in if he has a lead.

Yeah this is definitely middleman-sprockets -- I think coming from me deferring rendering/error handling. I'll have some time to dig in a bit more later today.

Hey @olliekav I've added some debugging output, you mind running from master and posting the Sprockets Debug: ... lines? It looks like there's assets being marked as processable, but then the set sprockets_path isn't found in the sprockets lookup paths. The debug lines should give us some insight into what the asset is, and how we're telling sprockets to find it.

@stevenosloan Sure, I'll do this today. Thanks!

Hey @stevenosloan, apologies for the delay getting this back to you. Output below...
== Sprockets Debug: #<Middleman::Sitemap::Resource path=fonts/263905/027807B1003227F6A.css>
== Sprockets Debug: #<Middleman::SprocketsExtension::SprocketsResource path=fonts/263905/027807B1003227F6A.css>

So it looks like a problem with the fonts from typography.com, it's self hosting and I had original issues with them in the build so I ignore them and add them back in after_build (see below).
# Build-specific configuration
configure :build do

  # Minify CSS on build
  activate :minify_css

  # Minify Javascript on build
  activate :minify_javascript

  # Enable cache buster
  activate :asset_hash

  activate :gzip

  # Use relative URLs
  # activate :relative_assets

  # Fix bug with build errors and typography fonts
  ignore 'fonts/*'

  after_build do |builder|
    print "After_build fixes... "
    FileUtils.cp_r(Dir['source/fonts/'],'build/')
    puts "done."
  end

end

I'm guessing it may be something to do with that after_build method?

Awesome, thanks @olliekav -- I think I can see what's happening. Looks like I might have goofed when I allowed sprockets to handle assets outside of css_dir or js_dir. It'll take me a little bit to get a fix up, but I think adding source to your sprockets path should work for you in the meantime.
activate :sprockets
sprockets.append_path File.join( root, 'source' )

@stevenosloan Yep, that's working for me. Thanks for being so prompt looking into it. Much appreciated!

@olliekav should be good on master now, thanks for the report.


App Crash from Firebase Remote Config

Description
The app is crashing immediately after opening, as indicated from Crashlytics as it's tagged under "Early crashes". The crash is happening randomly.
When the app starts up I have made sure that  func fetch(     withExpirationDuration expirationDuration: ,     completionHandler: ((RemoteConfigFetchStatus, ?) -> [Void])? = nil ) method is only invoked once to avoid any multiple calls
I am also using RemoteConfigRealTimeUpdate as well to fetch latest values whenever changes happen.
Can anyone provide me guidance on how I can resolve this crash as it's pointing to RCNConfigRealtime.m:449 and I have no insight on what's causing it
Reproducing the issue
No response
Firebase SDK Version
10.15
Xcode Version
15.2
Installation Method
Swift Package Manager
Firebase Product(s)
Remote Config
Targeted Platforms
iOS
Relevant Log Output
Fatal Exception: NSGenericException
0  CoreFoundation                 0xec678 (Missing UUID 5a6c1f41bf7032f6a1d65b894dd21362)
1  libobjc.A.dylib                0x2bc80 objc_exception_throw
2  CoreFoundation                 0x170f00 (Missing UUID 5a6c1f41bf7032f6a1d65b894dd21362)
3  My App                         0x1164020 __53-[RCNConfigRealtime fetchLatestConfig:targetVersion:]_block_invoke_2 + 449 (RCNConfigRealtime.m:449)
4  libdispatch.dylib              0x26a8 (Missing UUID c49a5fc2c87339139603004f5d0175e7)
5  libdispatch.dylib              0x4300 (Missing UUID c49a5fc2c87339139603004f5d0175e7)
6  libdispatch.dylib              0x12998 (Missing UUID c49a5fc2c87339139603004f5d0175e7)
7  libdispatch.dylib              0x125b0 (Missing UUID c49a5fc2c87339139603004f5d0175e7)
8  CoreFoundation                 0x36f9c (Missing UUID 5a6c1f41bf7032f6a1d65b894dd21362)
9  CoreFoundation                 0x33ca8 (Missing UUID 5a6c1f41bf7032f6a1d65b894dd21362)
10 CoreFoundation                 0x333f8 (Missing UUID 5a6c1f41bf7032f6a1d65b894dd21362)
11 GraphicsServices               0x34f8 (Missing UUID ac07ebbed8bc3e55a13a07bb548734d1)
12 UIKitCore                      0x22c8a0 (Missing UUID 2d5384466e403c108a5f559c938077a0)
13 UIKitCore                      0x22bedc (Missing UUID 2d5384466e403c108a5f559c938077a0)
14 My App                         0x11990 main + 26 (AppDelegate.swift:26)
15 ???                            0x1bc9badcc (Missing)

If using Swift Package Manager, the project's Package.resolved

Expand Package.resolved snippet

{
  "pins" : [
    {
      "identity" : "abseil-cpp-binary",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/google/abseil-cpp-binary.git",
      "state" : {
        "revision" : "bfc0b6f81adc06ce5121eb23f628473638d67c5c",
        "version" : "1.2022062300.0"
      }
    },
    {
      "identity" : "afnetworking",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/AFNetworking/AFNetworking.git",
      "state" : {
        "revision" : "ffae2391ab0c29dc88eb0a58d2f5b2c2c27cadbf",
        "version" : "4.0.1"
      }
    },
    {
      "identity" : "firebase-ios-sdk",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/firebase/firebase-ios-sdk",
      "state" : {
        "revision" : "8a8ec57a272e0d31480fb0893dda0cf4f769b57e",
        "version" : "10.15.0"
      }
    },
    {
      "identity" : "googleappmeasurement",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/google/GoogleAppMeasurement.git",
      "state" : {
        "revision" : "03b9beee1a61f62d32c521e172e192a1663a5e8b",
        "version" : "10.13.0"
      }
    },
    {
      "identity" : "googledatatransport",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/google/GoogleDataTransport.git",
      "state" : {
        "revision" : "5056b15c5acbb90cd214fe4d6138bdf5a740e5a8",
        "version" : "9.2.0"
      }
    },
    {
      "identity" : "googleutilities",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/google/GoogleUtilities.git",
      "state" : {
        "revision" : "c38ce365d77b04a9a300c31061c5227589e5597b",
        "version" : "7.11.5"
      }
    },
    {
      "identity" : "grpc-binary",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/google/grpc-binary.git",
      "state" : {
        "revision" : "f1b366129d1125be7db83247e003fc333104b569",
        "version" : "1.50.2"
      }
    },
    {
      "identity" : "gtm-session-fetcher",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/google/gtm-session-fetcher.git",
      "state" : {
        "revision" : "5ccda3981422a84186387dbb763ba739178b529c",
        "version" : "2.3.0"
      }
    },
    {
      "identity" : "interop-ios-for-google-sdks",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/google/interop-ios-for-google-sdks.git",
      "state" : {
        "revision" : "2d12673670417654f08f5f90fdd62926dc3a2648",
        "version" : "100.0.0"
      }
    },
    {
      "identity" : "kingfisher",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/onevcat/Kingfisher.git",
      "state" : {
        "revision" : "c75584ac759cbb16b204d0a7de3ebf53ea6b304d",
        "version" : "7.9.0"
      }
    },
    {
      "identity" : "leveldb",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/firebase/leveldb.git",
      "state" : {
        "revision" : "0706abcc6b0bd9cedfbb015ba840e4a780b5159b",
        "version" : "1.22.2"
      }
    },
    {
      "identity" : "libwebp-xcode",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/SDWebImage/libwebp-Xcode.git",
      "state" : {
        "revision" : "4f52fc9b29600a03de6e05af16df0d694cb44301",
        "version" : "1.2.4"
      }
    },
    {
      "identity" : "lottie-ios",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/airbnb/lottie-ios.git",
      "state" : {
        "revision" : "4a6058cbbdfe4f74aeae92c8bd51ad3b0de2a1ee",
        "version" : "3.3.0"
      }
    },
    {
      "identity" : "nanopb",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/firebase/nanopb.git",
      "state" : {
        "revision" : "819d0a2173aff699fb8c364b6fb906f7cdb1a692",
        "version" : "2.30909.0"
      }
    },
    {
      "identity" : "panmodal",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/slackhq/PanModal.git",
      "state" : {
        "revision" : "b012aecb6b67a8e46369227f893c12544846613f",
        "version" : "1.2.7"
      }
    },
    {
      "identity" : "promises",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/google/promises.git",
      "state" : {
        "revision" : "3e4e743631e86c8c70dbc6efdc7beaa6e90fd3bb",
        "version" : "2.1.1"
      }
    },
    {
      "identity" : "sdwebimage",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/SDWebImage/SDWebImage.git",
      "state" : {
        "revision" : "3312bf5e67b52fbce7c3caf431b0cda721a9f7bb",
        "version" : "5.14.2"
      }
    },
    {
      "identity" : "sdwebimagewebpcoder",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/SDWebImage/SDWebImageWebPCoder.git",
      "state" : {
        "revision" : "4416805045174938b11edb69a86e9f7dd0699952",
        "version" : "0.9.1"
      }
    },
    {
      "identity" : "skeletonview",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/Juanpe/SkeletonView.git",
      "state" : {
        "revision" : "739a4f5d78731bebc48811c75fa9e1f4c4cef23e",
        "version" : "1.30.4"
      }
    },
    {
      "identity" : "swift-protobuf",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/apple/swift-protobuf.git",
      "state" : {
        "revision" : "ab3a58b7209a17d781c0d1dbb3e1ff3da306bae8",
        "version" : "1.20.3"
      }
    },
    {
      "identity" : "swiftygif",
      "kind" : "remoteSourceControl",
      "location" : "https://github.com/kirualex/SwiftyGif.git",
      "state" : {
        "revision" : "db0c122b671bc9760385e0355be00eede3b7bb44",
        "version" : "5.4.3"
      }
    }
  ],
  "version" : 2
}

If using CocoaPods, the project's Podfile.lock

Expand Podfile.lock snippet

Replace this line with the contents of your Podfile.lock!

Hi @nik6018, could you share a sample app that reproduces the issue? Thanks.

Hi @rizafran I am unable to reproduce this issue every time and it's occurring in very rare occasions

@nik6018, could you share your full stack trace of the issue? Based on the stack trace you provided, it looks like there are missing UUIDs and the crashes aren't symbolicated.


2.3.0 release

Hi, is there any reason there is no 2.3.0 release on Github, only on npm? Is 2.3.0 considered stable?

Great catch @hyperknot - sorry we didn't see this sooner. v2.3.0 is stable and we just forgot to push the tag to github. Pushed it now and it should be fine.


deltaTime parameter in Body#update does not work as expected.

I'm doing this:
const vector = Vector.create(400, 0);
Body.setVelocity(body, vector);
const deltaTime = 500;
Body.update(body, deltaTime, 1, 1); // body, deltaTime, scaleTime, correction;

I'm assuming that passing 500ms in the deltaTime my body will get 200px moved to the right, but it does not matter if i use 1 billion ms or -1ms, it keep moving 400px exactly, so i'm interpreting that parameter badly?
Body.update(body, 16.6, 1, 1);

Body.update(body, 500, 1, 1);

Up to and including the release 0.18.0 velocity and speed units are (effectively) pixels per deltaTime ms which I think is what you're seeing, for these versions: normalise the result if not using a fixed deltaTime.
Note: this is due to become built in among other changes as part of #777 so please track that and keep an eye on release notes.

Sorry i don't get it 😅
If the movement is effectvely why the deltaTime literally do anything:
// If i'm passing a higher number in the `deltaTime` parameter.
console.log(Number.MAX_SAFE_INTEGER); // =><PHONE_NUMBER>740991

// WHY this is moving the body how it's running at `1000 / 60`?
Body.update(circle, Number.MAX_SAFE_INTEGER, 1.0, 1.0);

So i don't understand why MatterJS is fixing internally the deltaTime or whatever is happening with the Verlet integration.
Maybe i need to use Engine.update but i just want to move a body that was in the past to the present, so i need to simulate it for a higher deltaTime and i don't want to call multiple times the update function in 1 tick, can you help me with a simple example of this if it's not hard please 🙏🏻.

I'm not quite sure but worth trying out the latest release 0.19.0 as that includes #777, which has some changes related to this topic that might help.


Servo class - is speed adjustable?

I'm having a hard time as I am not a real coder, just a hacker.... Can I specify the speed that the servo moves to the target, in the motorised blinds example?  When moving the slider, the servo moves very quickly to the new position i.e. as fast as it can!
It seems the bit I need is "update actual servo position"......
if(hTiltTarget->updated()){                         // check to see if shade tilt angle was updated
      hTiltCurrent->setVal(hTiltTarget->getNewVal());   // set current value of tilt to match target value
      hTiltServo->set(hTiltTarget->getNewVal());        // <--- update actual servo position with ServoPin->set(degrees) method
    }

but in PWMPin.h there's only
class ServoPin : public LedC {
  uint16_t minMicros;
  uint16_t maxMicros;
  double minDegrees;
  double microsPerDegree;

I am new to OO programming so following along as best I can!

The HomeSpan Servo class does not provide a method to set the speed of change (though it's a good idea and might be something I can add in the future).  However, you should be able to accomplish this by having your code only change the hTiltCurrent in small steps to approach hTiltTarget, rather than set it to match hTiltTarget all at once.
The easiest way to do this would be to manage the process from within the loop() method for the Service instead of within the update() method.  Within the loop() method you can compare the current and target states, and if different increment the current state a little towards the target state.  Since the loop() method gets called every cycle, this will very quickly cause the current state to match the target state.  To slow things down you can use the timeVal() method for the current state and check to see whether a few seconds have elapsed since the last change.  If not, don't update current state.  If so, update it to approach target state.  By judicious choice of the increment by which you update current state, in combination with whatever timeVal() period you choose, you should be able to fine tune the speed of the Servo update.

However, you should be able to accomplish this by having your code only change the hTiltCurrent in small steps to approach hTiltTarget, rather than set it to match hTiltTarget all at once.

Thanks for the timely reply, I have been trying this today actually - in the update() method.
It works.....with jerkiness and other problems that i suspect are because things are changing value while i am using delay() inside the update() method.  In other words, just what you have pre-emptively said :-)
Will move it to the main loop and use the presumbaly non blocking timeVal() route, thanks!


Flow types

🙏

What "Flow Types" means ?

flow type definitions. similar to typescript type definitions
https://flow.org/en/docs/types/

If this done the IDE, like Visual Studio Code, will automatically hint function signatures, etc.
For example:
import type { PusherUser } from '@pusher/chatkit';

Closing as stale. This is in our internal backlog but is unlikely to be scheduled any time soon. Contact support if this is a feature that matters a lot to you!


collections should be defined at the project root and not relative to playbooks path

collections was being referenced from the playbooks relative directory and should be from the projects root directory.  updated roles for consistency on path joining.  thanks @andreas-marschke for the original find, updating the current codebase as there has been so many changes.
if a playbook is in <proj_root>/sub_folder/playbook.yml the previous code expects collections to be in the sub_folder directory <proj_root>/sub_folder/collections/requirements.yml.
my proposed changes sets the collections directory to always be in the projects root folder as it does for roles directory currently.
Now it will look for collections in <proj_root>/collections/requirements.yml as expected.
this resolves issue https://github.com/ansible-semaphore/semaphore/issues/1144 for No collections requirements file found

Hi @RusticPotatoes sorry, I missed your PR.
I merged this PM which do same that yours: https://github.com/semaphoreui/semaphore/pull/2001


Symlink makes it impossible to import package

Go doesn't follow symlink in import parts. Because of this symlink it is now impossible to import this package:
% cat main.go
package main

import "github.com/altinity/clickhouse-operator/pkg/apis/metrics"

func main() {
	_ = metrics.NewExporter
}

% go mod tidy
go: finding module for package github.com/altinity/clickhouse-operator/pkg/apis/metrics
go: found github.com/altinity/clickhouse-operator/pkg/apis/metrics in github.com/altinity/clickhouse-operator v0.0.0-20220307063955-cecb54ad1ade
go: finding module for package github.com/altinity/clickhouse-operator/pkg/client/clientset/versioned
example imports
	github.com/altinity/clickhouse-operator/pkg/apis/metrics imports
	github.com/altinity/clickhouse-operator/pkg/client/clientset/versioned: module github.com/altinity/clickhouse-operator@latest found (v0.0.0-20220307063955-cecb54ad1ade), but does not contain package github.com/altinity/clickhouse-operator/pkg/client/clientset/versioned

I suggest that all import statements get changed to the new path. I don't see any reason to keep the old imports and have this symlink in place.

The clickhouse prometheus endpoint doesn't expose enough to make the Altinity alerts config work. What is the harm in making it possible to import code from this package? We have successfully done this for the last year. The only thing that needs to change now is this symlink.

What is the harm in making it possible to import code from this package
We could change it in future and not have resource to maintain it as library with backwards compatibility

You don't have to guarantee backwards compatibility.  But being able to import in the past and not being able to import now is annoying.

code generator reworked
please check 0.23.3 version
hope it helps


PP-7422: Handle bad result returned by connector

This change updates post-flex controller to save valid Worldpay 3DS Flex credentials only and only if the result is valid. This change is necessary because the result can contain the value other than "valid" and "invalid". It adds 1 more test to Cypress to test the following scenario.

Connector returns result = bad data after checking Worldpay 3DS Flex credentials.

Please note that Self Service application has no way to check that response's body is null or empty before processing it. If the response contains no body or other data,  post-flex controller will attempt to retrieve result from the response's body and this will cause Self Service application to display a generic problem page.

LGTM. Can we have another test where connector doesn't return a 200?

It is already covered by the previous PR (https://github.com/alphagov/pay-selfservice/pull/2428). It tests should display generic problem page when checking 3DS Flex credentials fails. checking 3DS Flex credentials fails is where the connector will return 500.

LGTM. Can we have another test where connector doesn't return a 200?

It is already covered by the previous PR (https://github.com/alphagov/pay-selfservice/pull/2428). It tests should display generic problem page when checking 3DS Flex credentials fails. checking 3DS Flex credentials fails is where the connector will return 500.


Create Blog Post “2020-05-iterating-on-code-mil”

Automatically generated by Netlify CMS

Deploy preview for dds-mil ready!
Built with commit c1a94d112f26c8718690ce84b5cedcc6f59abc13
https://deploy-preview-164--dds-mil.netlify.app


Allow for fetching of user information by userName

In order to list user profiles in the fabric8-ui context menu, the UI (client) needs to be locate and load users by their name.

Duplicate of #970  ?

Yeah. Closing.

This was more of a story to track one of the required features in the Entity(User) area. Let me find a better way to track it.

@VineetReynolds if you have a story then you can link the existing #970 to it. If you don't but want to have one then you can reopen this issue and convert it to a story with a link to #970.

@alexeykazakov Yes, reopening and linking.

I'll working on re-wording the description to reference the tasks better.

fixed in #1196


Crash with BCLib when running on Server

Describe the bug
Crash with BCLib when running on Server(Forge 47.1.3),But BCLib works well on client(Forge 47.1.3)
Steps to reproduce
1.Install a brand-new Forge Server
2.Add Connector(beta10),Forgified FabricAPI(0.87.0+1.9.6+1.20.1) and BCLib
3.Start the server
4.Wait for crash
Logs
https://pastebin.com/af53F6GT
Additional context
No response

Interesting. I haven’t tested in a server environment but why not use Essential mod?

How is it related?

My suspicion here is @Environment methods are not being stripped properly on Forge. We'll have to remap them to use Forge's @OnlyIn annotation instead.

Interesting. I haven’t tested in a server environment but why not use Essential mod?

Because I want to use connector in my server😭


moment-from-now does not return unit as per locale

Thanks for the awesome addon.
I've been struggling to get this to return the unit, years in this case, in the locale I've set on the helper:
{{moment-from-now "Sat Jul 01 1978 21:18:08 GMT-0400 (EDT)" locale='ar' hideSuffix=true}}
the figure returned looks good but the unit doesn't get changed to what it's supposed to be:
39 years
I've already set includeLocales: true in config.environment, is there something I missed?
Appreciate any hint, thanks.

There isn't an issue and I validated that with a test:
https://github.com/stefanpenner/ember-moment/commit/49e7fa4ec1c914733006aff7df4c8625725a628b
The only thing I can guess is you have another addon which is also requiring momentjs and that is taking presedence over the one ember-moment (ember-cli-moment-shim) brings in.
yarn why moment to see which other addons might be requiring moment as a dependency (not a devdep)

@jasonmit nice one, I didn't have yarn (maybe I should!) but I trimmed my package.json removing "ember-welcome-page": "^2.0.2" and that did the trick, thanks!


Add ability to change verbosity of io.strimzi logging inside StrimziKafkaContainer

Currently we have method
public StrimziKafkaContainer withKafkaLog(Level level) {
        String log4jConfig = "log4j.rootLogger=" + level.name() + ", stdout\n" +
            "log4j.appender.stdout=org.apache.log4j.ConsoleAppender\n" +
            "log4j.appender.stdout.layout=org.apache.log4j.PatternLayout\n" +
            "log4j.appender.stdout.layout.ConversionPattern=[%d] %p %m (%c)%n\n";

        // Copy the custom log4j.properties into the container
        this.withCopyToContainer(
            Transferable.of(log4jConfig.getBytes(StandardCharsets.UTF_8)),
            "/opt/kafka/config/log4j.properties"
        );

        return self();
    }

but it would be great to also modify io.strimzi logger verbosity. Motivation comes from here [1]
[1] - https://github.com/strimzi/test-container/pull/95#pullrequestreview-2415237708

Triaged on 14.11.2024: This makes sense and should be implemented.


(HTTP code 500) server error - received unexpected HTTP status: 504 Gateway Time-out

Expected behavior
Actual behavior
Information about the Issue
(HTTP code 500) server error - received unexpected HTTP status: 504 Gateway Time-out
Steps to reproduce the behavior

...
...

I am getting the same issue when trying to create a docker image


vnso-nsdp-platform three fail pull requests that flags some errors that needs assistance in identifying the issue

Hi NSDP platform team & Brock
we're trying to upload and update some of our existing menu item pages for standalone indicators and adding some csv files, but we keep getting failed pull requests, three of the pull request are still there, two of the fail pull request have a common error statement which was screenshot and attached with this issue.

here are the links to the pull requests:
https://github.com/Vanuatu-National-Statistics-Office/vnso-nsdp-platform/pull/462
https://github.com/Vanuatu-National-Statistics-Office/vnso-nsdp-platform/pull/463
https://github.com/Vanuatu-National-Statistics-Office/vnso-nsdp-platform/pull/464

@jah-ky @hugo-e-pigott I've put up a pull-request to upgrade to 2.2.0-beta1, which just got released. This seems to be helping - the tests are passing. I think you could merge that PR, and then you could merge develop into jah-ky-patch-10, jah-ky-patch-11, and jah-ky-patch-11-1. Let me know if there are any issues with the upgrade.
FYI the full release 2.2.0 should be released in the next few weeks.

@brockfanning done a few pull requests today with jah-ky patch -10 and few others but still fail, should we wait until full release that is in next few weeks?

@jah-ky I took a look at a few of the recent failures, and there may be explanations to why they fail. Here are some comments for each one:
https://github.com/Vanuatu-National-Statistics-Office/vnso-nsdp-platform/pull/462
This PR is adding a menu item for /government/. But if I go to https://vanuatu-national-statistics-office.github.io/vnso-nsdp-platform/government/ then I get a page-not-found. If I remove the /, and go to https://vanuatu-national-statistics-office.github.io/vnso-nsdp-platform/government instead, I get a page. So the fix there might be to use /government instead of /government/.
https://github.com/Vanuatu-National-Statistics-Office/vnso-nsdp-platform/pull/463
This PR is adding a new data file. Open SDG assumes that this is a new indicator for the NSDP framework. And so it is looking for a goal image called "government". In order to avoid this problem, for any "standalone" indicators, you should upload the indicator configuration first, and make sure to set standalone to true. That is my guess - let me know if that doesn't work or doesn't make sense.
https://github.com/Vanuatu-National-Statistics-Office/vnso-nsdp-platform/pull/464
This PR is adding a data file called "jackson.csv" and an indicator configuration file called "government.yml.yml". I believe the "jackson.csv" is going to cause the same problem as in #463 (described above). I also believe that the double .yml.yml extension may be what is causing the problem here.
https://github.com/Vanuatu-National-Statistics-Office/vnso-nsdp-platform/pull/466
This one, again, is adding a standalone data file but there is no indicator configuration to tell Open SDG that the indicator is "standalone". So this is the same problem as #463 described above.


Update sdk doc

Description
Update the SDK docs and small refactoring.
Motivation and Context
The docs were misaligned with the latest changes.
How Has This Been Tested?

[x] Covered by existing integration testing
[ ] Added integration testing to cover
[x] Tested locally
[ ] Have not tested

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.

/rebase


Gmail tabs not being modified

Tab Modifier version
Browser: Chrome or Opera
Chrome version 75.0.3770.100
Extension version: X.Y.Z
0.22.0
_Can be found here: https://chrome.google.com/webstore/detail/tab-modifier/hcbgadmbdkiilgpifjgcakjehmafcjai?hl=en
Expected behavior
Setting a rule with "contains google.com" or "contains mail.google.com" should trigger the rule when gmail is opened (with the URL mail.google.com).
Actual behavior
No rules are triggered with gmail, although on google.com or google searches the rules trigger normally.
Steps to reproduce the behavior

Create new rule with condition "contains google.com"
Open mail.google.com
Rule isn't triggered

Hi @quincy2112
I just tried and it works. Can you export your tab rules and paste the JSON here please?
Best

{
"settings": {
"enable_new_version_notification": false
},
"rules": [
{
"name": "Google",
"detection": "CONTAINS",
"url_fragment": "google.com",
"tab": {
"icon": null,
"muted": false,
"pinned": true,
"protected": false,
"title": null,
"title_matcher": null,
"unique": false,
"url_matcher": null
}
}
]
}

Tested with drive.google.com and it didn't work either, if that helps.

This rule should pin the tab. Is the tab already pinned?
Disable Pinned mode and set any title and refresh the tab.

It's not pinned already. But I did as you said and changed the title, and I'm still getting the same problem. Google.com has its title changing to test, and nothing is happening to https://mail.google.com/mail/u/0/#inbox
{
"settings": {
"enable_new_version_notification": false
},
"rules": [
{
"name": "Google",
"detection": "CONTAINS",
"url_fragment": "google.com",
"tab": {
"icon": null,
"muted": false,
"pinned": false,
"protected": false,
"title": "Test",
"title_matcher": null,
"unique": false,
"url_matcher": null
}
}
]
}

I'm doing the same and it's OK. Can you open the Developer Console and check if any error is raised?

Yes, seems like you're onto something there.
Unchecked runtime.lastError: Could not establish connection. Receiving end does not exist.

Another extension on your Chrome may interfer with Tab Modifier, can you disable all extensions and let TM enabled and try again?
See: https://stackoverflow.com/questions/54619817/how-to-fix-unchecked-runtime-lasterror-could-not-establish-connection-receivi

That didn't help, unfortunately.

Weird!
If you disable TM and refresh the tab with the console, the error message remains or disapears?

It remains.
There are other error messages besides that one, but that's the only one that appears in both mail.google.com and drive.google.com, so it's most likely the culprit here.

I have the same error on gmail without the TM extension. It's raised by Hangout chat and not related to your issue.
Can you try a rule on another website? Like here github.com?

It works correctly.
Other google pages, like google.com or google translate, work fine. They don't have the Hangout chat error either.

Ok great, not sure if this will work for you in the future in mail.google.com...
Thanks for your feedback. If you consider the issue as closed, I let you close it!
Best

That's disappointing, it was gmail I wanted to use the extension for. Do you think the error message could be at all related, or is it a coincidence?
Thanks for the help.

I have the same message and TM is working fine on my computing for gmail. So hard to understand where/why this is not working for you.
The previous issue is on the same topic: https://github.com/sylouuu/chrome-tab-modifier/issues/190
I can't reproduce the "error" on Windows or Mac. :(

Okay, well thanks for the help.


docs: Add documentation for updating multiple parameters at once

Adds guidance in the README on how to handle updating multiple parameters at once.

Fixes #105

Summary by CodeRabbit

Documentation

Enhanced the README.md with clarifications on query parameters usage in SvelteKit, including a new section on updating multiple parameters.
Corrected a typo in the warning section for improved accuracy.

Style

Made formatting adjustments in multiple files, including consistent indentation and line breaks for improved readability.
Added newlines at the end of files where necessary to adhere to coding standards.

Yeah sorry I just installed and ran format at the end.

I will just close this and reopen it to git rid of the extra changes since its easier


Move fake::Kernel's data into a new type, KernelData.

Prior to this change, fake::Kernel had two conflicting responsibilities:

It is the public interface to the fake kernel, used by unit tests.
It stores the fake kernel's data.

This initially appeared to be the correct design, but it interacts poorly with system calls. In particular, system calls are highly reentrant (a Yield call can invoke an upcall that can call other system calls), and can call other methods that use the fake kernel (e.g. to queue an upcall). This reentrancy can result in many Rc<fake::Kernel>s existing in separate stack frames, which gets confusing.
To make writing the reentrant code easier, this change pulls the data out into KernelData. KernelData is stored in a thread local variable that replaces the thread local variable that previously held a Weak<fake::Kernel>. KernelData is data-only by design (i.e. it lacks behavior), which makes it easier to write code that is reentrancy-safe.
Note that this change drops some of the fake::Kernel leak checking (namely, the check that there isn't a fake::Kernel present when thread-local storage is deallocated). That check is less necessary now that we no longer use Rc to own fake::Kernel, and is also quite a bit more verbose to implement.

bors r+


Remove ;

hi
i suggest to make dart (flutter) code more simple, so why not remove ; like kotlin language

Hey @halimpuckjava since this is a change in dart, please open an issue here - https://github.com/dart-lang/sdk

Closing with the above comment, this is a language proposal not specific to Flutter


path/filepath: try GetFinalPathNameByHandle with VOLUME_NAME_GUID

EvalSymLinks calls into normaliseLinkPath, which will now also use GetFinalPathNameByHandle with VOLUME_NAME_GUID to resolve reparse points to paths.
Fixes #39786

@googlebot I signed it!

@googlebot It was fixed by design!

Superseded.


knife bootstrap against OSX 10.10.5 node giving error at first chef-client run (post chef omnibus install)

Recent attempt to knife bootstrap a OSX node (10.10.5) giving me an error when bash script is trying to execute "chef-client" for the first time, after omnibus installer successfully installs chef-client.  It installed latest 12.4.3-1.
Changing chef-client to 12.3.0 fixed the issue, so something's not right with the latest 12.4 release at least not for OSX:

Connecting to unity_mac_node.foo.bar
Password:
unity_mac_node.foo.bar Installing Chef Client...
unity_mac_node.foo.bar   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
unity_mac_node.foo.bar                                  Dload  Upload   Total   Spent    Left  Speed
100 18736  100 18736    0     0  20058      0 --:--:-- --:--:-- --:--:-- 20059
unity_mac_node.foo.bar Downloading Chef 12 for mac_os_x...
unity_mac_node.foo.bar downloading https://www.opscode.com/chef/metadata?v=12&prerelease=false&nightlies=false&p=mac_os_x&pv=10.10&m=x86_64
unity_mac_node.foo.bar   to file /tmp/install.sh.883/metadata.txt
unity_mac_node.foo.bar trying curl...
unity_mac_node.foo.bar url  https://opscode-omnibus-packages.s3.amazonaws.com/mac_os_x/10.10/x86_64/chef-12.4.3-1.dmg
unity_mac_node.foo.bar md5  e0caf8a0bd8b4140191fdfe7946da27c
unity_mac_node.foo.bar sha256   32d290cb5648ea600d976717fa32fc1e213e4452f10dc7b481f4e9aa7200293c
unity_mac_node.foo.bar downloaded metadata file looks valid...
unity_mac_node.foo.bar downloading https://opscode-omnibus-packages.s3.amazonaws.com/mac_os_x/10.10/x86_64/chef-12.4.3-1.dmg
unity_mac_node.foo.bar   to file /tmp/install.sh.883/chef-12.4.3-1.dmg
unity_mac_node.foo.bar trying curl...
unity_mac_node.foo.bar Comparing checksum with shasum...
unity_mac_node.foo.bar Installing Chef 12
unity_mac_node.foo.bar installing dmg file...
unity_mac_node.foo.bar Checksumming Protective Master Boot Record (MBR : 0)…
unity_mac_node.foo.bar Protective Master Boot Record (MBR :: verified   CRC32 $A63E199D
unity_mac_node.foo.bar Checksumming GPT Header (Primary GPT Header : 1)…
unity_mac_node.foo.bar  GPT Header (Primary GPT Header : 1): verified   CRC32 $1D64EDE0
unity_mac_node.foo.bar Checksumming GPT Partition Data (Primary GPT Table : 2)…
unity_mac_node.foo.bar GPT Partition Data (Primary GPT Tabl: verified   CRC32 $443CB9AF
unity_mac_node.foo.bar Checksumming  (Apple_Free : 3)…
unity_mac_node.foo.bar                     (Apple_Free : 3): verified   CRC32 $00000000
unity_mac_node.foo.bar Checksumming disk image (Apple_HFS : 4)…
unity_mac_node.foo.bar ..............................................................................
unity_mac_node.foo.bar           disk image (Apple_HFS : 4): verified   CRC32 $85A4C8ED
unity_mac_node.foo.bar Checksumming  (Apple_Free : 5)…
unity_mac_node.foo.bar                     (Apple_Free : 5): verified   CRC32 $00000000
unity_mac_node.foo.bar Checksumming GPT Partition Data (Backup GPT Table : 6)…
unity_mac_node.foo.bar GPT Partition Data (Backup GPT Table: verified   CRC32 $443CB9AF
unity_mac_node.foo.bar Checksumming GPT Header (Backup GPT Header : 7)…
unity_mac_node.foo.bar   GPT Header (Backup GPT Header : 7): verified   CRC32 $82A9B24E
unity_mac_node.foo.bar verified   CRC32 $38C25161
unity_mac_node.foo.bar /dev/disk3           GUID_partition_scheme
unity_mac_node.foo.bar /dev/disk3s1         Apple_HFS                       /Volumes/chef_software
unity_mac_node.foo.bar installer: Package name is Chef Client
unity_mac_node.foo.bar installer: Upgrading at base path /
unity_mac_node.foo.bar installer: The upgrade was successful.
unity_mac_node.foo.bar "disk3" unmounted.
unity_mac_node.foo.bar "disk3" ejected.
unity_mac_node.foo.bar Starting first Chef Client run...
unity_mac_node.foo.bar bash: line 100: chef-client: command not found

Having absolutely the same problem:

Mac OS X version: 10.10.5
Chef-client version: 12.4.3

Picking "12.3.0" version for a bootstrap works just fine. Later upgrade to 12.4.3 also works good, the problem happens only on boostrap

Because of 10.11.x not having a writable /usr we now setup symlinks in /usr/local/bin/chef-client.  On package install /usr/local/bin is created if it does not already exist.  Most likely the PATH is not being setup correctly (and even if the installer tried to setup the PATH it can't modify its parent's settings).
The upgrade probably works because the symlinks in /usr/bin are setup by the 12.3.0 install and then not removed correctly (possibly another bug there, but mostly a cosmetic one).

chef-full itself should /probably/ enforce that /usr/local/bin and /usr/bin MUST be on its own PATH -- proabably just by appending those two to its PATH.

We did a great big pile of work to make sure this works. If that's not the case with modern versions of chef please open a new ticket.


docs/hellonode.md Is missing 'gcloud container clusters get-credentials'

Currently it has:

It’s now time to deploy your own containerized application to the Kubernetes cluster! Please ensure that you have configured kubectl to use the cluster you just created.

With just the link out, which is easy to miss

@pjfitzgibbons

It is on line 139, a couple sentences above the "Create your Pod" heading.

@jeffmendoza this seems fixed. Can we close the ticket ?

Looks good!

@jeffmendoza Yep, good now.  Thanks!


Enhancements to Monte-Carlo g-formula

As noted in #73 and #77 there are some further optional enhancements I can add to MonteCarloGFormula
Items to add:

[x] Censoring model

[ ] Competing risk model

Testing:

[x] Test censoring model works as intended (compare to Keil 2014)

[ ] Test competing risks. May be easiest to simulate up a quick data set to compare. Don't have anything on hand

The updates to Monte-Carlo g-formula will be added to a future update (haven't decided which version they will make it into)
Optional:

[x] Reduce memory burden of unneeded replicants

I sometimes run into a MemoryError when replicating Keil et al 2014 with many resamples. A potential way out of this is to "throw away" the observations that are not the final observation for that individual. Can add option low_memory=True to throw out those unnecessary observations. User could return the full simulated dataframe with False.

When I release this version, it also removed TimeVaryGFormula as mentioned in #70 whether this is part of v0.6.0 or earlier (likely going to be a 0.5.x release)

Before adding the censoring model, I am going to see if I can speed the g-formula up a bit. I am going to manually run patsy myself to pull the matrix then multiply with numpy directly.
As the benchmark, the g-formula in d4a3f2c takes about 130 seconds in the test set-up

Interacting with the matrix via patsy and the estimated betas, doesn't save me any time really. Still about 130-140 seconds. The item to target for speed improvement in this approach would be np.dot

There are some difficulting getting the competing risk g-formula to work (np.random.multinomial doesn't like an array of probabilities). I also don't have a good comparison data set currently. I built some of the code structure for competing risks. I will add this into a later release (once I resolve some of these issues)


In Jupyter mode have the link target set to "_blank"

Type of changes

[x] Bug fix
[x] New feature
[ ] Documentation / docstrings
[ ] Tests
[ ] Other

Checklist

[x] I've run the latest black with default args on new code.
[ ] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
[ ] I've added tests for new code.
[x] I accept that @willmcgugan may be pedantic in the code review.

Description
Closes https://github.com/Textualize/rich/issues/1905
I've yet not updated CHANGELOG.md since I do not know which version of rich with this pr ship with, nor when the version will be released. I have not updated CONTRIBUTORS.md since I do not think this is a significant contribution. If you want I can do those.

I've also noticed a similar thing for exporting HTML.
https://github.com/Texualize/rich/blob/6216ee0fccb6c40fc363ae69ea353c2e77c487fb/rich/console.py#L2091

Codecov Report

Merging #1945 (6216ee0) into master (e839bfb) will decrease coverage by 0.01%.
The diff coverage is 99.02%.

 Click here to find out more.

Impacted Files
Coverage Δ

rich/default_styles.py
100.00% <ø> (ø)

rich/markdown.py
100.00% <ø> (ø)

rich/syntax.py
99.27% <97.72%> (-0.34%)
:arrow_down:

rich/pretty.py
99.71% <98.88%> (-0.29%)
:arrow_down:

rich/__main__.py
100.00% <100.00%> (ø)

rich/_inspect.py
100.00% <100.00%> (ø)

rich/color.py
100.00% <100.00%> (ø)

rich/console.py
100.00% <100.00%> (ø)

rich/measure.py
100.00% <100.00%> (ø)

rich/progress.py
98.35% <100.00%> (+0.23%)
:arrow_up:

... and 3 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 ae5865e...6216ee0. Read the comment docs.

You can start a new Unreleased section in the CHANGELOG if it's not there already.
You should add yourself to CONTRIBUTORS. The size of your contribution doesn't matter.

Okay I'll do that

@willmcgugan done!

Thanks


Make environment builds more robust w.r.t. .orchest being git ignored

Describe the problem this improvement solves
Orchest expects the .orchest/environments directory to be versioned (or, at least, to not be .gitignore'd). This is because an environment build will take a snapshot of the project, and said snapshot will exclude files and directories according to the .gitignore file. However, some users would like to not version any content of the .orchest directory.
Describe the solution you'd like
The build should work regardless of the .orchest directory being in the snapshot or not. Making the build read the environment properties and setup script prior to the snapshot should be feasible. Note: the PR fixing this should include some changes to docs/source/fundamentals/environments.md to remove the notion that .orchest/environments shouldn't be git ignored (i.e. revert https://github.com/orchest/orchest/commit/02c2fa4caaf2cccd770721265b812caa437e86c0.

hey @fruttasecca can i start working on it ?


4809 fix extra cr

[x] Tests added / passed
[x] Passes flake8 dask

Fixes #4809.
Includes the following new test, which demonstrates the issue, and did, indeed, fail prior to the fix here.
def test_to_csv_line_ending():
    df = pd.DataFrame({'x': [0]})
    ddf = dd.from_pandas(df, npartitions=1)
    expected = {b'0\r\n', b'0\n'} # either/or
    # For comparison...
    # unexpected = {b'0\r\r\n'}
    # This test addresses GH4809, and checks that only (at most) one
    #  '\r' character is written per line when writing to csv.
    #  In case it's correct (on UNIX) to have no '\r' at all, this test
    #  considers either '\r\n' or '\n' as appropriate line endings,
    #  but not '\r\r\n'.
    with tmpdir() as dn:
        ddf.to_csv(os.path.join(dn, 'foo*.csv'), header=False, index=False)
        filename = os.path.join(dn, 'foo0.csv')
        with open(filename, 'rb') as f:
            raw = f.read()
    assert raw in expected

LGTM, merging. Thanks @btw08.


Add basic support for --pre

Description
This is a temporary solution for adding support for pre release based gems.  An example for this is the octopress gem:

Which currently only has pre-release gems, and can not be installed in any for currently.
Notes for the future
This is a hack, and I don't really like solving the problem in this way, but it is the simplest for the time being.  In reality, we really should move away from a single file installation and move towards a more structured library.  I have outlined what I think is required to accomplish this here:  https://github.com/sportngin/brew-gem/issues/16
QA
Not the easiest to QA, but here are two possible ways to get this code in place:

Modify your brew-gem bin file locally with the changes from this PR.  Should be located here:  /usr/local/Cellar/brew-gem/CURRENT_VERSION_NUMBER/bin/brew-gem.

Check out this branch, make a tar of the bin directory, and install the formula using the following (requires that brew-gem is not installed currently):

$ curl https://raw.githubusercontent.com/Homebrew/homebrew/master/Library/Formula/brew-gem.rb > brew-gem.rb
$ brew install brew-gem.rb

Once you have done one of the above, try installing octopress by doing the following:
$ ALLOW_PRE_RELEASE_GEMS=1 brew gem install octopress

I think this is an ok workaround for now. I agree that we should make this more intuitive going forward.


passthrough server does not implements messageSelfAndDeferRetirement correctly

See messageSelfAndDeferRetirement in PassthroughMessengerService

To flesh this out a bit more, the implementation of messageSelf includes some additional logic (of most obvious importance is PassthroughMessageCodec.createInvokeMessage) that is conspicuously absent from messageSelfAndDeferRetirement.
We will need to take some time to look at this and see if this missing call is the only problem in the logic.


feat: add forge-std submodule for testing

Motivation
the forge-std submodule was missing causing solc import errors:
Compiler run failed
error[6275]: packages/contracts-core/contracts/test/NomadBase.t.sol:7:1: ParserError: Source "forge-std/console.sol" not found: File outside of allowed directories.
import {console} from "forge-std/console.sol";

Solution
add forge-std as submodule via forge install foundry-rs/forge-std
Note* I had to unignore the lib folder (default dir for submodule imports), not sure if this causes side-effects with yarn/javascript builds
if so I can move this to another dir
FOUNDRY_PROFILE=core forge build  now works

PR Checklist

[ ] Added Tests
[ ] Updated Documentation
[ ] Updated CHANGELOG.md for the appropriate package

So, we had a looong discussion internally, in an effort not to use git submodules to manage dependencies. The problem we wanted to avoid was that random updates to the library would dirty the git tree.

that's reasonable
gotcha, closing this

we'll probably open a similar PR with the dependency vendored :)


Camera Source Preview Always Looks dark

Preview Generated using CameraSource always seems to look very dark when Front camera is used. This can be experienced with GooglyEyes Test app too.
Test Phone used: Pixel XL.

I also had the same issue with nexus 5 and I got it resolved by decreasing the frame rate. The default value is 30 fps. I changed it to 20 and worked fine for the device. You should consider changing frame rate.
mCameraSource = new CameraSource.Builder(context, detector)
                .setRequestedPreviewSize(width, height)
                .setFacing(camId)
                .setRequestedFps(15.0f)
                .setAutoFocusEnabled(shouldIUseAutofocus)
                .build();

@sudipbhandari126 How does frame rate relates to brightness of the preview? Is it a bad work from google guys, or something? I tried it and it works though.

I'm certainly no expert but I believe that maybe the FPS setting affects the shutter-speed and that would probably have an effect on the brightness/exposure


Make GITHUB_TOKEN Optional

It should be possible to use the app without supplying the GITHUB_TOKEN env var (in which case only public repos would be accessible).

solved


Twiss fails when importing a json file with Line.from_json under Windows 11

Dear all,
When I import a Line from a json file (diretcly generated by XSuite with xt.Line.to_json), I get an odd behaviour of XSuite.
Indeed, under Windows and Ubuntu, the import works well and I get the right feature of the line (the number of elements, total length and attributes of the elements seem all right when I use get_table).
However, when I use Twiss just behind, I get the right behaviour with Ubuntu but the method fails with Windows.
As an example, I put in attachment a simplified json file.
The test was done with the last release Xsuite 0.6.0.
Thank you for your answer and the great work you do.
lattice_disp_suppr.json
The code I have run is the following:
import xtrack as xt
# Loads the json file
lattice = xt.Line.from_json('lattice_disp_suppr.json')
lattice.build_tracker()

# Prints the attributes of the line; works with Linux/Ubuntu and Windows 11
lattice.get_table().show()

# Works only with Linux/Ubuntu.
# Fails with Windows 11 (elements are not corretly loaded with a length of 0  for all of them for instance)
tw = lattice.twiss(method="4d")
print(tw)

In fact we do not us support Windows. Our suggested way for running Xsuite in Windows is to do it within WSL.


[Bug?]: File change breaks Prod if no hard refresh

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
I have Page1 and Page2 if i change something on Page2 and then using Page1 to navigate to Page2 i get error
error loading dynamically imported module: https://****/assets/about-52919eb7.js

I guess its due to the hash of the file name change but my current client session still working with V1 until i do hard refresh.
Is there any plan to work around this?
Expected behavior 🤔
Page2 should load without issue even if changes have been made and pushed to prod

Ive tested with Next.js it looks like its doing hard page refresh before continue fetching new components

Hmm.. a client side navigation has no way of knowing it is out of date, except from the response.. and we fetch JS in parallel. Unless they have setup some background pinging of versioning I don't see how Next could ever know this. My understanding was that Vercel keeps previous deploys around. Its also possible I suppose that that fetch data before JS and in that case they could know, of course that would be very inefficient.
In any case this is an idea and if anyone has ideas on how to implement it we can consider making a request.


Fix zayd prefetch roms

prefetch-abuse-idelafter.gba

prefetch-abuse-str.gba

prefetch-abuse-master.gba

Roms as used here are checked in but also in this zip
prefetch-abuse.zip
No idea what this is testing but it's likely related to the missing few timing tests in #5 and possible also to the other zayd dma timings #75

Tests here are from https://github.com/GhostRain0/PrefetchAbuse

Each column in the test refers to a different wait state configuration:

Column
WaitState
Region 0
Region 1
Region 2

1
0x4000
4,2
4,4
4,8

2
0x4010
4,1
4,4
4,8

3
0x4004
3,2
4,4
4,8

4
0x4014
3,1
4,4
4,8

Except in the prefetch-abuse-str case I only fail timings on colums 2 and 3, so wait states with 1 cycle for seq access to region 0. I then only fail those in 2 rows (4 & 7).
The rows here refer to the number of idle states before the test is executed (caused by mul spam). Row 4 is "IDLES_VERYLONG" which is mul r3, r8, r11 where r11 is 0xAAAAAAAA. Row 7 is mul r3, r8, r11 followed by mul r3, r8, r10.
These are just statements rather than any judgement on what's happening.
Also none of this is about why the str case fails so badly across the board. I think that might be something to do with wait states for str to ROM space whilst bus is in use by prefetch


Why all method have the NullPointerException problem?

When I run benchmark on my computer, every method can't run and return nullpointerexception like this. I'm really puzzled about this problem. What may cause this?

every check contains this problem...nullpointerexception

have you marked your implementation with @Service? did you succeed to run the program in the interactive cli mode?


Create MinWiz.com.md

Hi. I have added my website. I hope it's OK I didn't use all lowercase letters in the domain name... like this it makes it a bit easier to read/remember. If it's not OK with you I can remedy it immediately in a new commit.
Best regards,
Laurentiu

Nice site!
If you could make the site name all lowercase that would be helpful (to keep the listings consistent)
Also - multiple Pingdom tests show your website sitting at 3.2KB, so maybe update that as well 👍

Nice site!
If you could make the site name all lowercase that would be helpful (to keep the listings consistent)
Also - multiple Pingdom tests show your website sitting at 3.2KB, so maybe update that as well 👍

I have updated as requested.
Also, I don't know if before archiving the site you were also using pull requests, but as I see how popular your site is maybe it's a good idea to deactivate "build on pull requests". Otherwise you will run out of build minutes in no time :)
Regards,
Laurentiu

I have updated as requested.
Also, I don't know if before archiving the site you were also using pull requests, but as I see how popular your site is maybe it's a good idea to deactivate "build on pull requests". Otherwise you will run out of build minutes in no time :)
Regards,
Laurentiu

Thanks for the updates! Also thank you for the heads up about the build on PR - completely overlooked that...

Thanks for the updates! Also thank you for the heads up about the build on PR - completely overlooked that...


getData and setData

Codecov Report

Merging #70 into dev will decrease coverage by 0.11%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##               dev      #70      +/-   ##
===========================================
- Coverage     43.6%   43.49%   -0.12%
  Complexity     492      492
===========================================
  Files          101      101
  Lines         3325     3334       +9
  Branches       244      244
===========================================
  Hits          1450     1450
- Misses        1757     1766       +9
  Partials       118      118

Impacted Files
Coverage Δ
Complexity Δ

...main/java/com/webank/wecross/exception/Status.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...webank/wecross/stub/bcos/BCOSContractResource.java
16.21% <0%> (-1.44%)
7 <0> (ø)

.../wecross/stub/jdchain/JDChainContractResource.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

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 566f191...8c3b86e. Read the comment docs.


CLI commands puts SQLite files in the nuget folder containing dotnet-ef

A relative path in the SQLite connection string + dotnet-ef  = the root for the relative path is the folder containing dotnet-ef.dll. This is because Microsoft.Data.Sqlite relies on AppContext.BaseDirectory, which is Package.Current.InstalledLocation.Path.

To reproduce:
    public class TestContext : DbContext
    {
        protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
        {
            optionsBuilder.UseSqlite("Filename=./test.db");
        }
    }

dotnet ef migrations add First
dotnet ef database update

Makes sense; that's the application's base directory. We'll need to use some sort of override mechanism that M.D.Sqlite can use. Is there a .NET Core equivalent to AppDomain.SetData()?

Ironically, if we just let SQLite use the current directory, it would be correct. :smile:


Show all JSDoc @typedef properties when forming object of that type

Issue Type: Feature Request
Problem
When making references to custom types, you cannot really see what that custom type actually is actually composed of.
For example, I have a class called OnlineUser which takes an optional parameter of custom type OnlineUserOptions:

When hovering over a reference to the class or forming the object to pass as OnlineUserOptions, only the name of the type is shown in the documentation that is displayed.  If you don't know/remember what properties comprise OnlineUserOptions, then the only way to find out is by beginning to type random characters:

Suggestion
It would be very useful if instead, as soon as you begin the object literal, it would show all the properties of the custom type, like so:

It would also be nice if it was possible to click on the name of the type to jump to its definition (maybe through use of the @link tag?) or hover over it to view its details as though you were hovering over its name in the actual @typedef.
VS Code version: Code 1.50.0 (93c2f0fbf16c5a4b10e4d5f89737d9c2c25488a3, 2020-10-07T06:10:52.432Z)
OS version: Windows_NT x64 10.0.19041

Can you please share some example code (as text) that demonstrates this issue

Sure, here you go:
class OnlineUser {
  /**
   * @typedef {Object} OnlineUserOptions
   * @prop {boolean|true} connected
   * @prop {string|"default.png"} pfp
   * @prop {timeout=} timeout
   */

  /**
   * @param {number} userID
   * @param {string} username
   * @param {OnlineUserOptions} opts
   */

  constructor (userID, username, opts = {}) {
    this.userID = userID;
    this.username = username;
    this.connected = opts.connected || true;
    this.pfp = opts.pfp || 'default.png';
    this.timeout = opts.timeout;
  }
}

var user = new OnlineUser(1, "epiqu1n", {  });

If you delete the , {  } in the last line and then begin to retype it, VSCode shows the constructor preview with the type of opts being OnlineUserOptions but nothing more, and there is no way to jump to the type definition.

It'd be nice if this showed up automatically. FYI Ctrl-Space will bring up the completion list; no random characters required

Ah that is actually quite helpful thank you.  But yeah automatic would still be very nice.

Another common case where this would be useful is for callback arguments. The best way to properly describe callbacks with jsdoc is @callback, but that has the same problem as @typedef: Intellisense only shows the name. The workarounds I've found are as follows, and none of them are very satisfying:

Use the function syntax, e.g. @param {function(number, string): string}. Works in VSCode and JSDoc accepts the syntax, but you don't get any argument names or descriptions, and the generated documentation just shows function.
Use double braces, e.g. @param {{(arg1: number, arg2: string) => string}}. Works in VSCode, but JSDoc generates a syntax error. Also no descriptions.
Define a dummy function and describe it with its own JSDoc comment, then pass it as @param {DummyFunction}. Works in VSCode and JSDoc (VSCode prefers typeof DummyFunction but sets the type of the parameter anyway), but requires defining an extra function which isn't great (especially if you end up having to expose it).


standard_init_linux.go:207: exec user process caused "exec format error"

how can i fix this?
pi@raspberrypi:~/archivebox $ cat /proc/cpuinfo | grep Model
Model           : Raspberry Pi 4 Model B Rev 1.2
pi@raspberrypi:~/archivebox $ sudo docker run -v $PWD:/data -it archivebox/archivebox init --setup
standard_init_linux.go:207: exec user process caused "exec format error"
pi@raspberrypi:~/archivebox $

archivebox seems to be the project i really find useful but i cant manage to run it on my pi with docker

Use the :dev image instead of :latest:
sudo docker run -v $PWD:/data -it archivebox/archivebox:dev init --setup

Use the :dev image instead of :latest:
sudo docker run -v $PWD:/data -it archivebox/archivebox:dev init --setup

Thank you!


ImportError undefined symbol cblas_ddot

OS: Arch Linux 64bit
$ pkg-config --libs cblas
-L -lcblas -lblas
$ file /lib/libcblas.so
/lib/libcblas.so: ELF 64-bit LSB shared object, x86-64, version 1 (SYSV), dynamically linked, BuildID[sha1]=016679535567a6065cc9f5d0cdcbeb3f5830aa93, stripped
$ nm -D /lib/libcblas.so | grep ddot
0000000000008a60 T cblas_ddot

CMake's output:
-- Found PkgConfig: /usr/bin/pkg-config (found version 0.29.1)
-- Checking for module 'cblas'
--   Found cblas, version 3.6.1
-- Checking for module 'lapack'
--   Found lapack, version 3.6.1
-- Found BLAS and LAPACK via pkg-config

I see Dlib doesn't check for cblas_ddot if it finds BLAS using pkg-config https://github.com/davisking/dlib/blob/master/dlib/cmake_utils/cmake_find_blas.txt#L54. I added the lines checking for cblas_ddot before the return statement and CMake said it can't find cblas_ddot. According to this thread, I replaced CHECK_FUNCTION_EXISTS with CHECK_LIBRARY_EXISTS and CMake finally found cblas_ddot.
But I also wrote a simple C program to test cblas_ddot and it works fine. Which means even though Dlib does not check for cblas_dot, it should be able to find it at runtime. Right? I've checked link.txt and whether cblas_ddot is found or not, cblas and lapack are linked. So there probably is a problem when packing Dlib into a shared library.

Cool.  So it didn't link right for some configuration of arch linux and you fixed cmake so it did?

Nope. It still doesn't work.
$ ldd dlib.so
	libblas.so.3 => /usr/lib/libblas.so.3 (0x00007f8c8c7d5000)
	liblapack.so.3 => /usr/lib/liblapack.so.3 (0x00007f8c8bf9b000)
$ nm -D /usr/lib/libblas.so.3 | grep ddot
000000000000f5a0 T ddot_
$ nm -D /usr/lib/liblapack.so.3 | grep ddot
                 U ddot_
$ nm -D /usr/lib/libcblas.so.3 | grep ddot | xclip
0000000000008a60 T cblas_ddot
                 U ddot_
0000000000008c20 T ddotsub_

It's clear now the final shared library got linked to libblas.so and not libcblas.so. cblas_ddot is only available in libcblas.so. Maybe we could replace cblas_ddot with ddot_ which is available in all the three files. But why is libcblas.so not linked?

Uff..you're right. The output of pkg-config contains a -L with no argument and -lcblas follows it. Maybe gcc treated -lcblas as the argument of -L and that's why libcblas.so is not linked. Never encountered this before. Removing -L and it works now :)

Sweet

For fellow arch linux users, if anyone facing same issue as above and wondering what @mljli means, just replace ./dlib/cmake_utils/cmake_find_blas.txt line 48
set(blas_libraries "${BLAS_REFERENCE_LDFLAGS}")
with
set(blas_libraries "-lcblas;-lblas") ..
But make sure the output of
pkg-config --libs cblas
is
-L -lcblas -lblas

What exactly is the conclusion here? pkgconfig on arch linux is broken and shouldn't be used at all?  Maybe it should just be removed from the cmake script all together. I've never been a big fan of pkgconfig anyway.

I "solved" the issue for now by installing the python-dlib package from AUR and reinstalling my pipenv dependencies


Make resources/linux/bin/code.sh POSIX-compliant to remove bash dependency

I'd like resources/linux/bin/code.sh to be POSIX-compliant to remove the dependency on bash.

Why? Does this actually cause a problem?

I'm unable to start VS Code on my system which doesn't have bash. The linked PR allows me to start it because I have dash instead.

Verified by review.


Clear language data after opening file with a different extension

PR details
I'm surprised that no one wanted to fix it, although the only bug I'd say is that you would have open CTRL+L 2 times when opening a file with a different extension to actually get the comments.

I marked the PR as a draft because I thought it was bugged when I opened a markdown file and saw no comments (even though there wasn't any markdown extension so therefore there is no comments added)

I'm going to work on the auto complete persisting issue

Header
Header

Cell
Cell

Cell
Cell

@face-hh Can you review this PR?

I'm going to try to fix the bug with Bussin Reels as described in the PR description.

I'm going to try to fix the bug with Bussin Reels as described in the PR description.

Didn't do jack. 💀


email_from_field as an array?

I'm trying to configure elastalert to read an array from data which looks like:
"orgManagers": [
<EMAIL_ADDRESS><EMAIL_ADDRESS>], ...

And my alert configuration looks like:
email_from_field: "orgManagers"
Now this works as a single email address, but I need to send the alert to multiple recipients not known by the alert itself...those recipients come from the data. I've looked at enhancements but it looks like only the email field supports multiple. Ideas?

Ah, I only added support for a single address. I'll try add this feature in soon. Here's a quick patch you could try until I add this feature:
diff --git a/elastalert/alerts.py b/elastalert/alerts.py
index 1c6895e..b4e383d 100644
--- a/elastalert/alerts.py
+++ b/elastalert/alerts.py
@@ -391,6 +391,8 @@ class EmailAlerter(Alerter):
                     to_addr = [recipient]
                 elif 'email_add_domain' in self.rule:
                     to_addr = [recipient + self.rule['email_add_domain']]
+            else:
+                to_addr = recipient
         email_msg = MIMEText(body.encode('UTF-8'), _charset='UTF-8')
         email_msg['Subject'] = self.create_title(matches)

Thanks! Yeah it should always be valid address. :)

@Qmando It should work with email distribution list(DL) too, right?

@prayagupd Yup!

@Qmando Would a pull request be helpful for this?

@bobbyhubbard : Did you get this working ?

Apologies, havent had a moment to validate it. Hoping to do so soon

From: chettu<EMAIL_ADDRESS>Sent: Friday, August 11, 2017 9:26:19 AM
To: Yelp/elastalert
Cc: Hubbard, Bobby; Mention
Subject: Re: [Yelp/elastalert] email_from_field as an array? (#1064)
@bobbyhubbardhttps://github.com/bobbyhubbard : Did you get this working ?

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Yelp/elastalert/issues/1064#issuecomment-321827901, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AE1SSNveGc3hvHqkyghuIRQ-kYFLZ44uks5sXGSLgaJpZM4NL8f3.

CONFIDENTIALITY NOTICE: This email and any attachments are for the sole use of the intended recipient(s) and contain information that may be Garmin confidential and/or Garmin legally privileged. If you have received this email in error, please notify the sender by reply email and delete the message. Any disclosure, copying, distribution or use of this communication (including attachments) by someone other than the intended recipient is prohibited. Thank you.


Image Assets for Level Editor

I pulled in tilecodetextures in the other branch and just use the extracted assets for the bg files


Apps: support multiple dao+model per app

Refactor model+daoFactory into a DataConfig. Apps like the Event/Convention lobby display would need daos for events, presenters, locations, or other browseable things. Moving model and daoFactory into a separate model also makes it easy to make admin browsers, as the menu can select which dataConfig you want and then a single BrowserConfig targets a single DataConfig. Everything (except dao content) should still serialize inside a single AppConfig.
Apps that require only a single DataConfig will just have an array of one DataConfig.
AppConfig.dataConfigs = [];

Found some bugs in Admin app creation picker list, Questionnaire model picker

Refactor to allow export of the DAOs that the DataConfigs make available, so a DAOKeyView could use them. i.e. each DataConfig produces a DAO on the view context.

One more bug in DAO wizard pages found


feat: Add ASP.Net Core with Angular sample

Signed-off-by: Akash Nayak<EMAIL_ADDRESS>

Do we want to rename the package as dotnet5angular to be consistent with our other samples

Done. I have also updated the samples.


may be not need .vscode folder, need .editorconfig

.vscode/settings.json
{
    "editor.tabSize": 2
  , "typescript.tsdk": "./node_modules/typescript/lib"
}

the editor.tabSize can controlled by .editorconfig.
but now, the user setting and workspace setting polluted by it.
It seemed like a good idea that remove .vscode and add .editorconfig file

Yes, I agree with you that tabSize is not necessary at here, and should be removed.(I just did it)
However, vscode/settings.json have some feature that not supported by .editorconfig.
I.e: files.exclude to hide directory/files in the ide interface.

but the project does not use these features, such as files.exclude.
Modern editors support .editorconfig.
Wouldn't it make more sense?

Dust to dusts, dirt to dirts, ash to ashes

tabaSize to .editorconfig, files.exclude to .tsconfig
all not to .vscode/settings.json

No, I do not agree with you about this because of<EMAIL_ADDRESS>is totally different as<EMAIL_ADDRESS>;-)
See: How do I hide certain files from the sidebar in Visual Studio Code?

sorry,  i make a mistake on it.

@huhuanming No problem buddy, and thank you for the warm-hearted suggestion.
Please feel free to let me know if you have any other good ideas. :)

sweet, thx.


Some times in dependency, we can see : "Undefined"

Not present on "CustomField" tab. But reproduced on "VisualComponent" tab.
Should be just a problem of using the correct property when getting the name of the root node...

Copy/Paste issue.
For example, seen in "VisualComponent" VFP:
if (dep) return helper.html.render.dependencies(r.id, r.fieldName, dep);
the "r" variable does not have the "fieldName" property, in this case the code should use "name" instead.
Found the same issue in multiple VFP.


New PR View: Bad button layout

Duplicate of https://github.com/microsoft/vscode-pull-request-github/issues/5030


Collision that blocks movement

Entities need to be able to block the movement of other entities into their occupied space. My initial thinking is that this should be achieved by a two-stage process, where moving entities first announce their intent to move to a certain position, and a collision system then checks the destination, applies any shifts that may be required to avoid intersecting collision boxes, and moves the entities appropriately.

The appropriate solution for this is of course to use a physics engine, will integrate Rapier.


Flutter Pub uses 1.5.4-hotfix.2 instead of 1.6.2

Version 0.11.0 needs to use flutter version 1.6.2 however the pub website and analysis still uses 1.5.4-hotfix.2.
This is breaking version 0.11.0 and should not be used until resolved.

Turns out that Stable is only supported. From now on this package can only support Stable and not Beta with which I was targeting
https://stackoverflow.com/questions/56729997/flutter-package-targeting-min-version

Released v0.12.0


Pick by node label not working

I was helping Chad debug some of his GrizIt code, and we discovered that PickByNodeLabel is not working.
I've done some further testing since then, and it appears that the query always returns node 1. We actually have a test for this query in our test suite, but the baseline is also incorrect. This may or may not be related, but the "OrignalNodeLabels" does not appear in the labels menu, whereas "OriginalZoneLabels" does (zone label query seems to work).

but the "OrignalNodeLabels" does not appear in the labels menu,

You mean that menu item isn't appearing in the list of available variables for the Label plot menu?
There is code in current Mili plugin that looks for these with mc_load_conn_labels (for zones) and mc_load_node_labels (for nodes). If the node labels are not appearing, then doesn't it mean the input Mili database doesn't have 'em?

Yeah, that was my first thought as well, and that might be the underlying issue. Chad was under the impression that these labels should always be available in mili because they're just derived from the node ids, though, which could mean that they're either not being retrieved correctly or not being accessed/made available correctly. In instances where these labels aren't available (either in mili or other databases), we should probably raise some kind of error when a user calls PickByNodeLabel rather than just returning the first node.

Could it be the version of Mili the plugin is linked with? I mean, if the Mili lib isn't responding with node ids when asked, that could also cause the problem.

That's definitely possible. I'll start testing this on the new plugin branch and switch to the latest mili version.

As a workaround for GrizIt alpha, I'm going to convert the label pick to a node id pick in the GrizIt picker module. The node labels are really just the "node" followed by it's id.

@ChadCooper67 can you comment on this?

I will do some further testing tomorrow, but I believe this was resolved today during the label refactor.

Scratch that. It seemed to resolve the label being wrong, but the actual pick operator is still not performing correctly.

This is resolved now. The issues seemed to stem from two areas:

The node labels were not being carried through visit from the plugin.
The node id was being overwritten in avtPickByNodeQuery.

The fixes are on the task/maguire7/2019_01_mili_plugin_rewrite branch.


Suggestion: use default prettier settings

I noticed the Token Extension examples look odd and thought there wasn't any linter set up, and then I saw https://github.com/solana-labs/solana-program-library/pull/7115.
Suggestion: use two spaces per prettier defaults, StandardJS, node style guide, idiomatic JS, AirBnB style guide, MDN, Google Style guide, codepen, and jsfiddle.
Not my own personal preference, but if you want people to use your examples, it's better to fit in.

This is our Solana-wide Prettier config. I hate it too, but I didn't get here early enough to have a say.
See also https://github.com/solana-labs/eslint-config-solana.
In any case, your suggestions belong on the https://github.com/solana-labs/prettier-config-solana repo.


Harmonize import and reimport logic

This PR is something I wanted to do for over a year now. At first there were not enough unit / integration tests to do this big refactoring. After creating these tests and merging them, I got sidetracked on other PRs. Then there were some other big PRs going on, so I wanted to wait a bit to avoid too many conflicts. But now I need this PR because I am going to do some changes in a next PR around the import/reimport.
This PR:

moves all import and reimport code into one place, a new folder importers;
so no more 95% identical copy-and-pasted code between ui and api (re)import;
extracts some common logic between import and reimport into an utils module;
standardizes messages and notifications;
support close_old_findings setting on UI Import;

The test cases worked well, as I had to fix some small things found by them.

it's an awesome pull request! need!
Just one thing, is it possible to add unit tests for importer/re-importer ?

Made some comments to improve some parts but ti's good as is for me as it's improving a lot the part arounf import/reimport

Just one thing, is it possible to add unit tests for importer/re-importer ?

There are quite some unit tests that allowed us to have this PR, i.e. test_import_reimport.py, test_jira_import_and_pushing_api.py, dedupe_test.py. Including import -> reimport -> reimport with changed reports.


Can't select element to relate to from modal

Hi!
When trying to add a (reverse) relationship from a Many to Many field, the element select modal opens up empty (no elements to select from).
Probably because of changes introduces in Craft 3.1, ref: https://craftcms.com/guides/updating-plugins-for-craft-3-1#update-element-source-key-references
So it seems this is easily fixed by changing
'section' => $relatedSection->id

to
'section' => $relatedSection->uid

in craft-manytomany/src/fields/ManyToManyField.php> getInputHtml()
Thank you for providing a nice plugin!

Craft 3.1.15 with project.yaml

I have this issue as well.

@svale : this was recently fixed, so you might want to try again if it's working for you now :)

Fixed in 2.0.0


about symbols...

in onedark https://github.com/joshdick/onedark.vim
all symbols, like '=' or '+-!' etc, have a color (just watch image)
maybe need add?

I agree with you, purple for operators looks better in many languages.
You can customize highlight groups:
require('onedark').setup {
	highlights = {
		Operator = {fg = '$purple'},
		TSOperator = {fg = '$purple'},
	}
}

See https://github.com/navarasu/onedark.nvim#customization

thanks.

oops, I forgot.


Valid mod operations cause a div/0 exception

https://github.com/qmonnet/rbpf/blob/ceb7fa67946a7a6e9cff5a4e03ae721a1ecf10b6/src/jit.rs#L380

Looks like this was done on purpose, why?

I took this code from uBPF, so I'm not exactly the “original author” here :).
As I understand it this should be a conditional jump, triggered only if the divisor is 0 — in which case this is the behaviour we expect to see. How did you trigger it? Are you sure you were not trying to divide by 0?

Yeah, was triggered by a x % 10

I didn't manage to reproduce it :/. Do you have a reproducer, by any chance?

Possibly related to https://github.com/qmonnet/rbpf/pull/89