Exposing callable parameters for DigestAuth.challenge

NonceKeeper is package-private, so there's no way to construct one,
meaning there's no way to call challenge.
Mirror the same parameters from apply, and maintain the previous
challenge method for bincompat.
I'm happy to make changes if desired.
Thank you

Thanks for digging into this! Since you've done the work to preserve bincompat (appreciated!) would you be able to retarget this at a stable branch? series/0.23 unless you are on series/0.22.

@armanbilge It would be helpful to have this on 0.22, I'm happy to open a PR against 0.23 as well if that's how things work around here 🙂

No, series/0.22 is perfect. Lower branches are merged up so it'll find its way to series/0.23 and main :)

NonceKeeper is package-private, so there's no way to construct one

Unless there's a need to share a NonceKeeper the approach here seems reasonable 👍  Alternatively, if NonceKeeper shouldn't be shared, then this is definitely the right strategy :)

NonceKeeper is package-private, so there's no way to construct one

Unless there's a need to share a NonceKeeper the approach here seems reasonable 👍 Alternatively, if NonceKeeper shouldn't be shared, then this is definitely the right strategy :)

I don't believe there's any reason to share the NonceKeeper

Is there anything blocking this?  I'm looking to rev up a release train, and stayed out of this one because it already had two capable reviewers.

Yeah, I think this one is in a weird place. IIRC what's currently here fixes the immediate problem.
However, looking deeper the NonceKeeper should probably be reworked to not use synchronized { ... } blocks, but this will require some additional constraints.
So if we merge as-is we are basically introducing APIs that we will end up deprecating once this is fixed properly.

I haven't had time to rework NonceKeeper yet, but since it's private, could we expose the desired public API that NonceKeeper will eventually fill as a stopgap?

I'll do that early this week as to hopefully hit the release

I'm queuing up a release now (there's always something in flight), but there's plenty more, including this, to justify another.  I'm not opposed to two releases this week, which would test Arman's new website machinery.

You're right, we can't change the return type of the existing method. However, we can deprecate it, and add a new method.
The new method can still be called apply as long as the signature changes, which I believe it will have to because we will need to change the typeclass constraint.

@armanbilge Unless I'm missing something, [F[_]: Sync]() to [F[_]](implicit F: Sync[F]) has the same signature:
  def apply[F[_]: Sync, A](
      realm: String,
      store: AuthenticationStore[F, A],
      nonceCleanupInterval: Duration = 1.hour,
      nonceStaleTime: Duration = 1.hour,
      nonceBits: Int = 160,
  ): AuthMiddleware[F, A] = ???

...'s javap:
  public <F, A> scala.Function1<cats.data.Kleisli<?, org.http4s.ContextRequest<F, A>, org.http4s.Response<F>>, cats.data.Kleisli<?, org.http4s.Request<F>, org.http4s.Response<F>>> apply(java.lang.String, scala.Function1<java.lang.String, F>, scala.concurrent.duration.Duration, scala.concurrent.duration.Duration, int, cats.effect.Sync<F>);

vs
  def apply[F[_], A](
      realm: String,
      store: AuthenticationStore[F, A],
      nonceCleanupInterval: Duration = 1.hour,
      nonceStaleTime: Duration = 1.hour,
      nonceBits: Int = 160,
  ): F[AuthMiddleware[F, A]] = ???

...'s javap:
  public <F, A> F apply(java.lang.String, scala.Function1<java.lang.String, F>, scala.concurrent.duration.Duration, scala.concurrent.duration.Duration, int, cats.effect.Sync<F>);

Since the only thing that changes is in the return position, I still don't see a way to make this both binary compatible and deprecated with the same name.
Could do applyF as a stopgap?

@armanbilge Alright. server / mimaReportBinaryIssues is clear, applyF uses F[_]: Concurrent to permit future Semaphore[F] usage, I think this is good to go.

Oops. Was relying on server / Test / compile, got bit by incremental compilation fail.

OK. Rebased, clean ; test passed, scalafmtAll'd, mimaReportBinaryIssues, and we're ready to go again.

Personally I'd feel a bit funny shipping this without a matching implementation

So, given this, are we back at where we started, just exposing a new challenge method with callable parameters, pending the NonceKeeper rewrite, or are we blocked until that gets done?

We should let another maintainer weigh in.
series/0.22 is approaching EOL so maybe it's okay to just eat the caveats and go for the minimum-viable-fix. But I think we should see this fixed properly on series/0.23 up, at least.
FWIW I don't think it should be too challenging to fix. The only change really is gating the NonceKeeper with a semaphore instead of relying on a ConcurrentMap.

To clarify, I actually agree with your objection. I started trying to rework NonceKeeper, since that needs to be done regardless. If we could get a callable version of challenge in 0.22 that'd be grand, but not if y'all are uncomfortable with it.

Alright. Rolling this one back to pre applyF, as the future commits are in the other draft PR. Presumably the tests will all still pass and we can make some progress here.l Bincompat should still pass, and we don't have the confusion apply vs applyF to deal with, it's just the surgical exposure of a method that was previously uncallable.

I'm a bit confused about the relationship between this PR and https://github.com/http4s/http4s/pull/6138. https://github.com/http4s/http4s/pull/6138 is looking promising so I expect you can fix https://github.com/http4s/http4s/issues/6068 in that PR :)

The intent of this PR is to just get the challenge method merged into 0.22 before it gets closed, and the NonceKeeper changes would move on to 0.23+
This might be too much to ask, but I figured since we've got direction on the future work, we could get this one moving.

Gotcha! I think we should be able to land https://github.com/http4s/http4s/pull/6138 in 0.22 :)

Gotcha! I think we should be able to land https://github.com/http4s/http4s/pull/6138 in 0.22 :)

Got it, I misunderstood the direction of the conversation, no worries

#6138 looks like it's in the home stretch, closing this one out.


Query API when using a tag filter, returns only one value

I have in my database multiple points with the same tag.
I have this query
"SELECT \"O13\",\"tag\" FROM vc200 WHERE \"tag\"='40060'"
When using the influxdb->query api, it returns only one value:
vc200, O13=0.000000000000000000,tag=40060.000000000000000000,
When using influxdb->execute() api, it  returns all the tagged values:
{"results":[{"statement_id":0,"series":[{"name":"vc200","columns":["time","O13","tag"],"values":[["2023-11-10T08:38:52.940788Z",1,"40060"],["2023-11-10T08:38:52.9447803Z",0,"40060"]]}]}]}
Is there some option I am missing?

It works, my fault to iterate the results...


Feature - Time Block Input

Once clicked, a Time Block can receive input that will be used to indicate a scheduled event for that time block.  The user can then input text to the time block to record an event.

Feature already present.


Stop using startswith in API request if possible to avoid unnecessarily expensive query for the API

The SDK should do a direct comparison wherever possible.
See: https://balena.zulipchat.com/#narrow/stream/346007-balena-io.2FbalenaCloud/topic/SDK.20uuidOrId

Currently we allow customers to use short UUIDs in most endpoints per #326 and we need to use startswith for these ones. All other cases will try to fetch for the full UUID/ID which is the exact same behavior as the js-sdk one.


Please publish to maven central or maven.google.com

Currently, flexbox-layout releases appear to only be published to jcenter and are not present on maven central. JCenter has a long history of artifact integrity issues and we try to avoid it at all costs. I recognize that maven central's publishing process is sometimes considered more tedious, but this signing/verification process is part of why it's preferred.
https://blog.autsoft.hu/a-confusing-dependency
https://twitter.com/jakewharton/status/1073102730443526144?lang=en

This has taken on some more immediacy due to the imminent shutdown of jcenter: https://jfrog.com/blog/into-the-sunset-bintray-jcenter-gocenter-and-chartcenter/

👍  +1 to this one, please make this a priority.

I'll upload it to either google maven or maven central.

"JCenter repository will keep serving packages for 12 months until February 1st 2022" just as an update

@thagikura Any updates?

Although it's not due to be properly disabled until 2022, JCenter has had numerous problems over the last few days and is currently serving me 403s right now, with further planned brownouts planned for April, so the sooner this can happen the better.

