[Format Range] Format in a specific range POC

Add application range var to the Formatting Context
Pass TokenSyntax to the PrettyPrinter
PrettyPrinter uses the leading and trailing trivia in case when token is not in applicable range.
Verify its work by having some minimal tests

Previous discussions here: https://forums.swift.org/t/range-formatting/28060
PS. I was thinking it would be a good idea still to make a PR and continue discussions here, potentially improving the PR and eventually merging.

Thanks for getting this started! Apologies for the delay in replying to your latest forum post.
This looks like a reasonable start. However, to call attention to something I mentioned in my latest reply on the thread, I think just inserting the entire TokenSyntax into the formatting token stream won't do the correct thing on its own, because we also treat comments as special kinds of formatting tokens in the pretty printer (we extract them from the trivia of a token). So I think you could end up with comment tokens in the formatting token stream preceding the syntax token whose TokenSyntax has leading trivia that also contains those comments, and depending on how you're processing both of those things, you might end up printing those comments twice if you have the pretty printer print .comment tokens and print all the trivia of the TokenSyntax instance?
So I think instead what we need to do is consider that anywhere that we insert "text" (whether it's syntactic text or a comment), we should also keep track of the whitespace that preceded it. That would mean updating the .comment and .syntax cases to store that information; when we iterate over the trivia in the TokenStreamCreator, we'd have to partition it as we scan it to retain that leading whitespace. It also means we probably have to treat trailing trivia with care, in the handful of cases where it's present.
That may help us make a more informed decision about what whitespace to preserve and what whitespace to discard depending on whether we're inside an application range or not.
cc @dylansturg, who has also been thinking about this problem.

We need support for range formatting to support a mechanism for toggling formatting on chunks of code (e.g. imagine some chunk of code that the author wants to format exclusively by hand). I'm excited to see good progress on this!
I experimented with a similar approach to this one, where the entire TokenSyntax is inserted into the output stream by the PrettyPrinter. I ran into some difficult edge cases trying to suppress the correct non-syntax text (i.e.comments & newlines). I think that Tony's suggestion of slicing the trivia to store the original whitespace on the comment and syntax tokens could avoid those edge cases, and overall be more reliable to implement.
Regarding the implementation in the PrettyPrinter, I think we should continue processing the tokens to record indentation when in a non-formatted section. That will allow the PrettyPrinter to resume using the correct indentation in the next formatted section.

Thank you guys for the comments.
Now as I've written a test that formats a part of a doc comment I have encountered that comment is written twice. This is exactly the issue you guys have been pointing to.
@allevato it took me a while to get what exactly do you mean, but now I get it.
This means .syntax case contains also leadingTriviaTokens array which corresponds to leading trivia represented as [Token] array.
Then in the processTokenAsInOriginalInput we need to drop all the leading trivia tokens which were already printed.
Imagine we want to range format 2 out of 3 line doc comment. This means we will have an array of tokens that contains .comment cases and a .syntax(syntaxToken, leadingTriviaTokens) case. Before printing the text of the syntaxToken we need to print the leading ones, but only the ones which were not already printed
AFAIU this would lead to something like this, where didPrintToken function returns true if the token was already printed and else otherwise. How it will be computed is yet to be defined
  private func processTokenAsInOriginalInput(_ token: Token) {
    if case let .syntax(tokenSyntax, leadingTriviaTokens) = token {
      leadingTriviaTokens
        .drop(while: didPrintToken)
        .forEach(write)

      writeRaw(tokenSyntax.printableText)
      tokenSyntax.trailingTrivia.write(to: &outputBuffer)
    }
  }

  private func didPrintToken(_ token: Token) -> Bool {
    return printedTokens.contains(token)
  }

Did I understand it correctly?

I think we should continue processing the tokens to record indentation when in a non-formatted section. That will allow the PrettyPrinter to resume using the correct indentation in the next formatted section.

@dylansturg this is exactly the problem I am facing right now. What is the "correct indentation"?
Is there a concrete specification of how the range formatting should work in these kinds of edge cases, like formatting only a part of a statement or a part of an expression.
E.g. what should be the result, if we want to format only the line containing function h (line 3)
func f {
func g {
func h {}
}
}

should the result be indented by 1 indentation unit, or 2 (since it's indentation stack going to have 2 units, one for func g and another one for func h?
These kinds of choices obviously have an impact on how range formatting should be implemented. Unfortunately, I didn't find any spec for his.

What is the "correct indentation"?
Is there a concrete specification of how the range formatting should work in these kinds of edge cases, like formatting only a part of a statement or a part of an expression.

Unfortunately, I'm not aware of a written spec with these answers. 😞@allevato might have a more formal definition of what we should do when formatting a block of code that is inside of a file. We should certainly consider the behavior of other related formatters (e.g. clang-format) and expected use cases (e.g. formatting a block from a source editor extension).
I'm not sure if anyone else will agree, but I think referencing clang-format could be helpful. We might want to see if it makes sense to match how clang-format handles indentation in and around formatted ranges.

Imagine we want to range format 2 out of 3 line doc comment. This means we will have an array of tokens that contains .comment cases and a .syntax(syntaxToken, leadingTriviaTokens) case. Before printing the text of the syntaxToken we need to print the leading ones, but only the ones which were not already printed

A couple notes here:

The .comment cases should also keep track of leading whitespace trivia that preceded them.
Your solution still stores the whole TokenSyntax in the .syntax case, but I think that overcomplicates matters.

Imagine the following scenario:
    /// doc comment
    func foo() {
    }

IIRC, the leading trivia for the func keyword looks roughly like this:
[
  .newlines(1), .spaces(4), .newlines(1),
  .docComment("/// doc comment"),
  .newlines(1), .spaces(4),
]

If we use comment pieces as "pivots" (because those show up in the formatting token stream as "text" we care about), we can "partition" that trivia array around them. So, I think we'd want to represent that in the formatting token stream like this:
[
  ...,
  .comment("/// doc comment", leadingTrivia: [.newlines(1), .spaces(4), .newlines(1)]),
  .syntax("func", leadingTrivia: [.newlines(1), .spaces(4)]),
  ...,
]

In other words, as we scan the trivia of a token looking for its comments, we'd also need to build up those lists.
We'd also need to do something about trailing trivia, in the few cases where it occurs. That might just be another associated value in .syntax.
I haven't completely thought this through, but at least if we retained that information into the pretty-printer, we might have what we need to be able to make an informed decision about whether to print what was originally there or to print what the layout algorithm says should be printed, depending on whether we're in a format range or not.
 In some situations we may just have to ignore the exact range and adjust it to something more sensible.

Unfortunately, I'm not aware of a written spec with these answers. 😞@allevato might have a more formal definition of what we should do when formatting a block of code that is inside of a file. We should certainly consider the behavior of other related formatters (e.g. clang-format) and expected use cases (e.g. formatting a block from a source editor extension).

@dylansturg is correct, we don't really have a spec in this case. My recommendation would be to look to other formatters as prior art and consider what they do, but to also consider what makes sense within the constraints of the algorithm we have implemented.

@allevato thank you for your detailed description.
The only piece that is missing from the puzzle is how do other Tokens fit into this design (e.g. .open and .close cases). Are they also in the leading trivia part of the .comment or .syntax cases or do they exist separately in the token stream.
In the example above what we will have is something like this:
[
  ...,
  .comment("/// doc comment", leadingTrivia: [.newlines(1), .spaces(4), .newlines(1)]),
  .open,
  .open,
  .open,
  .syntax("func", leadingTrivia: [.newlines(1), .spaces(4)]),
  ...,
]

Should these open cases be in .syntax case as leading trivia as well?
What about the .spaces case. Sometimes it is in the leading trivia but is ignored by the tokenStreamCreator, yet now it should be in the leadingTrivia and if we put everything the leadingTrivia associated object of .syntax and .comment cases then how we do know when to ingore the cases and when not to?

Hey @denis631 are you still working on this? I was trying to implement this feature in SourceKit-LSP and I saw your pull request on swift-format. Let me know if you are still working on this or not?
@allevato

@rabingaire no I am not atm.
I hope you can benefit from the discussion, but my question written in the previous comment is still opened 🤷♂

The Swift project moved the default branch to main and deleted master branch, so GitHub automatically closed the PR. Please re-create the pull request with main branch.
More detail about the branch update - https://forums.swift.org/t/updating-branch-names/40412


Redefinition of functions with MBEDTLS_PLATFORM_MEMORY and MBEDTLS_PLATFORM_CALLOC_MACRO

Description

Type    : Bug
Priority: Minor

Bug
OS
windows xp sp3, ubuntu
mbed TLS build:
Version : mbedtls-2.9.0-apache.tgz
Compiler: vs2010, gcc4.9
{{{
There is a compile error,
package   : mbedtls-2.9.0-apache.tgz
file      : platform.c
line      : 40
condition : enable MBEDTLS_PLATFORM_MEMORY, MBEDTLS_PLATFORM_CALLOC_MACRO and MBEDTLS_PLATFORM_FREE_MACRO in config.h
current code:
#if defined(MBEDTLS_PLATFORM_MEMORY)
fixed code:
#if defined(MBEDTLS_PLATFORM_MEMORY) &&
(!defined(MBEDTLS_PLATFORM_CALLOC_MACRO) || !defined(MBEDTLS_PLATFORM_FREE_MACRO))
below is output snapshot in vs2010:
------ Build started: Project: mbedTLS, Configuration: Debug Win32 ------
platform.c
e:\work\gitroot\libpro\src\mbedtls\library\platform.c(61): error C2365: 'calloc' : redefinition; previous definition was 'function'
c:\program files\microsoft visual studio 10.0\vc\include\stdlib.h(598) : see declaration of 'calloc'
e:\work\gitroot\libpro\src\mbedtls\library\platform.c(62): error C2365: 'free' : redefinition; previous definition was 'function'
c:\program files\microsoft visual studio 10.0\vc\include\stdlib.h(599) : see declaration of 'free'
e:\work\gitroot\libpro\src\mbedtls\library\platform.c(67): error C2106: '=' : left operand must be l-value
e:\work\gitroot\libpro\src\mbedtls\library\platform.c(67): warning C4550: expression evaluates to a function which is missing an argument list
e:\work\gitroot\libpro\src\mbedtls\library\platform.c(68): error C2106: '=' : left operand must be l-value
e:\work\gitroot\libpro\src\mbedtls\library\platform.c(68): warning C4550: expression evaluates to a function which is missing an argument list
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========
Eric Tung<EMAIL_ADDRESS>}}}

Offhand, I'm not sure the design is supposed to accommodate this configuration. However - we'll look into it, and maybe it needs better documentation or more checks, if it isn't broken as it first appears.
Thanks for the feedback!

In fact, we are using the memory pool.
{{{
#ifndef MBEDTLS_CONFIG_H
#define MBEDTLS_CONFIG_H
#if defined(_MSC_VER) && !defined(_CRT_SECURE_NO_DEPRECATE)
#define _CRT_SECURE_NO_DEPRECATE 1
#endif
#include "../../../pro/pro_util/pro_a.h"
#include <stdlib.h>
void*
PRO_CALLTYPE
ProCalloc(size_t count, size_t size);
void
PRO_CALLTYPE
ProFree(void* p);
#define MBEDTLS_PLATFORM_CALLOC_MACRO ProCalloc // from a pool
#define MBEDTLS_PLATFORM_FREE_MACRO   ProFree   // from a pool
...
}}}

ARM Internal Ref: IOTSSL-2358

from a breif look, this seems like a duplicate of #1642
please confirm

Closing as a duplicate of #882.


An error during creating a C++ project with a name starting with a lowercase letter

During creating a project with a name starting with a lowercase letter I get this error message:

Finally created project has no necessary auto-generated source files (*.cpp).

Hello,
this error is on purpose and should help you to full fill the PLCnext runtime system naming conventions. Your project name has to start with an upper case letter. Otherwise it will not run on the PLC.


cifar10_eval.py Error

python cifar10_eval.py
I tensorflow/stream_executor/dso_loader.cc:108] successfully opened CUDA library libcublas.so locally
I tensorflow/stream_executor/dso_loader.cc:108] successfully opened CUDA library libcudnn.so locally
I tensorflow/stream_executor/dso_loader.cc:108] successfully opened CUDA library libcufft.so locally
I tensorflow/stream_executor/dso_loader.cc:108] successfully opened CUDA library libcuda.so.1 locally
I tensorflow/stream_executor/dso_loader.cc:108] successfully opened CUDA library libcurand.so locally
I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:925] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
I tensorflow/core/common_runtime/gpu/gpu_init.cc:102] Found device 0 with properties:
name: GeForce GTX TITAN X
major: 5 minor: 2 memoryClockRate (GHz) 1.076
pciBusID 0000:01:00.0
Total memory: 12.00GiB
Free memory: 281.36MiB
I tensorflow/core/common_runtime/gpu/gpu_init.cc:126] DMA: 0
I tensorflow/core/common_runtime/gpu/gpu_init.cc:136] 0:   Y
I tensorflow/core/common_runtime/gpu/gpu_device.cc:839] Creating TensorFlow device (/gpu:0) -> (device: 0, name: GeForce GTX TITAN X, pci bus id: 0000:01:00.0)
W tensorflow/core/common_runtime/bfc_allocator.cc:213] Ran out of memory trying to allocate 68.02MiB. The caller indicates that this is not a failure, but may mean that there could be performance gains if more memory is available.

