Deploy protocol contracts on new chains

Closes #1626
This PR deploys the protocol contracts on the following chains:

boba-ethereum
boba-avalanche
boba-bnb
boba-moonbeam
arbitrum-nova
sx-network
sx-network-testnet
aurora
aurora-testnet
godwoken
godwoken-testnet

Used deterministic deployment whenever possible

@Ashar2shahid the E2E test failure will be resolved once you update this branch to the latest master

Added

aurora-testnet
godwoken-testnet
sx-network-testnet

All Contracts are getting verified locally @bbenligiray this should be good for review

arbitrum-nova
All contracts verified on nova.arbiscan.io
AccessControlRegistry
AirnodeRrpV0
RequesterAuthorizerWithAirnode
aurora
(etherscan being discontinued from 1st Feb)
blockscout gives error when clicking on the code tab to verify the contracts
aurora-testnet
(etherscan being discontinued from 1st Feb)
it looks like etherscan doesn't see it as a contract, maybe because of deterministic deployment
NomicLabsHardhatPluginError: Failed to send contract verification request.
Endpoint URL: https://api-testnet.aurorascan.dev/api
Reason: The Etherscan API responded that the address 0x92E5125adF385d86beDb950793526106143b6Df1 does not have bytecode.
This can happen if the contract was recently deployed and this fact hasn't propagated to the backend yet.

I tried sending transactions to the functions a few days ago but etherscan still doesn't see it as a contract
boba-avalanche
All Contracts verified on blockexplorer.avax.boba.network
AccessControlRegistry
AirnodeRrpV0
RequesterAuthorizerWithAirnode
boba-bnb
All Contracts verified on blockexplorer.avax.boba.network
AccessControlRegistry
AirnodeRrpV0
RequesterAuthorizerWithAirnode
boba-moonbeam
All Contracts verified on blockexplorer.bobabeam.boba.network
AccessControlRegistry
AirnodeRrpV0
RequesterAuthorizerWithAirnode
godwoken
All Contracts verified on gw-mainnet-explorer.nervosdao.community
AccessControlRegistry
AirnodeRrpV0
RequesterAuthorizerWithAirnode
godwoken-testnet
Verification API not responding on gw-explorer.nervosdao.community
AccessControlRegistry
AirnodeRrpV0
RequesterAuthorizerWithAirnode
sx-network
Verification API not responding on explorer.sx.technology
AccessControlRegistry
AirnodeRrpV0
RequesterAuthorizerWithAirnode
sx-network-testnet
Verification API not responding on explorer.toronto.sx.technology
AccessControlRegistry
AirnodeRrpV0
RequesterAuthorizerWithAirnode
While I was verifying the contracts on each testnet, I was sending test transactions to call the functions of each of the contracts. sx-network-testnet deterministic deployments were not callable even though the local verification passes. I remember running into this issue with the sx-network deterministic deployment as well, which is why I then did a undeterministic deployment. I did the same here. Just to be sure I went back and tested the functions of all contracts on all chains that fail verification.

Created issue for verifying contracts on block explorers: https://github.com/api3dao/airnode/issues/1630


Error "navigator is not defined" in NodeJS

npm install command:

npm i -S @scatterjs/core @scatterjs/eosjs2<EMAIL_ADDRESS>@babel/runtime

Minimal code to replicate:

const ScatterJS = require('@scatterjs/core');
const ScatterEOS = require('@scatterjs/eosjs2');
ScatterJS.plugins(new ScatterEOS());

Full Error Message:

