added support for parsing of space delimitted paths

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[x] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[ ] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[ ] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[ ] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
[ ] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

$watch's expression parse is intentionally limited - it's meant as a convenience API only, so anything more complex than dot-delimited paths should use a function getter instead.

@yyx990803 @HerringtonDarkholme just coming back to this, shouldn't watching include any valid javascript property path? expression parse just has to separate on dots, that's literally all, and it would support minuses, plusses, spaces, etc... ? if I can do data(){ return { "some property" : true } } I should be able to do { watch: { "some property" } }
come on?

if I can do data(){ return { "some property" : true } } I should be able to do { watch: { "some property" } }

What if you have this:
{
  data () {
    return {
      'some.property': true,
      some: {
        property: false
      }
    }
  },
  watch: {
    'some.property' (val) {
      // ?
    }
  }
}

if I can do data(){ return { "some property" : true } } I should be able to do { watch: { "some property" } }

What if you have this:
{
  data () {
    return {
      'some.property': true,
      some: {
        property: false
      }
    }
  },
  watch: {
    'some.property' (val) {
      // ?
    }
  }
}

yeah that's a good point but who in their right mind uses dots in their property names :')

do you have any idea to use the love streaming api in laravel php
On Thu, 5 Dec 2019 at 17:13, Nikolaj<EMAIL_ADDRESS>wrote:

if I can do data(){ return { "some property" : true } } I should be able
to do { watch: { "some property" } }
What if you have this:
{
data () {
return {
'some.property': true,
some: {
property: false
}
}
},
watch: {
'some.property' (val) {
// ?
}
}
}
yeah that's a good point but who in their right mind uses dots in their
property names :')
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/vuejs/vue/pull/7356?email_source=notifications&email_token=ANGDLPGAD64PMQOEZA5GZVTQXDV7HA5CNFSM4EKA5YOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGAQAUI#issuecomment-562102353,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANGDLPGN65BXUH2GJJKOZATQXDV7HANCNFSM4EKA5YOA
.

@Alinawaz-786 what?

Live streaming Videos
Anyone have any idea to live streaming videos api
any how to use this API

@Alinawaz-786 this is not the place to ask about that? I don't know sorry.


Add how to mock scoped modules to Manual Mocks doc

This adds info on how to mock scoped modules to the Manual Mocks docs page as it wasn't clear how this process would work.
(changelog modification pushing shortly once I have a PR ID)

Codecov Report

Merging #5638 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5638   +/-   ##
=======================================
  Coverage   62.28%   62.28%
=======================================
  Files         216      216
  Lines        7898     7898
  Branches        3        4    +1
=======================================
  Hits         4919     4919
  Misses       2978     2978
  Partials        1        1

Continue to review full report at Codecov.

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


any idea how to import this via pods in swift?

found it: @import iOS_Slide_Menu;

You could import in bridging header file

that's the only way


Add support for $key index

to run 2i tests mvn -Pitest clean verify   -DargLine="-Dcom.basho.riak.2i=true"

+1 Looks good. Test is logical and passes without problems


tools: parameterise config file paths

PR solving https://github.com/Half-Shot/matrix-appservice-discord/issues/507 (Hard-coded configuration paths in tools).
Changes compile, pass the lint and unit tests, but weren't tested manually.

Looks good!


Inconsistent crashes when switching scenes

Godot version
4.0.3.stable
System information
Arch Linux - RX 570 - Ryzen 5 3400G - Vulkan
Issue description
When running the project, if I change scenes, for some reason the game sometimes crashes. It can work fine 20 times in a row, and then crash out of the blue, but it always happens when switching scenes, at no other time.
Here is the piece of code I use to change scenes with a transition:
extends CanvasLayer

func change_scene_to_file(target: String) -> void:
	$AnimationPlayer.play_backwards("fade")
	await $AnimationPlayer.animation_finished
	get_tree().change_scene_to_file(target)
	$AnimationPlayer.play("fade")

func reload_current_scene() -> void:
	$AnimationPlayer.play_backwards("fade")
	await $AnimationPlayer.animation_finished
	get_tree().reload_current_scene()
	$AnimationPlayer.play("fade")

The problem is not me sending an incorrect scene file path. I have corroborated that the path I am giving when it crashes is proper.
I launched godot through the terminal to see what it says when it crashes. Here's the result:
Godot Engine v4.0.3.stable.arch_linux - https://godotengine.org
Vulkan API 1.3.246 - Forward+ - Using Vulkan Device #0: AMD - AMD Radeon RX 570 Series (RADV POLARIS10)

ERROR: FATAL: Index p_index = 0 is out of bounds (count = 0).
   at: operator[] (./core/templates/local_vector.h:155)

================================================================
handle_crash: Program crashed with signal 4
Engine version: Godot Engine v4.0.3.stable.arch_linux
Dumping the backtrace. Please include this when reporting the bug to the project developer.
[1] /usr/lib/libc.so.6(+0x39ab0) [0x7ff789562ab0] (??:0)
[2] /usr/bin/godot(+0x3da8c91) [0x5560e4399c91] (??:?)
[3] /usr/bin/godot(+0x3d09a4a) [0x5560e42faa4a] (??:?)
[4] /usr/bin/godot(+0x43da93c) [0x5560e49cb93c] (??:?)
[5] /usr/bin/godot(+0x4df1b3c) [0x5560e53e2b3c] (??:?)
[6] /usr/bin/godot(+0x4423665) [0x5560e4a14665] (??:?)
[7] /usr/bin/godot(+0x4a6fbc5) [0x5560e5060bc5] (??:?)
[8] /usr/bin/godot(+0x4425009) [0x5560e4a16009] (??:?)
[9] /usr/bin/godot(+0x420c1ec) [0x5560e47fd1ec] (??:?)
[10] /usr/bin/godot(+0x3d8c58b) [0x5560e437d58b] (??:?)
[11] /usr/bin/godot(+0x3d8c58b) [0x5560e437d58b] (??:?)
[12] /usr/bin/godot(+0x3d8c58b) [0x5560e437d58b] (??:?)
[13] /usr/bin/godot(+0x3d8c58b) [0x5560e437d58b] (??:?)
[14] /usr/bin/godot(+0x3de039a) [0x5560e43d139a] (??:?)
[15] /usr/bin/godot(+0x4230fee) [0x5560e4821fee] (??:?)
[16] /usr/bin/godot(+0x3de3db5) [0x5560e43d4db5] (??:?)
[17] /usr/bin/godot(+0x3338e86) [0x5560e3929e86] (??:?)
[18] /usr/bin/godot(+0x48ba7a8) [0x5560e4eab7a8] (??:?)
[19] /usr/bin/godot(+0x2e2d545) [0x5560e341e545] (??:?)
[20] /usr/bin/godot(+0x2e8a234) [0x5560e347b234] (??:?)
[21] /usr/bin/godot(+0x10fbd89) [0x5560e16ecd89] (??:?)
[22] /usr/bin/godot(+0x10a18b9) [0x5560e16928b9] (??:?)
[23] /usr/lib/libc.so.6(+0x23850) [0x7ff78954c850] (??:0)
[24] /usr/lib/libc.so.6(__libc_start_main+0x8a) [0x7ff78954c90a] (??:0)
[25] /usr/bin/godot(+0x10afda5) [0x5560e16a0da5] (??:?)
-- END OF BACKTRACE --
================================================================

Steps to reproduce

Change scenes with get_tree().change_scene_to_file()
With some "luck"... crash

Minimal reproduction project
N/A

Could you use the debug build from https://github.com/Calinou/godot-debug-builds/releases/tag/v4.0.3 to trigger this crash again and get a stacktrace with debug symbols?

This shows that it's crashing in the 3D physics collision solver, so that's helpful!
Check the line numbers where it's crashing in the 4.0.3-stable might help physics contributors guess what the issue could be. But of course the best to really identify the problem would be to have a clear way to reproduce it, though I know it's not necessarily easy to create a reproduction project for an intermittent crash.

There are physics forces being applied until the very last frame when changing scenes. Could that be causing a problem?

I've been investigating the issue, and I just can't find a reason for it. Maybe I could give someone here access to the repo, see if they can manage to debug it. Someone who has a better understanding of the inner workings of the engine.
I have found that going from level 1 to level 2, then restarting level 2 is a guaranteed crash. Going to level 3 is a guaranteed crash as well (but not directly loading the scene). Also, restarting level 3 is a guaranteed crash.
I know this is meaningless without access to the project, so if someone is willing to dig deeper into this, please let me know.

