[Update Request]: RustDesk.RustDesk 1.2.1

What type of update are you requesting?
A new version of an existing package
Current Package Identifier
RustDesk.RustDesk
Package Version
1.2.1
Please describe the changes you would like to see
There is a new release (1.2.1) available.
https://github.com/rustdesk/rustdesk/releases/download/1.2.1/rustdesk-1.2.1-x86_64.exe

Hi, please Update, the latest Version is 1.2.3-1 which includes a critical security fix:
https://github.com/rustdesk/rustdesk/releases/tag/1.2.3-1
https://github.com/rustdesk/rustdesk/discussions/6444
https://nvd.nist.gov/vuln/detail/CVE-2024-25140

Recommend closing this issue; a newer version is already present in the WinGet community repository.


GET response doesn't show unidirectional ManyToOnes

I've finally got ApigilityDoctrine working but I've noticed that when I make a get request OneToMany column looks like this:
"app": {
    "__initializer__": {},
    "__cloner__": {},
    "__isInitialized__": false
},

Any idea why?

@tad3j Kind of outdated, but I had the same issue today and I wanted to share my experience fixing it.
I could fix it only by creating a new service Doctrine-connected for the odd entity ("app" entity in your case). After that, new requests worked like a charm and I got the entity back as part of the _embedded object.

@santiph thank you , your message helped !


Upgrade Material Icons to 3.0.1

3.0.1 has been relased on Sep 2, https://github.com/google/material-design-icons/releases

It's already using 3.0.1?

I use the local_shipping name, and it shows as missing Icon.
Just found out, it should be the 'local-shipping' name, different from the
doucmentation
Thanks :)
On Mon, Nov 21, 2016 at 5:46 AM, Joel Arvidsson<EMAIL_ADDRESS>wrote:

It's already using 3.0.1?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/oblador/react-native-vector-icons/issues/348#issuecomment-261808292,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXCyEQW3WkxCi8bI5MXY19Ou4n2kodYks5rAL-zgaJpZM4K3a1c
.

--
JavaScript, Go, Linux and Open Source

g http://GutenYe.comuten.me
github.com/g http://Github.com/GutenYeutenye


gktcord4: add nativeBuildInput wrapGappsHook

Description of changes
In #200179 we noticed some application crashes triggered during file uploads: "Settings schema 'org.gtk.gtk4.Settings.FileChooser' is not installed". This appears to be because we failed to wrapGAppsHook, this has been corrected.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1500


Add option to restrict redirects to parts of SSO app URLs

Coverage: 87.526% (+0.1%) from 87.414% when pulling 921303cc4695873d086ec22610079c916d278e59 on feature/restrict-app-redirects into a435c77b75f2653f5d920108ff8e6e17c1ddf391 on main.


feat: update Sphinx

Had to trigger it manually since we had asked dependabot to stop updating Sphinx.
7.2.0 is failing though...

Codecov Report

Merging #115 (1c5c2bb) into main (d92df55) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #115   +/-   ##
=======================================
  Coverage   94.20%   94.20%
=======================================
  Files           3        3
  Lines          69       69
=======================================
  Hits           65       65
  Misses          4        4

Looks like it works @AA-Turner! Thanks for your support!


vitesse-lite dev startup problem      [unocss] entry module not found, have you add import 'uno.css' in your main entry?

[unocss] entry module not found, have you add import 'uno.css' in your main entry?

We temporarily close this due to the lack of enough information. Please provide a minimal reproduction to reopen the issue. Thanks.

I have this problem occasionally. Fixed it by restarting server.
Not knowing how to reproduce yet, might look into it later.

@antfu Maybe... resolvedId will not be fired when the dev link is not opened in browser within 10 seconds, and resolved will be false always.
After 10s, the setTimeout's callback in the setWarnTimer method will be fired.

I had same problem

I'm getting the same issue was this really not solved?


Reduce Dependencies

We are installing git and ssh into the nib image but I'm not sure they are currently required. Can you think of any reason we need them @blankenshipz?

I don't think we need git anymore, I honestly don't remember why we needed ssh. I imagine it was a dependency of having git


Fixes (another) IRGen compiler crash caused by the new large types ABI

radar rdar://problem/34277023
Fixes a corner-case when getting a function ref as an argument and/or returning a function ref. In some cases wherein the signature of the referenced function was modified, we did not propagate that change correctly.

@swift-ci Please smoke test

Linux build failure has nothing to do with my PR:
Asynchronous wait failed - Exceeded timeout of 1.0 seconds, with unfulfilled expectations: Repository lookup expectation, Repository lookup expectation

@swift-ci Please smoke test linux

@swift-ci Please smoke test

@swift-ci Please smoke test


Can't access containers after upgrade due to "Cannot read property 'DeletedTime' of undefined"

After upgrading to 1.21.3, clicking on some containers displays "Cannot read property 'DeletedTime' of undefined" and won't show any folders in the container.

@MisterDoug99 do you know if these containers are in an ADLS Gen2 account? If they are then you would see (ADLS Gen2) next to the account node:

Or if you have attached the container you would see that next to the name of the container itself.

Yes, they are in an ADLS Gen2 account.

The Properties field is missing in the list response. I will relax the code that access that field optionally.

We have released 1.22.0 which includes the fix to this bug.

This repro's on 1.25.1

@pwehland We were told that the service issue was fixed in a previous major release except for pre-prod storage accounts. Are you using a pre-prod storage account? We are heavily relying on the service to return us the properties of the blobs/directories otherwise quite a few features won't work in Storage Explorer. If you can capture a Fiddler trace demonstrating the fact that the service is not returning the properties correctly, we can create a support ticket to the service team to investigate.
Here is instructions on how to get a fiddler trace https://docs.microsoft.com/en-us/azure/storage/common/storage-explorer-troubleshooting?tabs=Windows%2C2204#network-logs


Indented formatting?

I've fallen in love with the indented rule formatting output by the live sass compiler for VS Code. Also, CodeKit has a similar option.
"expanded, compact, compressed, nested"
Any chance this option can be added to your roadmap?

Currently in the preferences we have the CSS Style which allows you to choose Expanded or Compressed. I should be able to add Compact and Nested.

Sorry @luxuryluke because under the hood we are using the dart sass version only "Expanded" and "Compressed" are supported. Also see https://github.com/sass/dart-sass/issues/442


Why is only dino_vits8 supported?

In the examples, if you change the model_type to anything other than dino_vits8 the code crashes because of an assert in ViTExtractor.extract_saliency_maps. What needs to change to properly support other model types?

+1
This is weird, hope the authors can give us an answer.
Thks

@juancamilog I read the code again and find out that this may be because the authors have only tried the head_idxs = [0, 2, 4, 5] for vits. Since vit-b/l/g have different number of heads, the suitable head_idxs also should be found.
But that's not a simple question...

Hi!
The saliency maps used in the co-segmentation, part co-segmentation and correspondences examples is acquired by aggregating heads 0, 2, 4, 5 of Dino_vit8s. We removed heads 1 and 3 as they empirically attended bg areas. It is also possible to change the code to use different Dino_vits aggregating all heads, but would require adjusting some of the hyper parameters in each application.
LMK if you have further questions 🙏

Thks for your reply. If using vit models with more heads, the 0,2,4,5 idx can keep the same? I'm not sure the larger model's head will get the similar attention result.
By the way, I have tried using dinov2's weight, but find out the result is even worse. It seems that the patch_size will significantly influence the foreground seg results. The smaller the patch_size, the better the result can be get. Do you have any ideas about using dinov2's feature? Because I only found DINO_vit14 pretrained model in their repo.

@RickyYXY  did you get it working for vitb?


Instagram_video_downloader

Closes issue
#18 (instagram video downloader : intermediate)
This is a draft pr, as requested by @sonnynomnom .

[ ] My project tutorial topic has already been aproved by the Codédex team
[ ] My project tutorial file renders properly in mdx
[ ] The content written is my own, and not plagiarized from other sources
[ ] All code written is commented or documented thoroughly
[ ] I have already completed a self-review of the project tutorial
[ ] I will comply with changes from the Codédex team about my project tutorial

@sonnynomnom please have a look.

@Bobliuuu
Changes are done.
But I just used another approach to do this task, this is one is faster and efficient should update the file with new code?

@Abubakrce19 You can start on the project tutorial now!
If you want to use a different code for your tutorial, please change the code on your branch so we can review it!

Hi @Abubakrce19, that project template is just a template for you to create your project tutorial!
You need to edit it to write your tutorial, and explain your code using the template provided.
Let me know if you have more questions 🚀

@Bobliuuu
Thank you for responding on the week days❤ too.
I am currently working on this project advancement by creating a gui for this.
And adding more options like downloading profile picture of any account.
I will probably create a new pr for this and close this one.
In the new code I am explaining in detail by comments.
Also the project tutorial will e ready soon.
How can i get the header image template which every project tutorial had?
I am always open for your valuable suggestions.

Hi @Abubakrce19,
Sounds good! We will create the header image template for you, so don't worry about that!


Need tooling support for PR #15435

Description:
HTTP inbound authentication modules are refactored with #15435. With that change org.ballerinalang.langserver.workspace.WorkspaceTest is marked as broken due to a dynamic failure and need to be fixed properly.

Fixed with https://github.com/ballerina-platform/ballerina-lang/pull/15441 and https://github.com/ballerina-platform/ballerina-lang/pull/15332.


Fix dataframe deprecation warning

Corresponds to a message on slack,
task = load_reduced_ames();
┌ Warning: getindex(df::DataFrame, col_ind::ColumnIndex) is deprecated, use df[!, col_ind] instead.
│   caller = load_reduced_ames() at datasets.jl:18
└ @ MLJBase ~/.julia/packages/MLJBase/vdhww/src/datasets.jl:18

This is due to the update https://discourse.julialang.org/t/release-announcements-for-dataframes-jl/18258/38
PS: this only makes sense along with a compat requirement to have DataFrames be >=0.19.0 otherwise it will error; so this should be merged when we're happy to use the latest DataFrames (currently clashes with requirements from us to have Categorical Arrays to be of a specific version)

Thanks for that.
I think a better solution is to use MLJBase.selectcols, which does column selection through Tables.jl interface. That way we really don't care what version of DataFrames is being used, assuming DataFrames updates their Tables.jl implementation, which I imagine they would. In any case, this shifts responsibility for the warning to where it belongs.
As I am refactoring Base anyhow, I will do this. Thanks.


Newlines are lost when proceeded by a closing template tag

Describe the bug
If a template tag is the last part of a line, then the adjacent newline character is lost.
To Reproduce
Steps to reproduce the behavior:
If a template tag is the last part of a line, then the adjacent newline character is lost.
Steps to reproduce the behavior:

create a template with a line that ends in a closing tag:line before data
<%= it.data %>
line after data

render template with data

Expected behavior
Newline is preserved
Screenshots

Package & Environment Details
Node 20.11.1
Additional context
https://replit.com/@kevinkace/eta-tests

This is expected behavior! If you want to change this, you can change autoTrim in the config from [false, 'nl'] (default) to [false, false].


Consolidate common logic in Field class

Things done:

Basic tidying up of all ResidentField subclasses
Consolidate common logic into Field class
Moved old ResidentFields class items into respective ResidentField subclasses

Benefits:

Easier to add in new fields
Reduces need for testing (Free code coverage)

To consider:

Create VenueField class which extends Field to house Venue data

Codecov Report

Merging #118 (f6d315a) into master (1812dc1) will increase coverage by 0.47%.
The diff coverage is 74.41%.

@@             Coverage Diff              @@
##             master     #118      +/-   ##
============================================
+ Coverage     71.79%   72.27%   +0.47%
+ Complexity      519      514       -5
============================================
  Files            78       77       -1
  Lines          1521     1486      -35
  Branches        159      159
============================================
- Hits           1092     1074      -18
+ Misses          384      367      -17
  Partials         45       45

Impacted Files
Coverage Δ

...4hdb/logic/commands/modelcommands/EditCommand.java
94.59% <ø> (ø)

...ain/java/seedu/rc4hdb/logic/parser/ParserUtil.java
98.03% <ø> (ø)

.../logic/parser/commandparsers/AddCommandParser.java
100.00% <ø> (ø)

...logic/parser/commandparsers/EditCommandParser.java
93.93% <ø> (ø)

...gic/parser/commandparsers/FilterCommandParser.java
93.10% <ø> (ø)

...ain/java/seedu/rc4hdb/model/resident/Resident.java
98.24% <ø> (ø)

...eedu/rc4hdb/model/resident/ResidentDescriptor.java
100.00% <ø> (ø)

...n/java/seedu/rc4hdb/model/util/SampleDataUtil.java
27.27% <ø> (ø)

...c/main/java/seedu/rc4hdb/ui/ResidentTableView.java
0.00% <0.00%> (ø)

...java/seedu/rc4hdb/model/resident/fields/House.java
71.42% <66.66%> (+24.36%)
:arrow_up:

... and 20 more

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

Reopen branch to remove excess commits.


GPS location not available in Open Camera

I've just noticed that Open Camera is not tagging my photos with GPS location, despite the settings saying it should, and the Open Camera app having permissions for location, and location services being activated. Location is working for Google Maps, so why not Open Camera?
Already updated to V07 (Astro Slide).

Possibly this is related to https://issuetracker.google.com/issues/162643544?pli=1#comment6
Since the Astro's built-in camera doesn't appear to support adding GPS location data, it looks like we're SOL.


Documentation: Kind multi-node install

Signed-off-by: Afshin Paydar<EMAIL_ADDRESS>Description
Documentation Update
Install Calico on Kind multi-node cluster

@afshinpaydar-binary Thanks for the contribution! We've added this PR to our agenda for an upcoming docs team meeting. We'll be in touch after we've had a chance to consider it.

