[Bug]: Error: Protocol error (Target.setAutoAttach): Target closed

Bug description
puppeteer.launch({}).then(browser => {   return browser.pages().then(async (pages) => {      const page = pages[0]      ....      ....   }) })
Puppeteer version
10.2.0
Node.js version
14.16.0
npm version
6.14.11
What operating system are you seeing the problem on?
macOS
Relevant log output
(node:49324) UnhandledPromiseRejectionWarning: Error: Protocol error (Target.setAutoAttach): Target closed.
    at /Users/WoodyHu/Workspace/web_frontend/node_modules/puppeteer-core/lib/cjs/puppeteer/common/Connection.js:217:63
    at new Promise (<anonymous>)
    at CDPSession.send (/Users/WoodyHu/Workspace/web_frontend/node_modules/puppeteer-core/lib/cjs/puppeteer/common/Connection.js:216:16)
    at Page._initialize (/Users/WoodyHu/Workspace/web_frontend/node_modules/puppeteer-core/lib/cjs/puppeteer/common/Page.js:184:26)
    at Function.create (/Users/WoodyHu/Workspace/web_frontend/node_modules/puppeteer-core/lib/cjs/puppeteer/common/Page.js:176:20)
    at /Users/WoodyHu/Workspace/web_frontend/node_modules/puppeteer-core/lib/cjs/puppeteer/common/Target.js:72:88
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:49324) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:49324) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

m1 chips

M1 chips

Also having this problem at AWS Lambda.

I'm guessing it's related to the nodejs version and not with the mac chipset.

Please try it out with the latest version and re-open if it still happens.


support for nuttx RTOS

Please find attached my working port to Nutt-X on STM32. I noticed that bmp3_defs.h and bmp.h could be merged. Also, the dev structure should be pass to the read routines directly so that they may be accessed through an opaque pointer (the "dev" structure on the target OS). Thank you very much for the great development and clean coding.

Hi @jafrado,
We would like to avoid platform specific code in the driver which will prevent it from being compiled for other targets. Nevertheless, there are some comments that you have made that are still valid. In order to validate them and include them in our internal design guidelines, can you comment on which compiler you use and the related build options?

Sure, I can rip out the nuttx defines. Would you like me to resubmit a patch without that?
Using GCC from ARM arm-none-eabi-gcc -v reports:
gcc version 6.3.1 20170620 (release) [ARM/embedded-6-branch revision 249437] (GNU Tools for ARM Embedded Processors 6-2017-q2-update)

I just pushed another set of mods, removed the system dependencies and added a Makefile for you to test build.

The array initializes below, they are not supported on older compilers:
uint8_t reg_data[BMP3_P_T_DATA_LEN] = { 0 };
struct bmp3_uncomp_data uncomp_data = { 0 };

Hi @jafrado ,
can you change to a newer dialect by adding the option -std=c99. I hope that helps.

Hi @BoschSensortec, thank you for your reply. NuttX follows C89 and all code outside of architecture specific directories must be compatible with C89. So adding that to fix an initialization statement written in C99 won't work because there are a myriad of spots which simply won't compile.
Related,  the macros S{8,16,32,64}_C,  U{8,16,32,64}_C are kernel type macros found in
linux/include/asm-xxx/int-ll64.h and should also be defined under the
#ifdef KERNEL which means this driver has only every been built for Linux.
Removing this bmp_defs.h gets rid of additional code needed for the kernel case, and I have
provide options which will work on any embedded ARM without an MMU.
Please have a look.

Update SensorAPI architecture.


os/mac/pkgconfig: add pc files for Big Sur 11.1

Big Sur 11.1 beta annoyingly introduced a new SDK path - /Library/Developer/CommandLineTools/SDKs/MacOSX11.1.sdk
This broke various builds which depend on the various macOS provided pkgconfig files.
This pull request adds a new directory for 11.1, updated the paths in the pc files and checked the versions.
Not sure if that's the way you want to do it going forwards, seeing as it'll break at every macOS point release, but it fixes it for now.

Thanks so much for your first contribution!  You rock, @thalamus!

Not sure if that's the way you want to do it going forwards, seeing as it'll break at every macOS point release, but it fixes it for now.

Yeh, it's what we'll do for now but I agree this is not ideal and we'll need to change this approach at some point in future.

Thanks, appreciated. I'm always happy to hunt down and fix something if I'm able to and it's nice to be able to contribute something back.


Updated string construction and fixed import issue caused by __ver…

…sion__.

Is this what is causing pip install git+https://github.com/ghrcdaac/cumulus-api.git@f93fb6ff98f98b6d31bd530fa24dc92ed61fd795 to fail?

Exactly, I didn't see it because I didn't test the install with a new conda environment.


[5598] Resolve N+1 issues on drafts/registered page

