etcdserver: support RetryMsgProp

When MsgProp is dropped at rafthttp, raft loop is notified and add it
to the retry queue. When raft loop thinks it is a good time to retry it,
it proposes the data to raft state machine. If it cannot find a good time,
it drops proposals.
I have tested it manually. It works really well, and reduces the number of dropped messages from 12 to 4 in my test. The left 4 dropped because it is hard to tell when stream is broken.
fixes #3380

@yichengq We have discussed this a while ago. The previous decision is to retry proposal at rafthttp layer. Can you explain why we want to propagate the retry logic into etcdserver layer?

@yichengq Is it because you want to find the leader (or the new leader) before retry?

@xiang90 Yes. If it loses the connection with current leader, there is several probability:

the leader is dead. Resend messages cannot help but repropose works.
the leader is isolated from the internet for election timeout. Resend and repropose both work. Considering the cluster has new leader now, old message needs to redirect once more.
the leader is connected again in short time. Resend and repropose both work.

I choose retry propose to make it work better. I thought about retrying it in rafthttp, but it needs to understand leader switch and modifying messages then.

It works really well, and reduces the number of dropped messages from 12 to 4 in my test. The left 4 dropped because it is hard to tell when stream is broken.

I am not sure I understand this correctly. Can you explain more?

It works really well, and reduces the number of dropped messages from 12 to 4 in my test. The left 4 dropped because it is hard to tell when stream is broken.

I started 3 local machine and keeps proposing at one follower at 10req/sec. Then I killed the leader and checked how many proposals lost. Before this PR, it dropped ~12 proposals. After this PR, it dropped ~4 proposals, and retried other ~8 proposals successfully. The ~4 dropped is because rafthttp cannot know steam broken immediately and keeps sending some proposals in this period.

The ~4 dropped is because rafthttp cannot know steam broken immediately and keeps sending some proposals in this period.

Do you have a future plan for solving this?

Do you have a future plan for solving this?

No. My first thought is that it is hard to tell whether messages are lost in transfer process, and I definitely don't want to propose the same data twice. But it is possible that part of them is dropped when they are cached at lower level locally and have not sent out yet.

My first thought is that it is hard to tell whether messages are lost in transfer process,

Why? I think we assign each proposal a unique id, right?

We do assign each proposal a unique id. But it is hard to use:

it needs the remote to tell it whether or not the proposal is received, and this communication is hard, e.g., when the remote is disconnected.
it proposes possibly lost ones anyway. When apply, it needs to take care of not applying entries with same ID, which makes the logic complicated.

We do assign each proposal a unique id. But it is hard to use:

In my opinion, a member can always simply re-propose the proposal that it does not apply after a while. The current leader just need to drop a duplicate proposal.

Let me try.

I have tried and have some problem with it:
I need to maintain previous apply IDs to check duplication. To make it determined, I want to keep fixed number of previous apply IDs in the memory. The number is large considering it could apply up to 20000req/s and max request timeout is 105s. So I cannot recover this map when restart, and this may result in apply difference.

The number is large considering it could apply up to 20000req/s and max request timeout is 105s.

I do not think 30MB is BIG. And we do not have to keep this for 105s in most of the cases.

So I cannot recover this map when restart, and this may result in apply difference.

I do not understand why this will result apply difference.

I do not understand why this will result apply difference.

I thought it wrong: it will not result apply difference, but may lead to apply one propose twice.
Here is some scenario:

A, B, C is etcd cluster, and A is the leader.
C proposes and A receives
connection between A and C breaks
A commit it successfully with B
A stops, B restarts
B becomes leader with vote from C
C proposes again
B should reject this proposal, but it doesn't have the apply map.

Apply twice is still bad.

@yichengq
I think the scenario you just described is not a problem. One thing I am still confused is that you do you think B will lose the appliedMap after restart? Why cannot get the applied IDs from the proposals in wal again?

Why cannot B get the applied IDs from the proposals in wal again?

it could apply up to 20000req/s and max request timeout is 105s, so it needs to load 2million entries to rebuild the apply ID map. It is doable(I cannot recover is wrong), but it is kind of weird to do it.

it could apply up to 20000req/s and max request timeout is 105s, so it needs to load 2million entries to rebuild the apply ID map.

Why 2m is a problem? Why this is weird at all?

Why 2m is a problem? Why this is weird at all?

We only keep 10k for apply from snapshot point now. It is surely doable. Will try.

We only keep 10k for apply from snapshot point now.

Well... do you think it is reasonable to do a snapshot erver 10k when the request rate is 20k/second?

Close this PR because it doesn't solve the issue well. It cannot retry all proposals during leader election while no machines down, and users will still see request timeout error.


Simplify buffering in TCPConnection

Prior to this commit, the buffering in TCPConnection was overly
complicated. This was a carry over from the old style of buffer
handling. However, after Dipin Hora's commits to bring over the "faster
tcp" changes from Wallaroo, the buffer handling no longer makes sense.
Prior to this commit, a TCPConnection would be provided two variables to
control read buffering: init_size, max_size. The read buffer would start
at init_size and would grow to max_size.
Before "faster tcp", the buffer would only grow in size if we read the
entire buffer's worth of data in one recv call. After the data was read,
the buffer would be truncated and then handed off to the
TCPConnectionNotify. This was very wasteful. If you had a buffer of 128
allocated bytes and only read 4, then the 124 allocated bytes would be
truncated away and the buffer would be reallocated.
With "faster tcp", we allocate a large buffer and when reading data, we
chop it off the buffer as needed. This means, that "grow the buffer"
doesn't make much sense anymore. The buffer will continually shrink in
size as we slowly chop more and more off. This meant that with the code
as it is prior to change, we would always grow to max buffer size.
If we are always going to grow to max buffer size, it makes sense to
simplify the code and our interfaces and only allow for setting the
buffer size. This commit makes that simplication. Now we will:

allocate a buffer of size read_buffer_size
reallocate when the buffer is empty
reallocate if the caller is using expect and the buffer is too small
to hold the expected payload

This is a breaking change for TPCConnection and TCPListener as it
changes the constructor interfaces.

Can you please add a link to PR #3178 in the commit and PR description?

@jemc comment amened.

@SeanTAllen it seems like this change cannot handle the scenario when _expect > _read_buffer_size (or it can but i'm not following the logic correctly).
Can you please clarify/confirm?

Correct. Expect is capped at whatever the buffer size is.

And I forgot to open a PR for that change to expect.

That seems like a significant functional change beyond the "simplify buffering" as the title of this PR suggests.

I thought I had already opened a PR to address expect

Okey dokey. I'll wait to see that PR. Thanks.

The core change there is
  fun ref expect(qty: USize = 0) ?=>
    """
    A `received` call on the notifier must contain exactly `qty` bytes. If
    `qty` is zero, the call can contain any amount of data. This has no effect
    if called in the `sent` notifier callback.
    """
    if qty <= _read_buffer_size then
      if not _in_sent then
        _expect = _notify.expect(this, qty)
        _read_buf_size()
      end
    else
      error
    end

@dipinhora https://github.com/ponylang/ponyc/pull/3197


MdTooltip leaking dom nodes when used in element inside *ngIf

Bug, feature request, or proposal:
Bug: MdTooltip is leaking dom nodes when used inside *ngIf
What is the expected behavior?
When parent element is destroyed by *ngIf, it's child nodes should be released and collected by garbage collection.
What is the current behavior?
None of the nodes inside element containing *ngIf are cleared from memory when there is element with MdTooltip -directive.
What are the steps to reproduce?

Open chrome devtools and take heap -snapshot.
Toggle tooltip item visible ten times.
Press garbage collection
Take a new heap -snapshot
Compare with latest snapshot
There are 10 new MdTooltip and 10 text nodes created and 0 deleted. So delta is +10.

<button (click)="showTooltip = !showTooltip">Toggle tooltip</button>

  Show tooltip

XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX

Example plunker:
https://plnkr.co/edit/6ayP5LKlM62GUwCpP6Fg?p=preview
What is the use-case or motivation for changing an existing behavior?
Save memory
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular 4.2.3
Material 2.0.0-beta.7
Is there anything else we should know?
I'm not too familiar regarding memory management so correct me if I'm misunderstanding how this should work.

Sounds similar to https://github.com/angular/material2/issues/4499. Does that look like what you're experiencing?

Yes, it's looks exactly same. Don't know why I didn't find that issue.
I also tested without hammerjs and there were no leak. So it seems to be related to that.
dupe #4499


pipeline, api: handling of AS0 measurements?

In https://github.com/ooni/explorer/issues/495 we only mitigated the issue by changing Explorer. This issue is here to remind us that we wanna discuss how to implement a more comprehensive backend side solution to the issue.
I will facilitate a conversation on this issue in one of the upcoming backend meetings.

See https://github.com/ooni/pipeline/pull/326, https://github.com/ooni/api/pull/194, and https://github.com/ooni/api/pull/195.

@FedericoCeratto told me earlier everything is either deployed or being deployed! So, we can close this issue now!


add links & custom image

Move over UI for adding links from pleuston and add to publish flow. Also implement functionality from there to add an image by adding a special link type and if present, use that image in place of category images.

Download ZIP files for category banners
Category (fallback) images changed to 2:1 format with full width size, based on current data asset view (645x323px).
Available in both 1x and 2x scale.
1x scale .JPG at 100% quality:
banners 1x-100.zip
2x scale .JPG at 80% quality - part 1:
banners 2x-80 part1.zip
2x scale .JPG at 80% quality - part 2:
banners 2x-80 part2.zip
Sample preview for quick reference:


Feature request: Optional support for a template engines

As a user and HTML coder, I would love to not write repetitive HTML. To solve this, I'd like to see some handler that enables us to use EJS, Handlebars, Nunjucks (or something else) so I can include separate chunks of html into my static templates.
I use serve to build static websites before I add them as a template in a CMS or static generator. But also in static JS, I'd love to add chunks of html into a template to stay organized and non-repetitive.
I hope it validates as a feature. As a hint, this package deals with collecting and assigning template engines: https://www.npmjs.com/package/consolidate. This way, the user can npm install the template engine of choice, and consolidate could handle the inclusion to serve with that template engine.
I hope this suggestion fits in the core ideas of serve. I use this daily as a tool because it does the job great! Thanks a lot for maintaining this project.

Hi @automagisch
We would expect your static site generator (SSG) framework to convert templates to HTML.
This doesn't really fit with the purpose of serve which serves static files.
Instead, you could try Next.js for React templates or Hugo for Markdown templates.
Then you can deploy your project too ZEIT Now by connecting your Git Repository in couple clicks.  Each time you git push it will deploy!
You get HTTP2, TLS 1.3, Brotli compression, and many other features automatically.
I hope that helps!


Accessibility bugs

Fixing two accessibility bugs pertaining to 200% text scaling and radio button invocation with Narrator.
Description

Added text wrapping property.
Updated NavigationView options' RadioButtons to use more specific Checked events instead of more generic Click events.

Motivation and Context

Bug 19597343: Text Scaling Desktop: XAML Controls Gallery : Information present for RichEditBox is clipped in 200% Text Scaling.
Bug 19440518: Narrator Desktop : XAML Controls Gallery : Unable to change the position of navigation pane using caps+enter

How Has This Been Tested?

manual verification
Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

As a bit more context: The Click handlers need to be replaced with Checked handlers because those are expected to be triggered by automation peers too.
The Checked/Unchecked events are the abstract ones that are universally called no matter which input is used.
For RadioButtons in general, we need to use the Checked events over the Click events.


Add ShapeTexture2D

A possible implementation of godotengine/godot-proposals#1126
ShapeTexture2D
Similiar to other procedural textures like GradientTexture2D and NoiseTexture2D, but instead generates simple regular shapes.
Main use cases:

Placeholder for prototyping, intended to be changed later.
As particle texture, where simple shapes go a long way.

ShapeTexture2D generates a texture with specified colored shape with transparent background, with optional border. The shape can be any regular polygon or star (and circle is a polygon with enough vertices). Texture can also be antialiased by supersampling.

As concerns were raised that this might not be what was wanted (the original proposal was quite vague on details), I decided to make a proof of concept draft.
This is mostly functional with following caveats:

Doesn't support corner rounding, which is often wanted for nicer results.
Border width is calculated in a simple, but incorrect way (sharper corners cause narrower borders).

@AThousandShips note that this is a work in progress and more of a proof of concept to test if this is an useful addition, I'm not sure it needs to be reviewed yet. I'll be correcting stuff which automatic checks catches however, to get some artifacts for easier testing.

Having it pass so people can test artifacts is kind of relevant :) that's why I reviewed, I'm not doing a review on functionality etc.

Fair enough. Just didn't want to waste your time. Since you're here however, can you help with this. I'm not entirely sure what this means:

Is it just about spaces at the end of line, or does it want me to add indent there too and separate 4; to a separate line?

Both I think, there's trailing space, and also rules about indentation here which are confusing with clang

IMO this type of implementation is excellent! Is this PR intended to be a proof-of-concept or something you'll want to get through? Just so I know if I should try helping with reviews.

I'm not hard determined to get this through, but I do think it is a nice feature. Most projects need some circle textures etc., so creating them in-editor feels logical. Is it useful enough? I don't know, but hopefully we'll get some feedback now that it's testable.
So for now, it's more POC so that it could be tested in action. I also proposed making this similiar api for a vector based node, so most of the code can be simply copied over to implement that as well.

As particles are one of main use cases I had in mind, I would love to get feedback from @QbieShay about this.
Is the feature set good for this use, or should it have something more? Or something less? Or is this just a fool's errand?
Also I forgot to thank @AThousandShips earlier, thanks for the help! You don't get thanked enough, but you seem to be everywhere ensuring that prs are high quality 👍

Thank you! Means a lot, I am happy my nit-picking is appreciated, and I've had enough experience with clang to know all the little folleys lol

I think this is a great feature! We hae lots of people that make lowpoly/pixel art games that could benefit from this.
I think it'd be good if it was easily compatible with pixel art and lowpoly games (not sure if you need extra feature for pixel art)
For me personally, I try to avoid using built-in textures unless they are saved to disk. I work on big project and a lot of built-in textures where isnt necessary can start slowing down the project. I believe that, in order to get good results with these textures it's likely that they will be upscaled in size, so I think UX wise it'd be better if the creation of the texture prompts a file save dialogue like visual shader.
I hope this makes sense ^^
Bonus for me (no need to implement it in this PR): a gradient fill option for this texture type (border and fill separate from each other) and outline mode being either hard outline or outer glow.

For me personally, I try to avoid using built-in textures unless they are saved to disk. I work on big project and a lot of built-in textures where isnt necessary can start slowing down the project. I believe that, in order to get good results with these textures it's likely that they will be upscaled in size, so I think UX wise it'd be better if the creation of the texture prompts a file save dialogue like visual shader.

This would require saving the rasterized image, which is a destructive operation (you can't change its parameters afterwards, it's just an image like any other).

Procedural textures are only rendered when they are first used in a scene, so having procedural textures in scenes that aren't loaded shouldn't impact project load times.

This not very good imo, causes the game to randomly stutter if you raster at decent resolution.
I think saving the resource to disk by default creates a better workflow for the poor tech artist that will come later in the pipeline to fix stutters

This not very good imo, causes the game to randomly stutter if you raster at decent resolution.
I think saving the resource to disk by default creates a better workflow for the poor tech artist that will come later in the pipeline to fix stutters

Saving the .tres file alone isn't enough, you need to save the rasterized image alongside. This could be done in a non-destructive manner if we have a prerendering system similar to what fonts do already, but it's quite a lot of added complexity.

What I'm saying is that if these textures are saved separately by default, it will be easier for an eng/tech artist to write a tool (or modify the engine) in a way that at export-time the textures are rasterized. Otherwise if you embed them in the scene you'll have to go fish for textures everywhere, it's not practical.

I think such caching option should be implemented on Texture level, so that it would be usable for noise and gradient textures as well. But likely it's an idea for another proposal.
@Calinou thanks for noticing bleeding problems!

@QbieShay: I think it'd be good if it was easily compatible with pixel art and lowpoly games

Do you mean using something like Bresenham's algorithm? Need to think how to approach this. Currently the texture is drawn by checking if each pixel is inside a polygon (fill or border polygon). Triangulation and filling might be faster and allow for better outline, but needs testing.

a gradient fill option for this texture type

Texture filling would be more flexible and less hassle to implement (no need to duplicate GradientTexture2D functionality in its entirety). We'll see if it will be performant enough.

outline mode being either hard outline or outer glow.

Again with current approach might be a bit hard, but I'll make a note of this.
Thanks for feedback!

Before this gets too far along, I want to express two concerns:

This PR doesn't resolve https://github.com/godotengine/godot-proposals/issues/1126. The PR asks for a node to draw simple shapes. This PR is a tool for authoring textures. Textures come with a huge drawback in that they create aliasing, the texture size needs to be adjusted for scale and they add a lot of overdraw compared to using a shape.
To me, this sort of thing should always be an addon. It is just way too specific and niche to be core. Godot is not an asset authoring tool and artists are always going to need more flexibility and control than we can provide. We should let the content authoring tools focus on content authoring while we focus on the game engine.

I think the idea is really cool. But this should be a GDExtension or a GDscript plugin instead of a part of the engine

I don't think we need to implement a "Shape2D node" like the proposal suggests (moreover it's a really vague proposal from 3 years ago). What matters is that the use cases are addressed. Whether the use case is through a Sprite2D with a ShapeTexture2D, or a Shape2D node, I don't think matters here.
Also I think the suggestion that it's in addon territory is shaky, to me it looks similar to the inclusion primitive meshes, which are too useful to discard.

Alright, fair enough @clayjohn ! I came in the conversation late and I didn't really ask myself "should this be core"
However i still think out of this discussion we need to discuss the rasterization of procedural textures for export, which would benefit also a plugin like this one.

So is this PR dead or can we expect this to get merged at some point? Would be very welcome for my current project...

@PPillau this is still being discussed based on the comments above, and still needs some work even if no changes to the approach was decided on by the team

I think the general vibe is that this is not going to be part of the core, so I didn't pursue this further - the proposal that this is based on, altough approved at some level, is very vague in details. If somebody wants to turn this into a plugin feel free to do so.
I also tested making similiar feature in gdscript by generating svg snippets and loading them in via Image.load_svg_from_string which gave some promising results but had some problems like no control over antialiasing and white fringe around the shape (I think the latter might be a bug, needs more testing...).

and white fringe around the shape (I think the latter might be a bug, needs more testing...)

https://github.com/godotengine/godot/issues/82579

@MewPurPur thanks, I actually just discovered the solution myself. I think the documentation for load_svg_from_string should hint about using fix alpha border if this is intented behavior (I wrote about it in the rocket chat just now - link ).

Hey! For anybody interested in this feature, I implemented ShapeTexture2D as a gdscript addon - check the repo here. I'll submit it to the asset library soonish when I feel that the feature set/api is good.
@PPillau Since you said that you could use this in your current project, I would love to hear if the addon is suitable for your use cases :)


Pass environment variable to databaseName input parameter

Is it possible to pass such string "$($env:COMPUTERNAME)_ABC" to databaseName parameter? I tried many things:

"$($env:COMPUTERNAME)_ABC" with quotas and without
"$(dbprefix)_ABC" with and without quotas, where $dbprefix defined as included template variable

name: dbprefix
value: $env:COMPUTERNAME
or
name: dbprefix
value: $($env:COMPUTERNAME)

My template 'lifecycle' is following:
===azure-pipelines.yml:
name: ...
trigger: ...
variables: ...
stages:

template: ..\tmplt-stage-deploy2dev.yml
parameters:
stagename: Deploy2Dev
sqlsteps:
- template: tmplt-steps-sql.yml

===tmplt-steps-sql.yml:
steps:

task: SqlDacpacDeploymentOnMachineGroup@0
displayName: 'Deploy using : dacpac'
inputs:
dacpacFile: '$(Pipeline.Workspace)\sql\ABC.dacpac'
ServerName: $(sqlserver)
databaseName: "$(dbprefix)_CDB_Portal"

===tmplt-stage-deploy2dev.yml:
parameters:

name: stagename
name: sqlsteps
type: stepList

stages:

stage: ${{ parameters.stagename }}
variables:

template: tmplt-var-dev.yml
jobs:
template: tmplt-job-deploysql.yml
parameters:
jobname: deploysql
environment: 'dev1'
sqlsteps: ${{ parameters.sqlsteps }}
...

===tmplt-job-deploysql.yml
parameters:

name: jobname
type: string
name: environment
type: object
name: sqlsteps
type: stepList

jobs:

deployment: ${{parameters.jobname}}
displayName: 'Deploy SQL'
workspace:
clean: outputs
environment: ${{ parameters.environment }}
strategy:
runOnce:
preDeploy:
steps:
- checkout: self
deploy:
steps:
- download: none
- task: DownloadPipelineArtifact@2
displayName: Download SQL
inputs:
source: 'current'
artifact: 'sql'
path: '$(Pipeline.Workspace)\sql'
- ${{ each step in parameters.sqlsteps }}:
- ${{ each pair in step }}:
${{ pair.key }}: ${{ pair.value }}

===tmplt-var-dev.yml
variables:

name: sqlserver
value: 'server'
name: dbprefix
value: $($env:COMPUTERNAME)

The output is always like that:
##[debug]Executing command: C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\Common7\IDE\Extensions\Microsoft\SQLDB\DAC\150\SqlPackage.exe  /SourceFile:"C:\DeployAgent_work\1\sql\ABC.dacpac" /Action:Publish /TargetServerName:"server" /TargetDatabaseName:"$($env:COMPUTERNAME)_ABC" /p:BackupDatabaseBeforeChanges="True" /p:ScriptDatabaseOptions="False" /p:DropIndexesNotInSource="False"

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

ID: 6b7e6f9e-5aae-51b7-b237-282d0af40018
Version Independent ID: 33966728-39bc-37f6-af71-e90564fb7193
Content: WinRM SQL Server DB Deployment task - Azure Pipelines
Content Source: docs/pipelines/tasks/deploy/sql-dacpac-deployment-on-machine-group.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

UPD: using $(Agent.MachineName) instead of $env:COMPUTERNAME solve the my problem. But still interesting is it even possible to use environment variables in this place?

@VerdonTrigance -- thank you for your question and update. It looks like you're working through an issue with your scenario or implementation, rather than an issue with the documentation. Here are a couple of options where you might consider asking your question:

Azure DevOps Support Bot
Azure DevOps on Stack Overflow

I am closing this issue now.  You are welcome to @ mention me for any followup.
We hope to hear from you again.

@VerdonTrigance -- thank you for your question and update. It looks like you're working through an issue with your scenario or implementation, rather than an issue with the documentation. Here are a couple of options where you might consider asking your question:

Azure DevOps Support Bot
Azure DevOps on Stack Overflow

I am closing this issue now. You are welcome to @ mention me for any followup.
We hope to hear from you again.

Of course I read these topics, but they didn't help! They are talking that I may use it, but as you see it doesn't work. Probably it does not fit actual thing?


Testcase 41A with loinc code 55752-0

In testcase 41A we need to translate Loinc code 55752-0 to 'Klinische gegevens'. In the fhir message this loinc code is sent without any description.
At the moment we aren't able to show 'Klinische gegevens' in our GP soft because we are applying following rules.

If a loinc code is used we show the dutch translation if available
If the translation isn't available we show the description which is sent in the fhir message
If none of the above is applicable we show the code

For test case 41A this means we show code 55752-0 which I think is not clear for a GP. What should we do? We want to avoid hard coded descriptions for loinc codes in our code as these can change. How should we solve this?

Any update on this question? @bdc-ehealth

@ValerieNexuz ,
The only solution conforming with the rules, is that we should notify the sender that he should provide a description if the LOINCcode is not in Retam. If you need a technical solution, you could request a label from a freely available LOINC terminology server such as tx.fhir.org, but this should only be a last (non normative) resort.
The sad thing is that we cannot check this in an automated way, because we have no access to an API for ReTaM.


command?

Describe Your Problem:
Is it possible to trigger my tapo p110 with a curl command?
Switching on/off in the homebridge UI is working.
Logs:

Plugin Config:
{
    "name": "Tapo P110",
    "username"<EMAIL_ADDRESS>    "password": "Passw0rd",
    "devices": [
        {
            "name": "TapoP110",
            "host": "<IP_ADDRESS>",
            "type": "Plug",
            "updateInterval": 30,
            "timeout": 2
        }
    ],
    "platform": "TapoP100"
}

Screenshots:

Environment:
Homebridge-IP=
http://<IP_ADDRESS>:8581/
no docker

Plugin Version: homebridge-tapo v1.4.17
Homebridge Version: v1.6.1
Node.js Version: v16.20.1
NPM Version: 9.8.1
Operating System: Debian GNU/Linux 11 (bullseye)

@dukeluke not with an simple curl command. You would need to authenticate first in order to create the key pair needed for encrypting the request payload before using curl to talk to the REST API.


linkProp issues

Hi,
I just tried this plugin but got this error by using the example :
Property or method "linkProp" is not defined on the instance but referenced during render. Make sure that this property is reactive, either in the data option, or for class-based components, by initializing the property
&
"Error in render: "TypeError: _vm.linkProp is not a function"
I'm not really sure what I'm doing wrong, any one could help me?
Thanks!

Hi @luminous8 - did you find the solution to your problem? Would it help anybody else that may come across this issue?

Yes! Actually I didn't get it right how to properly use it. I get it by looking at the example.
Maybe in the doc you should precise that even we import like this import VueBreadcrumbs from 'vue-breadcrumbs' we do not need to import it as a component as it is already declared and simply use in with this tag in your code


Allow response data to be used as tag values

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
UI Feature
🔖 Feature description
I was hoping, if there was a way, to allow the repose of a request to be parsed and then turned in tags or even tag values. For instance, I have a simple Python application that returns the status of all/one of my access points using the router's API (Synology SRM API if you were wondering). A sample response looks like this
{
  "node": {
    "name": "Office AP",
    "is_re": true,
    "model": "MR2200ac",
    "uptime": "18h 13m 41s",
    "uplink": {
      "is_ethernet_connected": true,
      "wireless_uplinks": []
    },
    "wifi_radios": [
      {
        "band": "2.4G",
        "current_channel": 7,
        "current_frame_rate_rx": 0,
        "current_frame_rate_tx": 20,
        "current_rate_rx": 0,
        "current_rate_tx": 4909
      },
      {
        "band": "5G-1",
        "current_channel": 132,
        "current_frame_rate_rx": 34,
        "current_frame_rate_tx": 245,
        "current_rate_rx": 6019,
        "current_rate_tx": 299481
      },
      {
        "band": "5G-2",
        "current_channel": 36,
        "current_frame_rate_rx": 7,
        "current_frame_rate_tx": 30,
        "current_rate_rx": 542,
        "current_rate_tx": 5756
      }
    ],
    "network_status": "online",
    "node_status": "good",
    "node_status_msg": "good"
  }
}

✔ Solution
I would like below on my dashboard

where the values of the fields are being populated from the response data.
❓ Alternatives
No response
📝 Additional Context
No response

If there's a way to achieve this via Push monitor type I would be really appreciative if you could me in the right direction 😄

@george-dadeboe
I think this is a duplicate of https://github.com/louislam/uptime-kuma/issues/686
If you agree, could you please close this Issue, as duplicates only create immortal zombies and are really hard to issue-manage?
If not, what makes this issue unique enough to require an additional issue? (Could this be integrated into the issue linked above?) ^^

@CommanderStorm I believe you are right. will close this issue.


[flutter_releases] Flutter beta 2.8.0-3.1.pre Framework Cherrypicks

Flutter beta 2.8.0-3.1.pre Framework
Scheduled Cherrypicks

commit: f4f23ecb5

Gold has detected about 2 new digest(s) on patchset 3.
View them at https://flutter-gold.skia.org/cl/github/93448

Landing with Mac module_test_ios, the fix for the test is landing in https://github.com/flutter/flutter/pull/93556


Korki copying my changes :')

https://github.com/korki43/gauth-export/pull/3
This PR basically grab changes from Korki, who are basically taken from that PR.

Nothing interesting to grab. Closing.


doh-server: Got tls remote error: bad certificate

I self-signed the cert, but still got error msg
http: TLS handshake error from xxx.xxx.xxx.xxx:64532: remote error: tls: bad certificate

Have you installed and trusted your self-signed certificate on your client machine?
Anyway, if you just need to obtain a valid and free TLS certificate, you may want to use Let's Encrypt.

I get the cert and key from https://www.sslforfree.com/, but doh-server service can't start

I get the cert and key from https://www.sslforfree.com/, but doh-server service can't start

Would you please give me detailed error message? I guess there is a problem in the code.
I usually use doh-server behind an Nginx or Caddy load balancer, so I configure TLS there and don't use this feature often.

How to see the error message?

How to see the error message?

Use -verbose to get detailed log, with any possible error messages printed out.

If I use Nginx with https, then proxy_pass to doh-server port 80?

You may have doh-server listen on any port. (e.g. [::1]:64532 or <IP_ADDRESS>:64532.)
Just leave cert and key as empty so it would be a plain HTTP service.

Should I forward header information or other details?

If you want to use the ECS feature (required for GeoDNS), you should correctly pass X-Forwarded-For or X-Real-IP. Otherwise, you don't need to forward anything.
An example configuration:
location /dns-query {
    proxy_pass http://localhost:64532;
    include /etc/nginx/proxy_params;
}

Sorry, where can I print out the detail
service doh-server ?

Sorry, where can I print out the detail
service doh-server ?

If you use systemd to launch the service, check your journalctl -u doh-server.
If you directly execute doh-server to launch the service, check your terminal output.

The TLS private key is not accessible to the program.
There are several possible solutions:

Edit /usr/lib/systemd/system/doh-server.service and change the line User=nobody to any other user name that can access the key. (e.g. root is a possible choice, although not the best choice)
Or, change the owner and permission of your key so doh-server can access it.

If your machine is shared with other users, you may need to pay attention to the permission issue and make sure the private key is inaccessible to other people.
Also, if you use Nginx, you don't need to configure TLS at doh-server side. Leave them empty is okay.

I changed to root, and the crt&key also root:root

I use caddy server as HTTPS server, then proxy it to doh-server

If this works, that's okay.

I tried step 1, changed from nobody to root, and the crt&key also root:root, failed

The major reason is you didn't put a full certificate bundle with CA and intermediate certificates. (Certificate order)

I've the full certificate bundle, where should I put, in cert or key

I've the full certificate bundle, where should I put, in cert or key

Did your Caddy work? You can simply proxy Caddy to doh-server, and don't configure TLS here.

I tried Nginx but failed, Caddy works !

That's great!

Caddy's Let's Encrypt integration can help you automatically renew your certificate.
You may try that.
Here is an example Caddyfile configuration with automatic Let's Encrypt integration:
https://dns.dns-over-https.com {
        log     syslog
        errors  syslog
        gzip
        proxy   /dns-query      http://[::1]:8053 {
                transparent
        }
}


Terminal access for AGL

I'm running AGL Linux on Intel Minnowboard Max. Want to know how to access the terminal in AGL. I can now only access the UI. So would like some help in this regard.
thanks in advance

The easiest is to use a serial cable and to connect with a serial console
emulator such as minicom.
Alternative is to ssh in the unit as root.
Obviously in the last option requires to know the up address allocated by
the dhcp.
Personally I run a second eth interface with a USB adaptor on my pc with a
dnsmask acting as shop server for that interface,  so it's easy to control
which up address is given to the Minnow.
There is no display of the up address in the AGL menus. There is an open
bug on that  topic. Contributions are welcome.
Dominig
On 30 Apr 2018 06:35, "Prasad PR"<EMAIL_ADDRESS>wrote:
I'm running AGL Linux on Intel Minnowboard Max. Want to know how to access
the terminal in AGL. I can now only access the UI. So would like some help
in this regard.
thanks in advance
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/automotive-grade-linux/docs-agl/issues/114, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABI0oPUoEvCGJR8VHSUJVproUx4lsmvCks5ttqKYgaJpZM4TsJIi
.


