Running without Docker

Hi Authors,
Thanks for building this environment, that's a really great contribution.
I was wondering if it's possible to extend the codebase and either get rid of the dependency on Docker or make it compatible with other container technologies (such as Apptainer)?
The reason I'm asking is that Docker is not available on the Canadian cluster (and probably on some other clusters too) because of its security risks (https://docs.alliancecan.ca/wiki/Apptainer#Other_Linux_container_technologies).
Thank you.

Hi @evgenii-nikishin thanks for your comment and kind words! We really appreciate it :)
Thanks for pointing out this concern, that is a good point and stringent barrier of entry that our team has not considered so far.
I think between the two options you presented, my guess is that making it compatible with other container technologies (i.e. Apptainer) would likely require less engineering. I have put this down as a to-do and will be sure to look into it, but I think it will take me some time to deliver on this (~1 month).
On the other hand, it seems to me that removing the docker dependency and containerized execution in general means that you'd be interested in running InterCode directly on a machine's a code interpreter or executable? I think this would be possible, but as a word of caution, it may subject the machine to LLM code generations that might be cause irreversible modifications to the host system (i.e. rm -rf ...).
Between these two choices, do you have a preference for one or the other? I personally think the first option (compatibility with other container tech.) is better, but I'll prioritize whatever you prefer!

Thank you for the quick response! I think making it compatible with Apptainer should be a better alternative.
As for removing the containerized execution altogether, I wouldn't want that as indeed it might lead to unsafe behavior; I was just thinking that maybe there are other ways to create a sandbox without docker.

Gotcha, sounds good! I'll give it a shot and see how things go - will do my best to make it happen! I'll leave this open and ping here when I have an update.


Fix "Remove duplicate key in dictionary" issue

This pull request automatically fixes all occurrences of the following issue:
Issue type: Remove duplicate key in dictionary
Issue details: https://www.quantifiedcode.com/app/project/gh:jochym:phonopy?groups=code_patterns/%3A7r1WUUmA
To adjust the commit message or the actual code changes, just rebase or cherry-pick the commits.
For questions or feedback reach out to<EMAIL_ADDRESS>Legal note: We won't claim any copyrights on the code changes.
Cheers,
Cody - Your code quality bot

Indication of other mistake. Corrected in aaa3a55. Closing.


Module 0 not created

We need to add Module 0 from
Module 1: Pre-digitization Curation (Staging)

Demo issue. Closing.


Unclear available operator output properties

Before submitting the issue

[X] I have checked for Compatibility issues
[X] I have searched among the existing issues
[X] I am using a Python virtual environment

Description of the bug
dpf.operators.mesh.node_coordinates() does not output node coordinate values. After creating the operator and inputting the mesh:

coords_op.outputs
Available outputs:

 -   coordinates
 -   coordinates

coords_op.outputs.coordinates()
AttributeError: 'Outputs' object has no attribute 'coordinates'
coords_op.outputs.fields_container()
AttributeError: 'Outputs' object has no attribute 'fields_container'

Steps To Reproduce
from ansys.dpf import core as dpf
data_sources = dpf.DataSources()
data_sources.set_result_file_path(fileName))
model = dpf.Model(data_sources)
coords_op = dpf.operators.mesh.node_coordinates()
coords_op.inputs.mesh.connect(model.metadata.meshed_region)
Which Operating System are you using?
MacOS
Which Python version are you using?
3.7
Installed packages
ansys-dpf-core==0.5.2
ansys-dpf-gate==0.2.1
ansys-grpc-dpf==0.6.0
cachetools==5.2.0
certifi==2022.6.15
charset-normalizer==2.1.0
future==0.18.2
google-api-core==2.8.2
google-api-python-client==2.54.0
google-auth==2.9.1
google-auth-httplib2==0.1.0
googleapis-common-protos==1.56.4
grpcio==1.47.0
httplib2==0.20.4
idna==3.3
modeid==2.5.1
numpy==1.21.6
packaging==21.3
pandas==1.3.5
progressbar2==4.0.0
protobuf==3.20.1
psutil==5.9.1
pyasn1==0.4.8
pyasn1-modules==0.2.8
pyparsing==3.0.9
python-dateutil==2.8.2
python-utils==3.3.3
pytz==2022.1
PyYAML==6.0
requests==2.28.1
rsa==4.9
scipy==1.7.3
six==1.16.0
tqdm==4.64.1
uritemplate==4.1.1
urllib3==1.26.10

I have had this same issue on Windows.

I figured it out:
mesh_coord_op = dpf.operators.mesh.node_coordinates(mesh=result_mesh)
mesh_coords = mesh_coord_op.outputs.coordinates_as_field.get_data()
Is there a way to update the documentation to say coordinates_as_field rather than coordinates?

Hi @nick-hou,
Yes indeed it is not clear.
When an output can have several types, such as here a Field or a FieldsContainer, then the output name can be extended with the type name like you saw.
We need to improve the generated doc on our side indeed.
Thanks for raising this issue!


Ability to return parameter IDs in a complex structure control

Values returned from complex structures may be hard to identify which ID maps to what value.

We need a way to allow the parameter IDs to be returned with the values, for example:
"data": {
    "form": {
      "structuretableReadonlyColumnStartValue": [
        {
          "field": "Cholesterol",
          "structuretable_sort_order_readonly_int": 5,
          "structuretable_sort_order": "Ascending"
        },
        {
          "field": "BP",
          "structuretable_sort_order_readonly_int": 6,
          "structuretable_sort_order": "Ascending"
        }
      ]
    }

This is implemented with https://github.com/elyra-ai/canvas/pull/169 and https://github.com/elyra-ai/pipeline-schemas/pull/25
A new type property has been added to the complex type definition schema. This defaults to array and can be set to object. The parameters will be read in and passed out in this format defined.


Source games don't go fullscreen

When launching a Source engine game with fullscreen enabled in the game settings, it does not launch fullscreen.

This does not happen in other games that are not based on the Source engine, I've tested OMORI and Unturned and both launch fullscreen (although in Unturned waybar acts a little funky).

Current workaround is using the keycombo that activates fullscreen or maybe setting a window rule for every Source game.

it's the game's fault. Not much else to say. Maybe also X11's fault.

Oh lol then I'm the one acting funky.
Guess I'll have to create an issue at the CS:GO repo.
Also thanks for creating Hyprland, it works more intuitively than any window manager I've ever used.

Guess I'll have to create an issue at the CS:GO repo.

I'd advise against. It doesn't matter + it's prolly hyprland's fault.

It's not only source games. Running any game through gamescope --fullscreen results in the same issue. Gamescope launches in windowed mode but at the resolution of the monitor.

afaik gamescope just doesn't pass those events.

I have the same issue with Hearts of Iron IV, but if you do some magic to launch the game using the system SDL2 library instead of the bundled one and set SDL_VIDEODRIVER=wayland the game launches in fullscreen mode, so it's probably an XWayland issue

Tested gamescope --fullscreen on sway and kde plasma wayland. It seems to work perfectly, so it's a Hyprland/XWayland issue.


Update LTS 8.17

We should consider removing the dependency of this library and use the base library instead. For now, this will allow us to use it with latest lts from stackage.
I've already built the library and everything is properly working. (we don't have CI for this fork)

Thanks. We're using this version of the library for the website, we should consider opening a pull request instead to extend the base library.


Configurations file added into conf.d or as a secret not being read

When trying to test the Docker version, I tested the conf.d setup for my MOTD as well as setting up opers.conf with a secret. Both were setup following the directions in the README.md file. The MOTD redirection to my custom one failed to be seen and my opers.conf file was never read. In digging through the running container, I found no reference to conf.d in any config file or script. I also found the only secret being checked for is the SSL certificates and that is in the entrypoint.sh script. Can someone check to see if a connection got severed somewhere in the development or if I am doing something wrong.

I did some more digging and the config.sh script has references to conf.d and any conf file in /run/secrets. Looks like the issue is that these calls to add the configs are not being placed in inspircd.conf which causes it not to do anything with those files.

It looks like after a few updates, this is fixed.


Can I use tfgo for tensorflow 2.11.x?

Hi @galeone,
I would like to try tfgo in a production setup with tensorflow 2.11.0. I found that your installation guide is pointing to tensorflow version 2.9.1. Any suggestion highly appreciated.

It shouldn't be a problem. The C library is compatible, but there could be
some operations that haven't been mapped to the Go bindings.
In a few days I'll upgrade the TensorFlow' fork go-gettable to TensorFlow
2.11 (or whatever is the latest version) so we can get sure that everything
works fine and the support is complete
On Sat, Feb 18, 2023, 00:54 Tanvir Shahid @.***> wrote:

Hi @galeone https://github.com/galeone,
I would like to try tfgo in a production setup with tensorflow 2.11.0. I
found that your installation guide is pointing to tensorflow version 2.9.1.
Any suggestion highly appreciated.
—
Reply to this email directly, view it on GitHub
https://github.com/galeone/tfgo/issues/81, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAJSDBXNRAAB5TEXIGG5U3WYAFTNANCNFSM6AAAAAAU77S5MM
.
You are receiving this because you were mentioned.Message ID:
@.***>

@galeone Thanks for your prompt reply. The current stable version of TensorFlow is 2.11.0.
It would be really helpful if you release the TensorFlow fork go-gettable to TensorFlow 2.11.0. I will wait for the release and will test it intensively. For the time being I am going to close this issue then.
many thanks


Simplify convert and avoid dirty worktree

Description
When running make convert, it builds oapifixer in the tools/oapifixer/ directory. This unnecessarily dirties the worktree.
This change updates the Makefile to use a go run invocation of the tool so that the executable isn't written to the worktree at all.
Type of change

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

How Has This Been Tested?
make convert was run with and without this change to ensure that there was no difference in the generated output.
Checklist (check all applicable):

[ ] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard to understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

@TimonOmsk do you have the rights to handle the merge?


chore: add shellcheck linting

Problem

Bash is really really hard.
Solution
Add the https://www.shellcheck.net/ linter to our code and CI
Currently I only add it to the nix setup. We can defer merging this until everyone is on Nix if we want.
Closes #548

Would be nice to have this merged just to run it in CI

I think this PR is orthogonal to nix, @d4hines just picked nix because it was the easiest way to get it into CI since we have the infra for it already


Server respond with 404 for some images in Osekai Badges

https://osekai.net/badges/

meow

this is not our fault, osu! refuses to load so many images at once. make sure you're using @1x and not @2x

Any thoughts on splitting it into pages?

would prefer not to but might be worth looking into


An optimization of the network communication architecture for improving the running of hyperledger fabric on kubernetes, where the random port of the pod affects the node's p2p communication

Current Status
When running Hyperledger Fabric with Kubernetes, the port of the node's pod is random, while the ports of the Orderer and Peer anchor nodes are written in the genesis block gennis.block. When the node is restarted, the port will change, which will cause the node to be unable to establish p2p communication.
Expected
In order to improve the orchestration and deployment capabilities of Hyperledger Fabric on kubernetes, it is recommended to optimize the network communication architecture.
Solution
Introduce Zookeeper as a decentralized registration center, and each organization maintains a registration center. The specific design is as follows:

Allow each organization to connect to its own Zookeeper cluster, ZNode=/hosts/{domain name}
The {domain name, node ip, external random port} of the blockchain node (Orderer, peer, CA) is stored on the Zookeeper cluster
When the blockchain node (Orderer, peer, CA) starts, it immediately reports the {domain name, node ip, external random port} of this node to ZNode
When the blockchain node (Orderer, peer, CA) starts, it starts to monitor all changes in /hosts/{domain name} on ZNode.
(a) When the data changes, update the local /etc/hosts
(b) When the data changes, update the port number for the node to establish a connection with the peer node

Please let us know if you plan to work on this.
Yes!

And where would Zookeeper run? If it runs in K8S you have the same problem, and if it runs outside of it, then why can't you run Fabric outside of Kubernetes too?

Can't you specify the service port when using Kubernetes and/or use an ingress with a static port?


Avoid crash when app omits config

gave up due to inexplicable unsigned commits appearing in this branch after rebasing


Please continue dev

Please continue dev and powerup chrome OS

Seconded. Or at the very least let us know what Google recommend we use as an alternative with a similar feature set.

maybe opensource community should get into it, anyone interested?

Let's do it.
On Wed, May 11, 2016, 6:51 AM Prateek Gupta<EMAIL_ADDRESS>wrote:

maybe opensource community should get into it, anyone interested?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/GoogleChrome/chromedeveditor/issues/4061#issuecomment-218465086

--
Sent from my Maz–Ruh Legitness (obviously not an iPhone).


Add Getting Starter secction to README

README Update
Please add a__getting started__ section to the README so folks know how to start with this project.

Added Getting Started section in README file.


Access component

This PR removed IRenderedComponent<TComponent>.Instance method in favour of IRenderedComponent<TComponent>.AccessComponent.
On top, I removed cut.InvokeAsync as without a direct instance (and AccessInstance wrapped in InvokeAsync) there should be no need anymore.

Obviously the documentation has to be updated.

Open tasks:

[] WaitForAssertionAsync also should have an overload with TComponent (like WaitForStateAsync)


Can't delete Pinniped concierge using Yamls

What happened?
With previous versions of Pinniped, e.g. 0.7.0 it was possible to deploy and delete Pinniped concierge just using the Yaml files. For example:
# Deploy
kubectl apply -f https://get.pinniped.dev/v0.7.0/install-pinniped-concierge-crds.yaml
kubectl apply -f https://get.pinniped.dev/v0.7.0/install-pinniped-concierge.yaml

# Delete
kubectl delete -f https://get.pinniped.dev/v0.7.0/install-pinniped-concierge.yaml