Context
https://trello.com/c/XrhWvkwh/5598-n1-query-traineesreviewdraftscontrollershow
Getting N+1 queries on the drafts/registered pages for academic cycles/apply applications (some cached some aren't). This also happens every time filtering via JavaScript is initiated.
Changes proposed in this pull request

Add necessary includes

Guidance to review
Before:

Important business (does not apply)

 N+1 Query Trainees::ReviewDraftsController#show


Remove IOException never thrown in S3Resource methods

Type: Feature
Is your feature request related to a problem? Please describe.
Several methods of the S3Resource object throw IOExceptions in their signature. But Exception is never thrown in those methods.
Use cases where this is a problem :
I use listObject() method in S3Template. I want to use contentLength or lastModified fields of S3Resource to sort the list, but I need to catch IOException in the comparator.
Maybe I missed something but I think that we can remove them.
Describe the solution you'd like
Remove IOException declaration in method signatures in S3Resource.
I know that S3Resource inherits AbstractResource. So I don't know if it is a problem for you to remove the exception in some S3Resource methods. Moreover, most uses of S3Resource are made via the S3Template, so there's no reference to the parent class.
If it is ok for you, you can assign me the ticket, I can make a PR quickly.
Additional context
Some examples :
        @Override
	public long contentLength() throws IOException {
		if (headMetadata == null) {
			fetchMetadata();
		}
		return headMetadata.contentLength;
	}

	@Override
	public long lastModified() throws IOException {
		if (headMetadata == null) {
			fetchMetadata();
		}
		return headMetadata.lastModified.toEpochMilli();
	}

Thanks @alexisgra for reporting and apologies for late reply. Are you still up for creating a PR?

Thanks @maciejwalkowiak for your answer. Yes, I can create a PR. I do it quickly!


ERROR:  role "rdsadmin" does not exist

I got this while trying to restore an aws backup locally.
dbbackup.db.exceptions.CommandConnectorError: Error running:  psql  --host=localhost --port=5432 --username=postgres --no-password --set ON_ERROR_STOP=on --single-transaction proddb2 ERROR:  role "rdsadmin" does not exist
Is there a way to resolve this?

Have you tried to create the rdsadmin role locally? https://www.postgresql.org/docs/13/database-roles.html

I created the role locally using pgAdmin and it worked perfectly, thank you!

Glad to hear.


Support for comments

Would it be possible to add support for comments in JSON?

This guy already did it https://github.com/neuroo/json-ast

@vtrushin ah thank you very much


🍒 release 0.11 - fix broken links (#4426)

Signed-off-by: kcoriordan<EMAIL_ADDRESS>(cherry picked from commit 2b082e9c48da482907fbb8626c6949adf8ab2d62)

This cherry-pick is slightly modified for the release-0.11 branch to avoid the change to the addons/packages/app-toolkit/0.2.0 file (which doesn't exist on this branch.


Switch to Alpha Blitting in GDI

Just a feature that would be cool!

I got this far, and it doesn't actually blend... not sure why.  Anyone else, any ideas?
}

void CNFGBlitImage( uint32_t * data, int x, int y, int w, int h )
{
	static int pbw, pbh;
	static HBITMAP pbb;
	if( !pbb || pbw != w || pbh !=h )
	{
		if( pbb ) DeleteObject( pbb );
		pbb = CreateBitmap( w, h, 1, 32, 0 );
		pbh = h;
		pbw = w;
	}

	int i;
	for( i = 0; i < w*h; i++ )
	{
		data[i] = rand() | 0x01000000; //(data[i] & 0xffffff ) | ((rand()&0xff)<<24);
	}

	SetBitmapBits(pbb,w*h*4,data);

	SelectObject( CNFGlsHDCBlit, pbb );
#ifdef GDI32NOUSEALPHA
	BitBlt(CNFGlsHDC, x, y, w, h, CNFGlsHDCBlit, 0, 0, SRCCOPY);
#else
	// If you use this, you will also need to link to -lmsimg32
	static const BLENDFUNCTION BlendFunc = { .BlendOp = AC_SRC_OVER, .BlendFlags = 0, .SourceConstantAlpha = 255, .AlphaFormat = AC_SRC_ALPHA };
	AlphaBlend( CNFGlsHDC, x, y, w, h, CNFGlsHDCBlit, 0, 0, w, h, BlendFunc );
#endif
}


SCM-Revision in MANIFEST.MF is always UNKNOWN

elastic-apm-agent-1.30.1-SNAPSHOT.jar!/META-INF/MANIFEST.MF
Manifest-Version: 1.0
Created-By: Maven Jar Plugin 3.2.0
Build-Jdk-Spec: 11
Implementation-Title: co.elastic.apm:elastic-apm-agent
Implementation-Version: 1.30.1-SNAPSHOT
Implementation-Vendor: Elastic Inc.
Agent-Class: co.elastic.apm.agent.premain.AgentMain
Can-Redefine-Classes: true
Can-Retransform-Classes: true
Can-Set-Native-Method-Prefix: true
Multi-Release: true
Premain-Class: co.elastic.apm.agent.premain.AgentMain
SCM-Revision: UNKNOWN

This also happened in the released jars

When removing these lines, the revision is properly populated again:
https://github.com/elastic/apm-agent-java/blob/61f5ee0248f78ced4c32f2209f9733c687afc494/elastic-apm-agent/pom.xml#L17-L18
Not sure if that has unintended side-effects, though.
To be save, we could also add a maven profile that's active when git.commit.id.abbrev is not set to provide a default value.


eMMC/SD card goes into read only mode

It is impossible to use the BBBlue, it always goes into read-only mode whenever there is a write attempt to eMMC or the SD card (32gb).
It also gives Input/Output error after it goes the read-only file system mode.
It's a very irritating bug, any fixes? or Is it a hardware issue?

Issue migrated to beagleboard/beaglebone-blue#40


steel toe initializer is broken server down

Initializer does not work when clicking explore

[ ] adding Dependencies should be in drop down
[ ] add asp identity users/roles/claims mgmt. dependency

Hi @ridingmower,
Sorry about that, apparently something is wrong with our site monitoring software and we didn't get any outage notifications, so I'll have to look into that. The service should be back up now

It's still broken. Just tried to navigate to https://start.steeltoe.io/ and clicked on EXPLORE:


Removed duplicated Wallet text from Amateur Larry

Sentence "Amateur Larry's wallet holds a detective agency card (...)" had "Amateur Larry's wallet" twice in the beginning.

Thanks for spotting this and submitting a pull request!


Docker har-to-k6 not working (ENOENT: no such file or directory)

The har file exists, but the docker image is always returning a ENOENT error (e.g. using docker run grafana/har-to-k6:latest new-recording_19332.har > my-k6-script.js).
docker run grafana/har-to-k6:latest new-recording_19332.har > my-k6-script.js
WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested
Converting 'new-recording_19332.har'
Error reading archive file:
ENOENT: no such file or directory, open 'new-recording_19332.har', at path: new-recording_19332.har

OS Info:
ProductName:		macOS
ProductVersion:		14.1.1
BuildVersion:		23B81
Docker version 24.0.6, build ed223bc

Workaround that worked for me.
I created that har directory and put my .har file there. Then:
docker run --rm -v $PWD/har:/converter/har grafana/har-to-k6:latest har/new-recording_162038.har >my_script.js


Add Flux

This PR adds Flux to the marketplace using the dev install mode.

Hi @stefanprodan good to see you raising the PR ,  can we have uninstall.sh as well for removing all components that gets installed in install.sh

@saiyam1814 I used Linkerd as a template, didn't had uninstall.sh, I'll add it now.

Thank you for pointing out I will raise an issue for that with Linkerd team :)

I've created a cluster to test this out, but the container images download is stuck:
$ ./flux/install.sh
[INFO]  Downloading metadata https://api.github.com/repos/fluxcd/flux2/releases/latest
[INFO]  Using 0.12.3 as release
[INFO]  Downloading hash https://github.com/fluxcd/flux2/releases/download/v0.12.3/flux_0.12.3_checksums.txt
[INFO]  Downloading binary https://github.com/fluxcd/flux2/releases/download/v0.12.3/flux_0.12.3_darwin_amd64.tar.gz
[INFO]  Verifying binary download
[INFO]  Installing flux to /Users/stefanprodan/.flux/bin/flux
► checking prerequisites
✗ flux 0.12.1 <0.12.3 (new version is available, please upgrade)
✔ kubectl 1.21.0 >=1.18.0-0
✔ Kubernetes 1.20.2+k3s1 >=1.16.0-0
✔ prerequisites checks passed
✚ generating manifests
✔ manifests build completed
► installing components in flux-system namespace
◎ verifying installation
✗ source-controller: deployment not ready
✗ kustomize-controller: deployment not ready
✗ helm-controller: deployment not ready
✗ notification-controller: deployment not ready
✗ install failed

$ k -n flux-system get po
NAME                                      READY   STATUS    RESTARTS   AGE
helm-controller-5bbd94c75-ttl9x           0/1     Pending   0          8m57s
notification-controller-d66bdc787-dtcrx   0/1     Pending   0          8m56s
source-controller-86d84856cf-lslvg        1/1     Running   0          8m56s
kustomize-controller-97dbdcb99-pmp6r      1/1     Running   0          8m57s

k -n flux-system describe po helm-controller-5bbd94c75-ttl9x
Name:           helm-controller-5bbd94c75-ttl9x
Namespace:      flux-system
Priority:       0
Node:           k3s-flux-test-78a61e82-node-1bb7/
Labels:         app=helm-controller
                pod-template-hash=5bbd94c75
Annotations:    prometheus.io/port: 8080
                prometheus.io/scrape: true
Status:         Pending
IP:
IPs:            <none>
Controlled By:  ReplicaSet/helm-controller-5bbd94c75
Containers:
  manager:
    Image:       ghcr.io/fluxcd/helm-controller:v0.9.0
    Ports:       9440/TCP, 8080/TCP
    Host Ports:  0/TCP, 0/TCP
    Args:
      --events-addr=http://notification-controller/
      --watch-all-namespaces=true
      --log-level=info
      --log-encoding=json
      --enable-leader-election
    Limits:
      cpu:     1
      memory:  1Gi
    Requests:
      cpu:      100m
      memory:   64Mi
    Liveness:   http-get http://:healthz/healthz delay=0s timeout=1s period=10s #success=1 #failure=3
    Readiness:  http-get http://:healthz/readyz delay=0s timeout=1s period=10s #success=1 #failure=3
    Environment:
      RUNTIME_NAMESPACE:  flux-system (v1:metadata.namespace)
    Mounts:
      /tmp from temp (rw)
      /var/run/secrets/kubernetes.io/serviceaccount from helm-controller-token-xm4gq (ro)
Conditions:
  Type           Status
  PodScheduled   True
Volumes:
  temp:
    Type:       EmptyDir (a temporary directory that shares a pod's lifetime)
    Medium:
    SizeLimit:  <unset>
  helm-controller-token-xm4gq:
    Type:        Secret (a volume populated by a Secret)
    SecretName:  helm-controller-token-xm4gq
    Optional:    false
QoS Class:       Burstable
Node-Selectors:  kubernetes.io/os=linux
Tolerations:     node.kubernetes.io/not-ready:NoExecute op=Exists for 300s
                 node.kubernetes.io/unreachable:NoExecute op=Exists for 300s
Events:
  Type    Reason     Age    From               Message
  ----    ------     ----   ----               -------
  Normal  Scheduled  9m46s  default-scheduler  Successfully assigned flux-system/helm-controller-5bbd94c75-ttl9x to k3s-flux-test-78a61e82-node-1bb7

After 15 minutes the container images from GHCR have been downloaded:
$ k -n flux-system get po
NAME                                      READY   STATUS    RESTARTS   AGE
source-controller-86d84856cf-lslvg        1/1     Running   0          16m
kustomize-controller-97dbdcb99-pmp6r      1/1     Running   0          16m
helm-controller-5bbd94c75-ttl9x           1/1     Running   0          16m
notification-controller-d66bdc787-dtcrx   1/1     Running   0          16m

Flux is working fine after the images got downloaded:
kubernetes-marketplace $ flux create source git podinfo --url https://github.com/stefanprodan/podinfo
✚ generating GitRepository source
► applying GitRepository source
✔ GitRepository source created
◎ waiting for GitRepository source reconciliation
✔ GitRepository source reconciliation completed
✔ fetched revision: master/ef98a040c89180a4f39c0ab01dac47e6c3fced08
kubernetes-marketplace $ flux create kustomization podinfo --source podinfo --path kustomize --prune
✚ generating Kustomization
► applying Kustomization
✔ Kustomization created
◎ waiting for Kustomization reconciliation
✔ Kustomization podinfo is ready
✔ applied revision master/ef98a040c89180a4f39c0ab01dac47e6c3fced08

Thanks Stefan, the team has been notified about the issue and will be looking into this. PR looks good to me and I will merge it

Has anyone who uses Flux made use of this app? I don't think we've fully captured the use case with this change at all.
flux bootstrap still must be run following the install of this app from the marketplace. The bootstrap process for Flux normally installs the stack, so this app is completely redundant without the bootstrap process being included in the app's installation.
I imagined the Flux integration for Civo would effectively make a cluster zero-touch. A field should be exposed in the UI for this app so you can specify a git repo along with an SSH private key for Flux to bootstrap with.

the user input while installing the app is not there, I would request you to put that in the suggestions area in your Civo account and we can open it for voting and then implement it as a feature that can help cater this use-case

As stated in the PR description, this one-click deploy is for dev/test clusters only. I don't expect people to use this for production, instead you should use flux bootstrap as documented here: https://toolkit.fluxcd.io/guides/installation/
Having a UI where you would paste your Git token sounds really dangerous to me, if Civo stores the token in their logs then this could be a major security risk. Imagine that someone gets hold of those logs and then it can access your Git repos, delete them, push code, etc. In the end is about trust, the flux CLI runs on your own machine, if you run the CLI on someone else's computer aka the cloud, then you must trust the vendor that it can keep your secrets safe.

Outside of this specific use case; I strongly believe a feature for user-defined variables and secrets would reduce the friction of setting up a cluster and could expand the available use-cases.
If I'm trusting Civo with the data in my clusters and the secrets stored in the cluster, a few variables in addition to that for the setup flow is not a massive deal, especially if it reduces the time to go-live.

Having a UI where you would paste your Git token sounds really dangerous to me, if Civo stores the token in their logs then this could be a major security risk.

I'd say the same goes for everything, from password and payment fields to the Kubernetes tokens required to interact with the cluster. I don't doubt Civo can handle some additional secret material if it results in better UX.


Sending ADDRESS in merge field - Doesn't make it through to Mailchimp

I've been trying to figure out how to send ADDRESS in my merge fields (everything else is fine), but I can't seem to get the format right. Anyone else run into this?
Whether I pass a string:
ADDRESS: "123 Some St, Toronto, ON, CA"
or I try to pass parts (some googling pointed me to this):
ADDRESS: { addr1: "123 Some St", city: "Toronto", state: "ON", country: "CA" }
It doesn't save. When I made it a required field on the Mailchimp side it throws an error saying "Please enter a complete address" so obviously I'm just not nailing the format.
Wasn't able to find any Gibbon specific documentation on passing nested merge field values like this.

Hi @rapind. Can you share the line or two of Ruby you're using to try to set an address merge field?
I'd expect the second format to work assuming you're nesting the field properly when your making your request.
Gibbon maps Ruby syntax onto the MC API directly so as you say it's a matter of figuring out what the API expects and matching it.

@amro sure thing, but I don't think it tells you any thing more. The problem is the ADDRESS tag, everything else makes it through to Mailchimp just fine.:
    gibbon.lists(mailchimp_list_id).members(lower_case_md5_hashed_email_address).upsert(
      body: {
        email_address: user.email,
        status: "subscribed",
        merge_fields: {
          FNAME: user.first_name,
          LNAME: user.last_name,
          EMAIL: user.email,
          PHONE: user.phone,
          ADDRESS: user.address
        }
      }
    )

In the above I've tried a user.address string value of: 123 Some St, Toronto, ON, CA, and a user.address hash value of { addr1: "123 Some St", city: "Toronto", state: "ON", country: "CA" }. Neither of which makes it through. I've also trued a json version of the hash: E.g. address.to_json

I looked into this and at first I thought there was a bug in MailChimp's API (spoiler: there's not).
I first I tried your snippet and ran into the same problem. I intentionally fudged the type of one of the fields (zip code) in my case to make it numeric and the API returned a validation error (because the zip is supposed to be a string). At that point, I wondered if this worked at all since, as I mentioned, Gibbon simply takes Ruby syntax and maps it onto HTTP requests. It's a really simple transformation. So...I tried Curl:
curl --request POST \
--url 'https://us1.api.mailchimp.com/3.0/lists/<list_id>/members' \
--user 'anystring:<api_key>' \
--data<EMAIL_ADDRESS>"status":"subscribed", "merge_fields": {"FNAME":"Bob", "MYADDY": {"addr1":"1234 Someplace Way", "addr2":"", "city":"Atlanta", "state":"GA", "zip":"30318", "country":"US"}}}' \
--include

