[homework] 20201030

Hi @jkim237 :)
이번주 숙제입니다: 다음주 (월) 20201030 일까지 해주시고, 같이 풀어나갑시다!

항상!  :)
코딩 후 PR을 작성하여 제출해주세요!

Language: Javascript

공식문서 <= now, this is your friend. Say hi!

1. History

Check this video
Answer these questions:

A) What is a Programming Language? (check this article)
B) What is Javascript? (One sentence is enough!)
C) What are the pro and cons of Javascript? (Just list a few ;))

2. Syntax:
Follow the content of this site

Variables
Variables types (data types): primitives

Learn Javascript online

Choose one or another site you want and keep learning. A solid base is very important!

Codecademy
FreeCodeCamp

A1: What is a Programming Language?
Programming Language is a translator between the binary codes(Computer's language) and the human language. There are 2 types of programming languages. High-level languages and low-level languages. High-level languages are more like human languages. So, they're easier to understand for human beings. But sometimes, they're slower than the low-level languages.
A2: What is Javascript?
The programming language that can make the website to interact with the users.
A3: What are the pro and cons of Javascript?
Pros:

Easy to Understand and use
It's fast
Versatility
Server Load

Pros:

Client-side Security
Browser Support
Single Inheritance


Use CBOR for state serialization

Use CBOR for state serialization

Fixed by https://github.com/Nitrokey/oath-authenticator/pull/26 / https://github.com/Nitrokey/oath-authenticator/commit/7af1d8b908dae5a9ab12502f8970014801bdc37b


Add setup instruction to the webiste README.md file

Add an instruction to the website environment setup.
Changes a wording of a step.

@manthey I just realize we could add a simple instruction. And, the word setup feels better to me, but I am open to any suggestions.


Raise your hand

The hand rises only for a couple of seconds, it works very strangely

The code is implemented properly.
https://github.com/grishka/Houseclub/blob/63d04453e853f3ff5c8df93ae9ee762a4053f40b/Houseclub/src/main/java/me/grishka/houseclub/VoiceService.java#L257-L263
The hand raise notification popup is only shown for several seconds, but the hand is still raised when you check from the actual client.


Unable to read the Product Label and Serial Number (CON-1400)

I encountered a strange problem where I was able to read properties such as VendorId and ProductName, but not ProductLabel and Serial Number. However, my chip tool was able to read them, and I don't know where the problem lies
Last week it was still readable, but strangely now it suddenly becomes unreadable
chip-tool:

code:
if(dataProvider.GetProductName(device_info.productName, sizeof(device_info.productName)) == CHIP_NO_ERROR)
{
   USER_LOGI(TAG,"device_info.productName :%s",device_info.productName);
}
if(dataProvider.GetHardwareVersionString(device_info.HardwareVersionString, sizeof(device_info.HardwareVersionString)) == CHIP_NO_ERROR)
{
   USER_LOGI(TAG,"device_info.HardwareVersionString :%s",device_info.HardwareVersionString);
}
if(dataProvider.GetVendorId(device_info.VendorId) == CHIP_NO_ERROR)
{
   USER_LOGI(TAG,"VendorId :%d",device_info.VendorId);
}
if(dataProvider.GetProductId(device_info.ProductID) == CHIP_NO_ERROR)
{
   USER_LOGI(TAG,"ProductID :%d",device_info.ProductID);
}
if(dataProvider.GetSetupDiscriminator(device_info.Discriminator) == CHIP_NO_ERROR)
{
   USER_LOGI(TAG,"device_info.Discriminator :%d",device_info.Discriminator);
}
char ProductLabel[64] ={0};
if(dataProvider.GetProductLabel(ProductLabel,strlen(ProductLabel)) == CHIP_NO_ERROR)
{
   USER_LOGI(TAG,"ProductLabel :%s",ProductLabel);
}
char ReadSerialNumber[32];
if(dataProvider.GetSerialNumber(ReadSerialNumber,strlen(ReadSerialNumber)) == CHIP_NO_ERROR)
{
   USER_LOGI(TAG,"ReadSerialNumber :%s",ReadSerialNumber);
}

log

sdkconfig
sdkconfig.txt

@MaplestoryAlen ,
char ProductLabel[64] ={0};
if(dataProvider.GetProductLabel(ProductLabel,strlen(ProductLabel)) == CHIP_NO_ERROR)
{
   USER_LOGI(TAG,"ProductLabel :%s",ProductLabel);
}
char ReadSerialNumber[32];
if(dataProvider.GetSerialNumber(ReadSerialNumber,strlen(ReadSerialNumber)) == CHIP_NO_ERROR)
{
   USER_LOGI(TAG,"ReadSerialNumber :%s",ReadSerialNumber);
}

I guess the above code snippet should have device_info.ProductLabel like device_info.Discriminator if you are setting them and then using here to retrieve them in the code.

@MaplestoryAlen , any updates on the issue?

，此问题是否有任何更新？

I have given up using Matter's API to read, I will use IDF's NVS to read


Thread – New message button and green line appears when go back to main chat from thread

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.40-1
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5006354
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Go to https://staging.new.expensify.com/
Log in
Open any chat and send 10 messages
Right click on the firs message and select Reply in thread
Send a comment in a thread
Go back to main chat via header link

Expected Result:
New message button not appears
Actual Result:
New message button and green line appears
Workaround:
Unknown
Platforms:

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/a03f9516-afd5-4e6d-8520-8bf79febe16d

View all open jobs on GitHub

This looks like the most likely culprit: https://github.com/Expensify/App/pull/49367/files

@marcaaron I doubt that's the culprit because this issue has  been reported  previously in #49525 a week before #49367 was merged

cool! I based my guess entirely on the name of the PR.

The unread marker appearing on coming back from thread is fixed with this PR #49770. That can be CPed to staging.

@dangrous says this one is fixed via the CP!


Feature request : Sort My List (or any other list) by label

I'm submitting a ...

[ ] bug report
[X] feature request
[ ] support request

General information
With a large list, it's getting sometime hard to find what you want to see.  So I wish I could sort My List by label, and sort it again 'by default' if I need it.
For now, I was able to implement the label sorting by adding xbmcplugin.addSortMethod(g.PLUGIN_HANDLE, xbmcplugin.SORT_METHOD_LABEL) in the Kodi/Listings.py file in the 'sort_nothing' case.
But even if I replace SORT_METHOD_NONE by SORT_METHOD_UNSORTED, I can't sort it again 'by default' until I exit the view and get in again. This is the expected behavior for sorting (https://forum.kodi.tv/showthread.php?tid=174921), but if you have an idea to make it work, that would be awesome.
Thanks!
Smeulf.

I don't understand why you need to order the page result,
is just 100 results per page,
I'll never put sorting on the lists
for these reasons:
-the paged lists are a workaround because kodi does not support them
-the sort method mix the next/previous "buttons"
-ordering the results paged with the kodi sort method, involves a mixture per page, which does not respect the queries returned by the service
so the sort order can be applied only via query through the appropriate option in the settings->views

Well, even 100 results per page, it's sometime confusing for me if it's not sorted.
About the next/previous page, if I remember correctly, there's a way to put then at the beginning or the end of a list.  The 'Extra' addon uses it as far as I remember.
But it's ok for me if you don't want to implement it ;)
Just now, I'm trying to determine what view I should use to have My List ordered by title. Can you give me some tips ?
Thanks.


perf: replace Beam effect with Ring effect for Doomfist indicator gui - [merged]

Merges doom-indicator -> staging

mentioned in commit 132049d3597437f0b28e31baaa19c42f8c69641d

In GitLab by @ghost1 on Oct 9, 2023, 15:15
:tada: This MR is included in version 1.0.1 :tada:
The release is available on GitLab release.
Your semantic-release bot :package: :rocket:


Building on flow

this PR implements a new docs structure for building on flow section according to notion

There's no description or issue linked. Can you explain what one should be looking for here?

Will this go live with the missing content if it's merged?

yes, it will, that's why it's a draft
I think you can branch off this PR and when everyone finishes working on this section, we can merge it
alternatively I can remove top bar menu item and merge it
this section will be kinda hidden without


Missing core patch for "bold" and "italic" commands

Looks like we need core patches to properly support <strong> and <em> tags - the strong and italic commands do not work as expected.
In the latest version of Chrome, the queryState() method for these two commands appear to depend on CSS artifacts, e.g. whether the content happens to inherit the CSS rules font-weight: bold or font-style: italic.
You can verify this in the demo, by simply adding one of these CSS attributes to the editor <div>, and you will see the "bold" and "italic" buttons on the toolbar appearing as active.
I don't know why it behaves this way, but it's clearly wrong - whether your CSS happens to apply certain presentation rules has nothing to do with the <strong> and <em> tags, in terms of semantics.
My concrete case is a <blockquote> tag, which has font-style: italic applied to it by a CSS rule - this shouldn't mean we don't get to use <em> inside it, and it definitely should not reflect on toolbar state.
The <em> tag strictly means emphasis, it doesn't mean italics - HTML5 separates the semantic meaning of emphasis from the presentation artifact of an italic type-face.
With that said, Scribe uses the standard behavior of <b> and <i> tags by default, unless patched with something like scribe-plugin-formatter-html-ensure-semantic-elements - so I guess it's debatable whether this belongs in the core, or in that plugin?
I will patch this up for my own purposes tomorrow - if you can let me know where you think this feature belongs, I can submit a PR, if you'd like?

Hmm, attempted a quick patch by adding this to patches/commands/bold
      boldCommand.queryState = function () {
        var selection = new scribe.api.Selection();
        var boldNode = selection.getContaining(function (node) {
          return node.nodeName === 'B' || node.nodeName === 'STRONG';
        });

        return scribe.api.CommandPatch.prototype.queryState.apply(this, arguments) && !! boldNode;
      };