281MB is probably not enough to run CIFAR10.
Could you please run nvidia-smi to see what other binaries might be using up memory?
Sherry

Thanks sherry. Here is what got
+------------------------------------------------------+
| NVIDIA-SMI 352.63     Driver Version: 352.63         |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  GeForce GTX TIT...  Off  | 0000:01:00.0      On |                  N/A |
| 28%   68C    P2    79W / 250W |  11891MiB / 12284MiB |      1%      Default |
+-------------------------------+----------------------+----------------------+
+-----------------------------------------------------------------------------+
| Processes:                                                       GPU Memory |
|  GPU       PID  Type  Process name                               Usage      |
|=============================================================================|
|    0      1189    G   /usr/bin/X                                     178MiB |
|    0      2090    G   compiz                                         123MiB |
|    0      2313    G   /usr/lib/firefox/firefox                         2MiB |
|    0      2586    C   python                                         111MiB |
|    0      2686    C   .../dan/anaconda2/envs/tensorflow/bin/python   111MiB |
|    0     18361    G   itksnap                                          8MiB |
|    0     26483    G   MATLAB                                          63MiB |
|    0     32609    C   python                                       11257MiB |
+-----------------------------------------------------------------------------+
�

Maybe close a few of those applications before trying to run cifar?

after closing, some of the apps works fine but strange error
pciBusID 0000:01:00.0
Total memory: 12.00GiB
Free memory: 11.49GiB
I tensorflow/core/common_runtime/gpu/gpu_init.cc:126] DMA: 0
I tensorflow/core/common_runtime/gpu/gpu_init.cc:136] 0:   Y
I tensorflow/core/common_runtime/gpu/gpu_device.cc:839] Creating TensorFlow device (/gpu:0) -> (device: 0, name: GeForce GTX TITAN X, pci bus id: 0000:01:00.0)
2016-08-16 20:54:12.681518: precision @ 1 = 0.866
E tensorflow/core/client/tensor_c_api.cc:485] Enqueue operation was cancelled
[[Node: input_producer/input_producer_EnqueueMany = QueueEnqueueMany[Tcomponents=[DT_STRING], _class=["loc:@input_producer"], timeout_ms=-1, _device="/job:localhost/replica:0/task:0/cpu:0"](input_producer, input_producer/RandomShuffle)]]�

The observed message is expected when we close the queue. Is eval working for you?

Yes. Thank you


illegal operation on a directory, open './images/'

Mac needs rights to access folder in in application to stream data and write in folder. run "sudo nano npmrc" to add file on mac.

Issue resolved by passing file name instead of directory name.


Attempting to use API Keys

After I uncomment lines 27 and/or line 30 to use the API keys, the script will just tank throwing errors. Only after I comment the lines back out with the script work.

Can you explain a bit more? Which line's file did you commented, what are you running? what's the error? Screenshots always help.


Notion template: This content does not exist

It's not a bug of code. but template.
Describe the bug
On Readme, "this Notion template" link is broken
To Reproduce
Steps to reproduce the behavior:

Click "this Notion template" on readme

Expected behavior
Template shows up
Screenshots
If applicable, add screenshots to help explain your problem.

Platform:

OS: Doesn't related
Browser: Doesn't related
Version: Doesn't related

how to find this notion template? @craigary

Same as your problem .

@craigary Please fix this issue.

Same here :)

"You do not have access to CRAIG's BLOCK. Please contact an admin to add you as a member." how I should become the member: )

Here is the template example:
https://whitesspace.notion.site/cc92abe4c3584e7186b0873bfa41751f?v=5cc83ea98e9845adb1124a99170ef342
@MPThLee @stuchilde @adviks @ggteixeira @SBFPGLN

@designedbyclu thank u

This has been fixed.


Add GridFunction::ProjectDiscCoefficient with averaging for VectorCoefficients [vec-project-avg]

In classes GridFunction and ParGridFunction, add a new projection method, ProjectDiscCoefficient, that works with VectorCoefficients performing averaging over elements.

@Dan2997925, does this work for you?


onMove return false does not work correctly when nesting

As shown in the image below the onMove behaves oddly when you want to disallow some elements from being dragged into the root.
http://jsbin.com/huniluhuhi/edit?html,css,js,output

Hi @Dumpen,
Can you please share the link to the plunker which you made a fix.
Also can you please share the links of samples of nested lists with Sortablejs.
Thanks in advance,
Bhasker

@bhaskerchari Here is a very simple example with the fix applied http://jsbin.com/qaxozolevo/edit?html,css,js,output
I added the "draggable" to the root and child items.

Hi @Dumpen ,
Thanks for the sharing the link it looks like you have used wrong ids in the template. I corrected them and sharing the updated link for reference for someone else.
http://jsbin.com/kihijup/1/edit?html,css,js,output
Thanks,
Bhasker


Validate for valid email address

With this we now validate for valid user email addresses.
Fixes: #55

:+1: Perfect! I'll merge when tests pass

Thanks for the review!


Integrate tremor-query into tremor-tool

Currently tremor-query exists as an custom binary, instead it should be available as part of tremor-tool as tremor-tool query ...

superseded by #200


Detects if biometrics enrolment have changed

Detect changes in biometrics, notifies the user, and disabled biometrics authentication
Acceptance Criteria
Scenario: Change enrolled biometrics
Given the user has previously enabled biometrics
and have changed the enrolled biometrics
When the user opens the app
Then
Notifies user that biometrics have changed
and disables biometrics authentication
and the user us prompted to use app PIN
Wired Frames

User Flow

Can we split the screen lockout into a separate issue? It seems like a distinct change

@knguyenBC In the wireframes, I'm not sure if you want it to replace the default Enter PIN string with Please enter your wallet PIN. in all cases or just when the biometrics enrolment changes. I chose to interpret it as just for the biometrics enrolment, but let me know if you would like it changed for all cases.

Resolved by: https://github.com/hyperledger/aries-mobile-agent-react-native/pull/466

Resolved by: https://github.com/hyperledger/aries-mobile-agent-react-native/pull/466

Please change the default "enter PIN" to "please enter your Wallet PIN"

@Wade please change the default to "Enter your wallet PIN". I mistakenly added the "please" in the previous comment.

Based on the react-native-keychain that we are using, BIOMETRY_CURRENT_SET only works for iOS but not for Android.

@wadeking98 , has there been any update in the underline library by chance?

Clecio and I went through this a while ago. In order to implement this we need to set the access-control type to  BIOMETRY_CURRENT_SET which isn't supported on android. https://github.com/oblador/react-native-keychain#keychainaccess_control-enum

Clecio and I went through this a while ago. To implement this we need to set ACCESS_CONTROL to BIOMETRY_CURRENT_SET but that isn't currently supported in android. Read the note here: https://github.com/oblador/react-native-keychain#keychainaccess_control-enum


Removing flux_install and flux_sync data sources.

Towards #615

Please rebase your fork with upstream, this branch is out-of-date with the base branch.


Add endpoint_status creation script

Add an endpoint status creation script per the moderator meeting. Will be adding a note into the release draft as well.
associated documentation

[x] Give a meaningful name to your PR, as it may end up being used in the release notes.
[x] Your code is flake8 compliant.
[x] Your code is python 3.6 compliant (specific python >3.6 syntax is currently not accepted).
[x] If this is a new feature and not a bug fix, you've included the proper documentation in the ReadTheDocs documentation folder. https://github.com/DefectDojo/Documentation/tree/master/docs or provide feature documentation in the PR.
[ ] Model changes must include the necessary migrations in the dojo/db_migrations folder.
[ ] Add applicable tests to the unit tests.
[x] Add the proper label to categorize your PR.

I suggest to merge it right now so you can add a download link to it in the documentation PR so people can start using already now even before 1.9.0 is there and without the need to clone the repo/dev branch.

Where should I add it in the documentation? In the 1.9.0 release section?