I tried to debug this. Apparently, (although I'm not certain), the issue is caused by accessing nodes belonging to the scene that's being unloaded.
For some reason, when I restart a level that crashes, an Area3D that's present on the level sends a signal that a body entered, then it checks a property of that body. It is then that the crash occurs.
Oddly enough, if I remove this Area from the level, that piece of code doesn't execute of course, but an _input function I have on an autoloaded UI controller does, and it crashes there.

I'll leave the previous comment merely to provide more information, but on closer inspection, it seems that it was consistently crashing at the same line because of time, not because that line was actually doing something wrong. If I remove that piece of code, then it consistently crashes on some other line of code. If I remove that one, then on another one, and so on.
All these lines of code have no relevance to each other. Some are input functions, others are functions called from signals, one of them is even a variable defined when the scene is read (@onready var ...).
There's no pattern to follow here. I can't find a fault in my code at all. This must have something to do with the engine itself.
I looked through the engine code too, I can't really understand it since I don't know how it all works and is tied together. It seems to be trying to detect a collision tho, from what I can gather, which is odd.
Removing the player, which is the only rigidbody in the scene, does not solve the issue.

I figured it out. It's grid maps. The thing that was causing the crash was a grid map. If I remove it, it works no problems.
Problem is, the grid map is the level. Guess I will drop them. People have warned me about them being limited and outdated, guess they were right.
At least we know now where to look to fix this.

I'm trying to cobble together a test project to reproduce the issue. I don't know what exactly my project has that makes it be, but I'm not finding a way to do it.
It's weird too, since all my levels are composed of the same nodes. A GridMap, an Environment, the Player, a custom node for the player camera, a Goal which is just a grid map with a specific shape and some particle effects, and maybe some extra things for that particular level.
All the other levels may crash, but Level 3 is the only one that consistently crashes. It does have a difference with other levels, it contains meshes directly imported from blender. Although at some point, level 1 did have meshes (but those where gltf).
However, removing the meshes does not solve the issue. Only removing the gridmap resolves it.

Further developments. The GridMap in Level 3 was slightly moved. It's position was Vector3(0.008, 0.00, -0.02). After moving it to Vector3(0, 0, 0) the crash doesn't occur anymore.
I then moved it back to it's previous coordinates by pressing Ctrl+Z and the crash happened again, consistently when I restarted the level.
I then put it at 0,0,0 once more, and the crash fixed, and upon setting the Vector3(0.008, 0.00, -0.02) coordinates by hand the crash also didn't happen again.
I am now unable to reproduce the consistent crashing.
Copying these values to the test project in hopes to get a reproduceable crash didn't make it crash.
GridMaps are messy.

Hey, don't let our lack of response discourage you. Things take time to get noticed sometimes. But you're doing a great job researching and identifying the issue, so thanks a lot!

@bugbountyguy I would like to thank you, I was having an issue where every time I switched scenes in the main menu of my game while running on a real Android device I would get a non-crashing error in the engine saying [can_process: “!is_inside_tree()” is true. Returned: false]
I applied the timer and now the issue is gone, thank you


Incorrect browser title when navigating between properties

When I am on the page for a particular property, like https://topology.pi-base.org/properties/P000011, the page correctly shows "P11: Regular" in the browser tab.  Then I click on one of the links for one of the properties that appear in the list of theorems on that page, for example Lindelöf (P18) as part of T30.  It correctly goes to https://topology.pi-base.org/properties/P000018, but the browser tab still shows "P11: Regular" instead of the expected "P18: Lindelöf".
And same kind of problem when navigating from a Theorem page to another Theorem page.  For example, from https://topology.pi-base.org/theorems/T000107 and clicking on T14.
Or navigating from spaces to spaces.  For example, from https://topology.pi-base.org/spaces/S000077 and clicking on the link for "Irrational numbers".

pretty sure I know what's happening here - I'll try to make a quick fix

Actually it wasn't what I thought - I thought we needed $: title = 'foo', but that was already there. I got it to work by using {#key title}<Title {title}/>{/key} but it'd be better to not require reinstantiating that component each time the route changes. Actually, I have one more idea...

Putting the #key in the title component itself did the trick.

Always nice when the bugs are easily squashed 😅


How to use the plugin?

I am kind of new to unreal, and I tried to follow the document and is confused. In the second step of "How to use the plugin" section, it told us to bind to "On Generate PCM data", there there's no such thing in my editor. A lot of funtions seems to be missing, not pretty sure something went wrong like I am using the wrong version of Unreal or the document is not updated so the name is wrong?

Are you using the GitHub or Marketplace version? GitHub has the latest version, but Marketplace hasn't been updated yet. Update on Marketplace with latest changes coming soon.
If you are using the GitHub version, then before deploying the AudioAnalysisTools plugin, you also need to deploy the RuntimeAudioImporter, as the plugin depends on it.

Hello there! I'm having this same issue as well. I've downloaded both RuntimeAudioImporter and AudioAnalysisTools from GitHub. Both are properly deployed, but OnGeneratePCMData is missing in my editor as well.

@cjlovelace Once the sound wave is imported, you will be able to bind to this delegate from the imported sound wave.

This delegate can only be bound from an Imported Sound Wave object.

Here is the detailed information on how to use the plugin: https://github.com/gtreshchev/AudioAnalysisTools/wiki/How-to-use-the-plugin


if can? GainControl  provide  kAgcModeAdaptiveAnalog    mode?

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

I would like to use webRTC 's audio denoise funtion(Auto Gain Control). to do this,There are thress mode:kAgcModeAdaptiveAnalog\kAgcModeAdaptiveDigital\kAgcModeFixedDigital. but in the webrtc-java,i can't find the  kAgcModeAdaptiveAnalog mode.


[DUOS-1891][risk=no] Implement SupportRequestType enum

Minor to change to #1651
Creates enum for type field of a support ticket, as detailed https://developer.zendesk.com/api-reference/ticketing/tickets/ticket-requests/#json-format

Have you read CONTRIBUTING.md lately? If not, do that first.

Label PR with a Jira ticket number and include a link to the ticket
Label PR with a security risk modifier [no, low, medium, high]
PR describes scope of changes
Get a minimum of one thumbs worth of review, preferably two if enough team members are available
Get PO sign-off for all non-trivial UI or workflow changes
Verify all tests go green
Test this change deployed correctly and works on dev environment after deployment

How come this PR did not have any checks run against ti?

@rushtong Same as the other one View details button was playing hide and seek.


Allow capability to patch OSAD submodule files.

There may be times where it is desirable for rpc-openstack to change the SHA of the os-ansible-deployment submodule, but not necessarily match a tag. In order to pull in improvements between tags with minimal code disruption, we should provide some sort of patching mechanism.

Closing this, as the change is merged.


Add Lint/DeprecatedAttr cop

Is your feature request related to a problem? Please describe.
attr with a second boolean parameter has been marked as deprecated since Ruby 2.5:
https://github.com/ruby/ruby/commit/7b19e6f3fdf8b0238752cb1561dfe42a283f5308#diff-1b7f91891e9afd5c2c43fea4d989d0c4f8495cca99ac3ff5f825a7b30c435055R2309-R2310
https://ruby-doc.org/core-2.5.0/Module.html#method-i-attr
I would like a rule to flag such deprecated uses of attr. It could also autocorrect to their non-deprecated equivalents.
Describe the solution you'd like
# bad
attr :foo, true
attr :bar, false

# good
attr_accessor :foo
attr_reader :bar
attr :baz

Describe alternatives you've considered
Optionally, we could also encourage attr_reader :baz over attr :baz as a stylistic opinion, but that doesn't relate to deprecation, and thus feels out of scope here.
Additional context
N/A

It will extend Lint/DeprecatedClassMethods instead of new cop. However, it seems that an extension that can notice the arguments is required.

Thanks @koic ! 😍


Online demo now available

An online demo of this is now available
https://huggingface.co/spaces/jiayong/Dreamoving

How to use the demo to a server? don't know how to start it

It seems it only upload the video and image instead of actually running stuffs, maybe have to wait for a long time

@dreamoving @ArcherFMY et al. The button "Cartoon Video Generation" cannot be clicked, so the huggingface demo is not working. Any help with this? Thx & rgds,

@dreamoving @ArcherFMY et al. The button "Cartoon Video Generation" cannot be clicked, so the huggingface demo is not working. Any help with this? Thx & rgds,

We are working on it~

@dreamoving @ArcherFMY et al. The button "Cartoon Video Generation" cannot be clicked, so the huggingface demo is not working. Any help with this? Thx & rgds,

We are working on it~

it's also not currently returning results on Huggingface even with default photo & video


ci: gh action to validate pr title

Fixes #423
What

adds a GH action

As @cristiam86 mentioned, we might be able to then use https://github.com/marketplace/actions/changelog-from-conventional-commits to create release notes
Why
To adhere with our contributing standard
Testing done

pointing the PR to main doesn't trigger the workflow (needed for release PRs which are not following conventional commits)
action has run correctly on this PR

Decisions made

I've used https://github.com/ytanikin/PRConventionalCommits since it seemed to be the most maintained and with nice features
I've removed the pre-commit hook to check conventional commits: since we are squash merging we are only interested in the PR title check

Checks

[ ] I have tested this code
[ ] I have reviewed my own PR
[ ] I have created an issue for this PR
[ ] I have set the PR name to the issue name

@AgustinRamiroDiaz would it be possible also to include these two checks?

It complies with the PR template
It should be pointing to staging

@cristiam86 sorry, I didn't read carefully the issue description

I've added another GH action for "It should be pointing to staging"
https://github.com/yeagerai/genlayer-simulator/issues/423#issuecomment-2317490753


Support for external devices when using docker compose

When following Edgehog in 5 minutes, Edgehog is set up using Docker Compose.
With this method, connecting devices simulated on the same host machine running the Edgehog instance is possible.
However, some Edgehog devices are intended to run on external hardware.
This should be supported by the Docker compose setup to facilitate first-time users in trying out Edgehog.

Closing as addressed by #759


"purpose": "Create WalletAccount" - "type": "MethodUnavailable". We use our own API and deploy the contracts. Method error

I used EVM Paris and deployed the contracts correctly, and I can read them through the same API. However, when interacting, the following error appears:

I've already deployed the same contracts to other EVM chains and it worked. Only Besu is still giving me this error. We use w3.py and I believe this is not the problem.

Hi @macfarla Thank you for your answer!
We are using eth_sendRawTransaction.
We are debugging and it returned this error:

The Invalid TraceId warning can be ignored as per https://github.com/hyperledger/besu/issues/6440 (there's also a workaround there to silence that specific log warning)

Can this issue be closed with the ignore / suggestion above?

@ Afonsodalvi did you find out where the "Method not found" is coming from?

Closing this, if you are still having an issue feel free to reopen with more details


cannot import 'LightHander' or 'ColorLightHandler' for typings

Hello,
First of all, thank @mihai-dinculescu for your work in this library.
So, while all we need is to import the ApiClient to use this library, I am trying to add Type support for intellisense in VSCode so I am importing:
from tapo import ApiClient, LightHandler, ColorLightHandler

and my editor now looks like this:

However, this happens when I run the code:

This is the list of all my imports, I hope this issue isn't related to shadowing or something.
import asyncio
import colorsys
import sys
import TouchPortalAPI as TP
import yaml
from functools import wraps
from typing import Awaitable, Dict, Optional, Tuple, TypeVar, Union
from argparse import ArgumentParser
from TouchPortalAPI.logger import Logger
from tapo import ApiClient, LightHandler, ColorLightHandler

So my questions is, were LightHandler and ColorLightHandler not intented to be imported like this? If so, how to import them?
Thank you kindly for your help pointing me into the right direction.

They were supposed to be exported.
I've just merged a fix to main.
Thank you for reporting this issue!

Thank you for fixing this issue. Has this push to main fixed the package dependency? I mean, to take advantage of this fix, shall I only reinstall the dependency with pip?


Automatic creation of rabbitmq virtual hosts

Using MassTransit <IP_ADDRESS>2 with Rabbit mq.
MassTransit is able to dynamically create and configure exchanges and queues. But it is unable to create virtual hosts in rabbit mq - despite the information being available in the connectionstring.
All I'm met with is the exception
System.Net.Sockets.SocketException: 'An existing connection was forcibly closed by the remote host'

Is it a feature you want to support? Or should I create my own tool that fetches the exchanges and creates the missing ones?

Is this a blocking call? Should this method not be async and this call awaited?
var result = client.PutAsync(url, content).Result;
So its more like:
var result = await client.PutAsync(url, content);


Verifies lang exists before assigning it

What's this PR do?
Verifies the system language is actually installed before giving it to a user. Default to global en if it doesnt exist.
How should this be manually tested?
Create a new user and spoof the country code as GB or CA.
What are the relevant tickets?
Fixes #5256

:+1:


Support for video element

Is there any planned support for a video element? should it be/does existst a vlc/ffmpeg native component for desktop? Or the correct way should be throu a webview?

I've done some progress a long time ago, I can push what I got but is half-cooked and needs further work.

can it be a plugin?
On Sun, 13 Nov 2016 at 14:29, Víctor Martínez<EMAIL_ADDRESS>wrote:

I've done some progress a long time ago, I can push what I got but is
half-cooked and needs further work.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ptmt/react-native-macos/issues/96#issuecomment-260180431,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA9SU2df9iJV4qn0aPbrUH0O5dhY2QQGks5q9vShgaJpZM4JD9tb
.

Closing this, it sounds better as a plugin like react-native-video.


As a user, I want to export contacts

... so that I can easily integrate with existing data.

Close issue as this has been done.


[FR] Domain blacklist

We have data-domains [1] for whitelisting specific domains and ignoring everything else.
I wanted to black-list only my localhost domain, and thus thought that perhaps having a blacklist would be useful?
Thank you for the great tool btw!
[1] https://umami.is/docs/tracker-config

@kiprasmel You can do that by setting the IGNORE_IP environment variable, https://umami.is/docs/environment-variables

@kiprasmel You can do that by setting the IGNORE_IP environment variable, https://umami.is/docs/environment-variables

Oh that's great, thanks. I guess we can close this.
I now realize that anyone could just take your website's tracking script and add it to theirs and spam your analytics with useless data -- is that right? Might be an issue 😅

Oh that's great, thanks. I guess we can close this.
I now realize that anyone could just take your website's tracking script and add it to theirs and spam your analytics with useless data -- is that right? Might be an issue 😅


Removing all the azure things!

What's the motivation for removing Azure completely?

@nrb Mostly because it's too hard to support it. I have not worked on this one, but it seems like the Azure API and Go wrapper are not working very well for now.
Here's a comment by Kris.

Thanks @xmudrii.  Would it be more likely to stay if running on Azure had some measure of corporate support helping to make sure it worked?
(Please note I'm not committing here, but gathering information)

@nrb I reached this conclusion while work at Microsoft. So I have little faith that 1) corporate backing would change anything or even if it did 2) probably won’t happen anytime soon.
The problem is with virtual machine scale sets. Those just flat out don’t work. Or at least they didn’t for the few months I tried to hack on them.
Note we COULD create an implementation calling the virtual machine API directly as the machine API will solve the same problem virtual machine scale sets was supposed to solve.
2nd note that the VMSS problem was just the straw that broke the camels back. The ARM API simply fails something on the order of 70% of requests - so we are just biting off even more work that terraform has already solved.
Happy to talk more tomorrow if you’d like!

