Pull /json/protocol in non-headless?

I was expecting remote:true to pull /json/protocol when using a recent Chrome that's not headless.
But it appears that a UA sniff for headless is required to get that to work. Perhaps it should just be checking against chrome version only?

Ah I see you completely changed this in master recently. :)
Perhaps you now default to remote and let the user force to local themselves?

Yep, it's remote by default since 1457e73c4893a23241406d9a174930471da2d30c. Users can alter this behavior by setting local: true or by passing a custom protocol.
I also recently removed the UA check (d4c110316016a66d0ce1f19ab0954f4a5f7da135) since in all the cases the protocol was fetched from /json/protocol and the check merely ruled out unknown implementation, e.g., Opera, CEF, etc. #300

awesome. that's great. :)


Justify Text Content

Is there a way to justify text?
If not kindly add the functionality.

Yes, it is possible 😄 I have updated the documentation to provide a better visual clue about this functionality.
https://www.questpdf.com/api-reference/text.html#text-alignment

Thank you.


fix: encode IDs as JSON map keys properly

The encoding/json docs explain that you need to provide a MarshalText
method to encode integer types as map keys, otherwise they will be
formatted as a string version of the decimal number.
Providing MarshalText and UnmarshalText also uses those methods as a
fallback for MarshalJSON and UnmarshalJSON, so we no longer need
explicit versions of those latter methods.
Apparently Sources were using IDs as map keys and were providing the
,string attribute on the JSON tag on the struct. This was not correct so
that attribute has been removed. Existing sources will no longer be
readable as a result of this change.
Fixes #13277.

