(Possibly) Incomplete display of glyphs when there are a lot of glyphs in a font

Hi
Very nice to use your extension, avoids moving out of VSCodium to an external viewer !
But I encountered a small issur with "Iosevka NF" font, a Nerd Font with a lot of glyphs :
The top bar with buttons for groups of 200 glyphs is spilling outside the right edge of the window, with button "3400-3599" partially visible.
So even on a 34'' display not all the buttons can fit the width even with the left toolbar (Explorer/Search/etc) closed.
Possible solutions:

Move this button bar into a container that will make the flow fit inside (à la html "display" property set in a div)
Make the size of display groups configurable instead of default 200 ; at least from an extension settings property, but an input/select in the font display page would be much better.
In standard HTML this is trivial, but I know nothing about VSCode extensions, so may be that's not as simple as in HTML.
That would be great anyway !

That's odd, I'm not able to reproduce this in VS Code so this might only be an issue in VS Codium. Would you mind attaching a screenshot and the version of VS Codium you're using? I'd love to try and fix this!

https://ibb.co/sFzmkK3
Windows 10 + latest VSCodium.
FWIU VSCodium should behave the same as VSCode except on telemetry, intellectual property / logos from MS, access to MS marketplace. That's theory, but as often things are more complicated/subtle IRL...
[ I'm using VSCodium rather than VSCode because I'm concerned with telemetry and blackbox build of the binary from behind MS Corp walls. . MS Visual Studio Enterprise is another great tool, but I'm moving away from it for the very same reason. ]


fix: fix nil pointer error with StatefulSet labels when set

what
Fix nil pointer error with StatefulSet labels when set, introduced in #356.
why
    [ERROR] templates/: template: atlantis/templates/statefulset.yaml:8:22: executing "atlantis/templates/statefulset.yaml" at <.Values.statefulSet.labels>: nil pointer evaluating interface {}.statefulSet

tests

references
https://github.com/runatlantis/helm-charts/pull/356#discussion_r1516867355

Thanks @peikk0 🙏 can you please bump the patch version for the chart?

@GMartinez-Sisti done 👍🏻


Explore options for designing and creating a mini controller to communicate with the K8s API server

The Mini Controller is a logical subsystem within the BBP (Branch-based Planner) executable. It is designed to interact with the Webhook Receiver (another logical system in the same program) and handle incoming Pull Request (PR) information.
Mini Controller Functionality
The Mini Controller is responsible for the following tasks:

Receive PR information from the Webhook Receiver.
Locate and manipulate various objects based on the received PR information.
Post a readable plan back to the PR.

Implementation Steps
To implement the Mini Controller, we may follow these steps:

Set up Go-routines

Ensure that the Webhook Receiver and Mini Controller live in their own Go-routines within the BBP program.

Receive PR information
Upon receiving PR information from the Webhook Receiver, perform the following steps:

Locate the GitRepository object using the Git repo information from the PR. Refer to this object as g1.
Locate the Terraform object using the path information from the PR. Refer to this object as t1.
Locate the secret that stores the tfstate of t1. Refer to this Secret as s1.
Pause t1 (e.g. via controller-runtime annotation)

Clone objects and configure

Clone objects g1 and t1 into new objects, say g2 and t2. Link them together, so t2.sourceRef == g2.
Duplicate s1 as s2. So t2 would plan against tfstate information in s2.
Configure t2 by setting spec.approvalPlan to an empty string and spec.storeReadablePlan to human. This will allow it to create a readable plan in a ConfigMap, say cm1, after each planning step.

Post plan back to PR

Read the plan from the content of cm1 and post it back to the PR.

@chanwit : What exactly will the mini controller do?

I'll refine its tech details and get back to you.

ping @yitsushi I added tech details as a guideline to implement the mini controller. Please feel free to discuss.

I think I found something that does not match with the current workflow of the tf-controller.
I'll do another revision to improve it and get back to you.


gnupg22: 2.2.20 -> 2.2.21

Motivation for this change

Version update https://lists.gnupg.org/pipermail/gnupg-announce/2020q3/000446.html
Fix reproducibility by fixing SOURCE_DATE_EPOCH usage

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

/marvin opt-in

The release notes look okay to me.

I'm sorry this fell off my radar. I've reconfirmed that everything looks good with a local build. Thanks!


Missing CSS attributes: orphans, widow

Missing CSS attributes: orphans, widow

This issue was imported from Google Moderator
Moderator votes: +32
Issue added by Apff on 2014-06-04
To vote this issue up or down, simply include +1 or -1 in your comment.

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

:+1:
Renaming this issue from widow -> widows might help people find it. didn't show up when I searched for widows.

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

Now available at http://caniuse.com/#feat=css-widows-orphans


Change payment reference to confirmation number in payment confirmation

also, debit-card-failure has “Your reference number” … i think that should remain, right?

translation files can wait ;)


Update test ZARR path

Context
One test uses real (i.e., not specifically for testing) data. The scientific project publishing the data has moved the file from the nuc_morph_data folder to the nuc-morph-dataset folder. The new location is visible here on quilt. The file at the old path will soon be removed.
Change
Update the path in the test.
Testing
pytest bioio_ome_zarr/tests/test_s3_read.py

Test failures in CI appear unrelated. The test I touched is passing in CI.

Test failures in CI appear unrelated. The test I touched is passing in CI.

We need to get the unit tests back to green asap or else remove them completely.  I think a prior PR in here will fix them but is waiting for a bioio-base version bump first.

Merged #25 into this branch

Merged #25 into this branch (I probably should have waited to merge this one first)

Undid this with a force push so this PR is now simple again


Feature/help button

@Ren22 having a screenshot would be awesome :)

@intsco makes sense :)

Actually, I don't know what is better to have a pop-up window or open the document in another browser tab.
We should also ask @LachlanStuart or @theodev

@intsco we discussed it with Lachlan before and agreed upon that pop window will be a better option. Imo it's good because it doesn't interfere the process of filling metadata by sending user to another page. But the minor thing that I can add is a button to open it a new window from pop-up, if user is struggling with the resolution

@LachlanStuart do you think this PR can be merged?


⚠ Secret Website 3 has degraded performance

In c0cd691, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2088 ms

Resolved: Secret Website 3 performance has improved in e2fb7b8.


[composer] e2e tests

@wittjosiah would you mind listing out the e2e tests you think we need here?

This was progressed by https://github.com/dxos/dxos/pull/4967 and https://github.com/dxos/dxos/pull/4968 which setup tests for drag and drop in the stack storybook and within the composer application.
The one other test that we need for the composer-baseline is a test to ensure it catches protocol changes as expected. I have this test drafted on a branch and will land it soon.
There are a lot of other e2e tests to write but this is for a baseline. The other tests will be written as we promote experimental stuff to being on-by-default within Composer.


User upload from gateway home directory

Currently user can upload data files from their own computer.  We also need a way for the user to be able to upload files from their home directory on the gateway

the shinyFiles package @ https://rdrr.io/cran/shinyFiles/man/shinyFiles-package.html seems useful for server-side browsing


Don't show check mark when overpass returns no results

Entered by friend Sam Bolstad while testing at his house :+1:

Expanding on this, the loading feature should have a success/error/info callback that dictates the icon/color - something like:
dc.util.loader = function(yes, typeClass) {
    var loader = $('<div>').addClass('dropchop-loader');

    if (yes) {
      $('body').addClass('dropchop-loading');
      $('body').append(loader);
    } else {
      $('body').removeClass('dropchop-loading');
      $('.dropchop-loader').addClass(typeClass).fadeOut(2000, function(){ // class that has icon built in
        $(this).remove();
      });
    }
  };

