python3Packages.ha-philipsjs: 2.7.5 -> 2.7.6

Motivation for this change
Update to latest upstream release 2.7.6
Change log: https://github.com/danielperna84/ha-philipsjs/releases/tag/2.7.6
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 146098 run on x86_64-darwin 1

2 packages built:

python38Packages.ha-philipsjs
python39Packages.ha-philipsjs

Result of nixpkgs-review pr 146098 run on x86_64-linux 1

3 packages built:

home-assistant
python38Packages.ha-philipsjs
python39Packages.ha-philipsjs


Update index.css

Issue:

This repository is open for beginners but please don't send unnecessary PRs.


CAS token value is allowed to be negative

For CAS command, I noticed that the user can pass in a negative CAS token value, and memcached will accept it and casts the value into a positive integer via strtoull(). See https://github.com/memcached/memcached/blob/master/proto_text.c#L1950
cas key 0 0 1 -18446744073709551615
b
EXISTS

Is this behavior expected? I am not sure if it makes sense to allow the user to pass in a negative CAS token value. It seems like it is more appropriate to return CLIENT_ERROR bad command line format back to the client.

It also came to me as a surprise. Apparently safe_strtoul() calls strtoull() in C, which will cast a negative integer into a positive integer.
See https://github.com/memcached/memcached/blob/master/util.c#L54

Ah I'm thinking of another part of the code that does the negative check.
Not sure if this is even worth fixing; did you find this on audit or from a user bug report? If the user gives a bizarre CAS it just won't match, but if I change this behavior someone out there might start getting an error when they otherwise weren't.

I found this out when we were doing some internal testing on memcached during development. I don't think it was reported by one of our memcached users.

Fwiw I'm leaving this open for now as I'm a bit overbusy, but offhand I'm not sure if it's reasonable to fix it since it might make someone's buggy but working code, and make it fail.


chore(main): release 4.0.0

:robot: I have created a release beep boop
4.0.0 (2024-03-08)
⚠ BREAKING CHANGES

