Kafka Subscriber stops subscribing and doesn't recover automatically

Dapr runtime 1.12.2 on EKS
We have a Kafka PubSub subscriber, working fine, then we got the following 2 error message in the sidecar:

Processing Kafka message: ds-applicationtenant-created/0/9 [key=]"

encountered a retriable error while publishing a subscribed message to topic tenant-created, err: Post \"http://<IP_ADDRESS>:8087/subscription/tenant-created\": context canceled"
Too many failed attempts at processing Kafka message: ds-applicationtenant-created/0/9 [key=]. Error: context canceled.

Immediately after, the following logs were output (which looks like kafka client is attempting to reinitialise)
time="2023-12-12T12:04:57.361666911Z" level=debug msg="consumer/broker/0 closed dead subscription to ds-applicationtenant-created/0\n" app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-ksgjg scope=dapr.contrib type=log ver=1.12.2
time="2023-12-12T12:04:57.364608263Z" level=debug msg="consumer/broker/1 closed dead subscription to ds-applicationuser-updated/0\n" app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-ksgjg scope=dapr.contrib type=log ver=1.12.2
time="2023-12-12T12:04:57.364694874Z" level=debug msg="consumergroup/session/sarama-9a9a9c70-d096-41a5-9ce2-d57c5d3c3b91/74 heartbeat loop stopped\n" app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-ksgjg scope=dapr.contrib type=log ver=1.12.2
time="2023-12-12T12:04:57.364714625Z" level=debug msg="consumergroup/session/sarama-9a9a9c70-d096-41a5-9ce2-d57c5d3c3b91/74 released\n" app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-ksgjg scope=dapr.contrib type=log ver=1.12.2
time="2023-12-12T12:04:57.364731435Z" level=debug msg="Starting loop to consume." app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-ksgjg scope=dapr.contrib type=log ver=1.12.2
time="2023-12-12T12:04:57.364793657Z" level=debug msg="client/metadata fetching metadata for [ds-applicationuser-updated] from broker REDACTED-kafka-kafka-0.REDACTED-kafka-kafka-brokers.dotm-services.svc:9092\n" app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-ksgjg scope=dapr.contrib type=log ver=1.12.2
time="2023-12-12T12:04:57.366867612Z" level=debug msg="client/metadata fetching metadata for [ds-applicationuser-updated] from broker REDACTED-kafka-kafka-0.REDACTED-kafka-kafka-brokers.dotm-services.svc:9092\n" app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-ksgjg scope=dapr.contrib type=log ver=1.12.2
time="2023-12-12T12:04:57.373229493Z" level=debug msg="client/coordinator requesting coordinator for ds-application.dbx from REDACTED-kafka-kafka-0.REDACTED-kafka-kafka-brokers.dotm-services.svc:9092\n" app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-ksgjg scope=dapr.contrib type=log ver=1.12.2
time="2023-12-12T12:04:57.37419421Z" level=debug msg="client/coordinator coordinator for ds-application.dbx is #2 (REDACTED-kafka-kafka-2.REDACTED-kafka-kafka-brokers.dotm-services.svc:9092)\n" app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-ksgjg scope=dapr.contrib type=log ver=1.12.2
time="2023-12-12T12:04:57.47748149Z" level=debug msg="consumer/broker/1 accumulated 1 new subscriptions\n" app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-ksgjg scope=dapr.contrib type=log ver=1.12.2
time="2023-12-12T12:04:57.477578492Z" level=debug msg="consumer/broker/1 added subscription to ds-applicationuser-updated/0\n" app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-ksgjg scope=dapr.contrib type=log ver=1.12.2

At this point the kafka partition was one message behind.
Then there was a exactly 5 minute period of no logs, after which the following logs were produced

time="2023-12-12T12:14:42.452646345Z" level=debug msg="ClientID is the default of 'sarama', you should consider setting it to something application-specific." app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-ksgjg scope=dapr.contrib type=log ver=1.12.2
time="2023-12-12T12:14:42.452767447Z" level=debug msg="client/metadata fetching metadata for all topics from broker REDACTED-kafka-kafka-0.REDACTED-kafka-kafka-brokers.dotm-services.svc:9092\n" app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-ksgjg scope=dapr.contrib type=log ver=1.12.2
time="2023-12-12T12:14:42.456015284Z" level=debug msg="Connected to broker at REDACTED-kafka-kafka-0.REDACTED-kafka-kafka-brokers.dotm-services.svc:9092 (registered as #0)\n" app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-ksgjg scope=dapr.contrib type=log ver=1.12.2
time="2023-12-12T12:14:51.967077716Z" level=debug msg="client/metadata fetching metadata for all topics from broker REDACTED-kafka-kafka-0.REDACTED-kafka-kafka-brokers.dotm-services.svc:9092\n" app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-ksgjg scope=dapr.contrib type=log ver=1.12.2

Then no further logs for 15 minutes.
Kafka partition is still one message behind.

So, given that nothing had happened by this point, we decided to restart the subscribing pod.
Success! - the message is consumed immediately, interestingly it had expired (not sure if this is a red herring though)

Processing Kafka message: ds-applicationtenant-created/0/9 [key=]
dropping expired pub/sub event ede85cb8-9f54-4705-8420-4fca90f40466 as of 2023-12-12T12:21:09Z

time="2023-12-12T12:23:51.437188279Z" level=info msg="dapr initialized. Status: Running. Init Elapsed 15267ms" app_id=dbx instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-c949w scope=dapr.runtime type=log ver=1.12.2
time="2023-12-12T12:23:51.538416198Z" level=debug msg="consumer/broker/1 accumulated 1 new subscriptions\n" app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-c949w scope=dapr.contrib type=log ver=1.12.2
time="2023-12-12T12:23:51.538541168Z" level=debug msg="consumer/broker/1 added subscription to ds-applicationuser-updated/0\n" app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-c949w scope=dapr.contrib type=log ver=1.12.2
time="2023-12-12T12:23:51.538436258Z" level=debug msg="consumer/broker/0 accumulated 1 new subscriptions\n" app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-c949w scope=dapr.contrib type=log ver=1.12.2
time="2023-12-12T12:23:51.538727619Z" level=debug msg="consumer/broker/0 added subscription to ds-applicationtenant-created/0\n" app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-c949w scope=dapr.contrib type=log ver=1.12.2
time="2023-12-12T12:23:51.539936474Z" level=debug msg="Processing Kafka message: ds-applicationtenant-created/0/9 [key=]" app_id=dbx component="general-purpose-pubsub (pubsub.kafka/v1)" instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-c949w scope=dapr.contrib type=log ver=1.12.2
time="2023-12-12T12:23:51.540069626Z" level=warning msg="dropping expired pub/sub event ede85cb8-9f54-4705-8420-4fca90f40466 as of 2023-12-12T12:21:09Z" app_id=dbx instance=sciplat-svc-REDACTED-manager-deployment-859fd76f4c-c949w scope=dapr.runtime.processor.pubsub type=log ver=1.12.2

So my question is, did the message expiring (via the TTL built into dapr Pubsub ttlInSeconds) cause the entire sidecar to stop consuming from that partition?
My initial thinking was yes, it must be related to TTL...however the processing error occurred at 12:04:57Z and the message didn't expire until 12:21:09Z So there is a 15 minute window there where the message had not yet expired, but the kafka client was not doing anything.
So if its not the TTL, what is the problem?

Going back to the original 2 original log message
encountered a retriable error while publishing a subscribed message to topic tenant-created, err: Post \"http://<IP_ADDRESS>:8087/subscription/tenant-created\": context canceled : tracked down to here
Too many failed attempts at processing Kafka message: ds-applicationtenant-created/0/9 [key=]. Error: context canceled. : tracked down to here
What does context canceled mean as the detail here? It's not very helpful!
My guess is given the first log message, its trying to relay the message to the app channel, but the app channel is not available for some reason? maybe a connection timed out to the app channel?
So, assuming that the app channel goes away for some reason, maybe its under load / busy, and is unable to serve any more requests for a period of time, what is the recourse here?
My assumption is that if the app channel can't serve any more requests, and therefor the retries have exhausted, then there is little point of the sidecar remaining in the kafka ConsumerGroup. My guess is, It should gracefully eject its self from the consumerGroup, and then begin a process of attempting to rejoin, otherwise the sidecar just gets stuck with no further redelivery attempts.
Another assumption is that while all this is happening the Kafka client inside Dapr is still sending heartbeats to Kafka, even though it's not attempting any further redeliveries, hence why the sidecar hasn't been forcefully ejected from the consumer group by Kafka server.

Looks like there are a bunch of un-documented options for Kafka that allow configuring the retry behavior: https://github.com/dapr/components-contrib/blob/79adc565c17ad8936048896591cd205a6609ad67/common/component/kafka/kafka.go#L123-L128
I don't see this documented anywhere and it's unclear what the defaults are

@ItalyPaleAle it defaults to true :)
https://github.com/dapr/components-contrib/blob/02c6b21ec6216ffce45d47cc1dbb1a6e674f370f/pubsub/kafka/kafka.go#L108

do not close

This issue is pervasive in all of our pubsub components. Once a consumer errors out.. nothing reinits that consumer.
https://github.com/dapr/components-contrib/blob/9acfcc16b81f9d19c47dc4572d0a00c4b24518eb/common/component/kafka/consumer.go#L352C1-L354C6
But the same is also true for Azure EventHubs https://github.com/dapr/components-contrib/issues/3325

