avg function doesnt work on real type

avg on Real column like select avg(sr_return_amt) from store_returns in tpcds leads to following exception
VeloxRuntimeError: type->childAt(0)->kind() == TypeKind::DOUBLE (BIGINT vs. DOUBLE) Input type for final aggregation must be (sum:double, count:bigint) struct

CC: @spershin

We are trying to take care of this (along with other issues) at the moment.

@spershin I submit my fix for this issue in https://github.com/facebookincubator/velox/pull/1087. Please take your time to review it as it blocks TPC-DS now

@spershin Sergey, would you summarize what's the current plan to address this and similar problems?

Hi Frank,
I had left a comment on this PR that might have got lost.
Another approach to this solution could be to handle the order of the arguments in the function VeloxExprConverter::toVeloxExpr in presto_cpp. Is that solution not feasible ? That fix might be cleaner than adding these new signatures into Velox.


Add support React-Native new Version.

[x] Review the documentation: https://docs.viromedia.com/
[x] Search for existing issues: https://github.com/viromedia/viro
[x] Use the latest ViroReact release: https://docs.viromedia.com/docs/releases

Environment
Please provide the following information about your environment:

Development OS: Mac, Windows, Linux
Device OS & Version: What Android OS version (7.0, 8.0, etc.) or iOS version (11.0, 11.2) is your device running on?
Version: ViroReact version and React Native version
Device(s): What device(s) are you are seeing the issue on (i.e. iPhone X, Samsung Note 8, etc)

Description
Describe your issue in detail. Include screenshots if needed. If this is a regression, let us know.
Reproducible Demo
Let us know how to reproduce the issue. Include a code sample, screen capture, video recording. The more information you provide, the better we can support you.

You can simply compile it from source using the latest react native version.

Could u write a step-by-step on how to do that, i know that sounds like a noob request but i'm a noob indeed.

Have you fixed this issue?

Clone the repo.
Go to test folder
Be sure react-native ands react stands for the latest version in package.json
npm install
Navigate to android folder and from there run: ./gradlew assembleGvrRelease
Go back to root folder
Run ./prepareRelease.sh

@doranteseduardo i am new to the react viro, i have created a react native app using "react-native init sample" command,
The versions are,
"react": "16.13.1",
"react-native": "0.63.2",
And i have added the  "react-viro": "2.17.0"
Now i dont know how to run the app getting errors.
I need to any extra setup for react viro?
Please help me.

Do the steps I mentioned above to get an up-to-date version of react viro, at the end you will get a zip file.
Clear the folder node_modules/react-viro and unzip the file in there.
Finally, you can run cd android; ./gradlew clean; cd ..; npx react-native run-android

<npm WARN deprecated<EMAIL_ADDRESS>fsevents 1 will break on node v14+ and could be using insecure binaries. Upgrade to fsevents 2.

<EMAIL_ADDRESS>postinstall E:\Projects\TestApp\viro-master\test
./setup_react_viro.js

'.' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>postinstall: ./setup_react_viro.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\ZEB10\AppData\Roaming\npm-cache_logs\2020-09-30T04_55_10_693Z-debug.log`` >
Getting this error, in package.json file I have changed the react and react native versions
"dependencies": {
"react": "16.13.1",
"react-native": "0.63.2",
"react-native-config": "0.11.7",
"react-native-device-info": "1.3.0",
"react-redux": "^5.0.5",
"react-viro": "../",
"redux": "^3.7.2"
},
but it shows react-viro '../' is not recognized as an internal or external command

Do not worry about it, continue with the process mentioned above.
The script breaks in the post-installation, but I have had no problems at all.

Do not worry about it, continue with the process mentioned above.
The script breaks in the post-installation, but I have had no problems at all.

Thanks for your answer.
I need to change the react and react native version to current version right?

Yes, both of them.

Running ./gradlew assembleGvrRelease giving this error:
Task 'assembleGvrRelease' not found in root project 'android'. Some candidates are: 'assembleRelease'
Please help me with fixing this

It seems you haven't linked your project with Viro properly.
Be sure to do not skip the add flavors steps.

It seems you haven't linked your project with Viro properly.
Be sure to do not skip the add flavors steps.

I got this error after I run npm install in updated test folder. I am new to RN and am not sure how to fix this. Please help

Issue got resolved!! Thank you @doranteseduardo

Hi @Akash76

Hi @doranteseduardo i have linked all the packages. In mobile when i start the app it asks for 'Choose your experience' when i click AR , I could see only the Blank screen.
Whether do we need API key to process this? Or we can use only Viromedia apk to check the scene?
FYI: And in scene i have used HelloWorld to display

You're using an outdated package, Viro no longer requires an API key.

@doranteseduardo  this is my package version   react-viro: 2.17.0

And can we test only in testbed or in AR supported mobile can we check?

@doranteseduardo this is my package version react-viro: 2.17.0
And can we test only in testbed or in AR supported mobile can we check?

Could you please open a new issue report with all the needed info?
That way it'd be easier to help.

@doranteseduardo  i have created a new issue, Kindly check it


Modifying treeData in state does not trigger an onChange or other event.

Adding a node to the treeData outside of the component and modifying state.  I'd like it to trigger a event to know that data has changed.  It takes expanding/collapsing or moving an item for the UI to reflect this.
Ideally this should happen when state changes (treeData).  Am I doing something dumb?  Do I have to explicitly trigger an event?
Any help appreciated, thanks.

Hm.. I designed it so that it would update to reflect any changes in the state, without having to explicitly trigger anything. Are you modifying the object in-place?
e.g.,
this.state.myTree[3].bar = "foo";
this.setState({ myTree: this.state.myTree });

Every data operation I did in the library (and in the tree data APIs) did something more like
const index = 4;
this.setState({ myTree: [
    ...this.state.myTree.slice(0, index),
    { ...this.state.myTree[index], bar: "foo" },
    ...this.state.myTree.slice(index + 1),
] });

I forget, but I think that's called a pure function, or treating objects as immutable.
If this doesn't help, do you mind posting the relevant parts of your code?

Thanks for the expedient reply.  I'm simply adding an item to the initial data array, similar to the first option.  It may be less elegant than the immutable approach, but why should this matter? (my ignorance)
this.state.treeData.push({ title: 'foo' }); this.setState({ treeData: this.state.treeData });
Shouldn't it also update when I send in new props?  (Thinking out load - could some lifecycle events not being run because of react-router?)
Key is everything is visibly 'refreshed' (i.e. - the node added to the UI) as soon as I click on the expansion or collapse of a node or after a drop event.
Is there something that needs to be toggled for visibility?  Is it that I am adding a new node rather than editing an existing one?

Glad to hear!
More so than my library, I think it might have something to do with how React detects changes in state. Assigning new values to an object without recreating it makes it so the previous reference is unchanged, and objects are passed around by reference (I think).
Here's a little proof-of-concept from the terminal:
$ node
> a = {b: 1}
{ b: 1 }
> c = a
{ b: 1 }
> c.b = 2
2
> a === c
true

Hi,
How do we get index?
Can we use treeIndex to modify the tree node title?
Thanks

So I should be able to use the functions like addNodeUnderParent when hosting this in a PureComponent with no issue?


Initial Build Task fails

expected behavior Build Task Completes
actual behavior Build Task Fails
steps to reproduce Follow set up for Docker and vscode on windows, up to the point where you execute Run Build Task for the first time
system configuration Windows 11, Docker Desktop, Visual Studio Code