I have been told that Sources can be removed (https://github.com/influxdata/influxdb/issues/13325), so removing the ,string part from the JSON tag on the Source struct should be noncontroversial.

I'm nervous about merging a breaking change like this shortly before our upcoming milestone, but it might be better to get this in sooner rather than later, to avoid having to do a migration.


FR: env flex & new Plugins - Todo list

[ ] flexible/async-rest - shouldn’t be using Docker
[ ] flexible/endpoints - does appengine:deploy work?
[ ] flexible/* - See if we really want to use EnvVar’s to pass values,
perhaps we should modify to use web.xml context vars.
[ ] flexible/memcache — NEED WORK

Obsolete


Don't crash when reading header if 'main.tf' not found

Prerequisites
Put an x into the box(es) that apply:

[x] This pull request fixes a bug.
[ ] This pull request adds a feature.
[ ] This pull request enhances existing functionality.
[ ] This pull request introduces breaking change.

For more information, see the Contributing Guide.
Description
There was a regression added in #217 in which if there's no main.tf file available in the module, executing any commands of terraform-docs fails with Error: open main.tf: no such file or directory error. This PR handles the situation and:

silently ignore the error if the file missing is main.tf
shows error and exit 1 if the file missing is anything else (i.e. --header-from was explicitly set by the user)

Issues Resolved
Fixes #232
Checklist
Put an x into all boxes that apply:

[ ] I have read the Contributing Guidelines.

Tests

[x] I have added tests to cover my changes.
[x] All tests pass when I run make test.

Documentation

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Code Style

[ ] My code follows the code style of this project.

Many thanks for the fix.


Coordinate off

Hi David,
Great plugin, but the image map always have the left coordinate off 380px (for example we set 920 but the plugin is dropping to 541). Any idea what could be wrong?
Thank You,
Vicky.

Can you make a simple example page and I'll have a quick look.

Hi David, Thank you for your answer. This is the link to the page
http://www.pwcworkday.com/index.php?id=110
As you can see the links are in the middle of the banner, and they should
be on the right side of the n
banner.
Thank you for your help.
On Sat, Sep 26, 2015 at 3:27 AM, David J. Bradshaw<EMAIL_ADDRESS>
wrote:

Can you make a simple example page and I'll have a quick look.
—
Reply to this email directly or view it on GitHub
https://github.com/davidjbradshaw/image-map-resizer/issues/28#issuecomment-143406278
.

You need to size the coords to size of the unscaled image, not the displayed size set in your HTML.


Encumbrance bar rendered incorrectly when sheet permission is 'Observer'.

The bar renders incorrectly from the perspective of the Observer:

fixed


Unrequire phrase ethereum

phrases should not be required for creating the client or reading the blockchain
a conditional was added during the constructor to check if a phrase was passed, and consequently not call client.setPhrase( if it is not
~i also noted that the RootDerivationPaths was not defined in the eth package~ looks like it was just my editor.

(#376 should also resolve this issue)


UV index is always 0

Hello,
I use OpenWeatherMap and my UV index is always 0, although the weather station provides a correct value.
I use the Eve (with grouping) compatibility.
Thanks

I have the same problem

Are there any logs produced?
Looking at the code, it seems that UVIndex and DewPoint are handled specially. So is DewPoint also an issue?

Yes, the dew point always shows -50 degrees C.

See https://github.com/naofireblade/homebridge-weather-plus/issues/274#issuecomment-1898786420

Active work on this issue is in #274,
Duplicate of #274

Closing as a duplicate of #274
This report is older, however, active discussion is happening in #274


Shows larger color selector on drawtool options

Displays a full color selector instead of the simple 8-element palette.  Original 8 colors are still available as a palette.

Just a single line will add that functionality or are you missing other code?

No other code is needed, the change is complete as is.


Xiaomi Multimode Gateway 2 (EU) - Device Offline

Currently I have two gateways in my network:

Xiaomi Gateway v3 - ZNDMWG03LM
Xiaomi Multimode Gateway 2 (EU) / Xiaomi Smart Home Hub 2- ZNDMWG04LM - lumi.gateway.mgl001

Both are in the same network. I was able to add integration for the first one, without any problems.
But I can't connect to my second gateway.
Gateway is added to my Xiaomi Home app and is online. I used mail and password to add integration in Home Assistant. So when I clicks 'Add integration -> Xiaomi Gateway 3' I see option for my second gateway (ip address is correct):

But every time I try to add it, I received error "can't connect to gateway" - or similar. And in system logs I see info "Device offline":

Do any of you have any ideas what could be wrong?

Are you sure that IP address correct? Your server in same LAN network segment? And you don't have any restrictions between local devices?

Ok, server was in subnet <IP_ADDRESS>/24 and gateway was in <IP_ADDRESS>/24. I moved gateway to the same subnet as server, and now it's working 👍
Second gateway (Xiaomi Gateway v3 - ZNDMWG03LM) is working even if it's in different subnet 🤔
Thank you for your help!

The problem will be only after gateway restart.
Integration uses miio protocol only for opening telnet.
But miio protocol won't answer on request from another subnet.
The first gateway will execute the command from the other subnet, but will not respond to it.
How the second gateway will behave, I do not know. I haven't tried it.


d.ts files are getting ignored when entrypoint is a js file

which leads to jsr resorting to trying to infer types trivially, and as a result - failure to use the library altogether.
mcve
deno.json
{
    "name": "@foo/bar",
    "version": "1.3.5",
    "exports": "./mod.js"
}

mod.js
const exports = {};

exports.ns = {};
(function(ns) {
    ns.bar = "egg";
})(exports.ns);

export const ns = exports.ns;

mod.d.ts
export declare namespace ns {
    const bar = "egg";
}

the above is similar to commonjs output from tsc for typescript namespaces

expected result
jsr uses d.ts files for typings and doesn't try to infer them on its own
actual result
jsr warns about "slow types" when publishing and advises to add d.ts files (which are already there):
warning[unsupported-javascript-entrypoint]: used a JavaScript module without type declarations as an entrypoint
 --> /path/to/mod.js
  = hint: add a type declaration (d.ts) for the JavaScript module, or rewrite it to TypeScript

  info: JavaScript files with no corresponding declaration require type inference to be type checked
  info: fast check avoids type inference, so JavaScript entrypoints should be avoided
  docs: https://jsr.io/go/slow-type-unsupported-javascript-entrypoint

btw, linked docs might be out of date? there's nothing mentioned about js entrypoints

additional context
due to this bug, export const ns from the mcve, whose type can't be trivially inferred, results being unusable at all:
// some other package
import { ns } from '@foo/bar'

console.log(ns.bar) // errors here, ns is not a namespace

In Deno, you have to explicitly link your .d.ts files to your .js files because Deno's module resolution algorithm does not probe for sibling files. See https://docs.deno.com/runtime/manual/advanced/typescript/types#using-the-triple-slash-reference-directive.
In your case, you want to add a /// <reference types="./mod.d.ts" /> to your JS file.

i believe error message/docs could be improved then.
this difference is currently not mentioned anywhere in jsr specifically, so coming from node background this behavior is pretty unexpected

Yes - we'll improve the docs and error message.

@lucacasonato I just ran into this as well. Would you consider adding a types field to jsr.json specifically for JS entrypoints to specify their .d.ts files?
The reason is that this is a bit of a mess to implement for each package, as including the triple-slash directive before tsc runs causes errors, so my process ends up like this:

Run Rollup
Run tsc on the rolled up file
Prepend triple-slash directive to file

I understand that Deno doesn't crawl around the directory looking for .d.ts files, so maybe allowing us to explicitly specify the type declarations file in jsr.json could improve the developer experience?

You can now specify // @ts-self-types="./types.d.ts" instead of the triple slash reference comment. This doesn't break TypeScript so you don't need to do it after type checking :)

Ooh thank you! I will give that a try.


Antehandler for blocking validator configured sender

Closes: #XXX
need to change to checktx-only
Description

For contributor use:

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer

Codecov Report

Merging #1282 (8c39515) into main (7d45d8e) will increase coverage by 0.01%.
The diff coverage is 32.00%.

@@            Coverage Diff             @@
##             main    #1282      +/-   ##
==========================================
+ Coverage   20.08%   20.09%   +0.01%
==========================================
  Files         203      204       +1
  Lines       26786    26811      +25
==========================================
+ Hits         5379     5387       +8
- Misses      20409    20424      +15
- Partials      998     1000       +2

Impacted Files
Coverage Δ

ante/sendblock.go
32.00% <32.00%> (ø)

Continue to review full report at Codecov.

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

The desired goal imo is for validators to have a mempool block for addresses only being able to send to other addresses, for when they wish to help with scammed mnemonic recovery, recover staked / bonded assets.
The prior state of the art involved was brittle - relying on spamming the mempool, with a race condition to get your tx first. (And getting as many validators to be locally spamming near the time of unlock)
The idea of this method is to allow validators who agree to add a mempool filter locally.


feat(internal): Bump kraft cloud commands to the latest version

Prerequisite checklist

[x] Read the contribution guidelines regarding submitting new changes to the project;
[x] Tested your changes against relevant architectures and platforms;
[x] Ran make fmt on your commit series before opening this PR;
[ ] Updated relevant documentation.

Description of changes

Once https://github.com/kraftcloud/go-sdk/pull/11 is merged, the second commit needs to be updated and then this will be ready for review.
GitHub-Depends: https://github.com/kraftcloud/go-sdk/pull/11

Closed so @nderjung stops looking at it and starts looking at #1018


the offline pm broke.

the user is online, but still it says its offline and send pm.

happens sometimes, not my fault. Underlying lib is showing them as offline ;[

yeah this will be fixed when bot gets upgraded to 1.0 (when its out), until then, either don't use it or expect it to screw up sometime ;p


slide view loaded event not firing

Slide view loaded event is not firing on iOS (probably on Android too, I've not tested)
nativescript 2.3.0
tns-core-modules 2.3.0
tns-ios 2.3.0
nativescript-carousel-view 2.0.1
simulator iOS 9.3
How to reproduce:

Add this snippet to app/Views/Slides/slider-view.ts in the demo app:

export function onLoaded(args) {
    console.log('slide loaded')
}

Make sure slider-view.xml exposes loaded event (should be there by default)

Run the demo on iOS

Tap on "Add Page" and see console messages

@terreb i am releasing a new version with a fix for this one. onLoaded() will be called, that will also will allow you to add gestures to labels.

3 days past since you closed the reported issue but it's still here in the latest version.

Theres no issue with the plugin. You must be doing something wrong on your end.

Ok, I got it. You are not going to fix the plugin of multiple issues, but rather give your ridiculous statements that someone doing something wrong. I don't know whether your employer punish you for not closed issues and you close them without checking or you want that community think that plugin is free of issues. But in any case your behaviour is more than weird.

onLoaded event is not fired, just fix this

Are you trying to bully me? Because if thats the case, you are getting completely wrong. This is something i do in my free time and im employee of no one in that case.

Just re-read your own comments here. You said "i am releasing a new version with a fix for this one. onLoaded() will be called, that will also will allow you to add gestures to label" agreeing that there is an issue with loading event and closed the issue without releasing it. Then I reminded that the issue is still here, loaded event is still not firing. Your comment "Theres no issue with the plugin. You must be doing something wrong on your end." Is not this weird? Same in another issue You first said there was no issues with my example and all was working for you, but then you said "Set orientation to horizontal (this doesnt work, i had same issue in xamarin)". In third issue:  This plugin is a really must-have for NativeScript and I'm actually trying to test it and reveal issues helping community and you to make it better, but you position is kind of strange.

I said i am releasing a fix for onLoaded and i will, i just wanted to ship another useful functionality with it: page indicators.
Yes, the fix for onLoaded will allow you to add tap events programmatically to labels. Thats a workaround.
When i say horizontal stacklayout doesnt works and thats its also an issue with Xamarin version of the plugin i couldnt solve, i am offering a workaround if you use a grid instead.
Theres nothing strange on that. Strange is you getting upset and trying to be bossy.
I am leaving 2 issues you reported opened because i cannot offer a workaround to them, particularly CSS not being applied. I believe its critical and it should be solved. I am working with {N} team in that one.
I will release a quick fix today for onLoaded().
Thanks.

I also believe this is a really useful plugin for the community and i am also trying to make it as stable and flexible as possible. Thanks.

Version 2.0.2 released. Now you can use onLoaded() to hook up tap event to labels.
https://docs.nativescript.org/cookbook/ui/gestures

Awesome, it works perfectly, thank you @alexrainman !


node save_image not work as base from ComfyUI

normal node image_save can use prefix about say as here https://blenderneko.github.io/ComfyUI-docs/Interface/SaveFileFormatting/
can use value from variable %node.widget% , your version of this node has more customization, but lost old functionality.

Yes, the custom node I made for saving images has a specific use case that is covered by its implementation. It's not meant to be a general replacement for the ComfyUI node.


Mixed versions across workspaces

Hi there.
We are evaluating to replace Lerna with Yarn workspaces + Changesets and need input if the following concept is possible with changesets:
  "workspaces": [
    "libs/*",  // All packages in `libs` should be versioned independenly
    "main/*",  // All packages in `main` should have one fixed/locked versioning
  ]

We would ideally have an independent versioning (like all packages can get a custom version) for all packages inside the libs/ workspace and a fixed version across packages (like in LErna’s fixed/locked mode) under main/.
Is this possible with changesets?
Cheers 😊

It's possible - you could use linked config option for all packages in the main directory. Note that this works slightly in a different way than Lerna's fixed/locked mode but I think that it should suffice for the majority of use cases.
There have been some requests for the "true" fixed/locked mode so we could consider adding support for this somehow but at the moment we don't have manpower to design such a feature.

Hi @Andarist
So grateful for your feedback on this. I already saw the linked config option, but the example made me puzzle as how to always force each package inside the linked array to update to one and the same version.

There have been some requests for the "true" fixed/locked mode so we could consider adding support for this somehow but at the moment we don't have manpower to design such a feature.

Sure – we could look into helping out.
How would you describe the difference between Lerna’s fixed/locked mode and Changesets fixed mode?
Thanks again and have a good one.

How would you describe the difference between Lerna’s fixed/locked mode and Changesets fixed mode?

If by "Changesets fixed mode" you are referring to our linked mode then:

if you have a set of linked packages and you publish multiple packages from that set then all of those releases will have the very same version
but if you only release some subset of packages from that set of linked packages then those packages that don't have to be released won't be released. Whereas with Lerna's fixed mode all packages are always released together, even if some of them have not been touched

The fixed mode is being implemented in https://github.com/changesets/changesets/pull/690 . Please check out if it matches your expectations


Support translating Java methods with method-mapped signatures [moved]

This is Issue 185 moved from a Google Code project.
Added on 2013-06-03T14:53:23.000Z by<EMAIL_ADDRESS>Please review that bug for more context and additional comments, but update this bug.
Original labels: Type-Enhancement, Priority-Medium

Should be fixed using ObjectiveCName annotation.


Migrate changes from ZEP1 to main

This cherry-picks the first (smaller) changes from the ZEP1 review PR to the main branch. The commits included are
3a29e362a30f68dd7d6738380a9d35980911d6a5 dbddeb47d38759692161f34c9b6b1b3bc7764c0d 978cbd2283f56130ffcbb798937c5e86470430bc b63e64dcf530f195b19691e06a44c85916f33291.

cc @joshmoore @MSanKeys963

@jstriebel, the thinking is that batches of changes from that work will be periodically applied (and now merged) to the mainline?

Exactly, I think the rendered docs are great to get an impression of the current state of the spec, so it helps that they reflect the changes. I just cherry-picked those changes anyway to test https://github.com/alimanfoo/zarr-specs/pull/1, and thought this PR might be helpful.


Test setup for a TRAPI

This may be partially obsolete now - merging to keep the notebooks and update in future.


Can not fund wallet with ok coin

Aloha trying to buy a nft and need to fund my wallet.
clicked buy with Ok Coin ( already have an account with them)  the bridge opens and then redirects after showing buy page with an error saying not allowed to provide service. I am a US customer currently sitting in Portugal.  Ok Coin says it is a problem with Hiro wallet not them . help?

My understanding has been that Okcoin disallows purchases from any user who has registered their account in the US even when they're subsequently traveling overseas in another country.
However, are you saying that Okcoin support has indicated that you as a US-registered user should be able to purchase while overseas?
If so, we can contact their team for clarification.

This is precisely what I assumed however the customer rep said it was not an issue on their end.
I tried both via VPN and without. However when I am on their site - no VPN - I can buy crypto fine when logged into my account. IF they are blocking only the wallet fund then that is inconsistent.
Sent from ProtonMail for iOS
On Fri, Apr 15, 2022 at 9:40 AM, Mark Hendrickson @.***> wrote:

My understanding has been that Okcoin disallows purchases from any user who has registered their account in the US even when they're subsequently traveling overseas in another country.
However, are you saying that Okcoin support has indicated that you as a US-registered user should be able to purchase while overseas?
If so, we can contact their team for clarification.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

I think the distinction here is that you can indeed buy STX via Okcoin on their regular site, but this particularly purchase flow from the wallet into their "instant buy and deposit" flow is restricted to only non-US users (i.e. users who have KYC'ed with identities from other countries and are accessing the flow from other countries as well).
This would explain the differences you see as well, though it's not technically a wallet problem but rather a question of how the availability supported for this particular purchase flow at this time.

Closing as exchange specific


add documentation about instruction duration

Summary
The deprecations introduced in https://github.com/Qiskit/qiskit/pull/13247 did not notify via docs.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.9%) to 88.003%

Files with Coverage Reduction
New Missed Lines
%

crates/accelerate/src/two_qubit_decompose.rs
1
92.13%

crates/qasm2/src/lex.rs
2
92.48%

Totals

Change from base Build<PHONE_NUMBER>1:
-0.9%

Covered Lines:
34968

Relevant Lines:
39735

💛  - Coveralls


Extend the CLI to also call API methods

This PR:

Extends the CLI to also call API methods
Adds the API feature to StratisD

Lovely, thank you.


Fix update citation

During the last massive upgrade (to get a connection to JabRef Desktop running), the plugin does not select the current referencere any more.

This is also an issue in the main branch right now. It works if you click the small arrows left to the reference.


Updating PostgreSQL version 10 to version 14 in dockerized Zulip instance

Upgrade from Zulip 5 to Zulip 6 requires updating PostgreSQL version from 10 to 14.
Upgrade is supposed to be done by following these instructions. However, they are only possible, if Zulip installation is NOT dockerized. When installing Zulip with Docker, postgresql database runs in a separate container. Therefore, pg_upgradecluster command in Zulip container does not make sense and will fail, because the old database is not running locally.
So how are we supposed to upgrade the Zulip database in this case?

In order to make this possible, I've developed a database upgrade utility container based on tianon/docker-postgres-upgrade. It installs PostgreSQL versions 10 and 14 simultaneously and performs an update via pg_upgrade. Additionally, the required pgroonga, hunspell-en-us, and Zulip stop words are installed.
Please find the utility and usage instruction here: https://github.com/dmikushin/zulip-postgres-upgrade
Unfortunately, I'm not an expert in this topic, maybe there is a simpler way to upgrade?

Our instructions for upgrading PostgreSQL with docker-zulip are at
https://github.com/zulip/docker-zulip/blob/main/UPGRADING.md#upgrading-zulipzulip-postgresql-to-14-version-60-0-and-above
(linked from the docker-zulip readme). Let us know if this doesn’t work for you.

I totally missed that, thank you!


Added paging to main list

Adding paging and showing excerpts on the posts list.
Screenshots
Dark mode

Light mode

Super cool!

@Ninuce did help with her HTML and CSS skills ;)


build error with xCaddy.

I have an existing xCaddy build which used to work but now it fails:
# syntax=docker/dockerfile:1
FROM caddy:builder-alpine AS builder

ADD --keep-git-dir=false  https://github.com/luludotdev/caddy-requestid.git /requestid

RUN xcaddy build \
    --with github.com/lolPants/caddy-requestid=/requestid \
    --with github.com/caddyserver/transform-encoder

FROM caddy:alpine

COPY --from=builder /usr/bin/caddy /usr/bin/caddy

with the following error:
39.09 go: downloading google.golang.org/genproto v0.0.0-20230913181813-007df8e322eb
968.6 go: module google.golang.org/genproto/googleapis/api/expr/v1alpha1: Get "https://proxy.golang.org/google.golang.org/genproto/googleapis/api/expr/v1alpha1/@v/list": dial tcp <IP_ADDRESS>:443: connect: no route to host
968.6 go: module google.golang.org/genproto/googleapis/rpc/status: Get "https://proxy.golang.org/google.golang.org/genproto/googleapis/rpc/status/@v/list": dial tcp <IP_ADDRESS>:443: connect: no route to host
968.6 2024/02/28 21:26:23 [FATAL] exit status 1
------

how do I fix this error? I traced a comment to https://github.com/googleapis/go-genproto/issues/1015 but I am unsure how to resolve this dependency issue?

This has nothing to do with xcaddy. This is a network issue in your infrastructure. Basically, this says the go compiler doesn't know how to go to the listed IP addresses from your network.


Fix Telemetry

Fix:

[x] Add build date/time to distinguish between release and development version.
[x] Handle cancel world loading.

Not sure if we want to merge this into master (to make sure we detect development cases as soon as possible) or revision.

@omichel may I let you handle the new parameter on the server side ?

test me

Build Passed :ok_hand:
You can safely merge 'fix-telemetry' into 'master'.

This is done, the new parameters (operation=cancel and build=<unix_time_stamp>) should be taken into account in the database and displayed properly.

This is done, the new parameters (operation=cancel and build=<unix_time_stamp>) should be taken into account in the database and displayed properly.

Perfect, thank you!

test me

test me

Build Passed :ok_hand:
You can safely merge 'fix-telemetry' into 'master'.

Build Failed :confused:
https://cyberbotics2.cyberbotics.com:8443/job/Webots/400/consoleText

test me

test me

Build Failed :confused:
https://cyberbotics2.cyberbotics.com:8443/job/Webots/403/consoleText

test me

Build Passed :ok_hand:
You can safely merge 'fix-telemetry' into 'master'.

👍


[Sequoia 15.2] yabai --load-sa "could not spawn remote thread"

Issue
$ sudo yabai --load-sa
could not spawn remote thread: (os/kern) invalid argument
yabai: scripting-addition failed to inject payload into Dock.app!

Context
$ csrutil status
System Integrity Protection status: disabled.

$ nvram boot-args
boot-args       -arm64e_preview_abi

$ yabai -v
yabai-v7.1.5

Have: uninstalled, installed from HEAD, rebooted.
It was working prior to me rebooting my computer; have been using yabai for ~ 3 years. --load-sa errors have happened before but the usual tricks (cited in other existing issues) haven't fixed it this time round.
Happy to provide any additional context.

I resolved this by building from source.
I suspect the issue stemmed somehow by my use of arch -arm64 brew install ... as opposed to brew install ..., though without knowing more about architecture-specific differences for yabai I can't say much beyond that.
Seems unrelated to the MacOS update, so I'm closing this issue.


WIP: examples: notebooks: Create use-case example notebook for 'Transfer Learning'

The notebook consists of 2 approaches to transfer learning. Feature Extraction and Fine-Tuning with additional custom layers provided by the user.
The first approach just calls the model with trainable = False.
The second approach shows how to add more layers to pre-trained models with trainable = True.
Visualizing predictions.

Progress :

[x] Squashed unnecessary part.
[x] Fixed timeout during training.
[x] Fixed macos error by installing and enabling ipywidgets.
[x] Enhance code descriptions.
[x] Fix features.
[x] Remove the finetuning approach
[x] Add links to documentation for cached_download, source.dir, and model entry points

Codecov Report

Merging #1151 (96e46fc) into master (fec5c65) will decrease coverage by 0.07%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1151      +/-   ##
==========================================
- Coverage   84.67%   84.59%   -0.08%
==========================================
  Files         157      157
  Lines       10205    10201       -4
  Branches     1678     1678
==========================================
- Hits         8641     8630      -11
- Misses       1215     1221       +6
- Partials      349      350       +1

Impacted Files
Coverage Δ

tests/test_notebooks.py
100.00% <100.00%> (ø)

dffml/util/data.py
64.08% <0.00%> (-2.77%)
:arrow_down:

tests/util/test_net.py
98.36% <0.00%> (-1.64%)
:arrow_down:

dffml/base.py
84.38% <0.00%> (-0.34%)
:arrow_down:

dffml/df/memory.py
79.64% <0.00%> (-0.26%)
:arrow_down:

dffml/util/testing/consoletest/commands.py
75.16% <0.00%> (-0.06%)
:arrow_down:

dffml/source/df.py
76.54% <0.00%> (ø)

dffml/noasync.py
43.63% <0.00%> (+0.53%)
: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 fec5c65...96e46fc. Read the comment docs.

@pdxjohnny @sakshamarora1 can we please get this one reviewed first as I haven't had any feedback on this one yet.

Instead of doing Model.load to use the entrypoint-loading, I think it would make more sense to import it from dffml_model_pytorch

@pdxjohnny wanted dynamic loading to be used in these tutorials.

What does the Accuracy2 signify here?

Made quite a lot of changes, doesn't make sense afterwards. Thanks for the catch.

I think we should show the train output here, to make it more insightful.

I don't get any outputs on train at all, other than warnings about my CUDA and my local paths(say "hashim/dffml/" etc) so I cleared the train outputs.
Wasn't sure if this is dffml and jupyter interaction or if outputs are to be expected at all.

Also, the feature extraction example doesn't have the end layers replaced to accommodate the rockpaperscissors use-case.

Yeah, I wasn't sure whats the default on it and interaction between pytorch and dffml, so I asked you about that one on gitter.

Similarly, in the fine-tuning example, the out_features should be 3 and not 17.

Thanks for the catch, I was using the flower dataset so forgot to adjust that one shifting to rps. The other features are wrong as well.

Both the cases are almost the same with just the trainable bool parameter being true and false in them. So maybe we should
show either one, just like here - https://intel.github.io/dffml/master/examples/flower17/flower17_pytorch.html

That does make sense @pdxjohnny final thoughts on this?
Thanks @sakshamarora1 for the amazing review! I'll update these changes asap.

Can we move ipywidgets to the dev extras_require?

I don't get any outputs on train at all, other than warnings about my CUDA and my local paths(say "hashim/dffml/" etc) so I cleared the train outputs.
Wasn't sure if this is dffml and jupyter interaction or if outputs are to be expected at all.

Importing the logging module and setting it to INFO will work as far as I think. Try playing with that.

[x] move ipywidgets to dev extra_require
[x] ignore warnings in jupyter notebook
[x] set logging to INFO and stream to stdout to avoid the red background.
[x] Hide the logging cell from docs
[x] Remove the personal user handle from paths in JSON so it doesn't show on docs

@pdxjohnny @sakshamarora1 the logs seem good now.


Teacher tool iframe

I wanted to get out a foundation for the iframe since our work is growing and it will be useful to have the iframe for the  teacher tool.
There are a couple of things that I would like to expand upon for iframe to be more generic and usable across any situation in the future:

Move it to react-common so it's in one place and easily accessible
Make the props more extensible to allow an app's react-redux store to be hooked into the iframe. If we do this, we can refactor skillmap's iframe.
Make it so the iframe can respond to a wider range of event listeners.
Investigate more on different architectures. I think having the component the way it is right now is fine, but I've also considered having a redux store for the iframe itself because then it can be extended on via actions and dispatch functions, but I'm uncertain if that is the best way forward.

I'm gonna wait to merge this. I still need to change the file name for the frame, and let this rebuild, but I need to log off.


[Feature request] Take cursor acceleration into account when revealing hidden dock

Suggestion: When the dock is in autohide mode, do not unhide the dock unless the mouse reaches a certain threshold of downwards velocity when it is moving to the dock's trigger area.
Benefit: This will prevent users from accidentally revealing the dock and obscuring the bottom of windows they may be trying to work with during regular use.
Not sure if this is feasible/possible to implement in this project, but just a suggestion because it works really well in  GNOME's dock to shell extension's implementation of it.

No idea on how to measure the pointer speed here, as I just use the enter-notify-event. I could only think about adding some delay, and not showing the window if the pointer leaves the hotspot area before the delay ends.

After some further consideration (on boring shopping with my wife), I think I could, to some degree, measure the speed the hotspot was entered with. I need to install GNOME and see what you're talking about, in order not to implement the feature I haven't even seen.

My roommate has GNOME on his setup, I’ll ask him if I can screen record the dock behavior if you’re interested. A slight delay would also work well!

Here’s a video of the behavior, sorry about the poor quality, OBS on GNOME wasn’t cooperating with Pipewire and I didn’t want to spend too much time troubleshooting a DE I don’t use: https://youtube.com/shorts/vUA6-XJ8OfY?feature=share

I imagined it something like this. Will try to reproduce this behaviour, but I'm not sure if what I'm thinking about is going to work well. It'll turn out soon.

So, it could be implemented this way: https://youtu.be/DAP22C-aGF0
What do you think?

That is exactly what I had in mind! 👍

It needs some more work, but should be ready soon.

That’s awesome man, thank you so much for the great work!

I'm not 100% happy about it, but it works. You may check the add30 branch.

I'm having trouble building it, but that's probably an issue with me not the project, I very rarely build things from source straight from github so I'm probably doing something wrong. When I run make build after running make get I get the following errors: ./tools.go:69:12: cannot use window.Change (variable of type sway.WindowEventChange) as type string in struct literal
./tools.go:87:33: cannot use eventHandler (variable of type swayEventHandler) as type sway.EventHandler in argument to sway.Subscribe:
swayEventHandler does not implement sway.EventHandler (missing BarStateUpdate method)
./tools.go:117:33: cannot use eventHandler (variable of type swayEventHandler) as type sway.EventHandler in argument to sway.Subscribe:
swayEventHandler does not implement sway.EventHandler (missing BarStateUpdate method)
make: *** [Makefile:11: build] Error 2
Seems like some kind of mismatch with my sway version, maybe?

No idea. Are you on Arch Linux? Do you use nwg-shell or just the dock? One way or another, it must wait till tomorrow, as it's almost 4 am in my place.

I'm on Arch and have the whole nwg-shell installed, but not using nwg-panel. No worries man!

Awesome! THank you so much for the quick work
On Mon, Nov 14, 2022 at 6:44 PM Piotr Miller @.***>
wrote:

Closed #30 https://github.com/nwg-piotr/nwg-dock/issues/30 as completed
via #31 https://github.com/nwg-piotr/nwg-dock/pull/31.
—
Reply to this email directly, view it on GitHub
https://github.com/nwg-piotr/nwg-dock/issues/30#event-7809355564, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ANTWYSPF5644PLHTJF5CBHDWILMIXANCNFSM6AAAAAAR6G3QDM
.
You are receiving this because you authored the thread.Message ID:
@.***>

You're welcome. Thanks in advance for further feedback. :)

I just wanted to drop back and say that it works great! Thank you again man! This really improved the usability of the dock for me

I'm glad you like it. I'm just adding a corresponding to nwg-shell-config:


cannot create directory '/var/lib/mysql/mysql': Permission denied

Hi
I'm trying to run a galera-cluster on Kubernetes. It works pretty ok, by the way ;)
When using official mariadb image, container successfully initializes /var/lib/mysql directory mounted from host machine with the following (default) permissions:
[root@localhost docker-multinode]# ll /data/
total 0
drwxr-xr-x. 2 root root 40 Jan 18 11:10 mysql

However, when using galera cluster setup from this image, it fails to initialize mysql_data dir due to permission errors, even though volume setup/host permissions didn't change.
mysqld.sh: Initializing MariaDb data directory...
mkdir: cannot create directory '/var/lib/mysql/mysql': Permission denied
Fatal error Can't create database directory '/var/lib/mysql/mysql'

The latest information about mysql_install_db is available at
https://mariadb.com/kb/en/installing-system-tables-mysql_install_db
mysqld.sh: Failed to initialized data directory. Will hope for the best...
mysqld.sh: ----------------------------------
mysqld.sh: Starting with options: --console --init-file=/tmp/bootstrap.sql --wsrep-on=ON --wsrep-new-cluster --wsrep_cluster_name=dbcluster --wsrep_cluster_address=gcomm:// --wsrep_node_address=10.
1.7.2:4567 --wsrep_sst_auth=xtrabackup:123 --default-time-zone=+00:00
2017-01-18 11:10:05<PHONE_NUMBER>45856 [Note] mysqld (mysqld 10.1.19-MariaDB-1~jessie) starting as process 22 ...
2017-01-18 11:10:05<PHONE_NUMBER>45856 [Warning] Can't create test file /var/lib/mysql/galera-seed.lower-test
2017-01-18 11:10:05<PHONE_NUMBER>45856 [Note] WSREP: Read nil XID from storage engines, skipping position init
2017-01-18 11:10:05<PHONE_NUMBER>45856 [Note] WSREP: wsrep_load(): loading provider library '/usr/lib/galera/libgalera_smm.so'
2017-01-18 11:10:05<PHONE_NUMBER>45856 [Note] WSREP: wsrep_load(): Galera 25.3.18(r3632) by Codership Oy<EMAIL_ADDRESS>loaded successfully.
2017-01-18 11:10:05<PHONE_NUMBER>45856 [Note] WSREP: CRC-32C: using hardware acceleration.
2017-01-18 11:10:05<PHONE_NUMBER>45856 [Warning] WSREP: Could not open state file for reading: '/var/lib/mysql//grastate.dat'
2017-01-18 11:10:05<PHONE_NUMBER>45856 [ERROR] WSREP: Could not open state file for writing: '/var/lib/mysql//grastate.dat'. Check permissions and/or disk space.: 13 (Permission denied)
         at galera/src/saved_state.cpp:SavedState():49
2017-01-18 11:10:05<PHONE_NUMBER>45856 [ERROR] WSREP: wsrep::init() failed: 7, must shutdown
2017-01-18 11:10:05<PHONE_NUMBER>45856 [ERROR] Aborting

Could you please give me a hint on what could I do in this case?
This happens here https://github.com/colinmollenhour/mariadb-galera-swarm/blob/master/mysqld.sh#L13

Setting chmod -R 777 /var/lib/mysql on host does solve the problem.
Going back to chmod -R 775 /var/lib/mysql on host bring up the issue again.

Hi Maxim, unfortunately I have no experience with Kubernetes so I don't know if the problem is specific to Kubernetes or not but I have not had that problem on Docker Swarm or Kontena although I have been using named volumes so that might be the difference. The mysqld.sh script is run as "mysql" user using gosu, so try adding a chown to start.sh while the user is still root. I think chown is appropriate rather than chmod 777 for security reasons.

Thanks for the answer! Will try and get back to you

The question is not specific to Kubernetes, since Docker is still backing it up.

I added a "chown" in the start.sh script to make sure the directory is owned by mysql. I don't know if this will fix your issue or not but I don't know what else would. Thanks!


Update Personal Health Journal form for accessibility

🛠 Fixes Issue : "#23 - Accessibility Issues on Form"
👨💻 Changes proposed:
Fix accessibility issues on the Personal Health Journal form, identified by WAVE, specifically:

Orphaned Form Labels (form labels are present, but aren't correctly associated with corresponding form controls)
Missing Form Labels (form controls do not have corresponding labels)

To fix this, the <label> elements need to be associated with the <input> elements. I changed the values for each of the  <label>'s for attributes to match the corresponding <input>'s id attributes.
✔ Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[X] This PR does not contain plagiarized content.
[X] The title of my pull request is a short description of the requested changes.
[X] I am a participant of Hacktoberfest 2023.

📷 Screenshots
The form itself no longer has any WAVE errors.

Closes #23

Please upgrade your repository and resolve the conflicts.


after update powertoys run input area background color not correct

Microsoft PowerToys version
0.81.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce

after i changed windows theme one times, it will be ok whatever change theme to light or dark

✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

/dup #30206


Pool small messages inside the runtime

The runtime should pool small messages under a tunable size threshold to minimize memory allocation costs. This is especially important for applications (like Spectre) that use parameter marshalling, since the message object is hidden inside the runtime and can't be pooled by the application. This could be as simple as keeping a free list of fixed size messages on each PE. AMPI already does this with its AmpiMsgPool. Charm in fact already does this for some messages allocated via CkAllocSysMsg(), but that's only effective for a very limited set of messages.
Related to issue #34.

There is an important twist: allocation on one PE that is guaranteed to be freed by the same PE can be handle as above. The other case, where allocation and free happens on different processors can create a problem IF there is imbalance in where messages (say for tasks) are created and where (i.e. on which PE of the same logical node) they are freed. To handle the latter requires some more sophisticated implementation Within that, I'd distinguish 2 cases: If we know the imbalance will be strong and recurrent vs mostly balanced but may get out of whack over time. One strategy that will work for both: use address ranges or a bit field in the "header" to identify the allocating PE. On Free: if it is local, return to the pool, if not, queue it up (i.e. just add a pointer to it) in a simple fixed size message, of say 16 pointers. When it is full, send it to the source PE, which can add it to its pool. We used to have such an allocator a loooong time ago. (returning blocks of memory this way is probably also good for TLB hits, ,instead of  letting blocks float from PE to PE via pools).


Use translation methods instead of direct property

Fix #1646

@marktopper Now there is another error... Call to undefined method TCG\Voyager\Models\Role::translatable()

Thanks, will look into it now.

Have made a PR for it #1721


Help setting up snapshots

Versions

jest-glamor-react version: 1.4.0
enzyme version: 2.9.1
enzyme-to-json version: 1.5.1
glamor versions:

3.0.0-3 via gatsby-plugin-glamor
2.20.25 via @storybook/react and jest-glamor-react

node version: 8.1.4
yarn version: 0.27.5

Configuration
// package.json
"devDependencies": {
    "babel-eslint": "^7.2.3",
    "babel-jest": "^20.0.3",
    "babel-preset-es2015": "^6.24.1",
    "babel-preset-react": "^6.24.1",
    "enzyme": "^2.9.1",
    "enzyme-to-json": "^1.5.1",
    "glamor": "^3.0.0-3",
    "jest": "^20.0.4",
    "jest-environment-node-debug": "^2.0.0",
    "jest-glamor-react": "^1.4.0",
    "react-addons-test-utils": "^15.6.0",
    "react-test-renderer": "^15.6.1",
  },
"jest": {
    "setupTestFrameworkScriptFile": "<rootDir>/testUtils/customMatchers.js",
    "testEnvironment": "jsdom",
    "snapshotSerializers": [
      "enzyme-to-json/serializer"
    ],
    "transform": {
      "^.+\\.js$": "babel-jest",
      "\\.(jpg|jpeg|png|gif|eot|otf|webp|svg|ttf|woff|woff2|mp4|webm|wav|mp3|m4a|aac|oga)$": "<rootDir>/testUtils/fileTransformer.js"
    }
  }

// <rootDir>/testUtils/customMatchers.js
import { matcher, serializer } from 'jest-glamor-react'
expect.addSnapshotSerializer(serializer)
expect.extend(matcher)

Setup
For this component:
// src/components/ProfilePicture/index.js
import React from 'react'
import { css } from 'glamor'
import picturePath from '../../assets/profile.jpg'

const ProfilePicture = ({ path = picturePath, alt = 'Profile Picture', side = 140 }) => {
  const pictureSide = `${side}px`
  const pictureStyle = css({
    width: pictureSide,
    height: pictureSide,
    borderRadius: '50%',
    borderWidth: side < 100 ? '2px' : '4px',
    borderColor: '#ffffff',
    borderStyle: 'solid'
  })

  return (
    <img {...pictureStyle} src={path} alt={alt} />
  )
}

export default ProfilePicture

With this test:
// src/components/ProfilePicture/__tests__/ProfilePicture.test.js
import React from 'react'
import * as enzyme from 'enzyme'
import toJson from 'enzyme-to-json'
import ProfilePicture from '../../ProfilePicture'

describe('<ProfilePicture />', () => {
  let wrapper

  describe('with default props', () => {
    beforeEach(() => {
      wrapper = shallow(
        <ProfilePicture />
      )
    })

    test('renders the component', () => {
      expect(toJson(wrapper)).toMatchSnapshotWithGlamor(`enzyme.shallow`)
    })
  })
})

The generated snapshot is:
// src/components/ProfilePicture/__tests__/__snapshots__/ProfilePicture.test.js.snap
// Jest Snapshot v1, https://goo.gl/fbAQLP

exports[`enzyme.shallow 1`] = `
<img
  alt="Profile Picture"
  data-css-iqk43n=""
  src="profile.jpg"
/>
`;

Problem
The generated snapshot does not contain any CSS class. I was expecting to have css-iqk43n with:
{
    width: 140px;
    height: 140px;
    border-radius: 50%;
    border-width: 4px;
    border-color: #ffffff:
    border-style: solid;
}

Suggested solution
I don't really know, since I followed the README. The only difference is that I use glamor's {...pictureStyle} syntax and the README has className, but I tried that and it didn't work.
Thanks in advance.

That's quite odd. It should work with {...pictureStyle}, (we do have tests for it). And it definitely work for me with the className. I don't have time to dig into this issue, so if you could dig a little deeper to figure out what's up that'd be great. Thank you!

I've setup a repository to help investigating this issue: https://github.com/ricardocosta89/jest-glamor-issue-14
After a debug session, I've come to find out that in jest-glamor-react/dist/serializer:

print is being called with val:

{
  "type": "img",
  "props": {
    "data-css-iqk43n": "",
    "src": "../../assets/profile.jpg",
    "alt": "Profile Picture"
  },
  "children": null
}

getSelectors returns ["[data-css-iqk43n]"]

getStyles returns "", as this is the result of:

var styles = styleSheet.tags.map(function (tag) {
    return (/* istanbul ignore next */tag.textContent || ''
  );
}).join('\n');

