make the unreal settings more readable

Changelog Description
This PR is to make the unreal settings more readable
Resolve #149
Additional review information
Please build and install addon before testing
Testing notes:

Launch Unreal
Load and Publish(Load anything and publish render)

Missing conversion of settings overrides.

Should we merge this or there is something else we need to do to improve the settings to be more readable>

@moonyuet Im all in for merging!


Create automation for deploy pipeline

todo

I'll start looking at this now. Will keep it simple at first with a manual invocation.


(.AuthService .isLoggedIn)-.AUTH_PROVIDERS range is broken

In the README we discuss the query: (.AuthService .isLoggedIn)-.AUTH_PROVIDERS on examples/AuthService.ts. It appears this query has regressed.
My hunch is that this originates with the range change in which we search for child selectors to resolve a range (to ensure we never look backwards). My hunch is that we need to restore the old behavior if a child end selector is missing. That said, I'll need to look more closely at the code.

This is a false alarm - see 56ae625 where the test passes. Closing.


"discovery" implies that Swarm supports service discovery

Just as a heads up, I have had several people get confused about what "discovery" is in Swarm. They mistakenly assume it is a way for containers to do service discovery (e.g. how does my web server find my database).
I guess it's service discovery for Swarm nodes, but I wonder if there's a better word that could be used so people don't mistake it with a service discovery system they can use for their apps.
For reference, terms used by existing pieces of software:

Mesos seem to use slave "registration" around their docs.
Riak uses the word "membership" in that context.
etcd also use "discovery"

Valid concern (I think). Given that join is used, "membership" seems like a close match to that for terminology. It does sound a bit awkward though (membership service?)

I agree, I think we should say Node Discovery service

/cc @moxiegirl

@vieux adding "node" to discovery service isn't really going to help.  Because the notion of service discovery is so dominant.
Can someone define for me "discovery backend" I had this question when I started working the getting started docs and thought it meant service discovery. Is that the same or different than the topic here?
Don't "nodes join a swarm" that seems a natural way to say it.  Swarm service...or 'join a colony" meaning a particular swarm.  I definitely think we should find a way to remove the overloaded discovery --- register with a swarm....


Rewrite the validate_hif.py to use the new schema

Please let me know if I deleted something I shouldn't have.

@nwlandry FYI

@nwlandry FYI

Would you like me to review?

Yes, please take a look a the scripts/hif.py. It had a lot of duplication and many things seemed old.
Thanks!

Hi @colltoaction --- my only suggested change is to customize the output to improve the clarity of the error messages. Perhaps one way to handle this is to utilize the "messages" field. I think that @pszufe is best to review this PR, since he had an idea of what the output should be.

Sounds good, thanks for taking a look. @pszufe I'm merging and we can continue this work on the main branch.


Correction for raw affiliation Lakehead University, Thunder Bay, Ontario, USA

Correction needed for raw affiliation Lakehead University, Thunder Bay, Ontario, USA
raw_affiliation_name: Lakehead University, Thunder Bay, Ontario, USA
new_rors: 023p7mg82
previous_rors:
works_examples: W2890109985
contact: 03bcc30c4a2e4a1c792d2ce119e64c69:5c058ac7 @ ourresearch.org

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


Incorrect result returned by a NLJ query with filter (SQLancer-NoREC)

Describe the bug
The following query should return an empty set (verified in postgres and DuckDB), but in DataFusion it returns 1 row as output.
Reproducer in datafusion-cli
/*DML*/CREATE TABLE t1(v0 BIGINT, v1 STRING);
/*DML*/CREATE TABLE t2(v0 BOOLEAN, v1 BIGINT);
/*DML*/CREATE TABLE t3(v0 STRING, v1 BOOLEAN);

/*DML*/INSERT INTO t1(v1) VALUES ('1381560824');
/*DML*/INSERT INTO t2(v1) VALUES<PHONE_NUMBER>);
/*DML*/INSERT INTO t2(v0) VALUES (true);
/*DML*/INSERT INTO t3(v1) VALUES (false);

> SELECT * FROM t1 CROSS JOIN t2 LEFT JOIN t3 ON ((t1.v0) == (t2.v1)) WHERE ((t3.v1) < (t2.v0));
+----+------------+------+----+----+-------+
| v0 | v1         | v0   | v1 | v0 | v1    |
+----+------------+------+----+----+-------+
|    |<PHONE_NUMBER> | true |    |    | false |
+----+------------+------+----+----+-------+

More details: If this query is run without filter, the result will be
> SELECT * FROM t1 CROSS JOIN t2 LEFT JOIN t3 ON ((t1.v0) == (t2.v1));
+----+------------+------+------------+----+----+
| v0 | v1         | v0   | v1         | v0 | v1 |
+----+------------+------+------------+----+----+
|    |<PHONE_NUMBER> |      |<PHONE_NUMBER> |    |    |
|    |<PHONE_NUMBER> | true |            |    |    |
+----+------------+------+------------+----+----+

