[Snyk] Security upgrade @docusaurus/preset-classic from 3.2.1 to 3.5.0

Snyk has created this PR to fix 1 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

docs/help-center/package.json
docs/help-center/package-lock.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Missing Release of Resource after Effective Lifetime SNYK-JS-INFLIGHT-6095116
  631

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

close in favor of #657


legacy references still

Across spoonx libraries you have changed lib references from github:/spoonx/* to just npm:*
There are still references in /dist/* sources to 'spoonx/aurelia-api' in this project so it's still loading that old reference as a dependency.

@meash-nrel That should only be on the branch, and there shouldn't be a tag for it yet. If there is, it's our fault and it'll have to be fixed.

@RWOverdijk dist= new build and we didn't make a new build on master for some time

To be extra clear. dist is only updated once there is a release

Ahh @doktordirk  beat me

It has to be extremely clear now haha


Only outputs about 500 bytes of text ?

hear -i some.mp3 > text.txt
or
hear -i some.mp3
... only outputs the same first 500 bytes of text
macOS Monterey version 12.4
Thoughts?
chris

Unless you pass the -d flag, which specifies on-device speech recognition only, the audio will be sent for processing to Apple's servers. They seem to have a hard limit on how much they're willing to process in one session.
Try adding the -d flag and everything should work. If not, let me know! This could be better documented, thank you for pointing it out.

No worries, and the -d flag worked perfect ... thank you for this!

I have the same issue with with non-English audio.  For completeness of the thread I will record it here. OS X 12.4, Intel using
hear -l pt-PT -d - i ...
I get the clear error message (which was kind of expected)
On-device recognition is not supported for pt-PT.
Has anyone tried longer clips with non-English languages on Apple Silicon?

You might want to add this info to the man page -- I had the same issue and needed to see the call-out to the flag here to figure it out.


Webapp

In this pull request, the following tasks were implemented:

When the home page loads, the webapp shows the list of currencies that could be filtered out by name or code,

Along with the currency name, I displayed some numeric values per currency that come from the API.

When the user clicks (or taps) on a category item, the application navigates to the details page.

In the details page, the webapp retrieves data from the API to show detailed data in the category.

When the user clicks on the "Back" button (<), the user navigates to the home page.

Add Testing to the app

The data retrieved from the API is stored in the Redux store.

use of a Filter stateless component to filter the data retrieved from the API.

Every page (the main page and the pages for each item) have a unique route within the SPA.

Thank you, @VICTOR-CHUKWU, for reviewing my project. I will implement the optional changes you requested.


Public URL of the trust store or URL to obtain public keys for verification with 3rd party app?

Hello,
we would like to integrate the validation into our gastro software so that our staff does not need to use their personal smartphones with the "CovPassCheck" app.
Are there publicly available trust stores or is there a publicly available URI where one can obtain the public key to a given kid from a given issuer or something like a JWKS?
Being forced by government regulations to grant access to indoor venues only for tested or vaccinated people, we would prefer an easy way to check the official digital vaccination certificates of the guests.
Kind regards,
Alex

This is a duplicate of https://github.com/eu-digital-green-certificates/dgc-participating-countries/issues/10
Please see https://github.com/eu-digital-green-certificates/dgc-participating-countries/issues/10#issuecomment-862060966 and the following discussion.


Make Schema Editor responsive

The editor itself on bs3 should be responsive. I guess it's a integration/ bs2 backport issue


CI build is failing

See: https://github.com/android/architecture-samples/runs/7899962768?check_suite_focus=true

Try adding -Dorg.gradle.parallel=false to gradle config


ImageService.Instance.SetPauseWork(true) appears to cancel tasks, rather then to pause them

FFImage 2.1.8
PCL
Android API 19
the same code snippet as previous issue (https://github.com/luberda-molinet/FFImageLoading/issues/341) results in all the tasks finish instantly with Cancelled: True when I trigger ImageService.Instance.SetPauseWork(true).
ImageService.Instance.SetPauseWork(false) subsequent call does not restore the tasks either.

The method name is quite misleading. It actually cancels all tasks. I think we must obsolete that method name and add some new methods for pausing / cancelling work.

Thanks for fast reply!
I would really appreciate if you can implement the initial idea since i try to improve RecyclerView performance with the recomendations from wiki - https://github.com/luberda-molinet/FFImageLoading/wiki/Advanced-Usage#scroll-high-performances

The original idea was to add image loading tasks once again manually. Notice this:
      // Here you should have your custom method that forces redrawing visible list items
      _myListView.ForcePdfThumbnailsRedraw();

But I agree, that could be solved with a simpler way. Eg. leaving only last N loading tasks and cancelling the others, etc, etc.

It's fixed, you can test it in newest nuget prerelease.


support GHC 8.10

only template-haskell 2.16 supports GHC 8.10

Thank you! I'm now making use of this.

Would be fantastic to merge!

@JonasDuregard Are you around? Might you be interested in adding another maintainer to help?

CI is up at https://github.com/chris-martin/sized-functors/actions, to demonstrate that the patch works.

Please merge and publish a revision.

https://github.com/tscholak/sized-functors/pull/1 updates this PR to support GHC 9; not sure if it would be better as a PR to this repo instead?


ORM: iORm

iORM. [Delphi]  Delphi ORM interface based useful to develop desktop and mobile application.

Thanks, added!


fix(versions): update Activiti/activiti-cloud-gateway versions

UpdateBot pushed helm dependency: common to: 1.1.25

UpdateBot commands:
updatebot push-version --kind helm common 1.1.25


Lack of SVG Fill Color Customization

Description
Greetings,
I'm reaching out regarding an issue I've come across while using Typst for document writing. Currently, there seems to be a limitation in changing the fill (color) of an SVG image within Typst. I'm keen to inquire if there's an existing method to modify the fill color, or if not, whether there's potential for implementing this feature. I am also interested in taking on the task of addressing this issue and would greatly appreciate any guidance on the best approach to achieve this enhancement.
Use Case
Users often require icons to complement text in their presentations. By enabling SVG fill color changes, icons can be more deliberately aligned with text aesthetics, enhancing the overall visual impact of documents.

Is there an SVG-specific feature that allows changing all colors used inside of the SVG? As far as I'm aware, there isn't so the only way to achieve this would be to replace all "fill" attributes in the SVG with the color you want to fill it with, which I think would be a very brittle approach. So I'm not sure how this should be implemented on our side.
Do you have an example of an SVG file you want to recolor?

I was writing a document on material 3 for a university assessment and wanted to use some material 3 icons from the material 3 website : https://fonts.google.com/icons, I have also attached a material 3 clock svg file as an example.

I see. Well, currently this is not possible without resorting to hacks. I think from the technical perspective, it would be possible to implement, but I think the main issue is in the details of how this should be implemented in practice:

How should the user be able to indicate the color? Should there be another attribute in the image function? If so, this would mean that the function has an additional attribute which only works for SVG files and not raster images, which is not so nice in my opinion.
Which colours should exactly be replaced? Should only simple fill colours be replaced or also fills like gradients and patterns? What about fills inside of masks?

But I think this feature has been requested before, so maybe it does mean that many people want it and we should add it. But for now, in the case of simple SVGs like the one you posted, it should suffice if you just change it manually by specifying the color yourself. In your case, changing it to the following will make the whole thing blue. You can change the color by changing it inside of the "fill" attribute:
<svg xmlns="http://www.w3.org/2000/svg" height="48" viewBox="0 -960 960 960" width="48">
<path fill="blue" d="M480-80q-82 0-155-31.5t-127.5-86Q143-252 111.5-325T80-480q0-83 31.5-156t86-127Q252-817 325-848.5T480-880q83 0 156 31.5T763-763q54 54 85.5 127T880-480q0 29-4 58t-12 56q-11-11-24.5-19T810-397q5-20 7.5-40.5T820-480q0-142-99-241t-241-99q-141 0-240.5 99T140-480q0 141 99.5 240.5T480-140q58 0 110-18.5t95-50.5q10 12 22 22t26 16q-52 43-116.5 67T480-80Zm300-161q-17 0-28.5-11.5T740-281q0-17 11.5-28.5T780-321q17 0 28.5 11.5T820-281q0 17-11.5 28.5T780-241Zm-153-46L453-468v-225h60v201l159 160-45 45Z"/>
</svg>

Alright. Thank you

Does this solve your issue?

No, the decode method does not seem to work for the vg, I download from the material 3 website, I tried changing the black color to another color but it remains the same ,however , I downloaded a random svg from the internet and the methods works correctly on it,  the code i used to text the random svg from the internet : #let original = read("clock2.svg")
#let changed = original.replace(
"#000000",
green.hex(),
)
#image.decode(original,width:25pt)
#image.decode(changed,width:25pt)
"

The code i used to text the clock svg from the material 3 website : "#let original = read("clock.svg")
#let changed = original.replace(
"#000000",
green.hex(),
)
#image.decode(original,width:25pt)
#image.decode(changed,width:25pt)"

But the colour does not change here , can this somehow be fixed, attached both the icons as well

The non working file does not contain any color information. Black seems to be the default, not explicitly mentioned. I'll have a look...

Thank you

#let colorize(svg, color) = {
  let blk = black.hex();
  // You might improve this prototypical detection.
  if svg.contains(blk) {
    // Just replace
    svg.replace(blk, color.hex())
  } else {
    // Explicitly state color
    svg.replace("<svg ", "<svg fill=\""+color.hex()+"\" ")
  }
}

= Old
#let c1 = read("c1.svg");
#let c2 = read("c2.svg");
#let c1 = c1.replace("#000000",green.hex())
#let c2 = c2.replace("#000000",green.hex())
#image.decode(c1,width:25pt)
#image.decode(c2,width:25pt)

= New
#let c1 = read("c1.svg");
#let c2 = read("c2.svg");
#let c1 = colorize(c1, green);
#let c2 = colorize(c2, green);
#image.decode(c1,width:25pt)
#image.decode(c2,width:25pt)

try that ... it's prototypical though

will do

it works thanks again

Note: color.hex() is now color.to-hex().

For your information, I'm working on a plugin that will allow you to do that easily. I hope that I will be able to get it done within the next few weeks/months.

The idea is to make a plugin that allows to preprocess SVGs. Currently, this will probably only include two things, but this list might be extended if I get new ideas:

Allow changing the color of all strokes/fills.
Allow changing the text font/size/etc. of all text elements.

It will be a real WASM plugin and use usvg to read the SVG into memory, change the properties accordingly and then write it again. The advantage of doing it this way is that it is much more reliable (e.g. it doesn't matter whether the current fill color is specified as fill="color" or via some CSS propery, as long as it's a valid SVG, it will handle it properly).

i understand, it seems suitable. Will you post a link here, once ready?

Sure. 👍

The only way I could see this being supported natively is with SVG's currentColor, but I think it's unsupported in usvg and it's also unclear what the API would be in Typst. The string-level preprocessing or plugin approach is quite flexible, so should we just close this?

The best way would be using stylesheet injection and then creating a Typst plugin that uses that to provide the functionality. Maybe at some point I will keep working on it.
But I agree that this is out of scope for Typst itself, so we can close this.


Creating a Bucket

Issue on page:        https://github.com/opentelekomcloud-docs/object-storage-service/blob/main/api-ref/source/apis/operations_on_buckets/creating_a_bucket.rst
We received customer feedback that x-obs-bucket-object-lock-enabled is not working. Instead x-amz-bucket-object-lock-enabled has to be used. Can you please check and either correct the API or the documentation. However, I hope this is the right context here. The full feedback was:
About the WORM feature, I was able to create the buckets with WORM feature using the OBS API.
Postman has a feature to calculate AWS S3 signature, which is the same method used by OTC, Huawei Cloud and other S3 similar services.
The biggest problem was that OTC documentation has wrong information. The header "x-obs-bucket-object-lock-enabled" should be
"x-amz-bucket-object-lock-enabled". Took a while to find that, but now it is working

Hello @gekimmel ! The WORM feature is available in both Native and S3 API, but right now there is no documentation for the S3 part (we expect to receive it closer to the end of January). So in the case of S3 API - it is correct, instead of x-obs- headers the x-amz- headers should be used. However there is no mistake in the documentation (as far as I can tell) - the link to the page you provided is for the Native API, and it works with x-obs- headers.

FYI: information on the WORM feature is now available also for S3 documentation.
https://github.com/opentelekomcloud-docs/object-storage-service/blob/main/doc/s3api/source/operations_on_buckets/put_bucket.rst :

Note on enabling WORM
x-amz-bucket-object-lock-enabled in the Optional request headers table
Sample Request and Response for Creating a Bucket with WORM Enabled


Next13-progressbar causing hydration issues

Describe the bug
Bug is found when I added this to my providers.tsx (along with redux) and hydration errors occur.
To Reproduce
Steps to reproduce the behavior:

Create providers.tsx
Paste in the provided code (whatever was in documentation + my redux provider code):

"use client";

import { store } from "./store";
import { Provider } from "react-redux";

import { Next13ProgressBar } from 'next13-progressbar';

export function Providers({ children }: { children: React.ReactNode }) {
  return (
  <Provider store={store}>
    {children}
    <Next13ProgressBar height="4px" color="#DE2f2f" options={{ showSpinner: true }} showOnShallow />
  </Provider>
  )
}

Expected behavior
I expected it to work (which is does) but I did not expect hydration errors.
Screenshots
No screenshots but here is my error msg:
Error: Hydration failed because the initial UI does not match what was rendered on the server.

Warning: Expected server HTML to contain a matching <style> in <body>.

See more info here: https://nextjs.org/docs/messages/react-hydration-error

Desktop (please complete the following information):

OS: MacOS
Browser Chrome
Chrome Version 120.0.6099.109 (Official Build) (arm64)

Smartphone (please complete the following information):
N/A (using macbook)
Additional context
Error is dismissible but I do not like the hydration error there, as it could cause complications in the future. Idk if its a problem with my code structuring or with the lib.

Putting it outside the redux provided  tag also does not work btw

Nvm, i found the issue. I had nested layout.tsx files, and I put the provider in the root one. That caused the errors. I will be restructuring my project to fix this

Yep, issue was with my code. I structured my code in a way where new body's were being mounted improperly. I will be removing the  tags in my nested layouts to fix this issue.

Okay. Thanks for clarifying


Incorrect Certificate thumbprint algorithm used in OAuth2Util in identity-inbound-auth-oauth

Describe the issue:
SHA-1 algorithm has been used in IS 5.7.0 and older versions for calculating the certificate thumbprint in oauth related components. This has been changed to SHA-256 in newest IS releases ( 5.10.0 etc). In IS 5.11.0, this has changed back to SHA-1 mistakenly
How to reproduce:
Start the server and check the JWKS URL (https://localhost:9443/oauth2/jwks)
Expected behavior:
SHA-256 thumbprint in KID and other places
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: IS 5.11.0

Hi @dimuthnc
Can we close this issue now?


More celery tasks guidance

Following on from this slack thread about celery tasks: https://octoenergy.slack.com/archives/C0CBS8P2L/p1585237819113600
This is the outcome of what @fmarani and I ended up doing with adjusting the error handling in octoenergy.interfaces.tasks.dunning.dunning_send_action_comm.
I've avoided mentioning Loggly in the conventions as this is a public document, but here's a screenshot from a dashboard we made in Loggly. It shows a huge number of failed tasks, but a lot of them probably just aren't a problem. Better to have these tasks succeed, but include debugging data in the return value, if that's something we need to dig into. My argument is that it would be easier to build and interpret monitoring tools if we follow this convention - but I appreciate it's a matter of opinion.

Closing, as it didn't feel like this was a general enough problem.


Mocky online: Wrong status code in a special case

Steps to reproduce

Create a 202 response
After that, duplicate the tab (browser) and create a 200 response (Since the preselected status code on page load is 200, just click generate)
Check the response status code for the 200, it will be 202.

A new Mocky version have been released, the code base have been fully renewed.
All previously tickets are closed. Feel free to reopen a ticket is your problem is still present.

A new Mocky version have been released, the code base has been fully rewritten.
I close all previously opened tickets. Feel free to reopen a ticket is your problem is still present.


Problem with Sql Trasformation

Description
I try to execute this script pg/sql with one parameter
"SELECT *"+
"FROM   pg_settings pg"+
" WHERE  pg.source = :desc;"
When paste the code the conversion is done but when execution the script, it return this error
Errore SQL [42601]: ERROR: syntax error at or near "sc"
Posizione: 61
With the version community edition 22.3.2,  the same script run correctly.
DBeaver Version
Community edition  22.3.5 (20.03.2023)
Operating System
windows 10 64 bit
Database and driver
PostgreSQL 11.12
Steps to reproduce

open dbveaver
connect with db
open new script
copy and past the script to trasformate
execute the script

Additional context
No response

Hi, @biggio78
Could you show me what query do you see in Query Manager when execute it? You can find instructions of how to use Query Manager here https://dbeaver.com/docs/wiki/Query-Manager/

Never mind. I reproduced the issue. It is happening because of \r only instead of \r\n at the end of the query.

here the result after insert the value ('default') of the parameter "desc"
SELECT *
FROM pg_settings pg
WHERE pg.source 'default'sc

As a temporary workaround you could try copying the query you paste and paste it again. It helped me.

The issue was solved in scope of #18982 and fix will be available in DBeaver 23.1.0.
If the issue will appear again for you in the new DBeaver version, please let me know.


Add custom vector type for doctrine

Add a custom vector type for managing vector in doctrine
Fixes https://github.com/theodo-group/LLPhant/issues/38

Thanks @chr-hertel and @kraynel . I've finished the PR. Feel free to make comment. I'll release it in the latest version


Spatial softmax instead of global avg pooling?

Hi, thank you for your beautiful code :heart:
In section III.B of your paper you mention that you replace the global average pooling in the ResNet with a spatial softmax.
However, I cannot find where this is done in your code.
I can only see where you change the batch norm for a group norm
https://github.com/columbia-ai-robotics/diffusion_policy/blob/0d00e02b45e9e3f37f4eeb68bff076b68d9e9d44/diffusion_policy/model/vision/multi_image_obs_encoder.py#L62-L69
and where you remove the fully connected final layer
https://github.com/columbia-ai-robotics/diffusion_policy/blob/0d00e02b45e9e3f37f4eeb68bff076b68d9e9d44/diffusion_policy/model/vision/model_getter.py#L15
but not where you change the average pooling.
Am I missing something or did you actually use average pooling, contrary to what's stated in the paper?

Hi @lorenwel good question! All end-to-end diffusion policy experiments are actually trained with hybrid_image_policy, which directly loads robomimic's vision encoder instead using the image_obs_encoder you saw.
Their implementation of ResNet18 encoder has SpatialSoftmax pooling enabled https://github.com/ARISE-Initiative/robomimic/blob/b5d2aa9902825c6c652e3b08b19446d199b49590/robomimic/models/base_nets.py#L705.
Our implementation of vision encoder is only used for pretrained vision backbone (r3m, imagenet) experiments.

Got it! Thanks for the swift response.


[move] IR parse errors must have locations-(87)

Motivation
[move] IR parse errors must have locations.
Test Plan
CI/CD Tests covers the test case.

/canary

:sunny: Canary successful

/land


freebsd.libc: fix build with LLVM 16+

Description of changes
Noticed failures in cross-trunk, e.g. https://hydra.nixos.org/build/243111774
According to https://github.com/freebsd/freebsd-src/commit/2ba84b4bcdd6012e8cfbf8a0d060a4438623a638, this is to be expected now that we have bumped to LLVM 16.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build pkgsCross.x86_64-freebsd.libcCross


confusing:  Users of the application have to separately install the .NET runtime.

Type of issue
Other (describe below)
Description
Publishing your app as framework-dependent produces an application that includes only your application itself and its dependencies. Users of the application have to separately install the .NET runtime.
Most of our customers are publishing to Azure (or another cloud) that contains the .NET runtime. We need better wording and should mention when you pub Azur App Svc framework-dependent, Azure has the runtime installed.
@adegeo @IEvangelist  please suggest better wording
After we agree on the wording I'll do a "you" ectomy. It reads much better to say
Publishing the app as self-contained
than
Publishing your app as self-contained
The doc is shorter and more accurate. The zillions of consultants that read this don't need to roll their eyes by all the "your"'s.
Page URL
https://learn.microsoft.com/en-us/dotnet/core/deploying/#framework-dependent-deployments-fdd
Content source URL
https://github.com/dotnet/docs/blob/main/docs/core/deploying/index.md
Document Version Independent Id
89967250-8c3b-1999-078f-94ad7518ed49
Article author
@adegeo
Metadata

ID: 85eba478-0374-bb9b-8fa7-d5cf7d520603
Service: dotnet-fundamentals

So are you thinking something like

Publishing your app as framework-dependent produces an application that includes only your application itself and its dependencies. Users of the application have to separately install the .NET runtime. If you plan to run the app in the cloud, make sure that the target framework is already available on the host.

@Rick-Anderson So are you thinking something like

Publishing your app as framework-dependent produces an application that includes only your application itself and its dependencies. Users of the application have to separately install the .NET runtime. If you plan to run the app in the cloud, make sure that the target framework is already available on the host.

That's too nebulous.
Publishing your app as framework-dependent produces an application that includes only your application itself and its dependencies. Users of the application have to separately install the .NET runtime.  The bold might be technically right but it reads like on Azure with a released version of .NET, you have to separately install the .NET runtime.
We should feature Azure first as that's where are customers are going and where we want to encourage them to go. I wouldn't start out telling them you have to separately install the .NET runtime.  I'd start out saying something like
Publishing your app as framework-dependent produces an application that includes only your application and its dependencies, excluding the .NET runtime. If you plan to run the app on Azure App Services and are using a released version of .NET, the .NET runtime is already installed on the service, so you don't need to install it separately.
AWS has eons of latency and they don't bundle the runtime for doggie years after the release. I'm not too worried about explaining AWS and other clouds.

@Rick-Anderson I don't agree with that direction (but I'll welcome feedback from the product group). We have tons of customers doing desktop and console apps. This documentation is speaking to .NET in a generic sense from the product's point-of-view. If we want to improve the content with some sections talking about cloud stuff, I think that's fine, but I don't think re-tuning the entire doc to be cloud-first is the appropriate direction.
I would expect the user to actually research their cloud service and what it has prior to deploying something to.
Adding @richlander @KathleenDollard who helped drive this article.

@Rick-Anderson but I don't think re-tuning the entire doc to be cloud-first is the appropriate direction.

I don't want to re-tune the entire doc, just this section, just the following minor change
Publishing your app as framework-dependent produces an application that includes only your application and its dependencies, excluding the .NET runtime. If you plan to run the app on Azure App Services and are using a released version of .NET, the .NET runtime is already installed on the service, so you don't need to install it separately.

I would expect the user to actually research their cloud service and what it has prior to deploying something to.

Why should they have to research and hunt around to find out you don't typically need to install .NET?
When you publish an ASP.NET Core app to Azure, a message is displayed "publishing framework-dependent" When the user googles "framework-dependent", this is what they get, the problematic text:

Publish framework-dependent
Apps published as framework-dependent are cross-platform and don't include the .NET runtime. The user of your app is required to install the .NET runtime.

The user of your app is required to install the .NET runtime.

That's so confusing to new users. Jane publishes a web app but the users of her app have to install the .NET runtime?

I would avoid mentioning cloud or Azure at all in this little section. Application publishing is very generic.
How about this:

Publishing an application project as a framework-dependent application produces a set of binaries that includes app code and dependencies but not the .NET runtime. The app can be run in any environment or machine where the correct version of the .NET runtime has been installed.

I think that users would be better served with technology-specific publishing articles. This would help SEO much more than putting blurbs in the content that doesn't get picked up:

How to publish an ASP.NET Core app
How to deploy an ASP.NET Core service on Azure
Publish a Windows Forms app
.NET Aspire deployments
Tutorial: Publish a .NET console application using Visual Studio Code

Now that I look at the article, not just the text here, I think the page needs to be re-written. It was written for a different time, back with .NET Core 1.0 and before we had a good cloud story.
You can see that in this image. The TOC says "deployment models" and the articles says "publishing overview". Those are not the same thing and are the crux of this discussion.

Ideally, the article would lead with a discussion of common deployment targets and how it is straightforward to produce apps and other artifacts (like containers) for them. And then the rest would be links to other content, along the lines of what @adegeo suggested. This approach would also enable us to talk about cloud services sooner, per what @Rick-Anderson suggested. We'd then move all of the framework-dependent and friends content to a different topic.
I've always found this topic to be an abrupt intro into the general topic of deployment. I've never found a good way of describing that.
I'm happy to help on this if you want to share a branch.

Great points everyone. Each article in this area has kind of evolved independently and there hasn't been an overall story. Perhaps we should start with planning a high level overview of what we want the area to describe.

Tagging @IEvangelist who has done some work on this article recently.

Tagging @IEvangelist who has done some work on this article recently.

Just read through all of the issue comments, I agree that this seems a bit dated, and each area has evolved independently. It makes sense to refocus on this area all up and plan on updates.

I think to start, we can just move the current content to its own topic and write a new intro. That should help us determine what's actually needed. I can start that if folks want.

I'll take a look at redesigning and rewriting this content in Dec. Thanks everyone!


Possibility of users changable settings?

This link describes the process by which a user can change extension settings.
I could not find a settings file referring to this extension (something of the form<package_name>/<extension_name>.jupyterlab-settings). While a file can be created by the user I am unsure which settings would be alterable.
Are there any settings currently alterable?

Thanks for raising this here, and apologies for the late reply.

Are there any settings currently alterable?

No, there are not. What settings do you think you would want to set?

Hi there. The most desired for me would be:

changing the relative size of the code window and table window (e.g. make either window take up the whole screen while using it)
make the table window dark themed

Otherwise, awesome extension! Thanks for the reply.

Hi @mcclaassen ,
Thanks for the feedback!
It feels to me like:

changing the relative size of the code window and table window (e.g. make either window take up the whole screen while using it)

This would be best served by being able to resize the two sections, rather than hard-coding a setting somewhere?

make the table window dark themed

jupyterlab-sql should just respect the current theme.
What do you think?

I agree with

This would be best served by being able to resize the two sections, rather than hard-coding a setting somewhere?

Typical mouse based window resizing functionality would be great.
As for

jupyterlab-sql should just respect the current theme

The coding-window does respect the theme but (for me) the table-window (where the query results are displayed) is still white. With the black theme of the coding-window (and the inability to reduce the table-window), this can be quite glaring.
Cheers!

Thanks for clarifying.


Bug 2099968: Only use credentials that are provided by the azure-inject-credentials container

The CSI driver supports reading Azure credentials either from a given secret or from a configuration file provided via an environment variable. Our injector container creates a valid credentials file that we pass in to the CSI driver via an environment variable.
Credentials provided by secrets have precedence over those provided via the environment variable. The CSI driver defaults to kube-system/azure-cloud-provider if no secret is provided via CLI arguments.
This secret doesn't exist in regular OCP clusters, but it does exist in Azure Red Hat OpenShift, but with limited data. This cause some some operations to fail due to lack of sufficient permissions.
In this patch we overwrite the default secret and we make sure to use the credentials provided by the azure-inject-credentials container.
CC @openshift/storage

/bugzilla refresh

/retest

/lgtm


Send instructions error "The device is not registered on the server"

Hey Anton,I tried to website demo send instructions is an error is as follows: "The device is not registered on the server" ,  I try to use Intellj IDEA configuration traccar but always fails -"{Error running traccar:GWT Fecet is not configured for module 'traccar' } and {Error running traccar:No GWT modules found is 'traccar'} ",My computer system is Linux version, you are MAC configured Course, difference is very big,Do you can write a Windows and Linux Intellj IDEA configured Course?  thank you! My email is<EMAIL_ADDRESS>

"The device is not registered on the server" means that your device didn't send any data. What is the device id and port?
As for Intellij, it looks like you are doing something wrong. Traccar is not a GWT project.
I am using Linux as well.

@tananaev     I fill in the data on the android app is Device id = IMEI and port = 5055, This is wrong?
Thanks to reply me.

That is correct if you are using the latest version.

@tananaev Is the latest ,From the website to download the app.

What do you see on the status screen on the app? Any location updates there or errors?
Also, if you want me to check on the server side, I need to know your account name or device id.

@tananaev My account name is<EMAIL_ADDRESS>and device ID is:<PHONE_NUMBER>64131

I don't see any messages from your device in the logs. What do you see on the status screen in the app?

@tananaev My application data is the device ID is:<PHONE_NUMBER>64131, server address: demo.traccar.org, server port: 5055, frequency: 300. The state is open .

In the app there is a status screen (menu -> status) which shows information about location updates and errors.

@tananaev Please find I send you an email containing pictures

You can find my email address in profile, but your should be able to attach photos here as well.

@tananaev android application send data status :send failed

It means that your device can't connect to the server.


dataframe_regression out of the latest pytest-regressions-2.0.2

Hi @leonardo-ostan-4ss,
Yes, dataframe_regressions was introduced after that release: https://github.com/ESSS/pytest-regressions/blob/master/CHANGELOG.rst.

I updated to version 2.0.2 but this fixture is not there as you can see from screenshot
Best regards,
Leonardo Ostan
Developer
+5541996538622 | www.4subsea.com
/ Sent from my phone
On 20 Oct 2020, at 13:03, Bruno Oliveira<EMAIL_ADDRESS>wrote:

Hi @leonardo-ostan-4sshttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fleonardo-ostan-4ss&data=04|01|leonardo.ostan%404subsea.com|346fc7987507456b99c008d874e7c919|cdf4cf3dde2349cfa9b0abd2b675f253|0|0|637387886144427789|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=TX0oXaxH33tLwpavoYDzxbc1KWWWuuOV4AeystNNgEY%3D&reserved=0,
Yes, dataframe_regressions was introduced after that release: https://github.com/ESSS/pytest-regressions/blob/master/CHANGELOG.rsthttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FESSS%2Fpytest-regressions%2Fblob%2Fmaster%2FCHANGELOG.rst&data=04|01|leonardo.ostan%404subsea.com|346fc7987507456b99c008d874e7c919|cdf4cf3dde2349cfa9b0abd2b675f253|0|0|637387886144427789|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=5i8ljRzRbsx1sxeJcYRG1K24qCLQKQc5vlXyeyQmPw8%3D&reserved=0.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FESSS%2Fpytest-regressions%2Fissues%2F36%23issuecomment-712771339&data=04|01|leonardo.ostan%404subsea.com|346fc7987507456b99c008d874e7c919|cdf4cf3dde2349cfa9b0abd2b675f253|0|0|637387886144437785|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=pyAhilE51%2FIEOdiviM6dD5xvofkeclyfmI67kQ3lWKk%3D&reserved=0, or unsubscribehttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANNFQDX6NWZVLJLW4T23HCDSLVVAHANCNFSM4SX3LBIQ&data=04|01|leonardo.ostan%404subsea.com|346fc7987507456b99c008d874e7c919|cdf4cf3dde2349cfa9b0abd2b675f253|0|0|637387886144437785|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=t060mkt9jHOITpeHYHfFUnKR3U0pQ6GlOKNvb2sgpjU%3D&reserved=0.

I updated to version 2.0.2 but this fixture is not there as you can see from screenshot

Yes, that fixture will be available in 2.1.0, which has not been released yet.


Exemples of GET with query string params

Hi, in the test project there is no example of a GET request with query string parameters, is there one somewhere?

Yes, it works the same like the other params:

https://github.com/javalin/javalin-openapi/blob/main/openapi-specification/src/main/kotlin/io/javalin/openapi/OpenApiAnnotations.kt#L62


sched: Don't duplicate caller file handler when creating kernel thread

Summary
kernel thread should have only the starndard file i/o just like idle thread
Impact
Kernel thread doesn't inherit the caller file handler anymore
Testing

@xiaoxiang781216
My automatic test failed with this PR.
It seems that the lpwork and the hpwork do not start now.
Please confirm what is happening now.

@xiaoxiang781216 My automatic test failed with this PR. It seems that the lpwork and the hpwork do not start now. Please confirm what is happening now.

Let's revert for now and continue more close look on this.

@xiaoxiang781216 My automatic test failed with this PR. It seems that the lpwork and the hpwork do not start now. Please confirm what is happening now.

I think I find the root cause:
https://github.com/apache/incubator-nuttx/blob/master/sched/group/group_setupidlefiles.c#L77-L104
The opening handle in the above code belong the caller task, not the new task.


522 Connection Timed Out

Self explanatory.Why is it showing "Connection Timed Out"?

AniList is down right now


Run "dev dash up" as part of the Ansible playbook?

Looks like dev dash up runs by default when you call dev up, but in my case I didn't have any project to run with dev up but just wanted to run some "peripheral" container and use nginx-proxy for it.
Eventually I figured I need to run dev dash up, but maybe it's worth adding it to the playbook or at least mention in the README.

That's a great suggestion. I will add it to the README.

I found it difficult to create a machine, make sure it is up, and then bring up the dns containers on it. Is there a way to reliably do that?

@manojlds Is that part of this issue or a different issue? What problems are you running in to?

@silvamerica I misunderstood the issue. My problem was creating the machine and spinning up the containers for dns via the playbook doesn't work as the VM will be restarting when you try to start the containers.


Streaming _json

Has anyone considered streaming json to the client?
In the _json function you're calling $json = ($encode) ? json_encode($data, $option) : $data; and then sending a response with the result as part of the payload.
That uses a lot of memory, $data itself might be huge, but then the $json is basically double that with json encoding overhead added.
But what about instead: Send the response headers immediately without content-length, and then loop over the data to be encoded, turn it into json, until a buffer fills up, and then send the buffer to the client, repeat until data is finished.

I have tried this - using a header('connection: close') on regular php works great.  However for some reason flight doesn't allow the "connection" header to be modified.  I'm looking to blame my webserver, but it cooperates outside of flight.

Putting this in a route works:
       header('X-Accel-Buffering: no');         header( 'Content-type: text/html; charset=utf-8' );         echo 'Begin ...
';         for( $i = 0 ; $i < 10 ; $i++ )         {             echo $i . '
';             flush();             ob_flush();             sleep(1);         }         echo 'End ...
';

Hey,
I just fought this issue myself, i got it working atlast, so, here's working pseudo code:
// Ensure headers are sent before streaming data to client
$response = Flight::response();
$response->header('Content-Type', 'application/json');
// Ensures that $sent is set to true, so that _stop does not mess with our buffer
$response->send();

// Start building the output json
echo '{"something":[';

$cnt = 0;

while (($port = $ports_db->fetch_assoc())) {
    if ($cnt > 0) {
        echo ',';
    }

    $cnt++;
    echo json_encode($port);

    ob_flush();
}

echo '}';

I had a "fun time" debugging why the last two or so objects kept being missing from my output, or with my buffer being replaced with the last echo.
Turns out that _stop does some shenanigans to the buffer, so to prevent that, just get the response object immediately, call send() and then do your streaming code.
Good luck to anyone trying this!

Thanks for your help with this! I'll close the issue for now.

I'd argue that my hack is a workaround and not a proper solution. This issue was requesting a proper way to be added to the framework.

I agree with you, but I just see this as being the minority of usage, not the majority of usage. In time we'll have other Flight packages and we'll probably have something called JsonStream or something like that that is not part of the core, but will allow for functionality like this. I have to do something similar to this for work as json_encoding() 1000's of objects runs out of memory in our case.

So then just add it as a feature, many people are probably hitting this issue, just like you and me, only a tiny percentage of people check Issues, much fewer reports them.

I'll reopen this so we can put a pin in it for later.

In doing some research, it looks like we'd have to make some modifications to the following:
Engine.php
We'd have to modify the filter for start so that it likely doesn't run, or handles things differently.

Then in the _stop() method, we'll need to possibly do an ob_get_flush() instead if there's some var set to stream instead of capture the output buffer.

I think the other comments on this thread will really help get this pointed in the right direction.
Also, no idea why it hates the Connection header?

Finally got this in place. https://github.com/flightphp/core/pull/547 Feedback welcome!


[rancher-monitoring] CPU and memory usage doubled in Rancher/Workload (pods) and Rancher/Workload dashboards

All versions up to 100.0.2 of the rancher monitoring chart seem to be affected.

The solution is to include a filter for container!="POD", container!="" as else the parent cgroups memory and cpu usage will be added to the usage of each individual container which results in a doubling of metrics.

I don't see this fix actually merged into any of the releases, and it still happens in rancher-monitoring:103.0.3+up45.31.1. Am I missing something?

Duplicate of https://github.com/rancher/rancher/issues/44726


net: lib: http_server: give application access to request headers

Allow application to access request headers of a dynamically handled request.
This can be done by the application providing a pointer to an array of header structures in the dynamic resource detail. These contain the relevant field names the application is interested in, and buffers for the HTTP server to populate the header value into, so doesn't significantly increase the memory requirement of the server itself. If the application does not use this feature, there is no need to change anything in the resource detail (new fields can just be zero-initialised).
It would be good to get some feedback on whether this would be a useful addition to the HTTP server, and whether this approach towards the feature seems OK?
My rationale behind this is that it's a first step towards supporting some kind of authentication of clients (e.g. HTTP basic auth or a cookie-based approach), for which we would also want the server to be able to send application-defined headers and status codes. But I imagine the request headers could be useful for other purposes too.
There's still a few things to do:

HTTP2 support
Add tests
Remove "test code" from sample application, and perhaps add a more useful demo of how this can be used

And a few open questions:

At the moment only dynamic resources are supported, would support for other resource types be useful? I think this would be more complicated to implement, since for static resources there is currently no callback from which the application could do anything with the headers.
If headers are too long to fit in the application-provided buffer, they are dropped without any indication to the application. It is logged at a DBG level in the HTTP server, but perhaps some flag to indicate a header which is too long might be useful?
For HTTP2 (not implemented yet), the :path pseudo-header will be needed to lookup the correct resource detail. But I don't see any specification in RFC 9113 of what order the headers should be sent in. So in theory it's possible that other headers are sent before the :path, and these headers would be dropped since the server does not know whether the application needs them. In practice I don't know whether any client application would actually send other headers before the :path though.

This can be done by the application providing a pointer to an array of header structures in the dynamic resource detail. These contain the relevant field names the application is interested in, and buffers for the HTTP server to populate the header value into, so doesn't significantly increase the memory requirement of the server itself.

That's true for simple cases, however we'd need a separate set of buffers for every registered resource interested in headers, so again, I don't think it'd scale well... And we need a separate buffers, since we can't predict whether for example when client A access resource A, there won't be a client B who would want to access resource B at the same time.
To be honest, the more I think about this, the more it seems to me that some sort of dynamic allocation in such case is the most reasonable/scalable way to go. A rough idea I have would consist of

A dedicated heap (which could be of size 0 if the application is not interested in headers) for the server to allocate buffers for headers that were registered by the application
An array of struct http_server_request_header or similar, preallocated at the client context level, with size configurable with Kconfig, or those entries could be dynamically allocated too and form a linked list. Each header could get an extra status field, indicating whether header is OK, or whether it's damaged in some way (like the aforementioned header too long case),
The resource callback could get an additional parameter, with a pointer to the array/list of headers.

To be honest, the more I think about this, the more it seems to me that some sort of dynamic allocation in such case is the most reasonable/scalable way to go

Agreed, that would cover the HTTP2 case and now that I think about it an application would likely want the same header fields for multiple URIs anyway, which makes allocating buffers at a resource level problematic.
The other alternative that comes to mind is some sort of argc / argv inspired approach where we have a static buffer per client to store any relevant headers that the application is interested in. Header fields and values would be put into the buffer separated by a NULL character, and the argv equivalent would be populated such that you have argv[0] = first header field, argv[1] = first header value, argv[2] = second header field etc.
This adds a bit of work on the application end (as you don't know which index corresponds to which header), but doesn't need any dynamic allocation. Both the buffer size and the argv length could be configurable to suit the application needs.
Don't think I'll get a chance to look at this soon as I'll be on holiday, but will have a go at putting an alternative together when I'm back

Trying out a different approach, using a single static buffer per client for headers to be stored in. This now supports HTTP2 as well.
The application can register headers to be captured globally for the HTTP server using an iterable section macro.
The server stores any relevant headers that the application has "subscribed" to into the buffer for every request (since URI may not be known before headers are received). Each header name and value is separated by a NULL character so that they can just be treated as strings, and a pointer to each string is stored in an array of struct http_header for the application to access later from a dynamic resource callback.
There is also a status stored, so that the application can tell is any headers were present but not able to be stored due to a lack of space. This is just a per-client status, so it's not possible to tell which header exactly was dropped.
The feature can be disabled by Kconfig to avoid any additional memory requirements for applications that don't need this feature, and the header buffer size and header array length can be configured.
I've left the original commit in there for now in case I need to refer back to it, but will squash down if/when it becomes ready for review.

I like the idea of client telling which headers it would be interested in, this way we could avoid extra memory use. Could we perhaps get all the headers by setting the capture value to *, just in case the application would like to get all the headers?
What about if the application would like to set some header values, could we use this in both direction?

The "wildcard" to capture all headers sounds like a nice addition.
I did have some thoughts on handling the other direction, where the application wants to set headers and status codes to send back to the client. But I think this will be tricky to do well without breaking changes to the HTTP server API, so might be worth doing as a separate PR. As it stands, this PR allows existing applications to work without any change to API or resource usage.
For sending headers from the application, my first thought on how to do it would be to add a function like:
int http_server_send_response(enum http_status status, const struct http_header *headers,
                              size_t header_count, const uint8_t *body, size_t body_len);

which can be called from within the dynamic resource callback to send a response (or partial response), instead of passing the data back to the server via the buffer parameter. Then change the return value of the dynamic resource callback to be:

0 --> Success, application has sent full response and callback does not need to be called again.
-EAGAIN --> Success, but application requires callback to be called again to send more data.
Any other negative error code --> Some error occurred

This has the added advantage of the dynamic resource callback only being called once in simple cases where the response can be sent in one go, rather than having extra application logic to check if a response was already sent for this request, and then return 0 from the callback to indicate it doesn't need to be called again.
There would need to be some extra logic in the server to handle things like:

Combine the server's own response headers with the ones specified by the application (perhaps allow application to override some of them, but need to be careful about e.g. Transfer-Encoding)
If application calls http_server_send_response with headers and a body, it can't send more headers or a different status code on subsequent calls (only more body data is allowed).

There might be some issues with this I haven't though of - I might have a go at putting together a draft PR to see how it fits together, but it won't be until the end of August at the earliest

Rebased and added some tests

@rlubos @jukkar any thoughts on this approach now that HTTP1/HTTP2 are both supported?
If it seems OK I can tidy up the commit history, remove the "test code" from the sample application and add some documentation updates before marking as ready for review.
I also added a separate PR as a proof of concept of how response headers could be sent in the other direction: https://github.com/zephyrproject-rtos/zephyr/pull/77982

I agree, the proposal looks good, just make it normal PR for a proper review.


LoRaWAN Gateway App not available

The LoRaWAN Gateway item says:
"The included gateway is a Tabs Hub that works with TheThingsNetwork."
The referenced PDF from 2017 says that:
"The Hub may be installed into an existing WLAN network as a repeater or create a new WLAN network while acting as a WiFi Access Point. In both scenarios the Hub is configure via TrackNet’s Tab Application available for both iOS and Android."
The Tabs.io App on the Google Android Playstore (March 21, 2018, Version 1.3.15, Requires Android version 4.1 and up) seems to be incompatible with all my devices, both new and old (Android version 10 and 9), so I suspect the Tabs.io App is too old.

Is there a supported configuration app?
Is there another way to configure the hub - like from a browser on a computer attached to one of the ethernet LAN ports?

Has anyone done this before?
Any documentation?

Should I just shut up and plug it in and hope that the default configuration doesn't expose my LAN to exploits via predefined passwords and other settings?

Thanks for your report.

We reconfigure the device to be just a LoRaWAN gateway, and disabled all TrackNet/Tabs features because the company does not exist anymore and things like the app don't work.
Specifically the wifi access point / repeater behavior was disabled also.
The device can be reconfigured via ssh but it's not documented in the official docs. If you are interested, I can help you get access to it.
It's a fair point to ask about security of plugging a device in your network. As far I could see when i configured the device, it's pretty safe because now that all extra features are disabled, it's the same network behavior than the open source DIY gateways in which a simple udp connection is opened as outgoing to the TTN router and there's no incoming connection possible.
I will address some of these points and questions tomorrow at the workshop, and I can hand out reconfig scripts for those how want to hack the device.

If you're eager to play with it, here's the ssh user
 TABSGWIP="<IP_ADDRESS>"
 TABSGWUSER="root"
 TABSGWPASS="tracknet"

You need to connect it on the black lan port to the ethernet port on your computer

@derrickoswald does that answer your questions?

I removed the credentials in case these are used in other devices out there in the wild, per recommendation of @tamberg

I would add to that makezurich-hardware-intro.md section something along the lines of:
Regarding security:
The device is configured to be just a LoRaWAN gateway, and all TrackNet/Tabs features are disabled (because the company does not exist anymore and things like the app don't work).
Specifically the WiFi access point / repeater behavior was disabled also.
It should be pretty safe because now that all extra features are disabled, it's the same network behavior that the open source DIY gateways have, in which a simple UDP connection is opened as outgoing to the TTN router and no incoming connection is possible.
The device can be reconfigured via ssh. If you are interested, SSH credentials can allow you to connect to it on the black LAN port to the Ethernet port on your computer. Ask one of the organizers for the SSH credentials.
Consider it closed with the above documentation.

Cool! Thanks! I just added it (after a bit of a rewording) https://github.com/make-zurich/makezurich-hardware-intro/blob/master/README.md#lorawan-gateway

Thanks @gonzalocasas. I shortened the text a bit. A small thing: Is any setup required, or does it go to your TTN account?

No setup required. When connected, it will work as "unclaimed gateway". If you want to claim it, to get access to the traffic tab on the TTN console, you need to register it (the usual setup with "using legacy pkt fwd"). The EUI of the gateway is printed on the bottom and also on the bags, but it's a 6-bytes EUI which needs the two extra bytes FFFE to be inserted right in the middle. For example, if the printed label says (the EUI is the second one in the label): (92) 580011AABBCC, you need to register using 580011FFFEAABBCC.


Not works with RequireJS

I try to load on a different ways to load with require but always have the same error :
mainHome.js:1 Uncaught ReferenceError: Layzr is not defined
Passing as path or require(['libraries','ender.min', 'layzr.min'].

@malcocerGit dist files have been updated in the latest release, and importing errors should be resolved


Update recipientfilter-properties.md

| CountryCode  | CountryCode  |Integers   |This property contains the numeric country/region designation from ISO 3166. For more information, see Country Codes - ISO 3166.   | Remove this
Please check Content Idea Request 87260:Change in the Filterable properties of the Dynamic Distribution Group change request

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/exchange-server/recipient-filters/recipientfilter-properties.md
:white_check_mark:Succeeded
View (exchange-ps)

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/exchange-server/recipient-filters/recipientfilter-properties.md
:white_check_mark:Succeeded
View (exchange-ps)

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/exchange-server/recipient-filters/recipientfilter-properties.md
:white_check_mark:Succeeded
View (exchange-ps)

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/exchange-server/recipient-filters/recipientfilter-properties.md
:white_check_mark:Succeeded
View (exchange-ps)

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


timing api calls

just some timings prints to check if DB is stalling!

Codecov Report

Merging #324 (f9d78ea) into master (e13563b) will decrease coverage by 0.04%.
The diff coverage is 14.28%.

@@            Coverage Diff             @@
##           master     #324      +/-   ##
==========================================
- Coverage   29.95%   29.90%   -0.05%
==========================================
  Files          32       32
  Lines        2367     2374       +7
==========================================
+ Hits          709      710       +1
- Misses       1658     1664       +6

Impacted Files
Coverage Δ

lnbits/core/crud.py
24.26% <14.28%> (-0.55%)
: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 e13563b...f9d78ea. Read the comment docs.


Allow for obeying of .gitignore

I typically have to run codemod on several directories in my working repository with the -d option. I would run it in the root of my working repository, but it includes some large irrelevant files that slow codemod down. Is there a way to have it obey .gitignore?

This would be a good feature! Maybe codemod could use git ls-files or something to find the list of files, instead of Python's os.walk?


Excluding tests on windows 32bit

Some tests that are already excluded on windows 64bit should also
be excluded on windows 32bit for the same reasons.
I've also tidied up some formatting, as tabs seem the rule for this
file, and it seems a few spaces snuck in. Fixed.
Signed-off-by: Adam Farley<EMAIL_ADDRESS>

I think I'm ok with this, although all the formatting changes make it harder to review than I'd like. It looks like there's just two extra sections that have been added here for win32 - is that correct?
I'll defer to Shelley for formal final approval though :-)

Yes, that's correct. Apologies for making things harder for reviewers with the formatting.
For other reviewers, most of the changes are formatting changes. The only real additions are on lines 465-469 and 1150-1154.

Reviewing with whitespace changes hidden make things much easier here https://github.com/adoptium/aqa-tests/pull/2827/files?diff=unified&w=1


[DOCS] Fix sphinx warnings

Based on #4908, this is the first PR to remove all doc warnings from Python doc strings. After this fix, total number of doc warnings is reduced to 103 on my local.

Hi @comaniac
Thanks for submitting the PR.

No the warnings you listed are not covered in this PR. This PR now only covers Python files. I was planning to send another PR for RST, but I can also commit to this one if you prefer.

Thanks @comaniac


libffi: Avoid clash with builtin malloc and libc malloc

A different source file, closures.c, includes dlmalloc.c but sets
various preprocessor defines before doing so to make sure it's built
appropriately.  Building a seperate dlmalloc.c.o can possibly generate
incorectly defined and conflicting symbols.

For future reference, see #67 and #68


⚠ Bofrost.es has degraded performance

In fb6471c, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1408 ms

Resolved: Bofrost.es performance has improved in 5eea25d.


Dkryvchuk/284 fingerprints to df

Ouch, look like we have not merged it yet :( Let's find a way to proceed


#2470 navigation tab on lineage view

Description
the lineage view page has the tabs for navigation now
Related Issue :
#2470
How Has This Been Tested?
go to [Management]-[Explore data]-[metadata]-[lineage view]
there are the navigation tabs
if it works, then ok
Need additional checks?
Types of changes

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

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

Additional Context

run build
deploy to 5

@ksparknot
need more vertical space between tabs and contents.

run build
deploy to 5

run build
deploy to 5


use executor-agnostic timers

This PR replaces the tokio Timeout with a pure futures-based approach: race the request with a delay from the futures-timer crate.
I haven't extensively tested this, but for my minimal use case it seems to work.
Supersedes https://github.com/google/tarpc/pull/296 and fixes https://github.com/google/tarpc/issues/297

Thank you! Things have been a bit crazy here but I'll try to get a review done this weekend.

futures-timer tracks timers in its own thread which makes it a mini-runtime on its own. Therefore I am strongly opposed to merging this because this would mean much bigger overhead to placate a minority of users (tokio is used by more async prod than all other runtimes combined).
A better solution would be to generalize the timer behaviour, but doing that would be non-trivial.

Yeah, I don't think that futures-timer is a desirable solution. Thanks for the effort though!


Call to undefined session

I try to use your sessionHandler, but can't make it work!
I don't use composer, so i install the SessionHandler.php, and create the table in my db.
Then i try to do this... i will write you what i can see:
`
$dsn = 'mysql:host=localhost;dbname=bd_prueba';

try{

$dbh = new PDO($dsn, 'user_prueba', 'Abc123.');

}  catch (PDOException $e) {

    echo 'Conection Fail: ' . $e->getMessage();

}

var_dump($dbh);

require_once 'SessionHandler.php';

$config['secondsUntilExpiration'] = 7200; // 30 minutes

$Session = new SessionHandler($dbh, $config);

var_dump($Session);

$Session->setData('nombre', 'Carlos');

`
When i run this code i can see this....
`
object(PDO)#1 (0) { } object(SessionHandler)#2 (0) { }

Fatal error: Call to undefined method SessionHandler::setData() in /var/www/vhosts/....../sessions.php on line 9

Can you help me?
Ty very much!

It looks like the object is not being instantiated. Try using the namespace as in:
$Session = new WolfMoritz\Session\SessionHandler($dbh, $config);

By the way, you also need to provide a salt in the config array. If the custom session handler was being instantiated it would have thrown an error that the salt was not set.
Thank you very much mate!


http/1.1 protocol error: headers count exceeds limit

istio ver : 1.19.4
2023-12-14T08:44:56.591559Z	debug	envoy client external/envoy/source/common/http/codec_client.cc:170	[Tags: "ConnectionId":"502"] Error dispatching received data: http/1.1 protocol error: headers count exceeds limit	thread=23

I want to increase http header count and http header size
The problem is that it is not possible with envoyfilter.
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  name: http-header-count-size-control
spec:
  configPatches:
  - applyTo: CLUSTER
    patch:
      operation: MERGE
      value:
        common_http_protocol_options:
          max_headers_count: 200 # default is 100
        max_request_headers_kb: 96 # 96 KiB is the maxumum allowed, default is 60 KiB.
  - applyTo: NETWORK_FILTER
    match:
      listener:
        filterChain:
          filter:
            name: "envoy.filters.network.http_connection_manager"
    patch:
      operation: MERGE
      value:
        typed_config:
          "@type": "type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager"
          common_http_protocol_options:
            max_headers_count: 200 # default is 100
          max_request_headers_kb: 96 # 96 KiB is the maxumum allowed, default is 60 KiB.

not working this enovy filter

use commonHttpProtocolOptions and maxHeadersCount?

it works for me, istio version 1.20.0.
EnvoyFilter and Pod's namespace is?


Add Support for CEN-IO devices

Updated CEN-IO class inheritance of EssentialsDevice to CrestronGenericBaseDevice

DIGIN-104
RY-104
IR-104

Tested in the field with CEN-IO-IR-104 and CEN-IO-RY-104.  Merged hotfix/huddle-room-interfaces branch.

This PR also addresses a missing interface on the HuddleRoom.


Environment.host is None in init event

Describe the bug
It seems there's no way to write initialization code that is guaranteed to run after the host is known. A work-around is to always specify -H on the command line and use it in the init event.
The docs at https://docs.locust.io/en/stable/api.html#locust.env.Environment.host also don't mention that this property can be None. Similarly, https://docs.locust.io/en/stable/api.html#locust.event.Events.init gives the impression that the event is called after everything is known, particularly the environment.
Expected behavior
Initialization logic should know what the target host is and I expect that Environment.host is known in the init event.
Actual behavior

When not specifying -H on the command line, then environment.host is None within the init event.
Steps to reproduce
@events.init.add_listener
def on_locust_init(environment, **kwargs):
    print(environment.host) # None

Start with locust.
Environment

OS: Ubuntu 20.04
Python version: 3.8
Locust version: 2.8.4
Locust command line that you ran: locust
Locust file contents (anonymized if necessary): See above

What you're suggesting is not possible - init happens before host is set by the user. But maybe you can use test_start?

Ah yes, that is in fact what I needed. I somehow had the impression it runs more than once per host, still trying to learn all the concepts. Thanks!

There's a detailed example here: https://github.com/locustio/locust/blob/master/examples/test_data_management.py


Water Sensors: IWY KS

Site completely offline. No apparent reason in data strings.

Cover rep 2 the middle sensor is going to be replaced, it is reading two readings sometimes.


API v4 Bug: Delete Body Unused

Hello,
It looks like the values are not set in the DELETE method but are set elsewhere.
See here: In the DELETE method code, the data parameter in the function call is never used (and thus the values variable is not either), whereas they are used in the PATCH method (and the other methods).

In the PATCH method code (and for other methods), the request is composed as expected.

@joshwolff1 This issue was fixed for python and released in v4 0.30.1

Thank you for the prompt fix.
I hate to re-open this, but I just noticed that GET has the same issue. I didn't notice this before


DRILL-3562: Query fails when using flatten on JSON data where some do…

…cuments have an empty array

Added set for ListWriters tracking to keep empty arrays for further initializing in ensureAtLeastOneField method.
Added check to avoid schema generating with field type "Late" and mode "Optional", replaced it to "Int" type in FlattenRecordBatch class.
Added unit tests to cover cases querying Json with empty arrays with flatten.

Looks good.

@amansinha100, could you please review new changes?

LGTM.  +1


adding l2 distance numba and numpy versons

This PR contains the changes for adding l2 distance numpy and numba versions as a part of the redesign.

Ran the test, all passed. Ready to merge.


Bound methods of the same Module have same hash but gets recompiled when jitted

jax.jit treats two functions with the same hash as the same function, allowing them to share the compilation cache. For example,
# Let's define a closure
y = 1.
def foo(x):
    print("Tracing!")
    return y * x

# Let's jit the same function, which of course will have the same hash, i.e. `hash(foo) == hash(foo)`
jitf1 = jax.jit(foo)
jitf2 = jax.jit(foo)
jitf1(0.)  # this will print "Tracing!"
jitf1(0.)  # this won't
jitf2(0.)  # this won't either

With equinox modules, bound functions share the same hash if they belong to the same Module type:
class MyModule(eqx.Module):
    y: jnp.ndarray
    def foo(self, x):
        print("Tracing!")
        return self.y * x

m1 = MyModule(y=1.)
m2 = MyModule(y=2.)
assert hash(m1) != hash(m2)   # `eqx.Module` implements its own `__hash__` so that their hashes are the same iif their PyTree leaves are the same.
assert hash(m1.foo) == hash(m2.foo)  # however methods bound to different instances have the same hash; not sure how, maybe from the `Partial` shimming?

However, when we jit the bound methods, they will recompile despite having the same hash.
eqx_jitf1 = jax.jit(m1.foo)
eqx_jitf2 = jax.jit(m2.foo)
eqx_jitf1(0.)  # this will print "Tracing!"
eqx_jitf1(0.)  # this won't print
eqx_jitf2(0.)  # this will print "Tracing!"

Is there any way to match the behavior of the two setups? This way we don't need to recompile Module.foo if only the PyTree contents get updated.

On second thought, I think this is a case where we're jitting two different closures. So it's expected that we'll trace twice and compile twice, since m1.foo is equivalent to a closure like partial(MyModule.foo, m1).
On the other hand, I'm not sure why m1.foo and m2.foo have the same hash.
Closing for now, since I don't think there's actually an issue! Thanks!

Right, your explanation is correct. I think the cache entries are keyed by object identity (id(...)) of the wrapped function, rather than by __hash__ and __eq__.
I think that's arguably a bug with jax.jit, given that the cache entry are keyed by __hash__ and __eq__ for the (static) arguments that are passed. You might like to raise an issue on the JAX issue tracker.
As a fix: you should find that eqx.filter_jit is smart enough to handle this case, and that it doesn't recompile.

If you have the time, would you mind expanding a bit on the caching differences? Which entries are cached by id and which entries are cached by __hash__ and __eq__? That way I can understand the situation better and file a better report upstream.
Also how did you manage to get filter_jit handle this case when jax.jit doesn't?

So traced arguments are keyed by .shape and .dtype, whilst static arguments are keyed by __hash__ and __eq__.
I'm actually not certain what the wrapped function is keyed as. I'm speculating that it's id(...). I do think that it would be reasonable for this to be keyed by __hash__ and __eq__, though.
As for filter_jit: it's possible to make the function be keyed by __hash__ and __eq__ by doing something like:
def call(fn, args, kwargs):
    return fn(*args, **kwargs)

call = jax.jit(call, static_argnums=0)

def keawang_jit(fn):
   def wrapped_fn(*args, **kwargs):
        return call(fn, args, kwargs)
    return wrapped_fn

And this is something that filter_jit does under the hood (alongside its other improvements to jax.jit).

Got it thank you!


How to send a response to a write()?

Hi,
noise-network is nice library that gets away with dealing with IP addresses and ports to discover and communicate with peers. I am experimenting with it to build a secure communication channel between two parties, but I am not able to figure out how the "server" can respond to client's write() request. In the example provided, encryptedStream.on() allows the server to receive the data. Is there a way to send the response back to the client, say acknowledging the request?
`server.on('connection', function (encryptedStream) {
console.log('new encrypted stream!')
// encryptedStream is a noise-peer stream instance
encryptedStream.on('data', function (data) {
// What should be done here to respond to the client?
})
})`

You should be able to do that using encryptedStream.write(data) (it's a writable stream as well) :)


Fix "Writing generators" code example + add source_url to enable link to code on HexDocs

Nice library! :D
I noticed a small error in the writing generators guide. When calling create_new_elixir_file, the igniter argument was specified twice.
Also adding source_url to mix.exs so that all functions have code links in HexDocs.

🚀 Thank you for your contribution! 🚀


Goroutines deadlock when AccountMixer and AutoTicketPurchase are running simultaneously

Procedure To Replicate

Start up the account mixer with some account balance to be mixed.
Start the auto ticketbuy goroutine.
A few minutes later the application becomes unresponsive due to goroutine deadlock.

-24 17:26:30.116 [INF] DCR: Mixing output c4b0f982b17f027c389d033a74b0c4595097774ec3597431fed25aae8d5e66ed:2 (0.52268092 DCR)
2024-07-24 17:26:30.142 [INF] DCR: Returning address (account=2 branch=1 child=178)
2024-07-24 17:26:30.142 [INF] DCR: Mixing output 4b0951e6fe9e5d8a0e841a315dd78e5b0f3027fe38137d2921cfa10631aca41f:2 (0.15993685 DCR)
2024-07-24 17:27:24.432 [ERR] UI: Error adding tx and block notification listener: listener_already_exist
^C2024-07-24 17:28:04.481 [INF] UI: ...Initiating the app shutdown protocols...
2024-07-24 17:28:04.483 [INF] UI: Displaying the shutdown wallets view page
2024-07-24 17:28:04.483 [INF] DLWL: Shutting down libwallet
2024-07-24 17:28:04.483 [INF] DCR: Cancelling sync. May take a while for sync to fully cancel.
2024-07-24 17:28:04.483 [INF] DCR: [1] Stopping cspp mixer
2024-07-24 17:28:04.488 [INF] SHWL: Set discovered accounts = true for wallet 1
2024-07-24 17:28:04.490 [ERR] DCR: AccountMixer instance errored: context canceled
2024-07-24 17:28:04.490 [ERR] TKBY: wallet.MixAccount:: wallet.MixOutput(c4b0f982b17f027c389d033a74b0c4595097774ec3597431fed25aae8d5e66ed:9): context canceled
2024-07-24 17:28:04.490 [ERR] DCR: [1] Ticket buyer instance canceled
2024-07-24 17:28:04.534 [INF] DCR: Address manager shutting down
2024-07-24 17:28:04.534 [INF] DCR: [1] Indexing transactions start height: 1438212, end height: 1438220
2024-07-24 17:28:04.544 [INF] DCR: Sync fully cancelled.
2024-07-24 17:28:04.567 [INF] DCR: [1] Transaction index finished at 1438220, 71 transaction(s) indexed in total

Issue seems to have been fixed in master. I suspect enforcing the cancelFuncs shutdown might have addressed this


Optimal filter should have optional no-weight regions at start and end

Original report by Joseph Fowler (Bitbucket: joe_fowler, ).

We could easily analyze the effect of using shorter records if we were to allow creation of optimal filters with leading and/or trailing sections of zero weight. These filters should then be used as normal and allow regular processing to proceed.

Original comment by Joseph Fowler (Bitbucket: joe_fowler, ).

Test that the short filter and the shortened filter are identical.
The code fails this test! Reopen #143.

Original comment by Joseph Fowler (Bitbucket: joe_fowler, ).

Oops. Upon testing that a long filter, when cut at both ends, comes out the same as a short filter, I found that the implementation is wrong. Test it and then fix it!

Original comment by Joseph Fowler (Bitbucket: joe_fowler, ).

Fix bug in filters with start/end zero padding.
Also a better test that checks for cutting 0 from end and/or start.
Fixes #143 again.

Original comment by Joseph Fowler (Bitbucket: joe_fowler, ).

Removing milestone: v0.7 (automated comment)


Infinite load of chat report

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.76-2
Reproducible in staging?: Needs reproduction
Reproducible in production?: Needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @blimpich
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1716836416505109
Action Performed:

Open the app
Navigate to chat with<EMAIL_ADDRESS>
Expected Result:
Report should load
Actual Result:
Infinite load and console spits out a lot of logs for forever
Note: last message with<EMAIL_ADDRESS>was a submitted expense
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
logs-2024-05-27 18_56_36.281.txt
https://github.com/Expensify/App/assets/38435837/8f4b2bff-270a-44ca-b66c-a0f17737d845
https://expensify.enterprise.slack.com/files/U0646GC9801/F075P5JBXKK/trace-20240527t115650.json
Add any screenshot/video evidence

View all open jobs on GitHub

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

Marking this as a duplicate of the older actions bug, see slack thread for details (internal only)


[Event Parameter Change] Codeunit 333 "Req. Wksh.-Make Order" - procedure Code - Event OnAfterCode

We would like the parameter-/s PrintPurchOrders added to the following event.
Codeunit 333 "Req. Wksh.-Make Order" - procedure Code - Event OnAfterCode
Original:
OnAfterCode(ReqLine, OrderLineCounter, OrderCounter);

[IntegrationEvent(false, false)]
local procedure OnAfterCode(var RequisitionLine: Record "Requisition Line"; OrderLineCounter: Integer; OrderCounter: Integer)
begin
end;

Change:
OnAfterCode(ReqLine, OrderLineCounter, OrderCounter, PrintPurchOrders); // <-- ADDED

[IntegrationEvent(false, false)]
local procedure OnAfterCode(var RequisitionLine: Record "Requisition Line"; OrderLineCounter: Integer; OrderCounter: Integer; PrintPurchOrders: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 44362.


Feat: changing vim.g.rose_pine_disable_background

Please changes this like to transparency or what. as it will make user understand the setup much more easily

Transparency of the background should be handled by your terminal emulator. I know kitty has this option, unsure about others.

We can reopen if I misunderstood your issue


fix(cli-e2e): make it great again

What is it?

[ ] Feature / enhancement
[ ] Bug
[ ] Docs / tests
[x] Other

Why is it needed?
addresses this task.
Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have ran pnpm change and documented my changes
[ ] I have add necessary docs (if needed)
[ ] Added new tests to cover the fix / functionality

Thanks @naorpeled! Is this still a WIP?

Thanks @naorpeled! Is this still a WIP?

Yes, planning on finalizing today/tomorrow. Will let you know when it's ready

Hey,
@maiieul and @shairez
can we get this merged in? 🙏

@naorpeled sorry took this long, it's also because CI was broken for some time ^^

@naorpeled sorry took this long, it's also because CI was broken for some time ^^

All gucci bud!
after this will be merged, I'll be work on the next PR that is related to the CLI :)

Sorry for the delay @naorpeled !
Thank you very much for this 🙏 ❤


buffrs install fails when the proto/vendor/directory does not exist

~% git clone git@gitlab.helsing-dev.ai:infrastructure/forward.git
~% cd forward/agent
~/forward/agent% buffrs install
Error:   × failed to install dependencies for `hs-infrastructure-forward-agent`
  ╰─▶ No such file or directory (os error 2)

~/forward/agent% mkdir -p proto/vendor/
~/forward/agent% buffrs install
:: installed<EMAIL_ADDRESS>~/forward/agent%

Putting aside #192, it looks like the intended behavior is indeed to fail to run when proto/ does not exist:
https://github.com/helsing-ai/buffrs/blob/4a6d0e9084638f40909052aa1894bc20facbc01b/src/package/store.rs#L58-L67
So the question behind this issue is now: should the proto/ directory be created automatically?

Yes, but in install we should check for existence and initialize it if it doesn't exist.


Is there any way to disable preview on completion?

Title.

Currently, No.
Feel free to PR.

Currently, No.
Feel free to PR.


updated code parts for increased compatibility

changed main(){} to int main(){return 0;}
in order to be more copy paste friendly
and compatible to more IDE's and operating systems
personal note: no need to make it more difficult to a beginner to learn C.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@ThomasPapapaschos, everything LGTM! Thank you for helping us improve this section of the guide. Looking forward to your next contribution! :+1: :+1:
Great catch here @CodingSolo. Thanks for your help QAing this!


Update dependency org.easymock:easymock to v5

This PR contains the following updates:

Package
Type
Update
Change

org.easymock:easymock (source)
test
major
4.3 -> 5.0.0

⚠ Dependency Lookup Warnings ⚠
Warnings were logged while processing this repo. Please check the logs for more information.

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
As this PR has been closed unmerged, Renovate will ignore this upgrade and you will not receive PRs for any future 5.x releases. However, if you upgrade to 5.x manually then Renovate will reenable minor and patch updates automatically.
If this PR was closed by mistake or you changed your mind, you can simply rename this PR and you will soon get a fresh replacement PR opened.


pixman: 0.42.2 -> 0.43.0

While at it:

added a trivial updater
migrated to meson build system as autotoocs-based is dropped upstream

Changes: https://lists.freedesktop.org/archives/pixman/2024-January/005008.html
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Instead of tracking which features should be enabled on which CPU that require code like:
  # All auto-enabled have to be disabled explicitly on unsupported targets:
  #   https://gitlab.freedesktop.org/pixman/pixman/-/issues/88
  ++ lib.optional (!stdenv.hostPlatform.isAarch32) "-Dneon=disabled"
  ++ lib.optional (!stdenv.hostPlatform.isAarch32) "-Darm-simd=disabled"
  ++ lib.optional (!stdenv.hostPlatform.isAarch64) "-Da64-neon=disabled"
  # TODO: does not get enabled on `stdenv.hostPlatform.isLoongArch64`
  ++ [ "-Dloongson-mmi=disabled" ]
  ++ lib.optional (!stdenv.hostPlatform.isMips32) "-Dmips-dspr2=disabled"
  ++ lib.optional (!stdenv.hostPlatform.isPower) "-Dvmx=disabled"
  ++ lib.optional (!stdenv.hostPlatform.isx86) "-Dmmx=disabled"
  ++ lib.optional (!stdenv.hostPlatform.isx86) "-Dsse2=disabled"
  ++ lib.optional (!stdenv.hostPlatform.isx86) "-Dssse3=disabled"

I enabled mesonAutoFeatures = "auto" to rely on auto-detection by toolchain. Actual feature usage at runtime is defined by CPU capabilities, not the build-time configuration.


Update Cosmos (Atom) asset names to be chain specific

Currently, both "Cosmos (Atom)" assets have the same name

This makes for a confusing UX when searching for the asset - we should include the chain name/identifier in the asset name.

closed in https://github.com/shapeshift/lib/pull/477


[Bug]: Error when we use UUID/ULID

What happened?
Hi Mokhosh!!
I am using ULID in the model, so when I try to drag the item to another status or edit it, an error occurs...
I found that the problem is that all declarations of the $recordId parameter were defined as integer
int $recordId
This way I found all the declarations and changed it to:
int|string $recordId
The changed methods were in the following files:
"src/Concerns/HasStatusChange.php"
"src/Concerns/HasEditRecordModal.php"
In this second file, we have the variable
public ?int $editModalRecordId = null;
Changed to:
public null|int|string $editModalRecordId = null;
And the method:
getEditModalFormSchema(?int $recordId)
Changed to
getEditModalFormSchema(null|int|string $recordId)
Important:
I also had to change the function calls:
onStatusChanged(int|string $recordId, ...)
getEditModalFormSchema(null|int|string $recordId, ...)
In the page generated through the "make" command.
Reminder:
It is necessary to change the test files and functions generated in the "make" command
Well, I believe that as the use of UUID or ULID is common, this correction is interesting...
How to reproduce the bug
Use UUID or ULID as record IDs. Then try to change them...
Package Version
v2.3.0
PHP Version
8.1.10
Laravel Version
10.10
Which operating systems does with happen with?
Windows
Which browsers does with happen with?
Chrome
Notes
No response

Thanks for reporting. I'll look into it.

I wrote new tests for models using ulid, and tests are passing, but this will be a breaking change, as it will change the signature of some KanbanBoard methods that are usually overrode.
I'll ship this in v3.

Great!!
In my project the changes have already been made and so far everything is ok... Of course, if I need to update or, certainly, use it again I will go to version 3...
Thank you for your attention and for the code that worked perfectly for my needs!!

As a temporary fix:

Make the board extends Filament Page class ToDoBoard extends Page implements HasForms

Copy the all data (properties and methods) from KanbanBoard to your board

Create your own HasEditRecordModal and HasStatusChange traits and make $record_id to be a string

I got it working with UUID this way.

hello, any updates regarding this feature?

As a temporary fix:

Make the board extends Filament Page class ToDoBoard extends Page implements HasForms
Copy all data (properties and methods) from KanbanBoard to your board and make $record_id to be a string
Create your own HasEditRecordModal and HasStatusChange traits and make $record_id to be a string

I got it working with UUID this way.
P.S.: Great work. Very usefull module.

thanks, this worked for me too

An alternate solution to prevent you from implementing several classes is to use composer to override the HasEditRecordModel class like this:
As part of your project composer.json:
    "autoload": {
        "files": [
            "path-to-your-override/HasEditRecordModel.php"
        ],
        "exclude-from-classmap": ["vendor/mokhosh/filament-kanban/src/Concerns/HasEditRecordModal.php"]
    },

Then in path-to-your-override/HasEditRecordModel.php:
<?php

namespace Mokhosh\FilamentKanban\Concerns;

use Filament\Forms\Components\TextInput;
use Filament\Forms\Form;

trait HasEditRecordModal
{
    public bool $disableEditModal = false;

    public ?array $editModalFormState = [];

    public ?string $editModalRecordId = null;

    protected string $editModalTitle = 'Edit Record';

    protected bool $editModalSlideOver = false;

    protected string $editModalWidth = '2xl';

    protected string $editModalSaveButtonLabel = 'Save';

    protected string $editModalCancelButtonLabel = 'Cancel';

    public function mount(): void
    {
        $this->form->fill();
    }

    public function recordClicked(string $recordId, array $data): void
    {
        $this->editModalRecordId = $recordId;

        /**
         * todo - the following line is a hacky fix
         * figure why sometimes form schema is created before this
         * method when a RichText is present in the form schema
         **/
        $this->form($this->form);
        $this->form->fill($this->getEditModalRecordData($recordId, $data));

        $this->dispatch('open-modal', id: 'kanban--edit-record-modal');
    }

    public function editModalFormSubmitted(): void
    {
        $this->editRecord($this->editModalRecordId, $this->form->getState(), $this->editModalFormState);

        $this->editModalRecordId = null;
        $this->form->fill();

        $this->dispatch('close-modal', id: 'kanban--edit-record-modal');
    }

    public function form(Form $form): Form
    {
        return $form
            ->schema($this->getEditModalFormSchema($this->editModalRecordId))
            ->statePath('editModalFormState')
            ->model($this->editModalRecordId ? static::$model::find($this->editModalRecordId) : static::$model);
    }

    protected function getEditModalRecordData(string $recordId, array $data): array
    {
        return $this->getEloquentQuery()->find($recordId)->toArray();
    }

    protected function editRecord(string $recordId, array $data, array $state): void
    {
        $this->getEloquentQuery()->find($recordId)->update($data);
    }

    protected function getEditModalFormSchema(?string $recordId): array
    {
        return [
            TextInput::make(static::$recordTitleAttribute),
        ];
    }

    protected function getEditModalTitle(): string
    {
        return $this->editModalTitle;
    }

    protected function getEditModalSlideOver(): bool
    {
        return $this->editModalSlideOver;
    }

    protected function getEditModalWidth(): string
    {
        return $this->editModalWidth;
    }

    protected function getEditModalSaveButtonLabel(): string
    {
        return $this->editModalSaveButtonLabel;
    }

    protected function getEditModalCancelButtonLabel(): string
    {
        return $this->editModalCancelButtonLabel;
    }
}

Run composer dumpautoload and go.


fix: update doc to refer to github-actions oidc provider

Closes: #3039

Summary
The documentation was incorrectly pointing to github as provider, while it needs to be github-actions.
Release Note
NONE
Documentation
This is a fix for documentation

There is no change here. Can you double-check your commits?
thanks

You’ll need to update the description of the flag where it’s defined, then run make docgen

You’ll need to update the description of the flag where it’s defined, then run make docgen

apologies, i didn't get that the documentation was generated directly from the cli commands.


Update scala3-library to 3.2.1

Updates org.scala-lang:scala3-library from 3.1.3 to 3.2.1.
GitHub Release Notes - Version Diff - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (3.1.3).
You might want to review and update them manually.
build.sbt

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang", artifactId = "scala3-library" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.scala-lang", artifactId = "scala3-library" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, old-version-remains, commit-count:1

Superseded by #240.