1.0 Discussion

Just creating this as a place holder for discussions pertaining to a 1.0, will formalise something eventually.
Things I've been thinking about:

Spies

Rough implementation in #283
API needs discussing and finalising

Docs

#90
@GrahamCampbell and @WouterJ have made some progress, looking to start on migrating to sphinx shortly

API

Whole public API needs examining and finalising

Stubs vs Mocks behaviours

IMO stubs should always be byDefault, i.e. you can override them
I would love to break the API completely, so that shouldReceive defaults to once, rather than zeroOrMoreTimes, along with a stub method, as I think the language is in conflict, but I think that's probably too much of a BC break.

Issues

We need to fix and/or close issues as necessary

User Feedback

Need to speak to user base to see what we're missing

API Verifying mocks

Above and beyond allowMockingNonExistentMethods
Checks against method signature (for primitives and arg count etc)
Checks against docblock return type
Could be (configurably) silent when class/interface doesn't exist
Could be (configurably) silent when method doesn't exist

Newly added:

PHP version support

PHP 5.3.x is EOL
See #297

Coding Standards
Review state of matchers #129

I think strictness with regards to objects with($obj) and with(mustBe($obj)) are backwards compared to primitives.

@davedevelopment Should we consider moving the list to a set of separate issues under a new label?

@padraic I think that would be a good idea, if indeed we agree on a list of things. This really was just me brain dumping things :)

I did a few new ones already, being full of sugar and caffeine ;). I'd suggest creating them as RFC issues with the Towards 1.0.0 label and we'll see how they play out.


MTC Generator

MTC Generator development in progress.

MTC.Encoder is done, by and large.
MTC.Generator wraps MTC.Encoder and will manage the abstraction to locate/start/stop MTC MIDI event streams. This should be ready for field testing in the next couple days ideally.

Sync test harness has been updated and preliminary field testing is solid.
Just doing some final testing and documentation updates now.
Will push a new release to master branch soon.

MTC.Encoder and MTC.Generator added in PR #8 and new 0.1.5 release pushed to master.

Feature tests well. A few improvements and bugfixes will be added but this thread can be closed now.


Add tests to get_infra_failure_info

What does this PR do?

Add tests to get_infra_failure_info
Motivation

I had some errors in the CI:

https://gitlab.ddbuild.io/DataDog/datadog-agent/-/jobs/598354063
https://gitlab.ddbuild.io/DataDog/datadog-agent/-/jobs/598354059
https://gitlab.ddbuild.io/DataDog/datadog-agent/-/jobs/598354043

And I wanted to try test the function easily so I added some tests
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Add play_hurt_animation mech

play_hurt_animation mech
Flashes an entity red and can also rotate the player's camera in a specified direction.
Also add note in the animate command's meta about this mechanism being available to specify the damage direction for 1.20+
Tested on versions 1.20.1 and 1.19.4 (to be sure it won't be present and won't cause issues)
Requested by pigler

Done


Add react-a11y-proptypes rule.

Hi @t-ligu, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Liubin Guo). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

@HamletDRC I added react-a11y-proptypes rule.


support MODE= in MSGBOX

temp = msgbox("Hello from BBj's " + client$ + " client!", BBjSysGui.MSGBOX_ICON_INFORMATION, "Greetings", MODE="theme=primary")

@StephanWald
Duplicate of  #45 ?


Player clips through the walls on 30 FPS

Steps to reproduce

Build game
$ ./build/nothing --fps 30 ./levels/level-01.txt
Just mess around

Observed

Expected
Player doesn't clip through the walls that easily.

Fixed in #231


how to force to update collection of ColumnSeries

Bug or Feature Request?
N/A
Description
how to force to update collection of ColumnSeries looks like it push it by default when state changed
How to reproduce
Live demo demonstrating bug

Versions

React JSX Highcharts version: latest
Highcharts version:  latest
React version: latest

Would you mind creating a demo of the problem? I'm not quite sure what you mean.

I've created the app
https://stackblitz.com/edit/react-iq27yr
if you uncheck/check [02pm]  checkbox it will disappears and appears in the end on series.
If you uncheck/check [09am]  checkbox it will disappears and appears in the end on series BUT I am expecting to be appear in the start of series (in the place it was before unchecked)
Did I miss smth or dont understand smth? Thanks

