Can't Authenticate using app password.

Describe the bug
Can't Authenticate the google_home integration in HA.
I have looked at various old similar issues and tried the resolutions there.  None have worked.
Issue appears to be Cannot authenticate.
Have previously had this configured though.
Rebuilt server moving to Ubuntu 22.04 and never been able to reconfigure.
Also tried the following but would not work due to issue getting MAC address.
I included the MAC address of a google home on my network and also output from logs above and output is below.
python3.11 get_tokens.py
This script generates tokens that can be used when making requests to the Google Home Foyer API.
There are 2 kinds of tokens used here:

Master token - Is in the form aas_et/*** and is long lived. Needs Google username and password.
Access token - Is in the form ya29.*** and lasts for an hour. Needs Master token to generate.
If you do not want to store the Google account password in plaintext,
get the master token once, and set it as an override value.
It's safer/easier to generate an app password and use it instead of the actual password.
It still has the same access as the regular password, but still better than using the real password while scripting.
(https://myaccount.google.com/apppasswords)

[] Getting master token...
[!] Could not get master token.
[] Master token: None
[] Getting access token...
[!] Could not get access token.
[] Access token: None
[*] Done.
To replicate
add google_home integration via HACS
restart ha
attempt to configure integration and authenticate
add username (without @gmail.com) and use app password

Version of the integration
If you are unsure about the version check the github release here.

1.9.17
glocaltokens==0.7.0
gpsoauth==1.0.2
requests==2.28.2
requests-toolbelt==0.10.1
urllib3==1.26.14
uuid==1.30

Logs

2023-03-06 18:40:51.369 DEBUG (MainThread) [glocaltokens.client] Initializing new GLocalAuthenticationTokens instance.
2023-03-06 18:40:51.369 DEBUG (MainThread) [glocaltokens.client] Set GLocalAuthenticationTokens client access_token, homegraph, access_token_date and homegraph_date to None
2023-03-06 18:40:51.370 DEBUG (MainThread) [glocaltokens.client] Set GLocalAuthenticationTokens client username to "******", password to "***************", master_token to "" and android_id to
2023-03-06 18:40:51.370 DEBUG (SyncWorker_6) [glocaltokens.client] There is no stored master_token, logging in using username and password
2023-03-06 18:40:51.370 DEBUG (SyncWorker_6) [glocaltokens.client] There is no stored android_id, generating a new one
2023-03-06 18:40:51.370 DEBUG (SyncWorker_6) [glocaltokens.client] Generating android id...
2023-03-06 18:40:51.371 DEBUG (SyncWorker_6) [glocaltokens.client] Generated android id: a7574b3e185fd258
2023-03-06 18:40:52.826 ERROR (SyncWorker_6) [glocaltokens.client] [!] Could not get master token.
2023-03-06 18:40:52.827 DEBUG (SyncWorker_6) [glocaltokens.client] Request response: {'Error': 'BadAuthentication'}
2023-03-06 18:40:52.828 ERROR (MainThread) [custom_components.google_home]

Closed, because it's a duplicate of https://github.com/leikoilja/ha-google-home/issues/599


Remote Transcoder has errors trying to play a file on remote server, works fine with local transcoder as a target

I've attached the logs I pulled from plex when attempting to play a file that requires transcoding with a remote transcoder. This works fine on the local transcoder, (adding localhost with prt add_host) so it must be a missing dependency on my ubuntu server or something.
plex failed transcode.txt

This looks like a cross-platform issue.  The server on OS X is passing environmental variables to the Ubuntu machine that are incorrect for that platform.
From your logs:
Sep 06, 2016 01:46:51.315	DEBUG	Job running: FFMPEG_EXTERNAL_LIBS='/Users/Alex/Library/Application\ Support/Plex\ Media\ Server/Codecs/6209f91-1114-darwin-x86_64/' XDG_CACHE_HOME='/Users/Alex/Library/Caches/PlexMediaServer/' XDG_DATA_HOME='/Applications/Plex Media Server.app/Contents/Resources/' X_PLEX_TOKEN='xxxxxxxxxxxxxxxxxxxx' '/Applications/Plex Media Server.app/Contents/MacOS/Plex Transcoder' '-codec:0' 'mpeg4' '-codec:#0x01' 'mp3_at' '-i' '/Volumes/Shares/Media/Videos/TV Shows/America\'s Test Kitchen/Season 9/America\'s Test Kitchen - S09E09 - Bringing Home Italian Favorites.avi' '-filter_complex' '[0:0]scale=w=min(720\,iw):h=min(480\,ih):force_original_aspect_ratio=decrease[0]' '-map' '[0]' '-codec:0' 'libx264' '-crf:0' '16' '-pix_fmt:0' 'yuv420p' '-maxrate:0' '2250k' '-bufsize:0' '4500k' '-r:0' '29.969999999999999' '-preset:0' 'veryfast' '-x264opts:0' 'subme=2:me_range=4:rc_lookahead=10:me=dia:no_chroma_me:8x8dct=0:partitions=none' '-force_key_frames:0' 'expr:gte(t,0+n_forced*7)' '-map' '0:#0x01' '-codec:1' 'aac_at' '-ar:1' '48000' '-channel_layout:1' 'stereo' '-b:1' '256k' '-f' 'dash' '-use_timeline' '0' '-min_seg_duration' '7000000' '-skip_to_segment' '1' '-time_delta' '0.0625' '-avoid_negative_ts' 'disabled' '-map_metadata' '-1' '-map_chapters' '-1' '-movflags' '+faststart' 'dash' '-start_at_zero' '-copyts' '-vsync' 'cfr' '-y' '-nostats' '-loglevel' 'quiet' '-loglevel_plex' 'error' '-progressurl' 'http://<IP_ADDRESS>:32400/video/:/transcode/session/pb2mcpr7yde/progress'

You can see the paths for FFMPEG_EXTERNAL_LIBS, XDG_CACHE_HOME, XDG_DATA_HOME, etc. are all wrong.  We currently don't have a mechanism to translate paths from one platform to another, so this is something that we'll have to look into.

Of course! Duh. So right now your Master and Slave both need to be the same platform, or at the very least have the same paths to the external libraries. Is that right?

@alexholliz For now, yes, master and slave need to be the same platform.  Obviously we'd love to get this working between machines running on different platforms.  If you look at the prt.log on the slave machine, you should see some information about the command that is used to launch the transcoder.  If you're feeling up to it, you can patch PRT on the Ubuntu machine to remap the paths since most paths should be constants for the given platform (with the media path being the main exception).
To help get started, I just created a wiki page here with the paths used on various platforms.

I'll give that a go and report back

@alexholliz Great!  Also, feel free to update the wiki page with the paths that you're seeing on OS X.  I've added the ones that I could get from the log file that you posted earlier.

I was actually pretty confused on how to statically set the FFMPEG libraries on the ubuntu machine, any idea where I should look for that?

The paths are set as environmental variables.  The best place to go about remapping these is in the build_env  method.  What I would do is loop over items in os.environ, look for variables that need to be remapped (such as FFMPEG_EXTERNAL_LIBS) and remap them.

I also noticed that I don't have a prt.log on my ubuntu machine. Is there somewhere I should look for that specifically? mlocate couldn't find it.

Should be set to /tmp/prt.log by default.


Try to launch PyLint crashes my PTVS

It was the first time I try to launch PyLint, so PTVS proposes me to install it. It installed PyLint 1.6.2.
Just after that (3 seconds), PTVS crashed:

After restarting, I tried again but crashes each time.
Using a Venv Python 3.5 on PTVS 2.2.40315.00 and VS 14.0.25123.00 Update 2

I'm currently updating to test on newer version

You should have error messages in Event Viewer under the Application log. If you could copy those into the issue it'll probably be really easy to fix this.

Still broken with PTVS 2.2.40623.00 and VS 14.0.25422.01 Update 3
You mean this:

You need details view of the three?

Just the one with the call stack in it - there's probably a ".NET Runtime" event nearby with it.

Error Handler Exception: System.ServiceModel.CommunicationException: There was an error reading from the pipe: The pipe has been ended. (109, 0x6d). ---> System.IO.IOException: The read operation failed, see inner exception. ---> System.ServiceModel.CommunicationException: There was an error reading from the pipe: The pipe has been ended. (109, 0x6d). ---> System.IO.PipeException: There was an error reading from the pipe: The pipe has been ended. (109, 0x6d). at System.ServiceModel.Channels.PipeConnection.OnAsyncReadComplete(Boolean haveResult, Int32 error, Int32 numBytes) --- End of inner exception stack trace --- at System.Runtime.AsyncResult.End[TAsyncResult](IAsyncResult result) at System.ServiceModel.Channels.ConnectionStream.EndRead(IAsyncResult asyncResult) at System.Net.FixedSizeReader.ReadCallback(IAsyncResult transportResult) --- End of inner exception stack trace --- at System.Net.Security.NegotiateStream.EndRead(IAsyncResult asyncResult) at System.ServiceModel.Channels.StreamConnection.EndRead() --- End of inner exception stack trace --- at System.Runtime.AsyncResult.End[TAsyncResult](IAsyncResult result) at System.ServiceModel.Channels.TransportDuplexSessionChannel.EndTryReceive(IAsyncResult result, Message& message) at System.ServiceModel.Dispatcher.DuplexChannelBinder.EndTryReceive(IAsyncResult result, RequestContext& requestContext) at System.ServiceModel.Dispatcher.ErrorHandlingReceiver.EndTryReceive(IAsyncResult result, RequestContext& requestContext) \r\n at System.Runtime.AsyncResult.End[TAsyncResult](IAsyncResult result) at System.ServiceModel.Channels.TransportDuplexSessionChannel.EndTryReceive(IAsyncResult result, Message& message) at System.ServiceModel.Dispatcher.DuplexChannelBinder.EndTryReceive(IAsyncResult result, RequestContext& requestContext) at System.ServiceModel.Dispatcher.ErrorHandlingReceiver.EndTryReceive(IAsyncResult result, RequestContext& requestContext)

Don't know how to present this better :(

There is a .NET Runtime 3 minutes before:
Error Handler Exception: System.ServiceModel.CommunicationException: There was an error reading from the pipe: The pipe has been ended. (109, 0x6d). ---> System.IO.IOException: The read operation failed, see inner exception. ---> System.ServiceModel.CommunicationException: There was an error reading from the pipe: The pipe has been ended. (109, 0x6d). ---> System.IO.PipeException: There was an error reading from the pipe: The pipe has been ended. (109, 0x6d). at System.ServiceModel.Channels.PipeConnection.OnAsyncReadComplete(Boolean haveResult, Int32 error, Int32 numBytes) --- End of inner exception stack trace --- at System.Runtime.AsyncResult.End[TAsyncResult](IAsyncResult result) at System.ServiceModel.Channels.ConnectionStream.EndRead(IAsyncResult asyncResult) at System.Net.FixedSizeReader.ReadCallback(IAsyncResult transportResult) --- End of inner exception stack trace --- at System.Net.Security.NegotiateStream.EndRead(IAsyncResult asyncResult) at System.ServiceModel.Channels.StreamConnection.EndRead() --- End of inner exception stack trace --- at System.Runtime.AsyncResult.End[TAsyncResult](IAsyncResult result) at System.ServiceModel.Channels.TransportDuplexSessionChannel.EndTryReceive(IAsyncResult result, Message& message) at System.ServiceModel.Dispatcher.DuplexChannelBinder.EndTryReceive(IAsyncResult result, RequestContext& requestContext) at System.ServiceModel.Dispatcher.ErrorHandlingReceiver.EndTryReceive(IAsyncResult result, RequestContext& requestContext) \r\n at System.Runtime.AsyncResult.End[TAsyncResult](IAsyncResult result) at System.ServiceModel.Channels.TransportDuplexSessionChannel.EndTryReceive(IAsyncResult result, Message& message) at System.ServiceModel.Dispatcher.DuplexChannelBinder.EndTryReceive(IAsyncResult result, RequestContext& requestContext) at System.ServiceModel.Dispatcher.ErrorHandlingReceiver.EndTryReceive(IAsyncResult result, RequestContext& requestContext)

I normally find-replace " at " with "\r\n at " and then embed the whole thing in triple backquotes.
The latter looks like ours, but unfortunately it's not at all obvious what the cause is. Still, I can try and repro it (or if you want to dig deeper and figure it out, I can help point you in the right direction?)

I'm always glad to help, but here I don't code C#, I'm not sure I have the necessary skill right now to dig deeper... :(

I tried to repro this with either or both windows not open at VS startup, but no lock so far.
The code that deals with the Error List ultimately just uses ErrorListProvider. It's not documented to throw, but perhaps it could. We could do defensive try/catch around the invocation of its constructor - but it's not clear what to catch.
For the output window, we do throw InvalidOperationException if we couldn't obtain SVsOutputWindow, or if we couldn't create the desired output window pane. We could handle this as well.
Although in both cases, it's not clear why any of these would fail in the first place; and if they do, the real root cause would be the reason for that failure. We should be able to show error list & output if they aren't there yet.

@int19h I agree, but I want the errors to go away :)
I'll add some general error handling. We can't do anything useful if this fails, and the E_FAIL error code isn't helpful for anyone, so may as well just suppress it.

Actually, the original issue has no relation to the one I linked, so really we should have a new issue for it. Then again, with no +1s on the other old issue, it was probably due for closure anyway, so I'll just hijack it and fix it :)


Singularizes and expands godocs of HTTPRoute hostname

Previously the hostname of a HTTPRouteHost was plural. This PR singularizes the field and adds more detailed godocs.
Note: The RFC 3986 requirement is carried over from IngressRule v1beta1.
/assign @bowei
/cc @ironcladlou @Miciah

I think most of the issues raised above are captured in the ongoing discussions on the issues, so we can merge this.
/lgtm
/approve

Please open an issue if you don't see something that should be tracked.


code

The paper named [Attention-Inception and Long- Short-Term Memory-Based Electroencephalography Classification for Motor Imagery Tasks in Rehabilitation] ,do you prepare to publish the code?

Hi, I do not have the authority to share the code for this paper. Please contact the first author of the paper.


keeping your tokens safe

So with ionic apps, i find that its possible to deconstuct the app (many websites that can do this) and get access to much of the code js
With your plugin (which i am using for facebook and it works great) all I give up in my code is the facebook id, but for a twitter auth I'm giving up my consumerKey and consumerSecretKey...is this not a potential security threat to my twitter account?
Love your thoughts on this.

You are correct, apps are very easy to decompile:
https://blog.nraboy.com/2014/11/extract-android-apk-view-source-code/
It is safe to use oauth providers that make use of implicit grants (client key only).  Oauth providers that make use of explicit grants (client key and client secret) only should be used at your own risk.
Regardless on how safe or unsafe certain providers are, I get endless requests to add them.  I gave up the job of cyber-nanny.  My best advice would be to know what you're getting yourself into and see if it is worth it to you.

No I get it, and I'm glad its there. Do you recommend any sorts of obufusion plugins or methods that you think would be be effective?

Always obsfucate / minify your JavaScript code.  It won't stop the dedicated people from breaking in, but it will certainly deter a lot of people.

If you don't mind sharing, what methods or libraries do you use?

See this:
http://blog.ionic.io/minifying-your-source-code/

Great post!! Thanks for all the work you are doing educating people. It's
much appreciated and helping ease the learning curve.
On Oct 9, 2015 2:20 PM, "Nic Raboy"<EMAIL_ADDRESS>wrote:

See this:
http://blog.ionic.io/minifying-your-source-code/
—
Reply to this email directly or view it on GitHub
https://github.com/nraboy/ng-cordova-oauth/issues/145#issuecomment-146988596
.


Add "yield" support to the syntax-only fork

Yield (for generators) is now supported by coffee-script. This pull request is a simple modification to reflect that change.

Completely missed this pull request. Thanks @cybeliak.


fix: stuck local build issue

Local build was failing due to CachedXMLHTTPRequest logic issues.
These logic issues involved the fact that the private functions of our plugin were overrided as intended (i.e. send()). But the defined properties (defined with Object.defineProperty) were still using the old plugin!.
Changed our plugin name and disabled the built-in plugin using the Module flag to prevent this weird behaviour.
Also, this means that hotpatch.sh is not needed anymore, so removed it from the build system.

Confirmed patch not necessary anymore


Some small suggestions

Just a few observations.
1. Reporting delay
The glanceability of the first chart is great. But the ones below it are a bit harder for me to understand. Possibly because I'm math'icapped. What I've been missing is a graph similar to the first one that shows death dates for the deaths that have been reported today only.
2. Navigate between dates
It would be interesting if I could go back and see what the graphs looked like on a specific date.
3. The font weight of the graph headers seems to be set a bit too high
This is clearly the most important point :)

I agree with suggestion 2.  That is what I came here to suggest!
Even just don't delete the old files from the web directory; then we can manually change the date in the filename for the png and see the old predictions.

Even just don't delete the old files from the web directory; then we can manually change the date in the filename for the png and see the old predictions.

The old graphs are available.

I've made an interactive graph where you can click and drag on the bottom graph (showing number of reported cases) which will filter the top graph showing actual death dates.
You probably want to select the leftmost bar and then drag to the right, but you can make other selections as well.

1. Reporting delay
The glanceability of the first chart is great. But the ones below it are a bit harder for me to understand. Possibly because I'm math'icapped. What I've been missing is a graph similar to the first one that shows death dates for the deaths that have been reported today only.

I just realized that by making the selection just one day wide you can see just what you are after. If you want you can then drag the selection to see other reporting days as well.

Thanks for answering @morberg! Re 1: I agree. Haven't figured out a better way to do it though. The important thing is to achieve some way of tracking the size of the delay over time. So one can see if its changing, like it did over easter.

I made an attempt in a different issue comment, but it was a bit cluttered. This might be better?

If you plot each individual reporting day it gives an understanding for each day, but gives no sense of trend:

I would also add a line like the 90 percentile but with the median.

@morberg Your interactive graph looks really good. I came hear to suggest something similar. What I'm really after is the historical forecast, if you can add the historical forcasts to your interactive graph I think it will be as perfect as can be!
With the forecast chaning each day we can get a grasp of how accurate it is, or isn't.

@wederbrand: Here you go.


fix: Formatting blockquotes in datatrackers rendered markdown

This adds a utility class for HTML blobs of content (ie dynamic content...content that we can't add classes) to format <blockquote>.
In the following example the "text" is a blockquote which has padding-left and border-left:

Fixes https://github.com/orgs/ietf-tools/projects/3/views/4?pane=issue&itemId=58443314

This gets one place where we are rendering markdown - would be format all the places.

@rjsparks which other ones should be updated? All the document_* templates?
document_ballot_content.html   document_draft.html           document_material.html       document_status_change.html
document_ballot.html           document_email.html           document_polls.html          document_subseries.html
document_bibtex.bib            document_format_buttons.html  document_referenced_by.html  document_subseries_top.html
document_bofreq.html           document_history_form.html    document_references.html     document_top.html
document_charter.html          document_history.html         document_review.html         document_writeup.html
document_chatlog.html          document_html.html            document_rfc.html
document_conflict_review.html  document_info.html            document_statement.html

@rjsparks Because we're changing the default formatting of <blockquote> to have a border-left I updated all the existing blockquotes to render without a border with those inline style="border-left: 0". Arguably those should just be <div>s for semantic reasons but I guess that can happen in another PR

Yes - some of those pages are crufty and may need to move out of the datatracker into another property or just go away.


Resize no longer accept null for auto scale at v0.15

Resize no longer accept null for auto scale as mention at http://sharp.readthedocs.io/en/stable/api/#resizewidth-height-options after update to 0.15

Hello, there's a passing test case for fixed height and null=auto-width at https://github.com/lovell/sharp/blob/master/test/unit/resize.js#L32-L41
I modified this locally for an example of fixed width, null=auto-height, which also passed:
  it('Fixed width', function(done) {
    sharp(fixtures.inputJpg).resize(320, null).toBuffer(function(err, data, info) {
      if (err) throw err;
      assert.strictEqual(true, data.length > 0);
      assert.strictEqual('jpeg', info.format);
      assert.strictEqual(320, info.width);
      assert.strictEqual(261, info.height);
      done();
    });
  });

Are you able to provide more details of the error and/or effects of the problem you're experiencing?

Sorry, false alarm.
My console some how return null instead of NaN when I attempt to parseInt(null).


Missing title attribute from tree branches/items

When implementing the Tree component in a narrow container, the branches and items are properly truncated, but there doesn't seem to be a way to add a way to display the text non-truncated.
After comparing the markup, it looks like the <span/>s are missing the title attribute that would give us the browser's hover text automatically.
I will have a PR shortly to fix this.

Great, thanks!


[azure-template-cpp] publish version 1.0.0-beta.1472690

Update vcpkg ports for Azure SDK release. This release may contain multiple ports.

Adding azure-template to release
cc: @danieljurek, @weshaggard


 Due to Civ vehicles being spawned at the same position as existing vehicles.
Added a check here to avoid this. Also set vehicle speed to limited to avoid road rage.

Yeah! Noted!

Done :)


Added support for cross-compiling to mingw

You can cross-compile for windows on a linux host with:
$ make TARGET=mingw

To test, you can run main.exe with wine (a bit slow, of course).

Very interesting. How did you install x86_64-w64-mingw32-gcc?

It's usually contained in the distros.
For example in Debian / Ubuntu the package is called: gcc-mingw-w64-x86-64-win32.
Otherwise you can get it from: https://www.mingw-w64.org/downloads/
In combination with wine it's quite easy and comfortable for building and testing the windows target and it works pretty well.

Thanks! I will have to try with wine. That's neat.
Should use $(TARGET) in clean rule too
Also at this point it's worth having a separate "configure" script or something. Makefile is getting hairy with control flow and redefining variables.

True!
I fixed the clean rule also for native windows builds where the executable also should have an .exe suffix.
Maybe make is still good enough, which might change when adding more targets.


Support bech32 encoding for CLI key derivation

Issue Number
ADP-195. No issue yet.

Overview

[x] Added a keyBech32Codec abstraction for inferring the encoding automatically
[x] Added support for reading input from stdin and chaining commands with |.

Comments
Commands can be chained and work with both hex and bech32 encodings!
$ cardano-wallet-byron key root --wallet-style icarus --encoding bech32 -- express theme celery coral permit dizzy nothing stadium lemon tube olive brass dutch find mask \
    | cardano-wallet-byron key public
xpub1k365denpkmqhj9zj6qpaxat4dh0xetpz2qyneerrputtnwwq4jr5zyehhkjdt0qzzerzfq9cpxpyl76g79lq3k26h8cmj8fereywv6c0g4qe2

$ cardano-wallet-byron key root --wallet-style icarus -- express theme celery coral permit dizzy nothing stadium lemon tube olive brass dutch find mask \
     | cardano-wallet-byron key public
b47546e661b6c1791452d003d375756dde6cac2250093ce4630f16b9b9c0ac87411337bda4d5bc0216462480b809824ffb48f17e08d95ab9f1b91d391e48e66b

Closing in favour of #1599.
While interesting, encapsulating all logic inside CliKeyScheme for the sake of roundtrips might be overkill here. The golden tests in #1599 should be enough for catching errors in the glue code.


engine: refactor into a reactive builder, so we can build in response to things like pod restarts, etc

Hello @landism, @maiamcc,
Please review the following commits I made in branch nicks/reactive:
796185c3234d118064b9ae07667627215373f99c (2018-10-19 19:27:51 -0400)
engine: refactor into a reactive builder, so we can build in response to things like pod restarts, etc

this isn't quite where i want it to be yet, but moves us in the right direction

@maiamcc of course!


feat: Add instance store volume option for instances with local disk

PR o'clock
Issue
Resolves https://github.com/terraform-aws-modules/terraform-aws-eks/issues/1212
Description
You can only supply EBS type volumes in worker launch templates.  This PR aims to add the ability to also specify instance store volumes so that instances with local disk may be utilized efficiently.
The README did not need to be modified because it references local.tf where this change happens, but I did modify that section along with comments as seems to be the pattern.  There are two values being added (Hard to see because of terraform fmt)

additional_instance_store_volumes
instance_store_virtual_name

Diff output without terraform fmt
diff --git a/local.tf b/local.tf
index 09d62f0..584ead4 100644
--- a/local.tf
+++ b/local.tf
@@ -40,6 +40,7 @@ locals {
     health_check_type             = null                        # Controls how health checking is done. Valid values are "EC2" or "ELB".
     health_check_grace_period     = null                        # Time in seconds after instance comes into service before checking health.
     instance_type                 = "m4.large"                  # Size of the workers instances.
+    instance_store_virtual_name   = "ephemeral0"                # "virtual_name" of the instance store volume.
     spot_price                    = ""                          # Cost of spot instance.
     placement_tenancy             = ""                          # The tenancy of the instance. Valid values are "default" or "dedicated".
     root_volume_size              = "100"                       # root volume size of workers instances.
@@ -71,6 +72,7 @@ locals {
     termination_policies          = []                          # A list of policies to decide how the instances in the auto scale group should be terminated.
     platform                      = "linux"                     # Platform of workers. either "linux" or "windows"
     additional_ebs_volumes        = []                          # A list of additional volumes to be attached to the instances on this Auto Scaling group. Each volume should be an object with the following: block_device_name (required), volume_size, volume_type, iops, encrypted, kms_key_id (only on launch-template), delete_on_termination. Optional values are grabbed from root volume or from defaults
+    additional_instance_store_volumes = []                      # A list of additional instance store (local disk) volumes to be attached to the instances on this Auto Scaling group. Each volume should be an object with the following: block_device_name (required), virtual_name.
     # Settings for launch templates
     root_block_device_name               = data.aws_ami.eks_worker.root_device_name # Root device name for workers. If non is provided, will assume default AMI was used.
     root_kms_key_id                      = ""                                       # The KMS key to use when encrypting the root storage device
diff --git a/workers_launch_template.tf b/workers_launch_template.tf
index f574fcc..52f40f2 100644
--- a/workers_launch_template.tf

# Worker Groups using Launch Templates

resource "aws_autoscaling_group" "workers_launch_template" {
  count = var.create_eks ? local.worker_group_launch_template_count : 0
  name_prefix = join(
    "-",
    compact(
      [
        coalescelist(aws_eks_cluster.this[*].name, [""])[0],
        lookup(var.worker_groups_launch_template[count.index], "name", count.index),
        lookup(var.worker_groups_launch_template[count.index], "asg_recreate_on_change", local.workers_group_defaults["asg_recreate_on_change"]) ? random_pet.workers_launch_template[count.index].id : ""
      ]
    )
  )
  desired_capacity = lookup(
    var.worker_groups_launch_template[count.index],
    "asg_desired_capacity",
    local.workers_group_defaults["asg_desired_capacity"],
  )
  max_size = lookup(
    var.worker_groups_launch_template[count.index],
    "asg_max_size",
    local.workers_group_defaults["asg_max_size"],
  )
  min_size = lookup(
    var.worker_groups_launch_template[count.index],
    "asg_min_size",
    local.workers_group_defaults["asg_min_size"],
  )
  force_delete = lookup(
    var.worker_groups_launch_template[count.index],
    "asg_force_delete",
    local.workers_group_defaults["asg_force_delete"],
  )
  target_group_arns = lookup(
    var.worker_groups_launch_template[count.index],
    "target_group_arns",
    local.workers_group_defaults["target_group_arns"]
  )
  load_balancers = lookup(
    var.worker_groups_launch_template[count.index],
:460
+++ b/workers_launch_template.tf
@@ -453,6 +453,21 @@ resource "aws_launch_template" "workers_launch_template" {

   }

+  dynamic "block_device_mappings" {
+    for_each = lookup(var.worker_groups_launch_template[count.index], "additional_instance_store_volumes", local.workers_group_defaults["additional_instance_store_volumes"])
+    content {
+      device_name = block_device_mappings.value.block_device_name
+       {
+         virtual_name = lookup(
+           block_device_mappings.value,
+           "virtual_name",
+           local.workers_group_defaults["instance_store_virtual_name"],
+         )
+       }
+    }
+
+  }
+
   tag_specifications {
     resource_type = "volume"

Checklist

[X] CI tests are passing
[X] README.md has been updated after any changes to variables and outputs. See https://github.com/terraform-aws-modules/terraform-aws-eks/#doc-generation

This patch doesn't seem to have any effect for me. It still insists on creating an EBS volume using the default values. I think because there is a non-dynamic block_device_mappings block above the dynamic one you've added in your patch.
https://github.com/terraform-aws-modules/terraform-aws-eks/blob/50da792bc7e5a8a7a270f8b00f526fc9f961ab14/workers_launch_template.tf#L372

This patch doesn't seem to have any effect for me. It still insists on creating an EBS volume using the default values. I think because there is a non-dynamic block_device_mappings block above the dynamic one you've added in your patch.
https://github.com/terraform-aws-modules/terraform-aws-eks/blob/50da792bc7e5a8a7a270f8b00f526fc9f961ab14/workers_launch_template.tf#L372

@rossigee The additional non-dynamic block_device_mappings should not be an issue.  This existed before my patch and a non-dynamic plus a dynamic existed before my patch too.
To make sure I fully understand the issue, are you saying that this patch does not work because it does not change the existing behavior of creating the EKS worker node root volumes as EBS volumes?  If that is the case, this patch is not intended to modify that behavior.  As you correctly noted, that non-dynamic block does indeed limit this module to creating EKS clusters which use EBS volumes as their root disk.
In the case that this is just a configuration issue, here is an example of my worker-group config relevant section:
    - name: "spot-colossal"
      override_instance_types: ["x1.16xlarge"]
      spot_instance_pools: 3
      asg_max_size: 2
      asg_desired_capacity: 0
      asg_min_size: 0
      kubelet_extra_args: "--root-dir /mnt/ephemeral/kubernetes --node-labels=${join(",", ["spot=true","node.kubernetes.io/lifecycle=spot","lifecycle=Ec2Spot","example.cloud/namespace=example-staging","example.cloud/node=owned"])} --register-with-taints=spotInstance=true:NoSchedule"
      additional_instance_store_volumes:
        - block_device_name: "/dev/sdb"

I would like to see the possibility to change the bootstrap (similar to eksctl preBootstrapCommands) and mount and format instance storage when the pod is provisioned, so thus I can use it in Spark on Kubernetes:
https://aws.amazon.com/blogs/containers/optimizing-spark-performance-on-kubernetes/
      - IDX=1
      - for DEV in /dev/disk/by-id/nvme-Amazon_EC2_NVMe_Instance_Storage_*-ns-1; do  mkfs.xfs ${DEV};mkdir -p /local${IDX};echo ${DEV} /local${IDX} xfs defaults,noatime 1 2 >> /etc/fstab; IDX=$((${IDX} + 1)); done
      - mount -a

Or is there any other workaround how to provide these instance volumes to pods?

@rossigee The additional non-dynamic block_device_mappings should not be an issue. This existed before my patch and a non-dynamic plus a dynamic existed before my patch too.

@jwitko - my bad, i was completely misunderstanding how this works. Thanks for the explanation.

@jwitko Thank you for opening this PR. Can you please rebase your branch from master ?

@jwitko Thank you for opening this PR. Can you please rebase your branch from master ?

No problem!  Should be done now.

Thanks @jwitko for your contribution.

I would like to see the possibility to change the bootstrap (similar to eksctl preBootstrapCommands) and mount and format instance storage when the pod is provisioned, so thus I can use it in Spark on Kubernetes: https://aws.amazon.com/blogs/containers/optimizing-spark-performance-on-kubernetes/
      - IDX=1
      - for DEV in /dev/disk/by-id/nvme-Amazon_EC2_NVMe_Instance_Storage_*-ns-1; do  mkfs.xfs ${DEV};mkdir -p /local${IDX};echo ${DEV} /local${IDX} xfs defaults,noatime 1 2 >> /etc/fstab; IDX=$((${IDX} + 1)); done
      - mount -a

Or is there any other workaround how to provide these instance volumes to pods?

I'm trying to configure something similar, is there a way to do that with this module?

@josecsotomorales yes - see https://github.com/terraform-aws-modules/terraform-aws-eks/blob/master/examples/eks_managed_node_group/main.tf#L219-L229


Fix wildcard version NuGet downloads.

This PR fixes: #331

Writing some tests for this case now.

Fixing tests now, I also realized it was not a good test because as soon as a new version of any of the ones I wrote tests for would break.

I don't see a code change in here that handles .* versions differently. Can you explain what is fixed?

When debugging I realized it wasn't an issue with handling * versions, but it was an issue with the TypedManager which only gets used by NuGet. It was trying to enumerate through extracted files every time, even if we weren't extracting the .nupkg file. There might be more that I'm forgetting, so I'll go debug and make sure that was all.
Also I found the issue with the NuGetPackageActions where when extracting the file it was the extension of the file + .nupkg which is incorrect.

I don't see a code change in here that handles .* versions differently. Can you explain what is fixed?

When debugging I realized it wasn't an issue with handling * versions, but it was an issue with the TypedManager which only gets used by NuGet. It was trying to enumerate through extracted files every time, even if we weren't extracting the .nupkg file. There might be more that I'm forgetting, so I'll go debug and make sure that was all. Also I found the issue with the NuGetPackageActions where when extracting the file it was the extension of the file + .nupkg which is incorrect.

Just confirmed it yeah, the PackageDownloader constructor did successfully add every version to the PackageVersions = List<PackageURL> but the TypedManager.DownloadVersionAsync is where it was failing because we were trying to assert that we had extracted every single artifact we downloaded, even when we didn't specify that we wanted to extract the file we downloaded.


added port argument to api.connect

hi there,
i noticed server argument to api.connect function parses it and gives port number automatically. In order to use standard tcp/ip port double colon (::) needed as seperator
i mostly use this library through ssh-forward, thus port numbers generally in 10k space.
specifying port number as an argument simpler than host + "::" + port imho, also looks better
only downside is that when specified, port argument overrides port if server was given with display number or port

@pvtmert I see you 👍 my comment.  Should we close this PR or do you want to refine it to address the issue a different way?

yeah closing now, (totally forgot last time :) )


addition of Ain Shams University

@realpebbers I am sorry, the domain cis.asu.edu.eg is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Mesos support?

Describe the feature request
Can Istio run on Mesos/Marathon directly? Such as a framework, or as containers managed by Marathon.
Describe alternatives you've considered
Now Kubernetes can run on DC/OS, and Istio can run on K8s. But this means other services running on Mesos (out of K8s service) can not access Istio directly.

Istio does not currently have native support for Mesos.However, you can take advantage of our integration with Consul to run Istio Mesh on Mesos with it.For more details, refer to the Consul installation.

@stevenlee87 Thanks. I will have a try.


NTR: Peyer's patch lymphocyte

Note - please check that the term does not already exist (check OLS: https://www.ebi.ac.uk/ols/ontologies/cl)
Preferred term label:
Peyer's patch lymphocyte
Synonyms
Peyer's patch resident lymphocyte; small intestine Peyer's patch lymphocyte
Definition (free text, please give PubMed ID in format PMID:XXXXXX)
Lymphocytes in Peyer's patches have a major role in driving the immune response to antigens sampled from the intestinal lumen.  PMID: 23525039
These cells also play a role in regulating the formation of follicle-associated epithelium and M cells in Peyer's patches by converting intestitial enterocytes into M cells. PMID: 9252325
Parent cell type term (check the hierarchy here https://www.ebi.ac.uk/ols/ontologies/cl):
Lymphocyte: http://purl.obolibrary.org/obo/CL_0000542
What is the anatomical structure that the cell is a part of? Please check Uberon: https://www.ebi.ac.uk/ols/ontologies/uberon
Peyer's patch: http://purl.obolibrary.org/obo/UBERON_0001211
Your nano-attribution (ORCID)
https://orcid.org/0000-0003-4183-8865
Any additional notes or concerns
Have Peyer's patch B cell and small intestine Peyer's patch T cell as child terms

@ngeorgeebi
Lastly, the new term CL_0009014 (Peyer's patch lymphocyte) will also be in this week's CL release. Thank you.

@ngeorgeebi
Lastly, the new term CL_0009014 (Peyer's patch lymphocyte) will also be in this week's CL release. Thank you.


Query Cost Estimation with Dry Run

Dry run allows you to see how much data will be processed during your query

Really good to have BigQuery in VS Code, one of the only things missing for us is the ability to track the potential cost of the query (the dry run stats we would get in the GUI).

Good job with the driver, I agree it would be really useful to see potential cost of the query before running it

Hi, I'm trying to scope this effort:
src/ls/driver.ts:
-Add the dryRunQuery in the BigQueryDriver class
src/extension.ts:

Handle user triggering and output.
I'm thinking: https://code.visualstudio.com/api/ux-guidelines/status-bar
vscode.window.setStatusBarMessage(Dry run: ${result.bytesProcessed} bytes, 5000);

It will have to be executed using the command palette. At least in this first approach.

The status bar seems like a pretty sensible place for the output.
(It's always visible unlike for example the extension sidebar)

It will have to be executed using the command palette.
You could also make the status bar element clickable to execute it I think?

For example, before dry run the status bar could say ► Dry Run Query, and then change to Dry Run: 12.5GB when clicked.

have anyone working on it?
I'm interested in working on this issue, since the scope is clear enough and I really need this feature

No, you are welcome to it!

Currently I'm still trying on several alternatives. While I could add new method into the driver.ts, I haven't figured out how to call this method from outside driver.ts (as sqltools seems like loading the files in src/ls/* dynamically).
I have tried importing vscode package to use the messaging part of it, but found this error:
ERROR: Error loading plugin BigQuery Driver for SQLTools Plugin Request ls/RegisterPlugin failed with message: Cannot find module 'vscode'

can you point me to any other resources for the next step?

I'm afraid I haven't looked into this in detail, so I'm not clear on the best strategy.
I opened this issue on the main SQLtools repo, perhaps you could add some details and tag one of the maintainers?
https://github.com/mtxr/vscode-sqltools/issues/1191

Currently I'm still trying on several alternatives. While I could add new method into the driver.ts, I haven't figured out how to call this method from outside driver.ts (as sqltools seems like loading the files in src/ls/* dynamically).
I have tried importing vscode package to use the messaging part of it, but found this error:
ERROR: Error loading plugin BigQuery Driver for SQLTools Plugin Request ls/RegisterPlugin failed with message: Cannot find module 'vscode'

can you point me to any other resources for the next step?

Do you have your code available so I can pull it down and look into this error?


Typos and minor fixes for image-builder

Description of changes:
Minor fixes for image-builder.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cherrypick release-0.11

/lgtm

/approve

/lgtm


Error when running Linear normalizer

Basic information

My operating system: Windows 10 Enterprise
My MZmine version: MZmine 3.0.11-beta (portable)

What happened

Please describe what happened and how it differs from what you expected to happen.

I tried using the "Linear normalizer" module in MZmine 3 but am getting the following error message:

This happens both when applying the Linear normalizer to the aligned feature list aswell as to individual feature lists.
In MZmine 2.53, the same module works for me, with the same settings.

Please include all parameter values for the MZmine modules you used.

First I ran the following batch queue: pilot_positive_batch.zip  (zipped because I could not upload xml directly)
Then I ran the Linear normalizer (Feature list methods > Normalization > Linear normalizer), with the following parameters:

Please upload screenshots of any errors and your log/mzmine.log file.
Error message: see above.
Log file:  mzmine_0_0.log

Please upload an MZmine project file that demonstrates the problem you are reporting.
Please let me know if this is needed.

Thanks for the report, ill look into it.

You can check out the latest development build here: https://github.com/mzmine/mzmine3/releases/tag/Development-release
The issue should befixed. If it persists, please open a new issue.

Indeed it works now. Thanks a lot for fixing it so quickly!


 Any ideas what's up with these errors?
/Library/Python/2.7/site-packages/slybot/plugins/scrapely_annotations/builder.py:366: ScrapyDeprecationWarning: Attribute `_root` is deprecated, use `root` instead
  elems = [elem._root for elem in self.selector.css(selector)]
Traceback (most recent call last):
  File "/usr/local/bin/portia_porter", line 42, in <module>
    schemas, extractors, spiders = load_project_data(Storage(project_dir))
  File "/Library/Python/2.7/site-packages/portia2code/porter.py", line 76, in load_project_data
    Settings())
  File "/Library/Python/2.7/site-packages/slybot/spider.py", line 58, in __init__
    settings, spec, item_schemas, all_extractors)
  File "/Library/Python/2.7/site-packages/slybot/spider.py", line 226, in _configure_plugins
    self.logger)
  File "/Library/Python/2.7/site-packages/slybot/plugins/scrapely_annotations/annotations.py", line 89, in setup_bot
    self.extractors.append(SlybotIBLExtractor(list(group)))
  File "/Library/Python/2.7/site-packages/slybot/plugins/scrapely_annotations/extraction/extractors.py", line 61, in __init__
    for p, v in zip(parsed_templates, template_versions)
  File "/Library/Python/2.7/site-packages/slybot/plugins/scrapely_annotations/extraction/extractors.py", line 70, in build_extraction_tree
    basic_extractors = ContainerExtractor.apply(template, basic_extractors)
  File "/Library/Python/2.7/site-packages/slybot/plugins/scrapely_annotations/extraction/container_extractors.py", line 65, in apply
    extraction_tree = cls._build_extraction_tree(containers)
  File "/Library/Python/2.7/site-packages/slybot/plugins/scrapely_annotations/extraction/container_extractors.py", line 144, in _build_extraction_tree
    parent = containers[parent_id]
KeyError: u'2de2-4d1f-97ab#parent'

Not sure what to try. Tried different virtualenv, pip2 vs. pip3. Could use some help before I rabbit hole too much further.


add a init_ local database file

don't merge it yet
@JiejayLan you should take a look at this

OKOK.


nuxt generate --watch

I know that issue #52 exists, but I'm not sure if it's a good idea to add a comment there.
I stumbled over this because I think the nuxt generate --watch command is what I need. I'm creating a static site/blog using Nuxt. I create a page for every blog post using the config property generate.routes. (there I create a route for every .md file in another folder in the project) There I pass all data I need to the page's asyncData method via payload.
Now this works fine if I use nuxt generate. But if I just use nuxt (which is just way more useful while development) the payload parameter is undefined. As far as I understand it only get's passed in the generate process because that's the only time that generate.routes gets processed.
Now the obvious solution would be nuxt generate --watch ‒ if it was a thing. I figured that I can also fetch the blog post's content inside the asyncData method. But it feels like I'm doing the work twice. Also this makes me wonder what's the point of the payload parameter.
Since generating static pages for e. g. GitHub Pages is officially supported this doesn't feel like an edge case to me. Or what do you think? Are there other solutions to this problem?

Personally, I'd go with the following, strongly inspired by the payload example in the docs.
module.exports = {
  generate: {
    routes: function() {
      return require('./myData.js').posts.map(postData => ({
        route: '/posts/' + postData.id,
        payload: postData
      }))
    }
  }
}

// Now we can access the payload from /users/_id.vue like so:
export default {
  async asyncData({ params, error, payload }) {
    if (payload) return { post: payload }
    else return { post: require('./myData.js').posts[params.id] }
  }
}

But if your payload is truly local, you might not want to pass a payload at all, and just do this instead:
module.exports = {
  generate: {
    routes: function() {
      return require('./myData.js').posts.map(
        postData => '/posts/' + postData.id
      )
    }
  }
}

//  /posts/_id.vue:
export default {
  async asyncData({ params, error }) {
    return { post: require('./myData.js').posts[params.id] }
  }
}

hm... yes, I think the second solution good enough. Technically I just read and parse local markdown files.

To be honest, I am not sure it's a good solution now.
What would happen on clientside navigation?
Please post your current code in these two sections.

My source code is at https://github.com/alinnert/linnertmedia
It's a very basic setup, the project is at a very early stage in development. So, there's not much additional code aside from "hello world". I followed the setup introduction "Starting from scratch". Other than that there's a nuxt folder that contains my custom code to read and parse .md files. (I haven't found a proper name yet.)
The page file I'm fiddling around with is ~/pages/blog/_slug.vue.
I've changed my code to look almost like in your example. But running nuxt now throws an error that it can't resolve the dependency fs (which I use to read the content of .md files).
This error gets thrown because I execute an imported function that uses fs from inside asyncData().
This error, however, does not get thrown if I call that function from nuxt.config.js.
Also, it only complains about fs, but not about path. I don't really understand what's happening here.
I mean, I could use require() instead of fs.readFile() to read the content of a .md file. But this only allows me to read one specific file, not the content of a folder (fs.readdir) to generate e.g. a list of blog posts.

Yeah, it wouldn't work like this, you'd end up shipping things like gray-matter to the client.
Why don't you pre-process your posts into html fragments and expose them as an endpoint?

I'm not sure if I can follow you. What kind of endpoint? But this also sounds like it's getting a too complex setup for a simple static website.

I left you a PR: https://github.com/alinnert/linnertmedia/pull/2


bump ruff to 0.8 and relock poetry

can we get rid of darglint yet?

can we get rid of darglint yet?

I think we need these rules to be out of preview : https://docs.astral.sh/ruff/rules/#pydoclint-doc


Email validation

It would be cool to add email validation.
It would help with spam and also the possibility of someone using someone else's email.

Will the site admin receive a notification before or after the commenter has verified their email?
Things to think about in terms of the backend:

mode in database
Can site admins manually override not-yet-verified commenters?
Where do not-yet-verified comments show up in the admin?

Other aspects:

How long will verification be stored? At the moment, cookies have a lifespan of 15 minutes (max-age). Do we want to ask commenters to re-verify their emails after that cookie has expired in order for them to add a new comment?
Will the "token" they receive in their email be valid beyond the 15 minutes, allowing them to re-obtain a cookie that lets them edit any or just recent comments with their email?


Enable codegen for core::slice::<impl [T]>::split_first

Description of changes:
Removes core::slice::<impl [T]>::split_first from the list of functions to be skipped during codegen. This can be done since ProjectionElem::Subslice was partially implemented (for slices) some time ago.
Resolved issues:
Resolves #207
Call-outs:

Testing:

How is this change tested? Running existing regression.

Is this a refactor change? No.

Checklist

[x] Each commit message has a non-empty body, explaining why the change was made
[x] Methods or procedures are documented
[x] Regression or unit tests are included, or existing tests cover the modified code
[x] My PR is restricted to a single feature or bugfix

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

Does this resolve the issues seen with the regex-syntax and backtrace crates mentioned in #207?

@zhassan-aws that seems to be the case. Please check the updated I posted in #207.


Version 1.4.0 breaking change when using TRIVY_SEVERITY environment variable

I'm not sure if this is an issue with the plugin itself, or with the underlying version of trivy that it's using, but as of 2 hours ago when the 1.4.0 release went out, my builds are all failing now. I was setting the TRIVY_SEVERITY environment variable in my task like so:
  - task: trivy@1
    displayName: 'Fail build for critical vulnerabilities'
    inputs:
      version: 'latest'
      docker: false
      path: '$(System.DefaultWorkingDirectory)'
    env:
      TRIVY_SEVERITY: 'HIGH,CRITICAL'
      TRIVY_IGNOREFILE: '$(System.DefaultWorkingDirectory)/.trivyignore'

and it seems to be ignoring that and using the value passed in on the command line now (which defaults to 'CRITICAL,HIGH,MEDIUM,LOW,UNKNOWN'). Is there a way to get the old behavior back without going in and modifying all of my dozens of pipelines to pass in the values I want on the command line?

Hi @cjbush, this was fixed and a new release was cut: https://github.com/aquasecurity/trivy-azure-pipelines-task/releases/tag/v1.4.1

Much appreciated @giorod3! Thanks for all the hard work!


use application.android.startActivity instead of foregroundActivity

Related to this issue
You might want to change this line to use startActivity on this line to ensure that the Activity is not undefined for the reasons discussed in the linked issue.

I have the same problem


Check event no ads return?

Hi,
You can add function to check event no ads return?
public void onAdLoaded();
public void onAdFailedToLoad(int errorCode);
public void onAdOpened();
public void onAdClosed();
public void onAdLeftApplication();
Thanks!

Hi,
I'll investigate it and turn back to you with some comments, Ok?
KR!

I Clockbyte,
I'm waiting!
Thanks!

maybe we can provide that given the new NativeExpressAd class,
this way users can specify their own custom AdListeners.
what do you think?

@mourjan I guess you are right :+1: As for me the main prio is to get stable and well-documented admobadapter. Your idea is perfect to get more flexibility.

@hoanganh2020 Hi,
You can see https://github.com/clockbyte/admobadapter/wiki/Cookbook#in-case-you-want-to-wrap-your-ad-blocks-into-a-viewgroup-cardview-framelayout-etc
At the moment NativeExpressAdViewHolder#isFailedToLoad() is the only way to get some info on the ad block. Hope it will help you. I suppose we'll extend and improve this class further with some events and other extra-states for ads.


BIM notebook not found on website

https://mobook.github.io/MO-book/notebooks/02/bim.html
This notebook doesn't work, but it is visible in the book outline. Is this intended?

Solved with a rebuild of the book.


Unexpected search behaviour in test

Ruby 2.4.0/Rails 5.1.4/elasticsearch-model 0.1.9/Elasticsearch 5.3.0
I'm using elasticsearch-model, and manually indexing in my test.
My test looks a little like this:
 test 'it can be searched' do
    VCR.use_cassette('this_test') do
      model_instance = ModelName.create(value: 'reddit.com')
      ModelName.import(force: true)

      result = ModelName.search('reddit.com').records.first

      assert_equal model_instance, result
    end
  end

however, result comes back nil. If I put a debugger in after the assignment to result, and repeat result = ModelName.search_with_elasticsearch('reddit.com').records.first, the record is assigned as expected (!?).
If I add ModelName.__elasticsearch__.refresh_index! after the call to import, the record is assigned as expected.
However, neither approach works when run as part of the full test suite, result is still set to nil in this context. When the test is run in isolation, either approach works.
I have tried disabling VCR & webmock and this has no affect. The VCR response recorded is as expected, the GET corresponding to the search is present and contains the correct result.

Hi, not sure about how VCR interferes with the test in general, if at all, but it is definitely needed to call the "Refresh" API in order to propagate the changes from your import — force first deletes the index — to the search.
It is strange if it works in isolation, but not as part of the whole test suite, since even if asetup somewhere would delete / re-create the index, the import and refresh_index! calls should make the change visible to the subsequent search...

This turned out to be my mistake, the VCR cassette was recorded with a specific ID, but of course the model created in test would not necessarily match, so whilst results were present, records couldn't be rehydrated from them.


Fix for issues #1 and #2

These are small changes to the README file. Thought I'd do it formally so you can see how the issues system and pulls work.
Jim

Changes to the README


add unit tests for func ConvertAddressToCidr, isProxyVersion

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: zackbutcher
If they are not already assigned, you can assign the PR to them by writing /assign @zackbutcher in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/pkg/networking/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @islinwb. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

@islinwb: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@islinwb: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test e2e-mixer-no_auth

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@islinwb: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test e2e-mixer-no_auth

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-dashboard.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test e2e-dashboard

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@islinwb: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test e2e-mixer-no_auth

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-dashboard.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test e2e-dashboard

prow/e2e-bookInfoTests-v1alpha3.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@islinwb: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test e2e-mixer-no_auth

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-dashboard.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test e2e-dashboard

prow/e2e-bookInfoTests-v1alpha3.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-pilot-multicluster-e2e.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@islinwb: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test e2e-mixer-no_auth

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-dashboard.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test e2e-dashboard

prow/e2e-bookInfoTests-v1alpha3.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-pilot-multicluster-e2e.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test istio-pilot-multicluster-e2e

prow/e2e-simpleTests.sh
f60373dd1a4d3280cf7324bd554295e6411f93d6
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: rshriram
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/pkg/networking/OWNERS [rshriram]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


cdl_runner::db_client: insert error

The biggest bottlenecks of the loader are really:

The fact we have to iterate through every record in order to determine if we need to delete a specific value.
The fact we don't batch together DELETEs/INSERTs at all.
The fact we're processing each table one at a time, without threading out.

All of these facts do slow it down considerably, and we've in the past avoided fixing these bottlenecks because we don't want people to rely on this full time working with Canvas Data. It's meant to be an example application showing users how to interact with the API, and how they could do things like "avoid historical refreshes".
We could one day down the road, fix these features, and turn the CDL into a fully supported solution, but that would have to be a product decision.
Originally posted by @SecurityInsanity in https://github.com/instructure/canvas-data-loader/issues/6#issuecomment-427165092

Hey @latifatbraimah ,
As mentioned in the issue you're posting this from. This is a known bottleneck, and not one we plan on fixing for the reasons outlined in this comment. I'm unsure as to why exactly this was moved to it's own issue.
However, if you have any more questions I recommend posting them on the original issue (which is still open).
Thanks,


Roll back #842 to fix IVF_PQ and BIN_IVF_FLAT recall degrade

Issue: #858 #859

Rolled back AVX for normalization, using SSE where possible

/lgtm


Can't build documentation with PYTHONPATH based module discovery from different directory than repository itself

Describe the bug
I think it relates more to mkdocstrings setup and path handling then pytkdocs, but please let me know if issue should be created there.
mkdocs can be used with --config-file switch to provide direct path to a config. Settings in mkdocs.yml like docs_dir are relative to the config path, not current working directory.
It seems different for mkdocstrings when one use PATH based module finding:
plugins:
  - mkdocstrings:
      handlers:
        python:
          setup_commands:
            - "import sys"
            - "sys.path.append('../')"
          rendering:
            show_category_heading: true
            show_root_toc_entry: false

That piece of code is not run relative to a config placement. Build succeeds when run inside repository, fails when run in any different area, it seems.
Expected behavior
Either an explicit way in config to add paths to PYTHONPATH relative to config itself, like:
plugins:
  - mkdocstrings:
      handlers:
        python:
          python_paths:
            - ..
          rendering:
            show_category_heading: true
            show_root_toc_entry: false

Or guarantee that dynamic setup_commands are run in directory where config is placed. I am afraid it might break some behavior if someone expects that.
Screenshots
ERROR    -  mkdocstrings.extension: Importing '<some module>' failed, possible causes are:
            - an exception happened while importing
            - an element in the path does not exist

Information (please complete the following information):

mkdocstrings version: 0.15.2

Indeed, pytkdocs is executed in a Python subprocess in the current working directory. Maybe you could export PYTHONPATH yourself in your shell? PYTHONPATH=path/to/sources mkdocs ...
And if you cannot change PYTHONPATH in the environment mkdocs runs into, but can change other env vars:
plugins:
  - mkdocstrings:
      handlers:
        python:
          setup_commands:
            - "import os, sys"
            - "sys.path.append(os.environ['PACKAGE_PARENT_DIR'])"

Otherwise yes, we would need to change the directory where pytkdocs is executed.

I build docs for multiple projects, without projects knowing I do that, let's say. So I can't guess what path is expected to be added to PYTHONPATH - project owners knows that and can configure it.
Currently the way how config is written indirectly creates requirement for a directory where mkdocs have to be run. I believe anchoring it to config parent directory would clear that ambiguity. But it would impact every existing configuration, where users got used to building docs not in repo root, right?
I think Sphinx doesn't have this problem, as it uses python script as a config - that allows me to use a current file directory to build relative paths.

So I'll finally be working on this :hot_face:
Here's the summary of what I'll do:

add a python.paths config option: a list of paths or none
the option will default to the mkdocs.yml's parent folder
its items will be:

inserted in front of sys.path for the legacy handler (pytkdocs)
used as search_paths option for the new handler (Griffe)

each path will be used as is when absolute, or relative to mkdocs.yml's parent folder when not
if the path is current working directory, skip

Examples:

mkdocs.yml in root, package in root
root/
├── mkdocs.yml
└── package/

plugins:
- mkdocstrings:
    handlers:
      python:
        paths: [.]  # actually not needed, default

mkdocs.yml in root, package in subfolder
root/
├── mkdocs.yml
└── src/
    └── package/

plugins:
- mkdocstrings"
    handlers:
      python:
        paths: [src]

mkdocs.yml in subfolder, package in root
root/
├── docs/
│   └── mkdocs.yml
└── package/

plugins:
- mkdocstrings"
    handlers:
      python:
        paths: [..]

mkdocs.yml in subfolder, package in subfolder
root/
├── docs/
│   └── mkdocs.yml
└── src/
    └── package/

plugins:
- mkdocstrings"
    handlers:
      python:
        paths: [../src]

Except for the first case, defining python.paths will be strongly recommended, even if the package's parent folder is added to python path automatically when running mkdocs in the project root, as only then would the -f option of mkdocs work properly.


[pulsar-broker] Fix: bookie-isolation placement-policy was not configuring rackaware policy

Motivation
BookieIsolation placement policy doesn't consider rackaware policy because it doesn't configure dns_resolver_class and because of that it doesn't select bookies by considering racks and it logs below message
21:48:09.331 [bookkeeper-ml-workers-OrderedExecutor-3-0] WARN  org.apache.bookkeeper.client.BookieWatcherImpl - New ensemble: [<IP_ADDRESS>:3181, <IP_ADDRESS>:3181] is not adhering to Placement Policy. quarantinedBookies: []

Modification
Configure ZkBookieRackAffinityMapping with BookieIsolation placement policy to apply rackaware policy.

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks


Preserve Source RPMs of build dependencies

Once #146 is merged, we should ensure that we preserve the Source RPMs of anything that is built from source, whether as the final target or as a dependency .

This is something that should be included with #150.

Sorry, I meant to link to the multi-spec PR(#146 ) not #150.

Closing since we don't currently plan to implement multi-spec


Confuse about the frame rate in your dataset

Thanks for your meaningful work and code!
I have a question about the frame rate in your dataset.
In the appendix of the paper,you show that the frame rate of the Panda-Video is 30 or 12 fps.Is your baseline result of MOT also obtained in that frame rate?
Since you only provide the 2 fps version of Panda-Video,How can I download the 30 or 12 fps version and test on it?
I would appreciate your feedback.

Yes, our baseline result of MOT also obtained in 30/12 frame rate.
The data volume is too large for 30 or 12 fps version (800GB in total), so it's very inconvenient to upload and download. If your research needs high frame rate video data, please leave a message. We can provide the data of one scene as a test first.


At some point the downloads die for no reason and wont download past a certain point

any idea how to fix this? i log back in but it still wont download more than like 12

any idea how to fix this? i log back in but it still wont download more than like 12

I am also facing the same problem
I was previously downloading the complete public profiles without me getting this error now I must log in to download the files This issue should be fixed in the next update

The output message says it all: you have to login to proceed.
Instagram limited anonymous access. Same thing as in #1514 or #1067.

The output message says it all: you have to login to proceed. Instagram limited anonymous access. Same thing as in #1514 or #1067.

Public profiles do not need to be logged in
I don't know why you don't want to allow public profiles to be uploaded without logging in
Login causes account ban Use login only with private accounts
instaloader is a failed tool
What is the use of this tool?
I will look for others

instaloader is a failed tool
What is the use of this tool?

No one is making you use it, feel free to find another and report back if you find a better alternative that has all the features of Instaloader. Instagram API limits what Instaloader can download anonymously. Try going on a browser without logging in and clicking more than 12 posts, it will redirect you to login.

thing is when trying to login it never lets me. it always says some shit like “must be in a text file” and i tried that but it didn’t work. idk how to login properly

idk how to login properly

Did you try 'instaloader -l ' ? Run it on the promt and it will ask u for your pw.
Then it will create a session which you can the use in your script with:
s = instaloader.Instaloader()
s.load_session_from_file(USERNAME)  # Load the created Session


Dev avatar targeral

avatar组件

how do u test ur component? no example? and npm run site also cant display correctly.

sorry, forget update router


Kotlin: 84. Largest Rectangle in Histogram

File(s) Modified: 84-Largest-Rectangle-In-Histogram.kt.
Language(s) Used: Kotlin.
Submission URL: _https://leetcode.com/submissions/detail/780602472/_

Thanks, @MaratKhakim!


Add metrics dapr config to filter high cardinality metrics

In what area(s)?
/area runtime
Describe the feature
Dapr runtime has several metrics categories. This is important when debugging dapr or customer problem. but some metrics, such as http, grpc, and actors, can create high cardinality label spaces which make aggregation complicaed. It leads to perf problem later. To address the problem, this proposes adding whitelist metrics option to dapr config like tracing config.

We filed the issue to configure the metrics group before, but we cut it because it was nice-to-have feature about that time.

Proposed design
By default, dapr will emit the minimum metrics. to enable grpc, http, and actor, user can add metric config and set the list of metrics that he or she wants to emit.
apiVersion: dapr.io/v1alpha1
kind: Configuration
metadata:
  name: tracing
  namespace: default
spec:
  tracing:
    samplingRate: "1"
  metric:
    included:
       - default
       - actor
       - grpc
       - http

Metrics group
TBD.
Consequence:

Reduced aggregation burden in opencensus and prometheus.
Improve the perf in runtime.

@yaron2 @amanbha @orizohar @msfussell @shalabhms Please review this.

/lgtm. Yes, we need to provide a way to specifically list out the group of metrics.

@yaron2 @amanbha @orizohar @msfussell @yolember Please review this proposal. I would like to get this to reduce the impact of performance along with this proposal - https://github.com/dapr/dapr/issues/1643


Mass Update not logging

I noticed there was mention of this, but someone marked it as solved.  I'm still having an issue, or is this something that is not possible at this time?
Model::where('user_id', $oldID)->update(['user_id' => $newID]);
This does not trigger any logging activity.  I haven't tried deleting yet.

That's because when you use update it uses Laravel's query builder. The logger can only listen to events sent out by Eloquent models.
If you want to use Eloquent instead, you'll have to retrieve the models, and update them in some sort of loop one by one.


BAG3

https://www.mdpi.com/2073-4425/13/2/363

This publication only provides data on one case and there is no larger cohort publication I can find. Let's not include this now.


Update talk title Michelle Itano

Updated talk title.

Looks good on https://snoopycrimecop.github.io/www.openmicroscopy.org/events/ome-community-meeting-2020/lightning-talks/
Merging


[Solution]Deb app install but don't open in window correctly or not at all.

Describe the bug
Created Deb Packages with yarn tauri build don't open in linux desktop After install in app-center. Appimage work no problem.
Platform and Versions (please complete the following information):

yarn run v1.22.4
[tauri]: running info

Operating System - Linux(5.4.0-42-generic) - linux/x64

Node.js environment
  Node.js - 14.6.0
  tauri.js - 0.11.1

Rust environment
  rustc - 1.45.0
  cargo - 1.45.0
  tauri-bundler - 0.9.1

Global packages
  NPM - 6.14.6
  yarn - 1.22.4

App
  tauri.rs - 0.9.1
  mode - embedded-server
  build-type - bundle
  CSP - default-src blob: data: filesystem: ws: http: https: 'unsafe-eval' 'unsafe-inline'
  distDir - ../build
  devPath - http://localhost:3000
Done in 3.53s.

Solution
Add to the tauri.conf.json bundle default parameters shortDescription and longDescription fe.
"bundle": {
      "shortDescription": "Tauri App",
      "longDescription": "Tauri app with rust",

This bug does not appear to be present any more. If you can still reproduce it, please let me know and I'll reopen this issue.


Update message

It would be good to update the message payload and delay.
Example use case is change the copy and time a user would receive a notification based on an external event.
API could be something like
updateMessage(messageId, payload, delay) and it would optionally update the payload or delay based on what is set.

Simply delete and resend the message. Updating a message payload is a very bad id when the messageid stays the same.
Once you change a payload it is a new message.


Report servers where an SSH KeyPair is assigned

One should only get access to a server through 'even'. That implies that you should never use KeyPairs from EC2. Every usage of a KeyPair is wrong and needs a report.

+1


Clarification regarding the response of the API /v1/inform-about-preceding-release

This API “/v1/inform-about-preceding-release”  is to get the application-name and release-number of the old release of an application.
We know that , in the initial configuration file of all the application , the application-name of the oldRelease is “OldRelease”.
If the registered application is the very first version and don’t have a previous version ,
then,
the application-name of the old-release is OldRelease.
So, the response of the /v1/inform-about-preceding-release will be something like this ,
{
  "application-name": "OldRelease",
  "release-number": "1.0.1"
}

Kindly please let us know shall we produce a result like this ?
(or)
we shall not send a response code!=200 meaning that there is no previous version of this application is available.
Also we shall look into a related issue in RegistryOffice  https://github.com/openBackhaul/RegistryOffice/issues/527

Yes, please produce a result like this:
{
  "application-name": "OldRelease",
  "release-number": "1.0.1"
}

The response code shall relate to the communication and not the content of the message.

Thank you for the clarification. Closing the issue.


not possible to render (Error during JPEG image loading)

Link to PDF file (or attach file here): OBW-OVK.pdf
Configuration:

Webbrowser and its version: Chrome 59.0.3071.115

Operatingsystem and its version: Microsoft Windows 7 Enterprise SP1

PDF.js version: 1.8.516

Is an extension: no

Steps to reproduce the problem:

Load attached pdf-file

What is the expected behavior? (add screenshot)
What went wrong? (add screenshot)
Pdf has been rendered as blank pages. Error trace:
Error: Invalid XRef stream header
util.js:296     at error (https://mozilla.github.io/pdf.js/build/pdf.worker.js:355:17)
at XRef_readXRef [as readXRef] (https://mozilla.github.io/pdf.js/build/pdf.worker.js:21322:29)
at XRef_parse [as parse] (https://mozilla.github.io/pdf.js/build/pdf.worker.js:20992:28)
at PDFDocument_setup [as setup] (https://mozilla.github.io/pdf.js/build/pdf.worker.js:29900:17)
at PDFDocument_parse [as parse] (https://mozilla.github.io/pdf.js/build/pdf.worker.js:29789:12)
at _util.Util.inherit.ensure (https://mozilla.github.io/pdf.js/build/pdf.worker.js:39550:28)
at Promise ()
at LocalPdfManager_ensure [as ensure] (https://mozilla.github.io/pdf.js/build/pdf.worker.js:39545:14)
at LocalPdfManager.BasePdfManager_ensureDoc [as ensureDoc] (https://mozilla.github.io/pdf.js/build/pdf.worker.js:39497:19)
util.js:282 Warning: Indexing all PDF objects
app.js:1056 PDF f5a022ce9c1cfd977eedadde6a39bd36 [1.2 - / -] (PDF.js: 1.8.516)
util.js:282 Warning: Error during JPEG image loading
util.js:282 Warning: Dependent image isn't ready yet
util.js:282 Warning: Error during JPEG image loading
util.js:282 Warning: Dependent image isn't ready yet
Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

TL;DR: The embedded image has been corrupted, its height value does not represent its actual height.

The referenced file (https://github.com/mozilla/pdf.js/files/1125123/OBW-OVK.pdf) contains a JPEG stream, with the following dictionary:
1 0 obj
<<
/Type /XObject
/Subtype /Image
/Name /Im0
/Width 2480
/Height 3473
/BitsPerComponent 8
/ColorSpace /DeviceGray
/Filter /DCTDecode
/Length 1080464
>>
stream
...
endstream
endobj

However, the referenced image's height is much larger than specified (65535 > 3473). The browser is unable to load the image, and hence the output PDF is blank. Note that Chromium's default PDF viewer is not able to render the PDF either.
I extracted image.jpeg, e.g. by opening Chrome's devtools, viewing the network tab and seeing the request to a blob:-URL (the image). Then, to download it, run the following from the JavaScript console in the devtools of the same tab:
var a = document.createElement('a');
a.download = 'image.jpeg';
// Replace the following with the blob:-URL that is displayed in the devtools
a.href = 'blob:http://localhost:8888/b437bc55-2b13-4401-b526-a248111ad8d4';
a.click();

Now, the analysis of the image shows that the height is 65535:
$ exiftool image.jpeg
ExifTool Version Number         : 10.55
File Name                       : image.jpeg
Directory                       : .
File Size                       : 472 kB
...
X Resolution                    : 300
Y Resolution                    : 300
Image Width                     : 2480
Image Height                    : 65535
Encoding Process                : Baseline DCT, Huffman coding
Bits Per Sample                 : 8
Color Components                : 1
Image Size                      : 2480x65535
Megapixels                      : 162.5

When I open the JPEG file in a hex editor and change the faulty height value FFFF (=65535) to 0D91 (=3473), then the image can be displayed just fine.
To find out which bytes I needed to modify, I read:

http://vip.sugovica.hu/Sardi/kepnezo/JPEG File Layout and Format.htm
https://en.wikipedia.org/wiki/JPEG#Syntax_and_structure
http://www.fastgraph.com/help/jpeg_header_format.html
https://stackoverflow.com/questions/18264357/how-to-get-the-width-height-of-jpeg-file-without-using-library

The PDF file was blank too in Chromium's PDF viewer and Okular, but shown just fine in Adobe's PDF reader.
A patch for this issue is available at #8628. I am however not 100% sure whether this broken PDF should be supported by PDF.js. Where did you get this PDF file?

I am supporting an application, that fetches e-mails from a server, convert all attachments to multi-tiff, and provide this converted files for further processing. This PDF is coming from somebody unknown to me! I am always confronted with the fact, that customers asks "Adobe Reader can render this file. Why is your program not able to process it" Anyway, thanks for providing this patch!

Thanks for the info. Let's try to land the patch. None of the existing tests fail, so it should be fine to land it.


Accessing current tab properties

Is there any way to access the current tab to update the name to the current folder? My Column View utilizes multiple instances of an IShellPage (ColumnShellPage.xaml) with their own instance of the FileSystemViewModel inside the root ModernShellPage.

@TyJOrtiz Did we answer your question?

Yeah


Several caching improvements:

Populate the cache with the latest object version when possible.
Add negative caching for getting object by id.
Use latest version cache for satisfying dynamic field reads

LGTM! Thanks for adding all the detailed comments!
I wonder if it also worth creating some kind of cache invariant checker that runs periodically in simtest to catch issues. Some invariants may be that it could compare objects in dirty/cache/db and in object_by_id_cache, to see if they actually match.

adding a debug check

LGTM! Thanks for adding all the detailed comments!
I wonder if it also worth creating some kind of cache invariant checker that runs periodically in simtest to catch issues. Some invariants may be that it could compare objects in dirty/cache/db and in object_by_id_cache, to see if they actually match.

adding a debug check

LGTM!


i don't really know where is the problem

i made this function it should be working it checks if a shared preference has ar value and it should enter this function, it does enter the function but it some how exists in "to" part after checking it using the breakpoints

thanks in advance

The thing is Google translation API uses pay at scale for translations, it's not free.
When first I added this method I thought people will donate so that we could use more translations, but that did not happen.
Google limited usage for us now, so I removed this method in the newer versions of localize_and_translate and you will not get any further translations.

@msayed-net oh ok, so what is the new function ?

I'm studying adding all possible translation methods with user-based API-Keys, so that every user become responsible for his usage

Please check versions 3.x.x which applies mentioned method


Getting wrong project key

Using latest version (18) of this oauth2.0 script in Google Sheets for a Fitbit project. No longer gets the correct project key.
Looking at:
https://github.com/googlesamples/apps-script-oauth2/commit/5b7cf96137c1b2abdb4532f07cdc07b9fded4bec
Is this working as expected for others? Authorization keeps failing and I'm seeing a different project key than expected.
EDIT:
When I run Logger.log(ScriptApp.getProjectKey()); inside my own project, it logs the project key that I see in the error, but that doesn't match the project key I see in my Project Properties.

Do you have a code sample to repeat the issue?

Logger.log(ScriptApp.getProjectKey());
Compare to project key in File > Project Properties.

@simonbromberg , oh! I see now. The key in Project Properties doesn't look correct. It's too long.

Hm, maybe a bug.

Ah, looks like you've already submitted it. Thanks
https://code.google.com/p/google-apps-script-issues/issues/detail?id=6098&q=Type%3DDefect&colspec=Stars Opened ID Type Status Summary Component Owner

Added a method and instructions to work around the issue. Once fixed in Apps Script, will back out the instructions.


Update faq.yml

updating the subsequent-sign-in.png screenshot.

#sign-off

@Gargi-Sinha - These changes need to be submitted via the private repo, not the public repo. I'll close this PR now.


Added all platform support for injected Trust Wallet

…at there is a Trust Wallet browser extension.
Description
I'm on the Trust team, and we have a Chrome browser extension released: https://chrome.google.com/webstore/detail/trust-wallet/egjidjbpglichdcondbcbdnbeeppgdph. Changing the platform param for Trust's injected wallet in Blocknative to be 'all' instead of 'mobile' so TW extension can be connected via blocknative-powered dApps.
Checklist

[x] The version field in package.json is incremented following semantic versioning
[x] The box that allows repo maintainers to update this PR is checked
[x] I tested locally to make sure this feature/fix works
[x] I have run yarn check & yarn build to confirm there are not any associated errors
[x] This PR passes the Circle CI checks

Closing in favor of https://github.com/blocknative/web3-onboard/pull/1463


Unable to import with parcel or webpack bundlers

Hello,
I tried importing the package, after installing it from npm, with parcel and webpack but both didn't work.
Can you please clarify in the docs how we should proceed to make it work.
Thanks.

I forgot to mention, but I'm getting a ReferenceError: kontra is not defined.
The thing is that this is happening even when I import the kontra.js file directly from a script tag.

How are you brining in the library through a script tag? I can understand it not working with webpack since it's not written in es6 module syntax (maybe that should be my next feature).
I ran npm install kontra, and then loaded it in an HTML file without problems.
<!DOCTYPE html>
<html>
<head>
  <title></title>
</head>
<body>
  <script src="node_modules/kontra/kontra.js"></script>
  <canvas width="600" height="200"></canvas>
  <script>kontra.init();</script>
</body>
</html>

I managed to make it work with your example.
But if i manually download the kontra.js file from your site, and try to import it in the index.html file in a PARCEL project, it doesn't work. This looks like an issue with how Parcel loads the scripts and tries to bundle them.
I really would like to be able to use both together, if you have any idea on how to make it work, please let me know.
Thanks again.

I am unfamiliar with parcel, does it work like webpack and require es6 module syntax to bundle imports?

Yes. But I now webpack has a way to import non-modules but I'm not too familiar with Parcel either. I just wanted to try it because it's very simple and because of this boilerplate: https://github.com/mtmckenna/js13kgames-parcel-starter .

Kontra was originally written to be a loaded globally through a script tag, so it doesn't use any type of module loader syntax (CommonJS, Import, etc.). I'm not sure I have enough time to change it to use es6 module syntax before the jam starts in 12 days, but I can add it as a todo for next year.

That's cool. It would be cool if this gets done in the future.
I'll keep an eye on it. Thanks.

I wanted to give parcel + kontra a try also

FYI, just modifying this.kontra by kontra in src/core.js seems to have fixed my error, but I'm still early in testing.

Seems to work. I can make a PR if you have automated testing.

Glad you got it to work. PR's are always welcome :)

@andre-lima v4.0.1 should let you load Parcle thanks to @niicojs. Give it a try and let me know if it works for you.

example on how I use it

Great. I'll test it once I get home.
Although, will it work with the npm package? It would be a lot more seamleas if we can just import kontra from 'kontra';

I'm using it from npm.
Might work directly with import 'kontra' but it's usually better to only import the module you are going to use in order to save some precious bytes.

Yep. It works importing with only import 'kontra'.
Thanks, guys.


improvement Add Backend tag location check

Proposed changes
Issue Number: close #22671

Add Backend tag location check.
Avoid user set a bad backend tag, cause create table and dynamic partitions failed.
For example, the default value for all backends tag is default, When setting the replication_allocation of a table, user use the following command: ALTER TABLE example_db.mysql_table SET ("replication_allocation" = "tag.location.tag1: 1");, it can set success, but tag1 is not exist, cause dynamic partition can't create.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.55 seconds
stream load tsv:          511 seconds loaded<PHONE_NUMBER>9 Bytes, about 139 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.2 seconds inserted 10000000 Rows, about 342K ops/s
storage size:<PHONE_NUMBER>4 Bytes

run buildall

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.31 seconds
stream load tsv:          512 seconds loaded<PHONE_NUMBER>9 Bytes, about 139 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          30 seconds loaded 861443392 Bytes, about 27 MB/s
insert into select:          29.1 seconds inserted 10000000 Rows, about 343K ops/s
storage size:<PHONE_NUMBER>2 Bytes


New: added method FailWithError.

This method allows you to fail a suite whenever you want, for instance you could fail a suite of tests in a hook!
Not sure if it's the best way to do that, I'm listening if you have a better approach!

Current coverage is 40.09% (diff: 0.00%)

Merging #62 into master will decrease coverage by 0.61%

@@             master        #62   diff @@
==========================================
  Files            14         14
  Lines          1621       1646    +25
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            660        660
- Misses          907        925    +18
- Partials         54         61     +7

Powered by Codecov. Last update 5a471a7...a7c6201

I will leave it hanging, if more than a few users will express a need for such a feature, then it could be implemented, but differently - by returning an ErrFatal in a step, instead of having a method in Suite.
the suite provides you with an option --stop-on-failure that should be enough, it also allows you to run a single scenario in case if you need to rerun it. You do not have to run the whole suite. this method confuses, user might think that is a preferred way. Please do not, make pull requests just for the sake of it. Any additional public method just adds garbage and complexity to understand the overall interface available to an user. I would rather look how to throw something away instead of adding.

By returning ErrFatal or using the option --stop-on-failure in one of the steps will not fixt the purpose of this PR.
I have proposed this PR to have the possibility to make fail a test from a hook (use case, you check if the env is clean at the end of the test, if not the test is a failure, you could panic but you don't want that). It's not something new for the user of cucumber as it exists exactly the same one in ruby-cucumber.
No comment about making a PR or not, be glad that some people wants to contribute, and maybe if someone needs something, someone else will need it as well. Purpose of the PR

I do not agree with such feature in general, because this is a test suite, not a production server, if something is terribly wrong with a test context, then you should fail fast and dump this context in order to fix your tests for good, if you need such a feature - most probably that means you have an already too complex suite or maybe you have false positive tests (which is wrong in the first place). The point is, that your test suite is not an application, it is meant to show you whether your feature scenarios pass or fail. if they are wrongly implemented, then implement them well, that is the end of the story. Cucumber ruby, has many things, which I would not like to have in godog.  How long do your tests run? If you use database, use transactions and roll them back on each scenario. If your suite is running longer than a minute, refactor.

We use godog for functional end-to-end testing, where our QA guys don't know how the system is working internally and write story for the product.
For the database story, they don't have access to it, as they have the product as a black box, only the public API are exposed.
We would like to add this kind of test in the hook to make sure that the tester does not forget to clean everything (because it often happens unfortunately...). We know that it's not a good practice in terms of pure testing, but it helps a lot to control what is going on in the test. And we make sure to have standalone tests at the end.

I can't add this in general because there is a concurrency option, which runs not only one suite in parallel. If I add a similar feature, I have to remove other, which I think is better.
I do not think it is wise to make end to end tests this way, in general you are making your application to support black box tests, even though, in most cases data should not be removed through public api endpoints.
I would recommend to use a fork or a before scenario hook to always clean the state before any scenario, without even trying to assert.


Git clone fails with Recv failure: Connection reset by peer

Windows Version
Microsoft Windows [Version 10.0.19044.2965]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Debian 12 and Ubuntu 20.04
Other Software
git version 2.39.2
Repro Steps
Just try to clone a codeberg repo
git clone https://codeberg.org/joostkremers/visual-fill-column.git
Expected Behavior
The repo gets cloned
Actual Behavior
 GIT_CURL_VERBOSE=1 git clone https://codeberg.org/joostkremers/visual-fill-column.git --verbose
Cloning into 'visual-fill-column'...
21:27:28.721266 http.c:699              == Info: Couldn't find host codeberg.org in the (nil) file; using defaults
21:27:28.834167 http.c:699              == Info:   Trying <IP_ADDRESS>:443...
21:27:28.963775 http.c:699              == Info: Connected to codeberg.org (<IP_ADDRESS>) port 443 (#0)
21:27:29.081604 http.c:699              == Info: found 420 certificates in /etc/ssl/certs
21:27:29.081677 http.c:699              == Info: GnuTLS ciphers: NORMAL:-ARCFOUR-128:-CTYPE-ALL:+CTYPE-X509:-VERS-SSL3.0
21:27:29.081706 http.c:699              == Info: ALPN: offers h2,http/1.1
21:27:29.090009 http.c:699              == Info: Recv failure: Connection reset by peer
21:27:29.090114 http.c:699              == Info: gnutls_handshake() failed: Error in the pull function.
21:27:29.090143 http.c:699              == Info: Closing connection 0
21:27:29.090167 http.c:699              == Info: Send failure: Broken pipe
fatal: unable to access 'https://codeberg.org/joostkremers/visual-fill-column.git/': Recv failure: Connection reset by peer

Diagnostic Logs
No response

I have set the MTU values for both my WiFi card and WSL to be the same, it's not the issue. I've also tried to update my network drivers, to no avail. And tried to disable the generation of /etc/resolv.conf and manually setting it to <IP_ADDRESS>, but that didn't change as well. I made sure that all the necessary ssl libraries are installed.

I can not reproduce this. /logs

I can not reproduce this. /logs

I shutdown WSL, ran the script, started WSL, attempted the same git clone, and saved the logs
WslLogs-2023-06-14_11-40-27.zip