[Bug]: country data phone code is incorrect for india

What happened?
in database seeder phone code for india was incorrect
How to reproduce the bug
follow installation steps
Package Version
2.0
PHP Version
8.3
Laravel Version
11.32
Which operating systems does with happen with?
Windows
Notes
No response

hi @inishantthakur can you help us with the correct one?

correct codes are available at https://countrycode.org/ also i noticed other countries codes are also not correct... like unites states is also not correct..

here is another link with data being in json format https://gist.github.com/anubhavshrimal/75f6183458db8c453306f93521e93d37


Add CA data to service account tokens if missing or different

If the token controller encounters a service account token secret with missing or mismatched CA data, it should update it

@mikedanese PTAL

Is there a way to know which version of a secret a given container has mounted?  Even if we took no action, it would be nice to know which Pods have outdated secrets.

Is there a way to know which version of a secret a given container has mounted? Even if we took no action, it would be nice to know which Pods have outdated secrets.

I don't think so. @pmorie, do you know?

/cc @rrati

@liggitt

Is there a way to know which version of a secret a given container has mounted? Even if we took no action, it would be nice to know which Pods have outdated secrets.
I don't think so. @pmorie, do you know?

There's an issue for that: #4949

I'll review today

This is LGTM thanks. @k8s-bot ok to test.


add select multi select overflow and filtering

when using multi select with a hierarchy the item columns need to support overflow and individual filtering. invision design for reference-

@davidmenendez Can we close this issue?


Set a BMP- File as Logo on POS-Devices

Payment service provider
CCV
Payment terminal
Basenext
Communication Logs
No response
More information
Hello,
can I send a logo file in POS device? If yes, how and in which format (size, bmp, png etc) can i send.
Thanks

No sorry

ok thanks


Multi-day events

Would be great to have multi-day events, this is a functionality in many calendars. E.g. if you need to block 3 days for a congress. This could also be helpful for project management - blocks of time for subtasks.
Thanks for considering it.

You can actually already do this by updating the end date of the event in the front matter.

the attribute is named: endDate


Migrate localStorage content from draw.io to diagrams.net

Copy everything from draw.io domain in localStorage and indexdb (probably using hidden iFrame) to indexdb on diagrams.net one time and mark somewhere that’s happened so it’s not done again

In the meantime, is there a manual process that can do this?

Done, will be available in the next release


Expose enableNetwork/disableNetwork methods

I have already made this request for the Firebase JS SDK, and it has since been implemented as described here and defined here. Next up, I would love to see this in AngularFirestore.
The Feature
Firestore has enableNetwork and disableNetwork methods that allow you to control your connection state with the online server. For many reasons, I would like to be able to disable the network on my app so that it is not constantly syncing or connecting to an outside server. This would ensure that all the data in the app is only the data in the client. The client would then only be able to connect to the external server again when enabling the network.
Use cases
A simple feature from my app:
My app is used by customers who often need to use our product when cell networks are not available. Prior to this offline period, the user activates a sync button to pull their most recent documents for download to view offline. While offline, they may view and edit these documents. When they regain connection, they manually sync again to upload their changes.
We prefer an explicit execution of the sync as opposed to the automatic syncing for a few reasons:

Customers themselves enjoyed the manual syncing more
Manual syncing requires less data
It makes it easier on our end to resolve conflicts and monitor the system
It makes it easier for us to maintain consistency with our desktop model without having to completely refactor to introduce Firebase
Testing is easier
It is easier to keep the app responsive while completely offline

I don't think we intend to wrap this API, but you should be able to access the method via the initialized firestore instance we expose, assuming you constructed with the name afs like so:
this.afs.firestore.disableNetwork()
Is that satisfactory or did you have something else in mind?

@connor-odoherty We will either document this or add a wrapper method. Would you be for tapping into the underlying Firestore instance or would you want a wrapper method?
constructor(private afs: AngularFirestore) {
  afs.firestore.disableNetwork();
}

Hmm, are there any Angular event cycle issues we could run into if we access the Firestore instance directly instead of in the context of Angular Firestore? I'm assuming not, but I'm still relatively fresh in my Angular knowledge

Hey @connor-odoherty! No event cycle issues, it's just the matter of aesthetics.
Thank you so much for your kind words! Keep filing those issues 👍


error: The following binaries use incompatible versions of Swift:

error: The following binaries use incompatible versions of Swift:
RealmSwift.framework/RealmSwift
/Users/xxx/Library/Developer/Xcode/DerivedData/App-cgazqddnhxynjacblakqrxuhwpcc/Build/Products/Debug-iphonesimulator/App.app/
Xcode 8,Swift 3

Swift requires that all dependencies be built with the same version of the compiler.


Shared caching layer for thanos queriers

Is your proposal related to a problem?
We are running about 10 Thanos querier replicas for scaling purposes and we have 100+ sidecar + prometheus edge clusters across the world.
For our setup, the fanout problem is huge because of the scale. For example:
Info requests to sidecars
This is not a big problem because Info Request and Response are relatively cheap. In our setup, (number of queriers x number of sidecars) requests are sent every time. It is okay when scale is small. However, when you have more and more Thanos Queriers and edge sidecars, this is not very efficient.
metadata and rules query requests to sidecars
metrics metadata and rules query is something hardly changed for us. Especially metrics metadata. This is where caching would benefit us a lot.
more use case in the future
From https://github.com/thanos-io/thanos/issues/1611, we proposed to have some bloom filter like datastructure for reducing unnecessary series calls. Ideally, this could be done by introducing more data reported from the Info API and  keep a bloom filter in queriers. If we can have a caching layer for the querier clusters then keeping the bloom filter up-to-date is not that expensive anymore.
Describe the solution you'd like
Have another type of cache for this use case. Maybe call it proxy cache? It is similar to caching bucket but this time we cache endpoint responses.
Also I think the new galaxy cache is very suitable for this usecase.
Describe alternatives you've considered
Have some kind of gRPC proxy to do caching/passthrough based on the requests. I don't do any investigation right now but maybe something already suits my usecase.

So, something like galaxycache but for gRPC calls? Did I understand you correctly?

So, something like galaxycache but for gRPC calls? Did I understand you correctly?

Yes

Is this commit version usable (in production)?
https://github.com/thanos-io/thanos/issues/5047#issuecomment-1012146576


Question about generation speed

Hi! Can someone explain why different tools using the same models show dramatically different results? I tried generating images using Draw Things and mflux with the same settings and prompt and got the following results:

Draw Things: ~1m20s
mflux with -q 8 setting: ~3m30s
mflux without -q setting: ~7m

I have a weak laptop (MacBook 2021, M1 Pro, 16GB), but the difference in performance is significant. Maybe I make something wrong?
Thank you.

@TuiKiken Hi. I have not looked into Draw Things but it is indeed impressive on their end if the difference is that large for similar settings. The generation speed depends on a few factors, such as the hardware, image resolution, quantization etc. The more RAM available the faster it will run (so closing other application in the background can help). If maxing out the memory it will typically resort to swap memory (disk storage) which slows things down.


Post debug task to stop hugo

When you stop debugging, the Hugo watch process isn't terminated. This can cause problems when starting a new debug session.
There may be a cleaner way of doing this. I've added a postDebug task which

Tries to extract the id for the Hugo watch process - ps -ef | grep -e ' hugo -w$'
Calls kill on that process id

See https://github.com/azurecitadel/azurecitadel/pull/83


Incorrect implementation of bool rxor

https://github.com/mwerezak/sphinx-lang/blob/f351b2298e6f02a3c02d42f288f6fba33e38716c/src/runtime/types.rs#L224-L226
it is using op_and. This looks like a copy and paste error.

 Thanks for catching this.


fix: deep_merge shows false errors when using key names that happen to be Object prototype methods

#1572 introduced a couple of minor issues. One I fixed in #1698 but there was still a minor issue where, if your config has keys which happen to be Object prototypes like toString, valueOf, etc, you'd get false conflict errors reported. It's a simple one-line fix. Not super high priority and unlikely anyone would every hit that issue but good to be robust.
Before submitting the PR, please make sure you do the following

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[x] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts

Maybe I don't need all those block comments, I can remove if you like.

I think the code comment is good. I'd probably remove the test comments since they refer to things that aren't issues anymore


mbedtls_net_set_nonblock / mbedtls_net_set_block doesn't work (IDFGH-2264)

Latest master (93a8603c545fb8e54741d6685146e2f3b874378d) using 2019r2 toolchain.
mbedtls_net_set_nonblock / mbedtls_net_set_block doesn't work.
Steps to reproduce: try using mbedtls_net_set_nonblock / mbedtls_net_set_block
Apparently it's because HAVE_FCNTL was not defined when building the C library.
https://github.com/espressif/newlib-esp32/blob/esp_based_on_3_0_0/newlib/libc/syscalls/sysfcntl.c

I've hacked around the C library problem, and it seems to work, but perhaps there are other issues with nonblocking sockets and that's why fcntl was disabled in the first place?

@szmodz can you please share a short code snippet with comments highlighting the issue?

This code doesn't work at all with the 2019r2 toolchain:
https://github.com/espressif/esp-aws-iot/blob/0ba1fe942785b2efd553d73124afaef2f0327f41/port/network_mbedtls_wrapper.c#L208
It always fails with

ESP_LOGE(TAG, "failed! net_set_(non)block() returned -0x%x", -ret);

If you need test cases, here they are (Espressif's own code):
https://github.com/espressif/esp-aws-iot/tree/master/examples
The problem also applies to esp-tls:
https://github.com/espressif/esp-idf/blob/93a8603c545fb8e54741d6685146e2f3b874378d/components/esp-tls/esp_tls.c#L191
Basically anything that uses esp-tls in nonblocking mode is broken with the gcc 8.2 based toolchains. It just goes unnoticed because the call is not checked for errors.

Any ACK on this? I think this problem needs to be fixed, as it affects multiple esp-idf components, but I can keep posting bug reports... ;-)

This is why it works with gcc 5:
https://github.com/espressif/esp-idf/blob/93a8603c545fb8e54741d6685146e2f3b874378d/components/vfs/vfs.c#L728
(attribute weak)
https://github.com/espressif/esp-idf/commit/48d9d507aaf50bbac023ac9fa888cb28c6bdf2c8

@szmodz

Any ACK on this? I think this problem needs to be fixed, as it affects multiple esp-idf components, but I can keep posting bug reports... ;-)

Thanks for reporting this issue. Fix has been merged for this internally, once ESP-IDF codebase sync with github, this issue will be marked as closed.


