[Snyk] Security upgrade snyk from 1.707.0 to 1.996.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `yarn` dependencies of this project.

Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
yarn.lock

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

713/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 6.4
Command Injection SNYK-JS-SNYK-3037342
No
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

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

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

Codecov Report
Base: 60.97% // Head: 60.97% // No change to project coverage :thumbsup:

Coverage data is based on head (75fa9a9) compared to base (d8b1702).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #363   +/-   ##
=======================================
  Coverage   60.97%   60.97%
=======================================
  Files           3        3
  Lines          41       41
  Branches        9        7    -2
=======================================
  Hits           25       25
  Misses         16       16

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Create Bridge Domain

Hello Carl,
i'm sebastian, i been trying to create a bridge domain, i wasn't able to do it,
but i could create tenant, vrf and app, but it fails to create the bridge domain
I'm running the scripts against an apic 3.1(1) version
this is the xls file that i'm using
aci_deploy.zip
this is the output:

Please can you help me, thanks a lot for your time
Best Regards

Ah I think I see what is going on. Can you change the status on your BD --> "created, modified"
This is one of the things I would like to overhaul in PDT -- I've coupled some things together that probably shouldn't have been -- for example BD/VRF binding. Running the spreadsheet as you've got this is the response I see:
{"totalCount":"1","imdata":[{"error":{"attributes":{"code":"103","text":"Cannot create Private Network (fvRsCtx); object uni/tn-angeles/BD-bd1/rsctx already exists."}}}]}
Basically the BD call is pointing to the VRF (to associate it there), but the VRF already exists, because the status is "created" it will basically fail if the object already exists.
Give that a shot and let me know if that clears it up! Longer term hopefully I will get around to updating things :)

No problem, glad its working for you!


Copy into command copying only one row from the blob storage specified not the whole table.

@shikharbsar Thank you for the feedback.
Please provide a link to the documentation in question and we will be happy to provide an update.

https://docs.microsoft.com/en-us/sql/t-sql/statements/copy-into-transact-sql?view=azure-sqldw-latest. This is the command I am talking about. It is copying only the first row not the entire row.

@shikharbsar Thank you for the update.
I was able to fix a couple of quote issues in the statements, however since the queries are based off of user-provided data, you will need to send this to<EMAIL_ADDRESS>Be sure to provide the specific query in question along with a test data set.


Do not allow registering a new account if email exists in db

This is using a built-in validation mechanism from the library django_registration: https://django-registration.readthedocs.io/en/3.3/forms.html#django_registration.forms.RegistrationFormUniqueEmail
Result:

 Handle new user submissions to avoid double accounts

Oh wow, that a result. A two word pull request was all it took! 💯  Merge away Oliwia :D


error with didFailWithError

i am getting this error for most of the time, it works mostly though -

any thoughts how we could fix this?

I will try to reproduce this issue - can you share how do you use it on the Javascript side? If I remember, this could happen as soon as either component is unmounted or the event has already been called.

The issue happens because NativeAdView doesn't set onAdError prop -> https://github.com/callstack-io/react-native-fbads/blob/master/src/withNativeAd.js#L71
I believe we can fix it by setting it to this.props.onAdError, so that whoever renders withNativeAd wrapped component, can set that prop to listen to errors.
Handling itself is handled within that wrapper anyway (in case of error - nativeAd is null and nothing is rendered).

You need to provide required onError prop for BannerView. Otherwise, when error occurs, your native code will throw.


One more Middle Man

Create a new middleware verifyMiddleware that is similar to the authMiddleware but now, it also returns not allowed response if user is not verified.
You will have to create the middleware in a new file name verify.middleware.js in the middlewares directory.

Make sure to pull before starting your work.
For any doubts or difficulty, you can ping me on discord, I'll try to help you out as soon as possible.
All the Best!!

CLAIM

Assigned to @TheDivineAurora

Claim


Deleting objects does not persist

After deleting objects (Events in your case), then closing and restarting the application, the objects that I deleted still show up. I was wondering if I was doing something wrong, since I am deleting my objects just as you are deleting your Event objects.

@connersimmons Thanks for catching that bug. It looks like that the changes from the main context aren't saved.
A solution would be to call self.persistenceManager.mergeWithMainContext(),
that causes the changes (the deletion) in the mainContext to actually save the changes, which will actually delete the stored objects from the datastore. Seems to me i forgot to actually do the merge/save call.
Try this:
func deleteAllEvents() {
       let retrievedItems = getAllEvents()

       //Delete all event items from persistance layer
       for item in retrievedItems {
           self.mainContextInstance.deleteObject(item)
       }

      //Merge /save changes on mainContext
      self.persistenceManager.mergeWithMainContext()

      self.postUpdateNotification()
}

@srmds I thought that may do the trick. I thought I had tried that, but I guess I didn't. Made that change to my code and it seems to be working great. Thanks for the quick response!


Process hangs during upload when trying to calculate file size

Summary
I have been using filepond on my website successfully for several days and recently iPhone users have been reporting an error.

Filepond just stays stuck at this screen with message "Waiting for size".
The code I use is as below:
FilePond.registerPlugin(

    // previews dropped images
    FilePondPluginImagePreview,
    // corrects mobile image orientation
    FilePondPluginImageExifOrientation,
    // encodes the file as base64 data
    // FilePondPluginFileEncode,
    // validates the size of the file
    FilePondPluginFileValidateSize,
    FilePondPluginFileValidateType, // validate type
    FilePondPluginImageResize,
    FilePondPluginImageTransform
);

