A PKCS7 SignedData can't be validated

I have a pkcs7 SignedData(see my attachment), it can't be validated by Bouncy Castle with follow code:
//load data from my attached file
FileInputStream file = new FileInputStream("SignedData.bin.txt");
ByteArrayOutputStream out = new ByteArrayOutputStream();
byte[] buffer = new byte[1024];
int length;
while((length = file.read(buffer)) > 0)
{
  out.write(buffer, 0, length);
}
out.flush();
byte[] encapSigData = out.toByteArray();
out.close();

//validate the SignedData
CMSSignedDataParser     sp = new CMSSignedDataParser(new JcaDigestCalculatorProviderBuilder().setProvider("BC").build(), encapSigData);

sp.getSignedContent().drain();

Store                   certStore = sp.getCertificates();
SignerInformationStore  signers = sp.getSignerInfos();

Collection              c = signers.getSigners();
Iterator                it = c.iterator();

while (it.hasNext())
{
  SignerInformation   signer = (SignerInformation)it.next();
  Collection          certCollection = certStore.getMatches(signer.getSID());

  Iterator        certIt = certCollection.iterator();
  X509CertificateHolder cert = (X509CertificateHolder)certIt.next();

  System.out.println("verify returns: " + signer.verify(new JcaSimpleSignerInfoVerifierBuilder().setProvider("BC").build(cert)));
}

Exception occurred at CMSSignedDataParser constructor:
java.lang.ClassCastException: org.bouncycastle.asn1.DERSequenceParser cannot be cast to org.bouncycastle.asn1.ASN1OctetStringParser
at org.bouncycastle.cms.CMSSignedDataParser.<init>(Unknown Source)
at org.bouncycastle.cms.CMSSignedDataParser.<init>(Unknown Source)
at org.bouncycastle.cms.CMSSignedDataParser.<init>(Unknown Source)

After some analysis, I find that the content of contentInfo in SignedData is a Sequence. It seems that bouncycastle can't accept a Sequence to be the content.
More details in stackoverflow: http://stackoverflow.com/questions/34197756/validate-pkcs7-signeddata-by-bouncy-castle-in-java
SignedData.bin.txt

Okay, the issue here is that this is a real PKCS#7 message, as opposed to a CMS one.
I think I've managed support for this the API. Try the latest beta at http://www.bouncycastle.org/betas 154b12 or later.
You might find it more useful just to use CMSSignedData as getSignedContent() will now return the actual ASN.1 object that was in the encapsulated data area.

@bcgit
It is OK in 154b12, thanks.

@bcgit
Does this fix also be included in bc-csharp?
I tried the latest bc-csharp V1.8.6, neither new CmsSignedData(byte[] sigBlock) or CmsSignedDataParser(byte[] sigBlock) can parse the PKCS#7 message.


Sheet not opening in StrictMode

I am trying this module in my page. However, I cannot see the sheet in view because we set transform value is css. Also, we do not have this behavior in the release environment (I deployed my website through GCP).
One more interesting thing is if I zoom in and out by cmd +/-, then I can see the sheet rendering in view when I am in development mode.
Is this a known issue of this module? Or is there any workaround to prevent this issue? Thank you.
CODE:
'use client';
import { Sheet } from 'react-modal-sheet';
import { useState } from 'react';
export default function Container() {
const [isOpen, setOpen] = useState(false);

return (

this is container

        <button onClick={() => setOpen(true)}>Open sheet</button>

        <Sheet
            isOpen={isOpen}
            onClose={() => setOpen(false)}
            snapPoints={[-50, 0.5, 200, 0]}
            initialSnap={1}
        >
            <Sheet.Container>
                <Sheet.Header />
                <Sheet.Content>

We will have product content here

                </Sheet.Content>
            </Sheet.Container>
            <Sheet.Backdrop />
        </Sheet>

)

}

Hi @Chrishuwei 👋🏻
This sounds like it could be a React StrictMode related issue. For some reason the motion library that react-modal-sheet relies on struggles with StrictMode causing the opening animation of the sheet to not trigger correctly. The reason things work for you in production is because StrictMode is only enabled in development mode.
Unfortunately I haven't been able to figure out a workaround yet. Currently the easiest fix is to just turn off/remove React StrictMode.

I updated the title to reflect the real root cause of the problem.
I'll keep this issue open so that other devs that encounter the same issue can see it and maybe someone has an idea how to mitigate the problem. I currently don't have much time to investigate this problem.

Are you using React 18 or 19?
I don't have any issue with StrictMode enabled in React 18


Are we able to control the new lines on tiggers pannel

I'm submitting a ...

[x] Question / Support request

Software Versions

Grafana
Zabbix
Grafana-Zabbix Plugin

5.0.1
3.2.7
3.8.1

What did you do?
when creating tiggers dashboard it puts everything on a new line, when that dashboard is for a tv, having 4 lines is takes to much space.

What was the expected result?
that you can have multiple things on one line, like the demo dashboard,

or be able to go back to the old way of displaying

What happened instead?
getting no option to control the wasted space

It's a bug when you use it with Grafana 5.x #522


cannot call method "helpers" of undefined when moving meteor code to src

Hi, so we moved our source code into a src directory (previously client, server, and both lived in root). Upon moving it, jasmine is unhappy about discovering client-side basic packaging? In particular Template.helpers seems to be undefined.
My feeling is that it has to do with mocks, but I find it odd that specifically switching the directory where code is loaded from causes issues. (this may be because I'm new to meteor).
Here's the basic log output:
W20150702-04:00:52.218(0)? (STDERR) connections property is deprecated. Use getConnections() method
W20150702-04:00:54.717(0)? (STDERR) [sanjo:jasmine]: The file "/var/www/src/client/templates/announcements/announcement.js" has syntax errors. [TypeError: Cannot call method 'helpers' of undefined]
W20150702-04:00:54.740(0)? (STDERR) [sanjo:jasmine]: The file "/var/www/src/client/templates/announcements/announcementCreate.js" has syntax errors. [TypeError: Cannot call method 'helpers' of undefined]
W20150702-04:00:54.746(0)? (STDERR) [sanjo:jasmine]: The file "/var/www/src/client/templates/home/home.js" has syntax errors. [TypeError: Cannot call method 'helpers' of undefined]
W20150702-04:00:54.752(0)? (STDERR) [sanjo:jasmine]: The file "/var/www/src/client/templates/layout/layout.js" has syntax errors. [TypeError: Cannot call method 'helpers' of undefined]
W20150702-04:00:54.758(0)? (STDERR) [sanjo:jasmine]: The file "/var/www/src/client/templates/nav/nav.js" has syntax errors. [TypeError: Cannot call method 'helpers' of undefined]
W20150702-04:00:54.764(0)? (STDERR) [sanjo:jasmine]: The file "/var/www/src/client/templates/users/userCreate.js" has syntax errors. [TypeError: Cannot call method 'helpers' of undefined]
W20150702-04:00:54.772(0)? (STDERR) [sanjo:jasmine]: The file "/var/www/src/client/templates/users/userEdit.js" has syntax errors. [TypeError: Cannot call method 'helpers' of undefined]
W20150702-04:00:54.785(0)? (STDERR) [sanjo:jasmine]: The file "/var/www/src/client/templates/users/userProfile.js" has syntax errors. [TypeError: Cannot call method 'helpers' of undefined]
W20150702-04:00:54.798(0)? (STDERR) [sanjo:jasmine]: The file "/var/www/src/client/templates/users/userSearch.js" has syntax errors. [TypeError: Cannot call method 'helpers' of undefined]
W20150702-04:00:54.810(0)? (STDERR) [sanjo:jasmine]: The file "/var/www/src/client/templates/users/username.js

Here's some more info:
-verbose output.
-versions file.
Let me know if I can provide any other information.

Yeah, seems to be a bug where the server unit mode tries to load client only files. Can you move your server unit mode tests to the server integration mode? I will remove the server unit mode in the next release.

@Sanjo Are you planning to move the unit testing modes into a separate package? or depreciate the functionality altogether?

My plan is to remove them altogether. If needed, I can improve the remaining server and client mode for unit testing.
I think with ES6 modules support in Meteor, unit testing will be better. And I will then work on a better unit testing solution for Meteor.
So for now you can use the integration modes for unit testing. If you use packages you can also write unit tests for packages with Jasmine (this also uses the integration modes).

I'm interested in the reasoning behind this move considering the amount of effort that went into building them after the depreciation of jasmine-unit. Just to clarify, will this mean all jasmine tests will run in the meteor context?

I think that more work would be necessary to make the unit modes really good. For me the value of this work is currently not high enough to do it. Also with my current approaches for the unit modes I don't leverage the Meteor Core. I think the right user journey for unit testing is a tight integration into Meteor (that will come in the next 6-12 months I think. And I will help.).
Also I think in the current state most of the people are better off just using the integration modes for all unit and integration tests they write. The unit modes have a lot more rough edges. By removing the unit modes I can make the user journey better.

Just to clarify, this will mean all jasmine tests will now run in the meteor context?

Yes. But you can still mock, stub and spy on things. This works on a test by test basis. spyOn does this already.
Let me know if you have any other concerns or problems with the integration modes. Feedback is welcomed.

And also no work is thrown away. I will use all the experience I gained for the future work. Karma may will be the client test runner for unit tests is the future (would make sense). Also the autostubbing will be used in some form. I just have to make it work for client also. Anyway the autostubbing work came from Jest ;-)


Add the ability to remove files when using file search

Implements godot-proposals#7881

I messed up my commit history by syncing the repo. Will be reopened soon.

I mean I'll make a new clean pull request

New pull request.


NuGet.Server shows version <IP_ADDRESS> on landing page

Install a recent dev build and hit F5:

This is likely due to the switch to ServerCommon build (which does not set AssemblyFileVersionAttribute).

Fixed with https://github.com/NuGet/NuGet.Server/pull/36.


The Statue

I'll ask an AI to generate a picture of a stone statue of me naked with a laptop in front of EPSIC.

MAIS QUELLE EXELLENTE IDÉE !!! METTONS-LA JUSTE ICI :

Oder da:
https://www.epsic.ch/wp-content/uploads/2022/05/Banner_1_DSC_3704.jpg


chore(package.json): limit localforage version

Closes #28.
As per #28 (opened @ 23 Oct), at localforage we are planning to release v1.5 that will embed the respective typings as part of the npm package.
This is hopefully going to happen before Christmas, so I guess a new ionic-storage bugfix release should be released with this change.
Otherwise the @types/localforage could conflict with the embedded localforage typings.
I will open a new PR for the next localforage release as soon as it gets released

Any feedback on this? Should we proceed and release lovalforage v1.5?

Oh, and happy new year!

Reposting

Any feedback on this? Should we proceed and release lovalforage v1.5?

Thanks, makes sense! Appreciate the heads up as well.


UI - no error message shows if the name exists when creating a cloud credential

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

run Rancher v2.2.4-rc20
add a new cloud credential from either the node template or the cloud credential page
enter a name that is already used
fill out all fields
click create to save

