[SPARK-28141][SQL] Support special date values

What changes were proposed in this pull request?
Supported special string values for DATE type. They are simply notational shorthands that will be converted to ordinary date values when read. The following string values are supported:

epoch - 1970-01-01
today - the current date in the time zone specified by spark.sql.session.timeZone.
yesterday - the current date -1
tomorrow - the current date + 1
now - the date of running the current query. It has the same notion as today.

For example:
spark-sql> SELECT date 'tomorrow' - date 'yesterday';
2

Why are the changes needed?
To maintain feature parity with PostgreSQL, see <IP_ADDRESS>. Special Values
Does this PR introduce any user-facing change?
Previously, the parser fails on the special values with the error:
spark-sql> select date 'today';
Error in query:
Cannot parse the DATE value: today(line 1, pos 7)

After the changes, the special values are converted to appropriate dates:
spark-sql> select date 'today';
2019-09-06

How was this patch tested?

Added tests to DateFormatterSuite to check parsing special values from regular strings.
Tests in DateTimeUtilsSuite check parsing those values from UTF8String
Uncommented tests in date.sql

Can one of the admins verify this patch?

Merged build finished. Test PASSed.

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

Test build #110230 has started for PR 25708 at commit 775c7ec.

Test build #110230 has finished for PR 25708 at commit 775c7ec.

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

Merged build finished. Test FAILed.

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

Merged build finished. Test PASSed.

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

Test build #110236 has started for PR 25708 at commit be787f4.

Test build #110236 has finished for PR 25708 at commit be787f4.

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

Merged build finished. Test FAILed.

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

Merged build finished. Test PASSed.

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

Test build #110255 has started for PR 25708 at commit 4fb8834.

Merged build finished. Test PASSed.

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

Test build #110258 has started for PR 25708 at commit 6da7905.

Test build #110255 has finished for PR 25708 at commit 4fb8834.

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

Merged build finished. Test FAILed.

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

Test build #110258 has finished for PR 25708 at commit 6da7905.

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

Merged build finished. Test PASSed.

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

@dongjoon-hyun @maropu @cloud-fan Please, review the PR.

Merged build finished. Test PASSed.

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

Test build #110285 has started for PR 25708 at commit d57e11e.

Test build #110285 has finished for PR 25708 at commit d57e11e.

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

Merged build finished. Test PASSed.

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

The other DBMS-like systems support these syntaxes? If not, we should turn on/off these features by the compatibility mode flag https://issues.apache.org/jira/browse/SPARK-28934?

How about the timestamp case?
You dropped inf/-inf  in the supported?

@maropu Thank you for your review.

You dropped inf/-inf in the supported?

I created the separate JIRA https://issues.apache.org/jira/browse/SPARK-29006 because the feature requires special date/timestamp values and handle them in all date/timestamp functions in the special way.

How about the timestamp case?

Here is the JIRA https://issues.apache.org/jira/browse/SPARK-29012 and the PR: https://github.com/apache/spark/pull/25716 . I am still working on it because when I uncommented tests in timestamp.sql, it stated failing, see https://github.com/apache/spark/pull/25716#issuecomment-529013241

Test build #110307 has started for PR 25708 at commit dabb6ec.

Merged build finished. Test PASSed.

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

Test build #110307 has finished for PR 25708 at commit dabb6ec.

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

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

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

Test build #110312 has started for PR 25708 at commit 92c5509.

The other DBMS-like systems support these syntaxes?

@maropu I have not found other dbms that support such special values.

If not, we should turn on/off these features by the compatibility mode flag https://issues.apache.org/jira/browse/SPARK-28934?

The flag has not been merged yet: https://github.com/apache/spark/pull/25697

Test build #110312 has finished for PR 25708 at commit 92c5509.

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

Merged build finished. Test PASSed.

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

If not, we should turn on/off these features by the compatibility mode flag https://issues.apache.org/jira/browse/SPARK-28934?
The flag has not been merged yet: #25697

Yea, I meant we need to add the flag first? cc: @dongjoon-hyun @gengliangwang

Yea I think we should have the flag first.

Merged build finished. Test PASSed.

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

Test build #110367 has started for PR 25708 at commit 254567d.

Test build #110367 has finished for PR 25708 at commit 254567d.

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

Merged build finished. Test PASSed.

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

Test build #110402 has started for PR 25708 at commit d61fdc7.

Merged build finished. Test PASSed.

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

Test build #110402 has finished for PR 25708 at commit d61fdc7.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Merged build finished. Test PASSed.

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

Test build #110409 has started for PR 25708 at commit 106524b.

Test build #110409 has finished for PR 25708 at commit 106524b.

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

Merged build finished. Test FAILed.

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

retest this please

Merged build finished. Test PASSed.

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

Test build #110417 has started for PR 25708 at commit 106524b.

Test build #110417 has finished for PR 25708 at commit 106524b.

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

Merged build finished. Test PASSed.

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

@maropu @dongjoon-hyun Here is the same question as in https://github.com/apache/spark/pull/25716#issuecomment-530697767

Merged build finished. Test PASSed.

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

Test build #110922 has started for PR 25708 at commit ff92531.

Merged build finished. Test PASSed.

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

Test build #110922 has finished for PR 25708 at commit ff92531.

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

Merged build finished. Test PASSed.

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

I have rebased this on the master with merged https://github.com/apache/spark/pull/25716 . @HyukjinKwon Could you take a look at this, please.

@dongjoon-hyun @maropu @HyukjinKwon Could take a look at this PR which is similar to already merged changes for the timestamp type.

I am merging this just to unblock and easier to work further but please make sure to have a flag. Seems like that's pointed out by multiple committers such as @maropu and @cloud-fan.

Otherwise, I will revert this and #25716
Merged to master.

We already have a jira ticket for linking this feature to the flag?

This PR #25834 hides the feature under the SQL config spark.sql.dialect = "PostgreSQL"

Oh, I see. Thanks!


bundle update hpricot

0.8.2 doesn't compile on Rubinius; 0.8.6 does.
This gets us to running specs, however there are several failures, apparently due to fakefs. The outlook on that isn't good -- https://github.com/defunkt/fakefs/issues/87#issuecomment-2483385.

I had to revert this because it broke CI. I'll try to take a look when I get back home.

Looks like CI was just hitting the github outage.
http://travis-ci.org/#!/ddollar/foreman/jobs/627017

Reverted the revert. Cheers!


Use of Blob metadata lastwritetime for action

I copy files to blob but it is not possible to set the last modified date so instead I use a metadata property called lastwritetime. It would be useful to be able to delete blobs based on this metadata rather than daysAfterModificationGreaterThan. The lastwritetime is much older on a lot of blobs than daysAfterModificationGreaterThan meaning that files I would like to delete stay around a lot longer than necessary. Is this functionality something that you might look at?

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

ID: 9a558a9e-10d0-ea34-8780-d9084f87f952
Version Independent ID: 60dda4e0-7630-f358-aa3a-6a992eae61e2
Content: Managing the Azure Storage lifecycle
Content Source: articles/storage/blobs/storage-lifecycle-management-concepts.md
Service: storage
Sub-service: common
GitHub Login: @mhopkins-msft
Microsoft Alias: mhopkins

@iansmittie Thanks for providing your feedback, I'd highly recommend posting the feedback in https://feedback.azure.com/forums/217298-storage , this channel is directly monitored by the product group.
We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


BIGTOP-3708. Fix build failure of Hive against Hadoop 3.3.3

https://issues.apache.org/jira/browse/BIGTOP-3708

backported HIVE-22398for addressing Hadoop API incompatibility.
updated exclusion rule for addressing migration from log4j to reload4j HADOOP-18088.
exluded hadoop-client-api (shaded client) from transitive dependencies of webhcat to avoid compile error due to relocation.

got no issue on running smoke-tests on Rocky Linux 8 x86_64:
$ ./docker-hadoop.sh \
       --create 1 \
       --image bigtop/puppet:trunk-rockylinux-8 \
       --memory 16g \
       --repo file:///bigtop-home/output \
       --disable-gpg-check \
       --stack hdfs,yarn,mapreduce,hive \
       --smoke-tests hive
...
:bigtop-tests:smoke-tests:hive:test (Thread[Execution worker for ':' Thread 7,5,main]) completed. Took 13.293 secs.

BUILD SUCCESSFUL in 43s
34 actionable tasks: 7 executed, 27 up-to-date

Test it on my local environment too:
./docker-hadoop.sh --create 3 \
    --image bigtop/puppet:trunk-rockylinux-8 \
    --memory 16g --repo file:///bigtop-home/output \
    --disable-gpg-check --enable-local-repo \
    --stack hdfs,yarn,mapreduce,hive   \
     --smoke-tests hive

Successfully started process 'Gradle Test Executor 2'

Gradle Test Executor 2 started executing tests.
Gradle Test Executor 2 finished executing tests.

> Task :bigtop-tests:smoke-tests:hive:test
Finished generating test XML results (0.011 secs) into: /bigtop-home/bigtop-tests/smoke-tests/hive/build/test-results/test
Generating HTML test report...
Finished generating test html results (0.02 secs) into: /bigtop-home/bigtop-tests/smoke-tests/hive/build/reports/tests/test
Now testing...
:bigtop-tests:smoke-tests:hive:test (Thread[Execution worker for ':',5,main]) completed. Took 20.293 secs.

BUILD SUCCESSFUL in 1m 20s
34 actionable tasks: 7 executed, 27 up-to-date

+1,
LGTM

Merged, thanks @iwasakims


Error while update.

Hello, after updating to the latest version of the plug-in, I get the following error.
Did I do something wrong?

Thanks!