ReferenceError: navigator is not defined
at C:\Users\chper\Documents\Deployer\node_modules\device-uuid\lib\device-uuid.js:1036:27
at Object. (C:\Users\chper\Documents\Deployer\node_modules\device-uuid\lib\device-uuid.js:1038:3)
at Module._compile (internal/modules/cjs/loader.js:778:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
at Module.load (internal/modules/cjs/loader.js:653:32)
at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
at Function.Module._load (internal/modules/cjs/loader.js:585:3)
at Module.require (internal/modules/cjs/loader.js:692:17)
at require (internal/modules/cjs/helpers.js:25:18)
at Object. (C:\Users\chper\Documents\Deployer\node_modules\device-uuid\index.js:14:18)

"device-uuid" was added as dep in a0253339465e19a4fd4c9b53057a63d71e655f29

I'm guessing you're running this in a nodejs environment?
We might have to add some global.navigator spoofing in there when the window object isn't present.

I'm guessing you're running this in a nodejs environment?
Yes. I wanted to use Scatter as a signature provider for a few of my nodejs scripts.

Can you try out the latest version up on npmjs and let me know if it works?

Can you try out the latest version up on npmjs and let me know if it works?

It doesn't work.
global.navigator should be assigned a value before import {DeviceUUID} from 'device-uuid';
And it looks like it shouldn't be assigned an empty object. Otherwise I get this error
TypeError: Cannot read property 'replace' of undefined
    at DeviceUUID.get [as parse] (C:\Users\chper\Documents\Deployer\node_modules\device-uuid\lib\device-uuid.js:981:38)

<EMAIL_ADDRESS>fixes this, but relay sockets on nodejs will have a defaulted nodejs_env value for the time being as there isn't really a usecase for them yet.
There needs to be a way to align nodejs env device uuids and browser device uuids for there to be any connections there, in the future.


fix: 100avatars-garden signed fetch

What does this PR change?
Fixes #1473
There were problems on solving the metadata included in the signed fetch operation:

metadata.sceneId: removed unnecessary prefix so only the scene hash is included
metadata.realm.hostname: reworked how the hostname is resolved for the current realm. When the /about data of the realm has missing comms, then its considered as main realm, so the hostname is set to realm-provider.decentraland.org. If the realm is a world, then is resolved as: worlds-content-server.decentraland.org/world/{ensName}. Otherwise is the host name calculated through the desired realm domain.

The expiration date format in the auth chain now includes milliseconds as it could be required for date parsing on external servers.
The current realm information we were providing to the SDK was wrong, specially the domain field. We were providing the realm base URL, but it is needed to provide the domain only, without the path as it is implemented in kernel: https://github.com/decentraland/unity-renderer/blob/4a2f0ecba4235cf481eebbfccc7d664fea96fc4d/browser-interface/packages/shared/apis/host/EnvironmentAPI.ts#L108. This is necessary to properly form the request to claim the wearable to the rewards api:
curl 'https://rewards.decentraland.org/api/campaigns/c88873d0-b65d-4b48-bd0d-434d9449e05b/rewards' -X POST
  -H 'Content-Type: application/json'
  -H 'Origin: https://decentraland.org'
  -H 'Referer: https://decentraland.org'
  -H 'x-identity-auth-chain-0: {"type":"SIGNER","payload":"0x86b04fbfccae6abfad341b8f6236a86f0a7e3753","signature":""}'
  -H 'x-identity-auth-chain-1: {"type":"ECDSA_EPHEMERAL","payload":"Decentraland Login\nEphemeral address: 0xC1F07E11FDD9d6636237Ae03A9c3f46AcfBF7745\nExpiration: 2024-08-07T16:01:24.545Z","signature":"0x..."}'
  -H 'x-identity-auth-chain-2: {"type":"ECDSA_SIGNED_ENTITY","payload":"post:/api/campaigns/c88873d0-b65d-4b48-bd0d-434d9449e05b/rewards:1722441919214:{\"origin\":\"https://decentraland.org\",\"sceneid\":\"bafkreic5dxyxeqqt63znrsw6burxjxkvqod3y74tmhleorztjn37miqja4\",\"parcel\":\"-55,-121\",\"tld\":\"org\",\"network\":\"mainnet\",\"isguest\":false,\"realm\":{\"hostname\":\"realm-provider.decentraland.org\",\"protocol\":\"v3\",\"servername\":\"main\"},\"signer\":\"decentraland-kernel-scene\"}","signature":"0x..."}'
  -H 'x-identity-timestamp:<PHONE_NUMBER>214'
  -H 'x-identity-metadata: {"origin":"https://decentraland.org","sceneId":"bafkreic5dxyxeqqt63znrsw6burxjxkvqod3y74tmhleorztjn37miqja4","parcel":"-55,-121","tld":"org","network":"mainnet","isGuest":false,"realm":{"hostname":"realm-provider.decentraland.org","protocol":"v3","serverName":"main"},"signer":"decentraland-kernel-scene"}'
  --data-raw '{"campaign_key":"fjg9fbNcSO+R9hn57RiXM8iIc9C2XUtIvQ1DTZRJ4Fs=.w1rItMmAZPh486pOe5bFOjp3jIxgksu/NfRp/M9AWm0=","catalyst":"https://realm-provider.decentraland.org","beneficiary":"0x86b04fbfccae6abfad341b8f6236a86f0a7e3753"}'

This is finally how the dcl login request looks:
curl 'https://v2.lowpolyhub.com:3000/loginWithDcl' -X POST
  -H 'Referer: https://decentraland.org/'
  -H 'x-identity-auth-chain-0: {"type":"SIGNER","payload":"0x43d803d72ba0a4a785f0e46d3b2366cd224e49df","signature":""}'
  -H 'x-identity-auth-chain-1: {"type":"ECDSA_EPHEMERAL","payload":"Decentraland Login\nEphemeral address: 0x08e9E6684fbcC07218742F949b4e001B99F691f3\nExpiration: 2024-07-31T14:53:29.435Z","signature":"0x..."}'
  -H 'x-identity-auth-chain-2: {"type":"ECDSA_SIGNED_ENTITY","payload":"post:/loginwithdcl:1721833159404:{\"origin\":\"https://decentraland.org\",\"sceneid\":\"bafkreic5dxyxeqqt63znrsw6burxjxkvqod3y74tmhleorztjn37miqja4\",\"parcel\":\"-55,-121\",\"tld\":\"org\",\"network\":\"mainnet\",\"isguest\":false,\"realm\":{\"hostname\":\"realm-provider.decentraland.org\",\"protocol\":\"v3\",\"servername\":\"main\"},\"signer\":\"decentraland-kernel-scene\"}","signature":"0x..."}'
  -H 'x-identity-timestamp:<PHONE_NUMBER>313'
  -H 'x-identity-metadata: {"origin":"https://decentraland.org","sceneId":"bafkreic5dxyxeqqt63znrsw6burxjxkvqod3y74tmhleorztjn37miqja4","parcel":"-55,-121","tld":"org","network":"mainnet","isGuest":false,"realm":{"hostname":"realm-provider.decentraland.org","protocol":"v3","serverName":"main"},"signer":"decentraland-kernel-scene"}'
  -H 'Content-Type: application/json'
  -H 'Origin: https://decentraland.org'
  --data-raw '{"wallet":"0x43d803d72ba0a4a785f0e46d3b2366cd224e49df","username":"Lorux0r"}'

How to test the changes?

Follow issue steps
You should be able to claim the wearable
Go to a world, ie: metadyne.dcl.eth and check signed fetch is correctly working
Also on goerli there are several scenes that uses signed fetch. All of them should keep working:

82,-6
78,1
83,-4
81,-9

Our Code Review Standards
https://github.com/decentraland/unity-renderer/blob/master/docs/code-review-standards.md

It is working fine on my end, with two different accounts either in mac and windows:


[feature] A DSL atop OAM primitives

We can also consider support a DSL atop OAM primitives. Candidate include CUE and cdk8s.

We will support CUE as Definition template, and compose-like file for complex needs beyond CLI.

I'm working on it.

fixed by #86


Updated to use FrooxEngine.Store.dll

Previous update broke the mod. All that was needed was to reference FrooxEngine.Store.dll.
If you are fine with it can I release a build if you are not going to?

Panda

Just finished finals. I've already been making local changes, I'll push against main, tag a release, then feel free to PR against that and rebase your changes.

For future reference:

Please add future credits / etc in CONTRIBUTERS.md rather than code comments
The code is released under an MIT license, no need to ask to make your own release / etc

👍


MCT for yolov5n.pt

Issue Type
Feature Request
Source
pip (model-compression-toolkit)
MCT Version
1.8.0
OS Platform and Distribution
Linux Ubuntu 20.04
Python version
3.8
Describe the issue
I tried to use MCT for yolov5n.
I changed tutorials/example_pytorch_mobilenet_v2.py for yolov5n.
I ran changed script and got error messages as follows,
in pytorch_post_training_quantization_experimental,

torch.fx.proxy.TraceError: symbolically traced variables cannot be used as inputs to control flow

Expected behaviour
pytorch_post_training_quantization_experimental pass normally.

But is this limitation of pytorch?
https://pytorch.org/docs/stable/fx.html#limitations-of-symbolic-tracing

if statements where the condition may depend on the input values of the program
↓
  File "/usr/src/app/models/yolo.py", line 207, in forward
    if augment:

Code to reproduce the issue
[in host]
docker pull ultralytics/yolov5
docker run --name yolov5n -it --gpus all ultralytics/yolov5:latest /bin/bash

https://github.com/ultralytics/yolov5/releases/download/v7.0/yolov5n.pt
copy yolov5n.pt to container's /usr/src/app
copy example_pytorch_yolov5n.py to container’s /usr/src/app/

[in container]
pip install model-compression-toolkit
cd /usr/src/app
python example_pytorch_yolov5n.py --representative_dataset_dir ./rep_dataset --batch_size 2

[a minimal snippet of example_pytorch_yolov5n.py]
# from https://github.com/ultralytics/yolov5/export.py
model = Ensemble()
ckpt = torch.load('./yolov5n.pt', map_location='cpu')
model.append(ckpt.fuse().eval() if fuse and hasattr(ckpt, 'fuse') else ckpt.eval())

    core_config = mct.CoreConfig(quantization_config=mct.QuantizationConfig(z_threshold=args.z_threshold))

    quantized_model, quantization_info = mct.ptq.pytorch.quantization_facade.pytorch_post_training_quantization_experimental(
        in_module=model,
        representative_data_gen=representative_data_gen,
        core_config=core_config,
        target_platform_capabilities=target_platform_cap
    )

Log output
cd /usr/src/app
python example_pytorch_yolov5n.py --representative_dataset_dir ./rep_dataset --batch_size 2

Starting Scanning Disk: ./rep_dataset
Finished Disk Scanning: Found 2 files
YOLOv5 ? v7.0-63-gcdd804d Python-3.8.10 torch-1.13.1+cu117 CPU

Fusing layers...
YOLOv5n summary: 213 layers, 1867405 parameters, 0 gradients
Traceback (most recent call last):
  File "example_pytorch_yolov5n.py", line 235, in <module>
    export(args)
  File "/usr/local/lib/python3.8/dist-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
    return func(*args, **kwargs)
  File "example_pytorch_yolov5n.py", line 224, in export
    quantized_model, quantization_info = mct.ptq.pytorch.quantization_facade.pytorch_post_training_quantization_experimental(
  File "/usr/local/lib/python3.8/dist-packages/model_compression_toolkit/ptq/pytorch/quantization_facade.py", line 109, in pytorch_post_training_quantization_experimental
    tg, bit_widths_config = core_runner(in_model=in_module,
  File "/usr/local/lib/python3.8/dist-packages/model_compression_toolkit/core/runner.py", line 92, in core_runner
    graph = read_model_to_graph(in_model,
File "/usr/local/lib/python3.8/dist-packages/model_compression_toolkit/core/runner.py", line 288, in read_model_to_graph
    graph = fw_impl.model_reader(in_model,
  File "/usr/local/lib/python3.8/dist-packages/model_compression_toolkit/core/pytorch/pytorch_implementation.py", line 130, in model_reader
    return model_reader(module, representative_data_gen, self.to_numpy, self.to_tensor)
  File "/usr/local/lib/python3.8/dist-packages/model_compression_toolkit/core/pytorch/reader/reader.py", line 147, in model_reader
    fx_model = fx_graph_module_generation(model, representative_data_gen, to_tensor)
  File "/usr/local/lib/python3.8/dist-packages/model_compression_toolkit/core/pytorch/reader/reader.py", line 87, in fx_graph_module_generation
    symbolic_traced = symbolic_trace(pytorch_model)
  File "/usr/local/lib/python3.8/dist-packages/torch/fx/_symbolic_trace.py", line 1070, in symbolic_trace
    graph = tracer.trace(root, concrete_args)
  File "/usr/local/lib/python3.8/dist-packages/torch/fx/_symbolic_trace.py", line 739, in trace
    (self.create_arg(fn(*args)),),
  File "/usr/src/app/models/yolo.py", line 207, in forward
    if augment:
  File "/usr/local/lib/python3.8/dist-packages/torch/fx/proxy.py", line 298, in __bool__
    return self.tracer.to_bool(self)
  File "/usr/local/lib/python3.8/dist-packages/torch/fx/proxy.py", line 174, in to_bool
    raise TraceError('symbolically traced variables cannot be used as inputs to control flow')
torch.fx.proxy.TraceError: symbolically traced variables cannot be used as inputs to control flow

Hi,
Thank you for bringing this issue to our attention. We appreciate your support.
Upon reviewing the issue, we have identified that the problem is caused by the limitations of an external library that our project relies on: "torch.fx".
The FX toolkit helps us to acquire a static graph representation from the PyTorch models which is essential for performing model compression manipulation like batch norm folding, for instance.
We are currently in the process of creating a tutorial that showcases how to utilize MCT with object detection models (such as YOLO). Once this tutorial is completed, we will notify you.
If you have any further questions, please feel free to reach out.
Regards
Idan

Hi,
The tutorial is available here: https://github.com/sony/model_optimization/tree/main/tutorials/quick_start
Best regards
Idan


Plugin scaffolding support

Describe the feature
https://github.com/swc-project/swc/tree/main/tests/rust-plugins/swc_internal_plugin is a simple example how someone can write a plugin.
But it is a moving target, which is constantly changing for internal integration test purpose. Also it is not good way to officially document / instruct a way to write plugin by pointing some internal test fixture.
There are 2 things roughly needed to write plugin - first, setting up package itself, then apply multivalue polyfill to enable multivalue without rustc's upstream support (which is not working fully yet).
We'd like to have simple, lightweight cli support to provide both capability for the plugin author.
Babel plugin or link to the feature description
No response
Additional context
No response

closed via #3536.


[Feature] Support GuardDuty

Amazon just announced a new security service named GuardDuty this week. https://aws.amazon.com/guardduty/
It has some useful rules that would be difficult to implement in StreamAlert, such as detecting logins from new ASNs, which requires correlation with past events.  When GuardDuty detects something, it writes to CloudWatch Events.  My request is:

Support the schema used by GuardDuty alerts.
Create a StreamAlert rule to alert on any GuardDuty alerts.

From the User Guide (p 44, https://docs.aws.amazon.com/guardduty/latest/ug/guardduty-ug.pdf), the format of the CloudWatch events is:
{
 "version": "0",
 "id": "cd2d702e-ab31-411b-9344-793ce56b1bc7",
 "detail-type": "GuardDuty Finding",
 "source": "aws.guardduty",
 "account": "111122223333",
 "time": "1970-01-01T00:00:00Z",
 "region": "us-east-1",
 "resources": [],
 "detail": {COMPLETE_GUARDDUTY_FINDING_JSON}
 }

We'll ship this in Q1 :)

Sorry for the delays. Finally started looking at this.  GuardDuty sends events to CloudWatch Events and they are in the same format as the existing events (https://github.com/airbnb/streamalert/blob/0bb9ae2b75f0ce9791180eb15b73fd20bf4157e3/conf/logs.json#L1096). Because StreamAlert already watches CloudWatch Events by default and the schema already matches, there isn't much to do other than create rules.  I'll have a PR for a sample rule shortly.  This will just be:
@rule(logs=['cloudwatch:events'], matchers=[], outputs=['slack:sample-channel'])
def guardduty(rec):
  if rec['detail-type'] == 'GuardDuty Finding':
    return True
  return False

We could turn on GuardDuty, similar to how we turn on CloudTrail.  GuardDuty is a little more expensive ($4/mo + potentially some extra fees) and it would require adding some privileges to StreamAlert.  That work should fall under a separate ticket if people are interested.  From the UI, "integrating" GuardDuty with StreamAlert is just turning GuardDuty on, because it flows directly into StreamAlert's existing data flows.


Implement network calls

Current dependencies on/for this PR:

main

PR #5

PR #6   👈

This stack of pull requests is managed by Graphite.

Merge activity

Nov 19, 2:51 PM: @kadhirvelm started a stack merge that includes this pull request via Graphite.


Unable to find MicrosoftWebDriver

Summary
Webdrivers is unable to find edge driver in the url: https://www.microsoft.com/en-us/download/details.aspx?id=48212 as the link is no longer available.
Debug Info
Please provide the following information for bug reports:

Webdrivers version: 4.1.2
Ruby version: jruby-<IP_ADDRESS>
Operating system / CI Environment: Windows 10
Browser and version: Microsoft Edge (Chromium) <IP_ADDRESS>

Expected Behavior
To download the latest edge driver.
Actual Behavior
  Unable to find MicrosoftWebDriver. Please download the server from
  https://www.microsoft.com/en-us/download/details.aspx?id=48212 and place it somewhere on your PATH.
  More info at https://github.com/SeleniumHQ/selenium/wiki/MicrosoftWebDriver.
   (Selenium::WebDriver::Error::WebDriverError)
  ./features/support/hooks.rb:6:in `block in <main>'
  c:\jruby-<IP_ADDRESS>\bin\cucumber:23:in `<main>'
    Given I am on the wb table page

@anilreddy Are you using a build from the Beta channel? The Dev and Canary builds are at version 78.x as of today.
Please provide the debug log so we can see what exactly is going on here.

@kapoorlakshya am using the Microsoft Edge (Chromium) Beta (<IP_ADDRESS>) and I have added the debug log and it is not generating any log file and am getting the same error which I mentioned above. Below is my config in the hooks.rb file
Before do
  Webdrivers.logger.level = :DEBUG
  Webdrivers::Edgedriver.required_version = '<IP_ADDRESS>'
  @browser = Watir::Browser.new :edge
  @browser.window.maximize
end

Ah, that's because watir is attempting to launch the original Edge browser (EdgeHTML) and not the Chromium one.
Unfortunately, you'll not be able to use the new Edge with watir right now. watir is using Selenium v3 and you need v4 to use the new Edge browser. You'll have to wait for the next major release (in November-December I think) that supports Selenium v4.
I'm closing this issue, but please feel free to continue the discussion if you have any questions.

Also, we removed support for MicrosoftWebDriver in v3.9.0, so you'll have to manually download it and provide it to Selenium if you want to use the original Edge browser.

Ah, that's because watir is attempting to launch the original Edge browser (EdgeHTML) and not the Chromium one.
Unfortunately, you'll not be able to use the new Edge with watir right now. watir is using Selenium v3 and you need v4 to use the new Edge browser. You'll have to wait for the next major release (in November-December I think) that supports Selenium v4.
I'm closing this issue, but please feel free to continue the discussion if you have any questions.

@kapoorlakshya could you please confirm whether the recent watir 6.17v supports edge chromium or not. I am currently using watir 6.16v and unable to launch edge chromium.

Ah, that's because watir is attempting to launch the original Edge browser (EdgeHTML) and not the Chromium one.
Unfortunately, you'll not be able to use the new Edge with watir right now. watir is using Selenium v3 and you need v4 to use the new Edge browser. You'll have to wait for the next major release (in November-December I think) that supports Selenium v4.
I'm closing this issue, but please feel free to continue the discussion if you have any questions.

@kapoorlakshya could you please confirm whether the recent watir 6.17v supports edge chromium or not. I am currently using watir 6.16v and unable to launch edge chromium.

@maheshmohan27 I don't believe so because Edge (chromium) support was added in Selenium 4.0.0.alpha3 (see changelog) and watir is still using 3.x in the gemspec. You might be able to point Selenium to the Edge (chromium) binary instead of Chrome and trick it into thinking it's running Chrome, but no promises that it'll work 😄

@maheshmohan27 I don't believe so because Edge (chromium) support was added in Selenium 4.0.0.alpha3 (see changelog) and watir is still using 3.x in the gemspec. You might be able to point Selenium to the Edge (chromium) binary instead of Chrome and trick it into thinking it's running Chrome, but no promises that it'll work 😄

@maheshmohan27 I don't believe so because Edge (chromium) support was added in Selenium 4.0.0.alpha3 (see changelog) and watir is still using 3.x in the gemspec. You might be able to point Selenium to the Edge (chromium) binary instead of Chrome and trick it into thinking it's running Chrome, but no promises that it'll work 😄

Thank you for the response.

@maheshmohan27 I don't believe so because Edge (chromium) support was added in Selenium 4.0.0.alpha3 (see changelog) and watir is still using 3.x in the gemspec. You might be able to point Selenium to the Edge (chromium) binary instead of Chrome and trick it into thinking it's running Chrome, but no promises that it'll work 😄

Thank you for the response.


Unable to build any images due to use of an internal registry in makefile and dockerfiles

https://github.com/neuvector/neuvector/blob/main/Makefile#L3-L4
In the Makefile as well as all of the dockerfiles, the registry is set to a hard-coded and presumably private/internal IP. This needs to be fixed to allow end-users to build Neuvector locally

This is a preview version and build tools are not supported at this time. This will be added in the future.

@gkosaka Seeing as the issue is not resolved and will be fixed in the future, why was this issue closed?
Keeping this issue open also educates any end-users (like myself) who wish to contribute to the project about the current state of the repository and project.

Sounds good let's keep it open.

Thanks @gkosaka! Would you have an ETA for the build system to be opened as well?

I've linked a milestone for build tools availability. In the interim, images can be pulled from docker hub at:
neuvector/manager.preview:5.0.0-preview.1
neuvector/controller.preview:5.0.0-preview.1
neuvector/enforcer.preview:5.0.0-preview.1
neuvector/scanner.preview:latest
neuvector/updater.preview:latest

Looking forward to your updates

I've linked a milestone for build tools availability. In the interim, images can be pulled from docker hub at: neuvector/manager.preview:5.0.0-preview.1 neuvector/controller.preview:5.0.0-preview.1 neuvector/enforcer.preview:5.0.0-preview.1 neuvector/scanner.preview:latest neuvector/updater.preview:latest

I find that both makefile and Dockerfile use a lot of IP addresses, which makes it impossible for me to compile. I am looking forward to your updates on this area, thank you very much.

Is there any progress in the milestone ？Thanks

any update this build problem?

Hope this issue could be solved soon. We added a kubernetes orchestration flavor for our k8s distribution, and want to verify it on our platform, but currently stuck on the local build.

You can just compile it based on the image he's released

Hope this issue could be solved soon. We added a kubernetes orchestration flavor for our k8s distribution, and want to verify it on our platform, but currently stuck on the local build.

You can just compile it based on the image he's released

The build problem is not just about the base image.
The neuvector/build image is not released yet, and run make_fleet.sh in normal golang:1.14 container will failed.
Some Makefile targets are expected to be executed outside project(eg: copy_*) and some expected to be executed inside project(eg: fleet).

Hope this issue could be solved soon. We added a kubernetes orchestration flavor for our k8s distribution, and want to verify it on our platform, but currently stuck on the local build.

You can just compile it based on the image he's released

The build problem is not just about the base image. The neuvector/build image is not released yet, and run make_fleet.sh in normal golang:1.14 container will failed. Some Makefile targets are expected to be executed outside project(eg: copy_*) and some expected to be executed inside project(eg: fleet).

这些你用它原有的不就行了

Hope this issue could be solved soon. We added a kubernetes orchestration flavor for our k8s distribution, and want to verify it on our platform, but currently stuck on the local build.

You can just compile it based on the image he's released

The build problem is not just about the base image. The neuvector/build image is not released yet, and run make_fleet.sh in normal golang:1.14 container will failed. Some Makefile targets are expected to be executed outside project(eg: copy_*) and some expected to be executed inside project(eg: fleet).

这些你用它原有的不就行了

There neuvector/build image is not released, the released production image does not have build tools.

Hope this issue could be solved soon. We added a kubernetes orchestration flavor for our k8s distribution, and want to verify it on our platform, but currently stuck on the local build.

You can just compile it based on the image he's released

The build problem is not just about the base image. The neuvector/build image is not released yet, and run make_fleet.sh in normal golang:1.14 container will failed. Some Makefile targets are expected to be executed outside project(eg: copy_*) and some expected to be executed inside project(eg: fleet).

Hope this issue could be solved soon. We added a kubernetes orchestration flavor for our k8s distribution, and want to verify it on our platform, but currently stuck on the local build.

You can just compile it based on the image he's released

The build problem is not just about the base image. The neuvector/build image is not released yet, and run make_fleet.sh in normal golang:1.14 container will failed. Some Makefile targets are expected to be executed outside project(eg: copy_*) and some expected to be executed inside project(eg: fleet).

这些你用它原有的不就行了

There neuvector/build image is not released, the released production image does not have build tools.

你加我wx聊吧，lp948630594


No support for ARM architecture

Will support for ARM be added in the future? With IBM's push towards the cloud and IoT, it is really frustrating this is still not supported.

This would be really great. I opened this Issue a few years ago but I did not get a positive response: https://github.com/IBM-Cloud/ibm-cloud-cli-release/issues/7

looking forward for this feature

I am currently implementing AI services on nVIDIA nano with IBM Edge Application Manager, it will be very helpful if I can use ibmcloud to store the docker container in IBM Container Registry.

It is more important now to have ARM options looking at how well M1 is doing.
I think it is inevitable.

ARM support is desperately needed. Developers are buying Apple M1 Silicon Macs because they are screaming fast with all day battery life and while the IBM Cloud CLI does work with Rosetta 2 emulation on macOS, many developers are developing in virtual machines and Docker containers which are all arch64 linux on the M1 Mac and the ibmcloud CLI does not work.
We really need native support for ARM now more than ever. Apple M1 is the future. It's only going to get worse.

+1 Not having arm support is a massive let down...

This is now needed for all of us using an Apple M1.

That would be nice for Raspberry Pi users too.

We have added as-is versions for ARM now. Please see https://github.com/IBM-Cloud/ibm-cloud-cli-release/releases/tag/v2.6.0 for more the latest available.

@steveclay  Congratulation on acknowledging the ARM community but... I still get the following error:
$ curl -fsSL https://clis.cloud.ibm.com/install/linux | sh
Unsupported Linux architecture: aarch64. Quit installation.

I expect the ARM version to be installed.

I have downloaded and installed Linux 64 bit ARM binary on my Raspberry Pi OS(64bit). And I have confirmed it worked! Thanks!!

@dotnsf Did you have to modify the install script? There is no way to get it installed unless you modify the install script or copy the files manually. Then it works, but it doesn't even install to begin with.
I added aarch64 as a case statement in the install script like this:
MARCH=$(uname -m | tr '[:upper:]' '[:lower:]')
case "$MARCH" in
"aarch64") ;;
"ppc64le") ;;
"x86_64") ;;
"x86_32") ;;
"s390x") ;;
*)

and then it worked, but I shouldn't have to do that.

@rofrano I didn't use installer version of this. I had just downloaded and expanded from tgz binary of "Linux 64 bit ARM (as-is)". It worked in my Raspberry Pi OS.


[IDEA] Revise the definition of Ports and the use of provided/required

There are a couple of issues with the way we currently define ports:
1/ The semantics of provided and required is a bit tricky, it is mainly used for:
a) All required ports should be connected for a component to run - but optional port are more explicit to do that.
b) It is only possible to connect required ports with provided ports - but actually the messages and in which direction they are sent or received are a better indication for the validity of a connector.
2/ Whenever we define ports we typically also define the "mirror" port which contains the same messages but swapping send and receive.
What if we could define the port once (like we do for messages) and the use it in componenents. The keywords "provided" and "required" can then be used for specifying if the port has to be mirrored or not.
Let me give an example to try to clarify:

