webchat.js:2 Uncaught SyntaxError: Invalid regular expression: /\p{P}/: Invalid escape

Is it an issue related to Adaptive Cards?
No
Is this an accessibility issue?
No
What version of Web Chat are you using?
Latest production
Which distribution are you using Web Chat from?
IFRAME
Which hosting environment does this issue primarily affect?
Electron
Which browsers and platforms do the issue happened?
Platform: Windows
Which area does this issue affect?
Development experience
What is the public URL for the website?
https://polarisbot-test.microsoft.com/
Please describe the bug
I am opening the bot using an iframe like this :

to open the bot in an electron application where the url is https://polarisbot-test.microsoft.com/
I consistently get this error on the console as provided below and then it is followed by this error
?mode=dark&copilot=fcshell:238)
    at fire (jquery-3.6.0.js:3500)
    at Object.fireWith [as resolveWith] (jquery-3.6.0.js:3630)
    at done (jquery-3.6.0.js:9796)
    at XMLHttpRequest. (jquery-3.6.0.js:10057)
I load the webChat.js this way in the bot code
<head>
<meta charset="UTF-8" />
<title>Polaris Bot</title>
<script src="https://cdn.botframework.com/botframework-webchat/latest/webchat.js"></script>
</head>

and then I use the window.webchat like this
  var directLine = window.WebChat.createDirectLine({
  token: data.Token

});
console.log("window.webChat with directLine", directLine);
// render web chat widget that enables chatting with polaris bot
window.WebChat.renderWebChat(
{
directLine: directLine,
userID: activityUserId,
locale: 'en-US',
styleOptions,
overrideLocalizedStrings: {
TEXT_INPUT_PLACEHOLDER: 'Type your message here. Tip: type "help" to get the list of chat commands supported. type "cancel" or "quit" to end an ongoing dialog with me.'
}
},
document.getElementById('webchat')
);
Do you see any errors in console log?
webchat.js:2 Uncaught SyntaxError: Invalid regular expression: /\p{P}/: Invalid escape
    at Module.51638 (webchat.js:2)
    at r (webchat.js:2)
    at Object.9316 (webchat.js:2)
    at r (webchat.js:2)
    at Object.67090 (webchat.js:2)
    at r (webchat.js:2)
    at Object.9998 (webchat.js:2)
    at r (webchat.js:2)
    at Object.34654 (webchat.js:2)
    at r (webchat.js:2)

How to reproduce the issue?
I face this issue strictly when I use
in the header
When I change the script to use  it works again
What do you expect?
I need to get rid of this error message
It strictly occurs when I use the latest version and not 4.15.9
What actually happened?
When I load my electron application, I have a window on the left side which basically loads the iframe with the bot url .
But the bot never loads, instead it throws errors as provided in the description
Do you have any screenshots or recordings to repro the issue?

Adaptive Card JSON
No response
Additional context
No response

@Amey819 what exact electron version are you using?