As our project already was using jitpack, I removed jcenter by fetching flexbox from jitpack instead, implementation("com.github.google:flexbox-layout:2.0.1") instead of com.google.android:flexbox, which needs adding a maven("https://jitpack.io") to allprojects { repositories { ... } }`, as can be seen https://github.com/persian-calendar/DroidPersianCalendar/commit/7a4bddd90fa53da2c53873a86c21d45f11d723b5 it isn't ideal but maybe as a temporarily solution.

Another option is the repository "https://dl.bintray.com/google/flexbox-layout/".
maven {
        url 'https://dl.bintray.com/google/flexbox-layout/'
}

Another option is the repository "https://dl.bintray.com/google/flexbox-layout/".
maven {
        url 'https://dl.bintray.com/google/flexbox-layout/'
}

Bintray is going away just like JCenter.

I'm not sure if I understand you correctly, but how is that another option when Bintray both gets shutdown?

Is there any update please? :-) It's been two months now.

There is a really good tool https://github.com/martinbonnin/vespene to migrate multiple artifacts from jcenter to maven central.

UPDATE 4/27/2021: We listened to the community and will keep JCenter as a read-only repository indefinitely. Our customers and the community can continue to rely on JCenter as a reliable mirror for Java packages.
https://jfrog.com/blog/into-the-sunset-bintray-jcenter-gocenter-and-chartcenter/

UPDATE 4/27/2021: We listened to the community and will keep JCenter as a read-only repository indefinitely. Our customers and the community can continue to rely on JCenter as a reliable mirror for Java packages.
https://jfrog.com/blog/into-the-sunset-bintray-jcenter-gocenter-and-chartcenter/

I believe existing JCenter packages still can't be updated, so this issue will still need to be addressed before the next release. Ideally sooner - I've lost trust in JFrog and this is the only dependency I still need on JCenter.

My team is trying to remove jcenter() and this library still not available on mavenCentral() or even google()

any updates?

Sorry for the delayed response. I'm still discussing which option works better (gmaven or mavenCentral) and I plan to decide it and upload it soon after I/O.

I know putting the library in androidx is quite troublesome but this lib kind of belongs in androidx imo.

We decided to upload the artifacts to google maven for better reliability in the future.
But due to the restriction of how google maven hosts artifacts, the groupId of this library needs to be changed from com.google.android to com.google.android.flexbox.
Given that the artifacts on jCenter exists indefinitely and we can't change the groupId for the past versions, I'm going to update the version of this library as 3.0.0, which has the same code as 2.0.1 (the latest version as of now) except for the change of the groupId. And upload the artifacts to google maven for the 3.0.0 version and onward.
FYI @yaraki

I have uploaded the artifact at https://maven.google.com/web/index.html?q=flexbox#com.google.android.flexbox:flexbox:3.0.0
It should be available by the following declaration.
dependencies {
    implementation 'com.google.android.flexbox:flexbox:3.0.0'
}

We'll make future releases on the google maven.

@thagikura Since the release version 3.0.0 is still not mentioned here, i cannot really follow if is just the latest version 2.0.1 being renamed to 3.0.0, or a new release. can you please provide more information in this regard?
How about the older releases? will they be migrated to the new repo as well?

@mohand1993 thanks for pointing that out. I mentioned the 3.0.0 release.
And it's a new release, but as I commented previously the functionalities are exactly same as 2.0.1. The only change was the groupId from com.google.android to com.google.android.flexbox.
Older versions are not migrated to google maven as it conflicts with the old artifacts on JCenter because we can't change the groupId for the past versions.

It's also not that necessary as bintray said they'll keep the repo up forever

3.0.0 release depends on old kotlin-stdlib version from jcenter

3.0.0 release depends on old kotlin-stdlib version from jcenter

<dependencies>
    <dependency>
      <groupId>org.jetbrains.kotlin</groupId>
      <artifactId>kotlin-stdlib</artifactId>
      <version>1.4.32</version>
      <scope>compile</scope>
    </dependency>
  </dependencies>

This kotlin-stdlib is on Maven Central
https://mvnrepository.com/artifact/org.jetbrains.kotlin/kotlin-stdlib/1.4.32

@AlanChiou sorry, my bad, I removed jcenter from gradle properties but forgot to add mavenCentral

@lucis AFAIK, Gradle would pick a newer version if you just specify it as a direct dependency. But due to group id change you'll probably need to exclude an old one:
implementation('io.intercom.android:intercom-sdk-base:9.0.1') {
    exclude group: 'com.google.android', module: 'flexbox'
}
implementation 'com.google.android.flexbox:flexbox:3.0.0'

Yarn has this thing called resolutions which allows specifying the version which transitive dependencies may be fetched. I was wondering if something like this was also achievable in Gradle.

I haven't thought super hard about whether this is what you need, but there is resolutionStrategy.
Perhaps:
configurations.all {
    resolutionStrategy {
        force 'com.google.android.flexbox:flexbox:3.0.0'
    }
}

A pet hate is Gradle snippets with no context about where they belong, so this lives in the top level build.gradle within the buildscript definition.

Thanks @gmk57 and @robpridham-bbc. Indeed, because of the groupId change, those solutions wouldn't work.
I'm trying using a different feature from Gradle on android/app/build.gradle:
buildscript {
    configurations.all {
        resolutionStrategy.dependencySubstitution {
                    substitute(module("com.google.android:flexbox:2.0.1"))
                        .using(module("com.google.android.flexbox:flexbox:3.0.0"))
                        .because("jCenter sunset")
                }
        }
}

I still have some other jCenter errors to fix before checking if it really worked, but it's supposed to do what I want.

I just ran into this issue recently. I had been using flexbox 1.1.1, but bintray is throwing errors and seems totally broken at the moment. (My first assumption had been that the build issues might have been related to log4shell, but I found it was jcenter when I took a closer look today.)
I tried moving the app to 3.0.0, which compiled fine, but my app is now crashing. Is there any specific guide from migrating off of 1.1.x to 3.0.0? I also had to update paperdb due to jcenter issues, so the crashes might not be on the flexbox side; I haven't had a chance to capture any stack traces yet. That said, the paperdb migration seems to primarily affect the group ID, while flexbox has more significant changes with the major version bump due to mentioned risk of breaking changes, so that's where I'm looking at the moment.


Unable to load JS file after running npm run Build command in shop package.

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
master
Issue Description
After fresh installation, when we run npm i and npm run build command in the shop package, there when we check the details at the Shop front, JS files are not loading and also there are a lot  of console errors.
Preconditions

Fresh Installation to be done.
In my case I have checked in both Debugger true and false mode.

Steps to reproduce

Go to Terminal -> Go to Shop Package -> Open integarted Terminal.
Run npm i
Once npm node modules are added, now run npm run build.
See error at shop front and check the console for errors.

Expected Result
There should  be no errors and also all the files related to JS should be working.
Actual Result
Please check the video for the reference.
Video
https://github.com/bagisto/bagisto/assets/92721837/81af0a52-c3a0-46e0-9715-5dff5b3fecd3
Screenshots
No response
Additional context
No response

@shivendra-webkul
for checking the concern you need to do a complete  new installation.

@devansh-webkul sir,
I have checked the PR and the issue is fixed. Please check the video for reference.
Video
https://github.com/bagisto/bagisto/assets/92721837/c5a5689e-a63b-4dfc-9007-7fe94d799e65


Neuroimaging

Neuroimaging/resources page

@aaronjnewman so, I was able to incorporate all your requests :D let me know what you think! I added images, and to make them side-by-side I had to use this html grid component. Unfortunately, with this, there doesn't seem to be a way for me to center the images when they are side-by-side. If it being left-aligned bothers you, let me know and maybe we can add a third image to each set to even it out? Or just make it one image for the EEG section and one image for MRI section, and center the images.

The images are great picks but I think they are too small as they are displayed. I think having them alongside paragraphs would work better, alternating image and text sides as on the Projects page. Look at the tages on the Projects page for guidance on doing this alignment.
I'm also realizing this website is waaay too wordy. This section is particulary bad - I think we want much simpler explanations of the techniques, aimed at potential participants from the general public, and kids. Later we could introduce maybe some "blog posts" that provide greater detail about the techniques as is currently on the neuroimaging page.
I'm not asking you to rewrite it now (although you're welcome to :) but I'm flagging it while I think of it - and also to warn you that the number and size of paragraphs may change - so don't worry for now about aligning every paragraph with an image.

Alright, I have cut some text and formatted the pictures similarly to the projects page. Let me know what you think!

Looks great! However the pictures need to be resized/optimized as I'd mentioned for others on other pages. They take a long time to load and when I checked they're full 15 megapixel originals (5568 x 3712 pixels), whereas the size in the web page is only 816x552. In other words, the images are 33x larger than they need to be. So, please resize/ optimize compression (save as compressed jpeg, not PNG) for those images. Also make sure there are no spaces in the image filenames (calling out "DSC_3734 copy.JPG"), as that can cause problems.
I've updated the text so it's a bit simpler (I think I'm happy w it now), and added a couple of images for the data science section.

Loving the new font!
I optimized and compressed the images I included. I am assuming the ones you added are already compressed? If yes, then I think we are good to go with this page! If not, I'd be happy to compress them

Yes, the ones I added are compressed and optimized, so good to go!


Added project name display using projectile

I've added a completely optional feature allowing users to display the name of the project the current buffer is in by using projectile. Projectile is a completely optional dependency here.

I've also changed the elcord--details-and-state function so that future additions would be easier.

At a glance this seems cool! I'll review later when I get a chance to properly look at the code and try it out. Thanks for contributing!

Ditto, I like the idea of project integration. Looks good but I'll review more in the morning. Thanks!

Yep sorry about the delay. Had a busier Friday than expected. I did take another look I'm good

Great news! I was away for a few days so I couldn't reply. I thought a bit about the limitation of only being able to display either line numbers or the project name; maybe display the line numbers in short form after the buffer details line like so: Editing elcord.el [91 of 1002]. And also add an option to do so even without the project name setting turned on.
Maybe an interesting idea ¯_(ツ)_/¯.
P.S.: I have very little knowledge about contributing to open source projects, so lets say I'm interested in trying to add said feature, would I just add commits to this request, or create a new one? Any help would be appreciated!

P.S.: I have very little knowledge about contributing to open source projects, so lets say I'm interested in trying to add said feature, would I just add commits to this request, or create a new one? Any help would be appreciated!

Sorry! I thought I already replied to this! I think maybe the best answer is to provide some sort of hook for a user to be able to override the details in their own configuration. The default value for this function would produce the line number text, but adding projectile for example would be as simple as overriding this function with something else that returns the project name.
Feel free to add to the current PR if you think that's something you'd like to work on. If not, I feel comfortable merging it.


V5.1.0 upgrade

Describe your changes and provide context
This registers the v5.1.0 upgrade to be used on environments NOT having EVM module. This will add the evm module to the store keepers and perform related store upgrades
Testing performed to validate your change
Tested upgrading from v3.9.0 to v5.1.0 locally
TODO: wait to merge until code frozen and also update changelog + remove TODO message

Codecov Report
Attention: Patch coverage is 0% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 60.93%. Comparing base (cbe2957) to head (56f58ce).
Report is 5 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1582      +/-   ##
==========================================
- Coverage   60.94%   60.93%   -0.02%
==========================================
  Files         366      366
  Lines       26020    26051      +31
==========================================
+ Hits        15858    15873      +15
- Misses       9159     9173      +14
- Partials     1003     1005       +2

Files
Coverage Δ

app/upgrades.go
40.74% <ø> (ø)

app/app.go
65.12% <0.00%> (-0.42%)
:arrow_down:

... and 1 file with indirect coverage changes


Option to format time period in seconds.

By adding the configuration option fmt: seconds, it is possible to display history time as seconds (instead of decimal hours). The default remains decimal hours so existing use should not be affected,
Description:
Related issue (if applicable): fixes #
Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#11031
Example entry for configuration.yaml (if applicable):
  - platform: history_stats
    name: Pump duration last 24 hours
    state: 'on'
    entity_id: input_boolean.pump_pressure_switch
    type: time
    fmt: seconds
    end: '{{ now() }}'
    duration:
      hours: 24

Checklist:

[ x] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[ x] There is no commented out code in this PR.
[ x] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[ x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ na] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[ na] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[ na] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Hi @jeremycook61,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Duplicate of #28343


[Feature] - District level data in the API

Is your feature request related to a problem? Please describe.
Would be ideal to have one level lower (i.e. district per province) data - seeing as this is already available.
Describe the solution you'd like
Have a set of endpoint to provide cases, deaths, recoveries, tests, etc per district within a province.
Describe alternatives you've considered
None.
Additional context
N/A.

The API updates would have to be done outside the project. Will take a look.


NVS partition size in application

Raising this issue as per the suggestion from the link https://github.com/aws/aws-iot-device-sdk-embedded-C/issues/163
Is there a way to get the NVS partition size from the application (from source code of examples/get-started/hello_world/main/hello_world_main.c

https://esp-idf.readthedocs.io/en/latest/api-reference/storage/spi_flash.html#_CPPv2N15esp_partition_t4sizeE

@hvishu As @negativekelvin says, and as you have already found yourself, you can use esp_partition_find_first function to obtain a pointer to esp_partition_t structure describing NVS partition. From there, you can get the partition size in bytes.
Please use esp32.com forum for "how do I..." kind of questions and use this issue tracker for reporting bugs in ESP-IDF or for feature requests.


Using other UART with other port pins

It is probably more or less a c++ beginner question as a problem with the library itself.
In your Modbus master example you use the following line to define the used serial port, which works fine for me with the onboard USB on my aiThinker ESP32s module:
auto port = std::make_shared<PL::UartPort>(UART_NUM_0);

I have tried to change that to UART_NUM_1 and UART_NUM_2, but it did not work on the ESP32 pins 9 and 10, which are assigned to the serial UART1 as descibed there.

UART
RX IO
TX IO
CTS
RTS

UART0
GPIO3
GPIO1
N/A
N/A

UART1
GPIO9
GPIO10
GPIO6
GPIO11

UART2
GPIO16
GPIO17
GPIO8
GPIO7

If I change the library code in pl_uart_port.h like this, then I get the communication working via pin 9 and 10, but this is not the desired way to achive the result to use another serial port.
Original:
UartPort (uart_port_t port, int rxBufferSize = minBufferSize, int txBufferSize = minBufferSize,
            int txPin = UART_PIN_NO_CHANGE, int rxPin = UART_PIN_NO_CHANGE, int rtsPin = UART_PIN_NO_CHANGE, int ctsPin = UART_PIN_NO_CHANGE);

Change:
UartPort (uart_port_t port, int rxBufferSize = minBufferSize, int txBufferSize = minBufferSize,
            int txPin = 10, int rxPin = 9, int rtsPin = UART_PIN_NO_CHANGE, int ctsPin = UART_PIN_NO_CHANGE);

To change the code in main.cpp to
auto port = std::make_shared<PL::UartPort>(UART_NUM_1, 128, 128, 10, 9, -1, -1);

did not work for me.
What would be the proper way to implement the use of UART1 with the pins 9 and 10 in the app_main() code?

Hello, @MichaelUray!
I think the problem is that, when you specify the pins in the constructor arguments, you also specify TX and RX buffer sizes as 128.
According to ESP-IDF's uart.c (apart from the case when TX buffer can be zero) these sizes should be greater than SOC_UART_FIFO_LEN which is 128. So you have to make them at least 129.
You can specify it as a number or use the class constant PL::UartPort::minBufferSize.
Please tell me if it solves te issue.

Hi @plasmapper,
thank you very much, you are right.
I missed SOC_UART_FIFO_LEN + 1 in the code and to change the buffer to 129 helped.
auto port = std::make_shared<PL::UartPort>(UART_NUM_1, 129, 129, 10, 9, -1, -1);
Is there actually a way to assign just some specific parameters in the constructor like this?
auto port = std::make_shared<PL::UartPort>(port = UART_NUM_1, rxPin = 10, txPin = 9);

No, not in C++ as far as I know.


Getting AWS SSM parameters fails ValidationException

I can not get it to work to load SSM parameters. I got a SSM parameter with name test-param and a single value. I following documentation without success. What am I doing wrong? tried those
SSM_EXAMPLE: ref+awsssm://test-param[?region=eu-west-2&mode=singleparam]
SSM_EXAMPLE: ref+awsssm://test-param[?region=eu-west-2&mode=singleparam]#/test-param
SSM_EXAMPLE: ref+awsssm://test-param?region=eu-west-2&mode=singleparam

vals eval -f values-default.yaml
> expand awsssm://test-param[?region=eu-west-2&mode=singleparam]: get parameter: ValidationException: Parameter name: can't be prefixed with "ssm" (case-insensitive). If formed as a path, it can consist of sub-paths divided by slash symbol; each sub-path can be formed as a mix of letters, numbers and the following 3 symbols .-_
        status code: 400, request id: 61a77757-efc8-467c-be52-612a8721a324

Found similiar issue but that was closes 2 years ago
https://github.com/variantdev/vals/issues/11

never this issue was on my end ref+awsssm://test-param?region=eu-west-2&mode=singleparam worked in the end


Rename and describe "script.py"

The name "script.py" isn't very descriptive, and I can't find what it's used for. It should either be documented in the README or have an explanation included directly in the file as documentation.

"script.py"has been renamed to crawler.py


Handle cases where project is unsynced

💻 Environment
Platform: All (library)
📒 Description
In some cases, the project or any other item could be not synced and also not marked as "needs sync".
I saw this with an un-synced project that did not have an id (it was NULL), the project was visible in the autocomplete but selecting it would clear the project selection.

First of selecting a non-synced project should still work
Secondly, the project that is not synced should be marked as to be synced when loaded

I wonder that old implementation doesn't store any project_guid after creating, how it handles if the project remains unsync?
I don't see any complain about the unsync project in old code, it means that it always works, isn't it?
Ref: https://github.com/toggl/toggldesktop/blob/b5064ec91eafd104b1d1c7d6baab59da3fe5db42/src/ui/osx/TogglDesktop/TimeEntryEditViewController.m#L396-L421


No network connectivity WSL 2 (not DNS/firewall related?)

Windows Version
Microsoft Windows [Version 10.0.22621.2361]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 22.04
Other Software
No response
Repro Steps
Upon fresh reinstall of WSL2 and Ubuntu 22.04 (although was identical for 18.04 previously), running
ping google.com
or
sudo apt-get update
fails (whether from command prompt with elevated privileges or not) as network cannot apparently be reached (see below).
I have seen various similar issues posted online which relate to the DNS lookup and /etc/resolv.conf and the WSL config /etc/wsl.conf. I've tried modifying these (e.g. to use the nameserver at <IP_ADDRESS>) in the ways suggested, but still have no luck.
This is a home machine, and there are no enterprise firewalls or anything like that getting in the way.
Expected Behavior
Ping and Ubuntu package update as normal
Actual Behavior
For the ping, hangs on:
PING google.com (<IP_ADDRESS>) 56(84) bytes of data.
For the new package download:
Ign:2 http://archive.ubuntu.com/ubuntu jammy-updates InRelease
Ign:3 http://archive.ubuntu.com/ubuntu jammy-backports InRelease
Ign:4 http://security.ubuntu.com/ubuntu jammy-security InRelease
Ign:1 http://archive.ubuntu.com/ubuntu jammy InRelease
Ign:2 http://archive.ubuntu.com/ubuntu jammy-updates InRelease
Ign:3 http://archive.ubuntu.com/ubuntu jammy-backports InRelease
Ign:4 http://security.ubuntu.com/ubuntu jammy-security InRelease
Ign:1 http://archive.ubuntu.com/ubuntu jammy InRelease
Ign:2 http://archive.ubuntu.com/ubuntu jammy-updates InRelease
Ign:3 http://archive.ubuntu.com/ubuntu jammy-backports InRelease
Ign:4 http://security.ubuntu.com/ubuntu jammy-security InRelease
Err:1 http://archive.ubuntu.com/ubuntu jammy InRelease
  Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4002:1::103). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4002:1::101). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4000:1::19). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4000:1::16). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4002:1::102). - connect (101: Network is unreachable) Could not connect to archive.ubuntu.com:80 (<IP_ADDRESS>), connection timed out Could not connect to archive.ubuntu.com:80 (<IP_ADDRESS>), connection timed out Could not connect to archive.ubuntu.com:80 (<IP_ADDRESS>), connection timed out Could not connect to archive.ubuntu.com:80 (<IP_ADDRESS>), connection timed out Could not connect to archive.ubuntu.com:80 (<IP_ADDRESS>), connection timed out
Err:2 http://archive.ubuntu.com/ubuntu jammy-updates InRelease
  Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4002:1::103). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4002:1::101). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4000:1::19). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4000:1::16). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4002:1::102). - connect (101: Network is unreachable)
Err:3 http://archive.ubuntu.com/ubuntu jammy-backports InRelease
  Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4002:1::103). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4002:1::101). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4000:1::19). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4000:1::16). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4002:1::102). - connect (101: Network is unreachable)
Err:4 http://security.ubuntu.com/ubuntu jammy-security InRelease
  Cannot initiate the connection to security.ubuntu.com:80 (2620:2d:4002:1::102). - connect (101: Network is unreachable) Cannot initiate the connection to security.ubuntu.com:80 (2620:2d:4000:1::19). - connect (101: Network is unreachable) Cannot initiate the connection to security.ubuntu.com:80 (2620:2d:4002:1::103). - connect (101: Network is unreachable) Cannot initiate the connection to security.ubuntu.com:80 (2620:2d:4000:1::16). - connect (101: Network is unreachable) Cannot initiate the connection to security.ubuntu.com:80 (2620:2d:4002:1::101). - connect (101: Network is unreachable) Could not connect to security.ubuntu.com:80 (<IP_ADDRESS>), connection timed out Could not connect to security.ubuntu.com:80 (<IP_ADDRESS>), connection timed out Could not connect to security.ubuntu.com:80 (<IP_ADDRESS>), connection timed out Could not connect to security.ubuntu.com:80 (<IP_ADDRESS>), connection timed out Could not connect to security.ubuntu.com:80 (<IP_ADDRESS>), connection timed out
Reading package lists... Done
W: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/jammy/InRelease  Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4002:1::103). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4002:1::101). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4000:1::19). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4000:1::16). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4002:1::102). - connect (101: Network is unreachable) Could not connect to archive.ubuntu.com:80 (<IP_ADDRESS>), connection timed out Could not connect to archive.ubuntu.com:80 (<IP_ADDRESS>), connection timed out Could not connect to archive.ubuntu.com:80 (<IP_ADDRESS>), connection timed out Could not connect to archive.ubuntu.com:80 (<IP_ADDRESS>), connection timed out Could not connect to archive.ubuntu.com:80 (<IP_ADDRESS>), connection timed out
W: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/jammy-updates/InRelease  Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4002:1::103). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4002:1::101). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4000:1::19). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4000:1::16). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4002:1::102). - connect (101: Network is unreachable)
W: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/jammy-backports/InRelease  Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4002:1::103). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4002:1::101). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4000:1::19). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4000:1::16). - connect (101: Network is unreachable) Cannot initiate the connection to archive.ubuntu.com:80 (2620:2d:4002:1::102). - connect (101: Network is unreachable)
W: Failed to fetch http://security.ubuntu.com/ubuntu/dists/jammy-security/InRelease  Cannot initiate the connection to security.ubuntu.com:80 (2620:2d:4002:1::102). - connect (101: Network is unreachable) Cannot initiate the connection to security.ubuntu.com:80 (2620:2d:4000:1::19). - connect (101: Network is unreachable) Cannot initiate the connection to security.ubuntu.com:80 (2620:2d:4002:1::103). - connect (101: Network is unreachable) Cannot initiate the connection to security.ubuntu.com:80 (2620:2d:4000:1::16). - connect (101: Network is unreachable) Cannot initiate the connection to security.ubuntu.com:80 (2620:2d:4002:1::101). - connect (101: Network is unreachable) Could not connect to security.ubuntu.com:80 (<IP_ADDRESS>), connection timed out Could not connect to security.ubuntu.com:80 (<IP_ADDRESS>), connection timed out Could not connect to security.ubuntu.com:80 (<IP_ADDRESS>), connection timed out Could not connect to security.ubuntu.com:80 (<IP_ADDRESS>), connection timed out Could not connect to security.ubuntu.com:80 (<IP_ADDRESS>), connection timed out
W: Some index files failed to download. They have been ignored, or old ones used instead.```

