Disable initial transition

Hi. I'm new to react-spring, but have a background in react-motion. My main goal while using this library was to make the code simpler and more readable, which I believe was one of the goals of this library in comparison with react-motion.
However, while migration from TransitionMotion to Transition, I've faced a problem, where I couldn't disable the animation for the first mount. In react-motion willEnter applies only to the new elements, not the initial ones, which makes me able to render the children for the first time without the animation.
However, when I try to move to react-spring's Transition, it applies from styles to the first children and animates the as if they have just entered.
This probably would be okay if it wouldn't produce performance issues. E.g. when I navigate to the page with my Components with transition, animation freezes for a moment, even when using native and animated.
So my question is - is it possible to disable the initial animation for components rendered via Transition, so that it only animates the entering/leaving nodes after the first mount.

For now try this: https://spectrum.chat/react-spring
I’ll add a flag or something to make this easier, any ideas?

Thinking of something else right now, why not simply omitting "from" ? If it doesn't have base styles  it can't animate the first enter styles. Actually, i like that even as a solution because it seems more logical than a flag.

As of @5.7.0 you have an additional props "initial" which you can use to override "from" on first mount. If you do <Transition initial={null} /> you effectively remove first-mount base styles, to it jumps to "enter" immediately. You can also use it to apply animated first mount styles in general.
I'm in the process of documenting it all, here's a temporary version listing all the current props: http://steady-impulse.surge.sh/transition

Great news!
I'll try this out soon and report if I meet any issues.
Thanks for the quick reply!


Implement entrainment/detrainment boundary conditions in EDMFX

