Illegal array length, must be 15

When I run python manage.py drive --model ./models/mypilot,and then choose LocalAngle,this mistake was occured:
Traceback (most recent call last):
File "manage.py", line 190, in
drive(cfg, model_path = args['--model'], use_joystick=args['--js'], use_chaos=args['--chaos'])
File "manage.py", line 131, in drive
max_loop_count=cfg.MAX_LOOPS)
File "/home/pi/env/lib/python3.5/site-packages/donkeycar/vehicle.py", line 93, in start
self.update_parts()
File "/home/pi/env/lib/python3.5/site-packages/donkeycar/vehicle.py", line 129, in update_parts
outputs = p.run(*inputs)
File "/home/pi/env/lib/python3.5/site-packages/donkeycar/parts/keras.py", line 75, in run
angle_unbinned = util.data.linear_unbin(angle_binned[0])
File "/home/pi/env/lib/python3.5/site-packages/donkeycar/util/data.py", line 38, in linear_unbin
raise ValueError('Illegal array length, must be 15')
ValueError: Illegal array length, must be 15
Every time I can only run python manage.py drive and then choose LocalAngle,but it does not achieve the desired result.

check manage.py.change drive and train use same model(categorical or linear)-------- 原始邮件 --------主题：Re: [wroscoe/donkey] Illegal array length, must be 15 (#308)发件人：easonzrq 收件人：wroscoe/donkey 抄送：NikulausRui ,State change I have the same question.
How do you fix it ?can you tell me
3Q
—You are receiving this because you modified the open/close state.Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/wroscoe/donkey","title":"wroscoe/donkey","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/wroscoe/donkey"}},"updates":{"snippets":[{"icon":"PERSON","message":"@easonzrq in #308: I have the same question.\r\nHow do you fix it ?can you tell me \r\n3Q"}],"action":{"name":"View Issue","url":"https://github.com/wroscoe/donkey/issues/308#issuecomment-430916478"}}}
[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/wroscoe/donkey/issues/308#issuecomment-430916478",
"url": "https://github.com/wroscoe/donkey/issues/308#issuecomment-430916478",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [wroscoe/donkey] Illegal array length, must be 15 (#308)",
"sections": [
{
"text": "",
"activityTitle": "easonzrq",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@easonzrq",
"facts": [
]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n"commandName": "IssueComment",\n"repositoryFullName": "wroscoe/donkey",\n"issueId": 308,\n"IssueComment": "{{IssueComment.value}}"\n}"
}
]
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/wroscoe/donkey/issues/308#issuecomment-430916478"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n"commandName": "MuteNotification",\n"threadId": 375088529\n}"
}
],
"themeColor": "26292E"
}
]

sorry,i cant find where to change the mode.can you tell me ?

check manage.py.change drive and train use same model(categorical or linear)-------- 原始邮件 --------主题：Re: [wroscoe/donkey] Illegal array length, must be 15 (#308)发件人：easonzrq 收件人：wroscoe/donkey 抄送：NikulausRui ,State change I have the same question. How do you fix it ?can you tell me 3Q —You are receiving this because you modified the open/close state.Reply to this email directly, view it on GitHub, or mute the thread. {"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/wroscoe/donkey","title":"wroscoe/donkey","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"[https://github.com/wroscoe/donkey"}},"updates":{"snippets":{"icon":"PERSON","message":"@easonzrq in #308: I have the same question.\r\nHow do you fix it ?can you tell me \r\n3Q"}],"action":{"name":"View Issue","url":"#308 (comment)"}}} [ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "#308 (comment)", "url": "#308 (comment)", "name": "View Issue" }, "description": "View this Issue on GitHub", "publisher": { "@type": "Organization", "name": "GitHub", "url": "https://github.com" } }, { "@type": "MessageCard", "@context": "http://schema.org/extensions", "hideOriginalBody": "false", "originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB", "title": "Re: [wroscoe/donkey] Illegal array length, must be 15 (#308)", "sections": [ { "text": "", "activityTitle": "easonzrq", "activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png", "activitySubtitle": "@easonzrq", "facts": [ ] } ], "potentialAction": [ { "name": "Add a comment", "@type": "ActionCard", "inputs": [ { "isMultiLine": true, "@type": "TextInput", "id": "IssueComment", "isRequired": false } ], "actions": [ { "name": "Comment", "@type": "HttpPOST", "target": "https://api.github.com", "body": "{\n"commandName": "IssueComment",\n"repositoryFullName": "wroscoe/donkey",\n"issueId": 308,\n"IssueComment": "{{IssueComment.value}}"\n}" } ] }, { "targets": [ { "os": "default", "uri": "#308 (comment)" } ], "@type": "OpenUri", "name": "View on GitHub" }, { "name": "Unsubscribe", "@type": "HttpPOST", "target": "https://api.github.com", "body": "{\n"commandName": "MuteNotification",\n"threadId": 375088529\n}" } ], "themeColor": "26292E" } ]

sorry,i cant find where to change the mode.can you tell me ?

check manage.py.change drive and train use same model(categorical or linear)-------- 原始邮件 --------主题：Re: [wroscoe/donkey] Illegal array length, must be 15 (#308)发件人：easonzrq 收件人：wroscoe/donkey 抄送：NikulausRui ,State change I have the same question. How do you fix it ?can you tell me 3Q —You are receiving this because you modified the open/close state.Reply to this email directly, view it on GitHub, or mute the thread. {"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/wroscoe/donkey","title":"wroscoe/donkey","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"[https://github.com/wroscoe/donkey"}},"updates":{"snippets":{"icon":"PERSON","message":"@easonzrq in #308: I have the same question.\r\nHow do you fix it ?can you tell me \r\n3Q"}],"action":{"name":"View Issue","url":"#308 (comment)"}}} [ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "#308 (comment)", "url": "#308 (comment)", "name": "View Issue" }, "description": "View this Issue on GitHub", "publisher": { "@type": "Organization", "name": "GitHub", "url": "https://github.com" } }, { "@type": "MessageCard", "@context": "http://schema.org/extensions", "hideOriginalBody": "false", "originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB", "title": "Re: [wroscoe/donkey] Illegal array length, must be 15 (#308)", "sections": [ { "text": "", "activityTitle": "easonzrq", "activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png", "activitySubtitle": "@easonzrq", "facts": [ ] } ], "potentialAction": [ { "name": "Add a comment", "@type": "ActionCard", "inputs": [ { "isMultiLine": true, "@type": "TextInput", "id": "IssueComment", "isRequired": false } ], "actions": [ { "name": "Comment", "@type": "HttpPOST", "target": "https://api.github.com", "body": "{\n"commandName": "IssueComment",\n"repositoryFullName": "wroscoe/donkey",\n"issueId": 308,\n"IssueComment": "{{IssueComment.value}}"\n}" } ] }, { "targets": [ { "os": "default", "uri": "#308 (comment)" } ], "@type": "OpenUri", "name": "View on GitHub" }, { "name": "Unsubscribe", "@type": "HttpPOST", "target": "https://api.github.com", "body": "{\n"commandName": "MuteNotification",\n"threadId": 375088529\n}" } ], "themeColor": "26292E" } ]

sorry,i cant find where to change the mode.can you tell me ?

check manage.py.change drive and train use same model(categorical or linear)-------- 原始邮件 --------主题：Re: [wroscoe/donkey] Illegal array length, must be 15 (#308)发件人：easonzrq 收件人：wroscoe/donkey 抄送：NikulausRui ,State change I have the same question. How do you fix it ?can you tell me 3Q —You are receiving this because you modified the open/close state.Reply to this email directly, view it on GitHub, or mute the thread. {"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/wroscoe/donkey","title":"wroscoe/donkey","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"[https://github.com/wroscoe/donkey"}},"updates":{"snippets":{"icon":"PERSON","message":"@easonzrq in #308: I have the same question.\r\nHow do you fix it ?can you tell me \r\n3Q"}],"action":{"name":"View Issue","url":"#308 (comment)"}}} [ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "#308 (comment)", "url": "#308 (comment)", "name": "View Issue" }, "description": "View this Issue on GitHub", "publisher": { "@type": "Organization", "name": "GitHub", "url": "https://github.com" } }, { "@type": "MessageCard", "@context": "http://schema.org/extensions", "hideOriginalBody": "false", "originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB", "title": "Re: [wroscoe/donkey] Illegal array length, must be 15 (#308)", "sections": [ { "text": "", "activityTitle": "easonzrq", "activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png", "activitySubtitle": "@easonzrq", "facts": [ ] } ], "potentialAction": [ { "name": "Add a comment", "@type": "ActionCard", "inputs": [ { "isMultiLine": true, "@type": "TextInput", "id": "IssueComment", "isRequired": false } ], "actions": [ { "name": "Comment", "@type": "HttpPOST", "target": "https://api.github.com", "body": "{\n"commandName": "IssueComment",\n"repositoryFullName": "wroscoe/donkey",\n"issueId": 308,\n"IssueComment": "{{IssueComment.value}}"\n}" } ] }, { "targets": [ { "os": "default", "uri": "#308 (comment)" } ], "@type": "OpenUri", "name": "View on GitHub" }, { "name": "Unsubscribe", "@type": "HttpPOST", "target": "https://api.github.com", "body": "{\n"commandName": "MuteNotification",\n"threadId": 375088529\n}" } ], "themeColor": "26292E" } ]

sorry,i cant find where to change the mode.can you tell me ?

open manage.py
def drive()
......
kl = KerasLinear()[I have changed to KerasCategrocial]
......
def train()
......
kl = KerasLinear()[same to drive]
......


Docker Swarm on top of Mantl

Why
I propose that support for Docker Swarm should be added to Mantl.
By having Swarm on Mantl, all the (available) resources on a cluster are exposed via the Docker API directly. Developers are already used to this API / model of working.
Swarm is ideal to run one-time-jobs in production easily. Think about database migrations (which could be run detached), or running some interactive script to generate a report (in which the attached input/output is really usefull).
Finally, Swarm could allow for easy development of microservices on top of Mantl. A developer could write the code for microservices on his local MackBook Air, and use Docker Compose to start a set of microservices via Swarm on top of Mantl, which would use way more memory and CPU than his MacBook Air could provide.
How
Swarm registers itself as a Mesos framework, uses it to allocate resources and uses the Docker Executor to start containers. So far, so good! However, it also requires access to the Docker API on each worker node to provide services like pulling/pushing of images and showing the logs. At the moment, only the /var/run/sock unix domain socket is enabled in Mantl. To expose the Docker API in a secure way, I propose to add a proxy container (just like is done with Consul), to avoid some issues in Swarm [1].
Furthermore, Mesos Framework authentication should be implemented in Swarm [2]. After some investigation, It appears that adding this will take about 100 lines of Go code.
To make sure that a crash of a single Swarm daemon does not take down all containers started via Swarm, Swarm’s high availability option should be used [3]. I suggest that Swarm should run on each Mesos master node, to provide the same level of availability as Mesos itself.
[1] https://github.com/docker/swarm/issues/1089
[2] https://github.com/docker/swarm/issues/1944
[3] https://docs.docker.com/swarm/multi-manager-setup/

Our contract with developers is a docker-compose file.  This would help a lot in crossing between development (compose) and production (marathon).

I'd love to see swarm running on mantl. Our preference is to be an addon installed locally instead of a framework on top of mesos.

@stevendborrelli what do you mean exactly? Swarm on mesos is registering itself as a mesos framework. If it would be added to Mantl as a addon, I imagine it would be in a HA setup, with swarm daemons running on the same nodes as the mesos masters.

After some research, I can't really find any information on integrating Swarm with Mesos's scheduling. Swarm will run jobs separately from Mesos/Marathon (which is fine, just something to be noted). I'll look into setting it up as an addon.

Here's a crazy thought: Swarm is basically installed by running a Docker container on each control and worker node. Maybe we can start these jobs via Marathon?

I have done some work on this, but haven't gotten swarm to elect a leader. It seems that it's having trouble connecting to Consul, though when I try the API interactions from the console using curl, they work just fine.
Branch here: https://github.com/CiscoCloud/mantl/tree/feature/docker-swarm
Error log here: http://pastebin.com/raw/yAisqgfR

+1 for THE SWARM

+1

@distributorofpain @lguminski If either of you has more experience with Swarm and would like to help debug my branch, that'd be greatly apprciated!

Would love to help... i am a bit of a newb though, so i cant say i would know where to begin helping...
Being a linux sysops guy, i would think i could help though :)
Top of my head looking at the log file output( this is where my lack of understanding of mantl core setup could get annoying):

Does mantl run consul on 8500?  Does it require a certificate to talk to?  I only ask because i see environmental variables like this:  CONSUL_SSL=true   CONSUL_SSL_VERIFY=false
I see thats running on <IP_ADDRESS>:8500  If thats in docker, wouldnt you stipulate an outside port?
Is that container setup to run on the bridge network?
 I'm making more progress now. Also, more details on Swarm on Mesos: the help text for swarm has this option:
   --cluster-driver, -c "swarm"						cluster driver to use [swarm, mesos-experimental]

So I think if Mesos support indeed exists, we don't want to use it (because it's experimental).

Second set of eyes thing :)
I just started playing with docker last week.  I had to setup a registry and client to test it.  I really like docker, i can see why people are running with it.  Heck even mantl brought in kubernetes now :)
I saw the demo on youtube of docker swarm running on messoshpere, very impressive.  Everyone is really charging forward with this stuff.