@xmudrii Thanks for the link to the comment, I had missed that.
@kris-nova I appreciate the context! I think all of that is sound and don't really have an argument for keeping it in. I think my one suggestion is to add some of that context to the commit message here for posterity.


layout_calculation in 3D?

Hi Vincent, this visualization looks really cool!
I try to visualize my model in 3D using your algorithm, I changed num_dim in network_graph.py to 3 and print out the positions in layout_calculation.py, but the vertices tend to collapse into 2D (the z-axis value tends to be very similar for all vertices as the visualization unfold)
Do you have any suggestions on how to solve it? Thank you in advance!

Thanks!
To be honest, I did not really test the layout algorithm for more than two dimensions. But in theory it should work. How the dimensions are used depends on the connection patterns in your graph. Here is my intuition: A fully connected neural network has no incentive to use the third dimension. It is more efficient to just spread out in a plane. For a 1D convolutional network, the channels might spread a bit into the third dimension. A 2D convolutional network should construct something similar to the diagrams we often see, with the layers stacked on top of each other.
Maybe this helps, else I can try to see if there is something conceptually wrong with the code for higher dimensions.

Thank you! yeah, maybe that's the reason:

A fully connected neural network has no incentive to use the third dimension
...
For a 1D convolutional network, the channels might spread a bit into the third dimension.

I tested it using a simple fully connected neural net and the resnet18_1d provided in your example_script.py, okay I will think about it, thanks!


Adds extra confirmation to payinvoice and sendpayment, can be bypassed by force.

Resolves #1455.
Also looks like gofmt wasn't run on this file since the last commit that checked-in closedchannels, so I ran it too.

Awesome! LGTM. Could you restructure your commits to better follow the contribution guidelines?

Done, renamed commits like "lncli:...", less than 50 chars descriptions and also squashed the 1-2 liner changes that I made during the review.

Thanks for the review. I've squashed into a single commit.

Thanks @rawtxapp, this is a feature i've been wanting for a while! PR LGTM, will need a rebase before merging. May not be merged immediately, so I'd hold off on that until we figure out a merge order

Thanks for the review, let me know whenever you guys want to merge it and I'll rebase.

Awesome, sounds good :)

Changed to using ctx.Bool and rebased on master.


gcc not found

I am using this Docker image in GitLab CI.
At some point I am installing some pre-commit hooks, in particular black which requires frozenlist as one
of its dependencies.
The pip installation fails because it seems that gcc is missing,
Failed to build frozenlist multidict
stderr:
      error: subprocess-exited-with-error

      × Building wheel for frozenlist (pyproject.toml) did not run successfully.
      │ exit code: 1
      ╰─> [36 lines of output]
          *********************
          * Accelerated build *
          *********************
          running bdist_wheel
          running build
          running build_py
          creating build
          creating build/lib.linux-x86_64-cpython-312
          creating build/lib.linux-x86_64-cpython-312/frozenlist
          copying frozenlist/__init__.py -> build/lib.linux-x86_64-cpython-312/frozenlist
          running egg_info
          writing frozenlist.egg-info/PKG-INFO
          writing dependency_links to frozenlist.egg-info/dependency_links.txt
          writing top-level names to frozenlist.egg-info/top_level.txt
          reading manifest file 'frozenlist.egg-info/SOURCES.txt'
          reading manifest template 'MANIFEST.in'
          warning: no previously-included files matching '*.pyc' found anywhere in distribution
          warning: no previously-included files matching '*.pyd' found anywhere in distribution
          warning: no previously-included files matching '*.so' found anywhere in distribution
          warning: no previously-included files matching '*.lib' found anywhere in distribution
          warning: no previously-included files matching '*.dll' found anywhere in distribution
          warning: no previously-included files matching '*.a' found anywhere in distribution
          warning: no previously-included files matching '*.obj' found anywhere in distribution
          warning: no previously-included files found matching 'frozenlist/*.html'
          no previously-included directories found matching 'docs/_build'
          adding license file 'LICENSE'
          writing manifest file 'frozenlist.egg-info/SOURCES.txt'
          copying frozenlist/__init__.pyi -> build/lib.linux-x86_64-cpython-312/frozenlist
          copying frozenlist/_frozenlist.pyx -> build/lib.linux-x86_64-cpython-312/frozenlist
          copying frozenlist/py.typed -> build/lib.linux-x86_64-cpython-312/frozenlist
          running build_ext
          building 'frozenlist._frozenlist' extension
          creating build/temp.linux-x86_64-cpython-312
          creating build/temp.linux-x86_64-cpython-312/frozenlist
          gcc -pthread -B /opt/conda/compiler_compat -fno-strict-overflow -DNDEBUG -O2 -Wall -fPIC -O2 -isystem /opt/conda/include -fPIC -O2 -isystem /opt/conda/include -fPIC -I/home/mambauser/.cache/pre-commit/repohsebl7b0/py_env-python3/include -I/opt/conda/include/python3.12 -c frozenlist/_frozenlist.c -o build/temp.linux-x86_64-cpython-312/frozenlist/_frozenlist.o
          error: command 'gcc' failed: No such file or directory
          [end of output]

      note: This error originates from a subprocess, and is likely not a problem with pip.
      ERROR: Failed building wheel for frozenlist
      error: subprocess-exited-with-error

This might be an issue of frozenlist, since the version that is tried to be installed was released 5 days ago, but gcc seems something more basic, so I thought it might be an issue of the docker image.

If you want gcc, then you'll need to install it.  It is available from the conda-forge channel.


TypeError: UbufP2PCommOverlap(): incompatible function arguments.

Hello,
When I configured --sequence-parallel and --tp-comm-overlap and started the training. It shows below information:
TypeError: UbufP2PCommOverlap(): incompatible function arguments. The following argument types are supported:
1. () -> None
Invoked with: tensor([[0., 0., 0.,  ..., 0., 0., 0.],
[0., 0., 0.,  ..., 0., 0., 0.],
[0., 0., 0.,  ..., 0., 0., 0.],
...,
[0., 0., 0.,  ..., 0., 0., 0.],
[0., 0., 0.,  ..., 0., 0., 0.],
[0., 0., 0.,  ..., 0., 0., 0.]], device='cuda:7', dtype=torch.bfloat16), 7, 2, 16, 2, 0, 0, 3, 0, 0, tensor([])
How to fix it? Thanks.

Hi @sallyjunjun -- TE does not maintain its own models/applications so it does not parse any command-line options like --sequence-parallel or --tp-comm-overlap. Are you using TE's TP comm overlap via another package like NeMo or a custom application? If so, I would recommend opening this issue with the developers of that package/application instead to make sure their code is invoking TE API correctly.
On a related note, UbufP2PCommOverlap is deprecated API so the application you're trying to run must be using an older version of TE. Please check in with that application's developers about the possibility of updating to the latest release.

Thank you for your suggestion. I will try to upgrade TransformerEngine first.

