Fixed tiny weather icon in hass v0.110+

The CSS property seems to have changed. Works this way for me.

Thanks!

Thanks for the quick merge!


[WIKI] cURL commands for API methods.

I managed to work out some of the missing API methods in the Wiki, and am putting the cURL commands below for anyone who needs them.
# Check if whatsapp account exists?
curl -X 'GET' \
  "http://<IP_ADDRESS>:3000/api/v1/whatsapp/registered?msisdn=$PHONE_NUMBER" \
  -H "Authorization: Bearer $TOKEN" \
  -H 'accept: application/json'

# Send text
curl -X 'POST' \
  'http://<IP_ADDRESS>:3000/api/v1/whatsapp/send/text' \
  -H 'accept: application/json' \
  -H "Authorization: Bearer $TOKEN" \
  -H 'Content-Type: multipart/form-data' \
  -F "msisdn=$PHONE_NUMBER" \
  -F 'message=Hello World'

# Send Image
curl -X 'POST' \
  'http://<IP_ADDRESS>:3000/api/v1/whatsapp/send/image' \
  -H 'accept: application/json' \
  -H "Authorization: Bearer $TOKEN" \
  -H 'Content-Type: multipart/form-data' \
  -F "msisdn=$PHONE_NUMBER" \
  -F 'caption=<Image Caption Here>' \
  -F 'image=@image.jpg;type=image/jpeg' \
  -F 'viewonce=false'

# Send Video
curl -X 'POST' \
  'http://<IP_ADDRESS>:3000/api/v1/whatsapp/send/video' \
  -H 'accept: application/json' \
  -H "Authorization: Bearer $TOKEN" \
  -H 'Content-Type: multipart/form-data' \
  -F "msisdn=$PHONE_NUMBER" \
  -F 'caption=<Video Caption Here>' \
  -F 'video=@video.mp4;type=video/mp4' \
  -F 'viewonce=false'

# Send document
curl -X 'POST' \
  'http://<IP_ADDRESS>:3000/api/v1/whatsapp/send/document' \
  -H 'accept: application/json' \
  -H "Authorization: Bearer $TOKEN" \
  -H 'Content-Type: multipart/form-data' \
  -F "msisdn=$PHONE_NUMBER" \
  -F 'document=@document.pdf;type=application/pdf'

I hope this will help fill in the gaps in the Wiki pages :)

Hai @AvinashReddy3108 thanks for your help may be you can help me by requesting pull request for the wiki.
Thank you so much :D

Sure, I'd love to. But I can't seem to figure out how to make a pull request for the Wiki.
This is exactly why I had to make this issue :eyes:

@AvinashReddy3108 thanks, i added to Endpoint Wiki,
i try to help to create better Documentation, but get busy with task from my work 😁
https://github.com/dimaskiddo/go-whatsapp-multidevice-rest/wiki/Endpoint

Thank you so much pak @ibnux for the update 🥂 😄


Prevent collapses to zero width when one of column cell is empty

#424

Coverage increased (+0.04%) to 52.806% when pulling e218a8590f68991125ac39d1ae662d1c2a588f5c on KaczuH:issue-424 into 0fdf499ebab50af3868a1d1e6ace1cc7b4f51317 on xhtml2pdf:master.


Quickstart

Hi All,
First of all, thank you all for your contributions and for bringing this project alive.
I just started using the Project and as instructed I followed the documentation for Quickstart - https://socotecio.github.io/django-socio-grpc/#quickstart
When I follow the steps I'll be getting an error when I try to execute the command,
python manage.py generateproto
This gives the following error,
Traceback (most recent call last):
  File "<project-path>/tutorial/venv/lib/python3.10/site-packages/django/utils/module_loading.py", line 30, in import_string
    return cached_import(module_path, class_name)
  File "<project-path>/tutorial/venv/lib/python3.10/site-packages/django/utils/module_loading.py", line 16, in cached_import
    return getattr(module, class_name)