That does not appear to be enough. The default behavior of executing the "bold" command results in the creation of a <span style="font-weight: normal;"> node - yikes.
So looks like we'll need a custom implementation of execute() for the "bold" and "italic" commands.
Alternatively, maybe we should consider adding new, entirely custom commands named "strong" and "em"? (Though I doubt anyone would actually want the "bold" and "italic" commands as-are, for any reason? I can't really imagine anyone is using scribe without the ensure-semantic-elements plugin?)
I will look into this some more tomorrow.


custody-server: SDFSOF-22 / SDFSOF-23: Process inbound/outbound payments (Withdrawal/Deposit) via FB

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
Implement webhook handlers for SEP-24/SEP-31 inbound/outbound payments
Why
So we can update SEP and custody transaction with payment info(status, date, Stellar transaction etc)
Known limitations
[TODO or N/A]

Reference Server Preview is available here:https://anchor-ref-pr879.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr879.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr879.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr879.previews.kube001.services.stellar-ops.com/

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Reference Server Preview is available here:https://anchor-ref-pr879.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr879.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr879.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr879.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr879.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr879.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr879.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr879.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr879.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr879.previews.kube001.services.stellar-ops.com/

Something went wrong with PR preview build please check


Add a lifecycle.Managed for AutoClosable's instances

I used ExecutorServiceManager as a guide.

Changes Unknown when pulling 2c5839b981971b1dc3a02db58e46743b1e4f1265 on frojasg:add-auto-closable-manager into ** on dropwizard:master**.

Thanks for the contribution, @frojasg. This seems useful.
Could you add a JavaDoc to AutoCloseableManager providing some context, please?

@evnm Thanks for the feedback.
I added some documentation. Please let me know if I need to improve it 😄

Coverage increased (+0.01%) to 84.404% when pulling 092d3ee623e92b2f54c3b359de5256662b7e161d on frojasg:add-auto-closable-manager into 2320acdedcfba67e83cd9f6b93bd84a616f62bf0 on dropwizard:master.

Coverage increased (+0.01%) to 84.404% when pulling bae197d5ac2d881ace0336b95834f9c466f30621 on frojasg:add-auto-closable-manager into 2320acdedcfba67e83cd9f6b93bd84a616f62bf0 on dropwizard:master.

Thank @evnm!

Thanks you for contributing!


Support custom request methods?

Hi,
Thanks for creating this great tool/vscode extension!
Is there a way to create a custom request method?
For example: SEARCH
The API that I use supports a SEARCH method (with a JSON formatted query as body).
Couldn't find anything in the docs.
Thanks!

This feature will be launched soon.

This feature is live! Use the image below for setup. Thanks for your feedback!


Logs for Container App missing

Please provide us with the following information:

This issue is a: (mark with an x)

[x] bug report -> please search issues before submitting
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
One of our Container Apps doesn't show most of the log entries in Azure Monitor (ContainerAppConsoleLogs_CL). However the logs are shown perfectly fine in the "Log stream". It's also working fine for all of our other container apps. What makes this even more strange is, that we have the same container app configuration in two resource groups (staging and production) and it's only broken in one of them. They use the same terraform code to deploy and the exact same image. Deleting the Container App and redeploying also didn't fix the issue.
A few of the logs were even logged, but they were missing the first part and therefor invalid json.

stderr seems to work fine. also some other logs. for the non working logs Stream_s is empty and not showing stderr or stdout
Steps to reproduce
Unsure how to reproduce, because it's only affecting one instance.
Expected behavior [What you expected to happen.]
Logs that are shown in "Log stream" should also reliably appear in ContainerAppConsoleLogs_CL.
Actual behavior [What actually happened.]
One of the container apps doesn't show the logs reliably in ContainerAppConsoleLogs_CL.

having same issue

Any news? We're having the same issue. For us it's two of our apps.
It's actually happening to the first two apps we deployed. It's possible that Logging to Log Analytics wasn't set up back then when they were deployed and updated afterwards. Could that be the reason?

Could you please share your environment resource id at acasupport[at]microsoft.com ?

Hi @cstickel sorry for the delay. Do you still see this problem? If yes can you please provide the information of the containerapps(both working one and non-working one): containerapp name, region, subscriptionId?

Hi @xwang971
I will take over for @cstickel . It now affects two services at our end as well. How should I transmit the requested data?

@hmg-hholland Hi, can you please send the information to<EMAIL_ADDRESS>please?

We are seeing a similar issue in our deployment, at least one container app has log entries that are visible in the log stream - but they fail to end up in log analytics.
There seems to be a correlation between messages that do and do not end up in log analytics, unfortunately the error messages are omitted 😬

It seems that the custom column limit in the Log Analytics workspace table ContainerAppConsoleLogs_CL is reached. We didn't figure out how to fix the issue though.

@cstickel Hi, really sorry for the delay. We were adding metrics to the environment and found there were no logs dropped there and yes you are right the reason why the logs are missing is because of the limit of the number of table columns since we are now parsing the output and allocate different columns for each key. Could you try sending everything to azure monitoring instead of log analytics for mitigation please? We'll work on the fix for the limit issue and would update here once there's update.

@xwang971 as said in our staging environment there are no logs dropped anymore, since we recreated the log analytics workspace table ContainerAppConsoleLogs_CL which is broken since then.

so since a week the edit schema link is disabled, and it did not inherit the retention and plan from the workspace

also it logs each log entry, but most columns are empty.
it's not even showing data in columns like ContainerName_s which should be filled always for container apps:

you can see in the operation table, that before we recreated the table there were actually logs dropped:

but actually on the other (production) resource group, we still get logs dropped, because we manually deleted columns in the edit schema which should not be used. but instead of getting deleted, they were converted from Custom Columns to Azure Columns which are not deletable.

this is even more concerning for us, because that's our production environment.
however, i can turn azure monitor logging on, on our staging environment, if that helps to figure out the problem, why there is data in most columns missing. didn't know that this option even exists... is this new? as far as i remember last year log analytics workspace was required for container app environments.
if i change it to azure monitor, do you have any requirements to which sources we send the data, or is it fine to just switch to Azure monitor in Logging options and then select again analytics workspace as target in diagnostic settings?

Hi, @cstickel, the logs from our end look good with all the required information like containerappName, environmentName, could you check with log-analytics team please?
To configure azure monitoring, yes you can select again analytics workspace as target in diagnostic settings. The only difference here is that azure monitoring as a middleware would format the logs in the certain way so that no new columns would be added. And it would forward logs to log-analytics table ContainerAppConsoleLogs(without _CL)
Example:

The log contains the whole line of log even it's in json format without parsing into new columns. This would mitigate the issue.
Again we are sorry for the inconvenience and working on fixing the issue directly sending logs to log-analytics. We would update here where there's progress.

@xwang971 thanks for the info, we weren't aware of the new logging configuration. it's pretty new, isn't it?
changed our staging environment and that works perfectly fine so far. we need some more time to test it and also deploy it to our production environments and then we'll give feedback.
so far the new solution with logs configuration through diagnostic settings looks conceptionally much better than the old one, and more aligned with the other azure products.
we'll give feedback if the issue is finally fixed for us until end of the week. thanks for all the support!

@cstickel Thanks for the feedbacks! Please let us know if there's any update/issue.

@xwang971 is there any specific size limit for logs? first impression is, that it works a lot better, but for quite large json objects (3k+ byte) the logs are missing. it's not an issue, we can just reduce the amount of information, but would be nice to know if there is any limit we should know about.

with the change to azure monitor all problems seem to be fixed. couldn't find any situation with missing logs anymore.
so from my point of view, the issue can be closed.
@ekjuanrejon
@WolfspiritM
@Torgeir-Hansen
do you guys still need this issue or can it be closed?
summary:

checking "operations" table if you're at the column limit (then it's the same issue we had)
changing to azure monitor in the azure container app environment should fix it (actually even without the issue this is a good idea imho)

sure

@xwang971  that's weird, because logs bigger than ~3kb didn't end up in the azure monitoring for us... for smaller logs it's working perfectly fine.
not an issue for us, but you might just want to double check if the 32kb value is correct.

@xwang971 seems that it's not the size, but maximum nesting, that causes logs to be dropped (and only the single entry that has the very deep nesting).
totally unrelated to this issue, so would close this one, because with monitoring logs seem to work fine.

this is a problem still

Issue persist. One time it works, one time not. The same query, nothing special


[Issue]: In version 1.1.11 the "nvm use ver_no" command is not working.

What happened?
In version 1.1.11 the "nvm use ver_no" command is not working. Always staying the same version. After I install 1.1.10 the nvm use command worked.
What did you expect to happen?
Set node version to an specific version.
Version
1.1.11 or newer (Default)
Which version of Windows?
Windows 10 (Default)
Which locale?
None
Which shell are you running NVM4W in?
No response
User Permissions?
Administrative Privileges, Elevated
Is Developer Mode enabled?
Yes
Relevant log/console output
No response
Debug Output
No response
Anything else?
No response

This sounds like you have a prior install blocking NVM4W. What is the output of nvm debug? It will tell you if this is the problem.

@coreybutler there is an issue with spaces in your windows user:
C:\Users\Guilherme Mussi\git\sudokujs> nvm debug
Guilherme Mussi is not using admin or elevated rights.

C:\WINDOWS\System32\WindowsPowerShell\v1.0\powershell.exe
Windows Developer Mode: UNKNOWN (user cannot read registry)

NVM4W Version:      1.1.11
NVM4W Path:         C:\Users\Guilherme Mussi\AppData\Roaming\nvm\nvm.exe
NVM4W Settings:     C:\Users\Guilherme Mussi\AppData\Roaming\nvm\settings.txt
NVM_HOME:           C:\Users\Guilherme Mussi\AppData\Roaming\nvm
NVM_SYMLINK:        C:\Program Files\nodejs
Node Installations: C:\Users\Guilherme Mussi\AppData\Roaming\nvm

Active Node.js Version: v16.13.2

No problems detected.

All is good, but some commands fail with:
S C:\Users\Guilherme Mussi\git\sudokujs> nvm use newest
exit status 1: 'C:\Users\Guilherme' is not recognized as an internal or external command,
operable program or batch file.

exit status 1: 'C:\Users\Guilherme' is not recognized as an internal or external command,
operable program or batch file.

@gmussi which version of Windows are you using (i.e. a non-English version)? The most common issue with "spaces" is that some versions of Windows aren't saving the settings.txt file in UTF-8 format. As a result, the spaces aren't escaped properly.

@coreybutler Windows 11, English.
The issue, my windows username has a space in it: C:\Users\Guilherme Mussi\
This is breaking some of the commands:
exit status 1: 'C:\Users\Guilherme' is not recognized as an internal or external command,

The nvm tool should account for this.

@gmussi - yes, I understand the problem. Spaces were resolved several versions ago, but encodings can mess with the PATH.
Usually this occurs in non-English versions. Since you're using an English version, can you tell me which? Despite being based on English, they still behave differently by locale.

United States (American English)
United Kingdom (British English)
Canada (Canadian English)
Australia (Australian English)
New Zealand (New Zealand English)
India (Indian English)
South Africa (South African English)
Ireland (Irish English)

I'm experiencing the same issue.
Currently using:

Windows Edition Windows 11 Pro N
Version 22H2
Build 22621.2361 (fully updated)
Windows Display Language and Regional Format: English (Canada)

C:\Users\Tyler Brownell\Code\spt-server>nvm use latest
20.8.0
exit status 1: 'C:\Users\Tyler' is not recognized as an internal or external command,
operable program or batch file.

C:\Users\Tyler Brownell\Code\spt-server>nvm debug
Tyler Brownell is not using admin or elevated rights.