when I install te with command ' pip install git+https://github.com/NVIDIA/TransformerEngine.git@stable ', I met the following error:
Collecting git+https://github.com/NVIDIA/TransformerEngine.git@stable
Cloning https://github.com/NVIDIA/TransformerEngine.git (to revision stable) to /tmp/pip-req-build-tzb04kj9
Running command git clone --filter=blob:none --quiet https://github.com/NVIDIA/TransformerEngine.git /tmp/pip-req-build-tzb04kj9
Running command git checkout -b stable --track origin/stable
Switched to a new branch 'stable'
branch 'stable' set up to track 'origin/stable'.
Resolved https://github.com/NVIDIA/TransformerEngine.git to commit e5edd6cc3d5a868bb3fe4e81088d22aab505a30d
Running command git submodule update --init --recursive -q
Preparing metadata (setup.py): started
Preparing metadata (setup.py): finished with status 'error'
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [8 lines of output]
Traceback (most recent call last):
File "", line 2, in
File "", line 34, in
File "/tmp/pip-req-build-tzb04kj9/setup.py", line 37, in
from torch.utils.cpp_extension import BuildExtension
File "/mnt/petrelfs/geruijun/miniconda3-new/envs/llm-cuda12.2-nemo/lib/python3.10/site-packages/torch/init.py", line 368, in
from torch._C import *  # noqa: F403
ImportError: /mnt/petrelfs/geruijun/miniconda3-new/envs/llm-cuda12.2-nemo/lib/python3.10/site-packages/torch/lib/../../nvidia/cusparse/lib/libcusparse.so.12: symbol __nvJitLinkComplete_12_4, version libnvJitLink.so.12 not defined in file libnvJitLink.so.12 with link time reference
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.
Which version of cuda and pytorch should I use?

when I upgrade TransformerEngine, I met the problem ModuleNotFoundError: No module named 'torch'. But I have installed torch indeed. Do you know how to fix it?
The detailed information is as follows:
Processing /mnt/hwfile/geruijun/code/Megatron-mwiacx/TransformerEngine
Installing build dependencies: started
Installing build dependencies: finished with status 'done'
Getting requirements to build wheel: started
Getting requirements to build wheel: finished with status 'error'
error: subprocess-exited-with-error
× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> [17 lines of output]
Traceback (most recent call last):
File "/mnt/petrelfs/geruijun/miniconda3-new/envs/llm-cuda12.2-nemo/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in
main()
File "/mnt/petrelfs/geruijun/miniconda3-new/envs/llm-cuda12.2-nemo/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
json_out['return_val'] = hook(**hook_input['kwargs'])
File "/mnt/petrelfs/geruijun/miniconda3-new/envs/llm-cuda12.2-nemo/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/in_process/in_process.py", line 118, in get_requires_for_build_wheel
return hook(config_settings)
File "/tmp/pip-build-env-ncz8mpb/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 334, in get_requires_for_build_wheel
return self.get_build_requires(config_settings, requirements=[])
File "/tmp/pip-build-env-ncz8mpb/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 304, in get_build_requires
self.run_setup()
File "/tmp/pip-build-env-ncz8mpb/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 522, in run_setup
super().run_setup(setup_script=setup_script)
File "/tmp/pip-build-env-ncz8mpb/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 320, in run_setup
exec(code, locals())
File "", line 37, in
ModuleNotFoundError: No module named 'torch'
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error
× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.
note: This error originates from a subprocess, and is likely not a problem with pip.
srun: error: HOST-10-140-60-3: task 0: Exited with exit code 1
pip show torch
Name: torch
Version: 2.1.0+cu121
Summary: Tensors and Dynamic neural networks in Python with strong GPU acceleration
Home-page: https://pytorch.org/
Author: PyTorch Team
Author-email<EMAIL_ADDRESS>License: BSD-3
Location: /mnt/hwfile/geruijun/miniconda3-new/envs/llm-cuda12.2-nemo/lib/python3.10/site-packages
Requires: filelock, fsspec, jinja2, networkx, sympy, triton, typing-extensions
Required-by: accelerate, accelerated-scan, causal-conv1d, deepspeed, flash-attn, peft, pytorch-lightning, sentence-transformers, timm, torchaudio, torchmetrics, torchvision, transformer-engine

This sounds like a problem with your conda environment. You may be trying to install Transformer Layer in a different environment than where you installed PyTorch.
Also, if you're interacting with Transformer Engine through another package like NeMo or Megatron-LM, I would recommend updating that to a newer version (following their instructions, not ours) and letting it pull the correct Transformer Engine version it depends on. Otherwise, you will likely run into API mismatches between them.

Ok, I see. Thank you for your help.


Add password to AddDebitCardPage in NewDot

Details
Adds a password requirement on the AddDebitCardPage.
We are ignoring the forgot password flow for now, as we still haven't landed on a solution.
Fixed Issues
$ https://github.com/Expensify/Expensify/issues/188051
Tests

Pull the changes in Auth and Web-Secure
Log into NewDot and navigate to Settings > Payments > Add payment method > Debit card
Try to submit an empty form and verify that you see an error for the password input.
Now fill in all the necessary information (enter 4242 4242 4242 4242 for credit card number), but use a wrong password.
Submit the form and verify that you see the following message: Incorrect Expensify password entered.
Now enter your correct Expensify password and submit the form again.
Verify that the card was added.

QA Steps
Steps above.
Tested On

[X] Web
[ ] Mobile Web
[X] Desktop
[X] iOS
[x] Android

Screenshots

Web
https://user-images.githubusercontent.com/22219519/148591820-5ca7b892-a9bc-413c-b4b7-f7e32a8cacf1.mov
Mobile Web

Desktop
https://user-images.githubusercontent.com/22219519/148591833-02f90fd7-f3d0-49d3-8623-a01e88421b1c.mov
iOS
https://user-images.githubusercontent.com/22219519/148593278-19b218aa-1d7b-4193-ae11-fa9a08b4b5f1.mp4
Android
https://user-images.githubusercontent.com/22219519/148596836-7155b605-69b5-454a-ae62-1ccdd55a45be.mov

This is ready for review! Since it needs to be deployed with the Auth and Web-Secure changes, I'll keep it on hold until all 3 PRs are approved!

Bumping for review!

Thanks for the review @MonilBhavsar! No longer on hold, merging.

@luacmartins looks like this was merged without passing tests. Please add a note explaining why this was done and remove the Emergency label if this is not an emergency.

: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.

All tests passed before merging. Removing Emergency label.

🚀 Deployed to staging by @luacmartins in version: 1.1.30-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @chiragsalian in version: 1.1.31-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


CSS classes are assigned at multiple levels, not just to outer container

Hi,
Love your widget, it's in my project bootstrap; it's great for widgetized homepage layouts in particular.
I'm unsure whether this has anything specific to do with the Genesis Framework that I often use, and the tags that it produces, but I find that the $cssClass values are added at multiple levels, ie:

Fishing
Trips

In this case fishing-trips link-block one-third appear twice, which is awkward as one-third is a relatively generic grid class that sets a percentage width.
I've forked a copy of the repo to add an extra param to optionally apply the styles only to the first wrapper tag:
https://github.com/hughc/enhanced-text-widget/commit/013cf73122802fd71af7fcb6886d50c8918aaf2f
Let me know if you can see the issue and would like a pull request.

Hey! Sorry for not responding sooner. This is most likely something with your theme, it does not do this on the WordPress default theme with v4.8.
See "test-class" here:

OK, my apologies; I work with subthemes of Genesis all the time, and was unaware that it's markup is non-standard. I can be more specific now.
The extra CSS class feature is not compatible with any Genesis theme, as the $before_widget code for all Genesis widgets includes an extra
, and your str_replace code will add any nominated classes to both
s in the $before_widget , ie:

Not sure if there's a clean way to resolve this, but at the end of the day, there's an underlying assumption in the use of str_replace that there's a single element wrapping the widget content, but Wordpress does not prohibit the use of multiple divs.
I'm using a private fork of the plugin in client projects now, disconnected from wordpress.org updates, but it would be useful to either update the docs to mention the incompatibility or else consider the PR  that you closed. My proposed code could be simplified to remove the checkbox and just use the str_replace_first function instead of str_replace, to improve compatibility.


Implementation difference to Detectron in the mask head.

In Detectron, the predicted mask is padded before resize. While in this repo, no padding before resize (see this line and this line), why?

This difference is caused by different mask preprocessings when computing the training target. We use binary masks and crop the region on the resized masks, while Detectron uses polygons and transforms it to target masks directly. Training target and inference pipeline need to be unified.

@hellock Could you explain more specifically why there is a difference between the two different mask formats? Why is padding needed in polygons format and not needed in binary mask format?


Propagate the underlying error of ApplyError::Can'tPay

https://github.com/paritytech/substrate/blob/51ec6283ff1fb3a1fd9e1844d83229e40e387a63/srml/executive/src/lib.rs#L275-L276
Related #2880

+1
This silent error took a lot of time from me.


graphql client: How to pass authorization header in mutation and query request

Is there a way to pass the authorization header in mutation and query request, or a way to override httpLink default headers when calling the graphql client.
For now what I did is I created a function that will build the client.
getGraphQLClient([String? accessToken]) {
  final httpLink = HttpLink(
    ConfigReader.getAppGraphqlEndpoint(),
    defaultHeaders: {'Authorization': 'Bearer $accessToken'},
  );

  return GraphQLClient(cache: GraphQLCache(), link: httpLink);
}

In my repository I called it every time when I am making a request.
Future<dynamic> currentUser(String accessToken) async {
    final client = getGraphQLClient(accessToken);

    final options = QueryOptions(
      document: gql(r'''
        query {
          currentUser
        }
      '''),
    );

    final QueryResult result = await client.query(options);

    if (result.hasException) {
      throw Failure(result.exception?.graphqlErrors.first.message);
    }

    return result.data!['currentUser'];
  }

@campanagerald take a look at this link
https://github.com/felangel/fresh/tree/master/packages/fresh_graphql

Rebuilding the client is the way.

@budde377
If we rebuild the client every time, won't we lost our cache?

That depend on what cache you use and how you provide it. The cache is an argument to the client so you can just persist it if you'd like.


Remove rmMode from Location object

The Swagger docs currently have a boolean rmMode attribute. According to @peterdudfield this shouldn't be there and is a SQLAlchemy artefact.

Thanks @TheLiamGuy for pointing this out

This doesnt seem to be in the swagger docs anymore, so ill close this


CUPS can hold for authentication jobs on printers without auth-info-required attribute