yeah where you added it already. bonus might be to put a warning at 1.8.0 as well that basically they should wait for 1.9.0 or run the script after every import :-)

Done!


Has anyone successfully trained on own dataset?

Has anyone successfully trained on own dataset?Or I can only test it but cannot train?

me too

Hi, I have trained and tested it on my own dataset. Actually, I used SegTrack v2.
And I put my works here: https://github.com/JosieHong/SiamMask_on_Your_Own_Dataset
I hope it is helpful.

Hi, I have trained and tested it on my own dataset. Actually, I used SegTrack v2.
And I put my works here: https://github.com/JosieHong/SiamMask_on_Your_Own_Dataset
I hope it is helpful.

could you please help me to create own dataset ?

hello! I want to on my own dataset. But in ./siammask_sharp/custom
if test:
p0 = torch.nn.functional.pad(f[0], [16, 16, 16, 16])[:, :, 4pos[0]:4pos[0]+61, 4pos[1]:4pos[1]+61]
p1 = torch.nn.functional.pad(f[1], [8, 8, 8, 8])[:, :, 2 * pos[0]:2 * pos[0] + 31, 2 * pos[1]:2 * pos[1] + 31]
p2 = torch.nn.functional.pad(f[2], [4, 4, 4, 4])[:, :, pos[0]:pos[0] + 15, pos[1]:pos[1] + 15]
else:
p0 = F.unfold(f[0], (61, 61), padding=0, stride=4).permute(0, 2, 1).contiguous().view(-1, 64, 61, 61)
if not (pos is None): p0 = torch.index_select(p0, 0, pos)
p1 = F.unfold(f[1], (31, 31), padding=0, stride=2).permute(0, 2, 1).contiguous().view(-1, 256, 31, 31)
if not (pos is None): p1 = torch.index_select(p1, 0, pos)
p2 = F.unfold(f[2], (15, 15), padding=0, stride=1).permute(0, 2, 1).contiguous().view(-1, 512, 15, 15)
if not (pos is None): p2 = torch.index_select(p2, 0, pos)
    if not(pos is None):
        p3 = corr_feature[:, :, pos[0], pos[1]].view(-1, 256, 1, 1)
    else:
        p3 = corr_feature.permute(0, 2, 3, 1).contiguous().view(-1, 256, 1, 1)

    out = self.deconv(p3)
    out = self.post0(F.upsample(self.h2(out) + self.v2(p2), size=(31, 31)))
    out = self.post1(F.upsample(self.h1(out) + self.v1(p1), size=(61, 61)))
    out = self.post2(F.upsample(self.h0(out) + self.v0(p0), size=(127, 127)))
    out = out.view(-1, 127*127)
    return out

p0,p1,p2 are not match p3. The outsize is 2525 for p3 and is 1717 for p0,p1,p2


Implement Digest Auth

