VS Code extension host launches 2nd VS Code Icon in dock on macOS

Issue Type: Bug

Start up VS Code on macOS 12.3.1 Monterey
Wait for the extension host to start up
Notice a second VS Code icon appear in the dock.

VS Code version: Code 1.66.2 (dfd34e8260c270da74b5c2d86d61aee4b6d56977, 2022-04-11T07:47:29.396Z)
OS version: Darwin arm64 21.4.0
Restricted Mode: No

System Info

Item
Value

CPUs
Apple M1 Pro (10 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 5, 5

Memory (System)
32.00GB (8.29GB free)

Process Argv
--crash-reporter-id 3d2f6136-acb4-4301-9e21-54d9feb3bd9c

Screen Reader
yes

VM
0%

Extensions (52)

Extension
Author (truncated)
Version

better-comments
aar
3.0.0

svelte-intellisense
ard
0.7.1

astro-vscode
ast
0.16.0

spellright
ban
3.0.64

vscode-tailwindcss
bra
0.8.3

better-toml
bun
0.3.2

path-intellisense
chr
2.8.0

postcss
css
1.0.9

vscode-eslint
dba
2.2.2

EditorConfig
Edi
0.16.4

vscode-npm-script
eg2
0.3.24

prettier-vscode
esb
9.5.0

vscode-svelte-snippets
fiv
0.5.0

auto-close-tag
for
0.5.14

auto-complete-tag
for
0.1.0

copilot
Git
1.15.5815

vscode-pull-request-github
Git
0.40.0

gitkraken-authentication
git
1.0.3

headwind
hey
1.7.0

JSDocTagComplete
Hoo
0.0.2

howivscode
How
0.1.0

caddyfile-support
mat
0.2.0

template-string-converter
meg
0.6.0

fluent-icons
mig
0.0.17

dotenv
mik
1.0.1

vscode-dotnet-runtime
ms-
1.5.0

playwright
ms-
0.2.3

live-server
ms-
0.2.12

vsliveshare
ms-
1.0.5527

debugger-for-edge
msj
1.0.15

svelte-autoimport
piv
1.0.4

material-icon-theme
PKi
4.16.0

prisma
Pri
3.13.0

vscode-thunder-client
ran
1.16.2

java
red
1.5.0

vscode-sort-json
ric
1.20.0

mdx
sil
0.1.0

shader
sle
1.1.5

stately-vscode
sta
1.7.6

addDocComments
ste
0.0.8

vscode-stylelint
sty
1.2.2

svelte-vscode
sve
105.16.0

pdf
tom
1.2.0

sort-lines
Tyr
1.9.1

vscodeintellicode
Vis
1.2.20

vscode-java-debug
vsc
0.40.1

vscode-java-dependency
vsc
0.19.1

vscode-java-pack
vsc
0.22.4

vscode-java-test
vsc
0.35.0

vscode-maven
vsc
0.35.1

fig
wit
0.0.6

markdown-all-in-one
yzh
3.4.3

(2 theme extensions excluded)

I also see this every now and then. But I don't think this has anything to do with the extension host.

Well, I’m not 100% sure either, but (I should have mentioned this) closing the second icon causes the extension host to quit.
On 2 May 2022, 15:19 +0200, Alexandru Dima @.***>, wrote:

I also see this every now and then. But I don't think this has anything to do with the extension host.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Can you try to reproduce with our nightly insider builds? You can give our preview releases a try from: https://code.visualstudio.com/insiders/

Can you disable spellright extension and see if the issue repros ? If not, please try our extension bisect utility https://code.visualstudio.com/blogs/2021/02/16/extension-bisect#_welcome-extension-bisect to narrow down the extension.
Duplicate of https://github.com/microsoft/vscode/issues/138712

@deepak1556 Yep, Spellright seemed to be the culprit! Can't reproduce this issue with Spellright disabled.


Porygon Conversion: Should not be permanent

I think a lot of the Weakness or Resistance getters may be permanently changing the affected value. We should probably take a look at them and make sure they are making new objects instead of modifying existing ones.


JAV-103 resolve PropertyPlaceholderConfigurer conflict

Coverage decreased (-0.0009%) to 85.374% when pulling 8e54d2d2ca4fb82eefd04445765fc0453cda715f on wujimin:PropertyPlaceholderConfigurer-conflict into c52c2467ace6e084ba941bd76b47cfdf177672aa on ServiceComb:master.


HttpClient logging

In the IJA_API I have added a feature to make it possible to view logging of HTTP requests and responses. It's very helpful when there is an issue and you want to see for HTTP traffic what is received and sent.
In the IJA_API there is also HttpRequest logging. Also this feature is very helpful. In the IJA_API this is regulated by a separate pipe (which has to be included in the configuration). An improvement would be to add it to WebServiceListener/RestListener/ApiListener.

Nice feature. We will do this in a more generic way so you can configure logging more fine grained on a separate page in the GUI

Semi-fixed in #1847. You will still need to manually configure (in the log4j4ibis.xml) which loggers/appenders you would like to use. The log level can be changed dynamically on the log settings page.

Extending the log is already possible, see #6142.
Spring boot like log config can be used to easily add new loggers through environment variables.
eq;
logging.level.root=INFO
logging.level.org.frankframework=DEBUG
logging.level.org.springframework=WARN


Run failure

I have tried running  this with various switches and every time I get
Failed request #1 while fetching . Exception message: The SSL connection could not be established, see inner exception..
Failed request #10 while fetching . Exception message: The SSL connection could not be established, see inner
exception.. downloaded ----------------------------------------   0% ::** -
System.Net.Http.HttpRequestException: The SSL connection could not be established, see inner exception.
System.Security.Authentication.AuthenticationException: Authentication failed, see inner exception.
System.ComponentModel.Win32Exception: The Local Security Authority cannot be contacted
.
at async Task System.Net.Security.SslStream.ForceAuthenticationAsync(TIOAdapter adapter, bool
receiveFirst, byte[] reAuthenticationData, bool isApm)
at async ValueTask System.Net.Http.ConnectHelper.EstablishSslConnectionAsync(
SslClientAuthenticationOptions sslOptions, HttpRequestMessage request, bool async, Stream stream,
CancellationToken cancellationToken)
at async ValueTask System.Net.Http.ConnectHelper.EstablishSslConnectionAsync(SslClientAuthenticationOptions
sslOptions, HttpRequestMessage request, bool async, Stream stream, CancellationToken cancellationToken)
at async ValueTask<ValueTuple<Socket, Stream, TransportContext>> System.Net.Http.HttpConnectionPool.ConnectAsync(
HttpRequestMessage request, bool async, CancellationToken cancellationToken)
at async ValueTask System.Net.Http.HttpConnectionPool.CreateHttp11ConnectionAsync(HttpRequestMessage
request, bool async, CancellationToken cancellationToken)
at async Task System.Net.Http.HttpConnectionPool.AddHttp11ConnectionAsync(HttpRequestMessage request)
at async ValueTask System.Threading.Tasks.TaskCompletionSourceWithCancellation1.WaitWithCancellationAsync(      CancellationToken cancellationToken)   at async ValueTask<HttpConnection> System.Net.Http.HttpConnectionPool.GetHttp11ConnectionAsync(HttpRequestMessage      request, bool async, CancellationToken cancellationToken)   at async ValueTask<HttpResponseMessage> System.Net.Http.HttpConnectionPool.SendWithVersionDetectionAndRetryAsync(      HttpRequestMessage request, bool async, bool doRequestAuth, CancellationToken cancellationToken)   at async ValueTask<HttpResponseMessage> System.Net.Http.RedirectHandler.SendAsync(HttpRequestMessage request, bool      async, CancellationToken cancellationToken)   at async ValueTask<HttpResponseMessage> System.Net.Http.DecompressionHandler.SendAsync(HttpRequestMessage request,      bool async, CancellationToken cancellationToken)   at async Task<HttpResponseMessage> System.Net.Http.HttpClient.<SendAsync>g__Core|83_0(HttpRequestMessage request,      HttpCompletionOption completionOption, CancellationTokenSource cts, bool disposeCts, CancellationTokenSource      pendingRequestsCts, CancellationToken originalCancellationToken)   at async Task<TResult> Polly.Retry.AsyncRetryEngine.ImplementationAsync<TResult>(Func<Context, CancellationToken,      Task<TResult>> action, Context context, CancellationToken cancellationToken, ExceptionPredicates      shouldRetryExceptionPredicates, ResultPredicates<TResult> shouldRetryResultPredicates,      Func<DelegateResult<TResult>, TimeSpan, int, Context, Task> onRetryAsync, int permittedRetryCount,      IEnumerable<TimeSpan> sleepDurationsEnumerable, Func<int, DelegateResult<TResult>, Context, TimeSpan>      sleepDurationProvider, bool continueOnCapturedContext)   at async Task<TResult> Polly.AsyncPolicy1.ExecuteAsync(Func<Context, CancellationToken, Task> action,
Context context, CancellationToken cancellationToken, bool continueOnCapturedContext)
at async Task PwnedPasswordsDownloader.GetPwnedPasswordsRangeFromWeb(int i, bool fetchNtlm) in
//src/HaveIBeenPwned.PwnedPasswords.Downloader/Program.cs:208
at async Task PwnedPasswordsDownloader.ProcessRanges(Settings settings) in
//src/HaveIBeenPwned.PwnedPasswords.Downloader/Program.cs:250
at async Task PwnedPasswordsDownloader.ProcessRanges(Settings settings) in
/_/src/HaveIBeenPwned.PwnedPasswords.Downloader/Program.cs:247
Hash ranges downloaded ----------------------------------------   0% 00:00:00
Finished downloading all hash ranges in 876ms (1.14 hashes per second).
We made 0 Cloudflare requests (avg response time: NaNms). Of those, Cloudflare had already cached 0 requests, and made 0
requests to the Have I Been Pwned origin server.
What am I missing?

Same here.  A flood of error message and an empty pwnedpasswords_ntlm.txt file:
<snip>
Failed request #10 while fetching . Exception message: The requested address is not valid in its context.
(api.pwnedpasswords.com:443).----------------------------------   0% --:--:-- -
Failed request #10 while fetching . Exception message: The requested address is not valid in its context.
(api.pwnedpasswords.com:443).----------------------------------   0% --:--:-- -
Failed request #10 while fetching . Exception message: The requested address is not valid in its context.
(api.pwnedpasswords.com:443).----------------------------------   0% --:--:-- \
Failed request #10 while fetching . Exception message: The requested address is not valid in its context.
(api.pwnedpasswords.com:443).----------------------------------   0% --:--:-- \
Failed request #10 while fetching . Exception message: The requested address is not valid in its context.
(api.pwnedpasswords.com:443).----------------------------------   0% --:--:-- |
System.Net.Http.HttpRequestException: The requested address is not valid in its context. (api.pwnedpasswords.com:443)
     System.Net.Sockets.SocketException: The requested address is not valid in its context.
       at void System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.ThrowException(SocketError error,
          CancellationToken cancellationToken)
       at void System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.System.Threading.Tasks.Sources.IValueTaskSource.
          GetResult(short token)
       at async ValueTask System.Net.Sockets.Socket.<ConnectAsync>g__WaitForConnectWithCancellation|277_0(
          AwaitableSocketAsyncEventArgs saea, ValueTask connectTask, CancellationToken cancellationToken)
       at async ValueTask<ValueTuple<Socket, Stream>> System.Net.Http.HttpConnectionPool.ConnectToTcpHostAsync(string
          host, int port, HttpRequestMessage initialRequest, bool async, CancellationToken cancellationToken)
  at async ValueTask<ValueTuple<Socket, Stream>> System.Net.Http.HttpConnectionPool.ConnectToTcpHostAsync(string host,
     int port, HttpRequestMessage initialRequest, bool async, CancellationToken cancellationToken)
  at async ValueTask<ValueTuple<Socket, Stream, TransportContext>> System.Net.Http.HttpConnectionPool.ConnectAsync(
     HttpRequestMessage request, bool async, CancellationToken cancellationToken)
  at async Task System.Net.Http.HttpConnectionPool.AddHttp2ConnectionAsync(HttpRequestMessage request)
  at async ValueTask<T> System.Threading.Tasks.TaskCompletionSourceWithCancellation`1.WaitWithCancellationAsync(
     CancellationToken cancellationToken)
  at async ValueTask<Http2Connection> System.Net.Http.HttpConnectionPool.GetHttp2ConnectionAsync(HttpRequestMessage
     request, bool async, CancellationToken cancellationToken)
  at async ValueTask<HttpResponseMessage> System.Net.Http.HttpConnectionPool.SendWithVersionDetectionAndRetryAsync(
     HttpRequestMessage request, bool async, bool doRequestAuth, CancellationToken cancellationToken)
  at async ValueTask<HttpResponseMessage> System.Net.Http.RedirectHandler.SendAsync(HttpRequestMessage request, bool
     async, CancellationToken cancellationToken)
  at async ValueTask<HttpResponseMessage> System.Net.Http.DecompressionHandler.SendAsync(HttpRequestMessage request,
     bool async, CancellationToken cancellationToken)
  at async Task<HttpResponseMessage> System.Net.Http.HttpClient.<SendAsync>g__Core|83_0(HttpRequestMessage request,
     HttpCompletionOption completionOption, CancellationTokenSource cts, bool disposeCts, CancellationTokenSource
     pendingRequestsCts, CancellationToken originalCancellationToken)
  at async Task<TResult> Polly.Retry.AsyncRetryEngine.ImplementationAsync<TResult>(Func<Context, CancellationToken,
     Task<TResult>> action, Context context, CancellationToken cancellationToken, ExceptionPredicates
     shouldRetryExceptionPredicates, ResultPredicates<TResult> shouldRetryResultPredicates,
     Func<DelegateResult<TResult>, TimeSpan, int, Context, Task> onRetryAsync, int permittedRetryCount,
     IEnumerable<TimeSpan> sleepDurationsEnumerable, Func<int, DelegateResult<TResult>, Context, TimeSpan>
     sleepDurationProvider, bool continueOnCapturedContext)
  at async Task<TResult> Polly.AsyncPolicy`1.ExecuteAsync(Func<Context, CancellationToken, Task<TResult>> action,
     Context context, CancellationToken cancellationToken, bool continueOnCapturedContext)
  at async Task<Stream> PwnedPasswordsDownloader.GetPwnedPasswordsRangeFromWeb(int i, bool fetchNtlm) in
     /_/src/HaveIBeenPwned.PwnedPasswords.Downloader/Program.cs:187
  at async Task PwnedPasswordsDownloader.ProcessRanges(Settings settings) in
     /_/src/HaveIBeenPwned.PwnedPasswords.Downloader/Program.cs:227
  at async Task PwnedPasswordsDownloader.ProcessRanges(Settings settings) in
     /_/src/HaveIBeenPwned.PwnedPasswords.Downloader/Program.cs:224
  at async Task PwnedPasswordsDownloader.ProcessRanges(Settings settings) in
     /_/src/HaveIBeenPwned.PwnedPasswords.Downloader/Program.cs:240
  at async Task PwnedPasswordsDownloader.ProcessRanges(Settings settings) in
     /_/src/HaveIBeenPwned.PwnedPasswords.Downloader/Program.cs:240

Hash ranges downloaded ----------------------------------------   0% 00:00:00

Finished downloading all hash ranges in 48,398ms (0.02 hashes per second).
We made 0 Cloudflare requests (avg response time: NaNms). Of those, Cloudflare had already cached 0 requests, and made 0requests to the Have I Been Pwned origin server.
PS C:\Users\adebruyn>

Try again using the latest version (0.3.12).

I ended up restarting the server after-hours and it worked perfectly after that.
I suspect it had something to do with the latest .NET Framework being installed and the installer not prompting for a restart.


5014c warning

Fixes #1231
@jenshnielsen

I think it would be good have a slightly more "luxurious" version of this check. I'm imagining something like having a function that can determine whether a given sequence is playable using a software sequencer (does it have illegal trigger waits, etc.) and another one that checks if a given sequence will trigger using the software sequencer. By implementing that, we will also be able to give very helpful warnings and error messages, along the lines of Error: trigger wait on element 3 is incompatible with software sequencer, but length (120 points < 250 points) of element 5 forces instrument to use software sequencer. Cannot proceed. Ultimately, we want to protect users and ourselves from ever having to figure out this stuff manually again.


Settings change apparently randomly

While scrolling up and down in Settings menu  offscreen settings change randomly making it impossible to save settings the way one wants. Huawei Honor 4C, Andr.4.4.2

Have you had this on any other app? The new Settings screen is using the Android compatibility library's AppCompatPreferenceActivity, so it would probably be the same with other apps.

First time I've seen this kind of behavior. :/ Should this AppCompat...-thing work on Android 4.4.2?

Yes, it should. It looks like #482 has a similar issue. It may just be a bug that has been fixed in recent versions of the support library.

Will there be a release in F-Droid with this patch in near future?

I don't control F-Droid releases or the content of them.


Implement repo.GetData Function

hi
the bellow  function is called but i don't know how and in witch directory i should implement the function?
var items = repo.GetData(out totalRecords,
options.GetFilterString("FirstName"),
options.GetFilterString("LastName"),
active,
options.GetLimitOffset(), options.GetLimitRowcount(),
sortColumn, options.SortDirection == SortDirection.Dsc);

help pleaseeeeeeeeeeee

hi @sortami ,
If you haven't figured it out already go to the following link in the repository and there is the implementation of the method,
[(https://github.com/joeharrison714/MVCGrid.Net/blob/master/MVCGridExample/Models/JobRepo.cs]
You can implement this method in your Repository class or in our Data Access Layer in general.


drizzle-orm: [Pg] Add char data type

Add missing char column data type to pg

@AlexandrLi thanks a lot for this PR!
Everything looks great, but some things are missing. It's our bad, as long as Contribution docs were not updated
Let me update update out Contribution docs and then explain what steps are missing in your PR. After that it will be merged and you will be mentioned in release notes
Before that, if you really need this type in your project, you can use custom-types docs to add char right away. Don't want to block you in using this feature

Thank you @AndriiSherman for quick response. I will align my PR once you'll update Contribution docs. I've created this PR because I've tried to use it in combination with drizzle-kit and introspect my existing db. In the result this type was not recognized. I've created issue in related repo https://github.com/drizzle-team/drizzle-kit-mirror/issues/9 maybe you'll be able to take a look too. Thank you for all your efforts, greatly appreciate!

@AlexandrLi contribution docs were updated a bit. Here are some things, that are missing in your PR

All commits should be signed. You can check docs on how to sign commits

You PR should contain additional tests for char data type. We have a list of tests in integration-test folder. I've added this info in CONTRIBUTION.md docs also. For your example you should either create additional pg-char.test.ts file and copy all before and after scripts and test how insert/update/delete and update are working with char symbol. Or you could add additional optional char field in pg.test.ts file for some of a table and create few test cases with insert/update/delete and update using char field

We are still in process of creating better docs, test structure and explanation on how to do things in our repo, but you are always free to join our Discord, where we have chat called contribution-questions where you can post any suggestions on improvement and we could guide you through contribution process to help making PR approved and merged
Hope this will help you

Thank you for the update @AndriiSherman. I will take a look as soon as I can.

Hello @AndriiSherman, I've added some test cases via signed off commit. Can you review, please?

@AlexandrLi everything looks great!
You have only 1 commit unverified
After you will verify this one - we are ready to approve and merge this PR
Next steps will be for us:

Сheck tests locally 1 more time
Add those changes to next release we are planning
Mention you in next release notes

I've messed up with signing. Sorry. I'll open new PR.

Here is the new one https://github.com/drizzle-team/drizzle-orm/pull/177
Sorry for inconvenience.


Lots of unused imports!

angular-flask / angular_flask / controllers.py
from flask import Flask, request, Response
from flask import render_template, url_for, redirect, send_from_directory
from flask import send_file, make_response, abort
from angular_flask import app
You only using about 1/2 of them.  For example "Flask" isn't needed since its imported from "from angular_flask import app".

Greetings! I'm a new contributor on Github and want to practice working in the open source community!  Any comments are welcome!
I've taken out the unused imports and updated formats to follow the PEP8 standard. Also, for my own machine the localhost is pointed to address <IP_ADDRESS> and thus I made the change in angular-lask/runserver.py.


Add cors permission

I hope this end point can be suitable for using for App.
It fixes issue #22

@mjzac thanks for reminding, I had forgot to add it . I have updated my PR now

@abdullatheef @mjzac  I have updated as required.
How can I add csrf excempt in Django rest framework?

@kurianbenoy We are not using rest framework here i think.
Either a middleware(for entire API) or may be decorator will help (specific APIs)

Closing the PR as I have updated a new PR #201


Add support for configuring and sending contexts to PagerDuty when triggering alerts.

When Alertmanager creates PagerDuty incidents it appends all of the alert data as custom details.  Links within the custom details are clickable, but they are not easily discoverable.  Alertmanager should support populating the Contexts element (from PagerDuty Event API V1 )  and Links/Images when PagerDuty Event API V2 supported.
This PR is still a work in progress, but the goal is to populate the Links section in the PagerDuty UI as shown below.

With this PR, how would one send one context per alert?

As is it won't.  Your comments in IRC showed that.  To make that work we could loop over all alerts and for each alert expand all context.  That should work, but it isn't really that graceful.
As an alternative, could we just allow the entire PagerDuty (or Webhook) request body to be a template?  Then I could have full control and do the looping via Range.

@stuartnelson3 Updated the description and provided an image of what the end goal in PagerDuty is.

@stuartnelson3

Is there something about the current configuration that makes you want to add the link context?
The link in the context is much more discoverable.

Is there a reason you just wanted link, and not image?
Ideally, this PR would support specifying images as well as the PagerDuty API supports that.


Added locale for Months in calendar display

Fixed:

Days not translated in the embed calendar display when using different locale
Months not translated in embed/codeblock calendar display when using different locale

Nice catch! I'll verify once I get home

looks good, thanks for the PR!


Incomplete symbol recognition for functions in javascript

Using "Go To Symbol..." (⇧⌘O) it seems to only recognize certain ways of declaring functions in javascript.

All three functions should show up when you search "searchable", right? Only "searchable" and "alsoSearchable" show up however.
In this case it looks like these cases work:

Anonymous function assigned as a property on and object when the object is instantiated.
Named function assigned as a property on an object after it's instantiated.

And this case doesn't work:

Anonymous function assigned as a property on an object after it's already been instantiated.

This works in Sublime Text and I'd love if it could work in VS Code as well as it's one of the main ways I navigate my javascript code (especially the thousands of lines of code for my game).

This issue is present in this case too:
When I have a fuction defined with
var testFunction = function() {}
Here is a screenshot with the issue:
http://imgur.com/wS66ROM

Can you please try to reproduce using the new Salsa JS infrastructure: https://code.visualstudio.com/updates/#_javascript-salsa-preview

Using Salsa JS on the same example as I wrote before I get IntelliSense knowing the function, but now when I try to go to symbol I got no suggestion ( the go to symbol ( @ ) is empty ).

Moving to salsa for investigation

This issue was moved to Microsoft/TypeScript#7003


unable to upload style

Maputnik version:v2.0.0
Browser:Firefox, Chrome
OS:macOS
Description of the bug:
We've used Maputnik for years to examine our custom styles, but just recently it has stopped working.
When we upload our custom style file, the map does not change to show the new style, even tho the left-hand panel shows the names of the layers from the new style
Steps to reproduce the behavior:

launch Chrome incognito (to be sure of a fresh start) and visit https://maplibre.org/maputnik. OSM Liberty is displayed by default
choose Open > Upload and select local json style file

expected: new style is displayed
actual: the layers in the left bar reflect the local json style, but OSM Liberty is still displayed on the map.
Style file or style URL:
I'm not able to share our style files, but the identical style files worked as recently as a few weeks ago, and are still working in our client apps with regular maplibre.
Screenshots:
video: attempt to upload style, from fresh state:
https://github.com/maplibre/maputnik/assets/5832047/089a4e37-f286-4265-8881-f073138ff983

Can you check if this is the same issue as: #871
i.e. if you refresh the browser after you load the style it will work?

When I refresh the browser after upload, the map looks like what I was expecting.
in #871 they went from an uploaded style to a built-in style and saw a merged result. When I go from the uploaded style (which I only get after refreshing) to the built-in style, the built-in style looks fine.
I was able to get into a merged state earlier...I took screenshots but have not been able get into this state again. following the steps in #871 has not produced it
map with merged state: square is OSM Liberty, rest is uploaded style

inspection with merged state: the source "omv: water" is from my style, the "openmaptiles: transportation" is from one of the built-in styles as my style has no source with that name

I don't mind if you want to close if you think its a duplicate.
If I'm able to replicate the merged state again I'll check and see if refreshing fixes it.

Closing as duplicate of #871.
I'll gladly reopen if this is not the same once the other issue is fixed.


merge tag 1.2.2 into master

Issue
NA
Description
Merge 1.2.2 into master

Gravitee.io Automatic Deployment
🚀 A prerelease version of this package has been published on Gravitee's private artifactory, you can:

use it directly by updating your project with version: 1.3.2-merge-1-2-2-SNAPSHOT
download it from Artifactory here

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

1.3.2
GitHub release

Your semantic-release bot :package::rocket:


EF Core

Excellent article. Can someone do the same for the Entity Core? I did the example in EF Core 3.1 code first and my data object does not showup in the object tree.

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

ID: b73b2e5a-9526-e664-a2da-556717d344e1
Version Independent ID: 93442b78-6869-139c-bdbc-9a1efa21b7a6
Content: Databinding with WPF - EF6
Content Source: entity-framework/ef6/fundamentals/databinding/wpf.md
Product: entity-framework
Technology: entity-framework-6
GitHub Login: @divega
Microsoft Alias: avickers

See https://github.com/dotnet/efcore/issues/20044 and https://docs.microsoft.com/en-us/ef/core/what-is-new/ef-core-5.0/plan#ef-core-platforms-experience


batch.gradle and iap.gradle does not exist.

Hi,
Steps :

Cloning the repo.
Open the project using Android Studio 3.0.1.

Grandle Sync error :
Error:Could not read script '/path/to/project/EasyBudget/Android/EasyBudget/app/batch.gradle' as it does not exist.
Are there something missing in the repo?
Thanks.

I am seeing this error too. Same version of Android Studio.

@feelinc, I looked at past issues and found out you must provide these files. See #2, #3.

Hi there,
You are indeed in charge of providing 3 files:

batch.gradle which contains the API keys for Batch SDK
iap.gradle which contains the Google Play license key
fabric.properties which contains the Fabric API data

All of those are under /app and a sample is provided for each one.
Good luck
PS: Feel free to investigate more than 3 seconds on your own before creating an issue on a repo. Looking at the error and going into the repository reported by the error would have been much more productive than waiting for a response here ;)

Another reminder: If you plan to copy the app or a part of the app for your own personal project, feel free to do so but 2 reminders:

Everything related to the EasyBudget name and brand is under copyright (includes the name, colors and images)
If you use any part of this code into your own app, you should include a mention according to the apache 2 license

Thanks for the info.
I was trying look for the suspected missing files using my editor file search, but not found, perhaps I'm typing them all wrong in the first place. So I might spent more than 3 seconds to search the files before creating this issue.

Also, I'm not sure what the reason you put the "Another reminder" here, because you can put it in the repo README if you really want to make very clear.
Of you don't want someone creating issue here, I think best if you close this repo.
PS : I had deleting this repo in my computer yesterday.

I'm closing this issue for now, no need to go further into it.
Good luck


examples_documentation.md

https://github.com/ICB-DCM/PESTO/blob/master/examples/examples_documentation.md currently does not list plotMultiStartsDiagnosis (probably none of the examples use that function)
also it states that all PESTO features are demonstrated in the Conversion reaction examples, which thereby evidently is not true (also features enabled via options.trace, options.save and options.tempsave are not demonstrated could be done via plotMultiStartsDiagnosis)

Maybe one little thing to that:
I'm currently working things out in a way, that basically all feature of PESTO are demonstrated in one of the three small examples:

conversion reaction
transfection
enzymatic catalysis
I think its not necessarily the best thing to demonstrate evrything in one example: This would mean ~40 figures popping up in the conversion reaction

We can break it into several files. All features should preferable be demonstrated on a rather fast running example. Doesn't have to be the conversion reaction, although I think it would be a good choice.

Definitely is a good choice, transfection is also fast.
I'm currently trying to figure out what's wrong with the catalysis...

I basically took care of this, now just waiting for this getting on the master branch


Ppp ugdg

Will be done in 1hr

Coverage decreased (-0.9%) to 82.519% when pulling 99a55b97404ed631e0c2af3e81ab4e023037e1f0 on junweiljw:PPP_UGDG into 52b4f78ead8d1bb6ac8fbeeeac4fc7c90c9c181e on CS2113-AY1819S1-W12-1:master.

Fast fingers Junwei


Should Only lockfile updates (ignore updates that require Pipfile changes) be the default option?

If I pin package with specific version in Pipfile as following, the dependabot will still open PR for updating that.
Will it be better to set only updating lockfile as the default option? since that is the most use cases.
I'm a new user of dependabot which is cool for me. Just to share a few thoughts. :D
[packages]
celery = "==3.1.25"

👋 @jxltom, and thanks for giving Dependabot a try!
A lot of users pin but still want updates, so we update both the Pipfile and the Pipfile.lock by default. There's a setting to only update the lockfile if you'd rather, or if you'd like Dependabot to ignore a specific dependency you can comment @dependabot ignore this dependency.
Keep the feedback coming!

OK, thanks for the reply. 😄


chore: auction results copy updates #trivial

The type of this PR is: Enhancement

This PR resolves CX-1065
Description
Make the following copy updates:

"Market Signals by Medium -> change to "Market Signals"
(In Info modal next to Market Signals) -> change last paragraph to
"In this data set, please note that the sale price includes the hammer price and buyer’s premium, as well as any other additional fees (e.g., Artist’s Resale Rights). The data set only includes works valued over $1,000."
"Yearly lots sold" -> change to "Average yearly lots sold"

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a CHANGELOG.yml entry or my changes do not require one.

I will merge this since it involves only small copy updates


Fuzz Wasmtime against the official Wasm spec interpreter

This change introduces a new fuzz target, differential_spec that compares the outputs of executing the first exported function of a Wasm module in Wasmtime and then in the official Wasm spec interpreter (using the wasm-spec-interpreter crate). This is an initial step towards more fully-featured fuzzing (e.g. compare memories, add v128, add references, add other proposals, etc.)

All, thanks for the comments. I've fixed most of the small stuff but I've left open the ones that I still need help on (how and when to build wasm-spec-interpreter, e.g.).

I rebased this on main and condensed it to five commits. I think this is almost good for a final review except for the issue I raised in https://github.com/bytecodealliance/wasmtime/pull/3124#discussion_r678631805 about unsafe behavior. Something is not right there and we should wait to merge this until it is resolved. I can make the fuzzer crash by running cargo +nightly fuzz run differential_spec locally:
INFO: Running with entropic power schedule (0xFF, 100).
INFO: Seed:<PHONE_NUMBER>
INFO: Loaded 1 modules   (699235 inline 8-bit counters): 699235 [0x5557756ba9da, 0x55577576553d),
INFO: Loaded 1 PC tables (699235 PCs): 699235 [0x555775765540,0x555776210b70),
INFO:     1425 files found in /home/abrown/Code/wasmtime/fuzz/corpus/differential_spec
INFO: -max_len is not provided; libFuzzer will not generate inputs larger than 4096 bytes
INFO: seed corpus: files: 1425 min: 1b max: 63b total: 52307b rss: 72Mb
#1024   pulse  cov: 16016 ft: 34393 corp: 418/12380b exec/s: 33 rss: 429Mb
=== Execution rate (572 executed modules / 1000 tried modules): 57.199999999999996% ===
#2048   pulse  cov: 17686 ft: 41583 corp: 838/28Kb exec/s: 28 rss: 429Mb
=== Execution rate (992 executed modules / 2000 tried modules): 49.6% ===

[...time passes]

#3227   NEW    cov: 18157 ft: 47066 corp: 1060/39Kb lim: 68 exec/s: 35 rss: 429Mb L: 47/68 MS: 1 CrossOver-
#3268   NEW    cov: 18157 ft: 47074 corp: 1061/39Kb lim: 68 exec/s: 35 rss: 429Mb L: 48/68 MS: 1 CrossOver-
#3309   NEW    cov: 18158 ft: 47076 corp: 1062/39Kb lim: 68 exec/s: 36 rss: 429Mb L: 64/68 MS: 1 InsertRepeatedBytes-
#3346   NEW    cov: 18158 ft: 47077 corp: 1063/39Kb lim: 68 exec/s: 36 rss: 429Mb L: 38/68 MS: 2 ChangeByte-InsertByte-
#3397   NEW    cov: 18158 ft: 47079 corp: 1064/39Kb lim: 68 exec/s: 37 rss: 429Mb L: 36/68 MS: 1 ChangeASCIIInt-
#3445   NEW    cov: 18158 ft: 47108 corp: 1065/39Kb lim: 68 exec/s: 37 rss: 429Mb L: 41/68 MS: 3 ChangeASCIIInt-InsertByte-CopyPart-
#3452   NEW    cov: 18158 ft: 47113 corp: 1066/39Kb lim: 68 exec/s: 37 rss: 429Mb L: 58/68 MS: 2 InsertRepeatedBytes-EraseBytes-
────────────────────────────────────────────────────────────────────────────────

Error: Fuzz target exited with signal: 11

No core dump is associated with this crash so it's hard to tell what is going on. I would appreciate someone else trying to run this, or even any suggestions on how to narrow down what is going wrong.

I tried again to get this working locally, and I believe my previous build error was due to an older ocamlc in PATH than what was needed here (according to The Internet). I figured out how to wrangle opam and such and I got things working with 4.12.0.
I got a different error than you did initially, and it's behavior I've never seen before. The fuzzer would just stop after a few hundreds runs and print that it detected a leak:

=================================================================
==44138==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 32 byte(s) in 1 object(s) allocated from:
    #0 0xaaaaac28c01c in calloc /checkout/src/llvm-project/compiler-rt/lib/asan/asan_malloc_linux.cpp:154
    #1 0xffffb372e3f0 in __cxa_thread_atexit_impl /build/glibc-pfQmxL/glibc-2.28/stdlib/cxa_thread_atexit_impl.c:106:27
    #2 0xaaaab0a73074 in std::sys::unix::thread_local_dtor::register_dtor::h901a681bd6e662af /rustc/25b764849625cb090e8b81d12d2bb2295d073788/library/std/src/sys/unix/thread_local_dtor.rs:36:9
    #3 0xaaaaaea5e224 in std::thread::local::LocalKey$LT$T$GT$::with::h9f753ffa090280d6 (/home/acrichto/code/wasmtime/target/aarch64-unknown-linux-gnu/release/differential_spec+0x3df3224)
    #4 0xaaaaaeb0f3dc in wasmtime_runtime::traphandlers::tls::raw::replace::h66677875f2db8192 (/home/acrichto/code/wasmtime/target/aarch64-unknown-linux-gnu/release/differential_spec+0x3ea43dc)
    #5 0xaaaaac512d60 in wasmtime_runtime::traphandlers::tls::set::h4626657d9defd11e (/home/acrichto/code/wasmtime/target/aarch64-unknown-linux-gnu/release/differential_spec+0x18a7d60)
    #6 0xaaaaac676480 in wasmtime::func::invoke_wasm_and_catch_traps::hd21244aafcd451e9 (/home/acrichto/code/wasmtime/target/aarch64-unknown-linux-gnu/release/differential_spec+0x1a0b480)
    #7 0xaaaaac679638 in wasmtime::func::Func::call_impl::hb80934ccfb58865f (/home/acrichto/code/wasmtime/target/aarch64-unknown-linux-gnu/release/differential_spec+0x1a0e638)
    #8 0xaaaaac717d64 in wasmtime_fuzzing::oracles::run_in_wasmtime::hb0614ee0e39ac4f8 (/home/acrichto/code/wasmtime/target/aarch64-unknown-linux-gnu/release/differential_spec+0x1aacd64)
    #9 0xaaaaac715330 in wasmtime_fuzzing::oracles::differential_spec_execution::h71bb42d8a0672244 (/home/acrichto/code/wasmtime/target/aarch64-unknown-linux-gnu/release/differential_spec+0x1aaa330)
    #10 0xaaaaac4ad54c in rust_fuzzer_test_input (/home/acrichto/code/wasmtime/target/aarch64-unknown-linux-gnu/release/differential_spec+0x184254c)
    #11 0xaaaab09d2d50 in __rust_try (/home/acrichto/code/wasmtime/target/aarch64-unknown-linux-gnu/release/differential_spec+0x5d67d50)
    #12 0xaaaab09d2428 in LLVMFuzzerTestOneInput (/home/acrichto/code/wasmtime/target/aarch64-unknown-linux-gnu/release/differential_spec+0x5d67428)
    #13 0xaaaab09d7534 in fuzzer::Fuzzer::ExecuteCallback(unsigned char const*, unsigned long) (/home/acrichto/code/wasmtime/target/aarch64-unknown-linux-gnu/release/differential_spec+0x5d6c534)
    #14 0xaaaab09db6d4 in fuzzer::Fuzzer::RunOne(unsigned char const*, unsigned long, bool, fuzzer::InputInfo*, bool, bool*) (/home/acrichto/code/wasmtime/target/aarch64-unknown-linux-gnu/release/differential_spec+0x5d706d4)
    #15 0xaaaab09dd4b0 in fuzzer::Fuzzer::MutateAndTestOne() (/home/acrichto/code/wasmtime/target/aarch64-unknown-linux-gnu/release/differential_spec+0x5d724b0)
    #16 0xaaaab09e0bbc in fuzzer::Fuzzer::Loop(std::vector<fuzzer::SizedFile, fuzzer::fuzzer_allocator<fuzzer::SizedFile> >&) (/home/acrichto/code/wasmtime/target/aarch64-unknown-linux-gnu/release/differential_spec+0x5d75bbc)
    #17 0xaaaab09f8974 in fuzzer::FuzzerDriver(int*, char***, int (*)(unsigned char const*, unsigned long)) (/home/acrichto/code/wasmtime/target/aarch64-unknown-linux-gnu/release/differential_spec+0x5d8d974)
    #18 0xaaaaac202930 in main (/home/acrichto/code/wasmtime/target/aarch64-unknown-linux-gnu/release/differential_spec+0x1597930)
    #19 0xffffb3719d20 in __libc_start_main /build/glibc-pfQmxL/glibc-2.28/csu/../csu/libc-start.c:308:16
    #20 0xaaaaac202a5c in _start (/home/acrichto/code/wasmtime/target/aarch64-unknown-linux-gnu/release/differential_spec+0x1597a5c)

SUMMARY: AddressSanitizer: 32 byte(s) leaked in 1 allocation(s).
INFO: to ignore leaks on libFuzzer side use -detect_leaks=0.

MS: 1 CopyPart-; base unit: 5080fd9c0bf811fe129fd0db4e1158b53926333c
0xa,0xf4,0xf4,0xf4,0xf4,0xf4,0xf4,0xf4,0xf4,0xf4,0xf4,0xf4,0xf4,0xf4,0xf4,0xf4,0xf4,0xf4,0xf4,0xf4,0xf4,0xff,0xf4,0xf4,0xf4,
\x0a\xf4\xf4\xf4\xf4\xf4\xf4\xf4\xf4\xf4\xf4\xf4\xf4\xf4\xf4\xf4\xf4\xf4\xf4\xf4\xf4\xff\xf4\xf4\xf4
artifact_prefix='./'; Test unit written to ./leak-8ce318eb459e74f96cb3214962087900c5ce2cea
Base64: CvT09PT09PT09PT09PT09PT09PT0//T09A==

This doesn't really have anything to do with Wasmtime, it's just standard registration of a TLS destructor... Also I'm not sure why the fuzzer would just halt after a bunch of runs and it feels like it spuriously detects the leak. Unsure!
Anyway after I passed -detect_leaks=0 to the fuzzer it so far hasn't crashed yet. Right now I'm at
=== Execution rate (124756 executed modules / 256000 tried modules): 48.7328125% ===

I'm also on an AArch64 machine, which may make a difference. @abrown what version of ocaml are you using? Perhaps you also need to update to 4.12.0 and they fixed something in the meantime?

Good to see you got things running! I also have have version 4.12.0 of ocamlc and ocamlopt.
Since this morning I tried to:

run the fuzz target in LLDB but that eventually fails with LeakSanitizer does not work under ptrace (strace, gdb, etc)
run the fuzz target in Valgrind but that immediately fails with Shadow memory range interleaves with an existing memory mapping. ASan cannot proceed correctly. ABORTING. (I think libFuzzer's ASAN and Valgrind's ASAN and interfering?)
force OCaml's Gc.compact on every invocation of the spec interpreter--still crashes
guard invocations of the spec interpreter with a mutex--still crashes
avoid detecting leaks, like you did (e.g. cargo +nightly fuzz run differential_spec -- -detect_leaks=0)--still crashes

I guess I should try this out on another machine in case there is something specific about a compiler, library, etc. that is local to mine. In any case, @cfallin and I were going to get together to look at this tomorrow afternoon and it would be great fun (😁) if you wanted to join.

Sure I'm happy to help out!

The latest rebased commits happily fuzz away on my x86_64 Linux machine, where previously the fuzzer would crash after ~2k-4k cases:
$ cargo +nightly fuzz run differential_spec
    Finished release [optimized] target(s) in 0.10s
     Running `/home/abrown/Code/wasmtime/target/x86_64-unknown-linux-gnu/release/differential_spec -artifact_prefix=/home/abrown/Code/wasmtime/fuzz/artifacts/differential_spec/ /home/abrown/Code/wasmtime/fuzz/corpus/differential_spec`
INFO: Running with entropic power schedule (0xFF, 100).
INFO: Seed: 62986658
INFO: Loaded 1 modules   (720350 inline 8-bit counters): 720350 [0x558bd3a2529a, 0x558bd3ad5078),
INFO: Loaded 1 PC tables (720350 PCs): 720350 [0x558bd3ad5078,0x558bd45d2e58),
INFO:     7169 files found in /home/abrown/Code/wasmtime/fuzz/corpus/differential_spec
INFO: -max_len is not provided; libFuzzer will not generate inputs larger than 4096 bytes
INFO: seed corpus: files: 7169 min: 1b max: 180b total: 725327b rss: 79Mb
#512    pulse  cov: 12621 ft: 21502 corp: 178/4261b exec/s: 51 rss: 301Mb
...
#22307  NEW    cov: 22079 ft: 93916 corp: 5068/507Kb lim: 187 exec/s: 52 rss: 649Mb L: 111/187 MS: 4 CopyPart-CMP-ChangeBit-CrossOver- DE: "\xff\xff\xff\xff\xff\xff\xff("-
#22438  REDUCE cov: 22079 ft: 93916 corp: 5068/507Kb lim: 187 exec/s: 52 rss: 649Mb L: 127/187 MS: 1 EraseBytes-
#22580  NEW    cov: 22079 ft: 93918 corp: 5069/508Kb lim: 187 exec/s: 52 rss: 649Mb L: 180/187 MS: 2 ChangeBinInt-ChangeByte-
#22612  REDUCE cov: 22079 ft: 93918 corp: 5069/508Kb lim: 187 exec/s: 52 rss: 649Mb L: 119/187 MS: 2 CopyPart-EraseBytes-
=== Execution rate (5510 executed modules / 20000 tried modules): 27.55% ===

The ratio of executable modules steadily decreases but that's a separate problem so I'm going to mark this ready for review.


FOLLOW THROUGH: Fix filters UI [2]

The UI for the filters is needs to be brought in line with PatternFly and is currently residing in two areas within the layout of the Planner Page.
VC's

the filter follows the PatternFly design
the filter takes into consideration the layout of other elements on that area of the page.

The pre-defined filters wireframe has been updated to properly reflect PatternFly's UI. The link can be found here. I will create an Enhancement request in the Fabric8-Planner area for updating the UI with this new wireframe.


Disqus comment not unique to each blog article

Howdy, so I found another bug outside of the disqus comment count not working, but it seems the comments are universal. So I created a comment on one blog article and that thread now shows on all blog articles
https://www.orren-life.com/journal
That is a link to our blog, no matter which article you go into it will all show the same comments

I have fixed this, turns out on the disqus.tpl, teh unique identifier kept showing as 0. This has now been corrected by changing it the the article ID


Add tests for CharSeq.{map,mapChars}

These two simple tests validate if the types returned by CharSeq.map & CharSeq.mapChars,
are those which we expect.

Great, many thanks!
It is confusing that Vector.equals checks for instanceof IndexedSeq. Could you chabge the check to Vector there?

it's not that simple, as this will narrow the types with which Vector can be compared to only Vector itself. So it won't be possible to compare it with for example CharSeq - which as a result will brake some tests.
I've found out that there might be some incosistencies in them, as in this test:
@Test
public void shouldMapNil() {
    assertThat(this.<Integer> empty().map(i -> i + 1)).isEqualTo(empty());
}

a result of CharSeq.map is compared against CharSeq, which is incorrect as CharSeq.map returns Vector. I'll try to review the failing tests and fix them.

yes, the behavior should be consistent. Vector should not be equal to CharSeq in any case.

I'm almost done - maybe it would be best to do it in another PR, to keep it atomic ?

I was lucky - 500th PR :) https://github.com/javaslang/javaslang/pull/500

:))

