Pause Event not picked up on embedded YouTube

Hi
Moving to v3 from 2.6 I have a found a couple of things (see also multiple events firing when jumping to new play position):
With YouTube embeds now there is no pause event logged though play and playing are fine, and pause is fine with local / direct video playback.
$('#myplayerid').mediaelementplayer({
success: function(mediaElement, originalNode) {
		 mediaElement.addEventListener('play', function() {
        $('#output').append( $('play') );
    }, false);

		  mediaElement.addEventListener('pause', function() {
        $('#output').append( $('pause') );
    }, false);

}})
Regards
Simon

@SMen1 Thanks for pointing this. The multiple events is expected since that's how YouTube works: first triggers a pause, then triggers a play and finally a playing event when jumping to a new position. I fixed the absence of pause event. Please download the latest version from GitHub and let me know

Thank you that response was speed of sound... the pause event is firing now for YouTube - brilliant - confirmed.
Regarding the "pause play, playing" when a user jumps to a new playing position I am expecting this but
for local direct video I am getting:
pause play playing pause play playing playing
and YouTube
pause play playing pause play playing play playing
Now I do have two videos on the same page to test this - each have their own id, and are accessed with a new instance of mediaelement - if I am wrong here then ok, my mistake, but just in case it is something,  I mentioning it again in this request.
so video 1, $('#myplayerid1') and then again for YouTube with id $('#myplayerid2')
$('#myplayerid1').mediaelementplayer({
success: function(mediaElement, originalNode) {
		 mediaElement.addEventListener('play', function() {
        $('#output').append( $('play') );
    }, false);

		  mediaElement.addEventListener('pause', function() {
        $('#output').append( $('paused') );
    }, false);
		  mediaElement.addEventListener('playing', function() {
        $('#output').append( $('playing') );
    }, false);

    }
});

In version 2.6 using the demo events example I was getting just one instance of pause play playing or the equivalent for direct video.
Many thanks
Simon

Ok I'll check your scenario and keep you posted

My Update - if I only have one video then it only fires the single sequence, so the question is : should it be firing twice even when only one video is playing or active at a time - perhaps I have to actively set something now, with more than one video on a page.
Many thanks for looking.

Yes it should fire once per active video; let me see what needs to be added/fixed once I make my test case

@SMen1 Thanks for the patience. Please download the latest version from GitHub and let me know if you are seeing the multiple events. I found out that a play was being triggered when it shouldn't. I made a good deal of testing and seems to be working for me. But just confirm

Hi, no problem at all. I have downloaded the latest but I am getting pausedplayplayingpausedplayplayingplaying for local mp4 and pausedplayplayingplayplaying for YouTube.
I have emptied my cache etc. reloaded, but I won't be able to properly reset everything and test properly until 9 ish am +2 GMT tomorrow. But I am glad to help and so will resume tomorrow.
I have had a couple of issues starting YouTube with jquery, and pausing the current player in play, but that might actually have been resolved with your latest version - so to confirm all the above.
Regards, and on the contrary, many thanks for your speed in responding.
Simon

@SMen1 I'm posting my HTML so you see how I tested it and the result I obtained using the latest version:
<!DOCTYPE html>

    <meta charset="UTF-8">
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <title>MediaElement.js 3.0 - audio/video unification library</title>
    <meta name="viewport" content="width=device-width, initial-scale=1">
    <link rel="shortcut icon" href="favicon.ico" type="image/x-icon">
    <link rel="icon" href="favicon.ico" type="image/x-icon">

    <link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/normalize/5.0.0/normalize.min.css">
    <link rel="stylesheet" href="../build/mediaelementplayer.min.css">

Video Player

                <video id="player1" width="340" height="260" preload="none">
                    <source src="http://clips.vorwaerts-gmbh.de/big_buck_bunny.mp4" type="video/mp4">
                </video>

                <video id="player2" width="340" height="260" preload="none">
                    <source src="https://www.youtube.com/watch?v=YE7VzlLtp-4" type="video/x-youtube">
                </video>

Any updates on this?

Many thanks, yes ... you caught me typing it in ... I think this might be browser dependent.
These results are after creating a new folder to source the v3 files and replicating your code, with the exception of the two videos (using my own), but with mp4 then YouTube as per your example.  I get the same as you with Chrome for player 2  (which is the sequence I was expecting). With mp4 though I was previously getting playing playing, but now it is very different with all browsers.
Screenshot shows firefox. I manually play, then do one single click forward on the  jump / progress rail.

I can send you a temporary link, or please advise.  I am hoping this helps

Send me a link so I can look at it please

OK
http://www.globaldistrict.com/_administration/members_images_includes/testing_medialementv3.asp
On 16/02/2017 13:51, Rafael Miranda wrote:

Send me a link so I can look at it please
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/johndyer/mediaelement/issues/2079#issuecomment-280322126,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH_yW4iFJ6TQC3F00pTbG2ymQETfegU-ks5rdEY4gaJpZM4MBmYY.

OK I'll check this and let you know

@SMen1 After a lot of testing I ended up confirming that this is an issue with Firefox, and even YouTube itself has the issue. To prove it, just copy the following markup and perform the same action of sliding the rail and you will see the same behavior (in this case, 2 playing actions being executed):
<!DOCTYPE html>

    <meta charset="UTF-8">
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <title>MediaElement.js 3.0 - audio/video unification library</title>
    <meta name="viewport" content="width=device-width, initial-scale=1">

So on my side there's nothing more I can do about this. Thanks for reporting this though

I ran your earlier test using Mediaelement's v2 pushing the players into
an array (demo/mediaelementplayer-events), and for whatever reason, this
catches any repetitions and refires, so while playing, clicking on the
time rail results as follows:
for mp4: playing is followed by single "playing"
for YouTube: playing is followed by a single "play playing".
I can't get this to work with the v3 mediaelement-and-player.js file so
could I make this a feature request in V3? It may be you would be
planning this anyway.
Very many thanks for  your testing and support.
Regards
Simon
Different technicque used as below for v2:
// make an array for the mediaelement players
mediaElementPlayers = new Array();
$('audio,video').mediaelementplayer(
{
success: function (mediaElement, domObject) {
       // add this mediaelement to the mediaElementPlayers array
       mediaElementPlayers.push(mediaElement);

       var events = ['play','playing','pause','ended'];

       for (var i=0, il=events.length; i<il; i++) {

         var eventName = events[i];
         mediaElement.addEventListener(events[i], function(e) {

     $('#output').append( $('' + e.type + '') );
             var animationplaying = $('#animationplaying').val();

         });
     }}

});

Yes please send the feature request and I'll take a look at it

OK I don't think I can label it as a feature request so posting as a new
issue. Many thanks - hope correct.
On 17/02/2017 13:38, Rafael Miranda wrote:

Yes please send the feature request and I'll take a look at it
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/johndyer/mediaelement/issues/2079#issuecomment-280638573,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH_yW1rnJOruBh9tYQDkRaDGxWPScWdMks5rdZThgaJpZM4MBmYY.

Yes I'm the one labeling them so don't worry


[path_provider] Address privacy manifest requirement

This plugin is listed on https://developer.apple.com/support/third-party-SDK-requirements/ as requiring a privacy manifest. It's not currently clear why; we'll need to clarify why it's there and then either

add a privacy manifest, and figure out how to embed it, or
have it removed from the list if it's there in error.

See https://github.com/flutter/flutter/issues/131940 for context.

The guidance we have received is:

If an SDK doesn’t collect any data, then a privacy manifest should be provided outlining that no data is collected.

So we'll need to bundle an "empty" manifest.


1.19 towers spawn too frequently

Towers spawn 200 blocks away from eachother. I can't find anything tower spawn frequency related in config.

you need to create a datapack where you change the spacing and separation values of the structure set https://gist.github.com/misode/45559d34627755ecaa52497daea83544


Default sorting of selection lists

Selection lists, if they don't have a "user interface feature: order ..." should be sorted by "user interface label" in language of user interface.  This suggests sorting real-time in browser when ontology is loaded?

Partially implemented.  Categorical pick list choices (which don't have a user interface feature order annotation) are now sorted alphabetically by the jsonify.py script.  However, it is up to client-side applications to sort according to a particular language (unicode) alphabet.


Pretrained models available?

Dear author,
Thank you very much for sharing your great work! I hope to know are there available pretrained SEANet models on Coco Dataset?
Your reply is greatly appreciated. Thank you very much!

We will upload the models based on various backbones as soon as possible.


Auto corrected by following Lint Ruby Style/Next

Auto corrected by following Lint Ruby Style/Next
Click here to configure it on awesomecode.io

Coverage remained the same at 96.801% when pulling e4aa0e6d795450328f4a61d6445eaa939b9a694c on awesomecode-lint-ruby-style-next-48016 into 22ecfa4bd363433d74aa04fa34c03f124f91425e on master.

Coverage remained the same at 96.801% when pulling e4aa0e6d795450328f4a61d6445eaa939b9a694c on awesomecode-lint-ruby-style-next-48016 into 22ecfa4bd363433d74aa04fa34c03f124f91425e on master.


emoji box collapsing on clicking outside post

Issue #319
emoji box is now collapsing on clicking outside of the post. Initially, it was not collapsing.

Thanks for your contribution :)


chore(release): enable forcing a patch release to release open-install-library updates

@Julien4218 I looked at that action's workflow earlier as well. The release logic change what you were requesting was trivial and didn't involve needing to change our implementation of that the actions-ecosystem/action-get-latest-tag GH action.


Maple will not work in our benchmarking setting

