Attach Avi Controller: Fails to add endpoint

Describe the bug

After upgrading from o11nplugin-vro-21.1.3.RELEASE.dar to o11nplugin-vro-22.1.2.RELEASE.dar the AVI endpoint is missing in the vRO inventory.
When running workflow: "Attach Avi Controller" I get error:
ERROR(com.vmware.avi/attachVroClient) Error in (Dynamic Script Module name : attachVroClient#2) java.lang.Exception: Controller with the same IP is already exists: "host.domain.com-admin"

Troubleshooting:
Removing and reinstalling o11nplugin-vro-22.1.2.RELEASE.dar
When running workflow: "Attach Avi Controller" I get error:
ERROR(com.vmware.avi/attachVroClient) Error in (Dynamic Script Module name : attachVroClient#2) java.lang.Exception: Controller with the same IP is already exists: "host.domain.com-admin"

AVI 22.1.2 Build 9086
vRO is embedded in vRA:
vRealize Orchestrator <IP_ADDRESS>65372
vRealize Automation <IP_ADDRESS>552 (20566543)
Reproduction steps

Install o11nplugin-vro-22.1.2.RELEASE.dar
Run workflow: "Attach Avi Controller"

...
Expected behavior
AVI Endpoint should be visible in vRO Inventory
Additional context
AVI 22.1.2 Build 9086
vRO is embedded in vRA:
vRealize Orchestrator <IP_ADDRESS>65372
vRealize Automation <IP_ADDRESS>552 (20566543)

Hi @bmarkussen  Can you please share the logs as well? We are not able to reproduce the issue.

I have renamed servernames and useraccount name. vco-server-app.log for node2 is attached.
Any other logfile you need?
vranode2-vco-server-app.log

FYI: I have tried to add the o11nplugin-vro-22.1.2.RELEASE.dar to an other environment.
In that environment there is no error when running "Attach Avi Controller", but the Avi endpoint is not shown in the vRO inventory.
Running "Attach Avi Controller" a second time gives the error:
ERROR(com.vmware.avi/attachVroClient) Error in (Dynamic Script Module name : attachVroClient#2) java.lang.Exception: Controller with the same IP is already exists: "host.domain.com-admin"
AVI 22.1.1 Build 9052  (Single node)
vRO is embedded in vRA:
vRealize Orchestrator <IP_ADDRESS>65372 (Single node)
vRealize Automation <IP_ADDRESS>552 (20566543) (Single node)

Hi @bmarkussen Can you please try to remove the vRO plugin and reinstall it and share the starting logs for the same?

Hi @mkhachane
I have done these steps and attached log files.

8:13CET Uninstall vRO Plugin
Waiting for vRO to restart
8:18CET Installing vRO Plugin
Waiting for vRO to restart
8:27CET Running "Attach Avi Controller" - Error:

2022-11-03 08:27:56.740 +01:00INFO__item_stack:/item1
2022-11-03 08:27:56.810 +01:00ERROR(com.vmware.avi/attachVroClient) Error in (Dynamic Script Module name : attachVroClient#2) java.lang.Exception: Controller with the same IP is already exists: avi-ctrl1.markussenit.dk-admin
2022-11-03 08:27:56.826 +01:00ERRORWorkflow execution stack:

item: 'Attach Avi Controller/item1', state: 'failed', business state: 'null', exception: 'java.lang.Exception: Controller with the same IP is already exists: avi-ctrl1.markussenit.dk-admin (Dynamic Script Module name : attachVroClient#2)'
workflow: 'Attach Avi Controller' (d541c0da-5105-4c9b-b73b-bae90340e31f)
|  'attribute': name=errorCode type=string value=
|  'attribute': name=host type=string value=avi-ctrl1.markussenit.dk
|  'input': name=controller type=string value=avi-ctrl1.markussenit.dk
|  'input': name=username type=string value=admin
|  'input': name=password type=SecureString value=NULL
|  'input': name=tenant type=string value=admin
|  'input': name=version type=string value=22.1.1
|  'input': name=token type=SecureString value=NULL
|  'input': name=token_based_auth type=boolean value=false
|  'input': name=ignore_warnings type=boolean value=true
|  'no outputs'
*** End of execution stack.
AVI 22.1.1 Build 9052 (Single node)
vRO is embedded in vRA:
vRealize Orchestrator <IP_ADDRESS>65372 (Single node)
vRealize Automation <IP_ADDRESS>552 (20566543) (Single node)
washed-console-logs-vco-server-app.log
washed-file-logs-vco-server-app.log

Hi @bmarkussen  Can you please try to remove resource endpoint from the vRO instance and try to run Attach Avi Controller Workflow? Below are the steps to remove the endpoint,

Navigate to assets tab of the vRO instance and remove the avi-ctrl1.markussenit.dk-admin from the resources.
Run Attach Avi Controller workflow.

Done - Same error.
I have made screenshot of the steps. see attached pdf file.
vRO AVI ERROR.pdf

I can reproduce simliar behaviour with an environment running vRO 8.10 (embedded instance, single node). When installing vRO plugin release 22.1.2 and running the attach workflow and using the plugin version number of 22.1.2 as the version input parameter value I see the AVI controller added to the inventory in vRO however it does not retrieve the configuration from AVI to expand the object in vRO inventory. If I detach and attach again using the AVI controller version number of 21.1.4 it appears in the vRO inventory and I can expand the controller to see the configuration present within AVI. What is the version input parameter used for that could be causing this behaviour. I can provide access to the lab environment internally to VMware employees for log analysis etc if required.

Hi @mkhachane
When do you expect to have time to give feedback on this Issue?

Hi @bmarkussen
We have fixed this issue, you will get the fix on the vRO plugin version 21.1.6 release.

Hi @bmarkussen We have fixed this issue, you will get the fix on the vRO plugin version 21.1.6 release.

Thanks - Perfect.
Do you have an ETA for 21.1.6?
Will there be a fix for 22.1.x as well?

Hi @bmarkussen
21.1.6 will be around end of this month.
Yes, it will be available for 22.1.x as well.

Hi @mkhachane
I have tested the following:
vRA/vRO 8.10 single node:
vRO Plugin 22.1.2.post1 (<IP_ADDRESS>)
Connection to Avi Controller 22.1.2 2p3 (Version parameter set to 22.2.1) is shown in inventory, and Avi object are listed
Connection to Avi Controller 22.1.1 2p2 (Version parameter set to 22.1.1) is shown in inventory, but Avi objects are not listed.
I will test futher in the comming days.

Hi @mkhachane
I'm sorry for the late reply - But unfortunately I still have issues.
@kskilling did you have a chance to test the updated plugin?
Environment A
vRA/vRO 8.10 single node:
Avi 22.1.2 2p3
vRO Plugin 22.1.2.post1 (<IP_ADDRESS>)
AVI Configured Cloud: VMware vCenter/vSphere ESX
First issue:
Attaching Avi Controller 22.1.2 2p3 (Version parameter set to 22.1.2)
Avi endpoint is shown in inventory, but the configured virtual service is not shown:

Second issue:
Trying to add Virtual Service, the Avi endpoint is shown, but the server_cloud is not shown in the dropdown:

Environment B
vRA/vRO 8.10 3 node cluster:
Avi 22.1.2 2p3
vRO Plugin 22.1.2.post1 (<IP_ADDRESS>)
AVI Configured Cloud: NSX-T Cloud
Attaching  Avi Controller 22.1.2 2p3 (Version parameter set to 22.1.2)
Avi endpoint is shown in inventory and the configured virtual service is shown as expected.
Third issue:
"Create Virtual Service NSX-T LB" fails with error:
2022-12-16 10:26:37.910 +01:00INFO__item_stack:/item8
2022-12-16 10:26:37.917 +01:00INFOWorkflow ID in action:-> 731515498
2022-12-16 10:26:37.998 +01:00ERROR(com.vmware.avi/finishVroWorkflow) Error in (Dynamic Script Module name : finishVroWorkflow#4)
{"error": "Tenant 'org.mozilla.javascript.NativeArray@4843d755' does not exist!"}:: Inside Rollback ::null
::null
2022-12-16 10:26:38.010 +01:00ERRORWorkflow execution stack:

item: 'Create Virtual Service NSX-T LB/item8', state: 'failed', business state: 'null', exception: '
{"error": "Tenant 'org.mozilla.javascript.NativeArray@4843d755' does not exist!"}:: Inside Rollback ::null
::null (Dynamic Script Module name : finishVroWorkflow#4)'
workflow: 'Create Virtual Service NSX-T LB' (6b35104b-2cff-4b2e-91f2-0dfa4cd7f205)
I hope you will be able to help me.
Thank you,
Brian

@bmarkussen I have now been able to do some basic testing and can confirm I also have issues with the latest release of the plugin. Here are my results:
Environment configuration

vRA with embedded vRO single node 8.10 (Version <IP_ADDRESS>552 (20566543))
AVI Controller 22.1.2 Build 9086
Plugin release 22.1.2.post1 (<IP_ADDRESS>)
Single virtual service configured on the AVI controller as a test.

Steps performed

Detached the existing instance of the AVI controller from vRO using the workflow from the plugin.
Attached AVI controller in vRO using the workflow from the plugin.
Checked the vRO inventory. vRO Inventory shows the AVI Controller but not the virtual server (same as Brian reported in the previous comment).
Ran the Create VS workflow, and the AVI controller instance and the virtual service are not populated in the drop-down input boxes.

We are not able to reproduce the issue on vRO 8.11 single node server. Can you please try with 8.11 vRO single node server?

We are not able to reproduce the issue on vRO 8.11 single node server. Can you please try with 8.11 vRO single node server?

I will upgrade my environment and test if the issue is solved. I will back to you within a few days.

Hi @mkhachane
I have upgraded the environment and get the same errors:
vRA/vRO <IP_ADDRESS>63650 single node:
Avi 22.1.2 2p3
vRO Plugin 22.1.2.post1 (<IP_ADDRESS>)
AVI Configured Cloud: VMware vCenter/vSphere ESX
Issue 1:
Attaching Avi controller is working, but the VS is not shown:

Issue 2:
When running workflow VS Create the server_cloud is not published:

I'm trying to get my vRO lab upgraded to 8.11 to complete testing to see if I still get issues. I will update this issue once I have results.

I have upgraded my lab to 8.11 (single node, latest AVI plugin version) and have the same behaviour as @bmarkussen is reporting. AVI controller appears in inventory after running the attach workflow but VS isn't shown. When running workflow VS Create the server_cloud is not published.
In my lab, the VS in AVI is unhealthy as it is just an empty instance created for testing, and does not have the associated networks available to report as healthy.

@mkhachane My lab is accessible to any VMware employee if you would like to reproduce the issue for investigation.

Create Virtual Service NSX-T LB" fails with error:

For Above issue, there is a workaround. It will be fixed in upcoming release.
Add following line after line number 2 in createHealthMonitorObj action as per the screenshot below,
Tenant = workflowRuntime.getAviVroClient().cred.tenant;

Hi @mkhachane
Thank for the workaround for "Create Virtual Service NSX-T LB" fails with error"
"Tenant = workflowRuntime.getAviVroClient().cred.tenant;" fixes the NSX-T issue.
/Brian

Hi @mkhachane
Then we have the other issue that both @kskilling and I see:
When using vCenter Cloud (No NSX-T configured)
When running workflow "VS Create" the server_cloud is not published.

Do you have a fix for this as well?
/Brian

Update on this issue:
Several version upgrades since initial creation of Issue 126:
Environment B
vRA/vRO 8.10 3 node cluster:
Avi 22.1.3
vRO Plugin 22.1.3
AVI Configured Cloud: NSX-T Cloud (NSX-T <IP_ADDRESS>)
"Create Virtual Service NSX-T LB" is working now.
I will close Issue 126.
Environment A
vRA/vRO 8.11 single node:
Avi 22.1.3
vRO Plugin 22.1.3
AVI Configured Cloud: VMware vCenter/vSphere ESX
First issue:
Attaching Avi Controller 22.1.3 (Version parameter set to 22.1.3)
Avi endpoint is shown in inventory, but the configured virtual service is not shown:
Second issue:
Trying to "Create VS", the Avi endpoint is shown, but the server_cloud is not shown in the dropdown:
I have opened a new Issue https://github.com/vmware/vrealize-orchestrator-plugin-for-alb/issues/136


Improved widget-3

Closes #961 . Improved widget-3.
Added group tag to parent to change button color when hovered over parent "li" tag.
Screenshots attached:
Before:

After:

Awesome @Dibyendu303


[YOSEGI-7] There is no JIRA link in the README.

What is the necessity of this update? What is updated?
I created a link of JIRA.
How did you do the test? (Not required for updating documents)

LGTM


Use .iter() in a few more for loops. Also uses a better .min and .max for f32

These changes combined give me a repeatable ~3-4% speedup over current main. There's probably another 5% available by rewriting some of the existing for loops as iter chains, but the early returns make it annoying/non trivial.
See the comments in https://github.com/svenstaro/bvh/pull/87 for the f32 asm comparison between std .min() and .max() and the included functions in this pr.

thanks! seeing the same kind of improvement on my laptop


fixed overflow of cardfooter

#54 fixed
i had successfully fixed the overflow of the footer present in the card

#68  closed and completed

you closed issue which i haven't solved yet that was #54 and you closed #68


Use proper repository for rh-mongodb32 in CentOS.

Now it is not necessary to use testing repository.
Fixes #129.

LGTM.

[test]


License?

Hello ,
Is there license for project? There is not on documentation
Sorry my bad English =]
Best,
Toussaint

Hi @toussaintmarier! Thanks for asking!
I've added the license to the repo- it's Apache-2.0.


fix(driver,userspace/libsinsp): use new PPME_CONTAINER_JSON_2_ events with large payload

Signed-off-by: Federico Di Pierro<EMAIL_ADDRESS>
Co-authored-by: Mark Stemm<EMAIL_ADDRESS>
What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area libsinsp
What this PR does / why we need it:
Deny new scap files with PPME_CONTAINER_JSON_2_ events from  be opened by old falco; moreover, new falco is now correctly able to open old PPME_CONTAINER_JSON_ events.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

Hi @FedeDP. Thanks for your PR.
I'm waiting for a falcosecurity member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: FedeDP
To complete the pull request process, please assign leogr after the PR has been reviewed.
You can assign the PR to them by writing /assign @leogr in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/ok-to-test

LGTM label has been added.  Git tree hash: 1ba222e6566b43cf0a6f0bb73a5c0737fb6fad7b

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Put myself on owner list.

Signed-off-by: Mike Ng<EMAIL_ADDRESS>I wish to become one of the maintainers of this project. Thanks. @RainbowMango

/assign RainbowMango

/lgtm
But I'm afraid you might need to be an org member first.

/lgtm But I'm afraid you might need to be an org member first.

Thanks! Requesting access here https://github.com/kubernetes/org/issues/3133

Got it.

Welcome~ Mike
Leave approval to @pmorie
/assign @pmorie

https://github.com/kubernetes-sigs/work-api/pull/15
I am going to sponsor Qiu Jian as the approver instead.


[v1.9] Backport of #19702

Backport of https://github.com/cilium/cilium/pull/19702
$ for pr in 19702; do contrib/backporting/set-labels.py $pr done 1.9; done

/test-backport-1.9


How to safely uninstall Fireshock by ViGEm ?

Hi there. Can someone tell me how I completely uninstall Fireshock?
I basically would like to revert my system back to the time before I installed Fireshock. And no I dont have a System backup. i uninstalled in my Programs already, but there are traces.
for example, i have this folder left: Windows\System32\drivers\UMDF
do i need to completele delete this folder? or is only the fireshock.ddl related to fireshock instead of the whole folder?
I would like to know a safe method to delete all traces of firechock.
EDIT : it is the same with vigembus. i uninstalled it, but there are still a few traces. like this folder here Windows\System32\DriverStore\FileRepository\vigembus.inf_amd64_dc012c700833063e
(i already used sc delete vigembus and deleted vigembus.sys)
can i delete safely the stated folder without anny worries?

Not a helpdesk, there's literally links to the right places scattered all around
Don't mess with Windows directories or you'll get bitten, use the proper tools like DriverStore Explorer

I found these tutorials and I did what it was stated in them. BUt still, there were leftovers and there were no information about these leftovers. Thats why I asked if it is safe to deleted the stated folders/files.

I see, DSE will be your friend there, it uses safe Windows routines to get the job done.


Unsupported Operation Exception when running benchmark from the CLI

Describe the bug
When running benchmark version 0.10 from the CLI with the below command
 bash kcb.sh  --scenario=keycloak.scenario.authentication.AuthorizationCode --server-url=http://localhost:8180 --realm-name=myrealm --username=user-0 --user-password=user-0-password --client-id=MY_CLIENT --client-secret=MY_SECRET--client-redirect-uri=http://localhost --scope=openid profile offlice_access

I received below errors from gatling report
j.l.UnsupportedOperationException: Can't connect to IPv6 remote localhost/[0:0:0:0:0:0:0:1]:8180 + from IPv4 local one /<IP_ADDRESS>:0
j.l.UnsupportedOperationException: Can't connect to IPv6 remote localhost/[0:0:0:0:0:0:0:1]:8180 + from IPv4 local one /<IP_ADDRESS>:0

It worked well on previous benchmark versions.
Version
Keycloak: 22.0.1, benchmark: 0.10-SNAPSHOT
Expected behavior
Works well like previous versions without error
Actual behavior
I received below errors from gatling report
j.l.UnsupportedOperationException: Can't connect to IPv6 remote localhost/[0:0:0:0:0:0:0:1]:8180 + from IPv4 local one /<IP_ADDRESS>:0
j.l.UnsupportedOperationException: Can't connect to IPv6 remote localhost/[0:0:0:0:0:0:0:1]:8180 + from IPv4 local one /<IP_ADDRESS>:0

How to Reproduce?
Running the benchmark version 0.10-SNAPSHOT from the CLI
https://www.keycloak.org/keycloak-benchmark/benchmark-guide/latest/run/running-benchmark-cli
Anything else?
No response

Looking the CLI, the parameters for --scope probably need quoting, so it would read:
--scope="openid profile offlice_access"

lease double-check, I haven't tested this myself yet. Setting the environment variable RUNNER_DEBUG to 1 should log additional information from the bash script to debug if the value is passed correctly.
About the localhost issue: Something seems to be off with IPv6 vs. IPv4 as outlined in the error message. Use the CLI option --log-http-always to show all responses received from Keycloak. It seems that there is some redirect to a port 8180 happening, which seems to be specific for your setup.

It seems this was a side effect introduced in #434 by @ddurham2 when a system doesn't have a public IPv6 interface, but has resolves localhost with an IPv6 address.
https://github.com/gatling/gatling/blob/6dbac4c2740f3200cbefcc7b1f7f36bb2732c8e3/gatling-http-client/src/main/java/io/gatling/http/client/impl/DefaultHttpClient.java#L776-L783
I've pushed a change to wrap the line default = default.useAllLocalAddresses in a new CLI option use-all-local-addresses which should restore the previous default.
Please give it a try and let me know if this works for you again.

Thanks. I tried and confirmed it works well.

Thank you @knighthedspi  for testing this change!


[Bug]: ERR_TOO_MANY_REDIRECTS using Custom HTTPS Domain with WordPress Service

Error Message and Logs
I created a fresh service using the Wordpress Without Database template and I assigned a custom domain name to the service using HTTPS.
After configuring the fresh install and connecting it to my MySQL database, I updated the WordPress Address (URL) under Settings -> General from HTTP to HTTPS. When you attempt to access the wp-admin page, you are presented with an ERR_TOO_MANY_REDIRECTS error.
The rest of the site works except for the admin panel, which is odd. I am using Cloudflare and set the encryption mode to Full (strict) with no luck.
Steps to Reproduce

Create a new WordPress service using a custom HTTPS domain name.
Once the site comes online and you log into the wp-admin portal, the WordPress Address (URL) under Settings > General is set to use the HTTP custom domain and not HTTPS.
Update the WordPress Address (URL)	 to the HTTPS address and click save.
The page will reload, and ERR_TOO_MANY_REDIRECTS will show.
WordPress

Example Repository URL
No response
Coolify Version
v4.0.0-beta.360
Are you using Coolify Cloud?
No (self-hosted)
Operating System and Version (self-hosted)
Ubuntu 24.04
Additional Information
No response

I believe I found a solution to this issue by adding the following to the top of my wp-config.php file.
 * Handle SSL reverse proxy
 */
if ($_SERVER['HTTP_X_FORWARDED_PROTO'] == 'https')
    $_SERVER['HTTPS']='on';

if (isset($_SERVER['HTTP_X_FORWARDED_HOST'])) {
    $_SERVER['HTTP_HOST'] = $_SERVER['HTTP_X_FORWARDED_HOST'];
}

Source: https://github.com/docker-library/wordpress/issues/412#issuecomment-623530572

You are most likely running into this issue: https://coolify.io/docs/knowledge-base/faq#too-many-redirections

You are most likely running into this issue: https://coolify.io/docs/knowledge-base/faq#too-many-redirections

I have it set to Full, same thing is happening. The docs aren't very clear.

You are most likely running into this issue: https://coolify.io/docs/knowledge-base/faq#too-many-redirections

I have it set to Full, same thing is happening. The docs aren't very clear.

Yeah, that did not work for me either. The only thing that worked was the solution that I posted above.
https://github.com/coollabsio/coolify/issues/3903#issuecomment-2408801588


Unable to resolve the name 'Mathworks_QLabs.getInstallationLocation'.

After running QLabs.setup, the following error message has been returned.
Unable to resolve the name 'Mathworks_QLabs.getInstallationLocation'.
Error in QLabs/getDownloadDir (line 364)
downloadDir = Mathworks_QLabs.getInstallationLocation('Quanser Interactive Labs and QUARC Home for MATLAB Simulink');
Error in QLabs.isInstallerPresent (line 218)
installerFilePath = fullfile(QLabs.getDownloadDir,QLabs.getInstallerFileNames());
Error in QLabs.setup (line 41)
if ~QLabs.isInstallerPresent()

An update:
It is required to download and install the toolbox first. Once install, unzip the zip file and run QLabs.setup will resolve the issue. All good now.

An update:
It is required to download and install the toolbox first. Once install, unzip the zip file and run "QLabs.setup" (case sensitive) will resolve the issue. All good now.


How I can implement the feedback feature

Hi folks, nice work on this incredible app.
I need some guidance on how to implement the feedback feature on my Laravel app. I couldn't find any docs about it.
I will really appreciate the help.
Have a great day!

Hey!
We've had it available back in the day when it was paid. We're now refactoring some stuff to make it more fluent and easier to implement. We'll be releasing this very soon including the docs.

@Cannonb4ll That would be great. Thank you very much!

It would be great if they could have this feature working.
Thank you very much for the development of this app.

Any news on the feedback feature ?
thanks, Denis


Added Ribbon analog devices and small syntax change

Added Ribbon analog devices to the table and small syntax change (EdgeMarc)

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@CarolynRowe PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@FilippSe.  This branch has conflicts.  Can you resolve the conflicts?  Or resubmit?

I see. Will resubmit tomorrow and email you.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Carolyn Rowe<EMAIL_ADDRESS>Sent: Tuesday, March 3, 2020 8:58:54 PM
To: MicrosoftDocs/OfficeDocs-SkypeForBusiness<EMAIL_ADDRESS>Cc: Filipp Seljanko<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/OfficeDocs-SkypeForBusiness] Added Ribbon analog devices and small syntax change (#4074)
@FilippSehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FFilippSe&data=02|01|Filipp.Seljanko%40skype.net|16c2913239ee4df366ef08d7bfdf9827|72f988bf86f141af91ab2d7cd011db47|1|0|637188839366463639&sdata=weIdztZ%2Bk3%2FXqmoAIFQ985zubmYLJU91DFxxObJ7sRU%3D&reserved=0. This branch has conflicts. Can you resolve the conflicts? Or resubmit?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-SkypeForBusiness%2Fpull%2F4074%3Femail_source%3Dnotifications%26email_token%3DANLL2A54ZSZ7GUMSOIQPGLLRFWYV5A5CNFSM4K7CNFWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENV7CCI%23issuecomment-594276617&data=02|01|Filipp.Seljanko%40skype.net|16c2913239ee4df366ef08d7bfdf9827|72f988bf86f141af91ab2d7cd011db47|1|0|637188839366463639&sdata=Bj5ObTrHSjdJmrVsz2Z%2BYLJvCw4bI8aZPmmX3osqcMA%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANLL2A7BSOOV4ENFPF5PXA3RFWYV5ANCNFSM4K7CNFWA&data=02|01|Filipp.Seljanko%40skype.net|16c2913239ee4df366ef08d7bfdf9827|72f988bf86f141af91ab2d7cd011db47|1|0|637188839366473627&sdata=cof211v3C1lLDlSjORT%2ByNA0E%2BJ6NwQCZj8lfRIV2iA%3D&reserved=0.

This was handled in another PR.  Closing this one.


Add kube-proxy non-privileged blog post

The post describes how the --init-only flag can be used to do privileged initialization in an initContainer, and run the main container with NET_ADMIN capabilities only

/sig network

What is the practice about the commits accepted from a PR review? My intention is to squash them, but keep the "Co-authored-by:" part.

One thing I'd mention is that the narrowed kube-proxy Pod still only meets the privileged Pod security standard. There's still an improvement because the running container doesn't need to run privileged.

@sftim
I must admit I wasn't aware of the Pod security standard until I read the User Namespaces blog yesterday. I will try to mention it in some way.

One thing I'd mention is that the narrowed kube-proxy Pod still only meets the privileged Pod security standard. There's still an improvement because the running container doesn't need to run privileged.

In theory you could run two separate pods, though that would be annoying in various ways (especially that there's no such thing as a run-once DaemonSet)...
We should figure out what can be done to make the situation better...

@uablrek you should add something somewhere about this being Linux-only / not for Windows. (Maybe explaining that it's because k8s on Windows doesn't have the equivalent of capabilities, so there's no way to give the pod just a little bit of privilege).

Following on from https://github.com/kubernetes/website/pull/43680/files#r1373331022
Another option: write a blog for https://k8s.dev/ that is aimed at a more informed audience.
If you write an article that is aimed at cluster lifecycle tool authors, you don't have to include a sample manifest marked as kind: DaemonSet, just some snippets.

The flip side though is that if we are writing on the Kubernetes blog, we need to write for an audience that includes people who'll copy and paste what we put here and then file an issue if it doesn't work. I want to control for that risk before the stream of issues turns up.

A complete updated manifest would be something like in https://github.com/kubernetes/kubernetes/pull/120864#issuecomment-1734841738. In your example, the volume mounts seem to be missing. If anybody do a copy/paste on those, nothing would work.
@danwinship proposed a blog in https://github.com/kubernetes/kubernetes/pull/120864#issuecomment-1753433220,
but this may not be a post with a suitable level for Kubernetes blog.
Perhaps I should close this PR, and a better option would be that a kubeadm maintainer write a blog post when this feature is supported by kubeadm?

Perhaps I should close this PR, and a better option would be that a kubeadm maintainer write a blog post when this feature is supported by kubeadm?

It's good to announce the feature at around the time we add it. If you're willing to shift this writeup to https://k8s.dev/blog - https://github.com/kubernetes/contributor-site - then we don't have to worry much about people expecting working YAML. The contributor blog has fewer readers and different expectations.
So that's one good option; another good option is to explain how to set up kube-proxy the new way. That kind of detail is an unmet need: we tell people that kubeadm can set things up for them, but if they want to learn how to do it by hand they are largely on their own (I can explain why - maybe on Slack? - if you'd like more context).
The trouble is that if we make a blog article that looks like advice on deploying kube-proxy, I really think it needs to explain what readers are hoping to find. Even if that isn't the same as what we / you had set out to write.
You can still pick, but from a smaller set of choices than you might like. I'd prefer an article that explains kube-proxy setup for Linux the new recommended way. If that's not on the cards, then an article on https://k8s.dev/blog feels like a good second place option.

I will move to https://k8s.dev/blog. I think it requires more understanding than copy/paste to update the kube-proxy manifest and get it right, even if a complete manifest is given as an example. It isn't really hard to do one time in KinD for example, but to maintain it in a live cluster, without help from kubeadm for example, requires skill.
I think it's better to keep this for experienced users and developers of installation tools for now. It can be brought to a broader audience when support in kubeadm exists.

This PR is moved to https://github.com/kubernetes/contributor-site/pull/452

@sftim Thanks a lot for your input. I have already received a query on that image to use for this, so the copy/paste syndrome is very real. I wasn't aware, and I guess you would be the one taking the hit :smile:


Compile errors on armhf

Trying to build on a Raspberry Pi 3B+, running up to date Debian Buster (Raspbian), clang-8 from Debian backports.
$ uname -a
Linux raspberrypi 4.19.118-v7+ #1311 SMP Mon Apr 27 14:21:24 BST 2020 armv7l GNU/Linux

Environment:
export CC=clang-8
export CXX=clang++-8
export PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
export RUNTIME_VERSION=gnustep-2.0
export PKG_CONFIG_PATH=/usr/local/lib/pkgconfig
export LD=/usr/bin/ld.gold
export LDFLAGS="-fuse-ld=/usr/bin/ld.gold -L/usr/local/lib -I/usr/local/include"
export OBJCFLAGS="-fblocks"

libobjc2 configured with:
cmake .. -DCMAKE_C_COMPILER=${CC} \
-DCMAKE_CXX_COMPILER=${CXX} \
-DCMAKE_BUILD_TYPE=Release \
-DUSE_GOLD_LINKER=YES

Throws this compilation error 8 times:
pi@raspberrypi:~/Documents/gnustep-build/debian-10-clang-8.0/libobjc2/build $ make -j2
[  1%] Building C object CMakeFiles/objc.dir/eh_personality.c.o
[  1%] Building C object CMakeFiles/objc-static.dir/eh_personality.c.o

/home/pi/Documents/gnustep-build/debian-10-clang-8.0/libobjc2/unwind-arm.h:208:54: note: expanded from macro 'COPY_EXCEPTION'
  memcpy((dst)->cleanup_cache, (src)->cleanup_cache, sizeof(struct cleanup_cache)); \
                                                     ^     ~~~~~~~~~~~~~~~~~~~~~~
/home/pi/Documents/gnustep-build/debian-10-clang-8.0/libobjc2/eh_personality.c:571:3: note: forward declaration of 'struct cleanup_cache'
                COPY_EXCEPTION(ex->cxx_exception, exceptionObject)
                ^
/home/pi/Documents/gnustep-build/debian-10-clang-8.0/libobjc2/unwind-arm.h:208:68: note: expanded from macro 'COPY_EXCEPTION'
  memcpy((dst)->cleanup_cache, (src)->cleanup_cache, sizeof(struct cleanup_cache)); \
                                                                   ^
/home/pi/Documents/gnustep-build/debian-10-clang-8.0/libobjc2/eh_personality.c:575:3: error: invalid application of 'sizeof' to an incomplete type
      'struct pr_cache'
                COPY_EXCEPTION(exceptionObject, ex->cxx_exception)

@triplef, I thought #163 was meant to fix this, can you take a look?

Sorry for the delay, I just opened a PR that should fix this. No idea why my previous change seemingly worked before for me...

Took me some while to come back to this. But I can confirm this is fixed. Compiles, installs, works for Linux armhf now.


Remove obsolete code from heat assembly

Postprocess-function is deprecated and not used anywhere.

Coverage increased (+0.6%) to 91.04% when pulling 1d79b1adeb66278481cc3cfe052f5f67b3080737 on remove_old_code into dc75cfda0363acfafbd2d3b000ff5158c8661f86 on master.


Add GitHub Actions deployment workflow for deploy_dev

@facing-n Adding worfklow for dev, had minor tweaks to the README during testing.

@facing-n once this gets approved I'll merge and proceed with rolling this out for dev.

@facing-n updated.


ci: do not include go-ceph in generic GitHub package rebases

go-ceph is an important package that we consume.  It is better to have that as separate rebase PR from Dependabot and not include it in the general GitHub package group.

Show available bot commands
These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

/retest ci/centos/<job-name>: retest the <job-name> after unrelated
failure (please report the failure too!)

@mergifyio queue


Improve card UI style

close #672
Hi @panwar8279 , I've Improve the style of card and fix alignment issue, please check

Please Create new PRs for this because it contains conflict issue.


jwt_token error resolved and err.response checks included in action

@root5533 @ivantha
Resolved two errors in the authentication code
1. Added check for err.response
2. token varialble declaration and wrongly assigned bug fixed as
const token = res.data.access_token

Fixes #194

@ivantha Please review!
Thanks :slightly_smiling_face:

@ivantha @Anmolbansal1 Done!


Memory Leaks - Instance.cpp

Hi,
in Instance.cpp an exception can be thrown, as a result the  "delete PScanner;" will not be called
adding try...catch...finally... or try...finally... should fix the issue
in case of an exception the YSScanner is not disposed
	List<YSMatches^>^ YSInstance::ScanProcess(int pID, YSRules^ rules, Dictionary<String^, Object^>^ externalVariables, int timeout)
	{
		YSScanner^ PScanner = gcnew YSScanner(rules, externalVariables, timeout);
		try{
			List<YSMatches^>^ results = PScanner->ScanProcess(pID);
			return results;
		}
		catch (System::Exception^ e){
			throw e;
		}
		finally{
			delete PScanner;
		}

	}

I apologize for such a big delay in response.
I will investigate this in the coming weekdays

Seems reasonable. Will fix that on Process & Memory & File methods in 1.3.1


Add reference architecture document for R

I decided against doing something with Azure Databricks here, mostly because I felt it would overcomplicate the document. We could potentially have a reference architecture document for Azure Databricks, but I think it should use something better supported like Python, Java or just SQL notebooks.

@infoverload Thanks for the review! I've pushed a fixup commit that should address your comments.


update github links after repository migration

Motivation/Description of the PR

Description of this PR, which problem it solves
Resolves #issueId (if applicable).
issue #2441

Applicable helpers:

[ ] WebDriver
[ ] Puppeteer
[ ] Nightmare
[ ] REST
[ ] FileHelper
[ ] Appium
[ ] Protractor
[ ] TestCafe
[ ] Playwright

Applicable plugins:

[ ] allure
[ ] autoDelay
[ ] autoLogin
[ ] customLocator
[ ] pauseOnFail
[ ] puppeteerCoverage
[ ] retryFailedStep
[ ] screenshotOnFail
[ ] selenoid
[ ] stepByStepReport
[ ] wdio

Type of change

[ ] :fire: Breaking changes
[ ] :rocket: New functionality
[ ] :bug: Bug fix
[ ] :clipboard: Documentation changes/updates
[ ] :hotsprings: Hot fix
[ ] :hammer: Markdown files fix - not related to source code
[ ] :nail_care: Polish code

Checklist:

[] Tests have been added
[x] Documentation has been added (Run npm run docs)
[ ] Lint checking (Run npm run lint)
[x] Local tests are passed (Run npm test)

Thanks!


Error in com.google.fonts/check/glyph_coverage with font missing only unencoded glyphs

My WIP font currently has all glyphs from GF Latin Core that have Unicode codepoints, but not the unencoded ones like periodcentered.loclCAT, etc.
Observed behaviour
Console output:
 >> Check:   com.google.fonts/check/glyph_coverage
    Desc:    Check Google Fonts glyph coverage.
    Files:   LibreDotMatrix-Regular.ttf

    ERROR Failed with IndexError: list index out of range
          ↳ File
          "C:\Users\neilb\AppData\Local\Programs\Python\Python310\lib\site-packa
          ges\fontbakery\checkrunner.py", line 194, in _exec_check
          for sub_result in result:  # Might raise.
          ↳ File
          "C:\Users\neilb\AppData\Local\Programs\Python\Python310\lib\site-packa
          ges\fontbakery\profiles\googlefonts.py", line 1068, in
          com_google_fonts_check_glyph_coverage
          f"{bullet_list(config, missing)}")
          ↳ File
          "C:\Users\neilb\AppData\Local\Programs\Python\Python310\lib\site-packa
          ges\fontbakery\utils.py", line 215, in bullet_list
          pretty_print_list(config,
          ↳ File
          "C:\Users\neilb\AppData\Local\Programs\Python\Python310\lib\site-packa
          ges\fontbakery\utils.py", line 210, in pretty_print_list
          return f"{joined_items_str} {glue} {str(values[-1])}"
    Result: ERROR

Expected behaviour
It seems that the font was correctly identified as missing glyphs from the set, but pretty_print_list is attempting to index an empty list here, as that list only ever contains encoded glyphs.
Perhaps com_google_fonts_check_glyph_coverage should be changed to report unencoded glyphs as well as encoded ones.
Resources and exact process needed to replicate
This could probably be reproduced with any font that covers GF Latin Core, minus one or more of the unencoded glyphs. Here are the fonts which produced the error for me.

I believe that this was fixed 3 days ago by PR #3833
If that's the case, then you'll either want to wait for the next release (v0.8.10) or use fontbakery from git main branch.
Please, let me know if we should re-open this issue in case you have evidence that there are still problems even after the 3833 fix.


👌📚 Use all-contributors

First issue to integrate all-contributors

@all-contributors please add @s-weigand for code ideas maintenance projectManagement infra test doc

@all-contributors please add @s-weigand for code ideas maintenance projectManagement infra test doc


ign -> gz Namespace Migration : gz-gui

See https://github.com/gazebo-tooling/release-tools/issues/711

#406 is merged, this PR needs a merge / rebase

Rebased!

@osrf-jenkins run tests please!


Reuters API stopped updating

The /v8 endpoint stopped updating on 11/28 (See https://github.com/RSS-Bridge/rss-bridge/issues/1618). I assume they migrated to a new Url scheme.

https://www.reuters.com/arc/outboundfeeds/news-sitemap/?outputType=xml looks promising

https://www.reuters.com/arc/outboundfeeds/v4/mobile/section/ Try this

If I replace $wire_url = 'https://wireapi.reuters.com/v8'; with $wire_url = 'https://www.reuters.com/arc/outboundfeeds/v4';, it works locally but I get a new error when trying to use it at my usual hosted location:
Details
Type: HttpException
Code: 401
Message: https://www.reuters.com/pf/api/v3/content/fetch/article-by-id-or-url-v1?query={"website":"reuters","website_url":"\/world\/us\/how-history-race-politics-divide-tennessee-gun-legislation-2023-12-28\/"} resulted in 401 Unauthorized
File: lib/http.php
Line: 26

again, this URL works in my browser but my rss-bridge instance is blocked. Reuters is actively squashing people trying to use their RSS feeds.

i think is fixed. let know otherwise


torch 1.12.1 does not work with intel ccl 1.12.0

Signed-off-by: Wang, Yi A<EMAIL_ADDRESS>

@sgugger @yao-matrix same with the PR with transformer

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Same comment as in https://github.com/huggingface/transformers/pull/19132
Those kinds of checks belong in your libraries directly, not Accelerate or Transformers as we can't constantly add/remove checks each time there is a new release (also PyTorch 1.12.1 was out a month and a half ago...)

@sgugger we will release a new oneCCL 1.12.1 to work with torch 1.12.1


08/22/2018 18:32 : Create Pull Request for branch master -> prTesting1_2018082218

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Inquiries about using Peta Linux

Thank you for the great article! I want to connect to the Internet by connecting a Wi-Fi dongle to the installed Peta Linux, but the iwconfig command does not exist, and even the normal sudo apt-get update does not work. What should I do in PetaLinux??  If you have any good articles on how to use Peta Linux, please recommend them please.

Hello!
Sorry for the late response. To connect to the internet, I suspect that the current SD card image should work.
In any case, you should try to look on this previous issue where I also discussed the possibilities to connect the board to the internet.
If any of that does not work, please tell me. At least I can warn the users for that problem.
I do not have any experience with Petalinux but surely there is something that can be done there to support internet connection if it does not work.
Good luck! :)

Hi!
I will be closing this issue for inactivity. If you want to develop any other question be sure to open a new issue.
Thank you for understanding! :)


Error in name of cell_format in paint_box() in ExcelPainter module.

Complex logic cell format missing '-' for middle row counts net...
i.e.
cell_format = cell_format + 'mrowN-NET'

should be
cell_format = cell_format + 'mrow-N-NET'

Resolved by #128.


chore(main): release utils 1.13.0

:robot: I have created a release beep boop
1.13.0 (2023-02-13)
Features

loadDependencies arg to exclude dev deps (5624aed)
loadDependencies now returns package tree (d31c542)
add shared functionality (6722e33)
forward events to Inspector (1862309)
get local package sizes (7acd48c)
graph generation (f6c9f4e)
initial release (84442e0)
make all methods support both promise&async (4facc94)
more standard graph node properties (a649f79)
support registry metadata collection (9d4c194)
support stringified license data (e1b8838)
update Sandworm to v1.3.1 (63da401)
update Sandworm to v1.3.2 (a77c1a9)

Bug Fixes

loadDependencies returns unique items (77bf47a)
loadSandworm config option name (de05544)
better deprecated manifest license handling (df50b52)
crash when post-processing empty graphs (94d35dd)
crash when using pnpm with no dependencies (b9f64f8)
json license data parsing (9a9779e)
package size no longer includes node modules (3f4e463)
remove disconnected nodes from all packages (192315e)
support older, non-standard manifest licenses (7504cb4)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/sandworm-hq/sandworm-utils/releases/tag/utils-v1.13.0 :sunflower:


RuntimeError: CUDA error: Out of Memory

When I launch:
autotrain llm --train --project_name my-llm --model meta-llama/Llama-2-7b-hf --data_path . --use_peft --use_int4 --learning_rate 2e-4 --train_batch_size 1 --num_train_epochs 1 --trainer sft
I have CUDA Out of Memory Error:
Traceback (most recent call last):
File "/home/llamaFineTune/myenv/bin/autotrain", line 8, in
sys.exit(main())
File "/home/llamaFineTune/myenv/lib/python3.10/site-packages/autotrain/cli/autotrain.py", line 36, in main
command.run()
File "/home/llamaFineTune/myenv/lib/python3.10/site-packages/autotrain/cli/run_llm.py", line 489, in run
train_llm(params)
File "/home/llamaFineTune/myenv/lib/python3.10/site-packages/autotrain/trainers/clm.py", line 105, in train
model = AutoModelForCausalLM.from_pretrained(
File "/home/amg/llamaFineTune/myenv/lib/python3.10/site-packages/transformers/models/auto/auto_factory.py", line 511, in from_pretrained
return model_class.from_pretrained(
File "/home/llamaFineTune/myenv/lib/python3.10/site-packages/transformers/modeling_utils.py", line 2859, in from_pretrained
max_memory = get_balanced_memory(
File "/home/llamaFineTune/myenv/lib/python3.10/site-packages/accelerate/utils/modeling.py", line 731, in get_balanced_memory
max_memory = get_max_memory(max_memory)
File "/home/llamaFineTune/myenv/lib/python3.10/site-packages/accelerate/utils/modeling.py", line 624, in get_max_memory
_ = torch.tensor([0], device=i)
RuntimeError: CUDA error: out of memory
CUDA kernel errors might be asynchronously reported at some other API call, so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.
Compile with TORCH_USE_CUDA_DSA to enable device-side assertions.
My NVIDIA GPU is NVIDIA GeForce RTX 3060 Laptop GPU 6Gb.
Any help would be appreciated.
Thanks in advance.

The Llama 2 7B model is around ~14gb in size. You may want to try using google colab.


Add alt tag to maker shadow

There is an old PR (https://github.com/Leaflet/Leaflet/pull/2112) that addressed alt text for icons. However in order to pass accessibility tests the same needs to happen with maker shadows.
I'm wondering if it's ok just set a default alt text for icons.
How to reproduce

Leaflet version:  1.0.2
Browser: Chrome 55
OS/Platform (with version) I'm using: MacOS Sierra
Create a map with default markers
Run wave accessibility tool or similar (https://chrome.google.com/webstore/detail/wave-evaluation-tool/jbbplnpkjmmeebjpijfedlgcdilocofh)
A list of errors related with missing alt text in makers are displayed

Acceptance criteria
Like marker icons, marker shadows should have alternative text

I think marker shadow should have alt="" rather than copying the alt of the main marker, since it's a purely presentational element without semantical meaning, and should be treated as such by screen readers.

@mourner I've created a PR that does that https://github.com/Leaflet/Leaflet/pull/5259

I agree with @mourner, researching a bit about the best accessibility practices for alt tags in "decorative" images, I found many recommendations to leave the tag empty, if only to avoid clutter for assistive tools like screen readers.
For example, there's the Web Accessibility Initiative recommendations for decorative images, which provides some examples as well.

@mourner @luiscarloscb I've updated the PR to leave shadow's alternative text attribute empty.


Option to add numberOfLines inside header in navigationOptions

Motivation
Inside my app I can't always know the title, and sometimes the title can be 4-5 words long. Since I don't want cropping in my header titles, i can make the size of the header and the font of the title change dynamically but I cant change the numberOfLines attribute.
This forces me to drop the font size very low if I want to fit a long title in one line. Enabling the option to manually set the numberOfLines gives me the ability to combine changing the header height, title fontSize and numberOfLines and make my header look and behave in a more flexible way.
Example
Here is an example of the default navigationOptions with long title:

and the code:
 static navigationOptions = {
	header : (options) => ({
		right: (<OKButton onPress={() => {}}/>),
		left: (<CancelButton onPress={() => options.state.params.onCloseModal()}/>),
		style: {
			backgroundColor : Colors.navigationBarColor
		},
		tintColor: Colors.defaultTextColor,
		titleStyle: {
			paddingLeft: 20
		}
	}),
	title : 'A Somewhat Long Title That Could Not Fit In One Line Without My Change'
}

and the same navigationOptions with numberOfLines option:

and the code:
 static navigationOptions = {
	header : (options) => ({
		right: (<OKButton onPress={() => {}}/>),
		left: (<CancelButton onPress={() => options.state.params.onCloseModal()}/>),
		style: {
			height: 100,
			backgroundColor : Colors.navigationBarColor
		},
		tintColor: Colors.defaultTextColor,
		numberOfLines: 3,  // <------------------------------------ THE CHANGE
		titleStyle: {
			paddingLeft: 20
		}
	}),
	title : 'A Somewhat Long Title That Now Sits In Three Lines Because Of My Change'
}

What happens during transitions between screens with different header heights?

The header changes as well, according to the values you provide.

Have you tested on iOS? What does it look like when navigating between a screen with different header heights there? What happens when swiping back?

Please post a screenshot/gif of the results on iOS

@ericvicenti Please forgive the uglyness of this screencast. I don't have access to my computer at the moment so I made a real quick demo with create-react-native-app and a trial app on a friends iPhone/Mac.
I'll try to provide a better looking example with my initial project wen i get back to office.
Thanks for your time

Any news on this one? thanks

+100 , i need this feature in my app ;)

That jump is pretty ugly.

@satya164  Yeap I totally aggree! But what I'm proposing is something optional. I'm not saying it should be done like this everywhere. But having the option, we can make for example a modal that has a bigger header in which case ugly transition is not a problem.
What I'm proposing doesn't change the state of the current header. It just adds the extra flexibility some of us need for whatever good or bad reasons.

In case of a modal you can use a custom header anyways if you are not concerned about ugly animation. But you should really rethink the design, because animation and the header look pretty ugly. Adding this to the library even as an option means make it super easy to create ugly designs, which I'm not a fan of.

Fair enough. :)
(You may close it then i guess)


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #358 (92e21cc) into main (2974dfe) will increase coverage by 12.36%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##             main     #358       +/-   ##
===========================================
+ Coverage   83.33%   95.69%   +12.36%
===========================================
  Files           2        1        -1
  Lines         126      395      +269
  Branches       31       53       +22
===========================================
+ Hits          105      378      +273
+ Misses         21       17        -4

Impacted Files
Coverage Δ

src/index.ts
95.69% <100.00%> (+13.69%)
:arrow_up:

example/src/main.ts

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 6952090...92e21cc. Read the comment docs.


fix mjs module export

Thank you. The library has been updated to d.js 14.7.1, as they made some ComandInteraction properties private that needed to be passed via the constructor instead, and module export fixed. The previous release of this library was based on a slightly older version.


Add SEA Equipment and Gun racks

About the PR

Adds the SEA vendors with minor additions that deviate from parity:
*Adds Officer's Jacket
*Adds MP Security Glasses
Removed Alpha Headset from SEATech Vendor
Media

Requirements

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

add: Added SEA Equipment and Gun Racks!

These are the wrong types of vendors, this pr will not be merged half finished, but thank you for doing the fills.
Also please remove the officer jacket as it's not part of their uniform for the sake of recognizability.
Sec glasses can stay.

do you want the kind that have limited selection? or what kind should i be using here?
How can i improve it.

do you want the kind that have limited selection? or what kind should i be using here? How can i improve it.

It should be the limited selection yes, all SEA players can access these vendors and get equipment from them, they work the same as the rifleman armor vendor when done correctly.
see Resources/Prototypes/_RMC14/Entities/Structures/Machines/Vending/Squad/rifleman.yml for how to implement it correctly


mobile: add knob for h3 keepalive

Commit Message: add a config knob to Java, Kotlin and C++ engines to set initial interval of QUIC keepalive probing.
Additional Description: also adjust the validation rule of initial_interval to be larger than 1ms instead of 1s.
Risk Level: low, interface change
Testing: unit test
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A

test failures look legit though, e.g. https://github.com/envoyproxy/envoy/actions/runs/11369001000/job/31625515074

/assign @alyssawilk
since it contains core code, Alyssa should merge

/retest

/retest

/retest

/assign @envoyproxy/api-shepherds

@alyssawilk and @abeyad, PTAL again!


group clients by job?

Greetings,
First of all ,thank you for your effort and making this an open-source project.
Is it possible to group clients by "job" as explained here : https://prometheus.io/docs/concepts/jobs_instances/
This is because my clients belong to different services like "api-service" , "kafka-connect","spark"..etc

That's a matter of service discovery and relabelling on the Prometheus end, which is no different when using PushProx.


Tutorial 3. An example of  billion-scale clustering

An example of billion-scale clustering (sift1b), which contains:

Download the SIFT1B dataset
Encode billion-scale data iteratively
Run clustering

Extreamly SImple! Great!
I'll try it.

With #8 fix, finally, I was able to run this python example with my Macbook. Great!


Disable the module based on condition

I need to set the condition for enabling or disabling the module based on a certain value. For example, if I use the count, the rules are destroyed and recreated on every terraform run. Can we add some variable "enabled" to the module to make life easy?

@tirth-unix apologies for the delay! Was curious if there would be an issue with just commenting out the module code and destroying/recreating the resources as needed.

Closing as stale


Test: multi-cursor inline completions

Refs: https://github.com/microsoft/vscode/issues/201308

[x] anyOS @roblourens
[ ] anyOS @andreamah

Complexity: 3
Create Issue

This iteration we have made Copilot inline completions (ghost text) compatible with multiple cursor positions. Previously the inline completions were visible and applied only at the primary cursor position. Generally speaking, the inline completions that are inserted at the secondary cursors correspond to the inline completions to the right of the primary cursor. This means that if there are inline completions or code/text to the left of the primary cursor, this is not inserted at the secondary cursor positions.‡
To test, please follow the following steps:

Install and enable the Copilot extension. Enable the setting github.copilot.editor.enableAutoCompletions.
Open a file and place multiple cursors at different positions
Write some code/text until you see ghost text appear
Verify that the ghost text appears at all cursor positions
Click on the Tab key. Verify that the whole suggestion is accepted and inserted as expected into the editor.
Repeat steps 3 and 4. Click on cmd+arrow (if on Mac) or ctrl+arrow (if on Windows). Verify that you partially accept the inline completion at all cursor positions.
Repeat steps 1 to 6 so that the cursor positions are not on the same column index. In the image below for example the primary cursor is located after for ( on line 22 and the secondary cursor is located after for on line 20. Verify that the inline completions are displayed and inserted correctly given the point ‡ above.

Is it expected that I can only get that completion toolbar to show up on the completion for the primary cursor location?

Hi @roblourens yes so this is something we have not yet developed. As you correctly mentioned the toolbar only appears at the primary cursor location, but we will aim to make it appear at the others.

Worked great for me, good work! 👍


Implement Agent Kit ☂

This is an umbrella issue to track work on the Agent Kit: a kit of nodes that are a bit more higher-level than core, json, templates, etc.
The nodes in Agent Kit are designed to enable creating autonomous and semi-autonomous agents through simple composition.
Some ideas for nodes:

[x] worker -- an LLM with an instruction to do something
[x] repeater -- loop over a worker (or a set of workers) until a condition is met
[ ] multiplier -- launch N workers in parallel to and return all of their contexts
[ ] reducer -- turn N contexts into a single context
[ ] resetContext -- reset context (flags might be: last -- just the last interaction, first and last -- the original question and the last result, etc.)
[x] structuredWorker -- a worker that takes in a JSON schema and returns results that  conform to that schema

Declaring victory 🚢


introduction of fully automated continous test (CT)

Continuous test shall be executed as part of the pipelines for all repositories using test-trigger
in case of dev/rel tag the test result shall be published in the WebApp.
Bosch internal WebApp instance existing.
OSS WebApp to be setup.

Hi Thomas,
Please refer the current status of this issue:

Change in OSS has been merged.
The BIOS pipeline is nearly completed (download built artifacts, AIO installations are fine) now with some adaptions for executing aio-testtrigger properly.

Thank you,
Ngoan

CT successfully introduced with 0.7.0


Possibility to make sweeping faster?

Would it be possible to make sweeping the district faster?

Sweep times are only limited by how fast you can click. This plugin will not click for you or otherwise request data from the server in a manner outside of clicking on each district.

Ohh alright thank you


Giving Error "unrecognized arguments:  "  in setup.cmd of Lab -23

In Lab 23-custom-search-skill, of file setup.cmd,
where it creates the azure storage, create blob container, then upload the files , creates azure cognitive services and azure cognitive search services.
But after running the setup.cmd, when I cross checked it in the portal, I found that it is failing in creating the blob container, and thus unable to upload the files.
I think there is a problem in setting up the azure storage key in the following below line in setup.cmd:-
set key_string=!key_json:[ { "keyName": "key1", "permissions": "Full", "value": "=! set AZURE_STORAGE_KEY=!key_string:" } ]=! call az storage container create --account-name ai102str!unique_id! --name margies --public-access blob --auth-mode key --account-key %AZURE_STORAGE_KEY% --output none call az storage blob upload-batch -d margies -s data --account-name ai102str!unique_id! --auth-mode key --account-key %AZURE_STORAGE_KEY%  --output none
Please refer the screenshot for the above error.

@drguthals , can you take a look?

One point is that you seem to be in a PowerShell console - the labs should be run in the provided VM in the Bash console

The labs are designed to be used either in the free online courses on Microsoft Learn or in the classroom-based course delivered by Microsoft learning partners - both of these include a hosted VM environment for the labs.. There are links at https://microsoftlearning.github.io/AI-102-AIEngineer/.
If you're using your own environment, you could try just switching the console type to a Windows command prompt instead of the PowerShell console and try that; but again, we don't really support your own environment - the labs are designed specifically for the hosted VM

The labs are designed to be used either in the free online courses on Microsoft Learn or in the classroom-based course delivered by Microsoft learning partners - both of these include a hosted VM environment for the labs.. There are links at https://microsoftlearning.github.io/AI-102-AIEngineer/.
If you're using your own environment, you could try just switching the console type to a Windows command prompt instead of the PowerShell console and try that; but again, we don't really support your own environment - the labs are designed specifically for the hosted VM

Yes, Sir I Followed the instructions.


A TaskCancellationException is thrown instead of a TimeoutException when a hash cannot be resolved in time

Short version:
When a hash cannot be resolved in time,
either because of a slow internet connection  or because the it's no
longer being hosted by any peer the library throws a TaskCancellationException.
I think it would be better if a TimeoutException is thrown instead because
that's what really happened internally.

Details:
In order to reproduce the issue, please consider the following example
which tries to Cat a hash that is no longer hosted:
var path = "QmWZGoYKxUCC7xeThZpttcCwwgmWDpbKNTaCZzY1vrY2PS";
var client = new IpfsClient();
await client.Cat(path);

Or an even faster way to reproduce it, which clearly shows the TimeoutException being
swallowed:
var path = "QmWZGoYKxUCC7xeThZpttcCwwgmWDpbKNTaCZzY1vrY2PS";
var host = new Uri("http://<IP_ADDRESS>:5001");
var httpClient = new HttpClient() { Timeout = TimeSpan.FromSeconds(1) };
var client = new IpfsClient(host, httpClient);
await client.Cat(path);

Both samples throw a TaskCancellationException instead of a TimeoutException.
Aside from having the TimeoutException thrown it would also be great if we
could tell the IpfsClient how long to wait before timing out (or even wait forever) without
having to create an HttpClient manually like so:
var httpClient = new HttpClient() { Timeout = Timeout.InfiniteTimeSpan };

Hi @timothyparez, thanks for the suggestion. In both cases I'm not too keen on the design. It seems like it would just hide details from the user. (eg. Taking a TimeSpan in the constructor only to set it in the HttpClient internally, or rethrowing an exception and hiding the stacktrace).
Thanks,
-TD

Hi,
Then aside from setting the TimeSpan in the constructor,
the fact that it throws a TaskCancellationException is rather confusing,
because at the core it's really caused by a Timeout and that's nowhere to be found in
the StackTrace. (At least I couldn't find it anywhere, so perhaps I'm missing something?)
Because of this, as a consumer of the library, I had to go digging into the source code to
confirm my suspicions. And what if one day you want to give the user the ability to actually
cancel a Get command, (since it may take a very long time to complete and the user might
not want to wait), how would the consumer differentiate between the two?
If you don't mind, please leave this issue open a little while longer,
even if you are 100% correct, I'd like to be able to follow your _reasoning.
PS: I think the working you are doing here is of great value, so thank you for that.

Hi @timothyparez, I'm more partial to the suggestion of throwing a TimeoutException instead of a TaskCancellationException, but I was under the impression that rethrowing exceptions is considered bad practice. I'm sure the .NET developers have a good reason to throw a TaskCancellationException instead of a TimeoutException, even though the latter might make more sense in the context of this library. But I do agree that there should be a way to cancel requests, and that is something to look to add in the future.
I will have a play around with some ideas over the weekend.

I really do not want you to rethrow an exception, I'm in the "that's evil camp" as well.
At first, when I submitted the issue, I figured you actually caught a TimeoutException somewhere
and it got swallowed, but it turns out you didn't.
I'm digging deeper in the code and it turns out the TaskCancellationException is thrown
by the HttpClient.GetAsync() implementation in the framework. And it seems there are a number
of people on StackOverflow who have questions about it as well.
(https://stackoverflow.com/questions/12666922/distinguish-timeout-from-user-cancellation?noredirect=1&lq=1)
At first glance the key 'may' be to provide your own CancellationToken.
(Can't verify now since I'm calling it a day).

I'm sure the .NET developers have a good reason to throw a TaskCancellationException instead of a TimeoutException

Might be something that slipped under the radar while designing it :-)

Been using the library for a while longer now and I find that to much
information is being hidden. For example I consistently fail to issue an ipfs get
on certain hashes using the library (HTTP code 500 again) but when using the command
line it works just fine.
Because the exception just bubbles up from the HttpClient implementation,
there is no further information and that makes it impossible to debug the issue.
No matter what goes wrong, all we get is code 500, no info as to why.
Most of the time rethrowing exception is bad, but I'm starting to wonder, in this
case, if properly analyzing the HTTP responses and throwing custom exceptions
might actually be a better solution.

Could you take a look at the latest commit in my fork.
Proof-of-concept cancellation support:
https://github.com/timothyparez/net-ipfs-api/commit/5d4e5e95481e0d3bd816ebb7fd15aaac029d6f06
If this is something you can live with, I might make some time to implement it throughout the library

@timothyparez Looks good! Could you add a some unit tests and an example to show how it can be useful in the README?
var cancellationToken = ... ;
var catResult = ipfs.Cat("Qm...");
await Task.Delay(1000); // Mimic ipfs.Cat hanging for a long time
cancellationToken.Cancel();

Just got back from holiday, will look into that asap

I've added it and created a pull request https://github.com/TrekDev/net-ipfs-api/pull/12
You'll have to forgive me if I messed up the branching a bit, I'm not a frequent GIT user (used to HG),
and apparently I was doing the work in my "master" branch which I then merged into my "develop" branch.


Fixed "Outline Becomes Off Center in Build" by enable "Read/Write"

Hi,
I faced the same problem "Outline Becomes Off Center in Build" as the previous guy.
but after I select the model file (.fbx)  and goto "Model" import setting in inspctor,  turn on "Read/Write Enabled", it fixed this problem for me.
just want to share if it could help someone's problem too.

@bodinmon Thanks, although when I click the Mesh in assets I can see "Read/Write Enabled: False" but can't change it, any idea how I can change it?


fix map with quoted_num

This PR fixes the problem to use quoted_num with map.

Excellent, thanks for this fix!


w5500: does not take care of the local-mac-address

Currently the W5500 driver does not take care of the local-mac-address. Instead the MAC address is randomized, which cause issues on mender side because we use the MAC address to authenticate the device.
A temporary patch is done in src/main.c line 175 and following to have a static MAC address.
This should be to remove when https://github.com/zephyrproject-rtos/zephyr/pull/48763 will be merged.

The fix has been integrated in the main branch of the zephyr project. Issue will be removed in the readme of this example soon. It is recommended to work with the latest commit ID of Zephyr (da633c614807349090ab3a92e18f041b4046cff8 today).

This issue remains open until the next Zephyr release will be published, then readme will be updated.

The issue is solved on upcoming v3.4. First release candidate is already available on Zephyr project.


Automates brew formula version bump for new release

Automates the process of updating the Brew formula version in the tap repository (ppc64le-cloud/homebrew-pvsadm) for each platform-dependent formula (linux_amd64, darwin_arm64, darwin_amd64) to reflect the latest release version upon new releases in this repository.
This PR addresses issue #667 and resumes the changes from #675, which was previously put on hold due to the limitations of the proposed solutions utilizing GitHub Actions:
dawidd6/action-homebrew-bump-formula
mislav/bump-homebrew-formula-action
Above actions are unable to correctly parse and update a nested/custom formula file as maintained in ppc64le-cloud/homebrew-pvsadm. Additionally, the approach of maintaining three separate formulas did not work out for the reasons outlined here: https://github.com/ppc64le-cloud/homebrew-pvsadm/pull/6
As discussed, alternate solution uses a custom Bash script to update the formula file. The script has been thoroughly tested against a personal forked repository.
NOTE:
It requires additional token to create PR using GH cli in the name GH_TOKEN. Required scopes include read:org, repo, workflow, write:packages

Depends on https://github.com/ppc64le-cloud/homebrew-pvsadm/pull/7


HIVE-25146: JMH tests for Multi HT and parallel load

Change-Id: I33c7fc1435f8798b0f53c0a20812c93be16d137b

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 @ramesh0201

+1. looks good to me,


Added an interpreter spec for check crystal i is running on linux.

What's the point of this?
It seems this spec checks if an installed crystal compiler has interpreter support? That doen't make any sense because such a compiler binary would be completely independent of the spec context.

What's the point of this? It seems this spec checks if an installed crystal compiler has interpreter support? That doen't make any sense because such a compiler binary would be completely independent of the spec context.

Because build 1.11.0 failed on Arch Linux, see https://forum.crystal-lang.org/t/crystal-1-11-has-been-released/6284/4?u=zw963
If I remember correctly, and it really running failed this time, this should should be the second time the interpreter has broken?
So, instead of .build/crystal i should work? test on this no side effect.

The interpreter is tested in Linux, see for example https://github.com/crystal-lang/crystal/actions/runs/7460577216/job/20299250229?pr=14189 .
There might be an issue with Arch, or with any Linux distribution not tested in our CI, but that's a different topic.

The interpreter is tested in Linux, see for example https://github.com/crystal-lang/crystal/actions/runs/7460577216/job/20299250229?pr=14189 .
There might be an issue with Arch, or with any Linux distribution not tested in our CI, but that's a different topic.

Hi, i thought this spec should be fix to let spec failed before merge #14198 @straight-shoota


Documentation bug: wrong port for broker

Your documentation gives this example:
mochad_dispatch -s hal9000 mqtt://mqtt.example.com:1833
However, the default port for my MQTT broker (mosquitto) is 1883. Had me confused for a minute!

You're exactly right.  That's very confusing for users, I'll have that fixed in the next few days

Fixed!  Thanks for the report :)


make wercker preview badge static

issue: https://github.com/badges/shields/issues/1961

Warnings

:warning:

This PR modified service code for wercker but not its test code. That's okay so long as it's refactoring existing code.

Messages

:book:

:sparkles: Thanks for your contribution to Shields, @niccokunzmann!

  Generated by :no_entry_sign: dangerJS


Fix cpe matching inconsistencies

Fixes #170
PR contents

[x] CPE matches are to be held in sets instead of lists
[x] CVE matches are to be held in sets instead of lists
[x] When having >100 matches, "random" 100 matches are presented, decide if OK and document
[x] When having includingEnd (etc.) versions, random CPE is selected into the CPEDataset with cpe.version == "-". This selection should be standardized (or all version ranges noted down) to keep CPEDatasets without excessive diffs
[x] start_version and end_version are not really utilized IIRC. But they are somehow processed in CVEDataset. We should check if by not utilizing them we omit some CVEs or not. If yes, we should document this in the code as this is highly unintuitive and tricky.
[x] Currently, if we (randomly) add some CPE with version interval into the CPEDataset, we don't actually use this interval during CPE matching (double-check). However, we use intervaled version as a key in dictionary of vulnerable CVEs, effectively discarding all other versions, this is kinda weird behaviour.

Example of weird diff in matched CPEs: https://gist.github.com/J08nY/f45ccd7974bc538c9d79d5b84ef69fd4
Comments

Only first 100 matches will be displayed, but these are now sorted on two keys to achieve determinism. First, they are sorted by their similarity score, if equal, then lexicographically by their cpe uri (both descending).
CPEDataset is enhanced with CPEs from CVEDataset only if any of the following works for the CPE under consideration:

considered_cpe.version == '-' and (considered_cpe.vendor, considered_cpe.item_name) are not present in the original CPEDataset
considered_cpe.version != '-' and (considered_cpe.vendor, considered_cpe.item_name, considered_cpe.version) are not present in the original CPEDataset

The filtering above fixes "duplicates" in the CPEDataset
start_version and end_version are now being utilized only inside CVEDataset to drive dictionary of vulnerable configurations. When CPE from CVEDataset is added to CPEDataset, new CPE is effectively created with start_version and end_version set to None. This fixes undefined behaviour when different (start_version, end_version) tuples made it into the CPEDataset based on randomness
Undefined versions are now normalized from three distinct values '-', '*', '' to single value '-'
CPE parsing was improved to include escaping of key characters. Still imperfect but good enough I guess.
This leaves the result of our matching too benevolent, the problem is eloborately described in #173, to be fixed separately.

@J08nY this should, to my best knowledge, fix all possible sources of inconsisentices in our CPE/CVE matching algorithms. Still, it created another problem (see #173), but that's a different story. I'll fix that soonish. Though, the code related to matching is quite complex due to the quality of JSONs distributed by NIST.


QubitOperator('X0 X0') should simplify to the identity

Instead, it prints out to
1.0 [X0 X0]

There's a similar thing with FermionOperator, which is that FermionOperator('0^ 0^') does not simplify to zero. However, we have normal_ordered to simplify FermionOperators, while there is no analogous function for QubitOperators. But since simplifying QubitOperators is cheap (and normal-ordering is expensive), I think it makes sense to deal with this kind of stuff upon initialization.

Also, QubitOperator('X0 Y0') should simplify to i Z0.

I mean, if we're going to do this I guess it should be able to do arbitrary multiplication. E.g. it should know how to interpret "X0 Y1 Z3 X2 Z3 Y0", right?

Yeah.
On Fri, Sep 14, 2018, 5:47 PM Ryan Babbush<EMAIL_ADDRESS>wrote:

I mean, if we're going to do this I guess it should be able to do
arbitrary multiplication. E.g. it should know how to interpret "X0 Y1 Z3 X2
Z3 Y0", right?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/quantumlib/OpenFermion/issues/446#issuecomment-421503546,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEGN5tQfgw7BYefd3_AYQNjctjEZO77Fks5ubDHwgaJpZM4Wb4bI
.


model.build() fails to initiazlize all layer weights

Hi, thanks for keras. I've been recently trying to play around and I've found that model.build() doesn't initialize all layer weights as is indicated in the documentation, instead one has to call build() on the individual layer.
Example:
from keras.layers import Input, Lambda
from keras.applications import VGG16
from keras.optimizers import Adam
import numpy as np

model = VGG16(include_top=False, weights=None, input_tensor=Input(shape=(224, 224, 3)))
model.summary()

# Now let's say that I want to add the vgg preprocessing after the 'input_1' layer
def vgg_preprocess(x):
    x -= np.array([123.68, 116.779, 103.939])
    return x

model.layers.insert(1, Lambda(vgg_preprocess, input_shape=(224, 224, 3), output_shape=(299,299,3)))

# at this stage I was hopping that `model.compile` would have taken care of everything but that was not true
model.compile(optimizer=Adam(), loss='categorical_crossentropy', metrics=['accuracy'])

# trying to print a summary of the model gives an error
model.summary()

# using model.build((224, 224, 3)) doesn't help either
model.build((224, 224, 3))
model.summary()

# the only solution was to use build on the layer
model.layers[1].build((224, 224, 3))

model.summary()

I also have another question, in the printed model summary the layer gets a random name and the output shape says multiple.
How can I set my own name plus I don't understand the output shape multiple is that good or bad? How can I define that properly?

Lambda layers can only execute differentiable theano/tensorflow functions, not arbitrary python code.

@kgrm got it, so instead I should change vgg_preprocess to something like this:
x = K.variable(..., ....)
mean = K.variable(...., ....)
def vgg_preprocess(x):
    x -= mean
    return x

symbolic_func = K.function([x, mean], vgg_preprocess)

# so then my Lambda layer should be something like?

Lambda(symbolic_func, input_shape=(224, 224, 3), output_shape=(299,299,3)

Assuming that something like the above is correct, which is not but assuming then what is the next step?
# Pseudocode
1. model_compile()
2. model.build()
or
3. model.layer_n.build()

P.S. Didn't had the time to check this since I'm not in front of the computer

Given a properly specified backend function in the Lambda layer, the model can be compiled with m.compile(). The layer itself can more easily be specified as
mean = K.variable([123.68, 116.779, 103.939])
layer = Lambda(lambda x: x - mean)

@kgrm thanks, for the quick reply, closing it.


HTML Generation

The PR primarily deals with separating HTML string generation from any other object that spits out HTML.
Motivation and Context

In order to keep classes like Cells and friends as simple as possible, and to prevent easy errors on the part of programmers, it's a good idea to avoid writing HTML by hand.

With the classes introduced in this PR, we can now create elements in Python without having to worry about how the strings look or how to print children, etc.

Approach

The HTML generation classes and their tests can be found in object_database/web/html
HTML String Generation
The idea here is fairly simple: any class that subclasses AbstractHTMLWriter (for now this is only HTMLElement and HTMLTextContent) must implement a method called print_on whose first argument is some kind of stream.
In our case we are using StringIO to build strings in a streamlike fashion.
When __str__ is called on any HTMLElement instance, it will create a new stream and pass it to its own print_on, which handles printing open tags, attributes, children, and close tags. Printing children is pretty simple: it will just call print_on and pass the working stream to each of its children in order, and so on recursively down the chain.
To test this out, try something like this:
parent = HTMLElement('div')
parent.add_class("container")
child_one = HTMLElement('img', is_self_closing=True)
child_two = HTMLElement('p')
child_content = HTMLTextContent("This is the paragraph content")
child_two.add_child(child_content)
parent.add_children([child_one, child_two])
print(parent)

Now this code is somewhat verbose, which is why we added other ways to do this (see chaining and constructors sections below)
Chaining
The example in the section above shows how writing this HTML can be fairly explicit. But we've added the convenience of chaining most public methods (ie, returning self) so that we can do things like:
element = HTMLElement('div').with_children(
    HTMLElement('img', is_self_closing=True).add_class("primary-img"),
    HTMLElement('p').add_class("main-typography").add_child(
        HTMLTextContent("This is the paragraph content")
    )
)
print(element)

This allows for some better composition.
Special Constructors
Some HTML5 elements are "self-closing," meaning they don't have any children nor to they use a totally separate closing tag. For example,
Something
 is not self closing, while  is self closing.
When using the plain class constructor, one would need to know ahead of time whether or not the tag is self closing or not the generate valid HTML5. So we decided to dyamically add class methods to HTMLElement that "do the right thing" for you:
div = HTMLElement.div().add_class("main")
img = HTMLElement.img().add_class("big")
div.add_child(img)
print(div)
#

#
#

Cells Proof of Concept
We have successfully used these classes as drop-in replacements for the HTML that is generated in
three Cells classes: Card, CardTitle, and CollapsiblePanel. So far these appear to load and work fine when running the database test server.
How Has This Been Tested?

We have created a full set of tests in object_database/web/html/html_gen_tests.py and all are currently passing.
All current Cells tests are also passing, as is the regular full test suite.
For testing the output, we run object_database/frontends/object_database_webtest.py and then load all relevant services in the browser, checking the console as we go. So far any errors that do appear in the browser console are identical to those on the current (as of this writing) dev branch and are non-fatal.
We have also run flake8 on the changed files and everything seems to be ok linting wise.

Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Shall we merge and then deal with other formatting issues in later PRs?

The enclosing parens mean you don’t need escape slashes
Sent from my iPhone

On Mar 25, 2019, at 4:27 PM, Eric Gade<EMAIL_ADDRESS>wrote:
@darth-cheney commented on this pull request.
In object_database/web/cells.py:

           ____contents__

       """.replace('__other__', other)

          .replace('__style__', self._divStyle())

          .replace("__header__", """

                   ____header__

           """ if self.header is not None else "")

   )

   body = HTMLElement.div().add_class(

       "card-body").add_class(

           other).add_child(

               HTMLTextContent(" ____contents__"))

Yeah. In fact, the format you've proposed @atzannes is what would normally be used in JS. I just haven't encountered "chainable" patterns in Python too often and wasn't sure how to format them properly and have linters pass. I believe you would need a \ at the end of each line in that construction though, right? Either way the "separate lines" approach is certainly easier to read.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub, or mute the thread.

Yes let’s merge and when you do the rest of the cells we can try to get a consistent style
Sent from my iPhone

On Mar 25, 2019, at 5:22 PM, Eric Gade<EMAIL_ADDRESS>wrote:
Shall we merge and then deal with other formatting issues in later PRs?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub, or mute the thread.


Allow multiple values for checkboxes with fill()

If there are multiple  elements in the DOM, I would like to be able to give multiple values in the fill method.
Example: http://www.pap.fr/ checkbox element name = "typebiens[]"

See pull request https://github.com/n1k0/casperjs/pull/20


Is the full zip file of the origin folder available?

It's pretty troublesome to download separately, is it possible to compress the folder and upload it somewhere?
(Copyright issues?)

Hi Shuo,
We will prepare the download script for these datasets soon.
Best,
Wuwei

Any update? Or if the prepare script for wikisql available? I cannot find it.

We cannot release the original data directly, therefore we prepared download script, please check readme file.
We will upload the wikisql processing file soon.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Tianshu Wang @.>
Sent: Saturday, August 19, 2023 4:28:37 AM
To: awslabs/unified-text2sql-benchmark @.>
Cc: Wuwei Lan @.>; State change @.>
Subject: Re: [awslabs/unified-text2sql-benchmark] Is the full zip file of the origin folder available? (Issue #1)
Any update? Or if the prepare script for wikisql available? I cannot find it.
—
Reply to this email directly, view it on GitHubhttps://github.com/awslabs/unified-text2sql-benchmark/issues/1#issuecomment-1684892656, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACDANXHL7DEECTN6RUYN7ATXWB2LLANCNFSM6AAAAAA3A3AH7Q.
You are receiving this because you modified the open/close state.Message ID: @.***>

Script to prepare WikiSQL has been added. To download the datasets please refer to this section of the ReadMe. Note: Some datasets require manual efforts in downloading.
Closing this issue.


Option to preserve header's title case

I am building a HTTP proxy with hyper, and I realize that headers are all converted to lower case for HTTP/1.x.
There was an issue about adding http1_title_case_headers for clients: https://github.com/hyperium/hyper/issues/1492 , but we don't have a way to preserve the response headers' title case from remote servers.
Some client applications are expecting to receive header with name that is exactly the same as the server returns, for example:

Nintendo Switch's connection checking will sent a GET request to http://ctest.cdn.nintendo.net/ and expecting a response with header X-Organization: Nintendo.

As always, I've suggest filing a bug with the thing that expects a certain casing, since in HTTP/1 header casing is defined as case-insensitive, and in HTTP2 and 3, it's forced to lowercase. So the Switch is definitely wrong here.
That said, I realize it's not always controllable, and so I'm exploring providing support in the C API for curl, and it might be possible to expose it as part of Rust's API as well... So far, my idea is to include an extra type in the request Extensions which is a mapping of HeaderNames to the original casing. If that type exists, it would use that to serialize instead of HeaderName, and then hyper would also collect a map for the response and put that in the response Extensions.

Isn't there now code to do that behind the ffi feature?

There is indeed code in hyper now, currently behind the ffi feature. If someone wants to contribute this, it'd be good to propose the API that is exposed to users, and then converting the code to be not just for the ffi feature.

PR: https://github.com/hyperium/hyper/pull/2480


Add support for names such as: name[] and name[abc]

Further to #18
Tested the following: https://codepen.io/ismailm___/pen/QzEbyZ?editors=1010

It does not work. See my comment

The same issue is also present in the current master branch: https://codepen.io/ismailm___/pen/QzEjgj?editors=1010
The issue is that checkboxes
I guess the fix would probably be checking if it's a checkbox before running fn.showOrHide() and then if it's a checkbox, extract all checked values and then do the match...

Not sure what you mean by the second point in your comment?


How to make mamute work with non root user?

The setup instructions say:

First of all, you need to setup an MySQL database. The default database name is mamute_development, the username is root and the password must be blank.

I don't want to use root user. How can I make it work with a non-root user that also has a non-blank password?

This may be a better question for http://meta.mamute.org/
If you want to change the user name/password for MySQL in development it can be changed by modifying it in the mamute/src/main/resources/development/hibernate.cfg.xml file.  Of course on the MySQL side you need to 'create database mamute_development' and then create a user with permission to use that database and the same id/password as you put in the hibernate.cfg.xml file.

Thanks. This is exactly what I needed. This info ought to be there in the instructions on the main page.
Date: Thu, 18 Feb 2016 14:38:39 -0800
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [mamute] How to make mamute work with non root user? (#236)
This may be a better question for http://meta.mamute.org/
If you want to change the user name/password for MySQL in development it can be changed by modifying it in the mamute/src/main/resources/development/hibernate.cfg.xml file.  Of course on the MySQL side you need to 'create database mamute_development' and then create a user with permission to use that database and the same id/password as you put in the hibernate.cfg.xml file.
—
Reply to this email directly or view it on GitHub.


Add license to python package metadata

The license was not set for the python package metadata directly. We
listed the trove classifier for the project being apache licensed but
without specifying the license field certain parsers don't know what the
library is licensed under. This commit fixes the oversight and sets the
license field to apache 2.0.

Coverage remained the same at 91.981% when pulling 3c406ece94e57637b746be6742fb79dac090e4c1 on mtreinish:add-license-to-pkg-metadata into c7ef54fb878356dcc9b98ca9bdb1c4c446b61386 on Qiskit:master.


[LW-10057] Account activated toast is displayed for already active account

Checklist

[x] JIRA - https://input-output.atlassian.net/browse/LW-10057
[ ] Proper tests implemented
[ ] Screenshots added.

Proposed solution
Checks if wallet is already activated.
Testing
Describe here, how the new implementation can be tested.
Provide link or briefly describe User Acceptance Criteria/Tests that need to be met
Screenshots
Attach screenshots here if implementation involves some UI changes

coverage: 51.404% (-0.02%) from 51.421%
when pulling 38c149e362148c3312605f88c3183e32cfe7256f on fix/lw-10057-account-activated-toast-displayed-for-already-active-account
into d05f524a7a03014f082fb6d33886b44dbb4a71dc on main.


Handle object not found error in bindings: S3, Azure Blob Storage, GCP Bucket, Huawei OBS

Description
Added specific error handling to object not found mentioning the key so user is not frustated with unnecessary long errors
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #2526 _
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

/ok-to-test

/ok-to-test


Direct specification of for_each context?

My real use case is a lot more complex but in essence with multiple for loops and exceptions we have to do the following in SoS:
input: for_each=[ {'var1': range(5)}, {'var2': range(5)}]

skip_if(var1 == var2)

Then when processing the data in Python I have to do
for var in range(5):
  for var1 in range(5):
    if var == var1:
      continue

For these cases the following method might be cleaner:
contexts = []
for var in range(5):
  for var1 in range(5):
    if var == var1:
       continue
    contexts.appen(dict(var=var, var1=var1)

input: for_each=contexts

basically we construct contexts directly and free sos from doing this, and the code can be easier to understood for users.
@gaow What do you think? Of course interface is another issue.

I think i've done contexts approach for some of my applications -- doesn't it work with the current SoS implementation?

Hmm, at least the current list of duct syntax means something else.  Did you simply loop through contexts and refer to its content as dictionary key?

Ha okay my usage is in DSC. For example a DSC module like this
method:
  n, p: (10, 20), (10, 5)
  @FILTER: n>p

Will be translated to the following for_each code in SoS:
input: for_each = {'_n,_p,_DSC_REPLICATE':[(_n,_p,_DSC_REPLICATE) for _DSC_REPLICATE in DSC_REPLICATE for _p in p for _n in n if (_n > _p and _n == 10 and _p == 5) or (_n > _p and _n == 10 and _p == 20)]}

so yes technically I use the context notion but my approach is to create the exact SoS syntax automatically like this. ...

ok, currently for_each is designed to to cross-product of multiple variables, less so for irregular context, for which something like
input: for_each = [
  {'A': 1, 'B': 2},
  {'A': 5, 'B': 7'}
  ]

looks to be more readable to me than your style,
input: for_each={'A,B': [(1, 2), (5, 7)]}

It could be too confusing to add another style to the already very complex syntax for for_each though.

Just for the record, for my application with a lot of directly specified "context", I am using
def from_contexts(contexts):
    keys = contexts[0].keys()
    return {','.join(keys): [[x[key] for key in keys] for x in contexts]}

input: for_each=from_contexts(contexts)

to convert list of cases into SoS for_each. The contexts were created using the following code
contexts = []
for risk_name in ['Lowrisk' , 'Highrisk']:
    for quarantine_days in [0, 10, 14]:
        for test_frequency in ['', 'M', 'MF', 'MWF', 'MTWTF']:
            for test_name in ['NOTEST', 'PCR90', 'PCR95', 'AG85']:
                if test_frequency == '' and test_name != 'NOTEST':
                    continue
                if test_name == 'NOTEST' and test_frequency != '':
                    continue
                if test_name != 'AG85' and test_frequency == 'MTWTF':
                    continue
                for rep in range(10):
                    contexts.append(dict(
                        risk_name = risk_name,
                        quarantine_days = quarantine_days,
                        test_frequency = test_frequency,
                        test_name = test_name,
                        name = f'{risk_name}_Q{quarantine_days}_{test_frequency}_{test_name}_{rep}',
                        general_community_infection_rate = 6.8e-5 if risk_name == 'Lowrisk' else 6.8e-4,
                        zone_a_community_infection_rate = 3.4e-6,
                        test_sensitivity = {'NOTEST': 0, 'PCR90': 0.9, 'PCR95': 0.95, 'AG85': 0.85}[test_name],
                        test_specificity = {'NOTEST': 0, 'PCR90': 0.99,'PCR95': 0.99, 'AG85': 0.985}[test_name],
                        zone_a_test_time = get_test_time(test_frequency, start=7, stop=quarantine_days + 46),
                        zone_am_test_time = get_test_time(test_frequency, stop=quarantine_days + 46),
                        zone_b_test_time = get_test_time('' if test_frequency else 'M', stop=quarantine_days + 46),
                        seroprevalence = 0.137,
                    ))

The major advantages, I think, are that

All elements are clearly defined in context, even those "constants" that are needed by the substeps.
the contexts can be used throughout my project as "cases" and I do not have to recreate them during simulation and during data analysis.

done.


Roll Engine from aff8465afd80 to a62795a77ba0 (2 revisions)

https://github.com/flutter/engine/compare/aff8465afd80...a62795a77ba0
2021-02-01<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from -iUKxTl_w... to blNDDMWIJ... (flutter/engine#24115)
2021-02-01<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from KfIfizZ7o... to IHHTb9H-B... (flutter/engine#24113)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from -iUKxTl_wJsC to blNDDMWIJolt
fuchsia/sdk/core/mac-amd64 from KfIfizZ7odhD to IHHTb9H-BB4C
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


Resend 2FA code button is not working

Problem
Nothing happens when you click the "resend" button on 2fa verify modal window

Expected Behavior
Code must be sent again
Steps to reproduce

Create an account
Enable 2FA
Go to /send-money
Send some Near
Wait for the first confirmation
Click on the "resend" button

@ihaid please make sure the fix works ok

@ihaid please make sure the fix works ok

@vgrichina I still see this issue on testnet


Unexpected connection behaviour when using a updateNodeInternals

Describe the Bug
I want to change height of node when starting connection. But it becomes invisible and node is not updating. How to fix it?
Your Example Website or App
https://codesandbox.io/p/sandbox/mystifying-meadow-lc6kfs?file=%2Fstyle.css&utm_medium=sandpack
Steps to Reproduce the Bug or Issue
Start connecting
Current connection becomes invisible
Expected behavior
Current connection visible and handle moves to the center (and doesn't stay in old center)
Screenshots or Videos
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

Can you check your codesandbox? It seems to be the easy connect example.

Can you check your codesandbox? It seems to be the easy connect example.

yep, sorry, copy wrong link https://codesandbox.io/p/sandbox/blissful-wilson-3zx9f7?file=%2Fstyle.css%3A12%2C16

This is indeed not intended behavior. This uncovered a potential improvement we should implement. You can track it over at #4520.
For now, I suggest you use nodeOrigin={[0.5, 0.5]} in your flow which leaves any centered handles in the same position and extends all nodes from a centered origin (which is what you'd want to do anyway I suppose).

This will be fixed by #4548