Set Wails Always on Top

before : wails popup behaves like normal window
after : wails popup will be always on top, user can minimize the window but it will not stack under if the user changes focus

Codecov Report
Patch and project coverage have no change.

Comparison is base (6e444bd) 39.39% compared to head (24c263b) 39.39%.
Report is 4 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #704   +/-   ##
=======================================
  Coverage   39.39%   39.39%
=======================================
  Files          20       20
  Lines        1480     1480
=======================================
  Hits          583      583
  Misses        831      831
  Partials       66       66

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

Since it's part of a TBD milestone we might not need to merge right now

@pivilartisant could you describe before/after behavior in the PR description?

Did you had the bug mentionned in the issue ?
Did you discussed with someone about using the AlwaysOnTop option as the solution ?
In my opinion, it's a bit overkill because, if I'm not wrong, that means that the window can't be "minimized" or behind another one, which can be annoying especially on small screens

Did you had the bug mentionned in the issue ? Did you discussed with someone about using the AlwaysOnTop option as the solution ?
In my opinion, it's a bit overkill because, if I'm not wrong, that means that the window can't be "minimized" or behind another one, which can be annoying especially on small screens

Window can be minimized, but indeed it will always be on top during any other interaction

Did you had the bug mentionned in the issue ? Did you discussed with someone about using the AlwaysOnTop option as the solution ?
In my opinion, it's a bit overkill because, if I'm not wrong, that means that the window can't be "minimized" or behind another one, which can be annoying especially on small screens

Do I didn't have the bug mentionned in the issue

Window can be minimized, but indeed it will always be on top during any other interaction

Did you had the bug mentionned in the issue ? Did you discussed with someone about using the AlwaysOnTop option as the solution ?
In my opinion, it's a bit overkill because, if I'm not wrong, that means that the window can't be "minimized" or behind another one, which can be annoying especially on small screens

No I did not have the bug mentioned in the issue unfortunately
Window can be minimized, but indeed it will always be on top during any other interaction

We chose to not go forward with this pr as it did not improve UX


Downloads page additional modifications

Addresses these comments: https://github.com/opentargets/platform-app/pull/348#issuecomment-827618253

@mirandaio the changes look good. The Google Cloud command works as it should.
The FTP files are not yet ready but I will check as soon as they are just in case.
It would be good if we can merge this PR


Undefined symbols for architecture arm64 > Symbol: OBJC_CLASS$_HKCDADocumentSample > Referenced from: _OBJC$CATEGORY_HKCDADocumentSample$_serialization in HealthData.o

Trying to build an iOS app which is originally (built in Unity) through gitLab using CI.
Added new package for HealthKit  and endedup with error "Undefined symbols for architecture arm64 > Symbol: OBJC_CLASS$_HKCDADocumentSample > Referenced from: _OBJC$CATEGORY_HKCDADocumentSample$_serialization in HealthData.o" while building the app. tried deleting cache pods and DerivedData .But no luck.
Thanks in Advance

Unity is unrelated to j2objc, and those symbols aren't generated by j2objc.

Unity is unrelated to j2objc, and those symbols aren't generated by j2objc.


Upgrade nixpkgs

It seems like the Nix version used when running the container is pretty old:

We need to upgrade nixpkgs to the latest possible version to keep up to date with the latest nix versions.

We ran into these errors in Universe after merging #1220
https://gitlab.com/fluidattacks/universe/-/jobs/5765178089
We decided to revert the changes temporarily

@dsalaza4 @rohaquinlop
We ran into these errors in Universe after merging #1220
/tmp/nix-build-integrates-front-build.drv-0
/nix/store/600g0if5farpw99rb6zbjgpswv4bb7j6-integrates-front-build/output/app
error: home directory '/homeless-shelter' exists; please remove it to assure purity of builds without sandboxing

We decided to revert the changes temporarily, which fixed the pipelines

I think this is more related to /integrates/front/build needing to use the HOME directory in newer npm versions

makeNodeJsLock isn't working


exception handling the Autonomio way

This needs to be done properly.

We need to have only specific exception handling (not general as we now have many)
The user by default only sees our custom error message that says what to do, not what the problem is
Trace for error goes in to a log
the log can be accessed through errors() command where you can do for example errors('last')
There needs to be a switch where the user can turn on debugging, where trace is always shown with the bespoke error message

For now, as errors are are very few given the level of automated transformation etc. we provide, let's not do this and just have the standard error messages come out when relevant. Closing.


Tracing.proto and Common.proto pollute global namepace

Tracing.proto and Common.proto introduce lots of symbol into global namepace, like Command, SpanType.. which conflict with my other header files.
Consider adding package <NAMESPACE>; to proto files, however, this will change method name from /TraceSegmentReportService/collect to /<NAMESPACE>.TraceSegmentReportService/collect which is not recognized by skywalking server. Maybe fix it manually?

@wu-sheng @lizan I think that this feature should be introduced before 0.1.0 because it will cause same problem when integrating Envoy.

This is more a issue of data-collect-protocol issue. @wu-sheng ?

We are diacussing there, how it works in today's envoy?

@liberize As for discussion with SkyWalking maintainers, we can't introduce package name because it will cause serious breaking change without C++ dedicated namespacing option like java. This problem is not caused on Envoy which is large C++ application in the world integrating with SkyWalking data collect protocol.
https://github.com/envoyproxy/envoy/blob/master/source/extensions/tracers/skywalking/skywalking_types.h#L11
So personally I recommend you to use namespace in your C++ application.

We also want to check, cpp tracer is not using your grpc client. Is there still a conflict?

It will be used in general, but envoy won't.

Even in general, tracing backend is different from the user codes. So, I am expecting no conflict.

This problem will caused when compile. It will conflict, such as between Command class on user's application it includes cpp2sky and Command class on data collect protocol definition. So It might have no relation.

@wu-sheng it is the protobuf generated class, so regardless of the gRPC client, it pollutes the global namespace.

so it might be inevitable if you don't use cpp2sky grpc stub.

I was expecting no one is using like this. Unless like envoy, it could take care of this smoothly already.
Another block is adding this breaks all changes in the main upstream, we can't make it easily. We have to wait next time the whole protocol upgrade, which are years away most likely.

it is the protobuf generated class, so regardless of the gRPC client, it pollutes the global namespace.

@lizan I don't know cpp, it there a chance to separate the compiling? In all language before, it has internal package system, not using this package option, like java package option. But once this is added, grpc service name changed.
I hope we could introduce how envoy resolve this, and don't have to change it. Such as  @Shikugawa idea, use our reporter only.

No the unfortunate part is protobuf doesn't allow overriding C++ namespace, which worth a separate issue in protobuf. Aside from that, I added recommendation to https://github.com/apache/skywalking/issues/6019#issuecomment-747194576.
Re: how it solved in Envoy, no it isn't, at Envoy we clearly namespaced everything else (including those C libraries which have clear prefix in the symbol names they exported), but it shouldn't be the excuse that SkyWalking proto can pollute the namespace.


Unable to retrieve new kc_access token

Title
Unable to retrieve new kc_access token
Summary
I am using louketo-proxy ad middleware between DEX and kubernetes dashboard.
When I open the kubernetes dashboard trough louketo-proxy I am redirected to DEX.
I fill in the login form with my credentials - and I am redirected to kubernetes-dashboard.
In cookies I don't see kc-state (there is kc-access).
When the token expires, it is not refreshed by refresh token.
I am redirected to login page.
In louketo-proxy logs i see following lines:
1.5937260426119459e+09	info	accces token for user has expired, attemping to refresh the token	{"client_ip": "<IP_ADDRESS>:40068", "email": "**********"}
1.5937260426120653e+09	error	unable to find a refresh token for user	{"client_ip": "<IP_ADDRESS>:40068", "email": "**********", "error": "authentication session not found"}

Environment

Server: DEX
Louketo: {"prog": "keycloak-gatekeeper", "author": "Keycloak", "version": "10.0.0 (git+sha: f24a5fb, built: 29-04-2020)"}
Louketo-proxy config:

image: quay.io/keycloak/keycloak-gatekeeper:10.0.0
args:
        - --listen=<IP_ADDRESS>:3000
        - --discovery-url=https://k8s-dex.my-k8s.com/
        - --client-id=$(CLIENT_ID)
        - --client-secret=$(CLIENT_SECRET)
        - --upstream-url=https://k8s-dashboard-direct-int.my-k8s.com/
        - --skip-upstream-tls-verify=false
        - --skip-openid-provider-tls-verify=false
        - --enable-default-deny=true
        - --enable-logging=true
        - --enable-refresh-tokens=true
        - --enable-session-cookies=true
        - --secure-cookie=false
        - --enable-encrypted-token
        - --encryption-key=$(ENCRYPTION_KEY)
        - --enable-metrics
        - --scopes=groups

Expected Result
The access token is rotated using the refresh token
Actual Results
The access token is not rotated and I am redirected to DEX login page.

Sounds like #571 .
I was facing the same issue. A update to louketo-proxy 1.0.0 seems to solve the problem


Script App\Composer::setup handling the post-create-project-cmd event terminated with an exception

I have run the code "composer create-project -s dev aimeos/aimeos myshoppingsite". Following there is an error that I can not find a solution to to it.

Please suggest how can I create my project "myshoppingsite".

Maybe it helps if you set the path to your php file in your PATH before executing composer:
export PATH=$PATH:/c/xampp/htdocs # if the php command is stored there

or
SET PATH=C:\xampp\htdocs;%PATH%

Dear Aimeos/Aimeos team,
unfortunately the error is the same. I have no clue what might be causing
the error.
[image: image.png]
On Sat, Dec 19, 2020 at 2:13 AM Aimeos<EMAIL_ADDRESS>wrote:

Maybe it helps if you set the path to your php file in your PATH before
executing composer:
export PATH=$PATH:/c/xampp/htdocs # if the php command is stored there
or
SET PATH=C:\xampp\htdocs;%PATH%
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aimeos/aimeos/issues/39#issuecomment-748452940, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AK3JK325M7X6IZVTJV2UXI3SVR4EDANCNFSM4VBVHF7A
.

--
Respectfully yours,
Abdulaziz Kholmatov.

My credentials: Windows 10 home, PHP 7.4, Server version: 10.4.16-MariaDB
On Sat, Dec 19, 2020 at 5:54 AM Abu Kholmatov<EMAIL_ADDRESS>wrote:

Dear Aimeos/Aimeos team,
unfortunately the error is the same. I have no clue what might be causing
the error.
[image: image.png]
On Sat, Dec 19, 2020 at 2:13 AM Aimeos<EMAIL_ADDRESS>wrote:

Maybe it helps if you set the path to your php file in your PATH before
executing composer:
export PATH=$PATH:/c/xampp/htdocs # if the php command is stored there
or
SET PATH=C:\xampp\htdocs;%PATH%
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aimeos/aimeos/issues/39#issuecomment-748452940, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AK3JK325M7X6IZVTJV2UXI3SVR4EDANCNFSM4VBVHF7A
.

--
Respectfully yours,
Abdulaziz Kholmatov.

--
Respectfully yours,
Abdulaziz Kholmatov.

Environment variables are added as well.
[image: image.png]
On Sat, Dec 19, 2020 at 6:08 AM Abu Kholmatov<EMAIL_ADDRESS>wrote:

My credentials: Windows 10 home, PHP 7.4, Server version: 10.4.16-MariaDB
On Sat, Dec 19, 2020 at 5:54 AM Abu Kholmatov<EMAIL_ADDRESS>wrote:

Dear Aimeos/Aimeos team,
unfortunately the error is the same. I have no clue what might be causing
the error.
[image: image.png]
On Sat, Dec 19, 2020 at 2:13 AM Aimeos<EMAIL_ADDRESS>wrote:

Maybe it helps if you set the path to your php file in your PATH before
executing composer:
export PATH=$PATH:/c/xampp/htdocs # if the php command is stored there
or
SET PATH=C:\xampp\htdocs;%PATH%
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aimeos/aimeos/issues/39#issuecomment-748452940, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AK3JK325M7X6IZVTJV2UXI3SVR4EDANCNFSM4VBVHF7A
.

--
Respectfully yours,
Abdulaziz Kholmatov.

--
Respectfully yours,
Abdulaziz Kholmatov.

--
Respectfully yours,
Abdulaziz Kholmatov.

Where is your PHP binary located?

php.ini file is located in C drive. Under the fine PHP.7.4. Is that the
information you are asking?[image: Untitled.png]
On Sat, Dec 19, 2020 at 6:29 AM Aimeos<EMAIL_ADDRESS>wrote:

Where is your PHP binary located?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aimeos/aimeos/issues/39#issuecomment-748481993, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AK3JK36XZOFBJUMBQ6RUFDTSVS2EHANCNFSM4VBVHF7A
.

--
Respectfully yours,
Abdulaziz Kholmatov.

No, what is the full path to your php.exe, e.g. C:\xampp\htdocs\php.exe?

No it is rather C:\xampp\php.php or C:\php 7.4\php.exe
On Sat, Dec 19, 2020 at 7:58 AM Aimeos<EMAIL_ADDRESS>wrote:

No, what is the full path to your php.exe, e.g. C:\xampp\htdocs\php.exe?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aimeos/aimeos/issues/39#issuecomment-748491761, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AK3JK326LO2TGGOWKVPFA5TSVTES3ANCNFSM4VBVHF7A
.

--
Respectfully yours,
Abdulaziz Kholmatov.

But I have tried to create the Aimeos project using "composer
create-project aimeos/aimeos myshopTest" even inside the   C:\php 7.4
and   C:\xampp\htdocs\ . But the same error :(
On Sat, Dec 19, 2020 at 8:42 AM Abu Kholmatov<EMAIL_ADDRESS>wrote:

No it is rather C:\xampp\php.php or C:\php 7.4\php.exe
On Sat, Dec 19, 2020 at 7:58 AM Aimeos<EMAIL_ADDRESS>wrote:

No, what is the full path to your php.exe, e.g. C:\xampp\htdocs\php.exe?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aimeos/aimeos/issues/39#issuecomment-748491761, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AK3JK326LO2TGGOWKVPFA5TSVTES3ANCNFSM4VBVHF7A
.

--
Respectfully yours,
Abdulaziz Kholmatov.

--
Respectfully yours,
Abdulaziz Kholmatov.

The space in your path is the problem. Can you try:
composer create-project aimeos/aimeos:2020.10.x-dev myshop

Dear Aimeos/Aimeos team!
Awesome! It worked. Thank you for your time and effort! :)
On Sat, Dec 19, 2020 at 8:53 AM Aimeos<EMAIL_ADDRESS>wrote:

The space in your path is the problem. Can you try:
composer create-project aimeos/aimeos:2020.10.x-dev myshop
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aimeos/aimeos/issues/39#issuecomment-748497713, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AK3JK36VABUHQMUYDXVC6VTSVTLAHANCNFSM4VBVHF7A
.

--
Respectfully yours,
Abdulaziz Kholmatov.


Fix batch delete for string document ids. (#330)

Pull Request
What does this PR do?
Fixes #330
PR checklist
Please check if your PR fulfills the following requirements:

[X] Does this PR fix an existing issue?
[X] Have you read the contributing guidelines?
[X] Have you made sure that the title is accurate and descriptive of the changes?

Oh very good point!
Will update that tomorrow by adding a bridging method along with a deprecation notice.

Hi @brunoocasali!
I readded the [Int] methods along with unit tests. Since using Int ids is "wrong" I've also added a deprecation notice so that people who use the old method will get notified to change to string values.

bors merge

bors merge

I've opened a PR to fix this bors issue 😇
https://github.com/meilisearch/meilisearch-swift/pull/333

bors merge


more stable temperature control with PID loop

for high-precision heating, something like https://gist.github.com/DzikuVx/f8b146747c029947a996b9a3b070d5e7 could be used to accurately control the CPU temperature :-)

Today I Learned what's a PID controller! This looks amazing.
Gonna have to try it over the weekend.
I've also noticed that sometimes the cpu eating processes finish before the carrots are thawed. When that happens, we aren't clearing the children array, which means we can't start new processes. The computer cools down.

Just put your macbook into a pot with water and use it for sous vide meat cooking :-)


VWAP indicator comparison to TA-Lib

I am not sure if this is simply my mistake or a bug (I guess someone would have seen it by now), but I was testing the VWAP indicator and I got something like the following

the red line (this library) is the result of:
data["vwap"] = vwap(
        data["high"],
        data["low"],
        data["close"],
        data["volume"]
    )

The golden line (TA-Lib) is the result of:
data["vwap2"] = volume_weighted_average_price(
        high=data["high"],
        low=data["low"],
        volume=data["volume"],
        close=data["close"],
        window=20
    )

Using the following data:
"ARKK", start_date="04/03/21", end_date="04/10/21", interval="1m" from index 0 to 385
What might be the reason for this?

Hello @esmeraldaliaj,
Thanks for providing some details but unfortunately it is not enough for me to reproduce and compare.
Also TA Lib (python wrapper) does not have volume_weighted_average_price method nor does it exist in the TA Lib C Library which the Python Wrapper is based on. Is volume_weighted_average_price a custom function of yours?
Additionally, VWAP is not a rolling average and has no window parameter. So that is one reason why there is a major difference in your chart.
Below is documentation of Pandas TA's VWAP. It is anchored by default to "D" (or "Daily") which was requested so it matches TradingView's Anchored VWAP. Since you are using "minute" data, you will need to change the anchor with the available types listed in the Pandas Library as noted in the vwap documentation below.

In summary, please provide more code as well as share your "ARKK" 1min data with me so I can be more helpful in clarifying the discrepancy.
Kind Regards,
KJ

@twopirllc you are absolutely correct, I meant to say I was comparing it to the function found here https://github.com/bukosabino/ta [from ta.volume import volume_weighted_average_price].
With this data
arkk.csv
This is nothing urgent by the way, just making sure that everything works fine. Otherwise great library.

Hello @esmeraldaliaj,
Apologies for taking so long to answering this.
The differences lie in that Bukosabino's TA uses a rolling sum for it's calculation while Trading View and Pandas TA utilize a cumsum. Bukosabino's is a custom version and not in line with the definition of VWAP and that is ok if it suits your needs and helps you resolve your analysis.

Trading View's and Pandas TA's

I have included some code that compares Pandas TA's VWAP Daily and Hourly and VWMA(20) with Bukosabino's TA VWAP(20).
import pandas as pd
import pandas_ta as ta
import ta as bta

def pdata(path="../arkk.csv", length=20, cols=["date", "open", "high", "low", "volume"]):
    df = pd.read_csv(path)
    df.drop(columns=["ticker"], errors="ignore", inplace=True)
    df.set_index(pd.DatetimeIndex(df["date"]), inplace=True)

    # Pandas TA
    df.ta.vwap(anchor="D", append=True) # Daily: red
    df.ta.vwap(anchor="H", append=True) # Hourly: blue

    # Pandas TA VWMA similar to Bukosabino TA VWAP: green
    df.ta.vwma(length=20, append=True)

    # Bukosabino TA: orange
    df[f"BTA_VWAP_{length}"] = bta.volume.volume_weighted_average_price(df.high, df.low, df.close, df.volume, length)

    return df.drop(cols, axis=1)

df = pdata(length=20)

title = f"ARKK [{df.index[0]} to {df.index[385]}]"
colors = ["black", "red", "blue", "green", "orange"]
df.iloc[:385].plot(figsize=(16,8), title=title, color=colors, grid=True)

Hope this helps clarify the differences, but use what is relevant and useful for your use case.
Kind regards,
KJ

Hello @esmeraldaliaj,
I assume by no response that the solution provided was sufficient. Thus I will be closing this issue in a few days.
Kind Regards,
KJ

This looks fine to me. VWAP is anchored. Human traders that study vwap will usually use multiple anchor points (pivot points, 1 week, 1 month, daily or session)


Visual artifacts in nav tree with recent Doxygen versions

I gave Doxygen 1.10.0 a try. There are some visual artifacts in the nav tree:

Looks like the doxygen-awesome-css website is having some similar issues:

Cheers!

I think I have the same issue

Doxygen 1.10.0
My Doxyfile

Same problem on Doxygen 1.10.0.

Thx for reporting! This should be fixed in https://github.com/jothepro/doxygen-awesome-css/commit/c4a2f644a05af4242c6b4c171601a11daa3850d3.


patch: Add USB page to navigation

Signed-off-by: Vipul Gupta (@vipulgupta2048) <EMAIL_ADDRESS>

Please make sure to read the CONTRIBUTING document before opening the PR for relevant information on contributing to the documentation. Thanks!

@balena-ci I self-certify!

@balena-ci rebase

@balena-ci rebase


Api Cleanup for language services

Created internal + public versions of

BlockKind
SpanKind
AcceptedCharacters

That way these types are only exposed through the VS apis and not
through the runtime API surface.
Also deleted RazorEditorParser. Yep. It's going to take significant work
to just port it to the language services assembly. Let's reevaluate this
when we get closer to the next foundational update.

Addresses #1304 and part of #1259

FYI Only


Suggest details sash can be hovered even when there are no suggest details

