Annotation line not drawn

Hi guys,
I'm completely new and honestly not a developer but I'm trying to integrate a line into our current chart.
https://noahsbox.com/?fts=0&preview_theme_id=79991996475
You can find a preview behind this link. I have replaced an image we had with the chart.js but having some difficulties getting the line drawn & a label displayed.
Have tried it now for quite some time and maybe someone here has more luck. Any help would be much appreciated!
<script>
  setTimeout(loadChart)
  function loadChart() {
var ctx = document.getElementById("Price");
var myChart = new Chart(ctx, {
    type: 'bar',
    data: {
        labels: [["Kitchen Box","Standard"]],
        datasets: [{
            label: 'Noahs Box',
            data: [149],
            backgroundColor: [
                'rgba(30, 56, 79, 1)',
                'rgba(30, 56, 79, 1)',
            ],
            borderColor: [
                'rgba(30, 56, 79, 1)',
                'rgba(30, 56, 79, 1)',
            ],
            borderWidth: 1
        },
     {
            label: 'Amazon',
            data: [198.30],
            backgroundColor: [
                'rgba(255, 153, 0, 1)',
                'rgba(255, 153, 0, 1)'
            ],
            borderColor: [
                'rgba(255, 153, 0, 1)',
                'rgba(255, 153, 0, 1)'
            ],
            borderWidth: 1
        },
                  {
            label: 'traditional Retailers',
            data: [250],
            backgroundColor: [
                'rgba(0,91,71, 1)',
                'rgba(0,91,71, 1)'
            ],
            borderColor: [
                'rgba(0,91,71, 1)',
                'rgba(0,91,71, 1)'
            ],
            borderWidth: 1
        }],
    },
    options: {
      responsive: true,
  	  maintainAspectRatio: false,
      legend: {
      	display: true
      },
      animation: {
       		duration: 15000,
        },
      title: {
			display: false,
       		text: 'Save More When You Bundle',
        	fontColor: '#1E384F',
        	fontSize: 14
        },
      scales: {
        yAxes: [{
          ticks: {
            beginAtZero:true
          }
        }]
      },
    },
     plugins: [{
      deferred: {
        yOffset: '50%', // defer until 50% of the canvas height are inside the viewport
        delay: 1000      // delay of 500 ms after the canvas is considered inside the viewport
      },
       annotation: {
			// Defines when the annotations are drawn.
			// This allows positioning of the annotation relative to the other
			// elements of the graph.
			//
			// Should be one of: afterDraw, afterDatasetsDraw, beforeDatasetsDraw
			// See http://www.chartjs.org/docs/#advanced-usage-creating-plugins
			drawTime: 'afterDatasetsDraw', // (default)

			// Mouse events to enable on each annotation.
			// Should be an array of one or more browser-supported mouse events
			// See https://developer.mozilla.org/en-US/docs/Web/Events
			events: ['click'],

			// Double-click speed in ms used to distinguish single-clicks from
			// double-clicks whenever you need to capture both. When listening for
			// both click and dblclick, click events will be delayed by this
			// amount.
			dblClickSpeed: 350, // ms (default)

			// Array of annotation configuration objects
			// See below for detailed descriptions of the annotation options
			annotations: [{
	type: 'line',

	// optional drawTime to control layering, overrides global drawTime setting
	//drawTime: 'afterDatasetsDraw',

	// optional annotation ID (must be unique)
	id: 'a-line-1',

	// set to 'vertical' to draw a vertical line
	mode: 'horizontal',

	// ID of the scale to bind onto
	scaleID: 'y-axis-0',

	// Data value to draw the line at
	value: 149,

	// Optional value at which the line draw should end
	//endValue: 152,

	// Line color
	borderColor: 'red',

	// Line width
	borderWidth: 5,

	// Line dash
	// https://developer.mozilla.org/en-US/docs/Web/API/CanvasRenderingContext2D/setLineDash
	//borderDash: [2, 2],

	// Line Dash Offset
	// https://developer.mozilla.org/en-US/docs/Web/API/CanvasRenderingContext2D/lineDashOffset
	//borderDashOffset: 5,

	label: {
		// Background color of label, default below
		backgroundColor: 'rgba(101, 33, 171, 0.5)',

		// Font family of text, inherits from global
		fontFamily: "sans-serif",

		// Font size of text, inherits from global
		fontSize: 12,

		// Font style of text, default below
		fontStyle: "bold",

		// Font color of text, default below
		fontColor: "#fff",

		// Padding of label to add left/right, default below
		//xPadding: 6,

		// Padding of label to add top/bottom, default below
		//yPadding: 6,

		// Radius of label rectangle, default below
		//cornerRadius: 6,

		// Anchor position of label on line, can be one of: top, bottom, left, right, center. Default below.
		//position: "center",

		// Adjustment along x-axis (left-right) of label relative to above number (can be negative)
		// For horizontal lines positioned left or right, negative values move
		// the label toward the edge, and positive values toward the center.
		//xAdjust: 0,

		// Adjustment along y-axis (top-bottom) of label relative to above number (can be negative)
		// For vertical lines positioned top or bottom, negative values move
		// the label toward the edge, and positive values toward the center.
		//yAdjust: 0,

		// Whether the label is enabled and should be displayed
		enabled: true,

		// Text to display in label - default is null. Provide an array to display values on a new line
		content: "Save 25-50%"
	},

	// Mouse event handlers - be sure to enable the corresponding events in the
	// annotation events array or the event handler will not be called.
	// See https://developer.mozilla.org/en-US/docs/Web/Events for a list of
	// supported mouse events.
	onMouseenter: function(e) {},
	onMouseover: function(e) {},
	onMouseleave: function(e) {},
	onMouseout: function(e) {},
	onMousemove: function(e) {},
	onMousedown: function(e) {},
	onMouseup: function(e) {},
	onClick: function(e) {console.log("Annotation", e.type, this);},
	onDblclick: function(e) {},
	onContextmenu: function(e) {},
	onWheel: function(e) {}
}]
		}

    }]
}
);
  }
</script>

