[BEAM-6754] Use subprocess instead of threads in loopback environment

To Take advantage of multi cores, we need to start separate processes to for worker harness.
Using subprocess instead of threads to run the sdk harness.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

R: @aaltay @mxm @robertwb

Run Portable_Python PreCommit

Agree we should use multiprocessing. We should also consider making this configurable (especially as loopback is used for lightweight testing.)

Agree we should use multiprocessing. We should also consider making this configurable (especially as loopback is used for lightweight testing.)

Multiprocessing doesn't seem to work with grpc.
Making it configurable seems right. Using experimental flag which defaults to subprocess.

I don't think the default should be subprocess. More futureproof might be to have a separate numprocesses setting which if 1 would use a threadpool and for > 1 use subprocess.

I don't think the default should be subprocess. More futureproof might be to have a separate numprocesses setting which if 1 would use a threadpool and for > 1 use subprocess.

Interesting idea. I think that changing the behavior based on number of process (1 and >1) might be bit unintuitive for the user. Also even with 1 process, multiple threads will be created if runner request for multiple SDKHarness.

Fair enough.
I still think it should be opt-in, as the primary advantage of loopback is
that it's as lightweight as possible.
On Wed, Mar 6, 2019 at 10:04 PM Ankur<EMAIL_ADDRESS>wrote:

I don't think the default should be subprocess. More futureproof might be
to have a separate numprocesses setting which if 1 would use a threadpool
and for > 1 use subprocess.
Interesting idea. I think that changing the behavior based on number of
process (1 and >1) might be bit unintuitive for the user. Also even with 1
process, multiple threads will be created if runner request for multiple
SDKHarness.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/7984#issuecomment-470275831, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAdqgef2o35zqdMrZ2iPwnoafiylf8a4ks5vUC1egaJpZM4bdR-U
.

Let's keep Python threads as the default for now. We have not yet experimented enough with processes yet.

Sounds good, updated the default to use thread.


Force bundle push with fresh bundle digest

It's a long story, but this is a hacky but effective way to ensure a bundle push will be triggered with image digests that are new and unique. If we ever need to do it again we can bump the number.
(Previously I've added some random comment or whitespace change, but I didn't want to keep doing that.)
Related to https://issues.redhat.com/browse/HACBS-1603

Codecov Report

Merging #264 (598a37e) into main (4c6bc31) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #264   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           66        66
  Lines         1660      1660
=========================================
  Hits          1660      1660

Impacted Files
Coverage Δ

policy/lib/rule_data.rego
100.00% <ø> (ø)


Branch Validation

Only execute the full script if the push event matches the configured branch. Need to make sure the content type is configured as "application/x-www-form-urlencoded" for the github webhook or modify the code(line 21&22) for application/json.

I checked your commit, would it cause let's say Github to show that the Webhook failed if the branch of the event does not match since nothing was returned?
I did my own implementation and made it return status 200... https://github.com/NanoCode012/simple-php-git-deploy/blob/b51314d848c6f2ab47f0ecbc74af6b66bb43f070/deploy.php#L230-L233

Hi,
I don't think it will cause any webhook failure as the script will still run and return 200 by default even if it's empty. However, I think your implementation is cleaner and will allow invoking the script directly or by any other CI/CD tools as needed (Something that I realized after submitting the PR).


[WFCORE-4999] Remove Zanata configuration as the service is no longer…

… used.
https://issues.redhat.com/browse/WFCORE-4999

@jamezp sorry, I merged #4234 that introduces a conflict in your PR. Could you please rebase and I'll merge it? thanks

Not a problem at all @jmesnil. I've rebased and removed the label.

@jamezp thanks


Object database custom proxy handler implementation

As discussed on the google group forums, it would be very useful if we could give Orient our own custom implementation of proxy object handlers. This would be especially useful in my use case to save the connection the POJO was created from in the proxy handler and therefore always get the appropriate connection in the pool when doing proxied calls.
This would allow a proxied POJO from Orient to be sent to any thread and make UI code or applicative layers independent from Orient which is desirable.
We would need to be able to set a custom factory for making our own version of the invoke method of the com.orientechnologies.orient.object.enhancement.OObjectProxyMethodHandler class.
Note that a custom for Lazy object collections would also be very nice, since these lazy lists may also make calls to the database and we do not want the application/UI layers to have to worry about connections set in threadlocal.

I think this does not make sense anymore, closing


Error in the documentation

When trying to access:
https://facebook.github.io/jest/docs/en/asynchronous.html
only the word "undefined" appears

Did a bit of searching. This is the commit to blame: https://github.com/facebook/jest/commit/f063f9f09bd9f0dabd66c97ac1370e7e712af1e7
It removed everything on that page and replaced that with "undefined".

Also submitted a pull request https://github.com/facebook/jest/pull/4503


Improve workflows

Seems like the diff check in ci.yaml is using a constant and not a variable. Fixes issue #20
And we want to exclude the images from the comparison since the buider produces different output each time it is run with identical source files.

And we want to exclude the images from the comparison

Yes, it is a frustration of mine. I have reached out to the maintainers, but have not heard back. I dug into how the images are generated, and it is fixable, but we are blocked until they respond. I will ping them again.
I kept the image check under the assumption that we would fix it quickly, and thus better to be sure it was changed. Clearly, it is not happening quickly, so your approach is better.


Passing a param from heroku/index.js to a certain state in script.js

Hi there.
I wanna know how can I pass a param/argument from heroku/index.js to a certain state in script.js
I have done like below
heroku.index
Promise.all([
        stateMachine.bot.releaseLock(),
        stateMachine.setState(ai_action), // set new state ADD_MOVIE
        stateMachine.receiveMessage(params.movie_keywords) // calling receive of ADD_MOVIE state and passing a string param i.e params.movie_keywords = "my_string"
    ]);
    res.end();

and in script.js I have defined an state named ADD_MOVIE here is the code
ADD_MOVIE : {
        receive: (bot, message) => {
            const movie_name_searched = message.text;
            console.log("movie_name_searched : "+ movie_name_searched);  // I am getting undefined here
        }
    }

QUESTION
What I am missing here. Am I doing any thing wrong here. please help. I wanna pass the param value from heroku/index.js to my ADD_MOVIE recieve state in script.js.

Oops my wrong. I did't looked very closely.
its message.text => undefined
I have done like const movie_name_searched = message; and it worked. here is my whole ADD_MOVIE state now
ADD_MOVIE : {
        receive: (bot, message) => {
            const movie_name_searched = message;
            console.log("movie_name_searched : "+ movie_name_searched);  // I am getting undefined here
        }
    }


Add taichi-splatting implementation

Hi there - it's not a full reconstruction system like the others - but still might be something people find useful. At the moment it's around 2-3x faster for a full forward/backward pass than the original taichi_3d_gaussian_splatting for scenes I've benchmarked and a good bit more flexible to use.

Very nice. Thank you for your contribution Oliver.

Link is wrong, correct is https://github.com/uc-vision/taichi-splatting

Link is wrong, correct is https://github.com/uc-vision/taichi-splatting

Hi thanks for noticing, it was already fixed already fixed and merged here;
https://github.com/MrNeRF/awesome-3D-gaussian-splatting/commit/7ab3fa86c157c4d0bec8601e62dbbbf301e0a580


Update containerd to 1.3.7