Hi,
Maple is a model that requires Multi-sample dataset input to generate labels for individual samples (see publication (esp Fig. 1) and the code snippet below), whereas in our cases we input individual samples from a dataset. This method might not work under our framework.
The code from their fit_maple function:
  if(is.null(covars))
  {
    print("Note, no supplied covariates. Will use orig.ident as sample indicator.")
    sample <- as.factor(meta[,"orig.ident"])
    W <- as.matrix(stats::model.matrix(~ sample))
    rownames(W) <- rownames(meta)

Whenever I run it within our system, it returns an error as generally our orig.ident has only one level for each sample we input.
error message:
Error in h(simpleError(msg, call)) :
  error in evaluating the argument 'x' in selecting a method for function 'as.matrix': contrasts can be applied only to factors with 2 or more levels

@shdam Could you check if this is the case or any workaround? I can put Maple as a low-priority implementation for now.

Hmm, I thought it would work anyway since the single-sample tests went well..
Oh well, it is built on spruce, which may then be the actual method that should be implemented, as that is for single samples.

I just saw that spruce is no longer maintained, so perhaps we should exclude maple as well, as it uses spruce.

If it can work without error, I'd be happy to include the method. But I keep on getting the aforementioned error message. Did you have the same experience? Do you think there's any workaround?

Aha I see! Thanks for the suggestion. I will try to run it after the modification.
We should also dicuss a bit whether to keep the method in the next meeting, given that it's no longer maintained.

Yeah, it's unfortunate since it seemed like a decent method, but the P.I. behind it now works in industry.
I am curious to hear how it performs :)

Changed to spruce and it seems to work. Close the issue for now


Read "arrow" (IPC and streaming) files usning org.apache.arrow.dataset.jni.NativeDatasetFactory in Java API

How to fetch "arrow" (IPC and streaming) files usning org.apache.arrow.dataset.jni.NativeDatasetFactory in Java API. This chapter is missing in documentation. It is possible to do in CPP/Python API

Hi @igor-suhorukov,
I would need to check, but I don't think this is implemented in Java Dataset yet.
Do you need to use Datasets for your application? (in other words, is your data spread over multiple files or too big to fit in memory?) If not, you should be able to use a FileReader load the data.

This functionality required to implement Arrow file/Stream input format in my use case to process large amount of existing geospatial ARROW format data in Apache Spark data source. Optimized Analytics Package (OAP) for Spark also can leverage this feature of Dataset on JVM


refreshDeps 和 pollingInterval同时使用，导致一直无限请求

Describe the bug
  refreshDeps: true,
  pollingInterval: 1000 * 60 * 5,
  pollingWhenHidden: false

refreshDeps 和pollingInterval同时使用，一直无限请求接口， pollingInterval设置的间隔时间无效
Steps to reproduce
No response
System Info
"vue-hooks-plus": "^1.8.9"

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct.
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a VueHooks Plus issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.
[X] The provided reproduction is a minimal reproducible example of the bug.

这边建议给一个最小复现呢？

@zaxlct 目前已经修复，更新到了 2.2.0 版本。原因是，timeRef.value = setTimeout(xxxx) ,被 watchEffect 监听到了，一致循环的进行 refresh 操作。


Is the documentation out of date?

I've had a few issues with this, the latest one being this part from the ScriptObject documentation:
"Notice that when using a function with pipe calls like {{description | string.strip }}``, the last argument passed to the string.strip` function is the result of the previous pipe. That's a reason why you will notice in all builtin functions in scriban that they usually take the most relevant parameter as a last parameter instead of the first parameter, to allow proper support for pipe calls."
Instead the opposite (the "intuitive" way to order arguments) proved to be true for me.
I am using Visual Basic .NET if this has any effect on this.
Class ScribanFunctions

        'this works as expected on {{ date | format "dd.MM" }}
        Public Shared Function Format(input As Date, formatString As String) As String
            Return input.ToString(formatString)
        End Function

        'this throws Scriban.Syntax.ScriptRuntimeException:
        'Unable to convert parameter #1 of type `System.String` to type `System.DateTime`
        Public Shared Function Format(input As Date, formatString As String) As String
            Return input.ToString(formatString)
        End Function

    End Class

Either the documentation is outdated/wrong or i am misunderstanding something fundamental.

Either the documentation is outdated/wrong or i am misunderstanding something fundamental.

Yes, it is outdated. PR welcome


Add zaros spiritual creatures to be killable

Description:

Added zaros spiritual creatures to be killable,

Info from:
https://oldschool.runescape.wiki/w/Spiritual_ranger#Zaros
https://oldschool.runescape.wiki/w/Spiritual_warrior#Saradomin
https://oldschool.runescape.wiki/w/Spiritual_mage#Zaros

Drop tables already existed

Copied same as bloodreaver PR for where to add, but slot in around existing spiritual mages (saradomin)

Is this ready?


FALCON-1804. Non-SLA feed throws NullPointerException

Looks good

+1. Will merge once the build validation completes

LGTM.

:+1:  @sandeepSamudrala can you please resolve the conflict in CHANGES.txt?

@ajayyadava : I rebased the pull request. This doesn't seem to be working of resolving conflicts each time.

Can this be committed?


Ability to turn off the the string that the user typed in on the autosuggest matching list

Description
Current behavior:
When user searches for a term in the autosuggest component, the suggested values contain the string that the user typed in as the first value of the searched term as the first one in the list. (screenshot attached)

Is there a way for our developers to turn this feature off so that suggested list would only include the values that match the input string?
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate requests

I guess it will be easy to introduce, there is hideEnteredTextOption, I'm just not sure why it isn't configurable.

Hello!
This behavior is a deliberate design decision. Users need to have an explicit way to decline suggestions and proceed with their current input.
In your case (search input, I guess) the  "Use: how" text should submit the search same way as clicking on any other suggestion would do.
Feel free to provide more comments and details about your use-case, but I will close the issue as currently not planned


BUG - Tooltip label without displayColors and label in Bar Charts

Expected Behavior
Setting displayColors = false without setting a label should show only the value in the tooltip label
Current Behavior
The value in the tooltip is preceded by a colon.
Steps to Reproduce

Create a bar chart
Set displayColors to false
Do not set a label for the dataset
Hover on a bar
The number is preceded by a semicolon
See an example here http://codepen.io/anon/pen/oYVQgO

Environment

Chart.js version: 2.4.0
Browser name and version: any

Cause of this is: https://github.com/chartjs/Chart.js/blob/master/src/core/core.tooltip.js#L71
Work-around is to set a different callback function in the config that doesn't insert the semicolon.

Exactly what I did to fix this issue, but I think it should be a default behaviour.

Great. I'm happy to look at a PR for this if you have time :)

I'm still seeing this exact issue on the latest version.


adding support for Amazon Linux in run.pp

$operatingsystemrelease has "20XX.XX" format on Amazon Linux boxes, which is obviously greater then 7.0, but Amazon Linux is still on SysVinit.

@ikalinin thanks, appreciated.


cmd/compile: devirtualization changes program behavior

Moved from https://github.com/golang/go/issues/38634#issuecomment-1084040645.
What version of Go are you using (go version)?

$ go version
go version go1.18 linux/amd64

It looks this problem started from Go 1.16.
Does this issue reproduce with the latest release?
Yes
What did you do?

package main

type I interface{ M() }

type T struct{}

func (T) M() {}

var pt *T

func f() (r int) {
	defer func(){recover()}()

	var i I = pt
	defer i.M()
	r = 1
	return
}

func g()(r int) {
	defer func(){recover()}()

	var i I = pt
	f := i.M
	defer f()
	r = 1
	return
}

func main() {
	println(f()) // 0
	println(g()) // 1
}

[edit]: another example:
package main

type I interface{ M() }

type T struct{
	x int
}

func (t T) M() {
	println(t.x)
}

func f() {
	var t = &T{1}
	var i I = t
	defer i.M()
	t.x = 2
	return
}

func g() {
	var t = &T{1}
	var i I = t
	f := i.M
	defer f()
	t.x = 2
	return
}

func main() {
	f() // 1
	g() // 2
}

What did you expect to see?
Same return results.
What did you see instead?
Different results.
Changing the i variable to global makes this inconsistency gone.

In f, i.M can be devirtualized to *T, while in g, it does not. So defer i.M() becomes defer pt.M(), and pt is evaluated as nil pointer dereference.
Changing i to a global variable make it can not be devirtualized in both cases.

It obeys the Go Specification of Defer statements and Selectors.
For defer i.M() in f, i is nil, and panic immediately.

If a deferred function value evaluates to nil, execution panics when the function is invoked, not when the "defer" statement is executed.

g uses the selector and gets the (*T).M method, then defer calls the method and results in run-time panic.

For a value x of type I where I is an interface type, x.f denotes the actual method with name f of the dynamic value of x. If there is no method with name f in the method set of I, the selector expression is illegal.

If x is of interface type and has the value nil, calling or evaluating the method x.f causes a run-time panic.

If a deferred function value evaluates to nil, execution panics when the function is invoked, not when the "defer" statement is executed.

So the r = 1 statement should be executed by the spec, right?

If a deferred function value evaluates to nil, execution panics when the function is invoked, not when the "defer" statement is executed.

So the r = 1 statement in f should be executed by the spec, right?

You're right... I went back and read the specs carefully. I think it should be like this:
when defer is executed, it will first calculate the function address and parameters (if any), and then invoke the function before returns. The problem is that nil is encountered when getting the function address.
Defer:

Each time a "defer" statement executes, the function value and parameters to the call are evaluated as usual and saved anew but the actual function is not invoked. Instead, deferred functions are invoked immediately before the surrounding function returns, in the reverse order they were deferred. That is, if the surrounding function returns through an explicit return statement, deferred functions are executed after any result parameters are set by that return statement but before the function returns to its caller. If a deferred function value evaluates to nil, execution panics when the function is invoked, not when the "defer" statement is executed.

Calls:

A method call x.m() is valid if the method set of (the type of) x contains m and the argument list can be assigned to the parameter list of m. If x is addressable and &x's method set contains m, x.m() is shorthand for (&x).m()

For the first example shown above, the outputs are 2 2 with gccgo.
Personally, I think gccgo is right here, for it is self-consistent.
But the outputs of the following program is 2 1 for gccgo and 1 1 for gc.
I don't know who is right here.
@ianlancetaylor @griesemer
package main

type I interface{ M() }

type T struct{
	x int
}

func (t T) M() {
	println(t.x)
}

func f() {
	var t = &T{1}
	var i I = t
	defer i.M()
	t.x = 2
	return
}

func g() {
	var t = &T{1}
	var i I = t
	f := i.M
	defer f()
	i = &T{2}
	return
}

func main() {
	f()
	g()
}