Just typing in a plaintext file and moving the mouse over the empty area makes the sash react to the mouse over / become active:

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Add support for Redis Cluster

Description
Add support for redis cluster by swapping out connection manager with connection cluster.
Type of change
Please delete options that aren't relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist

[x] Updated documentation if needed
[x] Tests added/amended
[x] bazel test //...  passes locally
[x] PR is contained in a single commit, using git amend see some docs

This change is

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@blakehatch can we close this if it is stale?

@aaronmondal can we automatically close stale PRs please with automation? I don't want a ton lingering so I can work to get whatever is broken over the finish line.


Tip: Bootstrap Template Another Way (BTAW)

Bootstrap Template Another Way
This setup is based on Modern Business template from Start Bootstrap using the following:
Grunt
Bower components
Assemble
handlebars
less
concat
uglify
fontawesome
With this configuration it provides a scaffold structure that can be simply combined to form a single web-page or a multi-page website.
https://github.com/pacav69/btaw/

This issue tracker is for reporting bugs or making feature requests.
Please post articles/announcements to other, more relevant community forums instead of here. Thanks!


protoc-gen-go: Implement UnimplementedServer with value instead of pointer

Is your feature request related to a problem? Please describe.
The UnimplementedServer generated by protoc-gen-go in 1.3.2 implements the server methods with a pointer to an empty struct.  This is pointless since the pointer isn't used for anything and forces an embedding struct to embed a pointer to nothing.
Describe the solution you'd like
Make UnimplementedServer implement the server methods with an empty struct value rather than pointer.  That way, users do not have to embed a useless pointer.
This is backward compatible because the method set of foo embedding *UnimplementedServer includes the method set of foo embedding UnimplementedServer.

All pointers to an empty struct are allowed to be identical, so the pointer is already no worse than a placeholder scalar (heck, the pointer can even be nil, as long as the function does not weirdly deref the pointer to an empty struct.)
So, in the end, the only issue at hand for switching from pointer to value, is… the size of one pointer in your service.  https://play.golang.org/p/vpqE3mdXiIL
I am not saying that such a change would not save space, but in context of one process memory space, that space savings ends up being pretty negligible. 🤷♀

\cc @dfawley who reviewed #785

PS: due to the way pointer receiver methods work, you should be able to use a non-pointer embedded value even if the code is generating pointer receiver methods. As per example playground above. So, does anything even need to change?

This is correct.  We've implemented the use of UnimplementedXxxServer in grpc-go, and we've done so by value and not pointer.

PPS: Note that the opposite does not apply. That is, if you have a non-pointer receiver, you could get a nil pointer deref when someone uses the receiver method on an embedded pointer type.

Also correct.  So if a user has embedded *pb.UnimplementedXxxServer, such a change would break them.  While I agree it probably would have been better to pass by value, I'm not sure we can safely change it now, and the value of such a change would be questionable.

PPS: Note that the opposite does not apply. That is, if you have a non-pointer receiver, you could get a nil pointer deref when someone uses the receiver method on an embedded pointer type.

I overlooked this.

PS: due to the way pointer receiver methods work, you should be able to use a non-pointer embedded value even if the code is generating pointer receiver methods. As per example playground above. So, does anything even need to change?

I also overlooked this, because you need to explicitly pass the address to satisfy interfaces: https://play.golang.org/p/GobufNmGHU0
All of this still prevents you from passing an empty struct implementation of a gRPC server everywhere, since you'll end up being forced to pass a pointer around at some point.
However, since this isn't a backward compatible change as I initially thought, it's probably not worth it.

All of this still prevents you from passing an empty struct implementation of a gRPC server everywhere, since you'll end up being forced to pass a pointer around at some point.

I don't understand this comment.  The only intended use of this type is to embed it in your service implementation to prevent newly-added methods from breaking your build:
type myFooServer {
  pb.UnimplementedFooServer
  ...
}

Otherwise you should ignore Unimplemented services completely.

You can't embed the UnimplementedServer as a value in an empty struct type Foo, and then pass an empty struct Foo to satisfy the interface of the Server.  You have to pass a pointer to an empty struct Foo to satisfy the Server interface.
I believe that with the way interface values are currently implemented the extra pointer value doesn't really matter, but conceptually it sucks that you can't embed UnimplementedServer and still implement the server interface by value of the embedee struct.

Everyone is so busy trying to microoptimize away pointers to a struct.