CB Lite Listener on iOS uses Digest authentication. See this method in CocoaHTTPServer that CB Lite Listener iOS depends on https://github.com/couchbaselabs/CocoaHTTPServer/blob/3fbeb3faf0d3f6c8d1e9ad050bc46d43fbbffd63/Core/HTTPConnection.m#L375-L390.
Currently, the cb-lite-java-listener implements basic auth https://github.com/couchbase/couchbase-lite-java-listener/blob/master/src/main/java/com/couchbase/lite/listener/LiteServlet.java#L152-L185
Implementing digest auth would enable authed P2P replication between iOS and Android for apps such as Photo Drop and provide security enhancements for PhoneGap apps (see issue https://github.com/couchbase/couchbase-lite-java-listener/issues/48)
Gist that might help to implement digest auth https://gist.github.com/slightfoot/5624590

@hideki What is the latest plan on android? Pls confirm..

Duplicated with https://github.com/couchbase/couchbase-lite-java-listener/issues/56

@hideki is this in android? pls confirm...

@NitzDKoder
Not yet implemented with Android


Fixes #3575 (Flickering colors while upload)

Description (required)
Fixes #3575
What changes did you make and why?

Do not show random colors as placeholders for contributions items
Theme contributionsItem background as mainBackground from attributes

Tests performed (required)
Tested betaDebug on Samsung S7 API 27

Codecov Report

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

@@           Coverage Diff            @@
##             master   #3602   +/-   ##
========================================
  Coverage      6.96%   6.97%
  Complexity      291     291
========================================
  Files           256     256
  Lines         11708   11706    -2
  Branches        954     954
========================================
  Hits            816     816
+ Misses        10821   10819    -2
  Partials         71      71

Impacted Files
Coverage Δ
Complexity Δ

.../commons/contributions/ContributionViewHolder.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

Continue to review full report at Codecov.

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


websocket close codes for code readability

WebSocket Close Code Number Registry

the filename can be 'codes' instead of 'status'

@hamraa  this is awesome but might be better suited as an addition directly to channels or even to https://github.com/django/daphne

Yes! You are right.
They're very useful codes.


Fix Animation Bezier Editor key selection bugs

This bugs fixes a regression introduced with #87250, which resulted in not being able to move multiple keys in Bezier editor at once anymore. It re-adds the accidentally removed delete shortcut and the cut shortcut, which interestingly kept working, as the shortcuts were still covered by the AnimationTrackEditor, but I thought to be safe they should be put back. Finally, this PR also fixes an old bug that snapped keys in the Bezier editor upon selection, which is not desirable, by adding a check if the mouse has been moved or not.

rebased on top of 5f05e2b

rebased on top of b7145638

Thanks!


fix(beta): reset callout headings to H2s

📖 PR App

🧰 Changes
RMDX wraps callout headings in
 tags, but historically, RDMD has marked these up as
s. (Not sure if there was a reason for this change, but it's making it hard to tell which docs will actually be impacted by the migration!)

[x] make RMDX callout headings use an

This PR was released!
🚀 Changes included in v6.75.0-beta.79


Add mongodb support

Checklist

[x] I have read the Contributing Guidelines
[x] I acknowledge that any submitted code will be licensed under the ISC License
[x] I confirm that submitted code is my own work
[x] I have tested the code, and confirm that it works

Enviroment

Operating System: artix linux [Linux 6.5.5-arch1-1-custom x86_64]
Node version: 20.8.0
npm version: 10.1.0
pnpm version: 8.8.0

Description
woo more databases. adds support for mongodb
Issues
currently, the schema for uploaded files expects that the s3 property is always a bool. this should be fine for now, but when those properties start getting used, it might need to be changed to use the mixed type.

[AUTOMATED MESSAGE] 🔴 This Pull Request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

[AUTOMATED MESSAGE] 🔴 This Pull Request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

[AUTOMATED MESSAGE] 🔴 This Pull Request has merge conflicts. Please resolve the conflicts so the PR can be successfully reviewed and merged.

[AUTOMATED MESSAGE] 🟢 Conflicts have been resolved.


Short-captions for Tables

Tables in Markdown do not come with attributes (under the current AST), and there is no way to alter the \caption{} command written by Pandoc for tables in LaTeX.

I was able to modify the behaviour of the caption command within a longtable, so that we can use the functionality of \caption[short]{long}. This was done by replacing one of the \longtable internal commands: \LT@c@ption.
This longtable macro now checks for a macro \pandoctableshortcapt when expanding the caption. It will use this custom macro in place of whatever is in the square brackets in \caption[short]{long}, even if there were no square brackets.
After putting that macro overwrite in the header, the filter parses the caption of a Table.

If it finds a short-caption attribute, it places RawBlocks around the Table to define and undefine \pandoctableshortcapt appropriately.
If it finds a .unlisted class, it places RawBlocks around the Table to define \pandoctableshortcapt as {}. This causes the \LT@c@ption macro to not write an entry to the lof.

I've done a little bit of testing, and it seems to work nicely.
I have no idea how robust it is though, particularly to other changes in the header-includes.

This PR is WIP because I haven't written a Makefile with tests yet.

OK, tests done.
If you think it would be better merged into short-captions, please let me know.

I know very little about pandoc-crossref and LaTeX longtables, but this looks like an excellent idea.
One thing I'm not sure about: parsing the caption string is potentially fragile, and it would be nice if pandoc could handle for us. How about using a special Span to add this information? E.g.,
Table: This is the *italicised long caption* of my table, which has a
very long caption.
[]{#tbl:full-of-juicy-data short-caption="Short caption for *juicy* data table."}

It's two more characters, but one could just check for an empty span within the caption, and then transfer its attributes to the table.

Ah, I see: pandoc-crossref expects {#tbl:table} markers. Maybe using Span just for the short caption would work:
Table: [short caption goes here]{.short-caption} The long caption follows {#tbl:table1}

pandoc-crossref
Yeah, sorry that wasn't particularly clear. The {#tbl:table1} is expected in the caption by pandoc-crossref—and so this filter deliberately sends it back in there.
I use pandoc-crossref with pandoc more often than not (I quite like its [@tbl:crossref] syntax). While I'm not too sure on how it works internally with tables, its net effect is to turn a Table [] into Div (tbl:table1) [Table []] (at least when converting to native?). When converting to LaTeX, it puts a \hypertarget{} right before the table (I believe this is so links go to the top of the table, not to the caption), and inserts \label{#tbl:table1} into the caption.
I don't really want to reimplement what pandoc-crossref does for tables, so I left it to build the Div and hypertarget.
Spans
lightbulb moment YES. Your suggestion of an empty span is completely the way to go here. I've just done some quick testing—I was completely unaware of how Pandoc parsed spans (and didn't think of them). Properties and all! Writing the caption parser felt pretty hacky, this is gonna be so much cleaner!
One of the aims for me was to make the syntax for table short-captions as similar as possible to the figure short-captions in this repo. I still think this is a reasonable aim?
Todo
I'll rewrite the parser to target the following syntax tonight (your original suggestion):
Table: This is the long caption []{#tbl:table1 short-caption="short-caption here" .unlisted}

This should be far less fragile than the current hacky parsing. I'll still put {#tbl:table1} back into the caption for pandoc-crossref.

OK, last commit moved the logic slightly.
Rest of the patches should have resolved your comments.

Great, thank you!


Automated Kubernetes version update 1.23.3 1.22.6 1.21.9 1.20.15

Automated changes by the cron-kubernetes-update GitHub action

This does not appear to actually add 1.23.3, despite being in the title
We probably need manual work for that

1.23 Looks like it's already in the repo. I'm going to guess this PR was open and then at some point it was manually added.


CSV to Oracle DB transfer error: Length too long for VARCHAR

When exporting a CSV to a database, the default mapping datatype for all columns is VARCHAR with length 32767, but this is too long for my version of Oracle leading to a failed data transfer. Changing the size to 4000 lets the transfer proceed. Is there a way to change the default size?
DBeaver 5.1.4
Windows 10, 14393.0

Currently it is not supported. Mark as FR.

See fix in 5.2.2


Document the Audit Commands

Document what the Audit Commands do and how to use them

Closed by #44


Add new InternalAffairs/NodeMatcherDirective cop.

As discussed on discord, this cop looks for node matchers that do not have an appropriate @!method directive.
This is really useful for code editors which support YARD directives, as it allows them to be able to jump to the pattern matcher definition from the usage, as well as introspect what arguments are necessary. Even without one of these editors, it makes arguments explicit in the definition (so it's not a guess what %1, etc. mean in the pattern).
I am going to create PRs for the other gems (especially rubocop-ast) next.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[ ] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Maybe it's because I'm not using YARD's editor jump, it's a bit hard to force this directive...

Not sure it's worth the extra work, but you could use NodePattern#positional_parameters and #named_parameters to build an exact signature, even for the more complex cases...

Unfortunately, I honestly don't feel that way. Most matchers are only used within the defined one cop (that is, within a single file). It's the rule I'm not happy to be able to repeatedly write (and read) comments similar to what is written in the code. So I pay homage to this work, but it's not fun the directive code comment.

So maybe these comments should be added only to the nodepatterns that are "global" (i.e. defined on Node or a subclass)?

I want to demonstrate why this is useful. I understand that the majority of pattern matchers only are used within a single file, but even with that constraint, I am constantly using my editor to jump between usages and definitions of any methods I encounter.
Pattern matchers dynamically define methods, in a way that the static analysis editors do (if any) cannot detect, so the editor does not know that a matcher exists. I am not very familiar with ctags, etc. but I would guess that these methods can't easily be detected by anything generating ctags either, for the same reason (nothing to tell any analysis system that def_node_matcher creates a dynamic method). I don't really see this as the same as type annotation, but I understand your feelings there.
Without annotations:

With annotations:

BTW, I am using YARD but it's already in use on the code, in a lot of places (I found 196 instances of yard comments excluding @example or @!method).  I understand that none of the other ones are forced as this would cop would do, and that's probably a big part of your dislike. I tried to make it so that the cop could auto-generate a good tag itself without needing user input, hoping to alleviate any issue there.

So maybe these comments should be added only to the nodepatterns that are "global" (i.e. defined on Node or a subclass)?

I would rather do this than nothing at all, but I still think it's useful and beneficial to have these for every pattern matcher definition.

Maybe @koic is mostly opposed to the obligation of doing this, so maybe it could be done now on all patterns, and enforced in the future only for rubocop-ast?

For the record - I'm fine with the proposed change, but I won't merge the PR until we address @koic's concerns. I don't mind adding metadata here and there (I used to write Java for a living :D ), but I understand his perspective as well.

I think @marcandre's suggestion is a good compromise. It will likely get out of sync, but we (I?) can fix new ones as they're encountered.
@koic would you be ok with that?

Yeah, I also wrote Java at work until Rails came along. At that time, Java 1.5 annotations and zero configuration helped us escape from XML hell. On the other hand, Rails's CoC and Ruby's DSL have made it unnecessary to write redundant configurations.
I understand that dynamic definitions difficult work well with code analysising, but I'm still concerned about annotation document and code repeats. (This is a tough one that is to solve redundancy using CoC.)
Maybe in the future it may be hoped that growth of RBS and LSP will solve it, but it may be a little further.
Thank you for your consideration and kind comments. Let me think for a while.

@koic great, thanks! Just to clarify, are you find with keeping the IA cop enabled, or do you want it to be disabled by default?

@dvandersluis I'll leave it to you :-) IMO this IA cop can be enabled by default. It can be chosen to be auto-corrected or disabled for each repository that used IA cops.


TLS 1.2

I've created a manifest for mpv, but the site is signed using TLS 1.2 which (at least for me) wasn't enabled by [Net.ServicePointManager]::SecurityProtocol.  Have you had this issue with other packages? If so, how do you handle it?
I got it to work by adding [Net.ServicePointManager]::SecurityProtocol = 'ssl3,tls,tls12' to my Microsoft.PowerShell_profile.ps1 and removing the -noprofile from my bash alias. So I'm curious if this is the right way to do it, and what the motivation behind the -noprofile argument for bash was.  I didn't want to submit a PR to scoop-extras before discussing this.
Thanks

I haven't encountered that issue before. I checked my [Net.ServicePointManager]::SecurityProtocol and it's only ssl3,tls too, so I'm guessing this will be a problem for everyone else too.
The way you're handling it by enabling this in your Powershell profile sounds good to me.
Do you mean -noprofile is being passed when starting a new Powershell session from an alias that's defined in Bash? I'm a bit confused about what's going on there, is that an msysgitbash thing? Or cmder, or console2? Personally I start Powershell using the default Windows console, and without the -noprofile flag, so I haven't encountered this.
To make this a no-brainer install, maybe we could create a package for enable-tls12 that is just a couple of powershell scripts to add and remove tls2 to [Net.ServicePointManager]::SecurityProtocol, both saved in the user's $profile and for the current session.

I have a Bash alias set in ~/.bashrc that's the same as you have defined here.
An enable-tls12 script sounds like a great idea. I'll work on that, and let you know what I come up with.

Oh right, thanks—I've updated that page to not include -noprofile.
Look forward to the PR 👍

I think TLS 1.2 is off by default in .NET because there were websites that break when you use it. (I'm not sure if that's still true in 2016, though.) Maybe the right thing to do is make it an option in the manifest, instead of changing the powershell profile?

Any update on this? I'm really missing a mpv bucket.

TLS 1.2 problems were fixed with f731c1b1b3b1b8a0c12fc4bd0295ee41b8985692 and i just added mpv https://github.com/lukesampson/scoop-extras/commit/de20be22c4e2f24283ea1f16c7a1d78c5c5eb785

Ooh awesome!


Implement delete_model_links API

1. in brief

related issue : https://github.com/ucbrise/clipper/issues/603
Delete the link between a model and an app.

2. Implemented unlink_model_from_app Clipper API in clipper_admin.

Description
unlink_model_from_app(app_name, model_name)

Parameters
app_name : str
The name of the application
model_name : str
The name of the model to link to the application

Raises
:py:exc:clipper.UnconnectedException
versions. All replicas for each version of each model will be stopped.

3. Implemented delete_model_links Management-Frontend API.

API path
POST /admin/delete_model_links

Request schema
const std::string DELETE_MODEL_LINKS_JSON_SCHEMA = R"(
{
"app_name" := string,
"model_names" := [string]
}
)";

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1813/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1818/
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1819/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/1856/
Test PASSed.


Change from travis to github actions

see tableau server client repo for example

https://github.com/tableau/tabcmd/tree/jac/pyinstaller

completed in https://github.com/tableau/tabcmd/commit/dcd5f2c651d5f105acaa8dfa9db27a64a84f95de


error: Error reading 'x\project.json' at line 5 column 26 : Package dependencies must specify a version range.

"HtmlAgilityPack": "1.4.9"
Fails on that line. I am just migrating for .net cli.

I've just updated the tools in Visual Studio and am getting the same error on most of my 'dependencies' rows in the project.json file. What's the fix?

Replace "Regan.Lib": "" by "Regan.Lib": "1.0.0-*"
On Wed, May 25, 2016 at 4:43 PM, Tim Regan<EMAIL_ADDRESS>wrote:

I've just updated the tools in Visual Studio and am getting the same error
on most of my 'dependencies' rows in the project.json file. What's the fix?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/dotnet/core/issues/99#issuecomment-221598328

I close the issue as the fix is my last comment.


add Example project with tests

implemented in: https://github.com/marinofelipe/UICurrencyTextField/pull/7


Disable napari-console button when launched from jupyter

Description
In https://github.com/napari/napari/pull/4240 the console button was disabled for ipython, but not jupyter. As a result, at present, if napari is launched from a jupyter notebook, e.g.:
import napari
viewer = napari.Viewer()

Then the console button is functional, but brings up a empty box—a non-functional console.

As the user is expected (?) to keep using the jupyter notebook, the console button should be disabled. So in this PR, I just add in_jupyter to the check introduced in https://github.com/napari/napari/pull/4240
Type of change

[x] Bug-fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

References

Fixes https://github.com/napari/napari/issues/5211
But sort of? From my look at the docs, the napari-console is not well documented to users, so need to make sure users arn't misunderstanding the console. I've been quite confused by this myself, plus https://github.com/napari/napari/issues/3338 is still a thing and it's really annoying.
I will note, that the napari-console is super powerful and a real game changer—every time I show it to someone it's like 🤯
Maybe also closes https://github.com/napari/napari/issues/687
How has this been tested?

[ ] example: the test suite for my feature covers cases x, y, and z
[x] example: all tests pass with my change locally
[ ] example: I check if my changes works with both PySide and PyQt backends
as there are small differences between the two Qt bindings.

Final checklist:

[x] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If I included new strings, I have used trans. to make them localizable.
For more information see our translations guide.

I don't get whats up with the test at all—linux, pyside6? Seems ambitious...
😢

I don't get whats up with the test at all—linux, pyside6? Seems ambitious...

Just a typical problem with PySide6...
From time to time, they do strange things with this. The version 6.4.0 of PySide6 and shiboken6  was released 5 hours ago.
But it looks like they repeat an old error:
I think that there is a need to block it here https://github.com/napari/napari/blob/main/tox.ini#L78
@tlambert03 do you have any idea how to handle this?

TLDR: code changes look good and I think we should probably merge this.

As the user is expected (?) to keep using the jupyter notebook, the console button should be disabled.

But I'm also a little unsure about the expected behavior here.
In the implementation of napari-console, we can see that there is no support for connecting the napari console and an iPython shell launched from a terminal.
But there is an attempt to connect the napari console to an interactive shell launched from a jupyter notebook.
On main (at cef660e74e7425a2b298262bd5695fa0927d36f2) on an Intel mac, this works - executing a command in either the notebook or in the napari console updates the shared state. I only tried a few basic operations though.
But if there are environments where this doesn't work (which seems to be the case), then I think we should probably disable this as you've done here.
More generally, my feeling is that if you're launching napari from iPython or a jupyter notebook (or some other interactive shell or repl), you probably don't want the napari console. I think is especially true for a notebook, where you might expect that the notebook maintains some kind of record of the executed code. But I'd be curious about usage and opinions from people who are actually using napari from notebooks.

I don't get whats up with the test at all—linux, pyside6? Seems ambitious...

I think if you merge main and push, that test should pass now after #5244 was merged.

On main (at cef660e) on an Intel mac, this works - executing a command in either the notebook or in the napari console updates the shared state. I only tried a few basic operations though.

Interesting, the OP of the issue https://github.com/napari/napari/issues/5211 also has x86 macOS 12.6
On my arm64 I can reproduce the issue with jupyter qtconsole as well (with freshly pulled main):
Jupyter QtConsole 5.3.2
Python 3.10.6 | packaged by conda-forge | (main, Aug 22 2022, 20:41:22) [Clang 13.0.1 ]
Type 'copyright', 'credits' or 'license' for more information
IPython 8.5.0 -- An enhanced Interactive Python. Type '?' for help.

import napari
viewer = napari.Viewer()

We're both running python 3.10 though. Maybe it's that? let me check.

On python 3.9 I get an error when opening the console after launching from jupyter qtconsole.
ERROR:tornado.general:Uncaught exception in ZMQStream callback
Traceback (most recent call last):
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/site-packages/zmq/eventloop/zmqstream.py", line 557, in _run_callback
    callback(*args, **kwargs)
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/site-packages/jupyter_client/threaded.py", line 123, in _handle_recv
    msg_list = self.ioloop._asyncio_event_loop.run_until_complete(get_msg(future_msg))
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/asyncio/base_events.py", line 623, in run_until_complete
    self._check_running()
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/asyncio/base_events.py", line 585, in _check_running
    raise RuntimeError(
RuntimeError: Cannot run the event loop while another loop is running
ERROR:tornado.general:Uncaught exception in zmqstream callback
Traceback (most recent call last):
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/site-packages/zmq/eventloop/zmqstream.py", line 578, in _handle_events
    self._handle_recv()
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/site-packages/zmq/eventloop/zmqstream.py", line 607, in _handle_recv
    self._run_callback(callback, msg)
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/site-packages/zmq/eventloop/zmqstream.py", line 557, in _run_callback
    callback(*args, **kwargs)
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/site-packages/jupyter_client/threaded.py", line 123, in _handle_recv
    msg_list = self.ioloop._asyncio_event_loop.run_until_complete(get_msg(future_msg))
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/asyncio/base_events.py", line 623, in run_until_complete
    self._check_running()
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/asyncio/base_events.py", line 585, in _check_running
    raise RuntimeError(
RuntimeError: Cannot run the event loop while another loop is running
ERROR:tornado.general:Uncaught exception in ZMQStream callback
Traceback (most recent call last):
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/site-packages/zmq/eventloop/zmqstream.py", line 557, in _run_callback
    callback(*args, **kwargs)
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/site-packages/jupyter_client/threaded.py", line 123, in _handle_recv
    msg_list = self.ioloop._asyncio_event_loop.run_until_complete(get_msg(future_msg))
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/asyncio/base_events.py", line 623, in run_until_complete
    self._check_running()
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/asyncio/base_events.py", line 585, in _check_running
    raise RuntimeError(
RuntimeError: Cannot run the event loop while another loop is running
ERROR:tornado.general:Uncaught exception in zmqstream callback
Traceback (most recent call last):
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/site-packages/zmq/eventloop/zmqstream.py", line 578, in _handle_events
    self._handle_recv()
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/site-packages/zmq/eventloop/zmqstream.py", line 607, in _handle_recv
    self._run_callback(callback, msg)
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/site-packages/zmq/eventloop/zmqstream.py", line 557, in _run_callback
    callback(*args, **kwargs)
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/site-packages/jupyter_client/threaded.py", line 123, in _handle_recv
    msg_list = self.ioloop._asyncio_event_loop.run_until_complete(get_msg(future_msg))
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/asyncio/base_events.py", line 623, in run_until_complete
    self._check_running()
  File "/Users/piotrsobolewski/Dev/miniforge3/envs/apple-TF210/lib/python3.9/asyncio/base_events.py", line 585, in _check_running
    raise RuntimeError(
RuntimeError: Cannot run the event loop while another loop is running

No errors if using a jupyter notebook, but same effect of a blank console.

I don't get whats up with the test at all—linux, pyside6? Seems ambitious...

I think if you merge main and push, that test should pass now after #5244 was merged.

@andy-sweet Sorry to be ignorant, but just want to be sure:
to do this I pull main, then checkout my PR branch, then git merge main and that's it? Or is this where I need to rebase? Still getting the hang of all this.
❤

@andy-sweet Sorry to be ignorant, but just want to be sure:
to do this I pull main, then checkout my PR branch, then git merge main and that's it? Or is this where I need to rebase? Still getting the hang of all this.

That's what I do.
The merge and rebase commands are two different ways of integrating changes from one branch to another. Merge only adds commits (including special merge commits) to a branch, while rebase rewrites commits. I always find the Atlassian git guide to be pretty great and it covers this topic too.
When using github's squash and merge, I generally don't really find rebase useful and I'm not bothered by merge commits, so I tend to avoid it. But there likely are some cases when it would be appropriate. And I think some people generally prefer it.

Holy cow, that was really helpful. ❤

@psobolewskiPhD : feel free to merge this when you want.
There might be two follow-up issues worth writing up, both outside of this repo.

Clarify the intended behavior (i.e. the console is disabled when launching from iPython or a Jupyter notebook) in the napari.org docs. The getting started page seems like a good place to do this.
Consider removing the connection to the jupyter notebook Python kernel in the napari-console plugin. I don't entirely understand this code, but I suspect the intent is to allow the console in napari to share the Python kernel started by Jupyter so that definitions made in the console can be used in the notebook (which doesn't seem particularly useful to me).


Explore noise addition to inputs or data aug.

which noise model is the better in our case
add noise to the input directly OR include it in the data augmentation?

Stale / no longer done by this repo


Java SCORE review for existing Codebase and Testing

What needs to be done
Review and Test the existing codebase for the xcall service already existing in the BTP repo for feasibility and identify the update required
Why it needs to be done
Identify the usability of the codebase and code changes required.
Tests
Link associated tests here
Stories
Link associated stories here
Acceptance Criteria
Describe how we can know whether the task is done.
Ex:

When restarting the relay, synchronization picks up from the most recently stored block

@CyrusVorwald-ICON  Current implementation of iconbridge contracts requires service handler to be registered on bmc contract. There is no provision to do such in case of xCall implementation. Since any contract can execute xCall's interface, this opens the possibility of an arbitrary contract from making calls to a costly destination (draining the relay fund in process). To avoid such DOS attack type problem, proper fee handling mechanism, one based on destination chain and/or routing involved, is required
This routing dependent fee handling is what has recently been added to BMC contract and is a function of BMC alone and not bound to xCall's exposed methods.
If we choose to develop xCall without new bmc-fee-mechanism, there won't be changes to estimated xCall timeline.
Else if we find it necessary, the timeline will have to be updated accordingly.
Note: the problem is present in current implementation of iconbridge as well. Though blacklisting restrictions could be used as a safeguard, a user could use multiple different addresses to avoid blacklist.

bmc-fee-mechanism can be added when it's ready. We're transitioning to IBC anyways, so let's table all this for now and do that instead.


Bump zigpy

Zigpy 0.53 introduces big-endian types, which will help us simplify some of the code that reimplements them.

Codecov Report
Base: 83.69% // Head: 83.69% // No change to project coverage :thumbsup:

Coverage data is based on head (34a5f9c) compared to base (11d693e).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##              dev    #2167   +/-   ##
=======================================
  Coverage   83.69%   83.69%
=======================================
  Files         257      257
  Lines        8225     8225
=======================================
  Hits         6884     6884
  Misses       1341     1341

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 83.696%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6884

Relevant Lines:
8225

💛  - Coveralls


Comic-book reader

WIP...

PR Checklist

Please check all that apply to this PR using "x":

[x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[ ] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

Fixes
Issue Number: 1377
What is the current behavior?
There is no viewer for comic book archives
What is the new behavior?
A simple viewer for comic book archives (CBR, CBT, CBZ, CB7)

I'm not sure if I should be adding this files inside the static folder:
https://github.com/lbryio/lbry-desktop/tree/comicbook-reader/static/webworkers

ES module and webworker bundle, ES module part is your interface to talk to library, use it like any other module, webworker bundle lives in libarchive.js/dist folder so you need to make sure that it is available in your public folder since it will not get bundled if you're using bundler (it's all bundled up already) and specify correct path to Archive.init() method

https://github.com/nika-begiashvili/libarchivejs#how-to-use

There still some work to do on the repo side: https://github.com/btzr-io/Villain

The viewer should also work on the website, but I'm not sure how to enable it yet :stuck_out_tongue:

This is awesome! We still have to make some changes to handle viewing any non audio/video content on lbry.tv.
Why does this need a webworker?

@seanyesmunt The worker is required by libarchivejs and it's for the extraction process (decompression of the archive)
https://github.com/btzr-io/Villain#webworker

@kauffj Do you know if could be any issues with adding this? https://github.com/nika-begiashvili/libarchivejs

@btzr-io I think this is fine. Can you just add a comment above the static scripts that explains why they are there?

I only took a quick glance, but I don't see why we couldn't add libarchivejs. It's worth considering: 1) whether archive support should be added to the SDK and 2) how this will work on the web, but neither are blocking considerations.

@seanyesmunt  in this file ?


feat: added storage wrapper to enable encryption of local data

Description
image of encrypted data in localStorage:

The flow of this is now:

Client provides a password
If the app starts and there is no masterKey wrapped in storage then this code generates one, wraps it, and stores it.
If the app starts and tere is a masterKey the app unwraps it from storage.
The app uses this key so app restarts will not affect the key

Verification

clone this branch locally
clone this branch: https://github.com/StephenCoady/ionic-showcase/tree/add-encryption of the showcase application and link the sdk to there
provide an encryptionPassword in the config of the app
npm run start the showcase application and the storage should resemble that of the screen shot above.
make some offline changes and they should also be encrypted
return online and the changes should be decrypted and synced correctly

An example of they key to provide from the app is:
and then pass this object under DataSyncConfig as encryptionKey
Checklist

[ ] npm test passes
[ ] npm run build works
[ ] tests are included
[ ] documentation is changed or added

This perfect! Really good. Before we merge I will check how this works on mobile in terms of perf and support

Coverals failing. Maybe we can unit test that

We probably should consider using web crypto API instead of using the CryptoJS. There are a few benefits of use it:

It is a webstandard and looks like supported by many browsers.
We don't need to include another lib.
The APIs are async and it can improve performance.

@wtrocki @wei-lee I have swapped this to use the web crypto api now.
If you guys get a chance could you take a look please. I know this won't be merged here but I am pushing here as it is easier to review from a PR. If you are both happy with this I will push it to a separate package.
@wei-lee I know we spoke about wrapping the key so that it persisted. I implemented this but the unwrapping was very tricky and I got a lot of errors. In the end I decided it wasn't worth the work to implement so I abandoned it. In its current form the user will need to pass a jwk and then everything should work fine. If you are interested in the wrapping code I wrote its here: https://gist.github.com/StephenCoady/f815bc00f8167c58e756412313998e64

The tests are failing because I have left in a console.log.
If this is ok @wei-lee I will extract it to a cleaner module with a few files to make it easier to read and push somewhere. I will add some unit tests then as well.
If you want to try it out I have pushed to the showcase branch listed in the verification steps above.

@StephenCoady I have left some comments. Let's move it out and add some unit tests for it.

moved this to a package here: https://www.npmjs.com/package/web-encrypted-storage
if we want to I can move the repo under aerogear. I will create a PR to update the docs then if we do.


catkin_make expects cmake_minimum_required 2.8.3 for metapackages

It seems that catkin_make always expects cmake_minimum_required(VERSION 2.8.3) for metapackages.
It checks that the metapackage CMakeLists.txt conforms exactly to the expected format here and the build fails if it doesn't. The format has a fixed cmake version that seems to come from here.
What is the rationale for imposing a fixed version? If someone wants to force a higher minimum version for certain packages then they won't be able to do the same for the metapackage containing those packages.
I see that recently the minimum version was bumped for a number of packages and metapackages. (https://github.com/ros/catkin/pull/1052, https://github.com/ros/ros_comm/pull/1869 and other PRs). All these will probably build with with catkin_make_isolated, but I think they will no longer build with catkin_make.

Seems like the check will need to be relaxed similar to https://github.com/ros-infrastructure/catkin_pkg/pull/274

Thanks for the report. Addressed by #1065.

@toliver wrote:

If someone wants to force a higher minimum version for certain packages then they won't be able to do the same for the metapackage containing those packages.

this sentence is strange: metapackages do not contain any other packages, and the contents of the CMakeLists.txt is fixed. Metapackages don't do anything.
I'm not saying it was wrong to relax the requirement, but there doesn't seem to be any adverse affects from stipulating a specific minimum version for metapackages.

there don't seem to be any adverse effects from stipulating a specific minimum version for metapackages.

Yes, there is. If the minimum version is 2.8.3 as it was required before this change newer CMake versions print a CMP0048 warning.

I would hardly call that an "adverse effect", but ok.
My point was that the OP implies this restriction being a serious one. Seeing as metapackages don't actually install anything, don't contain any CMake logic other than the prescribed content, don't "contain" other packages and don't have any way to enforce anything in other packages it really doesn't matter. Apart from an annoying warning from CMake when they are built.

@gavanderhoorn you are right that a metapackage doesn't contain other packages, but it does list or group other packages (not sure what the correct term would be).
Even if it is not necessary to do so, it seems that many developers think it is sensible to bump the cmake_minimum_required version of a metapackage when they bump the version of all the packages listed in that metapackage.
That has happened in ros_comm, moveit and others. After such changes the repository would fail to build with catkin_make.
I think important repos failing to build with catkin_make can be a serious problem for some people (like myself), which is why I posted this issue. And I'm glad it's been fixed. Thanks.


Feature Request - Z Tweak

While printing, I put down 4x perimeters and use that to tweak the z position (for smooshed filament).  It would be ideal if Repetier had a way to fine-tune (+/-0.025mm) z during the print -- at least during the first layer.  You could then store that in "First Layer Height".
It would be even better if you could send the initial perimeters as short commands (eg, each move is < 1 mm long) so you could adjust it in real time on the perimeters, but that might be asking too much.

what about a way to tweak bed and hotend temperature at z height plugin ?


Cannot use brand with url or email

The code below is invalid
const UrlSymbol: unique symbol = Symbol('Url')
const Url = brand(url(), UrlSymbol)

Argument of type 'UrlValidation<string>' is not assignable to parameter of type 'BaseSchema<any, any> | BaseSchemaAsync<any, any>'.
  Type 'UrlValidation<string>' is not assignable to type 'BaseSchema<any, any>'.
    The types returned by '_parse(...)' are incompatible between these types.
      Type 'PipeActionResult<string>' is not assignable to type 'SchemaResult<any>'.
        Type 'InvalidActionResult' is not assignable to type 'SchemaResult<any>'.
          Property 'typed' is missing in type 'InvalidActionResult' but required in type 'TypedSchemaResult<any>'.(2345)

Is it possible to support url and email?

Oh I misused the API, it works with
const UrlSymbol: unique symbol = Symbol('Url')
const Url = brand(string([url()]), UrlSymbol)


Add WS subscriptions, thread "unsubscribe" from FE to BE

It should be really easy for the FE to subscribe to "topics", and easy for the backend to send those updates to each FE

Add WS subscriptions, thread "unsubscribe" from FE to BE

Close because there hasn't been a request from the community -- we will start by supporting documenting websockets in Fern's documentation product.


Could not render this component, see the console. O is undefined.

I see the following error in OAS3
Name | Description
id *integer(path) |
Could not render this component, see the console.
Could not render this component, see the console.
The console:
TypeError: "O is undefined"
    value https://localhost/DonServices/api/swagger/swagger-ui-bundle.js:82
    render https://localhost/DonServices/api/swagger/swagger-ui-bundle.js:70
    _renderValidatedComponentWithoutOwnerOrContext https://localhost/DonServices/api/swagger/swagger-ui-bundle.js:70
    _renderValidatedComponent https://localhost/DonServices/api/swagger/swagger-ui-bundle.js:70
    performInitialMount https://localhost/DonServices/api/swagger/swagger-ui-bundle.js:70
    mountComponent https://localhost/DonServices/api/swagger/swagger-ui-bundle.js:70
    mountComponent https://localhost/DonServices/api/swagger/swagger-ui-bundle.js:13
    mountChildren https://localhost/DonServices/api/swagger/swagger-ui-bundle.js:70
    _createInitialChildren https://localhost/DonServices/api/swagger/swagger-ui-bundle.js:70
    mountComponent https://localhost/DonServices/api/swagger/swagger-ui-bundle.js:70
swagger-ui-bundle.js:70:61175

It works in Swagger2 though.

Can you provide a sample spec/project?

It starts like this:
  "x-generator": "NSwag v<IP_ADDRESS> (NJsonSchema v<IP_ADDRESS> (Newtonsoft.Json v<IP_ADDRESS>))",
  "openapi": "3.0.0",
  "info": {
    "title": "My API",
    "description": "My API",
    "version": "1.0.0"
  },
  "servers": [
    {
      "url": "https://localhost/DonServices/api"
    }
  ],
  "paths": {
    "/v2.0/alerts/{id}": {
      "get": {
        "tags": [
          "Alert"
        ],
        "operationId": "Alert_GetAlert",
        "parameters": [
          {
            "name": "id",
            "in": "path",
            "required": true,
            "schema": {
              "type": "integer",
              "format": "int32"
            },
            "x-position": 1
          },
          {
            "type": "string",
            "name": "Auth-Provider",
            "in": "header",
            "default": "Custom1",
            "x-enumNames": [
              "Custom1",
              "Custom2"
            ]
          },
          {
            "type": "string",
            "name": "Authorization",
            "in": "header",
            "required": true,
            "description": "auth access token"
          }
        ],

Having the same issue:
services.AddSwaggerDocument(document =>
{
    document.PostProcess = d =>
     {
         d.SchemaType = NJsonSchema.SchemaType.OpenApi3;
     };
};

Removing d.SchemaType = NJsonSchema.SchemaType.OpenApi3; fixes it for now, but it used to work.

Can someone provide a C# ASP.NET Core sample controller where we can reproduce this?


feat: Port Data.Nat.Periodic

[ ] depends on: #2209

This PR/issue depends on:

leanprover-community/mathlib4#2209
By Dependent Issues (🤖). Happy coding!

This PR/issue depends on:

leanprover-community/mathlib4#2209
By Dependent Issues (🤖). Happy coding!


I cant view my list anymore in the separate window, ive tried shutting insomnia down and restarting but no luck

I'm new to coding, I'm in a bootcamp currently working on one of my final projects, I'm trying to make a list of cars but I can't see the list in the normal window and I have no idea how to get it back. Like i said I'm new to everything especially insomnia, any help would be much appreciated.

Thanks for the report @gatzjames seems this is an unexpected UX issue.
@keithrickett0 If you place your cursor along the right edge of the window you should see that it becomes a horizontal resize icon. You can reopen the response pane by dragging to the left.


PageModel Class has worng name

According to the text the page file name is "Pages/Index2.cshtml" but in the code it was written "public class IndexModel2 : PageModel". So changed it to "public class Index2Model : PageModel".

see #5468


feat: Add support for allowed_instance_types on self-managed nodegroup ASG

Description
As started in https://github.com/terraform-aws-modules/terraform-aws-eks/pull/2552, this PR will add the allowed_instance_types option but on the self-managed ASG this time.
Motivation and Context
This option could be configured on the Launch Template or the ASG itself but it's easier to see it in AWS Console when it's managed at the ASG level and not buried on the LT (when the situation permits it). It also follow the existing module convention to work with instance attributes scaling.
Breaking Changes
None
How Has This Been Tested?

[X] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[X] I have tested it within our own EKS clusters

This PR is included in version 19.17.0 :tada:


Remove constructor for FileSource

FileSource currently has a constructor which checks whether a path exists before returning a FileSource. This however is unnecessary because each FileSource method has failure logic (through kubectl) for when paths don't exist.
Since FileSource is of type string, creation can be accomplished through conversion.

FileInput also doesn't need a constuctor for similar reasons

Actually it would be nice to catch non existent paths a little earlier, I am adding a path existence check to FileInput.


[Enhancement] Optimize diffRange method in SyncPartitionUtils

What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Problem Summary(Required) ：

Optimize diffRange method in SyncPartitionUtils;
Add a benchmark to test this optimization;

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 3.0
[x] 2.5
[ ] 2.4
[ ] 2.3

[FE PR Coverage Check]
:heart_eyes: pass : 6 / 7 (85.71%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/common/util/RangeUtils.java
3
4
75.00%
[53]

:large_blue_circle:
com/starrocks/sql/common/SyncPartitionUtils.java
3
3
100.00%
[]

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.5


Version Packages (next)

See docs/releases/v1.9.0-next.0-changelog.md for more information.

wooooo :tada:


Stuck on gulp build

When I run gulp build it just gets stuck. Last message I see is build Creating distribution directories.
Just a thought - it would be great if there was a prepared dist/ directory with latest build in the repo.

Same question - could you please include dist folder with the build?


API Specifications Weekly - March 28th, 2022

This is a weekly public meeting to discuss the top API specifications, including, but not limited to OpenAPI, AsyncAPI, JSON Schema, GraphQL, gRPC, and Websockets. Discuss ideas on how we can better cross-pollinate information across the communities, develop common extensions, as well as tooling, collections, and other resources that benefit the community.
Agenda:

Updates - Doc provides the updates across specs from the week.

Automating Community Count - Automated the pulling of weekly Twitter and Github numbers, will be doing Stack Overflow.
Twitter Spaces - Use Twitter Spaces as a place to ask questions about spec usage and adoption.
How to Add items to API Specification Weekly Report - Documenting the process.

OpenAPI - Review the projects currently occurring as part of OAI operations, the future of the marketing meeting, and any other relevant items.

Marketing (Repo)
Gluecon - Conference and booth.
Tooling (Repo) (Kanban)
Nordic API Tooling
Readme.io OpenAPI compatibility chart
Compliance Checklist - How can we move this forward as part of tooling work?
Education Resources - Phil Sturgeon and Jack Skinner working on OpenAPI.dev.

AsyncAPI - Talk about how we can better showcase what they are doing, follow their lead, and energize other areas based upon what is happening.

3.0 - The next version is heavily being worked upon.

JSON Schema - Understand how we invest more in JSON Schema, but also vocabularies for AsyncAPI and OpenAPI, but also forms and others.

Hiring - Community, developer, and wider.

GraphQL - Doc will be full-time on GraphQL beginning next week
API Spec Toolbox - Keep this in the conversation so we can make sure and use it as the central gathering for this work and discussion.

Workspace - Where automation for the space is happening.
Community Rep - Managing this meeting with organization now.

Workshops - Produce 101 series curriculum for training and workshops.

OpenAPI, AsyncAPI, JSON Schema, GraphQL, gRPC
AsyncAPI Training - They are building training and curriculum.

ASC - The API Specifications Conference, event, sponsorship, program.

Event Description
Program
Sponsorship

Please add any agenda items you’d like to discuss below, and we will meet weekly, and push forward projects, stories, collections, and tools to support our work.
Meeting:

March 28th, 2022 - 8:00 AM PST
Zoom

Calendar - New OpenAPI, but can we update Frank's work -<EMAIL_ADDRESS>

Community Repo for the OAI.


Improve Library Versions Dependabot

What does this PR do?
It improves issue for unsupported libraries by grouping different versions of the same library.
Feature provided in this PR collects all comments from the issue that contains unsupported libraries (that jobs from workflows matrix added) and groups them by library name.
The final issue should look like this:

I'm afraid I won't have time to review this PR this week.


Issues relating to using ann_file_train for Kinetics400 dataset

Hi,
When I train the model, I face the following issues either use "kinetics400_train_list_videos.txt" or "kinetics400_train_list_videos.js
on" :
1)txt file error:
Traceback (most recent call last):
File "/home/hk20/anaconda3/envs/vfs/lib/python3.7/site-packages/mmcv/utils/registry.py", line 51, in build_from_cfg
return obj_cls(**args)
File "/home/hk20/VFS/mmaction/datasets/video_dataset.py", line 43, in init
super().init(ann_file, pipeline, start_index=start_index, **kwargs)
File "/home/hk20/VFS/mmaction/datasets/base.py", line 67, in init
self.video_infos = self.load_annotations()
File "/home/hk20/VFS/mmaction/datasets/video_dataset.py", line 61, in load_annotations
filename, label = line_split
ValueError: not enough values to unpack (expected 2, got 1)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "./tools/train.py", line 181, in
main()
File "./tools/train.py", line 156, in main
datasets = [build_dataset(cfg.data.train)]
File "/home/hk20/VFS/mmaction/datasets/builder.py", line 37, in build_dataset
build_dataset(cfg['dataset'], default_args), cfg['times'])
File "/home/hk20/VFS/mmaction/datasets/builder.py", line 39, in build_dataset
dataset = build_from_cfg(cfg, DATASETS, default_args)
File "/home/hk20/anaconda3/envs/vfs/lib/python3.7/site-packages/mmcv/utils/registry.py", line 54, in build_from_cfg
raise type(e)(f'{obj_cls.name}: {e}')
ValueError: VideoDataset: not enough values to unpack (expected 2, got 1)
Killing subprocess 2017741
Traceback (most recent call last):
File "/home/hk20/anaconda3/envs/vfs/lib/python3.7/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/home/hk20/anaconda3/envs/vfs/lib/python3.7/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/hk20/anaconda3/envs/vfs/lib/python3.7/site-packages/torch/distributed/launch.py", line 340, in
main()
File "/home/hk20/anaconda3/envs/vfs/lib/python3.7/site-packages/torch/distributed/launch.py", line 326, in main
sigkill_handler(signal.SIGTERM, None)  # not coming back
File "/home/hk20/anaconda3/envs/vfs/lib/python3.7/site-packages/torch/distributed/launch.py", line 301, in sigkill_handler
raise subprocess.CalledProcessError(returncode=last_return_code, cmd=cmd)
subprocess.CalledProcessError: Command '['/home/hk20/anaconda3/envs/vfs/bin/python', '-u', './tools/train.py', '--local_rank=0', 'configs/r18_nc_sgd_cos_100e_r2_1xNx8_k400.py', '--launcher', 'pytorch']' returned non-zero exit status 1.
2)json file error
Traceback (most recent call last):
File "/home/hk20/anaconda3/envs/vfs/lib/python3.7/site-packages/mmcv/utils/registry.py", line 51, in build_from_cfg
return obj_cls(**args)
File "/home/hk20/VFS/mmaction/datasets/video_dataset.py", line 43, in init
super().init(ann_file, pipeline, start_index=start_index, **kwargs)
File "/home/hk20/VFS/mmaction/datasets/base.py", line 67, in init
self.video_infos = self.load_annotations()
File "/home/hk20/VFS/mmaction/datasets/video_dataset.py", line 48, in load_annotations
return self.load_json_annotations()
File "/home/hk20/VFS/mmaction/datasets/base.py", line 80, in load_json_annotations
path_key = 'frame_dir' if 'frame_dir' in video_infos[0] else 'filename'
KeyError: 0
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "./tools/train.py", line 181, in
main()
File "./tools/train.py", line 156, in main
datasets = [build_dataset(cfg.data.train)]
File "/home/hk20/VFS/mmaction/datasets/builder.py", line 37, in build_dataset
build_dataset(cfg['dataset'], default_args), cfg['times'])
File "/home/hk20/VFS/mmaction/datasets/builder.py", line 39, in build_dataset
dataset = build_from_cfg(cfg, DATASETS, default_args)
File "/home/hk20/anaconda3/envs/vfs/lib/python3.7/site-packages/mmcv/utils/registry.py", line 54, in build_from_cfg
raise type(e)(f'{obj_cls.name}: {e}')
KeyError: 'VideoDataset: 0'
Killing subprocess 2017835
Traceback (most recent call last):
File "/home/hk20/anaconda3/envs/vfs/lib/python3.7/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/home/hk20/anaconda3/envs/vfs/lib/python3.7/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/hk20/anaconda3/envs/vfs/lib/python3.7/site-packages/torch/distributed/launch.py", line 340, in
main()
File "/home/hk20/anaconda3/envs/vfs/lib/python3.7/site-packages/torch/distributed/launch.py", line 326, in main
sigkill_handler(signal.SIGTERM, None)  # not coming back
File "/home/hk20/anaconda3/envs/vfs/lib/python3.7/site-packages/torch/distributed/launch.py", line 301, in sigkill_handler
raise subprocess.CalledProcessError(returncode=last_return_code, cmd=cmd)
subprocess.CalledProcessError: Command '['/home/hk20/anaconda3/envs/vfs/bin/python', '-u', './tools/train.py', '--local_rank=0', 'configs/r18_nc_sgd_cos_100e_r2_1xNx8_k400.py', '--launcher', 'pytorch']' returned non-zero exit status 1.
I also attached the txt file that I am using, any idea how to solve this issue?
kinetics400_train_list_videos.txt

Hi @hamidehkerdegari
I have attached my file for your reference
kinetics400_train_list_videos.txt
.

Thanks @xvjiarui. This solved my issue.


ng-if doesn't parse typeof right

ng-if doesn't parse typeof statements correctly
error:
Syntax Error: Token 'e' is an unexpected token at column 8 of the expression [typeof e != 'boolean'] starting at [e != 'boolean'].
Code:

  Yes

  No

where the entries array is defined as [0, 1, 2, 3, true, 5, 6, 7, 8, 9]

ngIf evaluates Angular expressions (not JavaScript expressions) against the current Scope (not the global context, e.g. window). Things like typeof are not available on the Scope (nor are they supported in Angular expressions), thus you get an error.
You proper way to do it, is define a function on the Scope and then call is from the view:

  {{ $index + 1 }}. {{ item }}

// Inside the controller
$scope.isBoolean = function (value) {
  return typeof value === 'boolean';
};

Demo pen
--
That said, this is rather a general support question and as such should have been directed to the appropriate channels. GitHub is reserved for bug reports and feature requests.


Basic drag not working

How can I re-enable the basic drag to show more detail of the card?

Confirm, I think isn't working


Update Docs 404

          I know this is techincally correct, but this sentence reads weird to me. "We couldn't find what you're looking for so go here to find what you're looking for" . Seems odd. NOT a blocker, but maybe we can revisit the language? idk

Originally posted by @avatus in https://github.com/gravitational/docs/pull/225#discussion_r1087090643
Also:
Assuming we could wrap this content with the standard page layout component (which we should be doing), we could change it to something like this:
      We're very sorry - we could not find the page you were looking for.
      Please navigate to the Teleport <Link href="/docs/">
        Documentation
      </Link>{" "}
      or the Home Page, or try our search bar.

Otherwise:
      We're very sorry - we could not find this page.
      Try  the Teleport <Link href="/docs/">Documentation
      </Link>{" "}
      or the Home Page to find what you're looking for.

Originally posted by @alexfornuto in https://github.com/gravitational/docs/pull/225#discussion_r1087124256

Looks like this is complete (at least in docs/pages/404.tsx):
// 404.js
import Link from "next/link";
export default function FourOhFour() {
return (
<>
404 - Page Not Found
We're very sorry - we could not find the page you were looking for.
Please navigate to the Teleport
Documentation
{" "}
or the Home Page to find what you're looking for.
</>
);
}

IMHO I wouldn't consider this done until the 404 page included the basic page layout, so that users would have more navigation options, and make the page look less like a site under construction.
Prior art:

Compare to our current:

Looks like this will be addressed in the move to Docusaurus. e.g.


Drop D50 Luv making Luv now refer to D65 Luv

Resolves #127

We really only want to support multiple variants if we have a required reason. D65 is going to be the most expected variant with almost any color space unless the color spaces are inherently something else - think Prophoto RGB.
At this time, we have no intention of supporting both, so D65 is going to be the winner. If we decide to add HSLuv, it also requires D65.
The only reason we support Lab as D50 by default is that CSS requires it. If we had our way, we'd probably just support D65 there as well.
@gir-bot lgtm


[CLOUD_NATIVE] Create table stmt hangs forever when in fact there already encountered some S3 authentication errors in BE

Steps to reproduce the behavior (Required)

In shared_data mode, I'm trying to config s3 with Assumed Roleaws_s3_iam_role_arn = <role_arn>

Execute any CREATE TABLE statement, it will hangs forever without any user side error information printed.
Try to review be's log, there are some errors as following:

$ grep 'create table failed' be.INFO
W0720 22:55:37.425045 3800710 agent_task.cpp:192] create table failed. status: Internal error: starlet err Put object s3://xiaolong-staros/staros-root-0719/0c15900e-caa4-40a4-94dd-f49b4678d781/10007/SCHEMA_0000000000002718 error: You have no right to access this object because of bucket acl.
W0720 22:55:45.451227 3800709 agent_task.cpp:192] create table failed. status: Internal error: starlet err Put object s3://xiaolong-staros/staros-root-0719/0c15900e-caa4-40a4-94dd-f49b4678d781/10007/SCHEMA_0000000000002718 error: You have no right to access this object because of bucket acl.
W0720 22:55:49.466392 3800708 agent_task.cpp:192] create table failed. status: Internal error: starlet err Put object s3://xiaolong-staros/staros-root-0719/0c15900e-caa4-40a4-94dd-f49b4678d781/10007/SCHEMA_0000000000002718 error: You have no right to access this object because of bucket acl.
W0720 22:56:37.886595 3800710 agent_task.cpp:192] create table failed. status: Internal error: starlet err Put object s3://xiaolong-staros/staros-root-0719/0c15900e-caa4-40a4-94dd-f49b4678d781/10007/SCHEMA_0000000000002718 error: You have no right to access this object because of bucket acl.
W0720 22:57:36.468698 3800709 agent_task.cpp:192] create table failed. status: Internal error: starlet err Put object s3://xiaolong-staros/staros-root-0719/0c15900e-caa4-40a4-94dd-f49b4678d781/10007/SCHEMA_0000000000002718 error: You have no right to access this object because of bucket acl.
W0720 22:58:35.020584 3800708 agent_task.cpp:192] create table failed. status: Internal error: starlet err Put object s3://xiaolong-staros/staros-root-0719/0c15900e-caa4-40a4-94dd-f49b4678d781/10007/SCHEMA_0000000000002718 error: You have no right to access this object because of bucket acl.
W0720 22:59:33.614609 3800710 agent_task.cpp:192] create table failed. status: Internal error: starlet err Put object s3://xiaolong-staros/staros-root-0719/0c15900e-caa4-40a4-94dd-f49b4678d781/10007/SCHEMA_0000000000002718 error: You have no right to access this object because of bucket acl.
W0720 23:00:32.192379 3800709 agent_task.cpp:192] create table failed. status: Internal error: starlet err Put object s3://xiaolong-staros/staros-root-0719/0c15900e-caa4-40a4-94dd-f49b4678d781/10007/SCHEMA_0000000000002718 error: You have no right to access this object because of bucket acl.

When try hard to press Ctrl + C,  session terminated and some error logs printed on the termina

ERROR 1064 (HY000): Unexpected exception: Table creation timed out.
 You can increase the timeout by increasing the config "tablet_create_timeout_second" and try again.
To increase the config "tablet_create_timeout_second" (currently 10), run the following command:

admin set frontend config("tablet_create_timeout_second"="20")

or add the following configuration to the fe.conf file and restart the process:
tablet_create_timeout_second=20

Expected behavior (Required)
Can we make the stmt exit and throw timeout errors automatically when errors encountered?
Real behavior (Required)
StarRocks version (Required)
Lastest 3.1+ version

You can get the StarRocks version by executing SQL select current_version()

Close this issue, the real behavior is expected, a timeout error can throw out when tablet_create_timeout_second  * buckets count seconds have passed.
MySQL [wxl]> CREATE TABLE insert_wiki_edit_03
    -> (
    ->     event_time DATETIME,
    ->     channel VARCHAR(32) DEFAULT '',
    ->     user VARCHAR(128) DEFAULT '',
    ->     is_anonymous TINYINT DEFAULT '0',
    ->     is_minor TINYINT DEFAULT '0',
    ->     is_new TINYINT DEFAULT '0',
    ->     is_robot TINYINT DEFAULT '0',
    ->     is_unpatrolled TINYINT DEFAULT '0',
    ->     delta INT SUM DEFAULT '0',
    ->     added INT SUM DEFAULT '0',
    ->     deleted INT SUM DEFAULT '0'
    -> )
    -> AGGREGATE KEY(event_time, channel, user, is_anonymous, is_minor, is_new, is_robot, is_unpatrolled)
    -> DISTRIBUTED BY HASH(user) BUCKETS 1
    -> PROPERTIES("replication_num" = "1");
ERROR 1064 (HY000): Unexpected exception: Table creation timed out.
 You can increase the timeout by increasing the config "tablet_create_timeout_second" and try again.
To increase the config "tablet_create_timeout_second" (currently 10), run the following command:

admin set frontend config("tablet_create_timeout_second"="20")
or add the following configuration to the fe.conf file and restart the process:

tablet_create_timeout_second=20


Added toString for UInt64 to allow representing a 64-bit integer as a string.

This is very useful for derived projects from the nem2-typescript-sdk, since it allows representing heights, fees, etc. as strings (useful for user-interfacess, keys for databases, etc.). It supports any radix, and defaults to 10.

Pull Request Test Coverage Report for Build 629

13 of 13 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 60.515%

Totals

Change from base Build 613:
0.06%

Covered Lines:
6375

Relevant Lines:
9108

💛  - Coveralls

@rg911, Are the submitted changes better?


Add responsive blog

Add media queries for blog posts and navbar
Extract nav-links partial for collapsed and expanded navbar links
Extract Sass mixins:

center-element
add-flexbox

Style adjustments
Bunch of small refactorings
Browser screenshots
Aligned logo with blog content below

Aligned expanded navbar with blog posts below

@jglovier Hope this PR goes smoother without weird style inconsistencies.

Alrighty. I see to it as soon as I can.
No weird style inconsistencies this time?

@jglovier Here you go!
Smaller but still finger friendly

Blog Detail Page

Blog Index

@vis-kid thanks for addressing those issues. It's headed in the right direction, but it's not quite there yet. The biggest problem is that the menu is still too large for mobile.
Have you tested it on a mobile device or in Chrome's web inspector mobile preview? Your screencaps look just slightly smaller than desktop. I'm thinking that menu is still going to be really tall on actual mobile (likely overflowing the viewport), and the padding could go way down on those.

No problem. Will tweak it some more. Wanted to keep it as finger friendly as possible. I personally dislike it when you need to be careful to target links too carefully.

@jglovier hope this is closer to what you had in mind. :)
Various mobile screens

Looking good, @vis-kid. Thanks for all the extra fine tuning! :heart:

@jglovier Waahoo! Don’t mention it. Happy to!

The marketing page you mentioned in the tweet confused me a bit. 😊  Did you change your mind about the landing page? I thought the team page should be next on my agenda…

The marketing page you mentioned in the tweet confused me a bit.

Heh, sorry for the confusion. This whole site is the marketing site. :smile:

I thought the team page should be next on my agenda…

Please refer to #31 for our current prioritization. I'd like to see the Demo page get some attention next. Are you up for working on that? If not, feel free to pick whatever you're most interested to work on; it doesn't' necessarily have to be in our prioritization order.

I see. Got it! 😆
Whatever you need. Just not sure what you need for the demo page. Sort of like an Apple marketing page that takes screenshots and highlights the highlights? What did you have in mind?

@jglovier
For the marketing site in general, do you think we could use drawings of children from the hospitals who use this software?
It would be great to give the site a  human / humane touch like this and tell its story through the eyes of its youngest patients. Have a few ideas if you are interested in exploring this…

do you think we could use drawings of children from the hospitals who use this software?

That's a cool idea, but I'm not sure it's appropriate for the site mainly because it may give the impression that the software is just for children's hospitals, when really it's for any hospital. That being said, I'm open to other ways to incorporate some more imagery into the site.

Important concern, I agree. My take would be to counter balance that with the accompanying copy, emphasizing that it’s for everybody. Maybe a playful note that this site was designed by its youngest patients or something. Not easy to solve but totally doable I guess.  :)
Just think that this approach would make the the site stand out, departing from the usual look of donation based non-profits while giving the kids a voice of sorts. If we do this right, I feel this would enrich the site tremendously.