@mdempsky maybe we should not devirtualize if there's deref op? Or if there's a deref op, rewrite it to:
dt := i.(*T)
if dt == nil {
    defer i.M()
else {
    defer (*dt).M()
}

I'm under the impression this only affects defer and go statements. Is that the case?
If so, I'm fine just not devirtualizing defer/go calls at all.


Create PR template for this repository

Similar to Issue template, introduce a PR template. To begin with, one type might suffice.

Fixed in #1109


目标检测

相关知识问答

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Merge return pass Issue:857

Adds a merge return pass #857 .
Unifies all return in a function into a single return location using a new basic block and (if necessary) new phi. Does not perform any other optimizations.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Suggest you take a look at Mesa's algorithm.  It does the structured breakout.
See @cwabbot0's comment:
https://github.com/KhronosGroup/SPIRV-Tools/issues/857#issuecomment-334952559

Rebased, squashed, and pushed into master as a92d69b43


test(avm): use some matchers gtest functionalities to improve unit tests

Resolves #4495

Benchmark results
Metrics with a significant change:

note_trial_decrypting_time_in_ms (32): 119 (+182%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 295cd556 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,700
180,580
720,100

l1_rollup_calldata_gas
222,940
871,156
3,465,808

l1_rollup_execution_gas
316,131
989,604
3,686,186

l2_block_processing_time_in_ms
1,153 (-1%)
4,376 (-6%)
17,287

note_successful_decrypting_time_in_ms
310 (-4%)
989 (-5%)
3,693 (-1%)

note_trial_decrypting_time_in_ms
19.8 (-41%)
:warning: 119 (+182%)
148 (+3%)

l2_block_building_time_in_ms
17,793 (-1%)
70,219 (-1%)
282,371

l2_block_rollup_simulation_time_in_ms
13,093 (-1%)
51,594 (-1%)
207,918

l2_block_public_tx_process_time_in_ms
4,673 (-1%)
18,490 (-1%)
74,189

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
13,787
26,550 (+1%)

note_history_successful_decrypting_time_in_ms
2,400 (+2%)
4,804 (+4%)

note_history_trial_decrypting_time_in_ms
83.7 (-18%)
200 (-13%)

node_database_size_in_bytes
17,297,488
33,079,376

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
306 (-1%)
44,528
25,825

private-kernel-ordering
194
44,145
16,225

base-rollup
1,403
128,654
933

root-rollup
82.6 (-3%)
4,192
729

private-kernel-inner
436
71,331
25,825

public-kernel-private-input
241
32,287
25,825

public-kernel-non-first-iteration
236
32,329
25,825

merge-rollup
7.83 (-2%)
2,712
933

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
2 leaves
8 leaves
16 leaves
32 leaves
128 leaves
64 leaves
512 leaves
1024 leaves
2048 leaves
8192 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.2
10.6
12.8 (+2%)
16.9
22.9
63.6 (-1%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.9
17.5
23.0
31.6
47.0
143
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.594
0.593
0.546 (+2%)
0.524
0.478
0.440
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
N/A
N/A
N/A
74.4
47.5
240
462
902
3,548

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
N/A
N/A
N/A
159
96.0
543
1,055
2,079
8,223

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
0.460
0.488
0.437
0.432
0.429
0.427

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
N/A
N/A
N/A
104 (+1%)
56.9
348 (-1%)
689 (+1%)
1,366
5,407

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
N/A
N/A
N/A
197
104
691
1,363
2,707
10,771

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
0.493
0.497
0.475
0.473
0.474
0.471

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
56.6
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
94.1
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
0.578
N/A
N/A
N/A
N/A
N/A
N/A
N/A

Miscellaneous
Transaction sizes based on how many contracts are deployed in the tx.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
16,859
40,167

Transaction processing duration by data writes.

Metric
0 new commitments
1 new commitments

tx_pxe_processing_time_ms
549 (-1%)
1,304

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
0.507 (-3%)
575


Fixed metadata forwarding and added associated tests

In order for checkpointing to work in a pipeline, the metadata must be in a list and not a nested list. If the metadata is forwarded as a nested list, then managing it can get very complicated. Previously there were tests to verify that the reference counters were being incremented and decremented properly. The tests in this PR ensure that the metadata is forwarded properly for all functions that emit collections.

It looks like there's an intermittent failure that's unrelated to the changes?

I'm good with this, will merge over the weekend unless there are other comments.


The development container is broken due to incorrect configuration of Poetry

Describe the bug
The development container is broken due to incorrect configuration of Poetry
To Reproduce
try to load this repo into vscode and start devcontainer
Additional context
No response
Version
current
Platform
Win with WSL
Code of Conduct

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

No Duplicate

[X] I have checked existing issues to avoid duplicates.

I have already attempted to fix this issue, and the setup is working better now. #284
However, there is still a major issue:
When running pytest via a launch command, breakpoints are not being hit, making debugging tests nearly impossible.
If someone is familiar with configuring pytest to work with breakpoints in a development container with poetry setup, I would greatly appreciate assistance with this in the PR.


getModule of AppRegistry is not an object

Description
I upgraded the version from 0.67.2 to 0.69.0
I got the issues when runtime:
E0623 17:04:48.910380<PHONE_NUMBER> bindingUtils.cpp:27] getModule of AppRegistry is not an object
E0623 17:04:48.910419<PHONE_NUMBER> bindingUtils.cpp:29] react_native_assert failure: moduleAsValue.isObject()
Assertion failed: (moduleAsValue.isObject()), function getModule, file bindingUtils.cpp, line 29.
dyld4 config: DYLD_LIBRARY_PATH=/usr/lib/system/introspection DYLD_INSERT_LIBRARIES=/Developer/usr/lib/libBacktraceRecording.dylib:/Developer/usr/lib/libMainThreadChecker.dylib:/Developer/Library/PrivateFrameworks/DTDDISupport.framework/libViewDebuggerSupport.dylib:/Developer/Library/PrivateFrameworks/GPUTools.framework/libglInterpose.dylib
Version
0.69.0
Output of npx react-native info
System:
OS: macOS 12.4
CPU: (8) x64 Apple M1 Pro
Memory: 38.02 MB / 16.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 14.17.0 - /usr/local/bin/node
Yarn: 1.22.19 - /usr/local/bin/yarn
npm: 8.12.1 - /usr/local/bin/npm
Watchman: Not Found
Managers:
CocoaPods: 1.11.2 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.4, iOS 15.4, macOS 12.3, tvOS 15.4, watchOS 8.5
Android SDK: Not Found
IDEs:
Android Studio: 2021.2 AI-212.5712.43.2112.8609683
Xcode: 13.3/13E113 - /usr/bin/xcodebuild
Languages:
Java: 15.0.5 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.0.0 => 18.0.0
react-native: 0.69.0 => 0.69.0
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
The build is successful but the runtime has errors
Snack, code example, screenshot, or link to a repository

You picked new architecture but I see no reference of it in your issue description. Was that correct?

I was getting this error when new architecture with hermes is enabled and some value is undefined and I am trying to access its props.


Can't successfully validate CLI::prompt when field param has periods

Describe the bug
I am making a CLI command that prompts the user whenever there is a file to be overwritten. In my prompt I just say File already exists. Overwrite? and supply the array of options ['y', 'n']. Now when I am supplying an input, pressing any of Enter, y, n fails validation and say that input is not within the list.
CodeIgniter 4 version
dev-develop
Affected module(s)
CLI, possibly Validation also
Expected behavior, and steps to reproduce if appropriate

Create a controller:

use CodeIgniter\CLI\CLI;

class Home extends BaseController
{
    public function index()
    {
        if (!is_cli()) {
            exit('Needs to run on CLI');
        }

        if (CLI::prompt('We are at home. Leave?', ['y', 'n']) == 'y')
        {
            CLI::write('We\'re leaving.');
        }
    }
}

Run in terminal:

php public/index.php home/index

Fill in the prompt.
I suspect the culprit is in how Validation runs the process, perhaps in lines 177 or 178. But I'm not sure yet.
Temporary fix for this is obviously removing the period but I want to know is this the intended behavior also for prompts?

Cheers!
Context

OS: Win 10
Web server CLI
PHP version 7.4.7

It's a bug. We use "prompt text" as a rule field and dot is intended to be there only in purpose to handle arrays. We should replace . with something else before using validation class and then replace the text from error message back to use . again.
See: https://codeigniter.com/user_guide/libraries/validation.html#validating-keys-that-are-arrays

Maybe we can use a $tempfieldname to substitute for the prompt text to use in the validation process?

Oh, silly me... We can just use $label option and set $field as some constant value like "temp".

Hm... wait we can't do that, sorry.

What I meant was in line 285, instead of $field as validate's first argument, maybe we can use something else. Like "temp" as what you've said. Or "prompt".

Sorry, obviously it's not my day. The label option should work okay.


HTML/CSS: Body and Footer

#HTML/CSS: Header and Navigation
##User Story
AS A marketing agency
I WANT a codebase that follows accessibility standards
SO THAT our own site is optimized for search engines
##Acceptance Criteria
GIVEN a webpage meets accessibility standards
WHEN I view the source code
THEN I find semantic HTML elements
WHEN I view the structure of the HTML elements
THEN I find that the elements follow a logical structure independent of styling and positioning
WHEN I view the heading attributes
THEN they fall in sequential order

branch - feature/article-content_00 is the fix, yet there will be another branch to include comments.


Frost spec is misleading

Hi hi. First of all: thank you for this great tool!
Onto the issue: I think what the sim calls "frost" spec is misleading. The tool is just using the Arcane 40/0/21 spec and spamming frostbolt on it. We should rename/reword it in the way to make it clear that the PvE Frost 18/0/43 spec is not being used here. And instead, mentions that it is the Arcane spec with Frostbolt spamming.
On a side note, it would be nice to add full support for the PvE Frost 18/0/43 spec. But I guess adding a water elemental simulation would involve non trivial changes to the code.

Water elemental is not in the sim because

It takes a long time to split the code to work with pets
We don't have the actual numbers for all the stats of the water elemental.

I won't change the name of the spec since that option only changes which spells to use (frostbolt).
I might add a disclaimer that water ele isn't supported.

I won't change the name of the spec since that option only changes which spells to use (frostbolt)

Yeah but that's source of the confusion imho. Why name the sim's "frost" spec this way when is actually not using the PvE Frost 18/0/43 spec, it's using the Arcane 40/0/21 spec, like the "arcane" spec of the sim. Both the "arcane" and the "frost" spec are using 40/0/21 but with a different spell rotation.

@Cheesehyvel - I'm currently playing around with code to add a water elemental. My thinking was that we could just sample the water elem waterbolt stats and fit an appropriate distribution offline and sample from it during online. I've read in various places that the elem's stats may scale with the mage's stats - not sure if that true or if you know better but I was going to test the hypothesis in any event.

It does scale. We have some numbers but not all of them.
I've asked in mage discord a couple of times but no one seems to have the complete information.
Which means any implementation will probably be inaccurate and misleading.

What are some of those numbers we know? I'm not convinced the situation is as dire as you might imply. You're already using estimated resistances from TBC Beta from WCL (per code comments). Base stat scaling (int) should be extremely easy to estimate. I haven't looked at the data yet but my hunch is that if I pulled ~200 Kara logs we'd be able to make some decent enough assumptions about scaling such that the water elem's sim behavior would be statistically indistinguishable from the observations - we/I can always iterate from there. I'll poke around on the mage discord and look into some logs today.

The only confirmed data point is water bolt coefficient which seems to be 0.25 of the mages sp, which can also be calculated as 30% sp * 0.833 coefficient.
We're missing base and scaling data for mana/crit/hit.
We also need documented behavior of how the buffs in the sim affect the elemental, like BL/drums/jc neck/totems etc.
The elemental also seems to have a delay between casts which needs to be factored in.
Feel free to gather this data. I just don't care enough about frost spec to do it myself.


sonarr: <IP_ADDRESS>3 -> <IP_ADDRESS>1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for sonarr is: Smart PVR for newsgroup and bittorrent users
meta.homepage for sonarr is: https://sonarr.tv/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS>1 with grep in /nix/store/6xmsxsq3h1c86c82hr4gkhbdzs06imgj-sonarr-<IP_ADDRESS>1
found <IP_ADDRESS>1 in filename of file in /nix/store/6xmsxsq3h1c86c82hr4gkhbdzs06imgj-sonarr-<IP_ADDRESS>1

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

sonarr

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/6xmsxsq3h1c86c82hr4gkhbdzs06imgj-sonarr-<IP_ADDRESS>1 \
  --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 sonarr https://github.com/r-ryantm/nixpkgs/archive/19c17e1e20dcf2fa3f01162880c2e1d1adb9b839.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/19c17e1e20dcf2fa3f01162880c2e1d1adb9b839#sonarr

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/6xmsxsq3h1c86c82hr4gkhbdzs06imgj-sonarr-<IP_ADDRESS>1
ls -la /nix/store/6xmsxsq3h1c86c82hr4gkhbdzs06imgj-sonarr-<IP_ADDRESS>1/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 --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

sonarr

Maintainer pings
cc @fadenb @purcell for testing.

Result of nixpkgs-review pr 306420 run on aarch64-darwin 1

1 package built:

sonarr


add a new openwrt device tracker that works out of the box and consolidate the 2 other methods to one file

The current device tracker names are based loosely on how they integrate with openwrt.
It would make more sense to have just one platform called openwrt and be able to specify which api/implementation it should use. The new wireless api allows users to easily add your openwrt router to Home Assistant. This should work with all Chaos Calmer 17.x releases and might work fine with older openwrt (luci) releases as well.
Description:
Both ubus and luci (jsonrpc) platforms required you to install extra packages to your openwrt router. I have added a third method which does not need anything other than luci installed.
It logs in to luci using the regular login method and parses json entries from the wireless page. It uses requests.session to keep make use of keepalive/connection pooling and to store the session cookie.
Deleting the luci.py and ubus.py could also be done in a later phase but migrating to the new implentation is very easy and could be easily explained in release notes.
Added support for https on all three platforms with a fix to remove the Insecure warnings associated with disabling SSL verify.
Related issue (if applicable): fixes #
None that I know of
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):
device_tracker:
  - platform: openwrt
    api: wireless
    host: <IP_ADDRESS>
    radio: radio0.network1
    ssl: true
    verify_ssl: false
    dnslookups: true
    username: root
    password: !secret openwrt_password