thing fragment TimerMsgs {

	message timer_start()
	message timer_cancel()
	message timer_timeout()

	port timer {
		receives timer_start, timer_cancel
		sends timer_timeout
	}
}

thing MyTimer includes TimerMsgs {

	provides timer

	[...]
	event timer?timer_start()
	[...]

}

thing MyClient includes TimerMsgs {

	requires timer

	[...]
	timer!timer_start()
	[...]

}

// By default, the name of the port is the name of the port in the definition (here timer).
// If we need different names or several instance of the same port we can have

requires timer as timer1, timer as timer2

or

requires timer1:timer, timer2:timer

In this example I have assumed that the definition "port" is done from the point of view of the provider of the service so when a thing "provides" the port it is left as-is but when a thing requires a port, it is "mirrored".
This could help be a bit more compact because I see in my programs that the "double" definition of mirrored ports is a pain when making modifications.
Any opinions? Any good reasons for our current approach which I have missed?

I like it, it is definitely annoying (and error prone) to make the mirror port every time we use ports. And sometimes I make two fragments with each of those mirror ports (like a _cli and _srv) to keep them close together so that I don't make a mistake. Your proposal would automate that process.
I guess a port also defines a "service", so using your proposal, we would have definitions of services using messages and ports, and then the Things would require or provide a service (which I guess is in line with the original idea of the ports?).
One thing I don't fully understand, is how this changes anything related to your points in 1/? Practically, this proposal would just generate required and provided ports automatically? The semantics and connectors would be exactly the same?
Also, what about internal ports? Would that now be a completely different concept? It wouldn't be too bad if it is, related to #161