I rebase the code in this PR to current master, so it can be merged without any conflicts.

roger that!


Cannot read property 'toString' of null

🐞 Bug report
I'm trying to add scullyio to empty Angular(9.1.7) project. And found an error as title.
Description
🔬 Minimal Reproduction
ng new hello_scully --create-application=false
ng add @scullyio/init
Installing packages for tooling via yarn.
Installed packages for tooling via yarn.
    error into import SculyLib
Cannot read property 'toString' of null
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

💻Your Environment
Angular Version:

Angular CLI: 9.1.12
Node: 14.5.0
OS: linux x64

Angular: 9.1.12
... animations, cli, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router
Ivy Workspace: Yes

Package                      Version
------------------------------------------------------
@angular-devkit/architect    0.901.12
@angular-devkit/core         9.1.12
@angular-devkit/schematics   9.1.12
@schematics/angular          9.1.12
@schematics/update           0.901.12
rxjs                         6.5.5
typescript                   3.8.3

Done in 0.47s.

Scully Version:

"@scullyio/init": "^1.0.0-beta.0",

🔥 Exception or Error

    error into import SculyLib
Cannot read property 'toString' of null

... and I resolved it after create Angular application.
it may be better that scully logs some message instead of "Cannot read property 'toString' of null" :)
ng g application app --routing=false --style=scss
ng add @scullyio/init
# => success