And I ran into the same problem! At this point I noticed the response included the user's merge fields, which referred to the address field I added by the MailChimp-defined field name (MMERGE17 in my case) so I tried that instead and it worked:
curl --request POST \
--url 'https://us1.api.mailchimp.com/3.0/lists/<list_id>/members' \
--user 'anystring:<api_key>' \
--data<EMAIL_ADDRESS>"status":"subscribed", "merge_fields": {"FNAME":"Bob", "MMERGE17": {"addr1":"1234 Someplace Way", "addr2":"", "city":"Atlanta", "state":"GA", "zip":"30318", "country":"US"}}}' \
--include

I then tried the same with Gibbon and it worked as well.
gibbon.lists(<list_id>).members.create(body: {
  email_address<EMAIL_ADDRESS>
  status: "subscribed",
  merge_fields: {
    MMERGE17: {
      addr1: "1234 Someplace Way",
      city: "Atlanta",
      state: "GA",
      zip: "30318"
    }
  }
})

Hmm, I thought. So I looked back at how I'd defined the new address variable on my list:

Ah! The issue in my case was there's a display name, which I'd made look like a merge field, but I haven't updated the API-visible merge field name to the right.
Once I did so it all worked fine:

So what's odd, I think, is the API seems to validate data types for merge fields that "don't exist", but
ignores the data. I guess they do this to increase success rate of API requests when people make mistakes (e.g. you get the email address, at least).

OK so it looks like zip is required. Including zip fixes it for me. Wish they documented that, or that the error returned from their API was more specific. I think we can close this. Thanks for your help @amro !

Great. Glad you figured it out! Agree that the doc around this is not great

On Oct 14, 2018, at 11:31 AM, Dave Rapin<EMAIL_ADDRESS>wrote:
Closed #272.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

OK so it looks like zip is required. Including zip fixes it for me. Wish they documented that, or that the error returned from their API was more specific. I think we can close this. Thanks for your help @amro !

Thanks @amro and @rapind... after banging my head against a brick wall for sometime and then finding this, supplying a zip attribute was all I needed to get my address imports working.
My ADDRESS merge field data looks like this:
{
  addr1: address.address_line1,
  addr2: address.address_line2,
  city: address.city,
  country: address.country, # Needs to be 2 characters i.e GB, US
  state: address.county,
  zip: address.postcode # All important!
}

Kind of annoying that this still isn't documented by Mailchimp... at least as far as I could find anyway.


set logging level to debug

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.09%) to 75.498%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sis_provisioner/models.py
0
3
0.0%

Files with Coverage Reduction
New Missed Lines
%

sis_provisioner/models.py
1
55.83%

Totals

Change from base Build<PHONE_NUMBER>:
-0.09%

Covered Lines:
607

Relevant Lines:
804

💛  - Coveralls


Redirect /twitch to Twitch channel

Changes:

/twitch view removed
/twitch redirects to https://twitch.tv/freecodecamp

closes #7422

@alistermada thanks for the PR, looks good. Pinging @QuincyLarson to QA.

Awesome work, @alistermada !


Feature order by translatable

Resolve #19

Codecov Report

Merging #26 (799f9c7) into master (e5822ba) will not change coverage.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##              master       #26   +/-   ##
===========================================
  Coverage     100.00%   100.00%
- Complexity       117       120    +3
===========================================
  Files             15        15
  Lines            268       281   +13
===========================================
+ Hits             268       281   +13

Impacted Files
Coverage Δ
Complexity Δ

src/TranslationScopes.php
100.00% <100.00%> (ø)
8.00 <3.00> (+3.00)

Continue to review full report at Codecov.

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


IP2

With at least one day to spare :)

@abhutta0 @kelloggm

GameAreaController._players doesn't reflect the initial state when creating a new GameAreaController through the constructor
I was trying to use GameAreaController._players in one of the helper methods, and while debugging the tests, I noticed that the _players array doesn't reflect the correct initial state of GameAreaController.model.game.players.
While the constructor of GameAreaController sets this._model to the input parameter gameArea, this._player does not get updated with the players inside gameArea.game.players unless _updateFrom is called.
constructor(id: InteractableID, gameArea: GameArea<State>, townController: TownController) {
    super(id);
    this._model = gameArea;
    this._townController = townController;
  }

I am not sure if this was intentional, but we can either call updateFrom in the constructor or manually set the _players if we don't want to emit events at that stage.

Here are a set of notes (from my TA) on the assignment: https://docs.google.com/document/d/1Mxy8ks6kVmMF63oxHSkMNUCs0VLVvSJd/edit?usp=sharing&ouid=104647171837719831027&rtpof=true&sd=true
I'm extremely busy this afternoon, but I'll try to turn this feedback into GitHub comments on the relevant part of the assignment spec ASAP. But, I wanted you all to be able to see this right away, too, and not have to wait for me if you have cycles today.

Here are a set of notes (from my TA) on the assignment: https://docs.google.com/document/d/1Mxy8ks6kVmMF63oxHSkMNUCs0VLVvSJd/edit?usp=sharing&ouid=104647171837719831027&rtpof=true&sd=true
I'm extremely busy this afternoon, but I'll try to turn this feedback into GitHub comments on the relevant part of the assignment spec ASAP. But, I wanted you all to be able to see this right away, too, and not have to wait for me if you have cycles today.

Thanks! I am doing my last pass through the assignment applying all of these comments today and we are planning to launch later today/early tomorrow...

Just confirming that we are also releasing the solution of IP1 (with tests), as usual.

Just confirming that we are also releasing the solution of IP1 (with tests), as usual.

Yes

GameAreaController._players doesn't reflect the initial state when creating a new GameAreaController through the constructor
I was trying to use GameAreaController._players in one of the helper methods, and while debugging the tests, I noticed that the _players array doesn't reflect the correct initial state of GameAreaController.model.game.players.
While the constructor of GameAreaController sets this._model to the input parameter gameArea, this._player does not get updated with the players inside gameArea.game.players unless _updateFrom is called.
constructor(id: InteractableID, gameArea: GameArea<State>, townController: TownController) {
    super(id);
    this._model = gameArea;
    this._townController = townController;
  }

I am not sure if this was intentional, but we can either call updateFrom in the constructor or manually set the _players if we don't want to emit events at that stage.

Thanks! Fixed.

Thank you very much to @shashwatrathod , @kelloggm , and Martin's anonymous TA. I've improved the assignment, handout, and reference solution with this feedback. The autograder is updated, too. This should be ready to go to release, I think, @abhutta0 (probably will need to update pages to link to it, etc, but figured you would do that while announcing it)

I can merge and then update other pages as needed. Please let me know when you are done.

I can merge and then update other pages as needed. Please let me know when you are done.

I am done

@jon-bell  Points don't add to 200. I have updated the Task total to match scores given in text. But we still seem to be 10 pts short
Task 1-4 are 30+85+40+15=170.

@jon-bell Points don't add to 200. I have updated the Task total to match scores given in text. But we still seem to be 10 pts short Task 1-4 are 30+85+40+15=170.