Somehow back to what I proposed in #3, 6 years ago...

But as far as I can see, this seems to be an OK proposition.


Revise the descriptor API

An attempt to fix #125.
The key change here is the new include_in_config property of USBDescriptor, which specifies whether the descriptor should be included in a GET_CONFIGURATION response.
Descriptors with this property set to True will appear following their parent in the GET_CONFIGURATION response, in the same order in which they were added with the add_descriptor() method. They will not be fetchable with a GET_DESCRIPTOR request, and do not require the number property to be set.
Descriptors with this property set to False will not be included in a GET_CONFIGURATION response. They can be fetched with a GET_DESCRIPTOR request, and must have the number property set to an integer value, so that they can be retrieved by the identifying tuple of (type_number, number).
The USBClassDescriptor class is deprecated, but retained for backwards compatibility. It now simply sets include_in_config to True.
Stacked on #122 for now.

This is dependent on #122 but otherwise ready to go.

Rebased now that #122 is merged.


fix Vue warn in PhantomJS browser

fix ERROR: '[Vue warn]: Duplicate keys detected: 'formly_{field.key}'. This may cause an update error. in PhantomJS browser

@klarkc thanks heaps for the PR! I've instead chosen to make a new PR which doesn't have the modified package.json file, includes a unit test, and favours ${field.key} over '' + field.key. Just to keep everything up to spec :) You can check out that PR here #24


Could not capture all the windows desktop dialog in server side

Describe the bug
In Serve side : Windows Desktop App provide a new feature to avoid capturing from DXGi. once the display affinity setting (WDA_EXCLUDEFROMCAPTURE or WDA_MONITOR) is being set by any Windows APP dialog(supported from Windows 10 2004 version). the DXGi would not capture it. therefore . the dialog would be shown as a black board in client side.
To Reproduce
Steps to reproduce the behavior:

In server side .Open Microsoft Teams.
Set up a meeting.
sharing your desktop or document in any way (Keep it online).
In Client side. connect to server and you will see the “Sharing Window” is black

Expected behavior
what you see in server side, will be the same in client side
Screenshots
Application details
Normal case
Environment (please complete the following information):

OS: [Windows]
Version/Distribution: Windows 10 22h2
Architecture: [x64]:

So basically you'd like that what is supposed to happen doesn't happen. In other words, we shall not capture with DXGI as it works this way with windows that have been flagged...

@podunyu so basically the server is hiding stuff and you want the client do something about that?
I´m all ears for suggestions how to do that :laughing:
you could replace the windows RDP server here or patch your windows to not do that, but nothing we can do about that ;)


Cancelling Clothing Purchase removes Armor

Noticed that when you choose to not save changes, it removes armor every time

@llbooya Its an issue with all fivem-appearance variants out there. I will need to take a look to see if this can actually be fixed or not

Implemented in #179


chore: Documentation for Azure custom metrics

Dev Review rules:

3 PR max
at the second feedback, prefer meet instead of github conversation

Definition of Done:

[ ] Testing
[ ] No error logs in web console (except iFrame errors)
[ ] Create or Update documentation (README file for users/integrators)
[ ] Identify breaking changes with conventional commits
[ ] Check the test page in order to manually validate the feature
[ ] Clean code (regroup configuration variable, typos, no console.log...)

Ok, thanks


Catbot Responds Incorrectly on default corpus.

After Executing the below code after some interaction if we interact with the same words it answers in correctly.
Code:
from chatterbot import ChatBot
#from chatterbot.trainers import ListTrainer
from chatterbot.trainers import ChatterBotCorpusTrainer
from chatterbot.response_selection import get_random_response
chatbot = ChatBot('Green_Apex',response_selection_method=get_random_response,)
chatbot.set_trainer(ChatterBotCorpusTrainer)
chatbot.train("chatterbot.corpus.english")
"""-------------------------------------------------------------------------------"""
THRESHOLD= float("0.7")
i=""
while i!="Bye":
	i = input("YOU : ")
	statement = chatbot.input.process_input(i)
	confidence = chatbot.logic.process(statement)
	#confidence = str(response.confidence)
	response = chatbot.get_response(i)
	confidence = float(response.confidence)
	#print(response.confidence)
	#print (confidence)
	if i!="bye":
		if confidence > THRESHOLD:
			response = chatbot.output.process_response(response)
			response = str(response)
			print("Bot : "+response)
		else:
			response = "I don't understand what you mean"
			response = str(response)
			print("Bot : "+response)
	else:
		print("Bot : Bye Have a Nice Day")
		quit()

Chat response:

After some irregular inputs the input for pizza has to that irregular inputs

I have see the database files and saw that the data base inputs the responses to the same database. I don't want the database to be updated besides the corpus data. Chat bot should only respond to the corpus data. Can anyone show me the way and resolve  my query

Try to set your bot in read only mode https://chatterbot.readthedocs.io/en/stable/examples.html#read-only-mode

I have update the Query above. After Adding the read_only= true the response is as follows
can you please review my code
import chatterbot
from chatterbot import ChatBot
from chatterbot.trainers import ListTrainer
from chatterbot.trainers import ChatterBotCorpusTrainer
from chatterbot.response_selection import get_random_response
chatbot = ChatBot('Green_Apex',response_selection_method=get_random_response,read_only=True)
chatbot.set_trainer(ChatterBotCorpusTrainer)
chatbot.train("chatterbot.corpus.english")
"""-------------------------------------------------------------------------------"""
THRESHOLD= float("0.2")
i=""
while i!="Bye":
	i = input("YOU : ")
	statement = chatbot.input.process_input(i)
	confidence = chatbot.logic.process(statement)
	#confidence = str(response.confidence)
	response = chatbot.get_response(i)
	confidence = float(response.confidence)
	#print(response.confidence)
	print (confidence)
	if i!="bye":
		if confidence > THRESHOLD:
			response = chatbot.output.process_response(response)
			response = str(response)
			print("Bot : "+response)
		else:
			response = "I don't understand what you mean"
			response = str(response)
			print("Bot : "+response)
	else:
		print("Bot : Bye Have a Nice Day")
		quit()

@vkosuri  I got a trick, used read_only = True in my main script but to train my corpus files executed from different file which will remove the ambiguity with read_only see the sample code below
train.py
from chatterbot import ChatBot
from chatterbot.trainers import ChatterBotCorpusTrainer
chatbot = ChatBot('tushar')

class Trainer :
    x=str(input("Do you want to Train the model ?\nType Y for Yes and N for No : "))
    if x =="Y"or x =="y" :
        chatbot.set_trainer(ChatterBotCorpusTrainer)
        chatbot.train("chatterbot.corpus.english")
    elif x == "N" or x =="n":
        pass

bot.py
from train import Trainer
--------------------Rest of your code here---------------


Fix: Use https instead of http for download URLs.

This PR updates download URLs to use https instead of http, for safety.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@jemc

Please sign the CLA here.

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.

Nice catch.  Thanks

Thanks for the quick merge!


AZC0004 doesn't warn when sync/async method return types don't match

AZC0004 should give a warning when the return types of sync and async methods don't match.  It does not do this currently.

@annelo-msft Do you expect the analyzer to return the same message as AZC0004?

DO provide both asynchronous and synchronous variants for all service methods.

Even though the message is technically correct, it would be more helpful for users if we explicitly point out that the mistake is the mismatch of the return type. Could we expand the description for clarity or even introduce a new rule? I’m not sure if it is possible to modify the descriptions or add new guidelines in the docs.
What do you think?


Add ptpython to the shell_plus docs

This is already implemented in https://github.com/django-extensions/django-extensions/blob/master/django_extensions/management/commands/shell_plus.py but not documented. This PR corrects that.
I tested the resolution order to confirm this update is correct.
I also changed the capitalisation on bpython since that's how the project refers to itself: https://bpython-interpreter.org/about.html

Coverage remained the same at 38.41% when pulling 2ebf6fccd38069a72710ebdfa43a304e20216ae1 on keybits:patch-1 into 6ff63bb49eef4ad36805fe237904811ecaea30c0 on django-extensions:master.

Thanks 1


fix(client): add doctype to preview iframe

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #56660
The preview iframe is running in quirks mode because it's missing the doctype. Example of incorrect output can be found in the issue.
I tried a few things that didn't work, so I ended up adding it to the frame.write(). It's a simple solution and seems to work just fine.

While this does fix the specific issue, I don't think we want to force the preview out of quirks mode. If at all possible, we want the preview to exactly represent what the learner has written.
I was just looking into this and it isn't too tricky to make it conditional on the learner having added <!DOCTYPE html> in their code.
If it's alright with you, I can push my changes.

If at all possible, we want the preview to exactly represent what the learner has written.

The doctype isn't missing in the written code. For example in step 9 of the challenge linked to in the issue a doctype is already in the HTML that was written.
I just think all our previews are missing the doctype not matter what.

I do get what you are saying. If the camper had missed adding the doctype we would be fixing that. I'm just not sure what else to do. The iframe has to have a doctype and it doesn't get one even if it was added to the HTML in the editor.
BTW, I just noticed that it is also quite broken for the certificate projects. Here we somehow are adding the doctype to the body element.

Yep, that's all correct, to my understanding.
There's a couple of gotchas, but, in short, my fix was to prefix the content with <!DOCTYPE html> when the camper has added it. When I tried it, I ended up calling frame.write(includeDoctype ? '<!DOCTYPE html>' + content : content); (more or less). includeDoctype's value was conditional on the presence or absence of the doctype, unsurprisingly.

I moved the code to writeContentToFrame and pass writeToFrame the doctype if I find it in the sources.contents.


vmount updates

This makes sure the vmount point-to-location works & updates the gazebo submodule
gazebo demo: https://www.youtube.com/watch?v=7zJ9nNOydes
fyi @thedevleon

Nice!


refactor: add store_id foreign key to menu

개요
store_id(foreign key) was added to the menu.
작업사항
(Inner)joined the menu table and the store table
변경로직
get_store_menu function was added to crud.py.
An inner join occurs in this function.

고생하셨어요


authenticateSession test helper with Ember 3.0.0

Hello!
After upgrading our application to Ember 3.0.0, we have experienced the following issue: the authenticateSession helper seems to authenticate the user properly but fails to properly return its value: return app.testHelpers.wait(); in the helper code breaks with Cannot read property 'wait' of undefined.

This should be fixed in ESA 1.5.0: https://github.com/simplabs/ember-simple-auth/releases/tag/1.5.0

We have 1.5.1 in our package. I've tried it with 1.5.0 as well, with the same result. It fails on this line: https://github.com/simplabs/ember-simple-auth/blob/1.5.1/test-support/helpers/ember-simple-auth.js#L17

This works fine with Ember 3.0.0 in the dummy app. If you also updated ember-qunit to 4.3+ (I think) you'll have to use the new test helpers added with ESA 1.5.0 though. Maybe that's the issue?


Crowdsec logo in README.md shows NGINX logo

Hello,
I just discovered the project and the first thing I noticed that kind of confused my was that the logo in the readme shows an nginx logo not a HAProxy logo. Is this intentional?

No it wasnt, we started this project from our nginx project so there was a slight overlap. This is now fixed, thank you for bringing it to our attention as we may have missed it 😄