Therefore, no styles are available at this point inprint:
if (styles) {
  return `${styles}\n\n${printedVal}`;
} else {
  return printedVal;
}

Investigating the styles generation...
styleSheet:
{
  "isSpeedy": false,
  "tags": [{}],
  "maxLength": 65000,
  "ctr": 0,
  "injected": true,
  "plugins": {
    "fns": [null,null],
    "media": {"fns": []},
    "fontFace": {"fns": []},
    "keyframes": {
      "fns": [null]
    }
  },
  "inserted": {},
  "registered": {}
}

We can see that styleSheet.registered is empty.
Downgrading to<EMAIL_ADDRESS>produces a correct stylesheet:
{
  "isSpeedy": false,
  "tags": [{}],
  "maxLength": 65000,
  "ctr": 1,
  "injected": true,
  "plugins": {
    "fns": [null,null],
    "media": {"fns": []},
    "fontFace": {
      "fns": []}
    ,"keyframes": {
      "fns": [null]
    }
  },
  "inserted": {"iqk43n": true},
  "registered": {
    "iqk43n": {
      "id": "iqk43n",
      "style": {
        "label": [],
        "width": "140px",
        "height": "140px",
        "borderRadius": "50%",
        "borderWidth": "4px",
        "borderColor": "#ffffff",
        "borderStyle": "solid"
      },
      "label": "",
      "type": "css"
    }
  }
}

