IIS Integration With VS on same machine

Describe the bug
I have a solution downloaded on my local Windows 10 machine with an ASP.Net Core 2.1 project in it. I have configured IIS to host the site on a particular port (changing the .NET CLR version to No Managed Code) This is so I don't have to have Visual Studio open and debugging this site when i'm working on the front end. But i want to be able to open Visual Studio and set this project as the startup project and debug it without having to use attach to process constantly.
The issue is whenever I reboot my machine and the web site isn't running, when i try to load the website i get the HTTP Error 502.5 - Process Failure error page initially.  If i open Visual Studio and run the project 1 time, the site will load from there on without having Visual Studio open and attached.
If you deploy this same website to a server where Visual Studio is not installed, it will startup just fine.
To Reproduce
Steps to reproduce the behavior:

Using this version of ASP.NET Core 2.1.1
Create a blank site in IIS with No Managed Code configured in the app pool
Run this code: create an ASP.net core website and configure the site for IIS integration on the port configured in step 2
Reboot your machine
Try to load the web site in the browser without opening Visual Studio and hitting F5 to debug
See error

Expected behavior
The website should startup without issue
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
The following error is reported in the event log
Application 'MACHINE/WEBROOT/APPHOST/SOMESITE' with physical root 'C:\dev\development\SOMESITE' created process with commandline 'C:\dev\development\SOMESITE\bin\IISSupport\VSIISExeLauncher.exe -argFile IISExeLauncherArgs.txt' but failed to listen on the given port '47860'
Include the output of dotnet --info
.NET Core SDK (reflecting any global.json):
Version:   2.1.502
Commit:    c74ce8f29f
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.17134
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\2.1.502\
Host (useful for support):
Version: 2.1.6
Commit:  3f4f8eebd8
.NET Core SDKs installed:
2.1.202 [C:\Program Files\dotnet\sdk]
2.1.403 [C:\Program Files\dotnet\sdk]
2.1.500 [C:\Program Files\dotnet\sdk]
2.1.502 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.1.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.1.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.0.9 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
To install additional .NET Core runtimes or SDKs:
https://aka.ms/dotnet-download

The error you are getting references VSIISExeLauncher.exe, which is a tool used in VS with IISExpress. Check the web.config in the 'C:\dev\development\SOMESITE' directory: the processPath is probably C:\dev\development\SOMESITE\bin\IISSupport\VSIISExeLauncher.exe -argFile IISExeLauncherArgs.txt, which isn't what you want when running full IIS.
We recommend publishing your site (dotnet publish or publish in visual studio) to a different directory and pointing your IIS site there. Reusing output between IIS and IISExpress isn't supported.


Upgrade lottie-react-native to 6.4.0

Details

In this PR we:

upgrade the lottie-react-native library to 6.4.0 which has web support,
remove the previous library for web support react-native-web-lottie.

It's a first PR of two, as requested here.
Fixed Issues

$ https://github.com/Expensify/App/pull/29975#pullrequestreview-1704827961
PROPOSAL: N/A
Tests

[x] Verify that no errors appear in the JS console
[x] Check the animations working from QA steps (I verified checked animations in that list)

Offline tests

[ ] Load the app
[ ] Turn off the network in the developer tools
[ ] Go to the view with the animation (i.g. Settings -> Preferences)

QA Steps

[ ] Verify that no errors appear in the JS console
[ ] Verify that every lottie animation is working:

[x] ExpensifyLounge.lottie: Go to Settings -> Profile -> Lounge access
[ ] FastMoney.lottie: Empty Wallet Screen
[x] Fireworks.lottie: Configure 2FA authentication
[x] Hands.lottie: Login screen
[ ] Magician.lottie: Activate Physical Card page
[x] PreferencesDJ.lottie: Settings -> Preferences
[x] ReviewingBankInfo.lottie: ReimbrusementAccountLoading
[x] Safe.lottie: Settings -> Security
[x] SaveTheWorld.lottie: Fab -> Save the world
[x] WorkspacePlanet.lottie: Settings -> Workspaces

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

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

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native / iOS: Native
https://github.com/Expensify/App/assets/104823336/e669c822-c7d4-4a2f-a656-9d27eb1938fb

Android: mWeb Chrome / iOS: mWeb Safari

https://github.com/Expensify/App/assets/104823336/49d7758a-4cc4-4807-9af7-5486fe3b134c

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/104823336/bfc29751-ba3d-41b7-bcea-b64a8f48be70

MacOS: Desktop
https://github.com/Expensify/App/assets/104823336/232132e0-fb27-4093-93e7-c58c0f430ccd

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/roryabraham in version: 1.3.96-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/puneetlath in version: 1.3.96-15 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/puneetlath in version: 1.3.97-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


⚠ Thumbnails API Endpoint has degraded performance

In 980293e, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 1971 ms

Resolved: Thumbnails API Endpoint performance has improved in 1f18a0e.


Fix dataproc helper for large-cohort combiner

hailctl's--pyfiles flag didn't work properly: it only loads .py files so defaults.toml gets missing. Hardcoding the defaults config instead, most of it will be moved into images and references repos anyway.
We still don't guarantee that metamist will have records for all GVCFs, so falling back to checking the file system for combiner.

Ah I didnt see before -- also note the tests are failing


Adding quality flags to the commandLineArgs config does not work

I was considering just doing this as a pull request because it seems pretty simple, at least on paper, however, I then remembered that I am kinda stupid and gave up after realizing I don't know how VSC extensions work.
Honestly, a advanced user would probably get even more use out of a command to freely change command line arguments as a whole, but as previously stated, I'm kinda stupid, so I would only be editing output quality flags, hence why I suggested that it have its own dedicated command
At the end of the day, I can just go edit the command line args in settings and edit it there but I'm lazy and this seems easier.

Hey,
How do you imagine the feature? Are you alking about customizing extra command line arguments? This one should be in the settings.

Yeah I've been working with that, however you get some issues with the sideview if you change the quality like that because it exports to a different folder. In my head I imagine a command in the command palate that you can run, then you just get a list of low, medium, high, etc.
I bring up command line arguments because I'm sure more advanced users would get use out of it, but a quality command I think would be a more relevant feature.

Can confirm, adding '-qm' to the commandLineArgs config does not work as the extension seems to always look for something in the 1080p60 folder.

I will look into this issue. Thank you guys for raising it.

Same here, it's definitely a needed feature. Just wanted to raise +1. 👍

A hacky way to get around this is to add -ql to the command line args, create a symlink from the 480p15 file to the 1080p60 folder. It's gross but I prefer it to disabling preview, using -pql and then watching in vlc.
When you want your final render in high quality just remember to remove the -ql from the command line args or change it to --qp or -qk for even higher than 1080p.

FYI, an alternative way to do this is to use a manim configuration file.

FYI, an alternative way to do this is to use a manim configuration file. The extension is able to read and change its settings.

Can you advise on what to add to the manim.cfg because when I add quality = low it did not work.
Thanks

FYI, an alternative way to do this is to use a manim configuration file. The extension is able to read and change its settings.

Can you advise on what to add to the manim.cfg because when I add quality = low it did not work.
Thanks

Yes, it's low_quality

Oh, my apologies, I did not see that this ticket existed before creating these ones:

https://github.com/Rickaym/manim-sideview/issues/101
https://github.com/Rickaym/manim-sideview/issues/102

FYI available quality options that can be put into the config.cfg file are: 'fourk_quality', 'production_quality', 'high_quality', 'medium_quality', 'low_quality', 'example_quality'. All of them are defined here.


NameError: name 'Scratch2Py' is not defined

when using the example code it gives an error
NameError: name 'Scratch2Py' is not defined
all packages are imported

What all did you run? Did you run the pip install command?

Please provide more information

Closing as not enough information was provided


Whitespaces

Some websites include many whitespaces in their tags. For example see the title tag of https://www.fressnapf.de/p/penthouse-arthur.
Maybe apply a trim on all values? I did no pull request because there is another one open and maybe you have a better solution / more generic way or just want to include all whitespaces. So, just an issue and freedom to decide :dancer:

This is library with a purpose to get data, not to manage it. I would suggest to manipulate data yourself.

And I thought it was a library for link previews… ;-)
Is there any reason where you want a link preview with tons of whitespaces? Anyway, just sayin, nvm


Add CURIE->name lookup

Looks good! For curie filters you might instead go with a filter query (fq) for a slight performance improvement, but a q also works.  It also depends if we expect non exact queries, such as case differences, so q might actually work better.


Add automatic studyguidancegroups access updating to updater

Current implementation in feature branch. Before launching, investigate what is the best to do writes from updater to user-db

probably open user-db port from oodikonepannu + create certs for safe connection

acually we can just make a request to importer and check if person has any tutoringgroups.


feat: add internationalization support

[x] implement http backend and lazy-loading
[x] create initial locale resources

[x] arabic
[x] english
[x] german
[x] spanish (spain)
[x] farsi (iran)
[x] french (france)
[x] hindi
[x] italian
[x] japanese
[x] korean
[x] dutch
[x] polish
[x] portuguese (portugal)
[x] russian
[x] turkish
[x] vietnamese
[x] chinese
[x] norweigan

[x] serve translations for markdown documents
[x] detect browser locale
[ ] update label page component
[ ] update contributing.md
[ ] fix fallback for markdown docs without translations
[ ] structure markdown docs by document with each document having its own directory of translation files
[ ] fix menu header titles longer than menu container width

putting on pause for now as built in browser localization support is pretty good and until there is more community support to review it


DRILL-5564: Added finally block for stopWait() to avoid all situation…

…s where Drill able to miss stopWait() in case of exceptions (it can lead to assertions).

@parthchandra can you review this?


Adding an experimental multicluster ingress API

First stab at defining a MultiClusterIngress API.
There are 2 high level resources here: MultiClusterIngress and MultiClusterService.
Key differences with the current CLI model:

MultiClusterService can be used to create the required service in appropriate multiple clusters (based on a cluster selector). With the existing CLI model, users have to create the required service in all clusters.
MultiClusterIngress does not create Ingress resource in target clusters. Only services are created in target clusters (via MultiClusterService). This is different than CLI which creates Ingress resource in all target clusters.