Add my first file

I've created a new branch, created a file, and made a commit. Then I am pulling a request.

I was able to successfully merge the branch and delete it.


Air temperature goes to 0 on several devices.

Checklist

[X]  I am not using Home Assistant. Or: a developer has told me to come here.
[X] I have checked the troubleshooting section and my problem is not described there.
[X] I have read the changelog and my problem is not mentioned there.

Deploy method
Docker
Z-Wave JS UI version
8.14.2.c234053
ZwaveJS version
10.16.0
Describe the bug
As of 26 of march I got several devices which out of the blue, the temperature drops to 0 and then after a period of time it goes back.

I noticed this just a few day ago but it began one month ago and, I believe, two version back.
This happens on several devices but before a month ago I never had the problem.
To Reproduce
I can't not reproduce it myself, it comes as it is.
Expected behavior
Maybe connection problems ?
Additional context
Log from z-ui:
2023-05-01 00:39:31.447 INFO Z-WAVE: [Node 055] Is now awake
2023-05-01 00:39:31.739 INFO Z-WAVE: [Node 055] Metadata updated: 49-0-Air temperature
2023-05-01 00:39:31.741 INFO Z-WAVE: [Node 055] Value updated: 49-0-Air temperature 23.4 => 0
2023-05-01 00:39:31.982 INFO Z-WAVE: [Node 055] Metadata updated: 49-0-Illuminance
2023-05-01 00:39:31.984 INFO Z-WAVE: [Node 055] Value updated: 49-0-Illuminance 0 => 0
Log from zwavejs:
2023-04-30T22:39:31.445Z DRIVER « [Node 055] [REQ] [BridgeApplicationCommand]
│ RSSI: -75 dBm
└─[WakeUpCCWakeUpNotification]
2023-04-30T22:39:31.446Z CNTRLR « [Node 055] received wakeup notification
2023-04-30T22:39:31.448Z CNTRLR   [Node 055] The node is now awake.
2023-04-30T22:39:31.450Z CNTRLR » [Node 055] Multilevel Sensor CC values may be stale, refreshing...
2023-04-30T22:39:31.451Z CNTRLR » [Node 055] querying Air temperature sensor reading...
2023-04-30T22:39:31.461Z SERIAL » 0x011000a901370431040100250000000083e6                              (18 bytes)
2023-04-30T22:39:31.462Z DRIVER » [Node 055] [REQ] [SendDataBridge]
│ source node id:   1
│ transmit options: 0x25
│ callback id:      131
└─[MultilevelSensorCCGet]
sensor type: Air temperature
scale:       Celsius
2023-04-30T22:39:31.468Z SERIAL « [ACK]                                                                   (0x06)
2023-04-30T22:39:31.469Z SERIAL « 0x010401a90152                                                       (6 bytes)
2023-04-30T22:39:31.470Z SERIAL » [ACK]                                                                   (0x06)
2023-04-30T22:39:31.476Z DRIVER « [RES] [SendDataBridge]
was sent: true
2023-04-30T22:39:31.595Z SERIAL « 0x011d00a98300000c01b57f7f7f7f00000308000000030100007f7f7f7f7f06    (31 bytes)
2023-04-30T22:39:31.596Z SERIAL » [ACK]                                                                   (0x06)
2023-04-30T22:39:31.598Z DRIVER « [REQ] [SendDataBridge]
callback id:            131
transmit status:        OK, took 120 ms
repeater node IDs:      8
routing attempts:       1
protocol & route speed: Z-Wave, 100 kbit/s
ACK RSSI:               -75 dBm
ACK RSSI on repeaters:  N/A
ACK channel no.:        0
TX channel no.:         0
2023-04-30T22:39:31.735Z SERIAL « 0x010f00a80001370631050122000000b5ca                                (17 bytes)
2023-04-30T22:39:31.737Z SERIAL » [ACK]                                                                   (0x06)
2023-04-30T22:39:31.738Z CNTRLR   [Node 055] [Multilevel Sensor] Air temperature: metadata updated  [Endpoint 0]
2023-04-30T22:39:31.740Z CNTRLR   [Node 055] [~] [Multilevel Sensor] Air temperature: 23.4 => 0     [Endpoint 0]
2023-04-30T22:39:31.742Z DRIVER « [Node 055] [REQ] [BridgeApplicationCommand]
│ RSSI: -75 dBm
└─[MultilevelSensorCCReport]
type:  Air temperature
scale: Celsius
value: 0
2023-04-30T22:39:31.745Z CNTRLR « [Node 055] received current Air temperature sensor reading: 0 °C
2023-04-30T22:39:31.746Z CNTRLR » [Node 055] querying Illuminance sensor reading...
2023-04-30T22:39:31.748Z CNTRLR   [Node 055] No scale preference for sensor type 3, using the last-used scale 1
2023-04-30T22:39:31.758Z SERIAL » 0x011000a901370431040308250000000084eb                              (18 bytes)
2023-04-30T22:39:31.759Z DRIVER » [Node 055] [REQ] [SendDataBridge]
│ source node id:   1
│ transmit options: 0x25
│ callback id:      132
└─[MultilevelSensorCCGet]
sensor type: Illuminance
scale:       Lux
2023-04-30T22:39:31.765Z SERIAL « [ACK]                                                                   (0x06)
2023-04-30T22:39:31.766Z SERIAL « 0x010401a90152                                                       (6 bytes)
2023-04-30T22:39:31.766Z SERIAL » [ACK]                                                                   (0x06)
2023-04-30T22:39:31.766Z DRIVER « [RES] [SendDataBridge]
was sent: true
2023-04-30T22:39:31.906Z SERIAL « 0x011d00a98400000d01b57f7f7f7f00000308000000030100007f7f7f7f7f00    (31 bytes)
2023-04-30T22:39:31.907Z SERIAL » [ACK]                                                                   (0x06)
2023-04-30T22:39:31.907Z DRIVER « [REQ] [SendDataBridge]
callback id:            132
transmit status:        OK, took 130 ms
repeater node IDs:      8
routing attempts:       1
protocol & route speed: Z-Wave, 100 kbit/s
ACK RSSI:               -75 dBm
ACK RSSI on repeaters:  N/A
ACK channel no.:        0
TX channel no.:         0
2023-04-30T22:39:31.978Z SERIAL « 0x010f00a8000137063105030a000000b5e0                                (17 bytes)
2023-04-30T22:39:31.979Z SERIAL » [ACK]                                                                   (0x06)
2023-04-30T22:39:31.981Z CNTRLR   [Node 055] [Multilevel Sensor] Illuminance: metadata updated      [Endpoint 0]
2023-04-30T22:39:31.983Z CNTRLR   [Node 055] [~] [Multilevel Sensor] Illuminance: 0 => 0            [Endpoint 0]
2023-04-30T22:39:31.984Z DRIVER « [Node 055] [REQ] [BridgeApplicationCommand]
│ RSSI: -75 dBm
└─[MultilevelSensorCCReport]
type:  Illuminance
scale: Lux
value: 0
2023-04-30T22:39:31.987Z CNTRLR « [Node 055] received current Illuminance sensor reading: 0 Lux
2023-04-30T22:39:32.270Z SERIAL « 0x011300a80001740a320221340000008b000000b722                        (21 bytes)
2023-04-30T22:39:32.271Z SERIAL » [ACK]                                                                   (0x06)

@Miesje This seems to me a device problem, I mean as logs say the device is reporting 0 so 🤷🏼♂
So firstly I wolud suggest to check if there are fw updates for that device and then ask support to the manufacturer.
cc @AlCalzone


fix(modal): fix unexpected scroll on closing

General summary
Fix a bug only present in the product, releasing an alpha version to test it

Tested in the product :heavy_check_mark:


help

how to select data when table is inserted with sqltransactions

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

ID: 2fdf0875-ec95-805c-d1eb-774bb032822b
Version Independent ID: 8facbdab-29f6-e657-460c-18b4346757e3
Content: SqlTransaction Class (System.Data.SqlClient)
Content Source: xml/System.Data.SqlClient/SqlTransaction.xml
Product: dotnet-api
GitHub Login: @douglaslMS
Microsoft Alias: douglasl

@GalaxyFour Can you please clarify the case that you're asking about? Do you mean, how to select data when you have created the table inside a transaction, or how to select data when you have inserted the data inside a transaction? Are you running into a specific error or failure that you're trying to resolve?

@GalaxyFour Can you please clarify your question? See my comment from last week. Thanks.

i use trans = con.BeginTransaction(IsolationLevel.ReadUncommitted, "transInsert")
to edit or select inserted data in transaction


docs: add vault plugin upgrade docs

I guess that is probably best, then if there are any "improvements" to those docs, they should be done by Hashicorp. :) We have learned that with an HA cluster in kubernetes that the safest was to actually install another version, then change to the new version. So we always have two versions installed in our cluster... sortof a rolling update, I guess ;)


Improve Qualification UI performance

Is your feature request related to a problem? Please describe.
After adding SQL view to the Qualification UI , the APP-detail view is initialized loading 4 different sections:

app details,
stages
execs,
then SQL

with heavy workload, the UI may take more time to load.
Describe the solution you'd like
There are some improvement that can be done to the UI code to improve the performance. However, we are still restricted by the lack of web server which limits capability to use callbacks

Use asynchronous functions to create the datatables in parallel

async function parallelInit() {
  await Promise.all([
      (async () => console.log(await loadStagesDataTables()))(),
      (async () => console.log(await loadExecsDataTables()))(),
      (async () => console.log(await loadSQLDataTables()))(),
  ])
}

Now, as the Qualification tool is stable, we can cleanup the JS code. For example, the JS code creates a map to rename the fields. This was needed to isolate the UI as the scala code used to change field names and classes during the implementation.
Getting rid of those mapping steps implies that we can reduce the overhead of iterating through all the applications processing their fields.

Outdated. UI is currently not on the priority list.


How open multiple window?

Is your feature request related to a problem? Please describe.
I have several projects, and I don't want other people to be able to see my list of databases, I should only know in the shared project, not in other people's projects
Describe the solution you'd like
Please make like sql electron, I already like the ui of Antares, it's very good
Describe alternatives you've considered
just need to be able to open a few windows
Additional context
that's it, made it able to open several windows

Maybe you can add workspaces like in postman, 1 workspace has many databases, it can be organized, some databases belong to which workspace/company

Hi @adibpwd,

Maybe you can add workspaces like in postman, 1 workspace has many databases, it can be organized, some databases belong to which workspace/company

Regarding this need yesterday I released an update that allows you to organize your connections into folders, it probably suits you.

Hi @adibpwd,

Maybe you can add workspaces like in postman, 1 workspace has many databases, it can be organized, some databases belong to which workspace/company

Regarding this need yesterday I released an update that allows you to organize your connections into folders, it probably suits you.

Where that feature sir?

Drag one connection over another to create a new folder.
By right-clicking then you can change the color and name of the folder.
This feature has been present since version 0.7.0.

Drag one connection over another to create a new folder.
By right-clicking then you can change the color and name of the folder.
This feature has been present since version 0.7.0.

Oke thanks, i'll try this night

while the folder feature is quite helpful, but being able to open multiple windows will really help, thank you, I hope to be able to open multiple windows will be available in the near future :)

