schedule function emits with a parameter

I installed the package today and I noticed that whenever the schedule function executes the function we pass as an argument, it receives 1 parameter which is a date corresponding to the time at which it was ran. This is not documented and is quite annoying when passing functions to be called, expecting there to be no parameters passed. This isn't in the @types/node-cron package either so I assume it is a bug that I would be happy to PR a fix for, if you'd like.
As a note, this issue is caused by this line:
https://github.com/node-cron/node-cron/blob/fbc403930ab3165ffef7d53387a29af92670dfea/src/task.js#L17

Hi @ImRodry, yep, I've added this to make the date that the task started available inside the task... I don't know if this makes sense anymore...
However, there are two PR (#221 and #190) that are adding the received options on scheduling as an extra param to the task function. I think passing the options is a good thing to allows pass some data to the task, and I wonder about adding a startedAt to the options before pass that to the task function.
What do you think?

From those two PRs I prefer the implementation in #190 that allows you to choose which args get passed when executing, and I think that PR could just remove now as a whole since it will always be Date.now (or new Date() if you want the full object). That also has some limitations which is that the passed args are static, which makes it rather redundant. IMO the best option would be to call the function with no parameters, since whatever you were gonna pass can always be obtained in some other way since it’s static. That would require a semver major change of course, so let me know what you think of that.


Add documentation

This is still work in progress ...

@martindurant, @aaronspring this is ready for review when you get a moment. You can view the rendered version from https://intake-thredds--19.org.readthedocs.build/en/19/

can we give this a nicer name instead of name not found?

I would also add section in the tutorial about the different drivers and simplecache. Should I add this?

Great improvements throughout.
The notebook doesn't show that well on github, but maybe that doesn't matter.

can we give this a nicer name instead of name not found?

I thought I could just do
catalog = intake.open_thredds_cat(cat_url, name='noaa-ersst-catalog')

and intake would take care of the rest in https://github.com/NCAR/intake-thredds/blob/3d5e08c3d784f235f8139b8b1228629138da0eea/intake_thredds/cat.py#L8-L11
For some reason, the name argument doesn't get propagated to the base class and self.name stays None no matter what. @martindurant, I'm missing anything here?

Yes, you can pass name= to Catalog, and intake.open_thredds_cat is intake_thredds.cat.ThreddsCatalog.
Is this line resetting the value?

Yes, you can pass name= to Catalog, and intake.open_thredds_cat is intake_thredds.cat.ThreddsCatalog.
Is this line resetting the value?

👍🏽 Good catch. Thanks! Should be fixed now

Not sure what's causing the ModuleNotFoundError: No module named 'msgpack' error on RTD. It was happening before and I just started seeing it a few minutes ago. Will look into this later today.
Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/intake-thredds/envs/19/lib/python3.7/site-packages/sphinx/config.py", line 327, in eval_config_file
    execfile_(filename, namespace)
  File "/home/docs/checkouts/readthedocs.org/user_builds/intake-thredds/envs/19/lib/python3.7/site-packages/sphinx/util/pycompat.py", line 88, in execfile_
    exec(code, _globals)
  File "/home/docs/checkouts/readthedocs.org/user_builds/intake-thredds/checkouts/19/docs/source/conf.py", line 17, in <module>
    import intake_thredds
  File "/home/docs/checkouts/readthedocs.org/user_builds/intake-thredds/envs/19/lib/python3.7/site-packages/intake_thredds-0.0.post56-py3.7.egg/intake_thredds/__init__.py", line 8, in <module>
    from .source import THREDDSMergedSource  # noqa: F401
  File "/home/docs/checkouts/readthedocs.org/user_builds/intake-thredds/envs/19/lib/python3.7/site-packages/intake_thredds-0.0.post56-py3.7.egg/intake_thredds/source.py", line 3, in <module>
    from intake_xarray.base import DataSourceMixin
  File "/home/docs/checkouts/readthedocs.org/user_builds/intake-thredds/envs/19/lib/python3.7/site-packages/intake_xarray/__init__.py", line 6, in <module>
    from intake.container import register_container
  File "/home/docs/checkouts/readthedocs.org/user_builds/intake-thredds/envs/19/lib/python3.7/site-packages/intake/container/__init__.py", line 8, in <module>
    from .dataframe import RemoteDataFrame
  File "/home/docs/checkouts/readthedocs.org/user_builds/intake-thredds/envs/19/lib/python3.7/site-packages/intake/container/dataframe.py", line 11, in <module>
    from .base import RemoteSource, get_partition
  File "/home/docs/checkouts/readthedocs.org/user_builds/intake-thredds/envs/19/lib/python3.7/site-packages/intake/container/base.py", line 8, in <module>
    import msgpack
ModuleNotFoundError: No module named 'msgpack'


aws-c-http: 0.6.19 -> 0.6.20

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/aws-c-http/versions.
meta.description for aws-c-http is: "C99 implementation of the HTTP/1.1 and HTTP/2 specifications"
meta.homepage for aws-c-http is: "https://github.com/awslabs/aws-c-http"
meta.changelog for aws-c-http is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.6.20 with grep in /nix/store/23zrwxs8hmjplkwz1rm6g2y2nhsrb8d2-aws-c-http-0.6.20
found 0.6.20 in filename of file in /nix/store/23zrwxs8hmjplkwz1rm6g2y2nhsrb8d2-aws-c-http-0.6.20

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

276 total rebuild path(s)

276 package rebuild(s)

First fifty rebuilds by attrpath
appvm
arrow-cpp
arrow-glib
aws-c-auth
aws-c-http
aws-c-mqtt
aws-c-s3
aws-crt-cpp
aws-sdk-cpp
bundix
cabal2nix
cached-nix-shell
cachix
colmena
comma
common-updater-scripts
crate2nix
crystal2nix
dep2nix
dydisnix
fusionInventory
gnome.gnome-packagekit
gnome.gnome-software
gnome.simple-scan
go2nix
haskellPackages.cabal2nix-unstable
haskellPackages.cachix
haskellPackages.hercules-ci-agent
haskellPackages.hercules-ci-cli
haskellPackages.hercules-ci-cnix-expr
haskellPackages.hercules-ci-cnix-store
haskellPackages.nix-paths
haskellPackages.nvfetcher
haskellPackages.update-nix-fetchgit
hci
hercules-ci-agent
home-assistant-component-tests.environment_canada
home-manager
hydra_unstable
intensity-normalization
libnixxml
libsForQt5.discover
libsForQt5.packagekit-qt
libsForQt512.discover
libsForQt512.packagekit-qt
libsForQt514.discover
libsForQt514.packagekit-qt
libsForQt515.discover
libsForQt515.packagekit-qt
lispPackages.quicklisp-to-nix

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/23zrwxs8hmjplkwz1rm6g2y2nhsrb8d2-aws-c-http-0.6.20 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/23zrwxs8hmjplkwz1rm6g2y2nhsrb8d2-aws-c-http-0.6.20
ls -la /nix/store/23zrwxs8hmjplkwz1rm6g2y2nhsrb8d2-aws-c-http-0.6.20/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 45m and timed out

Maintainer pings
cc @r-burns for testing.

@ofborg eval

https://github.com/ofborg eval


broken intra conflicts and providers checks with input packages

Conflict checks are not done between AUR packages that are inputed (manually or via -Sua pkg), resulting in a broken ConflictsCheck() in some cases:
$ pacaur -S pacaur-git pacaur
:: Package(s) pacaur-git pacaur not found in repositories, trying AUR...
:: resolving dependencies...
:: looking for inter-conflicts...
:: pacaur and pacaur-git are in conflict. Remove pacaur-git? [y/N] y
:: pacaur-20130220-1 is up to date -- reinstalling
:: pacaur-git-20130220-1 is up to date -- reinstalling

AUR Targets  (2):

Name        Old Version  New Version

pacaur      20130220-1   3.4.7-1
pacaur-git  20130220-1   20130216-1

Proceed with installation? [Y/n] n

Here is pacman output for 2 conflicting packages, which is also what should be expected from pacaur with AUR packages:
$ pacman -S openjdk6 jre7-openjdk
resolving dependencies...
looking for inter-conflicts...
error: unresolvable package conflicts detected
error: failed to prepare transaction (conflicting dependencies)

Here's another example with provide:
pacaur -S neovim-git python-neovim
:: Package(s) neovim-git python-neovim not found in repositories, trying AUR...
:: resolving dependencies...
:: no results found for neovim

The package neovim-git of course provides neovim.

Provider of targets has been removed in e468c9415990d2ff438fe3053282b491fcbf36a6, prior to the implementation of #543. This should be checked and readded.

The providers issue has been fixed in 4243d7869c8f933899624d66c8b8d296fb00144f. No versioning check done on providers since it doesn't make sense on a rolling distrib.

Commit 4243d7869c8f933899624d66c8b8d296fb00144f introduces a regression with versioning check.
$ pacaur -S aura
:: Package aura not found in repositories, trying AUR...
:: resolving dependencies...
:: no results found for haskell-regex-pcre-builtin
:: no results found for haskell-aeson<1
:: no results found for haskell-servant-client>=0.9
:: no results found for haskell-servant>=0.9

$ pacaur -S aura
:: Package aura not found in repositories, trying AUR...
:: resolving dependencies...
:: no results found for haskell-regex-pcre-builtin
:: no results found for haskell-servant
:: no results found for haskell-servant-client

The faulty code has been removed in 11c66630f11cf590570f0a7502b6b12464b5e242. It doesn't seem necessary anymore anyway, since I cannot reproduce any of the issue regarding provider checks listed above.


adding toArray()-method

Hi,
I need a way to serialize the Larafirebase so I am able to store it in a database. However, I am not able not access the data due to the private fields.
Therefore, I added this toArray()-method.
It would be great, if you merge this PR.
Thank you very much for this project and your time!
~ Maschga

I just saw this repository: https://github.com/gg-innovative/larafirebase
Is this the newer one? Should I switch to this one?

Is this the newer one? Should I switch to this one?

@Maschga please do it, thanks.


[FEATURE REQ] #endif comment for more clear #ifdefs

Is your feature request related to a problem? Please describe.
Currently #ifndef's and other macro's do not have an indication on the ending #endif where they began.
Describe the solution you'd like
In a very basic example here, it would be nice if it was structured as so:
#ifndef _az_HTTP_HEADER_H
#define _az_HTTP_HEADER_H
...
#endif /* _az_HTTP_HEADER_H */

In a more complex example for the _az_cfg.h this code snippet would turn into:
#if defined(__clang__)

#pragma clang diagnostic ignored "-Wmissing-field-initializers"
#pragma clang diagnostic ignored "-Wmissing-braces"

#endif /* __clang__ */

#ifndef _az_CFG_H
#define _az_CFG_H

#ifdef _MSC_VER
#define AZ_INLINE static __forceinline
#elif defined(__GNUC__) || defined(__clang__)
#define AZ_INLINE __attribute__((always_inline)) static inline
#else
#define AZ_INLINE static inline
#endif /* _MSC_VER */

#if defined(__GNUC__) && __GNUC__ >= 7
#define AZ_FALLTHROUGH __attribute__((fallthrough))
#else
#define AZ_FALLTHROUGH \
  do { \
  } while (0)
#endif /* (__GNUC__) && __GNUC__ >= 7 */

#ifdef _MSC_VER
#define AZ_NODISCARD _Check_return_
#elif defined(__GNUC__) || defined(__clang__)
#define AZ_NODISCARD __attribute__((warn_unused_result))
#else
#define AZ_NODISCARD
#endif /* _MSC_VER */

#endif /* _az_CFG_H */

The C SDK for IoT currently does this and it makes it easier to follow opening and closing of these #ifdef's
Describe alternatives you've considered
Feel free to format the comment after the closing #endif how you would like.
Additional context
Add any other context or screenshots about the feature request here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[X] Description Added
[X] Expected solution specified

Can we use // style comments? /**/ are harder to comment out.

Can we use // style comments? /**/ are harder to comment out.

Totally cool with me. Seems reasonable

https://github.com/Azure/azure-sdk-for-c/pull/377

Closed since was merged. Thanks @antkmsft


[KYUUBI #4788][K8S][HELM] Use StatefulSet instead of Deployment

Why are the changes needed?

Use StatefulSet instead of Deployment, add a headless service for statefulset
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[x] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

btw, our production kyuubi changed to this version

@camper42 thanks for sharing, cc @turboFei @zwangsheng too

Codecov Report

Merging #5062 (97b76ea) into master (ac21e27) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #5062   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files         563     564    +1
  Lines       31273   31293   +20
  Branches     4096    4096
======================================
- Misses      31273   31293   +20

see 12 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Why is it needed to replace Deployment by StetefullSet, i.e. what kind of problem does it solve?
Would it be enough to use headless Service and Deployment?

as our deployment, we want to persist server log later, so statefulset + pvc maybe a better choice
and ref to #4788: Kyuubi server is actually stateful for batch API, especially the batch session recovery relies on stable IP address/hostname after restarting Kyuubi server

Which Service (headless or not) clients should connect to and use?

at the moment headless service only used to provide stable network ID. Our client is not in the same cluster as the kyuubi server, in this case we use kyuubi-thrift-binary's(LoadBalancer) external IP.
And I'm not sure which service it better for in-cluster client connect. maybe @pan3793 know more about it.

client -> DNS: kyuubi-headless.kyuubi.svc.cluster.local -> pod IP (may have DNS cache issue)
client -> kyuubi-protocol cluster IP -> pod ?

https://kyuubi.readthedocs.io/en/v1.7.1-rc0/deployment/high_availability_guide.html#client-side-usage

then it can randomly pick one of the Kyuubi service uris from the specified ZooKeeper addresses in the /kyuubi path.

in ZK HA mode, client direct connect to a randomly picked kyuubi server.
In kubernetes, whether we use headless services (DNS service discovery) or ClusterIP (load balancing), it does not involve zk, so can HA be achieved without zk?

I'm not sure which service it better for in-cluster client connect.

For THTIFT_BINARY (TCP) it does not matter, because it's a long connection, the whole lifecycle of a session while sticky in one Kyuubi Server Pod.
For REST (HTTP), I think headless should be better. In the batch submission (aka Spark Jar application), suppose the spark-submit was executed in the Kyuubi-Server-1, the following request to fetch local logs of spark-submit process must be handled by the same Pod, the current implementation is:

Client => [Kyuubi Server Service] => random Kyuubi Server Pod
2.1 If the Kyuubi Server Pod holds the batch session => handle and return response
2.2 If the Kyuubi Server Pod does not hold the batch session, check the state store (and external storage like MySQL) to find which Kyuubi Server holds it and forward the request to it, wait for the response, then pass through it to the Client

The 2.2 could be enhanced by

the response contains the Kyuubi-Server-1 Pod address
the client always(until network exception) tries to access the Kyuubi-Server-1 directly to avoid Kyuubi Server internal forwarding overhead.

In such a case, I would say Headless Service seems more suitable.

headless service kyuubi-headless.kyuubi.svc.cluster.local will resolve to:

Pod 0 IP
Pod 1 IP
...
Pod N IP

Pod DNS(stable network ID) will resolve to Pod IP:

kyuubi-0.kyuubi-headless.kyuubi.svc.cluster.local -> Pod 0 IP
kyuubi-1.kyuubi-headless.kyuubi.svc.cluster.local -> Pod 1 IP
...
kyuubi-N.kyuubi-headless.kyuubi.svc.cluster.local -> Pod N IP

Cluster IP/External IP will proxy to one Pod IP (and stick for a client IP if sessionAffinity enabled)

in the current implementation, no difference between service expose methods for kyuubi REST ?

the response contains the Kyuubi-Server-1 Pod address
the client always(until network exception) tries to access the Kyuubi-Server-1 directly to avoid Kyuubi Server internal forwarding overhead.

in this implementation, client needs to reconnect to the Kyuubi-Server-1 Pod address, if using HTTP, client can connect by Pod IP address, but if using HTTPS, we need a stable address?

@turboFei could you share some ideas about HTTPS and the Service type chosen?

@turboFei could you share some ideas about HTTPS and the Service type chosen?

we are using stateful set.
A stable FQDN is needed for kyuubi batch recovery.
For HTTPS, stateful set is not a must.

Thanks, merged to master

Supply a new advantage of StatefulSet over Deployment, Utils.findLocalInetAddress.getCanonicalHostName returns different values:

Deployment: Pod name, e.g. kyuubi-bd8f56ddb-59jn5
StatefulSet: FQDN, e.g. kyuubi-server-0.kyuubi-headless.spark.svc.cluster.local

Thus, switching to StatefulSet could overcome the errors like
Caused by: java.net.UnknownHostException: kyuubi-bd8f56ddb-59jn5

when kyuubi.frontend.connection.url.use.hostname=true (default is true)


Version check updates only on second spawn of modal

@artillect

Load the page
New version is caught but link does not get updated when opening the modal
Close the modal and reopen
Link styling updates correctly

I think I've managed to fix it, I'll clean up the branch and get a pull request started


doc: Updated to correct the example configuration settings

CC-1212

CI: No security warnings found

CI: Security warnings found!


Binding the control associated to the NavigationItems (instead of PageType)

Is your feature request related to a problem? Please describe
As far as I know, after checking the docs and the demo source code, binding the control that will be shown in the frame after navigating to the select item is not available. As I understand, the only way to set what will be shown, is set the PageType to a control derived from UIPage. I cannot use this approach as I am trying to follow MVVM, and this way I am unable to control the viewmodel of this page.
If I am wrong, please informe me!
Greetings
Describe the solution you'd like
Implement a way to bind the content associated with the NavigationItem from a viewmodel (similar to a ContentControl)
Describe alternatives you've considered
No response
Additional context
No response

Ive fleshed out an enhancement here (didnt see this one first)
https://github.com/lepoco/wpfui/issues/634


[Bug] how to connect to dcm4chee

Describe the Bug
i use the example and pacs use orthanc :
  const params = {
    StudyInstanceUID: '1.2.2046.79<IP_ADDRESS>30724.105506.19135',
    SeriesInstanceUID: '1.2.2046.79<IP_ADDRESS>30724.105506.07036',
    wadoRsRoot: '/dicom-web',
  }
  const imageIds = await createImageIdsAndCacheMetaData(params);

and i proxy /dicom-web to  http://<IP_ADDRESS>:8042/ . that works well.
but now i change pacs to dcm4chee 。what should i do can get the data?
Steps to Reproduce
like above
The current behavior
like above
The expected behavior
like above
OS
window11
Node version
20
Browser
firefox

you can check out ohif docs https://docs.ohif.org/configuration/dataSources/dicom-web


Bug fixes and usability

@KarlErickson
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Changes made in internal repo.
#please-close


Updating electron version

Hey @arnarthor
I'm working with this lib right now and wondering what path you might want to take for updates. I'm a little new to the electron world, so do you know if there's any issue to updating the electron dep for this library?
Currently it's set to ^2.0.8 in this repo, but I've had no trouble so far running it against ^4.0.4 in my fork https://github.com/Schmavery/bs-electron/blob/094ddbfbd5f6b930a0fdc57892050385f26f0308/package.json#L18
Does it make sense to submit a PR to update it?
(I was also wondering if there's a reason that you pushed the *.bs.js artifacts or if that was a mistake)
If that sounds good to you, I'd love to follow up with PRs with support for menus and whatever other functions we need to expose for our app.
Thanks for making this, saving a lot of time on my end already 😄

Yes of course, feel free to update Electron and whatever PRs you have ready or will have in the future. I haven't been following Electron that much recently since I just switched jobs so I'm not sure if anything is breaking in they're most recent updates, but I'd be glad to assist as much as you need. I have some electron apps using this that I can test the updates on.
If you want I can also give you access to the npm package so you can publish your updates to npm as well 👍
Just let me know what the easiest path for you is

Thanks! I'll close this now that https://github.com/arnarthor/bs-electron/pull/2 is open


How to add domain?

How to add domain?

You can't, better add a domain on reverse proxy nginx (like nginx)


Documentation not aligned in 3-Authorization-II/2-call-api-b2c/

Issue

Please provide us with the following information:

This issue is for the sample

    - [ ] 1-1) Sign-in with Azure AD
    - [ ] 1-2) Sign-in with Azure AD B2C
    - [ ] 2-1) Acquire a Token and call Microsoft Graph
    - [ ] 3-1) Protect and call a web API on Azure AD
    - [x] 3-2) Protect and call a web API on Azure AD B2C
    - [ ] 4-1) Deploy to Azure Storage and App Service
    - [ ] 4-2) Deploy to Azure Static App Service
    - [ ] 5-1) Call a web API using App Roles
    - [ ] 5-2) Call a web API using Security Groups
    - [ ] 6-1) Call Microsoft Graph using on-behalf-of flow
    - [ ] 6-2) Call a web API using Proof of Possession tokens
    - [ ] 6-3) Call a web API using Conditional Access Auth Context

