How to do components efficiently?

Probably I'm just missing something obvious, but given I have a component tree like:
           |---|
           | R |
           |---|
             |
      |------|------|
      |             |
    |---|         |---|
    | A |         | B |
    |---|         |---|
      |             |
  |---|---|     |---|---|
  |       |     |       |
|---|   |---| |---|   |---|
| C |   | D | | E |   | F |
|---|   |---| |---|   |---|

If my components look like:
function ComponentR (state) {
  return h('div', [state.R,ComponentA(state),ComponentB(state)])
}
function ComponentA (state) {
  return h('div', [ state.A,ComponentC(state),ComponentD(state)])
}
function ComponentB (state) {
  return h('div', [state.B,ComponentE(state),ComponentF(state)])
}
function ComponentC (state) { return h('div', state.C) }
function ComponentD (state) { return h('div', state.D) }
function ComponentE (state) { return h('div', state.E) }
function ComponentF (state) { return h('div', state.F) }

I might be able to integrate some kind of event listener or other mechanism into each components to receive updates from user interaction or data updates from a server, but by relying on the return statement, I guess I would always need to re-render the whole component-tree
What I imagine is, that given any update, only certain components need to re-render.

If the update is just affecting a "leaf component", it would just need to re-render itself.
If the update is affecting the root - the tree might be re-rendered, but some components between the "root" and the "leafs" can eventually be skipped, sometimes even whole component sub trees.

I guess, when it comes to react, methods like shouldComponentUpdate and the like are related to that "problem"
Questions:

What should I read to learn more about it?
Is vdom-thunk related to that problem? (If so, could someone point me to an example?)
Should I change the return logic and instead have something like "vdom manipulation", where the whole thing starts of making a "copy" of the current vdom, applying some updates and then updating the dom?

The answer is to use thunks
Relevant issue: https://github.com/Matt-Esch/virtual-dom/issues/303

thank you :-)

I was reading a bit about those vdom-thunks, but it's still unclear to me how I would apply them to implement the idea i try to sketch below.
Given the component structure from above, I would like to have a "render mechanism" that works like follows:
// NOTATION (for current state of vtree) - EXAMPLE:
// Tree[Version](Subtree1[Version](...),LeafComponent2[version],Subtree3[version](...),...)
//
// NOTATION (for a user or system induced update through interaction with a component)
// Component.called_by_user_or_system_with_random_update()

// EXAMPLE
 var e     = require("engine-vtree2dom")
 var opts  = {/* some opts for R */}
 var R     = require("component-r")(opts) // R's dependency structure => (see above posts)
 //R.push("R[0]( A[0](C[0],D[0]), B[0](E[0],F[0]) )")
 R.pipe(e)
 //e.vtree === "R[0]( A[0](C[0],D[0]), B[0](E[0],F[0]) )"

// SIMULATION (of interaction)
// for demonstration purposes I assume access to a reference to each component
// used by R or their child components

 A.called_by_user_or_system_with_random_update()
 //A.push("      A[1](C[0],D[0])                   ")//.pipe(R)
 //R.push("R[0]( A[1](C[0],D[0]), B[0](E[0],F[0]) )")//.pipe(e)
 //e.vtree === "R[0]( A[1](C[0],D[0]), B[0](E[0],F[0]) )"

 D.called_by_user_or_system_with_random_update()
 //D.push("                D[1]                    ")//.pipe(A)
 //A.push("      A[1](C[0],D[1])                   ")//.pipe(R)
 F.called_by_user_or_system_with_random_update()
 //F.push("                                 F[1]   ")//.pipe(B)
 //B.push("                       B[0](E[0],F[1])  ")//.pipe(R)
 //R.push("R[0]( A[1](C[0],D[0]), B[0](E[0],F[1]) )")//.pipe(e)
 //e.vtree === "R[0]( A[1](C[0],D[0]), B[0](E[0],F[1]) )"
 //R.push("R[0]( A[1](C[0],D[1]), B[0](E[0],F[1]) )")//.pipe(e)
 //e.vtree === "R[0]( A[1](C[0],D[1]), B[0](E[0],F[1]) )"

I hope this makes sense and I'm not totally on the wrong track. The above is more pseudo code and a pseudo process that i tried to sketch.
One question I have in mind is:

given: components are implemented
given: engine-vtree2dom is implemented and whenever it receives a new vtree, it diffs with the old one and applies the patches to the dom

How would I "mark stuff" in my vtree with this vdom-thunk thing during each processing step in the component hierarchy, which makes up my "vtree stream pipeline", so that i minimize the cost for creating patches, once the whole built-up vtree arrives at the engine-vtree2dom?
I would appreciate general feedback about the approach too :-)

Your example is hard to follow
We do it like this
Root -> [Child1, Child2]
On change, invalidate thunk cache
Child1.markThunkInvalid() -> Root.markThunkInvalid() -> re-render page
render page -> Root.render() -> [Child1.render(), Child2.thunk (no change)]
Code: https://github.com/Zorium/zorium/blob/master/src/z.coffee

I updated my posting - maybe it's now a bit more clear.
In your description:

you executed render.page(),
which executes Root.render(),
which executes [Child1.render(), Child2.thunk (no change)]
and so on ....
can you tell more about how render.page() gets triggered in the first place?

In my example the flow is reversed (I will use the flow described above):
/*
                 |-----|                  |------------|
                 | R-0 |                  | NAME-STATE |
                 |-----|                  |------------|
                    |
           |--------|--------|
           |                 |
         |-----|           |-----|
         | A-0 |           | B-0 |
         |-----|           |-----|
            |                 |
       |----|----|       |----|----|
       |         |       |         |
    |-----|   |-----| |-----|   |-----|
    | C-0 |   | D-0 | | E-0 |   | F-0 |
    |-----|   |-----| |-----|   |-----|
*/

"ChildA" gets updated (e.g. through the user clicking a button that is part of ChildA),
which executes "ChildA.render()" and A.pipes(R) the resulting NewVtree to its "parent",
A -> which executes "Root.render()" and R.pipes(engine) the resulting NewVtree to the "engine",
R -> which executes engine.patch(dom, NewVtree),

... my example goes on ...

"ChildD" gets updated by the user or system,
which executes ChildD.render() and D.pipes(A) the resulting NewVtree to its "parent",
D -> which executes ChildA.render() and A.pipes(R) the resulting NewVtree to its "parent",
... another update somewhere else somehow manages to "squeeze in" ...
"ChildF" gets updated by the user or system,
which executes ChildF.render() and F.pipes(B) the resulting NewVtree to its "parent",
F -> which executes ChildB.render() and B.pipes(R) the resulting NewVtree to its "parent",
B -> which executes Root.render() and R.pipes(engine) the resulting NewVtree to the "engine",
R -> which executes engine.patch(dom, NewVtree),
... the other update paths continues and finishes ...
A -> which executes Root.render() and R.pipes(e) the resulting NewVtree to the "engine",
R -> which executes engine.patch(dom, NewVtree),

So because thunks are lazy, you never actually call child.render() yourself (except within the thunk)
What happens is an upwards chain of 'dirty' flags, which only call child.render() during the engine.patch event (via thunk)
This line checks thunk dirtyness: https://github.com/Zorium/zorium/blob/master/src/z.coffee#L60
And this line dirties the thunk: https://github.com/Zorium/zorium/blob/master/src/z.coffee#L141
And when the Root node thunk becomes dirty (which should be any change in the tree because it bubbles up) re-render: https://github.com/Zorium/zorium/blob/master/src/router.coffee#L139

The main thought behind all of this is, that the need for a diff feels wrong in the first place.

Because components return virtual dom, an ancestor has to render them
This means, the subtree has to be traversed looking for more "dirtyness"

If there was something passed into each component, that can be used by the component to push updates, the need for a re-render of the component by a parent would vanish.
In the levelup ecosystem, you have a module called sublevel, which exposes the same API as a regular db object, but all keys in .get(..) or .put(...) or other operations are prefixed.
var ComponentA = require('component-a')
var ComponentB = require('component-b')

module.exports = function ComponentR (engine, db) {

  ComponentA(engine.sub('component-a'), db.sub('component-a'))
  ComponentB(engine.sub('component-b'), db.sub('component-b'))

  var containerA = h('div#' + engine.getID() + 'componentA')
  var containerB = h('div#' + engine.getID() + 'componentB')

  var vtree = h('div', [db.get('component-r'), containerA, containerB])
  engine.push(vtree) // updates ComponentR's container with new vtree of ComponentR

}

When ComponentA or ComponentB do an engine.push(vtree) this will update the content of containerA or containerB. Something like that is the general idea, that i'm trying to get at.
If, for example, ComponentR decides to replace the containerA with something else, then ComponentA would not be needed any longer.
Wouldn't something like that make diffing unnecessary?

Agreed, there should be a way to do partial diffs on subtrees.
I think the interface though should be a function passed to the component, perhaps through a hook.
class Component
  hook: (render) =>
    @state.subscribe =>
      render @render()
  render: ->
    h 'div', 'xxx'

My current workaround has changed since before, and is now a bit simpler.
TL;DR - onMount -> listen, onChange -> mark 'dirty', tell parent your dirty (propogate upwards) -> final parent diffs
(note: using streams all the way through was too slow)
https://github.com/Zorium/zorium/blob/master/src/z_thunk.coffee#L26

I'm not sure what you mean with partial diffs on subtrees.
The code example is hard to read for me too. I'm not used to using coffeescript and classes and es6 so heavily. I'm also not sure why and/or what the hook would do.
I imagine that inside...
function tracker (db, query) {
  return require('level-tracker')(db).trackable().track({ gte: query, lte: query, old: false})
}

module.exports = function ComponentR (engine, db) {

  // ...

  tracker(db, 'component-r/items').on('data', function (items_of_componentR) {
    engine.push(
      h('div', [
        items_of_componentR,
        containerA,
        containerB
      ])
    )
  })

  // ...

}

So after the ComponentR was initialized (see code snippet previous comment), the only thing that stays active are those kind of listeners or "piped streams" that a component sets up when initialized.
(See example above)
It basically says, that whenever an entry in the leveldb (=wrapper around indexDB) changes, the ComponentR is being informed and re-renders itself...

it is not rendered by a parent or ancestor and the parents and ancestors are not re-rendered
it does not re-render it's children, because if children need re-rendering, they can listen to changes in the leveldb and re-render themselves the same way ComponentR does

....that's the whole thought behind it.

I'm actually too much of a beginner to really understand virtual-dom's internals and I'm not sure if all the things necessary to make the described code work are in place - if not - support would be great, because then I can try to implement it myself when i have time.
I'm thinking already for some time about combining streams with virtual dom.
I feel a lot of people see Rxjs as too much of a learning curve to get into the functional standard vocabulary it seems to offer and instead prefer streams, where you make up your custom vocabulary over time. ...at least that's my current perception :-)
...anyway, i'd love to see how virtual dom would work with streams and how that would compare to all the Rxjs stuff I see popping up in lots of places :-)