Refactor from kroxylicious futures to JDK futures alternative

Type of change

Refactoring

Description
Remove the kroxy futures implementation and use CompletableFuture internally.
Wrap the CompletableFuture in our own InternalCompletionStage when passing it to a Filter so that we can throw an UnsupportedOperationException when the Filter Author tries to convert it to a CompletableFuture.
An alternative to #184
Additional Context
We don't want to maintain our own futures code at this point. The kroxy implementation has the advantages of a Promise/Future separation with no blocking methods available to shoot yourself in the feet with. By preventing the Author working with a CompletableFuture they will be encouraged to not block in their filter and avoid doing other odd behaviour like completing the future unexpectedly.
A downside is this introduces another object that needs to be garbage collected.

@tombentley what do you think of this implementation vs https://github.com/kroxylicious/kroxylicious/pull/184
This implementation is to decorate the CompletableFuture with our own CompletionStage implementation that throws if they try to call toCompletableFuture(). The other PR uses a CompletableFuture subclass instead, with all the blocking and completion methods overridden to throw exceptions.


Error on build: Cannot start Firefox

I followed the build instructions and did npm install then grunt.
The grunt process stopped with this error
Running "karma:unit" (karma) task
INFO [karma]: Karma v0.12.31 server started at http://localhost:9876/
INFO [launcher]: Starting browser Firefox
ERROR [launcher]: Cannot start Firefox
        Can not find the binary C:\Program Files\Mozilla Firefox\firefox.exe
        Please set env variable FIREFOX_BIN

I am running on Windows, with no Firefox.
Do I have to install firefox, or is there a way around this?
EDIT:
I edited the Gruntfile and uncommented the kindly provided browsers: ['Chrome'], line which was commented out. Is that the correct way? Is my build process done now?

Karma is the test runner, which is run by default when just calling grunt. It needs a browser to run the tests in. If you just want to build PIXI, you can use grunt build (or build-debug) which doesn't run any unit tests.
See: https://github.com/GoodBoyDigital/pixi.js/blob/master/Gruntfile.js#L214

Yeah, I am not using grunt on a regular basis, so was not sure.
In fact, I think I do not need to build at all... I was under the impression that I need to build in order to start developing in Pixl, but I then realized I just need the bin folder, and start with one of the examples. Correct me if otherwise please.
If I am correct, perhaps this should be mentioned somewhere prominent both in the main README and on the pixljs.com site.

You really only need to build if you make changes to Pixi itself. If you're just going to use it in a project, you can simply grab the prebuilt JS file in the bin directory, or under "Releases". :smile:
@GoodBoyDigital As @DannyBen suggests, maybe there should be a "Note: You do not have to build Pixi yourself to start using it. Just use one of the prebuilt files in bin" under "Build" in the readme?

Thanks, I will now have tinted bunnies rotating on my screen in no time :)

We will update the v3 readme to make it much more clear to get and use pixi. Right now, you have to build v3 so we haven't added those docs yet.

This should be good to go now.


Fix documentation for building RAG

The docs for the example on "Building RAG Application" here should follow the current API. The current example code fails.

Thank you for the PR!


Touchables not working inside Card

Since updating past v1.14.0, it looks like pointerEvents="box-only" is getting set on multiple components which prevents nested "touchables" from being clicked (TouchableHighlight, Button, etc etc).
I think that should be either set to "auto" or let a user pass that prop down (instead of forcing box-only which prevents any nested elements from being clicked).
I.e.
<Card pointerEvents="box-only"></Card>

I'm also seeing this problem. Perhaps related to https://github.com/xotahal/react-native-material-ui/pull/239


fix: supports null values in query strings

Description
When retrieving a document using the REST API and a where clause wherein exists: false and field equals null:
const query = {
  or: [
    {
     '[field]': {
        exists: false,
      },
    },
    {
     '[field]': {
        equals: null,
      },
    },
  ],
}

Using API's for finding:
// Server
const { docs, totalDocs } = await payload.find({
  collection: 'my-collection',
  where: query,
})

// Returns
[
  {
    id: '65e6192f3e8aae7eda734563',
    test: null,
    createdAt: '2024-03-04T18:55:43.945Z',
    updatedAt: '2024-03-04T18:55:43.945Z'
  }
]

// REST
const queryString = QueryString.stringify({
  where: query
}, {
  addQueryPrefix: true,
})

const { docs, totalDocs } = await fetch(`${apiUrl}/my-collection${queryString}`, {
  method: 'GET',
  credentials: 'include',
}).then((res) => res.json())

// Returns
[ ]

Solution
Added strictNullHandling to query string middleware
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] Existing test suite passes locally with my changes

This might need to be added to the Relationship component as well for instance.

Hey @denolfe we are experiencing this exact issue with our implementation, do you think there is any chance of this reaching a release soon or alternatively do you have any advice for how we could work around?

Hey @denolfe we are experiencing this exact issue with our implementation, do you think there is any chance of this reaching a release soon or alternatively do you have any advice for how we could work around?

@tomgoldsmith Yes, we will review this on Monday 👍

Awesome, thank you


chore(master): release 1.2.29

:robot: I have created a release beep boop
1.2.29 (2023-07-09)
Miscellaneous

deps: update dependency ergebnis/composer-normalize to v2.33.0 (7529673)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/php-cs-fixer-config/releases/tag/v1.2.29 :sunflower:


Enable SSL on postgrest.com

Get a cert and configure Read The Docs to use it

Can't you set up LE?

That's an option yeah. I use LE for my own domain, although it requires updating the cert with a cron job which is automated by my server. For the docs I may have to put it on my calendar to update the cert and save the changes on Read The Docs periodically.

A cron job can be setup to do that too. If you use DNS based verification and assuming setting cert on RTD can be scripted. I'm happy to help figure this out if you are interested.

Yeah I am interested. Can you provide me a checklist of what I would need to do?

I spoke too soon, and without thinking things through. Unfortunately, reading from http://read-the-docs.readthedocs.io/en/latest/alternate_domains.html#cname-ssl, looks like this isn't possible with RTD's hosted version directly, and for good reasons.
There is no option to make RTD generate its own LE cert and use it, either. Since they say they have a shortage of developers and ops people, it is unlikely that they'll consider implementing it, since it'll add to their support load.
I'm curious though, other than "having ssl is good", do we have any specific threat model we are trying to guard against? Postgrest is not a project of sensitive nature, and not having ssl shouldn't be an issue IMO.

Maybe we need to protect postgrest users from the ridicule of "web devs" and stigma that goes along with putting you "business logic" in the database, everybody know this does not scale :)

@phanimahesh you're right, we have no real threats -- I don't expect anyone to do a man in the middle attack to change the content of the docs.
SSL is just basic web hygiene, a nice-to-have. It leaves a bad first impression I think that postgrest.com is not secure. I'm surprised RTD doesn't support SSL for custom domains.

@phanimahesh then again, look at https://docs.haskellstack.org/en/stable/README/ for example. It's read the docs but has SSL...

Question, would a normal certificate help or an actual separate aws instance running the site?

Their service is nice because it has a control panel and git hooks so that I can activate certain branches as being public versions and when I push to them it builds and deploys. Not that we couldn't figure out a way to do it, but their setup is nice. Maybe could proxy it behind CloudFlare or something which can add SSL to sites.

I think that will do it actually. I switched nameservers to cloudflare and enabled ssl. It will probably take a little while to propagate. We'll see if it automatically gets SSL.

It's alive! https://postgrest.com/en/v0.4/

a redirect would be nice


Added load_only to query for GPS to road segment

When pulling the road segment info from lat/lon coordinates, there are two relevant functions, get_db_matching_location_interurban and get_road_segment_id. The one causing the slow processing is the get_db_matching_location_interurban functions.
Within it, three lines are causing the most delay: the query to the database, and the addition of two new columsn with calculations.
This PR is an update to the query code so that it runs faster.
Of the rows that add columns, both would run faster if the table was shorter (it currently returns 2078 rows). As there are only 1006 segment (by the table on redash), perhaps this could be slimmed down either by a more complicated query or by limiting the year (eg from 2017 instead of 2014).
Finally, the slowest  part of the code by far is the calculation of the distance point and adding it to each row. If there is some way to make the geod.Inverse function more efficient, that would also help speed up the run.

@aentin16 as discussed please make sure you use black for formatting

Ignore this for now - it started throwing errors

@aentin16 can I close this one?

yes - closing


Support for commands with a variable number of parameters, e.g. HMSET

Hi Ben
Is the HMSET operation supported?  This doesn't seem to work, using a Vec<(String, String)> type:

        let items: Vec<(String, String)> = vec![("one".to_string(), "uno".to_string()),
                                               ("two".to_string(), "dos".to_string()),
                                               ("three".to_string(), "tres".to_string())];

        resp_array!["HMSET", "mykey", items]))

Update:  Unfortunately, the solution is a bit more challenging:
    let items = vec![("one".to_string(), "uno".to_string()),
                     ("two".to_string(), "dos".to_string()),
                     ("three".to_string(), "tres".to_string())];

    let parts: Vec<RespValue> =
        vec!["HMSET".into(), "mykey".into()].into_iter()
        .chain(items.into_iter().flat_map(|(a, b)| once(a.into()).chain(once(b.into()))))
        .collect();

    let send_fut = redis.send(Command(RespValue::Array(parts)));

Yes, this is a bit of an edge case, when the number of parameters in a Redis command is variable it's a little bit ergonomic (well, quite painful) to construct the command.
If you don't mind, I'm going to re-open this issue as a reminder that a better way of handling these situation is needed.

I've just published version 0.4.1 which includes an append function to aide this type of scenario.
So now it's possible to do:
resp_array!["RPUSH", "keyname"].append(vector_of_values)

in addition to using resp_array! the usual way.

@benashford that's much better!


test to create a couple of variants

Given I am a logged in user
And I create a new experiment
When I select optional field
And I create the experiment
Then These show up in the created experiment

This is done and moved to closed with PRs:
https://github.com/timeades/Phrasee_tech_test/pull/13
https://github.com/timeades/Phrasee_tech_test/pull/14


[SPARK-36576][SS] Improve range split calculation for Kafka Source minPartitions option

What changes were proposed in this pull request?
Proposing that the KafkaOffsetRangeCalculator's range calculation logic be modified to exclude small (i.e. un-split) partitions from the overall proportional distribution math, in order to more reasonably divide the large partitions when they are accompanied by many small partitions, and to provide optimal behavior for cases where a minPartitions value is deliberately computed based on the volume of data being read.
Why are the changes needed?
While the documentation does contain a clear disclaimer,

