creating no_sir.plug: plug file for no_sir.py

no_sir.py is  to tell users to not use 'sir' when they do. #337
Reviewers Checklist

[ ] Appropriate logging is done.
[ ] Appropriate error responses.
[ ] Handle every possible exception.
[ ] Make sure there is a docstring in the command functions. Hint: Lookout for
botcmd and re_botcmd decorators.
[ ] See that 100% coverage is there.
[ ] See to it that mocking is not done where it is not necessary.

You need to put all of the code into one PR , https://github.com/coala/corobo/pull/448 .
I see you have created lots of PRs for this one issue.  If you do that again, you will be unassigned.  Ask for help.


Not setting options for {% element %} throws error

Hi,
if you don't set options for element it will fail as it passes null to $compiler->subcompile()
in Lib\Twig\Node\Element.php
line: 65
$options = $this->getNode('options');
if ($data!== null) {
    $compiler->raw(',');
     $compiler->subcompile($options);
}

I think it should check for $options not $data.
Thanks.

Good point 👍 , could you make a PR, don't have time to test and fix it right now

Has this been already fixed?
https://github.com/WyriHaximus/TwigView/blob/5da1061417bfbcf00482cf5a13498f05fcfd2419/src/Lib/Twig/Node/Element.php#L68-L72


feat: upgrade from node 16 to 20

https://github.com/JarvusInnovations/background-action/issues/185
Upgrades node to use v20 and also upgrades some of the internal actions to v4 so that they also use v20.
Not quite sure how to test this so any direction here would be helpful :)

You can automate most of these changes by using GitHub's Dependabot.

https://github.com/jayvdb/nose3/pull/7/files

Thank you @Multimo 🎉
@ryon @themightychris can you folks please take a look on that? Now all Actions shows warning about this deprecation. 🙏🏻

Can haz merge? 😄

@till: I need to check with @themightychris that the hologit part works with Node 20, if so we should be good to merge this

@till: @cclauss: I checked with @themightychris, the changes you made are correct, however, we need to push an update for hologit that supports Node 20 for them to work. I can't commit to when this would happen but he's aware of the issue. Once hologit supports Node 20, we should be able to merge this.

Thank you very much!


Revert "Allow using videos as screenshots"

Neither of the 2 videos work on Safari, we can't deal with codec issues. I used Gifski.app to convert one video and recreated the rest of the screenshots.
Reverts refined-github/refined-github#5271

Neither of the 2 videos work on Safari, we can't deal with codec issues.

It's Safari's strict sandboxing instead of codec issues:
[Error] Blocked script execution in 'https://user-images.githubusercontent.com/44045911/149634307-1ed0e554-6957-4afa-8458-9c5bb4f80ebb.mp4' because the document's frame is sandboxed and the 'allow-scripts' permission is not set.
[Error] Failed to load resource: Plug-in handled load (149634307-1ed0e554-6957-4afa-8458-9c5bb4f80ebb.mp4, line 0)
[Error] Blocked script execution in 'https://user-images.githubusercontent.com/44045911/149634307-1ed0e554-6957-4afa-8458-9c5bb4f80ebb.mp4' because the document's frame is sandboxed and the 'allow-scripts' permission is not set. (x32)

To keep video support, there are some options

It just seems to me like too much hassle, for something we don't really need.
And I don't see GIFs disappearing any time soon either, without them the Internet would just collapse in on itself ;)

It just seems to me like too much hassle, for something we don't really need.

With the latter option all things needed is to prefix a URL. Is that counted as "too much hassle"?

And I don't see GIFs disappearing any time soon either, without them the Internet would just collapse in on itself ;)

This is about GIF as a format, not as a name. "GIF" you see everywhere are in fact videos and <video>.

It just seems to me like too much hassle, for something we don't really need.

It just seems to me like too much hassle

So you are bother converting videos to GIFs instead of simply prefixing a URL to the videos?

for something we don't really need.

So is Safari being able to directly open one or two videos, which would play just fine in the Safari's extension options and other browsers, something we really need?
This PR and what this PR is reverting doesn't even make it into the latest Safari release.

I really don't understand why you're spending so much energy on this non-issue.  PNG and GIF work out of the box and do not need any "solution" whatsoever. I don't know how else to put it: Zero code needed, zero bikeshedding needed.


Introduce new Pod annotation for assigning specified MAC address to container veth

Description
Introduce new Pod annotation "cni.projectcalico.org/hwAddr" that will be used to assign a specified MAC address to the
container-side Veth interface
Added new unit tests that pass, as well as manually verified by creating a container within a cluster and examining the MAC address of eth0 from within the container to make sure it matched the address specified by annotation.

Related issues/PRs
fixes 5196

Todos

[x] Tests
[ ] Documentation
[x] Release note

Release Note

Add new Pod annotation for assigning specified MAC address to container veth

Reminder for the reviewer
Make sure that this PR has the correct labels and milestone set.
Every PR needs one docs-* label.

docs-pr-required: This change requires a change to the documentation that has not been completed yet.
docs-completed: This change has all necessary documentation completed.
docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

release-note-required: This PR has user-facing changes. Most PRs should have this label.
release-note-not-required: This PR has no user-facing changes.

Other optional labels:

cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

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


Create CMP and documentation on how to install argocd plugin

https://argo-cd.readthedocs.io/en/stable/operator-manual/config-management-plugins/#sidecar-plugin

https://github.com/helmwave/argocd-plugin/

Not just some barely working shell script that we don't test, we need separate binary that will work as a CMP. And that will accept arguments


[Bug]: Map breaks the responsiveness in contact page and miss-alignment of text in small screen sizes

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?

The Maps breaks the whole reponsiveness and layout of the page i want to fix this issue . please assign me .

Add Screenshots
No response
What browsers are you seeing the problem on?
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

Thankyou ! :


Onboarding widget broken

Which at_widget package you find issue with?
at_onboarding_flutter
On what platforms did you experience the issue?
MacOS
What version are you using?
at_onboarding_flutter : trunk
Flutter 3.24.0 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 80c2e84975 (9 days ago) • 2024-07-30 23:06:49 +0700
Engine • revision b8800d88be
Tools • Dart 3.5.0 • DevTools 2.37.2
What happened?
Onboarding widget is crashing with errors.
Steps to reproduce
1. Follow this guide - https://docs.atsign.com/tutorials/flutter-and-atsdk-app
2. Do step 2 from here - https://pub.dev/packages/at_onboarding_flutter
3. Pin at_onboarding_flutter to trunk of this repo
4. Try to run the app
5. Press "onboard an atsign"
6. Error occurs

Additional info
Note that prior to trying the trunk code, I tried 6.1.7 and it also hung:
Press "onboard an atsign" > press "already have an atsign" > press "upload atkeys" then it hangs forever.
Relevant log output
A Dart VM Service on macOS is available at: http://<IP_ADDRESS>:55648/rzVdEDmRsAI=/
The Flutter DevTools debugger and profiler on macOS is available at:
http://<IP_ADDRESS>:9100?uri=http://<IP_ADDRESS>:55648/rzVdEDmRsAI=/
flutter: AtOnboardingInitScreen: error - Null check operator used on a null value
flutter: #0      OnboardingService._getClientServiceForAtsign (package:at_onboarding_flutter/services/onboarding_service.dart:79:42)
flutter: #1      OnboardingService.onboard (package:at_onboarding_flutter/services/onboarding_service.dart:101:35)
flutter: #2      _AtOnboardingStartScreenState._init (package:at_onboarding_flutter/screen/at_onboarding_start_screen.dart:56:47)
flutter: <asynchronous suspension>
flutter:
flutter: PASS
2024-08-08 17:05:34.776 npt_flutter[82542:55862346] WARNING: Secure coding is automatically enabled for restorable state! However, not on all supported macOS versions of this application. Opt-in to secure coding explicitly by implementing NSApplicationDelegate.applicationSupportsSecureRestorableState:.
flutter: AtOnboardingInitScreen: error - Null check operator used on a null value
flutter: #0      OnboardingService._getClientServiceForAtsign (package:at_onboarding_flutter/services/onboarding_service.dart:79:42)
flutter: #1      OnboardingService.onboard (package:at_onboarding_flutter/services/onboarding_service.dart:101:35)
flutter: #2      _AtOnboardingStartScreenState._init (package:at_onboarding_flutter/screen/at_onboarding_start_screen.dart:56:47)
flutter: <asynchronous suspension>
flutter:
flutter: PASS
flutter: AtOnboardingInitScreen: error - Null check operator used on a null value
flutter: #0      OnboardingService._getClientServiceForAtsign (package:at_onboarding_flutter/services/onboarding_service.dart:79:42)
flutter: #1      OnboardingService.onboard (package:at_onboarding_flutter/services/onboarding_service.dart:101:35)
flutter: #2      _AtOnboardingStartScreenState._init (package:at_onboarding_flutter/screen/at_onboarding_start_screen.dart:56:47)
flutter: <asynchronous suspension>
flutter:

### Flutter analyze output

_No response_

### Flutter doctor output

_No response_

Related:
#699

@sachins-geekyants @Sardar1208 if you need an example: https://github.com/atsign-foundation/noports/tree/xc-npt-desktop-mvp