AttributeError: module 'tutorial.urls' has no attribute 'grpc_handlers'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "<project-path>/tutorial/venv/lib/python3.10/site-packages/django_socio_grpc/settings.py", line 111, in import_from_string
    return import_string(val)
  File "<project-path>/tutorial/venv/lib/python3.10/site-packages/django/utils/module_loading.py", line 32, in import_string
    raise ImportError(
ImportError: Module "tutorial.urls" does not define a "grpc_handlers" attribute/class

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<project-path>/tutorial/manage.py", line 22, in <module>
    main()
  File "<project-path>/tutorial/manage.py", line 18, in main
    execute_from_command_line(sys.argv)
  File "<project-path>/tutorial/venv/lib/python3.10/site-packages/django/core/management/__init__.py", line 446, in execute_from_command_line
    utility.execute()
  File "<project-path>/tutorial/venv/lib/python3.10/site-packages/django/core/management/__init__.py", line 440, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "<project-path>/tutorial/venv/lib/python3.10/site-packages/django/core/management/base.py", line 402, in run_from_argv
    self.execute(*args, **cmd_options)
  File "<project-path>/tutorial/venv/lib/python3.10/site-packages/django/core/management/base.py", line 448, in execute
    output = self.handle(*args, **options)
  File "<project-path>/tutorial/venv/lib/python3.10/site-packages/django_socio_grpc/management/commands/generateproto.py", line 53, in handle
    grpc_settings.ROOT_HANDLERS_HOOK(None)
  File "<project-path>/tutorial/venv/lib/python3.10/site-packages/django_socio_grpc/settings.py", line 156, in __getattr__
    val = perform_import(val, attr)
  File "<project-path>/tutorial/venv/lib/python3.10/site-packages/django_socio_grpc/settings.py", line 94, in perform_import
    return import_from_string(
  File "<project-path>/tutorial/venv/lib/python3.10/site-packages/django_socio_grpc/settings.py", line 113, in import_from_string
    raise ImportError(
ImportError: Could not import 'tutorial.urls.grpc_handlers' for GRPC setting 'ROOT_HANDLERS_HOOK'. ImportError: Module "tutorial.urls" does not define a "grpc_handlers" attribute/class.

I saw the django-grpc-framework project (https://github.com/fengsp/django-grpc-framework) mention this in the README.md file.
urlpatterns = []
def grpc_handlers(server):
    demo_pb2_grpc.add_UserControllerServicer_to_server(UserService.as_servicer(), server)

but this has to have generated stub classes.
Can you guide me on  thow to get the quickstart project up and running?

Hello @lahirujayathilake The published doc was not up to date because of mkdocs font rgpd policy. Just upgraded my mkdocs policy and published the latest change.
You should have a complete quickstart on https://socotecio.github.io/django-socio-grpc/#quickstart
I close this issue. If you face other problem do not hesitate to ask us.

Thanks, @AMontagu for the update. Yes, it solved that issue however, I'm getting the following error when python manage.py generateproto is executed
  File "<project-url>/tutorial/venv/lib/python3.10/site-packages/lark/lexer.py", line 478, in lex
    yield lexer.next_token(lexer_state, parser_state)
  File "<project-url>/tutorial/venv/lib/python3.10/site-packages/lark/lexer.py", line 398, in next_token
    raise UnexpectedCharacters(lex_state.text, line_ctr.char_pos, line_ctr.line, line_ctr.column,
lark.exceptions.UnexpectedCharacters: No terminal matches 'q' in the current parser context, at line 4 col 19

  optional string question_text = 1;
                  ^
Expected one of:
        * EQUAL

Previous tokens: Token('FIELDNAME', 'string')

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<project-url>/tutorial/manage.py", line 22, in <module>
    main()
  File "<project-url>/tutorial/manage.py", line 18, in main
    execute_from_command_line(sys.argv)
  File "<project-url>/tutorial/venv/lib/python3.10/site-packages/django/core/management/__init__.py", line 446, in execute_from_command_line
    utility.execute()
  File "<project-url>/tutorial/venv/lib/python3.10/site-packages/django/core/management/__init__.py", line 440, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "<project-url>/tutorial/venv/lib/python3.10/site-packages/django/core/management/base.py", line 402, in run_from_argv
    self.execute(*args, **cmd_options)
  File "<project-url>/tutorial/venv/lib/python3.10/site-packages/django/core/management/base.py", line 448, in execute
    output = self.handle(*args, **options)
  File "<project-url>/tutorial/venv/lib/python3.10/site-packages/django_socio_grpc/management/commands/generateproto.py", line 80, in handle
    protos_by_app = generator.get_protos_by_app()
  File "<project-url>/tutorial/venv/lib/python3.10/site-packages/django_socio_grpc/protobuf/generators.py", line 45, in get_protos_by_app
    self.current_existing_proto_data = self.parse_existing_proto_file(
  File "<project-url>/tutorial/venv/lib/python3.10/site-packages/django_socio_grpc/protobuf/generators.py", line 201, in parse_existing_proto_file
    proto_data = protoparser.serialize2json_from_file(existing_proto_path)
  File "<project-url>/tutorial/venv/lib/python3.10/site-packages/protoparser/parser.py", line 362, in serialize2json_from_file
    return json.dumps(_recursive_to_dict(parse(data)))
  File "<project-url>/tutorial/venv/lib/python3.10/site-packages/protoparser/parser.py", line 322, in parse
    tree = parser.parse(data)
  File "<project-url>/tutorial/venv/lib/python3.10/site-packages/lark/lark.py", line 581, in parse
    return self.parser.parse(text, start=start, on_error=on_error)
  File "<project-url>/tutorial/venv/lib/python3.10/site-packages/lark/parser_frontends.py", line 106, in parse
    return self.parser.parse(stream, chosen_start, **kw)
  File "<project-url>/tutorial/venv/lib/python3.10/site-packages/lark/parsers/lalr_parser.py", line 41, in parse
    return self.parser.parse(lexer, start)
  File "<project-url>/tutorial/venv/lib/python3.10/site-packages/lark/parsers/lalr_parser.py", line 171, in parse
    return self.parse_from_state(parser_state)
  File "<project-url>/tutorial/venv/lib/python3.10/site-packages/lark/parsers/lalr_parser.py", line 188, in parse_from_state
    raise e
  File "<project-url>/tutorial/venv/lib/python3.10/site-packages/lark/parsers/lalr_parser.py", line 178, in parse_from_state
    for token in state.lexer.lex(state):
  File "<project-url>/tutorial/venv/lib/python3.10/site-packages/lark/lexer.py", line 487, in lex
    raise UnexpectedToken(token, e.allowed, state=parser_state, token_history=[last_token], terminals_by_name=self.root_lexer.terminals_by_name)
lark.exceptions.UnexpectedToken: Unexpected token Token('CONSTANT', 'question_text') at line 4, column 19.
Expected one of:
        * EQUAL
Previous tokens: [Token('FIELDNAME', 'string')]

Is there something I'm missing?

@lahirujayathilake Can you provide a minimal reproduction of this ?
But first you can try to delete all your protofile and try generate again. The rror look like you already have a proto file with a syntax we do not support.

Close due to inactivity


Library name change request

Library repository URL
https://github.com/holgerlembke/st7567sfGK

I'm so sorry.
I wished it was clearer to me, that the name is the main indexing item and therefor can not be changed that easily.

Hi @holgerlembke. I am confirming your request.
Unlike submissions, this type of request is still handled manually. The Library Manager index maintainer occasionally comes through to clear all the outstanding requests. They will close this issue when it is done. You will get a notification from GitHub at that time.
After that, you can monitor the indexing of the library via its dedicated logs web page.
Information about that here:
https://github.com/arduino/library-registry/blob/main/FAQ.md#can-i-check-on-library-releases-being-added-to-library-manager

❤

Done:
2024/02/15 13:41:01 Loaded 6650 libraries from DB
Removing st7567sfGK
Success!

❤


[ci skip] added user danielnachun

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Can we merge this now that Azure is supposed to be fixed?


error on npm start

Unsure whether this is an FP error or environmental. But I've tried every fix I can find or think of, and any help would be appreciated.

How can this bug be reproduced?

Working install
Restart computer
Run NPM start and see error

What happened instead?
C:\xampplite\htdocs\mysite.dev\wp-content\themes\FoundationPress>npm start

<EMAIL_ADDRESS>start C:\xampplite\htdocs\mysite.dev\wp-content\themes\FoundationPress
gulp

[13:34:57] Requiring external module babel-register
fs.js:642
return binding.open(pathModule._makeLong(path), stringToFlags(flags), mode);
^
Error: EPERM: operation not permitted, open 'C:\Users\Evan.babel.json'
at Error (native)
at Object.fs.openSync (fs.js:642:18)
at Object.fs.writeFileSync (fs.js:1348:33)
at save (C:\xampplite\htdocs\mysite.dev\wp-content\themes\FoundationPress\node_modules\babel-register\lib\cache.js:48:16)
at _combinedTickCallback (internal/process/next_tick.js:73:7)
at process._tickCallback (internal/process/next_tick.js:104:9)
at Module.runMain (module.js:606:11)
at run (bootstrap_node.js:389:7)
at startup (bootstrap_node.js:149:9)
at bootstrap_node.js:502:3
npm ERR! Windows_NT 10.0.15063
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "start"
npm ERR! node v6.11.3
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: gulp
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'gulp'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the foundationpress package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     gulp
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs foundationpress
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls foundationpress
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\xampplite\htdocs\mysite.dev\wp-content\themes\FoundationPress\npm-debug.log
C:\xampplite\htdocs\mysite.dev\wp-content\themes\FoundationPress>node -v
v6.11.3
Please List the Following:

Windows 10 Pro 15603.608
Node 6.11.3
FoundationPress 2.10.3
foundation-sites: "6.4.1"
DesktopServer (rather than xampp)

Sounds like this problem arises from your user folder being read-only or if the file is marked hidden. See:
https://github.com/facebook/react-native/issues/8678

.babel.json was in fact hidden. Thank you.


Inconsistent conversion between transformer and metric

According to the documentation, the learned distance between x and y is expressed as sqrt((x-y).dot(inv(M)).dot(x-y)), which is equivalent to evaluating the Euclidean distance in the transformed space obtained from dot(X, L.T) for n x d data matrices X with L = inv(cholesky(M)).
However, if an algorithm does not learn a metric M, but its corresponding transformer L directly, the current implementation of BaseMetricLearner.metric() computes M as L.T.dot(L), which actually equals inv(M), according to the definition of L given above.
If I got the documentation right, then the following should hold:
learner.transformer().T.dot(learner.transformer()) == learner.metric()

This is, however, currently not the case for algorithms learning M (e.g., ITML, LSML or even the covariance method).

In addition, the definition of a "metric" is not even consistent across different learners: ITML and LSML learn a metric sqrt((x-y).dot(M).dot(x-y)), while the covariance method and SDLM (I guess) use inv(M) as stated in the readme.
Thus, the transformers returned by ITML and LSML are wrong!
I would suggest harmonizing the terminology towards the one currently employed by ITML and LSML, since it seems also predominant in literature, in my opinion. This would require removing the inv() from BaseMetricLearner.transformer() as well as changing all learners relying on the current terminology, such as ITML ans LSML.

Thanks for bringing this issue up. I agree that we need to be more consistent with the mapping between transformers and metrics.
I want to make sure we get this right, so I'm going to re-read some of the literature and see if I come to the same conclusion you did.

Thanks for looking into this!
I completely agree that a change like this needs to be made very carefully.
Just in case you come to the same conclusion as I, I've already tried to make the necessary changes in the Pull Request referenced above.
Since ITML and LSML return just wrong transformers at the moment, their performance could be significantly improved by fixing this issue.

After letting this languish for too long, I think I'm convinced. Thanks for pointing this out!


Question about acceptable AUC, improving AUC

Hi Tangram Team,
I've been working with tangram-sc and am seeking advice on improving the AUC (Area Under the Curve) of my models for 10x visium.

Could you provide any tips or best practices for optimizing the AUC when using tangram-sc? Are there specific parameters, preprocessing steps, or data characteristics that typically have a significant impact on AUC improvement in the context of single-cell data analysis?
Is there a range of AUC that can be considered acceptable?
We have duplicated two samples from the same subject and several subjects in total. These samples were integrated using Harmony. My question is: Would you recommend using all these integrated samples as one combined input for Tangram, or is it more beneficial to split them into individual samples and run Tangram separately on each?

Any insights or suggestions you could offer would be greatly appreciated!
Thanks in advance for your help!

Hi @boyangzhang1993!
Thank you for you patience as well as your interest in Tangram!

General best practices for Tangram are:

Making sure either both single cell data and spatial data are normalized or use raw counts for both.
Using a set of marker genes depending on the cell types in your dataset as training genes.
One key assumption is that the single cell and spatial data are not only from the same species and disease system but also from the same specimen. Tangram will still do a good job but for the most ideal results using single cell and spatial data from the same specimen is recommended.

If you take a look at the curve which is used to compute the AUC we contrast score and spatial sparsity. Here the score is the cosine similarity between the measured and predicted spatial gene expression.  Generally higher the sparsity of the genes, poorer the score; this is because Tangram attempts to correct the expression of these genes. So I would say AUC depends a little bit on the quality of dataset. If there are genes that are sparser in one dataset than another, that would reduce the AUC. To answer your question, I'm not sure if I can recommend an acceptable range of AUC. I would, however, recommend using this metric in addition to the diagnosis plots as well as the cell annotation maps to infer the quality of mapping by Tangram.

If the samples integrated by Harmony are from the same subject, it is OK to use the integrated dataset! However, if for each of your single cell datasets you also have a corresponding spatial dataset, then I would say map them separately.


Install CLI tools with V4 Emulator install

Let's consider adding an "Install Commandline Tools" menu option to install the CLI tools.  We can also see if we can then leverage autoupdate for CLI as well as the core emulator

Three separate work items here -

Add a "Install command line tools" menu option that presents a modal with information on what the CLI tools are and link to docs with a checkbox to 'auto update command line tools'. Clicking ok/ install in this dialog should install all cli tools. Should prompt user to install node and try again if node is not installed.
Have auto update checker also look for presence of CLI tool and if found and if auto-update is turned on for CLI tools, update them as well.
'Auto update command line tools' setting is manageable via emulator settings page.


Admin t-shirt

[ ] headline
[ ]  copyroom id and url
[ ]  vote t-shirt size / estimate button
[ ]  display name and room id
[ ]  display result
[ ]  start votin with "name"
[ ]  close voting button
[ ]  close room button


Update rockcraft.yaml to build reef ROCK

Changed package repository to point to Bobcat UCA

# sudo cephadm --image utkarshhere/ceph:latest bootstrap --mon-ip <IP_ADDRESS> --single-host-defaults
Creating directory /etc/ceph for ceph.conf
Verifying podman|docker is present...
Verifying lvm2 is present...
Verifying time synchronization is in place...
Unit systemd-timesyncd.service is enabled and running
Repeating the final host check...
docker (/usr/bin/docker) is present
systemctl is present
lvcreate is present
Unit systemd-timesyncd.service is enabled and running
Host looks OK
Cluster fsid: cab5e912-6c09-11ee-8e1e-00163e14ad55
Verifying IP <IP_ADDRESS> port 3300 ...
Verifying IP <IP_ADDRESS> port 6789 ...
Mon IP `<IP_ADDRESS>` is in CIDR network `<IP_ADDRESS>/24`
Mon IP `<IP_ADDRESS>` is in CIDR network `<IP_ADDRESS>/24`
Mon IP `<IP_ADDRESS>` is in CIDR network `<IP_ADDRESS>/32`
Mon IP `<IP_ADDRESS>` is in CIDR network `<IP_ADDRESS>/32`
Internal network (--cluster-network) has not been provided, OSD replication will default to the public_network
Adjusting default settings to suit single-host cluster...
Pulling container image utkarshhere/ceph:latest...
Ceph version: ceph version 17.2.5 (98318ae89f1a893a6ded3a640405cdbb33e08757) quincy (stable)
Extracting ceph user uid/gid from container image...
Creating initial keys...
Creating initial monmap...
Creating mon...
Waiting for mon to start...
Waiting for mon...
mon is available
Assimilating anything we can from ceph.conf...
Generating new minimal ceph.conf...
Restarting the monitor...
Setting mon public_network to <IP_ADDRESS>/24,<IP_ADDRESS>/32
Wrote config to /etc/ceph/ceph.conf
Wrote keyring to /etc/ceph/ceph.client.admin.keyring
Creating mgr...
Verifying port 9283 ...
Waiting for mgr to start...
Waiting for mgr...
mgr not available, waiting (1/15)...
mgr not available, waiting (2/15)...
mgr is available
Enabling cephadm module...
Waiting for the mgr to restart...
Waiting for mgr epoch 4...
mgr epoch 4 is available
Setting orchestrator backend to cephadm...
Generating ssh key...
Wrote public SSH key to /etc/ceph/ceph.pub
Adding key to root@localhost authorized_keys...
Adding host ubuntu-ceph-TVWA-6E50...
Deploying mon service with default placement...
Deploying mgr service with default placement...
Deploying crash service with default placement...
Deploying ceph-exporter service with default placement...


RemoteException When Running Tests via Xamarin Studio on OS X

If the Nunit Test Adapter Nuget package is added to a C# project file (.CSPROJ), then tests in the test assembly will fail with a "RemotingException: Unix transport error. See the following screenshot:

This  gist has some information from the Xamarin Studio logs about the issue.
Steps to Duplicate:

Create a new C# Class Library project in Visual Studio 2013 or Xamarin Studio 5.5.4 or higher.
Add the NUnit Test Adapter or NUnit Test Adapter with Framework package from NuGet.
Create a simple NUnit TestFixture and Test.
Run the test in Visual Studio - notice the test runs fine.
Save everything.
Open the solution and project in Xamarin Studio 5.5.4 (build 15).
Try to run the Nunit tests. You should receive the RemoteException shown in the previous screenshot.

A sample project is available for download here: https://dl.dropboxusercontent.com/u/24991646/XamarinBugzillaScreenshots/NUnitTestAdapterException.zip

Had a quick look at this. The problem seems to be that the assemblies the NUnitTestAdapter NuGet adds to the project as references all have Local Copy set to true if you install the NuGet package with Xamarin Studio. This is not the case if you add the NUnitTestAdapter NuGet package to the project with Visual Studio. This is because the NUnitTestAdapter NuGet package is using a PowerShell script (install.ps1) to set Copy Local to false for these assemblies. Xamarin Studio currently does not support running PowerShell scripts, at least not out of the box.
So right now we have a manual workaround of setting Local Copy to false in Xamarin Studio. Or you can remove the assembly references.
To test this I did the following:

In Xamarin Studio 5.8 (build 443) create a new NUnit library project.
Add the NUnitTestAdapter NuGet package (1.2).
Run the tests and see the "Unix transport error". The error in the Xamarin Studio's IDE log file is that NUnit.Core.RemoteTestRunner doesn't implement interface NUnit.Core.TestRunner. This then causes Xamarin Studio's test runner to crash.
Build - Clean to remove the assemblies from the output directory.
For the assemblies added by the NUnitTestAdapter NuGet package (nunit.core, nunit.core.interfaces, nunit.util and NUnit.VisualStudio.TestAdapter) set Local Copy to false.
Re-run the tests and now they work.

If the NUnitTestAdapter NuGet package is added to the project in Visual Studio first then you should be able to share the projects between Xamarin Studio and Visual Studio.
If the assemblies need to be referenced in the project and cannot just be in the tools directory another way to fix this is to might be to have the references defined in a separate MSBuild .targets file in the NuGet package. The Copy Local value could be set correctly in this separate MSBuild .targets file which would be imported into the project. That should work for both Xamarin Studio and Visual Studio, but not sure if it is worth doing since I think the common scenario would be to add the NUnitTestAdapter NuGet package to a project whilst using Visual Studio. In this case the project can be shared with Xamarin Studio and the test runner will work with both IDEs.

For inspiration on how to do this, you can look at the xunit implementation of the VS test adapter.

@OsirisTerje How do you want to deal with this issue? Document only?

If we can make the nuget package work as a solution package and not as a Project package it should also solve this issue, if I have understood the above correctly.  I Guess that is the best approach, so we could raise another issue on V3.0 to work as a solution package, and just add a reference to this.
The dll's don't really need to be referenced, so we could also check that route.


Unify the node and browser test suites

Problem
Currently we have two separate test suites for testing against node or browser. This is due to the need for the WASM to be initialised in the browser while not in node, otherwise the two suites are the same.
This causes a bit of pain ensuring that both suites are in sync.
Proposed solution
We should unify these two test suites and conditionally import the initialisation function and run it only when the test is running in a browser context.
Alternatives considered
No response
Additional context
No response
Submission Checklist

[ ] Once I hit submit, I will assign this issue to the Project Board with the appropriate tags.

This has been handled well enough by abstracting the data out into the shared folder. The different test dirs just holds the matchers, etc. now


Add Support for partial snapshots

We currently support Full Snapshots, but due to the number of options for partials they weren't implemented in the 1st pass.

This should probably be done after #109.

i think that i already implemented this in in #109 , or i'm misunderstanding the issue

Fixed in #127
merged in dev in commit fca215d405f3ab342b065d3c23ead9a652c7a466


Add a wiki: TTSCPedia

Link to origin wiki: https://mssam.fandom.com/
Link to destination (independent) wiki: https://wiki.therofl98.co/wiki/Main_Page

It looks like there's a large content disparity between the source Wiki and destination Wiki (2,444 vs. 240 content pages). Some of the missing pages do appear to have content on them as well and aren't just empty pages.

Declining for now, redirection likely not useful with the large content gap.


Replace remaining uses of <<< operator with .send

After merging https://github.com/apple/swift-driver/pull/1444 there are still a few uses of <<< left, which should be cleaned up. This fixes remaining deprecation warnings caused by the use of <<<.

@swift-ci test

@swift-ci test macos

@swift-ci test macos

@swift-ci test windows

@swift-ci test linux

@swift-ci test

@swift-ci test windows


Incorrect description for constructor StreamController.broadcast()

The description for constructor StreamController.broadcast() reads:

A controller where stream can be listened to more than once.
The Stream returned by stream is a broadcast stream. It can be listened to more than once.
A Stream should be inert until a subscriber starts listening on it (using the onListen callback to start producing events). Streams should not leak resources (like websockets) when no user ever listens on the stream.
Broadcast streams do not buffer events when there is no listener.

I think highlighted sentences should be removed as not relevant to the subject. They look like requirements for implementation, while should be a description of implementation.

The same sentence is also present in description of StreamController constructor.

StreamController({
void onListen(),
void onPause(),
void onResume(),
dynamic onCancel(),
bool sync: false
})
A controller with a stream that supports only one single subscriber.
If sync is true, the returned stream controller is a SynchronousStreamController, and must be used with the care and attention necessary to not break the Stream contract. See Completer.sync for some explanations on when a synchronous dispatching can be used. If in doubt, keep the controller non-sync.
A Stream should be inert until a subscriber starts listening on it (using the onListen callback to start producing events). Streams should not leak resources (like websockets) when no user ever listens on the stream.
The controller buffers all incoming events until a subscriber is registered, but this feature should only be used in rare circumstances.

It is describing the behavior that a user of the stream expects - that not listening to the stream should be safe, no significant resources should be leaked by keeping the stream alive and not listening to it.
It is intended as a guide on how to use the stream controller (don't start doing anything before you get an onListen), so I think the paragraph, or something like it, belongs on the stream controller.
We might want to tweak the wording to make the intent more clear.

Please, correct the wording. A Stream should ... sounds very strange in the description of the concrete implementation.


information missing on project pages

@dnoesgaard notice that we are not showing all info on the project pages.
Is this a deliberate choice @burkeker? Any specific plans for showing it in a different way, or simply an oversight?
compare
https://demo.gbif-dev.org/project/living-atlases/gbif-spain
with
http://drupaledit.gbif.org/living-atlases/gbif-spain

Done as 5fcdc4d9e00fefd76bda8c26a25a9d326d03a27c.

This is still an issue it seems: https://github.com/gbif/portal-feedback/issues/71
could you look into it @burkeker

Related to previous version. Not relevant any more.


Add a title to works from the METS transformer

Follows https://github.com/wellcomecollection/catalogue-pipeline/pull/1836, for https://github.com/wellcomecollection/platform/issues/5242
We assume the presence of a title in a bunch of places, including the design of the works pages – and indeed, the title will always be set on the WorkData for a visible Work – but this isn't enforced by the type system. We have WorkData { title: Option[String] } is because we don't set a title in the METS transformer. The reason we don't set a title in the METS transformer is that it'll be blatted in the merger – but there is a title in the METS files (and I checked every METS file to verify – yes, it really is there).
This PR updates the transformer to set a title on every METS work.
It doesn't update the internal model to assume the use of title – if we do that, we can't deploy it into the current pipeline, because the existing METS works without a title will break. My plan is:

Get this patch reviewed, merged and deployed into the current pipeline
Reindex just the METS works, so every METS work in the pipeline has a title
Start enforcing a required title in internal model, and deploy that into the current pipeline

Maybe an extra verification step of "all works at all stages have titles set in the elatic index"

Yeah, good suggestion.

Although TBH this is looking like it's going to be a sufficiently invasive change that I might just reindex instead.


Write tests for frost-toggle _changeTarget()

Closed via: https://github.com/ciena-frost/ember-frost-core/pull/279


MQTT Server - Read Values with Client

When we push data to the mqtt server of the adapter, we can't read the values with a other mqtt client. Is there an a other option to use a external mqtt server?
Thanks

Can you please describe your topic in more detail ... ideally with an example?

In the adapter, there is a MQTT included, we have a lot of other mqtt clients where we want to push data also to io broker. there a other adapter can be use.
The request is, the included mqtt server can be open for other mqtt sensors or a external mqtt server can be configure, that we have all mqtt information in one mqtt server

This one could also be integrated in this one: https://github.com/ioBroker/ioBroker.sonoff/issues/69


Error during setup due to wrong database name in .env

Issue Summary

The database name given in the .env file does not match that given in the create database instructions
This results in the following error during setup:

  File "/Users/ddigges/github/open-source-library-data-collector/venv/lib/python3.6/site-packages/pymysql/err.py", line 107, in raise_mysql_exception
    raise errorclass(errno, errval)
sqlalchemy.exc.InternalError: (pymysql.err.InternalError) (1049, "Unknown database 'open-source-external-library-data'")

Steps to Reproduce

Follow the setup instructions at the Contributing guide.
When you try to run the app using python app.py, the following error will appear:

  File "/Users/ddigges/github/open-source-library-data-collector/venv/lib/python3.6/site-packages/pymysql/err.py", line 107, in raise_mysql_exception
    raise errorclass(errno, errval)
sqlalchemy.exc.InternalError: (pymysql.err.InternalError) (1049, "Unknown database 'open-source-external-library-data'")

This is confusing for first time contributors.
Technical details:

Python Version:  3.6.3
Browser/OS: Chrome/Mac
Database/Version: Ver 8.0.12 for macos10.13 on x86_64 (MySQL Community Server - GPL)

Fixed by #141


Made Directionality forego dependency tracking for better performance.

Customer money is spending 20ms of startup time doing bookkeeping for Directionality  This gets rid of that bookkeeping which makes updates faster, but Directionality should be changed infrequently.  On some platforms change in text direction actually forces reloading the process anyways (iOS).
Local testing of this change to the increase stock_build_iteration 2.5% (21ms).
I tried to make this solution more general but there was apprehension to change the public API and it seemed like other Widgets didn't have the same draw as Directionality.
Examples of widgets using Directionality.of:

Semantics.getTextDirection
createLocalImageConfiguration
Transform.updateRenderObject
FittedBox.createRenderObject

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

The widget test is legitimately broken.  I'm going to take this PR out of review until I fixed it.

It turns out that you cannot visit your children at the point in time when the update is happening.  It seems the Element tree is also being rebuilt, so managing the dependency links may be the only way we have to talk to them.  That's a bit odd that the Element tree is not static but dependencies are since a rebuilt widget could technically not depend on Directionality when it is rebuilt, but it will already have been notified.  That's probably technically a bug, but doesn't manifest in a meaningful way.

Okay, I fixed the tests.  I previously wasn't recursing into the children elements and I had to make updates get filtered based on the bookkeeping we are keeping lower in the tree to match existing behavior.  I had to make one tiny addition to the Element api.

Some questions I have about this approach - how well does this scale in the odd case where users override directionality? What is the cost of updating the widget tree if the directionality does change?
Finally, how does the performance improvement scale to less constrained devices? I'm assuming the number comes from a wembly?

Local testing of this change to the increase stock_build_iteration 2.5% (21ms).

What does this mean? Did it make the benchmark worse?

Local testing of this change to the increase stock_build_iteration 2.5% (21ms).

What does this mean? Did it make the benchmark worse?

Nope, an increase in performance, a decrease in time.  I wouldn't recommend a change that doesn't help haha.

What is the expected performance benefit of this change?

In the stock_build_iteration test, run locally on my iPhone SE, the build times have improved 2.5% (21ms).  I think that benchmark is pretty representative of many of our users UIs.  I have profiles of the startup time of customer: money on low end Android devices which show it taking up 20ms of startup time in this bookkeeping as well.  Details can be found on b/229707977.

I have profiles of the startup time of customer: money on low end Android devices which show it taking up 20ms of startup time in this bookkeeping as well. Details can be found on b/229707977.

Does this change translate to a 20ms faster startup time for customer:money then?

Ahh Iphone SE, not bad

Another follow up question: can we identify other inherited widgets that frequently don't change?

Some questions I have about this approach - how well does this scale in the odd case where users override directionality? What is the cost of updating the widget tree if the directionality does change?

We are switching our algorithm from:

insertion: average O(1) to O(1), worst case O(n) to O(1)
lookup: average O(1) to O(n), worst case O(n) to O(n)

Insertions are happening when we are building the widget tree, lookups happen when the widget decides its data has changed and needs to notify its dependencies.
So, the performance will be somewhere between where it was before the change and a hot reload.  It won't be as costly as a hot reload because we are still filtering updates to widgets that actually depend on the widget.  I think in practice Directionality is so ubiquitous that the delta won't be that much different.

Finally, how does the performance improvement scale to less constrained devices? I'm assuming the number comes from a wembly?

My local testing was done on a modern iPhone and the supporting evidence for the change was the profiles of low end android devices.  I suspect this to be a win across the board.

Another follow up question: can we identify other inherited widgets that frequently don't change?

I think there is an opportunity to.  The other widgets I identified as candidates are [MediaQuery] and [Theme].  I know less about those and how much they change.  Also, making them UbiquitiousInheritedWidgets would mean making the code visible across multiple files and we had some reservations on making this a public API so I avoided it since 20ms was a big enough win.

I have profiles of the startup time of customer: money on low end Android devices which show it taking up 20ms of startup time in this bookkeeping as well. Details can be found on b/229707977.

Does this change translate to a 20ms faster startup time for customer:money then?

I didn't run the test for customer:money since it is a bit involved.  I felt I had enough evidence in hand to know it would be beneficial and I'd get the exact results for them as part of their regular testing.


[Fix #10846] Fix a false negative for Style/DoubleNegation when there is a hash or an array at return location of method.

This PR fixes #10846.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Btw, it seems to me the fix addresses a false negative, not a false positive.

Oops, you're right. I've fixed the problems, thanks.

Thanks!


In timeline, filter out dates we don't use somehow

It'd be nice to have a list of dates from config settings, but not display ones that have a magical value (1/1/1970, null) or are before a certain date or such.

Mmm, it seems the Welcome.tsx view should use a component to display with class deadline or deadlineExpired

        Registration Open: <ConfigDate name='registrationOpen' format={MDY} />

yeah, I thought about that, but we regularly left the date visible without emphasizing that it had expires for NW.  I was intentional, but perhaps it's not really such a good pattern.  It would be pretty simple to change.

I'm noodling with something, basically added a tab to settings with prefix config.timelineDates. Label, type date. Pull it into configuration so I can grab it wherever I need. That bit I have working and flipping it to an aray, sort, filter, I already have showing up fine.
There are already two styles (deadline, deadlineExpired), so I figure on writing a component that takes a config date to show/not show the card with the "if you access this after X date, contact ContactEmail" and can contextually show that, and then take all the dates in timelineDates, optionally check at a equal to or before date to filter out (e.g. 1/1/1970), though you could just delete them in config, and optionally the two classes to use for past and current/future dates.
Then whack all of the bottom half of Welcome for that component.

Like we dropped a few dates last year, dropped another date this year. And it makes it easy to create a role for that tab, rather than the explicit ones in Config.
Oh, and I use the name of the code entry as the label, as the database doesn't care about spaces, and other formatting for the code column.


Modify Some Chinese translations

ℹ Computer information

PowerToys version:
PowerToy utility:
Language: CN

📝 Provide where the issue is / 📷 Screenshots
Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form
❌ Actual phrase(s)

index
Actual
EN

1
颜色选择器
Color Picker

2
键盘管理器
Keyboard Manager

3
PowerRename
PowerRename

4
快捷键指南
Shortcut Guide

5
选择模式
Choose a mode

6
启用了编辑器模式的 Color Picker
Color Picker with editor mode enabled

7
从屏幕中选择一种颜色，将格式化值复制到剪贴板，然后再复制到编辑器
Pick a color from the screen, copy formatted value to clipboard, then to the editor

8
仅 Color Picker
Color Picker only

9
从屏幕中选择一种颜色，然后将格式化值复制到剪贴板
Pick a color from the screen and copy formatted value to clipboard

10
激活行为
Activation behavior

11
检查是否有更新
Check for updates

12
行为
Behavior ( In PowerRename )

13
用于使用“搜索和替换”或正则表达式进行更高级的批量重命名的 Windows Shell 扩展。
A Windows Shell extension for more advanced bulk renaming using search and replace or regular expressions.

14
Shell 集成
Shell integration

15
在上下文菜单上显示图标
Show icon on context menu

16
只在扩展的上下文菜单中显示(Shift + 右键单击)
Appear only in extended context menu (Shift + Right-click)

17
当按下 Windows 键时，显示包含 Windows 快捷键的帮助叠加层。
Shows a help overlay with Windows shortcuts when the Windows key is pressed.

18
背景的不透明度
Opacity of background

19
需要按住多长时间(毫秒)才能显示
Press duration before showing (ms)

✔ Expected phrase(s)

index
Expected
EN

1
屏幕取色器
Color Picker

2
快捷键助手
Keyboard Manager

3
重命名助手
PowerRename

4
快捷键速查表
Shortcut Guide

5
主题模式
Choose a mode

6
取色并编辑
Color Picker with editor mode enabled

7
从屏幕中取色，复制到剪贴板，并打开编辑器
Pick a color from the screen, copy formatted value to clipboard, then to the editor

8
直接取色
Color Picker only

9
从屏幕中取色，复制到剪贴板
Pick a color from the screen and copy formatted value to clipboard

10
激活方式
Activation behavior

11
检查更新
Check for updates

12
扩展
Behavior ( In PowerRename )

13
更高效的批量文件重命名的工具，允许对关键词替换和正则表达式。
A Windows Shell extension for more advanced bulk renaming using search and replace or regular expressions.

14
右键菜单扩展
Shell integration

15
在右键菜单工具图标
Show icon on context menu

16
只在扩展右键菜单中显示(按住Shift + 右键单击)
Appear only in extended context menu (Shift + Right-click)

17
长按 Windows 键时，显示包含 Windows 快捷键的速查表。
Shows a help overlay with Windows shortcuts when the Windows key is pressed.

18
背景不透明度
Opacity of background

19
需要按住多长时间(毫秒)
Press duration before showing (ms)

ℹ Why is the current translation wrong
There is no right or wrong, it just feels more natural.
Incidentally,  hope the team can create a json file for the localized language and publish it. This is more convenient.

@XHXIAIEIN we can't due to how the localization team's system works.  Their system is deemed the source of truth for what has and has not been translated.
https://ceapex.visualstudio.com/CEINTL/_workitems/edit/355594

@XHXIAIEIN we can't due to how the localization team's system works.  Their system is deemed the source of truth for what has and has not been translated.
https://ceapex.visualstudio.com/CEINTL/_workitems/edit/355594

The translations corrections were sent for revision but we cannot create a way to allow for others to tweak the translations. The localization team has to approve it

The translations corrections were sent for revision but we cannot create a way to allow for others to tweak the translations. The localization team has to approve it

Ok cool

Ok cool

Great news!  This was resolved in the newly released 0.33 version of PowerToys!  Head to https://aka.ms/installpowertoys to grab latest.


Bugfix: fix non-working which method

Hi there,
A colleague uses NixOS and it's an ultra-minimalist declarative functional OS... I think :)
Anyway - we were pulling MailCatcher in as a dependency into the project, and it broke with a weird popen3 exception somewhere in the ruby stdlib.
Anyway - we tracked it down to the nixos setup, and also at the same time, I noticed a bug whereby the argument to #which() was not being used and a hardcoded string was sent instead.
I've fixed the command arg thing, and also handled the exception Errno::ENOENT exception within mailcatcher with a more useful error message.
Hope you like :) and thanks for an amazing tool. It's a daily part of our lives.

Sorry I missed this, but the underlying issue was fixed by a manual ruby implementation:
https://github.com/sj26/mailcatcher/blob/main/lib/mail_catcher.rb#L44-L48


[WIP] Add libraries

Possible to toggle libraries on and off and have them included in the page markup.
To do:

[x] Update validation configuration appropriately (e.g. $ should be an allowed global iff you have jQuery enabled)
[x] Persist library choices along with sources
[x] Fix partial-preview bug (which is even worse now)
[ ] Make toolbar look reasonable

Follow ups:

[ ] Add additional validators (e.g. bootlint) where appropriate
[ ] Switch languages appropriately (e.g. JSX if you’re using React)
[ ] “Conflict” warnings? (e.g. you probably shouldn’t use Angular and React together)

Looking good!

@kinduff thanks! I’d say the only blocker to merging it is in fact the way is it looking—I’m so bad at design. But hopefully will be able to get a reasonable layout together in the next couple of days.

We could integrate Bootstrap or use some CSS framework to set it up quickly. WDYT?

@kinduff I have always found that Bootstrap makes me slower actually, it’s just kind of confusing. but will give it a shot if I run into a wall with hand-rolling it


LEO LEO Vertical LEDs

With an external reset cable connected to VRESET/RESETIN1 jack U5 will be counting to 10 while not receiving an external reset trigger (since the plugged jack breaks the normalled SELVRESET signal).
The vertical LEDs light up going down 4 ticks, then stay unlit for another 6 ticks until U5 - CD4017 counts up to 10 to restart from 1 again.
U4 - CD4024 counts binary in powers of 2 so the mix CV outs appear to loop through ABCD / DCBA.
As a result the vertical LEDs will show the wrong steps after the 6 blind ticks.
One possible fix is grabbing the SELVRESET signal from the nc switching terminal of the reset jack and sum it with RESETIN1 through a 1K resistor at R6 on the control PCB.
That way both counters stay in line but still follow external resets.
I haven´t tested this with linked Leo Leo modules so I can´t tell if there would be an issue.

Nice fix.
I used a diode instead of the 1K.
There also needs to be a diode added to the 4017 . Anode on pin 10 (5th step, Q4), cathode on pin 15 (reset).
If we ignore the longer divisions which aren't used  the 4024 is always running 00 01 10 11 in a 4 step cycle even if it doesn't get a reset.
The 4017 gets a reset every time the 4024 does, so they only go out of sync if the 4024 goes back to zero without a reset. With the diode the 4017 can only go 12341234...so now it's locked to the 4024.
I've not tested with a second module, but logically that's a fix.

Oh yeah, that looks like an elegant solution!
The way I had it didn't work with linked units and for now I just fed a clock divider to the reset. Didn't think that was already built in ;) will give it a try next time I'm at my workshop.


Access token for Azure Valid only for 1 hour

Steps to reproduce
Hi @isaiahwilliams
I am looking for some guidance. We are using the secure model for partner center but with Azure.
To login to Azure we have the following code:
$azureTokenParameters = @{
            RefreshToken = [System.Runtime.InteropServices.Marshal]::PtrToStringAuto([System.Runtime.InteropServices.Marshal]::SecureStringToBSTR($azRefreshTokenSecure))
            Scopes = 'https://management.azure.com/user_impersonation'
            Credential = $azCredential
            ApplicationId = ''
            TenantId = ''
            ServicePrincipal = $true
            ErrorAction = 'Stop'
        }
        $azureToken = New-PartnerAccessToken @azureTokenParameters

The returned access token we use in Connect-AzAccount. The return access token is valid only for 1 hour though. I have tried doing this:
https://docs.microsoft.com/en-us/azure/active-directory/develop/active-directory-configurable-token-lifetimes#cmdlet-reference
The policy that is set is:
$policyDefinition = @('{
    "TokenLifetimePolicy":
        {
            "Version":1,
            "AccessTokenLifetime":"23:00:00"
        }
    }')

Setting AccessTokenLifetime to higher value specifically for the application and for the service principal of the application. Assigning the policy even to both of them does not work as I still get access token of 1 hour with the above command. Can you help us? We need to have access token for longer than one hour because we have workflows that run more than 1 hour. How to do this with the secure model for partner center?

What steps can reproduce the defect?
Please share the setup, commandline for vstest.console, sample project, target
framework etc.

Expected behavior

Share the expected output

Actual behavior

What is the behavior observed?

Diagnostic logs

Please share test platform diagnostics logs.
The logs may contain test assembly paths, kindly review and mask those before sharing.

Environment

Please share additional details about your environment.
Version

Hi @isaiahwilliams have you managed to look into this. I am also trying to get clarification from people who support Azure AD but seems they are not so familiar with this authentication flow.
https://github.com/MicrosoftDocs/azure-docs/issues/47224

#please-close this is being handled in the other issue. It's critical that the resource have the policy applied to it, not the client.

@isaiahwilliams Yes, after latest findings I figured out that is the way. Any way you can help with convincing the Az PS team to do this? As this is for partner scenarios I think it might be good that someone can convince them that more and more partners will need this especially when security defaults become mandatory. I have commented on this issue:
https://github.com/Azure/azure-powershell/issues/6585
Either the commands needs to pick the latest token or they should allow refresh token login. The latter I think it is better as it will provide better experience.

@slavizh both the Azure and Partner Center modules use a similar strategy for authentication, which means it would not be too difficult add this ability. I know there has been discussions about this topic within the Azure PowerShell team, but I am not aware of any immediate plans. Fortunately that module is open source, so if the team behind module is not planning to add this feature hopefully the community can. Let me see if there is anything additional I can track down.

@isaiahwilliams I was asking more to reach to that team as representative for the of the partners. Last time when I had to convince them to retain non-interactive user authentication I had to battle with them for weeks until they can understand how critical is this for CSP. It is important that there is some collaboration on such initiatives inside Microsoft as my experience shows when I try to reach to multiple groups to solve issue turns into failure unfortunately.

There is active communications happening on this topic through a number of means. Since this has been taken offline I am going to close this issue.


Add a way to exit(1) if global coverage does not satisfies a certain %

Jest have this feature http://facebook.github.io/jest/docs/configuration.html#coveragethreshold-object and it would be nice to have a similar CLI arg to fail on CI :)

its current behavior is to exit(2) when the total coverage is lower than the threshold (the default is 80% and it can be tweaked using --threshold command line option).
There is no command line option to disable this behavior, mostly because checking the total coverage is its main goal (besides "generating some nice reports to be able to more easily fix the flowtype coverage issues" :-)), and unfortunately there is no support for detecting an fail on a negative trend of the total coverage across two versions... yet (but I'd really like to add this feature asap).
I've added some integration tests to catch any future regression in this behavior (and I will expand the integration tests further in follow-ups PRs) in #60 (the current behavior is described by the commit jest snapshots: https://github.com/rpl/flow-coverage-report/pull/60/files#diff-ea5f4745f50d2aba384d45c0092557e2)
@MoOx let me know if an exit(2) on coverage failure is still an issue in your use case.

unfortunately there is no support for detecting an fail on a negative trend of the total coverage across two versions... yet (but I'd really like to add this feature asap).

Do you mean something like https://github.com/Koleok/jest-coverage-ratchet? I use that and it's excellent, may provide some inspiration!


why  ddl2cpp not support primary key?

why not ddl2cpp not support primary key

change file ddl2cpp， line 262  "+ pp.Group(pp.delimitedList(ddlColumn | pp.Suppress(ddlConstraint))).setResultsName(" to
"+ pp.Group(pp.delimitedList(pp.Suppress(ddlConstraint) | ddlColumn)).setResultsName("
it  work fine for me.


Support Sketch 45 plugin update system

Hi there!
This is a quick note to remind you that Sketch 45 will include a plugin update system.
It would be awesome if you could add support to it (it's really easy!) before Sketch 45 comes out. Ideally, you should release an update for your plugin while we're still in Sketch 44, so your users will have a nice & easy experience when 45 is released.
For more details, please check http://sketchplugins.com/d/229-updating-plugins and http://developer.sketchapp.com/introduction/updating-plugins/
Thanks in advance!

👏

However, it seems updating the plugin installs an old version (which is not compatible with Sketch 45). If I download the package linked in the appcast.xml file manually, I get the correct version. I have clearly done something wrong, but I have no clue, what!

Try removing the ~/Library/Application Support/com.bohemiancoding.sketch3/PluginsWarehouse folder and relaunching Sketch

Ah, thanks!


Spring command not going trough its socket?

Maybe I am understanding it wrong. But isn't the spring command supposed to go trough the spring socket. Now it needs to escape to a shell, boot ruby, to finally pass trough the spring socket. Seems a bit unnecessary in terms of milliseconds or am I wrong?
Thanks a lot
The runner code:
https://github.com/guard/guard-minitest/blob/master/lib/guard/minitest/runner.rb

And how the spring client talks to spring
https://github.com/rails/spring/blob/master/lib/spring/client/run.rb#L26

Aaaaaan here is the guard free patch
task :yolo => :environment do
  Thread::abort_on_exception = true
  listener = Listen.to("test/") do |modified, added, removed|
    #total = modified.concat(added).concat(removed).uniq
    begin
      Spring::Client::run(["rails","test"])
    rescue SystemExit
      puts "Lol rly?"
    end
  end
  listener.start
  puts "Watching for test changes"
  sleep
end


Invalid train.sh script

Please use this template for the train.sh. Your script is setting not any environment variables.
https://github.com/JDACS4C-IMPROVE/Singularity/blob/develop/src/templates/train.sh

Updated train.sh, last fix https://github.com/JDACS4C-IMPROVE/TGSA/commit/ab109b841b21cb12f2f64d24f27a287113451d2f


Documentation improved

Hi, @imharjyotbagga
Please add some description in your PR.
Also, resolve the conflicts in your PR


Unnecessary Row Added After Opening File in Excel

Description
I am trying to create an excel file with dropdown using excelize. At first, I am trying to use validation.SetDropList(data) to create the dropdown but i realize that the data is too many and causing the excel file to be corrupted. So i decided to use validation.SetSqrefDropList(sqrefDropList, true), set the data on other cells and hide the cell that contain the data using xlsx.SetRowVisible(sheet, hcell, vcell, false). But something strange happen. After opening and saving the file, the excel file now contain empty rows that can be read and causing an error in my program because there is an extra rows that are not hidden but contains empty value.

Steps to reproduce the issue:

Set a dropdown using validation.SetSqrefDropList(sqrefDropList, true)
Hide the row that contain the data for dropdown

for index, val := range(valSplit) {
     val = strings.Replace(val, "\"", "\"\"", -1)
     file.SetCellValue(sheetName.Slug, collumnOption + strconv.Itoa(collumnRow + index), val)
     file.SetRowVisible(sheetName.Slug, (collumnRow - 1 + index), false)
}
sqrefDropList := fmt.Sprintf("$%s$%d:$%s$%d", collumnOption, collumnRow, collumnOption, (collumnRow+len(valSplit)-1))
collumnRow += len(valSplit)
validation.SetSqrefDropList(sqrefDropList, true)

Open the file using Microsft Excel and save it.
Check the XML of the sheet

Describe the results you received:
Rows properly hidden and excel file not containing any extra empty rows
<sheetData>
<row r="1" spans="1:18" ht="20.100000000000001" customHeight="1"></row>
<row r="2" spans="1:18" ht="20.100000000000001" customHeight="1"></row>
<row r="3"spans="1:18"></row>
<row r="4" spans="1:18"></row>
<row r="600" spans="183:183"></row> // this row contain the data
<row r="601" spans="183:183"></row>
<row r="602" spans="183:183"></row>
</sheetData>

Describe the results you expected:
Rows being hidden but there's an extra empty rows that can be read.
<sheetData>
<row r="1" spans="1:18" ht="20.100000000000001" customHeight="1"></row>
<row r="2" spans="1:18" ht="20.100000000000001" customHeight="1"></row>
<row r="3" spans="1:18" ht="21.95" customHeight="1"></row>
<row r="4" spans="1:18" ht="21.95" customHeight="1"></row>
<row r="5" spans="1:18" ht="21.95" customHeight="1"/>  // extra empty rows
<row r="6" spans="1:18" ht="21.95" customHeight="1"/>
<row r="7" spans="1:18" ht="21.95" customHeight="1"/>
<row r="8" spans="1:18" ht="21.95" customHeight="1"/>
<row r="9" spans="1:18" ht="21.95" customHeight="1"/>
<row r="10" spans="1:18" ht="21.95" customHeight="1"/>
<row r="11" spans="1:18" ht="21.95" customHeight="1"/>
<row r="12" spans="1:18" ht="21.95" customHeight="1"/>
<row r="13" spans="1:18" ht="21.95" customHeight="1"/>
<row r="14" spans="1:18" ht="21.95" customHeight="1"/>
<row r="15" spans="1:18" ht="21.95" customHeight="1"/>
<row r="16" spans="1:18" ht="21.95" customHeight="1"/>
</sheetData>

Output of go version:
go version go1.10 linux/amd64

Environment details (OS, Microsoft Excel™ version, physical, etc.):
Microsoft Excel version : 2013

I really hope you can tell me what is the problem, whether it is because of the excelize or not. Thank you.

Hi @xuri, thank you for answering my question but i am still not sure about why the problem only happen when i tried to hide the rows. Can you elaborate me more about the possible cause of that? because there's no extra cell added when i did not try to hide the cell.


Report failures in docs.sh

We should actually know when doc uploads fail...

Closes https://github.com/flutter/flutter/issues/44452, in principle...

cc @gspencergoog


Move to new ListTrade RPC

This replaces the old operator service's ListSwaps RPC with the new ListTrades one.
In accordance with the proto spec basically, new types have been added:

SwapInfo resuming info of the swap request message
SwapFailInfo resuming info of the swap fail message, if the trade failed
TradeInfo resuming info of a trade: besides those above, it contains:

the trade's id
the status including if the trade failed
the market (asset pair) it refers and the relative fee
both base and quote prices
in case the trade is settled, the Esplora URL of the tx including blinders to reveal the outputs
all timestamps of request, accept, complete, settle and exipry events in unix and UTC format

BONUS:

add the timestamp when completing a trade by calling trade.Complete(). This means that all the trades done 'till now by daemons outthere never recorded the completion timestamp.
fix wallet.ExtractBlindingDataFromTx in order to accept out blind keys and return also out blinding data besides those of the inputs.
add a ExtractBlindingData func to the application.TransactionManager helper.

This closes #292.
Please @tiero review this.

Done.


Created a molecule2 command

We will likely be dealing with subcommands differently in v2.
Due to clicks add_command usage, we cannot side load different
command usage based on a CLI flag (inception).  Instead we created
a molecule2 command to experiment with.

  py27-ansible19-unit: commands succeeded
  py27-ansible20-unit: commands succeeded
  py27-ansible21-unit: commands succeeded
  congratulations :)


Run selection in interactive window doesn't works using shortcut keys

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.58.1
OS Version: Windows 11 (10.0.22000)

Run selection in interactive window doesn't works using any shortcut keys (the default shortcut is similar to run in python terminal, so when I use that, It runs the code in terminal. I have changed it to a unique shortcut, but still doesn't work)

/extPython


Jar file

Greetings,
I see you updated to 1.12, but did not provide a jar file. You should definitely consider doing this, as I had to compile this entirely myself :/

Hey, sorry about that. I should probably make it more clear on the github that this project is abandoned. Anybody wishing to compile 1.12 themselves or distribute compiled versions of this library is free to do so, so long as they comply with the terms of the MIT license.


[New game]: SUSHI KITCHEN 👾

🎮 Game Request
Game logic and basic description

Command a fleet of cook to craft sushi!
Buy new machines, hired more crew, expand your kitchen.
This is a management game.

Point down the features
Game points
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @0xabdulkhalid !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @0xabdulkhalid ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


pulp_sync plugin should remove registry from push_conf

The pulp_sync plugin knows which docker registry it syncs from, so it should remove that from the push_conf.
This would make koji_import and store_metadata simpler and reduce code duplication.

Pulp integrations has been removed from atomic-reactor. Closing this.


combinable with vllm

Great work. How can I combine the eagle with vllm？

Thank you for your interest. We will support vLLM in our subsequent updates.

@Liyuhui-12 Great Job~! looking forward to it. May I know what's the timeline to expect EAGLE to be integrated into vLLM?

Thank you for your interest. We are in the process of writing the paper on EAGLE, and after completing the paper, we will integrate vLLM and others.


Feature/upgrade deps

This upgrade the dependencies and also generalizes the ga4ghhelper

Coverage decreased (-0.1%) to 79.745% when pulling bcc3b289918dfbb3f15c47082872219cc0cd7dbe on feature/upgradeDeps into a58e24e6d175c55b60f27492896fc54b1dcf15ab on develop.

@garyluu FYI, the threshold for coveralls is configurable. Probably worthwhile to change it to 0.5% or 1% or so to avoid random changes


[🐛] Cannot pick video with launchImageLibrary mediaType: 'mixed'  on Android

Description
"react-native-image-picker": "^4.1.1",
"react-native": "0.64.1",
Platform: Android,
AndroidVersion: 11,
DeviceBrand: Xiaomi.
I'm using launchImageLibrary with option mediaType: 'mixed' to pick video and images, on IOS it's ok but on my android (Xiaomi) it didn't show any video in the library
Thanks

Are you able to reproduce this with the example app?


SWARM-1909: Swagger: Default the context root to JAX-RS context root

Motivation
Based on discussions in SWARM-1667, it made sense to set the defaults for Swagger resources to be made available under the JAX-RS Application context path.
Modifications

Look at the IndexView of the archive and determine the JAX-RS Application path.
If the developer has not defined "swarm.deployment.*.swagger.root", then use the above path.
Updated test case.

Result
Verified the functionality.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you created a JIRA and used it in the commit message?
[x] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[x] Have you built the project locally prior to submission with mvn clean install?

retest me please.

retest this please

What's up with the CI? :)

Retest this please

Dunno. We need CI for our CI.

Retest this please

Was the Jenkins instance recently upgraded to the latest LTS?

Caused by: java.lang.UnsupportedOperationException: Refusing to marshal org.kohsuke.github.GHUser for security reasons; see https://jenkins.io/redirect/class-filter/

Might be related to JENKINS-48950. Can the maintainers of the CI please have a look at the issue?

Indeed that seems to be the case. We'll likely have to update a couple of Jenkins plugins.

retest this please

I would prefer that this be merged after #844 & #849 ... this PR will need a rebase after those get merged.

@kenfinnigan I have updated the content of SWARM-1909 to reflect the actual intent of this change. Can you please review it and let me know if it makes sense to have this feature?


[CodeCompletion][Sema] Add fix to treat empty or single-element array literals as dictionaries when used as such

In the single-element case, the element is treated as the dictionary's key.
    func takesDict(_ x: [Int: String]) {}

    takesDict([]) // diagnose with fixit to add missing ':'
    takesDict([1]) // diagnose with fixit to add missing ': <#value#>'
    takesDict([foo.<complete>]) // prioritise Int members in completion results -
                                // the user just hasn't written the value yet.

The above previously failed with a generic mismatch error in normal type checking (due to the literal being parsed as an array literal) and as a result code completion could not pick up the expected type from its context.

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test Windows

Build failed
Swift Test OS X Platform
Git Sha - d74d470eb9c3027dffd383e4b5a290f5795b7ef6

@swift-ci please test OS X Platform

@swift-ci please test


Update views.py

Change "" to None for handling empty dependent_fields.

Hi @codingjoe.
Your welcome. Thank you for the comments and your support.
I tried to check the suggestion but it did not result the expected behavior.

Hi @Iran-110,
I see you points. However, I still don't fully understand the use case. A test would certainly go a long way here as well.
Best,
Joe

Hi @codingjoe,
This commit is about the case that in the child widget, we set the dependency_fields. But, they can be null.
In the correct situation, if the user sets the parent field(s) to empty, we must see no options in the list of the child widget.
But now, all options are on the table!
Best regards,
@Iran-110


Feature: construct fractions without simplification

Fixes: #11

disabling simplification for fraction construction
minor bug fix in gcd about handling 0s correctly

Codecov Report

Merging #12 (16f609d) into master (1f6f224) will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #12      +/-   ##
==========================================
+ Coverage   91.51%   91.59%   +0.07%
==========================================
  Files          36       36
  Lines         554      559       +5
  Branches       42       42
==========================================
+ Hits          507      512       +5
  Misses         37       37
  Partials       10       10

Impacted Files
Coverage Δ

src/data/fractionNumber/PLFractionNumber.ts
100.00% <100.00%> (ø)

src/data/fractionNumber/fractionNumberFn.ts
100.00% <100.00%> (ø)

src/utils/math.ts
100.00% <100.00%> (ø)

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 1f6f224...16f609d. Read the comment docs.

@vektor8891 see my comment: #11

@maxinteger I will close the PR as agreed. I've submitted PR #16 to fix the bug in gcd.


Use Multiyaml reader everywhere

Currently the baseline tests and Discogapic generator use a different config parsing pipeline than the actual  Gapic pipeline, which means that baseline tests don't actually show what would be produced by the Gapic generator.

@hzyi-google Can you review the baseline changes? Can you confirm that the new baseline changes are what would be expected given the existing gapic config (on_success config)
If this seems right, I'll send this to another person to review the implementation logic changes.

LGTM. This is indeed what we wanted to achieve. Thanks for doing this.

PTAL @eoogbe


Invalid characters on folder names

I've encountered this on two ocasions:
Error: Redacted API returned an error while uploading a torrent: {"status":"failure","error":"The file or folder in the torrent that has the name 'Big | Brave - Nature Morte [2023] (CD - MP3 - 320)' contains the forbidden character '|'. Please recheck all files and folders in the directory and rename them as necessary and recreate the torrent.<br \/><br \/>\nNote: The complete list of characters that are disallowed are shown below:<br \/>\n\t\t : ? < > \\ * | \" \/\/ (Leading Space)"}
Error: Redacted API returned an error while uploading a torrent: {"status":"failure","error":"The file or folder in the torrent that has the name 'Therapy? - Hard Cold Fire [2023] (CD - MP3 - V0)' contains the forbidden character '?'. Please recheck all files and folders in the directory and rename them as necessary and recreate the torrent.<br \/><br \/>\nNote: The complete list of characters that are disallowed are shown below:<br \/>\n\t\t : ? < > \\ * | \" \/\/ (Leading Space)"}
Can you add for it to automatically replace those either just remove them completely or add an underscore (_) like they do on site.
Thank you.

i'll add an option to ignore such characters, thanks for letting me know :)

Fixed in #6


The order of the splits in sciq is not correct

https://datasets-server.huggingface.co/splits?dataset=sciq
gives test / train / validation (alphabetical)
while we expect train / validation / test
See https://huggingface.co/datasets/sciq/blob/main/sciq.py#L67

OK, thanks!
For reference, it's here: https://huggingface.co/datasets/sciq/discussions/2


Make args and argTypes available in meta.json

Is your feature request related to a problem? Please describe.
I find the Controls add-on very useful and am now exploring the possibility to generate snapshots for all available arg combinations (e.g., Button component with three variants + disabled).
This is an example of my current Button story:
import type { Story, StoryDefault } from "@ladle/react";
import { Button, ButtonProps } from "../index";

export default {
  title: "Core / Button",
} satisfies StoryDefault;

export const Index: Story<{
  disabled: boolean;
  minimal: boolean;
  variant: ButtonProps["variant"];
}> = (props) => <Button {...props}>Button</Button>;

Index.args = {
  variant: undefined,
  disabled: false,
  minimal: false,
};

Index.argTypes = {
  variant: {
    options: ["default", "primary", "danger"],
    control: { type: "inline-radio" },
  }
};

Describe the solution you'd like
Ideally, the args and argTypes of each story would be available in meta.json. This would allow me to create snapshots for every combination automatically by passing the control via query parameters (e.g., ?arg-variant=primary&arg-disabled=true).
{
  "about": {
    "homepage": "https://www.ladle.dev",
    "github": "https://github.com/tajo/ladle",
    "version": 1
  },
  "stories": {
    "core--button--index": {
      "name": "Index",
      "levels": ["Core", "Button"],
      "locStart": 8,
      "locEnd": 12,
      "filePath": "src/button/__stories__/index.stories.tsx",
      "meta": {},
      "args": {
        "variant": null,
        "disabled": false,
        "minimal": false
      },
      "argTypes": {
        "variant": {
          "options": ["default", "primary", "danger"],
          "control": { "type": "inline-radio" }
        }
      }
    }
  }
}

Describe alternatives you've considered

Duplicate the available arg information and make it available via the meta object
Add all necessary variants/combinations to each story
Add stories for all relevant variants/combinations

Good question, I would like to use something like this too. The issue is that args / argTypes are not statically analyzable and they need to be executed (if we want to stay Storybook compatible). So you can't just simply export them through meta.json. There was a similar discussion about this here: https://github.com/tajo/ladle/issues/127#issuecomment-1199902925
I see two solutions:

propose a new (different) format for controls that will be strictly static
add some argStatic mode that would attempt to statically analyze args/argTypes and throw an error if not possible Your example would work just fine:

Index.argTypes = {
  variant: {
    options: ["default", "primary", "danger"],
    control: { type: "inline-radio" },
  }
};

however, this would fail
const variant = {
  options: ["default", "primary", "danger"],
  control: { type: "inline-radio" },
}
Index.argTypes = { variant };

Does it make sense?

Thanks for the link @tajo–I always assumed args/argTypes are statically analyzable as well and it would be easy to export them. Are there any use cases where those are not strictly static?
Personally, I think the CSF compatibility of Ladle is a great feature and it is worth the wait until a mutual standard has been set.

Are there any use cases where those are not strictly static?

They could be static, it's all simple arrays/objects but it's not been enforced in Storybook ecosystem (same with story titles, which Ladle actually strictly enforces and diverges from Storybook on that since it's critical for code-splitting and other features). Typically, devs assign args into a variable so they can spread it out over multiple stories.

Closing, since this is not technically feasible. Anyway, we might have some better & analyzable controls API in the future.


Input Club Kira Keyboard support

I'd love to see some support put out for the Input Club Kira
https://kiibohd.github.io/wiki/#/Keyboards/Kira

I'm the main dev on Input Club keyboards. Currently working on some host-side drivers that should make adding keyboard support rather easy.

Awesome to hear this.
You should be able to get a general idea of how the integrations work from the existing ones here but if you need any help or details feel free to ask.

rad - thanks for the input guys


Custom Event Handling

Because there doesn't seem to be a robust way to do this, it is probably easiest to call Framework::ready() and Framework::interaction_create() manually in the custom event handler


[BEAM-7116] Remove use of KV in Schema transforms

Beam's KV type has no schema and due to special casing of KvCoder in Beam it is difficult to give it one. Here we modify the Beam schema transforms that return PCollection to instead return PCollection where the Row contains key and value fields. This is possible now that we support large iterables in schemas.

R: @TheNeuralBit

run sql postcommit

friendly ping

Run Java PreCommit

@reuvenlax @TheNeuralBit  AvroSchemaTest.testAvroPipelineGroupBy started failing after merging this commit. Do you think this PR is related?
https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Spark/6002/
org.apache.beam.sdk.Pipeline$PipelineExecutionException: java.lang.IllegalStateException: ValueIterator can't be iterated more than once,otherwise there could be data lost

Yeah this PR must be the cause. It's not clear to me why this only seems to be a problem with the Spark runner.. I'll file a jira and exclude the test from Spark for now.

@TheNeuralBit Thank you for confirmation.

#10358 should fix Spark ValidatesRunner

The short answer is that the failure is caused by a limitation of the Spark
runner - the Beam model allows the Iterable passed into a GroupByKey to
reiterated, but Spark is unable to support that.
The longer answer is that this is exposing a performance bug in the Group
transform. We were accidentally walking over the iterator, which we should
not be doing here. I'll send a PR to fix it.
On Wed, Dec 11, 2019 at 3:43 PM Brian Hulette<EMAIL_ADDRESS>wrote:

#10358 https://github.com/apache/beam/pull/10358 should fix Spark
ValidatesRunner
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/10151?email_source=notifications&email_token=AFAYJVMNEC7QSLVPSGN6B3TQYF3ITA5CNFSM4JO5WLU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGU6Q5Y#issuecomment-564783223,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFAYJVNBKIBVTN6JRD7YJZLQYF3ITANCNFSM4JO5WLUQ
.


Install ccache on all platforms, not just linux

r? @Manishearth

@bors-servo r+

:pushpin: Commit 9609d24 has been approved by Manishearth

:hourglass: Testing commit 9609d24 with merge 6230828...

:sunny: Test successful - travis


Make it wider

Hello,
I can't find anything about making the card on the Hass Dasbhoard wider (like type: card / type: vertical-stack).
Do I miss something or is this not implemented (yet)?

The card just expands to fill whatever space is available, so it really depends on what you put the card into. For example, to have the card fill the whole dashboard you could use a panel dashboard.
What are you trying to do?

I'm trying to make the panel bigger, I would like to give it a two columns width.

I'm trying to make the panel bigger, I would like to give it a two columns width.

But yet not the full width? Maybe something like layout card can help, I'm not sure. But this is a general Home Assistant question rather than anything specific to this card -- your question is really "How can I get any HA card to span 2 columns". Once you answer that, the card will just work within whatever space it is given -- the card cannot control the overall dashboard layout.

I don't think there is anything the card can do here, since it simply expands into whatever available space Lovelace gives it and it cannot really control this. Please re-open if you think there's something the card can/should do.


pyros_test: 0.0.4-0 in 'jade/distribution.yaml' [bloom]

Increasing version of package(s) in repository pyros_test to 0.0.4-0:

upstream repository: https://github.com/asmodehn/pyros-test.git
release repository: https://github.com/asmodehn/pyros-test-release.git
distro file: jade/distribution.yaml
bloom version: 0.5.21
previous version for package: null

pyros_test
* now travis checking both indigo and jade
* Merge pull request #2 <https://github.com/asmodehn/pyros-test/issues/2> from asmodehn/package_v2
  Package v2
* README fix
* package description fix
* package v2 wiht minimum required version for dependencies
* Contributors: AlexV, alexv

This should be fixed now, but I cannot relaunch the travis job.

Retriggering Travis...


fix backward compatibility for GenerationMode import

What does this PR do?
fix import GenerationMode for transformers<4.39

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@echarlaix could you please take a look?


New-PNPSite with PreferredDataLocation Error using Azure Automate- [BUG]

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

Are you using Invoke-PnPSiteTemplate or Get-PnPSiteTemplate? The issue is most likely related to the Provisioning Engine. The Provisioning engine is not located in the PowerShell repo. Please report the issue here: https://github.com/pnp/pnpframework/issues.
Is the issue related to the cmdlet itself, its parameters, the syntax, or do you suspect it is the code of the cmdlet that is causing the issue? Then please continue reporting the issue in this repo.
If you think that the functionality might be related to the underlying libraries that the cmdlet is calling (We realize that might be difficult to determine), please first double check the code of the cmdlet, which can be found here: https://github.com/pnp/powershell/tree/master/src/Commands. If related to the cmdlet, continue reporting the issue here, otherwise report the issue at https://github.com/pnp/pnpframework/issues

Reporting an Issue or Missing Feature
Error when executing New-PNPSite with -PreferredDataLocation GBR
Expected behavior
Expecting new site to be created within the Geo Location
Actual behavior
Creating new site
Creating site in Norway Geo Locaton (NOR)
Provisioning process has FAILED
New-PnPSite : {"error":{"code":"-2147024809, System.ArgumentException","message":"PreferredDataLocation"}}
At line:110 char:16

...  $newsite = New-PnPSite -Type TeamSite -Title $siteTitle -Description ...

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

CategoryInfo          : InvalidOperation: (:) [New-PnPSite], PSInvalidOperationException
FullyQualifiedErrorId : InvalidOperation,PnP.PowerShell.Commands.NewSite

Steps to reproduce behavior
Please include complete script or code samples in-line or linked from gists
$siteTitle = "Test Site"
$siteAlias = "TestSite"
$owner.Email =<EMAIL_ADDRESS>$GeoLocation = "NOR"
$newsite = New-PnPSite -Type TeamSite -Title $siteTitle -Alias $siteAlias -Owners $owner.Email -PreferredDataLocation $GeoLocation
What is the version of the Cmdlet module you are running?
PnP.PowerShell 1.12.0
Which operating system/environment are you running PnP PowerShell on?

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

is this issue resolved in the latest version ?

@cystatto - Can you please update to the latest release 2.1.x however you will need PS 7.2 or later to benefit from all the bug fixes and improvements.
PS5 support has been dropped in the new release.
Feel free to reopen if the issue persists. Thanks!


rustc, cargo: always build on big-sur

Motivation for this change
Things done

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

[ ] NixOS
[ ] 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.

I agree, but it's quite onerous to override them. I tried for quite a while.

Ah okay, dang

@evanrelf For a little more context on why it's difficult, it's because the rustc/cargo definitions are part of the rustPlatform attrs which I just found very hard to effectively override.

Also, once we upgrade to a newer Nixpkgs I presume that we can drop the Big Sur mandatory feature since I believe newer versions of Nixpkgs don't require bootstrapping things on Big Sur

Also, once we upgrade to a newer Nixpkgs I presume that we can drop the Big Sur mandatory feature since I believe newer versions of Nixpkgs don't require bootstrapping things on Big Sur

I agree with that, too. I think if we are careful when forward-porting patches from awake-21.05 that should be pretty easy to spot? Does it deserve some commentary?

Ok thank you!


Fresh install from C3 using stripe-node throws: Uncaught ReferenceError: process is not defined

Reproduction
https://github.com/johnpaulkiser/stripe-cf-test
System Info
System:
    OS: macOS 13.2.1
    CPU: (8) arm64 Apple M1
    Memory: 2.09 GB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 21.2.0 - ~/.nvm/versions/node/v21.2.0/bin/node
    Yarn: 1.22.21 - ~/.nvm/versions/node/v21.2.0/bin/yarn
    npm: 10.2.3 - ~/.nvm/versions/node/v21.2.0/bin/npm
    pnpm: 8.14.0 - ~/.nvm/versions/node/v21.2.0/bin/pnpm
  Browsers:
    Chrome: 121.0.6167.160
    Safari: 16.3
  npmPackages:
    @remix-run/cloudflare: ^2.6.0 => 2.6.0
    @remix-run/cloudflare-pages: ^2.6.0 => 2.6.0
    @remix-run/css-bundle: ^2.6.0 => 2.6.0
    @remix-run/dev: ^2.6.0 => 2.6.0
    @remix-run/react: ^2.6.0 => 2.6.0
    stripe: ^14.16.0 => 14.16.0

Used Package Manager
npm
Expected Behavior
dev server to start
Actual Behavior
[ERROR] service core:user:worker: Uncaught ReferenceError: process is not defined
at null.<anonymous> (sp0mwi1d06.js:30044:5) in node_modules/util/util.js
at null.<anonymous> (sp0mwi1d06.js:12:50)
at null.<anonymous> (sp0mwi1d06.js:30395:22) in node_modules/object-inspect/util.inspect.js
at null.<anonymous> (sp0mwi1d06.js:12:50)
at null.<anonymous> (sp0mwi1d06.js:30416:23) in node_modules/object-inspect/index.js
at null.<anonymous> (sp0mwi1d06.js:12:50)
at null.<anonymous> (sp0mwi1d06.js:30757:92) in node_modules/side-channel/index.js
at null.<anonymous> (sp0mwi1d06.js:12:50)
at null.<anonymous> (sp0mwi1d06.js:30957:26) in node_modules/qs/lib/stringify.js
at null.<anonymous> (sp0mwi1d06.js:12:50)

Seems similar to this?
https://github.com/remix-run/remix/issues/8448

Not sure if this repo is "responsible" however the discussion here leads me to think this is the best place to open it.
https://www.answeroverflow.com/m/1158324050737774592
I've tested a similar setup with plain workers project & next.js CF pages starter, both of which work fine.

Remix 2 now longer automatically installs node polyfills for you so you need to opt-into them in the remix config:
/** @type {import('@remix-run/dev').AppConfig} */
export default {
  ...
  serverNodeBuiltinsPolyfill: {
    modules: {},
    globals: {
      process: true,
    },
  },
};

I cloned your repo and added that, and then got an error from cloudflare:
✨ Compiled Worker successfully
 ⛅ wrangler 3.28.2
-------------------
[wrangler:inf] Ready on http://localhost:8788

▲ [WARNING] The package "node:process" wasn't found on the file system but is built into node.

  Your Worker may throw errors at runtime unless you enable the "nodejs_compat" compatibility flag. Refer to https://developers.cloudflare.com/workers/runtime-apis/nodejs/ for more details

Based on those docs I updated the npm run start script to be:
wrangler pages dev --compatibility-date=2023-06-21 ./public --compatibility-flags="nodejs_compat"

And then it it built without errors.  Can you confirm if that resolves your issue?

Yup that works! Opting into this polyfill in the remix config also works, is there any advantage to one or the other?

Opting into the process polyfill in the remix config was not enough in my quick test, I also needed to run wrangler with the nodejs_compat flag.  If setting the wrangler flag by itself also works without touching the config then I don't think there's a benefit to one or the other.


Error access with protocol HTTP2

Bug description
When we try access with protocol HTTP2 return thios error:
upstream connect error or disconnect/reset before headers. reset reason: connection termination
Expected behavior
Steps to reproduce the bug
After create gateway, virtualservice and destinationrule
https://raw.githubusercontent.com/yanolol/logs/master/PCOoMqqQlTibl5VF43RroiccrION8Qi9.yaml
We tried access with curl with protocol HTTP 1.1 works
Logs istio-ingressgateway:
[2020-01-02T16:17:33.193Z] "GET /appstorefront/_ui/desktop/common/images/flag.png HTTP/1.1" 200 - "-" "-" 4 759 3 3 "<IP_ADDRESS>" "PostmanRuntime/7.21.0" "9600c2b7-ee85-9105-a6d8-8998992a2fc0" "dev.app.com" "<IP_ADDRESS>:8088" outbound|8088||http2-dev-app-storefront.dev-app.svc.cluster.local - <IP_ADDRESS>:443 <IP_ADDRESS>:65438 dev.app.com -
But when we tried access with protocol HTTP2 like a browser return 503 and "upstream connect..."
Logs istio-ingressgateway:
[2020-01-02T16:18:18.260Z] "GET /appstorefront/_ui/desktop/common/images/flag.png HTTP/2" 503 UC "-" "-" 0 95 16 - "<IP_ADDRESS>" "Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/79.0.3945.88 Safari/537.36" "ca03e427-214b-9b26-9c0a-6c5777dcd7b1" "dev.app.com" "<IP_ADDRESS>:8088" outbound|8088||http2-dev-app-storefront.dev-app.svc.cluster.local - <IP_ADDRESS>:443 <IP_ADDRESS>:49180 dev.app.com -
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
Istio 1.4.2
Client Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.0", GitCommit:"2bd9643cee5b3b3a5ecbd3af49d09018f0773c77", GitTreeState:"clean", BuildDate:"2019-09-18T14:27:17Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.0", GitCommit:"2bd9643cee5b3b3a5ecbd3af49d09018f0773c77", GitTreeState:"clean", BuildDate:"2019-09-18T14:27:17Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}
How was Istio installed?
Helm/tiller
Environment where bug was observed (cloud vendor, OS, etc)
Calico
RHEL 7.6

I think you may need to make the protocol HTTP2: https://istio.io/docs/reference/config/networking/gateway/#Port

I think you may need to make the protocol HTTP2: https://istio.io/docs/reference/config/networking/gateway/#Port

My aplication runner only that protocol HTTP 1.1, in gateway is set with HTTPS, i tried to change like this, but doesn't work
  servers:
    - hosts:
        - dev.app.com
      port:
        name: http
        number: 80
        protocol: HTTP
      tls:
        httpsRedirect: true
    - hosts:
        - dev.app.com
      port:
        name: http-test
        number: 443
        protocol: HTTPS
      tls:
        mode: SIMPLE
        privateKey: /etc/istio/ingressgateway-certs/tls.key
        serverCertificate: /etc/istio/ingressgateway-certs/tls.crt

Is there any way to block HTTP2?

Guys, i resolve this problem addind https-grpc-service in my service.

Hi yanolol,
I'm experiencing the same issue with Istio 1.4.2. Can you provide more details how you added the https-grpc-service to make it work?
thanks,

@howardjohn do you know anything changed on HTTP2 in proxy 1.4.x? or any new configuration required? It worked on the same upstream service with Istio 1.3.1.

Hello @patrickppeng
I addind https-grpc in the service name port
apiVersion: v1
kind: Service
metadata:
  labels:
    heritage: Tiller
  name: my-aplication
  namespace: default
spec:
  clusterIP: <IP_ADDRESS>
  ports:
  - name: https-grpc-app
    port: 8088
    protocol: TCP
    targetPort: 8088

I am still trying to understand why this issue occurs, but I believe it is something related to TLS because of its authentication.

Thanks @yanolol. It works after specified the protocol on service port name.
I was confused with this documentation and now it makes sense...
https://istio.io/docs/ops/configuration/traffic-management/protocol-selection/


Video metadata processing no longer writes temp files

Should be merged after #288.
Metadata-finding subsamplers (FFProbeSubsampler and CutDetectionSubsampler) no longer take byte streams, write to a temp file, then operate on the temp file. Rather, we can pass a filepath directly to these subsamplers, and they will extract metadata without performing additional I/O operations.
After this I will do the same with the video processing subsamplers in the next pull request.
This pull request has been tested with my usual workflow, reproducing expected results.

can you rebase please ?

@MattUnderscoreZhang what speed difference do you observe? sharing wandb links would be helpful

I'm showing my noob status here, but I've never really used wandb. Could we maybe schedule a call sometime where you show me how to generate the links you're looking for?
Also, I don't think there should be very much speedup at this stage anyways. I still need to perform a temporary write at the beginning of sample processing, since I haven't touched the dataloaders yet. That is, it would be nice if the dataloaders passed filepaths rather than byte streams, but this will have to be a later change. As it stands, I think I'm currently only saving a single read/write. The major savings should be in the next pull request, when I update the actual video processing subsamplers.

I think it's quite important to check the speed for this kind of major change
It could get worse

Here's a check of this branch vs. main on WandB.
https://wandb.ai/bucketoffish/video2dataset?workspace=user-bucketoffish
good-cherry-2 is this branch, which you can see is slightly faster.
This test was done with 100 webvid videos, with 5 processes and 10 threads, 10 samples per shard, on my 2019 MacBook Pro.

Here's a comparison between this branch and the threading_fix branch. This is with a larger dataset, around 5k samples. You can see the results are reversed here, with this branch being slightly slower, by about 4%. Each of these branches took about 2.5 hours to run. I'm not sure how significant these results are (in terms of variance).

@iejMac do you have some numbers of how many vid/s you reached on webvid ?
@MattUnderscoreZhang how many workers are you using?

I'm using 5 processes with 2 threads each.

@rom1504 says at the bottom of this - https://github.com/iejMac/video2dataset/blob/main/dataset_examples/WebVid.md
230 video/s (14.4 videos/s/core) or 420 Mb/s

That seems to be for a download config with no video processing. My changes would not have any effect in that use case.

@MattUnderscoreZhang ok let's try to run with the same settings as in that example
Also it would be helpful to increase the number of processes and threads
5 and 2 are too low to catch problems

I tried replicating a run with the exact config used in the linked example. 16 processes with 16 threads each, with 1000 samples per shard. I ran on a vast.ai instance with a webvid results_2M_val dataset.
It's good we ran this test because unfortunately, it looks like this branch is definitely buggy. Something about the threading and multiprocessing is causing the run to freeze.
Looking at old commits and comparing to commit c6f3ed2 (the one right before my first commit), I see that the download worker refactor commit also has a threading problem (even with the threading_fix branch applied). The commit right before, e1b5d89, is fine. The speed comparison for this commit matches the older commit:

For now I recommend rolling back the download worker refactor commit. Fixing the threading issue will take some debugging, but I don't think I have the capacity for it right now. You can close this pull request if you want, and I'll come back and review this later.

This would need a rebase