@moretea @distributorofpain @lguminski I put in a PR and would love some feedback!


Support x509.ExtKeyUsageClientAuth

Our mTLS certificates have the x509.ExtKeyUsageClientAuth field set.  Crypto/TLS won't validate certificates unless told to accept it.

weird thing is the puppet ones also have it, so wy does it work with them:
$ openssl x509 -in /home/rip/.puppetlabs/etc/puppet/ssl/certs/rip.mcollective.pem -text
....
            X509v3 Extended Key Usage: critical
                TLS Web Server Authentication, TLS Web Client Authentication
....

The certificates I'm using only have the client auth set.

@ripienaar guess I should've started with the error message(output from choria server):
WARN[0004] Certificate does not pass verification as '': x509: certificate specifies an incompatible key usage: leaf contains the following, recognized EKUs: <IP_ADDRESS>.<IP_ADDRESS>.2, <IP_ADDRESS>.<IP_ADDRESS>.7  component=security ssl=file
WARN[0004] Received certificate 'XXX' certiicate did not pass verification: x509: certificate specifies an incompatible key usage: leaf contains the following, recognized EKUs: <IP_ADDRESS>.<IP_ADDRESS>.2, <IP_ADDRESS>.<IP_ADDRESS>.7  component=security ssl=file

http://oid-info.com/get/<IP_ADDRESS>.<IP_ADDRESS>.2 (client auth)
http://oid-info.com/get/<IP_ADDRESS>.<IP_ADDRESS>.7 (ipsec user) (not sure why we set this)

@ripienaar if you don't specify any ext key usages, then it'll default to using ExtKeyUsageServerAuth:
https://github.com/golang/go/blob/3813edf26edb78620632dc9c7d66096e5b2b5019/src/crypto/x509/verify.go#L771-L774
Could probably make this configurable.


Roughness map has no effect on environment map reflections

Environment

OS Version: Ubuntu 20.04
Ignition Fortress, source

Description

Expected behavior: Roughness maps should affect the sharpness of the environment map applied to objects.
Actual behavior: Roughness maps have no effect on the reflectivity from the environment map

Steps to reproduce

Create an object and apply an environment map.
Adjust the roughness value from 0 to 1. Note the reflection remains sharp and unchanged regardless of the roughness value.
A high roughness value should be making the reflection more diffuse like this.

Output
Based on this image on fuel it appears that roughness used to work as you would expect. Note the roughness map is causing the reflection on the body of the extinguisher to become diffuse.

Compared to current functionality where the reflection is 100% sharp and the reflection intensity is even on the entire model since the roughness map is not being used to control the roughness of the different surfaces of the model.

Ogre2 does support roughness maps on env maps.
However we need the texture to have mipmaps for that. I suspect that, for whatever reason, the env. map does not have mipmaps.
How is the env. map reflection loaded? How can we repro it?
Cheers

We did some more tests afterwards using SaintPetersBasilica.dds from ogre and we do see differences in the result. I now see a blurry / low res image in the reflection. I opened the dds file from ogre using gimp-dds and noticed that there are mipmaps and ours don't so that could be it. I think @ColeOSRF is still testing to see if it's possible to tune more params and produce the desired effect.


Clean codebase by replacing kafka-rest-client/...  with kafka-rest-client v1.0.0

Clean up from codebase by replacing it with kafka-rest-client v1.0.0

nice cool :)


Sending batch requests