Please note that this configuration is like a hint: the number of Spark tasks will be approximately minPartitions. It can be less or more depending on rounding errors or Kafka partitions that didn't receive any new data.

there are cases where the calculated Kafka partition range splits can differ greatly from expectations. For evenly distributed data and most minPartitions values this would not be a major or commonly encountered concern. However when the distribution of data across partitions is very heavily skewed, somewhat surprising range split calculations can result.
For example, given the following input data:

1 partition containing 10,000 messages
1,000 partitions each containing 1 message

Spark processing code loading from this collection of 1,001 partitions may decide that it would like each task to read no more than 1,000 messages. Consequently, it could specify a minPartitions value of 1,010 — expecting the single large partition to be split into 10 equal chunks, along with the 1,000 small partitions each having their own task. That is far from what actually occurs. The KafkaOffsetRangeCalculator algorithm ends up splitting the large partition into 918 chunks of 10 or 11 messages, two orders of magnitude from the desired maximum message count per task and nearly double the number of Spark tasks hinted in the configuration.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Existing unit tests and added new unit tests

Can one of the admins verify this patch?

ok to test

@noslowerdna Thanks for the contribution!
We leverage Github Action in individual's fork to run test in Github. Could you please follow the instruction below?
https://spark.apache.org/developer-tools.html#github-workflow-tests

The new calculation looks better to me, but let's have double confirmation.
cc. @tdas @zsxwing @viirya @gaborgsomogyi @xuanyuanking

Test build #142757 has started for PR 33827 at commit cbf0701.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47257/

Test build #142757 has finished for PR 33827 at commit cbf0701.

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

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

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47257/

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

@HeartSaVioR Thank you for the review. I think the documentation at https://spark.apache.org/developer-tools.html#github-workflow-tests needs to be slightly updated for this recent change: https://github.com/apache/spark/commit/2974b70d1efd4b1c5cfe7e2467766f0a9a1fec82?  I don't see a manual "Run workflow" option for the Build and test action, like the Benchmark action does have, as Benchmark still has a workflow_dispatch event trigger. Anyway after enabling the workflow for my fork, I am seeing a Build and test workflow run automatically queued now for new commits so it looks like the tests should be running soon.

Test build #142766 has started for PR 33827 at commit 75dc94d.

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47266/

