Angular 6 / RXJS 6 Compatibility

I'm submitting a ...
[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?

Bug

What is the current behavior?

Build Errors when upgrading to Angular 6 / RXJS 6

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via
https://plnkr.co or similar (you can use one of the templates above as a starting point).

upgrade rxjs to version ^6.0.0(-beta.2)

What is the expected behavior?

building without errors

What is the motivation / use case for changing the behavior?

keeping up to date with the new angular versions

Please tell us about your environment:

Angular version: 6.0.0-rc.0
Browser: all

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

ERROR in ./node_modules/ngx-modialog/plugins/bootstrap/bundle/ngx-modialog-bootstrap.es5.js
Module not found: Error: Can't resolve 'rxjs/operator/combineLatest' in '[...]node_modules/ngx-modialog/plugins/bootstrap/bundle'
ERROR in ./node_modules/ngx-modialog/bundle/ngx-modialog.es5.js
Module not found: Error: Can't resolve 'rxjs/operator/filter' in '[...]node_modules/ngx-modialog/bundle'

The Upgrade Guide can be found here: https://github.com/ReactiveX/rxjs/blob/master/doc/pipeable-operators.md

Sure, but I'll wait for official first

PR's welcome

For now you can just install rxjs-compat, which as the name suggests is a compatibility layer between RxJS v5 and v6 and everything seems to work fine.

With rxjs-compat, the whole RxJS gets bundled into the application.

Update: Angular CLI library support is limited at this point.

It's very hard to tame it to work with sub-packages (e.g. @ngx-modialog/plugins/bootstrap)
It does not support (by design) hot library reloading, i.e. it requires re-compiling and building for every change in the lib then reload the demo app.

I will start working on v6 changes, but again I want to switch this repo to use 100% supported build process so it will probably be nx...
I have to say, that this no hot rebuild thing for libs (2) will not last long the CLI guys will realise it soon enough :)

ok, repo is angular-cli based, including demo app + libraries and of course cli library build which should make maintenance easier.
I'v also publish to npm, version 6.
There are some issues with the demo site, will work on that.

Now I've got this issue with version 6.0.0 and boostrap 3

Hi, any progress? You are the last reason for rxjs-compat in our large application(

I guess this one is dead after two years no response  action :(.


✨ use controller-runtime Terminal error instead of custom Unrecoverable error

Fixes #400

We have a TerminalError‽‽

@tmshort controller-runtime does! And it serves the same purpose, except with controller-runtime integration (e.g. for tracking terminal error metrics)


fix(cicd): lock version of happy in clean_happy_rdev.yml

Reason for Change

the automated cleanup of rdev environments it not working

Changes

add
remove
modify

Testing steps

Either list QA steps or reasoning you feel QA is unnecessary
Describe how you made sure to know that your changes worked. Should allow someone else to go verify your code without in depth knowledge.
"Unit tested only", "tested in rdev by a, b, c, verifying feature worked by... ", "manually ran pipeline locally with these results: ..."

Notes for Reviewer

Closing because we are already always using the latest.


example of a parser fo floating point numbers using digit and recognize

Recognize digits separated by a '.', then use float::parse on the resulting string.

Fixed in eee44cbcb8b


GIX-1115: Stake SNS Neuron

Motivation
Add functionality to stake an SNS neuron.
Changes

Add public methods stakeNeuron and getNextNeuronAccount to SnsWrapper.
Add private method getNeuronSubaccount to SnsWrapper.
New array utils "bigIntToUint8Array" and "numberToUint8Array".

Tests

New tests for public methods in SnsWrapper.
New tests for new utils.

@mstrasinskis @peterpeterparker please review

@peterpeterparker ready for another review

@peterpeterparker revered to use js-256 library


shiroa:0.1.2

I am submitting

[ ] a new package
[x] an update for a package

Description:

Fixed warnings since typst v0.12.0.

I have read and followed the submission guidelines and, in particular, I

[x] selected a name that isn't the most obvious or canonical name for what the package does
[x] added a typst.toml file with all required keys
[x] added a README.md with documentation for my package
[x] have chosen a license and added a LICENSE file or linked one in my README.md
[x] tested my package locally on my system and it worked
[x] excluded PDFs or README images, if any, but not the LICENSE

[x] ensured that my package is licensed such that users can use and distribute the contents of its template directory without restriction, after modifying them through normal use.

Thanks :)


Evaluate GEP on Client Certificate Validation

We need to evaluate the following GEP in the Gateway API to determine its feasibility as written:
https://github.com/kubernetes-sigs/gateway-api/pull/2273
Acceptance

The above GEP is evaluated to determine if the new functionality present is feasibly implementable by NGINX.
The GEP has a review added explaining the results of the evaluation, including any improvements necessary to support NGINX, if applicable.

NGINX should be able to implement the GEP as written.
There's one field subjectAltNames that NGINX can't support natively, but that is possible to implement with njs: https://github.com/nginx/njs-examples?tab=readme-ov-file#certificates.
Other implementors have raised questions around this field and it may not be included in the final API.
I will keep monitoring this GEP as it progresses, but will close this issue.


[code review] Skip jack fixes for partial capture

@Soleone and @wisq could you give this a review?

Closing, will create a new pull request without the extra commit.


Rework types structure

Instead of exporting everything separately, we want to match OpenAPI specification more closely

Next step is cleaning up different response status types

Next step is adding error codes, there isn't a good example right now


Fix tvOS build breakage in RCTAnimation Xcode project

RCTSubtractionAnimatedNode.m was not added to the tvOS build of RCTAnimation... this PR fixes the issue.
Test Plan
tvOS CI should succeed after this change.
Release Notes
[TVOS] [BUGFIX] [RCTAnimation] Fix build breakage

Woops, does the tvOS CI tests not include RCTAnimation? Looks like it still passed :(

@facebook-github-bot shipit


Split boulder test cookbook into it's own repo

I was trying to reuse the acme_server test cookbook in another recipe, to properly test setup via apache and tomcat, and found that this would be useful for a few reasons:

it's currently not possible to use the test cookbook from another cookbook using berkshelf (see https://github.com/berkshelf/berkshelf/issues/1505)
the boulder server was hardcoded to use port 5000 for debug (among many other ports), and this conflicted with an atlassian app's required ports

Anyhow, happy to do the work my self and then transfer the repo back to you, but curious whether you would be interested in using it for testing this cookbook. It would be a straight-up port from this cookbook at first, but could be improved later :)

Just out of curiosity: why would you host your own boulder server? (apart from testing clients)

It was for testing that I wanted it, but those issues I mentioned made it slightly hacky (had to copy paste the acme_server cookbook in the meantime, and change it a bit)

Well, I split it out :)
https://github.com/patcon/chef-letsencrypt-boulder-server
There are some small changes for configurability.
I've added you a collaborator, and used all the same licensing. If you'd still prefer to have the cookbook officially under your account, happy to transfer it.

Also added it to community site and made you a collab:
https://supermarket.chef.io/cookbooks/letsencrypt-boulder-server

OK, converted this into a PR that starts using the the extracted cookbook. Kept the acme_server recipe as part of it because it seems centos still needs some messing around with localhost. Could move that into the external cookbook as well, if you have feelings on that (but skimming the issue made it seem temporary).
Ran the tests and this works fine on DigitalOcean.

Ah why not :)


Accept Date as input for set_battery_replaced service

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
Changed some batteries a few days ago, but unable to set the replacement date to anything but now().
Describe the solution you'd like
If possible, change the service set_battery_replaced to take an optional date to set the replacement date of a battery.
Describe alternatives you've considered
Maybe manually modifying the .storage info?
Additional context
None really.

I did look at this but since it's only going to be the first time you'll need to do this decided against it as it was getting complicated with UTC vs local time calculations for little long term use.


Adding details to checkbox component

Coverage remained the same at 78.689% when pulling e15b0282544bd5c8875305ff8240338bcd9f14a1 on sarahkf/checkbox-details into b530f2e6d7fbb81a161eca0a4399d26266567538 on master.


Blank page for SQL syntax error in using PostgreSQL

When I upgrade the Artalk to the latest 2.9.0. I meet the SQL syntax error in using PostgreSQL.
2024/09/04 13:32:05.313 ERROR [v2@v2.52.5/router.go:145] [DB] {"error": "ERROR: missing FROM-clause entry for table \"pages\" (SQLSTATE 42P01)", "elapsed": 0.000486761, "rows": 0, "sql": "SELECT count(*) FROM \"atk_pages\" WHERE (LOWER(pages.key) LIKE LOWER('%%') OR LOWER(title) LIKE LOWER('%%')) AND \"atk_pages\".\"deleted_at\" IS NULL"}

https://github.com/ArtalkJS/Artalk/blob/90d8503d0ee07cfe2e633385c487a2b1c72b743d/server/handler/page_list.go#L53C26-L53C31
This should be the root cause. You shouldn't append the where clause directly and the page.key isn't a good reference.
if p.Search != "" {
		q = q.Scopes(func(d *gorm.DB) *gorm.DB {
			return d.Where("LOWER(`key`) LIKE LOWER(?) OR LOWER(title) LIKE LOWER(?)",
				"%"+p.Search+"%", "%"+p.Search+"%")
		})
}

Or you can use GORM built-in method for querying the table name.
if p.Search != "" {
	q = q.Scopes(func(d *gorm.DB) *gorm.DB {
		tableName := app.Dao().DB().Model(&entity.Page{}).TableName()
		return d.Where("LOWER("+tableName+".key) LIKE LOWER(?) OR LOWER(title) LIKE LOWER(?)",
				"%"+p.Search+"%", "%"+p.Search+"%")
	})
}

Or you can use GORM built-in method for querying the table name.
if p.Search != "" {
	q = q.Scopes(func(d *gorm.DB) *gorm.DB {
		tableName := app.Dao().DB().Model(&entity.Page{}).TableName()
		return d.Where("LOWER("+tableName+".key) LIKE LOWER(?) OR LOWER(title) LIKE LOWER(?)",
			"%"+p.Search+"%", "%"+p.Search+"%")
	})
}

Thank you for your feedback and suggestions

@qwqcode Given this is a regression. I suggest that you should release a new version ASAP.

The nightly version has been released immediately.


Add new site muted.io

Add my new site built with 11ty: muted.io. Thanks for creating something amazing with 11ty! 😍

Thank you!


Scan module with a sast or dast as a pr check

See tflint, tfsec, checkov
These can be run in pre commit hooks and can be run as a pr check

there's a pending issue with checkov preventing us from using it unfortunately: https://github.com/bridgecrewio/checkov/issues/4205

You can enable githubs codeql as well

I'll be adding checkov soon to pre-commit 👍🏼


Cannot install lavis with python3.6.12

Thanks to all the contributors to this repository. It is a great job. But I have some installation problems cannot be solved right now.
I know the python_requires=">=3.7.0", but I have no idea about why python version requirement is necessary. After I change the python version to 3.6.12, this error is raised: module 'lavis' has no attribute 'processors', from transforms_video.py line 18. Could anyone help me to figure out?

``

Hi @slyviacassell , thanks for your interest.
Can you help confirm your installation completes in full?
Thanks