I was thinking about this last night when I noticed a similar issue with the "Import file from a URL" feature. Perhaps a more Nodejs-y convention would be passing an optional error object, and displaying an appropriate loading state icon. Something like...
dc.util.loader = function(yes, error) {
    var stateClass = (error) ? 'loader-complete-error' : 'loader-complete';
    /*......*/

        $('.dropchop-loader).addClass(stateClass).fadeOut(....
}

Operationally, it would look like
// Toggle loading state
dc.util.loader(true);

// Remove loading state with a success
dc.util.loader(false);

// Remove loading state with an error
dc.util.loader(false, error);

Thoughts?

On a related note (this might be a separate issue) it might be best to handle the loading state on a method-by-method basis instead of universally through dc.util.xhr. Because it is handled directly in that function, there isn't a good way to handle situations dc.ops.file['load-url'] where a valid request is made but an invalid GeoJSON object is returned.
By moving loading state trigger out of dc.util.xhr there is more freedom to validate objects before informing the user that the process is complete.

Excellent thoughts, @jczaplew. Definitely agree that taking the loader out of the xhr is a good call. What scenarios do we need unique icons/colors for?

success
error, mishandled response
error, unsuccessful query (bad URL)
successful query but "no data" (is this an error or more of an info notification?)

It seems like if we have these errors popping up, we'll want to notify what the error is. Do you think it would make sense to build that into the loader or just use dc.notify to pass a notification separately?

For notifying the user, I think that could be baked into the loader.
As for different icons/colors, it could possibly be generalized to

success - everything went a-okay
warning - something went right, something went wrong (good request/bad response or good request/no data, for example)
error - a lot went wrong

How about something like
dc.util.loader = function(yes, errorMessage, errorType) {
    var loader = $('<div>').addClass('dropchop-loader');
    var stateClass = 'loader-complete';

    if (errorMessage) {
        // If there is an error type and it equal 'warning', show the warning symbol, otherwise default to error
        stateClass = (errorType && errorType === 'warning') ? 'loader-complete-warning' : 'loader-complete-error' ;
        dc.notify('error', errorMessage, 6000);
    }

    if (yes) {
      $('body').addClass('dropchop-loading');
      $('body').append(loader);
    } else {
      $('body').removeClass('dropchop-loading');
      $('.dropchop-loader').addClass(stateClass).fadeOut(2000, function(){
        $(this).remove();
      });
    }
  }

?
That way the loader function is generic and error messages can be unique to the method producing them.

Looks good to me!
Semi-related: It seems important to have sticky notifications for errors so users can copy them without rushing before they automatically remove. Created a separate issue for this #193

The _loader.scss file probably needs some reworking to take care of the extra classes - the animation should be removed from the base class as well, and instead we can have a unique "loading" class:
.dropchop-loader {
  &.loader-loading {}
  &.loader-error {}
  &.loader-warning {}
  &.loader-success {}
}


CollectibleAssembly / AddApplicationPart => IISExpress crash

I got myself the latest .net core 3 preview and tested the new CollectibleAssembly thing.
I am loading an assembly containing a controller and add it as applicationpart using mvcBuilder.AddApplicationPart.
When I use 'isCollectible=false', everything works as expected, controllers actions can be browsed.
Using 'isCollectible=true' makes iisexpress crash after reloading the page couple of times.
Is this something that is supposed to work or can collectible assemblies not be used with application parts?
The usecase is to exchange applicationparts without stopping the whole application.
Thanks for your help.

IISExpress exit code is 0x80000003

@Hondo2019 do you have a repro? Are you able to reproduce on another machine?
Did you try to minimize the repro? Can you please share it?
@janvorli to comment ...

I did not succeed creating a repro yet, it seems to work in a simple scenario. I will keep trying.
Does the IISExpress exit code indicate anything?

@Hondo2019 I am sorry, I've missed this issue. The exit code 0x80000003 means a breakpoint was hit.
Can you please try to check if that still repros with the .NET core 3.0 Preview 5?

@Hondo2019 did you have a chance to try it with .NET core 3.0 Preview 5 or 6?

Sorry, havent had the time to play with the newer releases yet. I will have to deal with this issue again when the final release is here.

Hi! Just had the chance to install preview 6, everything seems to work now! It means something has been fixed?


Correct location of isIterable function

Just a small docs fix. The isIterable function is on Iterable (not Immutable).

Merged in 825e29ef98012d210f817905994529be09c45de2, thanks!


Code Review Item: the init script should include UI related components

The Unicon IAM team did a code review and suggested several small adjustments. This is one of those adjustments:
The init script can be called to initialize a new IdP deployment. It creates a basic config, keys, metadata, etc and exports it to the Docker host. It would be helpful to deployers to also export the views, css, images, and js directories as these usually are modified shortly after spinning up the new IdP.
If deployers want to use the baseline assets, the local files can just be removed from the local Docker image source.

Commit: 668d5f148f7c53d9948242cb0df38309ca002bb3


How can i run Autocmd on another opened buffer ?

I have vim open with 2  buffers split vertically, the second one is  output file or log file which gets updated outside of vim, so to achieve auto scrolling when that happens i'm doing this :
au TextChanged output:///* normal G(everytime the buffer change the cursor goes to the bottom)
my problem is : the scrolling only happens when i'm focusing the buffer or entering the buffer.
i tried to lookup  events other than TextChanged ,
i found this FileChangedShellPost but without luck it didn't work.

TextChanged is only fired when the buffer is changed from a command in the current Vim instance.
FileChangedShellPost is fired (for the most part):

after you execute a shell command with :!
after you execute :checktime
after you focus back the Vim terminal window (if your terminal supports the focus event tracking feature, or you are in the GUI)

See :help FileChangedShell:

Mostly triggered after executing a shell
command, but also with a |:checktime| command
or when gvim regains input focus.

Otherwise, the event is not fired, which is probably why your :normal G command is not executed when the file is changed outside Vim.

Anyway, if you're on linux and you have inotifywait, then you can try:
vim9script
&autoread = true
command -bar AutoRead AutoRead()
def AutoRead()
  if !executable('inotifywait')
    echomsg 'missing dependency: inotifywait(1); https://man7.org/linux/man-pages/man1/inotifywait.1.html'
    return
  endif
  var file: string = expand('%')->fnamemodify(':p')
  var cmd: string = printf('while true; do inotifywait --event modify %s; done', file)
  job_start(['sh', '-c', cmd], {out_cb: function(AutoreadOutCb, [bufnr('%')])})
enddef
def AutoreadOutCb(buf: number, ..._)
  execute 'checktime ' .. buf
  var winid: number = getbufinfo(buf)
    ->get(0, {})
    ->get('windows', {})
    ->get(0)
  win_execute(winid, 'normal! G')
enddef

On Ubuntu, inotifywait(1) is provided by the inotify-tools package:
$ sudo apt install inotify-tools

Same Vimscript code in legacy:
set autoread
command -bar AutoRead call s:AutoRead()
function s:AutoRead() abort
  if !executable('inotifywait')
    echomsg 'missing dependency: inotifywait(1); https://man7.org/linux/man-pages/man1/inotifywait.1.html'
    return
  endif
  let file = expand('%')->fnamemodify(':p')
  let cmd = printf('while true; do inotifywait --event modify %s; done', file)
  call job_start(['sh', '-c', cmd], #{out_cb: function('s:AutoreadOutCb', [bufnr('%')])})
endfunction
function s:AutoreadOutCb(buf, ...)
  execute 'checktime ' .. a:buf
  let winid = getbufinfo(a:buf)
    \ ->get(0, {})
    \ ->get('windows', {})
    \ ->get(0)
  call win_execute(winid, 'normal! G')
endfunction

You might also be interested in the vim-autoread plugin.

For more info, see:

:help 'autoread'
:help :checktime
:help :command
:help :command-bar
:help :def
:help :execute
:help :var
:help bufnr()
:help executable()
:help expand()
:help fnamemodify()
:help function()
:help get()
:help getbufinfo()
:help job_start()
:help printf()
:help win_execute()

For more questions, visit the google mailing list, the irc channel, or vi.stackexchange.com.

TextChanged is only fired when the buffer is changed from a command in the current Vim instance.
FileChangedShellPost is fired (for the most part):

after you execute a shell command with :!
after you execute :checktime
after you focus back the Vim terminal window (if your terminal supports the focus event tracking feature, or you are in the GUI)

See :help FileChangedShell:

Mostly triggered after executing a shell
command, but also with a |:checktime| command
or when gvim regains input focus.

Otherwise, the event is not fired, which is probably why your :normal G command is not executed when the file is changed outside Vim.
Anyway, if you're on linux and you have inotifywait(1), then you can try:
vim9script
&autoread = true
command -bar AutoRead AutoRead()
def AutoRead()
  if !executable('inotifywait')
    echomsg 'missing dependency: inotifywait(1); https://man7.org/linux/man-pages/man1/inotifywait.1.html'
    return
  endif
  var cmd: string = 'inotifywait --monitor --event modify ' .. expand('%:p')
  job_start(cmd, {out_cb: function(AutoreadOutCb, [bufnr('%')])})
enddef
def AutoreadOutCb(buf: number, ..._)
  execute 'checktime ' .. buf
  win_execute(bufwinid(buf), 'normal! G')
enddef

It installs an :AutoRead command which you can execute in the window displaying the buffer which you want Vim to automatically reload whenever the corresponding file changes, and for which you always want the cursor to be on the last line.
On Ubuntu, inotifywait(1) is provided by the inotify-tools package:
$ sudo apt install inotify-tools

Same Vimscript code in legacy:
set autoread
command -bar AutoRead call s:AutoRead()
function s:AutoRead() abort
  if !executable('inotifywait')
    echomsg 'missing dependency: inotifywait(1); https://man7.org/linux/man-pages/man1/inotifywait.1.html'
    return
  endif
  let cmd = 'inotifywait --monitor --event modify ' .. expand('%:p')
  call job_start(cmd, #{out_cb: function('s:AutoreadOutCb', [bufnr('%')])})
endfunction
function s:AutoreadOutCb(buf, ...)
  execute 'checktime ' .. a:buf
  win_execute(bufwinid(a:buf), 'normal! G')
endfunction

You might also be interested in the vim-autoread plugin.
For more info, see:

:help 'autoread'
:help :checktime
:help :command
:help :command-bar
:help :def
:help :execute
:help :var
:help bufnr()
:help bufwinid()
:help executable()
:help expand()
:help function()
:help job_start()
:help win_execute()

For more questions, visit the google mailing list, the irc channel, or vi.stackexchange.com.

yes this seems the way to do it, but it wont work in my case since inotfiywait wont watch output:/// for changes,  it's not file.
there is missing call in win_execute(bufwinid(a:buf), 'normal! G')
btw thank you for the long answer , you pointed me to the right path i will keep looking at it.

yes this seems the way to do it, but it wont work in my case since inotfiywait wont watch output:/// for changes, it's not file.

Then maybe a timer running forever and executing :normal! G every 1s:
vim9script
command -bar AutoScrollToBottom AutoScrollToBottom()
def AutoScrollToBottom()
  var winid: number = win_getid()
  timer_start(1'000, (_) => win_execute(winid, 'normal! G'), {repeat: -1})
enddef

command -bar AutoScrollToBottom call s:AutoScrollToBottom()
function s:AutoScrollToBottom()
  let winid = win_getid()
  call timer_start(1000, {-> win_execute(winid, 'normal! G')}, #{repeat: -1})
endfunction

See :help timer_start().

there is missing call in win_execute(bufwinid(a:buf), 'normal! G') (legacy)

I updated the snippets to fix this issue.

Thank you so much . it's working like intended .


How to user python?

How to use in python?

We're working on a Python release. For more information, please see https://github.com/google/tink/issues/248.


[CATALOG]: Test request

Contact details
<EMAIL_ADDRESS>Organization type
Microsoft employee
End-to-end business process
Hire to retire
Which business process area is this article related to?
Create and manage sales
Which business process is this article related to?
Create a sales order
Enter the name of the pattern or practice you are starting work on
Create a sales order in the retail point of sale
Add any additional comments or information you want us to know about this business process catalog entry. If this is an update to an existing row in the catalog, please be sure to indicate in the comments.
Comments go here
Code of Conduct

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

Closing the test.


Fix typo in README

Personally I would also add trailing . to each sentence, but that was not the overall style of the README so I didn't add them.

Thank you!


feat(caraml-store): Add chart for caraml store

Signed-off-by: khorshuheng<EMAIL_ADDRESS>Motivation
This is the charts for CaraML feature store, which comprises two components, registry and serving.
While spark operator is required for a complete installation, this chart currently only includes sufficient dependencies such that the services can be installed using the default value and running.
Modification
Checklist

[x] Chart version bumped
[x] README.md updated

@mbruner For the prometheus endpoint, what would be the preferred approach to expose it? Should the creation of prometheus monitor resource be part of the chart, or should it be handled independently?

@khorshuheng I think we should have monitors as a part of the chart guarded by a flag.
prometheus:
  monitor:
    enabled: false
    ...


Multiline annotation support

As a user when I have multiline metadata in my yaml then I want to see it formatted properly on the page.  This pertains to all Flux resources that are shown in detail in Weave GitOps.

Kustomization
Helm Release
Bucket
Git Repository
Helm Repository

Considerations
Given the sheer number of options that can be supported for multi-line text support we will only provide support for a limited number of those options in the beginning, unless a 3rd party library can be found that supports the translation.
Reference
https://stackoverflow.com/questions/3790454/how-do-i-break-a-string-in-yaml-over-multiple-lines
Acceptance Criteria
Add annotations to a Git Repository, Bucket, and Helm Repository in your git repository and have them sync with Flux.  When you open Weave GitOps and go to the detail pages of those specific Git Repository, Bucket, and Helm Repository objects you will see the following behavior.
All of the criteria from #2222 and #2223 still apply for this story.

When I mark a metadata tag with the multiline character > then the UI will adhere to the behavior outlined in the link above which is "interior line breaks are stripped out, although you get one at the end". E.g.

would appear as

When I mark a metadata tag with the multiline character | then the UI will adhere to the behavior outlined in the link above which is "turns every newline within the string into a literal newline, and adds one at the end". E.g.

would appear as

Note: the additional line is aligned with the value block of text.

When I mark a metadata tag with the multiline character " then the UI will adhere to the behavior outlined in the link above which is "" must be escaped by , newlines can be inserted with a literal \n sequence, lines can be concatenated without spaces with trailing \". E.g.

would appear as

Out of Scope
Block styles with block chomping indicator (>-, |-, >+, |+)
"Flow" scalar styles ( , ')

I confirmed that Kubernetes processes them for us.
metadata:
  annotations:
    metadata.weave.works/description-1: >
      Hello
      World
    metadata.weave.works/description-2: |
      Hello
      World
    metadata.weave.works/description-3: >-
      Hello
      World
    metadata.weave.works/description-4: |-
      Hello
      World
    metadata.weave.works/description-5: >+
      Hello
      World
    metadata.weave.works/description-6: |+
      Hello
      World
    metadata.weave.works/description-7: 'Several lines of text,
      containing ''single quotes''. Escapes (like \n) don''t do anything.
      .
      Newlines can be added by leaving a blank line.
      Leading whitespace on lines is ignored.'

Here's the internal json representation
    "metadata": {
        "annotations": {
            "metadata.weave.works/description-1": "Hello World\n",
            "metadata.weave.works/description-2": "Hello\nWorld\n",
            "metadata.weave.works/description-3": "Hello World",
            "metadata.weave.works/description-4": "Hello\nWorld",
            "metadata.weave.works/description-5": "Hello World\n",
            "metadata.weave.works/description-6": "Hello\nWorld\n",
            "metadata.weave.works/description-7": "Several lines of text, containing 'single quotes$
. Escapes (like \\n) don't do anything. . Newlines can be added by leaving a blank line. Leading wh$tespace on lines is ignored."
        },


Load/download extension using extensions.json adjacent to settings.json

I think it makes sense to move extensions to extensions.json for following advantages.

Easy to share and sync
Workspace specific extensions
Allow users to create extensions collection which user can enable or disable as collection
e.g csharp collection in extensions.json file will have on csharp related extensions and can be disabled in workspace extensions.json so unnecessary extensions are not loaded

Thx

@sandy081 @joaomoreno

This makes sense
It will also solve this issue : https://github.com/Microsoft/vscode/issues/14444

I don't understand what you're asking for, we support an extension.json file already.

Hey @roblourens
I am new to IDE, plus @shanalikhan also have similar year old issue. I didn't came across extension.json either at location C:\Users\user\.vscode-insiders or C:\Users\user\AppData\Roaming\Code - Insiders\user
So it will be awfully nice of you to point out if I am missing something.
Thx

We a have a couple features that address this:

Workspace recommended extensions, via a extensions.json file in your .vscode/ folder: https://code.visualstudio.com/docs/editor/extension-gallery#_workspace-recommended-extensions
Extension Pack extensions let you share and download a bundle of extensions.

Hope that helps!

Hey @robertohuertasm
That was extremely helpful information.
But I still feel its more inline with the design pattern of light weight text editors such as vscode to save all settings in json file including user-wide extensions.
Let me try to rephrase and update my proposal.

In user folder C:\Users\user\AppData\Roaming\Code - Insiders\user create extensions.js and environment.js

extensions.js should contain list of all extensions and there enabled status; this is more in line with design pattern and aggregate all vscode configrations in one folder.

environment.js is optional file where you can create collection of extensions. Lets say we have csharp, misc and cpp environment. Each can have collection of extensions that are available. Benefit of this is one can enable/disable any environment. Lets say you enabled misc and cpp in settings all csharp extensions will be not loaded but union of misc and cpp extensions will be loaded.

Let me know if I was able to explain the proposals decently ; I do strongly feel about extensions.js at user level to manage state of extensions is inline with design pattern.
Thx

Thanks for the details. We try to design the extension APIs so that the user doesn't need to do a lot of manual management like that - I have lots of extensions installed that are only useful in certain file types or workflows, but I've never needed to enable/disable them individually. It is possible to disable them for a certain workspace (by clicking the gear in the extensions viewlet) but I don't think that controlling it with a config file is necessary.
Instead, if there are extensions that need to be enable/disabled really often, we should figure out why that is and either file issues on the extension, or change the extension APIs so that they don't interfere.

Well json configs are for nerds and that what vscode copied from success of sublime. This allows you to check them is git, share and expand on each others work.
Anyways you already have UI for settings.json but still have file for it which I can sync accross platforms easy. Same true for extensions, people who don't wanna touch extensions.json file; you already have UI for it; I am just purposing the using json as store instead of internal store.
Thx

Got it. I just realized this is also tracked in https://github.com/Microsoft/vscode/issues/15442, there has been some other discussion about it if you check the other linked issues too.


Mobile device, gestures - modal content is inaccesible for about a second

I'm submitting a ..
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior
I use HammerJS for gestures. There is an event handler for press (touch & hold) and swipeleft. I'm using those events to open a modal with couple buttons. Click event of those buttons is inactive for (I think) a second, maybe half. I had to click twice after modal shows up. Same behavior if I want to dismiss a modal instantly after shows up. Also the X button in header has same behavior.
If I attach same modal to a button, without gestures, just click, anywhere on the page, everything is fine.
Expected behavior
It should be accesible instantly after show.
Reproduction of the problem
Try interact with modal content opened by any hammerjs event.
Please tell us about your environment:
Windows 10 1809, Visual Studio Code, npm, ng serve

Smart Modal version: 0.8.x
7.1.1

Angular version: 2.0.x
7.2.8

Browser:
72.0.3626.121 Windows and Android

Language:
TypeScript

Hi @troncomputers and thank you for this well-detailed issue.
Indeed, it's strange. Can you provide a StackBlitz example please? Thus we can help you and understand your issue better 😉
Have you tried to disable modal's transition effects? It may be blocking all HammerJs interactions. 🤔
I stay tuned. 🖖

@maximelafarie Hi 😉
Here is a StackBlitz example angular-snrbnc
To test this correctly you need to view this page from (F12) developer options and toggle a device toolbar. Try swipe left on red box and click anywhere to dismiss a modal or click a button quickly. I know that this is a silly problem. Propably there is no one who click that fast, but my modals acts like context menu and I'm opening them as close as possible to the pointer/finger so it can be fast, specially when someone get used to to the modal position and will know where to click fast.

Great, thanks @troncomputers! 🤘
I'll debug it and go back to you after that.
There's no silly issues, only issues that deserves to be fixed! 😉
We'll see if it's a library-related issue or not.
Stay tuned! 🖖

Hi @troncomputers and thank you for your patience!
After a lot of checks and debug, I found that all is good for me, and it seems to be an issue that is non-related to the library. I tried again and again to click on the buttons and even if I automate it, it works perfectly. 😉
Have you some other code that may delay the execution of any function or actions?
I stay tuned! 🖖

Hi! @maximelafarie
The example that I provided uses only hammerjs and ngx-smart-modal and its behavior is the same as in my app. I guess it might be problem with hammerjs and gestures events. I didn't check their code but I think there is some kind of timeout for press and swipe because those events are based on how long you hold or how far you swipe. Well... I like your smart-modal so I need to try/find another gesture library or fix problems. Thx for help 👍 great job 😄

Thank you for your fast reply @troncomputers!
As you said, it might come from hammerjs, but since it's a huge and famous library, I really need to investigate for a such buggy behavior from our side. 🤔
Thank you so much for the kind words! Please let me know if anything solved your issue. I stay tuned and I keep testing! 😉
You rock, thank you! 🤘🖖

@biig-io/frontend can anybody test on the stackblitz link below? 😄

@troncomputers Here's my fork of your StackBlitz example with some automation. Let me know if there's still a delay between hammerJs action and clicks! 😉 https://stackblitz.com/edit/angular-inup2i

@maximelafarie The issue is only on mobile. You have to enable mobile view in Chrome if you're doing this on a PC. There is no issue in normal, desktop mode.
Try this modification:
  ngAfterViewInit() {
    this.ngxSmartModal.get('DocumentOptions').onOpen.subscribe(() => {
      // this.looper = setInterval(() => {
      //   document.getElementById('edit-btn').click();
      // }, 100);
    });

    this.ngxSmartModal.get('DocumentOptions').onAnyCloseEvent.subscribe(() => {
      if (!!this.looper) {
        clearInterval(this.looper);
        this.Counter = 0;
      }
    });
  }

  tap() {
    this.Counter++;
  }

  count() {
    this.Counter--;
  }

        <div class="col-12">
          <button
            id="edit-btn"
            style="border-radius:15px"
            class="btn btn-block btn-new-primary mt-2"
            (click)="count()"
            (tap)="tap()";
            >Edit</button
          >
          <button
            style="border-radius:15px"
            class="btn btn-block btn-new-primary mt-2"
                        (click)="count()"
                                    (tap)="tap()";
            >Print</button
          >
          <button
            style="border-radius:15px"
            class="btn btn-block btn-new-primary mt-2"
                        (click)="count()"
                                    (tap)="tap()";
            >Cash</button
          >
          <button
            style="border-radius:15px"
            class="btn btn-block btn-new-remove mt-2"
                        (click)="count()"
                                    (tap)="tap()";
            >Delete</button
          >
        </div>

Click/Tap immediately after modal shows up. You will see that at the second click/tap your counter will be decremented and stay on last number.
I made a short video with behavior that I have 😃
Youtube Video

@troncomputers You rock that's so cool! 🤘 Sorry but your video seems to be deleted or isn't available. 😕

Video was private, I've changed it to nonpublic. It should be ok now 😉

Hi @troncomputers, please excuse me for the late answer! Did you find any workaround since this issue? I tried to reproduce and thanks to your record, I could see the bug! 👍

Hi @maximelafarie, I moved to Angular Material 🙈

Haha no problem! I'll try to solve this issue anyway! 😉
Happy coding! 🖖


Generate a model card ID using the author, name, and version.

Fixed by https://github.com/Data-to-Insight-Center/patra-toolkit/pull/18


[BUG] 'group_by_modality_length' argument ineffective

Thanks for your great work.
It seems a bug in https://github.com/haotian-liu/LLaVA/blob/2ca20de1ca76d7d121be5a53f8a46c6bef47a9cb/llava/train/train.py#L656
This will lead to the 'group_by_modality_length' argument becoming ineffective, as 'images' is never appears in the raw JSON file (self.list_data_dict).
We found that this will lead to some performance drop.
Looking forward to your response!

Thanks, this was introduced in PR #694, which is just reverted in #866. Thanks!


Clarification on conditional updates of sub-attributes

would there be an easy way to update attributes based on conditions on the sub attributes?
For example if my Dyntastic object looks like so
class Foo(Dyntastic):
    __table_name__ = getenv("TABLE_NAME", None)
    __hash_key__ = getenv("PARTITION_KEY", None)
    pk: str= Field()
    data: Data | None = Field(None)

class Data(BaseModel):
    id: str| None = Field(None)

and I would like to update my data field with some arbitrary condition on my data.id field:

foo = Foo(pk="some_pk")
data_dump = Data(id="bar")
foo.update(A.data.set(data_dump), condition = (A.data.not_exists() | A.data.id.ne("foo") )

It might be possible using A("data.id") to reference nested fields. Give that a try and see if it works for your use case.

It seems to result in a no-op even if the condition is correct,
my condition statement is something like this (id is int now):
foo.update(A.data.set(data.model_dump(exclude_none=True)),
                    condition=(A("data.id").lt(data.id)),require_condition=True,
                    refresh=False)

I don't really know if I'm still doing something wrong, maybe Attr is not being set correctly for nested fields?

I was able to confirm that this simple case works:
item.update(A("my_dict.a").set(100), condition=A("my_dict.a") == 1)
assert item.my_dict["a"] == 100

What behavior are you seeing? Since you have require_condition=True, do you see a ConditionException?


[5.7] makes prependMiddleware more similar to pushMiddleware

pushMiddleware created an array as needed.  prepend did not.

No tests.


Drug type ahead not acting as expected

When adding new Predictive EID or AID in V2, the drug typeahead is not as expected. In this example for Trastuzumab from v1.5 we expect this:

Instead currently I get this for EID:

Or for AID

This was already reported in https://github.com/griffithlab/civic-v2/issues/338 and #361 and fixed in https://github.com/griffithlab/civic-v2/pull/339. This should be live any moment now.

Also unable to select the desired drug "Trastuzumab" and seem forced to choose something wrong which contains the word instead:


Add AS-IS disclaimer to map page

As per updated guidance, we need to include this text some where on the maps page:
"The COVIDcast site content and Epidata API are provided on an AS-IS, AS-AVAILABLE basis and are not intended for diagnostic or treatment purposes, or for guidance on any type of travel.  Please refer to the Terms of Use."

@Letsgohyde let's figure out where this should go from a UX perspective first.

4 point DingBats font?

Let's go with option 1 here:

If it's not too much trouble, we'll add a method to dismiss it and set a cookie to remember that acknolegdement.

If it's not too much trouble, we'll add a method to dismiss it and set a cookie to remember that acknolegdement.

should this be part of covidcast or the website?

Covidcast.
Also the updated text is:
"The content of the COVIDcast site and Epidata API are subject to an AS-IS, AS-AVAILABLE basis and are not intended for diagnostic or treatment purposes, or for guidance on any type of travel. Please refer to the Terms of Use."


chore: upgrade to OpenTelemetry 0.23.0

Codecov Report

Merging #173 (c4c2f4b) into main (e337742) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #173   +/-   ##
=======================================
  Coverage   86.18%   86.18%
=======================================
  Files          12       12
  Lines         275      275
  Branches       62       62
=======================================
  Hits          237      237
  Misses         38       38

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 e337742...c4c2f4b. Read the comment docs.


Connect with appium

Hello,
I love the approach of this project, I am currently using a wrapped selenium hub for my company to provide device automation.
Currently i add nodes to selenium hub using appium through nodeconfig param as such:
{
  "capabilities": [
    {
      "browserName": "iPhone 7 (iPhone9,3)",
      "appium-version": "1.7",
      "platform": "iOS",
      "version": "11.2",
      "deviceName": "iPhone (11.2.6)",
      "maxInstances": 1
    }
  ],
  "configuration": {
    "cleanUpCycle": 2000,
    "timeout": 30000,
    "proxy": "org.openqa.grid.selenium.proxy.DefaultRemoteProxy",
    "port": 5555,
    "maxSession": 1,
    "role": "node",
    "hubPort": 4444,
    "hubHost": "<IP_ADDRESS>",
    "register": true,
    "registerCycle": 5000,
    "hub": "http://localhost:4444"
  }
}

However, when I switched to the jsonwire-grid I am getting this error from the grid console:
ERRO[0302] Can't find node by address[<IP_ADDRESS>:4723], storage: node not found
WARN[0302] api/proxy: Can't get node, Can't find node by address[<IP_ADDRESS>:4723], storage: node not found
WARN[0302] Can't unmarshal register json:  json: cannot unmarshal string into Go struct field Configuration.port of type int
INFO[0302] 500 POST /grid/register (<IP_ADDRESS>:60659) 0.000s

I am not sure where to start debugging since I have no clue how this grid has access to drivers or which version of selenium it is using (if it is using any).
Cheers!

@Bamieh Do u use real device or simulator? We have got https://github.com/qa-dev/jsonwire-grid-wda-agent for run iOS simulator. @podtserkovskiy Can u try or comment compatibility with real device?

@andrskom Yes I am using read devices. I have fixed the issue above by changing the configs to the following:
{
  "capabilities": [
    {
      "browserName": "iPhone 7 (iPhone9,3)",
      "appium-version": "1.7",
      "platform": "iOS",
      "version": "11.2",
      "deviceName": "iPhone (11.2.6)",
      "maxInstances": 1
    }
  ],
  "configuration": {
    "cleanUpCycle": 2000,
    "timeout": 30000,
    "proxy": "org.openqa.grid.selenium.proxy.DefaultRemoteProxy",
    "url": "http://<IP_ADDRESS>:2/wd/hub",
    "host": "<IP_ADDRESS>",
    "port": 2724,

    "maxSession": 1,
    "role": "node",

    "register": true,
    "registerCycle": 5000,
    "hubPort": 4444,
    "hubHost": "<IP_ADDRESS>"
  }
}

I removed "hub" and added "url" instead. Thanks a lot!


[RowActionCell] Colors don't adhere to design

Describe the bug
Colors were incorrect according the component design
See internal issue: https://github.ibm.com/wiotp/monitoring-dashboard/issues/1043
Environment/versions:

OS: MacOS
Browser: chrome
carbon-addons-iot-react version: v2.90

:tada: This issue has been resolved in version 2.95.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Apache custom log format

I saw there are already some questions about it, but I'm trying to make it work for about 2 hours, and I still don't see the light at the end of the tunnel.
I assume that you guys can do it in 2 min..
LogFormat "%t %{X-Forwarded-For}i %v %u "%r" %>s %b "%{Referer}i" "%{User-Agent}i""
Example:
[29/May/2018:15:15:51 +0300] <IP_ADDRESS> blabla.com - "POST /page/123 HTTP/1.1" 302 - "http://blabla.com/page/123" "Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/66.0.3359.181 Safari/537.36"
Thanks :)

This should do it :)
goaccess access.log --log-format='[%d:%t %^] %h %v %^ "%r" %s %b "%R" "%u"' --date-format=%d/%b/%Y --time-format=%T

Damn! You are good at your job 🥇
Thanks a lot, you saved me few more hours of frustration.

Glad to hear that did the job :)


regenerate credentials

Codecov Report

Merging #138 into master will decrease coverage by 0.03%.
The diff coverage is 33.33%.

@@            Coverage Diff            @@
##           master    #138      +/-   ##
=========================================
- Coverage   94.24%   94.2%   -0.04%
=========================================
  Files         104     104
  Lines        4862    4865       +3
  Branches       91      91
=========================================
+ Hits         4582    4583       +1
- Misses        280     282       +2

Impacted Files
Coverage Δ

modules/portal/app/controllers/VinylDNS.scala
68.18% <0%> (-1.05%)
:arrow_down:

modules/portal/app/models/UserAccount.scala
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

one thing i’m still trying to figure out: in the portal you can’t seem to regenerate your credentials multiple times in a row. You can do it once but then you need to refresh the page or go to a new page. I believe the session isn't updating with the regenerated access key on the angular side.

@britneywright you you force refresh the page via javascript after getting a success response?

@nimaeskandary I was trying to do it in an "angular way" but since that didn't work I'll use plain javascript.

latest CodeCov report: 94.26% https://codecov.io/gh/vinyldns/vinyldns/pull/138/diff
Coverage increased by 2.99% in VinylDNS.scala

Codecov Report

Merging #138 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #138      +/-   ##
==========================================
+ Coverage   94.24%   94.25%   +0.01%
==========================================
  Files         104      104
  Lines        4862     4878      +16
  Branches       91      100       +9
==========================================
+ Hits         4582     4598      +16
  Misses        280      280

Impacted Files
Coverage Δ

modules/portal/app/models/UserAccount.scala
100% <100%> (ø)
:arrow_up:

modules/portal/app/controllers/VinylDNS.scala
72.22% <100%> (+2.99%)
:arrow_up:

Continue to review full report at Codecov.

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

updated @pauljamescleary!


Add tests for REGEX operators

Thanks @marcosnils -- have you signed the CLA?

@otoolep just signed it :smile:

I will look into the go get failures.

Build script has been fixed, please rebase so CI testing can re-run.

Thanks @marcosnils


Fix and complete Boom (un)installation.

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.

@adidalal I don’t think this was supposed to be merged just yet (see https://github.com/Homebrew/brew/pull/1538). Made https://github.com/caskroom/homebrew-cask/pull/26939 for when it’s ready.

Whoops, didn't see that.


Add dashboard template

Add a dashboard.json template to manually import into NR1 dashboard.

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


EditForm: After you hit submit the first time, any new tabbing through fields tries to resubmit

Describe the bug
A clear and concise description of what the bug is.
When using the EditForm component, after the initial submission, if you tab through the fields again, every "Tab" action attempts a resubmission.
To Reproduce
Steps to reproduce the behavior:

Using this version of ASP.NET Core 3.0
Create and EditForm: (documentation)[https://docs.microsoft.com/en-us/aspnet/core/blazor/forms-validation?view=aspnetcore-3.0]. Create FirstName, LastName fields, etc.
Have your submit action do something where you stay on the page. For instance, I have a confirmation modal show.
Go back to a Field like "First Name", make a change to the content.
Tab over

For me, my Modal popup runs. This is caused only by the EditForm's OnValidSubmit attribute running my Submit function.
Expected behavior
A clear and concise description of what you expected to happen.
Tabbing through input fields should never submit a form.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Include the output of dotnet --info
dotnet --info results
.NET Core SDK (reflecting any global.json):
 Version:   3.0.100
 Commit:    04339c3a26

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.15063
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\3.0.100\

Host (useful for support):
  Version: 3.0.0
  Commit:  7d57652f33

.NET Core SDKs installed:
  1.1.14 [C:\Program Files\dotnet\sdk]
  2.1.202 [C:\Program Files\dotnet\sdk]
  2.1.508 [C:\Program Files\dotnet\sdk]
  2.1.509 [C:\Program Files\dotnet\sdk]
  2.1.701 [C:\Program Files\dotnet\sdk]
  2.1.801 [C:\Program Files\dotnet\sdk]
  2.1.802 [C:\Program Files\dotnet\sdk]
  2.2.300 [C:\Program Files\dotnet\sdk]
  2.2.301 [C:\Program Files\dotnet\sdk]
  2.2.401 [C:\Program Files\dotnet\sdk]
  2.2.402 [C:\Program Files\dotnet\sdk]
  3.0.100-preview8-013656 [C:\Program Files\dotnet\sdk]
  3.0.100-rc1-014190 [C:\Program Files\dotnet\sdk]
  3.0.100 [C:\Program Files\dotnet\sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.All 2.1.12 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.1.13 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.2.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.2.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.12 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.1.13 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.2.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.2.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 3.0.0-preview7.19365.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 3.0.0-preview8.19405.7 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 3.0.0-rc1.19457.4 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 1.0.16 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 1.1.13 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.0.9 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.12 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.13 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.2.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.2.6 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.2.7 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.0.0-preview7-27912-14 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.0.0-preview8-28405-07 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.0.0-rc1-19456-20 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 3.0.0-preview7-27912-14 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 3.0.0-preview8-28405-07 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 3.0.0-rc1-19456-20 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

I haven't tested this fully yet, but I think this was also occurring with just a plain html <form> tag used within Blazor Components. I had an issue a few weeks ago, and this may have been a related issue.

Thank you for filing this issue. In order for us to investigate this issue, please provide a minimalistic repro project (ideally a GitHub repo) that illustrates the problem.

Hi. We're closing this issue as we have heard no response from you for some time now. If you have more details and are encountering this issue please add a new reply and re-open the issue.

@mkArtakMSFT
Here's a repo: https://github.com/kevingates/Blazor_EditForm

@kevingates I haven't dug in to this super well, but it looks like a bug in your code. ShowModal is set when you submit -
https://github.com/kevingates/Blazor_EditForm/blob/master/EditFormBug/EditFormBug/Pages/Index.razor - but nothing resets it. Once you've submitted the form, any time the Index component is rendered, the modal is still meant to be displayed. I'd have a look at how the modal component is built in the BlazingPizza sample - https://github.com/danroth27/BlazingPizza/ if you need to figure out a pattern for this.

@kevingates I haven't dug in to this super well, but it looks like a bug in your code. ShowModal is set when you submit -
https://github.com/kevingates/Blazor_EditForm/blob/master/EditFormBug/EditFormBug/Pages/Index.razor - but nothing resets it. Once you've submitted the form, any time the Index component is rendered, the modal is still meant to be displayed. I'd have a look at how the modal component is built in the BlazingPizza sample - https://github.com/danroth27/BlazingPizza/ if you need to figure out a pattern for this.

@pranavkm
You are absolutely correct. I have the modal closing itself within the modal's code, but since the parent is only having its state updated (versus being reinitialized) you're right that I am have it re-shown. I have my callback function change my "modalSettings" to false after I hear back and it fixed the issue.
First, I apologize for posting a bug about my own mistake.
Second, I sincerely appreciate your help in pointing this out and for the Blazing Pizza example. I was looking for his code recently. I will bookmark this and maybe adopt his modal instead.
I will go ahead and close this issue.
Many thanks.


is this necessary to call fileWatchService's shutdown method twice?

there are two places in BrokerController.java that fileWatchService's shutdown method is called :
line 1276:1278

and line 1357:1359

i think the second call is unnecessary because the second call will do nothing as variable started is false in ServiceThread

so we can delete line 1357:1359 safely

I agree with you.


Mark broadcast notifications in the list

Hey, I just made a Pull Request!
This patch adds a little icon to mark a broadcast notifications in the list.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)


How to calculate average recall (AR) of each class

Hi,
After completing each epoch I am getting the following results, but it shows the average recall of all classes. I want to check the average recall of each class. Can anyone please guide me on how to do this?
Thanks.
I was using 5 classes and got the following results.
Average Recall (AR) @[ IoU=0.50:0.95 | area= all | maxDets=100 ] = 0.320
Average Recall (AR) @[ IoU=0.50:0.95 | area= all | maxDets=300 ] = 0.320
Average Recall (AR) @[ IoU=0.50:0.95 | area= all | maxDets=1000 ] = 0.320
Average Recall (AR) @[ IoU=0.50:0.95 | area= small | maxDets=1000 ] = 0.128
Average Recall (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=1000 ] = 0.303
Average Recall (AR) @[ IoU=0.50:0.95 | area= large | maxDets=1000 ] = 0.341

@ibrahim1611 python test.py xxxx --eval-options evaluation.classwise=True

@hhaAndroid It only shows the average precision (AP) of each class.

@ibrahim1611 Sorry. Currently no support.

@ibrahim1611 Sorry. Currently no support.

so how to do it ?

Option 1:
Go to mmdetection/mmdet/evaluation/metrics/coco_metrics.py  file and change classwise: bool = False to classwise: bool = True in COCOmetric initializer. This will create mAP values of different thresholds (same format as general mAP information) for each class as follows:

If you want to obtain AR values in the same way, you can go to line 520
if self.classwise:  # Compute per-category AP
               precisions = coco_eval.eval['precision']

Option 2:

Implement classwise precision and recall functions (taken from official GroundingDINO implementation)
import itertools
from tabulate import tabulate

def per_class_AR_table(coco_eval, class_names=None, headers=["class", "AR"], colums=6):
    per_class_AR = {}
    recalls = coco_eval.eval["recall"]
    # dimension of recalls: [TxKxAxM]
    # recall has dims (iou, cls, area range, max dets)
    assert len(class_names) == recalls.shape[1]

    for idx, name in enumerate(class_names):
        recall = recalls[:, idx, 0, -1]
        recall = recall[recall > -1]
        ar = np.mean(recall) if recall.size else float("nan")
        per_class_AR[name] = float(ar * 100)

    num_cols = min(colums, len(per_class_AR) * len(headers))
    result_pair = [x for pair in per_class_AR.items() for x in pair]
    row_pair = itertools.zip_longest(*[result_pair[i::num_cols] for i in range(num_cols)])
    table_headers = headers * (num_cols // len(headers))
    table = tabulate(
        row_pair, tablefmt="pipe", floatfmt=".3f", headers=table_headers, numalign="left",
    )
    return table

def per_class_AP_table(coco_eval, class_names=None, headers=["class", "AP"], colums=6):
    per_class_AP = {}
    precisions = coco_eval.eval["precision"]
    # dimension of precisions: [TxRxKxAxM]
    # precision has dims (iou, recall, cls, area range, max dets)
    assert len(class_names) == precisions.shape[2]

    for idx, name in enumerate(class_names):
        # area range index 0: all area ranges
        # max dets index -1: typically 100 per image
        precision = precisions[:, :, idx, 0, -1]
        precision = precision[precision > -1]
        ap = np.mean(precision) if precision.size else float("nan")
        per_class_AP[name] = float(ap * 100)

    num_cols = min(colums, len(per_class_AP) * len(headers))
    result_pair = [x for pair in per_class_AP.items() for x in pair]
    row_pair = itertools.zip_longest(*[result_pair[i::num_cols] for i in range(num_cols)])
    table_headers = headers * (num_cols // len(headers))
    table = tabulate(
        row_pair, tablefmt="pipe", floatfmt=".3f", headers=table_headers, numalign="left",
    )
    return table

and call them as follows in the same lines:
if self.classwise:  # Compute per-category AP
               info = ""
               AP_table = per_class_AP_table(coco_eval, class_names=cat_list)
               info += "per class AP:\n" + AP_table + "\n"

               AR_table = per_class_AR_table(coco_eval, class_names=cat_list)
               info += "per class AR:\n" + AR_table + "\n"

               print(info)

This solution will create the following output:


kubectl-gs should not set worker instance type or explicitly specify nodePools value if no customization parameters given

This code sets m5.xlarge, so the cluster-aws default is useless.

cc @yulianedyalkova this was the issue i mentioned last week.


List Page header - Hov text for "Group by" is not available

Validated on master-head - commit id: 16616de9c

List page header does not have Hov text for "Group by"

2.5.2
Rerpo steps.

Cluster Explorer, select namespace page,
Notice there are two buttons to list the namespaces by flat list or group by, but currently, if you mouse hovers over these buttons, the label for these buttons are missing

Verified the fix on Verified the fix 2.5-head 9ee39b9, and mater-head, a549b47.
These buttons now have

appropriate labels.


Test job guard

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

/test pre-main-reconciler-protecode-guard


Being able to disable prom annotations

What was changed
Default behavior should be to have the annotations disabled at the service level.
Note that it is already enabled at the global level by default:
server:
  metrics:
    annotations:
      enabled: true

Why?
That makes easier to configure if you use the prometheus operator for instance

Do you need more info?

This is backwards incompatible, you are changing the defaults such that new users would not get the annotations. You aren't considering the top-level setting.


accomodation of proteiomics

Many of the problem we observed today is due to the non-standard pre-processing of the proteomics data. Now the proper support for proteomics are added. The gene_annotation notebook, annot_coord_protein step shall generate a bed file with #chr start end ID and sample names as columns, just like all other bed.gz files, with following features:

The #chr will have "chr" prefix to the number
The ID is in the form of chr_gene.id_uniportID

RE the FIXME on susie. It is near impossible to unify the susie complete data interface with the susie RSS interface due to the drastic difference in the input they require and the way tasks are parallelized.


RF code not displayed in Raspibian Chromium

When learning the RF code, the code is not displayed when using Raspibian's Chromium.  When using Firefox, the code shows up in the status box on the actions screen.

I'm ussing Google Analytics to mesure the usage of Broadlinkl Manager.
If you are using Chrome, somtimes there is script error that causing an error.
will try to fix it


Can I directly zip two LoRAs trained by sd-script?

I didn't use the train_dreambooth_lora_sdxl  scripts provided by this project for training. Instead, I chose two models that I previously trained using Kohya sd-scripts.
When I ran the train_dreambooth_ziplora_sdxl , it showed
KeyError: 'unet.unet.down_blocks.1.attentions.0.transformer_blocks.0.attn1.to_q.lora.up.weight."

I didn't use the train_dreambooth_lora_sdxl  scripts provided by this project for training. Instead, I chose two models that I previously trained using Kohya sd-scripts.
When I ran the train_dreambooth_ziplora_sdxl , it showed
KeyError: 'unet.unet.down_blocks.1.attentions.0.transformer_blocks.0.attn1.to_q.lora.up.weight."

replace prefix  from "unet.unet"  to "unet."  in ziplora_pytorch/utils.py: merge_lora_weights function , in   line42

@wuliebucha I don't think it's that easy, the kohya key format is completely different, using underscores instead of .

@wuliebucha @Xynonners tried both, and it seems neither worked 😂. I printed tensors.keys() in this function and they seem to be in a format like lora_unet_input_blocks_7_1_transformer_blocks_0_attn1_to_k.lora_down.weight. It might require a more complex formatting.

@wuliebucha @Xynonners tried both, and it seems neither worked 😂. I printed tensors.keys() in this function and they seem to be in a format like lora_unet_input_blocks_7_1_transformer_blocks_0_attn1_to_k.lora_down.weight. It might require a more complex formatting.

yeah, there are no SDXL LoRA interops between diffusers and kohya, I've searched pretty much everywhere.
The format is completely different, down to the numbers in the keys.

merged_lora_weights_dict = merge_lora_weights(lora_weights, attn_name,prefix='unet.')
merged_lora_weights_dict_2 = merge_lora_weights(lora_weights_2, attn_name,prefix='unet.')

@xiaohaipeng Have you tried it on your side？ I have tried this but it still shows KeyError.

I tried format with reference to this sd-scripts codes and although I totally don't know the implementation details, it does work 😂.
The training stopped at 100% and it shows requires too much memory for my machine (A10, 24G) to finish.

@sunhengzhe have you tried to merge 2 models trained with train_dreambooth_lora_sdxl? Is it also running out of memory? Do you know what rank parameter was the lora that you're trying to merge was trained with? maybe that's affecting the amount of memory needed for the merge.

@pedropaf Sorry for lately reply, I modified unet = unet.to(torch.float32) to unet = unet.to(torch.float16) in train_dreambooth_ziplora_sdxl.py and it can be merged, but I didn't reproduce the paper. I will keep testing.


atq/viz 1985

From the update baselines
it looks for when there is only space for one line of text
this change makes it only show the cell value and not the cell label. I think that is wrong

Is test/rendering/baselines/multi-scenario-4-baseline.png and test/rendering/baselines/multi-scenario-5-baseline.png correct?
It looks like some of the overlay labels still overlap? Is that something that is included in this change?
Also just so that you are aware. It is possible to use Tobias github action script to update the rendering tests.

These were generated locally, when I used yarn run test:rendering, and this fix dosn't include overlay labels, its only for  labels and values in a tree node. So the overlay labels are still the same way.

From the update baselines it looks for when there is only space for one line of text this change makes it only show the cell value and not the cell label. I think that is wrong (see: dover in multi-scenario-2-baseline.png)

It is the same way the old treemap works : both are identical now, when it comes to showing the tree node labels & values : for Dover its hows only value if value labels are toggled ON.
Attached is a screenshot :

It is the same way the old treemap works

OK


Update import paths, examples and docs with new module name

Closes #92

Thanks @webstradev!

Isn't this breaking existing users?

Isn't this breaking existing users?

If the previous module is still at the same address it won't break for them, it just means they won't be able to get newer versions from that address.

Yes, but that's the price we pay for moving the repository to a different organization.
The error message is pretty helpful, though:
penberg@vonneumann xxx % go run main.go
main.go:4:2<EMAIL_ADDRESS>parsing go.mod:
	module declares its path as: github.com/tursodatabase/libsql-client-go
	        but was required as: github.com/libsql/libsql-client-go

@haaawk or are you thinking of some other solution to this?

You could fork the commit before this was merged in and leave it at the old address as well i f that org is still in your control

Yes, but that's the price we pay for moving the repository to a different organization.
The error message is pretty helpful, though:
penberg@vonneumann xxx % go run main.go
main.go:4:2<EMAIL_ADDRESS>parsing go.mod:
	module declares its path as: github.com/tursodatabase/libsql-client-go
	        but was required as: github.com/libsql/libsql-client-go

@haaawk or are you thinking of some other solution to this?
We could leave the package name unchanged for some time maybe.

Would it not be better to keep the older version on the old URL and than add a message that this repository is continued at the new url. That way you have backwards compatibility.
If we leave the package name unchanged for some time we just move the problem to later down the line right

@webstradev We of course have control over the original organization, but I am hesitant to make fork there because we may end up driving people to use the old version for new applications.
@haaawk I am not sure what leaving the old package name buys us because at some point in the future, we do want to move away from the libsql organization, then breaking apps just like we do now.

Can we tag an older commit (pre this mr) with a version that people could then use.
I think github will handle the redirect from old to new and then the go module would match the import path as well.
Might be worth a try.
Not sure if it will actually fix the breaking issue because new users would probably still need to pin the use of thatt specific version but just an idea.

@penberg I don't have the answer really. I guess we will see soon how many people are affected.

@penberg @haaawk Might I suggest we tag this on main, so that we can easily instruct users to update to a minimum of version X.
I'm happy to do some quick digging to see who is using this and open PR's on the respective repositories.


feature request: debrid support

Could you add debrid support to aria2 for downloading from file sharing sites? Or is there a dirtier solution to this?
I'm using alldebrid with jd2 but I prefer aria2 as download manager.

This is not aria2 repository but one of its WebUI.
Also, aria2 already provided an RPC interface, which can be used as a backend for any download link converter.


yamlscript: 0.1.72 -> 0.1.73

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/yaml/yamlscript/releases.
meta.description for yamlscript is: Programming in YAML
meta.homepage for yamlscript is: https://github.com/yaml/yamlscript
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.1.73 with grep in /nix/store/p2pinwvwfcnf0ak55952z2wzvrl7bd48-yamlscript-0.1.73
found 0.1.73 in filename of file in /nix/store/p2pinwvwfcnf0ak55952z2wzvrl7bd48-yamlscript-0.1.73

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

yamlscript

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/p2pinwvwfcnf0ak55952z2wzvrl7bd48-yamlscript-0.1.73 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A yamlscript https://github.com/r-ryantm/nixpkgs/archive/4750b9e80e67d26eddeeb0aa50daf7a42ea75781.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/4750b9e80e67d26eddeeb0aa50daf7a42ea75781#yamlscript

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/p2pinwvwfcnf0ak55952z2wzvrl7bd48-yamlscript-0.1.73
ls -la /nix/store/p2pinwvwfcnf0ak55952z2wzvrl7bd48-yamlscript-0.1.73/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

yamlscript

Maintainer pings
cc @stigtsp for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge


--nolivereload not working

From @rashnk on May 4, 2017 13:2
Short description of the problem:
I tried this, ionic serve --nolivereload but it does not seem to be working, it still detects changes and reloads the browser.
What behavior are you expecting?
when file change is detected, do not build and do not reload
Steps to reproduce:

run ionic serve --nolivereload
Make changes in some .ts or .html file

Ionic Info
Your system information:
ordova CLI: 6.4.0
Ionic Framework Version: 3.1.1
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.3.4
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 7
Node Version: v7.7.4
Xcode version: Not installed
Copied from original issue: driftyco/ionic-app-scripts#962

From @rashnk on May 6, 2017 10:27
how to disable live reload when run ionic serve ? i hope someone in ionic team will look into this issue

@jgw96 This is an app-scripts issue.


Need image_id option in Edit Products page for Amazon VM

https://github.com/booz-allen-hamilton/jellyfish-ux/issues/151

This is just a ProductType setting that needs to be added.

Shouldn't have closed the API ticket (just the UX one)

No longer valid, this is an issue for the jellyfish manage iq module.


bump pydantic to v2

Would be it be possible to bump pydantic to v2 as ray [serve] is now compatible with pydantic v2? See Ray PR.
Follow up on issue #423 and PR #429.

Hi @jens-create Thanks for bringing this up! We will definitely consider bumping up the version once they bump up the pydantic version in Ray.

Is there a follow-up on this?

Just an FYI, Ray just released their 2.9.0 version, which adds support for Pydantic >=2.5.0
https://github.com/ray-project/ray/releases/tag/ray-2.9.0
CC @WoosukKwon

Hi all,
That would be quite good to upgrade, less and less projects will expect pydantic 1.x so the pin on 1.10 on vllm is a becoming a headache.

bumping this! there is an open PR for it here: https://github.com/vllm-project/vllm/pull/2322


Remote Combination of Search and Filter

Hi Alen,
I am writing remote method to achieve the search and filter using the examples available in the docs, it works fine when they are done as standalone, however when i first search and then filter, am not sure how to find the search data(if the table is searched) in the filter function to start with in your example we always start filter from complete data. I did try to start the filter with state data but that doesn't work as it would have changed depending on filter values over time,
similar case for the other way around filter and search
could you help on this, please?

Hello, sorry for lately reply, and did you enable the remote mode on BootstrapTable?

Hi @AllenFang Thanks for your response, yes i did and they works fine as standalone search and standalone filter
In your example
  onSearchChange(searchText, colInfos, multiColumnSearch) {
    const text = searchText.trim();
    if (text === '') {
      this.setState({
        data: this.products
      });
      return;
    }

    let searchTextArray = [];
    if (multiColumnSearch) {
      searchTextArray = text.split(' ');
    } else {
      searchTextArray.push(text);
    }

    const data = this.products.filter((product) => {
      const keys = Object.keys(product);
      let valid = false;
      for (let i = 0, keysLength = keys.length; i < keysLength; i++) {
        const key = keys[i];
        if (colInfos[key] && product[key]) {
          const { format, filterFormatted, formatExtraData, searchable, hidden } = colInfos[key];
          let targetVal = product[key];
          if (!hidden && searchable) {
            if (filterFormatted && format) {
              targetVal = format(targetVal, product, formatExtraData);
            }
            for (let j = 0, textLength = searchTextArray.length; j < textLength; j++) {
              const filterVal = searchTextArray[j].toLowerCase();
              if (targetVal.toString().toLowerCase().indexOf(filterVal) !== -1) {
                valid = true;
                break;
              }
            }
          }
        }
      }
      return valid;
    });
    this.setState({
      data: data
    });
  }

I think whenever a search key is entered you always start with the complete data as i assume this.products holds the complete data
Let's say I did a search for few characters and the resulting data is less than this.products
and now i want to filter on this data by column in the table, in the example below the filtering starts with this.producs which is not the result of search
  onFilterChange(filterObj) {
    if (Object.keys(filterObj).length === 0) {
      this.setState({
        data: this.products
      });
      return;
    }

    const data = this.products.filter((product) => {
      let valid = true;
      let filterValue;
      for (const key in filterObj) {
        const targetValue = product[key];
        switch (filterObj[key].type) {
        case 'NumberFilter': {
          filterValue = filterObj[key].value.number;
          valid = this.filterNumber(targetValue, filterValue, filterObj[key].value.comparator);
          break;
        }
        default: {
          filterValue = (typeof filterObj[key].value === 'string') ?
            filterObj[key].value.toLowerCase() : filterObj[key].value;
          valid = this.filterText(targetValue, filterValue);
          break;
        }
        }

        if (!valid) {
          break;
        }
      }
      return valid;
    });
    this.setState({
      data: data
    });
  }

I could start with state data but that works only for the first time, every time it does filter from search on it should filter based on the search data

Hello @supiash1, very sorry for lately reply :) ok, let going to your case,
Actually, in the mode with remote, react-bootstrap-table will handle your case well, which means, when you search firstly and then filtering by column, react-bootstrap-table will do the filter base on the result of previous search. So how react-bootstrap-table to achieve that? Not very hard, react-bootstrap-table  have your all data and also save your filtered data  again, which means, in the internal, we save your original data and also have any array to save the filtered data(search + filter).
For the remote mode, my purpose is let you to decide how to handle/operate your data, so that react-bootstrap-table now is a stateless table, it only accept the data and show it, for any action(sort/filter/paging), they should be handle by you.
So back to your case, you can have an instance variable or a state to keep the last search/filter data so that you can do any action base on last data.
BTW, do you know the remote have a tricky part which is you can control which functionality need to perform as remote or not. you can check this example.
in that example, it tell the react-bootstrap-table only cellEdit, insertRow and dropRow need to handle as remote but other functionality should keep to handle by react-bootstrap-table. For me, I usually recommend to disable some functionality which will not interact with backend, for example, insert, delete, etc. but for the sorting, filter, search, it's seems like can be don be react-bootstrap-table if you operation will not interact with backend server. so this example show you how to disable/enable some functionality with remote
Back to you example, after I check your code, in the filtering or search, you seems like handle the data synchronize, no any AJAX call or something, just data change in local, so maybe you can thing about to let react-bootstrap-table to handle search/filter and maybe make insert/delete/edit to use the remote mode if you need to sync data with backend
Sorry for a long story with my pool english, hope you can understand what I talk, and please let me know your concern to questions, thanks

@AllenFang Thank you very much for the explaination, I am using only remote filter and search using  the remoteobj,
I could have let the reactbootstrap table do the search but  onSearchChange needs to behave differently as am trying to to do search across expand tables as well
Thanks for your idea i am thinking

Hold the SearchData & FilterData as a variable in the state everytime it search or filter
Whenever we search use the FilterData if it set else the complete data similarly for filter

Thanks

Hold the SearchData & FilterData as a variable in the state everytime it search or filter

maybe you can consider to save only the rowkeys for search/filtered data :)
Please let me know if still have any question or concern, thanks


Large transaction fails with "missing signature"

Hey there,
I was experimenting with my sharedrop python script for meetups and successfully created paperwallets with 10 preloaded addresses .. preloading happens in a single transaction. Hence, 2 withdrawal and 10 deposit operations.
However, I also tried with 100 addresses (http://pastebin.com/raw.php?i=5mFKw2wT) and just got a "missing signature" assertion..
{u'message': u'missing signature (31005)\n\n\n\n\n\n\n\n\n',
u'code': 31005,
u'detail': u'31005 missing_signature: missing signature\n\n    {"owner":"BTSQDikgmCjCWxxKjweq8TnrK9JG5kDePdFr"}\n    th_a  balance_operations.cpp:197 evaluate\n\n    {"*this":{"balance_id":"BTSLtTf5qG2gbGSMRzubGHJfz3V13Hq8izJn",
    "amount":100000,
    "claim_input_data":""}}\n    th_a  balance_operations.cpp:266 evaluate\n\n    {"op":{"type":"withdraw_op_type",
	    "data":{"balance_id":"BTSLtTf5qG2gbGSMRzubGHJfz3V13Hq8izJn",
		    "amount":100000,
		    "claim_input_data":""}}}\n    th_a  operation_factory.hpp:52 evaluate\n\n    {"op":{"type":"withdraw_op_type",
	    "data":{"balance_id":"BTSLtTf5qG2gbGSMRzubGHJfz3V13Hq8izJn",
		    "amount":100000,
		    "claim_input_data":""}}}\n    th_a  transaction_evaluation_state.cpp:243 evaluate_operation\n\n    {"trx_arg":{"expiration":"2015-03-26T05:58:51",
			    "reserved":null,
			    "operations":[{"type":"withdraw_op_type",
	    "data":{"balance_id":"BTSLtTf5qG2gbGSMRzubGHJfz3V13Hq8izJn",
		    "amount":100000,
		    "claim_input_data":""}},

What's the max size for a transaction?
What can be the reason for the "missing signature"?
Could someone label this issue as "minor"?

Fixed the issue. Wasn't related to size but to the zigzag/varint encoding of the asset_id:
https://github.com/xeroc/python-bitsharestools/commit/05b6fbaf004093e27afdbc67fef3b2213dd65634


refactor: provide a common reconciler

@lburgazzoli this LGTM, but just to be clear this is aligning reconcilers into just one?

@lburgazzoli this LGTM, but just to be clear this is aligning reconcilers into just one?

no, it provides a base reconciler that the specific reconciler extends


Bump meow from ^5.0.0 to ^6.1.1

As described here: https://github.com/mapbox/polyline/issues/63

Thanks for merging. Do you plan to release it? The thing is that we're getting npm audit errors in our CI.

Thanks for merging. Do you plan to release it? The thing is that we're getting npm audit errors in our CI.

If you want to put together a release PR, I can review and we can ping Tristen to also review and publish to npm.
However if we are going to have both a package-lock.json and yarn.lock we should try to ensure they are in-sync before releasing.

Sure, i will prepare PRs for both release & yarn.lock. Btw, do you think it's a good idea to have both lock files here? May be we just need to keep one of them (not sure which one though)?


Integrating MongoDB with Chatterbot Django Example

Is there a working example of integrating MongoDB with the Chatterbot Django example? I am having issues. I have tried using the MongoDB Engine and Django-NoRel and was successful to a point, but I am unable to get the admin page working. Can someone provide an example or offer some advice regarding getting the admin page to work with MongoDB?
Thanks!

@rdt712 Currently there is no working example, I will tag this is an future request. I found some references on Djanog mongo db https://github.com/django-nonrel/mongodb-engine


Repulsion prior documentation

Hi,
There are inconsistent descriptions of RepulsionLayer comments and the actual calculation formula.
U_repulsion_ij = (sigma_{ij} / r_{ij}) ^ exp_{ij}
and
energy = torch.sum((self.repulsion_parameters[0, :]/in_feat)
                           ** self.repulsion_parameters[1, :],
                           1).reshape(n, 1) / 2

Thanks for finding this! The repulsion prior is very simple, so for decent initial repulsion parameter choices I believe the network should be able learn corrections to the prior if needed. Regardless, the factor of two should not be there. I will add it to my list of things to do.


google fonts

fonts.googleapis.com
Instead of calling Google Fonts externally from the web source, what if you include them in the source?

Added :)


[libcpplocate] New port

New port. Also opened an issue on the project's GitHub https://github.com/cginternals/cpplocate/issues/47

What does your PR fix?
N/A

Which triplets are supported/not supported? Have you updated the CI baseline?
~Fails compilation on UWP. Waiting for response from maintainers to see if we want to investigate.~ ~Fixed with respective patch.~ Patch was reverted due to myself lacking knowledge about UWP. UWP is now marked as unsupported

Does your PR follow the maintainer guide?

Yes

Ready for review now?

Ready for review now?

@JackBoosY Yes! Thank you.

I was just double-checking some things and now I'm not sure what should be the correct version listing...
The most recent tag v2.2.0 is here https://github.com/cginternals/cpplocate/commit/01fa9d592e4b84d16d33009f4f977e5e60eab496 but the META_VERSION_PATCH is listed as 1, making the full version 2.2.1, which doesn't seem correct. I would have expected the v2.2.0 tag to point to the parent of the referenced commit to mark it as the last commit with that version number, right?
On the other hand, their published Ubuntu packages have the latest release as 2.3.0 (like this vcpkg PR) uses the commit that I am using for vcpkg 2.3.0 (https://github.com/cginternals/cpplocate/commit/23aeb69c23848ff52456179374b9dfc1ede357bf), but the branch named v2.3 is a different commit.
@scheibel (who looks to be the maintainer of cpplocate) could you help clear this up please.

Thanks for pursuing an integration into vcpkg!

I suggest you use the v2.3.0 release that I created just know.

@scheibel thank you for creating that tag and explaining the versioning for cpplocate. I appreciate your time and response(s) 😄

@JackBoosY I have pushed new commits for vcpkg to use the latest tag (v.2.3.0), so this is now ready for another review. Thank you!
I'm not sure what is making CI fail. Should I do anything to fix that?

Looks all passed, our pipeline test has some bugs.

Thanks @ekilmer for the PR and thanks @scheibel for making such an awesome library :)


Updated dockerfile

Overview:
Modified  Docker image to cbica/captk:190rc.
The Latest and 190rc tags have same size and same commit id.
Connects #3
Checklist:
docker file
Testing instructions:

Pull the code from this repo .
open docker file and check for : FROM cbica/captk:190rc

please update with a PR template format

please update with a PR template format

I have updated the PR with the template.


feat: add tirg image encoder

@bwanglzu @JoanFM emb mismatch is back

@bhavsarpratik can u try the following?
Can u copy this into jina subfolder jina/hub ... and without installing these dependencies run jina check . See if it complains about this Executor not being properly loaded?

@JoanFM It's not complaining

I think there is a problem with our builder


Rancher download script is not working

if I run the script "curl -sfL https://get.rancher.io | sh - " it crashes.
This link is causing the problem"https://github.com/rancher/rancher/releases/download/v2.6.0/rancherd-amd64.tar.gz"

System info: ubuntu 18.04

https://github.com/rancher/rancher/issues/33423

Also described on https://github.com/rancher/rancher/releases/tag/v2.6.0


Disconnect message bus from project selector on component removal

Fixes #1881.

so having this in addition to a disposable (for the cases where we use disposables) won't matter (no interference or anything)?

I think message bus connection is local to the code that calls connect() so it sort of "owns" its lifetime. I looks to be working fine so far.


Add a minimal README

clarify that this is just a library

:+1:
Please add more docs to this tool.

in progress and duplicate to https://github.com/sphereio/sphere-product-import/issues/13

@nkuehn please have a look https://github.com/sphereio/sphere-product-import/tree/documentation
suggestions if any


need function to get TranspileConfig for a backend

What is the expected enhancement?
The qiskit.compiler.transpile() function is to be decoupled from a monolithic backend, in favor of transpiling for some TranspileConfig options (basis_gates, coupling_map, initial_layout, optimization_level, etc.)
Still, we need a convenience function that makes it easy to get these TranspileConfig options from a backend.
transpile_config = qiskit.compiler.default_transpile_config(backend)

this would populate a transpile_config = TranspileConfig(basis_gates, coupling_map), and the user still has the option to add more fields like transpile_config.seed_mapper = 10.

@jaygambetta in your PR you wrote this function as:
[run_config, transpile_config] = default_compiler_configs(backend)

but what does it mean to return a run_config? if there's not an overwhelming usecase for this I'd rather keep the function returning a single output for ease of use.

In my opinion this idea of throwing everything into a config object just makes our software harder to use.  The user has to learn yet another object.  There is some benefit for having less used options in a config, but things like coupling_map and basis_gates should not be in the config class.  The compile should probably look more like:
compile(circuits, coupling_map, basis_gates=None,
        initial_layout=None, shots=1024, config=TranspileConfig())

where everything else is in the config.  Then the backend instance should allow for doing backend.coupling_map and backend.basis_gates since they are fundamentally the key things you want.  Then, in the simplest case you can do:
backend = IBMQ.get_backend('ibmqx4')
compile(circuits, backend.coupling_map, backend.basis_gates)

@ajavadia, Is a RunConfig not something like the number of shots, rep_times, etc.? It can be used to guarantee that two different compiles will create Qobjs that run with the exact same runtime settings?
@nonhermitian, I think promoting specific configuration variables too first level could set a dangerous precedent for future variables.

PR #2166 closes this issue "by making it moot".


Internalize ecstatic library

Ecstatic is no longer maintained, and so we cannot rely on any future updates. This PR internalizes it as the core module.
This PR also pulls in all of ecstatic's tests so we don't lose that coverage. Unfortunately these were written with tap while http-server's tests are written with vows. I don't have a strong preference for one over the other, but these need to be reconciled before this PR can be merged. I'm leaning toward converting to tap because that means rewriting only one file.
TODO:

[x] Consolidate test files to use either tap or vows, or both.
[ ] Ensure all tests run as expected

Fixes #568

In the interim, what about a minor version bump in the dependencies to sidestep a security concern?
https://snyk.io/test/npm/ecstatic/3.3.2

Remediation: Upgrade to<EMAIL_ADDRESS>

The PlayCanvas Engine repo has had a Dependabot warning about this for maybe 9 months now. I guess I could just patch the package-lock.json. But yeah, would be great if http-server bumped its ecstatic dependency to 4.1.3, as @reergymerej suggests. Any objections to me submitting a PR for that?

The PlayCanvas Engine repo has had a Dependabot warning about this for maybe 9 months now. I guess I could just patch the package-lock.json. But yeah, would be great if http-server bumped its ecstatic dependency to 4.1.3, as @reergymerej suggests. Any objections to me submitting a PR for that?

Sorry I got pulled away into other life things, I need to get back to this PR. Unfortunately ecstatic v4 has some major incompatibilities with http-server (a product of the two projects being far too closely coupled for so long), so simply bumping the dependency does not work. The goal of this PR is to remove the dependency entirely, fix the vulnerability internally, and go from there.
This PR just needs functioning tests right now if anyone has the time to take a look at them, I don't right now but will get back to it when I can.

Hi @thornjad, is there something you will need help with, then I will like to help out :)

@thornjad is this just about fixing the failing tests listed here https://travis-ci.org/github/http-party/http-server/jobs/762178038 ? If someone submitted a PR with a fix, what are the chances any maintainer would look at it?

Yeah tests passing is the only thing holding this up. I just haven't been able to find any time, a PR would definitely be welcomed and looked at.

Closing this draft in favor of #693


Fix dubbo.tag override by consumer.tag in NacosRegistry

Fix issue  https://github.com/apache/dubbo/issues/13184
(dubbo.tag override by consumer.tag in NacosRegistry, which causes TagStateRouter not worked properly when provider using interface-level registry.)

@AlbumenJ PTAL

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

Comparison is base (180ff97) 70.39% compared to head (6fffe8a) 70.37%.

Additional details and impacted files
@@            Coverage Diff             @@
##              3.2   #13592      +/-   ##
==========================================
- Coverage   70.39%   70.37%   -0.02%
==========================================
  Files        1606     1606
  Lines       69987    69988       +1
  Branches    10097    10097
==========================================
- Hits        49266    49256      -10
- Misses      16089    16102      +13
+ Partials     4632     4630       -2

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


[BUG] SRTM_3arcsec_uncropped.tif not recognized as supported file

Invalid API key
When I ran the ariaTSsetup.py on Anaconda prompt to process netcdf (.nc) files to make time series data. The DEM file which is an output from a code within the script gives a runtime error. When I check the DEM file by opening it on a notepad there seems to be an invalid API key ERROR.
To Reproduce
Steps to reproduce the behavior:

Follow the setup process as mentioned on the github page for aria-tools on an anaconda prompt command line
After installation go to ARIA-tools>tools>bin folder and run python ariaDownload.py -w (paste working directiory Nisar>L2_interseismic folder) -u (username) -p (password) -b (houston area bounding box) -m 30 -l 36 -s 20180101 -e 20211231 -o Download
After Downloading rub python ariaTSsetup.py -f (paste L2_interseismic folder) --mask Download
The image of the error is attached

OS- WINDOWS 10
Mode of install and use- anaconda prompt and jupyter notebook
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

ARIA-tools git tag: [e.g. git show --summary]
OS

Additional context
Add any other context about the problem here.

You haven't configured your api key for open topo. Follow installation instructions under heading: REQUIRED: Acquire API key to access/download DEMs


elf-dissector: init at unstable-2020-11-14

Motivation for this change
Nice tool for inspecting binaries and finding linking problems. Feels like an internal tool for KDE development, but useful to us.
https://invent.kde.org/sdk/elf-dissector
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 102048 run on x86_64-linux 1

1 package built:

elf-dissector