This issue is for a

    - [ ] bug report -> please search issues before submitting
    - [ ] question
    - [ ] feature request
    - [x] documentation issue or request

Minimal steps to reproduce

Follow exact steps to setup the two applications

Any log messages given by the failure

GET http://localhost:5000/hello 401 (Unauthorized)

Expected/desired behavior

GET http://localhost:5000/hello 200

Library version

7fe2cc66f0da2fbbbe33784df4258a59f6512309

Browser and version

Chrome, Edge, Firefox, Safari

Mention any other details that might be useful

It's very nice to have this kind of samples but the readme is not aligned with the application configurations.

API\config.json

Documentation mention tenantID when the key is tenantName
Documentation does not mention anything about the scopes defined in protectedRoutes when by default the value is "demo.read" when in the documentation the steps to follow creates access_as_user scope

SPA\src\authConfig.js

Documentation mention follow values like Enter_the_Application_Id_Here that does not exists.
Documentation does not mention anything about loginRequest and protectedResources

@JPardinas thank you so much for spotting this! Will update the README asap (but feel free to make a PR if you like)


Batched and recursive computations cause trouble to lx.linear_solve and sometimes break the lx.NormalCG with a fixed max_step.

Batched and recursive computations cause trouble to lx.linear_solve and sometimes break the lx.NormalCG with a fixed max_step.
Working example:
import jax
import jax.numpy as jnp

