Update walk-through documentation; call-out must-haves and alleviate likely problems

Description
As a user, I'd like to have a more complete overview of the Getting Started walkthroughs prior to starting, and not be blocked from completing while partway through due to insufficient permissions or misunderstanding of input data values.
Acceptance Criteria
Reference: [Done-Done Checklist] (https://github.com/Microsoft/code-with-engineering-playbook/blob/master/Engineering/BestPractices/DoneDone.md)

[ ] Should update the documentation for each walkthrough to identify the permissions needed to complete the task
[ ] Should update the documentation for each walkthrough to include a preparation worksheet of data values (resource names, etc) that should be defined or obtained ahead of time
[ ] Should update the documentation for each walkthrough to include callouts for each data value that needs to be captured for use later (and which cannot be known/defined ahead of time)
[ ] Should update the documentation for each walkthrough to more clearly describe the role/scenario which it is intended to support
[ ] Should update the documentation for each walkthrough to distinguish between those steps which need to be performed once within an organization, and those which need to be performed for each deployment pipeline
[ ] Should include any prerequisites needed for self-hosted build agents beyond what Azure prescribes for the agent itself

Resources

[ ] Getting Started walkthrough for Cobalt Developer use case
[ ] Getting Started walkthrough for Advocated Pattern Owner use case
[ ] Getting Started walkthrough for Application Developer use case

Tasks
Stories are intended to be completed in a single sprint; if task breakdown creates addition work then team should discuss promoting the Story to an Epic.
Reference: [Minimal Valuable Slices] (https://github.com/Microsoft/code-with-engineering-playbook/blob/master/Engineering/BestPractices/MinimalSlices.md)
Reference: [How to Write Better Tasks] (http://agilebutpragmatic.blogspot.com/2012/04/splitting-story-into-tasks-how-to-write.html)
Assignee should break down work into tasks here

I've added an item to the list to indicate self-hosted agent requirements.

I've added an item to the list to document places in the Azure portal where ui bugs incorrectly displays configurations. (Keyvault access policies requires refresh, App Reg Service Principals displays outdated reply_uri changes and requires tenant re-login)


Managed IAM Policy AWSCloudWatchLambdaInsightsExecutionRolePolicy not available

What happened?
According to this older issue https://github.com/pulumi/pulumi-aws/issues/1700 the CloudWatchLambdaInsightsExecutionRolePolicy was added as a constant in the AWS IAM managed policies list.
However, the item is not within the typescript defined list and typing the role that was added in the referred Github issue (and related MR) gives me back an error that the role does not exist. My best guess is that the previous fix forgot a list variable or something.
Steps to reproduce
new aws.iam.RolePolicyAttachment("lambda_role-lambda_insights-attachment", {
  role: signAndVerifyDeclarationsLambdaRole,
  policyArn: aws.iam.ManagedPolicies.AWSCloudWatchLambdaInsightsExecutionRolePolicy, // <-- this should return  "arn:aws:iam::aws:policy/AWSCloudWatchLambdaInsightsExecutionRolePolicy", but errors out instead.
});

Expected Behavior
No error when you use the IAM managed policy aws.iam.ManagedPolicies.AWSCloudWatchLambdaInsightsExecutionRolePolicy.
Actual Behavior
Error occurs.
Output of pulumi about
Dependencies:
NAME                VERSION
@pulumi/aws         5.23.0
@pulumi/pulumi      3.49.0

Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

The documentation on aws.iam.ManagedPolicies says Use ManagedPolicy instead. The plural version of the enum appears to be an early TypeScript only implementation whereas ManagedPolicy is supported between all languages.
This code works correctly:
aws.iam.ManagedPolicy.AWSCloudWatchLambdaInsightsExecutionRolePolicy

It appears we didn't surface this documenation as a hard deprecation to avoid causing existing users too much noise, though perhaps this would be worth revisiting in the future.

@danielrbradley I tried your line (with the singular version). However, I still get the error:
operation CREATE failed with "NotFound": Policy arn:aws:iam::aws:policy/AWSCloudWatchLambdaInsightsExecutionRolePolicy does not exist or is not attachable. (Service: Iam, Status Code: 404, Request ID: 1a1ecdb6-6d63-4415-a588-6725dc37954e)

I looked the policy up in my AWS account and matched the naming closely (watching for typo's), but all seems to be fine on that side. Any clue why this error occurs?

Ok, I've reopened so we can continue to track this.

@danielrbradley I checked the source code (search: https://github.com/pulumi/pulumi-aws/search?q=AWSCloudWatchLambdaInsightsExecutionRolePolicy).
And indeed the auto-generated files contain the wrong policy name: https://github.com/pulumi/pulumi-aws/blob/682d5dac3a211aa2cdae472eefe9721462433a27/sdk/nodejs/types/enums/iam/index.ts#L21
"arn:aws:iam::aws:policy/AWSCloudWatchLambdaInsightsExecutionRolePolicy", --> must become "arn:aws:iam::aws:policy/CloudWatchLambdaInsightsExecutionRolePolicy",

And indeed the auto-generated files contains the wrong policy name:
https://github.com/pulumi/pulumi-aws/blob/682d5dac3a211aa2cdae472eefe9721462433a27/sdk/nodejs/types/enums/iam/index.ts#L21
"arn:aws:iam::aws:policy/AWSCloudWatchLambdaInsightsExecutionRolePolicy", --> must become "arn:aws:iam::aws:policy/CloudWatchLambdaInsightsExecutionRolePolicy",

@danielrbradley I'm bumping this topic as you might have missed my update (with the problem and solution) on the previous comment. If this issue needs to be addressed somewhere else (/other team), please let me know so this issue can be resolved.

Ok, thanks for the detail @Jimmy89. It's in the backlog for further investigation 👍


Import person into Amy

Retrieve the person list from Sords
Convert and clean
Extend Bulk importer to handle extra fields (occupation, affiliation)
Deploy.

Hi @pherterich, @hughshanahan , please see details in your email. I've created the first import yesterday. Let me know if more is expected for this item (such as more alumni needing to be imported).


Merge Sort In Golang

Introduction to the Sort
Time Complexity Information Of All Scenarios
Code For The Algorithm

I want to work on this please assign me

I would like to work on this.

@ST1LLWATER I have already opened this issue - https://github.com/ST1LLWATER/Sorting-Algorithms/pull/87#issue-1014089820


Add new text-to-speech feature for reading new messages

This code implements a new feature that enables text-to-speech reading of newly generated messages. The feature includes a toggle for the audio, which can be found in the top right of the user interface next to the clear all messages button. The audio can be enabled or disabled by clicking on the volume icon. The system will automatically read out new messages when the audio is enabled, and a new message is generated in the selected conversation.

https://github.com/C-Nedelcu/talk-to-chatgpt
Could this be useful for TTS implementation?

A play/stop button next to each text would be cool with voice speed and type options.


[Bug]: Eufy Security add-on crashes during extended P2P to RTSP streaming

Client version
1.7.1
Node version
Unsure how to get this?
Operating System type
Linux
Operating system version
HASSIO (Fully updated)
Describe the bug
When I P2P stream an S100 wall light camera through the RTSP Simple Server and leave it running it will always fail after a failry short amount of time. I would like to be able to have a camera stream up for longer periods. The Android Eufy mobile app seems to have no problem with this. My recent test crashed the add-on after 21 minutes. The HASSIO error is the screenshot. Eufy add-on logs have nothing because it crashed before logging any errors.
To reproduce

Open the WebRTC card for the camera, click the 'Start P2P Stream' entity.
Wait till it falls over.
The moment it freezes, HASSIO registers the 'Connection to Eufy Security add-on is broken' error.
Eufy Security add-on is now non functional and must be restarted. The 'Watchdog' feature seems unable to recover/restart it.

Screenshots & Logfiles

Additional context
I get these stream failures regardless of the camera. The cameras have very strong and reliable connections. The length of time the stream will run is variable. The shortest might be 5 minutes, the longest around 20.

+1; although its not necessarily related to long term streaming; I will debug for the coming 24 hours to see if something pops up.
Right now I get the same message per above, addon remains running (50%+ CPU usage) with a log message:
2024-01-02 10:44:44.129  INFO  Client disconnected with ip: <IP_ADDRESS> port: 43094 code: 1006 reason: Abnormal Closure

Same for me

Today i experinced the same every time a p2p streaming started. After ~3 seconds the started, immediatelly stopped (Idle state). Next streaming probe make the addon hangs and eventually crashes. In the mean time the memory consumption goes up to ~2 GB. Once i saw an error aftercrash-beforrestart. Something about java out of memory. I don't know, what changed, a few days ago there was no problem like this. Eufy api changed maybe?

+1 same for me


Consider more than code

https://twitter.com/le_debut/status/1164414438570627072

Closing this in favor of #51


mgr: refact 'wait for all mgr to be up' task

There's no need to use shell module here.
Instead of using | python -c, let's use from_json filter.
Signed-off-by: Guillaume Abrioux<EMAIL_ADDRESS>

jenkins test dev-centos-container-all_daemons


race condition?

mp_Mapper.py:137
self.is_tracking_keyframe_shared[0] = 0
should be after mp_Mapper.py:146-151
newcam = copy.deepcopy(self.shared_cam)
newcam.on_cuda()
    self.mapping_cams.append(newcam)
    self.keyframe_idxs.append(newcam.cam_idx[0])
    self.new_keyframes.append(len(self.mapping_cams)-1)

because the tracker can update the shared_cam immediately to a new value.
and the original value will be overridden and lost.

Hello,
Thank you for your suggestion, and I agree with your suggestion.
I will correct that part!


Margin width option for EPUB

Description of the new feature / enhancement
Option to set margin width for epub files in reader settings.
https://user-images.githubusercontent.com/127277950/235130891-38f60885-d370-43a5-b4f4-390a94c38f9d.mp4
Scenario when this would be used?
To set custom margins.
Supporting information
No response

That is size under reader setting.

That is not possible, without causing unwanted side effects.


Web -  Clicking on browser back button before closing shortcut modal doesn't close modal

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open ND
Press CTRL + J to open the keyboard shortcuts
Click on browser back button

Expected Result:
Modal should close
Actual Result:
Modal stays open while app navigates back to the previous page
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.35-5
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/fa89f7a3-bc2b-45a7-a812-a0f5c629f8ef
https://github.com/Expensify/App/assets/93399543/6da2013f-69ba-45a2-9996-121103c0a6da
Expensify/Expensify Issue URL:
Issue reported by: @Nathan-Mulugeta
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688220937658199
View all open jobs on GitHub

Thanks @hungvu193. Closing in favor of the earlier issue.


irclogs clickable links

The links are untrusted, so nofollow.
cc/@Kwpolska

Merged a bit too fast, it looks like it’s not done correctly…

21:47:48 <Aeyoun> why isn’t the "Suggested configuration" section shown? https://plugins.getnikola.com/#recent_posts_json
 name="irclog-2">21:59:01 <Aeyoun> Fixed! :) https://www.google.no/search?q=site:plugins.getnikola.com

fixed by matching against all whitespace: [^\s]+

Broke on links ending with a line ending? Of course. Thanks for fixing it.


Routing to hardcoded ids does not work

Describe the bug
The router should allow me to add routes in the following way:
$routes->get( 'abcd/efgh', 'Pages::id/123' )
However, in the default Router (Codeigniter\Router), all the / in the$to parameter get overwritten by \. This breaks the functionality of the route since it starts looking at the Pages controller for the function id\123 (which does not exist).
Your own guidelines even have an example that has this kind of route in it.
$routes->add('blog/joe', 'Blogs::users/34');
I have not actually tested this, but I guess that it fails for the same reasons as why my route is failing.
As far as I can tell, the way I want to route corresponds roughly with the routing described in the tutorial.
This problem seems to occur to the following lines of code in the checkRoutes( string $url ) function (in the Router, lines 471 to 474).
elseif (strpos($val, '/') !== false)
	{
		$val = str_replace('/', '\\', $val);
	}

These lines translate all the / in $val to \. I don't know why this is done, but commenting those four lines fixes the problem for me (although I don't know what other consequences there are...).
CodeIgniter 4 version
Which version (and branch, if applicable) the bug is in.
The first Codeigniter beta version.
Expected behavior, and steps to reproduce if appropriate
I expect that my routes are routed to the controller with the 123 passed as the $id parameter.
Context

OS: [MacOS]
Using php spark serve
PHP version [7.3]

I took the liberty of creating a PR that removes the translation of / into \ in the $to part of the routes. However, I cannot oversee all the consequences of this PR so please only merge if it does not break anything else (although I cannot quickly see what it would break 👍 ).


installation.md: update shell syntax for setting env vars

I find this more readable; it also no longer requires bash (thought just replacing bash with sh would achieve that much).
Strictly speaking, env is not necessary on most shells, but I find it makes things clearer.
Test Plan
Ran the command locally, did not try compiling the docs.

Aside, loosely related (and hopefully helpful) suggestions:
I'm hoping you will also explain in the docs how to install to ~/.local/bin, with the same goal as https://github.com/astral-sh/uv/pull/6839. Using environment variables for that is fine.
Another minor FR I'd have is to mention these environment variables in the help message of the installer script, especially if you want to encourage people to use them.
Thank you for working on the installation script! It helps me feel more comfortable about eventually asking that people install uv to compile docs on my project, hopefully helping people who don't have a Python environment already installed.

Thanks this seems reasonable to me! I recently put up https://github.com/astral-sh/uv/pull/7107 — https://github.com/astral-sh/uv/pull/6839 which looked a bit too complicated for users and I've been working with the upstream team to make configuration of the installer simpler / clearer.

Another minor FR I'd have is to mention these environment variables in the help message of the installer script, especially if you want to encourage people to use them.

Can you open an issue for this so we can track it?

One small problem with this approach is now the Windows recommendation is a different style :/


Is it possible to combine multiple subdomains in a single certificate?

Hi,
certbot offers a functionality with which you can create a single certificate for multiple subdomains using a command like this:-
certbot-auto -d one.example.com -d two.example.com -d three.example.com -d example.org
Can we do the same with lua-resty-auto-ssl? Or does it create a different certificate for each of the sub-domains?
PS: It seems that dehydrated also offers a -d flag:-

--domain (-d) domain.tld         Use specified domain name(s) instead of domains.txt entry (one certificate!)

Thanks,
Akshat

@brickcap not yet, but maybe soon! :) See https://github.com/GUI/lua-resty-auto-ssl/issues/36 and https://github.com/GUI/lua-resty-auto-ssl/pull/57.
@GUI or @brickcap please close this, as we've got an issue already.


missing /joomla_english

Joomla module is linking to https://www.zitat-service.de/joomla_english and is getting 404 at the moment

There is https://www.zitat-service.de/start/joomla_english existing. but route to shorter https://www.zitat-service.de/joomla_english is missing

installed on production with
git pull and passenger-config restart-app


Allow logging in via oauth and SPCS Oauth

This is my first PR here, and there are a few things in this PR that are probably too opinionated, but this is an attempt to allow oauth and oauth from within SPCS containers to work. I'm very happy to make further changes or improve the code as necessary.


quotation fetch from field customer_type

In lead there's is no field customer_type instead we got field 'Lead is an Organization' (organization_lead) in lead, so moving logic of fetch_from in custom field json to quotaion js file
customer_type field as select field just like in Customer Doctype field (customer_type)
Added property setters for address doctype via python

@beingeek thank you for this. I expect that the customer will also wan't the definition of customer_type in py rather than js.
They have a scenario where Transactions are created via API where the JS might not take action in. Can you overhaul this?

I think there's some misunderstanding, let me elaborate. If i describe functionality of this app in fewest word then this app consist of:

Enhancing functionality of transactions Doctypes namely ("Quotation", "Sales Order" and "Sales Invoice") extended_tax_category.py and this file contain a fuction validate_tax_category_fields, which enhance transactions Doctypes validate event of   via doc_events hooks, and this will work perfectly fine  on validate event whenever a new record is created.

Added Customer fields and setting properties in newly created fields, I had 2 options for creating custom fields:
i. Creating fields via UI Customise Form and and then export those using fixture fields via fixtures and fields will be forced on database on every migration.
ii. Other option which i used in this app is to keep fields in a python file function and then create fields on after_migrate and after_install hook function, and i choose this option because this more handy and developer friendly with more control on process of field creation.

Also there was a requirement of setting properties on fields, now again we have two options:
i. Creating properties via creating 'Property Setter' records on after_migrate/after_install hooks, which i used to set properties of Address Doctype.
ii. Using javascript features like frm.add_fetch and frm.set_df_property which i avoided as client requirement, but for 1 case in Quotation requirement was to fetch customer_type from party_name conditionally, it quotation_to is 'Customer' then simply frm.add_fetch('party_name', 'customer_type', 'customer_type') and else if condition quotation_to is 'Lead' then it will get value of field organization_lead from Lead and compare if this field is checked then customer_type will be set for to 'Conpany' else 'Individual' will be set to field. In frappe right now there's no way to set fetch_from conditionally just like we features mandatory_depends_on and read_only_depends_on.


Can it generate png with doubled size (2x)?

Hi @jkphl .
If there any option for generating png sprites with 2x, 3x sizes ?
Thanks.

Hey @nicksyndicate,
the current (old) generation of node-iconizr doesn't support PNG scaling, sorry. The upcoming version will have that feature, though, due for release within the next two weeks.
Cheers,
Joschi


install adapter mihome-vacuum fails

Hi andré, thanks for the great image!
I am trying to install/upgrade iobroker adapter mihome-vacuum (https://github.com/iobroker-community-adapters/ioBroker.mihome-vacuum):
installation routine fails:
$ ./iobroker upgrade mihome-vacuum Update mihome-vacuum from @1.1.5 to @2.0.7 NPM version: 6.13.4 npm install<EMAIL_ADDRESS>--loglevel error --prefix "/opt/iobroker" (System call) gyp  ERR! configure error gyp ERR! stack Error: write EPROTO<PHONE_NUMBER>37408:error:1408F10B:SSL routines:ssl3_get_record:wrong version number:../deps/openssl/openssl/ssl/record/ssl3_record.c:332:gyp ERR! stack gyp ERR! stack     at WriteWrap.afterWrite [as oncomplete] (net.js:789:14)gyp ERR! System Linux 4.14.24-qnapgyp ERR!  command "/usr/bin/node" "/usr/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "configure" "--fallback-to-build" "--module=/opt/iobroker/node_modules/canvas/build/Release/canvas.node" "--module_name=canvas" "--module_path=/opt/iobroker/node_modules/canvas/build/Release" "--napi_version=5" "--node_abi_napi=napi" "--napi_build_version=0" "--node_napi_label=node-v64"gyp ERR! cwd /opt/iobroker/node_modules/canvasgyp ERR! node -v v10.19.0gyp ERR!  node-gyp -v v5.0.5gyp ERR! not ok  node-pre-gyp  ERR! build error  node-pre-gyp ERR! stack Error: Failed to execute '/usr/bin/node /usr/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js configure --fallback-to-build --module=/opt/iobroker/node_modules/canvas/build/Release/canvas.node --module_name=canvas --module_path=/opt/iobroker/node_modules/canvas/build/Release --napi_version=5 --node_abi_napi=napi --napi_build_version=0 --node_napi_label=node-v64' (1)node-pre-gyp ERR! stack     at ChildProcess.<anonymous> (/opt/iobroker/node_modules/node-pre-gyp/lib/util/compile.js:83:29)node-pre-gyp ERR! stack     at ChildProcess.emit (events.js:198:13)node-pre-gyp ERR! stack     at maybeClose (internal/child_process.js:982:16)node-pre-gyp  ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:259:5)node-pre-gyp ERR! System Linux 4.14.24-qnap node-pre-gyp ERR! command "/usr/bin/node" "/opt/iobroker/node_modules/.bin/node-pre-gyp" "install" "--fallback-to-build"node-pre-gyp ERR! cwd /opt/iobroker/node_modules/canvasnode-pre-gyp ERR! node -v v10.19.0node-pre-gyp ERR!  node-pre-gyp -v v0.11.0node-pre-gyp ERR! not ok  npm  ERR! code ELIFECYCLEnpm ERR! errno 1npm ERR<EMAIL_ADDRESS>install: node-pre-gyp install --fallback-to-build`npm ERR! Exit status 1npm ERR! npm ERR! Failed at the<EMAIL_ADDRESS>install script.npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:npm ERR!     /opt/iobroker/.npm/_logs/2020-04-12T11_43_21_011Z-debug.log
ERROR: host.ioBroker-Host Cannot install<EMAIL_ADDRESS>1
ERROR: process exited with code 25`
(sorry for bad log structure, unfortunatelly line brakes get not recognized when pasting?)
There is a post at github of mihome-vacuum which states to manually install canvas via "sudo npm install canvas --unsafe-perm=true", but that fails as well.
Happy to provide additional info!
Thanks a lot for your work!
Tim

updated to newest buanet/iobroker image , updated a lot of adapters in the last days (admin, web, javascript,...) and now i can install the mihome-vacuum adapter.
Unfortunatelly I don't exactly know what solved it.


fix extend for 2 channel sources

This PR fixes extend when using a 2 channel source (e.g. greyscale png w/ alpha). I've simply copied the fix for https://github.com/lovell/sharp/issues/411 from https://github.com/lovell/sharp/commit/e0d58266be05da72c1072c5c7048637113b172c0 and added a test to verify it.
Previously this resulted in the following error from libvips: linear: vector must have 1 or 2 elements.

Coverage remained the same at 99.49% when pulling ccb3aa6624644738d9029c7a40058fe57060f0d7 on ppaskaris:extend-bw-alpha into dd9d66ef20a059e7da705f78146af2f33509d31b on lovell:master.

Fantastic, thank you!


Move thread completion to end of chain

Thread completion events (metrics and hook execution) were firing earlier in 1.4.x, as documented by #377, and discovered via #327.  This PR moves them back to the rightful place at the end of the Observable chain.  Along the way, 2 unrelated bugs were fixed.

Hystrix.startCurrentThreadExecutingCommand() and the corresponding end Action0 were being invoked twice per command.  Now they both invoked once
As documented in #448, HystrixCommand.isExecutedInThread() was documented to only return true when it actually ran in the Hystrix thread.  In practice, it was returning true in the case that it was set up to run in a thread, but got rejected.  Fixing this made the implementation of the thread completion straightforward.

Hystrix-pull-requests #200 SUCCESS
This pull request looks good


Suggestion/Question: Would make sense to add redis-sentinel to support redis cluster?

Closed by https://github.com/socketio/socket.io-redis/pull/95.


opt close the batch mode of file split in default

Proposed changes
follow https://github.com/apache/doris/pull/34032, close the batch mode of file split in default. It's not stable currently.

run buildall


Vanilla Unet Bringup

Model card - https://github.com/tenstorrent/tt-metal/issues/13272
The ttnn implementation of vanilla UNet is in branch punith/vanilla_unet_inference
The test file are in path.
Current PCC is 0.93.
Pending issues related to vanilla UNet model:
#6326
#13324
#13336
Below are the path to run test of vanilla UNet module:
Unet - pytest tests/ttnn/integration_tests/vanilla_unet/test_ttnn_unet.py
Currently,
Torch convTranspose is used.
One torch convolution.
One torch maxpool
one torch sigmoid - keeping it to ttnn reduces the pcc to 0.0. Should check it.
Many TM operations like to_device, ttnn.reshape and from_device are used to bypass OOM issue and statical buffer issue in the pipeline. Will reduce it.

fyi @saichandax


Pull request pep8 errors

There are many different changes in this PR... please resubmit each change as an individual PR


feat: add support for Google Compute Engine

Coverage increased (+0.6%) to 97.403% when pulling 925ca10dfe0a70df6c1b8de536cf46403cdf34f4 on gce into 91efc038fd8f902c8e9ef14e320e5a233580076c on master.

Coverage increased (+0.6%) to 97.403% when pulling 323f454d7649c29387ec7a37f205296e00b651f6 on gce into 91efc038fd8f902c8e9ef14e320e5a233580076c on master.


appengine/xmpp: remove/relocate unused sample

https://github.com/GoogleCloudPlatform/golang-samples/tree/master/docs/appengine/xmpp does not appear to be used in official documentation and can potentially be removed or moved to a Community Tutorial.

XMPP is shut down, we can remove it.


🐛 Crashes when displaying ad banners in Listview on iOS

Bug report
When using adbanners every x entrys in a Listview, flutter crashes on iOS
Steps to reproduce
Steps to reproduce the behavior:

Implement Banner Ad in a Container
Call the Container within a Listview.separated
App crashes as soon as the first ad banner enters the viewing range

Expected behavior
The List scrolls down with an adbanner showing every x entries

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
[√] Flutter (Channel stable, 2.0.4, on Microsoft Windows [Version 10.0.18363.1440], locale de-DE)
[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    X Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.
[√] Chrome - develop for the web
[√] Android Studio (version 3.6)
[√] VS Code (version 1.55.0)
[√] Connected device (1 available)

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
Dart SDK 2.12.2
Flutter SDK 2.0.4
flutter_ac 2.6.10+116

dependencies:
- cupertino_icons 0.1.3
- device_info 0.4.2+10 [flutter device_info_platform_interface]
- expandable 4.1.4 [flutter]
- flutter 0.0.0 [characters collection meta typed_data vector_math sky_engine]
- flutter_localizations 0.0.0 [flutter intl characters clock collection meta path typed_data vector_math]
- flutter_styled_toast 1.4.0+1 [flutter flutter_localizations]
- google_mobile_ads 0.11.0+4 [meta flutter]
- http 0.12.2 [http_parser path pedantic]
- package_info 0.4.3+2 [flutter]
- path_provider 1.6.24 [flutter path_provider_platform_interface path_provider_macos path_provider_linux path_provider_windows]
- path_provider_ex 1.0.1 [flutter]
- provider 4.3.2+2 [flutter nested collection]
- shared_preferences 0.5.12+4 [meta flutter shared_preferences_platform_interface shared_preferences_linux shared_preferences_macos shared_preferences_web shared_preferences_windows]
- sqflite 1.3.1+2 [flutter sqflite_common path]
- uni_links 0.4.0 [flutter]
- url_launcher 5.7.10 [flutter url_launcher_platform_interface url_launcher_web url_launcher_linux url_launcher_macos url_launcher_windows]
- webview_flutter 1.0.6 [flutter]

dev dependencies:
- flutter_launcher_icons 0.7.5 [image args yaml]

transitive dependencies:
- archive 2.0.13 [crypto args path]
- args 1.6.0
- characters 1.1.0
- charcode 1.1.3
- clock 1.1.0
- collection 1.15.0
- convert 2.1.1 [charcode typed_data]
- crypto 2.1.5 [collection convert typed_data]
- device_info_platform_interface 1.0.1 [flutter meta plugin_platform_interface]
- ffi 0.1.3
- file 6.1.0 [meta path]
- flutter_web_plugins 0.0.0 [flutter js characters collection meta typed_data vector_math]
- http_parser 3.1.4 [charcode collection source_span string_scanner typed_data]
- image 2.1.18 [archive xml meta]
- intl 0.17.0 [clock path]
- js 0.6.3
- meta 1.3.0
- nested 0.0.4 [flutter]
- path 1.8.0
- path_provider_linux 0.0.1+2 [path xdg_directories path_provider_platform_interface flutter]
- path_provider_macos 0.0.4+6 [flutter]
- path_provider_platform_interface 1.0.4 [flutter meta platform plugin_platform_interface]
- path_provider_windows 0.0.4+3 [path_provider_platform_interface meta path flutter ffi win32]
- pedantic 1.9.2 [meta]
- petitparser 3.1.0 [meta]
- platform 3.0.0
- plugin_platform_interface 1.0.3 [meta]
- process 4.2.1 [file path platform]
- shared_preferences_linux 0.0.2+4 [file flutter meta path path_provider_linux shared_preferences_platform_interface]
- shared_preferences_macos 0.0.1+11 [shared_preferences_platform_interface flutter]
- shared_preferences_platform_interface 1.0.4 [meta flutter]
- shared_preferences_web 0.1.2+7 [shared_preferences_platform_interface flutter flutter_web_plugins meta]
- shared_preferences_windows 0.0.1+3 [shared_preferences_platform_interface flutter ffi file meta path path_provider_platform_interface path_provider_windows]
- sky_engine 0.0.99
- source_span 1.7.0 [charcode collection meta path term_glyph]
- sqflite_common 1.0.2+1 [synchronized path meta]
- string_scanner 1.0.5 [charcode meta source_span]
- synchronized 2.2.0+2
- term_glyph 1.1.0
- typed_data 1.3.0 [collection]
- url_launcher_linux 0.0.1+4 [flutter]
- url_launcher_macos 0.0.1+9 [flutter]
- url_launcher_platform_interface 1.0.9 [flutter meta plugin_platform_interface]
- url_launcher_web 0.1.5+1 [url_launcher_platform_interface flutter flutter_web_plugins meta]
- url_launcher_windows 0.0.1+3 [flutter]
- vector_math 2.1.0
- win32 1.7.3 [ffi]
- xdg_directories 0.1.2 [meta path process]
- xml 4.5.1 [collection convert meta petitparser]
- yaml 2.2.1 [charcode collection string_scanner source_span]

same here , looking for any solution , it happened with me after upgrading to flutter 2

Can you share the crash logs?

solved for me after flutter 2.2.0 update


/etc/init.d/ custom script not executed at boot

I want to use this image to build an XAgent (www.xqual.com)
I managed to define the dockerfile in a correct way and everthing's there when I boot the docker image.
I added a script called S50xagent in /etc/init.d and made it executable.
However the script is not triggered by rcS. It seems that even rcS is not executed at all.
I added the command touch /tmp/thisworks to the rcS script, but after boot, there's nothing as such in the /tmp folder.
How can I launch an application at boot in this image? the xagent daemon script will register the docker image to the XStudio server instance.
manual launch of the script works fine. The agent shows up in the server instance...

 You'll have to figure that out yourself to run systems that require/assume an init system.


Tailwind Classes not compiled using ViewComponents

I have been testing ViewComponts and I found this ViewComponts + Tailiwnd guide.
Example
The guide uses a CDN to use tailwind and everything works fine.
My problem is that using tailwindcss-rails the tailwind classes that are injected from a component are not rendered in the view.
I start server from Procfile

My tailwind.config:

My component:

Component view:

Error:

Classes are loaded in Html but not in css
I know where the error is but I can't find a solution without having to use a tailwind CDN.
PD: I have also used rails assets:clobber

This seems like tailwind not recognizing those classes are being used and removing them from the bundle. I think you also need to tell tailwind to check the component ruby files in the content: config in your tailwind.config

https://tailwindcss.com/docs/content-configuration#configuring-source-paths

Make sure to add the components folder in the paths array to Tailwind config, as seen on your second screenshot.
config/tailwind.config.js
module.exports = {
  content: [
    './public/*.html',
    './app/helpers/**/*.rb',
    './app/javascript/**/*.js',
    './app/views/**/*.{erb,haml,html,slim}',
    './app/components/**/*' # This is the new Entry
  ],


Vue not defined in "https://vue-test-utils.vuejs.org/guides/#using-nexttick"

Version
1.0.0-beta.31
Reproduction link
https://github.com/vuejs/vue-test-utils/blob/dev/docs/guides/testing-async-components.md
Steps to reproduce
Vue is not defined in the testing.js file.
What is expected?
Vue should be defined or there should be a reference to the $vm
What is actually happening?
If we run npm run test by copying and adding the code snippet, it throws error

You will need to do import Vue from 'vue' in the unit test at the top.
If you can submit a PR, that'd be great.
The current docs are not ideal for this reason - they do not always have complete examples. We are in the process of redoing the docs, which will be aimed at supporting Vue 3. If this is something you are in interested in contributing to, reach out on Vueland Discord.


Dependencies problem

Kindly, Can you provide the links of all dependencies for this library to run??

hi @iAyubAnsari thanks for asking. this repo is not updated during the last 3 years, sorry about that. but answering your question, you need only sshpass and rsync commands basically. in the remote server you need basically mysql, mysqldump and gzip. thanks!


added character count feature

No description provided.

@himanshujaidka We need a word count feature. So if an average word is 7 letters long, so if we allow 10000 words (i.e. a length of medium story). it should be 70,000 letters and note we have not taken into account the whitespace. Please fix this. And only provide word count, if you want to. Showing Character count is not of much use.
A simple logic for word count will be:
- Check if there are two or more consecutive spaces(replace that by one)
- Count number of comma(s)
- Count number of fullstop(s)
-  Count number of semi-colon(s)
- Count the number of spaces
- Sum them the above
- add 1(for the last word)
- Now for every fullstop ('.') or comma(',') or semicolon(';') followed by a space reduce the count by one.

    Eg: I am a dedicated, practical and collaborative developer

A reference to a similar (yet incomplete implementation is): https://www.w3resource.com/javascript-exercises/javascript-regexp-exercise-6.php
Please document your code more

Do not put this a separate tab, just but it below the input area. Check out how we have it for twitter

Don't forget to link the Issue Numbers

ok got this @smaranjitghose

@himanshujaidka any updates?


Expand output by default if only one function

In the output, if only a single function is profiled, I think it would be nice if it was expanded by default.

:+1:


Cannot get render tree in profile build

When executing the command driver.executeScript("flutter:getRenderTree"), the driver returns an empty string in a profile build of the app.  Debug apps return the full render tree.

The same holds true for driver.executeScript("flutter:getRenderObjectDiagnostics", element, map).

Issue reported in Flutter project. #70772


clover.Open("path") doesn't make the directory automatically if not found in clover/v2

package main
import (
	//c1 "github.com/ostafen/clover"
	c2 "github.com/ostafen/clover/v2"
        "log"
)
func main() {
	//db1, err := c1.Open("clover-db") // this works fine and create it automatically if not found
	db2, err := c2.Open("clover-db2")// this will complain that clover-db2 directory not found
	if err != nil {
		log.Fatal(err)
	}
}

Hi, i was trying to use clover a bit for testing purposes but  when trying to open database using the first version it works fine and create directory for database automatically.
but when using v2 it doesn't work that way as it doesn't make the directory automatically and gives this error if not created beforehand.
2024/09/23 21:57:23 open clover-db2/data.db: no such file or directory
so i wonder what is the difference between the two versions?

In the V2, boltdb is used by default. It looks like the bolt driver doesn't automatically create the folder, so just make sure it exists before opening your database

On 24/09/23 12:11PM, Ahmed Hossam wrote:

so i wonder what is the difference between the two versions?

did you update your v2 clover? seems #123 solved for that. :smile:
--
shane.xb.qian

In the V2, boltdb is used by default. It looks like the bolt driver doesn't automatically create the folder, so just make sure it exists before opening your database

I see, so that's how it works.

On 24/09/23 12:11PM, Ahmed Hossam wrote: so i wonder what is the difference between the two versions?
did you update your v2 clover? seems #123 solved for that. 😄
…
-- shane.xb.qian

@Shane-XB-Qian , how can i do that ? i just did go get github.com/ostafen/clover/v2 wouldn't it get the latest?


database restore point swagger and examples

moved from https://github.com/Azure/azure-rest-api-specs-pr/pull/374

Automation for azure-sdk-for-python

@sarvesh-s FYI, merged, thanks again for your efforts! 🙂

Swagger to SDK encountered an unknown error: (Azure/azure-sdk-for-go)
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/github_tools.py", line 29, in exception_to_github
    yield context
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/restapi/github.py", line 180, in rest_handle_action
    return rest_pull_close(body, github_con, restapi_repo, sdk_pr_target_repo, sdkbase)
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/restapi/github.py", line 311, in rest_pull_close
    rest_pr.create_issue_comment("Was unable to create SDK %s PR for this closed PR.", sdkid)
TypeError: create_issue_comment() takes 2 positional arguments but 3 were given


Email verification

In the documentation, it states that verification emails will go to:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>However, if I am using domain name xyz.your-domain-name.com (a subdomain), will the emails go to<EMAIL_ADDRESS>or<EMAIL_ADDRESS>I believe it is the latter, but would like to clarify and it would be good to clarify this in the article too
Thanks,
Adam

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

ID: beac0740-af6e-61ce-bcfa-e783ca181b7d
Version Independent ID: d97885ea-b051-5fde-bd14-ec977e34760e
Content: Tutorial - Configure HTTPS on an Azure CDN custom domain
Content Source: articles/cdn/cdn-custom-ssl.md
Service: cdn
GitHub Login: @mdgattuso
Microsoft Alias: magattus

@AdamSharif-MSFT Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@AdamSharif-MSFT  Verification emails can be handled differently depending on the certificate provider and some may or may not support sub-domain email addressing.  DigiCert as used in the article appears to only support domain emails (or a phone number as an alternative), see https://www.digicert.com/ssl-validation-process.htm.

The experience of email validation is poor. I have a WebApp up and running with an A record for www.mydomain.com so needed to complete setup using the cdnverify.www method with email for SSL, prior to switching my www A record for the requisite CDN CNAME .  I did not have MX records defined for www as I imagine most people don't either. Having configured MX and an email account, deleted the CDN endpoint to retry SSL validation I now find myself with an 8 hour wait for the previous failed validation to unwind. I have three requests please 1.) the documentation should make it abundantly clear that you must have email delivery for the subdomain already configured or face 8 hours delay before retry. 2.) Azure portal should prompt / display the exact target email addresses including the subdomain prior to proceeding . 3.) It should be possible to backout and retry SSL validation much faster than 8 hours.  Thanks.


pkg/oci: Use Singleton for getLocalOciStore

pkg/oci: Use Singleton for getLocalOciStore
Concurrent calls to different oci.Store instances leads to race condition on the files that are written to the disk. oci.Store is goroutine safe only against its own instance.
cc @mqasimsarfraz

What happens when multiple instances of ig are run in parallel? Is there also a race condition in that case?

What happens when multiple instances of ig are run in parallel? Is there also a race condition in that case?

Correct. And AFAIK that is not in the scope of oras-go. So we would need to implement that with e.g. filesystem locking or lock files

"Multi-process support is also another non-trivial work and it is not on the roadmap of oras-go as it requires an atomicity source such as databases, file systems with O_EXCL support, and etc." - https://github.com/oras-project/oras-go/issues/472#issuecomment-1876251180


Stuck in a logging in loop

I am using 578 version and it's constantly saying log in failed.

Old builds of Roblox are likely broken due to protocol changes. Not much I can do unfortunately.


Detected Buildpacks added to CF Exporter

I noticed that if the buildpack is not manually selected in the cf push or in the manifest, prometheus picks it up as "buildpack" and not the actual buildpack name
This pr should allow the cf_exporter to return detected buildpacks (selected by CF) instead of listing them generically as 'buildpack'

Thanks @jccarte!


define entity keys that are not resolver input args

Is there a way to define entity keys that are not an arg for a resolver?
I have a use case for an id mapper where the arg for the resolver is to take and generic id and return the map of all ids associated with it. each of the associated ids are the actual keys that the subsequent subgraphs will join on.

As below, size and weight_in_pounds come from another subgraph?
#[Object]
impl Query {
  #[graphql(entity)]
  async fn find_product_by_id(
    &self,
    #[graphql(key)] id: ID,
    size: Option<u32>,
    weight_in_pounds: Option<u32>
  ) -> Product {
    Product {
      id,
      size: size.unwrap_or_default(),
      weight_in_pounds: weight_in_pounds.unwrap_or_default(),
    }
  }
}

@sunli829 thank you this I found in the docs
here is my use case
subgraph A
type AA {
idA
idB
idC
}
resolver(id:number): AA (this searches all idA, idB, idC to find the record that matches)
subgraph B
AA {
idC
BB
}
resolvers
AA {
BB({idX}) => getData(idX)
}
I hope this explains better.
I wish I could be more detailed but there is some proprietary information in the actual code


[gui error report] Error: Initializing daemon...

Desktop: 0.38.0
OS: win32 10.0.19045 x64
Electron: 31.0.1
Chrome: 126.0.6478.36

Error: Initializing daemon...
Kubo version: 0.30.0
Repo version: 16
System version: amd64/windows
Golang version: go1.22.7
Found outdated fs-repo, migrations need to be run.
Looking for suitable migration binaries.
Need 1 migrations, downloading.
Downloading migration: fs-repo-15-to-16...
Fetching with HTTP: "https://trustless-gateway.link/ipfs/QmRzRGJEjYDfbHHaALnHBuhzzrkXGdwcPMrgd5fgM7hqbe/fs-repo-15-to-16/versions"
Fetching with HTTP: "https://trustless-gateway.link/ipfs/QmRzRGJEjYDfbHHaALnHBuhzzrkXGdwcPMrgd5fgM7hqbe/fs-repo-15-to-16/v1.0.1/fs-repo-15-to-16_v1.0.1_windows-amd64.zip"
Downloaded and unpacked migration: C:\Users\markr\AppData\Local\Temp\migrations1225523272\fs-repo-15-to-16.exe (v1.0.1)
Running migration fs-repo-15-to-16 ...
  => Running: C:\Users\markr\AppData\Local\Temp\migrations1225523272\fs-repo-15-to-16.exe -path=C:\Users\markr\.ipfs -verbose=true
The migrations of fs-repo failed:
  migration fs-repo-15-to-16 failed: fork/exec C:\Users\markr\AppData\Local\Temp\migrations1225523272\fs-repo-15-to-16.exe: The parameter is incorrect.
If you think this is a bug, please file an issue and include this whole log output.
  https://github.com/ipfs/fs-repo-migrations

Error: migration fs-repo-15-to-16 failed: fork/exec C:\Users\markr\AppDat

@GitMarkalsoHub the "The parameter is incorrect" usually means a problem with your Windows machine.
You could have a hardware or software error on the disk, virus infection, or broken antivirus software, or a driver.
Try

uninstalling ipfs-desktop
run Checkdisk against your C: drive and fix all errors it finds
manually install latest version

If this does not help, create support thread at https://discuss.ipfs.tech/c/help/13, perhaps other Windows users are able to help.


How to take numpy arrays as input not image files?

Checklist

I have searched related issues but cannot get the expected help.
I have read related documents and don't know what to do.

Describe the question you meet
Image file is value range 0 ~ 254 int type numpy arrays, and can be storaged as ".jpg, .png" image files.
However, I have CT images with value range -2000 ~ 4000, I can not save these into  ".jpg, .png" image files.
But mmclassification only supports image file as input.
Is there any way I can using numpy arrays as input?

You can do that by implement your own CTDataset refer to mmcls/datasets/cifar.py,  which put loading image input dataset. the transforms will be a little different, such as https://github.com/open-mmlab/mmclassification/blob/58b21ee56ff193dced1a02efa81e73bb92347029/configs/base/datasets/cifar10_bs16.py#L7-L14

Thank you so much!


Set babel cache directory inside build dir

This can potentially fix #3276 and other similar issues.
Use case it fixes:

Babel plugins do something that modifies files in .next directory (e.g. copies static files there)
Babel creates caches in node_modules/.cache
User makes some changes and next.js clears .next directory
Build runs again, but babel sees cache in node_modules/.cache. It skips running babel plugins that should modify files in .next
Suddenly the second time app is run .next is missing some files

I'm not 100% sure I'll fix things but has potential to make them better

I've checked it on local setup and it works wonderfully

Hmm it seems it might fix just use case when babel plugin outputs to .next. When webpack plugin outputs something, the behavious seems to be still unpredictable.


JavaScript

JavaScript
User Story
As a bootcamp student
I want the prework notes to be structured on a webpage
so that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I can see the four topics I learned
listed along with a suggestion on what I should study first

JavaScript file, coding and my notes added to my Prework Study Guide.


Remaining TCF UI components

Closes https://github.com/ethyca/fides/issues/3723
Description Of Changes
Adds generic UI components for more TCF components.
Here's a bunch of them combined into one view:

Code Changes

[x] Allow toggles to take children components so that we can nest more data inside of them

[x] Add a badge prop to toggles so that we can render a GVL badge

[x] Adds a "filter" button group

[x] Adds a cookies table

Steps to Confirm

Edit the fides-js-components-demo.html component so that tcfEnabled is true
Run turbo dev in privacy center
Visit /fides-js-components-demo.html and open the modal
See the hodge podge of UI components

Pre-Merge Checklist

[x] All CI Pipelines Succeeded
[x] Issue Requirements are Met
[x] Update CHANGELOG.md

Wow very nice @allisonking!


eosio-cpp error: fatal error: 'boost/hana.hpp' file not found

/usr/local/eosio.wasmsdk/bin/../include/eosiolib/multi_index.hpp:9:10: fatal error: 'boost/hana.hpp' file not found.
besides, i use cmake .  on "examples/hello/" which no error, but no wasm file generated.
what's the proper way to generate wasm file ?

Add TEMP_DIR='/tmp' to build.sh at line 12.

Resolved by PR #62


Support Catalyst

Do not call #available() when targeting catalyst
Using React Native with Catalyst is quite interesting, and being able to support BLE interactions from a React Native app on a MacBook is very exciting.
I've tried to do this today with a boilerplate react-native init app, and after installing react-native-ble-plx was greeted with the following error when building for catalyst.
Undefined symbol: Swift._stdlib_isVariantOSVersionAtLeast(Builtin.Word, Builtin.Word, Builtin.Word) -> Builtin.Int1

This seems to come from a few calls to #available. I'm not yet sure why this is not working, but seems the compiler cannot see these symbols when targeting catalyst.
This PR avoids checking for these iOS specific availability checks when targeting catalyst.
~Bluetooth is unsupported on catalyst~
~Secondly, which I hope to address is well for catalyst is the second problem when targeting catalyst.~
BluetoothLE is unsupported on this device

~This seems to come from call to centralManagerDidUpdateState in RxCBCentralManager.swift which for some reason returning unsupported.~
Just needed to check Bluetooth in App Sandbox under Signing & Capabilities and now it's working. 🙂

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

I do not have any iOS expert available at the moment – judging from this article #available() has macCatalyst target — shouldn't it be used?

It seems it's the call of #available() that fails, otherwise, for a platform that is not iOS 11.2 or later, the preexisting path should cover catalyst already? In this article Creating a Mac Version of Your iPad App it mentions only:

If you have source code referencing APIs unavailable to the Mac version of your app, enclose the code in a compilation conditional block that uses the targetEnvironment(): platform condition.

#if !targetEnvironment(macCatalyst)
// Code to exclude from Mac.
#endif

The bigger question that has come to me now is with the additional path of considering catalyst, if the previous values for iOS versions lower than iOS 11.2 are correct, for example MTU size?

These changes continue to work well with MultiPlatformBleAdapter when targeting catalyst. It would be helpful to get in 0.1.7 and used in react-native-ble-plx!


SAMZA-1977: Shutdown the container threadPool when stopping the AsyncRunLoop.

@xinyuiscool @sborya
Please take a look when you get a chance.

@shanthoosh what prompted this change?

@prateekm
When fixing SAMZA-1638, I noticed that the container threadPool in AsyncRunLoop was not shutdown at all.
Container thread pool is used to run window, commit, timer tasks in  AsyncRunLoop.  This will cause problems in standalone in the following scenario:

Due to a rebalance phase, StreamProcessor shuts down the SamzaContainer.
StreamProcessor receives a new JobModel (v2) and starts the SamzaContainer again.
Window, commit tasks from the unstopped container thread pool corresponding to  previous JobModel(v1) would still be running in the JVM.

I think this can cause duplicate processing  depending upon number of window tasks scheduled in the container threadPool before AsyncRunLoop shutdown.
Please correct me if I'm missing something here.

Does SamzaContainer.shutdownTask() shut them down?

@shanthoosh I understand, was just asking if you noticed any specific issues that lead to this change. Why is this closed? Is this not an issue anymore?


Add all inline actions to right-click menu

Changes

Fix download action which was missing from source physicals
Add all inline actions to the right-click menu (refer to example below)

I think the intent of upload/download was from your PC and not the workspace.  But from Merlin, these actions cannot access the local PC


Integrate events with subgraph

replaced requesting create and close product events with querying from graphql
removed retry
removed duplicated discount request from getProductListRequest (we already request it in getProductDetailsFromIds)

Previously we had some logic that incrementally loaded the events in chunks, without me getting into the code can you confirm that that's been removed

@oaksprout  yes that was removed, now instead of fetching events by blocks we do one call to the subgraph for createProduct events and another one for closeEvents, you can see the difference in number of calls on attached screens, on the bottom left. there are still other calls to get needed data such as current price, and for that sometimes balancers respond very long (3-5+ sec), I can investigate if that can be improved somehow
Old: 129 requests

New: 98 requests


New NGE node: Mapping

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14215/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14215/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14215/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14215/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14215/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14215/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14215/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14215/merge#BCU1XR#0

Visualization tests for webgl2 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14215/merge/testResults/webgl2/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for webgl1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14215/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for webgl1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14215/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for webgl2 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14215/merge/testResults/webgl2/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for webgl2 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14215/merge/testResults/webgl2/index.html
If tests were successful afterwards, this report might not be available anymore.

Visualization tests for webgl1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14215/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.


Remove template specification requirement before creating plaquette

Describe the bug
I shouldn't have to specify a template before I specify a plaquette.
Steps to reproduce the behavior:

Specify unit cell
I must specify template, when I should be able to specify a plaquette right away.

Expected behavior
Once the qubits have populated, I should be able to start clicking them and when 3+ have been clicked, specify a plaquette.
Screenshots and MP4s

Other context:
From discussions with @afowler .
Coding task
Bypass the template functions: hoist plaquette addition into the control flow function.

There might be another way to use the existing behavior, namely to specify
the plaquette footprint. Every plaquette should have the same footprint.
In our usual architecture the footprint is a square containing 5 qubits,
and all plaquettes would be defined on a subset of this.
On Fri, Mar 8, 2024, 12:50 PM Sam Burdick @.***> wrote:

Describe the bug
I shouldn't have to specify a template before I specify a plaquette.
Steps to reproduce the behavior:
1.Specify unit cell
1.I must specify template, when I should be able to specify a plaquette
right away.
Expected behavior
Once the qubits have populated, I should be able to start clicking them
and when 3+ have been clicked, specify a plaquette.
Screenshots and MP4s
image.png (view on web)
https://github.com/QCHackers/tqec/assets/16883876/35941c6c-1fff-49a9-b481-e2c556c32992
Other context:
From discussions with @afowler https://github.com/afowler .
Coding task
Bypass the template functions: hoist plaquette addition into the control
flow function.
—
Reply to this email directly, view it on GitHub
https://github.com/QCHackers/tqec/issues/186, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKAXTGYDBCVKGV2S427MBDYXIQCNAVCNFSM6AAAAABENNJ2MCVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE3TMNZUGY3DKNQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

@afowler

There might be another way to use the existing behavior,

I think templating should be have more granularity than how it is now, which is a connected rectangular subset of the qubits. We can keep the template class, and it will have to change to meet the needs of tasks 4 and 5.

namely to specify the plaquette footprint. Every plaquette should have the same footprint.  In our usual architecture the footprint is a square containing 5 qubits, and all plaquettes would be defined on a subset of this.

We should have a "Create square plaquette" and "create triangular plaquette" buttons, which can be recolored, rotated, translated, etc, and users can define circuits on top of them [Task 3]

In my opinion you should start without the ability to rotate plaquettes, I
think that will be quite hard to do in general. I'm ok with having to build
a plaquette for each orientation, particularly since the timing of the
gates will be different for different orientations.
On Fri, Mar 8, 2024, 1:42 PM Sam Burdick @.***> wrote:

@afowler https://github.com/afowler
There might be another way to use the existing behavior,
I think templating should be have more granularity than how it is now,
which is a connected rectangular subset of the qubits. We can keep the
template class, and it will have to change to meet the needs of tasks 4 and
5.
namely to specify the plaquette footprint. Every plaquette should have the
same footprint. In our usual architecture the footprint is a square
containing 5 qubits, and all plaquettes would be defined on a subset of
this.
We should have a "Create square plaquette" and "create triangular
plaquette" buttons, which can be recolored, rotated, translated, etc, and
users can define circuits on top of them [Task 3]
—
Reply to this email directly, view it on GitHub
https://github.com/QCHackers/tqec/issues/186#issuecomment-1986460370,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKAXTF25DATK3MDS5NDDCTYXIWETAVCNFSM6AAAAABENNJ2MCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBWGQ3DAMZXGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

In my opinion you should start without the ability to rotate plaquettes, I
think that will be quite hard to do in general.

@afowler I feel that depends on what angles you can rotate by. Right now, it's only 90 degrees. What other options do we want to provide?

I don't think we should even provide that in the beginning as I don't think
rotating will be very helpful.
On Fri, Mar 8, 2024, 2:30 PM Sam Burdick @.***> wrote:

In my opinion you should start without the ability to rotate plaquettes, I
think that will be quite hard to do in general.
@afowler https://github.com/afowler I feel that depends on what angles
you can rotate by. Right now, it's only 90 degrees. What other options do
we want to provide?
—
Reply to this email directly, view it on GitHub
https://github.com/QCHackers/tqec/issues/186#issuecomment-1986515226,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKAXTGLI26JWNQMCDPICVLYXI32BAVCNFSM6AAAAABENNJ2MCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBWGUYTKMRSGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Going to close this one out, and move the rotation and other options to another issue.

Will be presented on Wednesday meeting.


Gitify open on middle of double screen on Windows

🔍 Is there already an issue for your problem?

[X] I have checked older issues, open and closed

📝 Description
When first opened Gitify open somewhere in the middle of my 2 screens on my Windows 11 dual screen setup, as you can see in the picture below:

For some reason if I minimize it and open it again it is where it should (on top on the Windows bar, above the notification area, aka bottom right corner on my primary screen)
Here is my dual screen setup:

For details this rather a 3x screen setup where both screen (1 and 3) are duplicated. This is because I have a laptop that sit somewhere and 2 standalone monitor on my desk. Every monitor is running at 1920x1080. There are different refresh rate for what it worth.
🪜 Steps To Reproduce

Have a dual screen setup as described in the description.
Open Gitify

Gitify Version
5.6.0
Operating System
Windows
GitHub Account
GitHub.com
📸 Screenshots
No response

Closing as should be resolved in #1193

Reported as still occurring in 5.8.1

@camille-avatarmedical  - this looks like a potential fix
https://github.com/max-mapper/menubar/issues/435#issuecomment-1581628798

I've raised an upstream fix 🤞 https://github.com/max-mapper/menubar/pull/479

@camille-avatarmedical - would appreciate if you could test this pre-release build: https://github.com/gitify-app/gitify/actions/runs/9538457506 🙏


[Bug] Filters order is important?

Hi @vladkens,
Just tested v0.6.0, filtering to be specific. Either I found a bug or didn't understood docs or docs require clarification.
Test case 1
GHS_FILTER=t0mmili/*,!t0mmili/taiko-ci
OK. Shows all my public repos except taiko-ci.
Test case 2
GHS_FILTER=!t0mmili/taiko-ci,t0mmili/*
WRONG. Shows all my public repos, including other orgs I'm part of.
Test case 3
GHS_FILTER=t0mmili/*,!fork
OK. Shows all my public repos except forks.
Test case 4
GHS_FILTER=!fork,t0mmili/*
WRONG. Shows all my public repos, including other orgs I'm part of.
So, generally, it looks like negations should be placed in the end. Please clarify, what's the situation here.

Hi, @t0mmili. Order shouldn't have relevance. I added test here and it passed

I did further digging.
First, I compiled app locally and tried above test cases. Order is not relevant 😮
Next I went back to original test, which was done with docker image running behind reverse proxy. It still fails when negation is first...
I will try to inspect this further and leave some updates here.

@vladkens problem solved!
Previously my GHS_FILTER value didn't have quotations. This is correct and works ok:
    environment:
      GHS_FILTER: "!fork,t0mmili/*"

Hi, @t0mmili. Here an article about: https://binfalse.de/2024/05/31/docker-compose-quotation-pitfalls/
If you use KV environments format in compose.yaml, complex strings should be escaped with ". This is related to YAML parser

Thanks @vladkens, I did know that, just had a memory fog somehow 😄 Closing this one.


Support multiple dublin-core style schemata when reading SAF.  Consis…

…tent formatting for prefixes in SPARQL updates

Coverage decreased (-0.07%) to 26.392% when pulling 0fbb0d160dc9c51ef0c23d8c71bfd10881211bd5 on support-multiple-schemata-SAF into 6432f34d3589179212b973386c869c17021e82d5 on master.


chore: <IP_ADDRESS> changelog

Description
Add changelog for <IP_ADDRESS> release.
DOCU-3403
Testing instructions
Preview link: https://deploy-preview-6186--kongdocs.netlify.app/gateway/changelog/#3410
Checklist

[x] Review label added
[x] PR pointed to correct branch (main for immediate publishing, or a release branch: e.g. release/gateway-3.2, release/deck-1.17)

@outsinre , a few questions:

are there other dependencies like bumped versions to include?
the graphQL fix that's listed as a breaking change: what's the customer impact?
for the new expressions route feature, where can I find the docs that explain how to use? I checked https://github.com/Kong/kong/pull/11348 but didn't see there.

@outsinre , a few questions:
* are there other dependencies like bumped versions to include?

* the graphQL fix that's listed as a breaking change: what's the customer impact?

* for the new expressions route feature, where can I find the docs that explain how to use? I checked [feat(router): support http queries in expression kong#11348](https://github.com/Kong/kong/pull/11348) but didn't see there.

Hello @acgoldsmith ,

Please refer to https://github.com/Kong/kong-ee/pull/6649/ or https://github.com/Kong/kong-ee/blob/next/3.4.x.x/changelog/<IP_ADDRESS>/<IP_ADDRESS>.md on <IP_ADDRESS> changelog.
Regarding https://github.com/Kong/kong/pull/11348, I will ping @chronolaw to add a new user-facing doc PR.

@acgoldsmith This is @chronolaw 's router user-facing PR: https://github.com/Kong/docs.konghq.com/pull/6201


789 ens support

#789

React Queries

[x] addressToENS
[x] ENSToAddress

Update components

[x] Update RightMenuWallet
[x] Explorer: validate valid input (ENS or address)
[x] Explorer: search input field keeps ENS input

Others

[x] Check address is valid in addressToENS query
[x] Check ENS is valid in ENSToAddress query
[x] Support emojis for ENS domains (as well as emojis with longer sizes)
[x] Issue with explorer/wallet/{ens.eth} slug

explorer/wallet/address.eth doesn't refresh but explorer/wallet/address.eth/ works
Related issue: https://github.com/vitejs/vite/issues/2245

As part of the Frontend guidelines from ENS

Should use iconify or blockies to identify address if it is not shown fully to the user
Should add a tooltip or in the wallet modal show the full address to check the user is connected with the correct wallet
Should trim ENS name differently than address (trim from the end)

Useful addresses to test
0xkeyrock.eth = 0x1855f41B8A86e701E33199DE7C25d3e3830698ba
binance (no ENS) = 0x4a3ca0cb4dcf6bc573ca4b538cab5fb0e01b5f19
bancorians.eth = 0x6439abb662e3E5F0eb95DcD25A576A722383d8d2
bancoralchemist.eth = 0x21dBF2e0a5f377F7098439B64fb5A67a5d18AF45
😭😭😭.eth = 0xD75337ceF33ace428a18533A1fb791aCB1Eff239

[ ] ens address validation
open explorer
type any string sdkjfhskjhfs
see invalid wallet error message
add .[any letter]
expected: see the error if the ens name is invalid (doesnt exist or doesnt represent a wallet)
actual: there is no error message.

add .[any letter]
expected: see the error if the ens name is invalid (doesnt exist or doesnt represent a wallet)
actual: there is no error message.

As per the ENS frontend guidelines, ENS domains can be .xyz, .eth, .luxe, .kred, .art, .club, etc:

Not all ENS names end with .eth: ENS names normally end with .eth. However other top-level domains (TLD) are currently supported (.xyz, .luxe, .kred, .art, .club) and more will be in the future. Consider this if you are thinking about displaying the TLD part in the truncated view of long names.

Can change the regular expression check for only the domains from the list above, but it might restrict future ENS name endings.

Code LGTM - but please make sure you "audit" every new dependency you add (graphemer) - that it's not on any risk list and that it's maintained

Thanks, will do!


Build FreeNAS 9.3

This covers the basic task of building an installable ISO of FreeNAS 9.3, the stock version

Assigned to Jakub


board: nsim: doc update for debugging section

board: nsim: doc update for debugging section
doc update for debugging dection for #17547 issue.
Signed-off-by: Watson Zeng<EMAIL_ADDRESS>the simulator wil use console for output, to avoid sharing with the gdb debug console, which will make output promiscuous, we can not directly use the make debug command for debugging.
Fixes #17547

@ioannisg @galak ping for merge

It doesn't look like you took my suggested changes with grammar and typo fixes?

sorry, my fault, I forgot it.
Now, it's fixed.

@dbkinder can you re-visit


[DOCFIX] fix cn docs Running-Hive-with-Alluxio.md and Alluxio-Storage.md

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/18830/
Test PASSed.


Allow HTTPS only for fetching JWKS

Only allow issuerUrls with https protocol
Make sure X509 certs are validated when establishing a TLS connection

Done with https://github.com/janlisse/zio-jwt-validator/pull/6


[IBCDPE-918] PEGS Challenge Profile

problem
The PEGS challenge profile needs to be configured such that MODEL_TO_DATA.nf can make required communications with the correct Challenge project on Synapse.
solution
2 new profiles are added: One for round 1 of the model-to-data evaluation (validation), and one for round 2 (testing). These profiles are configured using the new parameters introduced to MODEL_TO_DATA.nf
testing & preview

no happy for the participants to see the log. Yes the staging site is only for organizers, the real site is this one https://www.synapse.org/#!Synapse:syn52817032/wiki/624336


Bump default version of Swift to 5.10.1

Verified manually that swift-sdk-generator itself can be cross-compiled when using https://github.com/swiftlang/swift-sdk-generator/pull/127.

cc @euanh

@swift-ci test

@swift-ci test


how do I get imgproc?

I've just downloaded this package and I'm doing some code but I'm not able to find "Imgproc". How do I import it?

Not sure where "Imgproc" is used. Is it throwing an error if your testing?

No error, I'm doing a code to change the color of certain sections of an image and for that I need Imgproc.

I manage to workaround that, now I get a "mat.push_back is not a function" message. My code looks like this:
let mat = new cv.Mat(1, 1, cv.CV_32SC3)
mat.push_back(0, item[0])
where item is an array of number.

See document https://docs.opencv.org/3.4/de/d06/tutorial_js_basic_ops.html
I think you can try things like mat.data[0] = item[0]


Applying a bottom margin to a TextBlock results in an unhandled Win32 exception

Trying to render
<TextBlock
  text="USPS"
  margin={{top: 0, right: 0, bottom: 8, left: 0}}
/>

results in an unhandled Win32 exception. Unfortunately my Visual Studio skills aren't good enough for me to figure out exactly what that exception is.

looks like the xaml Margin property is colliding with the react native core margin property which only supports numbers or strings.
Going to discuss this more in microsoft/react-native-windows#8158

I'm sure it's the same issue, but <TextBlock style={{marginBottom: 8}} text="USPS" /> also results in a crash. Just thought I'd mention it in case that's significant.

Actually that's a similar but slightly different issue - in that case it looks like RNW is sending the marginBottom property to the react-native-xaml viewmanager, without context that it should be inside of a style object

This should be fixed in recent versions, we will only assert when we find a property we don't recognize if we are running under the debugger (e.g. in VS). See https://github.com/asklar/react-native-xaml/blob/main/package/windows/ReactNativeXaml/XamlViewManager.cpp#L77


the "tcore" dependency is missing after a fresh clone

where can i get me some of that sweet topt-core code? the ClassCollection class is missing from the collection of class collections of collected classes.
i was trying to $ npm run build
https://paste.ubuntu.com/p/yVNVBTfzDx/

Did ya use npm install topt-core?
On Wed, Dec 30, 2020, 9:21 AM sloph<EMAIL_ADDRESS>wrote:

where can i get me some of that sweet topt-core code? the ClassCollection
class is missing from the collection of class collections of collected
classes.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Varunda/topt/issues/11, or unsubscribe
https://github.com/notifications/unsubscribe-auth/APY44DXLUAON7V3AHX5MPRLSXNOTJANCNFSM4VORCTMA
.

bro dude bro i had no idea you created an npm repo sick bro
we should modify package.json to include it
i ran npm install and that should have pulled in all the deps
right?

bro dude bro i had no idea you created an npm repo sick bro
we should modify package.json to include it
i ran npm install and that should have pulled in all the deps
right?

Probably? Idk I use npm link which messes with it I think
On Sat, Jan 2, 2021, 4:57 PM chunk<EMAIL_ADDRESS>wrote:

bro dude bro i had no idea you created an npm repo sick bro
we should modify package.json to include it
i ran npm install and that should have pulled in all the deps
right?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Varunda/topt/issues/11#issuecomment-753550803, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/APY44DXMX5ZX4LWLTXQMOU3SX66HVANCNFSM4VORCTMA
.

Probably? Idk I use npm link which messes with it I think
On Sat, Jan 2, 2021, 4:57 PM chunk<EMAIL_ADDRESS>wrote:

bro dude bro i had no idea you created an npm repo sick bro
we should modify package.json to include it
i ran npm install and that should have pulled in all the deps
right?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Varunda/topt/issues/11#issuecomment-753550803, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/APY44DXMX5ZX4LWLTXQMOU3SX66HVANCNFSM4VORCTMA
.


Mardown links

My README.md has a section linking to internal files like this:
 ## Resources
- [Class diagram - full](diagram_full.pdf)
- [Class diagram - simplified](diagram_simple.pdf)
- [Test coverage](coverage/index.html)

Now the funny thing is that although everything works just fine when I call yard from the command line, the generate links are different if the command was built from a Rakefile
From the command line, the links are like this:
Class diagram - full

From the Rakefile (notice it is adding 'http://')
Class diagram - full

I tested that is the output generating the docs with the task helper:
YARD::Rake::YardocTask.new

and with a shell command:
task 'yard' do
    sh 'yard'
end

My .yardopts has nothing special to it:
--no-private
lib/**/*.rb -

Closing out old issues.
I realize this is very old, but my guess is that this is due to an environment issue, not specific to YARD. YARD itself can't tell the difference between the Rake task and running the binary directly, those differences would be related to the environment that you're running YARD in. Not sure if you're even having this problem anymore, so I will be closing this. If you really are still struggling with this, do feel free to re-open.


uninitialized constant Kubeclient::GoogleApplicationDefaultCredentials

Hey:
require 'helpers/tcp'
require 'kubeclient'
require 'googleauth'

RSpec.describe '#at-to-gcp' do                                                                                before do
  auth_options = {
        bearer_token: Kubeclient::GoogleApplicationDefaultCredentials.token
  }
end

Gives me:
  1) #at-to-gcp when the echo-server service is running should be able to establish a tcp connection
     Failure/Error: bearer_token: Kubeclient::GoogleApplicationDefaultCredentials.token

     NameError:
       uninitialized constant Kubeclient::GoogleApplicationDefaultCredentials
     # ./spec/at_to_gcp.spec:9:in `block (2 levels) in <top (required)>'

Gemfile:
❯ cat Gemfile
# frozen_string_literal: true

source "https://rubygems.org"

git_source(:github) {|repo_name| "https://github.com/#{repo_name}" }

gem 'rspec'
gem 'kubeclient'
gem 'googleauth'

any ideas?

As far as I can tell we have no released version with the feature you are using.
@Stono are you looking to have this feature included in a release?

The README at github reflects master. If you want to view the README for the version you are using it's best to view the one distributed with the gem.
Anyway I understand a new release is required. I will try to get to that tomorrow. Please bug me if you get no response.
cc @cben

Ah that makes sense, thank you :-) I've worked around it for now so no rush:
module Helpers
  module Kubernetes
    def cluster(project_id, zone, cluster_id)
      return @cluster if not @cluster.nil?

      scopes = [
        'https://www.googleapis.com/auth/cloud-platform'
      ]
      service = Google::Apis::ContainerV1::ContainerService.new
      service.authorization = Google::Auth.get_application_default(scopes)

      cluster_info = service.get_zone_cluster(project_id, zone, cluster_id).to_h
      master_endpoint = "https://#{cluster_info[:endpoint]}"

      cert_store = OpenSSL::X509::Store.new
      master_ca = cluster_info[:master_auth][:cluster_ca_certificate]
      master_ca = Base64.decode64(master_ca)
      cert_store.add_cert(OpenSSL::X509::Certificate.new(master_ca))

      ssl_options = {
        cert_store: cert_store,
        verify_ssl: OpenSSL::SSL::VERIFY_PEER
      }

      log("Using #{master_endpoint} as Kubernetes host")
      auth_options = {
        bearer_token: service.authorization.access_token
      }

      @cluster = Kubeclient::Client.new(
        master_endpoint,
        'v1',
        auth_options: auth_options,
        ssl_options: ssl_options
      )
      @cluster
    end
  end
end

We just released 3.1.0 that should include this (changelog: #328).  Please test.

Closing as I believe this is solved — 3.1.* releases contain Kubeclient::GoogleApplicationDefaultCredentials.


Merge pull request #62 from aleksn41/master

update UI StatisticRequest Layout

this was not opened on purpose

this was not opened on purpose


Add pggen to p3skb

A convenient place to track which PPGen was used to lock the user's secret key
For use in keybase/keybase#292.
Cc: @oconnor663 for code review

Besides above comment and the resulting test change, this looks :+1:


Another has-part filter?

https://github.com/NCATSTranslator/Feedback/issues/778
In this case we have
A-partof->B<-partof-A-treats->X
But if we already know A treats X then what is this for?
Not sure if this is somehow made worse by conflation

So the basic idea is that we have a rule that says
A-partof->B<-partof-C-treats->X.  And the assumption is A != C.  But that's not explicit, so sometimes we get trivial results.
How much is the actual learning of this result based on these trivial results?  Based on the answer to that, this could be a worse rule than we thought...

We reviewed the use of has_part.  The robokop particulars that led to the use of this rule were not very impressive so we decided to remove it.


Pile dataset is no longer available..

Hi, thank you for the awesome work!
However, when I tried to run some experiments with your code, I noticed that the Pile dataset is no longer available on their website. I'm wondering if you know any alternative methods to get this dataset.
Thank you for your help!

Or, alternatively, is there a link you can share with us to download the PILE dataset other than the original source?

Hi, all. I also found this issue that the Eleuther AI just put the Pile dataset offline. But I found that there is still an available resources on Huggingface dataset. Please check: https://huggingface.co/datasets/EleutherAI/raw_deduplicated_pile/tree/main


missing response stream error events

The response stream is setup as a series of pipes, but node-fetch only listens for errors on the end of the pipe. Pipes don't send error events along the way. We've seen expected end of JSON input errors which are in fact probably caused by ECONNRESET's which are not surfacing because of this issue.

You will have to give us a concrete example, we handle errors on this 2 level (response and response body).

https://github.com/bitinn/node-fetch/blob/master/src/index.js#L62-L65
https://github.com/bitinn/node-fetch/blob/master/src/body.js#L197-L199

And you can see timeout handling on the top.

Great, sorry for not being more clear. The problem is here (and subsequent pipes related to compression):
https://github.com/bitinn/node-fetch/blob/master/src/index.js#L118
It's my understanding that the pipe won't forward error events:
https://groups.google.com/forum/#!msg/nodejs/lJYT9hZxFu0/L59CFbqWGyYJ
As a result, the response body error listener won't actually pick up error events from the response  stream source.

@randypuro Thx for the explanation, could you help to provide a failing test case?
Our test cases are pretty easy to work on:

you setup a simple server route that simulate what you experience, like this

then you try to access it and test for your expected result, like this

You just need to figure out how to replicate your situation.

Unfortunately, the node api isn't low level enough to simulate some of the use cases that might occur in the wild (eg. TCP level connection reset while reading data from the socket of a response stream which results in a read ECONNRESET).  Even calling socket.destroy() from the node server perspective won't trigger those same tcp level errors you might see from the client perspective I'm not sure how to help reproduce it, but we've seen the same issue, when using the request library, reported as a ECONNRESET whereas now using node-fetch, we see a seemingly successful request, but with an incomplete/partial response.

Alright so the only safe way to ensure we capture all error events relating to network is to listen to the initial res body.
It might take a while before we can properly look into this. Are you using a standard JSON API or a JSON stream?

@bitinn Is there a reason why we are piping res to a new stream in the first place? I was never able to figure this one out.

@TimothyGu It's quite possible my initial design decision is to prevent users from accessing the raw http response. So that they stick to the Fetch API.
Now that I looked at it, I am not sure this issue can be nicely resolved if you also happen to .clone() the response, by definition it must be teed (piped).

We see this issue just using the standard .text() api.

Did anyone figure out a workaround. In some error case, if we pipe the stream, the response never comes and it hangs forever.

I just ran into this issue, and want to confirm that it's still happening in the latest release (2.6.0).
@atishay I worked around some connections hanging by setting a timeout and retrying, manually.

I think I've fixed this in #768


Include Dockstore CLI version

Tooltester was set up to only include the webservice version (previously we had CLI and webservice on the same repo).
It needs to somehow handle the CLI version (as a flag/parameter passed to Jenkins)
┆Issue is synchronized with this Jira Story
┆Issue Number: DOCK-2416
┆Sprint: Backlog
┆Issue Type: Story

Not sure this is relevant with the AGC WES-cli move


make sure http redirects to https

currently the redirect does not happen, which can result in the following issues:

Trying to login to prod with globus auth:
Mismatching redirect URI.

GitHub login results in

Social Network Login Failure
An error occurred while attempting to login via your social network account.

Code: unknown, Error: None

Closing as no longer relevant


peek method

Can we have a func peek<R>(fn: (inout T) - R) async method that allow me to inspect and potentially modify the next element (and await if needed) without receive it?
Use case: I am working with a networking library and it essentially I am sending Data to the channel. I sometimes need to receive a single byte instead of a Data chunk. It will be too inefficient to make UInt8 channel. But when I received a Data, I can't only take a single byte and put the rest back to the head of the queue. Right now I need to maintain another buffer for this purpose and makes the code more complicated.
Happy to make a PR if needed.

While I think this could be implemented conceptually. It disagrees with the design principals of a channel implementation. By allowing data to be mutated on the buffer, the contract between the sender and receiver can be broken.
Additionally in cases where there are many receivers, using such a function could cause unpredictable behavior.
However I am happy to help you come up with a cleaner implementation giving the problem you are facing.
Are you able to share a simple example demonstrating (or describe in more detail) the data flow and how you are trying to mutate it? Do you have multiple threads reading/writing to this data?
Perhaps using a ring buffer or similar data structure would be more appropriate?

The mutation should be protected by locks so I don't see how it could cause unpredictable behavior.
This is my code:
public final class Stream: Sendable {
    let stream: QuicStream
    private let channel: Channel<Data> = .init(capacity: 100)
    private let nextData: ThreadSafeContainer<Data?> = .init(nil)
    private let _status: ThreadSafeContainer<StreamStatus> = .init(.open)

    public private(set) var status: StreamStatus {
        get {
            _status.value
        }
        set {
            _status.value = newValue
        }
    }

    init(_ stream: QuicStream) {
        self.stream = stream
    }

    public func send(data: Data) throws {
        guard status == .open else {
            throw StreamError.notOpen
        }
        try stream.send(data: data)
    }

    func received(data: Data) {
        if data.isEmpty {
            return
        }
        Task {
            await channel.send(data)
        }
    }

    func close() {
        status = .closed
        channel.close()
    }

    func abort() {
        status = .aborted
        channel.close()
    }

    public func receive() async -> Data? {
        if let data = nextData.value {
            nextData.value = nil
            return data
        }
        return await channel.receive()
    }

    public func receiveByte() async -> UInt8? {
        if var data = nextData.value {
            let byte = data.removeFirst()
            if data.isEmpty {
                nextData.value = nil
            } else {
                nextData.value = data
            }
            return byte
        }

        guard var data = await receive() else {
            return nil
        }

        let byte = data.removeFirst()
        if !data.isEmpty {
            nextData.value = data
        }
        return byte
    }
}

and
let byte = await stream.receiveByte()
let processor = getProcessor(type: byte) // determine the data type from byte
processor.process(stream) // loop call stream.receiveData() and process the data until the stream is closed

If you expect to have multiple consumers of receiveByte and deal with shared state (nextData). Perhaps Stream should be an actor instead of a class? If this doesn't work you could use a simple mutex to protect the whole receiveByte function. I expect the race issues in receiveByte are due to granular locking of nextData.
This would also allow you to store just a native Data without having to wrap it in a container and simplify the code.

I don't want to make Stream an actor because that will make all the methods asyn and many of them are used in C callbacks and can't be async.
Yeah a big mutex will work.


chore: remove unused declaration

Fixes #

💚 All backports created successfully

Status
Branch
Result

✅
v2-main

Questions ?
Please refer to the Backport tool documentation and see the Github Action logs for details


Fixes #28 by relying on ansible_distribution_release instead of ansible_...

...lsb. Adding a test kitchen test suite so I know the fix works

@PierrePaul - Any concerns about merging this?

Sure, sorry I completely missed it.

Seems all good, didnt have time to test it properly, but it seems good. Kitchen tests will be a welcome change in this project. I will merge right away and do more tests this week. Thanks!


WIP Foundation extensions

Leaving this here while I work on some other things.
Adds overloads to URLSession APIs which accept/return WebURLs. I'm still a little unsure as to whether it's a good idea.
For making simple requests, it's fine because we do WebURL -> Foundation.URL conversion, which may add percent-encoding in relatively rare cases but performs no other normalization, and is guaranteed to round-trip back to an identical WebURL (to the version with encoding). Since that is done as the first step, all URLs on the URLRequest and URLResponse objects include that percent-encoding, and everything seems consistent from the developer's perspective.
If redirects occur, things get a little more complicated. Foundation resolves the redirect on its own, and converting those URLs to WebURL may require normalization.
For example, if a redirect occurs to /foo/./bar, Foundation will literally make a GET request with that target (including the dot component), and as a result, the request may fail. WebURL would simplify that URL to /foo/bar, and the developer would be left wondering why the request to a proper URL suddenly failed, because they can't see the dot component.
FWIW, Chrome agrees with the WebURL result, and generally Safari does as well. Ugh. It would be much better if we could just have WebURL handle all of the redirect resolution logic.


make_controlled factory bug (setting m_max_dt to eps_rel and meps_rel and m_eps_abs to eps_abs)

Bug submitted here: headmyshoulder/odeint-v2#228

not a bug, closed over at upstream.


Fix missing attribute in form example

Related issues
Coming from the work on PR #1949
Description
We used the JS to set the focus on first error on submit.
We fortgot that we also need to give the data-focus-visible-added attribute to this element to display error correctly.
Motivation & Context
A11y improvement.
Types of change

Bug fix (non-breaking which fixes an issue)

Live previews

https://deploy-preview-1953--boosted.netlify.app/docs/5.3/examples/form/

Checklist
Contribution

[x] I have read the contributing guidelines

Accessibility

[x] My change follows accessibility good practices; I have at least run axe

Design

[x] My change respects the design guidelines defined in Orange Design System
[x] My change is compatible with responsive display

Development

[x] My change follows the developer guide
(NA) I have added JavaScript unit tests to cover my changes
(NA) I have added SCSS unit tests to cover my changes

Documentation

[x] My change introduces changes to the documentation and/or I have updated the documentation accordingly

Checklist (for Core Team only)

(NA) My change introduces changes to the migration guide
(NA) My new component is added in Storybook
(NA) My new component is compatible with RTL
[x] Manually run BrowserStack tests
[x] Manually test browser compatibility with BrowserStack (Chrome >= 60, Firefox >= 60 (+ ESR), Edge, Safari >= 12, iOS Safari, Chrome & Firefox on Android)
[ ] Code review
(NA) Design review
[ ] A11y review

After the merge

[ ] Manually launch Percy tests

@MewenLeHo I've added a note in the migration guide in https://github.com/Orange-OpenSource/Orange-Boosted-Bootstrap/pull/1953/commits/f8955cb47864ee5308c4c1d72632aa502ef671de since this example was already there in v5.2. OK for you?


Update to Python Batch

I updated and cleaned up the python batch stuff. I would appreciate your review. I expect the readme files are a bit of a mess, but if you can give it a try, I would be most grateful.
Thanks!

replaced by #62


ci: add hurl output to summary

Pull Request type
Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[x] Testing
[ ] Other (please describe):

What is the current behavior?
Beerus end-to-end test script does not output the test output, only which files have succeeded or failed
Issue Number: #476
What is the new behavior?
The hurl output of each file has been added to the bottom of the summary, allowing easier debugging, see why some endpoints failed and check that passing endpoints have the expected output.
Does this introduce a breaking change?

[ ] Yes
[x] No

Other information
The summary could be further improved. As of now it displays the hurl output as very-verbose, which is the only mode that displays the request response. We are reaching the limit of hurl + bash here, if we want a cleaner output, we need to either process the hurl output with a more advanced language, or change hurl to a tool that provides more options.
However if this output is satisfying we can keep it as is.

summary.md
example summary.md produceed (this is what is displayed in the CI job output)


Did not match ack and chunk

Once Failed to read response, many same errors and twice write occur after that. Here are logs:
2020-09-15T13:25:54.035897792+00:00 - [fluent-worker-pool][WARN] - Failed to read response, chunk: OTk3ZDcxNTktNjUzMC00NTZmLTljN2UtODliMzhkODE1NmUx, cause: Transient(NetworkError("Resource temporarily unavailable (os error 11)"))
2020-09-15T13:26:04.591909981+00:00 - [fluent-worker-pool][WARN] - Failed to read response, chunk: OTk3ZDcxNTktNjUzMC00NTZmLTljN2UtODliMzhkODE1NmUx, cause: Transient(NetworkError("Resource temporarily unavailable (os error 11)"))
2020-09-15T13:26:06.540205861+00:00 - [fluent-worker-pool][WARN] - Did not match ack and chunk, ack: OTk3ZDcxNTktNjUzMC00NTZmLTljN2UtODliMzhkODE1NmUx, chunk: YzAyZTVhYjgtYjRmNS00Y2JmLWEwMzYtMGQ1YzRkZGU1NWNl
2020-09-15T13:26:09.714639498+00:00 - [fluent-worker-pool][WARN] - Did not match ack and chunk, ack: NDNhNjFhY2MtZTlhMC00NWQ0LThkNGYtOWVmZTkyY2UyM2Q0, chunk: ZGEwNzMwZjktZTVmMC00ZDE5LWFmMmMtMzY1NWJkZTZhOGEx
2020-09-15T13:26:13.858623537+00:00 - [fluent-worker-pool][WARN] - Did not match ack and chunk, ack: ODAzYzA4ODQtMjdjYS00NTU2LTk2ZTgtZGZiYTE0OGU1YWIz, chunk: ZWI1YmU4NzItYTNlNS00ZDE5LTlkMTQtMjUxMGY3MWQ0MGIw
2020-09-15T13:26:14.173476908+00:00 - [fluent-worker-pool][WARN] - Did not match ack and chunk, ack: ZTBhODZmYjItOGE3NC00MjA0LTkwMzAtYzBkZTY5YTExMDYz, chunk: MDhmNzI1NWUtNmY0YS00MzU1LTk3ZDUtOTBmY2NmNDI4MTBi
2020-09-15T13:26:14.298042783+00:00 - [fluent-worker-pool][WARN] - Did not match ack and chunk, ack: MDhmNzI1NWUtNmY0YS00MzU1LTk3ZDUtOTBmY2NmNDI4MTBi, chunk: MjU0NTA1MTctMDA4Ny00NjRjLTkzM2QtNGUzOWUzNzBkMjU4
2020-09-15T13:26:14.438972065+00:00 - [fluent-worker-pool][WARN] - Did not match ack and chunk, ack: MjU0NTA1MTctMDA4Ny00NjRjLTkzM2QtNGUzOWUzNzBkMjU4, chunk: YmRjZjM3OGUtOWJkYi00NWI5LWJlMjYtNmNiNDI0MDYwZmEw
2020-09-15T13:26:14.656711886+00:00 - [fluent-worker-pool][WARN] - Did not match ack and chunk, ack: YmRjZjM3OGUtOWJkYi00NWI5LWJlMjYtNmNiNDI0MDYwZmEw, chunk: OGQzNDkyNjMtNmQ2Yy00OTc0LWIzZjItZTg3NzU3OTJlNDUx
2020-09-15T13:26:14.810611582+00:00 - [fluent-worker-pool][WARN] - Did not match ack and chunk, ack: OGQzNDkyNjMtNmQ2Yy00OTc0LWIzZjItZTg3NzU3OTJlNDUx, chunk: ZjYxMjIyMjctZTUyOC00MjUzLThjZmEtNGZlNDRlYzc2MGQy
2020-09-15T13:26:14.894403624+00:00 - [fluent-worker-pool][WARN] - Did not match ack and chunk, ack: ZjYxMjIyMjctZTUyOC00MjUzLThjZmEtNGZlNDRlYzc2MGQy, chunk: MzNhYWMzMDktMGU3OC00Y2Q3LThkOGEtNGM0MmVlNDZmZDM0
2020-09-15T13:26:15.301887782+00:00 - [fluent-worker-pool][WARN] - Did not match ack and chunk, ack: NzVkY2M1ZTYtMDMyNC00NTk0LThmNDItYjliMmJlYmJiNzRh, chunk: MzdjZTM5ZmYtOWZmNi00MTZmLTkxY2YtMWE3ODQxNWVhMmYz

There are 2 bugs.

The error "Resource temporarily unavailable (os error 11)" should be handled differently, or should be ignored. It means we will receive the response, but with a delay.
After this case all messages are sent several times. It depends on the settings write_retry_timeout and read_retry_timeout
by default write_retry_timeout is 30 secs, read_retry_timeout 10 secs. Messages are sent twice for these settings.
I see nested closure with backoff pattern in the method
fn write_and_read(&mut self, buf: &[u8], chunk: &str) -> Result<(), Error>, file https://github.com/tkrs/poston/blob/master/src/connect.rs.
External closure with write works successfully once, and nested read_backoff is failed after 10 secs. It means that external backoff also is failed, but writes to socket. So it will write again the same message on next backoff step because it has a 30 secs timeout. I thinк write_retry_timeout should be less or equal than read_retry_timeout.
Anyway the nested backoff is hard to understand and the kind of anti pattern. May be needs refactoring write_and_read function.

Thanks

Thanks for reporting. Is the version 0.7.0?

Thanks for reporting. Is the version 0.7.0?

Yes, version is 0.7.0

Hmm, wasn't any error/warn logs output before the above logs?

Hmm, wasn't any error/warn logs output before the above logs?

No, the first log is "Failed to read response"

Sorry for the late reply. I have published an attempt to solve this as 0.7.1.