Advantages:

Users dont have to create Services in target clusters. This can now be done automatically.
Since cluster selection can now be done with service instead of ingress, different paths on the same MultiClusterIngress can route traffic to different set of multiple clusters. For example, /foo can route traffic to clusters A and B and /bar can route traffic to clusters B and C for a single MultiClusterIngress.

Sharing the API definition early to get reviews. Please review and add comments.

cc @csbell @bowei @rramkumar1 @agau4779 @thockin @G-Harmon


Discuss performance benefits of native types

From https://github.com/perl6/doc/issues/372#issuecomment-326795282:

Native int, num and so on can be used to speed up thing, like shaped Array. They are not only useful with NC

Not only speed things up, but natively typed arrays can be a giant memory saving also where applicable.

Where could this go? nativecall page is not exactly the right place. And as native types go, we only have int documented. After https://github.com/perl6/doc/issues/372 all native types point to nativecall page but again that's not entirely right.

The int documentation is somewhat unfortunate in that it lists a load of methods you can call on a "native int". This not quite true, as you can't call a method, or indeed do anything object-y, on a native type. Calls to such methods will box the native integer into an Int and the method will be called on that (one can observe this by trying (my int $x = 42).WHAT and observing it says Int, not int - the reason is that the int was boxed into Int to have something to ask for the type object of).
Perhaps making the native int documentation describe what a native integer is, the sized variants, and its boxing behavior (with a link to Int documentation) would be more useful?

I can think either of a new performance issues page or the already existing traps page.

@JJ no-no, I think this should go directly to the same place where native types are discussed. Unfortunately, I don't know about a page like this.
To clarify, a person searching for “int” should be able to eventually land on a page which discusses packed native arrays and stuff like that. And there we can say why the user may want to do that at all (e.g. for performance reasons).

Currently searching for int gives https://docs.perl6.org/type/nativeInt which says “TODO”. Not good at all :(
There's also https://docs.perl6.org/language/nativecall but that's a different topic IMO.

FWIW the CaR grant branch that'll be merged after next release (2018-05-19) adds a "Numerics" language page that does mention native types as well as their performance benefits, including the case where they're lost because code does a lot of boxing.


add useful labels to helm templates for ztunnel

Please provide a description of this PR:
This adds some useful labels to our ztunnel helm installations which would normally be found only if you use istioctl. I think we should phase out operator.istio.io/component in favor of istio.io/component or something similar. Also IMO we shouldn't be capitalizing the component name. Ztunnel or Cni just look weird to my eye. Lets go with lowercase for the new label's value but left uppercase since that is what istioctl and other components have for their operator.istio.io/component value...

@ilrudie: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
a5c455a195f65c0281c0ef179c8298c0292d286f
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

any chance we can count not labeling things we install if installed via helm a bug and backport this?
@istio/release-managers-1-22

I do not want anything with "operator" in it, or 4 labels for the same thing.
If we simply add "app: ztunnel" to the objects missing it this lgtm. does that meet your goals?

If the goal here is to be a model helm chart, we should follow:
https://helm.sh/docs/chart_best_practices/labels/#standard-labels

I do not want anything with "operator" in it, or 4 labels for the same thing.
If we simply add "app: ztunnel" to the objects missing it this lgtm. does that meet your goals?

yeah. adding operator...: Ztunnel was painful and I don't like it.

@ilrudie: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
58813ff3f38becf79850b7cc651801a9ab4b75cd
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@ilrudie: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
28fd79d315c2686f9434bbfeb730204048e00781
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

In response to a cherrypick label: new pull request created: #52067


[P4_Symbolic] Adding table_hit_entries_pb.txt files

Keyword Check:
~/sonic-buildimage/src/sonic-p4rt/sonic-pins$ ~/tools/keyword_checks.sh .
Keyword check Passed.
Build Result:
/sonic/src/sonic-p4rt/sonic-pins$ bazel build $BAZEL_BUILD_OPTS ...
INFO: Analyzed 567 targets (0 packages loaded, 0 targets configured).
INFO: Found 567 targets...
INFO: Elapsed time: 0.580s, Critical Path: 0.00s
INFO: 1 process: 1 internal.
INFO: Build completed successfully, 1 total action
Test Result:
/sonic/src/sonic-p4rt/sonic-pins$ bazel test $BAZEL_BUILD_OPTS --cache_test_results=no ...
INFO: Analyzed 567 targets (0 packages loaded, 0 targets configured).
INFO: Found 382 targets and 185 test targets...
INFO: Elapsed time: 180.140s, Critical Path: 119.86s
INFO: 239 processes: 282 linux-sandbox, 18 local.
INFO: Build completed successfully, 239 total actions
//gutil:collections_test                                                 PASSED in 0.5s
//gutil:io_test                                                          PASSED in 0.5s
//gutil:proto_matchers_test                                              PASSED in 0.6s
//gutil:proto_ordering_test                                              PASSED in 0.6s
//gutil:proto_test                                                       PASSED in 0.6s
//gutil:status_matchers_test                                             PASSED in 0.5s
//gutil:test_artifact_writer_test                                        PASSED in 0.6s
//gutil:testing_test                                                     PASSED in 0.6s
//sai_p4/instantiations/google:middleblock_p4info_up_to_date_test        PASSED in 0.0s
//sai_p4/instantiations/google:sai_nonstandard_platforms_build_test      PASSED in 0.0s
//sai_p4/instantiations/google:sai_nonstandard_platforms_cc_test         PASSED in 0.7s
//sai_p4/instantiations/google:sai_p4info_fetcher_test                   PASSED in 0.7s
//sai_p4/instantiations/google:sai_p4info_test                           PASSED in 1.1s
//sai_p4/instantiations/google:sai_pd_proto_test                         PASSED in 0.0s
//sai_p4/instantiations/google:sai_pd_util_test                          PASSED in 0.5s
//sai_p4/instantiations/google:tor_p4info_up_to_date_test                PASSED in 0.0s
//sai_p4/instantiations/google:union_p4info_up_to_date_test              PASSED in 0.1s
//sai_p4/instantiations/google:wbb_p4info_up_to_date_test                PASSED in 0.0s
//sai_p4/instantiations/google/test_tools:table_entry_generator_helper_test PASSED in 1.9s
//sai_p4/instantiations/google/test_tools:test_entries_test              PASSED in 0.9s
//sai_p4/instantiations/google/tests:p4_fuzzer_integration_test          PASSED in 3.7s
//sai_p4/tools:p4info_tools_test                                         PASSED in 0.6s
//sai_p4/tools:packetio_tools_test                                       PASSED in 0.8s
//thinkit:bazel_test_environment_test                                    PASSED in 0.5s
//thinkit:generic_testbed_test                                           PASSED in 0.9s
//thinkit:mock_control_device_test                                       PASSED in 0.6s
//thinkit:mock_generic_testbed_test                                      PASSED in 0.7s
//thinkit:mock_mirror_testbed_test                                       PASSED in 0.7s
//thinkit:mock_ssh_client_test                                           PASSED in 0.0s
//thinkit:mock_switch_test                                               PASSED in 0.7s
//thinkit:mock_test_environment_test                                     PASSED in 0.1s
//thinkit:switch_test                                                    PASSED in 0.8s
//sai_p4/instantiations/google/tests:p4_constraints_integration_test     PASSED in 1.0s
Stats over 5 runs: max = 1.0s, min = 0.8s, avg = 0.9s, dev = 0.1s
//sai_p4/instantiations/google/test_tools:table_entry_generator_test     PASSED in 43.8s
Stats over 50 runs: max = 43.8s, min = 0.8s, avg = 2.8s, dev = 8.4s
Executed 185 out of 185 tests: 185 tests pass.
INFO: Build completed successfully, 239 total actions

Co-authored-by: kheradmandG<EMAIL_ADDRESS>


Entrypoint script doesn't work well with additional arguments

Running this image with additional arguments seems to cause an error. for example:
docker run -it ngrok/ngrok http --region=us --hostname=jjurlcbx.ngrok.io --authtoken=<TOKEN> 80
returns the non-intuitive error:
ERROR:  You must specify a single argument: a port or address to tunnel to.
ERROR:  You specified 2 arguments: [80 --config=/var/lib/ngrok/ngrok.yml]
ERROR:  For example, to expose port 80, run 'ngrok http 80'.

This command fails because the entrypoint.sh is attempting to append a --config to the end of the command and is failing.
We should be able to update the entrypoint.sh script to know where to add the additional --config should be appended to the args list.

Indeed, good find, and thanks for digging in to why it happens!
I think my preferred solution here would be to support flags after the positional argument within the ngrok agent, rather than munging flags in the docker entrypoint
It seems to me like ngrok http --config=config.yml 8080 --hostname foo.ngrok.io should be unambiguous, and I think it's less confusing if the agent just supports that in general too, which would have the additional benefit of fixing this issue.
I'll dig into whether there's some good technical reason we can't support flags in both those places, and based on that we can figure out what to do next.

+1, providing dynamic arguments is impossible now.
Additionally, exposing every argument as an env var would also be nice (e.g. NGROK_HTTP_REGION=eu).

This bug is fixed, as of a couple days ago, with the v3 agent release :tada:!
The ngrok/ngrok:latest tag, and the 3* tags all shouldn't exhibit this bug. Note, there are a few flag changes and such, so it may require a few more changes to update as well.
I'll close this with the recommendation to upgrade to the v3 agent to get the fix for this argument parsing issue.

As a not super relevant aside, the bug's also fixed on the internal v2 agent branch (commit 7dd9e4b0 for anyone with access to the repo), so if/when we have to do a security release for the 2.x agent, it'll also be fixed in that version. We don't plan to do a v2 release just for this bug though.


Questions on MCP's intended usage patterns