You need to register the plugin.
https://www.chartjs.org/docs/latest/developers/plugins.html
var chart1 = new Chart(ctx, {
plugins: [plugin]
})
and check this: https://github.com/chartjs/chartjs-plugin-annotation/issues/156
try this:
var myChart = new Chart(ctx, {
    type: 'bar',
plugins: [ChartAnnotation],
    data: {
        labels: [["Kitchen Box","Standard"]],
        datasets: [{
            label: 'Noahs Box',
            data: [149],
            backgroundColor: [
                'rgba(30, 56, 79, 1)',
                'rgba(30, 56, 79, 1)',
            ],
            borderColor: [
                'rgba(30, 56, 79, 1)',
                'rgba(30, 56, 79, 1)',
            ],
            borderWidth: 1
        },
     {
            label: 'Amazon',
            data: [198.30],
            backgroundColor: [
                'rgba(255, 153, 0, 1)',
                'rgba(255, 153, 0, 1)'
            ],
            borderColor: [
                'rgba(255, 153, 0, 1)',
                'rgba(255, 153, 0, 1)'
            ],
            borderWidth: 1
        },
                  {
            label: 'traditional Retailers',
            data: [250],
            backgroundColor: [
                'rgba(0,91,71, 1)',
                'rgba(0,91,71, 1)'
            ],
            borderColor: [
                'rgba(0,91,71, 1)',
                'rgba(0,91,71, 1)'
            ],
            borderWidth: 1
        }],
    },
    options: {
      responsive: true,
  	  maintainAspectRatio: false,
      legend: {
      	display: true
      },
      animation: {
       		duration: 15000,
        },
      title: {
			display: false,
       		text: 'Save More When You Bundle',
        	fontColor: '#1E384F',
        	fontSize: 14
        },
      scales: {
        yAxes: [{
          ticks: {
            beginAtZero:true
          }
        }]
      },
    },
     plugins: [{
      deferred: {
        yOffset: '50%', // defer until 50% of the canvas height are inside the viewport
        delay: 1000      // delay of 500 ms after the canvas is considered inside the viewport
      },
       annotation: {
			// Defines when the annotations are drawn.
			// This allows positioning of the annotation relative to the other
			// elements of the graph.
			//
			// Should be one of: afterDraw, afterDatasetsDraw, beforeDatasetsDraw
			// See http://www.chartjs.org/docs/#advanced-usage-creating-plugins
			drawTime: 'afterDatasetsDraw', // (default)

			// Mouse events to enable on each annotation.
			// Should be an array of one or more browser-supported mouse events
			// See https://developer.mozilla.org/en-US/docs/Web/Events
			events: ['click'],

			// Double-click speed in ms used to distinguish single-clicks from
			// double-clicks whenever you need to capture both. When listening for
			// both click and dblclick, click events will be delayed by this
			// amount.
			dblClickSpeed: 350, // ms (default)

			// Array of annotation configuration objects
			// See below for detailed descriptions of the annotation options
			annotations: [{
	type: 'line',

	// optional drawTime to control layering, overrides global drawTime setting
	//drawTime: 'afterDatasetsDraw',

	// optional annotation ID (must be unique)
	id: 'a-line-1',

	// set to 'vertical' to draw a vertical line
	mode: 'horizontal',

	// ID of the scale to bind onto
	scaleID: 'y-axis-0',

	// Data value to draw the line at
	value: 149,

	// Optional value at which the line draw should end
	//endValue: 152,

	// Line color
	borderColor: 'red',

	// Line width
	borderWidth: 5,

	// Line dash
	// https://developer.mozilla.org/en-US/docs/Web/API/CanvasRenderingContext2D/setLineDash
	//borderDash: [2, 2],

	// Line Dash Offset
	// https://developer.mozilla.org/en-US/docs/Web/API/CanvasRenderingContext2D/lineDashOffset
	//borderDashOffset: 5,

	label: {
		// Background color of label, default below
		backgroundColor: 'rgba(101, 33, 171, 0.5)',

		// Font family of text, inherits from global
		fontFamily: "sans-serif",

		// Font size of text, inherits from global
		fontSize: 12,

		// Font style of text, default below
		fontStyle: "bold",

		// Font color of text, default below
		fontColor: "#fff",

		// Padding of label to add left/right, default below
		//xPadding: 6,

		// Padding of label to add top/bottom, default below
		//yPadding: 6,

		// Radius of label rectangle, default below
		//cornerRadius: 6,

		// Anchor position of label on line, can be one of: top, bottom, left, right, center. Default below.
		//position: "center",

		// Adjustment along x-axis (left-right) of label relative to above number (can be negative)
		// For horizontal lines positioned left or right, negative values move
		// the label toward the edge, and positive values toward the center.
		//xAdjust: 0,

		// Adjustment along y-axis (top-bottom) of label relative to above number (can be negative)
		// For vertical lines positioned top or bottom, negative values move
		// the label toward the edge, and positive values toward the center.
		//yAdjust: 0,

		// Whether the label is enabled and should be displayed
		enabled: true,

		// Text to display in label - default is null. Provide an array to display values on a new line
		content: "Save 25-50%"
	},

	// Mouse event handlers - be sure to enable the corresponding events in the
	// annotation events array or the event handler will not be called.
	// See https://developer.mozilla.org/en-US/docs/Web/Events for a list of
	// supported mouse events.
	onMouseenter: function(e) {},
	onMouseover: function(e) {},
	onMouseleave: function(e) {},
	onMouseout: function(e) {},
	onMousemove: function(e) {},
	onMousedown: function(e) {},
	onMouseup: function(e) {},
	onClick: function(e) {console.log("Annotation", e.type, this);},
	onDblclick: function(e) {},
	onContextmenu: function(e) {},
	onWheel: function(e) {}
}]
		}

    }]
}
);
  }```

@asehdev thanks for your reply.
However, I have tried that before (initialise separately and not inline).
I have now tried your version in the following website versioN: https://noahsbox.com/?fts=0&preview_theme_id=82025087035
However, I can not get it to work and always the whole Graph disappears. I had a read through the other issue/thread you have posted but couldnt find anything that deals with this particular issue.
You help would be much appreciated!
Best,
Moritz

I am seeing an error on the console "Uncaught ReferenceError: ChartAnnotation is not defined at loadChart"
Did you import it properly, please check that?

It worked for me when all parameters for annotations has been put right into options object, not plugins object, like https://codepen.io/jordanwillis/pen/qrXJLW
So instead of options.plugins.annotation you can try to use options.annotation

It worked for me when all parameters for annotations has been put right into options object, not plugins object, like https://codepen.io/jordanwillis/pen/qrXJLW
So instead of options.plugins.annotation you can try to use options.annotation

This worked for me; however, the typescript declaration file does not allow for adding arbitrary values to Chart.ChartOptions, it seems to support the options.plugins.annotation way despite this not working at runtime.

Original issue appears resolved


Add integration test for rolling update

This test validates the procedure of a successful rolling update.
Ping @tonistiigi @dperny.
Signed-off-by: Dong Chen<EMAIL_ADDRESS>

Failure on windows seems genuine
# github.com/docker/distribution/registry/api/errcode
vendor\src\github.com\docker\distribution\registry\api\errcode\register.go:74: undefined: http.StatusTooManyRequests

i got the same windows failure in #24563

Ok I deleted my comment but then in my email there is actually a comment about #24563 failing on windows for the same reason. Now it passes, no changes made. Windows build may be flaky.

That win2lin ci box has wrong version of go. @mlaventure removed it.

Several changes:

Use API instead of CLI to get tasks.
Do not validate update-delay as it's hard to estimate run time in different systems. Also --update-delay is not meant to be accurate.
Validate each batch should change update-parallelism number of tasks.

Removing from milestone (it's not strictly part of the release!).

Thank @tonistiigi for pointing out I can compare map in Go. The code is much simpler now.

LGTM

ping @cpuguy83 looks like the PR is. updated PTAL

LGTM

Adding back to the milestone; it was mainly removed to not be a blocker for release


Update the error code of the csv delimiter

According to the spec: https://github.com/meilisearch/specifications/pull/221
The initial error code that was decided has been changed from invalid_index_csv_delimiter to invalid_document_csv_delimiter.
Originally implemented in https://github.com/meilisearch/meilisearch/issues/3442
@meilisearch/docs-team @meilisearch/integration-team

Thank you @irevoire! Sorry for the late feedback


Missing Line numbers on Error messages

I'm currently setting up waybar for me, and while errors are fine, I would have expected a more verbose output to troubleshoot / fix my issues by myself.
I.e.:
[2023-10-18 21:25:23.623] [error] Item '': No icon name or pixmap given.
Is not a big help, when my config grows pretty large and now I have no clue where to look. Maybe we could add the line number in the message, that would be great for future troubleshooters.

After some investigation. I found these errors is probably made by module Tray:


Deleting an unpublished DPUL exhibit returns "405 NOT Allowed" from nginx

Happened with ABC Books. The exhibit still gets deleted, the error message is just off-putting.

Wait a minute. think i figured it out. standby

I didn't. I think it errors but the delete still works.

not t a high priority.

This is the only one that still needs to be deleted, @hackmastera . otherwise we are good. Thanks!
https://dpul.princeton.edu/cicognara


Make this repo work again

List of pending activities:

[x] Merge #165
[x] Merge #177
[x] Merge #183
[ ] Discuss #176
[ ] Why is add_style taking a mutable reference? From #183
[ ] HRTBs in bar.rs example? From #183

Sounds good @AlixANNERAUD?

Yeah that seems a good start.

Hi @AlixANNERAUD for the lvgl-sys move, should we discuss it in the other issue or here is ok?

Hi @AlixANNERAUD for the lvgl-sys move, should we discuss it in the other issue or here is ok?

In the opened issue, I think. I already left comments.


[Doc]: main to master

Describe the bug

[ ] Information addition regarding the newest change through a PR.
[X] Typo error.
[ ] New category addition.
[ ] Refractoring sentences that make more sense.
[ ] Fixing broken links.
[ ] Refractors / reformating of the document.

@Rehan52004 where is the typor

it's a very small issue


Introduced Shiftboss for the distributed version.

@zuyu Somewhere in the files, perhaps in QueryExecutionTypedefs.hpp, would be good to explain the RebuildWorkOrders highlighting when it is issued and the flow in which this is handled. Looks good otherwise. Thanks!

The RebuildWorkOrder info would be added in the next PR where ForemanDistributed is introduced.


Release v3.4.0

Added

Use service for safeTxGas estimation ( #2040 )
Add Aave v2 Safe app ( #2127 )
Labels and styles for cancelling transactions  ( #1652 )

Fixed

Fix layout small issues ( #2071  )
Fix gas estimation in xDai network ( #1932 )

Chore

Code improvements for modals ( #2049 )

The issue for release 3.4.0 full regression issues - https://app.zenhub.com/workspaces/safe-multisig---web-5ce554debb310a35b2f8b6f8/issues/gnosis/safe-react/2134

I reverted commit adding aave v2 safe app per Aave's team request. They plan to integrate safe app support into their main dapp.

@dasanra
We have to merge #2154 to this release branch, it fixes the crashing of apps when inspecting the tx details
There issue with gas limit being pumped up to 1 million plus still happens for tx created from Apps (compound, idle, sablier, etc)


Input API

I can't find the location to add the API from OpenWeatherMap.
I'm using the Homey v2.0 (early 2019).
Where should this API be added, because at the moment I can't add any flows with Open Weather Map.

I've found the sollution. I've had to add the device first.


[FeatureRequest] go 1.16

Some recipes are complaining about requiring go 1.16:
note: module requires Go 1.16

https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=298725&view=logs&j=656edd35-690f-5c53-9ba3-09c10d0bea97&t=e5c8ab1d-8ff9-5cae-b332-e15ae582ed2d&l=1267
It appears that the latest version is 1.16.3: https://golang.org/dl/

I see the autotick bot just opened a PR automatically for 1.15.11. It seems strange that it's not picking up the 1.16 builds?


Running picturefill after pageload in ie8

So, I have version 2.2.0 working good in every browser except ie8. In ie8 it picks the wrong image (which I can live with).

In short: if yes try html5shiv.

yes, I'm using the picture element. I already have the html5shiv (v3.7 included in modernizr)

turns out that shiv is old and doesn't include the picture element. now loading 3.7.2 (but not from modernizr) and it does work. thanks for pointing in the right direction!


Set default arguments to None on Workflows Core / StructLogs

You hate to see it. Will re-visit this when you're back


Install node 20 from nodesource

The default from ubuntu too old for microvium.
Leave it up to the user to install on demand if required.

Thanks.  I need to fix CI to run sensibly on pull requests.


LUTECE-2021 : Improvements around back office passwords

Some improvements around BO passwords:

expire default passwords
deactivate sample accounts
require longer password
better inform the user of password size requirements
Use a secure RNG to generate passwords

OpenJDK8 doesn't support PBKDF2WithHmacSHA512 algorithm yet.
03/11/16 17:35:48 ERROR [http-nio-8080-exec-6] lutece.error - java.security.NoSuchAlgorithmException: PBKDF2WITHHMACSHA512 MessageDigest not available
java.lang.IllegalArgumentException: java.security.NoSuchAlgorithmException: PBKDF2WITHHMACSHA512 MessageDigest not available

Another algotithm should be used if PBKDF2WithHmacSHA512 isn't found for keeping Lutece compatible with OpenJDK.

We will have to see about expiring the default passwords. I'm not sure if this will actually prevent sites in production from using the default password, as people running those sites don't usually use ant, instead they they get a dump from someone running in a test environment where the password is the default password.

pushed a rebased version at https://github.com/lutece-platform/lutece-core/tree/rzara-LUTECE-2021
We need to make  aria-describedby work with EJO's new freemarker macros

The integrator will use the ant script initially, and will be asked to change the default password when testing/configuring. So I still think this will reduce the risk.

This was merged in 4d01bc0129334204bbb7abe4
Thanks !


GitLab integration

Hello,
When following the steps outilined in the "integration with GitLab Job" the Explorer page does not work as expected.
When navigating to the URL created according the the provided example, the page shows that the "File mode" is selected rather than "URL mode" and the report section is empty.
I know that my "url=" parameter is correct as when I click on the "URL" button and paste it in the URL input, the report is displayed correctly.
Otherwise it works great. Thank you for reading.
Tostt

It works fine for me, so I expect that there is an issue with the handling of the payload, or maybe you would need to url encode the parameter? Can you paste an example of the url (might as well be random values, we just need the structure and character set to debug this)

Thank you for getting back to me.
The URL in the GitLab log looks fine :
https://dbsystel.github.io/trivy-vulnerability-explorer?url=https://gitlab.mysite.com/api/v4/projects/653/jobs/4102/artifacts/trivy-results.json
When I click on the URL in GitLab (or paste it in a browser adress bar) it appears that "my" URL appears to have been encoded while "yours" was not  :
https://dbsystel.github.io/trivy-vulnerability-explorer/?url=https%3A%2F%2Fgitlab.mysite.com%2Fapi%2Fv4%2Fprojects%2F653%2Fjobs%2F4102%2Fartifacts%2Ftrivy-results.json#/
I tried with different browsers : chrome, vivaldi and edge with the same results. I'm not sure it this is related to the problem.

Sorry, that it took so long to reach back out to you. I figured out the problem. In #167 I switched to Hash based routing and introduced the regression that you found here. I decided to not fix anything, but rather to update the docs. The URL needs to include a Hash # before the URL parameter. So in your case it needs to be like this
https://dbsystel.github.io/trivy-vulnerability-explorer/#/?url=https://gitlab.mysite.com/api/v4/projects/653/jobs/4102/artifacts/trivy-results.json
Sorry for the inconvenience and thanks for creating this issue!

No worries, it works great now. Well done. Thank you for updating the docs.


Taxon constraint:  GO:0050321 tau-protein kinase activity (or obsoletion?)

Please provide as much information as you can:

GO term ID label:

GO:0050321 tau-protein kinase activity
should have been done here
https://github.com/geneontology/go-ontology/issues/27589

Request to add a taxon constraint:

**only in taxon: **

**never in taxon: **

Supporting evidence if available (e.g PMID):

this one was included too in via https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgeneontology%2Fgo-ontology%2Fcommit%2Ff82ca71d91462d55579dff018605d0000b7850b0&data=05|02|vw253%40universityofcambridgecloud.onmicrosoft.com|76020c14c71042061fc608dd12c6c1b2|49a50445bdfa4b79ade3547b4f3986e9|1|0|638687367076799147|Unknown|TWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D|0|||&sdata=9EvLZBURrvS8MXlDz7%2BIc1RdDo9Ss3IUKCzwZGR1PH8%3D&reserved=0.


Add release notes for July and August 2020

The release notes for July and August 2020 are missing

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 1528404e-3ead-d3aa-e0aa-59e5a8b4889a
Version Independent ID: d8c3bf1c-232e-a388-326d-830447baef19
Content: Manage Microsoft Defender Antivirus updates and apply baselines - Windows security
Content Source: windows/security/threat-protection/microsoft-defender-antivirus/manage-updates-baselines-microsoft-defender-antivirus.md
Product: w10
Technology: windows
GitHub Login: @denisebmsft
Microsoft Alias: deniseb

@brettaweston THANK YOU for raising this issue. I am looking into it.

Hello @brettaweston and thanks again for the feedback. We have updated the article and the changes will be live later today.


JerryScript 2.0

It would be good to release JerryScript 2.0 this year. No specific deadline so far.
What we should do before that is reworking the API. Some ideas:

delete obsolote functions
rework error flag to be an error value (error can only be a result of something, and cannot be used as input except unboxing its value).

delete obsolote functions

Which ones are those?
Quick search for "deprecated" through the code base revealed:
JERRY_DEPRECATED_API
bool jerry_get_object_native_handle (const jerry_value_t obj_val, uintptr_t *out_handle_p);
JERRY_DEPRECATED_API
void jerry_set_object_native_handle (const jerry_value_t obj_val, uintptr_t handle_p,
                                     jerry_object_free_callback_t freecb_p);

#pragma message ("using jerry-api.h directly is deprecated, please use jerryscript.h")

#pragma message ("using jerry-port.h directly is deprecated, please use jerryscript-port.h")

  JERRY_INIT_MEM_STATS_SEPARATE  = (1u << 3), /**< deprecated, an unused placeholder now */
  JERRY_INIT_DEBUGGER            = (1u << 4), /**< deprecated, an unused placeholder now */

Anything else (other than the "old"/current error flag APIs)?

Mostly these. Maybe the parse variant where the source is not provided, but that is not necessary. We should emphasize that pass source info if available, and pass NULL if not.
It will be a much smaller scope API rework that JerryScript 1.0.

I've got a few suggestions for extra API functions/improvements:

JSON C API - parse/stringify
Regex construction from API / maybe even exec/test
Allow jerry_is_feature_enabled to query builtin/es2015-subset objects
Add DataView for es2015-subset

please note I'm not sure how widely they would be used or needed.

(IMHO) If we want to remove functions we should mark them as obsolete as a first step. This way we can (hopefully) notify the users that there will be a bigger change. Related to this I think we should consider writing a migration guide or at least a change list between v1 and v2.

Snapshot generation API  rework #2259
Lets just collect the API rework patches here, and later we can write a changelog using these references.

Jerry parse rework: #2282

Rename the jerry_value_has_error_flag function: #2290

Rename the jerry_value_has_abort_flag function #2291

Rename the jerry_value_clear_error_flag function #2292

Rename the jerry_value_set_error_flag function #2293

Rename the jerry_value_set_abort_flag function #2294

Rename the jerry_get_value_without_error_flag function #2295

Replace the jerry_value_set_error_flag function with jerry_create_error_from_value #2367

Add operational mode for jerry_gc API call. #2385

Replace jerry_value_set_abort_flag with jerry_create_abort_from_value #2411

would 2.0 be a LTS version?

Remove jerry_get_arg_value auto-conversion function: #2425

We only do backward incompatible changes before releases. So yes, after these are landed, there will be no large change for a long time.

Simplify source evaluation options. #2431

Merge instance into context #2501

We use JerryScript in our products, and we need to rely on an officially released version. But version 1.0 is too old to meet the demand, and we expect the community to release version 2.0 as soon as possible. Do you have a release plan in the near future?

@wangzhikang123 Admittedly, the releasing process of the project has room for improvement. But still, "expect" is a somewhat strong word. Moreover, if you are using the project in products (which is great!), don't hesitate to take part in the public development of the project by contributing back improvements or outstanding bug fixes. Please, also refer to the governance wiki page ("expect" vs meritocracy).

JerryScript 2.0 is out. You can find the release here: https://github.com/jerryscript-project/jerryscript/releases/tag/v2.0


Settings tab default behaviour

I am creating a module and when I open the settings, the link tab is open by default. How can I stop this behaviour? Thanks

Also wondering how to set a default toggle for custom modules

cc: @fikrirasyid

@snewsh Hi, this seems to be unwanted behaviour. When settings modal is opened, the fist toggle should be the one that is automatically opened. We'll look into it.
@jamesfosker there is no official way to do so right now. AFAIK we intentionally make the first toggle opened when settings modal is opened.

@jamesfosker there is no official way to do so right now. AFAIK we intentionally make the first toggle opened when settings modal is opened.

It's not opening the first toggle though, I removed the link toggle it then opens the admin label toggle instead, instead of the first toggle which is my one.


Question: Error  inject repository after transpile

When i start my application in develop mode, all works fine, but when transpile files.
I use ts-node-dev to start app.
But when i transpile files with babel and start application, i get the error:
UnhandledPromiseRejectionWarning: ParamTypeMissingError: Cannot get reflected type for a "repository" method's [object Object]1. parameter of OrderService class. Make sure you have turned on an "emitDecoratorMetadata": true, option in tsconfig.json. and that you have imported "reflect-metadata" on top of the main entry file in your application.And make sure that you have annotated the property type correctly with: Repository, MongoRepository, TreeRepository or custom repository class type.
emitDecoratorMetadata is true;
reflect-metadata is imported;
File that start application
server.ts
import 'reflect-metadata';
import { Container } from 'typedi';

import connection from '@config/db';
import { server } from '@config/globals';

import logger from '@middlewares/logger';
import { RabbitClient } from '@utils/rabbitmq/RabbitClient';

connection.then(async () => {
  Container.set(RabbitClient, new RabbitClient());
  const app = (await import('./app')).default;
  app.listen(server.port, () => {
    logger.info(`Server running`, server);
  });
});

OrderServive.ts
import { Inject, Service } from 'typedi';
import { InjectRepository } from 'typeorm-typedi-extensions';

import { dbConnections, rabbit } from './../../config/globals';
import { Order } from './order.entity';
import { OrderRepository } from './OrderRepository';
import { OrderStatus, TransactionStatus } from './types';

@Service()
export class OrderService {
  @InjectRepository(dbConnections.mongo.name)
  private repository!: OrderRepository;

  @Inject()
  private publisher!: RabbitPublisher;

  ....
}

OrderRepository.ts
import { EntityRepository, MongoRepository } from 'typeorm';

import { Order } from './order.entity';

@EntityRepository(Order)
export class OrderRepository extends MongoRepository<Order> {}

github repo
https://github.com/vitordelfino/stock-management

github repo
https://github.com/vitordelfino/stock-management

this link 404s. Can you fix?

repo is now public

github repo
https://github.com/vitordelfino/stock-management

this link 404s. Can you fix?

change  repo to public


Fix PoolInternals QueueStrategy default

Fixing default for queue strategy

@djc any plans to cut a release for bb8 soon? We're actively using it in our project and would prefer to be in sync with cargo rather than use github.


Area has one side overlighted or underlighted

Godot version
4.0 stable
System information
Windows 10 Forward+
Issue description
Whenever I try to make room or closed space one side is always with wrong light. it has too much light or too litle light on one side
Example 1

How it looks on first side

How it looks on second side

Example 2

How it looks on first side

How it looks on second side

Also when I enable SDFGI some of this is fixed, but not all. (my light sources are done using glow and emmision)
I found this in personal project, also Directional light is used in open areas so removing it is not option for me.
Steps to reproduce
Have directional light in scene and in env disable ambient light source.
Minimal reproduction project
MRP_Light.zip

Have you tried rotating your directional light to an angle you like better?
The images look like the directional light is just facing one side of the room. In that case it is expected that the one side will light up more than the other. That's just how lighting works.
In games, without global illumination, light doesn't bounce off of a surface. Whatever comes directly from the light source is what you get. You have to add in the bounced lighting yourself using tools like ambient light (which you disabled), VoxelGI, LightmapGI, or SDFGI

even when I enable SDFGI it feels like light travels throu mesh
(screens with SDFGI enabled)

if you look at image you can say light is directed left forward, left side has more light then left
and the corridor is in this

and it should make shadow/remove any light going into building
this room would be fully dark

also ambient and reflected light are disabled because of this

Your scene's walls are infinitely thin:

This causes problems when using global illumination techniques. You need to make your walls thicker in your 3D modeling software to prevent light leaking.

It is more like directional light does not care about walls

@Calinou if you look at this images you can see big cubes that shoulds block light from DirectionalLight3D (SUN) inside corridor

Changing rotation

Disabling SDFGI

my understanding of DirectionalLight3D is that it can be SUN

erm seams I might have been blind and did not see ability to make it use shadows, but also depending on how I set culling it can sometimes just not generate shadow


CREATE DOCUMENTATION TEMPLATE

📋 Description
This pull request enhances the documentation for improved clarity and usability, especially focusing on helping new users set up and navigate the project more effectively.
🔨 Changes Made

Improved formatting and organization in ISSUE_TEMPLATE
Added clearer instructions in the Installation Guide
Updated Configuration Guide with additional examples and explanations

✅ Checklist
Before submitting the PR, please make sure you have completed the following:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation (if applicable).
[x] My changes generate no new warnings.
[x] I have tested it locally and it works fine.
[x] Any dependent changes have been merged and published in downstream modules.

🏷 Types of Changes
What type of changes does your code introduce? (Check all that apply)

[ ] Bug fix (non-breaking change which fixes an issue) 🐛
[x] New feature (non-breaking change which adds functionality) ✨
[ ] UI enhancement (non-breaking change which enhances UI) 🎨
[x] Documentation update 📚

🤝 Related Issues

Fixes #192

@notsoocool the task has been completed. Kindly review the PR

@notsoocool

@notsoocool the task has been completed. Kindly review the PR


autogenerate .my.cnf in CLI container

it would be very cool if the CLI container autogenerates an .my.cnf so the users can just run mysql in the cli.
As the CLI container itself doesn't really ahead if there is a mariadb or other service existing, we probably need to do some clever autodiscovery if mariadb is resolveable and based on that create the .my.cnf.
For User/Pass inside .my.cnf we can use the global env variables  MARIADB_DATABASE MARIADB_USER MARIADB_PASSWORD
Important: for the mariadb-drupal Image they are drupal where for mariadb they are lagoon, but we have a cli-drupal Image so we can overwrite that there

From what I've seen so far, intra-pod communication for this sort of metadata seems like it would be extremely tricky, add starting dependancies, and so forth. Could something like this benefit from a central secrets controller, such as #65.
Would it make sense for oc-build-deploy-dind to generate some passwords, which could then be pulled out of the environment and in multiple pods, such as MARIADB_USER? maraidb root password and maxscale password could also make use of this, perhaps via templates?
It could also be a prototype for other resources such as generating a key that is used for encrypting files in git.

this has been implemented


Tags incorrect for hashtags

Describe the bug
I created a note for a test starting with #test on primal.net. Tags result was "tags": [ ["t", "est"] ]
Steps to reproduce
Create a note containing a hashtag (maybe only starting with hashtag? Not tried)
Expected behavior
"tags": [ ["t", "test"] ]
Environment

Starting a note with a hashtag
hashtag starting with a t

Additional context
ID of the created note using primal.net here nevent1qqsfx3y678xl9nfnxfj87wxtthdacqmaef4gmcmd7cwg5vz3ma2r2scfcktqv
https://www.nostr.guru/e/93449af1cdf2cd3332647f38cb5ddbdc037dca6a8de36df61c8a3051df543543

Yes, I was able to duplicate the circumstance.

"tags": [     [       "t",       "est"     ]   ],
I will try to solve it

The problem
Slicing text from hashtag works only for words with blank space before the hashtag _#test
Example note: #test another #onlyzap for a project #zapathon #eurothon
The only word without a blank space before it is the word "test."
For the code:
// Parse hashtags to add to tags
  while((match = regexHashtag.exec(value)) != null) {
    hashtags.push(match[0]);`
  }