Checklist:

[X] The code change is tested and works locally.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[X] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Hello @joeripronk,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

1b652fdca635d5ac3b677aa4eddb751ec9f2db13 - This commit has something that looks like an email address (joeri@blacknoise.nl). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

Hi @joeripronk,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Hi @joeripronk,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Hi @joeripronk,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Hi @joeripronk,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Hi @joeripronk,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Hi @joeripronk,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

I am currency’s experiencing issues wirh both luci and ubus platforms so I have high hopes for this one. Is this going to get merged soon?

Please run tox, or as a minimum script/lazytox.py for a quick check before pushing commits. It should help not getting bitten by the hound.

This is too much device specific code, as already mentioned. We can't accept this until that is broken out into standalone python package and published on pypi.

I've deleted all the stale hound comments. Please run tox or at least script/lazytox.py as @kellerza mentioned, before pushing the next time.

I understand, I will close this pull request.

@joeripronk hope you continue the work on this

any news on this?


Dynamic Data Loaders

🚀 A dataloader with changable sampling behavior
A DataLoader that takes in a confusion matrix, or class specific recall at the end of the epoch. It then oversamples classes that are the least known to the network in the next epoch and hereby helps the network train more efficiently.
Motivation
I often work with highly imbalanced datasets, which I have previously been dealing with artificially oversampling classes that are less prevalent (label-balancing). Also I have been experimenting with extending this stratification from the plain labels to other confounders aswell (confounder-balancing). (* example provided at the end)
It would be great to make this process dynamic! Since the data stays the same and duplicating images seems a waste of memory and only the distribution changes over which samples are drawn from the data.
The idea is the following:
At the end of the epoch, I check how well the model is doing on the validation set and see if there are any subgroups (either on label or confounder level) and would pass the information on to my train dataloader (or its pytorch.utils.data.Sampler). Sticking to the example in the footnotes, I'd get the following validation errors:
Dog pics taken on the inside 0.4 MSE
Dog pics taken on the outside 0.3 MSE
Cat pics taken on the inside 0.5 MSE
Cat pics taken on the outside 0.2 MSE
Then I would call something like:
self.train_dataloader.sampler.distribution = [0.4, 0.3, 0.5, 0.2]
and self.train_dataloader.sampler would yield indices antiproportional to that distribution from its data_source, e.g. via torch.nn.functional.softmax(-torch.Tensor([0.4, 0.3, 0.5, 0.2])) =
My concern
While pytorch offeres many strategies to tackle this, e.g. subclassing torch.utils.data.DataLoader or torch.utils.data.Sampler, I am not sure, how Pytorch Lightning is handling the dataloaders in the back, where they are stored and what the implications of that would be for the awesome functionalities Pytorch Lightning offeres (especially data-parallel methods etc.)
As this would be my first Pytorch Lightning project, I am yet unfamiliar with the inside functionality, so any ideas from experts with the API are highly appreaciated. I'll keep you posted with my progress! Best wishes and thanks for your input!

Example stratification (label-/confounder-balancing):
If I tried to classify cats vs. dogs from images and I have 20% dogs in my dataset, I would duplicate each dog picture 4 times and had a balanced dataset (label-balancing). If the pictures in my dataset had one of two backgrounds (outside/inside) and I had an imbalance like 25% dogs-inside 25% dogs-outside 40% cats-inside 10% cats-outside, then I had 50/50 labels, but my network is more likely to classify pictures taken inside to be from cats, since the label is more prevalent in that subclass. Knowing that this irregularity exists, I would oversample cat pictures taken outside and a few more dog pics from both classes so I'd have a balance dataset (confounder-balancing). Of course, these imbalances are not exclusive and for me it just boils down to, finding out as much as I can over my data and trying my best to reduce the all imbalances I find, unless the groups are super small, or the clientdoesn't care and thinks classifying images based on the background of the image is something he wants to pay me for (happen's more often than you think).

@jonesy-git This is a good idea I too want to implement something like that. This issue should have been open.

@jonesy-git This is a good idea I too want to implement something like that. This issue should have been open.


New version: DEVCOM.LuaJIT version 2.1.1706185428

Pull request has been created with komac v2.0.2 :rocket:
Microsoft Reviewers: Open in CodeFlow

Hi @Exorcism0666,
This manifest adds Apps and Features entries that aren't present in previous PR versions. Should these entries also be added to the previous versions?
(Automated response - build 684.)


Update dev guide for vanilla qemu

Update https://github.com/kata-containers/documentation/blob/master/Developer-Guide.md to explain that qemu-lite is no longer needed.

Closing this issue as https://github.com/kata-containers/documentation/pull/68/files was merged


Can we change CodeCop rule AA0005?

Can we change CodeCop rule AA0005?
The CodeCop rules can be found at https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/devenv-codeanalyzer-codecop-rules and CodeCop rule AA0005 is as follows:

AA0005
Only use BEGIN..END to enclose   compound statements.
Only use BEGIN..END to enclose   compound statements.
Only use BEGIN..END to enclose   compound statements.
Readability
Warning
true

Is there a way to turn off a specific rule in CodeCop? Or better still, can I convince you to change this rule?
I think the above rule is a bad rule and CodeCop should actually be the opposite to this rule.
These should be the correct rules:

Description
Category
Default Severity
IsEnabledbyDefault

Always use BEGIN..END following an IF   statment.
Mantainability
Warning
True

Always use BEGIN..END following an ELSE   statment.
Mantainability
Warning
True

Like most style-related issues, I’m sure opinion will divided on this. However, my goal is to produce code that can be maintained without introducing errors. I do not believe the readability of code with my proposed rules is any worse than the current rule, in fact I think it is easier to see what is going on with my proposed rules. There is a possibility that you do not want to change this rule because so much standard Microsoft code will violate this rule, but I would ask you to question whether that would be the correct motivation behind which rules should be included. The rules are there to try to ensure that the code we produce is robust (maintainable, upgradable, reusable).
It’s not hard to find discussions on this topic, so I took the first hit from a search as an example and changed the syntax of the examples given to be al. The original topic can be found at https://stackoverflow.com/questions/2125066/is-it-bad-practice-to-use-an-if-statement-without-brackets
Let’s look at both options.
// 1 - Code is compliant with current rule
if SomethingIsTrue() then
  DoSomethingImportant()
else
  DoSomethingLessImportant();

// 2 - Code is compliant with proposed new rule changes
if SomethingIsTrue() then begin
  DoSomethingImportant();
end else begin
  DoSomethingLessImportant();
end;

The problem with the first version is that if you go back and add a second statement to the if or else clauses without remembering to add the begin and end, your code will break in unexpected and amusing ways.
Maintainability-wise, it's always smarter to use the second form.
For an interesting real world example of why omitting compound statements following an if or else can cause problems, see this discussion of the “Go to fail” bug that caused serious security vulnerabilities in 2014: https://www.imperialviolet.org/2014/02/22/applebug.html

I would actually propose to completely remove begin's and end's. All this unnecessary typing all day ;)
In my opinion, curly brackets would make AL code much more readable.
if (true)
  DoSomethingImportant();