Now, at least with version 0.18.0 (haven't tried previous ones), there is the error:
Error from server (NotFound): error when deleting "https://get.pinniped.dev/v0.18.0/install-pinniped-concierge.yaml": the server could not find the requested resource (delete credentialissuers.config.concierge.pinniped.dev pinniped-concierge-config)
Error from server (NotFound): error when deleting "https://get.pinniped.dev/v0.18.0/install-pinniped-concierge.yaml": secrets "pinniped-concierge-impersonation-proxy" not found

Full log:
customresourcedefinition.apiextensions.k8s.io "jwtauthenticators.authentication.concierge.pinniped.dev" deleted
customresourcedefinition.apiextensions.k8s.io "webhookauthenticators.authentication.concierge.pinniped.dev" deleted
customresourcedefinition.apiextensions.k8s.io "credentialissuers.config.concierge.pinniped.dev" deleted
namespace "pinniped-concierge" deleted
serviceaccount "pinniped-concierge" deleted
serviceaccount "pinniped-concierge-kube-cert-agent" deleted
serviceaccount "pinniped-concierge-impersonation-proxy" deleted
configmap "pinniped-concierge-config" deleted
deployment.apps "pinniped-concierge" deleted
service "pinniped-concierge-api" deleted
service "pinniped-concierge-proxy" deleted
apiservice.apiregistration.k8s.io "v1alpha1.login.concierge.pinniped.dev" deleted
apiservice.apiregistration.k8s.io "v1alpha1.identity.concierge.pinniped.dev" deleted
clusterrole.rbac.authorization.k8s.io "pinniped-concierge-aggregated-api-server" deleted
clusterrolebinding.rbac.authorization.k8s.io "pinniped-concierge-aggregated-api-server" deleted
clusterrole.rbac.authorization.k8s.io "pinniped-concierge-impersonation-proxy" deleted
clusterrolebinding.rbac.authorization.k8s.io "pinniped-concierge-impersonation-proxy" deleted
role.rbac.authorization.k8s.io "pinniped-concierge-kube-cert-agent" deleted
rolebinding.rbac.authorization.k8s.io "pinniped-concierge-kube-cert-agent" deleted
role.rbac.authorization.k8s.io "pinniped-concierge-aggregated-api-server" deleted
rolebinding.rbac.authorization.k8s.io "pinniped-concierge-aggregated-api-server" deleted
role.rbac.authorization.k8s.io "pinniped-concierge-kube-system-pod-read" deleted
rolebinding.rbac.authorization.k8s.io "pinniped-concierge-kube-system-pod-read" deleted
clusterrole.rbac.authorization.k8s.io "pinniped-concierge-pre-authn-apis" deleted
clusterrolebinding.rbac.authorization.k8s.io "pinniped-concierge-pre-authn-apis" deleted
clusterrolebinding.rbac.authorization.k8s.io "pinniped-concierge" deleted
rolebinding.rbac.authorization.k8s.io "pinniped-concierge-extension-apiserver-authentication-reader" deleted
role.rbac.authorization.k8s.io "pinniped-concierge-cluster-info-lister-watcher" deleted
rolebinding.rbac.authorization.k8s.io "pinniped-concierge-cluster-info-lister-watcher" deleted
Error from server (NotFound): error when deleting "https://get.pinniped.dev/v0.18.0/install-pinniped-concierge.yaml": the server could not find the requested resource (delete credentialissuers.config.concierge.pinniped.dev pinniped-concierge-config)
Error from server (NotFound): error when deleting "https://get.pinniped.dev/v0.18.0/install-pinniped-concierge.yaml": secrets "pinniped-concierge-impersonation-proxy" not found

A possible explanation is that the pinniped-concierge-impersonation-proxy gets deleted first and automatically deletes the associated secret. Also that credentialissuers.config.concierge.pinniped.dev pinniped-concierge-config gets deleted first.
What did you expect to happen?
Be able to deploy and then undeploy Pinniped Concierge without an error.
What is the simplest way to reproduce this behavior?
kubectl apply -f https://get.pinniped.dev/v0.18.0/install-pinniped-concierge-crds.yaml
kubectl apply -f https://get.pinniped.dev/v0.18.0/install-pinniped-concierge.yaml
kubectl delete -f https://get.pinniped.dev/v0.18.0/install-pinniped-concierge.yaml

In what environment did you see this bug?

Kubernetes version (use kubectl version):

Hi @castelblanque,
Thanks for creating this issue and providing all the details of the uninstall problem that you encountered.
The installation instructions have changed slightly since those older versions of Pinniped. Please see https://pinniped.dev/docs/howto/install-concierge/ for the updated instructions.
The way to use kubectl to install and uninstall the latest version of the Concierge is currently the following:
# To install:
kubectl apply -f https://get.pinniped.dev/v0.18.0/install-pinniped-concierge-crds.yaml
kubectl apply -f https://get.pinniped.dev/v0.18.0/install-pinniped-concierge-resources.yaml

Note that the file name of the second file is slightly different from what you mentioned above.
# To uninstall, delete them in reserve order:
kubectl delete -f https://get.pinniped.dev/v0.18.0/install-pinniped-concierge-resources.yaml
kubectl delete -f https://get.pinniped.dev/v0.18.0/install-pinniped-concierge-crds.yaml

The CRDs need to be deleted last to avoid the problem that you mentioned above.
Note that the other filename that you referenced above (https://get.pinniped.dev/v0.18.0/install-pinniped-concierge.yaml) is only intended for use with kapp, the installer tool from Carvel. kapp contains additional intelligence beyond what is offered in kubectl to handle these kinds of install/uninstall ordering issues automatically. Because kubectl lacks that ordering logic, we offer the other two files for use with kubectl.
Please let me know if this works for you!

Hi @cfryanr,
Many thanks for the explanation, it works like a charm!


NPM or Bower

How to install this using npm or bower ?

Sorry duplicate #8
probably needs to be described in readme

https://github.com/flowplayer/flowplayer-hlsjs#commonjs
https://github.com/flowplayer/flowplayer-hlsjs#building-the-plugin
?

To complicated. To many dependencies. Please add to bower with "dist" folder with compiled and minified version

If you are using npm with browserify or webpack, you can just run
npm install flowplayer flowplayer/flowplayer-hlsjs

then in your javascript code you do
var flowplayer = require('flowplayer');
require('flowplayer-hlsjs'); // Plugin gets injected to flowplayer

flowplayer('#mydiv', myConf);

We can look into publishing this also to bower

I dont use browserify or webpack or requirejs so I cant use require() function

Found this thread via Google and it was only mildly useful.   This should get documented somewhere...
To install flowplayer 6.05 and flowplayer-hls 1.03 via npm and use with systemjs and systemjs-builder.
Install via NPM
npm install flowplayer flowplayer/flowplayer-hlsjs
gulp-copy file structure (result)
lib
-flowplayer
--hls.js
---flowplayer.hlsjs.js
---hls.js
--flowplayer.js

systemjs.config.js
map: {
     'flowplayer': 'lib/flowplayer/',
     'flowplayer-hlsjs': 'lib/flowplayer/hls.js/',
     'hls.js/lib/index.js': 'lib/flowplayer/hls.js/hls.js'
}, packages: {
     'flowplayer': { main: 'flowplayer', defaultExtension 'js' },
     'flowplayer-hls': { main: 'flowplayer.hlsjs', defaultExtension: 'js' }
}

import / require
var flowplayer = require('flowplayer');
var engine = require('flowplayer-hlsjs');
engine(flowplayer);

usage
var instance = flowplayer($el, config);
Note there is a JQuery dependency that I didn't cover above, but make sure it is part of your systemjs package.

Relates https://github.com/flowplayer/flowplayer/issues/1131


add new simplified Chinese into Pods>QBImagePickerController>Resources, app crash

I need to localize the language into simplified Chinese. And I did the following:

Clone the repo down on local disk, and it's already been localized and contains a zh-Hans.lproj folder.
I copied the folder into Pods project > Pods > QBImagePickerController > Resources. It appears in the catalog and also in the same folder as ja.lproj
Directly run the build on device, it works well.
Archive the build and download it from a webpage and install, it will crash when enter into imagePicker.

I cannot figure it out why it will crash. please help or give me some hints. Thanks!!

I deleted all local files and reload the pod, and the crash is gone.

Hello, how do u finally get simplified Chinese localization working?
I add zh-Hans.lproj which contains QBImagePicker.strings for simplified Chinese to Pods project, but seems not working at all.


show "No data" section inside table + Uncaught Error: Invariant Violation fixes

Awesome! I did make a small update so that the paging content doesn't display an empty select / next button.
Thanks for putting this together :+1:


Fixing Erroneous validations

I'm using puppet 3.7.5-1puppetlabs1, and it failed because of the validations.
With this changes, validations pass

Hmm. We could have caught this if we were testing against 3.7 I suppose.
How do you feel about augmenting your PR with new test targets so this is caught in the future?
(stealing from https://github.com/garethr/puppet-module-skeleton/blob/master/skeleton/.travis.yml)

I've never done that. I suppose I can try.

I misunderstood you: I was thinking you were talking about adding more tests.
Here there are the new environments. By the way: I'm having lot of trouble with the template. Maybe in other PR...

Thanks!


SHA-512 in rust-arkworks

I noticed that both c are produced with SHA-256 in <./javascript>. Is it ok to that crate to do it with SHA-512?

Oh we are removing sha512 everywhere and only using sha256.

What would be corrected test vectors for https://github.com/plume-sig/zk-nullifier-sig/blob/e7490a8c42d70988d3903a15c448ab1dfa62a807/rust-arkworks/src/tests.rs#L215 in that case?

Feel free to just calculate that for i.e. the test anvil sk/pk in the implementation and use that as a test; if the other implementations all give the same result, we can be pretty confident that its calculating correctly!


Add jdbc tests for boolean interpreted as number

@cawallin

https://github.com/facebook/presto/commit/66010b074cf81ce63e94cc43cc7361ad1a3b4e23

looks good


Wrong claim type "name" after successful authentication

I initially raised a documentation issue, then investigated further.
Details here: https://github.com/MicrosoftDocs/azure-docs/issues/18110
tldr; The OpenIDConnect code is adding the users name with a type of name, but the ClaimsIdentity property is looking for a type of http://schemas.xmlsoap.org/ws/2005/05/identity/claims/name
line 73 in ClaimTypes.cs in the F12 source
public const string Name = ClaimType2005Namespace + "/name";

why you posted C# issue here?

I found a bug, searched for "GitHub OpenID Connect" and posted here. Apparently the wrong place.


Integrate Request evm env filling in ethapi

Fix https://github.com/paradigmxyz/reth/issues/9296

excuse me, merging main looked like it would merge cleanly but some deps have changed in https://github.com/paradigmxyz/reth/pull/9391, let me know if you want help fixing the merge conflicts

excuse me, merging main looked like it would merge cleanly but some deps have changed in #9391, let me know if you want help fixing the merge conflicts

Let me fix it.

remains to fix docs. check out the Makefile, it contains some helpful commands like make lint and make rustdocs

ur very welcome @nkysg , thanks for the appreciation, I believe you can learn a whole lot from contributing to reth 😎


Permissions error with pgml.embed

Getting the following error when I run this query
SELECT pgml.version() /* returns 2.5.3 */
SELECT pgml.load_dataset('tweet_eval', 'sentiment'); /* runs with no issues */
SELECT text, pgml.embed('distilbert-base-uncased', text)
FROM pgml.tweet_eval
LIMIT 10;

I get
SQL Error [XX000]: ERROR: called Result::unwrap()on anErr value: PyErr { type: <class 'PermissionError'>, value: PermissionError(13, 'Permission denied'), traceback: Some(<traceback object at 0x7f6c962a0c80>) }
(Running as superuser)
Can anyone advise? Thanks!

Resolved on Discord: https://discord.com/channels/1013868243036930099/1120302118230818889

Resolved on Discord: https://discord.com/channels/1013868243036930099/1120302118230818889


d3__WEBPACK_IMPORTED_MODULE_0__.hierarchy is not a function

const root = d3
.hierarchy(data)
.sum(d => d.value)
.sort((a, b) => b.value - a.value)
.eachAfter(
d =>
(d.index = d.parent ? (d.parent.index = d.parent.index + 1 || 0) : 0)
);
versions :
"react": "^18.2.0",
"d3": "^3.5.17"
how to solve this?
repository : https://github.com/mbasakCinedigmInd/d3-hierarchical-bar-chart.git
I created this repository.. please tell me you need anything else. I am desperately searching for a solution

The issue is that you are using D3 version 3. Try using the latest version of D3.

thank you..

hi... the graph is working.. but clicking on it does not work
i want to achieve this
https://github.com/john-guerra/d3-react-bar-chart-hierarchy

const bar = g
.selectAll("g")
.data(d.children)
.join("g")
.attr("cursor", d => (!d.children ? null : "pointer"))
.on("click", d => down(svg, d))
this onclick not working


drop python<=3.7 support

According to https://endoflife.date/python python 3.7 has been EOSed 27 Jun 2023.
Filter all code over pyupgracde --py38-plus.

Closing due to lack of response


[matio] Update to 1.5.23

There are no remaining changes in this PR after https://github.com/microsoft/vcpkg/pull/24706/ . Thanks for the submission though!


LanguageCLient-neovim: virttext support

Motivation for this change
There has been a new release of languageclient-neovim today with
support for neovim's virttext:
https://github.com/autozimu/LanguageClient-neovim/commit/273e5dc44cf1f860e40999ee47ad9cb214ffd94c
I am really eager to test it so I tried updating the package but I get
I get "error: no matching package named crossbeam-channel found"
I don't know the rust ecosystem so looking for help here.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

I think you can solve your error by changing the cargoSha256 attribute to for example<PHONE_NUMBER>000000000000000000000000000000000000000000, it should then fail to build with an error showing you the correct cargoSha256 like any other fixed-output derivation (e.g. a package source). Correcting that should solve the dependency not found error, at least it did when I had a similar error in my Rust project. From what I understand the cargoSha256 attribute locks down the Rust dependencies, so it won't find any other dependencies, but I find the error message a bit misleading.


temp just for code review Refactor install command and implement uninstall command.

This commit implements the uninstall tool command.
The uninstall tool command is responsible for uninstalling global tools that
are installed with the install tool command.
This commit heavily refactors the ToolPackage and ShellShim namespaces to
better support the operations required for the uninstall command.
Several string resources have been updated to be more informative or to correct
oddly structured sentences.
This commit also fixes --version on the install command not supporting ranges
and wildcards.
Fixes #8549.
Issue #8485 is partially fixed by this commit (--prerelease is not yet
implemented).

If this PR should not run tests please add text "skip[REMOVE_THIS]ci[REMOVE_THIS]please" (remove the marked text, no quotes).
Please add description for changes you are making.
If there is an issue related to this PR, please add the reference.

I've pushed an update that fixes everything mentioned here except the change to move the InstallPackage parameters to the constructor of the project restorer.  I'll be fixing that shortly.

@peterhuene sorry for the delay. Checked the latest https://github.com/dotnet/cli/compare/master...peterhuene:uninstall-command?expand=1
I think it is good to merge if once CI passes

We should probably close this :) shows up in my "mentioned PRs list"


tx and rx control

Are you using control for TE and DE?  Or are you depending auto direction control?

hello?

nevermind i figured it out, thank you!


Issue #22 make log level configurable and log as json

Codecov Report

Merging #24 (c0627fe) into master (15fd462) will decrease coverage by 0.59%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #24      +/-   ##
==========================================
- Coverage   89.94%   89.34%   -0.60%
==========================================
  Files          17       17
  Lines         746      751       +5
==========================================
  Hits          671      671
- Misses         61       66       +5
  Partials       14       14

Flag
Coverage Δ

unittests
89.34% <0.00%> (-0.60%)
:arrow_down:

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

Impacted Files
Coverage Δ

cmd/provider/config.go
100.00% <ø> (ø)

cmd/provider/main.go
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

Codecov Report

Merging #24 (c0627fe) into master (15fd462) will decrease coverage by 0.59%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #24      +/-   ##
==========================================
- Coverage   89.94%   89.34%   -0.60%
==========================================
  Files          17       17
  Lines         746      751       +5
==========================================
  Hits          671      671
- Misses         61       66       +5
  Partials       14       14

Flag
Coverage Δ

unittests
89.34% <0.00%> (-0.60%)
:arrow_down:

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

Impacted Files
Coverage Δ

cmd/provider/config.go
100.00% <ø> (ø)

cmd/provider/main.go
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Ignore old custom field data

Environment

Python version: 3.6.8
Nautobot version: 1.0.1

We used to have a custom field asw_vulnerabilityscan assigned to IP Addresses. It is no longer assigned to this model, but still exists and is assigned to other models. But our DB contains this field for 1036 of such IPs. Editing any of these is not possible as it crashes on the field being set.

AFAICT is it great to keep this data as long as the field exists since it is very easy to missclick and remove the field from some model by accident. But form validation should ignore these.
Steps to Reproduce

Create custom field
Assign it to some object
Create instance of such object
Fill in the custom field
Unassign the custom field from the object
Try to edit the created instance

Expected Behavior
No issues found.

Observed Behavior
Unable to edit, old custom field data breaks for validation.

I'm sorry, the reproducer might be incomplete. During process of fixing the data I found out assigning and unassigning the custom field starts RQ task nautobot.extras.tasks.provision_field, which takes quite some time. It timed out, which might be why there are some IPs with that custom field still set in the first place. Nonetheless form validation should not takes this data into account as user has no control over sending them.

Hmmm... This is interesting. So you're saying that the worker was unable to complete the task and it resulted in this bizarre behavior?

Hmmm... This is interesting. So you're saying that the worker was unable to complete the task and it resulted in this bizarre behavior?

Exactly (let me join this conversation as a @Thetacz colleague :-) ) - it is impossible to edit such an object (record) and this error is shown by nautobot.

Okay thanks. Need to try to reproduce this and I'll follow up!

So there are realistically two bugs here.

The custom field background tasks are not being run in an atomic transaction. This is an oversight on my part.
The job times out on larger data sets. This one will be easier to sort out in 1.1 with #223 which is already underway.

Additionally, we should probably create a command to normalize custom field data in the event one of these jobs fails. Muck like the trace paths command.


WIP: draft KIPs for Cross Margin and Advanced Orders

When opening a pull request to submit a new KIP, please use the suggested template: https://github.com/Kwenta/KIPs/blob/master/kip-X.md
We have a GitHub bot that automatically merges some PRs. It will merge yours immediately if certain criteria are met:

The PR edits only existing Draft PRs.
The build passes.
Your Github username or email address is listed in the 'author' header of all affected PRs, inside .
If matching on email address, the email address is the one publicly listed on your GitHub profile.

@Tburm updated fee sections


Feature/noise model refactor simple

Implements #280.
Sorry guys, this PR is not tiny and not trivial and somewhat higher priority (should go in by Friday, at the latest Monday morning).

@mpharrigan Thanks for pointing to inconsistent T2 definitions in signatures.
So, I think as is this is better than it was before. T2 and T1 are somewhat tricky concepts, see also this here: https://en.wikipedia.org/wiki/Relaxation_(NMR)#BPP_theory

@stevenheidel thanks, and that's a good call-out. I think we could make pyquil.noise.decoherence_noise_model and pyquil.noise.apply_noise_model private for now.

Ok, @mpharrigan @stevenheidel @ecp-rigetti I think I addressed everything.


examples broken

For example, jwttoken example broken with error: "Error Request must be POST".

It may be too late but you can either add
server.Config.AllowGetAccessRequest = true
in the main function and restart your server.
The other solution is to modify the html code and replace the href link by a form with a submit button
HTH