In my specific instance, this happens when an SMB printer that requires authentication does not have the auth-info-required attribute. When trying to print with it, the backend (smbspool) will exit with CUPS_BACKEND_AUTH_REQUIRED, leading to the following snippet of code, which holds the job for authentication: https://github.com/OpenPrinting/cups/blob/3a1a81d7b882c06451a39a19a87aca7ec44742e6/scheduler/job.c#L3458-L3474
Since the printer does not have an auth-info-required attribute specified, though, the UIs in gnome-control-center andsystem-config-printer (but I suspect other interfaces as well) break, because they cannot provide an authentication prompt given that CUPS replies that no authentication is needed. The job can, at that point, only be cancelled manually despite the "Job held for authentication" message.
Given that receiving a  CUPS_BACKEND_AUTH_REQUIRED exit code from the backend while having no auth-info-required on the printer is a clear sign of misconfiguration, I suspect that, in this situation, an error should be either printed in the logs or shown to the user. At the moment, no clear indication of a misconfiguration seems to be present.

@EliaGeretto So this sounds like a Samba bug - the backend should be sending the appropriate message to stderr when authentication is needed:
ATTR: auth-info-required=username,password
ATTR: auth-info-required=negotiate

In the meantime you can use the lpadmin command to tell GNOME what kind of authentication is required:
lpadmin -p PRINTER -o auth-info-required=username,password  (for username + password)
lpadmin -p PRINTER -o auth-info-required=negotiate          (for Kerberos)

So, just to clarify, you are supposed to be able to add a printer without specifying if it needs authentication and then the backend is responsible to tell CUPS what authentication info is needed? If that is the case, then I should report the bug to samba, indeed.
As for the workaround, I edited manually /etc/cups/printers.conf and added the appropriate AuthInfoRequired line there.

I have just tried to execute smbspool, is this the correct behavior?
AUTH_INFO_REQUIRED=none smbspool 'smb://printing.vu.nl/VU_Ricoh_Color_Printer' test test 1 1 /tmp/test.txt
ERROR: Session setup failed: NT_STATUS_ACCESS_DENIED
DEBUG: get_exit_code(nt_status=NT_STATUS_ACCESS_DENIED [c0000022])
ATTR: auth-info-required=none
DEBUG: Unable to connect to CIFS host: NT_STATUS_ACCESS_DENIED%

@EliaGeretto Yes, it is a Samba bug.

I gave another look at the logs and what is shown in the snippet above is indeed the problem. Since no auth-info-required is specified when adding the printer, CUPS will invoke smbspool with AUTH_INFO_REQUIRED=none. smbspool exits with the correct code, CUPS_BACKEND_AUTH_REQUIRED, but echoes back the auth-info-required attribute that was passed in instead of specifying which authentication information is required. I will report this to samba then, let me know if I made a mistake in my summary.
Regarding this issue, I still think it would be nice to check if a backend returning CUPS_BACKEND_AUTH_REQUIRED actually specifies which information is required for authentication. A nice error log reporting a possible bug in the backend would have made this easier to debug. In addition, maybe marking the job as failed in such a situation could be a better than holding it indefinitely. At this point, though, this is more of a feature request than a bug report. Let me know what you think.

I have reported the samba bug. See: https://bugzilla.samba.org/show_bug.cgi?id=15057

@EliaGeretto Another option is to force username,password if auth-info-required is none...

[master f4e8437dd] Set default auth-info-required='username,password' if a backend doesn't set an authentication method (Issue #373)

@EliaGeretto In those cases, the fix I made wouldn't be any worse - the default (none) would be replaced by username,password if the backend doesn't set one. But if you need Kerberos (negotiate) you just need to use lpadmin to change the default (lpadmin -p PRINTER -o auth-info-required=negotiate)

Fair, it's true that setting it to username,password by default when there is a bug in the backend will at least make it work in some cases. Still, when some other kind of authentication is required, users will now be shown a username and password authentication dialog that is unrelated to what they need. I don't know, I would have probably gone with logging an error as well, but you have definitely more experience here and this fixes my problem :+1:


org.sparkle-project.Downloader crashing only while running in Xcode in os_unfair_lock

Thanks for the great example project and documentation!
We have an app that uses some of this code, but running into this issue when running in Xcode:
https://github.com/sparkle-project/Sparkle/issues/1805
Any chance this you saw this issue as well?

ok, this was not an issue after i stopped using the downloader service.

Hey, I recently pushed some updates to this example code base, which are not yet reflected in the README.md file. I added conformance to the ObservableObjectProtocol, so that the CommandMenu can disabled the "update"-Button based on the SPUUpdater.canCheckForUpdates - property.


Missing Duration support

What abbout support for Durations?
I would need something like FmtDuration(d time.Duration) to get "20 days 10 hours 5 minutes 3 seconds"

Hey @Niondir that is definitely something I would like to add in some time in the future :)
I definitely want to get the data being generated from the new JSON CLDR files rather than the old XML to make additions a little easier first though and am finding it very hard to find time to get to it though.