@serapath I tried to clarify the idea in my proposal (in js). Hopefully you also see how streams are used (as state).
https://github.com/Matt-Esch/virtual-dom/issues/343
Having a 'tracker' or any database layer to manage state I think is going a bit too far. I like the idea of components themselves managing their updates, and keeping the interface as simple and pure as possible.

What I like is, that levelup can be backed by a memdb to just have it in memory but it could also use indexdb in the browser or it could replicate back to a server-side leveldb.
There could also be some kind of "bridge" that translates any kind of API into populating a client side leveldb and/or keeping the client side leveldb up-to-date
The levelup ecosystem is diverse and starting from a pure in-memory-thing it can be updated with whatever features are necessary for any given "app requirements" :-)
In the end - if I use something else, for any serious project, I have to sync with a database anyway, so making this more structured feels pragmatic to me.

@serapath seems outside the scope of virtual-dom.

yes, it's out-of-scope. sry :-)
But your example and mine included some kind of "subscribe to state" and updates trigger a "self-rerender". I agree the state thing, that offers a subscribe could be multiple things, doesnt need to be a levelup thing


Adjust doc link rendering with safety sections

Minor adjustment to how the doc links are rendered with the new safety sections.
See here for example:
https://github.com/MaikKlein/ash/pull/272#discussion_r386053611

@MaikKlein Can this be merged?


Count the number of binary strings

🚀 Feature
I want to add the code for calculating total number of binary strings for a given length which do not have
consecutive one's in it . I want to add this to dynamic programming. I want to work with java.
Have you read the Contributing Guidelines on Pull Requests?
Yes
Motivation
It will help  people in understanding the concept of dynamic programming better. Dynamic programming will also help in executing the solution for this problem in less time.
Pitch
It will help reduce the time complexity and as it is one of the standard problems it will  help people in understanding the concept of dynamic programming better.

I would like to work on this issue.please assign me as a part of Hakin codes @SKAUL05 @siddharth25pandey @Kajol-Kumari @iamrajiv

i would like to work on this issue in c++ as part of hakin codes
@SKAUL05 @Abhijit2505 @plazzy99 @Kajol-Kumari

I want to contribute to this issue. Assign me in C language.
@VijayaGB98


[bug 1137022] Overhaul settings and cleanup startup

nix fjord/base/monkeypatches.py and fold the code into
fjord/manage_utils.py
clean up startup
rework wsgi/playdoh.wsgi so it no longer imports manage.py for
side-effects (gross) but instead explicitly sets up the environment
properly
add settings_utils.py and infrastructure we can later use for settings that come from the environment

r?

^^^ That rewrites everything so far.
Instead of making all the changes to settings files, this reworks Fjord startup to clean it up a bunch, cleans up some related code and then adds settings_utils.py which has the functions we can use for settings that need to pull from the environment.
For now, I think we should just leave it like it is. We'll continue to use the existing settings system where you have base.py and local.py overrides things so we don't incur the pain of having two different settings systems co-existing for different environments like we're currently doing with requirements/vendor. Moving to AWS is months away, so I think it's better to not add complexity now.

This is ready for review.
After it passes Travis and review, I'll test it out on stage. I did make changes to playdoh.wsgi, but I suspect it'll either work or not work and be pretty obvious. We can fix those issues then. Then we'll land it in master and push to stage and prod.

It seems weird to me to add a bunch of dead code like this, even though it makes sense. I would personally start putting settings_utils.config calls in fjord/settings/base.py. The concerns you raised on IRC about those getting tested or not make sense though.
The changes to start up make sense, and the helpers you wrote for env-based settings seem sound.

^^^ That adds some uses of settings_utils.py things. Pretty sure they're all overridden in travis setup or local.py, so I think it's moot, but they're there.

This looks good to me, though if you have any clever ideas on how to fix that double "using fjord.settings" messages, that would be cool.
I'd really like to see what Travis says about this, but r+ from me.

^^^ That tweaks the list issues in config.
So, when you run the server with ./manage.py runserver, it actually tells you the settings being used:
Performing system checks...

System check identified no issues (0 silenced).
March 24, 2015 - 14:34:23
Django version 1.7.7, using settings 'fjord.settings'                  <<<
Starting development server at http://<IP_ADDRESS>:8000/
Quit the server with CONTROL-C.

So I'll just nix that print line.

^^^ Ok. I think that covers it. I'll test on stage and report back if there are issues.
Thank you!

Looks good on stage. I think this is good to go.

4bb85f8 Nix unneeded print line
e4e615d [bug 1137022] Add settings_utils.py uses in base.py
84b11d8 [bug 1137022] Rework startup; add settings_utils.py
Thank you!

Oh. The error that Travis got is the same thing I got on my laptop. I figured it was unrelated, but I guess not?

I think it's from one of the things we config'd. I'll look into it.

Should be fixed in pull #536.


TextBlock throws "Font family not supported" on Android with string containing "weird" character

Describe the bug
Rendering text on Android using string containing a "weird" character like in this one:
"TEST (AU)" (in unicode: "TEST%9D%20%28AU%29")
(it came out of a test database)
avaloniaui throws a "Font family is not supported" exception.
On Windows it renders like a □ (square) without exception.
Expected behavior
I expect it would accept any unicode character without exception, (how it renders it does not really matter).
Desktop (please complete the following information):

Version 11 preview

Should be fixed by #8963


Add rocm Containerfile

To build rocm Container Image for use with AMD GPUs

For your disk space issue, here's an action you can copy to free up some space. It may get you enough. https://github.com/instructlab/instructlab/blob/main/.github/actions/free-disk-space/action.yml

There is an easier, faster way if you just need more disk space for building a container with Docker. The /mnt partition has plenty of disk space.
https://github.com/tiran/instructlab-containers/blob/main/.github/workflows/containers.yaml#L82-L93
                  sudo mkdir /mnt/docker
                  jq '. + {"data-root": "/mnt/docker"}' < /etc/docker/daemon.json | tee /tmp/daemon.json
                  sudo mv /tmp/daemon.json /etc/docker/daemon.json
                  sudo systemctl restart docker.service

@ericcurtin you need to change the location of the docker storage.

This is the logic I plan on using to decide whether to pull a ROCm image or not FWIW:
#!/usr/bin/python3

import glob

i = 0
gpu_num = False
gpu_bytes = 0

# Loop through each file matching the pattern
for filepath in sorted(glob.glob('/sys/bus/pci/devices/*/mem_info_vram_total')):
    # Open the file and read its content
    with open(filepath, 'r') as file:
        content = int(file.read())
        if content ><PHONE_NUMBER> and content > gpu_bytes:
            gpu_bytes = content
            gpu_num = i

    i += 1

print(gpu_num)
print(gpu_bytes)

If we have an AMD/ROCm card with greater than 1GB use the AMD/ROCm card with the most VRAM. On the system I am testing with theirs a measly 512MB integrated graphics card on the AMD CPU, but an external graphics card with 8G VRAM...

This is ready @rhatdan will be reworked a bit when the Kompute stuff gets merged in

containah pushed https://quay.io/repository/ramalama/rocm

LGTM


Selecting date causes calendar redraw

https://codepen.io/stefanvh/pen/yLNLqbv
Describe the bug
In the example, if you select any date other than the current one, the calendar will redraw. This does not happen if you select a date in a different month (after going to another month with the previous/next buttons).
To Reproduce
Steps to reproduce the behavior:

Click on Feb 4 for example
Calendar will redraw

@stefanvanherwijnen This will be fixed in v1.3.17 release (later today)

released


Login via token

Login via Token, fixes #16
Pasting oc login command  (#13) still doesn't work properly, as the cluster value is not updated.
Signed-off-by: Fred Bricon<EMAIL_ADDRESS>

Also switching tabs changes the dialog height, making it jumpy. Someone should fix that ;-)


React.js apps: Datepicker not rendering properly or opening calendar dropdown

Describe the bug
React apps cannot seem to render the date-picker, whereas all other Auro components that we've used do render correctly. I've tried adding the date-picker to our create react app microsite, and it appears to be added to the DOM, but doesn't render correctly with styles, icons, and the dropdown doesn't open.

To Reproduce
Download this create react app repo: https://github.com/tjgrist/my-app and run it. You cannot open the date-picker, it appears only the 'Choose date' text displays.

Steps to reproduce the behavior:

Download the repo
Install the packages
Run the project
Open localhost:3000
Click on the date-picker

Expected behavior
Date-picker should display as it does on the auro demo page, and should have icons/styles and should show the calendar dropdown when you click it.
Screenshots
See shots above
Desktop (please complete the following information):

OS: MacOS
Browser Brave, Chrome, Firefox
Version Latest

Additional context
N/A

Have you tried to include the calendar npm package or cdn link?

@tjgrist  I had a chance to look at the code and see that calendar isn't added.  I ran into the same issue when I first tried to use it.  We should update our documentation to make that more clear.
If it turns out there is another issue though please let me know.

We're trying to install the peer dependencies for auro-calendar, auro-dropdown, etc. But those weren't listed as required installation dependences. What other dependencies -- including css -- might we need? @braven112

Adding auro-calendar and auro-input seems to work, but the styles aren't right. What am I missing?

I see that the first pic may be the z-index issue

We're trying to install the peer dependencies for auro-calendar, auro-dropdown, etc to make it render properly now, will update. But those weren't listed as required installation dependences. What other dependencies -- including css -- might we need? @braven112

@jason-capsule42 is going to work on updating the docs with the correct peer dependencies. It doesn't sound like you need anymore case files if you have wcss and design tokens.

I see that the first pic may be the z-index issue

#32 appears if you resize the browser window while the calendar is open. I think for you its missing a dependency but I'm not sure what one yet.

Yes, we should have the docs illustrate this clearly. An example we should follow is what we did with https://auro.alaskaair.com/components/auro/flight/install

Updated the title to align with the remaining styling issue on the input element.

@tjgrist following up on this... from what I can see, this issue was resolved by doing the correct install, right? Missing dependencies was the crux of the issue?

Closing this one as it appears to be an issue with the installation and we have created #35 to help with this.  If there is anything else please reopen this issue.

@blackfalcon @braven112 yes I figured it out... I needed to ensure I had all the other components installed: auro-calendar, auro-dropdown, and auro-input!


AzureAD SSO not working as described when self hosted: azure_oauth2_secret replaced by social_auth_azuread_oauth2_secret