the removed filter is ((t3.v1) < (t2.v0)), notice t3.v1 is all NULL here, so I guess somewhere during execution, NULL < NULL is incorrectly evaluated to true
To Reproduce
No response
Expected behavior
No response
Additional context
Found by SQLancer(https://github.com/apache/datafusion/issues/11030)

👍


Configuration to control if model is centered at root origin or bounding box origin

Currently, we stage a model by determining its bounding box, and centering it in the viewport based on its bounding box's origin. This behavior is a reasonable default for a lot of scenarios.
Sometimes it is desirable to center a model at the origin of the root node of its scene graph. We should add a configuration that allows the user to specify this centering strategy.
This is semi-related to #271, where @jsantell mentions that this configuration might be desirable in the AR placement context.

Sometimes it is desirable to center a model at the origin of the root node of its scene graph. We should add a configuration that allows the user to specify this centering strategy.

I'd expect this new flag to put the model's origin at the bottom center of the stage; "centering" seems like a misleading description in this case (IIUC).

This is semi-related to #271, where @jsantell mentions that this configuration might be desirable in the AR placement context.

This issue (again, IIUC) describes the current AR implementation: using the model's origin. Would this attribute be necessary to not change the AR behavior? Without this attribute, would AR models use their bounding box as the floor?

I'd expect this new flag to put the model's origin at the bottom center of the stage; "centering" seems like a misleading description in this case (IIUC).

This configuration could very well be some combination of the anchor and origin (working title for both) style properties as we have previously discussed them.
For context, I'm going to share our previously discussed definitions of anchor and origin here:

Style property
Values
Default
Definition

anchor
top bottom left right front back center
center
Changes how the model is positioned relative to its logical parent

origin
bounding-box natural
bounding-box
Changes the point that is considered the effective origin of the model

Would this attribute be necessary to not change the AR behavior? Without this attribute, would AR models use their bounding box as the floor?

I think the conservative answer here is that how a model is positioned in AR and framed inline should be determined by two distinct sets of configurations, and that configurations or defaults for one does not affect the configurations or defaults for the other.

As I'm beginning to embark on this work, it seems as though the addition of this API will be sufficient to describe user intention while remaining somewhat forward-compatible with other potential framing and controls configurations:

Attribute
Values
Default
Definition

align-model
center natural
center
Changes how the model is aligned when being framed. Natural alignment maintains the model's original position (after scaling) along the given axis. Supports one-, two- or three-value syntax, corresponding to xyz, x y and x y z axis configurations respectively (similar to how transform-origin works).

The word natural could probably use some bike-shedding.

Some potential alternatives to natural:

absolute
default


HOW TO CONNECT SQL docker database engine from another SSMS

Hello All,
I have successfully install IIS , .net 3.5, SQL containers successfully. I can able to browse IIS website. Now I am trying to access SQL docker from another sql server by giving my server IP with port number. But I could not able to establish SQL connection . Requesting to all, Could you please let me know how to access the DB from sql management studio by using the container’s IP Address along with the sql authentication credentials stored in the Dockerfile ?

Hello @ravindravv
This issue tracker is dedicated to tracking Docker bugs, proposals and pull requests to make changes to Docker's source code. This issue tracker isn't a general support forum.
Please keep in mind that opening issues to ask questions when getting started will make everyone who contributes to the project spend more time looking at issues.
The documentation found at https://docs.docker.com/ has many examples and use cases; it also explains what the various flags and options do.
This kind of question and general support questions are more adequate for the following:

the forums: https://forums.docker.com/
the docker-user mailing list https://groups.google.com/forum/#!forum/docker-user
the #docker channel on Freenode.
StackOverflow https://stackoverflow.com/questions/tagged/docker

I'm going to close this issue now because the discussion can continue even after the issue is closed. Please feel free to continue the discussion. There's no need to open a new issue on this topic.


Agent unable to connect to local collector

I have a collector running locally. And am starting a agent with command line options for my collector server/port as below. Couple of questions

Am not sure why the service discovery is enabled when I am explicitly passing the collector info
How do I met more info on the tchannel error?

$ go run ./cmd/agent/main.go -collector.host-port <IP_ADDRESS>:14267
{"level":"info","ts":1499793930.997451,"caller":"tchannel/bulider.go:95","msg":"Enabling service discovery","service":"jaeger-collector"}
{"level":"info","ts":1499793930.9975188,"caller":"peerlistmgr/peer_list_mgr.go:117","msg":"Registering active peer","peer":"<IP_ADDRESS>:14267"}
{"level":"info","ts":1499793930.9978952,"caller":"agent/main.go:52","msg":"Starting agent"}
{"level":"info","ts":1499793931.998254,"caller":"peerlistmgr/peer_list_mgr.go:165","msg":"Not enough connected peers","connected":0,"required":1}
{"level":"info","ts":1499793931.998381,"caller":"peerlistmgr/peer_list_mgr.go:172","msg":"Trying to connect to peer","host:port":"<IP_ADDRESS>:14267"}

Can you try running all-in-one from source? The components still communicate over the same ports.
$ go run ./cmd/standalone/main.go --span-storage.type=memory
{"level":"info","ts":1499795331.6866407,"caller":"tchannel/bulider.go:95","msg":"Enabling service discovery","service":"jaeger-collector"}
{"level":"info","ts":1499795331.6867087,"caller":"peerlistmgr/peer_list_mgr.go:117","msg":"Registering active peer","peer":"<IP_ADDRESS>:14267"}
{"level":"info","ts":1499795331.687073,"caller":"standalone/main.go:82","msg":"Starting agent"}
{"level":"info","ts":1499795331.68803,"caller":"standalone/main.go:117","msg":"Starting jaeger-collector TChannel server","port":14267}
{"level":"info","ts":1499795331.6881118,"caller":"standalone/main.go:124","msg":"Starting jaeger-collector HTTP server","http-port":14268}
{"level":"info","ts":1499795331.69117,"caller":"standalone/main.go:174","msg":"Starting jaeger-query HTTP server","port":16686}
{"level":"info","ts":1499795332.689505,"caller":"peerlistmgr/peer_list_mgr.go:165","msg":"Not enough connected peers","connected":0,"required":1}
{"level":"info","ts":1499795332.68965,"caller":"peerlistmgr/peer_list_mgr.go:172","msg":"Trying to connect to peer","host:port":"<IP_ADDRESS>:14267"}
{"level":"info","ts":1499795332.6903925,"caller":"peerlistmgr/peer_list_mgr.go:177","msg":"Connected to peer","host:port":"[::]:14267"}

There is an error generated (https://github.com/uber/jaeger/blob/master/pkg/discovery/peerlistmgr/peer_list_mgr.go#L174) that is not being logged anywhere.
Logging this error message would be beneficial to diagnosing connection issues. We are currently running into some sort of connection issue where the connection is taking longer than connCheckTimeout (250ms).
Would a PR with our changes to this part of code be beneficial, if so is there a place to look at for PR guidelines?

@jnewmano a PR would be great - see https://github.com/uber/jaeger/blob/master/CONTRIBUTING.md

I think this has been resolved by the attached PR #356, closing.


Add Plugin links to the README.md

Reason for change:

The plugin ecosystem, while awesome, isn't easy to find

Change:

Add a section to the README.md file to show some example plugins.
Also add a link to the beautiful web documentation site, which assumedly is probably most up-to-date going forward.

Great, thanks!


Electra Finetune on HLGD + Time, Model weights usage

Do we need to add some layers(768x1 and 2x2) and train, as mentioned in the paper? Or we can use classifier_electra_hlpc_1_f1_0.7442.bin directly to classify?
If we can use it directly, then can you please send a sample code?
Thanks

Hello Badrinath,
Good question! I've added a script that gives an example of how to use the model: model_classifier.py.
Note: I've also renamed the classifier model file to cls_elec_base_hlgd_0.74f1.bin, but it is the same that was uploaded previously.


Mainnet & Testnet Wei (Calculation) Issue

Hello you will see same contracts as Sepolia and Mainnet:
They are not working in the same process. Because the explorer data on the mainnet side is not calculated as wei. But in testnet (Sepolia) is working correctly as well. We are experiencing many problems because of this. Can this process be controlled and corrected?
https://sepolia.scrollscan.com/address/0xd73D676AA7935FEc7ADA64AE910237C735E13a64#readContract < sepolia
https://scrollscan.com/address/0xdca8d621e17f0f820b403bc7f9edab8fc771a8d6#readContract < mainnet

Mainnet1

Mainnet2

Sepolia1

Sepolia2

Sepolia3
So we are getting issue about that. Could you kindly help us with that?

The ERC20 standard allows tokens to specify their decimals. While it is usually set as 18 (i.e. matching the wei-to-ETH granularity), there are well-known exceptions like mainnet USDT, which uses 6. To make your code robust, you should consider decimals in your contract's logic.


Add DigitalOcean service discovery

See #4166
Signed-off-by: Julien Pivotto<EMAIL_ADDRESS>

See #4166

Added tests.

I still think that we should default to v6 if it's configured. Turning on v6 is still optional in DO, which means users that intentionally turn it on expect to be using it.

In SD the only time we don't pick a single value for the address is when that's simply impossible, namely when there's no source field that's always present. Otherwise we're introducing business logic, and that's what relabelling is for.
Public v4 is the only thing we know will be present, thus it is the right value to use.
In addition that v6 is enabled on the scrape target does not mean that it will work for Prometheus, such as if it's running inside docker or the machine Prometheus is running on didn't have v6 enabled.

:+1:

ipv6 not only needs to be enabled on the DO level but also requires manual changes on VM ; so it is also less likely to work

@roidelapluie Manual changes are only required if v6 is enabled after droplet creation. If you enable v6 at creation time, everything just works.
Basically, turning on v6 in DO is something that doesn't happen by accident.
Configuration problems outside of this are out of scope.  If there is v6 on, we use it by default. This is how it's supposed to be.

Why prefer v4:

all droplets have one by default
it serves as a unique identifier
it will work for most users

Why prefer private v4:

to not use public address to route promethes traffic - which is something which would NOT change the way exporters work so that is not really relevant

Why prefer v6:

best practice
people who enable it probably know what they do and want to use it

issues with v6:

Can be added when a droplet is created, breaking its unique id and all its metrics
Prometheus in DO would have a v4 but maybe not a v6
It would add small logic in the SD that might surprise/break users

middle ground:

default to v4 for this provider because that is how the cloud provider is designed (always a v4 is easy to remember and explain)
not making this a strict rule for all service discoveries
document how to use v6 or private addresses (like we do for marathon: provide an exemple prometheus.yml)

people who enable it probably know what they do and want to use it

I think that's a bit of an assumption. I have v6 enabled on some of my droplets but not others, and considering I don't have any DNS pointing at the v6 address I'd not expect anything to try to use it.

@roidelapluie Thanks for documenting the reasoning. I think this is a good enough compromise.


Area calculation differs between  OpenLayers and PostGIS

In OpenLayers I draw a 5 meter * 5 meter rectangle.
If I calculate the area of the geometry in OpenLayers I get 25 m^2 but in PostGIS I'm getting 25.04.
OpenLayer:
code snippet drawing the rectangle:
const factor = getPointResolution('EPSG:3857', 1, cursorCoordinate);
const forceHeight = 5 / factor;

Measuring:
const factor = getPointResolution('EPSG:3857', 1, firstCoordinate);
const squareMeter = polygon.getArea() * Math.pow(factor, 2);

//squareMeter = 25.00

PostGIS:
https://postgis.net/docs/ST_Area.html
SELECT
	ST_Area(ST_Transform(postgis_area_geometry, 4326)::geography, true)
FROM
	geometry_table
WHERE
	id=xxx;

//Result use_spheroid=true = 25.04
//Result use_spheroid=false = 24.98

I'm not sure if do something wrong in openlayers or in postgis. I'm not even able to understand what Point-Resolution means.
QGIS shows me 25.04 too..
Geometry in GeoJson 3857:
{"type":"Polygon","crs":{"type":"name","properties":{"name":"EPSG:3857"}},"coordinates":[[[856385.579794264,6863294.22127374],[856393.675958321,6863295.424757587],[856394.879442168,6863287.32859353],[856386.783278111,6863286.125109684],[856385.579794264,6863294.22127374]]]}

The OpenLayers equivalent of the use_spheroid=true method is the spherical getArea() method https://openlayers.org/en/latest/apidoc/module-ol_sphere.html#.getArea which for your polygon returns 24.944.  The use_spheroid=false method uses a more accurate ellipsoidal calculation which has no equivalent in OpenLayers (see #13572).  Any of these global calculation methods applied to small distances (less than 10 meters) will incur considerable rounding errors relative to the final result.  For polygons less than 1km across when using spherical mercator projection where the point resolution is easily calculated as cos(latitude) - something at latitude 60 appears twice the size of the same object at the equator (see https://openlayers.org/en/latest/examples/draw-and-modify-geodesic.html) - simply applying the point resolution to the planar area is likely to be as or more accurate than a more complex calculation.  However when polygon dimensions exceed 1km the variation in point resolution across the polygon would become significant and this simple method would become inaccurate.

Fortunately, none of my geometries should exceed 1 km, so I can go with your suggestion, thank you :)


Compare patches with different spatial resolution

Would it also be possible to compare 2 images with different spatial resolution?
e.g.
img_1 = 256 X 270 X 3
img_2 = 180 X 245 X 3
One hacky way of doing it to resize both images to same size and compare.
In the end, it network gives a feature vector of different length if image res are different for 2 images. But just curious in case you have tried out something.

Oh, no. They need to be resized. But are the images of the same thing in the first place (have some spatial correspondence)?

Yeah same but different resolution. So on certain websites you see thumbnail corresponding to articles and when you open the article you see high res image.

Great, resizing seems reasonable then. Keep in mind that many built-in resizing functions do not properly antialias: https://twitter.com/jaakkolehtinen/status/1258102168176951299


Getting 'Request content too large' after modifying content_max_length in api_sync_func

I am trying to finish up a container for our AI4E requirements and I am getting the following:
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">
<title>413 Request Entity Too Large</title>

Request Entity Too Large

{'message': 'Request content too large (441078). Must be smaller than: 10000'}

I have tried to set content_max_length in this call
@ai4e_service.api_sync_func(
    api_path="/detect/cardinalis_cardinalis",
    methods=["POST"],
    request_processing_function=process_request_data,
    maximum_concurrent_requests=5,
    content_types=ACCEPTED_CONTENT_TYPES,
    content_max_length=MAX_CONTENT_LENGTH,
    trace_name="post:detect_cardinalis_cardinalis",
)
def detect(*args, **kwargs):
   ...

I also tried modifying this for the Flask app itself via app.config['MAX_CONTENT_LENGTH'] = MAX_CONTENT_LENGTH. MAX_CONTENT_LENGTH = 1 * 1024 * 1024. I see the following code: https://github.com/microsoft/AIforEarth-API-Platform/blob/7987f350c7d74f2bad254b1b3fbac8ecce10804f/APIs/1.0/base-py/ai4e_service.py#L103-L105 which makes me think this should work. Any thoughts?
My code is available here: https://github.com/chiroptical/ai4e-example/tree/opensoundscape_0_4_4

The code in the master branch doesn't have this issue but I need python 3.7+ so it was easier to base the container on Ubuntu 20. If you need anything extra let me know.

I had a bug in one of my deployment scripts which cause this. Closing.


Fix: Hide paycheck report with error

Details

Filter out paycheck report on LHN
Fixed Issues

$ https://github.com/Expensify/App/issues/45104
PROPOSAL: https://github.com/Expensify/App/issues/45104#issuecomment-2218408297
Tests

Prerequisite: There are some paycheck report with error other than failed receipt or bill in the account
Onyx.merge("report_3597331137566440", {
        type: "paycheck",
        errorFields: {
            "notFound": {
                "1721083106781482": "Report not found",
                "1721140343800206": "Report not found",
                "1721149017870362": "Report not found"
            }
        },
})

See LHN
Verify that reports with type as paycheck or bill are hidden

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

Prerequisite: There are some report with type paycheck or bill in the account

Log into OldDot
Export some payroll and bill reports that you have access to to a CSV
Make sure those reports don't show up in NewDot LHN

PR Author Checklist

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

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

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

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

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

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

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

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

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

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

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

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

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

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

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/183d4a16-16dd-4cd2-821a-47e0aa390390

MacOS: Desktop

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

🚀 Deployed to staging by https://github.com/puneetlath in version: 9.0.10-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Cherry-picked to staging by https://github.com/mountiny in version: 9.0.10-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

Confirmed I'm no longer seeing the errored payroll report in my LHN.

🚀 Deployed to production by https://github.com/mountiny in version: 9.0.10-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/francoisl in version: 9.0.11-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Removed BaseSpatialOperations.truncate_params.

Unused since 32969c3931dea8488ee4ad2849eab43e31e5542e.

buildbot, test on oracle.

buildbot, test on oracle.

buildbot, test on oracle.

buildbot, test on oracle.


services/ticker: handle invalid TOML urls gracefully

What
Ensure the Ticker can gracefully handle invalid TOML urls.
Why
This PR fixes #2326. Due to invalid TOML URLs, the asset ingestion process was crashing on mainnet. I've found references to similar segfaults with net/http using invalid URLs. Maybe this means we're using an old version of net/http?
In any case, this fix should prevent this issue from happening again.

Done! @leighmcculloch should be good for another look :)