copy/paste bug :( they were right in the autograder spec but not updated on the assignment page. should be all set now (and the end-to-end CI test shows that the reference solution earns 180 points)


feat(UI): Added new AllInOneMenu and SwitchButton components.

These are necessary for the new window level feature.

Context
Addresses #3764 for the window level epic #3493

Changes & Results
Added new components.

Testing
Run storybook to see the new components in action.

Checklist
PR

[x] My Pull Request title is descriptive, accurate and follows the
semantic-release format and guidelines.

Code

[x] My code has been well-documented (function documentation, inline comments,
etc.)

Public Documentation Updates

[x] The documentation page has been updated as necessary for any public API
additions or removals.

Tested Environment

[x] OS: Windows 11
[x] Node version: 18.16.1
[x] Browser: Chrome 118.0.5993.120

Great stuff

Thanks.

I feel like the hover background is wrong, maybe your implementation or maybe even the design. I expected it to fill the region, or at least be symmetric with top and bottom

I will check the design and update accordingly.

Looks like there is a bottom layout shift, I guess it is understandable the the top would change (as of height change), but the bottom should remain still
Having said number 2, I think the menu should accept two more props regarding where to open (bottom or top) and also which direction (left or right) similar to our dropdown UI. I think right now you have implemented it so that it opens from top to bottom and to the right, but for window level we need it opens towards up and to the right. And also if we decide to put it on the top right corner, it should handle top to bottom opening and to the left

There are no and should be no implementation details in the menu about how the menu opens because it is JUST a menu and it is up to the code that opens it to position it and thus decide which way it opens. Put another way, the top, left, bottom and right position attributes that it is placed into the DOM with will determine this. This will be evident when it is used in OHIF.

It says Back to I guess this is a bug?

Yes this is a problem with Story Book I think, but I think I found a way around it.

I must apologize to you for https://github.com/OHIF/Viewers/pull/3774#issuecomment-1802416428 because although the positioning of the menu is handled by the parent that opens the menu, things like the arrows for the back button submenus need to change depending on whether the menu tends to open left to right or right to left. I will update. Thanks and apologies again.


sort team members by role

@keybase/react-hackers

I think this would be nicer if it were like this:
const order = {owner: 0, admin: 1, writer: 2, reader: 3}
// ...
memberInfo.toArray().sort((a, b) => a.type === b.type ? a.username.localeCompare(b.username) : order[a.type] - order[b.type])

@marco It does look nicer, but if a new role gets added the behavior will be undefined - not a huge deal though.


Doesn't work

Got this to work on a Mac after adding my Private Repo Url & a personal access token to the nuget.config file in my $HOME/.nuget/nuget folder.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a1a158fb-d1e3-a0d9-492e-3f78dde5d97c
Version Independent ID: 71055426-3ba1-27f3-11db-cadb688ae123
Content: Use dotnet with Azure Artifacts feeds
Content Source: docs/artifacts/nuget/dotnet-exe.md
Product: devops
GitHub Login: @elbatk
Microsoft Alias: elbatk

Closing this as it looks like you've figured it out @jordanchang . Feel free to reach back out if you're still having issues.


Restic don't follow 308 on swift backend

Output of restic version
restic 0.16.2 compiled with go1.21.3 on linux/amd64
What backend/service did you use to store the repository?
swift
Problem description / Steps to reproduce
When trying to init a repo on an swift backedn i get a FATAL when there is a 308 redirect in place
restic init -v  Fatal: create key in repository at swift:BUCKETNAME:/ failed: client.PutObject: HTTP Error: 308: 308 Permanent Redirect
The reason on the Openstack site is that by default the installation gives out http api links, and then do a 308 to https.
Expected behavior
Restic to follow redirect 308/301
Is this behavior intended? is there a risk to follow redirect? Or would it be ok to just add a follow redirects?

The reason on the Openstack site is that by default the installation gives out http api links, and then do a 308 to https.

This is a serious security vulnerability as any client sending an authenticated request to the HTTP API would leak the used credentials / auth tokens. The OpenStack / Swift APIs generally use bearer tokens which NEVER must be sent over plaintext connections.
In addition, it is seriously inefficient if each upload request has to be retried.


[AutoPR migrate/resource-manager] Add config section for swagger->sdk for Go

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2725

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/1421


Flag !icloud backup

Hello,
I am having an issue where my app is being rejected due to my sqlite database being too big for icloud backups.  Since it's just reloadable data how can I flag it not to be backed up?
Thanks,
Nick

Durp-a-Durp, should have been using .CachesDirectory instead of .DocumentDirectory


Improve coerce API so it does not need DFSchema

Which issue does this PR close?
Closes https://github.com/apache/datafusion/issues/3793
Rationale for this change

Permit making an API for https://github.com/apache/datafusion/issues/10181 that takes a &DFSchema rather than an owned
Avoid a copy and thus make the API marginally faster in some cases.

What changes are included in this PR?
Change coerce API from
    pub fn coerce(&self, expr: Expr, schema: DFSchemaRef) -> Result<Expr> {

to
    pub fn coerce(&self, expr: Expr, schema: &DFSchema) -> Result<Expr> {

Are these changes tested?
By existing CI
Are there any user-facing changes?
The API to call coerce is slightly changed. Note I am in the process of adding a proper fully baked API in https://github.com/apache/datafusion/issues/10181

Thank you for the review @jayzhan211


Testing, please ignore. (tuqher)

This is a bug bounty test - please ignore.. Please DO NOT APPROVE THIS! (tuqher)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.tuqher fzrcm seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add threshold parameter for object detection widget

Is your feature request related to a problem? Please describe.
Currently, the object detection widgets don't have a threshold parameter, so they just use the hardcoded one from the object detection pipeline, which is 0.9 as seen here.
However, with more and more object detection models coming, we definitely need an adjustable threshold button for the widget, as for instance now Deformable DETR checkpoints, which need a lower threshold to detect objects in a scene, simply say "no objects detected":

Describe the solution you'd like
Adding a "threshold button" to the inference widgets. Perhaps make it possible to set a default threshold in the model card.

Is this supported in the pipeline class already @NielsRogge ?

Hey @osanseviero, threshold is already supported in the pipeline class, I'm updating the pipeline to use the post_process_object_detection methods instead of post_process with this PR.

Cc @mishig25 and @OlivierDehaene maybe can help here (adding the slider in the widget and adding support in the inference API for this)


Jupyter/Python extension integration tests in VS code core

There have been two failures in the last week related to core changes (caught by integration tests on Jupyter).
Maybe we should add some tests in VS code core that run with jupyter/python? Is this outside the scope of VS code core testing? (It would require python being installed)

I don't think this is necessary, as the API stabilizes we might find the need for such a test, less & less useful.
Closing for now.


Sql Server 2017 support?

Hi,
we use this amazing tool for comparing the structure of SSAS with the relational structure and need this tool for sql server 2017.
Are there any plans in updating the latest version?

Yes please!

I've just added a pre-release with 2017 support can you please check that it works for you. There is no new functionality, it's just been recompiled against the 2017 references.

I've just tested the pre-release on our DEV SSAS 2017 box. Works like a charm!
Thanks a million!

Great, thanks for letting us know


Cannot get tests to work with fragments

I try to run my GraphQL queries against a mocked schema, and it runs fine, except for any queries that contain a fragment. For these, I keep getting this error:
Unknown fragment "SomeFragmentName".
My test setup:

I have the transform set in the jest.config.js exactly as  described in this project's README.
In my test file, I generate a schema using makeExecutableSchema and addMockFunctionsToSchema.
Then, I run the queries like this:

const execute = ({ query, done, variables = {} }) => {
    let queryString = query.loc.source.body

    const promise = graphql(
        //  schema
        schema,
        queryString,
        null,
        null,
        variables
    )

    return promise.then(({ data, errors }) => {
        if (errors) {
            return done.fail(`Errors in query:\n${errors.join('\n')}`)
        }
        expect(data).toMatchSnapshot()
        return Promise.resolve()
    })
}

An example query that fails to run looks like this:

#import '../../fragments/Widget.graphql'

query WidgetFragment($id: ID) {
    readWidgetFragment(id: $id) {
        ...Widget
    }
}

Maybe I'm overlooking something here, would be grateful for any help!

Has anyone figured out a way to get around this?

Any luck with this issue?

query.loc.source.body does not include the contents of the imported Fragment, only your actual query. The imported fragment is inquery.definitions.
If your client must take a string, then you can use graphql/language/printer to print out the complete query.
import { print } from 'graphql/language/printer';

graphql(schema, print(query), ...)


[Bug] Tags in import-scan don't work

Describe the bug
Tags passed as a slice in the import-scan functionality are not passed correctly to the API endpoint. They are currently represented in a Go slice format like [one two three] and are not correctly parsed as separate tags by DefectDojo.
To Reproduce
Steps to reproduce the behavior:

Try to import scan with multiple tags associated
Validate if tags are correctly parsed as separate ones in DefectDojo

Expected behavior
Tags should be in the format one, two, three and parsed correctly as  part of FormData.

Example of call that is parse as a single tag:
https://github.com/truemilk/go-defectdojo/blob/9116690a754f05e08fedf6c7842392afdb6f18ca/example/importscan/main.go#L43
Results in:
[AAA, BBB]


Reporting Bug on MultiSelectBox

Hello, I found a bug with <MultiSelectBox/> that I would like to report.
I am using  nextjs 13 (experimental app/ directory) and with the code below, the first SelectBoxItem that is being clicked disappears from the list. While any following clicks on the remaining items do not make them disappear.
"use client";
const headerList = ["item1", "item2", "item3"];
export default function HeaderSelection() {
    const { selectedHeaders, setSelectedHeaders } = useHeaderContext();
    return (
        <MultiSelectBox defaultValue={copiedTableHeaders} onValueChange={setSelectedHeaders}>
            {headerList.map((headerName: string) => (
                <MultiSelectBoxItem key={headerName} value={headerName} text={headerName} />
            ))}
        </MultiSelectBox>
    );
}

The workaround that I employed was to create a clone of the array. I don't really know why this version works.
"use client";
const headerList = ["item1", "item2", "item3"];
export default function HeaderSelection() {
    const { selectedHeaders, setSelectedHeaders } = useHeaderContext();
    // Deep copy to avoid bug on MultiSelectBox componenet
    const copiedTableHeaders: string[] = Object.assign([], selectedHeaders);
    return (
        <MultiSelectBox defaultValue={copiedTableHeaders} onValueChange={setSelectedHeaders}>
            {headerList.map((headerName: string) => (
                <MultiSelectBoxItem key={headerName} value={headerName} text={headerName} />
            ))}
        </MultiSelectBox>
    );
}

I am very new to tremor and believe it is a bug. However, please kindly let me know if this is intended.

Able to reproduce here https://codesandbox.io/p/sandbox/reverent-julien-vq93dk?welcome=true
It is probably happenning because the defaultValue prop is being mutated in the MultiSelectBox component.
This line - https://github.com/tremorlabs/tremor/blob/92b84bd1433c2a797e6fba02e51a54a75ef3eddb/src/lib/utils.tsx#L27

hey @hyukkyukang, thanks for using Tremor! This is not a bug but the Select components are intended to be used differently as shown in the docs in the Controlled state example section. In your case this would be implemented as follows:
"use client";
const headerList = ["item1", "item2", "item3"];
export default function HeaderSelection() {
    const [selectedHeader, setSelectedHeader] = useState("item1");
    return (
        <MultiSelectBox value={selectedHeader} onValueChange={setSelectedHeader}>
            {headerList.map((headerName: string) => (
                <MultiSelectBoxItem key={headerName} value={headerName} text={headerName} />
            ))}
        </MultiSelectBox>
    );
}

Note that you should use value instead of defaultValue since you are using the controlled version of the component. You can read more about controlled and uncontrolled components in React here.
Let me know if you face any other issue.


Kwun/update packages

Description

Fixes #
Checklist

[x] Targeted PR against correct branch.
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Added an entry to the CHANGELOG.md file.
[x] Re-reviewed Files changed in the Github PR explorer.

Tested without issue with the deep dependency fixes


Added more detailed documentation on Contributing

I added documentation explaining how to contribute in detail. This documentation now lives on the pumpkin website as well as the DOCUMENTATION.md file

Pls rebase so there is not spam of "merge branch 'Snowiiii:master' into master"


Order of elements in Array is not followed (after initial render)

If you change the order of HTML-elements (e.g. by a drag'n'drop script) corresponding to array-elements and rerender this array the old order is left in place.
view on CodePen
(I assume the behaviour is intended but how can I force a hard redraw?)

This is expected. You're not supposed to manually (or through script) manipulate rendered content. It will most likely break things, because especially when rendering Arrays, lit-html creates a special structure that it needs in order to re-render efficiently.
If you want to move elements, then change the order in the original array, and let lit-html render that array again. In most cases it will be nearly as efficient as switching DOM nodes around. If your elements are keyed (they have unique state), you can use the repeat directive to move elements around.
Think of lit-html as a "pure" function that maps a state to a certain DOM tree representation. If you want to change the DOM tree, then change the state, and let lit-html map the new state to the DOM tree.

I am seeing this as well. I want to implement Drag-n-Drop functionality. Most of the time it works but sometimes it does get wonky. What can we do about this?

This is expected behavior. You can do what ruphin explained in his comment, change the data structure and let lit change the order.

Coming from LitElement, does it make sense to have an LitElement API to force re-render? I mean, for drag and drop there's really no way to cancel it after the fact, and instead of hacking a workaround (shifting elements back and updating model), just try to force a fresh render?

The correct way to handle this is to have the drag-and-drop action change the datastructure in your element that describes the content (and it's order), and then re-render. You should always avoid manipulating elements manually.

I wrote a minimal example for drag'n'drop in lit-html: hoffis-eck.de/exampleDnDLitHtml/

@HoffmannP, thanks for sharing your code!!  There's so little info/examples out there, sharing like this helps the community. :)

I have mastered the art of drag and drop within the same list, but dragging and droppping from one list to another using lit-html is a real usability issue and I am still struggling through it. (I am building a kanban board)
Maybe the thinkers around here have a better idea on how this can be easily accomplished, but so far it's been a real pain when the dragging and dropping goes across Shadow DOM component boundaries with their own list properties...

@ernsheong I think there are two general ways to accomplish this:

1 You never mutate the DOM directly. The drag operation styles the element so that it follows the cursor with a CSS transform, but upon drop an event is fires to mutate the data, which then re-renders the DOM with the item in its new location as derived from the updated data.
-2 You preserve the dragged item by closely coordinating data and DOM updates via special directives. In this case you would keep the dragged item, but you have to tell one directive that it was removed and hand it off to another directive so that it knows it's there. This would essentially be a modified repeat() with an ability to adopt an item part and it's DOM. The cache() directive has some relevant code in it too.

cc'ing @rictic and @straversi who've both done some experiments in this area.

I agree with @ruphin and @justinfagnani. I did do a little bit of experimenting around animating the other items during a drag sequence: https://stackblitz.com/edit/lit-drag-n-drop
The next natural problem in my mind is: how to animate the dropped item into place. If you mutate the underlying list immediately upon dropping, then lit will re-render, and there's not chance for
a drop animation. One solution would be to mutate the list upon completion of a drop animation.

I wrote a minimal example for drag'n'drop in lit-html: hoffis-eck.de/exampleDnDLitHtml

Hi man maybe you still have example of dragn drop code litElement, because your link dont work, but it would be very helpfull for me and other

Here is the link to my repo: https://github.com/HoffmannP/exampleDnDLitHtml

@justinfagnani Would you be able to explain this a little further?

-2 You preserve the dragged item by closely coordinating data and DOM updates via special directives. In this case you would keep the dragged item, but you have to tell one directive that it was removed and hand it off to another directive so that it knows it's there. This would essentially be a modified repeat() with an ability to adopt an item part and it's DOM. The cache() directive has some relevant code in it too.

We would like to move elements from one element (with its own data/render) to another element (with its own data/render).
We are finding that on dragend, the element we are dropping is not the element we have in the DOM.
Can you explain further what you wrote above? Just a little detail would do wonders. Thank you either way.

@mercmobily what I'm describing is a bit of a complex operation that's similar to what the cache() directive does (it preserves detached DOM) but coordinated across multiple directives()/containers().
We've been thinking about these kind of helpers in terms of what we're calling "reactive controllers" recently - these are objects that can preserve state and utilize the component lifecycle across renders. There are equivalent techniques for raw lit-html, but it's easier to show with LitElement.
The idea is you'd have a helper that worked like repeat(), but it could detect moves across multiple lists. If an item is moved between lists, its DOM is actually moved too. Also, in order to perform FLIP-style animations, it might need to measure all item DOM before anything is moved.
Using it would look like:
ts```
animatedListsController = new AnimatedListsController();
render() {
return html
       ${this.animatedListsController.repeat(this.list1, (i) => html${i})}

       ${this.animatedListsController.repeat(this.list2, (i) => html${i})}
   ;
}

This is definitely something we should at least prototype, since we need to show it working even if we don't ship production-ready helper just yet.

Oh I see. The idea -- correct me if I am wrong -- is that if two different elements could potentially share elements thanks to DnD (or anything else), they would use the same animatedListController variable to show the list, and animatedListController would be smart enough to make sure that DOM elements would be preserved -- right?
I can't even start to imagine what on earth animatedListController would need to do in order do actually perform such magic! But, yes, that would do the trick for us. And yes, it would be a much needed feature.


Release 1443 patch 13

Release 1443 patch 13
Summary
Release new 1443 release patch that addresses latest high and critical severity security findings.  The patch should include latest published kernel patch. The number should be included in the release note, together with a summary of the vulnerabilities addressed.
Changes
List GitHub Issues, Pull Requests or other references to important changes for the consumers.

todo

Tasks

[ ] Identify packages that must be patched

[ ] linux 6.6.54 → 6.6.tbd

[ ] Complete the build (stage 1)
[ ] Complete the publishing (stage 2)
[ ] Publish the release note
[ ] Inform Gardener team the patch is ready.

path 15 released


Converting Keras model to TF model.

I tried to run the :CUDA_VISIBLE_DEVICES=None python3 convert_keras_to_tensorflow_serving_model.py". It gives the following error.
Loading saved Keras model.
Traceback (most recent call last):
File "convert_keras_to_tensorflow_serving_model.py", line 87, in
"dice_coef": dice_coef, "dice_coef_loss": dice_coef_loss})
File "/home/avx/venv/TF3_env/lib/python3.5/site-packages/keras/engine/saving.py", line 419, in load_model
model = _deserialize_model(f, custom_objects, compile)
File "/home/avx/venv/TF3_env/lib/python3.5/site-packages/keras/engine/saving.py", line 312, in _deserialize_model
sample_weight_mode=sample_weight_mode)
File "/home/avx/venv/TF3_env/lib/python3.5/site-packages/keras/engine/training.py", line 139, in compile
loss_function = losses.get(loss)
File "/home/avx/venv/TF3_env/lib/python3.5/site-packages/keras/losses.py", line 133, in get
return deserialize(identifier)
File "/home/avx/venv/TF3_env/lib/python3.5/site-packages/keras/losses.py", line 114, in deserialize
printable_module_name='loss function')
File "/home/avx/venv/TF3_env/lib/python3.5/site-packages/keras/utils/generic_utils.py", line 165, in deserialize_keras_object
':' + function_name)
ValueError: Unknown loss function:combined_dice_ce_loss

Thanks.  Please check the update to see if it works.

It is working now.


Add detection documentation to attack techniques (closes #48)

Easiest way to review is to spin up the docs locally:

Checkout branch

pip install mkdocs-material mkdocs-awesome-pages-plugin && mkdocs serve

cc @xen0ldog


About AEAD in command line.

Why does openssl 1.0.2 has AEAD in enc command but openssl 1.1.0 doesn't.

1.0.2 does not support AEAD in enc. This is the code in 1.0.2 that checks this:
https://github.com/openssl/openssl/blob/7a7afc559ebc0ad88390cc62bfc34c221d595831/apps/enc.c#L341-L344

@mattcaswell But i found in the command line (sorry for no screenshot)
And I just know 1.1.0 doesn't.
I truely found the "openssl will not support aead in command line and even future" words.
But I truely found the "aes-256-gcm" in 1.0.2 command line outputs too.
(I can't found when i use correct command, lt's in the error output.)
(sorry for no screenshot)

The command runs, but it is silently wrong.  Later releases warned that it doesn't work.

Closing this since AEAD ciphers don't work from the command line and never have.


4-spaces indentation for gitlab mode

Fixes https://github.com/thlorenz/doctoc/issues/61

This PR had a bug in it an broke the tests.
I implemented this feature myself here including tests.
Please add tests for features/changes in the future and at the very least ensure that existing tests still pass after you make your changes.
This shows that you respect my time as much as I respect yours ;)


Fix topo image cmap

hopefully fixes #6112
So far this makes the following two approaches consistent (see #6112 for screenshots of how they are currently not consistent):
epochs.copy().pick_types(meg=False, eeg=True).plot_topo_image()
epochs.plot_topo_image(layout=mne.channels.find_layout(epochs.info, ch_type='eeg'))

However, this makes many topo_image plots for MEG sensors not very informative since the images are mostly grey/white (at least with the sample data).  I'm very open to suggestions for other approaches.
cc @larsoner @jona-sassenhagen

However, this makes many topo_image plots for MEG sensors not very informative since the images are mostly grey/white (at least with the sample data). I'm very open to suggestions for other approaches.

For EEG data, the situation will be even worse, right? And for weaker signals than the brutal sample data paradigm.
epochs.plot_image filters/interpolates the image for this purpose.

I say let's wait a week and a half and @jona-sassenhagen @drammock you two can work it out together over coffee and pastries :)

Awesome :D

replaced by #6225


Windows : npm start error

Hi.
After npm install (no errors) I tried npm start and I get this error:
$ npm start

><EMAIL_ADDRESS>start C:\Users\examples\react-webpack-babel-hotreload-example
> cross-env NODE_ENV=development nodemon --watch webpack-dev-server.js --watch webpack.config.js webpack-dev-server.js

[nodemon] 1.11.0
[nodemon] to restart at any time, enter `rs`
[nodemon] watching: webpack-dev-server.js webpack.config.js
[nodemon] starting `node webpack-dev-server.js`
[8972] Webpack development server running at http://<IP_ADDRESS>:7000
[8972] NODE_ENV=[development]
webpack: wait until bundle finished: /
Hash: 1d7583f4f2a00ea844bd
Version: webpack 2.5.1
Time: 48148ms
        Asset       Size  Chunks  Chunk Names
      main.js  190 bytes       0  main
    vendor.js    2.61 MB       1  vendor
  main.js.map   84 bytes       0  main
vendor.js.map    2.96 MB       1  vendor

ERROR in ./src/app.jsx
Module not found: Error: Can't resolve 'history/createBrowserHistory' in 'C:\Users\examples\react-webpack-babel-hotreload-example\src'
 @ ./src/app.jsx 4:0-57
 @ multi ./src/app

ERROR in ./src/components/root.jsx
Module not found: Error: Can't resolve 'prop-types' in 'C:\Users\examples\react-webpack-babel-hotreload-example\src\components'
 @ ./src/components/root.jsx 2:0-35
 @ ./src/app.jsx
 @ multi ./src/app

ERROR in ./src/components/main.jsx
Module not found: Error: Can't resolve 'prop-types' in 'C:\Users\examples\react-webpack-babel-hotreload-example\src\components'
 @ ./src/components/main.jsx 10:0-35
 @ ./src/components/root.jsx
 @ ./src/app.jsx
 @ multi ./src/app

ERROR in ./src/components/route_example_1.jsx
Module not found: Error: Can't resolve 'prop-types' in 'C:\Users\examples\react-webpack-babel-hotreload-example\src\components'
 @ ./src/components/route_example_1.jsx 2:0-35
 @ ./src/components/routes.jsx
 @ ./src/components/main.jsx
 @ ./src/components/root.jsx
 @ ./src/app.jsx
 @ multi ./src/app

ERROR in ./src/components/route_example_2.jsx
Module not found: Error: Can't resolve 'prop-types' in 'C:\Users\examples\react-webpack-babel-hotreload-example\src\components'
 @ ./src/components/route_example_2.jsx 10:0-35
 @ ./src/components/routes.jsx
 @ ./src/components/main.jsx
 @ ./src/components/root.jsx
 @ ./src/app.jsx
 @ multi ./src/app

ERROR in ./src/components/index.jsx
Module not found: Error: Can't resolve 'prop-types' in 'C:\Users\examples\react-webpack-babel-hotreload-example\src\components'
 @ ./src/components/index.jsx 10:0-35
 @ ./src/components/routes.jsx
 @ ./src/components/main.jsx
 @ ./src/components/root.jsx
 @ ./src/app.jsx
 @ multi ./src/app

ERROR in ./~/material-ui/ReactFragment/ReactFragment.js
Module not found: Error: Can't resolve 'fbjs/lib/emptyFunction' in 'C:\Users\examples\react-webpack-babel-hotreload-example\node_modules\material-ui\ReactFragment'
 @ ./~/material-ui/ReactFragment/ReactFragment.js 23:21-54
 @ ./~/material-ui/utils/childUtils.js
 @ ./~/material-ui/RaisedButton/RaisedButton.js
 @ ./~/material-ui/RaisedButton/index.js
 @ ./src/components/index.jsx
 @ ./src/components/routes.jsx
 @ ./src/components/main.jsx
 @ ./src/components/root.jsx
 @ ./src/app.jsx
 @ multi ./src/app

ERROR in ./~/material-ui/ReactFragment/ReactFragment.js
Module not found: Error: Can't resolve 'fbjs/lib/invariant' in 'C:\Users\examples\react-webpack-babel-hotreload-example\node_modules\material-ui\ReactFragment'
 @ ./~/material-ui/ReactFragment/ReactFragment.js 27:17-46
 @ ./~/material-ui/utils/childUtils.js
 @ ./~/material-ui/RaisedButton/RaisedButton.js
 @ ./~/material-ui/RaisedButton/index.js
 @ ./src/components/index.jsx
 @ ./src/components/routes.jsx
 @ ./src/components/main.jsx
 @ ./src/components/root.jsx
 @ ./src/app.jsx
 @ multi ./src/app

ERROR in ./~/material-ui/ReactFragment/ReactFragment.js
Module not found: Error: Can't resolve 'fbjs/lib/warning' in 'C:\Users\examples\react-webpack-babel-hotreload-example\node_modules\material-ui\ReactFragment'
 @ ./~/material-ui/ReactFragment/ReactFragment.js 31:15-42
 @ ./~/material-ui/utils/childUtils.js
 @ ./~/material-ui/RaisedButton/RaisedButton.js
 @ ./~/material-ui/RaisedButton/index.js
 @ ./src/components/index.jsx
 @ ./src/components/routes.jsx
 @ ./src/components/main.jsx
 @ ./src/components/root.jsx
 @ ./src/app.jsx
 @ multi ./src/app
Child html-webpack-plugin for "index.html":
         Asset    Size  Chunks  Chunk Names
    index.html  568 kB       0
webpack: Failed to compile.

Thanks

Should be fixed now.


⚠ ClamAV has degraded performance

In d1aaa4b, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 199 ms

Resolved: ClamAV performance has improved in 5db462a.


Add support for subroutines operating in multiple scopes

The current way we set annotations only allows for subroutines to be used
in a single vcl state function.
Fastly is allows that and it also leads to copy-pasting functions with different
annotations, which is not a good pattern.
I only implemented them as annotations in the form @<state_name> instead of supporting them in the
function name because I think it will be weird to have sub foo_recv_miss_deliver
Fixes #84
Signed-off-by: Sotiris Nanopoulos<EMAIL_ADDRESS>

I would suggest one thing a user also can define multiple scopes by separating commas like @recv,miss, what do you think about it?

I can swap to that instead of multiple lines, that might be cleaner.

re @scope:

We should open an issue. My only concern is that this is a breaking change so I am not sure if there is a way to make it easier either via tooling or in some other way

@ysugimoto any middle ground solution that I could implement to make this pr better and also maintain backwards compatibility?

@ysugimoto any thoughts on the above?


Add support for num_layers in sequence generator decoder

Code Pull Requests
Address this item from project board.
Works for simple baseline for all cell types (rnn, gru, lstm), num_layers >= 1 with no Attention and no beam search.
Work remaining: support beam search and Attention

This commit completes the work.  PR is read for review.
Support for num_layers was accomplished with tf.keras.layers.StackedRNNCells.  Modification were made to allow SimpleRNNCell, GRUCell and LSTMCell types to work with StackedRNNCells with all parameter combinations shown in this table.

num_layers
Attention
Beam Search

1
No
No

1
No
Yes

1
Yes
No

1
Yes
Yes

>1
No
No

>1
No
Yes

>1
Yes
No

>1
Yes
Yes

In addition to the above change, this PR

add tests to tests/integration_tests/test_sequence_feature.py for the num_layer changes.
store sequence probabilities to disk as follows:

PROBABILITIES have shape b x s, where each element in the sequence is the probability of the token at that position in the sequence.
PROBABILITY have shape b, where the values are the sum of the logs of probabilities up to the length of the sequence, i.e., sum does not include probabilities for trailing  tokens.


Masonry: Implement graph early bailout feature

Summary
Tho determine the best position for the two column item the current behavior creates all the possible combinations of items on the graphBatch and then looks for the one with the minimum whitespace, although this works is very inneficient since we could have a good enough answer. To improve it this PR implements a new feature to exit early the algorithm when a certain whitespace threshold is found.
Tests
Manual tests showed a really awesome improvement using a threshold of gutter2 px, reducing the number of iterationr from 326 to ~4 (limited manual tests)
Unit test added
Notes

To enable this a whitespaceThreshold must be set, but this PR doesn't set one, that would be on another pr to determine the best threshold
The future of this feature is very related to the implementation of a dynamic batch size and used together to create a better positioning
The process to track metrics with the improvements will come on a future PR

@manvoid - this just caught my eye.
It was fun to read - what's the difference between good enough vs best?
Also, to run through all 324 combinations, could the operation of pre-sorting be done server-side to be faster?


Access to file chooser's text field

In save mode, I have a file name to suggest to a user, but it appears like I have no any way to do this. FileChooser.setSelectedFiles(FileHandle...) does provide option to select specific file programmatically, but it depends on displayed file views and drops files that are not visible right now. I guess it wouldn't be a big deal to simply provide setter for the file name text field. What do you say?

I can't provide direct access to it because after adding to stage, file chooser refreshes itself which only preserves actual selected files and not field text.
I added a method that let's you set default file name which will be used when chooser is added to stage and no other files was already selected.

Yep, this should solve the case. Thanks!


Add versionid header in complete multipart if enabled

addresses https://github.com/localstack/localstack/issues/3514

Codecov Report

Merging #3643 (7f6988d) into master (f918635) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #3643      +/-   ##
==========================================
- Coverage   94.38%   94.38%   -0.01%
==========================================
  Files         420      420
  Lines       46073    46076       +3
==========================================
+ Hits        43487    43488       +1
- Misses       2586     2588       +2

Impacted Files
Coverage Δ

moto/s3/responses.py
95.93% <100.00%> (-0.19%)
:arrow_down:

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 f918635...7f6988d. Read the comment docs.

Codecov Report

Merging #3643 (7f6988d) into master (f918635) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #3643      +/-   ##
==========================================
- Coverage   94.38%   94.38%   -0.01%
==========================================
  Files         420      420
  Lines       46073    46076       +3
==========================================
+ Hits        43487    43488       +1
- Misses       2586     2588       +2

Impacted Files
Coverage Δ

moto/s3/responses.py
95.93% <100.00%> (-0.19%)
:arrow_down:

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 f918635...7f6988d. Read the comment docs.

This is now part of moto >= 1.3.17.dev242


[exptrk] update to 2024-01-01

Library name
exprtk
New version number
2024-01-01
Other information that may be useful (release notes, etc...)
https://github.com/ArashPartow/exprtk doesn't look to be maintained anymore.
URL to the latest version : https://www.partow.net/downloads/exprtk.zip
Unfortunately, there is no version in filename so package can be break at anytime...

exprtk repository will be updated in the future. This issue should be pending.


Tabs not properly centering on wide viewports

When running an Ionic v2 app (beta 10) in a browser on an iOS device, if the device is placed in landscape mode, the tabs are not centered across the viewport.

Hello, this may have been fixed already, would you give ionic 2 nightly a try and see if that solves the issue? To use Ionic 2 nightly just run npm install ionic-angular@nightly. Thanks for using Ionic!

I am having the same problem when running on an iPad.
When I do an ionic serve it looks fine:

However on the iPad the tabs as off center:

I will try the nightly version.

@jgw96 Upgrading to the latest nightly version did not work for me.

I was able to reproduce this issue using the conference app with 2 tabs in Chrome simulating an iPhone and iPad. Thanks for the issue! I'll look into this soon.

It seems adding the following css fixes it, but I'll have to see what all this affects:
ion-tabbar {
  justify-content: center;
}

So this was actually fixed by https://github.com/driftyco/ionic/pull/7159 in master. You can add the above code to your Sass file or you can wait until beta 11 is released and we'll have some steps to upgrade. :) I've confirmed this is fixed though with that PR. Thanks for the issue!


Questions about the connectivity

This is a wonderful work!!
I have a small question about the connectivity metric used, which i could not find the implementation in your code?

Thanks for your attention! This metric is from an open-source code. Thus, we don't put this in our code. You can find this metric code in this implementation link. I think it will help you and we will launch related work in the future so you can pay more attention to it. https://github.com/MIA2I/SkeletalSimilarityMetric

Thanks for your attention! This metric is from an open-source code. Thus, we don't put this in our code. You can find this metric code in this implementation link. I think it will help you and we will launch related work in the future so you can pay more attention to it. https://github.com/MIA2I/SkeletalSimilarityMetric

Thank you for your reply! I will refer to the materials you kindly provided :) Nice work! Congrats to you!


Appstudio update test-component-pac-pkfe

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-pkfe-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
21 seconds
clone-repository

✅ Succeeded
35 seconds
build-container

✅ Succeeded
11 seconds
deprecated-base-image-check

✅ Succeeded
11 seconds
inspect-image

✅ Succeeded
38 seconds
clamav-scan

✅ Succeeded
9 seconds
clair-scan

✅ Succeeded
11 seconds
sbom-json-check

✅ Succeeded
10 seconds
label-check

✅ Succeeded
5 seconds
show-sbom

✅ Succeeded
8 seconds
show-summary

Pipelines as Code CI/test-component-pac-pkfe-on-pull-request has successfully validated your commit.


Small improvements in logging and proper support for 0 retry_limit

Various small improvements:

Include the error in the retry limit exceeded logs.
Log the error directly for driver issues, this will properly fill in the log message
retry_limit might be 0, in this case we don't need to post the retry message at all.

cc @wikimedia/services

Coverage decreased (-0.2%) to 95.332% when pulling c3afd32edbe2d10569bf5b62193382ef97f3756b on Pchelolo:log_improvement into 617b5e9af200ae4351b8f16385845d08ffa7e736 on wikimedia:master.


Recent senders- bugfix - handle topic edits

@amanagr it looks like you didn't squash as you'd intended (or maybe didn't push?)

I did. There were 3 commits.

I see.  I'd squash the remaining commits too, since its just adding some unused code without tests in the first one :).  I can review and take care of that if appropriate.

Sure! I actually, I am yet to find a balance between splitting changes in proper commits. I find it hard to  split one commit into multiple, so I usually split pieces of work into commits, so that they can be easily squashed later on.

Yeah, that's a fine approach.  But the way I'd think about it is that a commit should be a minimal atomically useful conceptual change.  I think it's rarely useful to split by which file is modified as is done here, for example; each commit is just one half of the system (and often is totally untested code until everything is included).  It's easy to re-split later in any case, and as a reviewer, it's easy to understand the hunks in different files independently if one desires.

This generally looks good, but can you fix it for the case that the two topics are on different streams?  That's going to come up soon.

This now also handles stream edit updates for topic.

@timabbott updated this.

@timabbott updated this with requested changes.

Merged, thanks @amanagr!  This should take a nice chunk out of the larger recent_senders PR :).


QueryReturnedNotFoundException: 404 Not Found

Hello guys! I was scraping 100 posts from each IG user then after the second user finished, it returned this error:
QueryReturnedNotFoundException: 404 Not Found
What does this mean?
Thanks!
The whole error is:
QueryReturnedNotFoundException            Traceback (most recent call last)
 in ()
4 for post in posts:
5     start = time.time()
----> 6     ig_name = post.owner_username
7     if num_users == 5:    #number of users to output
8         break
~\Anaconda3\lib\site-packages\instaloader\structures.py in owner_username(self)
171     def owner_username(self) -> str:
172         """The Post's lowercase owner name."""
--> 173         return self.owner_profile.username
174
175     @property
~\Anaconda3\lib\site-packages\instaloader\structures.py in owner_profile(self)
164                 # is contained.
165                 # Note that we cannot use Profile.from_id() here since that would lead us into a recursion.
--> 166                 owner_struct = self._full_metadata['owner']
167             self._owner_profile = Profile(self._context, owner_struct)
168         return self._owner_profile
~\Anaconda3\lib\site-packages\instaloader\structures.py in _full_metadata(self)
132     @property
133     def _full_metadata(self) -> Dict[str, Any]:
--> 134         self._obtain_metadata()
135         return self._full_metadata_dict
136
~\Anaconda3\lib\site-packages\instaloader\structures.py in _obtain_metadata(self)
126     def _obtain_metadata(self):
127         if not self._full_metadata_dict:
--> 128             pic_json = self._context.get_json("p/{0}/".format(self.shortcode), params={})
129             self._full_metadata_dict = pic_json['entry_data']['PostPage'][0]['graphql']['shortcode_media']
130             self._rhx_gis_str = pic_json['rhx_gis']
~\Anaconda3\lib\site-packages\instaloader\instaloadercontext.py in get_json(self, path, params, host, session, _attempt)
265                 raise QueryReturnedBadRequestException("400 Bad Request")
266             if resp.status_code == 404:
--> 267                 raise QueryReturnedNotFoundException("404 Not Found")
268             if resp.status_code == 429:
269                 raise TooManyRequestsException("429 Too Many Requests")
QueryReturnedNotFoundException: 404 Not Found

QueryReturnedNotFoundException means that a 404 HTTP response came while obtaining some of the requested data. Your call stack trace suggests that it happened while obtaining a post's full metadata structure, which queries the URL https://www.instagram.com/p/SHORTCODE/. Here, QueryReturnedNotFoundException most probably means that the Post does not exist.
My first guesses: If the Post was instantiated with Post.from_shortcode(), the shortcode might have been invalid. If the Post was returned by one of Instaloader's get_*_posts() methods, the unlikely case might have happened that the Post was deleted immediately before Instaloader could query its metadata.

The Post was instantiated from: posts = L.get_hashtag_posts('hashtag')
We're trying to get the usernames of the posts that have this particular 'hashtag'. That hashtag returns around 200 million posts. The weird thing is every time I rerun the code, 404 error is raised after every 150 - 200 posts scrapes from 2-3 usernames. I have time.sleep(60) for every user. I'm not sure if Instagram blocks me for a few seconds after I scraped a certain number of posts. Also, I am scraping the data but I am not logged in. I'm not sure if this has an effect on the error.

I am getting the 404 error right away not even after downloading 1 image using this simple #darthmaul tag, but #darthvader works. I wonder if we all get the same error using the same tags.

@aandergr can you please explain how 8764947 fixes that instagram may randomly return 404 on hashtag scans?

Of course it doesn't fix that instagram may randomly return 404. Obviously, we cannot do that. But it handles how Instaloader reacts when a 404 is returned.

Been getting this all day today on a perfectly working account, in a completely normal use case of:
L = instaloader.Instaloader()
L.login("username", "password")

word = "attention" # random, real account for testing purposes

profile = instaloader.Profile.from_username(L.context, word)

print(profile)


Possibility of Express Middleware for perm checking

I'm trying to work out how I might be able to perform an access control permission check using a code structure of the form:
  app.get('/admin/users',
    setRender('admin/users/search'),
    setRedirect({auth: '/'}),
    havePerm.readAny('profile'),
    adminusers.searchProfile);	// Users Search/results

rather than the current Express.js example methodology which amounts to predefining each permissions check and testing it within what is effectively the controller of an MVC application. It seems that such an express middleware would simply subclass the ac.can() method, returning an express-appropriate response rather than the true/false value, but I don't want to reproduce effort, if someone (3rd party) has already done this, or it's on the roadmap, completed in an unreleased branch somewhere.
Similarly, and more trivially, it might be worthwhile to include a mongoose plugin that inserts a role property in a known location in user models, to facilitate the above permission checking style, being performed where it should be in an MVC application.
If this is already out there, and I've simply overlooked it, please kindly point me to the relevent code/project. If not, I'm willing to undertake this feature development, but having taken a cursory look at the accesscontrol codebase, I'd need a little guidance as to the code structure, in order to implement it efficiently.
Thanks,
Colin

Hi Colin. I understand the need but I'm not willing to couple AccessControl with a framework or library. Not in this main repo at least. For this to be clean and maintainable, we need to embrace SRP.
I'm also currently not planning on releasing an official express middleware or mongoose plugin. I'd love to see them being developed though. You're very welcome to write one! I'll try and answer if you'll have questions.
As far as I know, there is some efforts for express integration by @pawangspandey. (no affiliation)

@onury Thanks for the quick reply. Your sentiments regarding SRP are quite reasonable, and I've felt the same about my projects over the years. I took a look at @ pawangspandey's efforts, which don't look like they'll meet my needs. Looks like I'll be tackling this one. I'm leaning toward stubbing out my own CRUD chain methods off .can/havePerm because so far it seems subclassing for chained calls in express middleware is structurally near impossible.

You don't have to use chained methods.
i.e. see #permission in API reference.

Closing this. Feel free to continue if you have further questions.


Example Component Recommender Analytic

As a concrete example of an analytic (referenced generally in #59), it would be nice to add a component recommender/completer to make sure everything is in place so we can experiment with models trained from https://github.com/symbench/spice-completion/.
For the purposes of this issue, the "mock" recommender could simply return the top 3 most commonly used components in the netlist. Concretely, it might be nice to have the mock recommender use the netlist representation (not the GME nodes) and return the top three components with confidences (probably just some hard-coded values for the mock analytic).

@brollb Thanks. This makes sense to me. I will get started on this mockup plugin.


[Feature Request]: Migrate AsyncStorage -> MMKV

Feature Request
I get it may not be wanted due to the fact most people know AsyncStorage, but do we think it would be a worthwhile upgrade to migrate from AsyncStorage to MMKV.
Pros

Significantly faster
MMKV has encryption support for secure local storage
Synchronous storage, so simpler application code

Cons

Slightly higher learning curve for people adopting ignite (but still easier to use due to synchronous storage)
Due to the fact it's built on JSI the the Chrome Debugger will no longer work and flipper is the alternative

Support
There appears to be an integration for persisting mobx store already too, though I haven't tested this yet  https://github.com/mrousavy/react-native-mmkv/blob/master/docs/WRAPPER_MOBX.md
Realised a major caveat to it would be that expo is not able to autolink JSI yet, but there appears to be a plugin that can be configured for expo managed projects https://github.com/mrousavy/react-native-mmkv/issues/157 - so again something that would require some investigation if this sounds good.
I wanted to confirm if this is something that is agreed would be desirable, and I'm happy to work on a PR if so!

Hi,
In react-native-mmkv installation, it says for Android:

Since react-native-reanimated also uses JSI, there will be conflicts if you install both libraries at the same time. That's why the installation steps are different:

Would be any problem with using ignite x react-native-mmkv x reanimated?
And also i see reactotron also uses AsyncStorage. We need to migrate it too.

Recipe over at the Ignite Cookbook! https://infinitered.github.io/ignite-cookbook/docs/MigratingToMMKV

Recipe over at the Ignite Cookbook! https://infinitered.github.io/ignite-cookbook/docs/MigratingToMMKV

Hey, the link is broken, can you update it please?

@caiorrs updated! https://ignitecookbook.com/docs/recipes/MigratingToMMKV

Thanks, @frankcalise, for the tutorial! I've applied your tips. Now, I'm looking to integrate MMKV with Reactotron instead of AsyncStorage here. Any advice on making this switch?

@bi3ri hey yeah we should update that recipe (would welcome a PR if you want to help confirm and verify the steps!) to include the Reactotron plugin setup documented here: https://docs.infinite.red/reactotron/plugins/react-native-mmkv/


.Net: Kernel function arguments cannot be of type Array

Describe the bug
Kernel function with argument of type Enum[] throws on invocation.
I am writing a kernel function that finds toys with the following signature:
public string FindToy(Color[] colors) {...}
public enum Color { Blue, Red, Green }
Then I ask in the chat question to invoke my function.

This is the parameters schema sent to openAI sdk:
{{"type":"object","required":["colors"],"properties":{"colors":{"type":"array","items":{"enum":["Blue","Red","Green"]}}}}}
This is the arguments in the model responses' ToolCalls:
"{"colors":["Blue","Red"]}"
When invoking in this function SK throws this exception:
Function failed. Error: Object of type 'System.String' cannot be converted to type 'SKCopilotToolkit.Sample.SampleMockScenario.Plugins.ToysPlugin+Color[]'.

To Reproduce
Steps to reproduce the behavior:

Register the function with the following signature to the kernel:
[KernelFunction("FindToy")] public string FindToy(Color[] colors)
Ask the chat completion service for toys in some colors
examine the errors in the logs

Expected behavior
The function will be called with array of Enum values [Blue, Red]
Platform

OS: Windows
IDE: Visual Studio,
Language: C#
Source: SK NuGet package version 1.5.0

cc: @SergeyMenshykh

Hi @EdenTanami, thank you for reporting this issue. Here are two options you can use until a long-term solution is found:

Register the JsonStringEnumConverter converter for your enum. For example - [JsonConverter(typeof(JsonStringEnumConverter))]
Modify your function parameter type from Color[] to string[].


fix: Use put_header/3 instead of put_new_header/3 in auth step.

This change uses put_header/3 instead of put_new_header/3 in the Req auth step. This allows the auth step to be reused later in the pipeline to replace the authorization header e.g. when credentials are expired and need to be refreshed.
Closes #297.

Thank you!


Add support for Microsoft.AspNetCore.OpenApi strategy in NSwag

In .NET 7, we introduced support for a new Microsoft.AspNeCore.OpenApi package that generates an OpenApiOperation definition for a given minimal API endpoint using its method info and metadata. Calling the following code, will generate an OpenApiOperation for the given endpoint and store it in the endpoints metadata.
using Microsoft.AspNetCore.OpenApi;

app.MapGet("/", () => ...)
  .WithOpenApi();

This OpenApi operation includes the same information that would've been generated by the ApiExplorer but is designed to be customizable, such as:
app.MapGet("/", () => ...)
  .WithOpenApi(operation =>
  {
    operation.Summary = "Some summary here";
  });

It's also meant to take higher precedence than operations derived from ApiExplorer. To support this in Swashbuckle, we currently favor the OpenApiOperation stored in metadata with the OpenApiDocument generated by SwaggerGen (see https://github.com/domaindrivendev/Swashbuckle.AspNetCore/pull/2404). We also patch in the OpenApiSchema generated by Swashbuckle.AspNetCore (see https://github.com/domaindrivendev/Swashbuckle.AspNetCore/pull/2441).
For NSwag, it might make sense to implement this as a separate MinimalAspNetCoreOpenApiDocumentGenerator to make the factoring a little cleaner.

Any update on this?
In .NET 6 there is any way to add the summary or description? I'm having issues with minimal api

Waiting for this too. The minimal API needs better documentation support

I have created a basic reproduction of WithSummary not working using NSwag and working using Swashbuckle. This project is using .NET 7.
MinimalApi.zip

.NET8 is comming  Any new ideas?

@RicoSuter hello, sorry for direct tagging you
If you give more info there, I'll try to make a PR for fixing this.
It's really annoying now, that it's not clear how to use minimal api + nswag. WithName/WithDescription/WithSummary and other stuff do nothing in UI and it's a bit challenging to find such issues (that you're not able to add comments to API with Minimal Api and NSwag for now) in discussion

This is blocking me from using NSwag currently, so I had to temporarily revert to Swashbuckle despite that not having a commit in almost a year.

Apparently ASP.NET Core/Microsoft does not just populate API Explorer with additional metadata but invented a new way to provide metadata OpenApiOperation metadata object... NSwag needs to also look at this and use this over API Explorer metadata...

@RicoSuter
There's two aspects. to this:
The WithName and WithDescription extension methods populate EndpointMetadata which is accessible via the API explorer. https://github.com/domaindrivendev/Swashbuckle.AspNetCore/pull/2173 showcases how these values can be read from EndpointMetadata via the API explorer types.
The WithOpenApi extension method sets metadata in a similar way and you can access it via the same types. https://github.com/domaindrivendev/Swashbuckle.AspNetCore/pull/2404 showcases this.

If you're struggling with this you can add your own OperationProcessor to use the built in "WithOpenApi", "WithDescription" etc if you'd like.
Here's an example:
public class DotnetOpenApiProcessor : IOperationProcessor
{
    public bool Process(OperationProcessorContext context)
    {
        if (context is not AspNetCoreOperationProcessorContext aspnetContext)
        {
            // Will still include this operation - set to false to exclude it.
            return true;
        }

        foreach (var metadata in aspnetContext.ApiDescription.ActionDescriptor.EndpointMetadata)
        {

            if (metadata is OpenApiOperation openApiMetadata)
            {
                context.OperationDescription.Operation.OperationId = openApiMetadata.OperationId;
                context.OperationDescription.Operation.IsDeprecated = openApiMetadata.Deprecated;
                context.OperationDescription.Operation.Summary = openApiMetadata.Summary;
                context.OperationDescription.Operation.Description = openApiMetadata.Description;
            }
            else if (metadata is EndpointSummaryAttribute summaryAttribute)
            {
                context.OperationDescription.Operation.Summary = summaryAttribute.Summary;
            }
            else if (metadata is EndpointDescriptionAttribute descriptionAttribute)
            {
                context.OperationDescription.Operation.Description = descriptionAttribute.Description;
            }
            else if (metadata is EndpointNameAttribute nameAttribute)
            {
                context.OperationDescription.Operation.OperationId = nameAttribute.EndpointName;
            }
        }

        return true;
    }
}

Can of course expand on this, I only added what I needed myself

The summary annotations work when you use the WithOpenApi extension method, as documented here: https://learn.microsoft.com/en-us/aspnet/core/fundamentals/minimal-apis/openapi?view=aspnetcore-8.0#add-endpoint-summary-or-description
But the WithSumary() extension method does not work


fix pytorch pin

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@hmaarrfk I'm pretty sure this is the package that needs to be fixed first, then pytorch_sparse should be able to build then pytorch_geometric

I agree, if this is a C package and requires pytorch, it will need to be fixed.

See suggestions: https://github.com/mikemhenry/pytorch_cluster-feedstock/pull/1

notice that the pytorch pin has been added to each configuration yaml file.

@hmaarrfk Thanks!

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

This might need pytorch sparse. Which i just triggered again

@conda-forge-admin please restart cis

@conda-forge-admin please rerender

@conda-forge-admin please rerender


More guidance for managing & revoking

At IPFS Camp in July, some potential users had questions about token management & revocation. @expede provided IRL answers that seemed to satisfy them; seems like that knowledge would be good to publish.

It could be in a "Token Management" section in the Introduction page, after Hashing.
If there's a backlog of similar-level questions, it could go in a new catch-all "Usage" or "FAQ" section.

Great feedback, thanks @mishmosh ! I'm currently exploring having someone develop new materials including for the website (possibly a redesign). Keep the feedback coming; it's helpful for deciding what needs to go in there :)

Keep the feedback coming; it's helpful for deciding what needs to go in there :)

In that case, see #78.


Title Label on each bar

Hi All,
I am new to rickshaw. I want to display text on top of each bar. Please let me know how to do that. Thanks in advance.

+1
Xaxis isn't great for bar charts if you want the label below the bar
hoverDetail doesn't work well and jumps around displaying information for incorrect bars
with a stacked bar it would be nice to be able to label each segment on the chart.


High-Performance Order Storage (HPOS) support

Hi, when is High-Performance Order Storage (HPOS) support planned for the plugin?
Right now, woocommerce don't let me swap to hpos because of this plugin. 1 out of 30

When will the incompatibility with HPOS be solved?

Hi, @reiny99,
We have added this to our planned upcoming releases.
We are closing this issue for now. If there are any further requests, please write to<EMAIL_ADDRESS>Thanks,
Unzer Plugins team


Support the ability to write data into Apache Hudi Tables

https://github.com/StarRocks/starrocks/issues/30503


Funky event order (active before start)

testing #31475
Have an extension with listener as shown below. Debug a node.js app that just interval's forever and notice the following order of events. Weird is that CUSTOM EVENT comes first (tho not always) and that CHANGE ACTIVE comes before START
CUSTOM EVENT a07ab282-ca16-437c-be61-8637ecc0fb6d scriptLoaded
CUSTOM EVENT a07ab282-ca16-437c-be61-8637ecc0fb6d scriptLoaded
CHANGE ACTIVE a07ab282-ca16-437c-be61-8637ecc0fb6d
START a07ab282-ca16-437c-be61-8637ecc0fb6d
CUSTOM EVENT a07ab282-ca16-437c-be61-8637ecc0fb6d scriptLoaded
CUSTOM EVENT a07ab282-ca16-437c-be61-8637ecc0fb6d scriptLoaded
CUSTOM EVENT a07ab282-ca16-437c-be61-8637ecc0fb6d scriptLoaded
STOP a07ab282-ca16-437c-be61-8637ecc0fb6d

    vscode.debug.onDidStartDebugSession(e => {
        console.log('START', e.id);
    })

    vscode.debug.onDidTerminateDebugSession(e => {
        console.log('STOP', e.id);
    })

    vscode.debug.onDidChangeActiveDebugSession(e => {
        console.log('CHANGE ACTIVE', e.id);
    })

    vscode.debug.onDidReceiveDebugSessionCustomEvent(e => {
        console.log('CUSTOM EVENT', e.session.id, e.event);
    })

Before we know that the debug session is up-and-running, some scripts have already been loaded.
but CHANGE ACTIVE should not come before START

@weinand This is what I am seeing now. The order seems correct, but the start and active event seem to be send twice...
START 4909af9c-79ae-4367-96cb-eacb5fde16d6
CHANGE ACTIVE 4909af9c-79ae-4367-96cb-eacb5fde16d6
START 4909af9c-79ae-4367-96cb-eacb5fde16d6
CHANGE ACTIVE 4909af9c-79ae-4367-96cb-eacb5fde16d6
CUSTOM EVENT 4909af9c-79ae-4367-96cb-eacb5fde16d6 script
CUSTOM EVENT 4909af9c-79ae-4367-96cb-eacb5fde16d6 scriptLoaded

@jrieken I can not repro this.
I am using mac, here's my extension.ts. Let me know what you are doing differently
'use strict';
// The module 'vscode' contains the VS Code extensibility API
// Import the module and reference it with the alias vscode in your code below
import * as vscode from 'vscode';

// this method is called when your extension is activated
// your extension is activated the very first time the command is executed
export function activate(context: vscode.ExtensionContext) {

    // Use the console to output diagnostic information (console.log) and errors (console.error)
    // This line of code will only be executed once when your extension is activated
    console.log('Congratulations, your extension "tsext-first" is now active!');

    // The command has been defined in the package.json file
    // Now provide the implementation of the command with  registerCommand
    // The commandId parameter must match the command field in package.json
    vscode.debug.onDidStartDebugSession(e => {
        console.log('START', e.id);
    })

    vscode.debug.onDidTerminateDebugSession(e => {
        console.log('STOP', e.id);
    })

    vscode.debug.onDidChangeActiveDebugSession(e => {
        console.log('CHANGE ACTIVE', e.id);
    })

    vscode.debug.onDidReceiveDebugSessionCustomEvent(e => {
        console.log('CUSTOM EVENT', e.session.id, e.event);
    })

    let disposable = vscode.commands.registerCommand('extension.sayHello', () => {
        // The code you place here will be executed every time your command is executed
        const configuration = vscode.workspace.getConfiguration('testExtension', vscode.window.activeTextEditor.document.uri);
        configuration.get('resourceConfiguration');
        console.log(configuration.inspect('resourceConfiguration'));
        console.log(configuration.inspect('resourceConfiguration'));
        // Display a message box to the user
        vscode.window.showInformationMessage('Hello World!');

    });

    context.subscriptions.push(disposable);
}

// this method is called when your extension is deactivated
export function deactivate() {
}

Here's a js program that I am debugging
function fib(i) {
    if (i < 2) {
        for(var t = 0; t < 10; t++) {
            console.log(t);
        }
        return 1;
    }

    return fib(i - 1) + fib(i - 2);
}

console.log(fib(20));

Come to my desk, I am debugging another extension

Should this have been closed?