(WIP #735) Outgoing webhook system for integrations

Fixes: #735.
Orignal PR: #1393.

Automated message from Dropbox CLA bot
@vabs22, it looks like you've already signed the Dropbox CLA. Thanks!

@vabs22 can you fix the CI errors?
How much have you modified this from the original version vs. just rebased the branch?

@timabbott i haven't made any major changes. The core idea remains same. However at some places, the file had been deleted in latest commits, so i had made changes at other appropriate locations. Also i have removed some already satisfied dependencies.

I have updated this to the discussed model, added tests and support for personal message triggers besides @-mention triggers.

I have updated this to the discussed model, added tests and support for personal message triggers besides @-mention triggers.

Nice!  Sorry for the very slow review.  I'd like to try creating a few commits that we feel good about merging so that we can iterate on the later stuff from there.
I gave this a read, focused on the first few commits, and here's some thoughts on the first commit:

Services should probably be called Service?
I'd call service_name simply name, since it'll be usually accessed as service.name anyway.  Similarly for service_api_key; I think it might make sense to call it service.token, since it's not a api_key in the same sense that other things called api_key are (i.e. it isn't useful for auth to Zulip; it's for auth to the third-party site).

With those changes, and some comments documenting the intended use of the fields in the Service table I'm happy to merge that first commit.
The 2nd commit doesn't do anything, but here's my thinking on the third.

Given our experiences with interactive bots, I think the parsing logic in Outgoing Webhook System: Modify Bugdown for outgoing webhook bot triggers. is unnecessary.  It seems like we should instead just have logic in do_send_messages that, if a user that has flags.mentioned  (and later, or flags.has_alert_words, though we should just defer that feature until we have the rest working, since it's probably trivial to add once the rest of the framework is in, but will create clutter for now) and also is an outgoing webhook bot, then that user's messages should go into the outgoing webhook processing queue.  We can include a queue worker that just logs things that go into it for testing, and merge that infrastructure.  Future commits can then build off of it.

Does that plan make sense @vabs22?

@timabbott yeah this plan makes sense.

@timabbott I have dropped the 2nd and 3rd commit and added a new commit with the above logic. Please give your thoughts.

@timabbott I have updated the commit, please take a look.

@vabs22 thanks for working on this!  I merged the first few commits.  Then, I rewrote the code for checking which users were mentioned to avoid unnecessary database queries and setup a no-op queue processor and merged that as 8881b5eb9f499312a5ea192475669acdcfbf7a41.
I've pushed an updated version at timabbott/outgoing-webhooks that you should base your next work on.
Note that the branch doesn't work anymore, because I've changed the format of what gets put into the outgoing webhook queue; we'll need to update the code to work with that model.
I'm actually entirely sure what to do as the next phase.  Reading "Outgoing Webhook System: Add OutgoingWebhookService and generic interface." , it's ~300 lines of code that are fairly hard to follow with a lot of layers of classes that aren't well documented.
Gotta go right now but I'll hopefully have some notes for what to do next soon.

@vabs22 this PR was merged via #4741, right?  Closing under that assumption.


Auto-configure git to use LF endings

According to Professor Google:
https://www.aleksandrhovhannisyan.com/blog/crlf-vs-lf-normalizing-line-endings-in-git/#a-simple-gitattributes-config
...adding this file will make git enforce LF endings when committing to the repository.

Will this avoid corrupting binary files? (Probably, but worth asking).

Unclear from the documentation I've seen, but when I ran git add --renormalize . it didn't change any of the few binary files we have in the repo.


Make Unix casing consistent

Pre-Submission Checklist

[x] Your pull request targets the staging branch of FreeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm run test-challenges. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[ ] Closes currently open issue (replace XXXX with an issue no): Closes #XXXX

Description

Stumbled upon a slight inconsistency in the casing of Unix. It was capitalized properly in all but one occasions. Being the perfectionist that I am, I couldn't leave it like that 😄

LGTM :+1


Allow using ~/%HOME% when providing clone path, by expanding user home dir

I personally find this more useful than typing the path every time

I haven't tested but I you have

I haven't actually, let me try it out

Ok @stoivo , it works just fine 👍


Include memory usage of child processes in physical-memory-mb metric

Symptom: Some samza job will crash due to YARN PMEM error despite the physical-memory-mb is much lower than the container size.

Cause: Current physical-memory-mb metric only calculates the RSS memory of the java process that runs the application but ignores all its child processes, including those that load tensorflow models and take a lot of memory.

Changes: Get all the child processes of the java process that runs the application, and sum their RSS memory with the RSS memory of the java process as the physical-memory-mb of the container.
Test: unit tests
Tested with samza-hello-samza

lgtm


Sending a Node Buffer as a field in a message in binary format.

Is it possible to send a nodejs Buffer object without first base64 encoding it into a string?
If it is, it would be a helpful example to have, since binary data transmission isn't something JSON can do.

Assuming that you are talking about protobuf messages, a bytes field can be used to send a Buffer object.
Our interop tests use this. The message definitions for those tests are here.

Oh, they do! It would be good to have the documentation link to those or to have a documented example of it.


new version of enhsp

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (da8ab70) 84.96% compared to head (31976f5) 84.94%.
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #554      +/-   ##
==========================================
- Coverage   84.96%   84.94%   -0.02%
==========================================
  Files         200      200
  Lines       26433    26407      -26
==========================================
- Hits        22458    22432      -26
  Misses       3975     3975

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


Spread operators in object literals cause crashes in Edge/EdgeHTML

Bug Report
naja built source contains spread operator (...) in object literals, which is ES2018 feature. But Edge 12-18 (with EdgeHTML core) throws syntax error, because they implement only ES2015.
In the migration guide from v1 to v2, there is only mentioned that IE is no more supported, no mention about Edge. But .browserslistrc list last 2 versions which mean that old Edges with EdgeHTML core do not match anymore.

Hello, thanks for the report!

It [Naja] is written using modern-day JavaScript, but compiled into a bundle that works in all modern browsers.

As far as I am aware, Windows users should already have the Chromium-based Edge which has auto-updates, and the legacy Edge will no longer be supported as of next March, so it doesn't really count as a "modern browser" to me.
But you're right that this should be better clarified in the documentation. I'll take a look at it when I have a spare moment.

Well, that's true – for Win10 users only. :-\ So what do you suggest for those who need to support older Edges for people on older Win platforms? Switch back to naja v1?
The spread operator we are talking about is not a big deal I'd say – unlike XHR vs fetch issues. There are 5 occurences of it in whole naja. Still, I do not know if there will be any other problem once this would be fixed. 🤷♂

Ok, good point. I am reconsidering this. As much as I'm not particularly in favour of it, since Win 8.1 is obviously still in the extended support period until 2023, it makes sense to "support" the legacy Edge as well.
However: does it auto-update? Do all users have the latest version (18)? If so, then it seems to be just a matter of the spread operator, which I'm willing to transpile. Supporting the whole legacy Edge range is a no go though, as the resulting bundle would be twice as big as it is now.

In corporate environments, support is usually even longer. It does makes sense to me to support old Edge as corporate users cannot update to new Edge even if they want. 😉

@jkuchar So you mean to support all versions of old Edge, or is Edge 18 enough like @jiripudil wrote above?

@jiripudil What about version 16+ (last 3 old versions). Are the bundle double size on all versions below 18?

3 old versions is totaly OK. I would think that even suporting current + two version before would be sufficent. This will cover cases when some machines are not updated for various reasons (e.g. someone is travelling and is always on mobile data, etc.)

Not double, no. But I'm not sure I like it, even if it's just an extra kilobyte.
Strictly speaking, Naja 2.0 has dropped support for Internet Explorer, a browser which is still supported until August 2021, so supporting outdated versions of a browser which will lose support five months earlier seems a bit ridiculous 🙂 especially when all Windows users should automatically have updated to the Chromium-based Edge.
I'm willing to add edge 18 to browserslist so that the bundled code works in it. But playwright cannot run legacy Edge anyway, so please be aware that I can't actually support it to such extent as to be able to guarantee that all tests pass and everything works.

I mean, you can always tweak your build configuration to further transpile naja if you need to support any ancient browser (Edge 16 is almost 3 years old). It just doesn't make sense to me for Naja itself to support anything else than the latest version.

I'm not sure about release cycle of old Edge. What I mean that we need to give at least month or two for sysadmins to update to latest version. It obviously does NOT make sense to support 3 years old Edge 16.

@jiripudil Yes, you're right. I did not know about the IE state. That said, it's pretty similar situation. So if you've decided to drop IE support, then it makes probably sense not to support Edge as well.
Maybe we can wait if someone else using naja will have such problem as well and decide then. Meanwhile, I can patch package or something in project.

I've created this quickly and not foundany further problems in Edge 18.
diff --git a/node_modules/naja/dist/Naja.esm.js b/node_modules/naja/dist/Naja.esm.js
index bd4b5c5..e61823c 100644
--- a/node_modules/naja/dist/Naja.esm.js
+++ b/node_modules/naja/dist/Naja.esm.js
@@ -1172,12 +1172,8 @@ class Naja extends EventTarget {
       url = url.href;
     }

-    options = { ...this.defaultOptions,
-      ...options,
-      fetch: { ...(this.defaultOptions.fetch || {}),
-        ...(options.fetch || {})
-      }
-    };
+    const fetchOptions = Object.assign({}, this.defaultOptions.fetch || {}, options.fetch || {});
+    options = Object.assign({}, this.defaultOptions, options, {fetch: fetchOptions});

     if (method.toUpperCase() === 'GET' && data instanceof FormData) {
       const urlObject = new URL(url, location.href);
@@ -1191,14 +1187,15 @@ class Naja extends EventTarget {
     }

     const abortController = new AbortController();
-    const request = new Request(url, {
-      credentials: 'same-origin',
-      ...options.fetch,
-      method,
-      headers: new Headers(options.fetch.headers || {}),
-      body: data !== null && Object.getPrototypeOf(data) === Object.prototype ? new URLSearchParams(data) : data,
-      signal: abortController.signal
-    }); // impersonate XHR so that Nette can detect isAjax()
+
+    const requestOptions = Object.assign({}, {
+		credentials: 'same-origin',
+		method,
+		headers: new Headers(options.fetch.headers || {}),
+		body: data !== null && Object.getPrototypeOf(data) === Object.prototype ? new URLSearchParams(data) : data,
+		signal: abortController.signal
+	}, options.fetch);
+    const request = new Request(url, requestOptions);

     request.headers.set('X-Requested-With', 'XMLHttpRequest');


Rename "Analysis" to "Analyse" in German menu

This PR renames "Analysis" to "Analyse" in the German menu of the page.
Analysis is a German word, but it is not correct in this context, see https://www.duden.de/rechtschreibung/Analysis:

Teil der Mathematik, in dem mit Grenzwerten gearbeitet, die Infinitesimalrechnung angewendet wird

OR

Voruntersuchung beim Lösen geometrischer Aufgaben

I tested this change locally & everything was still correct.

Here is a preview of this change:

@Ein-Tim

The naming has been confusing from the beginning and it was agreed in https://github.com/corona-warn-app/cwa-website/issues/1776 to make it an English language word like the other top menu items.

It's unfortunate that the English and German words are so similar but not exactly the same.
The alternative design change would be to rename all the menu items to German when displaying any German page e.g. instead of "Science" write "Wissenschaft". At this point in time that would be quite disruptive though.
In short, I recommend no change to the menu.

@MikeMcC399
I disagree here, it is OK to have menu items "Science" or "Community" as they are widely known and do not have another meaning in Germany.
This is not the case for "Analysis" which has a different meaning in German than in English.
Also, after clicking on "Analysis", the page already shows "Analyse" at the top, so this is only an alignment of the menu item to the name of the page.

I have changed the title of this PR to make clearer that this is an alignment and nothing else.

@Ein-Tim

I have provided an alternative suggestion to resolve the problem you highlighted in this PR. The new issue is https://github.com/corona-warn-app/cwa-website/issues/2352 which does not involve changing the language of a menu item, nor does it involve any formatting changes, so it is very easy to implement. You are welcome to comment there.

Closing in favour of #2390.


FEATURE: allow ctrl+backspace to delete all

Is your feature request related to a problem?

Essentially, I want to do ctrl+backspace and I can't because that textbox is probably one of those old windows forms textbox :D
What solutions have you considered?
Implement the shortcut functionality
What alternatives have you considered?
Ctrl+A, Backspace works fine
Additional context
I'm just suggesting this quality of life feature, super minor thing.
To be real, I wouldn't mind doing it.
I use the tool in a daily basis and OSS always needs people involved :)
I'll look at an implementation in the weekend if you don't mind.
Cheers,
Code of Conduct

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

Hi @diomonogatari,
Thanks for this feature request!
You're right, this is almost certainly due to the WinForm text box. I've been toying with a rewrite in something modern like Maui, mainly with the goal of making v3 available on Mac or even mobile eventually. So this is definitely going into that rewrite's feature list!
Unfortunately, the original authors haven't documented what tools they used to create the MSI installer for JiraStopwatch, so while I have been tinkering in my own fork, I haven't been updating this repo as much as possible to avoid silently breaking anything until I regain the ability to create and publish new versions. If you're ok with sticking to CTRL+A+BCK for now, I'll keep this issue open as a reminder for the rewrite? Otherwise you are most welcome to open a PR for the fix if you'd like to try and reverse-engineer the installer?
In any case thanks for the suggestion, I appreciate it!


Extload doesn't work

Such words like "broadcasting.send" SOMETIMES highlight. Atto saw it, like hours ago, but now it doesn't. Extload also just doesn't work, it's not supported or something? Why? I can provide more information if needed.

Happy to help! Seems like you're having some strange issues... Using the broadcasting extension is working for me, though.
From what I hear with the specific issue you're facing, you might want to try emptying your browser cache and hard reloading. The way to do this varies between browsers — on Chrome, open DevTools (press Ctrl + Shift + I) and then go back to atto and right-click the refresh button in Chrome, before selecting empty cache and hard reload.
If you're able to upload any screenshots of the issues you're facing, that might help so I can see what's going on. If you could provide me with your OS and browser versions and any JavaScript error logs that are shown in DevTools, that would also be appreciated!
You might also want to refer to the example programs in the atto documentation for the broadcasting extension as they are known to be working fine.
Just to note that the atto broadcasting services are provided by a web server that currently goes to sleep after a while and spins up on-demand, and so an initial call to broadcasting.host may take up to a minute to finish in some cases — though I don't think this relates to the issues that you're having.

Thanks! The hard reset actually fixed it. I don't know why it broke though, but maybe it won't occure anymore - I don't remember extload working before.

You might also want to refer to the example programs in the atto documentation for the broadcasting extension as they are known to be working fine.
The example were also breaking immidetaly after seeing "extload", "broadcast" or etc.

No worries — glad it's working for you now! The browser cache can be a bit of a pain sometimes... If you come across this problem again, please do feel free to let me know!

Yep, for some reason it keeps breaking and I should do a reset everytime. This is the only thing I know right now.

I tried testing it multiple times, this is how DevTools outputs differ:
After a normal reset (devtools doesn't work in that case):

Cache reset and hard reset:

Should I reopen the issue?

The errors and warnings I can see in DevTools are what I get too, so they shouldn't be related to the issue you're having. If you can take any screenshots of your atto code/syntax highlighting problems that you've seen and send them over, then I can reopen the issue and might be able to try and work out a solution to things that are going wrong (along with what version of Chrome you have/the OS you're running)!

Version: latest, 112.0.5615.121 (Official build), (64 bit)
Computer:

Intel(R) Core(TM) i5-8400 CPU @ 2.80GHz   2.81 GHz
16,0 GB RAM (15,9 GB free)
64 bit

After a normal refresh:

After clearing cache and hard reset:

Okay, I reckon that it might be something to do with the service worker that's preferring an old version of atto from when attoX didn't exist. Any invalid command is errored with Unexpected end of line as atto expects a variable assignment expression here (eg. in this case extload=5), as can be seen in the example below:

I've invalidated the service worker's version so it should hopefully be forced to redownload everything. If you do a hard refresh at your end, it should now hopefully be fine from here on! If not, then in DevTools, go to the Application tab at the top, then Service Workers section, then press Unregister — that should do the trick (service workers can be weird sometimes).

Thanks!! It actually worked. Now extload works and for i=0 to 50:draw random*500,random*500:next works correctly, which means that atto is in the latest version now.


bump version in package.json to v2.21.6

release version v2.21.6

 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.


Update core to 3.11.12

Fixes CVE-2023-26051 and CVE-2023-26052.

I think we can close this one, as I bumped the version as soon as the new image was published.

Duplicate of https://github.com/saleor/saleor-platform/commit/3392632eb1ffcc1425f6984964c7258d1a3390a7


Table column width miscalculated when value contains "accent" Unicode character

Information

OS: Windows
Version: 0.49.1
Terminal: Windows Terminal

Describe the bug
When I display strings containing Unicode accent characters in the table context:

the width of the table seems miscalculated
there seems to be a whitespace rendered after the characters

I tried troubleshooting (e.g. went through Best Practices -> Configuring the Windows Terminal For Unicode and Emoji Support) but with no difference. I am curious if this is a limitation of a Terminal, a bug, or if I am misusing the library and it can be fixed with a configuration change?
To Reproduce
static void Main(string[] args)
{
    var table = new Table();
    table.Border = TableBorder.Rounded;

    table.AddColumn("Field");
    table.AddColumn("Value");

    table.AddRow("Row 1", "1");
    table.AddRow("Row 2", "2 ąśłćż");
    table.AddRow("Row 3", "3 єшерти");
    table.AddRow("Row 4", "4 áb́ćd́"); // \u0301 is the issue here

    AnsiConsole.Write(table);
}

Expected behavior
A table is rendered with the same column width in all rows.
Screenshots

Additional context

Windows Terminal uses "Cascadia Code" font, as described in Best Practices
"Use Unicode UTF-8 for worldwide language support" checkbox is selected in my OS:

Thanks for your work and a great library! :)
Please upvote :+1: this issue if you are interested in it.

No, that doesn't seem to be a bug in the wcwidth library. That effect looks very much just like ye olde Windows Console behavior.
Your strings use combining marks. The combining mark uses one character cell in the console output buffer, hence the empty space you see. It's not possible to workaround this by shifting the cursor position one to the left after outputting a combining mark in an attempt to make the empty space available for output. Because then you are going to overwrite (and thus lose) the combining marks in the output. (Have been there, done that...)
For limited scenarios, string.Normalize() might be used to convert combining marks and their preceding character into single characters. However, this is not 100% bullet-proof and still might leave you with combining marks, as there are no single (Unicode) characters for all possible combinations of combining marks and their preceding character (as is the case with b́, for example).

@elgonzo Thank you for the explanation!
It indeed looks like an issue with specific terminals like cmd.exe rather than the library. I think there is nothing more to do on the side of Spectre.Console.
As an additional test, I pasted a simple echo "áb́ćd́éf́" to Windows Terminal and saw the same erroneous behavior with whitespace.
Workaround
I'll paste the workaround I ended up with in case someone with a similar problem finds this thread.

First, I use string.Normalize() to replace characters with ones showing wider compatibility where it's possible
Then, I remove accent characters remaining in the string. I lose some accent marks in the console output, but it's an acceptable tradeoff for me to keep the output readable.

Screenshot

Code
// Variant 1
// (accent marks were the only characters problematic for Windows Terminal that I found)
Console.WriteLine("Strings without normalization:");
var table = new Table();
table.AddColumns("Field", "Value");
table.AddRow("Row 1", "ABCDEF");
table.AddRow("Row 2", "ąęúłśż");
table.AddRow("Row 3", "áéúíüñ");
table.AddRow("Row 4", "абвцде");
table.AddRow("Row 5", "а\u0301б\u0301в\u0301ц\u0301д\u0301е\u0301");
table.AddRow("Row 6", "a\u0301b\u0301c\u0301d\u0301e\u0301f\u0301");
table.AddRow("Row 7", "👍👎👌👏👋👊");
table.AddRow("Row 8", "你好嗎？我很");
table.AddRow("Row 9", "🇵🇱🇧🇷🇨🇦🇺🇸🇬🇧🇦🇺");
table.AddRow("Row 10", "أبجد ه");
AnsiConsole.Write(table);

// Variant 2
Console.WriteLine("Normalized with `string.Normalize(NormalizationForm.FormC)`:");
var table2 = new Table();
table2.AddColumns("Field", "Value");
table2.AddRow("Row 1", "ABCDEF".Normalize());
table2.AddRow("Row 5", "а\u0301б\u0301в\u0301ц\u0301д\u0301е\u0301".Normalize());
table2.AddRow("Row 6", "a\u0301b\u0301c\u0301d\u0301e\u0301f\u0301".Normalize());
AnsiConsole.Write(table2);

// Variant 3
Console.WriteLine("Normalized with `string.Normalize(NormalizationForm.FormC)`, remaining accent characters removed:");
string RemoveAccentMarks(string input) => string.Concat(input.Where(c => CharUnicodeInfo.GetUnicodeCategory(c) != UnicodeCategory.NonSpacingMark));
var table3 = new Table();
table3.AddColumns("Field", "Value");
table3.AddRow("Row 1", "ABCDEF".Normalize());
table3.AddRow("Row 5", RemoveAccentMarks("а\u0301б\u0301в\u0301ц\u0301д\u0301е\u0301".Normalize()));
table3.AddRow("Row 6", RemoveAccentMarks("a\u0301b\u0301c\u0301d\u0301e\u0301f\u0301".Normalize()));
AnsiConsole.Write(table3);

Thanks again for the support!


Getting GMT timezone

Hi, why am I getting GMT timezone rather than my local timezone?

Did you find a solution?  I'm also suddenly GMT in iOS Simulator.  My laptop is set to Pacific Standard Time.  I tried restarting my laptop.


[Merge] Report on quality of execution payloads produced

Description
Give an indication of how good execution payloads are in blocks we produce:

Time between making the preparePayload call and the getPayload call (how long did we give the EL to produce the block?)
Gas used vs gas limit - how "full" was the produced block
For MEV may be able to report the rewards we were paid

Not sure if this should be a log message or metrics. May need to experiment to see what's more useful.

closing this ticket, the rest is covered in #5654


Feature/make integration tests independent

Added cms helpers to quickly login/logout/add/remove page/addplugin
added setup / teardown for every test, so they can be run separately
changed some tests because they were a bit too strict

This. Is. Awesome. :+1:

:+1:

LGTM :+1:


Added popup window

Creating dynamic works sections by JavaScript, added pop up window class styles and sources, finish the mobile and desktop popup window.

@BenMukebo, Hello, thanks for your comments, I have a question regarding the description part of each project, since the Figma template shows a specific text, should I leave the mobile and written description for all projects and customize the title and image What would change for each project when clicking on "see project"?


Could not resolve resource

SWA Extension doesn't work for me. Getting error: Could not resolve resource.

Steps to reproduce:

Install Extension
Log in

Version: 1.95.3 (system setup)
Commit: f1a4fb101478ce6ec82fe9627c43efbf9e98c813
Date: 2024-11-13T14:50:04.152Z
Electron: 32.2.1
ElectronBuildId: 10427718
Chromium: 128.0.6613.186
Node.js: 20.18.0
V8: 12.8.374.38-electron.0
OS: Windows_NT x64 10.0.19045

I created them via Azure Portal.

@yury-romanov We currently only support SWA built through GitHub Actions right now.

That's an interesting decision, considering that not only Static Web App created via Portal recommends to use this extension, but also that this information is not mentioned on the extension page.

Sorry for the lack of information on the Portal-- they didn't really confer with us on this.
Unfortunately, when we first developed the SWA extension, GitHub was the only method of creating.  Since then, the service has been releasing more ways to create apps, but we unfortunately haven't had the bandwidth to invest time in the SWA extension.
We are hoping to be able to spend some time on this soon though.


[Planting Flow] Modify Geolocate Button to match new designed buttons

While the geolocate button is a generic button style, I would like the button to be more prominent of a button on mobile. I want it to match the new design of button that is minimal.

Tasks

Geolocate On - Mode
Geolocate Off - Mode
Geolcate Disabled - Mode

@zoobot Im back working on the enhancing the ux for the planting flow. I am starting with the geolocate buttons. I started with making them larger for mobile, and giving different actions depending on the situation. The first action is geolocate is off/disabled, the second is geolocate is active, the last is the default state of geolocate when no action has been taken. I like this as the first step to making it clear for the

user when trying to locate themselves.


Check why some recepies don't build on singularity hub

Some containers are not built on singularity-hub.org.

Fixed in 3636087c6013ba2faeaba6cc243b69baa6e9cef3


Update git authentication method

Email from github:
On February 7th, 2020 at 21:47 (UTC) your application (SPDX Online Tools) used an access token (with the User-Agent python-requests/2.20.1) as part of a query parameter to access an endpoint through the GitHub API:
https://api.github.com/user
Please use the Authorization HTTP header instead as using the access_token query parameter is deprecated and will be removed July 1st, 2020.
Visit https://developer.github.com/changes/2019-11-05-deprecated-passwords-and-authorizations-api/#authenticating-using-query-parameters for more information.

@goneall Do we need to completely remove the github authentication and replace it with the oauth authentication or authorization headers.

Do we need to completely remove the github authentication and replace it with the oauth authentication or authorization headers.

That is my understanding from reading the github documentation at https://developer.github.com/changes/2019-11-05-deprecated-passwords-and-authorizations-api/#authenticating-using-query-parameters

@anshuldutt21 Did you get a chance to look into this issue?  @rtgdk do you know which features will be impacted by this?  I haven't gotten any emails recently from Github, so I'm not sure if this is still an issue.

@goneall No, I couldn't find a good alternative for the access token. Is there any problem with the authentication currently since in the mail it said access token parameter will be removed by July 1st.

@anshuldutt21 @goneall The issue is with passing "access_token" in query url as parameter(?access_token=)...instead we should pass it through the api headers. In the email, the api mentioned is "https://api.github.com/user" which is only used in testing test_submitlicense_api_with_valid_fields in api/tests.py which passes the token correctly (in headers). I am not sure why we got this error on Feb 7th and not later on since this test used headers from the beginning. @Ugtan Do you remember if you used this api with access token as params?

@rtgdk Correct. The only place we used the user api is in test_submitlicense_api_with_valid_fields. I don't think we have used access_token as a query string anywhere else in the codebase.
I'm not quite sure but I think one of the libraries we are using uses the access_token as a query parameter. It could be social-auth-app-django but I'm not sure. We can update the libraries to check if everything works fine. Also, I just notice that the deadline for deprecation is extended until May 2021 from the article.

Good news on the deadline being moved out to May 2021 (thanks @Ugtan for the research).  It would still be nice to close on this, so perhaps we can update the libraries.

I started getting the emails again about this being deprecated.
I found this issue in social-auth-app-django : https://github.com/python-social-auth/social-core/issues/430
which was fixed in PR https://github.com/python-social-auth/social-core/pull/428
which is in version 3.4.0.
We're current at version 2.1.0.
It looks like updating to version 3.4.0 should resolve this.

PLease reopen if we get that email again.


ARIA COMBOBOX MULTISELECT closing in Esc

Hi Bryan
I notice that for keyboard users who focus to the input and then activate the checkboxes using the arrow key navigation and Space to make the selections, pressing Esc closes the list. However for pointing device (mouse) users there is no intuitive way to dismiss the list after making selection/s, other than clicking on the X button, which is not 'obvious' as it resembles a cancel/close button visually and being red is more confusing as it seems destructive.
I suggest changing the image Close button to:

An HTML button
Replace the icon image role button, and have a visible label 'Apply'.

This
<img id="mobileCloseIcon" class="" src="img/close.png" role="button" title="Close Suggestion Dropdown" alt="Close" aria-label="Close Suggestion Dropdown">
To
<button type="submit">Apply</button>
Applies to:

ARIA COMBOBOX (NATIVE INPUTS, MULTISELECT EDITABLE WITH SUBSTRING MATCH
ARIA COMBOBOX (SIMULATED, READONLY MULTISELECT)

Sounds great to me, do you want to make that change? That’s just markup right? I can make any scripting changes if these are needed.
From: Laurence Lewis @.>
Sent: Wednesday, April 10, 2024 11:44 PM
To: WhatSock/apex @.>
Cc: Subscribed @.***>
Subject: [WhatSock/apex] ARIA COMBOBOX MULTISELECT closing in Esc (Issue #55)
Hi Bryan
I notice that for keyboard users who focus to the input and then activate the checkboxes using the arrow key navigation and Space to make the selections, pressing Esc closes the list. However for pointing device (mouse) users there is no intuitive way to dismiss the list after making selection/s, other than clicking on the X button, which is not 'obvious' as it resembles a cancel/close button visually and being red is more confusing as it seems destructive.
I suggest changing the image Close button to:

An HTML button
Replace the icon image role button, and have a visible label 'Apply'.

This

To
Apply
Applies to:

ARIA COMBOBOX (NATIVE INPUTS, MULTISELECT EDITABLE WITH SUBSTRING MATCH https://whatsock.com/Templates/Comboboxes/Native Inputs, Multiselect Editable with Substring Match/index.htm
ARIA COMBOBOX (SIMULATED, READONLY MULTISELECT) https://whatsock.com/Templates/Comboboxes/Simulated, Readonly Multiselect/index.htm

—
Reply to this email directly, view it on GitHub https://github.com/WhatSock/apex/issues/55 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AAOHK2AMIDWVXYS3C5PWXO3Y4YWJ7AVCNFSM6AAAAABGBWYRBWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGIZTOMBRHE4TCMI .
You are receiving this because you are subscribed to this thread.  https://github.com/notifications/beacon/AAOHK2D2AD5FM57VWDZ2SNLY4YWJ7A5CNFSM6AAAAABGBWYRBWWGG33NNVSW45C7OR4XAZNFJFZXG5LFVJRW63LNMVXHIX3JMTHIKVRXA4.gif Message ID: @.*** @.***> >

No problem @accdc will update and create a PR.

Bryan I have uploaded the following changes to 'ARIA COMBOBOX
(NATIVE INPUTS, MULTISELECT EDITABLE WITH SUBSTRING MATCH)'

Updated the icon to use a less destructive visual indicator for manually collapsing and setting the list.
Wrapped the new SVG icon in an HTML Button
The icon is permanent and not initially hidden. I found the sudden appearance of the icon when making selections jarring (personal opinion add the hidden class to the button if you wish this to only show when a selection is made from the listbox)
Added HTML autocomplete=off to the input to  prevent the browser making suggestions (should have done this before but missed it in previous testing)

I think I captured all the changes.


Added command line completion for :diffget and :diffput

:diffget and :diffput now complete file names which are in
diff mode and which are different than the current buffer.
No completion when current buffer is not in diff mode.

Codecov Report

Merging #5409 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #5409      +/-   ##
==========================================
- Coverage    82.7%   82.69%   -0.01%
==========================================
  Files         134      134
  Lines      147606   147615       +9
==========================================
- Hits       122074   122068       -6
- Misses      25532    25547      +15

Impacted Files
Coverage Δ

src/usercmd.c
94.87% <ø> (ø)
:arrow_up:

src/buffer.c
85.73% <100%> (+0.1%)
:arrow_up:

src/cmdexpand.c
82% <100%> (+0.07%)
:arrow_up:

src/mbyte.c
65.83% <0%> (-0.58%)
:arrow_down:

src/profiler.c
94.11% <0%> (-0.57%)
:arrow_down:

src/if_xcmdsrv.c
85.99% <0%> (-0.36%)
:arrow_down:

src/gui.c
63.43% <0%> (-0.21%)
:arrow_down:

src/channel.c
84.22% <0%> (-0.08%)
:arrow_down:

src/terminal.c
81.81% <0%> (-0.04%)
:arrow_down:

src/window.c
88.27% <0%> (+0.03%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

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


Token/Scope Resolution is Incorrect for C++, Java, and Rust

I just updated from insiders 1.49 to insiders 1.50, and after the update Token resolution is incorrect for C++, Java, and to a lesser degree Rust, causing certain tokens to be highlighted incorrectly. I don't know if this is occurring in other languages as well, as these are the only three I use.
For example, according the Editor Tokens and Scopes Inspector, member variables are being picked up as:
semantic token type: "member"
foreground: "entity.name.function.member, entity.name.function, support.function, support.constant.handlebars"
when they should be picked up as "variable.member" or similar
and exception specifiers are being picked up as:
textmate scopes: "variable.other.object.access.cpp, meta.body.class.cpp, meta.block.class.cpp, source.cpp"
foreground: "variable"
When these should be picked up as "keyword" or similar.
In Java I'm encountering things like member variables being picked up correctly in class declarations (picked up as member variables), but not when used in method definitions (picked up as local variables). I'm also noticing incorrect token resolution in Rust, but not nearly as strange (eg procedural macros being picked up as functions)
Extension installed that may be related:
vscode-clangd
cpptools
rust-analyzer
vscode-maven
vscode-java-dependency
redhat.java
OS: Windows 10 1903

I'll file one per language shortly. Thanks!


Add mongo datastore with specs

Ok, here's the results of my initial spike on this. All of my specs are passing, and I tried to keep them pretty close to what was in the specs for the other backends. Let me know what needs to be fixed/tightened up.

This looks really good. I just have a few minor points.

The mongo gem is not defined as a dependency anywhere. We can put it in the
Gemfile for now, but we probably need to figure out a better way for people
to install either the riak or the mongo client.

We are working on support for ruby 1.8, so do you mind converting the
hashes back to the old style?

Do you mind removing the class<<self in favor of def self.
methods? This will match our other data stores.

It looks like you use the environment in the db name rather than the
collection name. That seems correct, but your spec is still for collection
name. Can you update the spec (line 113)?

Cool. No problems. Thanks for the feedback!

I made a couple of changes, and then rebased and merged in:

I moved mongo from the gemspec to the Gemfile for now, until we figure out how to do conditional dependencies for different back-ends

The Curator.configure blocks were bleeding between tests, so I pulled them into around blocks

Thanks again for your contributions!


Multi depot

Draft for multiple depot support. This PR:

[x] Closes #305.
[x] Adds and passes relevant tests.
[x] Has well-formatted code and documentation.

With the new depot argument we can also finally update the argument order of VehicleType (breaking change) to be a bit saner. This should hopefully be the last time such an update is needed.
Benchmarks on VRPTW
Main from #395 (ec6ed4513dc6e4ae7710fe5423275e04d20466ac; job ID 6529558):
                MEAN       MIN       MAX  COV
      inst
obj   C1     41370.4   41351.2   41392.3  0.0
      C2     16192.3   16189.6   16195.9  0.0
      R1     47230.6   47197.5   47274.4  0.1
      R2     27763.8   27732.0   27796.4  0.1
      RC1    44236.3   44214.9   44273.0  0.0
      RC2    23250.6   23236.7   23268.5  0.1
iters C1     67931.2   61537.8   74987.3  6.1
      C2    116731.3  106070.6  130550.6  6.4
      R1     39931.5   37421.3   43141.4  5.2
      R2     59910.4   54129.5   66751.9  8.1
      RC1    43526.6   39641.3   47665.2  6.2
      RC2    62497.9   58802.7   70464.7  6.7

This PR (f0c10498735d536da11510c349ab8fb6ce197af6; job ID 6552495):
QUEUED

Notes:

It is essential that you add a test when making code changes.
This keeps the code coverage level up, and helps ensure the changes work as intended.
When fixing a bug, you must add a test that would produce the bug in the master branch, and then show that it is fixed with the new code.
New code additions must be well formatted. Changes should pass the pre-commit workflow, which you can set up locally using pre-commit.
Docstring additions must render correctly, including escapes and LaTeX.

I think this now mostly works.

@leonlan @wouterkool this should now be good to go. I've checked with a benchmark that performance on VRPTW has not degraded, so this should not impact our current set of supported VRPs. I have added support for multiple depots in this PR, and written tests to check the basic functionality is in place.
What's still missing are benchmarks on MDVRPTW instances. Those need to be formatted appropriately first, and for that I need to extend the VRPLIB format a bit with some new sections (particularly something like a VEHICLE_MAX_DURATION or MAX_DURATIONspecification. That's part of https://github.com/PyVRP/Instances/pull/6 and will also need an update to our read function. I propose we do that in a follow-up PR.

@wouterkool thanks for the review! Looks doable. I'll try to finish this over the weekend.

@wouterkool GitHub won't let me comment on this directly. In BPD, you wrote:

I think you can count depots vs clients here as depots have neighbours unassigned but I'm also fine with this solution.

I don't think this'd work with prize collecting, because then also unassigned clients have no neighbours. As far as I can tell we'd have no way to differentiate the two cases (depots vs. unassigned clients).

Doesn't TwoOpt.cpp also need a change? I think swapping the tails of two routes changes the distances from the last customers to (different) depots.

Yes, I'm working on that right now. Thank you and Joop for the failing tests!

How does VRPLIB interpret 'num vehicles' with multiple depots and do we support that (see comment in read.py)

Short answer for now is it does not. Leon came up with a very nice extension proposal in #252 that we can iterate on to get there. This will need some work in either case.

Should we also make a config for MD-VRPTW? The VRPTW config does not include the SwapRoutes operator now.

Yes, I think so. I'll add something in right now, but leave the details (tuning?) to the PR that adds the benchmarks.

This new implementation for TwoOpt now seems correct to me. @wouterkool could you have another look?


Rviz no tf data

Hi all , I had just picked up ROS recently for a project that I am currently working on regarding autonomous robots. So I was trying to get the mapping process going , I have with me a UPsqaured connected to a RPlidar A1 and I am accessing the UPsquared through my computer.  I launched the rplidar launch file first , then the gmapping launch file from another terminal. On the gmapping.launch terminal I got this error message :
[ WARN] [1577691725.782039473]: MessageFilter [target=odom ]: Dropped 100.00% of messages so far. Please turn the [ros.gmapping.message_notifier] rosconsole logger to DEBUG for more information.
I do not know how to solve this error and am stuck , please help.

Hi @benjamin1203 , this doesn't seem to be related to towr, so I'm closing this here. You should be able to find some anwers on ros.org (related to rviz). Best of luck!


@freetube/yt-comment-scraper will be archived

Hello,
We are planning on archiving the @freetube/yt-comment-scraper module in the near future. I thought I'd reach out to the modules dependants to let them know so they can migrate to a new module if they would like.
FreeTube is switching to https://github.com/LuanRT/YouTube.js but I'm sure there are other modules that you could switch to.
https://github.com/FreeTubeApp/yt-comment-scraper/issues/92

Look at #23 for fixes and removing @freetube/yt-comment-scraper module


Package Managers Working Group announcement

👷 work in progress 🚧

Closing as there isn't a working group anymore


I cannot create a world

When i use /mvcreate to create a world, it displays Failed. and i can't import that world.
command: /mvcreate create normal -t flat
Console Log: https://paste.ubuntu.com/p/JSG76dGcbZ/

Interestingly enough, I have been seeing more and more of these reports, so I'm beginning to think something's wrong.
Here are some things to try:

Make sure your server software is up to date, be it Spigot or Paper. CraftBukkit is generally not recommended these days and isn't a great choice.
Make sure you're Multiverse Core and its subplugins are up to date. The latest versions of each plugin are available at here.
Try deleting the generated world's folder (in this case, the create folder) and removing the entry from Multiverse Core's worlds.yml file. Then try to create it again.

Thank you for your answer, but that didn't help. It displays Failed. again.
Console log: https://paste.ubuntu.com/p/2dbF9qGkTz/

I'm using the latest version Multiverse-Core 4.1.0 and the server version is Spigot 1.15.2

I've been trying to reproduce this but I've been having some trouble. Can you please provide the following:

Which host you are using (or if you're self hosting)
Which version of Java your server runs on
The output of /version
The output of /mv version

The server is running on my computer (Windows 10).
Java version: Click here
/version output: Click here
/mv version output: Click here

I can use /mvcreate to create some worlds before, but now I can't.

You can't create any world?

Yes.

I'm a Chinese and I'm not good at English.

I used /mvcreate nether2 nether just now and here is the output in the console.

That's okay. Are there any errors on startup or any errors on shutdown?

Let me have a look.

Here's the output of Multiverse-Core.

I don't know if that's an error.

There aren't any errors when the server is shutting down.

Okay. I have some more questions:

Is your system's language English? If not, which language?
Do you have Administrator access and sufficient privileges to the folder containing the server?

My system language is Chinese.
I'm the Administrator user of my computer.

Can you try with Paper and provide the log if you get a similar error?

Paper?

Yes, PaperMC, a fork of Spigot.
Available here.

I'll try that later. Thank you.

I'm downloading PaperMC, but I still want to use Spigot.
Can you fix those errors in Spigot?
Thank you very much!

We'll fix the issue if it's an issue with Spigot. I just want to see if it also happens with Paper.

OK.

There aren't any errors when I use /mvcreate in Paper. I hope that never will be wrong.

Thanks for giving it a try. Can you upload your server.properties, bukkit.yml, and spigot.yml files?

Here's the server.properties.

spigot.yml
bukkit.yml

In your server.properties, make sure to set gamemode, then try to create a world!

Yeah! I can create world now! Thank you very much.

Multiverse-Core is really a good plugin.

Awesome! Thanks for your kind remark. I'll see if something can be done to prevent this in the future.

But I have another question:
When using /reload, the config of Multiverse-Core will not save, so I must use /stop to restart.
Can you fix this error?

I will look into it, although, using /reload is generally not recommended.

I know that, but /reload is faster than /stop, so I like to use /reload


"Disabling Domain Reloading" not working

Describe the bug

Enter Play Mode faster in Unity 2019.3

Project Settings -> Editor -> Enter Play Mode Option (true)
first play: "AtomScheduler" object created. UniMob working well
stop play mode.
second play: "AtomScheduler" object not created. UniMob not working

Expected behavior

When "Disabling Domain Reloading", AtomScheduler should be created.
static variable should be cleared.
https://github.com/codewriter-packages/UniMob/blob/59a60bbdd6630d6869858a703a7803bb30ccb46a/Runtime/Core/AtomScheduler.cs#L21
maybe this document useful.
unity document for Domain Reloading
Desktop:
Unity version:
UniMob version: 59a60bbdd6630d6869858a703a7803bb30ccb46a
Other plugins installed:

I added Domain Reload support in a domain-reload-support branch. Can you check it works correctly in your project?

your code works well. 👍
note: my solution
https://github.com/5minlab/UniMob/commit/98bbbea86bbc611e721e7247e260af37c5c9ff63
Actually, AtomScheduler is singleton. So I use OnDestroy to reset static variable.


🐛 BUG: dyld error on earlier MacOS versions

Which Cloudflare product(s) does this pertain to?
Wrangler
What version(s) of the tool(s) are you using?
3.80.4 [Wrangler]
What version of Node are you using?
20.10.0
What operating system and version are you using?
MacOS Big Sur 11.7.10
Describe the Bug
Observed behavior
A dyld: Symbol not found error is thrown when running wrangler on earlier MacOS versions.
I've observed the error being thrown on the following commands:

wrangler dev
wrangler d1 execute
wrangler r2 object put

I've observed that the latest version to work as expected is wrangler 3.78.9, i.e. the error is thrown from wrangler 3.78.10 onwards.
This issue seems to be related to another issue I had previously reported: 🐛 BUG: Minimum required version for local development #3307.
Expected behavior
No dyld: Symbol not found error is thrown.
Steps to reproduce

Run npm create cloudflare@latest to setup a new worker
Run npx wrangler dev to start a local development server
Throws error

 ⛅ wrangler 3.80.4
-------------------

⎔ Starting local server...
✘ [ERROR] dyld: Symbol not found: __ZNKSt3__115basic_stringbufIcNS_11char_traitsIcEENS_9allocatorIcEEE3strEv

    Referenced from:
  /Users/<User>/Desktop/<Project>/node_modules/@cloudflare/workerd-darwin-64/bin/workerd
  (which was built for Mac OS X 13.5)
    Expected in: /usr/lib/libc++.1.dylib

/Users/<User>/Desktop/<Project>/node_modules/wrangler/wrangler-dist/cli.js:29768
            throw a;
            ^

Error: write EPIPE
    at WriteWrap.onWriteComplete [as oncomplete] (node:internal/stream_base_commons:94:16)
    at WriteWrap.callbackTrampoline (node:internal/async_hooks:130:17) {
  errno: -32,
  code: 'EPIPE',
  syscall: 'write'
}

Run npm create cloudflare@latest to setup a new worker
Run wrangler r2 object put to put an object in an r2 bucket
Throws error

 ⛅ wrangler 3.80.4
-------------------

Creating object "<Object>" in bucket "<Bucket>".
dyld: Symbol not found: __ZNKSt3__115basic_stringbufIcNS_11char_traitsIcEENS_9allocatorIcEEE3strEv
  Referenced from: /Users/<User>/Desktop/<Project>/node_modules/@cloudflare/workerd-darwin-64/bin/workerd (which was built for Mac OS X 13.5)
  Expected in: /usr/lib/libc++.1.dylib

Error: write EPIPE

Run npm create cloudflare@latest to setup a new worker
Run wrangler d1 execute to execute an sql file on a d1 database
Throws error

 ⛅ wrangler 3.80.4
-------------------

🌀 Executing on local database <Database> (<UUID>) from .wrangler/state/v3/d1:
🌀 To execute on your remote database, add a --remote flag to your wrangler command.
dyld: Symbol not found: __ZNKSt3__115basic_stringbufIcNS_11char_traitsIcEENS_9allocatorIcEEE3strEv
  Referenced from: /Users/<User>/Desktop/<Project>/node_modules/@cloudflare/workerd-darwin-64/bin/workerd (which was built for Mac OS X 13.5)
  Expected in: /usr/lib/libc++.1.dylib

Error: write EPIPE

Hey unfortunately we don't support end of life OS versions, in line with workerd. We were actually just about to add this to the cloudflare wrangler docs literally today, so apologies for the inconvenience 😅


Bigtable: add an experiments flag to enable client side metrics

Add an experiments flag enable_client_side_metrics to enable client side metricshttps://cloud.google.com/bigtable/docs/client-side-metrics

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

BigQueryIOWriteTest known flake not related


Add Amazon Braket and cirq requirements files

Currently the directions to test out Benchpress are as follows: python -m pytest --benchmark-min-rounds=1 benchpress/*_gym which includes the Amazon Braket gym and Cirq gym; however, these two are not included in any of the requirements files.

Yes, we have not kept up to date with those sdks as of late since they have a larger disparity with Qiskit.  It is on the list of things to do though.


Unhandled exception when help icon clicked.

To Reproduce

Click the question mark

Screenshots

FWIW, it might be better to just list on the main screen the 4 keys and what they do, rather than clicking for additional information.


Assert.Throws in ToolLocationHelper_Tests

I randomly hit this test when breaking into a test assembly, and noticed it was SUPER verbose when it could be short and sweet.

@dotnet-bot test this please

@dotnet-bot test Windows_NT please


onRotate prop work only every second attempt

<ViroARSceneNavigator
	initialScene={{
		scene: RenderScene,
	}}
	style={styles.arView}
	viroAppProps={{
		isReady,
		onQuadPress,
	}}
/>

<ViroARScene
	onTrackingUpdated={onTrackingUpdated}
	anchorDetectionTypes="PlanesHorizontal"
	onCameraARHitTest={onCameraARHitTest}
>
	<ViroAmbientLight
		color={'#fff'}
		influenceBitMask={1}
		intensity={500}
	/>
	<ViroDirectionalLight color="#ffffff" direction={[0, -1, -0.2]} />

	<ViroBox
		ref={viroBox}
		rotation={rotation}
		onRotate={onRotate}
		position={[0, 0, -3]}
	/>
</ViroARScene>

const [rotation, setRotation] = useState([0, 0, 0]);

const viroBox = useRef(null);

const onRotate = (rotateState, rotationFactor, source) => {
	console.log('rotateState', rotateState);
	if (rotateState == 3) {
		setRotation(old => {
			return [
				old[0],
				old[1] + rotationFactor,
				old[2],
			]
		});

		return;
	}

	viroBox.current.setNativeProps({
		rotation: [
			rotation[0],
			rotation[1] + rotationFactor,
			rotation[2],
		],
	});
};

The cube rotates on every second attempt.
How can I make the onRotate prop run every time?

Project create from starter-kit
"dependencies": {
    "@viro-community/react-viro": "2.21.1",
    "lottie-react-native": "^5.1.3",
    "react": "17.0.2",
    "react-native": "0.65.1",
    "react-native-get-random-values": "^1.8.0",
    "uuid": "^8.3.2"
 },

onRotate don't work on Xiaomi phones (Redmi Note 10)
engine - arcore

Hi @illi-homz I'm experiencing the same issue on Xiaomi Redmi Note 10. I've kind a made this issue less painful by the following way:
I've explicitelly set the rotationPivot position to [0, 0, 0] on the node i am trying to rotate to.
With this setup I went down from 8 to 12 unsucessfull attempts to rotate to 2 to 4 attempts.
Tested like 100times on development and release builds
Not a proper fix just an improvement...

@illi-homz . I figured out that on react-native 0.67.5 and viro-react 2.23.0 all events works for me. Even onPinch and onRotate works really well on every device I've tesed and had issues before. I have to notice that it works good only if I have Bitcode enabled in Xcode for the whole project and every pods, same with proguard on android. Hope this helps.


⚠ DDoS-Relay EU-C-fra1-03 has degraded performance

In 6cfc014, DDoS-Relay EU-C-fra1-03 (http://<IP_ADDRESS>:42069/;csv) experienced degraded performance:

HTTP code: 200
Response time: 185 ms

Resolved: DDoS-Relay EU-C-fra1-03 performance has improved in 26b769a after 24 minutes.


Fixed invalid whitespace character causing inability to parse moment.js

Please see the attached screenshots from Chrome 56.0.2924.87.
It appears there is an invalid whitespace character in the source code in src/lib/locale/set.js. I removed that character and added a standard space which appears to have fixed the problem. I still ran the tests and confirmed all passed.
This can be reproduced by adding the following index.html file in the repo at the root of the project and loading it in chrome.
I should note that I put the whitespace replacement into the file referenced above, then ran grunt release and confirmed that it did indeed fix the issue.
# index.html
<html>
<head>
  <script src="moment.js"></script>
</head>
<body>
</body>
</html>

Thanks guys! After looking at some open issues I may try and help out in that respect as time allows. Appreciate the warm welcome :)

I'll cut a release in around 5hrs

YTMND @ichernev

Merged in 863f8617946bbd60bf5c5ace0fe79a7577fd8441

Enjoy 2.18.1

Hi guys,
Has this fix been released yet? We're getting loads of issues about this in our project graph3d.


Expose hot restart to flutter_driver

Use case
Writing e2e tests could be more automated, and less time consuming, if we could wipe app state at the end of any given test. Often, we don't want the state from the previous test to carry over to a new test or workflow that we're testing.
Proposal
Expose a method on FlutterDriver that performs a hot restart on the app that's running for the driver. (i.e. FlutterDriver.restartApp), or add a flag to the driver that can configure it to restart at the end of every test. (i.e. FlutterDriver.restartAfterEachTest = true`).
more context
My team is re-writting our company's app in Flutter from iOS. The QAEs are attempting to port the E2E tests from our current app with Flutter Driver. Currently, they're written using cucumber.
There actually is a cucumber port on Pub that offers this feature. It accomplishes this by writing an "R" to stdin to the vm running the app while driver is running. That library isn't great for our use case, I'm just pointing it out for context

This is very interesting and useful, at the moment it should be doable using requestData in this way:
  enableFlutterDriverExtension(handler: (message) async {
    if(message == "hot_reload") {
      Process.killPid(PID_OF_YOUR_FLUTTER_RUN_PROCESS, ProcessSignal.sigusr2);
    }
  });

await driver.requestData("hot_reload")

But this will add additional flakiness to the tests since as far as I know there is no way to understand when the hot reload process is one, so it might be possible that you start the hot reload and the test kicks too soon causing the test to fail since the app is not ready yet.
Also, in trying this one out I got the driver to crash with this error VMServiceFlutterDriver: Instance of '_WebSocketImpl' is closed with an unexpected code 1005 so I think it's probably possible to be implemented as a custom behavior but requires a bit more investigations.
Hopefully, somebody can point me in the right direction here.

cc: @jonahwilliams

actually, you cannot send that kill signal from the driver as it runs in the app, and the process runs on your pc.
What you could do is send that signal from your tests, we tried it but an hot restarts causes the driver connection to drop as @nerder  mentioned, at the moment we have a script that launches e2e tests and forces hot restart before launching the driver test. you need a separate test file for each time you need to hot restart but at least it works.
probably if you have a setup method that closes the driver connection, hot restarts and then reopens the driver connection than you would be able to have all of the hot restart logic inside your tests... will update with a solution if this works

This is how I achieve hot-restart in my flutter driver tests:
test_app.dart:
import 'package:flutter_driver/driver_extension.dart';
import 'package:gemini/main/main_mock.dart' as app;
import 'package:hydrated_bloc/hydrated_bloc.dart';

void main() {
  enableFlutterDriverExtension(handler: (command) async {
    switch (command) {
      case 'restart':
        await HydratedBloc.storage.clear(); // clearing cache between each hot-restart
        _startApp();
        return 'ok';
    }
    throw Exception('Unknown command');
  });
  _startApp();
}

void _startApp() {
  app.main(); // this is call to runApp(...)
}

in my tests in setUp before each test:
    setUp(() async {
      await driver.restart();
    });

and the extension:
extension DriverExt on FlutterDriver {
  Future<void> restart() async {
    await requestData('restart');
  }
}

@lhimo are you having this issue with integration_test? Since integration_test tests are written using flutter_test, the app state should be re-created whenever [pumpWidget()](https://api.flutter.dev/flutter/flutter_test/WidgetTester/pumpWidget.html) is invoked.


chore: bump kind version to v0.22.0

Description of the change:
We use kind as the environment for e2e tests. This PR updates kind to the latest release
Motivation for the change:
Architectural changes:

Testing remarks:

Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Bug fixes are accompanied by regression test(s)
[ ] e2e tests and flake fixes are accompanied evidence of flake testing, e.g. executing the test 100(0) times
[ ] tech debt/todo is accompanied by issue link(s) in comments in the surrounding code
[ ] Tests are comprehensible, e.g. Ginkgo DSL is being used appropriately
[ ] Docs updated or added to /doc
[ ] Commit messages sensible and descriptive
[ ] Tests marked as [FLAKE] are truly flaky and have an issue
[ ] Code is properly formatted

Hi @perdasilva is the vendor folder still needed given the fact go.mod and go.sum exist?

close in favor of #3222


Amit/bug fixes

Description
This PR addresses the following bugs in playground:

Changing of code not reflecting when resending a transaction or script
Changing of transaction parameters not reflecting when resending a transaction or script

Some of these fixes are done by reverting previously made changes in commits from a few months ago which I noticed were causing these bugs to come up on the production site. Specifically those changes that were reverted were put in to add some memoization for performance reasons. I don't have much context on how important those changes were, so would greatly appreciate some extra context around that.
Manually tested across many example scripts + transactions.
Reproduction steps which you can run on play.onflow.org and on this local version to see the difference in functionality:

Open Playground on fresh incognito browser.
Head to default created transaction.
Remove references to helloworld
Modify parameter like so: transaction(a: Int) {
Add log in prepare like so: log(a)
Fill in int param, and click send.
It often fails first time due to a different bug which I have not fixed (Error: GraphQL error: addresses must be 8 bytes). If so, just modify the code again (space and backspace)
Add more params and logs, and verify that clicking send works as expected for each change

FYI - This branch might be breaking some other things, so probably isn't ready for merging, but should serve as a reference for how to fix those above bugs at least as a base

afaik, Playground is not supporting incognito mode 🤔

afaik, Playground is not supporting incognito mode

That would explain why I couldn't test contracts in incognito :sweat_smile:
The transactions + script testing does work reliably in incognito if not deploying/using contracts.
Same test scenario can be run without incognito though.


Add total moves

Add a function to calculate the total moves of the player to move all the rings from rod A to rod C. Every time a ring is moved, a move will be incremented. The no. of moves can be displayed in the blank space on the bottom of the screen,  and the total score can be displayed while the game is over or while moving to the next level. If issue #5 is solved, add the moves in the navbar.

@ishanbagchi I can do this one.

@elijahnavarro, sure. Go for it.


wrong maker invoked on default window when multiple tabs open

I've got three tabs open in my session:

some sql files in various splits
some ruby files in various splits
nvimrc

If I:

switch to nvimrc
save the session and quit
restart neovim

...I see my nvimrc file has lots of warnings and errors from mri and rubocop (the ruby linters). If I then:

switch to the ruby tab
save session and quit
restart neovim
switch to the nvimrc tab

I see no errors or warnings.
It feels like a race condition of some kind, because if I switch from:
autocmd! BufWinEnter * Neomake
autocmd! BufWritePost * Neomake

to
autocmd! VimEnter * Neomake
autocmd! BufWritePost * Neomake

then I don't see that behaviour (but obviously I have to write a file to see new lint, which is not ideal).

This behavior should have improved by now (although there are still issues in that regard).
Closing this one for now, please re-open in case I'm wrong.


Make 'recently_added' page accessible for logged-in users only

Page is currently visible for everyone, it should be visible only for users who are logged in. Ideally, this should be the page to land on after logging in for researchers/editors/publishers.

The first part was easy; just a matter of some clicking in the admin. The landing page after logging seems extremely well hidden in the code.

Ha! Success!


Improve support for UMD module documentation

Having attempted to get JSDoc to document my modules, the results are, quite frankly, atrocious, without excessively verbose and repetitive annotation of documentation comments. This really destroys part of the purpose of documentation--I partially want to make sure via automatic generation that none of my code is undocumented, and if the documentation requires adding lots of notes just to get JSDoc to tell that it exists, then it's a problem.
Here is a pared-down example of the code I'm trying to document:
/**
 * This module contains a bunch of utilities for SASL implementers, partially
 * to help bridge differences between Node.js and web browsers.
 * @module sasl-utils
 */
(function (root, factory) {
  if (typeof define === 'function' && define.amd) {
    define([], function () { return factory(atob); });
  } else if (typeof exports === 'object') {
    // Shim functions for node.js
    function atob(str) {
      return new Buffer(str, "base64").toString("binary");
    }
    module.exports = factory(atob);
  } else {
    root.saslUtils = factory(atob);
  }
}(this, function (atob) {
/**
 * Convert a string containing base64-encoded data into a Uint8Array containing
 * that data.
 *
 * @param {String} str The base64-encoded string.
 * @returns {String}   The resulting base64-decoded string.
 */
function base64ToBinaryString(str) {
  return atob(str);
}

return {
  base64ToBinaryString: base64ToBinaryString,
};
}));

I've tried many methods to figure out how to get the return value registered, such as using putting /** @alias module:sasl-utils */ in many strategic places, but at no point could I get documentation to be detected except by adding /** @alias module:sasl-utils.base64ToBinaryString */ or similar code to every single function.
 I've done enough debugging of the issue that I've identified that the key issue is that the documentation for the functions is being parsed as "<anonymous>~base64ToBinaryString" and there is a correct "module:sasl-utils.base64ToBinaryString" being generated--but the correct one being generated is happening at the object literal in the return statement, which is not identified as referring to the function that's getting the anonymous tag. Therefore, the declaration in the returned object literal is dropped as undocumented and the documented declaration is dropped as being anonymous.
Unfortunately, even though I've poked the code enough to figure out that this is happening, I haven't figured out how to fix the object literal to merge the two declarations together. :frowning:

I don't have a better solution for you than using the @alias tag. If you can't live with that, you'll have to find a less "atrocious" tool that doesn't "destroy" your documentation and leave you feeling "irate."

So you are completely against my suggestion in the last paragraph of merging the documentation tags based on the returned object literal?


Scripts not reading in song's data folder

What is your question?
I found this weird issue of scripts not loading inside a song's data folder, how do I fix it?

it loads scripts inside a song's data folder just fine. can you show a video?

Fixed it myself

can you say how you fixed it? Also others can know how to?

You need to match the data folder to the song folder so that the scripts can load

You need to match the data folder to the song folder so that the scripts can load

alright, and yeah they do need to match since the game can’t find the song files if the song and data folders don’t match


Declare NavModel.router for TypeScript projects

Hi,
I'm using a singleton view model inside a child router.  When the view model is instantiated, the wrong router is injected (it's the AppRouter instead of the child router).
One workaround I found is to get the router from routeConfig.navModel.router in the activate method.
However, the router property is not declared in TypeScript so I have to use a cast to any.
It is possible to add NavModel.routerin TypeScript?
See code sample below:
@singleton()
@autoinject()
export class OrderList {
  orderService: OrderService;
  orders: Order[];
  router: Router;

  constructor(router: Router, orderService: OrderService) {
    this.orderService = orderService;
    //this.router = router; // <-- Not the router I want

  }

  activate(params: any, routeConfig: RouteConfig, navigationInstruction: NavigationInstruction): void {
      this.router = (routeConfig.navModel as any).router as Router; // This is the router I need

  }
}

The reason for this is that you've made a singleton, which lives at the root container where the root router also lives, so the child router can't be injected (because it's not existing at that level of the DI).
I've added the router property to the nav model as requested.


Realm Studio Version 11.0.0-beta.1 not compatible with Realm v10.8.0-beta.2

Goals
Browse the realm database generated using Realm v10.8.0-beta.2.
Expected Results
As per the doc, Realm Studio Version 11.0.0-beta.1 should be compatible.
Actual Results
Opening Realm files of format version 22 is not supported by this version of Realm
Steps for others to Reproduce
Use Realm v10.8.0-beta.2 with a swift application. Make use of the new feature, AnyRealmValue. Write to the DB. Try opening the file with the above mentioned version of Realm Studio.
Code Sample
NA
Version of Realm and Tooling
Realm v10.8.0-beta.2
Realm Studio Version 11.0.0-beta.1
Xcode version: ?
Version 12.5
iOS/OSX version: ?
11.1

HI @ravikirankhareedi You can report this on the realm studio repository page https://github.com/realm/realm-studio/issues, This will take you directly to the people taking care of this.


⚠ Bofrost.de has degraded performance

In 60dedcf, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 961 ms

Resolved: Bofrost.de performance has improved in db99973.


Debbuging with mariadb

Hello!! i try to o a debugg with mariadb and i have the error "Driver mariadb no support debugging"
I try to ativate debuging in Driver mariadbconnector J
https://mariadb.com/kb/en/about-mariadb-connector-j/#debugging
The problems persinst!! if mysql works i think what in mariadb should be work, please help!1 i need a do a debug
Im using DBeaver last version of comunity
Mariab server 10.5.8
Mariadb connector J Last version 2.7.2
Windows 10 with all last updates
My server mariadb is locally and imnot using a vpn or ssl o wherever

It is not implemented yet in DBeaver. We will be working on your ticket in 21.1.1 where we are going to improve debugger.

Hello @bfharreguy,
If you're talking about debugging stored routines, then it is not possible. MySQL and MariaDB don't provide any debugging API.
But you can debug the server itself:
https://dev.mysql.com/doc/refman/5.7/en/debugging-server.html


feature request: datasets and kernels delete (cleansing after competition)

I'd like to perform a cleansing and delete a lot of kernels and datasets that I generated during imet-2019-fgvc6.
The idea is to preserve just a few ones.
This functionality is available using web interface.  But it's a toil to click and confirm hundreds of times.
Now, competitions are turing into Kernels Only modality, and probably there are more people interested in delete the 90% of temp work.
Thanks in advance!

It is a complete disappointment, more and more competitions are kernel-only and must-have endpoints for full automation like this are still missing. Kernels UI is laggy and it takes at least 5 minutes to update dataset every single time. My last kernel competition until it is fixed.


Update more dependencies

Codecov Report

Merging #502 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #502   +/-   ##
=======================================
  Coverage   99.75%   99.75%
=======================================
  Files         213      213
  Lines        5203     5203
  Branches     1216     1216
=======================================
  Hits         5190     5190
  Misses         13       13

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 a2aeb31...8e312e2. Read the comment docs.


Fix bug with TAP reporter using file hashes as filenames

When I wrote this reporter, I assumed that the files was an array of
strings (partly because that is how it was documented). However, in
practice, it is an array of hashes, so this didn't work correctly.

@sds I have tested this locally and it is working correctly now. Any chance we can get a patch release out today?

I have another small commit in the works on top of this one, FYI


Strace_update

Description

Seconds column now displays up to 7 digits for uniformity.
Added naclsysbrk and naclsysmprotect syscalls.
Excluded time & clock-related syscalls and unified fnctl operations. Merged send/recv stats into sendto/recvfrom .

@rennergade @RusherRG @yizhuoliang @yashaswi2000 can you review the pr?

Great, thank you!


Add a CLI Interface

Hi everyone,
I’m not a Rust developer, so I’m not sure how viable this idea is, but I wanted to get your thoughts.
There’s another project called b3sum that provides a CLI tool for computing checksums using BLAKE3. For example, you can install it like this:
$ cargo install b3sum

And then run it on a file:
b3sum file

The output looks like this:
af46dfde5af4db763f9d2d087b4ffdbdf5ad51f4dffb53056ef42241ff144683  file

I use this tool in my own project, picohash, by calling it through a shell interface in Go:
exec.Command("b3sum", path)

However, for my use case, BLAKE3 feels like overkill. I don’t need cryptographic security—I’m only checking for bit flips or simple file integrity issues.
Would it be possible to add a lightweight CLI interface to this project for simpler use cases like mine? I’d love to hear your thoughts or suggestions!

I have just deployed v1.2.0 which should allow you to cargo install rapidhash and use rapidhash on the commandline (accepting both stdin or a file path). It doesn't have a --help flag yet though, I'll add that in a future release.
echo "example" | rapidhash
8543579700415218186

rapidhash example.txt
8543579700415218186

Just tested on a 23G file on a local Macbook and rapidhash is IO bound, while sha512 is CPU bound. Happy hashing!
sha512      18.42s user 4.41s system 99% cpu 22.847 total
rapidhash   1.21s user 5.88s system 77% cpu 9.145 total

Yeah the CLI is a lazy implementation and reads the whole file because I simply copied the old CLI example, let me rework it to stream the file and add the --help flag. Brb!

Actually on second thought, the problem is that rapidhash doesn't have a streaming implementation. I'll have a think about whether I can come up with an alternative. You can read here for more info: https://github.com/Nicoshev/rapidhash/issues/3

@lucasbalena I've added a file-specific streaming implementation, currently on another branch. The pull request is: https://github.com/hoxxep/rapidhash/pull/10
Would you be willing to test the branch, to check the performance and memory usage matches what we are expecting?
You should be able to install the streaming branch via:
cargo install --git https://github.com/hoxxep/rapidhash --branch streaming rapidhash

Results I'm seeing on an M1 Max MacBook, with rapidhash using ~1MB ram, and b3sum using up to ~10MB. rapidhash is single-threaded in design and does significantly less CPU work, so I've included both single and multithreaded b3sum commands.
rapidhash                1.84s user  2.73s system   99% cpu   4.568 total
b3sum                   16.76s user  2.31s system  844% cpu   2.259 total
b3sum --num-threads 1   14.77s user  3.34s system   99% cpu  18.124 total


Fix Travis CI build

Changed log

Fix Travis CI build.
When using PHPUnit 8.x verson, it will output following message:

PHP Fatal error:  Declaration of BasicTest::setUp() must be compatible with PHPUnit\Framework\TestCase::setUp(): void in /data/agent/vendor/mobiledetect/mobiledetectlib/tests/BasicsTest.php on line 510

To resolve issue, removing PHPUnit 8.x version definition on composer.json.

Putty, but looks like we can't upgrade without mobile detect upgrading :)


feat: enable nats core subscribe for TS

Fixes #49
Blocked by https://github.com/the-codegen-project/cli/pull/53

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Ignores files without the gzip extension

Patched predict_gene_expression.py to ignore all files that do not end in ".gz".
Added a python script for converting genotype data in the plink binary PED format to the "dosage" format. This script requires plink2 to be installed for it to work.

thank you @sritchie73 for the pull request!

Was I correct that predict_gene_expression.py can't handle missing data in the dosage file? I'm testing it out on a cohort where we have both genotype and gene expression but I'm having to throw out 70% of the SNPs.

For best results you need to impute your genotype using impute2 or Mach.
The university of Michigan offers an imputation server that would make
imputation pretty easy.
We are also developing metaxcan that only needs summary statistics. But
again you would need summary stat in hapmap2 snps for best results.
Haky
On Tuesday, October 27, 2015, Scott Ritchie<EMAIL_ADDRESS>wrote:

Was I correct that predict_gene_expression.py can't handle missing data in
the dosage file? I'm testing it out on a cohort where we have both genotype
and gene expression but I'm having to throw out 70% of the SNPs.
—
Reply to this email directly or view it on GitHub
https://github.com/hakyimlab/PrediXcan/pull/11#issuecomment-151657704.

Yeap: I have genotype data imputed to 1000 genomes (phase 1 version 3), but there are a large number of SNPs that have missing data for some samples: (i.e. something like chr1 rd12345 123456 A C 2 0 1 0 NA 0 1).
These cannot be read by get_all_dosages() (on line 65) because the numpy array throws an exception: it reads the "NA" as a string and complains.
As a quick work around, I've just filtered the genotype file for SNPs with any missing values, but this removes 6.8 million of the 7.2 million SNPs passing QC. Is there a numeric coding for these missing values (e.g. plink uses -9)? Otherwise I will try setting the NA's to 0 (i.e. imputing to the major allele).

For now our models only use hapmap2 snps so i would check how many snps you
lose that way. 2*MaF would be better than 0 in any case.
On Tuesday, October 27, 2015, Scott Ritchie<EMAIL_ADDRESS>wrote:

Yeap: I have genotype data imputed to 1000 genomes (phase 1 version 3),
but there are a large number of SNPs that have missing data for some
samples: (i.e. something like chr1 rd12345 123456 A C 2 0 1 0 NA 0 1).
These cannot be read by get_all_dosages() (on line 65) because the numpy
array throws an exception: it reads the "NA" as a string and complains.
As a quick work around, I've just filtered the genotype file for SNPs with
any missing values, but this removes 6.8 million of the 7.2 million SNPs
passing QC. Is there a numeric coding for these missing values (e.g. plink
uses -9)? Otherwise I will try setting the NA's to 0 (i.e. imputing to the
major allele).
—
Reply to this email directly or view it on GitHub
https://github.com/hakyimlab/PrediXcan/pull/11#issuecomment-151686760.


Approximate Stellar Evolution

I would especially like to simulate suoernovae.
This doesn't make too much sense before #49.

When generating a star, create a Taylor series for the time evolution of luminosity and maybe other properties.

Extracted supernovae into #92.

We probably need a time evolution for star data, and one for star appearance. It is not feasible to recalculate star data in every timestep.
Alternatively we could define a shortest variation timestep and only recalculate if we are not within that step any more.


Appreciation post and feedback

Hi,
I've discovered your app a few days ago and I really enjoy using it. I've installed the agent on my VMs and also on my 3 machines (RPI, Asustor and my Server). It's great to see all those stats with such a low power cost.
That being said, I have a few suggestions (I'm sorry if those are similars to someone else's):

That would be cool to have a setting to specify if it's a VM or a physical machine (I've added * in front of every VMs name and ~ for every physical machine) I like them being splited. So maybe a custom CSS tool would do the trick?
Can the cached/buffer memory be visible with a lesser bright green on the dashboard?
It would be awesome to have a little icon button on the right, beside the alert bell, that says "manage" so we can paste the URL link to a portainer instance, proxmox VMs or NAS admin page.

Thanks for developing this tool!
Loving it :)

Thanks, glad it's working well for you!
Grouping or tagging for systems is something I've had on the list since the beginning, I just haven't been able to get around to it yet.
I like your other suggestions as well. I'll try to add them, but it may take some time.


Indentation of addSmallInts example

In this example: https://ziglearn.org/chapter-1/#comptime
fn addSmallInts(comptime T: type, a: T, b: T) T {
    return switch (@typeInfo(T)) {
        .ComptimeInt => a + b,
        .Int => |info| if (info.bits <= 16)
            a + b
        else
            @compileError("ints too large"),
        else => @compileError("only ints accepted"),
    };
}

I was wondering, doesn't an indentation like this makes more sense?
fn addSmallInts(comptime T: type, a: T, b: T) T {
    return switch (@typeInfo(T)) {
        .ComptimeInt => a + b,
        .Int => |info|
            if (info.bits <= 16)
                a + b
            else
                @compileError("ints too large"),
        else => @compileError("only ints accepted"),
    };
}

I see what you mean, but the former is more consistent and does not violate zig fmt.


When scanning an item with synchronization errors, the scanned text will flash.

For example, scanning the "ghost item" generated when the host uses a two handed mod.

Or scan the invisible flashlight generated by using the scroll wheel to switch the item slot at the last second of interaction when the player places the open flashlight into the item cabinet.

Can you show me a video of this?

I seem to have solved this problem, the issue maybe caused by simultaneously using  the GoodItemScan and "FixPersonalScanner" from GeneralImprovements.
Perhaps you can add a tip on the MOD introduction page？


sst start: importing files does not work for python

The following code works deployed via sst deploy but does not work locally via sst start.
from hello import world

def handler(event, context):
 return {
   "statusCode": 200,
   "body": world()
 }

def world():
  return "hi"

This is because sst runs the python script from the project's root folder and python will lookup the world file/module there. In order to import the file you could do something like import hello from path.to.module.world, but then this would break the deployment package.
Do you guys have plans to address this or any workaround? Right now we're manually setting local imports while developing and then cleaning the paths before deployment. Not great experience but fortunately we just use Python for some small ML / image processing tasks.
Thanks for this awesome framework by the way. We just finished migrating our whole codebase from Serverless Framework and have 0 regrets :)!

Hey @bros4president thanks for hint!
Which version of SST are you using? And Can you share the workaround you are putting in?


Feature: Filter gong and outreach subgraphs

Type of feature
📦 Chore
Current behavior
OneGraph has been updated to have new subgraphs for gong and outreach.  Need to update our filtering to account for these.
Suggested solution
Update regular expression as in prior updates (such as in #31)
Additional context
N/A
Code of Conduct

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

Contributing Docs

[X] I agree to follow this project's Contribution Docs

good catch!


Reconciler progress tracking: if a Kubernetes resource does not reach the READY status, dump latest status into logs

Description
If a deployed resource does not start (getting into READY state) or reaching a timeout, the reconciler should try to retrieve the latest status of this component (maybe the whole object retrieved from Kubernetes) and print it to the logs.
AC:

If the progress tracker is identifying a resource which doesn't become ready, the status of this particular component should be retrieved from Kubernetes and dumped to the logs to simplify later analysis.

Reasons
Simplify analysis of components which didn't reach the READY state.
Attachments

Closed as duplicate of #604


test: use tag in gapic-libraries-bom validation

The java-cloud-bom was released today. The tag is "v26.13.0". This includes the recent change in validate-bom action that checks the dependency tree. https://github.com/googleapis/java-cloud-bom/blob/v26.13.0/tests/validate-bom/action.yml

It passed.


Not able to run npm test command

Select one ...  (check one with "x")
[x ] bug
[ ] feature request
[ ] enhancement

I  developed a clarity-angular component - clr-list and I am trying to test it but I have problems running npm test command. It fails with messages which I could not debug. It will be great if you can suggest a solution.
Expected behavior
Be able to run npm test command with no errors
Actual behavior
running the npm test command fails with karma exit code 1
Reproduction of behavior
vasil@rabotlivko:~/server/clarity$ npm test
><EMAIL_ADDRESS>test /home/vasil/server/clarity
> gulp test

[23:33:24] Using gulpfile ~/server/clarity/gulpfile.js
[23:33:24] Starting 'test'...
[23:33:24] Starting 'build'...
[23:33:24] Starting 'clean'...
[23:33:24] Finished 'clean' after 571 ms
[23:33:24] Starting 'sass:static'...
[23:33:24] Starting 'sass:clarity'...
[23:33:24] Starting 'sass:app'...
[23:33:24] Starting 'sass:icons'...
[23:33:24] Starting 'typescript'...
[23:33:24] Starting 'tslint:clarity'...
[23:33:24] Starting 'tslint:app'...
[23:33:24] Starting 'tslint:icons'...
[23:33:24] Starting 'tslint:tests'...
[23:33:24] Starting 'html:index'...
[23:33:24] Starting 'html:app'...
[23:33:24] Starting 'html:assets'...
[23:33:24] Starting 'typescript:icons'...
[23:33:24] Starting 'typescript:clarity'...
[23:33:30] Finished 'sass:icons' after 5.41 s
[23:33:30] Finished 'sass:clarity' after 5.5 s
[23:33:30] Finished 'sass:static' after 5.53 s
[23:33:31] Finished 'html:index' after 6.21 s
[23:33:31] Finished 'html:assets' after 7.05 s
[23:33:36] Finished 'tslint:icons' after 12 s
[23:33:45] Finished 'typescript:icons' after 20 s
[23:33:45] Starting 'bundle:icons:sfx'...
[23:33:46] Starting 'bundle:icons-lite:sfx'...
[23:33:46] Finished 'bundle:icons-lite:sfx' after 2.98 ms
[23:33:46] Starting 'bundle:icons:shapes'...
[23:33:46] Finished 'bundle:icons:shapes' after 3.08 ms
[23:33:46] Starting 'svg:icons'...
Completed writing core-shapes svg files
Completed writing commerce-shapes svg files
Completed writing essential-shapes svg files
Completed writing media-shapes svg files
Completed writing social-shapes svg files
Completed writing technology-shapes svg files
Completed writing travel-shapes svg files
[23:33:48] Finished 'svg:icons' after 2.37 s
[23:33:48] Starting 'svg:icons:zip'...
[23:33:53] Finished 'bundle:icons:sfx' after 8.15 s
[23:34:00] Finished 'tslint:tests' after 36 s
[23:34:02] Finished 'sass:app' after 37 s
[23:34:02] Starting 'sass'...
[23:34:02] Finished 'sass' after 62 μs
[23:34:05] Finished 'tslint:clarity' after 41 s
[23:34:11] Finished 'tslint:app' after 47 s
[23:34:11] Starting 'tslint'...
[23:34:11] Finished 'tslint' after 45 μs
[23:34:11] Starting 'typescript:icons'...
[23:34:11] Starting 'typescript:app'...
[23:34:11] Starting 'typescript:tests'...
[23:34:11] Starting 'typescript:tests:icons'...
[23:34:12] Finished 'typescript:tests:icons' after 760 ms
[23:34:13] Finished 'html:app' after 48 s
[23:34:13] Starting 'html'...
[23:34:13] Finished 'html' after 35 μs
[23:34:13] Finished 'typescript:icons' after 1.43 s
[23:34:17] Finished 'typescript:tests' after 6.17 s
[23:34:21] Finished 'typescript:clarity' after 57 s
[23:34:21] Starting 'bundle:clarity:js'...
[23:34:30] Finished 'bundle:clarity:js' after 8.78 s
[23:34:30] Starting 'bundle:zip'...
[23:34:30] Finished 'bundle:zip' after 33 ms
[23:34:30] Starting 'bundle'...
[23:34:30] Finished 'bundle' after 345 μs
[23:34:30] Finished 'svg:icons:zip' after 42 s
[23:34:30] Starting 'svg'...
[23:34:30] Finished 'svg' after 44 μs
[23:34:30] Finished 'typescript:app' after 19 s
[23:34:30] Finished 'typescript' after 1.08 min
[23:34:30] Finished 'build' after 1.1 min
[23:34:30] Starting 'karma:verbose'...

START:
27 07 2017 23:34:31.300:WARN [watcher]: Pattern "/home/vasil/server/clarity/dist/**/*.js.map" does not match any file.
27 07 2017 23:34:31.800:INFO [karma]: Karma v1.7.0 server started at http://<IP_ADDRESS>:9876/
27 07 2017 23:34:31.800:INFO [launcher]: Launching browser Chrome_Headless with unlimited concurrency
27 07 2017 23:34:31.809:INFO [launcher]: Starting browser Chrome
27 07 2017 23:34:32.264:INFO [HeadlessChrome 0.0.0 (Linux 0.0.0)]: Connected on socket -yMyPMBmFdRJujQaAAAA with id 41303491
27 07 2017 23:34:33.277:WARN [web-server]: 404: /base/selenium-webdriver/testing
HeadlessChrome 0.0.0 (Linux 0.0.0) ERROR
  {
    "originalErr": {
      "__zone_symbol__currentTask": {
        "type": "microTask",
        "state": "notScheduled",
        "source": "Promise.then",
        "zone": "<root>",
        "cancelFn": null,
        "runCount": 0
      }
    },
    "__zone_symbol__currentTask": {
      "type": "microTask",
      "state": "notScheduled",
      "source": "Promise.then",
      "zone": "<root>",
      "cancelFn": null,
      "runCount": 0
    }
  }

Finished in 2.341 secs / 0 secs @ 23:34:34 GMT+0300 (EEST)

SUMMARY:
✔ 0 tests completed
[23:34:34] 'karma:verbose' errored after 3.89 s
[23:34:34] Error: There are failing unit tests
    at /home/vasil/server/clarity/build/tasks/karma.js:32:27
    at removeAllListeners (/home/vasil/server/clarity/node_modules/karma/lib/server.js:380:7)
    at Server.<anonymous> (/home/vasil/server/clarity/node_modules/karma/lib/server.js:391:9)
    at Object.onceWrapper (events.js:293:19)
    at emitNone (events.js:91:20)
    at Server.emit (events.js:188:7)
    at emitCloseNT (net.js:1577:8)
    at _combinedTickCallback (internal/process/next_tick.js:77:11)
    at process._tickCallback (internal/process/next_tick.js:104:9)
[23:34:34] 'test' errored after 1.17 min
[23:34:34] Error in plugin 'run-sequence(karma:verbose)'
Message:
    There are failing unit tests
Stack:
Error: There are failing unit tests
    at /home/vasil/server/clarity/build/tasks/karma.js:32:27
    at removeAllListeners (/home/vasil/server/clarity/node_modules/karma/lib/server.js:380:7)
    at Server.<anonymous> (/home/vasil/server/clarity/node_modules/karma/lib/server.js:391:9)
    at Object.onceWrapper (events.js:293:19)
    at emitNone (events.js:91:20)
    at Server.emit (events.js:188:7)
    at emitCloseNT (net.js:1577:8)
    at _combinedTickCallback (internal/process/next_tick.js:77:11)
    at process._tickCallback (internal/process/next_tick.js:104:9)

Environment details

Angular version: 4.1.3

Clarity version:
0.10.0-alpha.5

OS and version:
Ubuntu 16.04

g    _                      _                 ____ _     ___
   / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
  / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
 / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
/_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
               |___/

@angular/cli: 1.2.0
node: 7.10.0
os: linux x64
@angular/animations: 4.1.3
@angular/common: 4.1.3
@angular/compiler: 4.1.3
@angular/compiler-cli: 4.1.3
@angular/core: 4.1.3
@angular/forms: 4.1.3
@angular/platform-browser: 4.1.3
@angular/platform-browser-dynamic: 4.1.3
@angular/platform-server: 4.1.3
@angular/router: 4.1.3
@angular/upgrade: 4.1.3

Browser: Chrome Version 59.0.3071.115 (Official Build) (64-bit)

@bashibozuk
Please make sure you have Chrome Canary installed on your machine. The tests use HeadlessChrome (included with Chrome Canary) to run the tests.

You cant have Chrome Canary on Linux - https://askubuntu.com/questions/309813/how-to-have-ubuntu-chrome-canary
And in my opinion a developer should be able to run clarity tests on Linux also on Windows.
ng test command runs on any OS for example. Could you please explain why the reason I am getting error with no message is that I am using Headless Chrome instead of Chrome Canary ?

@bashibozuk
Headless Chrome is part of Chrome Canary at this time.
As for Linux and Windows, @jeeyun might be able to advise there...

Hi @bashibozuk,
Can you change the following line in your local karma.conf.js:
  browsers: ["Chrome_Canary_Headless"],
to:
  browsers: ["Chrome_Headless"],
And see if you are able to run the test locally on Linux?
I agree with you that our scripts should allow running tests locally on any OS. At the time I wrote the script, Chrome only supported headless in Canary in some OSs while it was already supported in Linux. I made a team-centric choice in the script since all of our team members were able to use Chrome Canary to run tests locally. I just did a quick check, and it seems that the latest versions of Chrome support headless in Linux/Windows/Mac, so I will update our script accordingly.


Updates rubocop to 0.80.

Coverage remained the same at 99.451% when pulling 46cbb98a936a9d41557560b77e6c05d8bbc961ea on LeFnord:master into fada2560b6cc329a7dc7792d2f84ae47cfd2a8ed on ruby-grape:master.

Coverage remained the same at 99.451% when pulling 46cbb98a936a9d41557560b77e6c05d8bbc961ea on LeFnord:master into fada2560b6cc329a7dc7792d2f84ae47cfd2a8ed on ruby-grape:master.

Coverage remained the same at 99.451% when pulling 46cbb98a936a9d41557560b77e6c05d8bbc961ea on LeFnord:master into fada2560b6cc329a7dc7792d2f84ae47cfd2a8ed on ruby-grape:master.

Coverage remained the same at 99.451% when pulling 46cbb98a936a9d41557560b77e6c05d8bbc961ea on LeFnord:master into fada2560b6cc329a7dc7792d2f84ae47cfd2a8ed on ruby-grape:master.

Coverage remained the same at 99.451% when pulling 46cbb98a936a9d41557560b77e6c05d8bbc961ea on LeFnord:master into fada2560b6cc329a7dc7792d2f84ae47cfd2a8ed on ruby-grape:master.


Remove unused dependencies pyasn1 and cryptography (python-jose)

Closes #8

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Removing these two packages as I believe they were added to the first versions of the recipe via python-jose, and were not removed after we updated our code/deps.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

For next release I think. Assuming build passes and it validates the cylc --version command is working, and perhaps also some local testing to confirm it can to handle security without these two packages :thinking:

(draft to avoid accidental merge)

Superseded by #26


What does A32/A64 mean?

When using VPD for semantic segmentation, does A32/A64 mean that only use size 32/64 attention maps? In the source code, attn16, attn32, attn64 are all used, how can i reproduce the results of VPD_A32 and VPD_A64?

Thanks for your interest in our work! A32 means attn16+attn32 are used and attn64 means attn16+attn32+attn64 are used. A64 is used by default and you can set max_attn_size=32 to the UNetWrapper to enable A32.

Thanks for your reply! I'll try it.


SASS integration

[x] Pull dependencies from from https://github.com/sachioross/helix-project-boilerplate-sass
[x] Test SASS conversion on one of the blocks
[x] Update README.md.

@ahmed-musallam Please do have a look and let me know what you think.