[SPARK-24552][core][SQL] Use task ID instead of attempt number for writes.

This passes the unique task attempt id instead of attempt number to v2 data sources because attempt number is reused when stages are retried. When attempt numbers are reused, sources that track data by partition id and attempt number may incorrectly clean up data because the same attempt number can be both committed and aborted.
For v1 / Hadoop writes, generate a unique ID based on available attempt numbers to avoid a similar problem.
Closes #21558

Credit here should go to @rdblue when merging.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/384/
Test PASSed.

Test build #92181 has started for PR 21606 at commit 7233a5f.

I guess it depends on how picky we want to be there are other places that use attemptNumber that we could update to task id: InternalRowDataWriterFactory, memoryV2, and SimpleWritableDataSource, places that implement createDataWriter

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/388/
Test PASSed.

Test build #92187 has started for PR 21606 at commit c884f4f.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/389/
Test PASSed.

Test build #92189 has started for PR 21606 at commit 5efaae7.

Test build #92190 has started for PR 21606 at commit 227d513.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/391/
Test PASSed.

Test build #92181 has finished for PR 21606 at commit 7233a5f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #92192 has started for PR 21606 at commit a16d9f9.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/394/
Test PASSed.

Test build #92187 has finished for PR 21606 at commit c884f4f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92187/
Test PASSed.

Test build #92189 has finished for PR 21606 at commit 5efaae7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92189/
Test PASSed.

Test build #92190 has finished for PR 21606 at commit 227d513.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92190/
Test PASSed.

Merged build finished. Test PASSed.

Test build #92192 has finished for PR 21606 at commit a16d9f9.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92192/
Test FAILed.

test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/412/
Test PASSed.

Test build #92214 has started for PR 21606 at commit a16d9f9.

Test build #92218 has started for PR 21606 at commit 503852f.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/415/
Test PASSed.

Test build #92214 has finished for PR 21606 at commit a16d9f9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92214/
Test PASSed.

+1

+1

A general comment about the log messages: it seems pretty noisy to have "logInfo" messages for every task (doing so only in the "failure" paths would be better in my opinion); but I'm keeping the current log level.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/419/
Test PASSed.

Test build #92225 has started for PR 21606 at commit 47131c5.

Also, since this patch won't backport, I'll go ahead and send versions of it for branch-2.3 and branch-2.2 (which I think will be enough to also backport to 2.1).

just an fyi, I was looking at backporting to 2.2, looks like at least some write calls don't have the issue:
https://github.com/apache/spark/blob/branch-2.2/core/src/main/scala/org/apache/spark/rdd/PairRDDFunctions.scala#L1128
Looks like that was lost when things we refactored.
In fact a test that rolls it, not sure why that was added:
https://github.com/apache/spark/blob/master/core/src/test/scala/org/apache/spark/scheduler/OutputCommitCoordinatorSuite.scala#L325

Interesting. But I found the same code in a different place:
https://github.com/apache/spark/blob/branch-2.2/core/src/main/scala/org/apache/spark/internal/io/SparkHadoopMapReduceWriter.scala#L89

Test build #92218 has finished for PR 21606 at commit 503852f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92218/
Test PASSed.

Yeah so things like saveAsTextFile in 2.2 are ok but other functions like saveAsNewAPIHadoopFile and the dataframe writers have the issue, so we do need to backport

Test build #92235 has started for PR 21606 at commit e892937.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/426/
Test PASSed.

Merged build finished. Test PASSed.

Test build #92225 has finished for PR 21606 at commit 47131c5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #92235 has finished for PR 21606 at commit e892937.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92235/
Test PASSed.