I'm having the same issue, added that config and received a new error (did not modify the html code):
ERROR: Client authentication not sent


Minor update for node build

Allows to build node client on a fresh repo clone without addition voodoo.
npm install is needed to avoid such errors
> compile-protobuf-files
> cd src && pbjs -t static-module -o ProtobufMessage.js ../../glide-core/src/protobuf/*.proto && pbts -o ProtobufMessage.d.ts ProtobufMessage.js

sh: 1: pbjs: not found

(missing protobufjs-cli)

Maybe create a new issue to track the rest of the work that @avifenesh mentioned. I'm in favour of keeping PRs small, so I think this one is fine as is and the rest can be done in other PRs.

That was actually a request to enrich clean action, which I did.


Remove unnecessary div wrapper in Modal component

Notes for Reviewers
This PR fixes #
This div prevents the footer from utilizing its full width. If we set width: 100%, it resolves the issue. I checked all the modal footers, and there are no side effects from this change.
Before:

After

Signed commits

[x] Yes, I signed my commits.

@sudhanshutech can you please review this and merge it.

Yes I checked there is not side effect of this.


update eclipselink-spring-boot-starter to work with 1.4

After I upgrade to Spring Boot 1.4, I'm seeing this error:
Caused by: org.springframework.beans.BeanInstantiationException: Failed to instantiate [org.springframework.boot.autoconfigure.orm.jpa.EclipselinkJpaAutoconfiguration$$EnhancerBySpringCGLIB$$31f73832]: Constructor threw exception; nested exception is java.lang.NoSuchMethodError: org.springframework.boot.autoconfigure.orm.jpa.JpaBaseConfiguration: method <init>()V not found
        at org.springframework.beans.BeanUtils.instantiateClass(BeanUtils.java:154) ~[spring-beans-4.3.2.RELEASE.jar:4.3.2.RELEASE]
        at org.springframework.beans.factory.support.SimpleInstantiationStrategy.instantiate(SimpleInstantiationStrategy.java:89) ~[spring-beans-4.3.2.RELEASE.jar:4.3.2.RELEASE]
        at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.instantiateBean(AbstractAutowireCapableBeanFactory.java:1098) ~[spring-beans-4.3.2.RELEASE.jar:4.3.2.RELEASE]
        ... 32 common frames omitted
Caused by: java.lang.NoSuchMethodError: org.springframework.boot.autoconfigure.orm.jpa.JpaBaseConfiguration: method <init>()V not found
        at org.springframework.boot.autoconfigure.orm.jpa.EclipselinkJpaAutoconfiguration.<init>(EclipselinkJpaAutoconfiguration.java:53) ~[data-jpa-eclipselink-spring-boot-starter-0.10.0.jar:1.4.0.RELEASE]
        at org.springframework.boot.autoconfigure.orm.jpa.EclipselinkJpaAutoconfiguration$$EnhancerBySpringCGLIB$$31f73832.<init>(<generated>) ~[data-jpa-eclipselink-spring-boot-starter-0.10.0.jar:1.4.0.RELEASE]
        at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method) ~[na:1.8.0_51]
        at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62) ~[na:1.8.0_51]
        at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45) ~[na:1.8.0_51]
        at java.lang.reflect.Constructor.newInstance(Constructor.java:422) ~[na:1.8.0_51]
        at org.springframework.beans.BeanUtils.instantiateClass(BeanUtils.java:142) ~[spring-beans-4.3.2.RELEASE.jar:4.3.2.RELEASE]
        ... 34 common frames omitted

[WARNING]
java.lang.reflect.InvocationTargetException
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:497)
        at org.springframework.boot.maven.AbstractRunMojo$LaunchRunner.run(AbstractRunMojo.java:483)
        at java.lang.Thread.run(Thread.java:745)
Caused by: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'org.springframework.boot.autoconfigure.orm.jpa.EclipselinkJpaAutoconfiguration': Instantiation of bean failed; nested exception is org.springframework.beans.BeanInstantiationException: Failed to instantiate [org.springframework.boot.autoconfigure.orm.jpa.EclipselinkJpaAutoconfiguration$$EnhancerBySpringCGLIB$$31f73832]: Constructor threw exception; nested exception is java.lang.NoSuchMethodError: org.springframework.boot.autoconfigure.orm.jpa.JpaBaseConfiguration: method <init>()V not found

Any ideas? seems like in 1.4 the JpaBaseConfiguration now expects a constructor to be implemented in the starter

I will have a look.

@luizkowalski could you please try out 0.10.1 and report back any issues or close this issue if fixed.
Thanks,
Jörg

perfect! thanks @jbellmann

Is this issue resolved and verified to be working. I face this issue in Spring Boot 1.4.1 as well. Kindly advise.

@nkapre at least I hope so :-). @luizkowalski closed the issue and commented in a way that I assume it works. Please verify you use the latest version '0.10.1'.
If it seems you have the latest versions in use and the error still exists. Please provide more information about your project, dependencies or maybe stacktraces.

The version 0.10.1 is not on the repo. I downloaded, installed locally and worked

All releases made so far are available in maven-central. Upload to maven-central is part of release. Have a look at the 'Maven-Central-Badge'. Or do a search here.
Maybe you already tried it with Spring-Boot 1.4.3.RELEASE? Does it work for you?
Regards
Jörg

The bug does not occur in 1.4.3.

That's fine. So I will close this issue.
@luizkowalski thanks for fast response regarding this issue.
Regards
Jörg


File naming in project.

Now many files in repo called like linux_*. It is very confusing, because it is like standard *_linux.go, but not. Also tab completion is not very convenient with such naming.
What do you think about changing naming to standard postfix form?

SGTM


How can i use the CSP for WKWebView

Can I resolve the CORS issue of WKWebView through whitelist? I have seen a certain post where developers have resolved it though CSP or whitelist plugin but I am not able to figure out the way to resolve the issue. It would be a great help if someone share the solution.

Please share the post you are talking about.


[broker] Optimize TopicPolicy#deduplicationEnabled with HierarchyTopicPolicies

Motivation
This is one of the serial topic policy optimization with HierarchyTopicPolicies.
Update topic policy with HierarchyTopicPolicies comes with these benefits:

All topic policy related settings will goes into AbstractTopic#topicPolicies, easier to understand, check, review, and modify.
Unify policy update to AbstractTopic. And easier to find which policy is not applied to non-persistent topic yet. And we can easily add support for it.
Unify policy value with 3 level settings (topic/namespace/broker), and priority with topic > namespace > broker.  And it's easier to find which level settings is missing.
All values are updated at creation or by trigger. We can save some resource to update it or recalculate each time we use it.
etc.

Modifications
Add new field deduplicationEnabled in org.apache.pulsar.broker.service.AbstractTopic#topicPolicies.
Verifying this change

[ ] Make sure that the change passes the CI checks.

This change is already covered by existing tests, such as org.apache.pulsar.broker.service.persistent.TopicDuplicationTest
Some checklist for updating topic policy with HierarchyTopicPolicies.

[x] Broker level value set. And this configuration is not dynamic. (Not supported yet)
[x] Namespace level value updated.
[x] Topic level value updated. And it's only for persistent topic (Non-persistent topic support in #12830)
[x] Pre-existing unit test covers this change.

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation
Check the box below and label this PR (if you have committer privilege).
Need to update docs?

[x] no-need-doc

Code optimization.

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks


Enable COM APIs to log diagnostic and telemetry events and Log Corela…

…tion Vector Json to existing events

[ ] Have you signed the Contributor License Agreement?
[ ] Are you working against an Issue?


Upgrade Glyphish Gallery.app to v2.1

Thank you for the contribution. It was merged directly as 35f6a68d67ecb7e75be6ab17e5fcf080e95df8dd to keep commit history cleaner. Your contribution is still credited to you.


Keep optional nil values in the JSON.

In same cases optional values could mean the value can be null, but shall be presented in the JSON.

This behavior is already supported via the use of NSNull. We'll need a pretty compelling reason to add another way of doing it.
Update your PR to follow the rules in: https://github.com/jsonmodel/jsonmodel/blob/master/CONTRIBUTING.md
Then I'll reopen it.

Ok, I will check.
My main problem is that I have a lot of classes with a lot of properties. A lot of those properties are optional (in my case it means it can be null, but need to be presented in the JSON)
So by default if they are not set, the value is nil ==> they are removed from the JSON.
I know the NSNull, but it means I need to go through all the classes, all the properties, and if something is (Optional and nil), set it to NSNull to keep them in the JSON.


Update DevContainer Dockerfile to use SDK version 7

Proposed change

Update the DevContainer to use version 7 of the dotnet SDK in the Docker container.
Type of change

[x] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #793
This PR is related to issue:
Link to documentation pull request:

Checklist

[x] The code change is tested and works locally.
[x] Local tests pass. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[ ] I have followed the [development checklist][dev-checklist]
[x] The code compiles without warnings (code quality check)
[ ] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated for www.home-assistant.io

There will be a new one soon

Will make a new one. Did a bad rebase.


gowin.lic is outdated, and floating license server is not working.

Floating license server <IP_ADDRESS> is not working now. Although test connection is success. Maybe the license of floating license server is outdated too. Now I can't open GW_IDE.

same issue.

Bump?

Bump

Bump

I have found a solution to our problem. The server is working, but not for recent version of gowin IDE. Download 1.9.8: http://cdn.gowinsemi.com.cn/Gowin_V1.9.8_win.zip

Bump

I have found a solution to our problem. The server is working, but not for recent version of gowin IDE. Download 1.9.8: http://cdn.gowinsemi.com.cn/Gowin_V1.9.8_win.zip

You are awesome, thanks!

That has worked for me yesterday. All day. This morning it's back to "License verification failed".
I am using Gowin V1.9.8. I am using Floating License Server <IP_ADDRESS> port 10559. Test Connection works, but after Saving and restarting I get "License verification failed".

That has worked for me yesterday. All day. This morning it's back to "License verification failed". I am using Gowin V1.9.8. I am using Floating License Server <IP_ADDRESS> port 10559. Test Connection works, but after Saving and restarting I get "License verification failed".

Same.

This licensing issue has been the biggest obstacle to using this great (and cheap) FPGA board for DIY projects. I was excited about the new Tang 9K and about the 4K as well, but if the software cannot be easily licensed, that is a problem.

Just to track this, today (29 dec 2021) the license server is working again. Here is hope that it stays up.

This licensing issue has been the biggest obstacle to using this great (and cheap) FPGA board for DIY projects. I was excited about the new Tang 9K and about the 4K as well, but if the software cannot be easily licensed, that is a problem.

I filled out a simple web form and got a license of my own via email(*) in less than 24 hours.  Seemed pretty easy to me.
They ask for a MAC address which gives you a lot of options.  It would be easy enough for an average user to provide the MAC of a $10 USB Ethernet adapter and use that as a 'license dongle' for the software.
(*) As a result of requesting this license, I got an email from a Gowin sales guy.  After I responded with a quite lengthy email outlining all the reasons they would be smart to also support the open source toolchain for their chips, I haven't heard from him.

This licensing issue has been the biggest obstacle to using this great (and cheap) FPGA board for DIY projects. I was excited about the new Tang 9K and about the 4K as well, but if the software cannot be easily licensed, that is a problem.

I filled out a simple web form and got a license of my own via email(*) in less than 24 hours. Seemed pretty easy to me. They ask for a MAC address which gives you a lot of options. It would be easy enough for an average user to provide the MAC of a $10 USB Ethernet adapter and use that as a 'license dongle' for the software.
(*) As a result of requesting this license, I got an email from a Gowin sales guy. After I responded with a quite lengthy email outlining all the reasons they would be smart to also support the open source toolchain for their chips, I haven't heard from him.

What kind of info did you give them?  Ive sent two requests, two years apart, and havent got a single reply.

What kind of info did you give them? Ive sent two requests, two years apart, and havent got a single reply.

I looked at the times stamps.  My key was generated after I responded to the sales person. So I guess that might be tied together.
Here is what I sent:
“
Thanks for reaching out Scott. I’m a home hobbyist / enthusiast using Sipeed Tang Nano (different sizes) boards with your FPGAs for learning more about FPGA and hardware design. So I don’t have any particular requirements around FPGAs as I’m just an individual learning.
Thanks for making such reasonably priced FPGAs available as well as the EDA tools.  Using open source tools like LiteX with RISC-V cores makes your devices pretty convenient to use.  Also it makes it easier to get started with hardware work.  That your tools are freely available and not 50GB+ is refreshing.
I do think it would be in your interest to help projects like Apicula support your devices better. The more engineering types that use your products in their own personal projects, the more reach you have to get into their projects at work also.
Much like gcc has been a huge enabler for many embedded devices, open source tools under the Symbiflow umbrella (apicula, yosys, nextpnr) that work seamlessly with your devices can only benefit you in the long run.  What chip company doesn’t want to sell more chips right?  With great support from open source tools you can be the “go to” chip for hobbyist projects.
Thanks again for reaching out.
-Andrew
“


Fix example/react-todomvc's unexpected routing in iframe conditions

What this PR does / why we need it?
Add e.preventDefault() in  example/react-todomvc's <a> tag.
This is hard-coded solution, I recommend using button tag instead of anchor tag.
Any background context you want to provide?
<a href="/#" ...> is causing unexpected behavior in iframe's conditions.
In https://yorkie.dev/examples/todomvc example, when you click filtering options (All, Active, Completed), you will be redirected to iframe's parent url, which is https://yorkie.dev/#.
Also, using  tag for button that is not performing redirection is not recommended in sematic web. I recommend using button tag instead of anchor tag.
What are the relevant tickets?
None
Checklist

[x] Added relevant tests or not required
[x] Didn't break anything

 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.

If I change anchor tag to button, it will change appearance of this example, which I thought should be discussed before.
If it is okay to change, I will modify some component and CSS values to fit previous appearance

I modified anchor component with button and added some CSS values to fit previous appearance.


libc-builtin: Enhance buffered print for printf_wrapper

Move print_buf and print_buf to str_context struct to support multi-threading
when BUILTIN_LIBC_BUFFERED_PRINTF macro is enabled, and fix the issue of
losing some text.
ps. https://github.com/bytecodealliance/wasm-micro-runtime/issues/3430.

LGTM


FastAPI rewrite initial commit

Summary
Basic FastAPI project structure
Co-Authored-By: sarzz2
Checklist

[x] If endpoints were changed then they have been documented and tested.

[x] I have updated the docmentation to reflect the changes.
[x] I have updated the tests to support the changes.

[ ] This PR fixes an issue.
[x] This PR adds something new (e.g. new endpoint or parameter).
[x] This PR is a breaking change (e.g. endpoint or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)

why did you delete the cli?...

why did you delete the cli?...

Yeah alot of removed content was unnecessary


Links on a page to a page with the same URL are not followed when clicked

A bit of an unusual problem, but it looks like there's a feature in Lagrange, I presume to save data and reduce network requests, that prevents the browser from reloading a link when that link points to the current page. Unfortunately, at least one Geminispace game uses URLs to represent actions within the game, and so performing the same action twice in a row requires loading the same URL twice in a row.
This can be worked around by reloading the page instead of following the link, but it was a source of confusion and makes playing the game as normal more difficult. Something like a separate "this link goes to the current page" symbol would allow the same saving of data while not breaking Geminispace games that use URLs this way.

Could you give me an example of what Geminispace game please? I might be able to do a fix.

Could you give me an example of what Geminispace game please? I might be able to do a fix.

gemini://alchemi.dev/maze/ is the one I was playing.

gemini://alchemi.dev/maze/ is the one I was playing.

I think is because at some point in early development of Lagrange, I had to prevent reloading of the page due to some UI event where the URL wasn't changing but the navbar still triggered an "open" command.
Now I think this safeguard is obsolete, so I'll make it so that any page can be reloaded by clicking on links or bookmarks, retyping the URL and pressing Enter, etc.

Providing this doesn't have side effects that weren't immediately obvious, I'll include the change in v1.1.1 as a bugfix.


Cannot click "Post button" to post video

The code was unable to click on the post button to post the video

Same problem here. Gets stuck on line 359 in _set_video function of upload.py.

Yeah, mine fails to even set the caption/description as well, and the upload_video() function doesn't work as it returns a TypeError: exists: path should be string, bytes, os.PathLike or integer, not NoneType in line 611 of upload.py.

#116 @Victorinox13's approach worked for me but I also changed line 61 in config.toml to:
post_confirmation = "//div[contains(@class, 'tiktok-modal__modal-wrapper')]//div[contains(@class, 'tiktok-modal__modal-title') and contains(text(), 'Your videos are being posted to TikTok!')]".
This was because the TikTok website had changed the confirmation text to Your videos are being posted to TikTok! instead of the former Your videos are being uploaded to TikTok! and that had broken the post confirmation in line 600 of upload.py's _post_video() function.

I have created a PR (https://github.com/wkaisertexas/tiktok-uploader/pull/125) to address this issue...
The button was not being clicked because there were a couple of webDriverWait instances that were not being satisfied, preventing the click functionality from being reached. There's a little more to it but I'm quite sure that was the culprit.


customize calendar type

do you plan support another calendar type instead of Gregorian calendar ?
for example solar calendar ( jalali calendar )

If we are enough to organize the remaining issues, why not? However, it may take time since I'm not always sure when I'd have much time.
Could you also provide some examples of the calendar you're thinking of (any sample picture)?

@hyochan this library use solar calendar
https://github.com/Ali-Azmoud/persian_datepicker


All data table's headers aren't identified programmatically

URL: http://clarity-ks.surge.sh/tables
Tools: Web Accessibility Toolbar (WAT2.0)
Reproduced Steps:

Login Clarity Kitchen Sink with web browser.
Go to Tables tab.
Check all data tables in Tables tab.

Actual Results:
All data table’s headers are not identified programmatically.
For more details, please refer to "Tables-HeadresAreNotIdentified.png".
Expected Results:
All data table’s headers should be identified programmatically.
Fix Suggestion:

Each column header must have either scope="col-header name" or id="x".
Each row header must have either scope="row-header name" or id="x".
The headers attribute for each data cell must include all the superior parent's header, in order for the column headers to be properly identified.
For more details, please refer to "Table'sMarkup.png".

Note:
This issue also occurs on Typography tab.
Violation of Section 508 Standards:
Section 1194.22 (g) Row and column headers shall be identified for data tables.
User Impact: Blind.
Screen shot:

Tables-HeadresAreNotIdentified.png

I don't fully understand the requirement:
Each column header must have either scope="col-header name" or id="x".
According to W3C standards, only applicable values for scope are col, row, colgroup, rowgroup.
https://www.w3.org/TR/1999/REC-html401-19991224/struct/tables.html#adef-scope
Please, provide reference for the use of scope="col-header name"

@Jinnie
This isn't that big of a deal, unless it requires an update to the datagrid. Here is an example of use.
If this is something that requires changes to the datagrid code, it is out of the scope of an update to kitchen sink. Please move it out of this epic and back into the backlog if so. Thanks.


Please consider making case-sensitive find and replace the default

The find and replace functionality is case-insensitive by default. While tolerable for find, this is dangerous for find and replace. When using find+replace in case-sensitive programming languages, this may break code and introduce bugs.
Possible remedies:

Make the find+replace function case-sensitive by default
Make the default different depending on file type, just like code markup
making the relevant settings immediately available from the find+replace popup

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!

I understand not wanting to make case-sensitivity the unchangeable default, but would it be possible to make the default behavior a configurable setting (for in-file search, project search, and "add selection to next find match" matching)? I almost always want each of those to be case sensitive, and I find myself having to re-select case sensitivity on a regular basis.
You can currently select "Smart Case" as a default in settings; that setting could instead be a dropdown with "case sensitive", "case insensitive", and "smart case" options.


cdc/deployment: add kafka consumer docker file

What problem does this PR solve?

Issue Number: close #4932
What is changed and how it works?
add docker file for kafka-consumer.
Kafka-consumer is used by Test-infra for consistency check, we have to maintain it and build new image when the kafka-consumer get upgrade.
Check List
Tests

No code

Code changes
Side effects
Related changes
Release note
`None`.

/merge

/run-leak-test

/run-kafka-integration-test

/run-leak-test

/run-integration-test
/run-kafka-integration-test

/run-integration-test
/run-kafka-integration-test

/run-leak-test


Updating Austrian number requirements

Updated the link to the Austrian numbering plan (nothing changed, just the URL)
Updated the length requirements for ndcs numbers from 6 to 5

I have an example of a valid number for the Klagenfurt region that has 5 digits after the ndc code.
I'm not sure if the change included here is good or should I split 463 into it's own case?

Coverage decreased (-1.6%) to 96.814% when pulling b372fd57a42a2f03d871a638aa840cb81b9047c0 on scott2211:scott2211/update-austrian-number-length into 798ddb1da2aad54a44a6a588f545a659bb31db41 on floere:master.

@scott2211 Thanks for the useful PR, Scott! 😊 I'm unsure about splitting it into its own case since the numbering plan mentions 6 as the minimum length – what is the number?

Hey, the number is +4346349133. Example company listing here: https://finde-offen.at/klagenfurt-am-worthersee/georg-huainig-kfz-reparatur-und-handels-gesellschaft-mbh-43960

@scott2211 Thanks again! 🎉 Released in 2.18.8 😊🚀

Amazing, thank you!


Document billing requirements - cost recovery

Gather cost recovery requirements
List them for next ticket creation

Priority for Q3 - Cost recovery - capturing the spend per team to create JV
Lower priority (Q3-Q4) - billing information / cost and usage monitoring for teams.
Tickets then taken by Adin

Cost Recovery System Requirements
1. Overview
This document outlines the requirements for implementing an automated cost recovery system for team expenses using Azure FinOps toolkit and policy-based tagging.
2. Required Report Columns
2.1 Primary Data Columns

Account Coding (License Plate)
Subtotal (Usage amount in CAD)
GST Amount (5%)
PST Amount (7%)
SSC/Overhead Fee (6%)
Total Amount (Sum of subtotal, taxes, and fees)
Expense Authority Email
Expense Authority Name
Ministry Name

3. Current Process
3.1 Quarterly Cost Recovery Workflow

Create a spreadsheet with all required columns
Submit spreadsheet to finance team
Finance team creates journal vouchers
Finance team processes cost recovery

4. Technical Requirements
4.1 Azure Resource Tagging
Mandatory Tags

Account coding/License plate
Billing Group
Ministry name

Tag Implementation

Tags must exist at subscription level
All resources must inherit tags automatically from subscription
Implementation through Azure Policy

4.2 Resource Tracking Requirements

All costs must be trackable by account coding
Resources must be properly tagged for accurate cost allocation
FinOps toolkit must be configured to track costs by tagged attributes

4.3 Reporting Requirements
Automated Components

Ability to query FinOps toolkit reports by:

Account coding
Time period
Total spend

Manual Components

Generation of reports including:

Base usage from FinOps toolkit
Tax calculations (GST/PST)
Overhead fee calculations

5. Implementation Steps

Configure Azure Policy for automatic tag inheritance
Verify existing subscription-level tags
Implement automatic tagging policy
Set up FinOps toolkit reporting
Create manual reporting process for tax/overhead additions
Validate cost tracking accuracy

6. Success Criteria

All resources automatically inherit required tags
Costs are accurately tracked by account coding
Reports can be generated with all required columns
Data can be easily exported for finance team's journal voucher process

7. Stakeholders

Finance team (Journal voucher processing)
Expense Authorities
Azure resource owners


Player name replaced by storage type

In the latest development build, the storage method is shown in various messages instead of the player name:

Errr, that is really weird.
I'm not having that issue at all.

Lazy copy pasting 😕


wip

Context

Closes https://github.com/sourcegraph/cody/issues/3403

Test plan

@valerybugakov I'll do a cleanup and ship it, yep. I was starting this before but then we got the legal pushback.

@philipp-spiess yes we'll be able to filter down to a subset of users (e.g. just users who have seen an autocomplete suggestion) in Looker. insertedCharacters will include both Cody + user written code, right?
@akalia25 this event is probably a good candidate to exclude from the Amplitude pipeline, I can't imagine us doing this kind of analysis there.


Fix rounding issues

Continuation of #7041. This PR also addresses other rounding-related issues as well:

Changed all *.resize() functions from  *.resize(newWidth, newHeight) to *.resize(desiredWidth, desiredHeight), because not all dimensions are possible, i.e., the framebuffer cannot have fractional pixels. The new resize functions round to the nearest possible dimensions.
Selected to use Math.round() everywhere instead of Math.floor/ceil(). It naturally works better than the other candidates in cases of slight imprecisions, e.g. Math.round((widthInPixels / resolution) * resolution) === widthInPixels which not always true for floor and ceil without special care. Of course other choices would be valid as well.
Made sure that RenderTexture.width/height is always precisely the width/height in pixels divided by the resolution of the texture.
Added explicit rounding for the integer arguments of gl.viewport, gl.scissor, gl.drawImage. Again, we need to make sure to round especially in cases like  width * resolution which might not be integer even if width === widthInPixels / resolution: we could get one less than expected when this number is cast to int.
Removed rounding in a few places where it's probably unnecessary.
Fixed some of the ROUND_PIXELS issues in Mesh and Sprite (#7049).

Pre-Merge Checklist

[ ] Tests and/or benchmarks are included
[ ] Documentation is changed or added
[x] Lint process passed (npm run lint)
[x] Tests passed (npm run test)

Looks good, but I have to check all those places one or two more times :)

I'd really prefer this was not an omnibus PR. I know the issues being addressed are related but isolating this into smaller PRs makes it easier to review these issues individually. We don't have a lot of test coverage for these areas (and there are no tests added here) so the consequences can be subtle and difficult to observe.

PR was split in multiple parts. The source frame rounding doesn't really make sense after all, so you won't find it in the other PRs.


Picking up Dream Journal crashes the client

Picking up Dream Journal crashes the client till the mod is completely removed. It only happens when item picked up from the ground.

Removing mod from server and client fixed the issue.

I'm sorry you encountered this error.  I'm unable to reproduce the problem on my end, though.  Do you have a crash report that you could attach to this ticket so that I can investigate further?  Thank you.

okay so I have the problem too now, everything worked fine when in singleplayer, problem happened on a freshly started mp server...
looking at the server log it says the following
09.11 17:39:27 [Server] Server thread/ERROR [ne.mi.ev.EventBus/EVENTBUS]: Exception caught during firing event: Could not initialize class com.verdantartifice.primalmagick.client.books.BookHelper
09.11 17:39:27 [Server] INFO 	Index: 11
09.11 17:39:27 [Server] INFO 	Listeners:
09.11 17:39:27 [Server] INFO 		0: HIGHEST
09.11 17:39:27 [Server] INFO 		1: net.minecraftforge.eventbus.EventBus$$Lambda$4211/0x00000008016d2000@223c77ef
09.11 17:39:27 [Server] INFO 		2: HIGH
09.11 17:39:27 [Server] INFO 		3: ASM: class dev.architectury.event.forge.EventHandlerImplCommon event(Lnet/minecraftforge/event/TickEvent$PlayerTickEvent;)V
09.11 17:39:27 [Server] INFO 		4: NORMAL
09.11 17:39:27 [Server] INFO 		5: ASM: se.mickelus.tetra.effect.ItemEffectHandler@6cbdf8d onPlayerTick(Lnet/minecraftforge/event/TickEvent$PlayerTickEvent;)V
09.11 17:39:27 [Server] INFO 		6: ASM: class ironfurnaces.Config player(Lnet/minecraftforge/event/TickEvent$PlayerTickEvent;)V
09.11 17:39:27 [Server] INFO 		7: ASM: class dev.ghen.thirst.content.thirst.PlayerThirstManager onPlayerTick(Lnet/minecraftforge/event/TickEvent$PlayerTickEvent;)V
09.11 17:39:27 [Server] INFO 		8: net.minecraftforge.eventbus.EventBus$$Lambda$4211/0x00000008016d2000@f0c7a9e
09.11 17:39:27 [Server] INFO 		9: ASM: class com.aetherteam.aether.event.listeners.EntityListener onRiderTick(Lnet/minecraftforge/event/TickEvent$PlayerTickEvent;)V
09.11 17:39:27 [Server] INFO 		10: ASM: class com.aetherteam.aether.event.listeners.DimensionListener onPlayerTraveling(Lnet/minecraftforge/event/TickEvent$PlayerTickEvent;)V
09.11 17:39:27 [Server] INFO 		11: ASM: class net.mcreator.createsweetsandtreets.procedures.SugarCookiePropertyValueProviderProcedure onPlayerTick(Lnet/minecraftforge/event/TickEvent$PlayerTickEvent;)V
09.11 17:39:27 [Server] INFO 		12: ASM: class net.mcreator.createsweetsandtreets.procedures.HoldTheRainbowProcedure onPlayerTick(Lnet/minecraftforge/event/TickEvent$PlayerTickEvent;)V
09.11 17:39:27 [Server] INFO 		13: ASM: class net.mehvahdjukaar.supplementaries.common.events.forge.ServerEventsForge onPlayerTick(Lnet/minecraftforge/event/TickEvent$PlayerTickEvent;)V
09.11 17:39:27 [Server] INFO 		14: net.minecraftforge.eventbus.EventBus$$Lambda$4211/0x00000008016d2000@2e2514e
09.11 17:39:27 [Server] INFO 		15: ASM: class com.simibubi.create.content.equipment.bell.HauntedBellPulser hauntedBellCreatesPulse(Lnet/minecraftforge/event/TickEvent$PlayerTickEvent;)V
09.11 17:39:27 [Server] INFO 		16: net.minecraftforge.eventbus.EventBus$$Lambda$4211/0x00000008016d2000@3d3db215
09.11 17:39:27 [Server] INFO 		17: ASM: mcjty.lib.setup.ModSetup$EventHandler@7804aaa5 onPlayerTickEvent(Lnet/minecraftforge/event/TickEvent$PlayerTickEvent;)V
09.11 17:39:27 [Server] INFO 		18: ASM: mcjty.rftoolsutility.setup.ForgeEventHandlers@69f379dc onPlayerTickEvent(Lnet/minecraftforge/event/TickEvent$PlayerTickEvent;)V
09.11 17:39:27 [Server] INFO 		19: LOW
09.11 17:39:27 [Server] INFO 		20: ASM: se.mickelus.tetra.items.forged.VibrationDebuffer@2f832cd8 onPlayerTick(Lnet/minecraftforge/event/TickEvent$PlayerTickEvent;)V
09.11 17:39:27 [Server] INFO 		21: ASM: se.mickelus.tetra.items.modular.impl.toolbelt.booster.TickHandlerBooster@15e090a9 onPlayerTick(Lnet/minecraftforge/event/TickEvent$PlayerTickEvent;)V
09.11 17:39:27 [Server] INFO 		22: LOWEST
09.11 17:39:27 [Server] INFO 		23: net.minecraftforge.eventbus.EventBus$$Lambda$4211/0x00000008016d2000@7c7f3952
09.11 17:39:27 [Server] INFO java.lang.NoClassDefFoundError: Could not initialize class com.verdantartifice.primalmagick.client.books.BookHelper
09.11 17:39:27 [Server] INFO 	at TRANSFORMER/primalmagick@4.0.7/com.verdantartifice.primalmagick.common.items.books.StaticBookItem.m_7626_(StaticBookItem.java:191)
09.11 17:39:27 [Server] INFO 	at TRANSFORMER/minecraft@1.20.1/net.minecraft.world.item.ItemStack.m_41786_(ItemStack.java:580)
09.11 17:39:27 [Server] INFO 	at TRANSFORMER/minecraft@1.20.1/net.minecraft.world.item.ItemStack.m_41611_(ItemStack.java:911)
09.11 17:39:27 [Server] INFO 	at TRANSFORMER/createsweetsandtreets@1.0.0/net.mcreator.createsweetsandtreets.procedures.SugarCookiePropertyValueProviderProcedure.execute(SugarCookiePropertyValueProviderProcedure.java:30)
09.11 17:39:27 [Server] INFO 	at TRANSFORMER/createsweetsandtreets@1.0.0/net.mcreator.createsweetsandtreets.procedures.SugarCookiePropertyValueProviderProcedure.onPlayerTick(SugarCookiePropertyValueProviderProcedure.java:19)
09.11 17:39:27 [Server] INFO 	at TRANSFORMER/createsweetsandtreets@1.0.0/net.mcreator.createsweetsandtreets.procedures.SugarCookiePropertyValueProviderProcedure_onPlayerTick_PlayerTickEvent.invoke(.dynamic)
09.11 17:39:27 [Server] INFO 	at MC-BOOTSTRAP/net.minecraftforge.eventbus/net.minecraftforge.eventbus.ASMEventHandler.invoke(ASMEventHandler.java:73)
09.11 17:39:27 [Server] INFO 	at MC-BOOTSTRAP/net.minecraftforge.eventbus/net.minecraftforge.eventbus.EventBus.post(EventBus.java:315)
09.11 17:39:27 [Server] INFO 	at MC-BOOTSTRAP/net.minecraftforge.eventbus/net.minecraftforge.eventbus.EventBus.post(EventBus.java:296)
09.11 17:39:27 [Server] INFO 	at TRANSFORMER/forge@47.2.1/net.minecraftforge.event.ForgeEventFactory.onPlayerPostTick(ForgeEventFactory.java:920)
09.11 17:39:27 [Server] INFO 	at TRANSFORMER/minecraft@1.20.1/net.minecraft.world.entity.player.Player.m_8119(Player.java:285)
09.11 17:39:27 [Server] INFO 	at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.level.ServerPlayer.m_9240(ServerPlayer.java:467)
09.11 17:39:27 [Server] INFO 	at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.network.ServerGamePacketListenerImpl.m_9933_(ServerGamePacketListenerImpl.java:262)
09.11 17:39:27 [Server] INFO 	at TRANSFORMER/minecraft@1.20.1/net.minecraft.network.Connection.m_129483_(Connection.java:263)
09.11 17:39:27 [Server] INFO 	at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.network.ServerConnectionListener.m_9721_(ServerConnectionListener.java:142)
09.11 17:39:27 [Server] INFO 	at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.MinecraftServer.m_5703_(MinecraftServer.java:907)
09.11 17:39:27 [Server] INFO 	at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.dedicated.DedicatedServer.m_5703_(DedicatedServer.java:283)
09.11 17:39:27 [Server] INFO 	at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.MinecraftServer.m_5705_(MinecraftServer.java:814)
09.11 17:39:27 [Server] INFO 	at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.MinecraftServer.m_130011_(MinecraftServer.java:661)
09.11 17:39:27 [Server] INFO 	at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.MinecraftServer.m_206580_(MinecraftServer.java:251)
09.11 17:39:27 [Server] INFO 	at java.base/java.lang.Thread.run(Thread.java:833)
followed by aproblem with sending packets based on this and repeating this, both repeat 4 times

This issue seems to be happening on my dedicated server as well. Hosting on Debian, using Java OpenJDK 17, in a virtualized container for my friends. Whenever someone sleeps with the book it causes them to crash and then receive an internal server error upon the next login until their player data is removed manually. Anyone who is within the immediate area are also affected and crash.
Here is the error when they attempt to login:
[19:57:21] [Server thread/ERROR] [ne.mi.ev.EventBus/EVENTBUS]: Exception caught during firing event: Could not initialize class com.verdantartifice.primalmagick.client.books.BookHelper
        Index: 4
        Listeners:
                0: HIGH
                1: ASM: class dev.architectury.event.forge.EventHandlerImplCommon event(Lnet/minecraftforge/event/entity/player/EntityItemPickupEvent;)V
                2: NORMAL
                3: ASM: class net.mehvahdjukaar.supplementaries.common.events.forge.ServerEventsForge onItemPickup(Lnet/minecraftforge/event/entity/player/EntityItemPickupEvent;)V
                4: ASM: stepsword.mahoutsukai.handlers.ServerHandler@4cf87e6a useItem(Lnet/minecraftforge/event/entity/player/EntityItemPickupEvent;)V
                5: ASM: wayoftime.bloodmagic.util.handler.event.WillHandler@cd815c onItemPickup(Lnet/minecraftforge/event/entity/player/EntityItemPickupEvent;)V
                6: net.minecraftforge.eventbus.EventBus$$Lambda$4230/0x00007fb9f0a93708@4638f01b
                7: ASM: Block{minecraft:air} onPickup(Lnet/minecraftforge/event/entity/player/EntityItemPickupEvent;)V
                8: net.minecraftforge.eventbus.EventBus$$Lambda$4230/0x00007fb9f0a93708@51c22512
                9: net.minecraftforge.eventbus.EventBus$$Lambda$4230/0x00007fb9f0a93708@30351a9b
                10: ASM: org.cyclops.cyclopscore.advancement.criterion.ModItemObtainedTrigger@27241068 onPickup(Lnet/minecraftforge/event/entity/player/EntityItemPickupEvent;)V
                11: ASM: com.lothrazar.cyclic.event.ItemEvents@68405efd onPlayerPickup(Lnet/minecraftforge/event/entity/player/EntityItemPickupEvent;)V
                12: ASM: mob_grinding_utils.events.ChickenFuseEvent@1b4b157d startChickenFuse(Lnet/minecraftforge/event/entity/living/LivingEvent;)V
                13: LOWEST
                14: ASM: class com.hollingsworth.arsnouveau.common.event.EventHandler itemPickupEvent(Lnet/minecraftforge/event/entity/player/EntityItemPickupEvent;)V
java.lang.NoClassDefFoundError: Could not initialize class com.verdantartifice.primalmagick.client.books.BookHelper
        at TRANSFORMER/primalmagick@4.0.7/com.verdantartifice.primalmagick.common.items.books.StaticBookItem.m_7626_(StaticBookItem.java:191)
        at TRANSFORMER/mahoutsukai@1.20.1-v1.34.58/stepsword.mahoutsukai.effects.secret.GeasEffect.hasGeas(GeasEffect.java:143)
        at TRANSFORMER/mahoutsukai@1.20.1-v1.34.58/stepsword.mahoutsukai.effects.secret.GeasEffect.geasItem(GeasEffect.java:51)
        at TRANSFORMER/mahoutsukai@1.20.1-v1.34.58/stepsword.mahoutsukai.handlers.ServerHandler.useItem(ServerHandler.java:400)
        at TRANSFORMER/mahoutsukai@1.20.1-v1.34.58/stepsword.mahoutsukai.handlers.__ServerHandler_useItem_EntityItemPickupEvent.invoke(.dynamic)
        at MC-BOOTSTRAP/net.minecraftforge.eventbus/net.minecraftforge.eventbus.ASMEventHandler.invoke(ASMEventHandler.java:73)
        at MC-BOOTSTRAP/net.minecraftforge.eventbus/net.minecraftforge.eventbus.EventBus.post(EventBus.java:315)
        at MC-BOOTSTRAP/net.minecraftforge.eventbus/net.minecraftforge.eventbus.EventBus.post(EventBus.java:296)
        at TRANSFORMER/forge@47.2.4/net.minecraftforge.event.ForgeEventFactory.onItemPickup(ForgeEventFactory.java:494)
        at TRANSFORMER/minecraft@1.20.1/net.minecraft.world.entity.item.ItemEntity.m_6123_(ItemEntity.java:349)
        at TRANSFORMER/minecraft@1.20.1/net.minecraft.world.entity.player.Player.m_36277_(Player.java:560)
        at TRANSFORMER/minecraft@1.20.1/net.minecraft.world.entity.player.Player.m_8107_(Player.java:527)
        at TRANSFORMER/minecraft@1.20.1/net.minecraft.world.entity.LivingEntity.m_8119_(LivingEntity.java:2298)
        at TRANSFORMER/minecraft@1.20.1/net.minecraft.world.entity.player.Player.m_8119_(Player.java:241)
        at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.level.ServerPlayer.m_9240_(ServerPlayer.java:467)
        at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.network.ServerGamePacketListenerImpl.m_9933_(ServerGamePacketListenerImpl.java:262)
        at TRANSFORMER/minecraft@1.20.1/net.minecraft.network.Connection.m_129483_(Connection.java:263)
        at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.network.ServerConnectionListener.m_9721_(ServerConnectionListener.java:142)
        at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.MinecraftServer.m_5703_(MinecraftServer.java:907)
        at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.dedicated.DedicatedServer.m_5703_(DedicatedServer.java:283)
        at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.MinecraftServer.m_5705_(MinecraftServer.java:814)
        at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.MinecraftServer.m_130011_(MinecraftServer.java:661)
        at TRANSFORMER/minecraft@1.20.1/net.minecraft.server.MinecraftServer.m_206580_(MinecraftServer.java:251)
        at java.base/java.lang.Thread.run(Thread.java:840)
Caused by: java.lang.ExceptionInInitializerError: Exception java.lang.BootstrapMethodError: java.lang.RuntimeException: Attempted to load class net/minecraft/client/gui/Font for invalid dist DEDICATED_SERVER [in thread "Server thread"]
        at TRANSFORMER/primalmagick@4.0.7/com.verdantartifice.primalmagick.client.books.BookHelper.<clinit>(BookHelper.java:57)
        ... 24 more


:construction_worker: Set Up Continuous Integration

Is your feature request related to a problem? Please describe.
This repo lacks of CI
Describe the solution you'd like
Set up Travis CI and Coverage (maybe scrutinizer)
Describe alternatives you've considered
Test could run before a commit, however it's better to configure such features
Additional context
Mind to Partner Queue Solution

It's not possible to configure the Partner Queue Solution using conda, since some required bioconda packages are only available on *x86` architectures. Try to configure stuff using GitHub workflow.


Forbid vendor for all code

Quite often I've come to see "bad" dependencies being used in code to solve problems, that are already solved via different dependencies in the larger part of the codebase.
It would really help to have a way to forbid usage of such dependencies.
As an example:

Project uses github.com/stretchr/testify's assert and require packages
But new contributor adds dependency on github.com/gotestyourself/gotest.tools's similarly named packages

I propose we add option to ban dependency for all code, like this:
vendors:
    testify:
        in: github.com/stretchr/testify/**
    gotest.tools:
         in: github.com/gotestyourself/gotest.tools/**

forbiddenVendors:
    gotest.tools

Additionally, I propose we add a way to configure a message explaining how this restriction must be mitigated:
forbiddenVendors:
    gotest.tools:
         message: Instead use github.com/stretchr/testify

Hey, good idea. I'll do it in version 4 of the config when I have time, if no one does the PR before me. I will be glad to see pull-request if there is a desire to add this feature faster.

I don't mind making a PR implementing this. One thing I don't understand, why do we need to increment config version to use this feature? Doesn't seem like it break any backwards compatibility

Yes, it affects external tools. Each config version has a specific immutable data scheme that is not supposed to have any changes after release. (external tools don't expect new fields in same config version)
fyi:

plugin example: https://plugins.jetbrains.com/plugin/15423-goarchlint-file-support
schemes: https://github.com/fe3dback/go-arch-lint/blob/master/internal/services/schema/provider.go


HTML validation fails on links of categories with more than one word

Our site is a book library. Books have categories, which are displayed in a collection route with URLs in the following format:

http://localhost:3000/categories/[category_name]

Note that category names are unique in our database.
We have implemented this route. However, HTML validation fails in pages with links to categories with multiple words.

This issue would be considered resolved if these HTML validation errors are solved without changing the route structure.

[!NOTE]
We understand we could change the URL to take an id instead of the category name, but we are looking to use names for SEO-friendly pretty URLs.

Steps to reproduce

From the home page, right-click the page > View Page Source
Select all the code and copy it
Paste it on a HTML validator and validate it

To run the project
git clone https://github.com/milolucero/rails_library.git
cd rails_library
bundle
rails db:migrate
rails db:seed
rails s

Fixed on fc1f5b13d61ca3d973692159176492ab700ecc90 Thanks @ManWahLi !


#2 Split header using b';' and decode as ISO-8859-1

@ruairif Could you please elaborate shortly why it's needed? At least for the history.

Hi @vshlapakov @ruairif . This is currently breaking the extension for every site using cookies (when the spider is running on Python 3), so can we get this merged? Are there any pending changes?
(cc @starrify)

No objections, @ruairif  feel free to merge it if there's nothing else to add.

Thanks a lot @ruairif and thanks @jesuslosada for reminder :+1:

Thanks!


Question: Any way to autodetect the language?

It's amazing. Thank you so much for open-sourcing it. I've played around with it a lot and learned from it. Thanks for this.
Unfortunately, there doesn't seem to be a discussion tab, so I'm writing here instead.
Is there any way to automatically detect the language and/or scan for mixed language content? I'm attempting to capture my students' handwritten homework, which often includes descriptions in languages other than German on the same sheet of paper.
Thank you so much,
Johannes

Thanks for the issue!
Sorry, I don't know know how to do that.
But,  I guess Live Text (or ImageAnalyzer of VisionKit) automatically determines the language.
If I specify a language here,  but do not specify it, it will be automatically determined.


Add docker support for ReMo portal.

@glogiotatidis r?

.git directory is about 200mb and you can add it to .dockerignore to speed up the build process and shrink the size of the image

So this is in a good direction of a docker based development environment. There are still things to do to have a docker image ready for deployment, including 12factoring the app, launching a wsgi server, serving static files, etc.
That being said you can  skip https://github.com/mozilla/remo/pull/1050/files#diff-3254677a7917c6c01f55212f86c57fbfR13 altogether, since you're mounting the code repository in docker-compose anyway

lessc is missing from the docker image.

That's all for now, update it and ping me for another pass. Nice work! :+1:

@glogiotatidis pr updated. thanks for the review!

looks good, :candy: r+


Unable to uninstall js

Expected behavior
Should be able to uninstall gauge-js
Actual behavior
Getting error Unable to read plugin's metadata, removing ...\AppData\Roaming\gauge\plugins\js\2.3.2.nightly-2018-09-13 Failed to uninstall plugin js . remove ...gauge\plugins\js\2.3.2.nightly-2018-09-13\node_modules\grpc\src\node\extension_binary\node-v64-win32-x64-unknown\grpc_node.node: Access is denied.
Steps to replicate

Observed on Windows

gauge uninstall js`

Version
Gauge version: 1.0.3.nightly-2018-09-25
Commit Hash: b7ea2ee

When I tried to manually delete the folder it said this was being used by some other program. However all other programs were closed.

@sguptatw I'm not able to replicate this. gauge uninstall js working as expected for me.
Can you make sure all the gauge-js processes are closed?
The gauge-js is also used by vscode. So if you have a js project open in vscode, gauge will not be able to uninstall the plugin.


Node.js in builder Dockerfile upgraded to 6.9.2 (latest LTS release)

Can one of the admins verify this patch?

[test]

Current coverage is 68.00% (diff: 100%)

Merging #517 into master will not change coverage

@@             master       #517   diff @@
==========================================
  Files            27         27
  Lines          1266       1266
  Methods         269        269
  Messages          0          0
  Branches        250        250
==========================================
  Hits            861        861
  Misses          405        405
  Partials          0          0

Powered by Codecov. Last update a0f3fa0...76e123f


[nri-kube-events] k8s events forwarder 1.15.2

Is this a new chart
No
What this PR does / why we need it:
Bumps the agent version for nri-kube-events, and updates the bundle.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [mychartname])

Closing in favour of https://github.com/newrelic/helm-charts/pull/293


if zip file is bigger than 50mb don't bother to upload

i happened to have a big virtualenv when i installed zappa, it seems to pull everything in the venv by default which resulted in a big zip, ideally before going to upload that it could just barf and say its not supported. even better would be just pulling deps from requirements.txt file or dependency.

Your situation sounds different than mine, but I use Zappa for a lot of Flask deployments where the static folder gets sucked into the zip and uploaded with the rest of the materials.  This frequently pushes my zip well over 50MB.  It doesn't seem to impact me at all and if Zappa puked at a 51MB zip, I'd have a lot of trouble with my existing workflows.  Maybe we can create some flags around size limits or dependencies?

Hey @kapilt ! I think we met after your great talk at Serverless London..
You may be very interested in this commit that will hopefully be merged soon: https://github.com/Miserlou/Zappa/pull/548

yeah its actually on the basis of extracted size that it barfed not zip size. @Miserlou indeed we did, thanks, playing around with apigw approaches at the moment.

@Miserlou i had a look at #548, afaics that's the wrong solution, because i think the goal is actually just making zappa zipped lambdas smaller by default with the correct dependency stack.
ie. is the goal about keeping the size small ? or supporting large projects with over 250mb of runtime dependencies?
for keeping the size small which i think is the more universal win, conceptually, the solution in that PR feels like the wrong solution and just adds complexity and cold start latency, the right solution imo, would focus on keeping the zappa handler size minimal and getting a better default exclude. the fixed overhead on zappa handler should only be 1mb uncompressed (werkzeug and requestlogger) ~ +310k on the zip
afaics keeping the lambda size small needs a few things

[ ] better logical separation of runtime vs cli deps on zappa
[ ] use a better default exclude that takes into account the lambda env (no dist-info, or dups of lambda env modules)
[ ] zappa handler deps should be minimal, the callback for cli.certify seems to be the worse offender
[ ] always serialize config files as json in the lambda (add to lambda, to avoid carrying toml, yaml deps forward)
[ ] use explicit dependencies from a requirements.txt file instead of copying the whole virtualenv

some of this is easy and should dramatically cut down the zip file size, other parts require code refactoring, like the certify stuff. in general it feels like some refactoring would be good, the kitchensink class approach with cross call chains complicates separation of the cli env and the runtime env.

@Miserlou Do you have a sense of how challenging / how much work it would be to make Zappa operate on a "target" / different virtualenv that doesn't itself contain Zappa? I feel like this would go a long way towards addressing both this issue, as well as issue #525 and related issues.
In the approach I'm describing, the "target" virtualenv would have only those dependencies needed on the Lambda server, and none of the dependencies needed to upload and deploy, etc. This probably requires in some sense "introspecting" the target virtualenv without actually installing Zappa inside. My sense is that there would be lots of possibilities for this, but you're probably most familiar with what's involved.

Removing zappa would make my 'zappa update' upload take about 8 secs instead of the current 1 minute 6 secs (on my current network connectivity). The time spent waiting for uploads adds up over time. I'm looking at serverless-wsgi + sls as a workaround (in case others need an alternative too). Still, any update on this? I can't assist as i'm a first time python user.


Deprecate the CouplingMap.subgraph() method

Summary
The CouplingMap.subgraph() method has overlapping functionality with the
reduce() method as both return a new CouplingMap object with a subset of
nodes from the original. In practice the only difference between the 2 is
that subgraph() doesn't retain the input list order for the output
coupling map while reduce() does. Since there is likely a use case for
retaining that list order this commit deprecates the subgraph() method
so we don't maintain duplicate methods.
It's worth noting that the subgraph() method is probably slightly faster
as it just calls retworkx's subgraph. So if you don't care about node list order
You can still do use this method with the CouplingMap API by just doing
something like:
subgraph = CouplingMap()
subgraph.graph = original_cmap.graph.subgraph(node_list)

instead of:
subgraph = original_cmap.subgraph(node_list)

Details and comments

The deprecation is clearly fine - the failure is due to the relevant test not marking the call as deprecated.  If we want to support the form suggested in the commit message, perhaps we should change/add to the failing test (in test/python/transpiler/test_coupling.py line 199) to try that path.

@jakelishman it looks like github got confused again it said I dismissed your review by https://github.com/Qiskit/qiskit-terra/pull/6823/commits/f7a644a3ae65bdeb9cc12416256e9d18ba7e3952 which you pushed.


fix(enum): add suffix AsOf when generating the enum

Status

READY
Description
Fixes https://github.com/anymaniax/orval/issues/245
Todos

[ ] Tests
[ ] Documentation
[ ] Changelog Entry (unreleased)

The way you did it will rename all enum. We just need to add the suffix allOf when it's in a allOf

ok, misunderstood the problem. how do you detect when it is a allOf?

A simple enum schema:
components:
  schemas:
    LogEventLevel:
      enum:
        - Verbose
        - Debug
        - Information
        - Warning
        - Error
        - Fatal
      type: string

will generate:
export type LogEventLevel = 'Verbose' | 'Debug' | 'Information' | 'Warning' | 'Error' | 'Fatal';

export const LogEventLevel = {
  Verbose: 'Verbose' as LogEventLevel,
  Debug: 'Debug' as LogEventLevel,
  Information: 'Information' as LogEventLevel,
  Warning: 'Warning' as LogEventLevel,
  Error: 'Error' as LogEventLevel,
  Fatal: 'Fatal' as LogEventLevel,
};

and that causes a @typescript-eslint/no-redeclare error when we build our app.

Yes it's mostly to compile simple JavaScript but could be interesting to add a rule to disable eslint for that one. Explanation here

Yes it's mostly to compile simple JavaScript but could be interesting to add a rule to disable eslint for that one. Explanation here

Thanks for sharing that; agree it would be better to disable that eslint for the generated model.


[WIP] Various Improvements

Configure awaits
Avoid linq on hot path
Polyfills string.Contains(char) & StringBuilder.Append(ReadOnlySpan) for .NETSTANDARD2.0
Simplifies Peek & Consume Lexeme functions
Allow indexes with patch level differences to be deserialized

Ready for review / feedback.

Agreed on keeping the library safe. The allocation reductions for StringBuilder.Append are unmarkable.
There's a small perf improvement using Contains on modern runtimes as it skips logic to get the index position.
I'll do some benchmarking and figure out how many allocations we're saving with the LINQ changes soon -- and open up a new PR with a clean history.


Loco Interiors

[x] CabInput

Handled by CustomCabInputController & CabInputRelay
CabInputShunter, CabInputDiesel, CabInputSteam, CabInputSteamExtra, CabInputHandcar, CabInputCaboose
Lives on interior root, takes GameObject references to "C" control objects with ControlSpec attached
Controls brake, throttle, and reverser

[x] ControlSpec

Handled by ControlSetupBase derivatives, CabInputRelay to connect to loco controller
Lever, Button, Puller, Rotary, ToggleSwitch, Wheel
Attached to transform holding C control "grabbables", set up with properties for that control eg. "C throttle"
ControlImplBase is created from the spec when car is instantiated, this is found by CabInput

[x] ShunterDashboardControls, DieselDashboardControls

Horn, lights, sand, fan, engine shutoff
Take GameObject references to controls & LampControl components

[x] ShunterDashboardLampController, DieselDashboardLampController

Handled by CustomLampController, DashboardLampRelay
Listens to simulation events and turns on/off lamps based on changing values
Plays warning bell when turning on a lamp
Takes LampControl component references

[x] IndicatorsShunter, IndicatorsDiesel, IndicatorsSteam, IndicatorsTender

Handled by CustomCabIndicators, IndicatorInfo
Control the gauges & sliders in cabins on each Update() using values from loco controller
Take references to Indicator components in cab

[x] Indicator

IndicatorGauge (temp, fuel, oil, sand, etc), IndicatorGaugeLagging (only spedo?), IndicatorSlider (fire), IndicatorScaler (fire, water)

[ ] DoorsControllerShunter, DoorsControllerDiesel

Would it be possible to make interactive elements on other train cars? Like opening doors and windows and such.