@ctauchen How did the docs team meeting go?
FWIW, I can confirm that the contents of this doc are good (I've used kind to set up clusters like this many times)
I think the only questions are:

do you want this added to the docs at all? (I think its useful)
are there any style changes needed?

@lwr20 I completely lost track of this one -- apologies @afshin-deriv.
As for whether it's useful and belongs -- I'll defer to you and Casey.
It's well organized and written. We'll follow up with whatever style and formatting changes we need later.

Casey's merged this already, so I assume it must have his blessing :)


Error: The PEM format key (unnamed) is encrypted (password-protected), and no passphrase was provided in options.

Hello! I'm attempting to use this github action to connect to a Kubernetes cluster. I've setup all of the required environment variables, and provided the cluster ID. However, I'm getting a failure on the "PEM format key". This cluster has a public endpoint as well.
Below is the output in github of the workflow execution:
Note: ignore the fact that it's nested under uhg-actions and not oracle-actions. that's just how my company approves github actions, wrapping them in our own.
1##[debug]Evaluating condition for step: 'Configure kubectl'
2##[debug]Evaluating: success()
3##[debug]Evaluating success:
4##[debug]=> true
5##[debug]Result: true
6##[debug]Starting: Configure kubectl
7##[debug]Loading inputs
8##[debug]Evaluating: secrets.OKE_CLUSTER_OCID
9##[debug]Evaluating Index:
10##[debug]..Evaluating secrets:
11##[debug]..=> Object
12##[debug]..Evaluating String:
13##[debug]..=> 'OKE_CLUSTER_OCID'
14##[debug]=> '***'
15##[debug]Result: '***'
16##[debug]Loading env
17
Run<EMAIL_ADDRESS>18  with:
19    cluster: ***
20  env:
21    DOCKER_IMAGE_TAG: SNAPSHOT-0.0.4
22    OCI_CLI_USER: ***
23    OCI_CLI_TENANCY: ***
24    OCI_CLI_FINGERPRINT: ***
25    OCI_CLI_KEY_CONTENT: ***
26    OCI_CLI_REGION: ***
27::group::Installing Oracle Cloud Infrastructure CLI
28
Installing Oracle Cloud Infrastructure CLI
29  /usr/bin/python -m pip install oci-cli
30  Collecting oci-cli
31    Downloading oci_cli-3.18.1-py3-none-any.whl (32.6 MB)
32  Collecting oci==2.85.0
33    Downloading oci-2.85.0-py2.py3-none-any.whl (17.7 MB)
34  Collecting terminaltables==3.1.0
35    Downloading terminaltables-3.1.0.tar.gz (12 kB)
36  Collecting pytz>=2016.10
37    Downloading pytz-2022.4-py2.py3-none-any.whl (500 kB)
38  Collecting click==7.1.2
39    Downloading click-7.1.2-py2.py3-none-any.whl (82 kB)
40  Collecting six>=1.15.0
41    Downloading six-1.16.0-py2.py3-none-any.whl (11 kB)
42  Collecting cryptography<=37.0.2,>=3.2.1
43    Downloading cryptography-37.0.2-cp36-abi3-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (4.1 MB)
44  Collecting arrow>=1.0.0
45    Downloading arrow-1.2.3-py3-none-any.whl (66 kB)
46  Collecting PyYAML<6,>=5.4
47    Downloading PyYAML-5.4.1-cp38-cp38-manylinux1_x86_64.whl (662 kB)
48  Requirement already satisfied: certifi in /usr/lib/python3/dist-packages (from oci-cli) (2019.11.28)
49  Collecting jmespath==0.10.0
50    Downloading jmespath-0.10.0-py2.py3-none-any.whl (24 kB)
51  Collecting prompt-toolkit==3.0.29
52    Downloading prompt_toolkit-3.0.29-py3-none-any.whl (381 kB)
53  Collecting python-dateutil<3.0.0,>=2.5.3
54    Downloading python_dateutil-2.8.2-py2.py3-none-any.whl (247 kB)
55  Requirement already satisfied: pyOpenSSL<=22.0.0,>=17.5.0 in /usr/lib/python3/dist-packages (from oci-cli) (19.0.0)
56  Collecting circuitbreaker<2.0.0,>=1.3.1
57    Downloading circuitbreaker-1.4.0.tar.gz (9.7 kB)
58  Collecting cffi>=1.12
59    Downloading cffi-1.15.1-cp38-cp38-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (442 kB)
60  Collecting wcwidth
61    Downloading wcwidth-0.2.5-py2.py3-none-any.whl (30 kB)
62  Collecting pycparser
63    Downloading pycparser-2.21-py2.py3-none-any.whl (118 kB)
64  Building wheels for collected packages: terminaltables, circuitbreaker
65    Building wheel for terminaltables (setup.py): started
66    Building wheel for terminaltables (setup.py): finished with status 'done'
67    Created wheel for terminaltables: filename=terminaltables-3.1.0-py3-none-any.whl size=15354 sha256=f5f7dc4c23f403c340898473bba50c6345ca795a1e1b5663991a7727da72afeb
68    Stored in directory: /home/runner/.cache/pip/wheels/08/8f/5f/253d0105a55bd84ee61ef0d37dbf70421e61e0cd70cef7c5e1
69    Building wheel for circuitbreaker (setup.py): started
70    Building wheel for circuitbreaker (setup.py): finished with status 'done'
71    Created wheel for circuitbreaker: filename=circuitbreaker-1.4.0-py3-none-any.whl size=7503 sha256=d25c5d8b63c1547880d21c8de8da7cf443194e62968c93d3a5087b9adb502dbb
72    Stored in directory: /home/runner/.cache/pip/wheels/52/06/a4/3f1cdbebf72c6da0c6e0074d5f4f54bebfa6c479ca6cb56e9d
73  Successfully built terminaltables circuitbreaker
74  ERROR: launchpadlib 1.10.13 requires testresources, which is not installed.
75  Installing collected packages: pytz, pycparser, cffi, cryptography, circuitbreaker, six, python-dateutil, oci, terminaltables, click, arrow, PyYAML, jmespath, wcwidth, prompt-toolkit, oci-cli
76  Successfully installed PyYAML-5.4.1 arrow-1.2.3 cffi-1.15.1 circuitbreaker-1.4.0 click-7.1.2 cryptography-37.0.2 jmespath-0.10.0 oci-2.85.0 oci-cli-3.18.1 prompt-toolkit-3.0.29 pycparser-2.21 python-dateutil-2.8.2 pytz-2022.4 six-1.16.0 terminaltables-3.1.0 wcwidth-0.2.5
77  ::endgroup::
78Error: The PEM format key (unnamed) is encrypted (password-protected), and no passphrase was provided in `options`
79##[debug]Node Action run completed with exit code 1
80##[debug]Finishing: Configure kubectl

(Oracle restricts GitHub Actions too, so I understand the issue well).
On your actual issue: unfortunately the GitHub Actions don't support encrypted private keys. You will either have to remove the pass phrase from your key, or create a new key or a new service account with its own key.
Alternatively, you could spin up an OCI instance to run a self-hosted GitHub Runner that uses instance principal authentication instead of keys.

Okay, that makes sense. Do you know what private key in particular? Would
it be for the OCI CLI or kubectl CLI? I'm having trouble finding a .pem key.
On Fri, Oct 7, 2022 at 9:09 PM Avi Miller @.***> wrote:

(Oracle restricts GitHub Actions too, so I understand the issue well).
On your actual issue: unfortunately the GitHub Actions don't support
encrypted private keys. You will either have to remove the pass phrase from
your key, or create a new key or a new service account with its own key.
Alternatively, you could spin up an OCI instance to run a self-hosted
GitHub Runner that uses instance principal authentication instead of keys.
—
Reply to this email directly, view it on GitHub
https://github.com/oracle-actions/configure-kubectl-oke/issues/16#issuecomment-1272185933,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZC2HE7OJKXOSPORWL3Z7FLWCDCS5ANCNFSM6AAAAAAQ7ZXROM
.
You are receiving this because you authored the thread.Message ID:
@.***>

It's the OCI API/CLI key stored in the OCI_CLI_KEY_CONTENT secret.

Aha, of course. Thanks!
On Fri, Oct 7, 2022 at 11:27 PM Avi Miller @.***> wrote:

It's the OCI API/CLI key stored in the OCI_CLI_KEY_CONTENT secret.
—
Reply to this email directly, view it on GitHub
https://github.com/oracle-actions/configure-kubectl-oke/issues/16#issuecomment-1272212850,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZC2HE53PI4XFEHESRCGALLWCDSYLANCNFSM6AAAAAAQ7ZXROM
.
You are receiving this because you authored the thread.Message ID:
@.***>

You're welcome.

Closing as @Djelibeybi proposed the solution above.


Secure the API to use some form of client validation

As the ZVGQ Client, I can make authenticated calls to the api with basic security, so that only specific clients can make calls to the api.

Configure the Ember RESTAdapter to send a custom header: REFERENCE - http://emberjs.com/api/data/classes/DS.RESTAdapter.html

Committed: a9fe93dbe323820b14944676785e66ece803a641
API router filters all request by hostname, which is configurable. This should act as a stop gap until other clients are needed.

All api requests are filtered based on referrer in request.
Commit: b892c5da0e56bd4d5f05a4ed0d7d7ad20d08d4ac


add  Registration Update on Notification mode (out of specification)

It was triggered by #852.

I can confirm this works great in our scenario!

Thx @sdrsdr. This is now integrated in master.


[forwardport] use master branch of metadata only for release server versions

https://github.com/rancher/rancher/pull/23606

The original issue https://github.com/rancher/rancher/issues/23485

wait for the coming preview release to validate the fix, so we don't have to build customized images


Turn off verbosity of evaluation set metrics

Hi,
I am wonder if it's possible to turn off the print statements of the evaluation metrics? These only seem to get printed when a train and test set is provided as evaluation sets?
i.e.
[0]     test-auc:0.898594       train-auc:0.917665
[1]     test-auc:0.900673       train-auc:0.926697
[2]     test-auc:0.904338       train-auc:0.934706
[3]     test-auc:0.90517        train-auc:0.938172
[4]     test-auc:0.906147       train-auc:0.942411

I have tried using the verbose(false) method in the BoosterParametersBuilder, and tried setting a custom_evaluation_fn in TrainingParametersBuilder, but I still get the evaluation metrics printed to the console.
Thanks,
Justin

Wait, never mind. I think I misunderstood the purpose of the evaluation_sets method, which is to evaluate and print the metric.


Add license information

Hello!
I found this project from https://hackaday.com/2023/03/08/supercon-2022-alec-vercruysse-can-see-through-murky-water/
Could you please add information about the license of this repository — is it allowed to copy, change and use the code and design files in any purpose?
It would be great to see any popular open source license.

Hello, thank you for raising this issue! We've added an MIT license.

The MIT license — great! Thank you!


Add Explore sample app

Adding a sample app that provides similar maps, places, and routes functionalities to the Explore app in the Amazon Location Service Console.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hi @imraymondlee thanks for the new sample, looking forward to reviewing this early next week (likely on Monday 2nd).
Question: would you be open on including an amplify folder to create & deploy all the required resources with amplify init & amplify deploy? This would be in addition to the manual instructions that you have already included, it would just provide a quick alternative for those users who are into Amplify CLI.
If you like the idea you could reuse the amplify directory from this other sample. If instead this is part of a different content strategy and the instructions must stay as they are disregard the question.

That's a good suggestion, I will need to discuss with the team about that.

Spoke with the team, and we have decided that we will not be adding the Amplify CLI support at the moment. I have also received some additional feedback and pushed a commit for those changes.

Overall the demo looks good.
I found the Place Index & Routing features a bit disconnected though. For instance if I search a place using the geocoder component, and then click "routes" I would kind of expect the marker I just placed to be used in the route. Instead what happens is that the black marker stays there, and I have to select again start & destination points for my route:

If the above is too much work, at least make the black mark disappear & reset the geocoder state so that you don't have two completely separate things on the map I'm looking at.


Lens doen't display HPA and PDB configuration

Steps to reproduce the behavior:

Go to the Config menu
Click on HPA / Pod Disruption Budget
See errors:

On the clusters, we have HPA with autoscaling/v2 API version and for PDB apiVersion: policy/v1.
Lens version: 2023.1.110749-latest which is installed on Windows 11.

This is fixed by #6906


[Feature Request] split skeleton as a separate layer to show all the skeletons?

I would like to see skeleton only for all the neurons. Although I can set the transparency, but the neurons do not show up in 3D window. This is because of GPU memory limit? It seems that there is too much mesh data which I don't need it.
If skeleton could be a separate layer, then it won't take much memory without the mesh?

my gpu memory is almost full:
~$ nvidia-smi
Thu Jun 21 17:39:34 2018
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 384.130                Driver Version: 384.130                   |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  GeForce GTX 970     Off  | 00000000:03:00.0  On |                  N/A |
| 28%   44C    P0    58W / 151W |   3832MiB /  4034MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+

+-----------------------------------------------------------------------------+
| Processes:                                                       GPU Memory |
|  GPU       PID   Type   Process name                             Usage      |
|=============================================================================|
|    0      1291      G   /usr/lib/xorg/Xorg                           489MiB |
|    0      3005      G   compiz                                       127MiB |
|    0      3366      G   ...-token=37FD1669DDF42972CF1A433273A5325F  3100MiB |
|    0     11211      G   /opt/anaconda3/bin/python                      3MiB |
|    0     11238      G   ...-token=7BE58A4FC24F224750C9C7C67B2D7797    50MiB |
|    0     21523      G   ...-token=1EA3E72345E400165DEADF0E60D73BC8    48MiB |
+-----------------------------------------------------------------------------+

a switch to load and show mesh/skeleton should also work. Would expect this to be an easier change.

@jingpengw As a temporary workaround, Right-Click in the top bar (next to the coordinates) and try to increase the GPU limit to whatever your GPU can handle - let me know if that helps.

I have visualized them based on GLVisualize. don't need it for now.
@nkemnitz should we close this issue?

Na, it's fine. The feature would still be useful.


io.cyanite.http http server exception caught

I get the following error when connecting to graphite-api in grafana direct mode:
ERROR [2015-11-03 03:28:54,151] epollEventLoopGroup-3-1 - io.cyanite.http http server exception caught
java.nio.channels.ClosedChannelException: null

This also breaks all the graphs. Works fine in proxy mode but.

Unfortunately ATM only the proxy mode is working.

Indeed, only proxy mode works reliably for now.


the generated audio has a strange tone

When I train with the AIShell-3 dataset, the generated audio has a strange tone. Have you encountered similar issues?

yes, same problem.


fixes for dotty 0.23

Update build to support Dotty 0.23.0-RC1

fixes #67

Actually for reasons I don’t know, they went back to implicit in some cases.   Try putting it back to using and see what happens.   It didn’t work for me and I recall reading about it.
Sent from my iPhone

On Mar 19, 2020, at 8:40 PM, Seth Tisue<EMAIL_ADDRESS>wrote:

@SethTisue commented on this pull request.
In munit/shared/src/main/scala-3/munit/internal/MacroCompat.scala:

@@ -10,8 +10,8 @@ object MacroCompat {
inline implicit def generate: Location = ${ locationImpl() }
}

def locationImpl()(given qctx: QuoteContext): Expr[Location] = {
import qctx.tasty.{_, given}

def locationImpl()(implicit qctx: QuoteContext): Expr[Location] = {
using is the Scala 3 way now, I think (here and on line 24 too)

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Reverted to using.  I also noticed there's a yarn.lock file in the repo.  I wasn't sure if it was needed.  Presumably not--but I left it for now because it was there before.

@gzoller yarn.lock is the dependecies lockfile of yarn, the javascript package manager. It's used for the website, and it's correct that is included in version control (see https://classic.yarnpkg.com/en/docs/yarn-lock/)

Thank you @gzoller !


Infix function confuses the indent rule

Expected Behavior

I'm writing a DSL for expressing some rules, and use a lot of infix functions.
If write the following:
val rules = "rule1".all(
    "subrule1".all(
        date1 notBefore date2,
    )
).then(
    "subrule2".all(
        date3 notBefore date4
    )
) or "rule1".all(
    date4 notBefore date6
)

everything works as expected. If I use the infix then ktlint becomes confused
val rules = "rule1".all(
    "subrule1".all(
        date1 notBefore date2,
    )
) then(
    "subrule2".all(
        date3 notBefore date4
    )
) or "rule1".all(
    date4 notBefore date6
)

Observed Behavior
ktlint can't decide on how much it wants to indent the three last lines. It just keeps sending me around fixing it back and forth from 0 -> 4 -> 8 -> 4 -> 8 -> 4 -> 8 and so forth
ktlint --verbose says:
File.kt:48:5: Unexpected indentation (expected 8, actual 4) (parameter-list-wrapping)
File.kt:49:1: Unexpected indentation (expected 4, actual 0) (parameter-list-wrapping)

Your Environment

Version of ktlint used: 0.38.1

I'm in a similar situation, but the symptoms are a little more deterministic (so I hope the extra detail helps narrow anything down).
I'm using Exposed, which likewise has an infix-heavy DSL, and sometimes needs brackets for readability.
When trying a long chained expression like this:
AmendmentTable
  .innerJoin(PlanningSchemeAmendmentTable)
  .innerJoin(PlanningSchemeTable)
  .select {
    (PlanningSchemeTable.code eq schemeCode) and (
      (AmendmentTable.exhibitionStartDate eq null) or
        (AmendmentTable.exhibitionStartDate lessEq now())
    )
  }

the infix fun and causes ktlint issues with its closing bracket (wanting to be indented an extra step).
Changing it to
    (PlanningSchemeTable.code eq schemeCode).and(

(no longer infix) removes the reported violation.
Also, indenting it as advised removes the violation, unlike the above reported issue (where it alternates between errors).
Your Environment

ktlint version: 0.41.0? It's not clear from the gradle plugin, and doesn't appear in gradle dependencies.
Integration: org.jlleitschuh.gradle.ktlint - 10.0.0
Gradle: 7.0.1

I do agree with both of you, that the formatting of the examples above is not what you expect. Despite that, it can not be considered as a bug in ktlint. Analog to IntelliJ IDEA, ktlint formats binary expressions between parentheses in this way. From ktlint code (IndentationRule):
            // IDEA quirk:
            // val v = (
            //     true
            //     )
            // instead of expected
            // val v = (
            //     true
            // )
            nextLeafElementType == RPAR &&
                node.treeParent?.elementType == PARENTHESIZED ->
                0

If you disable the ktlint plugin in IntelliJ and reformat the code sample above, the formatted code is comparable with the result of ktlint.

I'm not using IntelliJ for ktlint, but that's interesting.
androa's original issue still sounds like a bug, because running ktlint multiple times gives inconsistent results ("0 -> 4 -> 8 -> 4 -> 8 -> 4 -> 8 and so forth").

Closed by #1284


enhance: support vagrantfile

Vagrant is a tool for managing the virtual environments.
It uses a hardcoded config file name Vagrantfile and uses syntax of Ruby.
Would like to support it through ruby's lexer.

Merged, thanks!


Add LightningChart and SciChart

Since you have commercial libraries you should add
SciCharts and LightningCharts
They are really similar, and look really high-quality
NOTE: I am in no way affiliated with any of the two companies, I just hope the offer some sort of community license.

Thanks for the suggestion. Changes were added.


tpl: Added Google Analytics GTM (gtag.js) internal template

Added an internal template that uses the new Google Tag Manager (gtag.js).
Added basic page data to be sent with the new Google Analytics call.
Added additional privacy flags to disable  google ad signals, good ad personalization signals, and sending enhanced page data to google.
Fixes #4327, Fixes #4479

Great PR. I haven't tested it, but at a glance it looks good. 👍

This is just a personal preference, but the whole "signal" term part in the configuration department could be left out, as it's understood by the parts left of _signal what they mean. At least for the hugo config part.

Hello,
Is this what we need in order to use the new Google Analytics 4 properties (https://support.google.com/analytics/answer/10089681?hl=en&utm_id=ad)?
Regards,
Antoine

@coliff @davidsneighbour do you feel this is going to be used? If so, any changes needed? Or should I just close it?

@coliff @davidsneighbour do you feel this is going to be used? If so, any changes needed? Or should I just close it?

I was under the impression that this was already merged. There was recently a patch for the analytics layout to work with the new shorter number format for GA4, which is only working with gtag.js, so this PR might be obsolete now. On the other hand, those privacy options are definitely not in yet and they should.

I was under the impression that this was already merged. There was recently a patch for the analytics layout to work with the new shorter number format for GA4, which is only working with gtag.js, so this PR might be obsolete now. On the other hand, those privacy options are definitely not in yet and they should.

Thanks for the reply @davidsneighbour. I might be missing it, but the only gtag reference I see in code is in a comment here:
https://github.com/gohugoio/hugo/blob/master/tpl/tplimpl/shortcodes.go#L28-L31
And there is one other related gtag PR with a keep label but it doesn't seem as robust as this one: https://github.com/gohugoio/hugo/pull/4208
That's great to hear that we're getting some sort of gtag update in though.

Thanks for the reply @davidsneighbour. I might be missing it, but the only gtag reference I see in code is in a comment here:
https://github.com/gohugoio/hugo/blob/master/tpl/tplimpl/shortcodes.go#L28-L31
And there is one other related gtag PR with a keep label but it doesn't seem as robust as this one: https://github.com/gohugoio/hugo/pull/4208
That's great to hear that we're getting some sort of gtag update in though.

Looks like https://github.com/gohugoio/hugo/pull/8287 was merged so should have some sort of GA4 support soon.

With the merge of #8287 is this PR still necessary?


ints -> uints in Player.cs

Unsigned is not necessary here, it will never make a difference, and may break code which depends on this.

Please close the PR.

Unsigned is not necessary here, it will never make a difference, and may break code which depends on this.

There's no code that depends on this at the moment. It makes sense to have uints as you can never have negative coins.


index: fix checkout with views

force dir load on view.ls() (used in new diff behavior)
use batched fs.info when possible in checkout

Will fix https://github.com/iterative/dvc/issues/8852

Thank you, @pmrowla ! 🙏


fix: set allowNull: true on non-required model attributes

Please check the following

[ ] Do the tests still pass? (see Run the Tests)
[ ] Is the code formatted properly? (see Linting (Formatting))
For New Features:

[ ] Have tests been added to cover any new features or fixes?
[ ] Has the documentation been updated accordingly?

Please describe additional details for testing this change
This addresses a console warning when connecting to a live db:
Warning: After transforming columnNames back to attribute names for model `season`,
 a record in the result has a value of `null` for property `bracketLink`.
Since the `bracketLink` attribute declares `type: 'string'`,
without ALSO declaring `allowNull: true`, this `null` value is unexpected.
(To resolve, either change this attribute to `allowNull: true` or update
existing records in the database accordingly.)

This doesn't pop up until connecting to a postgres database with pre-existing data, which is why it doesn't happen during local development. Basically in addition to the value not being required, it also needs to explicitly allow null to work properly with how postgres sets missing values as null

This needs a migration to move the old 0-valued match endTime's to null:
update "match" set endTime = null where endTime = 0;


Segmentation Fault [Google Colab]

Hi,
I was able to setup your environment successfully on Colab but while trying to run train/inference on the STPLS3D dataset, I get this error of segmentation fault (core dumped). Do you have any insights on why this might be happening?

Hi,
I was able to setup your environment successfully on Colab but while trying to run train/inference on the STPLS3D dataset, I get this error of segmentation fault (core dumped). Do you have any insights on why this might be happening?

Thanks for sharing your issue! Could you provide me a code for setting Mask3D in Colab if possible? Thanks!


[TheCodingLoveBridge] Fix PHP notice.

Notice was: "Notice: Undefined index: content in …\TheCodingLoveBridge.php on line 35"

Can confirm that this change indeed fixes the php notice. Thanks.

I noticed that this is now a wordpress blog and their feed is at https://thecodinglove.com/feed.
We can remove this bridge.

The difference between this bridge and their feed is that this bridge displays the animations inline as item content, while in the feed the media is an attachment. I dunno what's better...

The bridge is removed. Closing.


issues2/to a

https://github.com/tashirosota/ex-r_enum/issues/10

@mnishiguchi
In addition, the return value when doing something from another map is adjusted to tuple!
I would like to hear your opinions!

In addition, the return value when doing something from another map is adjusted to tuple!

Yeah, I personally like it for the following reasons

now all the key-value pairs are tuples and it is consistent across the library
logic is simpler

@mnishiguchi
Thanks！


It is not possible to use two authenticated sockets in the same process

Hi!
I'm currently working on a program which starts multiple ioloops in different threads. I also want to authenticate all of them, which is not working because of this lines:
        self.zap_socket = self.context.socket(zmq.REP)
        self.zap_socket.linger = 1
        self.zap_socket.bind("inproc://zeromq.zap.01")

in zmq/auth/base.py
Using that two times in the same process throws an 'Address already in use error' since the name of the socket is hardcoded.
Any ideas on how to make this work?

There can only be one authentication socket per Context. Do you want auth to be different for each of these sockets? If so, you will need to create separate Contexts for each authentication context. Otherwise, you should create a single authenticator and use it for all your sockets in the process.

I am creating several contexts. It is not working because the authenticator of both contexts want to create inproc://zeromq.zap.01, which is obviously not possible.

inproc URLs can only be used once in a given context, but there can be many in a process, as long as each one is on a different context. Do you want the authentication to behave differently for every context? If not, creating a new auth socket for each is probably not the right thing to do.
An example creating two sockets bound to the same inproc endpoint:
import zmq
s1 = zmq.Context().socket(zmq.PUB)
s1.bind('inproc://a')
s2 = zmq.Context().socket(zmq.PUB)
s2.bind('inproc://a')


Design - Data Table component

Summary: Design of the Data Table component.
Design on component branch and push to main Figma file when approved.
Acceptance Criteria:

[ ] Component Kickoff
[ ] Sync with Dev and Design
[ ] Research and consulting other design libraries
[ ] WCAG Goal can be completed using a screen reader.
[ ] WCAG Visual elements observe color contrast requirements.
[ ] WCAG Manual accessibility testing meets WCAG 2.1 AA requirements.
[ ] Provides equal service in French and English.
[ ] Sync with Dev and Design
[ ] Comparison with official policy, documented reasons for deviation if any
[ ] Design review
[ ] Policy review
[ ] Design - Different screen sizes
[ ] Design - Short content / long content (text overflow etc.)
[ ] Design - Error state (if applicable)
[ ] Design - Empty state (if applicable)
[ ] Component anatomy
[ ] Write draft of design guidance
[ ] Handoff from draft dev guidance to documentation author/lead
[ ] Update Figma version

https://design.canada.ca/common-design-patterns/tables.html

25% of the way through but there is still the 50% of the mobile to cover and the laptop part. So its gonna overlap next sprint


Added an Arithmetic Function and a Single Unit Test

I opted for a math function because testing for stdout in Rust is not the least time consuming thing in the world without a third-party package.  Would be happy to re-implement a true "Hello World" function, but would need more time.  The options are, third-party package to stub out stdout, or use the Command module in stdlib to have the program call itself and test what comes out of stdout.  I tried to pull this off, but unsurprisingly got an infinite loop of the program calling itself.
If we want doctests (the Python term, not sure what Rust calls them), I'd be happy to spend some time figuring that out.  Apparantly cargo test will only run doctests for libraries, not applications, so it would require a main.rs and a lib.rs (or whatever name).
Let me know if there are any other changes you think should be made.
Coveralls/Travis-CI will only need 'cargo test' in the script section of .toml I think.

Closing request because really I should be pull requesting into 'next'.


TypeError: fsevents.watch is not a function

Describe the bug
When running webpack serve --open using webpack-dev-server (https://github.com/webpack/webpack-dev-server), chokidar is throwing a TypeError: fsevents.watch is not a function exception.
Here's the full output:
> webpack serve --open

<i> [webpack-dev-server] Project is running at:
<i> [webpack-dev-server] Loopback: http://localhost:8080/
<i> [webpack-dev-server] On Your Network (IPv4): http://<IP_ADDRESS>:8080/
<i> [webpack-dev-server] On Your Network (IPv6): http://[fe80::1]:8080/
<i> [webpack-dev-server] Content not from webpack is served from './dist' directory
/Users/Severson/Projects/react-essentials-with-luna/node_modules/chokidar/lib/fsevents-handler.js:93
  const stop = fsevents.watch(path, callback);
                        ^

TypeError: fsevents.watch is not a function
    at createFSEventsInstance (/Users/Severson/Projects/react-essentials-with-luna/node_modules/chokidar/lib/fsevents-handler.js:93:25)
    at setFSEventsListener (/Users/Severson/Projects/react-essentials-with-luna/node_modules/chokidar/lib/fsevents-handler.js:149:16)
    at FsEventsHandler._watchWithFsEvents (/Users/Severson/Projects/react-essentials-with-luna/node_modules/chokidar/lib/fsevents-handler.js:351:18)
    at FsEventsHandler.initWatch (/Users/Severson/Projects/react-essentials-with-luna/node_modules/chokidar/lib/fsevents-handler.js:427:23)
    at FsEventsHandler._addToFsEvents (/Users/Severson/Projects/react-essentials-with-luna/node_modules/chokidar/lib/fsevents-handler.js:516:12)

Versions (please complete the following information):

Chokidar version: 3.5.2
Node version 16.13.0
OS version: MacOS 11.6 (Big Sur)

To Reproduce:
Using Node.js v16.13, run webpack server --open using webpack-dev-server.
Here's where webpack-dev-server is leveraging chokidar: https://github.com/webpack/webpack-dev-server/blob/d8d5d71c8ca495098e1ee30ebc72ffd657ad5ba0/lib/Server.js#L2233
Expected behavior
chokidar should not call fsevents.watch if it is not a function within the lib/fsevents-handler.js:93code base.
Additional context
I posted this bug initially to webpack-dev-server, and they directed me here.  Here's said issue:
https://github.com/webpack/webpack-dev-server/issues/4070

Somehow I resolved my issues.  (I'll close this issue).
I guess upgrading to<EMAIL_ADDRESS>-> 5.64.4 and<EMAIL_ADDRESS>-> 4.6.0 fixed it.
Thanks.

Can we reopen this one? I've got this issue in case where there was fsevents version 1 installed nearby: https://github.com/vitejs/vite/issues/18728

As far as I understand, it will be solved once vite upgrades chokidar
But I seem to remember them being blocked on that for whatever reason (probably waiting for me 😅)
I'll try chase up on that again


dart2js: cps_ir: Find TypeMasks for list and map literals and constants

The following, the ContainerTypeMask is stored on the element X and not the constant.
We have a hack in TypeMaskSystem to associate the type with the element.
static const X = const [1,2,3];

The hack does not work from anonymous constants:
var a = const [1,2,3];
print(a[0]);

Nor does it work for non-constant literals that have an inferred ContainerTypeMask
var a = [1,2,3];
print(a[0]);

The SSA backend seeds these types from TreeElements.
Perhaps we should do the same, but we would probably need a separate 'seedType' field since unreachablity is encoded as propagated type 'nothing'.

Primitive ListLiteral has an allocationSiteType field.
Perhaps we could simply do this for constants too.

Obsolete


Inaccurate vim emulation in marker placement when using insert-mode remappings

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: Code 1.81.1 (Universal) (6c3e3dba23e8fadc360aed75ce363ba185c49794, 2023-08-09T22:20:33.924Z)
OS Version: Darwin arm64 22.5.0

List of extensions:

Extension
Author (truncated)
Version

python
ms-
2023.14.0

vscode-pylance
ms-
2023.8.20

material-icon-theme
PKi
4.29.0

vscode-sort-json
ric
1.20.0

find-it-faster
Tom
0.0.25

vim
vsc
1.25.2

Steps to reproduce / description
I like to make a marker whenever I exit insert mode. Here's a toy example where I replace a word via this approach:
Starting with this file:
this_is_a_variable = old_name
new_name

Using the vim command sequence:
gg$ciw<esc>mfjyiw`fp

Yields the following modification to said file:
this_is_a_variable = new_name
new_name

This behavior is consistent between vim and VSCode.
In vim, you can automate making the marker f with an insert-mode remapping:
" when exiting insert mode, the marker f is made
inoremap <esc> <esc>mf

This shortens the command sequence and preserves the behavior when in vim:
before: gg$ciw<esc>mfjyiw`fp
after:  gg$ciw<esc>jyiw`fp

But making the equivalent remapping in the VSCode settings.json reveals a bug in VSCode's vim emulation. Applying the new command sequence, instead of this:
this_is_a_variable = new_name
new_name

you get this:
this_is_a_varianew_nameble =
new_name

The marker is displaced left of it's correct location, and by an amount equal to the difference in line length before versus after the change (c) operation.
Here is the settings.json used:
{
    "vim.insertModeKeyBindings": [
        {"before": ["<esc>"], "after": ["<esc>", "m", "f"]}, // FIXME
    ]
}

It would be nice if this were fixed, or if a hack existed to correct the behavior.

Wrong repo, sorry.


Can not find resource aws_api_gateway_rest_api defined in target terraform plan

Description
I'm writing a new feature scenario to check if all aws_api_gateway_rest_api resources defined are created as private APIs.
When running terraform-compliance, it seems the aws_api_gateway_rest_api resource type is not identified correctly in the terraform plan
* SKIPPING: Can not find resource aws_api_gateway_rest_api defined in target terraform plan.

To Reproduce
For the below feature file, if i use Given I have resource aws_api_gateway_rest_api defined, the scenario gets skipped with message SKIPPING: Can not find resource aws_api_gateway_rest_api defined in target terraform plan.
If I use the below more general approach to filter the resources, then the scenario doesn't get skipped and resource is identified correctly.
    Given I have any resource defined
    When its type is aws_api_gateway_rest_api

Feature File:
Feature: Ensure all APIGW resources are compliant
  Scenario: Ensure that APIs created with APIGW are private
    Given I have resource aws_api_gateway_rest_api defined
    #Given I have any resource defined
    #When its type is aws_api_gateway_rest_api
    Then it must have endpoint_configuration
    And it must contain types
    And any of its values must match the "(^PRIVATE$)" regex

Plan snippet:
  "planned_values": {
    "root_module": {
      "resources": [
        {
          "address": "aws_api_gateway_rest_api.this",
          "mode": "managed",
          "type": "aws_api_gateway_rest_api",
          "name": "this",
          "provider_name": "registry.terraform.io/hashicorp/aws",
          "schema_version": 0,
          "values": {
            "body": null,
            "endpoint_configuration": [
              {
                "types": [
                  "REGIONAL"
                ]
              }
            ],
            "minimum_compression_size": -1,
            "name": "public-api",
            "parameters": null,
            "tags": null
          }
        }
      ]
    }
  }

Sample Terraform Code:
resource "aws_api_gateway_rest_api" "this" {
  name        = "public-api"

  endpoint_configuration {
    types = ["REGIONAL"]
  }
}

Used terraform-compliance Parameters:
terraform-compliance -f . -p fail.json

Error Output:
* SKIPPING: Can not find resource aws_api_gateway_rest_api defined in target terraform plan.

Expected Behavior:
    Scenario: Ensure that APIs created with APIGW are private
        Given I have any resource defined
        When its type is aws_api_gateway_rest_api
        Then it must have endpoint_configuration
        And it must contain types
		Failure: types property in aws_api_gateway_rest_api.this resource does not match with (^PRIVATE$) case insensitive regex. It is set to [{'address': 'aws_api_gateway_rest_api.this', 'values': ['REGIONAL'], 'type': 'any resource'}].
        And any of its values must match the "(^PRIVATE$)" regex
          Failure:

1 features (0 passed, 1 failed)
1 scenarios (0 passed, 1 failed)
5 steps (4 passed, 1 failed)

Tested Versions:

terraform-compliance version: v1.3.14
terraform version: Terraform v0.12.31
python version:  Python 3.8.7

Nevermind, this is not a bug.
I believe this behavior is caused by the fact that I'm using resource in front of aws_api_gateway_rest_api

skips the test: Given I have resource aws_api_gateway_rest_api defined
passes the test: Given I have aws_api_gateway_rest_api defined


Logo Proposal

Hello...
I came across your amazing project and i wanted to help contribute to your awesome project by using my logo Design Skills to help create a logo for your project...
I am already working on some ideas... i will update it here for you to give feedback and possibly any change or tweek you may want...
Regards.

Hello...
Here is my design idea... please do let me know if you want any tweak or change in colour or design.
You can also share your own idea...
if you find this design good enough and you like it... please do let me know so i can send you all the files including the vector files for your use...

@chimzycash This is awesome!! Thanks for the contribution. The design with the blue background and white text will surly catch everyone's eye. Great Work. Notable credit will be given for your work and contribution! :)

@chimzycash would it also be possible to squeeze this linke somewhere on the big logo?
Simple, powerful, customizable file download manager library for Android.

@tonyofrancis i'm really glad you like the design... ;)
do you want the words SIMPLE, POWERFUL, CUSTOMIZABLE FILE DOWNLOAD LIBRARY FOR ANDROID
to be included in the files?

Like this?

Its missing the Library work in the statement but it looks great

@tonyofrancis i have added it...
is there any other thing you would want me to change so i can do it and get all the FILES READY for you to use

@chimzycash This is perfect! At the bottom right you can add created by: @chimzycash to the files. I would like others to know that you created it when they see the logo instantly. This Project is open source others need to know your contributed. Thanks.

@tonyofrancis
I am really glad you like and accepted the design... and also glad to contribute to your amazing project...
i will prepare all the files including the vector files and upload for your use...
I appreciate the fact that you want me to be recognized as the logo creator :) ... but it wouldn't look great and work well on a logo...
I will get the files ready for your use...

The files are ready for your use and merge
Here are the google drive files:
https://drive.google.com/drive/folders/1vn0qWqopQUwCF-WEwRNVTjwO4-7iqb3q?usp=sharing

@chimzycash awesome! I will update the project. Thanks!


Contracts: Handle struct column specified both at root and nested levels + arrays of structs

resolves https://github.com/dbt-labs/dbt-bigquery/issues/782
resolves https://github.com/dbt-labs/dbt-bigquery/issues/781

Description
Handles scenarios where a nested column type may be specified both at the struct-level (e.g. b.id: int, b.name: string) and parent level b: struct. Additionally handles when parent level data_type is used to specify an array of structs.
This PR resolves two issues -- initially I had aimed to just tackle 782 on its own by ignoring a top-level data_type if specified (preserving its constraints) but the top-level array specification was just a couple additional lines at that point, and figured reviewing / testing it all together would be more straightforward since the issues and affected codepaths were very closely related.

Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
scope of changes is entirely covered by unit tests 🎉
[ ] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

🎩 parent array
-- schema.yml
  - name: nested_array_field
    config:
      contract: {enforced: true}
      materialized: table
    columns:
      - name: my_array
        data_type: array
        constraints:
          - type: not_null
      - name: my_array.id
        data_type: integer
        description: Desc for field_1
      - name: my_array.name
        data_type: string
        description: Desc for field_2

nested_array_field.sql
select
  array( select struct(
    1 as id,
    'name' as name)
  ) as my_array

  UNION ALL

  select null as my_array

🎩 parent specification with constraints
schema.yml

  - name: nested_fields
    config:
      materialized: table
      contract: {enforced: true}
    columns:
      - name: a
        data_type: string
        description: "field a"
      - name: b.id
        data_type: integer
        description: "nested field b.id"
        tests:
          - not_null
      - name: b.name
        data_type: string
      - name: b
        description: "top-level"
        data_type: struct
        constraints:
          - type: not_null

nested_fields.sql
select
  'string' as a,
  struct(
    1 as id,
    'name' as name
  ) as b

  UNION ALL

select 'string2' as a, null as b

@colin-rogers-dbt - I've tagged you for review as this is follow-on work from https://github.com/dbt-labs/dbt-bigquery/pull/738.

🎩 - Missing data_type:
❯ dbt run --select nested_fields --project-dir ~/basic-dbt
17:16:08  Running with dbt=1.6.0-b8
17:16:08  Registered adapter: bigquery=1.6.0-b4
17:16:08  Found 5 models, 1 test, 0 snapshots, 0 analyses, 498 macros, 0 operations, 0 seed files, 0 sources, 0 exposures, 0 metrics, 0 groups
17:16:08
17:16:10  Concurrency: 1 threads (target='dev')
17:16:10
17:16:10  1 of 1 START sql table model dbt_marky.nested_fields ........................... [RUN]
17:16:10  1 of 1 ERROR creating sql table model dbt_marky.nested_fields .................. [ERROR in 0.83s]
17:16:10
17:16:10  Finished running 1 table model in 0 hours 0 minutes and 2.01 seconds (2.01s).
17:16:11
17:16:11  Completed with 1 error and 0 warnings:
17:16:11
17:16:11  Compilation Error in model nested_fields (models/nested_fields.sql)
17:16:11    Contracted models require data_type to be defined for each column. Please ensure that the column name and data_type are defined within the YAML configuration for the ['a', 'b.name'] column(s).
17:16:11
17:16:11    > in macro bigquery__get_empty_schema_sql (macros/utils/get_columns_spec_ddl.sql)
...


Integrate sentry for simulator

Fixes #4962
Describe the changes you have made in this PR -

Integrated Sentry into the CircuitVerse simulator to enhance error tracking capabilities.
Added try-catch blocks in key areas to handle and report errors more effectively.
Implemented error handling in specific modules, including app.js, canvasApi.js, circuit.js, simulationArea.js, and testbench.js.

fixed simulator zoom slider UI that was broken after bootstrap migration from #5135

Summary by CodeRabbit
Release Notes

New Features

Integrated Sentry for enhanced error tracking across multiple components and functionalities.
Updated the zoom slider layout for improved visual consistency.

Bug Fixes

Enhanced error handling in various methods to prevent application crashes and log exceptions.

Documentation

Corrected comments for clarity in the simulationArea.js.

Style

Adjusted CSS for zoom slider buttons to improve alignment and appearance.

These updates aim to improve user experience through better error management and a more polished interface.

@tachyons pls review if you find time
Thankyou !

implemented some of the changes the bots said, looks like there are many more, will make them soon...


aws-sdk version bump to v3

Resolves #71.

Please merge this pr as the fastlane 2.144 is now switching to aws-sdk v3. Having this plugin on v2 leads to a dependency conflict issue
 fastlane-plugin-aws_s3 was resolved to 1.6.1, which depends on
      aws-sdk (~> 2.3) was resolved to 2.3.0, which depends on
        aws-sdk-resources (= 2.3.0) was resolved to 2.3.0, which depends on
          aws-sdk-core (= 2.3.0)

    fastlane (~> 2.144) was resolved to 2.144.0, which depends on
      aws-sdk-s3 (~> 1.0) was resolved to 1.61.1, which depends on
        aws-sdk-core (~> 3, >= 3.83.0)

@jackyLianMT I can rebase PR with latest master, but still looking for reviewer :(

@jaximan, I have a new PR made here to do what you have done with this PR. I am pointing this out if you want to avoid having to rebase.

Closing this since this was completed in #81 when fastlane also changed to using v3. Thanks for the PR thought! ❤


TypeError: shell is not a constructor

Hi,
I have been trying to use node-powershell and I constantly get this error
"TypeError: shell is not a constructor"
I have installed it on multiple machines, different folders, etc and it all brings back the same error message

here is the sample code I'm using to test
var shell = require('node-powershell'); var ps = new shell({executionPolicy: 'Bypass', debugMsg: true, noProfile: true}); ps.addCommand('echo "node-powershell rocks"') .then(function(){ return ps.invoke(); }) .then(function(output){ console.log(output); ps.dispose(); }) .catch(function(err){ console.log(err); ps.dispose(); });

Hi,
I have the same problem.

Saludos.

I was able to fix it.

I'm also experiencing this issue, and the fixes previously mentioned (in this thread and in issue #16 ) are not working for me. Has anyone else found a solution?

For me I just had to go back to an old version and it just works in that way.
From: ZTechGuy @.>
Sent: Tuesday, July 19, 2022 8:16 AM
To: rannn505/child-shell @.>
Cc: jonkindheaven @.>; Author @.>
Subject: Re: [rannn505/child-shell] TypeError: shell is not a constructor (Issue #144)
I'm also experiencing this issue, and the fixes previously mentioned (in this thread and in issue #16 https://github.com/rannn505/child-shell/issues/16  ) are not working for me. Has anyone else found a solution?
—
Reply to this email directly, view it on GitHub https://github.com/rannn505/child-shell/issues/144#issuecomment-1188978529 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKE5C6EUMN5SV3SB4QO4TMDVU2L6FANCNFSM5IULHEMA .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AKE5C6CYU4YEPGYBMTM37E3VU2L6FA5CNFSM5IULHEMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOI3PF6YI.gif Message ID: @.*** @.***> >

@jonkindheaven what version you went back to?

I

For me I just had to go back to an old version and it just works in that way. From: ZTechGuy @.> Sent: Tuesday, July 19, 2022 8:16 AM To: rannn505/child-shell @.> Cc: jonkindheaven @.>; Author @.> Subject: Re: [rannn505/child-shell] TypeError: shell is not a constructor (Issue #144) I'm also experiencing this issue, and the fixes previously mentioned (in this thread and in issue #16 <#16> ) are not working for me. Has anyone else found a solution? — Reply to this email directly, view it on GitHub <#144 (comment)> , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKE5C6EUMN5SV3SB4QO4TMDVU2L6FANCNFSM5IULHEMA . You are receiving this because you authored the thread. https://github.com/notifications/beacon/AKE5C6CYU4YEPGYBMTM37E3VU2L6FA5CNFSM5IULHEMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOI3PF6YI.gif Message ID: @.*** @.***> >

Me too.
To fix, explicitly I did:
const ps = new Shell({
// error ^ TypeError: Shell is not a constructor with "node-powershell": "^5.0.1",
// FIXED !!! it works flawless with "node-powershell": "^4.0.0",

Just as a bump on this issue...
I was running ~5.0.1 and getting the same error as everyone above. I downgraded like @msdiniz suggested and that fixed my error.
I'm guessing there is something that was changed in versions over 4.0.0.
🤷🏼♂


Disable service link

Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #793 (871ef60) into master (7eb2679) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #793   +/-   ##
=======================================
  Coverage   12.01%   12.01%
=======================================
  Files         106      106
  Lines        4745     4745
=======================================
  Hits          570      570
  Misses       4115     4115
  Partials       60       60

Continue to review full report at Codecov.

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


Syntax Highlighting breaks after certain length of line

Prerequisites

[ ] Can you reproduce the problem in safe mode?
[ ] Are you running the latest version of Atom?
[ ] Did you check the debugging guide?
[ ] Did you check the FAQs on Discuss?
[ ] Are you reporting to the correct repository?
[ ] Did you perform a cursory search to see if your bug or enhancement is already reported?

For more information on how to write a good bug report or enhancement request, see the CONTRIBUTING guide.
Description
If a line exceeds a certain length, syntax highlighting stops working.
I'm not sure where to report this bug, is this something the language (in this case Lua) module has to fix or the editor itself?
Steps to Reproduce

[First Step]
Create a new file
[Second Step]
Example code

zoneIDs.SpeciesToZoneId = {
[1755]={[1014]=1014}, -- Plump Jelly
	[1233]={[782]=782}, -- Pocket Reaver
	[1200]={[930]=930}, -- Pygmy Direhorn
	[1907]={[1018]=1018}, -- Pygmy Owl
	[378]={[181]=181,[475]=475,[510]=510,[42]=42,[27]=27,[34]=34,[30]=30,[462]=462,[121]=121,[1024]=1024,[24]=24,[491]=491,[241]=241,[606]=606,[9]=9,[950]=950,[36]=36,[480]=480,[21]=21,[81]=81,[301]=301,[20]=20,[22]=22,[39]=39,[40]=40}, -- Rabbit
	[237]={[201]=201}, -- Ravasaur Hatchling
	[238]={[40]=40}, -- Razormaw Hatchling
}

[and so on...]

Expected behavior: [What you expected to happen]
Syntax highlighting works as expected
Actual behavior: [What actually happened]
Syntax highlighting breaks
http://i.imgur.com/wGRnPTf.png
Versions
Atom    : 1.14.0-dev-87e6776
Electron: 1.3.6
Chrome  : 52.0.2743.82
Node    : 6.3.0
Windows 8.1 x64
You can get this information from executing atom --version and apm --version at the command line. Also, please include the OS and what version of the OS you're running.

This is natural. Each line is capped to match a certain number of tokens, and no more.
It does this for performance reasons.

Thanks @Alhadis. This is a duplicate of https://github.com/atom/atom/issues/1667. Feel free to subscribe there for updates.


Simplify logic in multiple places

In particular, do not use assert when an if suffices.
Also changed this construct
a = condition ? value : a

to
if (condition) {
  a = value
}

because I find it suprising that it ends up self-assigning a to a when the condition is false.

Yes, but groovy logic use that a lot, that's why I put it almost everywhere ;-)

Do you mean that it’s done that way in example main.nf files?

I'll verify it tomorrow

Looking at the nf scripts from the CRG lab (https://github.com/cbcrg), they mainly use the ? : instead of the classical if then else.
I don't mind using either, but I guess it could be a decision to made as a coding best practices for nf workflow.
Shall we talk about that with other nf coders here at SciLife Lab?

I don’t have a problem with using the ternary operator; it is very convenient. What I do not think is good style is to use it to re-assign the same value as I wrote in my first comment above:
a = condition ? value : a

If it were
a = condition ? value1 : value2

it would be fine. But re-assigning a to itself hides what is going on.

Oh I see.
Sorry, I totally misunderstood...
I believed you wanted to remove all ternary if, not just the malformed ones...
I totally agree then...

Is this ok if we do that?
a = condition1 ? value1 :
    condition2 ? value2 :
    condition3 ? value3 :
                 empty

No.


Material Design icons are not included in the build

We use Vue Material in combination with the Material Design Icons node package. However, after building the electron-vue project using npm run build only placeholder texts are shown instead of icon images.

The icons are shown correctly as long as  npm dev run is used.

Node v8.1.2
NPM v5.0.3

How can we include the icons from the Material Design node module into the build?

@jmeinke
I would highly discourage using the material-design-icons package directly. It contains not only the font files needed for the icons, but also includes just about everything else that you don't need (like source files for iOS, Android, etc). This package was never intended to be used directly. When installing material-design-icons into your application you are bloating your final build size with an additional ~52 MB, which will also slow down your build time tremendously.
The recommended way of adding the icons to your application is to use the Google Font API by adding the following in your src/index.ejs.
<link rel="stylesheet" href="//fonts.googleapis.com/icon?family=Material+Icons">

To answer your question, it may depend on how you are importing the icons. In my testing, the below seems to work fine in both development and production.
<style>
  @import "~material-design-icons/iconfont/material-icons.css";
</style>

Final Build Size Comparison (macOS, uncompressed)
284.8 MB - using material-design-icons
232.5 MB - using the Google Font API

Thanks a lot for your comprehensive response!
We have used the solution you suggested (by adding a link to src/index.ejs) and now all the icons are shown. I've also removed 'roboto-fontface' from our node modules and readded it as a link like the icons.
This can be closed :+1:


Feature/1

Pull Request Check List
Resolves: #1

[ ] Added tests for changed code.
[ ] Updated documentation for changed code.

I just noticed CONTRIBUTING.md says "All pull requests, unless otherwise instructed, need to be first accepted into the main branch (master).", so i was on the right track and shouldn't have merged to develop. I thought that was only for hotfixes, or is Git Flow out of fashion now?


Autoprefixer and Grunt Multi Task format support?

Hi there,
I'm just trying to use the grunt multi task pattern, so I can invoke mulitple tasks with different arguments through autoprefixer (using the format defined at : https://github.com/gruntjs/grunt/issues/315).
autoprefixer: {
    standards: {
        options: {
            browsers: ['last 3 versions', '> 1%', 'ie >= 10', 'ff >= 26'],
            diff: true
        },
        apply: {
            src: '<%= app.css %>/app.css'
        }
    },
    ie9: {
        options: {
            browsers: ['ie 9'],
            diff: true
        },
        apply: {
            src: '<%= app.css %>/app-ie-9*.css'
        }
    },
    ie8: {
        options: {
            browsers: ['ie 8'],
            diff: true
        },
        apply: {
            src: '<%= app.css %>/app-ie-8.css'
        }
    }
},

However this generates no output:
$ grunt autoprefixer

Running "autoprefixer:standards" (autoprefixer) task

Running "autoprefixer:ie9" (autoprefixer) task

Running "autoprefixer:ie8" (autoprefixer) task

Done, without errors.

If i remove the nesting of tasks and just run the standards tasks like so, it executes fine - so there's nothing wrong with my paths.
autoprefixer: {
    options: {
        browsers: ['last 3 versions', '> 1%', 'ie >= 10', 'ff >= 26'],
        diff: true
    },
    apply: {
        src: '<%= app.css %>/app.css'
    }
},

Any ideas what I'm missing?
Many thanks!

Aha, I shouldn't be nesting files src so much...
autoprefixer: {
    standards: {
        options: {
            browsers: ['last 3 versions', '> 1%', 'ie >= 10', 'ff >= 26'],
            diff: true
        },
        src: '<%= app.css %>/app.css'
    },
    ie9: {
        options: {
            browsers: ['ie 9'],
            diff: true
        },
        src: '<%= app.css %>/app-ie-9*.css'

    },
    ie8: {
        options: {
            browsers: ['ie 8'],
            diff: true
        },
        src: '<%= app.css %>/app-ie-8.css'
    }
},


can not download models and fatures from the given link

I want to return the inception v3 models, but, when I run the retain.py, I didn't work anymore .it was failed because it can't down the models from the default link. when I down the models manully, it didn't work for me.
The link as follows: https://tfhub.dev/google/imagenet/inception_v3/classification/1
https://tfhub.dev/google/imagenet/inception_v3/feature_vector/1

Can you paste in more information of the error?

Probably the same issue as https://github.com/tensorflow/hub/issues/53.
Cannot download the model anymore.

wget 'https://tfhub.dev/google/nnlm-en-dim128/1?tf-hub-format=compressed' -O /tmp/module.tar.gz
--2018-07-16 16:12:12--  https://tfhub.dev/google/nnlm-en-dim128/1?tf-hub-format=compressed
Resolving tfhub.dev... <IP_ADDRESS>
Connecting to tfhub.dev|<IP_ADDRESS>|:443... connected.
ERROR: cannot verify tfhub.dev's certificate, issued by ‘CN=Puma-dev CA,O=Developer Certificate’:
  Unable to locally verify the issuer's authority.
To connect to tfhub.dev insecurely, use `--no-check-certificate'.

Looks like you have installed "puma-dev" which seems to be a software that used to take control over ".dev" domain. You should configure it to use some other domain such as ".test" or ".puma".

@andresusanopinto That may explains why. Thanks.


The reference of class() and id() in the p5.dom library should specify that they work as "getters" if no argument is given

I noticed that in the reference for other functions like  html(), value(), position(), style() and attribute() this behavior is described. As that was not the case for the class() and id() functions, I had to test them to make sure they worked in the same way.

fixed with https://github.com/processing/p5.js/commit/bfda023839fe25d8f6058cb32597612d43d37a4e


Add ThreadContextPermission for markAsSystemContext and allow core to perform the method

Description
This PR replaces a previous PR and takes a different approach to protect methods in the ThreadContext class. Instead of changing the access modifier, this PR shows how permissions can be declared to protect methods within the ThreadContext class that should not be accessible outside of the core without explicit permission.
With this change, plugins would be able to utilize the method but permission needs to be granted through an entry in the plugin-security.policy file. The permissions would be:
permission java.lang.RuntimePermission "markAsSystemContext";

Related Issues
Resolves https://github.com/opensearch-project/OpenSearch/issues/14931
Check List

[ ] Functionality includes testing.
[ ] API changes companion pull request created, if applicable.
[ ] Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@reta Opening up a PR to introduce a permission for markAsSystemContext. I tried to add a test in the qa module utilizing ExternalTestCluster, but I don't think its possible to define plugin-security.policy files for in-memory cluster + plugin. For a test, I would like to define a plugin like this along with its associated policy file, but I don't think its possible.
@Override
protected Collection<Class<? extends Plugin>> nodePlugins() {
    return Arrays.asList(RankEvalModulePlugin.class);
}

Do you plan to have it backported to 2.x? If yes, we need documentation update, plugin authors should be aware that permissions might have changed for this API.

@reta, yes I would like to backport this change or keep it in main and mark 2.x as @Deprecated or similar to signal to plugin developers not to use this method.
Where would be best to place developer documentation? I added to the javadoc of markAsSystemContext with steps to grant permission to utilize the method.

@reta, yes I would like to backport this change or keep it in main and mark 2.x as @Deprecated or similar to signal to plugin developers not to use this method.

:+1:

Where would be best to place developer documentation? I added to the javadoc of markAsSystemContext with steps to grant permission to utilize the method.

I think [1] would be the best place to mention the change. If we want to limit the impact, for 2.x we could consider replacing the permission check with deprecation warning.
[1] https://opensearch.org/docs/latest/install-and-configure/plugins/

I think [1] would be the best place to mention the change. If we want to limit the impact, for 2.x we could consider replacing the permission check with deprecation warning.

[1] https://opensearch.org/docs/latest/install-and-configure/plugins/

I don't think that's quite the right place, but we don't have developer docs on the documentation website. Long-term, I'd like Plugin Developer docs to have their own place on the development website or provide an SDK (similar to this repo) with a separate documentation website.

@reta How can the deprecation warning differentiate who the caller is though? The usage of the method is not deprecated in this repo so I would not the message to display every time the method is called, only if its called from a class outside of the core.

@reta How can the deprecation warning differentiate who the caller is though?

Something along these lines:
 SecurityManager sm = System.getSecurityManager();
        if (sm != null) {
            try {
                sm.checkPermission(ACCESS_SYSTEM_THREAD_CONTEXT_PERMISSION);
            } catch (SecurityException ex) {
                deprecationLogger.warn( .,... )
            }
        }

That's smart. Thanks @reta!

Should I open up a manual backport with the change from here or wait for backport bot to create a backport and push a commit to the branch?

Should I open up a manual backport with the change from here or wait for backport bot to create a backport and push a commit to the branch?

I will add backport label (so the changelog check passes), but we could take it from there


Add persistence to consul and couchdb

Solves Issue #39
This adds 2 requirements prior to install.
The User must define 2 PersistentVolumeClaims in the openwhisk namespace

openwhisk-consul
openwhisk-couchdb

I've also added an initContainer to consul, to handle the IP change.

I saw issue #23, This was a 'until that's done' solution for consul and a potential permanent solution for couchdb. Which would still require persistence.

close as out-of-date?

close as out-of-date?

thanks @dgrove-oss for looking into this one.


Adding column to MS2LDA decomposition Mass2Motif page with prevalence

Assuming we do calculate those for the fixed Mass2Motifs, we could add the prevalence in a column, to guide the user to those Mass2Motifs that are abundant in the sample.
We could also sort automatically based on the prevalence upon first arrival to the Mass2Motif page (instead off M2M number)
Alternatively, we can take the thresholded version (the degrees) which are there for sure!

add the summary page as for LDA to the decomposition


My node pubkey isn't visible on 1ml.com

Description
My node pubkey isn't visible on 1ml.com. I opened a channel and lightningto.me can't send any receivable bandwidth because as far as it's concerned, I don't exist
Expected Behavior
The node should pop up and I should be able to get bandwidth
Actual Behavior
I can't get bandwidth
Possible Fix
Initially I thought it could've been port forwarding, but I did that, restarted, and still no dice
Steps to Reproduce
No idea
Context
I need bandwidth so I can receive sats
Your Environment

Zap version: Latest
Operating System and version: Win10 Pro, 64bit

My node pubkey: 039ee81adab91c62cdc1a80fc052cfa1d7c6193acb72aadc42ad51fd3e906ea858

Zap opens private channels by default.
https://github.com/LN-Zap/zap-desktop/blob/c385954df6ee3362d4a7b9a43d0f614b3ca41c6b/renderer/reducers/channels/reducer.js#L425
If you open a public channel, your node will appear on 1ml after a few hours.

Are you aware of the option to do that in the UI? Because that was my first
thought, and I could never find it.
On Wed, Jul 29, 2020 at 11:06 PM githorray<EMAIL_ADDRESS>wrote:

Zap opens private channels by default.
https://github.com/LN-Zap/zap-desktop/blob/c385954df6ee3362d4a7b9a43d0f614b3ca41c6b/renderer/reducers/channels/reducer.js#L425
If you open a public channel, your node will appear on 1ml after a few
hours.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/LN-Zap/zap-desktop/issues/3610#issuecomment-666088293,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOMCU4YBRQWFGD6W5CNGSITR6DWTZANCNFSM4PHTLOBQ
.

https://github.com/LN-Zap/zap-iOS/issues/241

@WillWammer I think @githorray meant the Zap on iOS uses private channels by default. I believe desktop Zap doesn't do that by default. You have to specify you want a private channel explicitly.

https://github.com/LN-Zap/zap-iOS/issues/241#issuecomment-527202719

If you are using zap's build in lnd node then by all channels that you open will be private. We don't provide the option to make public channels when using zap this way since your node will not be a routing node.
Likewise, if you are running the built in lnd node, it won\t show up on 1ml.com because your node is not a routing node and is not broadcasting its address to the network (ie, we don't set externalIp since it's not a routing node and likely running behind some kind of home router).


Kill more clients, because its fun!

Change-Id: I2a8b4c59f5b20812f7f841bff28becaf8987e10a

:+1:

This file isn't tested in CI. Merging.


WIP Merge Bootlint scripts and convert it to JS.

WORK IN PROGRESS, DO NOT MERGE!
@jmervine: there are 2 issues at this point:

ln complains that the file doesn't exist which is right ("ln: Source file does not exist"); not sure how else I can create the symlink
uglify fails the first time; it's like it's running before http.get is done

I'll take a look later this afternoon.
On Sat, Dec 17, 2016 at 00:07 XhmikosR<EMAIL_ADDRESS>wrote:

WORK IN PROGRESS, DO NOT MERGE!
@jmervine https://github.com/jmervine: there are 2 issues at this point:

ln complains that the file doesn't exist which is right ("ln:
Source file does not exist"); not sure how else to create the symlink
uglify fails the first time; it's like it's running before http.get
is done

You can view, comment on, or merge this pull request online at:
https://github.com/MaxCDN/bootstrap-cdn/pull/827
Commit Summary

Merge Bootlint scripts and convert it to JS.

File Changes

M public/bootlint/README.md
https://github.com/MaxCDN/bootstrap-cdn/pull/827/files#diff-0 (5)
D scripts/lint.sh
https://github.com/MaxCDN/bootstrap-cdn/pull/827/files#diff-1 (24)
A scripts/update_bootlint.js
https://github.com/MaxCDN/bootstrap-cdn/pull/827/files#diff-2 (69)
D scripts/update_bootlint.sh
https://github.com/MaxCDN/bootstrap-cdn/pull/827/files#diff-3 (35)

Patch Links:

https://github.com/MaxCDN/bootstrap-cdn/pull/827.patch
https://github.com/MaxCDN/bootstrap-cdn/pull/827.diff

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MaxCDN/bootstrap-cdn/pull/827, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAzJBdRS5aqTqLt_oy9U19nHWK0AEaptks5rI5hcgaJpZM4LP1Gf
.

Thanks. Keep also in mind that the ln error might be because I'm on Windows and there's poor support for it. So we make it work on *nix, it's OK for me.

@jmervine: thanks for the very useful comments!

I refactored the error check like you suggested; everything's good there now.
I worked around the Windows symlink issue by just writing the version as simple file; please check out the branch to see if it looks like a symlink on your platform. If not, maybe I should try fs.link or something.
There's still the issue with uglify which I can't solve.

👍 on the windows symlink issue, I totally forgot that's how github handles symlinks
RE uglify, I'll try to look later today.

Never got a chance to look at this. I can try next week. Apologies.

@XhmikosR I finally looked at this. Sorry it took so long. Made some tweaks. LMK what you think. Happy to collaborate, revert and revisit, etc.
Cheers!

@jmervine: thanks!
A couple of things:

we need to pass a flag to uglify to make sourceMappingURL relative because currently I get //# sourceMappingURL=C:\Users\xmr\Desktop\bootstrap-cdn\public\bootlint\0.14.2\bootlint.min.js.map
did you try it on your machine after removing the symlink and 0.14.2 folder? Because on Windows things are messy with symlinks and I can't trust the results.

I believe I fixed source maps. So now only thing left is to verify the symlink  is valid on nix systems.

Oh, and we need to tweak the order of version check since as it is now, replace will throw if no version is specified.

Should we work to get 847 merged and the ES6'ify this, or the otherway, merge this, rebase to 847 and ES6 it there?

Your call, I don't mind rebasing either one.

I'd say merge #847 if you feel it's ready (I do) and then rebase this one. Probably cleaner.

@XhmikosR close?

It's not done yet either.


ENH: Disallow FieldmapEstimation objects with same sources

This PR implements a bi-directional hash-map (dictionary) instead of a set to track fieldmap estimator IDs.
Instead, the registry can be queried by passing in the fieldmap estimator ID, OR a tuple of path-sources (in which case it returns the ID).
This way, a ValueError is raised when one tries to establish a fieldmap estimation using the same set of images.
Resolves: #129.

Hello @oesteban, thank you for submitting the Pull Request!

In the file sdcflows/tests/test_fieldmaps.py:

Line 60:84: E231 missing whitespace after ','
Line 116:1: E303 too many blank lines (3)
Line 136:1: W293 blank line contains whitespace
Line 146:1: W293 blank line contains whitespace

To test for issues locally, pip install flake8 and then run flake8 sdcflows.


Set computer accounts as owned in bloodhound if local admin privs

If you achieve to gain local admin privileges you basically have control over the comuter account. Therefore this is account is set to owned in bloodhound now. (Even with NetBIOS name support for the domain 😁)

FYI: this is branched of https://github.com/Pennyw0rth/NetExec/pull/35 so it is already up-to-date if this one gets merged

We could probably delete the bh_owned module now

Agreed. When you look into these functions the whole block of code mostly the same across the protocols. I just added it to the end of the block but we should centralise a lot of the code probably, but I didn't want to do it in this PR. It is probably a bigger problem.
I can do it in this PR but we should imo open up another one where we tackle the big problem on its own.

With local auth

Reminder @NeffIsBack replace outer single quotes with double quotes and check for f-strings

Showcasing the use case when the domain was not found in the database (e.g. the user specified a wrong domain or used local auth):

For Administrator multiple User Accounts were found in the db so we exit. (Forest with several "Administrator" Accounts)
For the computer CASTELBLACK we successfully retrieve the account (even without a domain) and set it as owned.

(Final result of course without debug output)

Working example with --local-auth only adding the host to the database:


User.Identity.Name is always null

I've been trying to setup a new IdentityServer3 with AspNetIdentity for a few days now. I'm able to login using my existing Identity DB and that's all good but I can never get the User.Identity.Name to contain data.
I've tried multiple attempts at adding custom claims & scopes and adding scopes to clients.
Finally, I loaded up the sample repository and tested it out with the webforms sample since it already used the User.Identity.Name in it's About page.
Using WebForms sample client + AspNetIdentity sample server = User.Identity.Name is always null
Using WebForms sample client + SelfHost with Seq sample server = User.Identity.Name with data
Let me know if this isn't the correct place to put this.

Let me know if this isn't the correct place to put this.

This is a question for stackoverflow.

Can we get a link to the answer at least?

@omayhemo I think this is the link: http://stackoverflow.com/questions/42441771/user-identity-name-is-always-null-when-using-aspnetidentity-with-identityserver3
But it doesn't have an answer yet :(

https://leastprivilege.com/2016/08/21/why-does-my-authorize-attribute-not-work/

Yeah, my solution was to put something like this:
((ClaimsPrincipal)HttpContext.Current.User).FindFirst("sub").Value,

Thanks CesarD, I have already implemented a solution, I should've revisited here to share.
If you want to use some of the inherited methods in the Microsoft.AspNet.Identity.UserManager to pull useful claims (such as UserId) and make as few changes as possible to ASP.net Identity Integrations

Add a 'name' claim with the value from PreferredUserName
Add a 'NameIdentitfier' with the value from the sub.

That'll wire up what Asp.net Identity Expect to find using the canned functionality.  For example from an MVC controller User.Identity.GetUserId()

I updated my question on the stackoverflow link to say that I continued to manually add the claim.
I haven't had the time to dig into why the two sample projects behave differently.
Apologies for not keeping up with my own question!


Chat features

openbazaar-go now has support for chat, allowing peers to exchange strings. Duo Search is now implementing chat, and I am interested in knowing which chat features the reference client will have. That is, how will the reference client interpret the chat strings that are exchanged? In particular,

Will the reference client support links in the chat box?
Will the reference client allow for sharing inline images in the chat box?
Will the reference client allow for sharing inline videos in the chat box?
Will the reference client allow for any other chat features (e.g. markdown, link unfurling, HTML, voice recording, etc.) in the chat box?

Yes, links will be clickable
Probably not in 2.0
Probably not in 2.0
Probably not in 2.0

We may support images and video tags, so long as the media source is external.


[UIFlow 1.0] [JP] General menu items & some GUI items.

Translate the general menu items in UIFlow 1.0 into Japanese.
Such like,

"EXAMPLE": "Example",
"MANAGER": "Manager",
"CATEGORY": "Device :",

and so on.
Also some GUI items is translated, such like

"font": "Font",
"width": "Width",
"height": "Height",
"radius": "Radius",

and so on.

Translation is finished.


page_entries_info seems wierd

I use this helper with inherited resources
= page_entries_info(collection).html_safe

I just have 6 entries in page 2, but it still display 'Displaying users 26 - 56 of 31 in total'
the total count 31 is right,  but  I think it should displayed as 'Displaying users 26 - 31 of 31 in total'
What do you guys think?

:+1:
Im sure thats not right. hehehe
I already have a pull request that i have worked on this helper. Just give me a time to try to reprocude your error. =D
Any hints on what could be doing this will be appreciated too.


[Candidate 2v2 Freezing Bursa ZH v1] Rocks are visible in map border

Observed behavior
Rocks are visible in map border.
Expected behavior
Rocks do not extent into the border.
Screenshots

fixed


Sub-tasks not hiding when are maked as done

Steps:

I've installed the extension
Put my API token
Get the tasks properly
Get the subtasks of a task properly (it hides the subtasks that has already marked as done)
Mark a subtask as done
The subtask is marked in todoist but it's not hid
Subtask doesn't disappear with reloads

PD: Great job, I will help, if I have free time

I was researching a little bit about it. It's a feature of Todoist API. When a task is closed, it won't be shown in active task anymore, but when a subtask is closed, it will be shown as an active task, but marked as completed. I propose to cross out completed subtasks to difference them to the not completed ones.

Other choice it's not to show the completed tasks

I think its better to not show the completed tasks.
I will approve #9
Thanks for your contribution


fix: disable gh CLI ansi colors

Resolve completion issues described in #53 that occur due to ansi colors in gh responses.

wouldn't it be better to use system()'s env opt (and the equivalent in plenary's job?


[Issue] Autostart

OS: Windows 10-64 bit
GPMDP Version: 3.6.0
Issue:
At first (auto)start the player has a whitescreen. Even the window around is missing. However the little icon in the taskbar is expanding and except of help->about, Thumbs and media control fully working. GPMDP Settings are working too.
Workaround:
Use the quit button in the task icon and start it again.
Reinstalling GPMDP didn't help at all.
Reproduce:
Settings are:
general:
-minimize to tray
-own color
-Voice-control
-JSON API
-Playback API
-start with last opened site
-scroll song text
mini player:
-nothing

This sounds like auto-start is still pointing to an old version, please toggle the auto start setting (with an app restart in the middle)


Proposed SIG-Docs meeting agenda for 2023-02-13

Meeting Details

Date/Time: February 13, 2023 @ 10:30am PT
Location: Discord SIG-Docs Voice Room
Moderator: chanmosq
Note Taker: chanmosq

The SIG-Docs Meetings contains the history past calls, including a link to the agenda, recording, notes, and resources.
SIG Updates
What happened since the last meeting?

O3DE - Docs and Community Project Board
Upcoming chair nominations in Spring 2023 (TBD)

Open action items
Checkbox list of open action items from previous meetings. If these are closed before or during the meeting, mark them off.

[ ] https://github.com/o3de/sig-docs-community/issues/102

Newly resolved items

:white_check_mark: https://github.com/o3de/sig-docs-community/issues/97

Initial Agenda
The pre-approved agenda for this meeting is as follows:

Agenda item name - Presenter - Length of time

Description of the item

...

Discuss how to prioritize issues

Propose decreasing meeting cadence to once a month

Meeting notes
Roadmaps: Discussion about how to make roadmaps discoverable in the main webpage (o3de.org) and the O3DE Documentation (o3de.org/docs)

https://github.com/o3de/sig-docs-community/issues/102
To resolve this issue, we've agreed on the following steps:

Update the Contributing Guide (o3de.org/docs/contributing/):
Create a new page that lists each SIG, a brief description of each, and a link to their roadmap.

The description of the SIG should include the key features they support.
It may make sense to format this as a table: SIG, Description, Roadmap.
Page title suggestion: "SIG Roadmaps"
Add a brief introductory paragraph

Update the Contribute section on the home page (o3de.org/contributing):
Add respective links to the (above) Contributing Guide. The purpose of this page is to be a minimal landing page regarding contributing to O3DE.

Proposal for a Meta Roadmap
The purpose of a meta roadmap is to have an overview that highlights key roadmap items across all SIGs, supplemented with a combined view of all the SIG roadmaps for those who want more information. The overview page will likely have heavy influence from the Marketing team and be maintained by them (TBD, discussion with Marketing). The combined SIG Roadmap view will likely be a GitHub project that syncs all of the separate SIG roadmaps into one view.