I will add a new error message, Scully only work with an Angular app using the route module!


flask==1.1.1 breaks with with werkzeug==2.1.0

A recent update to the werkzeug code is breaking flask during server startup
Since flask==1.1.1 does not pin python requirement versions, it gets installed against the latest werkzeug==2.1.0
In the latest werkzeug==2.1.0, the response classes have been refactored and the BaseResponse class is no longer availab.e

Install flask==1.1.1
The runserver fails with an import error due to recent changes to werkzeug
Traceback (most recent call last):
  File "/usr/local/bin/moto_server", line 5, in <module>
    from moto.server import main
  File "/usr/local/lib/python3.8/dist-packages/moto/server.py", line 10, in <module>
    from flask import Flask
  File "/usr/local/lib/python3.8/dist-packages/flask/__init__.py", line 21, in <module>
    from .app import Flask
  File "/usr/local/lib/python3.8/dist-packages/flask/app.py", line 32, in <module>
    from werkzeug.wrappers import BaseResponse
ImportError: cannot import name 'BaseResponse' from 'werkzeug.wrappers' (/usr/local/lib/python3.8/dist-packages/werkzeug/wrappers/__init__.py)

Environment:

Python version: 2.8
Flask version: 2.1.1

You are using an unsupported version of the project, please update to the latest version if possible. Additionally, use a tool like pip-tools to pin your dependencies and control when you get updates. Be sure to run your tests with deprecation warnings treated as errors so that you get notified of these types of changes early.