The HTTP API and node sdk allow for sending batches of events (https://segment.com/docs/sources/server/http/#batch), but not the client-side javascript SDK. Is this something that could be added? It would be nice to batch events with a configurable flush interval.

Any updates on this?

This is currently not on our roadmap. But I created a ticket internally to track it as a possible future improvement.

@pooyaj :wave:
Wanted to check in on priority here.  Is there anything that we could do to help get this feature in the JS library?

@Austio 👋 I should correct my previous update, we are doing some fundamental improvements to JS library and request batching is in our plans. Not an exact ETA yet, but should be able to give a clear timeline in a couple of months.

@pooyaj heya.  Wanted to touch base and see how this was looking for batch support.

@pooyaj any updates on th ebatch support or plan for this issue?

@pooyaj any updates on th ebatch support or plan for this issue?

@Austio Sorry for the delay. We are working on a new version of our JS library, and hoping to get the beta out somewhere in Q1. Will update the thread as soon as we make the new version available.

@Austio Sorry for the delay. We are working on a new version of our JS library, and hoping to get the beta out somewhere in Q1. Will update the thread as soon as we make the new version available.

Hey @Austio and @markbrouch!
This is officially supported in Analytics.js 2.0 and documented here: https://segment.com/docs/connections/sources/catalog/libraries/website/javascript/#batching


Including power in beaconData cause malformed URL since builder.setInclu...

...deTxPowerLevel(false);

I believe my previous pull request tried to handle the same issue but it was resolved as an old release that no longer reflects fixed code.

I think the problem might be that the 16-bit UUID (FE D8) is included by Android by the addServiceData() so the first 2 bytes are duplicated. Try removing them and seeing how it works.

I tested the lastest master however the malformed url still existed. So in my branch, I added back the flags and Adv Tx Power and removed the repeated UUID and its working for me.

LGTM

Thanks! Glad you persevered and got this working.


-SubscriptionID parameter is not used

I am using the version 3.1.17.
It seems like passing SubscriptionID parameter has no affect. I still get report for all 17 subscriptions I have.

Hello @Claudio-Merola
I use cloudshell and use the following command:
./AzureResourceInventory.ps1 -TenantId xxxx-xxxx-xxxxx-xxxxxx -SubscriptionID xxxxx-xxxxxx-xxxx-xxxxx -Debug
This creates report for all subscriptions not just one.

Hello @demirag
You were right, there was a bug when using Cloudshell.
I just submitted a version 3.1.18 that should fix that.
Can you try that version and let me know if it works correctly?
Thanks

@Claudio-Merola Thank you. I can confirm now it works as expected.


Automate release process

So far, our release process is defined step by step in HOWTO_RELEASE. But as we grow we need to simplify it as much as possible.  With just one command we should perform:

Update NEWS
Create tag in repo following semantic version rules
Publish in the registry

Consider using semantic-release or similar tool.

There is no need to tackle this issue at the moment


Fix bounds in FStar.Char

The following code causes an exception when extracted and executed:
let test = String.string_of_list [Char.char_of_int 0xD800]

See https://github.com/ocaml-batteries-team/batteries-included/blob/77bd2aace3e16499ec71bf544270e169656c3c23/src/batUChar.ml#L51-L55= .
This pull request fix this issue.
Probably a test should be added for this, but I don't know where it should be written?

Thanks @TWal! You could raise an issue, then testcases (both +ve and -ve) in tests/bug-reports/BugN.fst, where N is the issue number, and add BugN.fst to tests/bug-reports/Makefile under SHOULD_VERIFY_CLOSED.


printing checklists does not print checkboxes

I'm not sure if this is an issue of flatnotes but if I create a checklist like
* [ ] foo
* [x] bar

and try to print the list there are no checkboxes on paper/pdf.
btw. a print mode without header, menu, ... would be nice.
Test:

[ ] foo
[x] bar

Hey @AlBundy33. I've had a look at this, and I've no idea why they're not included. If I toggle on the @media print simulator in dev tools they are still there. But not in print preview. I'm at a bit of a loss.
In regard to a print mode without header/menu, these are hidden in the latest release, so please check you're updated.

Hi @dullage, thanks for the fast reply.
I've modified the example in the first post - if I print this issue I can see both checkboxes.
Maybe you can have a look how github renders the fields.
Printing without header and menu works perflectly. 👍
But on notes with text that does not need a complete page an empty page is at the end of the print preview.

any news about ths issue?


Not working in latest Safari on Mac

The demo on your site works like a charm. The files downloaded from GitHub are not working. I will let you know if I can figure anything out.


New SequencingPolicy: IdentifierExtractingSequencingPolicy

Feature Description
We have various events in our application that have a common interface and thus a common identifier which is used for the sequencing policy. For instance, we have a TaskEvent interface which makes sure that TaskCreatedEvent and TaskDeletedEvent have a method getTaskId to provide the identifier. This payload of the event message - if it is of type TaskEvent - is casted to TaskEvent and the identifier is extracted via the getTaskId method. We use this type of policy multiple times and suspect a common pattern.

Current Behaviour
There are currently three types of default sequencing policies: Sequential, Sequential per Aggregate and Full Concurrent.
Wanted Behaviour
We suggest to implement a new IdentifierExtractingSequencingPolicy, which consists of the payload type, the identifier extracting function (the getter) and a fallback sequencing policy. If the event message is not of the given payload type, the fallback sequencing policy is applied. The policy should have two constructor methods:
public static <T, K> SequencingPolicy<EventMessage> createWithFallback( final Class<T> payloadType, final Function<T, K> identifierExtractor, SequencingPolicy<EventMessage> fallbackSequencingPolicy)
public static <T, K> SequencingPolicy<EventMessage> createWithoutFallback( final Class<T> payloadType, final Function<T, K> identifierExtractor )

In case of the createWithoutFallback method, the sequencing policy would be a (nested) ExceptionThrowingSequencingPolicy which would simply raise an exception.
If you are interested, I could provide the full code, as we implemented this feature already in a generic way for our application.

I have been in the habit of constructing policies like this in the past at users of the framework.
Hence, it makes sense to provide something for this within the framework.
So, if you are up for the task, feel free to provide a pull request for this @nils-christian.
Note that for infra components we tend to use the Builder paradigm instead of constructors.
This allows for greater flexibility to changing the required fields at a later stage without constructing more constructors.
If you're going to provide a PR, I'd thus like to ask you to use a Builder for this sequencing policy as well.

If I may suggest a name for the class, it would be something along the lines of PropertySequencingPolicy.
Implementation hints for whoever is picking up the PR:
Axon uses the property extraction mechanism in a few places. One example is the Saga's Association Values (see org.axonframework.modelling.saga.PayloadAssociationResolver#getProperty).
The static org.axonframework.common.property.PropertyAccessStrategy#getProperty method retrieves a Property from a given class, or null if such property doesn't exist. These properties can be safely cached in a Map. The value can then be extracted based on an instance of that class.

Closed by https://github.com/AxonFramework/AxonFramework/pull/1831.


 Here at chin123, we take feedback very seriously. We will be sure to take your opinions into account in the future!
Stay safe and best regards,
All of us at chin123

Done


NPE when no third party datasource configured

This is the pairing issue with https://github.com/actframework/act-ebean2/issues/6

fixed in 1.1.5


Intrusive GPU profiler

Hello,
I saw you are working on debug-inspection extensions and this reminds me I would like to implement an intrusive GPU profiler for Sokol. The idea would be to adapt the intrusive GPU profiler proposed in the project Dx11Base and extending it for OpenGL, Metal, and WebGL (if possible). The ImGui UI can stay the same.
Do you think this would be an interesting feature?

Yes sounds like an interesting feature, maybe add a minimal amount of instrumentation code to the sokol_gfx.h backends, and then have a separate header to render an UI via ImGui...
If you are experimenting in that direction, please keep me updated, I'd be interested how the backend integration would need to look like and I think it makes sense to add this functionality to "core sokol_gfx.h".

I agree to have specific instrumentation code in sokol_gfx and the rest in a separate header for the UI.
Regarding the API, it can be very similar than the one proposed in DX11Base.
// Add a member in the structure `sg_desc` to enable the GPU profiler. For example:
sg_desc desc();
desc.enable_gpu_profiler = true; // This will initialize the backend in consequence
sg_setup(&desc);

In the public API four new functions will be proposed and two macros
// - sg_gpu_profiler_start_frame(): start to sample the content of a frame
// - sg_gpu_profiler_start_timer(const char *name, unsigned r, unsigned g, unsigned b, unsigned a): create a main timer to sample the content of a frame
// - sg_gpu_profiler_stop_timer(const char *name): stop the timer previously started using its name
// - sg_gpu_profiler_end_frame(): stop to sample the content of a frame

The macros simplifies the creations of timer to sample specific parts of the rendering.
// - SOKOL_GPU_SCOPED_TIMEREVENT(name, r, g, b, a): Create a new event to sample in a frame and start a timer
// - SOKOL_GPU_SCOPED_TIMER(name, r, g, b, a): Create a new timer to sample a part of an event.

To give you an example of this API (and largely based on the code proposed in DX11Base),
// render loop
while (true) {
  // Start a new frame to sample
  sg_gpu_profiler_start_frame();
  const char* frameGpuTimerName = "Frame";
  sg_gpu_profiler_start_timer(frameGpuTimerName, 150, 150, 150)
  // Sample specific parts
  GPU_SCOPED_TIMEREVENT(GameRender, 75, 75, 75);
  {
    GPU_SCOPED_TIMER(Clear, 34, 177, 76);
    sg_begin_default_pass(&pass_action, cur_width, cur_height);
  }
  {
    GPU_SCOPED_TIMER(Shadow, 128, 128, 255);
    {
      GPU_SCOPED_TIMER(ShadowCasc0, 64, 64, 255);
      {
        GPU_SCOPED_TIMER(Shadow0, 64, 0, 255);
        // The GPU code to sample
     }
     {
        GPU_SCOPED_TIMER(PartShad0, 0, 64, 255);
        // The GPU code to sample
    }
    // Continue this for the rest of your code
  }
  // After sg_commit()
  sg_gpu_profiler_stop_timer(frameGpuTimerName);
  sg_gpu_profiler_end_frame();
}

The result could looks likes this using the UI proposed in DX11Base.
For the internal structures, again the ones proposed by DX11Base are very well done and use internally a graph (see the content of the class DxGpuPerformance). However, some lines must be replaced to be specific to each backend. For example, here and here.
I have not yet thought about the structures to implement for each backend.

I haven't tried it myself but what I've experienced from using remotery, metal API does not support per API call profiling and can only sample the whole command buffer.

@septag Good catch! That would mean than specifically for the Metal backend the macros SOKOL_GPU_SCOPED_TIMEREVENT and SOKOL_GPU_SCOPED_TIMER will have not no effect (e.g. defined empty).
The profiler will return only the the time associated with a sample ("Frame" in my example), but not the events inside.

FYI, I continued my review of available intrusive GPU profilers. I found tracy and this is more than I expected (CPU / GPU / memory profiler), plus the author is working on a docking version (see this screenshot).
It seems irrelevant for me to work on another profiler but someone might has counter arguments against this!
My main question now is about the possibility to see the sokol trace hook API with tracy

Another GPU profiler with a Dear ImGui user interface: https://github.com/dougbinks/microprofile


cutorch hangs on ec2+centos

Hi,
I try to deploy cutorch on ec2 gpu instance with centos7. I've successfully installed latest nvidia drivers and cuda 7.5, also successfully compiled and executed some cuda examples. After installing torch and cutorch/cunn, when I run th and require 'cutorch' it just hangs. top shows 100% cpu usage on ksoftirqd, nothing more happens. Can someone help me with that?
==============NVSMI LOG==============
Timestamp                           : Wed Sep  9 17:30:20 2015
Driver Version                      : 352.39
Attached GPUs                       : 1
GPU 0000:00:03.0
Product Name                    : GRID K520
Product Brand                   : Grid
PID USER      PR  NI    VIRT    RES    SHR S  %CPU %MEM     TIME+ COMMAND
63 root      20   0       0      0      0 R 100.0  0.0   0:07.50 ksoftirqd/5
20914 centos    20   0 36.876g 224948  31092 D  90.7  1.5   0:09.34 luajit
20902 centos    20   0  130024   1840   1308 R   0.3  0.0   0:00.15 top
1 root      20   0   56748   6640   3908 S   0.0  0.0   0:02.65 systemd
2 root      20   0       0      0      0 S   0.0  0.0   0:00.00 kthreadd
Best regards,
Sebastian

it's usually the JIT compilation that hangs for 2-3 mins (probably longer on EC2 instances). Should only happen once (after that it caches).
http://devblogs.nvidia.com/parallelforall/cuda-pro-tip-understand-fat-binaries-jit-caching/

Another scenario which seems to replicate this is when multiple itorch instances are running (or sleeping). Killing these instances resolved the problem for me.


CRAN check failing

Dear maintainer,
Please see the problems shown on
https://cran.r-project.org/web/checks/check_results_metajam.html.
Please correct before 2024-08-19 to safely retain your package on CRAN.
It seems we need to remind you of the CRAN policy:
'Packages which use Internet resources should fail gracefully with an informative message
if the resource is not available or has changed (and not give a check warning nor error).'
This needs correction whether or not the resource recovers.
The CRAN Team

For the record, the current failure is
── Error ('test-download_ISO_data.R:15:3'): test data URL with ISO
metadata from Research Workspace member node and with multiple data
tables ──
Error in h(simpleError(msg, call)): error in evaluating the
argument 'x' in selecting a method for function 'resolve': Timeout was
reached: [cn.dataone.org] Connection timeout after 10001 ms
Backtrace:
▆
1. ├─metajam:::download_ISO_data(...) at test-download_ISO_data.R:15:3
2. │ ├─dataone::resolve(dataone::CNode("PROD"), data_id)
3. │ ├─dataone::CNode("PROD")
4. │ └─dataone::CNode("PROD")
5. │   └─dataone (local) .local(x, ...)
6. │     └─httr::GET(CN_URI)
7. │       └─httr:::request_perform(req, hu$handle$handle)
8. │         ├─httr:::request_fetch(req$output, req$url, handle)
9. │         └─httr:::request_fetch.write_memory(req$output,
req$url, handle)
10. │           └─curl::curl_fetch_memory(url, handle = handle)
11. └─base::.handleSimpleError(...)
12.   └─base (local) h(simpleError(msg, call))
--
Brian D. Ripley<EMAIL_ADDRESS>Emeritus Professor of Applied Statistics, University of Oxford

Jenny's inputs:
I have lots of thoughts and strategies around this, but the TL;DR really is that I include skip_if_offline() in tests that should not run w/o internet access.
And skip_if_offline() also skips on CRAN.
https://github.com/tidyverse/googledrive/blob/ed574d9eb3308619fc3e735251bfd42e6cfdb8e2/tests/testthat/test-drive_cp.R#L30C3-L30C20
https://testthat.r-lib.org/reference/skip.html
testthat.r-lib.orgtestthat.r-lib.org
Skip a test — skip
skip_if() and skip_if_not() allow you to skip tests, immediately
concluding a test_that() block without executing any further expectations.
This allows you to skip a test without failure, if for some reason it
can't be run (e.g. it depends on the feature of a specific operating system,
or it requires a specific version of a package).
Show more
https://testthat.r-lib.org/reference/skip.html
One of the testing chapters in R Packages also has a lot of relevant content. Which basically boils down to some version of the above advice and some version of what you are already doing, in case that makes it feel better!
https://r-pkgs.org/testing-advanced.html#when-testing-gets-hard
https://r-pkgs.org/testing-advanced.html#special-considerations-for-cran-packages
And you can specify a host, which makes it feel (and maybe even be?) more authentic. I do that somewhere, but I guess not in the example above.

Lluis:
There is a book about this topic https://books.ropensci.org/http-testing/ in the case of vignettes an approach is to precompute them. You could also use a prerecorded request but that is more complicated.

Nicholas:
I wrote the rnassqs package, which is a wrapper around an API and benefited quite a lot from review by the Neal Richardson, the author of the httptest package. You might look at the test structure there for an example. The setup gets things going with mock API calls, and then the actual tests are mostly in test-request. Although the tests are a few years old so they may not match current best practices.


pycc compatibilty with versioneer

I notice that CC.distutils_extension monkey patches distutils.command.build_ext.build_ext with a modified subclass, this can lead to a bug when using with versioneer.
In versioneer.py:
    if "build_ext" in cmds:
        _build_ext = cmds["build_ext"]
    elif "setuptools" in sys.modules:
        from setuptools.command.build_ext import build_ext as _build_ext
    else:
        from distutils.command.build_ext import build_ext as _build_ext

    class cmd_build_ext(_build_ext):
        def run(self):
            root = get_root()
            cfg = get_config_from_root(root)
            versions = get_versions()
            _build_ext.run(self)
            if self.inplace:
                # build_ext --inplace will only build extensions in
                # build/lib<..> dir with no _version.py to write to.
                # As in place builds will already have a _version.py
                # in the module dir, we do not need to write one.
                return
            # now locate _version.py in the new build/ directory and replace
            # it with an updated value
            target_versionfile = os.path.join(self.build_lib, cfg.versionfile_build)
            print("UPDATING %s" % target_versionfile)
            write_to_version_file(target_versionfile, versions)

    cmds["build_ext"] = cmd_build_ext

It also patches build_ext with its customized version of build_ext.
So the following will fail:
setup(version=versioneer.get_version(),
      cmdclass=versioneer.get_cmdclass(),
      ext_modules=[cc.distutils_extension()])

Traceback (most recent call last):
  File ".../setup_example/test.py", line 8, in <module>
    import src.aot as aot
ImportError: .../setup_example/src/aot.cpython-39-x86_64-linux-gnu.so: undefined symbol: nrt_atomic_sub

However if we just switch cmdclass & ext_modules:
setup(version=versioneer.get_version(),
      ext_modules=[cc.distutils_extension(),
      cmdclass=versioneer.get_cmdclass()])

this will succeed. Because in the first case, the ultimate cmdclass will not use pycc's build_ext class (the cmdclass is created before pycc's money patch), but the second will.
I propose a simple fix here:
rather than patching the entire class, I patch only build_extension method:
    @classmethod
    def monkey_patch_distutils(cls):
        """
        Monkey-patch distutils with our own build_ext class knowing
        about pycc-compiled extensions modules.
        """
        if cls._distutils_monkey_patched:
            return

        _orig_build_ext = build_ext.build_ext.build_extension

        def build_extension(self, ext):
            if isinstance(ext, _CCExtension):
                ext._prepare_object_files(self)

            _orig_build_ext(self, ext)

        build_ext.build_ext.build_extension = build_extension

        cls._distutils_monkey_patched = True