This is the output: ["#test", " #onlyzap", " #zapathon", " #eurothon"]
You can see the empty blanks
And when slicing the hashtag:
tags = hashtags.map(h => ['t', h.slice(2)]);
This is the output: [["t", "est"], ["t", "onlyzap"], ["t", "zapathon"], ["t", "eurothon"]]
Solution
Trim words before slicing

  // Parse hashtags to add to tags
  while((match = regexHashtag.exec(value)) != null) {
    hashtags.push(match[0].trim());
  }

And then change the slice to first letter after hashtag
tags = hashtags.map(h => ['t', h.slice(1)]);
I made the change but I cannot push the commit

Hi. Thank you for reporting this. The fix similar to what @unwired01 proposed will be in the next release

@moysa
I see that I made a mistake and erased my pr code.
I can restore it if you need.

Hi, no need. The fix is currently on the dev branch and will be merged into main on the next release (https://github.com/PrimalHQ/primal-web-app/commit/53edb4da29bc7396e7972cd89ad1d60cd99c48e7). It isn't exactly your proposed changes line-by-line, but essentially the same.

Fix for this has now been released in v0.74.8. Closing


Neural Network support

Hi,
I'm attempting to use this repo as a base for a neural network, capable of sorting images of cats and dogs.
However, once installed, this is sending my PC into a boot loop.
Any advice?

Yes,
this is a no-brain.

There a library for it, just run this

It's recommended that you don't use docker for running your neural network. I've heard horror stories...

Possible duplicate of #130


Connection:: beforeExecuting() cannot be executed on all SQL

Laravel Version: 9.*
PHP Version: 8.*
Database Driver & Version: mysql8

Description:
Connection:: beforeExecuting() cannot be executed on all SQL, such as the SET command. See MySQL Connector.
It is expected that all SQL can be processed by Connection:: run().
In my application, I use database middleware, such as shardingsphere or mycat.
Select the data source by adding annotations before SQL, as follows:
/* ShardingSphere hint: dataSourceName=db_1 */select * form user;
In addition, in order to ensure the use of transactions, it cannot directly use the PDO method(PDO::beginTransaction() PDO::commit （） PDO::rollBack()) to execute transaction operations, but needs to execute transaction SQL. All the above needs to be annotated, which leads to the need for larravel to provide the method type to execute transaction operations.

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


Question: url to stream in browser

Hi,
I'd like to possibly use the url that is emitted from peerflix to stream video in the browser. I'm wondering what the best way to do this might be? Also, do I need to always have the vlc flag on in order to get the url?
Thanks.

Check out peerflix-server it's basically already done in that package.

I made an NPAPI Plugin for this:
https://github.com/jaruba/PowderNPAPI
Not sure how useful it will still be considering that NPAPI's will be deprecated in Chrome from September, but I think it's worth checking out. :)