import lineax as lx

p = 10
A = jax.random.normal(jax.random.PRNGKey(0), (p, p))
# define a linear function that is n_mat consecutive matrix-matrix multiplications
def linear_fn(x):
    x = x @ A.T
    return x

# use it in a batched way:

batch_size = 128
x = jnp.ones((batch_size, p))
target = linear_fn(x)  # get output

# define the operator

operator = lx.FunctionLinearOperator(
    fn=linear_fn,
    input_structure=jax.eval_shape(lambda: x)
)

and then on my machine
lx.linear_solve(operator, target, solver=lx.NormalCG(1e-2, 1e-2, max_steps=10))

raises XlaRuntimeError: INTERNAL: Generated function failed: CpuCallback error: RuntimeError: The maximum number of solver steps was reached. Try increasing max_steps.
This error happens regardless of batch_size in the above code and does not happen consistently when changing the shape of A.
Another concern is the slowness of the automatic solver: on my machine, %timeit operator.mv(x) gives 28.6 µs, and computing the transpose with transpose = operator.transpose() and then doing %timeit transpose.mv(target) gives 113 µs. But %timeit lx.linear_solve(operator, target, solver=lx.AutoLinearSolver(well_posed=False)) gives 466 ms, which is 3000 times more costly than a .mv and a transpose evaluation. This ratio gets worse as the batch size increases, which I don't think should happen -- the number of iterations should stay about the same.
With a different shape of A so that lx.linear_solve(operator, target, solver=lx.NormalCG(1e-2, 1e-2, max_steps=10)) does not break, I also observe the same problematic scaling.
Thanks for your help
PS: Thanks for the awesome library; it is really helpful!

I don't think this issue is caused by the batching actually, and it makes sense
it would fail for every batch size (or even no batch.)
CG is pretty susceptible to floating-point errors, especially NormalCG since the
condition number is squared. Generally this means taking a few more than N iterations
for an N x N matrix to correct the floating point errors that have accumulated.
Indeed, setting max_steps=12 or max_steps=13 should fix the issue.
Note that although this doesn't align with the math of CG, it is common
numerically. JAX and Scipy both also fail to solve this with 10 iterations for the same
reason, but JAX does not raise an error at all (quite bad!) and scipy passes its error
information silently via info, which is equivalent to running
lx.linear_solve(..., throw=False).
Regarding lx.AutoLinearSolve, this will never default to an iterative method. This
is because iterative methods struggle when dealing with dense matrices, which
is the typical use-case we expect when using lx.AutoLinearSolve. Specifically,
lx.AutoLinearSolve(well_posed=False) will default to lx.SVD() unless the operator
has the tag lx.diagonal_tag.

Thanks for your swift answer!
So the expected behaviour is that if after max_steps iterations convergence has not been reached, an error is returned?

Whether an error is thrown or not is controlled by the keyword-only argument throw, which defaults to True. So solution = lx.linear_solve(operator, vector, max_steps=max_steps) will throw an error if it does not converge in max_steps iterations, but solution = lx.linear_solve(operator, vector, max_steps=max_steps, throw=False) will not. Note that regardless this information will still be included in solution.results, see: https://docs.kidger.site/lineax/api/solution/.

Ok, this is very clear! Thanks.


QT lib path not found even given lib path

Hi!
When I am building the whole project, I found that the make programme can not find the QTlib even I have given the right path:
/home/idroid/Desktop/clion-2020.1.1/bin/cmake/linux/bin/cmake -DCMAKE_BUILD_TYPE=Debug -DCMAKE_PREFIX_PATH=/home/idroid/Qt/Tools/QtCreator/lib/Qt/lib -G "CodeBlocks - Unix Makefiles" /home/idroid/Desktop/Cheetah-Software
**** Mini-Cheetah build disabled ****
-- We are on a Linux system
-- Embedded is OFF
-- Printing is ON
-- Profiling is ON
-- User interrupt is ON
-- Floats are OFF
-- Long integers (64bit) are ON
-- Code coverage is OFF
-- MKL Pardiso: ON
-- Floats are OFF
-- Long integers (64bit) are ON
CMake Error at sim/CMakeLists.txt:19 (find_package):
Could not find a package configuration file provided by "Qt5Core" with any
of the following names:
Qt5CoreConfig.cmake
qt5core-config.cmake

Add the installation prefix of "Qt5Core" to CMAKE_PREFIX_PATH or set
"Qt5Core_DIR" to a directory containing one of the above files.  If
"Qt5Core" provides a separate development package or SDK, be sure it has
been installed.
I have use the CMAKE_PREFIX_PATH and I also have tried add
SET(Qt5Core_DIR "/home/idroid/Qt/Tools/QtCreator/lib") in CMAKELists.txt but still get the same error, why?
Please help, thank you!
Yujian BAO

What's your OS? It only supports 20.04, I couldn't get it working on my 18.04 machine either.

What's your OS? It only supports 20.04, I couldn't get it working on my 18.04 machine either.

I can build and run the code in 18.04.

change the Qt version number to which you install:
set(Qt5Core_DIR ~/Qt/5.12.0/gcc_64/lib/cmake/Qt5Core)
set(Qt5Widgets_DIR ~/Qt/5.12.0/gcc_64/lib/cmake/Qt5Widgets)
set(Qt5Gamepad_DIR ~/Qt/5.12.0/gcc_64/lib/cmake/Qt5Gamepad)


fixed #2006 - [closed]

In GitLab by @liyanq528 on Mar 26, 2019, 05:15
Merges issue#2006 -> dev

In GitLab by @wknet123 on Nov 13, 2019, 11:41
The test coverage for backend is %  38.92%  , check  console log

In GitLab by @wknet123 on Nov 13, 2019, 11:41
closed


Create an account?

When running the application for the first time, I am presented with this screen >
https://ibb.co/gOcPv7
Wondering how I go about creating an account, sorry for the oversight, just dont see an easy entrance into Active Collab from this window...
Active Collab Desktop 0.7.2
linux x64 4.13.0-32-generic

ActiveCollab is a paid service, thus you need to have an account and this inofficial app doesn't provide an account creation screen. Please refer to ActiveCollab: https://activecollab.com/pricing

Interesting, thanks for the info!


Possible to launch metahook through Steam?  (re: renaming metahook.exe to hl.exe?)

I'm running metahook through WINE (Crossover/macOS) and it's working great but I was wondering if I could launch it using Steam directly.  (ie. clicking Half-Life and [Play])
For example, can I rename hl.exe to something else and then rename metahook.exe to hl.exe?
This would make it much easier to start and I believe the arguments (-insecure and -game) could be added using the Steam launch arguments.
The reason this would be nice is because it wouldn't require separate shortcuts but more importantly these are difficult to create on Crossover (it's a pain).

For example, can I rename hl.exe to something else and then rename metahook.exe to hl.exe?

ofc you can, this is what i've done with svencoop.exe

ok - thanks!  I thought metahook.exe called hl.exe but it looks like metahook.exe replaces hl.exe.
thanks again!


How should val.txt, trainval.txt files be in NWPU VHR-10 dataset?

How should val.txt, trainval.txt files be in NWPU VHR-10 dataset?

How should val.txt, trainval.txt files be in NWPU VHR-10 dataset?

In NWPU VHR-10, we train our model based on train.txt, and test our model  based on test.txt

The layout is similar to this:

but there is no trainval.txt and val.txt

but there is no trainval.txt and val.txt

simularly, train.txt plays the role of specify training data, and test.txt plays the role of test.txt


Set property tab

Questions
Answers

Description?
Set tab property.

Type?
bug fix

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fixes PrestaShop/Prestashop#27075.

@PrestaEdit can you target the dev branch please?


Feature Proposal: Support for Displaying Foreign Currencies (e.g., Euro)

I'm writing to propose a new feature for LubeLogger that I believe would greatly benefit users, particularly those operating in regions where currencies other than the default are commonly used.
The proposal involves incorporating functionality into LubeLogger to allow users to view currency values in foreign denominations, such as the Euro (€), alongside the default currency. This feature would enhance the versatility of the application and make it more accessible to users across various regions.
Currency Selection: Users should be able to select their preferred currency from a list of supported options within the app's settings.
Thank you for considering this proposal.
Regards

Duplicate of hargata/lubelog#298 (misconfigured locale)


Add monthDate to renderCustomHeader render props function

When rendering a custom header and displaying more than one month, like this, it becomes possible to sync the month displayed in header and in calendar
fixes #2981

Can you add some specs to make sure this won't break in the future?

@martijnrusschen I added a test for the monthDate prop, I hope I got it right because with Dates it can be tricky for tests to work 24/7/365, please double-check

Looking good, thanks!


Alternate Art (only in japanese)

In the Deckbuilder app, cards with alternative art are now only displayed in Japanese, regardless of the selected language.

Fixed ty


Add pytest.raises(match=_) patterns in test_valid_models.py

This adds match=___ patterns to all uses of pytest.raises in test_valid_models.py.
Previously we were allowing the test to pass if any error was raised. In some cases the error was being raised for the wrong reason (e.g. scale=0 triggered a different validation error). We can avoid these false negatives by specifying which error should be raised.

Codecov Report

Merging #1350 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #1350   +/-   ##
=======================================
  Coverage   95.66%   95.66%
=======================================
  Files         122      122
  Lines        6406     6406
=======================================
  Hits         6128     6128
  Misses        278      278

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 fbaa779...ec995ad. Read the comment docs.


Run tests against postgresql 10

Description
Update the pytest.ini file as such that the tests run against PostgreSQL 10 by default.
Part of inmanta/inmanta-core#3595
Self Check:

[x] Attached issue to pull request
[x] Changelog entry
[ ] Type annotations are present
[ ] Code is clear and sufficiently documented
[ ] No (preventable) type errors (check using make mypy or make mypy-diff)
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Processing this pull request

Merged into branches iso4 in 34b4fe1c03b9cf58b1f4d2a80ef648d44d0dbf83


Fix MSVC warnings