In this way it will not be influenced by the order of get_cmdclass() & cc.distutils_extension()
If you feel this is reasonable I'm willing to make a PR.

NumPy has just made a breaking change to the way that they monkey patch distutils and we're going to have to redesign this. That isn't going to be immediate, but we will need it to happen before 0.57. See https://github.com/numba/numba/issues/8355
How urgently do you need this change? It might be better to wait for the redesign which should happen before next calendar year.

Not a big problem for me, since switching get_cmdclass() & cc.distutils_extension() can get around.

We still need to fix this at some point, so I've removed the 'stale' label here.


shortcut for night light

there should be a shortcut fo rnight light as each time when it turns on I have to exit my full screen and go to the action centre and then turn it off . if there will be a shortcut i can toggle it off or on more easily

fyi: you can turn it on via a schedule.

Maybe it can be combined with #1331?

A lot depends on what we have access to via public APIs.


bugfix compile error

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

This problem may occur because your version of thirdparty is wrong

This problem may occur because your version of thirdparty is wrong

but i'm weird: The thirdparty seems to have not been updated ， And this error is only compiled on arm

This problem may occur because your version of thirdparty is wrong

but i'm weird: The thirdparty seems to have not been updated ， And this error is only compiled on arm

1.2 require s2geo v0.9, master require s2geo 1.0, you can check this.

This problem may occur because your version of thirdparty is wrong

but i'm weird: The thirdparty seems to have not been updated ， And this error is only compiled on arm

1.2 require s2geo v0.9, master require s2geo 1.0, you can check this.
yeah，closed


Notebook Terminal does not use --marimo-monospace-font (or any other css var from custom.css)

Describe the bug
I wanted to open a terminal in Marimo, but my prompt was messed up, because I'm using nerd font and xterm in marimo could not render in properly.
I tried configuring --marimo-monospace-font as the documentation says it's the font used for code blocks, so I assume terminal would use it as well. But it does not, an it appears that there's no proper way to configure it.
 This is an OK fix, and essentially the same as I did manually. But it probably would be better if you had a separate css var for terminal font and fall back to monospace if the former is not set. As they are used differently: the monospace is presented to the user in code sections, where as the terminal one is only used during development


Safe expect batch

Wrong usage of the expecter may cause the tests to be flaky.
ExpectBatchWithValidatedSend adds the expect.BSnd command to the expect.BExp
expression. It is done to make sure the found match is in the result of
the expect.BSnd command and not in a leftover that wasn't removed from
the buffer.
Issue https://github.com/kubevirt/kubevirt/issues/3619 describes such
scenario.
ExpectBatchWithValidatedSend also prevents having two sequential expect.BExp which
cause unpredictable behaviour as happened in -
https://github.com/kubevirt/kubevirt/issues/3602 and explained in PR
https://github.com/kubevirt/kubevirt/pull/3693.
For simplicity, ExpectBatchWithValidatedSend also prevents two sequential expect.BSnd.
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
For some reason \n appears in the expecter buffer as \r\n.
So when trying to find in the buffer a previously sent command that ended
with \n it fails.
To workaround this issue, the \n suffix is trimmed from the expected send command.
Release note:

NONE

/test all

/test all

/test all

/test all

/test all

If you want, discuss with Dan, he suggested that the command + the retcode would be send to a function that would wrap both

If you want, discuss with Dan, he suggested that the command + the retcode would be send to a function that would wrap both

Can you please explain how this suggestion contradicts with the approach in this PR?

If you want, discuss with Dan, he suggested that the command + the retcode would be send to a function that would wrap both

Can you please explain how this suggestion contradicts with the approach in this PR?

It might not contradict, just up to you if you decide to check out what he suggested and either do it on other PR or whatever you deicde

/test pull-kubevirt-e2e-k8s-cnao-1.17

I think it's a good idea overall to have a function that sanitizes batches before running them.
Did you consider running stty -echo (or equivalent) after logging in? That would ensure BExp never sees what was sent.
IIUC,  stty -echo causes the send command not to be printed. Since that's the way BExp validates the expected value is the result of the send command, I'm not sure how it helps us. Or you suggesting it as another approach?

If you meant as another approach, than yes, it may reduce the flakiness since a lot of the problematic cases were caused by the BExp finding a match in the BSnd command instead of the actual output.
But validating the expected value is the result of the sent command covers more flaky cases (when the BExp finds a match which is left over of a previous command result).
Looking at the man page, maybe stty -ocrnl can help us with the \r issue since it translates carriage return to newline. I will try it. Thanks for the hint.

For all the prompt BExps, I'd recommend using raw strings (backticks) instead of interpreted strings (double-quotes), which would allow us to avoid escaping the escape character (\\)...

It seems that in most of the tests the fact that the value passes to BExp is regexp is ignored. So it is a good idea to go over all the tests and change most of the BExp to use backticks.
But I think it worth a separate PR.

@AlonaKaplan switching to backticks in a separate PR seems fair, I've been meaning to give it a shot, I'll try to remember it after this is merged.
Sorry for not being more explicit, but I suggested stty -echo because I'm not a huge fan of messing with the batch to "consume" every send on receive. (I also think it's ok to do multiple sends in a row)
As you touched on in a comment, serial consoles don't know how wide the terminal is and default to 80 (another thing that can be fixed with stty!).
In your tests, you encountered wraps after ~70 chars, meaning the prompt must be ~10 chars.
However, some distros show the hostname (VMI name) in the prompt, which is pretty long in the functests. In that case, the wrap can happen after ~20 chars instead of 70...

@AlonaKaplan switching to backticks in a separate PR seems fair, I've been meaning to give it a shot, I'll try to remember it after this is merged.
Sorry for not being more explicit, but I suggested stty -echo because I'm not a huge fan of messing with the batch to "consume" every send on receive. (I also think it's ok to do multiple sends in a row).

When doing multiple sends in a raw, we cannot be sure the expect found a match only after all the sends were executed. It is safer to have one send. Multiple.commands in the send can be split with ';'.
If for some reason we need to have multiple sends in a raw, we can use the regular expect.expectBatch for this use case.

As you touched on in a comment, serial consoles don't know how wide the terminal is and default to 80 (another thing that can be fixed with stty!).
In your tests, you encountered wraps after ~70 chars, meaning the prompt must be ~10 chars.
However, some distros show the hostname (VMI name) in the prompt, which is pretty long in the functests. In that case, the wrap can happen after ~20 chars instead of 70...

Why does it mean the prompt must be ~10 chars? It only means the the BExp will try to find a match only to the first 70 chars of the sent command. The length of the prompt or any other value in the expect is not limited.

/test pull-kubevirt-e2e-k8s-1.17

/test pull-kubevirt-e2e-kind-k8s-1.17.0-ipv6

/test pull-kubevirt-e2e-k8s-1.17

/test pull-kubevirt-e2e-kind-k8s-1.17.0-ipv6

Would be great please if at the end, if the commits can be squashed to several commits with topic on each,
for example

create the mechanism.
use the mechanism.
far more re-viewable in the future this way imo.

/test pull-kubevirt-e2e-k8s-1.16

/test pull-kubevirt-e2e-kind-k8s-1.17.0-ipv6

Would be great please if at the end, the commits will be squashed to several commits with topic on each,
for example
1. create the mechanism.

2. config consoles.

3. use the mechanism in the tests.

far more re-viewable in the future this way imo.

I've re-arranged the commits.

/retest

/retest

/approve

/retest

This is a great improvement.
Although it is a too late now, but for the sake of future cases like this, it would be nice to separate the main enhancement in the tooling and the various fixes.
The enhancement may come with a few fixes to show it works, but the rest will fit nicely in following PR/s.

We still need a follow-up to use the ExpectBatchWithValidatedSend instead of expecter.ExpectBatch where possible. This PR doesn't cover all the places.
Also we need follow-up to use RetValue where possible in BExp, the PR cover just the echo $? results.

@oshoval - couldn't quote reply. In this comment, do you mean add space after the #?

@oshoval - couldn't quote reply. In this comment, do you mean adding space after the #?

yep , just on the lines you added \\

Thanks
/lgtm

/hold
@EdDev please remove the hold if you are fine with it as well, (since there is an approve i added the hold)

/retesy

/retest

/hold cancel

/lgtm
/retest

/retest

/lgtm cancel
you said there is one issue found

/lgtm cancel
you said there is one issue found

My review stands, this looks fine to me.
The owner can hold it if there is something that needs investigation.
/lgtm

/hold
https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/3882/pull-kubevirt-e2e-k8s-1.16/1291187384536272896