If you want to see how to use peerflix-server, peep this in action. https://github.com/tomekmarchi/Mixee

:+1: Thanks guys, these suggestions are great and exactly what I'm looking to do. I'm going to fiddle more with the peerflix-server and see if I can get it to work with an app I'm working on. Cheers :)

If your working on an app, then the best way to do it is by using websockets to pass the link to the browser page.

@jaruba Yeah, I guess that was more my question as I think I was doing it in a very naive way. Here is how I am currently trying to do it:
app.get('/torrent-stream/:magnet?', function(req, res) {
  var cmd = exec('peerflix "' + req.query.magnet + '" --vlc', function (error, stdout, stderr) {
    res.send({ data: stdout })
  });
});

As you can see, I pass magnet link to server which then executes the peerflix command programmatically. This poses two problems:

A vlc instance is started automatically and I really want to just stream the video through the browser
The ip address is nested inside that stdout and I need to do work to strip it out of there.

Basically my question is, is there a better way to use peerflix to get this address without running an instance of vlc?

There's the webplay option that will open the browser for you: https://github.com/mafintosh/peerflix/blob/master/app.js#L36

cool, thanks @asapach! That is def getting closer to what I'm looking for.

@mjw56 Check out this stackoverflow example of how to use websockets to send data from an app to a web page.
Is there a particular reason why you are calling peerflix like that? You can get the peerflix http server link by requiring peerflix directly in any node.js environment.
var peerflix = require('peerflix'),
    engine = peerflix(magnetLink),

engine.server.on('listening', function() {
    var myLink = 'http://localhost:' + engine.server.address().port + '/';
});

Where magnetLink is any magnet link or link/path to a torrent file, and myLink is the streaming link.

@jaruba ah, sweet success. This is exactly what I am looking for.
I wasn't aware you could call peerflix with magnet link like that, so that's why I was doing it the naive way. This is the simple solution that I was looking for! I got something working using this with websockets, many thanks @jaruba.

Glad to help, this is the best solution because you could later on send more relevant data (like download %, number of peers, etc.), you could even make your app into a daemon/windows service by using a the Forever module. :)
You should also check out the torrent-stream Readme.md that will show all events, etc. that you can use for engine. The only difference will be that in the docs you will use peerflix() instead of torrentStream() to initialize the module.

I can second that notion I use this method except with express and socket.io the link I supplied has a ton of code already with all that setup feel free to use as you see fit. Great examples @jaruba ! Websocket method is by far the most effective.


[Docs]: invalid md file links

Current State
Description of the current state/issue:

Click the Getting Started link.
The 404 Page Not Found appears.

Link to the page:
https://apisix.apache.org/docs/apisix/installation-guide/
Screenshots (if relevant):
Click the Getting Started link.

404 Page Not Found

Desired State
Description of the change you are proposing:
Clicking on Getting Started,  the page should redirect to Getting Started.
Would you like to fix this issue?
Yes! Count me in

This issue is related to apisix repo, move this issue to
https://github.com/apache/apisix/issues/9500


tests: flash : common: rt10xx: NXP flash does not support negative operation

Describe the bug
test_negative_flash_fill cause hard fault for NXP RT series platforms
Please also mention any information which could help others to understand
the problem you're facing:

What target platform are you using?
mimxrt1060_evk
What have you tried to diagnose or workaround this issue?
this fails when the negative test merged
Is this a regression? If yes, have you been able to "git bisect" it to a
specific commit?
1a48a0174fd519f08703c1e1c0658c545bab3372

To Reproduce
scripts/twister -p mimxrt1060_evk -T tests/drivers/flash/common
Expected behavior
test pass
Impact
flash operation
Logs and console output
*** Booting Zephyr OS build v3.7.0-1447-g8617e4684937 ***
Running TESTSUITE flash_driver
===================================================================
Test will run on device is25wp064@0
START - test_flash_erase
 PASS - test_flash_erase in 0.048 seconds
===================================================================
START - test_flash_fill
 PASS - test_flash_fill in 0.042 seconds
===================================================================
START - test_flash_flatten
 PASS - test_flash_flatten in 0.043 seconds
===================================================================
START - test_flash_page_layout
start_offset=0x622000   size=4096       index=1570
page_count=2048
 PASS - test_flash_page_layout in 0.006 seconds
===================================================================
START - test_negative_flash_erase
TEST_FLASH_START = 0x0
TEST_FLASH_SIZE = 0x4000000
 PASS - test_negative_flash_erase in 0.005 seconds
===================================================================
START - test_negative_flash_fill

    Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/flash/common/src/main.c:373: flash_driver_test_negative_flash_fill: (rc < 0 is false)
Invalid use of flash_fill returned 0
 FAIL - test_negative_flash_fill in 0.016 seconds
===================================================================
START - test_negative_flash_flatten
 PASS - test_negative_flash_flatten in 0.001 seconds
===================================================================
START - test_negative_flash_read
E: ***** BUS FAULT *****
E:   Precise data bus error
E:   BFAR Address: 0x5fffffff
E: r0/a1:  0x80004ab0  r1/a2:  0x5fffffff  r2/a3:  0x600001ff
E: r3/a4:  0x80004aaf r12/ip:  0x800006f1 r14/lr:  0x8000070d
E:  xpsr:  0x81000000
E: Faulting instruction address (r15/pc): 0x6000c5ac
E: >>> ZEPHYR FATAL ERROR 0: CPU exception on CPU 0
E: Current thread: 0x80004100 (test_negative_flash_read)
E: Halting system