GC is expensive.  The "small" design choice of using pointers for all scalar values in protobuf 2 structs in Go results in a couple magnitudes worse performance (although I haven't saved the actual numbers to report).

That’s not a microoptimization, and that was not the only reason why proto3 eschewed pointers to scalars. It also made assigning constants into those values rather painful. Unfortunately, some other APIs (cough S3 cough) copied the idea at the time, and still remain annoying to use.
What I mean about microoptimizations is trying to get rid of the pointer to an empty struct Server. The use of pointers there is minimal, and puts zero load on the GC.


Downgrade cypress

What:
Fix failing Cypress tests

Why:
Latest version of Cypress introduced issues when calling .blur() on elements inside iframes.

How:
Downgrade and lock the Cypress version to 3.0.1. We could have used the force option as well but I felt like downgrading was the better option for now.
I've created an issue for this on their repo https://github.com/cypress-io/cypress/issues/2124
When this gets fixed or a good workaround is available we should unlock the version number again.

Checklist:

[x] Documentation N/A
[x] Tests
[x] Ready to be merged

[x] Added myself to contributors table

Hey! Downgrading instead of changing our tests is for sure a better option, thanks for contributing this 😄 👍
I think we should make a follow up issue about this before merging though, to make it easier to track when we can unlock the version again 🙂

I'll go ahead and merge this now. If someone wants to make a follow up that's fine 👍 thanks!

:tada: This PR is included in version 2.0.11 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Added temp docs banner announcement to invite Education Survey responses

Actually gotta check one thing....sry!

Thank you so much, @emdecr!


Update README.md

Correcting the command for 'Scanning projects locally (Docker container)' section
To scan with default settings

Thank you so much!


[Feat]: Add Back-to-top button

What feature?
I want to add back-to-top button on the result page, as user scrolls down then it gets annoying to scroll to top again.
Add screenshots
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@Yash636261 If user click Next on the pagination we are scroll them to the top.

@shyamtawli Please assign me this issue

Closing.


feat: anthropic prompt caching beta

Codecov Report
Attention: Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Project coverage is 79.26%. Comparing base (b736a4b) to head (7c64c81).

:white_check_mark: All tests successful. No failed tests found.

Files
Patch %
Lines

gptme/llm.py
0.00%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #94      +/-   ##
==========================================
- Coverage   80.11%   79.26%   -0.85%
==========================================
  Files          40       39       -1
  Lines        2580     2489      -91
==========================================
- Hits         2067     1973      -94
- Misses        513      516       +3

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


Feature: symbol key validation

Symbol runtype now has a call signature <K extends string>(key: K): SymbolFor<K>. You can use just const S = Symbol for non-keyed symbols, and const T = Symbol("T") for keyed symbols.

Coverage decreased (-0.6%) to 98.565% when pulling b0a6a1a566ecdd63ef8d6fb87caba45ed0df3908 on yuhr:feat/symbol-key-validation into fe12a4780c05207075dce9a473b73d1cbc449a45 on pelotom:master.


TGA import/loading does not support 16-bits-per-pixel files

Godot version
all
System information
all
Issue description
Currently, the TGA loading code in Godot fails on 16-bits TGA files. Looking at the code it seems this has simply not been implemented.
Would a PR to add this be accepted?
Steps to reproduce
Try to import a TGA file that uses 16-bits per pixels (RGB565 or RGBA5551).
Minimal reproduction project
No response

Would a PR to add this be accepted?

Sure :slightly_smiling_face:

Actually, looking further into the TGA reading code it also seems to assume that the alpha channel stored is always valid, while there is a bit in the header to indicate if this is the case (if the alpha isn't valid, it is simply ignored and replaced with 0xff).
Should I make a separate PR for that or would an "improve TGA file support" PR fixing both issues be ok?


Errand tracker UI [95% done]

Description

Implemented rating Modal for my Map Tracker!
Pretty much got the logic and functionality down, just styling now and making UI more palatable. Wanted to get these changes merged asap into dev for you guys to test out.

How to test

The rating button in the tracker will be grayed out until ETA hits 0
Both modal buttons star and frown face should close the modal upon interacting with them.
After rating the user once, the rating button will be grayed out again.

Notes & Issues
All that's left is styling a bit more, but pretty much done.

Gave up on the progress bar, but left it there for appearance lol
** Rating gets reset if I leave my component, and you are able to rate the runner again. Need to look into Async/local storage?

Screenies

Published to https://exp.host/@heartbleed/whimsical?release-channel=pr64


Editor menu dropdowns are missing a11y role attributes

When using the editor and clicking on a editor button "More formatting" a popover appears that correctly has a role="menu" attribute specified. The child items are missing their own role="menuitem" attributes.
To Reproduce
Steps to reproduce the behavior:

Go to Ask a question page
Expand the (...) dropdown "More formatting"
Inspect attributes of the button element

Expected behavior
The popover should follow ARIA menu role
Screenshots

Desktop (please complete the following information):

OS: Windows
Browser: Edge
Version 113

The best starting place for a possible fix if I were to try and tackle this: https://github.com/StackExchange/Stacks-Editor/blob/2124fd6ec1fb0b6569a4f9d2fea8686984120292/src/shared/menu/helpers.ts#L192-L203

This was resolved in https://github.com/StackExchange/Stacks-Editor/commit/f00095e87ab39ef2b05100e98c9cb6305b9ca182


Use same endpoint for download results as the results list view

BACKEND AND FRONTEND PULL REQUEST
Related Issue

Resolves #7948

Changes Proposed

Updates CSV results downloader to use new GetResultsForDownload query which hits the same endpoint at the main results list view. This makes the results download consistent with the results list by returning the data based on Results rather than TestEvents as the CSV download was previously doing.

Additional Information

This change should be okay with any concerns about GraphQL frontend backwards compatibility because it only adds a new query and the AOE survey data is ignored for the regular results list page.
Because we are no longer handling the download through TestEvents, but rather through Results directly, this simplifies the logic for the CSV data parsing so we don't have to worry about filtering out any feature disabled diseases from TestEvent data.

Testing

Deployed to dev6
Use various combinations of filters and make sure the download matches what is shown on the results list page

Ready for review but noting that the Chromatic CI checks apparently detected some difference with dropdown icons where it is saying that main has these strange overlapping icons while this PR doesn't. Maybe one of the previous Chromatic builds had some visual bug?
Main (seems like some visual bug)

PR branch (seems normal as expected)

FYI investigating an error Elisa found that so far only appears on dev6 instead of locally

Tested a variety of different filter configs in dev6 and everything LGTM. All of the new tests are also passing for me locally. Thanks for your great work on this Mike! I agree with you and Merethe that it makes much more sense to be querying these results from the backend.
One thing of note that is outside the scope of this ticket:
I find it odd that we do include the symptoms and their onset date from the AOE survey as their own respective cols in the CSV but we don't include the pregnancy survey question as a col. Does anyone know if this is a decision that was explicitly made as an implementation decision? If not, should we consider adding pregnancy and the other missing AOE Q's to the results download? CC @DanielSass @jayna-SkylightDigital

@mpbrown Something I'm noticing about these changes is that in the downloaded file if there is a date value in the "Symptom onset" column, it is showing up as "Invalid date".
I noticed in other envs we are able to display the correct date if there is a date in that field. Let me know if you can repro.

@mpbrown Something I'm noticing about these changes is that in the downloaded file if there is a date value in the "Symptom onset" column, it is showing up as "Invalid date".
I noticed in other envs we are able to display the correct date if there is a date in that field. Let me know if you can repro.

Good catch! I'll look into how it's parsing the date

Setting this back to Draft while I fix up the tests in TestResultTest

@bobbywells52 @emyl3 ready for re-review!


Org matching improvements

to build the lookup tables by reverse date order to prefer newer org names over old
don't look up name from participating org ref if that ref is in the block list
look up participating org ref from name if there is one and the ref is in the blocklist
obtain refs for all versions of name (eg. EN, FR) and pick first ref for which a preferred name can be obtained
maintain order of names from org.name.narrative

Pull Request Test Coverage Report for Build 947991001

42 of 43 (97.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 96.23%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

iati/lookups.py
42
43
97.67%

Totals

Change from base Build 940854549:
-0.01%

Covered Lines:
485

Relevant Lines:
504

💛  - Coveralls

Pull Request Test Coverage Report for Build 947991001
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

42 of 43 (97.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 96.23%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

iati/lookups.py
42
43
97.67%

Totals

Change from base Build 940854549:
-0.01%

Covered Lines:
485

Relevant Lines:
504

💛  - Coveralls


Fixed codecov file paths

no issue

WIP

Codecov Report

Merging #35 (e57ae7b) into main (fd42be7) will increase coverage by 82.29%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           main      #35       +/-   ##
=========================================
+ Coverage      0   82.29%   +82.29%
=========================================
  Files         0       27       +27
  Lines         0     2056     +2056
  Branches      0      259      +259
=========================================
+ Hits          0     1692     +1692
- Misses        0      360      +360
- Partials      0        4        +4

Impacted Files
Coverage Δ

packages/elasticsearch/lib/elasticsearch.js
92.85% <0.00%> (ø)

packages/request/lib/request.js
100.00% <0.00%> (ø)

packages/root-utils/lib/root-utils.js
88.46% <0.00%> (ø)

...s/bookshelf-pagination/lib/bookshelf-pagination.js
91.00% <0.00%> (ø)

packages/root-utils/index.js
100.00% <0.00%> (ø)

packages/validator/lib/validator.js
91.66% <0.00%> (ø)

packages/version/lib/version.js
100.00% <0.00%> (ø)

packages/pretty-stream/lib/pretty-stream.js
90.47% <0.00%> (ø)

packages/tpl/index.js
100.00% <0.00%> (ø)

packages/validator/lib/validate.js
45.00% <0.00%> (ø)

... and 17 more

Continue to review full report at Codecov.

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


Script file support

I'd like to be able to pass in a script file to evcxr for execution; something like
evcxr script.rs

(though maybe script files should have a different extension)
Such functionality should also optionally allow running commands interactively after the script executes. This would allow the use of script files as a sort of prelude for setting up the interactive environment. Currently, the only way I know of to set up the interactive environment in a predefined way is through copy-pasting commands.

Note, you can currently get some of this functionality via a ~/.config/evcxr/prelude.rs or equivalent on other platforms. It'd probably be pretty simple to allow arbitrary files to be loaded in a similar way. I'll try to add support for that at some point, but if you'd like it sooner, I'd happily review a PR.

+1 for me; I would like to execute a rust script from a Jupyter cell. It's not a good look to paste hundreds of lines inside a notebook, it would be much better to "export" it into a script, execute the script and then carry on with the post-processing in the notebook.


Optimize binary get_number implementation by reading multiple bytes at once

This PR improves performance for get_number implementation by reading multiple bytes at once, which saves calling overhead especially when interacting with file I/O. It adds get_elements to input adapters and allow them to select more efficient underlying calls to read multiple bytes if available
Performance for reading msgpack from C style FILE
develop
FromMsgpack/floats             166717521 ns    163715750 ns            4 bytes_per_second=52.4267Mi/s
FromMsgpack/signed_ints        171707844 ns    167527000 ns            4 bytes_per_second=51.234Mi/s
FromMsgpack/unsigned_ints      167388344 ns    164910750 ns            4 bytes_per_second=52.0468Mi/s
FromMsgpack/small_signed_ints  102578387 ns    100799000 ns            7 bytes_per_second=46.3689Mi/s

branch:
FromMsgpack/floats              61241576 ns     60370091 ns           11 bytes_per_second=142.174Mi/s
FromMsgpack/signed_ints         67659257 ns     62698083 ns           12 bytes_per_second=136.895Mi/s
FromMsgpack/unsigned_ints       59830260 ns     57518000 ns           13 bytes_per_second=149.224Mi/s
FromMsgpack/small_signed_ints   63264757 ns     61850167 ns           12 bytes_per_second=75.5688Mi/s

Questions:

for get_elements in wide_string_input_adapter, I encountered a compile error without it. If I don't implement it, tests are passing locally, which seems to make sense: if wchar is used, likely the content has non-ASCII text and we won't want to interpret it as binary numbers, where get_elements is currently used. I am falling back to get bytes one by one in this case just be safe. I think parsing binary from wchar doesn't really make sense and not going to do any optimization for it.
adding msgpack parsing via C FILE which was used for benchmarking to the benchmark.

Pull request checklist
Read the Contribution Guidelines for detailed information.

[X]  Changes are described in the pull request, or an existing issue is referenced.
[ ]  The test suite compiles and runs without error.
[ ]  Code coverage is 100%. Test cases can be added by editing the test suite.
[ ]  The source code is amalgamated; that is, after making changes to the sources in the include/nlohmann directory, run make amalgamate to create the single-header files single_include/nlohmann/json.hpp and single_include/nlohmann/json_fwd.hpp. The whole process is described here.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Specifically, I am aware of compilation problems with Microsoft Visual Studio (there even is an issue label for this kind of bug). I understand that even in 2016, complete C++11 support isn't there yet. But please also understand that I do not want to drop features or uglify the code just to make Microsoft's sub-standard compiler happy. The past has shown that there are ways to express the functionality such that the code compiles with the most recent MSVC - unfortunately, this is not the main objective of the project.
Please refrain from proposing changes that would break JSON conformance. If you propose a conformant extension of JSON to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

coverage: 99.951% (-0.05%) from 100.0%
when pulling ea8b03d86452f42b8512144e6fd68020e9d7cd82 on TianyiChen:msgpack-int
into 8c391e04fe4195d8be862c97f38cfe10e2a3472e on nlohmann:develop.

I like this idea! @TianyiChen any plans to continue working on this?

Sure, I have rebased the branch and added a from msgpack test using C FILE. Could you retrigger the CI and let's see what's missing right now?

coverage: 99.553% (-0.09%) from 99.647%
when pulling 006d23a8a3c3cbc650d31f7205eab69c6d3285b7 on TianyiChen:msgpack-int
into d333f089588961a419dfcc97b10dcd89bba19cbc on nlohmann:develop.

I could reproduce the benchmark results. Good job, thank you!
Before
FromMsgpack/jeopardy<PHONE_NUMBER> ns<PHONE_NUMBER> ns            1 bytes_per_second=38.3318Mi/s
FromMsgpack/canada              26770413 ns     26757962 ns           26 bytes_per_second=37.6418Mi/s
FromMsgpack/citm_catalog         9823588 ns      9819069 ns           72 bytes_per_second=33.2626Mi/s
FromMsgpack/twitter             10075707 ns     10066348 ns           69 bytes_per_second=38.0386Mi/s
FromMsgpack/floats             175452521 ns    175401500 ns            4 bytes_per_second=48.9339Mi/s
FromMsgpack/signed_ints        176531916 ns    176491750 ns            4 bytes_per_second=48.6316Mi/s
FromMsgpack/unsigned_ints      176015917 ns    175288250 ns            4 bytes_per_second=48.9655Mi/s
FromMsgpack/small_signed_ints  108130396 ns    107524500 ns            6 bytes_per_second=43.4686Mi/s

After
FromMsgpack/jeopardy<PHONE_NUMBER> ns<PHONE_NUMBER> ns            1 bytes_per_second=38.807Mi/s
FromMsgpack/canada              13543979 ns     13528462 ns           52 bytes_per_second=74.4518Mi/s
FromMsgpack/citm_catalog         8898435 ns      8895269 ns           78 bytes_per_second=36.717Mi/s
FromMsgpack/twitter              9764953 ns      9760380 ns           71 bytes_per_second=39.231Mi/s
FromMsgpack/floats              58513708 ns     58492750 ns           12 bytes_per_second=146.737Mi/s
FromMsgpack/signed_ints         59872042 ns     59853500 ns           12 bytes_per_second=143.401Mi/s
FromMsgpack/unsigned_ints       55908673 ns     55896000 ns           13 bytes_per_second=153.554Mi/s
FromMsgpack/small_signed_ints   61281736 ns     61260583 ns           12 bytes_per_second=76.2961Mi/s


Discussion of factors in PBR materials

We need to have a conversation about default color spaces for base color/emissive/specular factors for PBR materials. Right now we default to linear space for the factors, where textures for these inputs are in sRGB, So if I assign (128, 128, 128) in a base color texture for a material that sits next to a material that has its base color factor assigned to (128, 128, 128) in code they will look different unless the base color factor is converted to linear space.
I know this has been around since 3.3, but it seems to me that defaulting color space for base color factor to linear space is not intuitive to an artist creating the materials.

We actually did remove them a while back as it was confusing to have both and we wanted to align with GLTF.
Maybe it should be an inspector option only ?


Known issue: Generating median income estimates

Problem description:
In the 03-clean-IPUMS-data.R script, attempts to generate a median income by OCC code are currently throwing an error.
Details
Using srvyr::survey_median() on a tibble survey object grouped by OCC code results in the following error:

Error in dplyr::summarise():
! Problem while computing MED_INC = survey_median(INCTOT, vartype = c("ci")).
ℹ The error occurred in group 1: OCCSOC = "111021".
Caused by error in h():
! error in evaluating the argument 'x' in selecting a method for function 't': Can't merge the outer name ci with a vector of length > 1.
Please supply a .name_spec specification.
Run rlang::last_error() to see where the error occurred.

Currently working to figure out what's causing the issue. I think the problem might be missing replicate weights-- I've submitted for a new extract with replicate weights to see if adding those to the survey design fixes the error.

Fixed by using tidycensus package to pull down survey with replicate weights


Update f5.yml

Adding Total and Used Other Memory (Kilobytes) for F5

@ply1-ops
The OID's you added were part of a separate table that doesn't work where you put them, but I edited the commit to add in the whole sysMultiHostTable  and your OIDs.
On a single host system I dont expect it makes a different but sooner or later someone is going to try and add a stack of these things so we may as well set up the profile correctly.


Adding multiple relationships doesn't reset modal form and clears form events.

Bug Report
There's a couple of wonky behaviours around rich text relationships. I think they're related so I'm putting them as one issue.
It seems to be that if you add a relationship, select somewhere else, and add another relationship:

The form is still populated with the previous values.
The events that fetch the relationships are no longer bound to the select box.
The "Add relationship" button submits the main form rather than inserting a new relationship inline.

It's hard to explain so I've included a screen recording.
https://user-images.githubusercontent.com/3854592/112930401-2ab9cf00-9155-11eb-9df0-1b27f9f72e11.mov
Steps to Reproduce

enableRichTextRelationship on a couple of different modules.
Try to insert a relationship, should be OK
Try to insert another.
See issues described above.

Hey @richardvanbergen — thank you for this. We have fixes applied to issues 1 and 2, but 3 we are still working on.
Going to continue to chase it down. It has to do with an inner working of React concerning event bubbling out of Portals:
https://github.com/facebook/react/issues/19637
Wild stuff. Will report back shortly as now that we have identified the issue, we will likely be able to implement a workaround and get all 3 issues fixed at once!

It's ok I understand how sometimes it can be weird esoteric interactions with the core framework. I've had plenty of those myself 😅

This has all been resolved in #120! Deploying 0.5.2 as we speak!


Fix: wrong casing for .env 'BIN' variable

Closes #15

Also updates with new and tested .deb package


Upstream bug: unable to use Drawer with SSR because of bundle target strips out code

Main issue here: https://github.com/material-components/material-components-web/issues/4104

Should be fixed with: https://github.com/material-components/material-components-web/pull/4217

Closing this now as the fix has been merged. 🙃


IAM.Paginator.ListUserTags paginator is missing

Is your feature request related to a problem? Please describe.
ListUserTags mapping is available, but relevant IAM.Paginator.ListUserTags is not
>>> import boto3
>>> iam = boto3.client('iam', region_name='eu-west-1')
>>> iam.get_paginator('list_user_tags')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.8/dist-packages/botocore/client.py", line 750, in get_paginator
    raise OperationNotPageableError(operation_name=operation_name)
botocore.exceptions.OperationNotPageableError: Operation cannot be paginated: list_user_tags

Describe the solution you'd like
IAM.Paginator.ListUserTags paginator as per https://docs.aws.amazon.com/IAM/latest/APIReference/API_ListUserTags.html
    "ListUserTags": {
      "input_token": "Marker",
      "limit_key": "MaxItems",
      "more_results": "IsTruncated",
      "output_token": "Marker",
      "result_key": "Tags"
    },

@avoidik,
Thanks for the feature request. Paginators are managed by the service team so that all AWS SDKs can use them. I'll let them know of this request!


Is this PSToolkit proxy-capable?

Hi, I'm having problems with a proxy with proxy-authentication. I set the proxy IP, Port and bypass list with
netsh winhttp set proxy ...
Then I simply
$Wcl=New-Object System.Net.WebClient
$Creds=Get-Credential
$Wcl.Proxy.Credentials=$Creds
Then, using Windows Powershell, I'm able to get a token but all subsequent calls return no data.
get-hpedsccstoragesystem -verbose returns
VERBOSE: About to make rest call to URL https://eu1.data.cloud.hpe.com/api/v1/device-type1.
VERBOSE: The RestAPI Request failed with the following Status:
 HTTPS Return Code =
 HTTPS Return Code Description =
VERBOSE: Raw Response  =
VERBOSE: Null value sent to create object type.
VERBOSE: About to make rest call to URL https://eu1.data.cloud.hpe.com/api/v1/device-type2.
VERBOSE: The RestAPI Request failed with the following Status:
 HTTPS Return Code =
 HTTPS Return Code Description =
VERBOSE: Raw Response  =
VERBOSE: Null value sent to create object type.
With latest MS Powershell even obtaining a token fails, connect-dscc  -verbose returns
VERBOSE: Changing default TLS to 1.2 from 1.0
VERBOSE: Able to DNS Resolve the Single Sign On (SSO) Server Successfully : SSO.Common.Cloud.HPE.com
VERBOSE: Detected PowerShell 7, so can test-connection. Testing SSO.Common.Cloud.HPE.com
Connect-DSCC: The Single Sign-In Server SSO.Common.Cloud.HPE.com Cannot be connected to. /r/n Please fix the network enviornment before re-running this command.
Any ideas what I'm doing wrong?

I am looking into this. I have not tested this through a proxy.
generally this error occurs if you cannot make a DNS lookup for the SSO.common.cloud.hpe.com site.


chore(ci): add warning for external contributors force pushing

Description
Problem*
Resolves
Summary*
This PR adds a sticky comment to any PRs which are opened from another repository which warns against force pushing once we've started reviewing the PR.
Additional Context
Documentation*
Check one:

[x] No documentation needed.
[ ] Documentation included in this PR.
[ ] [Exceptional Case] Documentation to be submitted in a separate PR.

PR Checklist*

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Looks like we need to get this PR merged before we can test it out


Images do not scale to page width in transclusion

As the title sais, see for example, original page:

transclusion:

A quick workaround:

Create a new page (eg, transclusion-image-workaround )
Add the following to the page:

```space-style
.sb-markdown-widget-inline .content img {
  max-width: 100%;
}
```

Reload everything ( ctrl-alt-r )
Remember to delete the transclusion-image-workaround page whan a proper fix is in!


Package registration

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/548
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if Julia TagBot is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" a94cc8a749cac274553267ea4028a53dd4fb8fef
git push origin v0.1.0

@JuliaRegistrator register()

Registration pull request updated: JuliaRegistries/General/548
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if Julia TagBot is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" f5e870a47c419f15b81a586430d85fc579ed1a30
git push origin v0.1.0

@JuliaRegistrator register()

Registration pull request updated: JuliaRegistries/General/548
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if Julia TagBot is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" dc78909a714e263aebc2467433a3d0fa50907d94
git push origin v0.1.0

@JuliaRegistrator register()

Registration pull request updated: JuliaRegistries/General/548
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if Julia TagBot is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 4a975c273d66e8966eb480bed1ee03d45607cb6a
git push origin v0.1.0

@JuliaRegistrator register()

Registration pull request updated: JuliaRegistries/General/548
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if Julia TagBot is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 2b1ef4a96f89f851182fd43f886333dc9e4b26d8
git push origin v0.1.0


Can't use neovim after I deleted plugin's directory

Describe the bug
Ok, after I've run PlugClean command (I use vim-plug) to delete all unused directories in plugged folder, each time I open neovim, it shows the same error
To Reproduce
Steps to reproduce the behavior:

Install plugin
Remove plugin from list
Run :PlugClean
Open neovim

Expected behavior
Neovim as usual without this error
Version information

OS: linux
Neovim: v0.5.1

Additional context
I installed gkeepapi and keyring via pip and after I've removed plugin, I uninstalled them as well.
 Thanks a lot.


Change course start date to fix tests

Context
A test started failing when we got to 1 July. I think this is because the new funding for the next academic year has not been added to Register yet (tickets in progress) and this start date in this test now falls in the next academic year.
Changes proposed in this pull request

Temporarily change the start date until the funding has been added

Guidance to review
Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

While this will work for now, it'll still fail on the 31st July and while failing on one day rather than a whole month is significantly better it still might cause us headaches on that day. I was going to modify this file here so that funding was available for the current and next academic cycles.
https://github.com/DFE-Digital/register-trainee-teachers/blob/97f381adc6bf74d987a52dd4239982932f643655/spec/support/features/course_details_steps.rb#L114
That way the tests should be robust enough to deal with applications for the next academic cycle as well.

@kennyevil Yes, that would be a nice solution. If you've got changes ready to go, feel free to push them to this branch. Otherwise, we can merge this to unblock things, and push the other changes later on.


Add support for CowString

See here and here. In particular, it shouldn't be necessary to clone string data from a structure to put it into a Doc in all cases.

Looks like this shaved another ~1000ns off the benchmarks too.

Heh, nice...

Yeah, I'm seeing it down to around 5500ns or so, pretty sweet :]


chore: consistently use "kebab case" for context key values

Since the types of the context keys were changed in #155 we might as
well take the chance to make them consistently be the same style.
Consumers who are not using a constant will need to make a change
anyway.
BREAKING CHANGE

Decided not to do this since its not a lot of benefit for this breaking change


[Milestone] IMGUI: Expandable should use object editor/custom editor instead of drawing each field

After some digging, this seems impossible. For anyone who wants to try this feature, here are some problems I've encount which might help:

CustomDrawer of PropertyAttribute, it can create and draw editor by:
Object scriptableObject = property.objectReferenceValue;
UnityEditor.Editor.CreateEditor(scriptableObject);
editor.OnInspectorGUI();

Doing this will trigger the layout system, which will close the current property drawer. This means in list this will be drawn under the list

As most ScriptableObject uses CustomEditor rather than CustomPropertyDrawer, the propertyDrawer solution will have no point.

This is possible for UnityEngine.Editor to solve for top level fields, but it will still not work for list/array because list element uses propertyDrawer not editor drawer.


React-Native

Does this work with react-native?

@hire3d There is another open issue about react native, so will close this one in favor of https://github.com/Sitebase/react-avatar/issues/42


Embedded-Hal PR - SpiDevice transaction take an operation slice instead of a closure

This PR is in preparation of this PR
It will also resolve issue #99  and will supersede PR #223  & PR #138
The linked embedded-hal PR also already implements changes from current master embedded-hal::delay & embedded-hal::digital, therefore this PR made also changes to gpio.rs and mod.rs to make it work.

keep track with e-hal master -> i2c iter fn trait imps where removed https://github.com/rust-embedded/embedded-hal/pull/440

ci fails duo to https://github.com/rust-lang/cargo/issues/11939


Add dock widget support

Describe the feature
This would be somewhat similar to the already existing splits, except instead of having a gradient handle for how much space each component should get, there would be discrete buttons that toggle each component between being displayed or not.
In fact, it may make sense to add dock functionality as an option on splits, since you may want to use both at the same time; they are not mutually exclusive. I put an example of how this could look below, although we would probably need more than just a bool option, depending on where you want to place the dock and how many different modalities you want it to have; in #742, one of the example dock widgets appears to support going between hidden, small, and full size.
I can probably work on this soon, although it would be helpful to have some more feedback about dock positioning and modalities for intended use cases.
Relevant code
sp := core.NewSplits(parent).SetDock(true)
core.NewText(sp).SetText("First")
core.NewText(sp).SetText("Second")

The Rust based GUI framework Egui dock has docking type functionality.
It would be useful to create sub windows which can be resized, closed, dragged and docked.
The following shows examples.

The Drag-n-drop support in Cogent Code would probably need some improvements to support this egui-dock kind of functionality, including #33 for going across different windows and apps.  #84 is also related.  Our current event handling happens mostly on a per-Scene level, via the Events type, so coordinating across different Scenes (windows) would potentially require some additional global event coordination infrastructure.
In general, everything is a widget and widgets can be put anywhere, so there aren't any other obvious in-principle barriers to implementing such a thing in terms of the contents of the panels.
Overall, there are a number of issues with this kind of functionality that personally limit my interest in pursuing such a thing, but nevertheless, it would be good to ensure that the framework can support it if someone wants to implement it:

It doesn't translate well to mobile and web, which are single-window platforms, potentially with small screens.  In general our focus is on providing GUI designs that naturally translate across platforms, so you can CORE (code once, run everywhere) with one codebase.

As a user of apps that employ this kind of highly customizable layout stuff, I find myself quickly getting lost, and spend more time than I would like fiddling with the layout rather than getting work done.  Metaphorically, I feel that it is like the difference between C++ and Go: there is so much complexity and so many different ways to organize everything.. wouldn't it be nicer if there was a much simpler, general-purpose way of doing things?  Exerting a bit of austerity and discipline might end up producing improved usability.

For example, in the Cogent Code editor, a very simple "4 panel" design works quite well, with a file tree, 2 editor panels, and a collection of tabs for all other functions.  In this design, you always know exactly where to look for any kind of random thing like find results, debugger, etc.  Also, we make everything accessible using the app chooser, where you can search for all commands, files, etc, providing direct text-based access to all functionality in one spot.  OTOH, on smaller screens, it might be useful to have the tabs panel go along the bottom instead of the right.  But that is easily accomplished with a single toggle somewhere, and doesn't require any of the complexity of the egui_dock thing.  Also, we are planning to add a double-click on split handles to collapse and expand them, along with better builtin support for persisting tab splits.

In terms of a general-purpose implementation of this kind of approach, it seems like there are many app-specific decisions and constraints that would be needed, in terms of what kinds of things make sense to go where, and how it all connects together, that I wonder about the utility of a single framework for this functionality.  You could quickly end up in a bad space of parameterizing the functionality with complex declarative data structures, in a way that inevitably won't handle the one case you end up needing.. So a more component-based, flexible, code-driven implementation would be better.

Also all the little widget "handles" (e.g., tabs) for each panel need to live somewhere, so you need to build in spaces for these to live -- these seem like important app-specific design decisions in general.  In VSCode and Adobe products, these handles proliferate everywhere and are generally rather obscure and hard to identify, and they definitely need bits of space in various places..  lots of complexity to this it seems.

I agree with the premise that these types of complex configurable layouts are more suited to large screen desktop type applications, rather than single, smaller screen mobile type devices.
Additionally, I also agree a more simplistic approach is better across multiple platforms. This functionality would still be extremely useful in allowing users to organise multiple views in more complex GUI applications.

I agree with @rcoreilly about the general idea that excessively complicated GUI layouts are not good, and should be avoided if possible. However, I also recognize that sometimes more sophisticated functionality may be necessary.
I am certainly in favor of the changes proposed in #996, and I hope that those will fulfill most of your needs. If you have specific situations in which you need more functionality than #996 provides, please tell us about them and we will work with you on figuring out a way to structure it well. It is possible that may reveal a need for more advanced dock-like functionality. I don't want to implement things that aren't necessary, but I am absolutely happy implementing complex things if they are necessary for real use cases.

We implemented support for splits tiling in #1026, which moves us toward supporting complete dock functionality. This allows you to structure your entire app into one unified splits structure that can be organized through 2D tiling.
Soon, we will also implement support for automatic saving of these splits structures and support for end-user customization of tiling, which, in conjunction with potential future support for interactions with tabs and dialogs, should provide sufficient dock support to satisfy any potential dock use cases.


[cocoon] Stop monitoring flutter/plugins repo.

The flutter/plugins repo is now archived, so there's no need that cocoon does anything with it. (The repo is still accessible from cocoon's "repo" dropdown.)
The ecosystem team uses now flutter/packages as the main mono-repo for work.
Feel free to remove flutter/plugins from cocoon whenever!
Thanks!

Cleaned up via https://github.com/flutter/cocoon/pull/2861.

Awesome, thanks @keyonghan!


Error new update the nativescript

Error new update the nativescript

Could you elaborate on the problems? Are you upgrading to new nativescript version and after upgrade the plugin fails?

yes this is the problem

What are the errors? Does the app even launch? Or after launch there is a point where it fails? Is there some output you may provide so I can investigate it further?

When I run a run android after installation fails, and so I read it is by using java function deprecated.

Same problem<EMAIL_ADDRESS>send you an email.
basicly deprecated api usage, android platform


Move tokens into its own directory

Pull Request Test Coverage Report for Build 3297

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 3292:
0.0%

Covered Lines:
698

Relevant Lines:
698

💛  - Coveralls


Support 64 bit extent randint

Fix #2816.
Compared to #2823, this PR keeps using 32-bit RNG.

I think this is the way to go.

Jenkins, test this please

Jenkins CI test (for commit 072d8808a3270b21dbdb7bb0c8ca25f3b5c45210, target branch master) failed with status FAILURE.

Seems like some of the tests requires fixes 😭

Jenkins, test this please

Jenkins CI test (for commit aa698daec519c97f50900d6f31d47649f961953f, target branch master) succeeded!

I'd rather not regard the previous behavior as a bug because the error message is good.


Order of transactional emails

Sometimes the order of the transactional emails is wrong: the payment confirmation is sent before the order confirmation is sent. This is confusing for our customers. We would like the order confirmation to be send immediately when an order is placed, regardless of whether the payment succeeds or not. I'm not sure if this is the standard Magento flow or that this is due to the Adyen plugin.

Hi oblomovx,
What is the payment confirmation email ? The order confirmation email is send when the payment is made. The default flow is in Magento different from API payments and Gateway (redirect) payments.
Could you be more specific when you have this trouble and what the payment confirmation email is ?
Regards,
Rik
Adyen

Hi Rik,
With the payment confirmation email I mean the Invoice email. When a payment is completed, the order email and the invoice email are both "sent", but I have noticed that the order email is added to a queue that is sent via cron, while the invoice email is sent directly. This causes the invoice email to arrive earlier in the clients mailbox than the order email.
Also, some payment methods take a couple of minutes to be processed (paypal for example), during which the customer gets no email at all.
So, the best behavior in our view would be that an order email is sent immediately after pressing the "Order" button in the checkout. If the customer doesn't pay, he still receives this email. Then, after payment is successful, the customer receives the invoice mail.

Hi oblomovx,
I don't understand that the invoice mail can be send but the customer get's no email at all. Could you explain this ?
On the AUTHORISATION notification the order confirmation is send. If you are on auto capture then the invoice will automatically created. If you are on manual capture it will create an invoice when the payment is captured. With the latest version of Magento they have introduce to queue the order confirmation mail. What is the benefit of sending an invoice email to your customer ?
Regards,
Rik


Unhandled exception escaping from a worker thread!

librealsense
2.50.0 RELEASE

OS
Windows

Intel RealSense Viewer / Depth Quality Tool has crashed with the following error message:
Unhandled exception escaping from a worker thread!
Error type: Access Violation!

Hi @vstadnytskyi  Which RealSense camera model do you have, please?
Does the error still occur if you follow the procedure below?

Install the older RealSense Viewer version 2.48.0 by visiting the link below and downloading and running the file Intel.RealSense.Viewer.exe in the 'Assets' file list?

https://github.com/IntelRealSense/librealsense/releases/tag/v2.48.0

After launching the 2.48.0 Viewer, close it and launch the 2.50.0 version of the Viewer again.

Hi @vstadnytskyi  Do you require further assistance with this case, please?  Thanks!

Thank you, @MartyG-RealSense  for a speedy reply. The problem was with the faulty RAM on the windows 11 PC. It caused many strange behaviors including issues with Intel RealSense viewer.
After running memtest86 on the computer and extracting faulty memory, the issue with Intel RealSense disappeared.
Thank you!

It's great to hear that you achieved a solution, @vstadnytskyi - thanks very much for the update!

@MartyG-RealSense Thank you!

@Lumos2050  You are very welcome  :)


bump STS

This PR fixes #22565

https://github.com/microsoft/sqltoolsservice/compare/<IP_ADDRESS>...<IP_ADDRESS>

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.003%) to 41.711%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/dialog/configurePython/configurePathPage.ts
1
57.02%

Totals

Change from base Build<PHONE_NUMBER>:
0.003%

Covered Lines:
30336

Relevant Lines:
67333

💛  - Coveralls


[TEP-0104] Support Task-level Resource Requirements for TaskRun: Part #1 Fields Addition & Validation w/ Docs Updates

Changes
/kind feature
The related TEP:
TEP-0104: Support Task-level resource limits
TEP-0104: Scope Updates to Runtime
TEP-0104: Rename "resources" field as "computeResources"
The related FR thread:
FR: Task-level (and maybe Pipeline-level) resource requests and limits
This PR will include changes as:

Add the required "computeResources" field for TaskRun
Add the related validations
Update the user guide for this new feature

A follow-up PR will include changes to update compute resources configuration by the specified task-level resource requirements:

ref-to-be-added

TODO

[x] Update this PR only with field addition and webhook validation
[x] Add alpha FF to guard changes over webhook
[x] Update related docs
[x] Confirm approval and merge on TEP-0104, then start merging implementations

After the scope update & field name update:

[ ] Update field validation
[ ] Update the related user guidance

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Docs included if any changes are user facing
[x] Tests included if any functionality added or changed
[x] Follows the commit message standard
[ ] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in
(if there are no user facing changes, use release note "NONE")

Release Notes
As changes will be delivered in 3 PRs, the release notes will be added in ref-to-be-added.

agree. thinking of a bit the complexity, was planning to complete the logic Task-level first, then for "runtime overwriting" from TaskRunSpec. but let me do this perhaps (for an easier review):

field addition and validation on current PR
Task-level update on 2nd PR
"runtime overwriting" on 3rd PR

ok sounds good! Just one more note, new fields should be guarded behind the alpha feature flag

got this. So the feature flag checks will be mainly/only needed in reconciler code right? (not changes wekbook code before, let me have another check. perhaps also needed there)

The feature flag checks should be included in the webhook validation, and optionally the reconciler validation, for the same reason as before (fail as fast as possible).

thanks @lbernick. pushed changes for this round of reviews.
will follow

TEP PR got merged asap (reached out community slack channel, and will check in the following API WG session)
further refining feedback on the user guide docs

/retest

/hold cancel
since https://github.com/tektoncd/community/pull/703 is now implementable

/assign @lbernick  @dibyom

cc @ywluogg

/retest

/meow

/retest


Add a mechanism to reset the page an RV is bound to

I have a case where I'd like to reuse the same IRV with multiple different pages, but right now it can only be bound to the first page

Tracking this in #34


Running atom in Ubuntu (x86), after build + install,  throws 'Syntax error: Unterminated quoted string'

Description
I build atom from sources 1.10.0, 1.12.1, and 1.12.2. Build .deb and the install via package manager in Ubuntu.
Then, after successful installation, I run atom . and I get this error:
nmartell@oficina-desktop:~$ atom .
nmartell@oficina-desktop:~$ /usr/share/atom/atom: 1: /usr/share/atom/atom: ELF: not found
/usr/share/atom/atom: 1: /usr/share/atom/atom: Syntax error: Unterminated quoted string

Version 1.10.0 was previously installed, then I build v1.12.2 to upgrade, but I got that problems. Now I can't even install the v1.10.0. :cry:
Steps to Reproduce

Run script/build ----create-debian-package in order to build atom.
Install the resulting *.deb
Run atom . to open editor.

Expected behavior: Open Atom Editor
Actual behavior: Throws this error: /usr/share/atom/atom: 1: /usr/share/atom/atom: ELF: not found /usr/share/atom/atom: 1: /usr/share/atom/atom: Syntax error: Unterminated quoted string
Versions
nmartell@oficina-desktop:~$ atom --version
nmartell@oficina-desktop:~$

(Yes, nothing)

nmartell@oficina-desktop:~$ apm --version
apm  1.12.9
npm  3.10.5
node 4.4.5
python 2.7.12+
git 2.9.3

nmartell@oficina-desktop:~$ node --version
v4.6.2

npm --version
3.10.9

There was a problem with building Atom for 32bit Linux that was fixed in https://github.com/atom/atom/pull/13001. I believe this fix is currently only in beta. Try either building beta or build Atom with this PR. Sorry for the inconvenience.

I manual remove 'atom' references after uninstall ( #2195) and then I was able to install old version v1.10.0 for the meantime.
Thanks, @Ben3eeE I'll wait to upgrade in the next v1.13.0 release.

I'm still not able to run in x86 for atom 1.12.7. u.u
This time from Fedora 24:

script/build --create-rpm-package:
Node:	v4.7.0
Npm:	v4.0.5
Installing script dependencies
Installing apm
../deps/libgit2/deps/zlib/inflate.c: En la función ‘inflateMark’:
../deps/libgit2/deps/zlib/inflate.c:1507:61: aviso: left shift of negative value [-Wshift-negative-value]
     if (strm == Z_NULL || strm->state == Z_NULL) return -1L << 16;
                                                             ^~
../deps/libgit2/deps/zlib/inflate.c: En la función ‘inflateMark’:
../deps/libgit2/deps/zlib/inflate.c:1507:61: aviso: left shift of negative value [-Wshift-negative-value]
     if (strm == Z_NULL || strm->state == Z_NULL) return -1L << 16;
                                                             ^~
Installing modules ✓
Installing<EMAIL_ADDRESS>✓
Installing<EMAIL_ADDRESS>✓
Installing<EMAIL_ADDRESS>✓
# . . .
Installing<EMAIL_ADDRESS>✓
Wrote Dependencies Fingerprint: /home/nmartell/src/atom-1.12.7/node_modules/.dependencies-fingerprint a9bde3eb677ce25a725e258539ffdd1af43f7481
Copying assets to /home/nmartell/src/atom-1.12.7/out/app
Transpiling Babel paths in /home/nmartell/src/atom-1.12.7/out/app
Transpiling CoffeeScript paths in /home/nmartell/src/atom-1.12.7/out/app
Transpiling CSON paths in /home/nmartell/src/atom-1.12.7/out/app
Transpiling PEG.js paths in /home/nmartell/src/atom-1.12.7/out/app
Generating module cache for /home/nmartell/src/atom-1.12.7/out/app
Generating pre-built less cache in /home/nmartell/src/atom-1.12.7/out/app/less-compile-cache
Generating metadata for /home/nmartell/src/atom-1.12.7/out/app/package.json
Generating API docs at /home/nmartell/src/atom-1.12.7/docs/output/atom-api.json
Skipping Chromedriver download because it is used only on macOS
Dumping symbols in /home/nmartell/src/atom-1.12.7/out/symbols
Running electron-packager on /home/nmartell/src/atom-1.12.7/out/app with app name "atom"
Downloading electron-v1.3.13-linux-x64.zip
[============================================>] 100.0% of 42.63 MB (33.37 kB/s)
Packaging app for platform linux x64 using electron v1.3.13
Changing permissions for node files in /home/nmartell/src/atom-1.12.7/out/atom-1.12.7-i386
Copying non-ASAR resources to /home/nmartell/src/atom-1.12.7/out/atom-1.12.7-i386/resources
Writing LICENSE.md to /home/nmartell/src/atom-1.12.7/out/atom-1.12.7-i386/resources
Application bundle created at /home/nmartell/src/atom-1.12.7/out/atom-1.12.7-i386
Skipping creating debian package. Specify the --create-debian-package option to create it.
Creating rpm package for "/home/nmartell/src/atom-1.12.7/out/atom-1.12.7-i386"
Creating rpm package directory structure at "/home/nmartell/rpmbuild"
Copying "/home/nmartell/src/atom-1.12.7/out/atom-1.12.7-i386" to "/home/nmartell/rpmbuild/BUILD/Atom"
Copying icons into "/home/nmartell/rpmbuild/BUILD/icons"
Writing rpm package spec file into "/home/nmartell/rpmbuild/SPECS"
Writing desktop entry file into "/home/nmartell/rpmbuild/BUILD"
Copying atom.sh into "/home/nmartell/rpmbuild/BUILD"
Generating .rpm package from "/home/nmartell/rpmbuild"
Copying "/home/nmartell/rpmbuild/RPMS/i686/atom-1.12.7-0.1.fc24.i686.rpm" into "/home/nmartell/src/atom-1.12.7/out/atom.i686.rpm"

Why Downloading electron-v1.3.13-linux-x64**.zip (x64)?

The PR you referenced should fix the issue of downloading the incorrect Electron version but I believe it is only available in Atom 1.13.0 still only in beta and you are building Atom 1.12.7. Did you try to build with the changes in that PR?

No. I'll try with 1.13.0-beta8. Thanks.

It works atom-beta 1.13.0.

Great to hear that it's working for you 🙂


Add support for custom config in Vagrantdir

This allows us to put additional Vagrantfiles inside a "Vagrantdir", for
user customized boxes. Also add a .gitignore entry for a
user_provision.sh in the root, that I can use to make additional
provisions for my customized boxes. :)

Please also document this behavior in the README

An alternative name for Vagrantdir could be Vagrantfile.d (credits to Aksel), but anything works for me!


Add the new  webpages regarding valuable quotes

I would like to add some valuable quotes of great personalities all over the worlds and several life lessons(some of the incidents and experiences being shared by them, ex- A P J Abdul kalam sir) that may motivate the user.

I would like to add some valuable quotes of great personalities all over the worlds and several life lessons(some of the incidents and experiences being shared by them, ex- A P J Abdul kalam sir) that may motivate the user.

Sure!

Sir , Is there any discord channel for this project too.
On Wed, 2 Mar 2022, 6:41 pm VIDHI BHATT, @.***> wrote:

I would like to add some valuable quotes of great personalities all over
the worlds and several life lessons(some of the incidents and experiences
being shared by them, ex- A P J Abdul kalam sir) that may motivate the user.
Sure!
—
Reply to this email directly, view it on GitHub
https://github.com/arpit456jain/Cool-Front-End-Templates/issues/255#issuecomment-1056914700,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVK3BK2AMH6C5KY4G7JYVBTU55SIHANCNFSM5PXCQAWA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were assigned.Message ID:
@.***>

yes

I want to work on this under GSSOC2022. Here is my discord profile DHANANJOY DEY | Contributor#7903

Issues are assigned on an FCFS basis. You can check out other issues @sceary-expert

Is it completed? If not, reopen this issue when PR is ready

Sir, I have to raise new issue????

Sir I have reopened the issue today but it's showing failed and something like ---"issue labeler"


no_std + alloc crate

could you give https://github.com/brandonros/ureq_wasm_ui a look and help me track down where the std::io usage is coming from (it's in ureq crate but I'm  not sure exactly where)? Even if we stub it out with "panic if wasm32 platform", I could help implement "fetch" from there?

I can have a look tomorrow!
This PR fixes only ureq-proto, which is underpinning ureq. So there is still work I need to do in ureq itself. Is this test checking ureq-proto or ureq?


Is it possible to hot-reload template.yaml?

Hot reloading source files with aws-sam-webpack-plugin is such a wonderfull and convenient thing that it's hard to remember why things stopped to work after adding another lambda to template.yaml :)
So the question is is it really possible to hot-reload template.yaml when it's been changed?

I think this is a relative thing: https://github.com/webpack/webpack-cli/issues/15

You might be able to use nodemon to restart webpack but I can't see another solution. The only time the entry points can be set in the webpack.config.js is when webpack loads it. If they provide another way to set them then I'm happy to revisit this request.

After re-reading the Webpack docs I'm going to change the recommended webpack.config.js from entry: awsSamPlugin.entry() to entry: () => awsSamPlugin.entry() which will cause it to reload the entry points each time it builds. It will still only build once a source file is changed (not template.yaml) but you'll always be building from the most recently saved template.yaml.

Wow. I missed that entry-function feature in webpack. Thanks.
Yeah, I'm already using nodemon.
I know that webpack plugins can add dependencies to watch into webpack while building projects.
May be awsSamPlugin could do that for template.yaml?
Then it wouldn't need to use nodemon.
This will allow to escape repeating file names and is gonna be less error-prone.

P.S. There is also one trick, if awsSamPlugin.entry is not a method of a class, but just a field with a function, then it can be used like entry: awsSamPlugin.entry, without brackets.

I'm not planning to add further support for hot reloading based on the template.yaml changing.


Content provider context

Candidate facing content added.
Think we might need to do something with a subhead – maybe
Candidate response
to distinguish their response from guidance when the details component is open

Nice – how about ‘What the candidate sees’?


All sensors unavailable

I have tried removing the custom component folder and reinstalling with ver 2 and beta version 3 but still the same. All my sensors read unavailable? All of my zones are present but how do I monitor the sensors?

Can you let me know if this is still an issue with v3.0.1? If it is, can you post the sensor data for one of your sensors using these instructions?

Does not work with 3.0.1…here is the debug log for one of themThis error originated from a custom integration.Logger: custom_components.alarmdotcomSource: custom_components/alarmdotcom/init.py:158 Integration: Alarm.com (documentation, issues) First occurred: 6:19:46 PM (1 occurrences) Last logged: 6:19:46 PMALARM.COM DEBUG DATA FOR BASEMENT BACK DOOR: {"deviceType": 1, "openClosedStatus": 2, "state": 0, "isFlexIO": false, "deviceRole": 0, "displayStateText": "Not Monitored", "isMonitoringEnabled": false, "sensorNamingFormat": 4, "supportsBypass": false, "supportsImmediateBypass": false, "isBypassed": false, "managedDeviceType": 12, "hasState": true, "canBeRenamed": true, "canBeDeleted": false, "canAccessWebSettings": true, "canAccessAppSettings": false, "webSettings": 400, "canAccessTroubleshootingWizard": false, "troubleshootingWizard": null, "addDeviceResource": 0, "canBeAssociatedToVideoDevice": true, "associatedCameraDeviceIds": {}, "macAddress": "", "manufacturer": "", "isAssignedToCareReceiver": false, "isOAuth": false, "isZWave": false, "supportsCommandClassBasic": false, "isMalfunctioning": false, "canBeSaved": true, "canChangeDescription": true, "description": "BASEMENT BACK DOOR", "deviceModelId": 110, "canConfirmStateChange": true, "canReceiveCommands": false, "remoteCommandsEnabled": true, "hasPermissionToChangeState": true, "deviceIcon": {"icon": 317}, "batteryLevelNull": null, "lowBattery": false, "criticalBattery": false}Sent from my iPadOn May 28, 2023, at 1:16 PM, Elahd Bar-Shai @.***> wrote:
Can you let me know if this is still an issue with v3.0.1? If it is, can you post the sensor data for one of your sensors using these instructions?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Alarm.com is reporting an unknown state for the sensor ("state": 0). It also looks like the sensors aren't being monitored ("isMonitoringEnabled": false). What do you see at https://www.alarm.com/web/Security/Sensors.aspx?

Here is a screenshot of website. They state status ok and are monitored…Sent from my iPhoneOn May 29, 2023, at 10:16 PM, Elahd Bar-Shai @.***> wrote:
Alarm.com is reporting an unknown state for the sensor ("state": 0). It also looks like the sensors aren't being monitored ("isMonitoringEnabled": false). What do you see at https://www.alarm.com/web/Security/Sensors.aspx?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Do you have multiple partitions or systems?

I am pretty sure I only have one partition but not sure how to check to conform. I only have one alarm system. The fire and panic sensors seems to register fine?Sent from my iPhoneOn May 30, 2023, at 4:22 PM, Elahd Bar-Shai @.***> wrote:
Do you have multiple partitions or systems?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Thanks. I'm going to mark this as a duplicate of #241.
You can check your systems and partitions using the adc command line tool. Follow the instruction here to install the tool, then run the "Basic" command from the table. Check under the Systems and Partitions sections and post what you see in #241.


Markdown rendering?

What's your opinion on potentially supporting markdown in the "lists" / "posts" files?
I'm a frontend web developer by day, I could definitely help you build it into your frontend :+1:

Greetings!  I explicitly did not want to support markdown for lists.sh.
I have a GH discussion in this repo that talks about a possible markdown blog platform using the same tech under the hood: https://github.com/neurosnap/lists.sh/discussions/10
Feel free to chat there.


Check for async void

We should be checking for async void.

Not likely that we will pursue this. Closing


Added example on how to create custom Service Monitor

@alexanderdavidsen this PR didn't propagate to https://github.com/helm/charts/tree/master/stable/prometheus-operator when the migration was done.
Also, other chart-devs may not be aware. Currently the stable/prometheus-operator chart enforces the following label for serviceMonitorSelector
> kubectl get prometheus -o yaml --all-namespaces -o \ jsonpath='{.items[*].spec.serviceMonitorSelector.matchLabels}'

map[release:k8s]

However https://github.com/helm/charts/tree/master/stable/nginx-ingress gets created with release: nginx-ingress and hence the ServiceMonitor never gets picked up.
The solution is buried in this post migration PR and not in the current documentation. I can make a PR on https://github.com/helm/charts/tree/master/stable/prometheus-operator to fix this.


Add assert_eq to testing documentation

[ ] Tests added / passed
[ ] Passes flake8 dask

Thanks for adding this Matt. LGTM.

This merely documents a change that is already going to be included in the next release. Merging...


Improve test resource logic.

Current dependencies on/for this PR:

main

PR #334   👈

This stack of pull requests is managed by Graphite.


HTTP request hangs for some site

Checklist

[x] I've searched for similar issues.
[x] I'm using the latest version of HTTPie.

What are the steps to reproduce the problem?

hit the "Try Online" button of the homepage of https://httpie.io/
input command 'http https://www.cmegroup.com' and hit Enter
the output hangs without response body

What is the expected result?
The response body text (web page source codes)
What happens instead?
None. Empty.
Debug output
Please re-run the command with --debug, then copy the entire command & output and paste both below:
~ $ http --debug https://www.cmegroup.com
HTTPie 2.4.0
Requests 2.26.0
Pygments 2.9.0
Python 3.7.6 (default, Jan 30 2020, 09:44:41)
[GCC 9.2.1 20190827 (Red Hat 9.2.1-1)]
/usr/bin/python3
Linux 4.18.0-147.3.1.el8_1.x86_64

<Environment {'colors': 256,
 'config': {'default_options': ['--style=monokai']},
 'config_dir': PosixPath('/home/termible/.httpie'),
 'devnull': <property object at 0x7fbe6e995a10>,
 'is_windows': False,
 'log_error': <function Environment.log_error at 0x7fbe6e99a5f0>,
 'program_name': 'http',
 'stderr': <_io.TextIOWrapper name='<stderr>' mode='w' encoding='UTF-8'>,
 'stderr_isatty': True,
 'stdin': <_io.TextIOWrapper name='<stdin>' mode='r' encoding='UTF-8'>,
 'stdin_encoding': 'UTF-8',
 'stdin_isatty': True,
 'stdout': <_io.TextIOWrapper name='<stdout>' mode='w' encoding='UTF-8'>,
 'stdout_encoding': 'UTF-8',
 'stdout_isatty': True}>

>>> requests.request(**{'auth': None,
 'data': RequestJSONDataDict(),
 'headers': {'User-Agent': b'HTTPie/2.4.0'},
 'method': 'get',
 'params': <generator object MultiValueOrderedDict.items at 0x7fbe6e8ce4d0>,
 'url': 'https://www.cmegroup.com'})

Provide any additional information, screenshots, or code examples below:
The same problem occurs for https://www.nasdaq.com, while good for https://www.bing.com and many others.

This maybe network issue.
I verified all sites above work well with httpie.

I also verified the website in the given reproducer works correctly on the latest revision. If you still experience this issue, please let us know.


Allow a check to see if any rollouts are active?

I needed to check if there were any rollouts active for a specific user. Could be a useful feature.

@andy4thehuynh I think it has been implemented by https://github.com/FetLife/rollout/pull/90 ... can you please try it? :)

closing for now... feel free to reopen :)