/hold cancel
The error in test https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/3882/pull-kubevirt-e2e-k8s-1.16/1291187384536272896 doesn't see to be related to the PR. A random  \r was added to the command. Although I have no idea why it happened, a follow-up PR will be sent changing the failing test to have shorter line and not using sudo.


Update org.nodejs pkg to 4.1.1

-- https://nodejs.org/en/blog/release/v4.1.1/

Thank you for the contribution. It was merged directly as ad03432dc72186283caa4c26e77ca301635384d1 to keep commit history cleaner. Your contribution is still credited to you.


GradleBuildInfoExtractor: only add modules with artifacts to build info

This addresses "Gradle Build Info Extractor should not add modules without artifacts" https://github.com/jfrog/build-info/issues/342

@rpalcolea,
Thanks for creating this PR!
If a module does not include artifacts, but does include depedencies, it should still be added to the build-info. Otherwise, the info about the dependencies will not be recorded.
I guess that we need check whether a module does not include artifacts and depdencies.

Hi @eyalbe4
Thanks for looking at this
I did the change =)


implementing openssl decrypt block using openssl

snippet from: https://github.com/Yubico/yubikey-ksm/issues/24
also reindented php to be more readable in this file

tested on PHP 7.3 @ debian 10


Bug: Make TargetType support abstract classes

Describe the bug
Hi,
While trying to add support for MAUI WinUI in LibVLCSharp (https://github.com/videolan/libvlcsharp/pull/370), I've ran into what I believe is a regression. Indeed this behavior is supported in other XAML/UI flavors so I believe it should also work in WinUI MAUI.
Given this style:
<ResourceDictionary
    xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
    xmlns:vlc="using:LibVLCSharp.Platforms.Windows">

    <Style TargetType="vlc:VideoViewBase">
        <Setter Property="Template">
            <Setter.Value>
                <ControlTemplate TargetType="vlc:VideoViewBase">
                    <SwapChainPanel x:Name="SwapChainPanel" />
                </ControlTemplate>
            </Setter.Value>
        </Setter>
    </Style>
</ResourceDictionary>

And this base class:
[TemplatePart(Name = PartSwapChainPanelName, Type = typeof(SwapChainPanel))]
public abstract class VideoViewBase : Control, IVideoView
{
/// stuff
}

This works fine in UWP, WinUI and Uno. But in WinUI for MAUI, I get the following error logs and stack trace:
Exception thrown at 0x00007FFE1316B699 (KernelBase.dll) in LibVLCSharp.MAUI.Sample.exe: WinRT originate error - 0x800F1000 : 'Cannot apply a Style with TargetType 'LibVLCSharp.Platforms.Windows.VideoViewBase' to an object of type 'Microsoft.UI.Xaml.Controls.Control'.'.
Microsoft.ui.xaml.dll!00007FFD09D843D2: 800F1000 - E_NER_INVALID_OPERATION
Exception thrown: 'System.Runtime.InteropServices.COMException' in WinRT.Runtime.dll
Exception thrown at 0x00007FFE1316B699 (KernelBase.dll) in LibVLCSharp.MAUI.Sample.exe: WinRT originate error - 0x800F1000 : 'No installed components were detected.

stack:
 	KernelBase.dll!00007ff95768b699()	Unknown
 	Microsoft.UI.Xaml.Controls.dll!00007ff8c616f5bf()	Unknown
 	Microsoft.UI.Xaml.Controls.dll!00007ff8c5d6d0b3()	Unknown
 	Microsoft.UI.Xaml.Controls.dll!00007ff8c5d82e9e()	Unknown
 	Microsoft.UI.Xaml.Controls.dll!00007ff8c5dc42c1()	Unknown
 	Microsoft.UI.Xaml.Controls.dll!00007ff8c5dd89a8()	Unknown
 	[Managed to Native Transition]
>	Microsoft.WinUI.dll!ABI.Microsoft.UI.Xaml.IFrameworkElementOverridesMethods.MeasureOverride(WinRT.IObjectReference _obj, Windows.Foundation.Size availableSize) Line 30	C#
 	Microsoft.WinUI.dll!Microsoft.UI.Xaml.FrameworkElement.MeasureOverride(Windows.Foundation.Size availableSize) Line 673	C#
 	Microsoft.WinUI.dll!Microsoft.UI.Xaml.FrameworkElement.Microsoft.UI.Xaml.IFrameworkElementOverrides.MeasureOverride(Windows.Foundation.Size availableSize) Line 678	C#
 	Microsoft.WinUI.dll!ABI.Microsoft.UI.Xaml.IFrameworkElementOverrides.Do_Abi_MeasureOverride_0(nint thisPtr, Windows.Foundation.Size availableSize, Windows.Foundation.Size* result) Line 33	C#

This looks like the same issue as https://github.com/CommunityToolkit/Windows/issues/273.
I can reproduce the error 90% of the time when running the sample app.
Steps to reproduce the bug

Clone https://github.com/videolan/libvlcsharp/pull/370
Run the sample without https://github.com/videolan/libvlcsharp/commit/267bf18a11f59600a72c36c51d19d3c43142ce27
Observe native exception on start

Expected behavior
Support abstract classes in Style TargetType in MAUI WinUI. Like in regular WinUI/UWP.
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.6.3: 1.6.241114003
Windows version
Windows 10 (21H2): Build 19044
Additional context
No response

This seems to be an issue with maui. Please re-file the bug with their repo:
https://github.com/dotnet/maui


Start first pass on eslint

Still 280 errors left that needs to be fixed manually (these are all fixed automatically using eslint --fix) That's why I haven't enabled travis check for this.

Tested on localhost by running the server and labeling a couple of things.


Alternate Star Orgs Logos

Noting that these logos are adapted from "Death Star" by Sergey Demushkin licensed under CC BY 3.0 US.


Release tmt-1.18.0

Follow the steps below to create a new major or minor release:

[ ] Run the full test coverage using tmt -c how=full run
[ ] Use git log --oneline --no-decorate x.y-1.. to generate the changelog
[ ] Update README with new contributors since the last release
[ ] Add a Release tmt-x.y.0 commit with the specfile update
[ ] Create a pull request with the commit, ensure tests pass

Release a new package to Fedora and EPEL repositories:

[ ] Move the fedora branch to point to the new release
[ ] Create a source rpm using the make srpm command
[ ] Enable Fedora kerberos kinit<EMAIL_ADDRESS>[ ] Change to the fedora rpms git or fedpkg clone tmt
[ ] Check out the rawhide branch git checkout rawhide
[ ] Import the srpm using fedpkg import /path/to/the/srpm
[ ] Restore any files removed by fedpkg if necessary
[ ] Ensure the proposed changes are ok and commit them
[ ] Create a pull request against rawhide from your fork
[ ] After tests pass, merge the pull request to rawhide
[ ] Build the package for rawhide fedpkg build --nowait
[ ] Build package for all active releases git checkout f33 && git merge rawhide && git push && fedpkg build --nowait
[ ] Create a bodhi update for each release git checkout f33 && fedpkg update --type enhancement --notes 'Update title'

Finally, if everything went well:

[ ] Tag the commit with x.y.0, push tags git push --tags
[ ] Manually merge the original release pull request on github (to avoid rebase) git checkout main && git merge --ff-only <release_branch> && git push origin main
[ ] Create a new github release based on the tag above
[ ] If the automation triggered by publishing the new github release was not successful, publish the fresh code to the pypi repository manually using make wheel && make upload
[ ] Once the copr build is completed, move the quay branch to point to the release commit as well to build fresh container images.
[ ] Close the corresponding release milestone

To be rebased after regression fix #1564 is merged.

Full test suite executed, two tests failed:

/tests/provision/virtual/dependencies
/tests/run/shell

Seems to be related to nested virtualization as local re-run passed without any issue.

Internal tests executed manually, everything green.


Can I use a different font (like Palladio) in LaTeXStrings, and if so, how?

As my thesis uses the Palladio font, I'd like to have my Julia plots match it. Is there a way to use LaTeXStrings with a different font, and if so, how do I go about setting it?

This is similar to my question on MathTeXEngine:
https://github.com/Kolaru/MathTeXEngine.jl/issues/53
so the answer seems to be: not at the moment

I'll need to check myself, but if using Matplotlib is an option, Matplotlib can be used in Julia, If used via pyCall, rather than through the Plots.jl package (or Makie, in my case), then all Matplotlib features should be available.

This is a FAQ: https://github.com/stevengj/LaTeXStrings.jl/blob/master/README.md#faq-not-a-latex-renderer


Add new commands lockTrunk & unlockTrunk

Tested on 2024 Cadillac  Lyriq
unlockTrunk unlocks but doesn't lift up the trunk. One can then open and close the trunk by pushing a button on the vehicle. All doors remain locked.
lockTrunk locks the trunk if it was unlocked, it doesn't close the trunk.

Will do.
I'll probably just fork the whole thing and install it through HACS from my repo, my goal is to try to add controls to Home Assistant UI.


demonstration of issue with blank characters in expect/actual values

It all stated when I wanted to write a test which makes assertions on HTTP response headers:
@test 'HTTP response header \`HTTP/1.1 200 OK\` one first line for GET http://httpbin.org/' {
	run curl --silent --fail --head http://httpbin.org/
	assert_output -l 0 "HTTP/1.1 200 OK"
}

which fails with a very confusing message:

? HTTP response header \`HTTP/1.1 200 OK\` one first line for GET http://httpbin.org/
  (from function `assert_output' in file test/lib/bats/batslib.bash, line 270,
   in test file test/carriage-return.bats, line 8)
    `assert_output -l 0 "HTTP/1.1 200 OK"' failed

  -- line differs --
  index    : 0
  expected : HTTP/1.1 200 OK
  actual   : HTTP/1.1 200 OK
  --

the HTTP protocol specification. RFC2616 states at the beginning of Section 2.2 (Basic Rules (!)):

CR             = <US-ASCII CR, carriage return (13)>
LF             = <US-ASCII LF, linefeed (10)>
HTTP/1.1 defines the sequence CR LF as the end-of-line marker for all protocol elements except the entity-body

So this test fails because the actual result is literally HTTP/1.1 200 OK\r and we are expecting HTTP/1.1 200 OK.

In this case, the test is the issue, I should expect HTTP/1.1 200 OK\r. But the fail message is also an issue. It should be explicit to the user that the two strings were not the sames.
I started a StackOverflow question about this matter here and it seems one way to do it would be to change a bit the output.bash's functions to make use of `printf "%q" for the actual and expected values.