This PR fixes the multiple bug reports relating to containerd not pulling over HTTP, even when forced (https://github.com/containerd/cri/issues/1433).

Have tested locally, with an insecure registry:
microk8s kubectl create deployment nginx --image=<IP_ADDRESS>:5000/nginx:latest

Events:
  Type    Reason     Age        From                       Message
  ----    ------     ----       ----                       -------
  Normal  Scheduled  <unknown>  default-scheduler          Successfully assigned default/nginx-5c8fff9b6-dg969 to ip-172-31-22-112
  Normal  Pulling    4m12s      kubelet, ip-172-31-22-112  Pulling image "<IP_ADDRESS>:5000/nginx:latest"
  Normal  Pulled     4m8s       kubelet, ip-172-31-22-112  Successfully pulled image "<IP_ADDRESS>:5000/nginx:latest"
  Normal  Created    4m6s       kubelet, ip-172-31-22-112  Created container nginx
  Normal  Started    4m6s       kubelet, ip-172-31-22-112  Started container nginx


Should we still add missing charts OWNERS files?

Issue to track PRs adding OWNERS files to charts still missing them.
Note I'm not sure if it's worth the extra 148 PRs now that we're working toward the charts distributed search proposal 🤔 Then again, perhaps any little bit towards distributed ownership could be a help in the PR deluge (these ones at least would be quick & easy to review/merge)? Adding this issue for discussion.
find incubator stable -mindepth 1 -maxdepth 1 -type d '!' -exec test -e "{}/OWNERS" ';' -print | sort -u:

[ ] incubator/burrow
[ ] incubator/cassandra
[ ] incubator/check-mk
[ ] incubator/common
[ ] incubator/couchdb
[ ] incubator/distribution
[ ] incubator/drone
[ ] incubator/etcd
[ ] incubator/fluentd
[ ] incubator/fluentd-cloudwatch
[ ] incubator/gocd
[ ] incubator/gogs
[ ] incubator/goldfish
[ ] incubator/istio
[ ] incubator/jaeger
[ ] incubator/keycloak
[ ] incubator/keycloak-proxy
[ ] incubator/kube-registry-proxy
[ ] incubator/kube-spot-termination-notice-handler
[ ] incubator/kubernetes-vault
[ ] incubator/kubewatch
[ ] incubator/mysqlha
[ ] incubator/oauth-proxy
[ ] incubator/patroni
[ ] incubator/puppet-forge
[ ] incubator/redis-cache
[ ] incubator/riemann
[ ] incubator/spring-cloud-data-flow
[ ] incubator/tensorflow-inception
[ ] incubator/vault
[ ] incubator/vaultingkube
[ ] incubator/webpagetest-agent
[ ] incubator/webpagetest-server
[ ] incubator/xray
[ ] stable/acs-engine-autoscaler
[ ] stable/aerospike
[ ] stable/apm-server
[ ] stable/ark
[ ] stable/auditbeat
[ ] stable/aws-cluster-autoscaler
[ ] stable/bitcoind
[ ] stable/buildkite
[ ] stable/centrifugo
[ ] stable/chronograf
[ ] stable/cluster-autoscaler
[ ] stable/consul
[ ] stable/coscale
[ ] stable/dask
[ ] stable/dask-distributed
[ ] stable/dex
[ ] stable/distributed-tensorflow
[ ] stable/efs-provisioner
[ ] stable/elasticsearch-exporter
[ ] stable/ethereum
[ ] stable/express-gateway
[ ] stable/external-dns
[ ] stable/factorio
[ ] stable/filebeat
[ ] stable/fluent-bit
[ ] stable/gcloud-endpoints
[ ] stable/gcp-night-king
[ ] stable/gitlab-ce
[ ] stable/gitlab-ee
[ ] stable/grafana
[ ] stable/hackmd
[ ] stable/hadoop
[ ] stable/heapster
[ ] stable/heartbeat
[ ] stable/hlf-ca
[ ] stable/hlf-couchdb
[ ] stable/hlf-ord
[ ] stable/home-assistant
[ ] stable/hubot
[ ] stable/influxdb
[ ] stable/ipfs
[ ] stable/janusgraph
[ ] stable/kanister-operator
[ ] stable/kapacitor
[ ] stable/katafygio
[ ] stable/keel
[ ] stable/keycloak
[ ] stable/kiam
[ ] stable/kong
[ ] stable/kube-lego
[ ] stable/kube-ops-view
[ ] stable/kube-slack
[ ] stable/kube-state-metrics
[ ] stable/kube2iam
[ ] stable/kubernetes-dashboard
[ ] stable/kuberos
[ ] stable/kured
[ ] stable/lamp
[ ] stable/linkerd
[ ] stable/locust
[ ] stable/luigi
[ ] stable/mailhog
[ ] stable/mcrouter
[ ] stable/metabase
[ ] stable/metallb
[ ] stable/minecraft
[ ] stable/minio
[ ] stable/msoms
[ ] stable/mssql-linux
[ ] stable/namerd
[ ] stable/neo4j
[ ] stable/newrelic-infrastructure
[ ] stable/nginx-ingress
[ ] stable/nginx-ldapauth-proxy
[ ] stable/nginx-lego
[ ] stable/openebs
[ ] stable/openldap
[ ] stable/percona
[ ] stable/postgresql
[ ] stable/presto
[ ] stable/prometheus-adapter
[ ] stable/prometheus-mysql-exporter
[ ] stable/prometheus-postgres-exporter
[ ] stable/prometheus-rabbitmq-exporter
[ ] stable/prometheus-redis-exporter
[ ] stable/prometheus-to-sd
[ ] stable/rabbitmq-ha
[ ] stable/redis-ha
[ ] stable/rethinkdb
[ ] stable/risk-advisor
[ ] stable/rocketchat
[ ] stable/rookout
[ ] stable/sapho
[ ] stable/selenium
[ ] stable/sensu
[ ] stable/sentry
[ ] stable/signalfx-agent
[ ] stable/spark
[ ] stable/spartakus
[ ] stable/spotify-docker-gc
[ ] stable/stackdriver-exporter
[ ] stable/stolon
[ ] stable/sugarcrm
[ ] stable/sumokube
[ ] stable/sumologic-fluentd
[ ] stable/superset
[ ] stable/telegraf
[ ] stable/tomcat
[ ] stable/uchiwa
[ ] stable/unbound
[ ] stable/verdaccio
[ ] stable/weave-scope
[ ] stable/zeppelin
[ ] stable/zetcd

i can own some if needed.

@scottrigby cool, I will try to join the next meeting. the problem it is in the same time as my planning weekly meeting :(
but count me for any help

Before we make owners files we do need to make sure the people are trusted and have contributed. There are numerous people listed as owners who have not contributed before.
K8s has a trust system via sponsorships. How do we trust the people we add to owners files?

Should this list exclude deprecated charts? incubator/drone has been moved to stable and it has its OWNER file, the incubator chart is there only because of the deprecation process.

incubator/cassandra done :)

@cpanato The decision here was to add these OWNERS files only as needed (when people request to help maintain them), since our efforts would be better spent moving to the distributed charts hub than opening all of these PRs each with a chart bump (and the coordination with other PRs against those same charts also bumping the version).


kernel: add 4.8-rt1

with modinst_arg_list_too_long patch
that is missing from https://github.com/musnix/musnix/pull/36

building, will report back when done

@tg-x I look forward to yr report.  Thanks for digging into this.

Can't wait to try this out!!! I'll probably get to it later today... Will report back. Thanks @tg-x for figuring that out!

So nice thing was that I didn't have to rebuild the kernel for the extra patch to fix my computer not starting! :+1: @tg-x

it did compile and started booting, but after a few seconds the whole system slowed down so much that it took at least half an hour or more just to get a root prompt..
then messages appeared about several million of printk messages dropped..
which could be the reason the whole thing slowed down so much..
saved some dmesg output which I stopped after several minutes and 400MB..
I'll extract useful parts of it and post it here..

@tg-x I'm confused, you're saying it doesn't work for you? Or should it not be working for me for some reason? How can I check if I actually got the kernel installed properly?

here's some dmesg output
it seems the issue is around
[<ffffffff9aa99af5>] migrate_enable+0x65/0x130
[<ffffffff9af5b647>] rt_spin_unlock+0x27/0x40
a quick search shows some LKML discussion about something similar,
I won't dig into this further but wait for the next RT patch and try if that works..

@erlandsona no, it doesn't work for me, does it work for you?
if you did not rebuild it then you probably on an older kernel..
check what uname -a says..

(as a side note, the reason I was looking into this
because virtualbox kernel modules won't compile with 4.6-rt kernels,
which is fixed in 4.8, or can be also fixed for 4.6 with a patch)

Here's output from my uname. It doesn't suggest I'm running anything rt, I just figured uname wasn't showing the flag?
❯ uname -a
Linux erlandsona 4.8.1 #1-NixOS SMP Fri Oct 7 13:03:33 UTC 2016 x86_64 GNU/Linux

I'm sticking with 4.8.1 though for now because it seems to fix an issue with my fan speed.

4.8.2-rt2 is out, let's see if it works, build it now..
@erlandsona it should have -rt in there, you probably removed it from your config then..
should set
musnix.kernel.packages = pkgs.linuxPackages_latest_rt;
and should not set
boot.kernelPackages

I had boot.kernelPackages set to something I think... that makes sense. 👍 Thanks!

just booted 4.8.2-rt2, same issues as with 4.8.0-rt1

@tg-x that's odd, it seemed to work just fine on my machine? Granted, I'm no expert and I'm probably doing something wrong.

it's perfectly possible that this issue only occurs in certain cases and does not affect you
(didnt inspect it further, maybe something related to btrfs? there were also some btrfs-related functions in the backtrace..)

@tg-x: How's 4.8.2-rt3 working for you?

4.8.2-rt3 works, no longer have the issue above

Woot woot!


[BUG]: Pi Camera not working after completing the installation of the Photobooth system

Describe the bug
My Raspberry Pi basic environment is 3B+, and I installed Bullseye -ARM64-lite. Before running the install-photobooth script, I made some necessary configurations for the pi-camera (V1.3 ov5647), and the pi-camera was working fine. However, after running the install-photobooth script, the web service of the Photobooth started successfully, but the pi-camera stopped working.
Please note that this is a translation and may not capture the exact meaning or tone of the original text. It is always recommended to provide the original text along with the translation when communicating with developers for better understanding and accuracy.
Steps to reproduce the behavior
I performed standard updates and executed "sudo rpi-update".
Run raspi-config, I enabled "legacy camera support", disabled the "GL driver", and enabled "glmor".
sudo nano /boot/config.txt, I confirmed the following settings: dtparam=i2c_arm=on, dtparam=i2s=on, start_x=1, #dtoverlay=vc4-kms-v3d, dtoverlay=vc4-fkms-v3d, and dtoverlay=ov5647.
After completing the above steps,  reboot.
run "vcgencmd get_camera" shows "supported=1 detected=0, libcamera interfaces=1".
run "libcamera-still -n -o %s -q 100 -t 1 | echo Done" provides camera information feedback.
run"libcamera-still -o test.jpg" works fine.
after running the install-photobooth script,the web service of the Photobooth is running properly and can be accessed,
but at this point, the pi-camera stops working.
run "libcamera-still -n -o %s -q 100 -t 1 | echo Done"
feedback:
Done
[0:04:47.945102977] [794] INFO Camera camera_manager.cpp:299 libcamera v0.0.4+22-923f5d70
ERROR: *** no cameras available ***
Expected behavior
How to properly configure pi-camera support
Photobooth Version
The query shows that it is the latest version, installed through the installation script.
Browser and browser version
No response
Operating System
No response
Did it work before?
No
Latest working version
No response
Additional context
No response

Furthermore, the FAQ mentions the following information:
"Once done, you need to adjust the configuration. Open the admin panel in your browser at localhost/admin and make the following changes:
'Take picture command on Pi OS based on bullseye':
libcamera-still -n -o %s -q 100 -t 1 | echo Done"
However, I cannot find any configurable content in localhost/admin. Is there a more specific section or location for this configuration?

I get you installed the service for preview from DSLR , those disables the pi camera.
Run
sudo bash /var/www/html/gphoto/install-gphoto-python.sh`

Could it be the same issue as this one?
raspberrypi/libcamera-apps#534
To fix it, I had to add camera_auto_detect=1 in my /boot/config.txt file


a52dec: 0.7.4 -> 0.8.0

Description of changes
Closes #296886.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I actually tried bumping this a week back and I was facing atleast 8 build failures, though some looked unrelated. Didn't have to investigate further. Just thought should let you know.

 Probably ofborg can help?
@ofborg build python311Packages.paperwork-backend

I actually tried bumping this a week back and I was facing atleast 8 build failures, though some looked unrelated. Didn't have to investigate further. Just thought should let you know.

None of the failures are related. I haven't opened a PR for each of them, though.

Nice work @wegank


aligns sepolia with mainnet parameters

@noisekit can we get this merged please, i'm signing off for today, I want to send this to the chain please for testing before we go live with mainnet, to make sure we are good on everything.


Share code between TCP and HTTP connection pools

It'd be good if we didn't have to implement features twice as we add things like prefetch logic.

(forgot to tag #11529 with this)


One command validator up and running

The process of setting up a new node+tower (either for a miner or a validator) ain't easy.
Create an end-to-end script that goes over this documentation and setting up everything automagically.
Try to account for as many as possible OSs, distros and errors that may appear along the way.--
Incorporate either into the cli, and/or possibly as an external pre bash command, this one-command to rule them all.
Bounty
5000

If going with Rust, worth checking tools mentioned here

test comment

Very much agree @corynthian .
Luckily for us, @nourspace and @MichaelZelbel been working on the docker and are in good place from what I know.
I see the standard process to be shifting towards a setup which just requires you to change some placeholders in a config and be in your way.
As for (1), it makes sense. Would be happy if @simsekgokhan , @sirouk and @0o-de-lally chime in.
Can you pick the ticket if will be placed on the board? I think it is an important one as it is the entry point into the project. Ease of setup has much value.

Thank you @hemulin. Please refer to https://github.com/nourspace/0l-operations which I have been using since day 1 to operate anything 0L. I also managed to get my v6-testnet node to run using the same setup.
The idea is simple; a layered docker image that provides the necessary environments for different purposes–mainly running in production.
The docker-compose wraps the image with sensible configurations like commands, environment vars, ports, and volumes to properly isolate the project. Then there is a Taskfile that acts like a more readable Makefile with common tasks defined.
Once we are inside docker, there is no need to worry about OS differences. Providing different OS-dependent setups or packages doesn't seem like a viable solution for a production-ready network. It was indeed helpful to use tmux/screen throughout the development and bootstrapping phases, but this–in my opinion–better change to a more robust deployment solution.
Looking at other networks, those built on Cosmos SDK for example, I see they are mostly using Docker images which are often run not only with a simple docker-compose but rather in a K8s cluster with Helm charts. Some went further and created a fully-fledged K8s operator to manage the deployments, snapshots, recovery, and other operations.
Moving forward, I want to polish the Dockerfile I'm using and add more documentation. Next would be the move to K8s resources, either as Kustomize manifests or a Helm chart.
I agree with what @corynthian said regarding account generation and other steps. I also think such steps should be run outside any setup as they require use attention and most likely entering a mnemonic.

Generic images
https://github.com/strangelove-ventures/heighliner
Images to run on Akash network
https://github.com/akash-network/cosmos-omnibus
Cosmos Operator
https://github.com/strangelove-ventures/cosmos-operator


Recursively freeze generic objects

A new function, freeze.Object, should be the default function used by callers. This has nice symmetry with JavaScript's Object.freeze(). However, freeze.Object will be less efficient than an equivalent freeze.Pointer or freeze.Slice, so it may not be appropriate in all cases.
Object freezes its argument according to the following rules:

If the object is a pointer, the underlying data is recursively frozen, and the pointer is updated.
If the object is a slice, its elements are recursively frozen, and then the slice itself is frozen.
If the object is a struct, its fields are recursively frozen, and then the struct itself is frozen.

Different rules are needed for the top-level call to Object. Specifically, top-level pointers should not be frozen, only the data they point to.

Object is implemented now, but it's pretty ugly. I'd like to spend some time making it more readable and efficient. The biggest problem is the conversions between reflect.Value and interface{}.


Problems adding custom control in component

I have difficulties to add a custom control on my map. I put this question with the regarding code on stakeoverflow([https://stackoverflow.com/questions/47608576/leaflet-customcontrol-angular-2]), but since know one there is able to answer the question, i try it here. (This could definitly be added on the README.md)
Best regards

I added the ngx-leaflet tag in SO and added a comment.

appreciate the fast respond, thanks!

Following up in SO, gonna close this.


release-2.0: Key Concepts » Smart Contracts and Chaincode

assign to me

assign me to review

reviewed


Don't show whitespace before the preheader text

Don't show whitespace before the preheader text in Outlook App for Windows 10, possibly caused by the  .
This is an image that compares two preheader texts:

As you can see, the second email's preheader text shows trailing whitespace, while the first one doesn't.
Please do notice I added an image before and outside the white box of the template, but I think the white space probably comes from the  s

Thanks @AeonFr


pipx installation doesn't work

Describe the bug
Having trouble to install with pipx
To Reproduce
Steps to reproduce the behavior:

brew install pipx
pipx install llm-toolkit

Expected behavior
Installs fine
Screenshots

Environment:

OS: MacOS

This should be fixed now! Let me know if you run into this again.


Glassmorphism Card

Is your feature request related to a problem? Please describe.
Many times we need cards with having glassmorphism effect.
Describe the solution you'd like
I have created Glassmorphism cards.
Describe alternatives you've considered
I used HTML and CSS to make it.
Additional context
https://user-images.githubusercontent.com/98695632/186598928-6c4acee0-503f-424c-882e-725f702925ad.mp4

We already have a glassmorphism card !


Smart debounce

This has been asked and answered too many times. It's tricky to implement for users even since we provided the queryHook option.
We need to implement a debounce option on the SearchBox.
This is tricky to do it right, as @bobylito noted we could do it the "dumb way" (my first idea) and just use lodash debounce, done.
But this would lead to people using debounce and debouce searches to every 700ms which would be bad for the Algolia product overall.
@bobylito proposed to develop an adaptative debounce that would adapt to the user typing speed behavior. We liked the idea.
We can then see this feature as an addition to the overall Algolia experience, it's true that for some heavy typists, making one request per keystroke is not at all smart and efficient. And does not provide a better experience: there's no way we are gonna render anything if there's only say 15ms between two keystrokes.
First the network will take some time, then the rendering will take some time, and once that finished we may already be two keystrokes further.
PS: Should we also adapt to the network/response time? If the network response time * 2 > interval between two keystrokes, is it worth making a new request next time immediately?

Hi is there any progress in this area or do i have to resort to using lodash in an implementation of my own?

For now you have to use lodash yep and the queryHook (when using instantsearch.js)

User feedback linked to this: https://news.ycombinator.com/item?id=13962790

Will be handled by #1994


Regarding the upcoming  variational autoencoder feature

This is great stuff!
The README.md mentioned that there is a plan to support variational autoencoder, which I very much look forward to.  My understanding is that the vanilla VAE tends to create blurry images (perhaps due to the disentangled representation that it learns), and I wonder if the planned VAE feature will be based on the vanilla VAE, or something like the autoencoding beyond pixels, or adversarial autoencoders?

We are looking at different architectures right now such as VAE-GAN or PPGAN. I am busy with a paper deadline right now, and plan to add new architectures by the late Jan or early Feb. If you would like to help add a particular network, send us a pull request.


player.getVideoTracks is not a function

Have you read the FAQ and checked for duplicate open issues?
Yes.
What version of Shaka Player are you using?
Master
Can you reproduce the issue with our latest release version?
I am using the master one, I supose it's the latest.
Can you reproduce the issue with the latest code from master?
Yes, it's happening with it.
Are you using the demo app or your own custom app?
My custom one.
If custom app, can you reproduce the issue using our demo app?
The demo doesn't work at all (gives cors error).
What browser and OS are you using?
Browser is Vivaldi and OS is windows 10
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
No, at this time.
What are the manifest and license server URIs?

Not using licenses at this time.
What configuration are you using?  What is the output of player.getConfiguration()?

What did you do?

See the source code below (I am trying to get the video tracks and will need to get the audio tracks, so I can select which one will be played.
What did you expect to happen?
Have the list of video(s) track(s);
What actually happened?

Get the error:
Error code undefined object TypeError: player.getVideoTracks is not a function
    at shakainit.js:58
    at async initPlayer (shakainit.js:50)

Line 50 is where this code is executed:
var videoTracks = player.getVideoTracks();
and more, the autoplay doesn't work when defined at the <video DOM.
PS:
I changed the line of error to:
var videoTracks = player.getVideoTracks;
but now, I get the error:
shakainit.js:107 Error code undefined object TypeError: Cannot read property 'length' of undefined
    at shakainit.js:63
    at async initPlayer (shakainit.js:50)

line 63 is where there is this code:
for (var i = 0; i < videoTracks.length; ++i) {
If I click on Play button, the video plays.
Html code:

<!DOCTYPE html>

    <video id="video"
    poster="image.jpg"
    controls
    autoplay>

        Your browser does not support HTML5 video.
    </video>

Javascript code (shakainit.js):
const manifestUri ="address of the m3u8 stream";

function initApp() {
    // Install built-in polyfills to patch browser incompatibilities.
    shaka.polyfill.installAll();

    // Check to see if the browser supports the basic APIs Shaka needs.
    if (shaka.Player.isBrowserSupported()) {
        // Everything looks good!
        initPlayer();
    } else {
        // This browser does not have the minimum set of APIs we need.
        console.error('Browser not supported!');
    }
}

async function initPlayer() {

    const video = document.getElementById('video');
    const player = new shaka.Player(video);

    window.player = player;
    player.addEventListener('error', onErrorEvent);
    player.addEventListener('adaptation', onAdaptationChange);
    player.addEventListener('variantchanged', onAdaptationChange);

    try {
        await player.load(manifestUri).then(function() {
            console.log('The video has now been loaded!');
            var videoTracks = player.getVideoTracks();
            var activeTrack;

            var ul = document.getElementById('videoTracks');
            for (var i = 0; i < videoTracks.length; ++i) {
              var track = videoTracks[i];
              if (track.active) activeTrack = track;

              var text = track.width + ' x ' + track.height;
              text += ' ' + (track.bandwidth / 1024).toFixed(0) + ' kbits/s';

              var li = document.createElement('li');
              li.textContent = text;
              ul.appendChild(li);
            }

            if (activeTrack) {
              var aspectRatio = activeTrack.width / activeTrack.height;
              video.width = video.height * aspectRatio;
            } else {
              console.error('Unable to query aspect ratio!');
            }

            video.play();
        });

    } catch (e) {
        onError(e);
    }
}

function onErrorEvent(event) {
    onError(event.detail);
}

function onError(error) {
    console.error('Error code', error.code, 'object', error);
}

function onAdaptationChange(event) {
    var tracks = shakaPlayerInstance.getVariantTracks();
    for (iterator = 0; iterator < tracks.length; iterator++) {
        if (tracks[iterator].active) {
            console.log("Bandwidth: " + tracks[iterator].bandwidth);
            break;
        }
    }
}

document.addEventListener('DOMContentLoaded', initApp);

The function is getVariantTracks.


Create a security policy

Description of issue or feature request:
Similar to go-tuf, we should create a security policy so that researchers can properly disclose security issues.

Sure, good idea. However it doesn't look like go-tuf has a security policy set up. Did you mean python-tuf?

Sure, good idea. However it doesn't look like go-tuf has a security policy set up. Did you mean python-tuf?

(Inspired by the lack of it in go-tuf)

I'm not sure python-tuf has the best policy right now (depends on one person and PGP)


Does not work on Windows

Annoyingly I can't copy the text from my terminal when running this project, probably some control sequence does something weird but running the binary (built from source) prints the following:

The relevant line is here.
This definitely seems to be a windows thing which I'm not sure if you even want to support in the first place. In case you do,
the following works for me: (not that the cmd equivalent of ls is dir)
use std::process::Command;

fn main() {
  let mut list_dir = Command::new("cmd");
  let output = list_dir.args(["/C", "dir"]).output();
  dbg!(output.unwrap());
}

Of course this would probably require you to rewrite the parsing stuff. You could also experiment with spawning a powershell instance instead of a cmd one in case the output is easier to work with (doubt it's much different and it would probably also be slower but just mentioning it here).

The program hasn't even been tested on windows, I haven't thought about it either, however, if you plan on using the program I will try and make it cross platform

That would be cool :)
I can try and help with both testing and/or implementing certain parts of it given the proper guidance 😅

Yes sure, if you feel you can implement parts of it now, then go for it. If not I'll try to start it and see where it'll take us


feat: Relocate Probot repositories to Operate First

Related to: https://github.com/open-services-group/community/issues/231

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please ask for approval from tumido after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

/approve

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

Needs approval from an approver in each of these files:

OWNERS [tumido]

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


Edit doesn't maintain layer correctly

Issue
When Editing a label on the back silk layer, the resulting updated label is on the front silk.
Steps to reproduce

Create a label
Place it on the back silkscreen by pressing F to flip.
With the back silkscreen label selected open KiBuzzard, the label is loaded up to edit.
Update the label and click OK
The resulting label is on the front layer.

Additional info
If the label has a rotation off 90 degrees the resulting rotation is also wrong in the final label.
Version
KiBuzzard 1.3.0 on KiCad 6.0.9 (Win) installed from PCM

This issue is related to the method used to determine if the label required flipping.
The code has a hardcoded "B." string it tries to match. This matches "B.Cu" correctly. However KiCad lets uses input custom label names. Mine is set to "Front/Back". This breaks the check.
Ideally a solution would use the constant enum value for determining the layer rather than a user defined label that's returned from FOOTPRINT.GetLayerName()


pass

pass should be installed. http://www.passwordstore.org/

Created ansible role: https://github.com/lbischof/ansible-role-pass

Added pass role f20b59605db959432a0ca8ef7187f6f628c29203


Stop expanding PValues in DirectRunner visitors

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[x] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

A PValue always expands to itself, and these calls are unneccessary.

R: @kennknowles

LGTM


Response Body and ResposeBody Highlight Tabs showing different values

I am using RESTClient plugin in firefox Version 42.0.
It shows two different values Response Body(Ray) tab and Response Body(Highlight) tab.
Attached screenshot.

Response Body (Raw) shows correct response. Just look at the person_id value.
person_id in the above response is Long datatype.


add correction for the concatenation in path_list

add correction for the concatenation in path_list
add a when condition to ensure the correct conditions are met when constructing the link_list.

I was having trouble to create the generic rom folders due to the way the concatenation was done. This fixed the issue on Debian 11.2, 11.7 and 12

I also experienced concatenation issues on Debian 12.

This needs to be rebased against the testing branch, we don't accept pulls into main

This needs to be rebased against the testing branch, we don't accept pulls into main

Sorry I did not know, let me redo the PR. Also the Mister CIFS script has the same issue so I'm going to change it too.


SOQL : lstOpportunity = [SELECT Account.Group__c,Account.Id,Account.Ts_Id__c,Type FROM Opportunity where Id IN :mapOfOpp.keySet()]; In this example, " :mapOfOpp.keySet()" is not getting parse

I am trying to use this plugin to build few rules. But I have found few issue related to parsing SOQL.
Example -->
SOQL : lstOpportunity = [SELECT Account.Group__c,Account.Id,Account.Ts_Id__c,Type FROM Opportunity where Id IN :mapOfOpp.keySet()];
In this example, " :mapOfOpp.keySet()" is not getting parsed.
The above is just an example, I want to discuss with you few of these issue.
If possible, we will like to have a call with you over skype.
This is an high priority issue for us, as this has been escalated by client.
Taging @pagwekar

@sachinshinde729 - see #116 comments as the proposed fix in #119 should address this as well


Implement headers for messages

At the moment, buneary cannot publish messages with headers. Sending messages with headers should be enabled by implementing a --headers or --header option.
There are two possibilities here:

Either let the user pass a list of key-value pairs to the flag: --headers="key1=val1,key2=val2"
Or let the user specify multiple flags: --header key1=val1 --header key2=val2

@aligator PTAL

I would do it like the first one. As it avoids many --header flags


Configuration error from kube logs/debug

Configuration error from kube logs/debug
For "all-in-one"  deployment approach, there is a error shown as:

This error come from the
[https://kubeedge.io/en/docs/advanced/debug/]: Enable Kubectl logs/exec to debug pods on the edge
Casue the IPtables Rule not activate.  May be maintainer should remind user to enable "debug" && IPtable Rule, while configuration of Sedna-local_install && all-in-one deployment scrapt.

/ping @JimmyYang20


Switch to .NET 8 RC

Switching to .NET 8 is slated to bring performance improvements, and is the latest LTS version of .NET.
There are so some much needed changes to System.Text.Json to make #231 easier
AB#2102423

.NET 8.0.100-rc.1 was just released yesterday, and is a go-live release: https://dotnet.microsoft.com/en-us/download/dotnet/8.0

.NET release 8.0.0 was released on November 14th, 2023. See (8.0.0 release notes)[https://github.com/dotnet/core/blob/main/release-notes/8.0/8.0.0/8.0.0.md] for more details.

There is an ask to support .NET 8 for ARM64 for OSX in the sbom-tool via https://github.com/microsoft/dropvalidator/issues/848


chore: benchmark protobuf

Motivation

Track the performance of protobuf in libp2p-gossipsub

Codecov Report

Merging #320 (85873bb) into master (74c08b1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #320   +/-   ##
=======================================
  Coverage   79.80%   79.80%
=======================================
  Files          44       44
  Lines       11037    11037
  Branches     1013     1013
=======================================
  Hits         8808     8808
  Misses       2229     2229

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


mc.action gets dropped in ex commands

Config:
{
  "jake-stewart/multicursor.nvim",
  commit = "9f8a480",
  config = function()
    local mc = require("multicursor-nvim")

    mc.setup()

    vim.keymap.set("n", "x", function()
      print("Running action")
      mc.action(function(ctx)
        print("  action!")
      end)
      print("Ran action")
    end)

    vim.api.nvim_create_user_command("X", function()
      print("Running action")
      mc.action(function(ctx)
        print("  action?")
      end)
      print("Ran action")
    end, {})

    vim.api.nvim_create_user_command("Scheduled", function()
      print("Scheduling action")
      vim.schedule(function()
        mc.action(function(ctx)
          print("ran action!")
        end)
      end)
      print("Scheduled action")
    end, {})
  end,
}

When pressing the custom x mapping in normal mode, :messages reads as expected:
Running action
  action!
Ran action

However, :normal x silently fails to execute the action:
Running action
Ran action

This is also the case if we run the same basic code through a custom user command, :X in the above config:
Running action
Ran action

The only thing I can think to do is vim.schedule the mc.action so that it triggers asynchronously after the ex command is finished. So with the :Scheduled user command above:
Scheduling action
Scheduled action
ran action!

Is there anything smarter mc.action could do? I think ex commands are explicitly being short-circuited by this condition: https://github.com/jake-stewart/multicursor.nvim/blob/f68e66cb9d490662571f664c313b6eeca774b25a/lua/multicursor-nvim/input-manager.lua#L57-L59 I assume that's there so that when you're on the ex command line typing things, it doesn't feed input into the multicursors. But I'm also guessing self._cmdType remains truthy even after I hit <cr> and start executing my command that winds up trying to trigger the mc.action. I don't know enough about the control flow here to hazard a guess at any fixes, though.

It looks like we can just get rid of that code.
 if self._applying or self._inInsertMode or self._cmdType then
     return nil
 end

Everything seems to work without it.


Line #933 missing an escape character

Just a heads up that line  #933 is missing an escape character:
"\t\t\"**/.gitattributes":true,\n" +
vs
"\t\t\"**/.gitattributes\":true,\n" +

Awesome!  :) looks like the community already pushed me a fix for that! <3 the community


fix: Enable (Native) Wayland Support

This fix enables native Wayland on modern Linux distributions.
This is especially important when using fractional scaling on HiDPI displays, as RocketChat is blurry there and looks somewhat :hankey:y...
The x11 can be dropped as it is already covered by unity7 as per Snap desktop interface documentation: https://snapcraft.io/docs/desktop-interfaces
I added the necessary plug and the native wayland flag of the builder, plus 2 arguments that make electron choose the wayland compositor if it is available.

One more thing: The electron-builder update to v24 seems to be necessary in order to get the flags passed to rocketchat correctly. Seemed as if there is some kind of bug in v23.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


cmd/compile: switch over wrapped singletons not matching

(Note: this is my first attempt at writing generic code, the chances I am just wrong are high.)
crypto/elliptic has a few functions like P256() Curve and P384() Curve that return a singleton implementation of that curve. The singleton is part of the contract, because applications are expected to check what curve is in use by comparing it against the return value of elliptic.P384().
In CL 360015 I experimented with turning the concrete implementation of those singletons into a pointer to a generic type. (Feedback on the use of generics welcome!)
Unexpectedly, this led to a test failure that stems from this switch not matching the singletons anymore.
https://github.com/golang/go/blob/2ebe77a2fda1ee9ff6fd9a3e08933ad1ebaea039/src/crypto/x509/x509.go#L508-L521
I tried reproducing it with a contained example on the playground, but things worked as expected, so I am not sure what is going on. https://go2goplay.golang.org/p/9jnZvqnPDrh
/cc @griesemer

Can you whittle down (remove everything that's not needed to re-produce the problem) in the real code and send that CL? Then we can cherry-pick and investigate.

Hm, the switch is failing because the itab of the incoming interface doesn't match the itab returned by the calls.
There are 4 identically-named, but different, itabs in this binary:
00000000011f4e08 s _go.itab.*crypto/elliptic.nistCurve[*crypto/elliptic/internal/nistec.P224Point],crypto/elliptic.Curve
00000000011f4e50 s _go.itab.*crypto/elliptic.nistCurve[*crypto/elliptic/internal/nistec.P224Point],crypto/elliptic.Curve
00000000011f4e98 s _go.itab.*crypto/elliptic.nistCurve[*crypto/elliptic/internal/nistec.P224Point],crypto/elliptic.Curve
00000000011f4ee0 s _go.itab.*crypto/elliptic.nistCurve[*crypto/elliptic/internal/nistec.P224Point],crypto/elliptic.Curve

That's not good. We require that itabs be unique, so that we can type switch using equality of itabs.

The linker is not deduplicating the itabs because their _type fields are different.
The 4 types are
1190f40
1191020
1191100
11911e0

I'm not sure yet why the types aren't being deduplicated. They have the same name, and they look identical except for the ptrToThis fields.

I've made a small repro that I think is the same underlying cause.
I think we need type deduplication in the linker.
It doesn't look like we have such a thing currently. Kind of surprises me that it hasn't been needed to date. Yet tests without generics that should provoke the same problem ... don't. More investigation needed.

@rogpeppe prepared a self-contained testscript for this issue in #49245.
go run ./pkg1

-- go.mod --
module m

go 1.18
-- pkg1/main.go --
package main

import (
	"fmt"
	"os"
	"reflect"

	"m/pkg2"
	"m/pkg3"
)

func main() {
	i1 := pkg3.GetI()
	i2 := pkg2.Inst
	instEqual := i1 == i2
	if !instEqual {
		v1, v2 := reflect.ValueOf(i1), reflect.ValueOf(i2)
		ptrEqual := v1.Pointer() == v2.Pointer()
		typeEqual := v1.Type() == v2.Type()
		if !ptrEqual {
			fmt.Printf("pointer values are unequal\n")
		}
		if !typeEqual {
			fmt.Printf("types are not equal\n\t%v\n\t%v\n", v1.Type(), v2.Type())
		}
		os.Exit(1)
	}
}

-- pkg2/x.go --
package pkg2

import "m/pkg3"

var Inst = pkg3.GetI()

-- pkg3/x.go --
package pkg3

type I interface{}

type impl[P any] struct{
	_ byte
}

var instance = &impl[int]{}

func GetI() I {
	return instance
}

> go run ./pkg1
[stdout]
types are not equal
	*pkg3.impl[int]
	*pkg3.impl[int]

[stderr]
exit status 1


Implement dep-free argument parser

It seems that we could do the parsing with no dependency in Swift rather simply:
https://swift.org/blog/argument-parser/

Done via https://github.com/dirtyhenry/pomodoro-cli/pull/10.


publish method for datachannels

We could add publish and hasNewData methods to dataChannel:
void publish(lms::Time t= lms::Time::now())

It would set some internal time value.
bool hasNewData()

Would check the internal time value and return true after publish has been called only once for each module.

Is it really necessary to save a timestamp here? (Do you need the timestamp?

It would make sense to have the publish-method only for WriteDataChannel and hasNewData only for ReadDataChannel.


Datacapture, refactoring single choice options code to apply custom style.

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #1007
Description
Use .xml layout file with RadioButton to create radio button options. So that it will be inflated by MaterialLayoutInflater at run time as material theme is declared as datacapture ttheme.
Alternative(s) considered
Have you considered any alternatives? And if so, why have you chosen the approach in this PR?
Type
Choose one: Bug fix
Screenshots (if applicable)
Checklist

[x] I have read and acknowledged the Code of conduct
[x] I have read the Contributing page
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA
[x] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project
[x] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally
[x] I have built and run the reference app(s) to verify my change fixes the issue and/or does not break the reference app(s)


Create LICENSE

#39 apache 2 license

are you waiting for the contributors to agree to this license?


In Datagrid Total Number Of Records And Filter Records Should Be Displayed

Feature Request

Currently in data grids we don't know, how much amount of data is present, it will be good if we provide a total number of records and filter records. Just like other data tables.

Looking forward to this update


Functional test specs

Added page spec and tests for work list component and app component.

This change is

Can one of the admins verify this patch?

@naina-verma @sanbornsen @nimishamukherjee @SMahil can you please review this.


Selecting 'Save' pops up an error after trying to close the file in editor

Repro steps:

Drill down from Subscription ->web app ->Files, then select one file and make changes.
Click 'x' to close the file, then make sure that pops up an dialog.
Select 'Save' -> 'Upload', then check the result.

Expect:
Succeed to save the file.
Actual:
Fail to save the file with error.

Action: appService.fileEditor.onDidSaveTextDocument
Error type: Error
Error Message: Cannot update file after it has been closed.
Version: 0.9.0
OS: win32

Ok. Thanks for your update.


Reject leading/trailing whitespaces for token id

Follow up on https://github.com/CosmWasm/wasmd/issues/1206#issuecomment-1450149264

Agree on this - respectively I don't see why token ids should have whitespaces.
valid token ids:

alphaNumeric111
001
spaces between words
_underscore_

invalid token ids:

 leading spaces
trailing spaces


Fixed effect of beacon for assembling machine

Fixes for https://github.com/Teoxoy/factorio-blueprint-editor/issues/218

I can't check rebased commit because of rebased command yarn start(yarn start:backend) can't run on win =(

Oh, I need to update the contributing document to include rust.
I think to get up and running, you are missing rust and systemfd.
Let me know if it works and I will then update the docs.

How is the performance of the implementation for blueprints with quite a lot of beacons (100+)?

@Teoxoy sorry for a late reply,
I can not run rust (and backend) on windows 10, i get a lot of dependency errors.

How is the performance of the implementation for blueprints with quite a lot of beacons (100+)?

The complexity of algorithm is O(n) so the performance is directly proportional to the number of elements in the blueprint. Each beacon is ckecked is for being in a specific area in which it can have an effect on this building (using a few arithmetic operations).

Ok, thanks!
Just to let you know, it might take a while until I can merge this PR. I might want to optimize this feature by making it O(1) - since it will be needed for #21 and I'm still working on optimizing the backend for the new release process (everything is blocked by this right now).

Oh, I need to update the contributing document to include rust.
I think to get up and running, you are missing rust and systemfd.
Let me know if it works and I will then update the docs.

and cargo-watch


Support all float dtypes in F.mean_absolute_error

This PR is a part of #4582 and generalizes F.mean_absolute_error to support all float dtypes other than float32, with losing the condition of inputs.

Looks good.
Jenkins, test this please.

Jenkins CI test (for commit 2f7897b70d2e0597ed90a40ddc1cc7a29059c3bb) succeeded without errors!


Add SeisSol to package index

Added the SeisSol package listed on Issue: #232  to the package index.

I also thought this is a C++ project with legacy Fortran, but it seems it has physics stuff in Fortran and it is still developed in Fortran (see e.g. https://github.com/SeisSol/SeisSol/pull/536), so I would say this package fits into our index.
(If it was just Fortran wrappers, then I would say we should have a category for Fortran wrappers to other packages.)

For the record, the developers have told me they would like to remove Fortran soon. See also the comment here: https://github.com/fortran-lang/fortran-lang.org/issues/232#issuecomment-1099727757

As the package was still a part of checklist on #232 and the library and its documentation showed references to Fortran usability I figured it was still up for addition. While I did stick to the guide the criterias for package addition are a bit ambiguous to me as I am new to the language. If the package doesn't meet the requirements I will add a PR to revert these changes as needed.


When you focus on the cvv field, the card flips but the back side of the card dissapears

When you focus on the cvv field, the card flips but the back side of the card disappears. When it is flipping, I can see the back side of the card. But after it has flipped, I can just see the mirror image of the front side of the card.

Same. makes it unusable. looking for fixes in PR's that were submitted all the way back in August.

Here's a quick fix for you guys:

copy over the card.css into your own (I copy/pasted it from node_modules/card-react/lib/card.css) -- (you may want to prettify it)
search for the following css rule:

.jp-card .jp-card-back,
.jp-card .jp-card-front {
    -webkit-backface-visibility: hidden;
    backface-visibility: hidden;
    -webkit-transform-style: preserve-3d;
    -moz-transform-style: preserve-3d;
    -ms-transform-style: preserve-3d;
    -o-transform-style: preserve-3d;
    transform-style: preserve-3d;
    -webkit-transition: all .4s linear;
    -moz-transition: all .4s linear;
    transition: all .4s linear;
    width: 100%;
    height: 100%;
    position: absolute;
    top: 0;
    left: 0;
    overflow: hidden;

and add
z-index:3;
to it.

Sorry for the delay, Iv'e added the fix in version 1.2.5


gcc 9.1.0 compile error

gcc -c dns.c -o dns.o -D_DEFAULT_SOURCE
  dns.h:527:16: error: lvalue required as unary ‘&’ operand
  527 |  dns_rr_i_init(&dns_quietinit((struct dns_rr_i){ 0, __VA_ARGS__ }), (P))

Same here...

I created a fix for this issue;
https://github.com/wahern/dns/pull/34

still error. merge?

still error. merge?

The fix has not been merged. Also, the last commit to this project was in December 2016...


zuul: install docker with pip for the manager role

Since we are working within a venv, but the python-3docker package is installed via apt, it must be installed manually in the venv so that the facts of osism.services.docker work.

Recheck


Drop CUDA 10.x support.

CUDA 10.2 was released in Nov 2019, we should upgrade the default toolchain to CUDA 11 now.

TODOs

[ ] Revise codebase so that clang-tidy check can pass under a CUDA 11.0 environment.
[ ] Fix gtest MultiClassesSerializationTest.GpuHist on Windows


MocoCasADiSolver output_interval property does not work on Windows

On Windows, the generated files contain all zeros.
This is likely an issue with how we compiled our copy of Ipopt.

To debug this issue, I think it's helpful to look at the changes that I think I made to CasADi's FindIPOPT.cmake: https://github.com/casadi/casadi/compare/master...stanfordnmbl:1555e79ea59cbd25d05894f2e9b351d92bacf348


Refactor code to enable reusing VM in multiple tests

Enable -DeployVMPerEachTest switch.  By default LISAv2 will reuse previous deployment for test with same setupType and overrideVmSize. If -DeployVMPerEachTest is enabled, VMs will be deployed per each test. The logic applies to both Hyper-V and Azure.
Test pass on both Hyper-V and Azure, including scenario of

DeployVMPerEachTest not enabled, multiple test contains both different setupType/overrideVmSize and same setupType/overrideVmSize
DeployVMPerEachTest enabled, multiple test contains both different setupType/overrideVmSize and same setupType/overrideVmSize

@juhlee-microsoft @ader1990 @adriansuhov could you please review this PR? I would like this PR to get merged soon. It also fixes a bug that resource group doesn't get deleted after the test complete in Azure

Hi @lizzha - I'll run some tests using your code. If I'll have questions or something goes wrong, I'll let you know. I'll come back with an update as soon as the tests are done.

@adriansuhov the behavior change is in Azure side, without this change, each test case run will provision a new VM in Azure, but with this change, if the setupType and overrideVmSize are both the same, multiple test cases will share a same VM deployment. Please double check that.
Also this PR enables above change by sharing the same logic for Azure and HyperV, it makes sure not breaking the existing functionality on HyperV.

Comments addressed, also tested. @juhlee-microsoft please review again. Thanks

@adriansuhov Here're part of the logs of running multiple tests on Azure with same setupType and overrideVmSize. The test reuses previous deployment
11/19/2018 03:02:23 : [INFO ] ** Initial and Final Kernel Logs has same content **
11/19/2018 03:02:23 : [INFO ] ICA-RG-SingleVM-liz-test-PDVW25856-636782217429-role-0: Final Kernel logs collected and Compared ..SUCCESSFULLY
11/19/2018 03:02:23 : [INFO ] Skipping cleanup of Resource Group : ICA-RG-SingleVM-liz-test-PDVW25856-636782217429.
11/19/2018 03:02:23 : [INFO ] Removing all files logs from IP : <IP_ADDRESS> PORT : 22
11/19/2018 03:02:23 : [INFO ] dos2unix: converting file ..\TestResults\2018-19-11-10-46-32-1586\BVT-ROOT-DEVICE-TIMEOUT-CHECK\runtest.sh to Unix format...
11/19/2018 03:02:23 : [INFO ] Uploading ..\TestResults\2018-19-11-10-46-32-1586\BVT-ROOT-DEVICE-TIMEOUT-CHECK\runtest.sh to azureuser : <IP_ADDRESS>, port 22 using Password authentication
......11/19/2018 03:02:30 : [INFO ] Upload Success after 1 Attempt
11/19/2018 03:02:30 : [INFO ] .\tools\plink.exe -t -pw ****** -P 22<EMAIL_ADDRESS>"echo ****** | sudo -S rm -rf *"
11/19/2018 03:02:35 : [INFO ] rm -rf * executed successfully in 5.36 seconds.
11/19/2018 03:02:35 : [INFO ] All files removed from /home/azureuser successfully. VM IP : <IP_ADDRESS> PORT : 22
11/19/2018 03:02:35 : [INFO ] ~TEST END : BVT-ROOT-DEVICE-TIMEOUT-CHECK
11/19/2018 03:02:35 : [INFO ] CURRENT - PASS    - 2
11/19/2018 03:02:35 : [INFO ] CURRENT - FAIL    - 0
11/19/2018 03:02:35 : [INFO ] CURRENT - ABORTED - 0
11/19/2018 03:02:35 : [INFO ] Removed Completed background job ID 55.
11/19/2018 03:02:35 : [INFO ] Loading the test data for BVT-HVMODULES-CHECK
11/19/2018 03:02:35 : [INFO ] ~TEST STARTED : BVT-HVMODULES-CHECK~~~~~~~~~~~~~
11/19/2018 03:02:35 : [INFO ] Starting multiple tests : BVT-HVMODULES-CHECK
11/19/2018 03:02:35 : [INFO ] Collecting ICA-RG-SingleVM-liz-test-PDVW25856-636782217429-role-0 VM Kernel Initial Logs..
11/19/2018 03:02:35 : [INFO ] Uploading .\TestResults\2018-19-11-10-46-32-1586\BVT-HVMODULES-CHECK\zgrmmo2h.yry to azure
user : <IP_ADDRESS>, port 22 using Password authentication
......11/19/2018 03:02:41 : [INFO ] Upload Success after 1 Attempt
11/19/2018 03:02:41 : [INFO ] dos2unix: converting file ..\TestResults\2018-19-11-10-46-32-1586\BVT-HVMODULES-CHECK\run
test.sh to Unix format...

Here're part of logs of running two tests in Azure with same setupType but different OverrideVmSize
11/19/2018 02:55:34 : [INFO ] Try to delete resource group ICA-RG-SingleVM-liz-test-PDVW25856-636782212133 ...
11/19/2018 02:55:34 : [INFO ] Checking if ICA-RG-SingleVM-liz-test-PDVW25856-636782212133 exists ...
11/19/2018 02:55:36 : [INFO ] Triggering : DeleteResourceGroup-ICA-RG-SingleVM-liz-test-PDVW25856-636782212133...
11/19/2018 02:55:36 : [INFO ] CleanUP Successful for ICA-RG-SingleVM-liz-test-PDVW25856-636782212133..
11/19/2018 02:55:36 : [INFO ] ~TEST END : BVT-VERIFY-DEPLOYMENT-PROVISION
11/19/2018 02:55:36 : [INFO ] CURRENT - PASS    - 1
11/19/2018 02:55:36 : [INFO ] CURRENT - FAIL    - 0
11/19/2018 02:55:36 : [INFO ] CURRENT - ABORTED - 0
11/19/2018 02:55:36 : [INFO ] DeleteResourceGroup-ICA-RG-SingleVM-liz-test-PDVW25856-636782212133 is running.
11/19/2018 02:55:36 : [INFO ] Loading the test data for BVT-ROOT-DEVICE-TIMEOUT-CHECK
11/19/2018 02:55:36 : [INFO ] ~TEST STARTED : BVT-ROOT-DEVICE-TIMEOUT-CHECK~~~~~~~~~~~~~
11/19/2018 02:55:36 : [INFO ] Starting multiple tests : BVT-ROOT-DEVICE-TIMEOUT-CHECK
11/19/2018 02:55:37 : [INFO ] SingleVM
11/19/2018 02:55:37 : [INFO ] Checking the subscription usage...
11/19/2018 02:55:38 : [INFO ] Estimating VM #1 usage.
11/19/2018 02:55:39 : [INFO ]   Current Regional Cores usage : 278 cores. Requested:1. Estimated usage=279. Max Allowed cores:868/868
11/19/2018 02:55:39 : [INFO ]   Current standardDSv2Family usage : 11 cores. Requested:1. Estimated usage=12. Max Allowed cores:700/700
11/19/2018 02:55:39 : [INFO ] Estimating storage account usage...
11/19/2018 02:55:41 : [INFO ] Current Storage Accounts usage:80. Requested:1. Estimated usage:81. Maximum allowed:200/200.
11/19/2018 02:55:42 : [INFO ] Current Public IPs usage:35. Requested: 1. Estimated usage:36. Maximum allowed: 1000.
11/19/2018 02:55:42 : [INFO ] Current VNET usage:28. Requested: 1. Estimated usage:29. Maximum allowed: 1000.
11/19/2018 02:55:42 : [INFO ] Current Security Group usage:28. Requested: 1. Estimated usage:29. Maximum allowed: 5000.
11/19/2018 02:55:42 : [INFO ] Current Load Balancer usage:9. Requested: 1. Estimated usage:10. Maximum allowed: 1000.

The PSScriptAnalyzer failure has been fixed by PR #402 , manually trigger the rebuild pass: https://microsoft.visualstudio.com/OS/_build/results?buildId=12984721&view=logs

Thank you @lizzha for the explanation & changes!

@adriansuhov did several tests in various combinations to check the cleanup/deploy.
@juhlee-microsoft please sign-off and merge this, we have no further comments

@lizzha I can not merge it because of PRChecker rule. Can you rebase master if it was resolved in #402?

@juhlee-microsoft rebased, and check pass


LDAP : anonymous with TLS certificates ?

Hi All - we are evaluating the label studio with openLDAP integration, before we do that, did go thru the article - https://labelstud.io/guide/auth_setup.html ...question mark..we are using  anonymous access (TLS enabled with certificate).. so, is there any suggestions to be included in order to achieve the anonymous access with TLS certificates to be passed ? Please provide an example on how we can include in the label studio (docker-compose or kubernetes)?  Below is an example we set in all windows+linux host (sssd.conf) ..please provide an example means provide the lines as to how to achieve anonymous access with TLS (certificate), below lines are mandatory..
ldap_uri = ldap://ldap.example.com:389
ldap_default_bind_dn = cn=sssd,dc=example,dc=com
ldap_search_base = dc=example,dc=com
ldap_id_use_start_tls = true
ldap_tls_reqcert = hard
ldap_tls_cacert = /etc/pki/tls/CA/ldap.ca

Try to use this configuration
AUTH_LDAP_ENABLED=1
AUTH_LDAP_SERVER_URI=ldap://ldap.example.com:389
AUTH_LDAP_BIND_DN=cn=sssd,dc=example,dc=com
AUTH_LDAP_USER_SEARCH_BASES=dc=example,dc=com
AUTH_LDAP_BIND_PASSWORD=""
AUTH_LDAP_START_TLS=1
AUTH_LDAP_CONNECTION_OPTIONS=OPT_X_TLS_CACERTFILE=/path/to/ca.pem;OPT_X_TLS_REQUIRE_CERT=OPT_X_TLS_HARD


Tests for project generation

[x] Make sure steps (1/6, 2/6, 3/6) in the wizard are correct.
[x] Make sure default values are correct. (title exists, prompt text exists, default groupID exists etc.)
[x] Make sure steps and values are correct after hitting the back button (difficult)
[x] Make sure input validation is working as expected.
[x] Make sure saving defaults work (have to make sure you don’t edit the current settings.json file)
[x] Make sure project downloads, contains launch.json and tasks.json in the artifactId/.vscode folder.

Testing would be easier if there was a way to fire events manually, and if there was a way to detect if a QuickInput is currently being displayed on the current window.
If not, we would need to test values before await calls that wait for user input.

Fixed with https://github.com/redhat-developer/vscode-quarkus/pull/172


some question about process 2d datasets

i found that you crop the closest person for 3d datasets if have multiple person in one image, but for 2d dataset you may crop all persons in one image. why you process these datasets differently?
in addition, the cropped image may contain another person, won't this process bring ambiguity to the network？
thank you very much!

i found that you crop the closest person for 3d datasets if have multiple person in one image, but for 2d dataset you may crop all persons in one image. why you process these datasets differently?

The cropping process is the same regardless of datasets. Even if you crop the closest person for 3d datasets, there could be other people in the cropped image. And actually MuCo, which you are referring to, does not have multiple person in one image originally. It synthesizes multiple real person images to one image using depths.

in addition, the cropped image may contain another person, won't this process bring ambiguity to the network？

That is the challenge of crowded scenes, which 3DCrowdNet resolves. Please see the paper.

i found that you crop the closest person for 3d datasets if have multiple person in one image, but for 2d dataset you may crop all persons in one image. why you process these datasets differently?

The cropping process is the same regardless of datasets. Even if you crop the closest person for 3d datasets, there could be other people in the cropped image. And actually MuCo, which you are referring to, does not have multiple person in one image originally. It synthesizes multiple real person images to one image using depths.

in addition, the cropped image may contain another person, won't this process bring ambiguity to the network？

That is the challenge of crowded scenes, which 3DCrowdNet resolves. Please see the paper.

Thanks for your reply！
I found many up-to-down methods using single person datasets for training, these can prevent ambiguity during training, which the cropped image does not contain another person. while for inference, the cropped image often contain other persons, but it often regress the right person's smpl parameters. Does this mean that if other people are included in the cropped image during training, it will bring ambiguity to the network and make it difficult for training.
In 3DCrowdNet, the cropped image contain other persons even for training. but add 2d robust pose heatmap to resolve the ambiguity. Is my understanding correct？

Does this mean that if other people are included in the cropped image during training, it will bring ambiguity to the network and make it difficult for training.

No. I think you are confused with how deep learning works. Given accurate ground truth, a neural network becomes robust to the ambiguity during training. Then, the neural network performs better on those ambiguous input in test time.


Merge of PR #107, Allow Telegraf to output data to multiple locations beyond InfluxDB, such as Riemann or Kafka

\cc @jipperinbham @pauldix @toddboom
This PR supports Telegraf outputting data to multiple locations beyond just InfluxDB. This is a breaking change that requires some changes to the .toml Telegraf configuration file, basically users will need to change their influxdb entry from this:
[influxdb]
url = "http://localhost:8086"
database = "telegraf"

to this
[outputs]
[outputs.influxdb]
url = "http://localhost:8086"
database = "telegraf"

You can find more details in @jipperinbham's original PR here: https://github.com/influxdb/telegraf/pull/107

@sparrc thanks for making this separate PR to handle the conflicts
Everything looks good to me, just had the one comment.

Commented on the other one. The Output interface should have a Close() error function so that Telegraf can be shut down cleanly. +1 on the breaking change

@jipperinbham @pauldix
Added a commit with a Close() function, although it appears that we can't really do anything with the (I can make a separate PR on influxdb/influxdb to add a Close() function to the client)


Valid link treated as invalid

Bug Description
Using a valid link (e.g. http://www.google.com) still triggers invalid link notification.

Quick debugging showed that in case of google.com, the response code is 403 with the following message:
We're sorry...</h1><p>... but your computer or network may be sending automated queries. To protect our users, we can't process your request right now.
Tested with some other links and it seems to work fine generally.
Q: What's the expected behavior in case of the forbidden error message as in the case of Google? Should we display something else in the snackbar instead of just "Invalid link"? Thoughts?
cc @o-fernandez @swissspidy

As a user I don't expect any message in that case. Even If I add a URL like https://thiswebsitedoesnotexist.org on purpose.
Especially since the URL is only fetched to get metadata for a page (i.e. title, image, description).
If I add an inexistent URL, the metadata will be simply empty.
The only place where I expect an error message is if I pass a malformed URL or an empty string.

Yeah, I wonder if we just have basic validation logic instead of actually making requests/testing the URLs. Is there a simple library/rule set we can use to check for valid URL structure?

@o-fernandez Note that we aren‘t requesting the URLs to see if they‘re valid. We make requests to retrieve page metadata like the title. Some basic validation logic is done in that process.

@swissspidy that's great. Then how come google.com was showing as invalid, per the original bug report? Or did I misunderstand the bug?

Because the site blocks such automated requests like ours. We just need to catch such cases better / not treat them as errors.

Got it, yes, my bad in how I phrased the prior comment. Agree that we should not show the user "invalid" link in such cases.

Seems like a PHP-side solution is needed here. I'd see if we're caching the requests and look more into the user agents used etc. Nevertheless, I think we should reassign and bring down from P1 to P2.

Moving discussion for the PR, I think that if the URL doesn't return a 200 status, it should return an error. Other states should maybe just return an empty title, description and should not cache.

if the URL doesn't return a 200 status, it should return an error. Other states should maybe just return an empty title, description and should not cache.

So what is it, error or empty data on other states?
The case reported here is that a 403 status should not return an error, which is what #2254 handles.

See #2254 for instructions.

When I enter an invalid URL, I see the warning message:

If I navigate away from element and REselect it, the message is no longer there:

Should this message be maintained? Please advise if ok=as-is

@csossi Can you test again? The error message should now persist.

Not seeing an error message at all now:

@csossi Well, that's a different URL 🙂 I've made the URL test a bit more restrictive now, PTAL. Try with something simple like just http://.
I'll create a follow-up ticket about making the check more robust for other cases. For now, the focus is on persistence and appearance of the error message.

@csossi Another thing to verify: the fields for link description & icon should not be displayed in case of an error.

Verified in QA:

I'm able to enter in a garbage url and not getting a error message


Multiplying by 100 twice again?

https://github.com/NebraLtd/hm-diag/blob/992d3ef6ddb8bc7faa4ceaaa402b555d1bfc78f6/diagnostics-program/main.py#L193-L194
I think we are multiplying by 100 twice again here. This was fixed in #69 / #70 but the error seems to have been reintroduced?

Think this was just in different places. @robputt is tackling rewriting the repo, so let's sort that out.

@vpetersson ah ok cool just wanted to flag it so it doesn't get reproduced going forward

Yep. We will should have unittests to cover that.

Fixed in https://github.com/NebraLtd/hm-diag/pull/81

Closed by #92


Fix the incorrect score calculation in Tetris

The score in Tetris is displaying hex values and is also increasing too fast. This is likely caused by the DAA instruction implementation.

I fixed this in my gameboy emulator just today actually. Was because I wasn't handling the memwrite to ROM properly. The game writes to 0x2000 (presumably to activate ram) and if you allow it to write to the ROM at that point you get awarded crazy scores for lines.

@tswilliamson Thank you! It works now.


Container List empty

I'm not sure what happened but recently in my Kitematic, the container list is always empty no matter how many containers are running in the virtual machine.
Is this a known issue?
I'm using the latest Kitematic release and on Mac.

It is a known issue a PR is pending for it - Closing in favor of:
https://github.com/docker/kitematic/issues/1355


[distribution] Revert 64ccbe1 and use JDK11 for snapshots GH-2117

Reverts part of the https://github.com/javalin/javalin/commit/64ccbe17724261cb38fa59dabb0d4131f7e1449d
Related to https://github.com/javalin/javalin/issues/2117

Be aware that this build will fail, see: https://github.com/javalin/javalin/actions/runs/7944870197
The compiled templates need jdk17

Speaking about the increasing complexity of the core repo lately be like

🥲

Indeed it did: https://github.com/javalin/javalin/actions/runs/8011382210/job/21884487304

I've checked the cause, and it's not even about compiled templates, but simply the fact that JTE 3.x dropped support for older versions of Java at all

https://github.com/casid/jte/releases/tag/3.0.0

So we're basically blocked in this repo by this. 2 solutions:

Downgrade JTE from 3.x to 2.x
Release snapshots from JDK17

Neither is fun, but downgrading JTE is the least not fun...

https://github.com/javalin/javalin/issues/2124


Updates to beautify

Flow is exactly the same. Mostly focused on style and using the night theme instead of black since the font is prettier. I'd also recommend solarized if you're looking for another nice one, but the colors look good now.

I hope you dig the updates @lukebakken! Enjoy the presentation and the event.

Awesome, thanks!


[UI improvement] /officer - show description for 911 call

Describe the feature
Now officer can see call description only in modal by click on button.

Propose to show description in 911 calls table on page.

Trim text, if description is too long, make column fixed width and add popup with full description text.

Additional Context
No response

What do you think about a "call title"? I feel like showing the description within the table will remove quite some valuable space. So instead of showing the description, it'd show the call title

Call title? What is it?
I see Caller field in call modal. You talk about that?

it would be a new field, it would be like the description, but shorter.

This new field can/should be filled by user or it will be filled automatically?
Like Description field will be trimmed to 20 chars for example?

Yeah, why not both 😅

The new field that the user will have to fill in is extra work and a waste of time.
Can you technically display the description in the call list shortened at once? If the officer needs to see the full description, he can go to the call card.


ImportError: cannot import name 'FileReader' from 'robot.utils'

I get this kind of error with new version v1.1:
ImportError: cannot import name 'FileReader' from 'robot.utils' (/root/project/venv/lib/python3.7/site-packages/robot/utils/__init__.py)

Not sure if this related to virtualenv, but previous version worked fine.

What version of robot are you using? rflint 1.1 requires robot framework
3.2 (though I realize I didn't specify that anywhere)
On Wed, Jun 3, 2020 at 7:32 AM Jussi Vatjus-Anttila <
<EMAIL_ADDRESS>wrote:

I get this kind of error with new version v1.1:
ImportError: cannot import name 'FileReader' from 'robot.utils' (/root/project/venv/lib/python3.7/site-packages/robot/utils/init.py)
Not sure if this related to virtualenv, but previous version worked fine.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/boakley/robotframework-lint/issues/76, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAISMYSI5XCQ6QNIZIE5WGLRUY7ERANCNFSM4NRT65RA
.

That was the reason. I was using previous robot version. Please add some notes about that.


Moving builder to local function, updating the changefeed processor version to <IP_ADDRESS>

#sign-off

Creating a Demo directory and will add more demos there.


PythonCall test works but not using PythonCall on Windows

I installed PythonCall,jl version 0.9.12 under Julia version 1.8.5. On my Windows machine, the usual test of PythonCall passes without any problems. However, when I run using PythonCall, it never returns the prompt and hangs there. This does not happen on my Mac's. I am wondering what's wrong under Windows. I also have PyCall.jl on my Windows, which works perfectly.
Thanks for your help!!

In my case, regardless of the simple REPL or vscode, it hangs. I found out that the similar things happen for CondaPkg.jl, which I just reported: https://github.com/cjdoris/CondaPkg.jl/issues/81

After removing PythonCall.jl, wiping out ~/.julia/environments/v1.8/.CondaPkg folder and the file CondaPkg.toml in that folder, and finally adding back PythonCall.jl, solved my problem as I also posted at https://github.com/cjdoris/CondaPkg.jl/issues/81


Fix missing ListKW global instance

Fixes missing ListKW Kategory global instance passing ListKW.traverse() explicitly as parameter of sequence
Bumps Kategory annotations version to 0.3.7

👀 @raulraja @pakoito

When I checked on kategory ListKW didn't implement GlobalInstance, Maybe that's the subjacent cause. I don't know what the state of codegen w/ GlobalInstance is at the moment, what's the trigger for it to get added.

it gets added when you invoke one of the instances, I'm not sure this fixes the issue. I'll do some more investigation

Addressed here https://github.com/kategory/ank/pull/12 but no need to init on top, though this approach is also valid


Feature/ui kit modals

Overview
Adjusting the banners to fit the new design in the UI kit.

Additionally on mobile position and padding of the mobile relative to the screen isnt consistent. Sometimes it has full width, sometimes some padding, others there is the desktop 40px padding at the top.


Ruby format strings

Hello,
when packaging rhis ruby gem for Fedora I have found that the checking of the format string vulnerabilities is forcefully disabled here and that there is actually one potential format string issue.
This makes the compilation on Fedora fail (and most probably on Ubuntu and Debian who have similar hardening rules) .
Best regards
Michal Ambroz

Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.
After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

User @xambroz isn't covered by a CLA. They will need to complete the form at https://identity.linuxfoundation.org/projects/cncf

Regards,
CLA GitHub bot

I have signed the CLA now.

This mostly looks good to me, and I think it's a good change to have, but before merging I'd like to get a little more background:
When you experience the compilation error, how were you building the package? E.g., were you invoking rake gem:native or rake native gem from the root of the github repo, or were installing a source package (e.g., gem install grpc --platform ruby)?
Also, what compiler and version were you using?
I'd like to know the cause here a little better to see if we can put some sort of portability test on this.

Hello,
I was trying to package the gem to be installed as RPM for Fedora - using
the gem2rpm.
I was doing something like:
gem2rpm --fetch grpc > rubygem-grpc.spec
rpmbuild -ba rubygem-grpc.spec
As part of the build it is called:
export STRIP=true GRPC_CONFIG=dbg
gem build %{gem_name}.gemspec
CFLAGS is exported by the build system to include the optimalization and
options for the hardening.


Only one node serve proper certificate

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
When setting up Traefik on Kubernetes as DaemonSet...
What did you see instead?
only one node from the set serves the issued certificate, while others serves Traefik default ones.
What version of Traefik are you using?
2.10.5
What is your environment & configuration?
apiVersion: helm.cattle.io/v1
kind: HelmChartConfig
metadata:
  name: traefik
  namespace: kube-system
spec:
  valuesContent: |-
    deployment:
      kind: Deployment
    persistence:
      enabled: true
      storageClass: longhorn
      accessMode: ReadWriteMany
    providers:
      kubernetesCRD:
        enabled: true
        allowCrossNamespace: true
        allowExternalNameServices: true
    certResolvers:
      letsencrypt:
        email<EMAIL_ADDRESS>        tlsChallenge: true
        storage: /data/acme.json
    ports:
      websecure:
        tls:
          certResolver: letsencrypt

Add more configuration information here.
If applicable, please paste the log output in DEBUG level
No response

After restarting in the daemon set mode, when acme.json is populated with proper cerificates, All pods serves correct certs. This most likely indicates that acme config is not properly reloaded after certificate being issued.

Hello @majkrzak,
Thanks for your interest in Traefik!
Multiple instances are not "supported" with ACME, as mentioned in the documentation: https://doc.traefik.io/traefik/providers/kubernetes-ingress/#letsencrypt-support-with-the-ingress-provider.
I'm marking this issue as a question accordingly.


Create model architecture in model.py file

This file will expose a method:
def create_model_for_training(.....):
# ... and return the model architecture as below:
  return input, output, loss_function, optimizer ...

This will be used by the tl_trainer.py node. Having this as a separate file allows the architecture to be implemented independently from the trainer.

I suggest pretrained SSD_GRAPH or FASTER_RCNN. In this case, we have only to fine-tune the existing model.

Without any fine-tuning I was able to detect traffic light contours on the image in most cases.

The most elegant method is to fine-tune the model, and the CNN will classifier the traffic lights's color too.


[Objc] add @objc row selection & deselection methods using Int instea…

…d of Index
@kevin-hirsch Just some convenience accessors since the Swift typedef for Index isn't visible to Objective-C.  I can easily remove the pragma mark if you dislike it.

Thanks a lot for contributing to this project! 🙏
I've merged your request ✅


is it necessary to use #[instrument]?

I don't get any traces from the server layer with latest tonic (trace export of other things in my app work wel). Do I additionally need to use #[instrument(skip_all)] similar as seen in the exampe?

no you use #|instrument(...)] to create child span. you could have multiple causes, depending on how you configure it. Can you share more information (configuration code, OTEL_... env variable, log level, startup logs) or create a minimal sample reproducing your case?

Thanks for the troubleshoot section. I still cannot get it working but maybe the info from the first answer below can help.
check you only have a single version of opentelemtry (could be part of your CI/build), use cargo-deny or cargo tree

% cargo tree -i opentelemetry_api
error: package ID specification opentelemetry_api did not match any packages

    Did you mean `opentelemetry_sdk`?

I checked another thing:
cargo tree | grep opentelemetry
│   ├── opentelemetry v0.24.0
│   ├── opentelemetry-http v0.13.0
│   │   ├── opentelemetry v0.24.0 (*)
│   ├── opentelemetry-otlp v0.17.0
│   │   ├── opentelemetry v0.24.0 (*)
│   │   ├── opentelemetry-http v0.13.0 (*)
│   │   ├── opentelemetry-proto v0.7.0
│   │   │   ├── opentelemetry v0.24.0 (*)
│   │   │   ├── opentelemetry_sdk v0.24.1
│   │   │   │   ├── opentelemetry v0.24.0 (*)
│   │   ├── opentelemetry_sdk v0.24.1 (*)
│   ├── opentelemetry-semantic-conventions v0.16.0
│   ├── opentelemetry_sdk v0.24.1 (*)
│   ├── tracing-opentelemetry v0.25.0
│   │   ├── opentelemetry v0.24.0 (*)
│   │   ├── opentelemetry_sdk v0.24.1 (*)
├── tonic-tracing-opentelemetry v0.21.1
│   ├── opentelemetry v0.25.0
│   ├── tracing-opentelemetry v0.26.0
│   │   ├── opentelemetry v0.25.0 (*)
│   │   ├── opentelemetry_sdk v0.25.0
│   │   │   ├── opentelemetry v0.25.0 (*)
│   └── tracing-opentelemetry-instrumentation-sdk v0.21.0
│       ├── opentelemetry v0.25.0 (*)
│       └── tracing-opentelemetry v0.26.0 (*)

check the code of your exporter and the integration with tracing (as subscriber's layer)
This works as other spans in my app are exported.
/// Creates an default OpenTelemetry OTLP tracer
///
/// Sets service name and version according to the project's `Cargo.toml`
pub fn create_otlp_tracer(
    config: InitOtlpTracerInput,
) -> Result<opentelemetry_sdk::trace::TracerProvider, TraceError> {
    global::set_text_map_propagator(TraceContextPropagator::new());
    let mut map = HashMap::new();
    map.insert(
        "Authorization".to_string(),
        format!("Api-Token {}", config.auth_token),
    );

    let mut resource = Resource::new(vec![
        KeyValue::new(semcov::resource::SERVICE_NAME, config.service_name),
        KeyValue::new(semcov::resource::SERVICE_VERSION, config.service_version),
    ]);

    resource = resource.merge(&read_dt_metadata());

    opentelemetry_otlp::new_pipeline()
        .tracing()
        .with_exporter(
            opentelemetry_otlp::new_exporter()
                .http()
                .with_endpoint(config.endpoint)
                .with_headers(map),
        )
        .with_trace_config(
            opentelemetry_sdk::trace::Config::default()
                .with_resource(resource)
                .with_sampler(opentelemetry_sdk::trace::Sampler::AlwaysOn),
        )
        .install_batch(opentelemetry_sdk::runtime::Tokio)
}

check that log target otel::tracing enable log level trace (or info if you use tracing_level_info feature) to generate span to send to opentelemetry collector.
I used the info feature flag and also set my trace level to trace. still no traces from this crate.

Oh I found it to work now with the latest version

You have 2 version of opentelemetry (sorry opentelemery_api was before some refactor in opentelemetry)


 Expected String but got Null. Path 'facebook'.
Invalid type. Expected String but got Null. Path 'twitter'.
Invalid type. Expected String but got Null. Path 'linkedin'.

rd/eduardo-pires.yml

[Error] Unable to load file: eduardo-pires.yml via processor: Profile: eduardo-pires.yml is not in supported format: (Line: 18, Col: 18, Idx: 588) - (Line: 18, Col: 19, Idx: 589): While parsing a block mapping, did not find expected key.

[Error] dipti-chhatrapati.yml is not in supported format: (Line: 18, Col: 18, Idx: 608) - (Line: 18, Col: 19, Idx: 609): While parsing a block mapping, did not find expected key.

For more details, please refer to the build report.
 Expected String but got Null. Path 'facebook'.
Invalid type. Expected String but got Null. Path 'twitter'.
Invalid type. Expected String but got Null.

[Error] joel-oleson.yml is not in supported format: (Line: 18, Col: 18, Idx: 578) - (Line: 18, Col: 19, Idx: 579): While parsing a block mapping, did not find expected key.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


TypeError: 'int' object is not iterable

How to fix this problem?
When I do "python make_stl_data1.py", I got a errors. The error is:

root@hu-N551JW:/home/hu/dec-master/dec# python make_stl_data1.py
Building HOG feature extractor...
running build
running install
running build
running install_egg_info
Writing /usr/local/lib/python2.7/dist-packages/features-0.0.0.egg-info
Preparing stl data. This could take a while...
Exception in thread Thread-1:
Traceback (most recent call last):
File "/usr/lib/python2.7/threading.py", line 810, in __bootstrap_inner
self.run()
File "/usr/lib/python2.7/threading.py", line 763, in run
self.__target(*self.__args, **self.__kwargs)
File "/usr/lib/python2.7/multiprocessing/pool.py", line 325, in _handle_workers
while thread._state == RUN or (pool._cache and thread._state != TERMINATE):
AttributeError: '_MainThread' object has no attribute '_state'

How did you fix this issue?

@bhavikajalli I took some time to fix it, but I have deleted my dec.py. So, totally sorry to response you. I remember it's the hog shape not work.

@bhavikajalli it's senseless to waste time to fix the problem. Just keep you time to understand how the code running, if other datasets can work. Because the code of author is kind of mess.


insert_final_newline into editorconfig file

My .editorconfig file looks like this:
root = true

[*]
charset = utf-8
indent_style = space
indent_size = 4
end_of_line = lf
trim_trailing_whitespace = true
insert_final_newline = true

[package.json]
indent_size = 2

If I save it, a final newline is not added.

@glen-84
Make sure you're using latest version of extension. This feature seems to work fine:
https://github.com/Microsoft/vscode-editorconfig/commit/baba53fffb661f2f9234b26ee626ac45aadda0e0

I think it's because the file isn't saved yet, so it has no editorconfig configuration to read. This is a, "What came first? The chicken or the egg?" scenario.

@peterblazejewicz I'm using the latest version, and I just tested it again and it still doesn't work. Please note that I'm referring to saving the editorconfig file itself, not another file.
@jedmao If I save the file with insert_final_newline = true and then edit something and save it again, it still doesn't insert the newline.

@glen-84 I am able to reproduce this issue on my fork, editorconfig/editorconfig-vscode as well.

@glen-84 this is fixed in https://github.com/editorconfig/editorconfig-vscode/issues/9. Use the forked extension and it should work for you. You can thank @SamVerschueren for the fix.

@jedmao Cool, thanks guys.
I don't understand the relationship between this extension and your fork though – is this one going to be deprecated, or what is the story?

You can think as it as the same relationship as NodeJS and io.js. Development pace and response in this repository was just to slow. Look at the open PR's. You can't blame Microsoft for it, they are working hard to make VS Code even more awesome. So as for now, the fork is being developed and released separately from this one.

Also, the fork is on the official EditorConfig GitHub group and published accordingly. I would imagine that the Microsoft one will fade out of popularity as we continue to have faster iteration cycles on the fork.

Okay, thanks. =)

Is not adding final new lined for filename beginning with '.' eg .gitignore

Are you using the plugin from EditorConfig instead of this one? This one is deprecated...

the link on the extension page brought me here!

@SteveALee then I believe you are using the deprecated one. See here: https://github.com/editorconfig/editorconfig-vscode/blob/master/package.json#L22

thanks - I raised an issue there.
I see I clicked on the very unhelpful forked from link here
https://marketplace.visualstudio.com/items?itemName=EditorConfig.EditorConfig
It has more weight than the support link over on the right


Resolve URL to original route & params

Hello,
is there a way to resolve the original route and params of a URL?
i.e. I have the url rule 'read/<id>'=>'site/read' I want to be able to resolve the URL /read/10 to the original route and param ['site/read', 'id'=>10]
is this possible?
I can do this to the current request using Yii::$app->getRequest()->resolve() but I need to be able to do this for any provided URL.
Thanks.

Yes. You can create Request instance, set parameters needed and pass it to Yii::$app->getUrlManager()->parseRequest($request);.

$request = new \yii\web\Request();
$request->setPathInfo('read/10');
$routeAndParams = Yii::$app->getUrlManager()->parseRequest($request);

did the trick for me.
many thanks @samdark