Tailsitter recovery

This is the tailsitter recovery strategy integrated into the attitude controller.
Replaces #3244

@kd0aij This is the PR for the optimal recovery. If you hold the vehicle upright you should see normal attitude control in all three axes. If you start tilting the airframe you should see an increasing amount of roll actuation (differential elevons) and if the motors should start pushing the vehicle upright.

@tumbili This doesn't work for @kd0aij. Could you fit in a bench test and fix it up?

elevons and motors active during bench test now. Don't see much diff elevon though. Maybe my yawrate P gain is too low? Should be able to do a test hover after sunup and before snow :)

@LorenzMeier This is failing because of overflowed flash for px4fmu-v4. I thought the Pixracer hardware had more flash space than the Pixhawk?

Rebased, which will fix it.

Now that this passes, can we merge this? Is this test-flown and good to go?

@kd0aij Can you give this a go?

Applied via cherry-pick.


Added a Digital Clock

Added a digital clock  that shows time in Hours : Minutes : Secs  AM/PM format.
Along with the current date.

__


Update volunteers names on the about us page

Is your feature request related to a problem? Please describe.

We need to add a new name to the volunteer's section

Describe the solution you'd like

Add Sanjana Senevirathne

I would like to work on this


Home/end/pgup/pgdn navigation keys

(PgUp and PgDn usually move the viewport by its current width)
Ctrl+PgUp/Dn for year might be useful too.

Should home go to the first candle or the first full screen?
Maybe first candle is best if we want to study how a symbol has behaved from the beginning without spoilers.

Wonder how we should handle when focus is in for example bookmarks datagrid. Leaving it to navigate selected item for now.

Leaving page up & down as they are somewhat tricky if we want charts to be focusable. Example of this would be to click weekly chart and have page up move one screen of weeks.

Maybe first candle is best if we want to study how a symbol has behaved from the beginning without spoilers.

Typically it is the first full page, but first candle might be more useful. There could also be a Ctrl+Home variant for that.


arconnect beta v1.0.8 Hardware wallets don't support this API method currently.

Keystone wallet, didn't support signMessage method.

The keystone wallet integration only supports signing transactions for now.


Delete request pop-up does not close when requestee pays the money request

Details

Delete request pop-up does not close when requestee pays the money request
Fixed Issues

$ https://github.com/Expensify/App/issues/29050
PROPOSAL: https://github.com/Expensify/App/issues/29050#issuecomment-1751697684
Tests

Go to staging.new.expensify.com.
[User A] Send a money request to User B.
[User A] Open the IOU details > click three-dot menu > Delete request > Leave the pop-up open.
[User B] Pay the request.

[x] Verify that no errors appear in the JS console

Offline tests

Go to staging.new.expensify.com.
[User A] Send a money request to User B.
[User A] Open the IOU details > click three-dot menu > Delete request > Leave the pop-up open.
[User B] Pay the request.

QA Steps

Go to staging.new.expensify.com.
[User A] Send a money request to User B.
[User A] Open the IOU details > click three-dot menu > Delete request > Leave the pop-up open.
[User B] Pay the request.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

https://github.com/Expensify/App/assets/68128028/fe877443-2a85-456d-9a4e-d31d9cd7cb31

Mobile Web - Chrome
https://github.com/Expensify/App/assets/68128028/a7a9f35a-b24d-4571-9671-60e0ff48a5b0

Mobile Web - Safari
https://github.com/Expensify/App/assets/68128028/703a3d47-d4f4-4d90-9ad6-23511775655a

Desktop

https://github.com/Expensify/App/assets/68128028/bc397ce1-80be-4f48-8b4d-227d194d2a5c

iOS

https://github.com/Expensify/App/assets/68128028/ea55f7bc-ad8c-4e2e-b332-beb8655691ac

Android
https://github.com/Expensify/App/assets/68128028/89148222-34c0-468b-8e51-53c5e7c330ae

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/MonilBhavsar in version: 1.3.81-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/MonilBhavsar in version: 1.3.83-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Add alias for dotnet workload install command

Add an alias for dotnet workload list command
Fixes #3481
Please review
Thank you in advance

Hi @augustoproiete any plans to review it?

@Marusyk your changes have been merged, thanks for your contribution 👍

How can this be used? I couldn't find any examples or documentation.

I was able to use it like this:
public static class Program
{
    public static int Main(string[] args)
    {
        return new CakeHost()
            .UseSetup<Setup>()
            .InstallTool(new Uri("nuget:?package=MSBuild.SonarQube.Runner.Tool&version=4.*"))

    }
}
public class Setup : IFrostingSetup
{
    void IFrostingSetup.Setup(ICakeContext context, ISetupContext info)
    {
        context.DotNetWorkloadInstall("aspire");
    }
}

I was expecting it to be something like InstallTool.


There was an error running gulp, make sure gulp is running correctly in your project

Hi!
I' ve got this error when starting gulp in sublime.
This is my log file:
TypeError: path must be a string
at TypeError (native)
at Object.fs.openSync (fs.js:584:18)
at Object.fs.readFileSync (fs.js:431:33)
at requireGulp (C:\Users\Family\AppData\Roaming\Sublime Text 3\Packages\Gulp\write_tasks_to_cache.js:24:22)
at Object. (C:\Users\Family\AppData\Roaming\Sublime Text 3\Packages\Gulp\write_tasks_to_cache.js:83:12)
at Module._compile (module.js:413:34)
at Object.Module._extensions..js (module.js:422:10)
at Module.load (module.js:357:32)
at Function.Module._load (module.js:314:12)
at Function.Module.runMain (module.js:447:10)
07-14-2016 00:35:
fs.js:584
return binding.open(pathModule._makeLong(path), stringToFlags(flags), mode);                 ^
TypeError: path must be a string
at TypeError (native)
at Object.fs.openSync (fs.js:584:18)
at Object.fs.readFileSync (fs.js:431:33)
at requireGulp (C:\Users\Family\AppData\Roaming\Sublime Text 3\Packages\Gulp\write_tasks_to_cache.js:24:22)
at Object. (C:\Users\Family\AppData\Roaming\Sublime Text 3\Packages\Gulp\write_tasks_to_cache.js:83:12)
at Module._compile (module.js:413:34)
at Object.Module._extensions..js (module.js:422:10)
at Module.load (module.js:357:32)
at Function.Module._load (module.js:314:12)
at Function.Module.runMain (module.js:447:10)

Hello!
That's weird, looks like the script that handles writing the cache could not find the gulpfile.js
Just to see if I can reproduce this, how is your project structured and which folder is included in sublime? The gulpfile.js is at the root of the project and at the root on Sublime (in the sidebar) ?
Also, do you happen to see a .sublime-gulp-tmp.js file in your project ?
Thanks!

Hello!
Actually I tried very simple different projects just to test gulp in sublime. The gulpfile.js is in the root of the project and in the sidebar of sublime. Also gulp is installed global and locally.
Thanks a lot!

ok, let's try this, can you run node in the project root and input the following ?
$ node
var cwd = process.cwd()
var cachePath    = path.join(cwd, ".sublime-gulp.cache");
var tmpfilePath  = path.join(cwd, ".sublime-gulp-tmp.js");
var gulpfilePath = (function() {
    var allowedExtensions = [".babel.js", ".js"];
    for(var i = 0; i < allowedExtensions.length; i++) {
        var filepath = path.join(cwd, "gulpfile" + allowedExtensions[i]);
        if (fs.existsSync(filepath)) {
            return filepath;
        }
    }
})();
console.log("The gulpfilePath is", gulpfilePath)

var fileSrc = fs.readFileSync(gulpfilePath);

console.log("The fileSrc is (buffer)", fileSrc)

That's part of the source that creates the gulp cache for the package, and that's failing in your machine (but normally works)

Ok. Done. Here it is the result printscreen:

Oh I was expecting an error.
That piece of code you ran is the exact same that the one is breaking on the package. The error TypeError: path must be a string comes from fs.readFileSync at requireGulp which does:
var requireGulp = function(gulpfilePath) {
    var fileSrc = fs.readFileSync(gulpfilePath);
    // ...

The only thing I can think of right now is that there might be a problem with permissions?, it looks like:
*. When sublime runs that code, process.cwd() is returning undefined (or something invalid)
*. Or, fs.existsSync(filepath) can't find the file across partitions
Do you know if there is anything strange with the read access?
(thanks for taking the time with the follow up)

Yeah, gotcha! The problem was in my cmd reg settings.
I had HKEY_LOCAL_MACHINE\Software\Microsoft\Command Processor\AutoRun configured with cd /d c:\ value. It means that my default cmd path was changed to c:\. I cleared the value and now all is running correctly.
Thank you very much for the help.

Wohooo!
You're welcome, thanks for letting me know!


gd4.3.beta1 use own world 3d viewport can not get world 3d(return null)

Tested versions
gd4.3.beta1
System information
win10 gd4.3.beta1
Issue description
Window w = Window.new()
w.set_use_own_world_3d(true)
get_tree().get_root().add_child(w)
print(w.get_world_3d())#null

func _physics_process(delta):
     print(w.get_world_3d())#null

Steps to reproduce
N/A
Minimal reproduction project (MRP)
N/A

Is the first part in _ready? Where is that called?

Window/Viewport own_world_3d documentation.

If true, the viewport will use a unique copy of the World3D defined in world_3d.

You never set the world_3d resource so it created just an empty Ref copy.

It will be available with find_world_3d

@smix8 In godot 4.2, I test if set own_world_3d ,then get world_3d will get a different instance(wihtout set world 3d to a special instance)
maybe change in godot4.3


They're completely useless because the lights will just turn off for a long time, which makes it look like you're still not home. You're better off setting a fixed schedule or writing an API client to toggle lights based on your own random logic.


Unspecified error instead of actual error code when broker is killed due to attempt at disconnect

Hello,
I am writing an application code that wraps around an aiomqtt client, and I would like to be able to handle the case where the broker goes offline as a specific exception case.
A minimal dupe of my code with a local instance of a mosquitto broker is as follows:
import aiomqtt
import anyio

async def sub():
    async with aiomqtt.Client(hostname="localhost") as client:
        try:
            await client.subscribe("topic")
            async for message in client.messages:
                print(message)
        except aiomqtt.MqttError as e:
            # this gives disconnect reason if broker is offline
            print("sub: ", e)
            raise

async def main():
    try:
        await sub()
    except aiomqtt.MqttError as e:
        # this is always Unspecfied error if broker is offline
        print("main: ", e)

anyio.run(main)

When I kill my local broker, the error is raised with reason "Disconnected during message iteration" is raised within the test function, but from the main function, I haven't found a way to retrieve that error message. This seems to be because while exiting the context manager, this line attempts to disconnects and fails, and as such I can't retrieve the test exception message except in the extended traceback.
Is there any way to retrieve the inner error code and message in the outer function?

Hi there, thanks for this detailed issue! 😊
I agree with you that the line you linked shouldn't fail, and we should instead get back the "Disconnected during message iteration" both in the inner and outer function. We should probably wrap the self._client.disconnect() in a try/except, what do you think? Would you like to make a PR?

Sure, happy to give it a try!

Spoke too soon. Looking at it again, I'm on version 2.3.0 but was looking at line 755 of main..
So the unspecified error is actually from reraising the exception from the self._disconnected future and not the attempt to disconnect.
I can't figure out why the error is being re-raised as unspecified - maybe multiple exceptions are being set on the future and the final one is unspecified? Extending the example a bit brutally like this:
import aiomqtt
import anyio

async def sub():
    async with aiomqtt.Client(hostname="localhost") as client:
        try:
            await client.subscribe("topic")
            async for message in client.messages:
                print(message)
        except aiomqtt.MqttError as e:
            # this gives disconnect reason if broker is offline
            print("sub: ", e)
        finally:
            try:
                await client.publish("test", "test")
            except aiomqtt.MqttError as e:
                # this gives error code 4 if broker is offline
                print("pub: ", e)

async def main():
    try:
        await sub()
    except aiomqtt.MqttError as e:
        # this is always Unspecfied error if broker is offline
        print("main: ", e)

anyio.run(main)

I'm not sure which error I would expect to be re-raised.

Great investigation! 👍😊

[...] maybe multiple exceptions are being set on the future and the final one is unspecified?

We'd get a different exception (InvalidStateError) if that would be what's happening. But I think we're close!
I believe that the exception is set in the _on_disconnect callback. We're actually specifying a description for the exception here, namely "Unexpected disconnection", but it seems like this description is ignored in our exception class. Does that make sense?
There's definitely potential to improve here. Currently, the Client.__aexit__ method ignores any exceptions that were raised inside the context manager, and reraises the one that was set on the future by _on_disconnect instead. It might make more sense to bring the details of the disconnection -- even though it's only [code:128] Unspecified error -- into the exceptions that are raised by MessageIterator.__anext__ and Client.publish() and reraise those inside Client.__aexit__ (we have access to those as they get passed to Client.__aexit__). What do you think?
I feel like my text might a bit confusing, please let me know in case something's not clear!

I think from a user point or view, reworking the exception raising so that the "Unexpected Disconnection" message from the _on_disconnect is visible from the error re-raised within __aexit__ instead of "Unspecified error" is clear enough, since that's the root cause of the publish and message iterator errors - not sure if that's what you mean?

I think the stack trace should contain both:

the exception from MessageIterator.__anext__ (or Client.publish)
and the exception from _on_disconnect

This way not only know that/how we lost the connection, but also where our code failed due to this (e.g. MessageIterator.__anext__).
I think the current state of Client.__aexit__ suppressing the exception from MessageIterator.__anext__ (or Client.publish()) is not good practice. I'd raise the exception inside MessageIterator.__anext__ as raise from the exception from _on_disconnect. Then Client.__aexit__ should no longer suppress this exception by simply not handling it inside aexit, but instead should let it bubble up. Does that make sense? 🙂

Hm my current understanding is

disconnection happens --> _on_disconnect is triggered and exception is set on self._disconnected, however the message is hidden by the exception class
some other function such as message iteration or publish throws an error, we exit context manager
in __aexit__, we check self._disconnected and find the exception, without finding the other errors thrown within context manager

So three things to fix:

exception class adapted to let the disconnect message go through
in each function, before raising error, we check if self._disconnected has exception and raise it from the disconnect exception if so
in aexit we first raise the parameter exc if it is not None


Stardust/LED-star combo oddities.

I recently got a couple DIYino Stardust boards, and have been putting together a couple sparring sabers using a tri-cree RGB LED star as the light source and a main/aux switch combo on D12/D11. I've loaded 3 of the stock soundfonts on the flash storage.
I've been working with FX-SaberOS 1.1 along with the most recent repository version. The 1.1 release when loaded doesn't seem to actually run - there's a lot of noise/static - the "DIYino Lightsaber" boot-up wav half-plays then goes back to noise/static.
The most recent repository version when set to two-button compile, when the board boots it switches automatically to config mode, then sets to sound fonts - only one button works - and it won't actually switch back to action mode out of config mode.
Enabling Single Button mode allows FXOS to boot normally (and play the "DIYino Lightsaber" boot-up wav), and start out in action mode - but when switching to config mode it only allows adjusting the soundfont - I'm not sure if it has to do with the fact that it's not recognizing double clicks - but the end result is that the functionality of the whole thing is rather limited.
Does anyone have any experience with debugging this and where I might want to look to find the culprit?

I'm having a very similar issue with the Stardust -- same behavior with the initial boot sequence with default test sound, and similar behavior with two-buttons.  I also have an issue where the sound starts playing whenever I apply power, and when I do get the blade to light, the sound goes off.

Hi guys, are you sure you tell the code to use a Stardust instead of a Prime? You can do it in the Config_HW.h file
//#define DIYINO_PRIME
#define DIYINO_STARDUST

The topic is, on the Prime the D11 is a PWM pin and the D4 is the aux pin signal. If you wire up D11 as a button but use the Prime branch, it will force D11 to 0V (switching off LEDs) and this in turn will be interpreted as a long press on the aux pin, getting the saber to Config Mode right away.
To be honest all my 2-button sabers have a Prime, so I cannot try 2-buttons with a Stardust, but the only difference is the D11 pin (LED driver for Prime and aux pin for Stardust).
One-button mode should work fine, the problem you see there comes from not being able to make a proper double click. It needs some excercise. Actually one of the thing I will work on next year will be to eliminate the double click as much as possible.

Hey Andras.
I definitely have it on the Stardust branch - so I'm trying to figure out why it's doing the behavior it's stuck on.
Do you know why it might be acting strangely on Version 1.1, but doing okay on the current version?

Also is there a parameter for adjusting the period required for a double click to register?

I confirm that I also have it set as stardust.  I’ll start again in the morning and review the code to see if a prime vs stardust check is not working as expected.

I can confirm that one button mode works fine - the double-clicks DO have to be really really fast, but it seems part of the problem was that sleep mode's right after soundfonts, so if you're too slow it puts the board to sleep right after.

The parameters governing the button sensitivity required by the OneButton library are located in the Config_SW.h file:
/*

BUTTONS PARAMETERS
/
#define CLICK        200 //5    // ms you need to press a button to be a click
#define PRESS_ACTION    600 //200  // ms you need to press a button to be a long press, in action mode
#define PRESS_CONFIG    600 //400  // ms you need to press a button to be a long press, in config mode
//
Try playing around with the CLICK parameter to adjust the sensitivity of the double clicks.

Also there is an undocumented feature I dubbed hit-and-run or escape sequence, a hit (clash) detected in Config Mode will have you exit config mode (well, mostly, in volume mode it will set the volume to max).
As to why the 2-buttons mode will start in Config: I'm pretty sure it's because somewhere in the code the D11 is forced to 0V. I just need to find out where. Reason is that D11 is a PWM pin and on the Prime it controls a MOSFET gate.
I also do not really know why the 1.1 behaves differently to the master branch. Of course I alwways improve the code but I do not really remember changing anything other than support for the Pixies and implementing the sleep mode timer.
Can say more if I will have the time to look at the code myself.

I do not feel it's a wiring issue. I have both prime and stardust, and i could never get the two button to work on the stardust. I just put one button on my saber and called it good. I never mentioned it because I just settled for one button. I checked the manual a million times trying to wire 2 button on. It was definitely wired correctly. Anyways thought I'd share that similarity with you.

Bug found, fix is in work.
Quick fix for those who cannot wait ;o)
comment out all lines which start with:
digitalWrite(11

There are 2 such lines in Light.cpp

Thank you for finding it so quickly!  I haven’t even had enough coffee to open the code up yet.

I made a new release, check out the master branch, it works fine on my side.

I guess it works now as I had feedback to deny it :)

It works great, sorry for the lack of reply.


Charts doesn't show when using it inside angular material cards (mat-card)

I just added the angular2-chartjs into my project, I copy pasted the sample, and wasn't showing anything (container and canvas had width and height 0)
Then I found out that it's because I added the chart inside an angular material card :(

It's working absolutely fine for me!
Care to share some code?

Try to remove any divs encapsulating your chart or your card. It worked for me.

I have this same issue. I have a radar chart that will not show up. I have tried playing with different CSS settings and with and without divs, It does not make a difference


[FLINK-26167][table-planner] Explicitly set the partitioner for the sql operators whose shuffle and sort are removed

What is the purpose of the change
Explicitly set the partitioner for the sql operators whose shuffle and sort are removed
Brief change log

Introduce InputSortedExecNode
*Update ForwardHashExchangeProcessor to explicitly set Exchange for the nodes whose shuffle and sort are removed *

Verifying this change
This change added tests and can be verified as follows:

Extended ForwardHashExchangeTest the verify the plan about the changes

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure

@flinkbot run azure

LGTM

@flinkbot run azure


Controller not detected (TTRGB Riing Plus)

Latest release version of this software (<IP_ADDRESS>) does not detect my Riing plus controller. Everything is working normally in TTRGB Plus.
With TTRGB plus shut down or uninstalled, the controller is still not being detected.
Report from TTController
-------------------------------
»
»        OS: Windows 10 Enterprise 2004 [19041.1.x86fre.vb_release.191206-1406]
»        Build: 0.10.0-alpha+Branch.master.Sha.a826afd7df70a522c78f4785f357baa124b6a937
»
-------------------------------
Controllers
-------------------------------
»
»
-------------------------------
Sensors
-------------------------------
»
»  ITE IT8686E:
»        Temperature:
»                System #1 (/lpc/it8686e/temperature/0): 38.0 °C
»                Chipset (/lpc/it8686e/temperature/1): 41.0 °C
»                CPU (/lpc/it8686e/temperature/2): 44.0 °C
»                PCIe x16 (/lpc/it8686e/temperature/3): 33.0 °C
»                VRM (/lpc/it8686e/temperature/4): 41.0 °C
»
»  ITE IT8792E/IT8795E:
»        Temperature:
»                PCIe x8 (/lpc/it879xe/temperature/0): 34.0 °C
»                System #2 (/lpc/it879xe/temperature/2): 33.0 °C
»
»  AMD Ryzen 7 2700X:
»        Temperature:
»                Core (Tctl) (/amdcpu/0/temperature/0): 54.8 °C
»                Core (Tdie) (/amdcpu/0/temperature/1): 44.8 °C
»
»  NVIDIA GeForce GTX 1080:
»        Temperature:
»                GPU Core (/gpu/0/temperature/0): 44.0 °C
»
»  Samsung SSD 970 EVO 500GB:
»        Temperature:
»                Temperature (/nvme/1/temperature/0): 38.0 °C
»                Temperature 1 (/nvme/1/temperature/6): 38.0 °C
»                Temperature 2 (/nvme/1/temperature/7): 47.0 °C
»
»  TOSHIBA HDWD120:
»        Temperature:
»                Temperature (/hdd/0/temperature/0): 29.0 °C
»
-------------------------------
Press any key to continue...```

How can I get it to detect my Riing Plus controller? It is shown as supported in the documentation.

UPDATE:
I installed the latest dev build from a few days ago....alpha 30.4 and that picked up my controller!
For some reason it thinks the controller is a Riing quad but it does recognize it
HID
-------------------------------
»
»  [0x264a, 0x2260]: \\?\hid#vid_264a&pid_2260#8&3b90f6cf&0&0000#{4d1e55b2-f16f-11cf-88cb-001111000030}
»
-------------------------------
Controllers
-------------------------------
»
»  Name: Riing Quad
»        VendorId: 9802
»        ProductId: 8800
»        Version: 3.0.3
»        Ports:
»                1:
»                        Data: [PortId: 1, Speed: 50, Rpm: 986, Unknown: 255]
»                        Identifier: [9802, 8800, 1]
»                2:
»                        Data: [PortId: 2, Speed: 50, Rpm: 965, Unknown: 255]
»                        Identifier: [9802, 8800, 2]
»                3:
»                        Data: [PortId: 3, Speed: 50, Rpm: 986, Unknown: 255]
»                        Identifier: [9802, 8800, 3]
»                4:
»                        Data: [PortId: 4, Speed: 60, Rpm: 0, Unknown: 255]
»                        Identifier: [9802, 8800, 4]
»                5:
»                        Data: [PortId: 5, Speed: 60, Rpm: 0, Unknown: 255]
»                        Identifier: [9802, 8800, 5]
»        Available effect types: PerLed
»
-------------------------------
Press any key to continue...```

Just noticed that only the PerLed effect is available. Is this expected or should I be seeing more effects?

It is reporting Riing Quad because that must be what thermaltake includes now with all of their fans. It is the latest one and it supports all fans from Riing Plus to Riing Quad, if you had Riing Plus controller you would only be able to use Riing Plus fans. I think they finally figured out that having one controller supporting multiple fans is easier to manage than having a new controller for each new fan.
PerLed is an effect type that is supported by the controller, Riing Trio/Duo and Ring Quad controllers seem to only support this type. Riing Plus controllers had builtin effects, meaning you could for example set the controller to Spectrum_Fast type and the controller box itself would generate the effect without the need for software. PerLed means you just send colors for each led at 30hz from the software to create animations. All effect plugins are now of PerLed type and they generate the colors manually instead of relying on the controller box.


Do not build ACDC docs for aborted workflows

This is an improvement and, since I will most likely forget it tomorrow, better to keep this record.
I guess the easiest way of doing so, would be by creating a new job state.

Alan, this might be a little hard to do it, since document is pushed to acdc jobs failed not when status changes. So when workflow is aborted there are bunch of jobs already pushed to acdc.
Currently what we are doing is clean up after the fact.
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/ReqMgr/CherryPyThreads/CouchDBCleanup.py#L41
We can add "aborted" there. currently only "aborted-archived" is there.

I still did not investigate how it could be done, but the main reason for that is that ACDC building is a very very expensive operation for ErrorHandler, so if we could get rid of them in such cases, that would be a super improvement ;-) I will investigate it during this week ...

I think the key is when would you know whether workflow will be aborted. If you know very early on it reflect that not to push files to acdc. But workflow is aborted after many jobs finished the data will be already in acdc db. You could change the code to update acdc after certain state "completed" But I think that will be quite a lot of change.

Actually, my concern is not whether there are ACDC documents or not. Instead, I would like just to avoid building acdc documents for those jobs that were aborted in the agents (that were pending or running).

If you are just concern the jobs after aborted, I think it is pretty easy to add. I will take this issue. Don't spend time on this. :smile:

Exactly! Oh, that's great and you are the man!!! It's not urgent though, if you have more interesting things to work on, relax and go for that.

Hi Alan, it seems actually we are doing that already.
This is the place where files are uploaded to acdc server
https://github.com/dmwm/WMCore/blob/master/src/python/WMComponent/ErrorHandler/ErrorHandlerPoller.py#L194
and that is called by handleRetryDoneJobs
https://github.com/dmwm/WMCore/blob/master/src/python/WMComponent/ErrorHandler/ErrorHandlerPoller.py#L261
And the jobList is from
https://github.com/dmwm/WMCore/blob/master/src/python/WMComponent/ErrorHandler/ErrorHandlerPoller.py#L305
It only gets the job in retrydone status.  When workflow is aborted job status moved to "killed" and killed job doesn't move to "retrydone" status.
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/JobStateMachine/Transitions.py#L27
So it shouldn't put the inputdataset when job is aborted unless I am missing something in the code.

Yes, you're right. According to this code aborted jobs should not go to the acdc server.
Maybe I aborted my tests yesterday just when a hundred of jobs were failing....
Closing this issue, thanks Seangchan!


🚀 Feature Request: Full-height iframe in LivestreamDetailsModal

Describe the solution
(This is more a question than a suggestion.)
I am wondering if, when the LivestreamDetailsModal has a column layout (i.e. when the screen is sufficiently wide), it could be beneficial to make the iframe take up the full height of the modal instead of limiting its height to 9/16ths of its width.
Benefits:

Non-16:9 videos, especially vertical videos, would be shown in the largest dimensions possible since we would use all the available screen real estate. Below is a comparison.

Twitcasts would not awkwardly be contained in a 16:9 iframe.

I do not see any clear drawbacks, especially since both Twitch and YouTube seem to handle iframe dimensions which do not align with the video's aspect ratio.

I agree!
Indeed, when viewed on a PC, the extra space below the iframe is quite noticeable. (With the increase in vertical broadcasts, I think this is worth doing.)

Full height looks great! However, if we want to apply it we may not use <YouTubeEmbed> which is mentioned in #29
The reason is it will generate a wrapper and its size (width & height) could only be number or auto since the component prop type is set to number | undefined.

If we want to implement full-height we might need to do some workaround which I think it will be a little messy.
Besides, I have already done some workaround to make the Video UIs look the same while adding <YouYubeEmbed> and actually I don't really like it :(
Also, some functionalities like cover image lazy loading are missing. (ref: https://github.com/paulirish/lite-youtube-embed/issues/57)
Maybe we can keep <ResponsiveIframe> solution until we meet performance issue then see how <YouTubeEmbed> enhance solve the problem and create our own one

Do you mind sharing the motivation for https://github.com/sugar-cat7/vspo-portal/issues/29,

The motivation behind raising this issue was the hope of eliminating the logic for assembling URLs and gaining typed options by stopping the construction of iframe components on our own.
(However, I recognize that since we still need to create our own iframes for platforms other than YouTube, such as Twitch, tackling YouTube alone might not be very effective.)
As everyone has verified, if the Fragment is the reason it's difficult to handle with MUI styling, I believe it's fine to continue using the existing approach. :)

Yeah another benefit of using iframe is we can do customization for different components easily 👍
I can take this one if you don't mind.

Yes, let's put #29 on hold for now.
Feel free to adjust the assignment of tasks or break them down in a way that's most convenient for everyone!


Unable to launch instance.

Error: Error applying plan:
1 error(s) occurred:

aws_instance.InstancebyTerraform: 1 error(s) occurred:

aws_instance.InstancebyTerraform: Error launching source instance: MissingInput: No subnets found for the default VPC 'vpc-a14210c9'. Please specify a subnet.
status code: 400, request id: acf72520-3e0b-40e7-b963-1857e2735efa

Terraform does not automatically rollback in the face of errors.
Instead, your Terraform state file has been partially updated with
any resources that successfully completed. Please address the error
above and apply again to incrementally change your infrastructure.

Looks like there is no default subnet exists in your default VPC.  Create default subnet using AWS CLI, it should work.

To save others from googling I'll leave the below here
aws ec2 create-default-subnet --availability-zone ap-southeast-1a

To save others from googling I'll leave the below here
aws ec2 create-default-subnet --availability-zone ap-southeast-1a


Multilingual support

At the moment, the buttons and step-counter are English only, because it's hardcoded at https://github.com/easelinc/tourist/blob/master/tourist.js#L45 .
Would you mind making this a setting?

+1


Update coinbaseWallet Connector

Description
We are experiences issues when you connect with the coinbaseConnector.  It redirects you to the coinbase app, it opens the page in their in app browser. Enabling enableMobileWalletLink will fix this issue.
Additional Information
Before submitting this issue, please make sure you do the following.

[x] Read the contributing guide
[x] Added documentation related to the changes made.
[x] Added or updated tests (and snapshots) related to the changes made.

Changing the default is a breaking change so this will have to wait until v3. In the meantime, you can set this value yourself.
Docs
import { coinbaseWallet } from 'wagmi/connectors'

const connector = coinbaseWallet({
  appName: 'My Wagmi App',
  enableMobileWalletLink: true,
})

ah thought this wasn't included, Coinbase will default it on a few month tho as it's currently a beta feature 👍
but yeah we can set this manually


State of Repository

Hello guys, I am hayper, open source developer.
I will be taking leonardssh's place in maintaining this repository, his life is very busy.

Welcome Xharper


Increase clickable area of marker

How can I do it?

a very simple solution is to adapt the marker bitmap, setting a transparency area of some pixels around the icon.

In the older, ItemizedOverlay, you can override boundToHotspot(...) to change it to whatever you want.


Allow spaces before and after form inputs

An example of how I approached allowing the user to enter whitespace before and after an input where the UI will remove the whitespace before completing the form request.
If I do it this way over 28 files will have to be altered in this manner. Is there a more efficient way to do this?

Try https://github.com/khashayar/ng-trim-value-accessor

Sorry, think this was a bit more of a pain than we expected while discussing.
I'm not sure this is worth it unless a cleaner/easier way is found/suggested

What if we just had the validator display a message when there are leading and/or trailing spaces?
I think I've seen such UI on other sites, although I can't remember one right now.
It's less friendly than automatically fixing it for the user, but on the other hand it makes it very clear what is going on, and is easier to implement.

@wshands What is the status of this? It shows up in my PRs for review, but it is in draft status.
Would it make sense to either close it or move it out of draft status, whichever one applies?
Thanks.


Question: DebugInfoData and wasm_file

Hi,
I'm trying to prepare a PR to add split wasm support, and I think I'd like to set wasm_file on DebugInfoData to the name of the wasm file being processed.  Would that be an appropriate use of wasm_file?
Probably set it here:
https://github.com/bytecodealliance/wasmtime/blob/main/crates/wasmtime/src/compile.rs#L69
So I can use it somewhere around here
https://github.com/bytecodealliance/wasmtime/blob/2c6f38622d3b009293437839e5d75b4df64fa774/crates/cranelift/src/debug/transform/mod.rs#L88
if we encounter gimli::UnitType::Skeleton,  The wasm module will contain the name of the dwo, however if the dwo files were packaged into a dwp then that name is arbirtrary and as far as I can see we can load it by convention, i,e, wasm file with the extension changed to .dwp or perhaps it can be an option passed to wasmtime, but in a later PR.  Chrome loads .dwp by convention it seems, and emscripten follows that conventions.  (I'm getting this info from https://developer.chrome.com/blog/faster-wasm-debugging/)
Thanks,

Seems reasonable to me yeah, I had no idea this field even existed myself! I do think we'll want to be a little careful about the filesystem access though since using Module::new for example probably shouldn't go off reading the filesystem. When using Module::from_file, however, reading the filesystem seems reasonable, but we probably don't want to read the dependent object files for each function and we'll want to make sure it only happens at most once.

I wonder if I can ask for some more advice.  I would like to pass the dwarf in here
https://github.com/bytecodealliance/wasmtime/blob/85ffc394a77d06bea15f460c78251651d154aaae/crates/wasmtime/src/compile.rs#L661
At least that's my current thinking.  If I load the dwarf in Module::from_file then I can store it in Module or ModuleInner I guess, but how would I access that in link_and_append_code, pass it all the way around?

It'll likely want to get attached somewhere within ModuleTranslation along the way. For a Module::from_file that's probably threading an option down into Module::from_binary which indicates that reading from the filesystem is ok or similar

Thanks very much for the help, I have something working I think

Going to close this now as #8055 exists.  Thanks!


[Question]: Inconsistent results of embedding between llamaindex and huggingfacd API

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
Below are two pieces of codes:

the first one is the code that is used to retrieve the most similar top k by llamaindex API:

import chromadb
from llama_index.vector_stores.chroma import ChromaVectorStore
from llama_index.core.node_parser import SentenceSplitter
from llama_index.core import StorageContext, Settings, VectorStoreIndex
from chromadb.utils import embedding_functions
from llama_index.embeddings.huggingface import HuggingFaceEmbedding

embed_model = HuggingFaceEmbedding(model_name="dunzhang/stella_en_1.5B_v5", revision="038b58da67ca1432ed75b0e4a44bba403020d38a")
Settings.node_parser = parser
chroma_client = chromadb.PersistentClient()
try:
    chroma_collection = chroma_client.create_collection("entity_collection", metadata={"hnsw:space": "cosine"})
except Exception as e:
  if "already exists" in str(e):
    chroma_collection = chroma_client.get_collection("entity_collection")
  else:
    raise e

vector_store = ChromaVectorStore(chroma_collection=chroma_collection)
storage_context = StorageContext.from_defaults(vector_store=vector_store)
index = VectorStoreIndex.from_documents(entity_doc, storage_context=storage_context, embed_model=embed_model, show_progress=True)

retriever = index.as_retriever(similarity_top_k=30)
nodes = retriever.retrieve("holistic care")

the second codes blow testing the performance of given embedding model by purely huggingface API:

import numpy as np
from sentence_transformers import SentenceTransformer

def cosine_similarity(vec1, vec2):
    dot_product = np.dot(vec1, vec2)
    norm_vec1 = np.linalg.norm(vec1)
    norm_vec2 = np.linalg.norm(vec2)
    return dot_product / (norm_vec1 * norm_vec2)

queries = [
    "holistic care",
]
docs = [
    "holistic care",
    "Physical care",
    "Psychological care",
    "Social and family care",
    "Spiritual and cultural care",
    "Effective ways of communicating",
    "Assisting patients",
    "patients to make decisions",
    "emotional reassurance and support",
    "patients advocat",
    "Effective ways of communicating"
]

model_test = SentenceTransformer("dunzhang/stella_en_1.5B_v5", trust_remote_code=True).cuda()
query_embeddings = model_test.encode(queries)
doc_embeddings = model_test.encode(docs)
similarities = model_test.similarity(query_embeddings, doc_embeddings)
print(similarities)

the result of the second piece of code is shown as below cell
tensor([[1.0000, 0.6447, 0.6307, 0.6184, 0.6823, 0.4400, 0.5293, 0.6325, 0.6819,
         0.6945, 0.4400]])

So from the second piece of codes, we can at least find that for the identical entity (phrase), such as the "holistic care" as query in the example, the cosine similarity with itself will be 1, which meet our knowledge.
However, for the first piece of code, I used the identical embedding model, and also "holistic care" as a query to retrieve relevant text from chromaDB (I'm very sure there is a "holistic care" as a chunk in the database), but I cannot retrieve "holistic care" out, although it can (depends on how many chunks I'd like to retrieve), I always get very low similarity among them. What happened? is there any problem of my first piece of codes?

@dosu,
So any problem with my codes?

Is there anyone who can help me look into this problem? I have been stuck on it for almost a week. Any suggestions are welcome！

Is there anyone who can help me look into this problem? I have been stuck on it for almost a week. Any suggestions are welcome！

Solved! metadata of each node will make the retrieval abnormal, not sure why


Data model without ESP-Matter (CON-626)

There are examples of esp32 device in connectedhomeip repository (for example, lighting-app). Also there is an example zap-light in esp-matter repository. It isn't clear how Data model is configured in these cases.
Are there any information about that or can someone explain?

For examples except zap-light example in esp-matter, the date model is dynamically generated when running the examples. We will call node::create() and endpoint::create() to create the data model. And for zap-light and all the examples in connectedhomeip repository, the data model is statically generated by ZAP when building the examples. we will use chip_app_component_codegen and chip_app_component_zapgen to generate the data model codes in CMake file.

Thank you for answer. How to create own data model using ZAP? Should files .matter and .zap somehow changed?

Yes, you should change the .matter and .zap files to create own data model.

They cannot be modified manually. Right? ZAP tool is used to generate .zap. What tool is used to generate .matter?
Are there anywhere a step-by-step guide?

https://github.com/project-chip/connectedhomeip/blob/master/docs/code_generation.md Here is the code-generation doc.

@wqx6 thank you!


add an option to slow down consumer.File's polling when reaching EOF

I'm using consumer.File and it's stat'ing a slow moving log file 1100-1800 times a second, which is causing performance issues.

Looks ok to me, but this is a symptomatic fix. The main problem is that there are too many tests.
I created #137 to fix the core issue in 0.5.x.


Content-Type header [application/vnd.elasticsearch+x-ndjson; compatible-with=8] is not supported’)

When I attempted to connect to the ES server 7.10 using this ES client, I encountered an error message as follows:
The "Content-Type" header ["application/vnd.elasticsearch+x-ndjson; compatible-with=8"] is not supported.
I checked package.json and found that es-plastic rely on elasticsearch.js 8+ , could this be the cause of  problems?

yes it is.

yes it is.

Any suggestions for this ?  Many thanks in advance

Nope unfortunately, upgrade your ES db.


StyleCI

Are you having trouble configuring StyleCI? Is there anything I can help with?

@GrahamCampbell I've just set an example config on StyleCI and it works great thanks!

You do know the example config wasn;t actually intended to be used. It was just an example of what you can configure. It's actually totally broken.

The "default" config would be:
preset: recommended

linting: true

finder:
  exclude:
    - storage
    - vendor
  name:
    - "*.php"

That's basically what get's used as a base, then everything else you configure get's applied on top.

  contains:
    - "Foo"

That part of your config there basically means that zero files are actually getting checked by StyleCI which is why everything is green.

I've updated your config to something I think better matches your codebase.

You can see the results here: https://styleci.io/analyses/zG119X.

Again, thank you. I should have read the docs properly :)

Sure. You're not actually the first person to have copied the example config like that. I've seen it a few times before. Maybe we need to make that stuff clearer. :)


add connection mode for udp sockets (work in progress)

Hi,
This PR adds a connection mode for UDP sockets, which correspond to std::net::UdpSocket::connect().
This behavior is also described in 2008 POSIX standard.

This needs tests obviously, but I'm opening the PR to get early feedback.

This PR adds a connection mode for UDP sockets, which correspond to std::net::UdpSocket::connect().
This behavior is also described in 2008 POSIX standard.

Note that smoltcp, by design, makes no effort to conform to POSIX or behave like std::net. In particular the TCP sockets are nothing like TcpListener/TcpStream; that is the job of a higher-level abstraction to present them like that. They are rather like TCBs (transmission control blocks) from RFC 793.
Anyway. I am partial to the idea behind this socket, but I strongly dislike the added branching in the API. Not sure what to make of it.

I see, thanks for the explanation. I think in this specific case, I can indeed provide this abstraction at a higher level (for the context I'm trying to use smoltcp as the network stack for redox).

@little-dude It's virtually certain that you will need to provide a higher-level abstraction, at least for TCP and to account for blocking. Therefore it seems alright to also do that for UDP sockets.
By the way, I am rather happy about the use of smoltcp in redox--the existing redox network stack wasn't exactly inspiring...


DOC-7567: React with emojis and GIFs page needs animated GIFs for mobile app workflows

Summary
This PR adds visual workflows for the Mattermost mobile app tab on the React with emojis and GIFs page.
Ticket Link
Fixes #7567

[x] This PR follows Mattermost Contributing Guidelines.
[x] The changes are tested in a local docs build.

@cwarnermm Please review and suggest changes, if any. Thanks


Profile.get_followers() and Profile.get_followees() returning empty list

Instastalk 4.10
Python 3.10.2
Windows
This is my script:
L = instaloader.Instaloader( )
L.load_session_from_file(...)
p = Profile.from_username(L.context, ...)

# so far everything works as p contains the Profile instance

try:
      followers = p.get_followers()
except Exception as e:
      print("Ex")
      sys.exit()

try:
      followees = p.get_followees()
except Exception as e:
      print("Ex")
      sys.exit()

The execution does not stop, just followers and followees are both empty.

hi
how did you able to login and use instaloader
i am getting this error:
JSON Query to accounts/login/: Could not find "window._sharedData" in html response. [retrying; skip with ^C]
i am login using L.login(username,password)
please help

Is account public?


Shows the buttons without styling

Hi,
I applied the library in my project, wich is an app made with Electron and ReactJS and everything works fine, but, I don't know if I have to configure another property, but, I tried to implement the navbar on top and it's working but the buttons, like zoom in, zoom out, rotation, etc it's shows without styling:

I saw the example and my implementation is equal, I don't know why the buttons shows like the image that I attached.
I don't know if is an issue or I have to add other property, you can find my implementation in the following image:

I will be waiting for your response.
Thanks in advance.
Best regards

No that's it, you don't have to configure anything else.
It uses Bulma & Material icons for styling so somehow your css is overriding it's default style or its not getting installed.

Ohhh I see the ligth. Thanks @ansu5555. My app use bootstrap so, now, I know why the buttons styling are broken. Thanks for the information and I recommend to you add this dependency in the documentation, because, maybe, you will have to response the same to other users in several times.
Thank you very very much.
Best Regards,
Leandro

Thanks for the suggestion, closing this issue as your query is resolved.
Feel free to open an issue if anything is there.


adds alert message banner

Addresses Issue #815

Adds site-wide banner if value for alertMessage field is present on brandData (currently using a static variable. will change)
Adds an alert button and dialog modal on the viewer page in an attempt to mindful of over crowding the page.

@subdavis I think merge issues with this branch have been fixed.


small question

Hello,
I put th commande
./convert_yolo_trt.sh
YOLOv3 or YOLOv4. Input 3 or 4
4
Do you want to download the model? [Y/N]
y
What is the input shape? Input 288 or 416 or 608
416
Creating yolov4-416.cfg and yolov4-416.weights
How many categories are there?
An d i have this question What should be done ?
thank you for your help
Regards
Minion40

The number of categories should be the number of classes you have trained the model on. If you used the ./download_yolo.sh, then it will be downloading the yolo weights trained on MS COCO.
Thus, there will be 80 categories inside. I'll remove that prompt since it'll almost always be 80


Restructured to deployable service

Some updates to make down the line but looking good let's get this merged so we can make sure to start incrementally adding features with smaller PRs going forwards


ERROR: Unable to extract uploader id

Checklist

[ ] I'm reporting a broken site support issue
[ ] I've verified that I'm running youtube-dl version 2021.12.17
[ ] I've checked that all provided URLs are alive and playable in a browser
[ ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ ] I've searched the bugtracker for similar bug reports including closed ones
[ ] I've read bugs section in FAQ

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'https://www.youtube.com/watch?v=j-Sh-W0S0zM']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 2.7.17 (CPython) - Linux-5.3.0-28-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 3.4.6, ffprobe 3.4.6, phantomjs ., rtmpdump 2.4
[debug] Proxy map: {}
[youtube] j-Sh-W0S0zM: Downloading webpage
[youtube] j-Sh-W0S0zM: Downloading MPD manifest
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/youtube.py", line 1794, in _real_extract
    'uploader_id': self._search_regex(r'/(?:channel|user)/([^/?&#]+)', owner_profile_url, 'uploader id') if owner_profile_url else None,
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

Received the error tonight for the first and only time.

I received the same issue just now.  I'm thinking that YT must have changed their system recently.
Computer specifications

16" MacBook Pro M1
macOS 13.2.1

Output:
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://youtu.be/REpafNHV-ns']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Git HEAD: a222a5bbc
[debug] Python version 3.10.10 (CPython) - macOS-13.2.1-arm64-arm-64bit
[debug] exe versions: ffmpeg 5.1.2, ffprobe 5.1.2
[debug] Proxy map: {}
[youtube] REpafNHV-ns: Downloading webpage
[youtube] REpafNHV-ns: Downloading MPD manifest
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/opt/homebrew/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "/opt/homebrew/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "/opt/homebrew/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/opt/homebrew/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/extractor/youtube.py", line 1794, in _real_extract
    'uploader_id': self._search_regex(r'/(?:channel|user)/([^/?&#]+)', owner_profile_url, 'uploader id') if owner_profile_url else None,
  File "/opt/homebrew/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Similar issues:

#31622
#31621
#31620
#31619

Pinned issue:
READ THIS BEFORE OPENING A NEW ISSUE! #30839

Known issues, fixed in git master code:

YouTube: Unable to extract uploader id

+1 for quoting the log correctly
-1000 for not following the checklist despite including it


Optimize putScalar (and shape(), stride())

Consider the following code here:
https://github.com/deeplearning4j/nd4j/blob/d40d4e9ec748904f15c329178dee133be9d76f51/nd4j-backends/nd4j-api-parent/nd4j-api/src/main/java/org/nd4j/linalg/api/ndarray/BaseNDArray.java#L1074-L1074
Typical usage is to call myArray.putScalar(new int[]{...}, myValue)
Then, internally, for every single element

Get shape: Create a new int[] (and a new NioInt buffer?) using BaseNDArray.shape() https://github.com/deeplearning4j/nd4j/blob/d40d4e9ec748904f15c329178dee133be9d76f51/nd4j-backends/nd4j-api-parent/nd4j-api/src/main/java/org/nd4j/linalg/api/ndarray/BaseNDArray.java#L3763
Get stride: As above (new int[], new buffer object) https://github.com/deeplearning4j/nd4j/blob/d40d4e9ec748904f15c329178dee133be9d76f51/nd4j-backends/nd4j-api-parent/nd4j-api/src/main/java/org/nd4j/linalg/api/ndarray/BaseNDArray.java#L3763

By my count: typical usage of putScalar results in 3 int[] and 2 temporary buffer objects being created, for every single value to be put.
While we are at it: we should add a putScalar(int row, int col, double value) that doesn't result in array calculation, as per this: https://github.com/deeplearning4j/deeplearning4j/issues/1463

https://github.com/deeplearning4j/nd4j/pull/871


Feature Request: Mapping Component

Not so much as part of the core Forge repository, but it would be nice to have a centralized ESRI-based mapping component.  Tyler Spatial currently has something like this, but it's based on the 3.x ArcGIS API with is EOL on July 1, 2024, and also uses TCW 1.x instead of Forge 2.0.  There may already be plans/efforts to update it.  Tyler Public Safety has a 4.x implementation (not Forge-based), but an relatively early version of 4.x, and would require some heavy lifting to get it updated and decoupled from the application.
The ArcGIS JavaScript API and the Dojo Toolkit dependencies require specialized knowledge, and it would be beneficial to centralize this component, rather than internally develop multiple variations on it.  Would also benefit maintenance and support of the component.
Describe the component:
Most likely in a satellite repository, provide a map view with configuration and/or composable components to meet the requirements of various divisions.
Describe the use case:
Some examples:

Data visualization: plotting one or many points on a map (with clustering if necessary), heat maps
Address verification: helping resolve a multi-match intersection (e.g. horseshoe shaped road) or lat/long to address
Routing
Viewing/updating details of points in a layer (e.g. Fire Hydrants)

Required features:

Plot an arbitrary data source on the map.
Inspect plotted points for additional details, handling multiple points at the same location.
Display a heat map based on a data source of points.
Toggle layers on and off
Select layers from the map
Draw arbitrary shapes on the map and get coordinates.
Search/inspect map features
Potentially more, will require more discussions to define fully.

Nice-to-have features:
Will require more discussions to define.
Screenshots:
Can be provided on request.
Please complete the following information:

Will this be a contribution by you or your team? Probably a collaborative effort.
What platforms will this component be used on? all
Does this component require a responsive design? Y
I have searched existing issues before creating this request? Y

Additional context:
Creating this as a tracking issue based on a Frontend Governance Board discussion.

Just to add to this, I do know that the spatial team is currently in progress upgrading their mapping component to the 4.0 API, as well as upgrading Forge 2.x at the same time. I'm fairly certain it already supports a number of the features outlined above, and I hope with the new changes that it becomes more lightweight.
I don't have much more information other than that, but it may be worth meeting with them to discuss any other needs they may not be aware of as well.

This will not be part of the core/atomic component library. It may make sense as a separate library within the awesome-forge ecosystem, but work has continued with the spatial team.


zabbix-agent2-plugin-postgresql: 6.4.15 -> 7.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/zabbix-agent2-plugin-postgresql/versions.
meta.description for zabbix-agent2-plugin-postgresql is: Required tool for Zabbix agent integrated PostgreSQL monitoring
meta.homepage for zabbix-agent2-plugin-postgresql is: https://www.zabbix.com/integrations/postgresql
Updates performed

Golang update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 7.0.0 with grep in /nix/store/gdxra9k4y4qg4zzsbnv0gbwf3mxhbdvy-zabbix-agent2-plugin-postgresql-7.0.0
found 7.0.0 in filename of file in /nix/store/gdxra9k4y4qg4zzsbnv0gbwf3mxhbdvy-zabbix-agent2-plugin-postgresql-7.0.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

zabbix-agent2-plugin-postgresql

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/gdxra9k4y4qg4zzsbnv0gbwf3mxhbdvy-zabbix-agent2-plugin-postgresql-7.0.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/be0d1a09af10c5f7907ab4c3762b0957aa63b749#zabbix-agent2-plugin-postgresql

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/gdxra9k4y4qg4zzsbnv0gbwf3mxhbdvy-zabbix-agent2-plugin-postgresql-7.0.0
ls -la /nix/store/gdxra9k4y4qg4zzsbnv0gbwf3mxhbdvy-zabbix-agent2-plugin-postgresql-7.0.0/bin

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

1 package built:

zabbix-agent2-plugin-postgresql

Maintainer pings
cc @gador for testing.

waiting on #317463


i18n(fr): Update guides/site-search.mdx from #2009

Description
Update guides/site-search.mdx from #2009

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

fr
guides/site-search.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Data Form of the MaLa-ASR

System Info
torch 2.1
Information

[X] The official example scripts
[ ] My own modified scripts

🐛 Describe the bug
bash decode_MaLa-ASR_withkeywords_L95.sh

Hi, I'm currently working on reproducing the results of MaLa-ASR and have downloaded the slidespeech dataset from https://www.openslr.org/144/. While running the provided decoding script, I noticed that it requires the file located at /nfs/yangguanrou.ygr/slidespeech/${split}_oracle_v1/. Could you please clarify what the format of this file is? Do I need to preprocess the downloaded data in any specific way, such as splitting the audio based on timestamps?
Error logs
no file named test_oracle_v1
Expected behavior
Could you please provide the steps for data processing and explain the format of the data? Thanks, looking forward to your reply.

The location of the slidespeech dataset can be modified through config file "mala_asr_config.py".
You can change "/nfs/yangguanrou.ygr/slidespeech/${split}_oracle_v1/." to your own path.
The dataset requires four files: "my_wav.scp", "utt2num_samples", "text", "hot_related/ocr_1gram_top50_mmr070_hotwords_list"
"my_wav.scp" is a file of audio path lists. We transform wav file to ark file, so this file looks like
ID1 xxx/slidespeech/dev_oracle_v1/data/format.1/data_wav.ark:22
ID2 xxx/slidespeech/dev_oracle_v1/data/format.1/data_wav.ark:90445
To generate this file, you can get audio wavs from https://www.openslr.org/144/ and get the time segments from https://slidespeech.github.io/. It provides segments, transcription text, OCR results at https://speech-lab-share-data.oss-cn-shanghai.aliyuncs.com/SlideSpeech/related_files.tar.gz) (~1.37GB). You need to segment the wav by the timestamps provided in segments file
This related_files.tar.gz also provides "text" and a file named "keywords". The file "keywords" refers to "hot_related/ocr_1gram_top50_mmr070_hotwords_list", which contains hotwords list.
"utt2num_samples" contains the length of the wavs, which looks like
ID1 103680
ID2 181600
...
Sorry for the late reply, been busy lately, hope your reproduction goes well!

@yanghaha0908 Thank you for your answer. It is strongly recommended that this answer be written into the mala README file.

I have added it to the README.md file of Mala-ASR, refer to https://github.com/X-LANCE/SLAM-LLM/pull/168.


Update README.md

-Transistors and their appliances,
-we commonly concentrate on bipolar transistors used in most electronic appliances of these days.

thanks let's keep on building the repo

Be familier with transistors,capacitors,diodes,resistors and develope eagle eyes of atechnician,repair cafe is adone deal.


feat: Update Highlights docs

Current
We have "Fill content" and "Add and Summarize" buttons in the "Highlight suggestions" section that are not covered here.
Originally posted by @adiati98 in https://github.com/open-sauced/docs/pull/213#discussion_r1405228070
Solution

[ ] add directions on adding a highlight from the highlight suggestion.
[ ] Explain both the "Fill content" and "Add and Summarize" buttons
[ ] Remove the existing gif for the highlight
[ ] Create a new gif with one that shows the suggestions and name it "highlight.gif". (The naming is important so we don't break anything that's been linked elsewhere.)

Hi can i take this up?

Hi can i take this up?

Hey @oluwatobi2001. Just type .take in the comment section and you'll be automatically assigned.

.take

Hi @oluwatobi2001, are you still interested in working on this issue?
If not, I would like to take it. Let me know. Thanks!

Yes, I'm still interested.
On Wed, Jun 5, 2024 at 10:19 AM Viviana Yanez @.***>
wrote:

Hi @oluwatobi2001 https://github.com/oluwatobi2001, are you still
interested in working on this issue?
If not, I would like to take it. Let me know. Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/open-sauced/docs/issues/224#issuecomment-2149303301,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGW7FFK6FZPFO5TSVBTL7S3ZF3JZLAVCNFSM6AAAAABAJXZWJSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBZGMYDGMZQGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

.take

Hello! I have a couple of questions before submitting a PR for this issue:
First, I understand that this change should prioritize the Highlight suggestions section usage, so the current indications should remain in the docs but maybe below the new instructions and the new gif. Do you think this approach is correct, or does anyone have any other thoughts on this?
Second, I encountered an issue while using the "Add and summarize" button. As shown in the gif below, I receive an error message when I try to summarize a PR:

I am also not able to use the Auto-Summarize feature if I paste the link manually:

So I wanted to check if I am following the right steps here or if I am missing something.
Thanks a lot for any help :)

Hey @vivitt,
Sorry, I don't quite understand what do you mean in the first question. Can you elaborate more?
As for your question about summarize, I can confirm that I also have the same issue.
You might want to create an issue on our app repo to report this. 😊

Hello @adiati98 !
Didn't find a similar open issue, so I created one.
For my first question, I was confused because the How to Add a Highlight section now includes a set of bullet points and a GIF that need to be replaced. My bad for not explaining that clearly.
What I wanted to know is if should I document the "Fill Content" and "Add and Summarize" buttons by adding more bullet points below the existing ones, or is it better to reorganize the section by first documenting the "Highlight Suggestions" buttons and then the option of manually pasting a link?
Thanks!

Hey @vivitt,
I think reorganize would be better.
We can say something like, "There are two ways to add highlights, with the Highlight suggestions and manually." You can experiment with adding h3 here if it makes things clearer.
Then, follow the checkbox on the issue.
The instructions are all in bullet points for better read and understanding.
You can draft it first, and we'll see how we can improve and clarify them.
And feel free to let me know if you have further questions or need help. ✨

Thanks @adiati98! Here's a possible draft for organizing the instructions:
## How to Add a Highlight

There are two ways to adding highlights:

### Using Highlight Suggestions

- Go to [OpenSauced's Highlights feed](https://app.opensauced.pizza/feed).
- Click the "Post a highlight to show your work!" input.
- Search the relevant blog post, pull request, or issue in the "Highlight suggestions" section.
- Either use the "Fill content" button to paste the suggested link or the "Add and Summarize" button to paste the link and Auto-Summarize the content.

![highlights demo](example.gif)

### Adding a Link Manually

- Go to [OpenSauced's Highlights feed](https://app.opensauced.pizza/feed).
- Click the "Post a highlight to show your work!" input.
- Paste the URL to your blog post, pull request, or issue in the bottom input.
- Either use our Auto-Summarize feature or write your own summary.

What do you think? Any suggestions?
Thanks :)

Hey @vivitt,
Sorry for taking a while to respond.
This looks great! ✨
However, I can see a room for improvement based on below things:

The first two bullet points in each method are the same. We might be able to avoid repetition here.
The "Highlight suggestions" only suggest issue and PRs, not blog posts.

Below is a raw suggestion. Feel free to give other ideas.
## How to Add a Highlight

First, go to [OpenSauced's Highlights feed](https://app.opensauced.pizza/feed), then click the "Post a highlight to show your work!" input.

There are two ways to adding highlights:

### 1. Using Highlight Suggestions

- Search the pull request or issue that you want to add in the "Highlight suggestions" section.
- Click the "Fill content" button to paste the suggested link or the "Add and Summarize" button to paste the link and auto-summarize the content.

:::note

The "Highlight suggestions" are only suggesting issues and pull requests that you created. For blog posts, you need to add them manually.

:::

![highlights demo](highlight.gif)

### 2. Adding a Link Manually

- Paste the URL to your blog post, pull request, or issue in the bottom input.
- Either use our Auto-Summarize feature or write your own summary.


[Electron] Mac OS desktop app root tab bar view content visible in locked down app for a split second after having gone idle behind another window

Root tab bar content visible for a split second while

booting app; and

more concerningly, after having gone idle (been locked down) while the app was in the background, usually obscured by other applications' windows, especially while the app's About window is visible/key, when the app is then brought back to the foreground.

I've done some digging into this and think 2 comes down to a Chrome rendering issue, but I'm not entirely certain. 1 can probably be fixed by architecture improvements.
1 can only ever occur when there is no data, and so is not a security issue – just a UX one.
2 can occur while decrypted user data is being displayed so is a security issue which can probably be exploited by some kind of screenshotter.
2 has been solved in the iOS (Swift) app.
2 should not be an issue under Android (Cordova JS) because the app is explicitly locked down when backgrounded.

I found a way to reproduce this 100% of the time on a Mac.
Set your idle time low, to like 5s, so you don't have to wait long, then place the MyMonero desktop app window behind another opaque window.. then after it must have gone idle, cmd-tab back into the app. You should see 2, where the unlocked content is visible for what must be only one (?) frame.
Funny enough this does not happen when the app is obscured by a transparent window (such as Terminal), which leads me to strongly suspect again this is some interaction with Electron/Chrome and the rendering system/Quartz.

Doesn't look fixed in electron 2.0 / chrome 61


Disply in_public

WHY
Get public or private repos.
WHAT
print

$ ./bin/dockerepos get quay.io/wantedly/test
Repository:
	quay.io/wantedly/test
Visibility:
	private
Permissions:
	wantedly+wantedlybot(write)
	koudaiii(admin)
	infrastructure(admin)
	developer(write)


Error when compiling quantized pytorch model with vai_c_xir: [UNILOG][Check Failed: shift_cut >= 0][/home/xbuild/conda-bld/xcompiler_1606385139459/work/src/slnode/DPUv2/SlNodeDptConv.cpp:55][XCOM_DATA_OUTRANGE][Data value is out of range!]

Hello,
I quantized a pytorch model and got an .xmodel. During compilation,  I get the following error:
VITIS_AI Compilation - Xilinx Inc.
**************************************************
[UNILOG][INFO] The compiler log will be dumped at "/tmp/vitis-ai-user/log/xcompiler-20220810-215919-99"
[UNILOG][INFO] Compile mode: dpu
[UNILOG][INFO] Debug mode: function
[UNILOG][INFO] Target architecture: DPUCZDX8G_CUSTOMIZED
[UNILOG][INFO] Graph name: Resnet, with op num: 604
[UNILOG][INFO] Begin to compile...
[UNILOG][Check Failed: shift_cut >= 0][/home/xbuild/conda-bld/xcompiler_1606385139459/work/src/slnode/DPUv2/SlNodeDptConv.cpp:55][XCOM_DATA_OUTRANGE][Data value is out of range!]
*** Check failure stack trace: ***
Aborted (core dumped)

Has anyone had the same error or knows what might be causing this error message?
I use Vitis-AI Docker vitis-ai-cpu:1.3.411.
Hope you can help me.
Thank you!
Here is the quantized xmodel:
链接：https://pan.baidu.com/s/1yu4aLBg_wIKxxHuFEvMaSA
提取码：6m28

Hi.

Hi, @wgshun
I noticed there are lots of scale layers converted from BatchNorm layer in your model, this is the root cause leads to deployment problem, which will make CONV layers feature maps not quantization friendly, deployment failure or not-acceptable performance of final deployment.
Suggest moving the BatchNorm layers to the output position of CONV layers, then they will be merged info CONV layers and will get reasonable deployment result.


Refactor base indicator classes

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #1263

[ ] Tests for the changes have been added (for bug fixes / features)

[x] (If applicable) Documentation has been added / updated (for bug fixes / features)

[x] CHANGES.rst has been updated (with summary of main changes)

[x] Link to issue (:issue:number) and pull request (:pull:number) has been added

What kind of change does this PR introduce?
Move around some of the base indicator classes code. Split the missing values handling from the resampling handling. This way, indicators like return_level can reduce the full time axis and still perform some missing values handling.
Does this PR introduce a breaking change?
No, not yet.
Other information:
@huard @RondeauG, if I am not mistaken, return_level and fit currently have missing values handling disabled because it was impossible with the previous classes to have both this and full reduction of "time". Now that it is possible, should we activate it ?
Similarly, stats had the "Any" missing method forced. Is there a reason for that ? Here, I removed the argument, meaning it will use "from_context".

if I am not mistaken, return_level and fit currently have missing values handling disabled because it was impossible with the previous classes to have both this and full reduction of "time". Now that it is possible, should we activate it ?
Similarly, stats had the "Any" missing method forced. Is there a reason for that ? Here, I removed the argument, meaning it will use "from_context".

Absolutely ! For all 3, having full control on the missing argument would be a great quality-of-life change.

coverage: 90.674%. first build when pulling 71c52a797bedcb041821ff8fea8a25ea9c469d51 on indcls-fullreduce into 36dcbdf67b6006698d5bfaa4e36fe525f36e4134 on master.

Issue https://github.com/CSHS-CWRA/RavenPy/issues/307 is probably due to the fact that this PR made fit a subclass of ReducingIndicator. It was previously a subclass of Indicator, which has no missing value logic in it.
So the generic fit indicator essentially went from skipping missing value checks, to apply the missing algorithm from context (typically any). I think we should have highlighted this more clearly in the release notes.


Could not load NIB in bundle, after install with cocoapods

Hi, I switched form TSMessage to RMessage and when using it after pods installation I got the following error each time I call any of the Messages. It seems to fail loading de xib file. It happens in any call, for example:
[RMessage showNotificationWithTitle:@"Log In Error"
                                   subtitle:@"Username contains an error"
                                       type:RMessageTypeWarning
                           customTypeString:nil];

I get the follwing error:
2016-01-18 09:43:33.600 AppName[40137:3085852] *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'Could not load NIB in bundle: 'NSBundle </Users/MyUser/Library/Developer/CoreSimulator/Devices/XXXX/data/Containers/Bundle/Application/XXXXX/AppName.app> (loaded)' with name 'RMessageView''
*** First throw call stack:
(
	0   CoreFoundation                      0x0000000108dc1e65 __exceptionPreprocess + 165
	1   libobjc.A.dylib                     0x000000010883adeb objc_exception_throw + 48
	2   CoreFoundation                      0x0000000108dc1d9d +[NSException raise:format:] + 205
	3   UIKit                               0x00000001099893cf -[UINib instantiateWithOwner:options:] + 499
	4   UIKit                               0x000000010998bb5b -[NSBundle(UINSBundleAdditions) loadNibNamed:owner:options:] + 223
	5   RMessage                            0x0000000107d889e1 -[RMessageView initWithDelegate:title:subtitle:iconImage:type:customTypeString:duration:inViewController:callback:buttonTitle:buttonCallback:atPosition:canBeDismissedByUser:] + 561
	6   RMessage                            0x0000000107d86408 +[RMessage showNotificationInViewController:title:subtitle:iconImage:type:customTypeString:duration:callback:buttonTitle:buttonCallback:atPosition:canBeDismissedByUser:] + 664
	7   RMessage                            0x0000000107d8611e +[RMessage showNotificationInViewController:title:subtitle:type:customTypeString:] + 270
	8   RMessage                            0x0000000107d85cec +[RMessage showNotificationWithTitle:subtitle:type:customTypeString:] + 156
	9   AppName                             0x0000000107a2f6f2 -[LoginViewController validLogInParameters:password:] + 242
	10  AppName                             0x0000000107a30297 -[LoginViewController login:password:] + 215
	11  AppName                             0x0000000107a2dcc8 -[LoginViewController loginAction:] + 248
	12  UIKit                               0x00000001095cf194 -[UIApplication sendAction:to:from:forEvent:] + 92
	13  UIKit                               0x000000010973e6fc -[UIControl sendAction:to:forEvent:] + 67
	14  UIKit                               0x000000010973e9c8 -[UIControl _sendActionsForEvents:withEvent:] + 311
	15  UIKit                               0x000000010973daf8 -[UIControl touchesEnded:withEvent:] + 601
	16  UIKit                               0x000000010963e49b -[UIWindow _sendTouchesForEvent:] + 835
	17  UIKit                               0x000000010963f1d0 -[UIWindow sendEvent:] + 865
	18  UIKit                               0x00000001095edb66 -[UIApplication sendEvent:] + 263
	19  UIKit                               0x00000001195a2871 -[UIApplicationAccessibility sendEvent:] + 77
	20  UIKit                               0x00000001095c7d97 _UIApplicationHandleEventQueue + 6844
	21  CoreFoundation                      0x0000000108ceda31 __CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION__ + 17
	22  CoreFoundation                      0x0000000108ce395c __CFRunLoopDoSources0 + 556
	23  CoreFoundation                      0x0000000108ce2e13 __CFRunLoopRun + 867
	24  CoreFoundation                      0x0000000108ce2828 CFRunLoopRunSpecific + 488
	25  GraphicsServices                    0x000000010dcabad2 GSEventRunModal + 161
	26  UIKit                               0x00000001095cd610 UIApplicationMain + 171
	27  AppName                             0x0000000107a45c2f main + 111
	28  libdyld.dylib                       0x000000010ba1792d start + 1
	29  ???                                 0x0000000000000001 0x0 + 1
)

On the other hand I added the code manually and there are no issues. Does this happen to anybody else?
PS. Perhaps I should not raise this here, but I have noticed some minor differences from the documentation github and the code such as enumerations like
RMessageNotificationType... that should be RMessageType... .
Thanks.

Hi @ogonzalezadell Can you pull the latest changes to RMessageView and install the pod normal way and see if this issue still exists. I believe  #commit: 3d249a8 fixes your issue. Sorry for the wait been so busy with some other work.

@ogonzalezadell been two weeks with no response. I will assume your issue has been closed as I believe it has.

Yes It works properly. Thaks


exec?

Would you be open to implementing (or accepting a PR to do so) the exec endpoint? Our use-case requires it.

Totally open. Prs are welcome

This should then be closed?

as if https://github.com/softprops/shiplift/pull/44, I think so. we can reopen if not.


Cannot upload add-ons - validator fails to return

Describe the problem and steps to reproduce it:
(Please include as many details as possible.)
This happens on both the upload page and the validator page
I tried to upload a new version of conversations, and also an older version (3.1.7).  I also tested the nightly version and the release version of Firefox.
What happened?
The upload proceeds and the progress bar gets to 100%, but then never completes.
What did you expect to happen?
The validation should complete and I should be able to upload the add-on.
Anything else we should know?
(Please include a link to the page, screenshots and any relevant files.)

Fixed this a few hours ago.


Update es-ES.coffee

A translation for Thang has been proposed.

Thanks, @javatlacati

np! :smile:


Add header comment for autogenerated files

Is your feature request related to a problem? Please describe.
I work in code bases where it is not always clear that a file is autogenerated, so sometimes people try to change the autogenerated files, which cause lots of issues.
Describe the solution you'd like
Each autogenerated file should have a header block comment that states that it is an autogenerated file, maybe with metadata as well as to the date the file was generated, the version of graphql-code-generator that was used, etc.
Describe alternatives you've considered

Additional context

Hi @cktang88 ,
We decided not to add it automatically, but let the developers choose how and where to add it.
There is a a plugin called add you can use for that: https://graphql-code-generator.com/docs/plugins/add#examples


ERROR - "A configuration is pending." after failed configuration

In testing cookbooks which utilize the DSC cookbook I am finding that when a DSC configuration fails it appears to leave the target node in a pending state.  Have others seen this issue?
   1) PowershellCmdletException -  dsc_resource[DownloadSmbShareResource] (AcmeWebServer2::default line 27) had an e

rror: PowershellCmdletException: PowershellCmdletException
Powershell Cmdlet failed: A configuration is pending. Please run Update-DscConfiguration or
Start-DscConfiguration command with -Force parameter.
+ CategoryInfo          : ResourceExists: (root/Microsoft/...gurationManag
er:String) [], CimException
+ FullyQualifiedErrorId : MI RESULT 11
+ PSComputerName        : localhost
I propose this should be handled by the dsc cookbook. It should cleanup any pending DSC configuration work so the next attempt does not fail immediately

I am seeing this issue. What was the resolution?


Swagger/OpenAPI 2.0 issue on additional properties

When the addtionalProperties is used for specifying dynamic fields in the swagger 2.0 spec, it is encountering issue.

Hi @jprobinson attaching here a sample of the json file. (Putting it as a txt file)
addProp.txt
The dynamic fields i'm referring to is supposedly like this:
map<string,google.protobuf.Any> Attributes = 31;

Hi @jprobinson ,
I get the same error while trying to generate a proto file using command below:
openapi2proto -spec TS29510_Nnrf_NFManagement-Swagger20.json > TS29510_Nnrf_NFManagement-Swagger20.proto
The error I get is
error: failed to transpile: failed to compile OpenAPI spec to Protocol buffers: failed to compile definitions: failed to compile #/definition/NfInstancesResponse: failed to compile properties for NfInstancesResponse: failed to compile property _links: failed to compile object property _links: failed to get type [object]: failed to find type object
Attaching the input file with txt suffix.
TS29510_Nnrf_NFManagement-Swagger20.json.txt


types: mark options param of ImageryLayer.fromProviderAsync as optional

As the examples show, it is perfectly fine to not pass in the options object. However, in my typescript project, if I don't pass in the options, I get a type error. This is intended to fix that error.
This was brought up in https://community.cesium.com/t/error-when-initialize-imagerylayer-fromproviderasync-without-options/31665
I'm not used to typescript/jsdoc at all, so this might not actually be the right way to do this. If you tell me how to do this,
let me know and I can update it.

I just signed the CLA. I don't personally need myself in the contributors file. If YOU require me in there, I can add myself.


[git] Provide branches named equally to the tags of releases

When adding VMA as a submodule of a project one can't specify specific tag to stick to, whereas (e.g.) VulkanHeaders allows this via branches.
Currently git submodule update, called after git submodule add -b v3.0.1 https://github.com/GPUOpen-LibrariesAndSDKs/VulkanMemoryAllocator.git external/VulkanMemoryAllocator results in error "no branch named v3.0.1".
Currently checking out of a repository, that has VMA as a submodule, results in checking out HEAD of master branch of VMA's repository, if .gitmodules doesn't contain specific branch =  entry.

My understanding is that you want to use VMA as a submodule and pin its version to a specific release.
A submodule is tracked by the parent repo by in each commit saying which submodule commit should be checked out in the submodule. What the '-b' option does in the 'submodule add' command is adding a branch to track, so that you can easily check out the latest commit in that branch in the submodule when running 'submodule update' (by supplying '--remote'), instead of checking out the submodule commit specified in the parent repo commit. In your case, since you seem to want to stick to a certain release, I would simply add the submodule normally, cd down into the submodule, check out the release (tag) in the submodule, cd up to the parent repo, and add that submodule checkout to a commit in the parent repo.
That is of course more work than just adding a submodule with a branch, and checking out the last commit on that branch with the --remote option, but solves the problem without having to create branches for every release.
There are a few options in git for working with submodules which makes life easier which i highly recommend:

diff.submodule = log (so you get clearer container diffs when referenced submodule commits changed. Instead of SHAs, you will see a log with the changes between the commits).
status.submoduleSummary = true (so git status gets useful again when a referenced submodule commit changed).
submodule.recurse = true (automatically check out the commit(s) in the submodule(s) specified by the commit in the parent repo. Fail if there are local modifications in the submodule(s))


modification in password text section

Could you elaborate more? What text you will change?

i want to replace the hidden icon from it's current position to right-middle

Could you elaborate more? What text you will change?
i want to replace the hidden icon from it's current position to right-middle

Hey, i think it's been resolved by #123


⚠ Bofrost.de has degraded performance

In 99dbf9f, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1181 ms

Resolved: Bofrost.de performance has improved in 0e65275.