SimpleHealthKit - API Sender Onboarding

What You Need to Know

[Yes, signed on 10/14/2022] Terms of Service agreement signed

[Originally scheduled 10/24 but cancelled due to staffing issues]

[x] Kickoff meeting completed 11/3/2022

[x] Company’s intake form: [paste info below this]

Name: Ramtej Pinisetty

Job title: Program Manager

Organization: Simple HealthKit Inc.

Email address<EMAIL_ADDRESS><EMAIL_ADDRESS>

Phone number: +1<PHONE_NUMBER>

Organization that referred me to ReportStream:

Public health department(s) I report to: California and Louisiana

Current reporting method(s): HL7, CSV

To Do

[ ] create individual tickets for this epic

[x] RS team member who originates this ticket: complete the following when creating this Issue.

[ ] RS team member who originates this ticket: add any dependencies, as needed

@BerniXiongA6 to reschedule sender onboarding kickoff -- will update epic once the kickoff meeting is confirmed

Email from BX to sender - updated on 10/31/2022:

Hi Ramtej,
Thank you for your patience. Beginning this week, our onboarding team has been reduced by nearly 40% and we expect backfills hopefully by the end of the calendar year. That being said, our backlog for onboarding new data senders to ReportStream is roughly 3 to 4 weeks. We can certainly schedule a kickoff meeting with your team this or next week; however, we won't begin verifying test files until mid-November at best.
Would you still be interested in moving forward with becoming a ReportStream data sender, given this timeline? If so, we have some availability as follows:
Wed 11/2 at 2pm-2:30pm ET
Thur 11/3 Between 12:00pm-1pm ET
Thur 11/10 Between 3pm-5pm ET
Looking forward to hearing from you.
Best,
Berni

BX sent out this follow-up email post-kick-off to sender on 11/3/2022:

Hi Ram, Linus, and Abdul:
Thanks again for joining the sender kickoff meeting with the ReportStream team today.
As we discussed, here are the next steps:
Terms of Service (ToS):
If you haven't already done so, you'll need to sign this before we move to testing in production.
To review our TOS, visit https://reportstream.cdc.gov/terms-of-service
After reviewing, sign the ToS agreement form at https://reportstream.cdc.gov/sign-tos
Programmer's Guide:
Review the programmer's guide that Brick, our engineer, reviewed today: https://reportstream.cdc.gov/resources/programmers-guide
Generate dummy data (no PII):
If you need assistance with this or have questions that the programmer's guide can't answer, please reply to this email and we can have you meet with a ReportStream engineer
Our current backlog is out to 3-4 weeks, so you've got time to work on this!
Share the test file:
When it's ready, you can reply to this email and attach your sample test file (no PII).
Our engineers should be able to take a first look at validating the test file in mid-late November.
Again, here's a screenshot of the onboarding steps slide we briefly discussed today (a detailed version of these onboarding steps is included in the Programmer's Guide above):
We look forward to working with you!
Thanks,
Berni
P.S. Ram or Abdul, could you forward this email to Linus as I don't have his email address.

Update from Sender:
Validation test file sent for review:
Hope you and your team are doing well. We want to move forward with the next steps for the onboarding process.
We signed the TOS and reviewed the programmers guide. We will proceed with CSV format. Additionally, the guide mentions
that we can use an existing format that we use to report to CA. We are providing a test file in that format.
Please let us know when we can expect validation to finish.

@brandonnava To bring into next sprint

Hello,
User states: Hello James,
Thank you for reviewing our initial file. We made the necessary changes based on your guidance. Please take a look at our new file attached below.
Please let us know if there are questions and what the next steps are when this file passes your validation.
Best regards,
Ramtej Pinisetty"

Update from Sender:
Hello James,
As per your email, I have added the UTC offset to the following fields: date_result_released, test_result_date, order_test_date, testing_lab_specimen_received_date, specimen_collection_date.
I have also generated the public keys via RSA and have attached them to this email as well.
Staging: shk-rs-rsa-public-key.pem
Production: shkprod-rs-rsa-public-key.pem
Please let me know if anything else is required from our end.
Thank you and have a great day,
Abdul Ahmed @ Simple HealthKit

Update from Sender:
Hi James,
I wanted to check in to see if we can move to the next step. We are hoping to get this live by Feb 3rd. We have some new clients starting to test with us soon.
Best Regards,
Ramtej Pinisetty


[Bug]: "Monitor" tab Out of memory error