Environment (please complete the following information):

OS: (e.g. Linux, MacOS, Windows)
Toolchain (e.g Zephyr SDK, ...)
Commit SHA or Version used: v3.7.0-1447-g8617e4684937

Additional context
due to #77895 please reproduce this issue with v3.7.0-1447-g8617e4684937

@hakehuang I'm unable to reproduce this issue on cdb9166b810cb7b663:
TESTSUITE flash_driver succeeded

------ TESTSUITE SUMMARY START ------

SUITE PASS - 100.00% [flash_driver]: pass = 6, fail = 0, skip = 0, total = 6 duration = 0.321 seconds
 - PASS - [flash_driver.test_flash_copy] duration = 0.102 seconds
 - PASS - [flash_driver.test_flash_erase] duration = 0.060 seconds
 - PASS - [flash_driver.test_flash_fill] duration = 0.049 seconds
 - PASS - [flash_driver.test_flash_flatten] duration = 0.065 seconds
 - PASS - [flash_driver.test_flash_page_layout] duration = 0.026 seconds
 - PASS - [flash_driver.test_read_unaligned_address] duration = 0.019 seconds

------ TESTSUITE SUMMARY END ------

===================================================================
PROJECT EXECUTION SUCCESSFUL

Can you check on your side?

Can you check on your side?

they move this to another test and only Nordic platforms allowed ffd065f93e535d4cd5849e1f17a15717ce7ad173


Link mount/unmount and request/response events directly

Linked request/response and mount/unmount events with their eventIndex.

lgtm, is the idea to link these in the logging? This just sets the stage for accessing the prior linked event?

yes exactly. want to log the eventIndex of the corresponding pair directly.


Make permissions_caching_test more resilient against delays

Occasionally, auth_test:TestAuth.permissions_caching_test can fail unexpectedly. Investigation suggests that this may be due to the cache expiring before the test has a chance to run the query that verifies the cached permissions. This patch adds a retry in the case that the cache is found to have expired earlier than expected.

+1


Typing text in the bash: existing text is overwritten, new text is not inserted

A brief description
When I type text into the bash, the existing text is overwritten. The new text is not inserted.

Expected results
If I type new text, it should not overwrite the existing one. Outside of the bash, the result is the expected one.

Your Windows build number
14393.0

Steps / All commands required to reproduce the error from a brand new installation
Just an open bash.

@jepster What PC are you using? More specifically, what graphics card (if any)?
Does this happen in cmd.exe or powershell? or is it only bash.exe?
Could you post a screenshot?

I am having the same / similar issue.
Basically, lets say I want to re-run the last entered command in a bash shell, but tweak some item in the command string. I hit the up arrow on my keyboard to repeat the last command, I arrow key over to where I want to make the change.
Now when I use the delete key, it doesn't (visually) display the deleted characters being deleted. And it appears to overtype new characters on what is left there.
In actuality, it DOES change the text, as evidenced when pressing enter to submit the command, but its near impossible to actually do this reliably since the modification cannot be seen.

@zadjii-msft The issue appeared in the bash.exe. I've tried to use the bash.exe inside the PHPStorm IDE, but there I cannot use the arrow keys of my keyboard.
Here is some info about my PC. It is in german, but I guess that you are able to understand the system info from the context.

Here is info about my graphics card:

I have recorded a screencast of the behavior. You can get to the video by the following URL: https://drive.google.com/file/d/0B5vmNAcUsVUgSWtBS2pNTmFISGc/view
How to reproduce this bug?
At the beginning, the text typing is ok. I have opened a file in VIM, edited it a bit. Then the behavior was still ok. Afterwards I was installing a program via "sudo apt install youtube-dl". Then I wanted to remove it from the system by "sudo apt purge youtube-dl". Afterwards the problem appeared. However - you can see this in my screencast - the issue disappeared after a while if typing.
What could this have to do with my graphics card? I doubt that the issue is related to the graphics card.

Just posting to confirm the screencast video Jepster posted is the identical behavior I am experiencing. I am experiencing it on multiple systems of different configuration, including a Hyper-V Server hosted Win10 VM, indicating (to me) there is no GPU issues at all.
I also experience similar behavior in that it does not occur until after the bash shell has been open for some indeterminate amount of time.

@jepster @diabolicallyrandom Hey so we may have found a hint in figuring this out.
Does this happen specifically after using apt to install something? Or can this happen in a fresh conhost.exe instance without really doing all that much?
I believe there's some program somewhere in the stack that's emitting something to cause conhost to get in a bad state, if only occasionally.

For my part, I have always done something prior to it occurring of course. I am typically using SSH to connect to my Ubuntu server, or my OpenWRT router. However I also had it occur after a session in which I used apt to install image magic and work with some files on my mounted windows drives.
If I can fine the time I will try and replicate the behavior without doing anything in the shell, because I swear I had that happen too.
Just to confirm, this happens on command prompt bash, PowerShell bash, and launching the bash shortcut directly. And it usually (although sometimes) doesn't happen while I'm actively using it, but often it's when I use it and walk away for some time and come back.
Would there be a way I could provide some kind of debug logging that might help?

If you could help get a reliable repro, that'd be huge. This has proven to be wildly difficult to get a consistent repro from. Just now, I found that on a small enough window with a sufficiently large screen buffer, alternating sudo apt install nmap and sudo apt remove nmap can cause some input line corruption. However, there's a good chance I'll come back to my desk on Monday and it'll be gone

>Just to confirm, this happens on command prompt bash, PowerShell bash, and launching the bash shortcut directly.
For the record, all of those are all the same bash.exe running in conhost.exe. If it happened in plain old cmd.exe (no bash involved), now that certainly would be a bigger issue, but seeing how it's only a bash issue we can probably assume that there's some VT sequence that's messing with the console.

Guys, I was typing a lot in the bash and I had this buggy behavior after I have run some PHP command line application. Then - in the hope that this was the issue causer - I have run the app again. But the Bash was behaving correct. Could it be, that there is any issue with the memory of the bash? Because the problem was never appearing directly but only after a while of working in the Bash. That would also describe why this issue happens with "apt". Because "apt" creates a lot of output on installing any application. Well, this is only "guessing". Though a reasonable indicator.

I'm consistently seeing this problem on 14965. Vim is also affected. Demo: https://www.youtube.com/watch?v=eDd9A8YJCH8

I have the same problem as @lilred
SSH via bash on windows is almost unusable with that bug...

I eventually was able to get a consistent repro on this bug, so I'm opening MSFT:9871791 to investigate further. This one's probably the highest priority bug ATM.

I am glad to read that. Thanks for your quick feedback.

@zadjii-msft: Any news regarding the bug? Where can we see the state of "MSFT:9871791"? I guess this is a Microsoft bug tracker which is hidden for the public.

