Documentation/proxy: Explain more about Procfile sample.

Say what goreman is so it doesn't confuse. (line 17)

@gyuho A little clarification piggy-backed on your #3871

Great. LGTM. Thanks @joshix

LGTM


Fix signal onboarding

Fixes #1908
What changed in this PR and why
We decided to update our Signal onboarding to create a signal_onboarding_token similar to what we do with Telegram. We create the contributor in our database and instruct them to send us a message containing only the token, by which we can identify them. Once we identify them by their token, we can save their uuid and complete their onboarding.
We then send them messages with their uuid and identify them with incoming message by it as well.

@roschaefer this is failing the build, I need to revert some changes to tests that I thought needed to be changed and I also need to write tests for the new onboarding/sending/receiving flow.
You could test it out though as I think it works as I imagine it should.


Add compatibility / upgrade page

See

What do you think about adding a compatibility section like the one at the bottom of this page in DPF Composites? Or at least a section which informs the customers that 2025 R1 is needed to run all features. I couldn't find any reference to minimum version of the unified installer.

Originally posted by @roosre in https://github.com/ansys/pyacp/pull/709#pullrequestreview-2469866612

Yes, I think we should do that. The individual classes have a note

Added in ACP server version XYZ.

in their API reference, but it would make sense to collect that information onto one page.
I was also considering adding some description of how to upgrade from the beta version to the full release. While it was explicitly stated in the beta note that the API may change, I think we should still give a brief overview of changes and how to migrate the code.

Originally posted by @greschd in https://github.com/ansys/pyacp/issues/709#issuecomment-2507835883

I would add a section Compatibility under user guide.
I am not sure if we have to list all the details there but at least guide the user to where the information is available (API documentation).
There are quite many API changes: removal of ACPWorkflow, renamings, clean up of the module structure, etc. Depending on the changes, we can provide all details or list the most important changes only.


Spaces in password is not supported

So I've tried to connect to SSID that has some special characters in the password. One of those characters is space. Didn't go very well.
If I try to enter the config through web config, it replaces spaces with plus + signs.
If I use serial line config, password is cut off from the first space.

If you want to enter non-ASCII or special characters in the web interface
you have to use HTTP-style hex encoding like "My%20AccessPoint". This will
result in a string "My AccessPoint". With this hex encoding you can enter
any byte value you like, except for 0 (for C-internal reasons).
Am 11. Juli 2018 23:38:36 schrieb Vilius Sutkus<EMAIL_ADDRESS>
So I've tried to connect to SSID that has some special characters in the
password. One of those characters is space. Didn't go very well.
If I try to enter the config through web config, it replaces spaces with
plus + signs.
If I use serial line config, password is cut off from the first space.—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks Martin.


Task - User C appears on User B's LHN when task assigned to User B is changed to User C by User A

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

Action Performed:

Go to staging.new.expensify.com
As User A, create a task and assign it to User B
As User A, go to the task report and change the assignee from User B to User C
As User B, go to LHN

Expected Result:
User C will not appear in User B's LHN
Actual Result:
User C appears in User B's LHN and it disappears with not here page when User B opens User C's report
Workaround:
Unknown
Platforms:

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

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: 1.3.74-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/a064a018-d660-4d0b-86e7-398caa98b2d5
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

@thienlnam can you clarify the behaviour here to understand how this should work?

A task was assigned to userB, and therefore the taskReport created is a thread on the userA <> userB DM
The taskReport gets re-assigned to userC, and therefore the taskReport thread is moved to the userA <> userC DM which is now it's parent chat
userB loses access to the taskReport after being unassigned.

Is that how it works?

Ah close but the taskReport is independent of these threads
A task was assigned to userB

Task report is created
A reportAction linking userB to the task is added on the user A <> userB DM

The task gets re-assigned to userC

The task report is shared with userC, but should also remain shared with UserB and UserA
A reportAction linking userC to the task is added on the user A <> userC DM

UserB should not lose access to the taskReport
I think the issue here is that UserB should not be losing access to that task report

Oh, so taskReports don't have a parentChat like expense/iou/threadReports do?
What reason does userB have to still be shared the taskReport after being unassigned, and has this ever worked, so it's deemed a bug?

No they still have a parent like those reports - I might have been confused on the

the taskReport created is a thread on the userA <> userB DM

What reason does userB have to still be shared the taskReport after being unassigned

A task is basically a thread with a completion state, so if they participate in the report at all they would still have access to it.

and has this ever worked, so it's deemed a bug?

Yeah this should be working already so it would be a bug

No they still have a parent like those reports - I might have been confused on the

Cool, that's an accurate statement though, right? The taskReport has a parentChat and that's the DM between userA <> userB, like an expense/iou/chat report are considered threads of the parent.

A task is created by userA and assigned to userB
A taskReport is created, with the userA <> userB DM as the parentChat
userA re-assigns the task to userC and that shares the taskReport with userC
The taskReport remains as a thread of the userA <> userB DM, shared with userB, and despite now being assigned to userC.

A task is basically a thread with a completion state, so if they participate in the report at all they would still have access to it.

Are you saying then if a task is assigned to userB, but they don't participate in the taskReport (i.e comment, or edit it) then the wouldn't have access after being unassigned?

Yes that's correct - the taskReport has a parent (chat or task) and it is where-ever the task was created

The taskReport remains as a thread of the userA <> userB DM, shared with userB, and despite now being assigned to userC.

Yup, but in this situation there is another parent of the user A <> userC DM that links to the taskReport, but the true 'parent' of the task still remains in the reportAction from the userA <> userB DM. Let me know if that part makes sense as it is kind of confusing. There are many links to the taskReport depending on how many times you re-assign it, but the parent will always be the original reportAction that created the task

Are you saying then if a task is assigned to userB, but they don't participate in the taskReport (i.e comment, or edit it) then the wouldn't have access after being unassigned?

Ah no, they would still have access from being the assignee. We don't ever unshare the task report with someone once they have access to it

Cool, I think we need to consider updating the parentChat when the task is reassigned. Here's a good example of why when it comes to the logic being updated wrt to the GBR: https://expensify.slack.com/archives/C02MW39LT9N/p1695898060546159?thread_ts=1695897345.190449&cid=C02MW39LT9N

Took a brief look, but yeah this seems like it will take a larger conversation as updating the parentChat everytime the task is reassigned might have some other unintended consequences. I'm mostly OOO this week, but can start a convo next week about this

Cool, so let's move this issue on for proposals to resolve this one in the current imp:

The DM between userA <> userC shouldn't be popping up in the LHN of userB like in the OP. That's a bug in itself to resolve in the current implementation

Then yeah, let's talk about this one separately somewhere. How are we going to align taskReports with this notion of the higher context chat getting the GBR when that chat could very well not be one the assignee has access to.

The parentChat not being the DM shared between the task creator and current assignee isn't a solution that will scale with the wider notion of the GBR being added to the "higher context" chat, so we should rethink how to solve that.

Perhaps! @thienlnam might be able to confirm.

Yeah I think that might be the case - seems like we're sending the new chat report to all participants but should not be sending it to people that are not included in the chat

I can get a PR up for it this week

Sounds good!

Oh, should the reviewing label be on this?

Yup, PR has been merged now just pending deploy to see if it fixes this issue

Dope!

Should be resolved now, can we see if it's still a problem and then close if not?

@tjferriss Issue is not reproducible on latest build 1.3.90-1
https://github.com/Expensify/App/assets/78819774/2f4a5203-6fcb-4dae-ae3d-465f1ad8b4f8

Excellent - thanks for that! As this was reported by Applause and a backend PR, closing it out!


🚀 Feature: Autologout

🔖 Feature description
Description
The Autologout feature enables platform engineers to add a mechanism to log out users after a configurable amount of time of inactivity, and redirect them to the login page.
When enabled, the mechanism will track user actions (mouse movement, mouse click, key pressing, taps, etc.) in order to determine if they are active or not. After a certain amount of inactivity/idle time, the user session is invalidated and they are required to sign in again.
Glossary
An inactive user is either:

a user that is not performing any action on the app (no click, no mouse movement, no keyboard press)
a user whose session is still valid but has not open tabs in the browser pointing to Backstage

Idle timeout: it is the amount of time after which the user is automatically logged out
prompt message: it is a small prompt displayed to users some seconds before being automatically logged out. it is to inform them what it is going to happen if they don't perform any action.
User stories

As a platform engineer, I can enable/disable the autologout mechanism through the app-config
As a platform engineer, I can configure the idle timeout in the app-config
As a platform engineer, I can enable/disable the prompt message before idle
As a platform engineer, I can configure the amount of time before idle when the prompt message will be shown
As a platform engineer, I can configure the list of DOM events to track to determine if the user is active
As a user, I am automatically logged out after X minutes of inactivity
As a user, a prompt is shown by the app, X seconds before idle timeout, to ask me if I'm still active
As a user, I am automatically logged out after X minutes of inactivity even if I have no Backstage app tabs open
As a user, I am automatically logged out after X minutes of inactivity from every Backstage app tabs open in my current browser

🎤 Context
In my use-case, we have an internal constraint so that applications should log users out automatically after 60 minutes of inactivity.
✌ Possible Implementation
The implementation could leverage the React-Idle-Timer library which I already tested on different browsers and with different tabs of the same app open.
There will be a provider called <AutologoutProvider> that wraps the timers management provided by the react-idle-timer library. When onIdle event is triggered, the autologout provider will make a call to identityApi to sign out the user.
The <AutologoutProvider> will be installed in the App.tsx entrypoint, and will wrap the <Root> component of Backstage in this way:
// ... App.tsx contents
<AutoLogoutProvider>
    <Root>{routes}</Root>
</AutoLogoutProvider>

The behaviour of AutologoutProvider can be configured through some properties on the app-config. An example configuration could be this:
autologout:
    enabled: true
    idleTimeoutMinutes: 60
    promptBeforeIdleSeconds: 10

My only doubt at this point is, which plugin can export the AutologoutProvider?
Some options came into my mind:

A new auth plugin for frontend components related to Auth stuff
A whole new frontend plugin which just exports this AutologoutProvider

I am open for suggestions  and to hear your feedbacks on this feature request :)
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

Yep I think it could makes sense to add this as an optional extra. Probably coupled with a dialog that pops up that warns the user that they're about to be logged out? That's typically how I see this implemented anyway. Depending on the complexity of the implementation I think it could be placed in a few different places, maybe @backstage/core-components if it's small, but a new @backstage/plugin-auth-react if it's a bit larger. Could figure that out in a potential PR

