KFImage does not render when applying modifier

Check List
Thanks for considering to open an issue. Before you submit your issue, please confirm these boxes are checked.

[x] I have read the wiki page and cheat sheet, but there is no information I need.
[x] I have searched in existing issues, but did not find a same one.
[x] I want to report a problem instead of asking a question. It'd better to use kingfisher tag in Stack Overflow to ask a question.

Issue Description
When using a modifier that transforms a view which contains a KFImage into a Button. The download process seems to fail and the image does not show even if the callback for the KFImage returns successful.
What
I have view which contains the KFImage such as:
var body: some View {
    VStack {
        KFImage(url)
    }
}

And I apply a modifier to this view which will result in the content being inside the button.
self // <--- View which contains KFimage
    .modifier(
          Button(
              action: { //some action code.  },
              label: { content  // <--- View which contains KFimage }
         )
    )

Sorry for the short code description, I tried to simplify as much as I could to pass down the issue. First time creating an issue related to KF.
Other Comment
[Add anything else here]

@hcjordao
Thanks for reporting thi.
But I am still confusing which view is containing KFImage. Is it self or content or both?
Can you paste a working example in which we can reproduce this issue? So we can ensure we are in the same context and try to dig into it.

No problem 🚀
They are both the same view. Self is used because it will act like a function.
The function would look something like this:
viewWhichContainsKingfisherImage.modifierToButton()

This will result in the view being embeded inside the Button in SwiftUI

Did you figure out how to solve this? I've noticed when I have a KFImage in something like a Button, it doesn't download the image and update the view

I realized my issue is actually that SwiftUI natively changes the image type for Buttons and Navigation links. I had to set .renderingMode(.original) on the Image, and that fixed it for me!

That also solved for me thanks @ZMcGuckin 🚀

Glad I could help! It was bugging me as well and when I swapped King Fisher for another Swift Package I saw the solution there


2.0.2 binary shows wrong version

Steps to reproduce:
wget https://github.com/PeggyJV/steward/releases/download/v2.0.2/steward
chmod +x ./steward
./steward --version
steward 2.0.1

Bumping this. Same issue with ~ v2.1.1 -- come'on folks...

We've added a version check to the release pipeline


current position property

i need to know the current position of the top of the drawer in order to move elements around in the containing viewController.
i would use the origin of the PullUpController frame.. but this seems hacky and i need observation on this value for real time changes.
also i can't access topConstraint.
i suggest adding a public function pullUpControllerTopPositionChangedTo(_ position: CGFloat) after changes to topConstraint are made.
Edit:
i can see that this was merged to the repo but reverted in https://github.com/MarioIannotta/PullUpController/pull/7
why was this feature removed?

Hi @idanboa
the PR you mentioned was reverted because the same functionality has been integrated with this PR in a more swifty way.
But I guess that what you need is a little bit different.
I've added the closure
/**
  The closure to execute when the view controller's view is dragged.
  The point, expressed in the pull up controller parent coordinate system, is provided in the closure parameter.
*/
open var onDrag: ((_ point: CGFloat) -> Void)?

on the develop branch.
Could you please check if that works for you?
Thanks,
Mario

Hi, thanks for the response 👍.
this is a good start, but shouldn't this value be representing the top position at all times and not just on drag? there is an option to animate positions programmatically that this closure is not called for, basically every time a change is made to topConstraint?.constant. why not have a unified block that notifies all movements?
Disclosure: i am currently working with a forked version of the library to accommodate for special cases in my app 😊

I thought about that but then I was wondering: when you would like to be notified while changing the position programmatically? At the beginning of the animation or at the end?
Since we already have

open var willMoveToStickyPoint: ((_ point: CGFloat) -> Void)?
open var didMoveToStickyPoint: ((_ point: CGFloat) -> Void)?
to handle those actions so you just need a closure that notify you when the user is dragging the controller's view.

That's the best solution i could think to handle your use case whilst keeping the apis clean and simple.


Accept various parameters

This pull request permits any parameters not only String.
e.g.
log.verbose()
log.debug(1.2)
log.info(true)
log.warning(["a", "b", "c"])
log.error {
    return NSDate()
}
let optionalString: String? = "text"
log.severe(optionalString)

Thanks for this PR. Merged in, albeit with some changes. Using Any? instead of Any so returning nil from a closure is still an option. Also, have added a noMessageClosure property to allow for customization of the message that is displayed when you call a logging method with no parameters: log.debug().


LL.TS Test LL/CON/SLA/BV-129-C fails (split)

With the introduction of commit "Bluetooth: controller: split: fix LL/CON/MAS/BV-74-C tx timing" (0ac4433a12), the test LL/CON/SLA/BV-129-C fails on EBQ.
To Reproduce
Run LL/CON/SLA/BV-129-C on EBQ on "split" controller, without coded PHY.
Expected behavior
Without commit 0ac4433a12 the test passes - with it, it fails.
Impact
Qualification on Zephyr 2.2 will fail.
Additional context
Non-Nordic HW platform and link layer used.

CC @kruithofa @cvinayak


 documentation missing

Hello,
I would like to customize the Container arround every T3CeXXX, so I thought to customize the  component, but

I don't know how (maybe you can tell me here),
I don't know if it's the right place for dealing with margins, classnames, ... of the wrapper arround every T3CeXXX component

That Link is dead: https://typo3-headless.github.io/nuxt-typo3/frontend/lib/components/content/CeRenderer.js
Got it from here (at https://typo3-headless.github.io/nuxt-typo3/frontend/)

@spot-mathis the file you are looking for is https://github.com/TYPO3-Headless/nuxt-typo3/blob/master/lib/components/T3Renderer/T3Renderer.js
This component is responsible for iterating through content array  from specific colPos (TYPO3 API) and creating dynamic component based on current item type. You have to notice there is also condition to check whether it should render ContentElement Frame or not.
  // render loop of content elements
    return ctx.props.content.map((element, index) =>
      ctx.props.frame && element.appearance.frameClass !== 'none'
        ? renderFrame(element, index)
        : renderComponent(element, index)
    )

You can override this component in regular way like you override global components.
This file is written in render function, but you can use regular template in your own version.

So to override container which is responsible for wrapping of all content elements you should override page file which is called _.vue in pages directory (https://github.com/TYPO3-Headless/nuxt-typo3/blob/master/lib/pages/_.vue) - just copy this file to your application
there is:
<template>
  <div>
    <t3-dynamic
      :data="t3page.content"
      :type="t3page.appearance.backendLayout"
      layout
    />
  </div>
</template>

so this div you see above is wrapper for all content elements.
To handle margins between content elements we use T3Frame Component https://github.com/TYPO3-Headless/nuxt-typo3/blob/master/lib/components/T3Frame/T3Frame.js -> this component takes fields from appearance of current content element and setup specific classes:
class: [
          't3-ce-frame',
          `frame-${props.frameClass}`,
          `layout-${props.layout}`,
          `space-before-${
            props.spaceBefore.length ? props.spaceBefore : 'default'
          }`,
          `space-after-${
            props.spaceAfter.length ? props.spaceAfter : 'default'
          }`
        ],

We use fields space-before, space-after to setup specific margins, we use also frame and layout classes to do some specific work.
So to override single content element wrapper you can override T3Frame


Upgrade modal dialog with draggable functionality

Let's add this feature to modal dialog component that will be controlled by option called "draggable".

Not working on iPad.


refactor: try upgrade regex-automata

I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
What's changed and what's your intention?
This refers to #3043.
Checklist

[ ]  I have written the necessary rustdoc comments.
[ ]  I have added the necessary unit tests and integration tests.
[x]  This PR does not require documentation updates.

Seems some internal impl issues exist; the following cases failed:
index inverted_index::search::fst_apply::intersection_apply::tests::test_intersection_fst_applier_with_valid_pattern
mito 2sst::index::creator::tests::test_create_and_query_regex

@zhongzc it just compiled and passed test. I suppose you do some end to end test for where you need it in the index functionality ..
Also, to glue the regex-automata and fst together, "extra" checks are involved. I don't test the performance after this change.

Also, to glue the regex-automata and fst together, "extra" checks are involved. I don't test the performance after this change.
I'm not currently motivated to check the performance of fst matching regex statements. In my manual end-to-end performance testing, it is not a critical impact point.

I suggest you do some end to end test for where you need it in the index functionality ..
Once we have e2e testing one day, I will add enough test cases for index

I feel like it’s more correct to write it like this,
ref to https://github.com/BurntSushi/regex-automata/blob/0ba880134d649866fa15809dec9c6eae89cd7591/src/dfa/transducer.rs#L41-L74
impl fst::Automaton for DfaFstAutomaton {
    type State = StateID;

    #[inline]
    fn start(&self) -> Self::State {
        let config = Config::new();
        self.0.start_state(&config).unwrap()
    }

    #[inline]
    fn is_match(&self, state: &Self::State) -> bool {
        self.0.is_match_state(*state)
    }

    #[inline]
    fn can_match(&self, state: &Self::State) -> bool {
        !self.0.is_dead_state(*state)
    }

    #[inline]
    fn accept_eof(&self, state: &StateID) -> Option<StateID> {
        if fst::Automaton::is_match(self, state) {
            return Some(*state);
        }
        Some(self.0.next_eoi_state(*state))
    }

    #[inline]
    fn accept(&self, state: &Self::State, byte: u8) -> Self::State {
        if self.0.is_match_state(*state) {
            return *state;
        }
        self.0.next_state(*state, byte)
    }
}

@zhongzc Cool. Let me try it out.

It works. Burnt gives a more concrete implementation in https://github.com/BurntSushi/aho-corasick/blob/56256dca1bcd2365fd1dc987c1c06195429a2e2c/src/transducer.rs, which handles Anchored and Unanchored separately.
In our case, it should be always unanchored, meaning we don't ensure the patter match the whole string, but part of the string is OK.


Intellisense becomes very slow with Ionide F# extension

Issue Type: Bug
In any F# project of significant size the intellisense becomes unworkable slow.
VS Code version: Code 1.21.0 (9a199d77c82fcb82f39c68bb33c614af01c111ba, 2018-03-07T10:54:22.798Z)
OS version: Darwin x64 17.4.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-3740QM CPU @ 2.70GHz (8 x 2700)

Load (avg)
3, 3, 3

Memory (System)
16.00GB (1.99GB free)

Process Argv
/Applications/Visual Studio Code.app/Contents/MacOS/Electron

Screen Reader
no

VM
25%

Extensions (5)

Extension
Author (truncated)
Version

Ionide-FAKE
Ion
1.2.3

Ionide-fsharp
Ion
3.17.3

vscode-docker
Pet
0.0.25

language-haskell
jus
2.4.0

csharp
ms-
1.14.0

Reproduces only with extensions

This is a F# specific issue.
Moved to https://github.com/ionide/ionide-vscode-fsharp/issues/741. Please continue the discussion there.


feat: add startupProbe

Closes #1964

/test limit=local

/test limit=local

The startupProbe implementation is complex without adding enough benefits with the current reconciliation logic that relies on HTTP status calls.
Closing without merging, I think we can revisit the opportunity to add the startupProbe once we simplify the operator reconciliation loop.


Create fresco as an amuse.ext package

fixes #19

maybe also include the fresco.py standalone as script (will be copied to bin)??

Thanks, that would indeed be nice (I’m trying to figure out how that works :) )

I think adding scripts="fresco.py" to the setup call is enough..

but it needs a shebang

done in 0.5.1


Terra theme properties updates

Summary

What was changed:
Update the semantic variables to the latest build from the user experience team.  The version of stylelint was also updated to 15 to allow overrides and consistency with terra-core.
Why it was changed:
The change was made so that developers can use the most recent set of theme variables in development.
Testing

This change was tested using:

[ ] WDIO
[ ] Jest
[ ] Visual testing (please attach a screenshot or recording)
[ ] Other (please describe below)
[X] No tests are needed

Reviews
In addition to engineering reviews, this PR needs:

[ ] UX review
[ ] Accessibility review
[ ] Functional review

Additional Details
N/A
This PR resolves:
UXPLATFORM-9931

Thank you for contributing to Terra.
@cerner/terra

This is technically part of the initial release so I am not updating the changelog file as it would not make sense to provide an update for each UX update.


adds linuxkit build

Issue #, if available:
Description of changes:
This is the bare minimal to get us started building the various pieces that make up linuxkit and hook's kernel build.  This simply adds the build for the linuxkit bin and the images needed here.
I will have a follow that changes the hook build to rely/pull these built images instead of those from upstream.
I also plan to poke at and gradually update these images to follow our standard process and be al2 based.  Most of these images actually execute on the hook os via containerd as containers.  Those are the ones listed as onboot or services in the template.  For these we should be able to change them to whatever since it does not matter what the host is.  For the images listed in the init section of the template, those are what are added to the "host" itself, so like containerd and runc as well as the base OS, which is currently from upstream.
/hold
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/test all

/unhold


(ignore this for now!) API Calls tracking

Send metric to prometheus every time an api is called.
prometheus metrics:
api_called{endpoint="</api/...>"}

not gonna do it


Add aubrey description

@vphill @somexpert  just wanted to add a brief statement about what aubrey is. Does that work for you guys?

Works for me :thumbsup:

works for me 👍


Fix volume template for timescaledb-multinode

Fix for issue #275

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.svolland seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Please abandon this pull request, replaced by https://github.com/timescale/timescaledb-kubernetes/pull/277


the page show 500 error when I open my repository

Gitea version (or commit ref):
Git version:
Operating system:
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] No
[ ] Not relevant

Log gist:

Description
logfile:
..ules/context/repo.go:605:func1() [E] GetBranchCommit: object not found
system notices:
Failed to health check repository (/home/ubuntu/tp/gitea/repositories/tom/jerry.web.git): exit status 2 - error: refs/heads/master: invalid sha1 pointer 0465a55c16418068b7e8100832b045db6dee5d06 error: HEAD: invalid sha1 pointer 0465a55c16418068b7e8100832b045db6dee5d06 notice: No default references
Screenshots

I execute the gc,return(Application Version: 1.9.5):
exec(21:Repository garbage collection) failed: exit status 128() stdout: stderr: error: refs/heads/master does not point to a valid object! error: refs/heads/master does not point to a valid object! fatal: bad object HEAD fatal: failed to run repack : error: refs/heads/master does not point to a valid object! error: refs/heads/master does not point to a valid object! fatal: bad object HEAD fatal: failed to run repack
摘要

Have you moved your repositories? Check if the files are still there... 😰

@stepm2 do you have changed the location where repository files are stored in your config?
or did you move this directory

It looks like the repository on the filesystem has been corrupted. Are your disks fine? Have you had any unexpected shutdowns recently?
You should try going to a command line and try git gc in the git repository that Gitea uses. You could try pushing from a working copy again.
Please also note Gitea 1.9.5 is now quite old. We're undergoing quite rapid development and there have been several security bugs fixed since then.

It looks like the repository on the filesystem has been corrupted. Are your disks fine? Have you had any unexpected shutdowns recently?
You should try going to a command line and try git gc in the git repository that Gitea uses. You could try pushing from a working copy again.
Please also note Gitea 1.9.5 is now quite old. We're undergoing quite rapid development and there have been several security bugs fixed since then.

I did have an abnormal shutdown before, and I tried to use the git gc command, but returned the following message：

error: refs/heads/master does not point to a valid object!
error: refs/heads/master does not point to a valid object!
fatal: bad object HEAD
fatal: failed to run repack

this looks like the data broke on this abnormal shutdown :(
I hope you have a backup of it ( do you have some local repo closen) ?

Probably, I have a local backup, but I still want to find before history, thank you so much.

Best of lucks!


ISM policy behaviour with disk-based shard allocation settings

Hi!
I have a cluster with two host data nodes, one warm data node and one cold data node. I have a policy which moves indices from hot node to warm node, when index grows up to 1gb.
        "states": [
            {
                "name": "hot",
                "actions": [
                    {
                        "rollover": {
                            "min_size": "1gb"
                        }
                    }
                ],
                "transitions": [
                    {
                        "state_name": "warm"
                    }
                ]
            },
            {
                "name": "warm",
                "actions": [
                    {
                        "force_merge": {
                            "max_num_segments": 1
                        }
                    },
                    {
                        "allocation": {
                            "require": {
                                "data": "warm"
                            },
                            "wait_for": false
                        }
                    },
                    {
                        "replica_count": {
                            "number_of_replicas": 0
                        }
                    }
                ],
                "transitions": [
                    {
                        "state_name": "cold",
                        "conditions": {
                            "min_index_age": "14d"
                        }
                    }
                ]
            },
            {
                "name": "cold",
                ...
            },
            {
                "name": "delete",
                ...
            }
        ]

I'd like to know, what will happen if warm data node hits cluster.routing.allocation.disk.watermark.high threshold. Will indices (which should be moved from hot to warm) stay on hot nodes?
Is it possible to recover indices, which were corrupted because of no disk space on warm data node? These are typical log messages I got while warm node was out of space:
[WARN ][o.e.i.c.IndicesClusterStateService] [warm] [my-index-000024][0] marking and sending shard failed due to [failed recovery]
org.elasticsearch.indices.recovery.RecoveryFailedException: [my-index-000024][0]: Recovery failed from {hot-2}{...}{...}{...}{...}{dm}{data=hot} into {warm}{...}{...}{...}{...}{d}{data=warm}

Thank you in advance!

@Tarasovych How this question relates to open distro security plugin?


Explicitly disable asm solvers if XNACK is enabled

By @atamazov: From now on we need to officially support XNACK enabled GPUs. This PR adds very simple support in Solvers that use ASM kernels: so far, none ASM kernels were are ready for XNACK enabled GPUs, so we need to disable them.
Also reverts some incorrect fixes introduced in #809 and #1110 (details here)
Previous version of description
Description
As I mentioned in the #1093, some asm kernels may not support xnack, but this is not checked anywhere.
This pull request temporarily disables all asm solvers if xnack is enabled.
Next, asm kernels must be checked for support xnack.
Side effect
winogradRxSf2x3 now works correctly on the gfx90a with disabled xnack

@Slimakanzer Please revise all cases when some assembly Solver is disabled specifically for gfx90a but enabled for other gfx9 parts (see #809 for example). Actually gfx90a should be able to execute any gfx9 assembly shaders, so such changes should be reverted. The XNACK check should do the necessary job instead. Thanks!

Description updated. Converted to draft.

@Slimakanzer could you please take a look:

54 - test_tensor_vec (Failed) for Int8 HIP All Vega20
ref: http://micimaster.amd.com/blue/organizations/jenkins/MLLibs%2FMIOpen/detail/asm-xnack-workaround/2/pipeline

@junliume This also enables ConvBinWinogradRxSf2x3 for gfx90a:xnack- and removes W/A for #1093. Unfortunately we forgotten to update the PR description and (more importantly) commit message.


Commit 4021c59 ignores desktop option in

I have following group settings:
<Group>
<Name>chromium-browser</Name>
<Option>desktop:5</Option>
</Group>
Up until commit 4021c59 it worked as expected. But this commit introduced this code (client.c: line 418):
SetClientDesktop(np, currentDesktop);
Which brings active windows to current desktop regardless of group settings. It seems to do this whenever application state change so when my music player which is always on desktop 9 change track, it is brought to current desktop.

I think the solution here is to add another group option to either prevent an application from switching desktops or prevent it from switching desktops on raise.

As of snapshot 1431, there is a new group option, fixed, to keep the client on the specified desktop even when it is raised.  With this option, JWM will change to the desktop with the window rather than moving the window.  This probably doesn't completely address this issue, but it may help in some cases.

The fixed option addresses this issue for me, so I'm marking this as closed.


Mininum javac version in README

Adding information on README file about the minimum javac version required (the project uses javafx Alert class, which was introduced in 1.8.0_45)

Have a look at #139, I think you will like it :)

Is this still needed? As @Inventitech already updated it on #139

I don't think so! You can close this one!
On Fri, Dec 9, 2016, 21:46 Clinton Cao<EMAIL_ADDRESS>wrote:

Is this still needed? As @Inventitech https://github.com/Inventitech
already updated it on #139
https://github.com/ClintonCao/UnifiedASATVisualizer/pull/139
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ClintonCao/UnifiedASATVisualizer/pull/138#issuecomment-266118841,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGJzB_0f9CH_Snj5hwdr2W3tblmRCWVks5rGb4ngaJpZM4LJDkZ
.

--
Maurício Aniche
Postdoc researcher
Delft University of Technology
www.mauricioaniche.com
@mauricioaniche

Alright then


toggleControls rewrite

Fixes #883.

Makes utils.toggleClass() work like Element.classList.toggle(), by allowing the toggle argument (renamed to force to match Element.classList.toggle documentation) to be optional.
Removes unused method ui.checkFailed() and related code, since this code gave the allure of integrating with toggleControls.
toggleControls is split up in two methods.

The one on the plyr instance is "dumb" and ignores any state as long as the ui is supported and it's not audio. It also triggers the controlshidden and controlsshown event. Like utils.toggleClass, the actual toggle argument is optional.
ui.toggleControls toggles the controls based on state (loading, paused, controls.pressed andcontrols.hover). If any of those are true, or if you pass true as the force argument the controls will be shown. Otherwise not
All the event handling code is in listeners.js, and instead of deep nested conditions it checks and sets state in order to have ui.toggleControls handle it.

I removed mouseenter and touchend since mousemove / touchmove, handles these cases. At least in my tests in Chrome (with/without "device" simulation) mouseenter is always followed by mousemove and touchend should only happen after the hide timeout is already triggered by touchstart or touchmove.
The noTransition class is applied on focusin and removed on focusout regardless of the device being a touch device or not. Previously this was only for non-touch devices, but since touch devices can have keyboards this wasn't a safe assumption I think. I assumed it was added to prevent the browsers from forcing the focused element into view prematurely (causing the parent to scroll and look broken). This issue can be replicate on the plyr demo with the device simulator and focusing a button into view via keyboard.
There is an issue with the youtube object not setting paused to false before triggering the play event that I'll try to fix separately in the youtube/vimeo PR to avoid conflicts.

Nice work. I guess we should put some of this stuff in a branch other than master so I can test?
The error stuff was WIP but I guess I can come back to this PR to grab that again.

You can test a PR locally without merging. I use git-extras (recommended since it's very easy to switch and not much to learn) which has an alias git pr for it or this way without git extras.
I'm still working on the youtube/vimeo one since the events still aren't triggered in the right order with that PR as it is now. The paused property should always be set before the play and pause events are triggered for example. Now it's set after. Should be done tonight though.


Selection should clear input text

Since AngularJS 1.3.0 the "multi selection" doesn't clear the input text if an item is selected.
"multi selection" is working with AngularJS 1.3.0:
http://plnkr.co/edit/8SmB9sy1stlTn8wh6STp?p=preview
"multi selection" is not working with Angularjs 1.3.1 anymore:
http://plnkr.co/edit/Wgl0Y2S0RpVCZTtHqkE4?p=preview
The example https://github.com/angular-ui/ui-select/blob/master/examples/demo-multi-select.html with the AngularJS version 1.3.0-beta.17 is also effected.

I think this is also true for single selection. (also when no value is selected the input value should be cleared)
Right?

@maku Think so

bump. I'm also having this issue

Should be solved with https://github.com/angular-ui/ui-select/pull/703
Check updated Plunker


Pie Chart is not drawing label lines when toggling labels and redrawing

Version
4.2.1
Steps to reproduce
Pseudocode:
var option = pie chart with label: {show: false}
chart = echarts.init(...)
chart.setOption(option)
all fine ... chart is drawn without labels
now make a new
var option_label = pie chart with label: {position: 'outside', show: true}
and redraw with
chart.setOption(option_label)
chart is drawn with labels. BUT there are no lines lines to the labels
What is expected?
Labels should be drawn with lines.
What is actually happening?
Labels are drawn without lines

If the first draw is omitted, lines are drawn just fine.
The other way round first plotting with labels and then plotting without labels will remove the labels, BUT the lines are STILL HERE!

First draw without label, then with label

First draw with label, then without label

When drawn with label it should look like this

Please provide a demo at https://gallery.echartsjs.com/editor.html or http://jsfiddle.net

Here you are

Removing Labels
https://jsfiddle.net/4qcfbx1a/8/
when you click removeLabel, the label is removed, but lines are still there.

Adding Labels
https://jsfiddle.net/4qcfbx1a/9
when you click addLabel, the label appear, but lines are not drawn.

You should add labelLine option
https://echarts.apache.org/option.html#series-pie.labelLine

@susiwen8 is right.
@zevero You may use
option.series[0].label.show = true;
option.series[0].labelLine = {show: true};

Thank you a lot!


chore: fixes Devnet.Ts

Can you run npm run prettier ? It seems the diff has a lot of prettier difference 🤔

Can you run npm run prettier ? It seems the diff has a lot of prettier difference 🤔

also did in last commit


Allow me to have the option for two dock panels (one for debugger, the other for files)

I would like to be able to view the watch window and the source window at the same time.   I have a large monitor and find it frustrating to have to toggle between these when debugging. I would make this a configurable option, like the mini map, but I also thing it needs a menu command.  Perhaps even right click on the debug icon and "open in new panel" and you can drag the panel to dock left or right.

I have VC 1.11.1:
Windows 10

You basically want to be able to move the viewlet into the panel area. There is already a feature request open for this and I have it on the far away radar


[Pipeline Witness] Track individual test failures

It would be very helpful for flaky tests prioritization point of view to have a list of individual failed tests.
@mitchdenny do you think it's possible to do with the current Pipeline Witness model?
One day we might evolve it into something that files flaky test issues automatically.
cc @maririos

It is possible we can pull this off. In the event of a build failure, if we detect it is a test failure we make subsequent calls to Azure DevOps to interrogate the test results and then append them to the Scope string fields. Then, we can look at a report which effectively groups by that scope string and filters by test failure and it'll give you a picture of the biggest contributors to test failures.

Obsolete.


problem about retraining the model

Excuse me, I want to retrain the model in the Flyingthings3D dataset and have set the parameters according to the paper. But I find a strange thing: step=85000 and 475000, the optical flow epe error is reduced from 12 to 7. The disparity and disparity change epe error are still at 42 and didn't fall down. What's the problem behind it?
Besides, I wonder to know how to use the knowledge distillation for better performance.
Thanks very much.

Hi,
The network requires a long training time (especially in the synthetic domain). However, what do you mean by 42% epe error? If the network has a 42 epe error, there is something wrong (maybe in data or in training setup).
About distillation, I used it only in the real domain

Thanks for you reply.
Yes, the network has a 42 epe error with respect to the disparity and disparity change. Since  ./configurations/sceneflow.json lacks the keys and parameters for training mode, I have set up the configuration on flyingthings according to the paper. Maybe something was missing. Could you please provide the training setup for sceneflow.json? Thanks a lot!

Sorry to bother you again~
With the help of you, I have almost achieved the same performance on Flyingthings3D. However, when trying to finetune the model on KITTI just with the provided gt, I got the results as follows:F1-27%,D1-6%,D2-9%(Noc);F1-36%,D1-6%,D2-15%(All).
I think the flow outliers are far from the error 18.53% in the paper obviously. Is the ablation study of KITTI in terms of Noc or All? And I wonder to know if you employed cross-validation or prolong the training process or just finetune the model with the kitti.json?  Some suggestions are helpeful~
Thanks very much!

Hi,
The results are about OCC validation of the model fine-tuned on KITTI

Thanks a lot! I still have problems about the performance on KITTI dataset. The error is far from the normal value. I'll check it. Thanks for your attention to this matter.

In fact, I create txt for kitti_170 split according to your kitti_160.txt and use the last 30 pictures as validation. And do you mean I should modify "NN as interpolation" in "filters_setting{method :NN}" of the kitti.json in the configurations?
I didn't modify the parameters in the kitti.json before. After setting the filters_setting:method as NN, I got the following results with respect to OCC: F1-33%,D1-5%,D2-13%. Maybe something wrong with corr2d when run "python ops.py“. It is strange that only flow outliers is abnormal.
Thanks a lot !

Are you using the evaluate_partial_scene_flow script inside external_packages/kitti_test_suite for evaluation (make sure to change if you are using 170 images for training, and not 160)?

Thanks for your attention. For valuation, I use the KITTI official code implemented in MATLAB. By the time, I got the OCC results for 30 valuation set  as follows: F1-22%,D1-4%,D2-9%(All). I think they are closer to the results only using KITTI gt in your awesome paper~ Would you consider it is the reasonable result?
I am very grateful for your guidance.


Runtime error spec

Runtime errorspec changes:
Include slit user & runtime error log
Aggregate log into a yaml file

Coverage remained the same at 52.932% when pulling 708463a9f82b3911e641eaf209212e9b38d33328 on runtime_error_spec into 48a5bad172a82e9f7b006c27a3185375988d8f2b on errorspec.

Coverage remained the same at 52.932% when pulling 708463a9f82b3911e641eaf209212e9b38d33328 on runtime_error_spec into 48a5bad172a82e9f7b006c27a3185375988d8f2b on errorspec.

Coverage remained the same at 52.932% when pulling 708463a9f82b3911e641eaf209212e9b38d33328 on runtime_error_spec into 48a5bad172a82e9f7b006c27a3185375988d8f2b on errorspec.

Coverage remained the same at 52.932% when pulling 708463a9f82b3911e641eaf209212e9b38d33328 on runtime_error_spec into 48a5bad172a82e9f7b006c27a3185375988d8f2b on errorspec.


As a registered User, I want to be able to place my pieces on the board before the game in order to be prepared for the game to start.

[ ] A user can start placing his pieces onto the board before the game has started.
[ ] The game will not start unless both players have placed their pieces.
[ ] A user can only place pieces in his own half of the board.
[ ] Each user must place his 40 units, no more, no less.
[ ] Before all the pieces have been placed onto the board, the “confirm” button is disabled.
[ ] sopra-fs23-group-22/sopra-fs23-group-22-client#27
[ ] sopra-fs23-group-22/sopra-fs23-group-22-server#46
[ ] #81
[ ] #82
[ ] sopra-fs23-group-22/sopra-fs23-group-22-client#16
[ ] #83
[ ] sopra-fs23-group-22/sopra-fs23-group-22-client#30
[ ] sopra-fs23-group-22/sopra-fs23-group-22-server#49

tentative version: a very detailed skeleton code of the design, which might have errors and to be finalised and test together with front-end development. Since we cannot implement every perfectly at the moment due to lake of other implementation detail, such as data converted to back-end, front-end implementation etc.


[SPARK-9483] Fix UTF8String.getPrefix for big-endian.

Previous code assumed little-endian.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

/cc @davies

Jenkins, this is ok to test.

Merged build triggered.

Merged build started.

Test build #39580 has started for   PR 7902 at commit 17d54c6.

@mtbrandy  not 100% sure, but it might be more clear if you just have a top level branch based on endianness.

@rxin, regarding top-level branch -- I can do that, but was just trying to minimize duplicate code.

+1 for moving the endianness check to top level, otherwise LGTM.

ok, I'll modify to move the endianness check up.

Merged build triggered.

Merged build started.

Test build #39589 has started for   PR 7902 at commit ec31df8.

Test build #39580 has finished for   PR 7902 at commit 17d54c6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

@davies I will let you merge this.

Test build #39589 has finished for   PR 7902 at commit ec31df8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Merged into master and 1.5-branch, thanks!


Rename to Eno$Something

Seems like this will contain everything  but the landing page, so something like EnoCtfFrontend or literally  any other name would be more fitting

yarp you are right.

EnoCTFPortal
EnoCTFPlatform
EnoReg

EnoCTFPortal it is


Support HSV colors.

This branch adds support for HSV colors.  I implemented most of the API of HSL.  What's not implemented (If it's desired), can easily be implemented by converting the HSV to HSL, doing the computation, and converting back.

Closing, meant to send upstream.


Schema validation - A Default value not matching the type of  property does not error on validateSchema/Compile

What version of Ajv are you using? Does the issue happen if you use the latest version?
V7.0.3 and yes it happens for previous versions too.
Ajv options object
{ useDefaults: true }

JSON Schema
{
    "$schema": "http://json-schema.org/draft-07/schema",
    "type": "object",
    "properties": {
        "test": {
            "type": "boolean",
            "default": "mp3"
        }
    },
    "additionalProperties": false
}

Your code
https://runkit.com/parora611/ajv-issue
What results did you expect?
Error should have been thrown on compile.
Are you going to resolve the issue?
Not sure as of now.

This is not a bug, it is a design choice. Ajv does not validate defaults in the schema as part of schema validation. The problem is that with JSON schema there is no general way to determine whether a part of the data is valid according to the part of the schema - there may be dependencies between parts of the data/schema.
I was sure this issue was discussed before but I couldn't find it - please link if you come across it in some older issues.

This is not a bug, it is a design choice. Ajv does not validate defaults in the schema as part of schema validation. The problem is that with JSON schema there is no general way to determine whether a part of the data is valid according to the part of the schema - there may be dependencies between parts of the data/schema.
I was sure this issue was discussed before but I couldn't find it - please link if you come across it in some older issues.


Reorganize some callouts in etcd admin task

Improve https://kubernetes.io/docs/tasks/administer-cluster/configure-upgrade-etcd/

Aim to reduce the large number of note callouts

promote one to a caution.

Preview

/sig etcd


DBZ-92, DBZ-97 Makes logging more verbose and changes the snapshot reader to produce separate events for each DDL change

@rhauch: the CI build seems to fail because there are much more schema changes reported than expected (locally the build passes for me).
I'm not sure why this can happen, except for the fact that again test ordering matters and certain tests are run in different order locally versus on Travis. This most likely means that there are tests which insert data in the test case (not via the docker image) and never clean it up.

@hchiorean Great! Our team needs it.

amended the commit

/home/travis/build.sh: line 45: docker: command not found...not sure what that's about

@hchiorean docker: unrecognized service
https://travis-ci.org/debezium/debezium/builds/150970664#L156
I also meet several docker issues on Travis, but them didn't happen on my local Mac. One of them is https://issues.jboss.org/browse/DBZ-93 [travis] Unable to create data directory /zookeeper/txns/version-2

@hchiorean, @mvj3: Travis are having issues at the moment, though at this time they've identified the problem. I'll restart the job when they fully resolve the problems.

This pull request failed on Travis multiple times with errors like the following after the first build (recall that the build attempts to re-run maven upon an error, and the subsequent failures are different):
[ERROR] Failed to execute goal io.fabric8:docker-maven-plugin:0.15.12:start (start) on project debezium-connector-mongodb: I/O Error: Unable to start container id [39fdb7dcd6cb]: driver failed programming external connectivity on endpoint mongo1 (ae0bf53b1e6f17f44e47c298998223fda6510667ee088f0eb1590d3bf4a839cc): Error starting userland proxy: listen tcp <IP_ADDRESS>:27017: bind: address already in use (Internal Server Error: 500) -> [Help 1]

This error implies that the mongo container that we're trying to start is failing because the port 27017 is already being used. Either something is still messed up in the Travis environment, though late yesterday they reported everything is fine while today they've said people are still having problems and that people should contact them if that is the case. I've sent them an email asking whether this might be related to the recent problems or if port 27017 is now reserved in some way. I'll keep you posted.
In the meantime, @hchiorean perhaps you might try creating a pull request that changes the MongoDB port to something else (e.g., 28017). Unfortunately, it needs to be changed in two places; feel free to set a property to make this easier.

I'm not sure what's going on, but this builds locally. Since I'm certain that nothing in the PR is broken but instead there's something wrong with the Travis environment, I'm going to merge this anyway.

Travis-CI support responded, and they've upgraded their images to ensure MongoDB is disabled upon boot. Woot!


Orchestrator support: Argo Workflows

Is your feature request related to a problem? Please describe.
We run dbt-core (within Meltano - see https://github.com/elementary-data/elementary/issues/1160 ) using Argo Workflows. Per https://docs.elementary-data.com/deployment-and-configuration/collect-job-data#cant-find-your-orchestrator-missing-info I'm opening this issue because elementary doesn't currently support an Argo Workflows orchestrator.
Describe the solution you'd like
It would be great to have Argo Workflows metadata captured in the same orchestrator metadata model as others:
Orchestrator name: argo_workflows
Job name: job_name (Argo Workflows name)
Job ID: n/a ?
Job results URL: n/a ?
The ID of a specific run execution: job_run_id (Argo Workflows uid )
Job run results URL: n/a ?
Describe alternatives you've considered
not sure
Additional context
Would you be willing to contribute this feature?
Yes!

It seems to be possible to pass Argo Workflows orchestrator information today:
dbt run --select elementary --vars '{orchestrator: \"Argo Workflows\", job_id: \"{{workflow.name}}\", job_run_id: \"{{workflow.uid}}\"}'


Fix PHP 8.1 return type deprecation notices

Still missing in HeaderStream.
Got following message in our unit testing code:
PHP Deprecated:  Return type of ZBateson\MailMimeParser\Stream\HeaderStream::update(SplSubject $subject) should either be compatible with SplObserver::update(SplSubject $subject): void, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /home/runner/work/mobilityshop-api/mobilityshop-api/vendor/zbateson/mail-mime-parser/src/Stream/HeaderStream.php on line 51

Thanks @joelharkes

thanks for the merge! :)


Composer could not find composer.json file when compiling

I've installed box globally via
$ composer global require humbug/box:^3.0@alpha

And when I run
$ box compile

inside a package directory containing a composer.json I receive the following error:
? Adding files

In Factory.php line 287:

  Composer could not find a composer.json file in /tmp/box/Box16095
  To initialize a project, please create a composer.json file as described in the https://getcomposer.org/ "Getting Started" section

The directory where I run compile and in which I have box.json has a composer.json so I don't know what went wrong.

Could you share your config?

Here is my box.json config:
{
    "main": "bin/mybin",
    "directories": ["src"],
    "finder": [
        {
            "name": "*.php",
            "exclude": ["tests"],
            "in": "vendor"
        }
    ],
    "git-version": "release_version",
    "output": "build/mybin.phar",
    "stub": true,
    "check-requirements": false
}

I see, the issue is that Box detects the composer.json so it will try to dump the optimized autoloader, but as the file is not included (and that's a bug as it should) it will fail.
While I'm fixing this, you have 3 options:

Wait for the fix
include the composer.json file manually with the files setting for example
Remove the directories and finder entries. You can also remove main and stub as they should now be inferred by the composer.json values.

Out of curiosity, why disabling the requirements checker?

Actually I can't reproduce it, do you have other intermediate steps?
Also if you can try box compile --debug and give the .box_dump/.box_configuration file that would help

The only intermediate step is to call composer build which in turn is a composer.json script that calls box compile and a few cleanups.
.box_dump file available here: https://pastebin.com/VmXQd5vN (I cleaned up some names from the dump as it may or may not be sensitive)
--debug displays the additional following information:
? Adding files

In Factory.php line 287:

  [InvalidArgumentException]
  Composer could not find a composer.json file in /tmp/box/Box71116
  To initialize a project, please create a composer.json file as described in the https://getcomposer.org/ "Getting Started" section

Exception trace:
 Composer\Factory->createComposer() at /home/ojrask/.config/composer/vendor/composer/composer/src/Composer/Factory.php:576
 Composer\Factory::create() at /home/ojrask/.config/composer/vendor/humbug/box/src/Composer/ComposerOrchestrator.php:38
 KevinGH\Box\Composer\ComposerOrchestrator::dumpAutoload() at /home/ojrask/.config/composer/vendor/humbug/box/src/Box.php:204
 KevinGH\Box\Box->addFiles() at /home/ojrask/.config/composer/vendor/humbug/box/src/Console/Command/Compile.php:377
 KevinGH\Box\Console\Command\Compile->addFiles() at /home/ojrask/.config/composer/vendor/humbug/box/src/Console/Command/Compile.php:218
 KevinGH\Box\Console\Command\Compile->createPhar() at /home/ojrask/.config/composer/vendor/humbug/box/src/Console/Command/Compile.php:175
 KevinGH\Box\Console\Command\Compile->execute() at /home/ojrask/.config/composer/vendor/symfony/console/Command/Command.php:252
 Symfony\Component\Console\Command\Command->run() at /home/ojrask/.config/composer/vendor/symfony/console/Application.php:865
 Symfony\Component\Console\Application->doRunCommand() at /home/ojrask/.config/composer/vendor/symfony/console/Application.php:241
 Symfony\Component\Console\Application->doRun() at /home/ojrask/.config/composer/vendor/symfony/console/Application.php:143
 Symfony\Component\Console\Application->run() at /home/ojrask/.config/composer/vendor/humbug/box/bin/box:23

Ah yes, adding composer.json to files allows compile to succeed now.
I disable requirements checker as per my last issue where some file for it was not found when running compile.

@rask I've tagged a new release fixing all the issues reported for the alpha 2

Alright, thanks!


[Bug] Should change Tuple(tvm.ir.Attrs, List[relax.Type]) into Tuple[tvm.ir.Attrs, List[relax.Type]] in in python/tvm/script/relax/parser.py

When I import tvm and tvm.script with following code
import tvm
from tvm.ir.module import IRModule
from tvm.script import tir as T, relax as R

I got error msg as below.
Traceback (most recent call last):
  File "test4.py", line 4, in <module>
    import tvm
  File "/home/xwv/xwv/relax-relax/python/tvm/__init__.py", line 54, in <module>
    from . import script
  File "/home/xwv/xwv/relax-relax/python/tvm/script/__init__.py", line 20, in <module>
    from . import relax
  File "/home/xwv/xwv/relax-relax/python/tvm/script/relax/__init__.py", line 19, in <module>
    from .function import function
  File "/home/xwv/xwv/relax-relax/python/tvm/script/relax/function.py", line 26, in <module>
    from .parser import from_source
  File "/home/xwv/xwv/relax-relax/python/tvm/script/relax/parser.py", line 109, in <module>
    class RelaxTransformer(Transformer):
  File "/home/xwv/xwv/relax-relax/python/tvm/script/relax/parser.py", line 1133, in RelaxTransformer
    def parse_call_attr(self, expr: ast.Call) -> Tuple(tvm.ir.Attrs, List[relax.Type]):
  File "/usr/lib/python3.6/typing.py", line 1293, in __new__
    raise TypeError("Type Tuple cannot be instantiated; "
TypeError: Type Tuple cannot be instantiated; use tuple() instead

I have to change line 1133 of parser.py into
 def parse_call_attr(self, expr: ast.Call) -> Tuple[tvm.ir.Attrs, List[relax.Type]]:

Thanks!

Hi @lindylin1817, thank you for your interest in Relax and giving it a try!
It seems that you are using Python3.6, while Relax code needs to import some packages which are available starting with Python3.7. Could you try running the code with Python3.7 or Python3.8 and make sure synr==0.6.0 is installed?

@YuchenJin Yes, you're right. Thanks for pointing it out. I used Python3.6. Will it be helpful if such dependance could be added into the installation document? Right now, Relax version is sharing the same one with normal TVM. But I fully understand that Relax hasn't been formally merged back to TVM main.

@lindylin1817 Thanks for your suggestion! Currently the TVM installation doc writes "Python is also required. Avoid using Python 3.9.X+ which is not supported. 3.7.X+ and 3.8.X+ should be well supported however." We will make 3.7.X+ a requirement and update the installation guide once Relax is upstreamed (upstreaming starting next month).
Does using a newer version of Python also solve the other issue https://github.com/tlc-pack/relax/issues/193?

@YuchenJin Yes, it has been solved. But I got another error. Let me close this issue first.


File-based cookie store

Has anyone done a file-based cookie store? Just wondering.

Yes: https://github.com/JSBizon/file-cookie-store


We need to update the GWs finalizers

I actually don't know that this change suffice, but I think it should... Unsure how to best test that before merging.
See here, but we can discuss that tomorrow...

/cc @jsmolar

Other than @eguzki telling me this is nonsense, I'll merge that in my afternoon 🙏


add http client

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

133 of 377 (35.28%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-4.2%) to 90.145%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

join/network/src/httpclient.cpp
52
78
66.67%

join/network/src/httpresponse.cpp
25
53
47.17%

join/network/src/httpmessage.cpp
30
61
49.18%

join/network/src/httprequest.cpp
20
179
11.17%

Totals

Change from base Build<PHONE_NUMBER>:
-4.2%

Covered Lines:
4793

Relevant Lines:
5317

💛  - Coveralls


Use different sample rate

Can I use a different sampling rate like 22 kHz for fine tuning?

Yes - I think you don't need to change anything of the code for a new sampling rate if you only use imagenet_pretrain=True.
But our AudioSet pretrained model is trained with 16kHz audios, so it would be better if you resample your audios to 16kHz. The improvement of AudioSet pretraining is obvious for audio event classification tasks.
-Yuan

The problem is, that 16 kHz can only used for frequencies until 8kHz, where my data contains higher frequencies (>8 kHz

In that case, you can just use your sample rate and set audioset_pretrain=False when you initialize the AST model.
-Yuan

can I use the audioset weights for my case here?


Minor improvements

Add /Cache in .gitignore
Add tests

Thanks for the pull request. Just a few changes I would like you to make:
For consistency, can you docblock those 2 test methods? I've made an effort to document all my methods using docblocks in my code.
Remove the extra new line between the { and } (opening and closing of a method) - small cosmetic change to just retain consistency.
I appreciate you taking the time to integrate PHPUnit. Once you make those changes I will check and if all is good, I will accept the pull request. :+1:


Node Lock

So we are running into an issue where once we create a prefab we are no longer able to change any of the values including moving the game object around. In this case its a component which has a variable which points to a serializable class which has a game object as a variable.
Attached is a super simple project which demos the problem.
NodeLock.zip (removed)
There is a Test scene with a test object. if you grab it and try and move it, It wont. You can also not type in any attributes.
Please advise.

To add, If we break the prefab connection, make changes then apply them back to the prefab... this work and is our current work around... which sucks.

Thanks, I've downloaded the project and I will try to reproduce tonight.

How we looking?

I'm having trouble reproducing on 5.3.4p6. Can you try reproducing on that version? Or maybe I'm reproing wrong. Here's what I did:
I added a new object, added a Test component, added an element to the array. Then I made the object a prefab. I was still able to drag the object around in the scene hierarchy view after doing that.

Those are the same steps we are doing here. So you were able to drag the object around the scene right? not the scene hierarchy?

I didn't check dragging around the scene, let me try that.

I've found the issue, but I believe this is a regression introduced by Unity. I've submitted a bug to them (https://fogbugz.unity3d.com/default.asp?818240_m622mv347ksj8nl6). For the time being, you should be able to work around by adding [NotSerialized] to the UnityEvent type.

I've uploaded a new build to access that should work around the Unity bug.


Update main.py

Ok

?
On Mon, Sep 25, 2023, 8:09 PM ɴᴏᴏʙ - ᴍᴜᴋᴇsʜ @.***>
wrote:

Ok
—
Reply to this email directly, view it on GitHub
https://github.com/Noob-mukesh/MukeshRobot/pull/176#issuecomment-1733849867,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWJMRF5KL6RS3IJCO3LZTULX4GJSXANCNFSM6AAAAAA5F7M7NM
.


Make endpoints / lagoon base configurable

Right now the lagoon base url is hardcoded into the gatherer utils - this needs to be changed to allow us to point the gatherer at different Lagoons.
This should be accomplished by either/both env vars and parameters.

This is actually going to fall away, we're going to lean on insights to make this happen.


Fix-h2 version and remoe CVE ignores

Bump to 2.2.224 and remoe old 2.2.220 CVE overrides from the ignore list.

This was submitted in error.


I'm getting different AAID if using any app cloner apps like DO Multiple Accounts - Infinite Parallel Clone App, Parallel Space - Multiple accounts & Two face ect.

Hey,
I'm getting different AAID in my app if one is opened normally and one opened in the parrel space app (app cloner).

Hey @devVinit, I don't know how app cloner works, do you have any detail?

hey @edodusi, Yes so it basically helps in running two instances of the app on one device, people use it for running multiple accounts.
https://www.youtube.com/watch?v=FHESoFZ9zBg

@devVinit so apparently it's like you have two users on the same device, so having different AAIDs is correct.

Yeah, we should get two AAIDs but we don't. It was said to be unique but got duplicate in both cases while running app

Sorry @devVinit I'm not sure I understand the problem you're facing. This module is a wrapper around the native call to get IDFA and AAID, so the result you get is the same you would get if you directly call the native API.
So there's no issue with the module itself, it's just behaving the way it should.

We were using it as a unique device id, but when uses clone the app we get different AAIDs. But according to going through google android docs I'm able to understand that AAIDs will change in case of that.


Fix for reset/panic overriding the user's stop

Switch to an enum to make the possible states easier to follow and so we can check we're in the default state before committing to a reset or panic.
Closes https://github.com/microbit-foundation/micropython-microbit-v2-simulator/issues/86

URL to embed: https://review-python-simulator.usermbit.org/user-stop-wins/simulator.html


Slow checksum computation

Experimenting with larger (~200MB) file transfers I discovered that the checksum computation is very slow (~90 seconds for a 200MB file on a Ubuntu 20 with a relatively fast SSD). This will cause the inactivity timer to be triggered.
I think the reason is that the checksum function reads the file 4 bytes at a time:
https://github.com/ASU-cubesat/cfdp-rs/blob/main/cfdp-core/src/filestore.rs#L421
I think instead of the 4 bytes vector, it should use a vector of 1 or 2KB.

I think I was more worried about making sure everything fit on the four byte boundary than being smart the first time. I agree it should probably read in more data and just iterate over a larger vector between each read. A few KiB is probably a good start.

I'm doing some benchmarks on a implementation with a BufReader. Seems to give ~1000x speedup according to the rust bencher.


Current state of DynamicMap

Is there an easy way to get the parameters of the showed item in a DynamicMap?
dmap.keys() of dmap.cache shows all the keys available in the cache, not the current selected frame.
When using ipython.widgets.interact, one can do: object.kwargs or something.
This addition would it also make it possible to easilier to save the last state of a widget in a notebook.

I think this is a reasonable suggestion.
The attribute could be called args as that is how arguments are passed into the callback (if not a generator), following the key dimension order. We have some plans to use kwargs for something else in future...

I've made a PR (linked above) to add an args attribute. I'll wait for @philippjfr  to review it but first I better test it works as expected...

It seems to work fine. I noticed one instance where the args didn't match the state of the sliders (a bit disturbing!) but I can't seem to reproduce that issue now...

@vascotenner This should now be addressed by PR #1453 which has been merged.
You'll find an args and kwargs attribute on dmap.callback with the information you need. Closing.


Running in IDE

I wish to incorporate the summarization model as a function in my code, i.e. the function will summarize its input text and return it. However, when I paste the code of run_summarization.py in my my notebook and run it, I get the DuplicateFlagError which says that the flags have been defined twice. Can anyone help me with using the model as a function?

I have resolved this issue. To use this model as a function, follow the following steps, I am considering jupyter notebook usage:

Take out the part of code in run_summarization.py which defines all the flags and paste it in a cell of the notebook.
Define an extra flag 'f', in the same manner as abisee defines the flags. The arguments to the define function should be ('f',' ','kernel'). This is to prevent a possible error with the flags.
Paste the remaining part of run_summarization.py as your summarizer function. Call the main() function of the run_summarization again in your summarizer.
Keep the other .py files like batcher, decode, etc in the location where your notebook is saved, and also where python is installed.
The part where we separate the flag define statements and the remaining code is to ensure that we do not define the flags again and again when we call the function.
It might be necessary to restart your kernel if you face the duplicate flag error.


Zooming the editor breaks webviews

Steps to Reproduce:

Zoom in
Open up a webview

Zooming in the entire editor seems to mess up webviews. Other parts of VSCode scale properly, but a weird padding is added to the webview (or rather, the height and width properties are too small for the now-zoomed pane)
Changing the width and height of the webview (which are both manually styled in the HTML as 100%) to, for example, 120% if you've zoomed once, then it "fixes" it, but this has to be done manually for every webview which is obviously a big pain haha.

Duplicate of #5745


Community packages

Docs Build status updates of commit ed1afca:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azure-go-sdk-conceptual/azure-sdk-go-install.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[Import] crash importing a VTK file created with VTK9

A user has a crash importing a vtk mesh in medInria/MUSICardio. I have the same crash:
DEBUG - ven. déc. 3 16:05:35 2021 - "medDatabaseNonPersistentImporter created with uuid: \"{17c5cf52-184d-4ff9-b847-dc6fa9dbd7f4}\""
WARN  - ven. déc. 3 16:05:35 2021 - Warning: In /home/mathildemerle/Dev/medInria3.2/medInria-public_ExtProjs/src/VTK/IO/Legacy/vtkDataReader.cxx, line 494
vtkUnstructuredGridReader (0x7f0040003090): Reading file version: 5.1 with older reader version 4.2

WARN  - ven. déc. 3 16:05:35 2021 - Warning: In /home/mathildemerle/Dev/medInria3.2/medInria-public_ExtProjs/src/VTK/IO/Legacy/vtkDataReader.cxx, line 494
vtkPolyDataReader (0x7f0040003090): Reading file version: 5.1 with older reader version 4.2

[...]

DEBUG - ven. déc. 3 16:05:35 2021 - "Chosen directory contains  1  files"
DEBUG - ven. déc. 3 16:05:35 2021 - "Can read with:  vtkDataMeshReader"
DEBUG - ven. déc. 3 16:05:35 2021 - "Reading:  /home/mathildemerle/Dev/00.data/[NAME.vtk]"

ERROR - ven. déc. 3 16:05:35 2021 - ERROR: In /home/mathildemerle/Dev/medInria3.2/medInria-public_ExtProjs/src/VTK/IO/Legacy/vtkDataReader.cxx, line 3075
vtkPolyDataReader (0x7f0040003090): Error reading ascii cell data! for file: /home/mathildemerle/Dev/00.data/[NAME.vtk]

ERROR - ven. déc. 3 16:05:35 2021 - ERROR: In /home/mathildemerle/Dev/medInria3.2/medInria-public_ExtProjs/src/VTK/IO/Legacy/vtkDataReader.cxx, line 3075
vtkPolyDataReader (0x7f0040003090): Error reading ascii cell data! for file: /home/mathildemerle/Dev/00.data/[NAME.vtk]

DEBUG - ven. déc. 3 16:05:35 2021 - vtkDataMesh has no such property: "SeriesDescription"
DEBUG - ven. déc. 3 16:05:35 2021 - "vtkDataMeshReader : no metadata found in  \"/home/mathildemerle/Dev/00.data/[NAME.vtk]\""
DEBUG - ven. déc. 3 16:05:35 2021 - "Patient exists in the database, I reuse his Id"
DEBUG - ven. déc. 3 16:05:35 2021 - "Study exists in the database, I reuse its Id"
Erreur de segmentation (core dumped)

The 10th firsts lines are:
# vtk DataFile Version 5.1
vtk output
ASCII
DATASET POLYDATA
POINTS 50000 double
52.402816802 -207.29418758<PHONE_NUMBER>8 54.006912177 -206.67887237<PHONE_NUMBER>2 50.390363852 -205.74193916<PHONE_NUMBER>5
51.286296796 -206.9683942<PHONE_NUMBER> 52.61190033 -206.41700745<PHONE_NUMBER>5 51.49369812 -205.50900269<PHONE_NUMBER>2
53.805900574 -205.60600281<PHONE_NUMBER>8 52.759998322 -205.2440033<PHONE_NUMBER> 53.861639697 -204.34420062<PHONE_NUMBER>2
50.814076513 -204.0761275<PHONE_NUMBER>2 52.112998962 -204.22900391<PHONE_NUMBER>3 50.42950058 -208.41799927<PHONE_NUMBER>4
51.808200836 -208.15800476<PHONE_NUMBER>8 48.825055536 -208.06280443<PHONE_NUMBER>4 53.282901764 -208.19999695<PHONE_NUMBER>3

It's a known error from users, and some of them are using this script to transform the VTK in a VTP readable by medInria/MSC:
import pyvista as pv

def convert_VTKpolydataMesh_to_vtp(VTKlMeshFile, VTPlMeshFile):
    tetra_mesh = pv.read(VTKlMeshFile)
    tri_mesh = tetra_mesh.extract_geometry()
    tri_mesh.save(VTPlMeshFile)

if __name__ == "__main__":
    """
    """
    path = '_output.vtk'
    out_path = '_output_clean.vtp'
    convert_VTKpolydataMesh_to_vtp(path, out_path)

Maybe it could be better if medInria does not crash importing these vtk, and also it could be fine that medInria succeed importing them. I can send privately an example of a crashing VTK.

We have to check which version of vtk they used to create the file but it's look like a vtk 9 version where they change the way to write data can you send the whole file please so i can check if it was written by vtk 9 or not.

It is indeed a problem of vtk 9 version  you should change the name of the issue

Hi Pauline,
Indeed VTK 8 and 9 format are incompatible for the moment.
Can you provide the callstack to produce the segfault please (or the core dump + binaries).

solved in https://github.com/medInria/medInria-public/pull/954


Style.css unavailable

After build there is no style.css file in dist/ClientApp folder, there is a styles.js file instead. Where to get style.css file? or some other process need to be followed.
Please suggest.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8d2ba9b5-bc21-3e30-f97a-63a5d5a9f506
Version Independent ID: dec8cc95-9dcc-ee56-f9d7-808de79f087b
Content: Dynamics 365 Portal hosted custom event portal
Content Source: ce/marketing/developer/portal-hosted.md
Service: dynamics-365-marketing
Technology: marketing
GitHub Login: @Nkrb
Microsoft Alias: nabuthuk

@AshV Thanks for the feedback. Run the command ng build --prod --output-hashing none to generate style.css file.


Create/Delete a list

Would it be possible to have the ability to create (or even delete) a list on the fly? Sometimes I discover a follow that doesn't quite fit the current set of lists I have and would like to put them in a newly created one

Already there!  Click the hamburger menu at the upper left and create away!  To delete, click the trash can when you hover over a list name!

As above, already implemented.


add uloop (ubox) based websocket server

Coverage remained the same at 91.284% when pulling 9906a0eb46d4b44863280ef6db494d8e8fa46a9c on kcsoft:master into f86b1709ca962b9c95b1bf192c4c4b7398e79832 on lipp:master.

THX! Any chance you will get tests added?


msg-sim Linux implementation

Context
Since #54, we have an initial implementation for our network simulation crate for MacOS, but we're still missing one for Linux. The simulation crate allows us to start simulated endpoints in Rust tests that we can then use to emulate a real world environment. The MacOS one uses dummynet with some ifconfig commands and a hacky localhost alias to configure a simulated endpoint. We want to do the same for Linux, but luckily Linux has some better tools for these types of things:

ip command: with the ip command, we can create network namespaces, virtual ethernet devices, dummy interfaces etc. I think dummy interfaces specifically will be most useful in the beginning (it's basically a separate localhost but with normal MTU for more accurate simulation).
tc command: the tc (traffic-control) command can be used with netem (a kernel module) to shape traffic.

Some inspiration:

https://github.com/tylertreat/comcast
https://github.com/celestiaorg/bittwister

Other notes:

These commands all require root access. Maybe there's a way to utilize network namespaces to not need root access? Also the CAP_NET_ADMIN capability allows us to do this stuff without root but I don't think it's useful when running tests. And granting capabilities also require root.
What other things can we do with network namespaces? A completely simulated p2p network where each peer is in its own namespace and can have its own latency, bandwidth and packet loss rates? This will need veth or virtual ethernet devices as well.

I've given a look at this and here are some of my thoughts/notes:
On dummy interfaces
It's very easy to spin up a dummy interface on Linux and play with it, e.g.
# Make sure that the dummy kernel module is loaded,
# which provides support for the creation of dummy network interfaces
sudo modprobe dummy

# `ip link` allows to configure network devices. We want
# to add a new one, called `dummy0` of type `dummy`
sudo ip link add dummy0 type dummy

# We configure it by assigning an IP address to it.
# We also need to specify the network device
sudo ip addr add <IP_ADDRESS>/24 dev dummy0

# Verify the dummy interface is correctly configured using
ip addr show dummy0

# Now you should be able to ping it:
ping <IP_ADDRESS>

# To remove it
sudo ip link delete dummy0

However adding delay or loss using tc + netem doesn't work for some reasons. Internet suggests to work with namespaces, and I listened.
On namespaces and local P2P network
With ip it's easy to spin up linked virtual ethernet devices on a separated namespaces where each end of the link has its own IP address and network emulation rules.
In order to achieve a dummy interface using this technique, we can spin up one veth one a separate namespace, and one veth on the host environment, like so:
# create namespace ns1
sudo ip netns add ns1
# create veth devices linked together
sudo ip link add veth-host type veth peer name veth-ns1
# move veth-ns1 device to ns1 namespace
sudo ip link set veth-ns1 netns ns1

# associate ip addr to veth-host device and spin it up
sudo ip addr add <IP_ADDRESS>/24 dev veth-host
sudo ip link set veth-host up

# same but from ns1 namespace
sudo ip netns exec ns1 ip addr add <IP_ADDRESS>/24 dev veth-ns1
sudo ip netns exec ns1 ip link set veth-ns1 up

# this should work
sudo ip netns exec ns1 ping <IP_ADDRESS>

# this too
ping <IP_ADDRESS>

# add latency etc to veth-ns1 from ns1 namespace
sudo ip netns exec ns1 tc qdisc add dev veth-ns1 root netem delay 3000ms loss 50%

# this should be slow
ping <IP_ADDRESS>

# bonus: attach a toy web server in another terminal; should take a while to respond
sudo ip netns exec ns1 python3 -m http.server --bind <IP_ADDRESS> 8000

In the same fashion, we can spin up a local p2p network where each veth device lives in a separate namespace, then we have to link them, assign IP addresses and adding network emulations as desired
On the actual implementation
I think a good strategy would be:

extending the existing simulation code for Linux support -- I imagine something like dummynet.rs but for Linux only, let's say ip_tc.rs which contains the logic to create a veth between the host and a target which lives on a separate namespace with provided emulation parameters; then use such logic for Simulation::start with Linux target.
since it is possible to create a local p2p network on Linux with the above mentioned techniques, it'd be cool to create and use a P2PSimulator struct made for this purpose. I still have to see what can be reused to avoid major refactorings.

On root access
Here I think there is not much we can do, since modifying the network stack requires root access. I think it's better to ask for them instead of altering the user's system configuration (which would require root privileges anyway) and may feel hacky/sketchy. For example, both tylertreat/comcast and celestiaorg/bittwister require root access to run the binary.

Great writeup, thanks for clarifying all the steps. Also agreed on the strategy! One question: when adding delay / bandwidth to a veth inside of a namespace, is it symmetrical or only one way? If it's one way, it will be different from how it works on MacOS over the loopback interface.
Keep in mind that the API I've settled on for the simulator is by no means final, if you think you can improve on it or bump into any limitations, feel free to propose changes to it.
Let's start with just single endpoint simulation and we can then expand it to P2P.

One question: when adding delay / bandwidth to a veth inside of a namespace, is it symmetrical or only one way? If it's one way, it will be different from how it works on MacOS over the loopback interface.

It is asymmetrical/only one way! For instance, if I apply delay to the device veth-ns1 (on namespace ns1) like in the code snippet above, then it affects all the packets leaving veth-ns1.


[CLI] Files paths are poorly matched

While running cli to add components, many of the file paths are not precedded with the custom aliases (@/~), this causes undesirable issues, this happened with styled-system, I spend a over a half and hour figuring out why I am getting module found, I even installed the styled-system npm package, as I thought the CLI have missed it, than after half and hour banging my head and figuring out why it doesn't working, My eyes fell on the styled-system directory, and then preceding with the custom aliases fixed the problem, there are many other issues in the document as well, I am not here to critisize the component library, I am just reporting what other may face in future, also I love park ui

[tokens] Missing token: colors.red.9 used in theme.semanticTokens.colors.border.error
[tokens] Missing token: colors.red.9 used in theme.semanticTokens.colors.fg.error

Also these errrors in the config

While running cli to add components, many of the file paths are not precedded with the custom aliases (@/~), this causes undesirable issues, this happened with styled-system, I spend a over a half and hour figuring out why I am getting module found, I even installed the styled-system npm package, as I thought the CLI have missed it, than after half and hour banging my head and figuring out why it doesn't working, My eyes fell on the styled-system directory, and then preceding with the custom aliases fixed the problem, there are many other issues in the document as well, I am not here to critisize the component library, I am just reporting what other may face in future, also I love park ui

Yes, have added styled-system to include array in typescript config file

@abdulahd2008
I've updated the CLI so that a tsconfig.json is no longer required to install components. This benefits users with setups like nx workspaces that lack a tsconfig.json file.


Fixing header nav test QA-2146

Description

Fixed header navigation test failure
Added try except in header navigation function

Test:
VPN2, qa-orange
gauge run specs/gdc_data_portal_v2/header/navigation.spec

Test ran successfully, Approving:
Header Navigation
Open the data portal        ✔
Validate header featured tools navigation   ✔ ✔ ✔ ✔


social link not working

these links are not workin
Im a GSSoc cntributor, please assign me this issue, I will resolve it

currently am not looking for this . Thanks !


LNURL-auth with Phoenix no longer works

Description
When using Phoenix to login, the backend responds with 500 and this error is logged in the console:
app  | - error Failed to handle request for /api/lnauth?tag=login&k1=bc39afa971bfbc961cca0d49f0d5256b330ef1b27987d5d9aaeb69e63512f1b1&sig=3045022100fec2a4a4152c99a70785aeb8753ffa93f37e8fd53cac92d643c457e5d7b31ca10220738046ea8aa822b17c1ef41bcdd74be5bc179062de78affeeb078312cae01e8c&key=03cbf859c4fa54b5f5eb1e55cfadc2aaf663b6b8b6583938dce4bbe2d0688d81c2
app  | TypeError: fetch failed
app  |     at Object.fetch (node:internal/deps/undici/undici:11576:11)
app  |     at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
app  |     at async invokeRequest (/app/node_modules/next/dist/server/lib/server-ipc/invoke-request.js:21:12)
app  |     at async requestHandler (/app/node_modules/next/dist/server/lib/start-server.js:336:33)
app  |     at async Server.<anonymous> (/app/node_modules/next/dist/server/lib/start-server.js:152:13) {
app  |   cause: RequestContentLengthMismatchError: Request body length does not match content-length header
app  |       at write (node:internal/deps/undici/undici:10059:41)
app  |       at _resume (node:internal/deps/undici/undici:10037:33)
app  |       at resume (node:internal/deps/undici/undici:9938:7)
app  |       at [dispatch] (node:internal/deps/undici/undici:9286:11)
app  |       at Client.Intercept (node:internal/deps/undici/undici:9017:20)
app  |       at Client.dispatch (node:internal/deps/undici/undici:7772:44)
app  |       at [dispatch] (node:internal/deps/undici/undici:7991:32)
app  |       at Pool.dispatch (node:internal/deps/undici/undici:7772:44)
app  |       at [dispatch] (node:internal/deps/undici/undici:10556:27)
app  |       at Agent.Intercept (node:internal/deps/undici/undici:9017:20) {
app  |     code: 'UND_ERR_REQ_CONTENT_LENGTH_MISMATCH'
app  |   }
app  | }

Steps to Reproduce
Use "Login with Lightning" with Phoenix
Expected behavior
Login works
Additional context
This is probably related to us upgrading NextJS. It might be more adhering to HTTP spec where the Content-length header should not be set for GET requests. We suspect it is set with value 0. We did not confirm this though.
Creating this ticket here to track status (I forgot about this error and just encountered this error again). Maybe we need to report this to ACINQ and/or we try to accommodate clients which don't implement the HTTP spec 100%.

I think the move is to remove this upstream in nginx or something

Confirmed that Content-length header is set to 0 by Phoenix:
87.XXX.XXX.XXX GET https://sn.ekzyis.com/api/lnauth?tag=login&k1=1ce277587259d33312a372013f08c89a432022983574c671f9c2dc8744f36af5&sig=3044022040d1951e0bac9036850507e06656cf2a37678122f2cd1629924ed60955141ec202205c9996bf9d2e8a4fc3faed499f2ce41f1754580bca62168a625d6a97eda69246&key=03cbf859c4fa54b5f5eb1e55cfadc2aaf663b6b8b6583938dce4bbe2d0688d81c2 500 0.027ms sent=174 recv=483 Ktor client 0
This is my nginx log format:
log_format custom '$remote_addr $request_method $scheme://$http_host$request_uri $status ${request_time}ms sent=$bytes_sent recv=$request_length $http_user_agent $http_content_length';
Using proxy_set_header Content-Length "" for requests to /api/lnauth indeed fixes login with lightning.


Injects in instantiated components not working via SceneContext

I've started a new project in Unity 2018.1.0f2 using the latest release of Zenject 5.5.1 but I can't seem to get the scene context (with monoinstaller) to properly inject my game manager. The GameManager instance is created and the the IInitializeable/ITickable methods are running. But any [Inject] attributes in other classes are giving me a "ZenjectException: Unable to resolve type 'GameManager' while building object with type...".
I should note that the scripts in question are added using container.InstantiateComponent(mygameobject);
using Zenject;

public class GameInstaller : MonoInstaller<GameInstaller>
{
    public override void InstallBindings()
    {
        Container.Bind<GameManager>().AsSingle();
        Container.Bind<IInitializable>().To<GameManager>().AsSingle();
        Container.Bind<ITickable>().To<GameManager>().AsSingle();
    }
}

However, if I move those 3 lines of code to an installer (monoinstaller via prefab) in ProjectContext, the Injects work fine. Is there something I am missing here?

Where are you running container.InstantiateComponent ?

So the GameManager gets initialized and spawns cars which then get an AI component attached.
GameObject car =  _container.InstantiatePrefab(carPrefab);
CarBrain carBrain = _container.InstantiateComponent<CarBrain>(car);

This all happens in the Initialize method of the GameManager. Maybe things are going to fast? It does work if I set it up through the Project Context (but I only need it in this one scene of course).

Ok I made it match my structure more and figured out how to make it not work. I have a class called CarBuilder that prepares the prefab, adds the component etc. This class is bound in the project context, and apparently that is when it fails. Here is the project with the error:
ZenjectIssue.zip
If the CarBuilder is bound in the SceneContext (by commenting out line 8 in ProjectInstaller and uncommenting line 32 in TestInstaller) it works fine. But I need this class to be available everywhere.
I'm new to Zenject so it is entirely possible I am using it wrong, in which case, how would I properly set this up?

Oh! Thanks for clearing that up. Now I understand what was happening.
I need the CarBuilder in multiple scenes such as customization screens or the player lobby. But it doesn't really matter all that much if there is a new instance per scene, so that is a possible fix. For now I fixed it by passing the DiContainer from the GameManager to the CarBuilder via a parameter in the method >  CarBuilder.Build(DiContainer container). So that the objects and components are instantiated in the correct container. Is that a good way to handle such a case?

I would probably go the route of having a CarBuilder in each scene by running the same installer that binds it within each scene.  I would also use a factory instead of container directly.  But your way seems fine too


Polkadot.js app does no longer have developer page in permissioned network tutorial

Which page you are referring to?
https://docs.substrate.io/tutorials/v3/permissioned-network
What is the problem?

In the custom type mapping part, it seems that the PeerId configuration step can be skipped without affecting the rest of the tutorials.
Polkadot.js app is no longer has developer page in the latest version.

What could be done?
Delete that part or give a further explanation (why we don't need that now).

Good spot! Sorry about the trouble, yes it's awesome: you don't need custom type definitions manually now because of https://github.com/paritytech/substrate/pull/8615 :tada: - I have updated the flow to remove this. New users won't have to worry about this, and those with older versions can now figure out how to. Fixed this in #591


CLI model info dump

🎉 New feature
Related to https://github.com/ignitionrobotics/ign-gazebo/issues/313
Summary
Adding a model command that is able to print:

Whole model (model pose, model links, model joints, model bounding box)
Just pose.
All links.
All joints.
A given link.
A given joint.

For this, the command can be used the following way:
ign model -m [model_name] to get the full information of the model.
ign model -m [model_name] --pose  to get the pose information.
ign model -m [model_name] --links to get the links information.
ign model -m [model_name] --joints to get the joints information.
ign model -m [model_name] --link [link_name] to get a certain link information.
ign model -m [model_name] --joint [joint_name] to get a certain joint information.
Pendings in this PR:

[ ] Complete tests.

Description
This features implements a new command allowing the user to request information about a model in the running world. To get this information, a copy of the ECM is created making use of the serialized state message, which is acquired from the /state service, as shown in #859. From there, each desired component is taken from the ECM snapshot and serialized.
This command was going to be initially implemented along the gazebo command, but then it was separated into a new folder following the log structure, in order to separate the responsibilities of each command and also avoid any unwanted conflict between the subcommands.
Using this command you will be able to the following information:

Model name.
Model pose.
Model links.

Link name.
Link parent.
Link mass.
Link inertial matrix.
Link pose.

Model joints.

Joint name.
Joint parent.
Joint parent link.
Joint child link.

Model boundary box.

Bounding box name.
Bounding box parent.
Bounding box size.
Bounding box center.
Bounding box minimum corner.
Bounding box maximum corner.

Test it

Run a gazebo simulation
Try some of the aforementioned commands.

Checklist

[x] Signed all commits for DCO
[x] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[x] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

I will review it while the tests are finished.

Here is a little demo running the diff_drive world. You can see here the usage of the model command, requesting the list of available models in the world, the whole model description, and even the link list and then a link in particular. You can request all the joints and a particular joint the same way.
https://user-images.githubusercontent.com/38142548/124010446-336cac00-d9b5-11eb-9a6f-a87da75f3201.mp4
I am of course open to any suggestion about the way the information is displayed here.

I haven't found a world containing such component

We don't have any systems or examples using that component. Some system needs to create the AxisAlignedBox component for the entity, like SubT's VisibilityPlugin does.
My suggestion is to leave this functionality out for now. In the future we could add a simple system that enables bounding box calculation on demand (because it's too expensive to do it all the time).

CI is failing when running test because of :
Library error for [/usr/local/lib/libignition-gazebo3-model.so.3.8.0]: can't load /usr/local/lib/libignition-gazebo3-model.so.3.8.0

I haven't faced this error when running it locally though.

Probably adding a tutorial using the ign model cli at ignition-gazebo/tutorials would be needed, right @chapulina ?

Probably adding a tutorial using the ign model cli at ignition-gazebo/tutorials would be needed, right @chapulina ?

+1, that would help giving the tool some visibility, otherwise it's difficult for users to discover it. It doesn't need to be too detailed, just straight to the point showing that the tool exists and some examples. Thanks!

CI is failing when running test because of :
Library error for [/usr/local/lib/libignition-gazebo3-model.so.3.8.0]: can't load /usr/local/lib/libignition-gazebo3-model.so.3.8.0

154b625 fixes this. IGN_CONFIG_PATH wasn't correctly set when running the tests.

@chapulina Ready for the review!

I have various suggestions on #928


Text tool: different behavior when entering text without first clicking cursor on text field

Steps to Reproduce
1st case

Open Ketcher.
Select Text tool.
Click on canvas.
Change Text font size in drop-down list from 13px to 144px.
Click to Text field and enter any symbols.

2nd case

Open Ketcher.
Select Text tool.
Click on canvas.
Click to 'bold'/ 'italic'/ 'subscript'/ 'superscript' button.
Click to text field and enter any symbols.

Expected behavior:
Text tool behaves the same in both cases:
Resets all settings to default
OR
Formatting is applied even without first clicking on text field.
Actual behavior:
1st case: Text displays with a size of 13 pixels. Text size 144px is displayed in drop-down list from.
2st case: Text displayed by default, i.e. without 'bold'/ 'italic'/ 'subscript'/ 'superscript' formatting. 'bold'/ 'italic'/ 'subscript'/ 'superscript' button not highlighted.
See atach. screen video
https://user-images.githubusercontent.com/90637536/133398332-e68db6b1-839c-4ae0-bbda-b6f49c6cb2b6.mp4
Desktop:

OS: Windows
Browser: Chrome Version 93.0.4577.63, IE Version 20H2 (build 19042.1165), Firefox Version 92.0, Microsoft Edge Version 93.0.961.47

Release/2.4 Closed as fixed.


fix: accept empty String as issue message

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Question: Any reliable method to return sent message id??

Does anyone know how can I return the "last send message id" from sendMessageToID.
I tried to "mimic" the check() function from sendMessage with no luck :/
Im sure I can do some manual magic, but wondering if someone knows why chat.sendMessage() returns undefined.
Any hint appreciated.

I have half work done.
Created a function to check last messages copying from the old sendMessage function.
Now dealing with the return to python ...
I'll post again if I have lucky.

No luck, need help with the return at JsFunction, which actually just returns True False. I am a bit puzzled.
Seems more easy to just create a set of separate functions to handle that :-/ because I won't broke rest of functions.
Any help appreciated.


Broken link to "reset the existing credentials"

The link "reset the existing credentials" is broken. I believe it is just because of the leading "/", the target document is in the same directory.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 25a0d502-ce71-76d4-bf8a-060e63913b52
Version Independent ID: 69d7fa26-1a37-e477-0a2c-db58ce19a84f
Content: Use a service principal with Azure Kubernetes Services (AKS) - Azure Kubernetes Service
Content Source: articles/aks/kubernetes-service-principal.md
Service: azure-kubernetes-service
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@ngrigoriev
Thanks for your feedback! We will investigate and update as appropriate.


import/order opposite errors on local machine and gitlab

Hi<EMAIL_ADDRESS>on my local machine with Ubuntu shows list of errors. After I have fixed it, making push on gitlab repository. There is a auto checking shows opposite errors
For example
locally
modules/query.js:2:1
:heavy_multiplication_x:  2:1   redux-actions import should occur before import of constants/types                                            import/order
on gitlab
modules/query.js:2:1
:heavy_multiplication_x:   2:1   constants/types import should occur before import of redux-actions          import/order
Do you have any suggestions?

It is because of different versions of node.js


Fix #382 update jquery to version 3.3.1

lbry io always up to date

#395

Jquery 3.1.1 seems to not break any functionality also which is good.

@maximest-pierre thanks for acceepted this PR and all people know if the latest version of all software or coding is more better than older version . @tzarebczan thanks sir for Merged this commit and close issues


Instagram reel download error, unable to extract user name

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
INDIA
Provide a description that is worded well enough to be understood
Downloading from Instagram reel fails, using seal Android app to download the video
App version: 1.12.1 (11212)
Device information: Android 12 (API 31)
Supported ABIs: [arm64-v8a, armeabi-v7a, armeabi]
Yt-dlp version: 2<IP_ADDRESS>710
URL: https://www.instagram.com/reel/C5bAkBoOn3S/?igsh=M2tyb2ppMGtoMjk=
WARNING: [Instagram] unable to extract username; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
ERROR: [Instagram] C5bAkBoOn3S: Unable to extract video url; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
[Instagram] Extracting URL: https://www.instagram.com/reel/C5bAkBoOn3S/?igsh=M2tyb2ppMGtoMjk=
[Instagram] C5bAkBoOn3S: Setting up session
[Instagram] C5bAkBoOn3S: Downloading JSON metadata
[generic] Extracting URL: https://github.com/yt-dlp/yt-dlp/issues?q=
[generic] issues?q=: Downloading webpage
[generic] issues?q=: Extracting information
[generic] Extracting URL: https://github.com/yt-dlp/yt-dlp/issues?q=
[generic] issues?q=: Downloading webpage
[generic] issues?q=: Extracting information
WARNING: [Instagram] unable to extract username; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
ERROR: [Instagram] C5bAkBoOn3S: Unable to extract video url; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
WARNING: [generic] Falling back on generic information extractor
ERROR: Unsupported URL: https://github.com/yt-dlp/yt-dlp/issues?q=
WARNING: [generic] Falling back on generic information extractor
ERROR: Unsupported URL: https://github.com/yt-dlp/yt-dlp/issues?q=
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output

[CommonMistakes] Extracting URL: yt-dlp
[Instagram] Extracting URL: https://www.instagram.com/reel/C5bAkBoOn3S/?igsh=M2tyb2ppMGtoMjk=
[Instagram] C5bAkBoOn3S: Setting up session
[Instagram] C5bAkBoOn3S: Downloading JSON metadata

[debug] Command-line config: ['-P', '/storage/emulated/0/Download/Seal', '--newline', '--config-locations', '/data/user/0/com.junkfood.seal/cache/config.txt', '--no-cache-dir', '--ffmpeg-location', '/data/app/~~7m97sUxttba-OtC-BYfoqg==/com.junkfood.seal-1wiJQ5U3lwJwRlB8J-2eUw==/lib/arm64/libffmpeg.so', 'https://www.instagram.com/reel/C5bAkBoOn3S/?igsh=M2tyb2ppMGtoMjk=']
[debug] | Config "/data/user/0/com.junkfood.seal/cache/config.txt": ['yt-dlp', '-vU']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8 (No ANSI), error utf-8 (No ANSI), screen utf-8 (No ANSI)
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp-nightly-builds [ff38a011d] (zip)
[debug] Python 3.8.0 (CPython aarch64 64bit) - Linux-4.14.190-23974023-abM515FXXS5DXA1-aarch64-with-libc (OpenSSL 1.1.1t  7 Feb 2023, libc)
[debug] exe versions: ffmpeg 5.1 (setts), ffprobe 5.1
[debug] Optional libraries: Cryptodome-3.17, mutagen-1.46.0, sqlite3-3.30.1
[debug] Proxy map: {}
[debug] Request Handlers: urllib
[debug] Loaded 1810 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp-nightly-builds/releases/latest
ERROR: [CommonMistakes] You've asked yt-dlp to download the URL "yt-dlp". That doesn't make any sense. Simply remove the parameter in your command or configuration.
  File "/data/user/0/com.junkfood.seal/no_backup/youtubedl-android/yt-dlp/yt-dlp/yt_dlp/extractor/common.py", line 734, in extract
    ie_result = self._real_extract(url)
  File "/data/user/0/com.junkfood.seal/no_backup/youtubedl-android/yt-dlp/yt-dlp/yt_dlp/extractor/commonmistakes.py", line 25, in _real_extract
    raise ExtractorError(msg, expected=True)

WARNING: [Instagram] unable to extract username; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
ERROR: [Instagram] C5bAkBoOn3S: Unable to extract video url; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
  File "/data/user/0/com.junkfood.seal/no_backup/youtubedl-android/yt-dlp/yt-dlp/yt_dlp/extractor/common.py", line 734, in extract
    ie_result = self._real_extract(url)
  File "/data/user/0/com.junkfood.seal/no_backup/youtubedl-android/yt-dlp/yt-dlp/yt_dlp/extractor/instagram.py", line 487, in _real_extract
    video_url = self._og_search_video_url(webpage, secure=False)
  File "/data/user/0/com.junkfood.seal/no_backup/youtubedl-android/yt-dlp/yt-dlp/yt_dlp/extractor/common.py", line 1467, in _og_search_video_url
    return self._html_search_regex(regexes, html, name, **kargs)
  File "/data/user/0/com.junkfood.seal/no_backup/youtubedl-android/yt-dlp/yt-dlp/yt_dlp/extractor/common.py", line 1357, in _html_search_regex
    res = self._search_regex(pattern, string, name, default, fatal, flags, group)
  File "/data/user/0/com.junkfood.seal/no_backup/youtubedl-android/yt-dlp/yt-dlp/yt_dlp/extractor/common.py", line 1321, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)

Duplicate of #5967


release: v0.7.0

About this change—what it does
Release v0.7.0

All green https://github.com/aiven/aiven-operator/actions/runs/3966900107