Had to change activate.sh to ./scripts/bootstrap.sh to get Bootstrap to run (Not sure if relevant)
Running "Run Build Task" eventually spits out:
Done. Made 4027 targets from 255 files in 2848ms
+ ninja -C ./out/default
ninja: Entering directory `./out/default'
ninja: error: '../../third_party/nlfaultinjection/repo/src/nlfaultinjection.cpp', needed by 'obj/third_party/nlfaultinjection/repo/src/libnlfaultinjection.nlfaultinjection.cpp.o', missing and no known rule to make it
The terminal process "/bin/bash '-c', 'scripts/build/default.sh'" terminated with exit code: 1.

My build fails at the same step but in a different way. I also have the same config as @jimmyeao (Windows 11, etc.)
It took a while but I finally got the bootstrap to work and I thought I was out of the woods. However, when I go to "Run Build Task" I get:
> Executing task: scripts/build/default.sh <

scripts/build/../../scripts/activate.sh: line 1: bootstrap.sh: command not found
The terminal process "/bin/bash '-c', 'scripts/build/default.sh'" failed to launch (exit code: 127).

Following the readme tutorial ran into the same problem
Env:  Ubuntu20.04    @andy31415

Had the same error .
this is what I did to make it work
clone with this option : -c core.symlinks=true
this will create the symlink for some file
ex: "./stripts/activate.sh" is a symlink to bootstrap.sh
after restarting VSCode and before running Bootstraps:
in a terminal :
sudo chown vscode:vscode /workspaces/connectedhomeip/ -R
sudo chmod 777 /workspaces/connectedhomeip/ -R
will allow the submodule to be downloaded and properly install.
Hope this will help

Delete dir third_party/nlfaultinjection/repo

run command

source scripts/bootstrap.sh
source scripts/activate.sh
gn gen out/host
ninja -C out/host


Floating point issues (multipleOf)

Hey,
I have issues with multiple of validation. I guess it is to floating point inaccuracy. Is there any fix / workaround.
The following schema schould be valid but is not.
Schema: `{
"type": "object",
"required": [
"environ_impact"
],
"properties": {
"environ_impact": {
"type": "array",
"items": {
"type": "object",
"required": [
"amount"
],
"properties": {
"amount": {
"type": "number",
"minimum": 0,
"multipleOf": 0.0001
}
}
}
}
}
}`
JSON
`{
"type": "object",
"required": [
"environ_impact"
],
"properties": {
"environ_impact": {
"type": "array",
"items": {
"type": "object",
"required": [
"amount"
],
"properties": {
"amount": {
"type": "number",
"minimum": 0,
"multipleOf": 0.0001
}
}
}
}
}
}`
Thanks,
buti

Issue has already been discussed here:
https://github.com/Julian/jsonschema/issues/320


feat(multi_object_tracker): add debugger output processing_time/cyclic_time/elapsed_time, tentative objects

Description

🤖[deprecated] Generated by Copilot at 208708c
Improved the MultiObjectTracker node with debugging and performance measurement features. The code uses tier4_autoware_utils to publish metrics and tentative objects, and filters out unstable objects from the output. The changes affect multi_object_tracker_core.hpp and multi_object_tracker_core.cpp.

Related links

Tests performed

Notes for reviewers

Interface changes

Effects on system behavior

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

data observation
With rosbag replay simulation, there are large difference between outputs with/without delay_compensation flag.

delay compensate: true

delay compensate: false

with publish speed to 1.0
Here is the sample output:


p5 tips and tricks

Video suggestions to be added to "p5 tips and tricks" track

Custom Shapes

Yes, this would be great!

Also, let's add a page for 2D arrays! It's referenced in several videos on the site:
https://github.com/search?q=repo%3ACodingTrain%2Fthecodingtrain.com OTNpiLUSiB4&type=code
This came up b/c I am also referencing this video in the Nature of Code book and I'd like to link to thecodingtrain.com.
Open to other places where this video could go?
https://youtu.be/OTNpiLUSiB4

@shiffman If you still want to link to the 2d arrays video from NoC, you can merge #1162

#1162 was merged, I think we can close this issue?

Indeed!


modified layout context name

It was written incorrect activity class name ( layout context name)

I signed it!


Position information populate from directory but need to be overwritten

For both owner and support positions need to populate from the selected user in the directory api.  They need the ability to overwrite this position in the case the directory api is incorrect.

This is a bit of a complicated one from a data integrity vs. usability standpoint.
The current feature is possible with the current interface via:
As a System Admin:

importing a user
updating the user's information
creating the dataset with the updated user.

From an usability perspective, I could:

make it possible to imported user in real time from the datasets creation form
have edits to the various dataset owner fields update the user on dataset creation, or in real time.

From a security perspective though, this breaks security around user management, effectively giving "dataset owner" the ability to create/update arbitrary users.
We could limit the exposer, by giving them the ability to:

only create users as part of dataset creation, but not edit them after the fact.
only create users for emails that don't already exist.

@dpdavids Thoughts?

Hi @klondikemarlen I think if you can include a hyperlink to update their own user information that is sufficient.  But as an FYI an data owner and data support users do not need to be system users.  The example would be when a business analyst enters the information on behalf of the unit, particularly when it is open or subscribe, where no approval is required.  If they are a system user and their name is in either of these fields they have the ability to manage the record.

These would just be potential users.  Everyone within YG can be a user.  There may be cases where a business analyst will enter the dataset.  The dataset does not belong to them rather someone else would be the expert.  The owner of the dataset or the support user may or may not be IDP users.  But they are the people that understand the dataset the most.  ideally they would both be users but it is not manditory.

0005, Setting User Metadata, 2024-06-06

Closing in favour of multiple linked tickets.


Inliners, outliners

On your screenshot the black dot also visible near line below green one. Maybe there some error in code.
In my opinion simple example without generic types may have more profit for understanding and usage. There much more code in model, that in `GRANSAC.h`. It's not bad, but start thinking do we really need this abstraction with templates.

Closing this as a duplicate of #4


RTD config fix

Fix environment file name in .readthedocs.yaml.

This is a minor fix, so self-merging.


soulseekqt: fix things incording upstream tarball change, unpack use p7zip instead of binwalk, fix homepage

Motivation for this change
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Notify maintainers
cc @

Would you mind rewording your commit message to be more descriptive about why these changes are being made?

Like @grahamc wrote, could you describe in your commit message why this is needed. It's far from apparent.

done.

It fails to build
convert: unable to open image `SoulseekQt.png': No such file or directory @ error/blob.c/OpenBlob/2881.
convert: no images defined `/nix/store/f583rkdjbsdx7dbb7p3vf2hglv2nz03v-soulseekqt-2018-1-30/share/icons/hicolor/16x16/apps/SoulseekQt.png' @ error/convert.c/ConvertImageCommand/3235.

fix, i did a slight modification by removing patchPhase, i think it's simpler like that.


Routing mesh listens on IPv6, but does not function correctly

I created a redis service as follows:
docker service create --replicas 1 --name redis -p 6379:6379 redis

When I run telnet localhost 6379, telnet connects to ::1. The connection succeeds, but whatever I type, I never receive a response.
When I run telnet <IP_ADDRESS> 6379, things work as expected. I get responses to my commands from the redis daemon.
Client:
 Version:      1.12.3
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   6b644ec
 Built:        Wed Oct 26 21:39:14 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.12.3
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   6b644ec
 Built:        Wed Oct 26 21:39:14 2016
 OS/Arch:      linux/amd64

is this related to https://github.com/docker/libnetwork/pull/968, and https://github.com/docker/docker/issues/20569 ?

@thaJeztah No it is not related.

Verified that the same problem happens for non-swarm services also with userland-proxy=false. Issue here is that the iptables rules to do port forwarding is setup only for ipv4. With docker-proxy process it works because the proxy process binds to both v4 and v6 addresses.

@sanimej Should this be added to 1.13?

@thaJeztah Its not a must-have for 1.13.

Dispatching to @mavenugo for routing

Any update on this issue?

Any updates, please? It seems that the issue still exists in 19.03.6.


Fixed d165f05 to avoid missing dispatch and add actually nested support.

Am I right in thinking this PR should be closed since #108 was merged, @klueska ?


Technology comparison pages returning 400 error

@AliasIO  - I noticed that comparison pages for technologies are not working.
Example URL - https://www.wappalyzer.com/compare/vtex-vs-shopify/

Fixed, thank you.


Cannot find ways to generate workloads for given CNN models

Dear Authors,
I just installed the tool and ran it following the instructions, and now it can work well. However, I found that the workloads in the astra-sim\inputs\workload\ASTRA-sim-1.0 are pre-defined files. I also read the readme and know that each subsequent line in a file like the Resnat file describes a layer with the format of layer description including layer name, reserved variable, FP compute time, etc. However, I have no idea of how to get such information with only the CNN model and hardware platform parameters.
I tried to find some ways to generate such workload files and found that there should be a  "scripts/workload_generator/gen_astrasim_workload_input.py" file to generate such workload mentioned in https://github.com/astra-sim/astra-sim/issues/34. However, it seems that the file is not in the installation package based on the current version. I have attached the screenshot of the hierarchy of the installed files. I also cannot find this file in the github folder. Could you please let me know where to find these codes? If it requires other installation, would it be possible to provide any instructions? Thank you.

I found there is a scripts/workload_generator/gen_astrasim_workload_input.py code is in the astra-sim 1.0 branch. However, when I try to follow the instructions to compiling & running analytical backend as the network simulator, I got the following error.

I tried to bypass the compilation and generate the workload files using the gen_astrasim_workload_input.py directly but also met an error shown as follows.

It seems that the SCALE-Sim should be here after I build the backend. Could anyone provide any help? Thank you.

The workload generator, scripts/workload_generator/gen_astrasim_workload_input.py, is not compatible with ASTRA-sim 2.0. To generate synthetic workloads, please use the et_generator in Chakra.

Dear @TaekyungHeo,
Thank you for the explanation. However, my problem is that given a CNN network (e.g. Conv layers with input channel, output channel, kernel size, etc.) and NPU platform (e.g. hardware constraint for each NPU and the connection between each NPU), we can get the estimated latency. From et_generator, I found that the inputs are already represented as computation and communication for each node but I have no idea to generate these information is I only have the CNN network. Thus, I found the et_converter can convert the CNN workload (e.g. ResNet) into executed .et. However, after I study the et_converter, I found the workload files have already given the computation time, communication time, etc. for each Conv layer instead of the initial Conv parameters. That's why I am wondering if there is a way to know such workload information with the Conv parameters and hardware constraints. Thank you.


CASS-18606 try2

Thanks for sending a pull request! Here are some tips if you're new here:

Ensure you have added or run the appropriate tests for your PR.
Be sure to keep the PR description updated to reflect all changes.
Write your PR title to summarize what this PR proposes.
If possible, provide a concise example to reproduce the issue for a faster review.
Read our contributor guidelines
If you're making a documentation change, see our guide to documentation contribution

Commit messages should follow the following format:
<One sentence description, usually Jira title or CHANGES.txt summary>

<Optional lengthier description (context on patch)>

patch by <Authors>; reviewed by <Reviewers> for CASSANDRA-#####

Co-authored-by: Name1 <email1>
Co-authored-by: Name2 <email2>

The Cassandra Jira

removed the nodetool adoc files, they are auto-generated.
Also removed trailing white space that was added to a number of files.
(@polandll any chance can you update your editor/IDE to ensure these don't get added in the future, please)

Also noticed some vector search docs came in with this one.  Doesn't matter, it's coming in soon anyway…

Committed https://github.com/apache/cassandra/commit/1e2c88fff832d891b296165e9adda786182e850d


Is it possible to compile with iar compiler.

Is there an option to connect iar workbench or compiler with vcpkg
Thanks

I guess this functionality is related to CMake rather than vcpkg. Or you mean automatically integrate vcpkg paths into IAR?

Hi,
I mean both points.
Thanks

CMake with IAR could be possible: https://github.com/Kitware/CMake/blob/master/Modules/Compiler/IAR.cmake
But integrate the paths into IAR workspace may need some work

This issue hasn’t been updated in a year; if it is still an issue, please reopen.


Removed preset security policy cmdlets

Per email request

Docs Build status updates of commit 9a9c792:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Disable-ATPProtectionPolicyRule.md
:bulb:Suggestion

Details

exchange/exchange-ps/exchange/Disable-EOPProtectionPolicyRule.md
:bulb:Suggestion

Details

exchange/exchange-ps/exchange/Enable-ATPProtectionPolicyRule.md
:bulb:Suggestion

Details

exchange/exchange-ps/exchange/Enable-EOPProtectionPolicyRule.md
:bulb:Suggestion

Details

exchange/exchange-ps/exchange/Get-ATPBuiltInProtectionRule.md
:bulb:Suggestion

Details

exchange/exchange-ps/exchange/Get-ATPProtectionPolicyRule.md
:bulb:Suggestion

Details

exchange/exchange-ps/exchange/Get-EOPProtectionPolicyRule.md
:bulb:Suggestion

Details

exchange/exchange-ps/exchange/New-ATPBuiltInProtectionRule.md
:bulb:Suggestion

Details

exchange/exchange-ps/exchange/New-ATPProtectionPolicyRule.md
:bulb:Suggestion

Details

exchange/exchange-ps/exchange/New-EOPProtectionPolicyRule.md
:bulb:Suggestion

Details

exchange/exchange-ps/exchange/Remove-ATPProtectionPolicyRule.md
:bulb:Suggestion

Details

exchange/exchange-ps/exchange/Remove-EOPProtectionPolicyRule.md
:bulb:Suggestion

Details

exchange/exchange-ps/exchange/Set-ATPBuiltInProtectionRule.md
:bulb:Suggestion

Details

exchange/exchange-ps/exchange/Set-ATPProtectionPolicyRule.md
:bulb:Suggestion

Details

exchange/exchange-ps/exchange/Set-EOPProtectionPolicyRule.md
:bulb:Suggestion

Details

exchange/exchange-ps/exchange/exchange.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/mapping/serviceMapping.json
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/Disable-ATPProtectionPolicyRule.md

[Suggestion: file-not-redirected - See documentation] File exchange/exchange-ps/exchange/Disable-ATPProtectionPolicyRule.md was deleted without redirection. To avoid broken links, add a redirection.

exchange/exchange-ps/exchange/Disable-EOPProtectionPolicyRule.md

[Suggestion: file-not-redirected - See documentation] File exchange/exchange-ps/exchange/Disable-EOPProtectionPolicyRule.md was deleted without redirection. To avoid broken links, add a redirection.

exchange/exchange-ps/exchange/Enable-ATPProtectionPolicyRule.md

[Suggestion: file-not-redirected - See documentation] File exchange/exchange-ps/exchange/Enable-ATPProtectionPolicyRule.md was deleted without redirection. To avoid broken links, add a redirection.

exchange/exchange-ps/exchange/Enable-EOPProtectionPolicyRule.md

[Suggestion: file-not-redirected - See documentation] File exchange/exchange-ps/exchange/Enable-EOPProtectionPolicyRule.md was deleted without redirection. To avoid broken links, add a redirection.

exchange/exchange-ps/exchange/Get-ATPBuiltInProtectionRule.md

[Suggestion: file-not-redirected - See documentation] File exchange/exchange-ps/exchange/Get-ATPBuiltInProtectionRule.md was deleted without redirection. To avoid broken links, add a redirection.

exchange/exchange-ps/exchange/Get-ATPProtectionPolicyRule.md

[Suggestion: file-not-redirected - See documentation] File exchange/exchange-ps/exchange/Get-ATPProtectionPolicyRule.md was deleted without redirection. To avoid broken links, add a redirection.

exchange/exchange-ps/exchange/Get-EOPProtectionPolicyRule.md

[Suggestion: file-not-redirected - See documentation] File exchange/exchange-ps/exchange/Get-EOPProtectionPolicyRule.md was deleted without redirection. To avoid broken links, add a redirection.

exchange/exchange-ps/exchange/New-ATPBuiltInProtectionRule.md

[Suggestion: file-not-redirected - See documentation] File exchange/exchange-ps/exchange/New-ATPBuiltInProtectionRule.md was deleted without redirection. To avoid broken links, add a redirection.

exchange/exchange-ps/exchange/New-ATPProtectionPolicyRule.md

[Suggestion: file-not-redirected - See documentation] File exchange/exchange-ps/exchange/New-ATPProtectionPolicyRule.md was deleted without redirection. To avoid broken links, add a redirection.

exchange/exchange-ps/exchange/New-EOPProtectionPolicyRule.md

[Suggestion: file-not-redirected - See documentation] File exchange/exchange-ps/exchange/New-EOPProtectionPolicyRule.md was deleted without redirection. To avoid broken links, add a redirection.

exchange/exchange-ps/exchange/Remove-ATPProtectionPolicyRule.md

[Suggestion: file-not-redirected - See documentation] File exchange/exchange-ps/exchange/Remove-ATPProtectionPolicyRule.md was deleted without redirection. To avoid broken links, add a redirection.

exchange/exchange-ps/exchange/Remove-EOPProtectionPolicyRule.md

[Suggestion: file-not-redirected - See documentation] File exchange/exchange-ps/exchange/Remove-EOPProtectionPolicyRule.md was deleted without redirection. To avoid broken links, add a redirection.

exchange/exchange-ps/exchange/Set-ATPBuiltInProtectionRule.md

[Suggestion: file-not-redirected - See documentation] File exchange/exchange-ps/exchange/Set-ATPBuiltInProtectionRule.md was deleted without redirection. To avoid broken links, add a redirection.

exchange/exchange-ps/exchange/Set-ATPProtectionPolicyRule.md

[Suggestion: file-not-redirected - See documentation] File exchange/exchange-ps/exchange/Set-ATPProtectionPolicyRule.md was deleted without redirection. To avoid broken links, add a redirection.

exchange/exchange-ps/exchange/Set-EOPProtectionPolicyRule.md

[Suggestion: file-not-redirected - See documentation] File exchange/exchange-ps/exchange/Set-EOPProtectionPolicyRule.md was deleted without redirection. To avoid broken links, add a redirection.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


chore: Profile page UI updates;

We must change the text, DMed you

We must change the text, DMed you


chore: extract tff example logic from the root mod

Extract the distributed TFF example logic from the root Terraform module and put that in a dedicated scripts and README instructions. We're losing some automation, but we gain in modularity.

Can you please rebase this PR against the main branch ? You have conflict in some of your files

@laurentgrangeau done :)

@laurentgrangeau ping :)


Unable to find image 'autoware/autoware:1.8.0-kinetic' locally docker: Error response from daemon: manifest for autoware/autoware:1.8.0-kinetic not found.

Fill-out only one section depending on whether you are reporting a bug or a new feature.
Bug
Expected Behavior
Actual Behavior
Steps to Reproduce the Problem

Specifications

Ubuntu version:
ROS version:
Autoware branch:

New Feature
Describe where the new feature request comes from. Add a high-level diagram of how the new feature should work.

currently Autoware is not support 1.8 in docker. you can use  1.7.0-kinetic to start autoware

According to your suggestion,this problem is solved,Thank you!
ziyi.wang<EMAIL_ADDRESS>于2018年9月25日周二 下午8:12写道：

currently Autoware is not support 1.8 in docker. you can use 1.7.0-kinetic
to start autoware
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CPFL/Autoware/issues/1537#issuecomment-424317889, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AR3qbR9pAgCauitfHrPMkQQnHViWrEmFks5ueh2_gaJpZM4W4eBm
.

Thanks!

I am facing the same issue. Followed the Docker installation instructions as in: https://github.com/CPFL/Autoware/wiki/Generic-x86-Docker (Using the Autoware Docket setup Cast 1)
For the final instruction
$ sudo sh run.sh kinetic
I am getting the following error:
Use kinetic Shared directory: /home/root/shared_dir Unable to find image 'autoware/autoware:1.8.0-kinetic' locally docker: Error response from daemon: manifest for autoware/autoware:1.8.0-kinetic not found. See 'docker run --help'.
From going through similar issues here, I see that I have to use 1.7 instead of 1.8. How do I do this downgrade? Just an edit in the command (and run.sh file) or should I start a fresh installation with an older build?
Thanks

Followed @jacks808 's comment I modified the last line in run.sh and it seems to work fine. I mean just change the line  autoware/autoware:1.8.0-kinetic to autoware/autoware:1.7.0-kinetic and it works.
Here is the link to the original documentation : https://github.com/CPFL/Autoware/wiki/Generic-x86-Docker
On another note, since the documentation has an error, maybe you guys can make it clear in the quick start guide as well?

Sorry, I updated the documentation.

Wow thanks so much for the response. I wasn’t expecting anyone to really notice... thanks!
Sent from my mobile device. Please excuse brevity and typos, thanks!

On Nov 6, 2018, at 12:15 AM, Kenji Funaoka<EMAIL_ADDRESS>wrote:
Sorry, I updated the documentation.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Attempted to define built-in macro more than once

Hi, I'm getting an error when trying to compile a contract with a drink-based test.
[...]
error[E0773]: attempted to define built-in macro more than once
    --> /Users/maciej/.rustup/toolchains/stable-x86_64-apple-darwin/lib/rustlib/src/rust/library/core/src/macros/mod.rs:1449:5
     |
1449 |     macro_rules! cfg {
     |     ^^^^^^^^^^^^^^^^
     |
note: previously defined here
    --> /Users/maciej/.rustup/toolchains/stable-x86_64-apple-darwin/lib/rustlib/src/rust/library/core/src/macros/mod.rs:1449:5
     |
1449 |     macro_rules! cfg {
     |
[...]

Here's my Cargo.toml:
[package]
name = "stablecoin"
version = "0.1.0"
edition = "2021"
authors = ["Maciej Nadolski", "Yuchen Lin"]

[dependencies]
drink = "0.8.0"
ink = { version = "5.0.0", default-features = false }

[lib]
path = "src/stablecoin.rs"

[features]
default = ["std"]
std = [
    "ink/std",
]
ink-as-dependency = []
e2e-tests = []

Please help :)

can you add your test code please?

The contents of src/stablecoin.rs are an exact copy of lib.rs from examples/quick-start-with-drink:
#![cfg_attr(not(feature = "std"), no_std, no_main)]

/// This is the classical flipper contract. It stores a single `bool` value in its storage. The
/// contract exposes:
/// - a constructor (`new`) that initializes the `bool` value to the given value,
/// - a message `flip` that flips the stored `bool` value from `true` to `false` or vice versa,
/// - a getter message `get` that returns the current `bool` value.
///
/// Additionally, we use the `debug_println` macro from the `ink_env` crate to produce some debug
/// logs from the contract.
#[ink::contract]
mod flipper {
    use ink::env::debug_println;

    #[ink(storage)]
    pub struct Flipper {
        value: bool,
    }

    impl Flipper {
        #[ink(constructor)]
        pub fn new(init: bool) -> Self {
            debug_println!("Initializing contract with: `{init}`");
            Self { value: init }
        }

        #[ink(message)]
        pub fn flip(&mut self) {
            debug_println!("Previous value: `{}`", self.value);
            self.value = !self.value;
            debug_println!("Flipped to:     `{}`", self.value);
        }

        #[ink(message)]
        pub fn get(&self) -> bool {
            debug_println!("Reading value from storage");
            self.value
        }
    }
}

/// We put `drink`-based tests as usual unit tests, into a test module.
#[cfg(test)]
mod tests {
    use drink::{
        sandbox_api::contracts_api::decode_debug_buffer,
        session::{Session, NO_ARGS, NO_ENDOWMENT, NO_SALT},
    };

    /// `drink` automatically discovers all the contract projects that your tests will need. For
    /// every such dependency (including the contract from the current crate), it will generate a
    /// [`ContractBundle`](drink::session::ContractBundle) object that contains the compiled contract's code
    /// and a special transcoder, which is used to encode and decode the contract's message
    /// arguments. Such a bundle will be useful when deploying a contract.
    ///
    /// To get a convenient way for obtaining such bundles, we can define an empty enum and mark
    /// it with the [`drink::contract_bundle_provider`](drink::contract_bundle_provider) attribute.
    /// From now on, we can use it in all testcases in this module.
    #[drink::contract_bundle_provider]
    enum BundleProvider {}

    /// Now we write the simplest contract test, that will:
    /// 1. Deploy the contract.
    /// 2. Call its `flip` method.
    /// 3. Call its `get` method and ensure that the stored value has been flipped.
    ///
    /// We can use the [`drink::test`](drink::test) attribute to mark a function as a `drink` test.
    /// This way we ensure that all the required contracts are compiled and built, so that we don't
    /// have to run `cargo contract build` manually for every contract dependency.
    ///
    /// For convenience of using `?` operator, we mark the test function as returning a `Result`.
    ///
    /// `drink::test` will already provide us with a `Session` object. It is a wrapper around a runtime and it exposes
    /// a broad API for interacting with it. Session is generic over the runtime type, but usually and by default, we
    /// use `MinimalSandbox`, which is a minimalistic runtime that allows using smart contracts.
    #[drink::test]
    fn deploy_and_call_a_contract(mut session: Session) -> Result<(), Box<dyn std::error::Error>> {
        // Now we get the contract bundle from the `BundleProvider` enum. Since the current crate
        // comes with a contract, we can use the `local` method to get the bundle for it.
        let contract_bundle = BundleProvider::local()?;

        // We can now deploy the contract.
        let _contract_address = session.deploy_bundle(
            // The bundle that we want to deploy.
            contract_bundle,
            // The constructor that we want to call.
            "new",
            // The constructor arguments (as stringish objects).
            &["true"],
            // Salt for the contract address derivation.
            NO_SALT,
            // Initial endowment (the amount of tokens that we want to transfer to the contract).
            NO_ENDOWMENT,
        )?;

        // Once the contract is instantiated, we can call the `flip` method on the contract.
        session.call(
            // The message that we want to call.
            "flip",
            // The message arguments (as stringish objects). If none, then we can use the `NO_ARGS`
            // constant, which spares us from typing `&[]`.
            NO_ARGS,
            // Endowment (the amount of tokens that we want to transfer to the contract).
            NO_ENDOWMENT,
        )??;

        // Finally, we can call the `get` method on the contract and ensure that the value has been
        // flipped.
        //
        // `Session::call` returns a `Result<MessageResult<T>, SessionError>`, where `T` is the
        // type of the message result. In this case, the `get` message returns a `bool`, and we have
        // to explicitly hint the compiler about it.
        let result: bool = session.call("get", NO_ARGS, NO_ENDOWMENT)??;
        assert_eq!(result, false);

        Ok(())
    }

    /// In this testcase we will see how to get and read debug logs from the contract.
    #[drink::test]
    fn get_debug_logs(mut session: Session) -> Result<(), Box<dyn std::error::Error>> {
        session.deploy_bundle(
            BundleProvider::local()?,
            "new",
            &["true"],
            NO_SALT,
            NO_ENDOWMENT,
        )?;

        // `deploy_bundle` returns just a contract address. If we are interested in more details
        // about last operation (either deploy or call), we can get a `Record` object and use its
        // `last_deploy_result` (or analogously `last_call_result`) method, which will provide us
        // with a full report from the last contract interaction.
        //
        // In particular, we can get the decoded debug buffer from the contract. The buffer is
        // just a vector of bytes, which we can decode using the `decode_debug_buffer` function.
        let decoded_buffer = &session.record().last_deploy_result().debug_message;
        let encoded_buffer = decode_debug_buffer(decoded_buffer);

        assert_eq!(encoded_buffer, vec!["Initializing contract with: `true`"]);

        Ok(())
    }

    /// In this testcase we will see how to work with multiple contracts.
    #[drink::test]
    fn work_with_multiple_contracts(
        mut session: Session,
    ) -> Result<(), Box<dyn std::error::Error>> {
        let bundle = BundleProvider::local()?;

        // We can deploy the same contract multiple times. However, we have to ensure that the
        // derived contract addresses are different. We can do this by providing using different
        // arguments for the constructor or by providing a different salt.
        let first_address =
            session.deploy_bundle(bundle.clone(), "new", &["true"], NO_SALT, NO_ENDOWMENT)?;
        let _second_address =
            session.deploy_bundle(bundle.clone(), "new", &["true"], vec![0], NO_ENDOWMENT)?;
        let _third_address =
            session.deploy_bundle(bundle, "new", &["false"], NO_SALT, NO_ENDOWMENT)?;

        // By default, when we run `session.call`, `drink` will interact with the last deployed
        // contract.
        let value_at_third_contract: bool = session.call("get", NO_ARGS, NO_ENDOWMENT)??;
        assert_eq!(value_at_third_contract, false);

        // However, we can also call a specific contract by providing its address.
        let value_at_first_contract: bool =
            session.call_with_address(first_address, "get", NO_ARGS, NO_ENDOWMENT)??;
        assert_eq!(value_at_first_contract, true);

        Ok(())
    }
}

please try:

updating drink to 0.17.0 (this one is working with ink 5, version 0.8.x was for ink 4 if I remember correctly)
move drink to dev-dependencies

@MaciejNadolski98 did it help?


Fixed React version in HTML example.

React.createElement is undefined and throws an error when using react-0.10.0.

Yikes, thanks.


SPI Master - second device (spi_bus_add_device) does not work (IDFGH-91)

When using spi_bus_add_device to add two devices to a SPI bus with different CS lines, the second device does not operate properly if one is half duplex and the other full duplex.  If both are full duplex, the issue does not occur.

Can you expand a bit on 'does not operate properly'? What symptoms do you notice?

Due to time constraints, I did not have the opportunity to explore what exactly "does not operate properly" entails.
The configuration being tested is an ILI9341 SPI display as the first SPI device and a XPT2046 touch panel interface as the second SPI device.  The first device is configured in half duplex mode while the second is full duplex.  When either is the only device added (spi_bus_add_device), correct operation is observed.  When the display is added followed by the touch panel, the touch panel does not report correct data.  Changing the display device to full duplex yields correct operation of both devices.

Hi @OtherCrashOverride ,
Sorry for long long no-reply ....
The issue was reported quite long ago, during which we have introduced a similar issue on 4.0 and fix it on master (see: https://github.com/espressif/esp-idf/issues/4641). (The fix backporting to 4.0, 4.1 and 4.2 are in progress....)
Please help test again, or wait for patches to 4.0-4.2. (they will also close this issue upon merged)

Thanks for reporting and sorry for slow turnaround.

Fix on release/4.0 is https://github.com/espressif/esp-idf/commit/9e9ebed8612a36ea334ecd5a23d20c71afa72da2
Fix on release/4.1 is https://github.com/espressif/esp-idf/commit/9a8dbbc281ce642868b4b3f701525cf92458f0b3
Fix on release/4.2 is https://github.com/espressif/esp-idf/commit/e3fd4b097b350403a8dd1ec7cdc0f40b7f9aeedb

Feel free to reopen.


Implement GetAddressVersion interop

Summary or solution description
Added the address_version property
How to Reproduce
https://github.com/CityOfZion/neo3-boa/blob/85122abed851f38797b9220e76f17099bcfc7f0c/boa3_test/test_sc/interop_test/runtime/AddressVersion.py#L1-L7
Tests
https://github.com/CityOfZion/neo3-boa/blob/85122abed851f38797b9220e76f17099bcfc7f0c/boa3_test/tests/compiler_tests/test_interop/test_runtime.py#L647-L656
Platform:

OS: Windows 10 x64
Python version: Python 3.7

Task linked: CU-2naaa44 Implement GetAddressVersion interop


[ocilib] Disable warning C4191

In an internal version of Visual Studio, ocilib failed with following error:
E:\0207\vcpkg\buildtrees\ocilib\x64-windows-rel\v4.7.3-f3f6b8726c.clean\src\array.c(300,71): error C2220: the following warning is treated as an error
E:\0207\vcpkg\buildtrees\ocilib\x64-windows-rel\v4.7.3-f3f6b8726c.clean\src\array.c(300,71): warning C4191: 'type cast': unsafe conversion from 'int (__cdecl *)(OCI_Array *,void **)' to 'POCI_LIST_FIND'

This is by design, add the warning to the list of disabled warnings to fix this error.
I have submitted an issue on upstream: https://github.com/vrogier/ocilib/issues/297

Thanks!


Update AnimatedNode.js

Fix issue: TypeError: undefined is not a function, js engine: hermes in some animated modules

Summary

Changelog

[CATEGORY] [TYPE] - Message
Test Plan

@FonDorn in theory flow should catch this issue. Also, could you add a repro case, a test plan, and changelog?

Closing as there was no follow up. Feel free to re-open if the issue persists and you can provide a repro case


error generating manifest: unresolved nodes in v225.0.0

When I ran "./scripts/generate-manifest openstack" in v225.0.0 with no modification,
the error occurred like below.
2017/04/24 09:16:17 error generating manifest: unresolved nodes:
(( merge ))     in /service-v3/cf-rabbitmq-release/templates/cf-rabbitmq-infrastructure-openstack.yml  networks        (networks)

Hi,
As a result of using BOSH Links and the new BOSH CLI v2, we've changed the way we generate manifests. As a result the old spiff way is no longer supported.
You can find instructions on how to generate a manifest in our readme.
@BenChapman & @dlresende


Waiting for mining the transaction

Hi, I submitted my profile and I am waiting for my transaction to mine, however I had a problem with my internet and I had to reload the page and now I don't see the information, video and photo I uploaded, now it appears completely empty, then I check in Metamask and it tells me that my transaction to Poh is still pending. My concern is that my profile information is lost and when the transaction is approved I will lose my ETH. I would really appreciate your help to verify that my profile information is ok, or if I should cancel the transaction (which I would not like to do because I would lose the gas fee).

@sofiabaur If your transaction is pending (and visible on Etherscan), it means that your profile details were uploaded and submitted successfully. So it will work even if you refresh your browser before the transaction completes.
Please paste your ETH address here if you would like us to check the transaction status for you.


[Feature] keep the same options choose between web client and windows destop client

Motivation

Now the client version I used is v1.8.2(the latest), but I found some different options between web client and windows destop client. Some differences like the picture above. It make me confused. For example, I must use the windows destop client rather than the web client, because I need to close SSL Secure for test.
Detailed design
Like the title what I said, keep the same options choose between different clients. It will make same feeling for users.
Alternatives
More detail (optional)

Yes, @WangLei1993 thank you for your feedback, currently, our web client and desktop client are not fully synchronized, we will fix this issue immediately.

Now we've synced to the web, thanks for using it. http://www.emqx.io/online-mqtt-client


Assets with a "-" return error

Description
I am trying to upgrade to v3.0.7
I am running into an error with assets that have a "-" in the file name.
Steps

Create an asset with a "-" in it like file-name.jpg
Try and compile

Output
Liquid Exception: no implicit conversion of Symbol into Integer

Thanks!

Please provide a trace.  That exception is non-useful. Thanks.

Hmm I'm having trouble repeating this now. Not sure what I changed. Looks like might have been an ASCII vs UTF8 copy/paste issue.
Let me dig into more.

I think a bad _config was causing the issue. I had CDN under the 2.x syntax instead of 3.x syntax.
Strange how that went to the filename as the error.
Working now!


Gossipsub backpressure

Backpressure PR to our fork

lol, was messing with gh cli, didn't mean to merge this early. But it works.

Yeah nice. Two things I noticed.

We currently don't handle the cases where the queue's are full. I guess this is a future PR.  I.e we don't handle the errors when we try to send.
I know I'm missing something, but why are we using an unbounded channel for the priority queue and not just another bounded channel like we do with the non-priority?

1 - when the queue is full for every peer we return InsuficientPeers for individual peers we just drop the messages yeah, would you like to add extra logic for that?
2 - no worries :D, because DRAFT and PRUNE messages should still be sent if the queue is full, so I used an AtomicUsize for the cap of Publish messages


Intersection of Sets

Related Issue

#1469

Proposed Changes

Cpp file added
readme file added

Additional Info
In this, I Have explained all about the intersection:

What is the meaning of the intersection of sets?
properties of intersection in sets?
working example image

Checklist

[x] ✅ My code follows the code style of this project.
[x] 📝 My change requires a change to the README.
[x] 🎀 I have updated the README accordingly.
[x] 👀 I have read the CONTRIBUTING document.
[x] ✨ I have added tests to cover my changes.
[x] 🚩 All new and existing tests passed.
[x] 🌟 ed the repo

The intersection of two sets A and B is the set of all elements common in both sets as identical in both A and B set, The intersection of A and B is denoted as A ∩ B.

#1469

Any GCC compiler

[x] New feature (non-breaking change which adds functionality)

Output Screenshots

Screenshot #1

Code  Output

Please add screenshots of code with proper comments

of code with proper

ScreenShots in the readme or any other place and I have given well-defined comments in the CPP file @aditi944 .

You have to add screenshots in the readme

You have to add screenshots in the readme

Screenshots added @aditi944  now check readme.


Fixing small typo in signal source module

Simply fixed a typo from atored to stored

I just realized that I did not create a feature branch hen doing this pull request. I could close this request and open another from a branch different than master from my repository

Hi Damian,
thanks for your pull-request. It is generally required to open a new branch, but for a small PR is ok as it is.  The change has been pushed online.


Ocean density must be referred to as a reference density

For detail-oriented folks, the variable
https://github.com/CliMA/CLIMAParameters.jl/blob/f779db6fa86f0bfce9de4ee31757fed1fe8b75ed/src/Planet/planet_parameters.jl#L36
is somewhat stressful, as the ocean density is not constant.
This quantity is really a reference density that must be defined as part of the Boussinesq approximation; it's important to identify it as a "reference" (and therefore a parameter) rather than an absolute value.
It's appropriate to include this parameter here, because it ends up appearing in a few unrelated places in ocean codes (and also user's scripts). But I think it should be called ocean_reference_density so as to not cause unwarranted stress for users that want to carefully manage their science.

Sounds like a good idea. In the atmosphere, we append ref (e.g., T_min_ref) to indicate reference values.

For sure. That can work for mathematical notation form. Going along with #41 I think we should use the literal word reference to indicate reference values for English variable names.


Render the place image if no map data is provided

Problem Description:
There are records with no map but have images. These images are not rendered.
Expected Behavior/Solution:
If there is no map data and an image exists in the data, render the image.
Requirements:

[x] Add function to check if place has map data and images

Needed for promotion:
If an item on the list is not needed, it should be crossed off but not removed.

[x] Wireframe/Mockup - Kam
[ ] ~Committee discussions - Sarah~
[ ] ~Feasibility/Team discussion - Sarah~
[ ] ~Backend requirements - TBD~
[x] Frontend requirements - Kam
[ ] ~Are new regression tests required for QA - Amy~
[ ] ~Questions~

UAT/LUX Examples:

TST: https://lux-front-tst.collections.yale.edu/view/place/bb8f6825-7a0c-4085-95c5-3a4599dcbd23
PRD: https://lux.collections.yale.edu/view/place/bb8f6825-7a0c-4085-95c5-3a4599dcbd23

Dependencies/Blocks:
N/A
Related Github Issues:
N/A
Related links:

Bugherd: https://www.bugherd.com/projects/284041/tasks/2266

Wireframe/Mockup:

@kamerynB, @miketullo95  and @roamye - this one needs a mockup before promoting

@prowns A mockup has been added.
cc: @miketullo95 @roamye

Approved by UAT

Closing as this ticket was marked as Done the week of 6/3.


Capability to set a alternative default profile when launchen the terminal over the windows explorer context menu

Description of the new feature/enhancement
At the moment we can specify the default profile when the windows terminal is getting started. Under Windows 11 I miss the option, to specify which profile should be getting used when i start the terminal over the windows explorer context menu.
At least in my setup i´m normaly using WSL so thats my standard profile. But when I´m starting the terminal over the context menu, i want to use cmd. Of course i can manually open the cmd but then i´m losing the directory context and have to manually cd into the right path.
I would love to see an option where i can specifiy, which profile should be used when the terminal is getting started over the explorer context menu.
Proposed technical implementation details (optional)
Maybe a second dropdown in the settings?

Alternatively, would #6111 work for you/? Instead of specifying a different default profile for the context menu, just changing the menu to allow you to pick whichever profile you want?

@zadjii-msft yes, that would be working too. Looks like a more straight forward solution! I will close this issue in preference for #6111


Add FNV-1 and FNV-1a hash operations

Adds operations for the Fowler–Noll–Vo (FNV) hash algorithm.
This uses the native BigInt though eslint complains about it being undefined, unless I add es2020.
It seems the BigNumber.js library does not support bitwise operations, so I'm not sure what's a best approach for this.

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


Bug with pydantic when extending_ports on some components

When extending ports on some multi-port components, pydantic throws an error.
Code:
c = gf.components.crossing()
p = 3
c = gf.add_padding_container(c, default=0, top=p, bottom=p)
df = write_sparameters_meep(c, overwrite=True, animate=False)

Traceback:
Traceback (most recent call last):
  File "test_write_sparameters_meep.py", line 59, in <module>
    test_sparameterNxN_crossing(None)
  File "test_write_sparameters_meep.py", line 36, in test_sparameterNxN_crossing
    df = write_sparameters_meep(c, overwrite=True, animate=False)
  File "pydantic/decorator.py", line 39, in pydantic.decorator.validate_arguments.validate.wrapper_function
    import itertools
  File "pydantic/decorator.py", line 133, in pydantic.decorator.ValidatedFunction.call
    func.__dict__.update(kw)
  File "pydantic/decorator.py", line 200, in pydantic.decorator.ValidatedFunction.execute
    """
  File "/home/simbil/Github/gdsfactory/gdsfactory/simulation/gmeep/write_sparameters_meep.py", line 404, in write_sparameters_meep
    **settings,
  File "pydantic/decorator.py", line 39, in pydantic.decorator.validate_arguments.validate.wrapper_function
    import itertools
  File "pydantic/decorator.py", line 133, in pydantic.decorator.ValidatedFunction.call
    func.__dict__.update(kw)
  File "pydantic/decorator.py", line 200, in pydantic.decorator.ValidatedFunction.execute
    """
  File "/home/simbil/Github/gdsfactory/gdsfactory/simulation/gmeep/write_sparameters_meep.py", line 270, in sparameter_calculation
    **settings,
  File "pydantic/decorator.py", line 39, in pydantic.decorator.validate_arguments.validate.wrapper_function
    import itertools
  File "pydantic/decorator.py", line 133, in pydantic.decorator.ValidatedFunction.call
    func.__dict__.update(kw)
  File "pydantic/decorator.py", line 200, in pydantic.decorator.ValidatedFunction.execute
    """
  File "/home/simbil/Github/gdsfactory/gdsfactory/simulation/gmeep/get_simulation.py", line 131, in get_simulation
    if extend_ports_length
  File "/home/simbil/Github/gdsfactory/gdsfactory/cell.py", line 166, in _cell
    component = func(*args, **kwargs)
  File "pydantic/decorator.py", line 39, in pydantic.decorator.validate_arguments.validate.wrapper_function
    import itertools
  File "pydantic/decorator.py", line 133, in pydantic.decorator.ValidatedFunction.call
    func.__dict__.update(kw)
  File "pydantic/decorator.py", line 200, in pydantic.decorator.ValidatedFunction.execute
    """
  File "/home/simbil/Github/gdsfactory/gdsfactory/components/extension.py", line 154, in extend_ports
    layer=port.layer,
  File "/home/simbil/Github/gdsfactory/gdsfactory/cell.py", line 166, in _cell
    component = func(*args, **kwargs)
  File "pydantic/decorator.py", line 39, in pydantic.decorator.validate_arguments.validate.wrapper_function
    import itertools
  File "pydantic/decorator.py", line 132, in pydantic.decorator.ValidatedFunction.call
    func.__module__ = getattr(self, 'module', callermodule)
  File "pydantic/decorator.py", line 129, in pydantic.decorator.ValidatedFunction.init_model_instance
    callermodule = '?'
  File "pydantic/main.py", line 406, in pydantic.main.BaseModel.__init__
pydantic.error_wrappers.ValidationError: 1 validation error for Straight
cross_section
  none is not an allowed value (type=type_error.none.not_allowed)

Elapsed run time = 0.0665 s

Also happens with other non-straight components like coupler

Tentatively, it seems the issue occurs when get_simulation attempts to extend_ports in components.extension.py. This was changed recently in commit ec8b2c3 (https://github.com/simbilod/gdsfactory/commit/ec8b2c3ab5c83cbd195dad1629ed169571fd1adc)


Helm integration

Related to #101

I signed it!

This looks cool!!!
I'll do a proper review after GDC this week. In the meantime, would be great to get this rebased down to a single commit (or you can do it post review, up to you)

@markmandel fixed to a single commit

@markmandel could you please setup integration with Travis CI or smth? Next step for Helm integration is create chart repository (I believe we can use github pages to publish charts) and automate building process using CI.
@fooock what do you think?

I will rebase down to a single commit when the review is finished

Can we also update the page docs/installing_agones.md:

to change the link of the install.yaml
add a small section about helm
link  install/agones/ if users wants more details.

So took this for a spin, and my GameServer couldn't transition from Started to Ready. Looking at the logs for the sdk sidecar, I see the following:
{
    "error": "could not retrieve GameServer default/cpp-simple-hlqhp: gameservers.stable.agones.dev \"cpp-simple-hlqhp\" is forbidden: User \"system:serviceaccount:default:agones-sdk\" cannot get gameservers.stable.agones.dev in the namespace \"default\"",
    "level": "error",
    "msg": "",
    "obj": "RequestReady",
    "queue": "stable.agones.dev.default.cpp-simple-hlqhp",
    "source": "*gameservers.SDKServer",
    "time": "2018-03-29T01:40:48Z"
}

{
    "level": "error",
    "msg": "could not retrieve GameServer default/cpp-simple-hlqhp: gameservers.stable.agones.dev \"cpp-simple-hlqhp\" is forbidden: User \"system:serviceaccount:default:agones-sdk\" cannot get gameservers.stable.agones.dev in the namespace \"default\"",
    "stack": [
        "agones.dev/agones/pkg/gameservers.(*SDKServer).updateState\n\t/go/src/agones.dev/agones/pkg/gameservers/sdkserver.go:166",
        "agones.dev/agones/pkg/gameservers.NewSDKServer.func3\n\t/go/src/agones.dev/agones/pkg/gameservers/sdkserver.go:119",
        "agones.dev/agones/pkg/util/workerqueue.(*WorkerQueue).processNextWorkItem\n\t/go/src/agones.dev/agones/pkg/util/workerqueue/workerqueue.go:97",
        "agones.dev/agones/pkg/util/workerqueue.(*WorkerQueue).runWorker\n\t/go/src/agones.dev/agones/pkg/util/workerqueue/workerqueue.go:73",
        "agones.dev/agones/pkg/util/workerqueue.(*WorkerQueue).(agones.dev/agones/pkg/util/workerqueue.runWorker)-fm\n\t/go/src/agones.dev/agones/pkg/util/workerqueue/workerqueue.go:115",
        "agones.dev/agones/vendor/k8s.io/apimachinery/pkg/util/wait.JitterUntil.func1\n\t/go/src/agones.dev/agones/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:133",
        "agones.dev/agones/vendor/k8s.io/apimachinery/pkg/util/wait.JitterUntil\n\t/go/src/agones.dev/agones/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:134",
        "agones.dev/agones/vendor/k8s.io/apimachinery/pkg/util/wait.Until\n\t/go/src/agones.dev/agones/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:88",
        "runtime.goexit\n\t/usr/local/go/src/runtime/asm_amd64.s:2337"
    ],
    "time": "2018-03-29T01:40:48Z"
}

Looks like there is a RBAC misconfiguration somewhere.

Scope of this PR
Add the ability to install Agones using the Helm package manager. For it, is necessary to maintain installation compatibility also through yaml, which is the currently supported mode.
It would be a good option to keep the two forms of installation, since not everything people uses Helm, apart from the fact that the use of Helm adds a series of prerequisites to the installation of Agones (like install Tiller in the cluster). Also keep in mind that maintaining two versions can be a nightmare and potentially prone to errors.
Why we need this?
The install.yaml file is big, supports multiple configurations and It can grow over time. It would be interesting to be able to extract variables like names of services, namespace, port range etc. It is also interesting to be able to divide the install.yaml file into multiple parts, so that its edition is more simple.
Pros os using Helm

Template system that will do variable substitution
Template system that does includes. We can make folders for serviceaccounts, crds...
Support plugin development
Manage install, updates, upgrade and test operations
Manage dependencies through requirements.yaml
Chart versioning
Hooks

Cons of using Helm

Need to install the Helm tools
Need to install Tiller in the cluster
Keep two versions in sync can be a pain and potentially prone to error
Maintain support for old Helm versions
More?

Things to consider
Apart from completing the documentation, we need to complete a series of requirements to complete this functionality. Before ship this we need to verify it (missing things?).
Build tools
We need to modify the CI system to be able to

Create a chart
Package the chart
Lint the chart to help find issues
Upload the chart to a repo

For this process, we can make a Helm plugin (WDYT?). Also, we need to modify the make do-release task, to zips up the install dir and grabs both the install.yaml and the helm chart.
Testing Helm charts
It would be necessary to carry out a series of e2e tests to verify that the changes do not affect the performance of Agones when installed using Helm. An idea for this process would be to create a Vagrantfile that installs minikube and Helm, and perform a series of checks such as:

GameServer status is Ready
Install simple-udp program and expect the ACK response usign nc
Chart tests
More?

Add char to repository
It has been talked about in issue #153. At first, a good option would be to use Kubernetes charts repo, since It is the site that is normally used. To be able to upload the chart to this repository we must meet a series of technical requirements indicated in the following url:

https://github.com/kubernetes/charts/blob/master/CONTRIBUTING.md

Conclusion
When this functionality is finished, the user will be able to install Agones in two different ways

YAML file
Helm

We provide a complete documentation of installing Agones with any of the two available methods. We will have an automatic e2e test system. The continuous integration system will be able to package the necessary files to be able to install Agones in any of the two ways, and have the same behavior. We will have a repository to upload the chart(s).
Surely I forget something, opinions?

So this feels like a lot of work for one PR - I think this is great scope for ALL of the packaging, but can we break this out into several PRs to be easier to test, integrate, review and work on in parallel?
(I have thoughts on a few of the design above - and maybe that should be moved into #101 as the overarching ticket for all the PRs), but here is a breakdown of separate PR work that I think would make sense, rather than trying do it all in one go. Happily take opinions.

Create a helm chart that can be installed and used on a cluster, with documentation
Migrate the build/ tooling to use Helm rather than sed templating
Write some form of make helm-test target (implementation details to be discussed)
Discuss chart repository options and potentially implement (depending on if we decide we need one at this stage - I don't think we should be pushing this to the k8s repo until we are 1.0, but we should discuss)
Update release process to include the above.

WDYT? Smaller pieces of work makes things easier.
Mark

I'm all in for smaller pieces.


Don't use fully qualified class name when it is in the same package.

Even if we have use imports to false, we shouldn't write the fully qualified class name when it is a sibling class (both are in the same package).

I'd rather recommend to change deobfuscator code to change variable/field names to prevent such cases


Move verification script in flutter_tools/test/general.shard/android/gradle_test.dart to integration.shard

removing assignment because I will be OOO

@jonahwilliams is this still relevant?

IIRC the original intention was to move all slow tests that used real implementations to the integration shard, general should be realtively fast and hermetic. I'm unsure if that was ever done, but if it doesn't seem important feel free to close this bug.


How to set the tenant ID

When using the auth_code_grant.rs example I get this OAuth error:
AADSTS50194: Application (my app) is not configured as a multi-tenant application. Usage of the /common endpoint is not supported for such applications created after '10/15/2018'. Use a tenant-specific endpoint or configure the application to be multi-tenant.

I had the same error when I was trying to use the Python o365 library, and fixed it by just setting the tenant ID like this:
credentials = (client_id, client_secret)

account = Account(credentials, tenant_id='....')

How do I do the same thing with graph-rs-sdk?

@Timmmm Hey Tim, sorry I am just now seeing this. Let me see if I can find an example that shows how to do this.

@Timmmm Because we don't use the tenant id in any other place besides the URL's for requesting access tokens and refresh tokens, you would actually need to format this in the url itself:
   let tenant_id = "TENANT_ID";
   let access_token_url = format!("https://login.microsoftonline.com/{}/oauth2/v2.0/token", tenant_id);

   let mut oauth = OAuth::new();
   oauth
       .client_id("CLIENT_ID")
       .client_secret("CLIENT_ID")
       .access_token_url(&access_token_url);

You will want to do this for the authorization url as well as the access and refresh token url's. For instance:
Replace the word common in each url below:
    oauth
        .authorize_url("https://login.microsoftonline.com/common/oauth2/v2.0/authorize")
        .access_token_url("https://login.microsoftonline.com/common/oauth2/v2.0/token")
        .refresh_token_url("https://login.microsoftonline.com/common/oauth2/v2.0/token");

  // Replace the above with:
     oauth
        .authorize_url("https://login.microsoftonline.com/TENANT_ID/oauth2/v2.0/authorize")
        .access_token_url("https://login.microsoftonline.com/TENANT_ID/oauth2/v2.0/token")
        .refresh_token_url("https://login.microsoftonline.com/TENANT_ID/oauth2/v2.0/token")

Now that you bring this issue up though, I think adding a way to just set the tenant id would be a great addition. I will create a ticket for it so that it can be done in future work.
Let me know if this helps. Thanks.

Ah that's a lot easier than I was expecting, thanks!


react-router-redux is deprecated

Hello.
Recently, I had seen that 'react-router-redux' is deprecated.
This is quote from their repository.
This project is no longer maintained. For your Redux <-> Router syncing needs, please see one of these libraries instead: connected-react-router
I think that I could replace react-router-redux by connected-react-router.
What do you think about?

Hey @gretzky I'd like to implement either a switch to reach-router or at least replace react-router-redux before we release 3.7. Any reasons to go one way or the other? Why the preference for reach-router? I've never used it.

2 reasons so far that I prefer reach:

Way easier to use / simple and expected API. No gotchas yet
It's geared toward accessibility

But I'm open to anything, esp. if it's easier to stick w/ react-router in the meantime

@gretzky Check out #2375. For now I implemented a switch to connected-react-router so we can at least close this issue and have maintained deps only. Down for a switch to reach-router later but it's probably a larger project...

Closing this as we're about to merge a migration to connected-react-router.


prometheus_client.exposition generate_latest throws error if registry contains an Info metric

Using prometheus_client version 0.7.1.
Here is the relevant part of the stack trace:
    logging.info('Metrics:\n' + indent(generate_latest(PUSH_METRICS_REGISTRY), '    '))
  File ".../lib/python3.7/site-packages/prometheus_client/exposition.py", line 121, in generate_latest
    output.append(sample_line(s))
  File ".../lib/python3.7/site-packages/prometheus_client/exposition.py", line 79, in sample_line
    for k, v in sorted(line.labels.items())]))
  File ".../lib/python3.7/site-packages/prometheus_client/exposition.py", line 79, in <listcomp>
    for k, v in sorted(line.labels.items())]))
AttributeError: ("'dict' object has no attribute 'replace'", Metric(my_info, The metadata about ... such as its version, config, and deployment state, info, , [Sample(name='my_info_info', labels={'foo': 'fubar', 'version': '1.2.3', 'deployment_state': 'OFF', 'config': {}}, value=1.0, timestamp=None, exemplar=None)]))

Can you share the code to create that specific registry?

just a no args constructor:
from prometheus_client.registry import CollectorRegistry
PUSH_METRICS_REGISTRY = CollectorRegistry()

Oh, maybe you mean the metric too?  Here is an example:
info_metric = Info('an_info', 'The description for the info metric',
                               labelnames=['one','two'],
                               registry=PUSH_METRICS_REGISTRY,
                               namespace='myapp', subsystem='microsvc',
                        )

That's working fine for me.

Sorry, I am not sure what is different, but trying it on a clean system now and it works fine for me too. Closing as not reproducible.

Thanks for getting back to us.


'IQR' in boxplot caption

Changed boxplot caption to fit multiple satellite datasets in image

Pull Request Test Coverage Report for Build 683168040

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

Totals

Change from base Build 653660757:
0.0%

Covered Lines:
749

Relevant Lines:
877

💛  - Coveralls


Added support for python 3.12

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.14%. Comparing base (665c2b1) to head (6f057a2).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master       #4   +/-   ##
=======================================
  Coverage   94.14%   94.14%
=======================================
  Files          12       12
  Lines         785      785
=======================================
  Hits          739      739
  Misses         46       46

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


Preserve blank lines in new file for long sections

Fixes #821
The problem is in line 105 of .../util/Strings.kt
if (originalLine.isBlank()) {continue} ...
Changing that fragment fixes the problem:
for (originalLine in this) {
    if (originalLine.isBlank()) {
        list.add("")
    }
    else {
        list.add(originalLine.substring(maxIndent))
    }
}

This, however, introduces the problem of blank lines at the start of the new file.
Changing the following line
val text = document.getText(TextRange(startIndex, endIndex)).trimEnd().removeIndents()
to
val text = document.getText(TextRange(startIndex, endIndex)).trimStart().trimEnd().removeIndents()
messes up the indentation of the first line, preventing removeIndents() from doing its job.
I solved this in Strings.kt:
var noContentYet = true
for (originalLine in this) {
    if (noContentYet && originalLine.isBlank()) {
        continue
    }

    if (originalLine.isBlank()) {
        list.add("")
    } else {
        noContentYet = false
        list.add(originalLine.substring(maxIndent))
    }
}

PS next time it would be nice if you could branch from the latest commit on master, I now had to merge master to avoid downloading old Gradle and IDEA versions.

I've been having trouble with this. I tried a pull request, but since my previous pull hasn't been accepted yet, it wanted to create a pull request for 16 commits, instead of these two... So I checked out a new branch from where I hadn't done anything yet (which was November last year).
I'll keep it in mind.

Ah okay, I didn't know how familiar you were with a VCS workflow in general. So what you normally do, you pull master from the main repo, then you branch to a topic branch (which you named master for your first PR), then push it to your fork and create a PR. Then after a while you pull master again (make sure to pull from the main repo, for example by letting it track master on the main repo) and make a new topic branch, push to your fork etc.
I see you have updated your other PR with commits from this PR. I don't know why, but this is in general unwanted because merging that PR would also essentially merge this PR, which not what you want. I have removed the commits.


Add an OhmConnect sensor

Description: This adds a OhmConnect sensor.

OhmConnect monitors real-time conditions on the electricity grid. When dirty and unsustainable power plants turn on, our users receive a notification to save energy. By saving energy at that time, California does not have to turn on additional power plants and California's energy authorities pay you for that.

Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#754
Example entry for configuration.yaml (if applicable):
sensor:
  - platform: ohmconnect
    id: AbCd1e

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

[ ] Documentation added/updated in home-assistant.io

If code communicates with devices, web services, or a:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

@balloob Fixed

💃 🐬


Sourcemap for<EMAIL_ADDRESS>points to missing source files

Describe the bug
I keep getting a list of Sourcemap for<EMAIL_ADDRESS>points to missing source files error messages when I create and run a new slidev project.
How do I address these error messages?
It seems they are similar to the following Issues posts from vite repo:

Sourcemap for "xxxx" points to missing source files · Issue #5562 · vitejs/vite
plugin-react 1.0.6 triggers sourcemap and deoptimisation error/warning messages in dev server · Issue #5438 · vitejs/vite

#Started a new Slidev slide deck with initiate command
npm init slidev

  ●■▲
  Slidev Creator  v0.27.9

✔ Project name: … Talks
✔ Package name: … talks
  Scaffolding project in Talks ...
  Done.

✔ Install and start it now? … yes
✔ Choose the agent › npm
npm WARN deprecated<EMAIL_ADDRESS>Potential XSS vulnerability patched in v6.0.0.

added 318 packages, and audited 319 packages in 23s

55 packages are looking for funding
  run `npm fund` for details

3 high severity vulnerabilities

To address all issues (including breaking changes), run:
  npm audit fix --force

Run `npm audit` for details.

> dev
> slidev --open

  ●■▲
  Slidev  v0.27.20

  theme   @slidev/theme-seriph
  entry   /Users/USERNAME/Documents/GitHub/Talks/slides.md

  slide show      > http://localhost:3030/
  presenter mode  > http://localhost:3030/presenter
  remote control  > pass --remote to enable

  shortcuts       > restart | open | edit

Sourcemap for<EMAIL_ADDRESS>points to missing source files
Sourcemap for<EMAIL_ADDRESS>points to missing source files
Sourcemap for<EMAIL_ADDRESS>points to missing source files
Sourcemap for<EMAIL_ADDRESS>points to missing source files
Sourcemap for<EMAIL_ADDRESS>points to missing source files
Sourcemap for<EMAIL_ADDRESS>points to missing source files
Sourcemap for<EMAIL_ADDRESS>points to missing source files
Sourcemap for<EMAIL_ADDRESS>points to missing source files
Sourcemap for<EMAIL_ADDRESS>points to missing source files
Sourcemap for<EMAIL_ADDRESS>points to missing source files

To Reproduce
Steps to reproduce the behavior:

Create a new Slidev project with npm init slidev

Desktop (please complete the following information):

OS: macOS Monterey version 12.2.1
Browser: Google Chrome Version 99.0.4844.51 (Official Build) (x86_64)
Slidev version: Slidev  v0.27.20

Thank you for your time 🙇♂

This has been fixed in version 0.28.8


Missing AudienceRestriction in response token

According to SAML profiles specs in <IP_ADDRESS> :
The assertion(s) containing a bearer subject confirmation MUST contain an
 including the service provider's unique identifier as an .
Currently reponse token is
<saml:Conditions NotBefore="2018-05-22T08:43:29.424Z" NotOnOrAfter="2018-05-22T08:53:29.424Z"/>
It should be :
<saml:Conditions NotBefore="2018-05-22T08:43:29.424Z" NotOnOrAfter="2018-05-22T08:53:29.424Z">                          <saml:AudienceRestriction> 				<saml:Audience>http://localhost:8088/myApp</saml:Audience> 			</saml:AudienceRestriction> </saml:Conditions>

Fixed


Is there a Composer 8.2 image coming at all?

Sail Version: 8.2
Laravel Version: 9.x
PHP Version: 8.2
Host operating system: macOS

Is there a Composer image for PHP 8.2 coming at any point? I noticed it’s missing from this page:
https://hub.docker.com/u/laravelsail
I use this approach to install Composer dependencies when installing projects on a new machine, or on a fresh clone: https://laravel.com/docs/9.x/sail#installing-composer-dependencies-for-existing-projects
But that section of the docs say to use a laravelsail/php81-composer, which doesn’t work for me as that image contains PHP 8.1 and the project I’m working on requires PHP ^8.2, so the composer install commands fails.

Heya, thanks for letting us know. We now released a 8.2 image.

@driesvints Awesome, thanks!


How to get all previous states?

Hi all,
I am new to ngrx store. I have question how do I get all of my previous states.
Also I am interested on implementing undo redo. Is there any way to get it out of the box.
I know Redux Undo package has this feature do we also have in ngrx store ? https://github.com/omnidan/redux-undo

You need to use the devtools if you want to retain all previous states. https://github.com/ngrx/store-devtools

Thanks Brandon, I was aware of this tool this tool works with chrome extension. But my requirement was to provide undo redo feature in my app. Is this store dev tools can be deployed in production also. Can you point me to some sample app ?


refactor(tree-wide): add/update base styles

Updates the template, associated documentation, and userstyles with existing ::selection selectors. Closes #765.

Apologies for all the pings! Just some cleanup stuff here. If you have a chance it would be great if you could verify that no regressions have snuck in in your userstyle(s).


ImportError: cannot import name 'summary_op_util'

Tensorflow version：
1.13.1-cpu-windows
Project git version:
8d93b2eb65d30a36a1b14ba861fd2a3c59c56d0f
Error log:
Traceback (most recent call last):
  File "C:/Users/gt/Desktop/tensor2tensor-master/tensor2tensor/bin/t2t_trainer.py", line 25, in <module>
    from tensor2tensor import models  # pylint: disable=unused-import
  File "C:\Users\gt\Desktop\tensor2tensor-master\tensor2tensor\models\__init__.py", line 58, in <module>
    from tensor2tensor.models.research import rl
  File "C:\Users\gt\Desktop\tensor2tensor-master\tensor2tensor\models\research\rl.py", line 31, in <module>
    from tensor2tensor.models.video import basic_deterministic_params
  File "C:\Users\gt\Desktop\tensor2tensor-master\tensor2tensor\models\video\basic_deterministic_params.py", line 22, in <module>
    from tensor2tensor.models.video import base
  File "C:\Users\gt\Desktop\tensor2tensor-master\tensor2tensor\models\video\base.py", line 27, in <module>
    from tensor2tensor.layers import common_video
  File "C:\Users\gt\Desktop\tensor2tensor-master\tensor2tensor\layers\common_video.py", line 27, in <module>
    from tensorflow.python.distribute import summary_op_util as distribute_summary_op_util  # pylint: disable=g-direct-tensorflow-import
ImportError: cannot import name 'summary_op_util'

I'm also seeing this error.

This is fixed in master.


keep platform warning out of bundle show?

bundle show --paths is a very useful command for outputting paths of all dependencies, to use as arguments for other commands.
For instance, I make a gemgrep command that will grep through ALL dependency sources, with a bash function like this:
gemgrep () { grep "$@" -r . $(bundle show --paths); }

This used to work fine. But currently in some projects, executing bundle show --paths first prints out this line:
The dependency tzinfo-data (>= 0) will be unused by any of the platforms Bundler is installing for. Bundler is installing for ruby but the dependency is only for x86-mingw32, x86-mswin32, x64-mingw32, java. To add those platforms to the bundle, run `bundle lock --add-platform x86-mingw32 x86-mswin32 x64-mingw32 java`.

Perhaps I should fix that. But it ought not to ruin my use of bundle show --paths as an arg if I don't. But it does, since it includes that line, of course grep complains that the args aren't what expects, in particular it sees -add-platform and tries to use it as an arg for grep, and complains grep: unrecognized option --add-platform'`.
Is there any way to get a bundle show --paths that won't output warnings like this too, it will only output the actual output?  (Or if you sent warnings to stderr and actual output to stdout, I could at least pipe the warnings to /dev/null -- but nope, both are to stdout).
If not, should there be?