Is your feature request related to a problem? Please describe.
No problem, just some questions on intended usage patterns
Describe the solution you'd like
No solution suggestion, just some questions
Describe alternatives you've considered
I've read through the docs and have set up a Python server to explore
Additional context
Love the initiative, glad to see attempts of standardization. Some questions:
Are there any clients that exist besides Claude Desktop? Plans for any? Any repos implementing the client side?
Seemingly Claude Desktop doesn't work on Linux, is there a good way to validate behavior if developing on Linux?
The MCP Server is meant to run locally, installed by the user, not run somewhere in the cloud? How do you envision people who aren't technical to configure this? Model "Store" coming in the future?

For Linux, the best approach is using the inspector npx @modelcontextprotocol/inspector. There is cody and zed as clients with some restrictions. See the manual: https://modelcontextprotocol.io/clients
There are early thinkings on making servers remote capable, as a cloud service.
This feels more like a discussion and not an issue. If you have further questions please create a topic in the discussion forums


NTR: myelinating glial cell

Doug suggests this NTR. See:

https://sourceforge.net/p/geneontology/ontology-requests/11392/

We have

 * CL:0000328 ! myelin accumulating cell (undefined in CL, inherited from initial version of CL).

This includes 'myelinating Schwann cell', but not oligodendrocyte.

Anyway, I think this class is weird, and should be obsoleted with a consider to
CL:new 'myelinating glial cells'

CL also makes a distinction between plain Schwann cell and 'myelinating Schwann
cell'. Do we need the latter?

{{{

         is_a CL:0002573 ! Schwann cell [DEF: "A glial cell that ensheathes axons of neuron in the peripheral nervous system and are necessary for their maintainance and function."]
          is_a CL:0000218 ! myelinating Schwann cell ***  [DEF: "A neuroglial cell of the peripheral nervous system which forms the insulating myelin sheaths of peripheral axons."]

}}}

By the text defs, these should be merged. Are there unmyelinating Schwann cells?

A complicating factor is that 'myelin' is found in non-vertebrates but these
may be functional analogs, and the method of producing 'myelin' may be
different. See: PMID:19508741

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 9 Dec 2014 at 10:47

Perhaps this can be solved by having a disjoint with for myelinating and non-myelinating schwann cell
I do get that the def in schwann cell and myelinating schwann cell are difficult to differentiate, but I think the def looks ok. Perhaps a comment on schwann cell that says that it can be both myelinating and non-myelinating?
happy to also add the has_part some myelin sheath (GO:0043209) but i dont think there is a equivalence for non-myelinating, though maybe the disjioint with would help here? @cmungall


Ubuntu 22.04 vs ZFS = hung boot

Are you using the latest driver? Version from Apt repo (5.7.0-129)
Are you using the latest EVDI version? Version from Apt repo (1.13.1-18)
If you are using a DisplayLink device, have you checked 'troubleshooting'
on DisplayLink's website? Not listed
Is this issue related to evdi/kernel? Not... necessarily!
Linux distribution and its version : Ubuntu 22.04
Linux kernel version: 5.19.0-41-generic #42~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Tue Apr 18 17:40:00 UTC 2 x86_64 x86_64 x86_64 GNU/Linux
Xorg version (if used): ii  xorg           1:7.7+23ubuntu2 amd64        X.Org X Window System
Desktop environment in use: Ubuntu Gnome

Following the installation of the above package and rebooting, I was presented with a recovery console.
Purging the displaylink-driver and evdi packages restores my machine to it's former state, but this isn't really useful.
Following re-installing the driver, rebooting and returning to the recovery console, when digging into the logs, I found an entry showing that:
Apr 28 11:12:05 jonspriggs-Kratos-EL04R6 systemd[1]: Starting Wait for udev To Complete Device Initialization...
░░ Subject: A start job for unit systemd-udev-settle.service has begun execution
░░ Defined-By: systemd
░░ Support: http://www.ubuntu.com/support
░░
░░ A start job for unit systemd-udev-settle.service has begun execution.
░░
░░ The job identifier is 20.

Followed later by
Apr 28 11:13:05 jonspriggs-Kratos-EL04R6 systemd-udevd[1912]: 4-3.1.3:1.0: Spawned process '/opt/displaylink/udev.sh /dev /devices/pci0000:00/0000:00:14.0/usb4/4-3/4-3.1/4-3.1.3/4-3.1.3:1.0 usb-004-004-DisplayLink_PR09_DisplayPort_Dock_YVFJ093338 /dev/bus/usb/004/004' [2280] is taking longer than 59s to complete
Apr 28 11:13:05 jonspriggs-Kratos-EL04R6 systemd-udevd[1787]: 4-3.1.3:1.0: Worker [1912] processing SEQNUM=4598 is taking a long time
Apr 28 11:14:05 jonspriggs-Kratos-EL04R6 systemd[1]: systemd-udev-settle.service: Main process exited, code=exited, status=1/FAILURE
░░ Subject: Unit process exited
░░ Defined-By: systemd
░░ Support: http://www.ubuntu.com/support
░░
░░ An ExecStart= process belonging to unit systemd-udev-settle.service has exited.
░░
░░ The process' exit code is 'exited' and its exit status is 1.

Manually running the above command did not take a long time (in fact, it ran and completed in microseconds).
In some of the intervening logs, I also saw:
Apr 28 11:12:05 jonspriggs-Kratos-EL04R6 udevadm[1890]: systemd-udev-settle.service is deprecated. Please fix zfs-load-module.service, zfs-import-cache.service not to pull it in.

Correspondingly, I have adjusted the zfs-load-module.service (/lib/systemd/system/zfs-load-module.service) and zfs-import-cache.service (/lib/systemd/system/zfs-import-cache.service) files, commenting out Requires=systemd-udev-settle.service, which has worked-around this issue.
I concede that the ZFS services shouldn't depend on systemd-udev-settle, however, the fact that systemd-udev-settle is failing as a result of the DisplayLink or evdi service means that it's having a knock-on impact on other areas of the system.
Please can you investigate?

I have the exact same problem on my laptop!
And I know at least one other person that said to me that his laptop boots really slow but he did not really care why since it always booted. I had checked and he also used ubuntu+2 displaylink monitors through a dock.
The problem does not happen if the monitor connects directly to the pc (I have an exact same os and monitor on a tower). It seems to only happens when the monitors are connected to a dock.
The problem also goes away without any changes to the .services files if during boot you disconnect the dock from the pc and connect it after the boot (what I have been doing).


convert to modern ecma and standardize on esm. fixes #6

@jotson any thoughts on this?

closing in favor of https://github.com/mreinstein/random-gap?tab=readme-ov-file#loottable


Can't update a CSV database or need to use workarounds?