As discussed here,
I have changed types of indices to std::ptrdiff_t. For size types that can never be negative, I changed them to std::size_t, including types of the corresponding indexing operators.
Tested with MSVC 2022 ((19.30) on both 32-bit and 64-bit architecture.
I opened up this pull request to see if you think some of the changes are appropriate. Especially changes made in CMakeList.txt files, which may not be compatible and require further discussions & validations.

Thanks for your work. Could you rebase your changes on my master branch? Note that I force pushed the last commit and you apparently forked in the meantime :sweat_smile:

Thanks for your work. Could you rebase your changes on my master branch? Note that I force pushed the last commit and you apparently forked in the meantime 😅

Just did. I would prefer to merge to the warnings branch instead since this is an ongoing change that probably requires multiple pull requests. Or perhaps you prefer to do it all in one-go?

Just did. I would prefer to merge to the warnings branch instead since this is an ongoing change that probably requires multiple pull requests.

sounds good. I created a new warnings branch from main.

I gave your cmake changes a test run:

I like that catch2 is added using FetchContent. Especially since I use catch2 version3.0.0, which is probably not installed on everyones machine. You can remove this from the CI script as well: https://github.com/maxbachmann/rapidfuzz-cpp/blob/1c21403a1757c8b6ee94312cfcb084bc0268ce5e/.github/workflows/cmake.yml#L19-L25

the new warnings implementation works for me on clang/gcc :+1:

I deleted some pending "Reviews" made by myself. Apparently, GitHub has this "feature" to block me from replying to your reviews if I made reviews myself and that they are pending. (https://github.com/github/feedback/discussions/6893)

I deleted some pending "Reviews" made by myself. Apparently, GitHub has this "feature" to block me from replying to your reviews if I made reviews myself and that they are pending. (https://github.com/github/feedback/discussions/6893)

Wow that's a strange "feature".

I gave your cmake changes a test run:

I like that catch2 is added using FetchContent. Especially since I use catch2 version3.0.0, which is probably not installed on everyones machine. You can remove this from the CI script as well: https://github.com/maxbachmann/rapidfuzz-cpp/blob/1c21403a1757c8b6ee94312cfcb084bc0268ce5e/.github/workflows/cmake.yml#L19-L25
the new warnings implementation works for me on clang/gcc 👍

Thanks. I removed the catch2 installation from the CI script.

There are still some commented out parts in the CMakeLists files, which can be removed now that project_options is used. The rest looks good to me.

There are still some commented out parts in the CMakeLists files, which can be removed now that project_options is used. The rest looks good to me.

I am leaving out the commented out arguments of project_options, just in case they come in handy one day. I also did not add project_options to bench/CMakeLists.txt since gtest is giving me some trouble and I haven't been able to compile it properly.

I am leaving out the commented out arguments of project_options, just in case they come in handy one day.

Yes, I am talking about parts like #find_package(Catch2 3 REQUIRED), which will not be required anymore.

Let me know if they need to be included as well.

Having them in the tests should already be enough, since the tests cover the largest part of the library.

Looks good to me now. Is there still something missing, that you would like to get into this PR?

I was thinking about merging the implementation of MatrixVectorView and ConstMatrixVectorView into a CRTP-like design, but then again it is a trivial detail which may turn out to be more cumbersome than the current design.

It might be nice to reduce the code duplication this way. Since this is not really related to the warnings, you could do this in a separate PR.

Thanks for your work on this

I'll check with clang & GCC next as well.

I'll check with clang & GCC next as well.

Yes getting a couple of errors on clang / gcc ;)

I'll check with clang & GCC next as well.

Yes getting a couple of errors on clang / gcc ;)

Yeah, I was expecting that. I was getting clangd & clang-tidy static analysis warnings as I worked on the MSVC fix.


feat: add new Button

Overview
PR Checklist

[x] Related to designs: https://www.figma.com/file/N6TfoPI5OA7GanX6Zl8nWj/Buttons-✅?node-id=268%3A16
[x] Related to JIRA ticket: GM-121
[x] I have run this code to verify it works
[x] This PR includes unit tests for the code change

Description

📬Published Alpha Packages:@codecademy/gamut-labs@4.4.6-alpha.88e120.0@codecademy/gamut@17.0.0-alpha.88e120.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f90783753ac911063ea0b46--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.4.6-alpha.e010d4.0@codecademy/gamut@17.0.0-alpha.e010d4.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f907a4a8120680d9f4c3dfe--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.5.0-alpha.976768.0@codecademy/gamut@17.0.0-alpha.976768.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f91a8ec1b7bec00f9e518dd--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.5.0-alpha.3d1988.0@codecademy/gamut@17.0.0-alpha.3d1988.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f92dc17294205990e798dd5--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.5.0-alpha.cc57c7.0@codecademy/gamut-styles@7.1.2-alpha.cc57c7.0@codecademy/gamut@17.0.0-alpha.cc57c7.0@codecademy/markdown-overrides@0.3.6-alpha.cc57c7.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f92e2096c387fa63e8892b3--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.5.0-alpha.b02f7d.0@codecademy/gamut-styles@7.1.2-alpha.b02f7d.0@codecademy/gamut@17.0.0-alpha.b02f7d.0@codecademy/markdown-overrides@0.3.6-alpha.b02f7d.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f92e523cca55ea0f4d58e5f--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.5.0-alpha.965d71.0@codecademy/gamut-styles@7.1.2-alpha.965d71.0@codecademy/gamut@17.0.0-alpha.965d71.0@codecademy/markdown-overrides@0.3.6-alpha.965d71.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f931bb11080491ada8cbf97--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.5.0-alpha.c0f212.0@codecademy/gamut-styles@7.1.2-alpha.c0f212.0@codecademy/gamut@17.0.0-alpha.c0f212.0@codecademy/markdown-overrides@0.3.6-alpha.c0f212.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f93251c0e058d214f8b4ae7--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.5.0-alpha.20b5d2.0@codecademy/gamut-styles@7.1.2-alpha.20b5d2.0@codecademy/gamut@17.0.0-alpha.20b5d2.0@codecademy/markdown-overrides@0.3.6-alpha.20b5d2.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f93262398c2a52c3a7899ed--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.5.0-alpha.3658cf.0@codecademy/gamut-styles@7.1.2-alpha.3658cf.0@codecademy/gamut@17.0.0-alpha.3658cf.0@codecademy/markdown-overrides@0.3.6-alpha.3658cf.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f93281410804927878cbea4--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.5.0-alpha.23d137.0@codecademy/gamut-styles@7.1.2-alpha.23d137.0@codecademy/gamut@17.0.0-alpha.23d137.0@codecademy/markdown-overrides@0.3.6-alpha.23d137.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f932f37f6aab73c3e0c6d7d--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.5.0-alpha.efebfd.0@codecademy/gamut-styles@7.1.2-alpha.efebfd.0@codecademy/gamut@17.0.0-alpha.efebfd.0@codecademy/markdown-overrides@0.3.6-alpha.efebfd.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f933e7972e15244f6c9a3ba--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.5.0-alpha.422cb0.0@codecademy/gamut-styles@7.1.2-alpha.422cb0.0@codecademy/gamut@17.0.0-alpha.422cb0.0@codecademy/markdown-overrides@0.3.6-alpha.422cb0.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f96e72b0c018f2b22ba2498--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.5.0-alpha.35504b.0@codecademy/gamut-styles@7.1.2-alpha.35504b.0@codecademy/gamut@17.0.0-alpha.35504b.0@codecademy/markdown-overrides@0.3.6-alpha.35504b.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f96ec86fa9519380bd73665--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.5.0-alpha.493f2a.0@codecademy/gamut-styles@7.1.2-alpha.493f2a.0@codecademy/gamut@17.0.0-alpha.493f2a.0@codecademy/markdown-overrides@0.3.6-alpha.493f2a.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f96ed574db5c0358198095c--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.5.0-alpha.753984.0@codecademy/gamut-styles@7.1.2-alpha.753984.0@codecademy/gamut@17.0.0-alpha.753984.0@codecademy/markdown-overrides@0.3.6-alpha.753984.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f99ea4aefcb980147c42d5b--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@4.6.3-alpha.c3ee62.0@codecademy/gamut-styles@7.1.2-alpha.c3ee62.0@codecademy/gamut@17.0.0-alpha.c3ee62.0@codecademy/markdown-overrides@0.3.6-alpha.c3ee62.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-labs@4.6.3-alpha.4f5ef3.0@codecademy/gamut-styles@7.1.2-alpha.4f5ef3.0@codecademy/gamut@17.0.0-alpha.4f5ef3.0@codecademy/markdown-overrides@0.3.6-alpha.4f5ef3.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://5f99ed1a7867c00dd054df18--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://5f99ed287d6b421248de4be0--gamut-preview.netlify.appDeploy Logs

Dang it, merged the branch by accident...

Dang it, merged the branch by accident...


Implement memstats command in the debugger

Implementation of memstats command in jerry-debugger, python and html client.
Shows the allocated bytes, byte code bytes, string bytes, object bytes, property bytes and heap size.
JerryScript-DCO-1.0-Signed-off-by: Daniel Balla<EMAIL_ADDRESS>

Fixed those styling issues, and yes I also think it shouldn't be added afterall.

As heap_size is not used anymore, I removed it.

Please fix the mentioned style issues, so we can merge the PR.


How to load external video or webcam feed and process it on device side

I am trying an experiment, where i can verify the results of device side models ( old and new,  such as head pose estimation model ). To verify if new models are working correctly with the existing pipelines using blob file and the inference speed, I wanted to know if there is any way where I can use external video or webcam feed and process it on device side instead of host side.
Thanks

Hi @asagar60 ,
We have an example here:
https://docs.luxonis.com/projects/api/en/latest/samples/MobileNet/video_mobilenet/#video-mobilenetssd
If you want to reconstruct stereo depth as well, I'd recommend using Replay module:
https://docs.luxonis.com/projects/sdk/en/latest/features/replaying/
I hope this helps!


[WIP] Add documentation about how to use fastlane with travis

Before merging, this build should pass: https://github.com/Themoji/ios/pull/27

Moving this to a later point, no time to deal with Travis right now


Bug fix: VB-4000 Visitors not showing correctly on past visits

Call different endpoint for visitor information to include unapproved visitors in profile
A bug was discovered where visitors from past visits, who have been unapproved, show up as undefined in visit history on prisoner profile page.
Fixing by calling endpoint to get all social contacts instead of just approved

closing as ive opened a new PR here https://github.com/ministryofjustice/hmpps-manage-prison-visits-orchestration/pull/245


Is there a native way to query AppSync fror createdAt and upadtedAt?

Those fields exist in the DB but it seems like there is no way to query for them with GraphQL and the autogenerated resolvers.

Please feel free to re-open or comment on the issue if this does not answer your question.

Related discussion: https://github.com/aws-amplify/amplify-cli/issues/401
On<EMAIL_ADDRESS>after I added fields for timestamps and amplify push, I could query them:
type MyTable
{
  id: ID!
  createdAt: AWSDateTime # <-- added
  updatedAt: AWSDateTime # <-- added
  ...


Added Philips LCA003 and LCT016 for PowerOnState

This quirk adds the ability to set the PowerOnState (Off, On, LastState) for Philips LCA003 and LCT016.

Thanks for the contribution!


Fix mypy by ignoring that pytest_mock does not provide type stubs

Without this, mypy src/ tests/ fails.

If/when https://github.com/pytest-dev/pytest-mock/pull/395 gets shipped, this will not be necessary.

I think there was an error with my Python setup.


[go] Add debug command to Devfile

What does this PR do?:
This adds a missing debug command to the Go stack.
Which issue(s) this PR fixes:
—
PR acceptance criteria:

[x] Contributing guide
Have you read the devfile registry contributing guide and followed its instructions?
[x] Test automation
Does this repository's tests pass with your changes?
[ ] Documentation
Does any documentation need to be updated with your changes?
[x] Check Tools Provider
Have you tested the changes with existing tools, i.e. Odo, Che, Console? (See devfile registry contributing guide on how to test changes)
Checked with odo.

How to test changes / Special notes to the reviewer:
Running odo dev --debug and attaching a remote debugger to the local port forwarded by odo should work:

/ok-to-test

I would keep the old versions there as they are and added new versions with the changes.

I would keep the old versions there as they are and added new versions with the changes.

Done. I hesitated to do so in the beginning mainly because of the maintenance overhead, but yes, it makes sense to add new versions. I haven't changed the default version for now.

I would keep the old versions there as they are and added new versions with the changes.

Done. I hesitated to do so in the beginning mainly because of the maintenance overhead, but yes, it makes sense to add new versions. I haven't changed the default version for now.

Yes, let's keep the default versions as-is for now.  The Devfile samples may be dependent on them as parents and may affect parsing when moving up to a newer parent version (as I've encountered).  We will need to perform additional testing and/or modify the samples to get them to work.


Pixel 4A / Motorola G7

Both the Motorola G7 and Pixel 4A are unable to pick up emitted WiFi Beacons from the DJI Air 2S.
I'm unsure if this is an issue with the drone or the phones.
I had previously added some info for Motorola G7 and here's the info for the Pixel 4A.

I do not expect anything from this issue than an update to the devices list and closing it...  I have confirmed that scan throttling is off on both devices.

I have also created a content piece using this information if you'd like to see the issues live (https://www.youtube.com/watch?v=J45xw7oT71Y). This video will premiere in 6 hours from this exact message.

Thanks for the update. That is pretty strange since Beacon should be possible to pick up on pretty much all Android devices (iOS unfortunately does not have the necessary API for this).
Just for sanity checking, have you tried also with this app?
https://play.google.com/store/apps/details?id=org.opendroneid.android_osm
I didn't look through your video yet. I will do that when I find a spare moment.
PS: I expect a fair amount of DJI drones to be remoted ID compliant by now. The FAA maintains a list here: https://uasdoc.faa.gov/listDocs

@friissoren okay, that's helpful, let me get this installed and let me re-test.

Confirmed Beacons are fine on Motorola G7 after takeoff.

Confirmed Beacons are fine on Pixel 4A after takeoff.
@friissoren I'm closing this as no-action and sending an email to Dronetag for Drone Scanner.

@friissoren okay... now I have a different issue.
Same phones, under OpenDroneID-OSM they receive one beacon and no more updates... restarting the app will allow new beacons to be picked up.

Here's the video for OpenDroneID-OSM... packets are picked up right away (as little as 3m off the ground), however new beacons are not picked up during flight unless the app is restarted (https://www.youtube.com/watch?v=_ENWov1LQ8s)

It wasn't very clear from your video how long your were flying (but I assume that it was more than a few minutes). That should have allowed more than one "ping" to be received by the phone(s) but it is known that the Wi-Fi Beacon transmission method is not as good as Bluetooth, in terms of making sure the Android receiver devices will pick up the transmissions.
I don't know how DJI eventually implemented it. According to the remote ID specifications, it is possible to either send the pings on any Wi-Fi channel of your choice (minimum of 5 pings per second) or on channel 6 (2.4 GHz) or 149 (5 GHz) (minimum one ping per second).
Sine Wi-Fi defines a lot of different channel frequencies, the receiver has to cycle through those and listen on each channel for a little while before moving to the next. Due to this cycling and the Android drivers/SW stack/HW applying various power consumption optimizations which further reduces the active listening times (and I suspect those strategies can be different from one phone model to another) the update frequency on the receivers is not particularly reliable. Disabling the Wi-Fi throttle mode in the developer settings should help on this but apparently you had already done that.
Your comment that restarting the app would allow it to receive another ping is suspicious. I wonder if a bug has been introduced at some point. I don't have a DJI drone to test with. Is this behaviour something you are seeing consistently? How long does it take after restarting the app to get the first ping? Is that delay constant or varying?
BTW: I don't know if you noticed but in the Info view, at the top, you can see the delta in seconds to when the app previously received a ping from the transmitter. I couldn't follow that in the video but maybe you could try to keep the Info view open for a while and look at that?

Your comment that restarting the app would allow it to receive another ping is suspicious. I wonder if a bug has been introduced at some point. I don't have a DJI drone to test with. Is this behaviour something you are seeing consistently? How long does it take after restarting the app to get the first ping? Is that delay constant or varying?

So I'm gonna rerun the testing for you, but I saw the behaviour consistently. I was able to get pings almost instantly like as soon as I got off the ground (within 5 seconds). However I only saw one ping at the beginning, and every time I restarted the app I picked up new data. I'll get you some extra info (footage).

@friissoren here's the data in video form. This time I kept the info window open and found the issue.
https://www.youtube.com/watch?v=PEaZ9eraTdw
The Motorola G7 captures beacons much more effectively. Takes about 8-18s per beacon.
The Pixel 4A took over 2 minutes to capture the first beacon, and only captured more beacons after restart.
Takes between 55 and 170s per beacon.

Thanks for testing. So at least both phones are able to pick up the signals but with varying success.
I think you got the Wi-Fi throttle setting the wrong way. It needs to be turned off (by default it is on). Google considers the throttling to be something that improves battery life (but will limit the Wi-Fi scanning frequency).

I think you got the Wi-Fi throttle setting the wrong way. It needs to be turned off (by default it is on). Google considers the throttling to be something that improves battery life (but will limit the Wi-Fi scanning frequency).

Okay, maybe it is the throttling that was the issue... upon re-reading the setting, it is "on" for battery saving.

Hi, we had some issues with Drone Scanner while reading Wi-Fi Beacon messages, which is now resolved in the new 1.2.0 version. Please feel free to try it out and let us know if it works for you. Thanks, Lukas


[PORT] Change Adaptive-Expressions pre-built function classes from public to internal

Port this change from botbuilder-dotnet/master branch:
https://github.com/microsoft/botbuilder-dotnet/pull/4389

closes: #4312
Now prebuilt functions in Adaptive-Expressions are internal classes.
Changed projects

AdaptiveExpressions

Close this because this is never exposed in JS.


Configure DNS before configuring the second domain controller

I just tried this procedure. When promoting the second server, it fails with error message "an active directory domain controller for the domain <testdomain.com> could not be contacted".
The fix is to configure the DNS on the VNet first before configuring the second server.

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

ID: 60b0097c-668e-1d37-22b0-89d0bf649f8a
Version Independent ID: 49ccfef6-05bb-9c93-5a05-a2e2fe632b3c
Content: Install Active Directory Domain Services on an Azure virtual machine
Content Source: WindowsServerDocs/identity/ad-ds/deploy/virtual-dc/adds-on-azure-vm.md
Product: windows-server
Technology: ad-ds
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

@K-Dela-R : Do you suggest to simply move the Configure DNS section up before the Configure the second Domain Controller section? (see suggestion below)
Configure DNS
After promoting the new domain controllers in Azure, they will need to be set to the primary and secondary DNS Servers for the virtual network, and any on-premises DNS Servers would be demoted to tertiary and beyond. More information on changing DNS Servers can be found in the article Create, change, or delete a virtual network.
Configure the second Domain Controller
Connect to AZDC02 using the credentials you provided in the script.
.
.
.

That will do. I also suggest to rephrase "After promoting the new domain controllers in Azure..." to say "after promoting the first server..."

Looks good. Unless you want to edit the page yourself, I am prepared to do so and create a pull request with the suggested improvement.

Please do so. I have yet to learn how to go about GitHub. Thanks!


rhel85/90: install docs in rpm to fix missing man pages

When we firstly introduced the v2 manifests, we excluded docs from all rpm
stages because it doesn't make sense to include docs in edge images.
However, when we ported the other image types to v2, we left the flag on.
The side effect of --excludedocs is that we no longer install man pages into
the image. This the default behaviour of rpm and can be seen here:
https://github.com/rpm-software-management/rpm/blob/b88f43b9a3bf39a92ba8ca511d33fdc60f399c1e/macros.in#L555
This is actually quite a major regression in non-edge images, thus this commit
reverts this setting. It would be great to have the option not to install
docs in some rpm stages but as this is a difficult topic and we're already
past the deadline, we will consider this in the future.
I also regenerated all test manifests.
This pull request includes:

[x] adequate testing for the new functionality or fixed issue
[x] adequate documentation informing people about the change such as

[x] create a file in news/unreleased directory if this change should be mentioned in the release news
[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

As this needs an exception, I'm marking this as a draft for now.

I agree that we should revisit this later, this is mainly a quick fix so we fix this before the deadline.


Put up the maintenance page

As an application user, I would like to see a friendly and non-threatening alert page when the application is offline
When we are redeploying, or when we need to halt operations for some other reason, app users should see a friendly interstitial instead of being forcibly logged out.
Client application should understand certain error codes to indicate that the API server is offline, and provide users with a brief explanation as well as a friendly stress-relieving animal mascot. API and gateway should produce this code during either intentional or unintentional downtime.
If the application gateway notices that no API instances are available to serve a request, it should (and probably does) respond to any request with a 503 Service Unavailable error. The client should recognize this as a signal that the service is, in fact, unavailable, and should either put up a persistent warning that we're in offline mode (if we have an offline mode), or just put up a modal/interstitial saying "we have no application right now, but we'll keep checking." And then it should do some kind of polling of the back end to see when the server comes back up.
For scheduled or emergency maintenance, operations should be able to trigger this manually: either we should have a switch we can flip on the application gateway that tells it to consider all our instances as unhealthy for the duration, or we should have a switch on the back-end instance that makes it return a 503 for the health check on the gateway. (Most likely the former, since having to switch multiple instances on and off to engage and disengage maintenance mode would be bad; on the other hand, the gateway is not part of our development flow or our current QA instance, so we probably want both of these in order for local development of the maintenance page to work.)
Possible bells and whistles:

we could send a message along with the 503 status code that tells the client how long we expect the service to be down for
we could have a regular status ping that the client does to make sure that it is still logged in, and have that ping return a "application is going down at time X" message when we are approaching scheduled downtime (though really, we should be prioritizing eliminating scheduled downtime rather than sending out nice messages about it)

current work for a maintenance page is capture in https://app.zenhub.com/workspaces/simplereport-5ff8be8854ae8b000e378210/issues/cdcgov/prime-simplereport/741


Adding CRUD azure cli commands for Managed instance and Managed database resources

Added new Managed instance and Managed database CRUD commands.

Managed instance commands:

az sql mi create
az sql mi show
az sql mi list
az sql mi update
az sql mi delete

Managed database commands:

az sql midb create
az sql midb show
az sql midb list
az sql midb restore
az sql midb delete

This checklist is used to make sure that common guidelines for a pull request are followed.

[x ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[x ] I adhere to the Command Guidelines.

View a preview at https://prompt.ws/r/Azure/azure-cli/6428
This is an experimental preview for @microsoft users.

I am trying to check out your branch on my machine but https://github.com/petrajkogit/azure-cli is empty. Something weird going on?

Does this aim for sprint 38 (release on 6/5)?

@troydai Yes, this should aim the first next release, so I guess that is sprint 38 (release on 6/5)

Please fix the travis build failures.

This PR needs to be merged by the end of the day to make in to the sprint 38 (release on 6/5)

> az sql mi create -h

...
    --edition --tier -e           : The edition component of the sku. Allowed value is
                                    GeneralPurpose.
...

Is that right? GP is the only edition?

My only major comment is on --target-db versus --dest-name param names. And the travis build needs to succeed. Overall looks awesome!

@jaredmoo .  Not sure why I cannot reply directly on your question, but here it is.
We are currently in Public Preview only with GeneralPurpose edition. BusinessCritical is not yet available, and only few whitelisted customers can create those. I agree that this should be got from LocationsCapabilities API but I would do that as a follow-up.
As for why the build is failing, I believe it is due to python SDK. Do you know if it has been published?

0.9.1 has been released with MI support: https://pypi.org/project/azure-mgmt-sql/
You can see the build failure info by clicking 'Details'

At the moment it is failing in style check (https://travis-ci.org/Azure/azure-cli/jobs/386200482), you can repro this locally by running azdev style --pep8 --pylint --module sql

Thanks for addressing comments. Good to merge as soon as build succeeds.

@tjprescott  are your concerns addressed?

LGTM.  @petrajkogit try re-recording your tests to see if it passes CI.  If not, we can look deeper. I didn't see any of the common reasons why the tests would fail.

@tjprescott @jaredmoo Thanks a lot for your help. I am rerecording my tests again since that's why build is failing, and should fix it soon.

Codecov Report

Merging #6428 into dev will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##           dev   #6428   +/-   ##
===================================
  Coverage    0%      0%
===================================
  Files       11      11
  Lines      133     133
  Branches     9       9
===================================
  Misses     133     133

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 4c17d31...5186243. Read the comment docs.


Az login is giving me below error

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Related command

Describe the bug

To Reproduce

Expected behavior

Environment summary

Additional context

A web browser has been opened at https://login.microsoftonline.com/organizations/oauth2/v2.0/authorize. Please continue the login in the web browser. If no web browser is available or if the web browser fails to open, use device code flow with az login --use-device-code.
Decryption failed: [WinError -2146893813] Key not valid for use in specified state.. App developer may consider this guidance: https://github.com/AzureAD/microsoft-authentication-extensions-for-python/wiki/PersistenceDecryptionError
Please report to us via Github: https://github.com/Azure/azure-cli/issues/20231

@jiasli for awareness

@gunjan4425 Thank you for reaching out to us. did you get a chance to try https://github.com/Azure/azure-cli/issues/20231#issuecomment-1006381416 or any other solution provided here https://github.com/Azure/azure-cli/issues/20231.
Also do share your environment details with us az --version

yes. i tried to uninstall the azure CLI and tried to install via powershell but still it is giving the same error.
please find the azure cli version below
PS C:\Windows\system32> az --version
azure-cli                         2.37.0
core                              2.37.0
telemetry                          1.0.6
Dependencies:
msal                            1.18.0b1
azure-mgmt-resource             21.1.0b1
Python location 'C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\python.exe'
Extensions directory 'C:\Users\hp.azure\cliextensions'
Python (Windows) 3.10.4 (tags/v3.10.4:9d38120, Mar 23 2022, 22:57:10) [MSC v.1929 32 bit (Intel)]
Legal docs and information: aka.ms/AzureCliLegal
Your CLI is up-to-date.
Please let us know how we are doing: https://aka.ms/azureclihats
and let us know if you're interested in trying out our newest features: https://aka.ms/CLIUXstudy

I am also facing this issue with both az login and connect-azaccount.

and even if I login to connect-azaccount which does not show any errors at login, but then try run my terraform, I get the attached:

any suggestions?

I am also facing this issue with both az login and connect-azaccount.

and even if I login to connect-azaccount which does not show any errors at login, but then try run my terraform, I get the attached:

UPDATE:
Solved it as suggested before, by deleting the .azure folder in the C:\Users\username folder

Thanks, had the error in Visual Studio Tried to get token using Visual Studio. Access token could not be acquired.  Exception for Visual Studio token provider Microsoft.Asal.TokenService.exe : TS003: Error, TS004: Unable to get access token.  'AADSTS50076: Due to a configuration change made by your administrator, or because you moved to a new location, you must use multi-factor authentication to access . Nothing helped until I tried your solution, saved my day.


Allow use of preprocessors + CSS modules in .vue files

Closes #2370
@JeffreyWay Do you want me to back port this to Mix v5 or leave it for Mix v6 only?

Let's hold off for now.

How to get this working with React and module.scss? Any clue?
I've tried it, but get no luck.

How to get this working with React and module.scss? Any clue?
I've tried it, but get no luck.

Can you give an example of what you're doing and what you expect the result to be? (A reproduction repo + readme/instructions would help the most). When I test React + CSS modules it works as I expect (see: https://github.com/JeffreyWay/laravel-mix/pull/2732/commits/e0f2fba7df1b893a2c69fbe32a63593175ba0070)

Can you give an example of what you're doing and what you expect the result to be? (A reproduction repo + readme/instructions would help the most). When I test React + CSS modules it works as I expect (see: https://github.com/JeffreyWay/laravel-mix/pull/2732/commits/e0f2fba7df1b893a2c69fbe32a63593175ba0070)

@thecrypticace Here it's https://github.com/nmfzone/laravel-mix-repro.

@thecrypticace Here it's https://github.com/nmfzone/laravel-mix-repro.

@thecrypticace Ah, I found the problem. So, the css module is generated in the js folder (public/js), and I don't include it on my blade views. Is it possible to change the css module output file (extracted by mini-css-extract-plugin i guess)?

@thecrypticace Ah, I found the problem. So, the css module is generated in the js folder (public/js), and I don't include it on my blade views. Is it possible to change the css module output file (extracted by mini-css-extract-plugin i guess)?

Oh that's odd. When I import them I don't get an extra CSS file. It's included in the JS file:

I'll have to look further into it.

Oh that's odd. When I import them I don't get an extra CSS file. It's included in the JS file:

I'll have to look further into it.

@nmfzone I figured out the issue. It will be fixed whenever the next version of Mix is released.

@nmfzone I figured out the issue. It will be fixed whenever the next version of Mix is released.

@thecrypticace Awesome. But how about change the output of the extracted css? It seems that it's good to extract the css, but change the output to the public/css.

@thecrypticace Awesome. But how about change the output of the extracted css? It seems that it's good to extract the css, but change the output to the public/css.

It seems that Vue support that (change the extract path) https://github.com/JeffreyWay/laravel-mix/blob/master/src/components/Vue.js#L172. It's good to see that feature in React too I guess.

It seems that Vue support that (change the extract path) https://github.com/JeffreyWay/laravel-mix/blob/master/src/components/Vue.js#L172. It's good to see that feature in React too I guess.

Yeah, we should allow it to be configured globally for all css. Not just as a Vue feature.

Yeah, we should allow it to be configured globally for all css. Not just as a Vue feature.


AB#23605 - welcome page UX design bug fix

23605-welcome-website-UX-feedback

@vinayajoseph-aot, @ravirammysore  - can you guys look at the merge conflicts and resolve together please?
@vinayajoseph-aot  - I think max wanted the buttons on the top right closer to this

Have create a constant gap of 1rem * 3/2 as per the spacings - right now its still reactive and scales
(dont use what I have done as verbatim, just an example)

I also see that there is already a V2 of V2: please follow up if this is what we should be implementing here:


Possible SQL injection vector through string-based query construction.

Checklist

[x] I've searched for similar issues.
[x] I'm using the latest version of HTTPie.

Possible SQL injection vector through string-based query construction.
Location: superset/tests/integration_tests/csv_upload_tests.py:373
372             .get_sqla_engine()
373             .execute(f"SELECT * from {EXCEL_UPLOAD_TABLE}")
374             .fetchall()
Solution :
Sanitize and control the input data when building such SQL statement strings.

This is not httpie-related.


[WIP] Add io.js to .travis.yml

Needed to make it easier to test against different versions of node/io.js binaries locally and took the chance to add interim support for travis as well.
Marked as WIP because there is currently a pending PR which is necessary to make tests pass on<EMAIL_ADDRESS>I'll rebase this branch when/if it gets merged.

nvm already supported iojs,  travis-ci didn't upgrade there nvm yet?

We can wait travis supporting it, and then config can be simple.
https://github.com/travis-ci/travis-ci/issues/3108

@ruimarinho can you take out the commits to the other PR from this PR?

let's wait until travis updates nvm. not really worth adding all these hacks to the travis file. we know koa works on iojs right now...

i'm constantly checking travis ci for support by rerunning https://travis-ci.org/expressjs/serve-static/jobs/47808983 over and over until it works.

@dead-horse no, it's taking them years to update nvm 😢
@jonathanong I can remove the travis config and keep the env settings on the makefile to make it easier to do cross-platform tests locally, defaulting to io.js perhaps? In any case, I just wanted to see how travis would behave with io.js. Personally, I'd be fine adding it even if it gets removed next week :)

@jonathanong rebased and pushed an update for io.js 1.0.4. Travis still hasn't updated nvm yet. Let me know if you still think it's better to close the PR.

not really down with the travis.yml hack. i'd personally rather wait until travis actually supports iojs. @koajs/owners thoughts?

I also agree on waiting, at least that is my plan AppVeyor already does io.js; I don't know what is taking Travis CI so long.

waiting seems reasonable to me as well!

I agree, wait :-)

Alright, closing :) thanks all!

@ruimarinho wanna make a new PR for iojs? :D

@jonathanong done :)


Fixed possible typos

Breaking change
No breaking change
Proposed change
Identified possible typos
Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[x] Code quality improvements to existing code or addition of tests

Nope


How can I do word wrap in xfxEntry ?

Version Number of Control:  1.1.3
Device Tested On: Android and I phone
Simulator Tested On:

Affects

[ ] iOS
[ ] Android

Input  text given in xfxEntry is  not wrapping to the next line. Is there any property similar to line break mode like in labels ?
Please suggest some solution to slove this issue.

Whoah, I'm not sure how I missed your question, sorry.
The Entry is only a single line by design. What you want is an Editor, unfortunately I have yet to build an XfxEditor that will support floating labels.


Point editors to this tree-sitter

helix: https://helix-editor.com/

https://github.com/helix-editor/helix/blob/89a7cde2f05fbf2eae212d9301301a8377432be5/languages.toml#L579

neovim: https://neovim.io/

https://github.com/nvim-treesitter/nvim-treesitter/blob/2a496fa4e89034014dd054f04c061ba7d6023a0c/lua/nvim-treesitter/parsers.lua#L1952

zed: https://zed.dev/

https://github.com/crystal-lang-tools/zed-crystal/tree/nobody/crystal-tree-sitter

vscode

TBD

I was looking if these editors use the same set of captures for highlighting, but seems they don't.
Zed uses https://zed.dev/docs/extensions/languages#syntax-highlighting
NVim uses https://neovim.io/doc/user/treesitter.html#treesitter-highlight-groups
Helix uses https://docs.helix-editor.com/themes.html#scopes
Seems that nvim tried to use the same groups used in Helix.
Considering that nvim-tree-sitter require the queries to be in their repository, and Zed includes the supported parsers in their own repository https://github.com/zed-industries/zed/tree/c57cc35b030407a6e4e9f9cd990135dcdcf36e9e/crates/languages. What highlighting query format should we keep here in the repository?
tree-sitter highlight --check file.cr inform some errors:
Non-standard highlight captures detected:
* function.method.builtin
* keyword.return
* keyword.conditional
* keyword.exception
* string.regex
* type.definition
* function.method
* function.call

Probably because it checks the same captures used by Zed, since the tree-sitter guy is one of the founders.
So I think we should keep the Zed format here in the repository and move the nvim format to nvim-tree-sitter. What do you guys think?

Trying to get this working with helix, running into an issue where helix segfaults and I have no idea why. Created an upstream issue to hopefully get feedback to try and figure this out. Could be something wrong with this repo, or with how helix is building it.
https://github.com/helix-editor/helix/issues/12257

So I think we should keep the Zed format here in the repository and move the nvim format to nvim-tree-sitter. What do you guys think?

Could we keep all of them here and dup to their respective repos when changes are made? queries/zed/..., queries/helix/..., etc.

This is a neat idea: https://github.com/tree-sitter/tree-sitter-scala/blob/68773dfed7fee05b3549aaf52ebf8be7c7fe7d5f/.github/workflows/ci.yml#L126-L145

Now that I'm on a computer I checked the captures used by nvim and tree-sitter, they are the same 🎉
The tree-sitter ones:
https://github.com/nvim-treesitter/nvim-treesitter/blob/master/CONTRIBUTING.md
Nvim ones:
https://neovim.io/doc/user/treesitter.html#treesitter-highlight-groups
Zed ones:
https://zed.dev/docs/extensions/languages#syntax-highlighting
All the same.
TL;DR;
We can put all queries here!

I'm still not sure we can re-use the same ones for every editor, at least not for helix, due to difference in query precedence rules:
https://docs.helix-editor.com/guides/adding_languages.html#queries

In Helix, the first matching query takes precedence when evaluating queries, which is different from other editors such as Neovim where the last matching query supersedes the ones before it. See this issue for an example.

I think we can consider both the same until some Helix user complain 😅 , but try to avoid multiple queries match the same node seems like a good practice to me.

On going https://github.com/nvim-treesitter/nvim-treesitter/pull/7481

I'm still not sure we can re-use the same ones for every editor, at least not for helix, due to difference in query precedence rules:

For us, the value of tracked queries is as a reference -- it is much easier to see what changes need to be made to adapt to breaking parser changes if you've made similar changes already. (This also means that these queries are worthless if they're not covered by CI that is run on every PR.) Things like capture names or predicates are trivial customizations on top of that, so I wouldn't worry too much about those.
Regarding precedence: Helix is the odd man out; the official precedence is "last match wins", i.e., Neovim rules. (They have a PR up to switch, but it's going slooooowly.)

@clason I assume you're referring to the highlight support in tree-sitter test? https://tree-sitter.github.io/tree-sitter/syntax-highlighting#unit-testing
@hugopl I think this is the official list of tree-sitter highlight groups: https://github.com/tree-sitter/tree-sitter/blob/master/highlight/src/lib.rs#L28

I assume you're referring to the highlight support in tree-sitter test?

Not really; I don't use the upstream crates. I just mean it's important to keep the queries compatible with the grammar. (I use https://github.com/nvim-treesitter/nvim-treesitter/blob/main/scripts/check-queries.lua for that, of course, but as long as you can check that the queries don't refer to removed or renamed nodes, any way will do.)

The linter in nvim-treesitter already found some issues that I fixed in the PR but not here, I'll do that once I have time.


Update French translation

Here we have col description which contain a space unlike all other label classnames, maybe the build fail is due to that ?

#CancelAndHaveAGoodDay

@xerta555 I have opened a pull request (#369) that adds translation support for the import/export (backup/restore) modal functionality. Feel free to add the French translations once it is merged. Thanks for your contributions :)

@xerta555 I have opened a pull request (#369) that adds translation support for the import/export (backup/restore) modal functionality. Feel free to add the French translations once it is merged. Thanks for your contributions :)

My pr: https://github.com/spicetify/spicetify-marketplace/pull/370


tsp, support grouping of path+query+header+body properties in model, reflect it in convenience API signature

Source MVAD
apiview java: https://apiview.dev/Assemblies/Conversation/66bfc5cd130a4eb4a4e05b9d192c7e2b#3b0999fbf4bb4752af7fb60c958b129e
apiview .net: https://apiview.dev/Assemblies/Conversation/0eacd79245e14c6fa88d5991aac2e02f#51d7c0923d6a41cbbe89658a521cca38
E.g.
If tsp define (model or alias) as a group
Model ApiOptions {
  @path id: string;
  @query format: string;
  name: string;
  data: bytes;
}

op post(options: ApiOptions): OkResponse;

And Java API be
void postWithResponse(String id, String format, BinaryData options);

void post(ApiOptions options);

Enhanced case would involve multiple groups, e.g. post(Option1, Option2, Option3).

Before we get to TOM, wondering whether we'd like to support a @optionBag(name = "ApiOptions") decorator in TCGC, for the simple case that all path/query/header parameters and all body properties would be grouped to a model ApiOptions.

PoC https://github.com/Azure/autorest.java/pull/2140

And even simple template could get things complicated...
  GetFileFromBatchNode is RpcOperation<
    GetFileFromBatchNodeOptions,
    BatchResponseHeaders &
      FileResponse & {
        @body
        @doc("A response containing the file content.")
        file: bytes;
      },
    {},
    BatchError
  >;

op RpcOperation<
  TParams extends TypeSpec.Reflection.Model,
  TResponse extends TypeSpec.Reflection.Model,
  Traits extends TypeSpec.Reflection.Model = {},
  TErrorResponse = Azure.Core.Foundations.ErrorResponse,
  TraitContexts extends TraitContext = TraitContext.Undefined
> is Foundations.Operation<
  TParams & TraitProperties<Traits, TraitLocation.Parameters>,
  TResponse & TraitProperties<Traits, TraitLocation.Response, TraitContexts>,
  Traits,
  TErrorResponse
>;

https://github.com/Azure/typespec-azure/issues/2724

https://github.com/Azure/typespec-azure/issues/831


[FEATURE]: [Worldline] Use connector_response_reference_id as reference to merchant

:memo: Feature Description

Reference id are used to map transactions in the connector’s dashboard.
Hyperswitch manages several reference ids, such as payment_id, attempt_id, and connector_transaction_id for a single transaction.
However, merchants may encounter uncertainty when determining which ID to utilize in the connector dashboard to identify the payment.

:hammer: Possible Implementation

When we receive a response from the connector for the payment, we deserialize it and populate the response field in RouterData.
For the TransactionResponse type, we must fill the connector_response_reference_id with a corresponding reference id for the merchant to identify the transaction.
One might need to have exposure to api docs of the connector for which it is being implemented to decide what to fill in connector_response_reference_id.
You can check this PR for further reference https://github.com/juspay/hyperswitch/pull/1735

🔖 Note: All the changes needed should be contained within hyperswitch/crates/router/src/connector/
:package: Have you spent some time checking if this feature request has been raised before?

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

:package: Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

:sparkles: Are you willing to submit a PR?

@AkshayaFoiger
I would be delighted to work on this issue
so kindly assign me this
Thank You : )

Hey @Kota-Karthik , sure! I've assigned this to you.
In case you have any queries, you can ask them on this issue thread, or on our discord server, or on slack whichever you are comfortable with

Hey! @AkshayaFoiger  if @Kota-Karthik is not working on it can I take it up?

Hey! @AkshayaFoiger I would like to take this one up if @Kota-Karthik is not working on it anymore.

Hello @speedhs ,
I can see that you are already working in the issue 2219.
Kindly raise the PR for that issue, then later you can take this up, if it will be open.

Hello @speedhs  , we didn't hear back from you, so we are unassigning you from this issue. If you are interested, we will assign you with other issues, if the current one is taken.

@deepanshu-iiitu Can you assign this issue to me?

Hey @Harshwardhan9431 , sure! I've assigned this to you.
In case you have any queries, you can ask them on this issue thread, or on our discord server, or on slack whichever you are comfortable with


Updated 2 workflows and tutorials to use the latest version of Galaxy-ML

Thank you for contributing to the GTN! :yellow_heart:
Please check that:

[ ] Your pull request has a good title, e.g.

"Fix typo in ansible-galaxy tutorial"
"Add new transcriptomics tutorial covering a new sequencing technology"

[ ] Please replace this entire pull request message with a description of your changes, it will help us review your pull request faster
[ ] Check that your images are allowed to be re-hosted by the GTN

Once these are done, you're ready to go!

Consider sustainable computing with Draft Mode :green_heart:
Is your pull request finished? 100% ready to be merged? Or do you want some time to work on it first (which we encourage! It's great to have visibility)
If not, then please consider creating this pull request as a draft.
Regular pull requests will trigger automated test runs when they are created
and every time you update them.
As a small contribution to sustainable computing, we skip the most energy intensive of these tests for pull requests marked as drafts. Once your pull request is ready to go, then you can click a button saying "Ready for Review" and the tests will be run!

Hi @anuprulez,
I updated 2 workflows/tutorials and created this PR. When you get a chance, could you please take a look?
Thanks!

@kxk302 are you up to generating a workflow test for this one?

@kxk302 are you up to generating a workflow test for this one?

@bgruening Sure! Is there a code snippet I could refer to? Thanks

This here might help: https://training.galaxyproject.org/training-material/faqs/gtn/gtn_workflow_testing.html

Seems all the checks have passed. Ready for review!


4.0.0.RC1 Grails Incompatible - Server returned 403: CSRF token doesn't match.

The plugin does not work with Grails 4.0.0.RC1
Executing a script results in the following message:
Server returned 403: CSRF token doesn't match. Please refresh the page.
Setup:
sdk use grails 4.0.0.RC1
grails create-app demo
cd demo
# add runtime 'org.grails.plugins:grails-console:2.1.1' to build.gradle
grails run-app

visit: http://localhost:8080/console/index#new
and hit the play button/execute a script.

I just tested and it seems to be working for me. Does it work for you after a page refresh?

Refresh has no impact.  Interestingly, it will work in an incognito window.
Strange, I just tested 3.3.9 and it doesn't work either so I guess this isn't related to Grails 4.
I am using Chrome 74.0.3729.131 on Mac OS Mojave 10.14.4
This used to work fine.  Not sure what is going on here. Any ideas?

Safari works fine.

Any ideas on what might be causing the issue in Chrome?  I tried disabling all my extensions.  Still doesn't work.  Weird that it works in incognito mode.

Strange. It works for me in Chrome (same Chrome and Mac version). Does it work if you set
grails.plugin.console.csrfProtection.enabled = false

yes, and like I said, the csrfProtection works fine in an incognito window.  Anything I can reveal by inspecting the headers?

The execute call should send a X-CSRFToken header that matches what's stored in the session.
https://github.com/sheehan/grails-console/blob/master/grails3/plugin/grails-app/controllers/org/grails/plugins/console/TokenInterceptor.groovy#L14


chore(postgres): cleanly handle unconnectable database

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#421
#420
#419  👈
#418
#417
#416
#415
#414
#413
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


Add CI status badges

Coverage remained the same at 82.143% when pulling d0879e4724cd8f807637ad671ab972ffdeec5ae0 on status-badges into 6fac6ea9085da8a1964f8817f34756d224b70fd5 on master.


Proposal: Dockerfile with Tag variable at build

Every Image comes up with different tag name with their respective versions or with updated Container image.So can we have Tag variable  to a Dockerfile to get the Exact image instead getting all the images associated.
Example
Dockerfile
FROM dockerfile/ubuntu:<VERSION>

docker build  --tag--version 12.04

closing as dup of https://github.com/docker/docker/issues/9641

see also

@jfrazelle perhaps I'm misunderstanding, but I don't see a way in those other issues to do something like
FROM ubuntu:<version>

and dynamically provide the version. Perhaps there is a different way to accomplish that?
We have a build process that builds up multiple docker containers and our build is fairly slow because of that. They all share the same codebase, so if we could build one base image and tag it with the current build version, and then have the other dockerfiles reference that specific version during their build, that would fantastic.
base image
 FROM debian
 # prepare base image

docker build -t foo/awesomeapp:454
webapp dockerfile
FROM foo/awesomeapp:${BUILD_NUMBER}
# do webapp stuff

task runner
FROM foo/awesomeapp:${BUILD_NUMBER}
# prepare task runner

docker build -t foo/webapp:454 -e BUILD_NUMBER=454
docker build -t foo/task_runner:454 -e BUILD_NUMBER=454

@grossadamm Did you ever figure out a way to do this?  The fact that the issue was closed makes me think I'm missing something but I also don't see the other issues as solving this problem.
I'm also wanting to do something like:
FROM ubuntu:${RELEASE}

And can only come up with hacky solutions such as a wrapper to replace the value in a Dockerfile before build and such.

Nope, I never found a way to do it built in. What I ended up doing was using sed in our build process to build the dockerfile with the right tag.
Horrible blog post that details the process http://synergydatasystems.github.io/posts/build-docker-containers-from-custom-tag-dynamically/

@grossadamm Thanks for following up on this!  I actually ended up doing something very similar to this (at least for now).  I am very new to docker, but I'm having trouble balancing the DRY principle with what I see as the Docker-esque philosophy on Dockerfiles.

Also see https://github.com/docker/docker/issues/10899 and https://github.com/docker/docker/issues/9731

I see it kind of tricky because it impact the basic build principle of dockfile;
meaning:- every step of dockerfile execute in its own container and get save as a temp image for next step; with this thought; for first line to execute it has be static.
Unless we update FROM to perform extra step something like: if imagename[:tag] not found and appears to be more like variable then interpret it as variable
BTW; I am one add to people in queue waiting or this feature, as this can make CI look clean and not hacky
So, are there plans to enable this feature?

@rajivbhati12 please don't leave the same comment on multiple closed issues

This syntax is working for me:
ARG BASE_TAG

FROM something:${BASE_TAG}

And then:
$ docker build --build-arg BASE_TAG=some-tag .

Yes, correct, this is now possible
I think the docs need a slight correction, as it's possible to use an ARG in FROM (not ENV). See https://github.com/moby/moby/issues/18119, and the PR that implemented it https://github.com/moby/moby/pull/31352 (documentation: https://docs.docker.com/engine/reference/builder/#parser-directives
I would recommend setting a default, so that build doesn't fail if you don't specify a tag
For example
ARG TAG=latest
FROM ubuntu:${TAG}

I think this works as well;
ARG TAG
FROM ubuntu:${TAG:-latest}

Thanks guys I was really looking for this feature
Not sure if we have plans to support something like:
ARG TAG
FROM ${TAG}

@rajivbhati12 should work already;
docker build --build-arg IMAGE=busybox -<<'EOF'
ARG IMAGE
FROM ${IMAGE}
RUN busybox | head -1
EOF

Sending build context to Docker daemon  2.048kB
Step 1/3 : ARG IMAGE
Step 2/3 : FROM ${IMAGE}
 ---> f6e427c148a7
Step 3/3 : RUN busybox | head -1
 ---> Running in 7e8fec11c34b
BusyBox v1.28.1 (2018-02-20 17:56:45 UTC) multi-call binary.
Removing intermediate container 7e8fec11c34b
 ---> 3675d0d414ac
Successfully built 3675d0d414ac

:100: :)


MessageDeleted Event Add Source Channel

MessageDeleted(ulong, Optional, ITextChannel)
For there to be a way to get information about the message deleted (like where it was deleted) when the message is not in the cache.
Actually, was planning on searching the channel for the message but then realized it triggered because it was deleted.
But it might have some other uses so keeping this here.

#395 Does this, but hasn't been pulled yet.


Build MXNET to use NaiveEngine by default

How can I build mxnet to use NaiveEngine by default?

@mxnet-label-bot add [Question]

@cyrusbehr Yes if you can build from source. Change below line to the engine you want:
https://github.com/apache/incubator-mxnet/blob/9d7fc7cbee09de2694022995d0601cb4316e4988/src/engine/engine.cc#L35

Excellent thank you


feat: [PL-38818] Added Read Support for all connectors by name

Describe your changes
Comment Triggers

PR Check triggers

Build: trigger build
Sub Category Field Check: trigger subcategoryfieldcheck

trigger build

Please check subcategory field and build failures. Also add either id or name mandatory in the schema definition. Also add changelog.

trigger build


Sleep API 2.0: Implement pokemon box with limit

Details
Decide a limit, 50 pokemon saved per account?

50 feels small to me. I'd expect that most users will save ten or fewer, some users will try to save their entire in-game box, and a small handful of power users will want to also include hypothetical upgrades of their mons as well.
My expectation is that few enough people will want to do their entire in-game box that it'll be fine for data storage. Maybe implement a limit of 1k so that we can notice anyone doing anything really weird, and monitor how it goes for a while? I'd prefer no limit if it's feasible.

50 feels small to me. I'd expect that most users will save ten or fewer, some users will try to save their entire in-game box, and a small handful of power users will want to also include hypothetical upgrades of their mons as well.
My expectation is that few enough people will want to do their entire in-game box that it'll be fine for data storage. Maybe implement a limit of 1k so that we can notice anyone doing anything really weird, and monitor how it goes for a while? I'd prefer no limit if it's feasible.

Currently there are 15 teams stored taking up ~45kb, so 3kb per team. There are 100 pokemon stored for ~96 kb, so about 1kb per pokemon. I believe we have a total of 10GB disk for the database atm. Let's say we dedicate like 4GB to pokemon table. That's like 4 million pokemon saved. I think we should be good, even if we were to add more columns. To reach 4 million pokemon we probably have several 100k users and at that point we have bigger fish to fry.
It's a little risky though to not have any limit, so probably a strange catch limit like you suggested at the very least.
I'll update this ticket to not mention the limit though, we still need to implement a pokebox view.

1kb per Pokémon feels like a lot. I'm guessing that's because it's JSON, and everything is being stored as strings. If we switch to something like protobuf, that should drop significantly. Do you want me to investigate that?

1kb per Pokémon feels like a lot. I'm guessing that's because it's JSON, and everything is being stored as strings. If we switch to something like protobuf, that should drop significantly. Do you want me to investigate that?

Taking the discussion offline


FastConcurrentTLru's Constructor timetolive is not precise on liunx mac platform

FastConcurrentTLru use TLruLongTicksPolicy. this policy discard item by
[MethodImpl(MethodImplOptions.AggressiveInlining)]
public bool ShouldDiscard(LongTickCountLruItem<K, V> item)
{
    if (Stopwatch.GetTimestamp() - item.TickCount > this.timeToLive)
    {
        return true;
    }

    return false;
}

but the Stopwatch.GetTimestamp() has platform different result. see
https://stackoverflow.com/questions/67205283/stopwatch-gettimestamp-produced-different-results-on-linux-vs-windows

on macos platform the scale is 1000x

Thanks for reporting this, I'll take a look.

thanks

I don't have a Mac to repro this, please could you run the program below and give the results:
class Program
{
    static void Main()
    {
        Console.WriteLine(Stopwatch.Frequency);
        Console.WriteLine(TimeSpan.TicksPerSecond);

        Console.WriteLine(Stopwatch.GetTimestamp());
        Thread.Sleep(TimeSpan.FromSeconds(1));
        Console.WriteLine(Stopwatch.GetTimestamp());
    }
}

I will then use to sanity check my fix. Thanks!

the result is:
1000000000
10000000
10000000
1026619477472300
1026620481623899

this my macbook pro info:

Thanks for the info - a fix is now checked in.