Cool, so it looks like we can merge this? (And #21615 and #21616?)

Given the deafening silence, I'll merge the PRs myself, given there's a bunch of +1s from others.


reference to non-existing functions in arch/x86/core/intstub.S

 * Only the volatile integer registers are saved since ISRs are assumed not to
 * utilize floating point (or SSE) instructions.  If an ISR requires the usage
 * of floating point (or SSE) instructions, it must first invoke nanoCpuFpSave()
 * (or nanoCpuSseSave()) at the beginning of the ISR.  A subsequent
 * nanoCpuFpRestore() (or nanoCpuSseRestore()) is needed just prior to returning
 * from the ISR.  Note that the nanoCpuFpSave(), nanoCpuSseSave(),
 * nanoCpuFpRestore(), and nanoCpuSseRestore() APIs have not been
 * implemented yet.

most of the above functions were probably renamed and do not exist, need to clean this up and use current implementation functions in the docs.

Note the mentioned documentation says these APIs have not been implemented yet, so perhaps they never have, and we should just delete this paragraph except for the first sentence?

@andrewboie could you give us the names of the functions that replace the former nano* ones?


DBZ-6484 Support FQDN table name in history

https://issues.redhat.com/browse/DBZ-6484
Cleanup of https://github.com/debezium/debezium/pull/4639

@subkanthi Applied the changes, thanks a lot for the code!


Robust local splitting of ways which are part of route relations

(should) fix #7653, #8415
This reworks the handling of relation-membership of ways during a split operation:
Instead of fully re-sorting the whole relation every time a member is split (which is inherently susceptible to faults when encountering edge cases, partially incomplete or partially broken data), this performs the split as a local operation:

The new way is inserted into the relation before or after the existing member, depending only[^1] on how the old/new way connect to their neighboring members.
Disable the split operation when the neighboring members have not been loaded (i.e. because they are too far out of the viewport)
Disable the split operation on closed roundabout ways, where the split operation would potentially result in invalid routes

Adding this has the pleasant side effect that the complexity in add_member action could be reduced significantly, as it now does not have to keep track of insertPairs anymore.
The new algorithm works fine for:

existing test cases
situations where a route relation is loaded partially and the loaded members form a different, shorter route than the full set of relation members do
cases with unsplit roundabouts (e.g. relation/13092746)

[^1]: for cases where the split way and its neighbors form a loop which turns back to itself (i.e. situations like way-1 -> way-to-split -> way-1), an additional look-ahead is implemented to disambiguate the proper order

Here’s a pathological case that I had been meaning to use as a test in my own experiments toward a solution: https://www.openstreetmap.org/relation/15504667 traverses https://www.openstreetmap.org/relation/15508044 more than once, going over a different portion each time. It’s also a one-way relation, so the order definitely matters. It’s a very long relation,[^unmapped] so it would be impractical to expect the user to load its members one by one. It backtracks many times, sometimes doubling back immediately on a given roadway.
Would a split in this roundabout or one of the backtracks preserve the order? What if the roundabout had originally been mapped as a single closed way and added to the relation twice?
[^unmapped]: Actually, only a third of the actual length has been mapped so far.

Thanks for the feedback on this @1ec5 ! :bow:
I forgot to ask whether you have a suggestion to improve the "cannot split because of missing members" message now that we have the download button for the relations. Perhaps something like the following?

This line cannot be split because it is part of a larger relation which has only been partially loaded. Make sure that the way's relations are fully downloaded.

Yes, something along those lines:

This line cannot be split because a parent relation isn’t fully downloaded. Download the full relation.

Hopefully the user would be able to figure out where to download the members. Would it be too verbose to read before the message goes away?


Upgrade all components to Typescript

There are some JSX pages. Typescript is much nicer.

Done (more or less).


Percent Decode Url Parameters

In my application I have some routes with parameters that can be user defined and often include spaces. This means that there usually is a %20 instead of a space in the parameter which is unexpected from a dev perspective. This makes sure that when building the parameter map we percent decode the parameter.
This breaks a doc test and I have a proposed fix for it since it now requires the "ssr" feature to work. The test now only runs when the ssr feature is enabled.

So the question with params matching here is what happens on client side routing. After the initial page hit, which is rendered on the server, all other navigation is done on the client.
Did you test that eventuality?

There already exists a client side function for decoding which it should use for client side routing.
Regardless I did just finish testing it on my personal site and client side routing is working just fine.

Can you run cargo clippy and cargo fmt on it? That's what's making it fail the CI

I have, it didn't change anything

@benwis Clippy is complaining about an issue inside rkyv, here and everywhere. I haven't had time to look into it. These thing pop up sometimes because it's running nightly clippy.

Clippy is complaining about an issue inside rkyv, here and everywhere.

I am just starting off the bug hunt investigation .. and reporting what I see as I go
Sometimes just talking about it in a reasoned way allow others to effortlessly jump directly to the right solution.
In the build artefacts I see this error
running `cargo clippy --no-default-features --all-features` on leptos (19/120)
error: process didn't exit successfully: `/home/runner/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/bin/cargo clippy --manifest-path /home/runner/work/leptos/leptos/leptos/Cargo.toml --no-default-features --all-features` (exit status: 101)

From the main branch, here is what I interpret as the failing command
cargo +nightly clippy  --no-default-features --all-features
so we can peek what is behind the (exit status 101).
I get  a long list of errors, all of the the same form. So I am just recording the first two errors below
They are rkyv-0.7.43 relates
When I look at the full list of errors, looking for a entry point, a place in the leptos code which is a root of all this ... I get no joy...
What do we need to do to get rkyv to compile? in this instance
So I am expecting a the answer to be and complex nested inter-dependency
where a sub module we import has a bug and is not setting the correct feature flags to ensure compilation.
When I use cargo why rkyv or cargo tree to grapple with these issues.
I am doing something wrong as I can I see no dependence on rkyv.... so any advice would be welcome
error[E0277]: the trait bound `NonZero<i8>: ArchiveCopy` is not satisfied
   --> /home/martin/.cargo/registry/src/index.crates.io-6f17d22bba15001f/rkyv-0.7.43/src/copy.rs:108:33
    |
108 | unsafe impl ArchiveCopySafe for NonZeroI8 {}
    |                                 ^^^^^^^^^ the trait `ArchiveCopy` is not implemented for `NonZero<i8>`
    |
note: required by a bound in `ArchiveCopySafe`
   --> /home/martin/.cargo/registry/src/index.crates.io-6f17d22bba15001f/rkyv-0.7.43/src/copy.rs:99:35
    |
99  | pub unsafe trait ArchiveCopySafe: ArchiveCopy + Sized {}
    |                                   ^^^^^^^^^^^ required by this bound in `ArchiveCopySafe`

error[E0277]: the trait bound `NonZero<u8>: ArchiveCopy` is not satisfied
   --> /home/martin/.cargo/registry/src/index.crates.io-6f17d22bba15001f/rkyv-0.7.43/src/copy.rs:109:33
    |
109 | unsafe impl ArchiveCopySafe for NonZeroU8 {}
    |                                 ^^^^^^^^^ the trait `ArchiveCopy` is not implemented for `NonZero<u8>`

Clippy is complaining about an issue inside rkyv, here and everywhere.

I am just starting off the bug hunt investigation .. and reporting what I see as I go Sometimes just talking about it in a reasoned way allow others to effortlessly jump directly to the right solution.
In the build artefacts I see this error
running `cargo clippy --no-default-features --all-features` on leptos (19/120)
error: process didn't exit successfully: `/home/runner/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/bin/cargo clippy --manifest-path /home/runner/work/leptos/leptos/leptos/Cargo.toml --no-default-features --all-features` (exit status: 101)

From the main branch, here is what I interpret as the failing command
cargo +nightly clippy --no-default-features --all-features
so we can peek what is behind the (exit status 101).
I get a long list of errors, all of the the same form. So I am just recording the first two errors below
They are rkyv-0.7.43 related
When I look at the full list of errors, looking for a entry point, a place in the leptos code which is a root of all this ... I get no joy...
What do we need to do to get rkyv to compile? in this instance
So I am expecting a the answer to be and complex nested inter-dependency where a sub module we import has a bug and is not setting the correct feature flags to ensure compilation.
When I use cargo why rkyv or cargo tree to grapple with these issues. I am doing something wrong as I can I see no dependence on rkyv.... so any advice would be welcome
error[E0277]: the trait bound `NonZero<i8>: ArchiveCopy` is not satisfied
   --> /home/martin/.cargo/registry/src/index.crates.io-6f17d22bba15001f/rkyv-0.7.43/src/copy.rs:108:33
    |
108 | unsafe impl ArchiveCopySafe for NonZeroI8 {}
    |                                 ^^^^^^^^^ the trait `ArchiveCopy` is not implemented for `NonZero<i8>`
    |
note: required by a bound in `ArchiveCopySafe`
   --> /home/martin/.cargo/registry/src/index.crates.io-6f17d22bba15001f/rkyv-0.7.43/src/copy.rs:99:35
    |
99  | pub unsafe trait ArchiveCopySafe: ArchiveCopy + Sized {}
    |                                   ^^^^^^^^^^^ required by this bound in `ArchiveCopySafe`

error[E0277]: the trait bound `NonZero<u8>: ArchiveCopy` is not satisfied
   --> /home/martin/.cargo/registry/src/index.crates.io-6f17d22bba15001f/rkyv-0.7.43/src/copy.rs:109:33
    |
109 | unsafe impl ArchiveCopySafe for NonZeroU8 {}
    |                                 ^^^^^^^^^ the trait `ArchiveCopy` is not implemented for `NonZero<u8>`

I wouldn't worry about this, making clippy happy with rkyv is not required for us to merge it

Thanks, this looks good to me!
@martinfrances107 rkyv is a dependency of server_fn. We should probably change the CI so that it uses the equivalent of cargo clippy --no-deps and doesn't lint our dependencies in any case, but I don't see an obvious way to do that with cargo-hack, which is what's used at present.
I'll just run CI locally and merge.


Adding status code to the latency metric for the HTTP probe

Describe the feature you'd like and the problem it will solve
The latency metric for the http probe currently lacks a status code field. Including such a field would enable measuring success and failure latencies separately.
Implementing this feature with the current custom implementation for the distribution might be complex, but leveraging Prometheus' client_golang library could simplify the process.

Including such a field would enable measuring success and failure latencies separately.

You can already do that right? There is a resp_code metric that HTTP probes export separately.
There might still be value in tracking latency for individual response code, but such use cases are limited in probers as in most cases you want only certain response codes to be counted as success, and when you do that those are the only response codes that will contribute to latency.

We rely on the latency per status code metric provided by health-exporter. It helps a lot in various troubleshooting scenarios.
We wanted to use cloudprober instead of health-exporter but the unavailability of such metric is a blocker.

You can already do that right? There is a resp_code metric that HTTP probes export separately.

I think the resp_code metric only provides the request count, not the latency.

Dear @manugarg, do you agree with a PR implementing this feature or are you against it?
By this feature, I mean utilizing the client_go library for distribution metrics and enabling dynamic label values.


Using Mustache as template engine (now that it does not have a compile method)

The examples given in #331 no longer work because mustache has removed compile method. (Refer: https://github.com/janl/mustache.js/issues/346)
The browser now reports:
Uncaught TypeError: Object ... has no method 'compile'

:+1: thanks @kzap

+1 this worked for me too, thanks @kzap

very thanks @kzap


Streaming responses should have Content-Type set to application/x-ndjson

Currently streaming responses return text/plain but they should return application/x-ndjson . Later we should consider application/json (see #281) or text/event-stream for browser based clients

This was fixed in cff002b

Hi jmorganca, just trying to be helpful here:
The official approach your looking for is: https://datatracker.ietf.org/doc/html/rfc7464
Anything that says application/*json is supposed to be valid json. A stream of json objects is not valid json.
Interesting discussion about that here: https://github.com/spring-projects/spring-framework/issues/21283
Not every content-type can be streamed (depending on the client). But text/plain you can always stream. And is widely adopted.
text/event-stream has limited support. It also complicates reading with curl from the command line.
In summary:

Valid: text/plain
Better: text/plain; charset=utf-8 (since json is always utf-8)
Standard: application/json-seq (support unknown)
Limited support: text/event-stream
Invalid: application/stream+json (since the response is not json)
Invalid: application/x-ndjson (non-official, partial support)

More info:

https://www.w3.org/TR/activitystreams-core/#ex27-jsonld
https://www.w3.org/wiki/Activity_Streams/Examples
https://www.w3.org/wiki/Activity_Streams#Definition
https://developer.mozilla.org/en-US/docs/Web/API/Server-sent_events/Using_server-sent_events
https://jsonlines.org/ (! non-standard)
https://github.com/ndjson/ndjson-spec (! non-standard)
https://www.iana.org/assignments/media-types/media-types.xhtml (official, lists json-seq)

About #281, I think you meant: Accept: application/json rather than Content-Type.
If that's the case, that could be a great addition for those who want to.
However, since this is HTTP, there is no way to differentiate between a timeout and waiting for the response to be generated. Some clients also replay the request if the connection temporarily drops.
So for that to work, can I suggest the following (just a suggestion):
POST /api/generate HTTP/1.1
Host: localhost:3333
Accept: application/json

{ "model": "orca", "prompt": "hi" }

HTTP/1.1 202 Accepted
Content-Type: application/json
Vary: Accept

{ "id": "1234" }

GET /api/generate?id=1234 HTTP/1.1
Host: localhost:3333

HTTP/1.1 299 Generating
Content-Type: text/plain; charset=utf-8
Retry-After: 5
Refresh: 5
Cache-Control: no-store

Generating... This page will refresh after 5 seconds (in the browser).

HTTP/1.1 200 OK
Content-Type: text/plain

I'm just an AI.

Note that 299 Generating does not exist. But neither 4xx or 5xx is good for this edge-case. Any of the 2xx range will work. But since none actually describe this behaviour, you can always invent your own. This should be compatible with any client since 2xx simply indicates the request is valid and the server understands that request. Refresh is not official either. But it actually triggers a refresh in the browser. The Retry-After is the official one to use. Any OK response can be cached by a client, so using Cache-Control: no-store is advised.

What is the final solution for non-streaming response?

@Enhitech you can set stream to false in the /api/generate and /api/chat endpoints.


fix: updating to include correct counter

Description

Adds API Key AND IP instead of IP for resources

Testing required outside of automated testing?

[ ] Not Applicable

Screenshots (if appropriate):

[ ] Not Applicable

Rollback / Rollforward Procedure

[ ] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

:tada: This issue has been resolved in version 1.167.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Release version 0.0.3

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

Project coverage is 100.00%. Comparing base (330921f) to head (9182443).
 Consider uploading reports for the commit a8acf32 to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #102   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           33        33
  Lines          635       635
  Branches        37        37
=========================================
  Hits           635       635

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


DownloadingError: read ECONNRESET     at TLSWrap.onStreamRead (internal/stream_base_commons.js:201:27)

Ubuntu 18.04.
Truffle v5.1.2 (core: 5.1.2)
Solidity v0.5.12 (solc-js)
Node v13.2.0
Web3.js v1.2.2

When I execute truffle unbox pet-shop on terminal, I get error
✔ Preparing to download
⠼ DownloadingError: read ECONNRESET
    at TLSWrap.onStreamRead (internal/stream_base_commons.js:201:27)

I am expecting contents of directory to look like this (from https://www.dappuniversity.com/articles/the-ultimate-ethereum-dapp-tutorial)

But instead it looks like this

If you are behind organizational proxy, clone the repo, i.e.
git clone https://github.com/truffle-box/pet-shop-box.git
cd pet-shop-box
rm -rf .git # Remove the .git record
npm install


[Bug]: the ./setup.sh (now deleted) is mentioned again in the README, see Create your own Fabric Mill

What happened?
Under: Create your own Fabric Mill, ./setup.sh is mentioned again but missing:
But we go beyond just providing Patterns. We provide code for you to build your very own Fabric server and personal AI infrastructure!

To get started, just run the ./setup.sh file and it'll set up the client, the API server, and the API server web interface. The output of the setup command will also tell you how to run the commands to start them.

This is an interesting project to me, but installation was difficult and the README has mistakes.
Version check

[X] Yes I was.

Relevant log output
Nothing as ./setup.sh is missing.

Relevant screenshots (optional)
No response

we are using pipx now. go to the install section. I'll fix the readme


Question: How do I add more than two options to a confirmation box?

Please check all of the platforms you are having the issue on (if platform is not listed, it is not supported)

[x] iOS
[x] Android
[] UWP
[] NET Standard 2.0

Version of ACR UserDialogs Library
6.3.8
Question
In this Xamarin Forms Forum Post its stated that its easy to extend the library to handle a Confirmation Box with more than two options.  I was wondering if there was any documentation or examples of how to make this happen.

No - if you want to overwrite things, the basis is built on the IUserDialogs interface.  All methods on each platform are also marked as virtual.  You can change any aspect you want from there


c release notes for the 2021-06 release

This is an automated pull request to gather the c release notes for the 2021-06 release. See Producing Release Notes for details of the processing.

The c repo isn't ready for release note automation because it uses different tagging and changelog strategies that aren't currently supported.


[BUG 🐛]  Cannot build iOS with @react-native-firebase/messaging

Logs and Screenshots
    RNFBMessaging (from `../node_modules/@react-native-firebase/messaging`) was resolved to 21.6.0, which depends on
      Firebase/Messaging (= 11.5.0) was resolved to 11.5.0, which depends on
        FirebaseMessaging (~> 11.5.0) was resolved to 11.5.0, which depends on
          GoogleDataTransport (~> 10.0)

    VisionCameraFaceDetector (from `../node_modules/react-native-vision-camera-face-detector`) was resolved to 1.7.1, which depends on
      GoogleMLKit/FaceDetection was resolved to 0.60.0, which depends on
        GoogleMLKit/MLKitCore (= 0.60.0) was resolved to 0.60.0, which depends on
          MLKitCommon (~> 0.60.0) was resolved to 0.60.0, which depends on
            GoogleDataTransport (~> 3.2)

package.json file:
{
"dependencies": {
"@react-native-firebase/app": "^21.6.0",
"@react-native-firebase/messaging": "^21.6.0",
"react-native-vision-camera": "^4.6.1",
"react-native-vision-camera-face-detector": "^1.7.1",
"react-native-worklets-core": "^1.5.0"
}
}

Same here
{
"dependencies": {
    "@react-native-firebase/app": "^21.6.0",
    "@react-native-firebase/messaging": "^21.6.0",
    "@react-native-firebase/remote-config": "^21.6.0",
    "react-native-vision-camera": "^4.6.1",
    "react-native-vision-camera-face-detector": "^1.7.1",
  }
}

I tried upgrading to the latest version "GoogleMLKit/FaceDetection", "7.0.0"
then reinstalling the pod and there was no error related to firebase dependencies.

Same here
{
"dependencies": {
    "@react-native-firebase/app": "^21.6.0",
    "@react-native-firebase/messaging": "^21.6.0",
    "@react-native-firebase/remote-config": "^21.6.0",
    "react-native-vision-camera": "^4.6.1",
    "react-native-vision-camera-face-detector": "^1.7.1",
  }
}

I tried upgrading to the latest version "GoogleMLKit/FaceDetection", "7.0.0"
then reinstalling the pod and there was no error related to firebase dependencies.

The currntly version of @react-native-firebase/app I tested is 19.2.2.
I can't tell if other versions will work and currently I don't have time to work on a fix.
But you are really wellcome to open a PR with a fix


Remove branch option on pr trigger

All PR should be trigger tests, isn't it?

r? @yujif

Thank you!


Removed default data region creation for client nodes.

Added meaningful exception if there's no data region for the cache.

PR is not associated with a ticket. Please see how to associate
https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute#HowtoContribute-Creation
Closing PR, which seems to be test-only due to inactivity.
Feel free to reopen this PR if it is actual.


Incomplete information about the command

The documentation does not inform the user if the command is persistent across reboots. Please add this information.
[Enter feedback here]

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

ID: edd1faab-d61a-c715-9a3e-c559cbeeda36
Version Independent ID: 49e5af1d-5d8e-20d2-ec8e-49694c1255e3
Content: subst
Content Source: WindowsServerDocs/administration/windows-commands/subst.md
Product: windows-server
Technology: windows-commands
GitHub Login: @eross-msft
Microsoft Alias: lizross

#assign:IngridAtMicrosoft

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Added option for multiple Treadmill LDAP users

Related PRs:
https://github.com/Morgan-Stanley/treadmill/pull/233
https://github.com/Morgan-Stanley/treadmill-aws/pull/167
https://github.com/Morgan-Stanley/treadmill-deploy/pull/57

Updated to use TREADMILL_LDAP_ADMINS


Rn0.60

Hi bro,
Please check this for update sdk and change compile to implementation.
Testable rn0.60.4
Thank you

take at look to this repo: https://github.com/react-native-community/react-native-device-info/blob/master/android/build.gradle, better than hardcoding values


Snap Zones need a means for suppressing stash sound

Snap-zones may need to be "populated" without triggering the stash-sound associated with a player putting an item in a snap-zone.
This can occur on scene loading when the snap-zone may have a design-time initial_object or if a persistence mechanism wishes to populate a saved state.

Have there been any developments addressing this issue? I am also looking for a solution to this, specifically suppressing stash sounds when an initial_object is assigned to a snap zone.

wouldnt a if !initial_object: be enough, anyways am a bit out of loop, slowly tryin to get back to xr but i might look at some of these things


External domain name propagation to rancher

Can you squash, rebase, and remove WIP if done?

@ibuildthecloud done

LGTM


Allow passing timestamp to constructor of uuid7

Without this PR, the only way to instantiate a UUID7 is with the current moment in time. This PR allows passing a timestamp in ms instead.

Yes please! this would be very helpful.


v1.2.0 appears to be broken

Nothing seems to load

Agreed. Same issue with you. When I switch to the downloading page, the loading circle keeps spining and nothing is loaded. I tried to add a download task, while the whole page keeps the loading status.

fixed in v1.2.1


Experiment: TraceryDisambiguator

This gives us a basic way to customize prompts based on inputs.

TraceryDisambiguator is essentially a router

Tries a succession of routes
Collects up to n matches

A route is a protocol that implements match(:) async -> TraceryContext?

RegexRoute is a concrete implementation of the protocol and allows regular-expression-based matching.

TraceryContext contains a grammar and a start string

The start string is used to begin flattening
The grammar is used in PromptService to overwrite keys on our base grammar. This allows us to save "state" extracted from the input to the grammar.

RegexRoute should allow us to do simple keyword-based dispatch of prompts.
Future improvements:

Could be expanded to tokenize, lemmatize, or otherwise preprocess text before handing to routes
Could be expanded support other kinds of router, including on-device ML classifier, or even LLM.

Picking up work here https://github.com/subconsciousnetwork/subconscious/pull/1101


Cleanup

Removes dev packages and jupyter lab from conda environment. It should speedup binder and solve issue #40

Was wondering about those, thanks a lot!


Add no encryption option for SMTP settings

Related to #4311
Add option to configure SMTP settings without encryption.

Update app/Livewire/Notifications/Email.php and app/Livewire/SettingsEmail.php to include "No Encryption" option in the smtpEncryption field and update validation rules.
Modify app/Notifications/Channels/EmailChannel.php to handle the "No Encryption" option in the bootConfigs method.
Add set_transanctional_email_settings function in app/Livewire/Help.php to support the "No Encryption" option.
Update config/mail.php to handle the "No Encryption" option in the mail configuration.

I added a few fixes here and there, but not sure if setting to null will solve the issue as pointed our here.

Just after I wrote the last comment I found how to set it in the source code of Symfony Mailer.


Wallpapers bloating the repository unnecessarily

Already discussed on IRC, opening an issue to know what the other users think about removing the wallpapers from the repository.
It bothers me to wait 10-15s more for downloading sway just because of wallpapers that I'm not gonna using. This is kind of very personal preference, your users love personalizing their desktop and are not likely using these bundled wallpapers either.
Wallpapers as an artwork have nothing to do alongside a WM's codebase. I suggest to split them in a separate repository and set a solid color as background for the sake of simplicity and lightweightness.

It's not like it'd make any difference, when you clone the repo (with full history contents).

It makes a difference when you install sway-git on arch, because it means you fully reclone the repo each time you want to update the package. For me it should be on another repo.

I agree, it would be nice to make it a separate repo and package. I don't notice now because I have a decent connection. But when you are on a shitty connection like I was for the past few months then 10MB really is a lot :)
@Hummer12007 is right, it will clone the whole history including the wallpapers, even if they are removed at some point. It would make sense to patch makepkg to not clone the entire history IMO.
I have a CI server automatically building sway-git packages so cloning the whole repo is not a problem for me though :)

If I chose to remove it from the repo, I can purge it from the entire history of the repo if necessary.

But then commit IDs will change. Might be problematic with tags and references in the bugtracker.

It makes a difference when you install sway-git on arch, because it means you fully reclone the repo each time you want to update the package.
That is wrong. I keep the repos for my git packages around and don't clone them each time.

Anyways, I don't think it isn't too critical for a development tree. If you wish to provide a leightweight release, then creating a tarball without the images should be possible.

Those who really want a small copy of the repo for light development purposes can just git clone --depth=1 to only get the current head, no need to rewrite history.
It's really substandard for any real work, though I think if images are too much a size constraint, best not try compiling anything.

Issue closed in all disdain from the majority of feedbacks in favor of removing these shitty wallpapers.

I'm not going to break the repo's history by rebasing them out, so the bloat will always be there for clones. There's an option not to install them during the build.


Player crouch

While the player is holding down the crouch button (down),
they should enter a crouched state.
This state includes the following changes:

change hitbox
change animations
gets all new animations, with new Idle, Walk, etc. animations
change movement stuff

MovementAcceleration
MaxMovementVelocity
BaseFriction

The crouched state should probably have its own entity config,
but entering/exiting crouched state should be handled through actual code.

Crouching works with the abstract config "PlayerCrouch".

Closing for now.


NewStmt: ignore all leading comment blocks

Running queries with leading comments such as /* foo */ fail because of NewStmt not handling it. This change attempts to help fix that.
Fixes #46.
Note:

This should be an easy fix with regexp, but I'm guessing you are not doing that for performance reasons. So I just added an else if block. Let me know if you want me to change the for loop to a regexp-based fix.
I also noticed that you did not have ANY dependencies in your go.mod, so I avoided adding a dependency on asserts from github.com/stretchr/testify/assert and used simple built-checks in the unit-tests. Let me know if I should change this.

@sijms Can you please take a look at this when you find time?


use rbac.authorization.k8s.io/v1 for ClusterRole/Bindings where k8s >= 1.17

On kubernetes clusters 1.17 or newer, rbac.authorization.k8s.io/v1beta1 is deprecated.
https://v1-17.docs.kubernetes.io/docs/setup/release/notes/#deprecations-and-removals

fixes #516

E2E tests are not passing 😕

I'm gonna close this in favor of #609. Thank you for your effort!

I'm gonna close this in favor of #609. Thank you for your effort!


thumbnail properties missing

thumbnail.txt
This properties missing in Bootstrap 4.3.1 and present in bootstrap 3.3.7

The .thumbnail was dropped long ago: https://getbootstrap.com/docs/4.4/migration/#components


[FLINK-18850][Table SQL / Runtime]Add late records dropped metric for row time over windows

What is the purpose of the change
Currently all the row time over windows in blink planner runtime discards late records silently, it would be good to have a metric about the late records dropping.
Brief change log

add metrics of late records dropping to all row time over windows

Verifying this change
This change added tests and can be verified as follows:

RowTimeRangeBoundedPrecedingFunctionTest
RowTimeRangeUnboundedPrecedingFunctionTest
RowTimeRowsBoundedPrecedingFunctionTest
RowTimeRowsUnboundedPrecedingFunctionTest

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (no)

@flinkbot run azure

@libenchao Why this PR is closed?

@zhuxiaoshang I merged this manually, you can see it here: https://github.com/apache/flink/commit/b5b30650291e470a9635aef5f1348690bba78b29


Digest test failing on Rakudo 2017.04.3-235-gdc5eece built on MoarVM version 2017.04-64-g6d5ea04

$ perl6 --version
This is Rakudo version 2017.04.3-235-gdc5eece built on MoarVM version 2017.04-64-g6d5ea04

===> Testing: Digest:ver('0.3.4'):auth('Lucien Grondin')
t/digest.t .. ok
t/ripemd.t .. ok
# Failed test 'sha256'
# at t/sha.t line 12
t/sha.t .....1/2# expected: 'fbc1a9f858ea9e177916964bd88c3d37b91a1e84412765e29950777f265c4b75'
#      got: 'd7d04233b3a8427b7798817eae45cb70c7c24cfcc6740c2b44fb7aa823662036'
# Looks like you failed 1 test of 2
t/sha.t ..... Dubious, test returned 1
Failed 1/2 subtests

It failed in my travis build here

Works with 2017.04.3-47-gf0414c4, fails with 2017.04.3-235-gdc5eece
{master} ~/git/libdigest-perl6$ perl6 -v
This is Rakudo version 2017.04.3-47-gf0414c4 built on MoarVM version 2017.04-44-gf0db882
implementing Perl 6.c.
{master} ~/git/libdigest-perl6$ perl6 t/sha.t
1..2
ok 1 - sha1
ok 2 - sha256
{master} ~/git/libdigest-perl6$ ~/git/rakudo/install/bin/perl6 -v
This is Rakudo version 2017.04.3-235-gdc5eece built on MoarVM version 2017.04-64-g6d5ea04
implementing Perl 6.c.
{master} ~/git/libdigest-perl6$ ~/git/rakudo/install/bin/perl6 t/sha.t
1..2
ok 1 - sha1
not ok 2 - sha256

# Failed test 'sha256'
# at t/sha.t line 12
# expected: 'fbc1a9f858ea9e177916964bd88c3d37b91a1e84412765e29950777f265c4b75'
#      got: 'd7d04233b3a8427b7798817eae45cb70c7c24cfcc6740c2b44fb7aa823662036'
# Looks like you failed 1 test of 2

Narrowed down some more:
Works with 2017.04.3-210-g6435933, fails with 2017.04.3-211-gef29bb9
This is the commit that broke it:
https://github.com/rakudo/rakudo/commit/ef29bb9f41aa5d50638544db1ff966459c888f68
It affects the +> bit shift operator.

I don't know why it broke, but if you change
sub rotr(uint32 $n, uint32 $b) {
to
sub rotr($n, $b) {
the test passes...

The two cases differ when rotr is called with $n =<PHONE_NUMBER> and $b = 18.
In Rakudo 2017.04.3-210 rotr() returns<PHONE_NUMBER>0799 (correct).  In Rakudo 2017.04.3-211, rotr() returns<PHONE_NUMBER>4496 (incorrect).
Isolating those values and calling the exact same subroutine with the two versions of Rakudo both give the same (correct) answer.
Rakudo broke that code, but only in some weird circumstance.  The bug is in Rakudo, related to that commit, but I don't understand it and can't isolate it.  It does seem to be related to converting Int to uint32.  All is fine when I remove those.

rotr($n, $b) should not work properly, as it does not perform modulo 2**32 arithmetics, unless $n and $b are uint32, and I don't think they are.

I don't know why it broke, but if you change sub rotr(uint32 $n, uint32 $b) { to sub rotr($n, $b) { the test passes...

@MasterDuke17 would you know if that's caused by the glitches with uint32 types not being real uints or something?

In Rakudo 2017.04.3-210 rotr() returns<PHONE_NUMBER>0799 (correct). In Rakudo 2017.04.3-211, rotr() returns<PHONE_NUMBER>4496 (incorrect).

I can repro the test failure as well as test fixing itself if uint32 type constraints are removed, but not any issues with the shift ops or the rotr giving wrong value there ^ I can't reproduce that, at least via the bot:
<Zoffix_> c: ef29bb9f41aa5d sub rotr(uint32 $n, uint32 $b) { $n +> $b +| $n +< (32 - $b) }; dd rotr<PHONE_NUMBER>, 18
<committable6> Zoffix_, ¦ef29bb9: «27071659120799»

ATM, my suggestion would be to report this bug (by emailing to<EMAIL_ADDRESS>with some test case that repros the issue and to remove the uint32 type constraints in the routine, as the native candidates are currently disabled, so you're not getting any perf boost ATM anyway.

Disregard my comments above. Found it to be a SPESH issue. Filed as https://rt.perl.org/Ticket/Display.html?id=131306
SPESH stuff is beyond my current skill level, so I'm unsure when it'll get fixed.

This can be closed.
I added a kludge that works around the core issue in https://github.com/rakudo/rakudo/commit/2f22b701d4cf44fa43500edcc3450718fa7fd468 as well as tests to ensure it doesn't re-appear in https://github.com/perl6/roast/commit/c1d62112f3


Report 'no syntactic or textual changes' where appropriate

This seems to be implemented. Or if it's not, it could use some additional details as to what else is still needed.
However, I actually don't think this is super useful, especially when linters intersect with pre-commit hooks.
Assume that you have code that is all committed, but needs linting. With a linter like the Black linter for Python, it will actually reformat your source code. But when you go to view the changes with difft, the only thing it shows you is "No syntactic changes":
airflow/cli/commands/db.py --- Python
No syntactic changes.

And while that is technically correct, it's not super useful. I still need to be able to view the non-syntactic changes made by my linter.
 If one of my files was not completely linted, then Black will reformat a few files, and the pre-commit check will fail. Then when I go to view the unstaged changes with git diff, difftastic only shows me "No syntactic changes". Very much not useful for me, and this can happen for every commit.
When difftastic runs across a diff for a file that has no syntactic changes, I think it would be nice to emit a notification message at the top like "No syntactic changes" along with the parser that was used, while still displaying a normal diff underneath:
# Note: No syntactic changes (Python)
diff --git a/airflow/cli/commands/db.py b/airflow/cli/commands/db.py
index 4f8cca0ad..4f5ecf5c0 100644
--- a/airflow/cli/commands/db.py
+++ b/airflow/cli/commands/db.py
@@ -35,8 +35,9 @@ click_to_revision = click.option(
     '-r',
     '--to-revision',
     default=None,
-    help=("(Optional) If provided, only run migrations up to and including this revision. Note: must "
-          "provide either `--to-revision` or `--to-version`."
+    help=(
+        "(Optional) If provided, only run migrations up to and including this revision. Note: must "
+        "provide either `--to-revision` or `--to-version`."
     ),
 )
 click_to_version = click.option(

But besides this one small tweak, difftastic has been really great. Thank you for your efforts on this!

Yeah, I agree. I've actually made this mistake: accidentally reformatting some code and not immediately noticing because I used difftastic.
Long term I'd like to support this, but I'm still not happy with how the structural diffing works in all cases yet.


Tibialyzer 1.4.3

I downloaded the new version and it never opens, i have the .NET 4.6.2 as its recommended in the installation guide and it keeps crashing theres a pic of what happens when i try to open
http://imgur.com/a/K7tK1                <--- just happens for about 3 seconds
I can still open and use version 1.4.2, what can i do to fix this issue with the new version? thanks

Try deleting the settings.txt file in the Database folder and see if it solves your issue.

happen the same to me i try deleted settings.txt dosent work

Try reinstalling the. NET framework and see if it helps.

I downloaded lyzer <IP_ADDRESS> and it now opens smooth as always, anyways thanks mytherin for ur releases your the best 👍

still happening the same to me i try deleted settings and reinstall .net
other thing whenever i download the program again this open but after a few times cant open again so i need download again but the thing more important  i lose all my configuration !!
i put a photo here of some i think is not normal ahh and another thing im 64 bits windows 8,1


Migrate this to OCI

See https://helm.sh/blog/helm-oci-mediatypes/#:~:text=Helm has taken full advantage,that need to be maintained.

Done


can't hide take a screenshot in firefox 61

/*** blank-context.css ***/
/#screenshots_mozilla_org_create-screenshot,  / Take a Screenshot                    */
this setting doesn't work

The identifiers changed and a separator was added.
Try this:
/* #context-inspect + menuseparator:not([id]),           /* ********* Separator (FF61+) **********/
/* #screenshots_mozilla_org-menuitem-_create-screenshot, /* Take a Screenshot (FF61+)            */

Not working

Just checked again, it is working.
Did you remove the /* at line start?
Did you restart the browser?

It works now, sorry I didn't fully restart.
Many thanks!

https://github.com/stonecrusher/simpleMenuWizard/commit/69e2dd4370c3a0f370712864ccad587c7ea39d72


Cocopods: "cocoapods" demand requred even though it’s not a valid demand for the xplat agent

Repro:

Install cocoapods on a Mac - MacInCloud instances have it installed
Add and configure a Cocoapods Install Task to a VSO Build Def
Queue build with Mac agent

Expected: Runs the command
Actual: Can't find an agent
The cocoapods demand in task.json should be removed and replaced with a good error message speaking to where Cocopods can be installed from if the "pod" command is not found.

I'm on it

Bump on this one - I notice the cocoapods task still requires a cocoapods demand, but the xplat agent doesn't support it.
I know I saw this on the dogfooding DL as well. Is this expected to be fixed already?

Demand was removed several months back, so closing the issue: https://github.com/Microsoft/vsts-tasks/commit/1c6e617ad8fd1d4ba9ddaf55f3ddc8f8f8254ce1


[JENKINS-53727] Resolve plugin branch build problem

JIRA task: JENKINS-53727
This PR achieves:

[x] Fix mistake in checking yaml/yml extension.
[x] Upgrade plugin dependencies to newest versions.
[x] Resolve branch build problem by importing snakeyaml and  pipeline-model-api

@oleg-nenashev @martinda

Yes, because I chose the wrong Jenkinsfile format in testing (groovy instead of yml), therefore i thought there are syntax errors in the file.

:+1:


[ALLUXIO-2349]Improve error handling in GCSUnderFileSystemFactory#create

https://alluxio.atlassian.net/browse/ALLUXIO-2349
Right now we handle an exception by both logging and rethrowing the exception. This isn't great because the code which catches the thrown exception will also log it, so we will log the same exception multiple times.
We should remove the log statement.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/11839/
Test PASSed.

@nicefish666 do you know where the three "Signed-off-by: nicefish666" commits are coming from? Could we remove those?

yes,these could be removed

please check my lastest pr

Thanks for fixing - I'll close this PR and we can move to the new one


Provide an example for using token-exchange grant type with Keycloak and SonataFlow

The Token Exchange RFC might help users who need our workflows to act on behalf of resource owners. For example, Alice wants to call a workflow to access the Pet Store. The data retrieved by the Pet Store must be related and owned by Alice.
Alice can send a request to the workflow containing her access ID to the Pet Store but not the authentication secrets (e.g., username and password). The workflow will use its service token to exchange for a valid one representing Alice in the Pet Store before calling any REST functions to the resource service.
Resources:

https://www.keycloak.org/docs/latest/securing_apps/index.html#_token-exchange
https://datatracker.ietf.org/doc/html/rfc8693
https://github.com/keycloak/keycloak/discussions/26502

Done in the draft PR.


feat: add correlation coefficient example with program and client script

Correlation coefficient example program and client script.
Tests

[x] There are test for this new feature/change and all test are passing.

Nice - just run this and all good. The only question I have is should this live in the client_multi_party_compute directory? As it is for two parties
At some point i think we should refactor this to have an "Examples" directory where all tutorials and examples live. But for now this is fine, we can point people to it if we think it will be helpful to them

Maybe I can simply have one party? To simplify the example? I guess the multiparty folder is more to show the coordination for a multiparty computation.

Actually, why dont you put it in the main directory and call the directory somethig like correlation_coefficient_example. I think this is worthy of being in front of the users immediately ....wdyt?

We are at a point where we have a lot of examples. It may be a good idea to classify the examples into various folders. Some projects do that for users to find what they want easier.

Actually, why dont you put it in the main directory and call the directory somethig like correlation_coefficient_example. I think this is worthy of being in front of the users immediately ....wdyt? In the same way as the millionaire's example?

Yes, that could work. My suggestion is that we do it in a separate PR where we organise better all python scripts as well. We can have a folder for simple/basic programs where the intend is to show how to use nada. Then, we can have a set of programs/examples for "higher-order" use cases such as: euclidean distance, correlation coefficient, millionaire, voting, etc.


Add button to create new project on dashboard

Parent Issue: #644
Add button to create new project.
Wireframe

This is blocked until we have a new project page. Please don't start yet.

What about adding a row in the dashboard after the last project that is a link/button/something to add project? @chemichelle would that make the UX gods cry?

@taquitos adding 'add project' button to the last row of the project will become problematic when users start having many projects. If you're concern is that the 'add project' button in the header will be hidden after users scroll down, we can play with subtle animations where the header becomes skinnier and remains sticky (see an example from my iPhone here: https://drive.google.com/corp/drive/u/0/folders/1g1JMDL3WFfzwfuxlm7Wy3GUdG6Aud-PS )

Cool, makes sense 👍 I do really like the button in the header, and we can even defer on the header animations since most people will have less than 10 projects 😄


add radio buttons with option to change photo

This fixes #316
@anthonylord Any feedback on layout / content?
Before

After

I'm gonna go ahead and assume this one's ok to merge.


Update search-BST.md

📥 Pull Request
Description
add java code
Fixes #1295
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

label plz @ajay-dhangar

@ajay-dhangar

hello @ajay-dhangar can you please label it level or gssoc-ext for the points

@ajay-dhangar can you please label it since it helps me to gain points as Gssoc near to end

@ajay-dhangar Please add respective labels on this pr asap and dont forgot to add labels on any pr , it creates confusion among contributors that their pr points are not showing up on the leaderboard.
Core Team
Gssoc-ext

@ajay-dhangar Please add respective labels on this pr asap and dont forget to add labels on any pr , it creates confusion among contributors that their pr points are not showing up on the leaderboard.
Core Team Gssoc-ext

Sure

@ajay-dhangar Please add respective labels on this pr asap and dont forget to add labels on any pr , it creates confusion among contributors that their pr points are not showing up on the leaderboard.
Core Team Gssoc-ext

Sure

Thankyou for your response @ajay-dhangar


Cannot scroll/move cursor to the end of the line when using the "Base16 Tomorrow Dark" syntax theme.

Description:
With soft-wrap disabled and a long line of text that exceeds the available viewport width of the editor pane, erroneous behaviour is observed when attempting to go to the end of the line either via the (bottom) scroll bar or through the doesn't work properly and requires a repeat invocation to work as expected.
Steps to reproduce:
You can reproduce this either using the scroll bar or using a keyboard command (e.g. editor:move-to-end-of-screen-line with whatever key shortcut applies on your platform/configuration).

Open Atom with an empty file. Make sure soft wrap is disabled.
type a long string on a single line --- your string should exceed the width of the editor viewport.
Press End or try to scroll (using the scrollbar at the bottom of the editor pane) as far right as you can.

Actual results:
In the case of the end-of-line keyboard shortcut, the cursor does move to the end of the line, but both cursor and end of line content is not visible.
In the case of the scrollbar, you cannot go past a point which is just before the end of the line. The cursor remains hidden, as does the end of the line.
Expected results:
In the case of the scrollbar event, you should be able to actually read the content at the end of the line, with some appropriate padding.
In the case of the keyboard commands, you should see both end of line content and the cursor blinking there.
Notes:

Repeating the editor:move-to-end-of-screen-line command one more time does move to the end of the line. I couldn't find a way to achieve this using the scroll bar.
This issue was consistently reproducible on OS X.

Thanks for the detailed report, @cosmix, but I'm having trouble reproducing that exact behavior. Can you share which version of Atom you're using, which version of OSX, which type of Mac you're using (with retina screen?), and a GIF which clearly demonstrates the problem? (you can record one using this tool)
The behavior which I observed was this:

Opened a new file, soft wrap disabled
Triggered the lorem snippet to get a really long line
Positioned the cursor at the end of the line using editor:move-to-end-of-screen-line.
Observed that the cursor was definitely shown, but not completely -- seems like half of it went over the edge and is not visible. The "invisible" character marking the end of the line is not visible.
Triggering editor:move-to-end-of-screen-line again does not change anything.
Using the horizontal scrollbar gives the same behavior.

Atom 1.0.7-8d68680, OSX 10.10.4, MacBook Pro with a retina screen.
/cc @nathansobo for thoughts

Hello @izuzak. No retina screen here, OS X 10.0.4. Atom 1.0.6-f7a60c0 (release).
Here's a GIF showcasing the issue. In case I wasn't clear enough in my report, you can get to the end if you explicitly click with your mouse or move the cursor with the arrow keys. It doesn't work properly using End/Command+→ unless you do it twice. And it doesn't work at all by dragging the scrollbar.
Here's a capture from my box.

Let me know if there's something more I can help you with.

Thanks @cosmix :bow:

In case I wasn't clear enough in my report, you can get to the end if you explicitly click with your mouse or move the cursor with the arrow keys. It doesn't work properly using End/Command+→ unless you do it twice. And it doesn't work at all by dragging the scrollbar.

Yeah, that was clear. Strangely, I'm unable to reproduce that -- behavior is the same for me regardless of whether I use arrow keys, execute the editor:move-to-end-of-screen-line command or use the scrollbar.
Can anyone else reproduce this, just in case I'm missing something? /cc @atom/feedback

Ok, I tried getting rid of .atom, after keeping a backup copy of course and discovered the following: this issue, as shown above, is way more pronounced using 'Base 16 Tomorrow Dark'. It is much less of a problem using other themes, where, although the cursor does remain nearly out of sight (especially when using packages like minimap), the scrolling behaviour is much closer to my expectation. So, if anything, this might be related to the theme and not the core editor per se.

I updated the issue title to better match the comments following the description, that is that the issue is largely confined to the "Base16 Tomorrow Dark" syntax theme, although the underlying cause may be due to a bug in the editor rendering code. I have consistently reproduced this issue on other OS X machines, including a Retina MacBook Pro using the latest Atom release (1.0.7)

Confirmed as caused by Base16 Tomorrow Dark syntax them and fixed in my repo (viz. pull request above). Closing issue.


Per Request Cancellation Tokens For TSServer Do not Work

While working on https://github.com/Microsoft/vscode/pull/22437 that adds TSServer per request cancellation to VSCode, I keep hitting this error:
shell.ts:468 Error processing request. Mismatched request id, expected undefined, actual 4
Error: Mismatched request id, expected undefined, actual 4
    at Object.resetRequest (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/cancellationToken.js:57:27)
    at IOSession.Session.resetCurrentRequest (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:73291:40)
    at IOSession.Session.executeWithRequestId (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:73299:26)
    at IOSession.Session.executeCommand (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:73305:33)
    at IOSession.Session.onMessage (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:73325:35)
    at Interface.<anonymous> (/Users/matb/projects/vscode/extensions/node_modules/typescript/lib/tsserver.js:74477:27)
    at emitOne (events.js:96:13)
    at Interface.emit (events.js:188:7)
    at Interface._onLine (readline.js:232:10)
    at Interface.<anonymous> (readline.js:365:12)

It seems the expected request id is always undefined

@vladima thoughts?

Ported to release-2.2 in https://github.com/Microsoft/TypeScript/commit/d89553f63edab8d790e76afdfd4b541551732f6e


DwdWeather.query() doesn't returns a dictionary because 3 arguments are required

When using print(DwdWeather.query()) it only returns this message: TypeError: query() missing 3 required positional arguments: 'self', 'station_id', and 'timestamp' and not the expected list of all the possible keys.
I used Python 3.8.5 and the latest dwdweather2 release.

Dear Pnem,
the signature of DwdWeather.query() is
def query(self, station_id, timestamp, recursion=0):
    """
    Get values from cache.
    station_id: Numeric station ID
    timestamp: datetime object
    """

See also dwdweather/core.py#L533-L538.
However, I strongly recommend to move on to the Wetterdienst package. It is the successor library to dwdweather and dwdweather2 and is becoming more and more popular these days. We have been trying to get this en par with all the features offered by dwdweather2 and we have come pretty far these days. Enjoy!
With kind regards,
Andreas.
cc @gutzbenj


Adding tilde expansion and resolve to filepath for hashing

Closes #622 by adding expanduser and resolve to path handling.

PR https://github.com/openghg/openghg/pull/630 needs to go in before this.


Outputs correct Python version in manifest

Secure Coding Practices Checklist Link

https://github.com/RedHatInsights/secure-coding-checklist

Secure Coding Checklist

[ ] Input Validation
[ ] Output Encoding
[ ] Authentication and Password Management
[ ] Session Management
[ ] Access Control
[ ] Cryptographic Practices
[ ] Error Handling and Logging
[ ] Data Protection
[ ] Communication Security
[ ] System Configuration
[ ] Database Security
[ ] File Management
[ ] Memory Management
[x] General Coding Practices

Codecov Report
Patch and project coverage have no change.

Comparison is base (da41652) 72.94% compared to head (950f591) 72.94%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #330   +/-   ##
=======================================
  Coverage   72.94%   72.94%
=======================================
  Files          25       25
  Lines         743      743
=======================================
  Hits          542      542
  Misses        201      201

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Start Google Cloud Storage gcssync for experts

I’m following the instructions at Pub/Sub for Cloud Storage.  This is
combined with FIN gcsync service.

Before you begin
Starting at 4., topic name MUST match the bucket name.
PubSub,
select [x] Add default Subscription
[ ] Use a schema ?
[ ] Enable Message Retention?
Then, I added a mini subscription, which matches the GCSBUCKETNAME,
for the aggie-experts-fcrepo-mini topic/bucket
Go off book at item 5., not using the suggested service agent of the GCS
bucket manager.
I’m making aggie-expert-data-harvester the pub/sub admin for the project.
This makes most sense, as they are the ones writing to the GCS, or they will
eventually.  I gave this the role of PUB/SUB publisher.
Questions,

Why not use the default subscription aggie-experts-fcrepo-mini-sub?
Where are the dams subscriptions?

Notification Configuration
Setup notifications for new additions to our mini setup.
gcloud storage buckets notifications create gs://aggie-experts-fcrepo-mini --topic=aggie-experts-fcrepo-mini

FIN
Now, we’ll see if we can setup the gcssync w/ the passed configuration.
This first example tries to put everything at root.
{
    "sync" : {
	"containers" : [
	    {
	    "bucket" : "aggie-experts-fcrepo-{{GCS_BUCKET_ENV}}",
	    "basePath" : "/",
	    "direction" : "gcs-to-fcrepo",
	    "initDataHydration" : true
	}
	]
    }
}

This seems to work just fine.  I do get a number of errors during the
update, specfically from aggie-experts-gateway, but they don’t seem to cause
any issue.
{
    "name":"fin-server-generic-localhost",
    "hostname":"6cca38593553",
    "pid":19,
    "level":50,
    "err":{
	"message":"Cannot read properties of undefined (reading 'head')",
	"name":"TypeError",
	"stack":"TypeError: Cannot read properties of undefined (reading 'head')\n    at OcflObjectFile.#resolveContentPath (/fin/services/node_modules/@ocfl/ocfl/lib/object.js:69:70)\n    at async OcflObjectFile.asBuffer (/fin/services/node_modules/@ocfl/ocfl/lib/object.js:92:38)\n    at async DirectAccess.readOcfl (/fin/services/node-utils/lib/direct-access.js:290:21)\n    at async FinSearch.update (/fin/services/node-utils/lib/fin-cache.js:145:17)\n    at async FinSearch.onFcrepoEvent (/fin/services/node-utils/lib/fin-cache.js:49:9)\n    at async ServiceModel._onFcrepoEvent (/fin/services/gateway/models/services.js:390:7)\n    at async Subscription._onMessageCallback (/fin/services/node-utils/lib/activemq/stomp.js:185:11)"
    },
    "msg":"Cannot read properties of undefined (reading 'head')",
    "time":"2023-08-08T23:37:02.447Z",
    "v":0}


Give link to update ad sync service account

Official documentation now give procedure to change sync service account password

@jbpaux Thanks for this.  I did an edit pass and can we change it to the following:

[!WARNING]
If you change or reset the ADSync service account password, the Synchronization Service will not be able start correctly until you have abandoned the encryption key and reinitialized the ADSync service account password.
To do this, see Changing the ADSync service account password.

Then I can sign-off on it.  Thanks!

@jbpaux : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@MicrosoftGuyJFlo: I'm sorry - only the author of this article, @billmath, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


shorts aren't filtered in the search result of mobile youtube

What filter does this issue target?
https://letsblock.it/filters/youtube-shorts
What changes should be made?
Please add the rule to allow shorts filtering in mobile youtube.
Did you write a uBlock filter solving this?
m.youtube.com##ytm-search ytm-thumbnail-overlay-time-status-renderer[data-style="SHORTS"]:upward(ytm-video-with-context-renderer)
Additional notes
No response

Thanks @chedanix, I'm implementing it in https://github.com/letsblockit/letsblockit/pull/485. We already had a rule for mobile seach, but it targets a different container class. I just added a second container class to the upward search instead of duplicating the whole rule. This should lead to better performance.
Feel free to open a PR next time if you feel comfortable doing it!

Also, it seems like you have two github accounts: @chedani and @chedanix. I credited the first one as it's already in our contributor list, but I can switch to the second one if that's the one you plan on keeping.

Thanks @xvello. Please switch to @chedanix as the other one is my work account. The rule in the PR looks good. I didn't know you could use comma.

Done and done!
New version will be deployed in a couple of minutes. Your browser will get the updated rules in less than 24h, but you can manually refresh your list right now to grab it!


Update mpc_data_analysis.tex

All functions are mathematical - so simplified phrasing.

Edits and comments to .tex file


Changed name of the column in deliveries table

Changed name of the column in deliveries table

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How has this been tested?

[ ] Unit test(s)
[ ] Integration test(s)
[ ] Tested manually

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] I have checked my code and corrected any misspellings
[X] I have added frontend code and checked it for mobile & desktop

Task linked: CU-862jfe3b9 Delivery table changes


Distribution Request

Do you mind if we add this to the archassault repos? It'd be nice if it could have a license, we have been working on this one for a while.
Thanks

We would be happy to help with the cross compiling as we compile on i686, x86_64, armv6h and armv7h. If you would like the help.

@archey awesome ! big fan of archassault btw! would a BSD license be okay?

@archey also help with compiling would be more than welcomed :P

Sure sounds good to us, just make sure to include the license file so we can add it. Oh and thanks for the kudos, we do our best to pull in the latest mitmf as fast as we can. We compile on real arm hardware so its alot more stable then cross compiling, if you want some real help feel free to stop by freenode #ArchAssault and we can talk more

Did you compile it all from src or build  a pkgbuild? I only ask as we have a split pkgbuild that builds all of it, but its not working correctly so we haven't made it available yet

I actually did neither! this is basically a mish mosh of the original repo plus some stolen libraries and files from kali!

never could get the damn thing to compile!

yeah we did a bit of that as well, we got it to compile. Its just winexe fails with busted symbols Could not find symbol E_md4hash
Using the basic setup from kali's git repo, you should hop online if you want and we can talk about it more


storage_test: compilation failure with go 1.11

$GOPATH/src/github.com/cznic/ql/storage_test.go:423: dbg call has arguments but no formatting directives

The dbg does exist at the line you reported. I can easily remove it, but I believe the compilation failure in go1.11 is most probably a vet failure. dbg is not a well know function taking formatting string in its first arument, like fmt.Printf, for example.
I don't see any ql bug here. Please report this problem directly at the Go issue tracker. Thank you.

I think this is unfortunate false-positive but I fear a lot of people might come here for this.
Would dbg("Error %s", err) work as well?

Would dbg("Error %s", err) work as well?

Sure it will. But the language specification does not allow the Go compiler to reject a valid program, AFAICT. At least I recall I've been told that while discussing some other issue I forgot already about.
Anyway, if I forget a valid format directive in a call to fmt.Printf, log.Printf etc, I'm actually quite happy vet catches the problem. The problem is that there's no problem in a call to dbg with no formatting arguments. The function is this:
func dbg(s string, va ...interface{}) {
	if s == "" {
		s = strings.Repeat("%v ", len(va))
	}
	_, fn, fl, _ := runtime.Caller(1)
	fmt.Printf("dbg %s:%d: ", path.Base(fn), fl)
	fmt.Printf(s, va...)
	fmt.Println()
}

src
vet has no bussines in this case. Also, not every function with signature func(string, ...interface{}) is a printf-like function. That's why I believe this should be fixed in vet.
Well, and if the compiler guys reject that, I will be sad, but I will remove the offending line, of course. But please let's please first try to fix vet.

Thanks for looking into this -  I reported this as a probable error in go vet here: https://github.com/golang/go/issues/26486

@cznic I came here from the Go project issue.  The dbg function does indeed pass its argument to fmt.Printf, so the call to dbg("", err) does look wrong to me.  Why does it seem right to you?

@ianlancetaylor The code seen in https://github.com/cznic/ql/issues/203#issuecomment-406381754 handles specially the first argument of dbg when it's equal to "" in such a way that it becomes a valid format string of fmt.Printf.
It's not nice or idiomatic code, but I use the dbg a) for years b) as a quick and dirty debuging aid. In those years, I've typed the call probably some thousands of times. One will soon realize that automatically creating the right "default" format string is just making debugging a bit more comfortable.
tl;dr: The format string finally passed to fmt.Printf when the first argument to dbg is "" is a valid format string. I don't think vet is right when making such code fail the compilation and thus disabling go test to execute. Maybe I'm missing something, please let me know in that case.
It still holds that I'll fix it on my side if the decision is to keep vet "deep" inspecting user code w/o understanding that the "bad" case is actually handled correctly.

Ah, I see, sorry for missing that.
I think that on balance vet is making the right choice here but let's discuss that over on the Go project issue.

This should be now fixed at tip: https://go-review.googlesource.com/c/go/+/125039

I can confirm that go 1.11beta3 contains the fix. Thanks for your help!


DOC: read_excel() documentation for na_values should show default na values

I'd like to see read_excel show the same valid information for na_values as read_csv does. This ended up taking a lot of digging to find because it was unclear what the defaults were with read_excel().
read_csv() documentation shows the following for na_values
na_values : str or list-like or dict, default None
Additional strings to recognize as NA/NaN. If dict passed, specific per-column NA values. By default the following values are interpreted as NaN: ‘’, ‘#N/A’, ‘#N/A N/A’, ‘#NA’, ‘-1.#IND’, ‘-1.#QNAN’, ‘-NaN’, ‘-nan’, ‘1.#IND’, ‘1.#QNAN’, ‘N/A’, ‘NA’, ‘NULL’, ‘NaN’, ‘nan’.
read_excel() documentation does not include default NaN values
na_values : list-like, default None
List of additional strings to recognize as NA/NaN

Thx for the suggestion. PR is appreciated!

I wish to work on the issue. How do I start?

First, look at the contributing docs (http://pandas.pydata.org/pandas-docs/stable/contributing.html) which explains how to set-up a development environment and to make a pull request.
Specifically for this change, you can take a look at the read_csv docstring, which includes the correct values for na_values. So you could just copy the explanation from there.


--check flag isn't available

The documentation is pointing to a flag --check, but trying to run autoflake --check my_file.py I receive the following error:
usage: autoflake [-h] [-i] [-r] [--exclude globs] [--imports IMPORTS]
                 [--expand-star-imports] [--remove-all-unused-imports]
                 [--remove-duplicate-keys] [--remove-unused-variables]
                 [--version]
                 files [files ...]
autoflake: error: unrecognized arguments: --check

I'm on version autoflake 1.2
Is there a known work-around for this? I need this to include it in my CI integration. I would also be fine with the regular output of autoflake my_file.py, but the error code is always 0, even if it discovers changes...

This feature wasn't released to PyPI yet. To get the latest changes from Git, you can install from this repository rather than PyPI.
We've been due for a PyPI release so I went ahead and released the latest code to PyPI in version 1.3.


Server does not run in TravisCI

Send an issue and you are part of this great tool :)
Please describe the scenario you stuck in:

https://travis-ci.org/sourcerer-mike/crmp/jobs/133164174

What do you expect to happen?

WebServer runs in TravisCI.

What happened instead?

Server does not run.

This already works. Behat is testing the app.


Error downloading from KissManga

Bug report

Description
Data

OS
Ubuntu 18.04

Python version
3.6.5

Pip version
9.0.1

Manga-py version
1.5.4

URL
https://kissmanga.com/Manga/Binbougami-ga

To Reproduce:
Steps to reproduce the behavior:

Start downloading the manga
Once it gets to issue 19/20 it crashes
Profit?!

Additional context
98% (58 of 59) |##########################################################################################################  | Elapsed Time: 0:00:16 ETA:   0:00:00  File "/usr/local/lib/python3.6/dist-packages/manga_py/provider.py", line 88, in _check_archive
return not_allow_archive and is_file(_path)
File "/usr/local/lib/python3.6/dist-packages/manga_py/fs.py", line 57, in is_file
return Path(_path).is_file()
File "/usr/lib/python3.6/pathlib.py", line 1362, in is_file
return S_ISREG(self.stat().st_mode)
File "/usr/lib/python3.6/pathlib.py", line 1158, in stat
return self._accessor.stat(self)
File "/usr/lib/python3.6/pathlib.py", line 387, in wrapped
return strfunc(str(pathobj), *args)
100% (59 of 59) |############################################################################################################| Elapsed Time: 0:00:17 Time:  0:00:17
root@f307d70794fe:/Manga# manga-py https://kissmanga.com/Manga/Binbougami-ga
File "/usr/local/lib/python3.6/dist-packages/manga_py/provider.py", line 88, in _check_archive
return not_allow_archive and is_file(_path)
File "/usr/local/lib/python3.6/dist-packages/manga_py/fs.py", line 57, in is_file
return Path(_path).is_file()
File "/usr/lib/python3.6/pathlib.py", line 1362, in is_file
return S_ISREG(self.stat().st_mode)
File "/usr/lib/python3.6/pathlib.py", line 1158, in stat
return self._accessor.stat(self)
File "/usr/lib/python3.6/pathlib.py", line 387, in wrapped
return strfunc(str(pathobj), *args)

Hi. Thanks for the issue. I will check it

Hi. Did you check it? Are there any difficulties? Can I close this?

Works for me :+1:

Component
Value

OS
Ubuntu 16.04.6 LTS

Python version
Python 3.5.2

Pip version
pip 19.1.1

Npm version
3.5.2

Manga-py version
<IP_ADDRESS>

URL
https://kissmanga.com/Manga/Binbougami-ga

Closed as resolved


Unable to build on Windows

I'm running Node v14.0.0 and trying to use boardgame.io.
The README.md file says to install I should run npm install --save boardgame.io, however that returns an error:
[9:58:27 AM] D:/Code/mj/boardgame.io> npm install --save boardgame.io
npm ERR! code ENOSELF
npm ERR! Refusing to install package with name "boardgame.io" under a package
npm ERR! also called "boardgame.io". Did you name your project the same
npm ERR! as the dependency you're installing?
npm ERR!
npm ERR! For more information, see:
npm ERR!     <https://docs.npmjs.com/cli/install#limitations-of-npms-install-algorithm>

npm ERR!      C:\Users\smcro\AppData\Roaming\npm-cache\_logs\2020-04-25T01_59_36_055Z-debug.log
[9:59:36 AM] D:/Code/mj/boardgame.io>

The README.md also says I should run npm install; npm run build, however this also fails:
[9:54:45 AM] D:/Code/mj/boardgame.io> npm install

><EMAIL_ADDRESS>install D:\Code\mj\boardgame.io\node_modules\deasync
> node ./build.js

D:\Code\mj\boardgame.io\node_modules\deasync>if not defined npm_config_node_gyp (node "C:\Program Files\nodejs\node_modules\npm\node_modules\npm-lifecycle\node-gyp-bin\\..\..\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node "C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js" rebuild )
Building the projects in this solution one at a time. To enable parallel build, please add the "-m" switch.
  deasync.cc
  win_delay_load_hook.cc
     Creating library D:\Code\mj\boardgame.io\node_modules\deasync\build\Release\deasync.lib and object D:\Code\mj\boardgame.io\node_modules\deasync\build\Release\deasync.exp
  deasync.vcxproj -> D:\Code\mj\boardgame.io\node_modules\deasync\build\Release\\deasync.node
Installed in `D:\Code\mj\boardgame.io\node_modules\deasync\bin\win32-x64-node-14\deasync.node`

><EMAIL_ADDRESS>install D:\Code\mj\boardgame.io\node_modules\husky
> node husky install

husky > setting up git hooks
husky > done

><EMAIL_ADDRESS>postinstall D:\Code\mj\boardgame.io\node_modules\core-js
> node scripts/postinstall || echo "ignore"

Thank you for using core-js ( https://github.com/zloirock/core-js ) for polyfilling JavaScript standard library!

The project needs your help! Please consider supporting of core-js on Open Collective or Patreon:
> https://opencollective.com/core-js
> https://www.patreon.com/zloirock

Also, the author of core-js ( https://github.com/zloirock ) is looking for a good job -)

><EMAIL_ADDRESS>postinstall D:\Code\mj\boardgame.io\node_modules\docsify
> opencollective-postinstall

Thank you for using docsify!
If you rely on this package, please consider supporting our open collective:
> https://opencollective.com/docsify/donate

><EMAIL_ADDRESS>postinstall D:\Code\mj\boardgame.io\node_modules\nodemon
> node bin/postinstall || exit 0

Love nodemon? You can now support the project via the open collective:
 > https://opencollective.com/nodemon/donate

npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})

added 1633 packages from 847 contributors and audited 890206 packages in 95.93s

4 packages are looking for funding
  run `npm fund` for details

found 847 vulnerabilities (845 low, 2 moderate)
  run `npm audit fix` to fix them, or `npm audit` for details
[9:56:41 AM] D:/Code/mj/boardgame.io> npm run build

><EMAIL_ADDRESS>build D:\Code\mj\boardgame.io
> cross-env BABEL_ENV=rollup rollup --config rollup.config.js

packages/client.ts, packages/core.ts, packages/debug.js, packages/react.ts, packages/react-native.ts, packages/ai.ts, packages/plugins.ts, packages/master.ts, packages/multiplayer.ts, packages/internal.ts → dist/esm, dist/cjs...
[!] (plugin babel) Error: Cannot find module '@babel/compat-data/corejs3-shipped-proposals'
Require stack:
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>- D:\Code\mj\boardgame.io\node_modules\rollup-plugin-babel\dist\rollup-plugin-babel.cjs.js
- D:\Code\mj\boardgame.io\rollup.config.js
- D:\Code\mj\boardgame.io\node_modules\rollup\dist\bin\rollup
packages\debug.js
Error: Cannot find module '@babel/compat-data/corejs3-shipped-proposals'
Require stack:
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>- D:\Code\mj\boardgame.io\node_modules\rollup-plugin-babel\dist\rollup-plugin-babel.cjs.js
- D:\Code\mj\boardgame.io\rollup.config.js
- D:\Code\mj\boardgame.io\node_modules\rollup\dist\bin\rollup
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:1014:15)
    at Function.Module._load (internal/modules/cjs/loader.js:884:27)
    at Module.require (internal/modules/cjs/loader.js:1074:19)
    at require (internal/modules/cjs/helpers.js:72:18)
    at Object.<anonymous> (D:\Code\mj\boardgame.io\node_modules\@babel\preset-env\lib\polyfills\corejs3\usage-plugin.js:10:55)
    at Module._compile (internal/modules/cjs/loader.js:1185:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1205:10)
    at Module.load (internal/modules/cjs/loader.js:1034:32)
    at Function.Module._load (internal/modules/cjs/loader.js:923:14)
    at Module.require (internal/modules/cjs/loader.js:1074:19)

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: `cross-env BABEL_ENV=rollup rollup --config rollup.config.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\smcro\AppData\Roaming\npm-cache\_logs\2020-04-25T01_56_59_999Z-debug.log
[9:57:00 AM] D:/Code/mj/boardgame.io> node --version
v14.0.0
[9:58:27 AM] D:/Code/mj/boardgame.io>

This is a fresh install of node, and I have removed npm from my home directory in order to ensure there are no global modules installed.

 D:/Code/mj/boardgame.io> npm install --save boardgame.io
npm ERR! code ENOSELF
npm ERR! Refusing to install package with name "boardgame.io" under a package
npm ERR! also called "boardgame.io". Did you name your project the same
npm ERR! as the dependency you're installing?

You will  have to rename your project folder to add boardgame.io as dependency

I wanted to try the examples first.  Do you not need to install it?
Also, trying the examples failed for some reason (see the second command).

Did you git clone the boardgame.io repo?
Then you don't need to/can't do npm install --save boardgame.io because you already got all the files.

For the first problem,
Yes, I did clone the git repo.  I wasn't sure if npm install --save did something like save boardgame.io to my user profile directory, ensuring it was "installed".  I'll skip that for now.
For the second problem, I'm able to get a successful build after running npm install '@babel/compat-data'. So that does seem like a required installation step.

Hi @xobs, we’ve just merged a PR to fix Windows builds (#657).
To run the examples, you should clone the repository and run the following in the directory.
# install the dependencies boardgame.io needs
npm install
# start the examples server
npm start

The npm install --save boardgame.io is for adding the package to your own project, it can’t be run inside the boardgame.io directory.
I’m going to close this, but let us know if you run into any further issues!


Window doesn't show with taskbar in second screen

From Daniel https://groups.google.com/forum/?fromgroups#!topic/tomighty/d1BReS-xUww:

I'm using Tomighty in a Windows 7 system with two screens, but I have
the following problem: when the taskbar is placed in the second
screen, the main window of Tomighty doesn't appear (I guess it is
placed out of the bounds of the screen).
As a workaround, I move the taskbar to the first screen, start
Tomighty, select the option that allows dragging the window, drag the
window to the second screen, and move the taskbar to the second
screen. After this procedure, the window appears correctly in the
second screen when the taskbar is also placed in the second screen.
However, if I stop and start Tomighty, the window is hidden again.

+1


fix dedup.minScrapeInterval behavior in case of VMCluster replication

If replicationFactor is specified in VMCluster, the operator will add -dedup.minScrapeInterval=1ms to vmstorage and vmselect.
If vmselect has -dedup.minScrapeInterval option in extraArgs, the operator will not add -dedup.minScrapeInterval=1ms and will honor extraArgs.
problem: On the other hand, even if vmstorage has -dedup.minScrapeInterval option is extraArgs, the operator will add -dedup.minScrapeInterval=1ms. Instead, if vmselect has -dedup.minScrapeInterval option is extraArgs, the operator will not add -dedup.minScrapeInterval=1ms to vmstorage.
It's hard to imagine that only one of the extraArgs in vmselect and vmstorage has the dedup option, but I think it's a good idea to fix the process.
Signed-off-by: UMEZAWA Takeshi<EMAIL_ADDRESS>

Thanks for contribution!

Thanks for contribution!


The mute option on settings is turned on by default

I was not able to hear conversations when I joined rooms, so I checked my settings (for the first time), under the sound settings and I saw the mute option is turned on by default.

That sounds like a issue with your browser. Try giving the app voice permits

that setting actually means the sound you "hear" when you mute/unmute. a kind od a notification sound.


When displaying two or more MarkerClusters, adding/updating a marker that is already in cluster with new latlng causes error

[x] I'm reporting a bug, not asking for help
[ ] I'm sure this is a Leaflet.MarkerCluster code issue, not an issue with my own code nor with the framework I'm using (Cordova, Ionic, Angular, React…)
[x] I've searched through the issues to make sure it's not yet reported

First off, amazing how flexible MarkerCluster is and hard/clever work put in, this works totally fine when just displaying one MarkerCluster group, however, with multiple MarkerCluster groups an error occurs.
How to reproduce
When using two or more MarkerCluster groups which have markers/layers already added and then adding in a marker that is already added e.g. Marker[1234] with same latlng following section in MarkerClusterGroup.js gets an error due to parent not being defined.
				//First create any new intermediate parent clusters that don't exist
				var lastParent = newCluster;
				for (z = zoom - 1; z > parent._zoom; z--) {

In the image above it's clear the code gets to this point and fails to assign parent so then the if statement fails. You can see the parent value on the right side of the image. I see previous discussions about this, but I don't believe any discussions that the code should not end up with this error code because you are re-adding a marker with same or updated values. For example, a Marker with same latlng or with a different latlng will cause this error when multiple markerClusters but not with one. In this situation I could remove then add, but I believe that it would be better to allow updating the MarkerCluster and it should work the same as when only using one MarkerCluster. The common use case is real-time updates to various MarkerCluster groups. I wonder if a check needs to be put in place to skip this if parent is not defined.
MarkerClusterGroup.js:1022 Uncaught (in promise) TypeError: Cannot read properties of undefined (reading '_zoom')
Related issues are
https://github.com/Leaflet/Leaflet.markercluster/issues/860
https://github.com/Leaflet/Leaflet.markercluster/issues/635

Leaflet version I'm using: 1.7.1
Leaflet.MarkerCluster version I'm using: 1.5.3
Browser (with version) I'm using: Chrome 98.0.4758.102 (Official Build) (64-bit)
OS/Platform (with version) I'm using: Windows 10
also using this with Svelte (Javascript)

What behaviour I'm expecting and which behaviour I'm seeing
Adding a marker that is already in a layer should not cause error. This issue does not occur with one MarkerCluster group, but with two MarkerCluster groups. I would expect the parent to be defined. If having multiple MarkerClusters and then update one of them actually works elsewhere please point out the example.
Minimal example reproducing the issue
If what I am writing makes sense that it should be possible to handle adding a marker that is already in a MarkerCluster or updating a MarkerCluster that it is already in, I'll create an example but might not be simple.

[ ] this example is as simple as possible
[ ] this example does not rely on any third party code

Using http://leafletjs.com/edit.html or any other jsfiddle-like site.

I have implemented a solution to clearLayers() for each cluster and then add each set of Markers back to each cluster. This isn't ideal for real-time data as any update will close the popup since the marker is briefly removed and then added back in. I tried the following settings for a new popup to see if it just stays around even when removed, but does not work. L.popup({"autoClose": false, "closeOnClick": null})

I have same problem.
When i destroy map, clear everything this what i can.
On second instance i getting error
Cannot read properties of undefined (reading '_zoom')

When i adding marker One by one by addLayer method, when i use addLayers, i don't have any error, but still no markers on map.

I used this package https://www.npmjs.com/package/use-supercluster and it works fine


The demo of nth-of-type styled props is incorrect

Description
As title, the nth-of-type styled props seems not working in demo.

Root cause
The usage in sample is incorrect.
  <Box
    key={item}
    py="2x"
    px="3x"
    _nthOfType={['odd', {
      bg: 'blue:40',
      color: 'white:primary',
    }]}
  >
    {item}
  </Box>

correct one:
  <Box
    key={item}
    py="2x"
    px="3x"
    _nthOfType={{'odd':  {
      bg: 'blue:40',
      color: 'white:primary',
    }}}
  >
    {item}
  </Box>

Environment

mac OS Big Sur

Chrome, Firefox

Updated with PR-574


test as student promotion

fix ENG-925

Summary by CodeRabbit

New Features

Introduced a modal dialog for student promotions in the teacher dashboard.
Added text details about testing the student experience, progress tracking, and free courses to the curriculum section.

Enhancements

Integrated the new student promotion modal into the teacher dashboard navigation.

this can only be merged once this is merged: https://github.com/codecombat/codecombat/pull/7578