Result:

cannot save the template, but there is no error message showing the reason
the API response shows the error message

{"baseType":"error","code":"AlreadyExists","message":"secrets \"ec2\" already exists","status":409,"type":"error"}

Issue does NOT occur in v2.2.3

In general, it seems that there is no error message when the API for creating a cloud credential is invalid:
https://github.com/rancher/rancher/issues/20577
https://github.com/rancher/rancher/issues/19677
https://github.com/rancher/rancher/issues/19759
https://github.com/rancher/rancher/issues/21215

Version: master-head (7/25/19)
The UI now passes along the api error message:

I ensured this works correctly both via "Add Cloud Credential" modal from "Cloud Credentials" page and when creating a new node template (pick "Add New" button to make new cloud credential).


Error while building apollo project using CLion

Environment :- Build label: 0.5.3, CLion: clion-2018.1.5, Bazel plugin versiion: 2<IP_ADDRESS>.3
I am trying to build  apollo project in CLion where it is giving this error message,
INFO: Reading 'startup' options from /apollo/tools/bazel.rc: --batch_cpu_scheduling
INFO: Loading package: @local_config_xcode//
ERROR: /root/.cache/bazel/_bazel_root/540135163923dd7d5820f3ee4b306b32/external/intellij_aspect/intellij_info_impl.bzl:318:43: name 'cc_common' is not defined (did you mean 'java_common'?).
ERROR: Extension 'intellij_info_impl.bzl' has errors.
INFO: Found 1651 targets...
INFO: Building...
ERROR: command succeeded, but not all targets were analyzed.
INFO: Elapsed time: 2.269s, Critical Path: 0.00s
Prefetching genfiles...
Prefetching files...
Computing directory structure...
Committing project structure...
Workspace has 0 libraries
Workspace has 2 modules
Sync was successful, but there were BUILD file errors. The project may not be fully updated or resolve until fixed. If the errors are from your working set, please uncheck 'Blaze > Sync > Expand Sync to Working Set' and try again.
Updating in-memory state...
0 unique C configurations (0 reused), 0 C targets

Any help regarding this is appriciated.

cc_common comes from bazel:
https://docs.bazel.build/versions/master/skylark/lib/cc_common.html
https://github.com/bazelbuild/bazel/blob/95a40bea2d0488ba3afc803b9082b16cadef1deb/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/cpp/CcModuleApi.java
It's fairly new. Can you upgrade to a newer bazel version?

Yeap, sorrrrry to make a new issue. I upgraded to 0.15.x and it gave a new set of errors. But I was able to make them fixed. So you can close this issue.  :+1:

No worries, glad you got things working!

Hi @GPrathap are you using docker?  Or are you working directly on the host pc?

I was trying on using docker ubuntu 16:04. I did not work out for me. There were lots of dependencies were to update. I just gave up.


ECO-3658: Enable PayPal Express

Developer(s): @kkicheglovspryker

Ticket: https://spryker.atlassian.net/browse/ECO-3658

Release Group: https://release.spryker.com/release-groups/view/3540

PR Overview: https://release.spryker.com/release/pull-request-eco/Computop/40

merge: squash

Release Table

Module
Release Type
Constraint Updates

Computop
minor

Module Computop
Change log
Improvements

Introduced ComputopFacade::savePayPalExpressInitResponse() to persist ComputopPayPalExpressInitResponse to database.
Introduced ComputopFacade::savePayPalExpressCompleteResponse() to persist ComputopApiPayPalExpressCompleteResponse to database and change purchase items' OMS status.
Introduced ComputopClient::savePayPalExpressInitResponse().
Introduced ComputopClient::savePayPalExpressCompleteResponse().
Introduced DefaultShippingMethodPlugin to perform QuoteTransfer expansion with default shipping method.
Introduced ComputopPayPalExpressPayment transfer object to hold PayPal Express payment information.
Introduced ComputopPayPalExpressInitResponse transfer object to hold PayPal Express payment init response information.
Introduced ComputopPayPalExpressPrepareHandler to prepare order data for PayPal Express Checkout button.
Introduced ComputopPayPalExpressInitHandler to fill QuoteTransfer with all necessary data for successful order placement flow.
Introduced ComputopPayPalExpressCompleteHandler to perform ComputopApiClient::sendPayPalExpressCompleteRequest() and save ComputopPayPalExpressInitResponse and ComputopApiPayPalExpressCompleteResponse to database.
Introduced paypal-buttons molecule to render paypal buttons.

Constraint issues
Module version constraint error in Computop composer.json: "Eco.ComputopApi dev-feature/eco-3656/eco-3658-enable-paypal-express" seems invalid.
Perhaps a pseudonym for the next version can help you here to pass the verification.
"spryker-eco/computop-api": "dev-feature/eco-3656/eco-3658-enable-paypal-express as 1.1.0",

Constraint issues
Module version constraint error in Computop composer.json: "Eco.ComputopApi dev-feature/eco-3656/eco-3658-enable-paypal-express" seems invalid.
Perhaps a pseudonym for the next version can help you here to pass the verification.
"spryker-eco/computop-api": "dev-feature/eco-3656/eco-3658-enable-paypal-express as 1.1.0"

It has not helped unfortunately


docs: update README.md openapi section for additional clarity

The documentation section for the openapi implementation was a bit confusing to set up and where to set the patchNestJsSwagger() function in. I updated that ereadme section for more clarity for users stumbling upon that section

@MannyNe thanks for the contribution!
However, you don't need to call patchNestjsSwagger inside bootstrap, as shown in the example: https://github.com/BenLorantfy/nestjs-zod/blob/main/packages/example/src/main.ts#L6

Thank you for the clarification @BenLorantfy
I updated the docs again just to make it more clear for confused souls like me. I hope the update I made to the PR works. If not, you can close this PR. Thank you again, and great tool btw 😊

LGTM, thanks for the contribution 🙂

My pleasure 😊


fixed places related to heap hard limit configs

rename everywhere to heap hard limit instead of heap limit;
clarified when the GCHeapHardLimit and GCHeapHardLimitPercent configs are used;
switched the order of the link to GCHighMemPercent later to reflect its actual place in the doc (this also keeps the hard limit related configs together in the links section);

Internal previews

📄 File
🔗 Preview link

docs/core/runtime-config/garbage-collector.md
docs/core/runtime-config/garbage-collector

docs/standard/garbage-collection/fundamentals.md
docs/standard/garbage-collection/fundamentals

any more changes needed or could we merge this?


Make Task swappable

STATUS: ~COMPLETED~ WIP
PR for https://github.com/shacker/django-todo/issues/70

[x] auto-install bleach & swapper
[x] add dev script make_migrations.py
[x] Update doc for contributors
[x] Rename Task → BaseTask
[x] Add swappable class Task(BaseTask)
[x] Create schema migration
[x] Add data migration steps
[x] Get existing unit tests passing
User test fork in local project

[x] pip install -e git+https://github.com/Fandekasp/django-todo.git@feat/swappable_models#egg=todo
[x] ./manage.py migrate todoApplying todo.0011_swappable_task... OK

[x] Verify that migration went fineIn [1]: from todo.models import *
In [2]: BaseTask.objects.all()
Out[2]: <QuerySet [<BaseTask: First task>]>
In [3]: Task.objects.all()
Out[3]: <QuerySet [<Task: First task>]>

~Add unit tests to cover swappable capabilities~ Unnecessary, this should be swapper's responsibility. wagtail-review didn't add swapper tests either.
[x] Update usage doc
@shacker I don't know how you're generating the doc, but since you're including README.md in your MANIFEST.in file, you probably want to remove docs/index.md, which is a duplicate.
[x] Version up (would be nice to list contributors).
[ ] Fix bug happening in local project when running makemigrationERRORS:
<class 'todo.admin.AttachmentAdmin'>: (admin.E039) An admin for model "Task" has to be registered to be referenced by AttachmentAdmin.autocomplete_fields.
todo.Attachment.task: (fields.E301) Field defines a relation with the model 'todo.Task', which has been swapped out.
        HINT: Update the relation to point at 'settings.TODO_TASK_MODEL'.
todo.Comment.task: (fields.E301) Field defines a relation with the model 'todo.Task', which has been swapped out.
        HINT: Update the relation to point at 'settings.TODO_TASK_MODEL'.

Wow you've done a lot of work on this already! I just left a comment on #70  - I wish we could have talked through the implementation a bit before you wrote code. Anyway, let me know what you think over there.

Pausing this PR and getting some coffee. Will reply on the issue a bit later.

PR conversation in #70, closing this, as the goal can be more flexibly and less invasively solved via Django Signals. Thanks for the contribution though!


Broadcast types aren't stored

A new broadcast types 'Wegeniswerken' was added in the dashboard<EMAIL_ADDRESS>However, after publishing, the type didn't appear in the Notification Settings. Probably related to db.get in the uber transaction.

Hi @abom, can you have a look at this before you continue working on the polls feature?

I added a unit test and moved a line to the end of the function. Needs a proper fix but as long we use the uber transaction no real fix is possible. OSA and RT should be split to properly work.
https://github.com/our-city-app/oca-backend/commit/2680cad0847aa34a66ab82e34c1556d4871538eb


Updating events on event table does not work as expected

Reporter : @grainier
Jirra url : https://wso2.org/jira/browse/SIDDHI-25
Description :
In below siddhi query;

Query 1 will throw an exception Cannot create update for table 'dataTable', Cannot find attribute type as 'id' does not exist in '';.
If we use Query 2 to overcome above exception, it will update id column values of all rows which satisfy the condition dataTable.id != id.

@Import('dataIn:1.0.0') 	define stream dataIn (id int, name string); 	define table dataTable (id int, name string);
/* Query 1 */
from dataIn
select name
update dataTable
on dataTable.id == id;

/* Query 2 */
from dataIn
select name, id
update dataTable
on dataTable.id != id;

Fixed by introducing SET operator (samples). Therefore, above two queries can be re-written as follows;
/* Query 1 */
from dataIn
update dataTable
set dataTable.name = name
on dataTable.id == id;

/* Query 2 */
from dataIn
update dataTable
set dataTable.name = name, dataTable.id = id
on dataTable.id != id;


Add examples for testing clients and servers on Node.js

This adds exemplary tests for a service and a client application.

The mocking bits are not easy to parse but I don't see an alternative.

Can you elaborate which ones specifically?


fix domains docs

Add description for ID

Coverage decreased (-0.03%) to 71.348% when pulling 1ba7fd765cf105f69567e50635f96dc21a6b611d on SzySteve:docs/domains into 6c335ad9d69e77d90c03f68d7ed93706ba900899 on linode:pending-docs-updates.


Add reorientation to RPI also for train/val transforms

Currently, reorientation to RPI is explicitely specified only for inference_transforms:
https://github.com/sct-pipeline/contrast-agnostic-softseg-spinalcord/blob/59e6229a32fa68b20d8fae6e4cb73cd07a7a243b/monai/transforms.py#L44
This makes sense because spine-generic images used for train/val were already reoriented to RPI before training. So we need to reorient to RPI only images we run inference on.
But, with new data coming to the training, it would be a good idea to use Orientationd(keys=["image", lbl_key], axcodes="RPI") also for train_transforms and val_transforms.

