Typescript syntax highlighting xpath

Issue Type: Bug
Xpath strings with interpolated variables are being collored as comments instead of strings.

VS Code version: Code 1.44.0 (2aae1f26c72891c399f860409176fe435a154b13, 2020-04-07T23:31:18.860Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Xeon(R) W-2133 CPU @ 3.60GHz (12 x 3600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.66GB (14.78GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (6)

Extension
Author (truncated)
Version

better-comments
aar
2.0.5

insertnumbers
Asu
0.9.1

gitlens
eam
10.2.1

powershell
ms-
2020.4.0

sublime-keybindings
ms-
4.0.7

vscode-typescript-next
ms-
3.9.20200423

Please share the text of the code along with screenshots

return By.xpath(`//*[contains(@class, "${Blade.bladeClass}") and descendant::*[contains(@class, "fxs-blade-title-subtitle") and descendant::*[contains("Deployment",text())]]]`);

Works fine for me in VS Code insiders:

Does this reproduce in the latest VS Code insiders build with all extensions disabled?


Look into parallelized/pipelined instructions

Currently, when we execute KCL, we send a command-at-a-time, wait for the response and continue. This is because of a variety of complexities, including the one where UUIDs are minted server-side, not client-side, so we we need to wait for a return before we can continue with dependent operations.
There's a few ways out of this, but the two I see are:

Work through an architectural overhaul where the client mints IDs. This will be tough for operations that produce new surfaces, since it won't be obvious to the client (for instance, if I draw a box and extrude, you need to how how many new surfaces that will create, and name each one in a way the engine can use). I think that may become prohibitively hard, but could be possible.

Start to represent command dependencies, such that for a given command, we can understand what command(s) need to complete (and get a response we parse) first.

This issue is about 2. I don't know how possible 1 will be.
If we can create a world where 2 is true (for a given command, we know what command UUIDs we depend on results from), we should be able to do a topological sort on the leafs. Here's a rough sketch:

When the KCL executor is running, it'll produce as many modeling messages as it can and push them into the queue. It'll need some way to represent the dynamic information from the server (some sort of marker that represents "The UUID returned from the engine in response to this message")
When messages are sent or received, preform some sort of lightweight topological sort on the command DAG.

Count the number of "inbound" dependencies to a task (things waiting to hear back about its response from the server).
If the number of any inbound dependencies to a node in the DAG is 0 (a "leaf"), send that message to the server without delay.

When a message is received, update the messages in the queue by:

mark the message we just got a value back from as having had a response, noting any uuid(s) in response
remove its "outbound" dependencies in the DAG
sweep the DAG for new leaf tasks -- tasks without dependencies.

This would (hopefully!) allow us to split off unrelated components from the DAG, and run their tasks in parallel, so that we can render different (non-dependent) geometries at the same time.
Thoughts?

A very similar idea came up before when we were talking about the artifact graph, including topological sorting. But I forget what the motivation was. I think it was about adding other features, not performance, but I could be mistaken.
We do batch the sending of commands. Only certain commands force sending to the engine. So it's not quite as simple as a command-at-a-time anymore, but I agree that it could be better. extrude() is an example that forces sending.


Calling bot.beginDialog fails in Botkit.afterDialog

Defining a simple handler through Botkit.afterDialog that starts another dialog crashes.
Example:
controller.afterDialog(“how_you_doing”, async (bot) => {
    await bot.beginDialog("tell_me_more");
});

It crashes with:
Error in trigger handler Error: Call to beginDialog on a bot that did not receive a dialogContext during spawn

Looking at the code running the handler right here, it spawns a new bot instance passing it the step.context which is a TurnContext. This makes it crash later down the road.
Workaround
Define the after hook using the convo.after API.
Context:

Botkit version: 4.6
Messaging Platform: Web
Node version: 10.16
Os: MacOS

This is due to the fact that the bot instance passed into the after handler no longer has a valid dialog context, which is required to start new dialogs.  I'm not sure there is a great solution.


<EMAIL_ADDRESS>

[Enter feedback here]

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

ID: ade10500-7d5b-8b93-aac8-25d31f4392a9
Version Independent ID: 45c79efc-f024-7f8a-55ce-35e9bfa54b6a
Content: WinDbg Preview - Start a user mode session
Content Source: windows-driver-docs-pr/debugger/windbg-user-mode-preview.md
Product: windows-hardware
GitHub Login: @DOMARS
Microsoft Alias: domars

No issue mentioned.


Resolved Issue #92  -  UI is complete with Tailwind and Firebase updates.

I pulled the latest version and integrated my changes to the adminPage.js. Then, I created this branch to track the changes and compare for the pull request.

ccurrently working on doing the small UI changes and merging with #123
-janise


SFTP fails since 2.2.1 (still in 2.2.2)

Prerequisites

[ X] Put an X between the brackets on this line if you have done all of the following:

Checked that your issue isn't already filed: https://github.com/icetee/remote-ftp/issues

Description
Connection failed – Error: All configured authentication methods failed
Steps to Reproduce

Try to connect

Expected behavior: connection established
Actual behavior: Error
Reproduces how often: 100%
Versions
Atom    : 1.32.0
Electron: 2.0.9
Chrome  : 61.0.3163.100
Node    : 8.9.3
OS: Ubuntu 18.04.01 LTS (bionic)
Additional Information
The issue occured since I upgraded the remote-ftp today to 2.2.1 and persists with the upgrade to 2.2.2; regular SSH connections work without any issue. SSHFS mount also works.

Hi!
Only two things have changed

Add variable to style.less https://github.com/icetee/remote-ftp/commit/176ea22742efe0385ee6c472a0c05d4fa845fe88
Add new lines to CHANGELOG.md https://github.com/icetee/remote-ftp/commit/9b49f48a7c744638b90d606670c04d0dd9642fa9

He could not have caused any problems the update itself.
Please try remove .ftpconfig, restart Atom, maybe
Related of https://github.com/icetee/remote-ftp/issues/698 and https://github.com/icetee/remote-ftp/issues/83

Please try this commands:
apm remove remote-ftp
apm install<EMAIL_ADDRESS>

works

@icetee I downgraded, the error continues the same. In FileZilla I can connect normally.

Does Atom nodejs support ed25519? Seems there is something there.
Switching the key to rsa worked.

ssh2-streams has been updated within electron to support eded25519 (mscdex/ssh2-streams#100)
As remote-ftp is broken for anyone like me who upgraded all their keys, how hard is incorporating the this new version of the dependencies?


chore(deps): update packages, linting

:tada: This PR is included in version 3.2.0-dev.1 :tada:
The release is available on:

GitHub release
npm package (@dev dist-tag)

Your semantic-release bot :package::rocket:

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Fix failing QuickOpen unit tests

Fix QuickOpen unit tests after introduction of QuickSearchField - listens for Enter on keydown, not keyup; expects an "input" event to follow every text change (Enter key processing now waits until text change events have caught up so Enter key never takes effect if change event never comes)

Thanks Peter. The Tests are now passing through on both Windows and Mac.


Fix mailbox

Fixes a mistake with launching mailbox.

Testing it now.
Btw, should we make the api_key input a password?

Btw, should we make the api_key input a password?

I think so.

Yes, there is. Can you try with my suggestion first?
https://github.com/TECH7Fox/Asterisk-add-on/pull/92#discussion_r803015761


Central transaction log

Story
As a vega wallet user
I want to easily find my recent and ongoing transactions
So that I can discover the status and outcome of transactions if I missed it elsewhere
Questions

[ ] Waht happen to that list when we switch network ? Should we void it ? SHould we keep it, without mixing links ?

Acceptance Criteria

[ ] I can find a single list of all transactions, completed and ongoing, from all keys and wallets, from my current desktop session
[ ] I can click a transaction in the list to open the transaction details modal for full information
[ ] I can find my way to the complete transaction history on block explorer
[ ] I can navigate between the transaction list and wallet list

Tasks

[x] Explore and sketch
[x] Team and stakeholder review
[ ] Visual Design for:

[ ] in-app navigation
[ ] visual device to indicate activity in the transaction tab
[ ] transaction tab

[ ] Team review
[ ] Implementation
[ ] Team QA

Sketch

The layout of the transaction tab is based on the transaction component which can be found in Wallet → Key → View Transactions:

Additional details / background info

Parent ticket: https://github.com/vegaprotocol/vegawallet-ui/issues/74
Sibling tickets:

https://github.com/vegaprotocol/vegawallet-desktop/issues/487
https://github.com/vegaprotocol/vegawallet-desktop/issues/489

@ckpringle to add a way to see which network each transaction was on

Transaction Detail attached and in FIgma here ↳. Two options for displaying the error, could use the inline notification TBD


Typo "[ 1**,...n]"→"[,**...n]"

https://docs.microsoft.com/en-us/sql/t-sql/statements/create-partition-scheme-transact-sql?view=sql-server-ver15
#PingMSFTDoc

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Bash Script?

If I purchase a membership of a certain level, will I be provided with bash script to customize and add modules myself?

Sorry, no. At this point, the script is not open.

Fair enough, I was considering the platinum membership but not if script isn’t open.
Thanks anyway!
Duane A. Garner, C.R.S.S.,
Mobile: (502) 528 - 3336
Office:<PHONE_NUMBER>
Business: @.***
Infrastructure Architect, Critical Communications, United Parcel Service
http://ups.com
Chief Executive Officer, Threat Level Gaming, LLC   https://www.threatlevel.gg - https://discord.gg/tlgg
Chief Executive Officer, Catalyst Group, LLC & NextLevel Analytics, a division of Catalyst Group
https://www.catalystgroup.gg - https://cghq.me
Director of IT and Security, Eastern Shore Recovery Foundation, 501(c)3
https://esrecovery.org
This message (including any attachments) may contain confidential, proprietary, privileged and/or private information. The information is intended to be for the use of the individual or entity designated above. If you are not the intended recipient of this message, please notify the sender immediately, and delete the message and any attachments. Any disclosure, reproduction, distribution or other use of this message or any attachments by an individual or entity other than the intended recipient is prohibited.
On Dec 5, 2024 at 03:48 -0600, SmartHomeBeginner @.***>, wrote:

Sorry, no. At this point, the script is not open.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Deploy with custom domain (CNAME) on GitHub Pages

example: https://github.com/JuanVqz/juanvqz.github.io
Type of PR

[ ] feature
[ ] enhancement
[ ] bug fix
[x] other

Description
I was searching for a way to don't remove the CNAME file on deploy but I think this is an easy way to keep the CNAME in the gh-pages branch.
I'm not sure if this is the best place to put this information. sorry if not.
BTW, thank you for your contributions. 🤟
Checklist

[x] I have tested this
[ ] I have updated the documentation if applicable

Closing this since this repository is being archived. Thank you so much for your contribution, and I apologize for not seeing it before now. I’d definitely make this same PR to the Bridgetown docs if it still applies!!
Archive reason here


Syntax errors cause crashes

Hello.
This simple piece of code causes a crash due to an error in the provided expression. The text "Some error" never gets printed.
#include <cparse/shunting-yard.h>

int main()
{
    TokenMap tm;
    tm["x"] = 5;

    try
    {
        calculator calc;
        calc.compile("5&&&x"); // I would expect this to throw an error, or set some flag I could check
        printf("%f", calc.eval(tm).asDouble());
    }
    catch (...)
    {
        puts("Some error");
    }
}

Changing the expression to e.g. 5*x makes it print the correct result.
This is very unfortunate, since I expect the users of my application to be notified of an error if they input an invalid expression. Is there anything I'm doing wrong or is there any other mechanism for validating the expression?
I'm compiling with gcc on 64-bit MinGW. Everything links successfully.
g++ -std=c++11 main.cpp -I../includes ../libs/cparse/core-shunting-yard.o ../libs/cparse/my-features.o

Also setting the expression to e.g. 5x does not crash the application, but doesn't throw errors either. It evaluates to x, though. However, setting it to sin(5x) does crash it. sin(5*x) works correctly.
The crash kind of looks like a stack overflow to me. Perhaps these expressions cause infinite recursion somewhere in the code?
Now, I have defined a custom function (logarithm) and a custom operator (caret for exponentiation). Those work and I don't think they have anything to do with this problem, but it's worth mentioning anyway.

The error is for sure at calc.eval(tm) and not at asDouble(). Trying to store the result of eval() crashes the program before the result is obtained.

This also crashes without any errors thrown.
#include <cparse/shunting-yard.h>

int main()
{
    TokenMap tm;
    tm["x"] = 2;

    try
    {
        auto result = calculator::calculate("sin(5x)", tm);
        printf("%f", result.asDouble());
    }
    catch (...)
    {
        puts("Some error");
    }
}

Hello @AntonC9018, thanks for reporting and also with a minimal example with the bug, I will try to understand it and then I will get back to you =].

Uhm, ok this might be one of those pesky ones, because I was unable to reproduce the first example, so the steps I did were:
First I created a main.cpp file and cloned the master branch of the cparse project on a subdirectory cparse/.
In the main.cpp file I inserted your code:
#include <cparse/shunting-yard.h>

int main()
{
    TokenMap tm;
    tm["x"] = 5;

    try
    {
        calculator calc;
        calc.compile("5&&&x"); // I would expect this to throw an error, or set some flag I could check
        printf("%f", calc.eval(tm).asDouble());
    }
    catch (...)
    {
        puts("Some error");
    }
}

Then I built the core-shunting-yard.o file with make release -C cparse and finally compiled and executed the program:
$ g++ -I . cparse/core-shunting-yard.o main.cpp && ./a.out
Some error

Second Problem:
I was able, however, to reproduce the incorrect compilation success you reported with the following code:
#include <cparse/shunting-yard.h>

int main()
{
    TokenMap tm;
    tm["x"] = 5;

    try
    {
        calculator calc;
        calc.compile("5x"); // Using 5x instead of 5&&&x
        printf("%f", calc.eval(tm).asDouble());
    }
    catch (...)
    {
        puts("Some error");
    }
}

It returned as if the result was the number 5.0000, so I will start by trying to understand what happened here.

Also as a side note, the packToken type works with cout so you could do it like this:
#include <iostream>

// ...
            std::cout << calc.eval(tm) << std::endl;
// ...

Buf for debugging purposes, I am glad you used printf since it avoids implicit calls to AsDouble that would have happened using this trick

Ok, here's what I'm doing for the first example (this time without custom features):

Git clone the latest code with git clone https://github.com/cparse/cparse cparse
Compile with mingw's make (I'm on Windows 7). mingw32-make release -C cparse
Compile the code below with g++ -std=c++11 cparse/core-shunting-yard.o test.cpp

// test.cpp
#include "cparse/shunting-yard.h"

int main()
{
    TokenMap tm;
    tm["x"] = 2;

    try
    {
        calculator calc("5&&&x");
        auto result = calc.eval(tm);
        printf("%f", result.asDouble());
    }
    catch (...)
    {
        puts("Some error");
    }
}

Run the code with a. The program produces no output while Windows gives me a generic message that the program has stopped working.

I suppose you are on Linux. Since you're on Linux, this might be a platform-specific issue. Any clues?

Wait, this time it's even weirder. This code also crashed for me:
#include "cparse/shunting-yard.h"

int main()
{
    TokenMap tm;
    tm["x"] = 2;

    try
    {
        calculator calc;
        calc.compile("5*x");
        auto result = calc.eval(tm);
        printf("%f", result.asDouble());
    }
    catch (...)
    {
        puts("Some error");
    }
}

Whereas calc.compile("5"); works fine. calc.compile("5*x", tm); also crashes it.

Ok, I guess one also has to link against builtin-features.o. If I do this, though, calc.compile("5*x"); works, but calc.compile("5&&&x"); still just crashes the application, without any output. Trying it with calc.compile("y");, while no such symbol y has been defined or given value, prints the following:
# a
terminate called after throwing an instance of 'bad_cast'
  what():  The Token is not a number!

This application has requested the Runtime to terminate it in an unusual way.
Please contact the application's support team for more information.

"Some error" is never printed.

Thought I'd mention the compiler version too.
# g++ --version
g++ (Rev6, Built by MSYS2 project) 10.2.0

I've also tried to catch each type of error your code ever throws individually, to no avail. This code also results in a crash, without any prints.
#include "shunting-yard.h"
#include "shunting-yard-exceptions.h"

int main()
{
    TokenMap tm;
    tm["x"] = 2;

    try
    {
        calculator calc;
        calc.compile("(");
        auto result = calc.eval(tm);
        printf("%f", result.asDouble());
    }
    catch (const bad_cast& exc)
    {
        puts("Bad cast");
    }
    catch (const syntax_error& exc)
    {
        puts("Syntax error");
    }
    catch (const type_error& exc)
    {
        puts("Type error");
    }
    catch (const undefined_operation& exc)
    {
        puts("Undefined operation");
    }
    catch (const std::invalid_argument& invalid_argument)
    {
        puts("Invalid argument");
    }
    catch (const std::domain_error& exc)
    {
        puts("Domain error");
    }
    catch (const std::runtime_error& exc)
    {
        puts("Runtime error");
    }
}

Also tried running the tests. I have not had experience with C++ tests, so I might be wrong. I have compiled the test-shunting-yard.cpp with g++ test-shunting-yard.cpp core-shunting-yard.o builtin-features.o catch.cpp, then ran the executable. It also simply crashes.

The last thing I tried is compiling the entire thing, without linking to libraries. Compiling and running the code from above from scratch with g++ -std=c++11 shunting-yard.cpp packToken.cpp functions.cpp containers.cpp builtin-features.cpp test.cpp does indeed print Domain error. catch (...) also works fine with this. I guess this approach works for me.
Any ideas on why this would be happening?
Compiling without optimization flags likewise did not solve it for me.

Hello @AntonC9018 sorry I disappeared,  I had some things to do tonight, tomorrow I'll continue the investigation

About the tests, it should be enough to run make test.
About the 5*x indeed we need the builtin-feature.o, about compiling it just with y the compilation actually works, what throws is when you try to convert it to double, since it is only an unbound variable.
One thing you could try is to to:
try
{
    // ...
{
catch (const std::exception& ex)
{
    std::cout << ex.what() << std::endl;
}

Since all exceptions inherit from std::exception this should work unless we are talking about invalid memory access, which is always random =/.

The last thing I tried is compiling the entire thing, without linking to libraries. Compiling and running the code from above from scratch with g++ -std=c++11 shunting-yard.cpp packToken.cpp functions.cpp containers.cpp builtin-features.cpp test.cpp does indeed print Domain error. catch (...) also works fine with this. I guess this approach works for me.
Any ideas on why this would be happening?
Compiling without optimization flags likewise did not solve it for me.

This is very weird. But sadly this is normal when we are working with C++ on different platforms =/.
The g++ version has an output very different from yours but it is also on the 10.2.0 version:
$ g++ --version
g++ (GCC) 10.2.0
Copyright (C) 2020 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

That said, if compiling including all cpp files is working for you this is great, but I have no idea of how to fix this so that future users don't have this problem. If you find out it would be great if you could submit a PR with a fix.
So, I will focus on fixing that problem where it would not throw for the expression "5x" and evaluate to 5.0000, but I have little time for making these fixes, so this might take some time, hopefully not too much.

This code does print "Some error" on my MacOS:
#include <cparse/shunting-yard.h>

int main()
{
    TokenMap tm;
    tm["x"] = 5;

    try
    {
        calculator calc;
        calc.compile("5&&&x"); // I would expect this to throw an error, or set some flag I could check
        printf("%f", calc.eval(tm).asDouble());
    }
    catch (...)
    {
        puts("Some error");
    }
}

If I remove try/catch it aborts with an unhandled exception: terminating with uncaught exception of type syntax_error: Invalid operator: &&&. So it seems cparse works correctly in this case. I think the problem with exception not caught on MINGW relates to exception handling on this specific platform. Normally catch(...) should catch all exceptions even a mem violation.
(maybe this would give a hint: https://stackoverflow.com/questions/4149118/exception-handling-doesnt-work-with-qt-on-windows)

Interesting, but I think that the problem reported on stackoverflow is probably something different since he was able to fix it by recompiling everything from source instead of using the intermediary .o files the Makefile produce. But it might be a good place to start investigating.

@AntonC9018 I have just submitted a commit that should fix the problem where it was not throwing for invalid expressions such as 5x:
aed8ce0be3063596a232c24d8e9693ada0962353
Regarding the error with throw, I don't know how to fix it since I can't reproduce it, so if the commit above solves the other problem I think we should close this ticket.

Sure.
I ended up using a different library anyway.

Yeah, I figured xP, anyway thanks for reporting.


Getting started with a simple CloudFront distribution

Tutorial here

[x] Install AWS CLI
[x] Step 1: Upload your content to Amazon S3 and grant object permissions
[x] Step 2: Create a CloudFront distribution
[x] Step 3: Access your content through CloudFront
[ ] Extra: Update Origin to the previous tutorial's environment (the one with the Buddy CI with Docker)

Step 1, done

Distribution domain name: https://d37d6agsnkyapf.cloudfront.net

Success:

https://d37d6agsnkyapf.cloudfront.net/hello-world-html/index.html

Success (the cloudfront website is delivering the other website's stuff)

Though that's just by putting the domain of the original as the origin for CloudFront:

There's probably a way to do this more securely, but that's a question for later

Success:
I pushed changes to the main branch, and a little while later, the CloudFront domain shows this:


Fix extremely large sublayouts (1,000+ nodes, chains 100+ long)

Fixes a rare but serious bug creating broken layouts on clicking into clusterNodes with very many aggregateNodes.
On digging into this I found it seems to happen when the total of the (unscalable) gaps between nodes exceeds the space available.
Here are a couple of 'before' examples:
https://upload.clinicjs.org/public/1e48153768a19dcf5bac7c225571945beb1fa282f022a767e135386ba14eb96c/1005.clinic-bubbleprof.html#c3 - 5,068 aggregate nodes in one cluster node

https://upload.clinicjs.org/public/39a458eeb928801e0cb2ad69cfb2108358fa3c72be6ac44a4e9b6ff9d413827d/21608.clinic-bubbleprof.html#c14 - 1,115 aggregate nodes in one cluster node

This PR fixes this by taking the total length of these unscalable gaps in the longest chain in the layout and adding it to the height of the space which is used when calculating the first, pre-collapse layout, as well as applying the % increase in total height to the collapsing threshold. Then, after collapsing small nodes together, we can use the correct height because the sublayout will have been collapsed down to a reason number of nodes.
Those same profiles generated with this PR:
https://upload.clinicjs.org/public/81bd1274028e32b2e257edd921245f0d989587c3c93cf45ad60602087d896559/1005.clinic-bubbleprof.html#c3

https://upload.clinicjs.org/public/487567a841d65c480d0f10d37d33e0ac91abf864cb225cacaa221101ad331ee1/21608.clinic-bubbleprof.html#c14

Note - performance in these sublayouts is a lot worse than it should be (massively better in these broken cases than before this PR, but still much worse than it should be).
The number of nodes that have been collapsed shouldn't affect in browser performance much, but clearly does - which suggests that there may be either a memory leak or some unnecessary data retention somewhere in the node collapsing and related logic.
But that's something to fix in a separate PR - this PR takes a case that is completely broken and extremely slow, and makes it not broken and only moderately slow.

Code LGTM but I cannot test this properly. Not sure if this is me or the branch lost a commit on the way, but I keep getting super-nests instead of what's on the screenshots. Maybe it's also because of my screen size? Was this tested on a large monitor maybe? Here is what I'm getting:

Also changing dimensions to mobile size seems to break it completely. Can you reproduce?
Also one code question as I'm not 100% clear - I cannot get in my head the reason for creating LNs depth-first? I'm not sure how changing order affects this block - iirc it runs independently between scale and collapse sections?

Btw if that's more convenient here I think we are good to drop legacy stretchable height. I don't think we will have it back anytime soon

Can confirm same happens to me in the online sample

That's really strange. Also really strange is that the Clinic Upload links, which I tested earlier, now no longer work for me, showing the old behaviour.
I've done some experimenting and if I recreate the sample, I can replicate the clump-everything behaviour you mention if I zoom out a few times. So it's clearly relative to screen size - and collapsing should not be relative to screen size, so that suggests something has gone wrong here.

Yes, turned out the window size was creeping in to some of the variables used in node collapsing. Latest commit should fix that. 21608.#c14 ( https://upload.clinicjs.org/public/9aa506b2b6c45d0fc17cf47c2e9bd564deede05b54fcce1141ad1cd892eab22b/21608.clinic-bubbleprof.html#c14 ) should now have these nodes at all window sizes:

And 1005.#c3 (  )

When I inspect inside this one:

I get this:

I imagine the white part at the bottom is telling me it did not contain a chain fully?

That's an extremely hard case to fix. The latest commit reduces the likelihood and severity of that happening, and stops it happening on large screens, but it can still happen on medium or small screens. Preventing it completely requires calculating a dynamic collapse threshold which should be done as a separate PR, not a bolt-on to this one (details below).
This commit adds a check after calculating the scale factor to check that the longest stem does now fit. If it doesn't, it reduces the scale factor to fit or if that's not possible, to zero so that there's as little as possible (this can be removed in the next PR that will increase the collapse threshold until it does fit). Here's how that view now looks on a large screen / zoomed out ( https://upload.clinicjs.org/public/cabda786c514e49ad25ad137dae1d14730c3146a95777a111b407a4e4abd5642/1005.clinic-bubbleprof.html#x4-c3 ):

On a smaller screen here's the "damage limitation" scale temporary fix, which fits in as much as possible while still letting the user see what's the biggest:

We're seeing this here because we have an extremely long chain that, by chance, just exceeds the collapse threshold yet adds up to more absolute space than most windows can fit. It's rare but possible, so far not observed anywhere outside this one sublayout of 1005.
I've tried many different approaches to this:

Re-running the collapse logic with increasingly strict thresholds if it returns a view that doesn't fit - hits a long line of errors due to the parents not being inputted yet but being required for an additional collapse
Adjusting the scale weights to take the longest absolute into account - turned out to be unrelated to the problem at hand
Changing the handling of negative scale weights - again didn't address the root cause of the problem (but did help narrow in on it)
Changing the scale independent weight logic to use a fixed-dimensions scaled square and choose an appropriate scale weight same as the regular scale  logic - inconsistent and heaps of side effects

What should fix it completely is adding a calculation to collapse-layout.js which looks at the longest stems, checks if their absolute value exceeds the fixed-height value, if it does, iterates through the ancestors seeing which are collapsable with the current threshold, calculates if this will bring the absolute value below the fixed height space, and if it doesn't, increases the threshold and tries again. Then takes the smallest such threshold that will fit all stems, and uses that for the collapsing.
But that's something that should be done in its own PR with its own tests etc.

Regarding other points:

"drop legacy stretchable height" - yes but let's do that as a separate refactor. Not a priority, at the moment it can be quite handy for isolating tests to specific parts of the scale logic
"changing dimensions breaks it completely" - was fixed in commit "Ensure collapsing is consistent across window sizes"

And then if there isn't a rush to finish that PR, can see if there's anything in that dynamic collapse threshold calculation that can be expanded to address any of the cases in your improved collapse notes

I mentioned that this last commit helps improve the display in very small windows. Here's a before and after of the layout above the layout shown above ( #c3 ) :
Before, we get a crazy, disturbing "aerial bombardment" layout if the screen shrinks below a certain point:

After, it stays more reasonable. At some extra-small sizes a label doubles back on itself (I think the next PR should fix that), but it's minor, not something crazy that would freak out someone innocently resizing a window:
file:///Users/alan/Documents/GitHub/node-clinic-bubbleprof-samples/all-samples/1005.clinic-bubbleprof.html#c3

Let's have a proper gallery set up for these. They're gorgeous.


Add info about creation of flake.nix

Until, there was no info about the name, path or existence of the flake.nix
I am a newbie myself, so if I have messed this up, please be gentle. 🙂

I don't think that these are the right suggestions, flakes should not live in privileged locations, it makes more sense to define the flake in some git clone of the repo where you'll keep this code. In general it is up to the user to decide where to store this file.
Also, the following section describes what to put in the flake file, which I think makes it rather clear what to do.
This project is an MVP currently and although it's usable for what it can currently do, it is probably not really appropriate for people new to nix.

Thanks a lot for answering so swiftly 🙂
I literally asked the Matrix channel, and multiple people told me, flakes can live where they want. And that the mentioned paths are good, cause they live together with other Nix files there.
I am packaging this, so that is why "I have to decide for the user" where this is going to be.
So I will choose a place next to the home directory, I guess.

Yeah, I personally feel that they should not be root-owned, that just encourages using sudo for everything all the time.


Possibility of using few methods available C3.js and not in this react component

I just refer this documentation  and try to use load method which can used to populate the data to already existing charts. But checking the module I doubted it doesn't have load or flow method.
I have an intention of loading or updating data to existing chart for every 20 secs. Will it be possible to do with this library. How can a chart component changed with data?
Thanks for any suggestions!

Not sure if you've figured this out, but from looking at the source, it looks like the C3Chart component has a method called loadNewData that wraps the underlying chart's load method.


[Ignore, Test] spark-runner: upgrade to spark 2.0.2.

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-1.6%) to 68.401% when pulling e087d168adea781b7e218756148b02fac5d9c6a7 on peihe:spark-runner into 62294e64cb3f68e37cd0ed133070fb617334c09e on apache:master.

R: @amitsela

R: @aviemzur

@peihe this will probably work, since the breaking changes in API in Spark 2.x are small.
However, our plan currently is to support both Spark 1.6.x and Spark 2.x in the runner.
This is covered by BEAM-1920
I believe this is currently being worked on by @jbonofre you two can sync your efforts on this.

I replied on the mailing list about that. Indeed, I have a branch with a new Spark 2 runner. @peihe can you please sync before pushing any PR ? Thanks !

R: @jbonofre

I was also involved in this.
This is what I thought last I checked the Spark1.x/2.x difference:

We keep supporting only the RDD API, "Structured Streaming" does not provide a stateful operator that scans the entire state (last I checked..).
There was an idea to dynamically switch between the Iterators and Iterables since this was the only compile-time difference I noticed. If we can do this we can actually support both with minimal duplication.


Multi key selection

Added buttons and functionality for selecting multiple keys lighting at once.
(e.g. WASD, Num Row, IJKL, etc.)

Please let me know when you have finished your changes. I'll do verify and merge in the changes.

So all my changes for multi key selection are done. I changed the buttons to be coloured in the selected colour and not just outlined. Added a button to copy a colour from an already assigned key.
A known bug is the selectedColour when copying gets updated, but it is not reflected in the colourpicker.

I believe that you have to explicitly set the colour picker. If you are done with the changes, I'll merge them in.

Yeah, changes are done.


[FLINK-12073][table-planner-blink] Support appropriate precision and scale processing of Decimal

What is the purpose of the change
1.Make Decimal's output more precise than random.
2.Let Decimal process closer to mainstream databases such as Hive/SqlServer.
Verifying this change
ut
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (JavaDocs)

LGTM


Cosmetics

Every now and again I do a fresh install and fix some things that's been bothering me.

Can one of the admins verify this patch?


We should recommend people place their configs inside config/backpack/addon-name.php

See title

Should it be inside a folder addons to differentiate the backpack official configs from the public addons?
config/backpack/addons/addon-name.php

Note that the #8 PR doesn't add that extra folder.

Note that the #8 PR doesn't add that extra folder.

Changed my mind - sounds like a bad idea now. Sure, it'll be easier for the user. But since this will:

make the Backpack organisation look responsible for that package we have no control over;
make it super-difficult to install/create 2 addons with the same name;

I think it's better for each package to use his own namespace. So let's scrap this idea.

Ok makes sense now 😅


fix(lock): make import pool, create replica and share replica operations mutually exclusive(cherry-pick PR)

Cherry-pick PR of #1611

bors merge


Is it possible to search with larger image size like 224*224?

I'm trying to search the cell with images resized to 224*224, but it seems that huge memory is required...

Yes. You may need to 1) change the stem to reduce the size at the very beginning and 2) use smaller batch size.


problem with svg mode in firefox

Hi,
When I use svg in markers with the newest version 1.0.9 in Firefox I see the svg circle sprite and the old circle image in the same time (slipped vertically with some 10 pixels) and the number  under them slipped down outside of the sprite.
However this is ok on Chrome.
This is the definition of the marker:
{ icon: 'fa-number', shape: 'circle', markerColor: this.points[index].color, number: this.points[index].icon, iconColor: this.points[index].textcolor, svg:true }
with a modified older css version it displays ok on firefox, but not on chrome:
leaflet.extra-markers.GOOD_WITH_FIREFOX_BAD_WITH_CHROME.zip
My firefox is v.72.01. 64 bit

Closing as fixed in v1.20+


[DRAFT] Create helm chart

Signed-off-by: Nicklas Frahm<EMAIL_ADDRESS>Description
This PR provides a Helm chart to deploy Tinkerbell to Kubernetes.
Why is this needed
This is submitted in alignment with the discussion during the community meeting on the 15.02.2022.
How Has This Been Tested?

TBD
How are existing users impacted? What migration steps/scripts do we need?
This is a new feature, so users should not be affected.
Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[ ] added unit or e2e tests
[ ] provided instructions on how to upgrade

Any chance you'd revisit this? Would be a very nice addition to the deployment methods.

I will likely be creating a helm chart for Tinkerbell in the somewhat near future if this PR has lost activity.

We've created a new helm chart repository that has been incorporated into v0.7 of Sandbox. This issue can be closed.


SUP-1788: CircleCI docs tabulation

Transformed the CircleCI docs into a markdown tabulated format, with each main key of a Circle pipeline and all sub-key support listed (with touched up notes from the original)

@toote as with #79, I've tweaked the docs to use a combination of ✅, ⚠ and ❌ to compensate for the coloured circles for now - let me know what you think!


Make available as a deb package for Raspberry PI OS

Greatly simplify the process of getting started for less technical users.

Done!


[roctracer] Adding roctracer support to capture (but not export) the hipMemset* APIs

and a couple other minor changes
see individual commit messages for details.

Can one of the admins verify this patch?

jenkins: retest rocm-xla please


Good work

Dear nicholasadamou,
Hope you are fine.
I looked your OnionPi project and liked but i have a question. I have to ask this, sorry.
Are you member of 5Eyes, 9Eyes and 14Eyes?
Thanks.

Thank you for your message. No I am not a member of those organizations you've listed.
@hasanalizxc
Sent with GitHawk

Thank you for your message. No I am not a member of those organizations you've listed.
@hasanalizxc
Sent with GitHawk

@nicholasadamou Thank for your interest and quick reply. I have a pi3 modelB and will try this project as soon as.


Fix various byte compilation warnings in rust-mode-tests.el.

Migrate to lexical binding.
Declare functions that are conditionally-defined.
Migrate off the deprecated ‘cl’ library.
Don’t use ‘setq’ to introduce global variables and constants.
Avoid “unused lexical variable” warnings.
Remove some unused variables.
Stop using interactive-only commands.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @mookid (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Thanks.


Validation fix duplicate headings missing alt text

Global effort to fix build validation errors
@Rick-Anderson
The Content & Learning team is fixing build validation errors on docs.microsoft.com for the rest of H2. This will eliminate potential accessibility, security, and usability issues. This PR includes only build validation fixes and does not change other content. Please review within five business days and merge or comment in the PR with any changes you'd like to see. Thank you!


Add two  schools

MJM design Graphic : https://www.mjm-design.com/
Webstart : https://ecole-webstart.com/

@fauVictor Pull request merged. Thank you!


showcase: Add Neolace platform

I'd like to add our app Neolace to the Deno showcase.
The backend is entirely written in Deno and though it's currently closed source, we've released many open source modules that were required to build this project, such as deno-s3-lite-client, the first Neo4j driver for Deno (now official), Vertex Framework, and quantity-math-js :)

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


Contribution of PC and PV currents

Just a curious physics question that I wonder often: what is a decay rate of Lc that goes via PC and PV currents.
Dump it here, and remove from my head. We might check it one day.
Two clues to investigate it:

LS is eigenbasis for the parity operator
PC and PV do not interfere

Essentially, one just need the decay rate matrix in LS basis. One can either:

use the alternative LS model, or
map the default model to LS: compute couplings, validate exact equality, compute the rate matrix.

Here, one needs to modify the compute_decay_rates function from
https://github.com/ComPWA/polarimetry/blob/main/docs/intensity.ipynb
something in this direction,
# Assuming each resonance has an attribute 'couplings' that lists its couplings
all_couplings = [coupling for resonance in resonances for coupling in resonance.couplings]
n_couplings = len(all_couplings)

def compute_decay_rates_for_couplings(func, integration_sample: DataSample):
    decay_rates = np.zeros(shape=(n_couplings, n_couplings))
    combinations = list(product(enumerate(all_couplings), enumerate(all_couplings)))
    progress_bar = tqdm(
        desc="Calculating rate matrix for couplings",
        disable=NO_TQDM,
        total=(len(combinations) + n_couplings) // 2,
    )
    I_tot = integrate_intensity(intensity_func(integration_sample))
    for (i, coupling1), (j, coupling2) in combinations:
        if j < i:
            continue
        progress_bar.postfix = f"{coupling1.name} × {coupling2.name}"
        coup1 = to_regex(coupling1.name)
        coup2 = to_regex(coupling2.name)
        if coup1 == coup2:
            I_sub = sub_intensity(
                func, integration_sample, non_zero_couplings=[coup1]
            )
        else:
            I_sub = interference_intensity(func, integration_sample, [coup1], [coup2])
        decay_rates[i, j] = I_sub / I_tot
        if i != j:
            decay_rates[j, i] = decay_rates[i, j]
        progress_bar.update()
    progress_bar.close()
    return decay_rates

and run the computation for the LS model


Fix broken FilePicker parameter in README.md

Rectify the errors in the FilePicker example within the README.md file.

Nice


Error adding links

PS E:\ArchiveBox> archivebox add 'youtube.com'
[i] [2021-03-27 04:36:40] ArchiveBox v0.5.4: archivebox add youtube.com
    > E:\ArchiveBox

[!] Warning: Missing 6 recommended dependencies
    ! WGET_BINARY: wget (unable to detect version)
    ! SINGLEFILE_BINARY: single-file (unable to detect version)
      Hint: npm install --prefix . "git+https://github.com/ArchiveBox/ArchiveBox.git"
            or archivebox config --set SAVE_SINGLEFILE=False to silence this warning

    ! READABILITY_BINARY: readability-extractor (unable to detect version)
      Hint: npm install --prefix . "git+https://github.com/ArchiveBox/ArchiveBox.git"
            or archivebox config --set SAVE_READABILITY=False to silence this warning

    ! MERCURY_BINARY: mercury-parser (unable to detect version)
      Hint: npm install --prefix . "git+https://github.com/ArchiveBox/ArchiveBox.git"
            or archivebox config --set SAVE_MERCURY=False to silence this warning

    ! CHROME_BINARY: unable to find binary (unable to detect version)
    ! RIPGREP_BINARY: rg (unable to detect version)

[+] [2021-03-27 04:36:44] Adding 1 links to index (crawl depth=0)...
    > Saved verbatim input to sources/E:\ArchiveBox\sources\1616819804-import.txt
    > Parsed 0 URLs from input (Failed to parse)
    > Found 0 new URLs not already in index

[*] [2021-03-27 04:36:44] Writing 0 links to main index...
    √ E:\ArchiveBox\index.sqlite3

youtube.com is not a url, https://youtube.com is a url.


ims document OAuth2 Server-to-Server support and associated Workspace Configurations

With PR 166 we now have OAuth Support.
We still need however to document it.

with updated README.md files in the core and ims modules
with OAuth support built-in ims WorkspaceUtil and hence in our various *ServiceTester
as well as in our docs at https://developer.adobe.com/events/docs/guides/api/

Related:

#155
#163

the following associated PR https://github.com/adobe/aio-lib-java/pull/224 has been merged
consequently aio-lib-java version 2.0.0-SNAPSHOT has been released.
we will release a proper non snapshot version after a round of testing.


fix(cd): fix grepping the last release

The alpha deployment has been broken for 3 weeks, precisely since https://github.com/ratatui-org/ratatui/pull/495
(older first)

https://github.com/ratatui-org/ratatui/actions/runs/7304892972/job/19907742693
https://github.com/ratatui-org/ratatui/actions/runs/7360766607/job/20037290884
https://github.com/ratatui-org/ratatui/actions/runs/7428042304/job/20214757237

There was no log but testing locally I figured grepping the last release wasn't working because of \d not working with -E flag.
I used perl compatible regular expression -P, we might also use -E together with [0-9] if we prefer.
Once this is merged I think it's worth manually triggering the workflow to have an alpha after 0.25 and see if everything works.
EDIT: I also fixed a typo in a variable name https://github.com/ratatui-org/ratatui/commit/edd83a4cccafa103216685f66ca1b1d23ef7ec8e
EDIT 2: fixed the condition that was checking for breaking change in https://github.com/ratatui-org/ratatui/commit/afe44f061e30a0691614474e89c6d971cf3e7fc7

Local tests

I added fake tags to verify it greps the last release.

Manually triggered successful run https://github.com/ratatui-org/ratatui/actions/runs/7439071841/job/20238532651

Whoops ;). Thanks for the fix


The Common Crawl models: What data sets are they based on?

Hello!
I am writing my master's thesis on constructing corpora from the web. In it, I use the Common Crawl's WARC files to produce my corpus by extracting text and cleaning it myself.
The GloVe models trained on Common Crawl data is therefore a great point of comparison for me, but I can't seem to find what data it is based on: Is it the already extracted text provided by the Common Crawl (the WET files) or have you extracted text yourself from the WARC files?
Regards,
Kjetil Bugge Kristoffersen
MSc student,
Department of Informatics,
University of Oslo

Thank you. There's no one involved who would remember if they needed to strip HTML etc. or if they just used already extracted text?

See also #37


When bind to all interfaces is used IPV6 doesn't work.

When bind to all interfaces is used IPV6 doesn't work.
Consider changing it from
NET_BIND_ADDRESS="<IP_ADDRESS>"
to
#NET_BIND_ADDRESS="<IP_ADDRESS>"
May 13 03:15:17 tardis2 piaware[749]: mlat-client(859): Beast-format results connection with ::1:30104: [Errno 111] Connection refused
May 13 03:15:54 tardis2 piaware[749]: mlat-client(859): Beast-format results connection with <IP_ADDRESS>:30104: connection established
May 13 06:46:31 tardis2 piaware[749]: mlat-client(859): Beast-format results connection with <IP_ADDRESS>:30104: connection lost
May 13 06:47:31 tardis2 piaware[749]: mlat-client(859): Beast-format results connection with <IP_ADDRESS>:30104: connection established
May 13 09:19:28 tardis2 piaware[535]: Starting multilateration client: /usr/lib/piaware/helpers/fa-mlat-client --input-connect localhost:30005 --input-type dump1090 --results beast,connect,localhost:30104 --results beast,listen,30105 --results ext_basestation,listen,30106 --udp-transport <IP_ADDRESS>:7418:1982839014
May 13 09:19:29 tardis2 piaware[535]: mlat-client(586): Beast-format results connection with ::1:30104: [Errno 111] Connection refused
May 13 09:19:59 tardis2 piaware[535]: mlat-client(586): Beast-format results connection with <IP_ADDRESS>:30104: connection established
May 13 09:30:47 tardis2 piaware[535]: mlat-client(586): Beast-format results connection with <IP_ADDRESS>:30104: connection lost
Change script to comment out bind section
it now binds to IPV6
May 13 09:30:57 tardis2 piaware[1811]: Starting multilateration client: /usr/lib/piaware/helpers/fa-mlat-client --input-connect localhost:30005 --input-type dump1090 --results beast,connect,localhost:30104 --results beast,listen,30105 --results ext_basestation,listen,30106 --udp-transport <IP_ADDRESS>:7440:3602295029
May 13 09:30:58 tardis2 piaware[1811]: mlat-client(1861): Beast-format results connection with ::1:30104: connection established

Instead of setting the configuration variable NET_BIND_ADDRESS to <IP_ADDRESS> the script now comments out this variable in order to bind all interfaces.
Commit 376ab244f8c2f3127f0484d7046d006a7a640f93.


Add Code Analyzers to Duality Core Source

Summary
Switch the main development repository of Duality to Visual Studio 2015 to gain access to the new Analyzer feature and add some Analyzers to reduce reviewing workload on Pull Requests. This should not affect the minimum required version to develop plugins with Duality, only developing Duality itself will be upgraded. The framework version should not be changed.
Analysis

Visual Studio 2015 should provide access to the Analyzers feature
Code Analyzer availability is probably the main benefit of switching to VS 2015. This will allow to enforce coding style and other development guidelines and reduce the code review workload for new contributions.
For Duality plugins, nothing should be changed. Users can switch versions as they prefer and compatibility should be a bigger concern. VS 2015 is still comparatively new and there is no good reason to enforce this as a minspec for framework users.
Before doing this, make sure AppVeyor is able to compile Visual Studio 2015 projects and solutions. Edit: It's probably installed.

Progress:

Reviewed the available Analyzer ecosystem. A lot of them are big packages of rules where a large percentage isn't really usable for Duality or has never been a problem in code reviews, yet slows down compilation a lot. Some other rules that would have been expected or preferred do not exist.
Right now, a custom set of Duality-specific Analyzers seems preferable. That way, specific rules that are derived from code review experiences can be written and maintained one at a time.

Notes:

Apparently, the default way to deploy custom code analyzers is via NuGet package. Since there is no dependency between the analyzers and Duality itself, it would be a viable option to completely split the two and have all the Analyzers in a different solution file, or maybe even repository.

Notes:

Looking at the complexity of existing code style analzers, it seems like a bad idea to custom-write all required analyzers. Reconsider using existing style analzers.


Migrate to Kubernetes

Currently 52Poké uses Docker Swarm for container orchestration. Kubernetes has a lot advantages in config & secret management, infrastructure as code, volume management, integration with Linode Block Storage, etc.
We are beta-testing Linode Kubernetes Engine and we plan to deploy 52Poké to LKE when it becomes general available in Tokyo 2 region.
See wiki for details.

Linode Kubernetes Engine is now available in Tokyo datacenter!
https://www.linode.com/blog/kubernetes/lke-global-availability/

With final migration and DNS update pending, we have finished setting up the LKE cluster and needed resources.


Unable to find ReactNativePropRegistry (0.22.2 and 0.26)

This framework looks really good - Thank you!
Trying it now and running into this issue - I had used latest react-native 0.26 (react 15.0.2), also had it downgraded to 0.22.2 (react 0.14.8) - same issue. Any idea?

I've got the same issue 👍
Looking for a solution or a hint to solve this :)

I found a workaround by modifying my package.json dependency from "react-native": "^0.22.2", to "react-native": "^0.24",. After I did rm -rf node_module && npm install, I got the ReactNativePropRegistry.js in my node_module/react-native/Librairies/ReactNative and it works.

It has been fixed and is available in development branch. Not publishing now as we're planning on adding a few more features for the next release. Pull from the development branch if you need it urgently.

Looked through the code at development branch

The problem is that ReactNativePropRegistry from "react-native/Libraries/ReactNative/ReactNativePropRegistry" does not exist in RN 0.26 and above.
How can we resolve

#27

RN 0.26 supported with latest update.

I am using react-native v0.29 and still facing this issue. What will be the work around for this issue?

Which version of react are you using?

@sankhadeeproy007, I am using React V15.02

Can you check the react directory inside node_modules and see if the path exists?

@sankhadeeproy007,  fixed this issue by clearing the cache and re-installed node-modules

Which version of React and React-Native does the latest version of Native-Base support? I have the latest version of RN and R, and I'm facing the same problem.

same here

+1

same here

Issue is occuring again, looks like facebook removed the 'ReactNativePropsRegistry' file from version 15.4.1. It is never a good idea to depend directly on internal libraries used by a framework or calling internal APIs because they can easily change without backwards compatibility.

I have the same issue. I have
	"native-base": "^0.5.15",
	"react": "~15.4.0-rc.4",
	"react-native": "0.38.0",

I can't even run through the native-base tutorial.

I have found the same issue. I noticed though that  'ReactNativePropsRegistry' file is still there but in a different path now. Inside 'react-native/libraries' folder, if I recall correctly. Still, no idea how to update all files in native-base that refer to this lib.

I can with follow:
"native-base": "^0.5.15",     "react": "^15.3.2",     "react-dom": "^15.3.2",     "react-native": "^0.37.0"

Same issue

Fixed with latest release 0.5.18

"native-base": "0.5.13",     "react": "15.3.2",     "react-native": "0.32.0",
firstly i upgrade these dependency, then i got the error of RCTWebsocket (Android) and unknown module ......Nativebase/button.js. so i just revert back all the version changes to normal. but now im having this 'Unable to find ReactNativePropRegistry '. Someone please!

@thundenilla make sure the versions match up, "native-base: 0.5.15" and below will work with "react: 15.3.2" and below and "react-native: 0.35.0" and below. The possible causes to your issue could be that you are pulling in new version of react or RN whilst having an older version of native-base. Just to be sure, you could nuke everything in the npm cache and clear your node_modules then do a clean npm install. e.g.

npm cache clean
npm install
Could also help if your dependencies are exact e.g. "react": "15.3.2" instead of "react": "^15.3.2"

@Alappin so i just update the versions based on your recommendation. but somehow, its still there.

#336

same issue here with RN 0.45.1

@cinder92 Had the same issue (with 2.2.0) and solved it by upgrading to version 2.3.1 (https://github.com/GeekyAnts/NativeBase/releases/tag/v2.3.1)

not working. "react-native": "^0.47.0", "native-base": "2.3.1",


Wrap fix for FiniteDifferences into a Package Extension

When using ChainRulesTestUtils to test our frules and rrules, FiniteDifferences crashes because the serialization procedure FiniteDifferences.to_vec doesn't work well with Tensor{T,0}.
The reason is that to_vec tries to serialize the labels field which in the case of Tensor{T,0} is an empty tuple, Tuple{}.
Right now, the testing code has a small fix that would be nice to export as a Package Extension when Julia 1.9 arrives.
# fix for FiniteDifferences: `FiniteDifferences.to_vec` fails for Tensor{T,0}
function ChainRulesTestUtils.FiniteDifferences.to_vec(x::T) where {T<:Tensor}
    x_vec, Array_from_vec = ChainRulesTestUtils.FiniteDifferences.to_vec(parent(x))
    _labels = labels(x)
    _meta = x.meta

    Tensor_from_vec(y) = T(Array_from_vec(y), _labels; _meta)

    return x_vec, Tensor_from_vec
end

Fixed in a212c9fcb5a0dba5ff3d4bb908fac34bc6c41107

Triggering TagBot for merged registry pull request: https://github.com/bsc-quantic/Registry/pull/38


osx-arm64: missing qcollectiongenerator

trying to build pyside2-feedstock locally for osx_arm64 I run into this issue:
-- optional module Qt5XmlPatterns found (). Looked in: $PREFIX/lib/cmake/Qt5XmlPatterns
CMake Error at /Users/lo/mambaforge/conda-bld/pyside2_1652084189920/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_place/lib/cmake/Qt5Help/Qt5HelpConfig.cmake:14 (message):
  The imported target "Qt5::Help" references the file

     "/Users/lo/mambaforge/conda-bld/pyside2_1652084189920/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_place/bin/qcollectiongenerator"

  but this file does not exist.  Possible reasons include:

  * The file was deleted, renamed, or moved to another location.

  * An install or uninstall procedure did not complete successfully.

  * The installation package was faulty and contained

I looked in the filesI looked into the files and the file is indeed missing.

It looks like we need to add -make tools to the call to configure. Based on https://gitlab.kitware.com/utils/ci-utilities/-/merge_requests/10

Actually looking closer at this, I cannot find a reference to qcollectiongenerator in lib/cmake/Qt5Help/Qt5HelpConfig.cmake. Which version of qt-main are you pulling in?

thanks for taking care of this issue. Still I am not able to use this build as conda pulls the 5.15.2 build. Trying to pin 5.15.3 leads to a"conflicting packages" error. Any ideas?


Use Minio as Azure storage gateway

As Azure storage is not S3 compatible, Minio will need to be configured as a gateway for Azure storage
See Minio gateway for Azure

The latest Minio removed azure gateway

This is solved in https://github.com/gitpod-io/gitpod/pull/10717


Add text direction configuration for locales

The way we currently set right-to-left text direction is a little cumbersome and there is no easy way to switch direction between locales.
This PR adds text direction as a config option to the locale settings.

Fixed.


feat: add maximum length flag (#91)

adding the length flag in reference to #91.
i assumed the default value of the maximum length according to the 50/72 rule, but of course this is still an opinionated change and should be discussed according to what you guys think!

i still have an issue though with commands/prepare-commits-msg-hook.ts, should it:

pass the length as an argument from cli.ts to generateCommitMessage,
hardcode the length inside the generateCommitMessage invocation,
make the length argument optional with a default value of 50 (or something else)
something better around the process.argv.slice line? (probably bad for maintainability)

thx for reading!

I really want this feature, but sadly it has conflicts.

sure!


Internship: Add Fractal Badge

Add a link to Fractal's site as a thanks for hosting the internship
for the summer
TEST=manual
fractal_badge.png: https://patina-dev.nyc3.digitaloceanspaces.com/screenshot/Screenshot-2024-08-15T10:02:53-HenryChen-fractal_badge.png
Change-Id: If2b28920ba50ebb37d35d1a6c1df8686576d05e5

Code Review comments can be found at https://gerrithub.io/q/If2b28920ba50ebb37d35d1a6c1df8686576d05e5


Fix default target call

By default make begins by processing the first target. We included the test Makefile which contain multiple targets. So make called the first target from this Makefile. To solve this problem we put the default target to be the first target in the Makefile.

@bruntib 2000th pull request :tada:

:stuck_out_tongue:


Lookat behaviour of named tuple and/ or dictionaries in pytorch dataloader

Nested tuples are respected by pytorch dataloader. How is this for namedtuples?

@jasonkhadka can this be closed?


How to install doctrine/mongodb-odm on symfony 4.2 (windows)?

Hi there,
I'm trying to install mongodb-odm but with no result.
`λ composer require doctrine/mongodb-odm-bundle
Using version ^4.1 for doctrine/mongodb-odm-bundle
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Restricting packages listed in "symfony/symfony" to "4.2.*"
Your requirements could not be resolved to an installable set of packages.
Problem 1
- Installation request for doctrine/mongodb-odm-bundle ^4.1 -> satisfiable by doctrine/mongodb-odm-bundle[4.1.0].
- doctrine/mongodb-odm-bundle 4.1.0 requires symfony/console ^4.3.3|^5.0 -> no matching package found.`
I'm using  a php 7.3.5, mongodb 4.2.2 and symfony 4.2.
Anyone can help me? :)

As my colleague once said: "Read the error" :)

doctrine/mongodb-odm-bundle 4.1.0 requires symfony/console ^4.3.3|^5.0

I'm using a php 7.3.5, mongodb 4.2.2 and symfony 4.2.

There you have it. You're using a Symfony version (lower) that is no longer supported by doctrine/mongodb-odm-bundle you're asking for. Either upgrade Symfony or use one of the earlier bundle versions.


Repository tag author does not change on update from other author

Description
The author of a tag in the repository detail view does not change, if another author pushes an update for the tag. For example if a new latest tag was pushed.
Steps to reproduce

Author 1 pushes an image with the tag latest
Author 2 pushes another image with the tag latest

Expected behavior: The author name will be change.
Actual behavior: The creating author is displayed all the time.

Portus version: 2.4.3@5a616c0ef860567df5700708256f42505cdb9952

This issue still exists!

This issue still exists!

hi @websolutions-hamburg I am interested in this issue, have already solved several portus issues, maybe will solve this one as well...
Anyhow, to start with  :

There exists a 2.5 docker image of portus, see https://github.com/SUSE/Portus/issues/2241#issuecomment-589990298
try exact same test you ran, but this time docker image version 2.5 instead of the version you used.
my guess, (I bet 50% on that one)what the developers did in portus source code, is that :

the image tag author is set at tag creation time,
when the webhook coming from the registry, to portus is processed, every filed in the json object sent by registry, as payload for the webhook, does not contain an author field.
And the registry is right about that : there should not be any author field in the webhook payload, identity of the author should be infered by portus, based on the JWT token in the request, authenticating a portus user.
So I bet what the developers did is "just takethe payload and save it to database"
And I just described the fix without even opening the source code.

Waiting for your results with portus:2.5 docier image

Update : lost my bet, I wrote smthg utterly stupid, and learned more
Ok, so I think I lost my 50cents bet, and I wrote something stupid, so learned a deeper lesson on portus internals    :

No, when the registry send a webhook to portus, it should not use the ID token of the portus user who executed the docker pushcommand :

First, because portus doesn't want any user to have access to the notifications endpoint, reserved to registry
Second, because portus therefore wants to know it actually is the registry, who is sending the webhook, so the registry must have its own auth token, to authenticate to portus as the registry service sending a notification, not a user playing with his docker client. Or worse.
Third, because yes, it makes sense, that the notification webhook payload, the registry tells the portus who has just pushed the image ! Indeed, portus emphasizes so much on the accountability provided by portus, to a docker registry service.

I'm almost sure I found one field, in the registry's webhook payload json schema, one filed that is supposed to let the webhook recipient,  who docker pushed. (Or who signed the the pushed image ?) : see the  "pusher": "trustedbuilder", field, in the example below (coming from official docker docs, cf. https://docs.docker.com/docker-hub/webhooks/#example-webhook-payload) :

{
  "callback_url": "https://registry.hub.docker.com/u/svendowideit/testhook/hook/2141b5bi5i5b02bec211i4eeih0242eg11000a/",
  "push_data": {
    "images": [
        "27d47432a69bca5f2700e4dff7de0388ed65f9d3fb1ec645e2bc24c223dc1cc3",
        "51a9c7c1f8bb2fa19bcd09789a34e63f35abb80044bc10196e304f6634cc582c",
        "..."
    ],
    "pushed_at": 1.417566161e+09,
    "pusher": "trustedbuilder",
    "tag": "latest"
  },
  "repository": {
    "comment_count": 0,
    "date_created": 1.417494799e+09,
    "description": "",
    "dockerfile": "#\n# BUILD\u0009\u0009docker build -t svendowideit/apt-cacher .\n# RUN\u0009\u0009docker run -d -p 3142:3142 -name apt-cacher-run apt-cacher\n#\n# and then you can run containers with:\n# \u0009\u0009docker run -t -i -rm -e http_proxy http://<IP_ADDRESS>:3142/ debian bash\n#\nFROM\u0009\u0009ubuntu\n\n\nVOLUME\u0009\u0009[/var/cache/apt-cacher-ng]\nRUN\u0009\u0009apt-get update ; apt-get install -yq apt-cacher-ng\n\nEXPOSE \u0009\u00093142\nCMD\u0009\u0009chmod 777 /var/cache/apt-cacher-ng ; /etc/init.d/apt-cacher-ng start ; tail -f /var/log/apt-cacher-ng/*\n",
    "full_description": "Docker Hub based automated build from a GitHub repo",
    "is_official": false,
    "is_private": true,
    "is_trusted": true,
    "name": "testhook",
    "namespace": "svendowideit",
    "owner": "svendowideit",
    "repo_name": "svendowideit/testhook",
    "repo_url": "https://registry.hub.docker.com/u/svendowideit/testhook/",
    "star_count": 0,
    "status": "Active"
  }
}

Ok, so now I think we have serious data to solve this issue.

unstale

unstale

unstale


Cannot select any dates

We're using the date time picker in an Angular 2 app. The control is initialised to allow a date range which may be 1-20 years in the past up to todays date:
`
$(this.datePicker.nativeElement).datetimepicker(
    {
        format: "dddd MMM Do YYYY",
            maxDate: new Date(),
            minDate: this.model.InceptionDate,
            defaultDate: this.selectedDate
     }
 );

`
That all works fine, until the user zoons the control out to maximum zoom by clicking on the title of the control (at the moment, this shows a date range of 2000-2107). Once zoomed out, you cannot select any date in the dropdown to zoom back in - they all have the 'stop' icon on them.
You can close the picker by clicking somewhere else on the page, but next time you open it, the zoon is still at it's fullest and cannot be shrunk down. The only solution is to completely reload the page.
I was looking at handling the dp.change event, but that doesn't fire so can't help me. Is there some way to handle this elegantly? Can I restrict how far out people can zoom?
It's important to say, this problem only happens if you set the max and min dates. If these aren't set, then you always end up with something you can click on.

Hi, the latest version, 4.17.47, doesn't have this error.


feat: Adapt to<EMAIL_ADDRESS>

There are some breaking changes in<EMAIL_ADDRESS>Does the team have a plan to adapt to these changes?

We will eventually, but given how different it is from v1, it's likely going to take quite a while before we can migrate to it.


Support older version of esp-idf

Hello,
Right now this library doesn't support espidf 4.x.x, that is used by Arduino.
Making it compatible with espidf 4.x.x only requires changing
#ifdef CONFIG_IDF_TARGET_ESP8266
by
#if defined(CONFIG_IDF_TARGET_ESP8266) || ESP_IDF_VERSION_MAJOR == 4
Doing so, the library builds without any warnings (Probably because the esp8266 is stuck with espidf4.x.x ?).
That would allow people using Arduino to use the library with espidf WiFi functions.
Also, I've read issues #3 and #4

Hi!
Thanks for the corrections.

#if defined(CONFIG_IDF_TARGET_ESP8266) ||| ESP_IDF_VERSION_MAJOR == 4). This is an important addition. It will be accepted.
ESP_LOGE(TAG, ‘ESP-NOW message processing task internal error : at line %d’, LINE). This is an important addition. It will be accepted.
#define ESP_EVENT_BASE ZH_NETWORK. This is added only for information for doxygen documentation. This parameter has dependencies in the .c file and cannot be changed. Addition will not be accepted.
ZH_NETWORK_MAX_MESSAGE_SIZE: Changing this parameter in the user's main program code goes against my philosophy. The user should install this component for himself, change ZH_NETWORK_MAX_MESSAGE_SIZE if necessary and use it in all his projects. Changing the parameter in programmes can lead to differences and incompatibilities. The addition will not be accepted.
Moving header files. This is my programming style. Probably not the right one. But all my projects are done this way. The addition will not be accepted.

P.S. Since this repository is a mirror repository on my personal Git server, additions will be made to it, and updated on GitHub. Pull request will be closed with a full merge, but once changes are made to the repository on my Git server it will only be updated as described above. This will happen in a few days. At the moment some changes have been made to the component and it is being tested.
Thank you.


 So here is the sample where the expected result is not matching the reality:
import { compileExpression} from 'filtrex';
let myfilter = compileExpression("has_bla");
for (const x of [{ "has_bla": true }, { "has_nothing": true }]) {
    console.log(x)
    if (myfilter(x)) {
        console.log("Matched!")
    }
}

Result:
{ has_bla: true }
Matched!
{ has_nothing: true }
Matched!
//should not be matched the last one?

I faced this issue today. Filtrex seems to have a problem when a value is undefined in the object provided to the checker function (myfilter in your case), and it always resolves as true for boolean checks.
The following workaround seems to work fine in my case. I used a Proxy to make Filtrex think that the property exists, but with an empty string as value:
function createDataProxy(data)  {
  return new Proxy(data, {
    get(target, prop) {
      return prop in target ? target[prop] : "";
    },

    getOwnPropertyDescriptor(target, prop) {
      return prop in target
        ? Object.getOwnPropertyDescriptor(target, prop)
        : { enumerable: false, configurable: true, writable: true, value: "" };
    },
  });
};

const myfilter = compileExpression("has_bla");

myfilter(createDataProxy({ has_bla: true })) // true
myfilter(createDataProxy({ has_nothing: true })) // false

Side effect. This will make every value match to an empty string "".
I hope this helps

Okay great, then I am proved now that I have not overseen something ;-). I like your solution I will try it tomorrow, but it should be also fixed :-)

I agree. I don´t think the current behavior is expected or follows any logic.

I needed to update it to:
function filrex_proxy(data) {
    return new Proxy(data, {
        get(target, prop) {
            return prop in target && target[prop]!=null  ? target[prop] : "";
        },

        getOwnPropertyDescriptor(target, prop) {
            return prop in target && target[prop] != null
                ? Object.getOwnPropertyDescriptor(target, prop)
                : { enumerable: false, configurable: true, writable: true, value: "" };
        },
    });
};


gulp-uglify messing with code order

I have a stream of javascript files that include jQuery, another vendor library and some "in-house" javascript.
When I use uglify() on that stream, the other vendor library jumps on top of jQuery.
Why and how that happens, and how to fix it?
gulp.task('scripts', function () {

    var vendorFiles = gulp.src(mainBowerFiles())
        .pipe(gulpFilter('*.js'))
        .pipe(order([
            "**/jquery.js",
            "*.js"
        ]))
        .pipe(concat('vendor.js'))
        .pipe(debug({verbose:true}));

    var appFiles = gulp.src('app/js/*.js')
        // .pipe(jshint())
        // .pipe(jshint.reporter('default'))
        .pipe(concat('app.js'));

    return es.concat(vendorFiles, appFiles)
        .pipe(order([
            "vendor.js",
            "app.js"
        ]))
        .pipe(concat('app.js'))
        .pipe(uglify())
        .pipe(gulp.dest('./dist/js'));
});

Experienced this as well. @jonathandelgado's temporary fix worked for me.

Mind testing if the to-be 1.3.0 branch resolves this?
npm install git+https://github.com/terinjokes/gulp-uglify.git#uglifyjs-the-hard-way

@terinjokes I just did a test with your new code and the problem still exists for me. Please let me know if I can do some more tests.

@ronaldlokers - does the following syntax fix your issue:
.pipe(uglify({compress: {hoist_funs: false, hoist_vars: false}})

(This seems like this solves the issue for me as these are settings from http://lisperator.net/uglifyjs/compress that this library uses)

@scande3 ah, neither of these libraries wrap themselves in an IIFE? I suppose that could cause the issue seen here. I'll accept a pull request to add some documentation.

I don't know if this is still a valid issue, however I noticed similar behaviour when combining gulp-uglify with gulp-useref.
If I ran just gulp-useref then the order in the output file was the same as the order the scripts were declared. If I piped the output through uglify the order ended up being changed.
I found a similar issue being raised with Grunt uglify too and a solution posted in there also helped me. So basically the following config solved the issue:
.pipe(uglify({
    compress: {
        drop_console: true,
        hoist_funs: false
    }
})

Any solution for this?

@alvarotrigo I never received a code sample that showed the problem, but I believe this is an issue with the UglifyJS library this problem uses.

@terinjokes solved thanks to this answer.

@alvarotrigo The hoist_funs feature really shouldn't be breaking things: functions are hoisted to the top of the scope by the JavaScript engine during execution anyways. You should definitely file a bug with UglifyJS2.

@terinjokes I see. Make sense. Fair enough!

With no disrespect to anyone who's commented on this thread, this thread isn't tracking a specific issue. As it stands right now, there's nothing for me to do next here.
In preparing for the next major release, I'm going to go ahead and close out this ticket. At work we'd call this an "unsolved mystery". 👻
You're more than welcome to open new tickets with specifics and reproduction steps.
Thanks!


Add rshift validation for terminal states

Codecov Report

Merging #43 into master will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #43      +/-   ##
==========================================
+ Coverage   96.33%   96.36%   +0.03%
==========================================
  Files           5        5
  Lines         300      303       +3
  Branches       41       43       +2
==========================================
+ Hits          289      292       +3
  Misses         10       10
  Partials        1        1

Impacted Files
Coverage Δ

src/awsstepfuncs/state.py
95.43% <100.00%> (+0.05%)
:arrow_up:

Continue to review full report at Codecov.

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


RedirectUri and CacheStorage

If I specify a redirectUri via the constructor, shouldn't it be setting that value in the cacheStorage too?
new UserAgentApplication(
	environment.auth.webAppId,
	this.authority,
	this.authCallBack,
	{
             redirectUri: window.location.origin
	});

One example of a conflict:
if (window.parent === window && !isPopup) {
    window.location.href = self._cacheStorage.getItem(Constants.loginRequest);
}

Source code

The redirect Uri you configure is the one B2C will redirect to.
The following
 window.location.href = self._cacheStorage.getItem(Constants.loginRequest);
seems to always want to take you back to the page you were on when the loginredirect was initiated.

... I'm in a similar situation with an Angular SPA...

Hey @MarkHawcroft - any chance you can put that comment on #102?

Duplicate of #243


GitHub actions for CI/CD not publishing web app.

Current GitHub workflow results is a success execution, but the app reports
"Your web app is running and waiting for your content. Your web app is live, but we don’t have your content yet. If you’ve already deployed, it could take up to 5 minutes for your content to show up, so come back soon."
The published app.zip file is a nested collection of subfolders (\usr\share\dotnet\myapp) that results in the Web App not finding the web content.
This patch corrects the depth of the App contents in the deployed zip file.

Thank you so much, @rob-foulkrod!


How We can removed WeekView

Hi Team ,
Is any way we can removed WeekView?.
I pinned weekview on top but there is  some gap on top.

@kirankhalkar did you find a solution for this? I too am facing with the same issue. please help


How to reinstall rancher-nfs driver?

Hi Guys,
Is there anybody know how to reinstall rancher nfs driver on UI ?
Thanks.

Hi,

You can do this by deleting the nfs stack.

Then go to catalog and search for rancher nfs.

Then click Rancher NFS to install.

Hope this helps.

@arjuneyy Thanks!
@zhangalex Should do the job, let me know if you need any more help.

@arjuneyy @superseb
thank you very much !


Courses from the personal plan are not downloaded

Udeler | Udemy Course Downloader
1.13.2
Account Type?
Personal
Steps to reproduce
It used to be that a notification would pop up at the top and after you pressed it, it would become available for download. But that doesn't happen anymore. By the way, I had another course other than the personal plan, could it be because of this?
✔ Expected Behavior
It used to be that a notification would pop up at the top and after you pressed it, it would become available for download. But that doesn't happen anymore. By the way, I had another course other than the personal plan, could it be because of this?
🔍 Additional context
No response

This notification still exists and will always appear if no course is found!
When connecting, is any course from the personal account displayed?

fixed in new version 1.13.3


SR-2309: embed BlocksRuntime in libdispatch to eliminate external dep…

…endency
Eliminate the external dependency on libBlocksRuntime.so by
copying the blocks runtime code from swift-corelibs-foundation/closure
and building it into libdispatch.  Added a configure option to control
whether or not the blocks runtime is embedded; defaults to embedded on
Linux and not embedded on other platforms.  Since the blocks runtime is
embedded; we can take the link directive out of the module.modulemap file.
This change enables dispatch to be built and run on a system without
the libblocksruntime-dev package installed.

Can you remove the mention to libblocksruntime-dev from INSTALL then please? or maybe have it removed from the ohter pull-request that already is changing it right now? whichever you prefer.

yes.  I was going to update the list of dependencies after the other pull request landed.  I don't want to cause a conflict for @amraboelela.   We also should update to say that libpwq and libkqueue are optional.  I'll do both once the dust settles on INSTALL.md

sounds reasonable

Perhaps I do not understand the implications of this change but doesn't this change make this un-necessary?

Hi Ryan,  sorry if I got the help-string wrong.  I'll take a look and open a PR to fix it.   On linux configure should enable the embedded blocks runtime by default (so you don't need the configure flag).

It is no trouble @dgrove-oss. I merely wanted to bring it to your attention.
I realized that I did not actually need the flag after a few hours of playing around over the weekend. The reason I was exploring the flag in the first place was because I couldn't get it to build with the bundled blocks runtime.
After banging my head against a wall for a few hours I realized I was checking out a fork of the repository that did not have these changes. Suffice it to say I had a long weekend. :hurtrealbad:
Thank you for the work you are contributing here.


Slightly modifying DashboardIndex.cshtml prevents solution to be displayed; but compiles without any problems

Hi JaiQ,
John Ranger here again ;-)
I wonder if I am the only one which have the Problem below.
Currently, I want to modify the Dashboard to the application's Need.
To display some important values, I wanted to simply double the top 4 small boxes so that there would be 2 rows of 4 small boxes.
For this - within DashBoardIndex.cshtml, I copied the
<!--
Unfortunately, the visual studio compile&run (F5) process is not anymore able to display the dashboardIndex.html with any of the two Browsers (FF and Chrome) which I use in my dev Environment. I get a HTTP 404 (Server error in '/' application).
I had this also with the App when it was based on an older Serenity Version but this one is freshly built based on V1.9.13.
I don't understand the reason for this. From an HTML Point of view the code is correct (in my understanding). This actually should be the most easiest part for the whole Project as this is native MVC (in my understanding). I never had Display Problems when editing cshtml before outside of serenity Projects but within serenity, it seems to be extremely sensible.
Even when I now deleted the copied lines, the dashboardIndex.cshtml continues to bring the 404 error so it seems to have either something to do with Caching or something similar.
I have also already

terminated the IIS express process
cleaned and rebuilt solution
transformed T4 templates
Saved web.config to clear any caches

but this didn't resolve the Problem.
Is this a known effect to you and if yes, how can I prevent this but still modifying the Dashboard?
Thanks for your Feedback
and with Kind regards,
John

Sorry, I think I just have found the cause ;-)
it is because Visual Studio wants to directly load the cshtml page instead of loading the Standard entry page.
If I mark the .web Project and then compile&run the Project, it works.
Strange behavior of VS but eventually this helps someone which have the same Problem.
With Kind regards,
John


added the font Lotion : https://gitlab.com/nefertiti/lotion-dist

https://font.nina.coffee/
I'll email mx. belikova if this font goes live to see if they have a better description. I just copy-pasted the existing description on their site.
Rendered their name in lower case since they do that on their website.

How unique! Thanks so much! Looks good to me. (But yeah, a better description at some point would be great).


Refactor to be used by any fuzzy finder using vim.ui.select

Either of the following routes can be taken:

Use vim.ui.select
Support popular providers and use config option provider = 'telescope'

I'm leaning more towards (1) as most providers provide support to override the default implementation.
This will require updating the plugin name as well. For backwards compatibility, the telescope extension code will still be maintained.

Possible plugin name:

fuzzy-bookmarks.nvim
bookmarks-picker.nvim
browser-bookmarks.nvim

Initial implementation is in the feat/generic branch.


Add option to force reserve a locked or queued resource

We typically run into situation where we want to take out a bad resource form the pool programmatically, ignoring all the jobs queued or locking that resource.
This PR adds a new function doForceReserve that unlocks a resource and then reserves the resource.

Hello, just in case: is this the same use-case (and largely solution) as in PR #144 / issue #142 ?

Hello, just in case: is this the same use-case (and largely solution vector) as in PR #144 / issue #142 ?

Yes indeed! I will close this PR and upvote ton that


Notification actions stopped working after upgrade from 1.0.3 to 1.4.0

Feature request
Notification actions are an important part of iOS notifications which are missing from this package. I have been using https://github.com/holmesal/react-native-ios-notification-actions to handle notification actions in my app and they have been working well. But now after upgrade to 1.4.0, the actions have stopped working probably because of the move the newer UNUserNotificationCenter. I have no experience with iOS development due to which I am not able to make required changes. The react-native-ios-notification-actions is no longer maintained.
I request that the notification actions be supported by this package directly as it is an integral part of iOS notifications and it would avoid many such incompatibility problems in the future.
Why it is needed
iOS notification actions are an integral part of the notifications and should be supported by this package.
Possible implementation
It would require adding the notification categories, notification actions and passing the events to the JS side when an action is taken as mentioned here https://developer.apple.com/documentation/usernotifications/handling_notifications_and_notification-related_actions

@varungupta85 Notification Actions are now supported in v1.7.0 :tada:


Development

Modify the scores of PCA
pca.py.txt

Pull Request Test Coverage Report for Build 1361

5 of 10 (50.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 95.342%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pyod/test/test_knn.py
3
5
60.0%

pyod/models/knn.py
2
5
40.0%

Totals

Change from base Build 1283:
-0.1%

Covered Lines:
4114

Relevant Lines:
4315

💛  - Coveralls


[DeusVault] Adding the character sheet

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: < Deus Vault >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: < Deus Vault Commu > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: < Deus Vault > (i.e. Dungeons & Dragons, FATE)

[x] I have followed the Character Sheets Standards when building this sheet.

[x] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[x] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Adding the character sheet

Character Sheet Info Roll20 Internal Use only.

@gauthen Are these rules accessible somewhere or is this just a homebrew ruleset?

The rules are 100% homebrew and we are still working on them but we think the character sheet should not change too much.

@gauthen is there a document, something like a PDF, that people using this sheet can learn the game rules? How is the community supposed to learn the game and use the sheet?

Hello @mperes
I just added a link to the rules in the sheet.json


trim content of exit code file

@rultor release, tag is `0.11.1``

@rultor release, tag is `0.11.1``

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is `0.11.1``

@yegor256 Done! FYI, the full log is here (took me 1min)


Save & Quit Bug

If you edit the config of a mod and click save and quit and go back in the config right after without exiting the mod menu list GUI there are inconsistencies in the GUI. It seems to work properly if you edit a config, click Save & Quit, exit the mod menu mod list, and then go back to the config GUI but that extra step shouldn't be there. There's something off with the Save & Quit button.

This would be a problem with the mod that adds the config GUI and would have nothing to do with Mod Menu.


[v0.0.7] 3rd person camera broken with rr 2024.3.0

The 3rd person camera can't pan or rotate after the new update while this is installed. The 1st person works fine.

Fixed in the new mod update


⚠ Bofrost.at has degraded performance

In baf5f46, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2156 ms

Resolved: Bofrost.at performance has improved in e3f8154.


revert evendata back to original name

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Communication.CallAutomation


Teach package verifier about pipe tasks.

What this PR does / why we need it:
Collect keys from all pipetasks and feed that to template rendering in place of
an empty map. This has lots of room for improvement, but at least does not fail
on every single package which uses Pipe tasks.

Fixes #1398
Signed-off-by: Marcin Owsiany<EMAIL_ADDRESS>

@porridge as an FYI... the package verifier was intended to ignore new templating features it didn't know about...   I'm on board with writing a solution for pipes... but I would like to make sure that errors are not reported for unknown features.. than fix pipe verification... perhaps this does that... I will be looking at the solution shortly.

I agree with @zen-dog comments... also... It is worth the time to understand which commit caused this and why.   This package verify works with kudo v0.10.x and fails with v0.11.0.  There isn't a feature added that should have challenged this.
IMO there should be 2 distinct issues here:

what broke verification which includes pipes?  how did it break and lets resolve that.
Adding pipes specifically as a verification feature.   With this, I expect a pipe.ReferenceVerifier added to the verify.go.

With step 2 above... I would expect an error for all pipe defs which are incomplete...  pipes are more difficult to know if the var is used or not... which is unfortunate... but we should ensure that the pipe task is formed well.

@kensipe this PR addresses the first point, I just filed https://github.com/kudobuilder/kudo/issues/1407 about the second one.
@zen-dog I applied your suggestions.
PTAL


Change splash screen

As of now the splash screen is still the default one that comes bundled with React Native. I think this only pertains to iOS.

New splash screen added with b04ff05


Add PPA for bsdowl to the whitelist

Any update on this?  Is there anything blocking, unclear, whatever?

… ping …

… spoutnik …

Let's close this as whitelisting is not applied anymore.


Fix cluster restarting issues

This fixes some races when restarting a node in a cluster that may have hinted handoff writes queued on other nodes.
There were two issues:

The first was that the a node would start up and accept writes/queries possibly before the meta-store had finished apply the raft log.  This could cause sporadic errors.  In #3960, it caused writes to be dropped because the receiving node had no way of knowing that the write it received was for a shard that it should own but did not have in it's meta-store yet.
The second issue was that the tcp.Mux listener was handing demuxing in the same thread that accepted connections.  If a listener was slow or not ready to read off it's connection channel, connections would not be accepted and possibly timeout.  Because of 1, the restarting raft cluster node would sometimes get starved while trying to rejoin the cluster (because they share the same tcp.Mux listener).

A side-effect of fixing the first issue also make #3677 repeatedly fail which made it easy to find a subtle restart race in the test.
Fixes #3960 #3677

Seems reasonable, nice to fix that racy test.
+1 on a green build.


feat: Info page

Description
Taking the landing page that was created and putting it at /info. This page will be completely optional, however clicking the "View Wallet" button will still call setIntroCookie

@Pabl0cks Got that link added:


Improve Vulnerable Software Matching

To reduce false positives if the major version of the CPE being analyzed (3.2.12) is the same as a vulnerable software that contains two or more major versions marked with "and all previous versions" ( 3.2.7 and all previous versions, 4.0.1 and all previous versions) the match should only be based on the same major version (3.2.7 and all previous versions).
See Spring 3.2.12.RELEASE as an example for CVE-2014-1904.

The patch has been tested and resolves the described false positives.
Thanks for reporting the bug!


Validation Error while moving one API app from a resource groups with multiple API apps in some Resource Groups

While moving one azure API app from a resource group with multiple azure API apps within the same subscription, it throws a validation error. Tried the portal as well as PowerShell, no respite.
tracking id '40ea95b4-4c27-4738-9013-187690012a98', request correlation id '2ea747b7-bc73-49d5-9ead-94396a5ecdd5'.",
The error is "The resource '/subscriptions/{sub id}/resourceGroups/test-accounts/providers/Microsoft.Web/sites/test-v1-accounts-pxopt' is missing from the move request" but the api app I'm trying to move is test-v1-accounts-import. Both test-v1-accounts-pxopt and test-v1-accounts-import-pxopt are in same resource group

@ajayaldo
Thanks for the feedback! We are currently investigating and will update you shortly.

Thanks. I deleted the whole resource group, created a new one with two resources in it and tried to move one, still, I got the error.

@ajayaldo I replied to your other post as well. I have a break down of your latest failed migration for the mentioned subscription ID. Please email me directly at<EMAIL_ADDRESS>and I can share the results with you.
I've also sanitized this post of any potentially sensitive information.

Done..

@ajayaldo Since we provided you feedback externally regarding your move process, we will now proceed to close this thread. If you have further questions regarding this matter, please reopen the thread and we would be happy to continue the discussion.


Issue with the color of the submit button

The submit button is initially black, on entering dark mode it becomes green but on reverting back to light mode, the color still stays green. Assign me this issue.

Done

@adithyapaib I would love to work on this issue. Would you please assign this issue to me?