else
  DoSomethingLessImportant();

if (true)
{
  DoSomethingImportant();
  DoSomethingEvenMoreImportant();
}
else
  DoSomethingLessImportant();

The bracketed question is already very much under discussion in #63. Please join us there instead of new discussion threads ^^. @erikrijn
Currently there is no Microsoft statement.

I agree with David here - there should be a way to disable certain rules/warnings, should we want to.
Our reasoning, which is probably subjective:
while the official Microsoft style for AL might insist on having BEGIN/END only on compound statements, we always code our solutions for maintainability/ease of upgrade. If we need to add more lines of code to and IF branch in the future, having enclosing BEGIN/END from the start make for a cleaner modification code.

Hi @DavidRoys,
With the use of rulesets you can hide that warning.
Ruleset for the Code Analysis Tool

You can also have a look at: Using the Code Analysis Tools with the Ruleset in order to set your ruleset.

Those answers helped me to achieve what I needed thank you. I have another problem now with rule AA0139:

AA0139
Do not assign a text to a target with   smaller size.
Do not assign a text to a target with   smaller size.
Possible overflow assigning '{0}' to   '{1}'.
Design
Warning
true

I guess this is just a case of the rule not being smart enough. The following code triggers this rule:
Value := CopyStr(NewValue,1,MaxStrLen(Value));
I don't see any way of circumventing this rule. I guess I can change it to an info type rule so it doesn't get too annoying.

The issue on rule AA0139 has already been reported in #2291.
I guess turning into an info could be a good workaround for now.


Parallel NetCDF input output

I think that iris has no parallel implementation for reading or writing NetCDF files.
HDF5 supports MPI parallelization and could be a great idea to implement this under iris project.

hello @carlestena
i think that there are options on the way a HDF5 library is built that control this.
The testing uses a HDF5 build which is thread safe, and some parallelisation takes place.  I don't think there are tests which use an MPI parallelisation
I wonder whether this is an option that would be enabled in the HDF build, that libraries such as Iris would just use.
Do you think that there is a part of the Iris API which should be able to control this, if it is available?  It is not clear to me that this would be the case.
the h5py documentation seems to me to suggest that using mpi4py and h5py together.  perhaps a similar pattern could be used to get benefit without requiring any change to the Iris library
http://docs.h5py.org/en/latest/mpi.html


Problem with different Frequency for up and downlinks 868 vs 433

I am using a Heltec Hotspot that has been set to EU868 (The correct frequency for Zimbabwe where it is installed.) It was working fine until about 4 months ago, after which a Console "upgrade" broke it. Since then it appears that the Dragino LGT-92 that I am using to send tracking information to the hotspot is sending on EU868 but is receiving on 433. This is what I can see from the Console. I can't see how I can change this.
To Reproduce
Use a Dragino LGT-92 and a Heltec Helium Hotspot in Zimbabwe, All set up to use EU868 regional settings.
Expected behavior
I expected both up and downliks to show EU868 settings from the console.
Device Info (please complete the following information):

Production Environment
Console Device ID  91b0e325-2c39-4561-8725-c40977936ef4
Device dev eui / app eui   A840411FA182FF58
How often is device sending data  Every 2 mins
Region EU868 Zimbabwe
Hotspots (names) used to transfer data old-bone-swift

NOTE: do not post your device app_key
Additional context
Add any other context about the problem here.
Screenshots
If applicable, add screenshots (from Console) to help explain your problem.

Something weird is going on, you'll have to contact Heltec.
The gateway is obviously listening in the 868 range.
But the Region being sent in the state_channel_packet is EU433.
The region in the packet comes from and is signed by the gateway.

Helium has the incorrect frequency set in its channel plan for Zimbabwe. How can this be changed.


More network resources

It would be good to have:

[ ] ALB resources
[ ] Internet Gateway
[ ] NAT Gateway
[ ] VPC/Service Endpoint
[ ] Route tables
[ ] Subnets
[ ] Peered connections
[ ] Security Groups
[ ] NACLs
[ ] Virtual Private Gateways
[ ] VPN Connections
[ ] Client VPN Endpoints

I'm thinking about how to refactor code of print messages... I don't like the way of things are displayed on screen and with lot of new services this will be a big mess..
I'll insert these new services, but will keep on target how to re-design this...

It would be good to have:

[x]  ALB resources

Added ALB and CLASSIC LB check. There are 2 different classes to check cause boto3 implementation works this way..

It would be good to have:

[x]  VPC/Service Endpoint

VPC Service Endpoint is attached to a Network Load Balancer and there is no a simple way to get the information about NLB.. The response of boto3 method describe_vpc_endpoint_services() doesn't have subnet or vpc-id, just have AZ and I can't use this, it's not safe.
But it's not a problem.. When scripts checks NLB, will detect if is using VPC informed or not..


add CFPB from watson repository

This is version that reflects watson version of CFPB. I got the following warning, hope that this is ok:
/u/ilyashn/.conda/envs/bzsm2.2/lib/python3.9/site-packages/scipy/stats/_resampling.py:100: DegenerateDataWarning: The BCa confidence interval cannot be calculated. This problem is known to occur when the distribution is degenerate or the statistic is np.min.
warnings.warn(DegenerateDataWarning(msg))
/u/ilyashn/.conda/envs/bzsm2.2/lib/python3.9/site-packages/scipy/stats/_resampling.py:147: RuntimeWarning: invalid value encountered in scalar divide
a_hat = 1/6 * sum(nums) / sum(dens)(3/2)
/u/ilyashn/.conda/envs/bzsm2.2/lib/python3.9/site-packages/scipy/stats/_resampling.py:100: DegenerateDataWarning: The BCa confidence interval cannot be calculated. This problem is known to occur when the distribution is degenerate or the statistic is np.min.
warnings.warn(DegenerateDataWarning(msg))
/u/ilyashn/.conda/envs/bzsm2.2/lib/python3.9/site-packages/scipy/stats/_resampling.py:147: RuntimeWarning: invalid value encountered in scalar divide
a_hat = 1/6 * sum(nums) / sum(dens)(3/2)
/u/ilyashn/.conda/envs/bzsm2.2/lib/python3.9/site-packages/scipy/stats/_resampling.py:100: DegenerateDataWarning: The BCa confidence interval cannot be calculated. This problem is known to occur when the distribution is degenerate or the statistic is np.min.
warnings.warn(DegenerateDataWarning(msg))

We may go directly to 9 classes using link: https://www.consumerfinance.gov/data-research/consumer-complaints/search/api/v1/?date_received_max=2023-01-04&date_received_min=2022-01-04&field=all&format=csv&has_narrative=true&lens=product&no_aggs=true&size=340390&sub_lens=sub_product&trend_depth=5&trend_interval=month


Pre-trained model download link

Hi author,
Thank you for sharing your great work. May I know where do I retreive the pre-trained models in model zoo? I am seeing the config files but not the weights.
Thanks!

We will provide these pre-trained models next week.


LGTM.com - false positive: Too few arguments to formatting function

Description of the false positive
It appears there is an issue with figuring out the position of the format string with Haiku's kernel-only dprintf function.
URL to the alert on the project page on LGTM.com
https://lgtm.com/projects/g/haiku/haiku/snapshot/d26be28c9c1882a2fe59ca80fbcf9410053b590f/files/src/add-ons/kernel/bus_managers/acpi/ACPICAHaiku.cpp?sort=name&dir=ASC&mode=heatmap#xf27d4d8483491c15:1
https://lgtm.com/projects/g/haiku/haiku/snapshot/d26be28c9c1882a2fe59ca80fbcf9410053b590f/files/headers/private/file_systems/QueryParser.h?sort=name&dir=ASC&mode=heatmap#x37242992c01d5a0a:1
I've been experimenting with the query, and with the following changes, I'm seeing two results per dprintf call, one with the format string at index 0 (correct), one with the format string at index 1 (incorrect, seems a glibc GNU extension). Is it possible that it's picking up two conflicting definitions of dprintf? Shouldn't the shadowed definition be excluded?

from FormatLiteral fl, FormattingFunctionCall ffc, int expected, int given, int literal, int total
where ffc = fl.getUse()
  and expected = fl.getNumArgNeeded()
  and given = ffc.getNumFormatArgument()
  and literal = ffc.getFormatParameterIndex()
  and total = ffc.getNumberOfArguments()
  and expected > given
  and total >= 3
  and fl.specsAreKnown()
select ffc,
	"Format expects "+expected.toString()+" arguments but given "+given.toString()+" of "+total.toString()+"\n" +
	"target = " + ffc.getTarget().toString()+" format string at " + literal.toString() +"\n" +
	"first argument = " + ffc.getArgument(1).toString()

Thanks for digging into this. It sounds quite likely that our query is picking up GNU dprintf too and is having trouble distinguishing between the two.
We're already tracking this bug under #1971.

Ah, I didn't realise he'd filed a ticket, his thinking behind the __func__ macro was a good guess, but invalid :)