Found the root cause of the issue to be the file_selector package, I am repurposing this ticket to replace file_selector with file_picker, as it is better maintained.
When we first started using file_picker, it didn't have desktop support so we continued to use file_selector. But now file_picker seems to be the simpler option.

I know that both at_onboarding_flutter and at_backupkey_flutter use file_selector, there may be other packages which I am not aware of.

The commits in #887 serve as a good example of the changes that need to be made, but #887 is based on the wrong branch.

Our packages that uses file_selector:

[x] at_onboarding_flutter
[ ] at_backupkey_flutter
[ ] at_contacts_group_flutter


Added "name" Attribute to metadata.rb

Without it, a Vagrant run using Berkshelf fails, e.g.:
stderr: The following error occurred while reading the cookbook `application_nodejs':
Ridley::Errors::MissingNameAttribute: The metadata at '/tmp/d20150216-17470-1laf605' does not contain a 'name' attribute. While Chef does not strictly enforce this requirement, Ridley cannot continue without a valid metadata 'name' entry.

Fixed in #10


ES-1832: Move CPI generation from plugins to corda-sdk

ES-1832
Move the logic from creating a CPI file from the Corda CLI package plugin into a new Corda SDK library.

The logic for creating a CPI file is available in the Corda SDK library
The functionality of the Corda CLI plugin remains unchanged but uses the CPI file logic from the library:

Test coverage added for the SDK logic largely overlaps with the existing test coverage of the CLI plugin; however it's better to keep having both

The dependency from the Network plugin has been updated to use SDK code
I attempted to reduce the amount of refactor to the necessary minimum:

All logic remained unchanged, including internals of CPI building and signing, and CLI plugin interface
Most of the tests were copied and refactored to use the library; certain new/missing coverage was added as well

@anton-subbotin I started the combined worker smoke tests here - these were intended to be kicked off manually on plugin-related PRs

@anton-subbotin I started the combined worker smoke tests here - these were intended to be kicked off manually on plugin-related PRs

Thank you @YashNabar , all passed!


Make CI work on forks

Unfortunately github actions CI will only work on forks if there are no custom secrets.
So the npm package and docker image release process will have to be ran manually instead of via CI. Those are also in a separate workflow and github actions still won't run the workflow that does not depend on any secrets.
It's a worthy tradeoff to make it easier for contributors to get feedback on their changes.

This might be a new feature, but I think GitHub Actions supports running builds in forks with secrets, requiring approval before doing so (to avoid people trying to steal secrets using echo $ENV_VAR or whatever): https://docs.github.com/en/actions/reference/environments#environment-secrets


make methods name standard

use [A/HC/LC Pattern](A/HC/LC Pattern)

Naming Convention
Actions

get: get resource value
set: modify resource value
add: add a new resource
use: utilities
check: perform a validation
calculate: perform some calculation
draw: draw an structure

Exceptions

cell
multi_cell
table


Automatically reload code (restart server?)

If I understand correctly, currently if I make a change to any file I need to stop and start again the demo server to see the effect.
If would be nice if there was a mode that was monitoring all the files (or a subset of the files) and it automatically reloaded the development server.

Agreed.
This can be a good first issue, assuming a nice library in Rust can be found that does this easily, or alternatively, some kind of external command line tool (prefer written in Rust) that does it externally (like procman, watchman, etc)

@szabgab @jondot
you can install and work with cargo watch crate: https://crates.io/crates/cargo-watch

Looks like this might cut it, closing -- please reopen if needed


Implement LRU TLB

Implement LRU TLB with doubly link list.
However, the error of not being able to execute correctly has not been resolved yet.

Have you figured out the root cause of failing to execute init process?

not yet, I am still working on it.
Jim Huang @.***> 於 2024年7月6日 週六 下午11:21寫道：

Have you figured out the root cause of failing to execute init process?
—
Reply to this email directly, view it on GitHub
https://github.com/sysprog21/semu/pull/48#issuecomment-2211796322, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BCS72JZD6IHI2J3QAPERJZDZLADRHAVCNFSM6AAAAABJ7M4K5OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJRG44TMMZSGI
.
You are receiving this because you authored the thread.Message ID:
@.***>


git-secret

I just discovered git-secret on homebrew and thought of how this might have alleviated earlier teething pains we were having :)
Take with several grains of salt because I have no idea if this is helpful or what the state of the project is now, but maybe it makes for good inspiration? In principle, it makes sense that everyone on the team should have their own gpg keys and then be able to decrypt the project files when they've been added to the project.
Hope y'all are doing well!

I will check it out, but it's been a busy couple weeks so I'm keeping the issue open until I actually do check it out.  Thanks @JaKXz


Added hello world in Hmong

Account number: 6f9a58d18f9c544db46a0bc988aae8bee1ab7d74f04e490f6bafc9a6c8eb4c23

@sai-koumudi It's not listed in the ISO codes. Where did you get the code from?


Player marker is higher than map marker

The regular marker is located lower in the layer than the player marker. Is it possible to fix this?

A playersAboveMarkers config option has been added to index.html's liveAtlasConfig. It defaults to true to match old behaviour, but can be set to false which will display player markers below other marker types.


Setup periodic testing on internal openstack

We need to setup odo periodic tests on internal openstack

Can we also try to run our regular integration tests on that cluster? Mentioning it here since we discussed this in Cabal and no one seemed to object to the idea.

We can definitely run them, but we have the same problem that we had with container pipeline service, we cannot communicate.  So no webhooks from GitHub. Girish said he will try to chase that down on the side and I will also look into options.
Not having webhooks also means we cannot do stuff like block merge etc
Right now, the only option is polling, even then logs will not be available to those outside redhat, unless we can find a way to export them.
Will open a separate issue for that

I have setup a periodic job poc that tests unit tests and build binary on fedora 31 in PSI.
Next target:

add rest of tests
add win and mac

As per our last bjn call i am adding @prietyc123 to work on setting up mac and Wnidows host. We got a mac mini server and @prietyc123 is following up that front to set it up in bangalore lab from odo side. Also new windows host will  be pre and post configured with certain requirement to test run.

Tasks need to be done for periodic jobs:

[ ] Set up the machine with minimal configuration

[x] fedora
[ ] Windows
[ ] macOS

[ ] Run unit tests

[x] fedora
[ ] Windows
[ ] macOS

[ ] Run integration and e2e tests

[ ] fedora
[ ] Windows
[ ] macOS

We have setup a windows vm with high end configuration, installed the pre-req. But facing few challenges like:

ssh to windows (Releng-key failed to provide password less auth on windows. Though we have work around by passing password in the database, but need to figure out the long term solution)
Jenkins build trigger errors out as makefile and gcc path does not reflect on cygwin (Although all the required environment var set perfectly in the windows path, somehow its contradicting on jenkins. Not sure why)

What should be done next:

We almost broke cygwin while investigation. So need to setup a new windows VM with all the minimal configuration.
Due to contradicting behaviour of environment variable, we decided with add PATH in the database itself.

cc_ @mohammedzee1000 please feel free to add your points, if I missed something.

We have setup a windows vm with high end configuration, installed the pre-req. But facing few challenges like:

ssh to windows (Releng-key failed to provide password less auth on windows. Though we have work around by passing password in the database, but need to figure out the long term solution)
Jenkins build trigger errors out as makefile and gcc path does not reflect on cygwin (Although all the required environment var set perfectly in the windows path, somehow its contradicting on jenkins. Not sure why)
I have successfully installed gcc, now, but in process of investigation, broke cygwin.

What should be done next:

We almost broke cygwin while investigation. So need to setup a new windows VM with all the minimal configuration.
Now we know that cygwin, which is pre-installed should not be touched, instead, install mingw separately which provides gcc (apparently mingw is not installed correctly in cygwin)
Due to contradicting behaviour of environment variable, we decided to add PATH in the database itself.
By contradicting behaviour, we mean $PATH set in WIndows Environment Variables are not being taken in cygwin, which shows up with its own PATH values, no matter what is set in windows $PATH. Hence why we set the PATH it during test run time. These values should be predefined for a windows m/c as per installation

cc_ @mohammedzee1000 please feel free to add your points, if I missed something.

macOS set up has been configured on PSI. Looking forward to go with the unit and integration tests.

Setup is complete, but we have failures on windows and mac, which need to be fixed. QE is already on it.
Once they are fixed, we can reenable the tests.
/close


Prayer flick helper to show with prayers off

toggle option to have prayer flick helper plugin show even when no prayers are on

this was added in #5071


cleanups for 1.6

This series cleans up by removing old documentation and adds an automatic build test using Travis CI.
To make use of Travis, you have to follow https://docs.travis-ci.com/user/getting-started/ preferably before merging this series.


Removal of SharpZipLib

Description of problem
DNN Platform is currently using SharpZipLib for some Zip operations.  The usage of this should be reviewed and replaced with standard .NET Zip functionality if possible.
Description of solution
Remove third-party dependency
Description of alternatives considered
Leave the third-party solution

Unstaled, we do need to tackle this, this would also close several dependabot PRs

Duplicate of #2735


Changes needed in the frontend

Description
The frontend needs to be responsive to ensure a good user experience.
FormLab has to be rewrited to use react-hook-form
Objectives

Disable animations on News component
add media queries for mobile
turn navBar into a menu

All changes will be made in a branch front_responsive.

Steps Taken to Implement Responsive Changes and Rewrite FormLab with react-hook-form

Disabled Animations on News Component:

I introduced a flag to detect if the device is mobile (isMobile). If the device is not mobile, the animations from framer-motion are still enabled, otherwise, they are disabled for a smoother experience on mobile devices.

Added Media Queries for Mobile:

Media queries were added for devices with a screen width of 800px or less.
Most display: flex styles were adjusted to display: column, and the width of several components was set to 100vw to ensure the layout adapts properly on smaller screens.

Transformed Navbar into a Menu:

The isMobile flag was again used to provide a different menu layout for mobile devices.
On mobile, the navbar was replaced with a collapsible menu for better usability and space optimization.

Rewrote FormLab with react-hook-form:

I replaced the previous state management for form responses with the register method from react-hook-form to handle form data.
Form validation was integrated using zod for schema-based validation, ensuring proper error handling and cleaner code.

These changes were made in the front_responsive branch.


liblur improvements

Note: this changes the API for new_from_hidraw.

Hey Peter,
patch "lur: add lur_device_get/set_userdata" is missing the liblur.sym additions.

whoops, fixed. Branch updated to 379695fca236

urgh, sorry, forgot I had something else on that branch. HEAD is f0602bd, this is the correct one now

merged as of f233b6b.
I amended the commit "lur: add lur_device_get/set_userdata" to have the new symbols declared in LIBLUR_0.4.0.


[WIP] Test PR for fixing importing error

Codecov Report

Merging #227 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #227      +/-   ##
==========================================
- Coverage   81.89%   81.87%   -0.02%
==========================================
  Files         106      106
  Lines        5114     5109       -5
==========================================
- Hits         4188     4183       -5
  Misses        926      926

I confirm these errors are caused by RDKit latest version.


[jetson-xavier-nx-devkit] OEM authentication of kernel header failed!

I can't find any precise instructions on how to built a yocto image for an Xavier NX development kit but I did the usual thing of building a core-image-full-cmdline with meta-tegra gatesgarth branch and tried to create an SD card image with make-sdcard script.
I modified the flash.xml.in file and set the APP file to the .ext4 partition image generated by yocto to make it work.
Unfortunately the device fails to boot with the following boot log from UART console.
Is there anything else I should do before being able to boot from the SD card?

The wiki page could use an update to mention the Xavier NX dev kit, but the instructions are similar to the ones for the Nano dev kit:

Use the dosdcard.sh script to generate the SD card image
If this is the first time you're booting the NX dev kit from a Yocto-based build, make sure you also reprogram the QSPI flash so the bootloader and BCT match up with what you have on the SDcard.

For that second step:

Insert the SDcard you wrote into the slot on the device
Connect a USB cable from your host to the type B/OTG port
Put the device into forced-recovery mode via the jumper
Use ./doflash.sh --spi-only to program just the QSPI flash

The wiki page could use an update to mention the Xavier NX dev kit, but the instructions are similar to the ones for the Nano dev kit:

Use the dosdcard.sh script to generate the SD card image
If this is the first time you're booting the NX dev kit from a Yocto-based build, make sure you also reprogram the QSPI flash so the bootloader and BCT match up with what you have on the SDcard.

For that second step:

Insert the SDcard you wrote into the slot on the device
Connect a USB cable from your host to the type B/OTG port
Put the device into forced-recovery mode via the jumper
Use ./doflash.sh --spi-only to program just the QSPI flash

@madisongh thanks, this works. Sorry for the noise, I didn't know Nano is so similar.

@madisongh thanks, this works. Sorry for the noise, I didn't know Nano is so similar.


⚠ Bofrost.gr has degraded performance

In 403be15, Bofrost.gr (https://www.bofrost.gr) experienced degraded performance:

HTTP code: 200
Response time: 2584 ms

Resolved: Bofrost.gr performance has improved in f0fb554.


provide a way to close a bunch of pull requests

In classes that use the fork strategy, teachers may leave students' pull requests open for review, so that they can submit updates, etc. This may leave them with a bunch of open pull requests across one or more repositories, so it would be helpful to have a command that takes care of them in bulk.
My thinking is to expand merge_pull_requests:
$ teachers_pet help pull_requests
Usage:
  teachers_pet pull_requests --repository=OWNER/REPO

Options:
  --repository=OWNER/REPO
  --merge   # Merges pull requests. Cannot be used in combination with --close.
  --close   # Closes pull requests. Cannot be used in combination with --merge.
  ...

Perform an action on all open pull requests in a particular repository

On the fence about whether it should be an even more generic issues command, though that may be confusing since most people probably think about them (and treat them) as separate things.

Hmm, maybe not necessary after all...


2.0.x

@xing2858 Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


Fix screenshots in smoke tests

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 48.179%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
14525

Relevant Lines:
28635

💛  - Coveralls


Update README.md Static Linux Install Section

Static linux versions unzip to mmseqs2 directory instead of mmseqs directory

Thanks a lot for this pull request!


ONNX export makes incorrect assumption about label names

Description
ONNX implementation in mx2onnx.export_model:export_model() makes multiple incorrect assumptions about the symbolic graph which makes its implementation very fragile to any graph that doesn't fit its assumptions.
Incorrect assumptions:

Order that data is fed to network is the order that sym.list_inputs() lists them:

This is an incorrect assumption because the order data is fed to network (particularly a gluon block) is determined by custom block's API. However the order that data is listed in sym.list_inputs() is by the dependency in computation graph.

There can be one and only one label in the graph and this label is named after the last node of the graph on sym.get_internals().

This is clearly an incorrect assumption. Perhaps if the last node is called 'softmax' and the network's label was called 'softmax_label' (which happens to be the default label name) then this assumption will hold. However under any other condition, I don't think this is a valid assumption.

A proper implementation would be to change the export_model() API to accept a dictionary of name: shape for input instead of a list of shapes. Any input that is not in params or in inputs will then be assumed to be a label and may be ignored. However if incorrectly specified as unnecessary label, then ONNX export will fail.
For backward compatibility, if a list of shapes is provided as inputs, the current automatic name to shape mapping will happen but a warning log is created, warning user to use a dictionary and also warns about the default mapping between shapes and inputs.

@mxnet-label-bot add [onnx]

Thanks Sina for creating issue for it. I am aware of this issue and started working on the fix for it https://github.com/apache/incubator-mxnet/pull/12946 but got sidelined by other things. I will rebase the PR with latest changes and continue working on the fix.
I also have added optional parameters "label_names", "label_shapes"([name, shape]) to export_model API.


add underground tag to buildings

There are a couple of BART stations which are underground: https://www.openstreetmap.org/way/155378773
https://www.openstreetmap.org/way/160331904
But the tiles have them as just vanilla buildings at street level:
http://tangrams.github.io/explorer/#17.5/37.7896/-122.4013/name/Montgomery Stat
https://trello-attachments.s3.amazonaws.com/55f1d2933cfa613ec481fe6c/978x816/e9e8d46c82c10fdd49e3d4af117272b5/Screen_Shot_2015-09-10_at_2.55.37_PM.png

Let's think about moving underground buildings that are transit into the transit layer (they don't have anything to extrude and are more or less just flat polygons). Regular (above ground) buildings tagged with transit should stay in the building layer (but express they are transit related in the attributes).
See also:

http://wiki.openstreetmap.org/wiki/Key:location?uselang=en-US

http://wiki.openstreetmap.org/wiki/Key:layer?uselang=en-US

We'll add level and location to the buildings, but I suspect the larger issue is that the BART platforms should really be marked railway:platform and be included in the transit layer.
See how to do it right in Moscow:
http://www.openstreetmap.org/#map=17/55.75196/37.60592

location is coming thru, but doesn't look like layer is. @zerebubuth can you investigate why?
Here's Powell St in SF:
https://www.openstreetmap.org/edit?#map=18/37.78498/-122.40639
Before:
http://vector.mapzen.com/osm/all/18/41938/101314.topojson?api_key=vector-tiles-HqUVidw

After:
http://vector.dev.mapzen.com/osm/all/18/41938/101314.topojson?api_key=vector-tiles-HqUVidw

location is coming thru, but doesn't look like layer is.

Osm2pgsql makes explicit columns from tags in its style file, and layer is one of those. A side-effect is that it removes it from the tags hstore, so that query was coming back as NULL always. Should be fixed as part of #285.

Verified.


Move to pytest-cov

The incremental coverage scripts used by Cirq and OpenFermion-Cirq are nice because you can use comments to indicate that certain lines don't need to be covered. I think we should move to that system and stop using coveralls. In the past we have had to override the coveralls status check many times.

That's fine. I feel like this is a bit redundant with #414. Also, I would deem all of this very low priority.


Radarr & Sonarr connections failing

I have petio running in docker on a synology deployment with DSM 7.2.  I noticed that my requests were no longer processing correctly so I took a look at my admin settings.  I tried to test the existing connections and they failed.  When I cancelled out of the configuration screen, the configuration disappeared.  When I try to recreate, the connection test fails.  I tried with both my local IP as well as a DDNS address and both fail.
I checked the console and see this error.  I took the path (API key removed) and put it into postman and it returns the expected body.

This configuration had been working for nearly a year and nothing has changed.
Petio image detail

Radarr image is linuxserver/radarr:latest and the same for Sonarr

Your error shows port 7373 and postman you used 7878

Ugh.  That was a fat finger after multiple attempts with both radarr and sonarr.  The issue persists for both.  Here's the revised error screenshot:

When I try to revise the configuration through the UI and the test fails, the profile gets deleted and the json file with the stored config in the docker folder also resets to a blank object.

I tried using both the local IP address as well as the DDNS URL.  Neither works.

I also tried disabling the firewall on the synology unit to rule that out but it made no difference.
All of these packages are deployed in docker.  Petio has its own network assigned but radarr and sonarr are running on a network bridge.
But this configuration previously worked until about a week or so ago.  radarr and sonarr haven't changed other than an image update (I use the :latest tag) and so has petio.
Please let me know if there is anything I can provide to help troubleshoot this.

Connection reset is still a networking issue. Is it a reset with the local IP too?

Yes, same issue
On Sun, Oct 29, 2023, 7:20 PM HalianElf @.***> wrote:

Connection reset is still a networking issue. Is it a reset with the local
IP too?
—
Reply to this email directly, view it on GitHub
https://github.com/petio-team/petio/issues/853#issuecomment-1784263736,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXYBZATPNBLRTL67Q5NTDIDYB3QCVAVCNFSM6AAAAAA6U4S4ECVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBUGI3DGNZTGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Is there any update here?

Still no update?


Animated Scroll Bar

Added Animated Scroll Bar. Pls merge and level  label the PR under DWOC .

Pls review, merge and level label this PR under DWoC

Don't add so many colours it looks too kiddish

Pls elaborate what is your concern. I have made it animated gradient custom scroll bar. I think it is looking good🙄.

It looks good on sites where we want to show many colours at the same place but here were keep it minimal so the colour of black and blue with the decent colour and width will look great here.

Don't add so many colours it looks too kiddish

Pls elaborate what is your concern. I have made it animated gradient custom scroll bar. I think it is looking good🙄.

See it's not about if an individual element looks awesome or not. What matters is if it fits well with the rest of the system otherwise it would look really absurd.
The scrollbar you have made is really ideal for a dark theme based product page like those of ASUS ROG,etc. For a portfolio page of a person, perhaps not


Fix typst watch not working with some text editors

This fixes #1651.
As stated in the issue, the modified code unwatches all files for which we received a Remove event, and allows these files to be watched again if they still exist and the compilation still depends on them.
Note that I was only able to test this on Linux, I'm unsure whether this fix could create issues on other platforms.

Should I also include the corresponding issue's id or a link to the issue in the note ?

I think this comment is descriptive enough

Thank you!


Error Handling When Checking Version of Firefox

On line 175 of install.js there's a missing error within the try... catch statement, resulting in a syntax error when installing.

@googlebot I signed it!

This is not missing.
You're using an old, unsupported Node.js version that lacks support for optional catch binding.


Hack to fix RCODE_NAME_ERROR during shell

I have no idea what I'm doing, but this fixes the RCODE_NAME_ERROR on the client when initializing a shell from the server.
Error server side:
#<NoMethodError: undefined method stopped' for #<DriverConsole:0x00000000d78020>> /home/ubuntu/dnscat2/server/controller/session.rb:396:in block in feed'
/home/ubuntu/dnscat2/server/controller/encryptor.rb:244:in decrypt_and_encrypt' /home/ubuntu/dnscat2/server/controller/session.rb:394:in feed'
/home/ubuntu/dnscat2/server/controller/controller.rb:91:in feed' /home/ubuntu/dnscat2/server/tunnel_drivers/tunnel_drivers.rb:25:in block in start'
/home/ubuntu/dnscat2/server/tunnel_drivers/driver_dns.rb:316:in call' /home/ubuntu/dnscat2/server/tunnel_drivers/driver_dns.rb:316:in block in initialize'
/home/ubuntu/dnscat2/server/libs/dnser.rb:872:in call' /home/ubuntu/dnscat2/server/libs/dnser.rb:872:in block (2 levels) in on_request'
/home/ubuntu/dnscat2/server/libs/dnser.rb:843:in loop' /home/ubuntu/dnscat2/server/libs/dnser.rb:843:in block in on_request'
So session expects the stopped method/attribute to be available, but not all drivers will have it. Or at least that is my interpretation of the bug.
Shell commands still seem to fail with this patch. So I will keep on debugging.

Sorry for the delay.. I hadn't realized how simple this was, and I was waiting to have a chance to review it :)


Refactor player to extract Lookit-specific exit behaviors

Refs: https://openscience.atlassian.net/browse/LEI-267
Companion to:
Purpose
The player currently has certain custom behaviors that are Lookit-specific. We do not, for example, want ISP users to be able to skip to the last frame by pressing F1.
Refactor to move those behaviors inside lookit.
Summary of changes

Reorg code: move lookit-only functionality into Lookit.
Add a new beforeUnload hook to the player, controlling additional custom behavior that happens when a user leaves. For example, it can be used to display a toast message with more info
Add a new messageEarlyExitModal property, controlling text of the confirm-on-exit modal. (may not show this text in all browser, but method has to return a non-null value if you want the modal to appear)

Testing notes

All studies will show warning messages with unsaved data by default
ISP will not respond to the F1 key
Lookit will respond to the F1 key by skipping ahead to the last frame

Confirmed that ISP no longer responds to the F1 key. Also, a warning message does show up when attempting to close the browser or navigate to a different url, but this happens regardless of whether there is unsaved data. Is that intentional @abought?

Confirmed that ISP no longer responds to the F1 key. Also, a warning message does show up when attempting to close the browser or navigate to a different url, but this happens regardless of whether there is unsaved data. Is that intentional @abought?

In theory, the warning message should always have shown up- the removed code simply depended on a boolean property, which would have had the same value for both players:
https://github.com/CenterForOpenScience/exp-addons/pull/171/files#diff-85b6be66a5ce62f39ee05c9d04ceb67fL30
Will follow up.

As discussed on flowdock: the reason that ISP is suddenly getting an error message is because I moved the toast.warning call into lookit.
Although the player code used toast, neither the player nor ISP listed it as a dependency. Attempting to call a toast method from ISP was, before this PR, causing the beforeunload code to fail halfway through.

As @abought pointed out on flowdock, the message shows up depending on the allowExit variable:
https://github.com/CenterForOpenScience/exp-addons/pull/171/files#diff-85b6be66a5ce62f39ee05c9d04ceb67fR46
https://github.com/CenterForOpenScience/exp-addons/pull/171/files#diff-85b6be66a5ce62f39ee05c9d04ceb67fR62
It does not depend on unsaved data so the current behavior is correct. That was my only concern, so this is good to go 🎉


fix: apply optimistic updates from READ requests immediately

Details

In our API layer and the sequential queue we want to run READ requests after WRITE requests. This is to avoid a READ request overwriting optimistic data of a write request. This was introduced here:

https://github.com/Expensify/App/pull/12219

However, this currently prevents the optimistic updates of a READ request to be applied right away. This leads to bugs where components call READ commands but those don't update the onyx state right away.
Fixed Issues

$ https://github.com/Expensify/App/issues/48053
PROPOSAL: https://expensify.slack.com/archives/C05LX9D6E07/p1724701548092719?thread_ts=1724696592.356379&cid=C05LX9D6E07
Tests

Create a workspace
Go to workspace settings
Open "More Features"
Enable distance tracking
Desktop:

Open the chrome dev settings, go to network tab, and pick "Offline"

Mobile:

Disable any network

Change the rate for the distance tracking feature while offline, save it
Go online again
Confirm that there is no flickering / jumping of the old and new value

Offline tests

Same as tests
QA Steps

Same as tests
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 either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[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 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(theme.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 the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[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.

Screenshots/Videos

Android: Native

https://github.com/user-attachments/assets/ec5dff1f-90d1-4707-846a-579f5702db36

Android: mWeb Chrome

https://github.com/user-attachments/assets/fc61dbd8-b16c-4c77-8370-c6f22f0b1d7c

iOS: Native

https://github.com/user-attachments/assets/a9a9da8e-3c3b-42dd-bb16-40e33d9054fe

iOS: mWeb Safari

https://github.com/user-attachments/assets/2964e0c6-4644-4bac-9a6e-67632225e060

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/578f35bc-ac0b-4d35-b9a2-be54e7ea9db6

MacOS: Desktop

https://github.com/user-attachments/assets/d9b56009-32ed-4bdb-b9f5-6f2152a741a7

cc @roryabraham
I still need to run all tests from the previous PRs to make sure we don't break anything.

Will be reviewing this in a few hours.

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible 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)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

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

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (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] 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 either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[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 verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[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] 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] 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(theme.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.
[ ] 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 the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[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 reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

Okay so I am trying to test this on iOS and mobile web. I disconnected the Wifi and connected again but I do see the old value of tag being shown temporarily.
https://github.com/user-attachments/assets/70e710e8-67ea-45bb-874a-e218525ac00a

Hm okay, thats odd, will take a look!

Note: I am OOO until Wednesday, will pick up after. (Or @kirillzyusko if you have time meanwhile feel free to take a look!)

@hannojg @kirillzyusko Any updates on this one?

@mananjadhav I'm looking into it 👀

@mananjadhav I've tested the code (before and after modifications) and I see the behavior that you described in old code variant as well:
https://github.com/user-attachments/assets/94bb8f95-ed98-48ab-8ced-f9ac03caa09d
So it makes me thinking that you found another bug that present in both code versions and I think it's better to fix it in separate PR and open a new issue. What do you think?
The purpose of this PR was to fix an order of applying optimistic update (in case if we have write operations in queue) - and this PR its job well. The only thing is that you found a new bug, but I believe this bug present in current main as well, so I think it should be fixed separately. What do you think?

@kirillzyusko Isn't one of the test steps about not flickering between old and new value.

Go online again
Confirm that there is no flickering / jumping of the old and new value

@mananjadhav Yes its one of the test steps I wrote. However I was assuming that "there is no flickering / jumping of the old and new value" on the main branch.
Kiryl says that its also on the main branch (so my assumption was wrong)
I should probably update the tests steps accordingly and open a new issue, right?

Yeah @hannojg  please update the test steps accordingly for me to test and verify. I can see you've started the bug report flow already. Thank you.

Yeah @hannojg please update the test steps accordingly for me to test and verify. I can see you've started the bug report flow already. Thank you.

@hannojg @kirillzyusko quick bump

@hannojg quick bump.
@roryabraham do you have any test steps in mind? Or should I submit the checklist with the flicker knowing it exists on main?

Not sure about specific test steps here, but I'm not worried because this is how it used to work, and I only changed it accidentally/by mistake.

Going to just merge this because I am pretty sure it's right, I don't know what specific test steps would be needed anyways, and you did some basic smoke tests already.

: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.


⚠ gorzdrav.org has degraded performance

In fd42e95, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 2952 ms

Resolved: gorzdrav.org performance has improved in 8e70d44 after 5 minutes.


Automatic number of subscales for images

For now, there is an argument n_subscales with default 5. This should depend on the image size.
We could have an "auto" argument by default

Not needed


⚠ Turnos 24x7 has degraded performance

In db2ad45, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 981 ms

Resolved: Turnos 24x7 performance has improved in 76a22c7.


A2S_INFO change

The library is currently unusable with the change Valve detailed in a post about 9 months ago:
https://steamcommunity.com/discussions/forum/14/2974028351344359625/
This change is rolling out to more and more games, breaking compatibility of this library.
The updated specification can be found here: https://developer.valvesoftware.com/wiki/Server_queries#A2S_INFO

@ribasco Thanks for noticing the issue.
As of recent updates for various Source 1 games (such as the 9/17/2021 Team Fortress 2 update), fetching server info always leads to a timeout. I have several applications that depend on this library to fetch server info.
Is there a possibility of a fix version being available sometime soon?

@ribasco Thanks for noticing the issue.

Thanks for bringing this to my attention.

Is there a possibility of a fix version being available sometime soon?

I'm already working on it as we speak and decided to push this fix for v0.2.0. Will you be available any time next week for testing?

I'm already working on it as we speak and decided to push this fix for v0.2.0. Will you be available any time next week for testing?

Yes, I can help with testing.

Any updates on this?

Any updates on this?

Apologies if I haven't responded promptly. Just an update, I ultimately decided to re-write most part of the library and push the changes to v1.0.0 instead.
If you are still available for a quick test, the snapshot artifacts (1.0.0-SNAPSHOT) are now available at the sonatype repository.

Closing for now. This has been resolved in v1.0.0 release. Feel free to re-open if you encounter any more issues with the release.


Sound volume reset on tracks change

Volume settings don't saving between tracks

As for me we need a custom wrapper around html5 audio anyway. For UI purposes at least.


Fix axios definitions by removing enums and config generic

Remove the enums because the library expects strings here. The test shows how to implement a "typed" way anyway though. Also remove incorrect generic parameter.

@marcelbuesing thanks mate!


Filter rules by tag contains

Is your feature request related to a problem?
This is more like a "nice to have" as I can work around it.  It would be nice with type=rule to be able to filter on "(tag contains something)" rather than a serious of OR'd "(tag has something)."
Describe the solution you'd like
For example, for tagging unused rules I have several tags based on how long the rule has been unused - 90 days, 180 days, 1 year, COVID, Pre-COVID, etc. (the remaining are legacy tag names that don't correspond to a timeframe).  They all are prefaced by "UNUSED-" so a filter such as (tag contains UNUSED) or (tag contains UNUSED-) would match all of these.
I use this to look for previously "unused" tagged rules such as when I want to disable the rules for future deletion.
Describe alternatives you've considered
Currently, I filter by a series of OR'd (tag has something) elements - one for each of the UNUSED-* tags.
Additional context
I would like to cutdown on the time it takes to build the filter (not a big deal as I have previous ones, but I would have to update them if any new tags have been introduced).  Also, it would avoid errors of omission.

pan-os-php type=rule in=api://MGMT-IP actions=display 'filter=(tag has.regex /UNUSED-/)'


Sche redirect6

What problem does this PR solve?

Issue Number: Close #xxx
What is changed and how does it work?

Check List

Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Code changes

Has the configuration change
Has HTTP API interfaces changed (Don't forget to add the declarative for the new API)
Has persistent data change

Side effects

Possible performance regression
Increased code complexity
Breaking backward compatibility

Related changes

PR to update pingcap/docs/pingcap/docs-cn:
PR to update pingcap/tiup:
Need to cherry-pick to the release branch

Release note

None.

Codecov Report

Merging #19 (5b26cf4) into master (edf1f95) will increase coverage by 0.02%.
Report is 7 commits behind head on master.
The diff coverage is 64.95%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
+ Coverage   74.55%   74.58%   +0.02%
==========================================
  Files         441      442       +1
  Lines       47365    47792     +427
==========================================
+ Hits        35314    35644     +330
- Misses       8956     8990      +34
- Partials     3095     3158      +63

Flag
Coverage Δ

unittests
74.58% <64.95%> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.


pypi-dependencies wheel only installation

Problem description
It would be nice to be able to define a wheel-only = true option to the [pypi-option] so we don't required the ability to build in a specific environment to solve the dependencies.
uv has this option with the --only-binary, the naming should be easy for the users.

Users should be able to specify this for each dependency separately. They might want to use the binaries from one but not the other.
In UV you can specify this for each package
https://docs.astral.sh/uv/reference/settings/#no-binary-package
In Pip you can supply it multiple times
https://pip.pypa.io/en/stable/cli/pip_install/#cmdoption-no-binary
In Poetry you can also do that via the config file
https://python-poetry.org/blog/announcing-poetry-1.2.0/#opting-out-of-binary-distributions

Thanks for the info @thomas-maschler


Add examples visualizing molecules with virtual sites

Description
Provide a brief description of the PR's purpose here.
Checklist

[ ] Add tests
[ ] Lint
[ ] Update docstrings

#880 includes some visualizations of molecules with virtual sites


generic output adapter

with a "type" named parameter for a special output adapter we could make it as generic and flexible as any spec from the json schema. this can help have a generic use case fallback output adapter, as well as a powerful simple description of an output adapter for prompts to generate generators.

this may be linked to #92


Unable to set up Netlify CMS on gitpod.io

Describe the bug

I have completed all the instructions to set up Netlify CMS on my project.  It works fine when hosted on Netlify.  When I try and authorize Netlify CMS locally on gitpod.io I am unable to authorize it.  I says "Login with Netlify Identity" but when I click it nothing happens, it just goes grey. I have had more success when following the instructions to set up a "Git Gateway without Netlify", but ultimately that did not work either.  Can somebody please have a look at my repo (linked below), and explain to me like I'm five if I am doing something wrong.
To Reproduce
Follow the instructions on https://www.netlifycms.org/docs/add-to-your-site/#app-file-structure and https://www.netlifycms.org/docs/authentication-backends/
Expected behavior

I would expect the go through some form of authorization and then for that window to close and for the Netlify CMS to start.
Screenshots

Applicable Versions:

Netlify CMS version:  2.9.1
Git provider:  GitHub
OS: Linux (gitpod.io)
Browser version Chrome 74

Node.JS version: 10.15.3

CMS configuration

https://github.com/JamesGreenaway/victor-hugo
Additional context

I am using gitpod.io as opposed to localhost.

You won't be able to do this from gitpod, or any online ide, without setting up your own auth solution. Using actual localhost is supported by an auth setup specifically for localhost urls by Netlify, but on Gitpod you're not using a localhost url. There are community libraries for setting up an auth server if you're interested: https://www.netlifycms.org/docs/authentication-backends/#external-oauth-clients
Setting up Git Gateway on your own is not currently for the faint of heart, but if you want to try your hand, here's the best resource I'm aware of to get your started: https://github.com/hfte/netlify-cms-with-selfhosted-gotrue-and-git-gateway

ok thanks for your help


umlauts in headings font

The font used for headings don't have all the special characters for umlauted characters: ä ë ï ö ü ÿ Ä Ë Ï Ö Ü Ÿ
.. and there's a distinct difference between V3 and Legacy. (This is due to a different font being used).
V3 Render

Legacy Render

From gitter:

They use different versions of the fonts with slightly different sizes and spacings which would throw off Legacy. We could always do the work of manually filling in the legacy version of the fonts with any missing characters.
But this was one of the big efforts for V3: redoing all the fonts to be more error-free, consistently-sized, add missing characters, etc. It was never a priority to also add missing characters to the old broken font.

In favor of closing as a duplicate of #116?

#116 is the appropriate issue .. however is currently closed. Re-opening that.


PDB match: search

Use pHMMer (from HMMer /projects/microprot/tools/HMMer/) or HHsearch (from HH-suite) to search an amino acid sequence for hits against the PDB database.
suggested parameters for phmmer: -E 0.001 --domtblout
make sure the script is able to process multiple hits to different PDBs, i.e. gene has several domains, each of them matching to a different PDB structure.

added as PDB_match rule in Snakemake


[sysrst_ctrl,dv] Randomize clock gating to check interrupt

Signed-off-by: Madhuri Patel<EMAIL_ADDRESS>

This pull request is related interrupt synchronisation after disabling the clock gating. I tried to re-generate the issue fixed by [https://github.com/lowRISC/opentitan/pull/12027]. Below is the waveforms before the fix:

We are disabling the core clock after register configuration (cursor at Time A).
We have two interrupts at Marker 1 and Marker 2. We enable the clock at Marker 3 but it does not read the interurpts and thus the test fails giving below errors:
**UVM_ERROR (cip_base_vseq.sv:378) [sysrst_ctrl_edge_detect_vseq] Check failed act_pins == exp_pins (* [*] vs * [*]) has 1 failures:
* Test sysrst_ctrl_edge_detect has 1 failures.
* 0.sysrst_ctrl_edge_detect.531956189
Line 329, in log /scratch/google/goo_opentitan/madhurip/master/sysrst_ctrl-sim-xcelium/0.sysrst_ctrl_edge_detect/out/run.log
            UVM_ERROR @ 940596234 ps: (cip_base_vseq.sv:378) [uvm_test_top.env.virtual_sequencer.sysrst_ctrl_edge_detect_vseq] Check failed act_pins == exp_pins (0 [0x0] vs 1 [0x1])
            UVM_ERROR @ 940676234 ps: (csr_utils_pkg.sv:432) [csr_utils::csr_rd_check] Check failed obs == exp (0 [0x0] vs 1 [0x1]) Regname: sysrst_ctrl_reg_block.intr_state
            UVM_INFO @ 940676234 ps: (uvm_report_catcher.svh:705) [UVM/REPORT/CATCHER]
            --- UVM Report catcher Summary ---**

Below is the waveform with the fix:

Once the clock is enabled, interrupt is read correctly and the test passes.


Windows build fails on AppVeyor

Example build failure: https://ci.appveyor.com/project/bryceschober/conan-cpputest/build/job/sc8kv9y0ybnc9fu1

This issue currently blocks #1.

Until AppVeyor and the OS X Travis (#4) are fixed, they should probably get disabled. PRs and commits are marked as failed at the moment, even though they are Ok.
After both are working, they are re-enabled of course.

The next issues are related to the build as shared library.
I am working on it (https://github.com/matlo607/cpputest/commit/ced78240d23058ff37aaf929b4865383ffc6fa5d, https://github.com/matlo607/cpputest/commit/2f0de48c0e048e34392030da4a72412f1a4db131), the build works on Linux but I struggling on Windows to add __declspec(dllexport).
I keep you updated.


add url as explicit dependency

Motivation and Context
(https://github.com/stoplightio/elements/issues/2483)
Description
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

Hi @mmarti21  could you please review this?


Registry returned 404 for GET on https://registry.npmjs.org/esprima-six

I can't run npm start successfully because of the following error:
npm ERR! 404 Registry returned 404 for GET on https://registry.npmjs.org/esprima-six npm ERR! 404  npm ERR! 404 'esprima-six' is not in the npm registry. npm ERR! 404 You should bug the author to publish it (or use the name yourself!) npm ERR! 404 It was specified as a dependency of 'derequire'

@launchriot Ugh sorry about that... can you try https://github.com/BinaryMuse/chrome-fast-tab-switcher/pull/19 and let me know if it works for you?

@BinaryMuse yup, that seems to have fixed it! There are some other errors as well, but they don't seem to halt the installation process. Thanks! :)


Supported Contract::_fallback in smart switch

Dependent from https://github.com/tonlabs/TON-Compiler/pull/97
If Contract::_fallback(cell msg, slice msg_body) function is implemented, smart switch will call it in case of unparsed or wrong func_id.

LGTM, but I'd recommend adding regression tests for ISel. They are trivial, but they would work better then runtime ones if we break something.

Test added.


Command IObservables do not work within a UserControl's constructor

If I create a Subject for my button's command within the constructor of the UserControl, the button will remain disabled. If I set the command the same way after the Initialized event is passed, it will then work.
        public MainView()
        {
            AvaloniaXamlLoader.Load(this);
            DataContext = this;

            // Temporary fix for https://github.com/AvaloniaUI/Avalonia/issues/2562
            // Also a fix for button commands
            Initialized += (s, e) =>
            {
                tabs = this.FindControl<TabControl>("Tabs");
                teamBuilder = this.FindControl<TeamBuilderView>("TeamBuilder"); // teamBuilder would be null in the constructor because of #2562
                ip = this.FindControl<TextBox>("IP");
                port = this.FindControl<NumericUpDown>("Port");
                connect = this.FindControl<Button>("Connect");
                connectEnabled = new Subject<bool>();
                connect.Command = ReactiveCommand.Create(Connect, connectEnabled);
                connectEnabled.OnNext(true); // This would not do anything in the constructor
            };
        }

If it was not called from Initialized, it would not work, as I said before. These are identical conditions to the FindControl bug (#2562)

This was happening while I was using a PR package (the one from #2620) but now it is fixed after merging with master, although the FindControl one is not fixed.

Disregard the above, it actually still requires Initialized.

I just tried this both on 0.8 and the latest CI build. I created a new Avalonia application from a template and did:
public MainWindow()
{
    InitializeComponent();

    var btn = this.FindControl<Button>("btn");
    var enabled = new Subject<bool>();
    btn.Command = ReactiveCommand.Create(() => { }, enabled);
    enabled.OnNext(true);
}

In both cases, without the enabled.OnNext(true) line, the button was disabled, but when I added that line the button was enabled.
Am I missing something?

This does seem to be fixed in the latest builds, so I will close this. Oddly, the FindControl bug is still in the latest builds, even though this bug had identical conditions as that bug


Egress gateway for IPv6

The egress gateway feature is only supported for IPv4 for now. Implementing the same for IPv6 connections would require support for BPF-based IPv6 masquerading (cf. https://github.com/cilium/cilium/issues/14350).

@pchaigno I have done few good-first-issues. I would like to take this issue but I might need help on how to get started with it.

@h3llix This is blocked on https://github.com/cilium/cilium/issues/14350.

@pchaigno Any other issue that I can take?

@h3llix You can either take another good first issue, or dig through the list of unassigned issues. Are you looking to contribute on some particular component/feature?

Is this in the roadmap? Or what cilium release would include this feature? @pchaigno

Just curious if this issue with v6 support will be on a roadmap or released soon?

The blocker issue (IPv6 BPF masquerading) has been solved, so it is possible to work on this now. Although it's unlikely that this will make it to 1.15.

Any news here?

Bump.  This would be a great add for my dual stack clusters.

Can we have an update on this if it is going to be on the roadmap? or at least what is the size of the task to maybe try to contribute.


Backwards consistency - Update README.md

[skip ci]

I'm not sure this matters, I actually think the order doesn't (unless not associative, but that would never apply?).
Should you have rather all upwards? I guess it depends on what the code does.
[About the dots "\cdots", having those upper is I guess intentionally not the same style though.]


feat: add support for Slonik v35, v36, v37

Slonik v33 and v34 are still supported as long as the matching mocha-slonik versions do not cause conflicts in peer dependency.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Added name

Related issue

What changes does this PR do?
Screenshots (If any)
Questions

Kindly place your name in the line 283


The request.body should be destructed

https://github.com/carlosarroyoam/nodejs-api/blob/5131b42ee5fe341ddd7027bcac37920e30769576/src/api/controllers/user.controller.js#L101

Already solved


docs: improve structure and examples

The structure has far outgrown its features.
Usually you are searching for a specific thing to achieve like in a "How to" while the docs are more a sequential arbitrary sorted list of references.
By visually deconstructing the UI into smaller parts and providing more screenshots for the different config options, it should be easier for a user to discover information of interest
https://www.youtube.com/watch?v=qC1OYK5oqDo

Somthing that may currently be useful and not that long to implement: move paragraphs of Customization to children pages so that they can be easily browsed. https://mcshelby.github.io/hugo-theme-relearn/basics/customization/index.html
Does milestone 6.0.0 have any probable date set?

Soon™
But with reduced features. The docs rework was dropped to speed things up. Anyways this is not off the table.

While adding documentation for 7.0.0, it got evident that this issue need to be done now. Stay tuned.


Suggestion: use labels sparingly

Look at other popular and widely-used projects on GitHub, such as https://github.com/openssl/OpenSSL.git and https://github.com/OpenSC/OpenSC.git
Both have tons of issues to process, but an issue would have maybe one, at most two labels. Here I see it rare for an issue to have less than four labels, usually five or six.
Please reconsider, and stop throwing every possibly-remotely-applicable label at an issue - it's both confusing and annoying to the viewer. Pick one that's the closest, and stay with it.
My $0.02.

@mouse07410 That's totally valid, and I would lean that way as well. However, the Code.gov website will scrape issues from our open source repositories and display them on their site - along with other Federal Government open source projects - in a single UI. In order for that page to have filtering capabilities it uses labels like the ones you see here. So, for the time being, we'll need to keep the labels as they are.
(I'll leave this issue open for a bit just so others can comment/read if they wish.)


User should be able to delete database connection

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/183
Originally created by @bencodezen on 2018-11-27 19:52:33

Currently Docker stores the database connection in memory and requires a full restart in order to get a new connection.
There should be a way to drop the connection from the Analyze UI.
cc @jschatz1 to add details if I missed anything

View 3 previous comments from the original issue on GitLab


Replace Splish editor with Edtr.io

Supersedes #22 and #23.
Open tasks

[x] Integrate Edtr.io in client
[x] Integrate Edtr.io in server

[x] Especially: fake plugins for every content type as discussed
[ ] https://github.com/serlo/athene2/pull/884

Before merging / releasing

[ ] Test every content type (both legacy to Edtr.io & Splish Editor to Edtr.io)
[ ] Test pages
[ ] Test profiles
[x] If possible, wait for 100% Kubernetes because legacy backend has completely different deployment process for editor-renderer
[x] (Is always importFromHtml a breaking change?)
[x] (Check Backward-compatibility)

Pushed a couple of fixes, editor-renderer fails for some easy Splish editor state, though. Let's look into that together, might be missing something simple...

editor-renderer should be fixed now, also rebased on master

Need to explore why athene2-assets doesn't build anymore

Merged to get this into staging :) Still needs some polishing before going into production, though.


Kill launch

Now that copy is able to get and run images in #2281, maybe it is time to deprecate and remove launch?

Please no :). Then the command to launch containers would be lxc copy, which doesn't make a lot of sense.

Presence of launch and start does't make a lot of sense too. =)

Copy isn't capable of handling images at all, it only allows copying from existing containers and snapshots, it's very different from launch.

@stgraber why not to extend it to handling images? How images are so different from containers and snapshots? To me image is the same container, just with dynamic part.


Timbl pad

Picking up various bug fixes etc wile mainly new version to fix race  condition in  user participation management.

👍 , merging


Use array this.traceIndices directly in Plotly.restyle(), avoiding fo…

Avoid for loop overhead by passing this.traceIndices to Plotly.restyle
There is no need to send each index separately as [i]. Let Plotly do the work.

Disregarding the typo npm caught, I don't see anything wrong with this, from a functional point. Someone with more knowledge regarding Plotly might chip in their opinion.


Trying to build from Gitea / Forgejo fails with authentication required

Hello everyone,
I have a public repository on an installation of Forgejo (a fork of Gitea), and if I try to use envbuilder to build from the URL of this repository I get the following output under Coder:
envbuilder - Build development environments from repositories in a container
#1: 📦 Cloning https://git.1in9.net/raider/wroofauth to /workspaces/wroofauth...
Failed to clone repository: clone "https://git.1in9.net/raider/wroofauth": authentication required
Falling back to the default image...
#2: Deleting filesystem...
#2: 🏗 Building image...
[...]

Cloning with a normal git client works fine. Adding the ".git" at the end of the URL has no effect.
Feel free to reproduce this using my repo at https://git.1in9.net/raider/wroofauth
Kind regards,
Kevin

Hi, I have the exact same issue on bitbucket server

I have the same issue with envbuilder 0.2.4.

I got it working by explicitly setting access credentials for GitLab (the git repository containing the .devcontainer spec is located on my GitLab instance). However, the Git repository is public. There should be no need to authenticate to GitLab.
In contrast, envbuilder can access public GitHub repositories without problems. Is there a special handling for GitHub repositories? From the comments above, it seems that this problem only occurs with non-GitHub repos.

Maybe it's because https://github.com/coder/envbuilder/blob/main/envbuilder.go#L357-L358 passing Git username as   (blank) and password as   (blank)?
So GitHub might resolve it as not logged in but other git provider resolved it as no this user

Oh guys, I don't think this is a bug, because I initially encountered the same problem as you did, but I solved it by looking at the documentation – of course, I have to say that the information here is really incomplete, which took me half a day to figure out.
Here's my solution:

Firstly, you need to add an OAuth2 application on your Gitea/GitLab/Bitbucket, and the URI address is documented here; please modify it according to your actual situation.
Secondly, you need to add the relevant environment variables as guided in the documentation. Please note that all variables should not be enclosed in double quotes and ensure that the CODER_ACCESS_URL environment variable is set correctly.
Lastly, modify the template file by adding the following block of "coder_external_auth" data:
data "coder_external_auth" "any name you like here" {
    id = the CODER_EXTERNAL_AUTH_0_ID you set in the first step
}

And within the "resource" "docker_container" "workspace", add the following environment variables:
env {
    "GIT_USERNAME=oauth",
    "GIT_PASSWORD=${data.coder_external_auth.the name you set in the second step.access_token}"
}

and then click "Build" and then "Publish".

After completing all these steps,You will find that your workspace creation now includes an additional repository authorization option. Complete the authorization, and you will be able to perform a normal git clone.

Hi @AisonSu,
Your solution seem to work because envbuilder run authenticated queries against the git SCM provider.
However if for some reason we can't set up an oauth access, we're stuck.
(Except on GitHub, which works fine without authentication)
wolf-yuan-6115 is probably right about the root cause .

Hi, I have implemented a fix for this, you can test this using following image:
ghcr.io/nwrkbiz/envbuilder:latest


Feature/issue 628

For issue ga4gh/dockstore#628

Codecov Report

Merging #108 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #108   +/-   ##
========================================
  Coverage    65.27%   65.27%
========================================
  Files           93       93
  Lines         2508     2508
  Branches       317      317
========================================
  Hits          1637     1637
  Misses         702      702
  Partials       169      169

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 33d4383...e288ada. Read the comment docs.


BulkInsert - Invalid column name '$Action'

I'm trying to call a BulkInsert but I'm facing this: Invalid column name '$Action'.
List<ENTITY_TRIP> lst = new List<ENTITY_TRIP>();
for (int i = 0; i < 1000; i++)
{
lst.Add(new ENTITY_TRIP
{
TRANSACTIONID = 100001 + i,
EVENTID = 1,
UPDATEDATE = DateTime.Now,
USERID = "test"
});
}
objContext.Database.Log = Console.WriteLine;
objContext.BulkInsert(lst, options =>
{
options.CommandTimeout = 30;
options.BatchSize = 100000;
options.InsertIfNotExists = true;
options.InsertOnCondition = (s, t) => s.TRANSACTIONID == t.TRANSACTIONID;
options.KeepIdentity = true;
options.AutoMapOutput = false;
});

vnizampatnam,
I am unable to reproduce this issue. Can you please provide the call stack and any additional details you think is relevant.

I believe it's due to the database collation settings. I encountered the same issue when using BulkMerge. The collation of my database is Chinese_Taiwan_Stroke_CS_AS, which is case-sensitive. After changing $Action to $action on line 345 of DbContextExtensions.cs, it worked correctly.

The issue should be resolved in v1.9.1. Can you validate it?

I tried v1.9.1, and it worked correctly.


Fix send a PR link in about page

I wished to contribute from the "about" page but the link was broken.

Thank you


fixadd variant type in decimalv3's compatibilityMatrix table

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Add AccuracyEvaluator

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.09%. Comparing base (3e25420) to head (70270e4).
Report is 11 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #58      +/-   ##
==========================================
+ Coverage   93.58%   98.09%   +4.51%
==========================================
  Files          22       23       +1
  Lines         343      368      +25
  Branches       31       33       +2
==========================================
+ Hits          321      361      +40
+ Misses         22        7      -15

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Update istio deps (pkg and release-builder)

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
7710b5b9f804bc15e937439c7ba537d13d2a6814
link
/test gencheck_istio

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/test-infra repository. I understand the commands that are listed here.


Resolve issues with x64 Branch & Add user expected behaviour

This PR is kind of a continuation of #16. Once I tested the built binaries with x64 branch, it didn't seem to work and once I debugged it, it appeared that the buttons were never allocated in memory thus resulting in faulty RPC Data sent to Discord Client.
So this PR adds these three following:

Interpret the given buttonPrimaryLabel as string properly by using LUA->GetString() != NULL au lieu of type-checking. Consequently the given thing is interpreted according to user's intention

Set buttons dynamically: in case primary isn't supplied and only secondary is present, solely display the second one (instead of crashing)

Slightly change the implementation method for buttons

Add semver as a vector so users can check the library version by DiscordRPCVersion == Vector(1, 2, 1) (in this PR the version is declared as 1.2.1)

Apologies for the late response on this one! I'm just quickly testing this before I merge it but it all looks good.
I have a quick note on the semver stuff


fix: update minor grammar & spelling errors for a react challenge on stateless react components

[ x] I have read freeCodeCamp's contribution guidelines.
[ x] My pull request has a descriptive title (not a vague title like Update index.md)
[ x] My pull request targets the master branch of freeCodeCamp.
[ x] None of my changes are plagiarized from another source without proper attribution.
[ x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[ x] My changes do not use shortened URLs or affiliate links.

Closes #XXXXX
guide/english/certifications/front-end-libraries/react/create-a-stateless-functional-component/index.md

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


Expected behavior
Switch pro controller to connect as a switch controller and be able to move around and play.,
Desktop (please complete the following information):

OS: windows 11
Minecraft 1.19
Fabric fabric 0.14.8
Mods https://gitlab.com/drakosoldier298/vanillia-modpack/-/tree/1.19
Version 1.4.0
Branch unknown, release I presume

Does the controller name appear orange in the Controller Options? If so, try creating a custom mapping.


make sure we have tracking events for new flex features

Blocked by issue #1400

closed via #1451


Create API for users

Manage users via API

This should be both read and write. Manage it the best way possible.


[Form Recognizer] Update to support typed document field

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-formrecognizer


Add a config option to allow preemptive auth

Hi Jest Team,
I tried to use the basic auth feature added in 0.1.6
JestClientFactory factory = new JestClientFactory();
factory.setHttpClientConfig(
    new HttpClientConfig.Builder("http://localhost:9200")
        .defaultCredentials("global_user", "global_password")
        .build()
);

The problem is that it doesn't do preemptive auth and AFAIK there is no way to enforce it.
Example access log from nginx fronting a test cluster
<IP_ADDRESS> - - [26/May/2015:17:27:44 +0200] "POST /index/file/_search HTTP/1.1" 401 195 "-" "Apache-HttpClient/4.4.1 (Java/1.7.0_60)"
<IP_ADDRESS> - user [26/May/2015:17:27:44 +0200] "POST /index/file/_search HTTP/1.1" 200 912 "-" "Apache-HttpClient/4.4.1 (Java/1.7.0_60)"
<IP_ADDRESS> - - [26/May/2015:17:27:44 +0200] "POST /index/file/_search HTTP/1.1" 401 195 "-" "Apache-HttpClient/4.4.1 (Java/1.7.0_60)"
<IP_ADDRESS> - user [26/May/2015:17:27:44 +0200] "POST /index/file/_search HTTP/1.1" 200 132 "-" "Apache-HttpClient/4.4.1 (Java/1.7.0_60)"
<IP_ADDRESS> - - [26/May/2015:17:28:08 +0200] "POST /index/file/d6371446-cbbc-44af-9d3f-e299047fef79/_update HTTP/1.1" 401 195 "-" "Apache-HttpClient/4.4.1 (Java/1.7.0_60)"
<IP_ADDRESS> - user [26/May/2015:17:28:08 +0200] "POST /index/file/d6371446-cbbc-44af-9d3f-e299047fef79/_update HTTP/1.1" 201 96 "-" "Apache-HttpClient/4.4.1 (Java/1.7.0_60)"
<IP_ADDRESS> - - [26/May/2015:17:28:12 +0200] "POST /index/file/d6371446-cbbc-44af-9d3f-e299047fef79/_update HTTP/1.1" 401 195 "-" "Apache-HttpClient/4.4.1 (Java/1.7.0_60)"
<IP_ADDRESS> - user [26/May/2015:17:28:13 +0200] "POST /index/file/d6371446-cbbc-44af-9d3f-e299047fef79/_update HTTP/1.1" 200 96 "-" "Apache-HttpClient/4.4.1 (Java/1.7.0_60)"

As you can see, every request starts without sending credentials, it ends up in 401 and then is repeated with basic auth included and ends up as 200.
This is an http client design decision http://stackoverflow.com/questions/2014700/preemptive-basic-authentication-with-apache-httpclient-4 and an example way to work around this is presented in their svn https://subversion.jfrog.org/jfrog/build-info/trunk/build-info-client/src/main/java/org/jfrog/build/client/PreemptiveHttpClient.java
client.getCredentialsProvider().setCredentials(
                    new AuthScope(AuthScope.ANY_HOST, AuthScope.ANY_PORT),
                    new UsernamePasswordCredentials(userName, password)
            );
            localContext = new BasicHttpContext();

            // Generate BASIC scheme object and stick it to the local execution context
            BasicScheme basicAuth = new BasicScheme();
            localContext.setAttribute("preemptive-auth", basicAuth);

            // Add as the first request interceptor
            client.addRequestInterceptor(new PreemptiveAuth(), 0);

The problem is that I see no way to add request interceptor and no way to add this local context on each request.
This is an important feature in context of elasticsearch, as usually you have just one server with one user/pass for your app. So sending each request twice seems really a waste in case of elasticsearch under a heavy load.
Do you agree this is a valid feature to have? Any suggestions on the implementation? I think that from an API consumer point of view the best would be to add new method preemptiveAuth(boolean) to HttpClientConfig.Builder.
Cheers,
Igor

As of version 4.1 HttpClient automatically caches information about hosts it has successfully authenticated with. Please note that one must use the same execution context to execute logically related requests in order for cached authentication data to propagate from one request to another. Authentication data will be lost as soon as the execution context goes out of scope.

so it should be enough for us to use a

As far as I understood interceptor is only useful when you don't want to or can't set the context at each request execution, but we can easily do that so I don't see the need.
Another safer solution is not blindly doing preemptive auth but using the auth caching mechanism as described in here:

As of version 4.1 HttpClient automatically caches information about hosts it has successfully authenticated with. Please note that one must use the same execution context to execute logically related requests in order for cached authentication data to propagate from one request to another. Authentication data will be lost as soon as the execution context goes out of scope.

This way auth challenge is done only once and all we need to do is give the same context (which we let httpclient to fill in during first request) in all httpClient.execute calls. The problem now is context instances are not thread safe and we have an async client... Also I'm not sure if it is good practice to just use the same context for the lifetime of the client. Implementation of this would sure include some locks here and there so even though nicer I'm ready to give up on this solution unless someone comes up with a clean way to implement it.
Otherwise yes it's a valid feature, would you like to take it?

Thanks for the detailed reply.
I'd like to take this feature, but the thing it is that next two weeks I'm on PTO and then probably will have a lot of stuff to catch up. So I can't promise any ETA. If lets say in 1-1.5 months time there is no other person I'll give it a shot.

No probs. I don't think we'll have a new version/release in a month so I'll leave this for you (unless I spontaneously end up with a commit).

@puszczyk Have you get around to implementing this? If not I may start it.

Sorry, I do not think I can do it now, so I have to yield the right to implement it ;-)

Any updates on this issue?
If not I have a look

Please feel free to do so @david-tejedor :)

Hello all, do you intend to add this in the next release? It is very important for my use case and I believe for a lot of other users too.


chore: remove rootDir from tsconfig and fix the paths

Description
This removes the rootDir property from the tsconfig for TS to recognize
eventual files imported from other blackout packages, thus avoiding the
'rootDir' is expected to contain all source files error.
This also fixes the paths to use the packages folders we're working on,
instead of the node_modules.
Checklist

[x] The commit message follows our guidelines
[ ] Tests for the respective changes have been added
[ ] The code is commented, particularly in hard-to-understand areas
[x] The labels and/or milestones were added

Disclaimer
By sending us your contributions, you are agreeing that your contribution is made subject to the terms of our Contributor Ownership Statement

Question: After the removal of the rootDir, when you run yarn types, does the generated dist folder contains the correct layout (i.e. what is inside the src folder)? If not, then you might need to add the rootDir setting to the tsconfig.build.json files.

Indeed I had to add the rootDir! Can you double-check, please? Thanks 😄


Support doctor upload for subformulas

doctor does not seem to support upload of formulas that call subformulas. It would be great to extend the tool to support this, so we can fully leverage the power of subformulas.

It does under certain conditions. You have to make sure that the bundle of formulas it’s uploading contains all subformulas.
It certainly should also support your case, but there is a workaround.


Implemented adding 'Remark' command

As per tutorial~
Still a WIP

Tutorial done! Can be closed now


feat: Catalogs functionality (Presto)

SUMMARY

Functionality added to fetch catalogs and shortlist schemas based on catalogs for presto databases.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

TBD, working on it.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[x] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[x] Introduces new feature or API
[ ] Removes existing feature or API

Hey @RahulRaj97, thanks for your contribution! Could you post a screenshot for the dataset Modal if the Database(such as MySQL) doesn't have catalog?
BTW, although there are multiple layers of namespaces in database systems, many databases do not fully implement them. currently, Superset just uses Database/Schema/Table in both SQLLab and Explore. If we add Catalog in the dataset model, we should append this level into SQLLab.