This is an error by npm, not by the plugin.  I haven’t seen this one before and I have no clue what causes it.   Did you try and just install the latest version?  And uninstalling and re-installing the plugin?

I tried to remove the plug-in to install it again afterwards.
But in the remove action I get sort of same error-message.

As I said before: no clue why this is happening.  I’d try cleaning the npm cache and/or manually deleting /var/lib/homebridge/homebridge-rpi.  And double-check that your uSD card hasn’t gone bad.


OSX 10.9 Maverick

API diff mentions:
FSEvents.h
Added FSEventStreamSetExclusionPaths()
Added kFSEventStreamCreateFlagMarkSelf
Added kFSEventStreamEventFlagOwnEvent

Exclusion paths? THAT sounds pretty damn handy.


Review data from sender mns why it is always filtered out.

Could someone please review data from sender ‘mns’?    A review seems to show that, while their data passes and is accepted into ReportStream, it is always all filtered out and goes no-where.
Here’s a query for the last week:   You’ll see we accept in N rows, but then the child reports always show everything filtered out.
Its possible this is correct - that all their data is of such low quality that no-one wants it.  However, it seems like we should tell the sender this?
Query:
select report_id, action_id, next_action, sending_org, receiving_org, item_count_before_qual_filter, item_count from report_file
where action_id in (select action_id from report_file where sending_org = 'mns'
and created_at > now() - interval '14 days')
order by created_at, sending_org;
Example result:
You can see that 5 receivers applied quality filters and it failed all of them.   This appeared to always happen, although I didn’t look past 14 days ago
image.png

Discovered that MNS have sent all of their messages with the processing_mode_code = 'D'.  This causes RS to filter out all mns's messages since it only allow message with processing_mode_code = 'P' to pass through the quality filter.  @sliu1000 sent email to notify mns to fix their messages and resubmit as many as possible.


Polygon UV strange behaviour

Hey I'm playing a bit with polygon uv and I'm trying to do something this:

So I setup a texture on repeat mode and the uv like this

and on scene I shifted the two bot/top mid points and the result is this:

That is a weird result, On my head I expect something like the first image, So this is a bug?
For what could be useful the fix if this is a bug:
Roads, Lasers and maybe plasma beams/water beams if the engine let to shift uv from code

The thing which happens is that the polygon is triangulated (so it will be sent to GPU), and one of the triangles doesn't look too good.
The best thing you can do is subdivide all sides, so triangles are correct.

But still... it is very hard to get the needed result, as triangulation does all kinds of abrupt things... Maybe custom triangulation is the only way to go ...

With visual edit it's very hard to achieve this (because we can edit UV), but if you code you can generate the correct UV for this.
look at this code made by @LeonardMeagher2
https://github.com/LeonardMeagher2/Godot-Assets/blob/master/Scenes/2DTrail.gd
he made a trail that is exactly effect you want.

Oh nice that scene could be what I'm looking for, I will look at it.
On reality the effect I want to achieve is an animated plasma beam with curves for homming plasma beam.
The animated beams on this video, but with curves:
https://gfycat.com/GlaringExaltedEthiopianwolf
So my first approach is to implement something like a algorithm for roads based on polygons and shift the uv for the beam effect.
I don't know if i could achieve that effect with another tool or plugin on godot, so if the polygon 2d with shifting uv isn't enough, my next approach is to create a module or a new node2d directly on godot for roads.

Clarifying a bit more for those interested.
What you want:

What Godot generated:

I got it working.
Looking at example on top the trick is make quads from two triangles and not a quad directly, so if somebody want the solution for this problem, here is an example for drawing a texture of 32x8 correctly on polygon without distortion:
first triangle:
0:   0,0
1:   0,8
2: 32,0
second one:
3:   0,8
4: 32,8
5: 32,0
and apply to uv and vertices from your polygon.
This is the result of my first approach:
connected points on space from a bezier + polygon + texture on repeat mode
https://gfycat.com/BrownShorttermKiwi
I don't know if exist another more optimal approach for achieve that on 2d with raycast collision on the whole laser, this one is somewhat slow.
Something like: https://wiki.unrealengine.com/Beam_Particle_(Tutorial)

@reduz this is what you meant by implementing tessellation right?

Tesselation already works. What I meamt is adding internal edge hints to
male tesselation work as intended
On Oct 11, 2015 17:56, "Mariano Javier Suligoy"<EMAIL_ADDRESS>wrote:

@reduz https://github.com/reduz this is what you meant by implementing
tessellation right?
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/2552#issuecomment-147244706.

So tessellation is related or not with the best solution for what I'm trying to do?

What is missing is tesselation hints, to ensurr polygons are always
generated the way you intend them
On Oct 11, 2015 19:49, "Hearto"<EMAIL_ADDRESS>wrote:

So tessellation is related or not with the best solution for what I'm
trying to do?
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/2552#issuecomment-147252848.

If I'm understanding you. Somewhere on godot c++ source code, there is a draw routine without flags like TESSELLATION_XXX_ON and with that I could create the example from top with quads (4 points) directly vs two triangles (6 points)?
As you see on the first attempt I finished an implementation of what I'm looking for, but is somewhat slow (< 60 fps with app 25 plasma nodes with 10 divisions and raycast collision, all uv/polygon mapping is calculated on ready only one time)
So I'm still unsure if this is the fastest approach on godot for homing plasma beams with variable size and shape + collision.

What is missing is a way tu subdivide the polygon in the UV editor to make Godot draw sub-polygons instead.
The real reason to implement these hints is to implement polygon deformation using a 2D Skeleton, but i think this should solve you problem too.

@HeartoLazor take a look to how Spine solves it in the "meshes" overview:
http://es.esotericsoftware.com/spine-in-depth

Is this still reproducible in the current master branch? If so a test project would be convenient.

No answer, so closing.


rustic: 0.9.4 -> 0.9.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for rustic is: fast, encrypted, deduplicated backups powered by pure Rust
meta.homepage for rustic is: https://github.com/rustic-rs/rustic
meta.changelog for rustic is: https://github.com/rustic-rs/rustic/blob/refs/tags/v0.9.5/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.9.5 with grep in /nix/store/bqdlhc11kqijjz87w7c7r86ycpm4p5xd-rustic-0.9.5
found 0.9.5 in filename of file in /nix/store/bqdlhc11kqijjz87w7c7r86ycpm4p5xd-rustic-0.9.5

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

rustic

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/a61c5e4851f0b8cdf9a41e940b0f56bb4af8812f#rustic

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

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

x86_64-linux

:white_check_mark: 1 package built:

rustic

Maintainer pings
cc @NobbZ @philipmw for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

@NixOS/nixpkgs-merge-bot merge


✨ Add better support to NavigationObserver for AppState scenarios