Welcome to the mess that is Dapr @olitomlinson - too many cooks in the kitchen not meeting the restaurant standard :D

@berndverst Ouch!
At the very least if a consumer errors out I would expect the dapr process/sidecar to exit so that it can be recycled by whatever orchestration is in place (k8s restarting the pod, being the common example)
Having a stalled subscriber particularly for things like Kafka is very painful indeed. (more painful than a traditional transactional broker like Service Bus)

Rereading the code - it does seem like it would restart the subscriber. So not sure what is going on there. Not a low hanging fruit after all.
I will not be looking into this. Contributions welcome.

@berndverst Is there any plan to address this issue? A stalled subscriber makes the Kafka connector practically unusable in a production environment. If the consumer errors out and doesn't recover automatically it severely impacts reliability. At the very least, the sidecar process should have some os.Exit call when encountering a permanent error, allowing the pod to be restarted by Kubernetes. This would help maintain service availability until a permanent fix is implemented. Without this the Kafka connector cannot be considered production-ready.


Add support for custom timestamp format in history

As per discussion in https://github.com/robbyrussell/oh-my-zsh/pull/6589, this PR adds the ability to define a custom format for timestamps shown by the history command.

If HIST_STAMPS is unset or empty the default behavior is maintained and no timestamps are shown.
The custom formats mm/dd/yyy, dd.mm.yyyy, and yyyy-mm-dd are kept as well.
Otherwise HIST_STAMPS is used as a format string passed to the fc command via the -t option. The format string — as per fc's manual — "is formatted with the strftime function with the zsh extensions described for the %D{string} prompt format in Prompt Expansion." . More info on the accepted escape sequences are available via man strftime.

The PR also updates the zshrc.zsh-template  template with instructions.
See issue https://github.com/robbyrussell/oh-my-zsh/issues/6109, for further background.

I've updated your PR to simplify the case logic to include if $HIST_STAMPS is empty or undefined. Thanks for your contribution!


Large memory footprint of Cox-PH model

Hey fellas, thanks a lot for this package. I've been using it recently, and I noticed that, for some reason, it's taking a huge toll in my memory. I have a dataframe with 20k rows. Once I load the dataframe, I have a 4gb of memory in use. After running the Cox-PH model, it grows to 11gb, and stays that way, even if I place the whole thing inside a function.
sdf = CSV.read("sdf.csv", DataFrame);
function coxphcoef(sdf)
    X = Matrix(sdf[!,Not(:duracao)])
    y = sdf[!,:duracao];
    y = EventTime.(y,[true for i in y]);
    @show cphmodel = coxph(X,y);
    return cphmodel.β
end
coxphcoef(sdf);

What is going on here?
BTW, I'm in Julia 1.7.2. Using:
DataFrames v1.3.2
Survival v0.2.2

The structures that keep track of the various parts of the model fitting process, including the model object that's returned, keep some potentially very large arrays in memory. I'd be curious to know whether Julia will free memory appropriately with your coxphcoef function if you return copy(cphmodel.β); perhaps it can't otherwise prove that the entire cphmodel object isn't still needed since you're returning one of its fields. (Just a guess)


Addons Jest - Fix test matching

Issue: #4497
What I did

Changed test name matching to regex
When "jest" parameter is a string it should be mapped to array.

We can see this working in preview https://deploy-preview-4689--storybooks-official.netlify.com/?selectedKind=Addons|jest&selectedStory=withTests&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybook%2Ftests%2Fpanel
But I will add some unit tests anyway. Maybe as a separate PR


2.0.2 is out! "Purging xenos"

If there is a missing mod - it will be selected so user could easily find it
All code has been refactored (where needed)
Optimization: refactored GetLoadedMods() method in ModMergerForm - it's x1.15 times faster! 0.86 ms vs 0.99 ms
Optimization: JIT compiler uses Profiler so it could use multiple threads now - ModManagerForm starts x1.27 times faster! 434 ms instead of 550 ms (It has to improve calculation speed in all parts of application!)
After downloading a new version, old version of DoW Mod Manager will be deleted and a new shortcut will be created on the user's Desktop (thanks, FragJacker!)
In ModMergerForm Mod.State is an enum now instead of a string. That may increase some operations and reduce memory usage
There is no more storing an unnecessary reference to a new Form when it's been created
Added annotations for all methods in ModManagerForm
Small optimizations here and there

Alright I think It would be best to merge this into my other branch and turn it into 2.0.3 already. This way we can nicely wrap up all changes in a proper version history.

I am closing this pull request since all it's changes are included in pull request #16 plus some more.


AngularJS ng-grid- To show 'No Data Available' inside grid when its has empty data

Can you please provide a workaround to show a message to display something if the grid is empty, instead of showing a blank grid, it doesn't look good at all, I have tried to put  inside the div but my page is complex and the message is not shown where i wanted. how to fix this issue ?

@vijender1256 I guess you wanted to open this issue in the relevant repo: https://github.com/angular-ui/ng-grid


Branch fix UG typos and rephrase select components

In this PR I aim to address formatting issues in the UG and rephrase components adapted from AB3's UG to better fit PROperty's UG.

lgtm


Template setup

Created the template for new extension repositories.

Sure. Makes sense. I'll update the template with the latest decision.


ddev craft install step of new Craft CMS Composer project creation failing with bootstrap.php error

Is there an existing issue for this?

[X] I have searched the existing issues

Run a Diagnostic and Paste Link Here
https://gist.github.com/olets/5cf613fe736e1b7325cbd21a0c92b202 (I've ellided some things with #####)
Expected Behavior
Can install Craft CMS following the DDEV docs
Actual Behavior
Craft CMS installation fails.
Steps To Reproduce

Per https://ddev.readthedocs.io/en/latest/users/quickstart/#craft-cmsmkdir my-craft-project
cd my-craft-project
ddev config --project-type=craftcms --docroot=web --create-docroot
ddev start
ddev composer create -y --no-scripts --no-install craftcms/craft
ddev craft install

and then I get
PHP Warning:  require_once(/var/www/html/vendor/autoload.php): Failed to open stream: No such file or directory in /var/www/html/bootstrap.php on line 11

Warning: require_once(/var/www/html/vendor/autoload.php): Failed to open stream: No such file or directory in /var/www/html/bootstrap.php on line 11
PHP Fatal error:  Uncaught Error: Failed opening required '/var/www/html/vendor/autoload.php' (include_path='.:/usr/share/php') in /var/www/html/bootstrap.php:11
Stack trace:
#0 /var/www/html/craft(8): require()
#1 {main}
  thrown in /var/www/html/bootstrap.php on line 11

Fatal error: Uncaught Error: Failed opening required '/var/www/html/vendor/autoload.php' (include_path='.:/usr/share/php') in /var/www/html/bootstrap.php:11
Stack trace:
#0 /var/www/html/craft(8): require()
#1 {main}
  thrown in /var/www/html/bootstrap.php on line 11
Failed to run craft setup: exit status 255

Anything else?
No response

I do apologize. I must have made a mistake with a merge conflict, because the --no-install shouldn't be in there.
You got this from latest docs, which is unreleased, but

#4340
should fix this, please try it (everything the same, just leave out the --no-install)

Also the stable docs quickstart should work fine with current release. https://ddev.readthedocs.io/en/stable/users/quickstart/#craft-cms

Doc is fixed, thanks for reporting the problem.

Confirming that worked! Thank you!
Now I see the stable/latest url and the selector down in the corner, but didn't before and wouldn't have thought anything of it.


Issue found in rgb_to_hex.java

Issue found in rgb_to_hex.java.
r, g, and b must be initialized as type int in first method
incorrect way to print in first method

fixed issue


check offset is valid

check offset is valid based on current version

move to file offset to and check that is valid

store parsed data(vector or something) and check that offset is valid

TODO after #15 done

test with CJpgPasrseSample

Done


lint --nitpick: ignore lines python.versions, ruby.branch(es)?

Resolves https://trac.macports.org/ticket/61980
@ryandesign

This looks OK to me.

What about perl or php?


firebaseAnimatedList empty state

Empty states are important. If a query returns empty, there should be a simple way to display something to the user. This PR implements an emptyStateChild which defaults to an empty Container() to display if the query returns no data.

@kroikie can you review this, please?

Presumably this will need a test before it can be landed, e.g. verifying that transitioning to and from 0 members vs 1 or more members does the right thing.

@frederickcook If you have the time to address the feedback on this PR we'd love to see your contributions land! If you don't have the time that's fine, I'll add it to our list of patches to tweak and land when we get around to it.

Any progress on this?

@Hixie Sorry, I haven't spent enough time learning about animations yet to get this cleaned up and am busy trying to get a project shipped. If you're okay leaving it open I'd like to come back to it in a few weeks. If anyone else wants to work on it in the meantime that would be awesome!

No worries!

@thosakwe If you're interested in this one, looks like @frederickcook would be happy to accept your help!

Any progress on this or similar? It's definitely a feature I'd be interested in using. Definitely make the whole thing simpler.


Fixed #26028 -- Added an "Overriding Templates" howto.

I think this could be accepted as-is, but I'm more looking for guidance than immediate merging. Is this the right place to put this? Is this too much detail or not enough?
https://code.djangoproject.com/ticket/26028