Conclusion: The issue is a breaking change from glamor's v2 to v3. Will continue to investigate to see if I can find out exactly what is causing this.

Thanks for the update and the continuous investigation!

I can't seem to reach any conclusion. The difference is that, by the time print is executed, styleSheet already has the registered object with the correct information in v2 but not in v3.
We might need to ask in Glamor repo about this change from v2 to v3, since I'm not figuring out where the "magic" of populating styleSheet is happening. 🤔

cc @threepointone, who should we talk with about problems like this? Could anyone help us know what's changed in Glamor 3?

Cc @donaldpipowitch

I talked with @otbe about this. He really knows the internals :) I just opened issues and hit the publish button.
It looks like the registered cache was previously just an object (?) which was added as a property directly to styleSheet (see here). In v3 the registered cache has a Map-like API (see here and here). It is not exported as a property to styleSheet, but together with inserted as caches (see here).

We can see that styleSheet.registered is empty.

So maybe we just need to look into caches.registered?
Maybe @otbe knows more. He's the best ❤

@donaldpipowitch
I took a look at what is available with require('glamor') using VSCode debug and unfortunately no caches is available. Which is very weird, as all of the other exports are available (e.g. simulate, simulations, isLikeRule, cssLabels, cssFor, ...). 🤔

@donaldpipowitch I think your conclusion is correct. Both caches are no longe attached to the exported StyleSheet instance. As of now they are exported via caches.
@ricardocosta89 thats strange. I did a quick test and it works as expected:

@otbe I think it's something related with using the debug console in VSCode or the dist version of jest-glamor-react.
If I add a dummy const s = require('glamor') in the test file and set a breakpoint, then I have:
JSON.stringify(s.caches)
"{"inserted":{"cache":{}},"registered":{"cache":{}}}"

If the breakpoint is set on jest-glamor-react/dist/serializer.js (line 10), then _require does not have caches.

@kentcdodds
It seems there's a complete API change here. I tried to see if I could extract the same kind of string given the Map-like API, but I couldn't get very far. 😞
Do you confirm that the desired string for each style (the result of tag.textContent) is <nodeSelector>{<style>} (e.g. .something{padding:20px;borderWidth:2px})? Or is there something more?
Also, @otbe, just a question: Will this label always be present?

The value of tag.textContent should look something like this:
.css-5qt57y,[data-css-5qt57y]{color:red;font-size:20px;}.css-1evjunj,[data-css-1evjunj]{color:blue;font-size:20px;}.css-s7n3hr,[data-css-s7n3hr]{color:red;font-size:25px;}.css-1xzrot3,[data-css-1xzrot3]{color:green;}.css-13vote6,[data-css-13vote6]{color:green;font-size:20px;}

@kentcdodds
I have the matcher and index tests passing with the following to get the styles using glamor v3 (Object.values() would be helpful here 😞 ):
const styleCache = caches.registered.cache

const styles = Object.keys(styleCache)
  .map(style => cssFor(glamorCss(styleCache[style].style)))
  .join('\n')

const ast = css.parse(styles)

However, the custom-sheet test does not use glamor, and therefore it still fails... any advice on how to tackle this situation?
On another point, and I'd like @otbe's help here, for jest-glamor-react tests, the caches are available and populated correctly in serializer.js. However, when I copy the dist folder to node_modules/jest-glamor-react in my sample project, caches are no longer populated when serializer.js is executed. Any idea why? 🤔

Quick note - glamor's default version is now v2 again, the v3 version leaked out to being default on npm by mistake

Hmm... the gatsby plugin I'm using has "glamor": "next",. I'll check what glamor version is resolved now.

Dammit next still points to v3. Will fix that too in a bit.

Thanks @threepointone 🙏

Wahoo!

Well, at least now we know a bit of the path when v3 is ready. 😄 And I have learned some things.


NEXT-8591 - Filterable product list (in administration)

Description
The product listing in the administration is missing filters. You can only sort it by various columns but there is no filter yet. This makes it hard to manage a shop with a large amount of different products.

Example
Should be self-explanatory.

Is already being solved by shopware. Is covered by "Filters & Search for Admin Grids"

Is already being solved by shopware. Is covered by "Filters & Search for Admin Grids"


Warning about registering a non entity service schema

Checklist:

[ X] I updated to the latest version available
[ X] I cleared the cache of my browser