[Discussion] Deployment of .NET Core 1.0.12, 1.1.9, 2.0.9, 2.1.2 to Azure App Service

Discussion issue for https://github.com/Azure/app-service-announcements/issues/121

There isn't any information in the announcement post about timelines. When is this going to be deployed?

I would like to know as well. Even a rough idea like "hopefully within a week barring unforeseen circumstances" would be nice.

Mathew is right, the info is hidden in release note: https://github.com/dotnet/core/blob/master/release-notes/2.1/2.1.2.md#azure-appservices

The deployment is now complete.


gccgo: incorrect export data for duplicate interface

What version of Go are you using (go version)?

$ go version
go version devel +7d13c1b887 Fri Mar 1 13:46:39 2019 -0500 linux/amd64
$ gccgo --version
gccgo (GCC) 9.0.1 20190304 (experimental)

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
linux/amd64
go env Output
$ go env
GOARCH="amd64"
...
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"

What did you do?
Create three packages, "usetm", "tm" and "tm/another/tm".
First package $GOPATH/src/tm/tm.go:
package tm

import (
	"tm/another/tm"
)

type LocalM interface {
	tm.LocalM
}

func NewM(x int) LocalM {
	return tm.NewM(x)
}

Second package $GOPATH/src/tm/another/tm/tm.go:
package tm

type LocalM interface {
	Foo(x int) int
}

type CM struct {
	q int
}

func (cm *CM) Foo(x int) int {
	return x
}

func NewM(x int) LocalM {
	return &CM{q: x}
}

Third package $GOPATH/src/usetm/usetm.go:

package usetm

import (
	"tm"
)

func Blah() {
	tm.NewM(3)
}

Then cd to $GOPATH/usetm and issue a build with gccgo:
$ go build -compiler gccgo .
What did you expect to see?
Successful build
What did you see instead?
Build fails with:
$ go build -compiler gccgo .
# usetm
./usetm.go:4:4: error: inherited interface loop
    4 |  "tm"
      |    ^
./usetm.go:4:4: error: invalid recursive interface
$

Looking at the export data for GOPATH/src/tm:
Excerpt from $GOPATH/pkg/gccgo_linux_amd64/libtm.a
v3;
package tm
pkgpath tm
import tm tm/another/tm "tm/another/tm"
init ...
init_graph 0 1 0 ...
types 4 2 25 33 39
type 1 "LocalM" <type 2>
type 2 interface { ? <type 3>; }
type 3 "tm/another/tm.LocalM" <type 2>
func NewM (x <type -11>) <type 1>
checksum E8DA5660A96FEDE5919157B0A4433F5FDF8C99A4

Note the messed up types (cycle involving 2 and 3).  So the problem is really with the export data, not with the error itself (compiler it correct to report a cycle, since that's what the bogus export data says).

This problem seems to be happening fairly late in the game -- from
spending time in the debugger I don't see anything suspicious until it
comes time to emit types into the export data.
There are four types of interest:
T1  named type "LocalM" defined in /src/tm, points to T1
T2  interface type with embedded interface T3
T3  named type "LocalM" defined in /src/tm/another/tm, points to T4
T4  interface type with method Foo
During Export::prepare_types, the traversal visits all four types,
however when it visits T4 and Export::set_type_index() is called, the
lookup into "type_refs" decides that T4 is identical to T2 (according
to the criteria set up for Type_alias_identical in export.cc), so it
returns the type index for T2 instead of adding a new entry. This
effectively makes the second LocalM an alias of the first LocalM,
which is what introduces the cycle in the type graph.
When T2 is exported, the code in Interface_type::do_export
walks the list of methods and encounters the embedded interface, it
goes ahead and emits the reference to T3, hence the problem.
My feeling at this point is that for the purposes of the exporter, it
would be better to consider these types non-identical.  I will see about
adding a flag to Types_are_identical to do a more nuanced comparison
(in particular, to compare package name/path in this case).

Interesting.  One way to look at the problem is to observe that Interface_type::do_export works on parse_methods_, which includes references to embedded types, while Interface_type::is_identical works on all_methods_, in which all methods have been pulled in from the embedded types.  If Interface_type::do_export used all_methods_, then presumably there would be no problem.  But I doubt we want to do that.  So I think you're right that we need Types::are_identical flag to compare interface types by comparing parse_methods_ rather than all_methods_, or, in other words, to compare based on embedded interface types rather than only on the final method set.


Example given in the README not working

Hi,
first of all: Thanks for that great piece of open source software, love it! I've seemingly encountered a small issue:
The example given in the README.md seems to be broken, when clicking on the link
https://abxtests.com/?test=https://www.dropbox.com/s/c00dylhi7ekqkfw/demo.yml?dl=0
it just shows a blank screen. I've checked the YAML formatting and whether all audio links are valid, no problems in that regard. Maybe you want to investigate this further :)

I guess dropbox changed their api

I guess this has not worked in over 2 years 🙈
The format of the demo YML file was incorrect. I uploaded a new file and it works now.


Integrate pager duty and sentry for console

The Chore
Trigger a pager duty alert if there are a high number of errors for critical features
Tasks

[ ] Review current set up on governance app
[ ] Implement for console

there's need to have owner, manager, or admin permissions to do it. There is already some integration done. Without these permissions we can't check nor review it.


Fix env var inheritance in docker

Env vars need to be gotten while running the docker.sh (templated out as runner.sh), because they may not be available from the executor process.

closing since this was actually a separate issue


Just some polishing

@earthlng My first time using GitHub for something other than reporting stuff. I hope I'm not messing up somehow 😓

I hope I'm not messing up somehow 😓

no, you nailed it :)
Looks good, I'll merge it right away, thanks.


Fix(curriculum): Accept valid html for meta tag.

Checklist:

[x ] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)
[x ] My pull request targets the main branch of freeCodeCamp.
[x ] I have tested these changes either locally on my machine, or GitPod.

Closes #44894

Updated regex for step 3 to accept valid html and removed required space.
Step 17 is explicit to use a closing tag - unchanged.
Step 18 appears to work for valid html - unchanged.

@sstonaker I see you made the same changes in this PR: #44984
I suggest we close this PR, and just go with the other one - both are small enough that reviewing the changes is not too difficult.