I think the content here is quite good; direct and clearly explained.
It seems like this type of content fits best (given how Django's docs are structured) within a "how-to" guide rather than in the topic introduction, which is already quite long. I'd suggest we add a new howto/overriding-templates.txt. That would also give us room to expand this a bit; I think it should at least briefly discuss the alternative approach of overriding one app from another app. That may not be what we want to present as the preferred approach, but there are some cases where it's useful, and if we're claiming to discuss "how to override templates" it should at least be mentioned (maybe with an example where one of your apps overrides one of its admin templates?).
I also think (as a somewhat separate but related change), the existing topic guide should somewhere clarify the order that apps are checked for templates (I believe it's in INSTALLED_APPS order, but that should be verified) when APP_DIRS is set to True. I'd add this near the top, under Configuration, in the bullet point where APP_DIRS is discussed.

Haven't taken a detailed look but there are a few high-level comments. Please remember that commit message should be in past tense.

I believe I have made all the changes suggested; let me know if there's anything else that we would like in this doc.

It would be nice to add a link to this guide in docs/topics/templates.txt.
The documentation only talks about overriding templates that use same backend 'django.template.backends.django.DjangoTemplates'. It would be nice to mention about other backends too (I haven't used Jinja2 or any other backend yet so I don't know if that's possible).

Could you add a short example of overriding a HTML file?

(Sorry for duplicate comments. GitHub is having some problems at the moment: https://status.github.com/messages)

I've made some of the suggested changes. The things that were mentioned above as "nice to have" I agree would be nice to have. Are they necessary for getting this landed or can we start with what's here?
Thanks for the reviews, happy to do what's necessary to get this landed!

After updating, don't forget to uncheck "Patch needs improvement" on the ticket so that it returns to the review queue.

@phildini, do you have time to return to this?

I'm hoping to work on this soon; that said, if there's someone chomping at
the bit for it they can take it.
PJJ
http://philipjohnjames.com
On Thu, Dec 29, 2016 at 7:02 AM, Tim Graham<EMAIL_ADDRESS>wrote:

@phildini https://github.com/phildini, do you have time to return to
this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/django/django/pull/6720#issuecomment-269642162, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AArZV9a83oVtK0UzKg5sOfe67CP2yG9Eks5rM8tvgaJpZM4IuMXZ
.

Edited and merged in 7c9a83330169df1118f6bd690aed131e7c59638d, thanks!


Frame of video player not correct when in transition

Hi, I use your animator transition when presenting and dismissing detail video. The problem is the frames of two playView is totally different (diff position, width and height). So when dismissing, the player player doesn't fulfill the player view.
Hope you understand and reply. Thanks

Try 5.0.1 again , I think this issue fixed


fix(AV-1744): Removed dataset sorting by source

Updated ckan image version for AV-1744

Closing due to the image being updated in another pull request to a newer version


[release/1.4 backport] shimv1: downgrade poroccess missing log to debug

backport of https://github.com/containerd/containerd/pull/4519
relates to https://github.com/containerd/containerd/issues/4509
This Info log shows up for all exec processes that use the v1 shim
with Docker because Docker deletes the process once it receives the exit
event from containerd.

/cc @cpuguy83


CASE processing of ' is null' needs investigation

from Array aggregations:

I have an ARRAY column that either has data in the array, or does not (is empty). I need to run an aggregation on this ARRAY column such that an empty array is counted as one case, and all combinations from UNNEST(array_column) are treated as all other cases.

Using “is null” to identify empty ARRAYs seems to work…

mapdql> select count(*) from stories where cultures is null; EXPR$0 382241636 1 rows returned. mapdql> select cultures from stories where cultures is null limit 3; cultures {} {} {} 3 rows returned.

But when I try to combine “is null” with UNNEST(cultures) it doesn’t return rows with empty arrays. For example:

mapdql> select approx_count_distinct(p_id) as culture_count, case when cultures is null then 'unknown' when cultures is not null then UNNEST(cultures) end as culture from stories group by culture; culture_count|culture 2504197|a 1737845|b 2783827|c 451152|d 4 rows returned.

Is there a way to do what I’m trying to do without resorting to running two queries?

as per @dwayneberry ,

It appears that our CASE process for <arrayitem> is null> is not quite what we might want it to be, as what he is trying to do seems reasonable, we just don't seem to be getting correct answer.
I think it's likely this is a bug and should be investigated.
Looks like some stuff is a bit broken around null and not null processing in this area at a quick glance:

mapdql> select * from t6 where i1 is not null; i1 {1} {2, 3} {4, 5, 6} {} 4 rows returned. Execution time: 123 ms, Total time: 124 ms mapdql> select * from t6 where i1 is null; i1 {} 1 rows returned.

Calcite may be dropping filters with conditions involving NULL checks on arrays.
It will take some time to get a fix.
I would suggest trying the following workaround: check the condition result explicitly,
see if it's true or false, e.g. to get @dwayneberry 's first query to work, run it like this:
mapdql> select * from t6 where (i1 is not null)=true;

or
mapdql> select * from t6 where (i1 is null)=false;


when server end set new cookie value, $.cookie still return the old value.

Is jquery-cookie cache it?
When I refresh the page, $.cookie can return the new cookie value.

Hi, new issues should be opened at https://github.com/js-cookie/js-cookie/issues as from d72bb07.
Check out the latest release and confirm if the issue is fixed: https://github.com/js-cookie/js-cookie/releases


Stop Pages | Show predictions when we don't have arrival or departure times

Summary of changes
Asana Ticket: 🚉 Stop Pages | Show predictions when we don't have arrival or departure times
The data feeding into the <StopCard> component which displays on Transit Near Me and on Stop pages had status: null hardcoded into the prediction when processing data from the backend to frontend 🤦🏻♀
The other commit is because I saw a Train null -- now we check if there's a train_number first.
For example, looking at rail results for /transit-near-me?from=search-homepage&query=1%20International%20Place%2C%20Boston%2C%20MA%2C%20USA&latitude=42.35604&longitude=-71.0521404&address=1%20International%20Place%2C%20Boston%2C%20MA%2C%20USA
Updated version on left -------- original on right

Before getting review, please check the following:

[ ] Does frontend functionality render and work correctly in IE?
[ ] Have we load-tested any new pages or internal API endpoints that will receive significant traffic? See load testing docs
[ ] Are interactive elements accessible to screen readers?
[x] Have you checked for tech debt you can address in the area you're working in?
[ ] If this change involves routes, does it work correctly with pertinent "unusual" routes such as the combined Green Line, Silver Line, Foxboro commuter rail, and single-direction bus routes like the 170?
[x] Are the changes organized into self-contained commits with descriptive and well-formatted commit messages?

HA!  Omg, glad you found the hardcoded bit.  (I wonder why it was done in the first place?)
I'm trying to test it out, but seems like all the CR schedules are properly built atm with boarding status and time-based prediction.  To mimic the GTFS data that TRC provided in the screenshot (where arrival_time and departure_time is null), I tried just replacing line 30 in predictions/lib/parser.ex with nil.  I think that effectively recreates the scenario TRC was seeing, but not positive.
That caused all upcoming trips to be marked as "Scheduled", even if the train was currently boarding (which I believe is what TRC was seeing?)  Is this a proper way to test this?
MBTA.com:

Localhost:

@hannahpurcell A LOT of the time, the prediction.status is actually nil, so you might have to manually add one (in the same place you were editing) to see this in action.

@hannahpurcell A LOT of the time, the prediction.status is actually nil, so you might have to manually add one (in the same place you were editing) to see this in action.

ah, cool!  Thanks!  I changed line 34 to be "All aboard", which is the standard string on predictions, and I got the same result as the screenshot above.  Still seems like when prediction times are nil and board status exists, we're ignoring the board status?  (Or my testing is off!)

Hey I think you're onto something; if I edit the time to nil as you describe, the status stops showing too. Thanks!

@hannahpurcell It turns out trying to create predictions with nil time to render in Transit Near Me is surprisingly tricky! I took a try at it at the cbj/prediction-without-arrival-testing branch -- the results on TNM are super interesting/weird! Will dig into this more tomorrow. 🤔

We were filtering out nil prediction times pretty early on, so now I'm letting those show in the frontend but have to make a lot of upstream adjustments! I have some additional filtering to figure out but making good progress on this!

Closing as this'll all be redundant due to #1476 and the stops page redesign.


Fix tree title column size minimum width

Fixes #52891
The biggest part of #52891 has been fixed by merging #53001.
But a minor issue is left :
The column title minimum size calculation miss the margin offset of the title.
Few pixel are missing and title is always crop if no other item is wider than title.
The offset used seems to be the Stylebox bg margin.
https://github.com/godotengine/godot/blob/5aa099aaed359df6ff79fe31616b5601db2a42c5/scene/gui/tree.cpp#L3699-L3701
Just adding this offset is not sufficent. I've also added the right bg margin to get the correct width.
Maybe this need to be reviewed, i'm not sure of this right margin use even if the looking result is good.
Before:

You can see that Global Variable title is truncated.
After:

You can see that Global Variable title is entirely shown.

Thanks!


setuptools-ify

Forgive me for the directory rename, but it seems setuptools likes to impose its will on its user. I figured it's probably best to do changes like this earlier rather than later.

I'm not particularly attached to the current directory structure.  I didn't know the standard, so I copied how Reinteract did things.  But Reinteract uses autotools, and I have no intention of following that path.
The one thing I will insist upon is that the program run without being installed.  Right now, that doesn't happen, since the script can't find the slidedex package.  This could be fixed by using the LIBPATH method that I stole from Reinteract, but this would have to be adjusted during the install.
I'll take a look around to see how other Python programs handle installation before deciding whether to go with setuptools or not.

On Mon, 28 Nov 2011 16:20:27 -0800, rschroll<EMAIL_ADDRESS>wrote:

I'm not particularly attached to the current directory structure.  I
didn't know the standard, so I copied how Reinteract did things.  But
Reinteract uses autotools, and I have no intention of following that
path.

The convention I have seen (that setuptools and distribute strongly
suggest that you follow) is to place library modules in a package. The
easiest way to do this is to put them in a directory with the name of
the package (e.g. slidedex/) along with an init.py file.

The one thing I will insist upon is that the program run without being
installed.  Right now, that doesn't happen, since the script can't
find the slidedex package.  This could be fixed by using the LIBPATH
method that I stole from Reinteract, but this would have to be
adjusted during the install.

Ahh, very true. I might propose the approach that I just pushed to my
setuptoolsify branch. In this case, I moved the script to the root of
the tree (along with renaming with caps; generally I'm not a fan of caps
but this was necessary to avoid a name clash with the slidedex
package). Since the directory containing the script is in the module
search path, there is no problem finding the package.
Cheers,

Ben

I also added a dependency on VTE and fixed the quoting issue you pointed out. Good catch, I have a habit of using quoting styles interchangably.

I think I'd like to steal the system used by Quod Libet: a script in the top directory named slidedex.py which gets installed to prefix/bin/slidedex.  Unfortunately, there doesn't seem to be a standard way to do this.  There's a simple solution on Stack Overflow that'd probably work for us.  Quod Libet uses a more involved solution; here're the relevant bits:
from distutils.command.build_scripts import build_scripts as distutils_build_scripts
⋮
class build_scripts(distutils_build_scripts):
    description = "copy scripts to build directory"

    def run(self):
        self.mkpath(self.build_dir)
        for script in self.scripts:
            newpath = os.path.join(self.build_dir, os.path.basename(script))
            if newpath.lower().endswith(".py"):
                newpath = newpath[:-3]
            if newer(script, newpath) or self.force:
                self.copy_file(script, newpath)
⋮
if __name__ == "__main__":
    os.chdir(os.path.dirname(os.path.abspath(__file__)))

    from quodlibet import const
    cmd_classes = {"check": check, 'clean': clean, "test": test_cmd,
                   "coverage": coverage_cmd, "build_scripts": build_scripts,
                   "build_gobject_ext": build_gobject_ext}
    setup_kwargs = {
        'distclass': GDistribution,
        'cmdclass': cmd_classes,
        ⋮

A related question is, should we be using setuptools or straight distutils?  I've been trying to read up on the difference, and have only managed to confuse myself.  From what I can gather, setuptools likes to keep all data files within the module directory.  If we get templates working, I'd like to be able to put them somewhere easier to get to, so users can copy and modify them more easily.  But perhaps we should not worry about that until we have to.


add in temporary logging around client reconnect to track down where things are dying

We probably should just move this into main so it is present in future releases until we have figured out why sometimes reconnect seems to just hang. I don't think anyone has been reliably on the pre-release created from this branch

Codecov Report

Merging #75 (378c7e4) into main (505c24e) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main      #75   +/-   ##
=========================================
  Coverage     11.77%   11.77%
  Complexity       35       35
=========================================
  Files            65       65
  Lines          3363     3363
  Branches        358      358
=========================================
  Hits            396      396
  Misses         2911     2911
  Partials         56       56

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 505c24e...378c7e4. Read the comment docs.


add EoD Strike Missions, SM CMs to /kp

End of Dragons Strike Missions and their respective CMs should ideally be added to /kp, if that information is available in the API.

Absolutely! I'd appreciate if someone could collect the necessary IDs for it, otherwise, I'll do it when I've got time.

I don't think that was the correct list of achievements.
It's missing Legendary Aetherblade Hideout (Mai Trin CM) and The Path Of Most Resistance (Overlook Title Achieve). The wiki doesn't seem to have those IDs and I don't know how to get them.
Also I don't think that the non-group achievements are relevant for kp, they just make it extra noisy or show-off-y.
List of achieves I think should be removed:

"Leaps Abound"
"Total Coverage"
"Clarity"
"Out of Focus"
"A Test of Your Reflexes"

Thanks for the feedback! Your points are valid. I'll get onto it

{{Achievement table row
| id = 6433
| name = Legendary Aetherblade Hideout
| page = Strike Mission: Aetherblade Hideout
| description = Complete the Aetherblade Hideout Strike Mission with challenge mode active.
| tiers = Completed the Aetherblade Hideout Strike Mission; 5
}}

Ok that's Aetherblade CM


[stack] render a consistent document title when section is collapsed

This appears to be a duplicate of #6166 — is it?

yes dupe of #6166


Docs:

Page URL: https://github.com/microsoft/TypeScript-Website/blob/v2/packages/documentation/copy/en/reference/Advanced Types.md#distributive-conditional-types
Issue:
The text says 'Similar to union and intersection types, conditional types are not permitted to reference themselves recursively'
This confused me. Union and intersection types are permitted to reference themselves recursively aren't they?
For example on the same page in the Type Aliases section we define
type LinkedList<Type> = Type & { next: LinkedList<Type> };
Which is an intersection type which references itself recursively.  We can change that & to a | to create a (not very useful) union type that references itself recursively.
So is the original statement wrong, or is there a sense I'm missing in which it's correct?
Recommended Fix:
If it's wrong we can just take out 'Similar to union and intersection types' from the sentence I think.

Yeah, this line was created before that restriction was liften in 3.7 - it can be removed 👍🏻


tcp_proxy: increase stat on upstream request retry

Additional Description: increasing the upstream_rq_retry on retry in the tcp_proxy
Risk Level: low
Testing: integration tests
Docs Changes: none
Release Notes: none
Platform Specific Features: none

I agree, I'll pivot to adding upstream_cx_retry instead to keep it consistent

but is n't this tracked by upstream_cx_connect_fail though/

@ramaraochavali does this stat increase for each upstream connection failure (i.e retries count) or is it per downstream connection?

Once for each upstream failure


pasting does not work in weston-terminal (crashes weston-terminal)

Steps to reproduce:

Select a word
Open the context menu, chose Copy
Open the context menu again, chose Paste
Observe how the weston-terminal window closed instead of the word being pasted.

The feature works in weston, but is broken in sway.

Likely an issue with wlc. Clipboard is "partially working" in wlc. Paging @Cloudef.

Partially working mainly means that xwayland <-> wayland isn't handled. This sounds more like bug in wlc's data-device implementation.


[FLINK-8865][sql-client] Add CLI query code completion in SQL Client

What is the purpose of the change
This PR adds cli query code completion in SQL Client based on Calcite SqlAdvisor
Brief change log
This change added tests and can be verified as follows:

Added sql completer test

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

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

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (not documented, add afterward)

Thanks for review and suggestions, @twalthr , I have refactored the codes. Please have an another review when you have free time, thanks :)

Thank you @xueyumusic. I found a couple of issues regarding escaping. I will fix them and merge this.


Show Instagram livestreams video address

Hi dear developer of this program,
It is not really not a problem what I have with "youtube-dl", on the contrary, I like this program and I am glad that it exists.
Now to my question. I can see the video address from many websites such as Youtube. The video address enables me to download live streams.
Is there any way to display the video address on Instagram or even download the live stream directly?
Many Greetings
UnixCro

What is to be criticized now?  Do not you want to answer me?


Continued pretraining from pytorch model

Hi,
I'm curious, do you happen to know if there's a way to use a pre-trained model from Huggingface models and use that to initialize KEPLER training? I was hoping to initialize KEPLER with a RoBERTa pretrained on medical data and use KEPLER pretraining with medical knowledge graphs and medical MLM.
Many thanks,
Michal

Hi,
We did not try to convert Huggingface's checkpoints into fairseq checkpoints (used in KEPLER pre-training) since all our KEPLER pre-training are based on native fairseq framework.
For your question, my suggestion is to read this code, which is used to convert fairseq RoBERTa checkpoints into Huggingface's and you may inversely convert it in similar way.
Best regards,
Xiaozhi

Hi Xiozhi,
Yea, I slightly feared that would be necessary. Thanks for the confirmation!
Kind regards,
Michal


How to install on  SUSE LES and SQLite3 ? Documentation not clear

Gitea version (or commit ref):  gitea-1.9.5-1.2.x86_64.rpm from Link in Docu

Git version:  2.26.2

Operating system:
Distributor ID: SUSE
Description:    SUSE Linux Enterprise Server 15 SP1
Release:        15.1

Database (use [x]):

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

Description
Disclaimer: I'm a complete rookie when it comes to work with Linux, especially with only having access via putty. So please bare with me.
What have I done so far:

Went to Docu "Installation from Package"
Downloaded the rpm from the linked page for SUSE Link
run rpm -Uvh gitea-1.9.5-1.2.x86_64.rpm
created  /etc/systemd/system/gitea.service with contents I found at Link
ran systemctl start gitea
and systemctl enable gitea
systemctl status gitea gives me:

gitea.service - Gitea
   Loaded: loaded (/etc/systemd/system/gitea.service; enabled; vendor preset: disabled)
   Active: activating (auto-restart) (Result: exit-code) since Thu 2021-02-11 10:19:50 CET; 577ms ago
  Process: 1944 ExecStart=/usr/local/bin/gitea web -c /etc/gitea/app.ini (code=exited, status=203/EXEC)
 Main PID: 1944 (code=exited, status=203/EXEC)

Feb 11 10:19:50 grwbghvgit01 systemd[1]: gitea.service: Unit entered failed state.
Feb 11 10:19:50 grwbghvgit01 systemd[1]: gitea.service: Failed with result 'exit-code'.

As you can see ... I probably forgot everything. I also needed external instructions to somehow continue.
But that is my point: I think the official documentation is lacking guidance.
My issues with the installation documentation:

In what sequence should I do things? There is no "1. ... 2 ... 3..."

I guess I should start with "Database Preparation" despite it is the 3rd entry and not linked or referenced in other places..
the database preparation has no guidance on SQLite even though it's supported. So I just did nothing ...
So I continued with "installation from package" and scrolled to SUSE. The only information there is the Link to the download...
No information on what to do next ... just: Here is a link. So I just ran the rpm
Now I assume that I should go to the "Linux Service" part... Which left me with the above output

I hope you can see my frustration.
None the less... I would be happy if someone could give me guidance on how to do it the right way :)

SUSE is packaging a version of Gitea that is significantly out of date. I recommend using the https://docs.gitea.io/en-us/install-from-binary/ install from  binary instructions instead.
For more troubleshooting please join us in our chatroom https://discord.gg/Gitea


Document Chooser with Populate Title

Hi, I found that Document Chooser cannot populate the updated file title but Image Chooser does.
I think its good to add this feature and its not hard to implement it because the function already exists in image-chooser-modal.js

The relevant bits in the image chooser code are:
https://github.com/wagtail/wagtail/blob/main/wagtail/images/static_src/wagtailimages/js/image-chooser-modal.js#L116-L130

Is this already done?

@akshay-kapase no. Feel free to pick this up.

Hi, Can I work on this issue?

FYI, #7534 enhances the image title population with an override function, it might be good to copy this approach for the document files.
See also #4945 for context


Added element type enum file

This PR adds the C++ enum definition for element types that are used in sfelib.

Thanks! Why is it put under extern instead of src?

Thanks! Why is it put under extern instead of src?

I tried to follow the wlslib-cpp structure.

But this file is not extern, so I should move to src

But this file is not "extern", so I should move to src

@xmjiao Moved to src


Commands run with the && operator do not participate in pipes

This command:
headers 22 && headers 19 | grep Bundle-SymbolicName
Should cause the output of both calls to headers to go through the grep command, but only the last one does.

Fixed.


Is this project still in active development?

From my understanding, there are improvements being made around the place.
People are using Comfy.
Oleg might be too busy at times to get around to them.
@GBH, you might want to answer @calvinl when you see this message.

Yeah, pretty much. I don't really have time to work on this right now. I'm here to deal with any game-breaking bugs but it's been pretty stable for a long time now. I'll release a version to rubygems that contains all the small bug fixes we had over the last year, but there are no features to speak of.
Rails 5 will force a significant work to get this gem working again. I'd say it would require need a rewrite from the ground up just to implement things like nicer translation support and a proper publishing flow.

Great, thanks for the quick responses @Daniel-ltw and @GBH. I'm just sizing up some CMS platforms for the features I need in my next project. Would be nice to see some new features, but I understand time is a big issue. Thanks for all your hard work.

I use this topic as an opportunity to tell you how great I find this CMS! There are so many developer-friendly but also user-friendly approaches in this CMS that it is really a pleasure to work with it. I am super happy with the status quo, but I hope it doesn't die!

I think that would be better to focus on Rails 5 update rather than new features (it's pretty complete in that sense!)

Can you add Comfy on Gitter Oleg?

So I spend good month full time working on 2.0. So it's active. Will be ready the moment Rails 5.2 get into beta by the end of the year or so.
I have project created here: https://github.com/comfy/comfortable-mexican-sofa/projects/1
All fixes are more than welcome.
@Ch1mpooya I added gitter link to the README.


check authority abnormal in streamload http request

Steps to reproduce the behavior (Required)
curl -i --location --request PUT 'http://<IP_ADDRESS>:7030/api/database_test/tb_test/_stream_load' \
--header 'jsonpaths: ["$.id", "$.name", "$.dt"]' \
--header 'format: json' \
--header 'columns: id, name, dt, __op ='\''upsert'\''' \
--header 'strict_mode: true' \
--header 'Expect: 100-continue' \
--header 'Content-Type: text/plain' \
--header 'Authorization: Basic xxx' \
--data '{
    "id": "101",
    "name": "xhl",
    "dt": "2024-06-25"
}
{
    "id": "102",
    "name": "ly",
    "dt": "2024-06-25"
}'

Expected behavior (Required)
{"status":"FAILED","msg":"Need auth information."}

Real behavior (Required)
curl: (52) Empty reply from server

StarRocks version (Required)

You can get the StarRocks version by executing SQL select current_version()

@tracymacding i had fixed this bug in the #47639


Issue regarding touchpad

VSCode Version: Code 1.16.1 (27492b6bf3acb0775d82d2f87b25a93490673c6d, 2017-09-14T16:38:23.027Z)
OS Version: Windows_NT x64 10.0.15063
Extensions:

Extension
Author (truncated)
Version

python
don
0.7.0

cpptools
ms-
0.13.0

Steps to Reproduce:
1.minimize vscode window.
2.click taskbar to reload vscode.
3.use touchpad to scroll edit area, it will get stuck.
ps: If click restore, scrolling get smooth again.

Reproduces without extensions: Yes

Could be a duplicate of https://github.com/Microsoft/vscode/issues/35191

https://github.com/Microsoft/vscode/issues/35191


Rust Backend Rewrite

Rewrote the backend in Rust to allow for more stability in the API interface and allow expansion of the backend to communicate with games for saving / loading, as well as gatekeeper integration down the line.

Is there documentation (compilation/installation guide, environment variable sample file, etc)?

Got it running for a little bit. I had to use the .env.template inside the onboard/ directory and fill it out. It needed some log file paths, resolution, and a directory, presumably to store game data in. Now it's crashing with cryptic messages
X Error of failed request:  BadValue (integer parameter out of range for operation)
  Major opcode of failed request:  130 (MIT-SHM)
  Minor opcode of failed request:  3 (X_ShmPutImage)
  Value in failed request:  0x340
  Serial number of failed request:  389
  Current serial number in output stream:  390


format error

my yaml file:
docker:
    - image: ubuntu:14.04
    - image: mongo:2.6.8
      command: [mongod, --smallfiles]
    - image: postgres:9.4.1

i run this command:
consulator import -glue=, /usr/local/ns-configs/dev/

but the result is Error: Invalid type map[string]interface {} in array. Only strings, numbers and boolean values are supported.  did it support this format?

redis:
  - host: 1
  - k: 2

did it support this format?

i found that this format is fine:
redis:
  host: 1
  password: 2
  port: 3

No, as Consul is a simple key/value store - how would you expect your earlier examples to be represented in Consul?

You can put any config you like in there, so long as it can be represented as keys with values - and internally Consul only supports string values.
This works:
foo:
  abc: 1
  def: x

This works:
foo:
  bar:
    abc: 1
    def: y
  xyz:
    abc: 1
    def: z

This also works:
foo:
  bar: 1
  servers:
    - server1
    - server2
    - server3

In this special case the "servers:" array will be transformed into a string within consul, so you'd need to split it when you read the value back out.

got it,  i will try it again, thank you ^ ^


Injecting lit-html into Components or DOM Question

Discovering lit-html was amazing and learning how it is utilized in a larger component model like lit-element is also amazing. There are some nice wrapper and factory patterns in some of the demos I've seen, yet it seems that there is a requirement to pick only one option: lit-html or lit-element as a way to manage template literals.
Using Reactive and FSM driven logic I'm inclined to reach for just lit-html, yet a few static lit-elements could be leveraged to prop up the app. Using lit-html I can create or fetch a template and simply render it dynamically in a functional way.
Is lit-element prepared for this kind of external injection? Can the lit-html template be targeted at a specific lit-element component and become part of it? Or at least can the components style rules be applied to what is injected?
Injecting into a lit-element is one option but I'd like things to remain completely open ended, allowing straight DOM injection as well. I've just not seen much about these options mentioned so far. I'd be interested in learning everything I can about leveraging the strengths of both lit-html and lit-element together, using each without them cramping each others strengths if possible.

I am not quite sure I understand exactly what you mean.
If you want to create components based only on lit-html, you can still use components based on LitElement (or any other web component) in your template.
Likewise html returns a TemplateResult, so you can use it straight in your template too:
render() {
  const template1 = html`<p>Hello</p>`;
  return html`${template1} <p> world</p>`;
}

You could also straight up use HTML with unsafeHTML directive: https://lit-html.polymer-project.org/guide/template-reference#unsafehtml
In case of injecting stuff into other components for me most of the time for me property bindings are enough. If it is not I am working on an event based dependency injection solution for web components that I will release to npm soon: https://github.com/Christian24/webcomponents-di

What I am asking is straightforward. I just want to use the lit-html and the lit-element approaches to rendering together but independently to build the DOM. Yes, there is the property binding facility but I'm driving and evolving the DOM dynamically so I would control the injection myself, essentially working outside the typical web component approach.
Since lit-element has more support for component model structure and style I'm consequently asking how injected lit-html fragments will work. Will these fragments be isolated or be integrated under the style of a component they are injected into? Is there good or poor integration when using lit-html and lit-element side-by-side as alternate ways to render into the DOM?
Using lit-html inside the lit-element component model should not be the only option. It should also be allowed to be used in a side-by-side way together but independently. There should be more documentation and samples about this and not just the assumption that the web component model and it's property binding system is the only game in town.

UPDATE It was important for me to be able to target different DOM nodes and bind full Web Components to separate roots. I believe I have now found the solution in Svelte, which offers a flexible range of composition options. I like it's syntax and compiler approach and it's focus on productivity and how it does not obscure getting things done with technology like Web Components that is still emerging.


Next release date?

Hi @nxpfrankli ,
We are soon to do a release that requires one of the latest changes in UUU. I have already a working, manually built, UUU binary but I was wondering if you have an approximate date for the next release in this repo?
Thanks in advance.

Which one do you need?  1.5.4?

The one we need is at least 1.5.6.

I create a pre release https://github.com/NXPmicro/mfgtools/releases/tag/uuu_1.5.11
I will remove pre-release take after some month.

Many thanks @nxpfrankli .
When you remove the pre-release, will there be an official release to replace it?

If not problem, I just remove pre-release tag and update release notes.
If there are problem, new pre-release will be created.

Great, that will work fine for us. Thank you!


python312Packages.google-api-core: 2.17.1 -> 2.18.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.google-api-core is: Core Library for Google Client Libraries
meta.homepage for python312Packages.google-api-core is: https://github.com/googleapis/python-api-core
meta.changelog for python312Packages.google-api-core is: https://github.com/googleapis/python-api-core/blob/v2.18.0/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.18.0 with grep in /nix/store/fvfhfm0n8dqcx536a6lql0025wciramz-python3.12-google-api-core-2.18.0
found 2.18.0 in filename of file in /nix/store/fvfhfm0n8dqcx536a6lql0025wciramz-python3.12-google-api-core-2.18.0

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

263 total rebuild path(s)

263 package rebuild(s)

First fifty rebuilds by attrpath
authentik
barman
baserow
beancount
browsr
certbot-full
deja-dup
duplicity
duply
dvc-with-remotes
fava
gam
gcalcli
gcp-scanner
git-annex-remote-googledrive
goobook
gyb
home-assistant-component-tests.google_generative_ai_conversation
home-assistant-component-tests.google_mail
home-assistant-component-tests.google_pubsub
home-assistant-component-tests.google_tasks
home-assistant-component-tests.nest
lbdb
ledger2beancount
lieer
mycli
parsedmarc
pgadmin4
pgadmin4-desktopmode
prowler
python311Packages.aiogithubapi
python311Packages.beancount
python311Packages.beancount-docverif
python311Packages.bpemb
python311Packages.certbot-dns-google
python311Packages.chirpstack-api
python311Packages.cirq
python311Packages.cirq-google
python311Packages.cloudpathlib
python311Packages.dalle-mini
python311Packages.dbt-bigquery
python311Packages.django-storages
python311Packages.drivelib
python311Packages.duckdb
python311Packages.duckdb-engine
python311Packages.dvc-gdrive
python311Packages.dvc-gs
python311Packages.fastai
python311Packages.filedepot
python311Packages.fuzzytm

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fvfhfm0n8dqcx536a6lql0025wciramz-python3.12-google-api-core-2.18.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/87cfe588fbf01b3c03143523c9b90e06bacc6363#python312Packages.google-api-core

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fvfhfm0n8dqcx536a6lql0025wciramz-python3.12-google-api-core-2.18.0
ls -la /nix/store/fvfhfm0n8dqcx536a6lql0025wciramz-python3.12-google-api-core-2.18.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 120m and timed out

Maintainer pings

#300255


Switch CI to using out custom JXcore build

We only use SM on the devices so it's bad that we are using V8 in the VM.

Discussed offline that we are fine using V8 on desktop, but we need our custom build so that we can do PSK.

Here is what I did:
// Download and unzip http://jxcore.azureedge.net/jxcore/0312/release/jx_osx64v8.zip
$ cp <download-folder>/jx_osx64v8/jx $(which jx)
$ rm -rf ~/.jx/

Then I noticed that @cicorias has done a SpiderMonkey build even though the .zip package filename implies V8. This can be checked with:
$ jx -jsv
Mozilla SpiderMonkey v34

Because our desktop build doesn't currently work with SpiderMonkey, I couldn't complete this task today.
I'll continue after either of these two conditions is met:

A OS X build with V8 is release by @cicorias
I am able to make the desktop build work with SpiderMonkey

updated the CDN and initiated a purge. I verified jx -jsv is V8 after:
curl -o jx.zip http://jxcore.azureedge.net/jxcore/0312/release/jx_osx64v8.zip
unzip jx.zip
cicoriasmbp13@Shawns-MBP:~/Downloads/jx_osx64v8$ ./jx -jsv
Google V8 v<IP_ADDRESS>

This is now done and there is a success build after the update at https://github.com/thaliproject/Thali_CordovaPlugin/pull/673#issuecomment-200705817.


bug of data augmentation

In your code of augmentation:
https://github.com/happywu/Sequence-Level-Semantics-Aggregation/blob/c1efb0a7ba37394c14f32c5725bc5a0139c8651a/lib/utils/augmentations.py#L270

if overlap.min() < min_iou and max_iou < overlap.max():

In this line, the max_iou in INF. So max_iou will never be < overlap.max(). As a result, whatever the overlap is, the crop will be kept.
Is this a bug? Our you intend to keep all random crops?

Hi, yes, it should be a bug from the original code snippet. https://github.com/amdegroot/ssd.pytorch/issues/119.

Hi, yes, it should be a bug from the original code snippet. https://github.com/amdegroot/ssd.pytorch/issues/119.


Speed up linking for incremental builds.

Problem description
Incremental builds take a very long time due to the time taken to link _ttnn.so.
What's changed
Split ttnn into multiple sub libraries, were each sub library is built and linked independently. These sub libraries are then linked together to form _ttnn.so
Reduced build time from 150s to 21s.
Checklist

[ ] Post commit CI passes
[ ] Blackhole Post commit (if applicable)
[ ] Model regression CI testing passes (if applicable)
[ ] Device performance regression CI testing passes (if applicable)
[ ] (For models and ops writers) Full new models tests passes
[ ] New/Existing tests provide coverage for changes

@blozano-tt @afuller-TT fyi

This is clearly moving things in the right direction.
but it looks a bit odd.
why not take advantage of the folder hierarchy and add more CMakeLists.txt in the “sub libraries”.
this would make things more declarative, and there would be no need for functions or for loops.
I’m almost certain @afuller-TT will agree as we are becoming hive mind

As @blozano-tt said, this is not idiomatic CMake, but it is moving things in the right direction.
Declarative is preferred over Imperative.
Scoped CMakeLists.txt are preferred over a global CMakeLists.txt.  Think Lego pieces.
If you'd like a hand/pointers for improving it, LMK and I'll help you out.
Otherwise I'm fine with this going in as it is a step better.  And I'll be in that area a-couple-projects-from-now in service of packaging ttnn for various deliverable formats.  At that time I'll revamp the CMake files there.


Delivery Tasks - Recommending earlier available slots

As a manager,
I want to be suggested an earlier available slot for a driver,
So that I can optimize the drivers' schedules
Description:
When proposing the starting and ending time for a task, the Schedule class will suggest an earlier time slot that has the same length as the proposed duration, if available.

Merged


[fix] show more descriptive error if data returned from load is a non-POJO

Closes #7262

Tested it out with return json

with new Error

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

Thanks @dummdidumm, i've updated it.

Thanks @dummdidumm, updated.

Could you add a changeset by running npx changeset in the root and following the prompts (patch release)? Then we're good to go 👍

Added changeset


Database restore doesnt work with root user

Bug report
System info: Influx 1.0.1, CentOS 7 x64
Steps to reproduce:

Completely backup influx as root
Restore backup as root
(I did it on a completely fresh server. Not sure if that affects the result)

Expected behavior: A completely operational DB
Actual behavior: Influx used the root user to create the local files and lost access to those files. Error write failed for shard 143: engine: cache maximum memory size exceeded
Additional info: https://gist.githubusercontent.com/jimaek/5f7fe91cc5f689273bed64401e564db1/raw/5aea6861548a2f3d86940000f25576402bca29f5/gistfile1.txt
 chown -R influxdb:influxdb /var/lib/influxdb/ fixes the problem

The error you get is because the files are owned by root and influxdb cannot write to them so the cache fills up.  If you run a backup, you need to run it as the influxdb user or chown the files on restore.


Problem with the tcam_demo.cpp example

OS: Ubuntu 16.04
TIS version: 0.9.1
TIS version source: https://github.com/TheImagingSource/tiscamera/releases/download/v-release-candidate-0_9_1-1/tiscamera_0.9.1_amd64.deb
Camera type: DMK Z12GX236
test code: https://github.com/TheImagingSource/tiscamera/blob/master/examples/cpp/examples/tcam_demo.cpp

I built and ran the code. I could see that the camera is recognized and that the video formats and properties are printed (although without the automatics properties).
The problem is that I the  GTK window doesn't show (I see its existence on the sidebar but not on the screen) and the new_frame_cb callback isn't called at all.
Can someone please help me see what I'm missing here?
Thanks a lot!
Joel

The problem is that the example does not set a video format on the camera. Add a line similar to this somewhere before cam.start() and it should work:
    cam.set_capture_format("GRAY8", {640,480}, {30,1});

Thanks, that solved it


Drop support for windows 8 and 8.1

Refs https://www.electronjs.org/blog/windows-7-to-8-1-deprecation-notice
March milestone:

[ ] :runner: Push EOL message via top status bar and notification
[ ] @isidorn to help with updating requirements and FAQ section in the website

May milestone:

[ ] Flip switch in update server to disable updates

@deepak1556 we do this in April?

We can push the client side notification for March release. Or you mean do we make April release v1.78 as the last release ?

I meant the notification. So let's assign this to March so we do not lose track :)

Steps for the March 2023 milestone are complete.


Implement the select mode in the NamespaceSelector API

Previously, NamespaceSelector only supports selecting namespace-scoped
resources whose labels match those labels of the Namespaces that are
statically declared in the source of truth.
This commit updates the NamespaceSelector to support selecting
dynmaic/on-cluster Namespaces as well.
When it applies the NamespaceSelector, it sends out a LIST request to
get all Namespaces and select resources match the on-cluster Namespaces.

One outstanding comment, but lgtm.
/lgtm
/assign @janetkuo

/lgtm

/lgtm


I can't get it to work.

I tried to do it, but I get this.
Traceback (most recent call last):
File "D:\DDSP-SVC\train.py", line 94, in
train(args, initial_global_step, model, optimizer, loss_func, loader_train, loader_valid)
File "D:\DDSP-SVC\solver.py", line 83, in train
for batch_idx, data in enumerate(loader_train):
File "D:\DDSP-SVC\venv\lib\site-packages\torch\utils\data\dataloader.py", line 633, in next
data = self._next_data()
File "D:\DDSP-SVC\venv\lib\site-packages\torch\utils\data\dataloader.py", line 1348, in _next_data
return self._process_data(data)
File "D:\DDSP-SVC\venv\lib\site-packages\torch\utils\data\dataloader.py", line 1374, in _process_data
data.reraise()
File "D:\DDSP-SVC\venv\lib\site-packages\torch_utils.py", line 665, in reraise
raise exception
RecursionError: Caught RecursionError in DataLoader worker process 0.
Original Traceback (most recent call last):
File "D:\DDSP-SVC\venv\lib\site-packages\torch\utils\data_utils\worker.py", line 308, in _worker_loop
data = fetcher.fetch(index)
File "D:\DDSP-SVC\venv\lib\site-packages\torch\utils\data_utils\fetch.py", line 51, in fetch
data = [self.dataset[idx] for idx in possibly_batched_index]
File "D:\DDSP-SVC\venv\lib\site-packages\torch\utils\data_utils\fetch.py", line 51, in
data = [self.dataset[idx] for idx in possibly_batched_index]
File "D:\DDSP-SVC\data_loaders.py", line 187, in getitem
return self.getitem( (file_idx + 1) % len(self.paths))
File "D:\DDSP-SVC\data_loaders.py", line 187, in getitem
return self.getitem( (file_idx + 1) % len(self.paths))
File "D:\DDSP-SVC\data_loaders.py", line 187, in getitem
return self.getitem( (file_idx + 1) % len(self.paths))
[Previous line repeated 1988 more times]
File "D:\DDSP-SVC\data_loaders.py", line 186, in getitem
if data_buffer['duration'] < (self.waveform_sec + 0.1):
RecursionError: maximum recursion depth exceeded in comparison

Check whether the training dataset and the validation dataset are placed correctly and the preprocessing is performed correctly.

https://zenn.dev/aivoicelab/articles/ddsp-svc-tutorial?redirected=1
I used this site for reference.
I did it exactly like this.
Are there any areas where I am wrong?


sig: configure routes on tunnel interface

The SIG could automatically add route entries for all the configured remote networks via its tunnel interface.
This would allow to run the SIG without surrounding scripts to set up these routes; the expected behaviour is roughly equivalent to:
sig --config <.../sig.toml> &
for ipnet in $(jq  -r .ASes[].Nets[] <.../sig-cfg.json>); do
   ip route add ipnet dev sig
done

In this case the routing tables would be managed directly by the SIG itself, and it seems that the tun_routing_table_id would not need to be setup. Perhaps this new behaviour could be an optional with a configuration option to switch between the  "automatic" mode and the old "fully manual" mode.
On a similar note, it would be really nice if the SIG would also directly set a sensible MTU (equivalent to ip link set mtu ... dev sig) and a local address (ip address add .... dev sig), but unlike the routes discussed above, this information does not currently seem to be available in the SIG's configuration.

Done, https://github.com/scionproto/scion/pull/3974/commits/57a514459d7ad2a157e94051a9238a24e2aaccb2.


code":49, 52e

The username/password for the admin LDAP config is correct. IDEAS, SUGGESTIONS?
var ad = new ActiveDirectory({ url: 'LDAP://LDAPAUTH.LDAP.< MYDOMAIN >.org',
        baseDN: 'OU=SystemUsers,DC=< OUR >,DC=< DOMAIN >DC=local',
        username: '< VALIDUSER >',
        password: ' <VALID PASSWORD>' });

    var username = '< TESTUSERNAME >' ;
    var password = '< TESTPASSWORD >' ;

    ad.authenticate(username, password, function(err, auth) {
        if (err) {
            console.log('ERROR: '+JSON.stringify(err));
            return;
        }

        if (auth) {
            console.log('Authenticated!');
        }
        else {
            console.log('Authentication failed!');
        }
    });

ERROR: {"dn":"","code":49,"name":"InvalidCredentialsError","message":"80090308: LdapErr: DSID-0C0903A8, comment: AcceptSecurityContext error, data 52e, v1db1\u0000"}

I have special characters in my password how do I escape it?

This is likely an upstream issue. But I'll try to help you.
Can you give an example of what you mean by a special character?
i.e. ASCII 0x13

I apologize for piggy backing off this old thread but this is the only thread I see referencing the specific LDAP error that I am also seeing when trying to use this module.  I am specifically trying to understand if the issue is with the password for the bindDN (AD admin user account) and itself having special characters or if the issue is actually with special characters that the user's password has (via the login form).  I am using passport and combining this module within the authentication call.

To see if the problem is with the bindDN (used for searching, etc.), just configure and issue a findUser() or other API method call. If it works, then the problem is NOT with the AD admin user account.


fix(IAM): renamed UserOptions iam_secret to iam_client_secret to be consistent with other cores

This PR renames iam_secret to iam_client_secret and iamSecret to iamClientSecret so that the node sdk core will be similar to other cores wrt IAM-related terminology.

this is a breaking change

@germanattanasio and I discussed this PR and determined that the node-sdk currently is locked in on version 0.1.1 of the node core.   This means that we can "safely" (famous last words) make this change without breaking SDK users.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Security] Subresource Integrity

Would it be possible to add a security feature, so that each link to an asset would have a so called "integrity string". The integrity string is more thoroughly described here:
https://developer.mozilla.org/en-US/docs/Web/Security/Subresource_Integrity
Would really appreciate, thanks.

@piotrgolasz Hi Piotr! Thank you for your interest in this project! I'm sorry this is not visible well enough, but we're not supporting this project anymore as you can see from the last commit message: https://github.com/OpenBuildings/asset-merger/commit/668e4b7f85996cfee1f34361190e972c31257ac3
However if you use the project and would like to contribute you can create some pull requests and I would make you a collaborator so you're able to maintain the project.

Hi @hkdobrev,
thanks for such quick response.
I'll make two pull requests related to this task and #30.

Hey @hkdobrev, do you accept?

@piotrgolasz Hi, thank you for your work on these pull requests! I've written my comments on the respective pull requests. :-)

@hkdobrev I've added multiple hashes - in the config file you can set array as parameter, and that enables it. I added a comment in the config file, however I think we should also change README to reflect recent changes in this and previous pull request.
Haralan, please check it when you can ;)

@hkdobrev ?

Ok, I merged those myself.


[OpenCL] Add check to only cast opaque handles to cl::BufferDescriptor at runtime

Previous casting of void_args was too aggressive and attempted to cast non backend allocated handles, such as kernel scalar arguments, to cl::BufferDescriptors.

cc @masahi


Rocketchat importer

Add import functionality for RocketChat.
Note: The work is based on the Gitter importer. The code if far from being complete, ideal
structured or resource optimized, but it works, passes the linters and has docs.
Since I spend hours puzzling this together, I still think this is somewhat helpful to others.
Testing Plan:
Nope. Puzzling this all together took ways to much time. We need to drop this here and
ask the community to complete this sketch.
What works

users
avatars
public channel
Rocketchat threads to Zulip topics
private conversations
notes to one self
attachments

What does not work

huddles
emojies

I've tried this and I got thie error after the command:
./manage.py convert_rocketchat_data  --output ./tmp/export  --rocketchat_dump ./tmp/mongodump/parties/
Traceback (most recent call last):
File "./manage.py", line 50, in
execute_from_command_line(sys.argv)
File "/srv/zulip-py3-venv/lib/python3.6/site-packages/django/core/management/init.py", line 381, in execute_from_command_line
utility.execute()
File "/srv/zulip-py3-venv/lib/python3.6/site-packages/django/core/management/init.py", line 375, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/srv/zulip-py3-venv/lib/python3.6/site-packages/django/core/management/init.py", line 224, in fetch_command
klass = load_command_class(app_name, subcommand)
File "/srv/zulip-py3-venv/lib/python3.6/site-packages/django/core/management/init.py", line 36, in load_command_class
module = import_module('%s.management.commands.%s' % (app_name, name))
File "/srv/zulip-py3-venv/lib/python3.6/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 994, in _gcd_import
File "", line 971, in _find_and_load
File "", line 955, in _find_and_load_unlocked
File "", line 665, in _load_unlocked
File "", line 678, in exec_module
File "", line 219, in _call_with_frames_removed
File "/srv/zulip/zerver/management/commands/convert_rocketchat_data.py", line 8, in
from zerver.data_import.rocketchat import do_convert_data
File "/srv/zulip/zerver/data_import/rocketchat.py", line 8, in
import bson
ModuleNotFoundError: No module named 'bson'
How can I add the bson module ?
Thanks
Regards,

@dedisoft Did you ever get this working? Seeing the same bson error.

Closing in favor of the actively developed https://github.com/zulip/zulip/pull/18966.  Thanks @Benocs!


15.09 → 16.03 update breaks syncthing service with custom dataDir

When upgrading my system from 15.09 to 16.03-beta the syncthing service failed, it was working fine under 15.09 with the following configuration
  services.syncthing = {
    enable = true;
    dataDir = "/home/rycee";
    user = "rycee";
  };

Turns out that 44788bb2ce5ecb025b63c71226ab613a05a334d3 changed the way the dataDir attribute works and changing the configuration to
  services.syncthing = {
    enable = true;
    dataDir = "/home/rycee/.syncthing";
    user = "rycee";
  };

fixed the issue. I didn't find any mention of this backwards incompatible change in the corresponding release log. It would be nice with an entry for this in the release log.
CC: @garbas, @fazo96

@rycee: tnx ... looks like we missed to add this

fixed

Thanks for the quick fix!

Sorry for missing the notification!


[bug report] the select component has both filterable and multiple components. Click the shrink button and it doesn't take effect

Element UI version
2.15.2
OS/Browsers version
window10/chorme
Vue version
2.6.14
Reproduction Link
https://codepen.io/mashedpotato2018/pen/BaWOzgy
Steps to reproduce
展开下拉选项，点击收缩按钮无效
What is Expected?
收缩下拉选项
What is actually happening?
没有任何行为

Duplicate of #21117


Refactored perception pipeline tutorial from ros1 to ros 2

Description
I have a PR for refactoring perception pipeline tutorial based on ROS 2. But i have several question about my PR.

I used some external links for not storing big files such as video, bag file which is created by me. Is it okay to give a link to my google drive for bag file and a link to Youtube video for perception pipeline demonstration or should i change these links?

I used Intel's sdf file  in the tutorial which the relevant repository has Apache 2.0 License. Does that occur any problem for this organization? Btw, ros2/launch has also Apache 2.0 License. But it is not indicated in this repository. I need a feedback for this situation.

Checklist

[ ] Required by CI: Code is auto formatted using clang-format
[ ] While waiting for someone to review your request, please consider reviewing another open pull request to support the maintainers

Just for friendly pinging, @sjahr

I used Intel's sdf file in the tutorial which the relevant repository has Apache 2.0 License. Does that occur any problem for this organization?

Good question :thinking: did you use the sdf or the mesh file or both?

Good question 🤔 did you use the sdf or the mesh file or both?

I had added Intel's license content to mesh file like below.

The most of the sdf is mine however i took some camera measurement from them. If it is boring in your perspective, I may remove it.

@CihatAltiparmak Maybe this is helpful as a reference: https://github.com/moveit/moveit2_tutorials/pull/700/files

Hello @sjahr , i have added the depth image octomap updater to the perception pipeline tutorial. However i have found some bugs in depth image updater and i have a fix for it. Btw i've also sent PR to moveit_benchmark_resources.
My PR in moveit_benchmark_resources : https://github.com/moveit/moveit_benchmark_resources/pull/3
My PR to fix bug in depth image updater plugin: https://github.com/moveit/moveit2/pull/2963
After merging the above PR's, i believe we can finalize this work.

@sjahr I've applied your suggestion in moveit_benchmark_resources and here. I've added redirection to https://github.com/moveit/moveit_benchmark_resources repository. I hope this is the last dance here.

Oh and can you fix the broken link CI error? https://github.com/moveit/moveit2_tutorials/actions/runs/10389968570/job/28769302814?pr=906. Looks like you just need to redirect it to moveit_benchmarking_resources

Oh and can you fix the broken link CI error? https://github.com/moveit/moveit2_tutorials/actions/runs/10389968570/job/28769302814?pr=906. Looks like you just need to redirect it to moveit_benchmarking_resources

I know, it's searching for bag_files/depth_camera_bag directory. We need to merge my moveit_benchmark_resources pr first to fix CI

Oh and can you fix the broken link CI error? https://github.com/moveit/moveit2_tutorials/actions/runs/10389968570/job/28769302814?pr=906. Looks like you just need to redirect it to moveit_benchmarking_resources

I know, it's searching for bag_files/depth_camera_bag directory. We need to merge my moveit_benchmark_resources pr first to fix CI

@CihatAltiparmak I merged it ~1h ago. Can you run formatting and retrigger CI by pushing. Then this should be good to go

All CIes are passed. Let's roll.

Thank you for porting this!

@sjahr I'm glad a long-standing ticket https://github.com/moveit/moveit2_tutorials/issues/25 is finally closed by this PR.
Then, more than a year ago I made basically the same effort https://github.com/moveit/moveit2_tutorials/pull/700. You also commented you'd start reviewing https://github.com/moveit/moveit2_tutorials/pull/700#issuecomment-1645650394. I admit I hadn't kept asking for review recently though, I'm honestly a bit sad. It'd be great in the future if open PRs aren't discarded without any accolade like in this way.

@130s you're absolutely right and I am very sorry that this happened due to my inattentiveness. As you pointed out I even committed to reviewing your PR last year and then never did it :disappointed: This should not happen and you're right to be frustrated about my (in)action.
I think we should re-open your PR as this merged version is a simplified version of the ROS 1 tutorial that doesn't include the Detecting and Adding Object as Collision Object section that you ported as well and use your work to enhance and improve the existing version.

I want to write as well. Firstly really apologies for this action @130s, You are absolutely right. I already saw your PR before i didn't open this PR. As a newbie of MoveIt2, i strongly needed to learn perception pipeline application in Moveit and i had limited time. I took a look at MoveIt1's tutorial and it didn't be helpful for me as newbie. When i see your PR has no active works, i decided to continue to my work. I have added depth camera octomap updater and i found 2 bugs in MoveIt2.
But i just want to empathize the maintainer as well. When i open the review page of my PR, i saw a lot of complex works for maintainer side and a lot of PR's to review.
If you still want to help us, my PR can have some lack of information. @sjahr 's suggestions are also good.
Best for you

@sjahr @CihatAltiparmak You guys made my day. Thanks, thanks. I'll see what I can pick up from #700. Another good thing is now we have a new reviewer for perception pipeline :)


Add an option --features wayland or features x11

Feature request, please. :smiley:
First brought up in relation to #566 (closed), here: #566
Inspiration is the bevy game engine, their pull request's: #249
Motivation is Wayland on Linux seemingly not being able to position windows.

As Amr said in that other issue: This doesn't make sense for tao since we replaced the custom x11 and Wayland backends with GTK (because we need that for webkitgtk).
Even if we ignore that, this would not necessarily help you. Even if you choose (only) the x11 backend you have no guarantee that XWayland behaves like you want, but it gets even worse: you can't expect XWayland to exist on all Wayland systems in which case it would just crash.

Apparantly you can force x11 using GDK_BACKEND env var, see https://wiki.archlinux.org/title/GTK#Xorg_backend

We do that in the appimage because it crashes otherwise. Idk but the flag feels a little weird at times too, and it seems to be ignored sometimes for whatever reason (could have been caused by something else, i noticed that when I tested some other appimage stuff)
Aaand it doesn't help with the linked issue either iirc

Thank you both for taking the time to respond, I appreciate it.


Set up CI with Azure Pipelines

I created a public Devops pipeline to test and it automatically created this PR (which is NOT desirable).


[5.7] Do not try to interrupt/terminate a process if it isn't running

5.7 cherry-pick of #5674

@swift-ci please smoke test


Adding more form adaptions from rules

Creating new issue so this won't get lost.
textarea can have minlength and maxlength
Required is also available for select, textarea
Besides number and email, input can also be url (among other things, but those do not have a validator in Yii. Not sure if that would be viable to implement).

Need more research:
~pattern vs RegularExpression~ Already taken care of, I totally missed that :)
step - not a validation option in Yii
Originally posted by @Mister-42 in https://github.com/yiisoft/form/issues/65#issuecomment-767014816

The validator should not change the attributes of the forms.

@terabytesoftw I disagree. HTML validation attributes are alright to be generated based on validators configured.

@terabytesoftw I disagree. HTML validation attributes are alright to be generated based on validators configured. At least it is very convenient and allows to keep HTML-based validation and server validation in sync.

For example, if you use the required attribute and in turn use the Required::rule() validator, the form will only show the error of the HTML validator browser, it will never show the error of the Yii3 Validator, what is the point of doing a double validation here?

Benefit of having validation on the client side is to avoid extra form submits and get the validation result ASAP.
But that doesn't eliminate the need for server validation since one can easily modify HTML before submitting the form.

Benefit of having validation on the client side is to avoid extra form submits and get the validation result ASAP.
But that doesn't eliminate the need for server validation since one can easily modify HTML before submitting the form.

Ok I think I can implement them without needing the interfaces to wrap the validator, would you agree to remove them?

@terabytesoftw depends on the implementation.