Thanks for opening the issue! I missed this when creating release v2.1 -- this will be fixed in the next release!

closing as this is the default now and has been fixed


Issue creating new stack "no declaration" "volume section"

BUG REPORT INFORMATION
Description
If I'm creating new stacks, I get the error message:
"Deployment error
Named volume "edomi-backups:/var/edomi-backups:rw" is used in service "edomi" but no declaration was found in the volumes section."
Volume has been created via Portainer.
Steps to reproduce the issue:

Create new volume
edomi-backups
edomi-db
edomi-installation

Create new stack with this volume:
services:
edomi:
container_name: edomi
image: starwarsfan/edomi-docker:amd64-latest
ports:

"81:80"
"8080:8080"
"3671:3671/udp"
"50000:50000/udp"
"50001:50001/udp"
"22222:22"
dns:
<IP_ADDRESS>
<IP_ADDRESS>
volumes:
edomi-backups:/var/edomi-backups
edomi-db:/var/lib/mysql
edomi-installation:/usr/local/edomi
restart: on-failure

Technical details:

Portainer version: 2.1.1
Target Docker version (the host/cluster you manage):
Platform (windows/linux): linux

Working with Portainer 2.0.1 again, I'm using the docker build 2.1.1).

Hi
Thanks the post, is it possible to share the OS details, docker swarm version. also screenshots of working n non working scenarios

me too...

Hi @HimekoEx , is it possible to share the OS details, docker swarm version and the docker stack yaml. also screenshots of working scenarios.

You need to declare the volume separate:
Example:
version: '3'

services:
  nessus:
    image: tenableofficial/nessus
    restart: always
    container_name: nessus
    ports:
      - 8834:8834
    volumes:
    - nessus_data:/opt/nessus

volumes:
  nessus_data

Hi
pl share the OS details, docker swarm version also. i tried on ubuntu 20.04  n docker 20.10.5

]in volumes pl set external as true and try
volumes:
nessus_data:
external: true