@Amey819 please either use the ES5 build (https://cdn.botframework.com/botframework-webchat/latest/webchat-es5.js), or upgrade the Electron to the latest verison

@Amey819 the release date of the version was 2018-09-10. The version is using Chrome 57 along with NodeJS 8.9.3, which both are not supported already and have a lot of publicly discussed security vulnerabilities.
If you really can't, I suggest pinning WebChat to the latest working version for now until you get the chance to update.

The error seems to be specifically happening around parsing a string, Is there a way to debug exactly what is causing that error?

@Amey819 The error is caused by an older Chrome engine not recognizing the regular expression /\p{P}/ due to the lack of support of the \p escape class: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Regular_expressions/Unicode_character_class_escape
According to MDN the support was introduced in Chrome 64 while Electon v2.0.9 uses Chrome v56.


convert Buffer Parsing Exception messages to error code format

Looks like all the messages passed to throwBufferParseException are hardcoded and should be converted to error codes (camel case) and passed to createOdspNetworkError as the fluidErrorCode.  Or, we could decide it's better to keep all these under one error code and leave it as-is, and close this.
Originally posted by @markfields in https://github.com/microsoft/FluidFramework/pull/7788/files#r726618889

Hi, I was wondering if this is an issue that I could work on. If so, I will probably need some help on where to start since I'm new to contributing.

@mqnguyen5 Happy to help, and have your help! Start by looking at the code that calls the function where that log is, and see what kinds of values are passed as message. If they're all string literals describing different cases, then we're in business. The goal then would be to get that info in some fields other than error message. Either fluidErrorCode, or maybe a custom property "reason".
Leaving out some details for now, take a look and let me know what questions come up, happy to spell it out more as you familiarize yourself here.

We've moved away from error codes and back to messages, see #9249.


Default Netflix addon subtitles act like my kodi presets

Feature request
I'm submitting a proposal to ...

[ ] new feature
[x ] change an existing feature

Describe your request

I have my kodi presets on subs to show at the bottom of the video.Is there an option on the default Netflix subs as well??
Describe the use case

Describe an alternative

Development tips

Additional details or screenshots (if appropriate)
Specific details

Screenshots

you need to enable Kodi subtitles override settings to force change position of webvtt subtitles


Command scenario uses incorrect build names

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [x] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce
The documentation uses the wrong build name/paths when asking the user to run the samples.
In this section:
https://github.com/sseiber/MqttApplicationSamples/tree/main/scenarios/command#game_die-run-the-sample
the documentation asks the user to run "dotnet/command_producer..." and "dotnet/command_consumer..." when it should be "dotnet/command_server..." and "dotnet/command_client...".

After this PR 61 is approved (or denied and closed) I have another PR ready for this change.


Path is not updated in case-insensitive system when copying files

Coping files has some strange behavior when changing the path case in windows.
Also the task is up-to-date even if a folder case has being changed.
Expected Behavior
The folder path should be updated with the new case.
Current Behavior
The paths of the files copied to the destination are not updated and when these resources are accessed via linux, because it's case sensitive, the folders don't exist.
Steps to Reproduce (for bugs)
test-gradle.zip
In windows:
run gradle copyIssue
open src folder and change the case of the main folder
re-run gradle copyIssue
the task is up-to-date and the path isn't changed
Your Environment
windows 10
gradle 4.8

As far as Windows is concerned, the path did not change. So I'd say this is expected behavior.

I guess it gets terrible when the file you just copied then ends up going into something like a zip, which might be unpacked on a case sensitive filesystem where it no longer works correctly...

@lptr What do you think about this? Is there some way we could detect these changes without breaking performance?

I was a bit surprised that Gradle would "optimize away" case changes, and mark the task as UP-TO-DATE, so I dug a bit deeper. What actually happens is not that the task is up-to-date, as indeed Gradle detects the change and re-runs the task, as it is printed in the --info log and build scans:
The task was not up-to-date because of the following reasons:
--
Input property 'rootSpec$1' file src/main has been removed.
Input property 'rootSpec$1' file src/main/file.txt has been removed.
Input property 'rootSpec$1' file src/main/otherfile.txt has been removed.

The problem is that when Gradle asks for the testlocation/Main directory, the OS tells it that it it already exists. This is how a case-insensitive file system is expected to work. I don't think Gradle should handle this differently.


Add check for non-inclusive language

Add a check for usage of terms and language that is considered non-inclusive. We are using the woke tool for this with a wordlist that can be found at https://github.com/linux-system-roles/tox-lsr/blob/main/src/tox_lsr/config_files/woke.yml
CHANGELOG.md - cleanup non-inclusive words.

Would be great if the action (or at least the commit message or PR description) would have some human-readable description what does the CI action does and why.

Would be great if the action (or at least the commit message or PR description) would have some human-readable description what does the CI action does and why.

Sorry - we get so caught up in the tasks that the core team is focused on, we forget that other people might not realize what we're doing.
@nhosoi We should update all of the existing PRs like this:
PR title: Add check for non-inclusive language
PR body: Add a check for usage of terms and language that is considered non-inclusive.  We are using the woke tool for this with a wordlist that can be found at https://github.com/linux-system-roles/tox-lsr/blob/main/src/tox_lsr/config_files/woke.yml
in .github/workflows/woke.yml at the top-level it should have
name: Check for non-inclusive language
Then the PR status will look like Check for non-inclusive language / woke (pull_request)
@Jakuje is this sufficient?

Thank you for your suggestions, @richm. I think I updated the PR following your advice.
Sorry, @Jakuje, for giving you this surprise... Does this updated PR look better?

@richm, I put [WIP] to the PR title because once https://github.com/get-woke/woke/pull/252 (or its alternative) is merged, we may need to add a new option to the action to allow the warning-level words. Do you think it's ok to wait for the woke PR's fate? Or should we handle the woke PR separately?

Thanks for the improvements. Looks better. I have only concern about the modification of the changelog how it is done now.

Thanks for the improvements. Looks better. I have only concern about the modification of the changelog how it is done now.

Not sure what you mean - what's your concern about the changelog?

Thanks for the improvements. Looks better. I have only concern about the modification of the changelog how it is done now.

Not sure what you mean - what's your concern about the changelog?

Is it that the intention of the changelog is that it is readable by end users, and having lines like
- update to tox-lsr 2.4.0 - add support for ansible-test sanity with docker // wokeignore:rule=sanity

is going to be distracting?
We could tell woke to skip CHANGELOG.md completely - but then we run the risk that some sort of non-inclusive language getting into CHANGELOG.md, which would be even worse than having it in other code which isn't usually intended to be read.
We could edit CHANGELOG.md to change or remove those instances of non-inclusive language - this may be the best option.

yes, the linter hint might be distracting or confusing to users. I think editing the changelog not to contain potentially non-inclusive language should be preferred way (this is why we do it anyway, aren't we).

[citest]

[citest bad]

Thanks for your approval, @Jakuje! Since I have no privilege to do so, if you could merge this pr, I'd appreciate it. Thanks!


Exclude non-target labels from analysis

From a user request. They wanted to exclude labels such as "Off", "Camera", etc. from being included in analysis, which presumably means statistics and image covers. So if an image had 10 points, with 3 being Acropora, and 1 being Off, then the image cover would report 3/9 = 33% Acropora.
One way to implement this would be to add a boolean field to each labelset entry (LocalLabel), which is True if the label's included in analysis, and False otherwise. This could be a checkbox in the "Edit Label Entries" page.

We've gotten requests now and then to exclude annotations with such labels (Off, Camera, Unknown, Unclear, Shadow...) from being fed into classifier training, since those annotations could confuse the training process.

I can say with rather high degree of confidence that this won't help improve accuracy. It's a common (and understandable) misunderstanding of how ML behaves.

@beijbom Hmm, could you elaborate? Sounds like there's something I could learn from this, too.
Though now that I think of it, the different labels listed above have some different concerns (I'll also add Other hard coral):

Label
Don't want in coral cover analysis
Wide variety of appearances to learn
Visually similar to 'normal' classes

Camera
Y
N
N

Off / Transect hardware
Y
Y
N

Shadow
Y
?
?

Unclear
Y
?
?

Unknown
Y
?
?

Other hard coral
N
Y
?

Shadow: if it's near total darkness, then that should be a pretty consistent appearance. Arguably, it's actually desired for the classifier to label as shadow in that case. If it's borderline, just barely dark enough to not be sure, then it could be confused with other classes. But not sure if more confusing than, say, a point that's coral half-covered by algae.
Unclear: assuming this is kind of a blurry or murky appearance (or a combination of that and shadow), maybe this is again, something the classifier can (and arguably we'd want it to) recognize and label consistently.
Unknown: assuming this is "I can see this clearly but I just don't know what it is", ideally it's just a temporary annotation marked for later review, or it's consistently a small number of different things that the team just doesn't care to ID. If it's consistently a small number of different things, then the classifier should be okay to label that as Unknown. If many different things are being labeled Unknown (and there's a high enough ratio of such points), then arguably the team just needs more expertise to proceed with their study.
Other hard coral: this has the potential to 'intentionally' be a lot of different things. That is, even if the team can identify those corals, they choose not to do so since they're interested in only a specific genus or two for their study. However, we do want to recognize this differently from non-coral classes and we ideally want the classifier to label this consistently. That being said, CoralNet classifiers have tended to perform well at just that - recognizing hard coral vs. not hard coral.

So yeah, now that I think about it, maybe the 'bad classes for training' concern isn't that big a deal as long as annotators are pretty consistent. Though I'd still be interested to hear your view.

Yea, I guess there are two aspects of this. I was mostly referring to the former but I think both are valid in this case.

Let's say they really care about the distinction between Pocillopora and Acropora. Due to actual similarity and the feature we use, there is some overlap and so the accuracy is not 100%. Let's say its 95%. Let's now say they add a "Camera" class. There is now some new confusion between Pocillopora and Camera (perhaps if points are on the boundary) and some confusion between Acropora and Camera. My point here is that the original confusion (accuracy) remains. They will still get the 95% accuracy between the first two classes. In other words, the classifier has plenty of "capacity" to learn the new class without ruining what it can do for the first two. This may seem counter-intuitive for laypeople and I have heard this type of concerns many times before.

The other argument for keeping it is to make these things explicit. If they don't include, say, "Camera" in the training there is no way to know what the classifier will do for new points that fall on Camera. Perhaps it will end up skewing towards one of the corals more than some other, thus over-representing that class once they tally the stats.

So, yeah, I'd say keep all classes during training.
Excluding them when calculating cover stats is a whole different thing, and I agree it makes sense there in most cases. At least for some classes like camera where this is no correlation between the non-coral class and the underlying substrate). It could be problematic if there is heavy correlation, e.g. almost all "unclear" points are actually coral type X which happens to grow in deep groves and have a lot of algae around it. Just removing those when tallying could be problematic. So instead of saying. e.g.
40% coral type X, 50% coral type Y and 10% unclear (which tends to be coral type X), you are now dividing those 10% up equally between the two other types.
Does this all make sense?

Thanks for the insight, that does help a lot. The arguments regarding training sound like good material for the site FAQ or similar.
For the correlation concern with cover stats, I guess the concern should be mentioned in some help text, but it'll be the user's responsibility to adjust the calculations accordingly. Alternatively we could build some kind of 'weighted average' option (e.g. treat unclear as 80% X 20% Y), but that's probably too complicated for now.

For the correlation concern with cover stats, I guess the concern should be mentioned in some help text, but it'll be the user's responsibility to adjust the calculations accordingly. Alternatively we could build some kind of 'weighted average' option (e.g. treat unclear as 80% X 20% Y), but that's probably too complicated for now.

Yea, I'd assume most biologists would understand the statistics of leaving things out from the counts, so I'm not that worried about it.


70 and #225 Update home.css for better readability

@mathagician A good PR can also be made which adds comments to all this code, if you are interested.


Newly introduced weights in Embedding layer breaks the downstream layers such as ReversibleEmbedding in KerasNLP

The __init__ explicitly calls self.build() which causes issues with ReversibleEmbedding because it requires self.tie_weights to be set before self.build is called.
I propose that only call self.build() if weights is specified.
        if weights is not None:
            self.build()  # <- move to here
            if not (isinstance(weights, list) and len(weights) == 1):
                weights = [weights]
            self.set_weights(weights)

cc @mattdangerw

Thanks for the suggestion, I applied it.


Proposal for PSF / IRF denoising method

In Gammapy we are making our first experiences with analyzing "real" exported DL3 data from IACTs. One problem we have seen in the past are noisy PSF histograms, which caused problems (especially at low energies) when computing the reduced PSFs for analyses. Because of the noise, interpolation and extrapolation and computation of containment radii etc. becomes numerically unstable.
Last week I've thought a bit about the problem and sketched an idea for denoising PSF histograms, using anisotropic smoothing, which seems to give promising results. Here you can find the notebook:
https://github.com/adonath/notebooks-public/blob/master/psf-denoising.ipynb
Ideally (from the Gammapy side) the PSF, or IRFs in general, would be already noise-free on the DL3 data level (see also https://gamma-astro-data-formats.readthedocs.io/en/latest/irfs/irf_components/index.html#comments). So I thought it's rather in the responsibility of ctapipe to do the denoising / smoothing. Is there any interest, that I contribute such a method? Is there maybe already a place (sub-module) in ctapipe where such a method could fit?
@kosack and @lmohrmann

I am moving this to pyirf


[18.09 backport] bump buildkit to c7bb575343df0cbfeab8b5b28149630b8153fcc6

backport of https://github.com/moby/moby/pull/38023 for 18.09
this PR is based on https://github.com/docker/engine/pull/77
https://github.com/moby/buildkit/compare/520201006c9dc676da9cf9655337ac711f7f127d...c7bb575343df0cbfeab8b5b28149630b8153fcc6
Relevant changes:

https://github.com/moby/buildkit/pull/667 gateway: check for ReadDir and StatFile caps on client side
https://github.com/moby/buildkit/pull/668 dockerfile: fix ssh required option
https://github.com/moby/buildkit/pull/669 dockerfile: update default copy image
https://github.com/moby/buildkit/pull/670 solver: specify SSH key ID in error message when required key was not forwarded
https://github.com/moby/buildkit/pull/673 solver: fix possible nil dereference
https://github.com/moby/buildkit/pull/672 fix setting uncompressed label on content
https://github.com/moby/buildkit/pull/680 dockerfile: fix empty dest directory panic

fixes https://github.com/moby/buildkit/issues/679

janky and powerpc failures are unrelated


Documentation Fix: edits quick-start doc file section "The renderer process" segment

The 2nd paragraph in "The renderer process was a little difficult to read. The following line especially,

so it would be possible to interactive with

I have taken to revising the entire paragraph to make it flow easier.

Thanks!


SQLite.NET-PCL nuget should take dependency on VC Libs

If you create a UWP app in managed language (e.g., C#) and add SQLLite nuget to it, it will crash on runtime because sqlite3.dll cannot be loaded.
Workaround is to add "Visual C++ 2015 Runtime for Universal Windows Platform Apps" reference to the UWP app, alongside with SQLite.NET-PCL.

Well actually you need the SQLite for Universal Windows extension(?) which should add the VC++ Runtime.

I see, extensions does not add VCLibs, you just get warning:
The SDK "SQLite.UWP.2015, Version=3.12.2" depends on the following SDK(s) "Microsoft.VCLibs, version=14.0", which have not been added to the project or were not found. Please ensure that you add these dependencies to your project or you may experience runtime issues. You can add dependencies to your project through the Reference Manager.


fix(tooltip): adjusting theming values for tooltips

Description
Adjusting the background values for tooltips according the new Covalent UX
What's included?

Angular based tooltips theme fix
Component based tooltip themeing fix

Test Steps

[ ] npm run start
[ ] then open the docs and test any button with a tooltip
[ ] thennpm run storybook and open the local app to the tooltips story

General Tests for Every PR

[ ] npm run start still works.
[ ] npm run lint passes.
[ ] npm run stylelint passes.
[ ] npm test passes and code coverage is not lower.
[ ] npm run build still works.

Screenshots or link to StackBlitz/Plunker

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

v8.5.2
GitHub release

Your semantic-release bot :package::rocket:


Feat: testimonial section

Describe the feature
addition of testimonial section after "explore our free e book" section
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Assigned! @Anamika-Ratna
Make sure to attach before & after screenshots in your PR and include the issue number.
Also, please star the repo before working on it. Happy open-source contributing!

Hi @anuragverma108 ,
Kindly review my changes.
#744

The dark mode button in the RHS is by default on and when one turn it one it is going out of button.

See this.
Fix this & raise PR again


MODULE_NOT_FOUND for pdf.js

Need to install the following packages:
  @opendocsg/pdf2md
Ok to proceed? (y) y
node:internal/modules/cjs/loader:927
  throw err;
  ^

Error: Cannot find module 'pdfjs-dist/es5/build/pdf'
Require stack:
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:924:15)
    at Function.Module._load (node:internal/modules/cjs/loader:769:27)
    at Module.require (node:internal/modules/cjs/loader:996:19)
    at require (node:internal/modules/cjs/helpers:92:18)
    at Object.<anonymous> (/home/node/.npm/_npx/404aa1a6362bd464/node_modules/@opendocsg/pdf2md/lib/util/pdf.js:1:15)
    at Module._compile (node:internal/modules/cjs/loader:1092:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1121:10)
    at Module.load (node:internal/modules/cjs/loader:972:32)
    at Function.Module._load (node:internal/modules/cjs/loader:813:14)
    at Module.require (node:internal/modules/cjs/loader:996:19) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>  ]
}

Looks like pdfjs got updated in such a way that they broke the API you were using?

Any progress on this one @skepticalwaves did you get it working?

I have not because I have zero experience in js/node, just a user.

Is this fixed then? I tried to install it yesterday and had the same problem.

Just fixed it and published it in 8c6a7e4, give it a try! You're looking for v0.1.25.
Thanks for your patience!


Angular 7.2.5 TranslatePipe using EventEmitter

Hello,
I have updated my project to Angular version 7.2.5 with typescript 3.2.4.
Now I have a custom TranslatePipe that basically uses the TranslatePipe code.
After the update I have got errors that the subscription to the observable is not assignable to the EventEmitter.
So would it help for the next releases to change the EventEmitter in the pipe to Subscription type?
Although EventEmitter is for Component use?

I encountered the same problem as you and tried to fix with internal updates:
The current version on remote origin:
if (!this.onTranslationChange) {       this.onTranslationChange = this.translate.onTranslationChange.subscribe(       ...
Updated internally
if (!this.onTranslationChange) {       this.onTranslationChange = this.translate.onTranslationChange;       this.onTranslationChange.subscribe(       ...
Hope it usefull to you.
please share me If you found any solution.
Regards,


Assertion failure VK_ERROR_DEVICE_LOST and VK_TIMEOUT

Description
NOTE:  Screenshot and diagnostic log are attached below.
Loading a Vulkan capture on v1.7 breaks UI rendering and other features. This was first experienced from my own Vulkan application but I also loaded an example app from SaschaWillems/Vulkan, specifically the push constants example which can be seen in the logs. Switch between tabs and resize the window to see effects on UI.
Looking at the logs, we can see that there are many assertion failure:
RDOC 015088: [14:12:55]          vk_core.cpp( 278) - Error   - Assertion failed: '(vkr) == (VK_SUCCESS)' (vkr=VK_ERROR_DEVICE_LOST, VK_SUCCESS=VK_SUCCESS)

Furthermore, when I click different events in the Event Browser, another assertion failure comes up in the log.
RDOC 015088: [14:13:04]          vk_core.cpp( 493) - Error   - Assertion failed: '(vkr) == (VK_SUCCESS)' (vkr=VK_TIMEOUT, VK_SUCCESS=VK_SUCCESS)

Steps to reproduce

Open RenderDoc 1.7 and go to Launch Application
Select a Vulkan app in Executable Path e.g. something from SaschaWillems/Vulkan
Click Launch
Capture a Frame
Exit example application
Load capture
Switch between tabs, resize window, click events in the Event Browser

Environment

RenderDoc version: 1.7
Graphics API: Vulkan

System information outputted by NVIDIA control panel
NVIDIA System Information report created on: 03/28/2020 14:15:23

[Display]
Operating System:	Windows 10 Home, 64-bit
DirectX version:	12.0
GPU processor:		GeForce RTX 2070 with Max-Q Design
Driver version:		445.75
Driver Type:		DCH
Direct3D API version:	12
Direct3D feature level:	12_1
CUDA Cores:		2304
Core clock:		1185 MHz
Memory data rate:	12.00 Gbps
Memory interface:	256-bit
Memory bandwidth:	384.06 GB/s
Total available graphics memory:	16333 MB
Dedicated video memory:	8192 MB GDDR6
System video memory:	0 MB
Shared system memory:	8141 MB
Video BIOS version:	90.06.3C.00.55

20200328_DIAGNOSTIC.log

The log indicates that the Intel GPU is being used for replay which means it was also used for capture. I don't think Sascha Willems' demos do any sophisticated physical device selection - if you force it to use the NV gpu does that work? I think you can use -gpu 0 and -gpu 1 command line parameters and check what the overlay says is being used.
I also can't easily tell when that Intel driver version is from, but the CTS release listed is a fair bit older so I'd suggest checking to see if a newer Intel driver is available.

Hello, I just tried your suggestion and passed -gpu 1 as an argument. That seems to resolve the issue. The screenshot attached shows that Intel GPU is not being used. Regarding Intel drivers, I usually keep my PC up-to-date and from what I'm reading online that should also keep Intel graphics drivers up-to-date as well. Just to make sure, I went ahead and downloaded this utility https://www.intel.com/content/www/us/en/support/detect.html to scan for driver updates. With that, I don't see any graphics driver updates available.
Is the situation a real RenderDoc issue or should I just be mindful of what physical device is being used next time? To be honest, I had assumed that my NVIDIA GPU was being used this entire time so I didn't bother carefully checking, which is an oversight on my part. If this isn't a real issue then I apologize for wasting your time and thank you for taking a look regardless.

This looks like a driver bug then. If you're on a laptop then it wouldn't surprise me if you're stuck on older intel drivers because it seems like most laptop vendors cripple the driver updates and don't allow timely intel drivers to be installed. You could check in dxdiag for the actual reported intel driver version and search to see when it was released.
With your own vulkan applications I'd recommend always picking the discrete GPU over the integrated GPU, Sascha's demos I believe always picks physical device 0 by default. With a recent loader (which should come with the recent nvidia driver you have) then the system GPU preference order is used on windows, which means that the GPU connected to the display (i.e. the integrated GPU) is the first in the list.


[ALLUXIO-2649] add double quote. Correctly dereference JAVA and JAVA_HOME

https://alluxio.atlassian.net/browse/ALLUXIO-2649
If there is a space in JAVA_HOME, bin/alluxio and bin/alluxio-start.sh will not work.
For example, many Mac users' JAVA_HOME is:
/Library/Internet Plug-Ins/JavaAppletPlugin.plugin/Contents/Home
This is usually caused by running Oracle's official upgrade patch, for example from 1.6 to 1.8. Then your JAVA_HOME will be set like that.
This is not a unusually problem.For example, elasticsearch has this issue as well:
https://github.com/elastic/elasticsearch/issues/6942
I changed ${JAVA} to "${JAVA}" in some scripts to correctly dereference the variable.

Merged build finished. Test PASSed.

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

alluxio-bot, check this please

alluxio-bot, check this please

Merged build finished. Test PASSed.

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


fallbackEmojiStyles

Hi, I would like to know if there is any way that I could pull this object when clicking on an emoji in the picker?  or how can I add it in the Emoji Object obtained.

Hmm not ideal, but you could access it by rendering the Emoji component and then using a ref prop to access the component instance directly. Is there a reason why you're unable to use the Emoji component to handle the rendering?

we're trying to convert markdown formatted text containing emoj.colons and inserting emoji component in between the text does not seem possible.
example:
" *Hey* rigo :smile: , how are you?" should render as :
Hey rigo  how are you?

Ahh okay, do you think something like this would work? https://gist.github.com/jm-david/7f98c3ee4fe235c596cff26d7aa26b7b

Yep should do! :)

thanks david! going to try it. I have another question though, for EmojiSearch, the mapping is using the native , how can I replace the display from native to the set sheet?  screenshot below:

Are you using the Emoji component? Or just outputting the native emoji directly?

ah yes, i forgot i was display the native directly. thanks for the heads up. Using the emoji component now but it seems that it does not render the emoji:

however if i tried :santa::skin-tone-3: it displays the correct emoji :

nvm, just needed an update. hehe thanks!


Humanized date tests is failing

tests/test_formatter.py::test_humanized_date is failing all of a sudden, both on CI and locally.
It's also happening locally, on an updated virtualenv, so it's not a regression in a dependency. My guess is that's there's something funny like DST on HST which is breaking some assumption.

As of 2011, all member states of the European Union observe summer time; those that use CET during the winter use Central European Summer Time (CEST) (or: UTC+02:00, daylight saving time) in summer (from last Sunday of March to last Sunday of October).


Line chart Y-axis label positioning

Is there a way to move the Y-axis labels on a line chart from the left side to the right side?
Using this Plunker as an example (http://plnkr.co/edit/XKOmwcSMZKLbYNxA3kMb?p=preview), I'd like to move the 0, 1.25. 2.5, 3.75, etc to the right side of the chart.
It looks hackable in the ct-labels class to change the 'x' attribute in the foreignObject from -5 to something close to the right edge.
<foreignObject x="400" width="40" height="36" y="186.36363636363637" style="overflow: visible;">
6.25
</foreignObject>

What is a better way of going about this?

Currently this is not supported. You could use events to achieve this but I think we should open a ticket to make the label position configurable. I've opened #302 for this feature. Let's track any further discussion there.

Using the draw event you could do something like this easily: http://plnkr.co/edit/b5fouXXCWUesYrNufaoz?p=preview

That's a great suggestion, and should work for us. Thanks so much for the quick response!

Thanks @gionkunz for that plnkr. :relieved:
Just in case that someone is looking for a way to have the x axis on top of the graph, one can put this in the draw event
if(context.type === 'label' && context.axis.units.pos === 'x'){
    context.element.attr({
      y: 0
    })
}


[UX Test]: Home Page Design - Sources Table

Issue Name
Home Page Design - Sources Table
Test Purpose
This test will determine the optimal design of the Sources table on the Home Page. Currently, the Home page on the production site is a simplified view with Name and Description, while the design on the staging site includes 3 additional columns. The test will determine if the 3 additional columns are an improvement on the original design, and if filtering becomes confusing if these columns are not included.
Test Approach
The test will be conducted with 25 participants on the Lyssna platform. The test will also be sent to the GoFAIR US team.
Test Cost
$75
Test Time
The test is expected to take 1-2 days to set up, 1 day to collect results, and 1 day to analyze results.
Writeup URL
No response
Related WBS task
https://github.com/NIAID-Data-Ecosystem/nde-roadmap/issues/51
For internal use only. Assignee, please select the status of this issue

[ ] Not yet started
[X] In progress
[ ] Blocked
[ ] Will not address

Status Description
No response
Test status check list

[ ] This test has yet to be discussed between NIAID, Scripps, Leidos
[X] This test has been discussed/reported between NIAID, Scripps, Leidos
[X] This test has been scoped out
[ ] This test has been drafted in the platform in which it will be conducted
[ ] This test has been conducted
[ ] The results of this test have been analyzed

The results of this study can be found here: https://docs.google.com/presentation/d/1HAFuqGolPBYW4ixcjTeRkOdqnRvIvu_o/edit?usp=sharing&ouid=114065631315523104759&rtpof=true&sd=true

Per an email dated 2024.06.06 in an email requesting for more information on the study results, I am marking this issue as review for info

Hi @gtsueng, NIAID agrees to proceed with current design but may wish to revisit this design at a later date.

As the study result have been reviewed, I am marking this issue as pending close out and will close this issue after a week if there are no further comments to be addressed.


az blueprint assignment create from json parameter file

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az blueprint assignment create Extension Name: blueprint. Version: 0.1.1.
Errors:
Unable to build a model: Unable to deserialize to object: type, AttributeError: 'str' object has no attribute 'get', DeserializationError: Unable to deserialize to object: type, AttributeError: 'str' object has no attribute 'get'
Traceback (most recent call last):
python3.8/site-packages/msrest/serialization.py, ln 1416, in _deserialize
    found_value = key_extractor(attr, attr_desc, data)
...
    return data.get(found_key)
msrest.exceptions.SerializationError: Unable to build a model: Unable to deserialize to object: type, AttributeError: 'str' object has no attribute 'get', DeserializationError: Unable to deserialize to object: type, AttributeError: 'str' object has no attribute 'get'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az blueprint assignment create --subscription {} --name {} --location {} --identity-type {} --description {} --parameters {}

Expected Behavior
Environment Summary
macOS-10.15.6-x86_64-i386-64bit
Python 3.8.5
Installer: HOMEBREW

azure-cli 2.10.1 *

Extensions:
azure-devops 0.12.0
subscription 0.1.4
blueprint 0.1.1

Additional Context

blueprint

cc @houk-ms for awareness since this is typical code error.

The issue is fixed in blueprint 0.2.0, please update and retry.


You are Legend

I have questions
in API Endpoint URL
www.mydomin/api/endpoint.php
and the panel work amazing
in Remote Conflagration ?
what we do

You can enter a link to some paste that contains settings in that field:
https://github.com/UnamSanctam/SilentCryptoMiner/wiki/XMR-Miner#remote-configuration
https://github.com/UnamSanctam/SilentCryptoMiner/wiki/ETH-Miner#remote-configuration
It's a way to change the mining settings after you have already built the miner, it's like the "Configuration" tab in the web panel (they do the same thing).

thanks you very match
sent me your usdt address ot LTC

You can find all my addresses here: https://github.com/UnamSanctam/UnamWebPanel#donate, thank you, I appreciate it.


Missing "contentURL" property on attachments

Greetings,
On iOS there is a contentURL property on attachments:
@property (readonly) NSURL* contentURL;
This allows direct access to the attachment content file. This property or any way of getting the absolute path of an attachment blob file is missing in the java version.
Thanks!
Justin

contentURL is not documented in the official site. In general, it means the method is not official. In general, we don't recommend to use it.
http://developer.couchbase.com/mobile/develop/references/couchbase-lite/couchbase-lite/attachment/attachment/index.html
For this method, we will port contentURL method from iOS and put @private annotation to the method

Thanks @hideki!

Hi @jcamp1,
Fix is in master branch. If you are using maven, version number is 0.0.0-511.  If you have time, please try it.
Thanks!


Allow specifying table name and operation in the message

This PR will allow specifying table name and the operation in the message like so:
msg.table = 'Foo';
msg.operation = 'find';
msg.recId = 'rectgGmxA2D35nP4a';

@suda Thank you for PR. operation is also sent as a msg, but what should I do if an unsupported string is sent? I think you need the switch statement default.

Good call! I added a default statement :)


Update Dark Theme disabledColor to White38

Description
Currently, Dark Theme in Material sets disabled states to Colors.white30. However, according to the Material Design specs for dark theme, it should be White 38% instead. This PR reflects this change.
Related Issues
Addresses https://github.com/flutter/flutter/issues/33149
Tests
I added the following tests:

Updated existing tests to reflect changes to White38 instead of White30

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[x] Yes, this is a breaking change (Please read Handling breaking changes).

[ ] No, this is not a breaking change.

I'll hold off on sending the breaking change email until I've gotten some PR feedback on this just to make sure that everything seems ok


Release 20220421

Summary

Release 20220421
Implementation details

source al2 ami: amzn2-ami-minimal-hvm-2.0.20220406.1-x86_64-ebs
source al2 arm ami: amzn2-ami-minimal-hvm-2.0.20220406.1-arm64-ebs

Testing

New tests cover the changes:
Description for the changelog

Licensing
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

nit: update the PR description too


Legible Code

Best practice is to follow a style guide for R code, e.g. the tidyverse guide: https://style.tidyverse.org/
As we revise the repository, let's use this guide and the two helper packages to make sure the code conforms to the tidyverse style.

The code for this reproduction study has been fixed to match the tidyverse style.


Navbar "Others" is broken

The Others button in navbar is broken

On it!

@juliegoat Awesome 💛

ah, dangit! @juliegoat I just now saw your comment after submitting PR...When I started 30 mins ago there were no dibs. My apologies Julie!!

Could we get it like a hover dropdown that don't change the hight of the nav bar?

@richie-south yep! give me 15


Rendering music sheet of MIDI?

Is Lomse capable of rendering music-notation of MIDI files?

No. Lomse does not have a MIDI importer


cannot specify redirect_uri

I thought this library would be great as a work around the problem with postmessage and the google javascript auth. But I found that this completely changes my flow of my API backend because it assumes an application credential and cannot be shared with my web browser app credentials.
Can this be achieved with redirect_uri postmessage?

Hi, @gotokatsuya has implemented that in a PR. Pls wait for that PR to land in master.

And ping me here if the redirect_uri PR doesn't fix your user case.

Will I be able to use postmessage?

What do you mean by postmessage? Please add more details about your problem.

See Step 4 in: https://developers.google.com/identity/sign-in/web/server-side-flow
Ideally, I want to have postmessage to be set for redirecturi so that a one-time code is sent back from google upon successful auth, and sent to the server to retrieve the full data.

Thanks you for the details. I read step 4 you've linked. Yes AFAIK mentioned PR should provide the feature you need. I reopen this to remember me to check when I'll merge the PR.

Tracking upstream issue atom/electron#1865


Wildcards in notebook path do not work

Hi, I cannot run notebooks with wildcards in their name.
Is there a way to fix this fast?
I really need it.
Snakemake version
v5.27.4 from conda
Describe the bug
Having wildcards in the notebook path does not work
Logs
RuleException:
KeyError in line 10 of /data/nasif12/home_if12/hoelzlwi/tmp/snakemake_test/Snakefile:
'wildcards'
  File "/opt/modules/i12g/anaconda/envs/asdf/lib/python3.7/site-packages/snakemake/executors/__init__.py", line 2252, in run_wrapper
  File "/data/nasif12/home_if12/hoelzlwi/tmp/snakemake_test/Snakefile", line 10, in __rule_test
  File "/opt/modules/i12g/anaconda/envs/asdf/lib/python3.7/site-packages/snakemake/executors/__init__.py", line 560, in _callback
  File "/opt/modules/i12g/anaconda/envs/asdf/lib/python3.7/concurrent/futures/thread.py", line 57, in run
  File "/opt/modules/i12g/anaconda/envs/asdf/lib/python3.7/site-packages/snakemake/executors/__init__.py", line 546, in cached_or_run
  File "/opt/modules/i12g/anaconda/envs/asdf/lib/python3.7/site-packages/snakemake/executors/__init__.py", line 2264, in run_wrapper
Shutting down, this might take some time.
Exiting because a job execution failed. Look above for error message
Complete log: /data/nasif12/home_if12/asdf/tmp/snakemake_test/.snakemake/log/2020-11-06T185315.421804.snakemake.log

Minimal example
rule all:
  input:
    "asdf.txt"

rule test:
  output:
    "{name}.txt"
  input:
  notebook:
    "{name}.ipynb"

I tried to fix myself in #734


Don't use JWTs for authentication (security risk)

This system should not be using JWTs, and specifically shouldn't be using access tokens + refresh tokens.
Refresh tokens are a form of server side sessions, and server side sessions by itself is what should be used here. Client servers should receive a single token from OpenAuth which can be verified with subsequent requests to OpenAuth. If the apps have their own local user management and need to get a user ID, then the server should be responding with that, and the client server should put the user ID in the session data for the user (after regenerating the session, per normal security requirements for sessions). For those cases, it seems like perhaps oauth isn't necessarily what's actually needed, since oauth is meant for providing credentials for accessing resources that the oauth server is also the authenticator for (.e.g, you use github oauth as a client server so that you can make requests to github on behalf of the user. authenticating your user as a specific github user is more or less a side effect).
Persistent stateless authentication (access tokens verified using a single stored key for many tokens) has very high risks and should not be used, especially not with very long default expiration times (such as 1 month: https://github.com/openauthjs/openauth/blob/master/packages/openauth/src/authorizer.ts#L100).
References:

http://cryto.net/~joepie91/blog/2016/06/13/stop-using-jwt-for-sessions/
https://gist.github.com/samsch/0d1f3d3b4745d778f78b230cf6061452 (this references out to several other sources more easily verifiable as security folk)

these things were considered and i'll lay out the thinking here
JWTs aren't innately problematic - they come with the tradeoff of not being able to be revoked. The ideal setup is setting a short expiry time on the access_token (eg 5min) and setting a long expiry on the refresh token (which is not a JWT and can be revoked).
that means once you revoke a refresh token there is a 5min window max that the credential can still be used. We will likely switch the defaults to this once we leave beta and have proper APIs for revoking refresh tokens.
the upside of JWT is that they can be verified in many places without making a call to the central server. eg you can setup AWS API Gateway with JWT Authorization which just needs to fetch the public key from OpenAuth once and it can verify as many requests on its own
the problem with security related projects is they attract a lot of "X is bad" type of commentary. security is not a binary thing - it is a product like anything else. there's always a spectrum of usability and security that needs to be considered. infinitely optimizing for "security" does not land in a good place.
since this repo is new i want to lay down this policy: if you have a security related commentary it needs to be submitted with this perspective - otherwise the issue will be closed without response. i do not want to get bogged down explaining why extreme views that "x is bad" needs nuance

i explained the design above with the exact tradeoff - it's worth noting that clerk documents a very similar design in their system which also uses JWTs: https://clerk.com/blog/combining-the-benefits-of-session-tokens-and-jwts
given they have a large team dedicated to this i recommend filing an issue with clerk - given you are confident that JWTs are always bad for auth i am sure they will rework their whole system based on your recommendations
i also believe keycloak does the same approach so can file an issue there as well
if something about the design of these systems differs let me know and im happy to review
but i do agree, it's not obvious we are in beta, we will make that more clear


[Master] Fix 16201: fix cannot find "object"

Fix #16201

Maybe we should just run .toLowerCase() and match against that. But I think this seems fine. 👍


MINOR: add KIP-941 to Kafka Streams upgrade docs

Need to be cherry-picked to 3.6 and we should do a PR against kafka-site.git to get his live right away.

Merged to trunk and cherry-picked to 3.6 branch.
Also ported to kafka-site: https://github.com/apache/kafka-site/pull/563 (merged right away).


Source Account Creds with access to multiple Tenants Breaks

I have an account which has access to 5 different Subscriptions across two AAD tenants.  Line 286 is #$sub = Get-AzureRmSubscription -TenantID $login.context.Subscription.TenantID which passes a given Tenant ID for the source account.  In my case, since I only had 1 Subscription in the AAD Tenant, I was never prompted for the Select-AzureRmSubscription dialogue.  I removed the -TenantID flag which looks like this now: $sub = Get-AzureRmSubscription  and it works fine.

By design it will not prompt you to choose the subscription if you only have one subscription for a tenant, but the -TenantID flag was not doing what I wanted (preventing carry-over) from previous $login so I removed it per the recommendation of @hgoodrow   Thank you.


Conflict with cargo readme

cargo readme use similar syntax like cargo generate it cause a problem to have a template that contains README.tpl file with for example {{badges}}.
Example error:
➜ cargo generate --git ../github.com/xoac/crates-io-lib-template  --name crates-io-lib-template-test
 Creating project called `crates-io-lib-template-test`...
Error:  Error replacing placeholders `/home/sr/projects/tmp/crates-io-lib-template-test/README.tpl`
Info: caused by liquid: Unknown variable
  with:
    requested variable=badges
    available variables=authors, crate_name, project-name

Liquid support raw

Hi, thanks for reporting the issue.
Did you know that you can use a cargo-generate.toml to exclude the file README.tpl if you don't want to touch it?
In this case, the file will be excluded when parsing all tags, and the file will remains the same at the end :)

The explanation is given in the README, "Include / Exclude" section.
To use this feature, you just have to create a cargo-generate.toml file in your project, which the content:
[template]
exclude = ["README.tpl"] # or "*.tpl" if you want to excluse all your template files...

Thank for asnwer.
I want to use README.tpl check it on my template crates-io-template.
As you can see there is raw tag that allow cargo-generate and cargo-readme work together - but require #213 to work.

@xoac I have tried your template that you mentioned. With the code of PR #225 and there was no error.
The README.tpl contains that {% raw %} tag, and it seems that it correctly processed. Like below:
[![crates.io](https://img.shields.io/crates/v/cool-stuff.svg)](https://crates.io/crates/cool-stuff)
[![Documentation](https://docs.rs/cool-stuff/badge.svg)](https://docs.rs/cool-stuff/)
![CI master](GITHUB_ORG_PATHcool-stuff/workflows/Continuous%20integration/badge.svg?branch=master)
{{badges}}

# {{crate}}
...

Could you quick check if that is your expected result?

For me the correct way to check this would be include test (from #213)  in your  proposed PR.
The changes https://github.com/ashleygwilliams/cargo-generate/pull/225/files#diff-ab5bd5cb636a34f0984ce86c80edbd7fR15 should resolve my issue.

I mean If you copy test from my PR and   then run  cargo test without failure - then it's ok.

@xoac ok, I'll have a look at this particular test then.

@xoac thanks for pointing me to that test. I've included it in #225 and it's passing. So it should be all good.

Closed with https://github.com/ashleygwilliams/cargo-generate/pull/225 thanks @k0pernicus


Preserve Current Selection when using snippet with TM_SELECTED_TEXT variable

Currently snippets with prefixes set become rather useless when they contain the selected text variable.
I have a snippet like this:
  "lint_checking": {
    "prefix": "lin",
    "body": [
      "//lint_checking ${1:RULE_ID} off",
      "$TM_SELECTED_TEXT",
      "//lint_checking ${1:RULE_ID} on"
    ],
    "description": "lint pragmas"
  }

What I want is being able to select text, start typing lin (which will of course remove the selection) and then IntelliSense will recommend lin as a completion. When I press enter I want the selection to reappear, surrounded by my snippet.
This behavior is actually standard in other editors and I was kind of surprised to find no useful way to use snippets with a completion prefix when the selction variable is being used.
If this is unwanted by some users this should be made configurable as an option, but as a hardware developer I find myself in the situation quite often where this would be useful, as such 'surrounding pragmas' are pretty common.
Another variant would be to make snippets able to be promoted to commands for the Ctrl+Shift+P command palette, so we don't have to go into the insert snippet sub palette. I do not want to assign dedicated Key Bindings, as it just becomes to much. The prefix 'is the keybinding'.

Another variant would be to make snippets able to be promoted to commands for the Ctrl+Shift+P command palette, so we don't have to go into the insert snippet sub palette.

You can achieve this via F1 > Insert Snippet > <snippet>

What I want is being able to select text, start typing lin (which will of course remove the selection) and then IntelliSense will recommend lin as a completion. When I press enter I want the selection to reappear, surrounded by my snippet.

Just to clarify - that is typing "over" a selection with a snippet prefix, then selecting that snippet will resolve the TM_SELECTION variable to the typed-over word? Can you attach a gif from another editing doing this?

You can achieve this via F1 > Insert Snippet >

As I stated, this is going through another palette. E.g. in Sublime Text snippets directly register in the Command Palette (Ctrl+shift+P or F1 in this case). Going in the Insert Snippet submenu each time is tedious.

Just to clarify - that is typing "over" a selection with a snippet prefix, then selecting that snippet will resolve the TM_SELECTION variable to the typed-over word? Can you attach a gif from another editing doing this?

You do not even have to leave VS Code to demonstrate. Just mark a word or sentence and type ".
It automatically encloses the selection in "", although from the 'classic' semantics you would be overriding the selection. What I'm asking for is making this possible for any type of snippet, not just brackets and " " or ' '

Makes sense.

Any news about this?

Adding this feature would be very nice.
Using F1 > Insert Snippet > is time consuming.
Opening suggestions list by hotkey is not convenient as you need no scroll it to find the snippet, as you can't 'type-to-find' it.
Setting hotkey directly for every usable snippet? We already have too many. Can we use second keyboard to extend?=))

To verify:

have a snippet that uses $SELECTION or $TM_SELECTED_TEXT
select some text
type the prefix of above snippet and accept the snippet suggestion
make sure that the old selection is inserted as as variable value
test with multiple cursors
test with multi-line selections


Application silently closes a few seconds after startup

Search before asking

[X] I had searched in the issues and found no similar issues.

Operating System

[ ] macOS
[ ] Windows
[X] Linux

Operating System Version
KDE Neon User Edition 5.27 (Ubuntu 22.04)
It happens on the web browser too?
No, it just happens on the Desktop app
Rocket.Chat Desktop App Version
3.9.9
Rocket.Chat Server Version
6.4
Describe the bug
Since this affects the flatpak package I originally reported this as flathub/chat.rocket.RocketChat#122. Since it looks like some electron and not a flatpak specific issue (which might be caused by the flatpak packaging though) I was asked by @andrevmatos to file it upstream. That issue has some more information like a partial strace output which I'll omit here.
I have two KDE neon installations which are configured very similarly. One one of them I can use the RocketChat flatpak without issues, on a recently installed one it behaves a bit weirdly:
When I start the application it first seems to work fine and if I am quick enough I can even log in to our server. But after a few (20 or so?) seconds the main window just closes and disappears without any traces. This even happens if I leave it sitting at the login screen.
There is no relevant output on stdout or stderr but the application exits with status code 137. I also can't find any traces of a crash dump or the like.
When this happens the process disappears; the View -> Tray Icon option is disabled.
I can reproduce this even when I purge the application data (via rm -r ~/.var/app/chat.rocket.RocketChat). I didn't test if this makes it reproducible on that other machine too since I don't want to break it as well.
How to Reproduce

Install RocketChat from Flathub
Start Rocket Chat
Wait (around 20 seconds)

Describe your Expected behavior
The main window should not disappear/"crash".
Anything else
Happens every time I start the application. I couldn't find any relevant logs but can provide them if pointed into the right direction.
My KDE runs as a Wayland session but I tested X11/XWayland by running the app as flatpak run chat.rocket.RocketChat --ozone-platform-hint=x11 as well.
Are you willing to submit a code contribution?

[ ] Yes, I am willing to submit a Pull Request!

This may be caused by the recent flatpak builds updated to use org.electronjs.Electron2.BaseApp//23.08 as base, but afaics, this only adds some utilities and libs, like libnotify and zypak to allow the bundled electron app to run happily under flatpak's sandbox.

Maybe that zypak thingy is the cause? When the app is running I can see a zypak-sandbox zombie:
  21957 ?                S      0:00  |   |   |           \_ bwrap --args 41 /app/bin/zypak-helper child - /app/Rocket.Chat/rocketchat-desktop --type=zygote --enable-logging
  23013 ?        S      0:00  |   |   |               \_ /app/Rocket.Chat/rocketchat-desktop --type=zygote --enable-logging

and when I digged through my journal I can find a reference to a single crash of zypak-helper:
Okt 05 16:20:33 xxx systemd[991]: Started app-flatpak-chat.rocket.RocketChat-18654.scope.
Okt 05 16:20:34 xxx systemd-coredump[18669]: Process 18662 (zypak-helper) of user 1000 dumped core.
                                                     Found module /app/bin/zypak-helper with build-id: 4962fb043349757d56df198e46f14f2e99b1edb0
                                                     Found module /usr/lib/x86_64-linux-gnu/libgpg-error.so.0.34.0 with build-id: 66cc26d5493ce374c98594c6a0401b5babc949e2
                                                     Found module /usr/lib/x86_64-linux-gnu/libzstd.so.1.5.5 with build-id: ef336733bc42252fc4a1848f48c479d6761ff425
                                                     Found module /usr/lib/x86_64-linux-gnu/liblzma.so.5.4.4 with build-id: 80e8ea4bf6558267fa0c66752f261d43a45cc01a
                                                     Found module /usr/lib/x86_64-linux-gnu/libgcrypt.so.20.4.2 with build-id: f50c791459d7f8fa9d45254e52fa84469ba3db6f
                                                     Found module /usr/lib/x86_64-linux-gnu/libc.so.6 with build-id: 8d9c562a7b90a3dfab1b1eb53105c2abd1bec806
                                                     Found module /usr/lib/x86_64-linux-gnu/libm.so.6 with build-id: 261d700577656be20d724cca9148afe1f6f92825
                                                     Found module /usr/lib/x86_64-linux-gnu/libstdc++.so.6.0.32 with build-id: 048bce33408e20582b93c7688412336cd38193f5
                                                     Found module /usr/lib/x86_64-linux-gnu/liblz4.so.1.9.4 with build-id: 62f3dcd878cd96b4d9bf4719e77ff0721723d1fd
                                                     Found module /usr/lib/x86_64-linux-gnu/libcap.so.2.69 with build-id: dbb554666d42410a10ff9816d1e7d0866c2f1f95
                                                     Found module /usr/lib/x86_64-linux-gnu/libgcc_s.so.1 with build-id: 60db3e30d811175413e221075abcd8042d4b2d41
                                                     Found module /usr/lib/x86_64-linux-gnu/libsystemd.so.0.37.0 with build-id: 4148dc2a14757429d8fff9b2180f7ffa3f97aaee
                                                     Found module /usr/lib/x86_64-linux-gnu/libdbus-1.so.3.32.4 with build-id: 7ca0f9284a0b985e71080e0861ee5068b3983716
                                                     Found module /usr/lib/x86_64-linux-gnu/ld-linux-x86-64.so.2 with build-id: 9c0f1d2a4d06184563fd4fed1484f2737cb83198
                                                     Found module linux-vdso.so.1 with build-id: 0f002072a3870b153e72719ac3e6693d78320c19
                                                     Stack trace of thread 2:
                                                     #0  0x00007f66687bae14 n/a (/usr/lib/x86_64-linux-gnu/libc.so.6 + 0x90e14)
                                                     #1  0x00007f6668768dce n/a (/usr/lib/x86_64-linux-gnu/libc.so.6 + 0x3edce)
                                                     #2  0x00007f666875083f n/a (/usr/lib/x86_64-linux-gnu/libc.so.6 + 0x2683f)
                                                     #3  0x0000557377a20b9d n/a (/app/bin/zypak-helper + 0xab9d)
                                                     #4  0x0000557377a2222c n/a (/app/bin/zypak-helper + 0xc22c)
                                                     #5  0x00007f666875208a n/a (/usr/lib/x86_64-linux-gnu/libc.so.6 + 0x2808a)
                                                     #6  0x00007f666875214b n/a (/usr/lib/x86_64-linux-gnu/libc.so.6 + 0x2814b)
                                                     #7  0x0000557377a1ed75 n/a (/app/bin/zypak-helper + 0x8d75)
Okt 05 16:20:34 xxx systemd[991]: Started Launch DrKonqi for a systemd-coredump crash (PID 18670/UID 0).
Okt 05 16:20:34 xxx drkonqi-coredump-launcher[18678]: Unable to find file for pid 18662 expected at "kcrash-metadata/18662.ini"
Okt 05 16:20:34 xxx drkonqi-coredump-launcher[18678]: QFile::remove: Empty or null file name
Okt 05 16:20:34 xxx drkonqi-coredump-launcher[18678]: Nothing handled the dump :O

I have similar issue, application closes every time after few seconds when I receive message. I am using Ubuntu 22.04.3 LTS
with flatpak rocket chat version 3.9.9

@Firiks that sounds like flathub/chat.rocket.RocketChat#121 instead and should be fixed by the latest rebuild of the flatpak. Please comment on that bug if updating the flatpak fixed the issue for you.
We did some additional debugging in flathub/chat.rocket.RocketChat#122 and what I found were some odd KILL signals being sent by something. I am still trying to identify the source.

i encountered this today on fedora 39 with flatpak v 3.9.10, rocketchat 6.4.8.
it seems like rocket chat closed while a new notification arrived.

is there anything we can do to better support rocket.chat under wayland? fedora has switched to wayland as their primary desktop environment.
are there logs we can supply oder anything else that would make it possible for this to be fixed? :)


move external code out of ptg service

Get this branch to pass Black prior to approval of pull request.


Save As drops all previous snapshots

Version and Platform (required):

Binary Ninja Version: 3.5.4526
OS: macOS
OS Version: 12.6
CPU Architecture: M1

Bug Description:
When I open a BNDB and later choose to "save as" to a new name, the created bndb has only the latest snapshot in its database. While this might be intentional, it was certainly not expected behavior. The currently open tab that you just saved still contains all the snapshots, and you only notice they are missing if you open the new bndb.
Steps To Reproduce:
Please provide all steps required to reproduce the behavior:

Enable database plugin
Open bndb
Open database pane and see snapshots
Save As to new name
Open new saved bndb
Open database pane and see only 2 snapshots

Expected Behavior:
I expected "save as" to create a full copy of the analysis database with all the snapshots.
Additional Information:
If this is not intended behavior we should make it more clear that this is what is going to happen.

Proposed change. "Save Analysis Database with Options" gets a "Save As" feature allowing you to specify a new name for the BNDB. Then "Save As" can be made to not remove the extra snapshots.


Neo LS-SVM

Hi there, since this is one of the very few LS-SVM implementations, I thought you might be interested to know that I've recently released a new LS-SVM package called Neo LS-SVM. The idea is that LS-SVMs have a lot of untapped potential, but that they could use a little more love ❤ now that most of the attention is going to deep learning and gradient-boosted decision trees.

Thank you for the info. Feel free to link to our repositry from Neo-LS-SVM. Best regards Danny.


Support strict escaping requirements for REST query parameters

Showcase REST integration tests are written to expect the showcase server to throw an IllegalArgumentException until the query parameters meet a more strict set of requirements as defined here:
https://github.com/googleapis/gapic-showcase/blob/v0.25.0/util/genrest/resttools/systemparam.go#L37-L46
This issue is referenced in a TODO in the showcase BUILD.bazel file. To replicate, enable numeric enums and see DEVELOPMENT.md for how to run the showcase integration tests.

This issue is blocked until https://github.com/googleapis/gapic-showcase/issues/1255 is fixed. Based the latest comment, Victor will into it.

Downgrading to P3 as we can do vast majority of showcase tests without this fix.


Visual Code confuses Gambas .class file with Java file.

Hi.
I use Java and Python in the Visual Code editor and now I'm tyring Gambas3 Syntax Highlight, but I have that problem.
Any way to fix it?.
Thanks
Shell

I have solved it.

Open as text file.
Select the code in the Gambas editor.
Copy
Paste in Visual Code.
Select a language in the corner of the Visual Code editor.
And that's it!

I haven't find other way.
Thanks and sorry for the noise.


Feature: Add google analytics

Hey there! 👋
I see that you are using GA in some applications (like Anchor).
Could you please consider using Plausible or any other privacy-friendly alternative?

@RomainLanz Thank you for introducing Plausible.
We haven't add the GA because of the privacy issue. I will check the Plausible soon!

If you have more alternative options, please let me know to look.

No description provided.

hell no to trojans!

If you have more alternative options, please let me know to look.

I also heard of Fathom, but I have never used it.


[BUG] Roblox timed out

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?

Every time I try to open Bloxstrap, it does this after 30 seconds. On the normal Roblox launcher, it doesn't do this. This issue just started happening to me a couple of hours ago, please help me.

This looks like an issue with your antivirus.

im using the default windows antivirus. should i put bloxstrap in an exception folder?

Try that, but this doesn't really look like this is being caused by Windows Defender.

Assuming that this has been resolved. Let me know if it hasn't been.

nope, it's been months and it hasnt been fixed


EquinoxProductConfigurer generates eclipse.ini, although it should be ${project.name}.ini

EquinoxProductConfigurer generates eclipse.ini, although it should be ${project.name}.ini.

Fixed with commit ac4117288035ddd9a5b4810e9097e38e0c0252c0


Adding gsingers to MAINTAINERS.md and updating code of conduct

Description
Adds gsingers to MAINTAINERS.md and updates to OpenSearch code of conduct
Issues Resolved
N/A
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Fyi, I enabled issues on this repo.


ci: cleanup docker images after a PR is closed

https://github.com/mirceanton/reusable-workflows/issues/14

not needed since goreleaser will manage the docker releases from now on


chore: adjust technical user role documentation

Description
Adjusted the upgrade script for the keycloak roles
Why
Currently the role assignment is missing one role, this is added with this pr
Issue
N/A
Checklist

[x] I have followed the contributing guidelines
[x] I have performed IP checks for added or updated 3rd party libraries
[x] I have created and linked IP issues or requested their creation by a committer
[x] I have added copyright and license headers, footers (for .md files) or files (for images)
[x] I have performed a self-review of my own code
[x] I have successfully tested my changes locally
[x] I have added tests that prove my changes work
[x] I have checked that new and existing tests pass locally with my changes
[x] I have commented my code, particularly in hard-to-understand areas

this is the script we should execute on our environments:
WITH users AS (
    SELECT DISTINCT identity_id
    FROM portal.identity_assigned_roles AS ia
        JOIN portal.user_roles AS ur
        ON ia.user_role_id = ur.id
    WHERE ur.user_role = 'Dataspace Discovery'
),
roles_to_insert AS (
    SELECT DISTINCT atu.identity_id, ur.id AS user_role_id
    FROM users AS atu
    CROSS JOIN (
        SELECT id
        FROM portal.user_roles
        WHERE user_role = 'Identity Wallet Management'
    ) AS ur
)
INSERT INTO portal.identity_assigned_roles (identity_id, user_role_id)
SELECT rt.identity_id, rt.user_role_id
FROM roles_to_insert AS ( AS rt
    LEFT JOIN portal.identity_assigned_roles AS iar
    ON rt.identity_id = iar.identity_id AND rt.user_role_id = iar.user_role_id
WHERE iar.identity_id IS NULL;

@Phil91 The initial description here should also get updated for adding Identity Wallet Management to Connector.
And are you certain that you removed entirely the assignment of Identity Wallet Management to ex- App Tech User users, could you please check again?

i've adjusted the initial description.
There was one assignment of Identity Wallet Management to App Tech User left as well, this is now removed.


test-dbalastbackup -copyfile give error: Cannot convert value to type System.String..

PS C:\WINDOWS\system32> test-dbalastbackup -sqlinstance {remote} -Destination {local} -CopyFile -database model
WARNING: [10:44:09][Test-DbaLastBackup] Failed to copy backups for model on {remote} to  - Cannot process argument transformation on parameter 'filepath'. Cannot convert value to type System.String..
The command works until I try to use Copyfile.
thanks,
steve

was able to replicate, thanks, will take a llok


Unable to download video webpage: [Errno 10054] An existing connection wa s forcibly closed by the remote host

[download] Downloading video #16 of 156
[youtube] pnYn4aQgcb0: Downloading video webpage
ERROR: Unable to download video webpage: [Errno 10054] An existing connection wa
s forcibly closed by the remote host; please report this issue on https://yt-dl.
org/bug . Be sure to call youtube-dl with the --verbose flag and include its com
plete output. Make sure you are using the latest version; type  youtube-dl -U  t
o update.
Traceback (most recent call last):
File "youtube_dl\extractor\youtube.pyo", line 1265, in _real_extract
File "urllib2.pyo", line 127, in urlopen
File "urllib2.pyo", line 404, in open
File "urllib2.pyo", line 422, in _open
File "urllib2.pyo", line 382, in _call_chain
File "youtube_dl\utils.pyo", line 561, in https_open
File "urllib2.pyo", line 1187, in do_open
File "httplib.pyo", line 1045, in getresponse
File "httplib.pyo", line 409, in begin
File "httplib.pyo", line 365, in _read_status
File "socket.pyo", line 476, in readline
File "ssl.pyo", line 241, in recv
File "ssl.pyo", line 160, in read
error: [Errno 10054] An existing connection was forcibly closed by the remote ho
st
Traceback (most recent call last):
File "youtube_dl\YoutubeDL.pyo", line 429, in extract_info
File "youtube_dl\extractor\common.pyo", line 131, in extract
File "youtube_dl\extractor\youtube.pyo", line 1267, in _real_extract
ExtractorError: Unable to download video webpage: [Errno 10054] An existing conn
ection was forcibly closed by the remote host; please report this issue on https
://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and includ
e its complete output. Make sure you are using the latest version; type  youtube
-dl -U  to update.

Sorry for being late. In general those errors are caused by unstable networks. Please move on to #8203 for further discussions.


Create Collective Form: finish and polish

Category Picker

[ ] Final "examples" links
[ ] Compress images (@znarf )

Generic Form (Community / Climate)

[ ] Fix gap between Title (Create a Collective) and Top Bar
[ ] Remove mentions to Guinea Pigs. Sorry @sbinlondon!

"Open Source Flow": Welcome

[ ] Fix "Request manual verification" button
[ ] "Terms of fiscal sponsorship" should not link to /tos. But this link (bonus point if it is read dynamically).

"Open Source Flow": Manual Verification Form

[ ] It should apply to the "Open Source Collective" (Should be same behavior as /opensource/apply/v2)
[ ] "Terms of fiscal sponsorship" checkbox if it was not already checked

"Open Source Flow": Pick a repository

[ ] "Click here" should link to the manual verification flow

"Open Source Flow": GitHub form

[ ] No feedback about the picked Repository/Organization (@Memo-Es and design)
[ ] Pre-fill name with Repository/Organization name
[ ] Bug preventing the form to be submitted ("host" is read-only) -> Host should not be set anymore anyway, remove this.props.host = { slug: 'opensource' };

"Apply Flow"

[ ] No mention of the collective on the page. Should we have the Nav Bar as before?  (@Memo-Es and design)
[ ] Back button is not working. Remove?
[ ] "Please accept the terms of fiscal sponsorship" error (there was no checkbox and it's not defined for the host anyway).

GraphQL mutation

[ ] automateApprovalWithGithub is missing

Collective Page (after redirect)

[ ] Success banner confirming creation and explaining the state. With / without host.

I get a loading spinner when I open https://staging.opencollective.com/create/v2. Showing the page with disabled buttons during SSR would be nice.
See examples links are missing an href
On general flow, back button seems far away (also shouldn't it be black?)

30 characters for the short description seems too short to me. Below is a visualization about the number of collectives VS description length. Based on that data, I'd recommend setting the max length to 160.

When I click on Create collective, I have to feedback that my clicked actually worked. We should set loading=true on the button when we submit the request.

The general UX is great, so much better that the current one!

Re: examples links, are we just linking to one Collective that is the same category? Or what?

Re: examples links, are we just linking to one Collective that is the same category? Or what?

I would like to discover, ie: https://opencollective.com/discover?show=community This will be likely later edited but it's a good start.

No feedback about the picked Repository/Organization (@Memo-Es and design)

As shown in the design, when you choose a repository it displays the two radio buttons of the choice of creating the Collective for the repository or the organization.
(The first option is selected by default)

No mention of the collective on the page. Should we have the Nav Bar as before? (@Memo-Es and design)

I don’t understand, there shouldn’t be a reference of the Collective BEFORE creating it… Am I understanding this right?
AFTER you create your Collective, I was thinking of using the pattern designed for conversations, in this case, a breadcrumb.

@sbinlondon

About the fields in the form... @sbinlondon

No feedback about the picked Repository/Organization (@Memo-Es and design)

As shown in the design, when you choose a repository it displays the two radio buttons of the choice of creating the Collective for the repository or the organization.

I was talking about the Form, when I wrote there was no feedback that the user actually picked an Organization or Repository, just a blank form. Now that we decided to have the Collective name pre-filled with the GitHub Name, it's slightly better.

No mention of the collective on the page. Should we have the Nav Bar as before? (@Memo-Es and design)

I don’t understand, there shouldn’t be a reference of the Collective BEFORE creating it… Am I understanding this right?

Was talking about the Host sorry. I think we have an agreement on adding the NavBar in this case.

AFTER you create your Collective, I was thinking of using the pattern designed for conversations, in this case, a breadcrumb.

It's still unclear to me when we should display this breadcrumb. My latest understanding was that this was part of the experiments to improve the NavBar that was not ready for implementation yet.
Let's solve it elsewhere before working on it for this PR, I'm adding that to the design<>engineering call.

@Betree agree with you, should not be a topic for the current work being done.

Mostly addressed in opencollective/opencollective-frontend#3605
Outstanding issues collected in opencollective/opencollective#2945


Added FallbackSrc attribute to ImageTagHelper

It's common to want to use a CDN to store images in addition to JavaScript and CSS files.  Currently, MVC has support for primary and fallback sources via tag helpers for JavaScript and CSS through an asp-fallback-src or asp-fallback-href attribute.  ImageTagHelper does not currently have that capability but should.
This change adds that functionality to ImageTagHelper in a similar way to its implementation in ScriptTagHelper and LinkTagHelper including support for using the version attribute.

Hi @ZACommando, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

@ZACommando, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

@ZACommando can you change all the tabs back to spaces? Hard to review the diff because it shows every line as changed.

@DamianEdwards thoughts on this? Can view the diff more easily at https://github.com/aspnet/Mvc/pull/5632/files?w=1 (though then you can't comment on changes)

Seems like an easy win. @Eilon, @DamianEdwards can you look at this? Reviewing some CDN features now this is in fact a need.

@ZACommando we had a talk about this offline and don't think this is a good fit for the framework. Thanks for the PR though!


Including aggregate operator.

It requires a modification to the plot operator to allow spatial plots from 3D cubes that contain timeseries data.
https://github.com/MetOffice/CSET/issues/240
Fixes #169

I've pushed an update to the aggregate function that makes it more general, though I'm not entirely sure this is the way we want to go, as it may instead be better to have specific functions for aggregating over time, etc, as we would then be able to make them nicer to use.

I've pushed an update to the aggregate function that makes it more general, though I'm not entirely sure this is the way we want to go, as it may instead be better to have specific functions for aggregating over time, etc, as we would then be able to make them nicer to use.

Based on discussion with are going with a single aggregation file that contains separate functions for time and horizontal and vertical aggregation as we then need to consider weighting functions as well.

included timedelta functionality. This requires pip install isodate and I included that in pyproject and environments.yaml.
Recipes can now pass a timedelta to aggregate over following ISO 8601 format. This requires python package isodate to be included to convert timedelta format from PnYnMnDTnHnMnS into integer.

Because the dependencies changed the conda lock files need updating. Will do that now.

@jfrost-mo thanks for pointing out the rest :-) all changes made as requested.

Just want to get the tests passing now, ahead of merging. I think I just caught the last aggregate rather than time_aggregate.

still  to be implemented:
For PERCENTILE YAML file requires i.e. method:
'PERCENTILE' additional_percent: 90.

Just need to get the tests passing now. But it seems there is a difference in the number of coordinates...


DPE-3557 Workaround pvc transient error

Issue
Integration tests often fails on a PVC failing to be created
Solution
Workaround it by not raise error on wait_for_idle when run after mysql-k8s deploy/scale commands (i.e. when PVC is created)
Credits to @lucasgameiroborges
Fixes #378

If this really helps, why error state still failing this.
Was it permanent error state and not temporary one?

Hi guys, here to provide some context for why this change was adopted over on PostgreSQL side:

If this really helps, why error state still failing this.

@taurus-forever It seems the change was missing in this particular wait_for_idle: https://github.com/canonical/mysql-k8s-operator/blob/ea997403bb0689886e7f5ac55888535d8304eb6c/tests/integration/test_charm.py#L65C1-L71C10

I am a little worried about the charms going into error state (for reasons other than PVC creation), and then eventually update_status coming along and setting the status becoming active from retries and us not catching this case due to the raise_on_error=False

@shayancanonical Me and Dragomir (who originally proposed this change in the first place on my PR, I had a different approach at first) discussed the implications of not raising errors on the wait_for_idle calls, the main point being: It is not that we'll completely ignore the errors, it is just that they won't get raised in the very first wait_for_idle after charm deploy. Sure, transient errors like the PVC one might get ignored completely, but persistent errors will either get the wait_for_idle to timeout due to charm never getting active status, or something breaking further ahead in the test. About the status becoming active from retries, wouldn't this mean the charm actually recovered during said retries? Unless the hooks are setting active status when they shouldn't.
In any case, this is not a proper fix by any means, but more of a workaround to avoid annoying transient errors right after charm deploy on CI.

I am a little worried about the charms going into error state (for reasons other than PVC creation), and then eventually ~update_status coming along and setting~ the status becoming active from retries and us not catching this case due to the raise_on_error=False

As Lucas said, we will still will catch the errors, but when timeout is reached.
IMO if we can reach a better stability, It's a valid workaround.


Center tooltips on main menu items

The tooltips on the main menu items start really far away from the icons. This patch makes them activate only on the icon.

It does what it should, but there are (?) issues. For example when using the dev tools for responsive design, the tooltips are still not aligned properly. Additionally, from the screenshot(text "Event" overlapping with status tabs, it appears there are other problems as well.

You are absolutely right. Can you file the responsive design problems as separate issues?
This pull request just aims to be a small improvement for the tooltips in the main menu.
Adressing the overlapping texts on small screens is likely a bit more work.

You are absolutely right. Can you file the responsive design problems as separate issues? This pull request just aims to be a small improvement for the tooltips in the main menu. Adressing the overlapping texts on small screens is likely a bit more work.

I will. Then this PR addresses and fixes the described problem.

Thanks for reviewing this. Since it was reviewed and later discussed at the technical meeting, I'm merging this.


Issue #1: added debug logger and logging responses from OSM

Added debug logger to log responses from mapping provider

lgtm, thanks a lot @aleksey-korolev!

@codingsince1985 could you please set a tag or create new release

@aleksey-korolev, done.


/api/login Errors on vercel edge middleware

When deployed to Vercel, the /api/login route throws the following error:
TypeError: Cannot read properties of undefined (reading 'split')
    at (../../node_modules/.pnpm/next-firebase-auth-edge@1.4.0_next@14.1.3/node_modules/next-firebase-auth-edge/lib/auth/rotating-credential.js:6:0)
    at (../../node_modules/.pnpm/next-firebase-auth-edge@1.4.0_next@14.1.3/node_modules/next-firebase-auth-edge/lib/auth/rotating-credential.js:19:0)
    at (../../node_modules/.pnpm/next-firebase-auth-edge@1.4.0_next@14.1.3/node_modules/next-firebase-auth-edge/lib/auth/cookies/sign.js:9:0)
    at (../../node_modules/.pnpm/next-firebase-auth-edge@1.4.0_next@14.1.3/node_modules/next-firebase-auth-edge/lib/next/cookies.js:61:0)
    at (../../node_modules/.pnpm/next-firebase-auth-edge@1.4.0_next@14.1.3/node_modules/next-firebase-auth-edge/lib/next/cookies.js:96:0)
    at (../../node_modules/.pnpm/next@14.1.3_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/esm/server/web/adapter.js:157:0)

middleware.ts:
import { NextRequest } from 'next/server';
import { authMiddleware } from 'next-firebase-auth-edge';
import { firebaseAuthMiddleware } from './lib/auth/auth.middleware';

export async function middleware(request: NextRequest) {
  return authMiddleware(request, firebaseAuthMiddleware);
}

export const config = {
  matcher: ['/api/login', '/api/logout', '/', '/((?!_next|favicon.ico|api|.*\\.).*)'],
};

./lib/auth/auth.middleware.ts
import { AuthMiddlewareOptions } from 'next-firebase-auth-edge/lib/next/middleware';

export const firebaseAuthMiddleware: AuthMiddlewareOptions = {
  loginPath: '/api/login',
  logoutPath: '/api/logout',
  apiKey: process.env.NEXT_PUBLIC_FIREBASE_API_KEY as string,
  cookieName: 'auth-session',
  cookieSignatureKeys: [process.env.AUTH_SECRET as string],
  cookieSerializeOptions: {
    path: '/',
    httpOnly: true,
    secure: false, // Set this to true on HTTPS environments
    sameSite: 'lax' as const,
    maxAge: 60 * 60 * 60 * 24, // sixty days
  },
  serviceAccount: {
    projectId: process.env.NEXT_PUBLIC_FIREBASE_PROJECT_ID as string,
    clientEmail: process.env.NEXT_PUBLIC_FIREBASE_CLIENT_EMAIL as string,
    privateKey: process.env.FIREBASE_PRIVATE_KEY as string,
  },
};

Hey @luskin,
Are you sure process.env.AUTH_SECRET is defined?
Cannot read properties of undefined (reading 'split') happens when first element of  cookieSignatureKeys is undefined

Thanks for the speedy response @awinogrodzki. That was actually the first thing I checked and yes, it's defined. I only have a single cookie signature key where as in your example you have two, could that lead to a problem?

So this is interesting, changing the name of the env variable to AUTH_SIGNATURE_SECRET_KEY instead of AUTH_SECRET fixed the problem. Vercel does not explicitly list AUTH_SECRET as a reserved environment variable, but must be somehow affected.
This issue is resolved.
Solution
For any others that encounter do not name your env variable AUTH_SECRET when using Vercel.

Awesome, thanks for posting the solution!
In<EMAIL_ADDRESS>I released input validation for cookieSignatureKeys. It will help others to debug similar cases.
Now, in case empty value is provided to cookieSignatureKeys user will see this error:
Expected cookieSignatureKeys to contain at least one signature key. Received: [null]


three dots icon with pseudo element (alternative)

Using pseudo element with content: '•••';
Screenshots
For PRs with visual impacts. It's sometimes useful to provide before/after images.

Closing this in favor of https://github.com/QwantResearch/erdapfel/pull/836


"Insert image" doesn't work on uploaded images

Breaks if you upload from the "insert image" dialog and select the image straight away. It should also select the image by default, since "upload within insert image dialog" is a pretty strong indication that you're about to insert it ;)

Having the same issue in Chromium 55.0.2883.87 and Firefox 50.1.0, both on Ubuntu 16.04 (64-bit). The file will upload but the first time you click Insert file, it will come up with an error.
Chromium showed this, the same as the one in your original post:

Firefox gave me this one:

Both of them inserted the file without complaint the second time I hit the button. I haven't been able to figure out what's causing the error or what might be causing SilverStripe to produce different messages in different browsers.

This is a manifestation of silverstripe/silverstripe-graphql#14.

Just tested this on dev-master and it doesn't seem like this is an issue anymore, I've also checked the part of the code that has caused this error to occur and seems that has been patched up as well :D
https://github.com/silverstripe/silverstripe-asset-admin/blob/master/client/src/entwine/TinyMCE_ssmedia.js#L214


Spellcheck workflow never saves its output

It always fails the checking step, so the following step --- the one that saves the output --- does not run. See https://stackoverflow.com/questions/58858429/how-to-run-a-github-actions-step-even-if-the-previous-step-fails-while-still-f for ways to make it run.

@clubanderson
@7h3-3mp7y-m4n


Following the license change, modify the boilerplates in crypto/kmac/

Merged.
e06785a5154cad12e8192bc2aaed0747ce7d23aa Following the license change, modify the boilerplates in crypto/kmac/


JWT verify always fails

I have tried to use oauthlib.common.verify_signed_token, but it turns out it always fails, because pyjwt is checking for aud key in JWT payload, which can not be passed to verify_signed_token, as it can be seen: https://github.com/jpadilla/pyjwt/blob/1.0.0/jwt/api.py#L196:L205 (note that I have pyjwt 1.0.0, as it is listed in requirements.txt)
I have locally changed behavious of verify_signed_token to include **kwargs that jwt expects and when I provide audience parameter, check passes. This is my change:
def verify_signed_token(public_pem, token, **kwargs):
    import jwt

    return jwt.decode(token, public_pem, algorithms=['RS256'], **kwargs)

Also, this function expects RSAPublicKey and JWT is usually verified with JWK, and it is not trivial to convert (at least not for me). So, it would be great if we had helper methods for this conversion, but I will be happy with just being able to use verify_signed_token.
If pull request would help, please tell me, I would be happy to create on (given that proposed solution is ok).

Submit a patch if you have a path to getting it working.


Build without embed fonts

Just to make sure all changes are applied. This is the full change.

Wait a sec, I had another design change in mind....

Oh, what was your change?

Also the load_from_asset should be private for both, which I forgot to add.

Well I looked into making the perfect solution work but realised that will never be the case. But what I changed what I think makes the shaper api less prone to misuse is to remove asset as planed from get_or_load() and expose load_from_asset if embedded fonts are available.
That way the intent is more clear and it hopefuly prevents that oversight the next time.

My changes seem to address those issues. It removes the asset bool without consequences.

Aaaa shoot, it doesn't return None.

Wait actually, https://github.com/SirJson/neovide/blob/6b8911d7fd7e0218a2347be9b01657b657ab7cd3/src/renderer/caching_shaper.rs#L143
I think it falls back to this.


Graph should ignore right click so I can save image as

Graph should ignore right click so I can save image as

see https://github.com/beothorn/stackignite/issues/6


🐛 Bug Report — console.log() behaviour changed, possibly unintentionally?

Starting with version v1.20231023.0.
I noticed in #1294 that the console behavior had changed.
OS
This is on macOS, and I have not tested the behavior yet on Linux.
Description
Previously, when I did a console.log it would show up in stderr (explicitly mentioning that this did NOT output to stdout to avoid confusion).
Now, when I do a console.log, it does not show up at all, even with --verbose flag, and does not show up in stdout at all.
If I change this to console.error, it will show up in the stderr as it did before.
Questions for debug
Is this intended behaviour? If it should be showing up in stdout, let me know and I'll debug on my side as my usage of workerd could be the issue in this case as I do have an "odd" way to run workerd.
Should it now be showing up in stdout, or is there a different way intended way to handle logs now?

Yep, I had a feeling it could be this.
I was going to update this ticket if it did fix the issue once that version with your fix is done building on my fork but I guess you beat me to it.

Hey<EMAIL_ADDRESS>has just been released which should hopefully fix this. Please let me know if this works. 👍

Happy to report it worked 🎉, I'm also very happy to report I'm no longer seeing a dreaded [object Object] but the actual objects contents.


Warning Property defaultPreviewTitleText is not allowed in VSCode extension

Describe the bug
Before adding a new i18n property defaultPreviewTitleText in #255, but now the warning Property defaultPreviewTitleText is not allowed remains in the VSCode.
Link to a StackBlitz project which shows the error
No response
Steps to reproduce

create a new Tutorialkit project
open VSCode and install Tutorialkit VSCode extension
open src/content/tutorial/meta.md
add below i18n content:

i18n:
  defaultPreviewTitleText: '預覽'

see the warning Property defaultPreviewTitleText is not allowed when the mouse hovering defaultPreviewTitleText

Expected behavior
No show the Property defaultPreviewTitleText is not allowed warning.
Screenshots

Platform

TutorialKit version: 0.2.3
OS: Windows
Browser: not applicable
Version: [e.g. 91.1]

Additional context
The i18n contentSchema is in the @tutorialkit/types package:
https://github.com/stackblitz/tutorialkit/blob/5c1de69c0e4e233a25a2f9b70fbb1f6c93f12356/packages/types/src/schemas/i18n.ts#L109-L114
And this is code of load the contentSchema in the vscode extension:
https://github.com/stackblitz/tutorialkit/blob/5c1de69c0e4e233a25a2f9b70fbb1f6c93f12356/extensions/vscode/scripts/load-schema-worker.mjs#L2-L5
https://github.com/stackblitz/tutorialkit/blob/5c1de69c0e4e233a25a2f9b70fbb1f6c93f12356/extensions/vscode/scripts/build.mjs#L61-L71
So if rebuilding the vscode extension can this warning be resolved?

The extension bundles @tutorialkit/types. This means that we need to re-publish it every time we make changes to @tutorialkit/types.
So this should start working once we publish new version of extension.

There is now new version of the extension published that fixes this issue.


Bump version number to v0.8.0

Add split, allowing limiting maximum concurrent threads

Codecov Report

Merging #15 into master will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #15      +/-   ##
==========================================
+ Coverage   99.17%   99.25%   +0.08%
==========================================
  Files           2        2
  Lines         121      134      +13
==========================================
+ Hits          120      133      +13
  Misses          1        1

Impacted Files
Coverage Δ

dockerjudge/__init__.py
98.63% <100.00%> (+0.07%)
:arrow_up:

tests.py
100.00% <100.00%> (ø)

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 2ac2620...f4e11dc. Read the comment docs.


Update NSURLSessionConfiguration defaults to support OSX 10.12

#1806

CodeFlow for GitHub does not download executable files. If you trust this file and would like to run it, please download it yourself through GitHub.

?

Refers to: Helloworld.txt:1 in 657034c. [](commit_id = 657034c9ca5bfc542a82a4205dced07df3d26892, deletion_comment = False)


⚠ Samsung has degraded performance

In 0ecd5b9, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2415 ms

Resolved: Samsung performance has improved in f7673bc.


🌳 Feature - Switch to vis-timeline from vis

Overview
This PR switches the vis package to vis-timeline
Dependencies
N/A
Outstanding Tasks

[x] version
[x] update examples folder with latest version

Thanks for this change! Can you guys merge it?

Can someone please merge the changes? Thanks!

https://www.npmjs.com/package/react-vis-timeline

Thank you!!! :)


Creating README file in english

Translating the README.md to english

Coverage remained the same at 97.647% when pulling 4a5cc3556629b3c1469ff75c082ea85e7b0ec31a on andersonberg:readme-english into a02a27b4d6f2925a2fa988eb339ad527c45b399b on mstuttgart:develop.

No problem! :D