### Diagnostic Logs

_No response_

Following https://github.com/microsoft/WSL/issues/8365#issuecomment-1726534223 (i.e. upgrading WSL to pre-release and enabling the experimental networking mode fixed the problem for me :)


build: Update Google Play Services and CameraX

Update Google Play Services dependency to 18.3.0 and CameraX to 1.3.0-rc01

Thanks for the PR, but im waiting till the stable release of camera2 instead of the RC variant.


Adding Convenience Fns

for adding components/connecting components.  These avoid having to write your own try-catch blocks for everything by doing it for you.
One could change the behavior of the add and related functions, but this approach is non-breaking and totally backward compatible as it just adds new tools to use.
Right now all these are grouped together, and then assigned to the relevant classes, but they could also just be defined inside the relevant classes (although this allows a few to share one function).
I update the example script using these new functions too, so you can see how it changes the flow of writing such files.

Also makes it an installable module

Oh, I also added some __repr__ functions so the classes print something useful when you display them

This is similar to ideas I had. What about, though, if we didn't have two methods, add() andtryadd(), but rather one method with a optional argument: def add(self, attributes, strict_add=False). If strict_add=True, then it will throw an exception if the element already exists in the DB. If it is False, it just prints a warning. In both cases, it returns self.
So essentially it would work like the current add() if strict_add=True (except that it also returns self), and it will work like your tryadd() if strict_add=False.