Agree, thanks for that.. looks like I might not have put this in its own branch so the other PR brought it over.

Closing as this is being addressed in #44984


webtoons.com is possible?

From what I can tell it should be possible to make a bridge for that site, although we need more information. Find below the questions to consider. Let me know if anything is unclear.
General information
Please describe what you expect from the bridge. Whenever possible provide sample links and screenshots (you can just paste them here) to express your expectations and help others understand your request. If possible, mark relevant areas in your screenshot. Use the following questions for reference:

Host URI for the bridge (i.e. https://github.com):

Which information would you like to see?

How should the information be displayed/formatted?

Which of the following parameters do you expect?

[X] Title
[X] URI (link to the original article)
[ ] Author
[ ] Timestamp
[X] Content (the content of the article)
[ ] Enclosures (pictures, videos, etc...)
[ ] Categories (categories, tags, etc...)

Options

[ ] Limit number of returned items

Default limit: 5

[ ] Load full articles

Cache articles (articles are stored in a local cache on first request): yes
Cache timeout (max = 24 hours): 24 hours

[X] Balance requests (RSS-Bridge uses cached versions to reduce bandwith usage)

Timeout (default = 5 minutes, max = 24 hours): 5 minutes

Be aware that some options might not be available for your specific request due to technical limitations!
Additional notes
Keep in mind that opening a request does not guarantee the bridge being implemented! That depends entirely on the interest and time of others to make the bridge for you.
You can also implement your own bridge (with support of the community if needed). Find more information in the RSS-Bridge Wiki developer section.

Note that the built-in RSS is limited to the first two frames so is basically useless for many comics. It seems that webtoon doesn't actually care about RSS at all and just uses it to check the feature checkbox.
It would still be nice to have an rss-bridge feed that produces a usable RSS feed.

Open RSS provides these until there's a bridge
https://openrss.org/www.webtoons.com/en/fantasy/tower-of-god/list?title_no=95


Downloading SDO data times out in "Exploring the heliosphere..." notebook

Apparently there is a new requirement to register an email address in order to export data from the JSOC.  We should prompt for an email address before trying to load the data, and skip these cells if it's left blank.

Huh, it seems to be working now without supplying credentials.   But this may be a sign of things to come.


Circle with progress doesn't work on Android

When I test the following code onAndroid,
   let arr = [];
        for (var i=0; i<10; i++) {
            arr.push(<ProcessCircle
                animated
            borderWidth={0} progress={i/10}
             color="blue"
             unfilledColor="red"
             size={50}
              direction="counter-clockwise"/>);
        }

the first two works fine, but the follow seven don't draw the progress circle as expected.

closed, because of time flying


fixing candle writer

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

It fixes the candle-writer for livebots.

What is the current behavior? (You can also link to an open issue here)

When a livebot is startet (via cli), it fetches the local dataset and tries to stich it with the exchange data. The reader ends the DB connection once it's done reading and the candle writer fails silently, since the db connection is already closed by the reader.

What is the new behavior (if this is a feature change)?

The connetion stays open

Are you using postgresql?


Allow non CS entropy sources for psa_generate_random()

Workflow of this PR

starts with some renaming of entropy kconfigs in order to have them more aligned to Mbed TLS build symbols (which simplifies understanding their meaning/effect).
improves the description of PSA kconfigs related to RNGs.
align entropy gathering between legacy mbedtls_hardware_poll() and PSA mbedtls_psa_external_get_random().
and finally it adds a new kconfig CONFIG_MBEDTLS_PSA_CRYPTO_EXTERNAL_RNG_ALLOW_NON_CS to allow non-CS entropy sources also for PSA

this is meant to be used only for tests (ex: testing a psa_get_random()` also on platforms that do not have CS sources), but it must be disabled in production. A warning is raised whenever this feature is enabled.

Improvements from this PR
Prior to this PR when Mbed TLS's PSA crypto was to be built:

MBEDTLS_PSA_CRYPTO_LEGACY_RNG was to be used on platforms that do not have an entropy driver

this required also MBEDTLS_ENTROPY_ENABLED to add a Zephyr's entropy callback from Mbed TLS's entropy module

MBEDTLS_PSA_CRYPTO_EXTERNAL_RNG was to be used on platforms that have an entropy driver.

After this PR it will be possible to enable MBEDTLS_PSA_CRYPTO_EXTERNAL_RNG + CONFIG_MBEDTLS_PSA_CRYPTO_EXTERNAL_RNG_ALLOW_NON_CSPRNG on all platforms. The latter symbols must be disabled in production of course, but for CI purposes it allows a more uniform usage of Mbed TLS's PSA crypto across the Zephyr's tessts and samples.

@valeriosetti Actually, I didn't think of this earlier, but this probably deserves some migration guide entry at least for the renamed Kconfig options.


Problem with residue selection

Hello!
I have the following error (image attached) where I select a pair of residues (in the image 576 and 566, for example) but the sequence that it shows in the bottom starts in residues 76 and 67. I have tried selecting different points in the map but the same thing happens, it seems like there is a shift from the dot plot to the sequence. I am using gepard in a Mac, and I installed it via conda.
Another thing that it would be nice to do is scroll left and right with the mousepad (and not just the keys), to more easily reach residues in a long sequence.

Thank you!

Hello, thanks for reporting this bug. We will try to reproduce it. As we are currently busy with other projects, this might take several weeks. We will keep you informed here. Best, Thomas


Release/0.5

merge release/0.5 back to main

Codecov Report

Merging #340 (e67e97a) into main (ccc9da7) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #340   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           42        43    +1
  Lines          909       999   +90
  Branches       333       384   +51
=========================================
+ Hits           909       999   +90

Impacted Files
Coverage Δ

src/autocomplete/Autocomplete.tsx
100.00% <100.00%> (ø)

src/autocomplete/ResultList.tsx
100.00% <100.00%> (ø)

src/button/Button.tsx
100.00% <100.00%> (ø)

src/common/components/CheckboxRadioBase.tsx
100.00% <100.00%> (ø)

src/common/components/Conditional.tsx
100.00% <100.00%> (ø)

src/common/components/ErrorMessage.tsx
100.00% <100.00%> (ø)

src/common/utils/clientOnly.tsx
100.00% <100.00%> (ø)

src/formInput/FormInput.tsx
100.00% <100.00%> (ø)

src/formSelect/FormSelect.tsx
100.00% <100.00%> (ø)

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


Refactoring: Services, Tools

Types of changes

Bugfix
New feature

using rich-click

Refactoring

enhance typing hints
more documentation/comments
add some utests

Breaking change (any change that would cause existing functionality to not work as expected)
Documentation Update
Other (please describe)

Description

Checklist:

[ ] Add test cases to all the changes you introduce
[x] Run inv style locally to ensure all linter checks pass
[x] Run inv test locally to ensure all test cases pass
[x] Run inv secure locally to ensure no major vulnerability is introduced
[ ] Update the documentation if necessary

Steps to Test This Pull Request

Expected behavior

Related Issue

Additional context

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

22 of 23 (95.65%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 99.187%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

vhcalc/app.py
3
4
75.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
122

Relevant Lines:
123

💛  - Coveralls


Add test case for URL

What is the purpose of the change
Add test case for URL

Codecov Report

Merging #8006 (734d6f2) into master (9f6b536) will decrease coverage by 0.22%.
The diff coverage is n/a.

:exclamation: Current head 734d6f2 differs from pull request most recent head f742f2a. Consider uploading reports for the commit f742f2a to get more accurate results

@@             Coverage Diff              @@
##             master    #8006      +/-   ##
============================================
- Coverage     60.85%   60.62%   -0.23%
+ Complexity      491      455      -36
============================================
  Files          1089     1090       +1
  Lines         43876    43905      +29
  Branches       6413     6389      -24
============================================
- Hits          26699    26616      -83
- Misses        14208    14308     +100
- Partials       2969     2981      +12

Impacted Files
Coverage Δ

...e/dubbo/registry/consul/ConsulRegistryFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...g/apache/dubbo/registry/consul/ConsulRegistry.java
0.00% <0.00%> (-60.00%)
:arrow_down:

...nfig/configcenter/nop/NopDynamicConfiguration.java
62.50% <0.00%> (-12.50%)
:arrow_down:

...che/dubbo/remoting/transport/mina/MinaChannel.java
35.52% <0.00%> (-10.53%)
:arrow_down:

...o/remoting/transport/ChannelHandlerDispatcher.java
60.00% <0.00%> (-10.00%)
:arrow_down:

...g/apache/dubbo/remoting/p2p/support/FileGroup.java
41.50% <0.00%> (-9.44%)
:arrow_down:

...mmon/threadpool/support/fixed/FixedThreadPool.java
80.00% <0.00%> (-7.50%)
:arrow_down:

...ng/transport/dispatcher/all/AllChannelHandler.java
82.75% <0.00%> (-6.90%)
:arrow_down:

...moting/transport/netty4/NettyEventLoopFactory.java
30.00% <0.00%> (-6.37%)
:arrow_down:

.../threadpool/support/limited/LimitedThreadPool.java
83.33% <0.00%> (-5.56%)
:arrow_down:

... and 62 more

Continue to review full report at Codecov.

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


Microsoft Highly Confidential PDF Download?

Hi,
maybe the security of the document is no longer "Microsoft Strictly Confidential" or someone shared a document that should not be publicly available.
The article has a link to Validation Instructions for Data-at-rest Encryption for Microsoft 365 which points to https://aka.ms/CustomerKey/PublicPreviewValidation
This ends in a PDF file download which is classified as "Classified as Microsoft Highly Confidential"
Can you check this?

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

ID: 18803bfa-118a-09ee-947b-abf1ee996b7a
Version Independent ID: 87179b6d-d917-1aa9-259a-6f2de072c049
Content: Customer Key for Microsoft 365 at the tenant level (public preview) - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/customer-key-tenant-level.md
Service: o365-seccomp
GitHub Login: @KCCross
Microsoft Alias: krowley

@myfope Thank you for your feedback. We are checking this internally with the author and will provide an update shortly.
Thanks
Sri

@myfope Thank you for the feedback. We have removed the Classification on the file in the backend. The PDF does not contain the header now.
@yogkumgit The issue can be closed.
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @myfope Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


When anchor link is clicked, card glows

On larger views, the data cards on the home page are aligned horizontally. Clicking on an anchor link in the sidebar will take the user to an ambiguous place on the page (it may not be easy to identify which card matches the link just clicked). Use a javascript timed glow effect to signal to draw attention to the destination card.

Done


Typescript tsc: use tsconfig.json if present

This commit instructs the tsc compiler to search for a tsconfig.json file,
which indicates a typescript project. If found, tsc will be instructed to
compile the project instead of the individual file. If no tsconfig.json file
is present, single-file compilation will be used instead.

Please see #1778.  Sorry, but no.

Any more info on why this dynamic check is bad? I see similar things being done in other compilers, such as the Java compiler (where I found the pattern in the first place).
The current compiler does not provide a good experience for first-time vim users compiling a project; they will suddenly see compiler errors that are not present in any other common Typescript IDEs, which all support tsconfig files in this fashion.

Patches that change defaults are not accepted.
Patches that do things that can be easily achieved by changes to user's vimrc are not accepted, either.
The javac checker is a mess which is an order of magnitude more complex than any other checker.  It exists mainly for historic reasons, and it doesn't work particularly well with realistic-sized projects.  It should be simplified (or, better, killed), rather than duplicated elsewhere.
What javac does is a lot more involved than finding a config file.  Parsing pom.xml is not something that can be achieved by easy vimrc manipulations.
Fixing usability problems in external checkers is outside the scope of syntastic.
The main philosophy behind syntastic is that you, the user, are supposed to understand what you're doing, and syntastic shouldn't get in your way.  In most cases, making meaningful decisions automatically is not even possible, let alone desirable.
Keeping things backwards compatible is more important than making things smooth for first-time Vim users.  Frankly, first-time Vim users shouldn't use syntastic, to begin with.

I could go on, but you probably already disagree violently, so I'll stop here. :)

@lcd047 So how is it handled if you have two projects with separate tsconfig's? Or, with an arbitrary number of .d.ts files specified? For example, I get these types of errors when using syntastic:
...|1 col 20 error| TS2307: Cannot find module 'jquery'.
...|2 col 20 error| TS2307: Cannot find module 'lodash'.

@eggers You creatively use autocmds to set args to the appropriate combination of options for each project.  Something like this:
augroup some_name
    autocmd!
    autocmd BufRead,BufNewFile    */project1/*.ts
        \ let b:syntastic_typescript_tsc_args = '-p /path/to/config1.json' |
        \ let b:syntastic_typescript_tsc_fname = ''
    autocmd BufRead,BufNewFile    */project2/*.ts
        \ let b:syntastic_typescript_tsc_args = '-m es2015'
augroup END

Feel free to hate the Vim guys who came up with this way of doing things some ~20 years ago. :smile:

@lcd047, I am grateful for all the thought that you have put into this. As a heavy TypeScript user and syntastic fan, I would like to offer my opinion.
When tsc.vim was originally written, tsconfig.json did not exist, but it is now the de facto way of developing TypeScript projects. Searching for a parent tsconfig.json is the behavior implemented in Visual Studio Code, atom-typescript, tsuquyomi, and likely many other integrations. I understand that vim is not Visual Studio Code or Atom, and that vim users are expected to know what they are doing, however this is a good indication that the behavior provided by this patch is now expected by TypeScript users. By introducing and putting support behind tsconfig.json, Microsoft changed the expectation. In my mind, this makes a non-backwards compatible change worth considering.
I would also argue that searching for a parent tsconfig.json whenever a buffer is opened is not trivial. Many vim users (myself included!) are not anywhere close to proficient in Vimscript, but still find vim to be a huge productivity boost compared to other editors. You may see it as trivial, but I do not.
I'm writing this with the hope that you will reconsider. Either way, thanks again for all your work on syntastic!
@eggers and @mrtracy, in the meantime, using tsuquyomi's syntastic integration may work!

@jnetterf Did you happen to wonder what is the point of my list above, and how did I come up with it (and with the first two points in particular)?  At the time of this writing syntastic has 215 checkers for 102 filetypes.  In an ideal world, there would be 215 people that care about said checkers, or at least 102 people that care about said filetypes, and those people would maintain the corresponding checkers.  In the real world, you only get yours truly for the entire menagerie and the core.  Once in a while some kind soul ads a new checker, then runs away.  Seldom if ever anybody bothers to answer a question here about a checker they wrote, let alone post a patch to keep it up to date.  I'm afraid I value my sanity more than the expectations of the users of any of the individual 215 checkers.
Consequently, you have a few options:

Spend your energy arguing with me about defaults until I remove the checker.  It happened several times in the past, and we lived happily ever after.
Spend your energy convincing tsc maintainers to handle tsconfig.json automatically instead of relying on the editors out there to do it.
Spend two minutes understanding my solution above with autocmd and friting a few lines in your vimrc.
Maintain the tsc checker externally.

Your choice.

@lcd047 I understand where you're coming from -- syntastic is definitely an ambitious project, and it makes sense to limit the scope of accepted changes. Thanks again for maintaining it!


YouTube: youtube_dl.utils.RegexNotFoundError: Unable to extract Initial JS player n function name (but the video does download)

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2021.12.17
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log
$ youtube-dl -v "https://www.youtube.com/watch?v=Aw5XqnD6WwQ"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.youtube.com/watch?v=Aw5XqnD6WwQ']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.6.9 (CPython) - Linux-5.13.0-40-generic-x86_64-with-Ubuntu-21.10-impish
[debug] exe versions: ffmpeg 3.4.8, ffprobe 3.4.8
[debug] Proxy map: {}
[youtube] Aw5XqnD6WwQ: Downloading webpage
[youtube] Aw5XqnD6WwQ: Downloading MPD manifest
[youtube] Aw5XqnD6WwQ: Downloading player 534c466c
WARNING: [youtube] Unable to decode n-parameter: download likely to be throttled (Unable to extract Initial JS player n function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output. Traceback (most recent call last):
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/youtube.py", line 1514, in _n_descramble
    self._player_cache[player_id] = self._extract_n_function(video_id, player_url)
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/youtube.py", line 1487, in _extract_n_function
    funcname = self._extract_n_function_name(jscode)
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/youtube.py", line 1470, in _extract_n_function_name
    jscode, 'Initial JS player n function name')
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract Initial JS player n function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
)
WARNING: [youtube] Unable to decode n-parameter: download likely to be throttled (Unable to extract Initial JS player n function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output. Traceback (most recent call last):
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/youtube.py", line 1514, in _n_descramble
    self._player_cache[player_id] = self._extract_n_function(video_id, player_url)
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/youtube.py", line 1487, in _extract_n_function
    funcname = self._extract_n_function_name(jscode)
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/youtube.py", line 1470, in _extract_n_function_name
    jscode, 'Initial JS player n function name')
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract Initial JS player n function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
)
WARNING: [youtube] Unable to decode n-parameter: download likely to be throttled (Unable to extract Initial JS player n function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output. Traceback (most recent call last):
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/youtube.py", line 1514, in _n_descramble
    self._player_cache[player_id] = self._extract_n_function(video_id, player_url)
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/youtube.py", line 1487, in _extract_n_function
    funcname = self._extract_n_function_name(jscode)
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/youtube.py", line 1470, in _extract_n_function_name
    jscode, 'Initial JS player n function name')
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract Initial JS player n function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
)
WARNING: [youtube] Unable to decode n-parameter: download likely to be throttled (Unable to extract Initial JS player n function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output. Traceback (most recent call last):
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/youtube.py", line 1514, in _n_descramble
    self._player_cache[player_id] = self._extract_n_function(video_id, player_url)
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/youtube.py", line 1487, in _extract_n_function
    funcname = self._extract_n_function_name(jscode)
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/youtube.py", line 1470, in _extract_n_function_name
    jscode, 'Initial JS player n function name')
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract Initial JS player n function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
)
WARNING: [youtube] Unable to decode n-parameter: download likely to be throttled (Unable to extract Initial JS player n function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output. Traceback (most recent call last):
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/youtube.py", line 1514, in _n_descramble
    self._player_cache[player_id] = self._extract_n_function(video_id, player_url)
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/youtube.py", line 1487, in _extract_n_function
    funcname = self._extract_n_function_name(jscode)
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/youtube.py", line 1470, in _extract_n_function_name
    jscode, 'Initial JS player n function name')
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract Initial JS player n function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
)
WARNING: [youtube] Unable to decode n-parameter: download likely to be throttled (Unable to extract Initial JS player n function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output. Traceback (most recent call last):
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/youtube.py", line 1514, in _n_descramble
    self._player_cache[player_id] = self._extract_n_function(video_id, player_url)
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/youtube.py", line 1487, in _extract_n_function
    funcname = self._extract_n_function_name(jscode)
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/youtube.py", line 1470, in _extract_n_function_name
    jscode, 'Initial JS player n function name')
  File "/snap/youtube-dl/4630/lib/python3.6/site-packages/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract Initial JS player n function name; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
)
[debug] Default format spec: bestvideo+bestaudio/best
WARNING: Requested formats are incompatible for merge and will be merged into mkv.
[debug] Invoking downloader on 'https://rr1---sn-ov8vuxaxjvh-v8cl.googlevideo.com/videoplayback?expire=1650634353&ei=EVpiYvGiMoWhyAW61LWYDA&ip=<IP_ADDRESS>&id=o-ANPP78zOHlZpwbOzDVEOohnqVvBaEa_AsEx00oFNuLTx&itag=136&aitags=133%2C134%2C135%2C136%2C160%2C242%2C243%2C244%2C247%2C278&source=youtube&requiressl=yes&mh=SO&mm=31%2C29&mn=sn-ov8vuxaxjvh-v8cl%2Csn-n8v7znlr&ms=au%2Crdu&mv=m&mvi=1&pcm2cms=yes&pl=19&initcwndbps=1573750&spc=4ocVC8nskdHytRZ-Abo2OQrdvqbQ&vprv=1&mime=video%2Fmp4&ns=UR7MdGlhE0wk_sywZ77hfn4G&gir=yes&clen=32606163&otfp=1&dur=117.366&lmt=1520938931983944&mt=1650612283&fvip=15&keepalive=yes&fexp=24001373%2C24007246&c=WEB&n=5KbcBBNu7gokaDANH&sparams=expire%2Cei%2Cip%2Cid%2Caitags%2Csource%2Crequiressl%2Cspc%2Cvprv%2Cmime%2Cns%2Cgir%2Cclen%2Cotfp%2Cdur%2Clmt&sig=AOq0QJ8wRgIhAMNu0GOsnct2DB6lhkT9dvWt74CvGPGWiKG1_jiR3O6YAiEAlVxj9swGnUOyHOCML50h94TecrAdwaNSYA1O-BJquA0%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpcm2cms%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRAIgWAlj7E71lXT7K_PUPlWus9IUCjNL0GlOM5Cyzpq0eR8CIBsmOKGb0l5rg32giwuFDqcPjClwsSLF0PRytdr3piiT'
[download] Destination: [FFXIV OST] The Azim Steppe (Day)-Aw5XqnD6WwQ.f136.mp4
[download]   0.2% of 31.10MiB at 64.08KiB/s ETA 08:15

Description
The above warnings are displayed for me when trying to download https://www.youtube.com/watch?v=Aw5XqnD6WwQ.
The video does download, but slowly.

There's no n-descrambling in the released 2021.12.17 version.
Your yt-dl must be an older patched version. You need to update to the latest git master.
Duplicate of #30856.

Confirming v2021.12.17 fixes the "n function name" issue....
_- still looking for any open tickets for the throttling games on YT ( showed up ~ 7-14d ago )

was running a custom build from 2021.06.06 with drop-in patch
pulled latest, but throttling still there..._

TRACE:
wget https://youtube-dl.org/downloads/latest/youtube-dl
youtube-dl https://www.youtube.com/watch?v=croh7Qo3x9k
[youtube] croh7Qo3x9k: Downloading webpage
[download] Resuming download at byte 3595369
[download] Destination: TWO DEITIES from the BIBLE - can U TELL the DIFFERENCE - podcast 1234-croh7Qo3x9k.f135.mp4
[download]   1.4% of 274.52MiB at 77.74KiB/s ETA 59:26^C
ERROR: Interrupted by user
$ youtube-dl --version
2021.12.17

nm.. found the thread & pulled master > make = happiness ensued

I ran into the same problem on Ubuntu 21.10 with youtube_dl  2021.12.17. The problem is with the snap version.  Canonical installs the snap version with apt install. I took this error seriously because my download speed was low and the warning "Unable to decode n-parameter: download likely to be throttled" appeared. As advised in this thread, I downloaded youtube_dl directly from https://youtube-dl.org/downloads/latest/youtube-dl. This error was resolved. However, the download speed did not change. Still pretty low.

Youtube-dl doesn't have any explicit library dependencies so there's no reason to install it as a snap. What you thought was advice to install https://youtube-dl.org/downloads/latest/youtube-dl was actually suggesting to install the program from this GitHub repository.

Using a MacMini M1, brokwn for me as of 3rd Apr 2023.
I trued brew upgrade, downloaded directly from https://youtube-dl.org/downloads/latest/youtube-dl and also tried setting up a 3.10 venv and running as a module, all fail with the SAME error message:
(env) ➜  Downloads ./youtube-dl --verbose https://www.youtube.com/watch\?v\=EmQcQquj8Hs
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://www.youtube.com/watch?v=EmQcQquj8Hs']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.10.10 (CPython) - macOS-13.3-arm64-arm-64bit
[debug] exe versions: ffmpeg 5.1.2, ffprobe 5.1.2, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] EmQcQquj8Hs: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/Users/x/Downloads/./youtube-dl/youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "/Users/x/Downloads/./youtube-dl/youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "/Users/x/Downloads/./youtube-dl/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/Users/x/Downloads/./youtube-dl/youtube_dl/extractor/youtube.py", line 1794, in _real_extract
    'uploader_id': self._search_regex(r'/(?:channel|user)/([^/?&#]+)', owner_profile_url, 'uploader id') if owner_profile_url else None,
  File "/Users/x/Downloads/./youtube-dl/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Not sure how to proceed, I'll check out the code I guess, maybe there are issues with a pop installation of that library for an M1 ? Woldn't be the first time I've had problems, pyscopg2 was a pain for a while.

RESOLVED: FOund the patched version , worked ok.


Use proper language parser for explicit imports  check

From https://github.com/Plutonomicon/cardano-transaction-lib/pull/703#issuecomment-1178781769:

In the future we could look into using https://github.com/natefaubion/purescript-language-cst-parser to re-write this check. I don't see it taking longer than a days work, and imo it's better than forking purs-tidy and having to keep up with their changes.

This could also be a starting point to introduce further checks like ensuring all universally quantified type variables have an explicit kind, etc

I guess the plan is to write a PS linter then? This is something I've missed quite a bit

I would like to write something like that. Since this task is of lower priority, maybe doing little advances between tasks?

@Luis-omega it is not needed as my grep-based solution (make check-explicit-exports) performs reasonably well as time has shown

All right, should we close this ticket then?


Increase security by specifying network policies

K3S uses CNI Flannel, which does not support proper networkpolicies. Will park this for now, and revisit if relevant later (like Flannel gets full support, or K3S changes CNI)


feat: add invalid access/trigger token error handling

Adding invalid access/trigger token error handling:

GitLab returns 404 HTTP status code on requests with an invalid trigger token, as a security measure. Exit immediately.
GitLab returns 401 HTTP status code on requests with an invalid access token. Exit immediately.

Fixes: #34

Looks good to me, thank you very much!


python312Packages.tesla-fleet-api: 0.5.11 -> 0.5.12

Diff: https://github.com/Teslemetry/python-tesla-fleet-api/compare/refs/tags/v0.5.11...v0.5.12
Changelog: https://github.com/Teslemetry/python-tesla-fleet-api/releases/tag/v0.5.12
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 313165 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.teslemetry
python311Packages.tesla-fleet-api
python311Packages.tesla-fleet-api.dist
python312Packages.tesla-fleet-api
python312Packages.tesla-fleet-api.dist


Support raw Kubernetes manifests

I know this is a helm plugin, but it would be nice to be able to extract images from a directory or file or STDIN of raw Kubernetes manifests (not a helm chart).  Consider this use case.  I have a skaffold.yaml that describes how to install a half dozen helm charts and k8s manifests.  I would like to extract all the images in use by the charts in use by the skaffold file, including the values that are added to the charts.  skaffold render dumps the raw k8s manifests as YAML to STDOUT.  I would like to be able run skaffold render | helm images get -raw=-  and have it dump the images to STDOUT.
So in short I proposing that a flag is added called -raw that can take a directory, file, or - (for STDIN) and extract the images from there.  Otherwise I have to make a fake/temporary helm chart to feed into helm images get with these manifests.  I suspect helm template is effectively called within helm images get so we just need to expose that.

@ktarplee, that sounds interesting! However, I don't have any hands-on experience with skaffold yet. Let me read up on it before diving into possible solutions.


add a sharing modal dialog

fix https://github.com/dart-lang/dart-pad/issues/285.

@lukechurch @sethladd
This asks each time; we can tweak that if we want, to once per browser, or once per day.

s/publically/publicly

lgtm
Dear @devoncarew esq, whilst I admire your use of one of the more interestings corner of English, we should probably consider the more popular usage: https://books.google.com/ngrams/graph?content=publicly%2Cpublically&year_start=1600&year_end=2000&corpus=15&smoothing=3&share=&direct_url=t1%3B%2Cpublicly%3B%2Cc0%3B.t1%3B%2Cpublically%3B%2Cc0


Windows 8 Compatability

The test suite singletest.py does not appear to work with windows 8.
The issue seems to be with Serial, which all of the tests rely on.
After the test suite resets the target, the response received on serial is as follows:
HOST: Unknown property:
When viewed through a serial terminal:
{{timeout;5}}
{{host_test_name;hello_auto}}
{{description;Hello World}}
{{test_id;MBED_10}}
{{start}}
Hello World
[00]{{timeout;5}}
{{host_test_name;hello_auto}}
{{description;Hello World}}
{{test_id;MBED_10}}
{{start}}
Hello World

Replicating the same build environment on a windows 7 machine, all tests run successfully.
It could be a symptom of the virtual machine however...
I am running windows 8 on the VMware box, but my serial terminal can see it, why can't the script?

is the serial driver installed on the windows 8 machine?

@sg- the one referred to here? https://developer.mbed.org/handbook/Windows-serial-configuration

Also to clarify, you don't have the terminal and the test suite running at the same time right? I believe the test suite will try to "own" the serial port and if the terminal is running it won't be able to run correctly.

@bridadan I do not, it would give a different error if that were the case.

@jamesadevine yes

@sg- it is installed

@PrzemekWirkus do we have any machine with this OS which causes this issue?

@0xc0170 No we do not, and I do not think we "support" Windows 8.
For now we are working with Windows 7, Ubuntu LTSs and for some tools MacOS (Darwin).

@jamesadevine We do not support test suit running on VMs yet!


Remove old plotly dependency

removing plotly 3.6.0 line in setup

closes #14

🙈 thanks for this :)


UI Upgrade

What this PR does / why we need it:

Upgrade UI dependencies as such:
Node 14 -> Node 16
React 16 -> React 17
Elastic UI 32.x -> 66.0
gojek/mlp-ui -> 1.6.2
Joblib has also been upgraded to 1.2.0
Screenshots of updated UI below:

Does this PR introduce a user-facing change?:

None
Upgrade UI Dependencies Versions: Node 14 -> 16, Elastic UI 32 -> 66, React 16 -> 17
Upgrade joblib version to 1.2.0

Checklist

[x] Added unit test, integration, and/or e2e tests
[x] Tested locally
[ ] Updated documentation
[ ] Update Swagger spec if the PR introduce API changes
[ ] Regenerated Golang and Python client if the PR introduce API changes

Joblib has also been upgraded to 1.2.0

Can you make separate PR?

Can you add context why the upgrade is necessary in the PR description?


[Feature Request]Allow users to enable extensions with workspace file

The already-had extension 'Enable (Workspace)' option is kind of implicit and is hard to tell which extensions are workspace enabled and which are globally enabled. I want vscode to read from the .code-workspace file and enable the extensions which are written explicitly in the file.
It would also be better if I can install the extensions which will only be used in one folder in that folder. By that I mean if I want to install some extension locally just in the project folder like node modules.

/duplicate

@sandy081 Can you cite the issue number that this issue is a duplicate of? I tried but did not find it. Thank you!

https://github.com/microsoft/vscode/issues/40239


Feature #2375: Add support for ++debug to charmrun shell scripts

I also can't imagine how any of these would actually work as expected
Whether they work or not will depend on the environment and particular use. I frequently run mpirun -np N xterm -e gdb -ex r --args ./myProgram +whatever with the MPI machine layer successfully. If it works it works, if it doesn't it doesn't, but at least this PR adds support for at least trying.

other than multicore, which only has one process anyway so doesn't need to launch xterms.
I did this for consistency. Other machine layers also don't need to be launched with Charmrun if only one PE will be used and launch in an xterm window with just one PE. If you don't want xterm, launch the process with your debugger directly (skipping Charmrun), and this is true for all machine layers that support standalone invocation.

What would be more useful is documentation of how to run a Charm++ program in a parallel debugger on each platform.
That's outside the scope of the issue this PR resolves, and also not something I can write given no experience with parallel debuggers. I don't want to to block this PR based on this separate issue. If you want, create a new issue and assign it to someone with actual parallel debugger experience.


Help in general for using git from the terminal

This is the document made by Jess and Jasleen which summarizes common commands needed to make commits.

https://rhul-my.sharepoint.com/personal/zhap066_live_rhul_ac_uk/_layouts/15/Doc.aspx?sourcedoc={bf99300a-cfbb-4cc4-8ad9-ad47a7597ab2}&action=edit&wd=target(2_getting_started_with_git.one|39a8bf3f-bd69-4681-8922-3604ff76305a%2FGetting started with git (Jasleen and Jess\)|47533053-2f94-40f0-ab86-914a8d08233c%2F)&wdorigin=NavigationUrl


Add CI check to make sure that sc's deployment size doesn't exceed 24kb

Also consider adding coverage reports in the CI (using forge coverage)


Updated to login and function on <IP_ADDRESS>

Getting a parse error right here

Where?

Its a line comment, click the src/snapchat_agent.php link

Its a line comment, click the src/snapchat_agent.php link

Fixed with: https://github.com/mgp25/Snap-API/pull/105


Code coverage for project, and coverage bot

Hi, I think it will be helpful add coverage for repository like
And will be very helpful coverage bot to shows how to change coverage after pull requests (for example - https://codecov.io/

@YuriiPovkh thank for the suggestion.
I don't know how to added for .net core project. Do you know any .net project used codecov?

@thiennn can you create an account for SimplCommerce in https://codecov.io/ and add the SimplCommerce repository, after that I can add the require script to make this happen, this tool is free for open source projects and it gives you a rich STATS like this https://codecov.io/gh/WormieCorp/Localization.AspNetCore.TagHelpers

@thiennn this link https://joaorosa.io/2016/10/12/integrate-net-core-appveyor-opencover-and-codecov/ will helps you a lot


Unsubscribe all connectivity callbacks on Channel.close

Fixes https://github.com/grpc/grpc/issues/18995
There could be a better fix available, but seems like the code should be reworked more fundamentally anyway so this simple fix could be fine for now

cc @lidizheng

@lidizheng it's hard to test against it without refactoring the logic entirely (it'd be easier to fix the entire logic than to test against it)--the reason being the symptom (i.e. the exception) is thrown on a different thread.

I agree the text logic probably is more complicate than the fix itself. But it shouldn’t be a challenging one. You can create the callback with the access to the unittest object.
I will label this as help wanted, so if someone have bandwidth can work on it.

There's no test for any of these in Channel.close (hence the mess in the first place). If you feel this is not worthwhile progress to fix customer pains short term, then fine. Otherwise, I recommend merging and filing a "help wanted" issue for adding the test.

Known failures:

https://github.com/grpc/grpc/issues/18305
https://github.com/grpc/grpc/issues/17330


Revert "update"

Reverts xunpan/federation-v2#1

I don't want to add my commit but only sync-with federation v2. Why there has no simple button to do that?


README updated to reflect changes from rx2

Let me know if I missed any, pretty sure there are a few I did.

Lgtm wanna do the wiki too? Thank you so much for your continued contributions.

Yea sure, and np
On Tue, Jul 11, 2017 at 11:27 PM, Mike Nakhimovich<EMAIL_ADDRESS>
wrote:

Lgtm wanna do the wiki too? Thank you so much for your continued
contributions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NYTimes/Store/pull/243#issuecomment-314634350, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AIq06KuqoUPIW49okcjaGV0UJw5x2WqRks5sND0dgaJpZM4OVFGg
.

--
Kevin Cronly
New York University | Computer Science and Statistics
<EMAIL_ADDRESS>201-398-8671 <(201)%20398-8671>
https://www.linkedin.com/in/kevincronly/


Design dynamic fees

Deliverables

[x] Show dynamic fee next to current base fee in UI
[x] "i" tooltip when dynamic fee is in effect

https://www.figma.com/file/sghReDviy1lcr6kkNqlc0e/?node-id=6120%3A230761
link to figma


Issue with LocalDate as literal in 1.6+

Description
Our application is working just fine in 1.5, but when we try upgrading to 1.6, we all of a sudden start seeing query exceptions when trying to use LocalDate/LocalDateTime as literals in the criteria builder.
Expected behavior
Dates get properly parsed into the query
Actual behavior
We get an error message that says java.lang.IllegalArgumentException: org.hibernate.QueryException: unexpected char: '{' with the date being parsed into the query as {d '2021-07-12'}
Steps to reproduce
We're using a PaginatedCriteriaBuilder to build an EntityView query against a simple Entity model. We're not doing much more than just saying builder.where("field").leLiteral(LocalDate.now()) and we're getting the error message. Most of our queries use dates, so this is breaking our entire application. If I find time, I will setup a proper test case for this.
Environment
Version: 1.6+
JPA-Provider: Hibernate 5.3.20.Final
DBMS: MySQL 5.7
Application Server:  Wildfly 24

Thanks for the report. I'll look into it next week when I'm back from my vacation :)

How did it get parsed on 1.5? Because {d '2021-07-12'} seems a valid date literal to me. Were your prior tests also against Hibernate 5.3.20.Final?

@jwgmeligmeyling not sure, is there a way to check what was being parsed? The only way I can tell right now is because I'm receiving the error message. Yes, it was working on 1.5/1.5.1 on Hibernate 5.3.20.Final.

The exception suggests that the literal is passed through to HQL but it doesn't support the JDBC escape syntax properly. For this purpose, we actually created some emulations if I remember correctly, but these are conditional based on the Hibernate version. Are you running with an uber jar maybe @jcputney ?

@beikov No, we're just using the version built into Wildfly 24

It would be awesome if you could share a reproducer for this because we actually have a test for this. You can add your reproducer to that test if you like: https://github.com/Blazebit/blaze-persistence/blob/master/core/testsuite/src/test/java/com/blazebit/persistence/testsuite/TemporalLiteralRenderingTest.java

I was able to reproduce your issue and just opened a PR to fix this. In case you want to verify that your particular issue is fixed, let me know if the fix works for you :)


Mega Menu: Suspend 3rd level menu at desktop

Changes

Updates incorrect URL and replaces and with & in menu item text.
Adds method to suspend/resume trigger events on a flyout menu.
Suspends trigger events on the 3rd-level trigger at the desktop menu size so that the triggers (e.g. About Us > Careers) won't prevent the default link behavior.

Testing

On refresh clicking About Us > Careers just closes the mega menu. Run gulp build and see that going from About Us > Careers goes to the careers overview page.

Review

@jimmynotjim
@KimberlyMunoz
@sebworks

:+1:

:+1:


Disable ring-proof zk-rows with "test-vectors" feature

Superseeded by https://github.com/davxy/ark-ec-vrfs/pull/33


Defining both custom_vjp and custom_jvp

I was wondering is it possible to define both custom vjp and jvp for a function?
from jax import custom_vjp, custom_jvp, jacfwd, jacrev

def f(x):
 return x

f = custom_jvp(f)
f.defjvp(lambda *args: args)

f = custom_vjp(f)

def fwd(*primals):
 return (primals, 0.)

def bwd(res, ct):
 return ct
f.defvjp(fwd, bwd)
print(jacfwd(f)(1.))
print(jacrev(f)(1.))

Above fails with
TypeError: can't apply forward-mode autodiff (jvp) to a custom_vjp function.

Thanks for the question!
No, that's not currently possible. It's documented in the custom derivatives tutorial (see the subsection "Forward-mode autodiff cannot be used on the jax.custom_vjp function and will raise an error" for more).
Can you say more about why you might need to define both?

Aha, didn't see that sentence in the tutorial.
My use case is quite intricate and very particular.
I was trying to fix https://github.com/tensorflow/probability/issues/1045.
The idea is that oryx defines this "tagging" transformation; what I want is for that tag to preserve after jacfwd/jacrev/grad transformations.
So I wanted to override both the jacfwd and jacrev for the tagged value (actually, a function that returns those tagged values); the overridden behavior is to just tag the gradient values.
It's a bit hacky IMO, but it's at least an attempt at this.

There are a few ways you could solve this, but one would be to create your own version of Oryx's sow primitive that has different behavior under JVP and transpose transformations (another would be to expose more control over how sow itself behaves, along the lines of Oryx's existing support for controlling how sow behaves under a scan).
CC @sharadmv
(Separately, the most straightforward answer to "how do I have a custom JVP and VJP at the same time" is to create a full-blown Primitive.)

What are your thoughts on rather than tying harvest to sow exclusively, it looks inside a set of primitives that all have  name/tag/mode params. This could enable users writing their own with specialized semantics however they want, then adding their primitive to the set.

I'd like to allow request the ability to define both the custom_vjp and custom_jvp.
I work on PennyLane, where we register derivatives computed on quantum computers with JAX.
Currently, we compute the full jacobian under a pure_callback, then let JAX trace the computation of the jacobian product. This allows us to provide both the jvp and vjp.
If we were to use a pure_callback around either the jvp or the vjp instead, we could be more efficient with our resources. Unfortunately, we would then lose the ability to support both modes.
So, our lives would be much easier if we could register both the jvp and vjp and compute them both under a jax.pure_callback.

I also would like to have both custom_vjp and custom_vjp. I want to compute the Hessian matrix of a model, so it is good to have both rather than using grad twice.

+1 for allowing both custom vjp and jvp.

In my application space, many of the important operations are related to large matrices that are never actually formed explicitly (because they are much too large to fit in memory). For example, if $u, v, x$ are column vectors with 1,000,000 components, the function
$$ f(x) \coloneqq u (v^\top x) = A x, \qquad A \coloneqq u \; v^\top $$
has trivial implementations of its vjp ($dy^\top A = (dy^\top u) v^\top $) and jvp ($A \\\; dx = u (v^\top dx)$), but forming $A$ from the jvp definition and transposing it to implement the vjp is incredibly inefficient, if not impossible.
This example is trivial for the sake of demonstration, but the underlying "matrix-free" idea is important in Krylov methods and HPC applications.

For those who are interested in this topic, JAX team published a paper on this. It explains the underlying philosophy.
https://dl.acm.org/doi/abs/10.1145/3571236

Believe it or not, @dfm is about to fix this issue!

@mattjj Do you have a link to the relevant issue/PR? And should this be closed?

@carlosgmartin — I think @mattjj might have overstated the "about to fix" part of his comment :D. We're still iterating on design questions, and realistically I don't think that a good solution will land before https://github.com/google/jax/pull/23299. There is a proof-of-concept in https://github.com/dfm/jax/tree/custom-ad-2, but I don't have a timeline for when/what will be merged. Always happy to help with specific use cases while we wait!


avoid files having the same files inside

for instance the enviNFe_v3.10.xsd is including leiauteNFe_v3.10.xsd then both enviNFe_v3.10.py and leiauteNFe_v3.10.py have the same massive TNFe class

done by dealing only with the core XSD files


topology command: ascii visualization

Discarding AsciiART.. the text of the resources is not readable

The most promising tool to create ASCII graphviz graphs is http://search.cpan.org/~tels/Graph-Easy/bin/graph-easy (https://github.com/ironcamel/Graph-Easy). No activity in the last 14 years.
I tried it and fails with some unsupported DOT attribute:
❯ graph-easy /home/eguzki/tmp/topology.dot
Error in attribute: 'note' is not a valid shape for a node at /usr/bin/graph-easy line 90

The graph in dot format being
digraph  {

	n5[label="AuthConfig\nkuadrant-system/e2db39952dd3bc72e152330a2eb15abbd9675c7ac6b54a1a292f07f25f09f138",shape="ellipse"];
	n4[label="Authorino\nkuadrant-system/authorino",shape="ellipse"];
	n14[label="AuthPolicy\ndefault/toystore-authn",shape="note",style="dashed"];
	n2[label="ConfigMap\nkuadrant-system/topology",shape="ellipse"];
	n6[label="EnvoyFilter\ngateway-system/kuadrant-auth-kuadrant-ingressgateway",shape="ellipse"];
	n10[fillcolor="#e5e5e5",label="Gateway\ngateway-system/kuadrant-ingressgateway",shape="box",style="filled"];
	n12[fillcolor="#e5e5e5",label="Listener\ngateway-system/kuadrant-ingressgateway#http",shape="box",style="filled"];
	n9[fillcolor="#e5e5e5",label="GatewayClass\nistio",shape="box",style="filled"];
	n8[fillcolor="#e5e5e5",label="GatewayClass\nistio-remote",shape="box",style="filled"];
	n11[fillcolor="#e5e5e5",label="HTTPRoute\ndefault/toystore",shape="box",style="filled"];
	n13[fillcolor="#e5e5e5",label="HTTPRouteRule\ndefault/toystore#rule-1",shape="box",style="filled"];
	n1[label="Kuadrant\nkuadrant-system/kuadrant",shape="ellipse"];
	n3[label="Limitador\nkuadrant-system/limitador",shape="ellipse"];
	n7[label="WasmPlugin\ngateway-system/kuadrant-kuadrant-ingressgateway",shape="ellipse"];
	n14->n11[comment="Policy -> Target",style="dashed"];
	n10->n6[comment="Gateway -> EnvoyFilter"];
	n10->n7[comment="Gateway -> WasmPlugin"];
	n10->n12[comment="Gateway -> Listener"];
	n12->n11[comment="Listener -> HTTPRoute"];
	n9->n10[comment="GatewayClass -> Gateway"];
	n11->n13[comment="HTTPRoute -> HTTPRouteRule"];
	n13->n5[comment="HTTPRouteRule -> AuthConfig"];
	n1->n8[comment="Kuadrant -> GatewayClass"];
	n1->n9[comment="Kuadrant -> GatewayClass"];
	n1->n3[comment="Kuadrant -> Limitador"];
	n1->n4[comment="Kuadrant -> Authorino"];

}


Limit undo/redo to word text, not clues and titles

This is closer to the original intent.

My bad.  This was already fixed.  Never mind.


Latest fish master might make commandline sequencing easier

Latest fish master includes
c3cd68dda (Process shell commands from bindings like regular char events, 2024-03-02)
a583fe723 ("commandline -f foo" to skip queue and execute immediately, 2024-04-08)
which should make the execution order more natural.
This should address the note in the readme
The second commit means that commandline synchronizes properly. I wonder if that breaks/improves anything; tests seem to pass still.
The selection update after history search seems to be broken, I'll try to investigate. nevermind that also happens on 3.7.1
Overall this project is impressive. One thing that seems missing is an insert-mode way to accept autosuggestions.
(I'm not sure if I'd use it, I'm used to kakoune bindings)

there are also a bunch of improvements to Vi mode (see CHANGELOG.rst) that might benefit fish-helix.
Maybe you want to do set fish_cursor_end_mode inclusive to make sure that the cursor never selects the one-paste-last character. For Vi mode this improves autosuggestion behavior (for commands like $. incidentally I'm not sure how to go to the beginning/end of line with fish_helix_key_bindings. x is missing I think)
I think we'll still need some changes to fish to make it easier to write such bindings. But such changes should be easy and not contentious

@krobelus Thanks for the info and for the feedback!
x is missing, it's also number one on my wishlist, but I seem to never get to it :( Alt+E calls external editor, and it seems enough for rare cases.
You can accept autosuggestions in insert mode with Right Arrow key: many default fish bindings are still in place

Completely untested but I think for the one-line case, x should be easy to implement with something like commandline -f beginning-of-line begin-selection end-of-line... not sure
Yeah right arrow should  work for me, just would need to change habits (except on keyboards where it's not so easy to reach)
I also use alt-e a lot, it even synchronizes the cursor position now though only for Vim/Kakoune