Command Prompt - nvm  debug
Windows Developer Mode: UNKNOWN (user cannot read registry)

NVM4W Version:      1.1.11
NVM4W Path:         C:\Users\Tyler Brownell\AppData\Roaming\nvm\nvm.exe
NVM4W Settings:     C:\Users\Tyler Brownell\AppData\Roaming\nvm\settings.txt
NVM_HOME:           C:\Users\Tyler Brownell\AppData\Roaming\nvm
NVM_SYMLINK:        C:\Program Files\nodejs
Node Installations: C:\Users\Tyler Brownell\AppData\Roaming\nvm

Active Node.js Version: none
(run "nvm use <version>" to activate a version)

No problems detected.

The command does work when running with administrator privileges:
C:\Users\Tyler Brownell\Code\spt-server>nvm install latest
20.8.0
Version 20.8.0 is already installed.

C:\Users\Tyler Brownell\Code\spt-server>nvm use latest
20.8.0
Now using node v20.8.0 (64-bit)

C:\Users\Tyler Brownell\Code\spt-server>nvm debug
Running NVM for Windows with administrator privileges.

Administrator: Command Prompt - nvm  debug
NVM4W Version:      1.1.11
NVM4W Path:         C:\Users\Tyler Brownell\AppData\Roaming\nvm\nvm.exe
NVM4W Settings:     C:\Users\Tyler Brownell\AppData\Roaming\nvm\settings.txt
NVM_HOME:           C:\Users\Tyler Brownell\AppData\Roaming\nvm
NVM_SYMLINK:        C:\Program Files\nodejs
Node Installations: C:\Users\Tyler Brownell\AppData\Roaming\nvm

Active Node.js Version: v20.8.0

No problems detected.

@refringe interesting, and thank you for posting.
It's known that Windows requires users to have symlink permissions to run nvm use, so the problem here is the error message (or lack thereof) is misleading. This is not a space issue, rather a permission issue.

@coreybutler I can take this issue and add a more descriptive error message

Sure.  I'll want to review it,  but submit a PR and I'll look at it.

The command indeed works fine with administrative account, however when I have space in my user name and use Polish (non ascii) user name it does not work in version 1.11.1 for non admin user.
Shouldn't that work?
I saved my settings.txt file in UTF-8 to allow non ascii characters in my user dir name


Handle dynamic list of identity providers

This will take care of also hiding buttons from the custom layout when adding them for all possible identity providers, but the actual list is dynamic so some won't be available.
Issue: https://github.com/firebase/FirebaseUI-Android/issues/1775

@googlebot I signed it!

@googlebot I fixed it.

@robapaul88 thanks for this contribution!  BTW I changed the target branch to version-6.3.0-dev since that's where we're working at the moment.

Created https://github.com/firebase/FirebaseUI-Android/pull/1800 to have this pushed to the right branch, redirecting this from master seems to also affect some other files (probably reverting some other commits from 6.3.0-dev).


Object-aware autocomplete doesn't work in interactive windows

Environment data

VS Code version: 2.82.2
Jupyter Extension version (available under the Extensions sidebar): v2023.7.1002162226
Python Extension version (available under the Extensions sidebar): v2023.14.0
OS (Windows | Mac | Linux distro) and version: Local: Windows 10 / Remote: Scientific linux kernel 3.10.0-1160.11.1.el7.x86_64
Python and/or Anaconda version: Python 3.10.12
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Jupyter server running: "Python environment" kernel started by vscode, running via remote ssh extension

Expected behaviour
In an interactive Python window with active jupyter kernel, I'd expect the autocomplete to be aware of the objects in memory, for instance to suggest dictionary keys. This is how this works for the notebook interface:

Actual behaviour
However, in an interactive window (for instance, associated with a python "percent script"), this doesn't work

Steps to reproduce:

Create a dictionary

mydict = {"aaaa": 42, "bbbb": 35, "cccc": 22}

Type mydict[" and wait for autocomplete

Logs

Output for Jupyter in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Jupyter)

10:52:46.519 [info] Restart requested /Interactive-2.interactive
10:52:46.621 [error] Failed to get activated conda env vars for ~/apps/jupyter/bin/python
                 in 100ms
10:52:46.622 [error] Unable to determine site packages path for python ~/apps/jupyter/bin/python (Conda)
10:52:46.658 [info] Process Execution: ~/apps/jupyter/bin/python -c "import ipykernel; print(ipykernel.__version__); print("5dc3a68c-e34e-4080-9c3e-2a532b2ccb4d"); print(ipykernel.__file__)"
10:52:46.699 [info] Process Execution: ~/apps/jupyter/bin/python -m ipykernel_launcher --ip=<IP_ADDRESS> --stdin=9036 --control=9034 --hb=9033 --Session.signature_scheme="hmac-sha256" --Session.key=b"bbdde280-619d-436a-a3d8-7d631a78fcae" --shell=9035 --transport="tcp" --iopub=9037 --f=~/.local/share/jupyter/runtime/kernel-v2-48298fdHvH8cQDnYV.json
    > cwd: ~/transient/quarto-test