Go for it! Our app is using an older version of Rollout that doesn't have the new features.


Can't execute sql in mssql and alert Connection is closed

Describe the bug
Recently I found that i can't execute mssql sql by using the tool any more And alert message connection is closed,by the way, the same sql use microsoft mssql tool with no problem.i have test the connection correctly.
To Reproduce

create a connection to mssql 2008 R2
choose the sql
Ctrl+E Ctrl+E
Use another computer repeat the operation

Expected behavior
execute success like before
Screenshots

Desktop (please complete the following information):

SQLTools Version:0.21.2
VSCode Version: 1.40.0
Dialect Used/Version:MSSQL 2008 R2
OS: Windows 10

Additional context
the mysql connection is well and can unfold the database

I have the same problem, but I have not found a message supported by the MSSQL version.

@zhutoutou @wjfang90 Does your connection need encryption?

@zhutoutou @wjfang90 Does your connection need encryption?

No,I connected direct

@zhutoutou @wjfang90 Does your connection need encryption?

No,I connected direct

Me too

Did anyone find a solution to this issue?

Looks like this bug concerns version 2008 only. I had same alert "Connection is closed" without displaying a result for simplest "select getdate()" query, when sqltools connected to MSSQL Express 2008. It  didn't create tree of schemas and their objects too.
I installed MSSQL Express 2019, vscode works fine with this version of database.

This was fixed on newer versions.


Include a allowed_tlds property in AnyUrl

Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.4
            pydantic compiled: False
                 install path: .../virtualenvs/py-pydantic/lib/python3.7/site-packages/pydantic
               python version: 3.7.4 (default, Sep 26 2019, 11:32:05)  [Clang 10.0.0 (clang-1<IP_ADDRESS>)]
                     platform: Darwin-17.7.0-x86_64-i386-64bit
     optional deps. installed: ['typing-extensions']

I think it would make sense to have an allowed_tlds property in AnyUrl. Use case is very specific, but it would only require to change a couple of lines in the library, and it will allow things like:
class TorUrl(AnyUrl):
    allowed_schemes = {'http', 'https'}
    allowed_tlds = {'onion'}

I can prepare a PR if you like the idea.

Humm, I'm torn since it should be fairly easy for you to implement your own custom type to implement this.
Why not start by implementing your own class/type and we'll leave this open to see if anyone is interested.

Sure :)