// Select the file input and use create() to turn it into a pondd
FilePond.create(
    document.querySelector('input[type="file"]'), {
    acceptedFileTypes: ['image/*'],
    allowImageExifOrientation: true,
    imageCropAspectRatio: '1:1',
    imageResizeTargetWidth: 300,
    imageResizeTargetHeight: 300,
    imageResizeUpscale: false,
    imageResizeMode: 'cover',
    instantUpload: true,
    imageTransformOutputQuality: 85,
    // allowFileEncode: false,
    server: {
        process: {
            url: './upload',
            onload: (response) => {
                console.log("file successfully uploaded with id: " + response)
                imageId = JSON.parse(response)[0];
...

I do not have an iPhone so I am not able to reproduce the error. I will post more info as I receive. Everything works fine and dandy on my Android and iPad.

Environment
Version

OS
iOS 13.3.1

Device
iPhone 11 Pro Max

Browser
Safari & Chrome

Is this with latest versions of all plugins?

Yes, directly from the NPM cdn.
As a check, the validate size demos and demos on filepond site work fine on
the iPhone in question.
Any other test I can perform on my side to debug the issue?
On Tue, May 5, 2020, 11:57 PM Rik<EMAIL_ADDRESS>wrote:

Is this with latest versions of all plugins?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pqina/filepond/issues/490#issuecomment-624476175, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFXZZLQUL6H652SCZAZ4RTRQEC6FANCNFSM4M2FVC6A
.

Will run some tests

@rikschennink I have a few results from someone with an iPhone, the folder (https://drive.google.com/drive/folders/1MnLbXUBwH69DrbYhRmhqDoQ5PMn81I0t?usp=sharing) has images labeled as work_ and nowork_ for images that worked and did not. Hopefully, Google drive did not mess up the image exif data.

@rikschennink so some success in getting to the root of the error. I was able to recreate the problem in my Android (10.3.1) Firefox Beta (75.0.0-beta-6), and connecting it to my desktop firefox for remote debugging, I see the following error in the console.

And the timing since the error appeared matches the update to the filepond-plugin-image-resize .
What is interesting is, this does not happen for all images and all browsers. In a later nightly build of firefox (77.0a1), it worked fine. In the meantime, I will try to go back to the previous version of the plugin and update with my findings.

@chintanp Thanks so much for the detailed test reports, this really helped a lot in quickly solving this. I just pushed 2.0.6 which should fix this.

Works on my Android now. Thanks for the quick fix. I am tempted to version lock the library and plugins though. I am new to JS dev, so not sure what the best practices are for this.
If someone can test on iPhone and finds that the issue persists, then they can re-open.

Glad to hear it works 👍
I would advise to always version lock.


Horizontal Recyclerview inside Vertical Recyclerview

I want to create a dynamic loading view that contains horizontal recyclerview inside vertical recyclerview and vertical recyclerview should be inside Scrollview as the main view contains some more views components. So my dynamic hierarchy is as following:-
Scrollview [
| Viewpager --
| AdView --
| Vertical Items - Recyclerview {
1) [Horizontal Recyclerview {1.1, 1.2, 1.3} [Lazy Loading of Items]]
2) [Horizontal Recyclerview {2.1, 2.2, 2.3} [Lazy Loading of Items]]
3) [Horizontal Recyclerview {3.1, 3.2, 3.3} [Lazy Loading of Items]]
4) [Horizontal Recyclerview {4.1, 4.2, 4.3} [Lazy Loading of Items]]
.
.
.
[Lazy Loading of Items in Vertical Direction also]
}
| Some Another View --
]
Is this horizontal recyclerview inside vertical recylerview possible from Bravah ?
Would scroll behavior of Scrollview be smooth ?
Please suggest me.

horizontal recyclerview inside vertical recylerview could not fluent, you can Adjust the layout, or Use multiple layouts

@AllenCoder  Hey buddy, can you tell me how can I adjust my layout ?
like I want a dynamically recyclerview type of section adapter and inside that list I want nested horizontal dynamic items. This view is basically same as google play store view.
How can i get this approach ?


Add server API

Current dependencies on/for this PR:

main

PR #2   👈

This stack of pull requests is managed by Graphite.

Merge activity

Jan 29, 2:06 PM: @mertcanbs started a stack merge that includes this pull request via Graphite.


Facing an exception of "Failed to find library "leptonica-1.82.0.dll" for platform x64".

Current Behavior
Getting "Exception has been thrown by the target of an invocation" exception and "Failed to find library "leptonica-1.82.0.dll" for platform x64" as inner exception whenever we are calling a method of  new TesseractEngine(); using Tesseract dll.
Expected Behavior
Other (please specify in additional notes)
Reproducing a Behavior
Create a web application and add Tesseract dll to the references. Try to call  new TesseractEngine() method.
Device OS
Windows 10
Genshin Impact Version
5.2
Inventory Kamera Version
5.2.0
Screen resolution, screen mode, and UI scale
1920x1080
Additional notes and remarks
No response
Evidence
No response
Preflight Checklist

[X] I have checked for similar issues in the issue page, and I have not found any similar to mine.
[X] I am using the latest version of the scanner.
[X] I tried setting the scanning delay to maximum, and it still doesn't work.
[X] I have attached the video recording to the bug report.
[X] I have attached the error log to the bug report.
[X] I am available to later explains the issue.
[X] I have disabled all screen filtering software or plugins (i.e. HDR, Image Sharpening) while scanning.
[X] I use default in-game keybindings or have configured keybindings for the scanner.

Please be sure to follow the install instructions here. Usually a restart is required to fix.


Update to new Celo branding

Hey folks!
Could all icons that are shown be updated to match the new Celo branding?
That would be amazing!
Guidelines can be found at:
https://celo.org/brand-kit-symbol
https://celo.org/brand-kit-wordmark
Currently the old branding is still being used, which isn't consistent with the new Celo identity.
Examples from GoodGhosting.com:

thanks for pointing this out. :)

@Deepcryptodive I have released 0.5.0 with the new branding.


Argument error on update

We've just started getting this crash when tzdata tries to do an update of its data:
  crasher:
    initial call: Elixir.Tzdata.ReleaseUpdater:init/1
    pid: <0.2305.0>
    registered_name: tzdata_release_updater
    exception exit: {
{badarg,[{erlang,hd,[[]],[]},
{'Elixir.Tzdata.DataLoader',content_length_from_headers,1,[{file,"lib/tzdata/data_loader.ex"},{line,29}]},{'Elixir.Tzdata.DataLoader',do_latest_file_size,1,[{file,"lib/tzdata/data_loader.ex"},{line,51}]},
{'Elixir.Tzdata.ReleaseUpdater','loaded_tzdata_matches_iana_file_size?',0,[{file,"lib/tzdata/release_updater.ex"},{line,48}]},
{'Elixir.Tzdata.ReleaseUpdater',poll_for_update,0,[{file,"lib/tzdata/release_updater.ex"},{line,36}]},
{'Elixir.Tzdata.ReleaseUpdater',handle_info,2,[{file,"lib/tzdata/release_updater.ex"},{line,17}]},
{gen_server,try_dispatch,4,[{file,"gen_server.erl"},{line,601}]},
{gen_server,handle_msg,5,[{file,"gen_server.erl"},{line,667}]}]},
[{gen_server,terminate,7,[{file,"gen_server.erl"},{line,812}]},
{proc_lib,init_p_do_apply,3,[{file,"proc_lib.erl"},{line,247}]}]}

Best as I can tell, the HEAD data it's pulling no longer contains the Content-Length field. Indeed a quick test seems to bear this out:
$ curl --head https://www.iana.org/time-zones/repository/tzdata-latest.tar.gz
HTTP/1.1 200 OK
Date: Tue, 07 Mar 2017 00:30:29 GMT
X-Frame-Options: SAMEORIGIN
Content-Security-Policy: upgrade-insecure-requests
Last-Modified: Tue, 28 Feb 2017 21:18:44 GMT
Vary: Accept-Encoding
content-type: application/x-gzip
content-encoding: identity
Cache-Control: public, max-age=3600
Server: Apache
X-Cache-Host: www-varnish.iana.org
X-Cache-Hits: 1413
Accept-Ranges: bytes
Connection: keep-alive
Strict-Transport-Security: max-age=48211200

But of course I could be misreading the code.
Edit: We're running 0.5.10

Yeah I landed here as well. Except
Tzdata.DataLoader.download_new

headers look like this
[{"Date", "Tue, 07 Mar 2017 00:17:13 GMT"}, {"X-Frame-Options", "SAMEORIGIN"},
 {"Content-Security-Policy", "upgrade-insecure-requests"},
 {"Last-Modified", "Tue, 28 Feb 2017 21:18:44 GMT"},
 {"Vary", "Accept-Encoding"}, {"content-type", "application/x-gzip"},
 {"content-encoding", "identity"}, {"Cache-Control", "public, max-age=3600"},
 {"Server", "Apache"}, {"X-Cache-Host", "www-varnish.iana.org"},
 {"X-Cache-Hits", "20995"}, {"Accept-Ranges", "bytes"},
 {"Content-Length", "323983"}, {"Connection", "keep-alive"},
 {"Strict-Transport-Security", "max-age=48211200"}]

Which is list so :erlang.hd should work? Investigating further

I don't think it's going through download_new. To check for updates it hits latest_file_size which calls :hackney.head/4. So it's not doing a full download, only a HEAD call:
iex(1)> :hackney.head("https://www.iana.org/time-zones/repository/tzdata-latest.tar.gz", [], "", [])
{:ok, 200,
 [{"Date", "Tue, 07 Mar 2017 00:50:07 GMT"}, {"X-Frame-Options", "SAMEORIGIN"},
  {"Content-Security-Policy", "upgrade-insecure-requests"},
  {"Last-Modified", "Tue, 28 Feb 2017 21:18:44 GMT"},
  {"Vary", "Accept-Encoding"}, {"content-type", "application/x-gzip"},
  {"content-encoding", "identity"}, {"Cache-Control", "public, max-age=3600"},
  {"Server", "Apache"}, {"X-Cache-Host", "www-varnish.iana.org"},
  {"X-Cache-Hits", "805"}, {"Accept-Ranges", "bytes"},
  {"Connection", "keep-alive"},
  {"Strict-Transport-Security", "max-age=48211200"}]}

@bernardd Okay a little further debugging revealed a difference between
:hackney.get
:hackney.head

This version will work but is super junky but I didn't want to deep dive into hackney at the moment. (In the middle of merge with something else) But if you need to get unstuck it's there.
https://github.com/polymetis/tzdata
In particular this
https://github.com/polymetis/tzdata/commit/431044b31a2196ae98baec17451b5562070bd8f2

Thanks for the reports. It does indeed seem like they stopped returning a Content-Length header.

It looks like the Content-Length headers are back. I've been making alternative functionality that downloads the file and looks at the size - this could be used as a backup in case Content-Length headers go missing again.

Awesome, thanks. Whatever the correct behaviour if/when it goes missing is, I'm pretty sure crashing shouldn't be it :)

I guess this error I received last night is related?
GenServer :tzdata_release_updater terminating
** (ArgumentError) argument error
    :erlang.hd([])
    (tzdata) lib/tzdata/data_loader.ex:29: Tzdata.DataLoader.content_length_from_headers/1
    (tzdata) lib/tzdata/data_loader.ex:51: Tzdata.DataLoader.do_latest_file_size/1
    (tzdata) lib/tzdata/release_updater.ex:48: Tzdata.ReleaseUpdater.loaded_tzdata_matches_iana_file_size?/0
    (tzdata) lib/tzdata/release_updater.ex:36: Tzdata.ReleaseUpdater.poll_for_update/0
    (tzdata) lib/tzdata/release_updater.ex:17: Tzdata.ReleaseUpdater.handle_info/2
    (stdlib) gen_server.erl:601: :gen_server.try_dispatch/4
    (stdlib) gen_server.erl:667: :gen_server.handle_msg/5
Last message: :check_if_time_to_update

It looks like the Content-Length headers have gone away again:
iex(2)> :hackney.head("https://www.iana.org/time-zones/repository/tzdata-latest.tar.gz", [], "", [])
{:ok, 200,
 [{"Date", "Tue, 07 Mar 2017 13:36:41 GMT"},
  {"Expires", "Tue, 07 Mar 2017 14:36:41 GMT"},
  {"Cache-Control", "max-age=3600"}, {"X-Frame-Options", "SAMEORIGIN"},
  {"Content-Security-Policy", "upgrade-insecure-requests"},
  {"Last-Modified", "Tue, 28 Feb 2017 21:18:44 GMT"},
  {"Vary", "Accept-Encoding"}, {"content-type", "application/x-gzip"},
  {"content-encoding", "identity"}, {"Server", "Apache"},
  {"X-Cache-Hits", "43539"}, {"Accept-Ranges", "bytes"},
  {"Connection", "keep-alive"},
  {"Strict-Transport-Security", "max-age=48211200"}]}

Maybe there's a misbehaving server in their pool.

Is there a work around for this?

I hope to release a new version tonight that can get the file size without the headers.

Version 0.5.11 has been released: https://hex.pm/packages/tzdata
{:tzdata, "~> 0.5.11"}

Let me know if it works well for you.

Works perfectly here after the update

Working here too. I'm going to close this. Thanks heaps for the quick response.