10:52:47.316 [info] ipykernel version & path 6.24.0, ~/apps/jupyter/lib/python3.10/site-packages/ipykernel/__init__.py for ~/apps/jupyter/bin/python
10:52:59.247 [warn] StdErr from Kernel Process ~/apps/jupyter/lib/python3.10/site-packages/traitlets/traitlets.py:2548: FutureWarning: Supporting extra quotes around strings is deprecated in traitlets 5.0. You can use 'hmac-sha256' instead of '"hmac-sha256"' if you require traitlets >=5.
  warn(
10:52:59.247 [warn] StdErr from Kernel Process ~/apps/jupyter/lib/python3.10/site-packages/traitlets/traitlets.py:2499: FutureWarning: Supporting extra quotes around Bytes is deprecated in traitlets 5.0. Use 'bbdde280-619d-436a-a3d8-7d631a78fcae' instead of 'b"bbdde280-619d-436a-a3d8-7d631a78fcae"'.
  warn(
10:52:59.374 [info] Started Kernel jupyter (Python 3.10.12) (pid: 110237)
10:52:59.374 [info] Started new session 47062b55-4b7a-4265-afcc-189a923d64e6
10:52:59.376 [info] Shutdown old session 297192d9-cd1c-44ba-9201-7cfa8ad70b06
10:52:59.377 [info] Dispose Kernel process 106726.
10:53:13.339 [info] Generated code for 1 = <ipython-input-1-3f004a30bb3a> with 2 lines
10:53:13.366 [info] Kernel acknowledged execution of cell 2 @<PHONE_NUMBER>362
10:53:13.367 [info] End cell 2 execution @<PHONE_NUMBER>365, started @<PHONE_NUMBER>362, elapsed time = 0.003s

Is the actual behavior, in the input box, is mydict shown in the completion suggestions but then flagged as unknown. If so, that's the same symptom as https://github.com/microsoft/vscode-jupyter/issues/13018#issuecomment-1458873666, so maybe whatever is broken is in the same area as that issue was.


Build fails on Solaris variants (like SmartOS) due to missing sig_t

The libmagic source uses sig_t in compress.c where it bombs when building on SmartOS.
Please see https://github.com/FreeRADIUS/freeradius-server/issues/765.

IMHO this is something that should be fixed upstream in libmagic. I'm hesitant to start floating patches, let alone for a platform I'm not familiar with and would have a hard time maintaining external support for.

5.23 is in master, which may fix the sig_t issue. Please give it a try and let me know if it solves things for you.


Can't add custom type to table

I have a playlist object that contains a video object, i'm trying to add a video to the playlist table and i'm getting compile error, here's the error:

Here's the convertor code:
class VideoConverter extends TypeConverter<Video, String> {
  const VideoConverter();

  @override
  Video fromSql(String fromDb) {
    return Video.fromJson(json.decode(fromDb) as Map<String, dynamic>);
  }

  @override
  String toSql(Video value) {
    return json.encode(value.toJson());
  }
}

And the playlist table:
class PlaylistDB extends Table {
  IntColumn get id => integer().autoIncrement()();

  TextColumn get title => text().named('title')();

  TextColumn get videos => text().named('video').map(const VideoConverter())();
}

How can i insert specific video/list of videos to the playlist table?

The error is telling you that you need to wrap video in a Value. Have a look at the docs for updates, deletes, and inserts - the tl;dr is that Value is used to distinguish between "leave this column unchanged" when updating, "use the column's default value" when inserting, and "set this column to null" when inserting or updating.

Thanks you, it's working!
I'm saving a list of videos in the playlist object, it's working good, i can't find a way to remove specific video from the list, here's the code:
VideoModel video = VideoModel('videoId1', 'channelId', 'title');
await into(playlists).insert(PlaylistsCompanion.insert(
  title: 'video.title',
  videos: [video],
));

List<Playlist> result = await select(playlists).get();
await delete(playlists)..where((t) => t.videos);         <---- How remove specific video from List<Videos>
print(result);

class Playlists extends Table {
  IntColumn get id => integer().autoIncrement()();

  TextColumn get title => text().named('t')();

  TextColumn get videos => text().named('v').map(const VideoListConverter())();
}

class VideoListConverter extends TypeConverter<List<VideoModel>, String> {
  const VideoListConverter();

  @override
  List<VideoModel> fromSql(String fromDb) {
    return jsonDecode(fromDb).map((e) => VideoModel.fromJson(e)).toList().cast<VideoModel>();;
  }

  @override
  String toSql(List<VideoModel> value) {
    return json.encode(List<dynamic>.from(value.map((e) => e.toJson())));
  }
}

For those queries, I'd generally recommend a more typical relational design, where you have three tables (videos, playlist, video-in-playlist). That makes the select a lot more complicated, but it's much easier to remove a specific video from a playlist.
In your design, I think the easiest way would be to:

Select videos from the playlist
Remove the video in Dart
Use an update to remove the video in the SQL.

This could look something like this:
await transaction(() async {
  final playlist = await (select(playlists)..where((row) => /* filter for the playlist where you want to delete a video */)).get();
  final videos = playlist.videos.toList();
  videos.remove(videoToRemove);
  await (update(playlists)..whereSamePrimaryKey(playlist)).write(PlaylistsCompanion(videos: Value(videos));
});

No activity in over a year & seems to be resolved, closing


Investigate LED Strips

We need to look into using LED strips again, for signaling events to the driver. This could include drivetrain orientation (front/back), cargo possession, or even a mix of the two.
However, I think we should look into alternate ways to do this, as we used an Arduino over I2C last year, which may or may not have been entirely necessary.
NOTE: Please do NOT begin work on this issue until every other aspect of the robot has been tested in its entirety. This is a secondary functionality and is something that we can work on even after the robot has been bagged.

closing based on discussion today, will reopen as necessary


Multiple calls to Delete for non-existing key result in EOF error instead of ErrNotFound

While handling my deletion logic and ensuring that it is non-fatal to delete a non-existing key, I have found that deleting more than once can result in an EOF being leaked from the decoding of a nil value in the bolthold delete() call.
Here is a small reproduction:
func TestDeleteEOF(t *testing.T) {
    path := "/tmp/bolthold_delete_test.db"
    os.Remove(path)
    db, err := bh.Open(path, 0666, nil)
    if err != nil {
        t.Fatal(err)
    }
    defer db.Close()
    type Item struct{ Name string }
    if err = db.Insert("key", &Item{"Name"}); err != nil {
        t.Fatal(err)
    }
    empty := &Item{}
    for i := 1; i < 5; i++ {
        t.Logf("Delete #%d", i)
        err := db.Delete("key", empty)
        if err != nil && !errors.Is(err, bh.ErrNotFound) {
            t.Fatalf("unexpected error after delete #%d: %v", i, err)
        }
    }
}

// unexpected error after delete #2: EOF

I could catch the EOF as well as ErrNotFound, but I am not sure if an EOF could also come from a legitimate error talking to bolt.

Yeah it's hitting the end of an iterator I think.  I should capture that and treat it like a Not Found.
Thanks,

Thanks for the fast fix!


Update Travis CI config to use pylint and caching

pylint seems to be more comprehensive than pep8, so this PR will have Travis CI use that instead for linting.
In addition, caching is enabled on Travis for pip package installs.
@OpenPoGo/maintainers

Possibly make it use a wildcard? *.py, rather than specific directories

@jtdroste I tried that and it kept going through a lot of junk I didn't want.

pep8 is obsolete; it's been renamed to pycodestyle. also, is flake8 not good enough? i find it beyond sufficient.

Approved

:+1:


chore: create person card component (WEB-43)

Created person component with tests.

ImageUrl looked off vs default image url. Fixed now.


Fixed bug and updated for loop to use range

Vector iterator was declared without std:: namespace, which made compilation fail. I've updated this snippet to use a range for instead of iterators, which will make it compile and is also cleaner.

Actually, this has been fixed in the extension repo: https://github.com/Bhupesh-V/30-seconds-of-cpp/blob/master/algorithm/generate.md

It will be updated with the next release of the extension, then.


A more useful plot_convergence

Only plots cost convergence instead of trying to plot variables convergence
Ready for review @whoburg  @bqpd

(Bump)
(Should be a really easy review)

Attach an example plot and I'll merge it. :D

How's this?

:+1:

sweet. minor design discussion: should we plot this on a semilogy scale?

Hmm! I'd argue no; we're checking the effects of negynomials, which I wouldn't expect to span too many orders of magnitude?

Actually, gonna change my mind and argue yes, since the terminating distinction is reltol...

The downside being a lot of these plots end up being flat on a semiology scale (as suggested by @bqpd)

Can't we autoscale the y axis to prevent that?

Hm, yes. But why does the log scale help in this case?
On May 14, 2016, at 12:19, bqpd <notifications<EMAIL_ADDRESS>wrote:
Can't we autoscale the y axis to prevent that?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHubhttps://github.com/hoburg/gpkit/pull/706#issuecomment-219229037

Oh, because the sp solve terminates when the previous solution is close enough to the current solution in  logspace, so plotting that directly might be more easily readable?

Clearly there's arguments both ways, though. :p

Well then perhaps it makes most sense to plot the absolute difference from the final cost. Otherwise I'm not sure you can "see" the convergence condition being satisfied.
On May 14, 2016, at 12:25, bqpd <notifications<EMAIL_ADDRESS>wrote:
Clearly there's arguments both ways, though. :p
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHubhttps://github.com/hoburg/gpkit/pull/706#issuecomment-219229331

Hmm, any multiplicative distance (like reltol) should show up as a fixed delta-y.


Add Fragment Documentation

I think the content you've copied from the blog post needs to be reframed a bit. It's written toward an audience of experienced React developers. But many people who read the Fragments doc will be using React for the first time, and won't care about limitations that existed before 16.2 — in fact, it may just serve to confuse them.
Instead of "here's how you used to return multiple children, and here's how you do it now," let's reframe it as "here's why you might need to return multiple children, and here's how you use fragments to do it."

Updated copy to be more beginner-friendly @acdlite


HDDS-2252. Enable gdpr robot test in daily build

What changes were proposed in this pull request?
Updated test.sh script to include gdpr.robot test so that it gets triggered as part of daily build.
Link to Apache JIRA
https://issues.apache.org/jira/browse/HDDS-2252
How was this patch tested?
Started Docker
cd ~/ozone-SNAPSHOT/compose/ozone
./test.sh

/label ozone

+1, pending Jenkins.

Checkstyle issue is unrelated to patch. Filed HDDS-2257 to address it.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
87
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

0
shelldocs
1
Shelldocs was not available.

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

-1
mvninstall
43
hadoop-hdds in trunk failed.

-1
mvninstall
37
hadoop-ozone in trunk failed.

+1
mvnsite
0
trunk passed

+1
shadedclient
948
branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

-1
mvninstall
39
hadoop-hdds in the patch failed.

-1
mvninstall
41
hadoop-ozone in the patch failed.

+1
mvnsite
0
the patch passed

+1
shellcheck
0
There were no new shellcheck issues.

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
808
patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1
unit
27
hadoop-hdds in the patch failed.

-1
unit
24
hadoop-ozone in the patch failed.

+1
asflicense
31
The patch does not generate ASF License warnings.

2223

Subsystem
Report/Notes

Docker
Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1602/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1602

Optional Tests
dupname asflicense mvnsite unit shellcheck shelldocs

uname
Linux 533275c9d8f3 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 8de4374

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-1602/1/artifact/out/branch-mvninstall-hadoop-hdds.txt

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-1602/1/artifact/out/branch-mvninstall-hadoop-ozone.txt

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-1602/1/artifact/out/patch-mvninstall-hadoop-hdds.txt

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-1602/1/artifact/out/patch-mvninstall-hadoop-ozone.txt

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1602/1/artifact/out/patch-unit-hadoop-hdds.txt

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1602/1/artifact/out/patch-unit-hadoop-ozone.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1602/1/testReport/

Max. process+thread count
362 (vs. ulimit of 5500)

modules
C: hadoop-ozone/dist U: hadoop-ozone/dist

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1602/1/console

versions
git=2.7.4 maven=3.3.9 shellcheck=0.4.6

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@anuengineer , @adoroszlai thanks for review.
@elek Thanks for commit.


Write a Python script to upload some example datasets to Girder

I'm starting in on this... any thoughts on a separate repo to house the example datasets/toolchains?

Is there a need for them to be in a separate repo? I just pictured a few example datasets that can work with the candela components exposed in the reference app. The script would be one of the "quick start" steps so the user doesn't need to create the special collections/folders and do the uploads by hand.

That's fine... just trying to figure out the best place to put things.
Maybe something like this?
candela
    /exampleCollections
        /Data
        /Toolchains

Is there any particular way you imagine we'd say that /Toolchains/CarScatterplot is the default toolchain for the scatterplot component?

Since these are specific to the reference app, I'd imagine them going somewhere under the reference app root. Something like what you've proposed, except:

I would maybe just call the directory examples
If we don't want to call them "toolchains" we ought to think about a better name for them

Ah, I hadn't seen #93 yet :)

I like this
candela
  /app
    /resonant-reference-app
      /examples
        /Data
        /Toolchains

:+1:


Remove videos from limit count

When I set limit to 10 and 5 of them are videos, I see only 5 images. Would be great to load 10 images no matter how many videos are in. Or show videos as images.

Yeah, I agree, that is how the plugin should work. I'm just not sure what the best approach for that would be.  I'll have to explore this when I have some time.

+1


Gestures folder

How to use the gestures folder created by you?

Only the gestures.db is provided.How to recover the gestures folder from the db provided?

facing same problem here

so looks like if you run create_gestures.py, this will create that folder for you. Silly me


:interrupted status

If I reboot my worker when it runs Sidekiq::Status.status always returns :working, although the job is not running anymore. Is there a way to figure out that my job was interrupted?

I have the same problem,
after this
kill -TTIN ...
the job status still is working.

I have the same issue as well. Would be great to know what the job has been interrupted

@utgarda - Expecting you to close this after gem update :)

Fixed in #67


Avoiding confusion with "mutating state"

Hi Dan and the whole redux Dev Team,
I'd kindly ask you to review this little documentation change affecting the introduction/Three Principles chapter.
As proposed in the commit message I'd suggest to use the word 'change' instead of 'mutate' to avoid confusion with the concept of 'immutable state'. As we all know: While not enforced, making state objects immutable is a strongly recommended way of using redux (and only then redux plays well together with pure reducers, time traveling and so on).
I found myself felt a bit confused when reading the introduction for the first time. Furthermore I remember to have seen similar comments elsewhere here.
Maybe this little change helps people not getting on the wrong path in the very beginning.
Cheers and thanks for checking this,
Matthias

Seems pretty reasonable.  Thanks!


Bump Arc and Azdata extensions

Pull Request Test Coverage Report for Build 263793613

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

Totals

Change from base Build 263351593:
0.0%

Covered Lines:
19153

Relevant Lines:
45789

💛  - Coveralls


 This triggers the docs.yml action, which attempts to deploy the documentation even if Github Pages is not enabled. (It also creates a CNAME file specific to Bevy and should not be used in forks, even for testing.)
For an example, see this run.

Solution

Only attempt to deploy docs when running from the main Bevy repository.

This does not affect us checking cargo doc on pull requests, since that it done in ci.yml.

Testing
It's difficult to test this, but you'd probably:

Fork Bevy
Cherry pick this PR's commits onto the main branch of your fork.
Push another commit to the main branch, triggering Github Actions.
Check the Github Actions job summary to ensure that the build-and-deploy job is skipped.

cc @tychedelia :)


Extend Steward To Support CCIP Controls

https://governance.aave.com/t/arfc-extend-gho-stewards-to-arbitrum/18298

This proposal was introduced by https://github.com/bgd-labs/aave-proposals-v3/pull/502


Set up firebase

Closes #51.

(As part of installing AngularFire, we need to upgrade to Angular version 9.)


dev: improve testing of to_invoke_tx

Improve the testing of the rpc-core/src/utils.rs file by adding unit tests for to_invoke_tx. The tests are expected to cover edge cases in order to be accepted.

@greged93  I would love to take this!

@greged93 I would love to take this!

Assigned to you

@MdTeach need to wait on #597 to be merged

Hey @MdTeach, #597 was merged and moved the logic to the file primitives/starknet/src/transation/types.rs and the implementation changed to impl TryFrom<BroadcastedInvokeTransaction> for InvokeTransaction. Would be great if you are still able to provide tests for this.

sure @greged93


Fixed changing screen orientation on Android

What was broken
The main Android thread wasn't notifying the engine about screen orientation changes. Android still was rotating screen but the engine was rendering the scene like there was no rotation, which was causing the surface to be stretched and wrongly orientated. Below is the video demonstrating the problem.
video
Fix
VkThread.onSurfaceChanged() now calls VkRenderer.onVkSurfaceChanged() upon changing the screen orientation, which resets the window (by calling DisplayServerAndroid.reset_window()). Resetting the window is not enough though. All framebuffers need to be invalidated and recreated. This happens through the new method DisplayServerAndroid.notify_surface_changed(), which mimics the behavior from the Windows' build. Here is the video of the screen orientation with the applied fix.
video

@szymonm-google Sorry for the delay - The PR looks good! I've added two small comments, and it should be good for approval once they're resolved.

@szymonm-google Sorry for the delay - The PR looks good! I've added two small comments, and it should be good for approval once they're resolved.

No worries, thank you for the review!

Thanks!


Getting started guide improvements

Hi, I was following the Getting started guide and I noticed few places which can be improved.  There were few lines of code missing and without them it was not possible to implement a new cities feature.

Current coverage is 94.27% (diff: 100%)

Merging #69 into master will not change coverage

@@             master        #69   diff @@
==========================================
  Files            57         57
  Lines          3861       3861
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           3640       3640
  Misses          221        221
  Partials          0          0

Powered by Codecov. Last update 208d182...28b4619


Build error caused by futures-util=0.3.7

I feel like futures-rs messed up by making that update a patch version change only but alas, It would be great to have this fixed.
httpmock % cargo test

   Compiling httpmock v0.5.1-alpha.0 (/Users/user/Code/httpmock)
error[E0433]: failed to resolve: could not find `core_reexport` in `futures_util`
   --> src/lib.rs:202:19
    |
202 | use futures_util::core_reexport::time::Duration;
    |                   ^^^^^^^^^^^^^ could not find `core_reexport` in `futures_util`

error[E0433]: failed to resolve: could not find `core_reexport` in `futures_util`
   --> src/lib.rs:202:19
    |
202 | use futures_util::core_reexport::time::Duration;
    |                   ^^^^^^^^^^^^^ could not find `core_reexport` in `futures_util`

error[E0412]: cannot find type `Duration` in this scope
    --> src/lib.rs:1407:26
     |
1407 |     pub fn delay<D: Into<Duration>>(self, duration: D) -> Self {
     |                          ^^^^^^^^ not found in this scope
     |
help: consider importing one of these items
     |
196  | use core::time::Duration;
     |
196  | use crate::data::Duration;
     |
196  | use std::time::Duration;
     |
196  | use tokio::time::Duration;
     |

error[E0412]: cannot find type `Duration` in this scope
    --> src/lib.rs:1407:26
     |
1407 |     pub fn delay<D: Into<Duration>>(self, duration: D) -> Self {
     |                          ^^^^^^^^ not found in this scope
     |
help: consider importing one of these items
     |
196  | use core::time::Duration;
     |
196  | use crate::data::Duration;
     |
196  | use std::time::Duration;
     |
196  | use tokio::time::Duration;
     |

error: aborting due to 2 previous errors

Some errors have detailed explanations: E0412, E0433.
For more information about an error, try `rustc --explain E0412`.
error: could not compile `httpmock`.

To learn more, run the command again with --verbose.
warning: build failed, waiting for other jobs to finish...
error: aborting due to 2 previous errors

Some errors have detailed explanations: E0412, E0433.
For more information about an error, try `rustc --explain E0412`.
error: build failed

I also noticed a BUNCH of packages are out of date. Do you think it'd be possible to update them and as a check for any  lingering uses of "deprecated" code?

#20 seems to be a solution for this problem

Yes, #19 will fix this problem. Thanks! I fixated all dependency versions in #19 so this should not happen again.

Thanks! Any timeline for a 0.5.1 release?

Yes, I am waiting for CI test results and will release then.

Fixed in 0.5.1 .


Add note about new self-hosted config file field

Description of the change

Checklist
Please make sure that the proposed change checks all the boxes below before requesting a review:

[ ] I have reviewed the guidelines for contributing to this repository.
[ ] The preview looks fine.
[ ] The tests pass.
[ ] The commit history is clean and meaningful.
[ ] The pull request is opened against the main branch.
[ ] The pull request is no longer marked as a draft.
[ ] You agree to license your contribution under the MIT license to Spacelift (not required for Spacelift employees).
You have updated the navigation files correctly:

[ ] No new pages have been added, or;
[ ] Only nav.yaml has been updated because the changes only apply to SaaS, or;
[ ] Only nav.self-hosted.yaml has been updated because the changes only apply to Self-Hosted, or;
[ ] Both nav.yaml and nav.self-hosted.yaml have been updated.

If the proposed change is ready to be merged, please request a review from @spacelift-io/solutions-engineering. Someone will review and merge the pull request.
Spacelift employees should request reviews from the relevant engineers and are allowed to merge pull requests after they got at least one approval.
Thank you for your contribution! 🙇

@adamconnelly thanks, I can never decide whether to go capital or not in such lists. I'll do capital.

If it's any consolation, I recently realised that in US English it's typical to use a capital after a :, so "I love three things: Cats, cats, and cats", but in UK English you'd use lowercase because it's not a new sentence "I love three things: cats, cats, and cats".
My mind was blown :exploding_head:

@adamconnelly 🤐 that, I'll stay with the lowercase 😂


fix can't push git notes bug

Not  sure can f-i-x #16688, Because this nit will't cause crash bug.

Codecov Report

Merging #16689 (71b3384) into main (23a87a0) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main   #16689   +/-   ##
=======================================
  Coverage   45.39%   45.40%
=======================================
  Files         758      758
  Lines       85306    85308    +2
=======================================
+ Hits        38727    38735    +8
+ Misses      40298    40294    -4
+ Partials     6281     6279    -2

Impacted Files
Coverage Δ

routers/private/hook.go
34.31% <0.00%> (-0.14%)
:arrow_down:

modules/queue/workerpool.go
55.34% <0.00%> (+0.38%)
:arrow_up:

models/repo_list.go
78.43% <0.00%> (+0.78%)
:arrow_up:

modules/process/manager.go
75.30% <0.00%> (+2.46%)
:arrow_up:

modules/util/timer.go
85.71% <0.00%> (+42.85%)
:arrow_up:

Continue to review full report at Codecov.

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

Whilst I agree that this would fix the problem, the function HookPreReceive is a total mess right now.
Adding yet another else if block to that sprawling if else inside a for loop isn't going to improve things.
It needs a more substantial refactor. The code inside the if blocks needs to be farmed out to functions and the duplication related to the proc receive needs to be refactored away.
The internal server error return is incorrect as it should be a forbidden. But we need to think more carefully about which refs we're disallowing e.g. this code will now prevent any number of extensions to git and I'm not certain that that is intended. It's probably the case that this forbidden should be dropped or at least switchable.

Closing in favor of #16706


How does keras build batches depending on the batch-size?

I'm trying to implement a Sentiment-Classifier using keras. But i got some problems with the batch_size param. It might be kind of a specific problem and I didn't find anything online that helped me.
Here is an example how the model could look:
Input_1				Input_2
(None, 200)			(None, 200)
  |				   |
  |				   |
Embedding_1			Embedding_2
(None, 200, 200)		(None, 200, 200)
  |				   |
  |				   |
Reshape_1			Reshape_2
(32, 200, 200)			(32, 200, 200)
    \				   /
     \	                          /
      -----------        ---------
                 \      /
		Concatenate
		(32,200,400)
		      |
		      |
		      ...

Reshaping is done by a Lambda-Layer using the reshape-function from keras.backend:
output = Lambda(lambda x: bd.reshape(x, (batch_size, 200, em_dim)), name='Reshape_Batch_size')(embedded)
I reshape the outputs of both embeddings (also using the batch_size) because the keras Concatenate-Layer needs tensors with equal shape on each axis except from the one you concat on. To be able to do the reshape I fit the number of my samples to a multiple of the batch_size. If I want to train my network everything is working fine until the last batch of the epoch. There I got an error that the number of input_values do not match the expected number:

InvalidArgumentError (see above for traceback): Input to reshape is a tensor with 320000 values, but the requested shape has 1280000 [[Node: Reshape_Batch_size/Reshape = Reshape[T=DT_FLOAT, Tshape=DT_INT32, _device="/job:localhost/replica:0/task:0/device:CPU:0"](Embedding_basic_em/Gather, Reshape_Batch_size/Reshape/shape)]]

320000 = 8 * 200 * 200
1280000 = 32 * 200 * 200

This looks like keras is cutting 24 samples from the last batch. But if I look at the output of the training it tells me there are 32 remaining samples:
Epoch 1/8 17440/17472 [============================>.] - ETA: 0s - loss: 5.1635 - acc: 0.6796

Does anybody know how keras handles batches depending on the given batch_size param or what I'm missing here? Or maybe someone has a hint how i can do the concatination without reshaping my outputs.
My Configuration
I'm working on macOS Sierra version 10.12.6, using Python 3.5.3 from Anaconda 4.4.0 (x86_64). As keras backend I'm using tensorflow(1.4.0).  keras is on version 2.1.1
Thanks in advance

Can you provide the code for the full model?

Already figured it out. Just had to delete the reshape-layers


Add Rollbar access token and environment to ARM templates

 Set up Rollbar on Azure "development" and "production" environments

This looks good, but I think we also need to add the reference to the appSettings for the app service.

Ah yes, thanks!

I've added those two references in appSettings


setValue forKey not working with List

I have an object in Realm. I would like to retrieve that object and then work with an unmanaged version of that object. Unfortunately, it doesn't seem like Realm for iOS has any good copy options, so I tried to followed the "detachable" workaround by "anlaital" outlined in issue #3381.
My code is as follows:

This works for the most part. When on a property that is a list and has values, detachable.detached() does return the expect list copy. Unfortunately, detached.setValue(detachable.detached(), forKey: property.name) does not set the value for any Lists my object has.

Any thoughts on how I can properly assign my "detached"/unmanaged list to my object?

Realm lists don't support reassignment. This is why we recommend that they be declared as let rather than var; anything you can do with a var list you can't do with a let list is considered erroneous behavior.

Alright; as an alternative, is there a way to append to a list by key?

For what it's worth, I've had success with this approach:

I'm appending to lists (reference grabbed via getValue:forKey: ). The unfortunate part is I must be quite verbose when checking list types (note the as? List<Int>, as? List<String, and so on).

I'm glad you figured out something that worked, and I hope others can benefit from your approach! Feel free to open new tickets if you have additional questions.

@austinzheng You comment:

Realm lists don't support reassignment

Seems to contradict the documentation:

You can access and assign List properties as usual:

Am I missing it in the docs, or is it simply a little vague?

The documentation is either unclear or wrong. I think the original intent was to say you can modify the elements inside a List, including assigning new objects to a particular index, rather than assigning a new List instance altogether. I'll update it, thanks for bringing it to our attention.

@allenhumphreys Comment on my other issue (#5469) gave me a magnificent idea.
Instead of having the messy casting for lists like I do in post 4 (which incidentally crashes for objects), because this is a detached/unmanaged object, I should be able to safely do setValueForKey for lists.
I tested this and it seemed to work, but I would like to hear your thoughts @austinzheng.
Do you think it is safe to set/assign a list in this manner on the premise that everything is unmanaged?

If it's unmanaged I don't see why it would be problematic. Does it work?

Indeed it does. I ran a few test cases and they all seemed to work beautifully. However, I do look forward to using this in a project and, effectively, testing it more broadly.
I appreciate your input.

Thank you @Alarson93 this was much helpful :)
This solution works indeed.

If anyone needs the code from @Alarson93, especially when dealing with #3381 (which "accepted" is broken since it doesn't support List):
import Realm
import RealmSwift

public extension Object {
  public func detached() -> Self {
    let detached = type(of: self).init()
    for property in objectSchema.properties {
      guard let value = value(forKey: property.name) else { continue }
      if property.isArray == true || property.type == .object {
        // Realm List and Realm Objects
        let detachable = value as? Object
        detached.setValue(detachable?.detached(), forKey: property.name)
      } else {
        // Primitives
        detached.setValue(value, forKey: property.name)
      }
    }
    return detached
  }
}

public extension Sequence where Iterator.Element : Object  {
  public func detached() -> [Element] {
    return self.map({ $0.detached() })
  }
}

public extension List {
  public func detached() -> List<Element> {
    let result = List<Element>()
    self.forEach { element in
      if let element = element as? Object {
        // Realm Objects
        result.append(element.detached() as! Element)
      } else {
        // Primitives
        result.append(element)
      }
    }
    return result
  }
}

I actually have a slightly newer version of this.

protocol DetachableObject: AnyObject {
    func detached() -> Self
}

extension Object: DetachableObject {

    func detached() -> Self {
        let detached = type(of: self).init()
        for property in objectSchema.properties {
            guard let value = value(forKey: property.name) else { continue }

            if property.isArray == true {
                //Realm List property support
                let detachable = value as? DetachableObject
                detached.setValue(detachable?.detached(), forKey: property.name)
            } else if property.type == .object {
                //Realm Object property support
                let detachable = value as? DetachableObject
                detached.setValue(detachable?.detached(), forKey: property.name)
            } else {
                detached.setValue(value, forKey: property.name)
            }
        }
        return detached
    }
}

extension List: DetachableObject {
    func detached() -> List<Element> {
        let result = List<Element>()

        forEach {
            if let detachable = $0 as? DetachableObject {
                let detached = detachable.detached() as! Element
                result.append(detached)
            } else {
                result.append($0) //Primtives are pass by value; don't need to recreate
            }
        }

        return result
    }

    func toArray() -> [Element] {
        return Array(self.detached())
    }
}

extension Results {
    func toArray() -> [Element] {
        let result = List<Element>()

        forEach {
            result.append($0)
        }

        return Array(result.detached())
    }
}

Hi, can someone translate @Alarson93's solution in Objective C.

Good solution, almost all our realm related crashes are gone, except that now our app crash Object has been deleted or invalidated is pointing to the detaching process for the value getter:
guard let value = value(forKey: property.name) else { continue }

@loilee I am also getting this kind of crash. Have you reported it or solved it?

@Alarson93 ,
Your solution doesn't work if A's child object has A as a child.

@winstondu
Can you provide a simple example? (Pseudo-code is fine)
Also - what happens? Does it simply not map, does it crash, etc?


new user -- cron error from synology nas

hello, i just installed this docker container in my synology nas,
when i launch the execution command from the task scheduler i got an error.
i follow your instruction in a previous issue to remove '-t' and i tried:
docker run --rm -i simaofsilva/noip-renewer 'user' 'password'
and
docker run --rm simaofsilva/noip-renewer 'user' 'password'
i also tried
docker run --rm -i simaofsilva/noip-renewer:latest 'user' 'password'
and
docker run --rm simaofsilva/noip-renewer:latest 'user' 'password'
the response:
Le planificateur de tâches a terminé une tâche planifiée.
Tâche : No-Ip Renewal
Heure de début : Thu, 13 Apr 2023 20:54:22 GMT
Heure d'arrêt : Thu, 13 Apr 2023 20:54:52 GMT
État actuel : 0 (Normal)
Sortie/erreur standard :
Opening browser
Login successful
Confirming hosts phase
Error: Message: The element reference of is stale; either the element is no longer attached to the DOM, it is not in the current frame context, or the document has been refreshed
Stacktrace:
RemoteError@chrome://remote/content/shared/RemoteError.sys.mjs:8:8
WebDriverError@chrome://remote/content/shared/webdriver/Errors.sys.mjs:180:5
StaleElementReferenceError@chrome://remote/content/shared/webdriver/Errors.sys.mjs:461:5
element.resolveElement@chrome://remote/content/marionette/element.sys.mjs:674:11
evaluate.fromJSON@chrome://remote/content/marionette/evaluate.sys.mjs:255:31
evaluate.fromJSON@chrome://remote/content/marionette/evaluate.sys.mjs:263:29
evaluate.fromJSON@chrome://remote/content/marionette/evaluate.sys.mjs:263:29
receiveMessage@chrome://remote/content/marionette/actors/MarionetteCommandsChild.sys.mjs:74:34
Logging off
thanks for your help

so i connected to the website
i had "redemption days 7"
so i tried to manually confirm
i launch again the the scheduled task from my synology
and the response was:
Tâche : No-Ip Renewal
Heure de début : Thu, 13 Apr 2023 21:34:49 GMT
Heure d'arrêt : Thu, 13 Apr 2023 21:35:17 GMT
État actuel : 0 (Normal)
Sortie/erreur standard :
Opening browser
Login successful
Confirming hosts phase
0 hosts confirmed
Finished
Logging off
do you think the initial error came from the fact i was in redemption day?

It is hard to tell since NOIP has captcha mechanisms and may do changes in HTML code.


MetaTest

Hi,
Thanks for this Library
How are we supposed to do meta testing because accuracy u mentioned is of meta train episodes, right?

Hi! I think you are referring to the examples, where the accuracy is indeed always on meta-training episodes. The examples are meant to be very simple to showcase the library, and they are not meant to have complete features (e.g. evaluation on meta-test, different choices of datasets).
To have a complete (and better tested) implementation of an algorithm based on Torchmeta, I suggest you to look at pytorch-maml, which does have a separate test.py script for meta-testing.

yea , i am trying to implement prototypical nets ,i am finding pytorch-meta's functions to be different , how do i directly do it while loading the data in my protonet code .

An option is to follow the Prototypical Network example for the meta-training part, and then adapt the test.py script from pytorch-maml I linked above for meta-testing.

Thanks


Can't debug .Net Core web app

Environment data
dotnet --info output:
.NET Command Line Tools (1.0.0-preview2-003121)
Product Information:
Version:            1.0.0-preview2-003121
Commit SHA-1 hash:  1e9d529bc5
Runtime Environment:
OS Name:     Mac OS X
OS Version:  10.11
OS Platform: Darwin
RID:         osx.10.11-x64
VS Code version: 1.4
C# Extension version: 1.4.1
Steps to reproduce
I only update VS Code and C# extension. After that I can't debug application by VS Code.
Expected  behavior
Proper debug
Actual behavior
1: (2000) STDERR:   Expected in: /Users/karol/.vscode/extensions/ms-vscode.csharp-1.4.1/coreclr-debug/debugAdapters/libmscordaccore.dylib
1: (2559) "/Users/karol/.vscode/extensions/ms-vscode.csharp-1.4.1/coreclr-debug/debugAdapters/clrdbg" exited with code 133 (0x85).
1: (2562) <-logout
E output: {"category":"telemetry","output":"VS/Diagnostics/Debugger/MIEngine/DebuggerAborted","data":{"VS.Diagnostics.Debugger.MIEngine.DebuggerName":"CLRDBG","VS.Diagnostics.Debugger.MIEngine.LastSentCommandName":"-exec-run","VS.Diagnostics.Debugger.MIEngine.DebuggerExitCode":"133 (0x85)","VS.Diagnostics.Debugger.clrdbg.Version":"15.0.25626.0 built by: PREVIEW.DBG2","VS.Diagnostics.Debugger.clrdbg.OSFamily":"Darwin","VS.Diagnostics.Debugger.clrdbg.KernelRelease":"15.6.0"},"type":"output"}
E output: {"category":"stderr","output":"ERROR: CLRDBG exited unexpectedly with exit code 133 (0x85). Debugging will now abort.\n","data":null,"type":"output"}
## ERROR: CLRDBG exited unexpectedly with exit code 133 (0x85). Debugging will now abort.
E output: {"category":"console","output":"The program '/Users/karol/Projects/candyandco/CandyAndCo/bin/Debug/netcoreapp1.0/CandyAndCo.dll' has exited with code -1 (0xffffffff).\r\n\n","data":null,"type":"output"}
The program '/Users/karol/Projects/candyandco/CandyAndCo/bin/Debug/netcoreapp1.0/CandyAndCo.dll' has exited with code -1 (0xffffffff).
E exited: {"exitCode":-1,"type":"exited"}
E terminated: {"type":"terminated"}
C disconnect: {"restart":false}
E output: {"category":"telemetry","output":"VS/Diagnostics/Debugger/DebugCompleted","data":{"VS.Diagnostics.Debugger.ImplementationName":"Microsoft.MIDebugEngine","VS.Diagnostics.Debugger.EngineVersion":"14.0.30822.1","VS.Diagnostics.Debugger.HostVersion":"1.0.20822.1","VS.Diagnostics.Debugger.AdapterId":"coreclr","VS.Diagnostics.Debugger.DebugCompleted.BreakCounter":0},"type":"output"}
R: {"success":true,"message":null,"request_seq":5,"command":"disconnect","body":null,"running":false,"refs":null,"seq":0,"type":"response"}

@kcybul if you try to run clrdbg on the command line, do you see anything else interesting printed to stdout/err?
/Users/karol/.vscode/extensions/ms-vscode.csharp-1.4.1/coreclr-debug/debugAdapters/clrdbg --interpreter-mi

-gdb-exit

MacBook-Pro-Karol:debugAdapters karol$ pwd
/Users/karol/.vscode/extensions/ms-vscode.csharp-1.4.1/coreclr-debug/debugAdapters
MacBook-Pro-Karol:debugAdapters karol$ clrdbg --gdb-exit
-bash: clrdbg: command not found
MacBook-Pro-Karol:debugAdapters karol$

Also, from that directory you will need to run ./clrdbg --interpreter=mi.

@kcybul: Do you have any further information to provide here?

@DustinCampbell No.

Were you able to get it working?

@DustinCampbell No, but this is now old case, and I don't have this environment anymore.

OK, Thanks!


add new --format flag to status command to enable json output

This PR adds the functionality described in #1020.
I added a new --format flag that accepts a file format (for now json, but yaml or whatever else could be added anytime) and prints the result of the status command to stdout.
The test for it is not the most resilient as I couldn't figure out how to do it properly. I wanted to spy on console.log and verifiy it was called with the right JSON, but couldn't do it as the Expected and Received JSONs never matched, if anyone has any idea how to write a better test for this please let me know.

Hey, it would be great to have this feature. What needs to happen for it to get merged?

How can I help get this over the line? Parsing the out all the 🦋 from the output is easy but painful when every other CLI tool provides a way to get usable JSON.

How can I help get this over the line? Parsing the out all the 🦋 from the output is easy but painful when every other CLI tool provides a way to get usable JSON. @Andarist ?

Hey! is there something to do to get this merged ? lmk if i can do something 😉


Use VTKs OpenGL2 backend

I want to test this out specifically for Mac.

According to https://blog.kitware.com/new-opengl-rendering-in-vtk/, this was merged on June 20, 2014, so we would need at least VTK version 6.3. In VTK 6.3, both OpenGL and OpenGL2 backends should be present.
At UCL, Im currently using Qt 5.4.2 for MITK based projects, using a fork of MITK 2015.5.2. But Qt changed their OpenGL significantly in 5.5, and MITK/master now has a minimum Qt 5.6. So, the target is Qt 5.6 or newer.
When I first compiled this project with Qt 5.6, I got issues on Mac, similar to: https://github.com/opencv/opencv/issues/7606, and further googling suggests we may need to specify a minimum Mac SDK or xtools version.
Then, Im also aware that on a Mac, there was this issue https://github.com/PointCloudLibrary/pcl/issues/712 when I was integrating PCL, whereby VTK 7.1 would not build, so I dropped back to 6.1. So the first question is whether VTK 6.3 will compile at all, regardless of the backend. This commit https://github.com/Kitware/VTK/commit/a3e9fc90d95d2379618d6e6f0e630377dbe034ad suggests it should not. But then maybe a newer version of PCL will work?

Tested on Windows, Linux and Mac, but with OpenCV off. If OpenCV doesn't compile with a specific version, then that should be a separate ticket. This is about VTK / Qt and PCL versions.


[Bug]: Edit/Update comment is not working as expected

Edit/Update comment is not working as expected

Try save the updated comment , it keep on  adding "&amper; " how many time you hit "tick (save comment)"
button

Video Link : https://drive.google.com/a/redhat.com/file/d/0B1qUq_laG6COS1ZOWFh2WUwxc28/view?usp=sharing

@debloper / @nimishamukherjee  can you please look into it !

Fixed by #1335, closing this.


Task - Edit Type specimen

Type designator should be selected in two ways. The same way we add an original author. Either as a role or as a citation (preferable way). No I can add a citation only using a radio annotator. This should be a part of the form.

We removed type designator, there is no point to it according to you :).


False data on the Sick Nanoscan3 donel

While working with Sick nanoscan3 lidar model, I am getting false data points forming a line. It comes in the range of -1.5rad to -2.32rad always. We have checked the mountings and removed reflective objects nearby. The parameter min_intensities has been kept at 60.

I am using the parameters:
"frame_id": "front_lidar",
"sensor_ip": "<IP_ADDRESS>",
"host_ip": "<IP_ADDRESS>",
"interface_ip": "<IP_ADDRESS>",
"host_udp_port": 0,
"channel": 0,
"channel_enabled": True,
"skip": 0,
"angle_start": -1.98,
"angle_end": 1.85,
"time_offset": 0.0,
"general_system_state": True,
"derived_settings": True,
"measurement_data": True,
"intrusion_data": True,
"application_io_data": True,
"use_persistent_config": False,
"min_intensities": 60.0,
Please suggest the solution for this.

Any new inputs to this @sumitsingh0000 ?

Closing since no new input was provided, please reopen if new information arise


Activity Bar Badge doesn't show smoothly.

Hi, thanks for writing this extension. I was using Dark Code before I saw this color theme. Thank you for that.
But activity bar badge is not shown very clearly. You can look at following image

I have just fixed it 🚀

Thank you


Code Refactor (3x speed improvemnets) + new way to generate db files

Benchmarked both scripts, original code 10000-15000 iter/sec refactored code 30000-35000 iter/sec.

I will test this PR on weekend

This PR will be rejected due to following reasons:
- Unnecessary refactoring that brings increasing of memory consumption during program execution: 3347MB -> 4316MB.
- Unnecessary refactoring that breaks integrity of the repository - database directory name and files.
However,
Indeed PR version shows significant performance improvements over original code, but that solely because of the updated dependencies — newer version of secp256k1 uses updated version of rand (0.6 -> 0.8) under its hood, that affects core performance of the program — generation of new keys.
Furthermore, after updating dependencies original code shows better performance than PR - advantage of 0.67%.
Resolution:
Code dependencies will be updated to the latest (PR) one.


chore(eslint): set environment to reflect #9e79050

due to test framework migration from mocha to jest,
eslint env should also be set to jest

Thanks! This slipped down during the migration.

@okonet oopsss, I'm sorry 😔
I think I accidentally included package-lock.json in this PR.
want to remove it?
or how about if I submit a PR doing the same thing like https://github.com/luftywiranda13/eslint-config-bunny/pull/98?
so it'll never happen again in the future

Good idea. Please create a new PR for that.

sure, i'm working on it


Add Sven Co-op

This commit will add the Sven Co-op dedicated server utilizing the generic module.

Wow, you put a lot of work into this.
You don't think leaving a lot of these parameters to the server cfg file alone will suffice?

That is always an option, but what would be the purpose of doing that? It seems AMP's goal is to make hosting easier by avoiding config files and providing a user interface.
-------- Original Message --------
On Dec 22, 2021, 5:42 PM, Levin V. wrote:

Wow, you put a lot of work into this.
You don't think leaving a lot of these parameters to the server cfg file alone will suffice?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>


feat: multiple signatures verification

What would you like to be added?
A container image could be signed by one trust identity multiple times, or by multiple trust identities, either case will produce multiple signatures associated with the image.
Trust identity is the identity that the signer trusts, for X.509 PKI, it means the signing certificate subject. For example, "C=US, ST=WA, L=Seattle, O=acme-rockets.io, OU=Finance, CN=SecureBuilder".
Scenario 1: A container image was signed by the same trust identity multiple times.
A container image was signed by different teams in the same company wabbit networks. For example, the builder team built the container image and signed it by adding annotation build=123. Then the QA team verified it and signed it using the same signing certificate but adding a different annotation test=123. Then this image had two signatures from the same trust identity with different annotation
Scenario 2: A container image was signed by multiple identities

A container image was signed by different teams in the same company Wabbit Networks, for example, the builder team built the container image and signed it, QA team verify it and signed it again using a different signing certificate before promoted to production. Then this image had two signatures from different trust identities from the same company.

A container image was signed by the company Wabbit Networks and published to a public registry. The company ACME Rockets only allows verified images to be used by internal teams. The acquiring team of the company ACME Rockets verified the image and signed the image, and then pushed it into internal registry. This image had two signatures from different trust identities from different companies.

A container image was signed by the company Wabbit Networks using tooling cosign and published to a public registry. The company ACME Rockets only allows verified images to be used by internal teams. The acquiring team of the company ACME Rockets verified the image, signed the image using tooling notation and pushed it to internal registries. Both cosign and notation can sign images but produce different formats of signatures. This image had two signatures with different formats.

Ratify policy for multiple signatures verification
Ratify policy should be able to support multiple signatures verification, so that users have a fine-tuned policy for validating signatures based on different scenarios.

Pass verification by 1 signature out of N signatures. The trust identity could be one from a trust identity list or any identity
Pass verification by N signatures from out of M signatures, each of the N signatures has a different trust identity

Scenario-1 has lower priority than Scenario-2.
Anything else you would like to add?
The solution could be related to feature multiple verifiers that covered by this issue https://github.com/deislabs/ratify/issues/448
Are you willing to submit PRs to contribute to this feature?

[ ] Yes, I am willing to implement it.

For a small company, both teams could use the same signing certificate, but added different annotations to the signature payload.

@yizha1 This is an interesting scenario. We hadn't considered annotations being used in policy. Currently, Ratify generates verification reports that contain details at the artifact level. For notation, for example, the report looks like this:
{
  "isSuccess": true,
  "verifierReports": [
    {
      "subject": "artifactstest.azurecr.io/resource-1-5sigs100subjects@sha256:06a076a09592b744cdbd3f87cc54c15d4f5ef9be47626261c1b002291edb0b51",
      "isSuccess": true,
      "name": "notaryv2",
      "message": "signature verification success",
      "extensions": {
        "Issuer": "CN=wabbit-networks.io,O=Notary,L=Seattle,ST=WA,C=US",
        "SN": "CN=wabbit-networks.io,O=Notary,L=Seattle,ST=WA,C=US"
      },
      "artifactType": "application/vnd.cncf.notary.signature"
    }]
}

The notation verifier adds some custom fields such as the Issue and SN. If a user is adding custom annotations to their sigs, then they'd want those annotations showing up in the artifact's verifier report so that the rego policy can consider that.
Few follow up questions:

Does notation cli support custom annotations?
Is adding the annotations of the artifact a  verifier-specific implementation detail? or is it a global verifier report addition that should be added regardless of the verifier run?

@akashsinghal

Few follow up questions:

Does notation cli support custom annotations?
Is adding the annotations of the artifact a verifier-specific implementation detail? or is it a global verifier report addition that should be added regardless of the verifier run?

@akashsinghal

notation supports signing and verifying artifacts with user-metadata (annotation added to signature payload), see signing example and verifying example
It's for notation only.

I missed one scenario, it is about a container image that is signed by multiple signing tools, for example cosign and notation, which results multiple signatures as well. I will update the issue soon.

Thanks @yizha1 for pointing me to the user metadata functionality. I wasn't aware this was supported. @binbin-li do you think Ratify would need to add the user annotation to verifier report so that policy could be applied?

those annotations are verified by notation verifier not by the OPA engine. Adding it to verifier report might not help the scenario, but we can still add it to extensions field. Instead, it should be added in verifier config or somewhere else that can be updated dynamically.

@binbin-li I updated the issue description and added one case for scenario-2. Scenario-1 is less important than scenario-2

Discussed with @yizha1, Ratify already supports it via Rego Policy. But we need to provide some templates for the below scenarios to help users try it:

Multiple signatures are signed by different signing keys issued by the same root CA.
Multiple signatures are signed by different signing keys issued by different root CAs.


[ImageUpdate] Update Docker & K8s depl images

What this PR include?

Updated Docker deployment images
Updated K8s deployment images

Can one of the admins verify this patch?


Allow navbar items to be hidden based on path

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Motivation
I would like to show/hide navbar items based on the path. For example

/docs/cli - show a github link and version dropdown for our CLI
/docs/supabase-js - show a github link and version dropdown for our client libraries

Implementation idea
The NavBar Item has a field activeBaseRegex, which I thought I could re-purpose for this, but it appears this function doesn't run on all NavBar items (only "Navbar link"). If this can be applied to all top-level links, then we could use the navbar__link--active CSS class to show/hide navbar items
One more thing
Thank you! We really love Docusaurus at supabase 🦖 . I'd be happy to swizzle this and test it on a live site if someone could let me know what approach to take
Self-service

[ ] I'd be willing to do some initial work on this proposal myself.

Duplicate of #4389. You can find some useful tips there. I'm going to close this for now—if you have further questions feel free to either ask under that issue or open a discussion.

got it, thanks for the link to the other issue - it really helped. For anyone coming here, this is how I solved it:
I added a custom regex to my navbar item (supabaseCustomNavBarRegex)
  {
    to: 'gotrue',
    position: 'left',
    label: 'GoTrue',
    supabaseCustomNavBarRegex: '(^/gotrue$|gotrue/)',
  },

Then I swizzled the NavbarItem, and show/hide based on the regex:
import React from 'react'
import { useLocation } from '@docusaurus/router'
import { isRegexpStringMatch } from '@docusaurus/theme-common'
import NavbarItem from '@theme-original/NavbarItem'

export default function NavbarItemWrapper(props) {
  const { pathname } = useLocation()
  if (
    !props.supabaseCustomNavBarRegex ||
    isRegexpStringMatch(props.supabaseCustomNavBarRegex, pathname)
  ) {
    return <NavbarItem {...props} />
  } else {
    return null
  }
}

Great solution.
In terms encapsulation it's not 100% ideal because if the component itself decide to render null, the parent component (responsible for placing the navbar items in the navbar) wouldn't know it, and could still apply some margin/spacing that shouldn't be there.
We'll implement something similar, but we'll likely filter the navbar items in the parent component


NFS Demo link is not available

https://github.com/kubernetes-incubator/external-storage/blob/v5.2.0/nfs/docs/demo
The link used in the nfs readme (with the hyper link as - here) for showing the demo does not exist.
Can you upload the demo at the link or update the link with the correct one.

The demo guide was removed because it wasn't maintained, the quick start should have enough information, it's just less verbose/explanatory https://github.com/kubernetes-incubator/external-storage/tree/v2.0.0/nfs#quickstart. You can view the old demo here https://github.com/kubernetes-incubator/external-storage/tree/v2.0.0/nfs/docs/demo but not everything in it is true anymore, I will fix the broken link (by removing it)
.area nfs

/area nfs


Regurarly update UI based on extracted signal strength

User story

As a user
I want / need to see visually how the bluetooth signal of a BLE device changes based on the movement of my own device
So that I get an indication on where the BLE device could be located.

Further description
precondition:

#74
#71

Acceptance criteria

[ ] Regularly update the view to indicate distance to selected device
[ ] Connect with created view (#71)

Definition of Done

It builds without errors
code has been reviewed
Build has been made and deployed on a testing environment
Code coverage >75%
All acceptance criteria were met
Documentation updated whenever needed within planning documents (Tabs: Bill of materials, Documentations)
Branch/feature was merged into dev and then main

@domi1504 @jannikbmc  @Meisterlu @codingwithelijah
Quick update:
I connected the circle radius to a rough estimate of what the distance to the device could be.
Still pending:

Distance formula is dependent on some constants we need to poll from the device.
This requires us to be connected to the device (which is probably necessary anyway - see markus' mail).
@Wandinad I don't know if you already implemented the strength search button for the scan list. In case you didn't it should be obsolete anyway.
For us this means:

Get rid of the AvailavleBTDevice class and use the IDevice interface provided by the framework in the viewmodel instead. (Object needs to be passed back to the framework to connect).
Enable the bluetooth service to actually connect to devices and poll their characteristics (=> check out Gatt model).

Animate circle transitions.

@domi1504 @jannikbmc
We might want to use the following library for animations:
https://github.com/OliveTreeBible/Xamarin.Transitions
I will look into it later.

@NicoStellwag thank you for the update! So just for my understanding and to avoid misunderstandings, whenever we want to search the device via signal strength solution we need to be connected with to this specific device.

@Meisterlu
Apparently I misunderstood the meaning of "connect" when researching.
I thought of connecting to a device in terms of persistently pairing it with the smartphone, but all it meant was establishing a communication channel (which we already do to poll the rssi).
Regarding the other points:

We decided animation of the circle was not really necessary, the gui looks acceptable (for an MVP) as it is
Polling the additional constants from the devices to normalize the circle size can also be posponed after the mid release

@NicoStellwag alright! Makes sense, thank you for the update.


auto session.reload()

new page socket.session  synchronization.
rewrite socket.on

Does this get rid of the need for socket.handshake.session.reload(err => {})

Hi @liangzeng . Thanks for this. I missed this PR of yours... I think I was traveling when you pushed it ... Will check... It needs to be rebased tho but I'll take care of it


[Bug][Alert] Alert instance edit page partial parameter values are not displayed

Describe the bug
Alert instance edit page partial parameter values are not displayed

To Reproduce
Steps to reproduce the behavior, for example:

Go to 'Alert instance management'
Click on 'Edit button for an alert instance'
See error

Which version of Dolphin Scheduler:
-[dev]
I will fix it

@CalvinKirs
There may be some bugs in vue code in the module of ui.
There are the same problems in email alert instance edit page too.
At the same time, while updating the alert instance, some necessary colums were missing so after updated alert instance, some columns were missing and that lead to some runtime exceptions.
I've got no such ablilities to develop vue code, so,I just suggest some bug tips.

@CalvinKirs
There may be some bugs in vue code in the module of ui.
There are the same problems in email alert instance edit page too.
At the same time, while updating the alert instance, some necessary colums were missing so after updated alert instance, some columns were missing and that lead to some runtime exceptions.
e.g. while using email plugin, show type will missing after updating the alert instance.
I've got no such ablilities to develop vue code, so,I just suggest some bug tips.

I think what you said is the same as what I reported above


Better formatting for the Inspect dialog

Unfortunately, @kammitama5 adding properties to the inspect dialog, while useful, made it a bit ugly and harder to read.
Some ideas for cleaning this up:

Source formatting.  I'd like to format these more in the form they appear in the program itself.  For example, instead of lettering { text = 'Hashim' }, I'd want to see lettering("Hashim") when in CodeWorld mode, or lettering "Hashim" when in Haskell mode.  For transformations, we could do something like translated(..., 4, 7) (in CodeWorld mode), or translated 4 7 (in Haskell mode).  This means we need to do something mode-dependent here, either by passing in a formatter, or just detecting the mode inside the program.
Contextual tool tips. Unfortunately, the previous idea loses descriptions of the meaning of the arguments.  I am afraid this is a fundamental tension - the wordier the result, the less likely it is to be useful at a glance; but the terser, the less likely to be informative.  But we can at least add the description in a tool-tip.  So when you hover over the 4 above, a tooltip could pop up saying "horizontal = 4".
Smarter ellipsization.  Instead of ellipsizing the entire string, we could start by ellipsizing each of the arguments separately.  Since this takes less away from the usability, we could be a bit more aggressive at ellipsizing, making the labels shorter (and, therefore, the structure more clear).  When arguments are ellipsized, we can still attempt to show more of the argument in the tool-tip.
Type specific formatting. When rendering a color, we should consider rendering it in the same hue and saturation as the color itself (but not lightness, because it must remain readable).  In tool tips, we can go even further.  The tool tip for a color should show a swatch, as well as the RGBA and HSL values.  The tool tip for a point can show the point on a coordinate plane.  A list of points should probably have separate tool tips for its member points.

Another important improvement would be to distinguish between & and pictures.  They can remain the same constructor, but would need to be represented differently if expressions are displayed in source form.
At the same time, we can also do better at not collapsing & chains with explicit pictures calls.  That is, today, a & pictures [b, c, d] will result in Pictures ... [a, b, c, d].  This isn't a faithful representation of the source code.  If we knew that [b, c, d] was an explicit application of pictures, then we could avoid fusing an & onto it, and end up with Pictures ... [a, Pictures ... [b, c, d]], as expected.
This isn't a complete solution.   [Pictures ... [leftEye, rightEye], nose, mouth], but instead we get a flat list.  This is a harder problem to solve, though.

The current source formatting should be almost entirely compatible with HTML, unless there are string literals that look like HTML markup.  So one plan could be to switch the client to interpret labels as HTML, first, and then switch the codeworld-api library to start emitting better markup, including the tooltips and type-specific formatting.  This seems better than inventing a whole new language for argument lists just to support formatting.

I've now fixed the issue with & versus pictures.  Checking for overlap in the call stack let me do a much better job than I even expected, so expressions like a & b where b = b1 & b2 are properly grouped.  (This case is detected because the SrcLoc of the right-hand side is not contained within the SrcLoc of the top-level & expression.)

Remaining tasks:

Contextual tool tips. Unfortunately, the previous idea loses descriptions of the meaning of the arguments.  I am afraid this is a fundamental tension - the wordier the result, the less likely it is to be useful at a glance; but the terser, the less likely to be informative.  But we can at least add the description in a tool-tip.  So when you hover over the 4 above, a tooltip could pop up saying "horizontal = 4".
Smarter ellipsization.  Instead of ellipsizing the entire string, we could start by ellipsizing each of the arguments separately.  Since this takes less away from the usability, we could be a bit more aggressive at ellipsizing, making the labels shorter (and, therefore, the structure more clear).  When arguments are ellipsized, we can still attempt to show more of the argument in the tool-tip.
Type specific formatting. When rendering a color, we should consider rendering it in the same hue and saturation as the color itself (but not lightness, because it must remain readable).  In tool tips, we can go even further.  The tool tip for a color should show a swatch, as well as the RGBA and HSL values.  The tool tip for a point can show the point on a coordinate plane.  A list of points should probably have separate tool tips for its member points.