critical breaking change (#43)

Features

f42: set a better message (#42) (c355256)

Bug Fixes

critical breaking change (#43) (11da95d)
update the readme (#45) (71ea55f)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/jbdoumenjou/poc-changelog/releases/tag/v4.0.0 :sunflower:


feat: don't retry on critical errors (query, tx)

Resolves: #23
@race-of-sloths

🥁  Score it!
@akorchyn, please ask your peers to score the PR using @race-of-sloths score [1/2/3/5/8/13].
If no scoring is provided within 24 hours, this PR will be scored as 1 🦥


Add OKDO_UBLOX_ODIN_W2 platform

Description
Removed UBLOX from OKDO_UBLOX_ODIN_W2 platform

Pull request type

[ ] Fix
[ ] Refactor
[x] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

@MarceloSalazar

@mark-edgeworth please review and merge.

Hopefully, this is now resolved - please check and merged :)


Comparison with a DATE yields an incorrect result

Consider the following statements:
CREATE TABLE t0(c0 VARCHAR);
INSERT INTO t0(c0) VALUES (DATE '2000-01-02');
SELECT * FROM t0 WHERE DATE '2000-01-01' < t0.c0; -- expected: {2000-01-01}, actual: {}

Unexpectedly, the SELECT does not fetch a row. Interestingly, I found that when using the CLI, the first time the query is executed, the result is correct:
SQLite version DuckDB dc352a372
Enter ".help" for usage hints.
Connected to a transient in-memory database.
 It seems related to the filter in the scan:
explain_key	explain_value
VARCHAR	VARCHAR
[ Rows: 3]
logical_plan
PROJECTION[c0]
  FILTER[CAST[VARCHAR](CAST[INT32](2000-01-01))<c0]
    GET(t0)
logical_opt
PROJECTION[c0]
  GET(t0)
physical_plan
PROJECTION[c0]
  SEQ_SCAN[t0 c0>2000-01-01]

Query failed with message: Unoptimized result differs from original result!
Original Result:
c0
VARCHAR
[ Rows: 0]

Unoptimized Result
c0
VARCHAR
[ Rows: 1]
2000-01-02

This issue was related to a wrong check for the zonemaps in cases of > or < checks with pruned strings.
Also I guess the expected result was 2000-01-02 and not 2000-01-01? and a string and not a date?

Yes, you are right.


error when updating scenario_overlays

Hi @bbest,
I'm attempting to rerun scenario_overlays.Rmd with the new prioritir s4 solution (including Antarctica) in order to get updated % overlap of s4 with each IHO sea for manuscript.
When I get to the for loop on line 89:
for (layer in lyrs_clip$layer){ # layer = lyrs_clip$layer[1]
    lyr <- layers %>% filter(layer==!!layer)
    read_sf(lyr$raw_shp) %>%
      st_intersection(p_abnj) %>%
      mutate(
        area_km2 = st_area(geometry) %>%
          units::set_units(km^2)) %>%
      filter(
        st_geometry_type(geometry) != "GEOMETRYCOLLECTION") %>% # to remove points
      write_sf(lyr$hs_shp)
  }

I receive the following error:
although coordinates are longitude/latitude, st_intersection assumes that they are planar

Error in CPL_geos_op2(op, st_geometry(x), st_geometry(y)) :
  Evaluation error: TopologyException: Input geom 0 is invalid: Ring Self-intersection at or near point 3.5156249243664486 43.274444391192986 at 3.5156249243664486 43.274444391192986.

I think it may be having trouble with p_abnj on line 93 -- perhaps since this .Rmd file is now in the repo bbnj-scripts instead of bbnj? But I'm not sure how to resolve this. Help??

@bbest do we need to be concerned about using st_intersection on a sphere? see this github issue

sf is informed that these are long/lat Earth coordinates, and the arctic polygon is supposed to include the pole, but st_intersects says it doesn't, which is wrong. It's wrong because it assumes coordinates are planar, and you are warned about that assumption.

(reality is somewhat more complicated: every polygon on the sphere divides the sphere into two parts, depending on the ring direction; this would have to be taken into account)

I hope to extend sf with routines that do proper intersections on the sphere; R package s2 from Ege @rubak seems to be a good candidate for this.

Hi @mvisalli,
I think I was able successfully convert the GEOMETRYCOLLECTION of North Pacific in ihor.shp into MULTIPOLYGON by using st_collection_extract("POLYGON") then group_by(...) %>% summarize(). Need to jump on another call but wanted to share good news and will check in fixed outputs this afternoon.
Ben

Thanks @bbest I'll keep an eye out for updated results & will plug into manuscript when ready

Hi @mvisalli,
All set! Check out North Pacific Ocean now in the IHO seas revised (ihor):
https://ecoquants.com/bbnj-scripts/scenario_overlays.html#ihor


[New metric] Add EED

🚀 Feature
Add EED, a text metric used for NMT evaluation.
Sources:
Paper - EED: Extended Edit Distance Measure for Machine Translation
Repo
Motivation
Over the years a number of machine translation metrics have been developed in order to evaluate the accuracy and quality of machine-generated translations. Metrics such as BLEU and TER have been used for decades. However, with the rapid progress of machine translation systems, the need for better metrics is growing. This paper proposes an extension of the edit distance, which achieves better human correlation, whilst remaining fast, flexible and easy to understand. (Stanchev et al., 2019)

I'll handle this :]

@mathemusician could you comment here so we can assign this issue to you... :rabbit:

I got this.


WIP: Debugging added to DAO_PING.java

Coverage increased (+0.001%) to 22.872% when pulling 058d13415846956961a788480cf4c80a897673d4 on bjagg:JGroups-dao-ping into 47247bdb315d3715b5a9f9c840d25ac87af56332 on Jasig:master.


run an arbitrary SQL statement on the database not accurate

Hola Miguel,
Documentation indicated use db.text() however attribute does not exist.

Here is python modules in my enviroment.

Sorry about that. The correct way to do this is to importtext from sqlalchemy:
from sqlalchemy import text


List command fails when used with a pipeline

Do the following:
rbnotes list | head -1

Then, rbnotes fails.
Traceback says:
...lib/rbnotes/commands/list.rb:58:in `write': Broken pipe @ io_writev - <STDOUT> (Errno::EPIPE)

The symptom is caused when rbnotes writes many lines (over 300 lines in my environment) into the pipeline input and the next command discard the pipeline.  Rbnotes tries to write into the pipeline, but the pipeline was broken because of the next program exit.
The top level of rbnotes could catch the exception, then just ignore it.


swap out mongoose for native mongodb driver + simple-schema

https://www.npmjs.com/package/mongodb
https://www.npmjs.com/package/simpl-schema

This one is also worth considering: https://www.npmjs.com/package/apollo-connector-mongodb


How do I remove the description field?

Thanks for asking.
There is no way to remove description field now 🙇
But, I think better there's it. 👍
I'll implement a option you remove description.
Please wait.

Alternatively you could use CSS to hide the element:
.link-preview .og-description {
  display: none;
}


Embassyd is occupying excessive memory

Prerequisites

[X] I have searched for existing issues that already report this problem, without success.

EmbassyOS Version
0.3.0~3
Device
Laptop/Desktop
Device OS
Windows
Device OS Version
Windows 10
Browser
Brave
Browser Version
1.38.119
Current Behavior
Despite running only a few services, memory is filled by embassyd processes. It appears there may be a memory leak in EmbassyOS

Expected Behavior
Determine whether a memory leak is present
Steps to Reproduce
Running the following processes:
Bitcoin Core
Bitcoin Proxy
BTCPay Server
LND
LNDg (beta marketplace)
Embassy Pages
File Browser
Installed services but not running:
Mempool
Thunderhub
RTL
BAR
Syncthing
Synapse
Anything else?
No response

should be solved by #1505


[Bug Report]: Unexpected warning from export variables

System Info
System:
OS: Linux 5.13 Ubuntu 20.04.4 LTS (Focal Fossa)
CPU: (16) x64 Intel(R) Core(TM) i7-10700K CPU @ 3.80GHz
Memory: 39.11 GB / 62.26 GB
Container: Yes
Shell: 5.8 - /bin/zsh
Binaries:
Node: 16.15.0 - ~/.nvm/versions/node/v16.15.0/bin/node
Yarn: 1.22.19 - ~/.volta/bin/yarn
npm: 8.5.5 - ~/.nvm/versions/node/v16.15.0/bin/npm
Browsers:
Chrome: 111.0.5563.110
Firefox: 102.0
npmPackages:
@rspack/cli: latest => 0.1.1
@rspack/core: ^0.1.3 => 0.1.3
@rspack/dev-server: ^0.1.3 => 0.1.3
Details

Reproduce link
No response
Reproduce Steps
Seems like the there will be warning in the following, however, the build is successful
const bar = { foo: () => {} }
export const {foo} = bar

As far as I understand, there should be a fixtures cover the above code , like this https://github.com/web-infra-dev/rspack/commit/edd66e02e33c5c1dd72ad26a7e0c96ff051f64bb

I believe the above code is a correct, as they are used extensivily in @reduxjs/toolkit https://redux-toolkit.js.org/tutorials/typescript

I'm interested. Can I take a try?

I'm interested. Can I take a try?

sure

Looks like the problem locates in crates/rspack_core/src/tree_shaking/visitor.rs, where we didn't handle destructring.

Looks like the problem locates in crates/rspack_core/src/tree_shaking/visitor.rs, where we didn't handle destructring.

yeah

Fantastic work. Thank you @IWANABETHATGUY @HerringtonDarkholme


Stuck transaction

Describe the bug
I have a transaction that is currently stuck. It's been approximately 7 hrs since I've sent my transaction. I tried removing the transaction using the trash can icon on the top right corner of the transaction only for it to return in my transaction list upon restart of wallet.  The transaction ID given in the wallet is not showing up in the block explorer. I've looked into all my previous receive addresses in the block explorer and my coins seem to still be unmoved from my addresses. Does this means my funds are retrievable or are they lost in limbo? I tried entering my mnemonic in another wallet but realized my mnemonic is only 12 words plus a passphrase. I believe the other wallets are 24 word mnemonic.
To Reproduce
Not sure if this exactly reproduces the behavior or not...
these are just steps i took prior to having a stuck transaction.
Steps to reproduce the behavior:

Send multiple transaction to the wallet using multiple different receive addresses.
Once receive transactions are confirmed.

Select an address from one of your saved contacts.
For the amount press the Send Max button.
Send.
Go to the transaction list.
Transaction will have the satellite logo to the left of it and will not confirm nor will the trans id be in the block explorer.

Expected behavior
I expect the wallet to send my transaction and confirm.
Smartphone (please complete the following information):

Device: iPhone 7
OS: iOS 12.1.3
iOS Wallet: v0.14-beta.2

This was solved by using the right amount of transaction fee.


Add linter and syntax formatter

Add automated linting and syntax formatting. There are patterns for this in other RAC-developed apps:

autopep8
flake8
isort
implemented as a git pre-commit hook

Examples in RAC apps:
dart-digitization/add pre-commit
AS_export/add pre-commit


Proxy protocol support

Hi, Just a question.
I deployed this docker-mailserver image on kubernetes behind haproxy which sends proxy-protocol.
But seems amavis failed to send email to smtpd.
Please see following log messages.

Sep  7 05:53:59 mailserver postfix/smtpd[1062]: warning: haproxy read: unexpected EOF

I guess <IP_ADDRESS>:10025 expect to receive proxy protocol but amavis  didn't send.
I think I should modify master.cf file before launching.
So Is there any official way to configure master.cf file after docker-mailserver is deployed?
Thanks.
Full Log
Sep  7 05:53:20 mailserver postfix/smtpd[1053]: connect from st41p57im-ztfb14240501.us.com[<IP_ADDRESS>]
Sep  7 05:53:21 mailserver postfix/smtpd[1053]: C991582EA6D697E4B6C12: client=st41p57im-ztfb14240501.us.com[<IP_ADDRESS>]
Sep  7 05:53:22 mailserver postfix/cleanup[1058]: C991582EA6D697E4B6C12<EMAIL_ADDRESS>Sep  7 05:53:22 mailserver opendkim[184]: C991582EA6D697E4B6C12: st41p57im-ztfb14240501.us.com [<IP_ADDRESS>] not internal
Sep  7 05:53:22 mailserver opendkim[184]: C991582EA6D697E4B6C12: not authenticated
Sep  7 05:53:22 mailserver opendkim[184]: C991582EA6D697E4B6C12: DKIM verification successful
Sep  7 05:53:22 mailserver opendkim[184]: C991582EA6D697E4B6C12: s=04042017 d=us.com SSL
Sep  7 05:53:22 mailserver opendmarc[196]: implicit authentication service: mail.lazy.work
Sep  7 05:53:22 mailserver opendmarc[196]: C991582EA6D697E4B6C12: us.com pass
Sep  7 05:53:22 mailserver postfix/qmgr[1033]: C991582EA6D697E4B6C12<EMAIL_ADDRESS>size=2180, nrcpt=1 (queue active)
Sep  7 05:53:23 mailserver postfix/smtpd[1053]: disconnect from st41p57im-ztfb14240501.us.com[<IP_ADDRESS>] ehlo=2 starttls=1 mail=1 rcpt=1 data=1 quit=1 commands=7
Sep  7 05:53:59 mailserver amavis[1045]: (01045-01) (!)rw_loop: leaving rw loop, no progress, last event (select) 35.011 s ago
Sep  7 05:53:59 mailserver postfix/smtpd[1062]: warning: haproxy read: unexpected EOF
Sep  7 05:53:59 mailserver postfix/smtpd[1062]: connect from unknown[unknown]
Sep  7 05:53:59 mailserver postfix/smtpd[1062]: disconnect from unknown[unknown] commands=0/0
Sep  7 05:53:59 mailserver amavis[1045]: (01045-01) (!)O440fHx7_I21 FWD from<EMAIL_ADDRESS>-><EMAIL_ADDRESS> 451 4.5.0 From MTA() during fwd-connect (No greeting, dt: 35.012 s): id=01045-01
Sep  7 05:53:59 mailserver amavis[1045]: (01045-01) Blocked MTA-BLOCKED {TempFailedInbound}, [<IP_ADDRESS>]:36637 [<IP_ADDRESS>]<EMAIL_ADDRESS>-><EMAIL_ADDRESS>Queue-ID: C991582EA6D697E4B6C12, Message-ID<EMAIL_ADDRESS>mail_id: O440fHx7_I21, Hits: -0.322, size: 2367, dkim_sd=04042017:us.com, 37021 ms
Sep  7 05:53:59 mailserver postfix/smtp[1034]: C991582EA6D697E4B6C12<EMAIL_ADDRESS>relay=<IP_ADDRESS>[<IP_ADDRESS>]:10024, delay=39, delays=1.6/0.01/0.01/37, dsn=4.5.0, status=deferred (host <IP_ADDRESS>[<IP_ADDRESS>] said: 451 4.5.0 id=01045-01 - Temporary MTA failure on relaying, From MTA() during fwd-connect (No greeting, dt: 35.012 s): id=01045-01 (in reply to end of DATA command))

ConfigMap I used

https://github.com/yuanying/k8s-env/blob/master/charts/mailserver/templates/configmaps.yaml#L23-L35

No there is no option available yet

Personally I have no setup for testing. Would you like to suggest a fix in a PR?

Hi, sorry for my late reply.
How about introduce postscreen to this mailserver?
This will fix my problem indirectly.
Anyway I'll check this idea and may send PR.
Thanks!

It looks like a solution and improvement.


Test against php 8.2 and 8.3

@BackEndTea not sure why those two checks have failed

Will be resolved by #304.


Shrink the Gather's output shape

Shrink the Gather's output shape to avoid crash in AddGemm(). For example:
Gather output shape {1, 1, 256} -> shrink to 2d {1, 256}.  Gemm can only support 2d inputs now.

Thanks Mingming.
Interesting 🤔. There are a number of places where I'm passing the modified output tensor desc to CreateNodeOutput. I'll need to review each of these cases...
GraphDMLImpl::AddInstanceNormalization(...)
...
  output_tensor_desc.PermuteDimensions(tensor_dimensions_permutation, TensorDesc::Alignment::kTrailing);
...
  auto node_output = graph_desc_builder_->CreateNodeOutput(
      node, 0, std::move(output_tensor_desc));

Yes, all nodes need to be checked again. If the output shape has been reset in graph_dml_impl.cc and mismatch with the graph builder level, it's dangerous and will let device_->CreateOperator(&op_desc, IID_PPV_ARGS(&op)) failed. You can refer to the https://github.com/mingmingtasd/webnn-native/blob/dml_optimize/src/webnn/native/dml/GraphDML.cpp, you can see, I ensure the output shape can align with the graph builder level.

I mean aligning with graph builder level equals to webnn spec, anyway, we need to keep a specific shape for each node no matther which backend/implementation actually give.

Indeed. The reason it works as-is for so many operators is because the input and output shapes were the same anyway, or the output shape between DML and WebNN/ONNX were the same. So, this really hits operators where the necessary DML shape and the WebNN/ONNX size differ. My single operator tests would not have caught these because it shows the client-side builder shape, not the internal shape, and this is the first time we've glued together so many operators together.


Sign up through the sign-in page/Have the ability to log in through the 'MetaMask' button

1)Sign up through the sign-in page
When attempting to log in with a non-existent account, it should redirect to the sign-up procedure instead of displaying the 'register first' message.
4sign up.webm
2)Have the ability to log in through the 'MetaMask' button
On the website 'dapp.satt.com,' clicking on the 'MetaMask' button does not trigger any action.
3metamask.webm

It remains to sign up through the 'sign in' page with Facebook and Telegram


Remove capacity field from array types.

As I am working on zig bindings for Clay I notice that array types are defined to have a redundant capacity field. As far as I can tell tell this is just a value copy of the maximum amount of items in most cases.
If this does get removed I can cleanly convert between Clay's array types and zig's built in slice type making for a nicer development experience and if someone really does need the capacity they can get it from the arena.

Looking at the code, there are only two struct in the public api that have a capacity field. One is Clay_Arena which makes great use of it, and which is also more complex than just a simple slice. The other one is Clay_RenderCommandArray which also uses it.
The other thing to keep in mind is that even it they got removed, you still wouldn't be able to implement what you want. In zig slices have no guaranteed in-memory representation, so you can't export functions that have slices in their signature.
As for a solution, it's easy enough to write wrapper functions around clay's api that convert []const u8 to and from Clay_String. You can add the conversion functions as members so it's super easy to use:
pub const String = extern struct {
    len: usize,
        chars: [*c]const u8,

        pub fn init(string: []const u8) String {
        return String{ .chars = string.ptr, .len = string.len };
    }
    pub fn slice(self: String) []const u8 {
        return self.chars[0..self.len];
    }
};

And for Clay_RenderCommandArray which we need to iterate over in the renderers, it's easy enough to write render_commands.slice():
pub const RenderCommandArray = extern struct {
    capacity: u32,
        len: u32,
        items: [*c]RenderCommand,

    pub fn slice(self: RenderCommandArray) []RenderCommand {
        return self.items[0..self.len];
    }
};

Yes I agree, I am not advocating for the removal of capacity on Clay_Arena and im not sure why I thought this was needed in the first place. Chalk it up to a tired morning I guess.


Fixing required in Documentation

Hi,
Can you please change the documentation to:
"require" : {"pubsubhubbub/subscriber": "*"}
instead of
"require": { "pubsubhubub/subscriber": "*" }
Thanks

Good catch; this is now fixed. Thanks for the report!


Made share tier tests more reliable

The service returns a time truncated to seconds, so the time we are comparing to getLastAccessTierChangeTime needs to be truncated as well.

/azp run java - storage - tests

/azp run java - storage - tests

/check-enforcer override

Live test failures are due to eng sys issues.


React Metrics webapp

The following have been implemented in this branch:

React (CRA) + Redux toolkit set up
React Router set up
connection to the UK Carbon Intensity API
homepage with links to regions categorized by carbon intensity indices
regions list pages filtered by intensity index
region detail pages displaying the carbon generation mix for the forecast
styling based on reference
responsive layout
unit and integration tests

Project documentation video: https://drive.google.com/file/d/1DXaeBKwAhjZR5ZCZDOxT3l8LZYnHt5ZE/view?usp=sharing

Hi, @balatstar If you notice that "ridwanediallo approved these changes," it was an error and I apologize. Hence, please check the changes that have been requested.

I have re-recorded the video and updated the link on the PR description. Pasting the link here also for reference: https://drive.google.com/file/d/1g4VmI26BWdsLj2V-3XcLxp74jhoPQEIq/view?usp=sharing


Can't upgrade npm within node@4 on fresh brew install with its only package

Much has been said about brew/node/npm and permissions problems. Today I've encountered a configuration I haven't seen before, so maybe it's worth reporting.
Prerequisite: completely pristine OSX, no previous Homebrew files around

Install brew: /usr/bin/ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)"
Observe that /usr/local/lib is empty
Install node4: brew install node@4
Observe that there is now /usr/local/lib/node_modules/
Run npm install -g npm

💥
npm ERR! Darwin 16.4.0
npm ERR! argv<EMAIL_ADDRESS>"/usr/local/bin/npm" "install" "-g" "npm"
npm ERR! node v4.8.0
npm ERR! npm  v2.15.11
npm ERR! path /usr/local/lib
npm ERR! code EACCES
npm ERR! errno -13
npm ERR! syscall rmdir

npm ERR! Error: EACCES: permission denied, rmdir '/usr/local/lib'
npm ERR!     at Error (native)
npm ERR!  { [Error: EACCES: permission denied, rmdir '/usr/local/lib']
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   syscall: 'rmdir',
npm ERR!   path: '/usr/local/lib' }
npm ERR!
npm ERR! Please try running this command again as root/Administrator.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/user/npm-debug.log

At this point things are completely foobar because npm and node_modules were removed and no other version installed.
A quick fix is to uninstall/install the brew package:
brew uninstall node@4 && brew install node@4
Fun fact: if there is anything else in /usr/local/lib/, it works!
Same procedure as above, with a new 5):

Install brew: /usr/bin/ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)"
Observe that /usr/local/lib is empty
Install node4: brew install node@4
Observe that there is now /usr/local/lib/node_modules/
touch /usr/local/lib/dummy
Run npm install -g npm

Everything works 🚀
It is my understanding this is more likely a problem of how npm itself works and not a brew issue.
Note that this is when upgrading from node4's npm which is version 2.
I did not encounter such problems when I manually jumped e.g. from npm 4 to 3 or such, so this seems not a current bug of npm anymore.
But since an older version of npm is installed here, maybe brew can do something as a workaround.
thanks!

Run npm install -g npm

Why do you need to run this, out of interest?

Have node4 with a global default newer version of npm (default is npm2, we want 4).
Is there a better way to achieve this?

I don't know, I don't use NodeJS. Regardless, this doesn't seem to be a Homebrew bug, sorry 😭

IIRC this version of node isn't even supported by npm any more, but if you're going to keep the older nodes around I'd nudge that they shouldn't ship with anything older than the 3.x branch of npm.

@DomT4 Do all old nodes work with new NPMs? I have no idea about this stuff but assuming that's the case 👍 to someone updating these to their newest supported versions.

Up until the end of the 3.x branch at least some of the older Nodes were still actively supported. From the final 3.x release:
#### UPDATE SUPPORT MATRIX

With the advent of the second official Node.js LTS release, Node 6.x
'Boron', the Node.js project has now officially dropped versions 0.10
and 0.12 out of the maintenance phase of LTS. (Also, Node 5 was never
part of LTS, and will see no further support now that Node 7 has been
released.) As a small team with limited resources, the npm CLI team is
following suit and dropping those versions of Node from its CI test
matrix.

* [`c82ecfd`](https://github.com/npm/npm/commit/c82ecfdbe0b5f318a175714a8753efe4dfd3e4b3)
  [#14503](https://github.com/npm/npm/pull/14503)
  Node 6 is LTS; 5.x, 0.10, and 0.12 are unsupported.
  ([@othiym23](https://github.com/othiym23))

The https://github.com/npm/npm/commit/c82ecfdbe0b5f318a175714a8753efe4dfd3e4b3 commit referenced there shows it should be fine to apply the final 3.x release to 0.10, 0.12, and all the 3/4/5 releases.

@DomT4 BTW: currently every node and node@* installation is overwriting npm, so your current npm is going to be whichever npm was bundled with whichever node or node@* you installed last.

@ilovezfs Aye, I know. It was one of the reasons I pushed a while back for Homebrew to drop all of the older Nodes and nudge people towards nvm or such instead.
Homebrew really doesn't handle mixing and matching newer/older versions of node well, and I'm surprised there haven't been more issues. My recommendation was and pretty much still is that Homebrew only supports versions of Node that can ship/use the same version of npm without issue, but I appreciate that's not the direction the project has gone down, so just trying to minimise the variation really.

I think the npm's bundled with the node@* need to be confined to their opt prefix.

Did you see my comment the other day in this thread? I think this would be a neat idea for the keg_only versions of node at least.
You'd lose your npm packages every upgrade but I don't think that's too punishing given how infrequently some of these old nodes are updated now. Not like you'd be having to npm install -g the world every 3 weeks or something.

Now, Homebrew could steal a page out of something like nvm and essentially write a source script for each version of node that when placed in your shell profile then handled ensuring the desired node and any npm packages installed were in your $PATH and I actually think that'd be a really cool system, but I think if I suggested that I might get glared at by both Mike and ILZ 😸

I don't object to that in principle. I just wonder if we're equipped to do that well enough that it would actually be better than some "you really should use nvm for that" advice, a :lollipop:, and sending them on their way.

My advice remains that Homebrew should drop everything not supported by a current npm without some kind of significant rewrite of the current system, but I'm not sure how much that's on the table as an option. I'd almost be tempted to literally make brew install node@4 and such an alias for brew install nvm 😅. If culling isn't an option I agree the current system is equivalent to poking a bear with a stick and hoping it doesn't eat you at some point.

My advice remains that Homebrew should drop everything not supported by a current npm

Specifically, that would mean removing which ones?
bash-3.2$ ls -1 node@*
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

node@4 could stay and be updated to a current npm. I don't think npm love having to do it, but they do still run their test suite against that version of node as it was the previous LTS branch. They currently test:
node (Or node@7 if you prefer)
node@6
node@4

Or well, test isn't fair there. They test pretty much everything even semi-recent, so think "support" more than "test". The point more being if you attempt to use a modern npm against one of those unsupported node releases and something breaks you're own your own.

unique install on request events in the last 30 days
node        97,752 (and node --without-npm 2,249)
node@6       5,595
node@4       4,721
node@5         744
<EMAIL_ADDRESS>    403
<EMAIL_ADDRESS>    272

It would seem safe enough to kill node@5, <EMAIL_ADDRESS>and<EMAIL_ADDRESS>then and simply update node@6 and node@4 to use the same npm as node.
It wouldn't surprise me if the chunk of --without-npm installations there are still as a result of Homebrew's reputation for handling npm badly in the past and related gists/blog posts/tweets/etc. There was much drama around that way back 😢.
I do think at some point Homebrew is going to have to drastically change the way npm is installed for the versioned formulae at the very least, but if you wanted to punt that down the line this is the way to achieve that.

You'd lose your npm packages every upgrade but I don't think that's too punishing given how infrequently some of these old nodes are updated

👎  to that.
I'd rather we leave the versioned Nodes in here while they still have non-trivial numbers of installs and aren't causing us user issues. If/when the user issues come in: by all means let's kill them rather than fix them.

I don't think it makes sense to blow away the node modules on upgrade, but I do think it's time to remove node@5, <EMAIL_ADDRESS>and<EMAIL_ADDRESS>

https://github.com/Homebrew/homebrew-core/pull/10577

👎 to that.

Was mostly just throwing around ideas. It's not my preference, but I didn't want to be too much like "Hey here's my advice take it or leave it bye" 😄. We could always chase the pypy model if not, which would make them "permanent", but ensure each node/npm tree was unique.

Yeah it seems like it would be best to have one npm and one modules tree per node/node@*.

@DomT4 the following changes have been made:
<EMAIL_ADDRESS>made non-keg_only, conflicts with node and<EMAIL_ADDRESS>added.
<EMAIL_ADDRESS>made non-keg_only, conflicts with node and<EMAIL_ADDRESS>added.
node@4: npm upgraded to 4.1.2.
node@5: deleted.
node@6: npm upgraded to 4.1.2.
node: conflicts with<EMAIL_ADDRESS>and<EMAIL_ADDRESS>added.
Revision bumps all around.

Are you happy with the layout of things now or do you want me to plow some time into separating the npm trees across each node@ version in the kind of pypy model?

@DomT4 I'm not sure yet about that. I would appreciate it if you could take a look at enhancing the node DSL so that it's not hard-coding the assumption that the dependency is depends_on "node" as opposed to one of the node@* gang. See https://github.com/Homebrew/brew/blob/master/Library/Homebrew/language/node.rb#L17.
Regarding what, if anything, to do about the layout, separating the module trees intuitively seems like the soundest approach but it might be possible to continue with the status-quo, and keep rotating out formulae from keg-only to conflicts-with status as soon as they no longer work with the npm that's designated by upstream for the main node formula. Thoughts?

can be solved like this:
1.delete the path /usr/local/lib/node_modules under npm
2. brew reinstall node


Merge explore orgs into explore users

Now, there is only one explore page to search for accounts.
The existing setting to define the redirect behavior of / has been changed so that it no longer redirects to the no longer existing /explore/organizations.
As such, this refactoring is not breaking.
Additionally, fix incorrect visibility behavior for /explore/users.
It was correct for /explore/organizations.
Fixes #29803
After

(Note that both tester and test-org are orgs, test2 is a bot, and test is a normal user)

I don't think this is better than before. It's confusing mixed users and orgs.

Maybe add a https://primer.github.io/octicons/organization-16 besides the org name to distinguish?

@silverwind

Additionally, I'd like to keep the refactoring clean, so that we add in these enhancements afterward.
What do you think?

I can also add it in this PR, but I would have liked to first make a refactoring and then a UX enhancement.
What's your preference on that?

Don't really care, these are seldomly used pages. But I do have a slight preference because landing something "unfinished" into main branch is not so great.

@silverwind Done.
I've adapted the screenshots as well.

Icon and text seem slightly misalign. Maybe it needs a gt-df gt-ac gt-fw (or equivalent tailwind classes) on the parent?

Done

I don't think it should be called users, imho the most common use case is to search for organizations not users. Personally I would just drop user search from there but if combined it would be preferable to call the tab Organizations

(Alternatively, the distinction could also be repos, owner.
However, I don't think that's any better either as owner is only a code-internal concept we haven't communicated to our end users, and orgs are on all accounts users too)

Furthermore, you can still query users with it, so it would be false advertising since an org is a user but a user is not necessarily an org
Or what would you say about it?

(Alternatively, the distinction could also be repos, owner.
However, I don't think that's any better either as owner is only a code-internal concept we haven't communicated to our end users, and orgs are on all accounts users too)

Furthermore, you can still query users with it, so it would be false advertising since an org is a user but a user is not necessarily an org.
Or how do you see it?

If you look from business side of view (not how it's technically saved in database) these are both different things. But from use case scenario in this place you would most commonly look for organizations to get to it's repositories not user profile

If you think both UI are similar because we haven't displayed more information on the list pages. I think the direction is displaying more user or org-related information on the list page but not sharing it.

I also agree that the concept "User" and "Org" are fundamentally different. Although they share the same base table, but they are different things to end users, and it should avoid mixing them together.

just to add my 2c ...
while the backend may mean ORG and USERS are (almost) synonymous, to a user they are not.
From a UX perspective having the tab just called "users" will confuse people ... "I want to find the project page, where do I search".
From people behaviour there is a mental distinction between what a USER is and what an ORG is since an ORG is a collection of related repositories to meet a common goal . A user however is someone that interacts with gitea/git, is associated with a project (aka ORG) and may have a collection of repositories that at face value would appear to be randomly associated but means something to a user (forks of associated repositories [users or org], their own pet projects).
Likewise users are associated to org's via teams but orgs are not associated to a user.
NOW... from a streamlining of the UI... this might make sense but

it can't be called users BECAUSE users use this page, orgs do not and users use this page to find orgs or users
clearly indicate whether it is a USER or a ORG, especially as people use odd names and name orgs oddly

At least, add an option in sort to select which type of user to search, individual or organization.
Then, at least, we can have the same behavior.

I cannot imagine we have the requirement to search both users and orgs.


[WIP] [AA-1543] Implement versioning into Admin API

@pmcvtm @saa14
Please help review the changes.

Changes implemented to enable the endpoint to work with one api version.
MapApiVersion extension added to enable version specific grouping and update the endpoint url with version (ex: /v1/vendors)

Updated more implementation details on the ticket.
Thanks

Thanks for the foundational work here y'all. I opened a new PR (#311) since while revealing, much of this ended up not being a great fit for a solution as we were looking for it to function. This will be good to have for reference as the ASP.NET versioning library evolves with minimal APIs.


release(chart): changes for OpenEBS v3.1.0 release

Signed-off-by: Niladri Halder<EMAIL_ADDRESS>Why is this PR required? What issue does it fix?:
This releases a new helm chart which will use v0.9 of the NFS Provisioner and the NFS server images.
What this PR does?:
Updates the chart and app versions.
Updates the README.md
Does this PR require any upgrade changes?:
If the changes in this PR are manually verified, list down the scenarios covered::
Any additional information for your reviewer? :
https://github.com/openebs/charts/pull/299 is a dependant PR.

Codecov Report

Merging #131 (9dd82fa) into develop (41672bd) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #131   +/-   ##
========================================
  Coverage    50.12%   50.12%
========================================
  Files           38       38
  Lines         2789     2789
========================================
  Hits          1398     1398
  Misses        1285     1285
  Partials       106      106

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 41672bd...9dd82fa. Read the comment docs.


ldapsearch hangs after "connect success" from WSL 2 Ubuntu 18.04

Version
Microsoft Windows [Version 10.0.19042.1526]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
5.10.16
Distro Version
Ubuntu-18.04
Other Software
Trying to get ldap connectivity from WSL 2 but am having issues.  Testing with ldapsearch results in hanging after I get "connect success".  I am able to telnet from my WSL 2 instance to the ldap server over port 636 though which is odd.  I thought perhaps I needed to do some port forwarding but that didn't seem to work either.
Repro Steps
Reproducing will be difficult to impossible since this is on a corporate network contacting an enterprise LDAP domain controller.  However, setup Ubuntu 18.04 on WSL 2 and install ldap-utils.  Then try to connect to your ldap server via ldapsearch -d 1 -H your.ldap.server.com.
Expected Behavior
I expect to receive a successful connection and something to the effect of:

# extended LDIF
#
# LDAPv3
# base <> (default) with scope subtree
# filter: (objectclass=*)
# requesting: ALL
#
# search result
search: 2
result: 1 Operations error
text: 000004DC: LdapErr: DSID-0C090A71, comment: In order to perform this opera
tion a successful bind must be completed on the connection., data 0, v3839
# numResponses: 1

Actual Behavior
Using the -d 1 debug flag, I get the below that just hangs indefinitely:

ldap_url_parse_ext(ldaps://ldap.org.net)
ldap_initialize( ldaps://ldap.org.net:636/??base )
ldap_create
ldap_url_parse_ext(ldaps://ldap.org.net:636/??base)
Enter LDAP Password:
ldap_sasl_bind
ldap_send_initial_request
ldap_new_connection 1 1 0
ldap_int_open_connection
ldap_connect_to_host: TCP ldap.org.net:636
ldap_new_socket: 3
ldap_prepare_socket: 3
ldap_connect_to_host: Trying <IP_ADDRESS>:636
ldap_pvt_connect: fd: 3 tm: -1 async: 0
attempting to connect:
connect success

Diagnostic Logs
No response

From what we can tell, the LDAP search gets the IP of the WSL virtual NIC and therefore the response doesn't know how to get back to it.  Assume this is what bridged mode would fix.  We tried messing with portproxy and firewall but none of our efforts seemed to resolve the communication failure.


Window resize issue

View space issue
Changing the window size affects the size of the field of view. In the first photo we see one tree on the left. The second one has two and the third one has three. The size of the window should not determine the field of view.

Making the window size affect the field of view was a deliberate design choice.
This is a two dimensional game and I figured that would have been acceptable but I suppose that having a consistent FOV would make more sense, I will see what I can do.

Alright, I made it so that whenever you resize the window, the display will instead attempt to scale up or down rather than shrink/expand the FOV. The GUI still retains the same size whenever the window is resized though. Do you have any other suggestions?

The easiest way to fix this is to disable window resize using glfwSetWindowAttrib(win, GLFW_RESIZABLE, 0);, but this is not a very good option. You can implement an orthographic camera and use the window size aspect to update the camera matrix. This would be the best solution, but it would take some time and work.

I have implemented your suggestion in this commit
In all due honesty, I have realized that much of the code I have written for this project isn't particularly good and the change I have made is rather hacky but should hopefully work for now. Please let me know of any other thoughts you currently have on this, and if all is good I'll close the issue.


How to generate multiple payment addresses from a single secret key

I see that electrum wallet generates different payment addresses for receiving BTC. I'm trying to generate new addresses, but they are the same. Is it possible to make them different?
private function generateAddress()
    {
        $pubKeyHash160 = $this->_privateKey->getPublicKey()->getPubKeyHash();
        $pubKeyHashAddr = new PayToPubKeyHashAddress($pubKeyHash160);
        $this->_address = $pubKeyHashAddr->getAddress();
    }

I've found that electrum generates multiple secret keys -> public keys -> addresses from a single seed phrase. But I don't understand how to restore all the secret keys from it. Any thoughts?

Recently & possible related: https://github.com/Bit-Wasp/bitcoin-php/issues/792 https://github.com/Bit-Wasp/bitcoin-php/issues/794
What do you mean the addresses are the same? The address in your code returns the same value every time, or you're trying to change the address type?


!admins

The command to output to the user of the administrators who are currently online.

Would probably be a good idea to encapsulate code to fetch a list of admins in the core package. An array of remote and local admin lists to load in the config would probably be a good approach.


docker: Error response from daemon: invalid mount config for type "bind": bind source path does not exist: /home/$USER/Desktop/git/plm/workspace.

I got this error while trying to open as instructions
[2021-04-29T15:00:39.202Z] [PID 699429] Step 10/10 : USER $IMAGE_USER
[2021-04-29T15:00:39.202Z] [PID 699429]  ---> Using cache
 ---> c9d2149c1cc9
[2021-04-29T15:00:39.202Z] [PID 699429] Successfully built c9d2149c1cc9
[2021-04-29T15:00:39.570Z] [PID 699429] Successfully tagged vsc-plm-25bebab07de9ae7520e8688f7bc1d502-uid:latest
[2021-04-29T15:00:39.573Z] [PID 699429] [1237 ms] Start: Run: docker -v
[2021-04-29T15:00:39.611Z] [PID 699429] [1275 ms] Start: Run: docker events --format {{json .}} --filter event=start
[2021-04-29T15:00:39.615Z] [PID 699429] [1279 ms] Start: Starting container
[2021-04-29T15:00:39.615Z] [PID 699429] [1279 ms] Start: Run: docker run --sig-proxy=false -a STDOUT -a STDERR --mount source=/home/$USER/Desktop/git/plm/workspace,target=/home/developer/workspace,type=bind,consistency=delegated --mount source=/dev/bus/usb,target=/dev/bus/usb,type=bind --mount type=volume,src=vscode,dst=/vscode -l vsch.local.folder=/home/$USER/Desktop/git/plm -l vsch.quality=stable -l vsch.remote.devPort=0 --privileged --entrypoint /bin/sh vsc-plm-25bebab07de9ae7520e8688f7bc1d502-uid -c echo Container started ; trap "exit 0" 15; while sleep 1 & wait $!; do :; done
[2021-04-29T15:00:39.662Z] [PID 699429] docker: Error response from daemon: invalid mount config for type "bind": bind source path does not exist: /home/$USER/Desktop/git/plm/workspace.
See 'docker run --help'.
[2021-04-29T15:00:39.665Z] [PID 699429] [1329 ms] Start: Run: docker ps -q -a --filter label=vsch.local.folder=/home/$USER/Desktop/git/plm --filter label=vsch.quality=stable
[2021-04-29T15:00:39.705Z] [PID 699429] [1369 ms] Command failed: docker run --sig-proxy=false -a STDOUT -a STDERR --mount source=/home/$USER/Desktop/git/plm/workspace,target=/home/developer/workspace,type=bind,consistency=delegated --mount source=/dev/bus/usb,target=/dev/bus/usb,type=bind --mount type=volume,src=vscode,dst=/vscode -l vsch.local.folder=/home/$USER/Desktop/git/plm -l vsch.quality=stable -l vsch.remote.devPort=0 --privileged --entrypoint /bin/sh vsc-plm-25bebab07de9ae7520e8688f7bc1d502-uid -c echo Container started ; trap "exit 0" 15; while sleep 1 & wait $!; do :; done

Sorry for that, I created the folder workspace with a wrong name folder. Now working as expected.


Pulumi

https://www.pulumi.com/pricing/ - "Call Us"

5 years later - RBAC and SSO is for rich Enterprise only ((


Duplicate entries in 1993

This problem report was submitted by @gurujeet-ngc via email:
AGAGD shows a big spike of over 3000 games on a day in 1993. This is in error and a result of games from the Go congress being doubly counted. See analysis and recommended action below.

The doubling occurred on the reporting for the Go Congress where games were reported as occurring on 8/7/1993. The total games for tournament cong19930807a (1540) equals the sum of the games for cong19930807b (778) and cong19930807c (762).

Looking at just one player’s games, Pin_Player_1, it is clear that the 9 games played in ‘a’ reappear in ‘b’ or ‘c’. This is true for all other players as well.
Recommendation: delete tournament cong19930807 and its games from the database as they are erroneous duplicates. Additionally there is a tournament cong19930808 with a much smaller number of games that all appear to duplicate some of the games played in the previous tournaments. It is possible that these were self-pair games but should be investigated further to see if they are duplicates also.

These should not be deleted directly from the database. they should be removed using ratings tools at best.

What are the problems / risks associated with deleting them from the database?
What would be your recommendation as to how to remove them with the ratings tools?

I lost Steve’s original message. Can you forward – thanks.
From: Nate @.>
Sent: Monday, August 9, 2021 11:35 AM
To: @.>
Cc: Gurujeet @.>; @.>
Subject: Re: [usgo/agagd] Duplicate entries in 1993 (#217)
What are the problems / risks associated with deleting them from the database?
What would be your recommendation as to how to remove them with the ratings tools?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/usgo/agagd/issues/217#issuecomment-895325131, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQV5GIIZRVY3BBNU5FEQS6LT37YTXANCNFSM5B2F4VCQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email.

If we were just to use the ratings tools for this and use the revert function, would that not require the original [tournament_name].in file? This would be based on the current documentation for our aga-ratings tools.

I think it can just be done with a few SQL delete statements. Identify which of the tables have the duplicated tournaments and games. I don’t have the schema, but that should be pretty straightforward.
From: Michael @.>
Sent: Monday, October 4, 2021 7:01 PM
To: @.>
Cc: Gurujeet @.>; @.>
Subject: Re: [usgo/agagd] Duplicate entries in 1993 (#217)
If we were just to use the ratings tools for this and use the revert function, would that not require the original [tournament_name].in file? This would be based on the current documentation for our aga-ratings tools.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/usgo/agagd/issues/217#issuecomment-933917765, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQV5GIJPUT3MX6M54Z5KSJLUFIW3BANCNFSM5B2F4VCQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

How do we know this is in error? Is this based on a large spike or historical research?
This should not be done directly though SQL commands. We have an archive of over 4000 old tournaments. At a minimum this should be testes with the ratings tools before any other attempts are made.

The error was clearly documented with the submission report.
From: Steve @.>
Sent: Monday, October 4, 2021 9:51 PM
To: @.>
Cc: Gurujeet @.>; @.>
Subject: Re: [usgo/agagd] Duplicate entries in 1993 (#217)
How do we know this is in error? Is this based on a large spike or historical research?
This should not be done directly though SQL commands. We have an archive of over 4000 old tournaments. At a minimum this should be testes with the ratings tools before any other attempts are made.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/usgo/agagd/issues/217#issuecomment-933992947, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQV5GIOIJZHGAACMF6X5MITUFJKZTANCNFSM5B2F4VCQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Re-provision the lcd server on a new linode instance

We need to

[x] 1. reprovision the lcd server on a new linode instance
[x] 2. disable password login - only ssh login should be allowed
[x] 3. when 1 and 2 are complete, delete old linode lcd instance
[x] 4. alert linode team here https://cloud.linode.com/support/tickets/20831765 when OLD instance deleted.

Closing as complete.


Vietnamese localization

Hello @tommai4881, welcome on the romcal project!
Can you elaborate a little more? Do you wish to have the Vietnamese localisation? Or do you plan to contribute to make it happen? Depending of your time and what you can do, there is multiple way to contribute. Let us know if you need help!

@emagnier was faster than me to comment on this issue. :wink:
I’d like to add AFAIK there is no current or past romcal contributor who would speak Vietnamese. If you find some time to contribute to romcal, even the Vietnamese calendar would use some tending. You could also use the calebrations list from Wikipedia, although it might a better idea to double-check it from the Vietnamese Roman Missal.
Nevertheless, any kind of contribution is most welcome! :smiley:

I'm closing this issue since there is no relevant discussion.
Note: it might be related to the contribution to this PR https://github.com/romcal/romcal/pull/276.


Incorrect redirection and duplicate OTP emails

Issue
I've integrated vue-clerk into my Vue 3 project to enable email login with OTP. However, when attempting to log in using the email for OTP, the redirection to the "/sign-in/factor-one" page is not being recognized within my Vue router config. As a result, the page goes blank, and the intended component is not displayed.
Solution Attempted
I attempted to address the problem by creating children routes, namely "factor-one","factor-two" and "verify" under the "/sign-in" route. While this resolved the issue of displaying the component that handles OTP entry and allows user access to the application, it introduced a new problem. The OTP email is being sent twice, with the first redirection triggering one and the subsequent component load triggering another.
I would like to know if anyone else has encountered a similar issue and if there are any known solutions or workarounds.
References:
https://clerk.com/blog/remix-delightful-developer-experiences
https://clerk.com/docs/custom-flows/overview

Hi, with Clerk Core 2 integrated with the latest vue-clerk versions, are you still able to replicate the issue?

Thanks for the update! I have an incoming PR that will remove the @clerk/clerk-js dependency and lower the bundle size :)
Closing this

Hey, I'm closing this for now. Please use the latest version and check if the error still persists. If it does, feel free to reopen it!


Performance regression

After much time, I've narrowed down to a commit https://github.com/jshint/jshint/commit/813d97a8f5d96f2ab8320ad5da63662195f79d6c that has caused JSHINT to take ~30 seconds longer to complete. I can consistently produce faster JSHINT times by going back one commit to https://github.com/jshint/jshint/commit/8a231e29422deddd8aef5e9bd65632a9bebf4528, the commit right before the culprit was merged in.
TBH, I'm not familiar with the JSHINT codebase and don't fully understand what the culprit commit is doing but based on the commit message, that commit was supposed to improve performance.
Please let me know if there's anything I can do. Thanks!

Maybe this is a duplicate of #2532? (And fixed by #2538)

This does appear to be a duplicate and will close it as such. Thanks!


feat(framework): print log to both file and console

Summary
Add a logger pool to DefaultLogger, whenever we need a nested logger, we use file name to check if the pool has that logger, if so, we just return it to create a  new default logger, otherwise, we create a new logger
Key Points

[ ] This is a breaking change
[ ] New or existing documentation is updated

Description
Describe what this PR does, and aims to solve in a few sentences.
Does this close any open issues?
closes #1716
Current Behavior
Describe the current behaviour of this issue, if relevant.
New Behavior
Describe the new behaviour updated in this issue, if relevant.
Screenshots

Other Information
Any other information that is important to this PR.

can not be closed


Marquee block design updates and ESLint issues resolved

Addressed the Marquee block center variant naming convention (centered is deprecated but still backwards compatible)
Adjusted the Marquee block quiet variant to have a 600px default content width
Adjusted the Marquee block split, one-third variant to be split 50/50 on tablet viewport
Fixed all linting issues in marquee.css and marquee.js files

Resolves:

MWPW-131806
MWPW-131560

Test URLs:

Before: https://main--milo--adobecom.hlx.page/docs/library/blocks/marquee
After: https://rparrish-marquee-fix--milo--adobecom.hlx.page/docs/library/blocks/marquee

FYI, Im seeing some inconsistencies w/ some of the headline sizes. Im addressing now and will update.

FYI. This above mentioned type issues are fixed and looks GTG on my end.


feature: allow specifications of consumer options via listener

NOTE: Do NOT add any Jira links to this PR description or commits.  This is a public repository.
Reason for Change
Allows max_attempts to be set when initializing a listener
List of Changes

Expand method signatures of FastlyNsq::Listener and FastlyNsq::Consumer to allow options to be passed to the underlying connection object.

Risks
None that I can see.
Checklist
Put an x in the boxes that apply.
You can also fill these out after creating the PR.

[x] I have NOT linked to any Jira tickets.
[x] I have linked to all relevant reference issues or work requests
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added or updated necessary documentation (if appropriate)
[x] Any dependent changes have been merged and published in downstream

Recommended Reviewers
@fastly/billing

:rage: rubicon
version bump? (oh yeah we can do that when we release)
nsq-ruby 2.3.0 is what has max_attempts support so even though this should work with 2.2 might help to update the gemspec

@leklund updated.


video_temp/4wex2/voice.wav No such file or directory

I get an error so "video_temp/4wex2/voice.wav No such file or directory". Please help me

Which TTS type are you using? If you are using the Built-In TTS option, then this may have caused an issue.
Check your server logs to check for anything odd. Feel free to post a snippet of your log here.
Another issue with the Built-In TTS is that invalid characters may cause it to crash. If you are on Linux, you may need to install the proper dependencies like espeak.

Thanks @alamshafil but now i get an error like this:

Failed to generate video: Internal server error: ffprobe exited with code 8 ffmpeg version 7.0.2-static https://johnvansickle.com/ffmpeg/ Copyright (c) 2000-2024 the FFmpeg developers built with gcc 8 (Debian 8.3.0-6) configuration: --enable-gpl --enable-version3 --enable-static --disable-debug --disable-ffplay --disable-indev=sndio --disable-outdev=sndio --cc=gcc --enable-fontconfig --enable-frei0r --enable-gnutls --enable-gmp --enable-libgme --enable-gray --enable-libaom --enable-libfribidi --enable-libass --enable-libvmaf --enable-libfreetype --enable-libmp3lame --enable-libopencore-amrnb --enable-libopencore-amrwb --enable-libopenjpeg --enable-librubberband --enable-libsoxr --enable-libspeex --enable-libsrt --enable-libvorbis --enable-libopus --enable-libtheora --enable-libvidstab --enable-libvo-amrwbenc --enable-libvpx --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxml2 --enable-libdav1d --enable-libxvid --enable-libzvbi --enable-libzimg libavutil 59. 8.100 / 59. 8.100 libavcodec 61. 3.100 / 61. 3.100 libavformat 61. 1.100 / 61. 1.100 libavdevice 61. 1.100 / 61. 1.100 libavfilter 10. 1.100 / 10. 1.100 libswscale 8. 1.100 / 8. 1.100 libswresample 5. 1.100 / 5. 1.100 libpostproc 58. 1.100 / 58. 1.100 Unrecognized option 'show_streams'. Error splitting the argument list: Option not found

Now i get an error at below, we use ubuntu machine, and we dont use docker. We want to install as native auto shorts project

After looking at the log output, it appears you are using the built-in TTS option in Linux.
The built-in TTS option in Linux does not support exporting audio, so unfortunately you cannot use this option. You can use trying use the ElevenLabs voice API type with API key. (The error message needs to improved.)
If you are using the server with UI, you can add the API key to the .env file in root and choose ElevenLabs in the UI.
.env file:
...
ELEVENLABS_API_KEY="key here"
...


fix: added secrets manager crn validation when var.skip_cos_sm_auth_policy is false

Description

Release required?

[ ] No release
[x] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

/run pipeline

:tada: This issue has been resolved in version 8.11.1 :tada:
The release is available on:

GitHub release
v8.11.1

Your semantic-release bot :package::rocket:


Automatic artifact uploads

Regards the "container" step
This is no longer relevant as we're transitioning away from Jenkins. However, the feature itself is still heavily requested.
Once the cmd repo has been transferred (see #32) we can add a new issue there with the equivalent feature request, as well as closing this issue.
Todo

Feed build with URL to Wharf API
Add to trigger Jenkinsfile to give artifacts property to container step
Add to container Jenkinsfile to upload specified file (or files in folder) to given Wharf API for its BUILD_REF
Add logging of which files were uploaded

Example step config
container:
  image: mcr.microsoft.com/dotnet/core/sdk:3.1
  cmds:
  - dotnet test /repo/src/Iver.Spark.sln --results-directory /test-results
  artifacts:
  - /test-results

To consider: curl and wget will not exist on every image that's used. It's really bad habit to assume the image using is either Linux, have certain tools preinstalled, or even has permission to do some stuff.
Good option is to mount a second container, for example how they do it in their example found in the README.md:
https://github.com/jenkinsci/kubernetes-plugin#container-group-support
Trimmed down example of just that:
podTemplate(containers: [
    containerTemplate(name: 'main', image: '$CONTAINER_IMAGE', ttyEnabled: true, command: 'cat'),
    containerTemplate(name: 'artifacts', image: 'ubuntu:20.04', ttyEnabled: true, command: 'cat')
  ]) {

    node(POD_LABEL) {
        stage('Run container commands') {
            git 'https://github.com/jenkinsci/kubernetes-plugin.git'
            container('maven') {
                sh CONTAINER_CMDS
            }
        }

        if (CONTAINER_ARTIFACTS) {
            stage('Upload artifacts') {
                container('golang') {
                    sh"""
                    for file in $CONTAINER_ARTIFACTS/*; do
                        curl $WHARF_API/something/artifacts
                    done
                    """
                }
            }
        }
    }
}

This is outdated and will be resolved in https://github.com/iver-wharf/rfcs/pull/25


urllib[3] not being patched in AWS lambda/python3.6?

Hi, is python3 urllib / urllib3 not being patched? I'd greatly like to avoid using 3rd party bloated libraries for https calls.  I tried urllib, urllib3, and requests, and only requests seems to be traced.

It looks like patch_all() might be broken with urllib/httplib. For the time being, I would recommend manually patching urllib/httplib by calling the following method:
from aws_xray_sdk.core import patch
patch(('httplib',))

Let me know if this fixes it for you. We'll investigate the issue further for the time being

It's working with patch().  Thanks.

Hey, if you want to track what Boto is doing as well, you may also patch it. But keep in mind that you will see multiple subsegments that represent both Boto and Boto's underlying httplib call.
This is something that's in the works in order to provide a better customer experience.
I'm closing this for now since the main issue has been resolved. Feel free to open it back up if you have any other issues pertaining to this.

Can we repoen this as a documentation issue? A note about it like about how you use x-ray with django might make a big difference


Selection mle by condition

Just for testing, validating the divergence-based ascertainment bias correction using selectionMLE where our bias correction applies only to bases in the ascertainment condition, instead of evenly across all bases.

Closing this branch. Empirical testing shows that this approach doesn't work

.


Object reference not set to an instance of an object

See the end of this message for details on invoking
just-in-time (JIT) debugging instead of this dialog box.
************** Exception Text **************
System.NullReferenceException: Object reference not set to an instance of an object.
at ALVR.Launcher.SaveSettings()
at System.Windows.Forms.Form.OnFormClosed(FormClosedEventArgs e)
at System.Windows.Forms.Form.RaiseFormClosedOnAppExit()
at System.Windows.Forms.Application.ExitInternal()
at System.Windows.Forms.Application.Exit(CancelEventArgs e)
at ALVR.Launcher.Launcher_Load(Object sender, EventArgs e)
at System.Windows.Forms.Form.OnLoad(EventArgs e)
at MetroFramework.Forms.MetroForm.OnLoad(EventArgs e)
at System.Windows.Forms.Form.OnCreateControl()
at System.Windows.Forms.Control.CreateControl(Boolean fIgnoreVisible)
at System.Windows.Forms.Control.CreateControl()
at System.Windows.Forms.Control.WmShowWindow(Message& m)
at System.Windows.Forms.Control.WndProc(Message& m)
at System.Windows.Forms.Form.WmShowWindow(Message& m)
at MetroFramework.Forms.MetroForm.WndProc(Message& m)
at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)
************** Loaded Assemblies **************
mscorlib
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.7.3416.0 built by: NET472REL1LAST_B
CodeBase: file:///C:/Windows/Microsoft.NET/Framework64/v4.0.30319/mscorlib.dll
ALVR
Assembly Version: <IP_ADDRESS>
Win32 Version: <IP_ADDRESS>
CodeBase: file:///C:/Users/Ciunics/Desktop/ALVR/ALVR.exe
System.Windows.Forms
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.7.3324.0 built by: NET472REL1LAST_C
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Windows.Forms/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Windows.Forms.dll
System
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.7.3416.0 built by: NET472REL1LAST_B
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.dll
System.Drawing
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.7.3190.0 built by: NET472REL1LAST_C
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Drawing/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/System.Drawing.dll
MetroFramework
Assembly Version: <IP_ADDRESS>
Win32 Version: <IP_ADDRESS>
CodeBase: file:///C:/Users/Ciunics/Desktop/ALVR/MetroFramework.DLL
System.Configuration
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.7.3324.0 built by: NET472REL1LAST_C
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Configuration/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/System.Configuration.dll
System.Core
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.7.3362.0 built by: NET472REL1LAST_C
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Core/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Core.dll
System.Xml
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.7.3190.0 built by: NET472REL1LAST_C
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Xml/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Xml.dll
MetroFramework.Fonts
Assembly Version: <IP_ADDRESS>
Win32 Version: <IP_ADDRESS>
CodeBase: file:///C:/Users/Ciunics/Desktop/ALVR/MetroFramework.Fonts.DLL
Accessibility
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.7.3190.0 built by: NET472REL1LAST_C
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/Accessibility/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/Accessibility.dll
Microsoft.CSharp
Assembly Version: <IP_ADDRESS>
Win32 Version: 4.7.3190.0
CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/Microsoft.CSharp/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/Microsoft.CSharp.dll
************** JIT Debugging **************
To enable just-in-time (JIT) debugging, the .config file for this
application or computer (machine.config) must have the
jitDebugging value set in the system.windows.forms section.
The application must also be compiled with debugging
enabled.
For example:

When JIT debugging is enabled, any unhandled exception
will be sent to the JIT debugger registered on the computer
rather than be handled by this dialog box.

Did you fix it im having same problem.

Any chance you were able to solve this? I'm having the same issue.....


Implement Relationship Search

https://atlas.apache.org/api/v2/resource_DiscoveryREST.html#resource_DiscoveryREST_searchRelatedEntities_GET

Why relationship endpoints is not already implemented? I can contribute with that

We really appreciate If you can help with that @jmrobles.

@jmrobles do you have any update on this ?

Sorry @verdan, not yet, but it's still in ToDo


Change Test cases for new Time APIs

Purpose

This PR to change the test cases which use Time module APIs.
Fixes: #1079

Reviewers

@niveathika @BuddhiWathsala

Related PRs

https://github.com/ballerina-platform/module-ballerina-time/pull/68
https://github.com/ballerina-platform/module-ballerina-sql/pull/150

Migrations (if applicable)

https://github.com/ballerina-platform/ballerina-standard-library/issues/1079

LGTM, better if we can create an issue to track all the disabled test cases?

+1

LGTM, better if we can create an issue to track all the disabled test cases?

issue is created : https://github.com/ballerina-platform/ballerina-standard-library/issues/1098


fix: broken custom payment with Bidali

Description
This fixes the custom payment integration with Bidali that was broken by https://github.com/valora-inc/wallet/pull/5692
Because it renamed window.valora to window.walletApp.
Note: it would be nice to make this Bidali payment flow work multichain too. Right now it's only for the Celo network.
Test plan
Before
https://github.com/user-attachments/assets/f2d0946a-376e-4b2b-b029-d25e5659ef74
It's not showing the custom payment integration. Only WalletConnect or the QR code (which is not really usable as is in the mobile-to-mobile flow).
Note: the payment QR code causes an incorrect conversion, but that's another story, we'll tackle that separately.
After
https://github.com/user-attachments/assets/abe723c9-0a33-4737-81ac-de1834ab02b9
It's showing the custom payment integration "Pay with Valora".
Related issues

Part of RET-1292

Backwards compatibility
Yes
Network scalability
If a new NetworkId and/or Network are added in the future, the changes in this PR will:

[x] Continue to work without code changes, OR trigger a compilation error (guaranteeing we find it when a new network is added)

nice! should we use APP_REGISTRY_NAME instead? it seems like this integration with Bidali is specific to valora, so the same value shouldn't be used for other apps? even if that means the flow is broken on MS apps (perhaps we should gate the Bidali feature in config?)

Good question, I'd say for now we keep valora to avoid unintentionally breaking this in the future since it's something Bidali reads, the only specificity is that Bidali shows "Pay with Valora".
Not great, but we could easily ask Bidali to support a new appDisplayName we'd provide.
Anyway we should definitely decide what to do with this feature for Mobile Stack apps.


Building returns error 2

Idk if your still maintaining this, but whenever I try and build I get an error. Ive tried on linux and windows. on linux it gives me this:
~/Downloads/Dialga-master$ DEVKITPRO=/opt/devkitpro/ make
make all -f MakefileNSO BDSPVER=110 BDSPVERSTR=1.1.0
make[1]: Entering directory '/home/zrm9901/Downloads/Dialga-master'
make -C build110 -f /home/zrm9901/Downloads/Dialga-master/MakefileNSO
make[2]: Entering directory '/home/zrm9901/Downloads/Dialga-master/build110'
logger.cpp
In file included from /opt/devkitpro/devkitA64/aarch64-none-elf/include/c++/14.1.0/string:67,
from /home/zrm9901/Downloads/Dialga-master/include/logger.hpp:3,
from /home/zrm9901/Downloads/Dialga-master/source/logger.cpp:1:
/opt/devkitpro/devkitA64/aarch64-none-elf/include/c++/14.1.0/bits/memory_resource.h: In member function 'void* std::pmr::memory_resource::allocate(std::size_t, std::size_t)':
/opt/devkitpro/devkitA64/aarch64-none-elf/include/c++/14.1.0/bits/memory_resource.h:76:28: error: no matching function for call to 'operator new(std::size_t&, void*)'
76 |     { return ::operator new(__bytes, do_allocate(__bytes, __alignment)); }
|              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
: note: candidate: 'void* operator new(long unsigned int)'
: note:   candidate expects 1 argument, 2 provided
: note: candidate: 'void* operator new(long unsigned int, std::align_val_t)'
: note:   no known conversion for argument 2 from 'void*' to 'std::align_val_t'
make[2]: *** [/opt/devkitpro//devkitA64/base_rules:16: logger.o] Error 1
make[2]: Leaving directory '/home/zrm9901/Downloads/Dialga-master/build110'
make[1]: *** [MakefileNSO:136: build110] Error 2
make[1]: Leaving directory '/home/zrm9901/Downloads/Dialga-master'
make: *** [Makefile:18: starlight] Error 2
do you know how to fix this, Ive looked at all the issues in starlight and theere were no issues in the starlilght oddessy one so idk what to do. Thanks in advance if you see this!

just tried in wsl2 and got the same error


Better proxy support?

Hello,
Thanks for this awesome gem. I love python requests egg, and I am glad I am not alone in the Ruby community!
I wonder if there is any plan to support the http://user:password@ip:port format, proxy timeouts, and SSL proxies?
I am willing to dig into it if no one is already working on this.
Thanks!

We have generally decided to eschew supporting passwords in URIs, following suit from browsers and browser guidelines which deprecated this pattern:
https://developer.mozilla.org/en-US/docs/Web/HTTP/Authentication
See also: #477
I think there's a way to do SSL proxies already but offhand I don't see it. Perhaps @zanker knows?

We support CONNECT proxying, which only supports HTTP (to the proxy). I don't believe we support SOCKs proxies? It's been a while since I've messed with a SOCK proxy though.

Cool, I didn't knew it was deprecated, good to know.
Unrelated question, if we use multiple proxies to get the same page at the same time via threads, is there a way to cancel a processing requests in case one proxy is faster than the other?


Add a run_constrained for libopenblas on openblas.

To avoid libopenblas 0.3.6 and openblas 0.3.5 being installable in the same environment.

@isuruf we have:
  - name: openblas
    ...
    requirements:
      run:
        - {{ pin_subpackage("libopenblas", exact=True) }}

Isn't that enough? Or should we do:
  - name: openblas
    ...
    requirements:
      run:
        - {{ pin_subpackage("libopenblas", exact=True) }}
      run_constrained:
        - libopenblas =={{ version }}

Done in https://github.com/conda-forge/openblas-feedstock/pull/72


Reference Source hashes have changed, so links no longer work

This issue originally filed by @IanKemp in here https://github.com/dotnet/corefx/issues/20690. I am moving it to the right repo
Go to http://referencesource.microsoft.com/
Scroll down to the Link to a type/member text and click the link corresponding to that text: http://referencesource.microsoft.com/mscorlib/a.html#1f55292c3174123d
Notice that you are not directed to the appropriate line of code as you would expect. Instead, you are shown a page containing the text Don't use this page directly, pass #symbolId to get redirected.
This seems to have occurred because the file hashes (I assume) that are used for this functionality have changed. This affects Visual Studio extensions like Ref12.

While I'm utterly thrilled that this issue has finally been opened in the correct repo, I am less thrilled that, after two weeks of back-and-forth to achieve the aforementioned, the actual underlying issue remains unresolved.

I'm running into this issue for all interfaces.


feat(ListDecorator): Add setList() function. Useful for keeping a decorator/paginated list configuration intact while modifying the underlying list.

feat(ListDecorator): Add setList() function. Useful for keeping a decorator/paginated list configuration intact while modifying the underlying list.

Please use correct git tags in future. E.g. BUG <desc>, ENHANCEMENT <desc>, API <desc>.
type(class) isn't a bad standard, but it's not the one we'd like and our release notes generation doesn't support it sorry. =(

Noted. After I finish this PR: https://github.com/silverstripe/silverstripe-framework/pull/6472
I'll make sure to follow your standards.


Content block - mixed groups: Gray background behind some items

Viewed in Chrome
I am seeing gray backgrounds appearing behind some list items in Storybook.
Expected: All items should have a white background.
As seen in Storybook:

This should be retested after the dev refactoring is finished.


Add missing opening bracket in Datastore.java

if (userEntity == null) return null; would look cleaner.

I would suggest keep the brackets. It's easy to make mistakes without the brackets.
If you really want to make it cleaner, you can do:
return userEntity == null ? null : new User(email, (String) userEntity.getProperty("aboutMe"));


fix some misc

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

LGTM

others  LGTM


Using Marten's WriteTo throws swallowed exception when Wolverine tries to set empty status code

When using Marten's WriteTo with Wolverine, WriteEmptyBodyStatusCode generated code tries to set the status code to 204, after Marten has already started the response. This fail silently, and is only discoverable when reviewing logs.

Raised a PR with a possible fix, and a useless Unit Test.  Open to suggestions how to make the useless unit test actually test something


[fileNotFoundError: [Errno 2]] cdt.causality.graph.LiNGAM + No such file or directory: 'C:\anaconda\lib\site-packages\cdt\utils\R_templates\test_import.R'

Encountering a bug while using this package may occur. In order to fix the said bug and improve all users’ experience, it is highly recommended to submit a bug report on the GitHub issue tracker: https://github.com/FenTechSolutions/CausalDiscoveryToolbox/issues
Describe the bug
A clear and concise description of what the bug is.
Please mention
- cdt-0.5.23
- python3.7.12
- pytorch1.10.2
- gpu (availiable)
- The full traceback of the raised error if one is raised.
- A small code snippet to reproduce the bug if the description is not explicit.

Hello ! This issue might be linked to #128
I think finally it should be a windows compatibility issue... I'll look into this !
Best,
Diviyan


Keyboard shortcut conflict in FieldTextEditor

Pressing Meta + Enter submits the form AND also adds a new line in TextEditor.

:tada: This issue has been resolved in version 0.2.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Object type filter (or setting)

Related to #270, but not the same: allow setting/overriding the AP/AS object type, which also affects how platforms display posts.
E.g., I noticed the code looks for post formats (but only for the post type, though). That's good, but it could be expanded to other post types.
Still, relatively few sites use post formats anymore. (I normally don't. I do use CPTs, however. I could add post formats to them, but it still wouldn't work.)
A simple filter would probably be okay, to override whatever the plugin came up with.
This would also allow developers to map, e.g., "post kinds" (or other taxonomies) to object types.

I'm guessing apply_filters( 'activitypub_post', $array, $this->post ) will suffice, and it's already there. :-D


Cannot publish via neither UI nor API

Hi, I have a free account and I try to submit my first deviation.
Using UI: I upload to Stash without problems, click Submit Now button in DA, green Successful popup on the bottom right corner, then I'm immediately redirected to the 404 Oops page with URL 'https://www.deviantart.com/deviation/'.
So I tried via API, auth OK, simple entrypoints OK. Then I tried to publish the stashed image via /stash/publish endpoint. It returns success and I get the deviation UUID.
But even after several hours, the deviation does not appear in All gallery.
I expect I don't need to use galleryids in the /stash/publish call for the deviation to appear in All. If I need to fill it, what value it should be for All?
What am I doing wrong? And how do people submit any art via UI without 404?
Thanks!

what username are you posting from?

fixed, thanks


reopen files when renaming, reorganizing all files

Files that are renamed/reoganized and actually opened in editor are closed and renamed/reorganized files are opened afterwards.

@makuhn ,
thanks so much!  Really appreciate it!  Got it merged, and it's released in version 1.0.7!


Dungeon Mayhem: Make clever disguise more obvious when used

It's a bit hard to see at the moment

Made it more obvious now. If it's not obvious enough, too bad, it is a clever disguise after all.


Empty settings page for users with custom role having the manage_options capability

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the plugin and it is still present.

WordPress version
6.4.3
Cloudflare-WordPress version
4.12.6
PHP version
7.4.33
Expected result
A user with a custom role which has the manage_options capability, being able to purge the cache through the Settings → Cloudflare page.
Actual result
When clicking the "Cloudflare" admin menu item, an empty page is displayed and CONFIG_FETCH_ERROR and ZONES_FETCH_ERRORS errors occur in the console.
Steps to reproduce

Create a user with custom role, having the manage_options capability (e.g. using the Members plugin; https://wordpress.org/plugins/members/)
Visit Settings → Cloudflare

Additional factoids
It appears that the changes from https://github.com/cloudflare/Cloudflare-WordPress/pull/529 are causing the issues (released in version 4.12.3). The "Cloudflare" admin menu item requires the manage_options capability and the WordPress AJAX action cloudflare_proxy — which seems needed to load the settings page — is checking for the administrator role.
https://github.com/cloudflare/Cloudflare-WordPress/blob/dd13e1509194ee0a15c4f737082d39cdc226ad71/src/WordPress/Hooks.php#L82-L87
https://github.com/cloudflare/Cloudflare-WordPress/blob/dd13e1509194ee0a15c4f737082d39cdc226ad71/src/WordPress/Proxy.php#L56-L60
It might be better to check against the manage_options capability in the proxy too, so both will be checking the same requirement to access the settings page.

Also, as mentioned in the WordPress developer documentation at https://developer.wordpress.org/reference/functions/current_user_can/, checking against a role instead of a capability using current_user_can() is discouraged:

While checking against particular roles in place of a capability is supported in part, this practice is discouraged as it may produce unreliable results.

https://github.com/cloudflare/Cloudflare-WordPress/blob/dd13e1509194ee0a15c4f737082d39cdc226ad71/src/WordPress/WordPressAPI.php#L159-L165
https://github.com/cloudflare/Cloudflare-WordPress/blob/58db13b91fbd5e8613a8599d58cf05d04914d7e6/src/WordPress/WordPressWrapper.php#L39-L42
References
https://github.com/cloudflare/Cloudflare-WordPress/pull/529

Is there someone from the @Cloudflare team who can review the PR and merge it if possible?

As I have not heard anything since opening this issue in April, I reported it through Cloudflare support. The case ID is 01317769.


Generating words from trained RNN model: “Variable already exists, disallowed. Did you mean to set reuse=True in VarScope? ”

So I am trying to implemented this RNN word generator model in jupytor notebook. When I was trying to use the trained model to generate some words:
with open(os.path.join(cfgs['save_dir'], 'config.pkl'), 'rb') as f:
   saved_args = cPickle.load(f)

with open(os.path.join(cfgs['save_dir'], 'words_vocab.pkl'), 'rb') as f:
   words, vocab = cPickle.load(f)

with tf.Session() as sess:
   model = Model(saved_args, True)
   tf.global_variables_initializer().run()
   saver = tf.train.Saver(tf.global_variables())
   ckpt = tf.train.get_checkpoint_state(cfgs['save_dir'])
   if ckpt and ckpt.model_checkpoint_path:
       saver.restore(sess, ckpt.model_checkpoint_path)
       print(model.sample(sess, words, vocab, cfgs['n'], cfgs['prime'], cfgs['sample'], cfgs['pick'], cfgs['width']))

It works for the first time, but if I run the code again there is an error:
ValueError: Variable rnnlm/softmax_w already exists, disallowed. Did you mean to set reuse=True in VarScope?

Right now I have to shut down the ipynb file then run the code to get a new sample. Any idea to change the code to avoid this situation?

The problem is with anaconda as it is not clearing the cache immediately. Try using Pycharm it is working fine.

You could also try adding this line in the code tf.reset_default_graph() before the new session is created.


Failed to ssh login in the container.

I started the hongyizhao/deepin-wine:apricot docker image with x11docker with the bridge network mode, and try to login it with ssh. See the following info for the sshd running in the container:
werner@320a7a91cca4:~/Desktop$ ps -ef | grep sshd
root        2455       1  0 20:48 ?        00:00:00 /usr/sbin/sshd -D

But I don't know the password and I really don't set any password for the user. I want to know if x11docker sets the password when starting the container.
Regards,
HY

The password is x11docker, the same as with option --sudouser.

The password is x11docker, the same as with option --sudouser.

Thanks a lot. It does the trick. It seems that this is not explicitly stated in the documentation.

Thanks a lot. It does the trick. It seems that this is not explicitly stated in the documentation.

It seems that this is not explicitly stated in the documentation.

I've added a password note now in --help:
x11docker sets up an unprivileged container user with password 'x11docker'
and restricts container capabilities. Some applications might behave different
than with a regular 'docker run' command due to these security restrictions.
Achieve a less restricted setup with --cap-default, --sudouser or --user=root.

It seems that this is not explicitly stated in the documentation.

I've added a password note now in --help:
x11docker sets up an unprivileged container user with password 'x11docker'
and restricts container capabilities. Some applications might behave different
than with a regular 'docker run' command due to these security restrictions.
Achieve a less restricted setup with --cap-default, --sudouser or --user=root.


Support weakly typed data and search attributes

Basically skip the type checking for those cases to be more flexible. In some case that user want to implement an dynamic workflow that may need dynamic data or search attributes (especially with persistence locking)
Another idea is to let user define data/search attribute with type by prefix:
DataAttributeDef.createByPrefix("UserInteger-", Integer.class)

Also need https://github.com/indeedeng/iwf-java-sdk/issues/180

Data attributes should be easily supported.
But search attributes need to be registered into the namespace, how could it be dynamically created during running time?

To support the createByPrefix for data attributes, we could:

add a new filed isPrefix into DataAttributeDef.
in registry, add a new Map<String, Map<String, Class<?>>> dataAttributePrefixToTypeStore field to record the prefixes seperately, so that when validating the keys, the time complexity could be O(N), where N is the count of prefixes.

To support the createByPrefix for data attributes, we could:

add a new filed isPrefix into DataAttributeDef.
in registry, add a new Map<String, Map<String, Class<?>>> dataAttributePrefixToTypeStore field to record the prefixes seperately, so that when validating the keys, the time complexity could be O(N), where N is the count of prefixes.

@longquanzheng What do you think?

That looks good to me. I would probably do the same way as you described

Also I just realized that getAllDataObjects will be broken for now after this change. We may just add a comment for this behavior as edge case. (In fact, I think this API is not too important to have anyway)

Data attribute support in #189.


Error updating where-used list for CLAS ZCX_ADF_SERVICE_GRAPH

After a fresh setup of ABAPGit and the ABAP SDK activation failed with error message "Error updating where-used list for CLAS ZCX_ADF_SERVICE_GRAPH". Tested with S4 2021.

try triggering the syntax check from abapGit, does any of the classes contain syntax errors?

Syntax Check shows 10+ errors of types:

The final class "ZCX_ADF_SERVICE" cannot have any subclasses.
The REPORT/PROGRAM statement is missing, or the program type is INCLUDE.

yea...
suggest enabling https://rules.abaplint.org/superclass_final/

Hi @MartinPankraz,
We have moved latest changes pertaining to Microsoft graph. Could you please try overwriting graph via pull and let us know if you are still encountering this error i.e. 'Error updating where-used list for CLAS ZCX_ADF_SERVICE_GRAPH'.
Best Regards,
Sanju

Thanks for that @Sanjukum19! Looks better now. Failed on my end with different message:
_Generation ZREST_SCREEN terminated (see long text). Return code ZREST_SCREEN
Message no. EC801
Diagnosis
Termination when generating a status because there was no more storage space.
This means that either the parameters ZTTA/CUA_AREA and ZTTA/SHORT_AREA are not compatible or the user interface of the program is too big._

Hi @MartinPankraz,
Could you please verify the parameters' value in RZ11 transaction i.e., if ZTTA/SHORT_AREA is at 6 times higher as that of ZTTA/CUA_AREA. Here is the SAP note for the same: [https://me.sap.com/notes/0002491252]
Please let us know if the issue still exists.
Best Regards,
Sanju

Even raising to maximum for short_area doesn't change the activation bottleneck. Workaround is manual second wave of activation.

Will test with a CAL instance for another project the next 2 weeks. If the problem doesn't occur there we can close here.

Hi @MartinPankraz,
Thank you! Please let us know once you complete testing on S4 Hana appliance in CAL.
I too tried on ABAP on Hana appliance in CAL and did not encounter the above error 'ZREST_SCREEN terminated' while pulling. However, we could not confirm as the free credit got fully exhausted within a day before full testing could be completed.
Best Regards,
Sanju


<ScrollRestoration /> does not restore the scroll positions before DOM paint

Which project does this relate to?
Router
Describe the bug
It shows the top of the page then restore the scroll positions. This causes flickering.
Your Example Website or App
https://stackblitz.com/edit/tanstack-router-k6ncqr?file=src%2Fmain.tsx
Steps to Reproduce the Bug or Issue

At the / route (Home tab), scroll to the bottom of the home tab.
Click the About tab.
Click the Home tab.
I can see the red background of the Home tab title appear and then suddenly disappeared.

Expected behavior
No red background flickering
Screenshots or Videos
https://github.com/user-attachments/assets/22f05924-bb58-4894-9938-5a7903b51db4
Platform

OS: WInd
Browser: Edge
Version: 130.0.2849.46

Additional context
No response

I identified that the issue occurs because the onResolved event is triggered after the <HomeTab /> component has mounted. Since the <ScrollRestoration /> component performs scroll restoration when the onResolved event is emitted, it consistently restores the cached scroll position after <HomeTab /> becomes visible to the user.
I resolved this issue using a workaround. Here’s how I addressed it:

Removes the isTransitioning check in isAnyPending in Transitioner.tsx

With the above change, the onResolved event will be emitted before the useLayoutEffect in <Page /> component.

I understand that the above workaround is not a real solution to this issue. As isTransitioning is obviously useful, removing it might cause some serious bugs in the other places. Hope we can find a real solution to this problem.

I propose a solution which is to make the router emit a new event called onBeforePageMount.
The onBeforePageMount is designed to trigger actions that should occur just before a new page component mounts. This event allows developers to perform pre-render operations, such as scroll restoration, ensuring that these actions complete before the page content becomes visible to the user.


Run all extension-internal uses of Python "isolated".

For #10681.
This change updates all other internal uses of Python in the extension to run "isolated".
Note: This PR is based on #10941, so until that is merged please review only commits after the last commit on that PR).

Codecov Report

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

@@           Coverage Diff           @@
##           master   #10943   +/-   ##
=======================================
  Coverage   60.29%   60.29%
=======================================
  Files         580      580
  Lines       31815    31815
  Branches     4502     4502
=======================================
  Hits        19182    19182
  Misses      11662    11662
  Partials      971      971

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 460ce71...460ce71. Read the comment docs.


Refactor: Snippet Delete Modal

Implements #1607

Codecov Report

Merging #1608 (fb1b96c) into master (1e2dac5) will increase coverage by 0.72%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1608      +/-   ##
==========================================
+ Coverage   32.40%   33.12%   +0.72%
==========================================
  Files          52       52
  Lines         824      806      -18
  Branches       97       94       -3
==========================================
  Hits          267      267
+ Misses        468      451      -17
+ Partials       89       88       -1

Impacted Files
Coverage Δ

...nents/snippet-options/snippet-options.component.ts
24.13% <ø> (+9.24%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Do I actually need NVIDIA CUDA 12 rather than 11.8?

I just tried the instructions for implementing GPU Support e.g. installing NVIDIA CUDA Toolkit 11.8 and NVIDIA cuDNN 8.7.0 for CUDA 11.x, specifically, as per the readme (which involves some manual file moving around and PATH environment updating on Windows as per Nvidia's instructions at https://docs.nvidia.com/deeplearning/cudnn/installation/windows.html - restarted since then to ensure PATH updated properly everywhere and confirmed it can be found in the proper place when in my venv via which cublas64_11.dll (I use Git bash for my terminal in VSCode, but where via cmd works, too) and then the pytorch uninstall/reinstall pip commands specifying cuda 11.8
Then I tried to run a super simple test script:
from RealtimeSTT import AudioToTextRecorder

def process_text(text):
  print(text, end=" ", flush=True)

if __name__ == '__main__':
  with AudioToTextRecorder(
    spinner=False,
    model="tiny.en",
    language="en",
    # enable_realtime_transcription=True,
    realtime_model_type="tiny.en"
  ) as recorder:
    print("Say something...")
    while True:
      recorder.text(process_text)

But got this error:
Exception: Library cublas64_12.dll is not found or cannot be loaded

The stack trace stops in RealtimeSTT so I manually hunted around in the .venv files in vscode for a reference to cuda 12. The only reference matching a regex of cublas(.*)12 in ./.venv/* is the METADATA file for PyTorch i.e. .venv\Lib\site-packages\torch-2.2.2+cu118.dist-info\METADATA . Specifically:
Metadata-Version: 2.1
Name: torch
Version: 2.2.2+cu118
Summary: Tensors and Dynamic neural networks in Python with strong GPU acceleration
Home-page: https://pytorch.org/
Download-URL: https://github.com/pytorch/pytorch/tags
Author: PyTorch Team
Author-email<EMAIL_ADDRESS>License: BSD-3
Keywords: pytorch,machine learning
Classifier: ...
...

I'm fairly inexperienced poking around to resolve Python's specific dependency hell 😅 so I'm not sure if this represents the cause of the issue or not.
I tried the uninstall, then pip cache purge, and then the re-install in case a cahed wheel was the issue, but still have the problem.

Also ran into that once. You can just search for the cublas64_11.dll and create a copy in the same folder and name it cublas64_12.dll.  Thanks for pointing out the lib causing this.

Brill! Yeah, I'd hread about PyTorch dependency hell 🤣
Yep, that worked. Wow, that's lighting fast now 😄
faster whisper is happy with cuda 12 since v 1.0.0, so might be worth seeing if CUDA 12 will work and/or if it's time for a dependency uplift? 🤷♂
And thanks for this great repository 😊

RealtimeSTT will probably work with CUDA 12, so CUDA 11 is more a recommendation, because I personally can't test anything on 12. Lots of other libraries some of my projects depend on still need CUDA 11, so I can't upgrade.

There's are also new beam_size parameters, try model "distil-small.en" with beam_size_realtime 1 - that's incredibly fast.

Also ran into that once. You can just search for the cublas64_11.dll and create a copy in the same folder and name it cublas64_12.dll.  Thanks for pointing out the lib causing this.

You saved me bro, ur the goat