I re-excute the installation command pip install -e . and this is the log
Looking in indexes: https://mirrors.aliyun.com/pypi/simple
Obtaining file:///mnt/alitranx-nas/users/tianyuxin.tyx/projects/LAVIS
  Preparing metadata (setup.py) ... done
Requirement already satisfied: contexttimer in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (0.3.3)
Requirement already satisfied: decord in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (0.6.0)
Requirement already satisfied: einops>=0.4.1 in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (0.4.1)
Requirement already satisfied: fairscale==0.4.4 in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (0.4.4)
Requirement already satisfied: ftfy in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (6.0.3)
Requirement already satisfied: iopath in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (0.1.10)
Requirement already satisfied: ipython in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (7.16.2)
Requirement already satisfied: omegaconf in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (2.3.0)
Requirement already satisfied: opencv-python-headless==<IP_ADDRESS> in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (<IP_ADDRESS>)
Requirement already satisfied: opendatasets in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (0.1.22)
Requirement already satisfied: packaging in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (21.3)
Requirement already satisfied: pandas in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (1.1.5)
Requirement already satisfied: plotly in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (5.5.0)
Requirement already satisfied: pre-commit in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (2.17.0)
Requirement already satisfied: pycocoevalcap in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (1.2)
Requirement already satisfied: pycocotools in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (2.0.6)
Requirement already satisfied: python-magic in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (0.4.27)
Requirement already satisfied: scikit-image in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (0.17.2)
Requirement already satisfied: streamlit in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (1.10.0)
Requirement already satisfied: timm==0.4.12 in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (0.4.12)
Requirement already satisfied: torchvision in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (0.9.1+cpu)
Requirement already satisfied: tqdm in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (4.62.3)
Requirement already satisfied: transformers<4.22.0,>=4.15.0 in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (4.18.0)
Requirement already satisfied: webdataset in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (0.2.31)
Requirement already satisfied: wheel in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (0.37.0)
Requirement already satisfied: sentencepiece in /home/pai/lib/python3.6/site-packages (from lavis==0.1.0) (0.1.97)
Requirement already satisfied: torch>=1.8.0 in /home/pai/lib/python3.6/site-packages (from fairscale==0.4.4->lavis==0.1.0) (1.8.1+cpu)
Requirement already satisfied: numpy>=1.13.3 in /home/pai/lib/python3.6/site-packages (from opencv-python-headless==<IP_ADDRESS>->lavis==0.1.0) (1.19.5)
Requirement already satisfied: dataclasses in /home/pai/lib/python3.6/site-packages (from transformers<4.22.0,>=4.15.0->lavis==0.1.0) (0.8)
Requirement already satisfied: regex!=2019.12.17 in /home/pai/lib/python3.6/site-packages (from transformers<4.22.0,>=4.15.0->lavis==0.1.0) (2021.11.10)
Requirement already satisfied: filelock in /home/pai/lib/python3.6/site-packages (from transformers<4.22.0,>=4.15.0->lavis==0.1.0) (3.4.0)
Requirement already satisfied: sacremoses in /home/pai/lib/python3.6/site-packages (from transformers<4.22.0,>=4.15.0->lavis==0.1.0) (0.0.53)
Requirement already satisfied: pyyaml>=5.1 in /home/pai/lib/python3.6/site-packages (from transformers<4.22.0,>=4.15.0->lavis==0.1.0) (6.0)
Requirement already satisfied: huggingface-hub<1.0,>=0.1.0 in /home/pai/lib/python3.6/site-packages (from transformers<4.22.0,>=4.15.0->lavis==0.1.0) (0.4.0)
Requirement already satisfied: importlib-metadata in /home/pai/lib/python3.6/site-packages (from transformers<4.22.0,>=4.15.0->lavis==0.1.0) (4.2.0)
Requirement already satisfied: requests in /home/pai/lib/python3.6/site-packages (from transformers<4.22.0,>=4.15.0->lavis==0.1.0) (2.25.1)
Requirement already satisfied: tokenizers!=0.11.3,<0.13,>=0.11.1 in /home/pai/lib/python3.6/site-packages (from transformers<4.22.0,>=4.15.0->lavis==0.1.0) (0.12.1)
Requirement already satisfied: pyparsing!=3.0.5,>=2.0.2 in /home/pai/lib/python3.6/site-packages (from packaging->lavis==0.1.0) (3.0.6)
Requirement already satisfied: wcwidth in /home/pai/lib/python3.6/site-packages (from ftfy->lavis==0.1.0) (0.2.5)
Requirement already satisfied: typing-extensions in /home/pai/lib/python3.6/site-packages (from iopath->lavis==0.1.0) (4.0.1)
Requirement already satisfied: portalocker in /home/pai/lib/python3.6/site-packages (from iopath->lavis==0.1.0) (2.6.0)
Requirement already satisfied: decorator in /home/pai/lib/python3.6/site-packages (from ipython->lavis==0.1.0) (4.4.2)
Requirement already satisfied: prompt-toolkit!=3.0.0,!=3.0.1,<3.1.0,>=2.0.0 in /home/pai/lib/python3.6/site-packages (from ipython->lavis==0.1.0) (3.0.24)
Requirement already satisfied: traitlets>=4.2 in /home/pai/lib/python3.6/site-packages (from ipython->lavis==0.1.0) (4.3.3)
Requirement already satisfied: backcall in /home/pai/lib/python3.6/site-packages (from ipython->lavis==0.1.0) (0.2.0)
Requirement already satisfied: pickleshare in /home/pai/lib/python3.6/site-packages (from ipython->lavis==0.1.0) (0.7.5)
Requirement already satisfied: jedi<=0.17.2,>=0.10 in /home/pai/lib/python3.6/site-packages (from ipython->lavis==0.1.0) (0.17.2)
Requirement already satisfied: setuptools>=18.5 in /home/pai/lib/python3.6/site-packages (from ipython->lavis==0.1.0) (58.0.4)
Requirement already satisfied: pygments in /home/pai/lib/python3.6/site-packages (from ipython->lavis==0.1.0) (2.10.0)
Requirement already satisfied: pexpect in /home/pai/lib/python3.6/site-packages (from ipython->lavis==0.1.0) (4.8.0)
Requirement already satisfied: antlr4-python3-runtime==4.9.* in /home/pai/lib/python3.6/site-packages (from omegaconf->lavis==0.1.0) (4.9.3)
Requirement already satisfied: kaggle in /home/pai/lib/python3.6/site-packages (from opendatasets->lavis==0.1.0) (1.5.12)
Requirement already satisfied: click in /home/pai/lib/python3.6/site-packages (from opendatasets->lavis==0.1.0) (8.0.4)
Requirement already satisfied: python-dateutil>=2.7.3 in /home/pai/lib/python3.6/site-packages (from pandas->lavis==0.1.0) (2.8.2)
Requirement already satisfied: pytz>=2017.2 in /home/pai/lib/python3.6/site-packages (from pandas->lavis==0.1.0) (2021.3)
Requirement already satisfied: tenacity>=6.2.0 in /home/pai/lib/python3.6/site-packages (from plotly->lavis==0.1.0) (8.0.1)
Requirement already satisfied: six in /home/pai/lib/python3.6/site-packages (from plotly->lavis==0.1.0) (1.16.0)
Requirement already satisfied: identify>=1.0.0 in /home/pai/lib/python3.6/site-packages (from pre-commit->lavis==0.1.0) (2.4.4)
Requirement already satisfied: toml in /home/pai/lib/python3.6/site-packages (from pre-commit->lavis==0.1.0) (0.10.2)
Requirement already satisfied: virtualenv>=20.0.8 in /home/pai/lib/python3.6/site-packages (from pre-commit->lavis==0.1.0) (20.16.2)
Requirement already satisfied: nodeenv>=0.11.1 in /home/pai/lib/python3.6/site-packages (from pre-commit->lavis==0.1.0) (1.6.0)
Requirement already satisfied: cfgv>=2.0.0 in /home/pai/lib/python3.6/site-packages (from pre-commit->lavis==0.1.0) (3.3.1)
Requirement already satisfied: importlib-resources<5.3 in /home/pai/lib/python3.6/site-packages (from pre-commit->lavis==0.1.0) (5.2.3)
Requirement already satisfied: matplotlib>=2.1.0 in /home/pai/lib/python3.6/site-packages (from pycocotools->lavis==0.1.0) (3.3.4)
Requirement already satisfied: scipy>=1.0.1 in /home/pai/lib/python3.6/site-packages (from scikit-image->lavis==0.1.0) (1.5.4)
Requirement already satisfied: imageio>=2.3.0 in /home/pai/lib/python3.6/site-packages (from scikit-image->lavis==0.1.0) (2.15.0)
Requirement already satisfied: pillow!=7.1.0,!=7.1.1,>=4.3.0 in /home/pai/lib/python3.6/site-packages (from scikit-image->lavis==0.1.0) (8.4.0)
Requirement already satisfied: networkx>=2.0 in /home/pai/lib/python3.6/site-packages (from scikit-image->lavis==0.1.0) (2.5.1)
Requirement already satisfied: tifffile>=2019.7.26 in /home/pai/lib/python3.6/site-packages (from scikit-image->lavis==0.1.0) (2020.9.3)
Requirement already satisfied: PyWavelets>=1.1.1 in /home/pai/lib/python3.6/site-packages (from scikit-image->lavis==0.1.0) (1.1.1)
Requirement already satisfied: validators in /home/pai/lib/python3.6/site-packages (from streamlit->lavis==0.1.0) (0.20.0)
Requirement already satisfied: protobuf<4,>=3.12 in /home/pai/lib/python3.6/site-packages (from streamlit->lavis==0.1.0) (3.19.1)
Requirement already satisfied: tornado>=5.0 in /home/pai/lib/python3.6/site-packages (from streamlit->lavis==0.1.0) (6.1)
Requirement already satisfied: attrs in /home/pai/lib/python3.6/site-packages (from streamlit->lavis==0.1.0) (21.2.0)
Requirement already satisfied: blinker in /home/pai/lib/python3.6/site-packages (from streamlit->lavis==0.1.0) (1.5)
Requirement already satisfied: cachetools>=4.0 in /home/pai/lib/python3.6/site-packages (from streamlit->lavis==0.1.0) (4.2.4)
Requirement already satisfied: watchdog in /home/pai/lib/python3.6/site-packages (from streamlit->lavis==0.1.0) (2.2.0)
Requirement already satisfied: pyarrow in /home/pai/lib/python3.6/site-packages (from streamlit->lavis==0.1.0) (6.0.1)
Requirement already satisfied: tzlocal in /home/pai/lib/python3.6/site-packages (from streamlit->lavis==0.1.0) (4.1)
Requirement already satisfied: gitpython!=3.1.19 in /home/pai/lib/python3.6/site-packages (from streamlit->lavis==0.1.0) (3.1.18)
Requirement already satisfied: pydeck>=0.1.dev5 in /home/pai/lib/python3.6/site-packages (from streamlit->lavis==0.1.0) (0.6.2)
Requirement already satisfied: altair>=3.2.0 in /home/pai/lib/python3.6/site-packages (from streamlit->lavis==0.1.0) (4.1.0)
Requirement already satisfied: semver in /home/pai/lib/python3.6/site-packages (from streamlit->lavis==0.1.0) (2.13.0)
Requirement already satisfied: rich in /home/pai/lib/python3.6/site-packages (from streamlit->lavis==0.1.0) (12.6.0)
Requirement already satisfied: pympler>=0.9 in /home/pai/lib/python3.6/site-packages (from streamlit->lavis==0.1.0) (1.0.1)
Requirement already satisfied: braceexpand in /home/pai/lib/python3.6/site-packages (from webdataset->lavis==0.1.0) (0.1.7)
Requirement already satisfied: entrypoints in /home/pai/lib/python3.6/site-packages (from altair>=3.2.0->streamlit->lavis==0.1.0) (0.3)
Requirement already satisfied: jsonschema in /home/pai/lib/python3.6/site-packages (from altair>=3.2.0->streamlit->lavis==0.1.0) (4.0.0)
Requirement already satisfied: jinja2 in /home/pai/lib/python3.6/site-packages (from altair>=3.2.0->streamlit->lavis==0.1.0) (3.0.3)
Requirement already satisfied: toolz in /home/pai/lib/python3.6/site-packages (from altair>=3.2.0->streamlit->lavis==0.1.0) (0.12.0)
Requirement already satisfied: gitdb<5,>=4.0.1 in /home/pai/lib/python3.6/site-packages (from gitpython!=3.1.19->streamlit->lavis==0.1.0) (4.0.9)
Requirement already satisfied: zipp>=0.5 in /home/pai/lib/python3.6/site-packages (from importlib-metadata->transformers<4.22.0,>=4.15.0->lavis==0.1.0) (3.6.0)
Requirement already satisfied: parso<0.8.0,>=0.7.0 in /home/pai/lib/python3.6/site-packages (from jedi<=0.17.2,>=0.10->ipython->lavis==0.1.0) (0.7.1)
Requirement already satisfied: kiwisolver>=1.0.1 in /home/pai/lib/python3.6/site-packages (from matplotlib>=2.1.0->pycocotools->lavis==0.1.0) (1.3.1)
Requirement already satisfied: cycler>=0.10 in /home/pai/lib/python3.6/site-packages (from matplotlib>=2.1.0->pycocotools->lavis==0.1.0) (0.11.0)
Requirement already satisfied: ipywidgets>=7.0.0 in /home/pai/lib/python3.6/site-packages (from pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (7.6.5)
Requirement already satisfied: ipykernel>=5.1.2 in /home/pai/lib/python3.6/site-packages (from pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (5.5.6)
Requirement already satisfied: ipython-genutils in /home/pai/lib/python3.6/site-packages (from traitlets>=4.2->ipython->lavis==0.1.0) (0.2.0)
Requirement already satisfied: platformdirs<3,>=2 in /home/pai/lib/python3.6/site-packages (from virtualenv>=20.0.8->pre-commit->lavis==0.1.0) (2.4.0)
Requirement already satisfied: distlib<1,>=0.3.1 in /home/pai/lib/python3.6/site-packages (from virtualenv>=20.0.8->pre-commit->lavis==0.1.0) (0.3.6)
Requirement already satisfied: certifi in /home/pai/lib/python3.6/site-packages (from kaggle->opendatasets->lavis==0.1.0) (2021.5.30)
Requirement already satisfied: python-slugify in /home/pai/lib/python3.6/site-packages (from kaggle->opendatasets->lavis==0.1.0) (6.1.2)
Requirement already satisfied: urllib3 in /home/pai/lib/python3.6/site-packages (from kaggle->opendatasets->lavis==0.1.0) (1.26.4)
Requirement already satisfied: ptyprocess>=0.5 in /home/pai/lib/python3.6/site-packages (from pexpect->ipython->lavis==0.1.0) (0.7.0)
Requirement already satisfied: chardet<5,>=3.0.2 in /home/pai/lib/python3.6/site-packages (from requests->transformers<4.22.0,>=4.15.0->lavis==0.1.0) (3.0.4)
Requirement already satisfied: idna<3,>=2.5 in /home/pai/lib/python3.6/site-packages (from requests->transformers<4.22.0,>=4.15.0->lavis==0.1.0) (2.6)
Requirement already satisfied: commonmark<0.10.0,>=0.9.0 in /home/pai/lib/python3.6/site-packages (from rich->streamlit->lavis==0.1.0) (0.9.1)
Requirement already satisfied: joblib in /home/pai/lib/python3.6/site-packages (from sacremoses->transformers<4.22.0,>=4.15.0->lavis==0.1.0) (1.1.0)
Requirement already satisfied: pytz-deprecation-shim in /home/pai/lib/python3.6/site-packages (from tzlocal->streamlit->lavis==0.1.0) (0.1.0.post0)
Requirement already satisfied: backports.zoneinfo in /home/pai/lib/python3.6/site-packages (from tzlocal->streamlit->lavis==0.1.0) (0.2.1)
Requirement already satisfied: smmap<6,>=3.0.1 in /home/pai/lib/python3.6/site-packages (from gitdb<5,>=4.0.1->gitpython!=3.1.19->streamlit->lavis==0.1.0) (5.0.0)
Requirement already satisfied: jupyter-client in /home/pai/lib/python3.6/site-packages (from ipykernel>=5.1.2->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (7.1.0)
Requirement already satisfied: jupyterlab-widgets>=1.0.0 in /home/pai/lib/python3.6/site-packages (from ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (1.0.2)
Requirement already satisfied: widgetsnbextension~=3.5.0 in /home/pai/lib/python3.6/site-packages (from ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (3.5.2)
Requirement already satisfied: nbformat>=4.2.0 in /home/pai/lib/python3.6/site-packages (from ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (5.1.3)
Requirement already satisfied: MarkupSafe>=2.0 in /home/pai/lib/python3.6/site-packages (from jinja2->altair>=3.2.0->streamlit->lavis==0.1.0) (2.0.1)
Requirement already satisfied: pyrsistent!=0.17.0,!=0.17.1,!=0.17.2,>=0.14.0 in /home/pai/lib/python3.6/site-packages (from jsonschema->altair>=3.2.0->streamlit->lavis==0.1.0) (0.18.0)
Requirement already satisfied: text-unidecode>=1.3 in /home/pai/lib/python3.6/site-packages (from python-slugify->kaggle->opendatasets->lavis==0.1.0) (1.3)
Requirement already satisfied: tzdata in /home/pai/lib/python3.6/site-packages (from pytz-deprecation-shim->tzlocal->streamlit->lavis==0.1.0) (2021.5)
Requirement already satisfied: jupyter-core in /home/pai/lib/python3.6/site-packages (from nbformat>=4.2.0->ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (4.9.1)
Requirement already satisfied: notebook>=4.4.1 in /home/pai/lib/python3.6/site-packages (from widgetsnbextension~=3.5.0->ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (6.4.6)
Requirement already satisfied: pyzmq>=13 in /home/pai/lib/python3.6/site-packages (from jupyter-client->ipykernel>=5.1.2->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (22.3.0)
Requirement already satisfied: nest-asyncio>=1.5 in /home/pai/lib/python3.6/site-packages (from jupyter-client->ipykernel>=5.1.2->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (1.5.4)
Requirement already satisfied: nbconvert in /home/pai/lib/python3.6/site-packages (from notebook>=4.4.1->widgetsnbextension~=3.5.0->ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (6.0.7)
Requirement already satisfied: argon2-cffi in /home/pai/lib/python3.6/site-packages (from notebook>=4.4.1->widgetsnbextension~=3.5.0->ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (21.3.0)
Requirement already satisfied: terminado>=0.8.3 in /home/pai/lib/python3.6/site-packages (from notebook>=4.4.1->widgetsnbextension~=3.5.0->ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (0.12.1)
Requirement already satisfied: prometheus-client in /home/pai/lib/python3.6/site-packages (from notebook>=4.4.1->widgetsnbextension~=3.5.0->ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (0.12.0)
Requirement already satisfied: Send2Trash>=1.8.0 in /home/pai/lib/python3.6/site-packages (from notebook>=4.4.1->widgetsnbextension~=3.5.0->ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (1.8.0)
Requirement already satisfied: argon2-cffi-bindings in /home/pai/lib/python3.6/site-packages (from argon2-cffi->notebook>=4.4.1->widgetsnbextension~=3.5.0->ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (21.2.0)
Requirement already satisfied: testpath in /home/pai/lib/python3.6/site-packages (from nbconvert->notebook>=4.4.1->widgetsnbextension~=3.5.0->ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (0.5.0)
Requirement already satisfied: jupyterlab-pygments in /home/pai/lib/python3.6/site-packages (from nbconvert->notebook>=4.4.1->widgetsnbextension~=3.5.0->ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (0.1.2)
Requirement already satisfied: bleach in /home/pai/lib/python3.6/site-packages (from nbconvert->notebook>=4.4.1->widgetsnbextension~=3.5.0->ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (4.1.0)
Requirement already satisfied: pandocfilters>=1.4.1 in /home/pai/lib/python3.6/site-packages (from nbconvert->notebook>=4.4.1->widgetsnbextension~=3.5.0->ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (1.5.0)
Requirement already satisfied: nbclient<0.6.0,>=0.5.0 in /home/pai/lib/python3.6/site-packages (from nbconvert->notebook>=4.4.1->widgetsnbextension~=3.5.0->ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (0.5.9)
Requirement already satisfied: mistune<2,>=0.8.1 in /home/pai/lib/python3.6/site-packages (from nbconvert->notebook>=4.4.1->widgetsnbextension~=3.5.0->ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (0.8.4)
Requirement already satisfied: defusedxml in /home/pai/lib/python3.6/site-packages (from nbconvert->notebook>=4.4.1->widgetsnbextension~=3.5.0->ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (0.7.1)
Requirement already satisfied: async-generator in /home/pai/lib/python3.6/site-packages (from nbclient<0.6.0,>=0.5.0->nbconvert->notebook>=4.4.1->widgetsnbextension~=3.5.0->ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (1.10)
Requirement already satisfied: cffi>=1.0.1 in /home/pai/lib/python3.6/site-packages (from argon2-cffi-bindings->argon2-cffi->notebook>=4.4.1->widgetsnbextension~=3.5.0->ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (1.14.5)
Requirement already satisfied: webencodings in /home/pai/lib/python3.6/site-packages (from bleach->nbconvert->notebook>=4.4.1->widgetsnbextension~=3.5.0->ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (0.5.1)
Requirement already satisfied: pycparser in /home/pai/lib/python3.6/site-packages (from cffi>=1.0.1->argon2-cffi-bindings->argon2-cffi->notebook>=4.4.1->widgetsnbextension~=3.5.0->ipywidgets>=7.0.0->pydeck>=0.1.dev5->streamlit->lavis==0.1.0) (2.18)
Installing collected packages: lavis
  Running setup.py develop for lavis
Successfully installed lavis-0.1.0

May I know your platform?

The detailed information:

OS: ubuntu18.04
python: 3.6.12
torch: 1.8.0

When upgrading python to version 3.7, all the problems are resolved.


[RFC] Created Static Web App Exec Doc (using AI)

Created the static web app exec doc with the similarity blocks generated by AI. It works now. Had to get modify the original Azure doc to get around the Git authentication part, which could be reinstated once we allow users to add variables while running the exec doc. Let me know if there are any comments or question!

Ensure we use generic environment variables and random postfixes.
The webapp 'show' command at the end doesn't ensure a valid output.  You can also curl as an option.  See LAMP doc for an example.
Echo the url to the user at the end like in LAMP/AKS docs.
Delete the AI's message at the end about the markdown being useful.

made the changes and pushed with updates

@vmarcella: how to troubleshoot this part of the code where I am replicating the curl command to see if the website is live and working but it keeps running in my local ubuntu terminal although a valid URL has been generated.

Validate that the static web app is running by curling the application url:

runtime="5 minute";
endtime=$(date -ud "$runtime" +%s);
while [[ $(date -u +%s) -le $endtime ]]; do
    if curl -L -s -f $MY_STATIC_WEB_APP_URL > /dev/null ; then
      curl -L -s -f $MY_STATIC_WEB_APP_URL > /dev/null | head -n 9
      break
    else
      echo "Waiting for the static web app to be deployed..."
      sleep 10
    fi;
done

Results:

<!DOCTYPE html>
<html lang=en>
<head>
<meta charset=utf-8 />
<meta name=viewport content="width=device-width, initial-scale=1.0" />
<meta http-equiv=X-UA-Compatible content="IE=edge" />
<title>Azure Static Web Apps - Welcome</title>
<link rel="shortcut icon" href=https://appservice.azureedge.net/images/static-apps/v3/favicon.svg type=image/x-icon />
<link rel=stylesheet href=https://ajax.aspnetcdn.com/ajax/bootstrap/4.1.1/css/bootstrap.min.css crossorigin=anonymous />

@vmarcella: how to troubleshoot this part of the code where I am replicating the curl command to see if the website is live and working but it keeps running in my local ubuntu terminal although a valid URL has been generated.

Validate that the static web app is running by curling the application url:

runtime="5 minute";
endtime=$(date -ud "$runtime" +%s);
while [[ $(date -u +%s) -le $endtime ]]; do
    if curl -L -s -f $MY_STATIC_WEB_APP_URL > /dev/null ; then
      curl -L -s -f $MY_STATIC_WEB_APP_URL > /dev/null | head -n 9
      break
    else
      echo "Waiting for the static web app to be deployed..."
      sleep 10
    fi;
done

Results:
<!DOCTYPE html>
<html lang=en>
<head>
<meta charset=utf-8 />
<meta name=viewport content="width=device-width, initial-scale=1.0" />
<meta http-equiv=X-UA-Compatible content="IE=edge" />
<title>Azure Static Web Apps - Welcome</title>
<link rel="shortcut icon" href=https://appservice.azureedge.net/images/static-apps/v3/favicon.svg type=image/x-icon />
<link rel=stylesheet href=https://ajax.aspnetcdn.com/ajax/bootstrap/4.1.1/css/bootstrap.min.css crossorigin=anonymous />

Have you made any progress on this issue @naman-msft?

The converter files should not be in the top level of the repo, they should be in a specific folder.  Can you also add to the description of the PR the AI converter changes?  Right now it appears that the PR is just for the static web app.

The converter files should not be in the top level of the repo, they should be in a specific folder. Can you also add to the description of the PR the AI converter changes? Right now it appears that the PR is just for the static web app.


Add krishnan_macropad

:tada:

Hey! The bottom of the gerber looks a little messed up compared to the front, I would suggest making everything to have a 1mm edge around it.


Update My Wallet Address screen [Fiji]

Sketch Cloud:
https://sketch.cloud/s/jDODG/v/eKjDAx/a/W0Oyb0
Zeplin:
https://zpl.io/VQ7WkR4

Add Smart QR Code Scanner icon in the top right to quickly switch to camera
clicking "tomek.eth" copies to clipboard
display ENS + address below

@JamesSmartCell tweaked so the Ethereum address wraps with the same font size: https://app.zeplin.io/project/5d0356be2f25b319dcc5944b/screen/5dfaf2cf2149329a03c7b760

@ChintanRathod this one is not as trivial as it seems. You may need to extend the ENSHandler class to do a reverse-ENS lookup. Currently I don't think it has that ability. The reverse ENS lookup is operational elsewhere in the wallet, on the walletActivity page to detect ENS names for each wallet. If you want extra info please msg me.

Also see resolveWalletEns(Wallet wallet) in AWEnsResolver, it takes a Wallet and returns any ENS name.
ENS name in Ethereum is like DNS - you can assign a name which represents an address (as opposed to a name that represents an IP address).
There is also a convenience method provided by ENS that checks if there's any ENS name given an address (similar to whois). This is great for wallets.

OK> Clear. I will start on this issue.

@JamesSmartCell In title, it is mentioned "Fiji" so should this changes applied to "Fiji-implementation" branch?
I checkout from master branch for now, but if we need to checkout from "Fiji-implementation" let me know. I did my work done for My wallet screen changes.
There are two screens I can see on Zeplin.

Below is current implementation in application (only left side screen from Zeplin)

So right side view of Camera is pending. Will that change be in this issue or will be a separate one?
If we need to implement then I am unaware about the functionality of Camera view, so if you can shade of the purpose.

@JamesSmartCell Shall this ticket be closed as PR is merged.

Closed, based on confirmation from @JamesSmartCell  over phone.


Rename StatisticAnnouncement#policy_areas to topics

This preserves consistency as statistic announcements relate to topics
rather than policy areas.

It seems this may well be incorrect given the renaming that occurred was actually the inverse i.e. topic to policy area. According to alphagov/whitehall#2240 (although this didn't get merged but referencing here for intent), alphagov/whitehall#2249 and alphagov/whitehall#2349.
If not, then colour me confused. :see_no_evil:


Secrets in environment variables

Library name and version
Azure.Identity latest
Query/Question
Is it fairly safe to keep the secret in the system environment variables?  I am doing this for local development.  Our Azure subscription is behind protected Torus tenant so I cannot use Visual Studio account to authenticate.
The entire team will need to have a secret in their system env variables.
Thanks
Environment
Windows 11
Visual Studio 2022 v17.2

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Hi @jrmcdona  - Are you able to use AzureCliCredential or InteractiveBrowserCredential instead? Those would avoid having to store and distribute secrets.


fix: assert.typeof(xxx, 'object') should assert record or null

This refinement makes the assertion meaningful (before it was just asserting the equivalent of any).

What’s the reasoning for any over unknown in this case?

Turns out that using Record<any, any> | null is the only combination that gives the proper type narrowing one would expect assert.typeof(xxx, 'object') to do.  It's by no means perfect, but provides the features I need to use assert.typeof to eliminate non-matching type possibilities.
It narrows to the intersection of the type with any kind of object (including arrays, etc) or null.
Unfortunately I was never able to find a way to have it type the same as assert(typeof xxx === 'object').  Schade.


Android <IP_ADDRESS> GetUI()

Xamarin.Auth version <IP_ADDRESS>
GetUI method is returning object.
Previous code:
var intent = auth.GetUI(context);
StartActivity(intent);

not working anymore.

I'm facing the same issue, a cast throws an exception. The returned object from GetUI(...) is of type MainActivity.

Tried casting it to an intent?
StartActivity((Intent)auth.GetUI(context));

that throws an exception in my case (will provide further exception information later on)

I'm also having problems with this, no exception is thrown but it doesn't do anything.
I tried the following code : StartActivity(((Android.Support.CustomTabs.CustomTabsIntent)ui).Intent);
And that gives me a context menu to select the app to open it in, but it gives me a strange selection of apps, none of which is a Browser

I am having the same problem, does anyone know or have a solution for this issue?
Kind regards.
Sergio

Seems that the only solution for me was to downgrade back to <IP_ADDRESS>.

Breaking change for upcoming features!
See (for more details): https://github.com/xamarin/Xamarin.Auth/issues/150
This is breaking change for new feature using Native UI instead of embedded web browser
controls/widgets/views.
|--------------------|--------------------|------------------------------|
| OS                 |  Old/deprecated    | new/secure                   |

embedded browsers
Native UI

Android
WebView
[Chrome] Custom Tabs

iOS
UIWebView
SFSafariViewController

--------------------
--------------------
------------------------------

New flag in Authenticator class
IsUsingNativeUI

https://github.com/xamarin/Xamarin.Auth/blob/portable-bait-and-switch/samples/Traditional.Standard/references01projects/Providers/Xamarin.Auth.Sample.XamarinIOS/TestProvidersController.cs#L92-L103
https://github.com/xamarin/Xamarin.Auth/blob/portable-bait-and-switch/samples/Traditional.Standard/references01projects/Providers/Xamarin.Auth.Sample.XamarinAndroid/MainActivity.cs#L83-L96
Android
System.Object ui_intent_as_object = auth.GetUI(this);
if (auth.IsUsingNativeUI == true)
{
    // Add Android.Support.CustomTabs package
    global::Android.Support.CustomTabs.CustomTabsIntent cti = null;
    cti = (global::Android.Support.CustomTabs.CustomTabsIntent)ui_intent_as_object;

}
else
{
    global::Android.Content.Intent i = null;
    i = (global::Android.Content.Intent)ui_intent_as_object;
    StartActivity(i);
}

IOS
System.Object ui_controller_as_object = auth.GetUI();
if (auth.IsUsingNativeUI == true)
{
    SafariServices.SFSafariViewController c = null;
    c = (SafariServices.SFSafariViewController) ui_controller_as_object;
    PresentViewController (c, true, null);
}
else
{
    UIViewController c = (UIViewController)ui_controller_as_object;
    PresentViewController (c, true, null);
}

See

Hi moljac,
It now compiles and runs, however the Navigate event and the Auth.Completed event are never fired. Could we get some documentation on how to use this new functionality in Xamarin.Auth correctly?

@Jake-Barrow
I'm working on 3 things right now:

Auth.Completed, Auth.Error, Auth.Canceled for UWP, WinRT (both Windows Store and Windows Phone)
Auth.Completed for Chrome Custom Tabs (Android) and this is PITA beast. I'm using experience of other guys that did hit that problem (navigation with Custom Tabs) earlier.
Adding support for custom uri schemas for all platforms (missing from Xamarin.Auth inception)
documenting biggest changes

Now few questions:

Do you use <IP_ADDRESS> version (latest) of the nuget?
Did you set Native UI in OAuth2Authenticator ctor?
If not, then this is a bug, because by default Xamarin.Auth should still use old embedded browser approach (UIWebView and WebView). The switch to Native UI (SFSafariViewController and Custom Tabs) is planned for mid April.

The native UI will present providers' login page (FB, Google, ...) but I need to implement API for [Chrome] Custom tabs and SFSafariViewController how to raise events with minimal API breakage (from current API).
I hated myself for this "object" breaking change. I redesigned this part of APIs twice or three times, because CustomTabsIntent and CustomTabsActivity inherit directly from Java.Lang.Object and not Intent or Activity... So, switch to object was minimal damage solution.

@moljac
Yes I was using <IP_ADDRESS> and yes I did set NativeUI = true in OAuth2Authenticator ctor. So are you basically saying this is partially implemented functionality? If so that's fine, it just seemed like it was done and I assumed I was using the package incorrectly :)

@Jake-Barrow
Yes, this is currently partially implemented functionality -  you can navigate, but not subscribe to events.
I would like more than you guys to be done with this functionality, but Xamarin.Auth has grown a lot.
<IP_ADDRESS> was pushed 10-15 minutes ago. UWP events implemented (OnCompleted, OnError,...)
https://www.nuget.org/packages/Xamarin.Auth/<IP_ADDRESS>

So, following this guide doesn't work at all now? or?
https://developer.xamarin.com/guides/xamarin-forms/cloud-services/authentication/oauth/

cti = (global::Android.Support.CustomTabs.CustomTabsIntent)ui_intent_as_object;
what am I supposed to do with the cti object afterwards? as the code stands right now it does nothing, do I launch some Url?

@JakubHolovsky
OK. Did you set IsUsingNativeUI in the constructor? If yes why? The sample above with if/then was simply to illustrate upcoming feature.
Yes you or Xamarin.Auth (preferably) will launch some uri.
https://github.com/xamarin/Xamarin.Auth/blob/portable-bait-and-switch/samples/Traditional.Standard/references01projects/Providers/Xamarin.Auth.Sample.XamarinAndroid/MainActivity.cs#L134-L145

@Brosten

Guides are intended to guide you through implementing some functionality
in this case OAuth authentication. Did you try to implement OAuth according to
that guide?
If answer to 1 is No, then where does it say it does not work (at all)?

To help you with answers:

You would get one compile error with the code from guide in link you provided.
It works with API changes!

Original code from that link:
    rootViewController.PresentViewController(authenticator.GetUI(), true, null);

New API
    rootViewController.PresentViewController( (UIViewController) authenticator.GetUI(), true, null);

So only cast will be needed.

Jake-Barrow was using new API by turning on boolean switch in constructor of OAuth2Authenticator.

@moljac thank you for your reaction, however I am a bit confused:
What I don't get if this is an "upcoming" feature, why is it in published nuget package when it basically does nothing ("work in progress" - does it work? does it not work? should we be using it?)?
https://github.com/xamarin/Xamarin.Auth/blob/portable-bait-and-switch/samples/Traditional.Standard/references01projects/Providers/Xamarin.Auth.Sample.XamarinAndroid/MainActivity.cs#L75-L75
This line says "// Default - false"
but if I have a look at the constructor:
public OAuth1Authenticator(string consumerKey, string consumerSecret, Uri requestTokenUrl, Uri authorizeUrl, Uri accessTokenUrl, Uri callbackUrl, GetUsernameAsyncFunc getUsernameAsync = null, bool isUsingNativeUI = true)
it is "true" by default, is it referring to some other default then?

@moljac Sorry, I didn't tell... I'm on Android. Not iOS.
I'm stuck in AndroidOAuthLoginPresenter, the "Login" method. I do the "GetUI()", returning a CustomTabsIntent. Trying to pass this intent to StartActivity() doesn't work. Nor does passing the "Intent"-property of the CustomTabsIntent.
Can't really see where to go from here. Or should i not use the method described in my link above, using the OAuthLoginPresenter?

@JakubHolovsky @Brosten The functionality was accidentally committed to the Nuget package, so it is unfinished code published. If you follow the current documentation as normal, but change the constructor of the OAuthAuthenticator to have isUsingNativeUI = false it will follow the old functionality as normal.
There are some slight changes in the fact that GetUI() now returns an object. So follow the code here to use the old functionality for Authentication UI
Android:
global::Android.Content.Intent i = null;
i = (global::Android.Content.Intent)ui_intent_as_object;
StartActivity(i);

iOS:
UIViewController c = (UIViewController)ui_controller_as_object;
PresentViewController (c, true, null);

The other option is to downgrade the Xamarin.Auth package to <IP_ADDRESS> until the new functionality is finished so you can use the new NativeUI options

@JakubHolovsky
You re right fixed. The default was true. Sorry.
Just to fix "path to long" on Windows bot on our CI servers and <IP_ADDRESS> will be published

@Jake-Barrow @moljac thanks, even though I guessed that it was incorrect and passed false in the constructor (basically followed the example) none of the Error / Completed / BrowsingCompleted events were being triggered (was using <IP_ADDRESS> version before). Reverted back to <IP_ADDRESS> for now.

@Jake-Barrow The "only" accident was - wrongly set variable.
Nuget is published through Ci servers and setting up everything (new lanes/jobs) for alpha/beta testing was too much trouble. I decided to push silently those new features so I can test them.
As soon as I fix building Windows bits <IP_ADDRESS> will be published

@JakubHolovsky

none of the Error / Completed / BrowsingCompleted events were being triggered

This was for <IP_ADDRESS>?
I will separate my sample projects that use nuget dependencies into separate repo. They have 16-17 OAuth providers. Then you can test them with your API keys.
I retested Android and iOS samples yesterday with nuget <IP_ADDRESS> and was able to get auth_token. On Android there are Error events raised for localhost and <IP_ADDRESS> redirect_urls. IOS code was cleaned up last week, so I catch exception and parse url properly. Android tries to open it.

@moljac yep, correct, that was for <IP_ADDRESS>. It did re-direct me to Twitter login page, I logged in successfully and then nothing was triggered (my URLs remained the same as I as using in version <IP_ADDRESS>).

Twitter is OAuth1?
I have to check that inheritance chain. I am (actually we are - 2 interns testing) are mostly wokring on OAuth2.

@moljac yep, correct OAuth1

@moljac Ahh, setting the bool to false solved my problem.... Thanks for your fast feedback

@Brosten You re welcome (BTW no need to thank me)
I managed to compile <IP_ADDRESS> on CI server, just few tests and then it will be like it used to be.
For next few weeks (I hope week or two depending on other stuff) you will not need this boolean set to false, just continue use old constructors.

Hmm, just got stuck on my next step:
// UserInfoUrl = https://www.googleapis.com/oauth2/v2/userinfo
var request = new OAuth2Request ("GET", new Uri (Constants.UserInfoUrl), null, e.Account);
var response = await request.GetResponseAsync ();
Doing that returns "Error authenticating: invalid_grant". Any idéas?

@Brosten
Is this new OAuth app?
Read: did you create this app with API key etc on google portal recently?

@moljac Well, created it a mounth or two ago.

@Brosten
OK. This is why new API must be implemented. Current (OLD) API uses "embedded" WebBrowsers (WebView and UIWebView) and this is considered security hole. So, they banned all apps created after 2016-10-27 (27 - not sure, 2X where X=something) and this is why you get invalid _grant. Older apps will work until 2017-04-20 and then all apps using embedded browsers will not be able to use google as OAuth provider.
Currently I use old apps and new created for Facebook (they have posibility to control usage of embedded browser on their backend) and github for explicit flow.
I have 16-17 providers, but re-tested most of them like week or two ago. most of them worked.

Hi
I get this exception when I update to v<IP_ADDRESS>:
Exception while loading assemblies: System.IO.FileNotFoundException: Could not load assembly 'Xamarin.Android.Support.CustomTabs, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken='. Perhaps it doesn't exist in the Mono for Android profile?
File name: 'Xamarin.Android.Support.CustomTabs.dll'
   at Java.Interop.Tools.Cecil.DirectoryAssemblyResolver.Resolve(AssemblyNameReference reference, ReaderParameters parameters)
   at Xamarin.Android.Tasks.ResolveAssemblies.AddAssemblyReferences(DirectoryAssemblyResolver resolver, ICollection`1 assemblies, AssemblyDefinition assembly, Boolean topLevel)
   at Xamarin.Android.Tasks.ResolveAssemblies.Execute(DirectoryAssemblyResolver resolver)

When I add Xamarin.Android.Support.CustomTabs I get 6 new exceptions. Any idea how to solve this?

I tried version <IP_ADDRESS> and even though the default for OAuth1Authenticator isUsingNativeUI is false it still returns OAuth1Authenticator that has isUsingNativeUI true.
I believe it's because of https://github.com/xamarin/Xamarin.Auth/blob/portable-bait-and-switch/source/Xamarin.Auth.LinkSource/UINativeNonIntegratedBrowsers/WebAuthenticator.Native.cs
which is using "true" for it's default value while OAuth1Authenticator constructor (even though it's false) is not using the value from constructor anywhere.

I'm stuck on this issue as well. Can anyone help? I'm using <IP_ADDRESS> and trying to get Twitter to work. It crashes on the Start Activity with invalid cast. How can this be fixed? The "IsUsingNativeUI" Doesn't appear to be a member of Authenticator.

@shayo
See above Jake-Barrow explained in short what I explained 2 weeks ago - you need to cast.
Android:
global::Android.Content.Intent i = null;
i = (global::Android.Content.Intent)ui_intent_as_object;
StartActivity(i);

iOS:
UIViewController c = (UIViewController)ui_controller_as_object;
PresentViewController (c, true, null);
In the future Xamarin.Auth GetUI() will return CustomTabs which derive from Java.Lang.Object and it was least-damage breaking change to return System.Object and leave (force) user to cast to proper class.
HTH

@moljac . It only works partially (in Android). It does work for OAuth2 (say, facebook), but crashing on that cast when trying Twitter (I believe it is OAuth1).
[MonoDroid] System.InvalidCastException: Specified cast is not valid.
[MonoDroid]   at ClimbTag.Droid.OAuthLoginPresenter.Login (Xamarin.Auth.Authenticator authenticator) [0x0000d] in :0

OK. Clean & rebuild solved it, but I sill don't get the callback. It just continues to show the twitter web page.

To summarize - problem on Android:
<IP_ADDRESS> - crashing on casting from OAuth1
<IP_ADDRESS> - no crash, but no callbacks for Oauth1.
Can someone help?

Just saw that <IP_ADDRESS> is out.
Good news - Oauth1 doesn't crash on casting anymore.
Callbacks are still not working.

for who facing casing issue in ios  set auth.IsUsingNativeUI = true;

@ads What is "casing issue"?

when instantiating the OAuth1Authenticator and calling auth.GetUI() app is crash and gives can't cast exception but when setting IsUsingNativeUI to true its working

Can you try this with the latest version of Xamarin.Auth please.
Regards

no response after two weeks


Wrong result dealing with control escape chars

Colored output contains escaped chars like \x1b[30m;, so pattern containing m or ; may results wrong matches.
I am aware there's always a workaround to remove the color. Or should fzy remove escape code while  choices_fread?

fzy should not be aware of escape sequences, the matcher is doing the right thing here.  That's why instead of ls --color=always there is ls --color=auto.


Automatically manage entity proxies for the OMASs

OMRS repository connectors have the ability to store entity proxies to support relationships to entities that are stored/managed in a different repository.
Today the OMASs are responsible for creating the entity proxies but this is not optimal.  It would be an easy extention to the Enterprise OMRS Repository Connector implementation of addRelationship() to automatically create the entity proxies in a repository if needed.

More detail on the design ...
In the addRelationship method, after parameter validation, the enterprise connector should query both entities in the cohort to ensure they are real and cache them in local variables.
It then starts to loop thorugh the reposiotry connectors.  When it is trying to add the relationship to a specific reposiotry, it should test whether the repository knows about the entity (using getEntitySummary to pick up the proxies) and add the entity proxy if it does not.  If the repository does not support entity proxies then the enterprise connector should move to another repository.  If no repository supports both the relationship type, and has the entities (either full or proxy) then EntityNotKnownException is thrown.


[Bug]: Removing bottom navigation bar via ImageEditorCustomWidgets causes layout to skip when moving items

Package Version
3.0.4
Flutter Version
3.22.2
Platforms
Android, iOS
How to reproduce?
Not sure if this is the right way to remove the bottom navigation bar, but I am doing:
ProImageEditorConfigs(
  customWidgets: ImageEditorCustomWidgets(
    bottomNavigationBar: const SizedBox(),
  ),
),

This seems to work fine. However, when dragging items around after that. The layout shifts back to having the space for the bottom navigation bar (so everything shifts upward). The preview moves up and shrinks. And the "delete" drop target stays in the right place but the actual drop target itself seems to be offset upward as well.
Video demonstrating:
https://github.com/hm21/pro_image_editor/assets/869261/c7b20684-ace3-42e6-aee8-751b8756f829
Logs (optional)
No response
Example code (optional)
No response
Device Model (optional)
No response

Thanks for your bug report.
Yes, it's true, if you remove the appbar or bottombar you will run into trouble. In the example I also added this comment “kBottomNavigationBarHeight is important that helperlines will work” but I forgot to add it to the documentation in the customWidgets.
However, since I released version 3.0.0, is it maybe possible that I can change the code a bit that it will work. I will do some tests and give you later feedback here.

I have released version 3.0.5 which should resolve this problem. Please let me know if there is still an issue.

Amazing, thank you! It works great 😎


Feature_Request : Pricing New Page

Is your feature request related to a problem? Please describe.

Hello all,
The pricing section of the website only have 3 cards . I want to make the VIEW ALL button to be working.
Describe the solution you'd like

I want contributors to make a separate page of the website. When any user clicks on the VIEW  ALL button they maybe redirected to a page which includes the TOUR PLACE CARDS as same as it is.
Describe alternatives you've considered

The Pricing page might contain

the same price cards
With proper place name
with a star review
price
A book now button
also include the same hovering effect to all the place-cards
NB :Please include at least 10 place-cards to the website

Demo Video/Screenshot (Mandatory )
https://github.com/apu52/Travel_Website/assets/114172928/a2b7c64f-9a77-468a-892b-90db2753419c
Are you contributing under any open-source program ?

JWOC'24

Whoever will comment first on this issue with JWOC mentioned on it I will assign this issue to her/him

@apu52 I want to work on this issue.

@PranavBarthwal You are assigned. You can start working on this issue.
You have 4 days to complete it


webview audio record not working

Your project works perfect but audio record not working.
How can I enable audio record?

I use your project, I just change webview link.

I was asking what audio you’re trying to record and how you’re trying to record that audio.
Can you show some code? Either you’re trying to initiate the recording in Android/Java, or in JavaScript. There must be some code which does the recording.

Permissions for audio recording is not configured


Can´t recover wallet?

I created a wallet last week with a balance. Successfully sent an LN transaction out as well. Wrote down my seed words.
This week I opened the app (added as PWA on mobile on Brave) and balance is 0. In Settings, ¨Tap to reveal seeds words¨, the seed words are different than those I backed up.
I don´t see anywhere in the app to restore wallet, not even in ¨Secret Debug Tools¨

We don't have the recover flow just yet. Though you can use the words to recover the on chain funds on another wallet. Did you happen to make a backup of the state file at all? That's where the lightning data is stored.

Ok. Wasn't much - was just testing wallet out. I'll just wait until the recover flow is in and then test that out.

Really sorry about that. We're adding a warning about the recovery process on the next release.
Some questions about the Brave PWA on mobile:

Is this on iOS or android?
Do you happen to know if you've done any "clearing" or "wiping" of the browser manually since then?
So the PWA app that's installed on your mobile home screen, that's what was cleared out? What happens if you just access app.mutinywallet.com from the normal browser? Still wiped out?


Disabled C++ test batchnorm_test.cc - InGrad

https://github.com/apache/incubator-mxnet/blame/master/tests/cpp/operator/batchnorm_test.cc#L533
Test has been disabled as part of MKLDNN

The same comments as https://github.com/apache/incubator-mxnet/issues/10131


allow empty string responses

Fixes https://github.com/wheresrhys/fetch-mock/issues/517

Coverage remained the same at 96.991% when pulling a12d7ac038a6146fd108c1d18d2abb0f1870602e on empty-string-responses into 6fdb8671b02577155994bcd8ba01ae4319c0a8f7 on master.


Jiahao/code quality

Improve code quality. Fix minor issues.

Codecov Report

Merging #255 (0d52b65) into master (2ddc229) will decrease coverage by 0.28%.
The diff coverage is 28.12%.

@@             Coverage Diff              @@
##             master     #255      +/-   ##
============================================
- Coverage     68.38%   68.10%   -0.29%
- Complexity      594      596       +2
============================================
  Files            97       97
  Lines          2059     2069      +10
  Branches        210      213       +3
============================================
+ Hits           1408     1409       +1
- Misses          586      594       +8
- Partials         65       66       +1

Impacted Files
Coverage Δ

src/main/java/seedu/address/MainApp.java
0.00% <0.00%> (ø)

...main/java/seedu/address/commons/core/Messages.java
0.00% <ø> (ø)

...java/seedu/address/logic/commands/FindCommand.java
47.05% <0.00%> (-2.95%)
:arrow_down:

.../seedu/address/logic/commands/PasswordCommand.java
84.61% <0.00%> (-15.39%)
:arrow_down:

src/main/java/seedu/address/ui/LoginScreen.java
0.00% <ø> (ø)

src/main/java/seedu/address/ui/SetUpScreen.java
0.00% <ø> (ø)

.../java/seedu/address/commons/util/PasswordUtil.java
92.30% <100.00%> (+0.64%)
:arrow_up:

...rc/main/java/seedu/address/logic/LogicManager.java
86.36% <100.00%> (ø)

... and 1 more

Continue to review full report at Codecov.

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


Add '--wait' flag to partitions.js

This will keep a connection to ringpops open.

LGTM

Looks good!

Squashed the commits. I`ll merge as soon as travis-ci check is done.


Combine Taskbar buttons while hiding labels.

Describe the solution you'd like
A way to not combine taskbar buttons while still hiding labels, which appears to not already be a feature in Windows.

Describe alternatives you've considered
I've attempted using regedits to change the width of taskbar buttons to hide the text while grouping is on, to no avail. However, even if this had worked, it's clunky and would be hard to undo, so being able to easily change it with TaskbarX would be preferred anyways.
Additional context

Duplicate of https://github.com/ChrisAnd1998/TaskbarX/issues/124

To be able to focus on major/current bugs I had to close all issues. The issues section has gotten very messy to a point that I couldn't follow it anymore. Most issues are duplicates, very old or are already fixed/solved. Feel free to reopen this issue if you still have this issue on the latest version of TaskbarX to get my attention. All feature requests have been put in a list here: 💡| Feature requests #870
I hope you understand ❤


rc3.2 Delegated module initialization fails

When using delegated modules, the initialization of the module fails when attempting to resolve any imported modules in the delegate with the below error.
Initialization of sharing external failed: TypeError: webpack_modules[moduleId] is not a function

@SapientMatt can we do another zoom / do you have a reproduceable example?

Hi, yup let me set up an example today quick.
Can do zoom call at 11am again today or tomorrow if that works?

Issue replicated here
https://github.com/SapientMatt/module-federation-examples/tree/issue/976
Updated the nextjs-ssr-delegate-modules example
To replicate
npm install for both checkout and home
from example root (nextjs-ssr-delegate-modules)
npm run start:checkout
npm run start:home
or
npm run build:home
This is giving slightly differnt issues from what I saw before, with intialization failing with
"Initialization of sharing external failed: TypeError: Cannot read properties of undefined (reading 'call')"
and actual loading now failing with "webpack_modules[moduleId] is not a function"

okay ill give it a try

blind guess, but try import(federation/utiliits/src/importDelegateModule) instead of importing the whole utils and destructure off it.

@SapientMatt Use this for the time being, ill see if theres a good solution to the root cause issue
const { importDelegatedModule } = await import(
    '@module-federation/utilities/src/utils/importDelegatedModule'
  );

ive updated the example, its still in PR phase:
https://github.com/module-federation/module-federation-examples/pull/2973

@ScriptedAlchemy so after a bit of further digging and a bit of a different approach, I figured out that the reason it seems to be failing is that the library I'm importing, is not yet loaded. Making the lib, in my case a wrapper around your utils package, eagerly shared, seems to solve this problem.

Ok, the above works while debugging, but falls over when actually building


Terminal/console layout not correct using TextDialogs

I am using robot framework 2.8.6 with a third party TextDialogs library for purely text based dialogs (https://code.google.com/p/robotframework-textdialogslibrary/).
Using the library stand-alone works fine and I am able to create a full screen text dialog (in my case a yes/no choice menu). However, when I import it as a library in a test case and call my dialog, the dialog appears on the left of the terminal (SSH terminal to a remote machine running the tests) after executing the test using pybot. I have tried the --monitorwidth option but this has no effect on the dialog.
Since the stand alone python code in the library works just fine, I suspect an issue in pybot. Do you provide support for this?

We don't unfortunately have time to investigate problems with integration to third partly libraries or tools Please contact the developers of TextDialogs and ask how it should work. If there's something that should be changed in Robot Framework itself to make it work better, please submit a new issue.


custom authentication failure response

Hi Guys,
Great work here.
I read some previously closed issues namely #16 however, i am having the same issue with the authentication failure response.
<?php

namespace App\EventListener;

use Lexik\Bundle\JWTAuthenticationBundle\Event\AuthenticationFailureEvent;
use Lexik\Bundle\JWTAuthenticationBundle\Response\JWTAuthenticationFailureResponse;

class AuthenticationFailureListener
{
    /**
     * @param AuthenticationFailureEvent $event
     */
    public function onAuthenticationFailureResponse(AuthenticationFailureEvent $event)
    {
        $data = [
            'meta' => [
                'code' => 401,
                'status' => 'error',
                'message' => 'Bad credentials, please verify that your username/password are correctly set.',
            ]
        ];
        $response = new JWTAuthenticationFailureResponse($data);
        $event->setResponse($response);
    }
}

outputs
{"code":401,"message":{"meta":{"code":401,"status":"error","message":"Bad credentials, please verify that your username\/password are correctly set."}}}%

However, based on the class JWTAuthenticationFailureResponse i do not see a way to do this correctly. My work around is to use $response = new JsonResponse($data);

{"code":401,"message":{"meta":{"code":401,"status":"error","message":"Bad credentials, please verify that your username/password are correctly set."}}}

Yea because JWTAuthenticationFailureResponse constructor first argument is message and it will replace the default message in that class. If you wish to insert some custom data while maintaining the output format from that response, simply use $response->setData($data) instead.
Else, use $response = new JsonResponse($data); for your own output format is better choice.

Closing as explained. Thank you for the kind words and the discussion.


Capacity factors added to Components

Pull Request Description
What issue does this change request address?
#121
What are the significant changes in functionality due to this change request?
New input node for Components allows user to apply capacity factors to the nameplate capacity. <capacity_factor> is applied parallel to the <capacity> node. Currently only allows Synthetic History (ARMA) input to be applied as a per-unit-time capacity factor.
Adding new tests based off the initial wind workshop case (simplified with Sine ARMA). Results are tested against the previous way that the wind workshop case was implemented: <capacity> took a Synthetic history valued param and the <multiplier> would contain the intended nameplate capacity. A copy of that script also included in test directory.

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass.
[ ] 5. If significant functionality is added, there must be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large tes.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added, the the analytic documentation must be updated/added.
[ ] 9. If any test used as a basis for documentation examples have been changed, the associated documentation must be reviewed and assured the text matches the example.

PR is now ready for review


Update go version

The projects path has been updated and it's been vendored. Also allow you to use -o json, -o yaml, or -s.

/assign @micahhausler

/lgtm

Looks like you need to rebase @christopherhein

@nckturner updated.


Source tarball for v2.0.0 is different here and at pypi

I downloaded the sdist tarball from pypi and from github and surprisingly after unpacking they differ significantly:
$ diff -qr argcomplete-2.0.0-pypi/ argcomplete-2.0.0-github/
Only in argcomplete-2.0.0-pypi/: argcomplete.egg-info
Only in argcomplete-2.0.0-github/: common.mk
Only in argcomplete-2.0.0-github/: docs
Only in argcomplete-2.0.0-github/: .github
Only in argcomplete-2.0.0-github/: .gitignore
Only in argcomplete-2.0.0-github/: Makefile
Only in argcomplete-2.0.0-pypi/: PKG-INFO
Files argcomplete-2.0.0-pypi/setup.cfg and argcomplete-2.0.0-github/setup.cfg differ
Only in argcomplete-2.0.0-github/: tox.ini
$

The most important differences are the egg-info, docs, PKG-INFO, and tox.ini.  This makes the sdist tarball at pypi hardly usable.  For example, it is not possible to run tests using tox there.

I'm not sure how this is an argcomplete bug. The tarball on github is generated by github while the tarball on PyPI is generated by build. Also, just to be clear, I don't run the tests using tox and I never committed to supporting tox in the argcomplete API.
I checked and the tarball on PyPI appears to be usable for installing the package from source, which is what it's meant for. Please let me know if you find otherwise, and we can reopen this.


ARROW-17288: [C++] Adapt the CSV file format to the new scan API

This also adds a utility method arrow::compute::DeclarationToReader.
This does not fully satisfy ARROW-17288 as it only adapts the CSV file format.  However, this was the most complicated case, and I had to convert some of the common test utilities as well.  So I am hopeful the other formats will be more straightforward.

It's a little bit unclear to me what happens type-wise when csv column names are duplicated and types of duplicated columns don't match. Do we currently raise or promote?

This should be fully configurable by the schema evolution strategy.

The user will ask for a column in the dataset schema and we will know the name of that column
The CSV reader will supply its list of column names (which will contain the duplicate)
The evolution strategy will be given that list (the inspected fragment), the dataset schema, and the requested columns in the dataset schema and must tell us which columns to load from the CSV.

I think the default behavior picks one arbitrarily and tries to load it.  If the type for that column in the file doesn't match the type for the requested column in the dataset schema then a runtime error should be generated.
Happy to change the default to simply give an error when the fragment has duplicate column names.  At this point in time I don't think we actually infer the types but we could, so that we could support a "find first column with the right name and a compatible type".
In most cases, if the fragments have duplicate column names, any amount of guessing on are part isn't likely to be correct.
Sounds like a good test case to have though to verify whatever it is that we do.

I think the default behavior picks one arbitrarily and tries to load it. If the type for that column in the file doesn't match the type for the requested column in the dataset schema then a runtime error should be generated.

That sounds ok correctness wise, but also like it could make execution non-deterministic for preventable reasons?

I also lean towards having an explicit error if the column is ambiguous. Trying to type-match also feels somewhat brittle to me.

I also lean towards having an explicit error if the column is ambiguous. Trying to type-match also feels somewhat brittle to me.

There was a slightly related discussion here: https://github.com/apache/arrow/pull/13938#issuecomment-1224349673 where the following was suggested:

Hmm, it would be nice if this could work even without disabling schema evolution. Perhaps a heuristic is possible?

if a field name is unique, do as usual
if a field name is non-unique, require that it has the same number of occurrences in both schema, and iterate on those pairs in order

That also seems reasonable. As long as the behavior is well-specified.

It's also something that could be left for a future PR, IMO

It's also something that could be left for a future PR, IMO

ARROW-18388


add support for non-amd64 arch gcr.io images

Currently all the gcr.io images used in kubespray can only run on x86.
Also gcr.io has not fully support multi-arch docker images.
Add extra var "image_arch" (default is amd64) to support running other
platforms, like arm64.
Change-Id: I8e1c9af533c021cb96ade291a1ce58773b40e271

ci check this

This is very interesting. Thx for the PR. I was just thinking about ARM devices the other day!
Looks like yamllint failed. If you'll get it fixed up, I'll re-kick CI.

@ant31 @rsmitty Updated. PTAL. Thanks.

@mattymo Updated. PTAL. Thanks.

@xd007 Thank you !


Netlfiy Logo and Tooltip for Login and Sign Up Page [DRAFT]

Netlfiy Logo and Tooltip for Login and Sign Up Page
Issue Number(s): #191
Status: need to fix green waves on mobile view
Checklist

[ ] Requirements have been implemented
[ ] Acceptance criteria is met
[ ] Database schema docs have been updated or are not necessary
[ ] Relevant reviewers (EM) have been assigned to this PR

Critical Changes

Database change / migration to run
Environment config change
Breaking API change

Related PRs
PR Number(s): #number_of_pr
How To Test
The steps to test the functionality of your ticket. Include edge cases and testing of error handling, if applicable.

Looks great! The only fix needed is the green waves on mobile which you've already mentioned.


[ASCII-1986][e2e] Add tcp transport over ssh connection to host

What does this PR do?

Implement HTTPTransport and HTTPClient methods on an host to enable making http requests over the ssh connection.
Motivation

Avoid having to use host.Execute("curl ...") or host.Execute("Invoke-WebRequest ...") to perform http requests, and use "native" Go http instead.
This stems from a recent incident where the latest Windows AMI (amazon/Windows_Server-2022-English-Full-Base-2024.07.10) takes 3~4 more seconds to start Powershell, making each command considerably longer (each command starts a new instance of Powershell), which results in timeouts.
Additional Notes

This PR also updates the agent client to use this new feature instead of running commands.
Possible Drawbacks / Trade-offs

There might be an impact when reconnecting.
Describe how to test/QA your changes

Very cool improvement, congrats!


Add downsampled images as context instead of screenshot.png?

Would it be advantageous to keep a collage of downsampled previous images, maybe to 160px x 90px and just stack them in a line left to right, one after another, and constantly pass this image as additional context for each action like here is a timeline of previous states the model has traversed through?
FYI: I would be happy to draft and code this feature out!

Out of curiosity, Doesn't the Vision model has any sort of "Token Limit" like it has in GPT4?

Got it. I'll go draft up a PR for the previous image stacks! I'm planning on just making it an immediate previous message in pseudo_messages.
Also, regarding the token limit, correct me if I'm wrong, is the GPT4V context window up to 128k tokens?
If so, would there be a real concern about hitting the token limit? With some calculations, where x = one dimension of maximum image size you get x = 14044, meaning you can pass up to a 14044px x 14044px image. To hit the token limit, you would need to pass a lot of images to hit 2 * 10^8 total pixels.

Got it. I'll go draft up a PR for the previous image stacks! I'm planning on just making it an immediate previous message in pseudo_messages.
Also, regarding the token limit, correct me if I'm wrong, is the GPT4V context window up to 128k tokens?
If so, would there be a real concern about hitting the token limit? With some calculations, where x = one dimension of maximum image size you get x = 14044, meaning you can pass up to a 14044px x 14044px image. To hit the token limit, you would need to pass a lot of images to hit 2 * 10^8 total pixels.
@michaelhhogue @shubhexists

@klxu03 Can this be closed since #57 was merged?

closing since old and less relevant now


Custom Babel config

I'm trying to use styled-jsx along with mdx-go, but I'm breaking the code trying to modify the babel plugins: https://github.com/zeit/styled-jsx#getting-started.

Currently the webpack babel-loader is set to babelrc: false to avoid unintentional conflicts. Have you tried with a custom webpack config? (Not sure if that would work right now, but it might)

My current solution is to copy and paste the necessary code from lib/config.js and modify as I need:
// webpack.config.js
const path = require('path');

const babel = {
  babelrc: false,
  presets: [
    [
      require.resolve('@babel/preset-env'),
      { modules: false }
    ],
    require.resolve('@babel/preset-react'),
  ],
  plugins: [
    require.resolve('@babel/plugin-proposal-class-properties'),
    require.resolve('@babel/plugin-proposal-export-default-from'),
    require.resolve('@babel/plugin-proposal-export-namespace-from'),
    require.resolve('@babel/plugin-transform-runtime'),
    require.resolve('@babel/plugin-syntax-dynamic-import'),
    require.resolve('styled-jsx/babel'), // this is all I need for styled-jsx
  ]
}

module.exports = {
  module: {
    rules: [
      {
        test: /\.js$/,
        exclude: /node_modules/,
        use: {
          loader: require.resolve('babel-loader'),
          options: babel
        }
      }
    ],
  },
  resolve: { // I tried babel-plugin-module-resolver but this way is easier
    alias: {
      'components': path.resolve('./src/components'),
      'utils': path.resolve('./src/utils'),
    },
  },
};

A better solution could be a mdx-go.config.js file with custom and "secure" configs. As well as Next.js with next.config.js: https://nextjs.org/docs#custom-configuration.

Thanks! I don't think mdx-go should have its own config – it would add to the API surface area and this is really a babel configuration issue. With the changes from babel 6 to 7, supporting custom babel configs for all use-cases is very tricky, but I'd consider allowing the babel-loader to pick up custom configs for v7 only. Would need to test this out and make sure it works first

I just tested out a custom babel.config.js locally with styled-jsx and it seemed to work with the latest version... I'm wondering if you've run into a bug somewhere, so it'd be helpful if you could test this out on your end.
In my babel.config.js I have:
module.exports = {
  plugins: [
    'styled-jsx/babel'
  ]
}

Running<EMAIL_ADDRESS>as a global install


Adds the Commit Lint plugin

I think it's a good one! What was once 5 lines of Ruby in a Dangerfile is now over 450 lines of code in the form of a well-tested and documented gem.
Am I doing this right?? 😀  😀  😀
Check it: https://github.com/jonallured/danger-commit_lint

Failed to parse danger-commit_lint
commit_lint
Run each commit in the PR through a message linting.
Lint all commits using defaults

commit_lint.check

Warn instead of fail

commit_lint.check warn: :all

Disable a particular check

commit_lint.check disable: [:subject_period]

Methods
check - Checks the commits with whatever config the user passes.

  Generated by :no_entry_sign: danger

Hmmm, I'm green when I run bundle exec danger plugins lint on my machine - what am I missing??

This bit is wrong in the Dangerfile:     markdown "### Failed to parse #{plugin}" if $0 != 0 it should be unless $0.success? the $0 does not represent the error code, but an object, so it always fails

So this is almost perfect - ideally, I'd love to see your usage section move into the comments above the class, so that you know what the checks are by just looking at the overview we see above ( which is what'll show on the site. )
Right now you don't know what it's going to check for without going to a different repo - which ideally can get fixed in the next release - other than that, this looks awesome!

Great job 👍

Yeah, I think it'll really help to see this on the site and then I can tweak it from there. ❤  ❤  ❤


[Firebase messaging] iOS gets 2 times the payload of the message while Android gets it only once

I have noticed that iOS is receiving twice the payload of the message while android is getting the payload only once. iOS shows 2 notifications while android only get's one.

I also have noticed that it happens in some devices, for me, 2 iphones (8 plus and 7) behave on differente ways. The 8 plus recive 2 notifications while the 7 only 1.

If the problems still isn't solved, please ask at the new  home for Firestore at https://github.com/FirebaseExtended/flutterfire and link this issue to it. Thanks!

Hi @ZatacaSystems
for what I can see the issue is related to a FlutterFire plugin,
please open the issue in the dedicated github.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
thank you


Update package index with latest published versions

Update package index with latest published versions

closing and reopening for a fresh build


After upgrading td-agent from version 3.8.0 to 4.1.0, cpu utilization of td-agent got increased

Describe the bug
Td-agent of version 3.8.0 and elasticsearch of version 7.8.0 is deployed in kubernetes environment. After upgrading td-agent to verison 4.1.0, we observe that there cpu utilization is very high

In the fluentd version 3.8.0 , average cpu utilization is 0.4 core
In the fluentd version 4.1.0, average cpu utilization is 1 core

To Reproduce
Deploy fluentd of version 3.8.0 in kubernetes environment with cpu configuration of 1 core and 1gb ram with below configuration

After upgrading the fluentd version 4.1.0 with the same configuration cpu consumption of fluentd got increased to 1 core.
Expected behavior
Cpu consumption of fluentd should not increase after migrating from the version 3.8.0 to 4.1.0
Your Environment
- Fluentd version:
- TD Agent version:4.1.0
- Operating system: centos 7.9
- Kernel version:

Your Configuration

Your Error Log
There are no error messages

Additional context
No response

Could you check #3382?
Probably your issue is same with it.

@ashie it seems that there is a bug that it gets fixed with 4.2.0 version. Based on the configuration file given by Varun do you see any possible workaround, configuration wise?

Hi @ashie ,
Can you please confirm any this issue can be fixed by changing any configuration in td-agent or we have to upgrade to fluentd 4.2.0 ?
Regards,
Varun S

Sorry, it seems a different issue.
td-agent 4.1.1 (Ruby 2.7.3 & excon 0.80.1) is affected by #3382 but td-agent 4.1.0 (Ruby 2.7.2 & excon 0.78.1) isn't.

Hi @ashie ,
td-agent 4.1.0 (Ruby 2.7.2 & excon 0.78.1) --->Yes
Regards,
Varun S

Hi @ashie,
With the above configuration we are not facing any issues so closing this issue, But we are facing issue when http plugin is used. So different issue is raised for the same. Please find the bug id:- https://github.com/fluent/fluentd/issues/3324 .


Persist Table Filters in the API Explorer

This persists the table filters in the API explorer into the URL. So this implements kind of #2525, but only for the API explorer table. However I implemented it in a generic way, so that every page that is using a table could use it. This stores the table filters, whether the table filters are open and the current search query. This doesn't store order, page or rowsPerPage. These are more difficult and up for somebody else 😆
I don't like the result as it feels very complicated. CheckboxTree, Select and material-table don't feels like they were made for this. State isn't public, so I can't move the state upwards into other components. I fixed this for CheckboxTree and Select, but still think they need a refactor in the future. The solution on material-table feels hacky, but works 😞 But at least I like the hook and that it's just a couple of line to use this.
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)

Related to https://github.com/spotify/backstage/pull/2730

@fox32 does this replace #2730?

@freben I know you had some thoughts about the implementation for this feature, could you maybe take a look at this to see if it seems sane or if theres some things not considered?

@Fox32 does this replace #2730?

Not directly, this one is for the API explorer and the other one for the catalog. But it could replace it, if we decide to use the standard table for the catalog which is probably the long term goal. This solution is generic and could be used everywhere

Hey @Fox32 - we're gonna have an internal review of this stuff on monday as a group, sorry it's taking so long, but internally @ Spotify we've had some problems with our own internal implementation of this, so want to make sure that we don't have any of those issues with filters moving forward. Will get back to you next week on this!

Don't merge that yet, I would like to have a look at the comments first 😉 had no time to implement them yet😁

Alright, won't merge before 0.2.0 either way, so there'll be space to iterate :grin:

I added changesets and improved the hook to have a separate state. For now I would like to keep it this way and improve the open things in the future (reacting to url changes, uncontrolled vs controlled for Select and CheckboxTree).

If it builds, it's ready

@Fox32 wise words to live by ;)


Import extension host certificates Steps

There is in issue in the "Import extension host certificates" procedure.  Both Step 3 and Step 5 are the same thing...I would expect that Step 3 could be removed and steps 4 and 5 could be renamed to 3 and 4, respectively.

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

ID: c3b84311-7c1e-bc20-70e1-9e812ff27824
Version Independent ID: c33812da-28e5-639b-dbf2-ccf8ab413309
Content: Prepare for extension host for Azure Stack
Content Source: articles/azure-stack/azure-stack-extension-host-prepare.md
Service: azure-stack
GitHub Login: @mattbriggs
Microsoft Alias: mabrigg

@SqlTechMike step 3 creates the directories, and step 5 puts the certs into those directories. Step 3 also forces a directory structure, so I am unsure how they are the same.

I'm referring to just the "Import extension host certificates" portion of the process.  At this point all of the directory structures have been created and you are simply adding the Admin Extension Host and User Extension Host certificates.  Step number 3 reads "Import the certificate for hosting endpoint. Adjust the script to match your environment." with no script block.  Step number 4 reads "Import the certificate for the Admin hosting endpoint." with a script block.  Finally, step number 5 reads "Import the certificate for the hosting endpoint." with a script block.  The way this appears to me is that step 3 could be removed and the remaining steps re-numbered.
Please see the attached image for reference.
Thank you,
Michael

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

On Step 4 I am running into issues with the -ExtensionHostFeature parameter.  I have not narrowed it down yet but I know if I run that command on a machine with Azure Stack PowerShell 1.5.0, Azure.bootstrapper 0.4.0, the newest hybrid profile, and the newest Readiness Checker I get the following error:
PS C:> Start-AzsReadinessChecker -CertificatePath c:\certificates -pfxPassword $pfxPassword -RegionName dfw -FQDN azure
stack.nttdacloud.com -IdentitySystem AAD -ExtensionHostFeature
Start-AzsReadinessChecker : Missing an argument for parameter 'ExtensionHostFeature'. Specify a parameter of type
'System.Boolean' and try again.
At line:1 char:154

... N azurestack.nttdacloud.com -IdentitySystem AAD -ExtensionHostFeature

                                                ~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidArgument: (:) [Start-AzsReadinessChecker], ParameterBindingException
FullyQualifiedErrorId : MissingArgument,Start-AzsReadinessChecker

However, running that command on a system that is using the older versions of the bootstrapper, Azure Stack PowerShell, the older profile, and one version back of the Readiness Checker it works fine.

Update to my message above.  It looks like the newest Readiness Checker (1.1811.1101.1) maybe the issue.  As soon as I installed version 1.1811.1029.1 I had no issues with the -ExtensionHostFeature parameter.

@kristopherjturner and @SqlTechMike Thanks for your feedback. I'll be able to update shortly. #in-progress

I've updated the topic. Thanks again for your feedback. #please-close


Default value

It would be nice if we could use the value="" parameter on this inputs. For instance...
<input type="file" class="filestyle" id="NewPictureName" data-buttonName="btn-primary" value="<?php if ( isset($record[PictureName]) ) { echo $record[PictureName]; } ?>">

But at the moment the default value is not visible.
Thanks in advance.

Can not, the file field has unique features in html. Of course this does not work with or without the plugin.


[Text] Add onPressIn and onPressOut

Summary
We wanted to add a special style on a Text component when it's pressed.
Instead of wrapping it in a TouchableOpacity (which prevents Texts nesting) I added onPressIn and onPressOut to listen to touch events.
Changelog
[General] [Added] - Add onPressIn and onPressOut on Text component
Test Plan

yarn lint ✅
yarn test ✅

cc @necolas @trueadm @shergin what do you think about this?

Looks ok to me.
What we're trying internally atm is a Text that doesn't include the Touchable stuff but can be combined with the relevant responders to create a separate "TouchableText" component (closer to how View is combined with Touchable to create TouchableHighlight etc)

Thanks for this @Kerumen. Do you mind rebasing & resolving these conflicts and we'll import for review?

Hi @Kerumen, in an effort to clean up our outstanding PRs, and because this one is quite old and we haven't heard back from you, we're going to close this to focus on active ones. If you would like to pick this back up please comment and I'll be happy to reopen it! Thanks

@charlesbdudley I'm sorry, I'm in vacation right now. But I'm willing to rebase it! Can you reopen it and I do it when I come back, next WEEK? Thanks a lot.

see #31288


Colab link failing with latest release of dash_jbrowse(?)

Visit https://colab.research.google.com/github/GMOD/jbrowse-jupyter/blob/main/browser.ipynb#scrollTo=fHGlGE4FpGyn
Run all ends up failing with
FileNotFoundError: [Errno 2] No such file or directory: '/usr/local/lib/python3.10/dist-packages/Flask-2.2.5.dist-info/METADATA'

screenshot

running it twice maybe worked though(?)

Actually the jbrowse-jupyter requirements.txt hard requires v1.1.0 of dash_jbrowse. maybe can try to update it

probably can close, seems like we gotta just do some build work and its not related to latest release


PELE-477 Updates to site_finder clustering, pele.conf and adaptive.conf

All tests passed.

To review:

Features/adaptive
Simulations parameters
Residue checker

Everything seems fine. Running test prior merging.


Allow user to test the APIKey on the configuration screen

The only way to currently test the APIKey is to complete the configuration and see if it fails

Added "Test API Key..." button in Release 1.3.1