Stopping multiple items from being dragged.

I'm having some issues on touch devices when multiple items are accidently dragged at the same time especially when one or more of them are cloned. Items can end up stuck on the screen along the drag path. Is there any way to stop a second item from being dragged while a drag is already taking place?
Thanks.

Being able to track multiple items should not be possible. I'll look into it, thanks for reporting


removed sample for standard cluster as it was out of date

Resolves: https://github.com/samsung-cnct/k2/issues/115
Sample configs were not being maintained and were out of date/misleading.

LGTM, is a documentation change, and tests are failing due to a known bug: 18:54:19 fatal: [localhost]: FAILED! => {"changed": true, "cmd": ["kubectl", "--kubeconfig=/var/jenkins_home/jobs/k2-aws-pr/workspace/ci-pr-215-aws/ci-pr-215-aws/admin.kubeconfig", "apply", "-f", "/var/jenkins_home/jobs/k2-aws-pr/workspace/ci-pr-215-aws/ci-pr-215-aws/fabric/config.yaml"], "delta": "0:00:00.175496", "end": "2017-02-23 18:54:19.877673", "failed": true, "rc": 1, "start": "2017-02-23 18:54:19.702177", "stderr": "Unable to connect to the server: EOF", "stdout": "", "stdout_lines": [], "warnings": []}


affected/package:

What version of Go are you using (go version)?

$ go version
go version go1.18 linux/loong64

Does this issue reproduce with the latest release?

我使用了如下编译命令：
go build -x -buildmode=c-shared -o hello.so hello.go
报错如下：
-buildmode=c-shared not supported on linux/loong64
于是我修改1.18源码中部分相关文件，让其支持loong64
最终报错停留在了如下：
/usr/share/ubxgo/go/pkg/tool/linux_loong64/link -o $WORK/b001/exe/a.out -importcfg $WORK/b001/importcfg.link -installsuffix shared -buildmode=c-shared -buildid=VjSGCzgmjjJmng4hvW1p/WO5bAXmxALTE53mSBcMM/lLTTU0AieOjUvcGiNFbI/VjSGCzgmjjJmng4hvW1p -extld=gcc /home/uos/.cache/go-build/8a/8a684f2f6e11d470d52b1497952c1f44669b428ec4b5cbe654f378ab7a078f2b-d
#command-line-arguments
/usr/share/ubxgo/go/pkg/tool/linux_loong64/link: running gcc failed: exit status 1
collect2: fatal error: ld terminated with signal 11 [段错误]
compilation terminated.

使用的demo代码如下：
package main
import "C"
//export hello
func hello(value string) *C.char {
return C.CString("hello"+value)
}
func main() {
}
What did you expect to see?
loongarch上是不支持将go编译成so吗？
What did you see instead?

@golang/loong64

I think for producing shared objects, some kind of real relocation record support is needed, but unfortunately the current stack-machine-based relocations of LoongArch are poorly supported at best.
I think it's probably better to wait until https://github.com/loongson/LoongArch-Documentation/pull/50 is finalized and merged.

有任何进展吗？loongson/LoongArch-Documentation#57 已经关闭

有任何进展吗？loongson/LoongArch-Documentation#57 已经关闭

I think it's better to write in English...
Translation for non-Chinese speakers:

Any progress? loongson/LoongArch-Documentation#57 is already closed.

I don't know if the Loongson people have plans for this cycle; pinging others in @golang/loong64. I myself don't think I have enough time to make any promise though.


How to support login-by-SMS

Hi thanks for the project! I need to implement login by SMS, and I also want to use the oauth2 system (e.g. automatically handle refresh token, access token, etc).
Therefore, I wonder how should I implement it?
One naive way may be use token exchange. The procedure is like: User sends HTTP request with username + the sms verify code they received. Then my code checks that the code is right. If so, my code use admin account to interact with spring-authorization-server using the "token exchange feature", and thus get a "refresh token" for that specific user.
However, I am not sure whether this naive way is correct or not. In addition, https://github.com/spring-projects/spring-authorization-server/issues/60 seems not implemented yet though...

I see: it seems that I can do custom grant_type.


Support dynamic URI building for the upstream service

Is your feature request related to a problem? Please describe.
I would like to be able to build upstream service URI based on components of the request. For instance given a request with the path /api/{serviceName} I would like to build the upstream URI using the serviceName piece like so:
Map ports = Map.of("service1", 8080, "service2", 8081);
URI upstreamUri = "http://host:" + ports.get(serviceName) + "/";