Hi,every Good Share project should need a README,or a Tutorial

Hi,every Good Share project should need a README,or a Tutorial

thanks


Koji's custom kernel DGEMM_Aldebaran_PKFixedAtomic512Latest

Have you verified this kernel?
Did all tests pass?

For the 3 sizes using this custom kernel, they validated in Tensile.  rocblas-test passed fully (quick, pre_checkin and nightly).

I respect the author's choice of kernel name, hence keeping the "Latest" suffix.  We don't care about the color of the bike shed, do we?  If you insist, I'll rename it.


refine InterpreterDAG

What problem does this PR solve?
Issue Number: ref #4739
Problem Summary:
What is changed and how it works?
move some codes from InterpreterDAG to InterpreterUtils
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/merge


Debug resolving can only happen once, resulting in missed actions

In VSCode's implementation of DebugConfigurationProviders, resolveDebugConfiguration will only be called for the first provider. If a provider resolves into another--for example, this is how the Docker extension handles .NET and Python debugging--then the resolvers for the other are never called.
In the case of the Docker extension, we have a debug configuration type of docker. Depending on platform, this will resolve into coreclr (.NET), node (Node.js), or python (Python of course). Each of these supports some form of remote debugging which is how container debugging is done, but all are different, so they each need special platform-specific code in the Docker extension to resolve.
In the case of Python, we run our Docker-specific resolving code, and then there is additional code we expected the Python resolver to run--however, the Python resolver never gets called. Instead, it is handed directly to the Python debug adapter. As a result, there isn't a great way to fix https://github.com/microsoft/vscode-docker/issues/2455. The options are to either duplicate the logic that the Python resolver is doing, or otherwise ignore some user configuration in order to do it more simply. Neither is really ideal. What we'd prefer is for the Docker DebugConfigurationProvider to resolve the configuration into a python one, and then the Python DebugConfigurationProvider would also have a chance to run. I'm not entirely sure how VSCode would decide that it is done--perhaps if the type does not change between start and end of the resolve call--but there must be some way.
@int19h FYI