Works with
``´external: true```
Info:
´´´
Linux  4.19.0-16-amd64 #1 SMP Debian 4.19.181-1 (2021-03-19) x86_64 GNU/Linux
´´´
´´´
Client: Docker Engine - Community
Version:           20.10.7
API version:       1.41
Go version:        go1.13.15
Git commit:        f0df350
Built:             Wed Jun  2 11:56:47 2021
OS/Arch:           linux/amd64
Context:           default
Experimental:      true
Server: Docker Engine - Community
Engine:
Version:          20.10.7
API version:      1.41 (minimum version 1.12)
Go version:       go1.13.15
Git commit:       b0f5bc3
Built:            Wed Jun  2 11:54:58 2021
OS/Arch:          linux/amd64
Experimental:     false
containerd:
Version:          1.4.6
GitCommit:        d71fcd7d8303cbf684402823e425e9dd2e99285d
runc:
Version:          1.0.0-rc95
GitCommit:        b9ee9c6314599f1b4a7f497e1f1f856fe433d3b7
docker-init:
Version:          0.19.0
GitCommit:        de40ad0
´´´


dev-deploy_weather_agent

These changes include minor modifications to the weather agent itself and some configuration required to deploy it in the agent stack.

I've tried submitted requests to the agent and they are functional


Run isort over botocore and tests

Breaking #2472 into slightly smaller pieces so we can have a more reasonable chance at properly reviewing the changes. This will be a first pass to add isort to keep our imports organized and fix our redundant/unnecessary imports.

Codecov Report

Merging #2642 (36be9e1) into develop (6f30bcb) will decrease coverage by 0.02%.
The diff coverage is 97.91%.

@@             Coverage Diff             @@
##           develop    #2642      +/-   ##
===========================================
- Coverage    95.28%   95.26%   -0.03%
===========================================
  Files           60       60
  Lines        12226    12162      -64
===========================================
- Hits         11650    11586      -64
  Misses         576      576

Impacted Files
Coverage Δ

botocore/crt/auth.py
0.00% <0.00%> (ø)

botocore/eventstream.py
100.00% <ø> (ø)

botocore/exceptions.py
99.54% <ø> (ø)

botocore/history.py
100.00% <ø> (ø)

botocore/hooks.py
95.66% <ø> (ø)

botocore/regions.py
94.93% <ø> (ø)

botocore/httpsession.py
91.42% <91.66%> (ø)

botocore/__init__.py
100.00% <100.00%> (ø)

botocore/args.py
100.00% <100.00%> (ø)

botocore/auth.py
97.76% <100.00%> (ø)

... and 35 more

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 6f30bcb...36be9e1. Read the comment docs.


GDPR infringment

Hello,
I'm contacting you here because your blocklist contains self-hosted mono-user personal instance domain and because i'm a European citizen and my own domain is on this list, per se, such a list is covered by the GDPR because such a domain is a PII (personal identifiable information).
From now, this list infringes the GDPR by many means :

scrapping of public source (article 6)
no compatible legal basis with the original source (article 6)
no compatible purpose with the original source (article 6)
false or missleading information (article 6 & 16)
no information of the subject (article 14)
no right to oppose (article 21)
no right to object (article 17)

Given all those violations, you have 1 month to proceed with removal of all my PII from this repository based on article 15 (access request), 16 (right to rectify false information), 17(c) (right to object), 17(d) (unlawfull processing) & 21 (right to oppose) of the GDPR. Without any action in this timeframe, I will open a case on my national data protection authority (CNIL).
NB: For your information, and you will note the dramatically irony of this situation, but currently my Mastodon bio is not terrorism support but just a joke about the fact my own government start classify everybody as terrorist without any documented or factual criteria and was then sentenced by our constitutional concil to drop this bill
https://www.nextinpact.com/article/25371/103093-le-delit-dentreprise-individuelle-terroriste-devant-conseil-constitutionnel
Any resemblance to existing characters or situations is purely coincidental… :wink:

Your bot identifying blocable domains is not working at all, not to mention the multiple infringements to privacy and associated laws in multiple jurisdiction. Also, posting plain text blocklist on github is poor practice, dangerous, and harming individuals whose domains are listed.
You still have a chance to avoid lawsuit...

For any curious fedi admins reading this issue, the domain in question is imirhil.fr.
Please share this domain with other instances so they can add it to remove it from their blocklists.

I will happily remove it from mine, aeris, just please identify the Natural Person in question.

It's identifiable and not identified. And my state identity can be easily find.

Is this supposed to be a DSAR?
I am sorry, I'm not certain the person making the Article 15 request here (if making such a request on Github is even considered legally binding) is the person who claims I hold personal information about them.
A github account can be owned by anyone, as can a domain name.
Proof of identity could be difficult. Normally in such a situation, if I had the personal email address of someone stored on my server, I'd ask them to log into their account to verify their ownership of the data. Since I don't hold any such data, I am not certain how to verify proof of identity from a github account.
Even if the domain name in question had WHOIS information traceable to a specific person (and it doesn't, it's anonymous) ownership of a domain does not mean that the user of a single-user instance on one of the subdomains of that top-level domain is the same person. It would be fallacious to assume so, since the person who registers a domain might do it on behalf of a friend, family member, or colleague.

I think you have this a little backward.

All is proofed here : https://keybase.io/aeris

I just see a page with a lot of links on it. I'm not sure how that proves anything.

It's cryptographic proof with every proof verifiable.

Proof for Github is here
Domain is here

Okay, so please explain how you're going to prevent your so-called "identifiable" domain name from appearing on my server as a normal part of federation?

GDPR is per purpose. Normal federation is another purpose so another legal basis. And no trouble with that. Only trouble with blocklist sharing.

You should probably keep reading that GDPR thing, I bet you'll discover it has some things to say about whether it affects things that aren't businesses.

There is no business condition. Individual or NGO was fined too.

For listing a domain name? I don't believe you.

On this topic I will be the first for private individual, but there is many cases with blocklist and other "sentence" considered as unlawfull.
I already fined an association for mailing without BCC, and you will find other private individuals fined to 10.000€ here

Currently 103 fines on private individual.

As soon as you process PII, you must comply with all the GDPR. Whatever the purpose or context. There is no exception at all (except one for private purpose, which is stated by CJEU as not compatible by design with social network or publication purpose).

If you want some cases about public blacklist trouble:

association publishing list of doctors suspected of medical error here: unlawfull, forced to remove data and from search engine
government publishing list of people fined for canabis owning here : unlawfull, forced to remove data
employer using blacklist for employee screening [here] : unlawfull, fined 40.000€, forced to stop processing

💤
None of those are comparable.

There are. Same trouble, blacklist are not lawfull without balancing right of people in the blacklist. It's the same trouble in the 3.

And typically the first is very very similar to your case.
An association acting for public interest, publishing publicly a blacklist, without real control of reason of blacklisting nor data qualification, and without balancing right of people inside. Unlawfull, illegal, and forbiden.

I'm not seeing any similarity.

So just do nothing, and see you in court.
My current GDPR infringment list on my national data protection authority : https://twitter.com/aeris22/status/1604148288151642112
Will be very pleased to add one more row on it.

Sounds like someone here has read more the RGPD than the other...
And imagine you are saying that because you are thinking it doe apply to commercial purpose only, while there are plenty examples proving you were wrong
Imagine you are not honest and still don't admit you were wrong.

And take into account that managing social network or related stuff is not neutral from a legal point of view.
LCEN in France, ePrivacy, GDPR, DSA, DMA and 2000/31/CE in Europe, DMCA in US, just apply to you.
 DMCA address on my instance will work for this too.
So what's your stance on sovereign citizenship? I feel like that would be right up your alley.

Unbelievable.
You flagged something wrongly, just admit it, fix your mistake and move on.

🥱 I'll take you seriously when you mail me something I can actually show to my lawyer. DMCA address on my instance will work for this too.
So what's your stance on sovereign citizenship? I feel like that would be right up your alley.

 This has nothing to do with it. This is just plain basic privacy right under GDPR.
"sovereign citizenship" 😂 ça you be more off topic?

Unbelievable.
You flagged something wrongly, just admit it, fix your mistake and move on.

I'm sorry, was this addressed to the owner of imirhil.fr or someone else?

Your instance says:

Oliphant skews far to the left, many members are actively anti-fascist, anti-racist, and accept no trolling or harassment--and rarely federate with any servers who do. (If we do so, it's probably an accident or oversight.)

But adding single-user instances to your blocklists might be considered targeted harassment.

But adding single-user instances to your blocklists might be considered targeted harassment.

It's even more. Currently blocking such instance must be address as an account and not an instance.
Admins just divert possibility to block the account at instance level. If I was on mastodon.social for example, I won't be ban (spoiler : I have account on this instance too).

yawning_face I'll take you seriously when you mail me something I can actually show to my lawyer. DMCA address on my instance will work for this too.

You have. The first post is a legal request.

Maybe for Oliphant. Not me.

@aeris A random GitHub issue is not a "legal request". If you really think your baseless threats truly have any kind of value to them, consider contacting GitHub support and/or file a DMCA claim.
Your "my personal domain is on a random list and violates GDPR" makes zero sense. Does that mean malware IOC lists of domains, IP addresses, hashes, etc violate GDPR because they're "scrapping of public source (article 6)" (your words)?  How about those C2 malware bots that post when an HTTP endpoint hosting malware? Violates GDPR too? Hope you're ready to take down the entire cybersecurity industry then if you think this is a "GDPR violation".
Seems like you're using FBA here. You complain about "scrapping of public source (article 6)", but the tool made by kiwifarmers you're using is literally doing that. By your own words, isn't that also a GDPR violation? You gonna take that down with your baseless legal threats too? Or are you going to eat your own words and just continue to have your followers brigade this issue.

@aeris A random GitHub issue is not a "legal request". If you really think your baseless threats truly have any kind of value to them, consider contacting GitHub support and/or file a DMCA claim.

Your "my personal domain is on a random list and violates GDPR" makes zero sense. Does that mean malware IOC lists of domains, IP addresses, hashes, etc violate GDPR because they're "scrapping of public source (article 6)" (your words)?

No, because this is not PII in such a case, it's probability 0 for a malware or IOC to be operated under personal domain or IP, just… because people doing malware try as most as possible to NOT being identifiable.
The GDPR only apply for PII, so no PII, no GDPR.

You gonna take that down with your baseless legal threats too?

In this case, there is a little difference. The source PII is the blocking domain for single user instance and not the blocked domain. Blocked domain is also a little in trouble from GDPR point of view here, but from collateral damage.
And so, I will be happy to take legal action too (and against kiwifarmers, double interest)… if I can. Legally I can't because at this time, my own domain is NOT scrapped or stored directly, so I have legally no interested in actying and only a single-user SCRAPPED instance can take legal action. Any takedown action from me may be refused by an DPA.

 Bold choice.

Would you have happened to have ever heard of the Streisand Effect?

@aeris
Please use this form to file a DSAR request
I am closing this issue.


Permission tree update process will take an abnormal time when the result set is vey high from the DB side

Description:
Permission tree update process will take an abnormal time when the resultSet is very high from the DB side (when there are a large number of rows as a result for query [1]). This issue is noticeable for DB types which have cursor data fetching mechanism (fetch the defined number of rows and start the resultset looping without waiting for the completion of the data extract process. eg: oracle) with network latency.
In oracle, we can control the fetching row size using defaultRowPrefetch property. If the result set of [1] is very high there can be an unavoidable delay even we tune the resultset looping time using the relevant property.
Therefore we need to re-consider the logic/query[1] we are using to retrieve the permission data from the DB side.
Please take this issue as a future reference
Affected Product Version:
All kernel versions
Steps to reproduce:
This is a sample test using product apim

Setup wso2 apim + oracle setup
Set a minimum value for DefaultCacheTimeout in /repository/conf/carbon.xml (for Key manager node) to observe the issue with a small-time interval (eg: 1s)
The below query [1] should return a large number of rows (>50000). (To do that you can create multiple APIs with a large number of API documents or import an existing DB dump which contains a large amount of data)
Execute permission update process (you can invoke an API, start the server, invoke /services/APIKeyValidationService directly, etc)
Observe an abnormal time to complete the result set loop [2].

SELECT UM_ROLE_NAME, UM_RESOURCE_ID, UM_IS_ALLOWED, UM_ACTION, UM_DOMAIN_NAME FROM UM_PERMISSION, UM_ROLE_PERMISSION, UM_DOMAIN WHERE UM_ROLE_PERMISSION.UM_PERMISSION_ID=UM_PERMISSION.UM_ID AND UM_ROLE_PERMISSION.UM_DOMAIN_ID=UM_DOMAIN.UM_DOMAIN_ID AND UM_PERMISSION.UM_TENANT_ID=-1234 AND UM_ROLE_PERMISSION.UM_TENANT_ID=-1234
https://github.com/wso2/carbon-kernel/blob/6843f35704db771446067b4ae55052c2ad7b162e/core/org.wso2.carbon.user.core/src/main/java/org/wso2/carbon/user/core/authorization/PermissionTree.java#L1133

We need to re-look at the need of loading total permission tree.
We can make the permission tree branch loading lazily, when a permission of a particular resource is queried.
This will make a limited set of records loaded at a time and cached. Not the whole set of permissions which most of them are not used.


Philips hue signe floor lamp

Hi.
It seems that this philips hue signe floor lamp is not supported. Any plans to implement this?
https://www2.meethue.com/en-us/p/hue-white-and-color-ambiance-signe-floor-light/4080248U7

@Goliath2000 please add the device as a simple zigbee and go into the advanced config of this added device and post details here (Producent ID, Product ID, Apparaat ID, Profiel ID) so they can be added as well.
Product ID: xyz
Device ID: xyz
Profile ID: xyz

@Goliath2000 please add the device as a simple zigbee and go into the advanced config of this added device and post details here (Producent ID, Product ID, Apparaat ID, Profiel ID) so they can be added as well.
Product ID: LCF003
Device ID: 528
Profile ID: 49246

Thanks @Kvasy123, added to list of devices to be added.

https://github.com/sebbebebbe/com.philips.hue.zigbee/issues/174


network_tree vms missing quadicon on hover

There should be a vms icon displayed when mouse hover on vm in network_tree. It is kinda there but never shows.
Code is here host/_config.html.haml, miq_dynatree.js -  function miqOnMouseInHostNet(id) and function hoverNodeId(id)
@himdel have a look?

@miq-bot assign himdel

Braindump for now...

hoverNodeId can't handle compressed id's (easy fix)
the main reason it fails is missing getAbsoluteTop - but the styling is wrong anyway
should probably also handle templates, not only hosts and vms (different prefix - t)
the styling is wrong wrong wrong
creating an #outer and #inner ids with each quad element is ..more wrong :)

Fix for first thing on braindump: https://github.com/ManageIQ/manageiq/pull/8081

New issue introduced:


Window stops responding on "check for updates" in extension viewlet

Just run "Check for updates"
Window stops responding for a minute

I can repro it consistently but @ramya-rao-a can't. Let me know how I can help debug it...

@roblourens Can you get the logs (window and shared) and show me? May be enabled log level to trace to get more info.

Duplicate #54041


need to be able to link to a track

need to be able to link to a track
should auto-open the modal
url should change when clicking an event (& opening a modal) to allow easy copy pasting
easiest:  <url>#<event-filename> (eg <url>#c_1_data_and_ipfs-1)
nicer: <url>#<event-name-slug> (eg <url>#data-and-ipfs-mobile)

working on this now


Delete other instant files (.clean) as well during commit archival

Will take a closer look .. Does this now cover .compaction as well?

not yet -
this only cleans up copy on write metadata. #165 will refactor archival of all instants,  archive all the instants into a archive timeline log (not just commits). This PR just deletes old .clean files, which is needed for copy on write now.


add config field for number of words in passcode, and added english w…

…ord list

@adriaanvanrossum remember to add
  "REDIRECT_TO_GOVERNMENT": false,
  "PASSCODE_LENGTH": 3

in the config before next release

We need this https://github.com/BustByte/coronastatus/issues/313 ;)


Kwxm/metatheory/fix test2

plutus-metatheory:test2 was disabled in CI and would fail if you ran it locally.  This should fix the tests, after a fashion.  Those tests (now renamed to test-detailed)  run various Agda and Haskell evaluators on some examples and check that the evaluators agree.  The problem was that the test was attempting to re-parse the output and compare ASTs, but the Agda evaluators can only output textual PLC/UPLC with names (often just A or something similar) annotated with de Bruijn indices, and the parsers can't handle those.  To work around this I've modified  the plc and uplc evaluators to be able to output textual code with de Bruijn indices as well and modified the tests to compare canonicalised versions of the textual output (squashing up all the whitespace and replacing variables with just their de Bruijn indices).  This is kind of horrible, but it does work and saves us from having to worry about alpha equivalence.To fix it properly I think we'd either have to make the Haskell parser accept de Bruijn indices or get the Agda code to output unique names, and neither of these seem especially easy.
I've also rewritten the tests to use Tasty.HUnit, which we use quite a lot elsewhere: previously they were using something called detailed-0.9, which seems to be an obscure type of test suite that cabal knows about.  I don't really know what that is, but I've  been able to replace it with exitcode-stdio-1.0, which we use everywhere else.
I messed about with project.nix to try to re-enable the tests in CI, but I'm not sure if I've done that correctly.
I also deleted some files in the top level of plutus-metatheory/test that appear to be unused.

This is issue #6406.

This is issue https://github.com/IntersectMBO/plutus/issues/6406.

@kwxm you can write "Fixes <link-to-issue/#issueno" to automatically close the issue when the PR gets merged

This is issue #6406.

@kwxm you can write "Fixes <link-to-issue/#issueno>" to automatically close the issue when the PR gets merged

Ah, thanks.  I wondered if there was some way to make a PR and an issue interact sensibly.


https://ogr.iu.edu.tr/

https://ogr.iu.edu.tr/ correct student mail domain. University official domain is https://istanbul.edu.tr. in Turkey every university domains must be end with .tr.

@gbirkocak I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host ogr.iu.edu not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


Making the most of Concepts pages

What is the problem to solve?
Now that we've built them, how can we make the most of these pages?
What are the objectives?
Look at traffic sources to pages with format “/concepts/”. If social, would mean editorial might find it worthwhile to tag more stories for concepts. If direct, could indicate these users are researchers.
If the analysis is not conclusive, try using the top concept pages identified in #10194 .
What research technique is required?
GA4

Moving to Icebox as this work is dependent on Concepts being linked to from other pages (aside from Works pages)

Closing as by the time the lead digital analyst is in post this will need re-looking at.


Add to "Apps using Lib" section

Hey there,
today I will release my note app Neuronizer to production on Google play.
As I decided to use your library it would be nice to be listed in the app section.
Here are the links:
https://github.com/djuelg/Neuronizer
https://play.google.com/store/apps/details?id=de.djuelg.neuronizer
Greets,
djuelg

Moved: https://github.com/apl-devs/AppIntro/issues/325
@djuelg


Remove reply from handle_cast documentation

Just a little documentation fix.
Great lib. I'm rebuilding my connection in rethinkdb-elixir on top of it.

Thanks mate


Add integration test for reindex

We found some bug when reindex a non-KNN index to KNN index with neural-search ingest pipeline.  We fixed this issue https://github.com/opensearch-project/ml-commons/pull/1418.
It will be best if neural-search can have some integration test to cover reindex. So we can catch such issue earlier.

The bug was in ml-commons. Ideally, test should be in ml-common imo. We don't expect every ml-common dependency plugin to add reindex integ test to catch a bug in ml-common?

It will be challenging for ml-commons , as run reindex needs Knn and neural-search plugins

Understand the difficulty to have reindex test in ml-commons but in a long term, we should consider implementing some very basic ingest pipeline inside ml-commons for integ test purpose.

@heemin32 I would beg to differ little bit here. As a ingest processor is defined in neural search plugin, ideally it is the job of the neural search plugin to validate all the cases in which that processor can be run. Now running the processor on re-index is another way to calling that processor. Hence the integration tests should be in neural search for re-index, because ML Commons cannot cover all the cases in which how the processor can be invoked.
Having said all the things above, one thing we definitely need from ML Commons Plugin or MLCommons client side, in terms of security how the apis can be used. This will ensure that its not the clients who are letting MLCommons know that something is broken.

Having said all the things above, one thing we definitely need from ML Commons Plugin or MLCommons client side, in terms of security how the apis can be used. This will ensure that its not the clients who are letting MLCommons know that something is broken.

My point align with this comment. MLCommons should be able to find if something is broken without relying on integ test in other plugins. Neural search should have re-index integ test to increase test coverage but MLCommons also should have a test case to cover the bug regarding this PR https://github.com/opensearch-project/ml-commons/pull/1418.

@heemin32

implementing some very basic ingest pipeline inside ml-commons for integ test purpose.

My comment was mainly around this.

@heemin32

implementing some very basic ingest pipeline inside ml-commons for integ test purpose.

My comment was mainly around this.

Just one of option to catch the bug inside ml-common itself.


Rotate axis issue with 0.4.11

When axis is rotated in c3js (0.4.11 pre release) multiple x axis lines are visible.
axis.x.height is also not working when axis is rotated.

I am also seeing these issues with c3 JS 0.4.11

https://github.com/masayuki0812/c3/pull/1294 fixes 1. For 2, I'm investigating now.

For 2, padding.left should be used. So, all seems to be ok now.

This seems resolved, so closing.


Interface that merges with Namespace (nee. Module) is labled as Module in output.

If you have the following declarations:
interface Foo<T> {
  foo: T;
}
namespace Foo {
  export function createFoo(): Foo { … }
}

TypeDoc generates documentation that reads:

Module Foo<T>

And classifies this as a module, rather than an interface (or a combination of the two).
Note that in VS Code, whenever there are merged declarations they appear in quick info like this:
interface Foo<T>
namespace Foo

Related to #472


Add distributed_context

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

Project coverage is 100.00%. Comparing base (206b989) to head (4c6ebd6).
Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #435   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          126       126
  Lines         3645      3663   +18
  Branches       277       278    +1
=========================================
+ Hits          3645      3663   +18

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


Create Sequence Diagram

Create a sequence diagram for the system.

Sequence diagram for featured services, communication, like service and recommendations is created.

Sequence diagram for Admin Dashboard is created

.


skip bad lines

This is a benchmark review for experiment review_of_100_reviews_20240409.
Run ID: review_of_100_reviews_20240409/benchmark_2024-04-09T10-24-34_v1-16-0-95-g65fefaeef-dirty.
This pull request was cloned from https://github.com/kurianbenoy/samam-ml-verification/pull/1. (Note: the URL is not a link to avoid triggering a notification on the original pull request.)


fix(dspy): added type check on evaluator result

signature_opt_typed.py previously assumed that the result of evaluator() was necessarily a float, but if the user sets the flags return_all_scores or return_outputs, it returns a tuple, which is not sortable.

Thanks @jmadden12 !


fix #3106  MSVC handling MSVC_BATCH with targetdir which requires escaping

MSVC handling MSVC_BATCH with targetdir which requires escaping.
If the target dir has a space in it and using batch compiles then the
/Fo argument must end with two dirseps
/Fo".]target dir\\"

Fixes Issue #3106
Contributor Checklist:

[x] I have created a new test or updated the unit tests to cover the new/changed functionality.
[x] I have updated master/src/CHANGES.txt directory (and read the README.txt in that directory)
[ ] I have updated the appropriate documentation

Coverage increased (+7.6%) to 66.31% when pulling b42fd7c0e76ca252ce778e5258be1e770561704d on bdbaddog:msvcbatch_gh_issue_3106 into e7b15e2288afa13cab09943bb2a5fa723cfa789e on SCons:master.

Coverage increased (+7.6%) to 66.31% when pulling b42fd7c0e76ca252ce778e5258be1e770561704d on bdbaddog:msvcbatch_gh_issue_3106 into e7b15e2288afa13cab09943bb2a5fa723cfa789e on SCons:master.

Coverage increased (+7.6%) to 66.31% when pulling b42fd7c0e76ca252ce778e5258be1e770561704d on bdbaddog:msvcbatch_gh_issue_3106 into e7b15e2288afa13cab09943bb2a5fa723cfa789e on SCons:master.


Improve certificate error screen (part 2)

Insert zero width spaces to host names so the text renderer can break them in convenient places if necessary
Display the server name in bold in the error message

Looks nice. We'd have to check if that works with Weblate. We'd lose the lint checks for format strings, though.


Wrong link for CONTENTdm template on homepage

When you click on CONTENTdm under Templates on the homepage it brings you to https://collectionbuilder.github.io/skins/ instead of https://collectionbuilder.github.io/cdm/

Fixed it


Add Usernames_to_Exclude__c field to Trigger_Handler__c

And its related logic. Same as in NPSP. (The original creation of the field and its related logic is not very easy to find now because the field was renamed: https://github.com/SalesforceFoundation/Cumulus/commit/3c99e7df8856ffd4b2b92fe984e38070b8c4800f).

**lurch: add

Tracking W-021068

Included in beta release 1.48 (Beta 3)

Included in production release rel/1.48

Closing this as it was included in a prior EDA release.


[DF] Port Join logic to Datafusion

Is your feature request related to a problem? Please describe.
Remove references to Calcite in join.py and replace those references with correlating references to Datafusion. This also includes any base logic that might make use of Join syntax.
Describe the solution you'd like
Replace instances of Calcite LogicalJoin with instances of Datafusion Join and ensuring the result still behaves as expected.
Describe alternatives you've considered
None
Additional context
Success criteria means all pytests passing for test_join.py

Closed with PR 512


move to 7.15.2 + stop using AUTOREV

It seems it's a better practice to use a static commit id than AUTOREV.
I guess it's debatable, but I feel it's better to decide explicitely when you change version.

Thanks for the comment. I get your concern.
I would prefer the version to be explicitly set in a .bbappend file for these reasons:

Security updates
fix upcoming build issues


NEW Add box shadow to focussed and hovered elements in the editor

Resolves #270

I know @newleeland mentioned "just use something existing" but the designs had a much more defined shadow on the top. I think this is something we should address now rather than later if the UX peeps are just going to want changes.
Maybe @newleeland can provide some feedback before merging?

Sure. We have five variations of box shadows in core already, if this needs a sixth we should add it there


Update datastore id and image size in downloader config

Before publishing the downloader config from volumemgr, datastore id and image size need to be updated.

Integration test result:
https://jenkins01.devt.zededa.net:8080/job/Smoketest-EveImageCheck/218/testReport/

Doesn't some of this assume that we can cancel that 40G download? The downloader doesn't have such logic AFAIK

The downloader doesn't have the logic of canceling the running download process.


Role delete idempotent

Modifies role.delete() to check if role exists and no-ops if it does not.
I found this when testing the CloudWatch event stuff.  kappa delete failed when run more than once after an error.
Also cleans up some of the calls inside the Role class so that use of the IAM service is more consistent.

Maybe worth adding some sort of test for this?

Happy to add tests, although I don't see much existing test infrastructure here.  Would you suggest augmenting the test_deploy.py?  Or testing role.py with mocks?  Or something else?
I'm not familiar with the placebo tool that's used by test_deploy.py but I can dig in and figure it out if needed.

Testing role.py with mocks works for me :)

Great - I have some mock tests ready, but it looks like the placebo tests in tests/unit/responses/deploy/ are blowing up because list_roles is no longer called.
Do you know the proper procedure for maintaining that stuff?  Do we re-record them somehow?

If we don't call it anymore, remove that part of the test.

I'm looking at this test:
https://github.com/garnaat/kappa/blob/develop/tests/unit/test_deploy.py#L46
It uses some recorded JSON files and plays them back (faking out the AWS endpoint, it appears).  So I'm not sure how one preps new JSON responses when the call sequence changes.  I'm guessing this is what the kappa --record-path flag is for.
I was able to get things to pass by having _get_role() guard on the shape of the response:
    def _get_role(self):
        try:
            response = self._iam_client.call('get_role', RoleName=self.name)
            if response and 'Role' in response:
                response = response['Role']
            return response

But that's sort of wacky.  Since that worked it makes me wonder what these tests are really verifying.

I've pushed the unit tests (which I'm happy with), and a separate commit to role.py that doesn't change the behavior of the function when called against AWS but causes the test_deploy.py test to pass.
I'm not happy with that commit but pushed it just so you could see the difference.  I don't think it's terrible to merge that change, but I think a better solution is to re-record the files that drive test_deploy.py, or perhaps reconsider that test approach entirely.  The fact that my patch causes this test to work is surprising to me, as the test was recorded against a different set of AWS calls.

Looks like my random string function only works on 2.x - I'll fix that.


renamed module_name and target_name

The idea is to make sure you don't use common names that could be used by other modules.
This becomes important when you use "host override" feature of node-pre-gyp

Is this still relevant ?

Please can you  forward this change to:
https://github.com/abandonware/node-bluetooth-hci-socket/
FYI an updated version has been released:
<EMAIL_ADDRESS>


New button on Service Delivery related list

Critical Changes
Changes
Users will be able to create new Service Delivery records manually from the Service Delivery related list on Service Sessions.
Issues Closed
New Metadata
Deleted Metadata
Definition of Done
Refer to Asteroids DoD document to see any additional details for the items below
- [ ] Place holder data is incorporated (Refer to Case Management Sample Data document)
- [ ] Base axe-core JEST test included for any new LWC (No critical violations)
- [ ] CRUD/FLS is enforced in Apex
- [ ] Update & Test permission sets to account for new features and CRUD/FLS
- [ ] All modified classes are unit tested (Apex) at 100% coverage
- [ ] UX approval or UX not necessary

[x] PR contains draft release notes
[x] Attach work item to the pull request with Lurch **Lurch:attach W-000000 or **Lurch:add
[ ] All acceptance criteria have been met

[x] Developer
[ ] Code Reviewer
[ ] QA

[ ] QE story level testing completed

W-8225771


Add warning that nodelocal doesn't work in multi-node clusters

Closes #4982.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/39940cf1b7adfe3c8cf56bad1689d7c1ca246020/

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/a0d582d857b0bc25281a5a67baa902a4e8109570/

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/ea2360569a8b1961f133202dcf751c121cbd791e/

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/dd09fb5950ba4a06a58d45f1c0141672337e75f5/

we could explicitly say something like "thus, unless the --external-io-dir is configured to point to an NFS mount or other network-backed, shared storage, using nodelocal in a multi-node cluster will likely not work as intended."

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/207f3b650f8bfa8718123858b13221b378d3caf8/

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/d316a0b16449b15870944dd0c4c3722d0011cc09/


Question: App Registration Secret

Hi Jens,
Would be great if an Azure Keyvault secret could be used for the Access management Setup. Is that possible?

Great suggestion. Thank you.
Will think about how this capability can be added.
Ideally AZK will be an optional capability. Not all customers can or want to use AZK


🐛 BUG: Creating and Downloading D1 Backups Not Possible

What version of Wrangler are you using?
2.6.1
What operating system are you using?
macOS Ventura 13.0.1
Describe the Bug
It is currently not possible to create and download D1 backups through wrangler.
Reproduce

Create a new backup:

wrangler d1 backup create awesome_database

┌─────────────────────────────┬──────────────────────────────────────┬────────────┬─────────┬───────┐
│ created_at                  │ id                                   │ num_tables │ size    │ state │
├─────────────────────────────┼──────────────────────────────────────┼────────────┼─────────┼───────┤
│ 2022-12-07T12:47:01.802737Z │ f522a065-a429-4b45-bfb5-679eb0476969 │ 18         │ 1.38 MB │ done  │
└─────────────────────────────┴──────────────────────────────────────┴────────────┴─────────┴───────┘

Download the created backup:

wrangler d1 backup download awesome_database f522a065-a429-4b45-bfb5-679eb0476969 --output awesome_backup.sqlite

🌀 Downloading backup f522a065-a429-4b45-bfb5-679eb0476969 from 'awesome_database'

✘ [ERROR] Failed to download backup f522a065-a429-4b45-bfb5-679eb0476969 from 'awesome_database' - got 500 from the API

(Both the name of the database and the id of the backup have been replaced.)

Furthermore, running wrangler d1 backup download <name> <backup-id> fails with the following message:
✘ [ERROR] The "path" argument must be of type string. Received undefined

Only when adding --output the command runs. This is misleading, as under the --help page it states that this option is optional:

The creating and downloading backups issue has been resolved - there was an issue with the API
We still need to improve the --help in wrangler as per the second part of the issue.

Appreciate the fast feedback and help!

same issue

@YankeeTube can you describe the issue you're having?

Not sure if that's what they meant, but I just noticed that the API is giving me a 500 again. Both in wrangler and the dashboard:
{
  "result": null,
  "result_info": null,
  "success": false,
  "errors": [
    {
      "code": 7500,
      "message": "Internal error"
    }
  ],
  "messages": []
}

For my future reference, where is your preferred place to have us report issues that have less to do with the wrangler integration and more with the service (e.g. D1) itself? This repo, the Discord server or the Discourse instance?

Discord is preferred @martindisch!
Sorry about the API issue over Christmas/New Years - should be resolved now.

Closing this issue as resolved in https://github.com/cloudflare/wrangler2/pull/2548

Thank you!


Upgrade Nodes sometimes fails with "Internal Error"

I've attached a screenshot of the error below:

I don't know what triggers this problem but I'd say close to 50% of the time Upgrade Nodes has this problem and then I have to drag in all the new Operations and set their parameters and links to other Operations again

I've seen it before. Can you see traceback in terminal?

Never mind, it will be resolved in 1.2.0

now should be fixed


Added Favicon in index.html

Done !
Added Favicon made with Ideogram AI
Hope you accept it !

Response Please ! Have you seen my work !

Response Please ! Have you seen my work !

Fix it so I can preview


2 New options - Dir ignore Prefix and Ignore Sort

Option Dir ignore Prefix :
to be able to use, for example, the prefix "_" in included files.
Option ignore sort :
to be able to manually set the include order

This looks good to me. (Perhaps 'ignore prefix' might need to be renamed to something like 'ignore path' or 'ignore directory' or something?)
But, if you update the documentation in the README, I'll merge.
Thanks for the contribution and for dealing with my code mess! :D

Hi there!
I'm the one who thank you because your npm was very helpful for me. I needed a command to compile my JS with includes (like Scss does)  and yours was the only one I could find and I'm very glad I did =)
About the ignore prefix, I taught it was the more obvious name because :
the 'dir' part is because the option is only used with 'dir' option
The 'ignore-prefix' is for starting chars like '_' so it is not a file nor a directory but a part of the filename. Maybe 'dir-ignore-fileprefix'?
What you think? Or just 'ignore-fileprefix'?
This is the first time I work on a npm project so I not good at it but I'm a php/Javascript programmer myself so I can handle a bit of Javascript and it was a pleasure to help =)

are you there?

Sorry for the delay.
I am closing this pull-request for now because I have rewritten a lot of the code.
And I think that even though I didn't implement your specific feature request, it is possible to do what you wanted to do.
Instead of having the ability to manually set the concatenation order programmatically, you can now use a new directive // include which let's you include the file directly at the location of the comment. This way, you can have a completely custom order just be rearranging your // include statements (perhaps in an extra "wrapper" file or "bundling entry point" file.
(As for the "ignore prefix" option, it is no longer possible to include an entire directory - you have to specify an entry file. However, when/if the directory feature gets re-added, we may have to revisit this option.)
Let me know if you have any questions.


examples of questionable inferences from reasoner

Starting a ticket to capture examples where EQ statements produce questionable inferred parents
impaired contractility of urinary bladder detrusor smooth muscle (MP:0000741) reasoner suggests parent impaired contractility of intestinal smooth muscle (MP:0004005)

The EQ for abnormal urine enzyme level is using the GO term 'catalytic activity' this doesn't seem quite right and leads to the inferred parent abnormal enzyme/coenzyme activity

abnormal spermatogonia proliferation has the inferred parent 'abnormal primordial germ cell proliferation'

'impaired basement membrane formation' has the inferred parent 'abnormal metabolism'

abnormal vascular endothelial cell development has the inferred parent 'abnormal cell differentiation'

abnormal Harderian gland pigmentation has the inferred parent 'abnormal eye pigmentation'

abnormal renal tubule morphology has the inferred parent 'abnormal kidney epithelium morphology'

ocular albinism inferred to be child of abnormal metabolism due to use of melanin metabolic process in the EQ

abnormal fetal cardiomyocyte apoptosis is inferred to be a child of abnormal cardiomyocyte apoptosis but I don't think we want the fetal cell under the adult cell, may be an issue of the CL term used not matching the intent of the MP term

seminal vesiculitis has the inferred parent testis inflammation, I think this is due to an incorrect relation between testis and seminal vesicle in UBERON

Wow this is super useful! Thanks @sbello keep them coming!

maxillary sinus inflammation has the inferred parent osteomyelitis (bone inflammation, MP:0005190), I would not think of sinus inflammation as a bone inflammation but I'll need to think about this more and make sure I understand 'maxillary sinus'

increased and decreased systemic arterial diastolic/systolic blood pressure are inferred to be children of increased/decreased mean systemic arterial blood pressure - I think this is because the mean term is missing a nuance in the definition. We should probably review all the definitions under MP:0002842 and MP:0002843, note the diastolic ones currently have the rogue relation 'after' and I'm working on finding a real relation to use here.

absent pterygoid muscle has the inferred parent absent masticatory muscles - problem with absent meaning absent all of X vs absent some of X when there is more than one X
similar issue with absent teeth

intestinal obstruction (MP:0003270) has the suggested parent 'abnormal intestine morphology' the MP has this term under the physiology branch (child of abnormal gastrointestinal motility), obstructions do not have to be morphological


[Removal]:

Hotspot Name
Trendy Flaxen Chinchilla
Hotspot b58 Address
11YAh9WhaYpjoKzH1Pj91URqzfbRHVBiRPF5PADQLkMqpmA6bxm
Discord Handle
No response
Hotspot Manufacturer
PantherX
Removal Reason
I bought a used hotspot
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information
No response

We have taken into account this removal request in the next iteration of the denylist. It may take some time before it's processed. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.


Fails to compile if proc-macro2 is a dependency

I'm unsure which tree to bark up against, so I'm reporting this here:
If a crate has a dependency on proc-macro2 (any version, currently resolved to 0.4.9), compilation via cargo web build (on 0.6.4) fails with can't find crate for 'proc_macro' while compiling proc-macro2. The offending line in proc-macro2 is extern crate proc_macro;.
It fails for all targets, except native targets, on stable and nightly-2018-07-23.

This happens because the proc_macro crate is not distributed for the asmjs/wasm targets, i.e. here we have it:
$ ls ~/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib | grep proc_macro
libproc_macro-1ea4ba9e4f6cdbc1.so

and here we don't:
$ ls ~/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/wasm32-unknown-unknown/lib | grep proc_macro

You can easily reproduce this issue using only cargo:
$ cargo new foobar
$ cd foobar
$ echo 'proc-macro2 = "*"' >> Cargo.toml
$ cargo build --target=wasm32-unknown-unknown
   Compiling unicode-xid v0.1.0
   Compiling proc-macro2 v0.4.9
error[E0463]: can't find crate for `proc_macro`
  --> /home/kou/.cargo/registry/src/github.com-1ecc6299db9ec823/proc-macro2-0.4.9/src/lib.rs:50:1
   |
50 | extern crate proc_macro;
   | ^^^^^^^^^^^^^^^^^^^^^^^^ can't find crate

So if this is a problem for you then you should report it as an Rust issue.
However, why would you actually want to use the proc_macro2 crate with the web targets? AFAIK that crate is only useful when writing procedural plugins which use your native target instead of the one for which you're compiling for.

proc_macro was being pulled in due to a dependency on syn, which pulled proc_macro2, which pulled in proc_macro. There is a knob in syn to turn this off, so problem solved for now.

Reopening because I still think it's a bug with cargo-web:
This builds using cargo web build
[dependencies]
proc-macro2 = { version = "0.4", default_features = false }

This also builds using cargo web build
[dependencies]
stdweb = { version = "0.4" }

But with both dependencies
[dependencies]
proc-macro2 = { version = "0.4", default_features = false }
stdweb = { version = "0.4" }

... cargo web build fails due to missing proc_macro-crate. It does, however, build using cargo build.

Re-closing because after some tinkering I now think it's actually cargo's fault :-/


Cleanup whitespace in ssl_lib.c (tabs to spaces)

Nothing much to be said about this commit.

Looks better to me. I hope everything is correct now.

@dot-asm can you merge?  I am having trouble passing the commit hook again with this one.

I am having trouble passing the commit hook again with this one.

So am I... Hmm... It's midnight here, so I better postpone this till morning....

Committed. Thanks.
[Reason for failure was mixed-case e-mail address in our CLA list.]

@dot-asm it looks like there is still some case sensitivity issue with my email address. Did you change it to lowercase in the cla list ? Because now the cla-check fails again, see PR #4402.

Did you change it to lowercase in the cla list?

Yes, otherwise it was impossible to merge this request. But take it easy, it will be sorted out...

Don't worry, I am accustomed to computers having problems with my last name ;-)


RECEIVER_EXPORTED param added for registerReceiver API to support SDK34

PROBLEM AND SOLUTION:
registerReceiver API has mandated RECEIVER_EXPORTED param from SDK34 so added the param
VALIDATION:
Did sanity testing

@mgowtham - You may want to merge this PR as it is approved?

@mgowtham any plan to merge this PR?


🐛 JSON Parser: Unterminated string literal can make inner_string_text panic

Environment information
Unrelated, the issue is reproducible when nursery/noDuplicateJsonKeys is enabled.

What happened?
Our JSON parser allows JSON_STRING_LITERAL to include an unterminated string literal:
https://github.com/biomejs/biome/blob/a5a67ed6202c5a37a33641de949fbeb572a45381/crates/biome_json_parser/src/lexer/mod.rs#L573-L579
https://github.com/biomejs/biome/blob/a5a67ed6202c5a37a33641de949fbeb572a45381/crates/biome_json_parser/src/lexer/mod.rs#L620-L628
This will make the function inner_string_text panic when the unterminated string literal is just a single doublequote ":
https://github.com/biomejs/biome/blob/a5a67ed6202c5a37a33641de949fbeb572a45381/crates/biome_json_syntax/src/lib.rs#L118-L123
The function inner_string_text is called in the JSON nursery rule noDuplicateJsonKeys:
https://github.com/biomejs/biome/blob/a5a67ed6202c5a37a33641de949fbeb572a45381/crates/biome_json_analyze/src/lint/nursery/no_duplicate_json_keys.rs#L58
And this nursery rule is enabled by default when debugging or in a nightly build. So when changing the VS Code extension lspBin to a local debug build, and when a JSON file containing content like this is scanned by the extension, Biome will panic and keep reloading until it's killed permanently:
{
  "a": """
}

The above erroneous JSON can appear quite frequently when using code completion so it will kill Biome often.
Expected result
Unterminated string literal in an AnyJsonValue place should be a JsonBogusValue instead of a JsonStringLiteral.
However I don't know how to treat it when the unterminated string literal appears in the JsonMemberName place. The JSON grammar may need changing. And I don't have enough knowledge to modify the grammar.
Code of Conduct

[X] I agree to follow Biome's Code of Conduct

Wow, that's surprising!

Some ideas:

We could add a JsonBogusMemberName and AnyJsonMemberName = JsonMemberName | JsonBogusMemberName.
or, we could handle { "a } like a value with a missing key and colon. This could avoid introducing a new node.

Any opinion?

Some ideas:

We could add a JsonBogusMemberName and AnyJsonMemberName = JsonMemberName | JsonBogusMemberName.
or, we could handle { "a } like a value with a missing key and colon. This could avoid introducing a new node.

Any opinion?

I myself prefer the first option because I was expecting the parser to be strict and conformant to the spec, and leave ambiguous tokens as bogus nodes just as what Biome claims to do. But I also think the bogus part should be as small as possible so to not propagate the bogus state to the outer tree, and the parsing can be recoverable as soon as possible.
#2606 also reports other fail cases to trigger the error, that's also another reason why I think option 2 is not ideal because we'll have to evaluate all the possible fail cases and draw a line somewhere to decide whether something is unrecoverably malformed, which I think can be error-prone.

Same there. The first approach seems better because I think it is closer to the user's intention.
I suggested the second approach to limit the number of bogus nodes: It is unpleasant to work with bogus nodes.

I have started to work on this.
However, I wonder: why is it actually wrong to treat an underlined string as a string?
We consider unterminated objects and arrays to be objects and arrays. Should unterminated string be different?
The lexer gives an error. Could we just change inner_string_text to handle unterminated strings?

However, I wonder: why is it actually wrong to treat an unterminated string as a string?

Maybe because it can cause som unstable or unexpected behavior?
I haven't try if this would be an issue, but an example I can think of is when the string is ended with a newline without the quote, it is regarded as an unterminated string, but a formatter will still format it and it may remove the new line, and that unterminated string may then become something else and so the tree may be changed accidentally?

I haven't try if this would be an issue, but an example I can think of is when the string is ended with a newline without the quote, it is regarded as an unterminated string, but a formatter will still format it and it may remove the new line, and that unterminated string may then become something else and so the tree may be changed accidentally?

I am not familiar with the JSON formatter. However, the newlime is part of the string. Thus, the formatter should not remove ir?

Ok, I was wrong, the parser doesn't include the newline in the string. See an example in the playground.


Fix Debug Fail Go-to-def on JS class

Fixes https://github.com/microsoft/TypeScript/issues/57924#issuecomment-2016963512

@sandersn do you have an opinion on whether we should accept this PR or https://github.com/microsoft/TypeScript/pull/57628?


added aws kb server

Description
Addition of a new MCP server for AWS Knowledge Base retrieval, leveraging the Bedrock Agent Runtime to fetch information based on user queries.
Server Details

Server: aws-kb-retrieval
Changes to: tools (new tool added)

Motivation and Context
To enhance the capability of the MCP client by enabling it to retrieve information from AWS Knowledge Bases directly. This integration simplifies the process of sourcing relevant data for users.
How Has This Been Tested?
The server has been tested with an LLM client by integrating it into the Claude Desktop setup. Various scenarios, including different queries and Knowledge Base IDs, were tested to ensure functionality.
Breaking Changes
Users will need to update their MCP client configurations to include this new server and provide their AWS credentials.
Types of changes

[x] New MCP Server
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation update

Checklist

[x] I have read the MCP Protocol Documentation
[x] My server follows MCP security best practices
[x] I have updated the server's README accordingly
[x] I have tested this with an LLM client
[x] My code follows the repository's style guidelines
[x] New and existing tests pass locally
[x] I have added appropriate error handling
[x] I have documented all environment variables and configuration options

Additional context
This integration requires users to set up their AWS credentials securely. The server has been optimized for performance and security, with careful consideration given to error handling and edge cases.
This form is filled out based on the AWS Knowledge Base Retrieval server you've developed. Adjustments can be made as necessary to align with your specific implementation or any additional requirements.

I thought community servers should stay outside of MCP? I'm confused.


lsp: Fix inconsistent processing of ignores

In the LS linting, pattern/* would not work when it would in the CLI. This was related to how abs paths were resolved and is fundamentally caused by and issue with URIs being used as file names in the linter.
Now, all files from the LS lint are ignored based on their path with the workspace root URI trimmed.
https://github.com/user-attachments/assets/30118754-61d5-4265-b68b-9fac99c204d9

Passing URIs to a function that at least judging by its name expects a directory

I would actually like to rename the function as it's being used with URIs elsewhere already and is even mentioned in the comment for the function.
I will take another pass at this today.


Changelog didn't get updated with 55

Congrats with the beta release! 🎉
It would be helpful to know if there were any updates to the API. I couldn't find much here:
https://github.com/BohemianCoding/SketchAPI/compare/release/55
And the changelog didn't get updated:
https://github.com/BohemianCoding/SketchAPI/blob/release/55/CHANGELOG.json

It's everything that is in "unreleased".

Ok! That's what I figured. Thanks! 👍

Also, not really part of the API but related to plugin dev:

The way errors were logged right now was pretty bad:

const err = new Error("This is an error")
console.log(err)
---
{ onRun@/***/Application Support/com.bohemiancoding.sketch3/Plugins/Untitled.sketchplugin:8:22
  line: 8,
  column: 22,
  sourceURL: '/***/Application Support/com.bohemiancoding.sketch3/Plugins/Untitled.sketchplugin' }

throw new Error('this is an error')
---
Error: This is an error
line: 2
sourceURL: /Users/mathieudutour/Library/Application Support/com.bohemiancoding.sketch3/Plugins/Untitled.sketchplugin
column: 26
stack: onRun@/Users/mathieudutour/Library/Application Support/com.bohemiancoding.sketch3/Plugins/Untitled.sketchplugin:2:26

It was not consistent and not pretty.
Now it's consistent (and match how errors are logged in Node):
Error: This is an error
    at onRun (/Users/mathieudutour/Library/Application Support/com.bohemiancoding.sketch3/Plugins/Untitled.sketchplugin:2:26)

There is a new action "HandleURL": https://github.com/BohemianCoding/SketchAPI/blob/release/55/docs/actions/HandleURL.md which is super useful for oauth, etc.
there will be the new documentation #421 hopefully


USHIFT-216: Fix kube-apiserver readiness check panic on kubeconfig error.

Errors returned by BuildConfigFromFlags were sent to the error
channel, but control flow continued despite the nil rest.Config
pointer.

@benluddy, could you link this PR to a JIRA issue please?

/retest

@benluddy, could you link this PR to a JIRA issue please?

Done. I'll also move the change from /healthz to /readyz into a separate PR with its own issue.

/approve

/lgtm

Also fixes USHIFT-20


ping is always 0

I couldn't get any results.
I tried with Irancell, MCI, and Zitel. also used multiple vless ws tls configs. configs do work in v2rayN but when used in the script return 0 ping with the same fragment settings.
i even tried running it while another vpn is connected(Amnezia) and still got 0 ping

I couldn't get any results. I tried with Irancell, MCI, and Zitel. also used multiple vless ws tls configs. configs do work in v2rayN but when used in the script return 0 ping with the same fragment settings. i even tried running it while another vpn is connected(Amnezia) and still got 0 ping

Thanks for your feedback and thorough explanation. Kindly test this modified version of the script which keeps xraylogs.txt for the entire run, and open the logs after the run is completed to check for other problem that might cause the issue.

Same problem here. Running script on window 11 and it returns 0.00 ping for each test

For Windows 11, pleases test This version of the PowerShell script and let us know if the issue is resolved.


How to Restore deleted photos from oneplus 5?

This is the first time OnePlus has collaborated with a retail outlet to sell its smartphones offline in India. The company first had an exclusive partnership with Amazon India, and then branched out the capacity to its official OnePlus online store. Only recently, did OnePlus launch its experience stores in the two cities, with a promise for expansion in more cities eventually. It also sets up pop-up stores in various cities during initial launch days for users to be able to try out the smartphone, and buy it there instantly. But that is usually a temporary affair, after which users had to rely on Amazon or OnePlus online store for the purchase of the smartphone. Now, for the first time, OnePlus will make its OnePlus 5 flagship available in Croma retail outlets across the country from September 19.
Method to Restore deleted photos on oneplus 5:
This Samsung Data Recovery not only can help you recover photos form oneplus 5 smartphones, but also restore data from oneplus 5 and so forth. What’s more, it allows you recover lost pictures from the SD card directly are just a few clicks. Also, it can be used to recover contacts, text messages, videos and whatsapp from oneplus 5.
More info;
Transfer Data Contacts from Android to OnePlus 5

khs dkjskdj hskd nsd jsk sjd jsd fsdd


Sample show mission path

show mission path in orange and also show start pin and stop pin.
Fixes #48.

Note: if you add "Fixes #48" to the PR description, it synchronizes it on the kanban board, so that it closes the corresponding issue when the PR is merged ;-).
"Connects to #48" would just synchronize them, without closing the issue when the PR is merged. That's when an issue requires multiple PRs.


Fix a crash in ImporterMesh::create_shadow_mesh for non-triangle surfaces

optimize_vertex_cache_func assumes the input is a triangle mesh; when an imported mesh contains points/lines/triangle strips, this code should be disabled.
Introduced in #94241
Fixes #96869

Aside: in the linked issue report, the primitives in question are lines; creating shadow meshes for point/line primitives seems dubious. However, it's still valuable for triangle strips in case they get through here through the importer flow, so this check is probably still a good idea even if we end up adding some early outs for lines/points in the future.

Thanks!


Adding cert checkers to sensitive profile

-all sei-cert checkers to the sensitive profile, as it seems to be a good default to enable all sei-cert rules (for minimally security aware projects)
-remove all alpha checkers from sei-cert guideline, so when one enables them do not get unstable warnings
-the severity level of some checkers were missing, these were added

@dkrupp Please resolve the merge conflicts.

the

OK. done.


New feature：highlight a node's neighborhood when it is clicked

Hi  Yomguithereal,
As I mentioned before,  I think it will be cool to add the following feature in ipysigma:
"... highlight a node's neighborhood when it is clicked, by drawing every other nodes as grey."
This feature is also introduced in the webpage of sigma.js :  http://sigmajs.org/,  you can take a look.
Thanks,

Hi Yomguithereal,
Any progress on this issue？
btw. I also introduce ipysigma to my colleagues and they think it is cool too.

Just published a v0.5.0 with the feature. Tell me what you think.

Great. Thanks for your work!
I tried the new version today and found that :
(1)  automatic layout function is gone：in the previous version，there is a play button at the corner to activate the layout function. I cannot find it now.  I think the layout algorithm should be one of the core components, it can provide better visualization.
(2) the highlight function: when I click one node，only the selected node and its neighbors appear in the graph and other nodes do not appear. How about keep all the nodes in the graph and differentiate them in different colors? For example, the selected node and its neighbors can be red, and the rest of the nodes can be light grey. In this way, we can see how the selected ones are related to the rest of nodes in the big picture.
(3) label of the nodes：in the previous version and the latest version, some labels of the nodes will appear, but not all of them. And it seems that it is random.  How about hiding all labels by default and showing the label only when mouse over it ？
Recently I also tried other visualization tools for network analysis.  I still think your product is my best choice : )  Again, thanks for your work.


Exception label is unresizeble, uncopyable and uncontrollable

When we have lets-plot exception with a big text
"IllegalStateException : Can't detect type of pattern 'Let's Talk About Sex (1998)	Drama
One Man's Hero (1999).movieId(mean)' used in string pattern '@{Let's Talk About Sex (1998)	Drama
One Man's Hero (1999).movieId(mean)}'"
The error label on the PlotPanel have a minWidth of 1560, and breakes the layout making it impossible to resize plot area.
Please

Make this error label multiline
Make it possible for users to copy the error message or even a stacktrace for reporting
Set manually the max width to something like 15, to save the layout from explosion
Provide a possibility to control it by developers that want to use lets-plot library with swing. Let them to override some "createErrorPanel(t: Throwable) to make it possible to provide our own LAF for errors with reporting and styles"

Done in this fix:

A better default "error message" component: multiline, selectable text: DefaultErrorMessageComponent

You can plug your own "error message" component by overriding createErrorMessageComponent() in PlotSpecComponentProvider

See example: MyErrorMessageComponent.
And BTW, when you are done with your custom "error message" component could you share its code? We'd love to make it a default "error message" component :)

This is how first draft looks, without UI/UX review:

Looks awesome :)


fix: adhoc filter 'equals' doesn't let you save

before

after

Applies to equals and does not equal

Codecov Report

Merging #9652 into master will increase coverage by 6.28%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #9652      +/-   ##
==========================================
+ Coverage   64.03%   70.31%   +6.28%
==========================================
  Files         532      182     -350
  Lines       28897    17728   -11169
  Branches     2758        0    -2758
==========================================
- Hits        18503    12465    -6038
+ Misses      10211     5263    -4948
+ Partials      183        0     -183

Flag
Coverage Δ

#cypress
?

#python
70.31% <ø> (-0.26%)
:arrow_down:

Impacted Files
Coverage Δ

superset/views/database/mixins.py
58.92% <0.00%> (-21.43%)
:arrow_down:

superset/utils/cache.py
45.83% <0.00%> (-20.84%)
:arrow_down:

superset/db_engine_specs/postgres.py
80.00% <0.00%> (-15.00%)
:arrow_down:

superset/db_engine_specs/mysql.py
78.26% <0.00%> (-13.05%)
:arrow_down:

superset/views/database/api.py
83.90% <0.00%> (-3.45%)
:arrow_down:

superset/models/core.py
84.47% <0.00%> (-2.39%)
:arrow_down:

superset/security/manager.py
88.81% <0.00%> (-0.34%)
:arrow_down:

superset/views/core.py
75.01% <0.00%> (-0.23%)
:arrow_down:

superset/connectors/sqla/models.py
88.36% <0.00%> (-0.16%)
:arrow_down:

...rontend/src/components/VirtualizedRendererWrap.jsx

... and 348 more

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 5e4c291...a70c528. Read the comment docs.

Addressed comments, @villebro or anyone, can a get a formal stamp? Also feel free to merge.


add either, option, try, validation traverse. improve the signature of validation.sequence

vavr has Future.traverse and Future.sequence, but for option, try, validation and either it has only sequence.
This PR adds traverse for all of these. Traverse is a pretty standard FP function (arguably even more standard than sequence).
The PR also slightly improves the Validation.sequence signature, regarding variance of the parameters (in a separate commit).

Codecov Report

Merging #2257 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #2257      +/-   ##
============================================
+ Coverage     97.17%   97.17%   +<.01%
- Complexity     5344     5349       +5
============================================
  Files            94       94
  Lines         12288    12303      +15
  Branches       1596     1596
============================================
+ Hits          11941    11956      +15
  Misses          193      193
  Partials        154      154

Impacted Files
Coverage Δ
Complexity Δ

vavr/src/main/java/io/vavr/control/Option.java
100% <100%> (ø)
45 <1> (+1)
:arrow_up:

vavr/src/main/java/io/vavr/control/Try.java
95.75% <100%> (+0.04%)
99 <1> (+1)
:arrow_up:

vavr/src/main/java/io/vavr/control/Either.java
100% <100%> (ø)
50 <2> (+2)
:arrow_up:

vavr/src/main/java/io/vavr/control/Validation.java
99.07% <100%> (+0.01%)
68 <1> (+1)
:arrow_up:

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 a9b7b0e...bb0150a. Read the comment docs.


Once an element is added to the list it can't be deleted.

Using QFormBuilder Demo, an item can be added to the end of the list and for a time it's available for deletion.  But as soon as the next element is added, the garbage can icon disappears from that element seemingly permanently.  So other than rearranging the order of elements, you must start over from the beginning if an element is incorrectly added.
While this state of affairs is OK for one-off designs, it fails in the case of where the an end-user is allowed to dynamically change element ordering and remove existing entries.
I'm considering adding elements for video, audio, images, embedded iframes, and markdown for an app where the end-user is dynamically adjusting the layout.
Can the garbage can be permanently (perhaps optionally) available for each element?

Oh my god you're right, that used to work properly but doesn't now. I'm not sure when that broke. I'll work on a fix and let you know when it's done.

This was a flaw in the Demo and I have fixed it. Thanks for finding it!


Gray image error

If the input image is grayscale， I think you should use len(image.shape) == 2 instead of image.shape[2] == 1

Hi there,
Thanks for your comment. When dealing with grayscale images, of course we usually only have two dimensions, e.g. (H,W). However, since this package is meant to be used with PyTorch CNNs, we still need a channel dimension-- for example, you can use a torch.nn.Conv2d layer on a image of shape (1, 1, 224, 224) (Batch x Channels x Height x Width). However, if you try to use a Conv2d on a grayscale image of shape (1, 224, 224) (Batch x Height x Width), you get this error:
RuntimeError: Expected 4-dimensional input for 4-dimensional weight ..., but got 3-dimensional input of size [1, 224, 224] instead. So, I'm going to keep mandating that grayscale images have a bogus channel dimension.


where to find video dataset ？

In Huggingface dataset， I only see the video uuid in json files. How could I fetch those videos?
Thanks!

Hi, the dataset is based on Ego4D (https://ego4d-data.org/). Just sign the licenses (https://ego4ddataset.com/ego4d-license/), and they will send you the aws credentials. Then pip install awscli ego4d, fill in aws credentials via aws configure. Finally, follow https://ego4d-data.org/docs/CLI/ to download the video ;)

Hi, the dataset is based on Ego4D (https://ego4d-data.org/). Just sign the licenses (https://ego4ddataset.com/ego4d-license/), and they will send you the aws credentials. Then pip install awscli ego4d, fill in aws credentials via aws configure. Finally, follow https://ego4d-data.org/docs/CLI/ to download the videos ;)

Thank you for your careful reply！

Another question. The json file:
 "2d0c81c8-d7ff-4240-9ee7-4c71c152fb0a": {
        "8a889987-06aa-4778-bcd6-acd52bfe188f": [
            {
                "time": 0.8326981999999999,
                "text": "You pick up a jigsaw with your right hand."
            },

The ego4d.json file downloaded from aws：
 {
            "clip_uid": "7829e427-1b08-4226-a631-74e25b5d9e89",
            "video_uid": "2d0c81c8-d7ff-4240-9ee7-4c71c152fb0a",
            "video_start_sec": 0.0,
            "video_end_sec":<PHONE_NUMBER>4583334,
            "video_start_frame": -1,
            "video_end_frame": 3255,
            "clip_metadata": {
                "fps": 30.0,
                "num_frames": 3255,
                "video_codec": "vp9",
                "audio_codec": null,
                "display_resolution_width": 1920,
                "display_resolution_height": 1080,
                "sample_resolution_width": 1920,
                "sample_resolution_height": 1080,
                "mp4_duration_sec": 108.54,
                "video_start_sec": null,
                "video_duration_sec": 108.5,
                "audio_start_sec": null,
                "audio_duration_sec": 108.522,
                "video_start_pts": 0,
                "video_duration_pts": 1666560,
                "video_base_numerator": 1,
                "video_base_denominator": 15360,
                "audio_start_pts": 0,
                "audio_duration_pts": 5209056,
                "audio_base_numerator": 1,
                "audio_base_denominator": 48000
            },
            "s3_path": "s3://ego4d-unict/public/v2/clips/7829e427-1b08-4226-a631-74e25b5d9e89.mp4",
            "manifold_path": null
        },

The first id "2d0c81c8-d7ff-4240-9ee7-4c71c152fb0a" is the video_uid, I found in the ego4d.json file, but i can't find where's the second id "8a889987-06aa-4778-bcd6-acd52bfe188f". What's the meaning of the second id?

Hello, the second id is clip_uid. Since livechat is generated by Ego4D goalstep, and that benchmark is proposed after Ego4D released, you may find that clip_uid in Ego4D goalstep annotations ;)