@Iau time to time I still get the same error in our production
I'm using:
"tzdata": {:hex, :tzdata, "0.5.11", "3d5469a9f46bdf4a8760333dbdabdcc4751325035c454b10521f71e7c611ae50"

Erlang/OTP 19 [erts-8.2] [source] [64-bit] [smp:4:4] [ds:4:4:10] [async-threads:10] [hipe] [kernel-poll:false]

Interactive Elixir (1.4.1) - press Ctrl+C to exit (type h() ENTER for help)

and the stack trace is
GenServer :tzdata_release_updater terminating
** (ArgumentError) argument error
    :erlang.hd([])
    (tzdata) lib/tzdata/data_loader.ex:29: Tzdata.DataLoader.content_length_from_headers/1
    (tzdata) lib/tzdata/data_loader.ex:51: Tzdata.DataLoader.do_latest_file_size/1
    (tzdata) lib/tzdata/release_updater.ex:48: Tzdata.ReleaseUpdater.loaded_tzdata_matches_iana_file_size?/0
    (tzdata) lib/tzdata/release_updater.ex:36: Tzdata.ReleaseUpdater.poll_for_update/0
    (tzdata) lib/tzdata/release_updater.ex:17: Tzdata.ReleaseUpdater.handle_info/2
    (stdlib) gen_server.erl:601: :gen_server.try_dispatch/4
    (stdlib) gen_server.erl:667: :gen_server.handle_msg/5
Last message: :check_if_time_to_update

I'm using:
"tzdata": {:hex, :tzdata, "0.5.11", "3d5469a9f46bdf4a8760333dbdabdcc4751325035c454b10521f71e7c611ae50"

No you're not :) Because

lib/tzdata/data_loader.ex:29: Tzdata.DataLoader.content_length_from_headers/1

Line 29 in v0.5.11 is not a call to hd any more, but it was in 0.5.10. Have you actually pushed out the new version to your production system and/or reloaded it?

@bernardd good point! thanks I'll take a look

[error] GenServer :tzdata_release_updater terminating
** (MatchError) no match of right hand side value: {:error, :timeout}
(tzdata) lib/tzdata/data_loader.ex:52: Tzdata.DataLoader.latest_file_size_by_get/1
(tzdata) lib/tzdata/release_updater.ex:48: Tzdata.ReleaseUpdater.loaded_tzdata_matches_iana_file_size?/0
(tzdata) lib/tzdata/release_updater.ex:36: Tzdata.ReleaseUpdater.poll_for_update/0
(tzdata) lib/tzdata/release_updater.ex:17: Tzdata.ReleaseUpdater.handle_info/2
(stdlib) gen_server.erl:601: :gen_server.try_dispatch/4
(stdlib) gen_server.erl:667: :gen_server.handle_msg/5
(stdlib) proc_lib.erl:247: :proc_lib.init_p_do_apply/3
Last message: :check_if_time_to_update
State: []

hi, I am using tzdata 0.5.11. Does anyone know how to solve it?

@Ljzn Try version 0.5.12. If you don't have internet access or if the IANA servers are unresponsive that could happen. By default it will try again the following day. If an error happens and it cannot check for new versions, the existing version of the data will be used until it successfully finds and downloads a new version.


Stop using short commit hashes

We found a bug on a test project that was using git-queue.

The bug was caused because git-queue assumes a fix short commit hash (7 chars).
As I discussed with @da2ce7 we should not rely on short hashes. Short hashes should be only used for the user interface because they are easier to handle by humans.
It seems neither the SimpleGit nor the git commit command itself has an option to return the full commit hash.
This is the object SimpleGit returns after committing:
{
  author: {
    email<EMAIL_ADDRESS>    name: 'NautilusCyberneering[bot]'
  },
  branch: 'issue-22-job-commit-not-created',
  commit: 'df462c39',
  root: false,
  summary: { changes: 0, insertions: 0, deletions: 0 }
}

As you can see in that case the commit hash contains 8 chars. I asked them if there is an option to get the full hash. But we are pretty sure there is not because it seems git commit command does not have it either.
It seems the only way to get the full hash is with some "plumbing" git commands. I've already described here how to use git commit-tree to create the commit.
I guess we can use something similar in our SimplyGit wrapper.

Please see comment: https://github.com/steveukx/git-js/issues/782#issuecomment-1093301761


Update pyproject.toml and dependencies

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@23d2909). Click here to learn what that means.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@          Coverage Diff           @@
##             main     #33   +/-   ##
======================================
  Coverage        ?   0.00%
======================================
  Files           ?      17
  Lines           ?     436
  Branches        ?       0
======================================
  Hits            ?       0
  Misses          ?     436
  Partials        ?       0

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Bring back the database_driver parameter

we often develop applications multi database aware, one developer is using mysql the other postgresql, somtimes we even try to support sqlite.
the database driver is something environment specific, so it sould be in the parameters file. also with the new direction in which the new symfony versions are moving, specifically i mean the new dotenv component and the remove of the parameters.yaml file in symfony4, we should rething the decision of hardcoding the database driver.
it allows better testing if in my CI ifi can set the database_driver in the parameters.yaml file or via an environment variable.
currently i have to edit every fresh symfony-standard installation and move the database_driver in the parameters.yml file. also having the environment variable support since sf2.7 and enhaced in sf3.2 makes me belive in readding this back in the parameters.yml
personal opinion: even if mysql maybe is the most common database, we shouldn't promote such an inconsistent database. see http://www.cybertec.at/why-favor-postgresql-over-mariadb-mysql/ (i am using postgresql in my php projects since 1999.
related to: https://github.com/symfony/symfony-standard/pull/867 https://github.com/symfony/symfony-standard/pull/722#issuecomment-58039662 https://github.com/symfony/symfony-standard/pull/776

maybe somthing for a different issue but does anything speak against having database_path also in the parameters.yml and autohandle it in the code (like if null use the other dbal parameters)?
doctrine:
    dbal:
        driver: '%database_driver%'
        host: '%database_host%'
        port: '%database_port%'
        dbname: '%database_name%'
        user: '%database_user%'
        password: '%database_password%'
        charset: UTF8
        path: '%database_path%'

👎 Having different RDBMS per environment/user is an edge-case that we do not need to support out-of-the-box.

I agree with @Tobion. If you have the need for this in your project, you can easily adapt the config files while bootstrapping a new project.

I agree with @Tobion and @xabbuh. Nothing prevents you from doing the changes for your project, but that would not make sense to do it here as the vast majority of the projects out there don't have the same need.


ctrl+click opens the file in same editor group even if file is open in different editor group

Type: Feature Request
In MPLAB X IDE you can create split screen view with multiple editor groups, this can be done in VSC as well.
But when I ctrl+click on function call, variable, macro etc. VSC opens the file, with the definition, in same editor group even if the file is already opened in different editor group. In case of MPLAB X IDE it identifies that the file with the definition is already opened in the different editor group and shows you the definition there.
VS Code version: Code 1.74.3 (97dec172d3256f8ca4bfb2143f3f76b503ca0534, 2023-01-09T16:59:02.252Z)
OS version: Windows_NT x64 10.0.19044
Modes:
Sandboxed: No

See workbench.editor.revealIfOpen setting.

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!


v3 Unable to use few characters in Cycle

Hi!
Thanks a lot for your hard work! I was trying to implement this using the following template string but it is not working. I tried with other possible combinations and it was working fine. This is the following template string -
tmpl := `{{ string . "filePath" }} {{ bar . "<" "-" (cycle . "-" "\" "|" "/" ) "." ">"}} {{speed . }} {{percent .}} {{ rtime . " --%s remaining-- "}} {{ etime . " --%s elapsed-- "}}`

If I use the above template, it gets reset to default. All the elements after the cycle are ignored.
I am thinking it could be because there are some special characters?
-Pranav

Thank you!
Try to replace "\" to "\\" in cycle arguments. Looks like it's escaping in go templates.

@cheggaaa - Yep, that worked! I put in the double slashes and it worked and looking really nice! Closing this issue. Thanks!


[Bug] Content-Length and Transfer-Encoding headers break response

When response has both Content-Length and Transfer-Encoding: chunked headers, Bruno fails to show response.
Steps to reproduce

Call endpoint that returns both Content-Length and Transfer-Encoding: chunked headers.

Expected behavior

Show response body
Warn user about conflicting headers received

Actual behavior

No response body, error is shown:

Error invoking remote method 'send-http-request': Error: Parse Error: Content-Length can't be present with Transfer-Encoding

Justificaiton
Standard requires us to ignore Content-Length in case of non-identity Transfer-Encoding, like "chunked", "compress", "deflate", or "gzip":

"Messages MUST NOT include both a Content-Length header field and a non-identity transfer-coding. If the message does include a non-identity transfer-coding, the Content-Length MUST be ignored." (RFC 2616, Section 4.4, point 3)

(c) https://stackoverflow.com/a/3304186
UPD: new RFC removed the part about ignoring, though it would be great to warn user in this case, it might simplify troubleshooting.

This is the exact issue i am having with confluent rest api. I am unable to turn off Content-Length and therefore just get the error mentioned.
I was able to confirm this issue by using postman and turning off Content-Length in its hidden headers and my data was sent fine.
Is there any plans for this bug, possibly the option to view hidden headers and disable, as i am very impressed with bruno.


Update pydeck README

For #2929

Background

Change List

Coverage increased (+2.9%) to 82.756% when pulling 0250f93631355d53114f05efa67adb6af25117a2 on ajd/change-readme-pydeck into d7d413276047af3a4ccd642ae665100950b0aa82 on master.


ZAMG weather reporting "unknown" state

ZAMG weather is reporting an "unknown" state:

I am using Hass.io Supervisor Version 140 and HassOS 1.13.
Regards
Richard

The source (ZAMG) doesn't conditions.


kdeltachat: unstable-2021-05-16 -> unstable-2021-05-18

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 123575 at d4b4d149 run on x86_64-linux 1

2 packages built successfully:

kdeltachat
tests.trivial


No payload no marker

cli CoAP client always sets the payload marker, which may cause some CoAP server failed to parse the packet.
Also the current CoAP parser doesn't verify the payload marker. This PR,

Set payload marker when required in cli CoAP client.
Raise parse error when payload marker present but without payload.

Codecov Report

Merging #1645 into master will increase coverage by 0.05%.
The diff coverage is 22.22%.

@@            Coverage Diff             @@
##           master    #1645      +/-   ##
==========================================
+ Coverage   66.93%   66.99%   +0.05%
==========================================
  Files         166      166
  Lines       21102    21107       +5
  Branches     2588     2590       +2
==========================================
+ Hits        14124    14140      +16
- Misses       6062     6065       +3
+ Partials      916      902      -14

Impacted Files
Coverage Δ

src/cli/cli_coap.cpp
0% <0%> (ø)
:arrow_up:

src/core/coap/coap_header.cpp
61.35% <100%> (+0.37%)
:arrow_up:

src/core/net/udp6.cpp
82.85% <0%> (-1.91%)
:arrow_down:

src/core/thread/topology.hpp
89.02% <0%> (-1.22%)
:arrow_down:

src/ncp/ncp_base.cpp
28.8% <0%> (-0.78%)
:arrow_down:

src/core/thread/mle_router.cpp
77.52% <0%> (-0.54%)
:arrow_down:

src/core/net/ip6.cpp
54% <0%> (-0.24%)
:arrow_down:

src/core/thread/mle.cpp
81.7% <0%> (+0.14%)
:arrow_up:

examples/platforms/posix/radio.c
78.42% <0%> (+0.41%)
:arrow_up:

src/core/thread/lowpan.cpp
93.9% <0%> (+1.43%)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

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


Why is pulling layer 3a32959397f1 for in2ex/jenkins soooooo slow?

The image in2ex/jenkins:1.642.1.0 (c9ee245fe79b) consist of 20-30 layers. The pull is very good and fast, except for the layer 3a32959397f1. It is by far not the biggest layer. Even after restarting the pull this layer stays to lack or what ever?
Greetings

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Tried on different machines. Mac OS with docker-machine and virtualbox and a nativ ubuntu 14.04....

docker version:
Client:
 Version:      1.9.1
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   a34a1d5
 Built:        Fri Nov 20 13:12:04 UTC 2015
 OS/Arch:      linux/amd64

Server:
 Version:      1.9.1
 API version:  1.21
 Go version:   go1.4.2
 Git commit:   a34a1d5
 Built:        Fri Nov 20 13:12:04 UTC 2015
 OS/Arch:      linux/amd64

docker info:
Containers: 0
Images: 150
Server Version: 1.9.1
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 150
 Dirperm1 Supported: false
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.13.0-77-generic
Operating System: Ubuntu 14.04.3 LTS
CPUs: 8
Total Memory: 15.55 GiB
Name: jenkins
ID: 24Q3:UB34:53NO:EEBP:WAHS:5VMN:YVGF:NJQE:73M5:HVJ5:QV6C:XS3I
Username: dzwicker
Registry: https://index.docker.io/v1/

Could you open a ticket through<EMAIL_ADDRESS>This issue tracker is for tracking bugs in docker; we don't have access to logs in Docker Hub (and it looks like the image you're describing is a private repository), so cannot help you here.