Drag one connection over another to create a new folder. By right-clicking then you can change the color and name of the folder. This feature has been present since version 0.7.0.

+1 for multiple windows, it can be helpful to compare 2 tables or similar to like that.

There is some modification to be made but there should be no problem implementing the possibility of multiple windows.
It will almost certainly be a feature that will be implemented in future releases.

Thank u sir


Optional record fields can't be used in two different ways

Simple repro, run roc check on this code:
app "helloWorld"
    packages { pf: "https://github.com/roc-lang/basic-cli/releases/download/0.7.1/Icc3xJoIixF3hCcfXrDwLCu4wQHtNdPyoJkEbkgIElA.tar.br" }
    imports [pf.Stdout]
    provides [main] to pf

add : { a : U64, b ? U64 } -> U64
add = \{ a, b ? 1} -> a + b

main =
    x = add { a : 7, b: 8 }
    y = add { a : 7 }
    Stdout.line "$(Num.toStr x),$(Num.toStr y)"

Some reason, we can use add both with/without the optional field as add { a : 7, b: 8 } and add { a : 7 }.

Error message:
── TYPE MISMATCH in examples/helloWorld.roc ────────────────────────────────────

This 1st argument to add has an unexpected type:

11│      y = add { a : 7 }
                 ^^^^^^^^^

The argument is a record of type:

    { … }

But add needs its 1st argument to be:

    { b : Int Unsigned64, … }

Tip: Looks like the b field is missing.

────────────────────────────────────────────────────────────────────────────────

In some cases, omitting optional record fields can also cause a compiler panic. Example can be found here

For people looking for a workaround:
You can use:
Xml : {
    declaration : [Some Str, None],
    root : Str,
}

Instead of:
Xml : {
    xmlDeclaration ? Str,
    root : Str,
}

Another workaround: if you move the definition add into a separate module, then everything works fine.

A couple other reproductions in https://github.com/roc-lang/roc/issues/6101 and https://github.com/roc-lang/roc/issues/5444
5444 has a comment with some investigation: https://github.com/roc-lang/roc/issues/5444#issuecomment-1831507861


Hello world in go in bangla language

_ No description provided. _

Please, add to CONTRIBUTORS.md following the convention in the README.md.


KAFKA-3637: Added initial states

@guozhangwang @dguy @norwood could you please review? Thanks.

@dguy could you have a look too if you get a chance? Thanks.

Thanks @dguy . @guozhangwang could you have a final look? Thanks.

kafka.producer.ProducerTest > testAsyncSendCanCorrectlyFailWithTimeout FAILED.
This is unrelated to streams or this PR.

Looks like transient testReprocessingFromScratch error.

Thanks @guozhangwang

ResetIntegrationTest.testReprocessingFromScratch is failing intermittently

@guozhangwang the usual reset test fails on jenkins. I can't get it to fail locally. In general the reset test fails more often than it should. I don't know if this code tickles it more.

@enothereska Somehow this failure
org.apache.kafka.streams.integration.ResetIntegrationTest > testReprocessingFromScratchAfterResetWithIntermediateUserTopic FAILED
    java.lang.AssertionError: Condition not met within timeout 60000. Did not receive 10 number of records
        at org.apache.kafka.test.TestUtils.waitForCondition(TestUtils.java:279)
        at org.apache.kafka.streams.integration.utils.IntegrationTestUtils.waitUntilMinKeyValueRecordsReceived(IntegrationTestUtils.java:211)
        at org.apache.kafka.streams.integration.ResetIntegrationTest.testReprocessingFromScratchAfterResetWithIntermediateUserTopic(ResetIntegrationTest.java:171)
``

seems consistent on my local machine running your branch (mine is a macbook air)... that was why I was a bit hesitant merging it.

If you claim your local run works fine I will take your word then :)

I am investigating the test issue... :( Sorry for the mess.

Merged to trunk.

@mjsax I'm not sure if the new states in this PR tickled something in the reset tool, I'll also investigate further.


Add an option to define a target folder where to put the subset fonts

I would like the subset fonts to be created in a target folder different from the source folder where the full TTF is.
I can move the files myself of course, but this would be a little more convenient.
Thanks!

I tried to look at the code but didn't see where I could add this. If you have any advice, I can try to built it and make a PR.

There is an apparently undocumented --output parameter that does just what you want. It works with relative and absolute paths. I detected it in cmd.js.

@kid4git it works indeed, thanks! 👍
@zachleat should I make a PR to update documentation with this feature, or do you prefer to keep it undocumented for a reason we don't know (yet)?

I see there was a PR on the filamentgroup repo: https://github.com/filamentgroup/glyphhanger/pull/64

If you port that PR over I will merge it!

@zachleat done: #9

PR #9 already merged


Going from a (OwnedProcess, BorrowedProcessModule) to a OwnedProcessModule

Hi there! A friend and I have been using your crate for our projects and loving it - it's so much nicer than the crappy C++ injectors we used to repeatedly hack into shape 😅
I have a function A that launches an application (OwnedProcess), injects a module into it with a helper function B that takes a BorrowedProcess (returns BorrowedProcessModule), and I'd like to return a OwnedProcessModule from A.
Unfortunately, there doesn't seem to be a way to fuse these together other than using unsafe { ProcessModule::new_unchecked(process, pm.handle()) } which is a bit gnarly. Is it possible for you to add a utility for this?
Thanks 🙏

Actually, might be a moot point: there's not really a way to return a BorrowedProcessModule from B as the Syringe used to provide the process for the module is dropped within the function.
The reason I used a BorrowedProcess for my helper function was because there was no way to go from a OwnedProcessModule to BorrowedProcessModule for passing into syringe.get_procedure; perhaps this is best solved with OwnedProcessModule::borrowed?

I dont think a utility to go from (OwnedProcess, BorrowedProcessModule) to OwnedProcessModule is needed as you can use BorrowedProcessModule::try_to_owned.
As for your second comment: I agree that something like OwnedProcessModule::borrowed would be nice and I will add it in.

Thank you, that's much appreciated!


[Feature]

Description
Adding animated cursor effects can be yet another striking functionality under Beautify.
I shortlisted two cursor animations that I'd like to build for Beautify.

Swirl Cursor Effect
Dot Cursor Effect

I will build them purely using HTML, CSS and JS.
Screenshots
https://github.com/Rakesh9100/Beautiify/assets/123321320/832aee53-77ca-4862-af73-6542d6970991
https://github.com/Rakesh9100/Beautiify/assets/123321320/abfee7ab-8388-4329-8fac-e8bf76e2b1fb
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

@SrinivasDevolper @Rakesh9100  kindly review this and assign it to me.

@Nidhi-Satyapriya
Youu want to add these effect inside background component. Right!!

Hey @saurabhsingh720. I plan to introduce a new 'Animated Cursor' component where not just I, but anyone can add engaging cursor effects with a sample background (since animated cursor effect is the aim, not the background).
To start with this idea, I myself would like to add two of my favorite animated cursor effects (as mentioned above).
I hope I made it clear.

@Rakesh9100
Can it be needed??

Hi @SrinivasDevolper @Rakesh9100 ! Did you review this issue?

@Nidhi-Satyapriya
Mention the mentor and admin only.
@SrinivasDevolper is a contributor.
Understood!!

@Nidhi-Satyapriya
When admin reviews your issue then they will reply u in the comment.

Yes, idea seems good @saurabhsingh720
Please go ahead @Nidhi-Satyapriya

Pls provide your PR @Nidhi-Satyapriya

Yes sir, asap. @Rakesh9100

@Rakesh9100 I made a PR. Kindly review it at your earliest convenience.

@Rakesh9100  @saurabhsingh720 It's been over two weeks since I contributed. May I know the status?

@Rakesh9100


Fix TSLint warnings in msbot-connect-cosmosdb.ts

Proposed Changes
Fix the following warnings in the file msbot-connect-cosmosdb.ts:

typedef
no-any
interface-name
quotemark
no-empty
prefer-const
one-line
curly
triple-equals
newline-before-return
eofline

Testing
Travis will no longer report this issue.

Pull Request Test Coverage Report for Build 184

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 70.668%

Totals

Change from base Build 150:
0.0%

Covered Lines:
2592

Relevant Lines:
3521

💛  - Coveralls

Replaced with https://github.com/Microsoft/botbuilder-tools/pull/404


ModifyResponse doesn't modify Content-Length Header

Bug report
The ModifyResponse filter doesn't change the Content-Length header. The problem with that is, that most http clients are aware of the content-length and wait for aditional informations within the response or cut bytes in case it is too big. That is especially the reason why the toUpperCase calls in the examples works well because they didn't manipulate the size of the response.
The error occured with RC.1 of Spring Cloud Gateway.
Thanks for the amazing job you are doing with this project!

I believe this will be fixed with changes to spring framework. https://jira.spring.io/browse/SPR-17502


airdrop

Could somebody let me know how to make my own token built on tron airdropped to random trx holders??

@wutalamanzheng Please join the developers' Discord channel #smart-contracts to ask for help. Thanks. https://discord.gg/GsRgsTD


Cloudfront

Towards: https://github.com/giantswarm/roadmap/issues/1546
This PR improves the cloudfront service in order to act as a reconciliation tool.
Instead of just creating the cloudfront distribution once and leaving it there untouched, it now checks if the desired parameters are in place and updates the distribution in case any change is needed.
this also applies to tags.
Checklist

[x] Update changelog in CHANGELOG.md.

asking also hydra's review as this is used on capi as well.


[backport] Fix out argument in fusion ufunc

This PR is backport of #213.

jenkins, test this please.

LGTM


Patch to make w3m image backend work on FreeBSD 12

Just an extra possible directory to search for w3m-img before outputting that it wasn't found on the system.
It fixes my issue of being unable to output images with neofetch when using it on FreeBSD 12-CURRENT.

LGTM :+1:

Thanks!


Support conv2d dilation for x86

in https://docs.tvm.ai/tutorials/frontend/deploy_ssd_gluoncv.html#sphx-glr-tutorials-frontend-deploy-ssd-gluoncv-py  , which is an ssd tutorial, there is the statement:
SSD with VGG as body network is not supported yet since x86 conv2d schedule doesn’t support dilation.
I was working in some tflite reference code recently and noticed that they have a very clear implementation of dilation, if that helps.
It is the convolution code in
 tensorflow/lite/kernels/internal/reference/reference_ops.h
The dilation was used as a modification to vgg19   to support super resolution enhancements in a style transfer example I played with recently, with very good results, so I think it is a parameter you should support.

The computation definition for dilation conv2d already exists in topi. For x86, the major part missing is the schedule.

https://github.com/dmlc/tvm/pull/3308


Improvement: Unmarshal withdrawals from RPC blocks

This adds Withdrawals to blocks retrieved via RPC.

Thanks for merging (and also for the improvements!) 🙏


Feature: Implement IconBrowser

What? The IconBrowser is now (half) complete.
I probably broke a few things along the way with the FA-imports because I don't have access to the Pro version of FA.
Why? I was bored and I like this project, so I pulled and botched something together.
Hope this helps in the developement of this "awesome" project

I guess, the Composition API is used in this project instead of the Options API.

Yes this is debatable. You could rewrite everything into one or the other, but personally  I find the Options API more structured and readable. Probably because I don't know the Composition API...

Yeah, I like the Options API too, but @sebinside used the new Composition API in this project and when learning Vue, so I assume he's not familiar with the Options API.

I know both APIs and I like the composition API way more as it is more honest (kinda blue pill / red pill, matrix style). Also, it is the default in the vue documentation, and although there are no plans to deprecate the options API, I think, going with the default is always a good idea. Additionally, as this project already uses the composition API, I would opt for not mixing them (although it is technically feasible).
TL;DR: I will look at this PR in the next coding stream and see what can be reused. If you want to use the chance to learn and play around with the composition API, feel free to do so ;)