What happened?
Hi! In our team we use jaeger to monitor some of our serivces. We have really highloaded service, which execute a lot of operations in small time range.
The problem is that data on "Monitor" has very long loading time. In some cases, when there are aroung 2000 pages with metrics operations kuber kills jaeger pod with OOM message. Also it has side effect on metrics storage Prometheus, which tries to sent a lot of data for these 2000 pages, so it also killed with OOM message.
Steps to reproduce

Enable SPM
Wait a lot of metrics data loading

Expected behavior
We expect that data on Monitor page will be loaded separated for every page, not for all pages in one request, like it works now.
Relevant log output
No response
Screenshot

Additional context
Recently we moved Promethes from HDD to SDD, there are results on screenshots above. With HDD this request had ~ 2 min duration.
Jaeger backend version
1.48.0
SDK
OpenTelemetry .NET Automatic Instrumentation
Pipeline
For metrics:
OTEL AUTO INSTRUMENTATION -> OTEL COLLECTOR -> PROMETHEUS -> JAEGER
Stogage backend
Span storage: badger, Metrics storage: Prometheus v2.74.0
Operating system
Linux
Deployment model
Kubernetes
Deployment configs
apiVersion: apps/v1
kind: Deployment
metadata:
  name: jaeger
  namespace: prod
  labels:
    app: jaeger
spec:
  replicas:
  selector:
    matchLabels:
      app: jaeger
  template:
    metadata:
      labels:
        app: jaeger
    spec:
      containers:
        - name: jaeger
          env:
          - name: METRICS_STORAGE_TYPE
            value: prometheus
          - name: PROMETHEUS_SERVER_URL
            value: ***
          - name: PROMETHEUS_QUERY_SUPPORT_SPANMETRICS_CONNECTOR
            value: "true"
          - name: SPAN_STORAGE_TYPE
            value: badger
          - name: BADGER_EPHEMERAL
            value: "false"
          - name: BADGER_DIRECTORY_VALUE
            value: /badger/data
          - name: BADGER_DIRECTORY_KEY
            value: /badger/key
          image: jaegertracing/all-in-one:1.48.0
          resources:
            limits:
              memory: 4Gi
            requests:
              memory: 2Gi
          ports:
            - name: telemetry
              containerPort: 4317
            - name: web-ui
              containerPort: 16686

@tcaty Can you please share the otel collector setup which is writing the metrics? You can do a lot to make the spanmetrics connector write fewer metrics or fewer buckets, which should reduce the amount of data the monitor tab has to read as well.
We are planning an alternate solution in Jaeger v2 for this use case, but if you want the most control, using the spanmetrics connector is the best option.


Fluid waveform layers

I like this layers functionality, I can see how this is allows for many more possibilities! I also like that there's still the legacy interface of passing a buffer or two and having it display quickly.
A few nice pictures:
NMF 3 Components on Nicol:

NMF 3 Components on Nicol (mels): still not too revealing necessarily, but closer perhaps!

HPSS on Nicol (mels)

NMF 2 Components on Tremblay-ASWINE-ScratchySynth-M.wav

NMF 2 Components on Tremblay-ASWINE-ScratchySynth-M.wav

I cannot believe I didn't think of the obvious L and R layer view... this is fantastic!


update datepicker.js

On line 656:
Angular validation is triggered when date is an empty string because an empty string is an invalid date object. Setting date to null mitigates this issue.
This happens when the input type is set to 'date'

I like this change - good catch.

See related PRs #2513, #3344


[BUG] on Connect-PnPOnline in azure function using CertificateBase64Encoded

Notice
Many bugs reported are actually related to the PnP Framework which is used behind the scenes. Consider carefully where to report an issue:

Are you using Invoke-PnPSiteTemplate or Get-PnPSiteTemplate? NO, Connect-PnpOnLine in Azure Function
Is the issue related to the cmdlet itself, its parameters, the syntax, or do you suspect it is the code of the cmdlet that is causing the issue? the code