This issue should be fixed by https://github.com/Semmle/ql/pull/1987 (please allow approximately 2 weeks for the fix to reach LGTM).


Bot is not starting on first "Start" command only after pressing "Restart"

I am running grammy 1.9, express.js and Node 16.13. I recently started experiencing an issue that my bot does not start when pressing the initial "Start" button inside telegram. After pressing the Start button the "Restart" button appears. If I press this button again the bot actually listens to the start command. So I basically always need to type in /start twice to get my first response.

I have completely isolated the problem by starting a simple express server from scratch and using the default configuration described in the grammy documentation. The problem persists and lets me wonder if this might be related to a recent telegram update (https://core.telegram.org/bots/api#june-20-2022).
I am pretty much clueless where this could come from. Is anyone else experiencing and issue like this?
Thanks for your help!

Can you share the code?
Also, bots can't actually make clients display restart buttons. This is something that happens internally in Telegram, and that bots have no control over.

I am just running this code. For me it looks like its a telegram issue. I created a new Bot and it worked perfectly. However using another bot it does not work. Its also seems to be user specific. I talked to some friends and it seemed to work for them while it was not working for me. Any idea, what this could be related to? As you said the "Restart" is something telegram does.
const bot = new Bot(xxx)

bot.command("start", (ctx) => ctx.reply("I'm running on Heroku using long polling!"));

const runner = run(bot);

// Stopping the bot when the Node process
// is about to be terminated
const stopRunner = () => runner.isRunning() && runner.stop();
process.once("SIGINT", stopRunner);
process.once("SIGTERM", stopRunner);

This issue is unrelated to grammY.
There is likely a bug in Telegram. Can you narrow down which clients were used to reproduce the problem, and which versions of it?
Btw runner.isRunning() only checks if your bot is ruining but doesn't stop it. Did you mean runner.stop()?

Hey,
so I actually just use the bot.start() instead of the runner. I am using the <IP_ADDRESS>774 Stable Mac Version of Telegram Desktop Client.  The following is my new testing code. I just run it via node bot.js. Same problem bot only starts to react after pressing Restart and not initial Start.

//bot.js
const { Bot } = require("grammy")

const bot = new Bot('xxx')

bot.command("start",(ctx) => ctx.reply("This is my test message"))
bot.start()

I just created a complete new Bot via BotFather and the newly created bot works just fine with the code.

Hey,
so I actually just use the bot.start() instead of the runner. I am using the <IP_ADDRESS>774 Stable Mac Version of Telegram Desktop Client. The following is my new testing code. I just run it via node bot.js. Same problem bot only starts to react after pressing Restart and not initial Start.

//bot.js
const { Bot } = require("grammy")

const bot = new Bot('xxx')

bot.command("start",(ctx) => ctx.reply("This is my test message"))
bot.start()

I just created a new Bot via BotFather and the newly created bot works just fine. This only seems to be related to older bots.

If you know how other people can reproduce this reliably, you can open a bug a report at https://bugs.telegram.org. I will close this issue as it has nothing to do with grammY.


Cannot move through long lists item by item with VoiceOver

Description of issue
On iOS with VoiceOver, stepping through items truncates the list. It is necessary to use the scroll gesture to force the list to pick up further items and make them available for stepping through.
This is a known React Native issue: https://github.com/facebook/react-native/issues/13743
Works fine on Android.
Expected behaviour
User can step through the entire list using the back and forward gestures, or the skip gesture (e.g. in headline mode)
Steps to reproduce

Open the events list with VoiceOver on
step through the items until the 'next' gesture does not get you any further
swipe up with three fingers (scrolling to the "next page"), new items are revealed and can be stepped through

OS / Browser / Device model
iOS 11.1, iPhone 7

@charypar is this something we can fix? Or do we need to wait for React Native to fix this?

There might be some workaround, but I'm not sure. AirBnB have a fix for it in their RN fork and it looks like the original issue was closed...


This is a must-have

Is your feature request related to a problem? Please describe.
No.  This project looks like a must-have for every RN app. But then why is this not so popular.
Describe the solution you'd like
Are there any articles on WHY this must be used, HOW much improvement in performance, WHAT are the gotchas if present?
Describe alternatives you've considered
I must implement and test this on my app.
Additional context
None. This is not really a feature request, but more of a request for guidance and any experiences of this this library.

Hi @nidheeshdas
Thank you for issue.
I think I wrote quite detailed documentation for this library.
This documentation contains step by step guide how to set up this library in any project. Also it has few tutorials such as "how to find the biggest dependency"/"how to measure startup time".
If you think I missed something important there - please, write below what you would like to see in the documentation.

How do experimentalImportSupport and inlineRequires relate to this project?

@kuasha420 this library is built on top of inlineRequires and ram bundles. Via register HOC you can easily move all screens/components to RAM bundle and load them on demand. (Couldn't find any info about experimentalportSupport. I would be highly appreciate you if you can give me a link where I can read about it).
Did I answer on your question?

@kirillzyusko Thanks for the quick answer! experimentalportSupport has no documentation actually. But was added at the same time as inlinerequire. Not sure what it does.
By the way, is it a good idea to try this on a react-native-web app or try spitting in webpack way?
Cheers!

@kuasha420 Unfortunately I don't know a lot of details about react-native-web. If you are saying, that webpack is used in react-native-web under hood, then I highly doubt that you can use this library out-of-box.
I know you can use react-loadable for splitting bundle on web projects. And I think you can use this library with react-native-web as well.
In fact I was thinking of adding web support into this library. It was on the roadmap and I thought to make a 2.0.0 release with web support. The API of both libraries is quite similar to each other so it shouldn't be a problem.
But if you go ahead and know the way how to bring a web support into this library - PRs are highly appreciated.

Thanks for the detailed answer. I'll investigate the matter and try to come up with something in the coming days. Cheers

@kuasha420 if you're interested in bundle splitting on web, it would be a great idea to [consider Next.js](
Next.js by Vercel - The React Frameworknextjs.org) which splits code out of the box.
You can easily make a mobile and web app with expo, combined with next.js. I have RAM bundles enabled for the mobile apps, and Next does its bundle splitting on web.

@heysailor That's a good idea for a new project. I'm trying to do it in an existing codebase. I've done some experimenting with loadable-component and it's working pretty well. Just need to adopt the idea with this project for a potential PR. Problem is require vs dynamic import

@kuasha420 if you want to add dynamic import support to react-native app you can use babel plugin. But I'm not sure that files imported via dynamic import construction will be moved into RAM bundle. It needs to be checked.
Also, feel free to create a new issue. Because the stuff that we discuss here is not related to the title of the issue😅

@kuasha420 and @heysailor I created new issue: https://github.com/kirillzyusko/react-native-bundle-splitter/issues/10
And I will close this one. Feel free to share new info there.


feat: Popular cities name

I have added some popular cities name for accessing weather directly.
I have also created pull request related to this.
Can you add this under 'hacktoberfest'  and 'hacktoberfest-accepted' label?

@kanishk-001 Is there anything wrong with this issue, it's labelled invalid?


Remove riscv64 from build platforms

Description of the change

I'm hoping this will unblock the release of jammy-1.0.2.
The docker image ubuntu:22.04 is not available for riscv64, which limits options for providing support for it here.
------
 > [linux/riscv64 internal] load metadata for docker.io/library/ubuntu:22.04:
------
Dockerfile:2
--------------------
   1 |     ARG BASE_IMAGE=ubuntu:22.04
   2 | >>> FROM $BASE_IMAGE
   3 |
   4 |     ARG QEMU_ARCH
--------------------
ERROR: failed to solve: ubuntu:22.04: no match for platform in manifest: not found
Error: buildx failed with: ERROR: failed to solve: ubuntu:22.04: no match for platform in manifest: not found

Benefits

Unblocks release?
Possible drawbacks

cc @olof-nord who added this in 2021 via https://github.com/phusion/baseimage-docker/pull/604, back then the release was focal.
Applicable issues

https://github.com/phusion/baseimage-docker/issues/632
Additional information

You are the ones in charge of the project, no questions asked.
I do see this as a step backwards though, as risc-v is supported by Ubuntu 22.04.3 as well as 23.10.
https://ubuntu.com/download/risc-v
For risc-v docker Ubuntu images, I use https://hub.docker.com/r/riscv64/ubuntu at the moment.

I've opened https://github.com/phusion/baseimage-docker/pull/634 with an attempt to bring back support for riscv64. Cross-posting:

It is worth noting that the riscv64/ubuntu:22.04 image was last built in December 2022, so more than a year ago. It does not look supported anymore.


Walkthrough text updates and quickstart cleanup fix

Small updates to Multicluster Gateways Walkthroughs
Quickstart cleanup fix - the cleanup function no longer exists

/lgtm


Automate "close" and "release" Nexus commands

See: https://github.com/gradle-nexus/publish-plugin

README.md updated with directions as well.   Old publishMavenPublicationToOSSRHRepository task still available for now (this is the task that does NOT automatically close and release, forcing you to login to webpage and do it manually).


Fix: Team Builder State

The team builder has several major issues:

Saving teams does not set the correct team
Selecting a team may not open it correctly for editing
Up to a certain amount of teams, creating a new team will override a preexisting one
Editing a team may either clear or duplicate the team

Requirements

[ ] Rewrite the state logic in order to clear all these issues
[ ] Change how "choosing works" so that you "choose" a team in the homepage (like in Pokemon Showdown)

Potentially fixed if #78 is verified


Can I use template feature in file response?

I want to use template feature in my test framework.  just like "response" : { "file": { "template": "foo_${req.queries['foo']}.json" } }, that means the moco will return a file named foo_1.json. but it does not work.

This would be a nice feature!

I was just using moco thinking it supports this :( not only the file response should allow referencing variables and dynamicly evaluate the path to load before it tries to load the file, all features should assume you might be referencing something and evaluate/resolve the value first. For example, playback, if you could record an endpoint by using more clever configuration and change the file name to write? I use moco in my tests as cause I really like its JUnit integration, I am advertising moco all the time, now I wanted the stand alone version to mock a rest api and without this feature a lot of it becomes really not doable.


Entry point is invalid and release version is not working (Windows 11)

Issue 1:
The entry point is "dist-electron/main/index.js" but should be "dist-electron/electron/main/index.js" i guess?

Issue 2:
Once I change the entry point I can start the application in development mode but the release version which I created with yarn build won't start. It shows the following error message:
(node:24752) electron: Failed to load URL: file:///D:/electron-vite-react/release/2.1.0/win-unpacked/resources/app.asar/dist-electron/dist/index.html with error: ERR_FILE_NOT_FOUND
(Use `YourAppName --trace-warnings ...` to show where the warning was created)

Issue 1:
https://github.com/electron-vite/vite-electron-plugin/blob/5dd82a3ee5db6bd484f8b6ae66794b309b2e34d4/src/config.ts#L214-L227

The behavior like tsc.
Issue 2:
You must adjust according to the new build directory structure.
https://github.com/electron-vite/electron-vite-react/blob/d6f989c4f5099dbd84dc54fe3d7dc9a92d6691bc/electron/main/index.ts#L1-L13

@caoxiemeihao Thank you for your answer. If I understand it correctly, I have to adjust the entry point (main) to "dist-electron/electron/main/index.js" if I want to use preload.ts, right? At least this seems to work.
Unfortunately I cannot solve Issue 2. Can you please assist me with this one? I've printed out the directories in dev and release version to see what's going on but I don't know how I have to adjust the directories to get the release version working as well. Please see screenshots below:

Thank you in advance.

- process.env.DIST_ELECTRON = join(__dirname, '..')
+ process.env.DIST_ELECTRON = join(__dirname, '../..')

Okay, thank you very much!

I believe this is still happening on win 11
(node:24688) electron: Failed to load URL: file:///C:/Users/USER/AppData/Local/Programs/electron-vite-react/resources/app.asar/dist/index.html with error: ERR_FILE_NOT_FOUND

@juandl Is this caused ba upgrading the vite-plugin-electron version?


Access with C#

I've implemented the following steps using this Python SDK.

Access desired dataset in workspace
Fetch data from Azure Blob Storage
Append data from ABS to dataset
Update dataset in workspace
Now, I'm attempting to write an Azure Function in C# that would perform the same steps. Can you share any documentation for this implementation?

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

ID: c798198a-6a22-41a3-5bdc-5e4e06446a9b
Version Independent ID: 3fb2d7ca-27df-87a8-9fdf-99245438f2e9
Content: Access datasets with Machine Learning Python client library
Content Source: articles/machine-learning/team-data-science-process/python-data-access.md
Service: machine-learning
GitHub Login: @deguhath
Microsoft Alias: deguhath

@mujeeb91 Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.
@deguhath Hi, this is a doc-enhancement idea from our customer, can you please check it and update the document as necessary? Thanks a lot!

@garyericson - who should this be assigned to now.

@marktab - Can you look at this?
@YutongTie-MSFT - The author metadata value on this article has been changed since this PR was opened. The author is now @marktab.

@garyericson Hi, I can not assign to marktab since he is not in our repo. @marktab Can you please join the Azure.docs repo and assign to yourself? Thanks.

@YutongTie-MSFT it may be that I could not be added since my MSFT email was not connected -- but it looks connected now.

@YutongTie-MSFT also to be clear:  I do not see where I either "join" the repo, or "assign" this issue to myself

@mujeeb91 We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


How to layer a object above characters.. (lua stage)

Describe your problem here.
How to make an object layered above the characters using lua (stage)?
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No.

It's on the wiki


XCode Build Error - "ambiguous use of 'sqlite3_value_type'

A build of the current SQLite.swift snapshot, and XCode 9 encounters the following compile issues. Project has SQLCipher enabled, and was building successfully and working well in the past.
Connection.swift:557:24: error: ambiguous use of 'sqlite3_value_type'
switch sqlite3_value_type(value) {
^
SQLite3.sqlite3_value_type:1:13: note: found this candidate
public func sqlite3_value_type(: OpaquePointer!) -> Int32
^
SQLCipher.sqlite3_value_type:1:13: note: found this candidate
public func sqlite3_value_type(: OpaquePointer!) -> Int32
^
Connection.swift:559:79: error: ambiguous use of 'sqlite3_value_bytes'
return Blob(bytes: sqlite3_value_blob(value), length: Int(sqlite3_value_bytes(value)))
^
SQLite3.sqlite3_value_bytes:1:13: note: found this candidate
public func sqlite3_value_bytes(: OpaquePointer!) -> Int32
^
SQLCipher.sqlite3_value_bytes:1:13: note: found this candidate
public func sqlite3_value_bytes(: OpaquePointer!) -> Int32
^
Connection.swift:561:28: error: ambiguous use of 'sqlite3_value_double'
return sqlite3_value_double(value)
^
SQLite3.sqlite3_value_double:1:13: note: found this candidate
public func sqlite3_value_double(: OpaquePointer!) -> Double
^
SQLCipher.sqlite3_value_double:1:13: note: found this candidate
public func sqlite3_value_double(: OpaquePointer!) -> Double
^
Connection.swift:563:28: error: ambiguous use of 'sqlite3_value_int64'
return sqlite3_value_int64(value)
^
SQLite3.sqlite3_value_int64:1:13: note: found this candidate
public func sqlite3_value_int64(: OpaquePointer!) -> sqlite3_int64
^
SQLCipher.sqlite3_value_int64:1:13: note: found this candidate
public func sqlite3_value_int64(: OpaquePointer!) -> sqlite3_int64
^
Connection.swift:567:58: error: ambiguous use of 'sqlite3_value_text'
return String(cString: UnsafePointer(sqlite3_value_text(value)))
^
SQLite3.sqlite3_value_text:1:13: note: found this candidate
public func sqlite3_value_text(: OpaquePointer!) -> UnsafePointer!
^
SQLCipher.sqlite3_value_text:1:13: note: found this candidate
public func sqlite3_value_text(: OpaquePointer!) -> UnsafePointer!
^
Connection.swift:593:9: error: ambiguous use of 'sqlite3_create_function_v2'
sqlite3_create_function_v2(handle, function, Int32(argc), flags, unsafeBitCast(box, to: UnsafeMutableRawPointer.self), { context, argc, value in
^
SQLite3.sqlite3_create_function_v2:2:13: note: found this candidate
public func sqlite3_create_function_v2(_ db: OpaquePointer!, _ zFunctionName: UnsafePointer!, _ nArg: Int32, _ eTextRep: Int32, _ pApp: UnsafeMutableRawPointer!, _ xFunc: (@convention(c) (OpaquePointer?, Int32, UnsafeMutablePointer<OpaquePointer?>?) -> Swift.Void)!, _ xStep: (@convention(c) (OpaquePointer?, Int32, UnsafeMutablePointer<OpaquePointer?>?) -> Swift.Void)!, _ xFinal: (@convention(c) (OpaquePointer?) -> Swift.Void)!, _ xDestroy: (@convention(c) (UnsafeMutableRawPointer?) -> Swift.Void)!) -> Int32
^
SQLCipher.sqlite3_create_function_v2:1:13: note: found this candidate
public func sqlite3_create_function_v2(_ db: OpaquePointer!, _ zFunctionName: UnsafePointer!, _ nArg: Int32, _ eTextRep: Int32, _ pApp: UnsafeMutableRawPointer!, _ xFunc: (@convention(c) (OpaquePointer?, Int32, UnsafeMutablePointer<OpaquePointer?>?) -> Swift.Void)!, _ xStep: (@convention(c) (OpaquePointer?, Int32, UnsafeMutablePointer<OpaquePointer?>?) -> Swift.Void)!, _ xFinal: (@convention(c) (OpaquePointer?) -> Swift.Void)!, _ xDestroy: (@convention(c) (UnsafeMutableRawPointer?) -> Swift.Void)!) -> Int32

which version of SQLCipher are you building against?

Version v3.4.1 (last commit: df092f0a Dec 28, 2016). My projects were building and running until recently when I updated Xcode.
Thank you for looking into this.

Hm not sure. I've only tested the SQLCipher integration via CocoaPods (where it works fine).
I think the problem is that now SQLite3 is always imported. You'd have to define SQLITE_SWIFT_SQLCIPHER to avoid this, but then you'll probably get errors that the module SQLCipher can't be found.

Thank you for pointing out that integration via CocoaPods is working. I chose that route. Again, thanks for looking into this.


Authenticate jsha to use Google Search Console

This allows troubleshooting issues with why Google doesn't crawl docs.rs deeply enough.
Related to https://github.com/rust-lang/docs.rs/issues/1438#issuecomment-1174811787
Per https://support.google.com/webmasters/answer/9128668?visit_id=637926083810327397-791277710&rd=2:

Search Console offers tools and reports for the following actions:

Confirm that Google can find and crawl your site.
Fix indexing problems and request re-indexing of new or updated content.
View Google Search traffic data for your site: how often your site appears in
Google Search, which search queries show your site, how often searchers click through for those queries, and more.
Receive alerts when Google encounters indexing, spam, or other issues on your site.
Show you which sites link to your website.
Troubleshoot issues for AMP, mobile usability, and other Search features.

Per https://support.google.com/webmasters/answer/7687615#manage-owners&zippy=%2Cadd-or-remove-verified-owners, removing this handler later will deauthenticate me.
Here's an example of the sort of report I'll be able to get from Google Search Console:

I have no problem with giving them access to the webmaster console, but we should do it by having the infrastructure team setup the account and then granting jsha access to it, rather than having jsha setup their own account linked to docs.rs. I'll add that to my todo list.


Use prop-types instead of React.PropTypes

Updated PropTypes usage to remove react warnings for 15.5.x
Remove the lib folder from the repo (it shouldn't be there.. You get it by running npm run build

Resolves https://github.com/Adphorus/react-date-range/issues/130

Will publish this tomorrow.

Great @burakcan ! Will wait for your publishing today so we could stop getting these errors..

I'm waiting too :]

Published it yesterday


Clean search help results at script tab

Could you give a option to clean search help results or disable it?  My script tab get flooded with these history results together with scripts.

There is option for it if you use latest commit.
Settings -> Editor Settings -> Text Editor -> Group Help Pages

You can close both scripts and help pages with File > Close (Ctrl+W).

Wow..  I didn't know it!
Thank you!


Bind Etherpad user to Editor User

Create User
Find Pad
... or Create Pad
Get Group for Pad
... or Create Group
Add user to group for pad
Things to reference:
http://i.imgur.com/d0nWp.png
ep disable name change plugin
https://github.com/tykeal/ep_ldapauth/blob/master/ep_ldapauth.js#L64
https://github.com/wtsi-hgi/ep_sotauth/blob/master/ep_sotauth.js#L11
http://etherpad.org/doc/v1.5.7/#index_createauthorifnotexistsfor_authormapper_name
https://github.com/tomassedovic/etherpad-lite-client-js

Done in 7c208375a970ab303ccb59342dbefb944e4df138  and opengovfoundation/ep_api_auth@1a7ba89c0d707b54876d4dccb92f2225b17d2a0b and opengovfoundation/ep_api_auth@a521c00fd22dac5fba41c5bb8494b43ccba95c1c


Error occurs when using dataset with many series：TypeError: Cannot read property 'width' of undefined

Version
4.2.1
Steps to reproduce
var source = [["分类", "陆慧", "王荣平", "测试功能", "0001", "123", "钟馗", "丁为芹", "刘平", "沈鑫", "王阿娣", "袁彬", "张兰", "郑松华", "卢静", "杨晶晶", "郑浪云", "顾燕", "许晨晨", "林炳青", "陈娟", "严国平", "黄雯雯", "潘湘宇", "蔡雪燕", "陈锦秀", "陈静", "董荣", "付春艳", "郭中琴", "刘炜", "潘婷", "邵玉蓉", "童娓娓", "王婷婷", "杨明晶", "叶长琳", "翟洪艳", "张阳", "周成亚", "周玲玲", "周长娟", "黄燕", "唐萍", "王丽", "徐华浩", "朱雪梅", "朱银娟", "刘叶", "王婷", "王燕", "徐霞", "袁业伟", "陈静B2", "郭中琴A2", "黄雯雯B2", "黄妍", "姜楠", "孟冬梅", "倪娜", "潘婷A2", "唐萍B2", "王洪玉", "王荣平A2", "王燕B2", "韦文燕", "武一兄", "肖桑", "袁叶婷", "掌霞", "韦文燕A2", "柏希", "卞芹", "封粉淦", "韩桥荣", "吉海艳", "施园月", "徐洁", "周敏", "朱雪娇", "单晓红", "树菊", "徐爱霞", "郇英", "赵昕", "周云", "福田王五", "郭001", "李四", "杨002", "福田李四", "福田张三", "新建人员", "罗湖李四", "杨02", "杨彬002", "水贝李四", "水贝张三", "水贝赵六", "水贝王五", "张梦玲", "张育楠", "周晓东", "陈小娟", "陈亚玲", "陈艳", "彭春霞", "沈雄", "史玲玲", "孙玉梅", "王莹", "邢美瑶", "徐陈慧", "徐帆", "严婷婷", "叶海婷", "郑燕雪", "陈汉云", "程凤", "许蓉", "蒋晓梅", "李庆芝", "王娟娟", "王雅", "潘洪", "龙久梅"],
                ["A门店", 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]];
var series = Array(source[0].length - 1).fill({
        type: 'bar'
    });
option = {
    legend: {},
    tooltip: {},
    dataset: {
        source: source
    },
    xAxis: {type: 'category'},
    yAxis: {},
    series: series
};

What is expected?
不报错
What is actually happening?
TypeError: Cannot read property 'width' of undefined

getLineWidth	@	echarts.js:41774
getLayout.cartesian2d	@	echarts.js:41708
(anonymous function)	@	echarts.js:41538
DataDiffer.execute	@	echarts.js:29232
extendChartView._renderNormal	@	echarts.js:41588
extendChartView.render	@	echarts.js:41486
progressMethodMap.render.progress	@	echarts.js:24544
doProgress	@	echarts.js:23466
taskProto.perform	@	echarts.js:23389
(anonymous function)	@	echarts.js:28232
(anonymous function)	@	echarts.js:21424
each$1	@	echarts.js:528
Model.extend.eachSeries	@	echarts.js:21422
renderSeries	@	echarts.js:28221
render	@	echarts.js:28194
updateMethods.update	@	echarts.js:27573
echartsProto.setOption	@	echarts.js:27133

使用dataset，series数量过多并且其中有名称为纯数字时会触发这个异常，数量少时不会

This bug is with v4.2.1 and cannot reproduce since v4.3.0.
@gaoqiangz Please help check if v4.3.0 works for you.

@gaoqiangz Could you please help check if this problem has been fixed? Otherwise, it will be closed soon.

The problem has been fixed，thanks!

Removed from 4.5.0 because it's previously fixed.


⚠ ClamAV has degraded performance

In d2134b7, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 99 ms

Resolved: ClamAV performance has improved in 0d783e7.


HELP

When ever I add this library in my project the compilation freezes at this step
Indexing .pio\build\nodemcuv2\libFrameworkArduino.a
No idea how to fix it

Nothing related to library.
Change your board, disable Anti virus, reinstall Arduino IDE or reinstall OS


task_container should cleanup tasks inline

This was an issue identified with the asynchronous echo server where running out of file descriptors would cause tasks to never be cleaned up in the task container.  Cleaning them up inline when they complete should resolve this issue.

Closes #130

The issue around this seems to be branch shaking with a static lib, when statically linking some of the coroutine frame.Destroy functions are omitted, so I'll switch to dynamically linking for the forseeable future to fix this issue.


volumes [drives] referenced by device index (instead of drive letter) causes problems when indexes change internally

I lately added a new hard drive for More Games Install Space, and HIDHide stores the whitelist by volume reference (\Device\HarddiskVolume4) rather than by drive letter (D:\) and somewhere between adding the new drive, and a crash-reboot, the C:\ diskvolume changed from Vol4 to Vol6, and without access to an actual config file or the possibility to textedit the paths directly, I'm stuck with around 20 entries I'm having to rebuild one by one

That's unfortunate but I can't think of any recommendations other than scripting a fix using the CLI.


Block Fenced Code boxes broken on mobile

Long lines of code escape the box.

The overflow seems to be handled correctly to me.

This is on Firefox/Android.

The CSS property overflow: auto hidden  is used to handle the overflow, which should be supported by Firefox.
Please check against this page with the overflow: auto; demo to see whether your browser is compatible with this CSS property.

I think it might have to do with how I am showing complete posts rather than Summary. I will check. (Browser is fine on that test page)
Did you add a site configuration for showing full posts instead of Summary/Description? That's one thing I couldn't figure out with loveit.

No, there is no such configuration.

It works correctly on iOS Safari as well. So I think it may be caused by some browser compatibility issues...

Works fine on Kiwi browser (chromium based) too.  I will see if I can identify the source of the issue on firefox.

I think Firefox was caching the old state of the site with loveit's broken box. Thanks for your help. All good now.


fix(material): add properties to RaisedButton

Adds focusElevation, hoverElevation, and padding to RaisedButton.icon()
Description
Brings configuration parity to RaisedButton.icon() from the base RaisedButton.
Related Issues
Fixes #40696.
Tests
This doesn't change behavior, just adds additional configuration options for RaisedButton.icon().
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[ ] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
[x] No, this is not a breaking change.

This PR will need tests.

@HansMuller What might you suggest?
There are not currently any tests for RaisedButton() for padding, focusElevation, or hoverElevation:
https://github.com/flutter/flutter/blob/b11c5acaa6200a8d662705a312a2e5fb7714fe2e/packages/flutter/test/material/raised_button_test.dart
These tests are, however, on the generic MaterialButton():
https://github.com/flutter/flutter/blob/b11c5acaa6200a8d662705a312a2e5fb7714fe2e/packages/flutter/test/material/buttons_test.dart

You can test the padding parameter by verifying that the button's child appears where it's expected when padding is provided. Since RaisedButton et al are simple subclasses of MaterialButton and padding is essentially a pass-through parameter, it would be OK to write another MaterialButton test.

Hi @cpboyd are you planning on working on adding tests anytime soon? We could close this issue for now and re-open it when you plan on revisiting this PR.

@shihaohong I'm busy with non-Flutter projects at the moment.  I'll see about revisiting this later


Merge develop to main

Pull Request Template
Description
Merge current changes to main
Type of change
How Has This Been Tested?
Not tested
Test Configuration:

Environment:

Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have made corresponding changes to the architecture
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Trenger en approve til, @eaidland @equinor-jmel


[Bug] install-lsp-features fetches unsupported version of bash-language-server

NPM package bash-language-server provides IDE-like autocomplete/hinting for Jupyter in the install-lsp-features sample LCC (used here).
However, this week they released major version 5.0.0 which drops support for NodeJS v14... And currently, SageMaker Studio JupyterServer seems to use NodeJS v14 so the sample now fails to run.
Suggest pinning this install as e.g<EMAIL_ADDRESS>until Studio JupyterServer environment gets updated?

PR #35 merged


Prevent crashes during Airflow server forking on MacOS

Describe changes
I implemented/fixed _ to achieve _.
Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table and the corresponding website section.
[ ] I have added tests to cover my changes.

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

Tested and working on my M1 Mac 👍


Merged Mews.Fiscalizations.Uniwix with Mews.Fiscalizations.Italy

Next steps:

Add tests for the Italian fiscalization
Add GH secrets for the credentials
Add GH actions (pipelines) for Italy lib.
Update the documentation with code examples.

@marektresnak tested?

@TashaBubo Does it make sense for these libs? I don't think there's ever a situation that would require manual testing. This can only be manually tested when it's actually used in Mews, which requires a separate PR.


Monitoring database performance on site

After migrating this evening, we've experienced a couple of minor latency jumps
which have resulted in bot commands performing poorly.
We should find a way to accurately measure and find remedial action to ensure
latency is as low as possible between site and bot.
This should come in the form of some tracing solution within Django that allows
us to browse how much time is spent per database operation.

This is resolved now, we have this data via django-prometheus.
We can investigate tracing further in the future.