[PTRun][Program]Avoid loops due to symlinks

Summary of the Pull Request
What is this about:
Some types of folder shortcuts cause infinite loops in the Program plugin when searching for programs.
A good example of this are the shortcuts created inside node_modules by npm (Node.js' default package manager).
What is include in the PR:
When recursively looking for programs, don't navigate through directories with the Reparse Point attribute set. This will make it so we don't follow symbolic links, directory junctions or mount points.
How does someone test / validate:
This is not easy to replicate at all. These instructions can be used but still the project will be to small to cause issues before running into max_path_length and exiting. I'm mostly looking for a code check.
Quality Checklist

[x] Linked issue: #14483
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[x] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Do we actually want this in 51?

Do we actually want this in 51?

@crutkas, I was thinking .53. Do you want this cherry-picked for .51?

Yeah, let’s wait for 53. Just seems like we see this once every release it feels like


Recent Events - FFH Collation

Is your feature request related to a problem? Please describe.
Due to how the FFH publish their event results, it is currently bloating the recent events page.
Describe the solution you'd like
Unsure, personally I'd implement something on the FE side of things as it would have a much lesser impact on the codebase.
Describe alternatives you've considered
Collating events at the point of creation however it wouldn't give us the transparency with event IDs marrying up between our mirror and the root database.

Closed with the recent FFH rebuild


Initial draft to short circuit range queries

Description
[Describe what this change achieves]
Related Issues
Resolves #11251 #9541 #13566

Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] API changes companion pull request created.
[ ] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[ ] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@jainankitk @msfroh @reta would love some feedback

@jainankitk @msfroh @reta would love some feedback

Apologies @harshavamsi , will take a look once the release is cleared out (later this week), thank you

@harshavamsi Could you help me understand the definition and what all falls under ApproximateableQuery?
Approximation given me a perception that we are trying to approximate something and usually approximation is on a number. So is the intent behind ApproximatebleQuery is to approximate search results i.e. search results are not precise but approximate?
Going through the implementation of ApproximatePointRangeQuery its trying to early terminate the query based on the size, which I believe is not approximate but accurate. So my concern here is around nomenclature and confusion it might cause to developers.

@harshavamsi Could you help me understand the definition and what all falls under ApproximateableQuery?
Approximation given me a perception that we are trying to approximate something and usually approximation is on a number. So is the intent behind ApproximateableQuery is to approximate search results i.e. search results are not precise but approximate?
Going through the implementation of ApproximatePointRangeQuery its trying to early terminate the query based on the size, which I believe is not approximate but accurate. So my concern here is around nomenclature and confusion it might cause to developers.

@rishabhmaurya thanks for your comments, in the present state of approximateable queries we intend to give users approximate results instead of accurate results. Take for example, in this case we return an arbitrary set of 10,000 hits by scoring only 10,000 documents instead of all documents in the segment. In my opinion this is some form for approximation.
I do agree that we do not fully leverage the use of the term and are returning accurate 10k hits, but I introduced the framework here only for the purposes of adding more use cases later.

@reta @jainankitk @rishabhmaurya can you take a look?

@reta @jainankitk @rishabhmaurya can you take a look?

My apologies @harshavamsi , have difficulties find the time to look at it, I will try to go over the change on Friday (unless some folks would look before), thank you

in the present state of approximateable queries we intend to give users approximate results instead of accurate results. Take for example, in this case we return an arbitrary set of 10,000 hits by scoring only 10,000 documents instead of all documents in the segment. In my opinion this is some form for approximation.

@harshavamsi thanks for clarification. I still think its accurate result set and we are returning early based on partial scoring to speed things up. So is ApproximateQuery always approximating just score to speed things up? If that's true, should we call them ApproximateScoreQuery just like we have ConstantScoreQuery?

I'm wondering if we should gate this with a feature flag / experimental so it has time to bake?

@harshavamsi are there any benchmark number which you can share to understand the impact?
Also, can we run appropriate labels to run the benchmark - https://github.com/opensearch-project/OpenSearch/pull/14829

@harshavamsi are there any benchmark number which you can share to understand the impact? Also, can we run appropriate labels to run the benchmark - #14829

my change:
|                                        50th percentile latency |  range |     5.83704 |     ms |
|                                        90th percentile latency |  range |     6.73326 |     ms |
|                                        99th percentile latency |  range |     7.49136 |     ms |
|                                       100th percentile latency |  range |     7.51334 |     ms |
|                                   50th percentile service time |  range |     4.49783 |     ms |
|                                   90th percentile service time |  range |     5.49778 |     ms |
|                                   99th percentile service time |  range |     5.75205 |     ms |
|                                  100th percentile service time |  range |     6.01743 |     ms |
|                                                     error rate |  range |           0 |      % |

main branch:
|                                        50th percentile latency |  range |     208.131 |     ms |
|                                        90th percentile latency |  range |     231.806 |     ms |
|                                        99th percentile latency |  range |     244.009 |     ms |
|                                       100th percentile latency |  range |     247.398 |     ms |
|                                   50th percentile service time |  range |     206.753 |     ms |
|                                   90th percentile service time |  range |     230.711 |     ms |
|                                   99th percentile service time |  range |     242.908 |     ms |
|                                  100th percentile service time |  range |     246.346 |     ms |
|                                                     error rate |  range |           0 |      % |

@msfroh could you review once?

{"run-benchmark-test": "id_5"}

Benchmark Baseline Comparison Results

keyword-in-range,  query-string-on-message-filtered and query-string-on-message-filtered-sorted-num queries have 100% error rate. Failing with below error:
[ERROR] search_phase_execution_exception ({'error': {'root_cause': [{'type': 'null_pointer_exception', 'reason': 'Cannot invoke "org.opensearch.search.internal.SearchContext.query()" because "context" is null'}], 'type': 'search_phase_execution_exception', 'reason': 'all shards failed', 'phase': 'query', 'grouped': True, 'failed_shards': [{'shard': 0, 'index': 'big5', 'node': 'XKSDC8G3SxGC3n_6ApBqZw', 'reason': {'type': 'null_pointer_exception', 'reason': 'Cannot invoke "org.opensearch.search.internal.SearchContext.query()" because "context" is null'}}], 'caused_by': {'type': 'null_pointer_exception', 'reason': 'Cannot invoke "org.opensearch.search.internal.SearchContext.query()" because "context" is null', 'caused_by': {'type': 'null_pointer_exception', 'reason': 'Cannot invoke "org.opensearch.search.internal.SearchContext.query()" because "context" is null'}}}, 'status': 500})

@msfroh how does this look now?

{"run-benchmark-test": "id_5"}

hmm, we're not seeing any difference in benchmarks because this is behind a feature flag

I tried retrying the Mend Security Check, but that doesn't seem to be working.
As mentioned by @harshavamsi above, the bulk of the uncovered code is copy/pasted from PointRangeQuery.
Merging...

As mentioned by @harshavamsi above, the bulk of the uncovered code is copy/pasted from PointRangeQuery.
Merging...

Ideally we should avoid copy/pasting code, but I also don't see good way until PointRangeQuery is more extensible. Was hoping to do this sooner, finally got around to https://github.com/apache/lucene/pull/13711.

hmm, we're not seeing any difference in benchmarks because this is behind a feature flag

You can create a new config, see https://github.com/opensearch-project/OpenSearch/commit/f351c015ee41367f06d0038e7cbc52df6a8cd792 for concurrent search feature.
@harshavamsi


The "About this website" text seems to be quite outdated by now

At the bottom of http://www.minetest.net/community/, we have the "About this website" text, copied word-to-word from the old website.

It should probably not list those names, because that list basically just contained whoever had access to edit the previous site.

It should probably link to this repository instead of saying "please contact us on IRC or forums."

Maybe the whole thing doesn't even have to exist anymore?

Maybe the whole thing doesn't even have to exist anymore?

I agree, it can be removed.

Can we close?


Add event listener for document.fonts.ready

When fonts take time to render we make sure to wait until they are
all loaded. We also bump beta version to 1.

Did you test the potential issue mentioned at the linked discussion about the event possibly not firing if the fonts are cached? Or do we need to add further protection for that case?

Did you test the potential issue mentioned at the linked discussion about the event possibly not firing if the fonts are cached? Or do we need to add further protection for that case?

I didn't do any extensive caching testing, but what you referenced is not what we're using here. The comment you link to makes use of the dev tools API, while we're just awaiting the resolve of the document.fonts.ready promise which in my tests both resolved with and without a webfont to load.


🚀 Feature: Reduce Number Transit Journey Levels -- Phase 1

User Story
The initial translation of the Cube transit assignment scripts were faithfully translated to EMME. This included the translation of the fare matrix. Mechanically, this was done via an algorithm that considered the distance between services and their differences in fares, with a goal of creating fewer than the maximum number of 30 journey levels. Because EMME considers the fare in the path building stage, there is a severe runtime penalty for each journey level. An analysis of the algorithmically created journey levels revealed that a lot of them were working to create accurate representations of fares for low ridership services (that have complicated fare structures) -- see tree map below.

As a first step in assessing the impact of reducing the journey levels on runtime and outcomes, we will manually specify journey levels within tm2py, starting with @inrokevin's recommendations. Subsequent efforts will create a set of procedures so that the specifications can be derived from a set of GTFS files and guidance documents.
Progress:

[x] Sufficiently defined
[ ] Approach determined
[ ] Tests developed
[ ] User story satisfied
[ ] Doc strings
[ ] General documentation
[ ] Passing tests

Priority
High
Level of Effort
~80 hours
Resolution Ideas

Create a scheme in the model_config.yml file that specifies the desired journey levels. Per @inrokevin, this could look something like:

[transfer_fares]
    groups = [
        {group_id = 1, fare_systems = [1, 2, 3]},
        {group_id = 2, fare_systems = [4, 5, 6]},
        {group_id = 3, fare_systems = [7, 8, 9]},
    ]
    from_fares = [
        {group_id = 1, values=[0, 0.88, 1.22]}
        {group_id = 2, values=[-1.1, 0, 2.5]}
        {group_id = 3, values=[0, 0.44, 0]}
    ]

