Configurable flush interval

In https://github.com/logstash-plugins/logstash-codec-multiline/pull/13, we added the evict_timeout for flushing to flush the last data collected if there is no need new data appended to a file.  However, this is hardcoded to 1 hr right now.  We have requirements in the field for near real-time.  Can we expose this as a configuration (and maybe cap the minimum flush interval to something sensible)?

Maybe a duplicate or related to #11.
This is kind of doable.
Please note the following:

The Multiline codec when used in an input is incapable of flushing out-of-band events into the pipeline because of design problems with the pipeline <-> input interaction.
Eviction stuff was added to the IdentityMapCodec object.
The IdentityMapCodec is not a real codec, it does not subclass LogStash::Codecs::Base and therefore does not have config options. It was designed as a transparent wrapper to sit between an input and ANY codec where discreet 'streams' of data from an input needs separation.
It exists in this repo as a convenience because multiline is the first and obvious codec that would benefit.  It will probably move.
The eviction and flush was added because flush is part of the public API of a codec.
The default settings were chosen after some debate and can be set programatically by the input code that chooses to use IdentityMapCodec to wrap its codec.

In order to do this we could use a config option from the input. This makes sense because A) of the point above and B) the input understands the context of the data being ingested better than the codec.
It is, of course, possible to use a config option from the codec but then it needs be one that exists in all codecs and therefore defined in LogStash::Codecs::Base. I did not do this because I wanted this initial release to be independent of a LS core version until we ironed out any problems.
@suyograo, @ph, @colinsurprenant - WDYT?

There is another problem related to the flush, the current implementation the identity map memorize the block on the first decode block and use the same one when the tick is called.
This should work normally in the majority of the plugins but in the context of beats this could be problematic, since the queue can raise an exception and kill the eviction thread.
Maybe we should add a way to define a custom eviction method?

I am wondering maybe we can change how the eviction work, I see it as two problems.
A) Multiline: Should I flush the content of my buffer based on a max_ages?
B) IdentityMap: Should I remove unused codecs from my hash?
If we define the unused codec concept from B by if the codec doesn't have any buffered data we can say its not in use and I can free it. This make the codec responsable to flush itself on a periodic calls.
Also in this scenario we need to be able to define the call back that will receive the tick.

@ph - re: pushing events to the queue raising exceptions, do you suspect this or is the raised exception by design?
Either way I need to protect my Cleaner thread.

@guyboertje its by design and it only affect the beats/lumberjack code.
Let me give you a bit of background:
The problem is in the way the lumberjack protocol works and how the clients are designed, if the server doesn't send an ack to the client, the client will think something went wrong and try to reconnect and send his packets again (He has a connection timeout).
The server was stuck because the queue was blocked and the thread wasn't released. The client kept connecting to the server creating new threads and eventually made logstash OOM.
The current design of the queue doesn't have a concept of a timeout on push, this problem is mostly relevant to the TCP family of the inputs.

Timeout on push can be done with two threads and a SizedQueue of 1. The caller starts a timeout thread and a worker thread both with access to the SizedQueueOfOne and then waits on the SizedQueueOfOne. The worker thread pushes the event (maybe waits for access) and when done puts something in the SizedQueueOfOne. The timeout thread sleeps for Timeout and then puts something in the SizedQueueOfOne. Whichever thread completes first frees the calling thread - bingo, exception free timeouts.  If the worker thread dies the timeout thread will still free the calling thread.

@guyboertje I would like to sync on that, not sure I understand all your description.

@ppf2 - What do you consider a 'real-time' minimum setting to be?

@guyboertje I guess it depends on the performance implications of the flush :)  If its pretty lightweight, we can probably set the limit to something that is no more frequent than 1s or 5s, etc..

Fixed in #18


Mapping numbers are overlapped if atoms are close to each other

Steps to Reproduce

Launch Ketcher
Open
mapping_4_benzene.zip
Select Reaction Auto-Mapping tool
Choose any mode (Discard, Keep, Alter) and press OK

Expected behavior
Atoms are mapped. Numbers are displayed clearly
Actual behavior
If atoms are close to each other, then number are overlapped and not visible.
Screenshots

Desktop :

OS: Windows 10

@AndreiMazol  @r1z3rISGOD , in this case (see video) it's wrong working and Mapping numbers should be displayed clearly?

https://user-images.githubusercontent.com/90637536/133615515-e3d5e426-7b95-4e7c-a58d-9c6c94837542.mp4


http.client.InvalidURL: URL can't contain control characters