As stated in this document: https://github.com/zulip/zulip/blob/main/docs/overview/changelog.md
When using AzureAD SSO, the secret should be registered as social_auth_azuread_oauth2_secret and not as azure_oauth2_secret  in /etc/zulip/zulip-secrets.conf
I know it is well announced but we were not using that option at the time and I did not pay attention. I think it would be worth mentionning it in the official documentation (readthedocs) which is not the case right now.
Symptoms are that you have this line in logs when using the wrong variable name:
WARN [zulip.auth.azuread-oauth2] Your credentials aren't allowed

@delahondes thanks for the report; it's much appreciated. I'm not sure why this didn't get a reply in July but it likely had to do with my being on parental leave and usually dealing with this class of issue.

Hmm. This was changed in 16829bd84ba8a91d9b970ff098b3dd9ee3396f19 and the change is documented in the upgrade notes for Zulip 6.0.
https://zulip.readthedocs.io/en/latest/production/upgrade.html#updating-settings-py-inline-documentation is a step that you should follow during future upgrades -- that's our process to avoid having stale comments in /etc/zulip/settings.py.


Use custom error not SyntaxError

I believe SyntaxError should be reserved for errors in python syntax.
Chevron should use it's own exception(s) not hijack SyntaxError.

This could lead to some very ugly confusion if one catches SyntaxError thinking it refers to an error in mustache templates, when in fact there's a syntax error in your python.


Security: pass secrets with --from-file instead of over the command line

I'm following up on a recent incident where a user thought they had a secret leak when running this action.
Passing secrets over the command line is insecure as it can accidentally cause the secret to leak.  See https://help.github.com/en/actions/configuring-and-managing-workflows/creating-and-storing-encrypted-secrets#using-encrypted-secrets-in-a-workflow for details.
For kubectl create secret generic, a better option would be to use --from-file instead of --from-literal.  It would work like this:

Instead of taking an arbitrary arguments string, take a set of key-value pairs like this:

with:
  secrets: |
    username=${{ secrets.USERNAME }}
    password=${{ secrets.PASSWORD }}
    storage-account-key=${{ secrets.STORAGE_ACCOUNT_KEY }}

In the JavaScript for your action, parse these key-value pairs and write them to files as kubectl create secret generic expects (username.txt, password.txt, storage-account-key etc.).
Build up the kubectl command line to run with the --from-file arguments.

This will protect users from accidentally exposing their secrets by running this action.

If you're supplying the values for the secret in ${{ secrets.* }} format, aren't those secrets masked from logs by default?

Yes, but secrets can leak in other ways.  First is that it's exposed through the command line for the process.  For a self-hosted runner, that means that other processes on the machine can see the secrets there.
Second, there's the possibility of a partial secret leak.  If there's a parsing issue over the command line, a string like helloworl"d might get turned into helloworl.  This substring wouldn't get masked from the logs.  But, a user might see that and think that their secret had leaked.

@rgsubh Could you take up the fix for this?

Fixed and released in v.1.1.

Currently, the readme has the below syntax for creating generic secrets:

    - uses: azure/k8s-create-secret@v1
      with:
        namespace: 'default'
        secret-type: 'generic'
        arguments:  --from-literal=account-name=${{ secrets.AZURE_STORAGE_ACCOUNT }} --from-literal=access-key=${{ secrets.AZURE_STORAGE_ACCESS_KEY }}
        secret-name: azure-storage

Am happy to update the README with changes from this PR, but would like to know if the below syntax for generic secrets is correct?
with:
  namespace: 'default'
  secret-type: 'generic'
  secret-name: azure-storage
  secrets: |
    account-name=${{ secrets.AZURE_STORAGE_ACCOUNT }}
    access-key=${{ secrets.AZURE_STORAGE_ACCESS_KEY }}


Outdated version 2.0.2 in README

The URL git.io/antigen points to master branch that has got version Antigen v2.0.2 (5599e4a) still.

FYI: with version 2.0.2 the git plugin from oh-my-zsh completion for git commands broke some time ago (see error below), but with 2.2.2 is works well:
_git:58: _git_commands: function definition file not found

Hi @adambro Latest stable is 2.0.2, which is the one pointed by git.io/antigen, to get the nightly/develop you can use git.io/antigen-nightly.
I've to release 2.2.1 or 2.2.2 as stable in a few days.

OK, thanks for quick response. I have no idea idea about the versioning, but take into consideration that 2.2.2 fixed my problem with git plugin from OMZ. I've tried that on Ubuntu 17.04 box, because it worked well all the time with antergos-git AUR package.
Thanks a lot for your great work, Antigen is totally awesome!


[BEAM-1644] Move travis & jenkins into shared test-infra dir, and move k8s scripts there

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[X] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[X] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[X] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[X] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

Reasoning is discussed in the bug, but generally, we want to have a language neutral place for these kubernetes scripts to live, and also don't want a million top level directories for test-infra
cc @jasonkuster

Coverage remained the same at 69.828% when pulling 3468cec566bd24164922c0c063067d90e1c43849 on ssisk:create-k8s-home into 4a694cebb5deaf4f0d24bea18fa2194798a8ac0f on apache:master.

LGTM

retest this please

Coverage increased (+0.003%) to 70.178% when pulling c688d806969ed973c64c54a61699f7a1b6bdb5ff on ssisk:create-k8s-home into 836e8e4aab239dd41f81de2f1553850b3fd3d716 on apache:master.

retest this please

Coverage increased (+0.003%) to 70.178% when pulling c688d806969ed973c64c54a61699f7a1b6bdb5ff on ssisk:create-k8s-home into 836e8e4aab239dd41f81de2f1553850b3fd3d716 on apache:master.

Test failures so far:
https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/9376/
Failed Tests: 1
org.apache.beam.runners.apex.translation.ParDoTranslatorTest.testAssertionFailure
failure details: https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/9376/org.apache.beam$beam-runners-apex/testReport/org.apache.beam.runners.apex.translation/ParDoTranslatorTest/testAssertionFailure/
java.lang.AssertionError: assertion should have failed
at org.junit.Assert.fail(Assert.java:88)
at org.apache.beam.runners.apex.translation.ParDoTranslatorTest.runExpectingAssertionFailure(ParDoTranslatorTest.java:160)
at org.apache.beam.runners.apex.translation.ParDoTranslatorTest.testAssertionFailure(ParDoTranslatorTest.java:175)

https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/9381/
Failed Tests: org.apache.beam.runners.spark.translation.streaming.CreateStreamTest.testLateDataAccumulating
error details:
https://builds.apache.org/job/beam_PreCommit_Java_MavenInstall/9381/org.apache.beam$beam-runners-spark/testReport/junit/org.apache.beam.runners.spark.translation.streaming/CreateStreamTest/testLateDataAccumulating/
java.lang.AssertionError: Flatten.Iterables/FlattenIterables/FlatMap/ParMultiDo(Anonymous).out0:
Expected: iterable over [<1>, <2>, <3>] in any order
but: Not matched: <4>
so we've seen different errors both times so far

Coverage remained the same at 70.569% when pulling d098fbf92340fc50501047253d07533bdf1d6400 on ssisk:create-k8s-home into 571631a5ef83ef31ea9f652c5698cea3cc95ad43 on apache:master.

Changes Unknown when pulling ad8ee8b407d411d59cdd5f844e9a8e7bc9d59e3e on ssisk:create-k8s-home into ** on apache:master**.