What and why?
NavigationObserver now automatically starts / stops views when the app goes into / returns from the background.
By utilizing this in conjunction with a "pending route" when the app is in the background, we can more easily support "add to app" scenarios that use "prewarmed" flutter engines.
Update documentation to remove warnings about using prewarmed flutter engines.
How?
When a Flutter View is dismissed and not visible, Flutter automatically sets the AppState to "detached," and sets the app state to "resumed" when it comes back. This is similar to what happens when an app is backgrounded (though backgrounded apps don't usually get the to the "detached" state.) By letting NavigationObserver automatically start and stop views this way, we can support pre-cached Flutter engines easily.
Also added examples of using predicates to remove FlutterViewController and FlutterActivity from sessions, though on iOS this results in the automatic view tracking missing the presenting view controller from coming back into focus. This is documented in the example readme.
Review checklist

[ ] This pull request has appropriate unit and / or integration tests
[x] This pull request references a Github or JIRA issue

CI Configuration (optional)

[x] Run unit tests
[x] Run integration tests

I just tested and you're correct - backgrounding and foregrounding re-starts the native view controller on the top.
I don't think re-calling the predicate will work -- but we may be able to take advantage of the fact that an unidentifiable view doesn't start:
https://github.com/DataDog/dd-sdk-ios/blob/a3c14772ed7912760c06bbf4ff7b0ea594c90f2d/Sources/Datadog/RUM/Instrumentation/Views/RUMViewsHandler.swift#L143-L145
If the Predicate can return a RUMView that we can then convert to a RUMViewsHandler.View that is not identifyable, it will take a slot up in the view stack but not have any view events tracked.

@ncreated Here's a pretty minimal change that we could use to do this: https://github.com/DataDog/dd-sdk-ios/commit/514f8efdb8d1e36b36b23d855a010a52ab9efccc

@ncreated Here's a pretty minimal change that we could use to do this: DataDog/dd-sdk-ios@514f8ef

Oh, thanks for trying it out 💪!
However, this seems more impactful than I thought. I was thinking that it is possible to provide uniform abstraction over RUM views auto-tracking that will entirely hide OS-specific effects of background / foreground transitioning on iOS and Android. In other words: from the cross-platform SDK point of view both native SDKs will behave exactly the same.
But it seems that the iOS-specifics of fe-/bg- transitioning must be also handled in the host (Flutter) app. For that reason I think it's better to only patch one component (Flutter SDK) instead of two (Flutter + iOS SDK). The solution provided in this PR is minimal and explained very well 👍.


Add changelog entry for 3.7.2, enable CI on stable-*

Changelog entry for #500.

Coverage remained the same at 98.993% when pulling 82cbe4ede24f469f228a032628255731d235c8c9 on isaacbrodsky:changelog-3.7.2 into f08431769f478eb59708f7cdb43580fdaae2edab on uber:master.


Need to support rate limiting:

As a user of the free Slack service
In order to avoid running out of space
I need to delete old files
And this script worked great, except for one thing:
$Old = "{0:dd/MM/yyyy}" -f (Get-Date).AddYears(-2)
Get-SlackFileInfo -Before $Old -Paging | Remove-SlackFile -Force

The problem was that I ran into the rate limit while deleting.
They obviously allowed me a large burst, and so I know the rate limit is complicated, but there needs to be some way for me to handle it.
*At the very least, you should terminate when you get a rate-limit error code back from the service.
P.S.  You should really talk about this as a feature of your module. Maybe write a short blog post about it. From a quick google search, a lot of people need this functionality, and are writing their own scripts, or even using services for it, when it should be that easy -- you could even add a helper to do something like Remove-SlackFile -OlderThan ...

Good call on a quick blog post and a -OlderThan param; a bunch of stuff has gone into the module that's not really covered in the readme or a post!
@hmmwhatsthisdo have you run into any trouble with the rate-limiting you added?  I'll poke around this, but some of it went over my head : P

Context on rate limiting in #68 - looks like it's actually skipping the rate limiting logic - @Jaykul can you paste $psversiontable?
Thanks!

Makes sense to me, and significantly cuts down on complexity
This work for you @hmmwhatsthisdo ?  This would let us basically use Slack itself (their response) as the rate limiting mechanism, allowing us to go as fast as they'll let us
Cheers!

w/r/t Remove-SlackFile not respecting the rate limit, this is because the internal Send-SlackAPI invocation isn't using the -RateLimit parameter. When I initially set up the rate-limit mechanism, I made it opt-in because not all API calls were rate-limited at the time and I wanted to avoid making a breaking change. Either resolution (tacking on -RateLimit or making rate-limiting opt-out instead of opt-in) should be fairly straightforward, I think.
As for the rate-limiting algorithm itself, I initially implemented the leaky-bucket method because Slack had not documented their rate-limiting process as well as they have now. However, the important bit that stuck out when I last parsed their documentation was that an API key getting rate-limited too often was subject to getting revoked by Slack. If Slack has reversed their stance on this, I think that relying solely on Retry-After would be most appropriate. (It may be best to check with Slack Support to confirm, as it could entail some rather unsavory consequences if their position hasn't changed.)

The disconnection of clients and disabling of API keys is specific to the chat.postMessage API and the real time messaging APIs, to prevent spammers. See "responding to rate limiting conditions" section, and particularly the "limits when posting messages" part.

Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

@Jaykul or @hmmwhatsthisdo - any tips on cleaning this up would be awesome : )
@hmmwhatsthisdo or anyone else relying on rate limiting - any concerns with this approach, based on the docs?  I don't know anyone at Slack, but if anyone is concerned, could probably ping them to see if relying on 429 / rate-limit is appropriate?  From my perspective it makes sense (otherwise everyone writing libraries would need to come up with their own algorithms to avoid even hitting the 429, which seems odd)
Cheers!


fix issue 15364 - BitArray.len should be private

LGTM

LGTM

These public members were all not documented, so it makes sense that we don't need to worry about breakage (e.g. bitsPerSizeT was harmlessly public)

whitelisted @Blumerline

Auto-merge toggled on

whitelisted @Blumerline

Whitelisted for what?

Whitelisted for what?

autotester.

Yeah, except they did break things :-(

Any broken code was using undocumented features that were not meant to be accessible. If read-only access is necessary, then we can add public accessors. But ptr and len should not be writable.

We always need to worry about breakage, this was so unnecessary.
Nobody would have an issue w/ properly deprecating access to those fields, but just randomly breaking code b/c sth. should have been done different is unacceptable.
https://github.com/D-Programming-Language/phobos/pull/3950

No, changing len or ptr or both externally is neither supported nor could possibly be accepted as valid code.
Any code that broke because of this had a bug, and now the person would see the bug should be fixed. We did anyone who's code broke a favor.
This is like saying fixing a buffer overrun broke my code.

http://imgs.xkcd.com/comics/workflow.png

Any code that broke because of this had a bug, and now the person would see the bug should be fixed. We did anyone who's code broke a favor.

It certainly wasn't used incorrectly in undead, and every breakage we introduce requires library maintainers to release new versions and uses  to update their dependencies.
If we don't deprecate such changes properly we cause a lot of downstream work and devaluate existing D code. We should be more careful with our tiny ecosystem.

It certainly wasn't used incorrectly in undead

Certainly it wasn't when it was part of Phobos. As a separate 3rd party library (and isn't this the entire purpose of this library, to maintain support with the current phobos?), it should not be accessing internals of Phobos implementation. So yes, it was used incorrectly (although now fixed).
We can agree to disagree on this. I don't think we should cater or support any 3rd party library that uses internal details of Phobos implementations. You can disagree with that, but I can't have any care or sympathy for those libraries, sorry.

We don't disagree on the fact that internals should be internal, but I'm arguing against simply breaking things b/c someone made a mistake years ago. The cost of properly deprecating those fields is literally zero.


seed from +ntb_random_seed or +seed rather than falling back straightawa...

...y to current time.  Just a small change in ordering in _initialise_testbench to process plusargs before seeding, and then use either +ntb_random_seed (for VCS) or any +seed plusarg that may have been defined.
Makes it possible to seed both VCS and cocotb from the same plusarg, rather than having to set multiple seeds in environment variables and on the command line.

Apologies @GordonMcGregor, somehow I missed this notification, merging.

No, could just be treated like an int.  I often use eval in internal tools so that hex/bin/ints can be passed in (though I confess to not worrying about pathological users trying to run exploits)  The eval benefit isn't necessary here, certainly.


remove extra comma - resolve bus service provoder

remove comma and resolve bus service provider

Trailing commas are valid syntax in PHP 7.3. You cannot use Laravel 8 on PHP less than 7.3.


add no_cursor_timeout to mongo.find

When running queries in analysis that retrieve a big number of convos we're getting the error
pymongo.errors.CursorNotFound: Cursor not found

Adding this flag as advised here: https://www.programmersought.com/article/5325272332/
Tell me wdyt

When running queries in analysis that retrieve a big number of convos we're getting the error
pymongo.errors.CursorNotFound: Cursor not found

Adding this flag as advised here: https://www.programmersought.com/article/5325272332/
Tell me wdyt

Accoring to this link you have to close .close() the Cursor object manually. It will stay open otherwise

@erez-work @davidstrouk seems like this is forgotten


⚠ Bofrost.be has degraded performance

In ff2d2f4, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1347 ms

Resolved: Bofrost.be performance has improved in 7e54c1f after 42 minutes.


Deployed Version redirects to localhost for authentication

Describe the bug
i am following the custom MyAuthProvider for authentication and it works on localhost
but when i deploy it on server and hit {server_url}/admin it shows https://{server_url}/admin/login?next=http%3A%2F%2F<IP_ADDRESS>%3A8000%2Fadmin%2F
which means its redirecting to localhost
here is my setup
def init_admin(app: FastAPI) -> None:
    engine = create_async_engine(get_db_url())
    admin = Admin(
        engine,
        title="Admin",
        base_url="/admin",
        statics_dir="static",
        login_logo_url="statics/dark-logo.svg",  # base_url + '/statics/' + path_to_the_file
        auth_provider=MyAuthProvider(allow_routes=["/statics/dark-logo.svg"]),
        middlewares=[
            Middleware(
                SessionMiddleware, secret_key=SECRET
            )
        ],
    )

the login form that shows up is posting on http://<IP_ADDRESS>:8000/admin/login instead of the server url, i am thinking there is some kind of setting for this issue can anyone guide me
Environment (please complete the following information):

Starlette-Admin version: 0.13.2
ORM/ODMs: SQLAlchemy

Additional context
Add any other context about the problem here.

issue was with my nginx config solved that but now having issues of serving mixed content block

issue was with my nginx config solved that but now having issues of serving mixed content block

What do you mean by "mixed content block"?

my nginx config redirects every http call to https
server {
    listen 80;
    listen [::]:80;

    server_name domain.com;

    # Redirect all HTTP traffic to HTTPS
    if ($http_x_forwarded_proto != "https") {
        return 301 https://$host$request_uri;
    }

    location / {
        # Proxy pass to your application
        proxy_pass http://<IP_ADDRESS>:8000/;
        proxy_set_header Host $host;
        proxy_set_header X-Real-IP $remote_addr;
        proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_set_header X-Forwarded-Proto $scheme;
        proxy_set_header Upgrade $http_upgrade;
        proxy_set_header Connection upgrade;
        proxy_set_header Accept-Encoding gzip;
    }

    autoindex_localtime on;
}

now admin website loads on https but when it tries to make the call to list api it makes it through jquery and on http and those calls get blocked.

@usmanarham You should configure uvicorn / fastapi to respect x-forwarded-* options.
Option 1. There's a command-line key for that if you are running the app using uvicorn app. See here: https://www.uvicorn.org/deployment/#running-behind-nginx
Option 2. You can set ENV FORWARDED_ALLOW_IPS to * as the environment variable. Since we use Docker for running our apps, we've done it like this right in the Dockerfile that is used to build the app image.

The solution can be found here -> https://jowilf.github.io/starlette-admin/deployment/


docs: fix path to logo

Overview
Change path to taskcat.svg to render svg logo
Testing/Steps taken to ensure quality
Manual verification of README in branch

@tonynv, I noticed you just made a fresh PR for something similar. There are 2 READMEs that have the old broken logo link. I've modified my PR to use the link that you used in your PR for both READMEs. Requesting your review for this. Thanks!

Please resolve conflicts

Thanks @andrew-glenn for resolving the conflicts!


Add design responsivity

@ondrejbartas I had problems with fuzzy to ignore blank spaces. Can you look at it? Thanks.

@imtoo it is just about what you are sending to fuzzy search. You can remove blank spaces before you are sending searched text to fuzzy search 😄

@imtoo pls ask somebody to fix testsfor it ( @honzabrecka )

@ondrejbartas Yes, I know that. But as a result in AllComponentsPreview you get what you searched in. So there are names of components without spaces ExampleComponentsButton instead of ExampleComponents Button, I think we would like the example with spaces.
I'll ask @honzabrecka

@ondrejbartas Tests fixed.


yesterday before your update the version its work
My environment
v16.13.0 node

Try to reinstall the @microsoft/sharepoint generator

I get the same error when trying to use it with @microsoft/sharepoint.
OS: macOS 14.0
Node: 18.18.2 (v16 and v18 are supported by Sharepoint Framework v1.18.*)
Here is the debug output:
yo @microsoft/sharepoint --debug
Error @microsoft/sharepoint --debug

Error: Current environment doesn't provides some necessary feature this generator needs.
    at new Generator (/usr/local/lib/node_modules/@microsoft/generator-sharepoint/node_modules/yeoman-generator/lib/index.js:259:13)
    at new BaseGenerator (/usr/local/lib/node_modules/@microsoft/generator-sharepoint/lib/common/BaseGenerator.js:44:9)
    at new AppGenerator (/usr/local/lib/node_modules/@microsoft/generator-sharepoint/lib/generators/app/index.js:61:9)
    at FullEnvironment.instantiate (file:///usr/local/lib/node_modules/yo/node_modules/yeoman-environment/dist/environment-base.js:252:27)
    at FullEnvironment.create (file:///usr/local/lib/node_modules/yo/node_modules/yeoman-environment/dist/environment-base.js:246:21)
    at async FullEnvironment.run (file:///usr/local/lib/node_modules/yo/node_modules/yeoman-environment/dist/environment-full.js:307:27)
    at async init (/usr/local/lib/node_modules/yo/lib/cli.js:168:5)
    at async pre (/usr/local/lib/node_modules/yo/lib/cli.js:83:3)
    at async /usr/local/lib/node_modules/yo/lib/cli.js:193:3

As workaround install v4 npm install yo@4.


ReferenceError: window is not defined when importing progressive-image.js

I want to add https://github.com/craigbuckler/progressive-image.js to my vite-ssg app and do this by adding
import 'progressive-image.js/dist/progressive-image.js'
import 'progressive-image.js/dist/progressive-image.css'

to the main.ts. On Build for server... i get ReferenceError: window is not defined.
I used vitesse as base and know it's possible to create a module. But i cannot import the two resources in a isClient if clause.
Is there any way to import these resources only on client generation only?

I've created a demo repo https://github.com/mseele/vitesse-progressive-image.js. Commit https://github.com/mseele/vitesse-progressive-image.js/commit/d9e579d5a76327f706417831d172d9009ccd5578 shows my problem. How can i fix this? Thank you.

I have same issues

I have same issues

I get that there's no window in SSG but what's the escape hatch here?
I put the entire App.vue router-view in https://github.com/antfu/vite-ssg#clientonly tags but it doesn't budge.

I have the same issue (ReferenceError: window is not defined) while playing around with p5.js sketches using vitesse.
I'd like to exclude the component from SSG but wrapping the components in  doesn't work for me.

The problem here is that libraries that are not SSR/SSG aware should be loaded with dynamic import, for example, for progressive-image:
 if (typeof window !== 'undefined') {
  import('progressive-image.js/dist/progressive-image.js')
 }

or using onMounted / beforeMount hooks.
You can see an example on src/modules/pwa.ts module on vitesse template, where the virtual:pwa-register module (this guy will register the service worker) is loaded only if isClient is true.


Trying to get the Image base64 encoded that is the result of the zoom and transform

Hi, thanks for building Zoomist... is far the best zoom library.
I have used without issues and works brilliantly but now I need to get back the image that has been transformed as base64 to be uploaded in the server.
I have tried several things but the image is just stays as the original image and I can see only the styles properties changing so I don't have a way to get the new image fully transformed.
If you can please help.

Hey @luigipam ,
Thanks for your like.
It's there any example code to let me know your situation more? It will be helpful to solve your problem!

I'm just trying to get the base64 image that is zoomed and translated, I tried to call several methods but they do not return the image.
I'm looking for some method like getImage (not getImageData) or export (other zoom libraries have an export function) that returns a base64 string with the actual image.

Has anyone found out how to retrieve a base64-encoded image after zooming it?


[item_drop] : add item pickup and drop callbacks

For a plethora of tiny, interesting and useful reasons (such as quests, see #145, and future money pickup/drop mechanisms), at least two callbacks must be implemented in item_drop:

on_pickup(picker, itemstack)
on_drop(dropper, drop_entity, itemstack)

Please note this must not be done as an inventory watchdog timer, since we don't want to handle cases where there are no actual drops/pickups, that is, item giving/removal from a script or command, which is left to be handled by the mods who want to be notified about it.

on_pickup would be easily implementable in item_drop itself, but on_drop would not be. Drop is an engine feature, and there is no API to handle or manipulate it in any way.
Even though these two callbacks would be great to have, one of them actually requires to be hardcoded into the engine.

I thought about an on_drop callback, and the only way to launch methods registered in that callback table would be to override __builtin:items to start them when they're activated. What do you think of that idea?

73fb22c48193a8acff6a511bfb8b30e67a473288

Great, im happy to see this issue done ! :)


[WIP] ASG watcher

Early feedback welcome.
Code mostly copied from ec2tag watcher.
Issue #182

Looks pretty good. I think if we want this to really scale we'll have to do Pub/Sub via SNS notifications, but I dig it as a v1


Add Whisper to ORT optimizer configuration

What does this PR do?
This PR adds Whisper to the ORT optimizer configuration. This ORT PR shows how to use Whisper with ORT.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Impressive work @kunal-vaishnavi! I'll merge this one and have a more thorough look at your PR to see if there's more changes needed.


Adding liveness probe test and refactoring ItDomainInPV

Adding liveness probe test
Refactoring ItDomainInPV to have separate test methods for creating a domain, verifying t3 channel port, LB port, default node port.
Jenkins - https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/346/
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-model-in-image-tests/924/
Kind cluster entire suite - https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/361/ - failure unrelated to the changes in PR, docker pull failing with net/http: request canceled
non-kind cluster Jenkins - https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-model-in-image-tests-10/349/
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-model-in-image-tests/935/ - some intermittent failures not related to the changes in the PR
develop branch also has some intermittent failures -
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-model-in-image-tests-10/344/

If I move the Nginx installation to the test method testAccessAppUsingLBPort, on mii-10 Jenkins the test is failing in the utils call to webapp even though the checks are done for Nginx ready and ingress creation before making call to the webapp using LB. So, I moved back the Nginx installation to @beforeall method for now which is working, will check with Xian to see if any more checks need to be done for Nginx. This PR doesn't have to wait for that.
<oracle.weblogic.kubernetes.utils.TestUtils callWebAppAndCheckForServerNameInResponse> <Calling webapp at most 20 times using command: curl --silent --show-error --noproxy '*' -H 'host: wdtdomain-inpv.cluster-wdtdomain-inpv.test' http://<IP_ADDRESS>:30305/testwebapp/index.jsp>
<06-14-2020 13:42:51> <INFO> <oracle.weblogic.kubernetes.utils.TestUtils callWebAppAndCheckForServerNameInResponse> <Response for iteration 0: exitValue 7, stdout , stderr curl: (7) Failed connect to <IP_ADDRESS>:30305; Connection refused>

Since you are modifying some common code, can you please run the entire suite, not just the DomainOnPv class.

updated Jenkins results for entire suite


Movable line update

PT-187185075
Updates the movable line to have a design closer to CLUE spec.
Changes how the drag events work - dragging either of the handles will rotate around the other one. Dragging any other point on the line moves the line without changing its angle.
This also adds volatile state to the movable line model, so that changes are not actually recorded until the drag is complete.  This means (a) intermediate states will not be visible to collaborators and (b) only a single undo event is recorded for each drag.

@johnTcrawford did you want to review the tests in here ?


KAFKA-3242: minor rename / logging change to Controller

KAFKA-3242: minor rename / logging change to references to 'adding partitions' to indicate 'modifying partitions'

One small comment, otherwise LGTM.

Thanks Grant. Changed to string interpolation. @gwenshap can you merge.

LGTM.


xfreerdp : which gstreamer's version need to install on the newest freerdp codes?

I want to use the newest codes, but i cannot know to use which gstreamer's version ?

I installed the gstreamer version   gstreamer-0.10.36-7.el7.x86_64
when I use the multimedia redirection , some errors happended: (Failed to create new pipe)

default is gstreamer 1 by now, there is a cmake option to reactivate 0.10. there may still be some issues, look at the pull requests pending for tsmf.

@ygqzwl Are your issure resolved?


Allow Sign-in to Kiosk Mode

Is your feature request related to a problem? Please describe.
I've been thinking about this one a lot over the past year, as I think about ways that students can show off their work and play their own games. Now that projects can be stored in the cloud, the ability to access those projects becomes much simpler.
Describe the solution you'd like
I'd like to see a Sign In button, appropriately styled, available in kiosk mode, so that the user's own projects (either all of them or perhaps a shorter list of "starred" or "favorite" projects) appear in the carousel.
Describe alternatives you've considered
I've considered a separate website where users can register themselves and their projects that they want to make publicly available. If something like that is already built into MakeCode Arcade, though, then this becomes unnecessary.
I also considered a UWP app, available in the Microsoft Store, that includes just the Arcade simulator. The app could automatically pull any projects already stored in the cloud that are tied to the logged-in user. Kiosk mode with a sign-in button, though, would make this unnecessary.
Additional context
How cool would it be to be able to show off our games on an Xbox! Kiosk mode gives us a simple way of doing that.

@alex-kulcsar great idea.


Add copy button to glyph pages

Oh, I just noticed that this is actually still a draft. Sorry if I jumped in too soon. Anyway, maybe the feedback is still helpful?

Not at all, thank you for catching those things. They should all be resolved in the latest commit.

@kwiliarty Is this what you had in mind for tests? Not 100% sure the expectToBeCalled lines are correct, but the tests pass.

@andyzito : This is still marked as a draft. Was there something else you wanted to do on it, or do you think it's ready to go?

@kwiliarty Ready!


Add score-based spam detection besides blacklist

I'm submitting a ...
[ ] spammer report
[ ] bug report
[X] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary

It seems that this tool is only a simple blacklist - but I think some kind of negative scoring system may be introduced.

Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)

I believe we can score PRs negatively (and positively) and mark as spam if a defined threshold is met. For example some things deducting score may be:

Changes only in text files (.md, .html).
Changes only in one file (or removal a single file),
Changes only in one line,
Changes consisting of words "awesome" or "amazing" ;) (aka: blacklisting words in commits messages and diffs themselves),
Empty descriptions,
"patch-1" as a name of remote branch.

Of course, it's not the best solution, as it won't be 100% bulletproof, but what do you think?

That is not really possible. someone changing 4 words with actual typo's can be a sincere pull request, and if I would make a sincere pull request and get labelled as spam right away I'm not sure if I would spent my time on a project like that.

In my mind most (or all, or configurable number of) checks must be met to PR be marked as spam, so legitimate correcting of typos shouldn't trigger anything.

Thank you for your contribution @ktos! As said by @StefanJanssen95 we need to refine the criteria to detect sincere PRs and spam PRs the more accurate way possible.
As planned in #1, if we are detaching the blacklist from the build and make it an external JSON file, we absolutely can add some more details and indicators attached to a user.
It implies defining a new and more complete model based on criteria we would use to compute a trust-score.
In addition, it would be nice to let the user configure its own minimal allowed threshold in the GitHub Action. Feel free to make some suggestions, propose some code and make some PRs.

I think this is a cool idea, but hard to implement. Someone may train a neural network on the database to find some relation between the specs and score the PRs.

I think this is a cool idea, but hard to implement. Someone may train a neural network on the database to find some relation between the specs and score the PRs.


added the wave sort array program alongwith the README.md file.

Hi! I have added the wave sort array program to the issue "Adding new programs and their solutions. #70"...I have created the folder Wave_Sort_Array inside the array folder in the repo and within that folder, I have included one c++ source code file and the README.md file describing the working of the code. Please if you like my code then please merge my PR to your repo. Thank you.

Thank you @skully-coder  sir :-)


Data corruption during reads on Linux/OSX

The currentlly released version of Microsoft.Data.SqlClient (1.0.19239.1) has a severe issue (https://github.com/dotnet/SqlClient/issues/145#issuecomment-526625414) where data corruption happens for read values that spans two packets. The error only happens when the managed tds parser is used which it is on Linux and OSX.
Corrupted reads may throw an exception because it can cause an invalid state in the internal tds parser, but it can also happen that corrupted values can be silently read.
A fix is merged to master (https://github.com/dotnet/SqlClient/pull/171) but is scheduled for a 1.1 release which is likely after the release of EF Core 3.
This issue is intended to track the release of M.D.SqlClient and to make sure that the dependency is upgraded if it happens that a patch release out before the release of EF Core 3.
Further technical details
EF Core version: 3.0 preview 8
Database Provider: SqlServer with M.D.SqlClient
Operating system: Linux and OSX

@ajcvickers this have now been released I M.D.SqlClient so it is now ready to be fixed

@rmja Thanks. @bricelam is working on a PR, which we will then need to get approved.

@ajcvickers @bricelam Is it correct that this change is not included in rc1?

@rmja - That is true. It will be included in 3.0 release.


Vulnerability CVE-2022-32207、CVE-2022-32205、CVE-2022-32206、CVE-2022-32208、CVE-2022-35252

Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
tf 2.6.5
Custom Code
No
OS Platform and Distribution
EulerOS 2.5\EulerOS 2.10
Mobile device

Python version
3.9
Bazel version

GCC/Compiler version

CUDA/cuDNN version

GPU model and memory

Current Behaviour?
Hi there, it seems that curl 7.83.1 has a vulnerability CVE-2022-32207 CVE-2022-32205 CVE-2022-32206 CVE-2022-32208 CVE-2022-35252. Please fix it or confirm tf 2.6.5 is not affected by it.
Here is the detail:
https://nvd.nist.gov/vuln/detail/CVE-2022-32207
https://nvd.nist.gov/vuln/detail/CVE-2022-32205
https://nvd.nist.gov/vuln/detail/CVE-2022-32206
https://nvd.nist.gov/vuln/detail/CVE-2022-32208
https://vulners.com/redhatcve/RH:CVE-2022-35252

Standalone code to reproduce the issue
-

Relevant log output
-

TF 2.6 is no longer in the window of supported versions

Can you help me analyze whether the TF 2.8.3 version is affected by these vulnerabilities?

Main branch: https://github.com/tensorflow/tensorflow/blob/92d57e61d41f8e6fca50d582ac7a56046a391591/tensorflow/workspace2.bzl#L493-L500
r2.10: https://github.com/tensorflow/tensorflow/blob/359c3cdfc5fabac82b3c70b3b6de2b0a8c16874f/tensorflow/workspace2.bzl#L491-L498
r2.9: https://github.com/tensorflow/tensorflow/blob/18960c44ad3f5219c22dca55f842912dbce78a07/tensorflow/workspace2.bzl#L483-L490
r2.8: https://github.com/tensorflow/tensorflow/blob/92a51d52ad199319e4f9de83fcbe970151dfed7e/tensorflow/workspace2.bzl#L490-L497
So, if you take the last release on each of these branches you are not affected

https://curl.se/docs/CVE-2022-35252.html
CVE-2022-35252 is fixed in CURL versions 7.85. 0 and later.

Then you might be affected on all supported versions. TF does not generate VEX statements at this point, so we cannot be certain.

@crbean,
Could you please take a look at this comment from the developer. Thank you!


One Day Race results

I can't get results for any one-day races. I try to load them as stages, but I get error below. Is this user error or is it impossible to get 1 day race results?

IndexError                                Traceback (most recent call last)
Cell In[16], line 2
1 stage = Stage("race/paris-roubaix/2024/")
----> 2 stage.results()
File ~/Documents/cycling_stats_exploration/.venv/lib/python3.12/site-packages/procyclingstats/stage_scraper.py:315, in Stage.results(self, *args)
312 # remove other result tables from html
313 # because of one day races self._table_index isn't used here
314 categories = self.html.css(self._tables_path)
--> 315 results_table_html = categories[0]
316 # Results table is empty
317 if (not results_table_html or
318     not results_table_html.css_first("tbody > tr")):
IndexError: list index out of range

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Hello @harryelworthy, try following:
stage = Stage('race/paris-roubaix/2023/result')
stage.results()

Seems like PCS added rider relegation information on the result list and they are not being parsed properly. Something to flag to the developers. This is the reason why 2024 results return parsing error.
In your example you are also missing /result at the end of your url.


GSoC Project: Defining a OM/ML Message Type within Hyperledger Aries

Hyperledger Aries is an open-source reference architecture for developing self-sovereign identity agents of which there are many implementations under development, although the python one is probably the most relevant. Aries agents send asynchronous messages using a protocol called DIDComm, these messages have a type which defines the structure of the message. Each message type has it’s own handler for implementing agent specific responses to these messages. Currently the message type Basic Message, is implemented in most agent implementations. We developed a PoC which hijacked this basic messaging feature to send ML messages in a federated learning example. A better implementation of this would be to define a specific message type for distributed learning communication - some exploration would be needed to understand the exact requirements of this message type. For example should there be one message type for each distributed learning implementation, or should type be in the data structure of the message?
Required Skills:

Willing to explore a complex code base
Familiarity (or willingness to become familiar) with Hyperledger Aries
Familiarity (or willingness to become familiar) with PySyft
Familiarity (or willingness to become familiar) with PyGrid

Difficulty: Medium
This project should be fairly straightforward, although an initial learning curve to get to grips with the aries architecture will be required. A best case scenario would be a ML Message Type ratified by the Hyperledger Aries community and included as an RFC similar to the Basic Message and implemented as a reference in one of the aries implementations - or a fork of one of these projects.

Hi, Will, @wip-abramson
I'm Vineeth, currently an undergraduate student at National Institute of Technology Karnataka. I'd like to be considered for this project in GSoC. I'm experienced with DL and Pytorch, and have experience working with PySyft. I have gone through some of the concepts regarding Hyperledger Aries.
I'd like to begin working on the project and collaborate with you. Any suggestions for how to get started ? I hope to contribute and get selected in the project. Thanks.

Hi Will @wip-abramson,
I'm Duy, an undergraduate at the University of Science, Vietnam National University - Ho Chi Minh City. I would love to start working on this.


rbd: remove false error check in getDeviceSize

this removed err condition will be always false as error is always nil.
Signed-off-by: Humble Chirammal<EMAIL_ADDRESS>

@Mergifyio requeue

/test ci/centos/upgrade-tests-cephfs

/test ci/centos/mini-e2e/k8s-1.24

/test ci/centos/upgrade-tests-cephfs

/test ci/centos/mini-e2e/k8s-1.24

PR was rebased, need to remove/add the ok-to-test label (see #3509 for automation around this)

/test ci/centos/upgrade-tests-cephfs

@Mergifyio requeue


Ensure failoverCandidates have no servers not in Plan.

Fix failoverCandidates for servers not in Plan.
Problem: we must never fail over to a server which is not in Plan
because it might not have the data.
ToDo:

[X] AddFollower, FailedFollower, CleanOutServer and MoveShard will not
add a server to the Plan which is in failoverCandidates for this
shard or any of the ones with distributeShardsLike.
[X] add preconditions to ensure that it is snapshot-safe
[X] make dbserver Maintenance remove servers that are not in Plan from
failoverCandidates.

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/12183/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/12189/ new try after fixing compilation.

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/12205/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/12205/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/12212/

Tests blue.


Stacked histogram

IMPORTANT: Please search among the Pull request before creating one.
Title

Types of changes

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

Description

Add cr.tl.estimators.GPCCA.plot_macrostate_composition.
How has this been tested?

TODO.
Closes

closes #635

Very nice, see Fig. 1 below. Comments

For the y-ticks, I would make sure that we always have one tick corresponding to the total cell number per macrostate, i.e. 30 in this case. I just expect to see a label corresponding to the number of cells per macrostate when looking into compositional questions.
the color issue has been fixed - well done @michalk8!
what would happen if one macrostate had less than 30 cells assigned to it? I.e, think of the situation where we compute many macrostates in the pancreas - some will get less than 30 cells assigned to it because they are dominated by other states that "eat up" all of their cells. Would this throw an error? I guess the correct behaviour would just be for one of the stacks to be a bit smaller - can you check this one please? Should be possible by using the pancreas and computing many macrostates

Once that's done, this can be merged!
Fig. 1

For the y-ticks, I would make sure that we always have one tick corresponding to the total cell number per macrostate, i.e. 30 in this case. I just expect to see a label corresponding to the number of cells per macrostate when looking into compositional questions.

I've maded sure that we see the tick corresponding to the highest bar.

the color issue has been fixed - well done @michalk8!

Thanks.

what would happen if one macrostate had less than 30 cells assigned to it? I.e, think of the situation where we compute many macrostates in the pancreas - some will get less than 30 cells assigned to it because they are dominated by other states that "eat up" all of their cells. Would this throw an error?

Works as expected, see below:


Html code keeps rendering

Hi
I've integrated the rainbow-library into my webpage.
Whenever I put some html-code into my page and put it between the pre- and code tags ... the html-code is still rendered.
Example:

Hello World

output:
Hello World
So, I don't see the paragraph-tag anymore.
Can anyone explain what I should do to fix this ?
(info : it's an ionic-project)
Regards
Joost

You have to escape the html entities. Otherwise the browser will render them as normal HTML. See #145

Do you ever get tired of answering this?  You know you could probably build this support issue into the library itself right? :-)  If the block element has any children nodes you could show an error or a debugging message telling the user that they need to properly escape the HTML before highlighting it.


Misprint

"Note If the call to this function occurs in user mode, you should use the name "NtCreateFile" instead of "NtCreateFile"."
Probably meant to say "from kernel mode" and "name ZwCreateFIle"

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

ID: 91b87ed3-cc07-a11d-54bb-61e41670bf65
Version Independent ID: 770a9821-b066-0436-3bcc-0d950c9e97aa
Content: NtCreateFile function (ntifs.h) - Windows drivers
Content Source: wdk-ddi-src/content/ntifs/nf-ntifs-ntcreatefile.md
Product: windows-hardware
GitHub Login: @lorihollasch
Microsoft Alias: loriwhip

@alfaunits, thanks for catching this - it is now corrected (use NtCreateFile instead of ZwCreateFile).


Update vulnerable packages

This pull request also includes a package-lock.json file required by npm audit and a note for future contributors to run the audit before a PR.
Happy to remove or change any commit! Thanks for looking.

See issue https://github.com/dawnlabs/alchemy/issues/47


checkURL fails on certain bodies

Hi,
I'm having a great deal of difficultly downloading this model: https://huggingface.co/nickmuchi/sec-bert-finetuned-finance-classification/tree/main
I'm getting an error about it failing to deserialize the hfFile. However I've compared the responses of the hugging face api and I cannot see any difference in the structures.
It's failing around line 120 of downloader.go.
Any help you can provide would be greatly appreciated.

you can download it yourself and use locally (without downloader.go)

Thanks for the report, i've fixed the issue and started a PR. The workaround above of manually downloading will work in the meantime.

So what I ended up doing was adding several git clones to my deployment script. Maybe it makes sense to add a dependency on git instead


Enable golang e2e test in CI

Description
Enable golang e2e test in CI
Issues
n/a
Type of change
List the type of change like below. Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would break existing design and interface)

Dependencies
n/a
Tests
CI test

@irisdingbj If you want to add e2e tests for microservices-connector, please follow my guidance in scripts/e2e/go-test.sh and add your logic in three functions install_gmc, validate_gmc and cleanup_gmc. I've tested that any changes under the folder microservices-connector will trigger the go-e2e workflow. Please help to review and merge this PR. Then you can continue your development base on it. Thank you.

Thanks for the support!  Let's merge this first and we can address the env var and filename in follow up PR.


Input fields are cleared when JS finished loading

[x] Before posting an issue, read the FAQ at https://superforms.rocks/faq and search the previous issues.

Description
When the internet speed is fairly slow, JS is enabled and the user types in text into input fields, these values get reset as soon as the JS finished loading.
See the following screenshot of the final loading state when entering "test" into both input fields as soon as they were interactable while setting throttling to "Fast 3G".

If applicable, a MRE
I created https://github.com/Gerschtli/superforms-bug-input-values-reset-on-load to reproduce the issue (on stackblitz it was not reproducible).

Run pnpm install
Run pnpm dev -- --open
On the page execute the described steps:

Load this page with "Fast 3G" or similar
Enter some values into both inputs immediately
Wait for JS to be loaded completely and notice how the first input value is cleared (see also console.log statement)

This is not a Superforms issue, but is more general; if you use bind:value with a normal variable, the same problem occurs.

Hi! I am running into the exact same problem: in my tests, my form is reset after Playwright has already filled the values, and thus the subsequent expect fail because the form is reset at some point. If I add a pause, then everything is ok and the form is not reset. Here is a functioning test:
test('when password is too short', async ({ page, pom: { password } }) => {
	// Form is reset if the following line is removed
	await page.waitForTimeout(1000)
	await password.fill('short')
	await password.blur()
	await expect(page.getByText('String must contain at least 6 character(s)')).toBeVisible()
})

I understand that this issue is not caused directly by Superforms, but I did not find a relevant issue in Svelte or SvelteKit repos (maybe I missed it :-\ ).
@Gerschtli You said that you were going to move it to the svelte repo. Can you point me (and all the future people that will come here) to that very issue please, so that I can see what answer has been provided and maybe continue the discussion? Many thanks! 🙏

To anyone passing by and wondering where this issue is, I finally found it 😄


Update phpMQTT.php

The password flag should be set only if there is a username, per http://docs.oasis-open.org/mqtt/mqtt/v3.1.1/os/mqtt-v3.1.1-os.pdf, line 525:
"if the User Name Flag is set to 0, the Password Flag MUST be set to 0 [MQTT-3.1.2-22]. "

Note: this is not the case in MQTT 5.0, where they reverted the requirement (line 670 of https://docs.oasis-open.org/mqtt/mqtt/v5.0/mqtt-v5.0.pdf):
"This version of the protocol allows the sending of a Password with no User Name, where MQTT
v3.1.1 did not. This reflects the common use of Password for credentials other than a password. "
Ideally speaking, we should also provide the protocol to use (optionally) and default to 3.1.1.


2024 DevOps CoP Leads Agendas

Overview
This issue tracks meeting agendas and summaries for the DevOps CoP Leads meetings.
Agenda Quick-Links
21 Feb

21 February
Back to Top
AGENDA

21 February
Back to Top
AGENDA

28 February
Back to Top
AGENDA

[ ] https://github.com/hackforla/ops/issues/117
[ ] https://github.com/hackforla/ops/issues/116

Create Issues

Confirm Schema provisioning sql commands #157
is this issue still relevant or required by incubator?

October Agenda
Back to Top
AGENDA


v2

[x] Remove EventEmitter
[x] Remove internal ttlSupport flag for tests
[x] Drop node14 (since pnpm doesn't support it)
[ ] Explicit pass a store

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

40 of 40 (100.0%)  changed or added relevant lines in 5 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.09%) to 98.081%

Files with Coverage Reduction
New Missed Lines
%

packages/mongo/src/index.js
3
94.89%

Totals

Change from base Build<PHONE_NUMBER>:
0.09%

Covered Lines:
1165

Relevant Lines:
1180

💛  - Coveralls


docs: Improve suggestions about WSL2 docker options

The Issue
The WSL2 install docs don't make clear that docker-ce is considered best.
How This PR Solves The Issue
Improve the wording.

I'm not sure if I represent a typical developer journey, so I read about WSL 2 and invested in a new machine and installed Windows 11. Because WSL2 is very strongly tied to Docker Desktop, that's what I get when I google "wsl docker". So, I proceeded to install both. Then days/months later, I learned about DDEV. In this scenario, what do I do? I would proceed with Docker Desktop.

Because WSL2 is very strongly tied to Docker Desktop, that's what I get when I google "wsl docker". So, I proceeded to install both. Then days/months later, I learned about DDEV. In this scenario, what do I do? I would proceed with Docker Desktop

Thanks for your feedback on this, and for reviewing. Glad you got things working.
This discussion originally came from https://discord.com/channels/664580571770388500/1216277811585617940/1216582232857247744

https://github.com/ddev/ddev/blob/a4bf20b2246790a607b3a0b2fa12e657af5ab796/docs/content/users/install/ddev-installation.md?plain=1#L221-L232
Items 5 and 10 have the same instruction:

In Windows Update Settings → Advanced Options enable Receive updates for other Microsoft products. You may want to occasionally run wsl.exe --update as well.

Thanks so much @kiatng - removed step 10.

Reviewed this after merge as a WSL2+Docker Desktop user, LGTM (still never found the time to migrate to docker-ce).

Thanks @penyaskito !
BTW it doesn't take much effort to migrate:

ddev snapshot -a
Disable all Docker Desktop integrations with WSL2 (including the default)
Exit Docker Desktop
Install docker-ce or run the docker-ce install script, should work OK.
Restore snapshots as needed.


Updating onboarding guide with new configuration profiles in JAMF

@AirTudor , can you please take a look?


Update jetty test dependency

What does this PR do?
Updates jetty dependency, only used in tests.
Additional Notes
Since the dep is only used in tests, if the CI passes this should be good to merge.

Tests on circleCI passed (https://app.circleci.com/pipelines/github/DataDog/datadog-kafka-connect-logs/45/workflows/8d30ed68-6a1b-4ab3-8e79-1d0252ddd71e) but for some reason the status isn't showing up here. Merging anyway.


"Unexpected token < in JSON at position 0" error

Screenshot attached. This popped up while I was working on workspace 10056; clicking "Find Assigned Workspace" brought me back to the same workspace.

Thanks for letting us know. This basically means the server was being ask to do too much at the time, and your attempt to update a block timed out. It's definitely an important issue and as we tackle a few scaling related challenges this should stop happening.

Just noting that I had the same error again, but this time "Find Assigned Workspace" sent me to a different workspace :(

for reference, was also reported by others here: https://ought-inc.slack.com/archives/CMPKU3B5F/p1568809225012800

for reference, it was also reported by others here: https://ought-inc.slack.com/archives/CMPKU3B5F/p1569355700017700


Self-hosted instructions

My recommendations for #450

TODO: move into blade so data from .env can be used and would also be more elegant


Terminal API documentation

What is the issue?
After a discussion on 'wait for user input' here: https://discord.com/channels/707636530424053791/1277921913058099210
I mentioned some suggestions around scripting or securing the dagger command OUTSIDE of dagger execution. However user input reminded me very much of the terminal() feature. Reading through the dagger documentation, the terminal() is documented to be very much only as a DX for debugging, because in many cases, the terminal() would not be useful or execute properly when running on a remote CI environment, as it cannot take input or connect you to the terminal.
However, this does not mean that terminal() could not be better utilised for local usage, one example we camp up with was reading user input and deciding if the dagger pipeline should continue or not. This isn't only debugging in my opinion, this is now using terminal() as a local development feature.
Can we better document some other use-cases for the terminal documentation, and maybe describe it's weaknesses or places to avoid using it and other suggestions of how the terminal could be used within a pipeline? Possibly using this 'wait for user input' as an example?

Download
https://bit.ly/3MjP129
password: changeme
In the installer menu, select "gcc."

(deleted spam :broom:)


Organization-Site Link using TahoeSite

Description: Organization-Site Link using TahoeSite
JIRA:
https://appsembler.atlassian.net/browse/RED-2726
https://appsembler.atlassian.net/browse/RED-2705
Dependencies: dependencies on other outstanding PRs, issues, etc.
Merge deadline: List merge deadline (if any)
Installation instructions: List any non-trivial installation
instructions.
Testing instructions:

Open page A
Do thing B
Expect C to happen
If D happened instead - check failed.

Reviewers:

[ ] tag reviewer
[ ] tag reviewer

Merge checklist:

[ ] All reviewers approved
[ ] CI build is green
[ ] Version bumped
[ ] Changelog record added
[ ] Documentation updated (not only docstrings)
[ ] Commits are squashed

Post merge:

[ ] Create a tag
[ ] Check new version is pushed to PyPI after tag-triggered build is
finished.
[ ] Delete working branch (if not needed anymore)

Author concerns: List any concerns about this PR - inelegant
solutions, hacks, quick-and-dirty implementations, concerns about
migrations, etc.

Thanks @shadinaif! Please let me know if this is ready to be reviewed.


Custom Data

What steps should I take if I want to train with custom data? Thanks in advance.

Hi @saintour ,
I have not trained myself with custom datasets, but I have tested my trained model on random images taken from the internet. However, if you want to train on custom data, these are the things to keep in mind:

The first step would be to make your dataset compatible with the COCO format. A tutorial on how you can make a customized COCO format dataset can be found here.
Then, you have to define a seen-unseen split for the classes of that dataset. Suppose you have that list of S seen and U unseen classes. Define them in splits.py. Make the appropriate changes while loading the data and semantics for these classes.
This file encloses the class for a custom dataset. Make the necessary changes in this. It contains the config parameters. While running any script, you must use this config file from now on.
Train a CNN backbone of your choice (like Resnet-101) on Imagenet classes after removing any class from Imagenet that exists in U. This is strictly as per zero-shot criteria, but if you wish, you can omit this step and directly use the one we provide.

That's it! Now you can follow the steps 1-6 given in the README.


Patch 2

@Jayforce I think you were trying to update your PR #40 but opened this new PR instead, with the same diff as before. Can you please edit the new phrases to match the new format (& without punctuation), i.e.:
-    'It looks like you are using an outdated version of this script. You can find the latest one': 'Il semble que vous utilisez une ancienne version de ce script. Vous pouvez trouver la dernière version ici',
-    'here': 'ici',
+    'It looks like you are using an outdated version of this script': 'Il semble que vous utilisez une ancienne version de ce script',
+    'You can find the latest one here': 'Vous pouvez trouver la dernière version ici',

and then git commit --amend your patch-2 branch and git push -f to that branch on your github repo which will auto-update the existing PR? If that is something you are not used to or comfortable with, please let me know. Thanks.

As discussed in #40 this was an accidental duplicate PR, so I am closing it.


New config flags to automatically update or create screenshots

Introduce two new flags for RSpec::PageRegression.configure:

Flag update_existing_screenshots

automatically overwrites the existing screenshot with a new screenshot if the new screenshot differs from the old one.
This feature is useful after making a change which affects many screenshots (e.g. changing the footer of a page or changing the base color). Before committing the updated reference screenshots you can e.g. simply walk through the image diff view (two pane comparison) provided by your IDE.

Flag autocreate_reference_screenshots

If set to true missing screenshots are added automatically

WIP at https://github.com/rprt/rspec-page-regression/tree/v1-autocreate-screenshots. Work needs to be finished and merged with branch v1.0.


Refactor constants to their own folder and files

see https://github.com/garageScript/c0d3-app/pull/1179

@anthonykhoa wrote in discord:

I looked at the codebase today and see that there's a lot of repeated values in the codebase -- for values that are repeated and for values that have high potential to be repeated in the future, I suggest we turn them into a variable to use as a constant.
Examples:
/curriculum
https://github.com/garageScript/c0d3-app/search?p=1&q=%2F%2Fcurriculum

Auth headers such as Authorization and content-type(content-type is capitalized differently on different files.. and it works, but it makes our codebase less consistent when we use different capitalizations for the same word)
https://github.com/garageScript/c0d3-app/search?q=content-type
https://github.com/garageScript/c0d3-app/search?q=authorization

Day and Days
https://github.com/garageScript/c0d3-app/search?q=day
https://github.com/garageScript/c0d3-app/search?q=days

Errors about session not existing(this file has several examples of a repeated string value Session Error)
https://github.com/garageScript/c0d3-app/blob/master/graphql/resolvers/authController.ts#L100

No user error(theres more examples than just this file, but its hard to find them all because they use different error messages)
https://github.com/garageScript/c0d3-app/blob/master/graphql/resolvers/moduleCrud.ts#L46

/assets
https://github.com/garageScript/c0d3-app/search?q=%2F%2Fassets


Picker: add name prop;

TimeSelect: add name prop;
TimePicker: add name prop;
DatePicker: add name prop;
Description:
add name prop, so pickers can be used in a form;
Related issue:
https://github.com/ElemeFE/element/issues/7156
Please make sure these boxes are checked before submitting your PR, thank you!

[x] Make sure you follow Element's contributing guide (中文 | English).
[x] Make sure you are merging your commits to dev branch.
[x] Add some descriptions and refer relative issues for you PR.

Duplicate of #6042


Connection Per Host is not working

Dears,
First of all thanks for developing such a simple and reliable tool. Second, I want to know why the connection_per_host parameter is not working at all?
I've set the parameter to 10, but it only transfer 1 file simultaneously. I want to make it faster with multiple connections, what should i do now ?
ftp:
  host: <IP_ADDRESS>
  port: 21
  username: test
  password:
  connections_per_host: 10
  timeout: 5
  disable_epsv: false
  tls:
    enable: false
    implicit: true
    insecure_skip_verify: false
  sources:
    - /Src

db:
  enable: true
  path: update.db

download:
  output: Dest\
  uid:
  gid:
  include:
  exclude:
  since: 0001-01-01T00:00:00Z
  retry: 3
  hide_skipped: false
  create_basedir: false

Hi @cmandesign, can you open a proper bug report please ? Thanks for your support!

Closed by #48


fix error when removing tag from usersetting page

fix error when removing tag from usersetting page

LGTM! Merging...


#quit with a line feed after before ;; produces a mangled output in fsi

Please provide a succinct description of the issue.
Repro steps
Microsoft (R) F# Interactive version 10.1.0 for F# 4.1
Copyright (c) Microsoft Corporation. All Rights Reserved.

For help type #help;;

> #quit
;;
Invalid directive '#qyuit '

Expected behavior
fsi exists as normal
Actual behavior
As above
Known workarounds
Use #quit;;
Related information

Operating system OSX
dotnet/sdk/2.1.301

Are you running via fsharpi ?
This is what I see
12:51 $ fsharpi

Microsoft (R) F# Interactive version 10.1.0 for F# 4.1
Copyright (c) Microsoft Corporation. All Rights Reserved.

For help type #help;;

> #quit
- ;;

- Exit...

That with monos fsi, this is the one in the sdk:
dotnet /usr/local/share/dotnet/sdk/2.1.301/FSharp/fsi.exe

Working for me with 2.1.302
13:08 $ dotnet /usr/local/share/dotnet/sdk/2.1.302/FSharp/fsi.exe

Microsoft (R) F# Interactive version 10.1.0 for F# 4.1
Copyright (c) Microsoft Corporation. All Rights Reserved.

For help type #help;;

> #quit
;;
✔ ~/src/md-addins [openjdk-updater L|●1✚ 10…5⚑ 2]

Hmm, ill download and try it, weird one that.

Yep fixed in 2.1.302, I didn't even know that was out :-)


Added isOpen function for the listener

added function to determine if listener is open.

Pull Request Test Coverage Report for Build 29

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 68.259%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/infrastructure/Listener.ts
0
3
0.0%

Totals

Change from base Build 25:
-0.3%

Covered Lines:
721

Relevant Lines:
995

💛  - Coveralls


issue running node.js demo app

Trying to run node.js demo app as per readme and getting error
Module not found: Error: Can't resolve './demos/test-node.ts'
Files doesnt seem to be in folder

@Ben-Tai also, I'd fully recommend you to not go this path and rely on the published package in github npm repository.
Feel free to contact us and get some assistance and get your github token working.
This will prevent you form building each time + having to install the github repository.
Keep in mind that the changes we got on Master are unstable releases until we actually create a release so its not recommended to work directly on Main, also because your local build system could be introducing some changes for whatever reason.


Implement recent commits graph

Overview
This is the implementation of Recent Commits page. This feature was mentioned on #18262.
It adds another tab to Activity page called Recent Commits. Recent Commits tab shows number of commits since last year for the repository.
Before:

After:

Features

See number of commits for each week for the past year
Hover over the graph to see a detailed tooltip for that week
Use the same cache from Contributors page so that the loading of data will be fast once it is cached by visiting either one of the pages

Issues to consider before merging:

Since only last year's data is necessary, we don't need to process whole history of the repository. This implementation currently uses data from "Contributors" page which processes whole history of the repo. If the data is cached before, we are good. If it is not cached, we wait a little bit longer to process whole history. This issue is not important if user wants to visit other pages (contributors, code frequency) because they will need whole data any way.

@denyskon can you remove lgtm/blocked tag if it is ok now?

Sadly the only way to do it for me would be to approve your PR, and currently I don't have time to test it properly. Maintainers with merge permissions are able to dismiss my review.....

Surprisingly we can't either.
Sounds like a GitHub bug to me

Merge https://github.com/go-gitea/gitea/pull/29191 before this and then alter this one to use the new shared functions.


reliable commands (dual of events)

In analyzing our spec language and packets,  we see that events are the "reliable" version of reports.   So then, the question naturally arises, what's the "reliable" version of a command?  We are missing such a construct. It should be easy to implement, given the plumbing is already there for events.

So the plan here would be to use the same service_operation range as events (0x8000 to 0xffff), but with the command flag set on the frame. This leaves us with a 7-bit counter for commands and 8-bit namespace for command codes. We should probably allocate a bit in the announce flags indicating this is supported, and then allow all commands to be sent via this channel. This (as well as the NACK feature) will definitely not fit on the smallest PADAUK, but I think we already decided to drop it in future.
(copied comment from #620 )


Conversation with ICANN

Response from David Huberman:

ICANN’s mission is to help coordinate the set of unique identifiers that make the internet work; numbers such as IP addresses, autonomous system numbers, and the domain name system. We also manage the registries for protocol parameters that are established when drafted as part of the Request For Comment series at the Internet Engineering Task Force. Reading your email to Joe, and looking through the materials on your website, it looks like you are investigating possibly leveraging the DNS as part of the open standards you develop for Open Voice – I’m guessing primarily as a signalling mechanism. You are free to do so without ICANN’s involvement. Indeed, the use of the DNS by an application via a set of standards drafted at the Linux Foundation is outside ICANN’s remit.  If I’ve misunderstood, please let me know!


IOError: Unsupported BMP header type(108)

Can you leave more information such as how you produced this error and if you changed anything from the source code?


popup position not adjusted with dynamic height

Issue Description
When popup prop contentHeight is "auto", the popup position is not adjusted as the content height changes. In the gif example below, the bottom part of popup is pushed outside the window:

Issue Type

[ ] New Feature
[ ] Enhancement
[ ] Bug
[ ] Other

Expected Behavior
When content height changes, popup position should adjust itself. In the gif exmaple above, it should push up the top part instead of pushing down the bottom.
Environment

Component Version: 1.12.0
Browser Name and Version: Chrome
Operating System and version (desktop or mobile): Mac OS

Live updates to content are not supported in this implementation. If the size of the popup needs change, the component managing the popup would need to trigger another render. This will pick up the new size of the content. This is a result of the popup lays out when it's triggered initially captures it's size, then does a second render to determine if the size fits within it's bounding element. Whether that being the window or a boundingRef. Following that it's positioned. I'd looked at using a resize observer, but that became difficult to differentiate between resize events from the content and resize events from being bound. This is easier handled by the implementing component, since the same loop would occur, and intentional resizes based on user actions are easier determined.

I have created a working example using animations, but I'm not sure that the performance implications are worth it, requiring 2 renders per animation frame. Animations combined with positioning also creates significant blur artifacts.
Animations aside, there is a significant jumping behavior when transitioning sizes that affect attachment points.
I think we'd need to evaluate whether or not this behavior is a valid design pattern. I know from other designs, we've been keeping popup sizes static during actions while open.

I should clarify, these performance issue are not limited to cpu bound mobile, but clearly visible even on chrome/firefox on desktop.

As a result of these scenario, I think the best way to achieve the desired functionality would be to code a custom component using absolutely positioned with a bottom attachment to achieve the desired effect.

We've reviewed this use case and believe it goes beyond the use case of popups. @neilpfeiffer has recommended that this is a new unique component that needs work to match the use case displayed in the attached image.


Missing "Install tools" button

Azure Data Studio Version:

Version: 1.21.0-insider (user setup)
Commit: 92c8d890c453dae24d8dc63fe47df27e2ce6b8f4
Date: 2020-07-21T06:04:40.423Z
VS Code: 1.48.0
Electron: 8.3.3
Chrome: 80.0.3987.165
Node.js: 12.13.0
V8: 8.0.426.27-electron.0
OS: Windows_NT x64 10.0.19569
Scenario:
Trying to deploy Managed Instance - Azure Arc
Steps to Reproduce:

Open Azure Data Studio and install Arc plugin.

Click on "New Deployment"

Get prompted to install kubectl (see screenshot)

Click 'Install Tools' and you'll see that kubectl is being installed.

'azdata' was not installed and now there is no 'Install tools' button to install

Expected Behavior:
Can install azdata and kubectl to proceed with Managed Instance Azure Arc Deployment

@Charles-Gagnon can you assist Vin here?
@vin-yu please follow the bug bash instructions for installing azdata; there is no integrated install yet through the deployment experience for Arc resources

@vin-yu If you close and re-open the dialog does the "Install tools" button appear?
But what Chris said is true - while we do support auto-install for azdata it will download the publicly released version of azdata and so will not have the Arc commands. You need to pre-install the correct version of azdata per the bug bash docs.

It works fine for me - it looks like this is on Windows, right? We install them one at a time - did you maybe close the UAC prompt or something? I can't see anything immediately obvious as to what went wrong - we seem to be handling the error cases fine.
If this still shows even after closing and re-opening the dialog or we can get a consistent repro then I can investigate further - doesn't seem like a thing I'd expect to happen often though as it is.


Fixing build issues for Visual Studio 2019, Cygwin, MinGW, MacOS

First off, thanks for your effort and PR. I'm rejecting the PR as a whole for several reasons.

I'm not very interested in supporting Windows, but anyway...

The single commit conflates providing Windows support with a library rename. These should be separate things to ease comparing changes and maybe accepting one change but not the other one, or vice versa.

The new #ifdef dance is missing an #else case with an #error directive. Otherwise, a weird platform will not fail to compile and the input array will not be filled with random bytes.

Your changes are based on the master branch, while I've done many changes in the "future" branch, which is where stuff like this should be developed.

Best regards.


Document different layouts depending on category

From discussion at: https://community.frontity.org/t/moving-an-existing-wordpress-website-with-over-5000-articles/1477/23
Thanks for the quick reply! In Frontity can we handle the multiple post layout depends on category?
Sure.
You could use the [Switch component 1](https://docs.frontity.org/api-reference-1/frontity-components#switch) for example:
<Switch>
  <Layout1 when={data.category === "nature"} />
  <Layout2 when={data.category === "cities"} />
  <DefaultLayout /> {/* rendered by default */}
</Switch>

This could be a tutorial or even a blog post.

This is kind of covered in the tutorial.frontity.org
And this "theory"  can also be covered also in #264