Yep! the user will be informed with a dialog (I called it prompt) on what is going to happen, this is requirement R7.
I'll create a PR 🕺


Unable to use metadata filter values containing spaces

I'm using the Qdrant document store with Haystack, and would like to apply a metadata filter inside a query pipeline. This works fine when meatadata values don't contain any white spaces, however breaks when they do. With spaces, parse_json_path() raises a ValueError: Invalid path, see code here.
In my case, metadata values with spaces occurs because I have predicted category labels for documents that sometimes are more than a single word.
It appears parse_json_path() only accepts json paths with dot notation, and not also bracket notation. I believe if it were made more flexible to also accept json paths using bracket notation, providing a path with keys containing spaces would not raise an error.
This change would be very useful and make the Qdrant more versatile in handling more varied metadata values as filters.

Hi @ms130, indeed, this functionality is not currently supported in qdrant.
Btw, it seems like you're using qdrant in local mode.
Local mode mimics the behaviour of the server (e.g. qdrant in docker).
We'll add it to the local mode as soon as we add it to the server.
@generall @xzfc

Quoted field names are supported, e.g. to match {"a b": {"c d": "x"}}, use this:
models.FieldCondition(key='"a b"."c d"', match=models.MatchValue(value="x"))

Great, thanks it works if I use that formatting. Not sure if it's already mentioned in the docs, but would be helpful to mention this as the way to use field names containing spaces.


added operationId to metadata of request specs

Added a change to allow users to add operationId to the metadata of request specs. There have been requests to make the  OpenAPI spec require operationId for many reasons. Here is an example: https://github.com/OAI/OpenAPI-Specification/issues/1907.
If a user adds operation-id to the metadata of a request spec, it will be added to the path section of the schema.
First time contributor and I struggled to figure out where to add a test for this feature. Let me know if there is a place where you would like to see a spec for this.

@exoego - What are your thoughts on this feature to enable operationId to be passed in as metadata options on a spec?

Supporting operation-id itself sounds legit to me.
It is unclear to me that path_server_records or other props are added.

Supporting operation-id itself sounds legit to me. It is unclear to me that path_server_records or other props are added.

@exoego - Sorry for the confusion here, I was testing out another proposed feature to the gem and committed my change to this PR.  Then I'll submit another PR for the server record changes.

@exoego - Ok, this PR for adding operation-id is good to review again.

@exoego - We added :operation_id to  a test and updated the expected.yaml to have the operationId as well. Let me know if there are any other changes you would like us to make.

@exoego - If we can get this :operation_id PR merged in, I will open up another PR for another proposed new feature.

Sorry for the late response.
Thanks.


Vertical alignment of footnotes looks wrong

When footnotes are displayed on a page, the vertical alignment of symbol and caption looks to be "off".

Cannot be reproduced


pull banner from the current installation, if available

Part one of:
https://github.com/uProxy/uproxy/issues/2038
Next will be to do a similar thing for authorized_keys files.

:+1:


Add license header check to CI

Describe the solution you'd like
Add a step to the GitHub actions that verifies new PRs do not introduce new files without license headers. Also, verify that the current license headers are correct.

Replaced by #417


Add the main page, load discord and github logos.

Main Page is created
I've created main page for the new users. On the page, the login page is already accessible.
Finish:
- The responsive
- Add the redirection to EDP Unblock
- Add the 2FA security