Release with the issue:
v2024.9.3
Last working release (if known):
n/a
Browser and Operating System:
Safari, MacOS (doesn't matter)

Description of problem:
During the start of Homeassistant the log shows the following errors:
homeassistant  | 2024-10-03 09:20:41.187 WARNING (MainThread) [homeassistant.helpers.frame] Detected that custom integration 'irrigation_unlimited' registers an entity service with a non entity service schema which will stop working in HA Core 2025.9 at custom_components/irrigation_unlimited/service.py, line 73: platform.async_register_entity_service(, please create a bug report at https://github.com/rgc99/irrigation_unlimited/issues

and a sesond one
homeassistant  | 2024-10-03 20:03:12.730 WARNING (MainThread) [homeassistant.helpers.frame] Detected that custom integration 'irrigation_unlimited' registers an entity service with a non entity service schema which will stop working in HA Core 2025.9 at custom_components/irrigation_unlimited/service.py, line 79: platform.async_register_entity_service(, please create a bug report at https://github.com/rgc99/irrigation_unlimited/issues

This issued should be addressed, even if there is some time until 2025.9.

Expected: no warnings in the log file :-)
Javascript errors shown in the web inspector (if applicable):
None
Additional information:
HA is running on Docker on a Synology DS.

This is a duplicate of issue 185


Feature/ch145205/dialog

Shortcut: https://app.shortcut.com/cartoteam/story/145205/component-dialogs-modals

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 70.303%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1142

Relevant Lines:
1529

💛  - Coveralls

@builtbyedgar if you can't currently work in the project, ping us so we can see how to manage CR changes

Acceptance OK

@Josmorsot. You should take this one, when you can


Frontend - Home page responsiveness

Home page is not fully responsive. I want to work on it and make it responsive.

Hey @meet503186 thanks for raising this but we already have and issue raised for the same and so closing this one
Ref - https://github.com/HITK-TECH-Community/Community-Website/issues/721


NewMap should accept the incoming context

In order to encourage users not to drop the tags from the
incoming context, adjust APIs to accept the incoming context
for convience.
Fixes #110.

Depends on https://github.com/census-instrumentation/opencensus-go/pull/108.


Add ParagraphStyle for checked checkbox

Currently checkbox has a ParagraphStyle (6) but checked checkbox uses a diffrent style (7) which is not implemented.
This PR simply adds ParagraphStyle.CHECKBOX_CHECKED (set to 7) to allow parsing of checked checkboxes.
If needed i can provide sample .rm files with checkboxes for testing.

Thanks!


add dockerfile scanner

The README describes it. This remains a work-in-progress. Publishing for feedback.
It still misses some ENV interpolation. Working on fixing that.

There is lots left to do. I'm going to fix those yetus issues if I can, but more importantly, I'm going to get options for output formats.

This is getting there. The scanner itself works well, I managed to incorporate some of the actual dockerfile scanner libraries in, which makes our parsing of things like ENV unnecessary and therefore the whole thing less brittle.
It has 3 output formats: "list", "spdx", "spdx-json".
I am working on completing spdx and spdx-json. When those are done, this will be ready.

@eriknordmark this is ready for review. It does not yet include it in any processes, or have targets from the main Makefile. It just creates the tool in its own directory and has its own README and Makefile.
It can process one or several Dockerfile at once, and it outputs lists of dependencies in list, spdx, or spdx-json formats.

It keeps doing that, even though there is a comment in doc.go.


Remove unused files

analyticstories.conf will be the single source of truth for all story and search details

Removed analytic_stories.conf and use_case_library.conf, as these files are creating confusion
updated dist/ folders
updated git actions
updated bin/
removed/updated dashboards to use analyticstories.conf
minor yml updates
deleted folders that are not used

Dashboard:


Reduce permissions history bloat

I think I found part of why my plugin extension is getting slow: We're recording every permissions act ever made, and including it in the state provided with each UI update. This is unreasonable.
We need to either cut this history short, or store it in a separate portion of state that is not constantly included with the UI.
Related to https://github.com/MetaMask/metamask-extension/issues/7311

Some notes on this:

There are technically two logs in state, the history and the activity logs
The history log keeps track of which domains have had which permissions. It doesn’t have a limit, but it grows slowly, and I can’t imagine it’s causing our performance issues.
The activity log records, as you mentioned, everything, but it’s capped at 100 entries. That should not amount to many kilobytes of memory, but I haven’t benchmarked it.
With respect to logging, the other thing impacting performance could be the permissions logger middleware itself. It shouldn’t cause slowdowns on boot, however, as, IIRC, we saw on your laptop.
To see if logging makes a difference, we could just comment out the lines where the logger middleware is pushed to the permissions middleware stack in the permissions controller.

Not specific to snaps, tracked elsewhere.


What about docs

WTF should we do about docs

[ ] #7
[ ] #8
[ ] #9
[ ] Decide if we want to go with more complexity

I'm talking to myself - any one want to write some @adrianwarren

What we're gonna do is:

[ ] Create a docs folder
[ ] Create a new index.md document
[ ] Turn on Github pages
[ ] Decide if we want to go with more complexity

document.write('Fuck of prat')


Update boredcoco.md

Codecov Report
Base: 56.82% // Head: 56.82% // No change to project coverage :thumbsup:

Coverage data is based on head (dad6ec0) compared to base (2edc3b6).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #303   +/-   ##
=========================================
  Coverage     56.82%   56.82%
  Complexity     1137     1137
=========================================
  Files           187      187
  Lines          4350     4350
  Branches        508      508
=========================================
  Hits           2472     2472
  Misses         1700     1700
  Partials        178      178

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Exclude label from release notes generator

I'm pretty sure this changed silently, but this is the documented way to
do it.

@davidjray Better fix for the issue we saw yesterday. Would you please review?


feat(tasks): allow app engine host override

Allow providing appEngineHost as a param to the TasksQueueService constructor to create tasks on that queue as a httpRequest instead of appEngineHttpRequest. This allows overriding the consumer host URL.
Though we can entirely override the consumer URL, it's expected that this is still a GAE consuming requests via the tasks endpoint, so only the URL is changed. The Http request path and method is kept the same.
oidcServiceAccountEmail provides a way to authorize the incoming request, in which Google Cloud Tasks will pass an id token of the service account assosciated with that email: https://cloud.google.com/tasks/docs/reference/rpc/google.cloud.tasks.v2#oidctoken

Thanks - merging and tweaking directly in order to get this through quicker.


Fix unzipping files within sub directory

Problem
Unzipping a file with sub directories would fail as the fullPath string was a file path.
Solution
The last path component has been removed, so all intermediate directories can be created.
A sub directory with .gif has been added to the already existing bb8.zip along with a unit test.

Great improvement @gwjakewelton. Thanks a lot!

@gwjakewelton it seems that for some reason the test you added fails

@marmelroy Test fixed. Looks like I generated the test zip archive wrong. CI is now passing.

Awesome, thanks


Added Curve automated market maker

Grant Application
This application is (select one):

[x] Speculative (use this by default)
[ ] an RFP response

This application is (select one):

[ ] Public (fully)
[x] Public with private finances

Abstract
Curve AMM module will be a 9-week long project with the final aim to deliver the following:

Low slippage, high efficiency stablecoin exchange
High efficiency exchange tool for other homogeneous assets on Polkadot (e.g. wrapped assets)
Low risk fee income for liquidity providers.
Liquidity superfluidity with additional rewards from supplying liquidity to lending protocols such as Equilibrium and Acala.

Checklist

[x] The grants document has been read and understood.
[x] The Google Form will be completed accurately. Note that the Google Form requires the pull request URL.
[x] Abstract (above) is succinct and complete.
[x] The application is being included into the correct directory: either 'targeted' or 'speculative'.
[x] The application includes a project description.
[x] The application includes all names of team members.
[x] The application includes a description of the team's experience.
[x] The application includes all necessary links (e.g. GitHub and LinkedIn)
[x] The "Development Roadmap" section in the application has a timeline of development ("milestones").
[ ] The "Development Roadmap" section in the application has an estimate of funds required.
[x] The "Development Roadmap" section gives an indication of the team's long term plans.
[x] The "Development Roadmap" section includes documentation as a deliverable for at least one milestone.

Closing due to https://github.com/w3f/Open-Grants-Program/pull/225

Closing due to https://github.com/w3f/Open-Grants-Program/pull/225


A bug in audio related task

Describe the bug
In the audio task, when I click the audio waveform, the audio playing pointer does not move to the position of my mouse pointer, resulting in that every time I want to listen to a part in the middle, I always listen from the beginning.
To Reproduce

Go to 'labeling'
Click on waveform
See error that the audio playing pointer does not move to the position of my mouse pointer

Expected behavior
the audio playing pointer move to  the position of my mouse pointer.
Environment:

OS: Windows 10
Browser: chrome
Version: 0.7.0

Additional context
This problem does not exist in the ’interface preview‘ in the ‘Setup’, but it will appear when labeling.

@hlomzik

Hey, @RenJinan! It sounds strange, especially if it works in preview. Could you please provide your config? How long is your audio? And what the error do you see? Screenshots or browser console output would be very helpful.

Thank you very much for your reply! My configuration file is the same as the official sample, but the problem still exists. I made a GIF to show this problem and look forward to your help.

Hey, @RenJinan! It sounds strange, especially if it works in preview. Could you please provide your config? How long is your audio? And what the error do you see? Screenshots or browser console output would be very helpful.

@hlomzik have you any ideas?

No ideas sadly :(
@renjinan do you have errors in browser console?
I checked this config in LSF, it works well. I'll check in LS soon.

@RenJinan have you tried to use AudioPlus tag?

I hope it's fixed after 0.7.3 release.


fix: add space between sentences

This pull request restores an “eaten-by-JSX” space between sentences of the Proudly Open Source section.

Thank you 👍


FIX wait for CSV

the get method requires key with resource name.
That can't be provided therefore replaced with a simple
wait function for the moment.
NOTE: package/channel list still presents duplicates and
namespaces may also need error handling. Got some namespace errors.
Signed-off-by: acmenezes<EMAIL_ADDRESS>

/lgtm


Turn server showing me: HTTPS connection is disabled due to venerability.

Hello I ave installed everything collider stunt and turn .
But turn server showing me HTTPS connection is disabled due to variability issue.
I have installed Ubuntu 18.04 and turn server <IP_ADDRESS> is turn srever updata required.

do you have valid cert file?

Yes, I resolved the issue.
Actually, the issue with turn version. I have updated turn to the latest version and it works now.
Note i have used Ubuntu 18.04 so I have updated turn to the latest version


Audio Support for Dell XPS 17 9700

Audio does not seem to work out of the box for the Dell XPS 17 9700 NixOS (this also applies to most linux distros).
https://github.com/maaarghk/soundwire-dkms/commits/master seems to offer some sort of support for this, but I'm not entirely sure how to integrate this support into nixos.

Here is another resource describing how to do this in Ubuntu: https://stukev.com/fixing-the-sound-on-dells-xps-17-9700/

https://blog.fts.scot/2020/07/04/dell-xps-2020-how-to-get-audio-working-on-linux/ may also be relevant

Just ignore the fact that it is dkms, we just build all kernel modules upfront. Also ignore their makefile since it hard code paths to linux: https://github.com/maaarghk/soundwire-dkms/blob/master/Makefile#L10
You can build this kernel module similar to https://github.com/NixOS/nixpkgs/blob/master/pkgs/os-specific/linux/mbp-modules/mbp2018-bridge-drv/default.nix and thttps://github.com/NixOS/nixpkgs/blob/master/pkgs/os-specific/linux/mbp-modules/mbp2018-bridge-drv/default.nix
And make it available in linuxPackages.

Thanks @Mic92 but I'm still a little lost at what exactly I'm supposed to be doing here. The two examples you posted seems quite different to me.
I'm not super familiar with how how linux kernel modules work to be honest.
I guess what is confusing to me is that it is not totally clear what the build outputs of this soundwire-dkms package should be, or how exactly to produce them, especially without using the provided makefile. Should I be recursing in to the individual directories and running the makefiles there?
arch linux also seems to have a PKGBUILD for this exact package, and all they seem to do is:
 install -dm755 "$pkgdir/usr/src"
 cp -r "$srcdir/$_pkgname" "$pkgdir/usr/src/soundwire-$pkgver"

can this be used as a template?

Archlinux uses dkms to build linux kernel modules. In NixOS we don't need that, we build kernel modules not on the target machine but in our CI, which is faster and less brittle. If you run make -C ${kernel.dev}/lib/modules/${kernel.modDirVersion}/build  \       INSTALL_MOD_PATH=$out M=$(pwd) modules_install like in the macbook driver example it will automatically create the right installation directory.

FWIW the following patch should fix audio on the XPS9700 on all Kernels 5.11 and above
(tested on 5.13.9)
and in this case there's no need to build the module linked above.
Of course this doesn't help if you want to remain on the LTS kernel. (5.10.x)

@workflow Amazing! Where did you discover that those kernel flags were what is necessary to get things working for the  rt1308?

@IvanMalison It was well hidden in this discussion :)

@workflow Ah nice! Thanks so much for finding this. Might be worth contributing something to nixos-hardware with this tweak: https://github.com/NixOS/nixos-hardware/pull/291
Unfortunate that this does require an entire rebuild of the kernel... I wonder if they will allow a default setting that will do that.

I think if we merge something into nixos-hardware we can maybe consider this closed?

Ah, neat! https://github.com/NixOS/nixos-hardware/pull/291 sounds great :+1:
The problem still remains on the current LTS kernel though (5.10.x), which does require the module you linked to. I tried building it quickly but failed (probably just missing some dependencies).
Not sure if it's good practice to force XPS 9700 users off the LTS kernel by default. :thinking:
@Mic92 We might need some guidance here.

The problem still remains on the current LTS kernel though (5.10.x), which does require the module you linked to. I tried building it quickly but failed (probably just missing some dependencies)

Ah, I actually do have it building. Let me see if I can dig it up.

Yep, here is my working build: https://github.com/IvanMalison/nixpkgs/tree/soundwire
IIRC, it might be a little sloppy right now but it does build. Thanks for all the advice in helping me figure out how to make that work @Mic92 . Looks like the reason it ultimately didn't work is that the additional kernel flags that @workflow found were ALSO needed.

https://github.com/NixOS/nixos-hardware/pull/291 is ready to go and fully tested. I consider this issue resolved.

There's still a huge problem with the microphone quality, I'm currently looking into that @IvanMalison

Hmm looked into it briefly but found no solution yet. Working around this by connecting a headset with a simple USB-C to Audiojack in, which has its own "sound card"
$> record --list-devices
card 2: A [USB-C to 3.5mm Headphone Jack A], device 0: USB Audio [USB Audio]
  Subdevices: 1/1
  Subdevice #0: subdevice #0

Hmm looked into it briefly but found no solution yet. Working around this by connecting a headset with a simple USB-C to Audiojack in, which has its own "sound card"

huh the built in microphone seems to work somewhat okay for me.

Lucky you :) some folks here report that it might depend on the exact hardware used.
I'm seeing the following in dmesg and internal mic input doesn't work, although sound output is fine with our patches.
[   14.959044] sof-audio-pci-intel-cnl 0000:00:1f.3: use msi interrupt mode
[   14.969847] sof-audio-pci-intel-cnl 0000:00:1f.3: hda codecs found, mask 4
[   14.973853] sof-audio-pci-intel-cnl 0000:00:1f.3: Firmware info: version 1:7:0-47d07
[   14.973855] sof-audio-pci-intel-cnl 0000:00:1f.3: Firmware: ABI 3:18:1 Kernel ABI 3:18:0
[   14.973858] sof-audio-pci-intel-cnl 0000:00:1f.3: unknown sof_ext_man header type 3 size 0x30
[   15.002138] ACPI BIOS Error (bug): Could not resolve symbol [\_SB.PCI0.PEG0.PEGP.GPS.PCGP], AE_NOT_FOUND (20210331/psargs-330)
[   15.002143] ACPI Error: Aborting method \_SB.PCI0.PEG0.PEGP.GPS due to previous error (AE_NOT_FOUND) (20210331/psparse-529)
[   15.002147] ACPI Error: Aborting method \_SB.PCI0.PEG0.PEGP._DSM due to previous error (AE_NOT_FOUND) (20210331/psparse-529)
[   15.002195] ACPI BIOS Error (bug): Could not resolve symbol [\_SB.PCI0.PEG0.PEGP.GPS.PCGP], AE_NOT_FOUND (20210331/psargs-330)
[   15.002197] ACPI Error: Aborting method \_SB.PCI0.PEG0.PEGP.GPS due to previous error (AE_NOT_FOUND) (20210331/psparse-529)
[   15.002199] ACPI Error: Aborting method \_SB.PCI0.PEG0.PEGP._DSM due to previous error (AE_NOT_FOUND) (20210331/psparse-529)
[   15.110935] sof-audio-pci-intel-cnl 0000:00:1f.3: Firmware info: version 1:7:0-47d07
[   15.110938] sof-audio-pci-intel-cnl 0000:00:1f.3: Firmware: ABI 3:18:1 Kernel ABI 3:18:0
[   15.196865] sof-audio-pci-intel-cnl 0000:00:1f.3: Topology: ABI 3:18:1 Kernel ABI 3:18:0
[   15.204761] ACPI BIOS Error (bug): Could not resolve symbol [\_SB.PCI0.PEG0.PEGP.GPS.PCGP], AE_NOT_FOUND (20210331/psargs-330)
[   15.204768] ACPI Error: Aborting method \_SB.PCI0.PEG0.PEGP.GPS due to previous error (AE_NOT_FOUND) (20210331/psparse-529)
[   15.204868] ACPI Error: Aborting method \_SB.PCI0.PEG0.PEGP._DSM due to previous error (AE_NOT_FOUND) (20210331/psparse-529)
[   15.213115] sof-audio-pci-intel-cnl 0000:00:1f.3: ASoC: Parent card not yet available, widget card binding deferred
[   15.616226] input: sof-soundwire Headset Jack as /devices/pci0000:00/0000:00:1f.3/sof_sdw/sound/card2/input36
[   15.616266] input: sof-soundwire HDMI/DP,pcm=5 as /devices/pci0000:00/0000:00:1f.3/sof_sdw/sound/card2/input37
[   15.616299] input: sof-soundwire HDMI/DP,pcm=6 as /devices/pci0000:00/0000:00:1f.3/sof_sdw/sound/card2/input38
[   15.616330] input: sof-soundwire HDMI/DP,pcm=7 as /devices/pci0000:00/0000:00:1f.3/sof_sdw/sound/card2/input39
[   15.894115] ucsi_acpi USBC000:00: unknown error 0

@workflow Just to clarify, are you saying that analogue microphone input does not work, or that the built in microphone does not work, or both?

@workflow curious what your lspci output looks like.

Analogue microphone input works with strong distortions that I wasn't able to fix so far.
Built-in microphone doesn't work at all.
 I have exactly the same issue (but what do you mean by analogue vs built-in microphone?). I asked some questions in https://discourse.nixos.org/t/dell-xps-13-9320-microphone-not-working/40932

@workflow did you finally solved your microphone issue? I have exactly the same issue (but what do you mean by analogue vs built-in microphone?). I asked some questions in https://discourse.nixos.org/t/dell-xps-13-9320-microphone-not-working/40932

@tobiasBora Hey, it has been working now for a while yes, though I can't remember exactly what fixed it!
I'm still using the following kernel patches: https://github.com/workflow/dotfiles/blob/fe458ebe54c659baadc9ee0a503edd4b971864a1/machines/flexbox/system.nix#L22
Here's what I meant:
Internal: Built-in Microphone (working now :heavy_check_mark: )
Analogue: External mic connected via AUX out (audio jack) (:grey_question: not sure since I switched to using a USB-C adapter, then it works)

Thanks! I actually fixed my issue using sudo alsactl init the first tine and enabling sound.enable = true; to persist it accross boot. Cf https://github.com/NixOS/nixpkgs/issues/294048 for webcam support.

Thanks! I actually fixed my issue using sudo alsactl init the first tine and enabling sound.enable = true; to persist it accross boot. Cf #294048 for webcam support.

Without my kernel overrides the internal mic is still not working on 6.1.76.
Running sudo alsactl init gives the following error:
Found hardware: "HDA-Intel" "Nvidia GPU 93 HDMI/DP" "HDA:10de0093,1028098f,00100100" "0x1028" "0x098f"
Hardware is initialized using a generic method
alsa-lib main.c:846:(execute_sequence) exec '/bin/rm -rf /var/lib/alsa/card1.conf.d' failed (exit code 1)

likely because it's already enabled,
and sound.enable seems to be true by default.

No, this error is due to the fact that /bin/rm is not existent in NixOs. I created a true issue there https://github.com/NixOS/nixpkgs/issues/294170 but in the meantime I used a dirty workaround I should have listed, which is to run:
$ sudo mkdir /bin
$ sudo ln -s /run/current-system/sw/bin/rm /bin
$ sudo ln -s /run/current-system/sw/bin/mkdir /bin

This is quite dirty, but does the job until the above issue is fixed.

@tobiasBora Both the options and having sound.enable = true are required on my machine, but no manual running of alsactl init.
On a separate note, apparently the option for SND_SOC_RT1308 was removed again between 6.1.76 and 6.1.82 (or I overlooked something in the first place), so I'm upstreaming it here:

Another note in the long list of strange audio adventures on this Laptop:
After switching to pipewire, I now have to enable the following Alsa channel capture:

...after every reboot for the internal microphone to work.

Even with sound.enable = true;?

Even with sound.enable = true;?

sound.enable = true isn't recommended anymore with a PipeWire setup.
Enabling it anyways, the only effect seems to be to add an alsa-store systemd service:
● alsa-store.service - Store Sound Card State
     Loaded: loaded (/etc/systemd/system/alsa-store.service; enabled; preset: enabled)
     Active: active (exited) since Wed 2024-05-29 13:47:38 WEST; 10s ago
    Process: 52557 ExecStart=/nix/store/204ak4443zipfr2yi6y5wal05bck1qqg-coreutils-9.3/bin/mkdir -p /var/lib/alsa (code=exited, status=0/SUCCESS)

...that creates the /var/lib/alsa directly, which at least leads to the ALSA settings to be persisted once changed, so slight improvement in the sense that I no longer have to change the ALSA state on reboot.
Not sure if that interferes with Pipewire though.

Note that kernels 6.6.37 seem to break audio again.
This seems closely related to https://github.com/NixOS/nixpkgs/issues/330685, but none of the presented patches seem to work on this machine.
Relevant logs on bad kernel versions:
Aug 20 11:18:41 flexbox kernel: sof_sdw sof_sdw: ASoC: topology: add_route failed: -22
Aug 20 11:18:41 flexbox kernel: sof_sdw sof_sdw: ASoC: topology: could not load header: -22
Aug 20 11:18:41 flexbox kernel: sof-audio-pci-intel-cnl 0000:00:1f.3: error: tplg component load failed -22
Aug 20 11:18:41 flexbox kernel: sof-audio-pci-intel-cnl 0000:00:1f.3: error: failed to load DSP topology -22
Aug 20 11:18:41 flexbox kernel: sof-audio-pci-intel-cnl 0000:00:1f.3: ASoC: error at snd_soc_component_probe on 0000:00:1f.3: -22
Aug 20 11:18:41 flexbox kernel: sof_sdw sof_sdw: ASoC: failed to instantiate card -22
Aug 20 11:18:41 flexbox kernel: sof_sdw sof_sdw: snd_soc_register_card failed -22
Aug 20 11:18:41 flexbox kernel: sof_sdw: probe of sof_sdw failed with error -22

This should be reported upstream.

Updating Firmware to the latest released by Dell seems to fix this on kernels 6.6.48+ 🎉

This was required to fix the internal mic: https://askubuntu.com/a/1376570/203326


Need additional axiom connecting to NCBITaxon

current:
http://purl.obolibrary.org/obo/CARO_0001010 = (Viruses or Viroids or cellular organisms)
The elements on the union are all NCBITaxon classes, which is fine
However, CARO also has cellular organism http://purl.obolibrary.org/obo/CARO_0010004, which is needs to be declared equivalent to NCBITaxon 'cellular organisms'

This seems to have been done


Updated PR to add octokit pagination to listForRef/listFiles Github API calls

This builds on previous PR #745. Fixed bug picked up by @tianhaoz95 where awaiting the responses for the posted status checks returned an array of undefined values.
This was because the format of the data was not as I expected given the docs. Just adjusted function to return response.data for each page rather than response.data.check_runs.
I ran this locally and tested against a repository with more than 30 checks. Prior to this minor change the app crashed, now it succeeds. See screenshot below.

This examples shows more than 30 posted checks working well. There is one check that is not detected properly -- that is a bug on our side where I had change the name of the check and missed the changed in the checkgroup.yaml, so nothing to worry about

@tianhaoz95 Would you be able to have a look at this one? I've tested locally now as well -- before I got the same crash you had but now I can verify that this is working correctly.

Yes, I've just verified that it works! See https://github.com/tianhaoz95/check-group-test/pull/18/checks?check_run_id=5323335533 where I created a ton of fake checks :)
I think the unit setup does not run in the forked repo which I will look into later. I've run the test locally and it works:

I will force this to merge and it will be auto deployed in about 5 minutes.
Thanks for the fix!

🎉 Thank you! Confirmed that is all working on our side now


Unable to build on MacOSX

Describe the bug, including details regarding any error messages, version, and platform.
Latest pyarrow is unable to build on Macosx Sonoma
exceptions.
      [ 13%] Built target lib_pyx
      [ 15%] Generating pyarrow/src/arrow/python/lib_api.h, pyarrow/src/arrow/python/lib.h
      [ 15%] Built target cython_api_headers
      [ 18%] Building CXX object CMakeFiles/arrow_python.dir/cmake_pch_arm64.hxx.pch
      [ 18%] Building CXX object CMakeFiles/arrow_python.dir/pyarrow/src/arrow/python/init.cc.o
      [ 26%] Building CXX object CMakeFiles/arrow_python.dir/pyarrow/src/arrow/python/benchmark.cc.o
      [ 26%] Building CXX object CMakeFiles/arrow_python.dir/pyarrow/src/arrow/python/arrow_to_pandas.cc.o
      [ 26%] Building CXX object CMakeFiles/arrow_python.dir/pyarrow/src/arrow/python/datetime.cc.o
      [ 26%] Building CXX object CMakeFiles/arrow_python.dir/pyarrow/src/arrow/python/common.cc.o
      [ 28%] Building CXX object CMakeFiles/arrow_python.dir/pyarrow/src/arrow/python/decimal.cc.o
      [ 30%] Building CXX object CMakeFiles/arrow_python.dir/pyarrow/src/arrow/python/deserialize.cc.o
      [ 32%] Building CXX object CMakeFiles/arrow_python.dir/pyarrow/src/arrow/python/extension_type.cc.o
      In file included from /private/var/folders/l6/sfgrxfn92t180ltdcfc6_4l40000gp/T/pip-req-build-l9f1ldv3/pyarrow/src/arrow/python/extension_type.cc:22:
      /private/var/folders/l6/sfgrxfn92t180ltdcfc6_4l40000gp/T/pip-req-build-l9f1ldv3/pyarrow/src/arrow/python/extension_type.h:36:58: error: non-virtual member function marked 'override' hides virtual member function
        std::string ToString(bool show_metadata = false) const override;
                                                               ^
      /opt/homebrew/include/arrow/extension_type.h:53:15: note: hidden overloaded virtual function 'arrow::ExtensionType::ToString' declared here: different number of parameters (0 vs 1)
        std::string ToString() const override;
                    ^
      1 error generated.
      make[2]: *** [CMakeFiles/arrow_python.dir/pyarrow/src/arrow/python/extension_type.cc.o] Error 1
      make[2]: *** Waiting for unfinished jobs....
      make[1]: *** [CMakeFiles/arrow_python.dir/all] Error 2
      make: *** [all] Error 2
      error: command '/opt/homebrew/bin/cmake' failed with exit code 2
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for pyarrow
Failed to build pyarrow
ERROR: Failed to build one or more wheels
[notice] A new release of pip is available: 23.3.2 -> 24.0
[notice] To update, run: pip install --upgrade pip

Component(s)
Python

Would you like to refer to this document?

https://arrow.apache.org/docs/developers/python.html#building-on-linux-and-macos

It's strange that your pip didn't use wheel.
We have wheels for macOS: https://pypi.org/project/pyarrow/#files

We don't use prebuilded wheel but build them ourselves since it is our company policy, because we don't know where the binaries where built. However the same process worked for older versions of pyarrow.
It says:
On macOS, any modern XCode (6.4 or higher; the current version is 13) or Xcode Command Line Tools (xcode-select --install) are sufficient.
gcc --version
Apple clang version 15.0.0 (clang-15<IP_ADDRESS>)
Target: x86_64-apple-darwin23.4.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin

If you want to build from source, you should remove Apache Arrow C++ installed by Homebrew and build Apache Arrow C++ by yourself. Or you should wait for https://github.com/Homebrew/homebrew-core/pull/169664 .


Add packit support

Let's add support for packit so we can run our test on each PR

Can you please clarify how packit can help with running our tests?

copying a comment from @henrywang from a separate thread:

I think containers org already onboarded Packit. Packit will be a good choice for podman-bootc test. Packit does not consume github action runner and it's well integrated with testing farm and tmt. Packit does not need testing farm API token either. In this podman-bootc case, secret is not required and the only requirement should be nested virtualization. Testing farm can be configured to request a nested virtualization runner for both x86_64 and aarch64. So we should use Packit.


Failure running tests for SGE on CircleCI

A rather strange failure has cropped up on CircleCI, which is not reproducible locally. This failure occurs when trying to run the test suite. In particular, the issue occurs when trying to start up SGE in the container. Unclear as to why it is happening. For now, documenting the failure so that we can revisit it in the future.
ref: https://circleci.com/gh/nanshe-org/docker_nanshe_workflow/242

Opened a WIP PR ( https://github.com/nanshe-org/docker_nanshe_workflow/pull/87 ) to try and fix this. It is not yet complete and may not have time to work on it right away. So merely adding it to keep track of this work at this point.

Should point out this issue arises after updating conda from 4.3 to 4.5. Another interesting (possibly, but unlikely related) issue is ( https://github.com/conda/conda/issues/7193 ).

Fixed by PR ( https://github.com/nanshe-org/docker_nanshe_workflow/pull/87 ).


Request for Data Storage Capability in Quests API

What does your feature do?
I am currently working on a custom objective for the Quests plugin, where players must perform an action in various random locations. To achieve this, I need to track which areas players have already interacted with to prevent them from performing actions in the same area multiple times.
The current API does not provide a way to store or manage custom data such as the locations where players have interacted with specific blocks.
I have two solution I can do as a user:
Create my own File controller to manager this, The down side of this would be constantly opening and closing the file todo the checks as I have no way too keep it in memory.
The second solution is too create a custom plugin that can store and save arbitrary data well keeping it in memory.
At this time I can not offer any code to solve this problem within the API it's self as I do not know it enough.
Here is some a example of what I want it for:
package org.Tree.treeCraftQuests;

import java.util.HashSet;
import java.util.Map;
import java.util.Set;
import java.util.UUID;

import me.pikamug.quests.BukkitQuestsPlugin;
import me.pikamug.quests.Quests;
import me.pikamug.quests.module.BukkitCustomObjective;
import me.pikamug.quests.player.Quester;
import me.pikamug.quests.quests.Quest;
import me.pikamug.quests.storage.BukkitStorageFactory;
import org.Tree.treeCraft.TreeCraft;
import org.bukkit.Bukkit;
import org.bukkit.Location;
import org.bukkit.block.Block;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.player.PlayerInteractEvent;

    public class Intro1ChestObj extends BukkitCustomObjective {

    private static final Quests quests = (Quests) Bukkit
            .getServer()
            .getPluginManager()
            .getPlugin("Quests");

    public Intro1ChestObj() {
        setAuthor("Tree");
        this.setName("Intro1ChestObj");
        setDisplay("Interact with a number of chests in a specific world");
        setItem("CHEST", (short) 0);
        setCountPrompt("Enter the number of chests to interact with:");
        this.addStringPrompt(
                "ChestLoc",
                "Enter the world (world):",
                "lobby"
        );
    }

    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent event) {
        Block block = event.getClickedBlock();
        Player player = event.getPlayer();

        // did we find the treeCraft plugin?
        if (treeCraft == null) {
            Bukkit.getLogger().warning("TreeCraft plugin not found");
            return;
        }

        if (block == null || !block.getType().name().contains("CHEST")) {
            return;
        }

        assert quests != null;
        UUID playerId = player.getUniqueId();
        Quester quester = quests.getQuester(playerId);

        for (Quest quest : quester.getCurrentQuests().keySet()) {
            Map<String, Object> map = getDataForPlayer(playerId, this, quest);

            if (map == null) {
                continue;
            }

            String chestLocStr = (String) map.get("ChestLoc");

            if (chestLocStr == null) {
                Bukkit.getLogger().warning("ChestLoc is null for player " + player.getName());
                continue;
            }

            Location chestLoc = new Location(Bukkit.getWorld(chestLocStr), block.getX(), block.getY(), block.getZ());

            // NO WAY TO STORE THE CHEST LOCATION IN THE APIS SO THE PLAY WILL BE ABLE TO INTERACT WITH THE SAME CHEST MULTIPLE TIMES

            // log the interaction and this.getCount
            Bukkit.getLogger().info("Player " + player.getName() + " interacted with a chest at " + chestLoc.toString());
            Bukkit.getLogger().info("Player " + player.getName() + " has progressed " + this.getCount() + " chests");

            // Increment the objective progress
            incrementObjective(playerId, this, quest, 1);

        }
    }
}

How would people use it?
These features would be useful for many objective-related tasks, such as ensuring a player does not eat the same item twice, visit the same location more than once, or interact with specific blocks or objects only a predetermined number of times.
Well what I'm trying to achieve can be done in multiple ways. I'm trying to get players to search chests around a location.
In the custom objective I could just list all the chest locations or some other things I have not thought off, this feature would allow ease of use.

Hi there! You're correct that constantly checking a file would be expensive, but you should be able to load information to a global variable such as a HashMap<UUID, Location> and only update the file when changes are made.

Yeah, I thought I tried that and I could not get it to work the first time around So I asked in the discord and had a conversation. Turn's out it was just a me issue and you can just do that.
package org.Tree.treeCraftQuests;

import java.util.HashMap;
import java.util.HashSet;
import java.util.Map;
import java.util.Set;
import java.util.UUID;

import me.pikamug.quests.BukkitQuestsPlugin;
import me.pikamug.quests.Quests;
import me.pikamug.quests.module.BukkitCustomObjective;
import me.pikamug.quests.player.Quester;
import me.pikamug.quests.quests.Quest;
import org.Tree.treeCraft.TreeCraft;
import org.bukkit.Bukkit;
import org.bukkit.Location;
import org.bukkit.block.Block;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.player.PlayerInteractEvent;

public class Intro1ChestObj extends BukkitCustomObjective {

    private static final Quests quests = (Quests) Bukkit
            .getServer()
            .getPluginManager()
            .getPlugin("Quests");

    // HashMap to store chest interactions: key is player UUID, value is a set of chest locations
    private final Map<UUID, Set<Location>> chestInteractions = new HashMap<>();

    public Intro1ChestObj() {
        setAuthor("Tree");
        this.setName("Intro1ChestObj");
        setDisplay("Interact with a number of chests in a specific world");
        setItem("CHEST", (short) 0);
        setCountPrompt("Enter the number of chests to interact with:");
        this.addStringPrompt(
                "ChestLoc",
                "Enter the world (world):",
                "lobby"
        );
    }

    @EventHandler
    public void onPlayerInteract(PlayerInteractEvent event) {
        Block block = event.getClickedBlock();
        Player player = event.getPlayer();

        if (block == null || !block.getType().name().contains("CHEST")) {
            return;
        }

        assert quests != null;
        UUID playerId = player.getUniqueId();
        Quester quester = quests.getQuester(playerId);

        for (Quest quest : quester.getCurrentQuests().keySet()) {
            Map<String, Object> map = getDataForPlayer(playerId, this, quest);

            if (map == null) {
                continue;
            }

            String chestLocStr = (String) map.get("ChestLoc");

            if (chestLocStr == null) {
                Bukkit.getLogger().warning("ChestLoc is null for player " + player.getName());
                continue;
            }

            Location chestLoc = new Location(Bukkit.getWorld(chestLocStr), block.getX(), block.getY(), block.getZ());

            // Check if the chest location has already been interacted with by the player
            Set<Location> playerInteraction = chestInteractions.computeIfAbsent(playerId, k -> new HashSet<>());
            if (playerInteraction.contains(chestLoc)) {
                Bukkit.getLogger().info("Player " + player.getName() + " has already interacted with this chest.");
                // log the contents of playerChestInteractions
                for (Location loc : chestInteractions.get(playerId)) {
                    Bukkit.getLogger().info("Player " + player.getName() + " has interacted with a chest at " + loc.toString());
                }
                continue;
            }

            // Store the chest location
            playerInteraction.add(chestLoc);

            // Log the interaction and progress
            Bukkit.getLogger().info("Player " + player.getName() + " interacted with a chest at " + chestLoc.toString());
            Bukkit.getLogger().info("Player " + player.getName() + " has progressed " + this.getCount() + " chests");

            // Increment the objective progress
            incrementObjective(playerId, this, quest, 1);
        }
    }
}


added variable assignment in new activity example

Coverage remained the same at 99.79% when pulling 7baa392b6cd023f7f340456f944c0effd7101b6f on client_new_activity_example into 63e5d9afc7fa980d544652ba81fca8697c16d8e4 on master.

Coverage remained the same at 99.79% when pulling 7baa392b6cd023f7f340456f944c0effd7101b6f on client_new_activity_example into 63e5d9afc7fa980d544652ba81fca8697c16d8e4 on master.


External stylesheets/scripts

It might be a useful piece of advice for people who are not there yet with complete encapsulation of components.

I'm glad you managed to make this work, it will be useful for users I'm sure! I only have one comment with the PR

Merged :tada:


New actions

          1 Error

:no_entry_sign:
ScreenKey.swift#L19 - Sources/Screen/ScreenKey.swift#L19: variable used within its own initial value
        let traits = traits.map { $0.base }

  Generated by :no_entry_sign: Danger

          2 Messages

:book:
iOS: Executed 1 test, with 0 failures (0 unexpected) in 0.009 (0.016) seconds

:book:
tvOS: Executed 1 test, with 0 failures (0 unexpected) in 0.006 (0.013) seconds

  Generated by :no_entry_sign: Danger