[incubator-kie-drools#5705] [new-parser] no viable alternative after …

…contains operator
Issue:

https://github.com/apache/incubator-kie-drools/issues/5705

How to replicate CI configuration locally?

Build Chain tool does "simple" maven build(s), the builds are just Maven commands, but because the repositories relates and depends on each other and any change in API or class method could affect several of those repositories there is a need to use build-chain tool to handle cross repository builds and be sure that we always use latest version of the code for each repository.
build-chain tool is a build tool which can be used on command line locally or in Github Actions workflow(s), in case you need to change multiple repositories and send multiple dependent pull requests related with a change you can easily reproduce the same build by executing it on Github hosted environment or locally in your development environment. See local execution details to get more information about it.

How to retest this PR or trigger a specific build:

for pull request and downstream checks

Push a new commit to the PR. An empty commit would be enough.

for a full downstream build

for github actions job: add the label run_fdb

for Jenkins PR check only

If you are an ASF committer for KIE podling, login to Jenkins (https://ci-builds.apache.org/job/KIE/job/drools/), go to the specific PR job, and click on Build Now button.

@yurloc @gitgabrio @mariofusco Please review, thanks!

org.drools.model.codegen.execmodel.CompilerTest#testHashSet and org.drools.model.codegen.execmodel.MvelOperatorsTest passed except another issue (https://github.com/apache/incubator-kie-drools/issues/5708)
Before PR:
drools-model-codegen
[ERROR] Tests run: 2433, Failures: 397, Errors: 7, Skipped: 9
After PR:
drools-model-codegen
[ERROR] Tests run: 2433, Failures: 370, Errors: 7, Skipped: 9

Thank you, @yurloc !

Confirmed that build-chain is fixed so GHA reflects the PR branch.
2024-03-07T02:31:54.3357494Z ##[group]Checkout summary
2024-03-07T02:31:54.3358559Z [INFO] [apache/incubator-kie-drools]
2024-03-07T02:31:54.3360248Z [INFO]      Project taken from apache/incubator-kie-drools:dev-new-parser (Branch HEAD 7afbf251a8796cea644b12599b7b236aa2c5fc76)
2024-03-07T02:31:54.3362165Z [INFO]      Merged tkobayas/drools:incubator-kie-drools-5705-contains into branch dev-new-parser
2024-03-07T02:31:54.3363373Z ##[endgroup]
...
2024-03-07T02:36:23.2065568Z [ERROR] Tests run: 52, Failures: 2, Errors: 0, Skipped: 0, Time elapsed: 0.969 s <<< FAILURE! -- in org.drools.model.codegen.execmodel.MvelOperatorsTest
2024-03-07T02:36:23.2073196Z [ERROR] org.drools.model.codegen.execmodel.MvelOperatorsTest.testMatchesWithFunction[STANDARD_FROM_DRL] -- Time elapsed: 0.003 s <<< FAILURE!

Fixing progress: There were 457 failed tests (-63).


MacOS window title is not set

Describe the bug
The default window title is displayed:

f3d 1.2.1

F3D - A fast and minimalist 3D viewer
Version: 1.2.1
Build date: 2022-01-16 23:05:16
System: Darwin x86_64
Compiler: AppleClang <IP_ADDRESS>00032
RayTracing module: OFF
Exodus module: ON
OpenCASCADE module: 7.6.0 (no metadata)
Assimp module: 5.1.0
VTK version: 9.1.20220110 (build 20220110)

Is this still an issue @Meakk ?

Yes I think it's still there, I will test it soon.


Custom Controller Clone Overflow

Hey, first of all, really cool repo, I've experimented with a few of them and this one already being tested with bhop/surf stuff is a big plus for me
For some reason, when I try using a different controller, clones get generated till there's an overflow on game start
I tried implementing the movement from this which is pretty much the best source engine kind of movement thing for unity
https://github.com/Fragsurf/Fragsurf-2
Repro steps

Took all the scripts from the Movement in https://github.com/Fragsurf/Fragsurf-2 and moved it to your project.
Commented code that was referencing missing stuff (not needed since they are mostly triggers)
Add the SurfCharacter to a random empty gameobject
Add a camera as a child and set parent & child Layer to Player

From that point you have a working SurfCharacter within the project
I then set it up just like your prefab in the test scene but only using the teleportable script on the player and the script on the camera too for portal rendering
and I add this code to the SurfCharacter (which might need a few more tweaks to work)
    void ITeleportCallback.OnTeleport(Portal portal, float newScale) {
        // this.scale = newScale;

        // Transform the velocity through the portal
        MoveData.Velocity = portal.DirectionThroughPortal(MoveData.Velocity);

        // Transform the ground normal through the portal
        MoveData.GroundNormal= portal.DirectionThroughPortal(MoveData.GroundNormal);
    }

Hey, I'm glad to see someone using this project ^^
My first intuition is that you have a TeleportableObject component on your clone prefab (you should not use the same prefabs for the "original" object and the clone).

Appreciate the fast reply, yeah that was exactly it
Had to do another to make the teleportation work but doesn't seem like that's the right way to do it, cause it will teleport the player up aswell (cause i assume it's the center of the portal)
        void ITeleportCallback.OnTeleport(Portal portal, float newScale) {
           ...
            MoveData.Origin = portal.DestinationPortal.transform.position;
        }

How would you do it?

You need to call PositionThroughPortal with MoveData.Origin as paramter:
MoveData.Origin = portal.PositionThroughPortal(MoveData.Origin);

You will also need to apply a transformation to the rotation of your charater MoveData.ViewAngles:
Quaternion rotation = Quaternion.Euler(MoveData.ViewAngles);
rotation = portal.RotationThroughPortal(rotation);
MoveData.ViewAngles = rotation.eulerAngles;

Also, looking at the MoveData struct, there may be other fields that would need to be transformed through the portal (the Vector3  fields).


Bug in Python Application GitHub Action: Pylint Issues

https://github.com/CSC510-Do-Lorenc-McDavitt/DollarBot/actions/runs/11804942732/job/32886293447

This issue has been fixed by #29 because the pylint issues were fixed by the previous team from this semester. They just weren't fixed from the team that worked on the project a year ago.


Document MariaDB issues

We only have this on an anecdotal level, so it would be really good to have a proper black on white account of why we gave up on MariaDB at a previous point in time, and what would be required in order for us to move back.
Posting a reply in this ticket and marking it as closed would be totally sufficient, I imagine.
This is related to #477 but also a number of related discussions.

Also good old #258

Running on MariaDB now. Report issues separately.


[Integration] List All User Generated Links to 'My Links' Section

Design :-

Do's :-

Only list user specific links for specific users.

Wait for Completion of #72

I can work on this @BRAVO68WEB

You participating for JWOC or GSSOC?

I'm participating in GSSOC 22. I want to work on this issue. Please assign it to me.

OK

Assigning to @HarshKumarChoudary but wait for #72

I'm waiting till #72  is not over. Thanks.

@HarshKumarChoudary You may start working now #72 is fixed

Use this https://github.com/BRAVO68WEB/url-minify/commit/f3b175d87b4b9d78805dd28fc4ed7ac6a6974657

@HarshKumarChoudary Updates?

I would like to work on it. @BRAVO68WEB

Ok @Danko-chan


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_f7txo7hofv/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_f7txo7hofv/clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_f7txo7hofv/clone/recipe/meta.yaml:45: missing_description: The recipe is missing a description
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_15xv4poae6/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_15xv4poae6/clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_15xv4poae6/clone/recipe/meta.yaml:45: missing_description: The recipe is missing a description
Errors were found


Support PipeChannels that are half-closed to start with

This should be fairly straightforward to implement: We should have a PipeChannel that starts with only one file descriptor so is implicitly half-closed on the other one.
For more info see #288

Just for posterity, it seems like /dev/null does not currently work on Linux where EPERM is raised during the epoll_cfg syscall. I've worked around this by opening a pipe and immedately closing the uninteresting end.

@GeorgeLyon Yes, you're right. On Linux you need to burn a pipe or something :| .
This actually seems to be the same issue as #2444


How to exclude custom from Ag: search

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[ ] bash
[x] zsh
[ ] fish

Problem / Steps to reproduce
How to exclude custom folders not in .gitignore from Ag: search command.

In Ag --ignore exclude custom directories and files, for example:
ag --ignore '*.js' --ignore '*.txt' foobar
ag --ignore={'*custom_dir_a*','*custom_dir_b*'}  foobar
for me to exclude ./corber and ./vendor folders found great using :Ag foobar
The complete command based on yor refence is
command! -bang -nargs=+ -complete=file Ag call fzf#vim#ag_raw('--ignore={"*corver*","*vendor*"} '.shellescape(<q-args>), <bang>0)
but it doesn't work using
s
I obtain error E471: Argument required

To make ag have parameters similar to above, I copied the :Ag command from the fzf.vim plugin, and made my own function: -- note that I manually added my "--ignore=test --"
function! s:agsearch(query, ...)
  let query = empty(a:query) ? '^(?=.)' : a:query
  let args = copy(a:000)
  let command = '--ignore=tests -- ' . fzf#shellescape(query)
  return call('fzf#vim#ag_raw', insert(args, command, 0))
endfunction
command! -bang -nargs=* AgSearch call s:agsearch(<q-args>,fzf#vim#with_preview(),<bang>0)

@Dotrar You can just call fzf#vim#ag
command! -bang -nargs=* Ag call fzf#vim#ag(<q-args>, '--ignore=tests', fzf#vim#with_preview(), <bang>0)

Thanks, that was hard to find.

Related: https://github.com/junegunn/fzf.vim#vim-functions


Revert "Merge pull request #1109 from dstufft/s3-upload-speedup"

This reverts commit 9df010c4cf8d6d797cc505b8663ccd792b3d582b, reversing changes made to 41b78b788473e6c797c41ac9e42fdc058f96d565.

It appears codecov is having maintenance right now. I'ver verified that this still has 100% coverage locally.


Netbird - No Identity Providers work ! Keycloak - Login Error: User state: Unauthenticated

Describe the problem
After trying to get Netbird Self Hosted with Authentik and getting weird errors (https://github.com/netbirdio/netbird/issues/1684) and no help so far, I decided to give a try to:

Netbird + Zitadel -> This apparently causes some issues (Zitadel related i/o timeout for connection between PostgreSQL and Zitadel-Server, probably cause is 100% CPU usage on my VPS Server and extremely high memory/RAM usage in Zitadel)
Netbird + Keycloak -> This results in a "Login Error: User state: Unauthenticated".

Nothing can be extracted from the logs of keycloak nor netbird-management Docker Container.
Looking at https://github.com/netbirdio/netbird/issues/1590, I am also running into the "CORS Missing Allow Origin".
To Reproduce

Setup Keycloak and Netbird according to the current version of https://docs.netbird.io/selfhosted/identity-providers#keycloak, both running behind Traefik Reverse Proxy with Letsencrypt provided TLS Certificates.
Visit https://netbird.MYDOMAIN.TLD
Get Login Error: User state: Unauthenticated

Expected behavior
Netbird working correctly.
Are you using NetBird Cloud?
Netbird self-hosted.
NetBird version
netbird version
NetBird status -d output:
Not sure how to do this:

netbird-management container: netbird: command not found
netbird-dashboard container: netbird: command not found
netbird-signal container: Error: crun: executable file /bin/sh not found in $PATH: No such file or directory: OCI runtime attempted to invoke a command that was not found
netbird-coturn container: netbird: command not found

Screenshots
Additional context
Environment same as in https://github.com/netbirdio/netbird/issues/1684 (Podman 4.9.3).

hello @luckylinux can you share logs from the following containers:
dashboard:
docker compose logs dashboard

management
docker compose logs management

Can you also share the content of your setup.env? you can mask your domain names, and client IDs, just make sure your are using the same masking value for the same real values.

Sure @mlsmaycon
I had to adjust a bit the command though since I am using Podman and not Docker ...
podman logs netbird-dashboard
podman_logs_netbird-dashboard.log
podman logs netbird-management
podman_logs_netbird-management.log
setup.env
setup.env.txt

Are there any news on this? Also unauthenticated running via keycloak.
Also getting the same CORS error as above. all set up like in the documentation mentioned above.
`
content.js:364 getEmbedInfo
content.js:425 OEMBED https://netbird.MYDOMAIN.tld/?state=EmabodHrAU&session_state=bf19a860-aa59-40d2-847a-6e70c172b05b&iss=https%3A%2F%2Fauth.MYDOMAIN.tld%2Frealms%2FMYREALM&code=fe9d32ef-9d35-4c23-8350-598ac78cf784.bf19a860-aa59-40d2-847a-6e70c172b05b.b7a2ec9a-1fc9-4850-a12c-d5ee66d01aff#callback
2731-e746de7d02695f25.js:1 Checking to see if there is an authorization response to be delivered.
2731-e746de7d02695f25.js:1 Potential authorization request  https://netbird.MYDOMAIN.tld/ Object EmabodHrAU fe9d32ef-9d35-4c23-8350-598ac78cf784.bf19a860-aa59-40d2-847a-6e70c172b05b.b7a2ec9a-1fc9-4850-a12c-d5ee66d01aff undefined
2731-e746de7d02695f25.js:1 Delivering authorization response
/?state=EmabodHrAU&session_state=bf19a860-aa59-40d2-847a-6e70c172b05b&iss=https%3A%2F%2Fauth.MYDOMAIN.tld%2Frealms%2FMYREALM&code=fe9d32ef-9d35-4c23-8350-598ac78cf784.bf19a860-aa59-40d2-847a-6e70c172b05b.b7a2ec9a-1fc9-4850-a12c-d5ee66d01aff#callback:1 Access to fetch at 'https://auth.MYDOMAIN.tld/realms/MYREALM/protocol/openid-connect/token' from origin 'https://netbird.MYDOMAIN.tld' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.
auth.MYDOMAIN.tld/realms/MYREALM/protocol/openid-connect/token:1
   Failed to load resource: net::ERR_FAILED

2731-e746de7d02695f25.js:1
   Uncaught (in promise) TypeError: Failed to fetch
at FetchRequestor.xhr (2731-e746de7d02695f25.js:1:82495)
at BaseTokenRequestHandler.performTokenRequest (2731-e746de7d02695f25.js:1:79114)
at AuthorizationNotifier.listener (2731-e746de7d02695f25.js:1:56541)
at AuthorizationNotifier.onAuthorizationComplete (2731-e746de7d02695f25.js:1:65206)
at 2731-e746de7d02695f25.js:1:66271

`

Are there any news on this? Also unauthenticated running via keycloak. Also getting the same CORS error as above. All set up like in the documentation mentioned above.
       content.js:364 getEmbedInfo
       content.js:425 OEMBED https://netbird.MYDOMAIN.tld/?state=EmabodHrAU&session_state=bf19a860-aa59-40d2-847a-6e70c172b05b&iss=https%3A%2F%2Fauth.MYDOMAIN.tld%2Frealms%2FMYREALM&code=fe9d32ef-9d35-4c23-8350-598ac78cf784.bf19a860-aa59-40d2-847a-6e70c172b05b.b7a2ec9a-1fc9-4850-a12c-d5ee66d01aff#callback
       2731-e746de7d02695f25.js:1 Checking to see if there is an authorization response to be delivered.
       2731-e746de7d02695f25.js:1 Potential authorization request  https://netbird.MYDOMAIN.tld/ Object EmabodHrAU fe9d32ef-9d35-4c23-8350-598ac78cf784.bf19a860-aa59-40d2-847a-6e70c172b05b.b7a2ec9a-1fc9-4850-a12c-d5ee66d01aff undefined
       2731-e746de7d02695f25.js:1 Delivering authorization response
       /?state=EmabodHrAU&session_state=bf19a860-aa59-40d2-847a-6e70c172b05b&iss=https%3A%2F%2Fauth.MYDOMAIN.tld%2Frealms%2FMYREALM&code=fe9d32ef-9d35-4c23-8350-598ac78cf784.bf19a860-aa59-40d2-847a-6e70c172b05b.b7a2ec9a-1fc9-4850-a12c-d5ee66d01aff#callback:1 Access to fetch at 'https://auth.MYDOMAIN.tld/realms/MYREALM/protocol/openid-connect/token' from origin 'https://netbird.MYDOMAIN.tld' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.
       auth.MYDOMAIN.tld/realms/MYREALM/protocol/openid-connect/token:1

       Failed to load resource: net::ERR_FAILED

       2731-e746de7d02695f25.js:1

       Uncaught (in promise) TypeError: Failed to fetch
    at FetchRequestor.xhr (2731-e746de7d02695f25.js:1:82495)
    at BaseTokenRequestHandler.performTokenRequest (2731-e746de7d02695f25.js:1:79114)
    at AuthorizationNotifier.listener (2731-e746de7d02695f25.js:1:56541)
    at AuthorizationNotifier.onAuthorizationComplete (2731-e746de7d02695f25.js:1:65206)
    at 2731-e746de7d02695f25.js:1:66271

My problem is solved. The issue was that the user somehow hasn't had the default-roles-apps role, and with that no uma_authorization and no offline_access roles. Put the role back an it works.

@mlsmaycon : any News on this ? It's really unfortunate I'm not getting anything working.
At least the Webpage should show something useful IMHO, not just crash the entire thing (and the Logs don't really tell much).

I just noticed there might be something related to Traefik Reverse Proxy going on ...
At the moment I only had the Dashboard reverse Proxy, but trying a Netbird Installation from Scratch using ./configure.sh I get these Warnings:
The following forwards have to be setup:
- https://netbird.MYDOMAIN.TLD:443 -http-> dashboard:80
- https://netbird.MYDOMAIN.TLD:443/api -http-> management:443
- https://netbird.MYDOMAIN.TLD:443/management.ManagementService/ -grpc-> management:443
- https://netbird.MYDOMAIN.TLD:443/signalexchange.SignalExchange/ -grpc-> signal:80
You most likely also have to change NETBIRD_MGMT_API_ENDPOINT in base.setup.env and port-mappings in docker-compose.yml.tmpl and rerun this script.
 The target of the forwards depends on your setup. Beware of the gRPC protocol instead of http for management and signal!
You are also free to remove any occurrences of the Letsencrypt-volume netbird-letsencrypt

So actually I am missing the 2nd, 3rd and 4th Rule ...


Improvements for BlockOneDeploySystemContracts

Group common logic from BlockOneDeploySystemContracts of the different networks into a function
Add unit tests for the new function
Remove deprecated sdkerrors
Remove MessageType in the emitted events (this is not messages it could create an issue with the event indexer)

BlockOneDeploySystemContracts has been removed


Fix | Intermittent deadlock error by querying only the procedure created

sp_sproc_columns_100 returns all stored procedures that match the criteria. There is a chance that stored procedures which match the criteria are being altered/updated during the test. If we only query for the stored procedure we created in the test by restricting the SP to use the function name, this won't happen as the SP names are unique.

Good fix and analysis.


Incorrect package name.

Why package name is ello.co.ello?
Not co.ello.ello?

Thanks @aoisensi, this has been fixed.Gonna close the issue.

:+1:


Move active window to mouse cursor

Description of the new feature / enhancement
Press a hotkey, current active window is moved to your cursor, or full screen on the screen your cursor is currently on.
The feature request basicly lets you press a hotkey, it will move whichever window that is currently active to your mouse cursor, or it could full screen on the screen your cursor is currently on.
Scenario when this would be used?
Scenario 1
I have two monitors, main has windows machine and a server box connected to it. secondary has only the windows machine.
Some times i will have to switch inputs to the server box to troubleshoot any issues.
my main monitor keeps the windows machine output online when im displaying the server box output. if i wanted a terminal or a browser on my secondary monitor and i havent had it open on the secondary window, i would have to switch back to the windows machine output to drag the window onto the other screen.
Scenario 2
Sometimes a window's top bar can be offscreen or behind the task bar. you're unable to drag it into view or into a better position as you cant get your cursor on the draggable window title bar.
 Maybe i need to be more detailed.

Sorry, I totally misread the title and jumped to a solution.
I think this may be niche. Your cursor typically is near the active window.

it could be niche, but useful when needed.
the more common scenario out of the two is scenario 1. If your terminal or browser is on screen 1(or last positioned on screen 1), but currently screen 1 is not visible, you cant get the terminal or browser onto screen two without making screen 1 visible.
Its essentially alt tabbing to the window you want to move that is off screen, hit a hotkey and that window will be moved to your cursor position.

Scenario 3
You're presenting, sharing the screen (not an application), and you need to access some other application. You don't know what monitor that application will restore to when you click its Taskbar icon. This utility would allow you to move your mouse to a non-shared screen and restore the application window to that screen. There's precedent for the idea of restoring an application window to some other screen - right-click the Slack Taskbar entry to find Move Slack to Current Display:


Support ⌥ Delete to backspace whole words

Feature description
Ability to backspace a whole word by pressing Option + Delete
From talz000013 on the App Store

Available in latest beta release :
https://testflight.apple.com/join/QSrBK59z


Added Topmost (always on top) functionality to Windows

Added always on top functionality to Windows. Topmost functionality (always on top) can be achieved at window creation by setting WindowOptions::topmost to true, or after creation by calling Window::topmost(bool). This api works on MacOS and Windows.
I tested the Windows implementation using Wine. I'm assuming it will work properly on a real machine. On unix instead of calling unimplemented! The topmost function just returns void so that nothing breaks

@phillvancejr hey, can you update the PR with my suggested change above so this can be merged?

Hey @emoon I don't see any suggested changes above, did I miss something? I'm still kind of new to contributing

Yes. Just see above my previous message.

@emoon I don't see any message before
@phillvancejr hey, can you update the PR with my suggested change above
so this can be merged?

You don't see this?

Sorry you should see the message now (it was in preview mode for me)

Made this change, should be good to go.

Cool but what is up with all the file permission changes?

I actually don't know, since I didn't touch those files in this change. I'll check it out, regards!

Has the file permissions been fixed? Would like to get this merged/resolved

Ok fingers crossed, this should be correct. On OSX and Windows thetopmost attribute can be set at creation time using the WindowOptions struct or after creation using the topmost method. Unix has the same method but it does nothing ,as opposed to throwing an unimplemented error, so that there will not be any crashes.

Thanks!


Issue with /docs/getting-started-guides/kops/

This is a...

[ ] Feature Request
[x ] Bug Report

Problem:
After making the cluster with kops, you enter  kubectl cluster-info. This provides you with the URL to access the Kubernetes master node.  When you visit the provided URL, it prompts you with an HTTP basic auth.
What are the credentials needed?

The tutorial at (https://kubernetes.io/docs/getting-started-guides/kops/) does not suggest anything.
The output of kops update ... does not suggest anything
https://github.com/kubernetes/kops/blob/master/docs/aws.md does not suggest anything
The contents of the generated file when you enter kops edit cluster <cluster name> don't suggest anything.

I'm at a loss...
Proposed Solution:
Provide the credentials somewhere.
Page to Update:
http://kubernetes.io/...

That's the URL to the API Server and the credentials to access it are in ~/.kube/config and are automatically used by kubectl.
Maybe you're thinking of the Dashboard instead? That's an add-on that you have to install separately.

Yes, I thought this would be some kind of web interface.  I'll research the dashboard.


Feature: sort validators

Codecov Report

Merging #156 (5512a27) into master (f7eefb3) will increase coverage by 0.10%.
The diff coverage is 83.33%.

@@            Coverage Diff            @@
##           master    #156      +/-   ##
=========================================
+ Coverage    2.98%   3.09%   +0.10%
=========================================
  Files         493     493
  Lines        9101    9112      +11
  Branches     2608    2608
=========================================
+ Hits          272     282      +10
- Misses       8829    8830       +1

Impacted Files
Coverage Δ

...nchor-protocol/webapp-provider/tx/airdrop/claim.ts
0.00% <0.00%> (ø)

...hor-protocol/webapp-fns/queries/bond/validators.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Add thumbnails.

Add thumbnails to the item list.  Make this an option via the settings (depends on issue #39).

Added in PR #43.


Possible issue in K8s cluster with cilium

What happened?
Setup: kubermatic/kubeone on Hetzner VMs, with cilium addon enabled.
Container from gefyra run can't resolve database connection (K8s service in the cluster). Env variables are all correctly taken from the specified deployment.
A traceroute from the container to the database host shows that it can resolve the FQDN, as well as the IP of the service and seems to reach an IP address inside the cluster. Although I wasn't able to match that IP address to a pod, service or node.
I am of the opinion that it did work at some time in that cluster. However, I can't say for sure, it could also be that I wrongly remember that...
What did you expect to happen?
Container from gefyra run to work without issue, more specific to be able to reach the database host.
Please provide the output of gefyra check.
[INFO] Checking Docker client.
[INFO] Docker client: Ok
[INFO] Checking availability of Gefyra Cargo image...
[INFO] Gefyra Cargo: Available
[INFO] Docker: Ok
[INFO] Checking Kubernetes connection.
[INFO] Kubernetes: Ok
[INFO] Gefyra client version: 1.1.2
How can we reproduce it (as minimally and precisely as possible)?
One of our project clusters, get in contact with me for more infos.
What Kubernetes setup are you working with?

$ kubectl version
# paste output here

WARNING: This version information is deprecated and will be replaced with the output from kubectl version --short.  Use --output=yaml|json to get the full version.
Client Version: version.Info{Major:"1", Minor:"26", GitVersion:"v1.26.3", GitCommit:"9e644106593f3f4aa98f8a84b23db5fa378900bd", GitTreeState:"clean", BuildDate:"2023-03-15T13:40:17Z", GoVersion:"go1.19.7", Compiler:"gc", Platform:"linux/amd64"}
Kustomize Version: v4.5.7
Server Version: version.Info{Major:"1", Minor:"26", GitVersion:"v1.26.1", GitCommit:"8f94681cd294aa8cfd3407b8191f6c70214973a4", GitTreeState:"clean", BuildDate:"2023-01-18T15:51:25Z", GoVersion:"go1.19.5", Compiler:"gc", Platform:"linux/amd64"}

OS version

# On Linux:
$ cat /etc/os-release
PRETTY_NAME="Ubuntu 22.04.3 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.3 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=jammy

$ uname -a
Linux gutschi-ThinkPad-P14s-Gen-2i 6.2.0-32-generic #32~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Fri Aug 18 10:40:13 UTC 2 x86_64 x86_64 x86_64 GNU/Linux

Anything else we need to know?
No response

I investigated this issue and came across cilium/cilium#27758
It seems to be an issue with Cilium at the moment. I'll keep you posted.

@tschale try to look at this: https://github.com/cilium/cilium/issues/27758#issuecomment-1718282433

FWIW: Patching ConfigMap cilium-config with kube-proxy-replacement=strict worked here.

@SteinRobert I'd close this. It is very specific and we can't do anything about it.


CollapsingToolbarScaffoldState.toolbarState.progress changes without scrolling

CollapsingToolbarScaffoldState.toolbarState.progress value changes when you go back in navigation stack even though no scrolling is happening. If toolbar title size depends on progress value, it visibly changes from smallest to largest when navigating back to screen.
It seems that when navigating back, two progress values are dispatched, 0.0f and then 1.0f.

Can you provide a minimal code snippet to help me reproduce your issue?

I am using AnimatedNavHost from accompanist with no animation when switching between destinations. Regular NavHost with fade animation actually hides text size change, but adding breakpoint to text size calculation you can see that progress is dispatched multiple times without scroll
    @OptIn(ExperimentalAnimationApi::class)
    @Composable
    private fun Test() {
        val navController = rememberNavController()
        Column(modifier = Modifier.fillMaxSize()) {
            NavHost(
                navController = navController,
                startDestination = "first",
                modifier = Modifier.fillMaxSize().weight(1f)
            ) {
                composable("first"
                ) { FirstScreen() }
                composable("second") { SecondScreen() }
            }

            BottomNav(navController = navController)
        }
    }

    @Composable
    private fun BottomNav(navController: NavHostController) {
        BottomNavigation() {
            val navBackStackEntry by navController.currentBackStackEntryAsState()
            val currentDestination = navBackStackEntry?.destination

            BottomNavigationItem(
                icon = { Icon(Icons.Default.Home, contentDescription = null) },
                label = { Text("First") },
                selected = currentDestination?.hierarchy?.any { it.route == "first" } == true,
                onClick = {
                    navController.navigate("first") {
                        popUpTo(navController.graph.findStartDestination().id) {
                            saveState = true
                        }
                        launchSingleTop = true
                        restoreState = true
                    }
                }
            )

            BottomNavigationItem(
                icon = { Icon(Icons.Default.Home, contentDescription = null) },
                label = { Text("Second") },
                selected = currentDestination?.hierarchy?.any { it.route == "second" } == true,
                onClick = {
                    navController.navigate("second") {
                        popUpTo(navController.graph.findStartDestination().id) {
                            saveState = true
                        }
                        launchSingleTop = true
                        restoreState = true
                    }
                }
            )
        }
    }

    @Composable
    private fun FirstScreen() {
        val state = rememberCollapsingToolbarScaffoldState()
        CollapsingToolbarScaffold(
            modifier = Modifier
                .fillMaxSize(),
            state = state,
            scrollStrategy = ScrollStrategy.ExitUntilCollapsed,
            toolbar = {
                val textSize = remember(state.toolbarState.progress) {
                    (22 + (34 - 22) * state.toolbarState.progress).sp
                }

                Box(
                    modifier = Modifier
                        .background(Color.Gray)
                        .fillMaxWidth()
                        .height(120.dp)
                        .pin()
                )

                Text(
                    text = "First",
                    fontSize = textSize,
                    modifier = Modifier
                        .road(Alignment.CenterStart, Alignment.BottomStart)
                        .padding(
                            start = 16.dp,
                            end = 16.dp,
                            top = 18.dp,
                            bottom = 18.dp
                        ),
                )
            }
        ) {
            Column(
                verticalArrangement = Arrangement.Center,
                horizontalAlignment = Alignment.CenterHorizontally,
                modifier = Modifier
                    .fillMaxSize()
                    .verticalScroll(rememberScrollState())
            ) {
                Text(text = "First screen")
            }
        }
    }

    @Composable
    private fun SecondScreen() {
        val state = rememberCollapsingToolbarScaffoldState()
        CollapsingToolbarScaffold(
            modifier = Modifier
                .fillMaxSize(),
            state = state,
            scrollStrategy = ScrollStrategy.ExitUntilCollapsed,
            toolbar = {
                val textSize = remember(state.toolbarState.progress) {
                    (22 + (34 - 22) * state.toolbarState.progress).sp
                }

                Box(
                    modifier = Modifier
                        .background(Color.Gray)
                        .fillMaxWidth()
                        .height(120.dp)
                        .pin()
                )

                Text(
                    text = "Second",
                    fontSize = textSize,
                    modifier = Modifier
                        .road(Alignment.CenterStart, Alignment.BottomStart)
                        .padding(
                            start = 16.dp,
                            end = 16.dp,
                            top = 18.dp,
                            bottom = 18.dp
                        ),
                )
            }
        ) {
            Column(
                verticalArrangement = Arrangement.Center,
                horizontalAlignment = Alignment.CenterHorizontally,
                modifier = Modifier
                    .fillMaxSize()
                    .verticalScroll(rememberScrollState())
            ) {
                Text(text = "Second screen")
            }
        }
    }

I can confirm the issue, also seeing this when navigating back from a detail screen to a screen with expanded toolbar and there the title flickers (first is small and then big), because its font size depends on progress.
Could narrow it down to the height being the initial value (maximum int value) and thus progress being near 0 (because height is in the denominator for calculating progress).

@onebone any update on this?

I can confirm the issue and looks like its cause is about timing as @ChristopherKlammt said. As a workaround, I added minHeight, maxHeight to the rememerSaveable and it stopped being dispatched twice. It might be relevant to the fact that the composition at the very first time doesn't dispatch textSize change twice?

@onebone can you share the code for that?


The "pub" script in the repo doesn't support ARM

The sdk/bin/ directory in the dart-sdk repository has various shell scripts. The "dart" shell script tries to find the "best" out/ directory and run dart from there. The "pub" shell script, however, only looks for ReleaseX64.
This means that when you're using a local Dart repo checkout on an ARM system, and have locally built the tools, the "pub" script doesn't work, even though the "dart" one does.
Copying the logic from the "dart" script to the "pub" script fixes it.

This issue was moved to dart-lang/pub#1422

Re-opening here because the pub script is in the SDK repo and usually only touched by SDK maintainers.


Add index to components so the ones that are used across the app can be imported easier

:page_facing_up: Description:
Add index to components so the ones that are used across the app can be imported easier

:heavy_check_mark: Tasks:

Create the src/components/index.js file and add all the exports there
Replace the imports to use the new index

@loopstudio/react-devs

Close this in favor of moving forward to a ducks (features) approach for the boilerplate


Use bulk_insert to write to hudi, why is there no .commit file generated under the .hoodie folder?

flink1.13.6
When using bulk_insert to write to hudi, no .commit file is generated under the hoodie folder

Configuration:
'write. operation' = 'bulk_insert'
Results:
only
20220623224132001.inflight
20220623224132001.commit.requested

Is there any god can help me anwser zhe question?

Did you use the batch execution mode to run the Flink pipeline ? If you are using Flink SQL Client, try to set up this option:
set execution.type=batch;

Is it possible to set execution.type=batch; to read streaming data in Kafka for bulk_insert
data stream:
kafka->bulk_insert->hudi and synchronize to hive

I use the following way to run
bin/flink run -Dexecution.runtime-mode=BATCH -c demo.HudiDemo /XX/XX/XX.jar
where the table settings are as follows
         TableEnv. executeSql(
                 "CREATE TABLE t24(\n" +
                         "id int,\n" +
                         "num int,\n" +
                         "primary key (id) not enforced\n" +
                         ")\n" +
                         "with(\n" +
                         "'connector'='hudi',\n" +
                         "'path' = 'hdfs://XX:8020/XX/XX/t24',\n" +
                         "'table.type'='COPY_ON_WRITE',\n" +
                         "'write.operation' = 'bulk_insert',\n" + +
                         "'hive_sync.enable'='true',\n" +
                         "'hive_sync.table'='h24',\n" +
                         "'hive_sync.db'='db',\n" +
                         "'hive_sync.mode' = 'hms',\n" +
                         "'hive_sync.metastore.uris' = 'thrift://XX:9083'\n" +
                         ")"
         );

But still can't generate commit file:worried:, can't sync to hive at the same time
insert into t24 select * from dgen
The dgen table is 'connector' = 'datagen'
(checkpoint has been set)
:sob: :sob: :sob:

Can I use bulk_insert model  transfer cdc format data from kafka To hudi？I use flinkcdc split sqlserver logdata as cdc format data  to kafka and bulk_insert to hudi. but bulk_insert model not work.I dont know why.

@danny0405 Is not support bulk_insert from Streaming table to hudi?

expect u can answer my question，thank you.

Just use the insert operation instead for COW table, which should be what you request.

it work，thank you

To conclude:

in streaming ingestion, use INSERT operation to write parquets directly with optional async clustering for small files optimization.
in batch ingestion for bootstrap, we can use BULK_INSERT operation.


[Docs] documetation raises ValueError

What we need to document?
make docs raises the following error when Sphinx == version 5.2.0.post0.
 pickling environment... done
checking consistency... done
preparing documents... done
writing output... [  5%] 01_data_preparation

Theme error:
An error happened in rendering the page 01_data_preparation.
Reason: ValueError('too many values to unpack (expected 3)')
make[1]: *** [Makefile:20: html] Error 2

We will use version 5.1.1 for a while.


Prevent Nested Clocks In Aetherling's Map

As https://github.com/phanrahan/magma/issues/238 notes, the CoreIR wire-clocks pass fails when it has to deal with ports that are arrays of clocks.
This commit prevents map from lifting clock ports. Therefore, the wireclocks pass will wire up the clocks for each of the nested instances that have a clock port and doesn't have to deal with a top layer, array of clock ports.

Closing as #592 is a more general solution to the problem.


Fix memory leak in known_folder()

The documentation for SHGetKnownFolderPath states:

The calling process is responsible for freeing this resource once it
is no longer needed by calling CoTaskMemFree, whether
SHGetKnownFolderPath succeeds or not.

Currently the known_folder() function only calls CoTaskMemFree when
SHGetKnownFolderPath returns S_OK, causing a memory leak whenever
SHGetKnownFolderPath fails.
References:
https://docs.microsoft.com/en-us/windows/win32/api/shlobj_core/nf-shlobj_core-shgetknownfolderpath

Thanks, great catch!


Document how to avoid light leaks with GIProbe and SDFGI

master version of https://github.com/godotengine/godot/pull/47429.
See discussion on Rocket.Chat: https://chat.godotengine.org/channel/rendering?msg=KoaoQZzvkojiHrdwi

Thanks!


no  info shown for certain events in IntelliSense

Issue Type: Bug
IntelliSense doesn't show any details about the events onformchange, onforminput and onmousewheel, it just shows a blank line that's half the height of a normal line.
VS Code version: Code 1.41.1 (26076a4de974ead31f97692a0d32f90d735645c0, 2019-12-18T14:58:56.166Z)
OS version: Windows_NT x64 10.0.18363
Extensions (1)

Extension
Author (truncated)
Version

LiveServer
rit
5.6.1

Please provide steps to reproduce the problem
/needsMoreInfo

Usually when typing events, intellisense recognizes them as such and shows a popup box with the info for the event. When writing these events a popup box appears that doesn't contain any info, and the box height is half a line.

I cannot reproduce this on macOS. @s-e1 if you try onchange (which has a rather long description) does it reproduce?

@jrieken Have you ever seen this happening, and how may I find a repro?

Unsure... Could this be timing dependent? E.g that we somehow resolve details, assume they are there but then it failed?


[Hotfix - master] fix(Search): update search api

can close since we will have a hot release later today


Fix a test case to unblock the bots

missed the 5.1 re-branch by a couple of minutes - rdar://problem/48946543

@swift-ci Please test and merge

LLDB does not work with 5.1, should be fixed now?

@swift-ci Please test and merge

@swift-ci Please smoke test and merge

@swift-ci Please test Linux


src/testing/cover: missing discription for CoverBlock struct

fill in the missing discription for CoverBlock struct

Yes

ping @steuhs - Please take a look at the review comments :)


Issue: require all exposed fields to be annotated with @loadable

currently, they are loadable, but not marked as so
it's probably better to force the user to mark them as loadable to avoid confusion
they're not really loadable, there are some minor issues to fix, as well

This is a good first issue. Join the discord and I'll walk you through what has to be done. I think it will be relatively straightforward!

/assign :)

Okay so some random thoughts before we meet and hammer out details.
Background
loadable fields

If a client field selects selects a client field (e.g. UserDetail selects full_name), the initial parameter to UserDetail will include the result of calling full_name (or a renderable react component, if it was an @component client field.)
You can add @loadable to any client field. (Ideally, we would allow that on any field eventually!)
If you selected full_name @loadable, you would instead receive something that contains a function that, when called, will make a network request for the data needed by full_name. This returns a fragment reference, which you can pass to useResult to get the value back (or suspend, if the query is in flight.)

__refetch and re-exposed fields

Re-exposed fields (see schema-extension.graphql) and __refetch fields are similar, in that they're almost like fields that are always selected loadably. But, that function that makes the network request returns void instead of a FragmentReference.
Specifically, their type is (/* refetch has no param, the other one does */) => [string, () => void], whereas a LoadableField is (argsOrVoid) => [string, () => FragmentReference<ReadOutType>]

Steps
There are (at least!) four main tasks here, all of which should happen in separate PRs. 1-3 will land as a single commit, 4 can land separately:
Step 1: Enforce @loadable
The compiler should emit an error if we don't select the fields loadably
Step 2: types

Adding @loadable to __refetch in PetUpdated results in __refetch: LoadableField<void, Pet____refetch__output_type>, in isograph/demos/pet-demo/src/components/__isograph/Pet/PetUpdater/param_type.ts. This is correct
However, we need to change export type Pet____refetch__output_type = () => [string, () => void]; to be export type Pet____refetch__output_type = void in isograph/demos/pet-demo/src/components/__isograph/Pet/__refetch/output_type.ts

Step 3: Implementation of the refetch field
Right now, the __refetch field's inner function returns void. It should return a FragmentReference<void>.
Refetch fields and exposed mutation fields (at least for now) have nothing that you actually want to read as a result of calling them. But, you might want to suspend if the network request is in flight, get a loading indicator, etc. and the mechanism for that is to return a FragmentReference
Step 4: Unify some APIs
useImperativeExposedMutationField is now useless, and it can be replaced with useImperativeLoadableField! Woohoo!
The user can also call useClientSideDefer with refetch fields (maybe we should lint against that, that's likely to be a mistake?) and with mutations (which does make sense!)

Okay I'm down to try, would I encounter difficulties or be unavailable to continue working on the issue I'll make sure to keep you updated!
Looking forward to contributing though!


fix: try/catch discord send error

Wrap the discord send command in a try/catch
Trying to debug #103 , where the choice length is invalid. From the error, it seems the length is 0, which seems odd, since it should not be allowed.

Maybe better to handle it here ? https://github.com/snapshot-labs/snapshot-webhook/blob/master/src/events.ts#L81-L85 🤔

Not viable, unless we wrap each line inside a try/catch, else if something go wrong with sendDiscord, and you catch it, the sendWebhook and sendXtmp will never be executed


[WIP] nan-training-debug

Got to this function be tracing nans back from model outputs up the stack. Found that before the call to
coord_stack = camera_utils.radial_and_tangential_undistort(coord_stack, distortion_params) the coord_stack param is not nan and is nan after. When debugging within this function, I had to switch of torch scripting to insert a breakpoint. This caused the issue to go away, and none of the downstream breakpoints were hit either. When going to reference function in comment they do not seem to script this function. Not sure what within scripting is causing this to break
To reproduce the old error uncomment the torch.jit.script - and you should hit the breakpoint
If we are happy with this solution - I can remove the breakpoint / comment

Great find! It does seem to fix the NAN issue on the redwood2 scene. The only downside is that there is a non-negligible performance hit (I'm seeing that training is ~10% slower). Lets investigate this a bit more before merging, I would be nice if we didn't need to remove jitting. Maybe @kerrj has an idea, he originally added the jitting.

A quick idea would be changing eps in that function to something larger (maybe 1e-4 or something), it's possible lines 373 and 374 are exploding when the denominator is too small. If that doesn't fix it, pinpointing which line exactly is causing nans would be helpful, but that might be tricky inside a jitted function.

Tested to 30k iterations with eps=1e-3 and it seems to work

Tested to 30k iterations with eps=1e-3 and it seems to work

Yeah this seems like a better solution - it works for me as well.

Would be good to do a quality comparison to make sure it doesn't affect results

Would be good to do a quality comparison to make sure it doesn't affect results.

Here is what I get when testing on the poster dataset -
orange is eps 1e-3 and blue 1e-9 - seems fine to me

Tested on a fisheye dataset. Seems to work fine.

Yeah this seems like a better solution - it works for me as well, Ive reverted the jit and updated the eps value

I tried on another dataset (gardenspheres) - the eps change delayed the happening of  'nan', but still occurs in later iterations.


tile38_version and redis_version need to be different

Describe the bug
Some client libs (StackExchange.Redis) utilize the INFO.redis_version in order to determine client behavior. Since redis and tile38 don't share versions, this means that clients end up believing that the redis server is actually much older than it actually is. Because of this some features like checking replication status during heartbeats doesn't work, which means sentinel based replication apps freak out.
For example:
https://github.com/StackExchange/StackExchange.Redis/blob/7e9b90001e9ca648d771ec11c24a7392e7121ec9/src/StackExchange.Redis/ServerEndPoint.cs#L697
https://github.com/StackExchange/StackExchange.Redis/blob/7e9b90001e9ca648d771ec11c24a7392e7121ec9/src/StackExchange.Redis/RedisFeatures.cs#L96-L99
Expected behavior
I'm not sure what to do about this. My suggestion is to set it to a static version that represents the level of redis commands supported.  If so, i can create a PR for it.

Found a workaround


Connecting with a proxy

Hello,
I'm having an issue connecting and I'm not sure what the issue is. I've gone through all the other material and have tried everything and can't seem to find a solution. I have a hunch that it has to do with the single sign on. I was able to test a server without single sign on (but with a few other differences and was able to connect), but am struggling to connect to the servers with single sign on. Here is my code:
from TM1py import TM1Service
with TM1Service(
address='tm1test.time.com',
user='',   #i've left this as an empty string because we use SSO, but also tested with signin
password='' , #same
port=31000,
namespace='Cognos 7',
ssl=True,
gateway='http://C10TEST.time.com/ibmcognos/cgi-bin/cognosisapi.dll'
) as tm1:
server_name = tm1.server.get_server_name()
print(server_name)
When I run this code I get:
TM1pyException: Text:  Status Code: 401 Reason: Unauthorized
When I run the code with my credentials I get the error:
Error: Incorrect padding
Can you help me figure out what is going on? Should I be placing the empty string after user and password or something else? Or do you think the issue is something else?
Thank you for your help!
Best,
Christian

Hi,
does it work when you provide None  instead of empty string for user and password ?
The CAM namespace is case senstitive. This could also be a problem.

For me it works with this config.ini file:
[tm1srv04]
address=localhost
port=13254
user=
password=
namespace=CUBEWISE
gateway=http://localhost:80/ibmcognos/cgi-bin/cognos.cgi
ssl=True

Hi @christianthieme ,
What is your TM1 and TM1py version?
The gateway parameter is a new feature which is not available in the current release.
Cheers,
Vincent

Hi @vviau,
Thank you for the quick reply. I'm running TM1 version 10.2.2 FP5. I'm running TM1py version 1.2.1.

Hi @MariusWirtz ,
Thank you for the quick reply. I reconfigured my code to use the config file and am still getting the 401 code. How is this code collecting my credentials to pass to TM1? I tried entering my credentials and got the following errror: File "L:\Users\ccarl\AppData\Local\Continuum\anaconda3\lib\base64.py", line 87, in b64decode
return binascii.a2b_base64(s)
Error: Incorrect padding
I know the config file works because I ran it on a test server and it worked. The test server is running a newer version of security I think than the production servers. I'm just not sure what else to try or what else could be wrong.
I also tried running the code with None in the user and password field and got :
UnicodeDecodeError: 'utf-8' codec can't decode byte 0x89 in position 1: invalid start byte
Thanks

@MariusWirtz,
I've been doing a lot of testing and research, and it looks like the crux of my problem is there is a governance software that sits on top of our Cognos Servers and is blocking my API calls (I work at a very large, international company). I've found a non-dynamic work around for the moment by clicking the CAMURI link in a browser and pulling down the CAMPassport cookie and feeding it into the token generating function in the RESTservices.py. I am able to authenticate using this method.
Thanks for all your help during this process.
Best,
Christian

Happy to hear that you found a workaround !
If you have further questions on TM1py or enhancement requests just open an issue.
Cheers,
Marius


[SPARK-42706][SQL][DOCS][3.4] Document the Spark SQL error classes in user-facing documentation

What changes were proposed in this pull request?
Cherry-pick for https://github.com/apache/spark/pull/40336.
This PR proposes to document Spark SQL error classes to Spark SQL Guide.

Error Conditions

SQLSTATE Codes

Error Classes that includes sub-error classes (INVALID_FORMAT as an example)

Why are the changes needed?
To improve the usability for error messages for Spark SQL.
Does this PR introduce any user-facing change?
No API change, but yes, it's user-facing documentation.
How was this patch tested?
Manually built docs and check the contents one-by-one compare to error-classes.json.
Closes #40336 from itholic/SPARK-42706.
Authored-by: itholic<EMAIL_ADDRESS>
What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

cc @MaxGekk @cloud-fan

+1, LGTM. Merging to 3.4.
Thank you, @itholic and @cloud-fan @dongjoon-hyun @HyukjinKwon for review.


Support Static Egress IP

When the pod communicates with the outside, the Host IP goes to the Source IP.
Because of organizational firewall policies, the Static Egress IP of a particular pod or namespace is required.
For example, to connect a specific pod of a particular namespace to an external DB,  I must open a firewall between all the worker nodes and the external db.
In this case, it is possible to communicate with the external db even in an undesired namespace.
Although the egress policy might be the answer, it is basically hard to allow the entire worker node ip firewall to be opened for security reasons.
Below is a reference to the openshift OVS feature.
(New Feature) https://blog.openshift.com/how-to-enable-static-egress-ip-in-ocp/
(Legacy Feature) https://blog.openshift.com/accessing-external-services-using-egress-router/
/kind feature

would love to see this possibility too.. also have a requirement where we set up a firewall rule for an exception to the normal blocking of http/https outbound traffic.. but running the proxy in the kubernetes cluster requires us to pin what ip the egress traffic originates from (so a matching fw rule can be set up).

I am running into a similar issue where an external service only responds to a whitelisted ip, would love to see this feature implemented.

Is this on the roadmap? If not then is there an alternate way to achieve this? Would love to see this feature.


No gradient defined for op

System information

Have I written custom code (as opposed to using a stock
example script provided in TensorFlow):
auto entropy_0 = Xlogy(scope_loss.WithOpName("xlogy0"), input_labels_var, out_segmentation);
OS Platform: Windows7
VS2015
Python version: 3.6
Bazel version: 0.26.0
TensorFlow version: tensorflow 2.0
CUDA/cuDNN version: Cuda 10.0 && Cudnn 7.4.1
GPU : Geforce GTX 1070 8GB:

Hey, I got some error when I using op like Concat and Xlogy, which remind me that I have to add C++ gradient. But as https://github.com/tensorflow/tensorflow/issues/19944#issuecomment-556315768 says, the gradient op is existing, so how should I do to use the gradient op?
Thanks !

@minjac
Please share simple indented stand alone code, for us to replicate the issue faced by you in our local environment.

@minjac
Please share simple indented stand alone code, for us to replicate the issue faced by you in our local environment.

Scope scope = Scope::NewRootScope();
auto input = Placeholder(scope.WithOpName("input"), DT_FLOAT);

map<string, Output> vars;
map<string, Output> assigns;
vars["0"] = Variable(scope.WithOpName("w1"), { 1 }, DT_FLOAT);
vars["1"] = Variable(scope.WithOpName("w2"), { 1 }, DT_FLOAT);
assigns["0"] = Assign(scope.WithOpName("assign1"), vars["0"], Input::Initializer(0.f, { 1 }));
assigns["1"] = Assign(scope.WithOpName("assign2"), vars["1"], Input::Initializer(0.f, { 1 }));

vector<Output> weights;
weights.push_back(vars["0"]);
weights.push_back(vars["1"]);

auto mul1 = Mul(scope.WithOpName("mul1"), input, vars["0"]);
auto mul2 = Mul(scope.WithOpName("mul2"), input, vars["1"]);

auto concat = Xlogy(scope.WithOpName("xlogy"), mul1, mul2);

auto label = Placeholder(scope.WithOpName("label"), DT_FLOAT);
auto loss = SquaredDifference(scope.WithOpName("label"), concat, label);

vector<Output> grad_outputs;
TF_CHECK_OK(AddSymbolicGradients(scope, { loss }, weights, &grad_outputs));

int index = 0;
vector<Operation> out_grads;
for (pair<string, Output> i : vars)
{
	string s_index = to_string(index);
	auto adam = ApplyGradientDescent(scope, i.second, 0.01f, { grad_outputs[index] });
	out_grads.push_back(adam.operation);
	index++;
}
Tensor train_input(DT_FLOAT, {1});
train_input.scalar<float>()(0) = 0.1;
Tensor train_label(DT_FLOAT, {2});
train_label.scalar<float>()(0) = 0.1;
train_label.scalar<float>()(1) = 0.2;

unique_ptr<ClientSession> t_session;
vector<Output> ops_to_run;
for (pair<string, Output>i : assigns)
{
	ops_to_run.push_back(i.second);
}
t_session = unique_ptr<ClientSession>(new ClientSession(scope));
TF_CHECK_OK(t_session->Run(ops_to_run, nullptr));

vector<Tensor> out_tensors;
TF_CHECK_OK(t_session->Run({ { input, train_input},{ label, train_label } },{loss},out_grads,&out_tensors));

thanks

@minjac
I ran your code on nightly and face this error

I had not faced this error, because I ran my code on VS2015 by using tensorflow_cc.dll.

Gradient for Xlogy is not implemented yet. But you are welcome to send a PR! You can take a look at the existing C++ gradient implementations here and the python gradient for Xlogy here.

@minjac It looks like you are using an older Version of Tensorflow . Many bugs have been fixed in the latest version. Could you please execute your code using Latest Version 2.5 and let us know if the issue still persists? Thanks!


[Bug]: REST API UI issues

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Collating a bunch of issues here:

[ ] When saving a new API as datasource, the datasource creates two empty headers by default. This should not be the case because it breaks the UI at multiple places.
- Datasource view mode shows empty headers
- REST API shows inherited headers that never render
[ ] When you try to store an API as a datasource, the rules to display the "Save as datasource" option next to the URL bar is different from the one on the Authentication tab. I am always allowed to store an API as datasource from the Authentication tab. We could do with some messaging here.
[ ] The authentication type shown for REST APIs that do not use authentication is shown as DBAUTH in some cases
[ ] The headings "Key" and "Value" for the Headers tab looks different from the headings in the Params tab

Steps To Reproduce
https://user-images.githubusercontent.com/5298848/153391243-03b030b5-0d22-4792-a101-8068d8199cc4.mov

Environment
Production
Version
Cloud

Can disabling the Save as Datasource button in the Authentication tab be a solution for the second issue ?
Was unable to reproduce the third issue : (The authentication type shown for REST APIs that do not use authentication is shown as DBAUTH in some cases). @nidhi-nair Can you please add the steps to reproduce the issue ?

The authentication type shown for REST APIs that do not use authentication is shown as DBAUTH in some cases : Seemed to be a old issue which was not reproducible, so leaving it all other will be solved in the created PR.


Syntax error checking stopped working

Let me know if I should repost this to StackOwerflow.
Issue: My python files are no longer being checked. I can write whatever in the python code and it will not be underscored or marked as an error.
Expected: Red underscoring of syntax errors, imports not found etc...
I am using macOS 10.15.1

Declare "python.linting.pylint": true in .vscode> settings.json. This is a known issue. You can use flake8 until it is fixed. It is not affected by this bug.
Duplicate of #9637 , #9649
Bug also affected #9577
Similar to: microsoft/python-language-server#1868
Relevant to #7314
https://github.com/microsoft/vscode-python/issues/7314#issuecomment-535188164

Closing in favour of #9649


Remove older analysis

This pr aims to software garden this repo by removing older analysis. Please give any constructive feedback while reviewing this pr.

Questions:

Why remove older analysis?
Could these be used in the future?
Are there any analysis that could be used as supplemental?
Could these analyses be refactored to a misc analysis folder?

Negative results or analyses that did not work or were published are still important analyses and thus might have reason to stay in the repo depending on how you answer the above questions.

Questions:
* Why remove older analysis?

Older analysis tend to clutter the repo and detract from the main story as it relates to NF1, especially for datasets that don't provide much value due to sample size, quality, etc...

* Could these be used in the future?

Of course, statistical tests, exploring feature importances, and other analysis are common in many project, and could be applied to other datasets.

* Are there any analysis that could be used as supplemental?

Some of them probably could be used as supplemental, but I don't think it would be very useful to the story. Some of these analysis are also duplicates.

* Could these analyses be refactored to a misc analysis folder?

Possibly, see response to previous question above.

Negative results or analyses that did not work or were published are still important analyses and thus might have reason to stay in the repo depending on how you answer the above questions.
Yes, negative results can also be interesting.

Here is a screenshot of what the repo will look like after this PR:  Some comments based on the new structure of the repo from this branch:
1. I would recommend not having the "classify_genotypes" module if you plan on only having one method. It looks a bit odd to me in the screenshot to have two modules with the same folder name inside.

Agreed, I'll change that

2. In the 0.data_analysis repo, I recommend that you should leave the plate 4 analysis. Even though it isn't included in the manuscript, it was an important part of our process to get to our model and how we decided not to include it.

Which analysis in 0.data_analysis do you mean specifically? In the last NF1 manuscript meeting, one of my action items was to close the plate 4 correlations analysis pr. I think most of the plate 4 analysis deleted were older correlation analysis.

3. In the previous `figures` module, it included an important analysis used in a grant, so it might not be the best to delete it from the repo. I recommend moving it to the 0.data_analysis module so that it can still be referenced.

It looks like all of the code in the figures module are used to visualize plate 4 correlations. See my response to number 2 above.

4. Most of these modules now do not include READMEs (except for the `3.figures` module, which I don't think needs one yet). I recommend adding back descriptive READMEs per module prior to merging this refactor.

This pr is for removing older analysis. I'll be adding the READMEs in a future pr.

Overall, I think most of the data exploration should be kept. From my perspective, it would be okay to remove the old ML models that we don't use and are not optimal, since we don't plan to discuss model optimization in the manuscript.
I agree, I have some code I haven't pushed to gh sitting on my local pc, which I may also create pr(s) for.

Which analysis in 0.data_analysis do you mean specifically? In the last NF1 manuscript meeting, one of my action items was to close the plate 4 correlations analysis pr. I think most of the plate 4 analysis deleted were older correlation analysis.

I just thought of this while looking at the repo, so my only concern was not having access to data that we used in a grant. This might be a conversation at another time since we still will have access to the data in the history. I would recommend keeping the older analysis for now in case we believe it to be necessary to include the updated data. There is no right or wrong answer here from my perspective.

Which analysis in 0.data_analysis do you mean specifically? In the last NF1 manuscript meeting, one of my action items was to close the plate 4 correlations analysis pr. I think most of the plate 4 analysis deleted were older correlation analysis.

I just thought of this while looking at the repo, so my only concern was not having access to data that we used in a grant. This might be a conversation at another time since we still will have access to the data in the history. I would recommend keeping the older analysis for now in case we believe it to be necessary to include the updated data. There is no right or wrong answer here from my perspective.

@jenna-tomkinson I don't think I have removed any analysis we still need, and we eventually need to software garden this repo. With that said, since you are the project lead and are suggesting to keep the older analysis for now, I will keep this pr open, and wait for you to schedule next steps. This may be in the form of a conversation or pr comments.

@MattsonCam Let's plan on software gardening more now, and we can get that analysis back later if necessary. I agree this repo for the manuscript should be clean and concise, and keeping the old data isn't as helpful as say, adding that closed PR with the plate 4 data correlations back later.
We might even consider moving plate 4 analysis into the image-analysis repo if that makes sense. We have time to make a final decision, and I think your original idea to remove plate 4 is good now after thinking about it more given the scope of the manuscript.

@MattsonCam Let's plan on software gardening more now, and we can get that analysis back later if necessary. I agree this repo for the manuscript should be clean and concise, and keeping the old data isn't as helpful as say, adding that closed PR with the plate 4 data correlations back later.
We might even consider moving plate 4 analysis into the image-analysis repo if that makes sense. We have time to make a final decision, and I think your original idea to remove plate 4 is good after thinking about it more given the scope of the manuscript.

Sounds good @jenna-tomkinson! We can always talk more about the other repo later if we want

I think this has been cleaned up really well since I noticed there was a lot of old plate 1 and 2 models. I like that duplicate analysis was removed like UMAPs. I think most of the stuff removed are out of scope of the manuscript (but can be added back later if needed).
I am going to approve this as the changes do not impact the code to generate final manuscript figures (as far as I can tell). As well, I don't think this needs to be looked at again, but my two important things I have for you to address prior to merge is:
1. Please make sure to include the READMEs prior to merging, or the READMEs will be the next PR. Let me know either way.

2. Feel free to remove the old plate 4 correlations for now in 0.data_analysis. This can be further discussed at another manuscript meeting once we have another draft ready.

Nice job gardening the repo! 🏡

Thanks @jenna-tomkinson, will do!

I left a comment with questions regarding the deleted files. I am requesting changes for some sort of change log for deleted file and rationale for the deletion of that file. I think for tracking files and for understanding why files are being deleted this would be a great option.

Sounds good, I don't think a changelog belongs in main, so I'll include it here for reference:
Changelog
These are the changes made in this pr. These changes were completed in decending order, starting from the top of this document.
Removed

0.data_analysis/construct_phenotypic_expression_plate_4_fs_data

Justification: We decided to no longer include correlations of plate 4 constructs

0.data_analysis/plates_1_2_sig_test_nf1_sc_all_cellprofiler.ipynb

Justification: Plates beyond 1 and 2 have more samples are believed to be better quality

0.data_analysis/data

Justification: Plates beyond 1 and 2 have more samples are believed to be better quality

0.data_analysis/figures

Justification: Plates beyond 1 and 2 have more samples are believed to be better quality

0.data_analysis/nbconverted

Justification: Plates beyond 1 and 2 have more samples are believed to be better quality

0.data_analysis/plate_projection_cp_fs_data

Justification: Plates beyond 1 and 2 have more samples are believed to be better quality

figures

Justification: We decided to no longer include correlations of plate 4 constructs

0.data_analysis/plates_1_2_sig_test_nf1_sc_all_cellprofiler.ipynb

Justification: Plates beyond 1 and 2 have more samples are believed to be better quality

1.train_models/linear_reg_plate_1_2_norm_data

Justification: Plates beyond 1 and 2 have more samples are believed to be better quality

1.train_models/classify_genotypes/generate_eval_data.ipynb

Justification: This notebook aimed to correct an absent bug

1.train_models/classify_genotypes/data/nf1_final_model_eval_data.parquet

Justification: This data manifested from a notebook aiming to correct an absent bug

1.train_models/classify_genotypes/model_data.parquet

Justification: This data manifested from a notebook aiming to correct an absent bug

1.train_models/classify_genotypes/nbconverted/generate_eval_data.py

Justification: This code aimed to correct an absent bug

1.train_models/classify_genotypes/model_data.parquet

Justification: This data manifested from a notebook aiming to correct an absent bug

1.train_models/log_reg_plates_cp_fs_data

Justification: This folder aimed to train genotype classification models on individual plates and the combined 3 and 3 prime plates. Since then we have developed a more generalizable model by training on multiple plates

2.evaluate_models/classify_genotypes/genotype_probabilities

Justification: We decided to no longer include plate 4 data

2.evaluate_models/classify_genotypes/nbconverted/plate_4_sirna_genotype_probabilities.py

Justification: We decided to no longer include plate 4 data, so we are not interested in the model evaluation code

2.evaluate_models/classify_genotypes/plate_4_sirna_genotype_probabilities.ipynb

Justification: We decided to no longer include plate 4 data, so we are not interested in the model evaluation code

2.evaluate_models/log_reg_plates_cp_fs_data

Justification: This folder aimed to evaluate genotype classification models on individual plates and the combined 3 and 3 prime plates. Since then we have developed a more generalizable model by training on multiple plates

2.evaluate_models/classify_genotypes/model_evaluation_data/confusion_matrix_hyperparameters.parquet

Justification: This data manifested from a notebook aiming to correct an absent bug

2.evaluate_models/classify_genotypes/model_evaluation_data/metrics_hyperparameters.parquet

Justification: This data manifested from a notebook aiming to correct an absent bug

2.evaluate_models/classify_genotypes/model_evaluation_data/precision_recall_hyperparameters.parquet

Justification: This data manifested from a notebook aiming to correct an absent bug

1.train_models/utils

Justification: These utils were used for training older genotype classification models

1.train_models/log_reg_plates_3_3p_5_cp_norm_data

Justification: The splitting approach for training this model was not efficient and needed to be refined. Due to time constraints we decided not to refine this splitting approach

2.evaluate_models/log_reg_plates_3_3p_5_cp_norm_data

Justification: The splitting approach for training this model was not efficient and needed to be refined. Due to time constraints we decided not to refine this splitting approach

1.train_models/nbconverted

Justification: Code from an old model trained on plate 3 data
2.evaluate_models/log_reg_plate_3_3p_cp_renorm_fs_data
Justification: This is the evaluation of the model trained on the Null and WT cells of plates 3 and 3p. Since then we have developed a more generalizable model by training on multiple plates

3.interpret_models

Justification: Location where older models were interpreted. Since then, we interpret models in 3.figures

Moved

1.train_models/classify_genotypes/* to 1.train_models/

Justification: Simplify the folder structure

2.evaluate_models/classify_genotypes/* to 2.evaluate_models/

Justification: Simplify the folder structure


Fix clippy warning

Fix clippy warning

Change with_single_cert() to with_client_auth_cert()

For Rust compiler 1.71.0

Codecov Report
Patch and project coverage have no change.

Comparison is base (c5cb146) 5.32% compared to head (5488fb9) 5.32%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@          Coverage Diff          @@
##            main     #62   +/-   ##
=====================================
  Coverage   5.32%   5.32%
=====================================
  Files          7       7
  Lines        732     732
=====================================
  Hits          39      39
  Misses       693     693

Impacted Files
Coverage Δ

src/client.rs
0.00% <0.00%> (ø)

... and 2 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Broaden google-cloud-core dependency

What this PR does / why we need it:
Broadens dependency on google-cloud-core. The limitation to 1.4.* seems tighter than it needs to be. The version 1.7.2 (which is the latest before 2.0.0) passes unit tests and makes feast more interoperable as a library with other systems.
Which issue(s) this PR fixes:
Relates to https://github.com/feast-dev/feast/issues/928
Does this PR introduce a user-facing change?:
NONE

See https://github.com/feast-dev/feast/pull/2094


Add sdk for Microsoft.Subscription/subscriptionDefinitions

Adding an sdk client for https://github.com/azure/azure-rest-api-specs/tree/current/specification/resources/resource-manager/Microsoft.Subscription/2017-11-01-preview

Description

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] Please add REST spec PR link to the SDK PR
[ ] I have read the contribution guidelines.
[ ] The pull request does not introduce breaking changes.

General Guidelines

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[ ] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[ ] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
[ ] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

Travis build seems to have failed in Reservations - I don't see how it could be related to my changes.
Please take a look.


Implement delete command

See: https://github.com/minishift/minishift/blob/cf59990e69ce1267cd8f9b343e9518974afdf049/pkg/minikube/cluster/cluster.go#L124-L134


Add RPC head check + [goreleaser]

Description
Handle when unreliable RPCs (eg. boba, canto) return a head block hundreds or thousands of blocks behind the true head.

Will be merged in #1248


Note that the Tasit SDK will support meta transactions

Resolved in https://github.com/tasitlabs/TasitSDK/pull/16


mem-ruby: [WIP] Remove RubySystem static member calls from DataBlock

Currently allowing a constructor with no size. Eventually a size will be required.
Issues with CacheMemory::allocate / CacheMemory::allocateVoid in SLICC parser preventing removing it for now.
Change-Id: Ia63c2ad5cf0bf9d1cbdffba5d3a679bb4d3b1220

@powerjg I was just talking to Brandon, who had implemented this originally about 7 years ago or more, and we came to the conclusion that we might have to modify every protocol to have DataBlock (and WriteMask) ctors that take the size as an argument. I wasn't able to make the last dev meeting but I think this would be a good agenda item for the next one.

@powerjg I was just talking to Brandon, who had implemented this originally about 7 years ago or more, and we came to the conclusion that we might have to modify every protocol to have DataBlock (and WriteMask) ctors that take the size as an argument. I wasn't able to make the last dev meeting but I think this would be a good agenda item for the next one.

Current commits require no changes to protocol files. Minor changes to configs.

@abmerop should we do a merge commit or a squash? I.e., do these commits make sense on their own?

Sorry I missed the end of this, but seems just a squash/merge since I last reviwed.


VS Code Stable 1.9.0 - Extension does not get activated on Windows 7

VSCode Version: 1.9.0
OS Version: Windows 7

Steps to Reproduce:

Install VS Code 1.9
Install PowerShell extension 0.9.0
Create or open a PowerShell file (.ps1)
Notice that the extension does not activate and no extension commands are able to be executed (try "PowerShell: Run Selection").

A couple PowerShell extension users have reported that on Windows 7 and VS Code 1.9 the PowerShell extension no longer becomes activated when opening a .ps1 file (see PowerShell/vscode-powershell#488).  I have not personally been able to reproduce this issue so I'm looking for some ideas on what we can use to debug the problem.  Any ideas?
Thanks!
/cc @DerfOh @emmdeeess

cc @sandy081 @bpasero Any ideas?

@daviwil something in the console?
Please see if you can find any logged output that could help us find the source of the issue:

run code --verbose from the command line and attach the output
in Code select Help | Toggle Developer Tools and select the Console
attach any output you see in the Console you can find

Thanks!

code --verbose

←[93m[main 10:57:32 AM]←[0m Starting VS Code in verbose mode
←[93m[main 10:57:32 AM]←[0m from: c:\Program Files (x86)\Microsoft VS Code\resources\app
←[93m[main 10:57:32 AM]←[0m args: { _: [],
  help: false,
  h: false,
  version: false,
  v: false,
  wait: false,
  w: false,
  diff: false,
  d: false,
  goto: false,
  g: false,
  'new-window': false,
  n: false,
  'new-window-if-not-first': false,
  'reuse-window': false,
  r: false,
  performance: false,
  p: false,
  verbose: true,
  logExtensionHostCommunication: false,
  'disable-extensions': false,
  disableExtensions: false,
  'list-extensions': false,
  'show-versions': false,
  nolazy: false }
[12088:0208/155744:INFO:CONSOLE(70)] "Ctrl+Alt+ keybindings should not be used by default under Windows. Offender: ", source: file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js (70)
←[93m[main 10:57:44 AM]←[0m IPC#vscode-workbenchLoaded
[12088:0208/155748:INFO:CONSOLE(133)] "ERROR starting extension host", source: file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js (133)
[12088:0208/155748:INFO:CONSOLE(133)] "timeout", source: file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js (133)
ApplicationInsights:Sender [ { Error: unable to get local issuer certificate
      at Error (native)
      at TLSSocket.<anonymous> (_tls_wrap.js:1060:38)
      at emitNone (events.js:86:13)
      at TLSSocket.emit (events.js:185:7)
      at TLSSocket._finishInit (_tls_wrap.js:584:8)
      at TLSWrap.ssl.onhandshakedone (_tls_wrap.js:416:38) code: 'UNABLE_TO_GET_ISSUER_CERT_LOCALLY' } ]
ApplicationInsights:Sender [ { Error: unable to get local issuer certificate
      at Error (native)
      at TLSSocket.<anonymous> (_tls_wrap.js:1060:38)
      at emitNone (events.js:86:13)
      at TLSSocket.emit (events.js:185:7)
      at TLSSocket._finishInit (_tls_wrap.js:584:8)
      at TLSWrap.ssl.onhandshakedone (_tls_wrap.js:416:38) code: 'UNABLE_TO_GET_ISSUER_CERT_LOCALLY' } ]
←[93m[main 10:58:36 AM]←[0m Lifecycle#window-before-close 1
←[93m[main 10:58:36 AM]←[0m Lifecycle#unload() 1
←[93m[main 10:58:36 AM]←[0m Lifecycle#window-before-close 1
←[93m[main 10:58:36 AM]←[0m Lifecycle#window-close 1
←[93m[main 10:58:36 AM]←[0m Lifecycle#window-all-closed
←[93m[main 10:58:36 AM]←[0m Lifecycle#before-quit
←[93m[main 10:58:36 AM]←[0m App#will-quit: disposing resources

@DerfOh do you get this as soon as you open a *.ps1 file or actually right on startup?

This is right on startup with a ps1 file as one of the tabs saved using the hot exit feature. I will replicate this without the file being opened initially.

@DerfOh Can you go to Extensions viewlet and see what it says for the installed extensions? Does the reload button is active?

@sandy081 PowerShell is the only active extension and it is at it's latest version (0.9.0)
There is no option to reload the window.

I had a chance to do some more testing today - The UNABLE_TO_GET_ISSUER_CERT_LOCALLY error occurs with or without "http.proxyStrictSSL": false set in 1.8 and 1.9. It still occurs even after I had Security put <IP_ADDRESS> into WebSense to exempt it from SSL MITM. The only new error in the console is:
[7568:0214/152620:INFO:CONSOLE(133)] "ERROR starting extension host", source: file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js (133)
[7568:0214/152620:INFO:CONSOLE(133)] "timeout", source: file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js (133)
Is there some further logging I can enable for the extension host to find out why it is failing?

Looks like some key endpoints were changed in 1.9 that were not accounted for in our firewall exception list. Closing as this issue has been resolved for us, thanks for your time @daviwil, @bpasero, @sandy081, and @joaomoreno :+1:

Thanks for letting us know!


Frontend and AI disconnect from backend

After a certain period of time (ranging from 1-2 minutes) the AI frontend and AI entities receive EVENT_DISCONNECT for no discernible reason. This may be due to the heartbeat timeout, which is when the server pings the client periodically to check if the client is still alive. I tried setting the timeout to infinite, which fixed nothing. Only thought is that the client and server connection gets interrupted by something. Possibly need to throw time intensive calculations into a separate thread in order to keep the heartbeat alive?

Have we figured this out yet? @James-Aric

Has this been fixed?


Update Tokemak's Reward and Protocol Revenue

Going to condense this here too: https://github.com/messari/subgraphs/issues/581


mysql query was empty

Hey.
I have a problem with a query. I've got this error in the log folder;
[17:57:50] [plugins/mysql] error #1065 while executing query "": Query was empty (..\RAS\Script\System\settings.pwn:66 -> ..\RAS\Script\System\settings.pwn:285 -> C:\SeeLocal\pawno\include\smartcmd.inc:475)
This is the 66. line in the settings.pwn;
mysql_tquery(sql_ID, "SELECT * FROM settings",   "_ShowServerSettings", "d", playerid);
MYSQL LOG:
[17:57:50] [DEBUG] mysql_tquery(1, "", "_ShowServerSettings", "d") (..\RAS\Script\System\settings.pwn:66 -> ..\RAS\Script\System\settings.pwn:285 -> C:\SeeLocal\pawno\include\smartcmd.inc:475) [17:57:50] [DEBUG] CCallback::Create(amx=0x15e62b0, name='_ShowServerSettings', format='d', params=0x676de14, param_offset=5) [17:57:50] [DEBUG] CCallback::Create - callback index for '_ShowServerSettings': 147 [17:57:50] [DEBUG] processing specifier 'd' with parameter index 0 [17:57:50] [DEBUG] retrieved and pushed value '1' [17:57:50] [INFO] Callback '_ShowServerSettings' set up for delayed execution. [17:57:50] [DEBUG] created delayed callback with 1 parameter [17:57:50] [DEBUG] CHandle::Execute(this=0x7ca0e0, type=1, query=0x71a53b0) [17:57:50] [DEBUG] CHandle::Execute - return value: true [17:57:50] [DEBUG] mysql_tquery: return value: '1' (..\RAS\Script\System\settings.pwn:66 -> ..\RAS\Script\System\settings.pwn:285 -> C:\SeeLocal\pawno\include\smartcmd.inc:475) [17:57:50] [DEBUG] CConnection::Execute(query=0x71a53b0, this=0x697f020, connection=0x7db870) [17:57:50] [DEBUG] CQuery::Execute(this=0x71a53b0, connection=0x7db870) [17:57:50] [ERROR] error #1065 while executing query "": Query was empty (..\RAS\Script\System\settings.pwn:66 -> ..\RAS\Script\System\settings.pwn:285 -> C:\SeeLocal\pawno\include\smartcmd.inc:475) [17:57:50] [DEBUG] CConnection::GetError(this=0x697f020, connection=0x7db870) [17:57:50] [DEBUG] CCallback::Create(amx=0x15e62b0, name='OnQueryError', format='dsssd)
Could you help to solve this problem? How could it be empty? :/

Hey, sorry for the late response on this.
Do you hook and/or use includes that hook any MySQL natives?


CIV-0000 pulled over commits from test branch

JIRA link (if applicable)
N/A
Change description
Part of the PBAv3 issue, the tests were failing when the toggle was switched on, this PR is to amend those tests when the toggle is set to true.
BEFORE MERGING

ensure that the feature toggle method has the call replaced from its hard coded value of true
Remove the service image pointer in charts/

Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

Before Merging steps from PR title completed


fix checkpoint load error and stop updating paramters in evaluation stage

#3119
When testing QAT in NNI, users would run into these problems

checkpoint load fails, which will cause incremental training fail

in evaluation stage, parameters of weight quantization and output quantization should not be updated

when we load checkpoint and then evaluate immediately, we would see bad model performance because of the reset of inner variable :[steps]

this pr fix these problems

Here scale and zero point are defined with a 1-dimensioon tensor, but might got size changed to 0-dimension tensor during calculation by function call update_quantization_param(bits, rmin, rmax).
rmin and rmax  are calculated by torch.min() and torch.max(), they are 0-dimension tensor, and in update_quantization_param(bits, rmin, rmax), you calculated min value by rmin = min(rmin, 0),  after this step, rmin is still a 0-dimension tensor, such as Tensor(1.23) but not Tensor([1.23]).
Then when we save checkpoint, we might save 0-dimension tensor for scale and zero_point, And now BOMM!
checkpoint load is failed due to paramter size dismatch, because scale and zero_point are defined as 1-dimension Tensor but we see 0-dimension Tensor in checkpoint. this is fixed in commit 1ed0163d8c0e37d0209ba073e7a30d863362bc01

add cuda transformation in quantize calculation, because when using multi GPU training , we might have tensor type mismatch problem, i.e. we might have an error of adding a cpu tensor and a cuda tensor. Although I have no idea why this is happening ==.


Packer Vsphere postprocessor

Hi,
instead of calling ovftool manually after packer, did you try to use the specific postprocessor, https://www.packer.io/docs/post-processors/vsphere.html ?
Will it work ? :)

Hi @maurorappa. Initially, I'd have to say that I don't expect that this would work, since a vsphere endpoint is different to a vCloud Director endpoint. However reading through the docs, it would appear to use the ovftool program under the covers. I'd need to go through the source for the post-processor to work out if there is a way of passing the necessary parameters needed for a vCloud connection.
Alternatively, it is hypothetically possible to make use of the Local Shell post-processor (https://www.packer.io/docs/post-processors/shell-local.html) to automate the calling of ovftool. I've not tried that myself yet, but will probably be giving it a go soon.
Rob

Thanks Rob, I could use the ovftool directly as you suggested, but that post-processor is more elegant and portable :)
I'll keep you updated  with my findings


Linter Failing

Last 4 lines of linter details look like this:
./mathgenerator/mathgen.py:988:100: E231 missing whitespace after ','
make: *** [lint] Error 1
Makefile:4: recipe for target 'lint' failed
Error: Process completed with exit code 2.

218 lines above that that say something similar to the first line mentioned above. Mention missing whitespace and expecting blank lines.

@YuviGold You might be able to fix this.

Hey i checked up the errors, most of them are related to not spacing the variables after ',' and space around operators.

the issue is not  solved yet , spacing the variables after ',' and space around operators didn't made much of a change

@lukew3 I created a PR solving the current issues.
I added a format target so contributors would be able to use make format. It is supposed to fix the small linting errors.

actually i am new to git , like can you exaplain how to use it

should just create a pr


Dynasty not Destiny

In the /Card json files, you have a string called "is_destiny". Not to nitpick, but that deck is called the Dynasty deck. Recommend changing to "is_dynasty".

Fixed with d415819de56cac41711dd298b804c387e6c78177, thanks


Update plugin.js

remove useless code blocks
and added connect dev tool in developer mod

Hi @Xicy
The plugin is special since it's interpolate by nuxt.js at build time.


Reserve space between plot area & controls for content message

Plot controls are not in the same position after the content message appears.
The content message currently pushes all of the plot controls down when it appears. Please instead reserve space for banner under Scatterplot (and also under all other viz tools to accommodate future content messages)
We do not want there to be an oversized white space gap under the plot when there is no content message, so please make the content message less tall and remove any gaps between the content message and the plot controls

in 10/21/2022 EDA UX meeting we had decided that:

There can be more than 1 plot control banners
if 2+, the banners will be stacked.

I suggest we start by leaving space for 2 banners & see how it looks

@danicahelb I am afraid that I am confused about the behavior of the banner, so please correct me if I am wrong. Do you want to leave a banner permanently after a banner is called? Currently, the banner behavior is like this: a) make a banner disappear if an addition change of a plot control by user is irrelevant to banner. For an example with Lineplot Viz, a banner is shown if user toggles on the log scale control while the error bar toggle is on. Under this condition, if user toggles off the log scale control, the banner disappears; b) make a banner be replaced with another banner because additional user action to trigger a new banner is unrelated to the previous action. So we currently show single banner depending the state of user's actions and those actions are irrelevant each other.
For reference, speaking of a possible number of banners, there are five banners for Scatter plot Viz and four banners for Lineplot Viz. The former cases are: 1) no smoothed mean; 2) X log scale and Smoothed mean; 3) Y log scale and Smoothed mean; 4) X log scale and Best fit; 3) Y log scale and Best fit. The latter cases are: 1-2) while X log scale on,  toggle on Bin, and vice versa; 3-4) While Y log scale is one, toggle on Error bar, and vice versa. Assuming that we will always keep banners, e.g., a user took actions of 1) and 2) with Scatter plot, which will have two banners, then what should we do if user will take another action of 3)? Removing the banner 1), moving up/down the banner 2), and showing a banner 3) to the empty place? If this is what you want, it is not that simple task to implement those behaviors because certainly current banner does not have that feature: it is designed for a single banner at each case. Honestly, I cannot estimate exactly how long it would take to make them happen.

Hey @moontrip
(banner is wrong verbiage, we should use context message. i corrected my post above)
we do not want to leave a context message permanently after it is called. it should only be visible when the user selections are incompatible. the current behavior is what we want.
Scatter currently has only 2 content messages, and you cannot get message 2 at the same time as message 1:

Log scale is not available for plot modes with fitted lines. Why?

X log scale and Smoothed mean
Y log scale and Smoothed mean
X log scale and Best fit
Y log scale and Best fit

Smoothed mean(s) were not calculated for one or more data series. Why?

no smoothed mean for 1+ data series

Line plot should has only 2 content messages, and you CAN get both at the same time:

Log scale and binning are not available concurrently. Why?

while X log scale on, toggle on Bin
while Bin on, toggle on X log scale one

Y-axis log scale and error bars are not available concurrently. Why?

While Y log scale is one, toggle on Error bar
While Error bar is one, toggle on log scale

(note, i've updated the verbiage for the line plots for the transition from snack bars to content messages)
3-4) While Y log scale is one, toggle on Error bar, and vice versa.

@danicahelb Thank you for summarizing my questions much more clearly. Now it is much simpler to do than what I initially thought. May I ask one question? Since we have two possible messages at each Viz at the same time, I think that you don't mind to fix the location of each banner. For example, we have two rows for two banners. So, Message 1 is always shown at the first row, and Message 2 is at the second row. I think that a flexible position is quite tricky to implement, so I prefer to having a fixed position per message.

thats fine, thanks @moontrip!

looks great, thanks @moontrip!

My pleasure 😃


Release v0.9.5

I see there is a tag "v0.9.5" but no release.
Is this how it should be or did something go wrong?

Thanks, now it's released

I had to go and click resync in the terraform registry UI for the release to show up in the registry.


Update Jenkins version to 2.73.2 (LTS)

This update is important to get a security fix.

@carlossg  Perhaps it's just easier to update this instead of explaining over and over again why it's not needed? https://github.com/jenkinsci/docker/pull/598, https://github.com/jenkinsci/docker/pull/592, https://github.com/jenkinsci/docker/pull/583, https://github.com/jenkinsci/docker/pull/574, and now this?

I get it, that docker hub repo is deprecated.


LTC5505

"The LTC5505-X is an RF power detector for RF applications operating in the 300MHz to 3.5GHz range. A temperature compensated Schottky diode peak detector and buffer amplifier are combined in a small 5-pin ThinSOT package. The supply voltage range is optimized for operation from a single lithium-ion cell or 3xNiMH.
The RF input voltage is peak detected using an on-chip Schottky diode. The detected voltage is buffered and supplied to the VOUT pin. A power saving shutdown mode reduces supply current to less than 2µA.
The LTC5505-1 operates with input power levels from –28dBm to 18dBm. The LTC5505-2 operates with input power levels from –32dBm to 12dBm."
-Linear Technologies
┆Attachments: LTC5505 Datasheet.pdf | LTC5505 Board Example.png

➤ Gavin Allcorn commented:
300MHz to 3.5GHz