Using StaticFiles crashes instrumentation for fastapi

Description
My project uses some StaticFiles configs in FastAPI.  This broke with version ~0.43.0, version 0.42.0 works fine.
main-1  | 2024-06-29T15:00:35.805550267Z   File "/usr/local/lib/python3.12/site-packages/achernar/ltyabb/restapi/app.py", line 98, in
main-1  | 2024-06-29T15:00:35.805788495Z     logfire.instrument_fastapi(app)
main-1  | 2024-06-29T15:00:35.805802763Z   File "/usr/local/lib/python3.12/site-packages/logfire/_internal/main.py", line 869, in instrument_fastapi
main-1  | 2024-06-29T15:00:35.806061535Z     return instrument_fastapi(
main-1  | 2024-06-29T15:00:35.806068786Z            ^^^^^^^^^^^^^^^^^^^
main-1  | 2024-06-29T15:00:35.806072767Z   File "/usr/local/lib/python3.12/site-packages/logfire/_internal/integrations/fastapi.py", line 76, in instrument_fastapi
main-1  | 2024-06-29T15:00:35.806252844Z     mounted_apps = find_mounted_apps(app)
main-1  | 2024-06-29T15:00:35.806317922Z                    ^^^^^^^^^^^^^^^^^^^^^^
main-1  | 2024-06-29T15:00:35.806327324Z   File "/usr/local/lib/python3.12/site-packages/logfire/_internal/integrations/fastapi.py", line 40, in find_mounted_apps
main-1  | 2024-06-29T15:00:35.806541391Z     mounted_apps += find_mounted_apps(_app)
main-1  | 2024-06-29T15:00:35.806551991Z                     ^^^^^^^^^^^^^^^^^^^^^^^
main-1  | 2024-06-29T15:00:35.806555790Z   File "/usr/local/lib/python3.12/site-packages/logfire/_internal/integrations/fastapi.py", line 36, in find_mounted_apps
main-1  | 2024-06-29T15:00:35.806558391Z     for route in app.routes:
main-1  | 2024-06-29T15:00:35.806560188Z                  ^^^^^^^^^^
main-1  | 2024-06-29T15:00:35.806561856Z AttributeError: 'StaticFiles' object has no attribute 'routes'
Appears may have been introduced with https://github.com/pydantic/logfire/pull/259
Python, Logfire & OS Versions, related packages (not required)
fastapi >= 0.109
logfire >= 0.43.0

cc @sneakyPad

Thanks @tardis10. I see the problem. I'll write a fix and add a test for this as well.

Thanks for the quick response!

Fixed, please pip install -U logfire and try again.


Add Stepper to Questionnaire

Documentation of stepper
https://mantine.dev/core/stepper/

@rahul1 please add adaptive forms documentation here - for standardized extensions, etc.

We should leverage the Structured Data Capture (SDC) implementation guide as much as possible (http://hl7.org/fhir/uv/sdc/)
For the stepper, take a look at the adaptive-forms spec (http://hl7.org/fhir/uv/sdc/adaptive.html)
Other things for HG:

Automatic Population: http://hl7.org/fhir/uv/sdc/populate.html

https://github.com/medplum/medplum/pull/2667


object_storage: batching for S3 delete_objects

About this change - What it does
Fixes when S3 would receive a request to delete a large number of objects.
Why this way
Without batching, any requests to delete_objects containing more than 1000 objects would fail (the current limit ).

Codecov Report
Patch coverage: 92.85% and project coverage change: +0.17 :tada:

Comparison is base (a9eb857) 61.91% compared to head (cc8b6fa) 62.09%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #106      +/-   ##
==========================================
+ Coverage   61.91%   62.09%   +0.17%
==========================================
  Files          30       30
  Lines        3356     3366      +10
==========================================
+ Hits         2078     2090      +12
+ Misses       1278     1276       -2

Impacted Files
Coverage Δ

rohmu/util.py
42.10% <88.88%> (+42.10%)
:arrow_up:

rohmu/object_storage/s3.py
53.52% <100.00%> (+0.29%)
:arrow_up:

... and 2 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

I will let @fingon  merge this since I'm not sure about the process for merging rohmu PRs.

Merging here is entirely normal; it is just internal use packaging that is extra step.


Feature: add support for net8.0

Hi @ChristianSauer
first of all thanks for this project. With yesterdays release of net8.0 I would like to use this with the latest framework.
Hope I covered the few steps for this.

Thank you! Looks good


make WSLg Linux windows full first class citizens in the Windows window manager(s)

Is your feature request related to a problem:
It's well documented that WSLg desktop integration is only partial, for example snapping doesn't work yet, clipboard interaction is buggy and some keys like "Super" are not supported and don't have a good proper workaround.
xx@YYY:~/xsdata$ wsl.exe --version
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-1
WSLg version: 1.0.59
MSRDC version: 1.2.4677
Direct3D version: 1.611.1-81528511
DXCore version: 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp
Windows version: 10.0.19045.3930

Describe the solution you'd like:
can you guys more transparently communicate your roadmap and priorities on fixing WSLg Windows desktop integration?
the alternative (deactivate WSLg and use VcXsrv or similar) is not satisfactory.
Describe alternatives you've considered:
the non-solution is "run VcXsrv" or a similar X Server.
Additional context:
get inspiration from xorg on Windows, i.e. xwin source code or the VcXsrv equivalent xwin source code
they address and solve these issues.
and then communicate what you intend to do and make visible how and when you do it.

can't add labels window-manager, clipboard-integration, keyboard-layout ...

I share the same concerns. Have been using WSLg since it became available on win10 and it remains one of the major issues

When I first heard about and tried WSLg I thought I'd be able to stop dual booting and instead use WSL and WSLg for my Linux needs. Unfortunately using Linux graphical programs through WSLg has proven to be too frustrating and I now instead simply don't bother booting into Windows except on my work PC.


!>(RUGBY-STREAMs!) New Zealand vs England LIVE FreE Autumn Internationals rugby 02. 11. 2024

Is England v New Zealand on TV? Kick-off time, channel and how to watch Autumn Nations Series today
Everything you need to know as England begin their autumn Tests against the All Blacks
CLICK HERE TO WATCH LIVE STREAM NOW
CLICK HERE TO WATCH LIVE STREAM NOW
CLICK HERE TO WATCH LIVE STREAM NOW
England Vs. New Zealand All Blacks Lineups, Live Updates And Scores
Follow along for live updates from the England vs. New Zealand match, including lineups, scores, and the kickoff time from Allianz Stadium in Twickenham.
England vs New Zealand LIVE SCORE: Borthwick’s side host All Blacks in Autumn Internationals opener – updates
England vs New Zealand LIVE! Latest score and rugby updates from Autumn Nations Series
The All Blacks head to Twickenham with all eyes on the haka after Joe Marler controversy
Despite it feeling like a lifetime, it's only been three-and-a-half months since England played New Zealand in a pair of tests in July.
Those matches were electric, and All Blacks won both of them (you can watch them free here and here. There is also a YouTube embed below).
England have not played a test match since then. But New Zealand had a full slate during the Rugby Championship, taking second-place in that competition.
What Is The England Vs. New Zealand All Blacks Kick Off Time?
England vs. New Zealand All Blacks kicks off at 11:10 a.m. ET at Allianz Stadium, Twickenham, London, UK.
England and New Zealand kick off the Autumn Nations Series with a marquee clash at the renamed Allianz Stadium, Twickenham.
England: 15 George Furbank, 14 Immanuel Feyi-Waboso, 13 Henry Slade, 12 Ollie Lawrence, 11 Tommy Freeman, 10 Marcus Smith, 9 Ben Spencer, 8 Ben Earl, 7 Tom Curry, 6 Chandler Cunningham-South, 5 George Martin, 4 Maro Itoje, 3 Will Stuart, 2 Jamie George (c), 1 Ellis Genge
All Blacks: 15 Will Jordan, 14 Mark Tele’a, 13 Rieko Ioane, 12 Jordie Barrett, 11 Caleb Clarke, 10 Beauden Barrett, 9 Cortez Ratima, 8 Ardie Savea, 7 Sam Cane, 6 Wallace Sititi, 5 Tupou Vaa’i, 4 Scott Barrett (c), 3 Tyrel Lomax, 2 Codie Taylor, 1 Tamaiti Williams
Steve Borthwick’s side were beaten narrowly in both Tests on their July tour, and will be out for revenge against an All Blacks side still adjusting to life under head coach Scott Robertson. A storied rivalry has been given extra edge this week by clumsy comments from England prop Joe Marler, who suggested the traditional pre-match haka needed “binning” on social media.


docs: Fixed README redirection hyperlinks

Please briefly answer these questions:

what problem are you trying to solve? (or if there's no problem, what's the motivation for this change?)
The README.md file in crates/xline-client contains two incorrect hyperlinks, likely due to changes in the code structure.
what changes does this pull request make?
Only modified the README.md
are there any non-obvious implications of these changes? (does it break compatibility with previous versions, etc)

I'm sorry, I was not familiar with VScode git tool.


Does superset support integration with MongoDB?

I tried connecting using the following SQLAlchemy URI
mongodb://localhost:27017/test
and faced the following error:
ERROR: {"error": "Connection failed!\n\nThe error message returned was:\nCan't load plugin: sqlalchemy.dialects:mongodb"}
Is there any workaround? Please suggest.

5th option:
There is a tool called BI Connector from mongodb, this tool can map mongodb schemata to relational. It's used to interact e.g. with Tableau or other BI Tools. Unfortunately AFAIK it's only available in a paid mongodb version. https://www.mongodb.com/products/bi-connector

I have created a MongoDB connector for superset. It is still WIP. Feel free to check it out. https://github.com/ksangeet9ap/incubator-superset

I have created a MongoDB connector for superset. It is still WIP. Feel free to check it out.
https://github.com/ksangeet9ap/incubator-superset

I have created a MongoDB connector for superset under the name of cosmos. It is still WIP. Feel free to check it out. https://github.com/ksangeet9ap/incubator-superset

@mistercrunch Any plans on integrating NoSQL databases to superset?

so , how may anyone use superset with MongoDB .
Please, tell me any possible way of doing it

1、Mongodb-BI-Connector with sqlalchemy-mongobi
2、Presto MongoDB Connector

1、Mongodb-BI-Connector with sqlalchemy-mongobi
2、Presto MongoDB Connector
@yuxiaorui can you provide an example?

1、Mongodb-BI-Connector with sqlalchemy-mongobi
2、Presto MongoDB Connector

Could you explain how I would implement this?

5th option:
There is a tool called BI Connector from mongodb, this tool can map mongodb schemata to relational. It's used to interact e.g. with Tableau or other BI Tools. Unfortunately AFAIK it's only available in a paid mongodb version. https://www.mongodb.com/products/bi-connector

Its available for free now. I would like to know if someone was able to connect through it

1、Mongodb-BI-Connector with sqlalchemy-mongobi
http://docs.mongodb.com/bi-connector/master/
https://pypi.org/project/sqlalchemy-mongobi/
2、Presto MongoDB Connector
https://prestodb.io/docs/current/connector/mongodb.html

I tried using the 1st option. It gives me driver not found error. I installed it in the same venv also. but still no luck. Did anyone got it working this way?

I tried using the 1st option. It gives me driver not found error. I installed it in the same venv also. but still no luck. Did anyone got it working this way?

ERROR: Could not load database driver: mongobi
the exact error when testing connection using sql alchemy uri

Is there anybody using Superset and MongoDB?

I just made a proof of concept of querying a mongo database with superset through mongo bi connector : https://github.com/jbouzekri/docker-superset-mongo-poc/. It seems to work pretty well. It is not in the documentation but I have tested row level security and it works too.

@jbouzekri thank you for the POC! really appreciate your efforts.

@jbouzekri thank you for the POC! But I ran into some problems using it: pulling the code locally and then executing the 'docker-compose up' directive encountered an error and terminated the directive. Could you give me some help?

@Hugh-Guan : can you open an issue directly in the POC repository with the details of the error ? Thanks in advance

Thank you for your reply.  I have put forward an issue in the project

@Hugh-Guan : can you open an issue directly in the POC repository with the details of the error ? Thanks in advance

@jbouzekri Thank you for your reply. I have put forward an issue in the project

Have anyone tried integrating Superset with Azure Cosmos DB? It has a MongoDB API, I wonder if it works with sqlalchemy.

Have anyone tried integrating Superset with Azure Cosmos DB? It has a MongoDB API, I wonder if it works with sqlalchemy.

Do you mean RU or vCore MongoDB? Apparently vCore MongoDB is very close to the vanilla MongoDB in terms of protocol - https://learn.microsoft.com/en-us/azure/cosmos-db/mongodb/vcore/compatibility.


ipv6

Good afternoon, thanks for the script, but could you tell me, I have ipv6 and the server also supports ipv6, how can I use it?

ipv6 > ipv4 , u know ?

IPv6 inside the tunnel should be configured automatically during setup if the server has IPv6 available. Still, the client will connect to the server over IPv4.


Split files?

I was using laptop to source the mac functions and homebrew functions without actually running the script like so:
source laptop/common-components/shared-functions
source laptop/mac-components/homebrew
source laptop/mac-components/mac-functions

Unfortunately, I can't do this any longer. Any ideas?

My script looks similar to:
#!/bin/bash

source /dev/stdin  <<< "$(curl -s https://raw.githubusercontent.com/thoughtbot/laptop/master/mac)"
brew_install_or_upgrade 'qt'
brew_install_or_upgrade 'tcl-tk'
...

I simply wish to have access to the functions such as brew_install_or_upgrade because I find them extremely useful and I appreciate how well they are maintained in this repository.
I also wish to install certain packages with parameters (such as git) which are installed in a different way in the main laptop script.
brew_install_or_upgrade 'git' '--with-pcre' '--with-brewed-curl' '--with-brewed-openssl' '--with-brewed-svn' '--with-gettext'

There is no easy way to do this without an ugly hack using awk.
curl -s "https://raw.githubusercontent.com/thoughtbot/laptop/master/mac" | awk '/\(\) *{/ , /^}/'

source: https://gist.github.com/zmughal/9a630d6d9dd494ce6d9b
Any thoughts if using laptop functions is a supported or encouraged behavior? I don't think it would be too difficult to have two files: one with functions and another for business logic. The Makefile can concatenate them or you can curl one into the other as I do above.
cc @croaky

Another alternative is to put everything into a main() function which one can less hackily ignore.


Improve CLI help for subcommands

Description
In previous releases, the CLI help included basic descriptions for the following arguments:
  --install [<file>]          install plug-in from a local ZIP file
  --install [<url>]           install plug-in from a URL
  --install [<id>]            install plug-in from plugin registry
  --install                   reload plug-ins
  --uninstall <id>            uninstall plug-in with the ID
  --plugins                   print list of installed plug-ins
  --transtypes                print list of installed transtypes
  --deliverables              print list of deliverables in project
  -h, --help                  print this message
  --version                   print version information and exit

Current Behavior
Now that subcommand help has moved to dedicated dita subcommand --help with #3437, the top-level entries for the subcommands are empty, and nothing indicates that more info is available via dita subcommand --help:
$ dita --help
Usage:
  dita -i <file> -f <name> [options]
  dita --project=<file> [options]
Subcommands:
  deliverables
  install
  plugins
  transtypes
  uninstall
  version

Expected Behavior
To help users understand the purpose of each subcommand, it would be good if a basic description could be presented alongside each subcommand in the dita --help output, and find a place to indicate that details are available for each via dita <subcommand> --help:
$ dita --help
Usage:
  dita -i <file> -f <name> [options]
  dita --project=<file> [options]
Subcommands:
  deliverables      Print list of deliverables in project file
  install           Install or reload plug-ins
  plugins           Print list of installed plug-ins
  transtypes        Print list of installed transformation types (output formats)
  uninstall         Remove and delete plug-in
  version           Print version information and exit

  See 'dita <subcommand> --help' for details about a specific subcommand.

For subcommands with multiple arguments, it looks like the arguments are missing next to their descriptions:
dita install --help
Usage:
  dita install [<file> | <url> | <id>]
Arguments:
                 install plug-in from a local ZIP file
                 install plug-in from plugin registry
                 install plug-in from a URL

Expected to see something like this instead:
dita install --help
Usage:
  dita install [<file> | <url> | <id>]
Arguments:
  <file>       Install plug-in from a local ZIP file
  <url>        Install plug-in from a URL
  <id>         Install specified plug-in ID from registry

@jelovirt I just noticed that dita help is not currently listed among the subcommands, although it does work without the double hyphens.
Would you prefer to view that as an option (rather than a subcommand) for consistency with the dita subcommand --help syntax, or what's the rationale there?

dita help is there as a fallback. The preferred form is dita --help and dita subcommand --help.


Add tests for Add Todo command & related tests

Closes #128

Done up the tests for AddTodoCommand and it's related tests.


checkpatch: Update image for "checkpatch" target, reuse target in CI

Some minor adjustements to the way we run checkpatch:

Update the image for the checkpatch target in bpf/Makefile.bpf, so that running make -C bpf checkpatch uses the same image as CI.
Update CI action to use this checkpatch target. This is so that we only keep one reference to the Docker image instead of having it at two distinct locations.

Currently under test. Don't mind those horrible additional commits.

I validated that the CI action still reports errors, by temporarily adding test commits that were not respecting our guidelines (line too long in C code, commit object too long).

Everything seems to work as expected, I removed those test commits.

+1 - We should probably document a bit checkpatch on dev doc or contributor guide.

Hello @qmonnet, I was more thinking about something like that: https://github.com/cilium/cilium/pull/19879 ;-)


Ability to reformat only a given code segment

 Issue by erandidesilva
Tuesday Oct 10, 2017 at 13:54 GMT
Originally opened as https://github.com/ballerina-lang/composer/issues/3484

Version: 0.93
Browser: Chrome Version 61.0.3163.100
Can we have $subject?

Composer is discontinued hence closing the issue.


explicit_gribi_under_network_instance deviation removal from all gribi tests

remove deviation from all metadata.textproto files
Remove deviation from all gribi test files
Updated base_hierarchical_nhg_update_test.go  test logic order to accommodate removal of this deviation
"This code is a Contribution to the OpenConfig Feature Profiles project ("Work") made under the Google Software Grant and Corporate Contributor License Agreement ("CLA") and governed by the Apache License 2.0. No other rights or licenses in or to any of Nokia's intellectual property are granted for any other purpose. This code is provided on an "as is" basis without any warranties of any kind."

Pull Request Functional Test Report for #2166 / 0e32f3e8785d2cd7cc88c13d86c5ba9532460099
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

TE-11.2: Backup NHG: Multiple NHTE-3.7: Base Hierarchical NHG UpdateTE-3.6: ACK in the Presence of Other RoutesTE-3.1: Base Hierarchical Route InstallationTE-14.1: gRIBI ScalingTE-15.1: gRIBI ComplianceTE-11.3: Backup NHG: ActionsTE-13.1: gRIBI route ADD during FailoverTE-6.2: Route Removal In Non Default VRFTE-11.1: Backup NHG: Single NHTE-13.2: gRIBI route DELETE during Failover

Cisco 8000E

TE-11.2: Backup NHG: Multiple NHTE-3.7: Base Hierarchical NHG UpdateTE-3.6: ACK in the Presence of Other RoutesTE-3.1: Base Hierarchical Route InstallationTE-14.1: gRIBI ScalingTE-15.1: gRIBI ComplianceTE-11.3: Backup NHG: ActionsTE-13.1: gRIBI route ADD during FailoverTE-6.2: Route Removal In Non Default VRFTE-11.1: Backup NHG: Single NHTE-13.2: gRIBI route DELETE during Failover

Cisco XRd

TE-11.2: Backup NHG: Multiple NHTE-3.7: Base Hierarchical NHG UpdateTE-3.6: ACK in the Presence of Other RoutesTE-3.1: Base Hierarchical Route InstallationTE-14.1: gRIBI ScalingTE-15.1: gRIBI ComplianceTE-11.3: Backup NHG: ActionsTE-13.1: gRIBI route ADD during FailoverTE-6.2: Route Removal In Non Default VRFTE-11.1: Backup NHG: Single NHTE-13.2: gRIBI route DELETE during Failover

Juniper cPTX

TE-11.2: Backup NHG: Multiple NHTE-3.7: Base Hierarchical NHG UpdateTE-3.6: ACK in the Presence of Other RoutesTE-3.1: Base Hierarchical Route InstallationTE-14.1: gRIBI ScalingTE-15.1: gRIBI ComplianceTE-11.3: Backup NHG: ActionsTE-13.1: gRIBI route ADD during FailoverTE-6.2: Route Removal In Non Default VRFTE-11.1: Backup NHG: Single NHTE-13.2: gRIBI route DELETE during Failover

Nokia SR Linux

TE-11.2: Backup NHG: Multiple NHTE-3.7: Base Hierarchical NHG UpdateTE-3.6: ACK in the Presence of Other RoutesTE-3.1: Base Hierarchical Route InstallationTE-14.1: gRIBI ScalingTE-15.1: gRIBI ComplianceTE-11.3: Backup NHG: ActionsTE-13.1: gRIBI route ADD during FailoverTE-6.2: Route Removal In Non Default VRFTE-11.1: Backup NHG: Single NHTE-13.2: gRIBI route DELETE during Failover

Openconfig Lemming

TE-11.2: Backup NHG: Multiple NHTE-3.7: Base Hierarchical NHG UpdateTE-3.6: ACK in the Presence of Other RoutesTE-3.1: Base Hierarchical Route InstallationTE-14.1: gRIBI ScalingTE-15.1: gRIBI ComplianceTE-11.3: Backup NHG: ActionsTE-13.1: gRIBI route ADD during FailoverTE-6.2: Route Removal In Non Default VRFTE-11.1: Backup NHG: Single NHTE-13.2: gRIBI route DELETE during Failover

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

TE-11.2: Backup NHG: Multiple NHTE-3.7: Base Hierarchical NHG UpdateTE-3.6: ACK in the Presence of Other RoutesTE-3.1: Base Hierarchical Route InstallationTE-14.1: gRIBI ScalingTE-15.1: gRIBI ComplianceTE-11.3: Backup NHG: ActionsTE-13.1: gRIBI route ADD during FailoverTE-6.2: Route Removal In Non Default VRFTE-11.1: Backup NHG: Single NHTE-13.2: gRIBI route DELETE during Failover

Cisco 8808

TE-11.2: Backup NHG: Multiple NHTE-3.7: Base Hierarchical NHG UpdateTE-3.6: ACK in the Presence of Other RoutesTE-3.1: Base Hierarchical Route InstallationTE-14.1: gRIBI ScalingTE-15.1: gRIBI ComplianceTE-11.3: Backup NHG: ActionsTE-13.1: gRIBI route ADD during FailoverTE-6.2: Route Removal In Non Default VRFTE-11.1: Backup NHG: Single NHTE-13.2: gRIBI route DELETE during Failover

Juniper PTX10008

TE-11.2: Backup NHG: Multiple NHTE-3.7: Base Hierarchical NHG UpdateTE-3.6: ACK in the Presence of Other RoutesTE-3.1: Base Hierarchical Route InstallationTE-14.1: gRIBI ScalingTE-15.1: gRIBI ComplianceTE-11.3: Backup NHG: ActionsTE-13.1: gRIBI route ADD during FailoverTE-6.2: Route Removal In Non Default VRFTE-11.1: Backup NHG: Single NHTE-13.2: gRIBI route DELETE during Failover

Nokia 7250 IXR-10e

TE-11.2: Backup NHG: Multiple NHTE-3.7: Base Hierarchical NHG UpdateTE-3.6: ACK in the Presence of Other RoutesTE-3.1: Base Hierarchical Route InstallationTE-14.1: gRIBI ScalingTE-15.1: gRIBI ComplianceTE-11.3: Backup NHG: ActionsTE-13.1: gRIBI route ADD during FailoverTE-6.2: Route Removal In Non Default VRFTE-11.1: Backup NHG: Single NHTE-13.2: gRIBI route DELETE during Failover

Help

LGTM Nokia related flagz


Add CI tests

[x] migrate travis unittests to github actions as it's no longer being used
[ ] add integration test / simulate a full basic debugging routine

The first part was done in #354 by @brownts. 👍.

I've reworked the test harness so that it's no longer dependent on having VSCode run it.  That should be a much nicer workflow, since you don't have to launch another instance of VSCode to run the tests.  This just runs Mocha directly instead of through VSCode.  There was really no reason to run it through VSCode as the extension doesn't dependent on it.  In addition, I've also added functionality to collect code coverage and report it via CodeCov.  Lastly, I've added badges in the README to show the CI status as well as the coverage percentage, along with links to the CodeCov detailed information.
The tests can be run from the "npm:test" in tasks.json if you like (or running npm test from the command line), but it is much nicer if you actually install and use the "Mocha Test Explorer" extension.  That adds nice integration and provides a "Testing" tab in the VSCode Activity Bar.  You can run/debug/etc. tests from there with ease.

Thank you this sounds very usefull.
The next thing would be to increase the test coverage...


⚠ Samsung has degraded performance

In 8eb1c18, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2493 ms

Resolved: Samsung performance has improved in 618514a.


Fix syntax error in props table

The regex docs were missing because the pipe character was not escaped.

Thx!


Newer versions of AE2 charged certus quartz in SAG Mill

Newer versions of AE2 now have an ore dictionary "oreCertusQuartz" entry for Charged Certus Quartz, the core recipe for Charged now causes a conflict and should be removed.

In a bizarre turn, updating to the latest AE2 and overriding the Charged recipe in the user config seems to have resolved these issues. Versions without the ore dictionary update still seem to have the issue.
https://github.com/SleepyTrousers/EnderIO/issues/2207
https://github.com/BuildCraft/BuildCraft/issues/2609


improved-wiki

Hey @swapagarwal , As you suggested I have made changes. I hope now everything is fine and JARVIS is able to answer to question 'Who is sachin tendulkar?' ;-)

Hey @swapagarwal , I couldn't understand why it is showing

All checks have failed.

Could you please help me?

Thanks @shreyas95! :+1:


Readme: clarified “safeLoad” return type

Fixes https://github.com/nodeca/js-yaml/issues/507

@puzrin super quick as always, thank you.


BackupRestoreService, does it accept policies with pre-encrypted credentials?

We've recently started utilizing the Backup/Restore feature in Service Fabric and noticed that service is configured with a SecretEncryptionCertThumbprint. Its obvious that this will encrypt any unencrypted credentials that are provided with backup policies. My question is, if I pre-encrypt the credentials with the same certificate that was specified in SecretEncryptionCertThumbprint using Invoke-ServiceFabricEncryptText, and then provide that encrypted string in the policy, will the BackupRestoreService recognize this and decrypt as necessary?
I ask because this would help us from a DevOps perspective, ensuring we aren't leaving credentials in plaintext at any point during the deployment process.

@JustinKaffenberger , We don't have any solution for this problem for now. There is no input taken from user in storage description which tells BRS if the credentials are already encrypted by the user or not.
We encrypt only on the basis if SecretEncryptionCertThumbprint is provided or not. So, if you would encrypt the credentials and also input SecretEncryptionCertThumbprint to BRS then, double encryption will occur and policy will be useless.


Failed to reconcile resource could not reconcile istio

Hello guys,
I have a problem with istio using ingressgateway. I received this error log:
"timed out waiting for the condition\ncould not execute AfterUpdate func\ngithub.com/banzaicloud/istio-operator/pkg/k8sutil.Reconcile\n\t/go/src/github.com/banzaicloud/istio-operator/pkg/k8sutil/resource.go:153\ngithub.com/banzaicloud/istio-operator/pkg/resources/istiod.(*Reconciler).Reconcile\n\t/go/src/github.com/banzaicloud/istio-operator/pkg/resources/istiod/istiod.go:113\ngithub.com/banzaicloud/istio-operator/pkg/controller/istio.(*ReconcileIstio).reconcile\n\t/go/src/github.com/banzaicloud/istio-operator/pkg/controller/istio/istio_controller.go:355\ngithub.com/banzaicloud/istio-operator/pkg/controller/istio.(*ReconcileIstio).Reconcile\n\t/go/src/github.com/banzaicloud/istio-operator/pkg/controller/istio/istio_controller.go:239\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.6.2/pkg/internal/controller/controller.go:235\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.6.2/pkg/internal/controller/controller.go:209\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).worker\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.6.2/pkg/internal/controller/controller.go:188\nk8s.io/apimachinery/pkg/util/wait.BackoffUntil.func1\n\t/go/pkg/mod/k8s.io/apimachinery@v0.18.6/pkg/util/wait/wait.go:155\nk8s.io/apimachinery/pkg/util/wait.BackoffUntil\n\t/go/pkg/mod/k8s.io/apimachinery@v0.18.6/pkg/util/wait/wait.go:156\nk8s.io/apimachinery/pkg/util/wait.JitterUntil\n\t/go/pkg/mod/k8s.io/apimachinery@v0.18.6/pkg/util/wait/wait.go:133\nk8s.io/apimachinery/pkg/util/wait.Until\n\t/go/pkg/mod/k8s.io/apimachinery@v0.18.6/pkg/util/wait/wait.go:90\nruntime.goexit\n\t/usr/local/go/src/runtime/asm_amd64.s:1357\nfailed to reconcile resource\ncould not reconcile istio"
EKS version: 21
istio version: 1.7.8

The pods was in looping shutdown less than 2 minutes: "failed to reconcile resource could not reconcile istio"
Troubleshooting:
We change the HPA config
reboot all the pods
disable and enable the ingress-gateway / istio-gateway
Could you please, help me about it?

Hi!
What's your Istio operator version? How did you enable an ingressgateway, through the Istio CR or the MeshGateway CR? Could you pls share your Istio and/or MeshGateway CR yamls as well?
Thanks!

@Laci21 hello,
istio version: 1.7.8, using CR.
I've attached the file here because it’s too large:
istio-operator.yml.zip

Hi,
I'm looking for the Istio operator's version, not the Istio version. So docker image of the istio-operator pod.
CR status looks good, ingress gw pods are up and running fine?

Sure,
Istio operator's version: /banzaicloud/istio-operator:0.7.8
At the moment yes, are running fine. I'm try to understand what's happening.

Based on the error log it seems to be related to istiod deployment/pods. Please check that the istiod deployment and pods are up successfully as they should when you see this error.
There should a program argument as well for the Istio operator to enable debug logging. With that you might see more (devel-mode=true maybe).


Docs assume PowerShell is running on Windows

Prerequisites

[x] I have read and understand the CONTRIBUTING guide
[x] I looked for duplicate issues before submitting this one

Description
Look at the "replace prompt" section:
https://ohmyposh.dev/docs/installation#4-replace-your-existing-prompt
$startInfo.FileName = "C:\tools\oh-my-posh.exe"
$cleanPWD = $PWD.ProviderPath.TrimEnd("\")
$startInfo.Arguments = "-config=""$env:USERPROFILE\.poshthemes\jandedobbeleer.omp.json"" -error=$errorCode -pwd=""$cleanPWD"" -execution-time=$executionTime"

There are several assumptions that this is being run on Windows.
When I first ran this on macOS, it silently failed and gave me the PS> prompt.
Environment

Oh my Posh version: N/A
Theme: N/A
Operating System: macOS
Shell: PowerShell
Terminal: iTerm2

Steps to Reproduce

Install oh-my-posh3 with homebrew
Run script in https://ohmyposh.dev/docs/installation#4-replace-your-existing-prompt

Expected behavior: Need a tab for "PowerShell on non-Windows" or something like that
Actual behavior: None exists

Yes, that's correct. Unfortunately I can't nest the tabs, however, I can offer the same function the Module has, making this work regardless. Unix style paths work on Windows for Powershell so that shouldn't be an issue. Although it does make me wonder why one wouldn't opt for the model in that case.

@JanDeDobbeleer  just my own experience, when I found this project, I went to the docs and looked for "Installation", which led me here and had me do a manual installation. It wasn't until much later that I realized this module can also be installed much more simply from the PowerShell Gallery as described here.
Just an idea, maybe the installation docs for PowerShell should steer the user to the Install-Module route first.  I'm not sure if that fixes the macOS user story.  To be honest, I've never installed using the PowerShell Gallery :D

@TravisTX a high level info block could do the trick. I'll do the platform agnostic changed and add that for completeness. I get the feeling you're not the only ones.

@JanDeDobbeleer something worth considering is doing what starship does and bundle the init script into the product so that the scripts are a bit more friendly to look at.

@TylerLeonhardt the first working version is drafted, I'm going to add the other init scripts as well (and probably refactor the hell out of this). Should land somewhere this week.


How do I add TFLM to the new Ti platform (Cortex-R5F)

First of all, thank you for the resources you provided.
I am a researcher trying to use the single chip field for the first time.
After seeing many TensorFlow Lite official documents, a lot of information about Ti is that it only supports the Cortex-M series platform.
Now the hardware I am using is Ti AWR2944 with Cortex-R5F and I use CCS as IDE. How do I import TensorFlow Lite into my project?

You will need to port TFLM to this platform. I would suggest starting here: https://github.com/tensorflow/tflite-micro/blob/main/tensorflow/lite/micro/docs/new_platform_support.md
Once you implement the basic platform support, you should be fine to use TFLM with the portable reference kernels. Performance will probably not be good, but it will be functional.


Data table extension: Actions dropdown

Data table extension: Actions dropdown
Brief description
This issue covers the actions dropdown extension for data tables. The actions dropdown is a dropdown in the primary button location.
Details
Link to design
Links to other materials
e.g. sketch files, icons, images, prototypes
Design Maintainer(s)

Marion Brülls
Johanna Koval

Tasks
Before starting work on this epic, please review and complete the following.

[ ] Initial review of design/existing code

Working in Carbon for IBM Products package

[ ] Have you reviewed our
contribution guidelines?
[ ] Have you noted our
code guidelines?
[ ] Did you use the CLI to scaffold your new component?

Includes incorporating the ButtonMenu for primary action


CMS update issue

Seems that since the NetlifyCMS update, deploying a new instance of one-click-hugo-cms results in the right-hand pane of the CMS not having its usual WYSIWIG features.
Below: Normal right-hand pane view + current right-hand pane view with newly launched one-click-hugo-cms sites.
(Note: This also happens w/the NetlifyCMS-Gatsby site, with a slight difference: after initial deploy, once you update to NetCMS2.8.0, the right-pane loses its WYSIWIG display features.)

Netlify CMS has been updated to address this, you'll want to get 2.9.0 or greater.


KeyError: Caught KeyError in DataLoader worker process 0.

🐛 Bug
I've created a dataset following the approach on https://lightning.ai/lightning-ai/studios/convert-spatial-data-to-lightning-streaming
On using the dataset I get
KeyError: Caught KeyError in DataLoader worker process 0.
Original Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/torch/utils/data/_utils/worker.py", line 308, in _worker_loop
    data = fetcher.fetch(index)
  File "/usr/local/lib/python3.10/dist-packages/torch/utils/data/_utils/fetch.py", line 32, in fetch
    data.append(next(self.dataset_iter))
  File "/usr/local/lib/python3.10/dist-packages/lightning/data/streaming/dataset.py", line 274, in __next__
    data = self.__getitem__(
  File "/code/common/datasets/streaming.py", line 20, in __getitem__
    image_name, mask, image = super().__getitem__(index)
  File "/usr/local/lib/python3.10/dist-packages/lightning/data/streaming/dataset.py", line 244, in __getitem__
    return self.cache[index]
  File "/usr/local/lib/python3.10/dist-packages/lightning/data/streaming/cache.py", line 135, in __getitem__
    return self._reader.read(index)
  File "/usr/local/lib/python3.10/dist-packages/lightning/data/streaming/reader.py", line 252, in read
    item = self._item_loader.load_item_from_chunk(index.index, index.chunk_index, chunk_filepath, begin)
  File "/usr/local/lib/python3.10/dist-packages/lightning/data/streaming/item_loader.py", line 104, in load_item_from_chunk
    return self.deserialize(data)
  File "/usr/local/lib/python3.10/dist-packages/lightning/data/streaming/item_loader.py", line 112, in deserialize
    serializer = self._serializers[data_format]
KeyError: 'str'

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Run '....'
Scroll down to '....'
See error

Code sample
Dataset creation script:
import os
from litdata import optimize
from glob import glob
from typing import Optional, Tuple

# 1. List the files and associate the images and their masks
root = "/Volumes/Toshiba4TB/EDA/cloud_detection"
input_dir = f"{root}/tuning_split"
output_root = f"{root}/tuning_split_optimized"

def get_info(img_filepath: str) -> Optional[Tuple]:
    mask_filepath = img_filepath.replace("S2L1C.tif", "labels/manual_hq.tif")
    if not os.path.exists(mask_filepath):
        return None
    splits = img_filepath.split("/")
    name = f"{splits[7]}_{splits[8]}"
    return name, img_filepath, mask_filepath

def compress(data):
    name, image_filepath, mask_filepath = data
    return name, image_filepath, mask_filepath

train_images = glob(f"{input_dir}/train/*/*/S2L1C.tif")
train_inputs = []
for img in train_images:
    info = get_info(img)
    if info is not None:
        train_inputs.append(info)

val_inputs = []
val_images = glob(f"{input_dir}/val/*/*/S2L1C.tif")
for img in val_images:
    info = get_info(img)
    if info is not None:
        val_inputs.append(info)

test_inputs = []
test_images = glob(f"{input_dir}/test/*/*/S2L1C.tif")
for img in test_images:
    info = get_info(img)
    if info is not None:
        test_inputs.append(info)

# 4. Run the optimization
if __name__ == "__main__":
    optimize(
        fn=compress,
        inputs=train_inputs,
        output_dir=f"{output_root}/train",
        chunk_bytes="128MB",
    )

    optimize(
        fn=compress,
        inputs=val_inputs,
        output_dir=f"{output_root}/val",
        chunk_bytes="128MB",
    )

    optimize(
        fn=compress,
        inputs=test_inputs,
        output_dir=f"{output_root}/test",
        chunk_bytes="128MB",
    )

And the dataset:
class SegmentationStreamingDataset(StreamingDataset):
    """
    Segmentation dataset with streaming

    Args:
        input_dir (str): Local directory or S3 location of the dataset
        transforms (Optional[Callable]): A transform that takes in an image and returns a transformed version.
    """

    def __init__(self, *args, transforms=None, **kwargs):
        super().__init__(*args, **kwargs)
        self.transforms = transforms

    def __getitem__(self, index) -> dict:
        image_name, mask, image = super().__getitem__(index)

        with MemoryFile(image) as memfile:
            with memfile.open() as dataset:
                image = torch.from_numpy(dataset.read()).float()

        with MemoryFile(mask) as memfile:
            with memfile.open() as dataset:
                mask = torch.from_numpy(dataset.read()).long()

        sample = {"image": image, "mask": mask, "image_name": image_name}
        if self.transforms is not None:
            sample = self.transforms(sample)
        return sample

Expected behavior
No errors
Environment
lightning==2.2.1
lightning-cloud==0.5.64
lightning-utilities==0.10.0
lit==15.0.7
litdata==0.2.1

Additional context

I noticed in my dataloader script I was importing from lightning updating to from litdata import StreamingDataLoader appears to have resolved this issue


update to beta 12

When I attempted to update to beta.12 sometime last week, I got something like this error when I attempted to close the modal:
"Task does not support cancellation, or is already canceled."
This appears to be related to https://github.com/angular/angular/issues/7671. From what I understand it's actually an issue with zone.js.
Have you not experienced a similar issue?

I guess the "Task does not support cancellation, or is already canceled." issue was solved. I'm no longer seeing it. Thanks for the pr!

Releases as v0.4.3.


official API support

https://www.threads.net/@0xjessel/post/C1AqVMiySi6
issue to keep track of endpoints in the upcoming official API

https://developers.facebook.com/docs/threads
Any plans to support this?

thanks for showing me this, will refactor ASAP

@sleep9 the API you linked is in private beta, meaning I don't have the ability to test it in practicality. however, I'm preparing for the public release of the API and I'm setting up a branch (v2) for anyone who does have access to the API to use. when the API releases to the public, I will merge that branch into this one

Great news. Looking forward.

the official API has been released. I am closing this and publishing my changes immediately

yes, unfortunately there's not a lot to the Threads API at the moment. you can still use v1.9.0 if you want to use the old library functions. is there anything you need in specific?

Almost every endpoint is utilized. Will have maintain our own library that is a combination of the new and old libraries. Reviewing the code I am unsure which functions we can migrate to the new functions and which ones we need to maintain as documentation is lacking. Do you know where I can find the official API spec? Also do you know where I can find documentation on logging into threads using Instagrams oauth/2fa?

I would recommend not switching to v2 then. the Threads API documentation is here but it is very unfinished and references some endpoints that have a different structure than others. still, it has information on using Threads with an OAuth application

Is it possible to initialize v1.9.0 with an access token retrieved via threads oauth? Right now we are using the client.login method to authenticate...

as far as I know, no, because they're different endpoints

Yea thats unfortunate. While using v1.9.0 logins are frequently rejected which is a dealbreaker for a production app and the new threads API only supports a subset of the previous functionality.


Idea: simplify and streamline Qiskit and Azure Quantum workflow

Some ideas after discussing with @xinyi-joffre and @achocron to simplify and streamline how you can submit Qiskit/Cirq/Q# circuits to a Quantum Workspace:

Pass any kind of problem to workspace.submit:

circuit = QuantumCircuit()
... # Create your circuit here

job = workspace.submit(circuit, target="ionq.simulator", shots=1000) # This interface will be consistent between targets. We only change circuit.

This will make the interface more scalable and easier to use/explain to new users.

Make Qiskit-specific abstractions available for advanced scenarios. I.e., we keep the azure.quantum.qiskit.

Create a standardized Result object.

Result.to_qiskit_result() : Convert Result into qiskit.Result

There are a couple of downsides to this approach:

You can submit a lot of different payloads to a workspace, not only Qiskit and Cirq, but provider-specific formats and/or Optimization jobs which are not circuit based.
Even the same circuit can be serialized differently depending on the target, and only the target knows this serialization
For Qiskit and Cirq, our goal is not to ask the users to use our APIs, but to leverage the existing Qiskit/Cirq APIs, for those scenarios it already works as suggested.

As such, for now we're not planning to take this ideas and we might revisit them in the future.


Sysmon 12.01

Bump to latest version:

hopefully fixes crashes of 11.11

Seems that Github Actions are failing because unsynced mirrors for MSYS packages https://github.com/msys2/setup-msys2/issues/82

Manual test seems successful, will retry the pipeline one more time and if it fails again on packages I will merge.


GL.iNet Slate AX / Flint - Failed to update package

Failed to update the package.
404

Some of these repositories are generated by gl-inet-infra which do not exist, they're included in stock firmware too, you can ignore them.
Ill probably add a repo replacement file on first boot for cleanup.


Patch nightly build flakiness

This PR retries GCS Cache requests on BadRequest errors to eliminate the flakiness in our nightly builds.

The first attempt at running the tests failed with a Zenodo timeout, which seems weird to me. I thought we were trying to download directly from the GCS cache for the CI now?

Hmm, it looks like we added the --gcs-cache-path argument to our pytest configuration, but then didn't tell the CI to use it.

PR trying to use the GCS Cache in CI: https://github.com/catalyst-cooperative/pudl/pull/1858

Ohh I thought the GCS cache in pytest changes were buried in an EIA flakiness PR.

Is there a good reason not to just have the tests default to trying the GCS cache first all the time? If there's a local cache, that'll get priority. If the data isn't in the GCS cache, it'll fall back to pulling from Zenodo (and populate the GCS cache in the process?)

This is currently how the LayeredCache works. If the files aren't available in the local cache, then it falls back to the GCS cache if --gcs-cache-path is specified. Are you saying we should set it up, so folks don't need to specify --gcs-cache-path?


[Question] High error rate after training - why?

I'm training with a new set of two fonts and the goal is to use Tesseract to analyze individual chars (only capital letters and numbers), not entire words, but the results are far from being decent and it looks like I'm doing something wrong. Tesseract and Leptonica are installed by the scripts.
Inspired by the test set provided in this repo, I created these tif files with their correct gt.txt's:
From original binarized chars:

From two TTFs to TIF images with random text:

At the end of the data creation process I have 1869 mixed text lines.
First I ran the makefile with 10000 iterations as default, but the best error rate was high.
I thought it was a matter of needing more iterations, so I changed to 30000 but nothing got better. The following image shows the results:

Sometimes I can see the char train increasing instead of getting lower.
What am I missing here? Do I need more training data? Is my initial data not following any important concept?
I'd appreciate any help!

It's probably best to ask on the tesseract-ocr mailing list, we're only providing a frontend, your problem is caused by tesseract itself.
If it is indeed a problem with how we're laying out directories or defaults given in the Makefile, we'll be happy about a PR.

https://groups.google.com/forum/#!topic/tesseract-ocr/qiQLb2-_QlE

Well, it was me who opened that thread, hehe.
Thank you @kba!


User's guide

I am very interested in using RDFSharp Semantics to create ontologies and use them but I can not find any guidance on how to use this library.
Is there somewhere where it is explained or at least a manual of reference to the possible functions?

Dear David,
thanks for the interest in using the library.
You can start with the PDF documentation which accompanies the release.
Regards,
Marco


widen range of calendar

can we widen range i.e. starting from 1970 BS - 2099?

Date range has been extended to 1970-2099 BS


excuse me, wtf?

This is not a bug report. I just wanted to convey the feeling of happiness I had when I saw the update of the most legendary mod in the world to you, the dear mod maker. What you did was bringing tears to my eyes. Mowzies Mobs, Tinker's Construct, Twilight Forest, Dynamic Trees, Better Foliege... I think apocalypse is approaching
THANK YOU SO MUCH FOR THIS ABSOLUTELY FANTASTIC MOD :)

We appreciate it... but please please please do not open this as a bug report. Our discord is great for this sort of thing.


Reinitialize totals when ordering has been updated

At present any change to the order of totals requires a page reload. Reinitializing (deleting existing) totals and starting over would be a good option.

Published in v9.3.0.


Typemap refactor

Refactors how the type map is built to fix #20.  The type names are now defined as keys on the subclass array rather than in the model.
All of the tests pass with some slightly refactored to handle the change.
This is a breaking change.

I would expect to see a new test case the prove the problem that this change is expected to fix is truly fixed

Test case added which ensures the correct model types are instantiated when loading relationships.

Just to note on this, the failing tests on HHVM appear to be due to a bug in HHVM: https://github.com/facebook/hhvm/issues/7193
The bug causes the code to get into an infinite loop and crash.  I can work around it by checking $property->info->class if it's set as HHVM provides extra information which is correct.

@bostanio Any interest in this or anything you would like clarifying?

@alexwhitman I am interested in this...  +1

I added the test here and it passed without these changes<PHONE_NUMBER>0109bbcd4c764d975a614e8c42434a


feat: redesign landing page

:tada: This PR is included in version 0.41.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[Terminal >Set tab color]: Accessible name is not descriptive for 'Reset' button.

Windows Terminal version
1.12.3472.0
Windows build number
1.12.3472.0
Other Software
Test Environment:
OS: Windows 11 Version Dev (OS Build 22504.1010)
App: Windows Terminal Preview
Screen Reader: Narrator
Tool: Accessibility Insights for Windows Version 1.1.1741.1
Steps to reproduce
Repro Steps:

Open Windows Terminal.
Open Settings page using 'Ctr+,'.
Activate 'Actions' tab and then Add a new actions 'Set tab color' with shortcut key 'Ctrl + Shift + c'.
Open Narrator using 'Win + Ctrl + Enter' keys.
navigate inside Terminal window.
Press 'Ctrl + Shift + c' keys, then navigate to 'Reset' control inside Set tab color pop-up using tab key.
Observe the issues.

User Experience:
Screen Reader users UX will not be good if control name is not descriptive.
Guideline Reference:
https://www.w3.org/WAI/WCAG21/Understanding/name-role-value
Attachments:
Accessible name is not descriptive for 'Reset' button..zip
Expected Behavior
Accessible name should be more descriptive for 'Reset' button.
i.e. Screen Reade should announce 'Reset tab color button'.
Actual Behavior
Accessible name is not descriptive for 'Reset' button.
i.e. Screen Reade is only announcing 'Reset button'.

Closing the bug as issue has been fixed.


Home page for release 2

As described in #102 we need a catalogue home page for release 2. This catalogue home page can either be in wordpress or the catalogue and if in wordpress could technically hand it over to Ed or Ignacio to action....
If do in the catalogue then can modify the draft made in #110 to be appropriate for this release.

As mentioned in #102, this can't be done in wordpress because all catalogue pages are going to go under /catalogue now

Note "Featured" uploads are actually just 6 randomly selected uploads - for the purpose of the prototype for the working group

Have replaced the trending and stats graphs with search images for each resource type - it's too much to generate actual trending and stats graphs for the prototype and we can't really have mock ups as that could confuse the working group


added support for @keyframes for radium animation support.

This update adds support for @keyframes css that can be then used with Radium.keyframes().

Coverage increased (+0.1%) to 96.059% when pulling bf60b75c52e653726c27480a9b2398791421243f on ixaris:master into 67a5d58c73093c6b4b39d908153da8c7dba04ab7 on halt-hammerzeit:master.

Oh, good, I'm merging this.

Released 1.5.0


How is the value for lv_roller set if control by keyboard?

When controlling the lv_roller by keyboard, LV_GROUP_KEY_LEFT, LV_GROUP_KEY_RIGHT, LV_GROUP_KEY_UP, LV_GROUP_KEY_DOWN controls the selection of the roller and LV_GROUP_KEY_ENTER calls the callback but when the roller is defocus, the value is reverted to the original value. How do I set the value?
In lv_roller.c, the sel_opt_id is saved when the roller gets focus and is reverted when it lost focus because sel_opt_id != sel_opt_id_ori
else if(sign == LV_SIGNAL_FOCUS)
{
        ext->ddlist.sel_opt_id_ori = ext->ddlist.sel_opt_id;
}
else if(sign == LV_SIGNAL_DEFOCUS)
{
        /*Revert the original state*/
       if(ext->ddlist.sel_opt_id != ext->ddlist.sel_opt_id_ori)
       {
            ext->ddlist.sel_opt_id = ext->ddlist.sel_opt_id_ori;
            refr_position(roller, true);
        }
}

LV_GROUP_KEY_ENTER does not set sel_opt_id_ori. Is this intentional? If so, how is sel_opt_id_ori set?
else if (c == LV_GROUP_KEY_ENTER || c == LV_GROUP_KEY_ENTER_LONG)
{
            if(ext->ddlist.action)
                ext->ddlist.action(roller);
}

Hi,
It was a bug. It should have update sel_opt_id_ori.
I fixed it in the beta branch.
Thank you for reporting the issue!


Fix for travis build, replace reads2ref with reads2fragments

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1090/
Test PASSed.

Thanks, Michael!

Merged manually.


[index] printConsoleConfig (text mode) should print a report of the current selections

https://github.com/adobe/aio-cli-plugin-console/blob/master/src/commands/console/index.js
It shouldn't log only if the selection is set. So if nothing is set it should be:
You are currently in:
1. Org: <no org selected>
2. Project: <no project selected>
3. Workspace: <no workspace selected>

@moritzraho could you check?

taken care of by @rajarju


allow interval in holder cred rev check via ledger

Signed-off-by: sklump<EMAIL_ADDRESS>

@sklump It looks like holder/base.py should be updated to match indy.py?


Undocumented API used in one of the examples?

I'm looking at this line in one of the examples:
https://github.com/expressjs/express/blob/2c47827053233e707536019a15499ccf5496dc9d/examples/route-map/index.js#L25
It uses app[key]("/route/path", function) API which I did not find documented anywhere. The code is also quite old. Can someone shine a bit of light on it?

Right after I posted I realized that in js app["get"](route, function) == app.get(route, function). Closing the issue (I can't delete it).


Sync downsteam main with upstream main

Sync downstream main with upstream main

Dup of https://github.com/red-hat-storage/ceph-csi-operator/pull/9


Fix adding notified user w/o default events

Adding a new notified user on a job via the user interface would always
add on_success and on_failure, even if those checkboxes were not even
enabled in the form.  This commit fixes that.

Coverage decreased (-0.03%) to 65.6% when pulling 3b5252fa6e031d423ab49eecd70963676a2822bb on guidow:notified_wo_success_or_failure into d8d3d8ffab773995ad3a5c7bbd9e1ee03378b681 on pyfarm:master.

Coverage decreased (-0.03%) to 65.6% when pulling 3b5252fa6e031d423ab49eecd70963676a2822bb on guidow:notified_wo_success_or_failure into d8d3d8ffab773995ad3a5c7bbd9e1ee03378b681 on pyfarm:master.

Coverage decreased (-0.03%) to 65.6% when pulling 3b5252fa6e031d423ab49eecd70963676a2822bb on guidow:notified_wo_success_or_failure into d8d3d8ffab773995ad3a5c7bbd9e1ee03378b681 on pyfarm:master.

Coverage decreased (-0.03%) to 65.6% when pulling 3b5252fa6e031d423ab49eecd70963676a2822bb on guidow:notified_wo_success_or_failure into d8d3d8ffab773995ad3a5c7bbd9e1ee03378b681 on pyfarm:master.

Coverage decreased (-0.03%) to 65.6% when pulling 3b5252fa6e031d423ab49eecd70963676a2822bb on guidow:notified_wo_success_or_failure into d8d3d8ffab773995ad3a5c7bbd9e1ee03378b681 on pyfarm:master.

Coverage decreased (-0.03%) to 65.6% when pulling 3b5252fa6e031d423ab49eecd70963676a2822bb on guidow:notified_wo_success_or_failure into d8d3d8ffab773995ad3a5c7bbd9e1ee03378b681 on pyfarm:master.

Coverage decreased (-0.03%) to 65.6% when pulling 3b5252fa6e031d423ab49eecd70963676a2822bb on guidow:notified_wo_success_or_failure into d8d3d8ffab773995ad3a5c7bbd9e1ee03378b681 on pyfarm:master.


requests.exceptions.ConnectionError: HTTPSConnectionPool(host='nptel.ac.in', port=443): Max retries exceeded with url: /courses/106106198 (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f0c0361d490>: Failed to establish a new connection: [Errno 110] Connection timed out'))

I got a very good connection of 100Mbps don't know what is happening while getting the page using requests module.

Found this alternative.
response = urllib.request.urlopen(url).read()
soup = BeautifulSoup(response, 'lxml')

Found this alternative.
response = urllib.request.urlopen(url).read()
soup = BeautifulSoup(response, 'lxml')


ATA creation has some usability footguns

Problems

ATA (associated token account) creation can be done for non-system accounts which can lead to unrecoverable tokens (especially if an ATA is created for another ATA)
ATA creation is not idempotent which causes usability issues due to race conditions in RPC interactions

Proposed Solution
Add a new instruction for idempotent ATA creation which restricts owners to be system accounts

2. ATA creation is not idempotent which causes usability issues due to race conditions in RPC interactions

ref: https://github.com/solana-labs/solana-program-library/issues/1371

Closed in favour of https://github.com/solana-labs/solana-program-library/issues/1371


Additional options for revalidation and removing old responses

Closes #620, closes #622, closes #623

Fixes behavior with SQLite backend, cache_control=False, expire_after=0, and remove_expired_responses()
New session option:session = CachedSession(always_revalidate=True)

New remove_expired_responses() option:# Remove cached responses older than 7 days
remove_expired_responses(older_than=timedelta(days=7))

Codecov Report

Merging #624 (fd71901) into master (cf69e93) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #624   +/-   ##
=======================================
  Coverage   99.63%   99.63%
=======================================
  Files          23       23
  Lines        1654     1660    +6
  Branches      260      261    +1
=======================================
+ Hits         1648     1654    +6
  Partials        6        6

Impacted Files
Coverage Δ

requests_cache/backends/base.py
100.00% <100.00%> (ø)

requests_cache/backends/sqlite.py
100.00% <100.00%> (ø)

requests_cache/models/response.py
99.24% <100.00%> (+0.01%)
:arrow_up:

requests_cache/policy/actions.py
100.00% <100.00%> (ø)

requests_cache/policy/expiration.py
100.00% <100.00%> (ø)

requests_cache/policy/settings.py
100.00% <100.00%> (ø)

requests_cache/session.py
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 cf69e93...fd71901. Read the comment docs.

I'm still struggling to understand what "expire" means in the docs. Cache expiration, or response expiration? Imho it would be easier if they would be completely separate concepts. And are validate-only responses considered expired?

For documentation purposes, "expired" (in general caching terms) = "stale" (in HTTP caching terms). The policy for removing expired responses depends on the features available in each backend. Redis TTL, for example, is very different from MongoDB TTL (note: those are pre-release doc links, not yet released).
In that sense, cache expiration (which I refer to in the docs as "TTL," since that's typically what it's called at the database level) can be configured separately from response expiration. For SQLite, there is no native mechanism for automatically removing expired responses, so in that case there's no separate concept for "cache expiration" or TTL, aside from manually calling remove_expired_responses().
Does that make sense? I will try to think of a clearer way to explain that in the Expiration section.

And are stale, but validatable responses considered "expired" in the context of the docs?

Yes.

Does that mean
# Remove expired responses *and* responses older than 7 days
remove_expired_responses(older_than=timedelta(days=7))

will remove all stale responses that could be re-validated as well then?

will remove all stale responses that could be re-validated as well then, no matter how old they are?

Yes, you're right, now I see why that's confusing. So what you want is to remove only old responses, which that currently doesn't do. There are also some other conditions for removal that I may want to add in the future, so I'm thinking older_than should be moved to a separate method that doesn't have "expired" in the name, but as a separate keyword argument. Something like:
def remove(
    expired: bool = True,
    older_than: datetime = None,
    ...,
):
    """Remove responses according to the specified condition(s)"""

What do you think?

Yes, that looks better.
I'd also get rid of the "expire_after" option there, as it's kinda confusing that removal and mutating the cache entries happen in one function.


til::rle<T, S> - a run length encoded storage template

Introduces til::rle<T, S>. Stores a chosen type T in a run length encoded (compacted) format. For additional memory savings, the counter size S can be chosen as well.
References

#8446 - ATTR_ROW and TextAttributeRun are purposefully left behind as pass-throughs to not get in the way of the other hide/cleanup effort in this PR.
#8000 - Supports buffer rewrite work. A re-use of til::rle will be extremely useful as a column counter as we pursue NxM storage and presentation.
#3075 -  The new iterators on this allow skipping forward by multiple units, which wasn't possible under TextBufferCellIterator nor OutputCellIterator. It also allows a bulk insert in multiple ways via assign and fill. Getting stuck on inserting cell-by-cell through those iterators is part of the performance detriment left to this issue.
#8787 and #410 - High probability this should be pmr-ified like bitmap for things like chafa and cacafire that are changing the run length frequently (and probably causing many transient allocations).

PR Checklist

[ ] Closes #8741
[x] I work here.
[x] Tests added.
[ ] Tests passed.

Detailed Description of the Pull Request / Additional comments
Validation Steps Performed

[x] Ran cacafire in OpenConsole.exe and it looked beautiful
[x] Ran new suite of RunLengthEncodingTests.cpp supporting til::rle<T, S> and derived from the concepts of AttrRowTests.cpp which is obsolete/removed.

Known TODO list

[ ] - Deduplicate _at from const and non-const versions
[ ]

Known TODO list

[ ] - Deduplicate _at from const and non-const versions
[ ]

If I understand this correctly, RLE should be faster than we have now. But is there a way to measure the performance of RLE impl so that we do not have significant performance regression comparing the current ATTR_ROW? Just in case.

If I understand this correctly, RLE should be faster than we have now. But is there a way to measure the performance of RLE impl so that we do not have significant performance regression comparing the current ATTR_ROW? Just in case.

If I understand this correctly, RLE should be faster than we have now. Well at least not worse than we have now, if not faster. But is there a way to measure the performance of RLE impl so that we do not have significant performance regression comparing the current ATTR_ROW? Just in case.

It probably won't be immediately faster, but it will have the potential to unlock faster actions on behalf of its users. I made this PR such that it was taking the best parts of ATTR_ROW, genericizing them, and putting it back as a drop-in replacement for what we already had without changing the consumption style. The performance should likely be exactly the same.

If I understand this correctly, RLE should be faster than we have now. Well at least not worse than we have now, if not faster. But is there a way to measure the performance of RLE impl so that we do not have significant performance regression comparing the current ATTR_ROW? Just in case.

It probably won't be immediately faster, but it will have the potential to unlock faster actions on behalf of its users. I made this PR such that it was taking the best parts of ATTR_ROW, genericizing them, and putting it back as a drop-in replacement for what we already had without changing the consumption style. The performance should likely be exactly the same.

Requests from trying to use with new buffer:

replace(off, size, newsize, newthing) (optional -- I can construct a new one and assign, but ... will that work properly? I need to replace [one column] with [many columns]
resize(newSize, fillValue) let me specify the new fill value instead of extending the final run
insert seems more like assign. Can I insert, by pushing subsequent runs forward after the insertion point? Can I insert many? (this could be the basis for a replace where size != newSize)

What error do you get when you remove non-const-_at

Requests from trying to use with new buffer:

replace(off, size, newsize, newthing) (optional -- I can construct a new one and assign, but ... will that work properly? I need to replace [one column] with [many columns]
resize(newSize, fillValue) let me specify the new fill value instead of extending the final run
insert seems more like assign. Can I insert, by pushing subsequent runs forward after the insertion point? Can I insert many? (this could be the basis for a replace where size != newSize)

What error do you get when you remove non-const-_at

Since the new buffer works by counting columns per wchar_t, and sometimes a single character takes up more than one wchar_t, I occasionally need to replace a run that looks like 1 0 0 with 1, or a run that looks like 1 with 2 0. That's a replace+shrink/grow.
I implemented it by replacing the impacted region with 0 0 0 ... (for as many wchar_t as there are) and then replacing the first index with the true column number.
My current backing store grows as necessary when I do that, and it does not have a maximum size (it is a vector).

Since the new buffer works by counting columns per wchar_t, and sometimes a single character takes up more than one wchar_t, I occasionally need to replace a run that looks like 1 0 0 with 1, or a run that looks like 1 with 2 0. That's a replace+shrink/grow.
I implemented it by replacing the impacted region with 0 0 0 ... (for as many wchar_t as there are) and then replacing the first index with the true column number.
My current backing store grows as necessary when I do that, and it does not have a maximum size (it is a vector).

nit: maybe define a static constexpr value inside til::rle<T, S> called npos, which you can use in the signature for substr instead of numeric_limits?

nit: maybe define a static constexpr value inside til::rle<T, S> called npos, which you can use in the signature for substr instead of numeric_limits?

I can't assign from a pair of rle_const_iterators 😦
I had to add a hacky runs_cbegin and runs_cend so I could get a raw iterator over the std::pairs inside the rle so I could assign from one to the other.
I implemented replace manually in the consuming code ...
OLD:
_cwid.replace(begin, len, cadvs);

NEW:
auto old = _cwid.substr(uint16_t(begin + codeUnitDamage));
_cwid.resize(_data.size());
_cwid.fill(1, begin);
_cwid.fill(ncols, begin + (col - minDamageColumn));
_cwid.fill(0, begin + (col - minDamageColumn) + 1);
_cwid.fill(1, begin + (col - minDamageColumn) + glyph.size());
_cwid.assign(old.run_cbegin(), old.run_cend(), begin + codeUnitDamage);
_cwid.resize(_data.size());

I'm abusing fill to remake the entire end of the rle from the insertion point to the end.
Fill with 1, 1, 1, 1, 1
Fill with    2, 2, 2, 2
Fill with       0, 0, 0
Fill with          1, 1
Merge in original        2, 0, 1, 1, 1, 0

FINAL:
          1, 2, 0, 1, 1, 2, 0, 1, 1, 1, 0

I can't assign from a pair of rle_const_iterators 😦
I had to add a hacky runs_cbegin and runs_cend so I could get a raw iterator over the std::pairs inside the rle so I could assign from one to the other.
I implemented replace manually in the consuming code ...
OLD:
_cwid.replace(begin, len, cadvs);

NEW:
auto old = _cwid.substr(uint16_t(begin + codeUnitDamage));
_cwid.resize(_data.size());
_cwid.fill(1, begin);
_cwid.fill(ncols, begin + (col - minDamageColumn));
_cwid.fill(0, begin + (col - minDamageColumn) + 1);
_cwid.fill(1, begin + (col - minDamageColumn) + glyph.size());
_cwid.assign(old.run_cbegin(), old.run_cend(), begin + codeUnitDamage);
_cwid.resize(_data.size());

I'm abusing fill to remake the entire end of the rle from the insertion point to the end.
Fill with 1, 1, 1, 1, 1
Fill with    2, 2, 2, 2
Fill with       0, 0, 0
Fill with          1, 1
Merge in original        2, 0, 1, 1, 1, 0

FINAL:
          1, 2, 0, 1, 1, 2, 0, 1, 1, 1, 0

FWIW I have some "workaround" code that looks like this-

nit: maybe define a static constexpr value inside til::rle<T, S> called npos, which you can use in the signature for substr instead of numeric_limits?

std::string::npos is defined as a hard (size_t)-1 or the same as std::numeric_limits<size_t>::max().
So to me, numeric limits feels right. Unless you want me to statically define npos as a const public member to the numeric max>

nit: maybe define a static constexpr value inside til::rle<T, S> called npos, which you can use in the signature for substr instead of numeric_limits?

std::string::npos is defined as a hard (size_t)-1 or the same as std::numeric_limits<size_t>::max().
So to me, numeric limits feels right. Unless you want me to statically define npos as a const public member to the numeric max>

having a static size_type npos is the convention, so idk


Discussion: adding params for TS modules

Hi, tks you for a great plugin for Gophers.
I really like your idea of using Treesitter to extract struct/interface/method/... at the current cursor position. I wish to use those exported functions in my own plugin.
However, I might need to pass bufnr as a parameter to those functions. The current implementation will not allow that.
M.get_struct_node_at_pos = function(row, col)
  local query = M.query_struct_block .. " " .. M.query_em_struct_block
  local bufn = vim.fn.bufnr("") -- will only query at the current buffer
  -- ...
end

My idea is that we can pass bufnr as the 3rd parameter for those function. This should be optional and fallback to the current implementation if missing. I suppose this will not break any current behavior.
M.get_struct_node_at_pos = function(row, col, bufnr)
  local query = M.query_struct_block .. " " .. M.query_em_struct_block
  local bufn = burnr or vim.fn.bufnr("") -- fallback if bufnr is missing
  -- ...
end

Just wondering if you are open to updating. I will gladly make a PR for this modification

Thanks for the feedback.
I am wondering in what scenario may this been used?

I’m adding code actions for go tooling (goimpl, tag, etc…) via null-ls
The code actions handler is passed with bufnr and I think it is better to use this bufnr param.
Most of the case, I’m thinking that vim.fn.bufnr(“”) will work. Just want to avoid unexpected edge case

Thanks,  I updated a commit for this.


3.0.x:   use golang package from centos vault temporarily

(cherry picked from commit 42586f5822be59ab0e51aebd94a5f7b5f2b5b0ce)
What does this PR do?
Fixes #3087 for 3.0.x release
Which TC components are affected by this PR?

[ ] Documentation
[ ] Grove
[ ] Traffic Analytics
[x] Traffic Monitor
[x] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[ ] Traffic Router
[x] Traffic Stats
[ ] Traffic Vault
[x] Other cdn-in-a-box

What is the best way to verify this PR?
Build traffic_ops, traffic_monitor, traffic_stats
Build cdn-in-a-box
Check all that apply

[ ] This PR includes tests
[ ] This PR includes documentation updates
[ ] This PR includes an update to CHANGELOG.md
[ ] This PR includes all required license headers
[ ] This PR includes a database migration (ensure that migration sequence is correct)
[ ] This PR fixes a serious security flaw. Read more: www.apache.org/security

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2884/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2888/
Test FAILed.


Element link won't work while no page attachment

Bug Description

The element link feature only work while page define page attachment.
Expected Behaviour

The element link should work w/wo page attachement.
Steps to Reproduce

on a page without page attachment, add link on an element

preview, the element link won't work

back to editor, add a page attachment

preview again, the element link works

Screenshots

Additional Context

Plugin Version: Version 1.1.0-alpha
Operating System: Mac
Browser: Chrome

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Thanks for your report!
I see you are using our local dev environment—thanks for testing!
I really thought we had fixed this already. @miina can you take a quick look?

@lucemia I'm afraid I can't really reproduce this:

Create a new story
Add text element
Add link
Preview story
Link is there

On your step 2) from your report, when you go back to the editor after previewing the story, do you still see the link option in the Design Panel? Does it look like this:

Tried to reproduce this as well -- without luck.
The issue did exist but was fixed within https://github.com/google/web-stories-wp/pull/4323
I'm also seeing this icon on the screenshots:

That was also removed recently -- it hints as if the version used is not fully up to date. Thoughts?

The version above says 1.10-alpha, which would be the current main branch.
Maybe need to run npm run build

@miina @swissspidy
after npm run build , the issue is gone.
Sorry for misleading


ignore HttpOnly as a cookie name instead of throwing exception

I think it would make more sense to have this configurable via an constructor argument. So what about modify and update the pull request?

also merge into 3.2?

Trustin wanted to have a look at if its possible to merge it into 3.2 too. If you can write a patch that can be easily merged to 3.2 then even better ;) Thanks

pull request #50

Done.. Thanks again


Widgets get scrambled

Using Flutter stable 1.2, in release mode, we're getting strange scrambled images in some devices:

To the left you see SVG icons, created with flutter_svg, inside of a ListView. As you scroll the list and new icons come to view, the ones already in the screen get garbled. As if they are being drawn inside of the image buffer of the other one. You can even see parts of one icon inside of the other.
We make heavy use of flutter_svg (version 0.11.0+1), but we don't know if there's any relation to that package.
The screenshots above are from a Galaxy S7. Testing in a Galaxy S4 it's more difficult to reproduce it, but it occasionally happens. We have also tested in some other device where we couldn't reproduce it at all. All this happens installing via flutter build apk+adb -d install -r app-release.apk. If I install via flutter install I can't reproduce it, as of now. But it also happens less frequently in the emulator.
I don't know if that's related to issues https://github.com/flutter/flutter/issues/28484, and https://github.com/flutter/flutter/issues/22546, and https://github.com/flutter/flutter/issues/28227, and https://github.com/dnfield/flutter_svg/issues/115, but I would assume so.

[√] Flutter (Channel stable, v1.2.1, on Microsoft Windows [Version 10.0.17763.316], locale pt-BR)
• Flutter version 1.2.1 at c:\flutter
• Framework revision 8661d8aecd (2 weeks ago), 2019-02-14 19:19:53 -0800
• Engine revision 3757390fa4
• Dart version 2.1.2 (build 2.1.2-dev.0.0 0a7dcf17eb)
[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at C:\Users\Marcelo\AppData\Local\Android\sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-28, build-tools 28.0.3
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
• All Android licenses accepted.
[√] Android Studio (version 3.2)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin version 28.0.2
• Dart plugin version 181.5616
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
[√] IntelliJ IDEA Ultimate Edition (version 2018.3)
• IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA 2018.3
• Flutter plugin version 33.3.2
• Dart plugin version 183.5912.23
[√] Connected device (1 available)
• SM G930F • 874556d84d7f4d5231 • android-arm64 • Android 8.0.0 (API 26)

Hello @tvolkert , also having this issue with latest master channel.

@tvolkert Yes, unfortunately the issue is not fixed.

/cc @liyuqian can you have a look?

This is probably a duplicate of https://github.com/flutter/flutter/issues/28227


feat: add<EMAIL_ADDRESS>

Fixes for Msys2, adds new BAZEL_TOOLCHAIN_UTILS_LOCAL_TRIPLET environment variable to override host detection and Bazel 8 support.

Awaiting approval to run pre-submit as I have added Bazel 8.0.0rc1 into the matrix.


Add more courses

There's only two courses in the help guide so far. It would be great to have more courses on some of the other aspects of atto, too!
One example of a course to write is how to make subroutines using gosub/return.

I would like  to contribute can you guide me on this

Hey @HardikxLabeitazer, have you had much progress with regards to writing courses? Just that I'm considering about writing some courses myself too, and I don't want to end up writing any courses that cover the same stuff as what you've been working on!
Totally fine if you've been busy but are still planning to write a course, too.


Test endpoints for file uploads which response with JSON:API documents in v2

I have some API endpoints for file uploads, which response with JSON:API documents.
As far as I'm aware I can not use jsonApi() test builder provided by MakesJsonApiRequests to construct the requests. That one does not seem to support file upload at all. And I think support for requests, which aren't JSON:API compliant, is out of it's scope.
But using CloudCreativity\LaravelJsonApi\Testing\TestResponse is very helpful. As the endpoint response with a valid JSON:API document it makes writing the test way easier. Also the test is more inline with the rest of the test suite.
In v1 I was able to achieve what I need by just doing this:
$this->call('POST', '/v1/documents', $requestPayload, [], $files, []);
      ->assertCreatedWithId($expectedResponsePayload);

To be honest I'm not sure why this worked but it has worked.
In v2 this was not working anymore. $this->call() returns a Illuminate\Testing\TestResponse, which does not have the assertions provided by TestResponse. assertCreatedWithServerId() which replaced assertCreatedWithId in v2 wasn't available. I ended up casting the request to TestResponse manually to make it work:
TestResponse::cast(
       $this->call('POST', '/v1/documents', $requestPayload, [], $files, [])
   )
   ->willSeeResourceType($this->resourceType)
   ->assertCreatedWithServerId('http://examples.com/v1/documents', $expectedResponsePayload);

But this feels like way more boilerplate and I'm not even sure if casting a response to TestResponse yourself is considered part of the public API.
Is there a better way to do this in v2?

Hi! Yeah, sorry didn't put this change in the upgrade guide because it wasn't documented and didn't think it would be encountered by anyone except me.
Basically, in v1 the MakesJsonApiRequests trait changed the test response class by overloading the createTestResponse() method on the test class. Here it is in action:
https://github.com/cloudcreativity/laravel-json-api/blob/v1.7.0/src/Testing/MakesJsonApiRequests.php#L176-L179
In v2 I removed this because I thought it wasn't safe for a package to be doing this - it's kind of up to the developer what test response class they get on the generic Laravel test methods. As such, in v2 the Laravel JSON API test response class is only returned for JSON API requests - i.e. through the method chaining from $this->jsonApi().
If you want the Laravel JSON API test class returned for non-JSON API test requests, then overload the createTestResponse() method on your test class.
Alternatively, the Laravel JSON API test response class has a static cast() method, as you've worked out.
You're right though that it doesn't feel massively intuitive. Therefore, maybe we should add something to the $this->jsonApi() test request builder to make it easier to do a file upload and get JSON API content in the response?
Maybe:
$response = $this
    ->jsonApi()
    ->contentType('multipart/form-data')
    ->withPayload($payload)
    ->withFiles($files)
    ->post($uri);

What do you think?

Therefore, maybe we should add something to the $this->jsonApi() test request builder to make it easier to do a file upload and get JSON API content in the response?

To be honest I'm not sure how likely my use case is. It would add quiet some complexity to the library, wouldn't it? Not only for supporting file uploads but also for multipart/form-data.
I'm fine with the manual casting the response to CloudCreativity\LaravelJsonApi\Testing\TestResponse. I opened this issue mainly for two reasons:

I wasn't sure ifTestResponse::cast() is considered public API as it's not documented.
I worried that i missed something and that there is a better way to do it.

We have quite a lot of endpoints where the request content type is not JSON API, but the response content type is. So I think supporting that kind of request via the fluent test request builder is a useful thing to support. So will add it in for the next version.

Syntax for testing file uploads will be:
$this->jsonApi()->asMultiPartFormData()->withPayload($data)->post('/api/v1/avatars');
There is also a asFormUrlEncoded() method, and if any other request content type needs to be used the contentType() method can be used.
In the next major release (3.0).
Note it's not necessary to split files out from $data as I've noticed the Laravel test helpers do that already - so $data can contain a mixture of files and non-files as needed.

In the next major release (3.0).

Are you planning to release another major soon?

Yeah there's a major release in the pipeline as I accepted a PR that introduced a breaking change. It's only one change though so the upgrade path isn't very difficult. Going to sort that out to coincide with Laravel 8.


Support CTRL-F Searching in Docs Pages

Use case
I want to find the "theme" arguments to MaterialApp:

Go to the MaterialApp docs page
Press CTRL-F
Type "theme"
Press ENTER

What Happens
The browser hi-lights instances of "theme" on the page, starting with the entries under CLASSES on the left.  There are 67 instances of "theme" under CLASSES.  I feel frustrated.  This is terrible UX.
After pressing ENTER sixty-eight times, the browser finally hi-lights the first instance of "theme" in the actual MaterialApp documentation.  Then I can begin searching for the information I need.
What I Need to Happen
Searching within a Flutter documentation page should show matches in the documentation section first.
Proposal
Change the order of the content in the HTML file so the main documentation appears before the CLASSES sidebard.  Use CSS to position the sidebar on the left-hand side.

I reported this a year ago.

any update on this? Will it be on a roadmap soon?


Write a .mailmap file

Description
I was trying to make some charts about number of contributors, and was thrown off by the same person showing up multiple ways; so I started writing a .mailmap file to normalize idents.

For outside contributors who have used more than 1<EMAIL_ADDRESS>email address, I put zero effort in to determining which address they prefer; I just picked one arbitrarily.

I've gone ahead and moved myself to "Former employees"--I'm only here for another week.

This is the command I use to visually scan for idents that might be duplicates:
git log --all --format=$'%cN <%cE>\n%aN <%aE>' | sort | uniq -c

Related Issues
(none)
Testing
I've spent a lot of time looking at that above git log command looking for names that might be duplicates.
Checklist

[ ] Does my change need to be backported to a previous release? - no

What backport versions were discussed with the Maintainers in the Issue?

[ ] I made sure to update CHANGELOG.md. - no applicable changes
Remember, the CHANGELOG needs to mention:

Any new features
Any changes to our included version of Envoy
Any non-backward-compatible changes
Any deprecations

[ ] This is unlikely to impact how Ambassador performs at scale. - not a runtime change
Remember, things that might have an impact at scale include:

Any significant changes in memory use that might require adjusting the memory limits
Any significant changes in CPU use that might require adjusting the CPU limits
Anything that might change how many replicas users should use
Changes that impact data-plane latency/scalability

[ ] My change is adequately tested.
Remember when considering testing:

Your change needs to be specifically covered by tests.

Tests need to cover all the states where your change is relevant: for example, if you add a behavior that can be enabled or disabled, you'll need tests that cover the enabled case and tests that cover the disabled case. It's not sufficient just to test with the behavior enabled.

You also need to make sure that the entire area being changed has adequate test coverage.

If existing tests don't actually cover the entire area being changed, add tests.
This applies even for aspects of the area that you're not changing – check the test coverage, and improve it if needed!

We should lean on the bulk of code being covered by unit tests, but...
... an end-to-end test should cover the integration points

[ ] I updated DEVELOPING.md with any any special dev tricks I had to use to work on this code efficiently.

[ ] The changes in this PR have been reviewed for security concerns and adherence to security best practices.

I've pushed 4 new commits revising it further.


⚠ Bofrost.es has degraded performance

In c1700f6, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1357 ms

Resolved: Bofrost.es performance has improved in 69c16d1 after 6 minutes.


Link to playfair API docs from Google Search returns 404

Steps to reproduce:

From Google Search, type "charting flutter".
Click on the first link in the results (points to https://github.com/domokit/playfair which returns a 404)

The correct page to the Playfair API docs is here:
http://docs.flutter.io/flutter/playfair/playfair-library.html

@sethladd How do we get Google Search to correctly index our Flutter API docs?

I guess we could recreate the domokit/playfair repo and put a single README in there that redirects a user over to flutter.

cc @tonygentilcore can you help us create a repo in domokit and give @qchong the permissions to add a README to that repo?

I just requested removal: https://www.google.com/webmasters/tools/removals

Thanks. I think this will sort itself out over time. Going to close this, but let's continue to monitor our SEO.

@tonygentilcore no action required :)


error using makefile

Hi,
im trying to build the ghz benchmarking binary and run into some issues using the makefile:
Proto file(s)
List protocol buffer definitions
none
Command line arguments / config
List all command line arguments or config properties
none
Describe the bug
A clear and concise description of what the bug is.
/bin/bash: -c: line 1: syntax error: unexpected end of file
make: *** [setup] Error 2

To Reproduce
Steps to reproduce the behavior:
$ git clone https://github.com/bojand/ghz
$ cd  ./ghz
$ make setup
if [ ! -f /bin/tparse ]; then go get github.com/mfridman/tparse; fi;
if [ ! -f /bin/goimports ]; then go get golang.org/x/tools/cmd/goimports; fi;
go: finding golang.org/x/tools latest
if [ ! -f /bin/golangci-lint ]; curl -sfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b OME/go/bin v1.21.0
/bin/bash: -c: line 1: syntax error: unexpected end of file
make: *** [setup] Error 2

Expected behavior
A clear and concise description of what you expected to happen.
no errors - using make
Environment

OS: [e.g. macOS 10.14.3]: osx 10.14.6 (18G95)
ghz: [e.g. 0.33.0]
golang: go version go1.13.7 darwin/amd64 - no GOPATH set

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
>>Additional questions<<
1.) is there a way to use multiple binary data files during the benchmark?
ghz --protoset ./bundle.protoset \
  --call helloworld.Greeter.SayHello \
  -B ./hello_request_data.bin \
  <IP_ADDRESS>:50051

also assume greeter.proto included another proto def (e.g. import "tensor.proto"), how to add them here? - I guess that's the right way:protoc --proto_path=. --descriptor_set_out=bundle.protoset *.proto
2.) how can I print the grpc response to stdout?
3.) is there a way to output the serialized binary message which is send when I use "data":{} and define my json for the bundle.protoset?
4.) how can I set a fixed qps for the benchmark e.g. (10qps)

Hello, Makefile should be working better now.

There is no way to use multiple binary files for input data. protoset is a format for bundling multiple proto files into a single binary.
This can be achieved using --debug.
This does not really make sense. See answer in 1.
Use --qps.


Question: :pseudo + sibling

Is it possible to something like http://codepen.io/sarbbottam/pen/YGjXjq?editors=0100 with glamor?
If so, a working example would be great.
Thanks!

sure, something like this should work.
let rule = {
  opacity: 0,
  '& + label': {
    position: 'relative'
  },
  '& + label:before, & + label:after': {
    content: '""',
    position: 'absolute'
  }
  // etc...
}

// ...
<input type="checkbox" id="toggle" className={rule} />
<label htmlFor="toggle">Toggle </label>

Awesome. I would give it a try.
Would you consider to showcase such complex use case in the readme/docs? I would be happy to raise a PR.

Well... I've started a complete rewrite of the docs, mind if I ping you when I have something to share there? Would be happy to take docs PRs into that.

Not at all. Would love to contribute to this awesome project.
I have been looking around for the perfect CSS solution. Wasn't happy with the existing ones, so created Sheer. It is good but far from my desired one, no scoping, specificity issues. I looked at atomizer, which addressed those issues but would create a readability concern with the example I provided above. I loved css-module, but it does not seem to be in active development, tried few more ... and then gave up, went back to Sheer.
I did come across glamor earlier but did not quite follow/like the syntax
<div
  {...style({ color: 'red' })}
  {...hover({ color: 'pink' })}>
>

Visited again a while back, directed from Next.js. And guess what, the current syntax is so intuitive. I guess this is the CSS Solution rather Presentation Solution I have been looking for.
More than happy to contribute, if I may be of any help.
Thanks for reading!

Sheer looks great! I'll loop you in if I need any help, thanks! Lemme know how using glamor goes for you.

Did anybody get this method to work?  Trying to style a checkbox label but having difficulty implementing.

So, as I've recently discovered content: '', and  content: '""', are not the same. content: '', will not work, since JS needs quotes in a string literal. I struggled with this for a good half hour before tweaking the right knob.
(This may be a good gotcha to note in the readme?)
Hope this helps!

Wow @jguffey Took me forever to find this post, but so glad I did. Had the exact same problem. Thank you!