Update the theme and colors to make them match with the /login page (same background color and same color of "window")
You replaced the EDP Logo with a "retour" button. I think that it would be understandable if the EDP Logo redirected to /main-page and I think the button takes too much space and is too visible for the login page. If you really think that click on the logo is not a good idea to redirect to the main-page, consider changing at least the style of the button (for instance the goBackArrow in the feedback page).
Try (I don't know if you can) to change the branch on which you're about to merge with the pull request, instead (if you can) make the pull request on a new branch.

Otherwise thank you for your good proposition 👍

I changed what was requested, I changed the colors and the logo back. It is now the EDP logo. With my level in git, i join forked repo [ https://github.com/Magic-Fishes/Ecole-Directe-Plus/compare/main...Vooxify:Ecole-Directe-Plus:save ]. Tell me what I should do

So, with the recent commits, I couldn't integrate the extension page. I'll let you do it.


Consider control-plane flag on kubeconfig command

What this PR does / why we need it:
Consider control-plane flag on kubeconfig command
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Fixed an issue where the `--control-plane` flag was not considered on the `kubeconfig` command

cc @mandelsoft


[cinder-csi-plugin] Helm: Add extraArgs support for containers

What this PR does / why we need it:
Add an extraArgs value to the various containers that are deployed by cinder-csi. Implementation is similar to occm helm chart
Which issue this PR fixes(if applicable):
fixes #2159
Special notes for reviewers:

Release note:

NONE

/ok-to-test

/lgtm

@jichenjc can this be backported to release-1.23 and have a 2.1.1 chart release? Similar for release-1.24 and 2.2.1?

can this be backported to release-1.23 and have a 2.1.1 chart release? Similar for release-1.24 and 2.2.1?

usually we do backport to N-3 maximum (convention?) like 1.26 , 1.25,1.24 for now
so 1.23 might be too old , but if you propose I can accept if it's really needed and no big harm

@nikParasyr chart lint failing

@zetaab should be fixed.
For backports are the commits cherry-picked with the bot or i have to open PRs to related branches? (each backport needs a change to the commit for the chart.version)

/retest

/retest
not sure if i can trigger it though

/lgtm

/approve

/approve

@jichenjc There was no release made for this PR. namely 2.3.2. Also noticed there is no release for 2.3.1.
I have added a releaseNote now, but probably it's too late. What is required from my side to get a 2.3.2 release with this included?

um.. it should release latest release through this job
https://github.com/kubernetes/cloud-provider-openstack/blob/master/.github/workflows/release.yaml#L23
and yes, latest 2.3.1/2.3.2 not there yet, is it because of the actions not performed or the release NOte is mandatory?
if it is, we can create a new PR to trigger the release of helm chart..
# helm update
Command "update" is deprecated, Use 'helm repo update'

Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "cpo" chart repository
Update Complete.

# helm search openstack
NAME                                    CHART VERSION   APP VERSION     DESCRIPTION
cpo/openstack-cinder-csi                2.3.0           v1.25.0         Cinder CSI Chart for OpenStack
cpo/openstack-cloud-controller-manager  1.4.0           v1.25.0         Openstack Cloud Controller Manager Helm Chart
cpo/openstack-manila-csi                1.6.0           v1.25.0         Manila CSI Chart for OpenStack

and yes, latest 2.3.1/2.3.2 not there yet, is it because of the actions not performed or the release NOte is mandatory?

From checking the job its only triggered on pushing on release-* branches while this PR and the one for 2.3.1 are only on master. I'm wondering whether when release-1.27 is created if they charts will get published then :thinking:

@jichenjc https://github.com/kubernetes/cloud-provider-openstack/actions/runs/4374866122/jobs/7654829151 https://github.com/kubernetes/cloud-provider-openstack/actions/runs/4376969885/jobs/7659825574
it fails in release-1.25 and release-1.26 because the chart release exists already..
if I take for instance cinder-csi-chart, the chart release is 2.3.0 in release-1.25 and release-1.26. Then in master its 2.3.2. Anyways I think we should start promoting these chart versions correctly. We should always increase the minor version of chart when there is new kubernetes minor version. Example:
master:  2.4.0
release-1.26: 2.3.0
release-1.25: 2.2.0
release-1.24: 2.1.0
Always when anything in chart is modified, the release patch version should be increased (excluding kubernetes APP version).
Currently this version thing is kind of mess and that is why its not working.

master: 2.4.0
release-1.26: 2.3.0
release-1.25: 2.2.0
release-1.24: 2.1.0

so we should NOT accept chart version cherry-pick of release, instead, we need ask backport of general code
and submit a individual PR for version bump then it should be ok?
thus for this release, we may ask for version update to follow the above pattern and doc this process somewhere
@zetaab @nikParasyr


Remove export defaults from itwinui-react/src

Changes
Removes all export defaults from files inside of itwinui-react/src
Testing
N/A
Docs
N/A

yeah missed an import. all good now!

took care of the exports I missed!


CkEditor doesn't show Angular 14

I have many issues when I try to include this library in a Angular Project.
Has anyone tried to use this library with Angular ?
angular.json
"styles": [
              "src/styles.scss",
              "./node_modules/grapesjs/dist/css/grapes.min.css"
],
"scripts": [
              "./node_modules/grapesjs/dist/grapes.min.js",
              "./node_modules/grapesjs-preset-newsletter/dist/index.js",
              "./node_modules/grapesjs-plugin-ckeditor/dist/grapesjs-plugin-ckeditor.min.js",
              "./node_modules/ckeditor/ckeditor.js"
]

package.json
    "ckeditor": "^4.12.1",
    "grapesjs": "^0.21.1",
    "grapesjs-plugin-ckeditor": "^0.0.10",
    "grapesjs-preset-newsletter": "^1.0.1",

component.ts
import {Component, OnInit} from '@angular/core';
import grapesjs, {Editor} from 'grapesjs';
import preset from 'grapesjs-preset-newsletter';
import plugin from 'grapesjs-plugin-ckeditor'

@Component({
  selector: 'app-web-builder',
  templateUrl: './web-builder.component.html',
  styleUrls: ['./web-builder.component.scss']
})
export class WebBuilderComponent implements OnInit {
  public editor: Editor | undefined;

  constructor() { }

  ngOnInit(): void {
    this.editor = grapesjs.init({
      container: '#gjs',
      fromElement: true,
      height: '700px',
      width: 'auto',
      storageManager: false,
      panels: { defaults: [] },
      plugins: [preset, plugin]
    });
  }
}

Console output :
Failed to load resource: the server responded with a status of 404 (Not Found)
localhost/:1 Refused to execute script from 'http://localhost:4200/config.js?t=J5S9' because its MIME type ('text/html') is not executable, and strict MIME type checking is enabled.
localhost/:1 Refused to apply style from 'http://localhost:4200/skins/moono-lisa/editor.css?t=J5S9' because its MIME type ('text/html') is not a supported stylesheet MIME type, and strict MIME checking is enabled.
fr.js:1
Failed to load resource: the server responded with a status of 404 (Not Found)
localhost/:1 Refused to execute script from 'http://localhost:4200/lang/fr.js?t=J5S9' because its MIME type ('text/html') is not executable, and strict MIME type checking is enabled.
core.mjs:9171 ERROR TypeError: Cannot set properties of undefined (setting 'dir')
    at Object.f (ckeditor.js:248:46)
    at e (ckeditor.js:249:99)
    at Array.n (ckeditor.js:249:189)
    at q (ckeditor.js:249:338)
    at CKEDITOR.env.ie.d.$.onerror (ckeditor.js:250:354)
    at HTMLScriptElement.wrapFn (zone.js:766:39)
    at _ZoneDelegate.invokeTask (zone.js:406:31)
    at Object.onInvokeTask (core.mjs:26261:33)
    at _ZoneDelegate.invokeTask (zone.js:405:60)
    at Zone.runTask (zone.js:178:47)
handleError @ core.mjs:9171
localhost/:1 Refused to apply style from 'http://localhost:4200/skins/moono-lisa/editor.css?t=J5S9' because its MIME type ('text/html') is not a supported stylesheet MIME type, and strict MIME checking is enabled.

https://github.com/GrapesJS/ckeditor/issues/51


Implement compatibility with pandas 2.0

Description of PR
This PR implements changes necessary for tests to pass with pandas v2.0 and nbconvert v7.3.
Note that the pytest-legacy workflow and the py3.7-tests (where pandas 2.0 was not released) ensure that also older versions of pandas continue work with the required changes.
Closes #738

fyi @phackstock @gidden @coroa


Email submissions aren't getting sent the double confirmation email from MC

Alrighty, I've double checked everything and can't figure out why MC is not sending double confirm emails for submissions (I have this turned on per MC recommendation and have checked my spam folder). The submissions are being saved in Statamic and I'm not getting any console errors. Yet no confirmation emails nor new audience contacts.
Also I added the pages that use the MC signup form to be excluded from caching.
I only want to save form submissions to MC, not users.
I reckon something is wrong in my setup and would love your expert eyes. Thanks!
SETUP FILES
mailchimp.php: (includes my MC API KEY in real life)
<?php

return [

    'api_key' => env('MAILCHIMP_APIKEY'),

    users' => false,

    'users' => [
        'audience_id' => '',

        'disable_opt_in' => false,

        'check_consent' => true,

        'consent_field' => 'consent',

        'merge_fields' => [
            [
                'tag'=> '',

                'field_name' => '',
            ],
        ],

        'disable_opt_in' => false,
    ],

    'forms' => [
        [
            'audience_id' => '0c86e539bc',

            'disable_opt_in' => false,

            'check_consent' => true,

            'consent_field' => 'consent',

            'form' => 'Newsletter',

            'merge_fields' => [
                [
                    'tag'=> 'EMAIL',

                    'field_name' => 'email',
                ],
                [
                    'tag'=> 'SIGNUP',

                    'field_name' => 'merge_field',
                ],
            ],

            'disable_opt_in' => false,
        ],
    ],
];

Newsletter template
_newsletter_signup.antlers.html (The partial that includes the newsletter form)
{{ form:newsletter class="w-full" }}

    <input type="text" name="honeypot" class="hidden" />

      {{ if success }}

          {{ newsletter_signup:confirmation_text }}

      {{ else }}
        {{ fields }}

            {{ display }}

            {{ if field:input_type == 'email' }}
              {{ field }}
              <button
                class="btn rounded-l-none"
                type="submit">
                Sign up
              </button>
            {{ elseif field:handle == 'merge_field' }}
              <input class="hidden"
                      type="text"
                      name="{{ field:handle }}"
                      value="{{ newsletter_signup:merge_field }}" />
            {{ else }}

                {{ field }}

            {{ /if }}

        {{ /fields }}
      {{ /if }}

      {{ if success }}
        <button class="btn border-brand-green bg-brand-green btn--xs" type="submit" disabled="">
          <svg class="text-white" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 20 20" fill="currentColor">
            <path fill-rule="evenodd"
              d="M16.707 5.293a1 1 0 010 1.414l-8 8a1 1 0 01-1.414 0l-4-4a1 1 0 011.414-1.414L8 12.586l7.293-7.293a1 1 0 011.414 0z"
              clip-rule="evenodd" />
          </svg>
          submitted
        </button>
      {{ elseif errors }}

          {{ errors }}
            {{ value }}

          {{ /errors }}

      {{ /if }}

  {{ /form:newsletter }}

newsletter blueprint
resources/blueprints/forms/newsletter.yaml
sections:
  main:
    display: Main
    fields:
      -
        handle: email
        field:
          placeholder<EMAIL_ADDRESS>          input_type: email
          display: Email
          type: text
          icon: text
          listable: hidden
          validate:
            - required
            - email
          antlers: false
      -
        handle: consent
        field:
          inline: false
          display: Consent
          type: checkboxes
          icon: checkboxes
          listable: hidden
          options:
            'Yes': 'I agree to receive promotional emails and understand I can unsubscribe at any time.'
          validate:
            - required
      -
        handle: merge_field
        field:
          input_type: text
          antlers: false
          display: 'Merge Field'
          type: text
          icon: text
          listable: hidden
          instructions: 'This is a hidden field and edited via the Globals > Newsletter Signup'

Globals > newsletter
content > globals > newsletter_signup.yaml
title: 'Newsletter Signup'
data:
  header: 'Subscribe to the Workify blog'
  body_text: 'Get our latest insights on building a sustainable business as they''re released.'
  confirmation_text: 'Hurray! Check your email to confirm'
  merge_field: BlogSubscribe

Example Form Submission:
I know the merge tags are getting set correctly because here's a statamic form example YAML file:
(Also I tested to make sure the below email address would receive mail and it can)
email<EMAIL_ADDRESS>consent:
  - 'Yes'
merge_field: BlogSubscribe

MC settings

I can confirm "Enable double opt in" is enabled on the MC site
I've double checked the audience ID and merge tags.
I've double checked the API key Im using in my .env is correct and enabled

Chatted with Erin about this and he said it might have to do with the consent box being an array on the newsletter blueprint. (Its an array because its a checkbox)
So I turned off "check consent" in the MC config and removed the 'required validation' on the checkbox in the form fieldset and re-submitted an email:
email<EMAIL_ADDRESS>merge_field: BlogSubscribe

But alas I didn't get the MC email nor was a contact added to the MC audience.

@austriker27 is that email address already in mail chimp? If you WERE in there but then unsubscribed, you don't get asked again (as that allows someone to use your email to spam you)

@austriker27 can you please change your config from 'form' => 'Newsletter' to 'form' => 'newsletter' and try again?

confirmed via Discord, that solves the issue, closing.

@austriker27 can you please change your config from 'form' => 'Newsletter' to 'form' => 'newsletter' and try again?

I can confirm it works! Silly me, I was using the Title not the handle when I filled in the mailchimp.php config file.


Flutter run - is not work

This issue exist for fresh install flutter

I am already update my build.gradle repository like this :
repositories {
    maven { url 'https://maven.fabric.io/public' }
    mavenCentral()
    jcenter()
    google()
}

duplicate of #23404

Thanks @hariprasadiit
Merging with #23404


building long strings through concatenation is quadratic in complexity

I have a dictionary with 200,000 records.
Normally I process these records in js and I can read and write an 18MB file within a few seconds.  (Note:.. older file about 168,000).
However, when I tried doing this in Dart for our re-written product, I ran into problems with .fromJson parsing.  There seemed to be a limit of less than 90,000.
Further investigation had me try processing a csv manually.
Further investigation had me write a csv manually in case my data was bad.
Further investigation had me write a dart cli ...
The code is shown below and takes roughly 1 second per 1000 merely to do string processing in memory.
The flutter code of the same thing takes about 2.5 seconds per 1000.
So in flutter (for my real life dictionary import feature), we are talking about 600 seconds or 10 minutes.  I have not ran any of these scripts to completion.  They appear as "hung" even though they are just taking a long time.
Meanwhile.. I can read an sql file and pass db.Update(sql); in a few seconds.
For now, I will need to store my data online for downloading dictionary updates as Raw SQL transaction inserts.
I never thought I would report speed as a bug.  But when we are talking a few seconds in js versus 10 minutes, this appears as "hung" and counts as a bug.  I love dart flutter, but had I known about this limitation, I might have opted for something else.  on the other hand the framework and using db's does not require us to feel this pain until now.
Hardware: Lenovo IdeaPad 3 14ALC6
Ubuntu 22.04.1
[✓] Flutter (Channel stable, 3.3.5, on Ubuntu 22.04.1 LTS 5.15.0-52-generic, locale en_US.UTF-8)
$ flutter --version
Flutter 3.3.5 • channel stable • https://github.com/flutter/flutter.git
Framework • revision d9111f6402 (6 days ago) • 2022-10-19 12:27:13 -0700
Engine • revision 3ad69d7be3
Tools • Dart 2.18.2 • DevTools 2.15.0

import 'package:cli/cli.dart' as cli;
import 'dart:core';

void main(List<String> arguments) {
  //print('Hello world: ${cli.calculate()}!');

  String myString = "";
  // make 200k records
  for (int x = 0; x < 200000; x++) {
    myString += "$x, another variable$x, 8\n";
    if (x % 1000 == 1) {
      print("$x");
    }
  }
}

JavaScript is actually a relative outlier that decided to optimise string concatenation because people don't understand that building long strings through repetitive concatenation of immutable strings has quadratic time complexity. Most languages don't do that.
Most JavaScript engines use complicated zoo of different string representations to optimise things like concatenation and slicing. This improves performance of the code like the one you have written but comes with a lot of complexity and comes with its own performance pitfalls. Dart decided that it is not worth going this route.
The solution, as explained, is to use StringBuffer.
@lrhn do we want to update String.operator + documentation to maybe suggest use of StringBuffer?

The reason JavaScript is "faster" at string + is that they optimize specifically for using + to build a larger string incrementally. That also has a cost.
I worked on V8 years ago, and back then its internal string model was something like:

A string is either a plain, flat string (which can be one-byte per char for Latin-1 only strings, or two-bytes per char for the rest, plus some extra variations to account for externally allocated strings),
or it's a concatenation of two other strings, a "concat-string", which is represented as an object of the form (header, first string pointer, second string pointer).

This means you can do string concatenation in constant time. Yey.
It also means that you cannot do most simple string operations efficiently on a concatenated string. For example doing indexOf cannot do a simple memory sweep over the string contents.
Because of that, a lot of string operations start by flattening their receiver, if it's a concat-string. Basically, a concat-string is a "lazy concatenation", which gets forced when you try to use the string. In that way, it works like an implicit StringBuffer like the one in Dart (or Java), just without the explicit control.
When the string has been flattened, the concat string object becomes (header, full string, empty string), so that existing references to the object keeps working, but there is only a small constant overhead when using the string. (If the string gets moved by garbage collection, it just becomes the flattened string without the trivial concat wrapper).
You can write badly performing code in JavaScript too, like:
// Fast code:
var t0 = Date.now();
var s = "";
for (var i = 0; i < 50000; i++) {
  s = s + "xyzzy";
  var c = "xyzzy".charCodeAt("xyzzy".length >> 1); // Simple constant time operation on a string.
}
var t1 = Date.now();
// Slow code:
var s2 = "";
for (var i = 0; i < 50000; i++) {
  s2 = s2 + "xyzzy";
  var c = s2.charCodeAt(s2.length >> 2); // Still simple, right?
}
var t2 = Date.now();
console.log(t1-t0, t2-t1); // For me: 18, 5342

By forcing the implicit flattening at every step, you get the same quadratic behavior that Dart's s += ... gives you in a loop.
Dart (native) chose to not do what JavaScript does here, because it wanted a more predictable performance (and a smaller overhead) on string operations, which then doesn't have to first check if the string is a concatenation or not.
Instead Dart followed Java's strategy and introduced a StringBuffer that you should use for non-trivial concatenations, because then you can control when the flattening happens.
When compiled to JavaScript, there is no way around the JavaScript behavior, but you are still recommended to use a StrignBuffer (which just does string concatenation inside, but on a string which doesn't leak until you ask for it.)

Dart (native) chose to not do what JavaScript does here, because it wanted a more predictable performance (and a smaller overhead) on string operations, which then doesn't have to first check if the string is a concatenation or not.

Well, when we consider 10 minutes versus 1 second, then there is something to consider besides "pure" coding ethics.
I do think += is quite readable and I use it often in my code.  I was going to write.. "If the issue of lint is accepted, you can close this", but I evaluated how annoying it would be even for my own code.  I don't like lint warning messages and I assume many others will feel the same.  += will be lost, but it seems that the "experts" say it is better not to use it.  I'm a monk and a C++ dinosaur who started programming after 20 years of doing almost nothing.   I'm willing to accept what the experts say and even clean up my += code where such operations are insignificant.
I'm not sure  other people will like it.. similar to null safety, but it is better to use null safety.  I appreciate it even though it rendered many tutorials (when I was learning) obsolete.
So you can close it.. I guess the experts will decide about lint string buffer.
I appreciate this prompt detailed discussion on this issue and all of the hard work done on dart and flutter.  It is a great language and framework and no wonder why it is #1.
The other (original) issue I had was reading and parsing a very large json using the fromJson method of the model.  It too was very slow and seem to stop filling json arrays at near to 87,000 .  50k was fine.   This whole exercise of creating a 200k csv was to make my own data was free from corruption problems due to font conversion (myanmar to roman, etc).  I guess I'll create a json programatically and then see if size matters.  However, maybe there is a reverse issue of buffers?  I'm using the factory in the model by usual dart documentation way and it seems that factories do not really create objects and that is the point.

Yes! Thank you for the awesome explanations.  It would be cool if += was lazy like JS, but I think the lint and more explicit docs would go a long way.
The docs actually do hint at when to use StringBuffer, but it's never said explicitly. Maybe this would be a good step forward. StringBuffer does say it's "A class for concatenating strings efficiently." However, it doesn't really say when those advantages show. Also, String does not show examples of when StringBuffer would be a better option. I could see putting the example from the use_string_buffers lint into one or both of these places.
I think that doc change with the lint would reduce the confusion a lot.


Build Artifactory client with custom http.client.

[ ] All tests passed. If this feature is not already covered by the tests, I added new tests.
[x] This pull request is on the dev branch.
[x] I used gofmt for formatting the code before submitting the pull request.

fixes #289
This functionality can be used as follows:
httpProxy := "http://localhost:54321"
rtDetails := auth.NewArtifactoryDetails()
rtDetails.SetUrl(artifactoryUrl.String())
if httpProxy != "" {
      proxyUrl, err := url.Parse(httpProxy)
      if err != nil {
	      return err
      }
      rtDetails.SetHttpClient(&http.Client{Transport: &http.Transport{Proxy: http.ProxyURL(proxyUrl)}})
}
rtDetails.SetUser("username")
rtDetails.SetPassword("password")

serviceConfig, err := config.NewConfigBuilder().
      SetServiceDetails(rtDetails).
      SetDryRun(false).
      Build()

@AnalogJ,
Thanks for this contribution!
Wiull you be able to sign the CLA following the above comment added by the bot?

I have read the CLA Document and I hereby sign the CLA

recheckcla

@yahavi Hey any update on this issue? I'd love to see it merged.

Hey @eyalbe4 I dont see a test class associated with any of the structs that I modified. Can you point me at exactly which file you'd like me to modify? or where you'd like me to create this httpclient_test.go file?

Sure @AnalogJ -
I suggest we create a new file named httpclient_test.go and place it under the tests package (under the root of the project)). You'll find all the integration tests under this package.

hey @yahavi @eyalbe4

my branch has been rebased against the latest from master.
I re-organized the code a bit to match to better follow that current state of master
I added a simple failure test case as requested

I'd love to see this change merged, any feedback would be appreciated!

Hey @eyalbe4 @yahavi can you take a look at this PR?

Thanks so much for this great contribution @AnalogJ!
It is now released as part of v1.4.0.


mina-rosetta network/list endpoint returns code 2 json parse error

Not sure what expected result is - never run this before. But the ledger wallet wants something useful back that it is not getting.
mina-rosetta endpoint http://<IP_ADDRESS>/network/list
returns:
{"code":2,"message":"JSON parse error","description":"We encountered an error while parsing JSON.","retriable":false,"details":{"body":["Json_parse","Metadata_request.t"],"error":"Metadata_request.t","extra":"Request"}}

Hey, thanks for reporting your issue. Can you share your full command to that endpoint (or more context on how it's being called)? I think the issue may be that Rosetta requires you to provide {} as the request body even though no body parameters are required.
https://www.rosetta-api.org/docs/NetworkApi.html
omer@Omers-MacBook-Pro mina % curl --header "Content-Type: application/json" --request POST http://localhost:3087/network/list
{"code":2,"message":"JSON parse error","description":"We encountered an error while parsing JSON.","retriable":false,"details":{"body":["Json_parse","Metadata_request.t"],"error":"Metadata_request.t","extra":"Request"}}

omer@Omers-MacBook-Pro mina % curl --header "Content-Type: application/json" --request POST --data '{}' http://localhost:3087/network/list
{"network_identifiers":[{"blockchain":"coda","network":"debug"}]}

I probably should have closed this; apologies. This was me trying to figure out why ledger was returning ("Failed to get network identifiers") and snooping the ledger-app-mina code quickly, seeing this on line 136
network_resp = requests.post(MINA_URL + '/network/list'
and then just trying to hit the endpoint directly.
It turns out that with the correct parameters, that is not an actual problem, and I got ledger working yesterday afternoon. I should have come back to close sorry!

No worries! Saw there was discussion in Discord yesterday but didn't have time then to look closely.


Search for repositories

It starts getting quite difficult to search for repositories based on language since the number has exceeded 100 repos. Wouldn't it be better if we add a simple search based on language so people can take a look at languages which they prefer to contribute to.

Ya like php or java that is good with server side search engines. Java script or sql is good for fast processing.

I have submitted a Pull Request #69 implementing a search box. Please have a look!


feat(TPG>5.38)!: added deletion_policy to private service access sub-module and promoted to GA provider

Fix #585

Add deletion_policy to private service access sub-module
Moved private service access sub-module to GA provider

/gcbrun


Navigation menu issues when nesting pages

What should change
There are a couple of issues that are related:

Nested pages display every heading on every nested page in the navigation bar
If a page has nested pages beneath/within it, the headings for the top-level page do not appear in the navigation bar

1. Nested pages display every heading on every nested page in the navigation bar
At the moment, if you nest pages with multiple headings in the tech docs template, every heading for every page in the nest is visible from the navigation bar.
In this example, the API reference page has 10 pages nested beneath it. You can see two of these nested pages in the screenshot - Create a payment API reference and Get information about a single payment API reference:

This is a problem because, when you load one of the nested pages, the navigation bar jumps right to the bottom. If you're viewing one of the
It would be useful if it were possible to have a global (or per-page) setting to only display the headings for the current page. So the example above would instead look like this:

2. If a page has nested pages beneath/within it, the headings for the top-level page do not appear in the navigation bar
In this example, the top-level page API reference has headings that should be visible in the navigation bar, such as Endpoint. Instead, the navigation bars is only displaying the nested pages and the headings from the nested pages:

User need
As it stands, I haven't done any testing with users beyond the GOV.UK Pay team. Within the Pay team, however, everyone that went to use the API reference commented on the navigation bar and its impact on their use.
Issue 2 prevented them from easily jumping to the headings they needed on the top-level page.
Issue 1 caused general navigation issues, where the user would select, for example, nested page 1 but the navigation menu would jump to the bottom. The user needed to then scroll past 9 nested pages (and their headings) to re-orient themselves.

Yes please! This would make our documentation more readable e.g. https://user-guidance.services.alpha.mojanalytics.xyz/data-engineering-tools/


Add cacert.pem as its own package

Maybe it could now be added as a dependency to wget and curl

Sounds good!


Implement datetime_formats in DatetimeInspector

Description
I have enhanced the DatetimeInspector class to include the capability to detect the specific format of datetime data in each identified datetime column. This enhancement involves adding a new method detect_datetime_format which iterates over a list of user-defined and preset datetime formats, attempting to parse dates in a series using each format. The fit method has been updated to utilize this new functionality, thereby not only identifying datetime columns but also determining their specific formats.
Motivation and Context
It was requested in #95
How has this been tested?
The changes have been thoroughly tested with a series of new unit tests that cover a wide range of scenarios, including:

Successful format detection for uniform datetime series.
Detection failure in cases of mixed valid and invalid datetime strings.
Proper utilization of user-defined formats in format detection.

Types of changes

[ ] Maintenance (no change in code, maintain the project's CI, docs, etc.)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

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

Cool!  I will finish the code review as soon as possible.

Thank you!

Thank you guys! :grin:


change log added

change log added.

@rahulshukla-webkul
Here is the list of all the enhancement which are implemented in this release
[enhancement] slider path should also work in default theme
[enhancement] add configuration to enable/disable the image search feature
[enhancement] multi product images should be changed on mouse hover
[enhancement] add configuration to set admin url while install using bagisto:install command
[enhancement] add confirm box while moving an item to wishlist from the cart page
[enhancement] Add option to export auto generated coupons in cart rule
[enhancement] Add bulk category delete functionality
[enhancement] default shop content should be localized
[enhancement] hide out of stock items from homepage
[enhancement] Show product price discount percentage
[enhancement] schedule cron job to process newsletter campaigns mails
[enhancement] showing configurable product base on catalog rule price
[enhancement] Wishlist and Cart key in Product API
[enhancement] Add a option in the channel configuration to enable/disable wishlist
[enhancement] Add product number


Proxy docs show incorrect Docker TLS args

The docker TLS arguments are not hyphenated in the way that is depicted in site/proxy.md. Needs fixing on latest_release_doc_updates->1.3->master.

Note prog/weaveproxy/main.go had a similar issue; fixed by 385b0aa07f93bce6f42a2982214346ae8213b531


Property kafkacache.bounded.cache.expiry.secs doesn't seem to work

I'm using 4.0.2 and it seems property kafkacache.bounded.cache.expiry.secs doesn't work.
Further simplifying it, I've directly created an InMemoryBoundedCache instance since it is used by KafkaCache when expiry time is defined, put an item to it, and my expectation was that it had to be evicted after waiting longer than the defined expiry time was.
Cache<String, String> cache = new InMemoryBoundedCache<>(null, Duration.ofSeconds(1), null);
cache.put("1", "first");
Thread.sleep(3000);
assertTrue(cache.isEmpty()); // <- it fails

The implementation uses a Guava cache, which performs periodic maintenance during writes and occasionally during reads.  See https://github.com/google/guava/wiki/CachesExplained#when-does-cleanup-happen.


set position

Hello!
How can I set a screen position for the pngview image?

Do you want to set the initial offset from the command line?

Yes :)

OK let me have a look. Should be relatively easy to do.

it is working! Thank you!


Masquerade --> ThemeInterfacer [rootless] [1/2]

Masquerade is ThemeInterfacer now and rootly Interfacer won't be supported.

wait please


Update data_app.md

Fix typo
See also https://github.com/Snowflake-Labs/sfguide-data-apps-demo/pull/3

Thank you for the PR!


when I use the latest mmcls/models/losses/accuracy.py ,the topk is always 0.but the loss is decline

Hello, can you provide the entire config file?
Generally, cross-entropy loss declines if the model output gets closer to the ground truth.
For example,
>>> import torch
>>> import torch.nn.functional as F
>>> gt_label = torch.tensor([0])
>>> output1 = torch.tensor([[0.1, 0.1, 0.8]])
>>> print(F.cross_entropy(output1, gt_label))
tensor(1.3897)

>>> output2 = torch.tensor([[0.3, 0.3, 0.4]])
>>> print(F.cross_entropy(output2, gt_label))
tensor(1.1331)

Although all outputs pred is wrong, the cross-entropy still declines.

No update for a long time, closed.
If the problem is not solved, please open a new issue.


Building crates that work only on some platforms

Crates such as https://docs.rs/crate/core-foundation/0.4.1/builds fail to build on platforms other than macOS. In this case, it fails with error: native frameworks are only available on macOS targets, but the crate would build correctly on macOS targets ... docs.rs doesn't seem to even try to build on macOS, however, after this error.
Could it be changed in such a way that docs.rs tries to build on all platforms and later only show the subset of platforms where it succeeded to build?

This is a known limitation of docs.rs.
Docs.rs is only using a Linux host machine right now. I can implement building on OSX if someone donates an OSX server.

Huh? But I changed coremidi-sys in such a way that it would be cfg'ed out if the target is not macOS, so it would not result in the linking error, but instead just build an empty crate. This worked, I can see the documentation for macOS: https://docs.rs/coremidi-sys/2.0.2/x86_64-apple-darwin/coremidi_sys/
So this means that somehow building for macOS target does work, and docs.rs should do that even if the build for other targets fails.

Ahh I see, I thought OSX targeted crates requires an OSX host machine, but looks like they are like winapi crates and rust-std for OSX is enough to build this type of crates.
Yes, I agree docs.rs should try to build crates for all targets and decide default-target itself. This is a neat idea and thanks for suggestion. I'll see what I can do.

Thanks, that sounds great! 👍

Can I do anything to help get this implemented? I haven't looked too closely, but it looks like src/docbuilder/chroot_builder.rs is the only file that needs to be changed.

I'm actually hoping to go the other way and only build one target unless requested otherwise, see https://github.com/rust-lang/docs.rs/pull/532.
Would setting default-target to x86_64-apple-darwin fix your problem?

Yes. I think that should work.


VAE commandline args equivalent in colab to apply to all models

Following this guide: https://www.reddit.com/r/StableDiffusion/comments/yaknek/you_can_use_the_new_vae_on_old_models_as_well_for/
Looks like --vae-path could be applied to colab if defaulted in webui.bat? Or added as a checkbox for optional use.

for now, it is automatically merged in the "download model" cell


Azure Cloud Shell has acrylic on with default settings

Environment
Windows build number: 2004 build 19041
Windows Terminal version (if applicable): 1.1.2021

Steps to reproduce

In settings.json, make sure nothing is inside profiles.defaults
In Windows Terminal, open a Windows PowerShell tab, and notice that acrylic is off
Open an Azure Cloud Shell tab, and notice that acrylic is on
In Settings.json, set profiles.defaults.useAcrylic to false
Switch back to the Azure Cloud Shell tab, and notice that acrylic is off

Expected behavior
Because https://aka.ms/terminal-profiles-schema states that profiles.defaults.useAcrylic defaults to false, I expect/assume an Azure Cloud Shell tab would have acrylic off by default.
Actual behavior
Azure Cloud Shell tabs have acrylic on by default.

I was surprised to see acrylic there.
https://github.com/microsoft/terminal/blob/fda80eaf64b6f4bb41f248f5c6b355d672ad64f8/src/cascadia/TerminalApp/AzureCloudShellGenerator.cpp#L39-L41
If the non-default theme and acrylic are required by some branding guideline, then perhaps Windows Terminal should write those settings of the Azure Cloud Shell profile to settings.json when it creates that file. Doing so would at least make it easier to understand which settings are making Azure Cloud Shell look different from other profiles.

Fair points.

I'd like to take a look at fixing this, should the acrylic settings be written to the settings.json file, or should they be removed entirely?

Meh, you can probably just remove it from the Azure generator and be done with it :smile:


Correct parsing of YieldExpression

At long last, here is a fix for gh-2923! You'll note the very first thing this
patch does is factor out the Mozilla-JavaScript-1.7-specific logic into a
standalone function. I think we 'd all like to remove moz one day, but until
then, this seems like the safest/most readable way to change the ECMAScript
standard version. (Since we we have no means to verify conformance with the
Mozilla-specific extension and no bug reports for that implementation, I've
elected to maintain behavior there.)
@rwaldron Would you mind reviewing this for me?

Coverage decreased (-0.1%) to 97.523% when pulling 292e4698e3ca931592ef1db0ca0c79a038b7058d on jugglinmike:2923-yield-prec-on-asi-2 into 3ef7a0395cea5ec963a51f69b55a404f3f8e4578 on jshint:master.

@rwaldron Bugging you again because you mentioned that you didn't see this one.

Are you ok with the coveralls fail?

Coverage decreased (-0.02%) to 97.639% when pulling 525a993780d84d658582d0995c9cae45b840c101 on jugglinmike:2923-yield-prec-on-asi-2 into 3ef7a0395cea5ec963a51f69b55a404f3f8e4578 on jshint:master.

I was okay with the decrease in test coverage, but that's because I was being
lazy. When I went to address it, I found I could simplify the new mozYield
function slightly--when we're taking that branch, it's impossible for
state.inES6() to return false, so one branch can removed.
Thanks for keeping me honest! I'm happy to squash and force-push if you'd
like--let me know.

When this going to be added on releases?

@rwaldron Can you take another look?

Sorry for the delay in response :\
This looks good. I like that my tangentially related question about coverage lead to removal and simplification of a code path—that's rare!

LGTM!


Use UV without VENV (devcontainer)

uv pip install manim_automata

error: Failed to locate a virtualenv or Conda environment (checked: VIRTUAL_ENV, CONDA_PREFIX, and .venv). Run uv venv to create a virtualenv.
my dev env is a devcontainer, specifically the image mcr.microsoft.com/devcontainers/python:1-3.12-bullseye hosted at https://github.com/devcontainers/images/blob/main/src/python
I think .venv is useful to contain python dev envs, but in my case, the whole OS is contained. I shouldn't have to use .venv

Thanks, this is already supported -- you can pass --system or a specific Python interpreter with --python path/to/python. See the docs here: https://github.com/astral-sh/uv?tab=readme-ov-file#installing-into-arbitrary-python-environments.

But wait, --system doesn't work in GitHub codespaces

Yep, I am already tracking that one


Helix-Machines deployment parameter hash is insufficient

During deployments, we calculate a hash of deployment parameters which we use to determine whether anything has changed in an image definition (definitions/shared/*).  That code piece appears to have decayed a bit and not been kept up to date as the schema for our definitions is modified.  We should either take a pass and update the code, or refactor it to automatedly determine / compare parameter values.
Deployment parameter hash function: https://dev.azure.com/dnceng/internal/_git/dotnet-helix-machines?path=/src/DeployQueues/QueueManager.cs&version=GCa27e0dbf09977a1b51b56ae775aabf1d272ea5a0&line=873&lineEnd=903&lineStartColumn=1&lineEndColumn=1&lineStyle=plain&_a=contents
Related issue: https://github.com/dotnet/dnceng/issues/4156

Release Note Category

[ ] Feature changes/additions
[ ] Bug fixes
[ ] Internal Infrastructure Improvements

Release Note Description

IIRC bits are missing from the hash b/c the returned information lacks the necessary information. do you have an example of something that is returned but is ignored in the hash calculation❓


The model and loaded state dict do not match exactly

I use the code  “bash tools/dist_test.sh configs/inference/clip_end2end_faster_rcnn_r50_c4_1x_lvis_v0.5.py epoch_12_end2end_coco.pth 2 --eval bbox”，epoch_12_end2end_coco.pth is from the model you provided in  this github website.
however, I cannot review the result.
size mismatch for roi_head.bbox_head.zs_weights: copying a param with shape torch.Size([1024, 81]) from checkpoint, the shape in current model is torch.Size([1024, 1231]).
Do you kown what's wrong?

I use the code “bash tools/dist_test.sh configs/inference/clip_end2end_faster_rcnn_r50_c4_1x_lvis_v0.5.py epoch_12_end2end_coco.pth 2 --eval bbox”，epoch_12_end2end_coco.pth is from the model you provided in this github website. however, I cannot review the result. size mismatch for roi_head.bbox_head.zs_weights: copying a param with shape torch.Size([1024, 81]) from checkpoint, the shape in current model is torch.Size([1024, 1231]). Do you kown what's wrong?

same error, have you resolved this?

have you resolved this?


Use Integer instead of Fixnum.

Fixnum and Bignum are unified in Ruby-2.4 to Integer.
Using Fixnum prints a warning at every start with ruby-2.4.

👍 👍

Hi, is there anything I can do to help get this across the line? More of my users are switching to Ruby 2.4 and are hitting this.


Cleanup all annotations

Most viewers, including PDF.js and Preview do not seem to mind the annotations but it was preventing Adobe Acrobat printing a flattened document:

The error with the annotations does not only affect printing, "save as" is also not possible in Acrobat Reader and Acrobat Pro with a flattened pdf. In addition, e.g. the validation of the PDF with Datalogic's PDF Checker fails: "Unable to process document due to PDF Error".

The error with the annotations does not only affect printing, "save as" is also not possible in Acrobat Reader and Acrobat Pro with a flattened pdf. In addition, e.g. the validation of the PDF with Datalogic's PDF Checker fails: "Unable to process document due to PDF Error".

@btecu Can you clarify what is being done here? It looks like this logic would just delete the first annotation from a page after flattening a widget that belonged to it. If that is the intent, can you explain why that would be desirable? There's not necessarily any correlation between the first annotation on a page and the widget being flattened.

@btecu Can you clarify what is being done here? It looks like this logic would just delete the first annotation from a page after flattening a widget that belonged to it. If that is the intent, can you explain why that would be desirable? There's not necessarily any correlation between the first annotation on a page and the widget being flattened.

Can you provide a sample document that fails to be printed after flattening? Does this only happen to certain documents, or to all documents?

Can you provide a sample document that fails to be printed after flattening? Does this only happen to certain documents, or to all documents?

@Hopding Notice the loop - is deleting all annotations. As a sample, take a look at the document in the flattening test. It doesn't appear to be a special document but the issue happens with most PDFs that have a form.

@Hopding Notice the loop - is deleting all annotations. As a sample, take a look at the document in the flattening test. It doesn't appear to be a special document but the issue happens with most PDFs that have a form.

Ah, I see the loop now. Deleting all the annotations seems like overkill. I don't think users would expect that flattening a form would result in all pages with fields losing their annotations.

Ah, I see the loop now. Deleting all the annotations seems like overkill. I don't think users would expect that flattening a form would result in all pages with fields losing their annotations.

Looks like the field annotations are the ones that were causing the issue. Removing those along with the widget annotations seems to have fix the issue.
In addition, I've improved removeField so that it cleans up the annotations.

Looks like the field annotations are the ones that were causing the issue. Removing those along with the widget annotations seems to have fix the issue.
In addition, I've improved removeField so that it cleans up the annotations.


[BugFix]Fix truncate table throws NPE bug

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #16773
Problem Summary(Required) ：

Check existence of partition should be done from the existing olapTable, not the copiedTbl, copiedTbl will never change.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[x] 2.4
[x] 2.3
[x] 2.2

run starrocks_fe_unittest

[FE PR Coverage Check]
:disappointed: fail : 2 / 3 (66.67%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/server/LocalMetastore.java
2
3
66.67%
[4525]

@Mergifyio backport branch-2.5

@Mergifyio backport branch-2.4

@Mergifyio backport branch-2.3

@Mergifyio backport branch-2.2

@mergify backport branch-2.5 branch-2.4 branch-2.3 branch-2.2


resetOn does not work for mutations

We have multiple screens which use the same (mutation) action (let's call it UPDATE) and want to clear any error on unmounting of the screen by dispatching a RESET action.
Although the meta of UPDATE contains resetOn: [RESET], the mutation state does not get cleared properly.
Checking the code of src/reducers/network-reducer.js:34, only the queries object is being checked for reset actions via src/reducers/requests-reducer.js:128.

@ManuelHaag see resetRequests action in v0.27.0


DSPGROUP_TRUESPEECH wav file

Hi
I am using scipy.io.wavfile.read() method for reading a wav file.
I see that it does not handle DSPGROUP_TRUESPEECH files.
Is there a way i can read it using scipy/python ?!!

Hi @nellorebhanuteja, thanks for reporting. I am sorry I will have to close this issue. This is not a user forum, I would suggest you to ask your question on StackOverflow using the tag SciPy or Python.


[cleanup][pom.xml] Could org.apache.pulsar:buildtools:2.10.0-SNAPSHOT drop off redundant dependencies?

Hi! I found the pom file of project org.apache.pulsar:buildtools:2.10.0-SNAPSHOT introduced 28 dependencies. However, among them, 5 libraries (17%) are not used by your project. I list the redundant dependencies below (labelled as red ones in the figure):
Redundant dependencies
com.google.guava:listenablefuture:jar:9999.0-empty-to-avoid-conflict-with-guava:compile
com.google.code.findbugs:jsr305:jar:3.0.2:compile
com.google.errorprone:error_prone_annotations:jar:2.7.1:compile
com.google.j2objc:j2objc-annotations:jar:1.3:compile
javax.inject:javax.inject:jar:1:compile

Removing the redundant dependencies can reduce the size of project and prevent potential dependency conflict issues (i.e., multiple versions of the same library). More importantly, one of the redundant dependencies javax.inject:javax.inject:jar:1:compile induced dependency conflict in the dependency graph. As such, I suggest a refactoring operation for org.apache.pulsar:buildtools:2.10.0-SNAPSHOT’s pom file.
The attached PR helps resolve the reported problem. It is safe to remove the unused libraries (we considered Java reflection relations when analyzing the dependencies). These changes have passed org.apache.pulsar:buildtools:2.10.0-SNAPSHOT’s maven tests.
Best regards

Thanks for your contribution. Please refer the PR Naming Convention.

感谢您的贡献。请参阅 PR 命名约定。

Thanks for your contribution. Please refer the PR Naming Convention.

OK，I will finish it.

Closed as stale. Please rebase and submit a new patch if it's still relevant.


Array2d concat

concatenation for 2d arrays in numdart

How you suggest this feature? Can give me some examples?

What i mean is overriding the plus operator for Array2d similar to what is done for Arrary

Please, update to version scidart: ^0.0.2-dev.12 to get the Array2d add implemented.


Feature: 1. more complete report in json and cyclonedx-json. 2. error when get sbom from trivy or syft.

Request Description

In this moment like I know depscan can generate reports only in html and json, but this json not complete and don't have all information that exists in html, for example the priority of remediataion, or the known exploited vulnerabilities. I would be nice if we can get report in json format that contain all this information to parse it.
I noticed, that some sboms from trivy and syft in their components might miss some keys like 'version' or 'purl'. And if their miss depscan will fail. It's possible to ignore that components?

Additional Information
No response

For 1, can you check the .vdr.json file. It must have everything that is printed under the vulnerabilities attribute. For 2, can you attach a sample so we can guide towards sending a PR?

For 2:
trivy-sbom.json - this is sbom trivy what give keyerror: 'purl'
bom-source-trivy.json - this is sbom trivy that give keyerror: 'version'

@almaz045 Thank you. Would you like to become a contributor? To resolve KeyError in python we have to replace bracket based lookup ["purl"] with .get("purl"). Wrap in an if condition to handle None

Yes, of course. Should I do same thing for KeyError with 'version' or after fixing purl, will the problem with key error 'version' also disappear?

@almaz045 With both, please. like .get("purl", "") and .get("version", ""). Thank you!


Update guidance on device code lookup

Description
Currently on the device code lookup tool, when no device is found for a given search team, we instruct users to follow the instructions on the upload guide to look up the device code.

Since all LIVD table devices are now in our database, users should be able to find their device codes here without looking it up in the LIVD table.
What guidance do we want to include on the code lookup page if a device is not found? Should we instruct users to reference the LIVD table here (or should we keep that guidance in the upload guide)?
Related to #5527

@lauraykerr added a new story for some additional content updates we'll need following the LIVD sync

@lg-king I'd be in favor of removing all mention of the LIVD table from here and the guide. I think if users can't find their device, this message should send them to support.
To help mitigate that need, though, thinking we can expanding on "Please try a different search term" by providing a couple examples of the kinds of information users can search by (e.g. LOINC codes).
Do you know which fields the device lookup is currently searchable by?

That sounds good to me. It looks as if the user can just search by what's in the table - manufacturer, model, and disease type.

Recommended text:

No device found matching [search term].

Please try a different search term from the device's name or manufacturer. If you need help, contact<EMAIL_ADDRESS>

This looks good!


[pt-br] Add tasks/job/automated-tasks-with-cron-jobs.md

I added the page:
tasks/job/automated-tasks-with-cron-jobs.md
Issue #48281

PTAL @stormqueen1990

/label tide/merge-method-squash

/remove-approve


FIX-#2508: Defaults to pandas for read_csv if dialect!=None

Signed-off-by: Anatoly Myachev<EMAIL_ADDRESS>
What do these changes do?

If a csv' object is passed as a dialect, then Modin does not work, since this object is not serializable. If its name is passed as a dialect, then this dialect will not be found on the workers, since it is registered only in the main process.

[x] first commit message and PR title follow format outlined here

NOTE:  If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

[x] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] signed commit with git commit -s
[x] Resolves #2508
[x] tests added and passing
[x] module layout described at docs/development/architecture.rst is up-to-date

TBH this PR looks more like a workaround rather than a fix... a true fix would be to make non-None dialect work in a parallel way. Is that completely impossible? If so it makes sense to add it to some place in Modin docs.

TBH this PR looks more like a workaround rather than a fix... a true fix would be to make non-None dialect work in a parallel way. Is that completely impossible? If so it makes sense to add it to some place in Modin docs.

I fixed the correctness of the work - this is a fix, then what you are talking about is more like a performance fix. In my understanding, if we can at least make it work slowly - we should do it. It's still better.
I didn't have a chance to check if it can be done fast.

To me it looks as a fix, and make it work in a distributed way is a feature. When we implement a pandas function that works in a distributed way, we treat it as a feature.


MissingUniqueIndexChecker autocorrect is wrong when column is a function

Imagine a model with validates :name, uniqueness: {case_sensitive: false} (with PG this will use lower(name)).
This will make database_consistency report:
MissingUniqueIndexChecker fail MyModel lower(name) model should have proper unique index in the database
Generating a migration 'autocorrect' creates:
def change
    add_index :my_models, %w[lower(name)], name: :index_my_models_lower_name_, unique: true
end

Running this migration gives:
PG::UndefinedColumn: ERROR:  column "lower(name)" does not exist ..._add_my_models_lower_name__index.rb:3:in 'change'
The correct migration is:
 add_index :my_models, "lower(name)", name: :index_my_models_lower_name_, unique: true
database_consistency version: 1.7.23
Rails 7.1
Thanks for all your work on this gem!

Hi @stevegeek,
Thank you for noticing this! It should be fixed in 1.7.24. Please try it out and let me know if it isn't working.
https://github.com/djezzzl/database_consistency/pull/230

@djezzzl thank you!


LoadFont with "bold" weight will crash

I have an issue with getting my custom fonts to work in Macaw.
SVG file:
text id="A" font-family="Roboto-Bold, Roboto" font-size="30" font-weight="bold" line-spacing="40" fill="#FFFFFF"
It crashes in the following code for unexpectedly found nil while unwrapping an Optional value:
class func loadFont(name: String, size: Int, weight: String?) -> MFont? {
...
fontDesc = fontDesc.withSymbolicTraits(.traitBold)!
...
}
Hope you can fix it.
Thanks in advance.

Hey @flierz,
Couldn't reproduce your issue, but fixed possible crash. I think it might occurs because you have Roboto font, but don't have its bold version installed. Anyway, please use the master version to check it out.

My issue was resolved by updating to the current master branch version, thanks a lot!


aliyun-cli: 3.0.121 -> 3.0.123

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/aliyun-cli/versions.
meta.description for aliyun-cli is: "Tool to manage and use Alibaba Cloud resources through a command line interface"
meta.homepage for aliyun-cli is: "https://github.com/aliyun/aliyun-cli"
meta.changelog for aliyun-cli is: "https://github.com/aliyun/aliyun-cli/releases/tag/v3.0.123"
Updates performed

Golang update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.0.123 with grep in /nix/store/8m8fphl7pfz8k50fwlcrnk3gxigm2mvy-aliyun-cli-3.0.123
found 3.0.123 in filename of file in /nix/store/8m8fphl7pfz8k50fwlcrnk3gxigm2mvy-aliyun-cli-3.0.123

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
aliyun-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8m8fphl7pfz8k50fwlcrnk3gxigm2mvy-aliyun-cli-3.0.123 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A aliyun-cli https://github.com/r-ryantm/nixpkgs/archive/487319f568953990dfb4b52bfeef4a33e0046621.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8m8fphl7pfz8k50fwlcrnk3gxigm2mvy-aliyun-cli-3.0.123
ls -la /nix/store/8m8fphl7pfz8k50fwlcrnk3gxigm2mvy-aliyun-cli-3.0.123/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

aliyun-cli

Maintainer pings
cc @ornxka @kalbasit for testing.

Change looks okay https://github.com/aliyun/aliyun-cli/compare/v3.0.121...v3.0.123


Optimize material compute

Since we always set instanceCount = 1 and baseVertex = 0, we don't need to put those into the surfaceCommands buffer.
Use a macro to get the correct uvec2 part of the uvec4 array to avoid adding padding.
This slightly improves performace since there's less memory to fetch/write.

LGTM


Added copyright headers, added NOTICE.txt file, updated LICENSE.txt and THIRD_PARTY_LICENSES.txt

Change to prevent any copyright issues.

cla_bot is broken going to merge this on behalf of Isaac.


Please update the SDK constraint

Please update the SDK constraint on this package. I am getting error message on get:
Package reflectable has no versions that match >=1.0.1 <2.0.0 derived from:

pointycastle 0.11.1 depends on version >=1.0.1 <2.0.0
pub get failed (1)

Pointy Castle currently doesn't work for Dart 2.0. Somehow a lot of tests are broken, possibly because of an issue with the BigInteger class that is used everywhere from the bignum package.

@rickyazhari As you can see in #122, the holdup is in the SDK, I'm waiting for a BigInt.toBytes method to avoid having to use custom serialization methods.

@stevenroose Thanks for the update.

Can you check if this is solved with the new v1.0.0-rc1 release?

@yhua537 I was able to resolve a similar reflectable issue by using this fork

Unfortunately PointyCastle users have to wait because of https://github.com/dart-lang/reflectable/issues/136 unless there is a fork that doesn't uses reflectable package, that seams that is being addressed at: https://github.com/PointyCastle/pointycastle/issues/131
I'm really in favor of just removing reflection. I have myself not used them at https://github.com/leocavalcante/password-dart and https://github.com/leocavalcante/encrypt, but I can't upgrade this packages because of this dependency.

I'm planning to remove the reflectable dependency. Problem is that I don't have much time to work on Dart projects..

looks like reflectable 2.0.3 is fixing #136. Should this be resolve soon as well?
@modulovalue, thanks, I was using my own plugin for the moment so wasn't hurry.


Page URL not defined

I already have the 'Page URL not defined' message in Matomo.

Hi @othmenbahriyo can you please elaborate a bit? What app are you building and what page urls are you expecting in Matomo? Do you have specifig screens that should map to page urls to be visible in tracking?

want to display the URL of pages in Matomo like in the first image, but I'm getting the result shown in the second image
i use this code :
if (status === 'granted' || status === 'undetermined') {
await trackScreenView({
name: screenName,
userInfo: {
rec: 1,
action_name: screenName,
urlref: "/" + screenName,
url:  "/" + screenName,
lang: Localization.locale,
res: ${SCREEN_WIDTH}x${SCREEN_HEIGHT},
ua: deviceName,
new_visit: 0,
uid: user,
dimension1: 'V2',
dimension2: Localization.locale,
dimension3: authenticated['authenticated'] === true ? 'connected' : 'not connected',
dimension4: statusLocation,
apiv: 1,
},
});
}


[rtext] ExportFontAsCode() doesn't work without OpenGL context

[X] I tested it on latest raylib version from master branch
[X] I checked there is no similar issue already reported
[X] I checked the documentation on the wiki
[X] My code has no errors or misuse of raylib

Issue description
Whenever I try to export a font to code, the program I've written (attached below) segfaults if I don't do InitWindow() beforehand. Going into the functions with gdb I thikn this is because no OpenGL context has been initialised (sorry if that's the wrong terminology) and I would presume some bogus pointer accesses memory where it shouldn't.
Environment
Debian 12.4 x86_64
OpenGL version 4.60
Mesa Intel(R) HD Graphics 520
Code Example
#include "raylib.h"
#include <stdio.h>

int main(int argc, char *argv[]) {
    // Check if a filename is provided as a command-line argument
    if (argc != 2) {
        printf("Usage: %s <filename>\n", argv[0]);
        return 1;
    }

    // Get the filename from the command-line argument
    const char *filename = argv[1]; // Use a pointer to the argument, not a single char
    const char *fontFilename = TextFormat("%s.ttf", filename);
    const char *headerFilename = TextFormat("%s.h", filename);

    // Load the font
    InitWindow(1, 1, "Font"); //Would not work without this line
    Font font = LoadFont(fontFilename);

    bool success = ExportFontAsCode(font, headerFilename);

    if (success) printf("Font exported successfully to %s\n", headerFilename);
    else printf("Error exporting font to %s\n", headerFilename);

    UnloadFont(font);

    return 0;
}

@Sot-Mits @ubkp Yes, that's right and it is this way by design.


Host list: convert to RecyclerView.

This converts the rest of the view to the Material Design spec. Changes include item animation as well as the "shadow" semicircles which appear at the top and bottom of the list when scrolling. Furthermore, performance should be improved through automatic view recycling.
Note that the click handler and context menu handler were only moved and were left unchanged. RecyclerView use requires that these handlers are applied to the individual child views instead of being added to the list ViewGroup as a whole.

@khorimoto can you get ./gradlew check happy?

Done.

Whoops, looks like this caused some errors:
org.connectbot.StartupTest > localConnectionCanDelete[Nexus 7 - 5.1]  [31mFAILED  [0m
        java.lang.IllegalStateException: You need to use a Theme.AppCompat theme (or descendant) with this activity.
        at android.support.v7.app.AppCompatDelegateImplV7.createSubDecor(AppCompatDelegateImplV7.java:309)

Not sure if this is a real error, but might be if the ID was changed:
org.connectbot.StartupTest > localConnectionCanChangeToRed[SK17a - 2.3.3]  [31mFAILED  [0m
        android.support.test.espresso.NoMatchingViewException: No views in hierarchy found matching: with id: android:id/list
        If the target view is not part of the view hierarchy, you may need to use Espresso.onData to load it from one of the following AdapterViews:android.support.v7.widget.AppCompatSpinner@2affa350


[BugFix] Revert Breaking Change to ChatModule.Reload API

In multi-GPU setting, a hard constraint is that currently only shared library is supported, and a string path is required to be provided to Disco's session as:
sess.load_vm_module("/path/to/shared/lib.so")

And therefore, previously in PR #867, we made a breaking change to the signature of ChatModule.reload API as below:
- void Reload(tvm::runtime::Module lib, ...);
+ void Reload(String lib_path, ...);

However, it brings an unexpected breaking change, which on iOS and Android apps, we expect the library being bundled together as static system libraries rather than shared libraries, and in these cases, the string path is not available, making them crash after the PR merge.
This PR re-enables backward compatibility for those apps and also preserves the ability to run multi-GPU in TVM Unity's Disco runtime by allowing a union type:
void Reload(Union[tvm::runtime::Module, String] lib_path, ...);

Tracking issue: https://github.com/mlc-ai/mlc-llm/issues/936.

CC: @jinhongyii @MasterJH5574 @spectrometerHBH


Document-store -> vectorstore -> insert -> Error: documentStoreServices.insertIntoVectorStore - Error: documentStoreServices._insertIntoVectorStoreWorkerThread - TypeError: fetch failed

Describe the bug
While upserting a new Document Store getting a 500 error
To Reproduce
Steps to reproduce the behavior:

Adding an ollama embeddings nomic-embed-text:latest
Adding a Faiss as a Vector Store
Upsert
See error

Expected behavior
The success upsert message is expected, but the 500 error is coming
Status: 500
Error: documentStoreServices.insertIntoVectorStore - Error: documentStoreServices._insertIntoVectorStoreWorkerThread - TypeError: fetch failed
Screenshots

Flow
If applicable, add exported flow in order to help replicating the problem.
Setup

Docker running locally
Flowise Versionv2.1.3
OS: windows
Browser: chrome,

I have the exact issue,  it doesn't mater if I use docker or install all manually.

is your FAISS dimension compatible with nomic embed text embedding dimension?

same issue for me
Status: 500
Error: documentStoreServices.insertIntoVectorStore - Error: documentStoreServices._insertIntoVectorStoreWorkerThread - TypeError: fetch failed

@shighcloud-ea @HenryHengZJ @AlamokCR @dev3py I found the solution and its worked
In the Ollama node configuration, use http://host.docker.internal:11434 instead of http://localhost:11434.
Since Ollama is installed on the host machine and not within Docker, Docker containers cannot connect to the host’s services via localhost. By using host.docker.internal, you enable Docker to communicate with services on the host.


DRILL-5799: native-client: Support alternative build directories

Build failed with unrelated reasons to this PR:
[WARNING] Rule 0: org.apache.maven.plugins.enforcer.RequireFilesSize failed with message:
The file drill-jdbc-all-1.12.0-SNAPSHOT.jar is outside the expected size range.

                  This is likely due to you adding new dependencies to a java-exec and not updating the excludes in this module. This is important as it minimizes the size of the dependency of Drill application users.
/home/travis/build/apache/drill/exec/jdbc-all/target/drill-jdbc-all-1.12.0-SNAPSHOT.jar size (31071836) too large. Max. is 29000000/home/travis/build/apache/drill/exec/jdbc-all/target/drill-jdbc-all-1.12.0-SNAPSHOT.jar

Build issue has been corrected via another PR.

@paul-rogers Thanks, rebase fixed the build.

@parthchandra or @sohami, can one of you take a quick look at this one since you are familiar with our C builds?

+1. Thanks for the fix.


Predicate validation in parquet seems to miss something

    ddf = dd.from_dict(
        {"A": range(8), "B": [1, 1, 2, 2, 3, 3, 4, 4]},
        npartitions=4,
    )
    ddf.to_parquet(tmp_path, engine=engine)

    with pytest.raises(ValueError, match="not a valid operator in predicates"):
        unsupported_op = [[("B", "not eq", 1)]]
        dd.read_parquet(tmp_path, engine=engine, filters=unsupported_op)

This should raise but doesn't with dask-expr

cc @rjzamora if you have thoughts where this belongs

Just to clarify: We can still add an extra/extra earlier validation step to dask-expr. I'll take a look to see where that would be easiest.

Sorry, you are completely correct, it's raising when I call compute.
Ideally we would fail earlier, but this is by no means urgent, I'll change the title


Security config log message to debug, readme updates

A couple typo fixes for the README as well as changing the log level for the Security configuration reloaded message to DEBUG.

thx


DataManagementApi: Use management api in samples

What this PR changes/adds:
Add the data management api into the samples.
Why it does that
It's necessary to introduce the management api to all users of the EDC.
Further notes

Add default config to the basic launcher
Add data management api into sample 03 and add some documentations
Update samples above sample 03
Reorder extensions in settings.gradle.kts
Fix typos in README.md

Checklist

[ ] added edit appropriate tests?
[x] performed checkstyle check locally?
[x] added/updated copyright headers?
[x] documented public classes/methods? (nothing to document)
[x] added/updated relevant documentation?

Codecov Report

Merging #733 (bc3e029) into main (01914cd) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #733   +/-   ##
=========================================
  Coverage     56.57%   56.57%
  Complexity     2126     2126
=========================================
  Files           593      593
  Lines         13166    13166
  Branches        936      936
=========================================
  Hits           7449     7449
  Misses         5302     5302
  Partials        415      415

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 01914cd...bc3e029. Read the comment docs.

The minikube test fails because the consumer pod has actually two URLs now (one for each port), therefore the environment variable CONSUMER_URL in the minikube_test.sh file is in the wrong format.
The short term solution is to remove the data management api from sample 04.0's build script and config again.
Mid-term we need to come up with a better solution though. Linking @algattik since he initially authored the minikube test.

The minikube test fails because the consumer pod has actually two URLs now (one for each port), therefore the environment variable CONSUMER_URL in the minikube_test.sh file is in the wrong format.

Replacing the export CONSUMER_URL= in minikube-test.sh with the following should do the job:
ip=$(minikube ip)
port=$(kubectl get svc consumer-dataspace-connector  -o jsonpath='{.spec.ports[?(@.name=="http")].nodePort}')
export CONSUMER_URL=http://$ip:$port

Ready for review & merge!
\cc @paullatzelsperger @algattik

@florianrusch-zf could you resolve the conflicts, so we can merge?

@florianrusch-zf ping, let's see if we can merge this ASAP.

@paullatzelsperger @jimmarino I'm back and will work on it ASAP

@paullatzelsperger @jimmarino can be merged


Can't purge PPA

If you Completed all troubleshoot options, check below

[ ]  Did you Follow all installation instructions carefully from here?
[x] Yes
[ ]  Did you check All troubleshoot options from here?
[x] Yes

Desktop (please complete the following information):

Distro: KDE Neon (Ubuntu)
Desktop: KDE Plasma 6
Version: 22.04
PipeWire Related packages Version: 1.0.7-3~ubuntu22.04
Kernel version: 6.5.0-45

Describe the bug
Trying to run sudo ppa-purge ppa:pipewire-debian/pipewire-upstream
Packages won't fully downgrade. Renders broken packaging.
To Reproduce
Steps to reproduce the behavior:

sudo add-apt-repository ppa:pipewire-debian/pipewire-upstream
sudo apt-get update
sudo apt install gstreamer1.0-pipewire libpipewire-0.3-{0,dev,modules} libspa-0.2-{bluetooth,dev,jack,modules} pipewire{,-{audio-client-libraries,pulse,bin,jack,alsa,v4l2,libcamera,locales,tests}}
sudo ppa-purge ppa:pipewire-debian/pipewire-upstream

Expected behavior
Packages should just downgrade without causing broken packaging.

You probably need to remove ppa:pipewire-debian/wireplumber-upstream first.
I assume you did add it, as the instructions show, but your steps to reproduce the problem don't include adding the wireplumber PPA.
Not removing this first is probably causing many dependency issues.

Workaround, which worked for me:

Lookup the latest version via sudo apt info -a pipewire
Run the following to remove repository and install the current packages with the correct version:

sudo add-apt-repository --remove ppa:pipewire-debian/pipewire-upstream
sudo apt install pipewire=0.3.48-1ubuntu3 pipewire-bin=0.3.48-1ubuntu3 libpipewire-0.3-modules=0.3.48-1ubuntu3 libpipewire-0.3-0=0.3.48-1ubuntu3 libspa-0.2-modules=0.3.48-1ubuntu3

I hope this helps to resolve your problem.

@grimmjo thanks a lot!


Does Azure play any part in this?

This just looks like a locally hosted SignalR chatroom example, I don't understand that part the Azure resource plays in this - is it used at all?

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

ID: edb1fa6c-8433-882f-8b97-4d381f885b1d
Version Independent ID: aad3ec71-363f-3500-8bcd-dc040f71821e
Content: Quickstart to learn how to use Azure SignalR Service
Content Source: articles/azure-signalr/signalr-quickstart-dotnet-core.md
Service: signalr
GitHub Login: @sffamily
Microsoft Alias: zhshang

As I understood it, this is only locally and Azure will only be interesting from the next step forward:
https://docs.microsoft.com/en-us/azure/azure-signalr/signalr-concept-authenticate-oauth#deploy-the-app-to-azure

It's definitely confusing, I thought the sample app would be using Azure, as the docs are about a Quickstart for Azure SignalR Service, not just plain SignalR. I realized there were no connections to my SignalR instance on Azure. I checked the Quickstart from the SignalR Service profile I created on the Azure Portal and then I realized I wasn't configuring correctly Startup.cs. I missed the following:
services.AddSignalR()
                    .AddAzureSignalR();

And:
app.UseAzureSignalR(routes =>
{
routes.MapHub("/chat");
});

Thank you for taking the time to comment. You are correct; the example runs entirely locally. There is no Azure component to this example. We're currently revising the article to clarify the example.
This issue is nearly a duplicate of #48979 . I'll close this issue now and then close the other issue when the revised article is published.
#label:duplicate
#please-close


Publish TAR v2.0.0_spec

Preparation

[x] Double check all pull-requests being closed
[x] Double check all v2.0.0_spec issues being closed
[x] Double check Postman being in synch with OAS published on GitHub
[x] Create new folder structure and move files
[x] Create read.me in spec folder
[x] Test links in read.me in spec folder
[x] Create read.me in root folder
[x] Write short description about application
[x] Test links in read.me in root folder
[x] Create release branch (e.g. TAR_v2.n.m) from develop

Add basic elements in OAS in release branch in VS Code

[x] Add all paths of Service Layer - Basic Part
[x] Add all paths of OAM Layer - Individual Part
[x] Add Profile according to ProfileList
[x] Add Clients and Servers
[x] Delete ElasticsearchClient if not required
[x] Check, whether Common Components are up-to-date (don't overwrite individual extensions to the Common Components)
[x] Replace "=*-1-0-0-" by actual ApplicationName and ReleaseNumber
[x] Push into release branch

Publishing

[x] Create and merge pull-request into main
[x] Create release/label "v2.0.0_spec" in main (similar to example in ApplicationPattern)
[x] Postman: create new version with name "2.0.0_spec", but don't change it now!
[x] Postman: create new version with name "2.0.1" for future work
[x] Postman: copy OAS from main branch (label "v2.0.0_spec") into version with name "2.0.0_spec"

Completed.


How can I add an example to the "[]map[string]interface{}" struct?

https://github.com/swaggo/swag/issues/327
I saw this issue and learned how to add an example to "map[string]interface{}", but how do I add an example to "[]map[string]interface{}"?
I would like to output an example like the following.
"attachments": [
    {
      "title": "string",
      "path": "string"
    }
  ],

Have you tried the last example?
CustomType map[string]interface{} `json:"map_data" swaggertype:"object,string" example:"key:value,key2:value2"`

@ubogdan
Of course.
But this was not what I wanted.
[Definition output by that sample]
        map_data:
          additionalProperties:
            type: string
          example:
            key: value
            key2: value
          type: object

[Definition I want to output]
        map_data:
          additionalProperties:
            type: string
          example:
            - key: value
              key2: value
          type: object

@yukanke-saab
Were you able to get this working for []map[string]interface{}?