Project
MTC's ActivitySim/Emme Conversion, Task Order 5
Who should be involved?
Users: @lmz
Reviewers: @inrokevin, @gregerhardt
Risk
Yes. The outcomes from the assignment may be different due to changes in the fare assumptions. However, manually specifying a small number of journey levels should make the code simpler and more robust. Right now, the algorithm can return more than the maximum number of journey levels.
Tests
We want to run the following tests:

[ ] After updating the journey levels, do a congested assignment for the morning commute period using the observed demand from the on-board survey. The key performance measures are: runtime (expected to be reduced roughly by the ratio of the previous number of journey levels and the proposed number of journey levels) and boardings by operator (expected to be highly similar to the previous assignment of the on-board survey demand with the full number of journey levels.

I recommend being as aggressive as we reasonably can in setting these.  The tree map shows 21 journey levels.  The initial recommendations included 8 journey levels:

BART
Caltrain
Muni + VTA + AC Transit + All other Local Bus — use ridership-weighted mean flat fare across Muni, VTA, AC Transit
Muni + VTA LRT — use mean flat fare, perhaps weighted by boardings
Ferries — all, using a mean per mile fare
Express buses — all, using a mean per mile fare, perhaps weighted by boardings
ACE
Capitol Corridor

A few possibilities:

Can we combine Caltrain, ACE and Capitol Corridor into a single commuter mode journey level?  I'm not sure they really compete or have many transfers. ACE and Capitol Corridor have substantial IX trips anyway, so we're not really going to get them right.
Do we need to separate out Muni + VTA light rail from buses?  Don't they charge the same fares?

Those two changes could potentially get us from 8 down to 5 journey levels.

And one more question: Does EMME include the concept of fare links?  If so, can we use them to customize as we need?  For example, we could merge Ferries with express buses, but have an additional $10 fare link at the dock.

@gregerhardt
I was thinking we start even more aggressive. One thing to note is that the journey levels are states. So, if a system like Caltrain does not give any discounts after riding Caltrain, then it does not need a separate journey level. Said another way, being in the state of having already boarded Caltrain does not change how you interact with the other services. We can accurately (well, after transforming it to a distance-based approximation) represent the Caltrain (and Ferry, and BART) fares without them being separate journey levels. The only thing we get wrong is the transfer discount when moving to another operator. I did not fully understand this -- probably still do not, thankfully we have @inrokevin to help us -- when I wrote the original recommendation for eight.
I think to start we go down to 2 journey levels (plus the others we'll need for park and ride and to prevent walking all the way), as follows:

Base state -- have yet to board a transit service
Already boarded Muni state

This is a minimum configuration in which we are acknowledging the free transfers on Muni (which is important), but ignoring all other transfer discounts. We can then assign the on-board demand with this configuration and see what we need to add, e.g., the discount when going from AC Transbay to AC Local, which @e-lo said was a problem in a long ago SFCTA calibration.
@inrokevin: what do you think about this approach?

You have the right approach Dave. Small clarification (probably what you meant but just to be clear) the base state would be not yet boarded Muni, i.e. would include not having boarded at all and having boarded any other services. So this proposal would no longer differentiate initial and transfer boardings.
A 3-level approach (1. not boarded anything, 2. boarded muni, 3. boarded anything else) is something to consider if transfer behavior is really off. This would also allow preventing "walk all the way" trips at the same time.
Any additional operator-to-operator fare discounts should be considered in the context of the number of transfers. A review of operator-to-operator transfers and important fare discounts might reveal some reasonable "over-discount" options. SO, for example, for the AC Transbay / Local case: if few transfers from other operators occur we could give the "from AC Transbay" discount for all transfer boardings to AC Local, knowing that fares for some passengers will be under-represented.
Of course, I would recommend trying a simpler version first and seeing if there are problems and only adding complexity as necessary.
Fare links and virtual stops can be used to represent special transfer combinations. This can be much more efficient for very local and well defined cases, e.g. special transfers between ferries, or ferries to certain bus services. However, the network visualization and maintenance can be a little more difficult.

Thanks @inrokevin. Can you explain the "additional operator-to-operator fare discounts" a bit more? Is this another journey level or something else?

By  "additional operator-to-operator fare discounts" I am referring to the real world cases, for example the AC Transbay & Local transfer fare discount. Before we decide to allocate a journey level to representing this case, we should consider that there are options for representing the fares without journey levels.

Have no discount to AC Local, fares when transferring from AC Transbay will be too high. (This seems to be the default, but it might not make sense if the AC Transbay to Local is actually the most important operator-to-operator transfer)
Everyone gets a discount, all transfer boardings to AC Local get the AC Transbay discount, and fares when boarding from other services will be too low.
Split the difference: everyone gets half discount.
The fare can vary over the network, so nodes which have both transbay and local get a discount, all other nodes have the full fare. This can lead to some short illogical walks to a cheaper boarding stop, but may actually get the least number of trips wrong.

My main point was that in order to prioritize, we should consider the number of passengers affected by analyzing the operator-to-operator transfers.

Initial test results are in. The test includes three journey levels, as follows:

Not yet boarded transit
Boarded SF MTA
Boarded all other transit providers

Headline of run-time comparison: for the morning commute period congested assignment of the on-board survey demand:

Baseline assignment runtime: 12.47 hours (3 iterations)
Test assignment runtime: 3.45 hours (3 iterations)
Baseline skims runtime: 2.32 hours
Test skims runtime: 1.32 hours

See the below spreadsheet for additional details, including the compute spec.
observed-transit-assign-journey-level-runtime.xlsx
The two assignments give highly similar results (for the morning commute). See this Tableau workbook. The key differences are as follows:

Decline in Caltrain boardings from 23,200 to 20,300
Decline in Ferry boardings from 5,100 to 2,400

Recommend moving forward with this configuration for next round of testing and to discuss the user experience on our July 3 call. Additional calibration to the address Caltrain and Ferry can occur if the pattern replicates in the 2023 base year.
@lmz, @vivverma9, @gregerhardt, @FlaviaTsang, @inrokevin

@vivverma9
Here's what I'm thinking in terms of the user experience:
https://github.com/BayAreaMetro/travel-model-two-config/blob/3c11035556f25d2407faade3804468dc64b3a53b/2015-tm22-dev-sprint-03/model_config.toml#L865-L869

@vivverma9, @AshishKuls
Updates per our conversation
https://github.com/BayAreaMetro/travel-model-two-config/blob/fa7d224dd4151931e5818315e664a94908d2ee34/2015-tm22-dev-sprint-03/model_config.toml#L865-L873

Some additional runtime statistics for a test with four journey levels for morning commute assignments:

Baseline assignment runtime: 12 hours, 28 minutes
Test assignment with three journey levels: 3 hours, 26 minutes
Test assignment with four journey levels: 4 hours, 12 minutes

Excel: observed-transit-assign-journey-level-runtime-update.xlsx

Here are Tableau summaries showing the outcomes of the above two tests, with three and four journey levels respectively.
https://mtcdrive.box.com/s/4wd1vvt9vpn3wjk5hdreu2tsvh9ehjf5


Fix cdLog typos

When I try to run:
make run

I get errors:
# github.com/openshift/hive/pkg/controller/clusterdeployment
pkg/controller/clusterdeployment/clusterdeployment_controller.go:241: Entry.Errorf call has arguments but no formatting directives
pkg/controller/clusterdeployment/clusterdeployment_controller.go:245: Entry.Errorf call has arguments but no formatting directives
make: *** [Makefile:70: vet] Error 2

This PR fixes that.

Thanks!
/lgtm


Created map markers, Adjusted event view, added documentation to home…

… view

Additional TO-DOs:
Update FirebaseDAO to extract event information
Store UofT buildings along with their latitude and longitude to use in HomeView
Implement the feature to add multiple markers to the map depending on the user's event creation data (update HomeController, create event, firebase)
Use checkstyle and correct all styling errors
Complete all to-dos

Good job following clean architecture, try adding more tests to have more code coverage


Task3 Task4

SourceCode/2024/May 2024/Rajendra/Task2/project.json
main.xaml

Input Dialog display name is duplicated, consider renaming or combining similar activities.
Activity names are too generic and don't give a clear indication of what they do; rename them for better understanding.

Score: 4/5
Please note that the main issues with this file are:

Duplicate activity names
Inadequate naming conventions

These issues can be addressed by renaming activities to provide clearer indications of their purpose.

SourceCode/2024/May 2024/Rajendra/Task2/project.json
Main.xaml

The Input Dialog display name is repeated multiple times, which exceeds the allowed threshold of 1. Please consider renaming these to something more specific.
Default names for activities like Assign, Log Message, and Input Dialog are too generic. It's better to add a more descriptive title to make it clear what each activity does.

Score: 4/5
Please update the display names and rename activities with more descriptive titles. Also, check if an Automation Hub URL is defined in your project settings.

@rajendrayarlagaddausa333
Task -3
Code verified.
Task -4
Code verified.

@rajendrayarlagaddausa333
Task -2
Code verified.
Task -5
Code verified.
Task -6
Code verified.


Is .funcignore actually respected?

Just a heads up, as of ~9/2022, it appears that files and directories listed in .funcignore aren't respected.
https://github.com/Azure/functions-action/issues/36#issuecomment-1241928855
I just tested as well and I was not able to .funcignore the README.md file in my repo.
Maybe just a heads up that the line of this doc that includes ".funcignore" isn't quite accurate. Maybe point users to the comment above showing them how to ignore files is the DevOps pipeline(?)
Thank you
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: b2646ad4-34b7-fc5e-dcc9-d53a061df394
Version Independent ID: bbff4fda-24e4-2b62-24b9-94cda5f53bed
Content: Work with Azure Functions Core Tools
Content Source: articles/azure-functions/functions-run-local.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Thank you for your feedback! We will review and update as appropriate.

Was this addressed? I'm still having issues when running func scripts.

And .funcignore:
*.js.map
*.ts
.git*
.vscode
tsconfig*
local.settings.json
test
coverage
*jest*
*.md
Dockerfile
.dockerignore
.DS_Store
.env
.funcignore
.npmrc
package-lock.json
node_modules/@types/
node_modules/azure-functions-core-tools/
node_modules/typescript/
**/*.bin


Pin setup-go action

Changes
The the setup go action in the CodeQL CI job for security.
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[x] Has Tests included if any functionality added or changed
[x] pre-commit Passed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

/kind misc

It looks like it's test: TestStepTimeout that fails all the time. I will make a PR to disable that test for now, because this is costing us a large number of retests.

/lgtm


failed to open config file config.yml Warning

Thank You again for the great job creating wg-portal.
I’m running wg-portal on ubuntu 20.04, in a docker stack with several apps installed with docker compose. All containers are being update automatically with Watchtower (daily scheduled at 4:00 am).
Found two new issues:
First: Although wg-portal is working, despite of the issues #40, #41 and #42, the client configuration download button is not working; to download the client .conf file, I had to press the right mouse button and choose, “open link in a new tab”.
Second: Wg-portal log shows a warning after Watchtower updated the container last night:
INFO[2021-09-13 04:00:29] starting WireGuard Portal Server [master-9147fe3]... ,
INFO[2021-09-13 04:00:29] real working directory: /app ,
INFO[2021-09-13 04:00:29] current working directory: . ,
### WARN[2021-09-13 04:00:29] unable to load config.yml file: failed to open config file config.yml: open config.yml: no such file or directory, using default configuration... ,
INFO[2021-09-13 04:00:29] setup of service completed! ,
INFO[2021-09-13 04:00:29] starting web service on :8123 ,
Could it be a folder permissions issue? My docker-compose is setting WG_CONFIG_PATH to /etc/wireguard wich is a root acess folder. No config.yml is beyng created on that folder. Do I need to change this path to another folder where wg-portal as write permissions.
Sorry for the basic beginner questions.

Regarding the download issue, i can not reproduce this on chrome, which browser are you using?
And the warning can be ignored (I am gonna remove it in a future release) as you configure wg-portal via environment variables and not via config file.

Thank you for your answer.
I’m using Edge (Chromium) on my PC, and Chrome for android on my smartphone
Experiencing the issue on both.
Initially I had no problem downloading the .conf file.

Ok. Found the answer. I was using wg-portal inside a iframe tab from Organizrr. If I i open it directly on the browser, download button works as expected. Is there a way to make work inside the i-frame?

Can you check the chrome developer tools (ctrl + i) and see if there are any errors or warnings after you clicked the download button?

No right now, but i will check it later and repport back.

Ok. I believe this is the error:
"Download is disallowed. The frame initiating or instantiating the download is sandboxed, but the flag ‘allow-downloads’ is not set. See https://www.chromestatus.com/feature/5706745674465280 for more details."

There is nothing wg-portal can do. You have to change the iframe parameters, see https://html.spec.whatwg.org/multipage/iframe-embed-object.html#attr-iframe-sandbox


Upgrading SettingsFileGenerator workbook to V3.1+

The upgrade process from the SettingsFileGenerator.xls included in the Framework prior to V3.1 to the new binary or SpreadsheetML (XML) format supported in V3.1+ needs to be documented.

The recommended upgrade path is to copy and paste the settings data from the old workbook into the new SettingsFileGenerator.xml included with the V3.1+ Framework.  The new workbook does not include any macros, and has slightly different formatting.

If the old workbook is simply saved as XML, the XML may include formatting and print areas that negatively affect the settings exporter.  One such issue was reported by Jun Wang, as follows:

When I save current SettingsFileGenerator.xls to SettingsFileGenerator.xml, maybe because of cell merging or something else, some cells inside xml got ss:Index attribute.

Look at this example,

FailedPIMtoFOPUpdateSupportTeamEmail
<EMAIL_ADDRESS>

3rd cell in this row got ss:Index = "4", that means this will be 4th column instead of 3rd. The problem was some Stage setting column was put into TEST setting file!

To support this Index attribute, I did modify the code SpreadsheetMlSettingsFileReader.cs, and add following code from Line 85.

   if (cellsIterator.Current.GetAttribute("Index", cellsIterator.Current.GetNamespace("ss")) != string.Empty)
   {
    columnIndex = System.Convert.ToInt32(cellsIterator.Current.GetAttribute("Index", cellsIterator.Current.GetNamespace("ss"))) - 1;
}

This code will check if there is ss:Index attribute, and if yes, replace default columnIndex, this will make sure put correct column to correspond setting files.
This work item was migrated from CodePlex
CodePlex work item ID: '3126'
Vote count: '1'

[daeva@4/22/2009]
I've also experienced this issue.  I guess the fix described above hasn't been released, yet.

[UnknownUser@1/22/2010]

[UnknownUser@2/21/2013]

[UnknownUser@5/16/2013]

Issue closed by tfabraham with comment
This is addressed here:http://biztalkdeployment.codeplex.com/wikipage?title=Frequently%20Asked%20Questions


Form for providing feedback on docs page always fails

This is a pretty minor one, but when I go to add comments/feedback in the in-page issue form (love it btw ♥) it usually fails the first time and then I have to click the link in the red bar (as shown in the photo) to submit issue. It often gets the right context for the issue, but can fail to copy over what I typed in in the box on the docs page so I have to re-create it there.

Thank you for reporting this issue @crazy4pi314. Can you please link to the page that you are having trouble reporting feedback on?

It happens on every page I have tried in the last few weeks (~3-4 pages), the screenshot from above was on microsoft.quantum.canon.splitarray. I know it also happened when I filed this issue too: https://github.com/microsoft/Quantum/issues/271#issue-527610486
I don't remember the other pages off the top of my head. I can reproduce it every time by just trying to use the form on the page, this screen shot is from chrome on mobile, on the page for CNOT.

Thank you @crazy4pi314. We have reproduced the issue in the reference documentation pages. (The problem does not occur in the conceptual doc pages)

@anpaz-msft, the current target for issues in the API docs is the github.com/microsoft/quantum repo. Is this where we want API docs issues to go? If so, can you check which app integrations are installed on that repo please? (I don't have access)

@anpaz-msft checking in if the questions from @natke are clarified.


Small fixes

Small fixes/enhancements

I'll add a few examples when I've time.
About using << for those other add_x methods. I don't think that's a great idea, I think it'll be confusing IMO. << is mostly used as a shortcut for appending things to a stream. I think it would be confusing when using it with multiple arguments.
sheet << [a,b,c] seems ok. sheet << :chartype, {option: true} doesn't seem right to me.
Now that I'm thinking about it, maybe I can rewrite << so you can pass rows like this:
sheet << 'column1', 'column2', 'column3'
That looks pretty clean to me.
I don't think you want to use options if you're using <<, but thats just how I would use it :)


Add javascript examples to operations.

This change is

Review status: 0 of 1 files reviewed at latest revision, 2 unresolved discussions, some commit checks failed.

src/ops/compare.ts, line 105 at r2 (raw file):
Previously, ManrajGrover (Manraj Singh) wrote…
This should be changed from dl to tf.

+1

src/ops/compare.ts, line 136 at r2 (raw file):
Previously, ManrajGrover (Manraj Singh) wrote…
Let's add a line between initialization and executors.
const b = tf.tensor1d([2, 2, 2]);

a.lessEqual(b).print();

+1 here and elsewhere

Comments from Reviewable


Fix timeout when creating model and matrix

When saving a model, the matrix generation is taking a long time (30x30 = 900 sampled points). I'm getting the following error from Node:
node: stack Error: Max API Message Request timed out. id: u15569491665
    at makeError (/Applications/Ableton Live 10 Suite.app/Contents/App-Resources/Max/Max.app/Contents/Resources/C74/packages/Node For Max/source/lib/api/index.js:9:14)
    at Timeout._onTimeout (/Applications/Ableton Live 10 Suite.app/Contents/App-Resources/Max/Max.app/Contents/Resources/C74/packages/Node For Max/source/lib/api/index.js:178:12)
    at listOnTimeout (internal/timers.js:531:17)
    at processTimers (internal/timers.js:475:7)

I should take a look into asynchronous call to the function that is generating and saving the matrix.

As of now, two sets of files are generated:

A set for the visualizer:

model_XXX_YYYmodel-matrix-LS.data
model_XXX_YYYmodel-matrix-vis.data

A model for the latent space  as a TF model:

model.json
weights.bin

With the current version, I can do one or the other, not both at the same time for the timeout issue. I should look into async calls.

The issue seems to have been fixed in b6c837a, but still testing.


[Task] Improve test case coverage

Description
Improve test case coverage for shenyu-client, shenyu-admin, shenyu-bootstrap, shenyu-sync, shenyu-integrated-test, please commit your pull request about this issue or at me.
Task List
1.shenyu-admin
2.shenyu-bootstrap
3.shenyu-sync
4.shenyu-client
Result
Coverage： 80%

Hello, I want to participate in the Shenyu community. Can you assign me the task of the shenyu-sync?

Hello, I want to participate in the Shenyu community. Can you assign me the task of the shenyu-admin?

Hello, I want to participate in the Shenyu community. Can you assign me the task of the shenyu-admin?

Hello, I want to participate in the Shenyu community. Can you assign me the task of the shenyu-client?

@moremind
Hello, I want to participate in the Shenyu community. Can you assign me the task of the shenyu-sync?


about the base model result

plan_L2_1s:0.3348474009348846
plan_L2_2s:0.6005098198601633
plan_L2_3s:0.9474086193724374
plan_obj_col_1s:0.0
plan_obj_col_2s:0.0
plan_obj_col_3s:3.255844337443258e-05
plan_obj_box_col_1s:0.0019535065442469234
plan_obj_box_col_2s:0.002979097479976558
plan_obj_box_col_3s:0.006088428835334152
projects/configs/VAD/VAD_base_e2e.py
L2 is better， but collasion is worse than the paper‘s. Why?

loss_plan_col=dict(type='PlanCollisionLoss', loss_weight=1.0),
can I make the above "loss_weight" bigger?

Did you train the model yourself, or use the pretrained model? If using the pretrained one, I think VAD_base_stage_2.py should be used instead of VAD_base_e2e.py
Additionally, I think that they use another image normalization config in the paper and pretrained model. See https://github.com/hustvl/VAD/issues/18#issuecomment-1711608585 and https://github.com/hustvl/VAD/issues/9#issuecomment-1696708154

I train the model from resnet50-19c8e357.pth by the configs of "VAD_base_e2e.py“.  Is the "loss_weight" not important?

I was in a similar situation
-------------- Motion Prediction --------------
EPA_car: 0.6096603478939834
EPA_pedestrian: 0.3235923685435086
ADE_car: 0.8396922945976257
ADE_pedestrian: 0.8586953282356262
FDE_car: 1.1747483015060425
FDE_pedestrian: 1.189048409461975
MR_car: 0.12827822120866592
MR_pedestrian: 0.16290983606557377
-------------- Planning --------------
gt_car:4.503418636452432
gt_pedestrian:2.099042781793319
cnt_ade_car:3.6200429771439735
cnt_ade_pedestrian:1.077358859152178
cnt_fde_car:3.4264504786091035
cnt_fde_pedestrian:0.9533111935924985
hit_car:2.9869115061535454
hit_pedestrian:0.7980074233248682
fp_car:0.40378980269583903
fp_pedestrian:0.20296932994725533
ADE_car:3.1434736251831055
ADE_pedestrian:0.9509602189064026
FDE_car:4.025217056274414
FDE_pedestrian:1.1335331201553345
MR_car:0.43953897245555773
MR_pedestrian:0.1553037702676304
plan_L2_1s:0.30277927277533684
plan_L2_2s:0.5726086170295752
plan_L2_3s:0.949615497207263
plan_obj_col_1s:0.0
plan_obj_col_2s:0.0
plan_obj_col_3s:3.255844337443258e-05
plan_obj_box_col_1s:0.0034186364524321157
plan_obj_box_col_2s:0.005176792342254347
plan_obj_box_col_3s:0.011851273242745881
fut_valid_flag:1.0
projects//configs/VAD/VAD_tiny_e2e.py

Hello, I followed the docs instructions to download resnet50-19c8e357.pth, but when the code runs it redownloads resnet50-0676ba61.pth, have you encountered this?

I would like to ask why I can not use my own training pth file to test? I still get errors when I use my own pth:result_dict['ADE_'+cls] = all_metric_dict['ADE_'+cls] / all_metric_dict['cnt_ade_'+cls]
ZeroDivisionError: float division by zero

您是自己训练模型，还是使用预训练模型？如果使用预训练模型，我认为VAD_base_stage_2.py应该使用而不是VAD_base_e2e.py 此外，我认为他们在论文和预训练模型中使用了另一个图像规范化配置。请参阅#18（评论）和#9（评论）
Hello, I would like to ask what is the difference between the pth trained by myself and the pth provided by the paper?


cmd/compile: unused variable escapes to the heap

What version of Go are you using (go version)?
go version go1.10.3 windows/amd64
What operating system and processor architecture are you using (go env)?
set GOARCH=amd64
set GOBIN=
set GOCACHE=...
set GOEXE=.exe
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=...
set GORACE=
set GOROOT=C:\Go
set GOTMPDIR=
set GOTOOLDIR=C:\Go\pkg\tool\windows_amd64
set GCCGO=gccgo
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=...
What did you do?
I compiled the following program using `go build -gcflags="-m -m" to check if any of the variables escaped to the heap.
https://play.golang.org/p/iZCz8oXRoi__h
What did you expect to see?
I expected only y to escape.
What did you see instead?
Instead, both x and y escaped to the heap.
In my opinion, only y should escape, because there is no way for x to leave the scope of this function. Alternatively, the assignment of the address of x to k should be entirely removed.

CC @dr2chase

The algorithm we use is not that clever and just tracks flow by variable name.
If/when we get around to moving escape analysis into the SSA phase (there are several difficult prerequisites in the way of this happening -- and these are planned, e.g., moving call expansion into SSA) this will be fixed as a side-effect.
Since there's an easy workaround and big blockers for the proper fix, I would not commit to fixing this in 1.12.

One possible benefit, from talking to a guy at GopherCon (@cixel) who's interested in doing instrumentation for taint detection/analysis, is doing dataflow-informed instrumentation.  Idea is that we'd transform to SSA, do whatever instrumentation we want (that might, in some cases, have effects on escape analysis -- and might also have effects on stack frames), then run escape analysis etc.
This is still a little hand-wavy.  He's looking the code for tsan and msan, and modifying things at the AST level, but having the ability to do flow analysis would be helpful (I wonder if it might not also help tsan and msan)

Dup of #18300; TL;DR: escape analysis is not path sensitive.
See also #31501.


[ci:component:github.com/gardener/autoscaler:v1.24.0->v1.25.0]

Release Notes:
Synced the changes till v1.25.0 of upstream autoscaler

/cc @oliver-goetz

@rishabh-11 would be raising a seperate PR for updating the charts, this PR should be closed at that time.
why 1.62 milestone ? we will need a hotfix for this in 1.61 right?

/close
as<EMAIL_ADDRESS>should be used only for K8s >= 1.25 Shoots

/milestone clear


Consider removing dead code from repo

As part of the tightening up of the interface done in https://github.com/noir-lang/aztec_backend/pull/117. I noticed that we don't actually use a decent number of the functions which we've written.

hash_to_field in blake2s.rs.
compress_many in pedersen.rs.

We also don't use construct_public_key in schnorr.rs however that's useful for testing whereas the other two are not.
We should consider removing these functions if this package is being restricted to serve only the ACVM interface.

We've removed so much code from here that github thinks this is a solidity project.


install script for AWS EC2 server

Amazon Web Service EC2 offers a free server for 1 year with a new subscription. This is an install script for the ot-node to try and run a node on a fresh Amazon Linux 2 server.

Great! Thank you!


Custom th/tdClass not taking effect

Issue Type (delete the irrelevant ones)

[x] Question

Specs
What version are you using?
2.16.5
Expected Behavior
I expected the properties I've set for the custom th/td class to take effect.
Actual Behavior
Nothing happened. No changes took place.

``
columns: [         {           label: 'Name',           field: 'display_name',           thClass: 'custom-th-class'         } ]
What am I performing wrongfully so that the changes don't take place?

Hi @ricardomiguelvieira,
looks like that should be working. Here's a working example - you can compare your code against it:
https://jsfiddle.net/aks9800/dc1L3pza/
oh it could be that you've scoped your style so it doesn't apply to any nested component. Try removing scoped or using /deep/ for it to affect the child components
closing.


Minor changes

EF Core 3.0 is a .NET Standard 2.1 but no all version of EF Core.
in-memory and SQL Server providers are not included in current version of ASP.NET Core
Change version 2.2.2 to 3.0.0
Some paragraphs include information specific to  ASP.NET Core 2.x .

I don't think there is value in adding explicit version everywhere. The page should be most relevant for latest version. And for previous version add a note section in the doc or simple line if small. If big enough then add section at the end of page with previous versions.
cc: @aspnet/efteam

As a user of EF Core,  I think than the documentation must be relevant for the specific version that I use not only the latest. Is possible to have a way to select doc's version just like ASP.NET Core documentation?

There is a lot of overhead with maintaining multiple versions of the docs. There are also only a few versions supported at one time. For example, versions 1.0, 1.1, and 2.0 are not supported and version 2.2 goes out of support in December. This only leaves versions 2.1 and 3.0 as really relevant.
We decided that it's more pragmatic to keep the docs focused on the latest version and call out version specific content in aside text.

@bricelam @smitpatel Was the decision here not to take this change as-is? Should we close it?

We need to update the page after 0s and 1s PR but this PR does not address those and cannot be merged as is.

@jahbenjah Thanks for the proposal, but we're not going to merge this as it's taking things in a direction we don't want to go.


bug: save draft as shipment

If you create a draft from the shipment form, and then try to submit the draft and make it into a shipment the status remains the same as shipment.

https://github.com/user-attachments/assets/0f8c1568-bcf5-4ae8-8c05-46c1a96f05ac
I tested this issue on local by pointing on dev backend and my observations are -
User is able to

Create a draft
Edit and submit the draft
Fetch the Registered Shipment and the status is getting updated to "Submitted"

A screen recorded video is attached.

Seems like you can close this issue then @ashukequinor


[dv,random_reset] Enhance handling of random resets

Declare the intention to issue a random reset in the near future in order to halt transaction generators. Without this there are occasionally failure for stress_all_with_rand_reset because of timing out waiting for no outstanding accesses, either because the generators have too many pending transactions, or because of a single cycle with no outstanding accesses followed by more accesses.
Fixes #21531

Let's merge this first and fix sram_ctrl afterwards, since it seems to be an exception.


Added glossary entries to Glossary page

Proposed changes

Describe your changes to communicate to the maintainers why we should accept this pull request.
If it fixes a bug or resolves a feature request, be sure to link to that issue.

Types of changes
Please put an x in the boxes related to your change.

[ ] Minor Issues and Typos
[x] Major Content Contribution
[ ] Others

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to reach out. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING GUIDELINES
[x] I have signed the CLA
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Related issues

Please leave the issue numbers or links related to this PR here.

Further comments
If this is a relatively large content contribution, kick off the discussion by explaining why you would suggest the content contribution, etc...

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

Closed due to CLA sign problem possibly caused by incorrect email configuration in local git repo


Merge live into master

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/API/NuGet-Protocols.md
:white_check_mark:Succeeded

docs/Guides/Create-UWP-Controls.md
:white_check_mark:Succeeded

docs/TOC.md
:white_check_mark:Succeeded

docs/Visual-Studio-Extensibility/NuGet-API-in-Visual-Studio.md
:white_check_mark:Succeeded

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


SA1505: OpeningCurlyBracketsMustNotBeFollowedByBlankLine

http://www.stylecop.com/docs/SA1505.html

Grabbing this one


[Feature] [Tracing] Add OTLP Tracing support as a Zipkin trace input

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
We are declaring to support OTLP trace format through OTLP collector and forward to Zipkin receiver.
This works, but an extra component seems unnecessary.
OTLP trace format is logically the same as the Zipkin trace format, so, we just need to add a OLTP trace handler and an in-memory converter to Zipkin receiver.
Use case
receiver-otel:
  selector: ${SW_OTEL_RECEIVER:default}
  default:
    enabledHandlers: ${SW_OTEL_RECEIVER_ENABLED_HANDLERS:"otlp-metrics,otlp-logs"}
    enabledOtelMetricsRules: ....

We should add otlp-trace as an option to enabledHandlers to support this.
@kezhenxu94 Any suggestion?
Related issues
No response
Are you willing to submit a pull request to implement this on your own?

[ ] Yes I am willing to submit a pull request on my own!

Code of Conduct

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

To be clear, this would not forward any OAL sources like the previous Zipkin receiver.

Not sure what you want, OpenTelemetry agents can send traces directly to Zipkin server
https://github.com/open-telemetry/opentelemetry-java/blob/3b1c6d9108f7e25afe5d0fdae830458e8c1a481b/sdk-extensions/autoconfigure/README.md#zipkin-exporter

Some agents could, I am thinking wider support only.

It sounds kind of out of skywalking’s scope to convert one third party format (OpenTelemetry) to another third party format (Zipkin)


Autocomplete popup is huge and doesn't scroll

When I type into the compose box something like :s, or @g, there are a lot of possible completions. That's fine; we should display a reasonable scrolling view of them.
Instead, as of 17.1.98 (and some recent versions before it), the autocomplete popup seems to extend way beyond the top of the screen, and doesn't scroll. This looks broken, and makes it impossible to select any but the last few options.

#2919 is one PR aiming to address this. I think #2944 may also be intended to do something related. (@jainkuniya , is that right?)

Yeah, the mentions issue here is quite annoying and makes it impossible to mention some users.

@armaanahluwalia reports an additional, fascinating wrinkle:

The autocomplete seems to stop at the bottom of the list... if i try to scroll up it doesnt seem to work unless i keep one finger on the screen and use another finger to scroll further up which reveals more emojis above which are inaccessible

I don't reproduce this on my device (17.1.98 on Android 9); IIRC Armaan uses iOS, so this may be OS-dependent. For me, it's just stuck at the bottom of the list and I'm unable to scroll at all, even by experimenting with multiple fingers.
This wrinkle isn't really an independent or additional problem relative to the issue described in the OP, but might be a helpful clue in resolving it.

Yes I’m on iOS 17.1.98. Let me know when fixed so I can retest.

Updating back here with discussion from #2919: fixing this properly will be tricky (details on that thread), and we'll come back and do that after we lay some groundwork, namely #3017 and the work that it in turn depends on.
In the meantime we'll do a quick fix, like a fixed max height. @jackrzhang is planning to do that.

This was fixed (back in October) by #3030. The height isn't always right, but it's never enormous, which is a big improvement.


Python Implementation for Object Detection

The python implementation of object detection can be achieved by using the DNN module of cv2 and the YOLO weights. It would be great to have that here. I have the code ready, may I generate a pull request for validation?

Hi @AsRaNi1, Please raise a  PR  and follow the instructions mentioned in the contributing guide. Thanks!


svelte 4 support

Tested on svelte 4 and it seems to work perfectly.

Based on the previous version pattern, might want to switch the devDependencies entry to ^4.0.0 and the package version to 4.0.0. Is that accurate, @jacobmischka?

Any updates on this?

+1

I've also tested it on web and mobile and looks like svelte-flatpickr has no issues with svelte 4.

Would also really love to get this merged!

Tested in Svelte 4.2.0, with no issues. Waiting for this to be merged!

@jacobmischka I know that a lot of people have spammed you about this PR and I apologies for doing the same, but I don't think the constraint suggested by @danhstevens is correct and there is really no reason to limit the peer dependency only up  to Svelte 4.1 (there is already Svelte 4.2 available).
In my opinion the dependency constraint need to change to:

>= 3.31.0 < 5.0.0
>= 3.31.0 || ^4.0.0

Thanks for writing! I did that in master already in a separate commit before publishing: https://github.com/jacobmischka/svelte-flatpickr/blob/master/package.json#L29

Hi @jacobmischka, any chance of when the new package version will be published with this change?

@djavier-gigadat I think it is already published in npm as 3.3.3 version.

Thanks @ganigeorgiev!


Problem with functions sets that have function sets as their range in the arrays encoding

Description
Another bug discovered via https://github.com/informalsystems/apalache/issues/1588. Like https://github.com/informalsystems/apalache/issues/1748 it occurs in both SMT encodings, and seems to be related to cherry picking function sets. It differs in that it exposes the fact that the arrays encoding does not handle function sets whose range is itself a function set. This issue is to track this particular problem.
Input specification
------ MODULE Oracle --------
VARIABLES
  \* @type: Set((Seq(Bool) -> (Int -> Int)));
  witness,
  \* @type: Bool;
  found

Init ==
  /\ witness \in {[{<<TRUE>>} -> [{1} -> {2}]], [{<<TRUE>>} -> [{3} -> {4}]]}
  /\ found \in BOOLEAN

Next ==
  UNCHANGED <<witness, found>>

NotFound ==
  ~found
======================

The command line parameters used to run the tool
apalache-mc check --smt-encoding=oopsla19 --debug Oracle.tla
apalache-mc check --smt-encoding=arrays --debug Oracle.tla
Additional context
For the oopsla19 encoding we have:
at.forsyte.apalache.tla.bmcmt.RewriterException: Do not know how pick an element from a set of type: FinFunSet[CellTFrom(Set(Seq(Bool))), FinFunSet[CellTFrom(Set(Int)), CellTFrom(Set(Int))]]

For the arrays encoding we have:
com.microsoft.z3.Z3Exception: domain sort Cell_F%SQ_b_%F%Si_%Si and parameter sort (Array (Array Cell_Q_b (Array Int Int)) Bool) do not match

So it is picking out of a set of two function sets that map tuples to functions of functions? @thpani, Nitpicker is exploding my brain 🤣

Yikes! 😂
I hope this gets a bit better when we merge the shrinker in #1683!

It is a totally legal TLA+ expression. As every automatic tool, Nitpicker does not have any notion of common sense and thus it is not avoiding scary expressions. This is good!

Sure, I just meant we can probably produce smaller expressions that are easier to parse by humans 😉


Improvements from wiper hackathon

Description
Add improvements from the wiper hackathon
Azure DevOps PBI/Task reference
AB#11305
Checklist

[ ] Release workflow is passing

Hi Dominic, hi Markus,
was that all to take over?


Include twitter badge to Rosie

There is a way to add a twitter badge on README.md.
It must look as something like this: [![Follow](https://img.shields.io/twitter/follow/RosieDaSerenata.svg?style=social&label=Follow)](https://twitter.com/RosieDaSerenata)
And Rosie's account: https://twitter.com/RosieDaSerenata

Hi! can I claim the issue?

Sure @desiby! Please go on!

Closed by #111


gpioTerminate() hangs preventing program close using buildroot package

Using buildroot (git hash 2ef3e19b7b026ad1f6839092428419011ba5d0f4) and pulling in pigpio package for linux kernel build  Linux 4.19.97-v8 aarch64 on Raspberry Pi 4B
I've setup a small program that tests the pigpio library. My program successfully initializes the library with gpioInitialise() but hangs indefinitely on gpioTerminate().
pigpiod -v reveals the following: 7301
I'm not sure what version this is supposed to be.
Any ideas? Do I need an more updated buildroot package for pigpio?

The current release is 78.  A four digit version is use to denote a pre-release usually on the develop branch.  This one, 7301, may have escaped into the wild.

@stevecool2020 , was the problem resolved after upgrading the most recent version?  If so, I'd like to close this issue.

gpioTerminate() still hangs in pipgpio v78 built under Buildroot 2021.02.


[main] hot-fix: Fix the imagestream minimal-cuda sha

[main] hot-fix: Fix the imagestream minimal-cuda sha
JIRA: https://issues.redhat.com/browse/RHODS-12774

Merging this to get the fix right in.


Submitting change requests sometimes double submits

Need to further investigate

I have totally seen this happen in the past, but having trouble replicating it right now.


soc: renesas: ra: ra4m1: Migrate to FSP-based configuration

Change to use FSP to integrate with other Renesas RA series.

@soburi , I think you need to brush up the PR. There are some changes for RA2A1 are mixed into.

Update done.
I will remove DNM and proceed once the following PR in hal_renesas is approved.
https://github.com/zephyrproject-rtos/hal_renesas/pull/31

@iandmorris I am looking to make part of the RA4M1 implementation FSP-based. Could you please review and check on your boards.

@iandmorris I am looking to make part of the RA4M1 implementation FSP-based. Could you please review and check on your boards.

@soburi - just confirmed that blinky still builds and runs on the Mikroe Clicker RA4M1

@iandmorris I am looking to make part of the RA4M1 implementation FSP-based. Could you please review and check on your boards.

@soburi - just confirmed that blinky still builds and runs on the Mikroe Clicker RA4M1

Thank you for your confirmation. I would like to proceed with this.

@KhiemNguyenT
Could you take a look, please?

@gmarull
Could you please re-check your change request?

@KhiemNguyenT
Please approve if it is good for you.

@KhiemNguyenT
Please take a look.


Port raftLogGcWorker and its test.

@coocood PTAL, thanks.

@coocood PTAL thanks.

@coocood  PTAL again, thanks.

LGTM


Allow removal of .js-search-toggle-btn without exception in storefront

1. Why is this change necessary?
When you want to redesign the mobile search and change content of block layout_header_search_toggle in storefront/layout/header/header.html.twig you will receive the exception in the JS which concerns but is totally intentional. At least make it less critical.
2. What does this change do, exactly?
It allows the class .js-search-toggle-btn to not exist in the DOM without throwing an exception.
3. Describe each step to reproduce the issue or behaviour.

Extend template Storefront/storefront/layout/header/header.html.twig
Clear block layout_header_search_toggle

4. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Will add changelog late ™

I added the missing changelog file

Good job @JoshuaBehrens

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-17332
Please use this issue to track the state of your pull request.

Hi @JoshuaBehrens,
Thanks for your contribution

Thank you @NiklasLimberg for merging 💙


Modal shows behind modal-bakdrop layer on 3.3.4

Tested this behavoir on Firefox 31.7.0 and Chrome 42.0.2311.152 m
when click modal link Modal on show appear behind modal-bakdrop so i can't use it.

![modal](https://cloud.githubusercontent.com/assets/5117610/7654293/6d9af15a-faf2-11e4-87dc-dab679f7a746.jpg)

@abaciocco Could you please post a simplified JSBin demonstrating your problem as well the version of bootstrap which you are using?

Dear Kevin:
I set uop a JSBIN and it work, so it means another library is messing up my work. thank you so much for your fast answer and attention.
best regards!
http://jsbin.com/gecaxa/1/edit?html,output

Hi @abaciocco!
You appear to have posted a live example (http://jsbin.com/gecaxa/1/edit), which is always a good first step. However, according to the HTML5 validator, your example has some validation errors, which might potentially be causing your issue:

line 17, column 1 thru line 17, column 121: The aria-labelledby attribute must point to an element in the same document.
line 17, column 1 thru line 17, column 121: The aria-labelledby attribute must point to an element in the same document.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

You are mixing different versions of Bootstrap's CSS and JS, which is not advisable nor guaranteed to work properly.

My mistake! bad copy-paste! i'm am updating my code from 3.3.2 to 3.3.4, and this behavoir appear using full 3.3.4, but not using 3.3.2. thank for your answer!