We send warnings to stderr since bundler 2. Which bundler version are you using?

Aha, I am still on bundler 1.17.3, time to upgrade, sorry, thanks!

No problem, thanks for the feedback!


go/types,x/tools/go/packages: types.Implements() fails for interfaces with named types

Problem summary
It seems like the function types.Implements will fail if the Interface provided contains methods returning custom types (even if everything is defined in the exact same package).
Sample minimal code can be found here:

https://go.dev/play/p/I3UPHwUkMFb
Note: it is not runnable on go.dev/play (go executable is not directly available), needs to be copy-pasted.

Details
Specifically, given the below which compiles fine (note the assertion that the struct "something" implements both interfaces):
type MyInterface interface {
	ID() ID
	Description() string
}

type MySimplerInterface interface {
	Description() string
}

type something struct{}

var _ MyInterface = &something{}
var _ MySimplerInterface = &something{}
...

the check types.Implements(types.NewPointer(typesObj.Type()), interfaceSearched) will return true only for MySimplerInterface, where the types details have been loaded using go/packages
What version of Go are you using (go version)?
go1.20.5
What operating system and processor architecture are you using (go env)?
darwin / arm64

(attn @griesemer @findleyr)

This is working as expected.
Note that types.Implements is the same function that the compiler is using (it's an auto-generated copy of it), so it's unlikely that types.Implements fails when using it as a library function when the compiler accepts the equivalent source.
Here is your example, a bit further simplified so I could see what's going on (in general when sending a repro case, please minimize by removing all unnecessary code - it greatly helps us diagnosing problems).
The problem is that you are loading the same packages twice (interfacePks and searchedPks). User-defined types, in this case, the type ID gets a different "identity" in different loads. As a consequence, the type ID used in the interface method doesn't match the type ID in the struct method and then Implements fails. The problem goes away if you use say string in case of ID (not a user-defined type), or if you correct the code and load the packages only once. You can see in the simplified example (line 50, 51) that loading the packages only once solves the issue.
The type-identity properties should be better documented. I filed #61418.
Closing as working as intended.

Thank you for the quick reply. That is not ideal (e.g. the interface could be defined in a completely different place than the code I am trying to analyse against it, without a common root - sure, one can force the existence of the common root programmatically), but I understand it is the limitation of the current toolset.
Thank you once again for your help!


Can't find any way to play audio on grid fields

I'm trying to reproduce audio using grid. The idea is bring location of the audio and play it with a button or something. Is there any way to do it?
Thanks in advance.

You can write your own column formatter that creates a button in your grid cell.
It is up to you to provide a click handler for your button(s).

Not a bug report. Please ask questions at forums


Possibly broken by a recent Windows update?

My RPI 4 running raspindi has been working well for a few months now, until about a week ago.
Symptoms
raspindi source is detected, but video is just black. Other sources, like NDI's own screen capture, is still working.
What changed?
The only thing that changed was a recent windows update, although I'm not sure which one. It stopped working on multiple Windows PCs, while continuing to work on other Windows PCs and all Linux PCs (using the exact same RPI4).

The Windows PCs on which it does NOT work are all on update 10.0.19042.
The Windows PCs on which it DOES work are all on update 10.0.18363.
It's working on LinuxMint.

Whose fault?
At first, I though this was a problem with the obs-ndi plugin that I'm using. See https://github.com/Palakis/obs-ndi/issues/651 . But NDI Studio Monitor is showing the exact same symptoms.
Since nothing changed in raspindi, it's probably not the raspindi's fault.
However, since other NDI sources are still working, it might be possible to get this working again from raspindi's side?

Interestingly, there are a lot of recent reviews on the NDI HX Camera android app, reporting the same problem as above. This increases my belief that it might have to do with a Windows update. However, I can't find any official source from NDI or Windows talking about this problem.

Pretty sure that whatever update affected Windows 10 is actually baked into Windows 11. I have exactly this problem as described on a clean Windows 11 install with updates.

As I mentioned above, this issue seems to point to the Windows end.
NDI as a protocol is all handled by the (closed source) NDI library, so nothing has changed at that end.


Call to enhanced-resolve converts # in paths to \0#, which crashes eslint-plugin-import

Overview
When a path includes #, it seems that enhanced-resolve replaces it with \0# (https://github.com/webpack/enhanced-resolve#escaping). Unfortunately further up this causes a failure in eslint-plugin-import:
TypeError [ERR_INVALID_ARG_VALUE]: The argument 'path' must be a string or Uint8Array without null bytes. Received<EMAIL_ADDRESS>Occurred while linting c:\Development\SharpLab\source\#external\mirrorsharp-codemirror-6-preview\WebAssets\src\codemirror\create.ts:1
Rule: "import/namespace"
    at Object.statSync (node:fs:1581:10)
    at Function.ExportMap.for (c:\Development\SharpLab\source\#external\mirrorsharp-codemirror-6-preview\WebAssets\node_modules\eslint-plugin-import\lib\ExportMap.js:792:67)
    at Function.ExportMap.get (c:\Development\SharpLab\source\#external\mirrorsharp-codemirror-6-preview\WebAssets\node_modules\eslint-plugin-import\lib\ExportMap.js:791:463)
    at processBodyStatement (c:\Development\SharpLab\source\#external\mirrorsharp-codemirror-6-preview\WebAssets\node_modules\eslint-plugin-import\lib\rules\namespace.js:11:40)
    at c:\Development\SharpLab\source\#external\mirrorsharp-codemirror-6-preview\WebAssets\node_modules\eslint-plugin-import\lib\rules\namespace.js:82:47
    at Array.forEach (<anonymous>)
    at Program (c:\Development\SharpLab\source\#external\mirrorsharp-codemirror-6-preview\WebAssets\node_modules\eslint-plugin-import\lib\rules\namespace.js:82:18)
    at ruleErrorHandler (c:\Development\SharpLab\source\#external\mirrorsharp-codemirror-6-preview\WebAssets\node_modules\eslint\lib\linter\linter.js:1115:28)
    at c:\Development\SharpLab\source\#external\mirrorsharp-codemirror-6-preview\WebAssets\node_modules\eslint\lib\linter\safe-emitter.js:45:58
    at Array.forEach (<anonymous>)
task ts:eslint failed:
Error: Command failed with exit code 2: eslint ./src --max-warnings 0 --ext .js,.jsx,.ts,.tsx
    at makeError (c:\Development\SharpLab\source\#external\mirrorsharp-codemirror-6-preview\WebAssets\node_modules\oldowan\node_modules\execa\lib\error.js:59:11)
    at handlePromise (c:\Development\SharpLab\source\#external\mirrorsharp-codemirror-6-preview\WebAssets\node_modules\oldowan\node_modules\execa\index.js:114:26)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async exec (c:\Development\SharpLab\source\#external\mirrorsharp-codemirror-6-preview\WebAssets\node_modules\oldowan\dist\index.js:53:5)
    at async task.run (c:\Development\SharpLab\source\#external\mirrorsharp-codemirror-6-preview\WebAssets\node_modules\oldowan\dist\index.js:74:13)
    at async file:///c:/Development/SharpLab/source/%23external/mirrorsharp-codemirror-6-preview/WebAssets/build.ts:74:5
    at async Object.task.run (c:\Development\SharpLab\source\#external\mirrorsharp-codemirror-6-preview\WebAssets\node_modules\oldowan\dist\index.js:74:13)
    at async build (c:\Development\SharpLab\source\#external\mirrorsharp-codemirror-6-preview\WebAssets\node_modules\oldowan\dist\index.js:107:9) {
  shortMessage: 'Command failed with exit code 2: eslint ./src --max-warnings 0 --ext .js,.jsx,.ts,.tsx',
  command: 'eslint ./src --max-warnings 0 --ext .js,.jsx,.ts,.tsx',
  exitCode: 2,
  signal: undefined,
  signalDescription: undefined,
  stdout: undefined,
  stderr: undefined,
  failed: true,
  timedOut: false,
  isCanceled: false,
  killed: false
}

Why this library?
I could report this to enhanced-resolve, but their \0 escape seems to be well documented and by design. Happy to report there if you think that's the best option.
I could report this to eslint-plugin-import, however the problem depends on details resolver implementation, and eslint-plugin-import seems resolver-agnostic.
Proposed solution
Path returned by enhanced-resolve can be post-processed to replace \0# with #, which is the correct representation of the filesystem path.

Do you want to raise a PR for it?


Update rules engine so it knows how to handle triggers with parameters

Just an issue for the missing feature described here in https://github.com/StackStorm/st2/pull/3398.
See:

https://github.com/StackStorm/st2/pull/3398#issuecomment-300087815
https://github.com/StackStorm/st2/pull/3398#issuecomment-300105396

Notably the second comment for the proposed solution.

I believe this has been resolved by #3398.
@lakshmi-kannan can you please confirm?


Storybook Readme Appearing in Every Addon Panel with Webpack 4 & Storybook Alpha 4.20 upgrade

The Storybook Readme addon is appearing in every addon panel after upgrading to Webpack 4 and updating Storybook to the latest Alpha release per their recommendations here.
Here is a clean repository with code reproducing the error.
And a small gif and picture showing the error:

This is preventing us from using the storybook readme addon after upgrading to Webpack 4 and Storybook Alpha (which is needed with the Webpack 4 upgrade).
For clarity, we tried storybook-readme with both the @3.3.0 and @4.0.0-beta1 version. Neither one worked as expected.

Going to investigate the problem today.

will fix in next releast

Sorry for being impatient but is there an ETA for the next release containing this fix?
Cheers!

Going to release during next few days

released in 4.0.0


How to add private key to my .env file?

Hi, i am trying to add a private cert on my .env file, but i can't make it works.
I define my .env var like this:
PRIVATE_KEY="-----BEGIN RSA PRIVATE KEY-----
\nMIIEpQzXCCKCAQEA0ClVcH8RC1u+KpCPUnzYSIcmyXI87REsBkQzaA1QJeW4/B7g
....
-----END RSA PRIVATE KEY-----"

I am making a log to see what happen and i see a backslash on the end of each line:
[2017-06-24 02:53:42] local.DEBUG: asd {"private_key":"-----BEGIN RSA PRIVATE KEY-----\
MIIEpQzXCCKCAQEA0ClVcH8RC1u+KpCPUnzYSIcmyXI87REsBkQzaA1QJeW4/B7g\
...
-----END RSA PRIVATE KEY-----"

I don't know what i am doing wrong. Any suggestion i will be grateful.
Regards

Ok,i found my problem, was not about this package.
My problem was my code, here is what i was executing:
str_replace('|', '\n', $cert)]);

My mistake was the newline char: it was on single quotes. I had to put it on double quotes, like this:
str_replace('|', "\n", $cert)]);

@holic-cl Hi, had already encountered this problem and used base64 to bypass it and encode the multiline key and retain its integrity.

Sounds like another solution, thanks for the idea.
Do you know why it is happening this?, i guess something about how is encoding the text.

I also encode complex strings in base64. But this is a bug I think that needs to be modified.

You could store the file in another location, and store the location itself in the .env file.

You can probably just escape the newline sequences \n to read \\n and then remove the actual newlines from the text, turning the entry into a single line. For example, replace
PRIVATE_KEY="-----BEGIN RSA PRIVATE KEY-----
\nMIIEpQzXCCKCAQEA0ClVcH8RC1u+KpCPUnzYSIcmyXI87REsBkQzaA1QJeW4/B7g
....
-----END RSA PRIVATE KEY-----"

with
PRIVATE_KEY="-----BEGIN RSA PRIVATE KEY-----\\nMIIEpQzXCCKCAQEA0ClVcH8RC1u+KpCPUnzYSIcmyXI87REsBkQzaA1QJeW4/B7g....-----END RSA PRIVATE KEY-----"

Hi @sridesmet @ocram @ikee87, thanks for suggestions. I read good ideas here.

https://github.com/vlucas/phpdotenv/pull/271

Attention: if you have in your .env:
X="testtesttesttesttesttesttest \n testtesttesttest"
the end result is an empty string. (since 2.5, it was fine in 2.4). I have found that the regex in sanitiseVariableValue is the issue.
To make your .env compatible with 2.5 you should use, as someone else mentioned here, double backslashes, i.e.\n.

Multiline support will come in v3.0 of this package, and is being developed at https://github.com/vlucas/phpdotenv/pull/271.

@GrahamCampbell is this done yet ?

Yes.


Port to Godot4

I think we should port this to Godot 4 and make this an Official Demo Project.

Not this one.
I actually have a different version lying around based on Brihernandez' newer example https://github.com/brihernandez/FreelancerFlightExample
More accurate to Freelancer and with a more  dynamic camera.
Still based on Godot 3 though :D
I'll port it to Godot 4 and upload to Github once I find the time.

I can port this one to godot 4

Well go for it! I'm just saying myself, I'd rather get the newer one up to snuff and port that.
I still see value in this repo as the mouse joystick approach is a lot simpler. If your goal isn't getting something close to Freelancer it's still a great way to implement third person spaceship controls.

I am developing my own PC games and my control scheme is okay for a
FreeSpace clone but needs work for any tactical or strategy gameplay.
On Sat, Apr 29, 2023 at 2:34 AM Yannick @.***> wrote:

Well go for it! I'm just saying myself, I'd rather get the newer one up to
snuff and port that.
I still see value in this repo as the mouse joystick approach is a lot
simpler. If your goal isn't getting something close to Freelancer it's
still a great way to implement third person spaceship controls.
—
Reply to this email directly, view it on GitHub
https://github.com/Sockstah/Space-Shooter-Example/issues/1#issuecomment-1528688563,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH4ZR5YZXIFV3QCTM43CTF3XDSY5VANCNFSM6AAAAAAVPIEUY4
.
You are receiving this because you authored the thread.Message ID:
@.***>