This applies to both resolveDebugConfiguration and resolveDebugConfigurationWithSubstitutedVariables - we utilize both.
More broadly speaking, the problem is that the resolvers are only picked once based on the initial value of "type", but then the debug adapter factory is picked based on the final value. Thus, it's possible to get a config in the factory for a given type that has not passed through the corresponding resolver. If the factory relies on something done by the resolver, things break. We've always been assuming that if the extension registers both resolver and factory for the same type, they're always guaranteed to be called in order; not sure if the actual behavior is intentional, or changing "type" was not anticipated; note, though, that the docs don't place any constraints on how the resolver can change the config.

We are encountering this as well - it seems like there is a conflict where our debug resolver does not get called. Uninstalling other extensions causes it to work again. It started working when we uninstalled the docker extension. Given this thread, I'm not sure it is just the docker extension causing issues. I'd be happy to help investigate.

I have a fix for this in https://github.com/microsoft/vscode/pull/143054 just waiting for someone to review.

Thanks for the ping, looking at that


Update readme and minor changes

I try to use this tap to get data from bigquery, but have to fix several issues, so doing backmerge PR.

Fix readme.
Add output of state message to be compatible with any target.
Fix lost require lib in setup script.

@sphinks Thanks for improving the code and docs! Looking good.

@sphinks Can you review this (breaking change v0.3.0)
https://github.com/anelendata/tap_bigquery/pull/3
I also wanted to discuss the release on PyPi since you did that with the previous version. Thx!

@daigotanaka I'm not really good with Python, but I do not notice any issues. Also, I'm not sure I can be mentioned as author yet (just fix couple issues:)), I suppose contributor is much close to the truth :)
Do you want me to release it on PyPi?

@sphinks got it about contributor. Can you add me (daigotanaka) as a collaborator at https://pypi.org/manage/project/tap-bigquery/collaboration so I don't have to contact you every time I need to release? Thx!

@daigotanaka done, you can use it.

It is make sense!


Incomplete Output Functionality

The emulator requires the input file to include a value for idtl, which determines the output level. Currently, only 0 (standard) and 2 (full) are supported. Level 5 (full with text descriptions) is not implemented.

Could you make the issue title more specific?


ServiceLoader is not instantiating the command handler classes

@guilhermegps
While trying to implement NIP-42, I noticed some limitations with the current code.
NIP-42 expects the client to reply with a signed auth event when receiving an AUTH-message from a relay. Currently, this takes place in the onAuth method here.
However, to sign the event, you need to have access to the private key, which is not available in that class at the moment.
I have therefore introduced the concept of Context classes, where I store contextual information, including the private key. The context is created in the api module when instantiating a client object, passed along to the other modules, and is now available in the OnAuth command handler.
Additional major changes  (Branch 0.6-SNAPSHOT):

I noticed that for every request to a relay, we would open a new connection. This is not good. I have now created a ConnectionPool to keep track of the connections, and ensure that we only have one connection per relay.
One other change I have made is that I now have, for each relay command, e.g. Ok, Eose, Auth, Closed etc, a separate handler class, instead of a unique command handler class with a method for each command.
You will notice I have also renamed, added, removed a few modules. Hopefully this is more intuitive and cleaner.
etc.

I welcome your opinions and suggestions on this.
NOW, the Problem I am facing, is that the ServiceLoader is not instantiating the handler classes (See here).
I need a second pair of eyes to review and hopefully find what is missing, and why it's not working.
Could you please have a look when you have a moment?
Thanks!

Ok, I will take a look and answer you soon.

I'd like to simulate your problem here. How can I do this?
What is the test that you are trying?
What is the branch that I need to look? 0.6-SNAPSHOT ?
About your points:

Nice, good change. This is a really problem that we have.

Agree

I'd like to say that we need to have caution with the module changes, because this kind of change could broke application that is using an oldest version and decide to upgrade for this new version. Said it, I need to agree that modules more intuitive are better.

When you receive a message from a relay (ok, eose, notice etc) we use the service locator to instantiate the corresponding command handler class (default or custom) and run its handle method. This doesn't work, the service locator doesn't find the handler classes. This is what I'm struggling with at the moment. It seems so obvious but I can't find where the problem is.
Agree re. caution with module changes but I saw these modifications as necessary. For clients, this would hopefully not be too much to change, just a bit of refactoring, changing their custom handlers, and create separate classes for each instead.
Thanks.

@tcheeric
I found a problem with this change, the example of "filters"  broke, it's not necessary to sign a request for search events, they are public.

Well, I'm getting a problem when try to connect with relays. When happened any problem during the connection with relay, I get an error here  and all connections with all relays stop and no event is sent.
abr. 20, 2024 11:03:41 AM jdk.internal.net.http.HttpClientImpl$SelectorManager run
FINE: [HttpClient-1-SelectorManager] [96ms] HttpClientImpl(1) Next deadline is 3000
abr. 20, 2024 11:03:41 AM nostr.examples.NostrApiExamples lambda$0
SEVERE: java.util.concurrent.ExecutionException: java.lang.RuntimeException: Failed to connect to Relay(uri=nostr.zebedee.cloud)
java.lang.RuntimeException: java.util.concurrent.ExecutionException: java.lang.RuntimeException: Failed to connect to Relay(uri=nostr.zebedee.cloud)
	at nostr.util/nostr.util.thread.ThreadUtil.run(ThreadUtil.java:51)
	at nostr.client/nostr.client.Client.connect(Client.java:52)
	at nostr.api/nostr.api.Nostr.send(Nostr.java:145)
	at nostr.api/nostr.api.Nostr.send(Nostr.java:113)
	at nostr.api/nostr.api.EventNostr.send(EventNostr.java:46)
	at nostr.examples/nostr.examples.NostrApiExamples.sendTextNoteEvent(NostrApiExamples.java:196)
	at nostr.examples/nostr.examples.NostrApiExamples.lambda$0(NostrApiExamples.java:99)
	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:577)
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:317)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1144)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:642)
	at java.base/java.lang.Thread.run(Thread.java:1589)
Caused by: java.util.concurrent.ExecutionException: java.lang.RuntimeException: Failed to connect to Relay(uri=nostr.zebedee.cloud)
	at java.base/java.util.concurrent.FutureTask.report(FutureTask.java:122)
	at java.base/java.util.concurrent.FutureTask.get(FutureTask.java:205)
	at nostr.util/nostr.util.thread.ThreadUtil.run(ThreadUtil.java:48)
	... 11 more
Caused by: java.lang.RuntimeException: Failed to connect to Relay(uri=nostr.zebedee.cloud)
	at nostr.connection/nostr.connection.impl.ConnectionImpl.connect(ConnectionImpl.java:60)
	at java.base/java.lang.Iterable.forEach(Iterable.java:75)
	at nostr.connection/nostr.connection.impl.ConnectionPool.connect(ConnectionPool.java:40)
	at nostr.client/nostr.client.Client$RelayConnectionTask.execute(Client.java:90)
	at nostr.client/nostr.client.Client$RelayConnectionTask.execute(Client.java:1)
	at nostr.util/nostr.util.thread.ThreadUtil.lambda$0(ThreadUtil.java:42)
	... 5 more

abr. 20, 2024 11:03:41 AM nostr.examples.NostrApiExamples main
FINE: ================== The End

@tcheeric
I found a problem with this change, the example of "filters" broke, it's not necessary to sign a request for search events, they are public.

Yes, correct.

@tcheeric
I found a problem with this change, the example of "filters" broke, it's not necessary to sign a request for search events, they are public.

Yes, correct.