Yes, that's definitely a good option if you are willing to change the default behavior of the add and related functions.  I'll go with that and then see how using it for the readout chain goes!

Updated with those changes, works at least for the simple-db example code.

Implementation of the strict parameter looks good!
I presume the tryadd() helpers are no longer needed, in which case, can they be removed?
Instead of simple prints, should we use the warnings module or something similar for the warnings? Maybe eventually go with the full-fledged logging module? But for now I'd be happy to merge with prints and fix this later, unless you want to tackle it now.

Yes, the  tryadd stuff can definitely go (I forgot to commit that change, pushed now).
You're right that the warnings or logging module is probably the right way show warnings that things are already in the db and otherwise log what happens when things are run.  I'm happy enough with printing for now though, so I would vote to merge and come back to improve that later.


Waypoint Updater (Full)

Use /traffic_waypoint to change the waypoint target velocities before publishing to /final_waypoints. Your car should now stop at red traffic lights and move when they are green.

I will work on this

Please check my comments on #6


Incorrect Link

"What Is Hybrid Identity" link at the bottom instead redirects to Password Hash Synchronization page.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0c0b212d-56e7-5918-09aa-86e53dc1d83b
Version Independent ID: 7c34c9d0-fbd5-4db2-4587-2842835a09b9
Content: What is federation with Azure AD?
Content Source: articles/active-directory/hybrid/whatis-fed.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@DavidMMaxwell
Thanks for your feedback! We will investigate and update as appropriate.

Good catch, @DavidMMaxwell !
I created a pull request to fix the link. Changes should go live within 24 hours. https://github.com/MicrosoftDocs/azure-docs-pr/pull/86152
You are also welcome to make your own pull requests for these types of fixes.


Function getGroupMembeship does not respect custom production group names

In release 0.9953, the name of $jssGroup[0] is hardcoded as "----PRODUCTION----" at line 1195. Because of this, the function ignores any custom production group name set as jssGroup[0] at line 55.
A minor bug, to be sure, but worth fixing in the next update.

Update: I've worked a fix for this into my own deployment of Patchoo, waiting on Legal Dept. clearance before I push it back up here so everyone else can benefit.

Thanks heaps! I apologise for not responding or looking into this further. I've just started a new job and am setting up a completely new environment so there has been no time for me to work on any Patchoo dev.
I appreciate any PRs, go for it!


Improved: Agreement Roles - VIEW permissions (OFBIZ-12518)

Currently, a user with only 'VIEW' permissions, as demonstrated in trunk demo with userId = auditor, accessing the Agreement Roles screen, sees editable fields and/or triggers (to requests) reserved for users with 'CREATE' or 'UPDATE' permissions.
To see/test: https://localhost:8443/accounting/control/EditAgreementRoles?agreementId=8000
modified:

AgreementScreens.xml - restructured screen EditAgreementRoles to work with permissions
AgreementForms.xml - added grid AgreementRoles for users with VIEW permissions
additional cleanup

Hi Pierre,
thanks for your contribution!
Why did you change the specific field translation labels to common labels, e.g. uiLabelMap.PartyPartyId vs uiLabelMap.CommonParty? The field contains the partyId which makes uiLabelMap.PartyPartyId the correct label.
The same applies to PartyRoleTypeId.
Can you please change this back to the specific label, thanks.

Hi,
I agree with Michael, but we can do otherwise, reopening

Hi @JacquesLeRoux
With your label corrections you neglected the aspect that in several other screens (and templates) CommonRole and CommonParty as labels already have been applied in various forms in the codebase.
Are you with your corrections to this pull requests suggesting that contributors should favor archaic dogma over UX improvements?

@PierreSmits,
The reason I agreed with Michael in substance is because now OFBiz is really mature. So if you change a label id you also change the possible translation people have used on their side. I hope it clarifies my position
It was a time that we discussed about labels and their content. Paradoxally in 2009, about partyPartyId label we (French people) decided to replace all "Réf. acteur" by "Acteur". Since then we have also automatically added the names of the party. So it's OK.
I just found that we kept one for MarketingTrackingCodeVisitPartyId because there it makes sense. I could not find the discussion about that but https://issues.apache.org/jira/browse/OFBIZ-2958 is relevant.


Html and css layout home page

Description

Add html
Add CSS
Add webpack

Everything is fine. You need no fixing.
Descriptive pull request.
Linters are also working fine.
https://media.giphy.com/media/tIeCLkB8geYtW/giphy.gif
You can now merge into the development.


feat: use react-query logo as toggle button

this changes the toggle button to use the react-query logo instead of the "knobs" emoji.
not sure if you actually want to do this (if not, feel free to close). i did this mostly because on my default linux setup this emoji looked quite awful.

Fabulous!

:tada: This PR is included in version 1.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Some strange behavior with :w on WSL with an angular hmr project

I'm not sure if this is intended or not, but basically I had the basic hmr setup for angular, but in Oni when I saved a file with :w, that update would not propagate all the way into the browser. It would trigger a rebuild, but the browser's log said "nothing new" basically.
Then I switched to VSCode, and the updates would get into the browser.
Does Oni, when :w is executed, just store that change in a buffer and not write out? Or does it trigger a change before fully flushing it out so this is more of a race condition? When I reloaded the file in VSCode, the change was there, so I'm not sure.
If anyone would like a more step-by-step repro, or gist, or repo, just let me know. I had literally just one component in my angular app, so it is a pretty minimal repro; only manual step is setting up the hmr.
Thanks! ❤

What is the result of :w in just pure nvim or even vi on the console?

Good question - I should be able to get around to testing that again tonight or this weekend, will post back with results.
It could be a weird windows/wsl issue - I recently started dual-booting so I can try the same environment with real linux and see if it's different.


Resized logo to square 300 x 300 pixels.

Fixes Issue #76.

Changed the "small" image to be square and introduced a new "tiny" image with transparent background to appear correctly in dark theme browser tabs.


Change index_type to std::string_view instead of const char*

Signed-off-by: yudong.cai<EMAIL_ADDRESS>Issue: #157

/lgtm


Improve the Log level

I have checked the log outputs. Changed some output from NDN_LOG_INFO  to NDN_LOG_DEBUG.
Now we can switch between INFO and DEBUG log-level outputs.

Member

IMO, we should not perfect the application-specific details as these will be changed. Only IceFlow related details should be our priority. :)


[BUG] Docker missing browsers at /root/.cache/ms-playwright/

Context:

Playwright Version: 1.28.0-focal
Operating System: Linux
.NET version: .NET 6
Browser: All
Extra: Using docker (installed .net sdk 7.0.100 manually but not in use)

Code Snippet
Project docker file
# This is the docker file for the project itself

#See https://aka.ms/containerfastmode to understand how Visual Studio uses this Dockerfile to build your images for faster debugging.
FROM mcr.microsoft.com/playwright/dotnet:v1.28.0-focal AS base

# copying installed playwright browsers to correct exec. folder
RUN cp -r ./ms-playwright/ ./root/.cache/

# Updating to latest .Net sdk
ENV \
    # Unset ASPNETCORE_URLS from aspnet base image
    ASPNETCORE_URLS= \
    # Do not generate certificate
    DOTNET_GENERATE_ASPNET_CERTIFICATE=false \
    # Do not show first run text
    DOTNET_NOLOGO=true \
    # SDK version
    DOTNET_SDK_VERSION=7.0.100 \
    # Disable the invariant mode (set in base image)
    DOTNET_SYSTEM_GLOBALIZATION_INVARIANT=false \
    # Enable correct mode for dotnet watch (only mode supported in a container)
    DOTNET_USE_POLLING_FILE_WATCHER=true \
    # Skip extraction of XML docs - generally not useful within an image/container - helps performance
    NUGET_XMLDOC_MODE=skip \
    # PowerShell telemetry for docker image usage
    POWERSHELL_DISTRIBUTION_CHANNEL=PSDocker-DotnetSDK-Ubuntu-20.04