I did tried that change with at https://github.com/ztombol/bats/pull/3/files#diff-1f8d2e29761b13fe455cefab1ad4b777R113
which gave me:

 ? HTTP response header \`HTTP/1.1 200 OK\` one first line for GET http://httpbin.org/
   (from function `assert_output' in file test/lib/bats/batslib.bash, line 270,
    in test file test/carriage-return.bats, line 8)
     `assert_output -l 0 "HTTP/1.1 200 OK"' failed

   -- line differs --
   index    : 0
   expected : $'HTTP/1.1 200 OK'
   actual   : $'HTTP/1.1 200 OK\r'
   --

Does this sounds to you like a good feature to have?

This pull request should serve as the main place for discussion on the topic.
The favourable behaviour and output format presented in this comment is just a quick suggestion. Nothing is carved in stone. All suggestions are welcome. We should work it out together. :)

@thomasleveil , thanks for investigating this important issue. I strongly support implementing some kind of escaping strategy.
Depending on the speed the Standard Library pull request is progressing and the size of this patch set, it may end up becoming a separate pull request to Bats. For now, let's keep the discussion here. Thanks again! :+1:

Good point, we should look into this. You are right, shell-escaping using printf '%q' is the correct way to implement this. But there are a few desirable properties that your approach does not have.

Only escape when necessary. Forcing escaping on strings that does not contain non-printable characters make for an equally bad user experience.

Instead of having to figure this out:
-- output differs --
expected : $'Data\ saved!'
actual   : $'Error:\ No\ such\ file\ or\ directory:\ \`test.file\''
--

Not performing escaping when not necessary, gives cleaner output:
-- output differs --
expected : Data saved!
actual   : Error: No such file or directory: `test.file'
--

Escaping should be done only when necessary, and even then it should be explicitly stated where it was applied.
-- output differs --
expected   : HTTP/1.1 200 OK
actual (e) : $'HTTP/1.1 200 OK\r'
--

Handle multi-line string escaping. If the same escaping is applied to strings consisting of multiple lines, they will end up as long, single-line, escaped strings.

Escaping should be done line-by-line to preserve separate lines, and only on lines that require it to improve readability. Escaped lines should also be marked.
-- output differs --
expected (1 lines):
  Data saved!
actual (3 lines):
  Error: No such file or directory: `test.file'
e $'\tMake\ sure\ you\ have\ write\ permission!'
e $'\tTerminating...'

Another thing to consider is that the notation used to mark escaped lines must not conflict with the line highlighting (>) of refute_output -l.

Escape strings with trailing spaces. The space is a printable character, but when it appears at the end of strings it easily goes unnoticed.

@test 'Trailing whitespace' {
  run echo 'Success! '
  assert_output 'Success!'
}

Instead of the confusing output:
-- output differs --
expected : Success!
actual   : Success!
--

The escaped string should be shown:
-- output differs --
expected : Success!
actual   : $'Success!\ '
--

The following related issue makes the situation a bit more trickier.

run removes the trailing newline from $output and ${lines[@]}. Obviously, this makes it impossible to test for a trailing newline. This is why you have to expect $'HTTP/1.1 200 OK\r' instead of the $'HTTP/1.1 200 OK\r\n' as described by the specification.
I consider this a bug in Bats, which I haven't reported yet... It may be related to the $IFS problem. I haven't had the time to investigate it yet.
All tests below pass even though they clearly shouldn't.


Recording won't start if you go away from the app

When you go away from the app before the recording has started (the countdown phase) the app won't start recording.
This might be a bug - running on OP5T with Android Pie

Hmmm - I'm unable to reproduce that on my Pixel 3 XL with Android Pie.  🤔

Okay,
that might be also OnePlus' fault because my OP5T is running Open Beta Builds so they are a little bit unstable.
Thanks anyway for checking it with Pixel 3 XL!

I have the same oneplus 5t running the lastest Oxygen Oreo, i do not have any problem.


remove the aws build and deploy steps

we're in azure now - dropping the concurrent aws build and deploy support for production
Review checklist

[ ] First, the most important one: is this PR small enough that you can actually review it? Feel free to just reject a branch if the changes are hard to review due to the length of the diff.
[ ] If there are any migrations, will they the previous version of the app work correctly after they've been run (e.g. the don't remove columns still known about by ActiveRecord).
[ ] If anything changed with regards to the public API, are those changes also documented in the apiary.apib file?
[ ] Are all the changes covered by tests? Think about any possible edge cases that might be left untested.

You can also remove lines 45-49:
          if (env.TAG_NAME == 'production-release') {
             stage('Update production release tag') {
               newImage.push('production-release')
             }
           }

See #3386 for reference.

Done - anything else blocking?


chore(frontend): hide Remote Repository option on the Advanced search page

Checklist

[x] the contributor license agreement is signed
[x] commit message follows commit guidelines
[ ] tests are included
[x] screenshots are included showing significant UI changes
[ ] documentation is changed or added

Description of change
Hide Remote Repository option on the Advanced search page.

Hi @edalex-yinzi , on your screenshot, you were on normal search mode, but the remote search was hidden.  We only want to hide the remote search selector when we are on advanced search mode.

Sorry, I misunderstood the ticket. :disappointed: Now it has been fixed.

Normal:

Adv search:

We need to reopen this PR to target to release branch.

Oh rats, I guess you need to repoint them before deleting the branch. :disappointed:


How to Change B.C. year on title to Buddha year ?

Can I change B.C. year to Buddha year in a red circle?

@beergtx hl
Hi , you have solution right , can you explain how to convert year to bhudda please

I solve it
We can setTitleMonth with call interface format in lib


Can this library be used to connect an ESP-32 to a DMX wireless circuit board?

I'd like to connect an ESP-32 development board to a DMX wireless circuit board like this:
https://www.amazon.com/Wireless-Transceiver-Controller-Transmitter-Receiver/dp/B07GZ28DFL/ref=sr_1_73?keywords=dmx+wireless+receiver+transmitter&qid=1640913708&sprefix=dmx+w%2Caps%2C65&sr=8-73
If it can, any idea of wiring would also be helpful.
Thanks,
Steve

It looks like it is possible. To do so, you would want to attach a RS485 or DMX interface (like this one) to your ESP32, and then plug XLR pin 1, 2, and 3 into G, D-, and D+ inputs of your product, respectively. This would be the surest way to get this product working with the ESP32.
It looks like your product has a TX, RX, and Ground pin for a UART as well. You may be able to plug your ESP32 UART into those pins to transmit DMX but it isn't very clear if that is possible. It may just be a programming port for the product. If you are able to transmit DMX via straight UART, you'd likely need a logic level converter to convert the 3.3v ESP32 UART into 5v.
Any sort of wireless communication from the ESP32 to your product is not possible with this library.
Let me know if you have any other questions!

@someweisguy Thanks for the response. I really appreciate it.
I looked at the Sparkfun board, but it's overkill for what I'm looking to build. Really trying to tap into wireless DMX. Sounds like the RS485 is the way to go. Something like this?
https://www.amazon.com/ALMOCN-Adapter-Module-Converter-Indicator/dp/B08XLT21S6/ref=sr_1_9?crid=1XIQEPYN3VBN1&keywords=RS485&qid=1640989381&sprefix=rs485%2Caps%2C99&sr=8-9
In your sample code you talk about RTS: IO21. What is it used for and does it need to be attached somewhere?
I think I understand what you're saying about how it would be connected. I'll probably get some parts and give it a try. See what happens.

That will work!
The way that RS485 works (and DMX too, since DMX uses RS485) is that it sends data using the D+ and D- lines in a half duplex data bus. Half duplex means that it can only send data or receive data, but not both at the same time. Therefore, it is necessary to signal to your RS485 chip when you want to receive data and when you want to send data. That is what IO21 is used for. RTS stands for "Request To Send." Typically, you'll pull the RTS line low and that tells your RS485 chip that it should receive data, not send it. If RTS is high, it tells the RS485 chip that you are transmitting data. Often you'll see DE/RE instead of RTS - it stands for Driver Enable/Read Enable.
It looks like the product you linked doesn't have a RTS pin. It is possible that its RTS logic is automatic. There RS485 boards that will stay in "read" mode until it detects data on the TX line. When it does, it switches to "write" mode and transmits your data. I am guessing that is the case here.
Since you aren't using that pin, your setting communication pins step will look a bit different. You can call DMX_PIN_NO_CHANGE when you declare your RTS pin. Like this:
const int dmx_num = 2;
const int tx_io_num = 17, rx_io_num = 16;
dmx_set_pin(dmx_num, tx_io_num, rx_io_num, DMX_PIN_NO_CHANGE);

Let me know if you have any other questions! :)

That will work!
The way that RS485 works (and DMX too, since DMX uses RS485) is that it sends data using the D+ and D- lines in a half duplex data bus. Half duplex means that it can only send data or receive data, but not both at the same time. Therefore, it is necessary to signal to your RS485 chip when you want to receive data and when you want to send data. That is what IO21 is used for. RTS stands for "Request To Send." Typically, you'll pull the RTS line low and that tells your RS485 chip that it should receive data, not send it. If RTS is high, it tells the RS485 chip that you are transmitting data. Often you'll see DE/RE instead of RTS - it stands for Driver Enable/Read Enable.
It looks like the product you linked doesn't have a RTS pin. It is possible that its RTS logic is automatic. There RS485 boards that will stay in "read" mode until it detects data on the TX line. When it does, it switches to "write" mode and transmits your data. I am guessing that is the case here.
Since you aren't using that pin, your setting communication pins step will look a bit different. You can call DMX_PIN_NO_CHANGE when you declare your RTS pin. Like this:
const int dmx_num = 2;
const int tx_io_num = 17, rx_io_num = 16;
dmx_set_pin(dmx_num, tx_io_num, rx_io_num, DMX_PIN_NO_CHANGE);

Let me know if you have any other questions! :)


@emeralt/database-localfs

Implement database plugin that allows to store data in local filesystem

Done in a43392d, will be available in v0.7.0 ✨


Docker Compose development environment

Vagrant support was dropped in favor of Docker based development environments, but no replacement was provided. This should do.

Hello @martinblech, that sounds good.
What will I speak, it is only my opinion, and can not be the same as the rest of the team.
First, I'm still trying to understand how the Docker ecosystem works.
having said that, I have some questions:

All the structure of this project, assumes he is running on Ubuntu 14.04 Server/Desktop (Ok, it works almost unmodified on other Linux systems). This docker setting is based on what operating system?

Some libraries from Python dependencies, have C extensions that need to be compiled at installation time. The compile dependencies to build these python libraries in Ubuntu 14.04 are listed on requirements.apt file. How this would work in Docker?

If someone see this docker configuration and want to put in a production environment. From this configuration uses some images available on Docker Hub, to what extent I can trust in the security settings applied in these images (server hardening)? The Database password is auto-generated? The database is accessible outside the container?

I think some of the answers to these questions should be included in the documentation to avoid problems and wrong interpretations.
Every day that passes, I think that "Explicit is better than implicit" is really an incredible excellent idea.

Hello @luzfcb, thank you for your feedback.

It's supposed to be OS-independent, but to be honest I have only tried it in OS X via boot2docker. I wouldn't expect there to be any problems, as Docker images are supposed to be completely isolated from the host OS. Do you have a Ubuntu or Windows box at reach where you could try it?
All the dependencies build fine with pip install -r requirements/local.txt during the image build stage. It seems that the base python:3.4 image has all the compile dependencies.
Dockerfile and docker-compose.yml are only intended for development purposes. Perhaps a comment there stating so is enough?

Couldn't agree more about "explicit is better than implicit".

@luzfcb Please look at the comments I just amended to Dockerfile and docker-compose.yml.

@martinblech about 1, docker python3.4 is based on Debian Jessie
in order of dependency:

https://registry.hub.docker.com/_/python/
1.1. https://github.com/docker-library/python/blob/a467ae88510bf64dd4f35253d9a41af3b3cef7b6/3.4/Dockerfile#L1
https://registry.hub.docker.com/_/buildpack-deps/
2.1 https://github.com/docker-library/buildpack-deps/blob/44070019e0c6c4f83461f2058d110cc418da09c4/jessie/Dockerfile#L1
2.2 https://github.com/docker-library/buildpack-deps/blob/a0a59c61102e8b079d568db69368fb89421f75f2/jessie/scm/Dockerfile#L1
2.3 https://github.com/docker-library/buildpack-deps/blob/a0a59c61102e8b079d568db69368fb89421f75f2/jessie/curl/Dockerfile#L1
https://registry.hub.docker.com/_/debian/
3.1 https://github.com/tianon/docker-brew-debian/blob/b637abe5e26f8b283a28d63cdc0413926ed54596/jessie/Dockerfile#L1
https://registry.hub.docker.com/_/scratch/

Interesting see how the layers are assembled in a container docker ...

While having a development environment powered by compose is neat, I'd go one step further and add a configuration that works for dev and prod.
I'd suggest the following:

A configuration for production with four services: postgres, memcached, gunicorn and nginx as a reverse proxy. If celery support is enabled, add three more: rabbitmq, celery-worker and celery-beat.
A configuration for development with postgres and a django image that runs runserver_plus and mounts the code to the running container, so auto reloading etc. works.

As of compose 0.3.0 there's no good solution to inherit from a common configuration like cookiecutter-django does with common.py, so two files are needed.

A docker-compose.yml for production, so all is needed is to run docker-compose up -d on the prod server.
A dev.yml for development. There are two ways to run that:
Set the environment variable COMPOSE_FILE to dev.yml and run docker-compose up, or
Run docker-compose -f dev.yml up

In fact, I've ported a couple of projects that were started with cookiecutter-django to a working configuration with docker-compose.
If there's interest in that, I'd be happy to put something together over the next few days.

The extends keyword can be used for sharing common configurations between development and production .yml files. Here's an example from the Docker Compose docs: https://docs.docker.com/compose/extends/

Yes, but it has some limitations that make it pretty unusable. What we really need is a concept like include from #318 or global_extends from #1380.

I made a PR here https://github.com/pydanny/cookiecutter-django/pull/284
It is basically working like @martinblech's contribution for development with a few tweaks (env variables are set automatically in entrypoint.sh, for example) and a working configuration that can be used as a starting point to run it in production.

Closing as #284 encompasses this excellent work.


Allow disabling / remapping of display rotation hotkeys

Please allow us to disable the screen rotation hotkeys ctrl + -> and ctrl + <-
On Windows (and in my custom Linux config) the hotkey for changing virtual desktops is ctrl+windows+<left or right>, so every time I switch virtual desktops the display rotates. This is super aggravating!

Could you test #1598 (future v1.15), please? The modifier key is changed (and can be changed).

Could you test #1598 (future v1.15), please? The modifier key is changed (and can be changed).

Fantastic! I'll test it today.

Tested #1598 today, works exactly as I'd like it to. Thanks for your work! I'll close this issue.


Allow copying files from sandbox to host

This PR adds a CopyToHost function to the sandbox.Sandbox interface which allows copying files from the sandbox container to the host, after the sandbox has run. This will be used in a future PR to save logs of executed functions during the dynamic analysis import phase.
The current copy struct is slightly tweaked to allow dual use, and renamed to copySpec to avoid name shadowing.
Also fixed an apparent bug with the existing copy implementation where multiple copies are specified - podman cp only supports one copy at once.

Nice addition and good catch on needing to call podmanRun on each cp call individually. If you ran any sort of test to catch that it could be good to write that into a test case as well.

I just noticed it when I was reading the code... but agree that having some automated tests to catch this kind of thing would be good. We should add end to end tests into the CI pipeline, see #637


How to fix "too many indices for tensor of dimension 4"?

I tried cogvideox_I2V_example_01.json, and the image is 720X480, but...
CogVideoImageEncode Node got this error :"too many indices for tensor of dimension 4"

Finally, turn off "enable_tiling" in CogVideo ImageEncode node solved my problem.

I have the same error, unchecking enable_tiling didn't help :/

This is possibly because of the fix for the tiled_encode for diffusers 0.31.0, maybe it broke it for the old ones, works for me on the latest version of the nodes and diffusers though.

I activated the tiled decoding and it worked (I also changed other settings so it may be another one, anyway now it's working for me too).

Still not working for me after disabling tiling, any other ideas??

Still not working for me after disabling tiling, any other ideas??

With the nodes and diffusers both at latest version?

The custom nodes and comfy are the latest. How do I check the diffusers?

Depends on how your comfy is setup, you'd do pip show diffusers:
For portable:

And to update:
python_embeded\python.exe -m pip install -U diffusers
Anyway I updated the nodes now with possible backwards compatibility version, so that might work for you as well.

Thank you it seems to work now, probably due to your latest update. 👍👍

Also had the issue but fixed now!

The custom nodes and comfy are the latest. How do I check the diffusers?

Depends on how your comfy is setup, you'd do pip show diffusers:
For portable:
And to update:
python_embeded\python.exe -m pip install -U diffusers
Anyway I updated the nodes now with possible backwards compatibility version, so that might work for you as well.

It turns out I have diffusers 0.30.0
When I try to update it with the command you gave me, I get a window saying " This app cant run on your pc" to find a version for your PC, check the software publisher. Is it a problem with Python? or something else?


Square bracket for group aggregation

See #395

With the last commits I added a kind-of square bracket also to individual items:

If the majority doesn't like it, we can easly remove it again.


Seven Days to Die Undead Legacy Overhaul Templates

This Template is a modified version of @IceOfWraith's Seven Days To Die which makes it possible to run the Undead Legacy Overhaul Mod.
seven-days-to-die-undead-legacy.kvp

Added App.EnvironmentVariables for BepInEx to work for the Undead Legacy Overhaul

seven-days-to-die-undead-legacyconfig.json

Added config options for Undead Legacy Overhaul in their own category.
Added Booleans for stable and experimental installs.
Fixed the typo of Job to Jog for zombie speeds.
Added stable and experimental server setting config options.
Removed older server settings config options as they won't work with Undead Legacy
Changed EAC default to false as Undead Legacy will not run with it on.

seven-days-to-die-undead-legacymetaconfig.json

Added new Undead Legacy stable and experimental configs.
Removed configs older than 20.4.

seven-days-to-die-undead-legacyupdates.json

Added download, extract, and copy for Undead Legacy stable and experimental overhaul mods
Added server setting download for stable and experimental
Removed configs older than 20.4

seven-days-settings-undead-legacy-stable2582.xml

Added Undead Legacy Stable settings for server settings config.

RecipeFIlter
StarterQuestENabled
WanderingHordeFrequency
WanderingHorgeRange
WanderingHordeEnemyCount
WanderingHordeEnemyRange

seven-days-settings-undead-legacy-experimental2610.xml

Added Undead Legacy Experimental settings includes stable settings also.

DeathPenalty
POITierLootScale

Tested and working on Linux AND Windows.

I think I figured out how to merge this into the main template. Let me take a look this weekend and share with you how it looks.

sounds good I reopened this in the right way following the amp wiki here #224


An error occurred in the SDWebImageImageIOCoder

Info
Value

Platform Name
ios

Platform Version
8.3 or 9.3.2

SDWebImage Version
4.4.1

Integration Method
cocoapods

Xcode Version
Xcode 9

Repro rate
sometimes

Repro with our demo prj
No, it happen with our own app

Issue Description and Steps
The following is the infomation of the crash.

vImage
0x00000001815a0000 + 169520

1 vImage
vImagePermuteChannelsWithMaskedInsert_ARGB8888 + 304

2 vImage
vImagePermuteChannelsWithMaskedInsert_ARGB8888 + 304

3 CoreGraphics
_CGSConvertBGRX8888toRGBA8888 + 160

4 CoreGraphics
_argb32_image + 3232

5 libRIP.A.dylib
_ripl_Mark + 32

6 libRIP.A.dylib
_RIPLayerBltImage + 928

7 libRIP.A.dylib
_ripc_RenderImage + 260

8 libRIP.A.dylib
_ripc_DrawImage + 688

9 CoreGraphics
CGContextDrawImage + 408

10 Y
-[SDWebImageImageIOCoder sd_decompressedImageWithImage:] (SDWebImageImageIOCoder.m:233)

11 Y
-[SDWebImageImageIOCoder decompressedImageWithImage:data:options:] (SDWebImageImageIOCoder.m:180)

12 Y
-[SDWebImageCodersManager decompressedImageWithImage:data:options:] (SDWebImageCodersManager.m:124)

13 Y
__70-[SDWebImageDownloaderOperation URLSession:task:didCompleteWithError:]_block_invoke.279 (SDWebImageDownloaderOperation.m:0)

14 libdispatch.dylib
__dispatch_call_block_and_release + 24

15 libdispatch.dylib
__dispatch_client_callout + 16

16 libdispatch.dylib
__dispatch_queue_drain + 1448

17 libdispatch.dylib
__dispatch_queue_invoke + 132

18 libdispatch.dylib
__dispatch_root_queue_drain + 720

19 libdispatch.dylib
__dispatch_worker_thread3 + 108

20 libsystem_pthread.dylib
_pthread_wqthread + 816

Not often, but there are also probabilities.

It seems a OOM issue, during CGContextDrawImage.
If you're facing this issue, on low-memory device (such like iPhone 4S/iPhone 5). I suggest to firstlly disable the decompress feature.
SDImageCache.sharedImageCache.config.shouldDecompressImage = NO;
SDWebImageDownloader.sharedDownloader.shouldDecompressImages = NO;

I turned on the image hard-coder to play the gif in the UIImageView, which seems to be because by this reason.i don't want to use the FLAnimatedImage frameworks,because it will replace UIImageView in project

@yunjinghui123 please see similar issues marked with OOM label
Basically, if you are decoding big images, the app will run out of memory. You need to stop de decompression like instructed above, or use our options for scaling down large images to smaller ones.

Thank you very much, I will take your suggestions to optimize the code.


Store Pangeo platform configurations and how to install on EOSC in this repository

I already mentioned that in https://github.com/pangeo-data/pangeo-eosc/issues/5#issuecomment-1217817626.
I think we should at least be storing here the helm values.yaml file. Possibly in two separated files to keep secrets elsewhere. Thereis probably a way to store secrets on github via encrypted files. I used ansible vaults previously, this is usually linked to ansible, and there is probably alternatives.
We should also add some documentation on how to install the full stack. We could start from @sebastian-luna-valero https://github.com/sebastian-luna-valero/pangeo-eosc/blob/egi/EGI.md.

Happy to work on this on the next few days.
On Thu, 18 Aug 2022, 18:29 Guillaume Eynard-Bontemps, <
@.***> wrote:

I already mentioned that in #5 (comment)
https://github.com/pangeo-data/pangeo-eosc/issues/5#issuecomment-1217817626
.
I think we should at least be storing here the helm values.yaml file.
Possibly in two separated files to keep secrets elsewhere. Thereis probably
a way to store secrets on github via encrypted files. I used ansible vaults
previously, this is usually linked to ansible, and there is probably
alternatives.
We should also add some documentation on how to install the full stack. We
could start from @sebastian-luna-valero
https://github.com/sebastian-luna-valero
https://github.com/sebastian-luna-valero/pangeo-eosc/blob/egi/EGI.md.
—
Reply to this email directly, view it on GitHub
https://github.com/pangeo-data/pangeo-eosc/issues/7, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABIZB3MLKGFHKK6UDTKCN63VZZQHTANCNFSM565ZM23Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

That would be really nice @sebastian-luna-valero! Thanks.

I have worked with Ansible vault previously as well but I can't think of how to apply it here easily. I am not sure about the method to securely share/configure secrets in this context and due to the time limitation for the workshop I would just use the Copy functionality in Markdown document on GitHub for the time being and create a new issue to work on this for future editions of the workshop or the infrastructure.

That's fine by me @sebastian-luna-valero. Let's go this way for the time being.


add pranesh6876

Done

@pranesh6876 I am sorry, but this is not part of hacktoberfest anymore. Sorry for the issues.


Compatibility with react-bootstrap 0.29.x (deprecation of )

React bootstrap has some changes in 0.29 which means Input is being deprecated.
The console outputs these warnings:
Warning: `<Input>` is deprecated. Use `<FormControl>`, `<Checkbox>`, or `<Radio>`, with `<FormGroup>` and/or `<InputGroup>` as needed instead.
Use a `<FormControl.Feedback>` element.```