Console output of the jenkins run shows that the build succeeded - 2017-04-12T20:31:56.948 [INFO] ------------------------------------------------------------------------
2017-04-12T20:31:56.948 [INFO] Reactor Summary:
2017-04-12T20:31:56.948 [INFO]
2017-04-12T20:31:56.948 [INFO] Apache Beam :: Parent .............................. SUCCESS [  8.887 s]
2017-04-12T20:31:56.948 [INFO] Apache Beam :: SDKs :: Java :: Build Tools ......... SUCCESS [ 23.823 s]
2017-04-12T20:31:56.948 [INFO] Apache Beam :: SDKs ................................ SUCCESS [ 47.948 s]
2017-04-12T20:31:56.948 [INFO] Apache Beam :: SDKs :: Common ...................... SUCCESS [ 26.703 s]
2017-04-12T20:31:56.948 [INFO] Apache Beam :: SDKs :: Common :: Fn API ............ SUCCESS [ 57.870 s]
2017-04-12T20:31:56.948 [INFO] Apache Beam :: SDKs :: Common :: Runner API ........ SUCCESS [ 30.498 s]
2017-04-12T20:31:56.948 [INFO] Apache Beam :: SDKs :: Java ........................ SUCCESS [ 11.154 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: Core ................ SUCCESS [03:56 min]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: IO .................. SUCCESS [01:00 min]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: IO :: Common ........ SUCCESS [ 10.742 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: Runners ............................. SUCCESS [ 10.122 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: Runners :: Core Java Construction ... SUCCESS [01:04 min]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: Runners :: Core Java ................ SUCCESS [01:56 min]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: Runners :: Direct Java .............. SUCCESS [03:15 min]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: IO :: Elasticsearch . SUCCESS [01:21 min]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: IO :: Google Cloud Platform SUCCESS [01:26 min]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: IO :: Hadoop Common . SUCCESS [ 21.839 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: IO :: Hadoop ........ SUCCESS [  9.938 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: IO :: Hadoop :: input-format SUCCESS [ 43.659 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: IO :: Kafka ......... SUCCESS [ 27.039 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: Runners :: Spark .................... SUCCESS [03:03 min]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: Runners :: Google Cloud Dataflow .... SUCCESS [ 52.367 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: IO :: Hadoop :: jdk1.8-tests SUCCESS [01:18 min]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: IO :: HBase ......... SUCCESS [01:14 min]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: IO :: HDFS .......... SUCCESS [ 36.628 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: IO :: JDBC .......... SUCCESS [ 36.079 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: IO :: JMS ........... SUCCESS [ 25.569 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: IO :: Kinesis ....... SUCCESS [ 30.759 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: IO :: MongoDB ....... SUCCESS [ 44.615 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: IO :: MQTT .......... SUCCESS [ 26.196 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: Maven Archetypes .... SUCCESS [  2.523 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: Maven Archetypes :: Starter SUCCESS [ 27.338 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: Maven Archetypes :: Examples SUCCESS [ 40.976 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: Maven Archetypes :: Examples - Java 8 SUCCESS [ 26.149 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: Extensions .......... SUCCESS [  5.437 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: Extensions :: Jackson SUCCESS [ 26.110 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: Extensions :: Join library SUCCESS [ 19.041 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: Extensions :: Sorter  SUCCESS [ 39.880 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: Harness ............. SUCCESS [ 43.927 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: Java 8 Tests ........ SUCCESS [ 13.227 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Python ...................... SUCCESS [13:00 min]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: Runners :: Flink .................... SUCCESS [  1.777 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: Runners :: Flink :: Core ............ SUCCESS [ 54.917 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: Runners :: Flink :: Examples ........ SUCCESS [ 20.860 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: Runners :: Apex ..................... SUCCESS [01:16 min]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: Examples ............................ SUCCESS [  1.641 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: Examples :: Java .................... SUCCESS [21:16 min]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: Examples :: Java 8 .................. SUCCESS [ 27.892 s]
2017-04-12T20:31:56.949 [INFO] Apache Beam :: SDKs :: Java :: Aggregated Javadoc .. SUCCESS [ 32.321 s]
2017-04-12T20:31:56.949 [INFO] ------------------------------------------------------------------------
2017-04-12T20:31:56.949 [INFO] BUILD SUCCESS
2017-04-12T20:31:56.949 [INFO] ------------------------------------------------------------------------
2017-04-12T20:31:56.949 [INFO] Total time: 01:26 h
2017-04-12T20:31:56.950 [INFO] Finished at: 2017-04-12T20:31:56+00:00
2017-04-12T20:31:57.619 [INFO] Final Memory: 298M/1852M
2017-04-12T20:31:57.620 [INFO] ------------------------------------------------------------------------
Waiting for Jenkins to finish collecting data
channel stopped

Run Seed Job


Cast const empty string to PWSTR

Error C2440 'return': cannot convert from 'const wchar_t [1]' to 'PWSTR'
I don't think the error/warning is reproducible since it probably would've been noticed already...

Why does this line generate a warning out of dozens of functions with the same return value? 🤔


Could you please add an example with the new method signature? (and with a real PCM input as well if possible)

I am trying to use it but when a pass a real PCM byte array as input it never count a valid frame size
when this line is called it return a huge number.
double frameSize = API.GetFrameSize(pcmLength, SampleRate, Channels);
What could I be doing wrong?
Here is my code:
public Task<byte[]> ConvertAsync(byte[] content)
       {
           using var encoder = new OpusEncoder(Application.Audio, 48000, 2)
           {
               Bitrate = 128000,
               VBR = true // Variable bitrate
           };
           var opusBytes = encoder.Encode(content, content.Length, out var encodedLength);
           return Task.FromResult(opusBytes);
       }

I know it's late, but in case you still need help:
You have to encode your PCM data frame by frame.
The first parameter for Encode has to be a buffer with PCM data and a length corresponding to 2.5, 5, 10, 20, 40 or 60 ms (with regards to the waveformat used).
For example, with 16kHz 16bit mono, a 60ms frame occupies 60162*1 = 1920 bytes.
If you give a maximum encoded buffer length of 128 bytes, the maximum bit rate will be about 2 KB/s.
hth,
mav


BAVL-235 initial commit adding localstack for integration testing of events/messages.

LGTM. I'll need to study a bit more to work out any further impact on int tests.

Ideally I like to only spin up localstack for those tests that need/use it.  This requires some refactoring though.


Source of truth for MIxS 6.2 release and transition to schema sheets

On today's TWG call, we decided that the SOT for MIxS should be the TSV files formatted for schema sheets, instead of YAML. That way they can be edited in Excel. The TSVs should be transformed back into YAML as part of a PR (with each commit?).
However, we will keep YAML as the SOT for MIxS 6.2, then work on the schema sheets implementation either for a 6.3 version of for 7.0 (expected ~ Feb. 2024).
@turbomam @sujaypatil96 please review my notes above and make any corrections needed or ask clarifying questions here.

The stored source is truth will be LinkML YAML. We did discuss allowing contributors to make their initial submission in TSV. It was not clear to me whether the TSVs could be fully validateable schemasheets, TSVs that use schemasheets headers but are otherwise unconstrained, or truly free- form spreadsheets.
Regarding the use of spreadsheet applications like MS Excel, I say that people can use any tool they want, but they are responsible for sharing UTF-8 encoded, tasb delimited documents.
Who is going to be responsible for converting the submitted sheets to something that can merged into the schema? I.E. either validateable schemasheets or validateable LinkML YAML? I think we may be really underestimating how much work that will take.


Cannot increase mdf file size when using docker volumes on Windows 10

OS: Windows 10 Pro Version 1709 OS Build 16299.192
Docker: Docker Containers Version 17.12.0-ce-win47 (15139) Channel stable
I have a database where the entire mssql folder it contained inside a docker volume (this is for dev so having all those files someplace I can access is useful). Lately I've had an issue where once the mdf files are created after creating a new DB, they cannot increase the size of the file from when it was created. I tried uninstalling docker and re installing and using a different empty folder for the volume but still no luck. When I change it so that all the database files are contained inside the container everything seems to work fine. Here is the command I use to create the container:
docker run -e "ACCEPT_EULA=Y" -e "MSSQL_SA_PASSWORD=Password1" -e "MSSQL_PID=Developer"    -p 1433:1433 -v /C/Users/ranger1230/Documents/SQLDataBase:/var/opt/mssql
-d microsoft/mssql-server-linux:2017-latest
And here is the error message I receive when I try to do something that will require the database files to grow (yes Autogrow is enabled):
TITLE: Microsoft SQL Server Management Studio
Restore of database 'Episerver' failed. (Microsoft.SqlServer.Management.RelationalEngineTasks)

ADDITIONAL INFORMATION:
System.Data.SqlClient.SqlError: MODIFY FILE encountered operating system error 31(A device attached to the system is not functioning.) while attempting to expand the physical file '/var/opt/mssql/data/Episerver.mdf'. (Microsoft.SqlServer.SmoExtended)
For help, click: http://go.microsoft.com/fwlink?ProdName=Microsoft+SQL+Server&ProdVer=13.0.16106.4+((SSMS_Rel_16_5).170125-2137)&LinkId=20476

BUTTONS:
OK

@friism is the fix coming in next version of docker?

ping @ebriney for status

I think the issue is due to Linuxkit, dockerd is now running in a container and samba share is only available in this namespace.
I'm trying to figure out what is going wrong with that configuration.

@ebriney @friism I am currently running into the same issue. on Docker for Windows 17.12.0-ce-win47 (15139). I created the container like this:
docker pull microsoft/mssql-server-linux:2017-CU4
docker create --name mySqlServerExpress -e "ACCEPT_EULA=Y" -e "SA_PASSWORD=MyStr0ng!Pa$$w0rd" -e "MSSQL_PID=Express" --publish 1433:1433 --mount type=bind,source="%~dp0sqlData",target=/var/opt/mssql microsoft/mssql-server-linux:2017-CU4

And when I run it, I get the following error:
2018-03-12 14:51:16.28 spid6s      Database 'master' running the upgrade step from version 863 to version 864.
2018-03-12 14:51:16.35 spid6s      Error: 17053, Severity: 16, State: 1.
2018-03-12 14:51:16.35 spid6s      /var/opt/mssql/data/mastlog.ldf: Operating system error 31(A device attached to the system is not functioning.) encountered.
2018-03-12 14:51:17.42 spid6s      Error: 928, Severity: 20, State: 1.
2018-03-12 14:51:17.42 spid6s      During upgrade, database raised exception 9002, severity 17, state 0, address<PHONE_NUMBER>D0157E. Use the exception number to determine the cause.
2018-03-12 14:51:17.45 spid6s      Error: 9002, Severity: 17, State: 0.
2018-03-12 14:51:17.45 spid6s      The transaction log for database 'master' is full due to 'NOTHING'.
When can we expect a fix for that?
/cc @twright-msft

@gingters I did tries changing the way we do the mounting in the vm but cannot make it work again.
So unfortunately no news on that.
Why do you need to bind mount a local db?
Docker volumes are working, tough it's not local, you can use it as persistent volume and run debug container in parallel.

@ebriney I am currently creating a test-environment. But I think I can work around that by creating a volume, and then using a temporary container to copy the files from the host into the volume and then attaching the volume to the actual sql server container and run it from there (for eacht test run).
However, this is only a work around, and this issue should be fixed.

This looks like https://github.com/Microsoft/mssql-docker/issues/136

This seems to be resolved for me in the 2017-CU7 tag of the docker image. Currently running Docker version 18.03.1-ce-win65 (17513) with a container running mssql-server-linux:2017-CU7 and I am able to successfully restore my databases.
Is it resolved for everyone else?

I get this failure when attempting to restore a database from a bak file of 10,063,164KB in size. For my other database restores, with bak files up to 454,215KB there is not problem.


Buttons are not being pushed to new lines for each one

Description
Input
The code looked like this before beautification:
<button class="dsmButton extralarge">Extra Large</button><button class="dsmButton large">Large</button><button class="dsmButton regular">Regular</button><button class="dsmButton medium">Medium</button><button class="dsmButton small">Small</button>

Expected Output
The code should have looked like this after beautification:
<button class="dsmButton extralarge">Extra Large</button>
<button class="dsmButton large">Large</button>
<button class="dsmButton regular">Regular</button>
<button class="dsmButton medium">Medium</button>
<button class="dsmButton small">Small</button>

Actual Output
The  code actually looked like this after beautification:
<button class="dsmButton extralarge">Extra Large</button><button class="dsmButton large">Large</button><button class="dsmButton regular">Regular</button><button class="dsmButton medium">Medium</button><button class="dsmButton small">Small</button>

Steps to Reproduce
Environment
OS: MacOS
Settings
{
  "indent_size": "4",
  "indent_char": " ",
  "max_preserve_newlines": "-1",
  "preserve_newlines": false,
  "keep_array_indentation": true,
  "break_chained_methods": true,
  "indent_scripts": "normal",
  "brace_style": "collapse,preserve-inline",
  "space_before_conditional": true,
  "unescape_strings": true,
  "jslint_happy": true,
  "end_with_newline": true,
  "wrap_line_length": "0",
  "indent_inner_html": true,
  "comma_first": true,
  "e4x": true,
  "indent_empty_lines": true
}

@bitwiseman your suggestion didn't work for me until I had { unformatted: [], inline: [] }


Insomnium: 0.2.3-a init.

Description of changes
Insomnium is a fast local API testing tool that is privacy-focused and 100% local. For testing GraphQL, REST, WebSockets and gRPC. This is a fork of Kong/insomnia
https://archgpt.dev/insomnium

Themes work
Custom themes work
Plugins work
All basic functionality works

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[x] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

 Something along insomnium: init at 0.2.3-a would be better.
See the commit conventions for a reference: https://github.com/NixOS/nixpkgs/tree/master/pkgs#commit-conventions

As stated by the README, the app isn't being actively maintained anymore and the repo is archived so I think this should be closed.


Support returning a promise

I'd like the ability to just return a promise instead of having to use a callback.

Having dug into this and created a number of test-bench functions using promises...I think we actually do support it for most cases. If the return value of a user's wrapped function is a promise, they'll see metrics just fine.  I've reached out to the user to see if I understand their intended use correctly, but the test scenarios I've tried all return promises and provide our analytics.

@mrickard is there anything special that needs to be done to support this?  If so, some examples in the examples repo would be great to see.

@adjohn All of the tests I've run are using completely unaltered IOpipe-js, so I'd say nothing special, barring that caveat of some special use case. I'd be happy to throw together an example...or add to the instructions on the Install page.
Another thing about the examples repo is that we're showing security warnings due to older dependencies.  I'm happy to take on updating those, if that's a reasonable priority.  (Short-term Picard fixing should probably go above that in priorities, though.)

(Also, I see that the customer is on Slack, so I might hear an update to the DM thread.)


Support Safari web inspector debugging for Edge on iOS (and iPadOS)

In https://github.com/MicrosoftDocs/edge-developer/issues/559 the team said

Currently, we do not support iOS remote debugging. This is ultimately due to some nuances with how our Microsoft Edge mobile apps are built. On iOS, as I understand it, the Microsoft Edge app uses WKWebView which basically means that the underlying web platform that is rendering HTML/CSS/JavaScript is WebKit.

From a chrome developer blog it seems as of iOS 16.4, there is support for remote web inspector debugging from WKWebView now

Now that WKWebView allows Web Inspector debugging in release (as of iOS 16.4), we have added a setting in Chrome 115 to enable this feature and further improve the developer experience.

Can the edge mobile team please allow this feature for edge mobile on iOS/iPadOS?
AB#53153096

Thank you for filing this feature request, and apologies for the delay answering.
I'll bring this up to the team for consideration/prioritization.

Been trying to track down some subtle issues with Entra App Proxy and iOS Edge where this would be super helpful.  Hopefully since Chromium has had it for a bit it wouldn't be tough to expose the webView.isInspectable = true in the Edge instances of WKWebView sooner rather than later.


Consider improving the syntax for FlushPoints

Right now using FlushPoints look like this:
</head>
@{ await FlushAsync(); }
...

The suggestion is to make it look nicer
</head>
@await FlushAsync()
...

This would require returning a no-op value (null / empty HelperResult) from FlushAsync

Duplicate of #1547


sysutils/nut: add listen, make arguments more flexible

Closes #994
Partially closed #924
Extra arguments is now a CSVListField, so when you need pollonly you just add it :)

ok, will arguments ever user commas ? spaces should work I think
"partially close(d|s)" will probably confuse github and close the issue anyway. :)

Only dots, no comma or space. Space only in description, but this is an extra field. So, good for merge :)

Merged, thanks!


Border lands tree

This is weird. It's not exactly a skill tree with directed nodes. Rather, it's based on layers and prerequisites if I remember right.
The only directed edges are on the sides of the tree which I rather not think about right now.

I actually think I can make this already with LayeredLayout just by specifying the depth (which I haven't fully implemented) of every skillNode.
The thing that I can't do is layout stuff right on top of one another. My LayeredLayout layout algorithm would take the space available and divide it amongst the children equally, which would not stack over the rest of the children as in this picture.
It looks like the placement is arbitrary so no algorithm will do. It'll have to be defined by the user.
Here's my immediate idea
class Home extends StatelessWidget {
  const Home({ Key? key }) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return SkillTree<void, void>(
      delegate: CustomBoxDelegate(
        layout: [
          [null, '0', null],
          ['2', null, '1']
        ]
      ),
      nodes: [
        Node(id: '0'),
        Node(id: '2'),
        Node(id: '1'),
      ],
    );
  }
}

but that brings up issues of serialization but I'll get to that later

It looks like WoW has the same talent tree setup. They keep the idea of 5 points a layer to unlock the next layer. They have edges between nodes as well so that's good.


Move to using docker for dependencies and deployments

summary of stuff that changes.

settings rationalised into base, development and production
docker files

heltour:latest: contains the python env (the webapp, celery, and eventually the api worker)
heltour-static: a static files server. at somepoint we should talk about moving this stuff to a CDN.

docker compose to replace vagrant. I've been using this locally and with the new docker compose watch feature, this feels viable.
includes glbert's #527

still todo:

api worker
secrets
figuring out what to do with database stuff. I'm playing around with django fixtures and feels viable.


Add options to fix clipping and apply peak normalization to the generated soundscape

Right now, it's very easy for the user to generate a soundscape the clips if they're not careful with their choice of ref_db and the SNR values they give to foreground events.
This PR adds two new optional parameters to both generate and generate_from_jams:

fix_clipping - if True (default=False), will apply peak normalization in the case that the soundscape is clipping
peak_normalization - allows the user to apply peak normalization (abs(max) sample = 1) even if the soundscape isn't clipping

This PR addresses #131
Checklist:
generate:

[x] modify soundscape audio
[x] modify event audio
[ ] jams: update ref_db
[ ] jams: store fix_clipping and peak_normalization bools
[ ] jams: store scale_factor?

generate_from_jams:

[ ] modify soundscape audio
[ ] modify event audio
[ ] jams: update ref_db
[ ] jams: store fix_clipping and peak_normalization bools
[ ] jams: store scale_factor?

testing:

[x] peak_normalize
[ ] generate
[ ] generate_from_jams

NOTE: missing unite tests, coverage at 98% right now.

If we can unpin pyrsistent in this PR as well, that'd be great. We pinned it to be compat with 2.7 and 3.4, but we dropped support for those a few versions ago, so we can unpin.

ok @pseeth I believe I've addressed all of your comments. Let's CR and SqM.


How to remove SQL query from Copy as HTML output?

System information:

DBeaver 6.2.1 CE

Connection specification:

MySQL

Describe your question:
Is it possible to remove the SQL query from the 'Advanced Copy\Copy as HTML' output?
Thanks for your time.

issue is obsolete

I came looking for this same issue, and I don't understand the response. Copy as HTML still copies with the query. Has there been any addition of Copy as HTML without query option which I can't find?
I regularly copy-paste data from the output to Excel and have to take the additional step of removing the query from the first row. Would be nice to be able to define a keyboard shortcut for Copy as HTML without query.


In-cluster eventing functions code for UI tests

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)

/retest

/retest

/retest

/retest


[Feature] Add a doc and YAML for Ray Serve grpc support

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
https://github.com/ray-project/kuberay/issues/1107
Use case
No response
Related issues
https://github.com/ray-project/kuberay/issues/1107
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

@evalaiyc98 will take this issue.

Sure! Thanks.


fix: refactor hooks

add price context and theme context
remove unused hooks
refactor hooks to remove deeply nested graphql dependencies

😅. While testing the app locally I noticed that the bug with the sat balance showing as ... has returned.

What are the steps to reproduce the "..."? I haven't had that issue myself.


opt use correct column label when execute query in FE

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall

run buildall


Drop outdated BEACH_WAIT_FOR_SYNC env variable

This drops the BEACH_WAIT_FOR_SYNC env variable from the setup, as it is no longer needed without the devkit container and a recent PHP container.
See https://github.com/flownative/docker-beach-php/pull/19

This should probably be kept open for a while, until we have https://github.com/flownative/docker-beach-php/pull/19 merged and people have updated to it…

Ah, @kitsunet – did you read my comment?

Ah, @kitsunet – did you read my comment?

Ah, @kitsunet – did you read my comment?

I should have made this a draft, probably…

trigger-happy @kitsunet 😊

I thought I merged the other on Friday 😆

Yeah, but even then:

[…] and people have updated to it…

But, thinking about it now: This affects new projects, so those should pull a new image anyway.
Might be good enough… 😎

Exactly, I did expect this only to have any effect on new projects, which schould be fine...


Allow static pv to be correctly referenced.

Fix #94. The generated pvc name and the size must be passed as well for this to work

Sorry for the late review @RudiMT. I think it's a simple change and should work fine, the only question I have if it someone wanted to customize the name, they are still bound to the static -pvc at the end which might not work for someone's naming convention or something?

I'm fine with -pvc and without. The PVC name isn't a parameter currently, though, so there's no possibility for the users to use their own convention (and I didn't want to add that logic). The docs say "persistence.enabled: Configure a Persistent Volume as the default storage location. It accepts either a StorageClass name to create a PVC, or directly accepts the PV to use.", but don't say how the PVC's name is derived in either case.

@RudiMT Thanks! Please squash commits to one

Superseded by https://github.com/rancher/backup-restore-operator/pull/274


Guard against openssl's new strict DER checks.

This is a port of the patches 488ed32f2ada1d1dd108fc245d025c4d5f252783 and 8dccba6a45db0466370726ed462b9da2eae43bce made by theuni for bitcoin core to fix the problem with the new openssl versions being stricter on the format of DER encoded ECDSA signatures (which could split the network).

I'm going to review this PR today against the Bitcoin patch and get it merged if I don't see any problems building it. Thank you for submitting it, @glv2!


⚠ Bofrost.lu has degraded performance

In f7aa558, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1918 ms

Resolved: Bofrost.lu performance has improved in 98779b4 after 14 minutes.


Issue in get_attributes module in agouti_shred.py

Hi AGOUTI team,
let me start off by stating what an awesome program AGOUTI is, which exactly caters to my needs of recovering a fragmented genome assembly in my MSc-thesis. I'd be a little bit lost without it. A very minor issue i was encountering when shredding an assembly. As you may know annotators dump pretty much anything in the comment section, which is a problem for agouti.  It throws an error and exits, when its not in the =<attribute_name> format, which is not always the case (and was not in my case). In my case i fixed it by changing the get_attirbutes function to:
def get_attributes(attribute):
	dAttributes = {}
	tmp_attributes = attribute.split(';')
	for i in range(len(tmp_attributes)):
		tmp_attribute = tmp_attributes[i].split('=')
		if len( tmp_attribute) > 1:
			attributeName = tmp_attribute[0]
			attributeValue = tmp_attribute[1]
			if attributeName not in dAttributes:
				dAttributes[attributeName] = attributeValue

which is a very minor correction and simply gets rid of comments not in the desired format. It's not the best solution, but worked for me. Anyway, I just wanted to point out this very minor issue and thanks again for writing such an elegant program!
Best,
Nico

Hello Nico,
Thanks very much for using AGOUTI, and pointing out this potential problem!
Could you please give me a bit more info about your case, if you don't mind? For example, what annotator you were using? What does your "comment" section look like? I'd like to try to come up with a general solution so that everyone else won't have it.
Thanks again,
Simo

Hello Simo,
the problem arose using a gff pulled from the German cockroach genome project's ftp page:
ftp://ftp.hgsc.bcm.edu/I5K-pilot/German_cockroach/maker_annotation/version_0.5.3/
It was built using MAKER. I Don't know what the exact line is which is giving teh problem but if i do  		```
try:
attributeName = tmp_attribute[0]
attributeValue = tmp_attribute[1]
except:
print tmp_attribute
it prints
> ['']
so i guess somewhere there is an empty attibute passed to the get_attributes function, but i don't know where. Maybe there are empty lines. Sorry, I couldn't be of any more help, but maybe you can recreate the problem

Best,
Nico

Hi Nico,
These info are great. Let me play with it.
Simo

Hi Nico,
I just made a patch. The error you saw was due to the semicolon at the end of attribute field in the GFF file. The fix can also bypass cases in which there were attributes with no {}={} type of format.
Thanks for bringing this up.
I close this for now. Let me know if you get further problems.
Simo


[Refactor] make ~ContextWithDependency virtual

What type of PR is this：

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

Problem Summary(Required) ：

The destructor of ContextWithDependency is not virtual.
Luckily, all the usage of subclasses of ContextWithDependency are shared_ptr, so memory leak doesn't happen for now.
Checklist:

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

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

Bugfix cherry-pick branch check:

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

[x] 2.5
[x] 2.4
[x] 2.3
[x] 2.2

@Mergify backport branch-2.2 branch-2.3 branch-2.4 branch-2.5

@Mergifyio backport branch-2.4

@Mergifyio backport branch-2.3


docs: move TL;DR to the top of the readme

Proposal:

Change the TLDR: into TL;DR
Add more features, like create swap, accept swap
Describe the other fields, use example from the Uniswap Permit2

Can I get this issue? @0xneves

Let's go! Send us da pull request!

https://github.com/blockful-io/swaplace-contracts/pull/148

#148
Hey @0xjoaovpsantos will give you a hint on github:

You can write the following in your PR:
closes #<issue>
it will automatically close the issue when approved and also link the PR with the issue.
The way you are doing by referring to the PR in the issue won't suit the linking between them.

closed by #148


ui-listview load-on-demand auto scroll to top

when loading items on demand in RadListView. adding more item automatically scrolls to the top.
here is playground demo:https://play.nativescript.org/?template=play-ng&id=E2d9wx&v=2

@bhavincb please use this repository for issues related to RadListView


riscv: stack objects are mis-aligned

Describe the bug
In macro "Z_STACK_PTR_TO_FRAME", it does not round up the stack frame size to arch
stack pointer alignment. Therefore, the thread stack size is not rounded up when initializing,
but it is when restoring the context to run.  The bottom of thread stack is actually over it
was allocated at first.
To Reproduce
Enable CONFIG_STACK_SENTINEL and error will be triggered

Thanks for the fix....but that's very weird that the size of the frame itself isn't aligned to the stack pointer alignment, since it represents data pre-populated onto the stack. What arch did you see this on?

I found this issue on RISC-V at first. If we enable CONFIG_FPU and CONFIG_FPU_SHARING, the stack frame size is 414/418 for RV32/RV64. They are not 16-bytes alignment.
On the other hand, there is an member "soc_context" in stack frame for extension. It would be a potential risk for user if he does not notice it.
Some archs only need 4-byte alignment, they may not have this issue. If any archs need 8/16-bytes alignment, it would be better to add this protection. Therefore, I would like to add it for all archs instead of only RISC-V.

I found this issue on RISC-V at first. If we enable CONFIG_FPU and CONFIG_FPU_SHARING, the stack frame size is 41x4/41x8 for RV32/RV64. They are not 16-bytes alignment.

So here's the question: why is an irregular amount of data being put on the stack? If we are putting data on the stack to instantiate a new thread, how are we getting into a situation where the amount of data pushed onto the stack results in a C ABI violation?

@andrewboie
RISC-V standard calling convention requires stack pointer must be 16bytes aligned.
See: https://riscv.org/wp-content/uploads/2015/01/riscv-calling.pdf
18.2  RVG Calling Convention
If users use only C functions compilers keep SP 16bytes aligned, but in z_setup_new_thread(), it sets SP manually.
So Zephyr violates SP alignment rules of RISC-V.

I posted analysis and an alternate approach in the linked PR. Long story short, your stack object is itself mis-aligned as its base address and size are only aligned to Z_RISCV_PMP_ALIGN.

@katsuster @Teng-Shih-Wei ^^

Reopened...
We need to solve this issue. Current RV32 Zephyr printf() outputs very strange values by this problem.
===================================================================
START - test_pi
Pi calculation OK after 50 (high) + 118 (low) tests (computed -26815615863448970000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000.000000)
Pi calculation OK after 150 (high) + 360 (low) tests (computed -26815615863448970000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000.000000)
Pi calculation OK after 250 (high) + 595 (low) tests (computed -26815615863448970000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000.000000)
Pi calculation OK after 350 (high) + 828 (low) tests (computed -26815615863448970000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000.000000)
Pi calculation OK after 450 (high) + 1054 (low) tests (computed -26815615863448970000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000.000000)
 PASS - test_pi in 12.306 seconds


cannot load such file -- os/linux/glibc

Please, help. This is my error:
/usr/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in require'
/usr/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in require' /home/vlad/.linuxbrew/Library/Taps/homebrew/homebrew-core/Formula/glibc.rb:1:in load_formula'
/home/vlad/.linuxbrew/Library/Homebrew/formulary.rb:21:in module_eval' /home/vlad/.linuxbrew/Library/Homebrew/formulary.rb:21:in load_formula'
/home/vlad/.linuxbrew/Library/Homebrew/formulary.rb:34:in load_formula_from_path' /home/vlad/.linuxbrew/Library/Homebrew/formulary.rb:83:in load_file'
/home/vlad/.linuxbrew/Library/Homebrew/formulary.rb:74:in klass' /home/vlad/.linuxbrew/Library/Homebrew/formulary.rb:70:in get_formula'
/home/vlad/.linuxbrew/Library/Homebrew/formulary.rb:207:in factory' /home/vlad/.linuxbrew/Library/Homebrew/formula.rb:1186:in []'
/home/vlad/.linuxbrew/Library/Homebrew/cmd/install.rb:200:in check_ld_so_symlink' /home/vlad/.linuxbrew/Library/Homebrew/cmd/install.rb:213:in perform_preinstall_checks'
/home/vlad/.linuxbrew/Library/Homebrew/cmd/install.rb:91:in install' /home/vlad/.linuxbrew/Library/brew.rb:84:in ``<main>'

Please surround copied-and-pasted logs with triple back ticks. See GitHub Help / Quoting Code.

Try…
brew update
brew vendor-install ruby
brew update

If that doesn't help, please report the output of brew config.

Consider installing Linuxbrew in /home/linuxbrew/.linuxbrew/ if possible so that you can use precompiled binary packages (known as bottles) for non-relocatable formula like util-linux.
Another possible workaround for you is brew install --force-bottle util-linux, but no promises.
If it's an option for you, you could open a ticket with your information systems department to ask that they create a linuxbrew role account with home directory /home/linuxbrew.
The precompiled binary bottles of non-relocatable bottles can only be used if you install in /home/linuxbrew/.linuxbrew, otherwise they have to be built from source. See the documentation below. On macOS the default installation directory is /usr/local. On Linux the default installation directory is /home/linuxbrew/.linuxbrew.

http://docs.brew.sh/Installation.html#alternative-installs
http://docs.brew.sh/FAQ.html#why-do-you-compile-everything
http://docs.brew.sh/Bottles.html#cellar-cellar

Please note that we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you repeatedly fail to use the issue template, we will block you from ever submitting issues to Homebrew again.
As there is way too much information missing here, I will close this issue. We can reopen it if you provide all the needed information


Router demo component for docs

As a developer I want to see interactive demos for the router component on the docs page (similar to the react-router online demos).
DoD:

resolve the iframe renderer back / forward issue: https://github.com/vlukashov/vaadin-demo-helpers/issues/3
resolve the iframe renderer issues in IE11:
-- https://github.com/vlukashov/vaadin-demo-helpers/issues/1
-- https://github.com/vlukashov/vaadin-demo-helpers/issues/2
verify that all existing Vaadin Component demos still work with the new version of vaadin-demo-helpers when deployed to vaadin.com
verify that the new vaadin-router demos work when deployed to vaadin.com
~the UI of the router demo snippets is reviewed / updated by a UI designer~ (done)
merge the 'iframe-renderer' branch into the upstream vaadin-demo-helpers repo

The common DoD does not apply.

The IE11 issues are still there.
The rest is done:

all modifications were merged into vaadin/vaadin-demo-helpers (both into the 1.x branch / Polymer 2, and into the master branch / Polymer 3). A new version is released: vaadin/vaadin-demo-helpers#1.3.0
vaadin.com is updated (still in preview: http://preview.vaadin.com/router)


⚠ gorzdrav.org has degraded performance

In d89a4fc, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 1464 ms

Resolved: gorzdrav.org performance has improved in 469c58f.


Change sorting order of containers bt "Pull no"

Currently sorting containers using "Pulls no" sort containers by ascending order showing the newest or least popular containers first.
I suggest to change the sorting order or to add a third sorting options with "ascending" and "descending" options

We have fix this issue. Sorry for the delay.

Thank you

More than welcome, if you would like to see more features in the Web just let us know !!!!


[UI][Feature Request] Collapsible pinned tracks

I often run out of vertical space after pining a few particularly high tracks (rows) to the top. I wish I could temporarily squish rows (without unpinning) to peek at some other rows below. I like the way collapsed processes are presented. I wish pinned rows could do the same.
Example:

.

@octaviantu @chromy FYI

do we have any solid plan for this feature ?
I encounter this problem recently also.
and it would be more user friendly if we can add vertical scrollbar in the pin tracker area as well.

No further updates on this.

i have same problem, this feature is very useful, please

Folding this into https://github.com/google/perfetto/issues/429


Test for go mod or go vendor inconsistencies

📑 Description
Add script to test for go mod / vendor inconsistencies and add it to CI.
Additional Information for reviewers
✅ Checks

[ ] My code requires changes to the documentation
[ ] if so, I have updated the documentation as required
[ ] My code requires tests
[ ] if so, I have added and/or updated the tests as required
[ ] All the tests have passed in the CI

How to verify it

add new dependency but do not run go mod / go vendor
execute make verify-go-mod-vendor

Expecting PR to fail until https://github.com/ovn-kubernetes/ovn-kubernetes/pull/4924 merges

/home/runner/work/ovn-kubernetes/ovn-kubernetes/go-controller
ERROR: detected go mod / sum or vendor inconsistency after 'go mod tidy; go mod vendor':
 M go-controller/go.mod
 M go-controller/vendor/modules.txt

From run https://github.com/ovn-kubernetes/ovn-kubernetes/actions/runs/12389426536/job/34582429696?pr=4925

nice
/lgtm


shenyu-admin做多个实例， 在 shenyu-admin  页面上做增删改的接口， 只会调用 shenyu-admin后端 多个后端服务实例之一  服务接口， 这导致 shenyu-admin 同步到 shenyu-bootstrap 的时候，网关服务实例的数据不一致

Question
// your question here;
Please pay attention on issues you submitted, because we maybe need more details.
If no response anymore and we cannot reproduce it on current information, we will close it.

Which way is used for data synchronization?

Hi,
For the admin module, there is no need to use the cluster to start, or you can use only one web-console.
If there is a proxy before admin module, and you want to start the cluster also. I have two suggestions

you can use Http Long-Polling to synchronize data. It will regularly check the database for changes every 5 minutes.
The best way is to use third-party components such as nacos to synchronize data

pls in english


How do you get this to work?

Hi, I followed the README.MD and it's set up but what do I code to toggle between the states?

There's a toolbar icon you can use to toggle the states.

So you simply set the states with CSS and it automatically does the toggling?

Yes, just install the addon and use the toolbar to select the state you want to see, or add parameters to your stories to give them an initial pseudo state. You can also wrap your story components in a div with e.g. pseudo-hover class on it. See https://github.com/chromaui/storybook-addon-pseudo-states/blob/main/stories/Button.stories.js


Allow for separation boolean options when running the dragen-somatic-with-germline pipelines

The following options can be determined for germline and somatic tools separately

--enable-sort
--enable-map-align
--enable-map-align-output
--dedup-min-qual
--enable-duplicate-marking

Testing with
wfr.8571c811daa1461c85a29b2226b9d63b and wfr.60f92efbcb8141d988f176df272575e9

Successful tests with
wfr.322a9746b55c4e32942002f7e5cd3104

And
wfr.089c4b5f4bab4ba69b4fc1d801e2cb54

Now use a CWL Expression to collect the normal_bam_out from either the germline directory and the somatic directory


Improve README regarding global variable handling

When will you merge my doc patch?

Terrible sorry for the late merge, but it's merged. Thanks for pinging back and contributing :) :+1:

No problem, I was just going through my github repos to clean up the ones I've forked just to submit patches.


Installing from GitHub throws an error because tidyverse::select is missing

Used devtools::install_github("data-edu/dataedu") and got this error:
Error in select(s12_course_data, ) : could not find function "select"
Error: unable to load R code in package ‘dataedu’
Execution halted

I'm guessing library(tidyverse) needs to appear in a script somewhere, but not sure where.

can you try this again using the latest version of the package? I think(/hope!) this and #25 are resolved in that PR. I remember seeing these errors when building.

I often get errors with select() - maybe change to dplyr::select()?
On Thu, Jan 2, 2020 at 2:09 PM Joshua Rosenberg<EMAIL_ADDRESS>wrote:

can you try this again using the latest version of the package? I
think(/hope!) this and #25 https://github.com/data-edu/dataedu/issues/25
are resolved in that PR. I remember seeing these errors when building.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/data-edu/dataedu/issues/26?email_source=notifications&email_token=AC5LK6GEJL6UJJJTQWVALDDQ3ZCZPA5CNFSM4KCGC3NKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH7ILZQ#issuecomment-570328550,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC5LK6EOV5TJBLYCH5I7RDDQ3ZCZPANCNFSM4KCGC3NA
.

just pushed a change to do this

Awesome it works now! Thank you


Synthesis failed for python-recaptcha-enterprise

Hello!

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at 4ecda99 chore: add default CODEOWNERS (#21)
2020-10-30 01:12:18,572 autosynth [DEBUG] > Running: git checkout 00bbad4dfd6633cf4b5f9596c1f93b756bb5c776
Note: checking out '00bbad4dfd6633cf4b5f9596c1f93b756bb5c776'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at 00bbad4d feat: add AnalyzeIamPolicy and AnalyzeIamPolicyLongrunning RPCs
2020-10-30 01:12:18,591 autosynth [DEBUG] > Running: git checkout 6542bd723403513626f61642fc02ddca528409aa
Note: checking out '6542bd723403513626f61642fc02ddca528409aa'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at 6542bd7 fix(python_library): fix external unit test dependencies (#831)
2020-10-30 01:12:18,598 autosynth [DEBUG] > Running: git branch -f autosynth-700
2020-10-30 01:12:18,601 autosynth [DEBUG] > Running: git checkout autosynth-700
Switched to branch 'autosynth-700'
2020-10-30 01:12:18,605 autosynth [INFO] > Running synthtool
2020-10-30 01:12:18,605 autosynth [INFO] > ['/usr/local/bin/python', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']
2020-10-30 01:12:18,605 autosynth [DEBUG] > log_file_path: /kokoro/artifacts/logs/python-recaptcha-enterprise/700/sponge_log.log
2020-10-30 01:12:18,607 autosynth [DEBUG] > Running: /usr/local/bin/python -m synthtool --metadata synth.metadata synth.py --
2020-10-30 01:12:18,839 synthtool [DEBUG] > Executing /root/.cache/synthtool/python-recaptcha-enterprise/synth.py.
On branch autosynth-700
nothing to commit, working tree clean
2020-10-30 01:12:18,964 synthtool [DEBUG] > Using precloned repo /root/.cache/synthtool/synthtool
2020-10-30 01:12:18,972 synthtool [DEBUG] > Ensuring dependencies.
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/local/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/synthtool/synthtool/__main__.py", line 102, in <module>
    main()
  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.6/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/synthtool/synthtool/__main__.py", line 94, in main
    spec.loader.exec_module(synth_module)  # type: ignore
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/root/.cache/synthtool/python-recaptcha-enterprise/synth.py", line 22, in <module>
    gapic = gcp.GAPICMicrogenerator()
  File "/synthtool/synthtool/gcp/gapic_microgenerator.py", line 46, in __init__
    self._ensure_dependencies_installed()
  File "/synthtool/synthtool/gcp/gapic_microgenerator.py", line 255, in _ensure_dependencies_installed
    f"Dependencies missing: {', '.join(failed_dependencies)}"
OSError: Dependencies missing: docker
2020-10-30 01:12:19,033 autosynth [ERROR] > Synthesis failed
2020-10-30 01:12:19,033 autosynth [DEBUG] > Running: git reset --hard HEAD
HEAD is now at 4ecda99 chore: add default CODEOWNERS (#21)
2020-10-30 01:12:19,038 autosynth [DEBUG] > Running: git checkout autosynth
Switched to branch 'autosynth'
2020-10-30 01:12:19,042 autosynth [DEBUG] > Running: git clean -fdx
Removing __pycache__/
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/local/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/synthtool/autosynth/synth.py", line 354, in <module>
    main()
  File "/synthtool/autosynth/synth.py", line 189, in main
    return _inner_main(temp_dir)
  File "/synthtool/autosynth/synth.py", line 334, in _inner_main
    commit_count = synthesize_loop(x, multiple_prs, change_pusher, synthesizer)
  File "/synthtool/autosynth/synth.py", line 65, in synthesize_loop
    has_changes = toolbox.synthesize_version_in_new_branch(synthesizer, youngest)
  File "/synthtool/autosynth/synth_toolbox.py", line 259, in synthesize_version_in_new_branch
    synthesizer.synthesize(synth_log_path, self.environ)
  File "/synthtool/autosynth/synthesizer.py", line 120, in synthesize
    synth_proc.check_returncode()  # Raise an exception.
  File "/usr/local/lib/python3.6/subprocess.py", line 389, in check_returncode
    self.stderr)
subprocess.CalledProcessError: Command '['/usr/local/bin/python', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']' returned non-zero exit status 1.

Google internal developers can see the full log here.

Autosynth passed, closing! :green_heart:


Adding sample data

Aim is to close #60
Initial push is just the skeleton for getting a single sample dataset into the napari GUI

Latest push adds a new taskipy task that will generate all the necessary code to load a given sample. To do:

[ ] add tests
[ ] add documentation
[ ] decide on what sample datasets to use
[ ] add developer docs for updating enabled datasets

example of current state:
https://github.com/user-attachments/assets/9579a6fd-4c97-44a5-94d3-559467004c0f

thinking that maybe the options in the browser should indicate which field will be loaded

oh, could add some of the smaller timeseries too.

not totally set on the current list of sample data, but can always add more, so going to merge...


README.md outdated. go get is now used differently

What did you do?
go get github.com/pion/stun/tree/master/cmd/stun-nat-behaviour
What did you expect?
installing stun-nat-behaviour
What happened?
got error.
used go install github.com/pion/stun/tree/master/cmd/stun-nat-behaviour@latest and it worked

Has this changed again? I am getting
go: github.com/pion/stun/tree/master/cmd/stun-nat-behaviour@latest: module github.com/pion/stun@latest found (v0.3.5), but does not contain package github.com/pion/stun/tree/master/cmd/stun-nat-behaviour

an nothing will be installed in $GOPATH afterwards.

@yjeanrenaud This change is addressed by PR #119.

Fix is merged to master.

Has this changed again? I am getting
go: github.com/pion/stun/tree/master/cmd/stun-nat-behaviour@latest: module github.com/pion/stun@latest found (v0.3.5), but does not contain package github.com/pion/stun/tree/master/cmd/stun-nat-behaviour

an nothing will be installed in $GOPATH afterwards.

same issue

@Weltolk What command are you running? Where did you get the command from?

@Weltolk What command are you running? Where did you get the command from?

https://github.com/pion/stun/blob/master/cmd/stun-nat-behaviour/README.md
Linux server0 5.10.0-25-arm64 #1 SMP Debian 5.10.191-1 (2023-08-16) aarch64 GNU/Linux
go version go1.21.1 linux/arm64
go install github.com/pion/stun/v2/cmd/stun-nat-behaviour@latest
go: github.com/pion/stun/cmd/stun-nat-behavior@latest: module github.com/pion/stun@latest found (v0.6.1), but does not contain package github.com/pion/stun/cmd/stun-nat-behavior

@Weltolk What command are you running? Where did you get the command from?

I try the new commit, but same
go install github.com/pion/stun/v2/cmd/stun-nat-behaviour@latest
go install github.com/pion/stun/v2/cmd/stun-nat-behavior@latest

@Weltolk I am having a hard time reproducing this :/
$ docker run -it debian:sid /bin/bash

$ apt update && apt install golang ca-certificates

$ go version
go version go1.21.1 linux/amd64

$ go install github.com/pion/stun/v2/cmd/stun-nat-behaviour@latest
go: downloading github.com/pion/stun v0.6.1
go: downloading github.com/pion/stun/v2 v2.0.0
go: downloading github.com/pion/logging v0.2.2
go: downloading github.com/pion/dtls/v2 v2.2.7
go: downloading github.com/pion/transport/v3 v3.0.1
go: downloading github.com/pion/transport/v2 v2.2.1
go: downloading golang.org/x/crypto v0.12.0

$ ~/go/bin/stun-nat-behaviour

@Weltolk I am having a hard time reproducing this :/
$ docker run -it debian:sid /bin/bash

$ apt update && apt install golang ca-certificates

$ go version
go version go1.21.1 linux/amd64

$ go install github.com/pion/stun/v2/cmd/stun-nat-behaviour@latest
go: downloading github.com/pion/stun v0.6.1
go: downloading github.com/pion/stun/v2 v2.0.0
go: downloading github.com/pion/logging v0.2.2
go: downloading github.com/pion/dtls/v2 v2.2.7
go: downloading github.com/pion/transport/v3 v3.0.1
go: downloading github.com/pion/transport/v2 v2.2.1
go: downloading golang.org/x/crypto v0.12.0

$ ~/go/bin/stun-nat-behaviour

I don't know why, amd64 ubuntu have same issue
┌──(root㉿server1)-[~]
└─# uname -a
Linux server1 5.15.0-1040-oracle #46-Ubuntu SMP Fri Jul 14 21:47:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
┌──(root㉿server1)-[~]
└─# go version
go version go1.21.1 linux/amd64
┌──(root㉿server1)-[~]
└─# go install github.com/pion/stun/v2/cmd/stun-nat-behavior@latest
go: github.com/pion/stun/v2/cmd/stun-nat-behavior@latest: module github.com/pion/stun/v2@latest found (v2.0.0), but does not contain package github.com/pion/stun/v2/cmd/stun-nat-behavior
┌──(root㉿server1)-[~]
└─# echo $GOPATH

┌──(root㉿server1)-[~]
└─# ll ~/go
total 4
drwxr-xr-x 4 root root 4096 Sep 24 16:39 pkg

┌──(root㉿server1)-[~]
└─# go install github.com/pion/stun/cmd/stun-nat-behavior@latest
go: github.com/pion/stun/cmd/stun-nat-behavior@latest: module github.com/pion/stun@latest found (v0.6.1), but does not contain package github.com/pion/stun/cmd/stun-nat-behavior
┌──(root㉿server1)-[~]
└─# ll ~/go
total 4
drwxr-xr-x 4 root root 4096 Sep 24 16:39 pkg

I see the issue! Typo in the go install.  Typo in the go install.  I am so sry


host-specific aliases

I'd like to be able to specify an alias as only being available on a certain host (i.e., the current one).

Hi @dmd,
Can you please give me a concrete example because I am not sure I understand?
Thank you

This is probably too out-of-scope, but I have a shared $HOME (via NFS) across a number of different hosts. I was thinking it would be nice if I could say 'goto blah' on, say, my production server, and it would alias to /opt/mycompany/lib/blah, but if I said 'goto blah' on my development server, it would instead alias to /home/dmd/trunk/lib/blah.

I think this is probably too out of scope.

Unfortunately, I think you have to keep different files...
Thank you for reporting!


prague: update chain_id in 7702 set code txs to u256

Updates the EIP-7702 set code transaction type to have a nonce of type u256 instead of u64, see:

ethereum/EIPs#9143.

Please check that no other instances were missed.
Corresponding change in EEST:

ethereum/execution-spec-tests#1026

Cute Animal Picture

Keeping this on hold and unmerged until ACD since there are other proposals. See Update EIP-7702: remove chain_id in authorization tuple and this

This change was agreed upon in ACD 202 and is merged.


Refactor the PushSecret interface

Problem Statement
https://github.com/external-secrets/external-secrets/pull/2792 added typed corev1.SecretType argument to the PushSecret interface, but this does not make much sense since the argument can be used only from the Kubernetes providers. If an interface contains information for a specific member, that indicates something is likely to be wrong with the design. So I've decided to refactor the interface.
Related Issue
N/A
Proposed Changes
Change the interface to PushSecret(ctx context.Context, secret *corev1.Secret, data PushSecretData) error to accept a wider range of information.
Checklist

[x] I have read the contribution guidelines
[x] All commits are signed with git commit --signoff
[x] My changes have reasonable test coverage
[x] All tests pass with make test
[x] I ensured my PR is ready for review with make reviewable

So I've been working on something similar https://github.com/external-secrets/external-secrets/pull/2847

@shuheiktgw No problem at all. I think you're will be more relevant. Passing in the whole secret, I won't have to bother with passing in only the values, I think. And yours is probably more future proof too.

No worries! Thank you for your review, as always 👍
I'm merging this PR so that it won't block other PRs.

Nice, thanks. Now I can work on the pushing all secrets. And it will be a lot cleaner too. :)


Analyzer for no matching workload for gateway selector

Kiali implementation: https://www.kiali.io/documentation/validations/#_no_matching_workload_found_for_gateway_selector_in_this_namespace

Perhaps I am misunderstanding something. Isn't this validation covered in the gateway analyzer?
I see that the analyzer checks if there are any pods matching the gateway selector and then reports an error in case there isn't any matching workload present.
Again, perhaps missing a detail somewhere.

You're right. I think I may have gotten this and https://www.kiali.io/documentation/validations/#_more_than_one_gateway_for_the_same_host_port_combination mixed up, and created an issue for the wrong one. :) An issue exists for the host/port combo now, so I'll close this one. Thanks for your sharp eye!


Always On VPN

Can the app support Always on VPN and/or ability to turn on at startup?

Besides the starting on boot up, the rest is already possible, I think. Android has built in a setting for always-on VPN and mine has been running without disconnects for days now.


https://ai.marrvel.org / how to integrate to Scout and our analysis?

Hello developers,
MARRVEL has released a new tool called MARRVELAI. This tool integrates phenotype data to prioritize variants, a feature currently missing in Scout's variant ranking. MARRVELAI can also extract phenotype terms from journal text, which could be useful for extracting HPO terms from extensive clinical descriptions. You can find more information at https://ai.marrvel.org/.
It may be worthwhile to explore if this tool can be beneficial to us and whether it can be integrated with the Scout and Tip2Toe phenotyping tools.
Thanks,
Fulya

The software is available https://github.com/LiuzLab/AI_MARRVEL under GPL-3 and they have an s3 with data for it, but with the not uncommon "free for research, contact us for business". While many of our users are researchers, the clinical part is sometimes ambiguous in these models - someone who is less annoyed by stuff like that should probably investigate. 😁
It also remains to be seen what can be done with our current division of compute; they appear to have variant scores, but they are somewhat weighted by phenotype and standard runs would involve providing HPO-terms upfront.

Interesting tool. We could look into integrating it but ideally I think you should be able to launch this from scout as well. With the current setup the HPO terms needed would have to be included in the order and passed to the pipeline. We could do a small pilot and test it.

Right, and that would then presumably involve connecting more compute resources / queues to it. While it may or may not be worth it for this one, I am guessing a chunk of the other newer "AI" tools will also try to use phenotype information as input. It makes for an easier genetics problem.

yup, don't know how beefy the current VMs are but it sounds like we would need to connect to a more dedicated compute resource, be it cloud or local.

Precisely; no way with the current config - essentially all outward facing microservices are on the same (somewhat decently sized, but still) VM. We would need to be able to queue jobs in one form or the other.


Suggestions for keyboard manager functionality

Description of the new feature / enhancement
I use the keyboard Manager feature to set up a number of one-touch shortcuts, but I find it cumbersome to switch them occasionally. I thought maybe we could add a shortcut to this feature. And two or more mapping combinations can be set up to allow users to switch between different key mappings in different scenarios.
Scenario when this would be used?
can be set up to allow users to switch between different key mappings in different scenarios.
Supporting information
No response

/dup #1881


Fix typo

Fix Dependendencies -> Dependencies

You'll need to sign off on your git commit to pass the DCO check.


Results Page hangs on long loads

When loading a marginally large number of results (more than 10, less than 300), the results page progress bars just freeze.  The ingestion continues in the background, and reloading the Results page refreshes and restarts the bars, but they just freeze again in a short time.

I don't think there is much that I can do about this.  It is likely due to database load.  My guess is that the refresh routines are timing out and therefore are not finishing, so it doesn't start another one.  We might be able to shift the threshold a little by optimizing the ingestion queries, but beyond that, I don't think there is much that we can do.  And that will come in time as we work toward 1.4 or 2.0.  You might be able to change the number of scans ingested at the same time (2-3) and see if that works better.

Probably needs to be pushed to the rewrite.  For now, we just reload the results page when it hangs.

No need to review - pushed to 1.4.0


Foreground function for Container Widget and WidgetExt

I left a comment about a test failure.
There are also some formatting problems that need to be fixed. There is some valuable information here: https://github.com/linebender/druid/blob/master/CONTRIBUTING.md
Lastly, don't forget to add your PR to the changelog. You can also add yourself to the authors file.

Ready to be merged! @xStrom


Validate all cells on table mount

I have a specific use-case, which includes validating each cell of the table, as soon as the data is loaded:

A user uploads a json file to my Vue app
The table is mounted and the content of the json file is loaded into the table using v-model
Since the json file is user-created and does probably not match some validation criteria yet, I want to validate all cells as soon as the table-data is loaded and display the error messages (using the error-tooltip of the table) directly to the user (for each cell).

Is it already possible to archive this using the current functionality of the component (an example would be appreciated), or does it need some new implementation?
Thanks for the great component, really appreciate the work behind this project.

Since it involves row validation and cell validation. I suggest you trigger an update of each cell, by calling this.$refs.editor.updateCell( rowPos, name, content ). Here is some codes for your references:
tableData.forEach((row, pos) => {
   Object.keys(row).forEach(name => this.$refs.editor.updateCell(pos, name, row[name]))
})

The above code has not been tested, but it contains the all what you need


nvcc: command not found (Vitis-AI-3.0)

After building and entering docker, met this error
nvcc: command not found

And there is no bin file folder in path: /usr/local/cuda-11.3/

How to fix it in Vitis-AI-3.0? I have not met this problem in version 3.5, but i am using MPSoC, so whether 3.5 can be used for MPSoC devices？Or I must change to 3.0?
Thanks for your kind reply!

Yes, you must change to v3.0.
MPSoC is only supported to v3.0.
If you want to use cuda, can you try to install the missing part manually?

Thanks for your comment, should I use "opt_pytorch" for the target framework?

Hello, please did you solve this issue? cuz am having the same issue? I am using Vitis 3.0 to quantize yolov5, Ihave GPU NVIDIA GeForce RTX 4060, and am using CUDA 11.3.1 in vitis?

Yeah, I have solved this issue by installing CUDA 11.3 and cudnn manually

Yeah, I have solved this issue by installing CUDA 11.3 and cudnn manually

Could you please share with me the command lines how you installed it?? much appreciated, thank you.