So, I will let you check this problem and fix it.
You can test it with this example.

NOW, the Problem I am facing, is that the ServiceLoader is not instantiating the handler classes (See here).

About this problem, I observed that you instanced the listener in ConnectionImpl as an OpenListener, but how the WebSocket will know when call TextListener, CloseListener or whatever? It's not clear for me. I did a teste and got sucess just changed OpenListener to TextListener.
Log:
abr. 21, 2024 12:14:32 PM jdk.internal.net.http.HttpClientImpl$SelectorManager run
FINE: [HttpClient-1-SelectorManager] [15s 778ms] HttpClientImpl(1) Next deadline is 3000
abr. 21, 2024 12:14:34 PM nostr.connection.impl.WebsocketClientListeners$TextListener handleReceivedText
INFO: Received message ["OK","6d83330b8d77b5868589f247015f658da1a95eed93f5d817b486cc9ee07cfa22",true,""] from Relay(uri=nostr.mom)
abr. 21, 2024 12:14:34 PM nostr.connection.impl.WebsocketClientListeners$TextListener handleReceivedText
INFO: Creating the command context with message OkMessage(eventId=6d83330b8d77b5868589f247015f658da1a95eed93f5d817b486cc9ee07cfa22, flag=true, message=)
abr. 21, 2024 12:14:34 PM jdk.internal.net.http.websocket.WebSocketImpl$ReceiveTask processText
FINE: [HttpClient-1-Worker-0] [18s 383ms] [WebSocket] exit onText 2 returned java.util.concurrent.CompletableFuture@279f774e[Completed exceptionally: java.util.concurrent.CompletionException: java.lang.NullPointerException: Cannot invoke "nostr.context.impl.DefaultRequestContext.getPrivateKey()" because "this.context" is null]
abr. 21, 2024 12:14:34 PM jdk.internal.net.http.websocket.WebSocketImpl tryChangeState
FINE: [HttpClient-1-Worker-0] [18s 383ms] [WebSocket] change state from TEXT to IDLE true
abr. 21, 2024 12:14:34 PM jdk.internal.net.http.websocket.WebSocketImpl$ReceiveTask run
FINE: [HttpClient-1-Worker-0] [18s 383ms] [WebSocket] receive state: IDLE
abr. 21, 2024 12:14:34 PM jdk.internal.net.http.websocket.WebSocketImpl tryChangeState
FINE: [HttpClient-1-Worker-0] [18s 383ms] [WebSocket] change state from IDLE to WAITING true
abr. 21, 2024 12:14:34 PM jdk.internal.net.http.websocket.TransportImpl request

I observed either that the pool of connection is a little unstable. Maybe it was caused by the timeout, I don't know, but some times no connections are successfully established. We need to analyze carefully this pool and to be right that it's know under of control.

@tcheeric
I found a problem with this change, the example of "filters" broke, it's not necessary to sign a request for search events, they are public.

Yes, correct.

So, I will let you check this problem and fix it.
You can test it with this example.

Alright, will do.

NOW, the Problem I am facing, is that the ServiceLoader is not instantiating the handler classes (See here).

About this problem, I observed that you instanced the listener in ConnectionImpl as an OpenListener, but how the WebSocket will know when call TextListener, CloseListener or whatever? It's not clear for me. I did a test and got success just changing OpenListener to TextListener.
Log:
abr. 21, 2024 12:14:32 PM jdk.internal.net.http.HttpClientImpl$SelectorManager run
FINE: [HttpClient-1-SelectorManager] [15s 778ms] HttpClientImpl(1) Next deadline is 3000
abr. 21, 2024 12:14:34 PM nostr.connection.impl.WebsocketClientListeners$TextListener handleReceivedText
INFO: Received message ["OK","6d83330b8d77b5868589f247015f658da1a95eed93f5d817b486cc9ee07cfa22",true,""] from Relay(uri=nostr.mom)
abr. 21, 2024 12:14:34 PM nostr.connection.impl.WebsocketClientListeners$TextListener handleReceivedText
INFO: Creating the command context with message OkMessage(eventId=6d83330b8d77b5868589f247015f658da1a95eed93f5d817b486cc9ee07cfa22, flag=true, message=)
abr. 21, 2024 12:14:34 PM jdk.internal.net.http.websocket.WebSocketImpl$ReceiveTask processText
FINE: [HttpClient-1-Worker-0] [18s 383ms] [WebSocket] exit onText 2 returned java.util.concurrent.CompletableFuture@279f774e[Completed exceptionally: java.util.concurrent.CompletionException: java.lang.NullPointerException: Cannot invoke "nostr.context.impl.DefaultRequestContext.getPrivateKey()" because "this.context" is null]
abr. 21, 2024 12:14:34 PM jdk.internal.net.http.websocket.WebSocketImpl tryChangeState
FINE: [HttpClient-1-Worker-0] [18s 383ms] [WebSocket] change state from TEXT to IDLE true
abr. 21, 2024 12:14:34 PM jdk.internal.net.http.websocket.WebSocketImpl$ReceiveTask run
FINE: [HttpClient-1-Worker-0] [18s 383ms] [WebSocket] receive state: IDLE
abr. 21, 2024 12:14:34 PM jdk.internal.net.http.websocket.WebSocketImpl tryChangeState
FINE: [HttpClient-1-Worker-0] [18s 383ms] [WebSocket] change state from IDLE to WAITING true
abr. 21, 2024 12:14:34 PM jdk.internal.net.http.websocket.TransportImpl request

I have found a solution. Look at how I use the CompositeListener to pass all listeners.
Also, in the ConnectionImpl, I was throwing an exception in the connect when the connection was failing. This is what, I believe, was causing all subsequent connection attempts to fail. I have now removed the throwing of the RuntimeException and I am logging an exception instead. This seems to work.
A COUPLE OF THINGS that I forgot to mention:

You have surely seen that I am not using the jetty library anymore to handle ws connections
Also, please always add the URI scheme of the relay in the properties file, e.g. ws://localhost:3333 - Although I think probing for https and http is necessary, I don't think it should be handled in here, but at client level. I want to move away from it, also, in the spirit of simplifying things.

NOW, with these issues solved, my original problem should now become apparent: the ServiceLoader does not instantiate the command handlers (ok, eose, auth etc.)

I have finally found the issue with the ServiceLoader!


os-crowdsec: Internet dies shortly after plugin loads

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[X] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[X] The title contains the plugin to which this issue belongs