Let's not do any mixing of the APIs (in Germany we say "kein Mischmasch"). Rewrote my changes into the Composition API


Table alignment assumes that id is in the first column

This is not guaranteed and we should use the record header to determine the id column instead.

It's a reasonable assumption once #397 is fixed.


Add option to delete empty cells

Hello. I've added an option to remove empty cells. Please, consider merge it.

Good idea, thanks @kuriot! I've changed the flag to --remove-empty to be more explicit, and also added the functionality to the check subcommand.

This has been released to PyPI in nb-clean 1.5.0.


[Suggestion]: 1.21 support?

General Info

[ ] I would like to propose new content
[ ] I would like to propose changes
[X] I would like to propose another thing

Suggestion Description
Will you port this mod to 1.21?

thanks


Repository Layer

Hello! You did a great job!
I was thinking of using this package together with a layer of repositories (https://github.com/andersao/l5-repository), but still was not very successful.
Do you have any idea that can help me make this implementation?
Once again thank you so much.

I haven't used this repository layer yet. I will take a look and dig into the code and will let you know on updates.


Fix multiple domains

Fix issues with multiple domains by creating a dummy certificate for each one.
Fixes #79, #65

(this does not really solve the issue)


handle more ActivityPub message types for bookmark and comment CRUD

Right now we don't broadcast deletes if you delete a bookmark and we don't accept deletes or edits if someone tries to manage a comment they made. All the ActivityPub support is hand-rolled, perhaps we should move to a separate library to handle the breadth of ActivityPub messages and then implement our functionality on top of that?

progress in fd8924254c737328f1e00995825042772bffb5c8 -- we now handle sending bookmark delete to followers. more to come soon.

Closing this; there's some work remaining to be done documented in #115.


Text Extractor - Show language and keyboard shortcut

Description of the new feature / enhancement

Show the language that Text Extractor currently uses, e.g. in a small movable square in the corner.
Allow us to switch the language by using a keyboard shortcut, e.g. the one Windows already uses for that (Ctrl+Shift for me, Alt+Shift by default).

Scenario when this would be used?
When user wants to extract some text not in the default language, he needs to remember to use RMB -> switch language. Often, if TE is not used often, user can forget about it, which leads to bad output. First requested feature will help user with remembering to switch by providing a clear indication of the current TE language setting. Second is just a quality of life feature for those that don't want to use the mouse while already having hands on the keyboard.
Supporting information
No response

Possible Multilingual Text Extractor Solution
My solution in #25197 is a single hotkey that activates a customizable context menu.
 or
I simply crosspost here because.

There is a Translate Text option (which can potentially pop up a screen that immediately allows you to change Text Extractor language on a case-by-case basis); and
The menu is optionally customizable by advanced users to add extra things, like multiple options "Copy as Text, using English OCR" + "Copy as Text, using French OCR" + "Copy as Text, using Chinese OCR" (via the proposed menu customization API)

While this may not be exactly your suggestion; it is a potentially universal mainstream solution that aligns better with Microsofts' goals, that can be extended to cover your particular use case;

This has been worked on for the 0.74.0 release. Please update PowerToys to the latest version. 😉
https://github.com/microsoft/PowerToys/releases


Try to install gdal using the system version

Overview

Set default GDAL version to >3
Get GDAL version number from system in CI workflow

Related Issue / Discussion
Additional Information
Contribution Agreement
(as per https://github.com/geopython/pywps/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

[x] I'd like to contribute [feature X|bugfix Y|docs|something else] to PyWPS. I confirm that my contributions to PyWPS will be compatible with the PyWPS license guidelines at the time of contribution.
[x] I have already previously agreed to the PyWPS Contributions and Licensing Guidelines

I was looking at a more radical approach #638 :)

Made unnecessary by #638


readme typo fixes

Making readme consistently info where it's occasionally details, to be consistent.
In #### swarm.on('connection'... the args were getting mixed up info vice details

Awesome stuff.

Awesome stuff.


zetacore : clean up finalized cctxs

Finalized CCTX can exist in three states
OutboundMined : Can be removed
Reverted : can be removed
Aborted : we support refunding a aborted TX , these should not be removed
Considering the above information, we should remove CCTXs finalized from our state.
We keep track of finalized inbounds separately, so this would not affect the security of the system, making sure already processed cctxs cannot be submitted again.
https://github.com/zeta-chain/zeta-node/blob/c3d19297b4d345e9d7ff746f510a0e3e25eb6355/x/crosschain/keeper/msg_server_vote_inbound_tx.go#L121-L124

We'd need to keep them around long enough for the indexer to see them though right?

Not sure about this one tbh. CCTXs are the pillar of ZetaChain, would make sense to me to keep indexed and keep a tamperproof and uncensorable way to fetch these through node even if it brings more overhead on state.
Cleaning ballots are different for example, as used more internally in the protocol.

If we just need the information regarding the cctx , we can add as much as we need to an event which is stored as part of the tx result. @gartnera  We can use events for indexing as as well.
Keeping long-lived data as part of events might be beneficial because some nodes can choose not to store it by just setting it tx_index to null , or even setting it to plsql for richer searching.
Keeping it in application storage mandates every node to store it.
With that said , the CCTX data does not seem to be an immediate concern , and can be added to the backlog for now

If we just need the information regarding the cctx , we can add as much as we need to an event which is stored as part of the tx result. @gartnera We can use the events for indexing as as well.
Keeping long-lived data as part of events might be beneficial because some nodes can choose not to store it by just setting tx_index to null , or even setting it to psql for richer searching.
Keeping it in application storage mandates every node to store it.
With that said , the CCTX data does not seem to be an immediate concern , and can be added to the backlog for now


Require Unbound 1.6.6 or newer

Pull Request (PR) description
This PR updates the minimum supported version of unbund.  this allows
us to be a bit more generous when writing the first unbound.conf file
and fixes #286
I have picked unbound 1.6.6 as the minimum supported version.  this
seems to be the version available in centos 7 which i assume is the
oldest version we support based on metatdata.json
This Pull Request (PR) fixes the following issues
Fixes: #286

@b4ldr can you take a look at the failing tests?

@b4ldr can you take a look at the failing tests?

done, at least im pretty sure the unbound issues are unrelated but will double check

This feels like a major version bump to me. Perhaps we should first merge the other open PRs, get a release out and then do this major?

SGTM

README is IMHO the best place.

updated


[Normalize.css]=>[Others]

Normalize.css
a small library that makes browsers render all the element in the modern standards
Link: https://necolas.github.io/normalize.css/
Checklist:

[x] I have performed a self-review of submitted resource and its follows the guidelines of the project.

Redundant commits


Fixup some spelling errors in comments

We used an automated tool to check for spelling errors in our project and found some in CMake files we use from here, so I thought I would contribute the fixes back.

Thanks for fixing! ;)


Keep repository structure when cloning a GIT workspace

This will keep all the git repository structure instead of just copy the folders form the "path" parameter from the workspace definition, this will allow to use modules that are inside the same repository.
Fix #431

Hey @alfespa17 !
Tested out your changes on Gitpod with the following folder structure:
├── Makefile
├── service
│   ├── articles-counter
│   │   ├── Dockerfile
│   │   ├── go.mod
│   │   ├── go.sum
│   │   └── main.go
│   ├── fetch-articles
│   │   ├── Dockerfile
│   │   ├── index.js
│   │   ├── package-lock.json
│   │   └── package.json
│   └── post-article
│       ├── Dockerfile
│       ├── index.js
│       ├── package-lock.json
│       └── package.json
└── terraform
    ├── app
    │   ├── main.tf
    │   ├── output.tf
    │   ├── provider.tf
    │   └── variables.tf
    ├── deepfence-threat-mapper
    │   ├── main.tf
    │   ├── provider.tf
    │   └── variables.tf
    ├── modules
    │   └── lambda
    │       ├── main.tf
    │       ├── output.tf
    │       └── variables.tf
    └── registries
        ├── main.tf
        ├── output.tf
        └──  provider.tf

and can confirm plan works! Can't really try the apply, but I'd assume it works as well.
Great job thank you! Let me know whenever this can be merged and released 🙌

Hello @AppliNH
I did some final testing and I think I got everything working correctly, for now the build is failing downloading some buildpack image, I will try to merge this later so I can release a beta with this fix for the directory structure.

@AppliNH
This feature can be tested deploying the latest helm chart and overriding the following parameter:
executor:
  version: "2.17.0-beta.1"
``


Revamp

@rultor merge

@rultor merge

@abdealijk OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@abdealijk Done! FYI, the full log is here (took me 1min)


Update docs with Tower instructions

Add instructions to import Student inventories in Tower using the CLI
Add detail to change ec2_name_prefix

@marcosgm one of the exercises is to have the student input the inventory themselves.  Did you see the tower exercises?
This is what we had in the deprecated workshop-> https://github.com/network-automation/linklight/tree/master/exercises/networking/2.2-towerconfigure#step-6-import-the-existing-inventory
I am actually in the process of making new tower exercises since the webUI changed.  What are your thoughts?

no response from @marcosgm


Create single React component to represent a project post

Refer to the design found on our Figma page.
Each project post should include:

Title
Picture (optional)
Timestamp (last updated)
Description as plain text
User who posted (name)
Link to full project page
Placeholder button users to comment (give feedback)

Hey mate, will get on this issue - by placeholder do you mean the buttons should be static and not do anything at the moment?

Hi @jameskbecker! Thanks for reaching out to tackle this issue. We are still setting up all of our issues for this project and would love to have you hop on some of the development. This one we have being developed by one of the students working with us. I apologize if I haven't figured out the assigning process yet. I will add some more issues today and would love to have you tackle some of them.

Ok gotcha - if you're looking to get contributions you may want to focus on docs for setting up vscode. If you like i can open a separate issue with system specifics but I just get compilation errors when trying to run the web app within "clients/web-browser"

Please open a separate issue with your system specifics. I am developing on a Windows machine with WSL running Ubuntu 22.04 and am able to get everything up and running without errors following the BUILDING instructions.

Test comment


Plotjuggler 'Missing ROSType in library' on ros1 node published messages

Other nodes get the messages fine, but plotjuggler is trying to get type information in a way roslibrust nodes don't provide?
https://github.com/facontidavide/PlotJuggler/blob/main/plotjuggler_plugins/ParserROS/rosx_introspection/src/ros_message.cpp#L206
What I do is click `Streaming | ROS Topic Subscriber | Start' within plotjuggler, select the topic and press OK, and then get the error:

I can start up a roscpp/rospy node publishing to the same topic, get plotjuggler plotting it, then switch in the rust node and it'll still plot fine (presumably having already cached the type information it wanted).
This happened with an earlier version of roslibrust (883057f8d159b0298c243744d65971e746ec5dbc) and then I just now updated to the latest in master with same result.

Lucas thank you!!! Seriously, awesome to have an active user, and really appreciate these issues. I'm just using this crate in my own little world and its awesome to have someone catching these things.
I haven't used plot juggler before, but I'm aware of it, let me dig in and see if this will be an easy fix.

Okay after doing some digging.
Plot Juggler ultimately parses the message_definition= field of the TCPROS connection header (which we should be providing) via RosIntrospection::ShapeShifter here.
So my first guess / assumption is that RosIntrospection::ShapeShifter isn't giving them the message definition from us.

Well crap... turns out I was wrong... ROS does have the message_definition field in the connection header contain the fully expanded definition...
Apparently that field is supposed to contain the full output of gendeps --cat example here:
https://wiki.ros.org/roslib/gentools
Which is not what our current message generation is doing :poop: 100% a bug. Will be somewhat annoying to fix.

For my own reference when I dig into implementation, gendeps concatenation is dead simple:
https://github.com/strawlab/ros/blob/c3a8785f9d9551cc05cd74000c6536e2244bb1b1/core/roslib/src/roslib/gentools.py#L245

@lucasw Okay I think I have a fix up on a branch here:
https://github.com/Carter12s/roslibrust/pull/174
It is certainly closer to the actually ROS specification, but I haven't been able to directly test with PlotJuggler yet.


If Automatic Household Naming is disabled and existing naming rules have syntax error, exception is thrown

Critical Changes
Changes
When NPSP Settings > Household Naming Settings are saved, they are validated whether Automatic Household Naming is checked or not. Moreover, the settings validation will be done when NPSP Settings > Refresh Household Names button is clicked on and if the Automatic Household Naming is not checked. The Automatic Household Naming will be checked and the batch process will execute only if the Household Naming Settings are valid.
Issues Closed
Fixes #2476

@smarovich , can you please verify the Release Notes Changes section?

@smarovich @RadSFDeveloper For the release notes, I think we can make this clear by pointing out that it applies if you select 'Other' for a name format and type in an alternate format. (Yes, it will be checked regardless but if they use a default it will pass and they'll have nothing to worry about.) :-) Maybe we can even give an example of a simple invalid one that will trigger an error, such as {!FirstName}} instead of {!FirstName} that will make it fail.

@smarovich We should also point out that field names are case sensitive... so FIRSTNAME will generate an error, while FirstName will not. You'll also get an error with custom_field__C vs custom_field__c.

@RadSFDeveloper @judisohn why are field names case sensitive? That's never been the case as far as I'm aware.

@njjc , if API Names are not code sensitive, then there are numerous errors as a consequence of that. For example, use firstname or middlename in the Household Name Format. Depending on the field, the error might be displayed when Save button is clicked or it can save successfully, but the error is shown when you save a Contact or add a new Contact via Manage Households. Related issue was fixed here (see comments for the discussion): https://github.com/SalesforceFoundation/Cumulus/pull/2681
Moreover, if we are to display Household Naming fields at a Lightning page to be displayed or populated, they will not be displayed since these fields are case-sensitive.

@RadSFDeveloper and @njjc API Names by themselves are not case sensitive, however map and set keys are case sensitive which was the root cause behind the middle name error.

@RadSFDeveloper and I had a great conversation about this. API names are case insensitive and can always be properly resolved to a single field in apex regardless of case. We agreed that it's a poor user experience to necessitate entering API names with a specific case, which is what this pull request is currently doing. She is going to further edit this pull request to no longer require any specific case to be entered by the user, and carefully handle maps and sets on the backend to avoid duplicate fields due to case differences.

@njjc indicated we should not ask users to enter case-sensitive API Names. I am going to close the PR and change Household Naming processing to work without expecting users to enter case-sensitive API names.

@davidhabib FYI don't review this yet, more changes are coming.


New org: Center for AI Safety

https://safe.ai/about

Ready for review. Check https://github.com/riceissa/aiwatch/commit/1ce8744a26d0d3ab7dfd4d5124809889528d4493. I saved safe.ai/about in Wayback Machine and used this date as start_date, with 'year' as start_date_precision.


Redirect urls with colon causes error on the page.

Version
react-router-dom: ^4.3.1
react: 16.5.0
react-dom: 16.5.0

Test Case
https://stackblitz.com/edit/react-temicd
Steps to reproduce
Go to the above url, click on the test link on the page. The page will show error. It causes error only for the Redirect, but works fine in other cases. E.g. if you click on the Details 2 link it has the same url but it works fine.
Behavior
It causes error on Redirect, if the redirect to url value contains a colon e.g.
Failure case
<Redirect from="/test" to="/details/2018-10-02T00:00" />
But it works fine for Link,
<Link style={styles.link} to="/details/2018-10-02T00:00">Details 2</Link>  this works.

React Router uses path-to-regexp to compile path patterns into regular expressions for matching locations, which interprets colons as params. In order for the <Redirect> to support redirects to routes with params, it compiles the to prop passed to the <Redirect> and attempts to inject any matched params from the from path.
While not perfect, I think that that best solution would be to escape the colon (00\:00).

@pshrmn, Thanks, your solution worked. I will close the issue.


Docs issue?

Is this really one-to-many unidirectional example?
https://github.com/doctrine/doctrine2/blob/master/docs/en/reference/association-mapping.rst#one-to-many-unidirectional-with-join-table

Yes :-)


BSON dependency missing in pom.xml

Hi HP,
I think your current master is missing the following dependency in pom.xml:
<dependency>
  <groupId>org.mongodb</groupId>
  <artifactId>bson</artifactId>
  <version>${mongodb.driver.version}</version>
</dependency>

Best,
Jurgis

hi @pederpansen
actually I'd be surprised if that would be the case. tried to reproduce a missing dependency issue with the bson artifact you are referring to but couldn't.
local maven build works
local IDE build works
travis CI build works
also when I manually look into the assembly jar after the successful build the BSON classes in question are contained. the bson.jar should be included as a transitive dependency by maven based on the mongo-driver artefact.
can you please get back to me with more details in case you cannot build without explicitly listing the bson artefact in the pom file? it should work as is.

Indeed, I was not able to reproduce the build error on my private computer. I imagine my maven cache or my company's artifactory could have interfered.

Thx for you clarifying answer and closing the issue!

Pingback: I was not able to reproduce the issue. The .m2 cache moves in mysterious ways ;)


addUser RPC Service and RPCTestClient

Summary

Add a simple RPCTestClient to enable a quicker / easier way to test our RPC services. This is an optional build parameter in CMake. BUILD_RPC_TEST_CLIENT will need to be set to ON
Add a new RPC service named addUser which will allow adding of users whilst segs_server is running and DB is locked.

Issues closed
Closes #772
Closes #435

Apart from the things pointed out by @nemerle , this looks good to me. :+1:

@broxen @nemerle all changes should be made, could you give it a once over please? Had a merge conflict with @dracc changes to AdminRPC but should be resolved

@broxen @nemerle
Sorry before you merge, just noticed something has gone wrong in my conflict fixing / merging ... I have a file called compile_commands.json with 51,000+ code insertions... This isn't me, so not sure where it's come from.
Any ideas??

Looks like something spit out by QTCreator or Clang as it has build information.
I'd simply add it to the .ignore file.

Okay sorry worked it out, was a compilation database I generated in error from Qt. I've just deleted it.
Should be good for merge if you are happy with review and builds pass.
Cheers

Thanks @miles200! Once you've had a chance to merge conflicts, I'll get this in.

@broxen Must have just conflicted with the merge of the other PR :smile: Should be sorted.

Once CI clears I'll merge.


feat(icons): lazy load react-icons

Proposed Changes
While debugging in another package we saw that loading the icon package could take multiple seconds in Jest (using commonJs). To avoid this we wrap the component in a Loadable component. It should be transparent when using the library.
Potential Breaking Changes
The d.ts did change a bit, if developer were importing the component from /dist/some/path/sizeXPx it now use a default export
Acceptance Criteria

[ ] The proposed changes are covered by unit tests
[ ] The potential breaking changes are clearly identified
[ ] README.md is adjusted to reflect the proposed changes (if relevant)

is it me or this seems to make the plasma website load faster also? It's feels more snappy :eyes:


FLINK-36479 Remove deprecated method Table getSchema as

What is the purpose of the change
https://issues.apache.org/jira/browse/FLINK-36479  Remove deprecated methods Table#getSchema and Table#as
Brief change log
Remove deprecated methods
Verifying this change
This change is already covered by existing tests
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency):  no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers:  no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature?  no
If yes, how is the feature documented? not applicable

i'm sorry. I was trying to remove my commit which failed the jenkins check. But PR itself got closed.


setup-buildtools.cmd also install llvm it seems

Steps to reproduce.
Download LLVM or have LLVM already installed
What is the expected behavior?
Buildtools script to not use choco to install LLVM
What is the actual behavior?
Chocolatey download llvm

This script assumes that choco is available by default (it is on GitHub Actions runner). While LLVM is not (and by default, it is not available on GitHub Actions runner). Unfortunately LLVM installs different versions under the same directory all the time. One option is to check if directory already exists and avoid the installation. However, in this case it'd be best to also check that the version actually installed matches the one that SDK is known to work well with. What would be your recommendation?

You should be able to override this behavior by unsetting INSTALL_LLVM variable.

Does chocolatey have version checking already, I'm sorry I can check but thought I would ask.
What if I install X and I already have X installed?
With most packages when you already have something installed, the Chocolatey process will just perform the install again silently. Most times this means that it does nothing and in the end you have what you already had.
I think this is probably a non-factor all things considering, just wanted to mention what I saw.  Feel free to close.

I'm no longer actively working on core parts of the project. I'd let @lalitb comment on this. You can specify the version through the --version flag.. However, I don't think that the present setup script does specify concrete version. Maybe it'd make sense to make this configurable via env var.
Example:
choco install llvm --version=13.0.1
Chocolatey LLVM Package Version history


Update to 1.20.6 & Bump Version to 1.6.4

Also, the mappings were changed to use the official Mojang-Mappings. Making it incompatible with older Versions & Spigot.
Closes #23.

Still need to test this on all Plattforms, the Server Starts Fine. Any Feedback would be great.
Here is the Jar, if someone other will do some Testing,
bmm-paper-1.6.4-paper-all.zip

@MiraculixxT can you review that?

Thanks for the update!
Somehow GitHub didn't notified me about this issue before pinging...


Feature Discovery

I was doing a demo with feature discovery and I was throwing the following error
Error: No component factory found for FivFeatureDiscovery. Did you add it to @NgModule.entryComponents?
and literally copy the following code from your project
Github Feature Discovery

Thank you for your issue and sorry for the late reply.
Please import FivFeatureDiscoveryModule as an import to the AppModule this will register FivFeatureDiscovery as an entry component.
import { NgModule } from '@angular/core';
import { FivFeatureDiscoveryModule } from '@fivethree/core';

@NgModule({
  imports: [
    ...
    FivFeatureDiscoveryModule,
    ...
  ]
})
export class AppModule {}

Also you need to import it into your page or component module where you like to use the feature discovery.