Any news here?
I just spend some time looking into the CLDR files. What's wrong with the XML?
As of durations I found them in /common/main/*.xml <units> in three "length" types long, short and narrow. If we add the duration "units" we could even add all units in general.
Example from en.xml:
Long:
<unit type="duration-week">
	<displayName>weeks</displayName>
	<unitPattern count="one">{0} week</unitPattern>
	<unitPattern count="other">{0} weeks</unitPattern>
	<perUnitPattern>{0} per week</perUnitPattern>
</unit>

Short:
<unit type="duration-week">
	<displayName>weeks</displayName>
	<unitPattern count="one">{0} wk</unitPattern>
	<unitPattern count="other">{0} wks</unitPattern>
	<perUnitPattern>{0}/w</perUnitPattern>
</unit>

Narrow:
<unit type="duration-week">
	<displayName>wk</displayName>
	<unitPattern count="one">{0}w</unitPattern>
	<unitPattern count="other">{0}w</unitPattern>
</unit>

Hey @nikolay-turpitko nothing is wrong with the XML per se, I want to add this along with individual format methods for seconds, minutes, hours etc in case someone only wanted the total in say minutes only.
As I said, I just am having trouble finding time getting to it, PR's are welcome :)

@joeybloggs @Niondir
Guys, I'd like to help, but I have not so much free time as well.
Also, I'm not currently use this project, so not quite understand what kind of help do you need here.
If you want programming/testing/translation assistance, please be more specific and describe task in a plain English as concrete and detailed as you can. I can invest only so much of my free hours after my daily job, so it's difficult to me to investigate, digest, guess stuff and go into prolonged discussions. So, if you really need my help, please give me some context. What's going on here, what's the current state of affairs, what's need to be done, what do you think is appropriate to accomplish this task (your design view in plain English, preferably even step-by-step instruction), some test cases and references to required documentation, etc.

Thanks I will try to create a project plan in the coming weeks

BTW, (not directly related to the topic of this issue) I've stumped on some interesting related projects. Probably you are aware of them, but just in case:
https://godoc.org/golang.org/x/text/message
https://godoc.org/golang.org/x/text/feature/plural
I haven't tried them myself, currently I do not involved into any multilingual projects. Take a look, may be there is something useful.


fix: format root files only

Fixes #98
We should use getRootFileNames instead of getSourceFiles when retrieving a list of files to format; the latter gets every files in the dependency graph constructed from files included in tsconfig.json.

Codecov Report

Merging #99 into master will decrease coverage by 3.66%.
The diff coverage is 93.75%.

@@            Coverage Diff             @@
##           master      #99      +/-   ##
==========================================
- Coverage   99.63%   95.96%   -3.67%
==========================================
  Files           8       10       +2
  Lines         272      322      +50
  Branches       12       19       +7
==========================================
+ Hits          271      309      +38
- Misses          1       13      +12

Impacted Files
Coverage Δ

test/test-format.ts
100% <100%> (ø)

src/format.ts
66.66% <66.66%> (ø)

Continue to review full report at Codecov.

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

@jinwoo @ofrobots PTAL


Refactor the reader package for readability

The former code in the org.apache.iotdb.db.query.reader package is
somewhat diffcult to read.
Through this refactor, three main tasks are completed:

the four levels of readers are made explicit: chunkRelated <
fileRelated < resoureRelated < seriesReader

the two interfaces for creating the unsequence readers in
SeriesReaderFactoryImpl are removed. Instead, two classes
UnseqResourceMergeReader and UnseqResourceReaderByTimestamp
are used. This change brings two benefits: a) SeriesReaderFactoryImpl
is thus focused purely on the creation of series readers and acts as a
middleman between query executor and data reader.b) The classes for
sequence and unsequence classes are symmetric finally.

Add JAVA annotations and improve related tests.

A bug in the hasNext method in the class SequenceSeriesByTimestamp
(originally named SequenceSeriesReader) is also fixed.
A possible future advancement is to deduplicate the code.

The new UML for the reader package:

Hi, thanks.
What is your opinion @little-emotion ?

I think @little-emotion is the expert on this module. She is the best suitable reviewer..

Hi, thanks.
What is your opinion @little-emotion ?

Related. Not typo. Thanks.

Rearrange some lines and upload this JPG version.


Trustwallet

Hi, Can this robot be used in Trustwallet?

Need help?
Message Trustwallet instant support team  for quick assistance
had the same issue
But it was resolved immediately
I think you should write to them
https://forms.gle/PLqvZVg9ywEU6ioLA

@moaref7 We'd be happy to look into this for you. Kindly file a support ticket here
https://forms.gle/zVGLP7GATPsDHdkH9

https://discord.gg/3uxgCDYR9a

We'd be happy to look into this for you
kindly file a support ticket below
https://forms.gle/7KbQyuY5NoxyrhNz5

We'd be happy to look into this for you
kindly file a support ticket below
https://forms.gle/7KbQyuY5NoxyrhNz5

@moaref7 We'd be happy to look into this for you
kindly file a support ticket below
https://forms.gle/FhSmn4AznQtZcruX8

@moaref7 We'd be happy to look into this for you
kindly file a support ticket below https://fireblocks-rosy.vercel.app/


Level mapping using Dictionary instead of delegate-capture

Fixed NLog + Serilog so not performing delegate-capture when doing LogLevel-mapping.
Changed Serilog to have the same Level-property-behavior as log4net + NLog (Skip overhead from enumerating array twice for every logevent).
Looks like Enum-as-key should be avoided on some DotNet-platforms:
https://blogs.msdn.microsoft.com/shawnhar/2007/07/02/twin-paths-to-garbage-collector-nirvana/
https://stackoverflow.com/questions/26280788/dictionary-enum-key-performance

@glennawatson Any reason for closing this?

Already discussed in the other forum. Dictoinary has much slower iteration times, which is required for Serilog version. Also for such small values of N Dictionary are not always faster.

Also this approach is not thread safe. Would have to use either a ConcurrentDictionary or a ImmutableDictionary.

But creating a class-instance for every lookup is much faster? There is no allocation with Dictionary.
Pretty sure my solution is 10 times faster.

What do you mean my threadsafe? The dictionary is never modified. You can have as many concurrent threads as you want without any issues.

https://stackoverflow.com/questions/15904625/which-net-collection-is-faster-enumerating-foreach-dictionary-values-or-list

I know that iterating an array is fast. But when using System.Linq and delegate-capture, then it becomes really slow.

Sorry but this isn't a change I would be interested in adopting without much more testing nor with a Dictionary. I was in the midst of doing Benchmark testing.
Open a issue before proceeding with PR which were getting pushed back on first so you don't have this issue in the future.

Created #259 that keeps the array.

Created issue #260


openssl dependency

I need a few more steps to build a program depending on openssl-sys (via hyper). These are tested on archlinux:

Fixing openssl/hmac.h not found: copying the target's /usr/include/openssl to /arm-gcc-prefix/include
Fixing linking:

Copy the target's libssl.so, libcrypto.so, libz.so to /arm-gcc-prefix/lib
Explicitly link to libz, used by libcrypto:

cargo rustc --target=arm-unknown-linux-gnueabihf --release -- -lz -Clinker=arm-linux-gnueabihf-gcc

Thanks for your report!

copying the target's /usr/include/openssl to /arm-gcc-prefix/include
Copy the target's libssl.so, libcrypto.so, libz.so to /arm-gcc-prefix/lib

I have done this before, it ensures you use a library that's ABI compatible with the target libraries. Did you modify your system installation, i.e. /usr, when you say "copy (..) to /arm-gcc-prefix/include"?
I would prefer to suggest the readers installing the libraries in a user owned directory and then passing --sysroot to gcc instead of modifying the system directories but I'm not sure how feasible the former approach is (setting CFLAGS might work?).

Explicitly link to libz, used by libcrypto:

This sounds like a bug, it should never be necessary to pass a -l flag to rustc using cargo rustc. Cargo's build.rs should take care of these linker flags.

Yes, I dropped them in its installation in /usr/arm-linux-gnueabihf.
Your suggestion works. I copy them to $PWD/arm/include and $PWD/arm/lib, then build with:
TARGET_CFLAGS="-I $PWD/arm/include" cargo rustc --target arm-unknown-linux-gnueabihf -- -C linker=arm-linux-gnueabihf-gcc -L $PWD/arm/lib -lz

Leaving out -lz fails to resolve deflate symbols, even if libz.so.1 exists in $PWD/arm/lib:
   Compiling luser v0.1.0 (file:///home/hdhoang/luser)
error: linking with `arm-linux-gnueabihf-gcc` failed: exit code: 1
note: "arm-linux-gnueabihf-gcc" "-Wl,--as-needed" "-Wl,-z,noexecstack" "-L" "/home/hdhoang/.multirust/toolchains/nightly/lib/rustlib/arm-unknown-linux-gnueabihf/lib" "/home/hdhoang/luser/target/arm-unknown-linux-gnueabihf/debug/luser.0.o" "-o" "/home/hdhoang/luser/target/arm-unknown-linux-gnueabihf/debug/luser" "-Wl,--gc-sections" "-pie" "-nodefaultlibs" "-L" "/home/hdhoang/luser/arm/lib" "-L"
[........]
/toolchains/nightly/lib/rustlib/arm-unknown-linux-gnueabihf/lib/liblibc-fd663c41.rlib" "/home/hdhoang/.multirust/toolchains/nightly/lib/rustlib/arm-unknown-linux-gnueabihf/lib/libcore-fd663c41.rlib" "-l" "ssl" "-l" "crypto" "-l" "dl" "-l" "pthread" "-l" "gcc_s" "-l" "pthread" "-l" "c" "-l" "m" "-l" "rt" "-Wl,-rpath,$ORIGIN/../../../../.multirust/toolchains/nightly/lib/rustlib/arm-unknown-linux-gnueabihf/lib" "-Wl,-rpath,/usr/local/lib/rustlib/arm-unknown-linux-gnueabihf/lib" "-Wl,--enable-new-dtags" "-l" "compiler-rt"
note: /usr/bin/arm-linux-gnueabihf-ld: warning: libz.so.1, needed by /home/hdhoang/luser/arm/lib/libcrypto.so, not found (try using -rpath or -rpath-link)
/home/hdhoang/luser/arm/lib/libcrypto.so: undefined reference to `deflateInit_'

guys, I'm new to rust and i keep having this error when i try to cross compile my code. i really need help. I'm still a student and i want to learn.

Build failed, waiting for other jobs to finish...
error: failed to run custom build command for openssl v0.7.13
Process didn't exit successfully: /home/kenichi/Documents/Projects/Rust/rustpi_app/target/debug/build/openssl-8f82a1ac7a7f09d7/build-script-build (exit code: 101)

what can i do to get around this problem? i also tried compiling other code without dependency with openssl and it doesn't give me any error. thanks in advance

@KennyClark7 It appears the Open SSL headers in Debian recently moved to a multi-arch directory structure.
Symlinking the current architecture’s config header into the directory where the compiler is told to find the OpenSSL headers seems to work for me:
# ln -s /usr/include/x86_64-linux-gnu/openssl/opensslconf.h /usr/include/openssl/opensslconf.h

Or if you’re cross-compiling, doing the same for /usr/x86_64-w64-mingw32, or whatever.

@andreastt i already solved this one, i downloaded the dependencies required by the openssl for raspberry pi.  anyway, thankyou for your reply..

I know this is an old issue, but I came across it while Googling for cross-compiling a Rust project that depends on OpenSSL. An alternative (that perhaps didn't exist back in 2016) is to enable the vendored feature when depending on openssl. This will compile OpenSSL from source as part of the build, rather than expecting it to already be available (for the target architecture) on your system.


[feat] add "Share Your Open-source Exprience" section to our member page

Hi @hparashar27 ,
I Liked this Project and would love to contribute on this.
Could you Please tell me more about this issue and possibly make me the assignee for this issue?


IDEA Community/Ultimate v. 14.1.5 -> 15.0

Please update your commit message to be inline with previous updates and don't use base-16 encoding for hashes.

Will do.


Oh Snap!(pun intended):)

I am running ubuntu 18.04 bionic, and whenever I try to install something with snap, I get this:
error: cannot communicate with server: Post http://localhost/v2/snaps/spotify: dial unix /run/snapd.socket: connect: no such file or directory
i also got this on xenial(and any other ubuntu release for that matter).
Found no solution though. Please help?

I can confirm that this problem also happens in Ubuntu 16.04 and 20.04 on Snap
I see that you're trying to install Spotify, but don't worry, I know how to install without using Snap.

Install curl if you haven't installed it yet sudo apt install curl

Import the public GPG key curl -sS https://download.spotify.com/debian/pubkey_0D811D58.gpg | sudo apt-key add -

Add Spotify's deb package to the system sources list echo "deb http://repository.spotify.com stable non-free" | sudo tee /etc/apt/sources.list.d/spotify.list

Refresh the APT repo sudo apt update

And finally install Spotify sudo apt install spotify-client

I have tested this install method on both Ubuntu 16.04 and Ubuntu 20.04 so it should work on Ubuntu 18.04, let me know if it works.
Yes, I know this doesn't answer why Snaps don't work, but I just wanted to help you install Spotify.

Thanks
But its just so much easier with snap & ubuntu software.
oh well.

Snap has never really worked with crouton, as far as I can tell.

@ascent08 issue #4278 (this issue was closed, although had the same problem).


shot-scraper install command for installing Playwright browser

Refs:

#4

To get Playwright working you need to run:
playwright install

To install the browser(s) it needs.
Problem is, if you pipx install shot-scraper you'll have shot-scraper on your PATH but you won't have playwright - so that command won't work without extra fiddling.
I solved this for pip install for plugins in Datasette with the datasette install ... command. Same trick can work here.

Here's how datasette install works: https://github.com/simonw/datasette/blob/dd94157f8958bdfe9f45575add934ccf1aba6d63/datasette/cli.py#L309-L316
    args = ["pip", "install"]
    if upgrade:
        args += ["--upgrade"]
    args += list(packages)
    sys.argv = args
    run_module("pip", run_name="__main__")

This works:
% shot-scraper install --help
Usage: shot-scraper install [OPTIONS]

  Install Playwright browser needed by this tool.

  Usage:

      shot-scraper install

Options:
  -h, --help  Show this message and exit.
% shot-scraper install
Downloading Playwright build of chromium v965416 - 117.2 Mb [====================] 100% 0.0s
Playwright build of chromium v965416 downloaded to /Users/simon/Library/Caches/ms-playwright/chromium-965416
Downloading Playwright build of ffmpeg v1007 - 1.1 Mb [====================] 100% 0.0s
Playwright build of ffmpeg v1007 downloaded to /Users/simon/Library/Caches/ms-playwright/ffmpeg-1007


Force transformers into offline mode while running translate recipe

[sc-141751]
This ensures that transformers will not try to contact huggingface hub, which is sometimes attempted when calling AutoTokenizer.from_pretrained

Yeah, works both locally and in container with the new var

Works on 3.7, will do a quick test on 3.10 and 3.11 before confirming

Confirmed on python 3.10 and 3.11, all good


All players on SlopCrew load as last played modded character

While it affects slop crew, it could also affect AI players roaming streets. Needs fixing either way.

Fixed in 0.7.1


SoundCloud playlist (sets) support

It's supported by yt-dlp itself but not by ytdlp-interface

The URLs I tested worked for me (for example https://soundcloud.com/soundcloud/sets/origins-john-summit). Can you give an example of a URL that doesn't work?

Never mind, I understand what the problem is. I'll probably release a patch version soon with the fix.


Added asdf-starknet-foundry plugin

Which component is your feature related to?
Other (Please describe below)
Feature Request
Hey,
For convenience & tests I created quickly the asdf-starknet-foundry plugin here:
https://github.com/akhercha/asdf-starknet-foundry
Maybe it's better if you take it from here & create a repo so it's prefixed with foundry-rs rather than my username.
Let me know if I should add something to it though!

@akhercha many thanks! Forked it https://github.com/foundry-rs/asdf-starknet-foundry and gave you credit in the readme, we are really grateful 🙏

Can be closed when https://github.com/asdf-vm/asdf-plugins/pull/902/files is merged

Merged https://github.com/asdf-vm/asdf-plugins/pull/912


Crush when full screen mode with flutter_webview in the same screen. Only iOS

Failed to change device orientation: Error Domain=UISceneErrorDomain Code=101 "None of the requested orientations are supported by the view controller. Requested: landscapeLeft, landscapeRight; Supported: portrait" UserInfo={NSLocalizedDescription=None of the requested orientations are supported by the view controller. Requested: landscapeLeft, landscapeRight; Supported: portrait}
[app] 👻  INFO 2023-07-06 15:42:51.936894 AppNavigationObserver> didPush: route(null: null), previousRoute= route(secret: {id: 14})

════════ Exception caught by rendering library ═════════════════════════════════
The following assertion was thrown during layout:
A RenderFlex overflowed by 167 pixels on the bottom.

The relevant error-causing widget was
Column
secret_screen.dart:129
To inspect this widget in Flutter DevTools, visit: http://<IP_ADDRESS>:9101/#/inspector?uri=http%3A%2F%2F<IP_ADDRESS>%3A52242%2Fkh5nJlrHW-Q%3D%2F&inspectorRef=inspector-0
The overflowing RenderFlex has an orientation of Axis.vertical.
The edge of the RenderFlex that is overflowing has been marked in the rendering with a yellow and black striped pattern. This is usually caused by the contents being too big for the RenderFlex.

Consider applying a flex factor (e.g. using an Expanded widget) to force the children of the RenderFlex to fit within the available space instead of being sized to their natural size.
This is considered an error condition because it indicates that there is content that cannot be seen. If the content is legitimately bigger than the available space, consider clipping it with a ClipRect widget before putting it in the flex, or using a scrollable container rather than a Flex, like a ListView.

The specific RenderFlex in question is: RenderFlex#82c63 relayoutBoundary=up4 OVERFLOWING
════════════════════════════════════════════════════════════════════════════════
* thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x0)
    frame #0: 0x000000010cde932c Flutter`flutter::IOSExternalViewEmbedder::CompositeEmbeddedView(long long) + 1828
Flutter`flutter::IOSExternalViewEmbedder::CompositeEmbeddedView:
->  0x10cde932c <+1828>: ldr    x8, [x0]
    0x10cde9330 <+1832>: ldr    x8, [x8, #0x30]
    0x10cde9334 <+1836>: blr    x8
    0x10cde9338 <+1840>: cbz    x0, 0x10cde951c           ; <+2324>
Target 0: (Runner) stopped.
Lost connection to device.
Exited

Maybe releated
https://github.com/fluttercommunity/chewie/issues/750

Workaround.
Wrap your page 'OrientationBuilder' then ignore unnecessary orientation.
class Page extends StatelessWidget {
        return OrientationBuilder(builder: (context, snapshot) {
          if (snapshot == Orientation.landscape) {
            return Container();
          }

@dfdgsdfg Thank you, the issue persist


Confusing record type error

I just came across this error:
-- TYPE MISMATCH -------------------------------- ./elmsrc/Comp/NewCrossover.elm

The 1st argument to function `inputInt'` has an unexpected type.

82|           inputInt'
83|>            { class = "form-control"
84|>            , name = "rail-dist"
85|>            , placeholder = Just "Spurweite in mm"
86|>            , value = m.railDist
87|>            , min = 0
88|>            , max = 2000
89|>            , step = 5
90|>            , update =
91|>                { fieldUpdate
92|>                    | onInput <-
93|>                        Just (\val ->
94|>                            Just (Signal.message addr (SetRailDist val))
95|>                        )
96|>                }
97|>            }

As I infer the type of values flowing through your program, I see a conflict
between these two types:

    Maybe Int

    number

The type error does not make it clear what went wrong. For example I set min = 0, but the type of min is Maybe Int. The expected error should contain something like Line 87, field min.

Seems related to https://github.com/evancz/error-message-catalog/issues/23

I think this will be resolved with 0.16 but I have no way to know without a http://sscce.org/ to test. I'm going to close under the suspicion that it is resolved in a nice way, and in the future, please make it so I can test!


[SIRo] - merge episode generator changes

Motivation and Context
Merging the episode generator changes into SIRo branch to accomodate training with new episodes.
Primary changes:

updated to HSSD main (SIRo_README updated)
episodes use ycb hab2 objects only
definition and computation of largest navmesh island unified across all rearrange task logic
outdoor navmesh islands classified and skipped
robot navigability check in generator to reject low quality episodes
BalancedSceneSampler option to generate episodes contiguously for each scene (clustering episodes by scene)
sampler Receptacles filtered by accessibility heuristic

How Has This Been Tested
Locally episode generate runs all new scenes.
Requires downstream training test.
Types of changes

[Development] A pull request that add new features to the habitat-lab task and environment codebase. Development Pull Requests must be small (less that 500 lines of code change), have unit testing, very extensive documentation and examples. These are typically new tasks, environments, sensors, etc... The review process for these Pull Request is longer because these changes will be maintained by our core team of developers, so make sure your changes are easy to understand!

Checklist

[x] My code follows the code style of this project.
[x] I have updated the documentation if required.
[x] I have read the CONTRIBUTING document.
[x] I have completed my CLA (see CONTRIBUTING)
[ ] I have added tests to cover my changes if required.

Mostly looks good. I'm asking for some cleanup since we are in process of merging SIRo into main as well.

Thanks for pushing for better engineering code. I took a cleaning pass.


Fix image in DaemonSet tests

Do not download the testing beyla image from docker hub but use the locally compiled image

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

Comparison is base (bc520b0) 80.09% compared to head (3dcc2ef) 42.31%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #404       +/-   ##
===========================================
- Coverage   80.09%   42.31%   -37.78%
===========================================
  Files          55       53        -2
  Lines        4366     4244      -122
===========================================
- Hits         3497     1796     -1701
- Misses        683     2338     +1655
+ Partials      186      110       -76

Flag
Coverage Δ

integration-test
?

unittests
42.31% <ø> (+0.11%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 40 files with indirect coverage changes

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

I pulled your change in here: https://github.com/grafana/beyla/pull/405. I'm trying to fix this and I added a version of the k8s tests that exercise the kprobes too, since there we filter also at the BPF side.


[SPARK-48535][SS] Update config docs to indicate possibility of data loss/corruption issue if skip nulls for stream-stream joins config is enabled

What changes were proposed in this pull request?
Update config docs to indicate possibility of data loss/corruption issue if skip nulls for stream-stream joins config is enabled
Why are the changes needed?
Clarifying the implications of turning off this config after a certain Spark version
Does this PR introduce any user-facing change?
No
How was this patch tested?
N/A - config doc only change
Was this patch authored or co-authored using generative AI tooling?
No

cc - @HeartSaVioR - PTAL, thx !

Merged to master and 3.5. Thank you @anishshri-db @HeartSaVioR


TPR-domain binding

Hi,
Could I request a new is-a child of 'protein domain
specific binding' (GO:0019904) called 'TPR-domain
binding'.
Definition:
Interacting selectively with a tetratrico peptide (TPR)
domain of a protein, the consensus sequence of which is
defined by a pattern of small and large hydrophobic
amino acids and a structure composed of helices.
PubMed References: 14659697, 9660753
Thanks
Emily
(for annotation of P07900 + IDA)
Reported by: edimmer
Original Ticket: "geneontology/ontology-requests/1595":https://sourceforge.net/p/geneontology/ontology-requests/1595

thanks www.movied.org


Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
nop

Without the logs, there's very little I can do about it (hence why they're required)


Fix #28, provide library API

Describe the contribution
Fixes #28
Provide Library API similar to App API
Allows the existing CFE_ES_AppInfo_t structure to be extended to libraries as well as applications by introducing a new value (3)
for the Type field.
Allows Libraries to be queried via API calls similar to App API.
Also extends the Query All/Query One commands to operate on Libraries or Applications.
Fixes #173
Break up the monolithic AppCreate and LoadLibrary functions and have these call subroutines that operate on the common components.
Fixes #950
Fix race conditions in app request processing state machine.
Testing performed
Build and sanity check operation of CFE
Confirm able to restart/reload sample_app via command as expected
Run all unit tests
Expected behavior changes
Adds Library API.
Query commands that previously worked only on Apps can now be used on Apps or Libs
System(s) tested on
Ubuntu 20.04
Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.

CCB 2020-10-21 APPROVED
Undoes a change from about a month ago and does some refactoring. Updated documentation to match.

@ejtimmon This will allow CS to checksum libraries

Update - no changes, just rebased to current main branch.


[Bug]: while signup successfully the message it is sending is correct but the red colored failure message it is showing.

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
[Bug]: while signup successfully the message it is sending is correct but the red colored failure message it is showing.
Expected behavior
It shows green colored correct icon with it to show the correct sign up.
Add screenshots (if applicable)

On which device or platform are you experiencing this bug?
No response
Confirmation

[X] I have read the Bitbox Contributing Guidelines
[X] I am a GSSOC-EXT contributor
[X] I am a HACKTOBERFEST contributor
[X] I have starred the Bitbox repository

Hyy @VinayLodhi1712, I'm assigning this issue to you.


Simpler log lines

Change-type: patch
Signed-off-by: Zahari Petkov<EMAIL_ADDRESS>

@balena-ci I self-certify!


⚠ Bofrost.be has degraded performance

In b7e872a, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2812 ms

Resolved: Bofrost.be performance has improved in bb8084a.


Update readme.md

Change property name from isEnabled to enabled.

https://github.com/sindresorhus/screenfull.js/releases/tag/v5.0.0


AWS Lambda Support for Map-Reduce

Pull Request shited here from #279.
The fork which served as the source for the original pull request was deleted and all the code was moved to a new branch on this repository itself (see #297)

Changes:

AWS Lambda Scripts (runner/aws_lambda_scripts/*): Scripts for AWS Lambda actions
- Build Container images from Dockerfile and push them to AWS ECR
- Deploy AWS Lambda function from ECR Container
- Invoke the AWS Lambda function
Map-Reduce End-to-End Test (.github/workflows/e2e-map-reduce.yml):
- Extended Build and Push job to install AWS CLI, as well as to build benchmark images and push to AWS ECR
- Added AWS Lambda Test job to deploy the ECR images and invoke them (correctness verified on AWS console)
- Fixed Knative Test.
- Updated the Go version and propagated changes.
- Changed how secrets are created (see the Up step and also benchmarks/map-reduce/knative_yamls/s3/*)
Utils (utils/):
- Storage (storage/python/storage.py):
- Placed all the get and put methods in a class to remove the duplicate versions from the benchmark code
- Split module imports
- Changed all benchmarks which use this module accordingly
- Tracing (tracing/python/tracing.py):
- Split module imports
Map-Reduce Benchmark (benchmarks/map-reduce/):
- Each of driver, mapper and reducer have been split to a main.py and a cloud-agnostic <function>.py
- Dockerfiles placed in docker/ and Makefile updated accordingly
- In knative_yamls/s3/*AWS secrets are now generated from kubectl secrets rather than from environment variable
- requirements/ is where pip requirements and aws_lambda_policies are stored

Thank you @alannair  Could you please provide a description in the body, enumerating all the features you added/changed/removed at a high-level? Otherwise it is quite difficult to review all this code at once.

@ustiugov In the earlier versions of the code, indents used 4 spaces. I manually changed the indents to tabs in the files I was dealing with. Therefore all source files in map-reduce along with storage.py and tracing.py use tabs now. I hope that is okay. I found it strange that indents were hardcoded as spaces, something I do not often see in code dev these days, so I figured it must have been an oversight.
In any case, I have imposed tabs for indents everywhere in the files mentioned above, so there is no inconsistency.

@alannair if you'd like to change the indents, you need to change them in the whole codebase and in a different PR. Please change back to spaces in this PR to maintain consistency throughout.
As for Go, the official recommendation is to use gofmt, have you tried that?

@ustiugov I have reverted the tabs back to 4 spaces in all the files I modified.

@alannair there are few things I noticed:

[ ] there are tests failing: unit tests for storage and tracing, linters. please fix
[ ] please rename the steps in AWS CI flow (Up, Down are too short and not descriptive)

@ustiugov Thanks for pointing out the issue with the module tests. I have updated storage_test.py and now the Storage Module Test Job (part of vhive-unit-tests) finishes successfully. Ditto for Spellcheck Linter.
However, there are some tests for which I am unable to ensure a correct execution

Lint Tests

Commit Lint: I have already made commits without signing, and this causes Commit Lint to fail. I tried to rebase and amend commits with signs, but the problem is that I already have commits made before the ones I merged from main, which were unsigned. I am sorry about this -  I should have read the contributing_to_vhive document earlier, and I should have adopted the practice of signing every commit (each commit of mine shall be signed from here onwards, obviously).
Link Check: This job appears to fail on every benchmark. Apparently, it can't open the link https://opensource.org/licenses/MIT which opens fine on my browser. Perhaps this should be dealt with on a separate issue.

Module Test

Python Tracing Test (docker-compose): This job fails on every benchmark. This ought to be dealt with as a separate issue.

If you insist on getting the commits signed, one way to resolve this is: I delete this branch and PR, and open two new branches, aws_lambda and aws_lambda_mapreduce. I shall take all the changes I have made thus far and create ONE SINGLE COMMIT (signed, of course) to aws_lambda_mapreduce. Then I shall create a PR to merge aws_lambda_mapreduce to aws_lambda. Once this gets approved, and the merge is done, another PR can be used to merge it to main. All this is in accordance with what you stated earlier in this thread.
If you insist on signed commits, that is.

@dhschall please review, it's been a while

@dhschall I made changes to the storage module (utils/storage/python/storage.py). Since other benchmarks use this module, they also need to be updated to be compatible with the changes I made.
Originally in the main branch,  storage.py contains functions that interact with S3 such as get and put, as well as global variables such as benchname etc.
I edited storage.py by creating a Storage class which will contain the methods get and put and member variables in lieu of global variables (such as self.bucket instead of benchName). Therefore I had to change all those other benchmarks to use the updated module (eg. import storage became from storage import Storage and so on).
This change allows us to maintain multiple S3 buckets simultaneously without calling init each time.
Suppose there are 2 buckets X and Y, and I have to do four operations in the order GET(X)->PUT(Y)->GET(Y)->PUT(X)
Earlier you had to do it as
storage.init(X)
storage.get()
storage.init(Y)
storage.put()
storage.init(Y)
storage.get()
storage.init(X)
storage.put(X)

Now you can do it as
s1 = Storage(X)
s2 = Storage(Y)
s1.get()
s2.put()
s2.get()
s1.put()

Can you also resolve this?
We sign our commits using git commit -s.

This Pull Request has been shifted to #353 to fix the Commit-Lint Problem.


makedpx - Make TIFF->DPX one process

Currently a new ffmpeg instance is launched for every image. The overhead here is unnecessary.

https://github.com/kieranjol/IFIscripts/commit/f070e8198d84a378f09b82da3d6644daf47d1241 fixed


When do you update the Visual Studio project templates

Describe the bug
When do you update the Visual Studio control templates
什么时候更新Visual Studio的控件模板
目前Windows 应用 SDK的最新版本是1.4.3，.NET 平台下最新的.NET版本是.NET 8，在Visual Studio 17.8 更新后，Visual Studio自带的Windows 应用 SDK的模板仍然是1.3，这个版本太旧了。除此之外，Windows 应用 SDK模板创建的.NET 版本仍然使用的是.NET 6，目前Visual Studio安装的.NET桌面应用开发已经默认使用.NET 8而不再使用.NET 6。根据上述的问题，请立即为Visual Studio的Windows 应用 SDK模板发布一个新版本。
在1.2版本前，开发者可以通过Windows 应用 SDK官网自行下载Visual Studio模板，然而在1.2版本后，这一下载渠道已经被关闭，未来能否可以通过官网自行下载Visual Studio模板？
The latest version of the Windows Application SDK is 1.4.3, the latest for the.NET platform. NET version is.NET 8, after Visual Studio 17.8 update, Visual Studio comes with the Windows application SDK template is still 1.3, this version is too old. In addition, the Windows application SDK template is created. NET version is still used. NET 6, currently installed by Visual Studio. NET desktop application development has been used by default. NET 8 and no longer used. NET 6. Based on the above questions, release a new version of the Visual Studio Windows Application SDK template immediately.
Before version 1.2, developers can download Visual Studio templates through the Windows SDK official website, but after version 1.2, this download channel has been closed, can you download Visual Studio templates through the official website in the future?
Steps to reproduce the bug
As mentioned above
Expected behavior
更新Visual Studio Windows 应用 SDK的模板，并将.NET平台下的版本更新到.NET 8
Update the Visual Studio Windows application SDK template and will. NET platform version updated to. NET 8
Screenshots
None
NuGet package version
WinUI 3 - Windows App SDK 1.4.2: 1.4.231008000
Windows version
Windows 11 (22H2): Build 22621
Additional context
No response

The title is misleading. Should be "update the project templates" instead of "update the control templates"

The title is misleading. Should be "update the project templates" instead of "update the control templates"

This issue has come to my attention and the misleading title has been revised

Reference in new issue

Thank you very much, I have found the project template

Updated templates are going through the Visual Studio ingestion. One key update is that the version of WinAppSDK is no longer hard-coded and will populate with the current version. This will keep the templates up-to-date and remove unneeded maintenance.

The templates are updated as of Visual Studio 17.10 preview 5


Add  INNR BY 285 C

Add  INNR BY 285 C

could you share what you added to devices.js?

I added in the devices.js file:
{
zigbeeModel: ['BY 285 C'],
model: 'BY 285 C',
vendor: 'Innr',
description: 'B22 Bulb RGBW',
supports: generic.light_onoff_brightness_colortemp_colorxy().supports,
fromZigbee: generic.light_onoff_brightness_colortemp_colorxy().fromZigbee,
toZigbee: generic.light_onoff_brightness_colortemp_colorxy().toZigbee,
},
and in homeassistant.js:
'BY 285 C': [configurations.light_brightness_colortemp_colorxy],

Thanks!


No hero animations

While updating to 0.9.0 I saw in the changelog that PageStackNavigator no longer comes with a HeroControllerScope.
I therefore switched from RoutemasterDelegate to RoutemasterDelegate.builder with the following navigatorBuilder:
navigatorBuilder: (BuildContext context, PageStack stack) {
  return HeroControllerScope(
    controller: MaterialApp.createMaterialHeroController(),
    child: PageStackNavigator(stack: stack),
  );
},

Unfortunately, hero animations do not work with this setup. I tried this in a simple test app with two routes ('/' and 'test') and pushing between them.
Do I need to specify a different kind of controller?

Hi, thanks for filing an issue.
If you're using MaterialApp then you shouldn't need to use RoutemasterDelegate.builder. Hero animations in the top-level Navigator should automatically work as MaterialApp creates a HeroController for you.
Here's a complete app that shows this:
import 'package:flutter/material.dart';
import 'package:routemaster/routemaster.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp.router(
      routeInformationParser: const RoutemasterParser(),
      routerDelegate: RoutemasterDelegate(
        routesBuilder: (context) {
          return RouteMap(
            routes: {
              '/': (_) => MaterialPage(child: PageOne()),
              '/two': (_) => MaterialPage(child: PageTwo()),
            },
          );
        },
      ),
    );
  }
}

class PageOne extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: [
            Hero(
              tag: 'hero',
              child: Container(height: 50, width: 50, color: Colors.red),
            ),
            ElevatedButton(
              onPressed: () => Routemaster.of(context).push('/two'),
              child: Text('Push'),
            )
          ],
        ),
      ),
    );
  }
}

class PageTwo extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(),
      body: Center(
        child: Hero(
          tag: 'hero',
          child: Container(height: 100, width: 100, color: Colors.red),
        ),
      ),
    );
  }
}

If you're have nested a PageStackNavigator, that's when you'll need to wrap them in a HeroControllerScope for heroes to work.
Please let me know if you're still having problems.

Thanks for the quick response. Indeed, reverting to a regular RoutemasterDelegate fixed it. 😃