Describe the bug
In a recent update (I believe this exists in the last two releases), the crowdsec plugin kills the internet occasionally. Often times, it occurs shortly after the plugin loads. It will intermittently work (for 1-2 second periods every 10+ minutes). Sometimes pinging <IP_ADDRESS> works, but pinging google.com never works (regardless of what the DNS server or nameserver is set to). Connection to WAN stops working, but LAN works fine.
I believe this is related to this suricata issue. This has also been reported on reddit.
Tip: to validate your setup was working with the previous version, use opnsense-revert (https://docs.opnsense.org/manual/opnsense_tools.html#opnsense-revert)
To Reproduce
Steps to reproduce the behavior:

Install the latest OPNsense release
Install the os-crowdsec plugin
Activate it
Wait for the internet to stop working
Disable os-crowdsec
Internet comes back almost instantly

Expected behavior
Crowdsec to work, but not block all connection to WAN.
Relevant log files
Available upon request, but I'd prefer not to repro this unless I have to.
Environment
Software version used and hardware type if relevant.
e.g.:

Running in Proxmox:

Hi!
Thanks for the report, but please use https://github.com/crowdsecurity/crowdsec/issues in the future for crowdsec plugin issues, or they might be overlooked.
Is this still a problem today? I could not replicate on 24.1.3.
Thanks

I had disabled crowdsec when I originally posted it. I've just re-enabled it. Will report back tomorrow if I don't lose internet connection, but so far so good. If you didn't make any updates on your end, I'd imagine it's related to OPNsense's Suricata fixes that came out a few releases ago.

All good so far. I'll re-open if that changes.


allow multiple signatures

In the case where we had multiple SIGN requests in the attestation plan, the whole array of signature was being replaced rather than appended.

Test failure is:
2024-12-19T23:15:08.9244061Z [2024-12-19T23:15:00.692Z]  INFO Logs diverted to build/testbed.component-test.log
2024-12-19T23:15:08.9244532Z     utils_for_test.go:100: Using transaction latency threshold of 2s
2024-12-19T23:15:08.9244764Z     utils_for_test.go:527:
2024-12-19T23:15:08.9245678Z         	Error Trace:	/home/runner/work/paladin/paladin/core/go/componenttest/utils_for_test.go:527
2024-12-19T23:15:08.9247130Z         	            				/home/runner/work/paladin/paladin/core/go/componenttest/component_test.go:1663
2024-12-19T23:15:08.9247485Z         	Error:      	Condition never satisfied
2024-12-19T23:15:08.9247950Z         	Test:       	TestPrivacyGroupEndorsementConcurrent
2024-12-19T23:15:08.9248620Z         	Messages:   	Deploy transaction did not receive a receipt
2024-12-19T23:15:08.9248857Z     utils_for_test.go:537:
2024-12-19T23:15:08.9249832Z         	Error Trace:	/home/runner/work/paladin/paladin/core/go/componenttest/utils_for_test.go:537
2024-12-19T23:15:08.9251440Z         	            				/home/runner/work/paladin/paladin/core/go/componenttest/component_test.go:1663
2024-12-19T23:15:08.9251821Z         	Error:      	Expected value not to be nil.
2024-12-19T23:15:08.9252263Z         	Test:       	TestPrivacyGroupEndorsementConcurrent
2024-12-19T23:15:08.9252544Z --- FAIL: TestPrivacyGroupEndorsementConcurrent (8.26s)
2024-12-19T23:15:08.9252673Z FAIL
2024-12-19T23:15:08.9253017Z coverage: 59.8% of statements in github.com/kaleido-io/paladin/core/...
2024-12-19T23:15:08.9253322Z FAIL	github.com/kaleido-io/paladin/core/componenttest	61.891s

Feels like it need checking for relevance to this fix, as in the same general area and not one we're tracking via an issue as an intermittent failure

It is failing on the deploy which is way before we attempt any assemble and sign.  I can reproduce it locally if I run the test in a loop.  In a loop of 30, it failed once and passed 29 times.
So I think this is an intermittent bug and unrelated to this fix.  I'll raise a separate issue to track it.

fixes #486

test passed on re-run so raised https://github.com/LF-Decentralized-Trust-labs/paladin/issues/489 to track the intermittent failure.


Include option for preserve_fid

I'd like to read in layers which had an id and use that as a column in the table. ogr2ogr has this option as preserve_fid
I'm seeing now that there's always INCLUDE_FID=NO
https://github.com/duckdb/duckdb_spatial/blob/5cde7a2abead24e8fc81a33e8ee321188f4df2de/spatial/src/spatial/gdal/functions/st_read.cpp#L277
Is it possible to add this as an option to ST_Read?

Hi! Thanks for filing this issue!
If I recall correctly I disabled it due to difficulties with projection pushdown as the column order changes whether or not it's present. AFAIK the FID is always sequential (?), and DuckDB preserves insertion order so if you would import the data into a table could maybe use the rowid pseudocolumn instead.
I want to refactor st_read at some point in the future anyway so I'll try to revisit this issue at that point.

Thanks for the explanations! Unfortunately the FID doesn't always increment up from 1 sequentially- I came across this because I was converting a shapefile to GeoJSON while also running a SQL filter:
ogr2ogr -f GeoJSON -preserve_fid -dialect sqlite -sql "SELECT f.fid .... WHERE ..."

Since relying on the FIDs have caused annoyance and confusion elsewhere in my processing, I should be able to work around this for my current use case (I'll change the FID to an attribute so it shows up normally in the DuckDB table).


moshi-ir's proguard rule generation currently doesn't support the experimental K2 compiler

I switched to 1.8.20 and get following error:
e: moshi-ir's proguard rule generation currently doesn't support the experimental K2 compiler
Disable the K2 compiler by removing -Xuse-k2 flag or disable proguard rule gen.

How do I disable disable rule gen? I don't use proguard in my project.

Oh it's
moshi {
    generateProguardRules.set(false)
}

Sorry for false alarm.


Composer feature (#548)

Pull request opened by the merge tool on behalf of #548

Merged in 2e626a005d6639ff4d1e26c43fb7f97a87d50d5a


i.e. 11compatibility

i.e. 11 Compatibility
Description

I just realized published content is not running in i.e. 11.  Has anyone found any setting or workaround to get it to work in i.e. 11.  I am running the most current version of i.e.

I now realize the i.e, is not supported, but just wondering if by any chance has anyone got it to work.
Thanks
Aaron
Version

Platform

Web Version [X]
Windows [X]
Linux [X]

Hi
Internet explorer 11 is not supported by nunuStudio, IE lacks support for a lot of modern web API and its a bit hard to get it to fully work with nunuStudio.
Some older version worked with it but i had to leave support for IE eventually in order to adopt new "better" web API's.
The last stable version of IE 11 is now 3 years old, accoding to https://www.w3counter.com/globalstats.php IE11 is now used by less than 4% of the internet users, if youre a web dev dont worry!
Thanks a lot!
Cumps


Add Alert component + story + tests

This PR adds the Alert component, story, and tests.
The Alert component was updated with the following:

No longer requires alert object as a prop. Instead the props required are not nested and passed in at the top level (i.e. alert.alertText is now just text)
duration prop used to be in units of milliseconds, but is now in seconds (could go either way on this)
The component is now self-cleaning. Basically before, you would need something external to remove the alert component from the dom after it was dismissed. However, now it holds an exited state and will render null so that cleans itself up after exiting.

Other additions in this PR:

react-test-renderer & jest-emotion was installed to do snapshot testing
Alert story was added

 Convert Alert component

@smmullen90 addressed in 59e5850


[Safe Apps] Favorites

What is this feature about? (1 sentence)
This will allow users to mark regularly used Safe apps as favorites for easier access.
Why is it needed? What is the value? For whom do we build it?

Trello activity

Trello problem statement

More Safe Apps are added regularly, making it hard for users to access a specific app

It is hard to get an overview of all apps and discover new relevant apps

It becomes harder to quickly find and access a specific Safe App

High-level overview of the feature

The Safe app card gets a button to star/mark as favorite
Favorite apps get shown in the side bar
https://zpl.io/bALO4GB
Side bar is sorted same like the regular page, i.e alphabetically.

Closing this one as @Graeme-Code is working on a better Safe apps overview and depending on the outcome we should re-evaluate if we want favortes or not.


Support for multiple RSA signing/verification keys

We have a use case, when for some of our resources we need to have multiple signing keys. Currently, there are no option to have multiple RSA signing/verification keys. With JwtAccessTokenConverter implementation it is possible to set only one. Also, /oauth/token_key endpoint returns only one public key. It can possibly provide different key depending on the client_id or Principal.

@yyunikov Take a look at JwkTokenStore as it provides support for Providers that expose multiple signing keys via a JWK Set.

@jgrandja I am checking the code of JWKTokenStore for a while now, but I do not see how multiple keys are supported. The key handling is done by a JwtAccessTokenConverter that handles only one signing and verifying key pair. Could you please let me know what you meant in more details?

@matyee JwkTokenStore is meant to be used by Resource Server's only. It uses a JwkVerifyingJwtAccessTokenConverter (look at the constructor in JwkTokenStore) for verifying the signature and obtains the Public Key(s) using a JwkDefinitionSource which fetches the keys from the Authorization Server's JWK Set Url.
Just to be clear, this support is for Resource Server only. There is no support for the Authorization Server side where it manages and exposes these keys via the JWK Set endpoint.

@jgrandja  Thanks Joe...this works beautifully.   I'm wondering if there are plans to add Authorization server support in the future?

@chrisoberle There are no plans to add JWK support. As an FYI, this project is in maintenance mode as our efforts are focused in building the new OAuth support in Spring Security 5. Please see this blog for more details.
For your reference, see this sample which exposes a JWK Set via the Authorization Server using Nimbus JOSE library.


Adding APIs that the BlobStoreCompactor will eventually use

Moving some APIs from Log to PersistentIndex because PersistentIndex is always expected
to have an atomic view of the store
Adding APIs in PersistentIndex to fetch/operate on Index segments of a particular Log
segment
Adding APIs to log to add and drop segments.
Related tests

./gradlew build && ./gradlew test on OSx and Linux passed
Formatted
Primary reviewers: @pnarayanan, @nsivabalan
Unit testing

Class
Class, %
Method, %
Line, %
Test class

Log
100% (1/ 1)
100% (25/ 25)
99.3% (148/ 149)
LogTest

PersistentIndex
87.5% (7/ 8)
92.2% (47/ 51)
90.7% (470/ 518)
IndexTest

CompactionLog
100% (3/ 3)
100% (23/ 23)
95.5% (126/ 132)
CompactionLogTest

New function in IndexValue is also covered.

@pnarayanan All comments addressed.

@pnarayanan Comments addressed

@nsivabalan Addressed comments

@nsivabalan Addressed comments

built and coding style applied ? @pnarayanan I will go ahead and merge once @vgkholla confirms the build and coding style.

@nsivabalan Built on OSx and Linux and coding style applied.

Merging.


rickshaw-index: confirm indexing

-docs are counted on submission and compared to a query before exiting.

@atheurer
I think I fixed the problems that your were hitting in CI, but you need to rebase in order to get one of the fixes.


Add ol.size.hasArea

This pulls the hasArea function from map.js and puts it in size.js.  The compiled output is the same with and without this, so it is mainly a stylistic change.  I'm working on removing cases where we shadow variables.  An alternative would be to rename the argument for hasArea, but it seems more appropriate to have ol.size.hasArea.

Looks good to me. Please merge when Travis says so.


BUG - bitOrder element have maxOccurs=1

Describe the bug
on modbusInterfaceDescription the bitOrder element is allowed multiple times. We do not support multiple conversion schemes at the same time on the same device.
Expected behavior
the bitOrder element should occure exactly once

thanks. Pls. make sure that bitorder, wordswap etc, are still possible in parallel


PRO-1672: localized: false migration

_id is changing, which is impossible to do without creating a new document.
On Fri, Aug 20, 2021 at 6:01 PM Alex Bea @.***> wrote:

@.**** commented on this pull request.
In index.js
https://github.com/apostrophecms/redirect/pull/4#discussion_r693239311:

},

methods(self) {
return {

 addUnlocalizedMigration() {

   ***@***.***/redirect:unlocalized', async () => {

     const redirects = await self.apos.doc.db.find({

       type: self.name,

       aposMode: 'published'

     }).toArray();

     for (const redirect of redirects) {

       delete redirect.aposLocale;

       delete redirect.aposMode;

       delete redirect.lastPublishedAt;

       redirect._id = redirect._id.split(':')[0];

       if (redirect.aposDocId) {

         await self.apos.doc.db.removeMany({

Why not patch it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apostrophecms/redirect/pull/4#pullrequestreview-735411327,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAH27NP5YD7L6DRRP2WD7DT53GFJANCNFSM5CQZ325Q
.
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&utm_campaign=notification-email
.

--
THOMAS BOUTELL | CHIEF TECHNOLOGY OFFICER
APOSTROPHECMS | apostrophecms.com | he/him/his


When duplicating parts, BOM items of the same part get lost

Please verify that this bug has NOT been raised before.

[X] I checked and didn't find a similar issue

Describe the bug*
Some assembly parts I use have multiple BOM items of the same part. E.g.,  1 meter of cable A to use for connecting X to Y, 2 meter of cable A to use for connecting Y to Z.
When duplicating such a part, only one of these items gets copied. The rest is lost.
Steps to Reproduce

Create a new part with part attribute "Assembly" enabled.
Add a BOM item to the Bill of Materials. Any part will do. Add a note to the part, e.g., "Foo".
Repeat, but use a different text for the note, e.g., "Bar". The BOM now contains 2 items of the same part, as expected.
Duplicate the part. Now the issue becomes visible: the BOM only contains 1 item ("Foo").

Expected behaviour
The BOM should be copied completely.
Deployment Method

[ ] Docker
[ ] Bare metal

Version Information
Demo Server 0.13.0 dev
Please verify if you can reproduce this bug on the demo site.

[X] I can reproduce this bug on the demo site.

Relevant log output
No response

@JustusRijke thanks for reporting - fix incoming in https://github.com/inventree/InvenTree/pull/5347


how can i add a CylinderGeometry in the way of  adding  primitive

in this constrctor,i can't find position parameter,so how can i add a CylinderGeometry in the place where i want to put

You can code like this, I wish this can help you!
var geometry = new Cesium.CylinderGeometry({
  length: 1,
  topRadius: 10,
  bottomRadius: 10,
  slices: 12,
});

var m = Cesium.Matrix4.multiplyByTranslation(Cesium.Transforms.eastNorthUpToFixedFrame(
    Cesium.Cartesian3.fromDegrees(113, 36)),
  new Cesium.Cartesian3(0, 0, 0),
  new Cesium.Matrix4());

instances[i] = new Cesium.GeometryInstance({
  geometry: geometry,
  modelMatrix: m,
  id: ‘yourid’
});

}
viewer.scene.primitives.add(new Cesium.Primitive({
geometryInstances: instances,
appearance: new Cesium.MaterialAppearance({
material :  new Cesium.Material({
fabric : {
type : 'Color',
uniforms : {
color : new Cesium.Color(1.0, 0, 0, 1.0)
}
}
}),
faceForward : true
}),
allowPicking: false,
interleave: true,
releaseGeometryInstances: true
}));

Thanks for the code example @Hiwen.
@engineerhe please use the Cesium forum for general questions. You might also find this guide helpful on understand how to construct geometry with the primitive API in CesiumJS: https://cesium.com/docs/tutorials/geometry-and-appearances/


OperatingSystem library doesn't support pathlib.Path objects

OperatingSystem library should support also Python Path objects. Currently OperatingSystem library keyword which uses path or file path as argument, expect the argument to be as a string. Would it be possible to enhance library keyword to support also Python Path as arguments.
This could be useful when library keywords returns a path object and that object is example used in Get File. Currently users are required to do:
${path}    Library Keyword Return Path Object
${path}    Convert To String    ${path}
${data}    Get File    ${path}

But it could be handy to just do
${path}    Library Keyword Return Path Object
${data}    Get File    ${path}

All our libraries (and APIs) certainly should accept Path objects as paths. Which keywords have you had problems with?
A related question is that should keywords also return Path objects. For backwards compatibility reasons that's probably not a good idea in general, but if a keyword gets Path as an argument it could also return one.

OperatingSystem now supports pathlib.Path. Need to check are there other libs, especially XML and Process, that need this support as well.

Somewhat related, we probably should add type conversion support for pathlib.Path.

Added pathlib.Path support to Process and XML. With the former it was possible to give stdout and stderr as pathlib.Path but not stdin or cwd. WIth the later parsing worked but saving didn't. Still need to check do other libs use paths.

Also Screenshot used paths and that's now fixed as well. That ought to be all.


fixes spells display when single class heroes reach level 20

@ChrisPJohn , this is the fix for a bug mentioned on Nexus forums. I also took a chance to properly update the branch version. I suggest we bump versions on DEV as soon as we merge it into MAIN to ensure gamers get a proper update on UMM. Currently, MAIN still says <IP_ADDRESS>

@ChrisPJohn , this is the fix for a bug mentioned on Nexus forums. I also took a chance to properly update the branch version. I suggest we bump versions on DEV as soon as we merge it into MAIN to ensure gamers get a proper update on UMM. Currently, MAIN still says <IP_ADDRESS>

Yeah, apparently I forgot to commit the version bump this last time. I do try to keep the repository up to date with versions.


Letters mixed as numbers

Environment

Tesseract Version: 4.1.1
Platform: Darwin 19.6.0 Darwin Kernel Version 19.6.0: Mon Aug 31 22:12:52 PDT 2020; root:xnu-6153.141.2~1/RELEASE_X86_64 x86_64

Current Behavior:
Letters get mixed as numbers when nearby a number. For example 400g becoming 4009 is not desired.
Expected Behavior:
Tesseract OCR should not mix g with 9 unless it's desired behavior for some reason.
Suggested Fix:
Numbers with units are quite common and they should be trained well enough in original tessdata. Furthermore, if there's some additional logic that favors a number over letter based on its proximity to other numbers, there should be a parameter to turn it off.

It helps to provide more information for such reports. Please attach an image which shows the problem. Which language / script model did you use for the recognition?

Try -c load_number_dawg=0. I did not test it myself, but that should be the parameter to turn it off.

Thank you for your quick response.
Doesn't seem to help. I use Finnish language and psm 6. In this picture difference is quite obvious unless Tesseract OCR recognizes g as subindex 9. Let's have a moment to think which one is more common.
`tesseract -l fin --psm 6 -c load_number_dawg=0 "/Users/torava/Desktop/Näyttökuva 2021-2-1 kello 12.42.08.png" stdout
Avokado 7009`

However, in some cases picture is interpreted correctly.
`tesseract -l fin --psm 6 "/Users/torava/Desktop/Näyttökuva 2021-2-1 kello 12.37.56.png" stdout
400g`

I tested your first example with latest Tesseract. It works perfectly.

A test with Tesseract 4.1.1 from Homebrew on macOS also shows a perfect result.

Hi @stweil
tesseract --psm 6 -c load_number_dawg=0 "10.jpg" stdout

MMAHME!
ONEYATAHO! 5
┬ú110149

~ BHMMAHVE!
┬® OMEYATAHO! &
p936589 ae
I want to read the last line. Any suggestion to improve the detection.
Many thanks in advance for your help!
best regards Martin


ensureMessageFormat: fix activity prop check

Previous condition allowed copying of properties having undefined value and potentially properties containing other falsy values.

#1707 is good.


Module request: nixos/bore-scheduler

Project description

A set of patches to the linux kernel that add a new scheduler that optimizes a specific metric.
Metadata

homepage URL: https://github.com/firelzrd/bore-scheduler
source URL: https://github.com/firelzrd/bore-scheduler
license: gpl2+ (https://github.com/firelzrd/bore-scheduler/issues/42#issuecomment-2211440328)
platforms: linux

Add a :+1: reaction to issues you find important.

For extra clarification: this scheduler massively improves responsiveness / latency under load:

Perhaps also relevant for people running into this feature request, it has indirectly already been packaged by Chaotic-cx:
https://github.com/chaotic-cx/nyx
The relevant package being the CachyOS kernel: https://github.com/chaotic-cx/nyx/tree/main/pkgs/linux-cachyos
The CachyOS kernel has other tweaks as well though.
Could be worth its own feature request, since NixOS already packages the Zen kernel..


Error installing update

Hi,
Getting this error trying to update to the latest version...
Updating to<EMAIL_ADDRESS>failed.Hide output…

npm ERR! code E404
npm ERR! 404 Not Found - GET https://codeload.github.com/accodeing/copy-with-style/legacy.tar.gz/v1.4.0
npm ERR! 404
npm ERR! 404  'https://www.atom.io/api/packages/copy-with-style/versions/1.4.0/tarball' is not in the npm registry.
npm ERR! 404 Your package name is not valid, because
npm ERR! 404  1. name can only contain URL-friendly characters
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

npm ERR!  I have not been able to update the package on atom.io due to some error on their side. I'm not the only one with this problem, please see https://github.com/atom/apm/issues/572 for details and updates on that 🙁
I think you should be able to install it manually though (by downloading from Github), but I'm not prepared to give formal instructions about how to do that 🙂

Still waiting on an update on this issue from upstream 🙁

Hello,
maybe you can give INFORMAL instructions on how to install directly from your Repo?
Like back up this directory...
Clone Repo
Run Build?
Copy these files to apm registry
If failure restore directory.
Right now you are the popular game in town (google turns you up first)
and the install fails.

Copying from GitHub at least did the trick...
apm install accodeing/copy-with-style

Unfortunately, perhaps because you are using main instead of master, when subsequently checking for updates, I get the error:
Fetching outdated packages and themes failed.Hide output…
Exit code: 128 - fatal: couldn't find remote ref master


TokenHelper class not working on .net core

I was trying to use the TokenHelper class using .net core 3.1 web application. But AppForSharePointOnlineWebToolkit nuget package is unable to generate the code files. It works on .net framework projects though.
Also I tried with copying the content of the TokenHelper class too but most of the imports are not supported in .net core.
So is there any .net core compatible version of this class so that we can get clientcontext using accesstoken?

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

ID: 35a3017a-6d6c-237f-fc57-94247a8f7bf6
Version Independent ID: 93413445-18fa-9245-ea64-1c1d789dce1e
Content: Granting access using SharePoint App-Only
Content Source: docs/solution-guidance/security-apponly-azureacs.md
Product: sharepoint
GitHub Login: @VesaJuvonen
Microsoft Alias: vesaj

Any updates on this matter? i'm encountering the same problem

Any updates on this matter? i'm encountering the same problem

Hi, I've the same issue.


fix: add chinmay29hub

resolves #1016
Checklist for profile addition

[x] Name the PR fix: add <your-username>.
[x] You've added file named your-username.json in the profiles directory.
[x] You've linked the PR to the correct issue.

Hey there! Do you need any help with this PR?

No thanks, I had some issues with my editor 😄
On Fri, 11 Feb 2022, 9:49 pm David Leal, @.***> wrote:

Hey there! Do you need any help with this PR?
—
Reply to this email directly, view it on GitHub
https://github.com/EddieHubCommunity/awesome-github-profiles/pull/1018#issuecomment-1036380609,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQSGRITUCPJ6KAFDLHSAI5DU2UZJ5ANCNFSM5OEWXZ7A
.
You are receiving this because you modified the open/close state.Message
ID: <EddieHubCommunity/awesome-github-profiles/pull/1018/c1036380609@
github.com>


How do you use it?

Sorry, dumb question, but I successfully initiated my first vue extension project, the terminal says "DONE Compiled successfully in 51ms". Now what do I do? No extension appears in the Chrome extensions page, and doing "Load unpacked" from it to the "src" folder says the "manifest file is missing or unreadable".
Got any usage instructions?

Oh, I think I know what the issue was - after the thing prompts "Should we run 'npm install for you?'" I have to do an extra 'npm install' afterwards because they aren't the same? Well, I got it up and now I'm getting a bunch of socket connection errors, but I think I'm making some progress.


Fix the handling of unexpected user errors

Codecov Report

Merging #2022 into canary will decrease coverage by 0.25%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           canary   #2022      +/-   ##
=========================================
- Coverage    6.44%   6.18%   -0.26%
=========================================
  Files          86      86
  Lines        4079    4072       -7
=========================================
- Hits          263     252      -11
- Misses       3816    3820       +4

Impacted Files
Coverage Δ

src/util/get-files.js
96.89% <0%> (-3.11%)
:arrow_down:

src/util/read-metadata.js
100% <0%> (ø)
:arrow_up:

src/util/get-project-name.js
100% <0%> (ø)
:arrow_up:

src/util/hash.js
100% <0%> (ø)
:arrow_up:

src/util/error.js
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Expound upon the concepts of Nature, Risk and Quality of change

This article suggests some concepts about the published change: Nature, Risk and Quality.  Are there references to cite?
The article asserts that " it's not feasible to communicate the quality in the version number," and yet the article's cited example of a SemVer number on a package identifies the "-beta" suffix of the version as indicating the Quality of the change.  Can this be better explained?  At present it seems to be self-contradicting.
Both of these aspects go towards understanding when to use SemVer suffixes and/or Artifact Server View features.

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

ID: 28b723c2-424c-263a-292e-95547303b95f
Version Independent ID: ef570f9d-0cf4-6afb-2796-0d2a838289ae
Content: What are feed views? - Azure Artifacts
Content Source: docs/artifacts/concepts/views.md
Product: devops
Technology: devops-artifacts
GitHub Login: @ramiMSFT
Microsoft Alias: rabououn

@mikerains Thanks for your feedback! It's referring to the numerical segment of the version. I'm updating this shortly


test(client): do not resolve dataproxy url in tests

A previous PR removed network calls to resolve the data-proxy version in functional tests, but did not work in edge clients because process.env isn't accessible there (since it's polyfilled). Instead I switched to rely on the generated client stored engine version which is<PHONE_NUMBER>000000000000000000000000000000 only in functional in tests. This should fix the remaining flakyness.

(Merge after release)


Add ES5 support to CreateIndex action

Coverage remained the same at 54.846% when pulling 4de015b004e1c42203514ff35780cfb4e5c41975 on asafalima:GH-415-Add-ES5-support-to-CreateIndex-action into 0f6a143d00a013ad98cf6b76f7da207d76b526d7 on searchbox-io:master.

Coverage decreased (-0.03%) to 54.82% when pulling 037347139d15a72af3f3deac097b2951d3708b43 on asafalima:GH-415-Add-ES5-support-to-CreateIndex-action into 0f6a143d00a013ad98cf6b76f7da207d76b526d7 on searchbox-io:master.

@asafalima do you confirm it works for 2.x too?

@gsmet Yes

@ferhatsb This one looks good! Can be merged, I think.


Configuration Function for Collection Behavior Test

Adds React.BackboneMixin.ConsiderAsCollection
Allows for use of non-standard Backbone compatible objects

Resolves #32

Hi guys,
Any chance of a merge?
Cheers,
nathan
/cc @markijbema

@clayallsopp can you release a new npm package and such?

Done! https://www.npmjs.com/package/react.backbone

Thanks for the merge.  Happy New Year! :)


ESD:245: Add link to API Reference

Signed-off-by: Thomas Lucka<EMAIL_ADDRESS>

Codecov Report

Merging #19 into master will increase coverage by 0.6%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master      #19     +/-   ##
=========================================
+ Coverage   87.68%   88.29%   +0.6%
=========================================
  Files          87       57     -30
  Lines        5637     3682   -1955
=========================================
- Hits         4943     3251   -1692
+ Misses        694      431    -263

Flag
Coverage Δ

#ios
88.29% <ø> (+0.6%)
:arrow_up:

Impacted Files
Coverage Δ

MSDKUI_Demo/IconButton.swift

MSDKUI_Demo/OptionsViewController.swift

MSDKUI_Demo/ManeuversOverviewViewController.swift

MSDKUI_Demo/GuidanceDashboardViewController.swift

MSDKUI_Demo/ExtensionArray.swift

MSDKUI_Demo/RouteViewController.swift

MSDKUI_Demo/ExtensionUITableViewCell.swift

MSDKUI_Demo/ExtensionBundle.swift

MSDKUI_Demo/ExtensionNMAMapMarker.swift

...UI_Demo/GuidanceDashboardTableViewDataSource.swift

... and 20 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 4777074...e899ccf. Read the comment docs.