Hello,
I am having an error while using : python3 download_ko.py --org test.org --out KEGG-KO --concurrent 10
"Traceback (most recent call last):
File "/usr/lib/python3.6/multiprocessing/pool.py", line 119, in worker
result = (True, func(*args, **kwds))
File "download_ko.py", line 37, in download
file = urllib.request.urlopen("http://www.kegg.jp/kegg-bin/download_htext?htext=%s&format=htext&filedir=" % id)
File "/usr/lib/python3.6/urllib/request.py", line 223, in urlopen
return opener.open(url, data, timeout)
File "/usr/lib/python3.6/urllib/request.py", line 526, in open
response = self._open(req, data)
File "/usr/lib/python3.6/urllib/request.py", line 544, in _open
'_open', req)
File "/usr/lib/python3.6/urllib/request.py", line 504, in _call_chain
result = func(*args)
File "/usr/lib/python3.6/urllib/request.py", line 1353, in http_open
return self.do_open(http.client.HTTPConnection, req)
File "/usr/lib/python3.6/urllib/request.py", line 1325, in do_open
encode_chunked=req.has_header('Transfer-encoding'))
File "/usr/lib/python3.6/http/client.py", line 1285, in request
self._send_request(method, url, body, headers, encode_chunked)
File "/usr/lib/python3.6/http/client.py", line 1296, in _send_request
self.putrequest(method, url, **skips)
File "/usr/lib/python3.6/http/client.py", line 1139, in putrequest
raise InvalidURL(f"URL can't contain control characters. {url!r} "
http.client.InvalidURL: URL can't contain control characters. '/kegg-bin/download_htext?htext=aaa00001.keg&format=htext&filedir=' (found at least ' ')
"""
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "download_ko.py", line 114, in
main()
File "download_ko.py", line 110, in main
download_ko([i[0] for i in orgs], args.out, args.concurrent)
File "download_ko.py", line 69, in download_ko
returns.append(result.get())
File "/usr/lib/python3.6/multiprocessing/pool.py", line 644, in get
raise self._value
http.client.InvalidURL: URL can't contain control characters. '/kegg-bin/download_htext?htext=aaa00001.keg&format=htext&filedir=' (found at least ' ')"
Any idea how to resolve that?
Thank you

Hi. I think your issue relates to the repo https://github.com/FlyPythons/KEGGTools.


Re supply & NPCS

Can you allow us to set the models of the npc's? And while using M9K no ammo works from the re supply station. Sorry ;)

NPC's are created here: https://github.com/LouisK130/Orange-Cosmos-Roleplay/blob/master/ocrp/gamemode/shared.lua#L600-L631
Using this function: https://github.com/LouisK130/Orange-Cosmos-Roleplay/blob/master/ocrp/gamemode/server/npcs.lua#L8-L63

Can you please add an option for a model?

The 8th parameter to that function is the model to be used.


Value cannot be null when re-submitting multiple messages at once (batch)

Using version 5.0.3 now (but was also an issue before on 5.0.2), I can't select multiple messages from the list of Dead-letter messages and right-click to resubmit them all at once.

The log gives me the following error:
<09:55:00> Exception: En værdi må ikke være null.
Parameternavn: The BrokeredMessage parameter cannot be null.

Which translates into
Exception: A value cannot be null.
Parameter name: The BrokeredMessage parameter cannot be null.

If I choose only a single message it works as expected, but this is tedious when having a lot of messages in the DLQ. It used to work before upgrading to version 5.0.2 (don't remember what version I was running before as my hard drive crashed back in February)

@CasperWSchmidt, I was unable to reproduce that on a DLQ to a queue. Are the bodies or any of the properties empty? I tried with an empty label since you have that in your snippet, but that also worked for me.
I guess that this has something to do with the message properties, but may of course be something else like the queue properties. Can you try different combinations and see if you can find a configuration that works? Then we could determine what is wrong. I generated my messages using the built-in functionality in Service Bus Explorer.

@ErikMogensen All our messages have empty message bodies (we use Service Bus Attachment plugin created by @SeanFeldman), but not empty properties. Please not that moving a single message from DLQ to a queue is not a problem.
This has worked fine in production for over a year (both moving a single item as well as batching), but suddenly something changed with a new version of Service Bus Explorer :( - On the positive side of things, the new version of Service Bus Explorer automatically selects the right topic to move to from DLQ, which was a bit of a pain in earlier versions :)
Example:

Just an FYI, @CasperWSchmidt, soon you won't need the plugin as ASB will support up to 100MB message size.

@SeanFeldman how soon, is soon? Or put another way: Where can I read about that new size limit?

@esbenbach I don't work for Microsoft to be able to answer this question but it's already in preview.

Nah i know your not from MS - just figured you apparently had some info my
google fu couldnt dig up. And knowing there is a preview did not help my
google fu unfortunately. Thx for the info though.
On Wed, 2 Jun 2021 at 00:42, Sean Feldman @.***> wrote:

@esbenbach https://github.com/esbenbach I don't work for Microsoft to
be able to answer this question but it's already in preview.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/paolosalvatori/ServiceBusExplorer/issues/547#issuecomment-852532890,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKQP7QTTZKUY74M2B4Z5ETTQVO35ANCNFSM45ECRVEA
.

--
Esben Bach
cand.scient

@esbenbach my Google fu found this https://docs.microsoft.com/en-us/answers/questions/363726/increasing-the-message-size-of-the-service-bus.html 😊 so the new limit is only for premium 😒

During the Integrate remote conference today Clemens Vasters said that 100 MB is in public preview and the plan is that it will GA this year.


add error information for when sonatype fails

This adds the info so e.getMessage() when used which is printed when using --stacktrace.  Also, we log the same thing as well for those not using --stacktrace as this is a pretty important failure and one needs to know the issue that occurred from the remote end.

The result after using my local install of this plugin is something like this...
Caused by: org.gradle.api.UncheckedIOException: org.apache.http.client.HttpResponseException: 500:Server Error  body=[errors:[[id:, msg:Unhandled: Staging repository is already transitioning: orgwebpieces-1084]]]
at org.gradle.internal.UncheckedException.throwAsUncheckedException(UncheckedException.java:43)
at org.gradle.internal.reflect.JavaMethod.invoke(JavaMethod.java:78)
at org.gradle.api.internal.project.taskfactory.AnnotationProcessingTaskFactory$StandardTaskAction.doExecute(AnnotationProcessingTaskFactory.java:228)
at org.gradle.api.internal.project.taskfactory.AnnotationProcessingTaskFactory$StandardTaskAction.execute(AnnotationProcessingTaskFactory.java:221)
at org.gradle.api.internal.project.taskfactory.AnnotationProcessingTaskFactory$StandardTaskAction.execute(AnnotationProcessingTaskFactory.java:210)
at org.gradle.api.internal.AbstractTask$TaskActionWrapper.execute(AbstractTask.java:621)
at org.gradle.api.internal.AbstractTask$TaskActionWrapper.execute(AbstractTask.java:604)
at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeAction(ExecuteActionsTaskExecuter.java:80)
at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeActions(ExecuteActionsTaskExecuter.java:61)
... 68 more
Caused by: org.apache.http.client.HttpResponseException: 500:Server Error  body=[errors:[[id:, msg:Unhandled: Staging repository is already transitioning: orgwebpieces-1084]]]
at io.codearte.gradle.nexus.infra.SimplifiedHttpJsonRestClient.post(SimplifiedHttpJsonRestClient.groovy:67)
at io.codearte.gradle.nexus.logic.RepositoryPromoter.promoteRepositoryWithIdAndStagingProfileId(RepositoryPromoter.groovy:15)
at io.codearte.gradle.nexus.PromoteRepositoryTask.doAction(PromoteRepositoryTask.groovy:23)
at org.gradle.internal.reflect.JavaMethod.invoke(JavaMethod.java:75)
... 75 more

And you can also squash those commits into one.

@deanhiller Can you fix the things I mentioned in the code review to make this PR being merged?

I reworked it your changes and merged them manually.


Send MIME messages with SendGrid

Description of the change:
MIME messages can be sent, in more detail, using SendGrid.
Link to original source:

Closes #4233

hi @rahulpuroht - thanks for this PR! this is breaking our build because it's missing any of the md headmatter. For a good example of the headmatter, see this page: https://github.com/sendgrid/docs/edit/develop/content/docs/for-developers/parsing-email/inbound-email.md
Also, please make sure you're following our styleguide for both of your PRs

Done with head-matter .Please update if required any other actions.Thanks

Hello @rahulpuroht,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX

@rahulpuroht - thanks! I've given you the hacktoberfest credit, and I plan to do another in-depth review on this later today

Hello @rahulpuroht,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


HDFS-16356. JournalNode short name missmatch

https://issues.apache.org/jira/browse/HDFS-16356

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 53s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
34m 48s

trunk passed

+1 :green_heart:
compile
1m 26s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
compile
1m 17s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
checkstyle
1m  1s

trunk passed

+1 :green_heart:
mvnsite
1m 28s

trunk passed

+1 :green_heart:
javadoc
1m  0s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
1m 25s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
3m 18s

trunk passed

+1 :green_heart:
shadedclient
25m 15s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

-1 :x:
mvninstall
0m 49s
/patch-mvninstall-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch failed.

-1 :x:
compile
0m 54s
/patch-compile-hadoop-hdfs-project_hadoop-hdfs-jdkUbuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04.txt
hadoop-hdfs in the patch failed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04.

-1 :x:
javac
0m 54s
/patch-compile-hadoop-hdfs-project_hadoop-hdfs-jdkUbuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04.txt
hadoop-hdfs in the patch failed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04.

-1 :x:
compile
0m 49s
/patch-compile-hadoop-hdfs-project_hadoop-hdfs-jdkPrivateBuild-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10.txt
hadoop-hdfs in the patch failed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10.

-1 :x:
javac
0m 49s
/patch-compile-hadoop-hdfs-project_hadoop-hdfs-jdkPrivateBuild-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10.txt
hadoop-hdfs in the patch failed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10.

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 53s

the patch passed

-1 :x:
mvnsite
0m 50s
/patch-mvnsite-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch failed.

+1 :green_heart:
javadoc
0m 52s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
1m 21s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

-1 :x:
spotbugs
0m 48s
/patch-spotbugs-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch failed.

-1 :x:
shadedclient
13m 13s

patch has errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
0m 50s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch failed.

+1 :green_heart:
asflicense
0m 27s

The patch does not generate ASF License warnings.

87m 47s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3726/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3726

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 8aeb743276b6 4.15.0-147-generic #151-Ubuntu SMP Fri Jun 18 19:21:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / e750aebbdcbe87745faad2b8b91e5863db2cae77

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3726/1/testReport/

Max. process+thread count
517 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3726/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

I have the same situation. Is it have  missing settings some configuration items？


[PDI-19769] Select Value step ignores specified timezones when converting from TimeStamp to Integer

…

:white_check_mark: Build finished in 43m 39s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core

:ok_hand: All tests passed!
Tests run: 5321, Failures: 0, Skipped: 14    Test Results

:information_source:  This is an automatic message


Wrong value reported

Describe Your Problem:
Hi the purple air sensor 34143 is reflected with the wrong reading:
Website: 10 Minute Average US EPA PM2.5 AQI is now 102
Home: pm2.5 49ugm3 (air quality excellent)
Config:
**Logs:
Plugin Config:
accessories": [
{
"name": "BPanjang Air",
"sensor": 34143,
"conversion": "None",
"averages": "10m",
"aqiInsteadOfDensity": true,
"verboseLogging": false,
"accessory": "PurpleAirSensor"
},
Environment:

Plugin Version:1.6.2
Homebridge Version: 1.1.7
Node.js Version: v15.4.0
NPM Version: v7.0.15
Operating System:macOS

@deeloke looks like you were using US EPA conversion on the PurpleAir website, but in your plugin config you specified None, you should set it to EPA if you want the US EPA conversion.

OK, I have tried conversion EPA. When I configure Homebridge this way:

Conversions help accomodate different types of pollution with different particle densities: EPA
Reading Averages (default = realtime): 10 minute average

The Purple Air website reports:

10 Minute Average US EPA PM2.5 AQI is now 22

Homebridge reports:

PM2.5 Density 7

I am used to reading the EPA PM2.5 AQI standard. Is it possible to have this standard reported in homebridge/ Apple Homekit?
Kind regards,
-dee

Set aqiInsteadOfDensity to true?

Already is, here is my config
{
"name": "BPanjang Air",
"sensor": 34143,
"conversion": "EPA",
"averages": "10m",
"aqiInsteadOfDensity": true,
"verboseLogging": false,
"accessory": "PurpleAirSensor"
},

@deeloke it's been a while, are you still seeing discrepancy issues or can we resolve this? Thanks


Add datafusion to the DuckDB flavored h2o.ai benchmark

Is your feature request related to a problem or challenge?
DuckDB has revived the h20.ai benchmark: https://duckdb.org/2023/04/14/h2oai.html
Previously we have done various versions of the h20.ai benchmarks (e.g. https://github.com/apache/arrow-datafusion/issues/147) -- see full list here
@kszlim has requested that DataFusion be added to the new incarnation of the benchmark
https://github.com/duckdblabs/db-benchmark/issues/5
Describe the solution you'd like
Help add datafusion to the benchmark
Describe alternatives you've considered
No response
Additional context
No response

https://github.com/duckdblabs/db-benchmark/pull/18

Closed by https://github.com/duckdblabs/db-benchmark/pull/18


Changed AttachmentLayouts to AttachmentLayoutTypes

The types of Attachment Layouts class is called AttachmentLayoutTypes and not AttachmentLayouts.
https://docs.botframework.com/en-us/csharp/builder/sdkreference/d7/d87/class_microsoft_1_1_bot_1_1_connector_1_1_attachment_layout_types.html

@peterkaplan,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

+1

Thanks for your contribution!


No maven repository available anymore

Hello there,
we're using this library and were also using this maven.proleap.io repository. As the latest commit suggests (0a9a7e1) and as we see it, this repository is no longer available. Are there any plans to make this library available on the official maven repositories? Or is there another way to use this library with maven, besides building it from sources by ourselves and hosting it in our own maven repository?
Thank you for your support!

Hi Simon,
correct, we had to switch our hosting to a cloud provider and in that context lost FTP support, which is required for deployment with Apache Maven Wagon FTP. Due to time constraints we could not implement another solution such as e.g. Maven Central Repository, yet.
So right now, building from sources and hosting in your company maven repository would be the best solution, until we develop a solution on our side.
Besides: We feel honored that ThoughtWorks is using ProLeap COBOL parser. If there is interest in additional features or dialects for the parser, or if there are ideas for further development and applications, please let us know!
Best regards
Ulrich

Hi Ulrich,
thank you for your quick response!
Okay, so then we'll use our own maven repository for now. If you need any support in setting up the Maven Central solution, I'm happy to assist.
To your besides note, thank you very much for offering support, but currently we're more or less just playing around with this :D

Simon


[http-server-csharp] Optional parameters are not properly ordered in generated code

We have this snippet generated for [the contoso project](https://github.com/Azure/azure-rest-api-specs/blob/main/specification/contosowidgetmanager/Contoso.WidgetManager/main.tsp#L50):

        ///<summary>
        /// Creates or updates a Widget asynchronously.
        ///</summary>
        [HttpPatch]
        [Route("/widgets/{widgetName}")]
        [ProducesResponseType((int)HttpStatusCode.Created, Type = typeof(WidgetSuite))]
        [ProducesResponseType((int)HttpStatusCode.OK, Type = typeof(WidgetSuite))]
        public virtual async Task<IActionResult> CreateOrUpdateWidget([FromQuery(Name = "api-version")] string apiVersion, string widgetName, [FromHeader(Name = "Content-Type")] string contentType = "application/merge-patch+json", WidgetSuite body)
        {
            var result = await WidgetsImpl.CreateOrUpdateWidgetAsync(apiVersion, widgetName, contentType, body);
            return Ok(result);
        }

The required parameter body appears after another optional parameter.

Hi @markcowl this issue must be resolved to let todoApp generate compilable code - could you share some insights on why this has been put into the backlog?

This issue also shows up in the todoApp project's generated code: https://github.com/allenjzhang/typespec-e2e-demo/blob/2abc70f27d1e5344ce286462fa77ab887abbc228/todoApp/servers/aspnet/generated/controllers/AttachmentsOperationsControllerBase.cs#L36

Fixed in this PR: https://github.com/microsoft/typespec/pull/5417
Sample generated code, with simple todoApp service Impl here: https://github.com/allenjzhang/typespec-e2e-demo/pull/47


Mesh Saving problem

Hi, I met with problem in using rosservice call /voxblox_node/generate_mesh "{}" to save mesh. Here is lunch file.
try .launch.txt
The terminal running rosservice call /voxblox_node/generate_mesh "{}" shows that ERROR: transport error completing service call: unable to receive data from sender, check sender's logs for details
The terminal running roslaunch voxblox_ros try.launch  shows [voxblox_node-2] process has died [pid 3938, exit code -11, cmd /home/best_jill_forever/catkin_ws/devel/lib/voxblox_ros/tsdf_server -alsologtostderr pointcloud:=/camera/depth_registered/points transform:=/orb_slam_2_ros_node/transform_cam __name:=voxblox_node __log:=/home/best_jill_forever/.ros/log/d4b40912-b1a6-11e8-bfd3-a402b9214b48/voxblox_node-2.log]. log file: /home/best_jill_forever/.ros/log/d4b40912-b1a6-11e8-bfd3-a402b9214b48/voxblox_node-2*.log
And the log file shows that

Log file created at: 2018/09/06 15:31:45
Running on machine: love-mua
Log line format: [IWEF]mmdd hh:mm:ss.uuuuuu threadid file:line] msg
W0906 15:31:45.825906  3938 mesh_layer.h:56] Returning null ptr to mesh!

What could be the problem?

I fail to find out the reason but after I re-install voxblox, everything is well now.
Thank you!


scheduler: Fix bug in scheduler

The scheduler deletes containers as it loops over them. This could before cause
incorrect placements and going out of bounds. With this patch, we iterate over
the list of containers in reverse order to avoid these issues.

Cool, I agree that the commit message was pretty vague. Do you like the new one better?
Also, I'll use github.com/stretchr/testify next time!

Yep, looks good. Ready for @ejj

@ejj , as you suggested, I updated to keep a separate array of validly placed containers and assigning it to the minion at the end of the loop. I kind of don't like passing around that extra argument, though. Using a set of containers instead of an array would simplify the implementation of the check/delete. Might be better?

Actually, I take that back. The set would presumably make testing more annoying, since scheduling becomes non-deterministic


feat: add chocolatey support

Description

Adds chocolatey support

Related issue(s)
#599

TODO :-

[ ] Get version from release itself

All required changes are done @derberg.

I think you forgot to update docs?

Yeah, forgot to see it in the above-most one. Updated the installation docs just now.

/rtm


abstract type pattern is unchecked since it is eliminated by erasure

I got the following error in scala 2.10 once I upgraded from 1.1.0 to 1.1.1
https://travis-ci.org/julien-truffaut/Monocle/jobs/169164862

Can you point me at the corresponding source?

sure:
https://github.com/julien-truffaut/Monocle/blob/master/macro/shared/src/main/scala/monocle/macros/GenIso.scala#L116-L117
https://github.com/julien-truffaut/Monocle/blob/master/macro/shared/src/main/scala/monocle/macros/Lenses.scala#L58
https://github.com/julien-truffaut/Monocle/blob/master/macro/shared/src/main/scala/monocle/macros/Lenses.scala#L69
https://github.com/julien-truffaut/Monocle/blob/master/macro/shared/src/main/scala/monocle/macros/Lenses.scala#L80

Thanks @milessabin, you're fixed worked :)

For the record the fix (I think it is a fix rather than a workaround) is to use patterns of the form,
case n: TermName => ...
case n: TypeName => ...

rather than,
case n@TermName(_) => ...
case n@TypeName(_) => ...

@milessabin Can you explain why the latter doesn't work in 2.10?

It seems that the extractors are being viewed as non-stable.

Personally I would qualify that as an issue with macro-compat that requires a workaround in user-land.
I wonder what project required support for them in macro-compat in the first place: https://github.com/milessabin/macro-compat/commit/83ceac1c34b0ec9e4eeb7cc45fd41648555d1aee


[BUG] pip mode affects control panel on android

After i click pip mode close button, control panel not visible on player. We observed on android. On ios working perfectly

Hi @sedatb, I'm unable to verify your claim. Can you give us some more information please:

Version of the library you're using
A sample configuration
A recording of your claim
Steps to reproduce (if it's an atypical setup)

The issue persists. I only installed JWPlayer and directly added a video while enabling the pipEnabled: true feature. On the Android side, I can successfully enter PiP mode by pressing the PiP button. However, when I return (by any means), the player’s controls are not visible
"@jwplayer/jwplayer-react-native": "^1.0.3"
https://github.com/user-attachments/assets/8467d965-ac9a-429a-aa35-fef162841eb0


Strimzi name

Added the prefix Apache to the first use of the term Kafka in accordance with their rules.

@SamiSousa @galletti94 Apologies, could you take another look - I realised I should have applied similar updates to the AMQ Streams Operator too.

/lgtm

Thanks @SamiSousa - now squashed.


https:// not connect in java web socket library

I am using this library :- 'org.java-websocket:Java-WebSocket:1.3.0'
when I connect https url  throw exception unknown scheme https and  also use protocols ws:// or wss://(similar to http://, https://) but i get exception malformedurlexception unknown protocol wss

Hello @piyurabadiya,
have you tried the example provided here?
Greetings
marci4

No response after one months..
Looks like wrongy used constructor.
Closing issue.
Greetings
marci4


Put color scheme selector in tab bar(Or have it follow scroll)

Kind of annoying to have to scroll all the way back up to select color scheme(or it will be when we get more themes.)

This is a good point. Probably not too difficult with some sticky or absolute positioning.

It looks you need to make the whole container sticky, but then the header bar goes on top. We'd either have to account for that "Spicetify Marketplace" header item (add a background or something) or remove it. Or we could refactor the DOM for how the header bar and dropdown etc are laid out.

We could move schemes to the user menu as well.

We could move schemes to the user menu as well.

I'd vote to avoid that. The specific colour scheme probably isn't something you're gonna change a ton, and the user menu can get kinda crowded already. Makes sense to keep it in the marketplace app somewhere I think.

It looks you need to make the whole container sticky, but then the header bar goes on top. We'd either have to account for that "Spicetify Marketplace" header item (add a background or something) or remove it. Or we could refactor the DOM for how the header bar and dropdown etc are laid out.

I'm not great with CSS but I managed to get the sticky effect without the h1 element by setting its position to fixed.
I had a problem aligning the h1 to where it was originally so I had to take the padding of the .contentSpacing element and put it in the h1's left property.
.marketplace-header {
  position: sticky;
  top: 76px;
  z-index: 1;
  flex-direction: row-reverse;
}

.marketplace-header > h1 {
  position: fixed;
  left: 16px;
}

@media (min-width: 1024px) {
  .marketplace-header > h1 {
    left: 32px;
  }
}

@FlafyDev I'm not able to test this at the moment, would you mind including some relevant screenshots of what it looks like? If not it's fine and I'll try to take a look at it tomorrow.

I'm not able to test this at the moment, would you mind including some relevant screenshots of what it looks like? If not it's fine and I'll try to take a look at it tomorrow.

Sure:

Thanks @FlafyDev , looks great! Added, tested locally, and committed 👌


[updatecli][shell] Bump version to 1.19

[updatecli][shell] Bump version to 1.19
Report
Source:
	✔ [gomod] Get version in go.mod format(shell)
	✔ [latestGoVersion] Get Latest Go Release(githubrelease)
	✔ [minor] Get minor version in .go-version(shell)

Condition:
	✔ [dockerTag] Is docker image golang:1.19.5 published(dockerimage)
	✔ [goDefaultVersion-check] Check if defined golang version differs(shell)

Target:
	⚠ [update-go-version] Update .go-version(file)
	⚠ [update-golang.ci] Update .golangci.yml(file)
	⚠ [update-gomod] Update go.mod(file)

Changelog
Click to expand

no GitHub Release found for go1.19.5 on "https://github.com/golang/go"

Remark
This pull request was automatically created using Updatecli.
Please report any issues with this tool here

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2023-02-07T08:58:13.892+0000

Duration: 10 min 24 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
751

Skipped
5

Total
756

:robot: GitHub comments
Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.


ipython lacks readline & colours support

I'm guessing this happened during 4.0 / jupyter upgrade. Running ipython (from pythonPackages.ipython) gives:
$ /nix/store/hims3s84qgr9dnmvwr021yd9r7wr7620-python2.7-ipython-4.0.0/bin/ipython
WARNING: Readline services not available or not loaded.
WARNING: The auto-indent feature requires the readline library
Python 2.7.10 (default, Jan 01 1970, 00:00:01)
Type "copyright", "credits" or "license" for more information.

IPython 4.0.0 -- An enhanced Interactive Python.
?         -> Introduction and overview of IPython's features.
%quickref -> Quick reference.
help      -> Python's own help system.
object?   -> Details about 'object', use 'object??' for extra details.

>>>

Tab completion / readline doesn't work, and the prompt has no colours any more (I don't know whether this is due to readline missing or a separate issue).
/cc @FRidh

Thanks for your report. readline is indeed missing, causing both tab completion not to work and not showing any colors. I'll add the dependency.


Exception when closing UNIXServer with abstract socket address

Abstract socket addresses start with a '\0', which causes a raise in UNIXSocket#close
require "socket"

UNIXSocket.open("\0foo") do |c|
  c.close
end

string contains null byte (ArgumentError)
0x42a9e7: *CallStack::unwind:Array(Pointer(Void)) at ??
0x42a97a: *CallStack#initialize:Array(Pointer(Void)) at ??
0x42a94a: *CallStack::new:CallStack at ??
0x42600e: *raise<ArgumentError>:NoReturn at ??
0x433b28: *String#check_no_null_byte:String at ??
0x44080a: *File::accessible?<String, Int32>:Bool at ??
0x4407f8: *File::exists?<String>:Bool at ??
0x43da8b: *UNIXServer#close:Nil at ??
0x425b75: ??? at ??
0x4288c9: main at ??
0x7fb6031b6291: __libc_start_main at ??
0x42546a: _start at ??
0x0: ??? at ??

UNIXServer.new and UNIXServer.open should take an optional abstract parameter to inject the NULL byte.

UNIXSocket.new and UNIXSocket.open should also take the optional abstract parameter.
I'd like to do that as my first contribution, somewhere this week if you don't mind

Both UNIXSocket and UNIXServer could take an abstract = false argument, and pass it down to UNIXAddress;
UNIXAddress could take an abstract = false argument, and only prepend the null byte to path when creating the LibC::SockaddrUn, so @path never contains a null byte;
We could provide an UNIXAddress#abstract? method, and maybe an UNIXSocket#abstract? method;
UNIXServer#close shouldn't search/delete the path if it's abstract, but still set @path to nil.

I agree with all you points, but I would add UNIXServer#abstract? also (if the user want the information back)

oh yes, I forgot that inheritance, thanks

#codetriage
This no longer errors out, currently shows this trace:
root@ubuntu-s-1vcpu-1gb-nyc3-01:~# uname -a
Linux ubuntu-s-1vcpu-1gb-nyc3-01 4.4.0-131-generic #157-Ubuntu SMP Thu Jul 12 15:51:36 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

root@ubuntu-s-1vcpu-1gb-nyc3-01:~# crystal --version
Crystal 0.26.1 [391785249] (2018-08-27)

LLVM: 4.0.0
Default target: x86_64-unknown-linux-gnu

root@ubuntu-s-1vcpu-1gb-nyc3-01:~# crystal run foo.cr
Unhandled exception: connect: Connection refused (Errno)
  from /usr/share/crystal/src/socket/unix_socket.cr:0:7 in 'initialize'
  from /usr/share/crystal/src/socket/unix_socket.cr:18:3 in 'new'
  from /usr/share/crystal/src/socket/unix_socket.cr:40:5 in '__crystal_main'
  from /usr/share/crystal/src/crystal/main.cr:97:5 in 'main_user_code'
  from /usr/share/crystal/src/crystal/main.cr:86:7 in 'main'
  from /usr/share/crystal/src/crystal/main.cr:106:3 in 'main'
  from __libc_start_main
  from _start
  from ???

The work in #4056 hasn't been merged in yet, so it's unclear whether this is meant to work?
I've tried some strace to see what the client is doing when attempting to establish a connection.
## session 1 (server)
$ socat ABSTRACT-LISTEN:foo -

## session 2 (client)
$ netstat -l | grep foo
unix  2      [ ACC ]     STREAM     LISTENING     27903    @foo

$ crystal build client.cr && strace ./client
...
connect(10, {sa_family=AF_LOCAL, sun_path=@"foo"}, 110) = -1 ECONNREFUSED (Connection refused)

So it appears that something is being translated to the correct sun_path, but it's not able to make the connection? I'm not certain of the next steps here - but the original crash is gone.


Request all shared attributes

Hello everyone,
Is it possible to request all shared attributes and not just only 1 key?

It is not possible to request keys, at keast to my knowledge.
But this limitation is not from this library, but instead the ThimgsBoard API. See the official MQTT Documentaion for more info.
Alternatively, you can use the Shared_Attribute_Subscribe method, which subscribes to any changes or creation of new shared attributes.
For that simply pass the Shared_Attribute_Update instance, but keep the 2nd parameter empty. Because if no keys are configured you are subscribed to all key changes.

Yeah i see now, thnx.
Shared_Attribute_Subscribe without keys is not very useful right? Because if the device is offline it cant know that it was created to later request it.

Yes that is the case updates in between states of being offline will be lost. I'll check the API again to really ensure if it isn't possible to request all keys, because with REST it is.
But if it were it would be an undocumented feature, which would definetly need to be changed.
Will update this thread once I tested it, but I dont have high hopes.

@JellevanKraaij
Check this example: https://thingsboard.io/docs/pe/devices-library/esp32-dev-kit-v1/
Using Shared_Attributes_Request() method you can request attributes from the server once the device is connected. But you have to set attributes on the callback level.

Thanks for tip @skokovsergey, it is unfortunately required to list the attributes, you can't request all attributes without knowing theire names


Update template for specifying extraArgs

Currently, extraArgs has to be specified in a 'nonstandard' way otherwise we get an error
helm template .
Error: YAML parse error on schemahero/templates/manager.yaml: error converting YAML to JSON: yaml: line 37: could not find expected ':'

currently:
extraArgs: ['- --log-level', '- debug']
expected:
extraArgs: ['--log-level', 'debug']

https://github.com/schemahero/schemahero/blob/2e24a481338c3795abb9c0fdc20d27c86ef4606d/pkg/cli/managercli/run.go#L124-L128
There is no flag definition for log level, so it is not expected to be executed. It seems that the corresponding code needs to be modified.
In fact, even if extraArgs is specified, the following Pod logs are checked:
unknown flag: --enable-database-controller --log-level debug

https://github.com/schemahero/schemahero-helm/pull/12


Bug: Files opens behind all other open apps on first start

Description
WHen i open files for the first time it opens begind all other open apps. This is annoying and requires an extra click to open the app.
Steps To Reproduce

Open files

Requirements
Files should be opened in front of the other open windows like any other app does.
Files Version
<IP_ADDRESS>
Windows Version
10.0.23585.1001
Log File
debug.log

Thanks for the report, can you see if this is an issue in the preview version. https://files.community/download

Yes it is!

Thanks for checking, how are you opening Files and do you have Files settings > Advanced > Set as default enabled?

Often im opening through the taskbar, but even when another app opens files it goes to the background.
And yes Set as default is on

Same problem with other software:
https://github.com/M2Team/NanaZip/issues/86
https://github.com/microsoft/terminal/issues/6611

Personally don`t have the issue in terminal

Same problem here, but not only on first start. The app (window) opens in the background (behind all other windows from different apps) if I let files stay running after closing the window. The window should be pushed to the front, when I open Files.

Thought I fixed this issue.
It's interesting that the app opens behind the all other apps.

I "solved" the problem with adding
Win32Helper.BringToForegroundEx(new HWND(this.WindowHandle));
to MainWindow.xaml.cs - InitializeApplicationAsync at the end:
if (!this.AppWindow.IsVisible) {
// When resuming the cached instance
this.AppWindow.Show();
Activate();
}
Win32Helper.BringToForegroundEx(new HWND(this.WindowHandle)); // here
if (Windows.Win32.PInvoke.IsIconic(new HWND(this.WindowHandle)))
Instance.Restore(); // Restore window if minimized

Sure not perfect, but it works for now (as long as a developer has time to look into it).

Yeah I made that helper.
Could you open a PR for the fix?

I also still struggle with this issue, it was fixed in the beta build I tried but now it's back again


Dismissible RTL

Fix the dismissible demo in the gallery (make it actuall update when you pick something from its menu; give it a better affordance for resetting once you've dismissed everything).
Improve some docs.
Fix various flinging bugs with dismissible. Add tests for those cases.
Add a feature to flutter_test to support a drag-then-fling gesture (used by the flinging tests).

LGTM


Upload User Profile Picture from Local Storage

Feature Request
Feature Description
The user should be able to upload images from device local storage to the server and use the returned URL to show the profile picture in the app.
Parent Issue
#1093
Would you like to work on the issue?
Yes

Done in #1517

Great Job

Thanks


question on adding a new coin

Hi, I'm experimenting with adding a new coin. However, even though I'm pulling blocks from block #1, they all get relegated to orphan status because I have no 'prev'. Is there something you do to place the genesis block so that this doesn't happen? I'm totally perplexed.
In _peerRelayedBlock(void *info, BRMerkleBlock *block), where you have
    prev = BRSetGet(manager->blocks, &block->prevBlock);

    if (prev) {
        txTime = block->timestamp/2 + prev->timestamp/2;
        block->height = prev->height + 1;
    }

I have no blocks, so prev is always null, even though the first block I get from the server is the one where the prevblock hash is the genesis hash...
(or maybe you never start from the beginning?)
Update. I think I see it here

    for (size_t i = 0; i < manager->params->checkpointsCount; i++) {
        block = BRMerkleBlockNew();
        block->height = manager->params->checkpoints[i].height;
        block->blockHash = UInt256Reverse(manager->params->checkpoints[i].hash);
        block->timestamp = manager->params->checkpoints[i].timestamp;
        block->target = manager->params->checkpoints[i].target;
        BRSetAdd(manager->checkpoints, block);
        BRSetAdd(manager->blocks, block);
        if (i == 0 || block->timestamp + 7*24*60*60 < manager->earliestKeyTime) manager->lastBlock = block;
    }

maybe my checkpoints are defined correctly....

We don't always start from the beginning, only on a fresh install and then save the blocks to the local client's DB and then pass them in the next launch to minimize the syncing time.

I found the issue, and it is a mysterious. When _peerRelayedBlock is called, somehow my block gets scrambled. Before the call the block has version = 6, but when it gets into _peerRelayedBlock the version is 65541 and the rest of the data is scrambled also. Very strange.

This is the android project, please consider to create issues in the core project:
https://github.com/breadwallet/breadwallet-core


I can't execute an expect with my own Promise without awaiting

If I look at some TestController properties / methods from TestCafe, they all return a promise, eg.
SelectorAPI.count: Promise<number>
SelectorAPI.visible: Promise<boolean>
The expect method can accept actual: A | Promise<A>.
If I run my script and I have t.expect(x.count).eql(#), the assertion is perfectly executed so it handled the promise of count.
However if I run expect with on my own method which returns a promise, I get an error. eg.
myCount: Promise<number> = new Promise(resolve => resolve(4))
But now, when I run t.expect(myCount).eql(4) I get the following exception:
Attempted to run assertions on a Promise object. Did you forget to await it? If not, pass "{ all
owUnawaitedPromise: true }" to the assertion options.

And I know, running t.expect(await myCount).eql(4) will execute fine, but it is weird that the expect function can accept promises, but only promises from the testcontroller and not my own created promises.
Could someone please elaborate on this functionality and why expect cannot handle all promises?
Your Environment details:

testcafe version:                   1.8.1
node.js version:                    12.13.1

By default, only Selector and ClientFunction promises can be passed as the assertion's actual value. You can use the allowUnawaitedPromise option to assert your own Promise.

@Dmitry-Ostashev I don't know why you closed this issue. I know only certain types work, my question is why other promises are not able to work (without adding await or a assertionOption).
If you look at the types, .count(x) returns a normal promise, nothing special. So again my question.

Just an example:
Simple promise property:
const myCount: Promise<number> = new Promise(resolve => resolve(4));
Without any await or assertion options you get the error
Attempted to run assertions on a Promise object. Did you forget to await it? If not, pass "{ allowUnawaitedPromise: true }" to the assertion options.

I add the assertion option:
await t.expect(myCount).eql(4, { allowUnawaitedPromise: true });
Then I get the following result:
AssertionError: expected {} to deeply equal 4
So the allow unawaited promise option does not resolve the unawaited promise. I don't really know what this option has done at this moment.

Hi @stefanschenk,
When expect receives Selector or ClientFunction as an actual value, it calculates it using the Smart Assertion Query Machanism. In the case of your custom promise, this approach is obviously not applicable.
What are you trying to achieve when you pass a promise as an actual value? Do you want it to be awaited and its result to be checked? If so, you need to add await. If you want the promise object to be examined, use the allowUnawaitedPromise option.


GROOVY-7697: GroovyScriptEngine.loadScriptByName doesn't support envi…

…ronment variable "groovy.ast"
This PR fixes an issue where the "groovy.ast" dump (only god, its author and the ticket author knew this existed) would not be saved when a script was loaded by name via the GSE. In that case, the XStreamUtils instance was given an URI (String) instead of an absolute path and the code there broke.
Regarding error handling in XStreamUtils (as this was also mentioned in the ticket): I did not change anything into that direction as I am not convinced that such a secondary "add-on" feature should in any way influence behaviour in SourceUnit#convert.

When I run the build with the changes from the pull request locally (where tests run in parallel), this test always fails:

groovy.util.ProxyGeneratorTest > testUnknownMethodThrowsUnsupportedOperationException FAILED
groovy.lang.MissingMethodException: No signature of method: static groovy.util.ProxyGenerator.instantiateAggregateFromInterface() is applicable fo
r argument types: (java.util.LinkedHashMap, java.lang.Class) values: [[myMethodA:groovy.util.ProxyGeneratorTest$_testUnknownMethodThrowsUnsupportedOpe
rationException_closure16@6cb59bd9], ...]
Possible solutions: instantiateAggregateFromInterface(java.lang.Class), instantiateAggregateFromInterface(java.util.Map, java.lang.Class)

not sure how it could be related.

Thanks for hinting on that, @PascalSchumacher. I will have a look at the code again once I find some time this week(end).

Thanks. I'm not sure this is caused by the changes of this pull request, maybe the additional tests just reveal a thread-safety issue of other tests (which now run in parallel because of the new tests).

Thanks!
I tested a build with this PR on several different machines and didn't hit the test failure that @PascalSchumacher had.


Async support

It looks like app-model doesn't support running in an event loop? For instance, commands can only be executed asynchronously through threads.
Do you think it could support async GUI backends? I'm thinking about Textual in particular.

yeah, the motivating use case for app-model was napari (with a Qt event loop).  But I definitely tried to make it backend agnostic, and would love to support asyncio runtimes as well.  I don't have a ton of experience with asyncio (though I'd like to know more), so if you have thoughts on what the API should look like for registering/executing awaitable commands, or even just a simple example of how you'd like it all to work, that would be great.

would be as easy as simply calling and returning the coroutine object if it's a coroutine function? ( adding typing.Awaitable to the return type
    def execute_command(
        self,
        id: str,
        *args: Any,
        **kwargs: Any,
    ) -> Future | Awaitable:
        ...
        if iscoroutinefunction(cmd):
            return cmd(*args, **kwargs)

that appears to work in ipython with await reg.execute_command('some_id')

Maybe it's as simple as that, yes. If app-model works with an event-loop based framework like Qt, then it should also work with a framework like Textual.
I'll try to use it for jpterm. It would be awesome if we could seamlessly switch backends.

closing, since i think app-model already supports async (though if anyone finds a good counter-example, please feel free to note it here)


Bug: Last elements in basic_charset initialization are discarded

Describe the bug
When initializing a basic_charset with character arrays such as those returned by digits() and others, the last element of the array is incorrectly discarded.
https://github.com/ashvardanian/StringZilla/blob/152ed046701aceae2c3d4c995b0b661a9aa06d95/include/stringzilla/stringzilla.hpp#L192-L195
This is due to the use of count_characters - 1 in the loop that populates the basic_charset bitset.
https://github.com/ashvardanian/StringZilla/blob/152ed046701aceae2c3d4c995b0b661a9aa06d95/include/stringzilla/stringzilla.hpp#L283-L289
While this prevents including a null terminator for string literals like sz::char_set("x"), it causes incorrect behavior when handling character arrays that do not have a null terminator, resulting in the exclusion of the final character.
I believe this local block is the full extent of the affected code.
https://github.com/ashvardanian/StringZilla/blob/152ed046701aceae2c3d4c995b0b661a9aa06d95/include/stringzilla/stringzilla.hpp#L330-L341
Steps to reproduce
#include "stringzilla/stringzilla.hpp"

namespace sz = ashvardanian::stringzilla;

int main() {
        sz::string haystack = "239";

        // Test with null-terminated string
        assert(haystack.contains_only(sz::char_set("0123456789")));
        // Passes: null terminator is correctly discarded

        // Test with initializer list
        static std::initializer_list<char> all = {'0', '1', '2', '3', '4', '5', '6', '7', '8', '9'};
        assert (haystack.contains_only(sz::char_set {all}));
        // Passes: constructor for initializer list is called

        // Test with carray
        sz::carray<10> all_digits = {'0', '1', '2', '3', '4', '5', '6', '7', '8', '9'};
        assert(haystack.contains_only(sz::char_set {all_digits}));
        assert(haystack.is_digit());
        // Fails: '9' is incorrectly discarded
}

Expected behavior
No asserts
StringZilla version
v3.11.0
Operating System
Ubuntu 22.04.5
Hardware architecture
x86
Which interface are you using?
C++ bindings
Contact Details
No response
Are you open to being tagged as a contributor?

[X] I am open to being mentioned in the project .git history as a contributor

Is there an existing issue for this?

[X] I have searched the existing issues

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Proposed Fix

Using carray with Null Terminator

inline carray<11> const &digits() noexcept {
    static carray<11> const all = {'0', '1', '2', '3', '4', '5', '6', '7', '8', '9', '\0'};
    return all;
}

Using std::initializer_list<char>

inline auto const &digits() noexcept {
    static std::initializer_list<char> all = {'0', '1', '2', '3', '4', '5', '6', '7', '8', '9'};
    return all;
}

Thank you @alexbarev! Let's start with tests, and later merge a patch.


syntax for closing a {% when ... %} block within a match

Hey! I have enjoyed using this project and am unsure if anything like this would be possible.
The long and the short of it is that most constructs in the syntax have both a {% something %} and an {% endsomething %}, this is not the case for match (when) arms.
I understand why this is, and agree it should be the default. But, a lot of html templating tools (formatters in particular) work best when each block has a start and end.
This is a simplified example of the current syntax. Formatted with djlint, a common and widely used jinja formatter.
{% match self.lpn %}
  {% when None %}

Scan a LPN number to begin.

  {% when Some(lpn_value) %}
  {% match customer_returns %}
    {% when None %}

      No Customer Returns found for {{ lpn_value }}.

    {% else %}
  {% endmatch %}
{% endmatch %}

formatter settings
[tool.djlint]
custom_blocks = "match"
ignore_blocks = "when"

Notice how the content of the when arms in on the same indentation level as the when itself. Some people may want it that way, and it should be the default.
Proposed
What I am suggesting, would be to optionally allow closing when arms.
This is produced by the same formatter, with slightly different settings.
{% match self.lpn %}
  {% when None %}

Scan a LPN number to begin.

  {% endwhen %}
  {% when Some(lpn_value) %}
    {% match customer_returns %}
      {% when None %}

          No Customer Returns found for {{ lpn_value }}.

      {% endwhen %}
      {% when Some(customer_return_value) %}
      {% endwhen %}
    {% endmatch %}
  {% endwhen %}
{% endmatch %}

formatter settings
[tool.djlint]
custom_blocks = "match,when"
ignore_blocks = ""

Notice how there is now a clear difference in indenation levels. AFAIK, no formatter for html/jinja templates allows this type of template to be configured for, because it can't know where it ends.
I know the difference is minimal, I know it is not really the crates job to worry about stuff like this. But, I think even semantically, it does make sense for the user to have the ability to specifically close a when arm, even if proc macro just throws it away.
I am willing to write the PR for this, if open to the change

I think the when is kind of of close to elif/else: you can write them one after the other without an end. I'm personally not much in favor of adding an endmatch as it would make the syntax heavier but let's see what others think.

I think the when is kind of of close to elif/else: you can write them one after the other without an end. I'm personally not much in favor of adding an endmatch as it would make the syntax heavier but let's see what others think.

I am pretty sure you just mispoke but to be clear, I am arguing for the endwhen instead of the endmatch (which already exists).
But, yeah I do agree that more syntax kind of sucks. I am purely in the "it definitely shouldn't be mandatory, but it would be nice to optionally have". Then again, optional syntax can become a PITA to maintain alongside other changes.
In any case, I don't mind having a fork for a while to gauge if anyone else feels strongly about it. Thanks for being open to it at least!

Jetlag and reading on mobile. So yes, I meant endwhen. ^^'
Making it optional could be a nice improvement though.


Update classes to remove Redis name due to license changes

Phileas uses Redis for caching in various places - alerts, anonymization, span disambiguation, and policies.
While Phileas does not use Redis itself, these classes have names that include Redis, such as RedisAlertService, and its underlying AbstractRedisCacheService.
Phileas does have a dependency on the redisson client library which is available under the Apache license. Redisson also supports valkey, the Redis fork available under an open source license. AWS is offering valkey as a managed service.
The class naming with respect to Redis should be made generic to not imply that Redis is required for Phileas, when valkey is also an option.

The bigger idea here is that the cache should not have to be Redis (or Valkey). The abstract cache service should allow for a lot of implementations. Having the abstract class service named AbstractRedisCacheService is very restricting.


RFC for admin panel

What's the status for moving the admin panel to a SPA website?

Let me know what you need.
On Mon, May 21, 2018 at 5:54 PM Mark Moffat<EMAIL_ADDRESS>wrote:

Needing assistance.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mrvautin/expressCart/issues/10#issuecomment-390639151,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFmqGVro46YQ0OaWLjuyYonpqZZq8LPuks5t0rIAgaJpZM4TCcwN
.

--
Regards
-Tarang Hirani

Closing as don't have time and people don't seem interested. Can re-open if needed.


wit: sort packages in topological order when rendering WIT

Multiple packages in a single WIT file are now sorted in topological order of dependence, starting with the package with fewest dependents. In practice, this means that the "root" package, e.g. wasi:http would be listed first when serializing wasi:http/proxy and its dependents.
This means that other tools, like wit-bindgen can process generated WIT without additional arguments like --world.

That was a fun debugging journey! Thankful we had WIT → JSON → WIT round trip tests. Turns out we had an old JSON representation of wasi:cli.


[Event Change Request] Codeunit 8 "AccSchedManagement"

Hello, can you please the Event OnBeforeTestBalance set scope from local to global?
[IntegrationEvent(false, false)]
    local procedure OnBeforeTestBalance(var GLAccount: Record "G/L Account"; var AccScheduleName: Record "Acc. Schedule Name"; var AccScheduleLine: Record "Acc. Schedule Line"; var ColumnLayout: Record "Column Layout"; AmountType: Integer; var ColValue: Decimal; CalcAddCurr: Boolean; var TestBalance: Boolean; var GLEntry: Record "G/L Entry"; var GLBudgetEntry: Record "G/L Budget Entry"; var Balance: Decimal)
    begin
    end;

No, you cannot call events directly. You must subscribe to events.


data not found

hi,
amazing work, I also read your master thesis paper, I tried to run the code you attached but I got the error
data not found
NotFoundError (see above for traceback): FindFirstFile failed for: /home/neo/projects/deepLearning/data/image : The system cannot find the path specified.

Can you provide the bin file you generated for label images from AM-FED videos, I tried to run the code in you paper but with no luck


Polyfile parser does not parse 2D boundary type of pli file correctly.

Describe the bug
Polyfile parser does not parse 2D boundary type of pli file correctly.
See example file: Boundary01.pli.txt
To Reproduce
Steps to reproduce the behavior:
from pathlib import Path
from hydrolib.core.dflowfm.polyfile.parser import read_polyfile
fn = Path(r'Boundary01.pli')
read_polyfile(filename)
{'has_z_values': False, 'objects': []}

See the empty objects, while the 'Boundary01.pli' is not empty:
"
Boundary01
3    2
6.625216613133794E+005  1.524474171041823E+006 Boundary01_0001
6.625216613133794E+005  1.524910832891534E+006 Boundary01_0002
6.625216613133794E+005  1.525410832891534E+006 Boundary01_0003
"
Expected behavior
Able to parse this file for the 2D boundary.
Suggested solution
(by @arthurvd )
The third column of support point labels should be ignored by the parser (but NOT lead to an error). The header definition is leading: if it specifies 2 columns, then only read 2 values from each line, and silently ignore anything that's left at the end.
My motivation for this solution is: the Delft3D FM Suite currently exports with a support point label on each line, and we want to support all models files produced by our official release.
However, there is no need to parse and store those labels, because they are in fact ignored right now by the kernel, because frontend and backend are not using the same format currently (even though they are present in the .pli files)
This first need to be fixed in frontend+backend. Only then we will consider full support in hydrolib-core.
Version info (please complete the following information):

OS: [e.g. Windows]
Version [e.g. 0.4.1]

Thanks, @arthurvd and @veenstrajelmer for the explanation! Now I understand why the supported point labels are not supported by hydrolib-core. And I agree that the fix should first be the front- and back-end.
Since the pli file without supported point labels can indeed be read back to the GUI, I think we are good off with reading/writing pli files without supported point labels in hydromt-delft3dfm.
That means we won't support updating a model that was exported from GUI, which was acceptable.
If this behavior would change, we will seek dfm-tools for preprocessing the pli file.

If you got your code from some documentation, please let me/@arthurvd know so it can be updated.

Thanks! I did not get the code from the documentation, it was a small test of "Why my PolyFile object is empty" :)

Hi @xldeltares, I will re-open this issue, because I think hydrolib-core should support reading of boundary polylines currently produced by the FM Suite (both kernel and GUI support reading of pli point labels, only they are not used in the actual calculation).

Hi @xldeltares, I know that you are currently modifying boundary pli files, removing the last column with point labels.
Yet, we have fixed the support for this in hydrolib-core.
Could you pleas make some time to test the new support on your (unmodified) pli files?
The code is in:
https://github.com/Deltares/HYDROLIB-core/pull/482
Thanks!

@veenstrajelmer Thanks for picking this up!


make error

make on system with previous micro checkout gives these warnings:
make
tools/build-version.go:8:2: cannot find package "github.com/blang/semver" in any of:
	/usr/lib/golang/src/github.com/blang/semver (from $GOROOT)
	/home/user/p/go/src/github.com/blang/semver (from $GOPATH)
git pull
Already up-to-date.
git submodule update
go build -ldflags "-s -w -X main.Version= -X main.CommitHash=18ad74a -X 'main.CompileDate=July 12, 2017' " ./cmd/micro
cmd/micro/pluginmanager.go:16:2: cannot find package "github.com/blang/semver" in any of:
	/home/user/p/go/src/github.com/zyedidia/micro/cmd/micro/vendor/github.com/blang/semver (vendor tree)
	/usr/lib/golang/src/github.com/blang/semver (from $GOROOT)
	/home/user/p/go/src/github.com/blang/semver (from $GOPATH)
cmd/micro/command.go:16:2: cannot find package "github.com/dustin/go-humanize" in any of:
	/home/user/p/go/src/github.com/zyedidia/micro/cmd/micro/vendor/github.com/dustin/go-humanize (vendor tree)
	/usr/lib/golang/src/github.com/dustin/go-humanize (from $GOROOT)
	/home/user/p/go/src/github.com/dustin/go-humanize (from $GOPATH)
Makefile:15: recipe for target 'build' failed
make: *** [build] Error 1

These command fixed the error:
go get github.com/blang/semver
go get github.com/dustin/go-humanize

But there are more:
go build -ldflags "-s -w -X main.Version=1.2.1-dev.72 -X main.CommitHash=18ad74a -X 'main.CompileDate=July 12, 2017' " ./cmd/micro
# github.com/zyedidia/micro/cmd/micro/vendor/github.com/zyedidia/tcell
cmd/micro/vendor/github.com/zyedidia/tcell/encoding.go:124: cannot use "github.com/gdamore/encoding".ASCII (type "golang.org/x/text/encoding".Encoding) as type "github.com/zyedidia/micro/cmd/micro/vendor/golang.org/x/text/encoding".Encoding in return argument:
	"golang.org/x/text/encoding".Encoding does not implement "github.com/zyedidia/micro/cmd/micro/vendor/golang.org/x/text/encoding".Encoding (wrong type for NewDecoder method)
...
cmd/micro/vendor/github.com/zyedidia/tcell/encoding.go:138: cannot use "github.com/gdamore/encoding".ASCII (type "golang.org/x/text/encoding".Encoding) as type "github.com/zyedidia/micro/cmd/micro/vendor/golang.org/x/text/encoding".Encoding in assignment:
	"golang.org/x/text/encoding".Encoding does not implement "github.com/zyedidia/micro/cmd/micro/vendor/golang.org/x/text/encoding".Encoding (wrong type for NewDecoder method)
		have NewDecoder() *"golang.org/x/text/encoding".Decoder
		want NewDecoder() *"github.com/zyedidia/micro/cmd/micro/vendor/golang.org/x/text/encoding".Decoder
Makefile:15: recipe for target 'build' failed
make: *** [build] Error 2

$ rm -rf ../../../golang.org/x/text/encoding
$ go get golang.org/x/text/encoding
go build golang.org/x/text/encoding: no buildable Go source files in /home/user/p/go/src/golang.org/x/text/encoding

rm -rf ../../../golang.org/x/text
go get golang.org/x/text

Doesn't help. The same (wrong type for NewDecoder method) error.

rm -rf ../../../github.com/gdamore/encoding
go get github.com/gdamore/encoding

Doesn't help too.

Side issue - https://github.com/zyedidia/micro/wiki/Installing-Micro#makefile there is no documented deps command in Makefile.

So I did see the problem with blang/semver not being recognized when trying to run tools/build-version.go and have fixed that problem in the most recent commit. I have not experienced any of the other build problems you seem to have.
Make sure that your copy of micro is on your $GOPATH in the correct place. If you don't have anything of import on your gopath you could try removing $GOPATH/src and recloning micro go get -d github.com/zyedidia/micro/cmd/micro.

Are you still having problems with the make file?

No other problems except this warning:
tools/build-version.go:8:2: cannot find package "github.com/zyedidia/micro/tools/semver" in any of:
	/usr/lib/go-1.7/src/github.com/zyedidia/micro/tools/semver (from $GOROOT)
	($GOPATH not set)


[AutoPR compute/resource-manager] Swagger: Allowing for VMSS reference during VM creation

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6836

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/5524


Define atomic semantic interfaces for ForageUI

Description
Please include a summary of the change. Please also include relevant motivation and context. List any dependencies that are required for this change.
Also

Add any other relevant change made on this PR.

Notes

Describe your changes

Tests Added / Updated?

YES or NO. If not, why?

Screenshots

Current dependencies on/for this PR:

main

PR #46   👈

This comment was auto-generated by Graphite.

Hot dayum! Thanks for the speedy review Danny! You even beat the CI/CD checks! 💥


Support "terminal.integrated.automationShell" for environment activation

Environment data

VS Code version: 1.39.2
Extension version (available under the Extensions sidebar):  2019.10.44104
OS and version: Manjaro Linux
Python version (& distribution if applicable, e.g. Anaconda): 3.7.4 /dependent
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): virtualenv
Relevant/affected Python packages and their versions: N/A
Jedi or Language Server? (i.e. what is "python.jediEnabled" set to; more info #3977): whatever the default is

Expected behaviour
When fish is the integrated automation shell, the following should be run to activate an environment:
source <python.venvpath>/bin/activate.fish
Actual behaviour
When opening up a new terminal, or when trying to debug, the following gets run:
source <python.venvpath>/bin/activate
which is for bash shells. This doesn't activate the environment.
Steps to reproduce:

have this in your user settings json: "terminal.integrated.automationShell.linux": "fish",
activate a virtualenv environment
open up a terminal, or try to debug a script
first command that's run is source <python.venvpath>/bin/activate, which is for bash/zsh/dash, and will fail for fish and zsh

Logs
Not sure what logs would be useful, but here's an example when I try to debug a script
 ~/D/studproject  source /home/massiveatoms/virtualenvs/studieproject-ML/bin/activate              wo 13 nov 2019 21:28:05 -03
~/virtualenvs/studieproject-ML/bin/activate (line 82): Missing end to balance this if statement
if [ -n "${BASH-}" ] || [ -n "${ZSH_VERSION-}" ] ; then
^
from sourcing file ~/virtualenvs/studieproject-ML/bin/activate
        called on standard input

source: Error while reading file “/home/massiveatoms/virtualenvs/studieproject-ML/bin/activate”

^ that's just fish complaining because fish scripts have different syntax compared to bash, I think.

Duplicate of #8341 (which has a workaround for this problem).

Hey, could you perhaps point out the workaround?
Because I'm just not seeing it.
Also, I don't think our problems are duplicates.
In your "Actual behaviour" section, you can see that the first thing that's run is
source /Users/brettcannon/Repositories/brettcannon/caniusepython3/.venv/bin/activate.fish
The problem I'm having is that vscode is running  source ~/ENV/bin/activate instead of source ~/ENV/bin/activate.fish.
Furthermore, I can manually run source ~/ENV/bin/activate.fish and everything will work fine, while you said that's another way to reproduce the issue for you?
Also, (take this with a grain of salt, I'm new to fish) it kind of appears that it's a fish problem? I had a similar issue when I tried to get virtualfish (a virtualenv wrapper for fish) to work, and I ended up removing my fish configs and reinstalling it to get it to work.

@TinyAtoms Sorry! You're right, I misread your issue and didn't pick up on the "automationShell" part. I have reopened this as a feature request to support "editor.integrated.automationShell" for environment activation.
As for the config issue, if I you can find something wrong in https://github.com/brettcannon/dotconfig/tree/master/fish then please let me know, otherwise the I have had has bit enough people that I think it might be a subtle problem (it's even been reported to virtualenv with no solution yet).

Hmmm, I see nothing wrong (with my untrained eye) with your fish configs.
What version of fish and virtualenv are you using, btw? Maybe an update from one of them fixed the issue? I'm using  fish 3.0.2-2 and virtualenv 16.7.7 and my activate.fish activates without errors.

Latest fish and I'm using venv, not virtualenv (mainly because it's better integrated and I can fish things in the stdlib 😁 ).

Hmm, I just tried it (never hear od venv before), but it works fine for me
https://i.imgur.com/gsoxznP.png in plain terminal
https://i.imgur.com/4JneenD.png in vscode

Is this issue still happening with the latest version of the extension?

Because we have not heard back with the information we requested, we are closing this issue for now. If you are able to provide the info later on then we will be happy to re-open this issue to pick up where we left off.


Polls 1 of 3 gmails evry 10 instead of set 60 mins...

Expected behaviour
Tell us what should happen
Should only poll every hr but polls every 10 mins. 2 gmails and 1 other. Only does this with 1 big use gmail.
Actual behaviour
Tell us what happens instead
Polls evry 10 mins instead of set hrly.
Steps to reproduce

Disabled in Cell apps settings, cleared cache.
Restarted K9, changed polling to 30.
Repeated all above, set poll to hr.

Environment
K-9 Mail version: Latest Googleplay K9 update.
Android version: 4.1.2 Sony Xperia LT28i
Account type (IMAP, POP3, WebDAV/Exchange):
POP3.
Didn't do this before for many mths, until about few mths ago. I suspect after auto-updating via Playstore app.

Just to clarify, you have 3 accounts, all POP3, two of which are Gmail-based and one which isn't. And one of the Gmail accounts gets polled every 10 minutes instead of every hour?

Yes this one. About 1/2 the time. I just watched it with notifications bar pulled down, when I renabled my internet connection. It checked this email 1st. Showed new emails for this address while it also showed polling the other 2 addresses. When it was done polling, the note bar note showing new emails vanished. It started doing this week or 2 ago.
Cam Gibb.
On 31 March, 2016 5:01:12 AM EDT, Philip<EMAIL_ADDRESS>wrote:

Just to clarify, you have 3 accounts, all POP3, two of which are
Gmail-based and one which isn't. And one of the Gmail accounts gets
polled every 10 minutes instead of every hour?

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/k9mail/k-9/issues/1235#issuecomment-203833690

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

"When it was done polling, the note bar note showing new emails vanished. It started doing this week or 2 ago."
That's clearly a separate issue, probably related to #834 I guess... After/during the design overhaul there is discussion around doing a "New Emails" aggregated virtual folder which will solve this issue. No timeline for that though.

Thx. Your reply was polled automatically just now, 10 mins since last poll. Even though it shown next poll 50 mins. I was browsing FB with Google Chrome. This time your 2nd reply notification stayed in android upper notifications bar.
Should I try older version apk. ??
Where can I find it. ??
Thx from Cam Gibb.
On 31 March, 2016 5:34:47 AM EDT, Philip<EMAIL_ADDRESS>wrote:

"When it was done polling, the note bar note showing new emails
vanished. It started doing this week or 2 ago."
That's clearly a separate issue, probably related to #834 I guess...
After/during the design overhaul there is discussion around doing a
"New Emails" aggregated virtual folder which will solve this issue. No
timeline for that though.

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/k9mail/k-9/issues/1235#issuecomment-203850631

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

I installed 2 mths back, version 5.008, over the newest that came out mths before. Dunno if it was 009 or 010. To see if new constant frequent polling issue ceased. It didn't. Ended up with duplicate folders. Tolerated for mths. Eventually found mention of ridding dup'd folders with "repopulate folders". Dup'd folders gone as of few days ago. Still sporadically polls every minute or mins or 15 or 30 despite set for 60. Not a huge deal I guess except lags system and screen response at times. Should I can I try version older then 008. ?? When first started usung it about 2-3 yrs ago it had no issues with this cell that came out around 2011.

I believe this is no longer an issue in master.


Scalarisations Bug

Modified and Augmented Chebychev have incorrect contour lines, some issue with normalisation probably.

The functions used incorrect upper and lower bounds after normalisation, creating incorrect contours. Its now fixed.


Make output transformer execute using async/await syntax

Simplifies transformer experience and removes [major] caveat of being able to use an non-async (but created as async) implementation.
Questions:

[ ] Name for TranformerMetadata key for the value_fn being originally sync/async. Different mechanism?

Included within #964. Closing


Feature Request: Add suggestion when typing.

Sorry for late answer!
Yes, this can and will be implemented in a future release. As for now, I've added a quick fix by implementing the prop customElement.
The component that is passed with customElement will be displayed between the TextInput and the list with tags. In this way you can include a component containing a list of search results from your database.

<TagInput
    updateState={this.updateTagState}
    tags={this.state.tags}
    customElement={<YourListWithSearchResults/>}
/>

I've also added an example to the repo on how to implement this by using customElement.

Leaving this issue open until this is implemented in a better way!

@jimmybengtsson thank you :), I will look into.

Can the 'suggestionsArr' be used with a scrollview? When used together, it behaves strangely on the part that it touches.
customElement={                         <ScrollView showsVerticalScrollIndicator={false}>                         <Block style={{ marginTop: 10 }}>                           {this.renderSuggestions()}                           </Block>                           </ScrollView>                       }


[bugfix] workaround if some SegmentMapping refers to end of tar

What this PR does / why we need it:
Workaround if some SegmentMapping refers to end of tar (because the mapping doesn't record real remote mapping size), which could happen on EROFS meta indexes.
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 #
Please check the following list:

[ ]  Does the affected code have corresponding tests, e.g. unit test, E2E test?
[ ]  Does this change require a documentation update?
[ ]  Does this introduce breaking changes that would require an announcement or bumping the major version?
[ ]  Do all new files have an appropriate license header?

As pread(2) says,
On success, the number of bytes read is returned (zero indicates end of file), and the file position is advanced by this number.
It is not an error if this number is smaller than the number of bytes requested; this may happen for example because fewer bytes are actually available right now (maybe because we were close to end-of-file, or because we are reading from a pipe, or from a terminal), or because read() was interrupted by a signal.

Before:

After:


⚠ Hartman Foundation has degraded performance

In 80f00ee, Hartman Foundation (https://gordonhartman.com/) experienced degraded performance:

HTTP code: 200
Response time: 2861 ms

Resolved: Hartman Foundation performance has improved in e3308b7.


Getting a really clear definition of what this actually is?

This is not a troll question I think I am genuinely missing something awesome here and I want to understand.
I notice you have made references back to eslint and how this actually formats your code, for javascript I use vim with vim-autoformat and XO (which is built on eslint similar to standard I believe).
What I am trying to understand is how Prettier differs from what I currently have? is it a more aggressive reformat? Can it do more?
Sorry I am just really trying to get a grasp of what is new.
Cheers.

Less/no config, autofix for everything Babel supports (like babel-eslint), however only deals with styling unlike ESLint (a linter + autofixer), accounts for max-line-length automatically

It's an alternative to enforcing style by configuring a lot of little "nits" in eslint, and instead just forcible re-printing everything. If you are using XO which is built on eslint, it's depending on eslint's --fix behavior which doesn't actually fix a wide array of style. For example the stricted eslint config I know happily passes all of these:
foo({ num: 3 },
  1, 2)

foo(
  { num: 3 },
  1, 2)

foo(
  { num: 3 },
  1,
  2
)

Prettier always ensures that 100% of your code is formatted consistently. I hope you try it out, and it's fine if it's not for you!

@jlongster thank you that is exactly what I was after :) And I will definitely be trying it!


Send data to a LEGO Hub

Great help. Managed to read data using am ESP32. Thank you!
Have you been able to send data to a LEGO Hub.
I have been able to advertise the exact same data, and my ESP32 sees it as the same, except for two details.
My data have a name "ESP32", which I so far haven't been able to get rid of. And my ESP32 have another address. But the advertisement data are the same.
I must be missing something, like some flags, or payload or scan response, something. Any help ... any luck?

Yes I have been able to both send and receive from other devices. In my case not ESP32.
What you need is to be able to set the full advertising data. For example sending data with an iPhone to a MINDSTORMS hub is also not possible since apple restricts app developers to not change the manufacturer ID which needs to be the LEGO ID.
Is this your issue as well? I expect there should be a mode on the ESP32 that allows to change this.
Do you have 2 ESP32 boards? What is the exact message received on one ESP32 board if you send with the other one? And how is it different to the message of a MINDSTORMS hub?

Yes, I have many ESP32 all kinds including ESP32C3 (BLE 5.0).
Sending from a LEGO hub and from an ESP32 to another ESP32, I get this:
Name: , Address: b0:b1:13:9b:8b:d7, manufacturer data: 97035f06d2e4fb61626364
Name: ESP32, Address: ec:94:cb:72:ed:ea, manufacturer data: 9703e606d2e4fb61626364, txPower: 3

Top line from LEGO Spike Hub, bottom from ESP32.
Here aligned for ease of reading:
Name:      , Address: b0:b1:13:9b:8b:d7, manufacturer data: 97035f06d2e4fb61626364
Name: ESP32, Address: ec:94:cb:72:ed:ea, manufacturer data: 9703e606d2e4fb61626364, txPower: 3

As you can see, it is only the message number that is different. The message ID is "signal111" and I have verified that I got the right CRC32, but also send it from the hub to the ESP32 to make sure.
Also the LEGO ID is correct in the manufacturer data. I haven't tried to change the address to a look alike address. There are also flags, etc that can be set, which I haven't.
Do you by any chance have a piece of working code, just a cut out of the source with the action ... in any programming language? That could help.

A clear. I guess the fact that you have the name and the tx power leads to the MINDSTORMS hub not recognizing the message.

I think what you need is to be able to set the RAW advertising data, i.e., set each bit and not only the manufacturer data. In micropython that can be done using: https://docs.micropython.org/en/latest/library/bluetooth.html#broadcaster-role-advertiser

I can send all those data in the ESP32 as well:
(interval_us, adv_data=None, *, resp_data=None, connectable=True)

Interval in uS
Adv_data, which I am sending now
Scan Response data (resp_data), which is typically the same, but I tried with and without.
And a flag for connectable, also set false, true, and not set.
but I don't know what those (other than adv_data) should be set to.

Can you set the adv_type? I remember now that when I was implementing this in pybricks on the technic hub I could not set it to the correct advertising type and hence the MINDSTORMS hub would not recognize the data correct. The correct advertising type  should be ADV_SCAN_IND.
I hope that insight helps?

Ok,
I finally got it to work:

Build a record similar to the one showed in this repository
Remember to change the message number, LEGO only responds to the first unique record
Don't set any flags
Make a ScanResponse with same content as the AdvertisementData
Set AdvertisementType to ADV_NONCONN_IND

@NStrijbosch, thanks for your insights


pact-jvm-provider-junit not published to Maven Central

http://mvnrepository.com/artifact/au.com.dius doesn't have the pact-jvm-provider-junit jar available. It seems like it should, as most of the other libraries are published there.

It is not published automatically. It will be included in the next release, which I'll push out this weekend.

Published

When will pact-jvm-provider-junit 2.11 be published next? I'd like to try out Stephan's @PactUrl and @PactBroker.

I'll push out a release this weekend

Great, thanks! in the mean time, I'm trying to build pact-jvm locally. Not all projects are building. The only ones that are building are:
pact-jvm-provider-gradle
pact-jvm-provider
pact-jvm-model
pact-jvm-matchers
I made the following change to the build.gradle file in order to avoid trying to publish to sonatype.org:
changed lines 127-141 to:
uploadArchives {
    repositories {
        mavenDeployer {
            beforeDeployment { def deployment -> signing.signPom(deployment) }
            mavenLocal()
        }
    }
}

I'm new to gradle, but I do not think this change is what's preventing all of the projects to build. I'm using the following command:
gradle publishToMavenLocal
Any ideas? I'm most interested in the project pact-jvm-provider-junit.
Thanks

If you just want to build it into your local maven repo, just run
gradle install
On Fri, 11 Dec 2015 10:47 GeorgeSimon<EMAIL_ADDRESS>wrote:

Great, thanks! in the mean time, I'm trying to build pact-jvm locally. Not
all projects are building. The only ones that are building are:
pact-jvm-provider-gradle
pact-jvm-provider
pact-jvm-model
pact-jvm-matchers
I made the following change to the build.gradle file in order to avoid
trying to publish to sonatype.org:
changed lines 127-141 to:
uploadArchives {
repositories {
mavenDeployer {
beforeDeployment { def deployment -> signing.signPom(deployment) }
mavenLocal()
}
}
}
I'm new to gradle, but I do not think this change is what's preventing all
of the projects to build. I'm using the following command:
gradle publishToMavenLocal
Any ideas? I'm most interested in the project pact-jvm-provider-junit.
Thanks
—
Reply to this email directly or view it on GitHub
https://github.com/DiUS/pact-jvm/issues/184#issuecomment-163788675.

Never mind. I ran gradle install and the projects built.


Error while installing arcgis-desktop-> No such cookbook: limits

No such cookbook: limits
Please see screenshot below for reference

Chef client 13 is not yet supported by ArcGIS Chef cookbooks. Please use Chef client 12. Also make sure that 'limits' cookbook is present in C:\chef\cookbooks folder.


SUBMARINE-1207. Move server-core/workbench/entity to server-api/workbench

What is this PR for?
Move files under submarine-server/server-core/src/main/java/org/apache/submarine/server/workbench/entity to submarine-server/server-api/src/main/java/org/apache/submarine/server/api/workbench.
What type of PR is it?
Refactoring
What is the Jira issue?
https://issues.apache.org/jira/projects/SUBMARINE/issues/SUBMARINE-1207?filter=allopenissues
Screenshots (if appropriate)
No
Questions:

Do the license files need updating? /No
Are there breaking changes for older versions? No
Does this need new documentation? No

The PR title should be SUBMARINE-1207 instead of Submarine 1207.

hi @Ray02250418
Please change PR title to SUBMARINE-1207. Move ...

@jeff-901 Please help me review the code.


Car port:  CADILLAC ESCALADE ESV PLATINUM 2019

Checklist

[x] added entry to CarInfo in selfdrive/car/*/values.py and ran selfdrive/car/docs.py to generate new docs*
[x] test route added to routes.py
[x] route with openpilot: 168f8b3be57f66ae|2023-09-12--21-44-42
[x] route with stock system: 168f8b3be57f66ae|2023-09-13--17-22-33

* cloned OP to linux machine, ran ubuntu_setup.sh, enter pipenv shell, installed missing python dependencies, ran scons and build successfully, then ran selfdrive/car/docs.py which throws:
(openpilot) haiiro@haiiro-pc:~/optw/openpilot/openpilot$ ./selfdrive/car/docs.py
Traceback (most recent call last):
  File "./selfdrive/car/docs.py", line 14, in <module>
    from openpilot.selfdrive.car.car_helpers import interfaces, get_interface_attr
  File "/home/haiiro/optw/openpilot/openpilot/selfdrive/car/car_helpers.py", line 6, in <module>
    from openpilot.common.params import Params
  File "/home/haiiro/optw/openpilot/openpilot/common/params.py", line 1, in <module>
    from openpilot.common.params_pyx import Params, ParamKeyType, UnknownKeyName, put_nonblocking, \
ImportError: /home/haiiro/optw/openpilot/openpilot/common/params_pyx.so: undefined symbol: PyGC_Disable

I figure I'm just missing something here, but I tried!

I drove two additional road tests comparing the prior version of the PR branch with the current version.
Here is a route using the prior version of the PR branch:
168f8b3be57f66ae|2023-09-23--14-43-46--0
Here is the same route using the current version of the PR branch:
168f8b3be57f66ae|2023-09-23--18-27-23--0

Will check it out in a bit, but which do you prefer?

I think there is a noticeable improvement with the newer version.
On Sat, Oct 7, 2023, 1:01 AM Shane Smiskol @.***> wrote:

Will check it out in a bit, but which do you prefer?
—
Reply to this email directly, view it on GitHub
https://github.com/commaai/openpilot/pull/29913#issuecomment-1751601681,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRH6P6H3UF7XSQIBTTYIHTX6DOZJAVCNFSM6AAAAAA4Y4TMW6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJRGYYDCNRYGE
.
You are receiving this because you commented.Message ID:
@.***>

thanks!


Cant get paste working!

Cant get paste working!
Would be nice if we could change java version as well!

That should be better now

thank you :)


Refactor tasks to use ListArtifact as input

Several Tasks have non-BaseArtifact input fields. For instance, ImageQueryTask.input has a type of:
tuple[TextArtifact, list[ImageArtifact]]

The complexity of this type makes it very difficult to serialize the inputs of these Tasks in downstream abstractions like Events. Additionally, it make the Task API inconsistent across Task types.
This PR changes the input type to ListArtifact, making it consistent with other Task types. Note that the setter for the field is mostly unchanged, only the getter has changed.

@andrewfrench thanks for the review, just fixed both of your comments.


chore(main): release 2.1.0

:robot: I have created a release beep boop
2.1.0 (2024-07-20)
Features

cli: support for multiple paths in any position (#562) (5ff880b)
support for config files (js and cjs) (#560) (368f396)
support for config files (json and jsonc) (#557) (e954336)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wellwelwel/poku/releases/tag/v2.1.0 :sunflower:


Apple 2-factor authentication (2FA): To set the application specific password on a CI machine using an environment variable, you can set the FASTLANE_APPLE_APPLICATION_SPECIFIC_PASSWORD variable.

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description

I searched and tried a lot of solutions but unfortunately nothing worked for me. I could use Fastlane to deploy new beta on TestFlight before 2FA. After Apple en-forced me to set 2FA, I could not use Fastlane to deploy new beta.
Command executed

fastlane beta
Complete output when running fastlane, including the stack trace and command used

[Transporter Error Output]: Sign in with the app-specific password you generated. If you forgot the app-specific password or need to create a new one, go to appleid.apple.com (-22938)

Transporter transfer failed.

Sign in with the app-specific password you generated. If you forgot the app-specific password or need to create a new one, go to appleid.apple.com (-22938)

Your account has 2 step verification enabled
Please go to https://appleid.apple.com/account/manage and generate an application specific password for the iTunes Transporter, which is used to upload builds

To set the application specific password on a CI machine using an environment variable, you can set the FASTLANE_APPLE_APPLICATION_SPECIFIC_PASSWORD variable

Environment

 🚫 fastlane environment 🚫
Stack

Key
Value

OS
10.14.3

Ruby
2.6.0

Bundler?
true

Git
git version 2.17.2 (Apple Git-113)

Installation Source
~/.rvm/gems/ruby-2.6.0/bin/fastlane

Host
Mac OS X 10.14.3 (18D109)

Ruby Lib Dir
~/.rvm/rubies/ruby-2.6.0/lib

OpenSSL Version
OpenSSL 1.1.1b  26 Feb 2019

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
10.1

System Locale

Error

No Locale with UTF8 found 🚫

fastlane files:
`./fastlane/Fastfile`
# This file contains the fastlane.tools configuration
# You can find the documentation at https://docs.fastlane.tools
#
# For a list of all available actions, check out
#
#     https://docs.fastlane.tools/actions
#
# For a list of all available plugins, check out
#
#     https://docs.fastlane.tools/plugins/available-plugins
#

# Uncomment the line if you want fastlane to automatically update itself
# update_fastlane

default_platform(:ios)

platform :ios do
  desc "Push a new beta build to TestFlight"
  desc "This will also make sure the profile is up to date"
  lane :beta do
    # UI.important "Important Message"
    # UI.error "O NO!"
    # UI.message "Basic blah"
    answer = UI.select("Would you like to add a Changelog?", ["Yes", "No"])
    changelog = ask('Type Changelog message:') if answer == "Yes"

    # Get latest certs appstore
    match(type: 'appstore')

    increment_build_number(xcodeproj: "citizenapp.xcodeproj")
    build_app(workspace: "citizenapp.xcworkspace", scheme: "citizenapp")

    if changelog
      upload_to_testflight(changelog: changelog)
    else
      upload_to_testflight
    end
  end
end

`./fastlane/Appfile`
app_identifier("com.parabol.citizen.konya") # The bundle identifier of your app
<EMAIL_ADDRESS># Your Apple email address

itc_team_id("118983516") # App Store Connect Team ID
team_id("65LUFY6YWC") # Developer Portal Team ID

# For more information about the Appfile, see:
#     https://docs.fastlane.tools/advanced/#appfile

fastlane gems

Gem
Version
Update-Status

fastlane
2.117.1
✅ Up-To-Date

Loaded fastlane plugins:

Plugin
Version
Update-Status

fastlane-plugin-appicon
0.14.0
✅ Up-To-Date

Loaded gems

Gem
Version

did_you_mean
1.3.0

executable-hooks
1.6.0

bundler-unload
1.0.2

rubygems-bundler
1.4.5

forwardable
1.2.0

CFPropertyList
3.0.0

public_suffix
2.0.5

addressable
2.6.0

atomos
0.1.3

babosa
1.0.2

bundler
1.17.2

claide
1.0.2

colored
1.2

colored2
3.1.2

highline
1.7.10

commander-fastlane
4.4.6

declarative
0.0.10

declarative-option
0.1.0

digest-crc
0.4.1

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20180417

dotenv
2.7.1

emoji_regex
1.0.1

excon
0.62.0

multipart-post
2.0.0

faraday
0.15.4

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

faraday_middleware
0.13.1

fastimage
2.1.5

gh_inspector
1.1.3

jwt
2.1.0

memoist
0.16.0

multi_json
1.13.1

os
1.0.0

signet
0.11.0

googleauth
0.6.7

httpclient
2.8.3

mime-types-data
3.2018.0812

mime-types
3.2.2

uber
0.1.0

representable
3.0.4

retriable
3.1.2

google-api-client
0.23.9

google-cloud-env
1.0.5

google-cloud-core
1.3.0

google-cloud-storage
1.16.0

json
2.2.0

mini_magick
4.5.1

multi_xml
0.6.0

plist
3.5.0

rubyzip
1.2.2

security
0.1.3

naturally
2.2.0

simctl
1.6.5

slack-notifier
2.3.2

terminal-notifier
1.8.0

unicode-display_width
1.5.0

terminal-table
1.8.0

tty-screen
0.6.5

tty-cursor
0.6.1

tty-spinner
0.9.0

word_wrap
1.0.0

nanaimo
0.2.6

xcodeproj
1.8.1

rouge
2.0.7

xcpretty
0.3.0

xcpretty-travis-formatter
1.0.0

fastlane-plugin-appicon
0.14.0

generated on: 2019-03-11

Yes, tried: FASTLANE_APPLE_APPLICATION_SPECIFIC_PASSWORD and FASTLANE_SESSION but could not solve this problem.
Thank you for helping :)

Do what the error messages says: Create an app specific password, set the FASTLANE_APPLE_APPLICATION_SPECIFIC_PASSWORD variable and it should work. If it doesn't, post the complete output when running the command (as the issue template asks for).

Hello @janpio,
I've already mentioned that I've done these steps and post the output on the issue. Even after set the specific password variable, still could not pass the 2FA with fastlane. That's the problem and that's why I posted the issue.

... and post the output on the issue

No you did not - you posted a tiny bit of the output, not the complete output.
(From your "Environment" information for example I know that part of the output would be a warning that you should define a locale that you ignored.)


Empty envvar not panic

fixes #43

Pull Request Test Coverage Report for Build #126

20 of 20 (100.0%) changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 82.878%

Totals

Change from base Build #122:
0.3%

Covered Lines:
1002

Relevant Lines:
1209

💛  - Coveralls


Obelisk visual bug.

Aye, the rotating item over the obelisk seems to do a weird rendering behavior. When I stand near to it it appers over the obelisk but as I start to walk away from it the rendered item will start to move out from the block into the the opposite direction. It's not a big deal when there is air around the obelisk but if you put a solid block behind it, the item will be rendered inside in that block.
The emerald rendered halfly in the block behind http://i59.tinypic.com/24m9e0g.png as I get further away it completly dissapers.
v<IP_ADDRESS>9

This is fixed in dev builds.