Test build #142766 has finished for PR 33827 at commit 75dc94d.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SparkClassNotFoundException(
class SparkConcurrentModificationException(
class SparkDateTimeException(errorClass: String, messageParameters: Array[String])
class SparkFileAlreadyExistsException(errorClass: String, messageParameters: Array[String])
class SparkFileNotFoundException(errorClass: String, messageParameters: Array[String])
class SparkNoSuchMethodException(errorClass: String, messageParameters: Array[String])
class SparkIndexOutOfBoundsException(errorClass: String, messageParameters: Array[String])
class SparkIOException(errorClass: String, messageParameters: Array[String])
class SparkRuntimeException(
class SparkSecurityException(errorClass: String, messageParameters: Array[String])
class SparkSQLException(errorClass: String, messageParameters: Array[String])
class SparkSQLFeatureNotSupportedException(errorClass: String, messageParameters: Array[String])
case class AttachDistributedSequence(
case class AttachDistributedSequenceExec(

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

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

retest this, please

Test build #142848 has started for PR 33827 at commit aa5a2fa.

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47351/

Test build #142848 has finished for PR 33827 at commit aa5a2fa.

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

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

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

Thanks @noslowerdna for the contribution! I merged into master.

Nice improvement, thanks @noslowerdna!


Use knative.dev/test-infra

@chaodaiG This PR used an older version of dep that made unnecessary changes to Gopkg.lock. Can you run hack/update-deps.sh with the new version of dep and create a new PR so we can revert the unnecessary changes? Thanks!

sure, done


[site/blog][s]: markdowndb launch post

@olayway and please just merge once you have acted on feedback (or not) so we keep this moving rapidly 😄


[ENHANCEMENT] Alerts: remove metrics for removed Alertmanagers

So they don't continue to report stale values.
Fixes #6864

I see. Thank you for the updated PR. I guess it will work now. But adding a test would be better, of course.

I think this one can be merged.

@bboreham Gonna leave it up to you to do the final merge, just because it's been sitting here for a while and I want to make sure you're still happy with it all as well.


Modernize flask example

Hey there,
Thanks for making this!  It has been very helpful.   I had a a little trouble getting the flask example to run so I updated it to use the flask cli.  This makes sure app context was preserved during DB creation and population and simplifies the instructions.

Thanks, I added it to the revamp https://github.com/Pegase745/sqlalchemy-datatables/pull/110/commits/49fed6a0bb869734e839bd0097f4f0b04fc54a65


fix: Added multilingual and indic benchmark

This adds outlines for the multilingual and indic benchmarks as well as a few adjustments to existing benchmarks due to a few discovered bugs in the previous implementation.
It also add utilites for easily creating a table from a list of tasks:
tasks = mteb.get_tasks(tasks=[...])
tasks.to_latex()
tasks.to_dataframe()

Checklist

[x] Run tests locally to make sure nothing is broken using make test.
[x] Run the formatter to format the code using make lint.

Merging this in for now -  we will do a final review on the benchmark at the end


Fix setAttrs runs before head is ready

:link: Linked issue
resolve #75
:question:  Type of change

[ ]   :book:  Documentation (updates to the documentation or readme)
[x]   :lady_beetle:  Bug fix (a non-breaking change that fixes an issue)
[ ]   :ok_hand:  Enhancement (improving an existing functionality like performance)
[ ]   :sparkles:  New feature (a non-breaking change that adds functionality)
[ ]   :warning:  Breaking change (fix or feature that would cause existing functionality to change)

:books:  Description
Nuxt3 runs the updateDOM function directly before the head has htmlAttrs, and this is causing to clear existing attributes in the head; this happened during hydration

Please revert the format changes. Thanks.

Could you add some tests to it?

I tested this by patching it in my project (using yarn patch), and it didn't fix the issue. See video (the background is initially green to emphasize the flash).
https://user-images.githubusercontent.com/12532733/187232695-14d97813-ec03-4e45-b320-f517e87b36a5.mp4

Hey  @christian4leaflabs, If you're still interested in solving this one could you address the above comment and maybe add some tests in the nuxt3 e2e
Will close for now but please re-open if / when changes are made :)

This isn't stale, it's still an issue. And there's no currently proven fix that I'm aware of.

The issue isn't stale (https://github.com/vueuse/head/issues/75), the PR is though as it hasn't been updated since July without any progress towards merging it.

This PR doesn't solve the issue, anyway (I tried it). The only thing that worked for me was my debouncer to limit how many times the update method can be called, and this prevents the flicker.

Thanks for the details @ffxsam, would you mind sharing this code snippet in the issue https://github.com/vueuse/head/issues/75, it will be helpful when I attempt to fix it.

@harlan-zw Of course! I can't guarantee it's the most elegant. 😅 I was in a hurry and needed a fix, but maybe you can improve upon it.
Keep in mind, this is a yarn patch, so it's on the bundled index.mjs file, not the original source code. Also, this patch includes the fix for the &amp issue.

Diff file

diff --git a/dist/index.mjs b/dist/index.mjs
index 481b7343c20357bc198ebec977c5daaff0c25e24..ecfcaf61faece3b0a3a91fd14e60d7f20c947f10 100644
--- a/dist/index.mjs
+++ b/dist/index.mjs
@@ -57,8 +57,18 @@ var createElement = (tag, attrs, document) => {
   return el;
 };

+var debounce = (fn, delay = 0) => {
+  let timer;
+  return (...args) => {
+    clearTimeout(timer);
+    timer = setTimeout(() => {
+      fn(...args);
+    }, delay);
+  };
+};
+
 // src/stringify-attrs.ts
-var htmlEscape = (str) => str.replace(/&/g, "&").replace(/"/g, """).replace(/'/g, "'").replace(/</g, "<").replace(/>/g, ">");
+var htmlEscape = (str) => str.replace(/"/g, """).replace(/'/g, "'").replace(/</g, "<").replace(/>/g, ">");
 var stringifyAttrs = (attributes) => {
   const handledAttributes = [];
   for (let [key, value] of Object.entries(attributes)) {
@@ -300,7 +310,7 @@ var createHead = (initHeadObject) => {
     removeHeadObjs(objs) {
       allHeadObjs = allHeadObjs.filter((_objs) => _objs !== objs);
     },
-    updateDOM(document = window.document) {
+    updateDOM: debounce((document = window.document) => {
       let title;
       let htmlAttrs = {};
       let bodyAttrs = {};
@@ -332,8 +342,8 @@ var createHead = (initHeadObject) => {
       }
       previousTags.clear();
       Object.keys(actualTags).forEach((i) => previousTags.add(i));
-    }
-  };
+    }, 500),
+  }
   return head;
 };
 var IS_BROWSER = typeof window !== "undefined";
</details>


Fitness function

Could you please shed some light on Fitness function that is being used here?

Actually, you can choose which fitness function you would like to use in the "scoring" parameter. By default it uses the accuracy, so the individuals (or hyperparameters) that survive will be the ones with the best accuracy.


gitignore: ignore local copy of bids-examples

In the context of the CI runs, the bids-examples repo is downloaded and installed into tests/. It might be nice to support this workflow in local interactive usage, too. Adding it to the gitignore lets developers drop bids-examples in the same place as the test code is expecting it, without accidentally checking it into git on the main repo.

The Travis CI build failed on this, but that looks like just an npm timeout, not related to this PR's contents.


Reactive support for subscriptions

Thanks for the awesome library!
I was wondering if support for reactive subscriptions can be added. For example, instead of doing
Subscription sub = sc.subscribe("foo", new MessageHandler() {
    public void onMessage(Message m) {
        ...
    }
}, /*SubscriptionOptions*/);

doing something like the following (using reactor core)
ReactiveSubscription sub = sc.subscribeReactive("foo", /*SubscriptionOptions*/);
Flux<Message> stream = sub.stream();

The above is just a possible example, I don't know what the best way of such an implementation would look like.

I have been thinking about this a little bit.
It's not so hard to make a Flux from a subscription object via the Flux.genrate(...) part of the Reactor API. But the only backpressure support available would be buffering or dropping elements.
To add proper reactive support (effectively bridging a Flux via Nats Streaming) seems quite a bit more complicated.

I need reactive stream support as well with proper backpressure support. Note that this doesn't require a new library since as of Java 9 Reactive Stream support is built into Java via Flow. So if a NATS specific implementations of those interfaces are written that follow the Reactive Streams specification then it should work with reactive libraries like Project Reactor and RxJava.


WG: Clickmap: The flare on the last article only fades in for the first player

The fade effect doesn't display for any following players who finished, but instead the flare just appears with the rest of the screen.

Still seeing this issue, same as described above.


Add Vim snippets

Please consider adding ultisnips of ava for vim editor.
Thanks!
ref:

engine plugin: https://github.com/SirVer/ultisnips
common snippets plugin: https://github.com/honza/vim-snippets/tree/master/UltiSnips

I don't have any experience with Vim, but I did a tweet about it: https://twitter.com/ava__js/status/729190859313270785 ;)

@zhangyuheng I needed this anyway, so I created one here https://github.com/ahmedelgabri/vim-ava-snippets
One thing to note though that having a . in a trigger ex: t.is, etc... might be an issue but for now it's working maybe I'll change this to - instead.

Wow. That was fast. Thank you @ahmedelgabri! Added :)

@zhangyuheng Open issues on https://github.com/ahmedelgabri/vim-ava-snippets for any improvements or bugs.


Update eslint to v3.5. Fix lint errors. Move from gulp to npm task for…

… lint. Closes #44, #45.

Coverage remained the same at 92.13% when pulling abfc2b0d34350e0266f2c29e7cfe36192ca34bb7 on develop-npm-eslint-task into 7ddaa73c612471246d27a2930c5f395a5a8b1de8 on develop.

Coverage remained the same at 92.13% when pulling 8b224b029f710990940e35082a56ae5b7bd44ae3 on develop-npm-eslint-task into 7ddaa73c612471246d27a2930c5f395a5a8b1de8 on develop.


Added branch name (master) to CONTRIBUTING.md

New section (Branching strategy). This could be filled out a bit but I
don't know enough about your workflow to write it up with any more detail.
Explicitly mention master in the PR section.

:metal: I had made note to update this, but you beat me to it.


Hasura docs are no longer available

In previous version of the documents screenshots existed that showed how the flow of not just keycloak but also the hasura backend. Was this a mistake or on purpose?

Please find documentation in https://httpsomkar.github.io/keycloak-hasura-connector/


Add a go.mod file to support Go modules.

In order to work well with Go modules, packages should have a go.mod file and semantic version tags on release versions.
Here I have provided a rudimentary go.mod file for levigo.  If you accept this change, and if you are willing to also git tag v0.0.1 master and git push origin v0.0.1 after it is merged, that would begin the release history. I don't know whether you consider the API stable enough for a v1.0.0 release, but users will expect stability so v0 is a safe default.

Wilco! Thanks!

(And done as v1.0.0)

Thank you!


feat: Array of Structure Query Code sample

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[x] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> ☕
If you write sample code, please follow the samples format.

nit: The conventional commit for samples is doc. feat will add it as a feature to the release notes.


Model not updating when dynamic select element is manipulated via arrow keys

Steps

have a select which all options are generated from ng-options
focus the select (but do not select any option yet)
type "T"
type "T" again

What should happen?
at step 3, the model should be "TN"
at step 4, the model should change to "TX"
What happens and should not?
at step 4, the model stays to "TX"
Environment
Chrome 33 , angular 1.2 or beta 1.3
demonstration (taken from https://github.com/angular/angular.js/issues/2230):
http://jsfiddle.net/ADukg/5116/
or http://plnkr.co/edit/2R9YM6KF01bye4kPeoZL?p=preview
work around

hard code the empty item <option value="" /> inside the select
trigger a change event on keystrokes: $(document).on( 'keyup', 'select', function(){$(this).trigger('change')} );

related

https://github.com/angular/angular.js/issues/4216
https://github.com/angular/angular.js/issues/2230

I am experiencing the same problem as well in Chrome X and Safari Y. I had just created a Plunker and was about to submit a ticket when I saw this one. Here's my Plunker. http://plnkr.co/edit/Zsb3iiGeoc2Ep0OyaWek

Now I'm facing the same problem, I don't think it is a problem related to browser. Is there any way to tackle this problem?

@vipin030, could you elaborate why you don't think it is a browser-related problem ?
As explained elsewhere (and confirmed by @NateRedding), adding a explicit empty option solved the issue:
<select ng-model="..." ng-options="...">
  <option value="">--- Select (or any text you like) ---</option>
</select>

@gkalpak , I have added the explicit option tag into my selection tag, but the problem is still exist, I have tested in firefox and chrome, it is not working in both.
I have developed the same by using javascript for testing purpose, it is working fine on both the browser.

@vipin030, can you provide  the Angular code that is not working for you (or better yet provide a runnable demo, using CodePen, Plnkr, JSFiddle etc) ?

@gkalpak , please check the below plnkr example, it is not working for me, The browser which I have tested is Firefox 37.0.1. Please let me know your thoughts.
http://plnkr.co/edit/2xSGeVPc9sKUz2SCcbZz?p=preview

@vipin030, I tried your demo on latest Chrome and Firefox and it works as expected.
--
Note that there is one difference between Chrome and Firefox in respect to when the select's value is changed:

Chrome updates the element's value as soon as a new option is selected.
Firefox updates the element's value only after the element is blurred.

The spec is somewhat unclear regarding that, stating that the value should be updated as soon as the user in done interacting with it, but it is a matter of interpretation.
This might be a source of confusion sometimes.

@gkalpak , Please check my demo code again
http://plnkr.co/edit/2xSGeVPc9sKUz2SCcbZz?p=preview
You can see two selection box  there, second one using our pure javascript to show the value, it is working on every browser regardless of the version
Note:
Firefox also  updates the element's value as soon as a new option is selected.( Please check my demo and let me know why angular is not performing the same thing like pure javascript )

@vipin030, both versions work as expected (each :smiley:).
The difference is that in your pure-JS implementation you use onkeyup (which is fired more consistently across browsers), whereas Angular is using onchange (which fires differently on FF vs Chrome).
If you want to update onkeyup instead of onchange, it is easy enough to write a custom directive that listens for the keyup event and triggers a change event:
directive('changeOnKeyup', function changeOnKeyupDirective() {
  return function changeOnKeyupPostLink(scope, elem) {
    elem.on('keyup', elem.triggerHandler.bind(elem, 'change'));
  };
});

Updated plnkr

Closing as dupe of https://github.com/angular/angular.js/issues/9134


Boilerplate Repos Blog Post

Write a blog post about effective git strategies to use when building something based of a boilerplate repository.
Cover:

Forking :fork_and_knife:
Git remotes :satellite:
Pulling the latest upstream changes :boat:
Conflict resolution :facepunch:

https://bocoup.com/?p=3636&preview=true

Nits

By setting up the main boilerplate repository as it’s own remote

its

That means your local repository doesn’t know what its newly added

it's (or better broken out as "it has")
Content:

When I think about this approach to using boilerplates, I'm mainly concerned about the issue of trust with the upstream developer: will they and how often are they going to make breaking changes? Following this path with an untrustworthy upstream can lead to even more yak-shaving.

This seems like a huge argument in favor of releasing boilerplates versioned into NPM with sensible defaults and overridable configs, much like hjs-webpack. I'd rather test out an update to the foundation of my application as a versioned module than dealing with rebasing/merging.

Thanks @bobholt, I think your second content point is right on. As I was writing the final section I started feeling the same way. There's a challenge there to the boilerplate devs to produce something that can be consumed in that way. I'm not entirely sure how to work this into the post but I wonder how feasible it would be to structure the games boiler in such a way?
On the first point I think I can find places to insert the trust issue.

I got small things. I enjoyed reading this! It flows very logically and was easy to follow, even for me! ;)
Direct Clone
Using git to clone boilerplate repos is a great start, but it can sometimes be confusing how to proceed from here
Suggest "it can be difficult to know how to proceed"
Even if you know in the end this is where you’re going to end up I still advocate for configuring an upstream remote in the beginning.
Suggest "Even if you know this is where you're going to end up, I still advocate for..."

um sorry that was an accident. Reopened and readded Needs Review label


Mobx example

Hi, thnx for this great repo and example. Would be great though if mobx could be added as an example, together with the already included react-router v4.
Would that be possible? Thnx in advance!

Hi @jwamsterdam,
Yes, it would be cool to have an example with mobx. If you have time for it please submit a PR.
Thanks

@jwamsterdam @gabrielbull  I decided to do a mobx version of this. take a look and let me know what you think

Hi Gabriel, I haven't had a chance to take a look at it, but I definitely
will only my app is more mature!! Many many thanks for your effort and I
will let you know if it worked out!
2017-06-07 5:33 GMT+02:00 Layton Whiteley<EMAIL_ADDRESS>
@jwamsterdam https://github.com/jwamsterdam @gabrielbull
https://github.com/gabrielbull I decided to do a mobx version of this.
take a look and let me know what you think
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gabrielbull/react-router-server-complex-example/issues/2#issuecomment-306676839,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGoND2Le7qUpKLdvNvGVgJxA-zqRMUmCks5sBhoUgaJpZM4MyGRz
.

--
JW Hennink
Meester Cornelisstraat 80zw
2023DK Haarlem
06 230 66 490


Error is reset to null during invalidation

Describe the bug
When invalidating a query that thrown, its error field is reset to null during invalidation until it throws again. This is not consistent with the behavior of RQ v3 where the error field was retained. As I could not find any information on that in the migration docs or any option to fallback to the old behavior, I consider this behavior a bug.
If you consider this correct, could you provide some links explaining it or provide guidance about how to fallback to the previous behavior?
Your minimal, reproducible example
https://codesandbox.io/p/sandbox/rq-v5-invalidation-issues-forked-3nh2c9?file=%2Fsrc%2FApp.js
Steps to reproduce

Navigate to the sandbox
Open console
Click on "Invalidate queries" button
See that the error text and error from console disappears until its thrown again

Expected behavior
The error would be retained on invalidation, same as in v3.
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
OS - [macOS]
browser - Chrome
version - 122.0.6261.112
Tanstack Query adapter
None
TanStack Query version
5.27.3
TypeScript version
No response
Additional context
No response

this is on purpose - errors reset when going into loading state. This is the same behaviour as in v3 - here's a fork of your reproduction downgraded to v3 and it behaves exactly the same:
https://codesandbox.io/p/sandbox/rq-v5-error-not-kept-forked-5wz54w?file=%2Fsrc%2FApp.js%3A8%2C1


status bar behaviour changes when exiting the full screen

When exiting the full screen view, the status bar gets hidden. Status bar is not set as per a single view controller in the info.plist, so its not possible for me to update it for a single viewcontroller.

Can you share some more details, like a screen shot of current behavior and desired behavior?

I added the player in a viewcontroller and pushed this viewcontroller from another viewcontroller on an action. Now when I play the video, it expands to full screen. In the full screen view, if I tap on the cross button in the top left of the screen, the player exits the full screen but the status bar in the viewcontroller now gets hidden. My viewcontroller also has a navigationbar

@mukeshydv please suggest a solution

It's not an issue in the simulator but in the devices.  Checked in iphone 6, 8 and Xr

can you paste code of your viewcontroller which has video view?

Hi,
I just have an iboutlet for the youtubePlayerView. And one line in
viewdidload that is
videoplayer.loadWithVideoId()
In which I pass the video id
On Fri, May 29, 2020, 01:16 Mukesh<EMAIL_ADDRESS>wrote:

can you paste code of your viewcontroller which has video view?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mukeshydv/YoutubePlayerView/issues/21#issuecomment-635558026,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPEJ4M2WGFH2ZJKNIYV6YTRT25Q5ANCNFSM4NNAPQAA
.

try adding this code in the video view controller
    override var prefersStatusBarHidden: Bool {
        return false
    }

This can only be used if in the info.plist viewcontroller based status bar
is set to true. And in that case I'll have to set the status bar in all of
the viewcontrollers. That is just not possible.
On Mon, Jun 1, 2020, 00:35 Mukesh<EMAIL_ADDRESS>wrote:

try adding this code in the video view controller
override var prefersStatusBarHidden: Bool {
    return false
}

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mukeshydv/YoutubePlayerView/issues/21#issuecomment-636514122,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPEJ4JFXYTCVTSRMNUOPK3RUKTADANCNFSM4NNAPQAA
.

No, setting view controller based status bar is not required now, you can use this without changing info.plist.

https://github.com/react-native-community/react-native-webview/issues/62

Setting viewcontroller based status bar to true is still required to call prefersstatusbarhidden. And why have you closed the issue, when it's not resolved?

This is not library's issue, so closing it.
https://stackoverflow.com/questions/52136497/status-bar-disappeared-after-full-screen-video-in-wkwebview-only-in-ios-12

This is library's issue. I don't need to handle the status on any other
viewcontroller, but the viewcontroller in which your library is used,
requires it to b used in. It should be mentioned in the requirements
section of the library. Otherwise it's just misleading people
On Mon, Jun 1, 2020, 13:02 Mukesh<EMAIL_ADDRESS>wrote:

This is not library's issue, so closing it.
https://stackoverflow.com/questions/52136497/status-bar-disappeared-after-full-screen-video-in-wkwebview-only-in-ios-12
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mukeshydv/YoutubePlayerView/issues/21#issuecomment-636665748,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPEJ4J333LG7Q7ZYGMH34TRUNKPPANCNFSM4NNAPQAA
.

I'm busy for some time, can you open a PR with the fix? thanks.

I don't have a fix. I would just ask you to reopen the issue, may be
someone else could solve it
On Wed, Jun 3, 2020, 18:03 Mukesh<EMAIL_ADDRESS>wrote:

I'm busy for some time, can you open a PR with the fix? thanks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mukeshydv/YoutubePlayerView/issues/21#issuecomment-638167354,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPEJ4IWONAFYJCQ2CTENZDRUY7KLANCNFSM4NNAPQAA
.


add xiaomudk as maintainer

Signed-off-by: yxxhero<EMAIL_ADDRESS>@mumoshu
@itscaro
I would like to recommend a friend to join helmfile. He is involved in many open-source projects and has rich experience in software development. We can work together to maintain helmfile.  What’s you opinion? https://github.com/helmfile/helmfile/commits?author=xiaomudk
Recent Work: helmfile init

@mumoshu WDYT? Thanks very much.

Let's include https://github.com/helmfile/helmfile/pull/520 as @xiaomudk's recent big contribution!

@xiaomudk good work. Thanks very much.

Let's include #520 as @xiaomudk's recent big contribution!

@xiaomudk good work. Thanks very much.

@xiaomudk great job: https://github.com/helmfile/helmfile/pull/522

@mumoshu maybe we need to invite @xiaomudk to join the helmfile org.

@mumoshu @yxxhero Thank you very much, reliable partners!


Talked about in #30
Rename to infraction system.


Add a placeholder for trade toggle

Add a placeholder for /trade toggle to see if a player has /trade toggle enabled or disabled

this is already available
%tradesystem_status%


SORT-OBJECT offers a misleading error message

Steps to reproduce
[PSCUSTOMOBJECT] @{ 0 = 0 } | SORT 0

Expected behavior
0
-
0

Actual behavior
Sort-Object: Cannot convert System.Int32 to one of the following types {System.String, System.Management.Automation.ScriptBlock}.

Environment data

Name                           Value
----                           -----
PSVersion                      7.0.3
PSEdition                      Core
GitCommitId                    7.0.3
OS                             Microsoft Windows 10.0.18363
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Certainly misleading, though I think the right solution is not to report an error at all and actually perform to-string conversion (the implied -Property parameter is [object[]]-typed).

Why misleading? The message says clearly that PowerShell expects either a string or a scriptblock but not int. Since PowerShell expects strongly one from two types it does not know what would be right conversion. It is a generic code and we can not force to string conversion because there is a high risk to break something in Engine or in custom scripts/codes.
See how works internal List<MshParameter> ProcessParameters(object[] p, TerminatingErrorContext invocationContext).

It is misleading, because you obviously can convert a System.32 instance to a System.String instance.

It shouldn't be an error to begin with, because the only sensible interpretation of something that isn't already a string or a hashtable (with a calculated-property definition) to begin with is to convert it to a string.

While it's debatable whether to-string conversion should be applied to just any data type, something like 0 in argument mode - in the absence of quoting - is inherently ambiguous: is it the number 0 or string '0'?
Here, the only sensible interpretation is as a string.
Note that we already have a mechanism to resolve this ambiguity, but it requires a PSObject-typed parameter:
PS> & { param([psobject] $p) $p.GetType().Name; $p + 0; $p.psobject.ToString() } 1e2
Double
100
1e2  # cached string representation, as passed on the command line

It sounds like changing the -Property parameter to [psobject[]] wouldn't be a breaking change, right?
If in the end there is a technical reason that this cannot be implemented, the error message should be revised to say something like: "if this is a property name that looks like a number, enclose it in quotes".

There's no technical reason I can think of that this couldn't be implemented. Seems like it's most likely just a simple case of doing a type check where it probably should have been a type conversion, i.e., we're probably doing something like:
if Property.BaseObject is IDictionary
  # handle as calculated property
else if Property.BaseObject is ScriptBlock
  # handle as unnamed calculated property expression
else if Property.BaseObject is string
  # treat as property name to sort on

that last branch should not have a type check at all, and instead simply opt for LanguagePrimitives.ConvertTo<string>(Property)

Thanks, @vexx32 - that should indeed suffice - I just realized that barewords are implicitly passed in PSObject wrappers, so no change in the -Property parameter type is necessary.

Thanks, @vexx32, that should indeed suffice.
I got myself confused earlier: there isn't a need to change the -Property parameter type, given that the engine implicitly wraps in PSObject those arguments that were parsed as numbers whose default string representation differs from the one given, with the PSObject instance containing the original (string) representation, which, as you state, LanguagePrimitives.ConvertTo<string>() does utilize.
This ensures that passing a bareword such as 1e2, which becomes a [double] that by default stringifies as 100, is still represented as '1e2 when converted back to a string.

Why misleading? The message says clearly that PowerShell expects either a string or a scriptblock but not int.

The message clearly states that whatever comes over must be convertible to both a string and a script block because it is a problem when the value is not convertible to one of them.  A script block is convertible to a string but it is debatable whether a string is convertible to a script block, so for the clarity of exposition I shall assume that it is not.  I also do not know of any other type that is convertible to ScriptBlock.  That means the winner is… ScriptBlock 🎺🎺 and anything that is not one cannot be given as a parameter.  Nice?

that last branch should not have a type check at all, and instead simply opt for LanguagePrimitives.ConvertTo(Property) (which will correctly handle the string conversion for parameters parsed as numbers as well).

It would be a huge breaking change - current code explicitly throw on unexpected type. The code is generic. There can be any allowed types, ex., byte,int,long - why would you convert another type to string in the case?

current code explicitly throws on unexpected type. The code is generic.

Code that throws on an unexpected type is definitely not generic.

@iSazonov

why would you convert another type to string in the case?

It isn't another type, conceptually. In the example in the OP, the intent is clearly to pass a string, 0.
Yes, you can force interpretation as a string with explicit quoting ('0'), but you shouldn't have to, given that in argument-parsing mode quoting strings is optional - just like you're free to use Get-Date -UFormat 0 (to use a contrived example) without having to quote the 0, or just like Write-Host 1e2 and Write-Host '1e2' produce the same output.)
What we're seeing is the confluence of two behaviors:

The parameter binder parses a bareword that looks like a number literal as a number rather than a string, but, if it then binds to an effectively untyped parameter (object or PSObject)  wraps it in a PSObject that caches the original string representation as specified on the command line, leaving the door open to treat this argument as a string later.

However, as an optimization, this wrapping happens only  if the original string representation differs from the resulting number's default string representation.

Incidentally, this is currently broken for number literals with the d ([decimal]) and n ([bigint]) suffixes, specifically - see #13380

Sort-Object doesn't have distinct parameters for the distinct types it accepts as sort criteria and uses object instead.

Note that a [string]-typed parameter would indeed accept any type, given that PowerShell happily converts anything to a string.
If you're really concerned that existing code relies on Sort-Object throwing an error for non-string property arguments, we can more narrowly limit what non-string types are accepted to the primitive number types plus [decimal] and [bigint].
(However, due to the optimization that results in a PSObject wrapper only when truly needed, we can't reliably tell if a given value started out as a bareword, so an unambiguously-passed-as-a-number argument would succeed too; e.g.
Sort-Object (0)).

Incidentally: the code in question appears to be shared among cmdlets; for instance,

Yes, it is a generic code. There may be an unpredictable number of cmdlets where this is not the problem but the desired behavior.

As far as I'm aware, this code is used exclusively for property expressions (which would all benefit from fixing the issue). Are there other use cases?

Incidentally: the code in question appears to be shared among cmdlets; for instance,

Yes, it is a generic code. There may be an unpredictable number of cmdlets where this is not the problem but the desired behavior.

No, it is shared code.  I think we would all appreciate an example where converting a number to a string leads to a zombie apocalypse.


BugFix: Replace map with for loop

This inner function from PaperDC fails on my machine:
https://github.com/agdestein/IncompressibleNavierStokes.jl/blob/e5dd0d9613326367927480e0b0f21fe3f4ad008f/lib/PaperDC/src/train.jl#L14-L19
so I have replaced it with a for loop.
Checklist

[x] I have added my name to the CITATION.cff file.

Ah it should be with parentheses, instead of
filenames = map(Iterators.product(params.nles, params.filters)) do nles, Φ
end

it should be
filenames = map(Iterators.product(params.nles, params.filters)) do (nles, Φ)
end


NENAH005 - diffusion does not have same number of slices: dir-AP 64 and dwi dir-PA 44

This data set has to be discarded


The event that triggered the playbook is not passed to the playbook

Please confirm the following

[X] I agree to follow this project's code of conduct.
[X] I have checked the current issues for duplicates.
[X] I understand that ansible-rulebook is open source software provided for free and that I might not receive a timely response.

Bug Summary
Hello. I have a weird issue recently. The playbook invoked by the rulebook doesn't know about the event that triggered it:
TASK [debug] *******************************************************************
fatal: [localhost]: FAILED! => {"msg": "The task includes an option with an undefined variable. The error was: 'event' is undefined. 'event' is undefined\n\nThe error appears to be in '/tmp/run_playbookj7pb1b06/project/say-what.yml': line 7, column 7, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line
appears to be:\n\n  tasks:\n    - debug:\n      ^ here\n"}
The code with issue is:

debug:
msg: "Thank you!!!! {{ event.payload.sender }}!"
I am using this Dockerfile as base to build my image:https://github.com/ansible/ansible-rulebook/blob/main/Dockerfile.

Environment
Redhat8
Steps to reproduce
Use https://github.com/ansible/ansible-rulebook/blob/main/Dockerfile to build a images, deploy and run it, have the rulebook listen to port 5010. send a webhook message:
curl -H 'Content-Type: application/json' -d "{"message": "Ansible is super cool", "sender": "deployer node"}" http://localhost:5010/endpoint
Actual results
Playbook fails with:
TASK [debug] *******************************************************************
fatal: [localhost]: FAILED! => {"msg": "The task includes an option with an undefined variable. The error was: 'event' is undefined. 'event' is undefined\n\nThe error appears to be in '/tmp/run_playbookj7pb1b06/project/say-what.yml': line 7, column 7, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line
appears to be:\n\n  tasks:\n    - debug:\n      ^ here\n"}
The code with issue is:

debug:
msg: "Thank you!!!! {{ event.payload.sender }}!"

Expected results
playbook should run successfully and display: Thank you!!!! deployer node
Additional information
No response

Hi @ttuffin, thank you so much! It is working now.


How to copy on drag and drop it ?

Hi,
Do you find any solution to copy the item while dragging and drop ?


Date / time data

Gnuplot documentation says:

gnuplot supports the use of time and/or date information as input data. This feature is activated by the
commands set xdata time, set ydata time

How can I use these commands in Cryplot ?
Thanks

Sorry, but the example doesn't do what I want.
I don't want to display integers (hours, days or years) in x, but dates, specifying the format (e.g. mm/yy).
Is this possible?
Thanks

Ah, I hadn't seen the gnuplot function!
Thanks


Make ExtractAPI#annotations phase travel to entering typer, again

This is a revert of the change that was made in:
https://github.com/sbt/zinc/pull/86/commits/b0a8a91aaa5b6f55714986f7bf389ec8aa091ec0#diff-babf52667b7dd64d625e0bcddfd8da48R211
Here's my analysis of what happened.
Prior to creating this commit Greg was working on class-based
name-hashing in a branch in sbt/sbt, targeting sbt 0.13.  With this
commit Greg was re-syncing his changes with the changes that had
happened in the sbt/zinc repo, to be able to PR sbt/zinc.
However, there was an important difference in sources between sbt/sbt
and sbt/zinc. The Scala compiler/reflect APIs change over time, for
instance in scala 2.11 the "atPhase" method was deprecated in favour of
the brand-new, more precise, but otherwise identical "enteringPhase"
method.  But sbt/sbt didn't care about deprecations and just used
"atPhase" for all versions of Scala it supported (2.8/2.9/etc).  In
sbt/zinc, instead, the sources had been split into 2.10 and 2.11+
sources, and the 2.11+ sources had switched to the newer, non-deprecated
APIs. So "atPhase" had been replaced with "enteringPhase".
So when Greg was re-syncing the changes in his branch with the changes
in sbt/zinc he had to deal with (a) the sources being split and (b) the
compiler API differences. At this point I think 1 of the 2 following
plausible scenarios happened. Either:

For one reason or another Greg accidentally, and incorrectly, thought
it was the correct implementation, perhaps he forgot that atPhase and
enteringPhase are synonymous. So he made the 2.11+ code invoke
enteringPhase(currentRun.typerPhase.next) instead of
enteringPhase(currentRun.typerPhase), thinking it was the correct
equivalent to atPhase(currentRun.typerPhase) in the 2.10 code.

Greg intentionally, purposely, made, all of a sudden, a judgement
call, while syncing, dealing with merge conflicts and file path changes,
that ExtractAPI#annotations MUST phase travel to exiting the typer phase
(via enteringPhase(typer.next)). And then he (a) changed only the
2.11+ code, leaving the the 2.10 code travel to the wrong place and
(b) didn't mention it in his "highlight of less obvious changes" in the
commit message.

I find 1 to be much, much more plausible, so let's revert to what has
worked since its inception in 2010:
https://github.com/sbt/sbt/commit/af4f41e052e6e077c2d590ba8ddbfda19769f280#diff-d31107a2fcd3c8482cbac4ee198375b6R311
and today in the latest stable release, 0.13.13:
https://github.com/sbt/sbt/blob/v0.13.13/compile/interface/src/main/scala/xsbt/ExtractAPI.scala#L189

/cc @gkossakowski @retronym

We've been discussing this all day here.
My take: this change may or may have not affected the behaviour of Zinc. It's not clear yet. The fact that our current Zinc for 2.11 runs after typer means that we could be affected by the desugarings produced in typer, or weird synthesized members or transformation we're not aware of. As far as I know, typer is dealing with a lot of Scala features, not only typing (e.g. default arguments, named arguments, implicit desugarings). So perhaps this could have an effect on the way Zinc tracks dependencies (this is unlikely, but it could be possible).
Reverting the change sounds good to me, though. I think this was an accident and the fact that Zinc has been using atPhase(currentRun.typer) for more than 4 years makes me think this is the way it's supposed to be. Otherwise a good  explanation would exist in commit messages (which doesn't).
This is the change that introduced atPhase for the first time in ExtractAPI: https://github.com/gkossakowski/sbt/commit/af4f41e052e6e077c2d590ba8ddbfda19769f280.
@romanowski suggested to run this change on several well-known Scala projects and see what happens. I think that's a good idea, let's see what the results are.

I forgot to mention: this change removes a difference between the 2.10 and 2.11+ implementations, see this diff between the two:
--- internal/compiler-bridge/src-2.10/main/scala/xsbt/ExtractAPI.scala	2017-02-24 16:58:41.000000000 +0000
+++ internal/compiler-bridge/src/main/scala/xsbt/ExtractAPI.scala	2017-02-24
@@ -204,3 +211,3 @@
   private def annotations(in: Symbol, s: Symbol): Array[xsbti.api.Annotation] =
-    atPhase(currentRun.typerPhase) {
+    enteringPhase(currentRun.typerPhase.next) {
       val base = if (s.hasFlag(Flags.ACCESSOR)) s.accessed else NoSymbol

NOTE: Don't be confused,atPhase and enteringPhase are strictly equivalent:
https://github.com/scala/scala/blob/v2.12.1/src/reflect/scala/reflect/internal/SymbolTable.scala#L420

So:

this change may or may have not affected the behaviour of Zinc. It's not clear yet.

Given that none of the tests aren't failing, particularly the 2.10 tests, if this does affect the behaviour of Zinc then it must only affect Zinc when using Scala 2.11+.
Or there's simply is no test case that warrant this to be changed originally.
@jvican Would you prefer if I changed the 2.10 code to enteringPhase(currentRun.typerPhase.next)?

I think it's good as-is. Rgarding the change of behaviour, sorry for not being explicit, I meant change of behaviour for 2.11+, when the change was introduced.

I love PRs that have 100 times more description and discussion then actual code changes :)
Mentioned builds succeeded but this was 'smoke tests' (big codebase was compiled without any exception/errors, nothing more).
I think we should merge it.

Do we have tests to check the correct behaviour of annotations in Scala? Otherwise, it would be a good addition to the PR.

I don't.

@jvican

I like your emphasis on the two scenarios
It's obvious you don't know what's the real reason

I've been studying what phase travelling actually means, and I've found some informative commit messages from Paul:

https://github.com/scala/scala/commit/4573e06a278e9505b7cf1b6f1390e2833fb70fec
https://github.com/scala/scala/commit/da12146b8f3a5215e9685cd978d898131f67f1eb
https://github.com/scala/scala/commit/2e69ae41d9b4af6dd87e14e5f6f980c51e164e4c

So it would seem that since its inception 7 years ago and still today in 0.13.13 ExtractAPI has travelled to the beginning of the typer phase when doing whatever it does in "def annotations". Except for as of 1 month ago where the 2.11+ sources here (only pre-released) were switch so it runs in patmat (or whatever happens to be the phase after typer.

Okay, I'm weighing in again.
After giving it some thought, I think this PR fixes a bug in 2.11 sources. Running after typer is bad -- I think that some annotations are removed by the typer because it applies its transformations. For instance, varargs. If we have a method annotated with varargs and we remove it, it changes the generated bytecode (on which a potential user could be relying on). However, before this PR, Zinc would not invalidate it because it wouldn't detect a change in its API. The same could happen for other annotations that affect the bytecode generation of the API.
So, I would propose that we merge this change. I'm pretty sure it's the right fix.

Thanks Jorge, that sounds right to me too.

Okay, I've just double-checked. This does not only affect varargs, it also affects annotations like field and getter since typer is the responsible of synthesizing getters and setters.

In a weird way, thank god that the sources were duplicated and only one version was changed. It allowed for an easy way to discover this bug. It's the risk you are paying when you merge +11,154 −3,866 PRs.

Just to be sure I played with different annotations and compare output with and without .next and results was always the same.
Now am almost 100% sure that merge was good decision.


Question: what if growing the amount of width or height instead of doubling

When it has no room for more shelves, it doubles its width or height.
https://github.com/mapbox/shelf-pack/blob/master/index.mjs#L194
So it can potentially waste a lot of space as said on README.
What about to just add bin's width or height instead of double the size?
like:
if (w1 <= h1 || w > w1) {   // grow width..
    w2 = Math.max(w, w1) + w;
}
if (h1 < w1 || h > h1) {    // grow height..
    h2 = Math.max(h, h1) + h;
}

It takes less space. May this change make any side-effects?

Good question!  The library works like this because it was originally intended to pack glyphs into graphics texture memory, which tends to prefer the memory being allocated in dimensions that are powers of 2, and also to avoid costly reallocations.
We found that if the maps being shown used mostly Latin characters in the fonts, we didn't need to pack so many glyphs - 26 letters, 10 numbers, and a handful of punctuation.  They'd grow to a certain size and then stop and we didn't have to think much about memory.  But if the maps being shown used CJK characters, these sprites can grow much larger, so they would grow to fit about the number of glyphs that are being used on the visible map, then as the user panned the map around we'd swap different glyphs into the spritesheet as needed and avoid the reallocations.

Thanks for the very quick answer! So it can reduce the number of memory re-allocation times.
I'm using it to calculate and arrange bins for image sprites that the memory usage is not very critical :)


Running the devenv remote over SSH

Hi, for obvious reasons I'm currently not localized as close to my GPU-servers as I'd like, and I'm currently trying to set up the devenv over Remote:SSH to use with Swift for Tensorflow.  I believe I've set the paths correctly, so I'm a bit confused about the error message I'm being thrown. Any ideas? I'm currently trying to use the languageserver bundled with the Swift for Tensorflow (sourcekit-lsp). I'd be very grateful for any hints that can get me up and running.
[Error - 7:03:06 PM] Starting client failed
Error: spawn ENOTDIR
	at ChildProcess.spawn (internal/child_process.js:394:11)
	at Object.spawn (child_process.js:535:9)
	at /home/asbe/.vscode-server/extensions/vknabel.vscode-swift-development-environment-2.10.0/node_modules/vscode-languageclient/lib/main.js:354:40

EDIT: after setting swift.languageServerPath correctly in the remote .vsconfig/settings, I came to the next error message missing dependent swift tool, please configure correct "swift.path.swift_driver_bin"  which leaves me slightly more confused.

Hi @asbe!
Good catch! There was an error in the documentation.
You can remove swift.languageServerPath from your settings!
Instead use the following config.
{
  "sourcekit-lsp.serverPath": "absolute path to the sourcekit-lsp executable",
  "sde.languageServerMode": "sourcekit-lsp",
  // optionally
  "sourcekit-lsp.toolchainPath": "absolute path to your swift 4 tensorflow toolchain",
}

Let me know if this helped you!

Having the following config seems to work (almost - now I believe the issues are with the code I'm loading - random file from "swift-models") - I get completion and floating bubbles.
I had to set toolchainPath to the folder containing the local installation of Swift (i.e., ./usr/bin/...) - slightly confusing.
{
    "sde.languageServerMode": "sourcekit-lsp",
    "sourcekit-lsp.toolchainPath": "/home/asbe/projects/swiftexperiments/"
  }


ASP.NET Core 2.1 Kerberos delegation not working

Hi all,
we have an ASP.NET Core 2.1 API running on a Windows Server 2012 R2 in IIS.
The IIS is configured to use Kerebros authentication, SPNs are set correctly and the application pool identity is a custom domain user. Delegation is enabled, too.
The API is called by a browser and tries to connect to a SharePoint 2013 Server using impersonation and web requests.
When I call the API with my account, which is a local administrator on the Server hosting the API impersonation works perfect.
If a regular account tries to call the API an exception gets thrown:
System.Net.WebException: An error occurred while sending the request. Error 5 calling WinHttpOpen, 'Access is denied'. ---> System.Net.Http.HttpRequestException: An error occurred while sending the request. ---> System.Net.Http.WinHttpException: Error 5 calling WinHttpOpen, 'Access is denied'.
   at System.Net.Http.WinHttpHandler.ThrowOnInvalidHandle(SafeWinHttpHandle handle, String nameOfCalledFunction)
   at System.Net.Http.WinHttpHandler.EnsureSessionHandleExists(WinHttpRequestState state)
   at System.Net.Http.WinHttpHandler.StartRequest(WinHttpRequestState state)
   --- End of inner exception stack trace ---
   at System.Net.Http.HttpClient.FinishSendAsyncUnbuffered(Task`1 sendTask, HttpRequestMessage request, CancellationTokenSource cts, Boolean disposeCts)
   at System.Net.HttpWebRequest.SendRequest()
   at System.Net.HttpWebRequest.EndGetResponse(IAsyncResult asyncResult)
   --- End of inner exception stack trace ---
   at System.Net.HttpWebRequest.EndGetResponse(IAsyncResult asyncResult)
   at System.Threading.Tasks.TaskFactory`1.FromAsyncCoreLogic(IAsyncResult iar, Func`2 endFunction, Action`1 endAction, Task`1 promise, Boolean requiresSynchronization)
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.SharePoint.Client.ClientContext.GetFormDigestInfoPrivate()
   at Microsoft.SharePoint.Client.ClientContext.EnsureFormDigest()
   at Microsoft.SharePoint.Client.ClientContext.ExecuteQueryAsync()

Now, if I just open the Internet Explorer on the server hosting the API as a regular user the impersonation works from the clients computer.
Has anyone an idea?
Greetings from Hamburg!

@davidsh is this an issue you've seen with WinHttpHandler?

This seems like it's likely an issue in HttpClient not ASP.NET Core. Sounds like this is an issue for dotnet/runtime, since it's an issue with impersonation (WindowsIdentity.Impersonate I presume?)

@phibsi Can you clarify, are you using .NET Framework with ASP.NET Core 2.1? Impersonation isn't support in .NET Core, so I'm presuming you are?
Can you provide a runnable sample that illustrates what you're doing and where it's failing (I realize that it will depend upon per-machine configuration like Windows domains, etc.).

Clarification: ASP.NET Core doesn't do impersonation by default on .NET or Core, but you can do impersonation manually on either.

Hi @anurse,
I created a sample project to reproduce the issue: https://github.com/phibsi/sample-aspnetcore-impersonation
You need to publish the webservice to an IIS Website on Windows Server 2012 R2.
The Windows Server needs to be joined to an Active Directory. Kereberos requieres SPNs set to the Computer Account or Application Pool Account. DNS A-recoreds needs to be set for the services (Webservice and SharePoint).
You don't need to use Microsoft.SharePoint.Client, a simple HttpClient call reproduces the issue, too.

Hi @Tratcher,
Impersonation is done in https://github.com/phibsi/sample-aspnetcore-impersonation/blob/ec4aa0de0f2f44ee771c6cff0b0d2bf50155bc93/Services/ImpersonationService.cs#L64 using IIS, Kerberos and
var user = (WindowsIdentity)Context.User.Identity;
WindowsIdentity.RunImpersonated(user.AccessToken, action);

Does it work any better when the action being run under impersonation is not async (or is forced to complete synchronously within the impersonation)? We've seen issues in the past with the impersonation being reverted too soon.
https://github.com/phibsi/sample-aspnetcore-impersonation/blob/ec4aa0de0f2f44ee771c6cff0b0d2bf50155bc93/Controllers/HomeController.cs#L39-L44

Hi @Tratcher,
I updated the project and changed the async CSOM request to sync WebRequest:
https://github.com/phibsi/sample-aspnetcore-impersonation/blob/2e277d4f7d9de851a4e1391a68ec2db94eceb64d/Controllers/HomeController.cs#L94-L116
But I get the same error message :(
Any idea?

Ok, since this is primarily an issue with impersonation and outgoing requests I'm going to transfer it over to the folks that own those. This does have some similarities to https://github.com/dotnet/corefx/issues/38646, but not quite the same stack trace.


When the y-axis is used in percentage mode, different series cannot maintain the ratio

For example, this ma line jumps up and down when in percentage mode.
But the price mode is fine, what should I do?

Hi, do you have the same amount of data points in both series? could you please share the Codesanbox/Stackblitz with reproduction?

jsfiddle: https://jsfiddle.net/z4o19h6q/
Hi, this issue has nothing to do with the amount of data, any official example using percentage mode will also have this issue.
The problem should be caused by the fact that the percentage mode uses the first value as 0% of the currently seen data, but the technical indicator line must be different from the first value of the candlestick chart, so it will jump up and down when the seen range changes.
But what can I do to make the technical indicator not jump, like in the tradingview super chart, I can't find a configuration item for that!


Update json4s-native to 4.0.6

Updates org.json4s:json4s-native from 4.0.5 to 4.0.6.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.json4s", artifactId = "json4s-native" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.json4s", artifactId = "json4s-native" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Codecov Report
Base: 60.30% // Head: 59.99% // Decreases project coverage by -0.30% :warning:

Coverage data is based on head (d4a4248) compared to base (43826e3).
Patch coverage: 80.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #301      +/-   ##
==========================================
- Coverage   60.30%   59.99%   -0.31%
==========================================
  Files         153      153
  Lines        3252     3252
  Branches       87      102      +15
==========================================
- Hits         1961     1951      -10
- Misses       1291     1301      +10

Impacted Files
Coverage Δ

...rge/internal/domain/AggregateProcessingModel.scala
40.00% <25.00%> (-5.46%)
:arrow_down:

...ore/src/main/scala/surge/internal/SurgeModel.scala
95.83% <100.00%> (+0.18%)
:arrow_up:

.../surge/kafka/streams/KafkaStreamManagerActor.scala
55.69% <0.00%> (-10.13%)
:arrow_down:

...StreamsUpdatePartitionsOnStateChangeListener.scala
82.85% <0.00%> (-8.58%)
:arrow_down:

...health/windows/actor/HealthSignalWindowActor.scala
91.72% <0.00%> (-2.26%)
:arrow_down:

...windows/stream/actor/HealthSignalStreamActor.scala
60.00% <0.00%> (+26.66%)
:arrow_up:

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

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

Superseded by #409.


Problem with routes L5.3 and adminlte-laravel

The integration is good, all components is perfect.
Now, when show all routes:

The version 5.3 not have routes.php (this is the new structure)

Your ServiceProvider have this:

I need create my custom routes for example:

Not need Register
Need change names for login, password, etc.
Question, what is the best way for customize the routes?

My system:

PHP 5.6
Ubuntu 14.04
Laravel 5.3
Adminlte-laravel 3.1.11

Sorry for my english is very bad.

@ssheduardo you can disable adminlte service provider in config/app.php and create your own custom routes.php file (you can start with mine). You can also override/extend my ServiceProvider and override method defineRoutes.


[feat]: rdfs:domain and rdfs:range should be OPTIONAL

When I (optional)

have a rdf:Property without rdfs:doman and rdfs:range
like this spec

I expect

[x] Lavoratore.retribuzione to be shown as a match

Instead

it is shown as a not match
the rdfs:comment correctly shown ;)

Notes

https://w3id.org/italia/social-security/onto/contributions/retribuzioneOrariaEffettiva

@stefanone91

I keep viewing it as a non-match.
The rdfs comment is working fine for the property.

Works for me :)


Dev profile and portfolio

There are still some conflicts, need to resolve.

Confirm Changes, everything looks nice.


Breaking syft | grype or merging syft code that breaks grype tests should warn team

What would you like to be added:
The release of a new version of syft should exercise syft | grype and, if grype fails, warn the team that syft is ahead of grype, in the sense that the latest version of syft produces output the latest version of grype can't parse. The warning (e.g. slack alert to the team) should remind the team to promptly release grype so that the period where syft is ahead is very short.
Why is this needed:
https://github.com/anchore/syft/issues/1935 is an example where we had customer impact because a syft release went out that the team didn't realize was incompatible with latest grype, and so syft | grype for latest syft and grype was broken for a couple of days.

Though failing CI in a PR isn't an option, adding a comment on the PR when it will break grype could be a good option.

We discussed this issue again at retro, and the solution we were leaning towards was:

New commits on main in syft should create (or update if existing) a PR to grype that bumps syft in grype to the new commit.
Releases of syft should query for this PR and assert that its tests are is passing.
There should be a good way to override the checks in step 2 if needed.
When we release syft, we can then immediately merge this PR and release grype.

This way, regressions can be caught by grype's test suite, which is probably the right place for that.


Catalog all clang-tidy issues

It turns out clang-tidy has not been running in the CI linter jobs. #210 adds a script for running clang-tidy locally. All issues found with clang-tidy should be logged as bugs to be addressed.

Other tasks cover this now


Longest common substring

I would like to add this problem to the Dynamic programming(DP) section.
I will implement the code in C++ with proper comments , time complexity, space complexity and test cases.

Hey @tarun26091999 @rudrakshi99
I would like to do it in python.
could you please assign me this issue.

I will implement the code in C++ /C/python with proper comments , time complexity, space complexity and test cases.

I'm a GSSOC'21 participant and I would like to work on this issue in C++.
I will write clean code and use proper comments.
Thank you.

@tarun26091999
Please Assign this to me.I would like to code this in C++. I m a GSSOC'21 Partcipant .I will assure you that I will provide a commented code with efficient time complexity.
Regards
Madhvi .

present at #425


Automatically convert addresses in tests

The unit tests contain lots of addresses and private keys as strings.  Those are encoded in the Bitcoin formats in the upstream code, but need to be converted to Namecoin formats to be accepted by the code under test.
This used to be done by hand using e.g. the conversion tool from Namecoin Core.  But that is cumbersome.  In this change, we introduce a Python function frombtc which handles the conversion automatically.  So instead of explicitly converting string literals, we now only have to wrap them in frombtc(...) in the tests.  This makes them easier to understand, and also makes upstream merges easier (although it does not completely avoid merge conflicts).
In this commit, the change is just applied to test_bitcoin.py as a proof of concept.  More will come later.

@JeremyRand, please let me know what you think.  IMHO this makes our life easier (and, to be honest, also makes my life maintaining the Xaya fork easier).  If you think this is a good idea, I can apply this to all the other tests as well.

Approach ACK.

Thanks for the review!  Does your "approach ACK" mean that you want changes in details, that you want me to convert the other tests and add to this PR, or that we can merge this and follow-up with other tests in subsequent PRs?  I'm fine with each approach, so whatever you prefer.

@JeremyRand, please take a look at my last comment.  (In other words, please clarify whether you want any changes to this PR, you want me to add more test conversions to it, or we should merge it and add more conversions as follow ups.)
Also, did Travis get fixed?  If not, what tests besides pytest is it actually running?  Can't we just do them locally?

Does your "approach ACK" mean that you want changes in details, that you want me to convert the other tests and add to this PR, or that we can merge this and follow-up with other tests in subsequent PRs?

@domob1812 I haven't checked the exact changes (e.g. I haven't checked whether there are any typos in the addresses), but once I do that (and subject to a Travis check) I'm fine with merging this and deferring the other tests for future PR's.

Triggering a Travis rebuild....

ACK 54a1eb8de51996c437533309cbc2c863f874d2f4, I have not tested the code, but I have reviewed it and it looks OK, I agree it can be merged, subject to a Travis check.

Looks like this PR is throwing off the rebranding check.  Probably the easiest way to deal with this is to add some -v exemptions to contrib/check_nmc_rebranding.  Exempting electrum_nmc/electrum/tests/address_conversion.py from the bitcoin check, and exempting the electrum_nmc/electrum/tests/ folder from the bech32 and private key checks, would probably work fine.

Ah indeed, I was not aware of that rebrand-check script.  I've updated it now as you suggested, and it runs fine now for me.
It would be great if we could just exempt strings wrapped in frombtc from the address/privkey checks, but that seems not immediately possible since we only extract the strings for further greping.  But I guess the exemption of the test folder as you suggested is fine for the purpose here.
After all, rebranding the addresses in tests is not done because it would be user-visible, but to make the tests pass; so if we had missed an address, it would still fail some test.  Thus there is no real need for the rebrand checking of those anyway.

Triggering a rebuild...

ACK 26aff503d2ff1c693190af6ee7b363208e6ac35b; I have not tested the code, but I have reviewed it and it looks OK, I agree it can be merged.


chore(main): release 1.26.0

:robot: I have created a release beep boop
1.26.0 (2023-09-29)
Features

recurse video folders, update spinners to log extracting folders (#168) (5751b67)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/aidanaden/canvas-sync/releases/tag/v1.26.0 :sunflower:


Moving SourceControl to GingerCoreCommon

@MeniKadosh1, created a branch for SourceControl and can you please review that to merge the changes to master.

Please provide to Natisha local build to test source control operations with SVN and GIT solution


Cloud Code logs not working; VERBOSE=1 does work

I have setup parse server on AWS. I have connected my iOS app to it however I am getting errors returned to the xcode console. This code did not error on parse.com hosted server. In order for me to track down what the error is I need to see the cloud code logs.
I have set up parse dashboard locally and it connects however, there are no logs. Before you close this as a duplicate: I have setup FileAdapterLogger as stated in --> https://github.com/ParsePlatform/parse-server/issues/781
My elastic beanstalk server is healthy and the app calls the CC. If I use "verbose: 1" in my index.js I get a TON of output into the log files in parse dashboard. I need to see just the cloud code logs including errors in the error log.
This is my setup:
var logger = require('parse-server/lib/Adapters/Logger/FileLoggerAdapter').FileLoggerAdapter;

var api = new ParseServer({
  databaseURI: databaseUri || 'mongodb://xxxxxxxxxxxxxxxxxxxxxxxx',
  cloud: process.env.CLOUD_CODE_MAIN || __dirname + '/cloud/main.js',
  appId: process.env.APP_ID || 'xxxxxxxxxxxxxxxxxxxxxxxxxxxxx',
  masterKey: process.env.MASTER_KEY || 'xxxxxxxxxxxxxxxxxxxxxxxxxx',
  serverURL: process.env.SERVER_URL || 'http://xxxxxxxxxxxxxxxxxxxx/parse/',
  loggerAdapter: new logger({ logsFolder: './' })
});

Expected Results
I need to have cloud code logs working in locally hosted dashboard.
Actual Outcome
"No logs in the last 30 days"
Environment Setup

Server

parse-server version: latest
Operating System:    latest
Hardware:             latest
Localhost or remote server? (AWS, Heroku, Azure, Digital Ocean, etc): AWS

Database

MongoDB version: latest
Storage engine: latest
Hardware:        latest
Localhost or remote server? (AWS, mLab, ObjectRocket, Digital Ocean, etc): mLab

I've set this all up this week so every version is the latest

Logs in Dashboard are unfortunately pretty useless right now. Logs being such an important part of development I'm not sure why bringing them on feature parity with parse.com isn't high on the priority list.
I did post a similar request here https://github.com/ParsePlatform/parse-dashboard/issues/376

I could deal with having to use verbose. What I can't deal with is error messages like this:
2016-07-11T16:36:34.775Z - error:
2016-07-11T16:36:34.770Z - error:

What am I supposed to do with that? Pointing the app to parse.com doesn't return an error. The cloud code methods called all work just fine. The error handling I wrote in cloud code is returning
Error with trunkQuery.find() [object Object] (Code: 141, Version: 1.12.0)
However in the verbose log it prints out every object it is returning, which is hundreds of objects.

@tanmays we welcome pull requests and trying our best to provide a stable platform. We gladly welcome pull requests, that can simply be adding info logs for cloud code requests etc...
And yes you're right, I didn't put much focus into logging, as I don't use parse-dashboard for logging and I capture request/response logs differently.

So I have figured out the problem with my cloud code failing. I need to update it to run on Javascript SDK 1.7. I've been going through and changing current user and master key. However it still doesn't run. And again the log output is:
error:
Is there any better way to debug this error. My code is:
callback.error("Error with myQuery.find()"+error);
This returns the error to Xcode as such:
Error: Error Domain=Parse Code=141 "Error with myQuery.find() [object Object]" UserInfo={code=141, temporary=0, error=Error with myQuery.find() [object Object], NSLocalizedDescription=Error with myQuery.find() [object Object]}
Is there a better way to understand WHICH line the offending code is on? It's hard to update code for the new version without know which lines are breaking. I suppose I could go line for line and see if the syntax is right for 1.7 but that would take forever.