[5.5] remove factory comment

this comment seems like overkill, because it’ll show up in every
generated factory.
people can read the docs to learn about factories

i disagree, it's the purpose of stubs to help to create and understand things,
it need to be commented. Just create a factory from scratch, they are easy to write, if you do not want to delete this comment each time.

this is literally the only stub that is this overly verbose.
it is the purpose of documentation to help us understand things.
stubs are to simplify things we do a hundred times, and keep them consistent.
should we add a comment to all of our generated migrations that say, 'Migrations are how you define your table structure.' or to our seeders that say 'Seeders generate dummy data for your development environment.'? Probably not, because these are high level concepts that you learn in 30 seconds .
so your suggestion is I write it from scratch, which would take more time than deleting the lines?

i'm always writting from scratch, or copy pasting from an existing one, it's far more rapid than going to the console to generate it and then rewrite it because it don't suits your needs.

On this point, I would be glad to re-introduce /** @var \Illuminate\Database\Eloquent\Factory $factory */

@lucasmichot that is a must-have.

Oh yeah @deleugpn

That @var stuff is so gross.


Bump trunk to v0.13.1

Motivation for this change
Bump trunk version to v0.13.1.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

commit message should be trunk: 0.10.0 -> 0.13.1 instead

Result of nixpkgs-review pr 138263 at 813d00c6 run on x86_64-linux 1

1 package failed to build:

trunk

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

also didn't realize this is a duplicate of #137972


Docs/qbittorrent qbitrr new 3rd party tool

Pull Request
Purpose
Update qBitrr doc to reflect recent updates
Requirements

[x] These changes meet the standards for contributing.
[x] I have read the code of conduct.

All done

Not really, you removed the link to ombi and overseerr.
just add after the link: {:target="_blank" rel="noopener noreferrer"}
this was when they click on the link it will open on a new page/tab etc

Updated as explained

Should be all good now


Support Ecto Repo

Hello!
Ecto has a nice sandbox that allows us to run database hitting tests concurrently when using Ecto's connection pool.
It'd be nice to be able to use this with Rihanna. Currently the library is hard-coded to use a dedicated Postgrex connection, however if the user was able to pass a connection the user could pass an Ecto repo and thus make use of the sandbox.
This would also remove the need for the Rihanna.Job.Postgrex in many cases, halving the number of database connections that Rihanna uses. Configuration could be added to prevent this process from being started if desired.
In addition it would also enable the user to enqueue more jobs and perform business login with the same SQL transaction as the connection will be shared across both automatically with Ecto, enabling a little more safety.
Here's some prior art on adding Ecto integration to a SQL executing lib without coupling tightly https://github.com/lpil/yesql/commit/e692828c572781d75f3575a6910eb2848cc6d1ce
I will implement this feature if given the thumbs up.
Cheers,
Louis

Alternatively it could be specified via mix config, as with the other configuration in Rihanna.
By default it continues to behave as it does today, however this config could be specified like so:
# A named postgrex connection is used
config :rihanna,
  enqueue_postgres_connection: {:postgrex, :my_postgrex_connection}

# A named postgrex connection is used
config :rihanna,
  enqueue_postgres_connection: {:ecto, My.Repo}

In either of the above cases the Rihanna.Job.Postgrex process is not started.

Interesting idea. From what I understand, this is only for the process that enqueues jobs, is that correct?

What do you mean? :)

I mean, it's for Rihanna.enqueue - not the dispatcher?

This seems like a good idea. Feel free to implement :)

Do you like the config design above?

Are you sure it can't be passed in at runtime?

Yeah it looks ok.

It could be passed at runtime for sure.
I suggested that API because Rihanna doesn't currently accept configuration beyond the Postgrex config at runtime so I thought that would be more in keeping with the current API.


Unable to keep settings persistent for mAutoPause=False

Describe the Bug
The settings.ini file in the /gamefiles/FactoryGame/Saved/Config/LinuxServer is overwritten with default settings frequently.  This prevents the server setting mAutoPause=False from working which means the game pauses when the last person leaves.  We would like the game to continue to run even if no person is actively connected.   It appears that whenever the container starts the ServerSettings.ini file is overwritten with default values, resetting the mAutoPause to true.   If I try to make the file read only then the container fails to fully start.
Looking for a way to ensure that the configuration files are not overwritten each time the container restarts.
Your Runtime Command or Docker Compose File
version: '3'
services:
    satisfactory-server:
        container_name: 'satisfactory-server'
        hostname: 'satisfactory-server'
        image: 'wolveix/satisfactory-server:latest'
        ports:
            - '7777:7777/udp'
            - '15000:15000/udp'
            - '15777:15777/udp'
        volumes:
            - '/mnt/d/satisfactory:/config'
        environment:
            - MAXPLAYERS=8
            - PGID=1000
            - PUID=1000
        restart: unless-stopped

System Specs :
Windows 11 running Docker on WSL2

Hey :) As is explained in the README, the config files are overwritten with custom ones to improve the default server's performance. So to overwrite the commonly used values, we have a number of environment variables. Simply add AUTOPAUSE=false to your environment variables and you'll be set.


'dcos package repo add': Default to prepending (--index=0) rather than appending?

Feels like if someone's adding their own custom repo, default behavior should be to give that new repo higher priority than Universe.

This is cosmos behavior. We decided to have universe packages take precedence unless explicitly set not to.


enforce providerTags to be set when pending pact enabled

build failure due to https://github.com/pact-foundation/pact-jvm/issues/1422


WIP: Move state-modifying code to QS.update()

Make QS.make_query() read-only and ensure that consecutive make_query() calls generate results from the same state.

@tungen Would you like to help review this branch so it can be merged?

@lsc36 Should we merge now?


Add helper func for testimage import

Allows migration tests to be run by themselves
Signed-off-by: Michael McCracken<EMAIL_ADDRESS>

Ah, thank you!


Run cloud tests in series to avoid wfID conflict

Run cloud tests in series to avoid workflow ID conflict. When running parallel integration tests against the same cloud namespace it is possible if the same test runs at the same time the workflow IDs will conflict causing test failure.
example:
https://github.com/temporalio/sdk-go/actions/runs/7049993619/job/19189705118
Note: this isn't a perfect, the only way to properly fix is to run in separate namespaces or modify all the tests to add a UUID to every workflow and child workflow ID.

This only runs the TestBasic test, so you can probably just modify that one test
If we see this failure any more I'll do that


500 Server Error - No application encryption key has been specified

Describe the bug
Getting a 500 Server Error after pulling the latest image.
Logs from container show this:
[2023-10-04 00:43:29] production.ERROR: No application encryption key has been specified. {"exception":"[object] (Illuminate\\Encryption\\MissingAppKeyException(code: 0): No application encryption key has been specified. at /var/www/html/vendor/laravel/framework/src/Illuminate/Encryption/EncryptionServiceProvider.php:79)

#0 /var/www/html/vendor/laravel/framework/src/Illuminate/Support/helpers.php(307): Illuminate\\Encryption\\EncryptionServiceProvider->Illuminate\\Encryption\\{closure}()

#1 /var/www/html/vendor/laravel/framework/src/Illuminate/Encryption/EncryptionServiceProvider.php(77): tap()
#2 /var/www/html/vendor/laravel/framework/src/Illuminate/Encryption/EncryptionServiceProvider.php(60): Illuminate\\Encryption\\EncryptionServiceProvider->key()
#3 /var/www/html/vendor/laravel/framework/src/Illuminate/Encryption/EncryptionServiceProvider.php(32): Illuminate\\Encryption\\EncryptionServiceProvider->parseKey()
#4 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(908): Illuminate\\Encryption\\EncryptionServiceProvider->Illuminate\\Encryption\\{closure}()
#5 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(795): Illuminate\\Container\\Container->build()
#6 /var/www/html/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(937): Illuminate\\Container\\Container->resolve()
#7 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(731): Illuminate\\Foundation\\Application->resolve()
#8 /var/www/html/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(922): Illuminate\\Container\\Container->make()
#9 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(1066): Illuminate\\Foundation\\Application->make()
#10 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(982): Illuminate\\Container\\Container->resolveClass()
#11 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(943): Illuminate\\Container\\Container->resolveDependencies()
#12 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(795): Illuminate\\Container\\Container->build()
#13 /var/www/html/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(937): Illuminate\\Container\\Container->resolve()
#14 /var/www/html/vendor/laravel/framework/src/Illuminate/Container/Container.php(731): Illuminate\\Foundation\\Application->resolve()
#15 /var/www/html/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(922): Illuminate\\Container\\Container->make()
#16 /var/www/html/vendor/laravel/framework/src/Illuminate/Foundation/Http/Kernel.php(255): Illuminate\\Foundation\\Application->make()
#17 /var/www/html/vendor/laravel/framework/src/Illuminate/Foundation/Http/Kernel.php(213): Illuminate\\Foundation\\Http\\Kernel->terminateMiddleware()
#18 /var/www/html/public/index.php(55): Illuminate\\Foundation\\Http\\Kernel->terminate()
#19 {main}

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Environment (please complete the following information):

OS: [e.g. Ubuntu]
Architecture: [e.g. arm64, amd64]
Browser [e.g. chrome, safari]
Version [e.g. v1]

Screenshots
If applicable, add screenshots to help explain your problem.
Logs
If applicable, check the logs for any error that might of occurred.
Additional context
Add any other context about the problem here.

I've been having the same issue since v0.12.2 (and now on v0.13.1).  I'm running Docker version 24.0.6 on Debian 10 buster.

@colin-hall in either your mounted directory or volume there is a .env file. Can you delete it and restart the container?
Also worth noting, if you have vars in there you want to keep (i.e. SMTP config) copy the contents out and then add them to the newly generated file.

@alexjustesen Removing the .env file did the trick.  Starting up the container generated a differently-formatted .env file and I was able to plug in my custom SMTP values and everything's working.
Thank you for your help and for the great work on this project!

@Jbliske can you verify you're on v0.13.1? This version has a fix for the missing app key cache.
Also you can manually set a key following the faq doc section.

I am running with the Latest tag; just updated the container this evening.
I followed the instructions you provided and it worked.  They APP_KEY was missing entirely so I added it with the key.

@Jbliske it's also likely you have an old .env file which is enforcing a bad cache. I also recommend nuking it and letting it create a new one on startup.


Integrate OGE subplant id cleaning methodology into PUDL

After scoping in #2400 we've decided to integrate the OGE cleaning methodology for pudl subplant ids directly rather than try and reinvent it / directly integrate the BGA table with the EPACAMD-EIA crosswalk. At least for now.

One minor point on integrating the OGE subplant_id refinements into PUDL. I've been working with the OGE crosswalk data and have noticed that in some cases emissions_unit_id_epa is missing for retired combustors, I believe this is a result of the performance optimization in singularity-energy/open-grid-emissions#239 (first bullet of 'Other updates').
I would propose two (hopefully) minor modifications to OGE's method:

Include all required fields required for connecting EIA and CAMD data for all existing and retired generators/units for which the underlying EPA crosswalk has connections. I think those fields would be plant_id_eia, plant_id_epa, generator_id, emissions_unit_id_epa, and subplant_id.
Make subplant_id s stable across years. That this is not the case is mentioned in the other bullets in the above referenced PR that describes  the subplant_id.

I think these two are related and so that addressing one should address the other as well.


Support kubernetes version 1.14

swagger.json is removed in kubernetes version 1.14

kube-apiserver now only aggregates openapi schemas from /openapi/v2 endpoints of aggregated API servers. The fallback to aggregate from /swagger.json has been removed. Ensure aggregated API servers provide schema information via /openapi/v2 (available since v1.10)

This PR upgrades kubernetes-client version to 6.12.0, which supports 1.14

Pull Request Test Coverage Report for Build 2927

29 of 31 (93.55%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 90.051%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

data-access-layer/eventmesh/ApiServerClient.js
29
31
93.55%

Totals

Change from base Build 2924:
-0.01%

Covered Lines:
9170

Relevant Lines:
9843

💛  - Coveralls


Missing components

Describe the bug
The latest commit has deleted a bunch of components and code references, this does not seem intentional / is not documented anywhere.
Ragas version: commit/bad3a8e1d115e69bb4acbd9887ca5b6cb6a03fba
Python version: 3.10
e.g. langchain/ package is entirely missing

@jjmachan

hey welcome to the community :)
really sorry about that but we are removing these extra modules in preparation for our v0.1 release which is a major revamp. While the happy case has been kept migration free since a lot of internals have changed which will need to be migrated.
I would love to help you migrate (by debugging on call) this since you are on the dev branch and hopefully it will get you unblocked and will help we write the migration docs for others as well - what do you think

@jjmachan hey sure that’s okay. For starters, how is the EvaluatorChain migrated (from the missing langchain package)?
from ragas.langchain.evalchain import RagasEvaluatorChain
from ragas.metrics import (
    faithfulness,
    answer_relevancy,
    context_precision,
    context_recall,
)

# create evaluation chains
faithfulness_chain = RagasEvaluatorChain(metric=faithfulness)
answer_rel_chain = RagasEvaluatorChain(metric=answer_relevancy)
context_rel_chain = RagasEvaluatorChain(metric=context_precision)
context_recall_chain = RagasEvaluatorChain(metric=context_recall)

hey @nils-hde could you try this notebook https://colab.research.google.com/drive/1C1Epju1lVkXTQi2jBq1njrOrmkfg0eQS?usp=sharing
I know it's too late - sorry about that


Reusable retry script for CI

We occasionally see problems in CI where a job fails due to a transient connectivity issue. This introduces a reusable retry script with the goal of making failure-prone CI steps more robust.

This change is


Fixed documentation for ResourceMetricCollector.clear() function

The docstrings of ResourceMetricCollector referred to the clear() function as reset(), which is not existing.
Therefore, I replaced each mention of reset() with clear() in the documentation.
Issue Type

Improvement/feature implementation

Runtime Environment

Operating system and version: [e.g. Ubuntu 20.04 LTS / Windows 10 Build 19043.1110]
Terminal emulator and version: [e.g. GNOME Terminal 3.36.2 / Windows Terminal 1.8.1521.0]
Python version: [e.g. 3.7.2 / 3.9.6]
NVML version (driver version): [e.g. 460.84]
nvitop version or commit: [e.g. 0.10.0 / 0.10.1.dev7+ga083321 / main@75ae3c]
python-ml-py version: [e.g. 11.450.51]
Locale: [e.g. C / C.UTF-8 / en_US.UTF-8]

Description
Within the ResourceMetricCollector class, the clear() function was not used in the documentation for the class, nor the docstring of the function itself, but reset().
As the reset() function is not existing for ResourceMetricCollector, I replaced all occurrences of it in the documentation with clear() as well as changing the description of the clear() function to reflect it's name better.
Motivation and Context

The motivation is, that I was looking at the documentation and tried to use the reset() function for a ResourceMetricCollector instance, and encountered an error.
I had to look at the code to see that the function I have to use is called clear().
Testing
As I only changed the docstrings within the ResourceMetricCollector class, I did not test my changes.

Hi @MyGodItsFull0fStars, thanks for raising this. Would you mind also add an alias above the method to make both name work?
class ResourceMetricCollector:
    ...

    def clear(self, tag: str | None = None) -> None:
        ...

    reset = clear

    ...

Hi @XuehaiPan, thank you for the quick reply.
Sure, will gladly do so.
I will push your suggestions once I tested both variants.

@XuehaiPan I just pushed the commit that includes the reset alias for the clear function in ResourceMetricCollector.

Hey @XuehaiPan, here is a friendly reminder that I have added the requested alias for reset and the code should now be fine.


Remove XPMan from DUnitX.Loggers.GUI.VCL.pas

Because it creates a resource duplicate with Delphi auto-generated manifests.
Builind a project that contains this unit leads to this warning :
[dcc32 Conseil] H2161 Warning: Duplicate resource:  Type 24 (user-defined), ID 1; File D:\Code\delphi\projets\nextis\ui\Neptis.res resource kept; file c:\program files (x86)\embarcadero\studio\20.0\lib\Win32\release\WindowsXP.res resource discarded.
Removing XPMan solve the problem.

Fixed.


Nexus Keycode Small: remove "update credit"

This message type is not actually supported in Nexus Keycode embedded code, and we are reserving the message type for potential future uses.
Contribution Guideline Acceptance

[x] I have read and understand the Individual Contributors License Agreement.
[x] I accept the Individual Contributors License Agreement on the entire contents of this pull request.

Attribution
This PR template derived from:
https://github.com/skcript/PR-Template
Original template is:
Copyright (c) 2017 Skcript
Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
in the Software without restriction, including without limitation the rights
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
copies of the Software, and to permit persons to whom the Software is
furnished to do so, subject to the following conditions:
The above copyright notice and this permission notice shall be included in all
copies or substantial portions of the Software.

 Remove 'update credit' from smallpad and smallpad spec


LPC1766 support?

Hi, i'm trying to program an LPC1766 mcu, flashing it with a J-Link programmer, but i always receive a "Writing target memory failed" error from flash process (platformio run --target upload). My J-Link programmer is correctly working as I can flask other Cortex-M3 (STM32) mcu(s) without problems.
Could please you help me? I can give all my help, in order to create new board support for this particular mcu model.
All needed files and output are attached below.
Thank you in advance

I've tried to add a new board definition for LPC1766 (adapting the one for LPC1768, and changing maximum flash size to 208 Kbytes [256-48]):

{   "build": {     "cpu": "cortex-m3",     "f_cpu": "100000000L",     "mcu": "lpc1766",     "core": "arduino"   },   "connectivity": [   ],   "debug": {     "jlink_device": "LPC1766",     "svd_path": "LPC176x5x_v0.2.svd",     "tools": {       "cmsis-dap": {         "onboard": true,         "server": {           "arguments": [             "pyocd-gdbserver.py",             "-t",             "lpc1766"           ],           "executable": "$PYTHONEXE",           "package": "tool-pyocd"         }       }     }   },   "frameworks": [     "arduino"   ],   "name": "NXP LPC1766",   "upload": {     "maximum_ram_size": 32736,     "maximum_size": 212992,     "protocol": "mbed",     "protocols": [       "jlink",       "blackmagic",       "cmsis-dap",       "mbed"     ]   },   "url": "https://www.nxp.com/docs/en/data-sheet/LPC1769_68_67_66_65_64_63.pdf",   "vendor": "NXP" }

This is the content of my platformio.ini file:

[env:nxp_lpc1766] platform = nxplpc-arduino-lpc176x board = nxp_lpc1766 framework = arduino upload_protocol = jlink

and this is the full output from platformio run --target upload:

Processing nxp_lpc1766 (platform: nxplpc-arduino-lpc176x; board: nxp_lpc1766; framework: arduino)
Verbose mode can be enabled via -v, --verbose option
CONFIGURATION: https://docs.platformio.org/page/boards/nxplpc-arduino-lpc176x/nxp_lpc1766.html
PLATFORM: NXP Arduino LPC176x 0.1.0 #f6eba86 > NXP LPC1766
HARDWARE: LPC1766 100MHz, 31.97KB RAM, 208KB Flash
DEBUG: Current (cmsis-dap) On-board (cmsis-dap) External (blackmagic, jlink)
PACKAGES: framework-arduino-lpc176x 0.1.3, toolchain-gccarmnoneeabi 1.80201.181220 (8.2.1)
LDF: Library Dependency Finder -> http://bit.ly/configure-pio-ldf
LDF Modes: Finder ~ chain, Compatibility ~ soft
Found 10 compatible libraries
Scanning dependencies...
No dependencies
Building in release mode
Checking size .pio/build/nxp_lpc1766/firmware.elf
Advanced Memory Usage is available via "PlatformIO Home > Project Inspect"
DATA:    [          ]   3.5% (used 1133 bytes from 32736 bytes)
PROGRAM: [          ]   4.2% (used 9004 bytes from 212992 bytes)
Configuring upload protocol...
AVAILABLE: blackmagic, cmsis-dap, jlink, mbed
CURRENT: upload_protocol = jlink
Uploading .pio/build/nxp_lpc1766/firmware.bin
SEGGER J-Link Commander V6.54c (Compiled Nov  7 2019 17:05:53)
DLL version V6.54c, compiled Nov  7 2019 17:05:41
J-Link Command File read successfully.
Processing script file...
J-Link connection not established yet but required for command.
Connecting to J-Link via USB...O.K.
Firmware: J-Link ARM V8 compiled Nov 28 2014 13:44:46
Hardware version: V8.00
S/N: -1
License(s): RDI,FlashDL,FlashBP,JFlash,GDB
VTref=3.261V
Target connection not established yet but required for command.
Device "LPC1766" selected.
Connecting to target via SWD
Found SW-DP with ID 0x2BA01477
Scanning AP map to find all available APs
AP[1]: Stopped AP scan as end of AP map has been reached
AP[0]: AHB-AP (IDR: 0x24770011)
Iterating through AP map to find AHB-AP to use
AP[0]: Core found
AP[0]: AHB-AP ROM base: 0xE00FF000
CPUID register: 0x412FC230. Implementer code: 0x41 (ARM)
Found Cortex-M3 r2p0, Little endian.
FPUnit: 6 code (BP) slots and 2 literal slots
CoreSight components:
ROMTbl[0] @ E00FF000
ROMTbl[0][0]: E000E000, CID: B105E00D, PID: 002BB000 SCS
ROMTbl[0][1]: E0001000, CID: B105E00D, PID: 002BB002 DWT
ROMTbl[0][2]: E0002000, CID: B105E00D, PID: 002BB003 FPB
ROMTbl[0][3]: E0000000, CID: B105E00D, PID: 002BB001 ITM
ROMTbl[0][4]: E0040000, CID: B105900D, PID: 002BB923 TPIU-Lite
ROMTbl[0][5]: E0041000, CID: B105900D, PID: 002BB924 ETM-M3
Cortex-M3 identified.
PC = 1FFF0BA4, CycleCnt = 00038074
R0 = 7FFF8FDF, R1 = 10000124, R2 = 00000105, R3 = 40084000
R4 = 2009C000, R5 = 00000002, R6 = 40008000, R7 = 00000000
R8 = 4002C000, R9 = 2C6F4C01, R10= A4ED0126, R11= 95FC3191
R12= 1CCE656C
SP(R13)= 10007FB8, MSP= 10007FB8, PSP= B8BF109C, R14(LR) = 1FFF0CBF
XPSR = A1000000: APSR = NzCvq, EPSR = 01000000, IPSR = 000 (NoException)
CFBP = 00000000, CONTROL = 00, FAULTMASK = 00, BASEPRI = 00, PRIMASK = 00
FPU regs: FPU not enabled / not implemented on connected CPU.
Downloading file [.pio/build/nxp_lpc1766/firmware.bin]...
Writing target memory failed.
Reset delay: 0 ms
Reset type NORMAL: Resets core & peripherals via SYSRESETREQ & VECTRESET bit.
Reset: Halt core after reset via DEMCR.VC_CORERESET.
Reset: Reset device via AIRCR.SYSRESETREQ.
Script processing completed.
========================= [SUCCESS] Took 1.70 seconds =========================

It seems Writing target memory failed. would indicate its trying to write to outside the available flash region,
This framework is relatively generalised but was designed for Marlin and I'm not aware of any lpc1766 boards? are you intending to use it for your own firmware? the official mbed framework will be more generally compatible.
If you would like to attach the firmware.elf and bin, or make a PR with your modifications, I can have a look to see if I can help when I have time.

Thank you for the answer. The board is an hardware already done by a customer, and since we need to reduce time-to-market, i was trying to speedup development time using arduino framework and libraries.
Anyway, I'm doing some further investigations using other frameworks and programming hardware (building a BlackMagicProbe with a BluePill board). Seems there is the same problem using mbed framework (trying to upload a simple "blink"). So, the issue is still in the flashing process.
I'll give you a report asap.

Although this is oldish...  I see S/N: -1
This looks to be a clone J-LINK and SEGGER software will notice this and produce all sorts of weird errors, without actually telling you why. It seems to be "copy protection". I've even had the SEGGER software blank the chip in the J-LINK clone.


An error occurred building android apk with gradle, When I use feaweedfs for storage

Repetition steps:

system linux centos or ubuntu
environment prepare
a) install jdk1.8, ndk18, Android SDK 29
b) add environment variables
mount seaweed dir & cd
wget https://github.com/wildfirechat/android-chat/archive/refs/heads/master.zip & unzip master.zip
cd android-chat-master & ./gradlew build assembleDebug --info --stacktrace
i got the error

Execution failed for task ':chat:packageDebug'.

A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
java.util.zip.ZipException: invalid code lengths set

this is worked fine on local disk or NFS
thanks for any help

2. environment prepare
a) install jdk1.8, ndk18, Android SDK 29
b) add environment variables

Please add exact scripts.

what is the SeaweedFS version?

what is the SeaweedFS version?

please upgrade first.

Updates?

my colleagues are working on it
but i donot think the version is the problem, it is possible there is no impletemented about the system filelock on seaweedfs mount...

please try the new version. it has a lot of good fixes.


Feature/Balance-Operations

Coverage increased (+0.2%) to 85.321% when pulling a26d419341401c3a0e026119d8f76f6846f9b757 on Feature/Balance-Operations into b1f111097b7243cae3746d4f4e7b8816661d69cf on V2.


Cannot type in dialog

(NOTE: I did a flutter upgrade from  3.19.1 from 3.19.0, reran and it worked)
Steps to reproduce

Start Flutter App in Intellij (not debugging)
Show dialog for text entry
Receive stack trace of error

Expected results
Ability to enter text in text field
Actual results
Nothing shows
Code sample
Code sample
import 'package:flutter/material.dart';
import 'package:flutter/services.dart';
import 'package:flutter_riverpod/flutter_riverpod.dart';

import '../providers.dart';

typedef NameCallBack = void Function(String?);

class NewItemDialog extends ConsumerStatefulWidget {
  final String title;
  final NameCallBack callBack;

  const NewItemDialog({super.key, required this.title, required this.callBack});

  @override
  ConsumerState createState() => _NewItemDialogState();
}

class _NewItemDialogState extends ConsumerState<NewItemDialog> {
  late TextEditingController textController;

  @override
  void initState() {
    super.initState();
    textController = TextEditingController(text: '');
  }

  @override
  void dispose() {
    textController.dispose();
    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return CallbackShortcuts(
      bindings: {
        const SingleActivator(LogicalKeyboardKey.keyY, meta: true): oKAction,
        const SingleActivator(LogicalKeyboardKey.keyC, meta: true): cancel,
      },
      child: AlertDialog(
        title: Text(widget.title),
        content: SizedBox(
          width: 150,
          height: 80,
          child: TextField(
            // decoration: const InputDecoration(border: InputBorder.none),
            keyboardType: TextInputType.text,
            autofocus: true,
            maxLines: 1,
            cursorColor: Colors.black,
            controller: textController,
            onSubmitted: (value) {
              oKAction();
            },
          ),
        ),
        actions: [
          TextButton(
              onPressed: () {
                cancel();
              },
              child: const Text('Cancel')),
          TextButton(
              onPressed: () {
                oKAction();
              },
              child: const Text('New')),
        ],
      ),
    );
  }

  void cancel() {
    final dialogState = ref.read(dialogStateProvider);
    dialogState.newItemDialogShowing = false;
    Navigator.pop(context);
    widget.callBack(null);
  }

  void oKAction() {
    final dialogState = ref.read(dialogStateProvider);
    dialogState.newItemDialogShowing = false;
    Navigator.pop(context);
    widget.callBack(textController.text);
  }
}

Screenshots or Video

Screenshots / Video demonstration

Logs
Logs
======== Exception caught by services library ======================================================
The following assertion was thrown during a platform message callback:
A KeyDownEvent is dispatched, but the state shows that the physical key is already pressed. If this occurs in real application, please report this bug to Flutter. If this occurs in unit tests, please ensure that simulated events follow Flutter's event model as documented in `HardwareKeyboard`. This was the event: KeyDownEvent#b941e(physicalKey: PhysicalKeyboardKey#f2ed2(usbHidUsage: "0x000700e3", debugName: "Meta Left"), logicalKey: LogicalKeyboardKey#e3a44(keyId: "0x200000106", keyLabel: "Meta Left", debugName: "Meta Left"), character: null, timeStamp: 540:07:08.776294)
'package:flutter/src/services/hardware_keyboard.dart':
Failed assertion: line 505 pos 16: '!_pressedKeys.containsKey(event.physicalKey)'

Either the assertion indicates an error in the framework itself, or we should provide substantially more information in this error message to help you determine and fix the underlying cause.
In either case, please report this assertion by filing a bug on GitHub:
  https://github.com/flutter/flutter/issues/new?template=2_bug.yml

When the exception was thrown, this was the stack:
#2      HardwareKeyboard._assertEventIsRegular.<anonymous closure> (package:flutter/src/services/hardware_keyboard.dart:505:16)
#3      HardwareKeyboard._assertEventIsRegular (package:flutter/src/services/hardware_keyboard.dart:520:6)
#4      HardwareKeyboard.handleKeyEvent (package:flutter/src/services/hardware_keyboard.dart:643:5)
#5      KeyEventManager.handleRawKeyMessage (package:flutter/src/services/hardware_keyboard.dart:1167:37)
#6      BasicMessageChannel.setMessageHandler.<anonymous closure> (package:flutter/src/services/platform_channel.dart:235:49)
#7      _DefaultBinaryMessenger.setMessageHandler.<anonymous closure> (package:flutter/src/services/binding.dart:603:35)
#8      _invoke2 (dart:ui/hooks.dart:344:13)
#9      _ChannelCallbackRecord.invoke (dart:ui/channel_buffers.dart:45:5)
#10     _Channel.push (dart:ui/channel_buffers.dart:135:31)
#11     ChannelBuffers.push (dart:ui/channel_buffers.dart:343:17)
#12     PlatformDispatcher._dispatchPlatformMessage (dart:ui/platform_dispatcher.dart:737:22)
#13     _dispatchPlatformMessage (dart:ui/hooks.dart:257:31)
(elided 2 frames from class _AssertionError)
====================================================================================================

Flutter Doctor output
Doctor output
➜  note_master git:(main) ✗ fdoctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.19.0, on macOS 14.2.1 23C71 darwin-arm64, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.1)
[✓] Chrome - develop for the web
[!] Android Studio (version unknown)
    ✗ Unable to determine Android Studio version.
[✓] Android Studio (version 2023.1)
[!] Android Studio (version unknown)
    ✗ Unable to determine Android Studio version.
[✓] Android Studio (version 2022.3)
[✓] IntelliJ IDEA Ultimate Edition (version 2023.3.4)
[✓] VS Code (version 1.84.2)
[✓] Connected device (3 available)
[✓] Network resources

! Doctor found issues in 2 categories.
➜```

</details>

This only occured the 1x and only from Intellij. I will say that Intellij gets messed up when I leave the computer asleep for awhile. (I have 3 monitors which might be causing a problem). We can close this as I've only seen it the 1x time. Restarting fixed the problem.


Packaging: clean up prior run node_modules before install

@keybase/react-hackers CC @gabriel

we actually need to clean up the other node_modules, not just under shared.
During the build process we create desktop/dist desktop/build desktop/release. All those should be wiped

I think we clear those out in the build script already, so I think this is ok

Checked that after a local build run there are no files left over inside the client/ dir.  (There are some in /tmp though.)

@chrisnojima Here's where we del() those dirs at the start of a new run:
https://github.com/keybase/client/blob/master/shared/desktop/package.js#L55
https://github.com/keybase/client/blob/master/shared/desktop/package.js#L56
https://github.com/keybase/client/blob/master/shared/desktop/package.js#L120

the problem is those paths interfere with RN so they also need to be cleaned before that is ran

@chrisnojima Huh.  Are you sure? That doesn't make much sense -- our macOS build runs on an entirely different machine to our iOS/Android builds.  This isn't mentioned in the ticket, either.  Could you give more info about the symptom you're seeing?

if this doesn't happen on the actual build machines we can close.
this happens locally


Composer install fails with "Your lock file does not contain a compatible set of packages. Please run composer update."

After run this command: composer create-project --prefer-dist cretueusebiu/laravel-vue-spa
I found this error:
Your lock file does not contain a compatible set of packages. Please run composer update.
Problem 1
- league/commonmark is locked to version 2.0.2 and an update of this package was not requested.
- league/commonmark 2.0.2 requires php ^7.4 || ^8.0 -> your php version (7.3.10) does not satisfy that requirement.
Problem 2
- league/config is locked to version v1.1.1 and an update of this package was not requested.
- league/config v1.1.1 requires php ^7.4 || ^8.0 -> your php version (7.3.10) does not satisfy that requirement.
Problem 3
- league/commonmark 2.0.2 requires php ^7.4 || ^8.0 -> your php version (7.3.10) does not satisfy that requirement.
- laravel/framework v8.54.0 requires league/commonmark ^1.3|^2.0 -> satisfiable by league/commonmark[2.0.2].
- laravel/framework is locked to version v8.54.0 and an update of this package was not requested.

Try maybe a git clone instead and delete the composer.lock file


Add docs for Ad Up Technology bidder adapter

Add docs for Ad Up Technology bidder adapter
Corresponding PR is HERE .

@bretg
Thanks for reviewing.
Our adapter PR was merged yesterday. So this PR could be merged now, couldn't it?

@bretg
bump :)
Our Adapter was Released with v1.17.0

sorry for the delay


Move prefixes to Autoprefixer

Right now current browsers do not need prefixes for flexbox. So, maybe your plugin should add only final spec properties and prefixes should beAuutoprefixer job, because it adds only actual prefixes.

Safari and IE need the prefixes. I just tested this. I removed a couple of moz specific prefixes I had, which are no longer necessary.
Do you recommend adding a note on auto-prefixer dependency? I do understand that it is better to maintain the vendor-prefixing in one place,  but I still wonder if its a good idea to have this dependency on this plugin.

In PostCSS is is easy to adding new plugins. Anyway right now every PostCSS user has Autoprefixer.
So, I think you can miss prefixes at all and even do not have Autoprefixer in dependencies. Just add note, that you recommend Autoprefixer and ask users to put Autoprefixer after you plugin.

Excellent. Makes sense. Done.


Tries to access /home/root directory

I downloaded script to /root folder, logged in as root, and started script directly. Installation goes fine, byt when creating a user file, I get this:
Tell me a name for the client.
Use one word only, no special characters.
Client name: tower

Do you want to protect the configuration file with a password?
(e.g. encrypt the private key with a password)
   1) Add a passwordless client
   2) Use a password for the client
Select an option [1-2]: 1

Note: using Easy-RSA configuration from: ./vars

Using SSL: openssl OpenSSL 1.0.2g  1 Mar 2016
Generating a 256 bit EC private key
writing new private key to '/etc/openvpn/easy-rsa/pki/private/tower.key.T2Chj6gbYB'
-----
Using configuration from ./safessl-easyrsa.cnf
Check that the request matches the signature
Signature ok
The Subject's Distinguished Name is as follows
commonName            :ASN.1 12:'tower'
Certificate is to be certified until Oct  9 10:49:36 2021 GMT (1080 days)

Write out database with 1 new entries
Data Base Updated
cp: cannot create regular file '/home/root/tower.ovpn': No such file or directory
./openvpn-install.sh: line 939: /home/root/tower.ovpn: No such file or directory

Client tower added, the configuration file is available at /home/root/tower.ovpn.
Download the .ovpn file and import it in your OpenVPN client.

You should add a special case: root's home is /root, not /home/root. I guess, you always tested with sudo?

No, I actually always test with root. What OS are you running?

Ubuntu 16.04 as provided by OpenVZ cloud hosting. Downloaded script to /root, logged in as root over SSH and launched it.

Can you run this?
CLIENT="tower"

if [ -e "/home/$CLIENT" ]; then
	echo "1"
	homeDir="/home/$CLIENT"
elif [ "${SUDO_USER}" ]; then
	echo "2"
	homeDir="/home/${SUDO_USER}"
else
	echo "3"
	homeDir="/root"
fi

Sorry, I've missed notification from Github.
Your script chooses 3.

I had a same issue with Debian Jessie hosted by OVH. 'root' directory not existed
After create this directory, That's run !

Please run the script with bash -x openvpn-install.sh if it happens again!

bash -x openvpn-install.sh working fine for me. Thanks.

@angristan I also faced the same issue. It occurs if you use sudo while you are root.
If you run it directly like ./openvpn-install.sh it is not causing any probelm.

@angristan I also faced the same issue. It occurs if you use sudo while you are root.
If you run it directly like ./openvpn-install.sh it is not causing any probelm.


CDK Seed/stepfunction event logger

What I did:
I implemented a CDK construct that takes a list of StepFunction arn's and stores their execution history in dynamodb, or postgres (coming soon) or allows the user to customize their error handling. The event logger is further described here. The event logger
How I did it:
A lot of blood, sweat and tears. Mostly tears because of TypeScript's type checking, that is clearly very powerful, but I have no idea how to properly use.
The seed itself contains a CDK construct that defines:

2 SQS Queues (one main, one Dead Letter Queue)
A datastore (dynamodb for now, postgres coming soon)
A processing lambda
All necessary permissions for these resources to talk to each other.

The seed also contains a lambda directory with the code to process the stepfunction execution event
How to test it:

TODO: unit tests

Check out this very simple example CDK project that contains the CDK seed.
Run:

pipenv install --dev
cdk deploy stepfunction-logger-test

This will stand up the construct along with a very simple stepfunction that the construct will monitor. Invoke the stepfunction with any parameters for a successful execution, and with {test_failure: true} for a failed execution. Notice in both cases how new events are written to the dynamodb table. It's very easy to query failed events by querying the ExecutionFailed-Timestamp-Index for type = "ExecutionFailed".
To see the construct operating with a "custom" error handling lamdba, set custom_lamdba to True in stack/app.py and re-deploy the stack using the above command. This will re-deploy the stack with a "custom" lambda (in src/lambda) that will instead store events in S3. This is supposed to show how easy it is for the user to supply their own lambda to process the events. This custom lambda can just as easily point to an existing error storage system, fire off emails to responsible people, etc).
TODO:

[ ] implement CDK infrastructure for postrges
[ ] implement logic in SQS Message processor lambda to store events in postgres
[ ] implement unit tests
[ ] implement configurable + default values for dynamodb table and GSI capacity scaling + arbitrary indexes

I'll wait until you assign reviewers before doing a full review, but I think this construct serves as a great example of how we can write the infrastructure in TS but still use something like Python to buildout the business logic of code. 💯

@alukach @markdboyd This PR is ready for review, TODO issue is open here


chore: added secret manager team in codeowners

Description
Fixes #
Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.
Checklist

[ ] I have followed Sample Format Guide
[ ] pom.xml parent set to latest shared-configuration
[ ] Appropriate changes to README are included in PR
[ ] These samples need a new API enabled in testing projects to pass (let us know which ones)
[ ] These samples need a new/updated env vars in testing projects set to pass (let us know which ones)
[ ] Tests pass:   mvn clean verify required
[ ] Lint  passes: mvn -P lint checkstyle:check required
[ ] Static Analysis:  mvn -P lint clean compile pmd:cpd-check spotbugs:check advisory only
[ ] This sample adds a new sample directory, and I updated the CODEOWNERS file with the codeowners for this sample
[ ] This sample adds a new Product API, and I updated the Blunderbuss issue/PR auto-assigner with the codeowners for this sample
[ ] Please merge this PR for me once it is approved

@minherz can you please the header-check test as well.

Header-check seems to be an issue with many PRs. Looking into it.

@Sita04 did you get a chance to look into this?

@minherz can you please merge this?


Nonetype stats['mean_2d'] gradient error in train.py

Hello,
Thank you very much for providing such an amazing open-source resource.
I am currently building a custom dataset that includes SMPL-X parameters and training an avatar using a train.py script. However, I encountered an issue during the training process.
On line 51 of train.py, the "grad" variable of stats["mean_2d"] is calculated as None, which causes the code to fail.
The exact error is as follows:
Traceback (most recent call last):
  File "train.py", line 84, in <module>
    main()
  File "train.py", line 51, in main
    stats['mean_2d_grad'] = torch.stack([x.grad.detach() for x in stats['mean_2d']])
  File "train.py", line 51, in <listcomp>
    stats['mean_2d_grad'] = torch.stack([x.grad.detach() for x in stats['mean_2d']])
AttributeError: 'NoneType' object has no attribute 'detach'

How can I resolve this issue?
Thank you.

Are you able to run the code as it is without modifying it on existing datasets, such as NeuMan dataset?

Thank you for your prompt response.
As per your suggestion, I will review the NeuMan dataset.
May I kindly ask if the "test_split.txt" and "train_split.txt" files for the NeuMan dataset should also follow the naming format of 00000, 00001, and so forth?
It appears that the preprocessed NeuMan dataset provided does not include specific information regarding the required format.

They are from NeuMan dataset's official release, not made by me.

I confirmed that, as you suggested, the same "nonetype grad" issue occurs with the Neuman dataset as well.

This is what I can see from the current repo. I can't see any problems. What I did is 1) clone the repo, 2) make common/utils/human_model_files, 3) make data/NeuMan/data, and 4) run python train.py --subject_id citron.

Thank you for your prompt response.
As per your suggestion, I will review the NeuMan dataset.
May I kindly ask if the "test_split.txt" and "train_split.txt" files for the NeuMan dataset should also follow the naming format of 00000, 00001, and so forth?
It appears that the preprocessed NeuMan dataset provided does not include specific information regarding the required format.

I found that code to create such split file  from the code in data_io/Neuman_helper
https://github.com/apple/ml-neuman/blob/0149d258b2afe6ef65c91557bba9f874675871e4/data_io/neuman_helper.py#L149

Yeap I made the split files using that code. Let me share the split files in here. This is also added to README as well.
https://drive.google.com/drive/folders/1L5KC4QIRX_ljQ_vyrIXV11FgynnuCDb8?usp=sharing

Hello,
Are you using multiple GPUs when running the train.py script?
We are working in a multi-GPU environment, and when we use CUDA_VISIBLE_DEVICES=N, the error doesn't occur. However, when we don't use it, we encounter an error where "grad" is None.

Hi, my server has multiple GPUs, but I only use a single GPU to train/test ExAvatar.
Before training/testing, I execute 'export CUDA_VISIBLE_DEVICES=N' as you said, where N is the GPU ID that I'd like to use.
Without executing that cmd, the system will use multiple GPUs in default, which could make some problems when gathering computation graphs.

We have also confirmed this fact.
Once again, thank you for your amazing work, and we appreciate your help.


portico: Update text on confirm_continue_registration.

Not very confident about the html or css, but tested having a long translation:

Looks good, though I think it'd look better if the two buttons had the same width.  @synicalsyntax is this something you'd be up for doing a quick PR to clean up?

(You get to this page by trying to login with Google/GitHub when you don't have an account).


Trying to make it work with Unity.

First, thank you for putting this together.
I've been trying to use your wapper and put it into Unity but keep getting "AzureKinectException: Couldn't create tracker".
I'm pretty sure none of your code is the problem but I wanted to know if you're planning on trying to work it into Unity and if you're successful.

Does the sample app work for you? You might have to create a software instance if your gpu doesn't support CUDA

Ya, It works great but Unity is having a problem with it.

Are the native libs all getting deployed? Try also inspecting the exception. Sometimes there's more details/log into in the errors k4a throws.

I'm going to confirm that CUDA is installed properly and then see what more information I can get out of the exception/problem.

The sample app is configured to require CUDA, so if that works, you should be good there. My guess is the native libs aren't getting deployed correctly.

[2019-11-18 14:26:31.794] [info] [t=19312] [K4ABT] D:\a\1\s\src\TrackerHost\TrackerHost.cpp (482): VerifyDependencies(). Verifying Dependencies! [2019-11-18 14:26:31.795] [error] [t=19312] [K4ABT] D:\a\1\s\src\TrackerHost\TrackerHost.cpp (484): VerifyDependencies(). Cannot locate onnxruntime.dll [2019-11-18 14:26:31.795] [error] [t=19312] [K4ABT] D:\a\1\s\src\TrackerHost\TrackerHost.cpp (131): Create(). At least one dependent library is missing! [2019-11-18 14:26:31.795] [error] [t=19312] [K4ABT] D:\a\1\s\src\sdk\k4abt.cpp (38): tracker->Create(sensor_calibration, config) returned failure in k4abt_tracker_create()
So I look like is can't find onnxruntime.dll

onnxruntime.dll and dnn_model_2_0.onnx must be in the same folder as Assets

@SethKaminski I see. So it works now? I'm curious what the nuget package can do to ensure that happens? The nuget support in Unity has always been a little weird to me

Yes, it finally works. Sadly nuget doesn't natively work with Unity since it controls what happens with the C# side of the project.

Awesome! Would you mind documenting the steps you used to make it work, so others can follow them?


Azimuth can present FIPs for selection which aren't actually available

Selected an IP on Arcus when creating a platform with a FIP, creation failedwith a popup:
{"parameter_values":{"cluster_floating_ip":"External IP is not available."}}
Horizon showed the FIP as Active but not mapped to a fixed IP. However the CLI showed it was mapped to a port on the ilab-60 network which was not owned by the project being used.

Moved to https://github.com/stackhpc/azimuth/issues/140


Play Page - Level's Won Indictator

I should see some indicator for any levels I've beated.  Use green checkmark or make the card green.  Be creative.

I would try this one out


feat(vscode): automatically enable Hybrid Mode

Closes #4131
This introduces the default value of "auto" for vue.server.hybridMode, which automatically enables it based on the compatibility of installed TS plugin extensions with Hybrid Mode.
The following TS plugin extensions are currently known to be compatible:
Extensions

unifiedjs.vscode-mdx
astro-build.astro-vscode
ije.esm-vscode
johnsoncodehk.vscode-tsslint
VisualStudioExptTeam.vscodeintellicode
bierner.lit-html (Added in v2.0.14)

The following TS plugins are currently known to be incompatible.
Extensions

styled-components.vscode-styled-components
svelte.svelte-vscode (Track https://github.com/sveltejs/language-tools/pull/2317)
Divlo.vscode-styled-jsx-languageserver

NPM packages

typescript-lit-html-plugin
typescript-plugin-css-modules

If you know of any other compatible TS plugin extensions, please report them here so that we can update the whitelist.
To verify compatibility, simply set "vue.server.hybridMode": true in your VSCode settings and reload VSCode to check if TS support in .vue files is working as expected.

denoland.vscode-deno should be compatible, as long as its not enabled (which is not by default)
The setting for this is deno.enable and should be set on a workspace basis

I got the message that jpoissonnier.vscode-styled-components is incompatible, but I don't even have that installed.

I'd really love if this message stopped popping up every time I open one of my Nuxt applications. 🤷

got this

@rogadev You can disable svelte.svelte-vscode in vue project workspace to have Hybrid Mode automatically enabled, or set "vue.server.hybridMode": false to explicitly disable Hybrid Mode to avoid message box.

@Peruibeloko Did you find that installed denoland.vscode-deno and setting deno.enable to true breaks Hybrid Mode?

@johnsoncodehk I'll do some testing tomorrow on a clean project using pnpm create vite, but IIRC Deno doesn't even has to be enabled for Volar to report it

@k-paxian It is known that denoland.vscode-deno is incompatible with Hybrid Mode when enabled. I will check for nrwl.angular-console.

Same as @Soviut, I have the message about jpoissonnier.vscode-styled-components being incompatible but I don't have this installed.

@Kerumen I think the extension ID of jpoissonnier.vscode-styled-components has been changed to styled-components.vscode-styled-components, have you installed that extension?

I got a message with miaonster.vscode-tsx-arrow-definition. It seems to be working with it.

程序员可以不懂英文吗

Any chance for some clarification on why nrwl.angular-console is marked incompatible? Looking at https://github.com/nrwl/nx-console/tree/master/apps/nxls it seems like their language server is specifically focused on autocompletion for the Nx json configuration files, so I'm curious what's conflicting.

@luxaritas Via https://github.com/volarjs/volar.js/pull/216, nrwl.angular-console will be compatible with Hybrid Mode starting in the next version (v2.0.23).

Hybrid Mode is disabled automatically because there is a potentially incompatible MylesMurphy.prettify-ts TypeScript plugin installed.

Hybrid Mode is disabled automatically because there is a potentially incompatible ShenQingchuan.vue-vine-extension TypeScript plugin installed.
I don't quite get it with this message but found this PR page, is there anything need to do in Vue Vine?

@ShenQingchuan easy to resolve, add your extension to https://github.com/vuejs/language-tools/blob/master/extensions/vscode/src/common.ts#L47-L64. Created a pr #4543

@johnsoncodehk Can we kindly ask you to check for new reports? There is long time with no updates

@matvejs16 This is a merged commit, you're not going to see any meaningful updates here. They will announce any fixes in releases.

If you know of any other compatible TS plugin extensions, please report them here so that we can update the whitelist.
To verify compatibility, simply set "vue.server.hybridMode": true in your VSCode settings and reload VSCode to check if TS support in .vue files is working as expected.

I think you are incorrect.

@matvejs16 that doesn't say the maintainers will issue a report, it says that people with problems can report them here and they'll add them to the whitelist. They are not going to publish the whitelist here.
So if you encounter an compatibility issue, you can leave a message saying which package caused the incompatibility, but if all the compatibility issues are solved, then this thread will be very quiet.

Thank you, @Soviut for sharing your perspective. I now better understand the reasoning behind your comments and your position on similar issues.
To clarify my intentions, I raised this concern here because I observed that the extension responsible for compatibility issues had been reported more than four months ago, yet it still hasn’t been added to the whitelist. I understand that sometimes users may inquire without checking the latest updates, but in this case, I made sure to verify that the problem persists before commenting. With several years of programming experience, I approached this thoughtfully, not to hasten the process but to inquire constructively.
Thanks again for your insight!

@matvejs16 okay, that's reasonable. Just report which one you're still seeing and mention its absence from the whitelist.

Is there any updates on https://marketplace.visualstudio.com/items?itemName=MylesMurphy.prettify-ts extension as @said12345plus have commented earlier?

Literally any VSCode window I open I get a message

Hybrid Mode is disabled automatically because there is a potentially incompatible jpoissonnier.vscode-styled-components TypeScript plugin installed.

Not only for vue projects, but for literally any VSCode window I open. And I don't even have that "incompatible plugin" installed...

Literally any VSCode window I open I get a message

Hybrid Mode is disabled automatically because there is a potentially incompatible jpoissonnier.vscode-styled-components TypeScript plugin installed.

Not only for vue projects, but for literally any VSCode window I open. And I don't even have that "incompatible plugin" installed...

This 👆

I get this with Github Copilot Chat. Interestingly it is specifically from the "Github Copilot Chat" (which I never use) and not the main "Github Copilot", but I don't seem to be able to install one without the other.

I get this with Github Copilot Chat. Interestingly it is specifically from the "Github Copilot Chat" (which I never use) and not the main "Github Copilot", but I don't seem to be able to install one without the other.

I am getting the same
Hybrid Mode is disabled automatically because there is a potentially incompatible GitHub.copilot-chat TypeScript plugin installed.

I get this with Github Copilot Chat. Interestingly it is specifically from the "Github Copilot Chat" (which I never use) and not the main "Github Copilot", but I don't seem to be able to install one without the other.

Same Here
GitHub.copilot-chat TypeScript plugin installed.

I got the same issue. Are there any solutions now?

same here

same here

same here

same here

Same issue with Copilot Chat

I don't know what this means, take the best guess and remove this message:

Hybrid Mode is disabled automatically because there is a potentially incompatible GitHub.copilot-chat TypeScript plugin installed.

bytedance-microapp.microapp

Maybe the fix is coming.

https://github.com/vuejs/language-tools/issues/5044#issuecomment-2544999764
#5047

Have same error as a lot of people before for Vue.official

I've encountered this situation too.

Stop pollute this thread. One or two messages on this problem were enough. Wait for the fix and turn on HybridMode to true explicitly for now

VSC alert !! Hybrid Mode is disabled automatically because there is a pootencially incompatible GitHub.copilot-chat Typescript plugin installed.

Put this setting to explicitly enable hybrid mode and bypass the warning:
{
  "vue.server.hybridMode": true
}


Add options to enforce min/max exclusion

The range type doesn't currently support arbitrarily set closed/open/half-open intervals. This feature may be useful in various settings (it arose when mapping alert text to non-zero input values in https://github.com/phetsims/molecules-and-light/issues/180).
I'm mildly concerned about how JS handles certain floating point situations (e.g. var x = 0.1 * 0.2 = 0.020000000000000004) and how they would interact with an exclusive Range( 0.02, 0.03 ). In that case, x should not be included in the Range.
Proposal is to implement closed intervals by default, add option keys (excludeMin and excludeMax) for closed behavior, and create tests to ensure these additions don't break current usages.

As noted in Slack, these changes sound good. I don't think Range is used in performance-critical areas, so adding the options object should be totally fine (and helps for code readability).

I'm mildly concerned about how JS handles certain floating point situations

Yup, seems somewhat unavoidable. Generally the client should round values as necessary, and it's best to NOT round in range checks. Can always use Fraction if we need to handle those fractional cases with full accuracy.

Thanks for the best practices info @jonathanolson, I'll be sure to add that in the doc.

This looks really nice thanks for doing that!

Thanks @zepumph! I think I can close this.


[FEAT/UI]: Build a Hero Section

Can you elaborate what you need exactly. I can work on it, if you assign me @ayushsoni1010

Hey @saimaheshtaduri👋
Thanks for contributing to Spendify
Currently, I am working on the Login and Signup UI, as of now for the authorization setup I had made a general UI. I need a few designs for both UI and then later we have to convert them into code.


Adapt to latest witnet-ethereum-bridge, simplify contracts structure, add tests

This:

Uses the latest version of the Witnet libraries.
Removes the need for an intermediary contract, it queries Witnet directly from Monetizer.
Fixes the parametric URL interpolation so it works for any Youtube video ID.
Uses a safer deposit / checkViews / withdraw workflow.
Adds a test that covers the entire end-to-end workflow of the Monetizer contract.
Improves the available tooling.
Lints all files.
Adds support for GitHub Actions, so that migrations, tests and whatnot are run upon pushing or creating PRs.

What's missing:

Adapting the webapp to the deposit / checkViews / withdraw workflow (look at the test to see how the workflow goes).

Thank you very much for your help, I have merged your contract to the main branch in this pull request.

Brilliant!


Attached DocDB/Graph accounts should show error node if they fail to connect

Attached mongo accounts show a helpful error message in the tree, but docdb/graph accounts just show the "Load more..." node:

This was fixed in https://github.com/Microsoft/vscode-azuretools/pull/107


[BUG]  Import From Reactive Resume json v2 does not work

I have my CV saved with rxresu.me, I export my CV as a .json, and then I tried to re-import as .json v2, it doesn't work. The CV just says "undefined undefined"
I tried with .json "v1" and it works, but not the v2.

Desktop (please complete the following information):

OS: Windows
Browser Firefox
Version 104

Same issue here...

Now there is no 'JSON v1' anymore, just 'JSON' and 'JSON v2 from v2'.
So no more v1, just v3 and v2. That's why it doesn't work like before.
I am going to close this issue.


Upload Progress Component

As a user i want to be able to see the progress of my upload. @samspecial

Live link: https://companyfiles.zuri.chat/upload
Slack name: Samspecial.
Team: Galileo
Aspect worked on: Frontend


Few-Shot Learning dataset

Hi, thanks again for your code. I am curious if you have the plan to release the data preparation code for few-shot learning for FGVC-Aircraft, Oxford-Pets, Food 101, Stanford Card and Oxford Flower.

Hi @RaptorMai,
Please refer to https://github.com/ZhangYuanhan-AI/NOAH/#data-preparation and https://github.com/ZhangYuanhan-AI/NOAH/blob/main/lib/datasets.py for datasets preparation.


[BACKLOG-20818] Fixed issue with closing and adding database connection

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
mvn -Dsurefire.runOrder=alphabetical -B -fn -f 'pom.xml' -pl 'plugins/repositories/core' -P '!assemblies' -amd clean install && mvn -B -f 'pom.xml' -pl 'plugins/repositories/core' -P '!assemblies' -amd site

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'plugins/repositories/core' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  plugins/repositories/core/src/main/javascript/app/components/add/add.component.js
  plugins/repositories/core/src/main/javascript/app/components/database/select/select.html

@ddiroma


API v2 should not include the user's role in the responses

Currently all API v2 operations that return the User entity, include the user's role in the response.
The user's role is specific to hs_auth's internal implementation and hs_auth's consumers should not depend on the user's role, permissions should be checked via the GetAuthorizedURIs operation. As such, we should update the API v2 responses to omit the role whenever a User is returned.

The user entity: https://github.com/unicsmcr/hs_auth/blob/master/entities/user.go
To resolve this issue, it would be enough to omit the Role field from JSON responses. We are already doing something similar for the user's password already: https://github.com/unicsmcr/hs_auth/blob/master/entities/user.go#L27 (via the json:"-" directive). So to complete this issue we'd need to:

Omit the user's role from JSON responses similar to how we do it for the user's password
Create unit tests like the ones we have for omitting the user's password: https://github.com/unicsmcr/hs_auth/blob/master/entities/user_test.go


(rc4) ng test errors with 0 tests executed

Versions
Angular CLI: 6.0.0-rc.4
Node: 8.9.4
OS: darwin x64
Angular: 6.0.0-rc.4
... animations, cli, common, compiler, compiler-cli, core
... elements, forms, http, platform-browser
... platform-browser-dynamic, platform-server, router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.5.6
@angular-devkit/build-angular     0.5.6
@angular-devkit/build-optimizer   0.5.6
@angular-devkit/core              0.5.6
@angular-devkit/schematics        0.5.6
@angular/cdk                      6.0.0-rc.1
@angular/material                 6.0.0-rc.1
@angular/service-worker           1.0.0-beta.16
@ngtools/json-schema              1.1.0
@ngtools/webpack                  6.0.0-rc.4
@schematics/angular               0.5.6
@schematics/update                0.5.6
rxjs                              6.0.0-turbo-rc.4
typescript                        2.7.2
webpack                           4.5.0

Repro steps

clone https://github.com/angular/angular/pull/23234
cd angular/aio
yarn install
yarn ng test

Observed behavior
karma starts, but doesn't run any tests:
Chrome 65.0.3325 (Mac OS X 10.13.3): Executed 0 of 0 ERROR (0.021 secs / 0 secs)

as if no specs were loaded when karma started.
Desired behavior
unit tests run :)
Mention any other details that might be useful (optional)

I tried to scaffold a new cli project and copy the contents of the aio/src/ directory into it + install all missing dependencies and got the same result

I've been looking into this and it seems like something is preventing webpack from 'scooping' up all the spec files.
In test.ts, these are are the lines responsible for rounding up all specs:
// Then we find all the tests.
const context = require.context('./', true, /\.spec\.ts$/);
// And load the modules.
context.keys().map(context);/

In a brand new project you can add the line below after the ones above to see the list of tests ran:
console.log(context.keys());

You will see the following line in the karma output:
HeadlessChrome 65.0.3325 (Windows 10.0.0) LOG: ['./app/app.component.spec.ts']

In AIO with this PR, that doesn't happen. In fact, nothing at all is printed. Console logs after the require.context don't seem to run at all.
But if you go into src/environments/environment.ts and comment out the two following lines, tests run again:
import 'core-js/es6/reflect';
import 'core-js/es7/reflect';

The test will fail, but that doesn't surprise me since they are ran in JIT and thus need reflect.
But something weird is happening with reflect.

I think there an import somewhere that breaks things.
Trying to scoop up this single test works:
const context = require.context('./app/shared/', true, /registry\.spec\.ts$/);

this was due to our unusual polyfill setup. See the commit above for solution.


Added a wrapper around song list

torvalds.dev is analyzing the pull request

Based on the Coding Standards Document, here's what I can deduce from the Github PR diff mentioned above:

Modifed files:

src/core/MusicPlayerCore/index.ts
src/core/VueMiniPlayerCore/index.ts

Added files:

src/core/VueMiniPlayerCore/index_2.ts
src/core/VueMiniPlayerCore/new-index.ts

Here are the improvements that could be made to the code based on the coding standard:
Naming Conventions:

In the file src/core/MusicPlayerCore/index.ts, a new local variable P is introduced. Single-letter variable names, except for loop counters, are not recommended. A more meaningful and descriptive name could be used.
The function name PrevSong has been updated to Prev-Song. Avoid using hyphens in function names. Instead, consider using camelCase, for example prevSong.
Several variable names have been replaced with single letter variables: WrapperID to a and app to b. These changes make the code less descriptive and readable. Therefore, it might be ideal to revert them back to their original names.

Code Formatting:

There has been addition of blank lines (+ ) at the beginning of a few methods, like AppendSongOnHead. There's no need for blank lines here really, as they don't really improve the overall readability of the code.
The operator = in this.SongIdList = list was replaced to === which is a strict equality operator. If the intent is to assign list to this.SongIdList then it might be a typo and it should be reverted back to = to ensure correct assignment operation.
slider.clientWidth was replaced with slider.client-Width which seems like a typo, as client-Width is not a valid property.

It seems that this PR needs few changes according to the above recommendation to conform to the given coding standards document.


WebGL.Net via SkiaSharp currently unable to load jpgs

see: https://github.com/WaveEngine/WebGL.NET/blob/master/src/Samples/Samples/TexturedCubeFromAssets.cs#L73

See #228


fix(iOS): Allow the original delegate to handle non-Notifee notifications first

Issues
Fixes

#925
#984

Overview
For notifications received when the app is backgrounded or killed in iOS (through userNotificationCenter:didReceiveNotificationResponse:), the original delegate is not invoked for non-Notifee notifications. The existing code attempts to parse the unknown notification into a format Notifee understands before giving the original delegate a chance to handle it.
I've moved up original delegate so that it handles a non-Notifee notification first (if it exists). If there is no original delegate, then we fall back to parsing the unknown notification and allowing Notifee to handle it.
Testing

Jest tests all pass
iOS end-to-end tests all pass
I have an app that handles both Iterable and Notifee notifications.

Before this change: The Iterable notification was being handled by Notifee, resulting in a default behavior of launching to the app home screen.
After this change: Notifee falls back to the original delegate when encountering the Iterable notification, resulting in handling the notification correctly & launching the app to the correct screen.

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

@mikehardy, Just wondering if we can get this in as this may be more common case and the bug prevents one listener from other

Tried this.
messaging().getInitialNotification() works now when iOS app is terminated.

not stale
@mikehardy Is there a possibility to get this reviewed and merged? It would really help for projects that use libraries that depend on Notifee, but for projects where advanced notifee features are not directly required by the consumer (the developer), instead react-native-firebase is enough. This is exactly the use case I have and would greatly appreciate this change being merged. (same as in android, but that's a story for another PR)

If I correctly understand these changes, wouldn't the original delegate always get called instead of the notifee handler?
E.g when I use react-native-firebase?
Should there be an option to toggle that behaviour?

Good work @gotcha84 ! Looking forward to seeing this merged.

I can confirm that this solves #913 as well.
I would love to see this merged soon!

not stale

is there a plan to merge this PR?

Can someone please merge this?

thanks for the fix! 🙏

Thanks for fixing and merging it. Any idea when this will be included in an upcoming release?

@gotcha84 looks like this was merged, but has some linting issues
https://github.com/invertase/notifee/actions/runs/9472210287/job/26097204168
@helenaford any reason aside from linting issues to not get another release out with this included?

Ok. @gotcha84 i did the cleanup
@helenaford it's clean now.
https://github.com/invertase/notifee/pull/1073

Firstly, thank you, and when do you think it will be deployed?

@helenaford @mikehardy is it possible to merge the fix provided by @markalanevans #1073?
Currently, the library is incompatible with the latest RN versions since it fails the TS check.

Hey there -<EMAIL_ADDRESS>is published just now with these changes --> https://github.com/invertase/notifee/releases/tag/%40notifee%2Freact-native%407.9.0


Change type of flags in io_uring_prep_renameat() to unsigned int

According to:
https://man7.org/linux/man-pages/man2/rename.2.html renameat2
flags takes unsigned int
Signed-off-by: Ritesh<EMAIL_ADDRESS>

Signed-off-by: Ritesh<EMAIL_ADDRESS>
The Signed-off-by should be in the commit message, not
in the pull request body.


RAC-5464: check uncompleted wget download for *.zip

Background:
wget common.zip failure will still return 0...
then unzip will complain...

Testing:
(1) positive case
[Info] Downloading http://www.colorado.edu/conflict/peace/download/peace.zip
[Info] wget download successfully http://www.colorado.edu/conflict/peace/download/peace.zip
[Info] Checking zip file integrity for http://www.colorado.edu/conflict/peace/download/peace.zip
test of peace.zip OK

(2) negative case
[Info] Downloading http://*.*.19.21/job/Tools/job/Static-Files-Service/lastSuccessfulBuild/artifact/common/*zip*/common.zip
[Info] wget download successfully http://*.*.19.21/job/Tools/job/Static-Files-Service/lastSuccessfulBuild/artifact/common/*zip*/common.zip
[Info] Checking zip file integrity for http://*.*.19.21/job/Tools/job/Static-Files-Service/lastSuccessfulBuild/artifact/common/*zip*/common.zip
zip error: Zip file structure invalid (common.zip)
+ '[' 3 -ne 0 ']'
[Error] the download file(http://*.*.19.21/job/Tools/job/Static-Files-Service/lastSuccessfulBuild/artifact/common/*zip*/common.zip) is incompleted !

@PengTian0

BUILD on-build-config #543 : FAILURE

change from zip to unzip...
vmslave doesn't have zip installed ...orz...

@anhou  @PengTian0  @iceiilin

:+1:


Accessibility support for new assessments UI

https://openedx.atlassian.net/browse/MA-1032
Following 3 issues are now remaining and need some research:
1 - On sign up screen drop downs like Country, gender etc it only read the label like Country , gender.
2 - On tapping course view modes it says showing 5 items showing items 1 to 5 of 5.
3 - On tapping view on web icon it says showing 3 items showing items 1 to 3 of 3.

I'm not sure we want these changes that are " button". My understanding is that the typical UI label is enough ("Cancel", "Sign In") and a user of assisted technologies has a way of finding out that something is a button. That way, most buttons are already accessible, since they have just have action text, and so we mainly need to make sure the action for image views, image buttons, and custom views are labeled properly.

I've added the "button" suffix to custom views that we have made as it was asked, like, the Sign In button is actually a Relative Layout having a TextView and a ProgressBar ... so it was being read as "Sign in" only.
Also, it was something that was mentioned in the ticket as well:
10 - On Sign in screen on tapping "Sign In" button it only says "Sign in"

@aleffert should this PR be merged now ?

@miankhalid yes


Application is not asking for consent

Today I pulled new updates. I removed the registered app and tried to again go through the consent process. When I ran the app and logged in instead of app prompting for consent UsersSync page is shown. When I  initiated the user sync, app always throw Unauthorized error.

Closing as no repro text was shared


Migrate away from forecastio python library

Describe the issue
As mentioned in #321 it's probably time to migrate away from using the old forecastio python library as it hasn't been updated in about 6y and is prone to breaking in new python versions.
I'm not super familiar with HA so am not sure if we can just add in the models.py file into the integration and use that. I took a look at some official integrations and all of them seem to use an external library. Which means we may still need a library if you're planning to submit this as an official integration eventually.
If we still need a library to use then the options are

Use the pirate-weather-python library linked in that issue
Fork the forecastio library and use that fork instead (I have a fork here https://github.com/cloneofghosts/python-pirate-weather and am willing to add it to the organisation/invite you as a collaborator if you want)

I know the plan at one point was to re-write the integration using that library but the branch hasn't been updated in about a year so probably isn't the plan anymore.
Home Assistant version
No response
Integration version
No response
Troubleshooting steps

[X] I have updated my Home Assistant installation to the latest version.
[X] I have updated the Pirate Weather Integration to the latest version.
[X] I have gone through the documentation before opening this issue.
[X] I have searched this repository and API Repository to see if the issue has already been reported.
[X] I have restarted my Home Assistant installation.
[X] I have queried the API in my browser to confirm the issue is not with the API.
[X] I have not included my API key anywhere in my issue.
[X] I have written an informative title.

It's things like this that really show the limits to my computer science knowledge. I sort of understand the whole async/ sync stuff, and appreciate how to use modules and the like, but I'm really not comfortable with it.
For HA, it's both easiest and best (in the short term at least) to just drop the requirement and add the models.py script. It has very few dependencies, and really just organizes the response. We could even modify the sensors.py script to just use the json directly (like we do for the new fire sensor), but I like the idea of keeping some structure to it. Fewer dependencies is always better, and while I suspect there's some sort of benefit we could get from using a designated library, I don't know what it would be. PR with the change is coming momentarily.
However, for other projects, a Python library that makes calls and whatnot probably makes sense! I'm going to try to reach out to @bakobako to see if he'd be ok with moving that library into the project, and building off it from there, since it seems like a great start!

Sure we can do that for the short-term and the integration will eventually switch over to using that library? I'll keep mine around in case it'll be useful for someone in the future.

Yes, agreed! You've very much convinced me on that point. just made the initial push with the library changes, with the fire risk level next

Okay, sounds good. When it's ready to test you can make a PR and assign me to review to check to make sure everything still works before going live.
Again, not really sure what the policy is for HA integrations is so not sure if you'd need to maintain a separate library or if you can include it in integration.

Fixed up the lint errors though in the models.py file I added placeholder comments so it would at least lint but need to be changed. Also there's an instance of OpenWeatherMap in one of the files that'll need to be changed. Also had to create a function for the fire index since it was complaining about complexity.
Did a quick test and it seems to work though after I added the fire index sensor the fire index level sensor wasn't added and I see no way to select it.

@alexander0042 Just responding to the tag, I haven't worked on that code in a long time and don't plan to in the future since I no longer use it. Feel free to copy or modify anything you need. If you'd prefer, I can also delete the repo to avoid confusion if it's outdated. Just let me know.

Thanks for responding so quickly! What I'm planning on doing is forking it into our own organization, that way everything is in one place, but people can still trace the history back to your starting point!

Link for future reference: https://github.com/Pirate-Weather/pirate-weather-python

Link for future reference: https://github.com/Pirate-Weather/pirate-weather-python

I'll setup dependabot, lint action and codeql and create a PR to merge it in or if you invite me to the repo I can merge it myself. @alexander0042 I also noticed you didn't merge in the changes you made on your branch a while back but maybe you just forgot about it.

@alexander0042 I know its a bit late for this but it may be helpful in the future but I published my forecastio fork on PyPI yesterday https://pypi.org/project/python-pirateweather/


Make PopupMenu hide after its parent window loss focus

Previously PopupMenus were still on screen after their parent was minimized. It was quite annoying, especially in Editor, because it limits its fps to 1 per second when minimized, even if one of the popup menus is still visible. It's also worth mentioning that no other app I use behaves like this. Their popups always hide after minimizing the main window or they can't just be minimized while popup is visible.
Before
https://github.com/user-attachments/assets/632e1957-c731-46b6-8749-e64a220e4fe9
After
https://github.com/user-attachments/assets/6d9b0e1e-421f-495f-b422-0267a104f8a2
I decided to fix this  by adding two flags which are also visible from scripts:

Window::return_to_transient (default: false)
PopupMenu::hide_on_parent_unfocused (default: true)

Also I had to add auxiliary const method Window::get_transient_parent() to be able to use the "focus_exited" signal of the parent window from its child popup.
Previous behavior can be achieved by setting return_to_transient to true and hide_on_parent_unfocused to false.
Three things I'm not sure about are:

naming of the PopupMenu::hide_on_parent_unfocused and related stuff (I chose this name because there is Popup::_parent_focused() method.
using thread guards in setter and getter of the Window::return_to_transient flag. Honestly, I don't even know what exactly do ERR_MAIN_THREAD_GUARD and ERR_READ_THREAD_GUARD_V macros do
Haven't tested on .NET version of Godot.

Can't reproduce the bug on Windows.

Can't reproduce the bug on Windows.

Yep, it seems to be a Linux only issue (and maybe macOS), but it needs to be fixed somehow. Looks like it's due to how different OSes handle popups. And I don't think it'll be changed by them in the future. These are different approaches.

Honestly, I was waiting for the feedback about my naming before writing documentation, but ok, this is it.


remove "audit-service" log topic

Scope & Purpose
Enterprise companion PR: https://github.com/arangodb/enterprise/pull/1498
Remove unused log topic "audit-service"

[ ] :hankey: Bugfix
[ ] :pizza: New feature
[ ] :fire: Performance improvement
[x] :hammer: Refactoring/simplification

Checklist

[ ] Tests

[ ] Regression tests
[ ] C++ Unit tests
[ ] integration tests
[ ] resilience tests

[ ] :book: CHANGELOG entry made
[ ] :books: documentation written (release notes, API changes, ...)
[ ] Backports

[ ] Backport for 3.12.0: -
[ ] Backport for 3.11: -
[ ] Backport for 3.10: -

Related Information

[ ] Docs PR:
[x] Enterprise PR: https://github.com/arangodb/enterprise/pull/1498
[ ] GitHub issue / Jira ticket:
[ ] Design document:

It turns out that the log topic is still in use.


ceph_ansible: provide devices, monitor_interface, public_network

Ugh. On Ubuntu it is /sbin/ip whereas on CentOS it is/usr/sbin/ip. And it's not in $PATH. Hold off on this for the moment.

@andrewschoen any idea what happened here?
https://jenkins.ceph.com/job/teuthology-pull-requests/247/console

Apparently pushing "rebuild" in Jenkins and waiting ~15min for everything to provision/run doesn't actually update this :(

test this please

@andrewschoen so this time, test passed but reported failure:
https://jenkins.ceph.com/job/teuthology-pull-requests/249/console
So... where do I file bugs for this CI stuff?

test this please

test this please

testing is broken.  so be it.


Add blog post on PodSecurityPolicy historical context

This blog post aims at giving more historical context on the deprecation and removal of PSP for the reader to understand how PSP was firstly added and why it was necessary to remove it and replace it with something else. It's a bit of a sandwich article with explanations on what and why at the beginning, then some historical context, and then some explanation on why we change and for what.
I mostly retrieved the information by digging into Github issues, PRs and watching the SIG auth updates, if someone can bring new anecdotes or info, it would be appreciated.
Deploy preview here.
TODOs:

[x] search and replace all the absolute kubernetes.io links
[x] rework the timeline figure for appropriate logo and colors
[x] clarify the use of PodSecurityPolicy vs PSP, idem for SecurityContextConstraint vs SCC, along the article
[x] add or not the chronological evolution from Tabitha and Tim
[x] remove comments and TODOs in the article
[x] update the date of the article

References & resources
I put all these links in a code block to avoid cross-linking all these issues and PRs.
- 6287 WIP: Security Context [closed] https://github.com/kubernetes/kubernetes/pull/6287
- 7101 ServiceAccounts [merged] https://github.com/kubernetes/kubernetes/pull/7101
- 7343 Security context - types, kubelet, admission [merged] https://github.com/kubernetes/kubernetes/pull/7343
*The security context was reversed and then rereversed*
- 7893 WIP: Security Policy [merged] https://github.com/kubernetes/kubernetes/pull/7893
- 23217 Bring PodSecurityPolicy to usable state https://github.com/kubernetes/kubernetes/issues/23217
- 24600 PSP admission [merged] https://github.com/kubernetes/kubernetes/pull/24600
- 33080 Add authz to psp admission [merged] https://github.com/kubernetes/kubernetes/pull/33080
- 36184 Split PSP defaulting and validation https://github.com/kubernetes/kubernetes/issues/36184
- 52849 Prefer non-mutating PSPs, then order by name https://github.com/kubernetes/kubernetes/pull/52849 (this thing closes the two above issues)

- KEP 5 Pod Security Policy https://github.com/kubernetes/enhancements/issues/5

PSP design proposal - https://github.com/kubernetes/design-proposals-archive/blob/main/auth/pod-security-policy.md
Lots of information from:
2019 - SIG Auth Update and Deep Dive - Mo Khan, Red Hat; Mike Danese, Google; & Tim Allclair, Google - https://www.youtube.com/watch?v=SFtHRmPuhEw&
2021 - PodSecurityPolicy Replacement: Past, Present, and Future - Tabitha Sable & Tim Allclair - https://www.youtube.com/watch?v=HsRRmlTJpls

I love this timeline but it misses references and links to the associated PRs, maybe I will have the time to find more information or we will not include that.
I finally decided to add a "What's next" section instead.
- 1.3 - PSP added as beta feature.
- 1.6 - The resources to create PSP are available by default, no need for a feature
  gate anymore.
- 1.7 - It is possible to bind PSP into namespaces.
- 1.8 - Many fixes.
- 1.15 - Some feature additions, more pod fields to be controlled with PSP.
- 1.16 - First public replacement discussions.
- 1.21 - Deprecation.
- 1.25 - Removal.

Hi, @onlydole  can you serve as the technical reviewer for this as requested? Please just add a /lgtm if this is technically correct,
suggest if anything to add @bradtopol
/assign
/assign @onlydole

Hey folks! We'd like to get this blog out on or before release day ideally, and if that's not possible, very close to it. Is that doable? Happy to help in any way necessary!

Hi Mahé. This is a very interesting history. I made some minor suggestions that I hope will strengthen your message. Most of these are subjective, while some are based on rules of style or grammar. You are welcome to take or leave what you like. I'm free to answer any questions you have.

Thank you so much for taking the time to do this review, it's really valuable for me!! :)

(from Slack, @katcosgrove)

If we can specify a publication time for this as well, then perhaps 2pm Pacific, to make sure it comes out after the release blog?

/hold
OK to unhold this once the v1.25 release has happened. Ideally we also set a date that is after the v1.25 release blog publishes.

An approachable article backed by careful research. I like it.
/lgtm
/approve

An approachable article backed by careful research. I like it.

I'm glad you like it Tim 🤗! I guess everything is ready for publication in ~4 hours.

I guess everything is ready for publication in ~4 hours.

I think we're all set.

/hold cancel

/lgtm


Removing video content should also provide progress bars

While removing videos from an installation (for a language change for example) it is very time consuming and there are no progress bars to let the user know the process is still underway.

Hi @EdDixon - I haven't tried to use multi-language checks (especially videos too) for an install yet so this is interesting.  Some questions for you:

Did you remove the videos inside the ka-lite web app, using one of the installers, or manually in a file system of the OS?
Which OS did you use for this?

If in case you used the ka-lite web app for this, maybe we should file this on the ka-lite repo?

I think we can remove this issue, as it relates to the source package. The use case is also rather narrow, but in a broader sense, we need a "cleanup" command that can delete unused videos which will eventually occur if you upgrade from one KA Lite to another or if you just wanna copy in the videos that were downloaded last year.
But as KA Lite is going into "legacy mode", it's unlikely this can be done.


Tabs: Update live demo

Live demo needs to be updated to reflect v11 changes

Hoping we can structure the live demo a little closer to how we structured our docs:

Visuals:

Showing icon with size selector for line tabs:

the original design above is the ideal configuration that follows our current doc structure. it differs slightly from our storybook right now.
after chatting with @andreancardona, in the short term, we will stick with the current storybook structure and update the live demo component according to priority of the Storybook audit:
https://airtable.com/appOqWyfQBLznbeE7/tbldm9oxL5kMaRFh7/viw2hSBv9swEUJTNt?blocks=hide
Demo with current storybook structure and temporary live demo design:

Just wanted to point out, the knobs on the right directly map to what props are used in the react components themselves, with the exact naming a developer would use. The above idea of a kind with text & icons isn't a prop but rather a pattern a user would construct, and could lead to some confusion.


Web Cmdlets strip ContentType header on GET requests even with -SkipHeaderValidation

I have an API that expects to receive a Content-Type header in a particular GET request. When it does not get that header, it returns an HTTP 415 'Unsupported Media Type' response. I realize this API is probably non-conforming to the HTTP spec. Under normal circumstances, it is probably correct that PowerShell strips the Content-Type header from GET requests. However, the docs for SkipHeaderValidation make it seem like that would be a workaround.

This switch should be used for sites that require header values that do not conform to standards. Specifying this switch disables validation to allow the value to be passed unchecked. When specified, all headers are added without validation.

This will disable validation for values passed to the -ContentType, -Headers and -UserAgent parameters.

It should be noted that the API call works and the Content-Type header is passed normally in Windows PowerShell 5.1 despite the lack of the SkipHeaderValidation parameter.
Steps to reproduce
(Invoke-WebRequest https://httpbin.org/get -ContentType 'application/force-download' -SkipHeaderValidation).Content

# or

(Invoke-WebRequest https://httpbin.org/get -Headers @{'Content-Type'='application/force-download'} -SkipHeaderValidation).Content

Expected behavior
{
  "args": {},
  "headers": {
    "Content-Type": "application/force-download",
    "Host": "httpbin.org",
    "User-Agent": "Mozilla/5.0 (Windows NT 10.0; Microsoft Windows 10.0.17134; en-US) PowerShell/6.2.0"
  },
  "origin": "xxx.xxx.xxx.xxx, xxx.xxx.xxx.xxx",
  "url": "https://httpbin.org/get"
}

Actual behavior
{
  "args": {},
  "headers": {
    "Host": "httpbin.org",
    "User-Agent": "Mozilla/5.0 (Windows NT 10.0; Microsoft Windows 10.0.17134; en-US) PowerShell/6.2.0"
  },
  "origin": "xxx.xxx.xxx.xxx, xxx.xxx.xxx.xxx",
  "url": "https://httpbin.org/get"
}

Environment data

Name                           Value
----                           -----
PSVersion                      6.2.0
PSEdition                      Core
GitCommitId                    6.2.0
OS                             Microsoft Windows 10.0.17134
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

@markekraus

/cc @markekraus
From what I remember of some recent issues, there's an issue (by design? I'm not sure) where the cmdlet ignores Content-Type headers and the parameter when -Body is not set on the request.

Duplicate of  #9473 closing

For clarity, its the same underlying issue that causes both of these behaviors.

Setting an empty body does indeed workaround the problem. Ironically, the empty body causes a ProtocolViolationException on Windows PowerShell 5.1. But I can work around that too.


Executing async tests serially?

I work on the project that has many async unit tests.
In the codebase, many dependencies are not explicitly passed to the classes, but are global variables or static properties on the classes.
Testing involves mocking/stubbing these global variables, and I need all unit tests to be executed serially, so that two tests running concurrently wouldn't overwrite/access each other's mocks and stubs. I wonder how I can ensure my async tests are executed serially?
Online resources are conflicted on this question.
The mochajs docs just describes the 3 ways of wriring acync tests but doesn't explain if any of the tests run concurtently by default.
Stackoverflow suggests that one must pass the done callback version of the code to it(), then the tests will become sequential.
This tutorial says that the done callback is archaic and only relevant for the old codebases.

This open question has existed for quite a while without a concrete answer. Per #5027 we've moved question-asking to Discord. If you're reading this and interested, please:

Search other issues for what you're trying to do
Ask there or StackOverflow

Cheers!


Repaired node.parentCollection for switch statements

Explanation
After moving to estraverse for tree-iterator in #817, we introduced a regression when resolving node.parentCollection. We want to resolve the sibling nodes for a node by looking at the property its parent is referring to.
For example, in the script function a() {} function b() {}, the Program node would have a body of an array of 2 FunctionDeclaration nodes. If we wanted to find this array from the perspective of a, then we would look at its parent node and the key it used to access it (e.g. body).
In most cases for VariableDeclaration's, the containing node has this declaration inside of a body key (e.g. Program, FunctionDeclaration, BlockStatement).
https://github.com/estree/estree/blob/9a35a3d091af6ff9cf7fadfe27c49e22533b2bce/spec.md#programs
https://github.com/estree/estree/blob/9a35a3d091af6ff9cf7fadfe27c49e22533b2bce/spec.md#functions
https://github.com/estree/estree/blob/9a35a3d091af6ff9cf7fadfe27c49e22533b2bce/spec.md#blockstatement

Fwiw IfStatement/CatchClause/etc all have a BlockStatement as the children which is the proper container for these VariableDeclaration's.

However in the case of a SwitchCase, we use consquent as the key:
https://github.com/estree/estree/blob/9a35a3d091af6ff9cf7fadfe27c49e22533b2bce/spec.md#switchcase
Problem/solution
The problem we had was that tree-iterator wasn't stopping once it found the key. It continued traversing up to the root node (Program). As a result, the key was always body and this worked great for most cases (e.g. Program, FunctionDeclaration, BlockStatement). However, in the case of SwitchStatement, it would yield undefined.
In the case of our test failures, the parentCollection would be [undefined] which means a bad indexOf lookup, leading to an error thrown when accessing a property of undefined.
To remedy this, we stop looping at the first parent's key via a break.
In this PR:

Added regression tests
Added patch to tree-iterator

Fixes #916. Fixes #1163.
Notes:
It should be noted that parentCollection is only used by require-multiple-var-decl. Additionally, this touches _buildNodeIndex which may lead to performance speed up (yey).
https://github.com/jscs-dev/node-jscs/blob/v1.12.0/lib/js-file.js#L728-L738

Coverage increased (+0.0%) to 99.09% when pulling 4736b16f93992495b42286b1152ded84f9ae7469 on twolfson:bug/require-multiple-var-decl.sqwished into 95d611e817bcfef613ae2b785167ac82ec6c309d on jscs-dev:master.

Ah, I almost forgot. I have skipped adding regression tests to tree-iterator as it seems tree-iterator lacks any in-depth tests at all =/
https://github.com/jscs-dev/node-jscs/blob/v1.12.0/test/specs/tree-iterator.js
Let me know if you want me to add these/what they should contain besides the regression tests.

Nicely done! Thank you


No compile-error for parathesized await expression in sync function

According Dart specification (30rd Edition, June 2013) "16.30 Await Expressions"
"It is a compile-time error if the function immediately enclosing a is not declared asynchronous"
But if there are (any expression) no compile-error occurs in dart. For exapmle the code below produces no error in dartanalyzer. In dart it produces "NoSuchMethodError: method not found: 'await'"
f() {
  return true;
}

test() {
  await (null);
  await (true);
  await (1);
  await (x);
  await (x + 3);
  await ('hello');
  await (new Object());
  await (f());
}

main() {
  test();
}

Tested on Dart VM version: 1.13.0-dev.7.6 (Fri Oct 30 15:24:19 2015) on "windows_x64"

Not a bug. However, this error is simply a syntax error, because in the context
of a normal function, await has no special meaning. Closed


Add Spanish translations

I clarify that I am from Uruguay and I only put all the spanish so that they have a base

Thank you for your contribution!