The library still works, just with plenty of warnings in the console, but this will need to be addressed for when `<Input>` is eventually removed.

I'm working on a new version that will have a lot of improvements and support all updated dependencies including react-bootstrap. No ETA though.

Cool, that's good to hear.

Hello, do you have any progress with the new version? Thanks!

@heilhead Hello, do you have any progress with the new version? Thanks!

Well, react-bootstrap 0.30.0 now breaks compatibility with react-bootstrap-validation. Can confirm #67 works.


The dev map didn't who Candy now.

Expected Behavior
Candy for each type of pokemon.
Actual Behavior
Candy: Undifined
Steps to Reproduce
I have also tried to use some old but well functioned map submodule to replace the current one, But the issue is still there.
Run it and see :D
Other Information
OS: Win 10
Git Commit: 83463c2bcd5c4360adbc41c61e6572406fec54ce

Have you tried
git submodule init
git submodule update

https://github.com/OpenPoGo/OpenPoGoWeb/pull/98


Public Release for Fall24 meta-release r1.3

What type of PR is this?
Add one of the following kinds:

correction
subproject management

What this PR does / why we need it:
Preparation for public release r1.3 designated to correct an error detected in r1.2

hi @hdamker take a look to check if it makes sense to you. The idea is to keep in the docs why the r1.2 has been revoked and then remove it from the repo. Keeping onbly the r1.3.
do we need to include @camaraproject/release-management_maintainers ?

@fernandopradocabrillo looks good to me. I would like to have a short review by @tanjadegroot here as well.
@tanjadegroot: context is the discussion within https://github.com/camaraproject/KnowYourCustomer/issues/112 how to handle this case where kyc-match v0.2.0 in r1.2 can't be used (and the meta-release was not yet announced).

@fernandopradocabrillo to be on the safe side I have created a review issue in release management. Please bear with the team and wait for the approval from @camaraproject/release-management_maintainers

Thank you very much @hdamker!


Shift+Enter to create a new line while commenting

Issue
(Staging) The user can't start a new line while writing a new comment. Both Enter or Shift+Enter are considered as sending the comment (The user is able to press Enter to begin the new line while editing the existing comment though.)

Please check the video in this post

Steps to reproduce

Access any post as a logged-in user, write a comment but not press the icon on the right to send it yet
Try press "Enter", you will see the comment is sent
Try writing a new comment and then press "Shift+Enter", the comment is sent anyway

Expected behavior
The user should be able to start a new line while commenting by using "Shift+Enter"

Thanks @pemika-me
Thought - maybe you should always need to hit button to post the comment? Looking at other sites it seems only with messaging that enter submits (with shift+enter adding new line).
But if we stick with enter->post then we do at least need shift+enter. Maybe even provide a note? in Slack:

Such a good point! Thank you @joshmorel for your great perspective.
Dear PM @doheekim93 @dhruvdpatel @ppagarawal
May you consider Josh's comment and give us a decision?

By the way, I think just require the user to hit the button to post a comment is the simplest one - as it's more reasonable for the user who accesses our website on Mobile web. (I have never pressed Shift :arrow_up: + Enter button to create a new line on my mobile keyboard) So both Desktop web and Mobile web can work with the same logic. (Enter to create a new line, and only send/post a comment when the send icon is hit)

Typically, settings provide an option to decide what the enter should do. Here, I recommend that
Desktop: Enter must correspond to post action. Shift + Enter can be used for new line. (Think of UX here for using 1 device here instead of 2 devices to post)
Mobile: use the button to post. Pressing shift must toggle the send button to change to enter a new line.

@pemika-me so finally what should be the result o pressing Enter and Shift+Enter?

@AshBha243 Please refer to @ppagarawal comment above.

Hi team, I am the ticket's author and just decided to leave FP. Please contact PM or engineer lead in case you would need some info or review related to this ticket. Thanks. 😊

Thanks @pemika-me ! I'll take ownership of this one

Closing in favor of #1672


Delta table processing using astro-sdk

Please describe the feature you'd like to see
Databricks delta tables are being used heavily in MDM(Master Data Management) and CDC(Change Data Capture) projects.
I was thinking if there is a way in astro-sdk to submit query directly to Databricks delta table.Please refer to https://docs.databricks.com/delta/delta-intro.html
Describe the solution you'd like
Since the astro-sdk has support for parquet , it should be possible to execute a  CONVERT TO DELTA command on a parquet file, on which users can execute DML and DDL statements. But I was wondering whether we can submit spark jobs to Databricks cluster using aql ?
Are there any alternatives to this feature?
Submit a spark job using Livy Operator or the native SparkSubmitOperator on databricks
Additional context
Add any other context about the feature request here.
Acceptance Criteria

[ ] All checks and tests in the CI should pass
[ ] Unit tests (90% code coverage or more, once available)
[ ] Integration tests (if the feature relates to a new database or external service)
[ ] Example DAG
[ ] Docstrings in reStructuredText for each of  methods, classes, functions and module-level attributes (including Example DAG on how it should be used)
[ ] Exception handling in case of errors
[ ] Logging (are we exposing useful information to the user? e.g. source and destination)
[ ] Improve the documentation (README, Sphinx, and any other relevant)
[ ] How to use Guide for the feature (example)

Addressed as part of the below issues
#1328
#1353
#1249
#1248
#1450


getOption doesn't get option on newly initialized tom-select object

Hi there,
lately i have spotted a bug that my tests revealed. On newly initialized object, getOption method doesn't get an option, gets null. Only way i can get it is to refreshOptions(false) before.
https://jsfiddle.net/mArtinko5MB/dukmpx3t/9/
No idea if that is expected behavior, but it seems to me it's unnecessary to call refreshOptions before doing anything with the select.

I have come across this before and I'm kind of 50/50 on where to go with it.
getOption(value) returns the dom element that's used to display an option in the Tom Select dropdown. These dom elements are created only if they're needed. In a situation where there are thousands of options, dom elements may never be created for many of the options.
A solution may be to add a create_option argument so the function signature would look like this: getOption(value, create_option). Calling getOption('a',true) would create the dom element for 'a' if it doesn't already exist.

getOption('a',true) can now be used. docs


initial setting

[ ] install release-please-action
https://github.com/google-github-actions/release-please-action
[ ] install conventional commit lint
https://pub.dev/packages/commitlint_cli (flutter)
https://github.com/conventional-changelog/commitlint (others)
[ ] security setting

git hook 이란
https://sujin-park.github.io/web/git-prepare-commit-msg/


Clarification on style for functions with very long return types

Might be useful to include some guidance on how to deal with function return types that don't fit on a single line within the recommended line length limit, e.g.:
fn foo(
    &self,
    x: i32,
    y: i32
) -> fn(a: SomeLongComplexType, b: SomeOtherLongComplexType) -> Box<Future<Item = AnotherLongType, Error = ALongErrorType>> {
    ...
}

E.g. is it preferred to keep on a single line as in the example above? Or should it be split along similar lines to the outer function, i.e.
fn foo(
    &self,
    x: i32,
    y: i32
) -> fn(
    a: SomeLongComplexType,
    b: SomeOtherLongComplexType
) -> Box<Future<Item = AnotherLongType, Error = ALongErrorType>> {
    ...
}

Ouch. Good example, and that seems painful for several reasons, not least of which making it easy to confuse the two function-syntax types and the return type.
As far as I can tell, we don't have any guidance for when the return type is too long to fit on one line.
I would personally be inclined to write that this way:
fn foo(
    &self,
    x: i32,
    y: i32
) ->
    fn(
        a: SomeLongComplexType,
        b: SomeOtherLongComplexType
    ) -> Box<Future<Item = AnotherLongType, Error = ALongErrorType>>
{
    ...
}

I also don't mind writing the fn( on the same line as the ) ->, but I'd prefer to add extra indentation on the return-type function-type arguments and on its return type, and then not put the { on the same line.


waffle.io Badge

Merge this to receive a badge indicating the number of issues in the ready column on your waffle.io board at https://waffle.io/TWCable/grabbit
This was requested by a real person (user sagarsane) on waffle.io, we're not trying to spam you.

Added manually.


updated to use prometheus/client_golang v1.0.0

The latest github.com/prometheus/client_golang v1.0.0 changed the API

Thanks @tmelhiser


Document update for removed elements in Flow Exporter

This commit updated flow visibility documentation for removed fields originalExporterIPv4Address and originalObservationDomainId.
fixes #2398

Codecov Report

Merging #2401 (fba2897) into main (cc8e7e3) will decrease coverage by 17.68%.
The diff coverage is n/a.

:exclamation: Current head fba2897 differs from pull request most recent head 280beb7. Consider uploading reports for the commit 280beb7 to get more accurate results

@@             Coverage Diff             @@
##             main    #2401       +/-   ##
===========================================
- Coverage   59.64%   41.96%   -17.69%
===========================================
  Files         284      146      -138
  Lines       22170    18122     -4048
===========================================
- Hits        13224     7605     -5619
- Misses       7534     9831     +2297
+ Partials     1412      686      -726

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
41.96% <ø> (-0.09%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.


adsr hold calculation incorrect?

In the adsr function, the remainder for the hold is calculated by subtracting the attack, decay, and sustain from the duration. I think this should be attack, decay, and release instead, as those are the adsr time parameters, and sustain is a level parameter.
https://github.com/ctford/cljs-bach/blob/master/src/cljs_bach/synthesis.cljs#L92

Hmm, you're right, that does look wrong.