Masthead | Web Components | RTL enabled has rendering issues

Detailed description

Describe in detail the issue you're having.

The web components masthead has rendering issues in RTL:

Is this a feature request (new component, new icon), a bug, or a general
issue?

Bug

Is this issue related to a specific component?

Masthead

What did you expect to happen? What happened instead? What would you like to
see changed?

Match rendering to React version of the Masthead: http://ibmdotcom-react-rtl.mybluemix.net/?path=/story/components-dotcom-shell--default

What browser are you working in?

Chrome

What version of the IBM.com Library are you using?

v1.10.0
Steps to reproduce the issue

Go to http://ibmdotcom-web-components-canary.mybluemix.net/iframe.html?id=components-dotcom-shell--default
Inspect the page, and under the <html> element, add the attribute <html dir="rtl">

Additional Information

Hi @jeffchew thank you for reporting, we need to create another Storybook build and test there. For Storybook build the instruction is similar to React (using STORYBOOK_IBMDOTCOM_WEB_COMPONENTS_USE_RTL=true environment variable).

The remaining works:

Create documentation to how to use RTL from our library users' perspective. The one for carbon-web-components can be found at: https://web-components.carbondesignsystem.com/?path=/docs/introduction-custom-styles--page#dependency-injection
(Optional): Create RTL version of lit-html CSS build

@asudoh Are you working on this issue?  Just checking as you have not assigned yourself.

@RobertaJHahn Thank you for double-checking, not at this moment, but I can definitely look into this if this item should be prioritized.