RUN apt-get update \
    && apt-get install -y --no-install-recommends \
        curl \
        git \
        wget \
    && rm -rf /var/lib/apt/lists/*

# Install .NET SDK
RUN curl -fSL --output dotnet.tar.gz https://dotnetcli.azureedge.net/dotnet/Sdk/$DOTNET_SDK_VERSION/dotnet-sdk-$DOTNET_SDK_VERSION-linux-x64.tar.gz \
    && dotnet_sha512='0a2e74486357a3ee16abb551ecd828836f90d8744d6e2b6b83556395c872090d9e5166f92a8d050331333d07d112c4b27e87100ba1af86cac8a37f1aee953078' \
    && echo "$dotnet_sha512  dotnet.tar.gz" | sha512sum -c - \
    && mkdir -p /usr/share/dotnet \
    && tar -oxzf dotnet.tar.gz -C /usr/share/dotnet ./shared ./packs ./sdk ./sdk-manifests ./templates ./LICENSE.txt ./ThirdPartyNotices.txt \
    && rm dotnet.tar.gz \
    && dotnet help

FROM mcr.microsoft.com/dotnet/sdk:7.0 AS build
WORKDIR /src
COPY ["project/project.csproj", "project/"]
RUN dotnet restore "project/project.csproj"
COPY . .
WORKDIR "/src/project"
RUN dotnet build "project.csproj" -c Release -o /app/build

FROM base AS final
WORKDIR /app
COPY --from=build /app/build .

RUN mkdir traces

# Updating packages and installing cron
RUN apt-get update && apt-get install cron -y \
    && rm -rf /var/lib/apt/lists/*

# Copying script file into the container
COPY "project/run-and-post.sh" .

# Giving executable permission to the script file
RUN chmod +x run-and-post.sh

# Adding crontab to the appropriate location
ADD "project/crontab" /etc/cron.d/my-cron-file

# Giving permission to crontab file
RUN chmod 0644 /etc/cron.d/my-cron-file

# Running crontab
RUN crontab /etc/cron.d/my-cron-file

# Creating entry point for cron
ENTRYPOINT ["cron", "-f"]

Crontab file
22 11 * * * sh /app/run-and-post.sh >> /app/cron.log

Run-and-post file
#!/bin/sh
xvfb-run --auto-servernum dotnet test /app/project.dll

Describe the bug
I found out there is an issue when it comes to running dotnet test project.dll when it comes to docker compose up in combination with cron.
Whenever I use just the mcr.microsoft.com/playwright/dotnet:v1.28.0-focal image it installs the browsers in ./ms-playwright/ whereas, when running xvfb-run --auto-servernum dotnet test /app/project.dll it expects the browsers to be installed at ./root/.cache/ms-playwright/.
I've solved this by copying the files to that specific folder upon customizing the base image, but I am wondering if there isn't a better way to do this, and or if this is a bug or not.
I also noticed that when I run the dotnet command within the docker terminal (dotnet test project.dll) it does seem to work without having the browsers copied to the ./root/.cache/ folder.
Does anyone have any idea about this?
Ps. Devops is not my strong suit so I would not be surprised if it is my own doing, if so I would however appreciate some feedback 😊

Do you maybe set the PLAYWRIGHT_BROWSERS_PATH env var manually somewhere in your scripts? Since we set it in our Docker image, if its not set it will fallback in your case to /root/.cache/ms-playwright.


Not working when using cookie authentication

The cookie is not carried in the request header.
I'm not sure if it has anything to do with When using cookie authentication use document.cookie
This is the Cargo.toml file of my project:
[package]
name = "axum-web"
version = "0.1.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[workspace]
members = [".", "entity", "migration"]

[dependencies]
entity = { path = "entity" }
migration = { path = "migration" }
axum = { version = "0.7.2", features = ["multipart"] }
axum-macros = "0.4.0"
dotenvy = "0.15.7"
sea-orm = { version = "0.12.10", features = [
  "debug-print",
  "sqlx-mysql",
  "runtime-tokio-native-tls",
  "macros",
] }
serde = { version = "1.0.193", features = ["derive"] }
serde_json = "1.0.108"
tokio = { version = "1.35.0", features = ["full"] }
tokio-util = { version = "0.7.10", features = ["io"] }
tower-http = { version = "0.5.0", features = ["cors"] }
tracing-subscriber = "0.3.18"
jsonwebtoken = "9.2.0"
bcrypt = "0.15.0"
chrono = "0.4.31"
tower-cookies = "0.10.0"
tower = "0.4.13"
validator = { version = "0.15", features = ["derive"] }
thiserror = "1.0.51"
anyhow = "1.0.76"
futures = "0.3.30"
utoipa = { version = "4.1.0", features = ["axum_extras"] }
utoipa-swagger-ui = { version = "5.0.0", features = ["axum"] }

// ......

#[derive(OpenApi)]
#[openapi(
    paths(
        User::create_one, User::delete_one, User::user_signin
    ),
    components(
        schemas(
            Schema::RespError,
            User::RespForUser, User::UserInfo, User::CreateUser, User::DeleteUser, User::DeleteUserOpt, User::LoginUser)
    ),
    modifiers(&SecurityAddon),
    tags(
        (name = "User", description = "User items management API"),
        (name = "Post", description = "Post items management API")
    )
)]
pub struct ApiDoc;

pub struct SecurityAddon;

impl Modify for SecurityAddon {
    fn modify(&self, openapi: &mut utoipa::openapi::OpenApi) {
        if let Some(components) = openapi.components.as_mut() {
            components.add_security_scheme(
                "app_auth_key",
                SecurityScheme::ApiKey(ApiKey::Cookie(ApiKeyValue::new("app_auth_key"))),
            )
        }
    }
}

// .......

/// Delete User by id
///
/// Delete User by id. Returns either 200 success of 404 with RespError if User is not found.
#[utoipa::path(
        delete,
        path = "/api/v1/user/{id}",
        responses(
            (status = 200, description = "User delete done successfully", body = RespError),
            (status = 401, description = "Unauthorized to delete User", body = RespError),
            (status = 404, description = "User not found", body = RespError)
        ),
        params(
            ("id" = i32, Path, description = "User database id"),
            ("thoroughly" = Option<bool>, Query, description = "Whether to completely delete all user related information, default value is false")
        ),
        security(
            ("app_auth_key" = [])
        )
    )]
#[debug_handler]
pub(crate) async fn delete_one(
    State(state): State<AppState>,
    cookies: Cookies,
    PathParser(input): PathParser<DeleteUser>,
    QueryParser(opt): QueryParser<DeleteUserOpt>,
) -> Result<Response, KnownError> {
    // ......
}

// .......

I have only just learned rust, so please feel free to correct me if there is anything wrong with the questions I asked. Thank you so much.

Perhaps this is because the cookie is currently set with the Swagger ui path, therefore it will not be sent as a part of any requests routing through differing paths. That's what I think I'm seeing as the default behaviour. Am I missing a way to configure this in Swagger ui?


Add diagonal grid

This is one of the most requested features. It is scheduled for the upcoming new version.

This is done in the DEV branch and will be shipped with the new version.


Build errors with 18.0.2

Just added the toolkit ^18.0.2 to my project with @ngrx/signals ^18.0.2 and I get the following build errors after adding withDevtools to a single signalStore.
X [ERROR] TS2307: Cannot find module '@ngrx/store/src/models' or its corresponding type declarations. [plugin angular-compiler]
node_modules/@angular-architects/ngrx-toolkit/lib/redux-connector/create-redux.d.ts:1:42:
  1 │ import { ActionCreator, ActionType } from "@ngrx/store/src/models";
    ╵                                           ~~~~~~~~~~~~~~~~~~~~~~~~

X [ERROR] TS2307: Cannot find module '@ngrx/store/src/models' or its corresponding type declarations. [plugin angular-compiler]
node_modules/@angular-architects/ngrx-toolkit/lib/redux-connector/model.d.ts:4:60:
  4 │ ...ctionCreator, ActionType, Prettify } from '@ngrx/store/src/models';
    ╵                                              ~~~~~~~~~~~~~~~~~~~~~~~~

I have added the withDevtools to a global store and it builds fine, I only get the above error for a store I'm using for a component.

OK I need @ngrx/store installed as a dependency as well, is there a document to explain the setup and also what I can achieve with the dev tools?

@caboodal, that's a bug. we will make the redux-connector as a second entrypoint. by that, you only need @ngrx/store if you use the connector.
the only documentation so far is the README. there is not so much you can do at the moment. What kind of requirements would you have?

I was keen to be able to visualize the signal store state like I did with the redux pattern, that was incredibly useful. I can't find the Ngrx Devtools extension in the Chrome store to progress any further?

@caboodal It is the same Redux extension you use for the Ngrx Devtools. You just add withDevtools and the Redux Devtools will show the NgRx Signal Store.

Ah OK I thought I'd tried that, OK no worries I'll take another look.

@caboodal did it work?

@caboodal Can you do me a favor and check what compilerOptions -> moduleResolution in your tsconfig.json is set to? I get the same error with moduleResolution: bundler. Some other options like node10 worked fine.

@caboodal did it work?

Yes it did, I did uninstall it the other day so just reinstalled it and I can now see my state

We are getting the same build error (^18.0.2 versions). We've tried with moduleResolution of "node" and "node10" - no luck.

@ngrx/store

I installed @ngrx/store as a dependency to make it work, I assume once the bug is resolved we can then remove that superfluous dependency again.

As there seems to be a few responses I've reopened the issue.

Hi Team, We are using angular latest version and ngrx-toolkit as well. In our team also facing the same issue. Attaching the screenshot for reference.

@maniya108 sorry for that. the quick workaround is to install @ngrx/store. We will soon have the fix which puts the redux connector as second entry point, meaning, you need @ngrx/store only, if you USE the connector.

Starting vom 18.0.3, you should not get any compilation errors anymore.
If you use the redux-connector, you must import it via a secondary entrypoint (see docs) and then you require @ngrx/store.
@maniya108, @caboodal @the1979: can somebody of you confirm that it is now working for you as well? Unfortunately, I was not able to reproduce it.

This seems to be relevant to this issue despite my issue not being about the redux connector:
This update to 18.0.3 allowed me to pull out @ngrx/store of the dependencies to get the toolkit working on Stackblitz. No more error like the one in this issue.

Starting vom 18.0.3, you should not get any compilation errors anymore.
If you use the redux-connector, you must import it via a secondary entrypoint (see docs) and then you require @ngrx/store.
@maniya108, @caboodal @the1979: can somebody of you confirm that it is now working for you as well? Unfortunately, I was not able to reproduce it.

@rainerhahnekamp I can confirm that 18.0.3 fixes it for us. Thank you!

Great, thanks for your quick response!

Hi @rainerhahnekamp
Everything now building successfully without the need for a dependency on @ngrx/store, many thanks!
Sorry for the slow response, just returned from holiday :-)


cockroach dump removed from v22.1 & v21.2

Fixes DOC-2795 DOC-2843

Removed cockroach dump from v21.2 and 22.1
Added redirects to go to the remaining pages v21.1
Updated the note on v21.1 for clarity

This change is

Hey @andf-crl @taroface , I have added you both for the docs review.... not sure how you want to handle the "supporting" part.


Quality checks don't run

we had a few checkstyle issues.
The right way to fix this would be to upgrade gradle and use newer quality tasks.

PR merged


The Player SSL

Hello
the website

http protocol Working

With SSL protocol
when i try to set SSL and play one content i have playback error
How solve it ?
Thank you

Conseguiu resolver?


feat(compiler-sfc): introduce defineModel macro and useModel helper

Summary
This PR introduces a new macro called defineModel that enhances the developer experience when using two-way binding props. With defineModel, v-model props can be declared and mutated like a ref.

The macro is a

<template>
  <Comp />

  <Comp v-model="count" />

  <input v-model="count"/>
  {{ count }}
</template>

Comp.vue:

<template>
  <button @click="modelValue++">+</button>
  {{modelValue}}
</template>

So you have to write a bunch of excess code to fix it:
Comp.vue:

<template>
  <button @click="() => {counter++; modelValue++;}">+</button>
  {{counter}}
</template>

@Merkushin-AY You can see https://github.com/vuejs/rfcs/discussions/503 Local Mode Chapter.

When will this no longer be experimental and opt-in? Should I avoid using it in my project until it's stable?


is it possible to refactor flow so that I can get parsing as a separate library?

yes

Ok... So to be a little more helpful:
The parser is isolated from the rest of the system here. It does not have dependencies on anything outside of that directory.
You can call the parse_program function to pass in source text and receive an AST.
Also note (just for clarity) that Flow does not typecheck while parsing -- so this parser function only generates ASTs.


qt: Show all addresses without label in a less visible color

Staking address should also be in gray color if no label is assigned.

ACK
That looks better


Server Admin page for paid subscriptions

Create, edit, & delete paid subscriptions for organizations (coupons)
New PlanDialog.js, is heavily based on TaskDialog.js.
Work towards wevote/WebApp/issues/2488

Thank you @SailingSteve, I'm looking forward to playing with this! 👍


Update BQ Comment

Made a new PR in https://github.com/datacommonsorg/website/pull/1570, since this one included submodule change.


Move starter project for init task into dbt repo

Rather than storing the starter project cloned by dbt init in a separate repository (dbt-starter-project), we should store that code directly in the main dbt repo.
Originally, we stored the starter project separately so that we could make changes to it more frequently than we could cut new releases of dbt. Instead, over the past four years, we've cut many more releases to dbt than we have merged updates to the starter project :)
This helps us avoid annoying behavior when a specific dbt version requires a specific breaking change to the starter project (e.g. https://github.com/fishtown-analytics/dbt-starter-project/pull/8, https://github.com/fishtown-analytics/dbt-starter-project/issues/10).
We could take one of two implementation approaches:

Store starter project files in the https://github.com/fishtown-analytics/dbt repository, dbt init works more or less the same by cloning from the repo @ the tag of the current dbt version
Ship starter project files within each version of dbt, rewrite dbt init to just copy files (no cloning from remote)

Questions

Should we allow adapter plugins to override / reimplement the starter project? We already enable them to provide a sample profile for first-time users, which enables dbt init --adapter [adapter] (mostly: #2814).

Describe alternatives you've considered

Continue maintaining separate repo with starter project. Develop more rigorous plan for versions + tags.

IDE does not use the init command

Check what their init command does vs what we do
Why are they doing that?

Have you considered using https://github.com/cookiecutter/cookiecutter?

@ssassi Yes, we were just talking about it over in #3474 (which was just merged, and really closed this issue). I think there's a lot cookiecutter could get us. It's one step beyond what dbt init needs to do today, but if/as we add more complexity, we may decide it's worth the switch.


You can only charge US Accounts in Dollars

Now i'm trying to payment getting this error.
{"status":"error","message":"You can only charge US Accounts in Dollars","data":{"code":"ERR","message":"You can only charge US Accounts in Dollars"}}

Please, can you paste the construction of the RavePayInitializer?

I will be closing this issue since there's been no activity for some time. Please, feel free to reopen when you have more information,

I'm getting the same error too
var initializer = RavePayInitializer(
amount: 100,
publicKey: kFlutterwavePublicKey,
encryptionKey: kFlutterwaveEncryptionKey)
..country = "NG"
..currency = "NGN"
..email =<EMAIL_ADDRESS>..fName = "Kelvin"
..lName = "Forteta"
..narration = 'Payment for data plan'
..txRef = 'iu282787823hjhjsa8727878ayrem'
..subAccounts = []
..acceptMpesaPayments = false
..acceptAccountPayments = true
..acceptCardPayments = true
..acceptAchPayments = true
..acceptGHMobileMoneyPayments = false
..acceptUgMobileMoneyPayments = false
..staging = false
..isPreAuth = false
..displayFee = true;
// Initialize and get the transaction result
RaveResult response = await RavePayManager()
    .prompt(context: context, initializer: initializer);

print(response.message);

was actually caused as a result of ACH payment.


analyze LikePredicate without depending on Reference/Field.target()

In order to remove .target() on Field the ExpressionAnalyzer may not use
that property anymore.
This changes the like analyzation to only use the types which is enough anyway
(normalizeInputForReference does a lot more is it can also handles objects and
stuff which is only relevant for insert)

ack


chore: migrate to pnpm, remove daisyUI

Description

Linked Issues
Preview / Screenshot

Photo
Video

Checklist

[ ] I have followed commit guideline to create PR title (https://conventionalcommits.org/en/v1.0.0/)

📝 Changed routes:

/404
/

(view full size)
(view full size)

And 1 other route:

/api/hello

Commit e834b99490aefa76aa4d02c52d1fce22068e139c (https://nextarter-tailwind-tremor-m759u10pt-sozonome.vercel.app).


General repository cleanup

Codecov Report

Merging #321 (2ea849a) into master (8f9f23d) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@           Coverage Diff           @@
##           master     #321   +/-   ##
=======================================
  Coverage   64.35%   64.35%
=======================================
  Files          33       33
  Lines        3316     3316
=======================================
  Hits         2134     2134
  Misses       1182     1182

I'm going to merge this - the code coverage decrease is odd - it's mainly around syntax corrections and I don't quite understand how that decreases the coverage.  But without adding a bunch of new tests, I'm not sure how to correct that.  Everything else works as expected, though.


Updated UG description without diagrams

Diagrams to be added

Codecov Report

Merging #166 into master will decrease coverage by 0.10%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #166      +/-   ##
============================================
- Coverage     52.11%   52.01%   -0.11%
+ Complexity      566      565       -1
============================================
  Files           114      114
  Lines          2502     2507       +5
  Branches        307      307
============================================
  Hits           1304     1304
- Misses         1110     1113       +3
- Partials         88       90       +2

Impacted Files
Coverage Δ
Complexity Δ

...va/seedu/address/logic/commands/CommandResult.java
71.42% <0.00%> (-16.81%)
10.00% <0.00%> (ø%)

.../seedu/address/logic/parser/FindCommandParser.java
95.94% <0.00%> (-1.36%)
27.00% <0.00%> (-1.00%)

...java/seedu/address/logic/commands/FindCommand.java
83.33% <0.00%> (ø)
7.00% <0.00%> (ø%)

...java/seedu/address/logic/commands/ListCommand.java
85.71% <0.00%> (+0.71%)
7.00% <0.00%> (ø%)

Continue to review full report at Codecov.

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


Fix Unable To Estimate  Gas Error

Ethers is currently throwing an error when attempting to estimate gas for swaps. This may be a result of the RPC throwing an error when token approval is not yet set. A possible workaround is removing the from address.

Updated to use a default gas limit if the node returns an error. This limit can be made more dynamic based on the tx type, but ideally we can resolve the initial RPC issue.

Current solution is to set a default gas limit if the RPC estimation fails.


integration: add repairnator webhooks

We can integrate other tools by calling them through a web HTTP call, ie a kind of repairnator webhook.
This is useful for integrating proprietary systems and those that are not easy to run.
Example of payload;
{
  commit_sha:"aebabe567a";
  github_repo: "organization/repo",
  travis_build_id: "7887524",
  failing_tests = ["fooTesT"]
}

I'm not quite sure about the relevance of doing that, compared with running its own instance of Repairnator, or with directly inspecting Travis CI as we do.
But assuming there is a usecase, we should really specify:

The schema of the payload
How the users could be able to register to the webhook.

For 1, I don't agree with your payload example: only the Travis Build ID and the list of failing tests are relevant. The commit_sha might be completely artificial, the github_repo might change and those information are available through the Travis Build ID.
For 2, I propose that we store the list of hooks in the DB: for now we would only have to change insert or update new elements in the DB to add new hooks, and on the future it might be even done using a webpage.

For 1, I don't agree with your payload example: only the Travis Build ID and the list of failing
tests are relevant. The commit_sha might be completely artificial, the github_repo might change
and those information are available through the Travis Build ID.
OK. Could you update the issue description accordingly?

For 2, I propose that we store the list of hooks in the DB: for now we would only have to change
insert or update new elements in the DB to add new hooks, and on the future it might be even done
using a webpage.
Excellent idea, +1.

Next, we must design a way to callback repairnator with patches/


fix: Add validation for tags to ChocolateyPackagerValidator

Fixes #1313
Context

Adds validation for the tag list not being empty
Checklist

[x] Review Contribution Guidelines.
[x] Make sure all contributed code can be distributed under the terms of the
Apache License 2.0, e.g. the code was written by
you or the original code is licensed under a license compatible to Apache License 2.0.
[x] Update documentation when applicable.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks! 😄


[Bug]: Video stream freezes 1-2sec for a brief second after start of every HKSV recording

What happened?
Dual Wired Doorbell:
Every video recording made by HKSV has a momentary freeze for about 1-2 seconds right at the beginning. The recording usually plays fine (sometime a little choppy) after the initial freeze, but I end up missing the important action that happened during those few secs. Sometimes I only get the first few seconds of the event and the recording just terminates in the middle of the event. Homekit is not liking the current stream apparently. I have tried changing various plugin settings: the snapshot options, changing the snapshot refresh interval, changing a few setting in the advanced ffmpeg encoding section (render audio and video together). Tried changing settings in the Eufy app: streaming quality, video encoding format (high doesn't work at all with plugin/homekit), notification content types. Eufy combine 2 cameras into 1 video stream that looks like a 4:3 split view size format or a more tradition 16:4 type video size with a picture in picture. I've tried both formats and it still freezes. Nothing seems to fix this issue.
I have a strong wifi network and an Asus wifi 6 router that  is about 10-20 ft away. I'm currently just copying the stream since homekit accepts the doorbell stream as-is and it's fast to load up. Maybe I need to render ffmpeg with specific settings, but I don't know where to start. I have debug logs with the ffmpeg debug on.
Device Type
Doorbell (Supported)
Plugin Version
beta (Best Effort)
HomeBridge Version
v1.4.x (Supported)
NodeJS Version
v16 (Supported)
Operating System
Ubuntu 18.04+ (Supported)
Relevant log output
no specific error that I can identify

config
{
    "username": "*****",
    "password": "*****",
    "country": "US",
    "deviceName": "Eufy Security",
    "cameras": [
        {
            "serialNumber": "T8203P1122290007",
            "rtsp": false,
            "enableCamera": true,
            "videoConfig": {
                "audio": true,
                "debug": true,
                "vcodec": "copy",
                "useSeparateProcesses": true
            },
            "talkback": true,
            "hsv": true,
            "snapshotHandlingMethod": 3,
            "refreshSnapshotIntervalMinutes": 5,
            "unbridge": true
        }
    ],
    "platform": "EufySecurity",
    "ignoreStations": [
        "T8203P1122290007"
    ],
    "ignoreDevices": [],
    "cleanCache": true,
    "preferLocalConnection": true,
    "CameraMaxLivestreamDuration": 240,
    "experimentalMode": true,
    "enableDetailedLogging": true
}

Hi @streets97f
Just to be clear: the freeze happens only in the recordings, right? If you stream normally there is no such thing?
And to be clear about what 'freeze' means: You get a still image and after 1-2 seconds the recording jumps ahead, or do you get a still image and the recording continues after the short delay?
So is something clearly missing between the image and the rest of the video, or does it just begin with a still image (say a person) and this person just starts to move after 1-2 seconds?
The doorbell dual does not need a Homebase, right? So it's directly connected to the wifi?

It appears to happen only in the recordings. This doesn't happen when opening up a live stream. When opening up the recording I immediately get a normal video for about 1 sec. Then the video freezes, I see a still image for about 1-2 sec and then the video jumps forward and continues playing. The recording viewed in the Eufy app is normal..I assume because this recording is from the internal ssd card.
The dual doorbell is directly connected to the wifi without a homebase.

Can you share a bit more about your system specs?
Would it be possible for you to try a different system?
At the moment I am running the plugin on a windows homebridge system and utilising the Cisco h264 libopenh264 with pretty good results on an intel 8100 cpu.

I'm running HB along side home assistant in a docker on a ODROID-N2+ with 4GB RAM, 128gb eMMC. Home assistant really isn't running anything (few simple automations and some lights). I have low CPU load around 4-6%. The Eufy plugin is set yo copy and not currently transcoding. when streaming video or recording CPU load may go up to 35%. How do I load/set the lipopenh265 codec, because inputting "lipopenh265" in vcodec just produces an error. I don't currently have a more powerful system to test on.

The libopen codec is only available on windows systems as far as I know. From a quick google search I also don't think that hardware encoding is a possibility on the odroid, that we could use here.
Have you already updated to the latest beta.10? There are now separate video settings introduced for HKSV. So if you used copy before this might now be reversed to the default settings.
But nevertheless I'd like you to try default settings vs. copy preset. Is there a substantial difference regarding the freeze?

Yes I'm using beta 10.
I tried using the default in the separate HKSV setting and streaming settings. I also tried combinations of copy preset and default settings between HKSV and streaming setting. The HKSV recording quality was worse. Very jittery, freezing and jumping throughout the whole recording not just the beginning. CPU load still remains fairly low 5% and maybe 20% when streaming camera feed to iphone or when HKSV is recording.
I'm also getting some sort of memory leak and my ram is filling up to max over the course of a day or so. I have to restart homebridge to release all the memory being used, but I will make a separate issue topic for this.


CAPTCHA pls

We have a spambot problem and a plan to fix it, but we haven't put that plan into place. So implement our bespoke CAPTCHA system and force all users registering to pass it.

Completed WITH STYLE :stars:


hard to read text in spelling suggestions pop-over

In the pop-over containing the spelling suggestions, the selected item is hard to read. A lighter (white) font-color is probably better here.

@bmesuere are you using a package for spelling suggestions? can't seem to find them.

It's a core package: https://atom.io/packages/spell-check

Yeah, I found it a few minutes after asking :D … I still can't seem to find that popover though. The keyboard shortcut is listed as Cmd+: yet nothing happens on my screen. I do get the underline on misspelled words.

Figured I'd chime in on this one. On OS X if you right click a misspelled word and select correct spelling you'll get the pop up (I'm using spell-check 0.59.0). I also noticed the keybinding says cmd-: but the readme says cmd-shift-: which when I press works.

The contrast for me seems okay using atom material syntax and ui themes.

To try and repo the OPs problem, I switched to Syntax Theme Atom Light and switched my accent color to teal, it's a little harder to read but I don't find it as bad.

Thanks @am17torres, I'll take a look at this ASAP.


Allow selective downloading of photos and albums

This PR first refactors the zip archive code to make it more general and uses this afterwards to allow the download of multiple albums and photos. To enable this feature for guests, Photo::getPublic needed to be extended to support multiple photos as well.
This feature is not yet usable in the search view. I will create a separate PR for this, as it requires some discussion first.
Note also that the items.concat() approach is only necessary because separators in basicContext do not support the visible property yet. This is because a menu item is currently considered a separator if it has no properties. I think it should rather be determined by a missing title property?

This PR will be closed because I'm not actively working on Lychee anymore. The repository of Lychee will stay available, but the main development continues under the Lychee Organisation. A fork maintained by the community.


Implement missing validity checks from RFC

Your environment.

Version: 54fe03a09a940c65580b1095492a98bb1b195d1e

This was originally reported in a bug report by @kixelated on the srtp repo. I'm creating this report to submit a patch against.

From the RTP spec:
A.2 RTCP Header Validity Checks

[x]  RTP version field must equal 2.
[ ]  The payload type field of the first RTCP packet in a compound
packet must be equal to SR or RR.
[ ]  The padding bit (P) should be zero for the first packet of a
compound RTCP packet because padding should only be applied, if it
is needed, to the last packet.
[ ]  The length fields of the individual RTCP packets must add up to
the overall length of the compound RTCP packet as received.  This
is a fairly strong check.


docs: add doc for nydusify commit command

Relevant Issue (if applicable)
Details
add doc for nydusify commit command
Types of changes
What types of changes does your PullRequest introduce? Put an x in all the boxes that apply:

[ ] 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)
[x] Documentation Update (if none of the other choices apply)

Checklist
Go over all the following points, and put an x in all the boxes that apply.

[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.

This pr should after https://github.com/dragonflyoss/nydus/pull/1543


Scheduling Optimization from Multiple Sources?

Hey Kevin,
Found this library and wanted to explore before trying to code something up myself. Dealing with a non-lin optimization problem where I have constraints of having to have x tonnes of a commodity at a port at y date. I have three possible options where each option has 1. price & logistics cost, 2. time to port, 3. amount of commodity available.
Given those constraints, I'd like to solve for profit maximization, given limited ordering from each location.
Is this something supplychainpy can solve?
Thx.

Hi Oliver (@obeavers) ,
Sorry for my late reply. Unfortunately, the library cannot assist you with the optimisation problem you are exploring.
Pyomo is a popular library for optimisation modelling and might be more helpful. Please also have a look at the list hosted on this Wiki. I hope this is useful.
Cheers


builds sqlite3 with version 3270100

With this version you can create an in memory db and use VACUUM INTO 'somefile.db'; to export the database to the fs without using the backup api. Also prior versions do not support this use case.

Thanks, doing the update to 3.28 along with the new release for node 12.


 Duration Estimation Method Swap

Pull Request Key Information
Reason for change
Duration estimation was initially hard coded.
What was changed
Users are now able to set, per program, a duration estimation method to calculate emissions.
Intended Purpose
Allow users to choose a duration estimation method and create a framework for future methods to be incorporated easily.
Testing Completed
Manual testing - ran with the test simulation where P_Air was using the newly defined component-based estimation, while P_OGI was using the measurement-based-conservative method.
SampleOutputFiles.zip
All unit tests passed.
result.txt

results.txt


bug: able to add isolated event with invalid date for feb due to leap year

solution: check if it leap year and validate the date

Could you also change the example help command for isolated event to a date very far into the future.


YARN-2118

https://issues.apache.org/jira/browse/YARN-2118

Can one of the admins verify this patch?


GraphQL queries run in previews don't seem to be honouring GraphQL aliases or schema customization

I've been having quite a bit of trouble getting previews to work smoothly, and I notice they keep breaking because certain data fields, that are queried perfectly fine when the component is loaded normally (ie. not through preview), seem to become undefined and blow-up the preview.
For eg.: I see this in preview mode, but it renders fine normally:

For the error about buttons being undefined, I've managed to triangulate it to the fact that data that is being passed into the component during preview does not seem to name a field according to the GraphQL alias, but retains the original field name. That is, there is a field called items that I alias to buttons in my query, but this does not seem to work in preview:
Object logged out in preview:

Relevant portion of my GraphQL query:

On a different page, where I'm using the createResolvers API to add a new contributors field on one of my types:

The contributors field doesn't seem to be populated correctly in preview mode, but I can query it fine normally and in the GraphQL playground.
Broken in preview:

But totally fine when querying in the playground:

I suspect this is a bug which how this plugin is handling the merging of preview data with component data, but I don't know enough about the internals of what the preview HOC components are doing to know for certain.
Don't mind spending some time investigating and maybe opening a PR to fix, but wondering if the maintainers (@angeloashmore ?) with more context could help to point me in the right direction? And whether there have been similar issues reported before (I tried to dig through historical issues, but couldn't find anything exactly like this about broken previews).

I'm trying to find the issue where this was mentioned but previews don't support aliases.
I believe this is because this plugin uses the RestAPI and uses that for previews so has no knowledge of the aliases you have made.
This may be fixed in v4 potentially.

Thanks @jodiedoubleday for the response. I guess aliases aren't too big an issue for me since they can be circumvented easily with a separate mapping function to transform the query data.
But the null data with the customized schema worries me, there isn't an elegant workaround for that other than bringing the join operation into runtime code, which I'd rather avoid.

Thanks @jodiedoubleday, that's correct. Field aliasing won't work during previews since the GraphQL server Gatsby uses during development/build time is no longer there.
This document explains some of the preview limitations. It is written for the upcoming V4 release, but still applies to V3.
https://github.com/angeloashmore/gatsby-source-prismic/blob/beta/packages/gatsby-plugin-prismic-previews/docs/limitations.md
Your schema customizations fail for the same reason. Modifying the GraphQL schema in Gatsby only affects the development and build time API. During previews, we are only working in the browser without Gatsby.
However, what you are setting up could probably be accomplished without creating a schema customization. If your document has a contributors field that links to Contributor documents, the GraphQL API already returns those documents. This assumes your contributors field is a Group field containing a contributor Content Relation field.
See this part of the documentation: https://github.com/angeloashmore/gatsby-source-prismic#query-content-relation-fields
Your query could look something like the following:
query MyQuery {
  allPrismicArticle {
    nodes {
      id
      data {
        contributors {
          contributor {
            document {
              ... on PrismicContributor {
                id
              }
            }
          }
        }
      }
    }
  }
}

Thanks @angeloashmore for the detailed response and suggested workarounds! Still getting to grips with GraphQL so didn't realize the foreign key join could be approached that way. Will give it a shot and update and close this issue if all goes well, for any others who struggle with the same problems.

Alright, I've got previews to work by removing GraphQL aliases through:

Transforming the data via a separate data
Splitting a query for which the above wouldn't work (I needed to query the same field with two different parameters) out into separate useStaticQuery calls in two separte hooks files which I then imported and used in my component

And following the recommended approach of querying documents linked by a content relationship field worked like a charm.
I wonder if the limitations doc should be amended slightly to explicitly mention that schema customization is not supported as well? (Although that conclusion follows naturally from the fact that things are happening at runtime not build time, that's not necessarily obvious to everyone)

Great! Yes, adding this to the limitations document is a good idea. I'll open an issue to make sure it's updated.


Make protocols for which we have backing implementation public.

For protocols like IPropertyValue we have the underlying implementation, but these were private.
Getting value out of a runtime class (returned from an API) which implements this interface was difficult as the interface itself was private.
This change makes all the protocols for which we have backing implementation public.

:shipit:

:shipit:

Update projection binaries: Make protocols with backing implementation publicly available.


Fixes #26635: clarify "memory_cost" of Argon2PasswordHasher differs from command line utility

@timgraham
https://code.djangoproject.com/ticket/26635

merged in 9407cc966b02e5ef69b7f561a6b972aff5d9c2e0, thanks!


About pretraining and finetune data

Data
Will you release the pre-training data and finetune data or the data processing code mentioned in the paper?

pretraining data

finetune data

Pretraining code
Will you release the pretraining code and scripts?

Thanks

Sorry, we will not release the pretrained data because it is huge and very expensive to serve (also some commercial reasons), but they are public so that you can download it on your own.
We are planning to open-source the corrected LLAVA for SFT if the CogVLM paper is accepted by ICLR.


Optional correspondence paragraph

In case no corresponding author is required (e.g., for student papers), this change enables output (docx, html, and pdf) without the correspondence paragraph.
Usage
For .html and .docx, simply remove the corresponding: true (or set it to false) in the author options. For .pdf output, additionally set the new  nocorrespondence  option to true within the apaquarto-pdf format options.
A new if statement has been added to title.tex, which evaluates nocorrespondence. It defaults to displaying the correspondence paragraph when nocorrespondence is omitted from the YAML.
Minor Limitation
If no corresponding author has been set, but nocorrespondence: false (or omitted), then the PDF output will contain the start of the paragraph ("Correspondence concerning this article...") but no author information. An error message could be added for those cases, but I don't think it's strictly necessary.

PS: I think the title.log file can be deleted from the _extension folder, so I included this change as well.


perf: rescale screenshots on background thread

Not the solution for #377, but still useful

only downside: if there's a bad plugin simultaneously submitting long-running jobs to this executor, unhiding chat widget could take longer (but we can refactor this approach to schedule unhide on renderer thread instead of executor thread)

i've also been running this PR & haven't had any chat-hiding issues


hasteImpl

Summary
Introducing an optional HasteImpl of type {getHasteName(filePath: string): (string|void)} that returns the haste name for a module at filePath if it is a haste module or undefined otherwise.
I'm going to add a similar injectable config to React Native Packager.
This allows us to inject a custom implementation of haste's module id resolution.
Test plan
npm test

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@3414a99). Click here to learn what that means.
The diff coverage is 93.75%.

@@            Coverage Diff            @@
##             master    #2877   +/-   ##
=========================================
  Coverage          ?   67.19%
=========================================
  Files             ?      142
  Lines             ?     5139
  Branches          ?        0
=========================================
  Hits              ?     3453
  Misses            ?     1686
  Partials          ?        0

Impacted Files
Coverage Δ

packages/jest-runtime/src/index.js
85.02% <ø> (ø)

packages/jest-haste-map/src/index.js
95.37% <100%> (ø)

packages/jest-config/src/normalize.js
85.9% <100%> (ø)

packages/jest-haste-map/src/worker.js
94.59% <90.9%> (ø)

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 3414a99...0c6e8c0. Read the comment docs.


Improve num_qbits_needed

Change from eq. 15 to eq. 22 in the source paper.  This will add support to more than 2 state nodes.

Eq 22 does not appear to give the correct number for the lesser whistling model (for instance) -- equation might be wrong??

Should the equation have -2 within the max function?  That is consistent with the existing code and my understanding


Adding code for network firewall

Issue #, if available:
Description of changes: Adding code for the network firewall operator
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I have made the requested changes


v1.4 backports 2019-03-29

#7476 -- Fix issue where traffic matching L4-only rule would be redirected to the proxy then dropped (@joestringer)

Had to rebase to remove the changes in commit 55b0fd3d22a9. Minor fixup.

Once this PR is merged, you can update the PR labels via:
$ for pr in 7476; do contrib/backporting/set-labels.py $pr done 1.4; done

This change is

test-me-please

test-missed-k8s

test-docs-please

Hit #7546 . Will re-try.

test-me-please

test-me-please

test-docs-please

test-missed-k8s

Cilium health test failure

trying again.

test-missed-k8s

Datapath config tests failed due to identities not being allocated (most likely due to etcd-operator issues): https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-K8s/1882/

test-missed-k8s

All tests passed apart from unit tests, which I had to fix locally. I pushed that as the final commit. @joestringer can you PTAL at that commit?

Merging since unit tests passed locally + all other tests passed on Jenkins


WIP: Refactors Kube RBAC

Fixes: #291
This code failed e2e testing with:
2022-08-30T18:29:03.209Z	ERROR	runner/runner.go:53	failed to create new infra	{"runner": "infrastructure", "error": "failed to get serviceaccount envoy-gateway-system/eg: serviceaccounts \"eg\" is forbidden: User \"system:serviceaccount:envoy-gateway-system:envoy-gateway\" cannot get resource \"serviceaccounts\" in API group \"\" in the namespace \"envoy-gateway-system\""}

Signed-off-by: danehans<EMAIL_ADDRESS>

Codecov Report

Merging #292 (3f079e3) into main (c0a21c0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #292   +/-   ##
=======================================
  Coverage   58.19%   58.19%
=======================================
  Files          31       31
  Lines        2837     2837
=======================================
  Hits         1651     1651
  Misses       1087     1087
  Partials       99       99

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@skriss I create a separate Infra Manager Role, attach the Role to the Envoy Gateway ServiceAccount using a RoleBinding, but I still get the same error:
$ kubectl get role/infra-manager -n envoy-gateway-system -o yaml
apiVersion: rbac.authorization.k8s.io/v1
kind: Role
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"rbac.authorization.k8s.io/v1","kind":"Role","metadata":{"annotations":{},"name":"infra-manager","namespace":"envoy-gateway-system"},"rules":[{"apiGroups":[""],"resources":["serviceaccounts","services"],"verbs":["create","get","list","watch"]},{"apiGroups":["apps"],"resources":["deployments"],"verbs":["create","get","list","watch"]}]}
  creationTimestamp: "2022-09-01T18:13:42Z"
  name: infra-manager
  namespace: envoy-gateway-system
  resourceVersion: "76599"
  uid: 4d3197e5-0c3b-4038-811f-0072a03ebc8c
rules:
- apiGroups:
  - ""
  resources:
  - serviceaccounts
  - services
  verbs:
  - create
  - get
  - list
  - watch
- apiGroups:
  - apps
  resources:
  - deployments
  verbs:
  - create
  - get
  - list
  - watch

$ kubectl get rolebinding/infra-manager -n envoy-gateway-system -o yaml
apiVersion: rbac.authorization.k8s.io/v1
kind: RoleBinding
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"rbac.authorization.k8s.io/v1","kind":"RoleBinding","metadata":{"annotations":{},"name":"infra-manager","namespace":"envoy-gateway-system"},"roleRef":{"apiGroup":"rbac.authorization.k8s.io","kind":"Role","name":"infra-manager"},"subjects":[{"kind":"ServiceAccount","name":"envoy-gateway","namespace":"system"}]}
  creationTimestamp: "2022-09-01T18:13:42Z"
  name: infra-manager
  namespace: envoy-gateway-system
  resourceVersion: "76600"
  uid: 91431b52-3603-498a-bb06-97ce34a568d9
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: Role
  name: infra-manager
subjects:
- kind: ServiceAccount
  name: envoy-gateway
  namespace: system

$ kubectl get deploy/envoy-gateway -n envoy-gateway-system -o yaml | grep serviceAccountName:
      serviceAccountName: envoy-gateway

$ kubectl logs deploy/envoy-gateway -n envoy-gateway-system -f
...
2022-09-01T18:17:52.706Z	ERROR	runner/runner.go:53	failed to create new infra	{"runner": "infrastructure", "error": "failed to get serviceaccount envoy-gateway-system/eg: serviceaccounts \"eg\" is forbidden: User \"system:serviceaccount:envoy-gateway-system:envoy-gateway\" cannot get resource \"serviceaccounts\" in API group \"\" in the namespace \"envoy-gateway-system\""}

Looks like the namespace in the subjects is not correct (should be envoy-gateway-system, not just system:
subjects:
- kind: ServiceAccount
  name: envoy-gateway
  namespace: system

@skriss I replicated the Kube provider kustomization setup for the Infra Manager and it still failed to append the envoy-gateway-system namespace to the RoleBinding subjects[].namespace. I have removed the namespace appending from the infra kustomization setup.