Describe the solution you'd like
For the fluent routes (RouteLocatorBuilder) one solution might be to introduce another method e.g. uriBuilder which will have access to request details:
@Bean
public RouteLocator routes(RouteLocatorBuilder builder) {
  return builder.routes()
    .route("dynamic-route", predicate -> predicate.path(/api/**))
    .uriBuilder(request -> {
      String serviceName = "todo"; //extract service name from the reqeust's path
      return "http://host:" + ports.get(serviceName) + "/"
    })
    .build();
}

Describe alternatives you've considered
I couldn't find any alternatives. The closest I could find was the RewritePath GatewayFilter but it allows only for rewriting the path part of the URI.

I just stumbled on https://github.com/spring-cloud/spring-cloud-gateway/issues/276 and I think undocumented AbstractChangeRequestUriGatewayFilterFactory. It looks like it solves the problem stated above.

@dharezlak yes I think that is the best solution at the moment.  Unfortunately this means the URI of the route is pretty much useless which is kind of nasty.  I think we need a cleaner solution to this problem.


Error when checking VGG16 input shape with include_top=False

I'm trying to use Keras' implementation of VGG16 with grayscale images, channels first.
My training data is shaped as follows:
x_train shape: (7961, 1, 128, 128)
y_train shape: (7961, 1)
X_test shape: (1991, 1, 128, 128)
y_test shape: (1991, 1)

When I build my model:
model_vgg16 = VGG16(include_top=False,
                        weights=None,
                        input_tensor=None,
                        input_shape=(1, 128, 128),
                        pooling=None,
                        classes=CLASSES)

it throws this error:
File "/Library/.../keras/applications/vgg16.py", line 101, in VGG16
    include_top=include_top)
  File "/Library/.../keras/applications/imagenet_utils.py", line 116, in _obtain_input_shape
    '`input_shape=' + str(input_shape) + '`')
ValueError: The input must have 3 channels; got `input_shape=(1, 128, 128)`

This seems odd since I am using include_top=False and weights=None but it still seems to expect a 3 channel tensor and is passing my shape through imagenet_utils.py, which seems inflexible in allowing for anything < 3 channels (see _obtain_input_shape).
Is this desirable?
If I keep input_shape=None then I get a shape mismatch error at the beginning of training epoch 1.
Keras 2.0.4
Python 3.5

Although I do think that the flexibility is desirable, I just want to quickly point out for anyone else who reaches this thread what the argument, 'input_shape' specifies for vgg16:
       input_shape: optional shape tuple, only to be specified             ifinclude_topis False (otherwise the input shape             has to be(224, 224, 3)(withchannels_lastdata format)             or(3, 224, 224)(withchannels_firstdata format).             It should have exactly 3 input channels,             and width and height should be no smaller than 48.             E.g.(200, 200, 3) would be one valid value.

I'm having the same problem, but on TF 1.10.0
This is my line of code:
model = tf.keras.applications.VGG16(weights='imagenet',include_top=False,input_shape=(48,48,3))
And I get this:
ValueError: The input must have 3 channels; got input_shape=(48, 48, 3)`
Maybe I missed something? I admit I kind of new to using TF. I also hope this is the right place to post about the issue (Because I am not using Keras library directly, but rather through TF integration)

@amwfarid As you have set include_top=False, you must have input_shape=(3,48,48)


Support suites

Description
Add support for Suites
Motivation and Context
Suites allow

tests to be grouped
perform a common setup and teardown for a group of tests

How Has This Been Tested?
With example tests
Types of changes

New feature for supporting suites. Done backwards compatible.

Checklist

[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes if appropriate.
[x] All new and existing tests passed.

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "",
    "component": "backend",
    "pr-id": "1966",
    "commit-sha": "d5896be05216a5dff473aab41930f96c116e5993",
    "pipeline": "master",
    "org": "prismacloud/pcn"
  },
  {
    "project": "",
    "component": "apocheck",
    "pr-id": "4",
    "commit-sha": "fa91622f6283f00d734930a2d78382d342c5cb40",
    "pipeline": "master",
    "org": "PaloAltoNetworks"
  }
]


Project References Regression starting in Nightly 3.9.0-dev.20200313

Project References do not appear be honored in tsserver starting in Nightly 3.9.0-dev.20200313.  This results in an error in Visual Studio code when importing from 'some-pkg/dist/file' when that file has not yet been built.

TypeScript Version:  3.9.0-dev.20200313 and later

Search Terms:
references tsserver
Code
Minimal reproduction project attached. repro.zip
This seems to be an issue with tsserver, and so I can only reproduce it in Visual Studio Code (not with tsc -b).
The error is a regression starting in version 3.9.0-dev.20200313.  There was no error in 3.9.0-dev.20200312 and previous.
To reproduce, use attached repro.zip, here are simple instructions and the screenshot of the result.
unzip repro.zip
cd repro-ts-bug
yarn install
code .
code src1/one.ts

See issue in 3.9.0-dev.20200313:

This should work because of a project reference to src2/tsconfig.json from src1/tsconfig.json, and it does work in the previous build of typescript.

You can see this is a regression.  Installing the previous version and reloading the window demonstrates no error.
yarn add<EMAIL_ADDRESS>
And reload window.  No issue in 3.9.0-dev-20200312

Expected behavior:
No error, as in second screenshot
Actual behavior:
Error:
Cannot find module '../dist2/two' or its corresponding type declarations.

Playground Link:
Unable to demonstrate project reference issues with playground (as far as I know)
Related Issues:

@sheetalkamat, it appears this problem began with the merge of PR #37370, "Move useSourceOfProjectReferenceRedirect to program so other hosts can use it too, enabling it for WatchHost" with commit a76a16696d7
Is this a regression, or am I misunderstanding something about how this should work?

Don't repro the issue at all:

Hmm, indeed, I can only reproduce it on this machine.  It didn't occur to me to try other machines - sorry about that...
But something has definitely changed in 20200313, though, because on this machine every version before that works, but every version after does not.
I will try to see if I can see why.  Are there any hints you can give that might explain why this might work on one machine, but not on a nearly identical machine?

you could add --traceResolution and see whats going on in tsserver log..

Ah, it was a vscode extension that I long forgot was there; I should have thought of that.  It appears to be interfering in some way that breaks after 20200313.  I don't need the extension so I am just removing it rather than look deeper.  Thanks for the pointer, and sorry about wasting your time.

@warrenfalk glad we got it figured out; thanks for following up


feat(operator): add withLatestFrom

closes #209

I'd add a bit more tests to the spec, but otherwise looks good.

@staltz ... I know I'm waiting for that Virtual Time scheduler from @trxcllnt ... then we're going to port basically every test from RxJS 2/3 over.


demo: leaff demo

Using #9382 as a somewhat random example

Oh, that's quite neat!

I think it would be helpful to group by file there, rather than repeating the @@ Mathlib.LinearAlgebra.Finsupp @@ heading three times.

I think it would be helpful to group by file there, rather than repeating the @@ Mathlib.LinearAlgebra.Finsupp @@ heading three times.

Yeah there are two competing heuristics here, group by file or put "more important changes first". I opted for the latter as it seemed more helpful to me for long diffs, but if there was consensus for the other one I don't mind changing. Some PRs generate a lot of proof changes though which can be spammy otherwise

The one in particular that confused me was the attribute instance added to LinearMap.CompatibleSMul.finsupp_dom not being listed right next to added LinearMap.CompatibleSMul.finsupp_dom, so maybe my request was an x/y problem.

So currently there is a hand written list of priorities which controls order diffs are shown in https://github.com/alexjbest/leaff/blob/9dea5f69fb27cad88cfeef57b6af8489f8b40176/Leaff/Diff.lean#L189C22-L190C1
Things are ordered by priority then grouped by module. At the moment I made all priorities distinct, but I can definitely see the argument for changing that so that things like adds and removals and attributes still come first (grouped by module), and more boring stuff later, but with less new module headers the whole time than there are right now

I think I might be asking for something more basic like "don't list additions of attributes if the declaration itself is new"

I think I might be asking for something more basic like "don't list additions of attributes if the declaration itself is new"

This is another design decision I was indeed unsure of, I already implemented the reverse direction of not listing attribute removals when the decl is deleted. But I'm not yet sure what is most useful. For example if your proof breaks at a simp call, it's very helpful for the diff tool to tell you clearly which simp lemmas were added or removed without you having to think further

@alexjbest am I right that some form of this feature meanwhile has been implemented in CI and therefore this PR can be closed?
Please reopen it if I missunderstood! (for example as draft or "WIP" if not ready for review)

Well the pr was a demo of something that has not been merged as far as I know. Closing this makes sense #8479 was the actual pr


Căinelu De Sus, Hunedoara (Băița) - Is not written correctly

ENVIROMENT: ui.orase
URL: https://peviitor-ro.github.io/ui.orase/
Browser: Chrome
Device: Laptop
OS: Windows 7
STEPS TO REPRODUCE:

Open URL in browser
Type "Căinelu De Sus" in the search bar
Select Căinelu De Sus, Hunedoara (Băița) from the list

EXPECTED RESULTS:
The name of the locality written in the search bar is written correctly based off the law nr. 167/1929
https://legislatie.just.ro/Public/DetaliiDocument/189
ACTUAL RESULTS:
The name of the locality written in the search bar is not written correctly based off the law nr. 167/1929

Fixed! Poti retesta


welle-io: init at 1.0-rc1

Motivation for this change
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

Thanks a lot for reviewing. I updated the PR.

Thank you!


Updated AI template

Here is an updated template for the actual version of Adobe Illustrator.
I added Master Icon as the symbol, which is placed into all required sizes.
So You just open AI template, put your icon into one symbol and it's applied automatically for all sizes, easy :)

One more note: its as a separate file, coz not everyone is using the latest version of Adobe Illustrator, my file is saved in lates AI 2020.

A little fix - properly named slices :)


Date.prototype.getYear

You are using Date.prototype.getYear which is obsolete and not supported by standar js engines, in my particular case https://github.com/cesanta/v7/. Please use Date.prototype.getFullYear as suggested in https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Date/getYear

Pretty sure we'd take this as a PR that passed all the tests.
In case you haven't, note that you can retrofit your date object like this:
Date.prototype.getYear = function(){return this.getFullYear() - 1900;};

Where specifically do you see this in moment?

Yeah, I tried a couple ways to reproduce this, using moment().year() and moment().get('year') and moment().set('year',2016) all very clearly fall back to the FullYear function.
Adding and subtracting will change years to months before performing the operation.
Formatting will fall back to the tested get functions internally.
Don't know where this would be.

Ha, I didn't even think to verify that we actually use it. Switching this to troubleshooting.

Yeah, my quick review of the code agrees with Maggie's.

Going to close this for lack of response. @cancerberoSgx if you have more information on where you think Moment is using that function (and have confirmed you're using the latest version), then let us know and we'll reopen.

I double check and I was using an old moment version (1.1.6). The fault was that working with npm I was doing "npm install momentjs" instead of "npm install moment". https://www.npmjs.com/package/momentjs has that old version, I will ask the author to change the name or remove this package cause is confusing. Thanks and sorry for the trouble.


Cross-Site Request Forgery (CSRF) in microweber/microweber

I have tried to email multiple times per security.md
https://huntr.dev/bounties/9d3d883c-d74c-4fe2-9978-a8e3d1ccf9f3/

@peter-mw any update?

hi, the issue is fixed


Training script crashes at the end of training (saving checkpoints work)

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Expected Behavior
Complete training without crashing
Current behaviour
When the training is complete, the script crashes with the following error:
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: -9) local_rank: 0 (pid: 357394) of binary: /home/wgc/miniconda3/envs/axolotl/bin/python
Traceback (most recent call last):
  File "/home/wgc/miniconda3/envs/axolotl/bin/accelerate", line 8, in <module>
    sys.exit(main())
  File "/home/wgc/miniconda3/envs/axolotl/lib/python3.10/site-packages/accelerate/commands/accelerate_cli.py", line 47, in main
    args.func(args)
  File "/home/wgc/miniconda3/envs/axolotl/lib/python3.10/site-packages/accelerate/commands/launch.py", line 977, in launch_command
    multi_gpu_launcher(args)
  File "/home/wgc/miniconda3/envs/axolotl/lib/python3.10/site-packages/accelerate/commands/launch.py", line 646, in multi_gpu_launcher
    distrib_run.run(args)
  File "/home/wgc/miniconda3/envs/axolotl/lib/python3.10/site-packages/torch/distributed/run.py", line 785, in run
    elastic_launch(
  File "/home/wgc/miniconda3/envs/axolotl/lib/python3.10/site-packages/torch/distributed/launcher/api.py", line 134, in __call__
    return launch_agent(self._config, self._entrypoint, list(args))
  File "/home/wgc/miniconda3/envs/axolotl/lib/python3.10/site-packages/torch/distributed/launcher/api.py", line 250, in launch_agent
    raise ChildFailedError(
torch.distributed.elastic.multiprocessing.errors.ChildFailedError:
=======================================================
axolotl.cli.train FAILED
-------------------------------------------------------
Failures:
  <NO_OTHER_FAILURES>
-------------------------------------------------------

Interestingly, saving checkpoints during training all works. Only crashes at the very end.
Steps to reproduce

Download library and setup
Start training with any data
Wait for training to complete

Config yaml
No response
Possible solution
No response
Which Operating Systems are you using?

[X] Linux
[ ] macOS
[ ] Windows

Python Version
3.10
axolotl branch-commit
main
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this bug has not been reported yet.
[X] I am using the latest version of axolotl.
[X] I have provided enough information for the maintainers to reproduce and diagnose the issue.

I'm not using FSDP. I'm using the default config taken from examples/mistral, it has fsdp unset.
Could there be a bug in the code somewhere that's using unnecessary RAM? I have 256GB of CPU RAM, my dataset is 1GB, and I'm training a 7B model. It's hard to fathom in any world where I will run out of RAM.


Simplify driver-loader init container usage for users

Motivation
I am using your Helm chart with the driver-loader init container. I would have preferred to have the init container part of the template with most of the attributes already set instead of filling the extraInitContainers with the required values, i.e:
image:
  repository: falcosecurity/falco-no-driver

extraInitContainers:
  - name: driver-loader
    image: docker.io/falcosecurity/falco-driver-loader:latest
    imagePullPolicy: Always
    volumeMounts:
      - mountPath: /host/proc
        name: proc-fs
        readOnly: true
      - mountPath: /host/boot
        name: boot-fs
        readOnly: true
      - mountPath: /host/lib/modules
        name: lib-modules
      - mountPath: /host/usr
        name: usr-fs
        readOnly: true
      - mountPath: /host/etc
        name: etc-fs
        readOnly: true
      - mountPath: /root/.falco
        name: driver-fs
    env:
      - name: FALCO_BPF_PROBE
        value:

extraVolumes:
  - name: driver-fs
    emptyDir: {}

extraVolumeMounts:
  - mountPath: /root/.falco
    name: driver-fs

ebpf:
  enabled: true

That way I would not have to worry about the volumes mounts or other changes for the forthcoming chart releases.
Feature
Add the driver-loader init container fields inside the daemonset template and enable it with a simple feature flag like:
driverLoader:
  enabled: true
  image:
    registry: docker.io
    repository: falcosecurity/falco-driver-loader
    tag: 0.31.1
    pullPolicy: IfNotPresent

# Still, there is the need to set the Falco container pod:
image:
  repository: falcosecurity/falco-no-driver

/milestone falco-chart-2.0.0


How to sort using scan ?

is there any option to sort items according to some criteria using scan?

DynamoDB doesn't support ordering for scan.  You'll have to order the results after or use query.


Github username isn't needed for github-pages:deploy

The ng github-pages:deploy command currently asks for the username to create the remote url, but this can be obtained from the github repo create call.

Closed via https://github.com/angular/angular-cli/pull/4385.


[BUG] NodePublishVolume RWX CSI realpath failed to resolve symbolic links on microk8s

Describe the bug
The volume is created and attached (Longhorn UI tells me this)
But the pod where the volume is mounted does not start and has this lines in the log:
MountVolume.SetUp failed for volume "pvc-bee93258-1cfb-4d8a-a501-d34194e7a3fd" : rpc error: code = Internal desc = NodePublishVolume: failed to prepare mount point for volume pvc-bee93258-1cfb-4d8a-a501-d34194e7a3fd error exit status 1

To Reproduce
Steps to reproduce the behavior:

Install with helm chart
Create storage class

kind: StorageClass
apiVersion: storage.k8s.io/v1
metadata:
  name: longhorn-histdata
provisioner: driver.longhorn.io
allowVolumeExpansion: true
reclaimPolicy: Delete
volumeBindingMode: Immediate
parameters:
  numberOfReplicas: "2"
  staleReplicaTimeout: "2880"
  fromBackup: ""
  dataLocality: "best-effort" #Data should be where its needed
  replicaAutoBalance: "least-effort" #Data is not critical and can be redownloaded

Create PVC

apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: pvc-hist-data
spec:
  accessModes:
    - ReadWriteMany
  volumeMode: Filesystem
  volumeName: pv-hist-data
  resources:
    requests:
      storage: 20Gi
  storageClassName: longhorn-histdata

Mount with custom helm template:

        volumeMounts:
          - name: data
            mountPath: {{ default "/mnt/data/" .Values.persistence.mountPath | quote }}
      volumes:
      - name: data
      {{- if .Values.persistence.enabled }}
        persistentVolumeClaim:
          claimName: {{ .Values.persistence.pvcName }}
      {{- else }}
        emptyDir: {}
      {{- end -}}

Expected behavior
Volume is mounted and pod is starting
Log
If applicable, add the Longhorn managers' log when the issue happens.
Relevant log entries:
level=debug msg="Running nsenter command: nsenter [--mount=/proc/1/ns/mnt --net=/proc/1/ns/net --uts=/proc/1/ns/uts -- /bin/realpath -e /var/snap/microk8s/common/var/lib/kubelet/pods/13006cc4-eae5-433d-ab9c-dfe31da7f9a8/volumes/kubernetes.io~csi/pvc-bee93258-1cfb-4d8a-a501-d34194e7a3fd/mount]"
level=error msg="failed to resolve symbolic links on /var/snap/microk8s/common/var/lib/kubelet/pods/13006cc4-eae5-433d-ab9c-dfe31da7f9a8/volumes/kubernetes.io~csi/pvc-bee93258-1cfb-4d8a-a501-d34194e7a3fd/mount" error="exit status 1"
level=debug msg="mount point /var/snap/microk8s/common/var/lib/kubelet/pods/13006cc4-eae5-433d-ab9c-dfe31da7f9a8/volumes/kubernetes.io~csi/pvc-bee93258-1cfb-4d8a-a501-d34194e7a3fd/mount try reading dir to make sure it's healthy"
level=error msg="NodePublishVolume: failed to prepare mount point for volume pvc-bee93258-1cfb-4d8a-a501-d34194e7a3fd error exit status 1"
level=error msg="NodePublishVolume: err: rpc error: code = Internal desc = NodePublishVolume: failed to prepare mount point for volume pvc-bee93258-1cfb-4d8a-a501-d34194e7a3fd error exit status 1"

You can also attach a Support Bundle here. You can generate a Support Bundle using the link at the footer of the Longhorn UI.
Environment:

Longhorn version: 1.2.0
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): Helm Chart using ArgoCD
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: Microk8s 1.21

Number of management node in the cluster: 3
Number of worker node in the cluster: 3 (same nodes)

Node config

OS type and version: Ubuntu 20.04
CPU per node: 4
Memory per node: 16GB
Disk type(e.g. SSD/NVMe): SSD
Network bandwidth between the nodes: 500Mbit

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): Baremetall
Number of Longhorn volumes in the cluster: 1

Additional context
Add any other context about the problem here.

@GeroL
Can you double check if you hit this problem https://longhorn.io/kb/troubleshooting-none-standard-kubelet-dir/ ?

@joshimoo  Do you have any idea? The error comes from this line of code https://github.com/longhorn/longhorn-manager/blob/5da6ccfbba0c4ae441c6b5101ace541c42271f81/csi/nfs/nsenter.go#L143

Update @khushboo-rancher Will try to reproduce the issue in lab

As discussed with @PhanLe1010 the core issue seems to be the realpath lookup. @khushboo-rancher you can try installing microk8s via snap to see if this can be reproduced in that type of environment.

Thanks @joshimoo as we discussed in the team meeting yesterday, we want to wait for the user to answer this question first then @khushboo-rancher will spend effort reproduce the issue

Hi,
I managed to mount several RWO volumes (with deleted fsGroup entries in deployments due to another bug).
I did not test the RWX volumes again as I worked around it with simple hostpath volumes to unblock me.
I also do not have time this sprint to make another test round.

Hello @GeroL would you able to try again with kubelet --root-dir argument point to: /var/snap/microk8s/common/var/lib/kubelet
I am able to use this is helm3 command, and install local chart pull from longhorn/charts@674a553a9363996e4150e487ad6cfb59479db3a1
helm install longhorn ./longhorn --namespace longhorn-system --create-namespace --set csi.kubeletRootDir=/var/snap/microk8s/common/var/lib/kubelet

A working RWX volume for reference:
$ kubectl describe pvc/pvc-hist-data
Name:          pvc-hist-data
Namespace:     default
StorageClass:  longhorn-histdata
Status:        Bound
Volume:        pvc-4b192ed0-5b0c-4d1b-b8d9-f185d87d7587
Labels:        <none>
Annotations:   pv.kubernetes.io/bind-completed: yes
               pv.kubernetes.io/bound-by-controller: yes
               volume.beta.kubernetes.io/storage-provisioner: driver.longhorn.io
Finalizers:    [kubernetes.io/pvc-protection]
Capacity:      1Gi
Access Modes:  RWX
VolumeMode:    Filesystem
Used By:       <none>
Events:
  Type    Reason                 Age                From                                                                                Message
  ----    ------                 ----               ----                                                                                -------
  Normal  Provisioning           12m                driver.longhorn.io_csi-provisioner-76665df9b5-fvmwg_98848812-aa0c-4614-944b-fa929854043b  External provisioner is provisioning volume for claim "default/pvc-hist-data"
  Normal  ExternalProvisioning   12m (x3 over 12m)  persistentvolume-controller                                                               waiting for a volume to be created, either by external provisioner "driver.longhorn.io" or manually created by system administrator
  Normal  ProvisioningSucceeded  12m                driver.longhorn.io_csi-provisioner-76665df9b5-fvmwg_98848812-aa0c-4614-944b-fa929854043b  Successfully provisioned volume pvc-4b192ed0-5b0c-4d1b-b8d9-f185d87d7587

Same issue here with k3s v1.21.7+k3s1
longhorn-csi-plugin time="2021-12-07T21:33:03Z" level=info msg="NodeStageVolume: req: {\"staging_target_path\":\"/var/lib/kubelet/plugins/kubernetes.io/csi/pv/pvc-7eb2fb33-b229-48da-852a-3bbe25bcfe13/globalmount\",\"volume_capability\":{\"AccessType\":{\"Mount\":{\"fs_type\":\"ext4\"}},\"access_mode\":{\"mode\":5}},\"volume_context\":{\"fromBackup\":\"\",\"fsType\":\"ext4\",\"nodeSelector\":\"nvme\",\"numberOfReplicas\":\"2\",\"share\":\"true\",\"staleReplicaTimeout\":\"30\",\"storage.kubernetes.io/csiProvisionerIdentity\":\"1638912038313-8081-driver.longhorn.io\"},\"volume_id\":\"pvc-7eb2fb33-b229-48da-852a-3bbe25bcfe13\"}"
longhorn-csi-plugin time="2021-12-07T21:33:03Z" level=debug msg="trying to ensure mount point /var/lib/kubelet/plugins/kubernetes.io/csi/pv/pvc-7eb2fb33-b229-48da-852a-3bbe25bcfe13/globalmount"
longhorn-csi-plugin time="2021-12-07T21:33:03Z" level=debug msg="stat: try to evaluate whether nfs mount point /var/lib/kubelet/plugins/kubernetes.io/csi/pv/pvc-7eb2fb33-b229-48da-852a-3bbe25bcfe13/globalmount exist"
longhorn-csi-plugin time="2021-12-07T21:33:03Z" level=debug msg="stat: done evaluating whether nfs mount point /var/lib/kubelet/plugins/kubernetes.io/csi/pv/pvc-7eb2fb33-b229-48da-852a-3bbe25bcfe13/globalmount exist"
longhorn-csi-plugin time="2021-12-07T21:33:03Z" level=debug msg="Running nsenter command: nsenter [--mount=/proc/1/ns/mnt --net=/proc/1/ns/net --uts=/proc/1/ns/uts -- /usr/bin/realpath -e /var/lib/kubelet/plugins/kubernetes.io/csi/pv/pvc-7eb2fb33-b229-48da-852a-3bbe25bcfe13/globalmount]"
longhorn-csi-plugin time="2021-12-07T21:33:03Z" level=error msg="failed to resolve symbolic links on /var/lib/kubelet/plugins/kubernetes.io/csi/pv/pvc-7eb2fb33-b229-48da-852a-3bbe25bcfe13/globalmount" error="exit status 1"
longhorn-csi-plugin time="2021-12-07T21:33:03Z" level=debug msg="mount point /var/lib/kubelet/plugins/kubernetes.io/csi/pv/pvc-7eb2fb33-b229-48da-852a-3bbe25bcfe13/globalmount try reading dir to make sure it's healthy"
longhorn-csi-plugin time="2021-12-07T21:33:03Z" level=error msg="NodeStageVolume: err: rpc error: code = Internal desc = Failed to prepare mount point for shared volume pvc-7eb2fb33-b229-48da-852a-3bbe25bcfe13 error exit status 1"

Surprisingly:
ReadWriteOnce works flawlessly
ReadWriteMany having issues as above - yet volume is created, bound / attached but csi-plugin throwing logs as above ( and the app container using volume not coming up )

For v1.2.3 we switched to using a wrapper shell script that does the nsenter instead of having to maintain our own nsenter mounter. This should bypass the need for the realpath binary.
REF: https://github.com/longhorn/longhorn-manager/blob/v1.2.x/package/nsmounter

@lukaszraczylo if you want to verify that the change with the nsenter wrapper fixes the issue in your environment, you can try to deploy the RC chart on a TEST CLUSTER we don't support upgrades/downgrades from RC releases.
REF: https://github.com/longhorn/longhorn/tree/v1.2.3/chart

@joshimoo first of all - thank you so much for the fast reply! I haven't expected it :)
Second - and "the thing we've been waiting for" I can confirm that RWX mounts work as desired with 1.2.3 and problems do not exist anymore.

NOTE for QA, https://github.com/longhorn/longhorn/issues/2987#issuecomment-988293534
This change was part of the https://github.com/longhorn/longhorn-manager/pull/1064 for the csi driver FS expansion support issue: https://github.com/longhorn/longhorn/issues/2794
Since the expected functionality has been tested on our existing distro matrix / test clusters. This should be fine to close, unless you want to verify correct functionality on alternative k8s distros (Microk8s) for example.

Just in case: I've tested in on k3s running on Pi's ( if that helps )

cc @longhorn/qa

@kaxing Can you please quickly check this on Microk8s?

Hey @khushboo-rancher I am able to launch rwx-test workload on microk8s with version:v1.22.4-3+adc4115d99034.
Pushing the workload replicas to 128, both the cluster(1+3 4g ram nodes) and longhorn are holding up health.

I've been testing it since the issue was raised ( and the solution suggested ) and can confirm it's stable and has not caused any problems since then ( k3s @ RPI: 14 nodes total, two data nodes with nvme's, various RWX volumes mounted at least three times and operating under load and constant RW ops )

Thanks for the update @lukaszraczylo !

Validation - PASSED
Tested with master-head images with microk8s.
Details in this comment: https://github.com/longhorn/longhorn/issues/2987#issuecomment-992283392

Thanks @kaxing @lukaszraczylo


Fixed #27850 -- Made RelatedFieldWidgetWrapper call render() of the wrapped widget.

https://code.djangoproject.com/ticket/27850

Looks okay to me. @prestontimmons, any input?


”Maieru” is listed in the drop-down menu

Description:
"Maieru" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Maieru" in the search bar.

Expected results
The location is listed in the drop-down menu as "Maieru, Bistrița-Năsăud".


Please provide a valid Impression in the request !!

Although the impression is not mandatory in the documentation

if i dont provide any value
if i providse the default value ie. 15

It returns the error Please provide a valid Impression in the request !!

Yes, I have made that as mandatory so there is always a clarity on which specific size the user wants to request.. will have to check if passing 15 is still not working


#1267 closed and locked without opportunity to request clarification

Please explain (or link to a document which explains) why you felt it important to put in significant effort to modularize winston, as mentioned in #1267. While I find the dependency growth offputting, I might agree with your rationale and find great use out of winston. I am a little concerned about the finality of the response I received, but that need not end our association.

Please see my follow-up comment here https://github.com/winstonjs/winston/issues/1267#issuecomment-380886039
Modularity in software is important. By separating the formatters from the base transport from winston itself we allow those things to be used by other projects individually. This in turn grows the ecosystem around the project.
I do not see this line of discussion as fruitful in public – from my perspective it only serves to create FUD. If you wish to continue our discussion in private please feel free to email me.


add apache groovy

As stated here: https://groovy-lang.org/syntax.html
//
/* ~ */

Closed with 7d3c181.


Game Crashes

BEFORE SUBMIT A BUG, PLEASE READ THE FAQ PAGE HERE: https://github.com/derekhe/msfs2020-google-map/wiki/FAQ
Describe the bug
The game crashes after taking off ingame. Issues like the game loading and crashing started when I first installed the mod. I removed the files but the game does not work.
To Reproduce
Load up the game, take off, and wait a few minutes.
Screenshots
N/A
Desktop (please complete the following information):
MSFS2020 Steam Edition
Additional context
N/A

I also looked for the lines of code to delete but I could not find them.

If you are using Nvidia, and your driver is the latest, the cause is not google maps, it is the driver.
The game crashes with the latest nvidia driver, you have to revert to the old one.
497.09
https://piunikaweb.com/2021/12/04/microsoft-flight-simulator-crashes-after-updating-nvidia-drivers/

After upgrading from Version 497.09 to 497.29 of the GeForce Game Ready Driver, I no longer experience crashing to desktop when commencing a flight.


why are these exported. As well as some other style fixes

Sorry, but making those variables start with capitals was a deliberate decision: it serves as a marker of global variables.
Besides, they aren't really "exported" since they are in the main package and can't be imported anyways.


Fabric - create doc using git.

use git to create the presentation and documentation.

the documentation should be using "markdown" (md files)
And the presentation should be using "marp" (which is based on markdown)


There is an error when I use PySide6 6.4 and QtAwesome - AttributeError: Cannot reassign members.

versions：

python 3.9.13
pyside6 6.4
qtawesome 1.2.0

code:
import sys
from PySide6 import QtCore, QtWidgets, QtGui
import qtawesome

class MyWidget(QtWidgets.QWidget):
    def __init__(self):
        super().__init__()

        self.button = QtWidgets.QPushButton("click here")

        self.layout = QtWidgets.QVBoxLayout(self)
        self.layout.addWidget(self.button)

        self.button.clicked.connect(self.showMessage)

    @QtCore.Slot()
    def showMessage(self):
        msgBox = QtWidgets.QMessageBox()
        msgBox.setText("Hello world")
        msgBox.setStandardButtons(QtWidgets.QMessageBox.Ok)
        ret = msgBox.exec()

if __name__ == "__main__":
    app = QtWidgets.QApplication([])

    widget = MyWidget()
    widget.resize(300, 200)
    widget.show()

    app.setStyle(QtWidgets.QStyleFactory.create("Fusion"))
    sys.exit(app.exec())

error:
Traceback (most recent call last):
  File "D:\work\pyside6_study\ps1.py", line 3, in <module>
    import qtawesome
  File "D:\work\pyside6_study\venv\lib\site-packages\qtawesome\__init__.py", line 23, in <module>
    from qtpy import QtCore, QtWidgets, QtGui
  File "D:\work\pyside6_study\venv\lib\site-packages\qtpy\QtCore.py", line 116, in <module>
    Qt.MidButton = Qt.MouseButton.MiddleButton = Qt.MiddleButton
  File "shibokensupport/enum_310.py", line 529, in __setattr__
AttributeError: Cannot reassign members.

Hi @lizhuoxuan thank you for the feedback! As the traceback points out this is an issue over QtPy (for more info about the related issue, it is available here: https://github.com/spyder-ide/qtpy/issues/373).
Let us know if the info above helps!

Hi @lizhuoxuan thank you for the feedback! As the traceback points out this is an issue over QtPy (for more info about the related issue, it is available here: spyder-ide/qtpy#373).
Let us know if the info above helps!

Yes,it is very useful.thanks!


Setting Modes

Normal
Read Only
Write Only
Disabled

This is for discussing the effect that each of these will have.
The first and last are obvious, since htey already exist. I assume that
setMode(NORMAL) will act as though nothing changed (the default value of mode).
setMode(DISABLED) will be an alias of ->disable()
The harder ones to describe would be Read Only and Write Only.
In read only, what effect should using set() have. Should it return the same as it currently does, but not store the value?
In write only, what effect should get() have. Should it return null, the value that is currently stored in teh database (effectively, a force miss)

@tedivm input please

Yeah, you've got it.

The last one is still ambigious to me,
Should it return either:

Null; or:
The value currently stored in the database (and null if not exist)

It should return null. Returning what's in the database requires reading, which shouldn't happen when the system is in write only mode.

This still doesn't solve the problem I had initially then, which is to have a forceMiss mode.
I need it to always write, regardless of whether there is something in the database already or not.
But it also must read the correct value at the end.

Toss "forced miss" into the mix then.

@tedivm Should write only always write (IE. force miss too?)
Or should it still respect the expiration ?

@tedivm I have implemented them all, but am struggling testing the actual modes, since with all the stubs that get used etc, I don't actually understand how it is set up.
I'll submit the PR as is, and can you help me with the testing?
Thanks.

@tedivm Any input?


Add support for MCMC

I've implemented a simple MCMC method using the emcee library.
This fixes #21.
It required returning -inf when a parameter is outside of its
bounds. I've also had some trouble with our models returning nan, which we should definitely fix.
I think it's best to raise an exception if the model produces nan, and then make sure that the default distributions do a good job of not running into problems.

Tests please :smiling_imp:

LGTM :+1:


Can't update/delete connection from connection settings of left sidebar

Describe the bug
I am able to edit/update/delete a connection using ConnectionSettings component sidebar that can be loaded from within a chat. But, when I use the main navigation leading to ConnectionSettings page, all connections are being detected as "new" with "Add" being the only option. I think this is related to https://github.com/convos-chat/convos/commit/5f69a031590ea6d52304e18b5f1ccbe5aa559c75 where ConnectionForm started expecting
To Reproduce
Steps to reproduce the behavior:

Go to /settings/connections
Click on any connection
Try to update/delete

Expected behavior
Update button and delete button should be available.
Screenshots
Only add button is available
Additional context
diff --git a/assets/page/ConnectionSettings.svelte b/assets/page/ConnectionSettings.svelte
index c9aafbd9..b4deb859 100644
--- a/assets/page/ConnectionSettings.svelte
+++ b/assets/page/ConnectionSettings.svelte
@@ -1,4 +1,5 @@
 <script>
+import {getContext} from 'svelte';
 import ChatHeader from '../components/ChatHeader.svelte';
 import ConnectionForm from '../components/ConnectionForm.svelte';
 import Icon from '../components/Icon.svelte';
@@ -8,6 +9,8 @@ import {nbsp} from '../js/util';

 export let connection_id = 'add';
 export const title = connection_id === 'add' ? 'Add connection' : 'Edit connection';
+const user = getContext('user');
+$: connection = connection_id !== 'add' ? user.findConversation({connection_id}) : {};
 </script>

 <ChatHeader>
@@ -19,5 +22,5 @@ export const title = connection_id === 'add' ? 'Add connection' : 'Edit connecti
 </ChatHeader>

 <main class="main">
-  <ConnectionForm is_page="{true}"/>
+  <ConnectionForm {connection} is_page="{true}"/>
 </main>

Something like this seems to work.
But as soon as DELETE request is made, a POST request is made again which seems to be recreating deleted connection

Yikes! I guess I must have been in a hurry when restructuring the assets 😭
Hope it's OK I put your name on the commit.


support eslint:recommand rules

all project support eslint:recommand rules

all project support eslint:standard rules


Testing, please ignore. (sztyyu)

This is a bug bounty test. Please ignore.. Please DO NOT APPROVE THIS! (sztyyu)

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


Rename Microsoft.AspNet.HttpFeature to Microsoft.AspNet.Http.Interfaces

If you want to know why, ask @davidfowl!

https://github.com/aspnet/HttpAbstractions/issues/150

I'd have much preferred Ms.AspNet.Http.Features or Features.Interfaces, but shrug

It's more consistent with the other package names :shipit:

:shipit: from me too, though I won't ask @davidfowl why. If you need any assistance with the CI (as is often the case with renames), let me know.

too late :P and @tugberkugurlu agrees with me


Confirm non-utf8 proto string behavior

Describe the solution you'd like
In Go when we switched from gogoproto to common proto, we recognized that the former allowed invalid utf8 but not the latter. We have seen a potential case where a user may have set invalid utf-8 in a proto Failure message. We need to confirm whether Java proto library even allows this (in either direction if possible, but especially at marshal time), and then assess our options.

Confirmed the java proto library does not allow non-utf8 proto string


Update govuk frontend 1.1.0

Frontend release notes:
https://github.com/alphagov/govuk-frontend/releases/tag/v1.1.0

do want to do this and 1.1.1 separately? Or just go to 1.1.1?

Would be good to remove that merge commit and rebase off master if we need to.

done


fix: articles/active-directory/develop/active-directory-saml-protocol-reference.md

Escape element

@nschonni : Thanks for your contribution! The author, @CelesteDG, has been notified to review your proposed change.

Verified editorial correction. Thank you.


Time Series Forecasting Models using Python

Updated Readme, proper comments in code

@Rutikab12

@durbar2003 we can not accept   .py  file so please replace it with .ipynb file

okay


Make BIO_do_connect() and friends handle multiple IP addresses also in 1.1.1

This backports #11971 as requested: https://github.com/openssl/openssl/pull/11971#issuecomment-634679197

BTW, I cannot any more locally build OpenSSL_1_1_1-stable.
Even after make distclean and ./config no-dso I get:
/usr/bin/ld: ./libcrypto.so: undefined reference to `DSO_METHOD_openssl'

cannot reproduce your build problem, try make distclean and let use git clean afterwards,
maybe something is lingering.

@bernd-edlinger, git clean was an excellent hint!
It showed that (from checking out master in between) there were some remnant
crypto/bn_conf.h crypto/dso_conf.h crypto/opensslconf.h
and after removing those the build went fine again.

Yeah, moreover make distclean is broken in master.
But I tried to fix the PR stalled for months.  Now I have given up

This pull request is ready to merge

Merged - thanks @bernd-edlinger and @t8m!


Get RemoteAudioState without listener

Hi,
I am sorry to start this as an issue, since its more like a question.
So in my RN app, I have an option to disable video and go only with audio ( in which case I have other stuff connected to this state ).
BUT, if someone joins the channel AFTER someone in the channel disables his video, I am unable to get that information about users already in channel.
Listener RemoteAudioStateChanged does not trigger on init and I can see no function that would give me this info.
Would you know please?
Thanks in advance

Any updates ?

does UserJoined trigger?

_initEngine = async () => {
    this._engine = await RtcEngine.create(config.appId);
    this._addListeners();

    await this._engine.enableAudio();
    await this._engine.disableVideo();
    await this._engine.setChannelProfile(ChannelProfile.LiveBroadcasting);
    await this._engine.setClientRole(ClientRole.Broadcaster);
  };

  _addListeners = () => {
    this._engine?.addListener('JoinChannelSuccess', (channel, uid, elapsed) => {
      console.info('JoinChannelSuccess', channel, uid, elapsed);
      this.setState({ isJoined: true });
    });
    this._engine?.addListener('LeaveChannel', (stats) => {
      console.info('LeaveChannel', stats);
      this.setState({ isJoined: false });
    });
    this._engine?.addListener(
      'RemoteAudioStateChanged',
      (uid, state, reason, elapsed) => {
        console.info('RemoteAudioStateChanged', uid, state, reason, elapsed);
      }
    );
  };

  _joinChannel = async () => {
    if (Platform.OS === 'android') {
      await PermissionsAndroid.request(
        PermissionsAndroid.PERMISSIONS.RECORD_AUDIO
      );
    }
    await this._engine?.joinChannel(
      config.token,
      this.state.channelId,
      null,
      config.uid
    );
  };

this works fine for me.

Oh sorry I ment RemoteVideoStateChanged of course, my bad.
UserJoined triggers fine.

yes, disableVideo means diable video engine, you can try enableLocalVideo

For 'disabling video' we are using muteLocalVideoStream . My goal here is to find out the video state ( of everyone else ) once I join a channel . Just like what RemoteVideoStateChanged returns, but as I mentioned, that does not trigger on init. Is there any way how to get the video state anyhow else, but the RemoteVideoStateChanged listener?

that does not trigger on init
I can not understand this, RemoteVideoStateChanged will be triggered after you joined the channel.

Well, it does not trigger after I join. We are using RtcChannel, maybe the issue is there ?

I test it with RtcEngine, I will test it with RtcChannel tomorrow.

have you called publish?

if the remote user mute his video, you can't receive RemoteVideoStateChanged.

you can set no video state by default, and change state when you received RemoteVideoStateChanged .

Yeah you are right. I could have thought as well 😄 . Thanks for pointing out 👍🏼


Add test to prevent modifying /etc/hosts twice

What this PR does / why we need it:
Fixes an issue that causes docker swarm init to fail with BYO DNS
Which issue this PR fixes
fixes: #760

This change is

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@acs-bot  test this please

@benformosa , the test failed, because you didn't regenerate template.go file.
Do
git pull
make ci

Once all passed, template.go file get updated. Include that into this PR.

please run make build(preferable in the dev env, ./scripts/devenv.sh). Uur templates are loaded into the templates.go file.

@benformosa,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@benformosa, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

@rjtsdl, @JackQuincy Thanks for the advice. I've redone the commit, including the changes to template.go.


research finding users

I want your team to focus on social learning elements - and for that we need to use telegram api to find other users.
there is a chance that the bot api cannot do it-  so we might need to use the mtproto (or telegram api) to do this.
DOD:

[x] create a POC repo and install the needed telegram sdk
[x] note which data you need to run it (phone number? specail api key/token?)
[x] create fast api endpoints to search for users based on keywords. check if there is an option to serach by profile stuff
[x] create poc bot to communicate with server
[x] check that the functions work
[x] create a report detailing what is the goal, what needs to be done to achienve this and every other info
[x] send the report to me and the entire team in slack

server
bot


SW-6154 Removed permission checker for fetching deliverable category

#2493  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tommylau523 and the rest of your teammates on  Graphite

Merge activity

Oct 15, 4:23 PM EDT: A user merged this pull request with Graphite.


No hardcoded file paths

Motivation and Context

Closes #539
Throughout the worker there are multiple instances of hardcoded file paths. By using a PathProvider code duplications is mitigated and all the hardcoded endpoints can be easily found as they are all in once place.
Description

All the hardcoded file paths I found in the worker module have been moved to the pathprovider. The path providing functions follow the OS agnostic style laid out by filepath
The overall code has not been changed much, mostly just moved about, so the expected behaviour should also not have changed.
Steps for Testing

Prerequisites:

A machine running a Unix-based OS

Run go test github.com/TUM-Dev/gocast/tools/pathprovider
Ultimately these changes should incur no change in the behaviour of the overall project

Screenshots

none

There was a misunderstanding on my end regarding the difference between the new runner and the worker, so I don't think this PR makes sense any more. I'll close it, but I can re-do this for the actual runner if need be (but I don't think that will be the case).


cursor pointer not work

Hello Developer:
Thank you very much for developing this useful package.
I don't know if I should ask this question here, the cursor doesn't seem to work.
The simplest example, I used the HTML template you provided and added the following code to index.html .

pointer

Thanks!
Fix it.
You can use that feature starting from version 0.17.3


Please review int_to_bytes, bytes_to_int, lemmaBytesToIntIsTheInverseOfIntToBytes, lemmaIntToBytesIsTheInverseOfBytesToInt in beacon/helpers/Math.dfy

Please review

int_to_bytes,
bytes_to_int,
lemmaBytesToIntIsTheInverseOfIntToBytes,
lemmaIntToBytesIsTheInverseOfBytesToInt

in
beacon/helpers/Math.dfy

@saltiniroberto

I suggest moving the int_to_bytes/bytes_to_ints to the BytesAndBits.dfy file (ssz).
This is where the counterpart functions for uint8 are located.

The proof of the second lemma can be simplified using the first one. Here is the simplified version:

lemma foo(s: bytes)
    requires bytes_to_int.requires(s)
    ensures int_to_bytes(bytes_to_int(s),|s| as uint64) == s
    {
        calc {
            bytes_to_int(int_to_bytes(bytes_to_int(s),|s| as uint64));
            ==
            bytes_to_int(s);
        }
    }

Thanks for the review @franck44  and for pointing out a simpler proof the second lemma.
On the first point, however, I would prefer to leave int_to_bytes and bytes_to_int where they are for the moment as they are technically part of the beacon-chain spec, not of the SSZ spec.
Also, by keeping the current organisation is very straightforward to check that, for example, all the functions of Math helper section of the beacon chain specification have been verified.

[DONE]


Please add EncodeHints/EncodeHintType in qrcode.dart

I can't import EncodeHints

@xvrh
Oh sorry something is wrong
https://github.com/xvrh/zxing-dart/actions/runs/3181579826
I will try it

@xvrh
I found error.
Error:  found changed files after build. please run 'dart tool/prepare_submit.dart' and check in all changes
Can you rerun job?

@YojiroMotoba Thanks, I fixed it directly on master.

@xvrh Thank you for responding to it promptly! 😄


How to populate redis ?

Can you please post some example how to populate redis so I will be able to add "virtulhosts" and coresponting routes ?
Thanks.

You can configure Ceryx about how to connect to and use a Redis server, with the following environment variables:

CERYX_REDIS_HOST: the redis host to connect to - defaults to <IP_ADDRESS>
CERYX_REDIS_PORT: the redis port to connect to - defaults to 6379
CERYX_REDIS_PREFIX: the redis prefix to use in keys - defaults to ceryx

All configuration options are available in the README.


Updating the included resources.yaml to include latest bits.

I suspect the jenkins test is flaky at best at the moment.  Going to merge with Travis as the test of record.


dsniff requirements

1.dsniff compile time requirements:

libpcap-dev
libnet-dev
libdb-dev
libnids-dev
libssl-dev

Note: Please do not use the version of libssl1.1 otherwise an error will be reported during compilation

openssl

all requirenemts
apt-get install make gcc g++ gdb libpcap-dev libnet-dev libdb-dev libnids-dev libssl1.0-dev openssl -y

The mistake here should be libssl1.0-dev

The mistake here should be libssl1.0-dev


fix isinstance

Small fix. I think one recent AnnData version changed the return type from pd.DataFrame to pd.Series.

When tests pass I am going to update the changelog.

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

Project coverage is 83.56%. Comparing base (e6ed752) to head (2edc11c).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #393      +/-   ##
==========================================
+ Coverage   83.52%   83.56%   +0.03%
==========================================
  Files           8        8
  Lines        1687     1691       +4
==========================================
+ Hits         1409     1413       +4
  Misses        278      278

Files with missing lines
Coverage Δ

src/spatialdata_plot/pl/basic.py
89.32% <100.00%> (+0.10%)
:arrow_up:

src/spatialdata_plot/pl/render.py
89.86% <100.00%> (+0.05%)
:arrow_up:

@timtreis @Sonja-Stockhaus @melonora very small PR that fixes some bugs I got locally a while ago, can you please have a look?

Test pass, so for the sake of a release I'd be inclined to merge soon.

Thanks for the review, I will try to find the plot that was causing the problem.

Ok I looked more into this. What caused the problem is due to scanpy==1.10.3, now fixed in 1.10.4. Shown is the diff between scanpy 1.10.3 and 1.10.4. The left version leads to a pd.DataFrame when the argument to get.obs_df() is a list[str], while it leads to a pd.Series when a str is passed. The second always passes a list[str], so the return is consistently a pd.DataFrame.
We could also fix this by requiring 1.10.4 as a min version. I'll think about it and update this PR.

I tested it against scanpy 1.10.3 and 1.10.4 and the problem is addressed.

Merging since the problem is now addressed.


adjust layout to be more usable on a better range of screen sizes

@gissoo I noticed that the site doesn't look very good on Ivan's laptop. I don't know how common this screen size is, but thought you'd want to take a look and think about how we might adjust.  Ivan says his display resolution is at 1920x1080

completed in #172