Added demo
Please review
From: Will Hawker<EMAIL_ADDRESS>Sent: September-19-18 8:05 AM
To: whawker/react-jsx-highcharts<EMAIL_ADDRESS>Cc: Vladimir Vorobiev<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [whawker/react-jsx-highcharts] how to force to update collection of ColumnSeries (#154)
Would you mind creating a demo of the problem? I'm not quite sure what you mean.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/whawker/react-jsx-highcharts/issues/154#issuecomment-422776707, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ac1zm1iIJp66zAgtd-qvNK5FK20As7HBks5ucjLngaJpZM4WvBbT.

Hmm I would suggest using the series visible and showInLegend properties instead of an array filter?
React JSX Highcharts automatically adds/removes a series when a <Series /> component is mounted/unmounted.
Instead of
arr.filter(c => c.enabled)
  .map(item => (
    <ColumnSeries
      key={item.sortOrder}
      //etc
    />
  ))

Try
arr.map(item => (
    <ColumnSeries
      key={item.sortOrder}
      visible={item.enabled}
      showInLegend={item.enabled}
      //etc
      />
  ))

did not help
change the demo - behavior is the same

sorry it works


{Freeway-Bug}[Adaptive Cards - Product Video] - Caption does not define for video player under 'Product Video'.

Test Environment:
Device: iPhone 11
iOS Version: 16.7.1
App Version: 1.0 (2.4.0-beta.<IP_ADDRESS>)
Screen Reader: Voice Over
Repro Steps:

Launch Adaptive Cards iOS application.
Home screen will appear.
Activate Voice over
Navigate to "V1.6" control and select it.
Navigate to 'Scenario' control and activate it.
Navigate to 'Product Video.Json' and activate it.
Card will appear.
Navigate to video and select 'Play button. Navigate all the video controls.
Verify the issue.

Actual Result:
Caption does not define for video.
Expected Result:
Caption should be defined for pre-recorded video.
User Experience:
If Caption does not define on video. so, it would be difficult for users to know about video and purpose of video.
WCAG Reference:
https://www.w3.org/WAI/WCAG21/Understanding/captions-prerecorded
"Have feedback to share on Bugs ? Please help fill Trusted Tester Bug Feedback (office.com) “
https://github.com/microsoft/AdaptiveCards-Mobile/assets/95913748/06c24afd-bdb2-49d0-95ee-fde32aea010e

Hi @ardlank,
Hope you are doing great.
Could you please provide us ETA to fix this issue as due to this issue product is getting non-compliant.

Hey @vagpt, apologies for the delay on the update. We had to do some more investigation on the background for the issue and the existing behavior.
This actually isn't a bug right now. The native controls support captions however the video source needs to have the captions encoded in it. At that point the native video controls will update to support the caption option in the area you showed.
The video provided in the adaptive card test case does not have the captions encoded, and instead provides them in a different file. Processing separate caption files from the video source will be considered a feature enhancement.
We'll shift this to a separate feature to provide support for parsing and leveraging provided caption files with videos on iOS.

Discussed with Arvind Prakash and Ajay Sharma. Will be treated as feature item as is part of 3454571 work item.


DomeViewer - Starting Miniature View leads to dizzying non-vr view

Platform: Windows & VR Headset
When choosing Miniature View from the Menu during a VR Session, the camera image gets stuck to the head and the user is in a non recoverable state

is fixed
https://github.com/prefrontalcortex/DomeTools/pull/5


Updated status bar

Added status for the selected items count
Added the size of the selected item (right now it only shows the size if one item is selected)

Is it possible to also display how many items are in the current directory?

@XPoppyX It is, that just hasn't been implemented on an level yet so it will come in a future update.


[OpenAI.Inference] Adding functions support.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.AI.OpenAI


⚠ Bofrost.lu has degraded performance

In 1557bc8, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1223 ms

Resolved: Bofrost.lu performance has improved in 3770cf0 after 31 minutes.


Is there an off() functionionality?

Im user 'on' to bind select event.
wdtEmojiBundle.on('select', function (event) {
	XXXXXXXX
});

can do I unbind it?
is there an off() function?
Thanks.

is there a way to 'destroy' the emoji picker?

Hımm good point. Currently, we don't have that functionality.

I need this functionality so when it would be added?

You can write your own simple function and place it anywhere in the code. e.g.:
    wdtEmojiBundle.off = function (eventName, handler) {
        switch (eventName) {
            case "select":
                return wdtEmojiBundle.dispatchHandlers.select = [];
                break;
            case "afterSelect":
                return wdtEmojiBundle.dispatchHandlers.afterSelect = [];
                break;
            case "afterPickerOpen":
                return wdtEmojiBundle.dispatchHandlers.afterPickerOpen = [];
                break;
            default:
                console.error('wdt-emoji-bundle - Not supported event type!', eventName);
                break;
        }
    };

and just call it like:
 wdtEmojiBundle.off('afterSelect')

The function is a modified version of wdtEmojiBundle.on()


[Podspec] update podspec for 0.92.0

This is needed to push 0.92.0 to trunk because the documentation_url needs to point to https://realm.io/docs/objc/#{s.version}. /cc @segiddins

:+1:


Cluster not repainted for new bounded zoom level after fitbounds

I am calling fitbounds to center the map after marker creation. The issue is the clusters will be initially drawn as if the zoom level were before the fitbounds zooms in/out the map view. Zooming manually after will trigger cluster repaint and fix the issue. So if the map were zoom 3 initially, and the fitbounds zooms to 6, the clustering behaves as if it were still zoom 3.
EDIT: Actually, this issue happens even if I don't use fitbounds and use zoom().
var locations;
clearLocations();
$.ajax({
	type:'GET',
	url:'url.php',
	dataType: "json",
	data:{
		region:region
	},
	success:function(data){

		markerNodes = data;
		//total results for counter total number
		totalResults = markerNodes.length;

		for (i = 0; i < markerNodes.length; i++) {

			var id = markerNodes[i].id;
			var name = markerNodes[i].company;
			var distance = '';
			var latlng = new google.maps.LatLng(
				parseFloat(markerNodes[i].lat),
				parseFloat(markerNodes[i].lng)
			);

			//create marker and event listener attach function
			createMarker(id, latlng, name, distance, i);
			bounds.extend(latlng);

		}

		markerClusterer = new MarkerClusterer(map, markers, {imagePath: 'img/maps/m'});
		markerClusterer.setMaxZoom(minClusterZoom);

		if (bounds.getNorthEast().equals(bounds.getSouthWest())) {
			var extendPoint1 = new google.maps.LatLng(parseFloat(bounds.getNorthEast().lat() + 0.001), parseFloat(bounds.getNorthEast().lng() + 0.001));
			var extendPoint2 = new google.maps.LatLng(parseFloat(bounds.getNorthEast().lat() - 0.001), parseFloat(bounds.getNorthEast().lng() - 0.001));
			bounds.extend(extendPoint1);
			bounds.extend(extendPoint2);
		}

		map.fitBounds(bounds);
		//markerClusterer.repaint();

	}
});

I have the same issue, this is my workaround for now:
map.fitBounds(bounds)

setTimeout(() => {
  markerCluster.resetViewport()
  markerCluster.redraw()
})

@CrocoDillon That didn't quite work for me, but using your snippet I did figure out something for my use case.
google.maps.event.addListenerOnce(map, 'idle', function() {
	markerClusterer.resetViewport()
	markerClusterer.redraw()
});

Basically just calls the redraw on map idle. Would have been ideal if fitBounds had a callback.

Phew, glad I found this issue! Took me a while to figure out what was going wrong.
Here is the code I used, which is a combination of the code snippets provided by @CrocoDillon & @mikemania
google.maps.event.addListenerOnce(map, 'idle', function() {
  markerClusterer.repaint();
});


Switch deployment providers from Vercel to Netlify 🎉

Resolves #697.

As part of this PR, I transferred the required environment variables to Netlify.

As part of this PR, I gave the keys to Netlify to handle the acmcsuf.com domain.


Fix indent, typo and modify some mistake

chore: fix markdown style and remove unused space
feat: add missing part
fix: typo 'krik' to 'kirk'
fix: fix indent and use space instead of tab

Thanks!


Timeline to stable?

Hi,
I came across Coraza and this SPOE while looking for a HAProxy CE WAF.
It seems like the project is very active, with a full rewrite just two weeks ago (https://github.com/corazawaf/coraza-spoa/pull/103), but has been looking for a maintainer for almost two years.
Will this project be usable / production-ready in the coming few months? As ModSecurity is no longer supported starting this year, I can imagine that this project will become much more active.

Just so we leave it somewhere, ModSecurity has moved to a new stewardship below OWASP. See https://www.trustwave.com/en-us/resources/blogs/spiderlabs-blog/trustwave-transfers-modsecurity-custodianship-to-the-open-worldwide-application-security-project/


Test QML tutorials against latest PL master

Website build
Commit: 70bdae4307e2938f795b95e8cade8c654240ed44
Circle build number: 936
Built demos directory: https://936-214003948-gh.circle-artifacts.com/0/tmp/qml_demos.zip
Backreferences zip: https://936-214003948-gh.circle-artifacts.com/0/tmp/qml_backreferences.zip
Website zip: https://936-214003948-gh.circle-artifacts.com/0/tmp/qml_html.zip
Website url: https://936-214003948-gh.circle-artifacts.com/0/_build/html/index.html
Please double check the rendered website build to make sure everything is correct.


Add partial indexing support for skipping index

Description

Enable WHERE clause support for skipping index
a. Remove old UT, add more logging
b. Restrict column must be partitioned
Skipping index query rewrite
a. Enforces hybrid scan per query if skipping index is partial
b. Fix hybrid scan join bug: change from LEFT OUTER to ANTI SEMI join between source file list and index data
Restrict filtering condition on covering index be conjunction too (in case we want to support query rewrite in future)

Documentation: https://github.com/dai-chen/opensearch-spark/blob/add-where-clause-for-skipping-index/docs/index.md#skipping-index
Create Partial Skipping Index Test
Create skipping index with filtering condition on non-partitioned column should fail:

spark-sql> CREATE SKIPPING INDEX ON ds_tables.http_logs
         > (status VALUE_SET)
         > WHERE status = 200
         > WITH (
         >   auto_refresh=true
         > );

java.lang.IllegalArgumentException:
    requirement failed: status is not partitioned column and cannot be used in index filtering condition

Create skipping index with disjunction filtering condition (OR) should fail:

spark-sql> CREATE SKIPPING INDEX ON ds_tables.http_logs
         > (status VALUE_SET)
         > WHERE year=1998 OR month=6
         > WITH (
         >   auto_refresh=true
         > );

java.lang.IllegalArgumentException:
    requirement failed: filtering condition Some(year=1998 OR month=6) must be conjunction

Create skipping index with filtering condition on partitioned column should succeed:

CREATE SKIPPING INDEX ON ds_tables.http_logs
(status VALUE_SET)
WHERE year=1998 AND month=6 AND day >= 10
WITH (
  auto_refresh=true,
  extra_options = '{"myglue.ds_tables.http_logs": {"maxFilesPerTrigger": "10"}}'
);

POST flint_myglue_ds_tables_http_logs_skipping_index/_search
      {
        "_index": "flint_myglue_ds_tables_http_logs_skipping_index",
        "_id": "a4245e7651b50fe6e274e2e27f9c9808b0ce3d05",
        "_score": 1,
        "_source": {
          "file_path": "s3://daichen-benchmark/httplogs/http_logs_partitioned_json_bz2/year=1998/month=6/day=10/part-00383-76cb51e0-1b8f-41ea-8bfd-e77261483002.c000.json.bz2",
          "status": [
            206,
            200
          ]
        }
      },
      {
        "_index": "flint_myglue_ds_tables_http_logs_skipping_index",
        "_id": "199e16b74b231ad41156da61c95608c1232a9ee6",
        "_score": 1,
        "_source": {
          "file_path": "s3://daichen-benchmark/httplogs/http_logs_partitioned_json_bz2/year=1998/month=6/day=10/part-00385-76cb51e0-1b8f-41ea-8bfd-e77261483002.c000.json.bz2",
          "status": [
            206,
            400,
            200
          ]
        }
      },
      ......

Skipping Index Query Rewrite Test
Query rewrite should enforce hybrid scan and thus include unknown files in skipping index:

spark.sql("SELECT `@timestamp`, clientip, status FROM ds_tables.http_logs WHERE status = 500").explain

23/11/10 19:16:22 INFO ApplyFlintSparkSkippingIndex: Applying skipping index rewrite rule on filter condition Filter (status#4511 = 500)
+- Relation ds_tables.http_logs[@timestamp#4508,clientip#4509,request#4510,status#4511,size#4512,year#4513,month#4514,day#4515] json

23/11/10 19:16:22 INFO FlintSpark: Describing index name flint_myglue_ds_tables_http_logs_skipping_index
23/11/10 19:16:22 INFO FlintOpenSearchClient: Checking if Flint index exists flint_myglue_ds_tables_http_logs_skipping_index
23/11/10 19:16:22 INFO FlintOpenSearchClient: Fetching Flint index metadata for flint_myglue_ds_tables_http_logs_skipping_index

23/11/10 19:16:22 INFO ApplyFlintSparkSkippingIndex: Found skipping index Some(FlintSparkSkippingIndex(myglue.ds_tables.http_logs,WrappedArray(ValueSetSkippingStrategy(VALUE_SET,status,int)),Some(year=1998 AND month=6 AND day >= 10),FlintSparkIndexOptions(Map(auto_refresh -> true, extra_options -> {"myglue.ds_tables.http_logs": {"maxFilesPerTrigger": "10"}}))))
23/11/10 19:16:22 INFO ApplyFlintSparkSkippingIndex: Found filter condition can be pushed down to skipping index: Some(('status = 500))

== Physical Plan ==
*(1) Project [@timestamp#4508, clientip#4509, status#4511]
+- *(1) Filter (isnotnull(status#4511) AND (status#4511 = 500))
   +- FileScan json ds_tables.http_logs[@timestamp#4508,clientip#4509,status#4511,year#4513,month#4514,day#4515] Batched: false, DataFilters: [isnotnull(status#4511), (status#4511 = 500)], Format: JSON,
Location: FlintSparkSkippingFileIndex(1 paths)[s3://daichen-benchmark/httplogs/http_logs_partitioned_json_bz2], PartitionFilters: [], PushedFilters: [IsNotNull(status), EqualTo(status,500)],
ReadSchema: struct<@timestamp:timestamp,clientip:string,status:int>

Verified that partial skipping index can hybrid scan source file not included in index data (day=9)
spark-sql> SELECT `@timestamp`, clientip, status FROM ds_tables.http_logs WHERE year = 1998 AND month = 6 AND day >= 9 AND status = 404 LIMIT 5;
1998-06-10 01:04:01	<IP_ADDRESS>	404
1998-06-10 04:48:48	<IP_ADDRESS>	404
1998-06-10 16:07:35	<IP_ADDRESS>	404
1998-06-10 19:27:09	<IP_ADDRESS>	404
1998-06-11 00:42:28	<IP_ADDRESS>	404
Time taken: 31.411 seconds, Fetched 5 row(s)

spark-sql> SELECT `@timestamp`, clientip, status FROM ds_tables.http_logs WHERE year = 1998 AND month = 6 AND day >= 9 AND status = 404 ORDER BY `@timestamp` LIMIT 5;
1998-06-09 00:00:00	<IP_ADDRESS>	404
1998-06-09 00:00:00	<IP_ADDRESS>	404
1998-06-09 00:00:03	<IP_ADDRESS>	404
1998-06-09 00:00:03	<IP_ADDRESS>	404
1998-06-09 00:00:04	<IP_ADDRESS>	404
Time taken: 125.703 seconds, Fetched 5 row(s)

Issues Resolved
https://github.com/opensearch-project/opensearch-spark/issues/89
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Will resolve the conflicts and reopen for review later.


chore(deps): update common-dev-assets digest to 00b5683

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
63e86bc -> 00b5683

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

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

GitHub release
v1.4.5

Your semantic-release bot :package::rocket:


TreeView does not adjust its height when you open or close subtrees

I notice that when I put two TreeViews in a single vertical BoxLayout, they overlap.  This is because the height of a TreeView is always 100 pixels, regardless of how many nodes are open, regardless of the actual height of the view.

Still a problem in February of 2018.

@janssen how about a example to reproduce the issue?

I could write an example, but this is kind of a conceptual issue; I'm not sure that would help.  I'd wager that most TreeView widgets on a large data space (lots of leaf nodes) are deployed inside a ScrollView.  And ScrollView tracks height, not minimum_height.  So there's a lot of code containing some variant of this line:
            mytreeview.bind(minimum_size=lambda w, size: w.setter('height')(w, size[1]))

I think it's a bug to force programmers to keep having to write very similar lines of code.  Perhaps there should be three option in the constructor, "track_minimum_size=False", "track_minimum_height=False, "track_minimum_width=False", which would automatically add these bindings.


[BUG] Cannot enter full screen mode in Android

Cannot enter full screen mode in Android.

can you provide additional information about your bug report?

this issue is still persistent, is there any progress on this issue? and why does this happen any idea??


Integrated more functions into Twig

Now some functions are moved from ItemInterface to utils and other classes, there is no way they can be used inside a template.
This PR adds back some functionality.
As we need this to finish our KnpMenu 2.0 update for the Symfony CMF project, can this please be reviewed and merged quickly and be included in a new minor version?

ping, both PRs are now updated.

Sorry to bump again, but we need this PR to be merged to move towards releasing CMF 1.3.

Thanks for the review @stof, I've fixed your comments.

@WouterJ can you add tests covering Helper::getBreadcrumbsArray too ?

@stof done

anything more needed here?

LGTM

@WouterJ thanks!

Just looking through this and wondering if this will be in a tag release any time soon?

ping @stof @akovalyov for a new release tag.

I think @stof will work on it in the next days.

Otherwise, we can totally release a new tag, if it's ok with all of you.

@docteurklein :+1:


HTTP publishing body size limitation

Hello.
We are using pushpin for real-time updates in the last version of our product Tracim.
During our tests we got unpublished messages when their size is big. After investigation the problem is linked to be HTTP publishing which limits the content's body size. If I'm not wrong, it is this line https://github.com/fanout/pushpin/blob/e1312a54c0db0a4e381583de6b5e8d3708c67204/src/handler/simplehttpserver.cpp#L38
Publishing the same content through 0mq is fine.
Is the HTTP limit a "hard" one, meaning that pushpin is not meant to handle bigger contents? Or could it be made configurable?

I suppose this ought to be made configurable, and the size limits between HTTP and ZeroMQ inputs should be more consistent. However, large messages are not recommended as they can slow down queues. For comparison, Fanout's cloud service has a 64kB size limit.
Thought experiment: if you had to pick a different size limit and couldn't change it later (meaning it ought to suit your current needs and any future ones), what would you choose?
Also, what transport is being used by the subscribers?

OK, it was not our primary intent to publish large messages, more a side-effect of our time constraints.
Currently our biggest messages are around 4MB but we'll likely change their structure to not include all data which would make their size drop around 1kB. We are using Server-Sent Events as transport.
In the mean time we can use 0mq although I would have preferred to stick with HTTP publishing to avoid 0mq's persistent connections between our backend and pushpin. If you're willing to accept a PR making the HTTP body size configurable I can have a look at it.
Is the main effect of  "slowing down queues" a delay in message publishing?

With the HTTP streaming transport (which you are using if you're doing SSE), it is possible to use reliable streaming to set a URL to fetch data from and then publish a hint instead of the actual data. This will cause Pushpin to fetch the data from the URL and send it to the client, in which case the data has no size limit.
However, it might be easier to just increase the size limit. A PR for that would be great.
By "slowing down queues" I mean if you add a broker layer between your publishing code and Pushpin. Not a problem now but something to be aware of as you grow.

Thank you for the additional information.
I just created a PR to add configuration for http max body size and max headers size.


swarm label constaint is not working

Hi I am using swarm with compose to deploy two container into two different host. Where the docker daemon on each host is configured with different label. I am also using consul.
I got the error ERROR: unable to find a node that satisfies x.storage==ssd
There exist a machine with this label, but swarm is not finding the machine.
The docker compose yml file is:
app:
  image: flask/app
  links:
    - redis
  #environment:
   # - "constraint:x.pci==fpga"
app2:
  image: flask/app
  links:
    - redis
  environment:
    - "constraint:x.storage==ssd"
haproxy:
  image: tutum/haproxy
  links:
    - app
    - app2
  environment:
    - BACKEND_PORT=5000
nginx:
  image: nginx/loadbalancer
  ports:
    - "80:80"
  links:
    - haproxy
redis:
  image: redis

docker info
Containers: 3
 Running: 2
 Paused: 0
 Stopped: 1
Images: 8
Role: primary
Strategy: spread
Filters: health, port, dependency, affinity, constraint
Nodes: 2
 host1: <IP_ADDRESS>:2376
  └ Status: Healthy
  └ Containers: 2
  └ Reserved CPUs: 0 / 8
  └ Reserved Memory: 0 B / 12.32 GiB
  └ Labels: executiondriver=native-0.2, kernelversion=3.19.0-25-generic, x.storage=ssd, operatingsystem=Ubuntu 14.04.3 LTS, provider=generic, storagedriver=aufs
  └ Error: (none)
  └ UpdatedAt: 2016-02-17T19:49:07Z
 host2: <IP_ADDRESS>:2376
  └ Status: Healthy
  └ Containers: 1
  └ Reserved CPUs: 0 / 4
  └ Reserved Memory: 0 B / 16.45 GiB
  └ Labels: executiondriver=native-0.2, kernelversion=3.19.0-25-generic, x.pci=fpga, operatingsystem=Ubuntu 14.04.3 LTS, provider=generic, storagedriver=aufs
  └ Error: (none)
  └ UpdatedAt: 2016-02-17T19:49:29Z
Plugins:
 Volume:
 Network:
Kernel Version: 3.19.0-25-generic
Operating System: linux
Architecture: amd64
CPUs: 12
Total Memory: 28.77 GiB
Name: host2

docker version
Client:
 Version:      1.10.0
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   590d5108
 Built:        Thu Feb  4 18:36:33 2016
 OS/Arch:      linux/amd64

Server:
 Version:      swarm/1.1.0
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   a0fd82b
 Built:        Thu Feb  4 08:55:18 UTC 2016
 OS/Arch:      linux/amd64

This might be related to #1785.

@nishanttotla  Hi, thanks for the reply. Yes, it will be very useful to debug the problem.
I my case I have nothing running in the cluster; thus it should not have conflicts.
Do you have a swarm image with this modifications? It would be very helpful!

@marceloamaral try this branch https://github.com/nishanttotla/swarm/tree/list-conflicting-constraints
Although if you can wait for a bit for #1796 to get merged, you'll be able to use Swarm latest, which has some bug fixes.

@nishanttotla thank you! I will close this issue.

@marceloamaral thanks! Just checking, did you try out that branch? If so, did it help?

sorry @nishanttotla  I didn't try the branch yet. But the discussion itself was very useful. I could try to track the problem and I am sure that this modification will be very useful to debug.

Great, thanks @marceloamaral!


RuntimeError: Failed to parse Dictionary from the input stream

Hi, I have a trained model that sort of works fine but I want to improve its accuracy by retraining on new images through VoTT.
Here is what I tried.

Downloaded the app and ran VoTT.exe

Open a directory of images and added tags in the textfield.

Tag few images and export them

Ctrl+R to add model path - I have a trained model in c:\p\model\frcnn_svm.model along with a file name model.json with all the tags in it.

Click on review and bang! This is what I get from the developer console
            Error: CNTK Process failed with error code 1
           Error output:Traceback (most recent call last):
            File "C:\VoTT\resources\app\node_modules\cntk-fastrcnn\src\frcnn_detector.py", line 364, in <module>
           detector.load_model()
            File "C:\VoTT\resources\app\node_modules\cntk-fastrcnn\src\frcnn_detector.py", line 111, in   load_model
             trained_frcnn_model = load_model(self.__model_path)
            File "c:\local\Anaconda3-4.1.1-Windows-x86_64\envs\cntk-py35\lib\site-   packages\cntk\internal\swig_helper.py", line 69, in wrapper
             result = f(*args, **kwds)
             File "c:\local\Anaconda3-4.1.1-Windows-x86_64\envs\cntk-py35\lib\site-packages\cntk\ops\functions.py", line 1599, in load_model
             return Function.load(model, device)
             File "c:\local\Anaconda3-4.1.1-Windows-x86_64\envs\cntk-py35\lib\site-packages\cntk\internal\swig_helper.py", line 69, in wrapper
             result = f(*args, **kwds)
              File "c:\local\Anaconda3-4.1.1-Windows-x86_64\envs\cntk-py35\lib\site-packages\cntk\ops\functions.py", line 1513, in load
             return cntk_py.Function.load(model, device)
          RuntimeError: Failed to parse Dictionary from the input stream.

   [CALL STACK]
        > CNTK::operator>>
        - CNTK::Function::  Load
        - PyInit__cntk_py (x2)
       - PyCFunction_Call
       - PyEval_GetFuncDesc
       - PyEval_EvalFrameEx (x2)
       - PyFunction_SetAnnotations
       - PyObject_Call
       - PyEval_GetFuncDesc
       - PyEval_EvalFrameEx (x2)
       - PyEval_GetFuncDesc (x2)
       - PyEval_EvalFrameEx

        Selected GPU[0] Tesla K80 as the process wide default device.

          at ChildProcess.proc.on (C:\VoTT\resources\app\node_modules\cntk-fastrcnn\src\python_eval_client.js:82)
           at emitTwo (events.js:106)
           at ChildProcess.emit (events.js:191)
          at Process.ChildProcess._handle.onexit (internal/child_process.js:215)
           C:\VoTT\resources\app\src\lib\detection_algorithms\cntkfastrcnn\reviewer.js:65 undefined
           internal/process/next_tick.js:103 Uncaught (in promise) undefined
          _tickCallback @ internal/process/next_tick.js:103

Also I don't find a file named CNTK-Config.json under resources/app folder to specify the CNTK path. So I created a new file with the same name and added my CNTK path but it didn't pick. So I finally had top copy the CNTK folder in local.
Can you help me with the issue ?

VoTT Currently doesn't support SVM for evaluation since the requires extra pipelining I recommend Iterating without the SVM component and adding it back in when you go to production since the goal of the SVM component is speed performance. Always open to a pull request on that issue

I did try to load c:\p\model\frcnn_nn.model, one without SVM but I still get the same error. Any ideas ?

My bad! I had refresh the app. Its working now.
One more question:
How do I retain after exporting the tags ?
Should I place this exported tags and images in the existing positive and testImages directory ? Or I only retrain only on the exported tags by computing ROIs ?

Glad that helped. To retrain you should merge the directories.
Best of luck,
-Ari


Target Java 11 for extensions

This sets the source and target compatibility to 11 in the buildExtension.gradle script. This will help extension developers from unknowingly targeting a different version than what ghidra is targeting.

This should not be hard code to 11.  Preferably we should feed off of the setting established by the applications.properties.  See loadApplicationProperties.gradle and javaProject.gradle which uses the JAVA_COMPILER property.

This should not be hard code to 11. Preferably we should feed off of the setting established by the applications.properties. See loadApplicationProperties.gradle and javaProject.gradle which uses the JAVA_COMPILER property.
This could be done by using ghidraProps.getProperty('application.java.compiler ')

This makes much more sense so it doesn't need to be changed in the future. I'm also assuming the trailing space was a mistake correct?

This should not be hard code to 11. Preferably we should feed off of the setting established by the applications.properties. See loadApplicationProperties.gradle and javaProject.gradle which uses the JAVA_COMPILER property.
This could be done by using ghidraProps.getProperty('application.java.compiler ')

@ghidra1 for clarification this would use the version ghidra was built with and not the java version gradle is running in or whatever the user has installed correct?

That's correct.  It pulls the value from the application.properties file.

That's correct.  It pulls the value from the application.properties file.

Excellent. Is it a string? If not I think I have to convert it first.

Yes, looks like it's a string.  This is how it is used in javaProject.gradle:
plugins.withId('java') {
	sourceCompatibility = "${rootProject.JAVA_COMPILER}"
	targetCompatibility = "${rootProject.JAVA_COMPILER}"
}

When the GhidraDev plugin creates a new Ghidra project, it sets the compiler level based on the linked Ghidra installation's application.properties file.


Fixes to library to parse<EMAIL_ADDRESS>(comments<EMAIL_ADDRESS>

There are a lot of mail addresses written in the form<EMAIL_ADDRESS>(comments<EMAIL_ADDRESS>Currently library does not parse them correctly. (it would output undefined when parsing this name)
This is causing some issues with pgp (pretty good privacy) realisations:
https://github.com/openpgpjs/openpgpjs/issues/918
In this pull request I fix this behaviour to correctly parse displayName with @. Secondly comments are correctly parsed and outputted as concatenated string.

@jackbearheart gentle ping. Thanks!

This library aims to implement RFC 5322, while OpenPGP clients are more lenient than that. I've created a fork using these commits at https://github.com/openpgpjs/email-addresses, but it might not be appropriate to include them in the original.

Thanks for the PR! I appreciate you folks wanting to contribute this upstream if possible.
twiss is correct in my intent here. This PR as is isn't right for this library because it would change the default behavior to be non-rfc compliant. What we could do is add an option acceptAtSignInDisplayName (or something more concise) that enables this. There are some other options like this floating around.

ok, I will look at the option path.

@chesnokovilya concatenating comments like you did seems reasonable

I have updated PR. Now this feature is behind option atInDisplayName which is false by default.
@jackbearheart any other comments?

@chesnokovilya Great, thank you! I haven't had the chance to merge and release this yet, but from a brief look I think it is good. I'll do this when I have a moment.

Great

@jackbearheart Any chance this can be merged and a new release out out soon, please?

Yes, I'll do that now. Apologies for the delay...

I published this in version 3.1.0. I made some minor changes to preserve backwards compatibility. Thank you!

Awesome, thank you!


genrsa is not really deprecated although documented so

The notice from openssl-genrsa(1) manpage should be removed.

I'll fix it and open a pull request.

@t8m The history section of genrsa is only comprised of the statement: "This command was deprecated in OpenSSL 3.0.".
Removing it will make it so it will be empty. Is that fine or should something else be added there, if it is to be added then please let me know as I'm not versed enough to do so myself.

Replace the statement with This command was erroneously marked as deprecated in OpenSSL 3.0. It is not deprecated.

Done.
First issue fixed! Took longer than I expected (forgot to fork, then fixed the change in a different branch in the fork 🙈).

It was deprecated deliberately.  The erroneously part isn't correct.
I do agree with keeping the command and undeprecating it.

@paulidale No, it was not deprecated. There was originally a decision to deprecate all these gendsa/dsa/rsa/genrsa and similar commands but none of them was in the end deprecated for good reasons. The only deprecated command that should stay deprecated that is related is the rsautl command.

Tomáš Mráz wrote:

@paulidale No, it was not deprecated. ...
Then use another word - legacy , or obsolete, or etc. + reference to new model .

The only deprecated command that should stay deprecated that is related is the rsautl command.

sample, based on rsautl description "This command  The  command should be used instead."
Roumen


More features to the API

Describe the feature
I have two feature requests for the AdvancedSeasons developer API:
1) Expose the information in the API whether the a biome should get snow during the cold season.
I built my own system on top of AdvancedSeasons which simulates the vanilla snow accumulation behavior in all biomes, but currently I can't check whether it should snow in biome X. I could do it by manually defining that information for all the biomes in code, but it would be much more convenient to get that information from AdvancedSeasons directly.
2) Expose block or biome temperature(s) in the API. Currently only getting the temperature for the player is supported.
This would allow me to check if the block or biome temperature allows snow formation. MC exposes block temperature, but I dabbled with it and  it doesn't really seem to do me much good with AdvancedSeasons.
I suspect you might already have all this information easily available in AdvancedSeasons and would be just a matter of taking a few minutes to expose it in the API and updating the docs.
Thanks for considering this.

The first feature can be added easily, but the second may take more time. AS calculates the temperature only for the player, not for blocks or other entities. This is because AS take advantage of the light level to calculate the temperature.
However, we are planned to improve this mechanism to the future.

Done!


Background call for flutter webrtc client

The webrtc client gets unregistered when it goes background or is terminated.
I am using an asterisk for the SIP server.
When the client is in a background state, it cannot receive the call as it is unregistered.
how to achieve handle this issue?

Usually in such situation you have to implement PUSH notifications to start your application before sending call to it.

Usually in such situation you have to implement PUSH notifications to start your application before sending call to it.

Thank you @dmitry-sinina
Is there any server to act as proxy with asterisk to support RFC 8599, for Webrtc ?

@E2-Sterlin , You can use Push Notification as suggested by @dmitry-sinina, Or else, you can run SIP as a background service.


Ensuring that BigQuery jobs are tagged with the Dataflow step that launches them

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

r: @ajamato

Run Python 3.8 PostCommit

Run Python 3.8 PostCommit

Run Python 3.8 PostCommit


Add benchmark for querying a persisted block

Taking the useful benchmark from #397
Signed-off-by: Chris Marchbanks<EMAIL_ADDRESS>

Thanks!


Create Kyma step

Description
Changes proposed in this pull request:

implementation of a step which creates/updates Kyma resource

Related issue(s)

/hold

Open questions:

what should be the name of the Kyma resource?
what should be the pattern of labels?
do we need any annotations?

ptal @piotrmiskiewicz, go fmt in pre-control-plane-components-kyma-environment-broker job is complaining about formatting
[0100] | ✗ some files are not properly formatted or contain not formatted imports. To repair run make imports
[0100] | ./internal/process/provisioning/apply_kyma_step.go

/unhold


Accept extra env variables for jobs and ckan pods

@zelima can you please explain the use case here?

@akariv Right

Got it, thanks.
Don't forget to add the documentation for the extra variables in the CCO repo.


Usage of BeagleBone PRU

I wanted to know how I could write a python script for reading RC-signals on the beagle bone PRU?

If you're asking about the DSM functionality of the robotics cape library I have not implemented bindings for if yet.


VoxelSet editor

Create a in-engine editor for VoxelSets.

[x] preview
[x] add voxels
[x] remove voxels
[x] update voxels

VoxeSet has been reworked into a Resource and it's functionality has been revamped


A11y: Form Validation

Currently, our storefront form validation has several issues, particularly when it comes to multiple input errors. When there are multiple fields that have errors, only the first is highlighted with a text based indicator. The rest only use a different color outline, which is a failure of SC 1.4.1.[https://www.w3.org/WAI/WCAG22/Understanding/use-of-color|https://www.w3.org/WAI/WCAG22/Understanding/use-of-color|smart-link]
This is also problematic when using a screen reader. VoiceOver will only read the tooltip ‘Please fill in this field’ of the first error. You don’t have any information if there is more than one error, and you don’t even have any information which field has an error - you have to use context clues from the navigation.
{panel:bgColor=#deebff}
All storefront facing features will need to be updated to reflect this solution
{panel}

[public] Shopware automated response: This issue is linked to the internal issue: NEXT-39235
[created from NEXT-39235, comment 506368]

[public]

Status Change: Open

This issue is ready for implementation.
[created from NEXT-39235, comment 510206]


[PowerAutomate-TestEngineModule - Feature]: Documentation of Power FX Functions.

Is your feature request related to a problem? Please describe.
Provide documentation and examples for

MockAction()
OpenWorkflow()
BuildTest()
TriggerWorkflow()

Also add documentation on how other PowerFX functions like With can be used to validate the values returned from Power FX functions
Describe the solution you'd like
Describe what each function, why the function is important and provide examples of how each function can be used
Describe alternatives you've considered
No response
Additional context?
No response

Added to Power FX


Redesign Find a Gig List page public view

[x] Remove company name
[x] Remove search by company name
[x] Add description
[x] Add other detail

I changed the public view so that the company name is not included in the display or the filter.

Closed with pull request #238


UART: add Tx/Rx pin swapping

The alternative is to impl TxPin for all the Rx pins, etc. I think this version is more obvious in the documentation vs listing even more traits.

Wow, this is cool! No more endless PCB checking before sending to fab!
Love it


Referees Expire

Set an expires_at date for all referees. This should be +7 days from the date that the V/T application was submitted_at by the member.
If a referee fails to reply within 7 days, the application should be cancelled (perhaps create a new status as it doesn't fit into either Withdrawn, Rejected or Expired Automatically).
Emails to referees should be updated to ensure it is clear that they have 7 days.

It might make more sense to allow the candidate to choose another referee or send a reminder to the current referee, rather than cancelling their application and making them go through the process of submitting another application again.
If, after 3 attempts to get the reference (where the candidate has chosen a new referee or chosen to remind the current one), there is still no reference, the application would then be cancelled to free up the training space.
If a candidate does not choose an option within 7 days of the reference expiring, the application is then cancelled.

Agreed. Let's go ahead with that.


show signatures graphically irrespective of their cryptographic validation status

@luk-lo 's recommendation https://github.com/mozilla/pdf.js/issues/10507#issuecomment-458532183  (also in https://github.com/brokermint/pdf.js/pull/1) worked nicely for example in version pdfjs-2_4_456-es5-dist
Unfortunately, in pdfjs-2.6.347-es5-dist and pdfjs-2.7.570-es5-dist , the data.fieldValue remains empty
(in the attached example
minimum-sig7.pdf
the fieldValue of obj with id "28R" (signature annotation) is "27R" with 2.4.456, in the newer code this remains null  ;(

Signatures are currently not supported, since they cannot be verified, which means that any "hack" to attempt display them are also (obviously) unsupported; hence this is a duplicate of #1076

Signatures are currently not supported, since they cannot be verified, which means that any "hack" to attempt to display them are also (obviously) unsupported; hence this is a duplicate of issue #1076.

the data.fieldValue remains empty

It needs to be empty, given that signatures are currently not parsed, as explained by this comment: https://github.com/mozilla/pdf.js/blob/709c57a66b176d17db4b2e034d16daddd3860f33/src/core/annotation.js#L1051-L1054

Signatures are currently not supported, since they cannot be verified, which means that any "hack" to attempt to display them are also (obviously) unsupported; hence this is a duplicate of issue #1076.

the data.fieldValue remains empty

It needs to be empty, given that signatures are currently not parsed,  ...
Just strange that the signatures were correctly parsed in v2.4.456-es5-dist and now they are no longer , which is really sad.

as explained by this comment:
https://github.com/mozilla/pdf.js/blob/709c57a66b176d17db4b2e034d16daddd3860f33/src/core/annotation.js#L1051-L1055
As the title of my issue suggests, I have seen this comment before.

... "hack" to attempt to display ...
Looking at the number issues raised around this, it seems to be need for many

and in our empirical experience, the "attempt to display" is successful almost all the times - we never got any complaints.
But we got immediately lots of complaints once we upgraded beyond v2.4.456-es5-dist

Just strange that the signatures were correctly parsed in v2.4.456-es5-dist and now they are no longer , which is really sad.

If it "worked" previously, that was a bug rather a feature given that we've never actually supported signatures. Hence this should never have worked in the first place, and consequently it's not really a regression.

Looking at the number issues raised around this, it seems to be need for many

Yes, but validating signatures is no simple task, which is why it's not yet implemented.

the "attempt to display" is successful almost all the times

There's a very real risk in displaying unverified signatures, since users could then be fooled by invalid ones, which is why just displaying them is not a good idea and why we explicitly hide signatures (for now).

There's a very real risk in displaying unverified signatures, since users could then be fooled by invalid ones, which is why just displaying them is not a good idea and why we explicitly hide signatures (for now).

There are some users out there who _do_know_what_they_do and still think that the graphical display of unverified signature is a worthwhile action, therefore I suggest:
a) not to display them by default
b) but have a switch where it can be configured to be displayed
c) add some disclaimers (e.g. mouse-over ?) that the display doesn't imply it being valid

Closing since we're not going to do this until the signatures can be verified, which is tracked in #1076 but is non-trivial. If this "worked" before, it's indeed a bug and there is simply no guarantee that it will remain to work since it's clearly unsupported. If you want to display them anyway, that's entirely your responsibility and up to you to build. We're not going to provide a means to display signatures that might very well be invalid since that defeats the whole purpose of digital signatures.

display signatures that might very well be invalid since that defeats the whole purpose of digital signatures.

"Always being valid" is not_at_all the purpose of digital signatures.
The first purpose of digital signatures is to give a mean to

find out whether the content is still the same as it was at the origin. And therefore, it is very OK, to also show a signature that proves that this not the case and that the content has been altered (some do it with a red X vs. green check).
furthermore, many jurisdictions have come up with multiple levels of assurance / "expression of will"  (qualified signatures, etc.) as an additional dimension on top of the pure integrity ==> therefore signature validity often is not a binary or boolean topic, but way more involved.

Thus, as for suggestion c) you should refer the less validity-sure end-users to third-party services about signature validity. If you try to do all of it inside pdf.js for all jurisdictions, you will probably never get there.
Fore example there is https://validator.ch (https://www.e-service.admin.ch/validator/upload/all/en) with ELEVEN flavours of signature validity - and this for just one of the smallest countries of the planet.
Hence, please reconsider re-opening this issue and removing the "non-regression" of data.fieldValue w.r.t.  v 2.4.456 !
P.S.: I am also fine, if you by default put a big red X over each signature display to be on the safe side...

https://github.com/mozilla/pdf.js/pull/13214 seems to be positive news


this adds app_id attribute to neovide

Hi!
This is a little feature for Linux users that are on wayland only desktops. Over here, we don't have class or instance attributes, but an app_id.
This should not bring any breaking changes into the codebase.
Further information on the topic is over here

Updated for now. While this shouldn't introduce any problems, this is cutting short. In another PR, it would be great to actually include logic to decide between x11 and wayland features of glutin and adjust the window building to the correct one present.

@TimB87 would you be wiling to update #704 with what you think would be necessary?


Fixing README and Doc Index examples

Coverage remained the same at 78.085% when pulling e798dde0e36adeda3105bfe76662fac9a5ebbcdb on fix-examples into 7b02f120c2e8203e1edbd95a776c02ee5da224a2 on develop.


ModuleNotFoundError: No module named 'lsp_jsonrpc'

2024-10-13 00:02:41.433 [info] Name: yapf
2024-10-13 00:02:41.433 [info] Module: yapf
2024-10-13 00:02:41.433 [info] Python extension loading
2024-10-13 00:02:41.433 [info] Waiting for interpreter from python extension.
2024-10-13 00:02:41.433 [info] Python extension loaded
2024-10-13 00:02:41.433 [info] Server run command: d:\Portables\python-3.13.0-embed-amd64\python.exe c:\Users\1\.vscode\extensions\eeyore.yapf-2024.9.126091437\bundled\tool\lsp_server.py
2024-10-13 00:02:41.433 [info] Server: Start requested.
2024-10-13 00:02:41.491 [info] Traceback (most recent call last):

2024-10-13 00:02:41.491 [info]   File "c:\Users\1\.vscode\extensions\eeyore.yapf-2024.9.126091437\bundled\tool\lsp_server.py", line 44, in <module>
    import lsp_jsonrpc as jsonrpc
ModuleNotFoundError: No module named 'lsp_jsonrpc'

2024-10-13 00:02:41.496 [info] [Error - 00:02:41] Server initialization failed.
2024-10-13 00:02:41.496 [info]   Message: Pending response rejected since connection got disposed
  Code: -32097
2024-10-13 00:02:41.496 [info] [Info  - 00:02:41] Connection to server got closed. Server will restart.
2024-10-13 00:02:41.496 [info] true
2024-10-13 00:02:41.496 [info] [Error - 00:02:41] yapf client: couldn't create connection to server.
2024-10-13 00:02:41.496 [info]   Message: Pending response rejected since connection got disposed
  Code: -32097
2024-10-13 00:02:41.563 [info] Traceback (most recent call last):

2024-10-13 00:02:41.563 [info]   File "c:\Users\1\.vscode\extensions\eeyore.yapf-2024.9.126091437\bundled\tool\lsp_server.py", line 44, in <module>
    import lsp_jsonrpc as jsonrpc
ModuleNotFoundError: No module named 'lsp_jsonrpc'

2024-10-13 00:02:41.568 [info] [Error - 00:02:41] Server initialization failed.
2024-10-13 00:02:41.568 [info]   Message: Pending response rejected since connection got disposed
  Code: -32097
2024-10-13 00:02:41.568 [info] [Info  - 00:02:41] Connection to server got closed. Server will restart.
2024-10-13 00:02:41.568 [info] true
2024-10-13 00:02:41.569 [info] [Error - 00:02:41] yapf client: couldn't create connection to server.
2024-10-13 00:02:41.569 [info]   Message: Pending response rejected since connection got disposed
  Code: -32097
2024-10-13 00:02:41.569 [error] Server: Start failed: Error: Pending response rejected since connection got disposed
2024-10-13 00:02:41.630 [info] Traceback (most recent call last):

2024-10-13 00:02:41.630 [info]   File "c:\Users\1\.vscode\extensions\eeyore.yapf-2024.9.126091437\bundled\tool\lsp_server.py", line 44, in <module>
    import lsp_jsonrpc as jsonrpc
ModuleNotFoundError: No module named 'lsp_jsonrpc'

2024-10-13 00:02:41.635 [info] [Error - 00:02:41] Server initialization failed.
2024-10-13 00:02:41.635 [info]   Message: Pending response rejected since connection got disposed
  Code: -32097
2024-10-13 00:02:41.635 [info] [Info  - 00:02:41] Connection to server got closed. Server will restart.
2024-10-13 00:02:41.635 [info] true
2024-10-13 00:02:41.635 [info] [Error - 00:02:41] yapf client: couldn't create connection to server.
2024-10-13 00:02:41.635 [info]   Message: Pending response rejected since connection got disposed
  Code: -32097
2024-10-13 00:02:41.635 [info] [Error - 00:02:41] Restarting server failed
2024-10-13 00:02:41.635 [info]   Message: Pending response rejected since connection got disposed
  Code: -32097
2024-10-13 00:02:41.695 [info] Traceback (most recent call last):

2024-10-13 00:02:41.695 [info]   File "c:\Users\1\.vscode\extensions\eeyore.yapf-2024.9.126091437\bundled\tool\lsp_server.py", line 44, in <module>
    import lsp_jsonrpc as jsonrpc
ModuleNotFoundError: No module named 'lsp_jsonrpc'

2024-10-13 00:02:41.700 [info] [Error - 00:02:41] Server initialization failed.
2024-10-13 00:02:41.700 [info]   Message: Pending response rejected since connection got disposed
  Code: -32097
2024-10-13 00:02:41.700 [info] [Info  - 00:02:41] Connection to server got closed. Server will restart.
2024-10-13 00:02:41.700 [info] true
2024-10-13 00:02:41.700 [info] [Error - 00:02:41] yapf client: couldn't create connection to server.
2024-10-13 00:02:41.700 [info]   Message: Pending response rejected since connection got disposed
  Code: -32097
2024-10-13 00:02:41.700 [info] [Error - 00:02:41] Restarting server failed
2024-10-13 00:02:41.700 [info]   Message: Pending response rejected since connection got disposed
  Code: -32097
2024-10-13 00:02:41.759 [info] Traceback (most recent call last):

2024-10-13 00:02:41.759 [info]   File "c:\Users\1\.vscode\extensions\eeyore.yapf-2024.9.126091437\bundled\tool\lsp_server.py", line 44, in <module>
    import lsp_jsonrpc as jsonrpc
ModuleNotFoundError: No module named 'lsp_jsonrpc'

2024-10-13 00:02:41.764 [info] [Error - 00:02:41] Server initialization failed.
2024-10-13 00:02:41.764 [info]   Message: Pending response rejected since connection got disposed
  Code: -32097
2024-10-13 00:02:41.764 [info] [Error - 00:02:41] The yapf server crashed 5 times in the last 3 minutes. The server will not be restarted. See the output for more information.
2024-10-13 00:02:41.764 [info] [Error - 00:02:41] yapf client: couldn't create connection to server.
2024-10-13 00:02:41.764 [info]   Message: Pending response rejected since connection got disposed
  Code: -32097
2024-10-13 00:02:41.764 [info] [Error - 00:02:41] Restarting server failed
2024-10-13 00:02:41.764 [info]   Message: Pending response rejected since connection got disposed
  Code: -32097

@xiaomoy - hi, I test it on python=3.13 but works fine on my side. Could you check whether lsp_jsonrpc.py on c:\Users\1\.vscode\extensions\eeyore.yapf-2024.9.126091437\bundled\tool floder? You can reinstall this extension if lsp_jsonrpc.py not exists.

@xiaomoy - Could you run
import sys
print(sys.path)

in a .py file anywhere, like d:\Portables\python-3.13.0-embed-amd64\python.exe c:\Users\1\Desktop\python13_path.py, to check whether c:\Users\1\Desktop in the sys.path. I think this's the python interpreter issue, the embed suffix may not a suitable interpreter that automatic add the running directory to sys.path. Could u use another python interpreter or use the setting yapf.interpreter to choice another interpreter different from your current project needed.

@xiaomoy - the directory of the python entrance will insert to the head of sys.path. It seems like  the python-3.13.0-embed-amd64 will not complete this process automatically. You can add a code snipet in the lsp_server, like
import sys
import os
sys.path.insert(0, os.path.dirname(__file__))

Or could you tell me roughly what kind of scenario would require using that interpreter?  So I can consider whether it's a general case and then add that code on the extension side.


Simplify usage of select with dynamically selected items

Provide a select(value <-- selectedValueObservable), like React:
https://reactjs.org/docs/forms.html#the-select-tag

This should work with the value attribute, or not? Or am I overseeing something?


Backbone-specific addresses

Readiness checklist

[x] I added/updated unit tests.
[ ] I added/updated integration tests.
[x] I ensured that the PR title is good enough for the changelog.
[x] I labeled the PR.

@Dannyps The definition of "refactoring" is that it doesn't change behavior.
This changes in this PR is the complete opposite. Nothing will work anymore after we merged this.

Quite fair. I wasn't looking at the bigger picture!

@Dannyps There is a conflict in this PR. You should resolve it rather soon than later, as it will only become worse. ;)

@Dannyps Remember to update the target branch.


Cli + python test

Hello, I am using TON for my project and didn't find any solutions for simple communication under python.
Therefore, I wrote a comfortable layer for cli and interact through python subprocess.
(I started rewriting the entire code base, and then I realized that I'm too lazy :) )
Not fully tested yet.

And also I did not find an option to store data not in Uint8Array, so I did the conversion to hex-string


[T3A3][T13-A1] Aloisius Stephen

Split Address into more classes

@aloisiusStephen overall good job :+1:
there are a few minor comments that I hope will help you think about why and how certain things may be done better. please take note, ack and close the PR when you have read the comments.

Thanks for the comments! will take note of these points in the future


Target make safetest got broken

It used to be possible to only run the tests relying on Mocket - non-real socket ones - but it looks like something changed in the Redis client that is worth investigating.
It helps to overcome issues like the one mentioned in #272

coverage: 98.808% (-0.2%) from 99.025%
when pulling 5405e2ea410920c7a313adbb5228066881f35b3a on make-safetest-got-broken
into a5b5e34b8f981e033bf9694f4fec53481d933ade on main.

coverage: 98.808% (-0.2%) from 99.025%
when pulling 5405e2ea410920c7a313adbb5228066881f35b3a on make-safetest-got-broken
into a5b5e34b8f981e033bf9694f4fec53481d933ade on main.


chore: Use totalCount for offer notifications

This PR resolves ONYX-757
Description
Use totalCount for offer notifications because the workaround is not needed in production.
PR Checklist

[ ] I have tested my changes on iOS and Android.
[ ] I hid my changes behind a feature flag, or they don't need one.
[ ] I have included screenshots or videos, or I have not changed the UI.
[ ] I have added tests, or my changes don't require any.
[ ] I added an app state migration, or my changes do not require one.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Use totalCount for offer notifications - ole

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Dev changes (Use totalCount for offer notifications - ole)

  Generated by :no_entry_sign: dangerJS against 5fda080d964329e0851e350551e41e5d4e718baa


DRAFT: DBZ-121 Added options for how change events are mapped to topics

The default is to map all change events in each table to a separate topic, but this commit adds the ability to map all change events for all database tables to a database-specific topic, or to use a custom mapping via TopicMappingProvider implementation.
NOTE: This is still incomplete and not thoroughly tested.

@rhauch: I've also added a TOPIC_SELECTION_STRATEGY config option for the PG connector with the options of PER_TABLE(default) or PER_SCHEMA. This PR contains the missing pieces in TableSchemaBuilder to support this feature (i.e. correct event Key schema and value generator) so I will not attempt to make the same changes to TableSchemaBuilder, but rather rely on this once it's merged.

@hchiorean Awesome. Any preferences or suggested changes to names of anything? I do like TOPIC_SELECTION_STRATEGY better than TOPIC_MAPPING.

the only comment I would have based on the current proposed PR changes is that I'm not convinced of the benefit of the added complexity for the custom TopicMappingProvider option. For PG, there are only 2 options which make sense : per table and per schema.
As far as integrating with the rest of the code, it's hard to say until I actually get around to doing that, so I'm fine with it in its current form.

@hchiorean, agreed. Ideally we can completely hide how this is implemented, and instead use a configuration option that specifies something like per-table, per-database, per-schemas or whatever reads well. We can expand these if/when we add new built-in strategies.
DBZ-121 not only deals with expanding how the connector maps events to topics beyond just topic-per-table to also handle topic-per-database and topic-per-schema (whichever makes sense for the DBMS), but it also calls for a way to deal with sharded tables. Originally, I was thinking that dealing with shards is really just a different topic mapping strategy, since you may want to annotate the key to do this. There are so many different approaches to sharding that I'm not sure we can have a built-in approach that works for everyone. Thus the need for specifying a custom mapping strategy.
But just because we could use a custom mapping strategy doesn't mean the configuration should require an implementation class name. This still needs to be changed/refined.

I'm excited about these changes. Let me know if I can help!

@dasl- I hope to get back to this first. It is possible to limit the changes to only allow someone to simply customize the names of the topics, but not the table-to-topic mapping. This proposal tries to be even more flexible by allowing multiple tables to be mapping to a single topic, and to do this we'll likely need to augment the message keys since a table's primary key is no longer sufficient to distinguish the row in table A from perhaps a row from table B with the same primary key.
Having said all this, though, I've not really looked at this PR for some time, and I may want to do things differently. Any chance you want to take a whack at it, even if it's just a start/concept?

@rhauch My approach would probably be very similar to the one you took here. I would like to allow multiple tables to map to a single topic also.
I will try applying this patch and seeing how it works. Perhaps I will have a follow up code review with some additions.

Closed without merging. Instead, we've gone with using Single Message Transforms per #211.


[Core] Allow disablement option for tracing implementation

Tracing is enabled by default if "opentelemetry" has been imported and the azure-core-tracing-opentelemetry package has been installed.
However, in some cases, a user might not want this implicit enablement. For example, the azure-monitor-opentelemetry package allows users to disable Azure SDK tracing, so a way to explicitly disable tracing is needed to avoid the implicit enablement.
Here, we let users set the tracing_implementation value to the string "none" to ensure tracing is disabled in cases where it would be implicitly enabled. We don't use the Nonetype since a value of None currently has the implicit behavior.

API change check
API changes are not detected in this pull request.


Added files for node version in reactive native project

Summary:
Changelog:
[General][Added] - Added files for avn, nodenv, and other managers that set the node.js version in reactive native project including testing
Reviewed By: cortinico
Differential Revision: D37683291

i guess if we are currently using a .nvmrc file from nvm, we dont need this file?

i guess if we are currently using a .nvmrc file from nvm, we dont need this file?

exactly

How do we know if we need to add this file?


Include 'Connect-MgGraph -Scopes ' for PowerShell Code snippets

Is your feature request related to a problem? Please describe.
Issuing an MS Graph API call shows how to use the corresponding PowerShell cmdlet to perform that task, when the admin clicks "Code snippets" > PowerShell.  The example generated even includes which module to import.  What it lacks, and causes the command to fail, are the permissions that are required for the command to work. This forces the frustrated admin to have to locate the API documentation again in order to determine the permissions that are required. Once they know the permissions, they must re-authenticate with Connect-MgGraph while specifying the correct -Scopes.
Describe the solution you'd like
In addition to showing the Import-Module command, the "Code snippet" for "PowerShell" should also include Connect-MgGraph -Scopes  that will allow the command to run.
Describe alternatives you've considered
There is no alternative unless we want to continue to force frustrated admins to have to look for the API docs page so they can look up the permission. Giving them the scope will also help drive MS Graph PowerShell adoption.
Additional context
N/A

Hey @dweverett. Sorry about this.
A quick way to locate permissions for any query is on the Modify Permissions tab:

Also, to quickly locate docs, there is a button on the extreme right of the query textbox which takes you to the correct documentation

I understand that you prefer to have the scopes pre-populated on the cmdlet. Let me escalate this as a feature request and tag the appropriate teams.

Hey @dweverett. For now, permissions required for specific request urls are on the Modify permissions tab. Whenever they are not available on the tab, the next option is to check the documentation through the 'Learn more' button.
Appending scopes to the snippets will cause a subpar experience in cases where there are quite a number of scopes to consent to.


The /tests endpoint from the REST API is not returning all tests

What I'm working on
I'm working on an CI integration with Defect Dojo. We're collecting a series of reports through out the CI pipeline and then upload it to DD. The problem is that if you're on a Merge Request and commit changes many times, you'll end up with lots of the same tests on a given engagement. Thus, I was thinking on looking only at the most recent test for any given test type, in order to decide if the MR gate should be lifted or not.
Bug description
When I request the /tests endpoint for a given engagement like: .../tests/?engagement=875 the results list contains lots of tests and test types but never the most recent one.
Steps to reproduce
Steps to reproduce the behavior:

Create an engagement;
Upload various reports of the same type to it (which simulates the various runs of the pipeline during the lifecycle of a Merge Request);
Check on the UI the list of tests, you should see various tests having the same name on the engagement;
Check the ID of the most recent one, i.e., the highest ID;
Make a simple curl request or use postman to call the tests endpoint speciying the engangement and you'll see the highest ID you find in the browser is no the one you see in the REST API.

Expected behavior
It would be expected that the highest IDs should match.
Deployment method (select with an X)

[ X ] Docker Compose
[ ] Kubernetes
[ ] GoDojo

Environment information

DefectDojo version 2.34.3

Screenshots
The UI highlighting the most recent scan ID

When calling the /test endpoint, this is the highest ID I get for the test type of the previous image.

@rgcouto I have not been able to reproduce this. My own tests DO return the most recent test (in this case 2332, as below):

I do note that the default limit once the count goes over 25 comes into play. You have 29 results which suggests that perhaps there is a 'next' for paging in your call? Like so:

You can also try calling the curl command with the limit parameter set to 100 or something over your expected count. Are you still seeing this issue?

You're right, I didn't thought about pagination. My bad..
Thanks anyway! Great job!


Domestic Standing Order Consent UI missing standing order details

The Consent UI for Domestic Standing Order is missing information about the standing order i.e. frequency, payment amount, dates etc. Therefore, it does not give the PSU user enough information for them to decide to give or reject consent.
TO DO - check if International Standing Order UI has the same issue.

The initiation object has been changed for Standing order payments (domestic and international), now is the original initiation from the consent instead of a customised frequency details.
The UI needs to be fixed to display properly the frequency details.
The detail services needs to set a readable frequency to be displayed in the UI.

See DomesticStandingOrderConsentDetailsService#addIntentTypeSpecificData in RCS
See InternationalStandingOrderConsentDetailsService#addIntentTypeSpecificDatain RCS
See FRFrequency in commons


Reloading like crazy in deeper navigation stack

First thank you for your nice library, which saved me a lot of time writing boilerplate codes. But I just stumbled a weird issue.
Here's my code:
https://github.com/tamcy/asyncloadertest/blob/master/lib/main.dart
It is very similar to the example of this library with two changes:

it shows a bottom sheet when an action button is tapped.
there is another button which brings users to another page (I am reusing the same class in this demo).

And here's a video of what happens:
https://github.com/tamcy/asyncloadertest/blob/master/demo.gif
As you see, the code is working perfectly in the first (home) page. But when navigated to the second page or deeper,

the async loader seems to init itself twice from the look of the progress indicator, before actually showing the content. This already indicates something isn't quite right.
whenever a dialog (bottom sheet in this example) is shown, the async loader reloads itself, which is obviously not right.

Things become normal again when GlobalKey is not assigned to async loader, but then there is no way to access the current state (which I need to).
I am new to Flutter, so I'm not sure if I am doing things incorrectly. My apologize if it is the problem of my side. Thanks in advance.

Okay, this is because we can't assume build() not be called when there's no visible change. So we need to make the async loader an instance variable, and have it initialized in the initState() method. This also means that the class holding using the loader has to be stateful.


Music Assistant Fails to Start due to DB Error

What version of Music Assistant has the issue?
2022.8.4
The problem
I get an error on HA Startup in the HA Logs
Logger: homeassistant.config_entries
Source: custom_components/mass/__init__.py:190
Integration: Music Assistant ([documentation](https://github.com/music-assistant/hass-music-assistant), [issues](https://github.com/music-assistant/hass-music-assistant/issues))
First occurred: 3:34:25 AM (1 occurrences)
Last logged: 3:34:25 AM

Error setting up entry Music Assistant for mass
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 365, in async_setup
    result = await component.async_setup_entry(hass, self)
  File "/config/custom_components/mass/__init__.py", line 197, in async_setup_entry
    raise exc
  File "/config/custom_components/mass/__init__.py", line 190, in async_setup_entry
    await mass.setup()
  File "/usr/local/lib/python3.10/site-packages/music_assistant/mass.py", line 79, in setup
    await self.database.setup()
  File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/database.py", line 43, in setup
    await self._migrate()
  File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/database.py", line 216, in _migrate
    await self.set_setting("version", str(SCHEMA_VERSION))
  File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/database.py", line 60, in set_setting
    return await self.insert(
  File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/database.py", line 137, in insert
    return await self.get_row(table, lookup_vals)
  File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/database.py", line 115, in get_row
    return await self._db.fetch_one(sql_query, match)
  File "/usr/local/lib/python3.10/site-packages/databases/core.py", line 148, in fetch_one
    return await connection.fetch_one(query, values)
  File "/usr/local/lib/python3.10/site-packages/databases/core.py", line 257, in fetch_one
    return await self._connection.fetch_one(built_query)
  File "/usr/local/lib/python3.10/site-packages/databases/backends/sqlite.py", line 120, in fetch_one
    metadata = CursorResultMetaData(context, cursor.description)
  File "/usr/local/lib/python3.10/site-packages/sqlalchemy/engine/cursor.py", line 164, in __init__
    (
ValueError: not enough values to unpack (expected 5, got 4)

How to reproduce
it happens on startup
Relevant log output
Logger: homeassistant.config_entries
Source: custom_components/mass/__init__.py:190
Integration: Music Assistant ([documentation](https://github.com/music-assistant/hass-music-assistant), [issues](https://github.com/music-assistant/hass-music-assistant/issues))
First occurred: 3:34:25 AM (1 occurrences)
Last logged: 3:34:25 AM

Error setting up entry Music Assistant for mass
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 365, in async_setup
    result = await component.async_setup_entry(hass, self)
  File "/config/custom_components/mass/__init__.py", line 197, in async_setup_entry
    raise exc
  File "/config/custom_components/mass/__init__.py", line 190, in async_setup_entry
    await mass.setup()
  File "/usr/local/lib/python3.10/site-packages/music_assistant/mass.py", line 79, in setup
    await self.database.setup()
  File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/database.py", line 43, in setup
    await self._migrate()
  File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/database.py", line 216, in _migrate
    await self.set_setting("version", str(SCHEMA_VERSION))
  File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/database.py", line 60, in set_setting
    return await self.insert(
  File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/database.py", line 137, in insert
    return await self.get_row(table, lookup_vals)
  File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/database.py", line 115, in get_row
    return await self._db.fetch_one(sql_query, match)
  File "/usr/local/lib/python3.10/site-packages/databases/core.py", line 148, in fetch_one
    return await connection.fetch_one(query, values)
  File "/usr/local/lib/python3.10/site-packages/databases/core.py", line 257, in fetch_one
    return await self._connection.fetch_one(built_query)
  File "/usr/local/lib/python3.10/site-packages/databases/backends/sqlite.py", line 120, in fetch_one
    metadata = CursorResultMetaData(context, cursor.description)
  File "/usr/local/lib/python3.10/site-packages/sqlalchemy/engine/cursor.py", line 164, in __init__
    (
ValueError: not enough values to unpack (expected 5, got 4)

Additional information
No response
What version of Home Assistant Core are your running
2022.11.0b0
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Raspberry Pi

Duplicate of https://github.com/music-assistant/hass-music-assistant/issues/962

Ture - apparently enetered at the same time... Closing this one


When my app goes to background it's killing background service

Here is my Application
https://github.com/KashifAhmed/backgroundGeoLocaiton

I also ran the sampleApp but it's same when I suspend my app it's destroying background service do I need to add some thing in AndroidManifest or I need some thing to add on config.xml.

This is not an issue with my plugin — it’s an issue with your device.
Did did not answer my question about the state of developer-options that
you don’t have “Don’t keep activities” set.
[image: Inline image 1]
On Sun, Jul 12, 2015 at 5:56 PM, Kashif Ahmed<EMAIL_ADDRESS>wrote:

I also ran the sampleApp but it's same when I suspend my app it's
destroying background service do I need to add some thing in
AndroidManifest or I need some thing to add on config.xml.
—
Reply to this email directly or view it on GitHub
https://github.com/christocracy/cordova-plugin-background-geolocation/issues/209#issuecomment-120766409
.

--
Chris Scott
Transistor Software http://www.transistorsoft.com

I have checked the sampleApp both enable & disable "Don't keep activities".

What device make/model OS version are you testing this on?  try it on
another Android device.
On Sun, Jul 12, 2015 at 6:12 PM, Kashif Ahmed<EMAIL_ADDRESS>wrote:

I have checked the sampleApp both enable & disable "Don't keep activities".
—
Reply to this email directly or view it on GitHub
https://github.com/christocracy/cordova-plugin-background-geolocation/issues/209#issuecomment-120768012
.

--
Chris Scott
Transistor Software http://www.transistorsoft.com

The app which is on google play it works fine.on my mobile, I have Motorola Droid Razr M.

Same error on other devices I think I'm doing something wrong.

Remove my plugin
In a simple Hello World app, set up a watchPosition using standard plugin cordova-plugin-geolocation.  Send the location to your server via Ajax.
Does the problem still occur, where where the app dies when you "minimize"?


FindRootsWithin not always returning correct results

Hi @SeanJxie
I found that FindRootsWithin doesn't always return the correct roots.
Here is an example of a polynomial i used:
x^21 - 86400x +86399=0

When I used FindRootsWithin(1,100) I get the following roots: [14.912790 1.000000] : (14.912790 is not a valid root)
When I used FindRootsWithin(0,100) I got the following roots: [1.000000 1.687851] - which are correct.
When I used FindRootsWithin(-100,100) I got the following roots: [-14.765139 1.000000 1.687851] of which -14.765139  is not a valid root.

Hello @MartinSimangoDaemn
Thank you for pointing this bug out. It has been fixed as a result of the Polygo rewrite.
p := NewPolyFromString("x^21 - 86400x + 86399")
s := NewSolverDefault()

fmt.Println(s.FindRootsWithin(p, 1, 100))
fmt.Println(s.FindRootsWithin(p, 0, 100))
fmt.Println(s.FindRootsWithin(p, -100, 100))

Now, you can also find all roots at once:
fmt.Println(s.FindRoots(p))

Cheers,
Sean


LICENSE: Init Apache 2.0 license

Looks like this was missing in bootstrap.
Signed-off-by: Stephen Augustus<EMAIL_ADDRESS>/assign @saschagrunert @puerco @cpanato
cc: @kubernetes-sigs/release-engineering
/priority critical-urgent

/lgtm


mrp2_desktop: 0.2.1-2 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository mrp2_desktop to 0.2.1-2:

upstream repository: https://github.com/milvusrobotics/mrp2_desktop.git
release repository: https://github.com/milvusrobotics/mrp2_desktop-release.git
distro file: indigo/distribution.yaml
bloom version: 0.5.20
previous version for package: null

mrp2_desktop
* Package information update
* Initial commit
* Contributors: Akif

mrp2_viz
* Package information update
* RViz configs added
* Contributors: Akif

Sorry, you'll have to reopen this one. It was too close to your other pull request within this file.
Simply rerun bloom-release with same options as before but add --pull-request-only. You shouldn't run into this problem much in the future.
I'll close this one for now.

I have opened a new PR for this. For my upcoming PRs, should I wait merge of a PR to open another one if their alphabetical orders are close? Thank you.

Well if there are three merging the middle one first before opening the others would be best otherwise do one at a time if you're not sure.


I want to be able to comment and provide feedback on bugs. (US 5.0)

I want to be able to comment and provide feedback on bugs.
Acceptance Criteria

[x] Requires bug reports to have a comment section
[x] Database should store comments that have been made on a bug report
[x] A comment section should be displayed for each bug report
[x] Requires a form to enter a comment
[x] User should be able to add a comment

The following acceptance criteria was removed since long threads of comments are undesirable:

User should be able to reply to/comment on other comments


turn non-prod cloud runs yellow

We discussed that pipelines which run in pre-prod environments like Canary/PPE/INT should not be hard failures because they are not indicators of client library health and oftentimes fail due to service-related issues. Turning these "yellow" will result in warnings rather than failures.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Data Factory:0.17734918,Azure.Core:0.07873521,azure-spring:0.039057408'


[Android] XHR Response Headers empty

Raising issue to keep track of it. When I am using superagent which wraps XMLHTTPRequest on iOS - it works fine and I see all the headers. But same code on Android does not have the response headers, this is probably not getting exposed correctly on Android's wrapper.
xhr.responseHeaders is {}. Probably related to this comment: NetworkingModule.java#L183

Thanks for reporting this @paramaggarwal! Would be happy to review a pull request, would you like to look into it?

Just going to link the PR from here so somebody doesn't do duplicate work: https://github.com/facebook/react-native/pull/3062

I think it issue can be closed by PR https://github.com/facebook/react-native/pull/3709

Closing as https://github.com/facebook/react-native/issues/2861 was merged.


Disallow duplicate json keys

please remove whitespace changes

+1 once ws changes are removed

@davisp @rnewson the WS changes are a separate commit

@janl The issue is that the new whitespace is also wrong as its using two space indents instead of our standard 4. If you update to have correct indentation I'm fine leaving it as per SOP.

sorry this slipped, fixed in latest push


Added configuration for ReactJsx filter support

Here is the required configuration to make the ReactJsx filter available through this bundle.
This filter is not present in any tagged version of Assetic yet though, so you guys might want to wait before merging this. I successfully tested this PR's contents on a fresh Standard Edition clone where I forced kriswallsmith/assetic to dev-master.

Hi,
Any news on this ? AFAIK, the failing builds are unrelated to this PR's contents.

Thanks!

You're welcome :)
On a side note, is there any way this PR can get me this : https://connect.sensiolabs.com/badge/32/symfony-code-contributor or is it only for core contributions ?

Any timeline of getting this tagged?

I would like to know too, we still have to maintain a mirror of this in our own packages yet.


Update dokka version, fix knowledge task

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #2237
Description
Update dokka version, add missing configuration code to knowledge doc task.
Alternative(s) considered
Type
Choose one: Bug fix
Screenshots (if applicable)
Checklist

[x] I have read and acknowledged the Code of conduct.
[x] I have read the Contributing page.
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA.
[x] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach.
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project.
[x] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally.
[ ] I have built and run the demo app(s) to verify my change fixes the issue and/or does not break the demo app(s).

ok @vorburger i've disabled auto merge. but i'm fine with this being merged. so please feel free to.

ok @vorburger i've disabled auto merge. but i'm fine with this being merged. so please feel free to.

I've now tested this as part of working on #2483 and seen that it works great, so I'm merging; thanks @williamito !

This fails to auto-merge due to #2485. Either #2486 (if the team agrees) or a "real" fix of that problem will unblock it.
This is holding up #2483.


Evaluated core soroban-env-host error handling changes

What version are you using?
Evaluate the impact of core soroban-env-host error handling changes in
https://github.com/stellar/rs-soroban-env/commit/44cba6a72cd2cca9594de492298eea1a7ed0e031
https://github.com/stellar/rs-soroban-env/commit/1c3f7aef810c2692ee0c9ef2bfc690d9cbb8d647
https://github.com/stellar/rs-soroban-env/commit/94bc87a42d5ced70905f2c09f6ae69fa000954ba

@anupsdf without a summary of what changed, it's more productive to simply try the new version rs-soroban-env and see the impact. Can you give me the env and SDK commits so that I try it out?

Closing in favor of https://github.com/stellar/go/issues/5075


Using with Jeyklll

I am fairly new to Jekyll, after including extended-markdown-filter to my Gemfile, bundling and adding to extensions I get the following error:
undefined method '[]' for nil:NilClass
Looking at https://github.com/gjtorikian/extended-markdown-filter#within-jekyll I am not quite sure what it means to pass the context "emf_use_blocks" through the filter.

Ah, with Jekyll, this is meant to be used with another plugin in conjunction, https://github.com/gjtorikian/jekyll-html-pipeline.
A minimum config file might look like:
gems:
  - extended-markdown-filter
  - jekyll-html-pipeline

markdown: HTMLPipeline
html_pipeline:
  filters:
    - "ExtendedMarkdownFilter"
  context:
    emf_use_blocks: true

Let me know if that works out for you and I'll update the documentation!


publish can-note status

If can-note does not communicate change the availability status in Homeassistant.
Possibly realizable by uptime.

node is made unavailable in two cases:

can2mqtt is disconnected from mqtt broker
time since last heartbeat is greater than hearbeat interval read from the node multiplied by 2


Automatically apply govuk-main-wrapper-l class to  if beforeContent has no content.

possibly too magic?

Potentially, let's see if this comes up later.


big number of applauses

When a number of applauses is very big (like<PHONE_NUMBER>34234234234) it is not well centered

Wow, you must have a very popular blog!
I see the issue ...

Well, not popular at all.
Just need to cover all the possible behaviors.

:tada: This issue has been resolved in version 3.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Please add TWD!

Please add TWD!

In  need of this as well!!

Same

Hi @LawreyTan, @acidjazz & @ArchitW - as I understand it, this library uses the data provided by the European Central Bank (ECB). The ECB produces a list of currencies they currently support: https://www.ecb.europa.eu/stats/policy_and_exchange_rates/euro_reference_exchange_rates/html/index.en.html and TWD isn't currently on the list so I don't think it would be that simple to add this currency to the library, but it's maintainers may have ideas!


Fixed AddBanAsync(IUser,int) calling itself

... instead of AddBanAsync(ulong,int)
Fixes StackOverflowException which occurs when AddBanAsync is called.

How do you get nadeko flowers sorry for posting on a random conversation @Kwoth

LGTM


--no-auth flag no longer functioning on 1.19.0

Bug description
Using the --no-auth flag during start-up does not seem to work correctly anymore. Going to the url brings me to <url>/#/auth with blank username/password fields. Clicking Login generates this message in the log: http error: Cannot authenticate user. Portainer was started with the --no-auth flag (err=Authentication is disabled) (code=503)
Expected behavior
--no-auth flag means going to the url loads the home page instead of the authentication screen.
Steps to reproduce the issue:

docker run -p 9000:9000 -v /var/run/docker.sock:/var/run/docker.sock portainer/portainer --no-auth
Go to localhost:9000

Technical details:

Portainer version: 1.19.0
Docker version (managed by Portainer): 18.06.0-ce
Platform (windows/linux): Mac
Command used to start Portainer: docker run -p 9000:9000 -v /var/run/docker.sock:/var/run/docker.sock portainer/portainer --no-auth
Browser: Chrome, Safari, Firefox

Additional context
This setup seems to have worked until the update to 1.19.0. I've attempted clearing any cached data. Attempted opening in incognito mode and in all my other browsers (Chrome, Safari, and Firefox).

As an update, I realized that on the authentication page, an error pops up saying "Endpoints not found" as well. Still have not resolved the issue.

Looks like this issue has been introduced after portainer/portainer:1.18.1.
--no-auth works as expected with v1.18.1

It says "Endpoints is not defined" at the login page when --no-auth is used.

I still have the similar issue. I am using version 1.20.1. but I do not have "Endpoints not found". it asks for login only.


ci: Fix not outputting values from release-please

Codecov Report

Merging #2816 (224b94f) into develop (aa35277) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           develop   #2816   +/-   ##
=======================================
  Coverage     7.07%   7.07%
=======================================
  Files          219     219
  Lines        10802   10802
=======================================
  Hits           764     764
  Misses       10038   10038

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


How to make a global locale config, as well style

Hello,
I didn't see on docs a way to make a global configuration of locale and style. It's very annoying to add :locale every time i use a datepicker on a large app. Is there any way?

There's explicitly no way to do that, as I like my components isolated. I would suggest you add a higher-order component wrapping this one, which would set the :locale prop and CSS variables for styling, and would pass through any other props (attribute fallthrough makes that very easy: https://vuejs.org/guide/components/attrs.html#nested-component-inheritance)


Can't run championship race weekend as sessions won't complete (URGENT)

Describe the bug
Prior to the recent update we were having scheduling issues and had to manually start parent race weekend sessions after which linked child sessions would auto-start if configured so. Since the update scheduling is working but the sessions do not complete (staying perpetually as "In Progress"). I have since found this happens even after manually starting the sessions and with no ability to manually complete them we can't run our regular Monday evening championship race weekend events for our sim racing group (looking to fall back to Content Manager server in the interim which is not ideal but all we can do at this stage).
Realising it has something to do with the recent update we attempted to change back to the Stable Channel in our ES control panel but the server will not start no matter how many times we've tried and get the error shown under the screenshots section below.
Update 29/8/21 - I was given some advice by Emperor Servers support to import the results file to the session to manually complete it. Upon further testing where I joined myself to ensure a results file was generated to test this advice out, all sessions then completed without issue, showing that it was the lack of anyone connecting to my test sessions that stopped them from completing. I believe from memory on previous testing that this didn't use to be the case and sessions used to still complete without anyone joining (I could be remembering wrong though, no sure).
In any case, this issue report can be closed if it is deemed resolved or left open if sessions are meant to complete regardless of any entrants joining and I am happy to help test further if such is the case, which if so then I'm assuming Server Manager is meant to generate it's own blank results file or import a generic blank one in order to complete a session where no actual results were generated from entrants joining?
To Reproduce

Manually start a race weekend parent session
Wait for the session time to end
Server stops instead of progressing to next child event and parent event remains "In Progress" with no way to manually complete it

Expected behavior
The ability to manually or automatically via scheduling start a race weekend session and have it auto progress to child sessions once parent sessions finish.
Screenshots

Desktop (please complete the following information):
OS (ver): Windows 10 Pro (10.0.19042 Build 19042) & iOS (14.1)
Browser (ver): Brave (1.28.106) & Safari
Additional context
Any support ASAP would be greatly appreciated, if not resolved in 2 days time this will nearly take us out of action as a sim racing group and the Content Manager server backup solution has been known to be less than stable in the past for the amount of entrants we have in our events.

Added an update to the OP bug description.


feat: expose set_visible_on_all_workspaces()

Would allow for Spotlight-like panel windows.
Related:

https://github.com/tauri-apps/tauri/issues/2258
https://github.com/tauri-apps/tao/issues/136

StackOverflow answers for showing window over fullscreen app:

https://stackoverflow.com/questions/23503943/add-nswindow-in-front-of-fullscreen-app
https://stackoverflow.com/questions/29995376/show-nswindow-beneath-nsstatusitem-in-fullscreen

This might be how Electron implements it? https://github.com/electron/electron/blob/81c143318ba04c9aff2a5bcb80e4a1e3d55e019f/shell/browser/native_window_mac.mm#L1297-L1318

@srsholmes was talking about how this could work in https://github.com/tauri-apps/tauri/issues/2258 (as well as behavior across multiple spaces, which idk if there should be another issue for?)

related to tauri-apps/tao#185

@amrbashir I don't think showing over fullscreen apps is necessarily the same as showing on all workspaces, though I don't know for sure

I guess that is true, but it will be only for macOS.
I will transfer to tao.