Also, if we mix that approach with a couple more photos like that landing page already has, we can avoid that misconception that it is for children's hospitals…

@jglovier
any particular ideas / preferences for that Demo page?


[Botskills] Add migrate command

Purpose
What is the context of this pull request? Why is it being done?
Since a new manifest schema was released, the old skills had to be migrated manually from the skills.json file to appsettings.json file. So we added the migrate command to automate this work.
Changes
Are there any changes that need to be called out as significant or particularly difficult to grasp? (Include illustrative screenshots for context if applicable.)

Add --sourceFile argument where will be specified the file from where the skills will be migrated  (default is the skills.json file)
Add --destFile argument where will be specified the file where the skills will be migrated (default is the appsettings.json file)

Tests
Is this covered by existing tests or new ones? If no, why not?
They are not currently covered by tests. It's in the feature plan in order to finish with this feature.
Feature Plan
Are there any remaining steps or dependencies before this issue can be fully resolved? If so, describe and link to any relevant pull requests or issues.
We will continue updating the documentation, fixing tests, updating the VA/Skill.
Checklist
General

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated the appropriate tests
[ ] I have updated related documentation

Bots

[ ] I have validated that new and updated responses use appropriate Speak and InputHint properties to ensure a high-quality speech-first experience
[ ] I have replicated language model changes across the English, French, Italian, German, Spanish, and Chinese .lu files and validated that deployment is successful

Deployment Scripts

[ ] I have replicated my changes in the Virtual Assistant Template and Sample projects
[ ] I have replicated my changes in the Skill Template and Sample projects

Dupe of #204