It is our internal tracking number. It's under investigation, but I've been out for the holidays, so nothing new to report.
From: Peter<EMAIL_ADDRESS>Sent: Wednesday, November 23, 2016 12:20 PM
To<EMAIL_ADDRESS>Cc: Mike<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [Microsoft/BashOnWindows] Typing text in the bash: existing text is overwritten, new text is not inserted (#1307)
@zadjii-msfthttps://github.com/zadjii-msft: Any news regarding the bug? Where can we see the state of "MSFT:9871791"? I guess this is a Microsoft bug tracker which is hidden for the public.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/BashOnWindows/issues/1307#issuecomment-262620841, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ARgZ1q7iJ9r1LH3qwN1u1LVbwdWIryonks5rBJ_tgaJpZM4KmVS8.

Any news? (:

@jepster Nothing yet. I think I got it pinned down in the code, but I don't have a fix tested yet.

So, breakdown of what's going on here:
apt install (NOT apt-get install) uses '\x1b[1;{{height-1}}r' to set the scrolling margins of the terminal to one less than the entire terminal while it's displaying output (including that neat little loading bar of theirs). They then use '\x1b[1;{{height}}r' to reset the margins back to the entire screen height, effectively removing them.
Typically, setting the scroll margins is only done inside the Alternate Buffer on a terminal, eg, within an application like vim, and not usually on the main buffer. Resetting the margins by setting to the entire terminal height is fine on the alt buffer, because it is exactly the size of the terminal window (it has no scrollback).
However, conhost doesn't really have a traditional "scrollback", and if you set the margins to the entire terminal on the main buffer in conhost, then they would stay there, even once you scrolled away. This could create scenarios where certain text manipulations would be clipped to the old scroll margins (which were no longer on the screen). This would cause visual artifacts in the output buffer, even though the bash input line still logically knew the correct data.
I've put the fix in code review today. Thanks for all the help on this one everyone, pinning this one down was particularly challenging.

Thanks for your very detailed feedback! It's great that Linux gets better and better as a subsystem in Windows by your effort. Can you share with us how the process looks like, until we will receive your update on our Windows 10 systems? Do you have an idea how much time it will take more or less?

@jepster Basically, the process for a pretty trivial fix like this is as follows:

it'll bounce around code review a bit (for such a small fix, this doesn't take too long)
it then gets checked in to our teams source branch.
It now needs about 3 weeks to get up to winmain. This time varies depending on how stable winmain is, or if our sister team's quality remains good, or hey, even a holiday break (like the one at the end of the month) can put this off by a week. 3 is what I've found to be the average. Sometimes all the pieces line up just wrong and this can be MUCH longer.
Once it's in winmain, it's usually the following week's insider build that has the actual fix in it. I believe it goes into the fast ring then, but I don't really know enough to know how long it takes to go from Fast -> Slow.
And it'll hit non-Insiders whenever the Creator's Update hits

@zadjii-msft: Thanks for your quick reply. Sounds like a straight forward process.
I am an non-insider. Can you tell me, how much difference there is between the non-insider and the insider release?

@jepster Easier for me to just link this: http://www.windowscentral.com/all-you-need-know-about-windows-insider-rings

Just to verify, this fix should fix any instances of similar behavior, not just those triggered by apt, correct?

@diabolicallyrandom Theoretically yes. Turns out there was a few related bugs related to the scroll margins that I all just checked in. I'm marking this particular issue as "fixinbound" and I'll close once it gets to Insiders.
If it seems to resurface once those changes are in Insiders, then it's probably an unrelated bug causing those, and I'd love a new issue opened to track it, to help single out the root cause

OK Thanks!

The fix for this has just been released with Insiders build 15014.

I'm seeing this with the latest bits from the CU on bash 4.3.48(1). Should it not have been fixed?

@asampal I'm gonna refer you to #1756, which is basically tracking this issue since creator's update. apt install causing the input line to act weird should be fixed in Creator's Update

@zadjii-msft , I'm not sure I follow - are you saying that #1756 (which describes problems on the input line) should be fixed for me? In any case, I commented there as well.

I am having this issue to when I run the mongo shell in bash(babun). Is it the kind of thing that is just predetermined by the software you're using? Should I write to mongo and ask for a fix?

@JessePotterMusic If this is still happening to you, and you have some reliable repro steps, you should file a new issue here. That's the best way to get me to start investigating :)

It's still happening for me as well.

Also still having this issue. Surprised it hasn't been fixed yet.

If this is still happening to you, and you have some reliable repro steps, you should file a new issue here.

The issue this thread was tracking was fixed in 2017. If you're still seeing it, then it's likely a different bug, with similar symptoms.

I realize this issue is closed, but wanted to post here for folks like me who still see similar symptoms and happen across this link

In a possibly related issue (#2713) , I found a workaround that worked for me.

These work around items (Setting the text wrap checkbox and adjusting PS1) did not result in any noticeable difference in behavior, and text in remote sessions is still being overwritten.

This stopped for me after switching to Debian from ubuntu on wsfl.

Can confirm this for me as well.  I tried Ubuntu18.04 LTS and Ubuntu 20.04 LTS (both direct from Microsoft Store) and both had the overwriting issue.  When I installed Debian, I see normal behavior where text is no longer being overwritten.

export TERM=vt100 on my ubuntu/debian
this removes color from my terminal, instead I set it to this value:
export TERM=xterm-256color
now it works fine. It was xterm-color previously, set in my decade-old .bashrc file that I carried around.


iw: 5.16 -> 5.19

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/iw/versions.
meta.description for iw is: "Tool to use nl80211"
meta.homepage for iw is: "https://wireless.wiki.kernel.org/en/users/Documentation/iw"
meta.changelog for iw is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.19 with grep in /nix/store/h3i191jxlrna67f487300j5zv3lsjqhh-iw-5.19
found 5.19 in filename of file in /nix/store/h3i191jxlrna67f487300j5zv3lsjqhh-iw-5.19

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

9 total rebuild path(s)

9 package rebuild(s)

First fifty rebuilds by attrpath
aircrack-ng
airgeddon
i3status-rust
iw
linssid
linux-router
linux-wifi-hotspot
tlp
wifite2

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/h3i191jxlrna67f487300j5zv3lsjqhh-iw-5.19 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/h3i191jxlrna67f487300j5zv3lsjqhh-iw-5.19
ls -la /nix/store/h3i191jxlrna67f487300j5zv3lsjqhh-iw-5.19/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

9 packages built:

aircrack-ng
airgeddon
i3status-rust
iw
linssid
linux-router
linux-wifi-hotspot
tlp
wifite2

Maintainer pings
cc @viric @primeos for testing.

Result of nixpkgs-review pr 184225 run on x86_64-linux 1

9 packages built:

aircrack-ng
airgeddon
i3status-rust
iw
linssid
linux-router
linux-wifi-hotspot
tlp
wifite2


Dask arrays are replaced with different objects in a forked process worker

What happened:
I create a dask (lazy) array, then schedule an operation in a dask worker, which refers it as a free or global variable (i.e. not as an argument).
Within the worker code execution, the resulting (lazy) array object does not match the original in the parent code  :

A lazy masked array, accessed by a free variable within a process worker, has the meta type of a "normal" numpy array instead of a masked one.
additional custom instance properties on  lazy array do not exist in the worker
the "id" of the array object is changed (which does not happen for "ordinary" Python objects).

What you expected to happen:
The array object should appear the same in the worker as in the parent code,
-- as it does, when using a "threads" or "synchronous" scheduler.
It's possible that this behaviour is essential,
e.g. because a memory-copied object contains O.S. resources (like file handles) which cannot function in a forked process.
If so, however, that is really not obvious, and should probably be documented somewhere.
Minimal Complete Verifiable Example:
import dask
import dask.array as da
import dask.bag
import dask.config
import numpy as np

test_maskedarray = np.ma.masked_array(da.arange(3), [0, 0, 1])
global_lazy_array = da.from_array(test_maskedarray)
global_lazy_array.custom_property = 'exists'

def array_details(array):
    result_string = (
        f'  {array}'
        f'\n  id(x) = {id(array)}'
        f'\n  type(x._meta) = {type(array._meta)}'
        f'\n  .custom_property = {getattr(array, "custom_property", "<NONE>")}'
    )
    return result_string

@dask.delayed
def trial_worker(*args):
    result = array_details(global_lazy_array)
    return result

def run_case(**compute_kwargs):
    test_delayed_op = trial_worker(0)
    with dask.config.set(**compute_kwargs):
        result = test_delayed_op.compute()
    return result

print(f'array properties IN PARENT:')
print(array_details(global_lazy_array))

print('')
print(f'array properties WITHIN WORKER (threads) :')
print(run_case(scheduler='threads'))

print('')
print(f'array properties WITHIN WORKER (processes) :')
print(run_case(**{"scheduler":'processes', 'multiprocessing.context':"fork"}))

Sample result:
array properties IN PARENT:
  dask.array<array, shape=(3,), dtype=int64, chunksize=(3,), chunktype=numpy.MaskedArray>
  id(x) =<PHONE_NUMBER>24560
  type(x._meta) = <class 'numpy.ma.core.MaskedArray'>
  .custom_property = exists

array properties WITHIN WORKER (threads) :
  dask.array<array, shape=(3,), dtype=int64, chunksize=(3,), chunktype=numpy.MaskedArray>
  id(x) =<PHONE_NUMBER>24560
  type(x._meta) = <class 'numpy.ma.core.MaskedArray'>
  .custom_property = exists

array properties WITHIN WORKER (processes) :
  dask.array<array, shape=(3,), dtype=int64, chunksize=(3,), chunktype=numpy.ndarray>
  id(x) =<PHONE_NUMBER>81696
  type(x._meta) = <class 'numpy.ndarray'>
  .custom_property = <NONE>

Anything else we need to know?:

The same thing happens whether the operation is actioned via a dask Bag or a Delayed computation.
The problem is specific to a dask "processes" scheduler
-- results are as expected for a "synchronous" or "threads" schedulers.
replacing dask Bag/Delayed parallelism with a concurrent.futures.ProcessPoolExecutor does not show the same behaviour

Environment:

Dask version: 2.9.2 or 2022.05.2
Python version: 3.8
Operating System: linux
Install method : conda

@pp-mo Thanks for opening this!

It's possible that this behaviour is essential,
e.g. because a memory-copied object contains O.S. resources (like file handles) which cannot function in a forked process.
If so, however, that is really not obvious, and should probably be documented somewhere.

Indeed, this is to be expected with multiprocessing and distributed schedulers. I also agree that we can document this better, maybe in the Scheduling docs. Would you be interested in submitting a PR for this?
Here's a simpler example:
import numpy as np

from dask import delayed
from dask.distributed import Client

a = np.ones(shape=(3,3))

@delayed
def get_id(x):
    return id(x)

get_id(a).compute()

with dask.config.set(scheduler='threads'):
    print(get_id(a).compute())

with dask.config.set(scheduler='processes'):
    print(get_id(a).compute())

with Client() as client:
    print(get_id(a).compute())

Here's a simpler example:

. . .
@delayed
 def get_id(x):
     return id(x)
 . . .
get_id(a).compute()
. . .
with dask.config.set(scheduler='threads'):
    print(get_id(a).compute())
. . .

Actually, I really don't think this is the same thing :
since in this case the array is passed as an argument, what the called function will receive is an actual (computed) numpy array, and not the original lazy array object.
That is exactly why my example is using a global variable : because if it is passed as an argument, the Dask scheduler will compute it before passing to the worker.
((
N.B. aside ...
I have also trialled code that creates an inner function referring to a lazy array which is a call argument -- i.e. local variable in a call instance (closure), with the same results.
This technique is much like what one does for decorator functions.
This is actually really required for sensible usage, only I didn't show it because it is more complicated + confusing to read.
I mean something like this (not tested!) ...
def get_worker(this_array):
  @dask.delayed
  def inner():
    return array_details(this_array)  # 'this_array' is a free variable from the enclosing def

  worker = inner()  # call returns a delayed to process the specific array in this invocation
  return worker

worker1 = get_worker(dask_array_1)
worker2 = get_worker(dask_array_2)
result1, result2 = dask.compute([worker1, worker2])

))
So, the point of my query is that the array, as a general Python object should not need to be copied/duplicated/serialised when used like this -- unless it contains O.S.-specific resources, which I think it does not -- i.e. if it is just a set of information in memory like a "regular" Python object.
Because obviously, other "ordinary Python objects" are not modified in this way -- they exist for the forked worker process just as memory content, identical to the original.
Obviously I may be missing something here.
But I did at least confirm that referencing the array in the expected way does work successfully, if I replace the Dask scheduler with plain Python multiprocessing :  In such cases, the forked process does see an array identical to the original, and it can also be computed.
Apart from the "why", I also don't really even understand how the dask scheduler can "replace" this dask.Array object with something else in the forked process
-- perhaps it keeps references to all Dask objects~
--  or perhaps it scans the entire Python environment for variables containing Dask objects and replaces them that way?
That is just why this effect is such a big surprise :  how should the scheduler even "know about" that dask array which happens to be stored in a global variable (or a function closure, in the more complex, realistic code) ?

P.S. I certainly don't mean to disparage the docs, which have really got so much better in recent years !

how the dask scheduler can "replace" this dask.Array object with something else in the forked process

UPDATE:
I just discovered that the Dask scheduling is in fact copying other Python objects too.
E.G. with a global object created like this...
class NormalObject:
    pass

global_normal_object = NormalObject()
global_normal_object.custom_property = 'exists'

... a process worker (unlike a threaded one) does not see this object at the identical address.
So it must be a "copy".
Whereas, if the worker is a fork created via a concurrent.futures.ProcessPoolExecutor, then  it does have this object at the same address, as expected from a normal Linux fork, which has memory duplicated by "copy-on-write".
So I think serialisation really is involved ,after all, and is applies to the whole Python environment in some way.

@ian-r-rose Do you have thoughts on this issue and how we can improve the docs?

I think serialisation really is involved ,after all, and is applied to the whole Python environment in some way.

Update : I still don't get quite how+where this happens, but on reflection I'm more + more convinced that it is somehow inevitable.
That is ... you can't seriously expect workers in a Pool to have a code environment equivalent to a simple "fork", since they may be re-used.
So, I will muse a bit longer on how we can improve the description.
Meanwhile, good to see the #9240 "failure of serialisation" is now fixed ! :+1:


update

-backdoor-apk reconfigured to use apktool 2.2.2 in third-party folder because main debian repositories do not have yet this version available to install , and this version solves multiple compilation errors on new apk files .
-backdoor-apk can now work wit user apks outside its main directory.
-backdoor-apk will move the finished apk to its main directory under the name "app_rat.apk" and will make a cleanup of the files used during the compilation process .

@peterpt I appreciate the pull request but I can't merge these changes. I'll look into adding Apktool as a third-party tool.


Fix grammar issues in the JavaDoc comments

Comparison requires 'than', not 'then'.
Use 'a' instead of 'an' or vice versa.
The abbreviations 'i.e.' and 'e.g.' requires two periods.
Some words are spelled with hyphen
Some words spelled as one word instead of two
Repeated words (e.g. 'the the')
Missing comma
Plural instead of possessive
Possessive instead of "it's" (short for "it is")

I would replace "i.e." and "e.g." and other Latin abbreviation with their English equivalent. Back when I was writing books, our style guide did allow their use. Most non-native speakers (and some native) don't know the difference to boot.


build 0.0.9

[x] upstream requirements
[x] checked license
[x] created package from scratch with conda-skeleton pypi pymc-experimental

Linter check found the following problems:
The following problems have been found:
ERROR: clone/recipe/meta.yaml:19: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: clone/recipe/meta.yaml:21: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: clone/recipe/meta.yaml:40: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: clone/recipe/meta.yaml:53: http_url: http://github.com/pymc-devs/pymc-experimental is not https
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: clone/recipe/meta.yaml:19: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: clone/recipe/meta.yaml:21: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: clone/recipe/meta.yaml:40: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: clone/recipe/meta.yaml:53: http_url: http://github.com/pymc-devs/pymc-experimental is not https
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: clone/recipe/meta.yaml:21: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: clone/recipe/meta.yaml:58: http_url: http://github.com/pymc-devs/pymc-experimental is not https
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: clone/recipe/meta.yaml:21: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
INFO: clone/recipe/meta.yaml:48: invalid_url: https://www.pymc.io/projects/experimental/en/latest : URL domain redirect www.pymc.io ->
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: clone/recipe/meta.yaml:21: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
INFO: clone/recipe/meta.yaml:48: invalid_url: https://www.pymc.io/projects/experimental/en/latest : URL domain redirect www.pymc.io ->
Errors were found


Disabled HamburgerButtonInfo has the same foreground color as HamburgerMenu in Light theme

...as in subject. Is it possible to override default style of HamburgerButtonInfo? The problem is that there is DimGray color used by default in disabled state and it is "invisible" in menu.

There is no current way to do this.


API key as a class member without security handling:

Storing the API key as a public class member (std::string Api_Key;) is not secure. Sensitive data, like API keys, should be securely handled and not exposed in plain text. I would suggest secure API key storage and handling to prevent exposure
Regards Inderpreet Singh

Hi Inder, thanks for pointing it out I meant to change it but forget it.


Pagination tx

fix #61 and #173

Imagine you have 1 million transactions and accounts. The current implementation will query 15 latest, then 30 latest, then 45 latest, ... 1 million latest at a single query. This implementation is fine as a temporary solution. The current limitation is on the backend side, as I mentioned: https://github.com/nearprotocol/near-explorer/issues/61#issuecomment-574236190. It is still better than nothing, so let's have some code quality review and merge it, and refactor it later once the new backend indexing service is built.

fine, I will separate them into three, through and if you have better idea, just make high level component to let this better


Do not translate Huawei and Android

Tick the checkbox if you understand [x]:

[x] I have read and understand the pull request rules.

Description
Addendum to #2812 + remove iOS i18n leftovers
Type of change
Please delete any options that are not relevant.

Other

Huawei is kind of a special case since it does have a Chinese name 😂

What this even mean?

What this even mean?

It's just a name in Chinese with two words 華 (Hua) and 為 (Wei), that's why I kept it in the language files.

@Saibamen Given that Louis has raised the point that this is translatable in Chinese, I think this PR can either be reduced to just iOS or closed.

https://github.com/louislam/uptime-kuma/pull/3480


4.13 bloop app will not load

Describe the bug
appears to be a frontend problem. the app will not load onclick. qdrant.exe starts, uses 10% CPU and 12gb RAM, but the app will not
Expected behavior
bloop app should start normally
To Reproduce
i installed bloop bloop_0.4.13_x64_en-US.msi on windows 11 pro (updated today) on click qdrant.exe loads and causes fans to spin but bloop does not start
Processor	13th Gen Intel(R) Core(TM) i9-13900K   3.00 GHz
Installed RAM	64.0 GB (63.8 GB usable)
System type	64-bit operating system, x64-based processor
Graphics  NVIDIA GeForce RTX 3090
Graphics  NVIDIA GeForce RTX 3090
Screenshots or output
there is nothing to show bloop will not start
Additional context
i had 17 repositories including one really big one, maybe that has something to do with it. i have tried earlier versions and they will not start either. unfortunately i have seen this before in other programs, they will not start then 5 hours later or something they may start (and 12 instances load because of repetitive clicking). i think it has something to do with windows.

UPDATE:
clicked to start bloop about 2 hours ago, it would not start but qdrant.exe started and is using 15gb mem and about 10% cpu. after about 2 hours i noticed qdrant was no longer using cpu, so i clicked bloop icon on my taskbar and bloop app started, asked me to submit a crash report, after submitting report, app went blank meaning i could see the apps window or pane but bloop is non-functional.
i can now start bloop after letting qdrant.exe run for hours, but the app crashes every time and asks me to submit a crash report.
Processor 13th Gen Intel(R) Core(TM) i9-13900K 3.00 GHz
Installed RAM 64.0 GB (63.8 GB usable)
System type 64-bit operating system Win11 Pro, x64-based processor
Graphics NVIDIA GeForce RTX 3090
Graphics NVIDIA GeForce RTX 3090

@justinwj Do you still get this on the latest version of bloop?

@ggordonhall 0.5.7 also does not work on Windows 11

0.5.7 also doesn't load for me on MacOS Sonoma 14.0 (M1 Max) :/


[enhancement] removed account_money_enable flag && private key from preference

removed account_money_enable flag && private key from preference

Codecov Report

Merging #1159 into enhancement/v4 will increase coverage by 0.01%.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##           enhancement/v4    #1159      +/-   ##
==================================================
+ Coverage           25.96%   25.98%   +0.01%
==================================================
  Files                 336      336
  Lines               14422    14413       -9
  Branches             1305     1304       -1
==================================================
  Hits                 3745     3745
+ Misses              10225    10216       -9
  Partials              452      452

Impacted Files
Coverage Δ

...om/mercadopago/android/px/internal/di/Session.java
44.06% <0%> (ø)
:arrow_up:

...pago/android/px/plugins/model/BusinessPayment.java
0.83% <0%> (+0.03%)
:arrow_up:

...go/android/px/paymentresult/props/HeaderProps.java
54.9% <0%> (+3.05%)
:arrow_up:

Continue to review full report at Codecov.

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


Gatherer + Find-Object

Hi Mathieu,
Hope you are doing all well !
I found this interesting project: https://github.com/headupinclouds/gatherer. It is a cross platform gpgpu opengl (es) shader pipeline framework: ogles_gpgpu + qt5 (UI). It's really convenient to compile mobile or desktop version of QT5 based apps and leverage the GPU for the GUI interface (Camera in this case).
They have a repo with several packages to easily in the CMakeLists.txt:
Github account: https://github.com/hunter-packages
Adding find-object as a new package would be interesting to add value to the gatherer project. The only problem is the way the UI is templated in the current version of Find Object. Maybe if we migrate to a QML responsive design the UI of find-object: it would make things easier.
Did you ever work with the QML features of QT 5 ?
What do you think about adding find-object in the gatherer project ?
Would it help you to have a smoother visualization of the features detection/extraction ?
Cheers,
Luc Michalski

Hi Luc,
I am doing well! Hope you are too!
I didn't use QML yet, but I think it could not be so difficult to migrate the *.ui to *.qml. Not sure about if the ui would be setup the same way in c++ though (so that c++ code may have to be also be modified). Having the UI running on GPU could indeed save some time for Find-Object's feature processing. I have also seen that for Tegra, OpenCV has some optimizations to use the GPUs too.
cheers,
Mathieu


Suggest >>= _ -> to >>

From #362 by @mitchellwrosen, it occurs to me that we could suggest a >>= \_ -> b to a >> b... That can be encoded as an expression rule very easily, and even automatically does the right thing about free variables (a >>= \c -> b ==> a >> b will fail to apply if any variable is free in the revised expression but not the original).
However, I'm slightly hesitant because of incorrect monad instances, and too much time in a strict Haskell-like language where this doesn't work.

Done since it makes sense.


Replaced sha256sum with shasum, and fixed echo bug

I made some changes:

sha256sum -> sahsum -a 256
echo -n -> echo

First is for crosslinux support (e.g. + OS X)
Second fixes terrible bug for url in OS X (something like "https://dl.bintray.com/flant/multiwerf/-n v1.0.15"). "echo -n" just adds " -n" to version

Please verify that solution on other linux'es like centos/ubuntu/alpine

sha256sum -> sahsum -a 256

As i can see, shasum is a part of perl installation. And Perl is not available in modern linux or docker images by default. sha256sum is a part of coreutils or busybox and is available by default. The best approach here is to create a $OS dependent function and call shasum -a for macos and sha256sum for linux.

echo -n -> echo

Totally agree. -n flag is redundant here, because trailing newline is already removed by get_location_header.
So I merge this PR and create a new one with fixes for linux.
Thank you for testing on MacOS!


Improve AppearanceForIconCount

I have a few ideas about how this class can be improved.

Make the collection generic instead of expecting Appearance.
Reassignment is probably ok. I should remove the check.
The class should be initialised with a length (14 or 15) rather than using the boolean.
Is allowing a single key really a good idea? To keep it consistent, just take a 2-tuple.
Implementing __delitem__ makes it fit with MutableMapping. It certainly is mutable either way, so might as well.
When in the range, the mapping will always have a value. That's None if we haven't set it or we've deleted it. Perhaps raise an exception when it's None.

Implemented all sensible changes. As for the None thing, I just use a dictionary internally now so you'll get KeyErrors where appropriate.


[REVIEW]

Github Profile Link: https://github.com/Buddhad

You can also join the discord community here
Feel free to check out other cool repositories of CodeVisors Community here

Because I want to stay on top of everything that goes on in this organization and of course, as a good dev, you have to know about the challenges of this incredible world of programming


transport: resolve DNSNames when SAN checking

The current transport client TLS checking will pass an IP address into
VerifyHostnames if there is DNSNames SAN. However, the go runtime will
not resolve the DNS names to match the client IP. Intead, resolve the
names when checking.
/cc @gyuho

an alternative to this is to rip out the DNSNames check entirely and only check if IPAddresses is set

Codecov Report

Merging #7767 into master will increase coverage by 0.15%.
The diff coverage is 48.38%.

@@            Coverage Diff             @@
##           master    #7767      +/-   ##
==========================================
+ Coverage   75.61%   75.77%   +0.15%
==========================================
  Files         331      331
  Lines       26053    26085      +32
==========================================
+ Hits        19700    19765      +65
+ Misses       4917     4891      -26
+ Partials     1436     1429       -7

Impacted Files
Coverage Δ

pkg/transport/listener_tls.go
73.91% <48.38%> (-10.81%)
:arrow_down:

proxy/grpcproxy/register.go
69.44% <0%> (-13.89%)
:arrow_down:

pkg/testutil/recorder.go
66.66% <0%> (-3.71%)
:arrow_down:

pkg/netutil/netutil.go
77.94% <0%> (-2.95%)
:arrow_down:

rafthttp/transport.go
81.92% <0%> (-1.13%)
:arrow_down:

clientv3/watch.go
94.92% <0%> (-0.26%)
:arrow_down:

etcdserver/raft.go
88.39% <0%> (+0.24%)
:arrow_up:

etcdserver/server.go
80.25% <0%> (+0.59%)
:arrow_up:

clientv3/client.go
81.54% <0%> (+0.73%)
:arrow_up:

etcdserver/v3_server.go
71.19% <0%> (+0.95%)
:arrow_up:

... and 10 more

Continue to review full report at Codecov.

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


How to set cosine lr update in config

lr_config = dict(
    policy='cosine',
    warmup='linear',
    warmup_iters=250,
    warmup_ratio=1.0 / 3,
    step=[8, 11])

Something like that?

It seems that 'cosine' has been renamed to 'CosineAnnealing'.
https://github.com/open-mmlab/mmcv/issues/260
The following config worked for me :
lr_config = dict(policy='CosineAnnealing', min_lr_ratio=0.001, by_epoch=False)


💭 how can I setState from onRecordingFinished?

Question
I am trying to call a setState function from inside onRecordingFinished:
const [state, setState] = useState('')

const handleStartRecording = () => {
    setIsRecording(true);
    cameraRef.current.startRecording({
      fileType: "mp4",
      onRecordingFinished: (video) => {
        setState(state+1);
        console.log(video);
      },
      onRecordingError: (error) => console.error(error),
    });
  };

but state is still an empty string after the recording in finished. What am I doing wrong?
What I tried
I tried to useSharedValue() from react-native-worklets instead of useState but got the following error:
TypeError: Invalid attempt to destructure non-iterable instance.
In order to be iterable, non-array objects must have a [Symbol.iterator]() method.

This error is located at:
    in _default (at app/index.js:5)
    in _default (at useScreens.js:112)
    in Unknown (at useScreens.js:116)
    in Suspense (at useScreens.js:115)
    in Route (at useScreens.js:131)
    in Route(index) (at SceneView.tsx:132)
    in StaticContainer
    in EnsureSingleNavigator (at SceneView.tsx:124)
    in SceneView (at useDescriptors.tsx:218)
    in QualifiedSlot (at Navigator.js:116)
    in PreventRemoveProvider (at useNavigationBuilder.tsx:718)
    in NavigationContent (at useComponent.tsx:35)
    in Unknown (at Navigator.js:72)
    in QualifiedNavigator (at Navigator.js:52)
    in Navigator (at Navigator.js:115)
    in RNCSafeAreaView (at SafeAreaView.tsx:49)
    in Unknown (at Navigator.js:114)
    in DefaultNavigator (at useScreens.js:112)
    in Unknown (at useScreens.js:116)
    in Suspense (at useScreens.js:115)
    in Route (at useScreens.js:131)
    in Route() (at ExpoRoot.js:90)
    in RNCSafeAreaProvider (at SafeAreaContext.tsx:92)
    in SafeAreaProvider (at ExpoRoot.js:55)
    in wrapper (at ExpoRoot.js:89)
    in EnsureSingleNavigator (at BaseNavigationContainer.tsx:433)
    in BaseNavigationContainer (at NavigationContainer.native.js:105)
    in ThemeProvider (at NavigationContainer.native.js:104)
    in NavigationContainerInner (at ExpoRoot.js:86)
    in ContextNavigator (at ExpoRoot.js:64)
    in ExpoRoot (at qualified-entry.js:20)
    in App (created by ErrorOverlay)
    in ErrorToastContainer (created by ErrorOverlay)
    in ErrorOverlay (at withDevTools.js:18)
    in withDevTools(ErrorOverlay) (at renderApplication.js:57)
    in RCTView (at View.js:116)
    in View (at AppContainer.js:127)
    in RCTView (at View.js:116)
    in View (at AppContainer.js:155)
    in AppContainer (at renderApplication.js:50)
    in main(RootComponent) (at renderApplication.js:67), js engine: hermes
 ERROR  TypeError: Invalid attempt to destructure non-iterable instance.
In order to be iterable, non-array objects must have a [Symbol.iterator]() method.

This error is located at:
    in _default (at app/index.js:5)
    in _default (at useScreens.js:112)
    in Unknown (at useScreens.js:116)
    in Suspense (at useScreens.js:115)
    in Route (at useScreens.js:131)
    in Route(index) (at SceneView.tsx:132)
    in StaticContainer
    in EnsureSingleNavigator (at SceneView.tsx:124)
    in SceneView (at useDescriptors.tsx:218)
    in QualifiedSlot (at Navigator.js:116)
    in PreventRemoveProvider (at useNavigationBuilder.tsx:718)
    in NavigationContent (at useComponent.tsx:35)
    in Unknown (at Navigator.js:72)
    in QualifiedNavigator (at Navigator.js:52)
    in Navigator (at Navigator.js:115)
    in RNCSafeAreaView (at SafeAreaView.tsx:49)
    in Unknown (at Navigator.js:114)
    in DefaultNavigator (at useScreens.js:112)
    in Unknown (at useScreens.js:116)
    in Suspense (at useScreens.js:115)
    in Route (at useScreens.js:131)
    in Route() (at ExpoRoot.js:90)
    in RNCSafeAreaProvider (at SafeAreaContext.tsx:92)
    in SafeAreaProvider (at ExpoRoot.js:55)
    in wrapper (at ExpoRoot.js:89)
    in EnsureSingleNavigator (at BaseNavigationContainer.tsx:433)
    in BaseNavigationContainer (at NavigationContainer.native.js:105)
    in ThemeProvider (at NavigationContainer.native.js:104)
    in NavigationContainerInner (at ExpoRoot.js:86)
    in ContextNavigator (at ExpoRoot.js:64)
    in ExpoRoot (at qualified-entry.js:20)
    in App (created by ErrorOverlay)
    in ErrorToastContainer (created by ErrorOverlay)
    in ErrorOverlay (at withDevTools.js:18)
    in withDevTools(ErrorOverlay) (at renderApplication.js:57)
    in RCTView (at View.js:116)
    in View (at AppContainer.js:127)
    in RCTView (at View.js:116)
    in View (at AppContainer.js:155)
    in AppContainer (at renderApplication.js:50)
    in main(RootComponent) (at renderApplication.js:67), js engine: hermes

VisionCamera Version
4.0.0-beta.10
Additional information

[X] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar questions in the issues page as well as in the discussions page and found none.

I don't think this is related to VisionCamera.


Tap/Hold feature

Hi,
I saw that the Tap/Hold feature was on your todo list, any idea when it will be available ?
I'm using a Gherkin as my daily driver and well, this feature is essential.
Thanks !

Already implemented.
Here's an example


Does pulsar operator supports imagePullSecret

I am pulling the images for plusar cluster from a private docker registry instead of docker hub.
I could not find in the crd config anywhere we can pass imagePullSecrets.
Please let me know if this is supported somehow.

This feature is not currently supported and can be added later.

@sjmittal A little late to reply but you should be able to achieve this by attaching the pull secrets to the default service account for your pulsar namespace docs here


Change generator UI for the new map generator version

We are changing the parameters for the map generator here: https://github.com/FAForever/Neroxis-Map-Generator/pull/396

Do you have an idea why there is excessive space between the heading and the spawn number spinner? It's not because of the hidden command line field, because even if I enable that, there is still too much space under the heading.

I never was exactly sure, something with the gridpane I think

There is a weird interaction going on. When the heading text is the longest of all elements in the column, then it needs significantly more vertical space. This can easily be observed by shortening the text. I have no idea why this happens though


Invalid number literals causes crash

e.g. foo(123a).

Hey,
thanks for the bug report. I'm not able to reproduce this. Could you add an example source + query that triggers a crash?

Oh I think I have weggli configuration disabled, that is why. I can probably still reproduce if I re-configure my build. Is the screenshot sufficient?


DBZ-7456 Skip PK change LOB column reselect if column is excluded

https://issues.redhat.com/browse/DBZ-7456

/packit test --labels oracle


sudo: support insults (but don't enable by default)

I'm sorry. I've tried.
I've tried for the past half-year to get used to a system that doesn't scold me when I fat-finger my own password. That lets me simply try again - without strongly suggesting I should probably sober up first. Even apologising for my balderdash with a meek:
Sorry, try again.

I have failed.
This allows insults to be enabled in sudoers if desired.

Thanks!11!1!! This is a key feature that has prevented me and many others from using NixOS for the past several years. I am so glad that I can finally use the operating system I've been wanting to try for so long!

Thanks :+1: It was several weeks of hard work, but all worth it in the end.

Sadly, this adds 1.6MB to sudo which is a bit much.
This probably belongs in stdenv anyway.

Why not add this as a configurable option to the package? I would add
insultsSupport to the arguments there and then add some kind of option
to NixOS config to override this option in nixpkgs.

Are you not merging this?

@copumpkin: patience! Rome wasn't built in a day.
@oxij:

Why not add this as a configurable option to the package? I would add
insultsSupport to the arguments there

You read my mind! Almost. Hold tight for withInsults 2.0.

and then add some kind of option to NixOS config to override this option in nixpkgs.

Now you're just being silly.

I expect there to be a NixOS option to pipe my sudo insults through cowsay and lolcat by the end of this. Can there be? Pretty please?

...why optional?
This was an actual feature request. From an actual user. It was flagged as "important". I can't help but feel you're not taking them seriously. >:-(

No, my feature request is serious as well!


Disallow invoking Run-ALPipeline with keepContainer without specifying credential

If keepContainer is specified, credential must also be specified, as otherwise the container will be created with a random password and there will be no way to access it.

Please also remove lines 670 - 678 (the dump on whether credentials is specified) - there is no reason to dump this info anymore.


New World

Game Title
New World
Game Website
https://www.newworld.com
Anti-Cheat Software
Easy Anti-Cheat
Status of the Anti-Cheat
❔ Unconfirmed
Proof of Mention
N/A

duplicates #19