Reporting an Issue or Missing Feature
ISSUE
Expected behavior
Connect-PnPOnline -Url $url -ClientId $Env:clientID -CertificateBase64Encoded $Env:secret -Tenant $Env:tenantID -ReturnConnection -ErrorAction Stop
should connect to the requested sharepoint site
Actual behavior
Is working fine on local visual studio code.
the azure function can print the CertificateBase64Encoded string from the key vault.
(only thing I did was I changed the functions_worker_process_count from 1 to 10 and then back to 1)
Get-PnpException output I am receiving:
{
"Message": "The system cannot find the file specified.",
"Stacktrace": "   at Internal.Cryptography.Pal.CertificatePal.FilterPFXStore(Byte[] rawData, SafePasswordHandle password, PfxCertStoreFlags pfxCertStoreFlags)\r\n   at Internal.Cryptography.Pal.CertificatePal.FromBlobOrFile(Byte[] rawData, String fileName, SafePasswordHandle password, X509KeyStorageFlags keyStorageFlags)\r\n   at System.Security.Cryptography.X509Certificates.X509Certificate..ctor(Byte[] rawData, SecureString password, X509KeyStorageFlags keyStorageFlags)\r\n   at System.Security.Cryptography.X509Certificates.X509Certificate2..ctor(Byte[] rawData, SecureString password)\r\n   at PnP.PowerShell.Commands.Base.ConnectOnline.ConnectAppOnlyWithCertificate() in c:\PnPPowerShell\src\Commands\Base\ConnectOnline.cs:line 419\r\n   at PnP.PowerShell.Commands.Base.ConnectOnline.Connect(CancellationToken& cancellationToken) in c:\PnPPowerShell\src\Commands\Base\ConnectOnline.cs:line 245\r\n   at PnP.PowerShell.Commands.Base.ConnectOnline.ProcessRecord() in c:\PnPPowerShell\src\Commands\Base\ConnectOnline.cs:line 207\r\n   at System.Management.Automation.Cmdlet.DoProcessRecord()\r\n   at System.Management.Automation.CommandProcessor.ProcessRecord()",
"ScriptLineNumber": 12,
"InvocationInfo": {
"MyCommand": {
"Verb": "Connect",
"Noun": "PnPOnline",
"HelpFile": "PnP.PowerShell.dll-Help.xml",
"PSSnapIn": null,
"Version": {
"Major": 1,
"Minor": 5,
"Build": 0,
"Revision": -1,
"MajorRevision": -1,
"MinorRevision": -1
},
"ImplementingType": {
"IsCollectible": false,
"DeclaringMethod": null,
"FullName": "PnP.PowerShell.Commands.Base.ConnectOnline",
"AssemblyQualifiedName": "PnP.PowerShell.Commands.Base.ConnectOnline, PnP.PowerShell, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null",
"Namespace": "PnP.PowerShell.Commands.Base",
"GUID": "5005aa8f-cf7d-39ee-abf2-05d950e12e32",
"GenericParameterAttributes": null,
"IsSZArray": false,
"GenericParameterPosition": null,
"ContainsGenericParameters": false,
"StructLayoutAttribute": {
"Pack": 8,
"Size": 0,
"CharSet": 2,
"Value": 3,
"TypeId": "System.Runtime.InteropServices.StructLayoutAttribute"
},
"Name": "ConnectOnline",
"DeclaringType": null,
"Assembly": {
"CodeBase": "file:///C:/home/data/ManagedDependencies/210427154130965.r/PnP.PowerShell/1.5.0/Core/PnP.PowerShell.dll",
"FullName": "PnP.PowerShell, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null",
"EntryPoint": null,
Steps to reproduce behavior
Please include complete script or code samples in-line or linked from gists
##x# What is the version of the Cmdlet module you are running? 1.5.0
(you can retrieve this by executing Get-Module -Name "PnP.PowerShell" -ListAvailable)
Which operating system/environment are you running PnP PowerShell on?

[ ] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ x] Azure Functions
[ ] Other : please specify

@pieronegri I also encounted this issue however this is not a but in the module. You can solve this issue by adding the following function app setting:
WEBSITE_LOAD_USER_PROFILE = 1

The underlying issue is that, if you do not load a user profile in the function app, the environment does not provide a cryptographic keystore or something like that. So it cannot save the certificate data to the hosting environment.

@pieronegri this is working file for me and I am connecting with:
Connect-PnPOnline -Url $SiteUrl -ClientId $clientId -Tenant "$SPO_TENANT.onmicrosoft.com" -CertificateBase64Encoded $CERTIFICATE
In my case, $CERTIFICATE is an app setting that is a Key Vault reference to the certificate secret, so all is handled by Azure.

This is also what I am doing, but once in a while it fails...

@pieronegri I also encounted this issue however this is not a bug in the module. You can solve this issue by adding the following function app settings:
WEBSITE_LOAD_USER_PROFILE = 1

The underlying issue is that, if you do not load a user profile in the function app, the environment does not provide a cryptographic keystore or something like that. So it cannot save the certificate data to the hosting environment.

I will give this a try!

@pieronegri - any update on this ? If it is working fine locally but not in Azure function , I don't think it a PnP PowerShell issue but something to do with the AF configuration, so going to close this :)

@gautamdsheth 100% works in Azure Functions as I have created a few over the last months and works fine

Did a session at the M365 UK user group where I created some functions with certificates. Hope this can help someone: https://www.youtube.com/watch?v=QajesgmwOCM

sorry for delay. it's actually working with WEBSITE_LOAD_USER_PROFILE = 1


cmd/compile: global variable references have weird line numbers

package main

import "os"

func main() {
	x := os.Stdout
	x.WriteString("foo\n")
}

When I compile this with SSA, I get:
	0x0013 00019 (tmp.go:3)	MOVQ	os.Stdout(SB), AX

Note the line number.  It's the line number of the import statement, not the line number where os.Stdout is mentioned.
This bug comes from earlier in the pipeline than the SSA backend.  On input to SSA, we have:
.   AS u(2) l(6) colas(true) tc(1)
.   .   NAME-main.x u(1) a(true) g(1) l(6) x(0+0) class(PAUTO) f(1) tc(1) used(true) PTR64-*os.File
.   .   NAME-os.Stdout u(1) a(true) l(3) x(0+0) class(PEXTERN) tc(1) used(true) PTR64-*os.File

Note the l(3) on the os.Stdout reference.
I suspect the old compiler ends up using the line number of the AS, so it is ok, whereas the SSA compiler uses the line number for the NAME.
Probably a dup of #14742
See issue #15427
@davecheney

I think that was used to help reporting errors so the
line number points to the defining line (in this case,
the import statement.)

All references to an imported object share the same ONAME Node, so their line numbers (at least currently) have to correspond to the definition line rather than the usage line.

It looks like this should just be a matter of avoiding the s.pushLine(n.Lineno) call in (*state).expr for ONAME Nodes.  Maybe for named OLITERAL Nodes too.


[runtime] Improve detection of memory allocation failures

Detect failed allocations from posix_memalign.
posix_memalign doesn't write to its pointer-to-pointer parameter when it fails, so AlignedAlloc ended up returning stack garbage in that case. AlignedAlloc now initializes the value before passing it to posix_memalign, so that the initialized value is returned in case of failure.

Resolves rdar://80504684 and rdar://79314903

@swift-ci please smoke test

@swift-ci please smoke test

@kylemacomber I'm inclined to submit this to 5.5 as is, and add the performance-improving change we discussed earlier as a separate PR only to main.


Keyclock tasks

[ ] talk to the SSO and IDM folks about how to handle long keyclock expiry
[ ] make sure the log in stuff Sam did works
[ ] make sure the time out stuff Rob did for the time out issue.

This is needed because of the shift to standardized keyclocks.

duplicate


[bug]:  Error while invoking: Failed to load sd-1:main:unet model

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Windows
GPU
cuda
VRAM
10
What version did you experience this issue on?
3.0.2Post1
What happened?
All in painting models run error (except the SD1.5 inpainting).
In the WEB UI I have this error after "Invoke" :

and this message in the consol :
====ERR LOAD====
None: Cannot load C:\InvokeAI\autoimport\sd-1\main\RealisticVision v51 Inpainting because conv_in.weight expected shape tensor(..., device='meta', size=(320, 4, 3, 3)), but got torch.Size([320, 9, 3, 3]). If you want to instead overwrite randomly initialized weights, please make sure to pass both low_cpu_mem_usage=False and ignore_mismatched_sizes=True. For more information, see also: https://github.com/huggingface/diffusers/issues/1619#issuecomment-1345604389 as an example.
[2023-08-23 22:54:42,148]::[InvokeAI]::ERROR --> Traceback (most recent call last):
File "C:\InvokeAI.venv\lib\site-packages\invokeai\app\services\processor.py", line 90, in __process
outputs = invocation.invoke(
File "C:\InvokeAI.venv\lib\site-packages\invokeai\app\invocations\generate.py", line 217, in invoke
with self.load_model_old_way(context, scheduler) as model:
File "C:\Users\Utilisateur\AppData\Local\Programs\Python\Python310\lib\contextlib.py", line 135, in enter
return next(self.gen)
File "C:\InvokeAI.venv\lib\site-packages\invokeai\app\invocations\generate.py", line 171, in load_model_old_way
unet_info = context.services.model_manager.get_model(
File "C:\InvokeAI.venv\lib\site-packages\invokeai\app\services\model_manager_service.py", line 365, in get_model
model_info = self.mgr.get_model(
File "C:\InvokeAI.venv\lib\site-packages\invokeai\backend\model_management\model_manager.py", line 491, in get_model
model_context = self.cache.get_model(
File "C:\InvokeAI.venv\lib\site-packages\invokeai\backend\model_management\model_cache.py", line 198, in get_model
model = model_info.get_model(child_type=submodel, torch_dtype=self.precision)
File "C:\InvokeAI.venv\lib\site-packages\invokeai\backend\model_management\models\base.py", line 300, in get_model
raise Exception(f"Failed to load {self.base_model}:{self.model_type}:{child_type} model")
Exception: Failed to load sd-1:main:unet model
[2023-08-23 22:54:42,154]::[InvokeAI]::ERROR --> Error while invoking:
Failed to load sd-1:main:unet model
I read that some parameters have to be set, but don't understand why and where...
PS : I didn't have this issue with InvokeAI 2
Screenshots
No response
Additional context
No response
Contact Details
No response

Could you try clearing your models/.cache and restarting Invoke?

Thanks for your reply, delete the cache folder did'nt solve my issue (the folder was empty, be cause I already convert my models to diffusers with InvokeAI), But, I try to delete the converted model, redownload it, and convert it again... And it works !
I concluded that my models was converter between 3.0 and 3.02 :-)
Thanks


Error while generating TFRECORDS .

I'm encountering errors while generating tfrecords .  Help would be appreciated !
I'm attaching the log here :
python generate_tfrecord.py --csv_input=images/train_labels.csv --image_dir=images/train --output_path=train.record
Traceback (most recent call last):
File "generate_tfrecord.py", line 119, in
tf.app.run()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/app.py", line 125, in run
_sys.exit(main(argv))
File "generate_tfrecord.py", line 110, in main
tf_example = create_tf_example(group, path)
File "generate_tfrecord.py", line 99, in create_tf_example
'image/object/class/label': dataset_util.int64_list_feature(classes),
File "/home/tanush/tensorflow1/models/research/object_detection/utils/dataset_util.py", line 26, in int64_list_feature
return tf.train.Feature(int64_list=tf.train.Int64List(value=value))
TypeError: None has type NoneType, but expected one of: int, long

hi tanushpas , the same error. how to deal with it? lots of thanks.


Zlib CMAKE Error

Hello, I have problem with my opencv installation
I run this command .\vcpkg install opencv[core]:x64-windows
But it shows error like this :
F:\vcpkg>.\vcpkg install opencv[core]:x64-windows
The following packages will be built and installed:
opencv[core]:x64-windows

zlib[core]:x64-windows
Additional packages (*) will be modified to complete this operation.
Starting package 1/2: zlib:x64-windows
Building package zlib[core]:x64-windows...
-- Using cached F:/vcpkg/downloads/zlib1211.tar.gz
-- Using source at F:/vcpkg/buildtrees/zlib/src/1.2.11-f690224aeb
-- Configuring x64-windows
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:56 (message):
Command failed: ninja;-v
Working Directory: F:/vcpkg/buildtrees/zlib/x64-windows-rel/vcpkg-parallel-configure
Error code: 1
See logs for more information:
F:\vcpkg\buildtrees\zlib\config-x64-windows-out.log

Call Stack (most recent call first):
scripts/cmake/vcpkg_configure_cmake.cmake:273 (vcpkg_execute_required_process)
ports/zlib/portfile.cmake:22 (vcpkg_configure_cmake)
scripts/ports.cmake:71 (include)
Error: Building package zlib:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: zlib:x64-windows
Vcpkg version: 2018.11.23-nohash
Additionally, attach any relevant sections from the log files above.
Log error :
https://github.com/godblezzme29/log-error/blob/master/config-x64-windows-out.log
Based on this log, I found that there is problem with the ninja, Is there anyone has solved this problem?
Previously when I have VS 2015 and VS 2017 everything is work with no problem, but after I uninstall my VS 2015, it has error with the installation.
*update : I already re-install VS 2015 but it has same error
My environment :
VS 2017
Windows 10

#6416

PR #6420

Thanks for posting this issue. This issue has been fixed. So I am closing this issue for now.


Week 5: AWS Branch of SCC Training Material

Topics: Amend the following sections of the Selection Round Training material, in an appropriately named branch:

Tutorial 1: Section 3 - Launching your first AWS EC2 Instance
Tutorial 3, Section 8 - Spinning up a Second Compute Node Using a Snapshot
Tutorial 4, Section 4 - Automating the Deployment of Your AWS Instances Using Terraform

Tasks:

Create a feature branch that adds a section with pictures to the tutorial content
Submit a PR

To assist you in editing markdown and preparing your Pull / Merge Requests, you are strongly advised to review:

The first page from Selection Round Content Contribution to the Project, just that you will be creating a fork.
The section just below that, with the links to GitHub Markdown Syntax and Style.

Topics: Amend the following sections of the Selection Round Training material, in an appropriately named branch:

Tutorial 1: Section 3 - Launching your first AWS EC2 Instance
Tutorial 3, Section 8 - Spinning up a Second Compute Node Using a Snapshot
Tutorial 4, Section 4 - Automating the Deployment of Your AWS Instances Using Terraform

Tasks:

Create a feature branch that adds a section with pictures to the tutorial content
Submit a PR

On tutorial 3 section 8 we have found that it cannot be replicated on aws, we have tried but there no options to do so but the rest can be done we will submit the PR shortly

"On tutorial 3 section 8 we have found that it cannot be replicated on aws, we have tried but there no options to do so but the rest can be done we will submit the PR shortly"
As @nyameko  stated you were supposed to take a snapshot of your 1st compute node root block storage (EBS), it's a roots disk that contains the OS which can be used to create more compute node..


ValueBar in ViewPager

I have a ViewPager in which I can swipe between different fragments and integrated the ValueBar in one of these Fragments.
Now when I try to drag the ValueBar, the ViewPager always catches the motion-event and swipes to the next or previous fragment :(

Allright, I got this one on my own, you can do it just like this:
   // Disable siwping fragments in the ViewPager when you drag the ValueBar
    valueBar.setOnTouchListener(new OnTouchListener() {
        @Override
        public boolean onTouch(View v, MotionEvent event) {
            ViewParent parent = getParent();
            parent.requestDisallowInterceptTouchEvent(true);
            return false;
        }
    });


Vim patch 9.0.907 causes E1312 when temporary buffers are closed

Vim changed its behavior at patch 9.0.907 to prohibit autocommands from performing certain operations such as closing buffers. Consequently, closing a linediff temporary buffer now prints an error message like this one:
Error detected while processing WinEnter Autocommands for "<buffer=3>"..function linediff#controller#Destroy[6]..linediff#differ#CloseAndReset[1]
..linediff#differ#CloseDiffBuffer:
line    3:
E1312: Not allowed to change the window layout in this autocmd
A workaround is to change line 214 of autoload/linediff/differ.vim (in function linediff#differ#CloseDiffBuffer(()) from
exe "bdelete".bang." ".self.diff_buffer
to
call feedkeys(":bdelete".bang." ".self.diff_buffer."<CR>:<BS>")
which just executes the :bdelete command in a feedkeys() operation, clearing the :bdelete from the command line with a colon and a backspace.

Thanks for noticing, I have a mapping to close an entire tab, so I didn't realize this was happening when you close just one diff buffer.
I had some trouble getting the workaround to work for me in tests. I think it's a bit fiddly about key inputs. I tried to fix it instead by setting a timer to close the buffer immediately after the autocommand: https://github.com/AndrewRadev/linediff.vim/commit/b4d3c3f03eed00caa741c2650e13db5b7b7ce0ef
This seems to work fine for me and it passes the tests. Could you try it out as well with your own setup, let me know if it fixes the issue for you as well?

You're welcome.
My "git pull" command timed out (weird), so I just downloaded the file from the web page. It seems to work fine. I'll let you know if that changes. Thanks for fixing it, and so quickly.

Cool, I'll close this issue for now, but please reopen if you see any regression


Google server response changed this morning and causes PHP Client Library Error

Issue description
Between 8.00 and 8.30 CET this morning the response from the google reCAPTCHA servers changed
$recaptcha = new \ReCaptcha\ReCaptcha(MY_KEY, new \ReCaptcha\RequestMethod\SocketPost());
The google servers now return, the json is invalid
see below.
Has anybody else had this problem?
HTTP/1.1 200 OK
Content-Type: application/json; charset=utf-8
Date: Tue, 17 Mar 2020 11:39:47 GMT
Expires: Tue, 17 Mar 2020 11:39:47 GMT
Cache-Control: private, max-age=0
X-Content-Type-Options: nosniff
X-Frame-Options: SAMEORIGIN
Content-Security-Policy: frame-ancestors 'self'
X-XSS-Protection: 1; mode=block
Server: GSE
Alt-Svc: quic=":443"; ma=2592000; v="46,43",h3-Q050=":443"; ma=2592000,h3-Q049=":443"; ma=2592000,h3-Q048=":443"; ma=2592000,h3-Q046=":443"; ma=2592000,h3-Q043=":443"; ma=2592000,h3-T050=":443"; ma=2592000
Accept-Ranges: none
Vary: Accept-Encoding
Connection: close
Transfer-Encoding: chunked
5a
{
"success": true,
"challenge_ts": "2020-03-17T11:36:43Z",
"hostname": "<IP_ADDRESS>"
}
0
This breaks the function public static function fromJson($json) in the lib
Environment
We have seen the problem on 4 systems of different configurations.

Rowan, Is the google server json response valid?
5a
{
"success": true,
"challenge_ts": "2020-03-17T11:36:43Z",
"hostname": "<IP_ADDRESS>"
}
0

Same Problem here.
/ReCaptcha/ReCaptcha.php:96:string '69
{
"success": true,
"challenge_ts": "2020-03-19T11:48:43Z",
"hostname": "dummy.host.name.com"
}
0
' (length=113)

Did you see the fix from Lionel?
Just change SocketPost to CurlPost

Yes. The fix worked fine.
I just wanted to confirm the problem.

Also fixed by switching to CurlPost instead of SocketPost.

We also fixed the SocketPost by patching the fromJson() in the lib
It's not a proper fix but it works
It may be needed by some as the CurlPost needs a certificate
/**
 * Build the response from the expected JSON returned by the service.
 *
 * @param string $json
 * @return \ReCaptcha\Response
 */
public static function fromJson($json)
{
    $responseData = json_decode($json, true);

    if (!$responseData) {

        $hostname = isset($responseData['hostname']) ? $responseData['hostname'] : null;

        if (strpos($json, 'success') !== false && strpos($json, 'true') !== false) {
            return new Response(true, array(), $hostname);
        }

        return new Response(false, array('invalid-json'));
    }

    $hostname = isset($responseData['hostname']) ? $responseData['hostname'] : null;

    if (isset($responseData['success']) && $responseData['success'] == true) {
        return new Response(true, array(), $hostname);
    }

    if (isset($responseData['error-codes']) && is_array($responseData['error-codes'])) {
        return new Response(false, $responseData['error-codes'], $hostname);
    }

    return new Response(false, array(), $hostname);
}

The issue is that recently Google servers started to serve chunk-encoded HTTP responses. Until this is fixed in the library, if you can't switch to CurlPost what you need to do is edit SocketPost and decode the response as a potential HTTP chunk-encoded. You can do so either by using a custom function (plenty of examples around the web) or more conveniently if you are using / can use PECL, the pecl-http package includes a http_chunked_decode function. If you end up with a clean solution, feel free to PR it as well, hope that helps!

The issue is that recently Google servers started to serve chunk-encoded HTTP responses. Until this is fixed in the library, if you can't switch to CurlPost what you need to do is edit SocketPost and decode the response as a potential HTTP chunk-encoded. You can do so either by using a custom function (plenty of examples around the web) or more conveniently if you are using / can use PECL, the pecl-http package includes an http_chunked_decode function. If you end up with a clean solution, feel free to PR it as well. Hope that helps!

Thanks for the explanation.
I wondered why switching to CurlPost fixed the problem.

‘CurlPost’ does not experience the problem because ‘cURL’ handles and parses the chunk-encoded HTTP internally, while ‘SocketPost’ just uses a socket for a raw HTTP request with minimal parsing involved.

On 22 Mar 2020, at 09:35, Domgarth<EMAIL_ADDRESS>wrote:

Thanks for the explanation.
I wondered why switching to CurlPost fixed the problem.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

‘CurlPost’ does not experience the problem because ‘cURL’ handles and parses the chunk-encoded HTTP internally, while ‘SocketPost’ just uses a socket for a raw HTTP request with minimal parsing involved.

CurlPost does not experience the problem because cURL handles and parses the chunk-encoded HTTP internally, while SocketPost just uses a socket for a raw HTTP request with minimal parsing involved.

We fixed it that way:
/**
 * Build the response from the expected JSON returned by the service.
 *
 * @param string $json
 * @return \ReCaptcha\Response
 */
public static function fromJson($json)
{
    $responseData = false;
    if (self::isJsonValid($json)) {
        $responseData = json_decode($json, true);
    } else {
        $json_pattern = '/\{(?:[^{}]|(?R))*\}/xs';
        preg_match_all($json_pattern, $json, $matches);
        $filtered_json = trim(preg_replace('/\s+/', ' ', $matches[0][0]));
        if (self::isJsonValid($filtered_json)) {
            $responseData = json_decode($filtered_json, true);
        }
    }

    if (!$responseData) {
        return new Response(false, array(ReCaptcha::E_INVALID_JSON));
    }

    $hostname = isset($responseData['hostname']) ? $responseData['hostname'] : null;
    $challengeTs = isset($responseData['challenge_ts']) ? $responseData['challenge_ts'] : null;
    $apkPackageName = isset($responseData['apk_package_name']) ? $responseData['apk_package_name'] : null;
    $score = isset($responseData['score']) ? floatval($responseData['score']) : null;
    $action = isset($responseData['action']) ? $responseData['action'] : null;

    if (isset($responseData['success']) && $responseData['success'] == true) {
        return new Response(true, array(), $hostname, $challengeTs, $apkPackageName, $score, $action);
    }

    if (isset($responseData['error-codes']) && is_array($responseData['error-codes'])) {
        return new Response(false, $responseData['error-codes'], $hostname, $challengeTs, $apkPackageName, $score, $action);
    }

    return new Response(false, array(ReCaptcha::E_UNKNOWN_ERROR), $hostname, $challengeTs, $apkPackageName, $score, $action);
}

/**
 * Checking for JSON validity.
 *
 * @param string $string
 * @return bool
 */
public static function isJsonValid($string) {
    json_decode($string);
    return (json_last_error() == JSON_ERROR_NONE);
}

We are experiencing the exact same issue since today. We "fixed" this by using CurlPost instead of SocketPost.

Where did you make this switch?

I've released a super quick fix which is just requesting HTTP 1.0 instead of HTTP 1.1 and as a result - no chunking in the response.
That's a bit dirty though, so I'll update later on to actually handle a chunked response properly.

https://github.com/google/recaptcha/compare/1.2.3...1.2.4

#383


Window going TOP inside if my task bar position is top

Issue Type: Bug

Set Top to Tasksbar location on screen.
Open a vscode.
File -> New Window.
Look where is window head? it is not in proper position.

VS Code version: Code 1.31.0 (7c66f58312b48ed8ca4e387ebd9ffe9605332caa, 2019-02-05T22:35:56.624Z)
OS version: Windows_NT x64 10.0.17134

@mansres can you attach a screenshot

Please do refer steps which is I mentioned before. You can get it.

Looks ok to me:

It is looks like this to me.

/duplicate https://github.com/Microsoft/vscode/issues/50830


make the plugin able to download behind proxy on windows guest

Currently the plugin is not able behind proxy on windows guest.
I rewrote the PowerShell script to try to download without proxy. If the attempt fails, a new attempt is made using the environment variables http_proxy and no_proxy. The new script also works if the proxy requires authentication.
This also fixes the issue #80
The proxy configuration can be done using  then plugin vagrant-proxyconf.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@fabriciocolombo

Please sign the CLA here.

Closing this out since the CLA was never signed.


[sampleapp-android] Change make.py to apptools

Impacted tests(approved):new 0,update 42,delete 0
Unit test platform:[Deepin]
Unit test result summary:pass 42,fail 0,block 0

About the title: should be make_apk.py
And pending this update, since apptools does not support shared mode yet.

I will update this pr when apptools supports shared mode

I guess you guys want this to be closed and to make new PR in need, right?

Yes, I will close it.


fix: ui elements shifting to the right

The main layout elements are shifting to the right when the search bar is open
before:
https://user-images.githubusercontent.com/47347106/226173941-f18d6faa-9565-466b-bab1-151e15017623.mp4
after:
https://user-images.githubusercontent.com/47347106/226173953-6c83189b-fe85-464f-8b08-4714a17702e3.mp4

Is there any more natural way to do this?
Why does the scrollbar disappear underneath the model? Can we fix that instead?

If the scrollbar appears, it means that the main content below the modal can be scrolled, which is not desirable since we disable scrolling when the popup modal opens. This causes the scrollbar to disappear, similar to using overflow: hidden.

On further investigation, I found that @docsearch/react is adding a class DocSearch--active to the body element. Which is making the scrollbar hide and they are using overflow: hidden to do that. When we remove that class, the main content below the modal can be scrolled.


[#36] [Backend] As a user, I can sign out

#36
What happened

As users can sign in, the app must have an option for users to sign out. Signing out should completely remove any account-related footprints of the user.

Insight
For APIs:

https://survey-api.nimblehq.co/api/v1/me
https://survey-api.nimblehq.co/api/v1/oauth/revoke

Created UserUseCase.
Created UserRepository.
Created UserRequestConfiguration.
Test send API requests to get correct responses.

Proof Of Work
https://survey-api.nimblehq.co/api/v1/me

https://survey-api.nimblehq.co/api/v1/oauth/revoke

Yes, we will have unit test for UserUseCase and UserRepository in ticket #35 .


Add support for REPL Snowflake connectivity

Add support for REPL Snowflake connectivity

Duplicate of https://github.com/finos/legend-engine-ide-lsp/pull/216