Following @costachris
entr = (1/z)*(-4.013288 - 0.000968*Π₁ + 0.356974*Π₃ - 0.403124*Π₄ + 1.503261*Π₆
detr = (3.535208 + 0.598496*Π₁ + 1.583348*Π₃ + 0.046275*Π₄ - 0.344836*Π₆) + max_area_limiter

where max_area_limiter forces more detrainment when updraft area approaches max area. Without it, an overwhelming majority of TC simulations failed in the first couple of EKI iterations when updraft area maxed out.
The max area limiter is
max_area_limiter = 0.1 * exp(-10 * (maximum_area - updraft_area))
The Pi-groups are defined in here
Calibrations are unstable for deep convection (TRMM) and raining cumulus (Rico). Regardless, these should be enough to get started with EDMFX

Also, the above formulation is 1/length. We want it to be 1/time, for example to treat parts of it implicitly or to be able to limit the tendency my the model time step.
We should absorb the physical velocity into entrainment detrainment. This will give us the form we want

@costachris - looking at the units here. Shouldn't detr also be divided by z?


WunderGraph Router Doesn't Resolve Key Field with Multiple Keys When Using @external and @provides

Description:
I am experiencing an issue with WunderGraph Router where it fails to resolve a key field defined with multiple keys when using @external and @provides across federated subgraphs.
Subgraphs:
Subgraph A
type A @key(fields: "ax") @key(fields: "ay") {
  ax: ID!
  ay: String!
  az: String
}

Subgraph B
type A @key(fields: "ax", resolvable: false) @key(fields: "ay", resolvable: false) {
  ax: ID! @external
  ay: String! @external
}

type B {
  bx: ID!
  by: String
  bz: String
  a: A! @provides(fields: "ax")
}

Steps to Reproduce

Define Subgraph A and Subgraph B as shown above.
Query the router with the following:

Query 1 (Works as expected)
query Test1 {
  getB() {
    bx
    by
    a {
      ax
      az
    }
  }
}

This query successfully fetches the az field from Subgraph A
Query 2 (Fails to resolve correctly)
query Test1 {
  getB() {
    bx
    by
    a {
      ax
      ay
      az
    }
  }
}

This query does not fetch the ay field from Subgraph A and incorrectly attempts to resolve it from Subgraph B, despite it being marked as @external.
Expected Behavior
The router should be able to resolve the ay field from Subgraph A when queried alongside the other fields, given that at least one key is provided.
Version
WGC version: 0.66.0
router version: 0.115.0 (https://github.com/wundergraph/cosmo/releases/tag/router%400.115.0)

@apatelWU Please update composition and the router to the latest versions as a first step before reporting bugs.
Please kindly try again with the latest versions, and let us know the result.
Thanks!

New versions are working as expected. Thanks!


fix: escape test file path

In case special symbols are present in file path, jest won't find the actual file, thus failing the test run.
Example:
libs/store/entity/src/lib/+state/entity.reducer.spec.ts
Encountered this when trying to test ngrx-store spec files generated by @ngrx/schematics:store.

Closing in favor of https://github.com/nvim-neotest/neotest-jest/pull/91, vim.fs.normalize(pos.path) solves problem for me.


[3.x] Fix multiple ubershader bugs

I've kept the fixes in separate commits, since each one addresses one specific issue, and I believe this way is better to convey information to future maintainers.

Fix shadow flickering with async shader compilation
This mostly reverts the approach in https://github.com/godotengine/godot/pull/62628, which now the problem is better scoped, looks overengineered and instead focuses on the few cases where there's something to take care of.
Fix emission not working in the ubershader
Fix GI probes not working in the ubershader

Fixes #63346.
Fixes #64040.

Thanks!

Cherry-picked for 3.5.1.

Does this also fix #64038 (the exposure flickering issue I mentioned)?

Good question. I didn't test, but it may... Sadly, I have no time for testing it now.


Rename the require_verification config option to be clearer

Right now the require_verification config option isn't very clearly named especially when seen with the isolation_verification. So we should rename it to be better.

I am closing this issue as it has been resolved and integrated into mainline.


Add simple-flash

https://github.com/tamtamchik/simple-flash - Easy, framework agnostic flash notifications.

Thanks for the suggestion.


Fix dependencies

Some missing or misspelled dependencies. Checked on Ubuntu 19.04.

Thank you :+1:
In fact, we can even replace android-tools-adb by adb (I just did it: a13524e7f9d696635983776c548adaa1b9c8508f).


Finish #3 monitor usage

Closes #3

Codecov Report

Merging #10 (d7711fc) into develop (2baf466) will decrease coverage by 0.75%.
The diff coverage is 71.42%.

@@             Coverage Diff             @@
##           develop      #10      +/-   ##
===========================================
- Coverage    85.28%   84.53%   -0.76%
===========================================
  Files           22       24       +2
  Lines          367      388      +21
===========================================
+ Hits           313      328      +15
- Misses          54       60       +6

Impacted Files
Coverage Δ

lib/src/model/usage.dart
25.00% <25.00%> (ø)

lib/src/model/usage.g.dart
100.00% <100.00%> (ø)

lib/src/translator.dart
89.14% <100.00%> (+0.30%)
:arrow_up:

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 2baf466...d7711fc. Read the comment docs.


Fix batch_size sanity check logic for split_batches

What does this PR do?

Fixes #2285 and PR/2310 comment: Unfortunately, this broke Megatron DL
Some libraries may not follow PyTorch's design of BatchSampler.
If the batch_size cannot be found, a warning is left to notify the user.
Do anyone have suggestions for such situations?
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you read the contributor guideline,
Pull Request section?
[x] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.
@muellerzr @BenjaminBossan

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

While this PR makes the error user-friendlier - this approach swipes the problem under the carpet. If anything it should be an assert and not a warning. i.e. if the user wants split_batches this code path must not proceed.
But there is more to it - I have never set split_batches=True - so somewhere in the stack I reported at https://github.com/huggingface/accelerate/pull/2310#issuecomment-1893072023 Accelerate sets it to True by default w/o user's intent to do so.
I think you have this problem here as well:
https://github.com/huggingface/accelerate/blob/c0a94724223b34a0e2e93f1768c069584ba48ca0/src/accelerate/data_loader.py#L487C9-L492
it always calls sampler.batch_size. so that code also probably should check hasattr and assert if it is False.


ID Site Timeouts result in JSON Error

Steps to reproduce:

Open up ID Site. Leave it open for longer than the timeout (5 mins I believe)
Login with a user. (Not sure if it has to be valid or not)
Results in that error.

Action items:

Express-Stormpath should render this error properly, or just redirect someone back to ID Site which needs to render this error?

ID Site does redirect the user back to the service provider (SP) so that the SP can decide how to handle the error.  In this situation, some of our customers have chosen to re-direct the user back to ID Site with a new jwtRequest (and as such, this error is never seen by the end user).  However I still think that workflow is confusing, to be redirected back to ID Site without a hint about what happened.  I'm not sure what I want to do here.


dial tcp: lookup prometheus_postgresql_adapter: No such host

I run the docker-compose and receive:
timescale-prometheus-connector_1  | {"addr":":9201","caller":"main.go:280","level":"info","msg":"Listening","ts":"2020-08-27T14:15:17.645Z"}
prometheus_1                      | ts=2020-08-27T14:17:38.800Z caller=dedupe.go:112 component=remote level=warn remote_name=686501 url=http://prometheus_postgresql_adapter:9201/write msg="Failed to send batch, retrying" err="Post "http://prometheus_postgresql_adapter:9201/write": dial tcp: lookup prometheus_postgresql_adapter on <IP_ADDRESS>:53: no such host"
My prometheus.yml file
global:
scrape_interval:     10s
evaluation_interval: 10s
scrape_configs:

job_name: prometheus
static_configs:

targets: ['node_exporter:9100']
remote_write:

url: "http://prometheus_postgresql_adapter:9201/write"
remote_read:
url: "http://prometheus_postgresql_adapter:9201/read"

How to fix it?

Looks like you are using the wrong remote read and write URLs.
Can you try changing it to http://timescale-prometheus-connector:9201?

Looks like you are using the wrong remote read and write URLs.
Can you try changing it to http://timescale-prometheus-connector:9201?

I have modified the prometheus.yml file
run and receive
...
prometheus_1                      | ts=2020-08-28T14:07:33.809Z caller=dedupe.go:112 component=remote level=warn remote_name=380fe3 url=http://timescale-prometheus-connector:9201/write msg="Failed to send batch, retrying" err="Post "http://timescale-prometheus-connector:9201/write\": dial tcp: lookup timescale-prometheus-connector on <IP_ADDRESS>:53: no such host"
The docker-compose.yml is the same as here, I modified only
volumes:
- ./prometheus.yml:/etc/prometheus/prometheus.yml:ro
because of it is Windows.
===================
prometheus.yml
global:
scrape_interval:     10s
evaluation_interval: 10s
scrape_configs:

job_name: prometheus
static_configs:

targets: ['node_exporter:9100']
remote_write:

url: "http://timescale-prometheus-connector:9201/write"
remote_read:
url: "http://timescale-prometheus-connector:9201/read"
==================

The docker compose has been fixed and tested in #477


GPU-accelerated docker container

This works, but is more a dev env for confirming that CuBLAS is working. As per this issue the problem was the size of the prompt I was passing.
Dockerfile.txt

@gjmulder thanks!

@gjmulder   what do you mean by saying "does generate load on my NVidia 1080Ti, as expected:" in this issue ?https://github.com/ggerganov/llama.cpp/issues/1283

@gjmulder  by the way ,i tried the gpu dockerfile  ,it works ,great stuff thanks.

@shengkaixuan

what do you mean by saying "does generate load on my NVidia 1080Ti, as expected:" in this issue ?ggerganov/llama.cpp#1283

"BLAS calls are only called when the tensors meet certain dimensions. In your case, you need to pass a prompt with length of 32 or more tokens to main to make it use BLAS. After it processes the prompt, it will continue text generation without BLAS since the overhead in this case is too big and there is no benefit"

@gjmulder thanks for the dockerfile, I've added it to the root of the project Dockerfile.cuda, unfortunately I don't think I can provide a pre-built image through github actions.

Credit due to @jmtatsch


Delete scroll 'instance' when stream is destroyed (and when there is a failure?)

hi,
I think it would be nice if "the scroll" was automatically deleted when then stream is destroyed (and when it fails?), as it is now any defined scroll will linger on ES until the timeout ... which (if I read the ES docs correctly is not best practice) .. possibly it could [only] be triggered with an optional flag to the destroy method.
the effective line of code in your module would be something like this:
    this.client.scroll({ method: 'DELETE', scrollId: this.scrollId })

This is possible to implement, PR is welcome, of course :)


Add test to verify schemas are cached for Neo4j PGDS

Co-authored-by: Tobias Johansson<EMAIL_ADDRESS>

This can't be rebased properly because all Neo4j tests were removed. Should be added once these tests are retrieved.

Cherry-picked these changes onto  #862


[NDTensors][Debug] Fix Typo NB should be NC

Description
Typo in the contract code, it tried to create a tuple of size NB instead of NC which will fail if NC != NB

Codecov Report

Merging #1158 (54a7eef) into main (93447d0) will decrease coverage by 31.13%.
The diff coverage is n/a.

:exclamation: Current head 54a7eef differs from pull request most recent head 9d80058. Consider uploading reports for the commit 9d80058 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##             main    #1158       +/-   ##
===========================================
- Coverage   85.28%   54.16%   -31.13%
===========================================
  Files          87       86        -1
  Lines        8368     8314       -54
===========================================
- Hits         7137     4503     -2634
- Misses       1231     3811     +2580

see 41 files with indirect coverage changes


Adding additional tags (other than 'difficult')

Had to integrate additional tags in one of my projects so I added the following tags:
- crew
- occluded
- fake
- behindGlass
- reflection
Thought this might be useful to others as well.

how to git clone the repo from this pr?

how to git clone the repo from this pr?

It hasn't been merged. Looking at some other PRs, seems like @tzutalin is kind of inactive in terms of reviewing and merging new ones.
You can probably clone the fork under my repos instead.
Hope that helps!


Redraw the area before calling transition if doing a flow redraw

Fairly minimal change to fix the problem I've seen in #1880. This draws the area of the graph outside the drawing area before it is slided in avoiding the transition flicker.

Current coverage is 64.07% (diff: 100%)

Merging #1881 into master will decrease coverage by 0.01%

@@             master      #1881   diff @@
==========================================
  Files             1          1
  Lines          4406       4407     +1
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           2824       2824
- Misses         1582       1583     +1
  Partials          0          0

Powered by Codecov. Last update 955fdd6...3b54f08


Web page not updating

I am seeing targets from 6 hours ago.
I have it set to one hour
A manual refresh of chrome fixes it but that is not how it should work
Andrew

Yes, older positions don't get removed.


Publish debian and rpm packages to Buildkite Packages

We've added support for OIDC tokens to the package create endpoint built into Buildkite Packages now, so let's try it out! This adds another step similar to the Packagecloud task which uses an OIDC token to upload the agent packages to Buildkite Packages.
Testing this is a little tricky. I think we kinda just need to try it. All the changes are isolated to steps which are allowed to soft fail. The scripts are mostly copy/pasted from the proven Packagecloud scripts and then the differences I've tested in my terminal (e.g. https://github.com/buildkite/buildkite/pull/17083#issuecomment-2161873567).
Fixes PKG-7120.

Cheers for the feedback @moskyb. shellcheck runs clean. Mind taking another look?

Looks like the deploy tools container is unhappy with me, not sure why yet

https://buildkite.com/buildkite/agent-release-edge/builds/1275#01900ad3-e46c-4f19-8301-8b9f8f62f0c5

(I was using the wrong queue!)

@sj26 buildkite/agent isn't on the allowlist for the deploy queue any more - hopefully elastic-runners works great though

@yob yeah thanks, I think I originally copy/pasted the Packagecloud steps from an older branch which was still using the deploy queue — PEBKAC.

change change, everywhere, :cheese:

https://buildkite.com/buildkite/agent-release-edge/builds/1286#01900f34-e625-4497-895d-b3315aefb8ed/197


Renaming repo: change nteract to daf-nteract

Application or Package Used
Docker image and documentation.
Describe the bug
Repo is renamed from nteract to daf-nteract and Dockerfile and documentation need to be updated.

Final name: pdnd-nteract


MaxFreeTransactionsPerBlock limit doesn't work correctly

Testnet node configured with
  MaxTransactionsPerBlock: 500
  MaxFreeTransactionsPerBlock: 20

puts 500 free transactions in a block:
https://neoscan-testnet.io/block/2d612bea91111aa38d39365766f6a2c7309a6425edd68aa0194d2cb27f7a923f
https://neoscan-testnet.io/block/e7c4b81535398864bf47a92e691dd783371f546919ffab9d4d534edfa6c6b6bb
https://neoscan-testnet.io/block/c71d375ab4732f5e01a24c86ab00ba0e282f92d21aef81e6d2e4bf7c4dbf31cc
But it shouldn't.

Related to #370.


Fix image (and other) pasting

The current paste implementation tries to override pasting image data with a fake drag event, which no longer works (requiring two attempts at a paste so it can paste the created URL normally the second time).
I've changed the logic so we mess around with the pasteboard data only when we have image data being pasted without a URL, and then everything gets passed straight through as a normal paste event with no further hackery.

Screenshots and other copied image data now paste on the first try (at 99% JPEG quality, which is much smaller than PNG and will get ruined by Facebook anyway), which resolves #393
Copied file(s) and image(s) paste correctly
Displays an error when trying to paste a directory
Remove unnecessary MMFakeDragInfo dependency

@rsms any update on this? I'd love to see this long-standing bug fixed.

I hope too @raxod502

@rsms Ping?

@rsms Is this project still maintained?

Doesn't look like it... personally, I've gone back to using messenger.com now that it supports image paste in Safari (one of the main reasons I had for using this app).
The MIT license allows forking the project as long as the original author is left credited, so if someone feels like forking this and actively maintaining it, please go ahead and do that.
(while you're at it, I also have #345 and #410 hanging for over a year, so you might want to merge those in too 😛)

My use case was being able to jump unambiguously to Messenger with a global system hotkey. That would really be quite nice, but maybe I can learn to live without it. Alternatively, maybe https://messengerfordesktop.com/ works well enough as an alternative.

@raxod502 my issue with most messenger apps floating around (that one included, IIRC), was that they use Electron, which is a massive drain on RAM and battery, as opposed to this one which uses a plain WebView.


Question: Is there an API-only way to access Config.getRealConfig()?

In a mod of mine, I'd like to do the equivalent of
if (Tools.show(mode, Config.getRealConfig().getShowCropPercentage()))

but only using the API package. Is that currently possible?

Hmm not easily right now


Provider redirect uri?

I'd like to use Google as OAuth provider and when configuring this in their API manager they ask for a Authorized redirect URI

For use with requests from a web server. This is the path in your application that users are redirected to after they have authenticated with Google. The path will be appended with the authorization code for access. Must have a protocol. Cannot contain URL fragments or relative paths. Cannot be a public IP address.

Their example: http://www.example.com/oauth2callback
What do I set this to when using superlogin?
The readme on Adding providers only mentiones that the callback URL will be generated automatically.

The error that's shown in the popup was:

That’s an error.

Error: redirect_uri_mismatch

The redirect URI in the request, https://peteruithoven-superlogin.herokuapp.com/auth/link/google/callback, does not match the ones authorized for the OAuth client. Visit https://console.developers.google.com/apis/credentials/oauthclient/1034479641750-3hr02878scadridj4jqsgh6p70s646ci.apps.googleusercontent.com?project=1034479641750 to update the authorized redirect URIs.

Using https://mywebsite.com/google/callback seems to work.

So I'm testing this in the superlogin-demo and using using https://peteruithoven-superlogin.herokuapp.com/auth/link/google/callback works when linking to an Google account from the profile page, but when trying to register with it it failes. Here it seems to want:
https://peteruithoven-superlogin.herokuapp.com/auth/google/callback. So you need to
It's possible to give Google multiple Authorized redirect URI's, so apperently you need to register both.


hope add transfer case, thank you very much

hope add transfer case

Added https://github.com/sCrypt-Inc/runelib?tab=readme-ov-file#transfer


Fix word break for long Strings

Current behaviour with long strings
console.log(document.querySelector('img'))

console.log(document.querySelector('img').src)

New one

LGTM thanks for the pr!

@samdenty99 Is this included in 2.8.4, so that we can update it in CodeSandbox, too?

@lbogdan yep


nixVersions.nix_2_8: 2.8.0 -> 2.8.1

contains fix for https://github.com/NixOS/nixpkgs/issues/169193

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
22.05 Release Notes (or backporting 21.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

diff: https://github.com/NixOS/nix/compare/2.8.0...2.8.1

Relevant for the release as well: @NixOS/nixos-release-managers

Fixes nixos-rebuild switch to me. The diff is also quite short. So let's merge it!


Fix an error when multiple selection is allowed.

It added twice the countries selected. The fix it's quite easy and I added one test

Wow, great find. I never even though of using this in a multi-select. I'd say it was never explicitly supported.

thanks!


Test-Json not matching string format "duration"

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
Let's create a very simple json schema that use the string format duration :
{
	"type": "object",
	"additionalProperties":false,
	"properties": {
		"duration": {
			"type": "string",
			"format": "duration"
		}
	}
}

the format duration was added in the json schema "draft 2019-09" current version is "draft 2020-12". The goal of the duration format is to match ISO 8601 (see: json schema documentation )
and save it in a file named schema.json
and run the following command to test:
> Test-Json -SchemaFile .\schema.json -Json '{"duration":"test123"}'

or
> Test-Json -SchemaFile .\schema.json -Json '{"duration":"PT1D"}'

the result will be True when expecting False
Expected behavior
PS> Test-Json -SchemaFile .\schema.json -Json '{"duration":"test123"}'
False
PS> Test-Json -SchemaFile .\schema.json -Json '{"duration":"PT1D"}'
False

PS> Test-Json -SchemaFile .\schema.json -Json '{"duration":"P1D"}'
True

Actual behavior
PS> Test-Json -SchemaFile .\schema.json -Json '{"duration":"test123"}'
True
PS> Test-Json -SchemaFile .\schema.json -Json '{"duration":"PT1D"}'
True

PS> Test-Json -SchemaFile .\schema.json -Json '{"duration":"P1D"}'
True

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.4.0-preview.4
PSEdition                      Core
GitCommitId                    7.4.0-preview.4
OS                             Microsoft Windows 10.0.22621
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

There seems to be an issue regarding sending " within strings to programs that are not native pwsh, they aren't getting escaped in 7.4.0-preview.4

the same behavior happens when piping a file to the command:
> Get-content .\schema.json
{
        "type": "object",
        "additionalProperties":false,
        "properties": {
                "duration": {
                        "type": "string",
                        "format": "date"
                }
        }
}
> Get-content .\data.json
{"duration":"test123"}
> Get-content .\data.json | Test-Json -SchemaFile .\schema.json
True

Can you please revise the title of this issue, as it's a bit too specific, it'll likely cause many copies of it

There seems to be an issue regarding sending " within strings to programs that are not native pwsh, they aren't getting escaped in 7.4.0-preview.4, no matter the string type you use ""...""and '"..."' will result in the same thing

@brian6932
I got 2 different behavior when using '"..."' and ""..."" (see below) so I think it's not related
PS > echo "{"duration":"test"}" | Test-Json -SchemaFile .\schema.json
Test-Json: Cannot parse the JSON.
False
Test-Json: Cannot parse the JSON.
False
PS > echo '{"duration":"test"}' | Test-Json -SchemaFile .\schema.json
True

After some more digging I have noticed that none of the json shema string format options are working in  7.4.0-preview.4
but that in 7.3.5 moste of the string format option were working (only some  option like duration, regex were not woking)

/cc @gregsdennis

In JSON Schema, format  is an annotation by default.  That means it doesn't validate.
The two solutions I see here are:

Just internally configure JsonSchema.Net to validate format by default.  This may align with expected behavior, but you wouldn't be able to turn it off.
Add a new PowerShell parameter that enables that configuration, e.g. -ValidateFormat.

The fix is pretty easy; we just need to decide which to do.

@gregsdennis Thanks! I hope WG will do a conclusion. /cc @SteveL-MSFT

7.4.0-preview.6 fixes the issue I had

Uh... what was the fix? Where was it implemented (e.g. a PR)?

Not sure, but I don't have any issues with nesting quote types anymore '{"...":"..."}' seems to parse just find now 🤷

Oh, so your problem is with quotes in general, which isn't what this issue is about.

It seemed like the same underlying cause

Is there still the need for a fix/WG discussion, or was is just a quoting issue?

Is there still the need for a fix/WG discussion, or was is just a quoting issue?

@TobiasPSP This is still definitely a problem, and the quotes issue @brian6932 was experiencing was a red herring I believe.

After some more digging I have noticed that none of the json shema string format options are working in 7.4.0-preview.4
but that in 7.3.5 moste of the string format options were working (only some option like duration or regex were not woking)

I agree with @FormindAPE that it was mostly working in 7.3.x, I tried 7.3.6 that I had lying around and date-time was working but duration was not.
Using this schema.json:
{
  "type": "object",
  "additionalProperties": false,
  "properties": {
    "duration": {
      "type": "string",
      "format": "duration"
    },
    "datetest": {
      "type": "string",
      "format": "date-time"
    }
  }
}

In PowerShell 7.3.6:
PS > Test-Json -SchemaFile .\schema.json -Json '{"datetest":"test123"}'
Test-Json: DateTimeExpected: #/datetest
False

> Test-Json -SchemaFile .\schema.json -Json '{"duration":"test123"}'
True

In PowerShell 7.4.0:
PS > Test-Json -SchemaFile .\schema.json -Json '{"datetest":"test123"}'
True

> Test-Json -SchemaFile .\schema.json -Json '{"duration":"test123"}'
True

The two solutions I see here are:

Just internally configure JsonSchema.Net to validate format by default.  This may align with expected behavior, but you wouldn't be able to turn it off.
Add a new PowerShell parameter that enables that configuration, e.g. -ValidateFormat.

The fix is pretty easy; we just need to decide which to do.

@gregsdennis My vote is for doing 1, in a patch version of PowerShell 7.4.x, since this is essentially a possible unintentional breaking change from 7.3.x. then do 2 in maybe 7.5 or something like that?

I don't mind doing both of these as a single update, but I would need someone from the PS team to confirm.

cc: @iSazonov ☝

From document referenced above:

A JSON Schema validator will ignore any format type that it does not understand.

So I think it is optional feature for the cmdlet too. In any case we need WG conclusion.

From document referenced above:

A JSON Schema validator will ignore any format type that it does not understand. - @iSazonov

To be clear, the spec requires that format is an annotation unless the implementation is configured to validate or the format-assertion vocabulary is specified by the meta-schema.
Since, generally, the default meta-schemas do not reference the format-assertion vocab, it falls to the configuration of the lib.
@pearj is arguing that previous versions of PowerShell validated format, implying that such configuration was enabled by default, and that format now not being validated is a breaking change which should be reverted (enable format validation by default).
I'm happy to have to enabled by default, if that's what PowerShell wants to do, but it needs to have a switch to disable it.  (Personally I feel this is backwards, but I understand the historical context.)

Personally I feel this is backwards, but I understand the historical context

@gregsdennis Yes you are right, going to the WG sounded like it could take a long time, so I was hoping to restore functionality in a quicker timeframe.  But if either change requires WG approval then off by default probably makes the most sense then.

Cmdlet WG has reviewed this. In #20782 we recommended adding the parameter -Option to Test-Json to allow for more optional leniency. -Option takes a string array. We feel that Test-Json should by default perform strict validation according to the Json Schema. The ability to optionally include Format in the validation process is an added benefit that should be exposed through an additional option item in  -Option.


New version: Kubernetes.minikube version 1.31.1

Pull request has been created with WinGet Automation :rocket:
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/115077&drop=dogfoodAlpha

@wingetbot run


failed to require("postcss-selector-matches")

postcss-selector-matches uses "export default".
When import it by commonjs require, Babel transforms it to an object like "{ default: ... }".
So, in features.js:
require("postcss-selector-matches")(options)
needs to be fixed to
require("postcss-selector-matches").default(options)

Please just reinstall cssnext, postcss-selector-matches should fix that in 2.0.3


Shellhomework

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
completed homework
What did you learn from the changes you have made?
I learned to make directories
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
no
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
no
How were these changes tested?
by running the commands
A reference to a related issue in your repository (if applicable)
no
Checklist

[x] I can confirm that my changes are working as intended
yes

You have unfortunately made a pull request to the wrong repo. You shouldn't be making a pull request to uoft-dsi/shell, but rather your own forked shell repo. Please read the instructions on your onboarding Assignment Submission Guide more carefully.


Add 2 function

add 2 function: setExpression, getExpression

I have several problems with this pull request, I'll discard it and close it. Please fix and resubmit a new one.

Your pull request is based on an older version of EvalEx, please always use the latest version to submit pull requests
If you add methods, follow the code formatting used in the rest of the code
Include method documentation for all new methods, parameters, exceptions and return values
For all methods other than simple getters and setters, add apropiate unit tests


cmd2 is missing from requirements in 0.5.0

fxconfig introduced cmd2 as a dependency, but it was missed from requirement.txt

Actually, cmd2 is in requirements.txt, but requirements.txt isn't used in setup.py.


ConfigProjectSource might use ConfigGlobalSource

Given /home/user/project/.vintrc.yaml and /home/user/.vintrc.yaml, ConfigProjectSource will first find the project config, but then also the user's global config, which it then uses instead.
I think a sensible fix for this is to stop once the first config has been found.
Also, ignoring the global config explicitly might make sense.
Vint could stop at the user's home directory to address this?!
Also, ConfigGlobalSource should look at ~/.config/vint/config ($XDG_CONFIG_HOME/yamllint/config) first, which would work around this.

I shipped this bugfix on v0.3.11.


[DOCS] documentation sanic ASGI

Description
Sanic framwork now support ASGI
https://sanic.readthedocs.io/en/latest/sanic/deploying.html#running-via-asgi
documentation should be updated https://fastapi.tiangolo.com/alternatives/#sanic
Thank you all !

Thanks for the report and the PR @raphaelauv !


"Youtube-DL" does not see "Live Stream" on the link to the channel (YouTube)

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.11.23. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.11.23

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):

[debug] System config: []
[debug] User config: ['-o', 'C:\\youtube-dl\\download\\%(title)s-%(id)s.%(ext)s']
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.youtube.com/channel/UCFuhijRzLpSFAP8y1NlLVZQ/videos']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2018.11.23
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.14393
[debug] exe versions: ffmpeg 4.0.2, ffprobe 4.0.2
[debug] Proxy map: {}
...
<end of log>

"Youtube-DL" does not see "Live Stream" on the link to the channel (YouTube)
https://www.youtube.com/channel/UCFuhijRzLpSFAP8y1NlLVZQ/videos
Through a direct link, this "Live Stream" is downloaded well.
https://www.youtube.com/watch?v=UvQyvVE3-ac

My command line looks like this:
youtube-dl --download-archive archive.txt "https://www.youtube.com/channel/UCFuhijRzLpSFAP8y1NlLVZQ/videos"
Please help.

You must use live URL https://www.youtube.com/channel/UCFuhijRzLpSFAP8y1NlLVZQ/live.

Thanks, it's working.


Handling Categorical Data with PySpark API

Dear Team! I’m curious if there are any plans to enhance the xgboost Pyspark API by incorporating valuable features like enable_categorical and introducing partition-based splits parameters such as max_cat_to_onehot and max_cat_threshold. If such enhancements are in the horizon, could you share any expectations regarding the version in which these features might become available? Thank you!

I would like to get it done in this version, but then I can't make the promise since I have other things on hand.

Closing in favor of https://github.com/dmlc/xgboost/issues/8398 .


chore: dependabot special config

Describe the improvement
Configure dependabot so that it would try to fetch and update published Reassure version on a daily basis. This should be done in addition to any other regular dependabot setup.
Scope of improvement
Try to update Reassure package to the latest version on a daily basis for all examples.
Suggested implementation steps

Configure dependabot.

I've done potential setup, let's see how it works in practice in the following days.


[WIP] cms line services

Coverage decreased (-0.4%) to 84.404% when pulling d81905a9c88712e48f37dfabd047251910e0f740 on line-services into 706701cd79ab07daadd6570099ce5b07f63206a2 on master.


Try to set --keep-alive to 1

Fixes #26

Codecov Report

Merging #27 (be6dd7b) into main (fbc6f51) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #27   +/-   ##
=======================================
  Coverage   74.67%   74.67%
=======================================
  Files          12       12
  Lines         462      462
=======================================
  Hits          345      345
  Misses        117      117

Flag
Coverage Δ

linux
74.67% <ø> (ø)

windows
?

Flags with carried forward coverage won't be shown. Click here to find out 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 fbc6f51...be6dd7b. Read the comment docs.


Avoid deep copy pubkey2index map in EpochContext.copy()

resolves #1027  with Proxy approach:

Share the same pubkey2Index map in EpochContext.copy()
pubkey2Index is now a getter method returning pubkey2index map proxy with get method only

I would rather go in direction of exposing method to read required data like:
class EpochCtx {
   getValidatorIndex(pubkey): ValidatorIndex;

 // add methods for other data similary
}

wdyt @wemeetagain ?

@mpetrunic Yeah I think that would suffice. A get/set on EpochContext.
I don't think we should be engineering our node defensively at every level against internal data manipulation, or it will likely lead to an unmaintainable solution. I don't think we should be copying the EpochContext in every chain.getEpochContext() call, either. (We still should figure out how to copy the epoch context, w/o deep copying the validator indices tho)
As it stands, I could feed random data to the db controller, fork choice, gossip peers, etc and corrupt the node in a thousand different evil ways, given a reference to the beacon node. Not to say that this isn't an important issue, its just something that's better tackled at some boundary, at one layer, all at once, and optionally. (fake example solution: we have a SafeApi module that safely exposes the API w/o giving access to the underlying modules. We keep the reference to a beacon node in a closure, and only return the SafeApi.)
Here, I think the danger of overwriting / corrupting the pubkey map is in the case of deep eth1 reorgs. So whatever solution we figure, it should have that in mind. The solutions I've seen for that are either 1. ignore/explode on deep eth1 reorg. 2. special case on deep eth1 reorg (eg: recompute cache) 3. hierarchical approach as here: https://github.com/protolambda/zrnt/blob/master/eth2/beacon/epochs_context.go#L95
I'm leaning toward 1 or 2 atm since this is a highly unlikely scenario.

We might use something like https://github.com/immutable-js/immutable-js if we wanted to properly handle conflicts / deep copying

Thanks for clarifying that.


PHP 7 : unable to mock return types

Have tested with both bool and object return types, Mockery does not define a return type on an object mock. When the mock is passed as an argument, the declared return type complains:
class Stub {
     public function returnType() : bool
     {
           return true;
     }
}

class TestingCode {
    public function example(Stub $stub) : bool
    {
        return $stub->returnType();
    }
}

// test
public function testTestingCodeReturnType()
{
    $stub = Mockery::mock(Stub::class);
    $testing = new TestingCode();
    $this->assertTrue($testing->example($stub));
}

PHP Fatal error:  Declaration of TestingCode::example(Stub $stub) must be compatible with TestingCode::example(Stub $stub) : bool

Have you tried this against the master branch?

Every time I make a silly mistake like this I realize I just need to drink more coffee.


Labels should update when the symbol changes

When a user starts to search and select their symbol the SIDC code tag for that symbol does not update.
Here is an example:

I would expect the symbol tag to change as the symbol changes.  The SIDC code in the text box does not match what is in the tag.

This is the same as issue:
https://github.com/Esri/military-symbol-editor-addin-wpf/issues/183

@topowright In the example above, it looks like the "Infantry : Friend" symbol has not been selected in the list box. The preview and tags should update once this is selected.
However, we could:

If only 1 symbol returned from the search, just auto-select that symbol.
-or-
Just always auto-select the first symbol after a search

Addressed in PR #314 by updating current symbol with 1st symbol returned from search

with build 19 i can search for "civil" and the tags for the first search results are displayed without selecting anything. If I then search for "infestation" tags are updated based on the first search result before I click on anything.

[ ] add to issues addressed
[ ] updated checklist test

MT build 120 and MSE build 20 and DEV have updated doc


chore: updating VerifyMembership and VerifyNonMembership methods to use Path interface

Description

Updating VerifyMembership and VerifyNonMembership client state methods to use Path interface in favour of []byte. This removes the need for marshalling and unmarshalling the key path in 03-connection and lightclient modules respectively.

closes: #1884

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Codecov Report

Merging #2736 (cfab769) into main (cce91a1) will increase coverage by 0.04%.
The diff coverage is 60.00%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2736      +/-   ##
==========================================
+ Coverage   78.74%   78.78%   +0.04%
==========================================
  Files         178      178
  Lines       12313    12281      -32
==========================================
- Hits         9696     9676      -20
+ Misses       2190     2182       -8
+ Partials      427      423       -4

Impacted Files
Coverage Δ

modules/core/02-client/legacy/v100/solomachine.go
5.47% <ø> (ø)

...dules/light-clients/06-solomachine/client_state.go
83.21% <33.33%> (-4.38%)
:arrow_down:

...odules/light-clients/07-tendermint/client_state.go
86.09% <33.33%> (-3.21%)
:arrow_down:

modules/core/03-connection/keeper/verify.go
81.01% <100.00%> (+6.66%)
:arrow_up:

I think we are missing a changelog for the addition of the generic membership verification funcs.
Should we add one now or later?

I think we are missing a changelog for the addition of the generic membership verification funcs.

Either is fine by me


feat(#22): enable integration tests

Enable packs integration tests.
Closes: #22.

History:

feat(#22): repair one of the integration tests
feat(#22): repair all of the rest integration tests
feat(#22): fix all qulice suggestions

PR-Codex overview
This PR focuses on updating dependencies and making changes to the codebase for better readability and maintainability.
Detailed summary

Updated the version of jeo-maven-plugin dependency from 0.2.11 to 0.2.12 in pom.xml.
Modified the Opcode class to include a boolean flag for counting opcodes.
Updated the JeoDecompiler class to use the new Opcode constructor with the counting flag.
Added a new constructor to DecompilerMachine to accept arguments for the decompiler.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 3min)


Only two namespaces inspected

I am trying Polaris in a cluster with 11 namespaces, but only kube-syste, and polaris are shown in the report.
Any clue?
Thanks

Hey @marcecaro, thanks for checking in. Right now Polaris only validates deployments, so if your other namespaces don't have deployment resources, they won't show up. If they do have deployments, is Polaris logging anything useful?

Yes it is really useful for big solutions and it would help to keep metrics.
We have several teams that are still using replica controllers.  And some of them Daemons and ReplicaSets.
Are you going to cover those resources in the future?
Thanks

Yep, it's on our roadmap, we hope to get those kinds of resources included shortly. I've created an issue that can track progress on that feature here: https://github.com/reactiveops/polaris/issues/107. Thanks for letting us know it would be useful!

Thank you very much, and really nice work!


Allow for Custom Gradient Angle

Simple slider control to customize the gradient angle would be great.

I can look into this if required.


Thank you & Question(s)

Hello @tom-auger and thank you for publishing this.
I'm currently investigating the suitability of the library for my cross platform .NET standard based apps. It seems like a good fit, but there are a couple of questions I can't answer myself.
If you don't mind:

Are there any plans on adding async support (Task<T> instead of T and Task instead of void returns)  or better yet default everything to async ?
Are there any plans on adding deterministic random numbers support like libhydrogen has or at least allowing the plugging of custom RNG engines to support this scenario ?
Would you use the library in production yourself ? :)

Regards.

Hi,
Thanks for your interest!

I don't plan to add async methods, unless there is a compelling use case for them. The current API is best suited for encrypting small amounts of data in memory, so typically will not be a long running operation. To process large amounts of data it's best to encrypt it in chunks, so implementing something like the CryptoStream with async methods, on top of the SecretBox API would probably be the way to go.

I didn't add an API for random number generation as SecretBox simply uses the RNGCryptoServiceProvider for generating the key and nonce. The System.Random class would be best for deterministic keys, however I'm less keen on making the random number generation customizable as one of the main goals is to make the library hard to mis-use, and it could be hard to spot if your code was using a SecretBox with a deterministic RNG. If you're thinking of doing this for testing then instead I would recommend creating a mock SecretBox that implements ISecretBox and returns deterministic results that way.

Yes, to the same degree that I would use libhydrogen! I'm not aware of other projects using it in production, but I would very much it to be. Bear in mind the code hasn't been audited like some other mainstream crypto libraries, so consider whether it meets your security requirements.

Regards,
Tom

Thanks @tom-auger . Closing this.


feat: added support in env setup glue for fetching device details dynamically

:tada: This PR is included in version 1.2.0-dev.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Branch delete is not informative

Select a stash project with several branches.
Configure stash2slack
Delete from stash (e.g. git push origin --delete [somebranch])

Expected: an indication that "somebranch" was deleted
Actual: "Push on [repo name] by [author]. See commit list" with no indication of branch delete.

if the branch is deleted via git push origin --delete [branchname] then the following message is generated: {refname} deleted by {username} <{email_address}>.

note that if you delete a branch via the stash web interface no slack message is generated because no push has happened (plugin only subscribed for push and pullreq events)

Thanks. Very fast fix!


std-lib wrapper around BHSH (block header hash) opcode

@nfurfaro is this something you can take on?

Yes, assigned to me now.


[AutoPR eventgrid/resource-manager] EventGrid: README.md changes: Updated default tag for global settings and updates to multi-api settings.

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3928

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/2902


Jinja2 2.9.x broken

Looks like Jinja 2.9.x introduced some issue with the groupby filter used in the configurations task:
TASK [network-interfaces : configurations] *************************************
 [WARNING]: Exception in safe_eval() on expr: [_GroupTuple(grouper=u'eth0', ..., "msg": "AnsibleUndefinedVariable: 'unicode object' has no attribute 'device'"}

Tested with Ansible <IP_ADDRESS> and Jinja2 2.9.3.

https://github.com/ansible/ansible/pull/20014

I confirm that it is fixed, ansible now specifies jinja2 < 2.9.

@NicolasLM In master branch or in all current stable releases?

https://github.com/ansible/ansible/compare/v<IP_ADDRESS>-1...v<IP_ADDRESS>-1
https://github.com/ansible/ansible/compare/v<IP_ADDRESS>-1...v<IP_ADDRESS>-1

Thanks for this issue. I was able to fix it on my machine by removing the pip-installed version of Jinja2, so that Ubuntu's version is used. Didn't know what too look for, before i found this issue.

Ok, so this is not an issue of this module, but Ansible itself?
Can we close it here?

Yes, I would say so

Ok, I close the issue.


Does it support arm?

I have this: EM3566
Is it possible to use your library with my controller?

Hi, it looks like that board has an ARMv8 CPU and runs Linux, so .NET should support it!


Add ac742739cae0d411dfcb78ae99a7baee.exe_

Test file for https://github.com/fireeye/capa-rules/issues/185 - It's a program that just returns 0, which I then packaged up using IExpress

thank you!


Typescript & Babel

For me it sounds confusing using Babel on a typescript codebase (i.e both add js shims into the generated code).
Could you please tell why you made this choice? What benefits does Babel add for typescript development?
Please don't see it as a criticism, I'm genuinely curious about this.

Hi @gerardolima, babel is already used as a transpiler in nextjs, also I found a nice article showing some benefits of using TypeScript with Babel here: https://iamturns.com/typescript-babel/

Hi @gerardolima ,
Babel is great tool for compiling javascript.I understand why you’re confused like “why do i use babel with typescript?” Because i also have these kinds of confusions but i have great answers for them.

You can easy manage pollyfill with Babel. (https://babeljs.io/docs/en/babel-preset-env)
You can use babel plugins (https://babeljs.io/docs/en/plugins)
You can take part in the robust babel ecosystem (typechecking, error checking etc...)

Also you can read more,
https://dev.to/saltyshiomix/why-babel-with-typescript-56l5
https://blogs.msdn.microsoft.com/typescript/2018/08/27/typescript-and-babel-7/
http://artsy.github.io/blog/2017/11/27/Babel-7-and-TypeScript/

hey, @omeraplak, Typescript also injects polyfills, introduces its runtime, including module handling,  and produces code targeted to specific ECMA Script versions; how those relate to Babel ones? About type checking, I really can't see any improvement Babel can bring over TypeScript ... but anyway, thank you for your thoughts and for the articles


fixes

dokumentation
Added GreenSurvivors as authors
renamed blockRecorder package to just recorder to reflect that entities get recorded as well. Also, it makes codacy happy.
some minor fixes
added GameEnded trigger that allways fires if a game has come to an end (not only if it is in its endphase)

Ich glaube wir können das mal auf Dev mergen.

 Languages werden nicht alle richtig erkannt


Solves issue 1689

Fixes #1689

My code sample is in :

[ ] CPP
[✓] Java

Screenshot of working code sample  :

Checklist

[✓] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[✓] My code sample is formatted and works correctly

Merge conflicts, rebasing should help


Remake dark mode

https://1drv.ms/u/s!AiGYPzCg7KuggcxouaDm3DqznQZfig
Drag on C:\Users\YouUserFolder\.spicetify\Themes\Fluent and replace

What?

I dont know do that

Hi, yes, it would be much better as a pull request. I am closing this as you have made a separate issue for the same thing and I have replied to you there with instructions and information. https://github.com/williamckha/spicetify-fluent/issues/58


Sonic the Anonymous Hedgehog: Watch ezzeefonts.com

Sonic the Anonymous Hedgehog requests the watch of the watch_keyword ezzeefonts\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

ezzeefonts\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Ryan M in Charcoal HQ


How to update stop_sequence?

I'm trying to update the stop_sequence of a trip, but can't quite figure out how to reach in there. Any ideas?
for trip in feed.trips():
 seq = trip.stop_sequence()

 #remove stops with no pickup and dropoff allowed from the sequence
 seq = [s for s in seq if not (s['pickup_type'] == "1" and s['drop_off_type'] == "1")]

 #does not work
 trip.set('stop_sequence', seq)

 #does not work either
 trip.add_child(seq)

 #check for remaining stops
 for s in trip.stop_sequence():
    if s['pickup_type'] == "1" and s['drop_off_type'] == "1":
        print s['stop_id']

hi @laidig
Ah, good catch. It's not properly documented, and I just noticed I don't have methods for removing/reassigning associations. The stop_times associated with a trip are available in trip.children(), and trip.stop_sequence() are the children sorted by stop_sequence. The underlying attribute is trip._children and you should be able to manipulate this set, or assign it to your filtered list. This may be buggy but worth a shot.
>>> trip
<Trip BFC2>
>>> trip.children()
set([<StopTime None>, <StopTime None>])
>>> trip.stop_sequence() # children sorted by stop_sequence
[<StopTime None>, <StopTime None>]
>>> seq = [i for i in trip.stop_sequence() if i.get('stop_sequence') == '1']
>>> seq
[<StopTime None>]
>>> trip._children = set(seq)
>>> trip.stop_sequence()
[<StopTime None>]

Looks like it. Thanks!


Adaptive Card is only supported to Incoming Webhook O365 Connector type and not in any other O365 Connector types.

Add a Note after following note:

Adaptive Card is only supported to Incoming Webhook O365 Connector type and not in any other O365 Connector types.

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

ID: 3fb4394d-25b1-8752-3996-fe3a2934732d
Version Independent ID: 6d391381-89d0-117c-27cc-00043bd26a36
Content: Types of cards - Teams
Content Source: msteams-platform/task-modules-and-cards/cards/cards-reference.md
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: lajanuar

@surbhigupta12 - Could you please add below note here.
Adaptive Card is only supported to Incoming Webhook O365 Connector type and not in any other O365 Connector types.


can't get manual.py to work

hey. Newb here.
I've installed all the prereqs, except Requests[security] my macOS env reports that doesn't exist.
(sickbeard_mp4_automator) andy@MozzersMini sickbeard_mp4_automator-master % pip install requests[security]
zsh: no matches found: requests[security]

The error I get on running manual is to do with guessit. Can anyone help me?
Thanks
Manual processor started.
Python 64-bit 3.7.7 (default, Mar 10 2020, 15:43:33)
[Clang 11.0.0 (clang-1<IP_ADDRESS>)].
Guessit version: 1.0.3.
/Users/andy/sickbeard_mp4_automator/sickbeard_mp4_automator/bin/python
Loading config file /Users/andy/sickbeard_mp4_automator/config/autoProcess.ini.
Enter path to file:
#: /Users/andy/Downloads/Torrents/Complete/\ Dune1984Extended1080Pblurayremuxavcdtshdma51Rarbg.mp4
Traceback (most recent call last):
  File "manual.py", line 384, in <module>
    main()
  File "manual.py", line 372, in main
    tagdata = getInfo(path, silent=silent, tmdbid=args.get('tmdbid'), tvdbid=args.get('tvdbid'), imdbid=args.get('imdbid'), season=args.get('season'), episode=args.get('episode'))
  File "manual.py", line 112, in getInfo
    tagdata = guessInfo(fileName, tvdbid=tvdbid, tmdbid=tmdbid, imdbid=imdbid, season=season, episode=episode)
  File "manual.py", line 157, in guessInfo
    guess = guessit.guessit(fileName)
AttributeError: module 'guessit' has no attribute 'guessit'

You’re using a very old version of guessit
Guessit version: 1.0.3.
Update to the latest version

not sure how that happened, this is a brand new install. Ho hum.
thanks for the help. All working now. :-)


Operator Framework

CoreOS released the Operator Framework.
We should explore what the benefits of using it are, and whether it makes sense to base our work on it.

There's also https://github.com/giantswarm/operatorkit :)

Looked into both, and the OF seems to solve more of our problems.
It has a CLI tool that:

creates a project with the expected dir layout
includes/replaces the code generator, so we don't have to use it directly anymore (yes!)

It has the OLM, which should enable dependency management across operators, among other things.
OTOH, operatorkit is more opinionated about the structure of the controller, and it provides much more functinality in its libraries than what operator-sdk currently does. But I think this will probably change in the future as operator-sdk matures.

Closing this, for now it seems to me that OF is the best of the two, but it's definitely too early to think of switching.

Closed again for now, we'll revisit once the project is more mature.


Add guard for multiple node unmarshal

Fixes unmarshalling a node with multiple left and right children
Description
Add a check for unmarshalling a node when the input byte array with multiple left or right childs.
Mentioned in Here
Changes:

Add left and right flag to protect there only at most 1 left or right child when marshalling.
Add unit test for this part.

Types of changes
Leave only items that describe your changes, remove the rest. Remove this line too.

Bugfix (non-breaking change which fixes an issue)

Testing
Requires testing: Yes
Did you write tests??: Yes
Documentation
If this requires a documentation update, did you add one? No

@jelilat Thanks for your point out, I have update the TODO and fixed the spell error. I think I need to install a spell check to help me.

@jelilat @joshklop I have squashed all my commits, please have a re-review. Thanks

@IronGauntlets, can you force merge this? @jelilat wont be able to review soon probably

@liusanchuan Please rebase the branch and feel free to merge

It's my second PR succcessfully merged :)
Thanks for your guys, I've leaned spell checking, rebase and squash, golang effieiency optimization.
Juno is a amazing place, I just contribute several lines of code, but get so much tutorial for yours. For each respose, thank you again🥇 !
Hope I can do better and contribute more!


500 Internal Server Error

Hi Jess!
How's it going?
This theme is legit!
But here's a minor catch I found. So I recently upgraded to Yosemite and then my localhost stopped working. I get a 500 Internal Service Error when I load my local server. I couldn't figure out what was causeing this error. First, it thought it might be something in my httpd.config or username.conf, but nope, that wasn't it. Then I check the .htaccess tried commenting from line 68-71, and it worked! The 500 Error went away. so it's all good now.
    # RewriteEngine on
    # RewriteCond %{REQUEST_FILENAME} !-d
    # RewriteCond %{REQUEST_FILENAME}\.html -f
    # RewriteRule ^(.*)$ $1.html

So yeah, I don't think it's a major issue but I thought I'd share it anway.

closing as this is archived but thanks and sorry!!!


chore(release): publish branch main

:robot: I have created a release beep boop
generator-ui5-react: 1.3.0
1.3.0 (2024-12-14)
Features

add support for httpyac as means of using http scripts in VSCode (6df83bf)

Bug Fixes

make prettier work with decorators by specifying importOrderParsers (2f4d028)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/cpro-js/generator-ui5-react/releases/tag/generator-ui5-react-v1.3.0 :sunflower:


Can't get currentNode for sidebar

Hi,
I have an issue for get the current node of the children of a children.
Let me explain, I have a sidenav and a topnav on my website. The topnav is just for the NavNode where parentKey="RootNode". The topnav is my principal nav and the sidebar is used to display their children when the user is in the current node of one of the tabs in the topbar.
Like this :

Now my issue is : I cant get the current node when I want to edit my table in another page with the project Id passed on URL like this :

but I can go to this controller without an Id :

I found the solution to Create or Edit with an Id when I remove the line :
if (node.EqualsNode(Model.CurrentNode))
But the issue is my sidebar display all childrens of xml navigation file like this :

Those are the Shared files :
Topbar.cshtml  : (I post th topbar file on pastebin because indentation is bad when i paste in there.)
http://pastebin.com/NiL5sBZw
Sidebar.cshtml :
`@if (Model.HasVisibleChildren(Model.RootNode))
{
foreach (var node in Model.RootNode.Children)
{
    if (!Model.ShouldAllowView(node)) { continue; }
    if (node.EqualsNode(Model.CurrentNode))
    {
        if (!Model.HasVisibleChildren(node))
        {
            @:

                    @Model.AdjustText(node)

        }
        else
        {
            @:

                @Model.UpdateTempNode(node)
                @Html.Partial("Partials/NodeSideNavPartial", Model)

        }
    }
}

}`
NodeSideNavPartial.cshtml :
`@if ((Model.TempNode != null) && (Model.HasVisibleChildren(Model.TempNode)))
{
@foreach (var childNode in Model.TempNode.Children)
{
if (!Model.ShouldAllowView(childNode)) { continue; }
    if (!Model.HasVisibleChildren(childNode))
    {
        @:

                <i class="<EMAIL_ADDRESS>

    }
    else
    {

                <i class="<EMAIL_ADDRESS>
            @Model.UpdateTempNode(childNode)
            @Html.Partial("Partials/NodeSideNavPartial", Model) @* recursion *@

    }
}

}`
And the navigation.xml :
`

<NavNode key="Saisie" parentKey="RootNode" controller="Saisie" action="TableauDeBord" text="Saisie">
  <Children>
    <NavNode key="TableauDeBord" parentKey="Saisie" controller="Saisie" action="TableauDeBord" text="Tableau de bord" iconCssClass="fa fa-tachometer">
      <Children />
    </NavNode>
    <NavNode key="Kanban" parentKey="Saisie" controller="KanbanBoard" action="Index" text="kanban" iconCssClass="fa fa-share-alt">
      <Children />
    </NavNode>
  </Children>
</NavNode>

<NavNode key="Administration" parentKey="RootNode" controller="Projet" action="Index" text="Administration">
  <Children>
    <NavNode key="Projets" parentKey="Administration" controller="Projet" action="Index" text="Projets" iconCssClass="fa fa-file-powerpoint-o">
      <Children>
        <NavNode key="Fiche" parentKey="Projets" controller="Projet" action="CreateEdit" text="CRUD" preservedRouteParameters="id" componentVisibility="false">
          <Children />
        </NavNode>
      </Children>
    </NavNode>
    <Children />
    <NavNode key="Utilisateurs" parentKey="Administration" controller="Utilisateur" action="Index" text="Utilisateurs" iconCssClass="fa fa-tachometer">
      <Children />
    </NavNode>
  </Children>
</NavNode>

<NavNode key="Déploiement" parentKey="RootNode" controller="Deploiement" action="Alive" text="Déploiement">
  <Children>
    <NavNode key="Alive" parentKey="Déploiement" controller="Deploiement" action="Alive" text="Alive" iconCssClass="fa fa-tachometer">
      <Children />
    </NavNode>
  </Children>
</NavNode>

`
Now my question is : How can i display just the children of the current NavNode like "Adminisitration" in previous exemples ?
Thanx for helping and grieve if you do not understand me ^^

the CurrentNode property of the viewmodel is intended to be determined from the current url, it could be null sometimes if it can't determine which navigation node should have that url. Are you saying it is null when it should be able to match?
are you saying that it works if the url matches a controller/action but it is null if the url has query string paramters like controller/action?param=foo
?
it is possible to specify a startingNodeKey when invoking the navigation, ie the key of a node from the navigation.xml
@await Component.InvokeAsync("Navigation", new { viewName = "ChildTree", filterName = NamedNavigationFilters.ChildTree, startingNodeKey = "Administration" })

it is also possible to filter out nodes from a particular view using a filter name as in this example:
<NavNode key="NewSite"
                 parentKey="SiteList"
                 controller="SiteAdmin"
                 action="NewSite"
                 text="New Site"
                 componentVisibility="breadcrumbs,childtree"
                 viewRoles="ServerAdmins">
          <Children />
        </NavNode>

if componentVisibility is blank then it will not be filtered from any view, but if you populate componentVisibility then it will only be visible when the filterName matches one of the items in that setting, so for the example above the node will be filtered unless the the filterName is either breadcrumbs or childtree, so this item will only be visible in breadcrumbs or childtree, it own't be visible in topnav for example. I tend to use filter names that are the same as the viewName, but there is no requirement for filtername to be the same as viewname

I'm looking into the issue about CurrentNode being null, I think there are some cases where it is not resolving that as it should
I would think that if your goal is to show children nodes of current node in the sidebar then the sidebar view should be like:
if(Model.CurrentNode != null)
{
    foreach (var node in Model.CurrentNode.Children)
    {
      ...
     }
}

instead of using RootNode

but again for this to work CurrentNode must not be null

can you try dotnet restore --no-cache to get the new version 1.0.2-rc20160822
and see if it solves the problem where CurrentNode is null

Okay, i try your answer and now i have the current node when i'm in CreateController.
I will explain you again because i think i hurt explain my issue, then you need to look my screen for a better comprehension.

The red block is the topbar, in this topbar I have his children on green block which are displayed in the sidebar. In these children i can have children too, but they are not displayed in the sidebar because they are dependant of their father green block but i always want to display all children green block of the red block and display the father of blue block like the current tab of sidebar.
This is my goal, i hope my explain is better and you have an idea to help me otherwise no worries and thank you.

I think if you could make a sample project in a public git repo that demonstrates the problem, then I will understand it better and I could try to solve it

I'm sorry but it's a private project for a client, I will try to make a video of my issues in a few moment, maybe is can be better.

I don't mean for you to use your client's project, I mean try to make a small new project that illustrates the problem. that way I can see the actual code and I can try to fix it

Okay no problem i'll do this

maybe you could fork this project and create the problem in the demo app

I have done the project with your exemple, you can download in this url : https://github.com/alimtunc/CloudstribeBugTest/tree/master/cloudscribe.Web.Navigation-master
When you start the project, for test the bug you need to follow these instruction :

Click on administration Tab
Click on project tab
Click on "go to edit" button

Second bug :

Click on "utilisateur" tab and the tab of sidebar disappear

Sorry for the design of this project but it's not the principal problem, we can see carefully the issue. Maybe this project can show you my issues and thank for help.
I'm not sur I answer you this night but done what you have to do and I will see tomorrow.  Good luck

your project is missing some files, project.json and other files are missing but I copied them from my project.
but also your custom navigation views are missing, isn't there supposed to be a custom sideNav.cshtml and TopNav.cshtml?
can you add those?

I did get the null exception on /Projet/Edit but I fixed that by editing the navigation.xml
I changed this:
<NavNode key="Fiche" parentKey="Projets" controller="Projet" action="Create" text="CRUD" preservedRouteParameters="id" componentVisibility="false">
        </NavNode>

to this:
<NavNode key="Fiche" parentKey="Projets" controller="Projet" action="Edit" text="CRUD" preservedRouteParameters="id" componentVisibility="ChildTree">
        </NavNode>

the if an action is not included in the menu then there is no currentNode for it to find so you need a node for the Edit action, then filter it out of what views you don't want to show it in

for any controller/action to be found as the CurrentNode it must exist in the navigation.xml
I have a similar situation in my main cloudscribe project where certain nodes for things like new site, edit site etc I don't want in the top navigation so I filter them for only childnav or breadcrumbs, you may get some ideas looking at the navigation.xml file I'm using there

Yes sorry my bad and the custom Topnav and Sidenav are the default name, I just put my code into them.
I see the navigation xml and again my bad, I forgot to replace the "action='Create'" to "action='Edit'" but yes now you can see the real issue to display the sidebar, I will try with your navigation.xml file now.

but TopNav and SideNav don't exist in your repository so I don't have your code for them and cannot see the problem as you do. I see you added me to the repo so I could commit, but for some reason I get an error when I try to clone the repo so I just download the zip

Hi, I have found a solution to my issue and it work good ! And I seen the file .gitignore have add these ligne :
src/cloudscribe.Web.Navigation/project.lock.json src/NavigationDemo.Web/wwwroot/lib* src/NavigationDemo.Web/wwwroot/js/* src/NavigationDemo.Web/wwwroot/css/site.min.css src/NavigationDemo.Web/project.lock.json test/cloudscribe.Web.Navigation.Test/project.lock.json src/cloudscribe.Web.SiteMap/project.lock.json src/cloudscribe.Web.SiteMap.FromNavigation/project.lock.json *.json src/cloudscribe.Web.SiteMap.FromNavigation/project.lock.json
Why ? I don't know.. anyway, for explain my solution I modified the SideNav.cshtml  to :
http://pastebin.com/haJhKt7G
The TopNav.cshtml to :
http://pastebin.com/DdKqtGw6
And the SideNavPartial.cshtml to :
http://pastebin.com/GNVyLgvY
I can manage specific cases now because you fix the bug to know where we are if we are a children from a children and it's great :)
## Thank's for your help again and for being patient !

ok, glad it is working for you, so this issue can be closed?

Yes, I close, good continuation ;)


behavior changed after update from 0.7.3 to 0.8.1

here is the dom structure:
<div>
  <Content data-tip="content">
    <SubContent>
      <SubElement></SubElement>
    </SubContent>
  </Content>
  <ReactTooltip place="bottom" effect="solid" />
</div>

With 0.7.3, when I mouse over <Content> area or its subelements, the tooltip component will positioned relative to <Content> element.
But with 0.8.1, the tooltip will positioned relative to the current mouseover target. e.g. If I mouse over <SubContent> element, the tooltip will moved to the bottom of <SubContent>.
Just wonder if I have some misunderstanding about default behavior.

Looks more like a bug of the latest react-tooltip, I will check this later, thanks for report.


Add ability to filter by Elm version

Elm seems to change quite rapidly from version to version (for example 0.17 -> 0.18). It would be great to be able to filter examples by Elm version, then one can lookup how other people uses the latest versions. Especially for beginners, this could be of great help to be able to find quickly recent examples, as the tutorials are not always complete or up to date.

I like this idea, but I wonder if we would end up with out of date metadata. For example, someone adds a project when they first write it, say in 0.18. But will they remember to come back and update this repo when 0.19 comes out months later and they update their project?

Indeed, that would be a problem for people looking for old examples and then stumbling upon updated versions.
But I think the primary use case would be beginners, and they should be mostly interested by examples with the most recent version. Instead of providing a search field, a simple checkbox "Show me examples with the latest Elm version" would suffice?

one declares a project with version Elm 0.18
the website can filter all projects from the latest version, namely 0.18
later, Elm 0.19 is released and the website is updated to mirror that
projects declared as 0.18 no longer appear as "recent", unless manually updated.

Sure, we would miss some update but at least, there is less false positive results.
Next step of course would be to retrieve elm-version from the elm-package.json :)


add tests for all functions and components

Add tests for all functions
Create unit tests for all React components

She is already commit the fix linter error can you check it again please
On Wed, 22 Feb 2023, 9:06 pm Nemwel Boniface, @.***>
wrote:

@.**** requested changes on this pull request.
Hello @huzaifakhan0308 https://github.com/huzaifakhan0308 👋 ,
I am Nemwel your reviewer for this PR 🧔♂
While you made a great effort in this project, unfortunately, I cannot
proceed to review your code.
✖ Invalid Code Review Request ✖
As this is a pair programming task, kindly ensure that your partner Daniel
Edo makes contribution to the project.
Your Code Review Request will be marked as invalid in your Dashboard, so
please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
Please, remember to tag me @.***
https://github.com/Nemwel-Boniface* in your question so I can receive
the notification.
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
Invalid Code Review Request does not count into the code reviews limit.
—
Reply to this email directly, view it on GitHub
https://github.com/huzaifakhan0308/math-magicians/pull/7#pullrequestreview-1309594561,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3I36YHBYG3YN76FURCCV4DWYY2PHANCNFSM6AAAAAAVEPTAVU
.
You are receiving this because you were mentioned.Message ID:
@.***>


Add TapOutsideConfiguration widget

This PR adds TapOutsideConfiguration where you can define a custom default logic for onTapOutside.
TapOutsideConfiguration(
  behavior: AlwaysUnfocusTapOutsideBehavior(),
  // behavior: const NeverUnfocusTapOutsideBehavior(),
  // behavior: const CustomTapOutsideBehavior(),
  child: Column(
    children: [
      // This TextField will use onTapOutside from TapOutsideConfiguration
      TextField(),
      // Of course you can still define onTapOutside
      TextField(onTapOutside: (event) => print('Tapped outside')),
    ],
  ),
)

Custom logic can be define like this:
class CustomTapOutsideBehavior extends TapOutsideBehavior {
  const CustomTapOutsideBehavior();

  @override
  void defaultOnTapOutside(PointerDownEvent event, FocusNode focusNode) {
    // any custom logic here
  }
}

This PR implements also two simple AlwaysUnfocusTapOutsideBehavior and NeverUnfocusTapOutsideBehavior.
Resolves #150123
Pre-launch Checklist

[X] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[X] I read the Tree Hygiene wiki page, which explains my responsibilities.
[X] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[X] I signed the CLA.
[X] I listed at least one issue that this PR fixes in the description above.
[X] I updated/added relevant documentation (doc comments with ///).
[X] I added new tests to check the change I am making, or this PR is test-exempt.
[X] I followed the breaking change policy and added Data Driven Fixes where supported.
[X] All existing and new tests are passing.

I wonder if this could be handled by turning the tap outside behavior that happens inside of EditableText into an Intent. That would allow overriding of the action, allow passing of the FocusNode and event to the action, and not introduce yet another InheritedWidget that people need to discover.
It would be implemented similar to DeleteCharacterIntent/_DeleteTextAction, which is an overridable action on the text field in editable_text.dart.
Something like this:
class EditableTextTapOutsideIntent extends Intent {
	const EditableTextTapOutsideIntent({required this.focusNode, this.pointerDownEvent});

	final FocusNode focusNode;
	final PointerDownEvent pointerDownEvent;
}

And then, in editable_text.dart, you would create an action and make it overridable:
  late final Map<Type, Action<Intent>> _actions = <Type, Action<Intent>>{ // already exists
	// ...
  	EditableTextTapOutsideIntent: _makeOverridable(_EditableTextTapOutsideAction<EditableTextTapOutsideIntent>()),
	// ...
  };

class _EditableTextTapOutsideAction<T extends EditableTextTapOutsideIntent> extends ContextAction<T> {
	const _EditableTextTapOutsideAction();

  @override
  Object? invoke(T intent, [BuildContext? context]) {
    /// The focus dropping behavior is only present on desktop platforms
    /// and mobile browsers.
    switch (defaultTargetPlatform) {
      case TargetPlatform.android:
      case TargetPlatform.iOS:
      case TargetPlatform.fuchsia:
        // On mobile platforms, we don't unfocus on touch events unless they're
        // in the web browser, but we do unfocus for all other kinds of events.
        switch (intent.pointerDownEvent.kind) {
          case ui.PointerDeviceKind.touch:
            if (kIsWeb) {
              intentl.focusNode.unfocus();
            }
          case ui.PointerDeviceKind.mouse:
          case ui.PointerDeviceKind.stylus:
          case ui.PointerDeviceKind.invertedStylus:
          case ui.PointerDeviceKind.unknown:
            intent.focusNode.unfocus();
          case ui.PointerDeviceKind.trackpad:
            throw UnimplementedError(
                'Unexpected pointer down event for trackpad');
        }
      case TargetPlatform.linux:
      case TargetPlatform.macOS:
      case TargetPlatform.windows:
        intent.focusNode.unfocus();
    }
  }
}

And finally invoke it in EditableTextState:
  void _defaultOnTapOutside(PointerDownEvent event) {
	Actions.maybeInvoke(context, EditableTextTapOutsideIntent(event, widget.focusNode));
  }

To override it, you just define an Actions widget above the EditableText somewhere in the widget tree that maps EditableTextTapOutsideIntent to your custom action.
This way it would use an existing mechanism that is already part of EditableText, allows for overriding, and doesn't create a separate new mechanism that people need to discover and understand.
(fyi, I haven't tried compiling or running the above, it's just hot off the top of my keyboard)

(triage) @kubatatami Do you still have plans to follow up on the feedback given above?

@goderbauer Yes. I will try to follow up on the feedback early next week.

@gspencergoog @goderbauer I changed PR to use Intent and Actions. I used code prepared by @gspencergoog with minor changes. The only major changes I had to do were:

change order of TextFieldTapRegion and Actions
add a Builder widget to have a context that I can use to invoke Actions.invoke
use Actions.invoke instead of Actions.maybeInvoke because we want to be sure that some action is always invoked (default or custom)

Unsubscribe
On Fri, Sep 20, 2024, 1:36 PM Greg Spencer @.***> wrote:

@.**** commented on this pull request.
In packages/flutter/lib/src/widgets/text_editing_intents.dart
https://github.com/flutter/flutter/pull/150125#discussion_r1768949786:

@@ -387,3 +389,15 @@ class TransposeCharactersIntent extends Intent {
/// Creates a [TransposeCharactersIntent].
const TransposeCharactersIntent();
}

+/// An [Intent] that represents a tap outside the field.
+class EditableTextTapOutsideIntent extends Intent {

/// Creates a [EditableTextTapOutsideIntent].
const EditableTextTapOutsideIntent({required this.focusNode, required this.pointerDownEvent});

/// The [FocusNode] that this [Intent]'s action should perform on.

⬇ Suggested change

/// The [FocusNode] that this [Intent]'s action should perform on.

/// The [FocusNode] that this [Intent]'s action should be performed on.

In packages/flutter/lib/src/widgets/text_editing_intents.dart
https://github.com/flutter/flutter/pull/150125#discussion_r1768962906:

@@ -387,3 +389,15 @@ class TransposeCharactersIntent extends Intent {
/// Creates a [TransposeCharactersIntent].
const TransposeCharactersIntent();
}

+/// An [Intent] that represents a tap outside the field.
Please add more description here: when is it invoked, why someone might
want to invoke it or override it themselves, and a "See Also:" section with
a link to Action.overridable to explain how those work. This is the
public API doc that will be on the API docs site, so it is what will
explain this feature.
—
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/pull/150125#pullrequestreview-2318839123,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR7EZHVK4AHNZF7BOSPCC3LZXRMLRAVCNFSM6AAAAABJG2X7TCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGMJYHAZTSMJSGM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi @kubatatami , do you still plan to come back to this pr?

Hi @kubatatami , do you still plan to come back to this pr?

Yes. I will improve docs this week.

@gspencergoog @chunhtai More description added. Please let me know if there is anything else to improve.

A few nits here, but there's one thing that I think we should do before merging this. In the docs for onTapOutside, can you mention the new EditableTextTapOutsideIntent? I want to make sure both APIs are discoverable and avoid confusing users that might not realize setting onTapOutside will prevent triggering EditableTextTapOutsideIntent.

@justinmc Done. Please let me know if there is anything else to improve.


Compiled assets not used in production

There is a problem with the configuration where the assets url are not using the compiled/manifested versioning

I think #151 introduce a regression.

This is fixed in 91cc6e6dc8e0039e6f36f79d2207363f4359c593


Feature/map generator

Adding map_generator.py and test_map_generator.py to develop - some demo oriented code still in map_generator.py but commented out because superfluous to the program functioning.
requirements-dev.txt will also now contain the rest of the added libraries
file structure has been changed to deal with issue importing modules when folders were nested

Codecov Report

Merging #13 into master will decrease coverage by 2.35%.
The diff coverage is 95.91%.

@@            Coverage Diff             @@
##           master      #13      +/-   ##
==========================================
- Coverage     100%   97.64%   -2.36%
==========================================
  Files           3        4       +1
  Lines          36       85      +49
  Branches        1        8       +7
==========================================
+ Hits           36       83      +47
- Misses          0        1       +1
- Partials        0        1       +1

Impacted Files
Coverage Δ

dork/map_generator.py
95.91% <95.91%> (ø)

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 e51e5c6...a5af1a7. Read the comment docs.


Run scheduled tasks only on main repository

only on main repository

I'm not sure I follow what is mean by "main repository". Do you mean on non-forked repositories? So the proposal here is to make the Update * jobs only run on paketo-buildpacks/*?
You linked to this job as an example. I see it failing, do you want this to effectively no-op for forks?
Thanks

We talked about this change and the feeling is that we don't want to accept it. There are a couple of concerns 1.) It's possible that someone might want to fork a project and have the update jobs run. This change would prevent that. 2.) This change fails into a state where it doesn't error (it silently skips), and while I don't see anything wrong with the logic there, I can't help but worry that something will break in a future commit and silently fail, causing updates to be missed. The preference would be for it to fail in a noisy way, but I think that would defeat the purpose of the change you want to see.
Beyond that, my understanding is that Github Actions should not run on a fork automatically. You should see this warning when you fork the repo:

If you have moved past that warning (you might be able to revert? I'm not sure), you could always disable the individual actions you do not want to run or delete the actions in your fork.

ok! I think fork the project via cli.


[BUG] 404 page how-do-i-get-an-fusd-enabled-wallet

Current Behavior
clicking on How do I get an FUSD-enabled wallet?  leads to a 404 page on developers.flow.com https://developers.flow.com/fusd#how-do-i-get-an-fusd-enabled-wallet

Expected Behavior
leading to a page that shows how to setup fusd wallet
Steps To Reproduce
Try and fund testnet wallet with FUSD instead of Flow

Environment
MacBook, Google Chrome

What are you currently working on that this is blocking?
dApp on Flow that uses FUSD

FUSD has been deprecated and this token will be removed from the Flow Faucet


Adding the Dockerfile that could run onnxruntime server and  corresponding instruments

Adding the Dockerfile that could run onnxruntime server and  corresponding instruments how to use the docker

/azp run


SET REDIRECT misses some warning messages.

Reported by ansley b manke on 19 Jan 2011 23:32 UTC
Rather than re-open old #594, a new bug.
Referring to this message to the Users List,
http://www.pmel.noaa.gov/maillists/tmap/ferret_users/fu_2010/msg00809.html
It sounds like the "set redirect" should do what Patrick wants, but apparently we missed something.  There's a routine fmt/src/tm_split_message.F which is analogous to fer/utl/split_list.F.  Karl, we had pointed you to split_list as the place to implement the redirection; the same thing should be done in tm_split_message.F  too.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1793

Modified by karl.smith on 20 Jan 2011 16:07 UTC

Comment by karl.smith on 16 Feb 2011 00:52 UTC
TM_SPLIT_MESSAGE now just calls SPLIT_LIST to handle messages appropriately.

Comment by karl.smith on 16 Feb 2011 00:54 UTC
Test already in benchmarks: redirect stdout and then issue a "say" command.  The message was previously not redirected, but now it is.

Comment by ansley.b.manke on 12 Mar 2012 17:47 UTC
Yet more messages are missed, this time under PPLUS.  The routine ppl/symlib/cmderr.F writes some error messages and they are not redirected.  This is yet another Patrick Brockmann report; here is his example, where an intentional error is created.
$ cat redirect.jnl
set redirect/clobber/file="ferret_out.txt" STDOUT STDERR
set mode verify

plot i[i=1:100]

ppl set ME_text 'aaa bbb'

$ ferret -script redirect.jnl

At the screen I have
**ERROR Symbol not found              set ME_text 'aaa bbb

but not in the file ferret_out.txt.


fix: null pointer when removing changelog

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] MIT license headers are included in each file.

/check-enforcer override


Update Readme for Ruby 2.3 install windows

This is a great guide: http://installfest.railsbridge.org/installfest/windows


php74Extensions.blackfire: 1.75.0 -> 1.76.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for php74Extensions.blackfire is: "Blackfire Profiler PHP module"
meta.homepage for php74Extensions.blackfire is: "https://blackfire.io/"
meta.changelog for php74Extensions.blackfire is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.76.0 with grep in /nix/store/cg4lc2l6s8h1lv1h7ivb0asljjqhxvg5-php-blackfire-1.76.0

found 1.76.0 in filename of file in /nix/store/cg4lc2l6s8h1lv1h7ivb0asljjqhxvg5-php-blackfire-1.76.0

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

3 total rebuild path(s)

3 package rebuild(s)

3 x86_64-linux rebuild(s)
0 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
php74Extensions.blackfire
php80Extensions.blackfire
php81Extensions.blackfire

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/cg4lc2l6s8h1lv1h7ivb0asljjqhxvg5-php-blackfire-1.76.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/cg4lc2l6s8h1lv1h7ivb0asljjqhxvg5-php-blackfire-1.76.0
ls -la /nix/store/cg4lc2l6s8h1lv1h7ivb0asljjqhxvg5-php-blackfire-1.76.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

php74Extensions.blackfire
php80Extensions.blackfire
php81Extensions.blackfire

Maintainer pings
cc @jtojnar for testing.

Newer version already merged


move_doc_files: do not crash with which='none'

Fixes KeyError when selecting 'none' with the "Select docs_tool:"
option.

Hi @blueyed thanks for catching this :)
I also added your fix to https://github.com/pytest-dev/cookiecutter-pytest-plugin/pull/29/commits/9919b486c1c1b9015961b9eacc01a9643299e548 which is doing the same for cookiecutter-pytest-plugin

Thanks @obestwalter!


Change wait_for_vault timeout.

Try to fix the latest CI failure in wait_for_vault condition.

Why do we need vault timeout in integration tests?


[Bug]: Translator in custom field

What happened?
Is it possible to use translator in custom field like this:

if so, what is the correct way to use it
How to reproduce the bug
try using translator in custom field and get this error

Package Version
1.0
PHP Version
8.1
Laravel Version
10
Which operating systems does with happen with?
Windows
Notes
No response

this variable gender does not exist inside the plugin translate file.
You should create inside your project and pass directly.
Ex: __('gender')

thank you it's work 👍👍👍