I am a little confused.  Each sprint each engineer assigns issues to themselves as their time permits.  This is done by selecting your name under the Assignee drop down in the right column.  I am asking about this issue because I see that you have linked and worked on PR (#4107).  Once you begin working on an issue, then you should "assign" the issue to yourself.  It appears that there is additional work to do.  Are you going to finish the tasks included in your comments?  Should I add this issue to our engineering meeting agenda for additional discussion?

Thank you for clarifying where your question came from @RobertaJHahn!
Here's the story; @jeffchew created a PR that partially addresses this issue, which is #4107. I added a comment in this issue stating that there are more works to do.
Please let me know if you have any follow-up questions. Thanks!


SOAP Transport Improvements

A collection of issues centered on making improvements to the SOAP transport.

FYSA: @anshulkumar-usds @Adrian-Brewster I have added the following epic to include changes and improvements for the SOAP transport type based on code reviews.

@MauriceReeves-usds @brick-green Are SOAP improvements still on our radar or close as we move to a more RESTful world?

@brick-green is it safe to say we're not doing SOAP?

We're using SOAP for PA. So we won't be able to avoid it completely. Hopefully we won't have to add any additional STLTs though.

Closing as the SOAP transport works well enough now for PA. Will reopen or start fresh should the need for improvements arise down the road


Resolved the issue about lap working without timer

add screenshot of the ui change

here it is
Screencast from 07-10-22 07:49:04 AM IST.webm


[WIP] Create Docker container for tests

WIP
Docker container for tests

@alvations I'm trying to set up a docker container to run the tests, so I want to build it using the correct dependencies.
I can see that the Travis pipeline is checking for a specific java version, in a specific location. Could you please give me more details about this? For example, why is this check needed? And why do we use this specific location? I know it's a pretty old commit, but I hope you can help :)
https://github.com/nltk/nltk/blob/d13078e0441cd46f46b41a9db70dbde0ffacd4fe/tools/travis/pre-install.sh#L8

Thank you @bsolomon1124, very appreciated suggestions!
I am currently slowed down by a tendonitis, but I hope I'll be back working on this soon.

Hi @fievelk ... I hope you're better... looking forward to your return

Inactive, so closing. Happy to reconsider in future...


Dual value of constraints specified as variable attributes.

More of a question than a feature request.
Currently calling problem.constraints does not include the "implicit" constraints defined as variables attributes.
From the docs it looks like specifying variable constraints through attributes is the recommended practice.
Here a simple reproducible example on non negative least squares (my use case is significantly more complex but NNLS already shows the different behaviour).
# Non negative least squares
import cvxpy as cp
import numpy as np

rng = np.random.default_rng(seed=0)
A = rng.standard_normal((20, 4))
b = rng.standard_normal(20)
x = cp.Variable(shape=(4, ), nonneg=True)
obj =  cp.Minimize(cp.sum_squares(b - A @ x))
problem = cp.Problem(objective=obj, constraints=[])
_ = problem.solve()
print(problem.constraints)  # empty list

I found that problem.variables()[0].domain[0].dual_value of the solved problem is None.
However, specifying constraints=[x >= 0] and then calling problem.constraints[0].dual_value returns array([0.        , 1.66159569, 0.        , 0.        ]).
Any hint on what I'm doing wrong?

Hmm that's a good point, you currently can't access dual values for variable attributes. We can add documentation saying if you want dual variables you should specify the constraint explicitly.

Yes that hypothetically could happen but could be fixed using a wrap (https://www.cvxpy.org/_modules/cvxpy/atoms/affine/wraps.html).

@SteveDiamond would you be open to adding functionality where dual variables of attributes are propagated back from the solver?

I'm not sure this feature should be added. Wouldn't we need dual values for attributes that don't generate constraints like symmetric?

That's a good point. As far as symmetric / Hermitian goes, it would always be valid to set a dual variable to the matrix of all zeros (and to do that lazily).

Thinking about it more, I'm ok with adding dual variables for the attributes nonneg, nonpos, PSD, and NSD, accessible via domain as above. Covering other attributes is not so essential, and they don't have domain so the access pattern above wouldn't even be possible.


Update setup.cfg before deploying plugin

Just a small task: Update setup.cfg before submitting this project to pypi. Otherwise, generic email-addresses and github-urls will cause issues on pypi and napari-hub:
https://github.com/juglab/napari-n2v/blob/064b8dd0efef3ac12e8ca9b725af4f74ffab5871/setup.cfg#L5
I also propose to change the intended audience and topic here:
https://github.com/juglab/napari-n2v/blob/064b8dd0efef3ac12e8ca9b725af4f74ffab5871/setup.cfg#L13
and
https://github.com/juglab/napari-n2v/blob/064b8dd0efef3ac12e8ca9b725af4f74ffab5871/setup.cfg#L15
to something like this:
https://github.com/haesleinhuepf/napari-skimage-regionprops/blob/bda457cb30d15c057669e3cd847ab5e795c4b75d/setup.py#L44-L47

Good catch! Thanks!
Fixed in https://github.com/juglab/napari-n2v/commit/fef4e213e539c6dbdca79230c6b77b803c7008ca.


Enable launch with mixed-Python/C debugger from vcxproj

Based on this template, we should add a new extension to enable launching in mixed Python/C code from native projects.
This should only be installed when selecting Python native development tools, so it ought to be its own package.

Started work on this in https://github.com/zooba/PTVS/tree/issue-2750
Still need:

better handling of environment variables
special handling of PYTHONPATH
improvements to C++ template (and new templates)


celery containers exits immefiately in superset

I am running celery using the below docker command

sudo docker run --detach -e REDIS_SERVER_IP=xxx.xxx.xxx.xx -e REDIS_PASSWORD=x -e POSTGRES_SERVER_IP=xxx.xxx.xxx.xx -e POSTGRES_PASSWORD=x --volume $PWD:/etc/superset --volume $PWD:/var/lib/superset amancevice/superset celery worker --app=superset.sql_lab:celery_app --pool=gevent -Ofair

I have also set my database conection to allow async execution

but the container starts and exists immediately

meaning my sql queries remain in pending status

Please post your docker logs from the celery worker

Please post your docker logs from the celery worker

Please post your docker logs from the celery worker

File "/usr/local/lib/python3.6/site-packages/superset/sql_lab.py", line 43, in
from superset.db_engine_specs import BaseEngineSpec
File "/usr/local/lib/python3.6/site-packages/superset/db_engine_specs/init.py", line 36, in
from superset.db_engine_specs.base import BaseEngineSpec
File "/usr/local/lib/python3.6/site-packages/superset/db_engine_specs/base.py", line 54, in
from superset.models.sql_lab import Query
File "/usr/local/lib/python3.6/site-packages/superset/models/init.py", line 17, in
from . import core, datasource_access_request, schedules, sql_lab, user_attributes
File "/usr/local/lib/python3.6/site-packages/superset/models/core.py", line 55, in
from superset.db_engine_specs.base import TimeGrain
ImportError: cannot import name 'TimeGrain'
/usr/local/lib/python3.6/site-packages/psycopg2/init.py:144: UserWarning: The psycopg2 wheel package will be renamed from release 2.8; in order to keep installing from binary please use "pip install psycopg2-binary" instead. For details see: http://initd.org/psycopg/docs/install.html#binary-install-from-pypi.
""")
Traceback (most recent call last):
File "/usr/local/bin/celery", line 8, in
sys.exit(main())
File "/usr/local/lib/python3.6/site-packages/celery/main.py", line 16, in main
_main()
File "/usr/local/lib/python3.6/site-packages/celery/bin/celery.py", line 322, in main
cmd.execute_from_commandline(argv)
File "/usr/local/lib/python3.6/site-packages/celery/bin/celery.py", line 495, in execute_from_commandline
super(CeleryCommand, self).execute_from_commandline(argv)))
File "/usr/local/lib/python3.6/site-packages/celery/bin/base.py", line 289, in execute_from_commandline
argv = self.setup_app_from_commandline(argv)
File "/usr/local/lib/python3.6/site-packages/celery/bin/base.py", line 509, in setup_app_from_commandline
self.app = self.find_app(app)
File "/usr/local/lib/python3.6/site-packages/celery/bin/base.py", line 531, in find_app
return find_app(app, symbol_by_name=self.symbol_by_name)
File "/usr/local/lib/python3.6/site-packages/celery/app/utils.py", line 373, in find_app
sym = symbol_by_name(app, imp=imp)
File "/usr/local/lib/python3.6/site-packages/celery/bin/base.py", line 534, in symbol_by_name
return imports.symbol_by_name(name, imp=imp)
File "/usr/local/lib/python3.6/site-packages/kombu/utils/imports.py", line 57, in symbol_by_name
module = imp(module_name, package=package, **kwargs)
File "/usr/local/lib/python3.6/site-packages/celery/utils/imports.py", line 111, in import_from_cwd
return imp(module, package=package)
File "/usr/local/lib/python3.6/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 994, in _gcd_import
File "", line 971, in _find_and_load
File "", line 955, in _find_and_load_unlocked
File "", line 665, in _load_unlocked
File "", line 678, in exec_module
File "", line 219, in _call_with_frames_removed
File "/usr/local/lib/python3.6/site-packages/superset/sql_lab.py", line 43, in
from superset.db_engine_specs import BaseEngineSpec
File "/usr/local/lib/python3.6/site-packages/superset/db_engine_specs/init.py", line 36, in
from superset.db_engine_specs.base import BaseEngineSpec
File "/usr/local/lib/python3.6/site-packages/superset/db_engine_specs/base.py", line 54, in
from superset.models.sql_lab import Query
File "/usr/local/lib/python3.6/site-packages/superset/models/init.py", line 17, in
from . import core, datasource_access_request, schedules, sql_lab, user_attributes
File "/usr/local/lib/python3.6/site-packages/superset/models/core.py", line 55, in
from superset.db_engine_specs.base import TimeGrain
ImportError: cannot import name 'TimeGrain'

same question: https://github.com/apache/incubator-superset/issues/9589
Use the following command instead:
celery worker --app=superset.tasks.celery_app:app -Ofair

same question: #9589
Use the following command instead:
celery worker --app=superset.tasks.celery_app:app -Ofair

thanks it has worked.


Initially, CTRL+TAB toggle switches between the instruction dialog and the zones contrary to the instructions.

Microsoft PowerToys version
0.55.1
Running as admin

[X] Yes

Area(s) with issue?
FancyZones, FancyZones Editor
Steps to reproduce
We are running PowerToys in Windows 11.  While creating a grid layout In the Fancy Zones editor, we observed the following issues:

CTRL+TAB initially moves the focus between the instruction dialog (with the cancel button) and a zone, not between a zone and a resizer as indicated in the instruction box.  As you can imagine, this was most frustrating since we couldn't seem to select resizers to delete them or move them with the arrow keys.  For a new user, the whole editor appeared to function squirrelly until we figured out the anomaly!

Subsequent use of the CTRL+TAB feature continues to toggle between the instruction box and a zone, never the zones and resizers as indicated.

Before the user can TAB as expected, they must use CTRL+TAB to move focus from the instruction dialog box to a zone, then use the TAB key cycle through both the zones and resizers in sequence.   CTRL+TAB never moves between zones and resizers as instructed.

Ironically, when moving a vertical split with the left/right arrow keys, an up/down arrow key WILL toggle the focus between the zone and the resizer.  Unfortunately, the same is NOT true for horizontal splits.  Confusing?

We think the user should be able to move a resizer button with the arrow keys when hovering over the resizer and the cursor changes to a two-sided arrow.  As efficient as this would be, it is not the case.

Likewise, when a zone is the focus, the DELTETE key should delete that zone.

After using CTRL+TAB and the focus moves back to the instruction dialog; the zone focus resets and tabbing back to zones returns to zone-1 and not the last zone of focus.  Not a big issue unless you have a lot of zones!

We almost discounted the whole Fancy Zones PowerToy as a failed attempt to mirror other snap layout programs that function extremely well.  However, when we figured out what was wrong, we grew to really appreciate this utility.  Until some of these issues are fixed, other users are very likely to discount this useful little utility.  Thank you for allowing us to comment.
✔ Expected Behavior

CTRL+TAB should toggle between zones and resizer buttons.
LEFT/RIGHT arrow keys should toggle focus between the zone and resizer button when moving a horizontal split, just like the UP/DOWN toggles the focus when moving horizontal splits with the arrow keys.

❌ Actual Behavior

CTRL+TAB moves focus between zone and instruction dialog instead of zone and resizer button.
LEFT/RIGHT arrow keys appear to do nothing when moving a horizontal split, unlike a vertical split.

Other Software
Nothing

Can you confirm this issue is still relevant in v0.73? /needinfo


site: add external link - fixes #552

Fixes #552
Preview: https://stephenplusplus.github.io/gcloud-node/#/docs/master

That looks awesome. Nice!

Damn, that's neat. But it does look pretty cluttered. Can't we just set target="_blank" on all external links?

I think this is just to denote that they are external links. Any ideas on how to de-clutter?

Wait what exactly is cluttered? Are you saying the little external link icons are causing too much clutter? I'm seeing on average 1-2 per method, the occasional 3... I think it's useful to know when I'm leaving the docs...

I think I just needed a minute to warm up to it. I was probably looking at a particularly cluttered area. Sorry for the false alarm.

Cool - LGTM.


pass pip_args and verbose flag when determining package name

This passes pip arguments to the command that determines the package name. It also passes the verbose flag to the venv creation which suppresses the animation output when running in verbose mode. These were regressions in the 0.15 release.
cc @tsiq-rob, @tsiq-alberto
fixes #320

Since this is a serious bug, I also updated the pipx version to <IP_ADDRESS> and will publish a new version to PyPI once this is reviewed.

Thank you for such a swift follow up :)

Until we set up a test server with deterministic package content (which will have the nice side effect of speeding up all of pipx's tests), I'm not sure that adds any value to our unit tests. If you disagree, could you go into more detail on how you are proposing the test work?
_package_name_from_spec() now requires the arguments, so if they are not passed lint tests will fail. I guess I could write unit tests for that function?

Mostly I just somebody to try out the code (aka integration test) to make sure that it will fix the bug.
Does the OP of the bug (@tsiq-rob) have an example we could use to verify that the new code works?
It doesn't have to be a unit test (although there existed another index it would be nice.)

Oh I already did that
> pipx install git+https://github.com/psf/black  --verbose --force --pip-args='--extra-index-url=https://test.pypi.org/simple/'
pipx > (run_subprocess:107): running /home/csmith/git/pipx/venv/bin/python3.7 -m venv --without-pip /tmp/tmpd759jwj2
pipx > (run_subprocess:107): running /tmp/tmpd759jwj2/bin/python --version
pipx > (run_subprocess:107): running /tmp/tmpd759jwj2/bin/python -m pip list --format=json
pipx > (run_subprocess:107): running /tmp/tmpd759jwj2/bin/python -m pip install --extra-index-url=https://test.pypi.org/simple/ --no-dependencies git+https://github.com/psf/black
Looking in indexes: https://pypi.org/simple, https://test.pypi.org/simple/
Collecting git+https://github.com/psf/black
  Cloning https://github.com/psf/black to /tmp/pip-req-build-6o1xlp2e
  Running command git clone -q https://github.com/psf/black /tmp/pip-req-build-6o1xlp2e
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
    Preparing wheel metadata ... done
Building wheels for collected packages: black
  Building wheel for black (PEP 517) ... done
  Created wheel for black: filename=black-19.10b1.dev16+g1ab87a3-cp37-none-any.whl size=101532 sha256=4a2b27e6e44236e7b444ecb9761792f1e5e65edcbdf3dc07524ccbe2f6de7be8
  Stored in directory: /tmp/pip-ephem-wheel-cache-yvn3dbm9/wheels/80/17/07/88c1dba6753a7314e671700d8ff6d15770a8f1f61a925b181f
Successfully built black
Installing collected packages: black
Successfully installed black-19.10b1.dev16+g1ab87a3
pipx > (run_subprocess:107): running /tmp/tmpd759jwj2/bin/python -m pip list --format=json
pipx > (install_package_no_deps:232): Determined package name: 'black'
pipx > (_package_name_from_spec:104): Package name determined in 9.7s

OK, that's all I was thinking.
Although, unfortunately this example also works with our current code (just tried it) because black is both on regular pypi as well as the --extra-index-url test.pypi.org/simple.
But I guess at least everything seems to still be working.  I'm glad that --verbose works properly now, I noticed something weird in the formatting before but I didn't register it was my code's fault...
LGTM

If you inspect the commands it ran, you can see it passed the new arguments when determining the package name.
I just thought of a way to test this: Pass an invalid argument to pip, then confirm Cannot determine package name from spec is in the output. Then remove the invalid argument and confirm the installation succeeds.

For future reference (not gating this PR's merge) this uses a Github Pages URL to mock a simple pypi index.  If all we need is a separate index online.
https://www.freecodecamp.org/news/how-to-use-github-as-a-pypi-server-1c3b0d07db2/


Cirrus CI / musl Linux fails

See https://github.com/libprima/PRIMA.jl/runs/17824304342

We have removed these tests for now (see d85e73a0132a8cda00bb572ec1c497c895cd443b).


Include ng-sortable.js in bower distribution

bump

thx

Can you please cut a new tag w/semver to 1.2.1 to allow bower users to get ng-sortable.js?
tag 1.2.0 is currently latest, does not include this change of course

A new version would be appreciated - there's some other angular related improvements that aren't accessible at the moment.

https://github.com/RubaXa/Sortable/pull/439


[HOTFIX][TOOLS] dcos-launch is supposed to be duplicated there.

The first parameter to /venvs.sh is the name of the virtual environment, and the rest are the command itself to be run in that virtual environment.

Merging without checks because integration test checks are broken without this.


Alarm Clock in windows using Tkinter GUI

Script Title - Alarm
Brief - Alarm Clock based on Tkinter GUI which will be triggered at a particular time using windows task schedular after setting it for a particular time.
Instructions

Create a new folder for your script and file/folder name should be appropriate.
Create a README.md in your folder for program Instructions
add requirements.txt if needed

Programming Language

[x] Python

Happy Coding

Please assign me this issue.


Ability to pin import to top of the list

Thank you so much for keeping on the great work of this extension.
We've run into a slightly interesting issue where we need the ability to pin certain imports to the top of our page.
For example: import "reflect-metadata";, #!/usr/bin/env node; or import "source-map-support/register"
#!/usr/bin/env node;
import "reflect-metadata";
import express, { Application, NextFunction, Request, Response } from "express";

However we always have to manually add these back to the top, it would be great if we had a setting that allowed us to "pin" certain imports to the top of the list.
Thanks in advance,
N

@neilsoult I started sketching out something for me to use locally: https://github.com/Nimothy/typescript-imports-sort
If you agree with the approach, I can open up a PR.

@Nimothy sure, I'd be glad to accept a PR for this change.


Paste as "Plain Text"

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is.
Currently when I paste text into Tusk it always retains the original formatting. This is very annoying as I always I want the new text to have the format of the Tusk text. My current workaround is first I paste the text into NotePad and then copy from NotePad into Tusk.
Describe the solution you'd like
A clear and concise description of what you want to happen.
I would like Tusk to have an option in the Right Click menu to "Paste Text" and remove all copied formatting. A lot of other editors have this feature so it should be clear to understand.
So the options in the right click menu would be something like:

Cut
Copy
Paste
Paste Text

Additional context
Add any other context or screenshots about the feature request here.

I always use Ctrl+Shift+v to paste text without formatting.

@Amandaville Thanks!


Make catalogWatchTaskScheduler/configWatchTaskScheduler more sophisticated

This issue related to #146. A fix provided there is not suitable.
For example:

The consul is disabled for a local environment and my own configuration for TaskScheduler with 2 threads works well.
The consul is enabled for PROD configuration and there is a single scheduling thread as a result.

To fix this a spring-vault approach with TaskSchedulerWrapper can be used
https://github.com/spring-projects/spring-vault/blob/main/spring-vault-core/src/main/java/org/springframework/vault/config/AbstractVaultConfiguration.java#L324
The advise in the documentation is not very good:

The watch uses a Spring TaskScheduler to schedule the call to consul. By default it is a ThreadPoolTaskScheduler with a poolSize of 1. To change the TaskScheduler, create a bean of type TaskScheduler named with the ConsulConfigAutoConfiguration.CONFIG_WATCH_TASK_SCHEDULER_NAME constant

It doesn't work. It needs to enable bean overriding.
Why I have to add to my application configuration a bean with a wired name, just to have multiple threads in the TaskScheduler?

The simplest workaround is not to use a bean with the same name, but @Primary on the application configuration.
    @Primary
    @Bean(name = "app-scheduler-pool")
    public TaskScheduler appSchedulerPool() {
        ThreadPoolTaskScheduler threadPoolTaskScheduler = new ThreadPoolTaskScheduler();
        threadPoolTaskScheduler.setPoolSize(THREADS_COUNT);
        return threadPoolTaskScheduler;
    }

this was created before spring boot changed the default of bean overriding. The docs could be updated to reflect primary as an option. I don't understand how a wrapper will help.

@spencergibb Maybe a wrapper will not help. I just wanted to say that spring-vault has a separate TaskScheduler that doesn't influence my application scheduling logic at all. spring-cloud-consul does it.

After chatting with the team in the spring cloud kubernetes issue, we're doing a wrapper as a bean, we should do the same here.


Leveldb sorter lost data

What did you do?
Setup two tidb clusters and use ticdc replicating data between them.
Note, this bug only affects master branch.
What did you expect to see?
No data lost
What did you see instead?
Data lost.
Versions of the cluster
Upstream TiDB cluster version (execute SELECT tidb_version(); in a MySQL client):
TiCDC version (execute cdc version):
master 2022/03/09

As it only affect master branch, lower the severity.


No internet connection icon

For some upcoming polish work we are going to need a "no internet connection" icon added to Dialtone. This has been approved by @hynes-dialpad
I opened a branch in figma with the icon exported from google's material icon set
Exported icon:

Tracked in DT-616.


[FLINK-10726] [table] Include flink-table-common in flink-table jar

What is the purpose of the change
flink-table-common is not included in the flink-table jar which leads to failure in the streaming SQL e2e test.
Brief change log

Inclusion added

Verifying this change
Run ./run-single-test.sh ./test-scripts/test_streaming_sql.sh.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): yes
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

Looks good, if it passes the tests!

The build passed on my Travis. I will merge this shortly...


Documentation: MariaDBConfigStore

Suggestion
Document the MariaDBConfigStore integration with all the necessary details that explain what the store is and how to use it.
Motivation

Introduces users to the ConfigStore.
Improves usability of the store by providing usage examples.
Improves the completeness of the integration package.

Context

Integration docs: https://mariadb.org/documentation
Integration SDK docs: https://typeorm.io/data-source-options#mysql--mariadb-data-source-options
Example PR of a store doc contribution: #575

All yours @mrswastik-robot


Remove App Engine backend from GCLB

This just removes App Engine from the load balancer. I'll manually apply this on prod after it's merged.
I still need to understand what happens to Datastore if we delete App Engine entirely.

Do we also need to delete
resource "google_project_service" "app_engine" {

in gcp_apis.tf? Or does this need to get removed first

That's what enabled App Engine on the Google Cloud Project, and since I'm not removing App Engine yet (just the Load Balancer config) it should probably stay
(Not that removing it will do anything anyway, since it won't disable the api when destroyed)


Update dependencies associated with fast-xml-parser vuln

What
Updating oclif dependency version
Why
To include patch for the fast-xml-parser vulnerability

References
Vulnerability https://github.com/npm/provenance/security/dependabot/8
Related to https://github.com/github/npm/issues/12635

A new [PR] has created for addressing this issue


Added credentials section to on-gke setup instructions.

Thanks for opening a Pull Request!
Here's a handy checklist to ensure your PR goes smoothly.

[x] I signed Google's Contributor License Agreement

Please review Gregg's comment before merge.


chore: introduce prow-github-actions

This is an introduction to https://github.com/jpmcb/prow-github-actions, which will allow us to use the most basic slash commands. We'll see how it works and eventually configure more features later.

/lgtm

/approve

/lgtm

We will test on the next PR.


The .NET CLI tools cannot be located

From @Tobyee on July 27, 2016 5:56

VSCode Version: 1.3.1
OS Version: Win7-x64
dotnet Version: 1.0.0-beta-001598

Steps to Reproduce:

cmd: mkdir demo & cd demo
cmd: dotnet new
cmd: dotnet restore
cmd: code .

Error:

I'am sure that  .NET Core SDK for Windows was installed and dotnet command can be execute in cmd.
See alse:
https://github.com/dotnet/cli/issues/3858
Copied from original issue: Microsoft/vscode#9831

@Tobyee do you have the latest version of the C# extension (check in the extensions panel)? Assuming so, what do you get from:

Start->Run, cmd.exe
where dotnet.exe
dotnet --info

@gregg-miskelly
Thanks for you tips. I found the reason.
It's because I have a old version cli tools installed that influence on the new one.
The old version install is:

The new version install is:

I uninstall the old one then it fixed.

@Tobyee glad you solved your issue. I will close this issue then.

What do Linux users do?

@EliasCK if you are running into a problem related to this and would like some help, or have a concrete suggestion, please open a new issue so we can help understand what is going on on your machine.


clever install clever-odm fails

$ node --version
v0.10.38
$ cd test001038/backend/modules/
$ clever install clever-odm
Attempting to install clever-odm...
✔      Found CleverStack Backend (node-seed) in /Users/xxx/test001038/backend...
├── Searching for modules...
├──   Searching NPM...
├──   Searching Bower...
├── ⚠ Installing clever-odm...
├── Installing NPM modules for clever-odm...
✔    Successfully installed clever-odm...
├── Installing module peerDependencies...
├── ⚠ Running grunt tasks for module clever-odm...
ReferenceError: runTask is not defined

I believe i have fixed the issues in Release 1.2.7 - can you please run npm install -g cleverstack-cli and try again?  (if your running that on linux/mac then you may need to use sudo)

Node is installed in ~/node, so no sudo is required.
Fix confirmed! Thanks for your effort.


Moving data and query in within subject code block for latency issue in QA

This PR is a hotfix for the latency issues observed in current QA (v5.0.9) due to the monitoredLogCounts query and its required sites and targets data used outside within subjects block, causing unnecessary calls to the query for non-within subjects experiments.

@danoswaltCL @bcb37 We don't see any possibility for optimization of the below query as it uses only single left join. The hotfix in this PR should suffice for the current latency issue. In case we still observe this, we can look for some more possibilities.

@danoswaltCL I was going through within subjects experiment testing for exclusions logic and noticed that the within subject unitOfAssignmennt was removed from the UI in your last commit here. When do we plan to get this back in UI?


SQL Server 2012 client names appear to be wrong

They're all the 2008 R2 names.  These are probably all wrong.

This has been fixed


Add Iterators for slots

closes https://github.com/ViewComponent/view_component/issues/1620
What are you trying to accomplish?
Add a parameter similar to the iteration context provided when rendering collections, ViewComponent could provide contextual information, such as #size, #index, #first?, and #last?, when rendering slots that have been defined with renders_many,
additionally provide a collection counter to slots defined with renders_many.

What approach did you choose and why?
I chose to use ActionView::PartialIteration to get the iteration count and the iteration context because that is what was used in the collection iteration counter and iteration context helpers
Anything you want to highlight for special attention from reviewers?

@Spone I fixed that spelling mistake a re-review would be much appreciated

@joelhawksley I added some docs. I leveraged a lot of the content from the collection docs. A  second set of eyes would be much appreciated

@joelhawksley a re-review of this pr at some point would be much appreciated

@reeganviljoen sorry for the delay, I lost track of this one!
@corasaurus-hex would you be up for re-reviewing first?

@reeganviljoen I've pushed a few minor refactors while I review this ❤

Thank you @joelhawksley , I see the tests are failing I will see if I can fix that

@corasaurus-hex I'd still be interested in your review ❤

@joelhawksley @corasaurus-hex @Spone @BlakeWilliams after much thought I have decided to abonden this pr because of the very valid points braught forth


Subclass Tokens

Token types should have subclasses with token-specific metadata (for instance a gene pair might have a gene1 and gene2 field with the corresponding tokens for those genes as fields)

Tokens are now subclass-able while still working within the openapi schema. Next step is deciding which tokens need additional metadata attached.


DB Trigger:  On client_agreement create and delete ---> create and delete plan_confirmation record

When no longer relevant client agreements are deleted, the plan confirmation record needs to get cleaned up.

Only fire if there is a plan for that agreement_id
Make sure to get the most recent plan if there are dupes
client_agreement on create  makes appropriate plan_confirmation record for given clients.
client_agreement on delete deletes appropriate plan_confirmation record for given clients.

@micheal-w-wells Would it be possible to just add a plan_confirmed column to client_agreement instead? And remove the plan_confirmation table? Since we won't have more than 1 plan per agreement now.

@calebissharp that sounds pretty good, as long as we have a dedicated column for tracking the date signed rather than going off changed date on the table (there's a job that touches those regularly).

Hmm that's a good point. Doable though. Any other gotchas you can think of? As far as I can think, it's just a one-to-one relationship.

Not necessarily a gotcha but very related is the need to notify staff that there are more people to sign the RUP.

Notifications will be another beast altogether, regardless of how the actual submissions are tracked.

Resizing to reflect reality.  Also despite chatter here Caleb and I decided it would make sense with the time we have to just go the db trigger route.

Steps to test:

Create or find a RUP with a client or two on it.
Change who is linked on it
You should get the right number of AH awaiting signatures, and they should be able to complete them without any wonkyness with awaiting confirmations/signatures. on the web page and pdf.

@LisaMoore1 @ZoeSimon can probably hold off testing this until Caleb's client linking changes come in, then we can test it all in one shot.

@micheal-w-wells Can we move this to in progress until it is ready to test so that only things that need action “today” are in Review/QA?

@LisaMoore1 I have very high confidence this works as I've tested it lots myself and I have been using the SQL to fix the ones that pop up in metabase.  Because this is a database only change, i realized I could apply it to prod ahead of time.  I've done so, as well as fix the remaining ones that we got alerted about on the weekend.  As this is all done I'm going to close it.


Omit Dockerfile table on no vulns

This omits the Dockerfile table when no Dockerfile vulns are found.

Merging to main for manual verification.


Added default task which runs all tests

This makes bundle exec rake work, more consistency with other 💎  projects.

Best PR cc @KrauseFx

Thanks 👍


Cannot access baidu for the evaluation code and model

baidu.com is not available outside of China.
Could you share it to Google Drive or some other storage, please?

You could use this link for example: https://drive.google.com/drive/folders/1WWtput3Utb5hvxs7nBycqTiXbwl_OtB2?usp=sharing
Or this one: https://www.icloud.com/iclouddrive/0ff9FMBqAmk2C9fuk-T1XobHA#ML_Shared


Can't build on ArchLinux

Hi
I'm trying to build this package because it's needed for mod_oauth2 but I'm getting some file missing errors when going with make on Arch Linux. The first one is:
In file included from src/server/apache.c:22:
include/oauth2/apache.h:34:10: fatal error: httpd.h: Nie ma takiego pliku ani katalogu
   34 | #include <httpd.h>
      |          ^~~~~~~~~
compilation terminated.

And I'm guessing that same will go for: http_config.h, http_log.h and mod_auth.h as I can't find them anywhere in the source folder as well.
My steps (after extracting tar) were: ./autogen.sh, ./configure, make.
So what am I doing wrong here?

you need to install the Apache development files; on Ubuntu etc. that would be the apache2-dev package and possible you'll need to provide a path to the include files using the CPPFLAGS env var

be aware that mod_auth_openidc 3.x that depends on this library is not production ready yet; you may want to stick with 2.x for the time being

I didn't even know that there's a 3.x branch. I was trying to update to 2.4 but stumble upon missing mod_oauth2 error and installing it gave me missing liboauth2 that also had problem, as seen above. ;)
After installing everything I still don't get headers for mod_auth_openidc defined with OIDCClaimPrefix "OIDC-". All I'm getting are REDIRECT_OIDC...which probably means I didn't install everything as I should. :(

no, it's an apache thing https://groups.google.com/forum/#!topic/mod_auth_openidc/QEyxhna39zw

Argh...And I was fighting with the proper installation when the problem is completely elsewhere. Thanks for pointing that out.


[vcpkg baseline][pangolin] Add features, fix dependency pybind11

Since most of the cmake options related to dependencies in pangolin are ON by default, and will be set to ON or OFF according to the result of searching for dependencies, and these options are not set in portfile.cmake, if these dependencies are installed first , it may cause build issues.
This PR will:

Add features according to cmake options
Change the default dependency to features and set them as the default features.
Force other options to OFF.
Fix pybind11 build issue:

/home/linuxbrew/.linuxbrew/bin/ld: /home/byxrolland/vcpkg/installed/x64-linux/debug/lib/libavcodec.a(vc1dsp_mmx.o): relocation R_X86_64_PC32 against symbol `ff_pw_9' can not be used when making a shared object; recompile with -fPIC

Fix libuvc build type and usage issue.
Fix libusb usage issue on non-Windows (libusb requires libudev on non-Windows).

Fixes #19960.
Already tested all features / core on x64-windows, x64-windows-static and x64-linux.

Okay, all features are tested again.

Thanks @JackBoosY :D


[FEATURE] add anchor point (left/center/right) + sign hanging mode (recto/verso)

demo data has been improved

we have some display issues that are arising (a bit more than before) due to the display of 2 signs on the same row/level (left+right)
example:

if we explode it a bit (offset):

what we could do:

detect when there is a single type of anchor (left/center/right) for the azimut and in this case center the signs (in the above example, the 2 columns starting with Neuchâtel and Vallorbe would get centered ( at the cost of losing the visual information of the anchor)
try to calculate when there is a covering of sign and add an offset for the aizmut


Safety question: converting back to Vec

Is it safe to convert an OwnedBuf into a Vec<u8> if it was created from a Vec<MaybeUninit<u8>> or vice versa?

I had some thoughts on generalizing the drop to buffer recovery: https://github.com/tokio-rs/tokio-uring/issues/216#issuecomment-1412168484 I'd appreciate your thoughts.


merge package template from arlac77/npm-package-template

Updated standard to latest version

Coverage remained the same at 87.302% when pulling a16ef797a9a59f42d3d79e7a01f326f69a42df1d on template-sync-1 into 9f452458c2aff0854f118f9fbec015a813e0acf0 on master.

Coverage remained the same at 87.302% when pulling a16ef797a9a59f42d3d79e7a01f326f69a42df1d on template-sync-1 into 9f452458c2aff0854f118f9fbec015a813e0acf0 on master.

Codecov Report

Merging #99 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #99   +/-   ##
======================================
  Coverage    87.3%   87.3%
======================================
  Files          16      16
  Lines         126     126
======================================
  Hits          110     110
  Misses         16      16

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 9f45245...a16ef79. Read the comment docs.


Cannot deploy simple flask app - werkzeug problem

I am deploying a simple app. All 3 files are in the project root.
Here is my app.py
from flask import Flask

app = Flask(__name__)

@app.route('/', methods=['GET'])
def lambda_handler(event=None, context=None):
    return 'hello world!'

if __name__ == '__main__':
    app.run(debug=True)

Here is my zappa_settings.json
{
    "dev": {
        "app_function": "app.lambda_handler",
        "aws_region": "us-west-2",
        "profile_name": "default",
        "s3_bucket": "my-bucket",
        "memory_size": 128,
        "keep_warm": false,
    }
}

Here is my requirements.txt
argcomplete==1.8.2
base58==0.2.4
boto3==1.4.4
botocore==1.5.40
certifi==2017.4.17
chardet==3.0.4
click==6.7
docutils==0.13.1
durationpy==0.4
Flask==0.12.2
future==0.16.0
futures==3.1.1
hjson==2.0.7
idna==2.5
itsdangerous==0.24
Jinja2==2.9.6
jmespath==0.9.3
kappa==0.6.0
lambda-packages==0.15.1
MarkupSafe==1.0
placebo==0.8.1
python-dateutil==2.6.0
python-slugify==1.2.4
PyYAML==3.12
requests==2.18.1
s3transfer==0.1.10
six==1.10.0
toml==0.9.2
tqdm==4.14.0
troposphere==1.9.4
Unidecode==0.4.20
urllib3==1.21.1
Werkzeug==0.12
wsgi-request-logger==0.4.6
zappa==0.42.2

I receive this error when opening the site after zappa deploy dev. CloudWatch logs look normal (no traceback).
"{'message': 'An uncaught exception happened while servicing this request. You can investigate this with the `zappa tail` command.', 'traceback': ['Traceback (most recent call last):\\n', '  File \"/var/task/handler.py\", line 433, in handler\\n    response = Response.from_app(self.wsgi_app, environ)\\n', '  File \"/var/task/werkzeug/wrappers.py\", line 903, in from_app\\n    return cls(*_run_wsgi_app(app, environ, buffered))\\n', '  File \"/var/task/werkzeug/test.py\", line 903, in run_wsgi_app\\n    buffer.append(next(app_iter))\\n', '  File \"/var/task/werkzeug/wsgi.py\", line 704, in __next__\\n    return self._next()\\n', 'StopIteration\\n']}"

Help.

Changing requirements.txt solved it.
Flask>=0.12
zappa>=0.17.6


Add link to whitelisted java methods that can be used in SpEL

The Spinnaker SpEL documentation around java is missing an explanation that not any java method can be run using SpEL, only whitelisted methods.
This adds this explanation + link to the code with the whitelisted methods.
@tomaslin

might make sense to not link to master but rather to a known hash

@tomaslin right. done.

thank you very much for your contribution


WIP: Storybook 4 integration

Release Type: Dev improvement

Fixes https://x-team-internal.atlassian.net/browse/XP-2439
Description
Please check this comments for all the details and background: https://x-team-internal.atlassian.net/browse/XP-2439?focusedCommentId=11977&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-11977
Having troubles to make Stylegudist work with latest React. It worked with prev version we had, but once I updated to React 16 to make Storybook 4 happy, it started failing. Tried upgrading Styleguidist -related libs versions, but still unlucky 😠 Spent several hours on making it work as expected.
So probably we’ll have to complete migration to Storybook first and then merge it. Not sure what is the best approach tho. My initial idea was to bring Storybook into the flow asap so we can start using that to write new component and convert existing ones gradually.
That’s another error I get when run npm start:

OK, keeping it in progress so we can contribute to it by converting existing components into stories. Let's stick to the rule: 1 story per component - 1 commit
TODO

[ ] Transfer existing components' specs over to stories
[ ] Export Storybook as a static web app that we can deploy to GitHub pages or any static hosting service - https://storybook.js.org/basics/exporting-storybook/
[ ] Implement structural snapshot testing (similar to current setup) - https://storybook.js.org/testing/structural-testing/
[ ] Anything else?

Checklist
Before submitting a pull request, please make sure the following is done:

[x] set yourself as an assignee
[x] set appropriate labels for a PR (In Review or In Progress depending on its status)
[x] move respective Jira issue to the IN REVIEW column
[x] make sure your code lints (npm run lint)
[x] Flow typechecks passed (npm run flow)
[ ] my change requires a change to the component's documentation (.snap)
[ ] component's documentation (.snap) is changed or added accordingly
[ ] if any snapshots have been changed, verify that component still works and looks as expected and update the changed snapshot
[ ] manually tested the component by running it in the browser and checked nothing is broken and operates as expected!

Steps to Test or Reproduce

install new deps: npm i
run npm run storybook
gradually convert existing components specs into stories: one component - 1 commit

Some things to follow when writing Stories

remember to run npm t after the story is written (I’ll automate that soon)
use all-lowercase for story names just to make it consistent
keep the same stories grouping as in current Styleguidist case (you can see how I did that in existing stories)
instead of console.log we now use an action() addon that is more handy and flexible
you can always check existing stories for an example

Screenshots

To keep things in sync, rebased this branch with latest sprint-12

We also updated SVgIcon in last PR. So will keep it here not to forget to reflect in Storybook.

I believe I updated that when I did the rebase, but please confirm when you have the time.


Use Case insensitive header comparison for newer OkHttp compatibility

This is needed because from OkHttp 3.3.0 onwards, Headers.toMultimap() lowers the case for header names. This will be fixed in 3.4.0 as Headers.toMultimap will return a TreeMap which does case insensitive get. However, Feign again wraps the response headers in https://github.com/Netflix/feign/blob/master/core/src/main/java/feign/Response.java#L50-L51.
In general, http-remoting shouldn't be making assumptions about the header's case sensitivity.

few nitpicks, otherwise this is fine. thanks Aakash!

lgtm; thanks for iteration

Thanks Aakash!


Fixed issue #7170

Issue Reference
#7170

@akathuria289
This is working fine, please find the screenshot Here

@akathuria289
The Inventory of bundle products is not updating after creating the refund.

@Sanjaybhattwebkul Please check

@akathuria289
Still not working. the quantity of the product is not increasing after creating refund

@Sanjaybhattwebkul it`s working fine https://webkul.chatwhizz.com/share/view-recording/63e0969daac9534fac237f4a

@akathuria289
Not working. Please create multiple inventories and again reproduce the issue.

@Sanjaybhattwebkul Now check

@akathuria289
Not working, in the case of bundle products the product inventory is not updating after creating a refund.

@akathuria289
Not working, getting multiple input boxes for quantity while creating shipping, please find the screenshot Here

@akathuria289
The product quantity is not updating after creating a refund of bundle products.

@Sanjaybhattwebkul It`s working fine please check again


Merge unable to parse column names

Hi, we have a list of columns that we need to update which follow a naming convention like:
Attr_227e7d6a-876c-4ace-92f3-ffc3945a8ed6
where the latter is a UUID maintained by our systems.
When I try to update my table via the merge command, I get the following error:
Possibly unquoted identifier a15-71da97ac0646 detected. Please consider quoting it with back-quotes as `a15-71da97ac0646`(line 1, pos 35)

== SQL ==
source.Attr_be685ab2-75b8-440e-8a15-71da97ac0646`

Which seems to me as if something is unable to parse the column name correctly.
This is how I defined my statement  (shortened, we actually have >150 columns that follow this schema)
changedUpdate = {
    "Expired_Date": "source.Expired_Date",
    "Content_Native": "source.Content_Native",
    "Attr_02272e10-a11e-4ba5-a596-9839102b972a": "source.Attr_02272e10-a11e-4ba5-a596-9839102b972a",
    "Attr_04952d40-d747-4df2-ba09-f342fc1a94f2": "source.Attr_04952d40-d747-4df2-ba09-f342fc1a94f2",
    "Attr_5c7e61de-92e5-4ff1-8520-04ed73df95e0": "source.Attr_5c7e61de-92e5-4ff1-8520-04ed73df95e0",
    "Attr_5cee25e9-4fb7-4643-be52-19a05ef32e31": "source.Attr_5cee25e9-4fb7-4643-be52-19a05ef32e31",
    "Attr_be685ab2-75b8-440e-8a15-71da97ac0646": "source.Attr_be685ab2-75b8-440e-8a15-71da97ac0646",
    "Attr_bee8dce0-6fe2-4c64-b095-569f8e8f69d4": "source.Attr_bee8dce0-6fe2-4c64-b095-569f8e8f69d4",
    "Attr_c08a022b-c21b-48d0-96a3-a57496ce4c7b": "source.Attr_c08a022b-c21b-48d0-96a3-a57496ce4c7b",
    "Attr_c4100c8e-c9a5-4b58-8563-3c72110fee08": "source.Attr_c4100c8e-c9a5-4b58-8563-3c72110fee08",
    "Attr_c4fa0587-5519-4f27-a01d-b40e4f5c5c61": "source.Attr_c4fa0587-5519-4f27-a01d-b40e4f5c5c61",
    "Attr_c8f9a2e3-ed1e-4132-a591-aa2fb72dd49d": "source.Attr_c8f9a2e3-ed1e-4132-a591-aa2fb72dd49d",
    "Attr_cd048ad3-7966-4f0e-bcfe-cd74292fd059": "source.Attr_cd048ad3-7966-4f0e-bcfe-cd74292fd059",
    "Attr_d25ecfaf-e743-4e62-9dde-1cd9018f294d": "source.Attr_d25ecfaf-e743-4e62-9dde-1cd9018f294d",
    "Attr_d2d2f131-20aa-4f34-9a8b-1fcd904bde5e": "source.Attr_d2d2f131-20aa-4f34-9a8b-1fcd904bde5e",
    "Attr_d54bf424-e4c1-45c1-a7a3-aafbd5c6e2fb": "source.Attr_d54bf424-e4c1-45c1-a7a3-aafbd5c6e2fb",
    "Attr_d599ec96-55c2-4bb9-8319-6ea21cc6319e": "source.Attr_d599ec96-55c2-4bb9-8319-6ea21cc6319e",
    "Attr_d669fa3d-4bc0-4a0c-9788-ff16c0594640": "source.Attr_d669fa3d-4bc0-4a0c-9788-ff16c0594640",
    "Row_Hash": "source.Row_Hash"
 }

(
target.alias("target").merge(df.alias("source"), "target.Document_ID = source.Document_ID")
.whenMatchedUpdate("target.Row_Hash != source.Row_Hash and target.Deleted_Flag = 0", set = changedUpdate)
.whenNotMatchedInsertAll()
.execute()
)

When I delete the line in question I basically run into the next error:
AnalysisException: cannot resolve `source.Attr_d669fa3d` in UPDATE clause given columns target.`Document_ID`, .. etc.

which, I guess shows that the column name somehow gets truncated at the first dash?
Note: I don't want to use updateAll, as I have a set of columns which should stay the same/have a hardcoded value depending on the match.

This is actually correct parser behavior; the - characters are being interpreted as minus operators. As the error message notes, you can escape the column name with backquotes in order to pass the entire thing as a column name.

Thanks, my bad there :)


NEW: integrate scheduler into docker

Just run "schedule:work" as a separate Artisan command once your container is started.


adds channel utils for mux and demux

Proposed Changes

add util to join N channels
add util to Clone N channels
closes https://github.com/projectdiscovery/utils/issues/193

@Mzack9999 , yeah do you think we should implement both of them or one of proposed .  if we have a choice i think we should probably use error channel approach since utils is library the error can be properly propogated to downstream repos


Fix favicons not showing Kubeflow logo

Currently, some services are picking up the default favicon from Docsy (our theme) when linking to the Kubeflow website.
For example, try linking to Kubeflow website on slack, you will see a strange default icon:

This PR fixes all issues with favicons and explicitly requests that people reindex them by adding a parameter ?v=2 to the URL paths.
You can confirm this PR works by looking at the preview and seeing if the favicon appears in your browser.

I have confirmed that this PR fixes the issue on slack:

@james-jwu @jbottum @andreyvelich @terrytangyuan can one of you please review or approve this website PR?
It looks bad if our favicon does not work properly, lol.

@terrytangyuan the new favicons were generated using the industry standard tool from the SVG of the logo:

https://realfavicongenerator.net/

Thanks
/lgtm
/approve

This issue was also affecting Google, lol:


Add C++ module for xmake project

Is your feature request related to a problem? Please describe.
xmake build build my project fine. But using xmake project to generate a compile_commands.json won't work properly because C++ modules are not present in the json.
Describe the solution you'd like
Add modules in the built source among others (listed in the function _sourcebatch_is_built of the file /plugins/project/clang/compile_commands.lua for example)
Describe alternatives you've considered
For now, I'm using xmake b -vD to get xmake module build command that I copy/paste into the compile_command.json
Additional context
No response

https://github.com/xmake-io/xmake/pull/4199


type assertion are a TypeScript only feature. Convert your file to a TypeScript file or remove the syntax.🐛

VS Code version
1.87.2
Extension version
2.2.2
Biome version
1.6.1
Operating system

[ ] Windows
[X] macOS
[ ] Linux

Description
I have a react application, biome is stating that the javascript files have the following issue:
"type assertion are a TypeScript only feature. Convert your file to a TypeScript file or remove the syntax."
It is highlighting the returned elements of the function which are normal JSX.
Biome is even highlight this with the error above:
return

;
The Biome extension in VS code is the issue, we have Biome running with our pre-commit husky hook and everything is normal.
Steps to reproduce
I cannot share the code.
Expected behavior
Biome does not mention typescript errors as this is just a javascript project and file.
Does this issue occur when using the CLI directly?
Not sure / Not applicable
Logs
No response

Closing as duplicate of https://github.com/biomejs/biome/issues/2085


 I've implemented it with what I knew.
This PR:

adds a key binder so that when one type a letter while having a RadioList opened, it automatically switches to the next entry starting with that letter. This behavior is the same than in explorer for instance.

Demo code:
from __future__ import unicode_literals
from prompt_toolkit.formatted_text import HTML
from prompt_toolkit.shortcuts.dialogs import radiolist_dialog
result = radiolist_dialog(
    values=[
        ('red', 'Red'),
        ('green', 'Green'),
        ('blue', 'Blue'),
        ('blue2', 'Blue2'),
        ('blue3', 'Blue'),
        ('bluec', 'blue. hi, im case sensitive'),
        ('orange', 'Orange'),
        ('orange', 'Orange2'),
        ('blue4', 'Blue again'),
    ],
    title='Radiolist dialog example',
    text='Please select a color:'
)

Then type, for instance, "B" (note the cap letter), several times
Surely, you all know a better way to do this, using what already exist. If you have any good advices on how to do this in a better way, please tell me

Codecov Report

Merging #627 into master will decrease coverage by 0.05%.
The diff coverage is 8.33%.

@@            Coverage Diff             @@
##           master     #627      +/-   ##
==========================================
- Coverage   70.92%   70.86%   -0.06%
==========================================
  Files         134      134
  Lines       12662    12674      +12
==========================================
+ Hits         8981     8982       +1
- Misses       3681     3692      +11

Impacted Files
Coverage Δ

prompt_toolkit/widgets/base.py
29.09% <8.33%> (-1.08%)
:arrow_down:

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 deb7488...9c53d74. Read the comment docs.

Hi @gpotter2, I added two remarks. If you have some time to look at that, I can merge it. Thanks again!

Just updated the PR according to your comments

Updated

Thanks a lot, @gpotter2!


How is the average memory access latency measured or calculated?

In the HDagg paper Section "Executor Evaluation" it is said that "The average memory access latency is used as a metric to measure locality." and "PAPI’s performance counters are used to measure architecture information needed in computations related to the locality and load balance metrics."
In the sptrsv_profiler.cpp example, the PAPI event list is:
https://github.com/sympiler/aggregation/blob/da293bb1d1060bc390ad785978f8452943a8909c/profiler/sptrsv_profiler.cpp#L106-L111
I wonder how is the memory latency obtained from the above metrics?

It is based on the average memory cycle defined in the computer architecture book (see page 75).
PAPI does not give you all counters you will need, and it changes per architecture. We used something like below:
def compute_memory_cycle_for_one_group(row, arch_params):
    dl1_miss = row['PAPI_L1_DCM'].values
    dl2_miss = row['PAPI_L2_DCM'].values
    dl3_miss = row['PAPI_L3_TCM'].values
    dl1_access = row['PAPI_LST_INS'].values
    l1_mr = dl1_miss / dl1_access
    l2_mr = dl2_miss / dl1_miss
    l3_mr = dl3_miss / dl2_miss
    l1_access_cost = arch_params['L1_ACCESS_TIME']
    l2_access_cost = arch_params['L2_ACCESS_TIME']
    l3_access_cost = arch_params['L3_ACCESS_TIME']
    mm_access_cost = arch_params['MAIN_MEMORY_ACCESS_TIME']
    avg_mem_cycle = l1_access_cost + l1_mr*(l2_access_cost + l2_mr*(l3_access_cost + l3_mr*mm_access_cost))
    exec_cycle = avg_mem_cycle  * dl1_access
    return exec_cycle

You will need some architecture parameters. You can improve the code by finding more accurate counters.

Thanks, that makes sense!  I find that Vtune also provides an average latency metric, but it's good to calculate and verify it from scratch.


Should go through all expects and ensure that they are reasonable

I have a few places left that I think we probably crash a bit too eagerly.

Removed most of these


[composer] Misc editor issues.

Smaller

[x] Decorator bug with large code blocks.

https://github.com/dxos/dxos/issues/5701

[ ] Do no remove decorations when traversing via up/down key until after delay.
[ ] Render images when scrolling through document.
[ ] @Mention (selective autocomplete)

Auto-complete on typing if @ otherwise require ctrl-space (e.g., to complete page link dialog)
Show "/" tooltip on blank line with popup menu.

[ ] Smooth scroll into view; ideally able to position in the middle of the viewport.

Larger

[ ] Language extension for schema.

Provide example for simple extension to JS language (e.g., https://effect-ts.github.io/schema/modules/Schema.ts.html)

[ ] Hierarchical editor (e.g., similar to logseq to replace current OutlinerPlugin):

drag-and-drop rows (current dxos/mosaic lib allows DND across components -- e.g., to/from sidebar).

Scrolling issues:

don't expand links until delay
images don't render until select
jitters when scrolling

Uploading Screen Recording 2024-02-27 at 10.53.56 PM.mov…

PR https://github.com/dxos/dxos/pull/5799 seems to help with the images loading late and flickering when scrolled into view.


Added success and timeout callbacks

Hi! I had a situation recently where I needed to know if an image did successfully load or not.
In my case I used a transparent 1x1 gif instead of a grey placeholder, because I showed a spinner animation as the image's background (css). But because my loaded images were transparent in some places, the spinner showed through when the full image appeared. So I added a success callback to be able to remove the spinner when an image had fully loaded.
There was also a case when the data-original url did not actually exist. Then the spinner would always be visible (not a very good user experience). So I also added a timeout callback (default 10 seconds) to remove the spinner then.
This makes the plugin a little bit more dynamic, with little extra plugin code. Take a look and see if you like it! :)

Hey! This is really helpful! It's just what I need... I hope this could be merged into stable soon.

Thanks for the patch! However load and appear events were implemented elsewhere so I am closing this now.

great :)


Typo to align expected outcome in Learn Module

#sign-off

#sign-off

Addresses user-reported feedback (e.g., AB#365106)

Addresses user-reported feedback (e.g., AB#365106)


add support for wildcard operator .

Add support for more operators from python regex, .* or .{100}
But do notice that once we add these, we are no longer strictly EBNF format, rather an enhanced ENBF.

@nathanrchn closed related to #21

Solved in #21


Getting error when I click on Login and Contact Support- Transactional Emails being rejected by Yahoo.

Expected content
Actual content
Link to page:

Hey @tylerwall - you can contact our support team about this issue through this form - https://support.sendgrid.com/hc/en-us/requests/new#login-issue
Closing since this isn't an issue specific to our docs.


Add "Delete" button for users to be able to delete files in copilot-chat-app sample

Hello !
I was trying to add a "delete" button in the copilot-chat-app sample to be able to delete files, it seems to work but I don't know if I'm doing it right. I am still a beginner in building webapps, so I was wondering if I could get some help here. (I don't know if a pull request is the best way to do it ? but I thought it might be a nice feature to add to the sample). I made a couple of changes in DocumentsTab and useFile and added DocumentDeleteService and DocumentDeleteController, if someone can help me pinpoint where I might be doing something wrong, I would be really grateful.
Thank you !

@teresaqhoang @TaoChenOSU I think you may be working on something similar already?

Hello @kenzabenjelloun,
Thank you for your PR!
This is a very nice feature to add, and we are happy that you are considering it.
Document handling is an essential part of the chat copilot app. Each imported document will generate data in 3 places:

The memory source repository
The chat message repository
The memory storage (in the form of embeddings)

To completely remove a document, we need to remove the date generated in all these 3 places. This PR only removes the memory sources. We encourage you to add implementation to complete the deletion of documents.
P.s. To delete a resource, you should use the delete request: https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods/DELETE.

Hi @kenzabenjelloun - This is a great start! I've added some additional suggestions on top of what my colleagues said earlier.
Are you still interested in pursuing this PR? I just want to check since I haven't seen any updates from you in a couple weeks.

If yes - I recommend you review the suggestions and take a look at Tao's comment about how to remove all the document's data
If no - no worries! If I don't hear from you within the next couple days, I will go ahead and close this PR. If you want to come back and work on it at a later date, you are welcome to reactivate it.

Hi @gitri-ms ! Yes, I would still like to work on it. Can I just get some pointers on when the embeddings are generated in the code ? I'm having a bit of trouble figuring it out (I thought it would be directly after the document is imported but I don't see it in DocumentImportController?)

FYI - DocumentImportController is being deleted in this change: https://github.com/microsoft/chat-copilot/pull/152
At this stage, introducing the feature might be best accomplished after https://github.com/microsoft/chat-copilot/pull/152 is merged.
Note: Your change and this update may overlap: https://github.com/microsoft/chat-copilot/pull/193

Hi @kenzabenjelloun - thank you again for this contribution. However, given the large refactor coming with #152, we are unable to accept this change at this time. For that reason, I am going to close this PR for now.
We may be interested in revisiting it with you once the semantic memory refactor code comes in. However, I do want to note that while we welcome community contributions, we have limited engineering resources to assist with implementation of features that aren't in our project roadmap.
Going forward, for new features like this, we would appreciate if you would first open an issue with us to discuss the feature you want to bring in and its design/prioritization.
Alternatively, you are always welcome to maintain a fork of Chat Copilot and build on it there! I also encourage you to join our Discord community to engage with other developers building on SK and Chat Copilot, as they might have ideas on how to go about implementing new features like this.


[Events]Verify that, after canceling the request, a pop-up notification appears

Environment: Windows 10 Pro Version 22H2 OS 19045.3803 Google Chrome Version 120.0.6099.200 (Official Build) (64-bit)
Reproducible: sometimes
Build found: 06.02.2024
Preconditions:
1.The site https://www.greencity.social/#/greenCity is opened
2.User signed in
3.Events page is opened
4.One event created by another user is present.
5. An event created by another user was found.
Steps to reproduce:
1.Click 'More' button
2.Click 'Join' event button
3.Click 'Cancel request' button
Actual result:
'Cancel request' button has been replaced by 'Join event' button
Expected result:
'Cancel request' button has been replaced by 'Join event' button and a pop-up notification appeared: "You canceled your request".
User story and test case links
E.g.: "User story #3270
https://jira.softserve.academy/browse/GC-4979
Labels to be added
"Bug", Priority ("pri: "), Severity ("severity:"), Type ("UI, "Functional"), "API" (for back-end bugs).

There are not any requirements to show pop-up:
Authorised User is able to cancel a join event request by clicking on the button “cancel request”:
a. it is replaced by a button “Join event” on the event’s card and on the event’s profile page.


[Task]: Improve the after all cleanup approach in tests and make effective use of DirtiesContext

SubTasks

Discussion thread 1: https://theappsmith.slack.com/archives/C06Q3A97USE/p1716462397624419
Discussion thread 2: https://theappsmith.slack.com/archives/C06Q3A97USE/p1716888655039179

Closing this as we are going ahead with using of DirtiesContext in most of our test cases wherever we want to cleanup context before we begin a test.


add PSA labels to openshift-infra in guest cluster

After https://github.com/openshift/hypershift/pull/2097, conformance test [sig-storage] PersistentVolumes NFS when invoking the Recycle reclaim policy should test that a PV becomes Available and is clean after the PVC is deleted. began failing.
https://prow.ci.openshift.org/view/gs/origin-ci-test/logs/periodic-ci-openshift-hypershift-main-periodics-4.13-conformance-aws-ovn/1625150475412180992
This is because the recycler pod is being blocked by PSA
Recycle failed: unexpected error creating recycler pod: pods "recycler-for-nfs-9mqrj" is forbidden: violates PodSecurity "restricted:latest": allowPrivilegeEscalation != false (container "pv-recycler" must set securityContext.allowPrivilegeEscalation=false), unrestricted capabilities (container "pv-recycler" must set securityContext.capabilities.drop=["ALL"]), restricted volume types (volume "vol" uses restricted volume type "nfs"), runAsNonRoot != true (pod or container "pv-recycler" must set securityContext.runAsNonRoot=true), seccompProfile (pod or container "pv-recycler" must set securityContext.seccompProfile.type to "RuntimeDefault" or "Localhost")

Standalone addressed this in https://github.com/openshift/cluster-kube-controller-manager-operator/pull/647

/test e2e-conformance

/test e2e-conformance

/lgtm

/test e2e-aws


Handle failed requires & missing bindings

This PR does two separate but closely related things. Check the commits individually if you'd like to separate them.
First: it moves resin-cli-errors into the CLI itself, and converts it to TS en route. This is a separate module that isn't really useful outside the CLI, doesn't need to be separate, and would be better inlined here. I'll deprecate the separate npm lib version later.
Second: it adds error handling & a nice message for require & missing binding errors. This will fix #872 (by providing a helpful message so users can do a complete uninstall & reinstall to recover for themselves), which lets us rerelease #870.
I've also included a nice message in the 2nd part for non-x64 users. Right now this shouldn't come up much, if at all, but it'll be super useful when we look at #845.

Tests fail because the API is down, sigh.


haskellPackages: mark linux-only packages

Motivation for this change
ZHF: #122042
cc @NixOS/nixos-release-managers, @NixOS/haskell
A small subset of Haskell packages that are currently failing on Darwin are simply not supported on Darwin by design, but not marked as such. This PR adds a helper and starts a list of these packages. I also included cpuid and hsignal under this helper umbrella, because they do a similar thing, but for x86-only.
Haskell in Nix has the unsupported-platforms list, but it's flexibility is limited, only allowing to eliminate specific platforms. For example, marking a package as not supported on Darwin would've worked for a while with x86_64-darwin,  but then we got aarch64-darwin. Similarly, marking a package as Linux-only is impossible: there is no opposite supported-platforms, plus we're continuously adding Linux platforms like riscv64-linux.
I think fixing this properly is outside the scope of ZHF and the upcoming release, and hopefully this stop-gap solution is okay for now.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

These settings belong into configuration-hackage2nix.yaml.

I disagree. The unsupported-platforms mechanism is too cumbersome for these kinds of changes: It doesn't allow us to reuse lib.platforms and it requires us to expand the list of unsupported platforms as new platforms are added to haskell packages or even nixpkgs itself.

I am a bit torn on the issue. I feel like if this is a only a temporary fix it would be okay to do it via the yaml file (I mean there won‘t be any new platforms added before branch off, will they?) And if this is not temporary we should also do it via the yaml file but need to change hackage2nix a bit.
Introducing this solution will introduce an inconsistent split. Between some platform being configured via yaml and some via nix.

I want this as an intermediate solution at the very least because it allows us to clean up the haskell-updates jobset in a simple and maintainable way.
In the future, I'd like to rework how platforms work in hackage2nix by allowing us to prescribe meta.platforms and meta.badPlatforms (which would be the successor to unsupported-platforms). The only conceptual trouble left to solve is how to distinguish between platforms as strings (like we have in unsupported-platforms) and named platform lists from lib.platforms.

I think I agree with @maralorn here.
If this is just a temporary fix, then it doesn't really matter how we do it for the 21.05 release.
However, in the long run, I'd like to go with a solution that is automatable (well, really, just some solution that makes us do minimal work).  Up until now, most of our automatable solutions have been based on yaml files.
@sternenseemann Do you have some sort of long-term vision for how platform-related stuff should work in the Haskell infrastructure?  I guess I'm mainly interested in what exactly the three of us will have to do for this.  I don't personally want to deal with problems on multiple platforms more than my current level of almost-completely-ignoring-it.  (Although don't get me wrong, I do like our current approach of trying to push more of the burden on drive-by-contributors.)

I'm not familiar with Haskell, nor am I a big Nix contributor, but my 2 cents:
I don't think this can be automated, unless upstream lists this somewhere in package metadata. But I don't see anything on hackage or in the cabal files of these packages. So pushing this to drive-by contributions (and efforts like ZHF) is probably the right solution for now.
What could help longer term is more automated nixpkgs-review of PRs on Darwin machines, like @r-rmcgibbo currently does for x86_64-linux and aarch64-linux. (Though that sort of review is currently not a requirement for merging.)
Right now, listing the two Darwin platforms in the existing unsupported-platforms is sufficient as a stop-gap to satisfy Hydra. But technically the list of 'non-Linux' is a lot longer:
nix-repl> lib.filter (x: !(lib.hasSuffix "-linux" x)) lib.platforms.all
[ "i686-cygwin" "x86_64-cygwin" "x86_64-darwin" "i686-darwin" "aarch64-darwin" "armv7a-darwin" "i686-freebsd" "x86_64-freebsd" "aarch64-genode" "i686-genode" "x86_64-genode" "x86_64-solaris" "js-ghcjs" "mmix-mmixware" "aarch64-netbsd" "armv6l-netbsd" "armv7a-netbsd" "armv7l-netbsd" "i686-netbsd" "mipsel-netbsd" "powerpc-netbsd" "riscv32-netbsd" "riscv64-netbsd" "x86_64-netbsd" "aarch64-none" "arm-none" "armv6l-none" "avr-none" "i686-none" "msp430-none" "or1k-none" "powerpc-none" "riscv32-none" "riscv64-none" "vc4-none" "x86_64-none" "i686-openbsd" "x86_64-openbsd" "x86_64-redox" "wasm64-wasi" "wasm32-wasi" "x86_64-windows" "i686-windows" ]

I think a long-term solution should at least:

incorporate lib.platforms, for better matching, and because distribution-nixpkgs is prone to be out of sync. (allKnownPlatforms is very limited)
Allow matching the opposite direction (ie. supported-platforms), which is the convention for Nixpkgs in general (meta.platforms = lib.platforms.linux;)
Allow combining sets (linux ++ darwin)

I pushed a version that lists these in the YAML just for x86_64-darwin, which should be enough for Hydra.
I cannot test it, though, because regenerate-hackage-packages.sh fails:
maintainers/scripts/haskell/regenerate-hackage-packages.sh: line 19: warning: setlocale: LC_ALL: cannot change locale (C.UTF-8): No such file or directory
Starting hackage2nix to regenerate pkgs/development/haskell-modules/hackage-packages.nix ...
hackage2nix: constraint cassava ==<IP_ADDRESS> cannot be resolved in Hackage
This could be due the package being missing in the hackage directory
or the file system not being case sensitive.

CallStack (from HasCallStack):
  error, called at hackage2nix/Main.hs:205:34 in main:Main
hackage2nix: thread blocked indefinitely in an MVar operation

I created a case-sensitive APFS volume for this, so that should not be the problem. I also don't believe the LC_ALL setting is the cause, though it may be a separate issue. (Seems like C.UTF-8 doesn't exist on Darwin. I have the same issue in #122634. I tried with en_US.UTF_8, no dice.)

Thanks, this looks good to merge in for now at least.


Allow third party download.

Holy Christ just go complain to curseforge.

I really don't know how many times I have to say this. Complain to curseforge. They don't pay me for third party downloads. If you complain, it could make them change their mind and then we both win.


MUD dev-contracts watcher failed to deploy config or contracts changes

[MUDError]: Gas estimation error for registerSystem('0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0','73,110,99,114,101,109,101,110,116,83,121,115,116,101,109,0','0x51E8A72176b53f50388418471d771C2805f65e4B','true'): execution reverted
use pnpm mud dev-contracts --tsgenOutput ../client/src/mud -rpc xx

can you share your repo to be able to reproduce this issue? to which RRC are you deploying? For development it's recommended to use a local one

I plan to use chainlink, so I can only test on the test network. The version is the latest version. The RPC node is  https://polygon-mumbai.g.alchemy.com/v2/APIKEY. How can I use the online version in the front end? I tried it and there were many errors. Can you give an example version?

can you share your repo to be able to reproduce this issue? to which RRC are you deploying? For development it's recommended to use a local one

https://github.com/aLIEzsss4/LinkSpark/blob/master/packages/contracts/src/systems/HeroSystem.sol

for deploying to a testnet try the mud deploy command instead of mud dev-contracts. The former waits for transactions to be confirmed before sending the next one while the latter one just hammers the transactions out because the local node is fast enough to handle it. (Also the latter one redeploys every time you change a file, which is probably not what you want when deploying to a remote network)

for deploying to a testnet try the mud deploy command instead of mud dev-contracts. The former waits for transactions to be confirmed before sending the next one while the latter one just hammers the transactions out because the local node is fast enough to handle it. (Also the latter one redeploys every time you change a file, which is probably not what you want when deploying to a remote network)

How to use the Mumbai version in the frontend? I have tried it and there are many errors. Can you give an example version?

If the deployment succeeded you should be able to specify a chainId param to the client url. Could you share a screenshot of the errors you're seeing?

we just rebuild our deploy pipeline in https://github.com/latticexyz/mud/pull/1702, so going to close this as "outdated" for now but feel free to open with reproduction steps if this is still an issue


packbox command fails if Bhyve enabled

Logging this as an issue for support purposes. It can be closed.
potman packbox was getting stuck and exiting:
==> virtualbox-iso: Using SSH communicator to connect: <IP_ADDRESS>
==> virtualbox-iso: Waiting for SSH to become available...
==> virtualbox-iso: Timeout waiting for SSH.
==> virtualbox-iso: Cleaning up floppy disk...
==> virtualbox-iso: Deregistering and deleting VM...
==> virtualbox-iso: Deleting output directory...
Build 'virtualbox-iso' errored after 20 minutes 59 seconds: Timeout waiting for SSH.

The problem stems from having Bhyve enabled, and the solution is to disable it.
sysrc vm_enable="NO"

then reboot
Afterwards potman packbox runs normally.
Useful to also watch the logs in ~/VirtualBox VMs/box/Logs/VBox.log to isolate where the problem is, if any.

Relevant https://wiki.freebsd.org/bhyve#Q:_Can_I_run_bhyve_and_VirtualBox_on_FreeBSD_at_the_same_time.3F


Formatting error in http://git-scm.com/docs/user-manual

There is a formatting problem in the source code of page http://git-scm.com/docs/user-manual that makes almost half of it be rendered in a <pre> element that displays the page source in the original markup language instead of being converted to HTML.
The issue is in the paragraph that stars with "The index is a binary file (generally kept in `.git/index`)..."

I reported this here because I couldn't find a contact page on git-scm.com. I found it out in the meantime and reported the problem there. Closing this issue.


[camera] [API 10+, API 11+]  Samsung S24 crash - reboot phone

Steps to reproduce
The only thing you have to do is start the camera several times. This also happens on other devices but on the S24 with Android 14 approximately on attempt number 6 the phone crashes. It doesn't crash, not even the app, but the entire phone and it resets.
Expected results
Do not close the app and even less reboot the phone
Actual results
The  phone is rebooting on the S24 and on other devices like the Tecno camon 19 pro just close the app
Code sample
Repo
https://github.com/tyba-co/document_selfie_verification
Steps

clone repo
install dependencies
run example
take Picture
Repeat Photo
take Picture
Repeat Photo
take Picture
Repeat Photo
take Picture
Repeat Photo
take Picture
Repeat Photo
take Picture
Repeat Photo

Screenshots or Video

Screenshots / Video demonstration
https://github.com/user-attachments/assets/187f5a66-c497-4f5d-98bf-1b26704dcf89

Logs
Logs
I/Camera  (29142): startPreview
I/Camera  (29142): CameraCaptureSession onConfigured
I/Camera  (29142): Updating builder settings
D/Camera  (29142): Updating builder with feature: ExposureLockFeature
D/Camera  (29142): Updating builder with feature: ExposurePointFeature
D/Camera  (29142): Updating builder with feature: ZoomLevelFeature
D/Camera  (29142): Updating builder with feature: AutoFocusFeature
D/Camera  (29142): Updating builder with feature: NoiseReductionFeature
I/Camera  (29142): updateNoiseReduction | currentSetting: fast
D/Camera  (29142): Updating builder with feature: FocusPointFeature
D/Camera  (29142): Updating builder with feature: ResolutionFeature
D/Camera  (29142): Updating builder with feature: SensorOrientationFeature
D/Camera  (29142): Updating builder with feature: FlashFeature
D/Camera  (29142): Updating builder with feature: ExposureOffsetFeature
D/Camera  (29142): Updating builder with feature: FpsRangeFeature
I/Camera  (29142): refreshPreviewCaptureSession
I/CameraManagerGlobal(29142): Camera 0 facing CAMERA_FACING_BACK state now CAMERA_STATE_ACTIVE for client com.example.example API Level 2 User Id 0
W/System  (29142): A resource failed to call Surface.release.
E/System  (29142): Uncaught exception thrown by finalizer
E/System  (29142): java.util.concurrent.RejectedExecutionException: Task android.hardware.camera2.impl.CallbackProxies$SessionStateCallbackProxy$$ExternalSyntheticLambda0@aa1fe05 rejected from java.util.concurrent.ThreadPoolExecutor@f3cde5a[Shutting down, pool size = 1, active threads = 0, queued tasks = 0, completed tasks = 9]
E/System  (29142):      at java.util.concurrent.ThreadPoolExecutor$AbortPolicy.rejectedExecution(ThreadPoolExecutor.java:2082)
E/System  (29142):      at java.util.concurrent.ThreadPoolExecutor.reject(ThreadPoolExecutor.java:842)
E/System  (29142):      at java.util.concurrent.ThreadPoolExecutor.execute(ThreadPoolExecutor.java:1374)
E/System  (29142):      at java.util.concurrent.Executors$DelegatedExecutorService.execute(Executors.java:678)
E/System  (29142):      at android.hardware.camera2.impl.CallbackProxies$SessionStateCallbackProxy.onClosed(CallbackProxies.java:104)
E/System  (29142):      at android.hardware.camera2.impl.CameraCaptureSessionImpl.close(CameraCaptureSessionImpl.java:588)
E/System  (29142):      at android.hardware.camera2.impl.CameraCaptureSessionImpl.finalize(CameraCaptureSessionImpl.java:894)
E/System  (29142):      at java.lang.Daemons$FinalizerDaemon.doFinalize(Daemons.java:370)
E/System  (29142):      at java.lang.Daemons$FinalizerDaemon.processReference(Daemons.java:350)
E/System  (29142):      at java.lang.Daemons$FinalizerDaemon.runInternal(Daemons.java:322)
E/System  (29142):      at java.lang.Daemons$Daemon.run(Daemons.java:131)
E/System  (29142):      at java.lang.Thread.run(Thread.java:1012)
W/System  (29142): A resource failed to call Surface.release.
E/System  (29142): Uncaught exception thrown by finalizer
E/System  (29142): java.util.concurrent.RejectedExecutionException: Task android.hardware.camera2.impl.CallbackProxies$SessionStateCallbackProxy$$ExternalSyntheticLambda0@a896d68 rejected from java.util.concurrent.ThreadPoolExecutor@2497581[Shutting down, pool size = 1, active threads = 0, queued tasks = 0, completed tasks = 8]
E/System  (29142):      at java.util.concurrent.ThreadPoolExecutor$AbortPolicy.rejectedExecution(ThreadPoolExecutor.java:2082)
E/System  (29142):      at java.util.concurrent.ThreadPoolExecutor.reject(ThreadPoolExecutor.java:842)
E/System  (29142):      at java.util.concurrent.ThreadPoolExecutor.execute(ThreadPoolExecutor.java:1374)
E/System  (29142):      at java.util.concurrent.Executors$DelegatedExecutorService.execute(Executors.java:678)
E/System  (29142):      at android.hardware.camera2.impl.CallbackProxies$SessionStateCallbackProxy.onClosed(CallbackProxies.java:104)
E/System  (29142):      at android.hardware.camera2.impl.CameraCaptureSessionImpl.close(CameraCaptureSessionImpl.java:588)
E/System  (29142):      at android.hardware.camera2.impl.CameraCaptureSessionImpl.finalize(CameraCaptureSessionImpl.java:894)
E/System  (29142):      at java.lang.Daemons$FinalizerDaemon.doFinalize(Daemons.java:370)
E/System  (29142):      at java.lang.Daemons$FinalizerDaemon.processReference(Daemons.java:350)
E/System  (29142):      at java.lang.Daemons$FinalizerDaemon.runInternal(Daemons.java:322)
E/System  (29142):      at java.lang.Daemons$Daemon.run(Daemons.java:131)
E/System  (29142):      at java.lang.Thread.run(Thread.java:1012)
W/System  (29142): A resource failed to call Surface.release.
E/System  (29142): Uncaught exception thrown by finalizer
E/System  (29142): java.util.concurrent.RejectedExecutionException: Task android.hardware.camera2.impl.CallbackProxies$SessionStateCallbackProxy$$ExternalSyntheticLambda0@24ee267 rejected from java.util.concurrent.ThreadPoolExecutor@73bf814[Shutting down, pool size = 1, active threads = 0, queued tasks = 0, completed tasks = 9]
E/System  (29142):      at java.util.concurrent.ThreadPoolExecutor$AbortPolicy.rejectedExecution(ThreadPoolExecutor.java:2082)
E/System  (29142):      at java.util.concurrent.ThreadPoolExecutor.reject(ThreadPoolExecutor.java:842)
E/System  (29142):      at java.util.concurrent.ThreadPoolExecutor.execute(ThreadPoolExecutor.java:1374)
E/System  (29142):      at java.util.concurrent.Executors$DelegatedExecutorService.execute(Executors.java:678)
E/System  (29142):      at android.hardware.camera2.impl.CallbackProxies$SessionStateCallbackProxy.onClosed(CallbackProxies.java:104)
E/System  (29142):      at android.hardware.camera2.impl.CameraCaptureSessionImpl.close(CameraCaptureSessionImpl.java:588)
E/System  (29142):      at android.hardware.camera2.impl.CameraCaptureSessionImpl.finalize(CameraCaptureSessionImpl.java:894)
E/System  (29142):      at java.lang.Daemons$FinalizerDaemon.doFinalize(Daemons.java:370)
E/System  (29142):      at java.lang.Daemons$FinalizerDaemon.processReference(Daemons.java:350)
E/System  (29142):      at java.lang.Daemons$FinalizerDaemon.runInternal(Daemons.java:322)
E/System  (29142):      at java.lang.Daemons$Daemon.run(Daemons.java:131)
E/System  (29142):      at java.lang.Thread.run(Thread.java:1012)
W/System  (29142): A resource failed to call Surface.release.
E/System  (29142): Uncaught exception thrown by finalizer
E/System  (29142): java.util.concurrent.RejectedExecutionException: Task android.hardware.camera2.impl.CallbackProxies$SessionStateCallbackProxy$$ExternalSyntheticLambda0@358deb2 rejected from java.util.concurrent.ThreadPoolExecutor@5d3e03[Shutting down, pool size = 1, active threads = 0, queued tasks = 0, completed tasks = 7]
E/System  (29142):      at java.util.concurrent.ThreadPoolExecutor$AbortPolicy.rejectedExecution(ThreadPoolExecutor.java:2082)
E/System  (29142):      at java.util.concurrent.ThreadPoolExecutor.reject(ThreadPoolExecutor.java:842)
E/System  (29142):      at java.util.concurrent.ThreadPoolExecutor.execute(ThreadPoolExecutor.java:1374)
E/System  (29142):      at java.util.concurrent.Executors$DelegatedExecutorService.execute(Executors.java:678)
E/System  (29142):      at android.hardware.camera2.impl.CallbackProxies$SessionStateCallbackProxy.onClosed(CallbackProxies.java:104)
E/System  (29142):      at android.hardware.camera2.impl.CameraCaptureSessionImpl.close(CameraCaptureSessionImpl.java:588)
E/System  (29142):      at android.hardware.camera2.impl.CameraCaptureSessionImpl.finalize(CameraCaptureSessionImpl.java:894)
E/System  (29142):      at java.lang.Daemons$FinalizerDaemon.doFinalize(Daemons.java:370)
E/System  (29142):      at java.lang.Daemons$FinalizerDaemon.processReference(Daemons.java:350)
E/System  (29142):      at java.lang.Daemons$FinalizerDaemon.runInternal(Daemons.java:322)
E/System  (29142):      at java.lang.Daemons$Daemon.run(Daemons.java:131)
E/System  (29142):      at java.lang.Thread.run(Thread.java:1012)
F/crash_dump64(31960): crash_dump.cpp:554] failed to attach to thread 973: Permission denied
F/crash_dump64(32045): crash_dump.cpp:554] failed to attach to thread 1202: Permission denied
F/crash_dump64(32126): crash_dump.cpp:554] failed to attach to thread 973: Permission denied
F/crash_dump64(32213): crash_dump.cpp:554] failed to attach to thread 1202: Permission denied

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.19.6, on macOS 14.1.2 23B92 darwin-arm64, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2023.2)
[✓] VS Code (version 1.93.1)
[✓] Connected device (4 available)
    ! Error: Browsing on the local area network for Ricardo’s iPhone. Ensure the device is unlocked and attached with a cable or associated with the same local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)
[✓] Network resources

Based on the error log triggered, it seems similar to https://github.com/flutter/flutter/issues/108941, so please follow-up in it for further updates.
Closing this as a duplicate.


Add example for Transform's path_arc

Overview: What does this pull request change?
This was very handy for me when trying to swap two parts of an equation around without them getting muddled together first. Thus, I want it to be more clear how to use the path_arc feature of Transform, so I added an example showing how the different angles change the movement path.
Links to added or changed documentation pages
Transform Page
https://user-images.githubusercontent.com/6819944/175700022-efb9c6b3-d278-4905-8bc1-85c8c093c19a.mp4
Reviewer Checklist

[ ] The PR title is descriptive enough for the changelog, and the PR is labeled correctly
[ ] If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
[ ] If applicable: newly added functions and classes are tested

This looks good. Thank you for your work! But i'm not sure if we can support more Animations in the Documentation right now. @behackl might take a look in a few days. Till then we will just keep this here till we know for sure.

But you can add it as a template though!
https://github.com/ManimCommunity/manim/tree/main/manim/templates

Thank you for the excellent suggestions.

Anything else to address?


CONDA_ENVS_PATH ignored?

I am experimenting with using CONDA_ENVS_PATH to make various environments visible to our jupyterhub users. In the script for starting up the user's singleuser-server this variable is set and I can confirm it also in the output of a "conda info -a" that I run before the user's jupyter process is started. Also all of the envs from CONDA_ENVS_PATH are visible there. However, in the running jupyter notebook I can only see the envs which belong to the path of the base env and the user's default location at ~/.conda/envs. It seems that CONDA_ENVS_PATH is ignored when searching for available kernels.
I am using nb_conda_kernels  2.2.1.
Thanks

Sorry for the delay on this one. But nb_conda_kernels is not responsible for enumerating the environments—that it delegates, as it should in my view, to conda itself. In particular, if conda info --envs cannot see an environment, then nb_conda_kernels will not be able too, either.

So I guess the question is, are you sure that the CONDA_ENVS_PATH environment variable is making it to the process that is running nb_conda_kernels? My guess is that it is not.


Add goanpeca/setup-miniconda

This is used to install Miniconda environments. If a new Python section header is eventually created, it should go there.

@henryiii your changes have been merged, thanks for your contribution 👍


Update init params for controller to include ble layer

Problem
Android app crashes on trying to provision over WiFi.
It turns out the BLE layer is not passed into the controller and this crashes with 'INVALID_ARGUMENT' since android has no device layer.
Summary of Changes
We have 4 different init functions in the controller, 2 of which are effectively deprecated and take in way too many arguments. Switched them to have 2 init methods with 'params' arguments instead.
For now tested that code compiles (build and test all passes) and android app does not crash anymore and connects to the device (BLE is active).

@mspang ?

@andy31415 sorry, conflicts :(


Fix regression: Only write register address with no value.

Sorry @generationmake :bow:
I'm quite sure i messed up the library during cleanup, can you test the version in this branch if it still works?
Cheers, Alex

Hi @aentinger
I tested it with my sensor and it works like expected.
so please merge.

Thank you @generationmake :bow: