[txs] Require users to enter account address if they changed authkey

Motivation
Users can change their account's authkey by using the command txs authkey --new_key $NEW_AUTH_KEY, however the current toolchains would not work with the new authkey. Reason being CLI requires users to enter their mnemonic and then derives authkey, keypair,  and account address from said mnemonics. But the new set of authkey/new mnomenic isn't linked with the old account address, so there is no way to get the correct account address.
This PR requires the user to provide their mnemonic alongside with an optional old account address if they changed their account's address
Test Plan
None
Related PRs
Key rotation #65

Hello thanks for this PR.
I don't think we want to be asking for user input in this module. For library type modules which are imported elsewhere (e.g. GUI) there will be no way to input data. Also I think we should follow Tell Don't Ask pattern here https://martinfowler.com/bliki/TellDontAsk.html.
There are two ways we should retrieve the user address:

From command line flags for each CLI app such as --account
Load from AppConfig using the 0L.toml

Also for future reference there is a rust library called Dialoguer.

This looks correct thank you. Can you rebase this against the V6 branch?

Admins, I think this can be merged. @sirouk @hemulin

LGTM


More Travis deploy script fixes

#14306

Escape characters in the base64
Don't echo back the keystore
Limit the iOS version to 2 dots only
Don't print back the cert repo url


@anyone => Fixes indentation.

I’m just gonna assume that these are correct now, reviewing so much whitespace on GH is not really doable imo.

They're all correct, yeah.


Reduce console output

ARRRR! Using this plugin, the Cap'n is a bit too talkative for me - it would be nice to have a way to not hear anything from him (including the CaptainHook Composer Plugin line) or only hear from him when something has actually been done within a call of composer update or install.
It's probably just one of my weird OCD symptoms but I would prefer if there is no output when it just reports that all of the hooks are already installed - it just weirds me out a bit having so much additional output after an ordinary composer update.
Maybe this could be done by setting an option in the extra section of composer.json with a few different values, among them one that will never output anything and one that will only output stuff when there have actually been any changes to the actual hooks.

Sounds doable, I'll have a look at it

The captainhook install command does currently not abide to the captainhook.json verbosity setting.
Would it be suffice for you if that would be the case so you could configure verbosity there?
This is my composer update output after fixing it locally and setting the Captains verbosity to `quiet?
$ composer update
Loading composer repositories with package information
Updating dependencies (including require-dev)
Package operations: 0 installs, 0 updates, 0 removals
Generating autoload files
5 packages you are using are looking for funding.
Use the composer fund command to find out more!
CaptainHook Composer Plugin
  Using CaptainHook config: /Users/sf/Development/github.com/captainhook/captain-install plugin/captainhook.json

If it's up to me I'd rather have no output at all when the plugin calls the install command and no changes are done to the hook files.
If I am getting this right if I set the verbosity setting in the config to quiet, the actual hook actions don't output anything any more either and just silently fail. That's not what I'd want it to do.
Sorry for being such a pain... :)

No pain at all, I totally get it. I will think about it a bit more and see what I can come up with.


fix(proxy): do not register the server socket into server socket list

There is an issue: the proxy doesn't correctly shutdown. You can run the tests and there are about 15 not working.

@fabienvauchelles how do you run the test? I didn't see any issue on my side for the last days.

You can run the test with:
DOTENV_FILE=packages/backend/test/src/assets/storage-file.env npm run test

Here is the content of the packages/backend/test/src/assets/storage-file.env:
PROXY_REFRESH_DELAY=200
FREEPROXY_REFRESH_DELAY=200
CONNECTORS_REFRESH_EMPTY_DELAY=500
PROXIES_REFRESH_EMPTY_DELAY=500
ASSETS_PATH=dist/scrapoxy/assets
STORAGE_TYPE=file
STORAGE_FILE_FILENAME=scrapoxy-test.json
PYTHON=/home/fabien/.virtualenvs/scrapoxy/bin/python
JAVA=/usr/bin/java
AUTH_GITHUB_CLIENT_SECRET=fake_secret
AUTH_GOOGLE_CLIENT_SECRET=fake_secret
FRONTEND_JWT_SECRET=frontendsecret
BACKEND_JWT_SECRET=backendsecret

I've got also a config for distributed test packages/backend/test/src/assets/storage-distributed.env:
PROXY_REFRESH_DELAY=200
FREEPROXY_REFRESH_DELAY=200
CONNECTORS_REFRESH_EMPTY_DELAY=500
PROXIES_REFRESH_EMPTY_DELAY=500
ASSETS_PATH=dist/scrapoxy/assets
STORAGE_TYPE=distributed
STORAGE_DISTRIBUTED_MONGO_DB=scrapoxytest
PYTHON=/home/fabien/.virtualenvs/scrapoxy/bin/python
JAVA=/usr/bin/java
AUTH_GITHUB_CLIENT_SECRET=fake_secret
AUTH_GOOGLE_CLIENT_SECRET=fake_secret
FRONTEND_JWT_SECRET=frontendsecret
BACKEND_JWT_SECRET=backendsecret

looks like this.sockets.add(socket); does the job. i really want to see it in prod!


Adding even more HLVR entity keys

Should I leave this PR open for now in case you find some new keys?

Sure, I'm looking for some more worldspawn stuff actually

I don't think I'm going to be finding any more of these any time soon.


The amount of bots small

Idk why but i put a lot of checked socks and cant get more than 10 bots, somethimes if i put onboarding timer 500 i get about 30 and it goes down to 10 again

wats your skype ill help you

sejo.dedic4

agar.io patched alot of proxies

I still get 200 bots, just buy HQ proxies and put only 1 bot per ip

i still have 400 snake aka @Lukkez

site of sell hq proxies ?

Now i have 600 =p


s3fs-1.79

This updates s3fs to the latest version and updates the dependency from fuse4x to osxfuse. :beers:

Would you mind rebasing here against the master? Thanks!

@DomT4 -- done!

Merged in https://github.com/Homebrew/homebrew-fuse/commit/cd398442656a21749ad4dad9fb92b1d184e7cf3a. Thank you for your contribution to Homebrew @slnovak; we appreciate it! :smiley_cat:


Word filter for notifications to Telegram

I use PagerMon to receive messages from the Dutch emergency services. All notifications arrive well and are forwarded to a Telegram channel. We only want to receive notifications in Telegram of a specific area. Can an option be built in that notifications with certain words will only be pushed to Telegram? This is very important to us because we now receive useless notifications.

I also use it for telegram. but what is your isseu then?
I only see reports of cap codes that I wish for. What then word filters for?

As I see you are also Dutch so you probably know 112pers. They have a word filter that you can set. Only reports with certain words will come through or not. For example, we only want reports from Vlissingen. Now I cannot filter that because the ambulances come to multiple places in Zeeland. So I would like to set a word filter so that the messages with certain words only come in telegram.

Has been discussed previously.
The best way would be to implement globally for all notification plugins, but not something that anybody has started, Sorry.

Duplicate of https://github.com/pagermon/pagermon/issues/109


Adding raspi_temperature to documentation index for kinetic

I'd like raspi_temperature to be indexed and documented on ros.org.

This seems to be a duplicate of #18098 which has already been merged.
Also your branch has diverged from master and shows as currently not mergable (probably because it conflicts with the already merged patch?).


iron does not honor prompt color set in ipython configuration

I noticed that the IPython REPL does not honor the prompt color scheme I specified in my ipython_config.py
I am trying to use the red and green specified by the Gruvbox Scheme.
When I start ipython in command line, the color shows up fine.

As you can see, I overrode the Token.OutPrompt and Token.Prompt colors using highlighting_style_overrides.
However, the same exact IPython started by iron does not reflect these overrides (despite the fact that the override values show up when I query get_ipython().highlighting_style_overrides

I took a quick look at the source code, and I don't think you are hardcoding this anywhere.
But other than this issue, iron.nvim is the bees knees.
Thanks

whoops. Sorry, didn't realize it was opening ptipython

Don't worry, no harm done!
Glad you enjoy using iron.nvim!
Cheers,
Henry


libgcc: fix Darwin cross compile

I guess this is a fallout from the build variables reorganisation in 1ac5398: the previous infix notation was changed to a suffix notation.
With this patch, Linux versions of libgcc can be cross compiled on Darwin hosts again. Compilation on Linux still works.
Things done

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

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

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

Thanks!

Thanks! Sorry I missed this grepping;

Filed under open-source benefits: crowd-sourced grepping. ;)
It was fun diagnosing this down to such an obvious fix. Thanks for merging.


Everything has been patched

yes

paste the code from the .js file in to this website:
https://skalman.github.io/UglifyJS-online/
it will minify the code. Paste this code into a bookmark with the prefix "javascript:" (ex. javascript:minified_code_goes_here)
click on the bookmark when you want to use it and it should work


Rename JSON Schemas and example files

Both the schema and example filenames had hang-overs from the pre-v1.0 API which potentially cause slight confusion e.g.

use of name meaning input/output id (fixed in the docs and schema descriptions already)
a 'tables' schema which now only applies to 'active' as there is no longer a single 'staged' map
a mix of 'base' vs. 'root'
a single schema which used a mix of lowercase and uppercase in the filename ('sourceID'), inconsistent with the API endpoint it describes and all other filenames

So, while none of this is critical, I'd like to merge this into the v1.0.x-review branch for consideration.

Also note that only two of the example files are currently being linted, due to https://github.com/AMWA-TV/nmos-lint-scripts/issues/3, so I hope to resolve that before we approve v1.0.x-review.


[WIP] Ubuntu 18.04 + k8s 1.12 + containerd

I started some tests to make a stable solution with Ubuntu 18.04, but it isn't really working yet.
When/if merged:
close #154

@xetys I wonder if I could help with this one. Can you explain what is exactly the problem with the upgrade?


multicam pointcloud

I want to control two cameras to get the pointcloud ,then save it as ply format,i also want to control the exposure and the emitter,Can you recommend some useful examples for me to learn? Thank you very much.

Hi @wuhaoqi-hub   If you plan to control these functions by writing your own program, could you tell me please which programming language you are using please (C++, C#, Python), etc.

c++------------------ 原始邮件 ------------------
发件人<EMAIL_ADDRESS>发送时间: 2020年10月13日(星期二) 晚上11:04
收件人<EMAIL_ADDRESS>抄送: "wuhaoqi-hub"<742842498<EMAIL_ADDRESS>主题: Re: [IntelRealSense/librealsense] multicam pointcloud (#7561)

c++

Thank you very much for the confirmation that you are using C++.
Some links to programming information for multicam point cloud, emitter and exposure are provided below.  I have listed one example link for each type of function.  Please let me know if you need further examples after studying these.
MULTICAM POINT CLOUD MERGING
https://github.com/IntelRealSense/librealsense/issues/2556
EMITTER ON / OFF
Code for controlling the emitter on / off state with the RS"_OPTION_EMITTER_ENABLED instruction can be found in the section headed librealsese2 in this link:
https://github.com/IntelRealSense/librealsense/wiki/API-How-To#controlling-the-laser
MANUAL EXPOSURE
There are separate exposure controls for depth and RGB sensors.
RGB exposure
https://github.com/IntelRealSense/librealsense/issues/3224
Depth exposure
https://github.com/IntelRealSense/librealsense/issues/5528

thank you very much!


Adding support to rename json fields in generated structs

I'm looking for a way to manipulate the generated JSON fields on structs.
These changes allow to do that without breaking existing configurations.
I'm wondering if you think this is a good idea or a good way to implement it.

I'm so sorry I never left a review for this PR. I know getting a message eight months later isn't any good, but I think it's better than getting no message ever.
As for the PR, applying renames to JSON would be too big of a breaking change for right now. I understand that sqlc is missing functionality around managing struct generation. It's something that I'm looking forward to fixing in the future.


Remove some unneeded bindings

just clearing up that these aren't used.

👍

Codecov Report

Merging #175 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #175   +/-   ##
=======================================
  Coverage   97.08%   97.08%
=======================================
  Files           4        4
  Lines         274      274
  Branches       19       19
=======================================
  Hits          266      266
  Misses          8        8

Impacted Files
Coverage Δ

...re/src/main/scala/org/typelevel/paiges/Chunk.scala
98.21% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


i49-Y-renaming

It is renamed but not included in archetypes

@daren-thomas

@JIMENOFONSECA: good on the renaming. I'm merging this!
What do you mean about the archetypes? I didn't understand that comment?


Is crow.marine.usf.edu down?

http://crow.marine.usf.edu:8080/thredds/dodsC/FVCOM-Nowcast-Agg.nc.html seems to be down.

@jcothran is listed as the contact on https://www.ngdc.noaa.gov/docucomp/collectionSource/list?recordSetId=2604653&componentId=&serviceType=THREDDS&serviceStatus=&serviceUrl=&search=List+Collection+Sources

<EMAIL_ADDRESS>
It is up now. I guess the system was rebooted recently. Tomcat is not set up to auto-start on reboot. I will have to look into that.


Added dialog module features.

Added dialog module features. I briefly hand tested the functions and everything seems to be working well.

An off-topic comment: I noticed in the doc examples the crate is referenced as tauri_api, but it's actual name is tauri-sys. I could see this becoming confusing.

An off-topic comment: I noticed in the doc examples the crate is referenced as tauri_api, but its actual name is tauri-sys. I could see this becoming confusing.

Yup true, thanks for catching that!

@bicarlsen I changed the module signature quite significantly to use builders instead of option objects, that felt more idiomatic and nicer to use, it also reflects the rust-version of this api much better.


Add subtask to FileMigrationTask to fix file permissions

Based on https://github.com/silverstripe/silverstripe-framework/pull/8924/files

Renamed from "fix file" to "fix folder", added a bit more context on how that's been caused by secureassets only (might not apply everywhere, nearly exclusively a CWP issue), and added something to the "file migration" doc. Otherwise lookin' good


forward only runs one wheel

The forward command only runs one wheel. Reverse runs both.
Anybody seen this behavior?
Running on revision f of the board.

Have you found a solution yet? I'm also seeing this behaviour. I swapped my micro SD with another and found the problem went away.
I'm guessing it's something to do with GPIO pin usage on the original card. The second SD was a clean installation, the only major difference between it and the first SD is that I ran sudo apt-get dist-upgrade on the second.
May try yet another SD to see if I can replicate the problem.

I've now got exactly the same issue. Have a camera plugged in my zero too.

For me the solution was to remake my SD card and not run dis upgrade. Run apt-get update; apt-upgrade; It still works after that finished.


Fix error labels order and price fields links

Fix pricing field error label not linking to the form
#109072494
Toolkit forms use question IDs for linking to the question input.
This is the same for multi-field questions: instead of using the
field name for priceMax/priceMin we should use the parent quesiton
id.
Sort form errors to match the question order
errors_map is sorted according to the question order within the
section.
Returns an OrderedDict to preserve the question order since frontend
templates expect errors to be a dictionary.
Move get_error_messages logic to the ContentQuestion
Similar to .get_data, .get_error_messages for individual quesiton
should be the responsibility of the question class, so we move the
logic to ContentQuestion.get_error_messages.
In order to check that the API didn't return any errors that don't
belong to the current section we compare the error fields with the
list of section fields, and if there are any errors that are not
part of the current section question we raise an exception.
This also takes care of ordering the errors to  match the question
order without sorting since the initial iteration is based on
section question order.

:+1:


WIP: Add tests for update stale and fix any problems found

Coverage increased (+12.5%) to 90.169% when pulling a6826bf542f95baf77e19e87a4ce0af663ee2bae on update-stale into b29468e98cbfb614fea30e9b5541815b8ad39256 on master.


Fix found lldb-server for NDK 26

With Android NDK 26 changes path to lldb-server:

Now used only one lib directory "lib" instead of two "lib" and "lib64"
Path to clang stuff now include only major version code, instead of full version with patches.

With this PR, the extension tries to find lldb-server in:

In old library path with "lib64" directory
In new library path and only for major clang version code
In new library path with full version code

I force-pushed fixes

Thanks a lot for the PR. I'll try to publish a new version soon with this fix. :)


Update windows-ansi to 0.0.5

Fixes https://github.com/com-lihaoyi/mill/issues/2441

CI should pass once windows-ansi 0.0.5 is sync-ed to Maven Central

CI is mostly green, right? 🤔

Thanks, Alex!


DOC: specify and explain the format of a Y92 grouping

Description
The idea of a "Y92 grouping" is referred to in the docstrings, but it is not well defined. A section in the documentation for this is in order.
Tasks

[ ] Add an explanation of Y92 groupings to the documentation

Duplicate of #75 (this issue was created first, but I am removing this one because it is contained within the newer issue)


100% CPU

Type: Bug
I disabled all extensions except the below ones that are absolutely required for my work. Left my PC open for 3 hours, got back. VSCode was using 100% CPU. I assume the whole time. This behavior won't suffice, I can't work if the fan is spinning 10,000rpm all the time.
VS Code version: Code 1.72.0 (64bbfbf67ada9953918d72e1df2f4d8e537d340e, 2022-10-04T23:20:47.539Z)
OS version: Linux x64 5.15.0-52-generic
Modes:
Sandboxed: No
Remote OS version: Linux x64 5.4.0-1030-aws

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-11800H @ 2.30GHz (16 x 2300)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
1, 1, 1

Memory (System)
31.10GB (13.79GB free)

Process Argv
--unity-launch --crash-reporter-id 99537c5f-bf0a-4d29-90c7-25bab3f544c8

Screen Reader
no

VM
0%

DESKTOP_SESSION
ubuntu

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
ubuntu

XDG_SESSION_TYPE
wayland

Item
Value

Remote
SSH: elixir-import

OS
Linux x64 5.4.0-1030-aws

CPUs
Intel(R) Xeon(R) CPU E5-2686 v4 @ 2.30GHz (8 x 2299)

Memory (System)
31.36GB (30.28GB free)

VM
0%

Extensions (20)

Extension
Author (truncated)
Version

better-comments
aar
3.0.2

vscode-icontheme-nomo-dark
be5
1.3.6

macos-modern-theme
dav
2.3.18

good-taste-theme
fae
2.4.0

vscode-graphql-syntax
Gra
1.0.4

mayukaithemevsc
Gul
3.2.3

better-dockerfile-syntax
jef
1.0.2

keen-neutral-icon-theme
kee
1.0.8

dotenv
mik
1.0.1

remote-ssh
ms-
0.90.1

remote-ssh-edit
ms-
0.84.0

material-icon-theme
PKi
4.21.0

solarized-go
rag
0.1.0

vscode-field-lights
sve
0.0.11

vscode-nginx
wil
0.7.2

python
ms-
2022.16.1

vscode-pylance
ms-
2022.10.40

vscode-jupyter-cell-tags
ms-
0.1.6

vscode-jupyter-slideshow
ms-
0.1.5

better-align
wwm
1.1.6

(67 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vscaat:30438848
vsclangdf:30486550
c4g48928:30535728
dsvsc012:30540252
azure-dev_surveyone:30548225
pyindex848cf:30577861
nodejswelcome1cf:30587006
fc301958:30595537
2e4cg342:30596373

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.72.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

/needsPerfInfo

Thanks for creating this issue regarding performance! Please follow this guide to help us diagnose performance issues: https://github.com/microsoft/vscode/wiki/Performance-Issues
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Feature/classdiagrams

Add class diagrams to Guide.md
This seems pretty useful for explaining the typeclass hierarchy to people. And it is much nicer than having this done as part of the build. The only downside I can see is that there is this gravizo watermark on the images.
Review by @non or @tixxit

Hi @rklaehn! We had a pass through the issues/PR with @non during nescala. We are slightly reluctant to introduce a dependency on a non-free service.
What are the alternatives? Can we introduce a SBT task that delegates to graphviz?


Migration for shell workable tests to e2e

What type of PR is this?
/kind feature

What this PR does / why we need it:
This PR migratees shell workable tests to e2e
Which issue(s) this PR fixes:

xref #1062
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

@wzshiming Please review this PR.I have also updated the testdata that will be changed due to adding some more arguments in the cluster.

/ok-to-test

@wzshiming I have made the changes.Please take a look

@wzshiming I think we need to set the context first because the current context is not set otherwise the tests will fail

@wzshiming The issue has been fixed.

LGTM, then squash all the commits

@wzshiming I have squashed the commits

/retest pull-kwok-e2e-test-main

/test pull-kwok-e2e-test-main

The pull-kwok-e2e-test-main was always stuck in TestBenchmarking/Benchmarking/Create_pods

/test pull-kwok-e2e-test-main

/approve
/lgtm


(#889) add LocalTime asserts

I think we should, as LocalTimes are made for this kind of glanularity

But time.minutes shouldBe 12 isn't that hard to type either. It wouldn't hurt to add though

True
On Mon, 15 Jul 2019, 20:30 Leonardo Colman Lopes<EMAIL_ADDRESS>wrote:

But time.minutes shouldBe 12 isn't that hard to type either. It wouldn't
hurt to add though
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kotlintest/kotlintest/pull/893?email_source=notifications&email_token=AAFVSGVQQ7RDHMRDMGK6VQTP7UQCFA5CNFSM4ID3VJQKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZ7NL3Q#issuecomment-511628782,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFVSGTVOYKDGR6FBBL4FH3P7UQCFANCNFSM4ID3VJQA
.

Yeah time.minutes shouldBe 12 is fine.
Merging this.


[5.1] Fix Pivot Timestamps Convertion to Carbon When Is Filled

This is related to Issue
#10243.
When an instance of Pivot is created and date attributes such as
created_at and updated_at are passed, it tries to convert those to a
Carbon instance using the connection’s sql grammar from the parent.
At the time the pivot is being filled, no table neither connection is
already set by the parent model, so the pivot resolves a grammar from
the default connection.
This happens when you have models that use a MySql Connection and other models that use other connection type (SqlServer, Postgres…).
The fix consist on moving $this->setTable($table); and
$this->setConnection($parent->getConnectionName()); before
$this->forceFill($attributes); on the Pivot’s class constructor.
I tried to change the testPropertiesAreSetCorrectly test in
DatabaseEloquentPivotTest.php to reproduce the use case, as the current
test does not test that.
But got an error from PHP Unit pointing to this method in the Model
class:
1) DatabaseEloquentPivotTest::testPropertiesAreSetCorrectly
BadMethodCallException: Method Mockery_33_Illuminate_Database_Query_Grammars_Grammar::getDateFormat() does not exist on this mock object

/**
 * Get the format for database stored dates.
 *
 * @return string
 */
protected function getDateFormat()
{
    return $this->dateFormat ?:
$this->getConnection()->getQueryGrammar()->getDateFormat();
}

I think this is related to the mocking instance, but my testing
knowledge is not enough to fix it.
Hope this helps!, if you need more details i am able to give you if
needed.

Tests failing?

Yes, i already made a change to the test to test this, but i have not much experience with test, can you give it a look?
How to mock the getQueryGrammar method?
Thank you

I fixed the test.

Thank you @taylorotwell and @GrahamCampbell


Improve Java Icon

Ever since I been using Discord integration, One thing has bothered me. That is the Java icon on the rich presence. Since it's the most shown icon, I think it should be less cluttered and have a cleaner look.
NOTE: This is my first Github Issue/Suggestion So, I wanted to do something small. Sorry if I did anything wrong. Also, I'm a big perfectionist, so this is why I suggested this.
Different Photo

Updated Photo

I think it's a good idea to give the all icons a little border, or at least the once that extend into the bottom right corner too much (although others could potentially profit from it too, especially when used as small icon).
Do you still know much you shrank that one?

Sadly I don't remember how much I shrink it down by. If you would like, I can do other images that need to be edited for you. ￼


'ApolloNextAppProvider' is not exported from '@apollo/experimental-nextjs-app-support' (imported as 'ApolloNextAppProvider').

Hi,
My apollo-client file:
import {  ApolloClient, InMemoryCache } from "@apollo/experimental-nextjs-app-support";
import { HttpLink } from "@apollo/client";

export default function makeClient() {
    const httpLink = new HttpLink({
        uri: 'http://localhost:3000/api/graphql',
        fetchOptions: { cache: 'no-store'}
    })
    return new ApolloClient({
        link: httpLink,
        cache: new InMemoryCache()
    })

}

My Layout file:
import { Inter } from "next/font/google";
import "./globals.css";
import { ApolloNextAppProvider } from "@apollo/experimental-nextjs-app-support";
import makeClient from "@/lib/apollo-client";
const inter = Inter({ subsets: ["latin"] });

export const metadata = {
  title: "Create Next App",
  description: "Generated by create next app",
};

export default function RootLayout({ children }) {
  return (
    <html lang="en">
      <body className={inter.className}>
        <ApolloNextAppProvider makeClient={makeClient}>
        {children}
        </ApolloNextAppProvider>
        </body>
    </html>
  );
}

Error:
Import trace for requested module:
./src/app/layout.js
 ⚠ ./src/app/layout.js
Attempted import error: 'ApolloNextAppProvider' is not exported from '@apollo/experimental-nextjs-app-support' (imported as 'ApolloNextAppProvider').

Appreciate any suggestions or help
Thanks

Hey @lalithasj 👋
Are you using the "use client" pragma in either your client file or layout file? ApolloNextAppProvider is only available in Client Components so its possible that Next.js thinks you're trying to build these as Server Components where ApolloNextAppProvider is not available.

Hi @jerelmiller,
Thank you for your response. ApolloNextAppProvider in my layout file (app router) and there is no 'use client' statement above that file. Appreciate any help

You'll need to add "use client" in order for this to work. That will enable this as a client component and will make that import available. Try that out and see if that helps.

Hi @jerelmiller ,
Thanks for your suggestion. Now it's working by adding 'use client' and removing metadata.

Great to hear!


[python] Fix many typeguard warnings

Some post-1.6.0 housekeeping.
We've been seeing literally hundreds of pytest warnings per CI run, which is frictional to sort through on CI fails.
Regarding the dominant error which is no type annotations present -- not typechecking {}'.format(function_name(func)) I could not easily find any where location. (Maddening.) But by comment-out bisection on tiledbsoma.io I found _create_and_open_coll and _create_and_open_collection (and @overload variants), the existence of which on a typeguarded import results in these warnings.
I found https://github.com/agronholm/typeguard/issues/108 and in particular https://github.com/agronholm/typeguard/issues/108#issuecomment-1379547357 indicating that this is a fault of older typeguard versions.
Meanwhile thanks to yours truly on https://github.com/single-cell-data/TileDB-SOMA/pull/1115 we're currently pinned to typeguard<3 which deprives us of the fix for the above.
On the other hand, typeguard 4 has a couple syntax changes to accommodate -- which this PR dutifully implements.

Green on my laptop with Python 3.9; red CI for Python 3.7 .....

@ryan-williams please take over this PR
https://github.com/single-cell-data/TileDB-SOMA/issues/1116#issuecomment-1989370557

Re-enabling typeguard on most of tiledbsoma (see tests/__init__.py changes) surfaced a ton of type-checking errors.
Current status:
= 172 failed, 1472 passed, 1 skipped, 3 xfailed, 3222 warnings in 258.96s (0:04:18) =

It was much higher before! I fixed a bunch, and learned many typeguard/mypy tricks. Cautiously optimistic I can deal with the rest tomorrow.

also, sorry if it didn’t come across in my previous message, but overall this change is a great improvement! my comments are relatively small in the scope of the entire thing.

@thetorpedodog can you approve or are these still changes outstanding?

(R CI failures believed to be unrelated to this change, should be resolved later today…)


Add: style.table (table style)

Need to research table_style and all it's bits. Might require some additional helpers for certain parts of it.

Example from Helmod:
default_gui["helmod_table-odd"] = {
	type = "table_style",
	-- default orange with alfa
	hovered_row_color = {r=0.98, g=0.66, b=0.22, a=0.7},
	cell_padding = 1,
	horizontal_spacing = 10,
	vertical_spacing = 2,
	horizontal_padding = 1,
	vertical_padding = 1,
	odd_row_graphical_set =
	{
		type = "composition",
		filename = "__core__/graphics/gui.png",
		priority = "extra-high-no-scale",
		corner_size = {0, 0},
		position = {78, 18},
		opacity = 0.7
	}
}

Table styles from base game (core/prototypes/style.lua):
    table_style =
    {
      type = "table_style",
      horizontal_spacing = 5,
      vertical_spacing = 5
    },
    electric_network_sections_table_style =
    {
      type = "table_style",
      cell_padding = 5,
      horizontal_spacing=0,
      vertical_spacing=0,
      -- same as frame
      column_graphical_set =
      {
        type = "composition",
        filename = "__core__/graphics/gui.png",
        priority = "extra-high-no-scale",
        corner_size = {3, 3},
        position = {8, 0}
      }
    },
    slot_table_style=
    {
      type = "table_style",
      horizontal_spacing = 2,
      vertical_spacing = 2
    },
    control_settings_table_style =
    {
      type = "table_style",
      horizontal_spacing = 5,
      top_padding = 20,
      vertical_spacing = 7
    },
    browse_games_table_style =
    {
      type = "table_style",
      -- default orange with alfa
      hovered_row_color = {r=0.98, g=0.66, b=0.22, a=0.7},
      selected_row_color = default_orange_color,
      cell_padding = 1,
      horizontal_spacing = 20,
      top_padding = 5,
      vertical_spacing = 5,
      odd_row_graphical_set =
      {
        type = "composition",
        filename = "__core__/graphics/gui.png",
        priority = "extra-high-no-scale",
        corner_size = {0, 0},
        position = {78, 18},
        opacity = 0.7
      },
      column_ordering_ascending_indicator =
      {
        type = "monolith",
        top_monolith_border = 0,
        right_monolith_border = 0,
        bottom_monolith_border = 0,
        left_monolith_border = 0,
        monolith_image =
        {
          filename = "__core__/graphics/arrows/indication-arrow-gui-ascending.png",
          priority = "extra-high-no-scale",
          width = 44,
          height = 24,
          scale = 0.5
        }
      },
      column_ordering_descending_indicator =
      {
        type = "monolith",
        top_monolith_border = 0,
        right_monolith_border = 0,
        bottom_monolith_border = 0,
        left_monolith_border = 0,
        monolith_image =
        {
          filename = "__core__/graphics/arrows/indication-arrow-gui-descending.png",
          priority = "extra-high-no-scale",
          width = 44,
          height = 24,
          scale = 0.5
        }
      }
    },
    browse_mods_table_style =
    {
      type = "table_style",
      -- default orange with alfa
      hovered_row_color = {r=0.98, g=0.66, b=0.22, a=0.7},
      selected_row_color = default_orange_color,
      cell_padding = 1,
      horizontal_spacing = 20,
      top_padding = 5,
      vertical_spacing = 5,
      odd_row_graphical_set =
      {
        type = "composition",
        filename = "__core__/graphics/gui.png",
        priority = "extra-high-no-scale",
        corner_size = {0, 0},
        position = {78, 18},
        opacity = 0.7
      },
      column_ordering_ascending_indicator =
      {
        type = "monolith",
        top_monolith_border = 0,
        right_monolith_border = 0,
        bottom_monolith_border = 0,
        left_monolith_border = 0,
        monolith_image =
        {
          filename = "__core__/graphics/arrows/indication-arrow-gui-ascending.png",
          priority = "extra-high-no-scale",
          width = 44,
          height = 24,
          scale = 0.5
        }
      },
      column_ordering_descending_indicator =
      {
        type = "monolith",
        top_monolith_border = 0,
        right_monolith_border = 0,
        bottom_monolith_border = 0,
        left_monolith_border = 0,
        monolith_image =
        {
          filename = "__core__/graphics/arrows/indication-arrow-gui-descending.png",
          priority = "extra-high-no-scale",
          width = 44,
          height = 24,
          scale = 0.5
        }
      }

    },

@Helfima any idea what difference is between cell_padding and *_spacing / `*_padding' is?
All the cell elements have their own padding, the spacing between rows/cols is set by *_spacing, the space between the table element and its contents is set by *_padding (eg. top_padding)...


Add Ubuntu 20.04 LTS envrionment

Ubuntu 20.04 LTS (https://releases.ubuntu.com/20.04/) is now out and makes sense to add it as a supported environment while deprecating Ubuntu 16.04 LTS.

This is a duplicate of https://github.com/actions/virtual-environments/issues/228

Hi @rigtorp! Thank you for your feedback! Please check details about Ubuntu 20.04 in related issue https://github.com/actions/virtual-environments/issues/228.


Formatting issues in Token API page

Description:
Please see the screenshot below from the Token API page

@ruthryi  can this be closed?


mihomo configuration help

Hello,
I installed the luci-app-nekoclash on my router with architercure mipsel_24kc , i have a probem trying to set up the configuration for mihomo .
I have my own VPS and i have v2ray rurring there. i can use my android phone with V2RAY app and it working fine with my configuration. I tried many time to make it works in my router , but i failed to create the configuration. Could you help me , please ?
This is my configuration that i use on my v2ray , but i want to use it in my router with your tool
vless://683c305c-c9e1-4cd4-b645-7e2ceb02f283@<IP_ADDRESS>:443/?type=tcp&encryption=none&security=tls&sni=my.yahoo.com&alpn=h2%2Chttp%2F1.1#Mex-tcp
or
- name: Mex-tcp   type: vless   server: <IP_ADDRESS>   port: 443   uuid: 683c305c-c9e1-4cd4-b645-7e2ceb02f283   cipher: auto   tls: true   alterId: 0   flow: xtls-rprx-direct   servername: my.yahoo.com   udp: true   skip-cert-verify: true
Can you create a whole or full configuration this way i can only copy and paste it in the uci-app-nekoclash , please ?

You can directly use the link without the need for conversion, removing the "proxies:" at the beginning.

The default configuration is provided, and you can use it directly.

@Auroraxpm Can show me how to add it and use it with my configuration ? Maybe with screenshots or video , please

Join the Telegram group for feedback


Crystal 0.34 compatibility

This PR brings compatibility with Crystal 0.34.

As Errno is now an enum, there was an issue comparing Errno.value (which now returns an enum value) with ZMQ::Util.errno which still returns an integer, leading to raised exceptions.
I also updated broken examples that was not compatible with recent crystals (use of Time.now and Time::Span.ticks).

Thank you.


customFileTypes not overriding default file types

Per #8144 I would expect to be able to set a custom file type association to override the default syntax. Here's my config.cson:

Based on this block in grammars-spec.coffee I would expect that my config above would use the .scss syntax on .css files, but I am not seeing that behavior. Am I reading this incorrectly or doing something wrong?
I am running Atom v1.0.9 on OS X v 10.10.4
cc: @maxbrunsfeld

Your file type specification is incorrect; change "*.css" to "css".

@atom/feedback Would it be worth having some type of validation here, given we could implement one without too much complexity?

:+1: that worked, thanks @thomasjo
cc: @balexand you mentioned you had a similar issue, check this out.

I think validation would be good. I think some documentation is necessary too.

@mmcbride1007 Thanks for pinging me with this. That worked.

:+1: for documentation on this.


Drop Scala 2.10

We agreed on this in #66

Just a small neat (really sorry for bike-shedding): most our "infrastructure/bump" tickets have their "standard" easy-searchable titles, like https://github.com/snowplow/snowplow/issues/3285. This is in the case contributor needs help in implementing them and wants to search prior art.

Sorry about not following your conventions, still trying to learn. 🙌


libCamera make messy :)

Hi,
If there is a good-old days version of the opencv and camera without libcamera would be perfect. bulseye make life harder and ask more cpu :)
thanks

@MyraBaba ,
I agree with you, LibCamera isn't the most wise decision the Raspberry Pi foundation made.
However, if you don't want frame buffers to overflow, or large inference times, you need to use a streamer.
Both, GStreamer and ffmpeg use more CPU than LCCV. Where LCCV uses 10%, GStreamer needs 25%. See here
That's why I use Bullseye and LCCV.
Feel free to modify the code for Buster and GStreamer. Given the repo's here, it can't be that difficult.

@Qengineering  I am modifying let you know.
what will be the best model for heavy intersection road not just like vertical one and also small vehicles?
I tried Yolov7 with large model , quite good but computational gpu expensive.


Display a confused warning message when dequeueing an expired message

Storage Explorer Version: 1.2.0
Platform/OS Version: Windows 10/MacOS High Sierra/Linux Ubuntu 16.04
Architecture: ia32/x64
Build Number: 20180627.6
Commit: 6fdb639d
Regression From: Not a regression
Steps to Reproduce:

Go to Subscription -> Storage Accounts.
Open one queue -> Add a new message and make sure it will expire in a short time.
Wait for it to expire then dequeue the expired the message.(Without refreshing the queue editor)
Check the popped warning message.

Expected Experience:
It should display '... has been removed or expired...'
Actual Experience:
Dispaly a confused warning message.

It says "...has been removed or modified by someone else". Two possibilities. Either the message doesn't actually exist in the queue anymore or it's been modified. I'm not sure how this message is confusing.

Hi Craig, I'm confused about the warning message:
The expired messages are hidden and don't dispaly in the current queue, but its not that have been removed.


assign instance profile to bastion

Works:
$ aws ssm start-session --target i-123abc123abc123abc

Starting session with SessionId: service-deployer-123abc123abc123abc
sh-4.2$ psql --version
psql (PostgreSQL) 12.7
sh-4.2$ exit

(Painfully pieced together from https://docs.aws.amazon.com/systems-manager/latest/userguide/session-manager-getting-started.html and the rabbit warren it links to...)


PySpark rdd methods (collect, toLocalIterator) under moto return empty arrays

PySpark are bindings to Spark clusters. PySpark methods that interact with RDD's on the localhost are empty until the moto context. Below is a simple method of reproducing the issue.
Python 2.7.13 (default, Jan 19 2017, 14:48:08)
[GCC 6.3.0 20170118] on linux2
Type "help", "copyright", "credits" or "license" for more information.

Welcome to
      ____              __
     / __/__  ___ _____/ /__
    _\ \/ _ \/ _ `/ __/  '_/
   /__ / .__/\_,_/_/ /_/\_\   version 2.0.1
      /_/

Using Python version 2.7.13 (default, Jan 19 2017 14:48:08)
SparkSession available as 'spark'.

>>> import moto
>>> moto.__version__
u'0.4.31'
>>> data = [{'a': 1}]
>>> spark.createDataFrame(data).collect()
[Row(a=1)]
>>> with moto.mock_s3():
...     spark.createDataFrame(data).collect()
...
[]

The moto should not conflict with the rpc between python and the java process.

Can you try testing this against the master branch of Moto? My guess is that it is no longer an issue.

Awesome! It's working for me now, I'm curious about the root cause of the problem though. Do you know what patch in particular solved the issue?

I'm not positive; there was a large refactor of the S3 code.


Add jonjohnsonjr

Welcome Jon! You should now be able to ssh to<EMAIL_ADDRESS>from any IPv6 enabled host. Please let me know if you have any problems connecting.


multiple commands per projects

Hi!
I realy like hotel, but one of the big problemes I see is that I can specify only one command on a project.
I would like to know if I would be possible to put multiple commands for a project like I do in my package.json files (npm)
You could then have a sort of dropdown of whatever to choose a command and maybe specify a default one.
Thank you in advance, regards

Hi @benavern,
Good question and suggestion. I'd rather keep it simple. I fear that with multiple commands, it would make the CLI and interface much more complicated. The CLI would need to operate at project level and/or command level.
Would any of these solutions work for you?
1 Use custom names
hotel add 'cmd1' --name app1
hotel add 'cmd2' --name app2

2 Use subdomains
hotel add 'cmd1' --name cmd1.app
hotel add 'cmd2' --name cmd2.app

3 define multiple hotel commands in package.json
{
  "scripts": {
    "activate1": "hotel add cmd1",
    "activate2": "hotel add cmd2"
  }
}

And then in your terminal
$ npm run activate1
$ npm run activate2

4 Add commands to hotel
You don't have to add only servers to hotel, you can also add commands:
hotel add 'hotel add cmd1' --name 'project:activate1'
hotel add 'hotel add cmd2' --name 'project:activate2'

This is maybe what is the closest to your idea. It's not a dropdown, but you can use them to switch between multiple commands for one project from hotel interface.

I like the idea that hotel could read the package.json and provide a switch button for each script it finds inside.
something like this :

The cli could be very similar : when hotel add is called it adds a command rather than overriding.
The same way, hotel rm removes a command rather than deleting the project & when there are no commands left, no project appears in the hotel GUI


Suggest macOS alternative to Rufus

Specify that Rufus is only an option for Windows.
Suggest Apple Boot Camp Assistant (built-in) as an alternative for macOS users.

Not sure what to suggest for Linux other than possibly the following online search:
Linux "ms-sys" bootable Windows ISO

As far as I've seen, BootCamp is only a solution in a minority of cases (it refuses to create an external USB drive on later versions, handling everything external), but @jhermsmeier can probably tell.

Yeah, Boot Camp Assistant only flashes to USB thumb drives (no USB HDDs, no SD cards, etc afaik), so that makes it a bit of a limited alternative.

Still better than suggestion a Windows only alternative.

Cool, yeah, up to @jhermsmeier really.

Still better than suggestion a Windows only alternative.

Yeah definitely, I think so to – was just confirming the limitations.

Awesome, thanks a lot @da2x !


Grasp plugin doesn't work with a gazebo issue

There is a gazebo issue that makes the gripper open and close due to an unknown reason, while you do a motion grasping an object. I thought this plugin would fix that, but as the arm opens and closes, the object is falling down. Do you know any workaround?.
Thank you,

This plugin is meant to release the object once the gripper is not touching it any more. So it's not suitable to fix this bug. Is there an issue in Gazebo for this bug?

@JenniferBuehler  No I don't think there is such issue on this bug. I don't know why it is happening, but it is totally strange as the same thing worked in the Gazebo 2.2.3, but now when upgraded to kinetic we use gazebo 7.12.0 here comes the problem of the gripper, when doing a motion after grasping the object. I created a question in the gazebo answers(http://answers.gazebosim.org/question/20490/issue-with-the-post-grasp-motion-in-gazebo/). I recently created an issue on gazebo(https://bitbucket.org/osrf/gazebo/issues/2528/issue-with-the-post-grasp-motion-with-the)

Ok, thanks for the links. Let's see if there is a response to the issue soon.
Meanwhile, I'll close this, because the grasp plugin unfortunately cannot be used to fix the issue.
Cheers


update packages

:white_check_mark: Build TheIdServer 2.1.1-maintain-1-0-01-0084 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/359075596b by @)


Integration into mobile apps

Is there a way that this can be integrated as a realtime text-to-speech engine for stuff like mobile phones? As well, is WhisperSpeech at that level of realtime inference yet or is there more to be done?

The inference for the currently released models is quite well optimized in PyTorch. If this is too slow then there are also smaller models available (tiny and base) which are quite a bit faster.
Unfortunately I am not an expert in how difficult it is to export these autoregressive models to CoreML or TensorFlow-Lite. They are quite a bit different from traditional image recognition models since we have to run the model hundreds of times to generate the whole sequence.
That said any tutorial on running OpenAI Whisper or Large Language Models on mobile devices would be a great resource since WhisperSpeech is works very much like Whisper or a small LLM.


There is a small problem in AngularJS Git Commit Message Conventions on google docs

Related PR: https://github.com/angular/angular.js/pull/12032

The type is mandatory and the scope is optional

is wrong. It should say The header is mandatory and the scope of the header is optional.
You could also say "The type and subject is mandatory and the scope is optional in the header, and other parts are optional".
I also think a git repo would be easier for us to contribute than google docs...
@petebacondarwin

This has been changed in our contributing.md yesterday. I am not sure we can edit this google doc, or if this change should be tracked here. Wdyt @petebacondarwin ?

I have tweaked the Google doc further. We should probably allow public comments to make it easier to collaborate on that document.

Thanks @petebacondarwin , Looks good for now. Just a few things:

I updated table of contents (as there is some mismatch with the new titles). Not sure if you see my suggestion.
When I open the doc, I'm in "Suggesting mode". I think it would be better to be in "Viewing mode" by default. Not sure if you can change this though. Or is it just my machine.

Thanks @stevemao. I don't think we can control what the default view you get is...


Version catalogs should work in buildscript dependencies

This was previously reported as part 2 of Issue 16652.
Expected Behavior
I expect centralized dependency versions to work everywhere I specify dependencies. This includes my buildscript clause.
In myproject/settings.gradle.kts:
enableFeaturePreview("VERSION_CATALOGS")
...

In myproject/gradle/libs.versions.toml:
[libraries]
shadowJarPlugin = "com.github.jengelman.gradle.plugins:shadow:6.1.0"

In myproject/build.gradle.kts:
buildscript {
  dependencies {
    classpath(libs.shadowJarPlugin)
  }
}
...

Current Behavior
> Configure project :
e: /Users/jwilson/Development/myproject/build.gradle.kts:20:15: Unresolved reference: libs

Context
I’m using Gradle 7.
Workaround
This is a workaround. It is verbose, doesn’t have IDE support, and is inconsistent with my other dependencies blocks.
buildscript {
  fun libs(lib: String): String {
    val versionCatalog = project.extensions.getByType(VersionCatalogsExtension::class).named("libs")
    val dep = versionCatalog.findDependency(lib).get().get()
    return "${dep.module.group}:${dep.module.name}:${dep.versionConstraint.displayName}"
  }

  dependencies {
    classpath(libs("shadowJarPlugin"))
  }
}

@swankjesse
Better workaround that allows for typesafely accessing the versions:
buildscript {
  repositories {
    gradlePluginPortal()
    google()
    mavenCentral()
  }
  dependencies {
    val libs = project.extensions.getByType<VersionCatalogsExtension>().named("libs") as org.gradle.accessors.dm.LibrariesForLibs
    classpath(libs.sqlDelight.gradlePlugin)
  }
}

I have a similar problem, but in a different place. I tried to define dependencies from service catalog for sub-projects and it does not work either.
Example:
subprojects {
   dependencies {
      implementation(libs.abc)
   }
}

Extension with name 'libs' does not exist. Currently registered extension names: ...

I have a similar problem, but in a different place. I tried to define dependencies from service catalog for sub-projects and it does not work either.
Example:
subprojects {
   dependencies {
      implementation(libs.abc)
   }
}

Extension with name 'libs' does not exist. Currently registered extension names: ...

Looks like you're having this issue, which is unrelated

@swankjesse Is this feature VERSION CATALOG supported by Android Studio? I tried it but autocomplete ain't working.

What version of Gradle should this fix show up in?  (I just tested Gradle 7.1 and this seems to still be broken)

I just tested it too with 7.1 and it doesn't work yet. I guess it was fixed too late to come into 7.1. Hopefully it will be included in next release.

Paul's answer in Groovy
        def libs = project.extensions.getByType(VersionCatalogsExtension).named("libs") as org.gradle.accessors.dm.LibrariesForLibs
        classpath(libs.gradlePluginHiltAndroid)

On Gradle 8.5, the solutions above didn't work. In Groovy, this is working
buildscript {
    dependencies {
        def libs = project.extensions.getByType(VersionCatalogsExtension).named("libs")

        classpath(libs.findLibrary("myPlugin").get())
    }
}

libs can't be casted to org.gradle.accessors.dm.LibrariesForLibs. The type is org.gradle.api.internal.catalog.VersionCatalogView, which exposes findXX functions that return Optional.

I had a problem accessing Version Catalog dependency from my Groovy build.gradle file.
Used the following syntax to 'fix' it:
buildscript {
        dependencies {
            classpath "${libs.plugins.your.awesome.plugin.get()}"
        }
    }

get() simply returns a full String version of your dependency. In my case, I can use it normally in classpath.


WIP: SSL on IRC connection

I thought this would be easy -- but so far, it has just been a rabbit hole of sorts.
So I'm posing my current results.
#496
Issues:

[ ] Messages are not forwarded from Zulip=>IRC, the connection breaks and we get:

irc_mirror_zulip has left the room (Quit: Read error: error:1408F119:SSL routines:ssl3_get_record:decryption failed or bad record mac).

[x] No scheduler object, so reconnections aren't scheduled and the whole bridge dies

Hi @benjaoming, are you still interested in working this?

@PIG208 - I would be happy to comment/review on any work extending on this, but realistically I don't have time to finish it. I don't think there's anything in this PR that couldn't be taken further.
I'll try to ask around if anyone would be interested in picking it up.

This PR is now updated with the conflicts caused by black linting.


[enhancement]: Swap foreground and background points on Auto Mask

Is there an existing issue for this?

[X] I have searched the existing issues

Contact Details
skunkworxdark
What should this feature add?
The ability to swap the foreground and background picked points.
Alternatives
invert the mask itself
Additional Content
No response

I added the ability to invert the selection. I think this better addresses the use-case than switching the points, because it leaves the model's internal "semantics" alone. That is to say, the model is designed to select objects, not the inverse of objects. Closing as completed but happy to discuss further if there's a use-case for inverting the points.


duplicate symbol '_rebind_symbols' in:  FLEX/libFLEX.a(flex_fishhook.o)

duplicate symbol '_rebind_symbols' in:
/Users//Library/Developer/Xcode/DerivedData/-aqijccxgiqwcgqaelnmggjvzpszk/Build/Products/Debug-iphoneos/FLEX/libFLEX.a(flex_fishhook.o)
/Users//Library/Developer/Xcode/DerivedData/-aqijccxgiqwcgqaelnmggjvzpszk/Build/Products/Debug-iphoneos/fishhook/libfishhook.a(fishhook.o)
duplicate symbol '_rebind_symbols_image' in:
/Users//Library/Developer/Xcode/DerivedData/-aqijccxgiqwcgqaelnmggjvzpszk/Build/Products/Debug-iphoneos/FLEX/libFLEX.a(flex_fishhook.o)
/Users//Library/Developer/Xcode/DerivedData/-aqijccxgiqwcgqaelnmggjvzpszk/Build/Products/Debug-iphoneos/fishhook/libfishhook.a(fishhook.o)
ld: 2 duplicate symbols for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

fishhook 能不能做成依赖库，集成在项目中和其他第三方 冲突了

Fixed in 4.1.1


Implement follow-on queries

Fixes #12. This is based on Glen's change, but refactored a little.

LGTM. Thanks!


Throw error if instanceID node is missing.

@theywa found that the latest enketo-core returns XML that lacks instanceId. We believe this is a bug.

also tagging @kkrumlian

I think this might be because you're using an old incompatible version of enketo-transformer. There have been some major changes recently (OC-related). Are you experiencing this in any version of enketo-express, or enketo-express-oc as well? Or after upgrading to the latest enketo-transformer (and enketo-core)?

Thanks MartijnR, I tried downgrade enketo-transformer version to 1.7.4 and that solved the bug( checked on participant, edit, and anonymous submit in IE and modern browser ), is it okay to not used latest enketo-transformer version when using latest enketo-core version?

No that's definitely not okay. I added a warning to the changelog.
Can you help me reproduce the issue you have with the latest enketo-core and the latest enketo-transformer? E.g. by completing these reproducible steps:

Start instance of latest enketo-transformer published on npm
Edit enketo-core app.js to use this instance
Edit index.html to use non-minimized js
Start enketo-core with grunt develop
Open form: ......... (link to XForm)

Sure, following your example I found out that this bug only happen to xml file that generated from xls file. No meta and instanceId tag found on it. Here is the link of xml file (  pp.xml ) that generated from xls file ( pp.xls ) also this is example of worked xml file ( birdwatching.xml )

Ah ok, thanks for checking. So the XForm is the issue. The major change in enketo-xslt (and enketo-transformer), also means enketo no longer tries to correct this by adding a missing node. I think this is better, however, enketo should have thrown an error that this form is invalid.

to maintain backwards-compatibility the instanceID is now automatically re-added again (a temporary addition) in enketo-transformer 1.8.2


JSON preview using jq shows files as empty

What system are you running Yazi on?
Linux X11
What terminal are you running Yazi in?
Konsole v23.08.1
Yazi version
yazi 0.1.5
Did you try the latest main branch to see if the problem has already been fixed?
Not tried, and I'll explain why below
Describe the bug
When tab_size is set to 8 (which is the default tab width in a Linux terminal and therefore the preferred one when looking at file diffs for example), JSON files are displayed as empty.
This is because --indent option of jq only allows values in range from -1 to 7 (0 stands for "no whitespace", and -1 stands for using tabs), and yazi is neither aware of that nor does it have any fallback behaviour; this applies to the latest commit in master as well: https://github.com/sxyazi/yazi/blob/24fd587431e91ecb52344a285227497e4ec5d9d3/yazi-core/src/external/jq.rs#L10
(This arbitrary limit exists because for some inexplicable reason jq uses bitmask flags to store indent width…)
Note that I have 2 JSON files which are getting displayed in preview with tab_size set to 8; though considering they have a different colour scheme, I'm guessing jq is simply not applied to them (due to their huge size or perhaps the number of lines).
Expected Behavior
Preview of a JSON file should display the file contents – at least in plain text if the parser failed to process it.
To Reproduce
Set tab_size to 8 and navigate to a JSON file.
Configuration
In yazi.toml:
[preview]
tab_size = 8

Anything else?
When the problem is in the number of spaces, it's still possible to display the requested width by using requesting tabs and replacing each tab at the start with respective number of spaces (e.g. with regexps). Which is probably what the tab_size option is normally used for, anyway…
But either way, a fallback behaviour is necessary since it's perfectly possible for the file to contain invalid JSON (which would cause the parser to bail). …And likely not just for jq since I've seen similar problems with JS files that were apparently deemed invalid. (To compare: MC syntax colouring is primitive and warty, but at least it doesn't cause an empty screen when there's a misplaced comma in the file… or rather, it works just fine pretty much at all times, even if most of the file itself contains gibberish)

Thank you for the detailed explanation of this issue. tab_size=8 causing empty is definitely a potential bug, and I hadn't noticed it – in fact, I never use an 8-tab indentation. I have fixed it in https://github.com/sxyazi/yazi/pull/320, let me know if it works for you.
As for the invalid JSON fallback, I never implemented it, so it's not a bug "but" an "enhancement". Anyway, I've also implemented it in that PR.
BTW, previously there was already a fallback to the built-in highlighter when jq was not installed.

let me know if it works for you

It does. Though I'd like to note that build.sh doesn't run on my system because of +stable (probably cuz I don't have rustup installed). Also I believe it's customary to include build instructions into the main README file (or at least a link to the respective doc).

I never implemented it, so it's not a bug "but" an "enhancement"

The sentiment is familiar but I disagree in this case: the base functionality of displaying file contents doesn't work when a parser fails, therefore it's buggy. Displaying the contents is the base behaviour of this functionality, and should work at least at the most basic level for any file (“the file is just way too big” might be a passable excuse for a failure to display its contents, but “the code in this file wouldn't compile” hardly qualifies as such). An “enhancement,” on the other hand, would be a new feature (or “sub-feature,” I guess) like highlighting of syntactically invalid files.

I think our disagreement lies in the definition of "functionality".
Your definition is too "broad", such as "a preview feature", while I habitually break it down into smaller units like "JSON preview feature", "image preview feature", and "PDF preview feature". You can certainly refer to a "can't preview invalid JSON" as a "can't preview", but this doesn't provide any convenience for our maintenance.
GitHub's issue tracker is a place for developers to conveniently collect and categorize problems. We are used to addressing a specific and particular problem in a single "bug" or "enhancement" issue. I understand that you encountered an issue with build.sh during testing, but please raise it as a separate build-related issue. I just don't want to see a single issue getting cluttered with several "unrelated" problems being solved under it; it can be quite frustrating for me. I know you might argue that they are related: I need to build while testing previews! But please consider it from a software engineering perspective.
Finally thanks for the test, I will merge it now.

Your definition is too "broad"

That's not correct: there is, in fact, a "preview" feature in your app, and it has a baseline expectation of "the file contents being displayed". A "JSON preview" (or an "image preview" for that matter) is an extension of it, which can be considered on its own, but doing so doesn't remove the concept of a "preview" as the base one. These two concepts both exist within the application, and they both have their own areas of concerns; wherein the correct one must be addressed. The lack of a general fallback that would display the file contents as plain text is not a "JSON preview issue", its a "preview issue"; it concerns the general handling of rendering previews, not just one specific extension thereof.
(…As for the build.sh thing, I wasn't trying to report it as a bug so much as I was indicating that most users would have more trouble following your suggestion of trying out this code than they'd care to deal with.)

What you mentioned about the fallback when jq fails to parse invalid JSON, is specific to the JSON previewer. You will encounter this issue only when you depend on jq and use the JSON previewer. You can simply not install jq and use the built-in highlighter without any issues.
This problem is not related to the basic preview functionality. You can look at the code of that PR; it only modifies the logic of the JSON previewer and is unrelated to the basic preview functionality.
Additionally, I disagree with your classification of this as a bug; this is incorrect. I would only consider something a bug in the following cases:

If a feature is described in the documentation, but doesn't work as expected.
If a feature is mentioned in the release logs, but doesn't work as expected.
If there is code in the implementation of a feature, but it doesn't work as expected.

In fact, "fallback when jq fails to parse JSON" doesn't fall into any of the above categories. Also, I don't think it falls under the category of "baseline". What's the likelihood of it happening? I use Yazi every day and have never encountered parsing failures. I'm not sure how you define "baseline", maybe:

I assume it should be considered baseline, because there is a possibility of parsing failure.
There is information about its fallback in MC, so consider MC as the baseline (because you mentioned MC, but I've never used it and don't know its specific workings and the probability of encountering this issue, whether it also detects file types through file, or whether it also relies on jq).

Regarding build.sh, it's just a script I made for my own convenience, not for users. It doesn't appear anywhere in the user documentation. On the contrary, the documentation explains how to build from the source: https://yazi-rs.github.io/usage/installation#build-from-source. Also, if you take a closer look at its contents (I assume you should have looked - don't run any scripts directly; it may harm your computer), it builds binary files for 4 platforms (macOS aarch64, macOS x86_64, Linux, and Windows). I'm not sure why you would need it.


Action Overflow Menu disappears on device rotation

When i perform this sequence of events, the menu disappears:

Click on ... (More Options)
Orientation Change

It is perfectly normal for context menus to disapear on device rotation. Pop up any '...' menu or a context menu in an app like Chrome or the Play Store and rotate your device: the menu will collapse. While I don't believe this behaviour is necessarily prescribed, neither is it even remotely uncommon.  In this particular case, this is a function of the platform.


RTD + Sphinx docs

Add sphinx docs

@hellysmile do you have a RTD account?  Could you make one and get this repo setup w/ it?

Does http://django-activeurl.readthedocs.io/en/latest/ works for You?

Yes, let's add it to the repo description?

Right now docs is bit limited, it is just symlink from readme...
Do You think it makes sense? If Yes, You are more then welcome!

Sphinx needs an .. toctree:: in the index.rst, e.g. https://raw.githubusercontent.com/tony/libvcs/master/doc/index.rst
What do you think about breaking README.rst up into pages?

You are welcome to do it Your way!

It'd be helpful to get push access on this repo. Are you ok w/ that? And pypi/rtd? Could you add my username to those so I can push releases and update rtd settings?  I'm tony on RTD and git-pull on Pypi.

rtd access was just granted!
I am not fully sure regarding future of django-activeurl, maybe I need to talk from some one from https://github.com/jazzband for example and move it there. Ill wait about pypi for a bit.

Yes anything to make the project under better maintainership is good. 👍

Can you replace the link to pypi with RTD?  http://django-activeurl.readthedocs.io/en/latest/

Replaced

I've fixed some code styling today and matrix build
I can do release, when You say that we are ready and docs ready!
Thanks for great job!

@tony ping ;)

I think we're ok to tag a new release right now. no need to wait. We can always update docs after through master.
remember the version is stored in about.py now
When you release, do this:
git tag <version>
git push --tags
that way we got a mark in git for what we released, along with pypi.

just released tag and pypi!


i18n: Add occurrences location in .po and sync messages with Transifex

Description

Extract translation keys directly from src directory instead of using .js bundles
Add occurrences details in .po (filename and line number)
Sort .po content by file location
Sync messages with Transifex

Why
This change should make future integration of Transifex messages easier to review, now that .po files will be generated more consistently.

Parsing .jsx files with the latest version of gettext, or another parser would be more robust. Let's forget this for now.
New i18n messages have been included in #670


Ctrl+C doesn't work as expected

Run the simplest:
from osim.env import ProstheticsEnv

env = ProstheticsEnv(visualize=False)
observation = env.reset()
for i in range(200):
    observation, reward, done, info = env.step(env.action_space.sample())

and try to stop it with Ctrl+C. Instead of stopping it will only print:
^C<built-in function Manager_integrate> returned a result with an error set

It seems it's a fairly general error related to error handling in python3 (and an exception thrown in OpenSim integrator) https://github.com/zulip/zulip/issues/6153
It's beyond the scope of this repo but relevant, I'll try to see if there are some workarounds.

It seems it's a fairly general error related to error handling in python3

It seems it's a fairly general error related to error handling in python3

The issue is not about a general error returned a result with an error set. It could be anything, let's say it's some RuntimeError, doesn't matter.
The issue is about handling this exception inside osim.py.

It's beyond the scope of this repo but relevant

It's within the scope of this repo. In this repo there is a Python code that calls C++. And this Python code is not handling exceptions coming from C++ well.

Good call, exceptions from the integrator were not passed.

Oh, nice, thank you Łukasz :)


Equivalent of caffe iter_size in TF

Can we get the equivalent of caffe's iter_size parameter in TF? This accumulates gradient calcs over several GPU cycles before doing the weight update. It effectively allows a larger batch size. TensorFlow doesn't natively have this but some ppl seem to have implemented sth like it themselves, e.g
https://stackoverflow.com/questions/42156957/how-to-update-model-parameters-with-accumulated-gradients
I think it'd be a useful parameter to have as part of official TF ... or is there some easy way to implement this functionality already?
Shaun

You can achieve this using tf control flows. .I'm going to mark this as "contributions welcome" since I think it would be a nice feature and would benefit the whole community.

I really want a iter_size, or I have to turn to pytorch
Anyone could do something?

Anyone could do something?

It's easy to implement and you can take the code from here:
https://github.com/tensorpack/tensorpack/blob/45ebac959f34507f29176fc12d327f3cc9ff7468/tensorpack/tfutils/optimizer.py#L133-L202

Anyone could do something?

It's easy to implement and you can take the code from here:
https://github.com/tensorpack/tensorpack/blob/45ebac959f34507f29176fc12d327f3cc9ff7468/tensorpack/tfutils/optimizer.py#L133-L202

Thank you for providing so excellent repo. Now I have installed and I am preparing for testing on my previous code. If there is no problem, I'd like to introduce it to others.

Thanks. To use this optimizer it takes is only one extra line of code:
        from tensorpack.tfutils.optimizer import AccumGradOptimizer
        myopt = tf.train.GradientDescentOptimizer(0.01)
        myopt = AccumGradOptimizer(myopt, niter=5)
        train_op = myopt.minimize(loss)

Thanks. To use this optimizer it takes only one extra line of code:
        from tensorpack.tfutils.optimizer import AccumGradOptimizer
        myopt = tf.train.GradientDescentOptimizer(0.01)
        myopt = AccumGradOptimizer(myopt, niter=5)
        train_op = myopt.minimize(loss)

OK,  got it and I am running a program with AccumGradOptimizer. I worry that batch norm will go wrong because someone before gave his code with the same target but I came across many bugs.
So I am testing.
Another problem is learning rate. I should enlarge learning rate according to niter, or not?

I worry that batch norm will go wrong

If your per-iteration batch size is too small and make BatchNorm fail, iter_size will not help you.

I should enlarge learning rate according to niter, or not?

What my optimizer does is written in the code:

An optimizer which accumulates gradients across :math:`k` :meth:`minimize` calls,

and apply them together in every :math:`k`th :meth:`minimize` call.
This is roughly the same as using a :math:`k` times larger batch size plus a
:math:`k` times larger learning rate, but uses much less memory.

Whether you want to adjust learning rate on top of that is up to you. Based on common folk wisdom you do not need to adjust learning rate further.

I worry that batch norm will go wrong

If your per-iteration batch size is too small and make BatchNorm fail, iter_size will not help you.
In that case you'll need cross-GPU BatchNorm (which is also supported in tensorpack), or GroupNorm as a replacement (which I authored).

I should enlarge learning rate according to niter, or not?

What my optimizer does is written in the code:

An optimizer which accumulates gradients across :math:k :meth:minimize calls,
and apply them together in every :math:kth :meth:minimize call.
This is roughly the same as using a :math:k times larger batch size plus a
:math:k times larger learning rate, but uses much less memory.

Whether you want to adjust learning rate on top of that is up to you. Based on common folk wisdom you do not need to adjust learning rate further.

Your repo helps me a lot. But I find that Global Step is not supported while some repos train networks with dynamic learning rate, or other tricks, where Global Step is necessary. So I would like to know whether it's difficult to implement with Global Step? Or other reasons?

But I find that Global Step is not supported while some repos train networks with dynamic learning rate, or other tricks, where Global Step is necessary. So I would like to know whether it's difficult to implement with Global Step? Or other reasons?

It takes about three lines of code to support it.
It was not there earlier because tensorpack already manages global_step by other methods, rather than by the optimizer.

But I find that Global Step is not supported while some repos train networks with dynamic learning rate, or other tricks, where Global Step is necessary. So I would like to know whether it's difficult to implement with Global Step? Or other reasons?

It takes about three lines of code to support it.
It was not there earlier because tensorpack already manages global_step by other methods, rather than by the optimizer.

WOW! So fast updating! What an enthusiastic man!Thank you.

@paragon00,
Could you please take a look at this comment from the developer and try to update to the latest stable v2.9 and let us know if the issue still persists. Thank you!


Multisampling in FrameBufferSystem

My minimal implementation for #6288
Demo: https://www.pixiplayground.com/#/edit/ZbmXFH7nt11L70Ux2jUy8
🎁 Added

Framebuffer.#multisample  - number of samples that user wants. By default its 0 so shortcut if (multisample)  can be used.

Extra operation blit that has to be called after renderer.render if you think that msaa is enabled on renderTexture. Operation corresponds to low-level gl.blitFramebuffer and requires extra internal framebuffer object to bind the texture.

How it works
API is secure - It won't fail in case of WegGL1 or if there's not enough samples.
Internally, colorTextures[0] for MSAA renderbuffer, so, theoretically, MRT still should work. I didn't test it though. However, colorTextures[0] still exists - that's where we blit the data for later usage.
Why not RenderBuffer object

MSAA is slow and eats extra memory.
WebGL2 doesn't exist in Safari.
I didn't see ThreeJS projects that use MSAA in production.

I do not want to add new objects and abstractions in vanilla PixiJS for a case that cant be used in production.
However, it is good as a temporary solution for demos and prototyping games. Basically, its a Toy.
Two-three hundred lines of code are not much, we just mirror WebGL2 API a bit so people can experiment how it looks.
Future:
Both PR's should be small.
RenderTextureSystem
Add high-level blit operation, that can will either use framebuffer blit, either texture copy either scaling filter, that will enable trivial WebGL1 fallback.
FilterSystem
We can enable usage of custom texture pools per object, specify that we need particular renderTexture for dealing with a container, and that texture can have multisampling.
What cant be in vanilla:
MSAA is slow. I don't believe anyone will use it in production even after those PR's.
I'm working on a real solution that can be used in production by heavy projects:

real WebGL1 antialiasing fallback
caching system with atlases
several antialiasing options for containers / graphics
solution for strange conflation in MSAA :

If this PR will be merged, I won't need to add framebuffer hacks in the plugin.
Another plugin
I hope that this thing covers @SukantPal case for filters, maybe not right now but after next  FilterSystem PR.

Also, "4. Strange conflation with MSAA" - I think that is because you are not clearing the frame buffer with the background before rendering to it.

Oh, no, macs, my bane! I hate them really, I dont have one. I'll ask for a testing device on work tomorrow.
There shoulnd't be a problem with clearMode, because its an input texture that should be cleared anyway. Maybe I'll add clear there in case of forceClear just to be compliant.
Exactly, case 4 is done when something is already drawn, and the problem is that users will do it anyway. We need a way to mitigate the damage, my solution is to apply AA to all shapes befure they start combining them.

@ivanpopelyshev Why are using resolution=.125 in your demo?

MacOS/ iOS WebGL2 support is worst.

MacOS/ iOS WebGL2 support is worst.

@eXponenta The other PR works on my machine (the one with Renderbuffer classes). It is actually a problem with this PR. @ivanpopelyshev What machine are you testing on, does it have WebGL 2?

His machine has fully WebGL2 support.

@eXponenta WebGL 2 support depends on the browser I guess, not on the machine.

It isn't fully true.

@eXponenta Well, MacBooks of course have the graphics ability to support WebGL 2. The browsers are supposed to bind the WebGL API to a OpenGL backend.

The heck is going on on apple devices? We'll know tomorrow when I get said devices from test team! Stay tuned!

Codecov Report

Merging #6296 into dev will decrease coverage by 0.29%.
The diff coverage is 24.56%.

@@            Coverage Diff            @@
##              dev    #6296     +/-   ##
=========================================
- Coverage   77.57%   77.27%   -0.3%
=========================================
  Files         186      186
  Lines       10083    10140     +57
=========================================
+ Hits         7822     7836     +14
- Misses       2261     2304     +43

Impacted Files
Coverage Δ

packages/core/src/framebuffer/Framebuffer.js
49.05% <100%> (+0.97%)
:arrow_up:

packages/core/src/framebuffer/FramebufferSystem.js
56.39% <22.22%> (-11.76%)
:arrow_down:

packages/core/src/filters/FilterSystem.js
81.69% <50%> (-0.42%)
:arrow_down:

Continue to review full report at Codecov.

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

 Added a unit test for it. Now it should work on mac devices that somehow support webgl2

Need to test this thing on iOS 13.2 Safari  . @SukantPal whats your device?

@ivanpopelyshev It works now! (macOS Catalina, 2018 i7 MacBook Pro).

@bigtimebuddy @GoodBoyDigital I don't think we should really delay this any longer. If you want, we could finalize this pull request.
I just want you to consider whether we should require users to do a blit after rendering to a multisample render-texture. I think that'll be a big source of bugs.
We might also need to consider blend modes for multisample render-textures. Renderbuffer blits don't support "blending", and this will break some expected behaviors. Instead of blitting directly onto the "texture", we might need to blit into a secondary texture and then render that onto the actual render-texture (via a shader that will blend).

Instead of blitting directly onto the "texture", we might need to blit into a secondary texture and then render that onto the actual render-texture (via a shader that will blend).

This PR doesn't actually forbid a blitting with shader, it doesnt include it on low-level. I use extra filter for blitting in my fork, and I think it has to be part of high-level object, alongside with WebGL1 fallback and some other things.

@ivanpopelyshev I don't understand what you said. "This PR doesn't forbid blitting with a shader". How? First of all, blitting can't be done by a shader (by definition). It just copying the pixels from a render buffer onto a texture (no blending, direct copy).
"I use extra filter for blitting in my fork, and I think it has to be part of high-level object" - that doesn't make sense.
All my complaint is that we're going to have to add a line after rendering to use multisample in PixiJS -
app.renderer.render(circle, buf);
app.renderer.framebuffer.blit();

What is the problem if the blitting is handled as a part of the rendering pipeline. I don't think any user would expect to a blit after calling render. Do you have an example of a framework that uses these semantics? You said you wanted "users to know about the blit operation". Now @bigtimebuddy & @GoodBoyDigital will have to decide whether that reason is enough to separate blitting from the rendering process.

@ivanpopelyshev I still don't understand why your example uses 1/8th resolution on the textures. That just nullifies the effect of multisampling.

I still don't understand why your example uses 1/8th resolution on the textures. That just nullifies the effect of multisampling.

To actually see that it works.

What do you see on a 4K screen? If you’re on a 4K screen, it will still just render as normal.
On my screen, it has jagged edges and doesn’t look like a circle. You’re supposed to use 1x resolution so the multisample makes the effective resolution 8. The whole point is to multisample each pixel 8 times, not multisample 8 pixels 8 times.

Codecov Report

Merging #6296 into dev will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #6296   +/-   ##
=======================================
  Coverage   74.70%   74.70%
=======================================
  Files         105      105
  Lines        6053     6053
=======================================
  Hits         4522     4522
  Misses       1531     1531

Continue to review full report at Codecov.

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

Codecov Report

Merging #6296 into dev will decrease coverage by 0.42%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             dev    #6296      +/-   ##
=========================================
- Coverage   74.7%   74.27%   -0.43%
=========================================
  Files        105       85      -20
  Lines       6053     4615    -1438
=========================================
- Hits        4522     3428    -1094
+ Misses      1531     1187     -344

Continue to review full report at Codecov.

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

@ivanpopelyshev Should we have a fallback to higher resolution if multisampling is not supported on the device? When forcing WebGL 1, the half-red pixels do not show up.

@bigtimebuddy @GoodBoyDigital I approved this - but I did have concerns about requiring a blit operation after render. There is also no documentation regarding this (is there?)

@SukantPal good catch! I put it in PIXI.Framebuffer#multisample and linked from PIXI.MSAA_QUALITY

Thanks everyone for getting this feature through!


Discover Page is empty

Altme version: 2.18.2 (555)
Device: iPhone12
I wanted to add the blockchain network card to my imported account. However, the "Discover" page is empty. Therefore, I can't add any card. I'm not using a profile either (default Altme).

Hello @dansan566
Thank you for submitting your first issue to this repository! A maintainer will be here shortly to triage and review.
In the meantime, please double-check that you have provided all the necessary information to make this process easy! Any information that can help save additional round trips is useful! We currently aim to give initial feedback within two business days. If this does not happen, feel free to leave a comment.
Please keep an eye on how this issue will be labeled, as labels give an overview of priorities, assignments and additional actions requested by the maintainers:
"Priority" labels will show how urgent this is for the team.
"Status" labels will show if this is ready to be worked on, blocked, or in progress.
"Need" labels will indicate if additional input or analysis is required.
Finally, remember to use the official Decentralized Dapp Protocol to initiate a chat with the support team to get immediate support via our Decentralized Portal.
Note: Use the chat 💬 icon at the bottom of the page to get started.

As mentioned in #3152, I was able to add the credential with version 2.17.0. All of a sudden, the Discover page is not empty anymore in 2.18.2:

I deleted comment from @morarhen97 , we don't know this person and the links provided were... suspect at least.

@dansan566 AssociatedAddress credentials are now created automatically when accounts are created so you won't see it in discover anymore.
The empty discover screen with default profile is strange, it may be specific to testing version 2.18.2 (555). We will see if this happens again (currently I don't have the issue when creating a new wallet).


New Drafts Created for Unread Messages on Every Loop

Issue Report: New Drafts Created for Unread Messages on Every Loop
Issue Description:
When looping through unread messages in the application, a bug is causing the creation of multiple drafts for the same message thread. This results in repetitive replies being generated and sent to the same message, cluttering the communication and potentially causing confusion for users.
Steps to Reproduce:

Open the application and log in to the user account.
Navigate to the message center or inbox where unread messages are present.
Observe that the application creates drafts for each unread message.
Receive subsequent new messages in the same thread or conversation.
Note that the application fails to remove previous drafts and instead creates additional drafts for the same message thread.

Expected Behavior:
The application should only create one draft per message thread when looping through unread messages. If subsequent new messages are received in the same thread, the application should remove the previous draft and generate a fresh draft for the updated conversation.
Actual Behavior:
The application currently generates multiple drafts for the same message thread during the loop, leading to repeated replies and unnecessary clutter in the messaging system.
Impact:
This issue negatively affects the user experience as it causes confusion and may result in redundant messages being sent to recipients. It also increases the workload for users who have to delete duplicate drafts manually.
Proposed Fix:
To resolve this issue, the following changes are suggested:

Modify the loop logic responsible for creating drafts to check if a draft already exists for a specific message thread.
If a draft exists for the thread and subsequent new messages are received, remove the previous draft and create a fresh draft for the updated thread.
Ensure that the application does not keep replying to the same message over and over within the loop.
Implement unit tests to verify the correct behavior of the new logic and prevent regressions.

Additional Notes:
The issue appears to be localized to the loop that handles unread messages and draft creation. Care should be taken while implementing the fix to avoid unintended side effects on other parts of the application related to message handling and draft creation.
Attachments:
No attachments are currently available for this issue report. Further investigation may require access to the application's source code and relevant logs.
Priority:
The priority of this issue is deemed High, as it impacts the core functionality of the messaging system and affects user communication. The prompt resolution of this bug is crucial to maintaining a smooth user experience.

Thank you for your detailed bug report. We've investigated the issue and made changes to prevent the creation of multiple drafts for the same message thread.
Specifically, we've adjusted the application to only fetch unread emails from threads that don't already have a draft. This was accomplished by getting a list of all drafts, extracting the thread IDs from these drafts, and excluding these thread IDs when fetching unread emails.
However, please note the following limitations of this solution:

The Gmail API's users().messages().list() method only returns the first 100 messages that match the query. If there are more than 100 unread emails, some of them might not be processed.
If a new email arrives in a thread after the draft has been created but before the user sends the draft, the new email won't be included in the draft.

These limitations were deemed acceptable for the current use-case of the application. We recommend testing this fix in your environment to ensure it meets your needs.


Switch to structured logging

Review all log messages and rework them to use structured logging for parity with other system elements.

@sameo, @sboeuf  - thinking about this, it might make sense to add a logrus.Entry field to types such as Pod and Container since that way:

the constructors could add in some basic fields (such as podID / containerID).
the object-specific functions could then call something like...pod.Logger.WithField("error", err).Error("failed to foobar the pod")

... and it would expand to a useful log entry. Thoughts?

In the end, I took a slightly different approach in #426 by just adding a logging receiver as that seemed safer, given that there are places were objects are created without a constructor function.


HELP ME

I run the grabber on my pc to test it, and i forgot that it would block the antivirus sites.
Now i cant unlock them.
I have already tried running unblock_sites.py multiples time using different apps but it doesnt change anything.
Please help me on what i should do now to unblock the sites..
And yes whenever i run unblock_sites.py multiple black cmd windows open but close instantly..

You need to open cmd in the folder by typing cmd in the address bar of the Extras folder and then typing python unblock_sites.py

@gabuja just goto he extra directory and run the unblock_sites.py or maybe open cmd in admin and cd to tat directory and then run py file

@gabuja just goto he extra directory and run the unblock_sites.py or maybe open cmd in admin and cd to tat directory and then run py file

Thank you so much it worked ur great!


Fixing user model test's last test which fail or report a false positive

fixing up a user model test which was not setup correctly without the async done() callback, which led to false postivies. Adding timeouts to the test ensures that the test completes in time, otherwise mocha's 2s timeout will fail the test

addresses issues with https://github.com/meanjs/mean/issues/770 and https://github.com/meanjs/mean/issues/486

@ilanbiala  @mleanos review please?

@ilanbiala we're good?

I'd like to push this fix in as well as other tests I have worked on in another PR ASAP so we can get that tests safety next as wide as possible.
Anything else @codydaig @ilanbiala @mleanos ?

Yes LGTM.

Great, thanks.


Closes #42. Implement anonymous functions

This PR introduces anonymous functions into the language. Anonymous functions are created using the fn keyword, followed by one or more clause definitions, and finally terminated with an end keyword.
Clause definitions are indicated using a "stab" (->). Note that this is not an operator (it can't be overloaded, nor used in any situation other than clause definitions inside anonymous functions.
Clauses are constructed by the stab, a parenthesized parameter list (even if no parameters are given), and a block body using either the {...} or do...end syntax.
The main use case of anonymous functions is to define complex behavior locally, without having to leak entries into higher scopes (e.g., current_self). They also have the benefit of being more terse than regular method definitions, making them even more useful in scripting contexts.
More details about how this syntax was decided on can be seen in #42. More discussion on the syntax is always welcome.
Here's an example of an anonymous function being assigned to a local variable, then called a few times with various arguments:
foo = fn
  ->(1) { 2 }
  ->(2) { 4 }
  ->(3) { 6 }
  ->(a) { a + 2 }
end

IO.puts(self.foo(1)) #=> 2
IO.puts(self.foo(2)) #=> 4
IO.puts(self.foo(3)) #=> 6
IO.puts(self.foo(4)) #=> 6
IO.puts(self.foo(5)) #=> 7

Note: this PR only implements anonymous functions as a concrete concept. For them to be fully utilized, more changes to allow for function capturing and usage as block parameters will be needed.


Please invite me to the GitHub Community Organization

Name
Smruti Ranjan Lenka
Discord Username (if applicable)
Smruti Ranjan Lenka#6621
Additional Context
I see this opportunity as a way to contribute to an exciting organization and wants to become a part of it.

It's great having you contribute to this project
If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord Server and our GitHub Organisation - we help and encourage each other to contribute to open source little and often 🤓 . Any questions let us know.

Invitation sent for the GitHub Organisation. Welcome to the community 🎉#
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see, you can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/jaidevstudio/people
Like this

Tips for practising:
Remember contributing to open source is not just about code, its about collaboration, communication and adding value
I hope that helps

Please check your email or organisation page 🙏


CRC32 returns variable results based on platform

Apology - after additional testing it seems the feeding mechanism has issues between the architectures

OK. I'm closing it then.


Tasks (tests, suites, hooks) that are too fast show a duration of 0ms in Task results

Clear and concise description of the problem
I was writing a custom reporter and noticed that many of the tasks I was working with were showing 0ms as their duration. Upon further digging, I noticed that Vitest uses Date.now for a lot of measurements related to timing.
I asked @sheremet-va why that is, and he mentioned that global.performance required an import from node:performance at the time of writing. Importing from node:performance would've broken browser mode for Vitest, so the team elected to use the more compatible (but less precise) Date.now to take measurements.
Suggested solution
Now that global.performance is available in all node versions that Vitest supports, we can go through everything to do with timings and swap from using Date.now to using global.performance
Alternative
We could decide not to do this.
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

I'd be happy to take on this issue. Could you assign it to me? 👀


Make y optional in dask.array.learn

See https://github.com/dask/dask-ml/pull/4
Alternatively, I could just copy this module over to daskml and deprecate it in dask.array? That'd probably be best.

Deprecating dask.array.learn also sounds fine to me


Generic pp for summaries

Currently, the pretty-printer displays an ASCII version of the aggregated summaries.
The idea is to provide a generic representation, Document.t and, then give a printer to a specific file format:
+--------------+       +--------------+      +-----------------+
|              |       |              |      |                 |
|  Summary.t   +------>|  Document.t  +----->|   PP_<type>     +-------> Output
|              |       |              |      |                 |
+--------------+       +--------------+      +-----------------+

As there is now lib_context recorder, this is not needed anymore. Feel free to reopen.


Trick google colab to get webui applio running

If you guys somehow nuke Linux Out-of-Memory Killer (maybe editing systemctl things since in colab we are the root user) on colab and use smart ways to clone it we could still using applio webui. also since we are using .venv for google colab things it will be harder to detect

@laynz28 ???

ops sorry my bad


✨ data workflow: various enhancements

AI-tools maintenance

Combine "Expert" and "Datasette Oracle"
Add GPT4o
Update cost estimates
Remove "Meta Expert"

Wizard theme

Align theme of Wizard with OWID's site

Chart diff

Add reject option

Indicator upgrader

Fix error distribution tab ⚠

@Marigold I've added a reject option to chart-diff. I think we should now go and add support on the chart-sync side for this.
As we've discussed in other threads, I think that the behaviour of chart-sync should be:

check if all charts have been reviewed (either approved or rejected)
if some is pending (i.e. approval_status='pending'), owidbot should raise an error and CI/CD should fail (as now).
once the PR is merged, only those charts with approval_status='approved' should be merged (i.e. overwrite master counterparts if existent). The remaining ('rejected' or 'pending' if user ignored the CI/CD error) should be dropped.


Switch to ruff

Fixed in https://github.com/s0undt3ch/python-tools-scripts/commit/f7519250a2e0303f8a3731f60f75fdfb6da464b4