I might have missed some page or example test, but a brief review of the wiki and some example tests, I don't seem to find much examples of inserts & updates of different database types. Seems most of the examples for various DBs are for selects where you can also dump the output of a select from one DB into another via "select into"
I was trying something like this basing code on the select and update examples:
var alasql = require('alasql');
alasql('UPDATE csv("/data/monitoring/logs/staging/statsdb.csv",{headers:true}) SET notes = "it works" WHERE device = "iPod2"');
but I get:
/Users/daluu/temp/node_modules/alasql/dist/alasql.js:1818
throw new Error(str);
^
Error: Parse error on line 1:
UPDATE csv("/data/monitoring/l
----------^
Expecting 'SET', got 'LPAR'
at Object.parseError (/Users/daluu/temp/monitor/node_modules/alasql/dist/alasql.js:1818:15)
at Object.parse (/Users/daluu/temp/monitor/node_modules/alasql/dist/alasql.js:1883:22)
at Function.alasql.parse (/Users/daluu/temp/monitor/node_modules/alasql/dist/alasql.js:3982:16)
at Function.alasql.dexec (/Users/daluu/temp/monitor/node_modules/alasql/dist/alasql.js:4136:19)
at Function.alasql.exec (/Users/daluu/temp/monitor/node_modules/alasql/dist/alasql.js:4112:17)
at alasql (/Users/daluu/temp/monitor/node_modules/alasql/dist/alasql.js:121:18)
at Object. (/Users/daluu/temp/monitor/query.js:20:1)
at Module._compile (module.js:460:26)
at Object.Module._extensions..js (module.js:478:10)
at Module.load (module.js:355:32)
Is updating a CSV DB directly not supported? Or did I just have the wrong syntax?
If not supported, what's best workaround? Use an in memory DB (or other) and then overwrite the CSV with in memory version to do the update via select into csv from in memory DB?
It would be nice to clarify on the wiki which DBs are and are not supported for the various SQL commands, because otherwise, one might assume it works against all.

Hmm... following examples on the site to create a DB, table (with fields/schema roughly matching CSV file), then select into newly created table from CSV seems to result in empty result "[]", or just returns the count of rows when I print "res" return value but not the actual data.
Querying directly with select * from csv works fine.
Would be nice if an example could be presented that shows you how to use this library to update a CSV database (then select back out the results to console log to verify the update).

Thank you for a good input

It would be nice to clarify on the wiki which DBs are and are not supported for the various SQL commands, because otherwise, one might assume it works against all.

I really agree. Lets work on that.
As I understand the library whats suported is SELECT INTO - so no update to files
I will have to dig into some examples and will get back to you. Probably tomorrow...

@mathiasrw Do we need to extend functionality with this statement?
UPDATE CSV(...) SET ... WHERE ...

UPDATE CSV(...) SET ... WHERE ...

is a very good area to improve the library. I suggest we put it on the list of "really good things to implement after the critical ones".
If we implement this on CSV( we must offer the same functionality for the other export functions as forexample XLS(, JSON(  and so on

Yes. The same for :
INSERT INTO CSV('a.csv') (a,b,c) VALUES (1,2,3);
DELETE FROM XLS('a.xls') WHERE a = 10;
UPDATE CSV('a.csv') SET b=20 WHERE a = 10;

We can have a problem with ```INSERT``` syntax, because parser can not
understand what is this:

INSERT INTO CSV('a.csv',{headers:true}) VALUES (1,2,3);
INSERT INTO one(a,b,c) VALUES (1,2,3);

This functionality can be useful for command-line version of AlaSQL.

2016-01-11 16:11 GMT+03:00 Mathias Rangel Wulff<EMAIL_ADDRESS>
> UPDATE CSV(...) SET ... WHERE ...
>
> is a very good area to improve the library. I suggest we put it on the
> list of "really good things to implement after the critical ones".
>
> If we implement this on CSV( we must offer the same functionality for the
> other export functions as forexample XLS(, JSON( and so on
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/agershun/alasql/issues/383#issuecomment-170544646>.
>

We can have a problem with INSERT syntax, because parser can not
understand what is this:

We have a lot of reserved words. Guess we have to add CSV, XLS ... and so on to the list of reserved words so that if one has a table named csv they have to
INSERT INTO `one` (a,b,c) VALUES (1,2,3);

INSERT INTO CSV('a.csv',{headers:true}) VALUES (1,2,3);

What will happen if the headers value is not set?

Closed as its put on on hold on The Long List - only to be reopend when its time has come.


play-app: transformation from PCM to HTML or handonstable does not support feature groups

Fixed by 2e120ae303bfffb7f279a85756e8ef2f79999bad


support for CONSTRUCT queries in the FnO definition of the function.

In addition to sparql SELECT queries, there is now also support for CONSTRUCT queries. The query handling is done by Comunica.

Cool! Would love to see a test/example file of that 👼


Doesn't work on NEX-6

Users report an app crash

Installed 0.2, app still crashed
TWEAKLOG.TXT

Can you try v0.3 please?

Worked like a charm, thanks for your work ma1co

Awesome, thank you for testing :)


Sony PS5 Controller on Mac

Hello, thank you for a great plugin. I noticed that on mac os PS5 DualSense controller is called "Sony DualSense". I patched my local version, maybe you can also add this one to the list

Thanks 👍  I've added "Sony DualSense" as an extra PlayStation identifier.


Throwing HubException from server should bypass EnableDetailedErrors flag

Just like ASP.NET SignalR

Yep, that makes sense. We should do it.

I have better idea... implement possibility to extend/overide some functionalities of exceptions parsing  and handling.
It will open possibility for implementation of:
Throwing HubException from server should bypass EnableDetailedErrors flag
and a at least hundred of other strategies...

I don’t think we’re going to be providing any extensibility here. We’re going to do what ASP.NEt SignalR does and see if satisfies 90% of users. We have no desire for pre-mature extensibility that a majority of users don’t need.

Done here: c7ebae47ea3569183a27c1edd745d79da7c8f159


Angular2CookieLawService compatibility with Angular Universal

Fixes #29

@exequiel09  I had to revert this PR as it was breaking the build.
You can see the log from https://travis-ci.org/andreasonny83/angular2-cookie-law/builds/438295127#L566. Try to fix the problem, then raise a new PR. Thanks.

That can be fixed easily, i'll send a new PR.


When active, background gets much lighter

This can be undesirable, especially when you want it to use atoms background color.

Uhh... can you describe this problem in more detail? I've never run into it.
Screenshots, info about your setup (OS, Atom version, theme, etc) would be very useful.


Added option to not copy messages.

New Pull Request Checklist

[x] I have read and understood the CONTRIBUTING guide

[x] I have read the Documentation

[x] I have searched for a similar pull request in the project and found none

[x] I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

[x] I have added the required tests to prove the fix/feature I am adding

[x] I have updated the documentation (if necesarry)

[x] I have run the tests and they pass

[x] I have run the lint and it passes (pod lib lint)

This merge request fixes / reffers to the following issues: ...
Pull Request Description

Codecov Report

Merging #832 into master will increase coverage by 0.01%.

@@            Coverage Diff             @@
##           master     #832      +/-   ##
==========================================
+ Coverage   22.92%   22.94%   +0.01%
==========================================
  Files          16       16
  Lines        3899     3900       +1
  Branches      279      279
==========================================
+ Hits          894      895       +1
  Misses       3003     3003
  Partials        2        2

Impacted Files
Coverage Δ

Classes/DDLog.h
88.23% <ø> (ø)
:white_check_mark:

Classes/DDLog.m
57.23% <100%> (+0.05%)
:white_check_mark:

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 f2b127e...fa30fa4. Read the comment docs.


iothub_client.pyd for python 3.4 32bit

OS and version used: <Windows 10>

Python runtime used: <3.4 32bit>

SDK version used: <10-30-2017>

Description of the issue:

Has anyone been able to rebuild the iothub_client.pyd for python 3.4 32bit?
I was able to rebuild all the "c" sub folders including the BOOST files and OpenSSL. The "c" samples also work. Rebuilding the python bindings against python 3.4 32 bit presents many errors which I can not resolve. I tried to rebuild to python 3.6 using VS 2015 and the 32bit tools and also get errors (although different ones). I am not sure I am linking against the "regenerated" "C" libraries ...there seems to be some missing steps in the instruction.
Code sample exhibiting the issue:

Console log of the issue:

@dvescovi1
Please provide more details and make sure you are following the steps correctly. We are currently building the SDKs with 3.6 for the release. We build them with 3.4 a while ago.
Best Regards,
Zoltan

@dvescovi1
Closing this issue because there is no activity. Please open a new one if the problem still exists.
Best Regards,
Zoltan


Create configuration for CircleCI

CircleCI needs a config.yml file inside .circleci folder to automate CI / CD processes

Added the given config.yml
# Use the latest 2.1 version of CircleCI pipeline process engine.
# See: https://circleci.com/docs/2.0/configuration-reference
version: 2.1

# Define a job to be invoked later in a workflow.
# See: https://circleci.com/docs/2.0/configuration-reference/#jobs
jobs:
  # Below is the definition of your job to build and test your app, you can rename and customize it as you want.
  build-and-test:
    # These next lines define a Docker executor: https://circleci.com/docs/2.0/executor-types/
    # You can specify an image from Dockerhub or use one of our Convenience Images from CircleCI's Developer Hub.
    # Be sure to update the Docker image tag below to openjdk version of your application.
    # A list of available CircleCI Docker Convenience Images are available here: https://circleci.com/developer/images/image/cimg/openjdk
    docker:
      - image: cimg/openjdk:17.0
    # Add steps to the job
    # See: https://circleci.com/docs/2.0/configuration-reference/#steps
    steps:
      # Checkout the code as the first step.
      - checkout
      # Use mvn clean and package as the standard maven build phase
      - run:
          name: Build
          command: mvn -B -DskipTests clean package
      - save_cache:
          paths:
            - ~/.m2
          key: v1-dependencies-{{ checksum "pom.xml" }}
      # Then run your tests!
      - run:
          name: Test
          command: mvn test
      - save_cache:
          paths:
            - ~/.m2
          key: v1-dependencies-{{ checksum "pom.xml" }}

# Invoke jobs via workflows
# See: https://circleci.com/docs/2.0/configuration-reference/#workflows
workflows:
  sample: # This is the name of the workflow, feel free to change it to better match your workflow.
    # Inside the workflow, you define the jobs you want to run.
    jobs:
      - build-and-test


Bug fix 3.4/multi bugs

Scope & Purpose
Give up trying to persist follower info in agency for collections that have been dropped in the meantime. The previous implementation cycled forever, occupying one scheduler thread just with the retries.

[x] Bug-Fix for a released version (did you remember to port this to all relevant release branches?)
[x] The behavior in this PR can be (and was) manually tested (support / qa / customers can test it)

Testing & Verification
This change is already covered by existing tests, such as scripts/unittest shell_client --cluster true.
https://jenkins01.arangodb.biz/view/PR/job/arangodb-matrix-pr/5843/

Tests blue


add Baidu on the powered-by page

Please answer these questions before submitting pull request
add power by

skywalking have been running in a test environment before and will be deployed online next week.
In the future ,i would like to share this.Our department belongs to the tob industry, the data size will not be very large.

Codecov Report

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

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

Interesting, due to #4793, it shows you care about the large scale deployment very much.

🙈In Friday,our es cluster is restarted due to some failure, the es cluster store logs data and some skywalking index in test env and preonline env.
Before I knew it, I thought that shards num was for segment record,so i set the number is 5
Because of too many shards in the cluster, the restart time is so long
so,I commit the PR
https://github.com/apache/skywalking/pull/4793

That PR is meaningful, I remembered some people asked about this, but they didn't submit the pull request to fix that in the upstream. Thanks.

I remember the 7.0.0 merge some metrics together.
Nowadays,I think skywalking's pressure on es may mainly come from unreasonable allocation of shards,Hope this PR can solve the pressure on es
https://github.com/apache/skywalking/pull/4793

Nowadays,I think skywalking's pressure on es may mainly come from unreasonable allocation of shards,Hope this PR can solve the pressure on es

If you have some monitoring data to prove this, welcome to let us know. Reducing shards should be a good thing. And also in 8.0.0, we removed the register, which should help to get rid of lock in the es level.

ok,i would be focus on it
When there is time, I will continue to mention some PR,do you think the following content makes sense?

spring annotation plugin add regex match,avoid intercepting all spring beans
dynamic get config from kubernetes configMap

I don't know why the pipline fail ,cloud you follow up @wu-sheng

Could you start separated issues to discuss these with details? It makes more people to follow the context.

Ok,I would to open a new discuss


Fixing a text editing bug happening when text field changes.

During text editing, if one changes between text fields, sometimes the size/transform was not being updated.
Description
This was due to the check in this editable text:
if (size != _lastSize || transform != _lastTransform)
When there is a new connection lastSize and lastTransform was not being cleared.
By carrying lastSize,lastTransform to TextInputConnection we are making sure that when there is a new connection, the old values won't be cached.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[X] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[X] I signed the CLA.
[X] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[X] I updated/added relevant documentation (doc comments with ///).
[X] All existing and new tests are passing.
[X] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[X] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
[X] No, this is not a breaking change.

The IOS platform errors seems irrelevant. I'm merging the PR.


chore(release): prepare release 0.3.1

Pull Request Test Coverage Report for Build 77

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.554%

Totals

Change from base Build 76:
0.0%

Covered Lines:
477

Relevant Lines:
484

💛  - Coveralls

Pull Request Test Coverage Report for Build 77

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.554%

Totals

Change from base Build 76:
0.0%

Covered Lines:
477

Relevant Lines:
484

💛  - Coveralls


replace np.bool with bool as per numpy request

The usage of numpy.bool has been deprecated as of Numpy 1.20

Needs to run through formatting


Virtualization kernel extensions / KVM on Jetson / Xavier

would these patches be a decent addition to the build tree as bb appends for virtualization / KVM
https://github.com/b-man/Xavier-KVM

Did you find the first two patches to be necessary? I see the third is just to fix a warning. It's been a couple releases since I tried it, but I was able to run KVM with Debian and linux-yocto guests with only the GIC device tree changes and a custom kernel config, so I'd be curious where the first two changes are applicable.
In the latest l4t, note that the default device tree includes the GIC device tree changes.

according to the documentation on the nvdia forum the rest were added for winders 10 arm64
https://forums.developer.nvidia.com/t/guide-to-enable-kvm-on-the-xavier/119777
and the recently merged windows patch. Be nice to see it added as a bitbake recipe so we can Yoctoize the Xaviers, since i have also meta-virtualization with k8s and docker-nvidia, going for kubevirt / k8s on Xavier..... requires a working libvirt / kvm .... I also previously inquired about XEN which is whats preferred, or making xvisor work for the Jetsons

The kernel patches are backports from upstream, so I'd be OK with submitting them as a PR in the linux-tegra-4.9 repo.
If you can isolate the specific config settings that need to be enabled, the config changes could be applied in a .cfg file as part of a linux-tegra bbappend under external/meta-virtualization, conditional on DISTRO_FEATURES containing kvm.  That would be the least intrusive.


chore: use local images instead of imgur

we don't want to rely on external sites for image hosting


SAK-48263 lessons: fixed edit item dialog

https://sakaiproject.atlassian.net/browse/SAK-48263

@csev Could you give this a test out? It seems ok to me.


Mipsel32 GOT relocations not applied properly

Version 2.4.2846 (2021-06-03 17:42:23) - latest stable

Can you provide a binary please?


Release 0.1.0

@JuliaRegistrator register(branch=release)

@JuliaRegistrator register(branch=release)

Registration pull request created: JuliaRegistries/General/15629
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 8e07cdc8293ed73c3739fac693aa4d768a0cb02a
git push origin v0.1.0


Split SourceKitLSPServer.swift into multiple files

The file was a little large and contained multiple types that can easily be split off. Now you can scroll to the top of the file and see the members of SourceKitLSPServer.

@swift-ci Please test

@swift-ci Please test Linux

@swift-ci Please test Windows


Bunny command sets the description of the embed to the permalink

This shouldn't be the case as its not clickable and therefore should be removed

Blocked until Random Bunny 2.3 is updated


Problem with touch screen devices

I've integrated this plugin in my web app and it's working fine in desktop, but issue occurred while doing the same with touch devices. How can I integrate this plugin with touchscreen device, Please help me regarding same?

Because there is a lot libraries that support touch gestures, when I created xZoom I decided not to implement my own into xZoom, instead create as much as possible API functions with which everyone will be able to integrate it with touch gestures or anything else, their preferred way. If you take a look here from mobile device it works with touch screen (tap is opening, then when it is opened double tap opening MagnificPopup or Fancybox in their examples, plus movement is working too):
https://payalord.github.io/xZoom/
On that example page, xZoom integrated with HammerJS version 1. But there is already HammerJS version 2 available. But I don't have enough time to create normal examples in xzoom-sandbox for this.

This example will show how to integrate with HammerJS v2: https://github.com/payalord/xzoom-sandbox/tree/master/Hammer.jsV2


IterableMap Go implementation

Implement IterableMap in golang to use it in precompiled contract. It should work with a storage(load and dump(save) modified values to storage

fixed by https://github.com/Taraxa-project/taraxa-evm/pull/42


How does this pointer get deleted?

Thanks a lot of sharing your repository.
Do you know how this pointer can be deleted?
IOptimizationProfile *optProfile = builder->createOptimizationProfile();

https://github.com/cyrusbehr/tensorrt-cpp-api/blob/main/src/engine.cpp#L126

See NvInfer.h
The builder retains ownership.


Enable the list macro to call the external link macro

Enable the list macro to call the external link macro so that external links in a bulleted list can inherit the icon and all the correct attributes.

Originally posted by @jrbarnes9 in https://github.com/ONSdigital/design-system/discussions/233#discussioncomment-943939

Double check HTML content filter in Service Manual


Support running scripts

It will be nice if eclisp can read a script and execute it always, instead of running in a REPL mode.

Done. Can be ran in this format:
./eclisp -f script.l

Done. Can be ran in this format:
./eclisp -f script.l

Where script.l contains the expression to evaluate.


FilterSpider: Remove process_file_url and process_text_url

These methods are identical, so there isn't a need for two of them.
Moreover, they're tiny and single-purpose, so it's much more explicit to
just do the 'meta' creation inline.

I think this looks ok - were those methods overriden anywhere in the codebase?

looks good.


Please invite me to the GitHub Community Organization

Name
Simran Gupta
Discord Username (if applicable)
Simran#0909
Additional Context
The first time, I saw Eddie on a youtube CodeForCause with Kunal Kushwaha. I got so much influence from him.
I like this community and want to be part of this community because this is a very inclusive community and help beginners to learn.
Lastly, I love both slogans by Eddie. The first one is "Collaborate first and code second" and the second one is "Open Source is for everyone".

Welcome to EddieHub, @Simran-Gupta234! We hope you learn and grow your skills with us! 😄👋🎉


Post Multipart and json

Hi
im trying to post a file and along with json like this fiddler request:
`---------------------------acebdf13572468
Content-Disposition: form-data; name="fieldNameHere"; filename="3.jpg"
Content-Type: image/jpeg
<@INCLUDE C:\Users\user0.1\Pictures\3.jpg@>
---------------------------acebdf13572468
Content-Disposition: form-data; name="model"; filename=""
Content-Type: application/json
{"ptid":1004,"i":null,"id":0,"bo":null,"t":"rev pink 5","hc":"hello","vs":1,"vi":"http://dornian.COM","hi":false,"iw":0,"ih":0,"po":["55555","hhhhh","iiiii","vvvvv","aaaaa"],"co":["aaaaa","vvvvv","iiiii","22222","11111"]}
---------------------------acebdf13572468--`
but i keep getting this error:
 {"Message":"An error has occurred.","ExceptionMessage":"Object reference not set to an instance of an object.","ExceptionType":"System.NullReferenceException","StackTrace":"   at System.Web.Http.ApiController.<InvokeActionWithExceptionFilters>d__1.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at System.Web.Http.Dispatcher.HttpControllerDispatcher.<SendAsync>d__0.MoveNext()"}
can you help me?

post your code

i fixed it, thanks anyway

for anyone who's wondering how
      RequestParams params = new RequestParams();         params.put("model", new ByteArrayInputStream(jsonObject.toString().getBytes()), "", "application/json");         if (imagePath != null)             params.put("image.jpg", new File(imagePath), "image/jpeg", "image.jpg");         params.setForceMultipartEntityContentType(true);         client.post(url, params, handler);


Enhancement: issue an error when defining attributes that are not part of the model

Summary
Defining a field on a factory (e.g: with PostGenerated) that is not defined on the target model will result into the field being completely ignored.
When making an error, e.g: a typo in the field name on the factory, it's hard to know what's going on.
I'd like an error to be thrown telling me that I tried to define a field that doesn't exist on the model.
Basic Example
from dataclasses import dataclass

from polyfactory import PostGenerated
from polyfactory.factories import DataclassFactory

@dataclass
class Pet:
    name: str
    person_id: str

@dataclass
class Person:
    name: str

class PetFactory(DataclassFactory[Pet]):
    __model__ = Pet

def add_pets(name, values, *args, **defaults):
    return [PetFactory.build()]

class PersonFactory(DataclassFactory[Person]):
    __model__ = Person

    pets = PostGenerated(add_pets)

person = PersonFactory.build()

print(person)

This will output the following and silently ignore pets:

Person(name='lEoXzCHKCZmVcFmEeoqP')

Drawbacks and Impact
Improved developer experience.
Unresolved questions
No response

[!NOTE]
Check out all issues funded or available for funding here: https://polar.sh/litestar-org

If you would like to see an issue prioritized, make a pledge towards it!
We receive the pledge once the issue is completed & verified

This feels reasonable to me though I think the throwing of the error would only work for defining a field as PostGenerated or Use. It's also possible to define fields as just a classmethod with the same name as the field, but in that case, there's no way to know whether that method was added for some extra functionality or as a way to set the value of that specific field in the model.

Also, would you like to work on this?

I can, yes. Will try to come up with a PR asap.

I created a draft MR in https://github.com/litestar-org/polyfactory/pull/405 @guacs before I move ahead and tests etc, can you give it a look and see if you like the approach?


v-for type is unknown

Vue version
3.2.25
Link to minimal reproduction
https://sfc.vuejs.org/#eNrtlsGK2zAQhl9lMBTvwsZOdunFdQKFwlLooYfeohwUW461a0tCmiQbgt+9I9lOk5beStmDk4M9M9I/v0Y+fOfoszHJYS+iLMpdYaVBcAL3BhqudssYXbxiSrZGW4QzWFFBB5XVLcS0KWaKKfEWilKhsBUvBHx9tnpvfvDtF44czkwByDIDh1aq3ScfKt6Km8T6VZzGxCYDrk6U7rw6MFVo5RCM1cbBEkpRSSW++ygP2jvf7Zt0mN12Xm+8xuruPnQYZf6+mrTXfmUQ7S3Hi/ihj3rH8fMl4XQrsCa7C0rXoml0/EBNbkqPY8n//iw/hfLlT8ME6IL8tYXH3y140//fxtP7sDFNY5rGNI1pGtOlTJcyXco/sMHUxmNWnvbwR7BHAYrWNBwFRQB5KQ9QNNy5ZUwMhZzwy3oqJKjCfLtH1J7TKDjMKm2XLAqMRUD4C7ZY1HvpZViEfDvbogpphsy3DU8SNKvzGe56Ce5uIe0+8bOBrstT45369WnvoLeakld6y9OrE1Do8NT0h0kuJxhGX0pHCwk+q0a8BVIEOMoS6wwW8/mHkKEh0c7B87BvWPNxbsZdtZC7mrDyKrXlxas/iSpnhW60zQigy6E2JI61RDF2oVsITqOHqEfuWctN8uK0IjwPjdlQcCzKxq+HRYTiPmZRjWhclqauKjzUv7hE211Kb4ndK5StSIRrZ1urj05YEu7vhTp3UfcTW5jJNQ==
Steps to reproduce
The warning message doesn't show up on SFC Playground, but I get unknown type for group in v-for="group in groupList"
What is expected?
group should receive type IGroupTabData since it is iterating an array of IGroupTabData
What is actually happening?
I get unknown type for group in v-for="group in groupList"
System Info
System:
    OS: Linux 5.15 Ubuntu 22.04.1 LTS 22.04.1 LTS (Jammy Jellyfish)
    CPU: (8) x64 Intel(R) Core(TM) i7-9700 CPU @ 3.00GHz
    Memory: 5.97 GB / 15.45 GB
    Container: Yes
    Shell: 5.8.1 - /usr/bin/zsh
  Binaries:
    Node: 18.12.1 - ~/.nvm/versions/node/v18.12.1/bin/node
    npm: 9.2.0 - ~/.nvm/versions/node/v18.12.1/bin/npm
  Browsers:
    Chrome: 107.0.5304.121
  npmPackages:
    vue: ^3.2.25 => 3.2.41

Any additional comments?
I am using VS code for my IDE.
I solved this issue by doing type casting but I want to know if there's a better solution for this.
<p>{{ (group as IGroupTabData).name }}</p>

I've also tried below but it did not work:
const props = defineProps({
  groupList: {
    type: Array as PropType<IGroupTabData[]>,
    default: []
  }
});

This project I am working on did not had tsconfig.json.
After creating one with basic options, the issue was gone.


dbeaver/#20624  added Redshift Keywords.

Added Keyword  for Redshift

Thank you for contribution.

Looks nice now. Thanks for the contribution.


Faster gaussjacobi( 100000, .9, -.1 ) computation time

Fix #89
current master:
julia> @time gaussjacobi( 100000, .9, -.1 );
  4.262193 seconds (21.32 k allocations: 7.956 GiB, 30.34% gc time)

on v0.4.4:
julia> @time gaussjacobi( 100000, .9, -.1 );
  0.927837 seconds (5.28 k allocations: 1.320 GiB, 32.22% gc time)

with this PR:
julia> @time gaussjacobi( 100000, .9, -.1 );
  0.746913 seconds (3.59 k allocations: 1.255 GiB, 21.73% gc time)

Ah, I realized the conflicts I got was because of the git squash (#85).

Ah, I realized the conflicts I got was because of the git squash (#85).

I've added more tests, and seems like ready to merge!

I've added more tests, and seems like ready to merge!


Change Data Capture (CDC) Doc Update

Re: Change Data Capture (CDC)
Issue Description
We say this only works for "a targeted use case" but do not clearly explain what that use case is.
Suggested Resolution
Explain the use case we support

Thanks, @nstewart. Just a bit down the page, we do say a little more about potential usage of this feature.
@lhirata, can you work with @rolandcrosby to identify the use cases we want to highlight more clearly? If we're not ready to do so, we could also just remove that clause form the callout Nate is mentioning.


Fixed the OpenLiberty javac compilation issue.

This feature has already been added on other branch. Closing.


New Feature - Flippening Widget

Create a new section / set of widgets focusing on eth / btc  price and market cap comparison.
Probably should include a lot of projection features letting people play around with different input parameters and then having a simulation projecting eth / btc market cap / price into the future.
Open Questions:

Exact style of visualization.
Projection algorithm and input parameters
Focus only market cap flippening or also price flippening ?

I'm envisioning an extremely clean chart showing the historical ETH/BTC marketcap ratio, with a logarithmic y-axis. (Everyone focuses on the ETH/BTC price ratio when actually we should be looking at ETH/BTC marketcap ratio.) In terms of style I'm thinking something similar to "ETH supply—200Y projection", i.e. a simple line with a glow effect, no visible x- or y-axis, halo effect background, tooltip on hover.
In addition to the historical logarithmic ETH/BTC marketcap ratio line I would add:

a point showing the presale (where 2,000 ETH was being sold for 1 BTC and the total marketcap was 72M ETH)
a horizontal line for the flippening (with a 🐬  emoji somewhere)
a trend line going from the presale ratio to present day ratio (maybe with a 🦇  emoji somewhere)

shipped!


Support LOAD INPATH

https://github.com/hortonworks-spark/spark-llap/commit/7f4ea06262a48afaf2ac97d63b42d6f7e9c1ff23 close this.


Improve Accessibility of the Comment Textfield

The accessibility of the textfield for leaving a comment could use a little improving. A label would help identify it, as with the other fields. Also, for some reason, Firefox on Linux gets confused by it and reads a blank line, but Safari on a Mac has no problem identifying it as a textfield.

Some basic approaches mentioned in https://github.com/posativ/isso/pull/353#issuecomment-346217728

Should be fixed by https://github.com/posativ/isso/pull/861


ISPN-13382 Register JMX for Single Port Transport

https://issues.redhat.com/browse/ISPN-13382

@tristantarrant ISPN-11385 Remove remote-command-executor from schema I think you have the wrong JIRA number in this commit.

@tristantarrant but https://issues.redhat.com/browse/ISPN-13385 is related to the REST API: Retrieve a payload with mutable attributes and the value of each


Issue #25 Update the sbt-web plugin to the latest version so sbt-ecli…

Update sbt-web plugin

it appears that sbt-web now requires java7

There is sbt-web 1.3.0 already

Fixed by (ugrades to latest sbt-web)

#43


Implement /components/schemas/Result.Bgpv4Request

Please cover request and result:
/components/schemas/Result.Bgpv4Request
/components/schemas/Result.Bgpv4

We will revisit according to the requirement


Some RGBA images cause broken backgrounds due to incorrect RGB conversion

Currently, the load_image function in train_utils uses the PIL image.convert("RGB") method for converting the loaded 4 RGBA channel images to the expected 3 channel RGB format.
This is incorrect for some edge cases, and causes the background to become distorted.
As an example, loading this RGBA image results in heavy background artifacts. when used with the test code below:
from PIL import Image
from library.train_util import load_image

img = load_image("test.png")
Image.fromarray(img, 'RGB').save("test_convert.png")

To verify this issue, a LoRA was created using that single image as the dataset, using a caption that includes the words "white background". Seed and everything else was fixed between the two tests.
The top row is the latest code using a fresh VENV. The bottom row is the proposed fix, which I will create a PR for shortly. It uses the Image.alpha_composite function with a new blank white image to handle the alpha channel.

They're not artifacts. Those colors really exist in the png, they're just fully transparent. Dropping the alpha channel makes them visible again. Most image editors automatically set transparent pixels to black but for some reason the artist disabled that.

@feffy380 Interesting, seems like you're right. I kind-of just assumed it was some funky conversion logic but I guess that explains it.
I've created a quick script to check for hidden/corrupted backgrounds.
Currently testing it against my smaller 300K dataset to see what comes up. The current hit rate is 101 out of ~10000 images, only considering PNGs, so seems pretty rare.

@kohya-ss Thank you. That solution is indeed more optimal, I'll close my PR and will follow the one you linked instead.
Leaving the issue open until that PR gets merged in case someone else runs into this same behavior.


Crash when text has html tags

When i try to display a markup text which contains html tags (also has some malformed tags in it) the app crashes when using MarkdownUI
Tested with this text
**CC-Attribution-NonCommercial**\n\nBased on a scan of a replica of the \"Mayflower\" ship, processed by @spogna and available [here](https://skfb.ly/6u8T6).\n\nLike in the [Mosquito in Amber](https://skfb.ly/6xWCO), most of my attention was spent on normal maps, roughness and refraction settings to try to make this model as realistic as possible.\n\nI actually had to create a secondary internal surface for the bottle, and also relied on clear coat options for the glass material in order to try to give the model a feeling of thickness. \n\nTo try to get the opacity right, I used refraction on the outer layer (normals offset checked), and alpha-blending on the inner one.\n\nAll of the modeling was done with Blender, but the critical part for the model aesthetics relies on Sketchfab materials and post-processing.\n\n
\n>\n> Another ship in a bottle\n> \n> Has got no place to sail\n> \n> To see the waves getting darker every hour\n> \n> The stars have lost their power\n> \n> In the sky\n> \n\n
\n**Beck**, *Ship in a bottle*
Screenshot from Xcode
I solved this by stripping out html tags.
  func deleteHTMLTags() -> String {     return self.replacingOccurrences(of: "<[^>]+>", with: "", options: .regularExpression, range: nil)   }
The problem is that when you get markdown text from an api and which can contain malformed Markdown that includes html tags or other user created malformed tags, it could crash your app.
Questions if this this should or could go into MarkdownUI ?

I can't reproduce the crash with the text you provided, neither in the simulator nor in my device. Tested with Xcode 12.4 / iOS 14.4.2.
Here's a couple of screenshots:

i had another look and you're right. The problem was on my site and how i structured my views.
I used a state var to show/hide the text and also a function to get the text. The problem lies somewhere there.
if self.showMoreDescription {    Markdown(Document(getDescrption(showMore: true))                     ).markdownStyle(                       DefaultMarkdownStyle(                         font: .system(size: 12, weight: .regular),                         foregroundColor: .white,                         codeFontSizeMultiple: 1                       )                     ).accentColor(Constants.yellow)              }
I'm closing the issue. Sorry for the trouble...

i had another look and you're right. The problem was on my site and how i structured my views.
I used a state var to show/hide the text and also a function to get the text. The problem lies somewhere there.
`if self.showMoreDescription {
Markdown(Document(getDescrption(showMore: true))
).markdownStyle(
DefaultMarkdownStyle(
font: .system(size: 12, weight: .regular),
foregroundColor: .white,
codeFontSizeMultiple: 1
)
).accentColor(Constants.yellow)
}`
I'm closing the issue. Sorry for the trouble...

same issue, how you fix it?

i had another look and you're right. The problem was on my site and how i structured my views.
I used a state var to show/hide the text and also a function to get the text. The problem lies somewhere there.
`if self.showMoreDescription {
Markdown(Document(getDescrption(showMore: true))
).markdownStyle(
DefaultMarkdownStyle(
font: .system(size: 12, weight: .regular),
foregroundColor: .white,
codeFontSizeMultiple: 1
)
).accentColor(Constants.yellow)
}`
I'm closing the issue. Sorry for the trouble...

@ppoh71  Hi, i find  the same issue, when i use  html   render the markdown, page reopen  crashed.   Do you fix it ?

How do you render HTML markdown using Markdown UI? I just get the plain Html tags and text


Usage information incorrect: loadConfig doesn't exist

Using v2.2.1, there is no exported member for loadConfig in graphql-config, unless I'm missing something here?
import {loadConfig} from 'graphql-config';

async function main() {
  const config = await loadConfig();

  const schema = await config.getDefault().getSchema();
}

I also assume this will return a GraphQlConfig type, which then has no exported members for getDefault().

@mrowles The API you're using is v3 not v2, please run:
$ yarn add graphql-config@next


[BUG] ResourceManager - Subscription Id and Name/DisplayName fields mapped incorrectly

Library name and version
Azure.ResourceManager 1.0.0-beta.6
Describe the bug
The Subscription response has the Id and Name/DisplayName fields mapped the wrong way around.
Expected behavior
The fields are mapped correctly.
Actual behavior
Issues:

Name is actually the SubscriptionGuid
DisplayName is actually the SubscriptionGuid
SubscriptionGuid is the Name

Reproduction Steps
var resourceManagerClient = new ArmClient(new DefaultAzureCredential());
await foreach (var subscription in resourceManagerClient.GetSubscriptions().GetAllAsync(cancellationToken))
{
  // Do something here
}

Environment
No response

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Appears to have been fixed in Azure.ResourceManager v1.0.0-beta.7.
https://github.com/Azure/azure-sdk-for-net/pull/25807


add status_code for response

Not sure if this is something that you want to do Michael; also, not sure if we should expose the rest of the response parameters...

Hi @fferegrino Thanks so much for the PR. It looks solid. But I'm having a hard time of seeing when I would use this in a HTML / Web flow outside of APIs which wouldn't use the template wrapper.
What did you have in mind for responses? For 201s that only makes sense in the terms of APIs. For 200s, that's the default. 302 is a return RedirectResponse (approx, class name?). The one that may be something we'd use is 404 and showing a proper error page.
I might just be game to accept this just to make pretty 404s easier, but what else would we do with this capability?

Yeah, styled 404's was what I had in mind.

Hi @fferegrino
Thanks again for this PR. It is a good idea to support 404s. But I think I'd like a different way to communicate it.
Let's say you request this URL:
https://site.com/items/17
We'd look in the DB, and if there is an item with ID 17, we'd return that to the template and render it. But if it's not found, we want to return the HTML of an error page, the status 404, but leave the URL in place.
I don't see how your PR makes that work without going to a new view/URL. But I love the idea of your PR for this.
So I took the ideas and added the feature. Check it out here:
github.com/mikeckennedy/fastapi-chameleon#friendly-404s-and-errors
With this, any time you want a friendly 404 page for a given view, just call fastapi_chameleon.not_found(). It'll raise a special exception noticed by this library. That will tell it to render templates/errors/404.pt and return a 404 status code but leave the URL in place without redirecting.
Thanks!

Cool, closing this issue!


fix a behavior the toggle button when a modal is open

Current behavior:
It switches the visibility of the main panel, but not a modal panel
Variants:

Hide the toggle button when a modal panel is showing

Toggling a visibility a modal panel with the admin panel together

the second option is selected


Toolbar headline section created

Hello !
I just created a toolbar and a beautiful attractive design in my header section of the portfolio.
I included some svg images and other elements.
Please can you check and give me any feedback to help improve my skills ?
yours faithfully 👌

thanks alot for your reviews !
I really appreciate and i'm confident that i've follow every instrunction as mentionned above.
I just committed my last changes about the work I did.
Please let me know if everything is doing well know
🤗


Run graceful recovery tests in pipeline

Proposed changes
Problem: As a user, I want to automate graceful recovery tests
Solution: Added a job to run graceful recovery tests in pipeline
Testing: Running it as part of pipeline
Please focus on (optional): If you any specific areas where you would like reviewers to focus their attention or provide
specific feedback, add them here.
Closes #1910
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[x] I have read the CONTRIBUTING doc
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have checked that all unit tests pass after adding my changes
[x] I have updated necessary documentation
[x] I have rebased my branch onto main
[x] I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

Release notes
If this PR introduces a change that affects users and needs to be mentioned in the release notes,
please add a brief note that summarizes the change.

Looking in the pipeline that was run for the functional tests, looks like because of some label the test is getting skipped:
Will run 2 of 17 specs
------------------------------
[BeforeSuite]
/home/runner/work/nginx-gateway-fabric/nginx-gateway-fabric/tests/suite/system_suite_test.go:248
  [SKIPPED] in [BeforeSuite] - /home/runner/work/nginx-gateway-fabric/nginx-gateway-fabric/tests/suite/system_suite_test.go:130 @ 05/29/24 00:22:[45](https://github.com/nginxinc/nginx-gateway-fabric/actions/runs/9278039607/job/25528584310?pr=2045#step:14:46).584
S [SKIPPED] [0.015 seconds]
[BeforeSuite]
/home/runner/work/nginx-gateway-fabric/nginx-gateway-fabric/tests/suite/system_suite_test.go:248

  [SKIPPED] NFR tests can only run in GKE
  In [BeforeSuite] at: /home/runner/work/nginx-gateway-fabric/nginx-gateway-fabric/tests/suite/system_suite_test.go:130

Be sure to name your branches appropriately so the PR gets tagged the right way. These are test changes, not a feature or enhancement.

Why did the bot add the enhancement label and remove the test label 🤔

@bjee19 Because the branch is named with feat instead of test.

Be sure to name your branches appropriately so the PR gets tagged the right way. These are test changes, not a feature or enhancement.

I'll ensure to update the tag before merge! @bjee19 @sjberman


Refactor some sequential holidays

Proposed change

Refactor some sequential holidays (for cleaner code).
Type of change

[ ] New country/market holidays support (thank you!)
[ ] Supported country/market holidays update (calendar discrepancy fix, localization)
[x] Existing code/documentation/test/process quality improvement (best practice, cleanup, refactoring, optimization)
[ ] Dependency update (version deprecation/upgrade)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (a code change causing existing functionality to break)
[ ] New feature (new python-holidays functionality in general)

Checklist

[x] I've followed the contributing guidelines
[x] I've run make pre-commit, it didn't generate any changes
[x] I've run make test, all tests passed locally

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

28 of 28 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
10963

Relevant Lines:
10963

💛  - Coveralls


Quota type

NPE fix in the BnP
API consistency for QuotaType.
Quota set, tries all nodes before rethrowing the last exception. ( Consistent with verifyOrAddStore behavior, where it skips the failed nodes while adding the store).

Thanks for making these changes! LGTM.


Release 5.0.5

Need #716, then rebase it.

some PR was backported to fluent-package-v5, then rebased with it.

As https://github.com/fluent/fluent-package-builder/pull/732 was merged, it should pass all test cases.

I've forgot updating fluent-plugin-systemd.

updated to ship fluent-plugin-systemd 1.1.0.


Regenerate requirements using python 3.6

Sure enough, this breaks for new versions of python.

I don't know what to make of this: generating the requirements using 3.6 or 3.7 makes newer versions fail. Generating with anything but 3.6 makes 3.6 fail.

I think the right thing to do is to have different versions for different versions of Python. I have no idea idea how to generate that but there's py_version in requirements file format which can be used to declare that version x is for Python older than some version and version y is for Python newer than the version.


Add Wordpress

For personal (and for some maybe public) use I would like to include a blogging platform like Hugo or Ghost. What do you guys think? If you are also interested what platform would you choose and why?

I guess it depends how much meat you want around it and what you actually want to achieve.
For example, Hugo (and Jekyll) are static content generators that generate static HTML from markdown committed to git. They run on any web server that serves HTML.
Ghost on the other hand (and similar things like Wordpress) are full blown CMSs that require being "installed" and run from on an app server...
I've used all of the above, each of them have their positives and negatives - you just need to pick the right tool for the job.
In terms of ansible-nas, it might be fun to add wordpress, it's a nice platform to use...Ghost I know less about but would still accept a PR if there was interest in creating one.

I too would vote for WordPress.

Portainer has a wordpress template that is easly set up.

Portainer has a wordpress template that is easly set up.

I can see how a WordPress integration may be useful. As @PurpleNinja225 pointed out, you can easily spin up a WordPress container using Portainer, but you will need to provide all  the Traefik middleware labels yourself, which may or may not be a problem  for the user.
One issue with ansible-nas adding WordPress support would be to allow the user to define multiple websites, since for some people it may not be a "one off" thing.
Personally I'm going the Portainer route right now, and may think about contributing to this issue with  a PR in the future if I find easy enough to implement multiple websites support


Elastic BackPressuredBufferSubscriber

The current implementation of BackPressuredBufferSubscriber stops the publisher when the buffer is full and will not continue it until it's empty again. This can be suboptimal in case both the publishing and subscribing rates are changing.
Another parameter could be introduced in BackPressure for a lower threshold, below which the upstream could be already signalled to continue. The default value could be half the buffer size.
Introducing this would allow us to have more elastic data pipelines.
I am not sure if the current implementation can be easily changed to support this, alternatively it could be done as a new overflow strategy.

Here is some, maybe naive, implementation from the top of my head
https://gist.github.com/akozhemiakin/4934a5ab463f7d2104c3bb35dd88a2a8
If it looks fine I can prepare pull request based on it

Thanks for replying and for looking into this. Sorry, but i'm not an active user of the project. I opened this ticket a long time ago while I was experimenting with the library, so it would take a while for me to refresh my brain on this. I suggest creating a PR anyway if you also find this valuable. If not just let me know and I can close the ticket.


Permalinks

As a user, I want to click a link for a provider and navigate to a page that i can then share the url with a friend or bookmark for later

from MSD: Ability to show single provider on map

I think this also covers "Save the results" functionality that MSD have asked for

#closes 7


Missing XML Response Content Type with IHttpActionResult and UseXmlSerializer

I have a bit of a strange situation. I have reproduced the problem with the dummy action below. Basically, if the return type of the action is IHttpActionResult and the Web API config.Formatters.XmlFormatter.UseXmlSerializer setting is used then Swagger UI does not list XML in the response content types. If I look at the Swagger definition, the 'produces' array does not contain text/xml or application/xml which I assume is the underlying reason for the UI issue.
[SwaggerResponse(HttpStatusCode.OK, Type = typeof(Blah))]
[SwaggerResponse(HttpStatusCode.NotFound)]
public IHttpActionResult Get(int id)
{
    if (id != 1)
    {
        return NotFound();
    }

    return Ok(new Blah { Id = id, Text = "test" });
}

If I change the return type to object, or revert back to the standard data contract serializer, the content type list is populated correctly. The actual API works fine in all cases. It is just Swagger which is not cooperating.

I think I know what's going on. When I inspect the ApiExplorer, WebApi's built-in metadata layer which SB heavily depends on, it also doesn't list the XmlSerializer as a supported response formatter.
This is due to the dynamic nature of IHttpActionResult. When ApiExplorer tries to determine the SupportedResponseFormatters for an action, it asks each configured formatter if it can serialize the action's response type (see MediaTypeFormatter.CanWriteType). Unlike the JsonSerializer, which is less strict, the XmlSerializer returns false in the case of IHttpActionResult.
To workaround this issue, you can wire up a simple OperationFilter that hardcodes the supported mime-types for each operation:
// SwaggerConfig.cs
c.OperationFilter<MimeTypesOperationFilter>();

// MimeTypesOperationFilter.cs
public class MimeTypesOperationFilter : IOperationFilter
{
    private static readonly string[] SupportedAccepts = new[] { "application/xml" };
    private static readonly string[] SupportedContentTypes = new[] { "application/xml" };

    public void Apply(Operation operation, SchemaRegistry schemaRegistry, ApiDescription apiDescription)
    {
        operation.consumes = SupportedAccepts;
        operation.produces = SupportedContentTypes;
    }
}

Closing due to inactivity


hello, where is the code to input the images and ground truth?

I couldn't find where the ground truth code  input into the model.  And ground truth shape is?

It's under core/sample.


Visdom Is Extremely Slow With Many Points

I'm using Visdom to plot the training loss on every mini-batch onto a line plot. After only a few epochs (around 2000 mini-batches = 2000 points on the line plot,) my Visdom browser window becomes extremely slow and unresponsive.
Is there a way for Visdom to optimize plotting line plots containing many points?

Hi @enchainingrealm. As of now we don't have backend optimizers for averaging out points on dense plots to reduce density, and standard plotly plots choke under heavy load. Still, so long as you don't have too many of these dense plots, you can render using webgl as noted in our docs and demonstrated in our python demo.

Visdom is still slow with WebGL enabled, but now at least its usable. I'm closing this issue since you mentioned that averaging out points that live in the same x-coordinate (on screen) is not yet implemented, and that was what I was looking for.


matchAttribute, matchClass and shouldMatch ?

Hi there!
Instead of writing huge css selectors like this:
$$("[onclick*=submit]:not([name*=search]):not([class*=search]),[onclick*=subscribe]:not([name*=search]):not([class*=search]),[name*=submit]:not([name*=search]):not([class*=search])")
I wanna write:
$$("[onclick*=submit],[onclick*=subscribe],[name*=submit]")
.exclude(matchName("search"))
.exclude(matchClass("search"))
.exclude(matchAttribute("search")) // any attribute with matched substring
.exclude(matchAttribute("bla-attr", "search")) // the attribute with matched substring
I have started to code this but I realized that I need an architect vision...
There are a lot of methods to work with text but the same can't be said about the classes or other attributes.
What methods should I extend?
Have a nice day!

@vasilevichra We have a performance problem with this approach.

When you write huge CSS selector, selenium webdriver executes ONE request to the browser. It's fast.
When you write $$("css").exclude(C1).exclude(C2) etc., webdriver will need to execute many requests:

load all elements matching "css"
iterate all elements, and check every one of them, probably executing a separate request for each element.
As a result, we get the famous "N+1 select" problem.

Close old issue. Feel free to reopen and start disscussion again if it still needs to implement.


Vocable iOS - Recents category feasibility

Recently on iOS we removed the concept of "Preset" categories and made all categories custom categories. This means the preset categories that come with the app can be deleted and edited.
There are some exceptions to this rule, e.g. Recents; you  can not add/delete phrases for the Recents category.
This ticket is to capture what we want to do with Recents. The original intent was to house the most recently used phrases, but now that the category can be deleted do we still want the Recents functionality?  Also if the category is renamed, the phrases are still saved to the category.

Just a note that the term "preset" is still important in referring to the predefined, out-of-the-box categories and phrases. I guess it refers more to a state of the app in that a newly installed app uses presets and the app can be returned to presets by clicking "Reset" in Settings.


Stringify Duration field in Stop

Currently the duration field in log.Stop is set to be a time.Duration type, which is then formatted by the handlers. see here:
https://github.com/apex/log/blob/a0bcece0a725e2b62f01d948b25ba3c22ebfb792/entry.go#L144-L146
With the JSON handler, this ends up looking like this:
"duration":1935807,
Calling .String() on the duration ends up with a more human-readable:
"duration":"1.935807ms"
I realize that the JSON handler is meant to be machine-parseable, but think the second is still machine-parseable while being much more human-readable.
Any thoughts? Would be glad to submit a PR with this change.

I'd prefer to keep the json one purely "structured" so you can query against those as numerical values, though I'll admit nanosecond resolution is a bit overkill haha


flake in TEST_P(IntegrationTest, DISABLED_MultipleRequests) {

https://dev.azure.com/cncf/envoy/_build/results?buildId=139488&view=logs&j=767be981-567e-57d8-68c3-2140ede0a0bd&t=67a76341-37f4-5b49-db38-acbb8a59af82&l=637
[ RUN      ] IpVersions/IntegrationTest.MultipleRequests/IPv6
[2023-06-06 22:31:40.729][361706][critical][assert] [contrib/generic_proxy/filters/network/test/integration_test.cc:200] assert failure: result. Details: Timed out waiting for data. Got '\0\0\0Mfake_fake_fake|service_name_0|/path_or_anything|hello|version:v1;stream_id:1;\0\0\0Mfake_fake_fake|service_name_0|/path_or_anything|hello|stream_id:2;version:v1;', waiting for 81 bytes.
[2023-06-06 22:31:40.730][361706][critical][backtrace] [./source/server/backtrace.h:104] Caught Aborted, suspect faulting address 0x3e9000584ea
[2023-06-06 22:31:40.730][361706][critical][backtrace] [./source/server/backtrace.h:91] Backtrace (use tools/stack_decode.py to get line numbers):
[2023-06-06 22:31:40.730][361706][critical][backtrace] [./source/server/backtrace.h:92] Envoy version: 0/1.27.0-dev/test/RELEASE/BoringSSL
[2023-06-06 22:31:40.730][361706][critical][backtrace] [./source/server/backtrace.h:96] #0: __kernel_rt_sigreturn [0xffffa07e178c]
[2023-06-06 22:31:40.730][361706][critical][backtrace] [./source/server/backtrace.h:96] #1: abort [0xffffa0556aac]
[2023-06-06 22:31:40.741][361706][critical][backtrace] [./source/server/backtrace.h:96] #2: Envoy::Extensions::NetworkFilters::GenericProxy::(anonymous namespace)::IntegrationTest::waitForUpstreamRequestForTest() [0xa262cc]
[2023-06-06 22:31:40.751][361706][critical][backtrace] [./source/server/backtrace.h:96] #3: Envoy::Extensions::NetworkFilters::GenericProxy::(anonymous namespace)::IntegrationTest_MultipleRequests_Test::TestBody() [0xa28ca8]
[2023-06-06 22:31:40.761][361706][critical][backtrace] [./source/server/backtrace.h:96] #4: testing::internal::HandleExceptionsInMethodIfSupported<>() [0x1bdfe68]
[2023-06-06 22:31:40.772][361706][critical][backtrace] [./source/server/backtrace.h:96] #5: testing::Test::Run() [0x1bdfd08]
[2023-06-06 22:31:40.782][361706][critical][backtrace] [./source/server/backtrace.h:96] #6: testing::TestInfo::Run() [0x1be0e58]
[2023-06-06 22:31:40.792][361706][critical][backtrace] [./source/server/backtrace.h:96] #7: testing::TestSuite::Run() [0x1be1bc4]
[2023-06-06 22:31:40.802][361706][critical][backtrace] [./source/server/backtrace.h:96] #8: testing::internal::UnitTestImpl::RunAllTests() [0x1bef0cc]
[2023-06-06 22:31:40.812][361706][critical][backtrace] [./source/server/backtrace.h:96] #9: testing::internal::HandleExceptionsInMethodIfSupported<>() [0x1beeb80]
[2023-06-06 22:31:40.823][361706][critical][backtrace] [./source/server/backtrace.h:96] #10: testing::UnitTest::Run() [0x1bee9dc]
[2023-06-06 22:31:40.833][361706][critical][backtrace] [./source/server/backtrace.h:96] #11: Envoy::TestRunner::RunTests() [0x15520bc]
[2023-06-06 22:31:40.843][361706][critical][backtrace] [./source/server/backtrace.h:96] #12: main [0x1550c84]
[2023-06-06 22:31:40.843][361706][critical][backtrace] [./source/server/backtrace.h:96] #13: __libc_start_main [0xffffa0556e10]

seen this a couple of times, finally getting around to filing

will try to fix this.