Move Omnibus directive related stuff from ClientOrders to PublicOffer read model

Related to #224.
The goal is to move Omnibus price info from the ClientOrders to the PublicOffer read model.

The thing is - we are supposed to show it to clients in the moment they are making an order too, right? They might not even look at the Products page and go straight to making an order. And we're using there the ClientOrders::Product, not PublicOffer::Product.
Unless - we're not really going to inform about promotions when client is making an order. The directive requires to show this information when informing consumers about the promotion / special offer. So if we only show promotions on PublicOffer part, then we should be fine with keeping it there.

Yes, PublicOffer is a view for showing promotions.
Thanks, I'm closing this.


how do I download it?

I want to download it as a gif, but only a picture comes out..

I want to download it as a gif, but only a picture comes out..

@Gigimi931 these are not the animated versions of the emoji. The animated ones can be found at https://animated-fluent-emoji.vercel.app/

I want to download it as a gif, but only a picture comes out..

Hello!
@Gigimi931 these are not the animated versions of the emoji. The animated ones can be found at https://animated-fluent-emoji.vercel.app/


Implement IOList BIFs

Implementation for #153.

[x] Implement and test iolist_to_binary/1
[x] Implement and test iolist_size/1
[x] Implement and test iolist_to_iovec/1
[ ] Maybe add testing for each binary sub-type / representation (erlang docs, enumerated in typed_terms.rs (ProcBin, HeapBinary, SubBinary, MatchContext)?

I'm starting with pretty naive implementations here, to get my feet wet. This is my first Rust programming, and first exposure to both the Lumen dev environment and to iolist, so I've gotta find a handhold somewhere.
It's taken a bit of digging to find good information on the use cases for iolist vs the similar list_to_binary/1. (My initial implementations delegate to list_to_binary/1.)
I'm beginning to understand the use case of optimizing memory usage when streaming data such as when writing to a file or socket. The iolist functions can allow for more efficient list operations and memory usage. Handling improper lists is part of this use case. Helpful resources on this: here, here, here, here.

I'm not sure whether adding test coverage for the binary sub-types is important. Interested to hear what a reviewer thinks about that.

@KronicDeth do you have any feedback regarding whether it's important to test with various binaries that would result in different internal typed terms? (ProcBin, HeapBinary, SubBinary, MatchContext)
Seems like I saw something possibly related on #200 .

You definitely need to test subbinaries to show that you handle non-binary ones correctly.

I rebased onto develop and fixed up some stuff that seems to have changed.
I was not successful at constructing a MatchContext that I could pass in as a term from the test module.
I tried adding a test for ProcBin, which seemed to expose the fact that the list_to_binary/1 (to which iolist_to_binary/1 currently delegates). Opened #368 to track that and proposed #369 to resolve it. So the resolution of this PR is now blocking and that resolution of those.


Custodian output logs for scheduled policy (Lambda)

If I've set up my policy to run on a schedule through Lambda, where can I define where the output logs will go? Can I send them to an S3 bucket? Or are there no logs when Custodian runs as a Lambda function?

Specifically looking for the resource.json. I see Lambda/Custodian taking actions on my resources, such as marking them, but cant find a way to access or save resource.json

Figured solution to push resource.json from tmp folder to S3 bucket

Hi, could you share the solution here,i am also looking for similar kind of solutions.


insufficient information

Closing this issue as there is no reference to a topic. Please open a new issue. Thanks.


Get deep attribute with orm_mode

Question
Hi!
How can I access to a deep attribute with orm_mode = True?
For example:
Response I want:
{
  "fake_A": 1,
  "fake_d": "fuzzy"
}

My models:
class A:
    a = 1
    b = '2'

class B:
    d = 'fuzzy'
    c = A()

Pydantic models:

class Fake(BaseModel):
    a: int = Field(alias='fake_a') # source='c.a'  like django?
    d: int = Field(alias='fake_d')

     class Config:
         orm_mode = True
         allow_population_by_field_name = True

Fake.from_orm(B())

On other framekors like Django you can get the a attribute of Class A throung Class B with source='c.a'
How can I do it with Pydantic?

Yes, but currently it's not trivial.
You need to subclass and customize GetterDict:
https://github.com/samuelcolvin/pydantic/blob/44616e35cb0ddd97af07b95d7209176a29bfea49/pydantic/utils.py#L347-L352
Then use that as the getter_dict attribute of BaseConfig:
https://github.com/samuelcolvin/pydantic/blob/44616e35cb0ddd97af07b95d7209176a29bfea49/pydantic/main.py#L118


health_checks: add option to always log successful health checks

Additional Description: Resolves #32119. This allows the option to always log successful health checks. On the first successful health check, only logAddHealthy is called. On consecutive successful health checks, logSuccessfulHealthCheck is called.
Risk Level: low (config guarded)
Testing: unit tests
Docs Changes: API docs
Release Notes: none
Platform Specific Features: none

/assign @botengyao
PTAL on API @lizan

/retest

/wait

Done. @lizan can you PTAL?

/retest


fix some data races in Server

This avoids some issues with Server.PacketConn and Server.Listener being
accessed outside of their locks.
Fixes miekg/dns#244.

LGTM


jcenter shutdown

https://jfrog.com/blog/jcenter-sunset/

I create a brancg WebRTC to handle this issie. Since org.webrtc:google-webrtc:1.0.32006 is only available from jCentral, the options are to replace this library with something like "org.jitsi:webrtc:124.0.0", or include a jar of "google-webrtc" in the project. Using "org.jitsi:webrtc" caused some problem of unresoved functions, so I think I will build the jar next.

Using org.jitsi:webrtc caused some problems with unresolved functions,

why?

include a JAR of google-webrtc in the project

please don't include binaries
try to use webrtc from Maven, eg. https://central.sonatype.com/artifact/im.conversations.webrtc/webrtc-android/versions

@licaon-kter @toliuweijing
Thanks for the suggestion.
// implementation 'org.webrtc:google-webrtc:1.0.32006'
implementation "im.conversations.webrtc:webrtc-android:119.0.1"

This builds the project, but I get an error at runtime:
java.lang.UnsatisfiedLinkError: No implementation found for long org.webrtc.SoftwareVideoEncoderFactory.nativeCreateFactory() (tried Java_org_webrtc_SoftwareVideoEncoderFactory_nativeCreateFactory and Java_org_webrtc_SoftwareVideoEncoderFactory_nativeCreateFactory__)
                                                                                at org.webrtc.SoftwareVideoEncoderFactory.nativeCreateFactory(Native Method)
                                                                                at org.webrtc.SoftwareVideoEncoderFactory.<init>(SoftwareVideoEncoderFactory.java:23)
                                                                                at org.webrtc.DefaultVideoEncoderFactory.<init>(DefaultVideoEncoderFactory.java:20)
                                                                                at org.avmedia.remotevideocam.display.customcomponents.VideoViewWebRTC.initializePeerConnectionFactory(VideoViewWebRTC.kt:150)

How can Iclude these native libraries? Thanks for your help.

Unfortunately I can't help with the rest of the implementation.
Note that your old webrtc was from Aug 2020 and full of vulnerabilities, that aside, newer versions changed some stuff indeed. Hope you find a way to use it for this app.

@licaon-kter
Are you OK if we use this library:
org.webrtc:google-webrtc:1.0.30039
from here:
maven {
            url 'https://raw.githubusercontent.com/alexgreench/google-webrtc/master'
        }

Maybe replace it in the future.

No, not really, the list of accepted repos is here (jcenter will be removed soon): https://gitlab.com/fdroid/fdroidserver/-/blob/8cf4503a00f6e0ccb93f5a9089cf660f20b0045b/fdroidserver/scanner.py#L456-L475
also, using a 4 years old webrtc will get you a Knownvuln flag :( , ref: https://gitlab.com/fdroid/fdroiddata/-/merge_requests/11496

Reording can fix the UnsatisfiedLinkError exception, because PeerConnectionFactory.initialize loads the native library into memory.

However, the PR still doesn't work due to native crash. Need to investigate more. cc @izivkov if you have more insight.

@toliuweijing
Yes, I got that far in branch WebRTC.
I'm working if the OpenCV has something to do with requiring hardware support.

@toliuweijing
Yes, I got that far in branch WebRTC.
I'm wondering if the OpenCV has something to do with requiring hardware support.

I don't think so. I didn't use any fancy CV stuff but of course you can verify by reverting the commit.

@toliuweijing
Any luck with the WebRTC library replacement?

@izivkov appologize for late reply. I didn't have enough time to investigate this. My plan is to debug it by building a sample webRTC app using the new dependency "im.conversations.webrtc:webrtc-android:119.0.1".
Will see if I get some time late this month or next month.

No worries, thanks for the update!
On Mon, Sep 16, 2024 at 5:05 AM Jim Liu @.***> wrote:

@izivkov https://github.com/izivkov appologize for late reply. I didn't
have enough time to investigate this. My plan is to debug it by building a
sample webRTC app using the new dependency
"im.conversations.webrtc:webrtc-android:119.0.1".
Will see if I get some time late this month or next month.
—
Reply to this email directly, view it on GitHub
https://github.com/izivkov/RemoteVideoCam/issues/28#issuecomment-2352367915,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7M37SIMTEFETNS4JY7NNDZW2NMNAVCNFSM6AAAAABMWGTUV6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJSGM3DOOJRGU
.
You are receiving this because you were mentioned.Message ID:
@.***>


Improve debugging styled components

When defining a Styled Component, it is named Anonymous.

This basically affects the debugging experience.
I've been thinking maybe stitches can provide a babel plugin to infer the names based on the file name, etc. Or stitches can name the Component like this "StyledComponent(section)". I'm not sure though what would be the better approach for this. But for now, I can say it's hard to determine the styled-components in React dev tool.

That's a great suggestion! Thanks for raising it.
Let's see what we can do here.

@denniscual One way to improve this right now, is to add displayName to your components. Here's a CodeSandbox demo.

@hadihallak @christianalfoni Do you guys have any ideas on the least obstructive way of improving the debugging experience?
If possible, I'd like to avoid the need for babel plugins. But I don't know

@peduarte yeah, we can create a default label internally similar to what @denniscual suggested: StyledComponent(tagName).
Will push this today but I also think that a babel plugin will be needed later as it's a lot cleaner to infer the name from the constant containing the component instead of using a default based on the tag but we can do this as a step 2 kind of thing

@hadihallak sounds great.
If we can provide a good default experience, I'm fine with using babel as an enhancement (opt-in)
Thanks! <3


No cookie sent with POST request when using auth().form()

Hi,
It seems like when using form authentication the cookie is not send with the POST request.
Example:
given()
    .auth().form(this.username, this.password,
        new FormAuthConfig("/login", "username", "password")
	    .withLoggingEnabled()
            .withCsrfFieldName("_csrf"))
    .baseUri(baseUrl)
    .relaxedHTTPSValidation()
    .redirects().follow(false)
    .cookies(COOKIE_ID, this.cookie)
    .log().all()
.when()
    .get("/login");

Responses
Request method:	GET
Request URI:	        https://localhost:8089/login
Proxy:			<none>
Request params:	<none>
Query params:	        <none>
Form params:	        <none>
Path params:	        <none>
Headers:		        Accept=*/*
Cookies:		       LOGIN_SESSION=0F1B08EC8E7E53E19B449A9FDBBAC249
Multiparts:		<none>
Body:			<none>

Request method:	POST
Request URI:	         https://localhost:8089/login
Proxy:			<none>
Request params:	<none>
Query params:	        <none>
Form params:	        username=***
				password=***
				_csrf=***
Path params:	       <none>
Headers:		       Accept=*/*
				Content-Type=application/x-www-form-urlencoded; charset=ISO-8859-1
Cookies:		       <none>
Multiparts:		<none>
Body:			<none>

In the first GET-Request the cookie is set, however, it is missing in the POST request. This is why I get a 403 as a response, because the server does not know to which session the CSRF token belongs to.

Hm ok! Perhaps you could try to provide a pull request for this? Would be really helpful.

I would like to, however, I'm not familiar with groovy. Maybe I'll take a shot at it...


Feature Request: allow custom working days

Hey!
I am working 80% and I would love to be able to select my work days to see how this impacts the holidays.
Thanks!

Lovely! Thank you so much @zachd!


updated social-links and logo

Description

Check List (mark all applicable checkboxes)

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots (if applicable)

Original
Updated

original screenshot
updated screenshot

its for
https://github.com/WeMakeDevs/classroom-monitor-bot/issues/161

Make sure to update all files and not only README.md.

okay then I'll make a new PR soon.., thanks


re-work lite clients into 'Browsers & Mobile'

Hey all, I've restructured the "Lite Clients" track to include browsers and provide a goal for this half-day workshop. I'd love your feedback!
@RangerMauve & @matheus23, I'm asking for both of you (or someone from your org) to give lightning talks to help support this goal of moving the ball forward on IPFS in weird places. I'll wait for both of your feedback & approval before merging.
I'm also seeking feedback from folks I'd love to have attend:
@achingbrain
@autonome
@ikreymer
@lidel
@dignifiedquire
My first request for feedback: who else should we be trying to get to attend?
closes #31

I can commit to preparing a lightning talk :+1:

add a "DIY content routing" session using Reframe for a "lite client" approach to DHT alternatives


When loading csb files, prevent repeated loading of plist files

Describe your changes
Issue ticket number and link
Checklist before requesting a review
For each PR

[ ] Add Copyright if it missed:
- "Copyright (c) 2019-present Axmol Engine contributors (see AUTHORS.md)."

[ ] I have performed a self-review of my code.
Optional:

[ ] I have checked readme and add important infos to this PR.
[ ] I have added/adapted some tests too.

For core/new feature PR

[ ] I have checked readme and add important infos to this PR.
[ ] I have added thorough tests.

Is there any reason that you can't use SpriteFrameCache::isSpriteFramesWithFileLoaded to check if a sprite sheet is already loaded?
For instance, you've added _loadedPlists, and use it like this:
if (!_loadedPlists.contains(plist + png))
{
    // ...
}

Does the following not work?
if (!SpriteFrameCache::getInstance()->isSpriteFramesWithFileLoaded(plist))
{
    // ...
}

We need to reload

removeUnusedSpriteFrames may release unused sprite frames

Sorry, I'm not quite sure that I understand what you mean.
isSpriteFramesWithFileLoaded is implemented as follows:
bool SpriteFrameCache::isSpriteFramesWithFileLoaded(std::string_view plist) const
{
    return isSpriteSheetInUse(plist) && isPlistFull(plist);
}

It checks if the sprite sheet entry is full via isPlistFull, so if any frames were removed through removeUnusedSpriteFrames, then isPlistFull would return false, which means isSpriteFramesWithFileLoaded returns false.  Isn't that the case?

ok, so we don’t even need the _loadedPlists variable?

ok, so we don’t even need the _loadedPlists variable?

Sorry, yes, that is what I meant.  Have you checked if SpriteFrameCache::getInstance()->isSpriteFramesWithFileLoaded works for you? If it does work, then you do not need _loadedPlists.

Let me try


docs.go not support markdown

when markdown has follow text,the  ``` confilct with docs string ```
{
"code":0,
"data":interface{},
"msg":"success"
}

follow

We have bug report and feature request templates. None of them where used.
Good luck.

Comment ticks have been fixed #466 so the docs.go get generated well . Meanwhile i've started making some tests with markdown and looks very unfinished to me.


The --animation-xml parameter seems to do nothing

Was trying to create a netanim viz so I can see what was happening, but it doesn't seem to be creating the file. Tried a relative and absolute path. No feedback, or errors indicating a problem.

Oh ya I may have commented that out so it stoped spewing those files for the simulation runs. It ran waaaay faster when I had that code commented out.

Ah yeah makes sense - encoding can be really slow sometimes


Reindexing not supported within Collection container

Reindexing isn't happening within stereocolletion of SDF file.
See kuva translated sdf and sendoff reindexed sdf for comparison: stereo_comp.zip

To be clear, the file in the attachment ending in _reindexed was generated from sendoff, not Kuva - correct?


⚠ Fortnite has degraded performance

In e5ed534, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1658 ms

Resolved: Fortnite performance has improved in 8e85349.


RDS: BackupRetentionPeriod should be unified to 30

BackupRetentionPeriod is handled differently between AWS and azure, as well as between SDX and Distrox. All of them should have a meaningful value and set to 30 days for HA and a different value for non-HA scenarios.
See detailed description in the commit message.

retest

Can you please include the jira id in the commit?

Can you please include the jira id in the commit?

Thank you for detecting it. Changed.


Compound/composite keys and slickgrid

I didn't see anything in the guide about interacting with tables with Compound or Composite keys. Since SlickGrid expects a single column to be primary/unique, it blows up as soon as the table has duplicates in one of the columns marked as PrimaryKey in the row class (or possibly the IIdField?)
Have any examples for how to get Serenity to generate an auto-incrementing fake id column for SlickGrid to use?
Thanks!

What about adding an expression field to row concatting your composite key values, and using it as ID field. Another option is to add an identity column to table if possible, you dont have to make it primary key.

Another option is to use a fake id client side like i did in GridEditor base class.

Latest template has a view without ID sample

Using an expression field fixes the grid view, create/edit dialog and inserts write to the db, but updates and deletes fail with 'Record not found' exception. E.g. from the repository's Retrieve() call, request.EntityId = '3.1.1' which succeeds, but then on Delete(), request.EntityId = 3 which then throws that exception when Process() is called. Any idea what I'm missing?
Thanks!

Check IdProperty on dialog.cs

[IdProperty(RequirementScriptRow.IdProperty)]     [FormKey("Review.RequirementScript"), LocalTextPrefix(RequirementScriptRow.LocalTextPrefix), Service(RequirementScriptService.BaseUrl)]     public class RequirementScriptDialog : EntityDialog<RequirementScriptRow>     {         protected RequirementScriptForm form;         public RequirementScriptDialog()         {             form = new RequirementScriptForm(this.IdPrefix);         }     }
Looks ok, RequirementScriptRow.IdProperty resolves to "Id" which is the correct column. Here's the json it retrieves when the dialog loads:
{"Entity": 	{ 	"RequirementId":3, 	"ScriptId":1, 	"OsId":1, 	"Id":"3.1.1" 	} }

Why request.EntityId is 3 on delete then? There must be RequirementId left somewhere. Did you transform T4.

Yeah...can't seem to find it. Clean, rebuilt solution, Transformed T4, built Script project. Same problem. Searching solution for RequirementId, not seeing it anywhere it shouldn't be. I'll try again tomorrow. Thanks for the help!

Still not sure what's going on. Here's the json being passed around when opening the Edit dialog.
Request: {"EntityId":"3.1.2"}
Response: {"Entity":{"RequirementId":3,"ScriptId":1,"OsId":2,"Id":"3.1.2"}}
Update: {"Entity":{"RequirementId":"3","ScriptId":"1","OsId":"2","Id":3},"EntityId":3}

Aha, found it. Serenity.Externals.js line 587:
var v = parseInt(id, 10);
is converting "3.1.2" into 3.
You're more familiar with this so perhaps you can suggest a fix for handling strings as Id fields.

I'll have a look into this, definitely a bug

Forcing it to an int (e.g. 312) still causes problems on update/delete, this time the generated SQL looks like this and it complains that 0 rows were affected (Expected 1)
exec sp_executesql N'UPDATE [dbo].[RequirementScript] SET [RequirementID] = @p1, [ScriptID] = @p2, [OS_ID] = @p3 WHERE (cast(RequirementID as varchar(20)) + cast(ScriptID as varchar(20)) + cast(OS_ID as varchar(20)) = @p4)',N'@p1 int,@p2 int,@p3 smallint,@p4 nvarchar(4000)',@p1=3,@p2=2,@p3=2,@p4=NULL
Maybe using an Expression for the Id column isn't the best route...I could just add an Identity column on the db.

Yes, adding an extra ID column shouldn't hurt but there are some who use tables with string IDs, this bug would affect them too, so i'll try to resolve it anyway. Thanks.


tf_library syntax error: '+' operator applied to incompatible types (select of string, list)

Forgive me if I'm missing something obvious here (new to Bazel) but the tf_library Bazel rule seems to assume that tfcompile_flags is a string, while similar rules allow lists (cc_binary's copts for example). This is a gotcha that probably could be fixed easily.
Example
tf_library(
  name = "graph",
  cpp_class = "Graph",
  graph = "graph.pb",
  config = "graph.config.pb",
  tfcompile_flags = ["--target_cpu='core-avx2'", "--xla_enable_fast_math=false"]
)

will crash upon build with '+' operator applied to incompatible types (select of string, list)
The solution is to manually concatenate options in the BUILD file:
tf_library(
  name = "graph",
  cpp_class = "Graph",
  graph = "graph.pb",
  config = "graph.config.pb",
  tfcompile_flags = "--target_cpu='core-avx2' --xla_enable_fast_math=false"
)

@calthome Are you referring to tfcompile.bzl (below)?
https://github.com/tensorflow/tensorflow/blob/da3aa93758508a37741dad1d8b28e62782330171/tensorflow/compiler/aot/tfcompile.bzl#L148
I am new to Bazel as well though I think using " ".join(tfcompile_flags or []) instead will allows tfcompile_flags to be a list.

Created a PR #12769 for this issue.

I support making this change, so long as it maintains support for passing a string and the items in the list are shell quoted. I left that feedback in the review that @yongtang was generous enough to send us.


Improve layout implementation

Move from grid layout to nested rows and columns (or another solution) to allow for more dynamic UI layouts

Done, next step is to add components to rows.  Also widths for columns and heights for rows

Resolved


Mailer From and Reply

Currently the mailer seems to error if the email section FROM in the config is supplied.
When the config line reads: FROM = "Gogs"<EMAIL_ADDRESS>I get this error
2015/08/23 21:47:11 [.../runtime/asm_amd64.s:2232 goexit()] [E] Async sent email 0 succeed, not send emails<EMAIL_ADDRESS>info: Subject: [Test-Repo] Test(#1), send issue mention emails err: mail: missing phrase`

When I have this FROM =<EMAIL_ADDRESS>it still does not use this value and instead uses the email of the user that triggers the event.
I think it would be better if people want to functionality of being able to reply to the person that triggered the email event is to use the FROM address and then use the Reply-To: email header with the user's email.

Tested with <IP_ADDRESS>2 Beta and <IP_ADDRESS>5 Beta according to the configuration page

Thanks your feedback!

I think it would be better if people want to functionality of being able to reply to the person that triggered the email event is to use the FROM address and then use the Reply-To: email header with the user's email.

I'm confused by this line. So you say  it still does not use this value and instead uses the email of the user that triggers the event., and then you want to reply to the user who triggered the event(which is the way Gogs does right now).

It does not use the FROM email as the email the email has been sent from.
As you said it sends it as the user sent the email, not the server that is sending it.

OK... Gogs uses FROM you configured on app.ini when sends so-called system e-mails, and use user's e-mail as FROM for sending mention e-mails is on purpose that shows who actually mentioned you.
Does my explanation of the behavior make sense to you?

Yes!
That make sense.
I am looking for all messages to be sent from the server, and the ones that the user triggered instead use the Reply-To: flag. All emails are being mail from the server and not the person themselves, the current way makes it seem that the person sent that email, not the server. This causes issues as a smtp server is sending mail as a non-verified address that it doesn't own.

If I use<EMAIL_ADDRESS>it should always send from that as it, as domain.com is the domain that I own and I have rights over, not gmail.com or whatever email domain that the user's email is hosted on.

The server should always send from the from address, but the user can still reply to the original sender using the Reply-To: header.

OK, I understand now!
And I agree your point about makes it seem that the person sent that email, not the server.

Hi @goldbattle , by merging https://github.com/gogits/gogs/pull/1596 to develop branch, this issue is claimed to be fixed. Can you help test again? Thanks!

Closed as implemented.

Maybe it's too late to say now, but I was also experiencing error if I set FROM value to the form of "name"<EMAIL_ADDRESS>it said something about missing angle address or what.

@JokerQyou someone reported this before, I think the solution was using:
"Name"<EMAIL_ADDRESS>Two "`" around your value in custom app.ini.


Proxy RTSP stream to SRT?

Describe the feature
I would like to proxy a stream from an RTSP IP camera to SRT.
For example, I have a RTSP IP camera (hanwha.local), attached to an ARM linux board (camera001) published like so:
#[ YML on camera001 running mediamtx 1.4.1]
paths:
  client-stream:
    source: "rtsp://hanwha.local/camera"
    sourceOnDemand: yes

I would like to pull that stream on demand:
#[ YML on server running mediamtx 1.4.1, where I pull the WebRTC stream ]
paths:
  camera001:
    source: "srt://camera001.local:8890?streamid=read:client-stream&pkt_size=1316"
    sourceOnDemand: yes

Currently I get an empty stream when I attempt to do this from a WebRTC client, so I am thinking this is a new feature.
Architecture is roughly like so:
[Hanwha_Camera] -- RTSP --> [ARM camera001] -- SRT --> [Server] -- WebRTC --> [Chrome Browser]

Hello, please post the server log of the MediaMTX instance you're trying to connect to with a web browser, with option "logLevel" set to "debug", as explicitly mentioned in the instructions, otherwise it's not possible to understand what's going on.

closed since user did not provide required data.


MacOS Scan Output Error

I use in macos 13.2.1 (22D68). I see this error output
`
Host is likely running Unknown OS!
---------------------Starting Port Scan-----------------------
sed: 1: "/elapsed/{s/.undergoin ...": bad flag in substitute command: '}'
sed: 1: "/% done/{s/.About (. ...": bad flag in substitute command: '}'
sed: 1: "/elapsed/{s/Stats: (. ...": bad flag in substitute command: '}'
sed: 1: "/remaining/{s/.* ((.\ ...": bad flag in substitute command: '}'
sed: 1: "/elapsed/{s/.undergoin ...": bad flag in substitute command: '}'
sed: 1: "/% done/{s/.About (. ...": bad flag in substitute command: '}'                             ] 0% done
sed: 1: "/elapsed/{s/Stats: (. ...": bad flag in substitute command: '}'
sed: 1: "/remaining/{s/. ((.\ ...": bad flag in substitute command: '}'
sed: 1: "/elapsed/{s/.undergoin ...": bad flag in substitute command: '}'
sed: 1: "/% done/{s/.About (. ...": bad flag in substitute command: '}'                             ] 0% done
sed: 1: "/elapsed/{s/Stats: (. ...": bad flag in substitute command: '}'
sed: 1: "/remaining/{s/. ((.\ ...": bad flag in substitute command: '}'
sed: 1: "/elapsed/{s/.undergoin ...": bad flag in substitute command: '}'
sed: 1: "/% done/{s/.About (. ...": bad flag in substitute command: '}'                             ] 0% done
sed: 1: "/elapsed/{s/Stats: (. ...": bad flag in substitute command: '}'
sed: 1: "/remaining/{s/. ((.\ ...": bad flag in substitute command: '}'
sed: 1: "/elapsed/{s/.undergoin ...": bad flag in substitute command: '}'
sed: 1: "/% done/{s/.About (. ...": bad flag in substitute command: '}'                             ] 0% done
sed: 1: "/elapsed/{s/Stats: (. ...": bad flag in substitute command: '}'
sed: 1: "/remaining/{s/. ((.\ ...": bad flag in substitute command: '}'
sed: 1: "/elapsed/{s/.undergoin ...": bad flag in substitute command: '}'
sed: 1: "/% done/{s/.About (. ...": bad flag in substitute command: '}'                             ] 0% done
sed: 1: "/elapsed/{s/Stats: (. ...": bad flag in substitute command: '}'
sed: 1: "/remaining/{s/. ((.\ ...": bad flag in substitute command: '}'
sed: 1: "/elapsed/{s/.undergoin ...": bad flag in substitute command: '}'
sed: 1: "/% done/{s/.About (. ...": bad flag in substitute command: '}'                             ] 0% done
sed: 1: "/elapsed/{s/Stats: (. ...": bad flag in substitute command: '}'
sed: 1: "/remaining/{s/. ((.\ ...": bad flag in substitute command: '}'
sed: 1: "/elapsed/{s/.undergoin ...": bad flag in substitute command: '}'
sed: 1: "/% done/{s/.About (. ...": bad flag in substitute command: '}'                             ] 0% done
sed: 1: "/elapsed/{s/Stats: (. ...": bad flag in substitute command: '}'
sed: 1: "/remaining/{s/. ((.\ ...": bad flag in substitute command: '}'
sed: 1: "/elapsed/{s/.undergoin ...": bad flag in substitute command: '}'
sed: 1: "/% done/{s/.About (. ...": bad flag in substitute command: '}'                             ] 0% done
sed: 1: "/elapsed/{s/Stats: (. ...": bad flag in substitute command: '}'
sed: 1: "/remaining/{s/. ((.\ ...": bad flag in substitute command: '}'
sed: 1: "/elapsed/{s/.undergoin ...": bad flag in substitute command: '}'
sed: 1: "/% done/{s/.About (. ...": bad flag in substitute command: '}'                             ] 0% done
sed: 1: "/elapsed/{s/Stats: (. ...": bad flag in substitute command: '}'
sed: 1: "/remaining/{s/. ((.\ ...": bad flag in substitute command: '}'
sed: 1: "/elapsed/{s/.undergoin ...": bad flag in substitute command: '}'
sed: 1: "/% done/{s/.About (. ...": bad flag in substitute command: '}'                             ] 0% done
sed: 1: "/elapsed/{s/Stats: (. ...": bad flag in substitute command: '}'
sed: 1: "/remaining/{s/. ((.\ ...": bad flag in substitute command: '}'
sed: 1: "/elapsed/{s/.undergoin ...": bad flag in substitute command: '}'
sed: 1: "/% done/{s/.About (. ...": bad flag in substitute command: '}'                             ] 0% done
sed: 1: "/elapsed/{s/Stats: (. ...": bad flag in substitute command: '}'
sed: 1: "/remaining/{s/. ((.\ ...": bad flag in substitute command: '}'
sed: 1: "/elapsed/{s/.undergoin ...": bad flag in substitute command: '}'
sed: 1: "/% done/{s/.About (. ...": bad flag in substitute command: '}'                             ] 0% done
sed: 1: "/elapsed/{s/Stats: (. ...": bad flag in substitute command: '}'
sed: 1: "/remaining/{s/. ((.\ ...": bad flag in substitute command: '}'
sed: 1: "/elapsed/{s/.undergoin ...": bad flag in substitute command: '}'
sed: 1: "/% done/{s/.About (. ...": bad flag in substitute command: '}'                             ] 0% done
sed: 1: "/elapsed/{s/Stats: (. ...": bad flag in substitute command: '}'
sed: 1: "/remaining/{s/. ((.\ ...": bad flag in substitute command: '}'
sed: 1: "/elapsed/{s/.undergoin ...": bad flag in substitute command: '}'
sed: 1: "/% done/{s/.About (. ...": bad flag in substitute command: '}'                             ] 0% done
sed: 1: "/elapsed/{s/Stats: (. ...": bad flag in substitute command: '}'
sed: 1: "/remaining/{s/. ((.*\ ...": bad flag in substitute command: '}'
`

install gnu-sed and setting up an alias within the script to point to the gnu-sed solves this issue.
MacOS doesn't ship with GNU-sed which this script expects.

Use Homebrew to install GNU-sed, then replace all instances of sed with gsed


Powershell 7.3.0 does not show posh-git status bar

System Details

posh-git version/path:
PowerShell version: 7.3.0
Git version:  2.37.1.windows.1
Operating system name and version: [Version 10.0.19045.2251]

Issue Description
I am experiencing a problem with Powershell 7.30 in windows terminal (ver 1.15.2875.0) specifically that the status bar oes not appear in folders with git repository.
Troubleshooting done so far:

Un-&Reinstalled git, Windows Terminal and posh-git
Checked Execution-Policies; posh-git is loded correctly

My suspision would be that the methods to add the status bar to the PowerShell promt have changed because of the autotabbing for git working as usual.

I can confirm the same issue with the same versions mentioned above.
My temporary workaround is to use the Windows PowerShell profile (being version 5.1.19041) within Windows Terminal, instead of PowerShell (Core). The posh-git status bar shows as expected there.

I'm suspecting this line is not returning correctly:
https://github.com/dahlbyk/posh-git/blob/b8c1f17262263072e10bfe5067c62ed64ad12e64/src/GitUtils.ps1#L331

Same issue here with PowerShell 7.3.0

I'm using PowerShell 7.3.1 and seeing the same issue.

I think I found a workaround. If you replace lines 326-331 in ./src/GitUtils.ps1 with the code below, it works. For some reason, I needed to get rid of the "2" after the branch argument in the status command. Also, it doesn't like having an empty string in the pathspec argument, hence not including $untrackedFilesOption when it's empty.
https://github.com/dahlbyk/posh-git/blob/b8c1f17262263072e10bfe5067c62ed64ad12e64/src/GitUtils.ps1#L326-L331
$psMajor = $($PSVersionTable.PSVersion.Major) -as [int]
$psMinor = $($PSVersionTable.PSVersion.Minor) -as [int]

$status = $null
$untrackedFilesOption = if ($settings.UntrackedFilesMode) { "-u$($settings.UntrackedFilesMode)" } else { "" }
if ($psMajor -ge 7 -and $psMinor -ge 3) {
    if ($untrackedFilesOption.Length -gt 0) {
        $status = Invoke-Utf8ConsoleCommand { git -c core.quotepath=false -c color.status=false status $untrackedFilesOption --short --branch }
    }
    else {
        $status = Invoke-Utf8ConsoleCommand { git -c core.quotepath=false -c color.status=false status --short --branch }
    }
}
else {
    $status = Invoke-Utf8ConsoleCommand { git -c core.quotepath=false -c color.status=false status $untrackedFilesOption --short --branch 2>$null }
}

I think I found a workaround. If you replace lines 326-331 in ./src/GitUtils.ps1 with the code below, it works. For some reason, I needed to get rid of the "2" after the branch argument in the status command, as well as the ">$null". Git doesn't like having an empty string in the pathspec argument either, hence not including $untrackedFilesOption when it's empty.
https://github.com/dahlbyk/posh-git/blob/b8c1f17262263072e10bfe5067c62ed64ad12e64/src/GitUtils.ps1#L326-L331
$psMajor = $($PSVersionTable.PSVersion.Major) -as [int]
$psMinor = $($PSVersionTable.PSVersion.Minor) -as [int]

$status = $null
$untrackedFilesOption = if ($settings.UntrackedFilesMode) { "-u$($settings.UntrackedFilesMode)" } else { "" }
if (($psMajor -ge 7 -and $psMinor -ge 3) -or $psMajor -gt 7) {
    if ($untrackedFilesOption.Length -gt 0) {
        $status = Invoke-Utf8ConsoleCommand { git -c core.quotepath=false -c color.status=false status $untrackedFilesOption --short --branch }
    }
    else {
        $status = Invoke-Utf8ConsoleCommand { git -c core.quotepath=false -c color.status=false status --short --branch }
    }
}
else {
    $status = Invoke-Utf8ConsoleCommand { git -c core.quotepath=false -c color.status=false status $untrackedFilesOption --short --branch 2>$null }
}

Confirmed fixed in PS 7.3.x, also works in previous versions.
My code replacement in GitUtils.ps1 was slightly different:
Replace this line (236ish) with @tcomben s fix:
$status = Invoke-Utf8ConsoleCommand { git -c core.quotepath=false -c color.status=false status $untrackedFilesOption --short --branch 2>$null }

I realised I encountered this issue because I installed poshgit using chocolatey, which, according to https://community.chocolatey.org/packages/poshgit, it's latest version stalled at <IP_ADDRESS>.
Installing latest via
git clone https://github.com/dahlbyk/posh-git.git && .\install.ps1
fixed it for me.


travis build pipeline for windows

Apparently pkg-config in Windows does not care about the PKG_CONFIG_PATH. That's causing pkg-config to not find seabolt17.pc.
We have that file. And we know the location of that file. We need to figure out how to make it visible to pkg-config.

seabolt17.pc is now visible pkg-config
The new issue: There's a directory containing C header files. We need to make it visible to the C compiler.


Static Analysis Issues

Boost: 1.71.0_1574063579
Beast: 266
Hi @vinniefalco thank you for the great work!
We are using the static analysis tool coverity which reports us the following issues:

In websocket/impl/write.hpp

198            net::async_write(impl.stream(),
199                buffers_cat(impl.wr_fb.data(), cb_),
200                    beast::detail::bind_continuation(std::move(*this)));
201            bytes_transferred_ += clamp(fh_.len);

CID 135675 (#3-4 of 4): Identical code for different branches (IDENTICAL_BRANCHES)
identical_branches: The same code is executed when the condition impl->check_stop_now(ec) is true or false, because the code in the if-then branch and after the if statement is identical. Should the if statement be removed?
202            if(impl.check_stop_now(ec))
203                goto upcall;
   	implicit_else: The code from the above if-then branch is identical to the code after the if statement.
204            goto upcall;
205        }

in websocket/impl/write.hpp

263            impl.wr_fb.clear();
264            detail::write<flat_static_buffer_base>(
265                impl.wr_fb, fh_);
266            n = clamp(remain_, impl.wr_buf_size);

CID 135672 (#11-12 of 12): Unchecked return value from library (CHECKED_RETURN)
9. check_return: Calling boost::asio::buffer_copy(boost::asio::mutable_buffers_1 const(boost::asio::buffer(impl.wr_buf.get(), n)), this->cb_) without checking return value. This library function may fail and return an error code. [Note: The source code implementation of the function has been overridden by a builtin model.]
267            net::buffer_copy(net::buffer(
268                impl.wr_buf.get(), n), cb_);
269            detail::mask_inplace(net::buffer(

in websocket/impl/read.hpp

602        upcall:
603            impl.rd_block.try_unlock(this);

CID 135670 (#5-8 of 8): Unchecked return value (CHECKED_RETURN)
24. check_return: Calling maybe_invoke without checking return value (as is done elsewhere 10 out of 12 times).
604            impl.op_r_close.maybe_invoke();
605            if(impl.wr_block.try_unlock(this))
606                impl.op_close.maybe_invoke()
607                    || impl.op_idle_ping.maybe_invoke()
608                    || impl.op_ping.maybe_invoke()
609                    || impl.op_wr.maybe_invoke();
610            this->complete(cont, ec, bytes_written_);

in core/impl/basic_stream.hpp

642template<class Protocol, class Executor, class RatePolicy>
643basic_stream<Protocol, Executor, RatePolicy>::

CID 131433 (#2 of 2): Uncaught exception (UNCAUGHT_EXCEPT)
exn_spec_violation: An exception of type boost::wrapexcept<boost::system::system_error> is thrown but the throw list noexcept doesn't allow it to be thrown. This will cause a call to unexpected() which usually calls terminate().
644~basic_stream()
645{
646    // the shared object can outlive *this,
647    // cancel any operations so the shared
648    // object is destroyed as soon as possible.
   	fun_call_w_exception: Called function throws an exception of type boost::wrapexcept<boost::system::system_error>. [show details]
649    impl_->close();
650}

Hi @erikzenkerLogmein ,
Many thanks for reporting this.
Are you able to give me the command line use used to create this output? It would make it much easier for me to test against an MCVE.

Obviously you would need a Coverity license, and the program installed... I think the author gave sufficient information for this issue to be actionable (or not actionable, as is more likely the case).

Yeah, they just quoted me $12,000 for a 5-pack. No discount for open source maintainers. It's a bit above my budget :-)

The last one, impl_->close(); is already fixed in master, and the others look like false positives.

Thx @madmongo1, for fixing #1876!
@vinniefalco so I guess unchecked return value is intentional.
Thus, we will update to the upcoming boost version and mark the other issues as intentional.


privatednsnamespace READY=FALSE

What happened?
When creating a new privateDNSNamespace it becomes never ready (READY=FALSE)
How can we reproduce it?
Create a privateDNSNamespace:
apiVersion: servicediscovery.aws.crossplane.io/v1alpha1
kind: PrivateDNSNamespace
metadata:
  name: testnamespace.svc.cluster.local
spec:
  forProvider:
    region: eu-central-1
    name: testnamespace.svc.cluster.local
    vpc: vpc-123a125c
  providerConfigRef:
    name: dev-1234567890

It never becomes ready:
Status:
  At Provider:
    Operation ID:  k43dmiizk4aavpwxahj524zy2nmibeib-kq2xlla5
  Conditions:
    Last Transition Time:  2021-06-21T18:07:34Z
    Reason:                Creating
    Status:                False
    Type:                  Ready
    Last Transition Time:  2021-06-21T18:07:34Z
    Reason:                ReconcileSuccess
    Status:                True
    Type:                  Synced
Events:                    <none>

What environment did it happen in?
Crossplane version:
Crossplane version: v1.1.0
Provider AWS: v0.19.0

hi @RafalMaleska ;)
can you check against aws-api your status ?
because in general privatednsnamespace is working on my side:
apiVersion: servicediscovery.aws.crossplane.io/v1alpha1
kind: PrivateDNSNamespace
metadata:
  name: example-privatednsnamespace
spec:
  providerConfigRef:
    name: default
  forProvider:
    region: eu-central-1
    description: "test"
    name: crossplane-private-dns-namespace-test
    vpc: vpc-aab73bc0
    tags:
      - key: cluster
        value: "my-cluster"

Status:
  At Provider:
    Operation ID:  6acudes4m6hepmczrig5bexhvppzqkfw-kq87n4af
  Conditions:
    Last Transition Time:  2021-06-22T15:37:30Z
    Reason:                Available
    Status:                True
    Type:                  Ready
    Last Transition Time:  2021-06-22T15:36:27Z
    Reason:                ReconcileSuccess
    Status:                True
    Type:                  Synced

aws servicediscovery get-operation --operation-id 6acudes4m6hepmczrig5bexhvppzqkfw-kq87n4af
{
    "Operation": {
        "Id": "6acudes4m6hepmczrig5bexhvppzqkfw-kq87n4af",
        "Type": "CREATE_NAMESPACE",
        "Status": "SUCCESS",
        "CreateDate": "2021-06-22T17:36:28.023000+02:00",
        "UpdateDate": "2021-06-22T17:37:08.175000+02:00",
        "Targets": {
            "NAMESPACE": "ns-ngfbsrd3zfnp3qnj"
        }
    }
}

Hi Chris, for me, the operations Status was also SUCCESS, however, the Condition Type Ready Stand weiterhin auf False

@haarchri the privateDNSnamespace is created and managed by crossplane but for some reason the status is READY=FALSE.
This may have further impact when using it in a composition.

okay in our environment we see
 Reason:                Available
    Status:                True
    Type:                  Ready

with same Crossplane and Provider Version - IAM for Provider is enabled  to Check Operations ?

I've been able to achieve the same (Reason Available, Ready True) when importing existing namespaces ( w\ external-name).
We only experience this issue creating new namespaces. Could you try this @haarchri ?

@blut added PR #740 in my testing it was a problem if the k8s resource is without description ...


ArgoCD Integration - Pull Model

The current OCM and ArgoCD integration is primary push model, which means the ArgoCD controller on the hub cluster needs to directly communicate with the target managed clusters. This approach does not use OCM's hub and spoke cluster post registration secured communication channel.
We are proposing implementing an ArgoCD integration pull model that leverages the OCM registration, placement and manifestwork APIs. We will introduce a new CRD call the MulticlusterApplicationSet that will evaluate cluster placements and create manifestwork wrapping ArgoCD application template to the target clusters. Once the ArgoCD application template lands on the target clusters, the ArgoCD controller will evaluate the template and deploy the application.

After discussing with the ArgoCD community. We will not introduce a new CRD.
Instead we will:

Enhance and leverage our existing GitOpsCluster to help with importing OCM managed clusters to ArgoCD cluster inventory.
Create a new controller that wraps Applications with ManifestWork so that the Application will be propagated to the managed cluster.


Dont show dock icon

As per the discussion on here, this change hides the dock icon. I made the change locally, build the app and it's working!

ugh realized it still doesn't work.

what lol that was the fix??? jeez. thanks!


remove unused options

These options have no influence on the package

@ericLemanissier  thanks for the suggestions. The options are not used atm but some of them have to be applied in the def _configure_cmake(self) function via cmake.definitions . I am aware that a lot of them are not useful but all of them should not be thrown away just because I'm not using them on the _configure_cmake(self) function in the recipe now.
I'll remove the unwanted options in my original MR

ok!


Check if ssh-agent is running for openstack deployment

@ilausuch what description? Do you think current one is missing something?

Indeed seems to be missing the why?
Plus this may conflict with the caasp4 deployment, which need an ssh-agent but has to create it itself and store the sock. Would like to see one caasp4 run with this check

Well, this was something I already did for kvm deployment and just forgot to include it in openstack part.

Plus this may conflict with the caasp4 deployment, which need an ssh-agent but has to create it

Does it? This CI step that starts ssh-agent is executed before deploy_caasp4 step, right?

Does it? This CI step that starts ssh-agent is executed before deploy_caasp4 step, right?

It should but tbh I dont know what the check does :)

My comment was about the PR description that is empty.

OK, I could add something to PR description , but more important is the commit message itself


Updates to api-doc and missing validation

Overview
This PR includes the following proposed change(s):

Fixed bug wherein the app would break due to custom validation trying to validate something that wasn't there
Added user_name array where it was missing before

Type of change

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Refactoring / Documentation
[ ] Version change

if your change is a breaking change, please add breaking change label to this PR
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[x] New and existing unit tests pass locally with my changes

/rebase

Gonna bring this in too, need to unblock form submission for testers.


The -p flag causes exiting to be delayed

Using the -p x flag to add a delay between requests also means that hitting ctrl+c takes x seconds to exit,

Would this need to be solved with the context package and cancel each worker goroutine or do we simply need the main goroutine to listen for keyboard interrupt and exit the process?


Use es2019 build target

This is a fix for https://github.com/nhost/nhost/issues/1623 and https://github.com/nhost/nhost/issues/1607
It seems that if the target to esbuild is es2019 it transforms the nullish coalescing (??) operator, which causes problem in React Native and Expo. Starting from es2020 it does not transforms those.
From the documentation I thought we should use es2020, but actually we need es2019 to make it work.
https://esbuild.docschina.org/content-types/#javascript
I am not sure of all the implications of the build targets, but I guess targeting older JS versions is overall safe.

I'm closing this in favor of #1665. Could you take a quick look at that PR @beepsoft?
There is a quite recent issue with RN Expo + Web though that is tracked here: https://github.com/expo/expo-cli/issues/4639. I also experienced this in the example project.


Using environment markers in install_requires fails

The changelog for 20.5 seems to indicate that environment markers should be supported in install_requires, but I was not able to get that to work. Specifically, the example from PEP 496 causes an error.
Environment:
pip (9.0.1)
pkg-resources (0.0.0)
setuptools (32.3.1)
wheel (0.30.0a0)

Procedure to reproduce:
python
Python 2.7.12 (default, Nov 19 2016, 06:48:10)
[GCC 5.4.0 20160609] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> from setuptools import setup
>>> setup(install_requires=["pywin32 > 1.0 : sys.platform == 'win32'"])
error in setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers; Invalid requirement, parse error at "': sys.pl'"

The same error occurs when the argument is passed to setup_requires, but the exception is not caught.

PEP 496 is a draft. The relevant (and accepted PEP) is PEP 508.
You need to use ; and not :.


Make registry auto update when new custom nodes added

This would allow the user to use newly pulled docker images without the need to restart SkiffUI.

Converted to issue and solved


Malformed user configs explode violently

If JSON is incorrect, it seems to be trying to load the file multiple times and printing a stacktrace each time.

This seems to require some kind of config validation. Maybe node-convic is worth a look

I think simply avoiding retrying in a loop and printing the same error endlessly should be good enough. I don't think we need full blown validation. Most settings will just error gracefully if they are malformed, for example resulting in impossible to connect to nonsense server using nonsense port.

Is this issue already in 1.5.0 or did we add this in pre-releases? My guess is that it's been there since beginning of time.
If so, let's remove the 2.0.0 milestone for now or we'll never get to release it.

Okay so I did look into it a little bit, and of course it was not what I was expecting...
It would be easy if it was a retry in loop, but it turns out this is how we watch the user folder (config, code) 😮
We could create an ignored array to avoid re-parsing broken files, but that would kill the point of auto-loading as file could not be fixed without restarting the server anymore...
So until we do proper file/directory watching on new or changed files (something like this or better), there is nothing we can do to get rid of this IMO.
I'm removing the 2.0.0 milestone as fixing this specific issue will require a much bigger work on the autoload feature, but hopefully we'll get there someday...

This was fixed in 2.0.0 by using require on the config.

I actually don't think this has been fixed, per my previous comment.

Yeah you're right, I misread.


feat: Align metrics

Codecov Report
Attention: Patch coverage is 72.86432% with 54 lines in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

vicinity/backends/basic.py
72.22%
25 Missing :warning:

vicinity/backends/usearch.py
45.00%
11 Missing :warning:

vicinity/backends/annoy.py
63.63%
8 Missing :warning:

vicinity/backends/faiss.py
78.94%
4 Missing :warning:

vicinity/backends/pynndescent.py
78.57%
3 Missing :warning:

vicinity/utils.py
90.00%
2 Missing :warning:

vicinity/backends/hnsw.py
90.90%
1 Missing :warning:

Files with missing lines
Coverage Δ

vicinity/__init__.py
100.00% <100.00%> (ø)

vicinity/vicinity.py
98.86% <100.00%> (+1.18%)
:arrow_up:

vicinity/backends/hnsw.py
95.23% <90.90%> (-3.04%)
:arrow_down:

vicinity/utils.py
95.74% <90.00%> (-4.26%)
:arrow_down:

vicinity/backends/pynndescent.py
95.65% <78.57%> (-1.13%)
:arrow_down:

vicinity/backends/faiss.py
92.50% <78.94%> (ø)

vicinity/backends/annoy.py
89.41% <63.63%> (-5.53%)
:arrow_down:

vicinity/backends/usearch.py
85.13% <45.00%> (ø)

vicinity/backends/basic.py
78.94% <72.22%> (-11.38%)
:arrow_down:


Azuma crashing game

using any lightning based effect from azuma crashes server
unsure about other effects atm but still playing with the rest of the mod...

Again. I need logs. Also this is a mod issue, not a pack issue. Therefore could you report all these crashes to the mod repos?


[semantic-sil] Update DIMemoryObjectInfo so that it recognizes destroy_value where it recognizes strong_release

[semantic-sil] Update DIMemoryObjectInfo so that it recognizes destroy_value where it recognizes strong_release
rdar://31521023

@swift-ci smoke test and merge


Fit to current pm2

Thanks for a useful capistrano plugin. This changes is not compatible for now. However, since pm2 now recommends the use of ecosystem file and our way of usage also changes accordingly, I think that capistrano-pm2 needs to follow it as well.
Behavior

task
current behavior
new behavior

restart
start or restart gracefully
only exec restart

reload
nothing
only exec reload

start_or_restart
nothing
only exec startOrRestart

start_or_reload
nothing
only exec startOrReload

Add attributes

pm2_options: options for pm2 CLI
pm2_config: ecosystem file for pm2

@tomhanoldt hello?

Hello,
sorry for my late answer, and thanks for your PRs.
What do you thinl of taking ownership of this repository? …I use it not any more in production, so maybe is a good point for you to maintain and extend this?
Best Tom

Am 10.02.2019 um 12:30 schrieb Tomohisa Oda<EMAIL_ADDRESS>@tomhanoldt https://github.com/tomhanoldt hello?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/creative-workflow/capistrano-pm2/pull/10#issuecomment-462124565, or mute the thread https://github.com/notifications/unsubscribe-auth/ACKffE7rYf98N4QZ3IEN49r0MC8LjJEOks5vMALwgaJpZM4XjNL8.

Tom Hanoldt - Creative-Workflow
Gaudystraße 17A
10437 Berlin
Tel:  +49 (0)176 49564869
E-Mail<EMAIL_ADDRESS><EMAIL_ADDRESS>Web: www.tomhanoldt.info https://www.tomhanoldt.info/
https://www.xing.com/profile/Tom_Hanoldt https://de.linkedin.com/pub/tom-hanoldt/b0/a44/925 https://www.tomhanoldt.info/blog https://www.github.com/creative-workflow https://www.facebook.com/profile.php?id0010040683738

Thank you for your reply. I fork it and use it in production environment. I am interested in ownership. If you do not mind, I welcome the repository transfer. Also, the authority of Gem.


Release latest master

Hi Jon,
I've finally been able to check out Magnolia; what a slick project!
We'll be testing it in combination with spray-json at Driver and put together a small proof-of-concept library. We're using annotations on types, a feature that is only available in the unreleased master version of Magnolia. Is there anything we can do to help get out a release soon?

Hi Jakob!
I've just made a quick release now (as it has been quite a while) and the pending changes I had been waiting to finish are almost certainly not going to get finished this week...
So you should be able to get version 0.7.0 from Maven Central quite soon!
You might also like to have a look at https://github.com/propensive/adversaria/ which could potentially provide the same annotations functionality in an ortthogonal way, (though hasn't been tested much in combination with Magnolia yet).
/cc @kevinwright @sirthias @joroKr21

And thanks for the positive feedback! Makes me very happy to hear people using it successfully! :)

Wow, that was quick! The PR has barely been in there for a week...

It's more in anticipation that it might take me a while longer to finish off the "v1.0" updates I had planned, so let's get this version out the door and then I'll take it from there... ;)

Hey, works for me.  Gives that extra touch of stability for building other things on top of.

and by "other stuff", I of course mean adding attribute serialisation to Xylophone - using annotations to denote elements vs attributes


⚠ Samsung has degraded performance

In b8f9afe, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 1830 ms

Resolved: Samsung performance has improved in 49a8a85.


iOS 13 and Photo Notifications - No Sensor/Switch Showing

Hello,
I tried tonight to enable iOS 13 Photo Notifications via Homebridge Config UI X. I am using a Wyze Cam 2, and the video feed works fine in HomeKit. I am using homebridge-camera-ffmpeg v2.0.0, Homebridge is v1.1.1 and node.js v12.18.1, all on a Raspberry Pi 3 using Raspbian Buster.
When I enable iOS 13 Photo Notifications, and then restart Homebridge, I do not get any new dummy sensor/switch appearing in HomeKit. For good measure, I also did a full reboot of the Raspberry Pi and still no change.
For fun, I also tried turning the "doorbell" feature on, but that also did nothing.
For reference, here is the info from config.json (which as I mentioned I am using Homebridge Config UI X to modify):
{
"cameras": [
{
"name": "Wyze_A",
"manufacturer": "Wyze",
"model": "Wyze Cam v2",
"motion": true,
"doorbell": false,
"videoConfig": {
"source": "-rtsp_transport tcp -re -i<EMAIL_ADDRESS>"stillImageSource": "-rtsp_transport tcp -re -i<EMAIL_ADDRESS>"maxStreams": 2,
"maxFPS": 8,
"maxBitrate": 564,
"vcodec": "copy",
"audio": false
}
}
],
"platform": "Camera-ffmpeg"
},
Hoping someone has a suggestion/solution on how to make this work. Thanks!

did you have camera-ffmpeg setup before you updated to 2.0.0?

did you have camera-ffmpeg setup before you updated to 2.0.0?

Yes - I had it setup with this camera prior to the update.

And just for clarity, I never tried the iOS 13 Photo Notifications option prior to the update, so that was never turned on until now.

Inspired by your comment, I fully uninstalled the plugin and then reinstalled it. Set up the camera again from scratch and now the dummy switch/sensors appear to be showing. Thanks again.


Add palette picker/editor experiment

Adds an experiment for use in pxt-arcade that lets you change your project's color palette!
Obviously this has gone through zero design iterations, so don't expect it to turn into anything other than an experiment anytime soon. Most of the palettes came from lospec.com via @jwunderl's pxt-color extension.
I went through and reordered the colors so that they are somewhat close to our default palette. This makes it so that the gallery sprites are still somewhat usable.

(in future, we could add support for importing using e.g. the .hex format output of lospec

or gimp palette format

but that's more on the 'possibly nice to have' side~)

So I can no longer do this ?

Why???

@LucasMayhew this hasn't been pushed to beta yet. in the meantime, you can still override the palette in the usual way

Okay

any idea when it will be pushed like in the next few weeks or so

Also it is soo hard to change the palette the old way

I am sooo Hyped for this thing

@LucasMayhew this is now in beta. you can turn it on like this:

Click the gearwheel menu
Click "About"
Click "experiments"
Click "Change Color Palette"
Click "Go back" and wait for the page to reload
You should now see "Project Settings" in the gearwheel menu when you have a project open

YES YES YES YES YES YES YES YES YES YES YES YES YES YES YES


Cannot play multilplayer with Essential

https://mclo.gs/6wkgacf It always brings up this msg whenever we try joining eachothers worlds, we have no idea what is causing it other than it's proably because of kotlin

That is an issue with Essential. If you can disable using QUIC for networking, that may fix the issue.

More technical information
The part of the log discussing this issue is here:

[17:39:07] [Server Connector #2/WARN]: Failed to convert "quilt.zfs://org-jetbrains-kotlin-kotlin-stdlib-1.9.10.i0:0/" to file:
java.lang.UnsupportedOperationException: Only the default FileSystem supports 'Path.toFile()', class org.quiltmc.loader.impl.filesystem.QuiltZipPath '/' does not!
	at org.quiltmc.loader.impl.filesystem.QuiltBasePath.toFile(QuiltBasePath.java:580) ~[quilt-loader-0.23.1.jar:?]
	at gg.essential.util.HelpersKt.findCodeSource(helpers.kt:156) ~[Essential (fabric_1.20.1).jar:?]
	at gg.essential.util.ForkedJvm$Companion.defaultClassPath(ForkedJvm.kt:72) ~[Essential (fabric_1.20.1).jar:?]
	at gg.essential.sps.quic.jvm.ForkedJvmQuicConnector$Companion.quicConnectorClasspath(ForkedJvmQuicConnector.kt:58) ~[Essential (fabric_1.20.1).jar:?]
	at gg.essential.sps.quic.jvm.ForkedJvmQuicConnector$Companion.access$quicConnectorClasspath(ForkedJvmQuicConnector.kt:45) ~[Essential (fabric_1.20.1).jar:?]
	at gg.essential.sps.quic.jvm.ForkedJvmQuicConnector.(ForkedJvmQuicConnector.kt:20) ~[Essential (fabric_1.20.1).jar:?]
	at gg.essential.sps.quic.jvm.ForkedJvmClientQuicStream.(ForkedJvmClientQuicStream.kt:21) ~[Essential (fabric_1.20.1).jar:?]
	at gg.essential.network.connectionmanager.ice.IceManager.createClientAgent(IceManager.java:398) ~[Essential (fabric_1.20.1).jar:?]
	at net.minecraft.class_2535.redirect$jnp000$injectLocalChannel(class_2535.java:4088) ~[minecraft-1.20.1.i0:0/:?]
	at net.minecraft.class_2535.method_52271(class_2535.java:344) ~[minecraft-1.20.1.i0:0/:?]
	at net.minecraft.class_412$1.run(class_412.java:99) ~[minecraft-1.20.1.i0:0/:?]
[17:39:07] [Server Connector #2/ERROR]: Couldn't connect to server
java.lang.UnsupportedOperationException: Failed to find class kotlin.Unit jar location
	at gg.essential.util.ForkedJvm$Companion.defaultClassPath(ForkedJvm.kt:72) ~[Essential (fabric_1.20.1).jar:?]
	at gg.essential.sps.quic.jvm.ForkedJvmQuicConnector$Companion.quicConnectorClasspath(ForkedJvmQuicConnector.kt:58) ~[Essential (fabric_1.20.1).jar:?]
	at gg.essential.sps.quic.jvm.ForkedJvmQuicConnector$Companion.access$quicConnectorClasspath(ForkedJvmQuicConnector.kt:45) ~[Essential (fabric_1.20.1).jar:?]
	at gg.essential.sps.quic.jvm.ForkedJvmQuicConnector.(ForkedJvmQuicConnector.kt:20) ~[Essential (fabric_1.20.1).jar:?]
	at gg.essential.sps.quic.jvm.ForkedJvmClientQuicStream.(ForkedJvmClientQuicStream.kt:21) ~[Essential (fabric_1.20.1).jar:?]
	at gg.essential.network.connectionmanager.ice.IceManager.createClientAgent(IceManager.java:398) ~[Essential (fabric_1.20.1).jar:?]
	at net.minecraft.class_2535.redirect$jnp000$injectLocalChannel(class_2535.java:4088) ~[minecraft-1.20.1.i0:0/:?]
	at net.minecraft.class_2535.method_52271(class_2535.java:344) ~[minecraft-1.20.1.i0:0/:?]
	at net.minecraft.class_412$1.run(class_412.java:99) ~[minecraft-1.20.1.i0:0/:?]

There are a couple main points:

Essential (trying to use QUIC for secure communications) is attempting to load the kotlin.Unit class, a default class in Kotlin's standard libraries.
a Path.toFile() call works only for files in the operating system's default filesystem (e.x. the mods folder, but not the inner data of mods themselves)

The first point there is what leads to the second point. Essential is attempting to find the "default" classpath in order to properly undergo QUIC communications. As a result, it searches through some files.
Below is an approximation of the original code based on a Vineflower decompilation of Essential:
// ForkedJvm.kt
class ForkedJvm {
	// ...
	companion object {
		fun defaultClassPath(): String {
			val sourcePaths = listOf(
				ForkedJvm::class.java,
				Unit::class.java,
				// ...
			).map {
				findCodeSource(it) // declared in Helpers.kt
					?: error("Failed to find class $it jar location") // The message above
			}

			// ...
		}
	}
}

This is a relatively simple process, but upon trying to locate the kotlin.Unit class (the second call), it fails in findCodeSource which propagates the error by returning null.
// Helpers.kt
fun findCodeSource(clazz: Class<*>): CodeSource? {
	var url = clazz.protectionDomain?.codeSource?.location
	// ... Some modifications to the URL. This ends up being a Quilt ZipFS URL eventually.
	// if url ends in .jar, return a Jar code source
	// else:
	try {
		val dir = url // These lines are where the problem occurs. The lines converting the URL into a Path
			.toURI()  // likely are inlined from some other function originating likely from either Kotlin's
			.toPath() // standard libraries or some helper somewhere else in Essential. This ends up causing an
			.toFile() // <-- UnsupportedOperationException
			.toPath() // when Quilt's ZipFS is erroneously given a request to convert to a (native filesystem) File.

		return CodeSource.Directory(dir)
	} catch (e: Exception) {
		LOGGER.warn("Failed to convert \"$url\" to file:", e)
		return null // This is where the null is returned, causing the error to propagate.
	}
}


[Bug] Can't edit and save files inside a compressed file

Describe the bug
When I try to edit/modify a file that is openable by Notepads, it refuses to properly save the that file INSIDE a compressed file. And on top of that, if I indirectly want to Edit it, it opens as an empty file instead.
To Reproduce

Create a txt file, write something in it, then save.
Using 7-zip/winrar whatever, compress that file.
Open the compressed file, try to edit it, then save or open/view it directly and see its empty/nothing was changed.

Expected behavior
It should be saved normally inside a compressed file. With default notepad, it works perfectly find.
Screenshots
Even better, a video!
https://github.com/0x7c13/Notepads/assets/81096046/87c27118-71e1-447c-a8cc-4b4bb0e43a01
Also once finished with the edit/modify, the archiver program should prompt the following message: (it happens with regular notepad save)

Desktop (please complete the following information):

OS: Windows 10 22H2 19045.4123
Version latest <IP_ADDRESS>

Additional context
I got tired of the white theme on notepad, that's why I fell in love with Notepads, although this issue annoys me, please have a look, thank you!
P.S: I think it has to do with the fact that Notepads.exe is located in a hidden/restricted folder by default (Program Files/WindowsApps), compared to Notepad.exe, which is in system32.

I am not sure if OG notepad does something automatically for re-saving a file in archive. Not sure how to achieve this with UWP tho.

OG Notepad doesn't automatically re-save it, but as you can see on that small picture, it allows this option.
I think it has to do with the fact that this program's exe is in a restricted folder (UWP), hence 7-zip can't access it normally and unlock its full potential.
Any chance for a UWP-less version perhaps, just to test it out? Thank you.


Lessons In Chemistry

Book Title?
Lessons In Chemistry
Author?
Bonnie Garmus
Genre of Book?
Historical Fiction

Assigned this to you @86pushkar24 🚀

Can I also get assigned?

Can I also get assigned?

Hey @DanielZhang111 , this issue has been closed but you can raise a new issue for any other book. Please read the Contribution Guidelines

Ok thanks


Surface Razor Components in View Components topic

Fixes #11986
I assume it's best to allow the section to explain out the VC scenario first. I'll try that first here.

@guardrex  how about https://docs.microsoft.com/en-us/aspnet/core/blazor/components

Yeah ... I guessed that it would be an either way scenario on the link. The components topic jumps in with both feet, so that's why I was a little concerned. The reader will prob click out to the overview anyway, so let's go with the direct link.


Verification of control plane requires username and password

The verification of the control plane suggests that we check component status to see how things are looking:
https://github.com/kelseyhightower/kubernetes-the-hard-way/blob/master/docs/08-bootstrapping-kubernetes-controllers.md#verification
When running this I'm prompted for a username / password which I don't recall configuring.
searbe@controller-0:~$ kubectl get componentstatuses --kubeconfig admin.kubeconfig
Please enter Username:

Perhaps I'm missing something - how do I authenticate myself here, assuming I've followed the guide correctly up to this point?

This appears to work if  --kubeconfig admin.kubeconfig is ommited. I'll leave this issue open for other's judgement as I'm not sure whether the option should be removed, the configuration file should be tweaked, or whether I missed something else along the way.

It works without the kubeconfig flag because you are using the deprecated insecure access on <IP_ADDRESS>:8080 which will be removed in the future.
If you are getting prompted for a password it maybe because the admin.kubeconfig is not setup correctly.


svg icons

emojicon e1a-honeybee is missing.
We need to make it bit more mobile responsive with some media queries.

About the names. honeybe is a name of the shortname bee, names is a such kind of text, for an instance :
iphone:
   name : 'mobile phone'

there should be some js conversion into shortNames when intending to use regular/long names .


Add a way to access the script's hash from inside the script itself

Feature description
Would it be useful to have procedures in the kernel to obtain the context's script hash?
Why is this feature needed?
While implementing https://github.com/0xPolygonMiden/clob-example/pull/3 I needed to get the note script's hash from inside the note script itself. I ended up compiling and hashing the script and adding the digest to the note's inputs.
Maybe this case is too niche to deserve a dedicated procedure.

I think this is a good functionality to add. It could be done via a relatively simple kernel procedure - e.g., something like miden::note::get_script_hash. This procedure would return the hash of the currently executing note or error out if we are not currently executing any notes.

Addressed by #995


Add 3rd party content warning to Ingress Controllers concept

Relates to https://github.com/kubernetes/website/issues/20232
/kind cleanup

/lgtm
Thanks.


FancyZones does not support outlook

Microsoft PowerToys version
"0.59.0"
Running as admin

[X] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
I cannot highlight zones for the Latest Outlook 365.
excluded apps window does not include outlook.exe or anything else.
✔ Expected Behavior
all other applications can be assigned to a FancyZone, but outlook.
❌ Actual Behavior
No response
Other Software
No response

jut to confirm 0.58.0 version worked fine with outlook. this is a new issue for 0.59.0

@tall27 https://github.com/microsoft/PowerToys/issues/18657 - have you checked that?

There are several bug reports for this issue relating to multiple applications. They are likely all related to the same root cause.

@tall27 #18657 - have you checked that?

That was it!
I've unchecked "show this window on all desktops" and outlooks works again as expected.
Thank you for the tip.


error while running

Exception in thread Thread-1:
Traceback (most recent call last):
File "C:\python37\lib\threading.py", line 926, in _bootstrap_inner
self.run()
File "C:\python37\lib\threading.py", line 870, in run
self._target(*self._args, **self._kwargs)
File "C:\python37\lib\site-packages\plyer\platforms\win\libs\balloontip.py", line 208, in balloon_tip
WindowsBalloonTip(**kwargs)
File "C:\python37\lib\site-packages\plyer\platforms\win\libs\balloontip.py", line 141, in init
self.notify(title, message, app_name)
File "C:\python37\lib\site-packages\plyer\platforms\win\libs\balloontip.py", line 190, in notify
raise Exception('Shell_NotifyIconW failed.')
Exception: Shell_NotifyIconW failed.

@bunnysworld, I think this error is in package plyer , not pyleri

i reinstalled the package after that facing the same issue

Closing this issue as it is not related to pyleri


[HUDI-4258] Fix when HoodieTable removes data file before the end of Flink job

What is the purpose of the pull request
Fix concurrency issue when endInput event is handled before the previous events. It could lead to removal of data files before the end of Flink job.
Brief change log

added new method executeSync to NonThrownExecutor
endInput is handled using executeSync in StreamWriteOperatorCoordinator now

Verify this pull request
This change added tests and can be verified as follows:

added unit test to TestStreamWriteOperatorCoordinator

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@hudi-bot run azure

Thanks for the fix @trushev , i have left a minor comment ~

Thanks for the review :)

@hudi-bot run azure

There were CI issue, so I squashed and rebased on the latest master to trigger a new pipeline

@hudi-bot run azure

Pipeline is finally green, I think this PR can be merged


Restart failed downloads

The download of the video file will not skipped if the previous attempt was interrupted (zero size).

Thanks


Server / Kernel title bar appears to "vibrate" when you move a slider

Don't have an easy way to capture gifs right now. But create something like a widget slider and then drag it back and forth. The title bar changes between busy and idle. But the size of the text area is not locked, so the entire bar changes size. So it looks like it's vibrating. Not a big deal, but very visually distracting.

Thanks for your feedback. This issue will be addressed once VS Code adds native support for editing notebooks. Please see https://github.com/microsoft/vscode/issues/91987. You can experiment by using VS Code insiders, which defaults to the new notebook experience.


Failed to create arsession

When i run augmented images project the app shows a toast "Failed to create ArSession"

Solved the issue

I am also facing the same problem while trying the "augmented images" sample.


Initial @web/test-runner reporter

This still has some problems and has caused me to question a few things but it overall does work. This doesn't include browser information yet (trying to figure out where to get it).

I'll resolve conflicts in a bit

Ok should be good to review now.


Support to Hugo extend for linux arm64

Hi,
I'm using Hugo Extend for my sites in the Linux Arm server (Ampere), I built Hugo for that, but I think it could be great to release a Hugo Extend version for Linux ARM64. I take a look the goreleaser file, and it seems that it just adds a line there. e.g.:
[...]

    id: hugo_extended_linux
    ldflags: -s -w -X github.com/gohugoio/hugo/common/hugo.buildDate={{.Date}} -X github.com/gohugoio/hugo/common/hugo.commitHash={{ .ShortCommit }} -X github.com/gohugoio/hugo/common/hugo.vendorInfo=gohugoio
    env:
      - CGO_ENABLED=1
    flags:
      - -buildmode
      - exe
      - -tags
      - extended
    goos:
      - linux
    goarch:
      - amd64
      - arm64

[...]

Is it right? I can create PR if a good thing. :)

I take a look the goreleaser file, and it seems that it just adds a line there.

No, that will not work.
We build this from one single Docker container and each target GOARCH needs a working build setup for C/C++. Turns out, this isn't trivial (it's easy to do AMD64 OR ARM64, but not both at the same time).
That said, this is duplicate of #8257


feat: Qdrant datetime filtering support

Qdrant v1.8 introduced support for datetime filtering.
This PR updates the Qdrant integration to support the same in Haystack.

Blocked by https://github.com/qdrant/qdrant-client/issues/532.

Hey, @Anush008!
Thanks for the PR.
Ping me when the bug is fixed...

Hi @anakin87. Could you please rerun the CI?
qdrant-client==1.8.1 should now fix this.

Released: https://pypi.org/project/qdrant-haystack/3.2.0/


~ Fix required validation for single selection

Example with single select http://embed.plnkr.co/2lN4NRFN8VF6kUZqPs0M/

thanks but it already works without this change. however, i needed to update the css on the demo so the red highlight would show up

thx a lot!


add a way to see interactions/replies from non-bridged users

Right now replies and likes from users who have enabled the bridge are mirrored on the other side, but replies and likes from users who haven't obviously can't be handled the same way. But I think it's still pretty important to be able to see them. This is easy on the Bluesky side (just open the post on bsky.app), but not in the other direction.  You actually don't know if maybe one of your posts went viral there and you got 20 comments, but you only know about 3 of them from users who are bridged. For me this is a pretty important missing piece if I wanted to "sunset" my Mastodon account and only post on Bluesky with a mirror via Bridgy, because I'd want to see what people are replying on Fedi.
@snarfed said that these responses should be showing up here: https://fed.brid.gy/bsky/mackuba.eu/notifications, but they don't seem to be. I've asked<EMAIL_ADDRESS>(who isn't following the bridge account) to make a few comments on my bridged Bluesky account, and these replies aren't shown anywhere, e.g.: https://mastodon.social/@rademaker/112814874617446470 (the Bridgy notifications list only shows that he followed me, but no further interactions).

Thanks for filing! Yeah we started short circuiting out of processing earlier for non-opted-in accounts a while ago; sorry for misleading you here.
I'm a little cautious about this for non-opted-in accounts, but only a little, it's hopefully reasonable.
However, re:

For me this is a pretty important missing piece if I wanted to "sunset" my Mastodon account and only post on Bluesky with a mirror via Bridgy, because I'd want to see what people are replying on Fedi.

Totally get it, that's an entirely reasonable desire, but I'm not sure if BF will be able to fully serve it. BF is only really designed to bridge interactions that are bridgeable, not to collect and serve all interactions. I can consider it, and look into it, but I'm not too optimistic just yet.

This is easy on the Bluesky side (just open the post on bsky.app), but not in the other direction

Actually, now that I think about it, it would also be useful for Mastodon accounts bridged to Bluesky, since this way you can easily check responses to a single post, but not to any of your posts in general.

Now that service DMs are implemented, would it be reasonable to provide (some) notifications that way?
If yes, then this probably should be an opt-in feature that the bridging-enabled- and help-messages mention.
It should also be possible to activate it for follows and replies independently of boosts and likes, since the latter could get spammy if supported.


feat(kubelet): Create an agent pod only

Description

This provides the user with the ability to create only a single pod containing an infrastructure agent container when combined with kubelet.enabled == false and ksm.enabled == false.
This is useful when you already have your kubernetes infrastructure monitored by newrelic at the infrastructure layer and you want to retrieve the length of some redis keys which are relevant at the application layer.
For more details of the motivation for this change please see https://github.com/newrelic/nri-kubernetes/issues/943
Type of change

[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] New feature / enhancement (non-breaking change which adds functionality)
[ ] Security fix
[ ] Bug fix (non-breaking change which fixes an issue)

Checklist:

[ ] Add changelog entry following the contributing guide
[ ] Documentation has been updated
[ ] This change requires changes in testing:

[ ] unit tests
[ ] E2E tests

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


As a user I can add a task with a from/to date/time range so that I can add tasks which take some time to complete.

I will start by changing the DueByDate and DueByTime class to store a range of dates and times, minimally two.

[x] Implementation (38848d29b163a5699448eee2920d9b8c35e52fe2)
[x] Tests

Implementation

User must surround their task details in 'single apostrophes' as in add 'detail'. Otherwise, an error will be thrown.
Users cannot enter ambiguously detailed tasks, as in add ''. An error message asking if block was meant if thrown.
User is now able to enter start date, time, end date, time if they require.

add 'new task' from 10/13 3pm to 10/14 4pm

If user types in an earlier end date before the start date, CMDo assumes he has made an error and does the correction for him.

add 'sala' from 10/12 to 10/11         # adds task 'sala' from 10/11 to 10/12

If user types in an earlier end time before the start time on the same day, CMDo also assumes he has made an error and does the correction for him, though they may mean 1pm the next day. This is intentional.

add 'sala' from 3pm to 1pm             # adds task 'sala' from 1pm to 3pm on the same day

Refer to issue #35 for follow up.


ARROW-2603: [Python] Allow date and datetime subclassing

When converting a pandas dataframe holding subclasses of date/datetime objects, arrow raises an ArrowInvalid exception
import pandas as pd
import pyarrow as pa
import datetime

classMyDate(datetime.date):
    pass

date_array = [MyDate(2000, 1, 1)]
df = pd.DataFrame({"date": pd.Series(date_array, dtype=object)})

table = pa.Table.from_pandas(df)

Codecov Report

Merging #2061 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2061      +/-   ##
==========================================
+ Coverage   86.35%   86.35%   +<.01%
==========================================
  Files         242      230      -12
  Lines       41184    40345     -839
==========================================
- Hits        35564    34840     -724
+ Misses       5620     5505     -115

Impacted Files
Coverage Δ

cpp/src/arrow/python/numpy_to_arrow.cc
88.99% <100%> (ø)
:arrow_up:

python/pyarrow/tests/test_convert_builtin.py
99.59% <100%> (+0.02%)
:arrow_up:

cpp/src/arrow/python/python_to_arrow.cc
88.29% <100%> (ø)
:arrow_up:

python/pyarrow/tests/test_convert_pandas.py
94.88% <100%> (+0.09%)
:arrow_up:

rust/src/buffer.rs

rust/src/error.rs

rust/src/memory.rs

rust/src/list.rs

rust/src/datatypes.rs

rust/src/builder.rs

... and 7 more

Continue to review full report at Codecov.

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


chore: Update training-operator tag

5ef6c405df2bb1bf1d3ede988cd43433eff2e956 is the last commit built from post-submit job. We will need this commit tag to release beta training operator version
/cc @kubeflow/wg-training-leads

/test kubeflow-tf-operator-presubmit

/approve


Fix on GET /users/:term/:id/campaigns when reportbacks are deleted on Drupal

What's this PR do?
When reportbacks get deleted on Drupal, they're not removed from Northstar. So when Northstar queries for them, there is no 'data' object that comes back in the response and it dies. This PR just does an isset check as a quick fix to prevent the process from terminating.
Questions
Thoughts on if we'll ultimately want to remove the reportback id from the user's document on Northstar too? Potentially could happen here - which feels iffy. Or could happen as an event triggered from Drupal when a reportback is deleted there.

:+1:
we should probably handle removing the reportback/signups from drupal.

Cool. I'll create an issue for it.


Restore lines for WSL and SSH domains

See discussion from this comment.
Currently there is no way to restore the lines for WSL and SSH domains, but it seems like there might be a way to do so as per the discussion in #39.

Continuing discussion from #39 here:

I agree. This is promising but is a separate feature and could use more testing.
I think the solution will involve some variation of the following Wezterm lua functions to get the pane_id and return the scrollback text from it:
root.pane:pane_id() -- 23 was the pane_id for my SSH domain session
wezterm.mux.get_pane(23):get_lines_as_escapes(1000))

It might also make sense to abstract the get_lines_as_escapes into its own local function.

That should be the same behaviour as just root.pane:get_lines_as_escapes(1000) when saving the pane

Continuing discussion from #39 here @joncrangle:

I agree. This is promising but is a separate feature and could use more testing.
I think the solution will involve some variation of the following Wezterm lua functions to get the pane_id and return the scrollback text from it:
root.pane:pane_id() -- 23 was the pane_id for my SSH domain session
wezterm.mux.get_pane(23):get_lines_as_escapes(1000))

It might also make sense to abstract the get_lines_as_escapes into its own local function.

That should be the same behaviour as just root.pane:get_lines_as_escapes(1000) when saving the pane

I found I was unable to inject output in panes within domains, and found this in the docs:
"Not all panes support this method; at the time of writing, this works for local panes but not for multiplexer panes."

For now I see several directions this could be taken.

Do not save the text for non-local domains
use get_lines_as_text to store the text only and then send it to the pane with send_text forcing either the user to ctrl-c or we insert a ctrl-c control sequence at the end.
Spawn the pane with a command like printf <text> ; sh

One of these are great tho :neutral_face:

For now I see several directions this could be taken.

Do not save the text for non-local domains
use get_lines_as_text to store the text only and then send it to the pane with send_text forcing either the user to ctrl-c or we insert a ctrl-c control sequence at the end.
Spawn the pane with a command like printf <text> ; sh

One of these are great tho :neutral_face:

I'd be interested in whether others have feedback, because this is not a feature I'm overly concerned about for my personal workflow.
This plugin currently satisfies the pain point I was having by reattaching to SSH and SSHMUX sessions. SSHMUX sessions are already very similar to tmux in that running processes are continued as long as the Wezterm daemon on the server side is still running. I am personally less concerned about the scrollback, but can completely understand if someone else would find it useful.

For now I think not saving lines for non-local panes is the best option, but if someone is reading this and has input on this, please feel free to provide ideas or feedback 😄


Set default selected index to 0

By consistency with ipywidgets.

cf https://github.com/jupyter-widgets/ipywidgets/pull/1495


Forex class use incorrect SekType (FOP but should be CASH)

Unless I'm misunderstanding something, the Forex class  sets the secType property incorrectly. It should be of type CASH instead of FOP.
This works:
const contract: Contract = {
  symbol: "USD",
  exchange: "IDEALPRO",
  currency: "CAD",
  secType: SecType.CASH,
};

ib.reqHistoricalData(4002, contract, '20221126 15:59:00 US/Eastern', "1 M", BarSizeSetting.DAYS_ONE, "MIDPOINT", 1, 1, `false);

but this does not:
const contract = new Forex("EUR", "USD") // this does not work, because the Forex class set secType to FOP (which is dedicated to options)

ib.reqHistoricalData(4002, contract, '20221126 15:59:00 US/Eastern', "1 M", BarSizeSetting.DAYS_ONE, "MIDPOINT", 1, 1, false);

Thanks for the bug report.


Haxelib icon should only be used for .haxelib

+ add haxe_libraries (used by a Haxe package manager called lix)

Urm.. well, the relevant docs section doesn't seem to mention what the folder is called exactly.
https://lib.haxe.org/documentation/per-project-setup/
But I can point you to the relevant sources:
https://github.com/HaxeFoundation/haxelib/blob/56390adc9c804463cda1dff279bf70be08743432/src/haxelib/client/Main.hx#L160
haxelib newrepo creates a folder named .haxelib, and it definitely needs the dot to be considered a library repository.

Looks like @robertohuertasm and you missed the dot when the PR was submitted (https://github.com/vscode-icons/vscode-icons/pull/224/files)

I think at that initial stage in the plugin history the dot was automatically supported.


create-reproducible with flat design files

openlane.steps.Step

load()'s pdk_root flag can now be passed as None
create_reproducible():

Added flatten to public API, which flattens the file structure with the exception of the PDK
Modified behavior of flatten to allow including the PDK (which isn't flattened)
Generated run_ol.sh now passes ARGV to the final command (backwards compatible with old behavior)

Primary use for this: run command now accepts --pdk-root flag

Fixed bug with ejected reproducible scripts keeping some nix store paths.

@kareefardi I isolated the changes related to reproducible creation specifically. Will handle the rest in a separate PR


How to add module php-gettext

How to enable php gettext module?
I try enter shell and install from apt but failed
Thanks

Horrible process but the solution came out okay enough.
In your site directory (eg /opt/easyengine/sites/mysite.com make a file named php.Dockerfile:
FROM easyengine/php:v4.6.6
USER root
RUN docker-php-ext-install gettext; \
	docker-php-ext-enable gettext
RUN sed -i 's/;extension=gettext/extension=gettext/' /usr/local/etc/php/php.ini
USER www-data

Edit /opt/easyengine/sites/mysite.com/docker-compose.yml and edit the first service definition to be like this:
version: '3.5'

services:

  php:
    #image: easyengine/php:v4.6.6
    build:
      context: /opt/easyengine/sites/mysite.com/.docker
      dockerfile: /opt/easyengine/sites/mysite.com/php.Dockerfile
    restart: always
# rest of file below
# ...

Then so we have a blank build context directory for the docker container, run mkdir /opt/easyengine/sites/mysite.com/.docker
Then run either ee site enable mysite.com --force or cd to the site directory and run docker-compose -f docker-compose.yml  up -d --build php.

Honestly though after reading some of the easyengine docker files I will not be using it again in the future.


chore(Web components): configure docs in storybook

Closes #6423
The current storybook for web components does not include docs, configure the same.
What did you change?
Updated storybook configuration to show content from *.mdx files to show in the docs section.
How did you test and verify your work?
Storybook

Should double check if #6300 changes are included since we closed that as a duplicate.

Should double check if #6300 changes are included since we closed that as a duplicate.

I've double checked with closed issue:

Docs tab is working as expected now.
Imports are updated.
For links to samples, since we don't have stackblitz samples set up for web components and the ones in .mdx files point to core carbon, this section is removed. we could update this once we have set up stackblitz samples for web components package.


Fix missing faces when loading quad meshes

Due to unwanted side effect when disabling automatic triangulation in assimp loader.
Fixes #228

There is another thing to fix: after 1 animation step, the mesh is flat rendered.

Ok just found out why the mesh is flat rendered.
This is due to the activation of RADIUM_WITH_TEXTURE, which makes the AssimpGeometryDataLoader to load duplicate vertices.

Ok, so what do we do ? You just need to disable textures and it's good, right ?


Split IteratorUtil up into 2 non-API classes

Moved methods not useful to users to non-API classes
Created IterConfigUtil and SystemIteratorUtil for internal use
Cleaned up loadIterators methods and created IterLoad class to assist

I could have put the source iterator and classCache objects into IterLoad but this introduced generics to the class.  I was also thinking of writing IterConfigUtil in a way to not have any generics.

All ITs are passing.

@keith-turner did you want to look at these changes any closer or you OK?


Dramaqueen

https://github.com/reb-hash/github-slideshow/issues/19

https://github.com/reb-hash/github-slideshow/issues/19

#19


adding in examples and tests for amend2 generators

This contains examples for amend2 generators
Will address https://github.com/spotify/ratatool/issues/93

Codecov Report

Merging #101 into master will increase coverage by 0.74%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #101      +/-   ##
==========================================
+ Coverage   70.67%   71.41%   +0.74%
==========================================
  Files          24       24
  Lines        1081     1102      +21
  Branches      125      125
==========================================
+ Hits          764      787      +23
+ Misses        317      315       -2

Impacted Files
Coverage Δ

...tify/ratatool/examples/scalacheck/GenExample.scala
100% <100%> (ø)
:arrow_up:

...la/com/spotify/ratatool/samplers/AvroSampler.scala
100% <0%> (+3.12%)
:arrow_up:

Continue to review full report at Codecov.

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

👍 Good to merge when tests pass


PSReadline <IP_ADDRESS> problems on $ErrorActionPreference = 'Stop'

PSReadline <IP_ADDRESS> is not working sometimes in the following scenario

Start PowerShell without loading PSReadline, i.e. without the profile
Invoke commands

PS> $ErrorActionPreference = 'Stop'
PS> ipmo PSReadline

As a result, PSReadline seems to be not working. Interestingly, on Win7 x32 it started to work after a while. On two other machines (Win8 x64, Win7 x64) it does not work.
The problem did not exist in PSReadline <IP_ADDRESS>

Thanks - it looks like PSReadline is failing on the first command line, but the module is loaded, and the second command line will work as expected.
If you looked at $error, you'd see something like:
Cannot convert argument "options", with value: "PSConsoleUtilities.PSConsoleReadlineOptions", for "SetOptions" to type
"PSConsoleUtilities.SetPSReadlineOption": "Cannot convert the "PSConsoleUtilities.PSConsoleReadlineOptions" value of
type "PSConsoleUtilities.PSConsoleReadlineOptions" to type "PSConsoleUtilities.SetPSReadlineOption"."
Cannot convert argument "options", with value: "PSConsoleUtilities.PSConsoleReadlineOptions", for "SetOptions" to type
"PSConsoleUtilities.SetPSReadlineOption": "Cannot convert the "PSConsoleUtilities.PSConsoleReadlineOptions" value of
type "PSConsoleUtilities.PSConsoleReadlineOptions" to type "PSConsoleUtilities.SetPSReadlineOption"."
At C:\Users\Jason\Documents\WindowsPowerShell\Modules\psreadline\PSReadLine.psm1:31 char:13

        [PSConsoleUtilities.PSConsoleReadLine]::SetOptions($options)

        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [], MethodException
FullyQualifiedErrorId : MethodArgumentConversionInvalidCastArgument

Sorry, this is a little bit not clear to me. Is the problem confirmed? I.e. is it to be fixed?
BTW. In some cases $Error is populated and the PSReadline works. And sometimes $Error is empty and PSReadline does not work. Weird...

The error is (mostly) harmless - the fix is to just remove the line of script causing the error - it was unnecessary.  It does mean I broke the scenario of loading history yourself (HistorySaveStyle.SaveNothing), but I'll bet few people use that.

I just pushed a fix for the problem I saw.  Let me know if you still see the problem with this fix (you can just edit PSReadline.psm1, you don't need to build anything.)

What you see and what I have is different. When I posted the issue I never saw this error. I did check $Error, I am sure. When you answered, I tried again and I saw this error just once. Now I cannot repro this error.
But the most important PSReadline does not work for me when I load it with $ErrorActionPreference = 'Stop'. This is not "harmless".

I have tried the updated PSReadline.psm1. The problem seems to be fixed. Thank you!


Attribute relay message processing errors as client errors rather than client errors

All errors in processing incoming relay messages are treated the same: as a server error, even if it's a problem coming from the client. For now we've increased the threshold on acceptable number of server errors, but we need to properly attribute client errors as client errors, make a separate chart, and reduce the alarm threshold for server errors.

Closing; these errors will be attributed as they occur. The major ones have already been fixed.


Fixed typo in messages.po

@mistercrunch can you please review the changes?

I think this specific msgid was generated dynamically in the past but does not exist in the source anymore. That's why it shows as commented. Translators shouldn't translate strings that do not exist anymore (commented out).  It's not bad to leave them there in case they were to resurface.
In case of lines with typo that have been corrected in the source, we could delete them or just leave them there. Fixing them here wouldn't do anything.


Add bash completion for docker {node rm,swarm leave} -f

Ref: #28196
ping @sdurrheimer for zsh completion

Thanks @thaJeztah @vdemeester @albers .
It is a great work!