the score of problem may be surrounded by ( )

落ち
https://atcoder.jp/contests/wupc2019/tasks/wupc2019_a

配点 : \(100\) 点
 などと \( \) で囲まれてるのが原因。同様のものはこのコンテストだけ
$ oj dl -n https://atcoder.jp/contests/wupc2019/tasks/wupc2019_a
[x] problem recognized: AtCoderProblem.from_url('https://atcoder.jp/contests/wupc2019/tasks/wupc2019_a'): https://atcoder.jp/contests/wupc2019/tasks/wupc2019_a
[x] load cookie from: /home/user/.local/share/online-judge-tools/cookie.jar
[x] GET: https://atcoder.jp/contests/wupc2019/tasks/wupc2019_a
[x] 200 OK
Traceback (most recent call last):
  File "/home/user/.local/bin/oj", line 7, in <module>
    onlinejudge._implementation.main.main(args=sys.argv[1:])
  File "/home/user/.local/lib/python3.6/site-packages/onlinejudge/_implementation/main.py", line 279, in main
    run_program(namespace, parser=parser)
  File "/home/user/.local/lib/python3.6/site-packages/onlinejudge/_implementation/main.py", line 252, in run_program
    download(args)
  File "/home/user/.local/lib/python3.6/site-packages/onlinejudge/_implementation/command/download.py", line 52, in download
    samples = problem.download_sample_cases(session=sess)  # type: ignore
  File "/home/user/.local/lib/python3.6/site-packages/onlinejudge/service/atcoder.py", line 550, in download_sample_cases
    return self.download_content(session=session).sample_cases
  File "/home/user/.local/lib/python3.6/site-packages/onlinejudge/service/atcoder.py", line 543, in download_content
    self._cached_content = _AtCoderProblemContent_from_html(resp.content.decode(resp.encoding), problem=self)
  File "/home/user/.local/lib/python3.6/site-packages/onlinejudge/service/atcoder.py", line 507, in _AtCoderProblemContent_from_html
    score = _AtCoderProblemContent_parse_score(soup)
  File "/home/user/.local/lib/python3.6/site-packages/onlinejudge/service/atcoder.py", line 490, in _AtCoderProblemContent_parse_score
    return int(utils.remove_suffix(utils.remove_prefix(p.text, '配点 : '), ' 点'))
ValueError: invalid literal for int() with base 10: '\\(100\\)'


[FR] Change the links into "Internal Link" to support features like page preview

Could you please consider changing the links of Outlinks/backlinks into Obsidian's "internal link", so that some basic functions (like mouse hover link to preview) could be triggered?
https://github.com/user-attachments/assets/fbe15f1b-5b7a-4225-83a4-d64cdde7a876

I also tried this feature, but it is not working on my end, and I'm not sure why.

me too! And not to forget link to headings and blocks and also link display text! This would just awesome!

@Moyf @ksdavidc

Update the plugin in Obsidian 🎉

https://github.com/jparkerweb/rich-foot/releases/tag/1.8.0


TargetRpc with NetworkConnection Only Works for Observers

Describe the bug
TargetRpc with a NetworkConnection parameter fails if the target client isn't an observer of the calling object.
[IMPORTANT] How can we reproduce the issue, step by step:
Consider a player object with Network Proximity Checker and this script attached.
If 2 players are within range, the TargetRpc works, but if they're out of range, it fails.
Expected behavior
TargetRpc with a NetworkConnection parameter should always deliver.
Desktop (please complete the following information):

OS: Desktop
Build target: Desktop
Unity version: 2019.4 LTS
Mirror branch: master

In order to do this mirror would have to spawn the object on the client before the targetrpc call and then to not cause issues despawn it instantly, right?
That might not be the right thing to do in 90% of the cases

The assumption is the object is spawned on the target client, yes.  The example given is a player object.

The point was that TargetRpc with a conn param should bypass any filter for "observers".  If there's a spawned object instance for the indicated client connection, the msg should be sent.

@MrGadget1024 TargetRpc can only be applied to observers because otherwise the client wouldn't know who to applied to.
for example, if client sees only himself and gets a TargetRpc for a monster that is on the other side of the map.
the monster isn't spawned on that client.
so the client wouldn't know what to do with the rpc. there's no monster with that netId, there is no component with that rpc, etc.


View for JScholarship Redirects

This adds a View for exporting a file that is similar
to the format used by Apache's RewriteMap.
To access the endpoint for this view:
curl -X GET https://test.digital.library.jhu.edu/jscholarship_redirects_rest\?query=\(ss_type\:islandora_object\)\&items_per_page\=500\&offset\=0 > out.csv

That will only get the first 500 items. You'll want to use the offset parameter
to go through until you have exported metadata for all the items in the repo.
This would preferably be done in a script.
The resulting file will look like:
dspace_identifier "Citable URL"
https://jscholarship.library.jhu.edu/handle/1774.2/57496 https://test.digital.library.jhu.edu/node/5321

To use the resulting file as a RewriteMap the headers should be removed and the root URL
should be removed from the jscholarship URLS:
/handle/1774.2/57496 https://test.digital.library.jhu.edu/node/5321

Then in the JScholarship Apache config (after moving the file to /etc/httpd/conf/redirects.txt):
  RewriteEngine On
  RewriteMap redirects "txt:/etc/httpd/conf/redirects.txt"
  RewriteCond ${redirects:$1} !=""
  RewriteRule ^(.*)$ ${redirects:$1} [redirect=permanent,last]

When I ran the above curl statement against the cloud test instance I get records like this (with spaces as separator):
dspace_identifier "Citable URL"
https://jscholarship.library.jhu.edu/handle/1774.2/57496 https://test.digital.library.jhu.edu/node/5321
https://jscholarship.library.jhu.edu/handle/1774.2/55393 https://test.digital.library.jhu.edu/node/6472
https://jscholarship.library.jhu.edu/handle/1774.2/55389 https://test.digital.library.jhu.edu/node/6473
https://jscholarship.library.jhu.edu/handle/1774.2/49491 https://test.digital.library.jhu.edu/node/6474
https://jscholarship.library.jhu.edu/handle/1774.2/55376 https://test.digital.library.jhu.edu/node/6475
https://jscholarship.library.jhu.edu/handle/1774.2/49493 https://test.digital.library.jhu.edu/node/6476
https://jscholarship.library.jhu.edu/handle/1774.2/49495 https://test.digital.library.jhu.edu/node/6477
https://jscholarship.library.jhu.edu/handle/1774.2/49494 https://test.digital.library.jhu.edu/node/6478
https://jscholarship.library.jhu.edu/handle/1774.2/55387 https://test.digital.library.jhu.edu/node/6479

Running against my local setup with this PR and some test data I get this (with commas as separator):
dspace_identifier,"Citable URL"
,https://islandora-idc.traefik.me/node/45
,https://islandora-idc.traefik.me/node/44
http://jscholarship.library.jhu.edu,https://islandora-idc.traefik.me/node/38
http://jscholarship.library.jhu.edu,https://islandora-idc.traefik.me/node/39
http://jscholarship.library.jhu.edu,https://islandora-idc.traefik.me/node/40
http://jscholarship.library.jhu.edu,https://islandora-idc.traefik.me/node/41
http://jscholarship.library.jhu.edu,https://islandora-idc.traefik.me/node/42

I have a few items with no DSpace URl. You'll probably want to add a filter condition on the view to check the DSpace identifier - or maybe you have a different way of skipping the records that do no have DSpace URL's.
Looks good overall with a few minor things that should probably be changed.

@bseeger Thanks for catching the separator! I also added a filter for the DSpace identifier.

I can review this today again.  Note that the test failures are due to a testcafe upgrade and John created a PR to lock in a working version yesterday. https://github.com/jhu-idc/idc-isle-dc/pull/291

This probably resolves: https://github.com/jhu-idc/iDC-general/issues/450


Addition of CPlayer structure

Change Request

Description
Creating CPlayer class to associate functions of SAMP to them, this structure will ease out memory management, easy function access. This class should inherit the CEntity class.

Issue isn't closed yet, CPlayer still has few developments left.


Update requirements.txt

This layout provides a clear structure, separating the description section from the list of packages. It's easier to read and understand for anyone viewing the file on Git.

closing this because (probably) chat-gpt generated pull request without value


Add RedisInsight Docker Extension

https://github.com/collabnix/redisinsight-docker-extension

Added.


GH-849 Cache collection of FieldInfo for relationship fields

Cache the collection of FieldInfos for the relationshop fields as it is already done for the property fields.

Description

Caches the collection for FieldInfos for better performance when reading entities with @Relationship annotations from the database.
Related Issue

#849
Motivation and Context

This changes leads to a better performance when reading linked data from the database and mapping it to an entity model that uses @Relationship annotations
How Has This Been Tested?

Run the tests and also tested it in our software.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Thanks, Torsten, once again! I have seen this but it will have to wait a couple of days, but we will bring this in.

Thanks a lot. I took the freedom and polished the method itself a bit.


what diff with set sdk with session or global

by test，vofx use -g or -p both create a new bash process and add sdk path on PATH，i don't know what diff with them?

@yimiaoxiehou Please join Discord https://discord.gg/85c8ptYgb7 ;)

I told u before https://github.com/version-fox/version-fox-plugins/pull/11#issuecomment-1925684486, because of you did not configure the shell hook correctly. ;)


Optimize memory usage of delete_expired_checkouts task

:information_source: This is a 3.8 port of https://github.com/saleor/saleor/pull/11175
I want to merge this change because it optimizes memory usage of delete_expired_checkouts task.
Memory usage :chart_with_upwards_trend:
Before: 604MB (200k objects, no batching), exec time: 25.22s
After: 27MB (200k objects, 2k batches), exec time: 23.16s

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: CLOUD-2177 Fix delete_expired_checkouts task which executes long and consumes a lot of memory


worklog --comment flag do not work together with prompt

Describe the bug
Worklog --comment flag with prompt mode will not get the the comment updated. To reproduce
jira issue worklog add TEST --comment "Test"
? Time spent 15m
? What's next? Submit

Worklog added to issue "TEST"

However when checking the entry in jira the comment/description is missing . This is fine if with --no-input flags supplied
Please provide following details

JiraCLI Version:

(Version="v1.3.0", GitCommit="ed84fe74f68ba1b2cf0504a931ca09a03b2c9fed", CommitDate="2023-01-02T09:59:32+00:00", GoVersion="go1.19.4", Compiler="gc", Platform="linux/amd64")

Are you using Jira cloud or on-premise jira server?

on-premise
SERVER INFO
-----------

Version:         8.13.4
Build Number:    813004
Deployment Type: Server
Default Locale

What operating system are you using? Also mention version.
Host centos7.9 - docker container

What terminal are you using? Also mention version.
bash

To Reproduce
jira issue worklog add <ISSUE> --comment "Test comment"
Expected behavior

Screenshots

Additional context

Hi @sarathsprakash, this should be fixed with #562. Thank you for reporting!


Write Core Provenance Headers

Add functionality to write a standard set of core provenance headers to any file or dataset (see CTAO documentation on the header definitions).  This should support:

generating and validating a dictionary of core headers
function to write that dictionary to HDF5 dataset attributes and ECSV headers (at minimum), and JSON output

This should be part of the Provenance system, probably associated with the Activity, which means at least some of this should be defined for all Tools, and maybe even finer-grained in case a Tool writes many datasets.

fixed by #1221


New workflow for xcdat mean climate metrics

In this PR:

CDAT in mean climate was replaced by xCDAT.
The mean climate driver was re-written from scratch with using some of existing functions, with substantial re-organization of the directory for mean climate metrics.
Many of class-based objects that over-complicates code maintenance were deprecated (moved to 'deprecated' directory, and will be removed later).

This PR is intended to replace the previous PR, #884.

Comparison of metrics obtained from cdms-based and xcdat-based codes for pr global looks promising!

Test results for CMIP6 models proves great consistency between results from xcdat-based and cdms-based codes.

@acordonez I think this PR is ready for review. When you have a chance, could you take a look and also try run demo notebooks to ensure this change is not breaking the demo?

@gleckler1 once this PR is merged, the capability of getting annual cycle using xcdat is going to be applicable to the obs4mips workflow in more efficient way.


Deprecate Postgis in our database and API

In our API and in our database schema, we depend on Postgis.
This is adding complexity in developer setup for not much use.
We should aim to stop depending on it.

As part of https://github.com/opencollective/opencollective-api/pull/7457, we're moving the only column that depends on Postgis to JSONB thus removing the need for this dependency.
We'll need to have a second pass to update the docs and consolidate the migrations: if we just ask people to use postgres without postgis, their first npm run db:migrate will fail when trying to migrate the column.
To summarize, the next steps are:

[ ] Update test/dbdumps/opencollective_dvl.pgsql to make sure it includes all the current migrations
[ ] Move all migrations to migrations/archives
[ ] Remove all references to postgis from the code, CI config files and documentation

Depends on https://github.com/opencollective/opencollective/issues/5515

Postgis is gone! I'll keep an eye on our channels to look for regressions


flutter run and flutter apk fail on basic flutter created project

ProcessException: No such file or directory
Command:
/home/appname/dev/android-sdk-linux/build-tools/23.0.2/aapt package
-M android/AndroidManifest.xml
-A /tmp/flutter_tools7u0DqJ/assets
-I /home/appname/dev/android-sdk-linux/platforms/android-23/android.jar
-F /tmp/flutter_tools7u0DqJ/app.apk.unaligned
-S /home/appname/dev/bitbucket/appname-web/src/main/dart/appname-flutter/android/res /tmp/flutter_tools7u0DqJ/artifacts
dart:io                                                           Process.runSync
package:flutter_tools/src/base/process.dart 98                    _runWithLoggingSync
package:flutter_tools/src/base/process.dart 69                    runCheckedSync
package:flutter_tools/src/commands/apk.dart 100                   _ApkBuilder.package
package:flutter_tools/src/commands/apk.dart 286                   _buildApk
package:flutter_tools/src/commands/apk.dart 414                   buildAndroid.
dart:async                                                        _SyncCompleter.complete
package:flutter_tools/src/flx.dart 197                            buildInTempDir.
dart:async                                                        _SyncCompleter.complete
package:flutter_tools/src/flx.dart 295                            assemble.
dart:async                                                        _SyncCompleter.complete
package:flx/signing.dart 44                                       CipherParameters.seedRandom.
===== asynchronous gap ===========================
dart:async                                                        _Completer.completeError
package:flutter_tools/src/commands/apk.dart 423                   buildAndroid.
dart:async                                                        _SyncCompleter.complete
package:flutter_tools/src/flx.dart 197                            buildInTempDir.
dart:async                                                        _SyncCompleter.complete
package:flutter_tools/src/flx.dart 295                            assemble.
dart:async                                                        _SyncCompleter.complete
package:flx/signing.dart 44                                       CipherParameters.seedRandom.
===== asynchronous gap ===========================
dart:async                                                        _asyncThenWrapperHelper
package:flutter_tools/src/commands/apk.dart                       buildAndroid
package:flutter_tools/src/commands/apk.dart 451                   buildAll.
===== asynchronous gap ===========================
dart:async                                                        Future.Future.microtask
package:flutter_tools/src/commands/apk.dart                       buildAll
package:flutter_tools/src/commands/run.dart 158                   startApp.
===== asynchronous gap ===========================
dart:async                                                        Future.Future.microtask
package:flutter_tools/src/commands/run.dart                       startApp
package:flutter_tools/src/commands/run.dart 109                   RunCommand.runInProject.
dart:async                                                        _SyncCompleter.complete
package:flutter_tools/src/runner/flutter_command.dart 35          FlutterCommand.downloadToolchain.
dart:async                                                        _SyncCompleter.complete
package:flutter_tools/src/toolchain.dart 56                       Toolchain.forConfigs.
dart:async                                                        _SyncCompleter.complete
package:flutter_tools/src/toolchain.dart 44                       _getCompilerPath.
dart:async                                                        _SyncCompleter.complete
package:flutter_tools/src/artifacts.dart 214                      ArtifactStore.getPath.
===== asynchronous gap ===========================
dart:async                                                        _asyncThenWrapperHelper
package:flutter_tools/src/commands/run.dart                       RunCommand.runInProject
package:flutter_tools/src/runner/flutter_command.dart 87          FlutterCommand.run.
dart:async                                                        _SyncCompleter.complete
package:flutter_tools/src/runner/flutter_command.dart 133         FlutterCommand._getDevicesForCommand.
dart:async                                                        _SyncCompleter.complete
package:flutter_tools/src/device.dart 60                          DeviceManager.getAllConnectedDevices.
===== asynchronous gap ===========================
dart:async                                                        _asyncThenWrapperHelper
package:flutter_tools/src/runner/flutter_command.dart             FlutterCommand.run
package:flutter_tools/src/commands/run.dart 86                    RunCommand.run.
dart:async                                                        _SyncCompleter.complete
package:flutter_tools/src/dart/pub.dart 43                        pubGet.
===== asynchronous gap ===========================
dart:async                                                        _asyncThenWrapperHelper
package:flutter_tools/src/commands/run.dart                       RunCommand.run
package:args/command_runner.dart 176                              CommandRunner.runCommand.
dart:async                                                        Future.Future.sync
package:args/command_runner.dart 129                              CommandRunner.runCommand
package:flutter_tools/src/runner/flutter_command_runner.dart 201  FlutterCommandRunner.runCommand
package:args/command_runner.dart 103                              CommandRunner.run.
dart:async                                                        Future.Future.sync
package:args/command_runner.dart 103                              CommandRunner.run
package:flutter_tools/executable.dart 72                          main...
===== asynchronous gap ===========================
package:stack_trace                                               Chain.capture
package:flutter_tools/executable.dart 65                          main.

flutter doctor
Flutter root: /home/appname/dev/github/flutter.
[✓] Android toolchain - develop for Android devices (installed)
[✓] Android SDK - enable development for Android devices (installed)
[✓] Atom development environment - a lightweight development environment for Flutter (installed)
[✓] Atom editor (installed)
[✓] Flutter plugin - adds Flutter specific functionality to Atom (installed)
Flutter from https://github.com/flutter/flutter.git (on alpha)
flutter revision: 4860e63f444cf3786a6b136430f3058a7b851041 (12 hours ago)
engine revision : a53faaf7983025f83a2059021a409c6e98beb58d

This is on a brand new project generated by flutter create.  flutter devices sees my phone.  flutter run or flutter apk from command line, same error. flutter build seems to work.
Ubuntu/VirtualBox
I checked the existence of all the fixed files.  Seems to be related to one of the temp paths.  Any ideas?

I wonder if this could be https://github.com/flutter/flutter/issues/2076 ?

Yup!  The following fixed it:
sudo dpkg --add-architecture i386
sudo apt-get -qqy update
sudo apt-get -qqy install libncurses5:i386 libstdc++6:i386 zlib1g:i386

OK.  Will close this as a duplicate of #2076.


[Feature]: add ctb

Guidelines

[X] I agree to follow this project's Contributing Guidelines.

Description
person(given = "Philipp",
family = "Baumann",
email =<EMAIL_ADDRESS>role = c("ctb"),
comment = c(ORCID = "0000-0002-3194-8975")),
Problem

there is no problem

Proposed Solution
you can add me as ctb :-) cheers
Alternatives Considered
no

Sounds sensible to me! Yet, if you plan to submit your package to CRAN anytime soon, it might be better if we just import your package rather than forking some of your functions. This would also simplify downstream maintenance in the long run. In case submitting to CRAN is not an option for you, feel free to submit a PR here if you like your code change/improved.

I suggest you keep the copy of the function like this. I plan to submit https://github.com/spectral-cockpit/opusreader2 to CRAN only, since the heurististics used in the succcessor is not bullet proof. But let's discuss about integrating opusreader2 😎

you may want to add Pierre Roudier, too @pierreroudier

agree about CRAN, valid point! Let's maybe discuss in person, we can also have a call since we are on the way to anyway submit opusreader2 to CRAN, and also we have some neat downstream utilities regarding workflow and metadata management. Currently, I am checking out some funding to allow me to work on it, be it either private labs or larger research collaborations. :-)

Great! Let's keep the fork then until opusreader2 is on CRAN. Let me know if I can help with anything. Should we arrange a call date via email? @wincowgerDEV would you like to join?

Would love to join! Next week Wednesday onward is best for me.
On Thu, Sep 21, 2023, 2:36 AM Philipp Baumann @.***>
wrote:

Great! Let's keep the fork then until opusreader2 is on CRAN. Let me know
if I can help with anything. Should we arrange a call date via email?
@wincowgerDEV https://github.com/wincowgerDEV would you like to join?
Hi @zsteinmetz https://github.com/zsteinmetz and @wincowgerDEV
https://github.com/wincowgerDEV that sounds excellent. I will email you
later today and we can arrange :-) next week starting from Tuesday works
well for me. Talk soon.
—
Reply to this email directly, view it on GitHub
https://github.com/wincowgerDEV/OpenSpecy-package/issues/144#issuecomment-1729214082,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGMUJU37TWBLEVG724E4J7TX3QDCBANCNFSM6AAAAAA45GWDVI
.
You are receiving this because you were mentioned.Message ID:
@.***>

@philipp-baumann and @pierreroudier, got you added to the contributors now, you'll be listed on the next release.


Update minLength.json

Updates the language for the test that verifies that one grapheme is too short when checked against minLength of 2

Also ref: https://json-schema.slack.com/archives/CT8QRGTK5/p1701324578397949

Looks good to me too, just change it in the other drafts too

change it in the other drafts too

Oh! Please also check for maxLength tests as well.

Maybe we should pick some characters that actually make sense -- as far as I can tell "\uD83D\uDCA9" isn't valid utf8.
I did some digging in the history and the test was added here: #52 - and indeed this is UTF-16, not UTF-8. we should fix that.

No, JSON requires that it be broken out into the surrogate pair.

@spacether Just wanted to make sure you saw this request to add this change to other drafts as well.


Feature/ffmpeg from google

Issue:
FFmpeg linux version failed to download or download corrupted file or python can't handle with archive type.
Solved:
FFmpeg is prepared as zip package on pype's google drive containing ffmpeg for all OS. File is publicly accessible so anyone can download it without autemtication.
Testing:
Run pype deploy and see.

Moved to https://github.com/pypeclub/pype-setup/pull/22#discussion_r423871347


Question: How is the decompression handled?

Hello everyone,
I have a question regarding how decompression is handled with mitmproxy. I am trying to read some https requests, and I just see a mess.
The HTTP Response includes:
Content-Type: application/octet-stream
Accept-Encoding: gzip
In order to read the http body do I have to decompress the http body by myself or is it already decompressed but just maybe encrypted or compressed from the application?

I am trying to read some https requests, and I just see a mess.

Please elaborate how you are currently reading them. In the mitmproxy UI? Using an addon?
gzip should be handled transparently, e.g. in the mitmproxy UI it says "[decoded gzip]".
What type of response are you expecting? application/octet-stream suggests that it might be a binary format and judging from your question you don't know which format?

The HTTP Response includes:
Content-Type: application/octet-stream
Accept-Encoding: gzip

I hadn't read that part carefully. Are you sure this is a response? Accept-Encoding is for requests.


Modified export from Haag-Streit EyeSuite software.

When cooperating with another university hospital, I found that the way the export of the csv-files from EyeSuite that I suggested is not available everywhere. Therefore, I have added an export template which is installed with the package (inst/octopus_template.txt). I have also changed the hyperlink in the help file to a link where I can modify the contents (that is not the case for the RPub repository).

Thanks. I have tried to merge but it says merging is blocked. Could you do that? I am not sure exactly what is required.

Hi,
could you approve the pull request again?
It seems to be a new safety precaution. All commits that are merged to the master branch must be signed with a key. I have added a signed tag to the commits, so they should be verified now. However, I cannot find a way to continue the merge.
Maybe if you approve again?
 Maybe we could have a zoom meeting? It is not urgent, however.
Cord

Am 04.10.2023 um 17:25 schrieb Iván Marín-Franch @.***>:
Thanks. I have tried to merge but it says merging is blocked. Could you do that? I am not sure exactly what is required.
—
Reply to this email directly, view it on GitHub https://github.com/imarinfr/vf1/pull/9#issuecomment-1747031108, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADSN4J6TGB3CHGLFOTWGFCLX5V5XJAVCNFSM6AAAAAA5SQURL2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONBXGAZTCMJQHA.
You are receiving this because you authored the thread.


Feat: script for converting N5 datasets to OME-Zarr

The script takes a BigDataViewer-style multiscale N5 (on disk or cloud) and converts it to OME-Zarr. If there are fewer scales in the N5 than desired, the pyramid is generated from scratch.
Benchmarks
In this case all scales exist in the N5, so no downsampling was done.
Source N5 (multiscale, GZip compressed): s3://aind-open-data/exaSPIM_650010_2023-04-04_17-42-51_fusion_2023-04-23_11-35-00/fused.n5
Dest Zarr (multiscale, Blosc Zstd compressed): another S3 bucket
Machine: Allen HPC
N Cores: 64
Raw data size (all pyramid levels): 89187 GiB
Conversion time: 65563s
Speed: 1.36GiB/s
N5 chunk size: (256, 256, 256)
Zarr chunk size: (256, 256, 256)

@camilolaiton yeah, I was thinking about doing that but it's possible the downsampling method could be different, and since it's not possible to always know how the initial pyramid was generated I figured it was safer to do it from scratch.

Ohh I see, the fact that we don't have access to the downsampling metadata is inconvenient. Are we including those in our pyramids?
Not sure if this would help, but I am adding it to the OMEZarr .zattrs file:

I'll be updating the OMEZarr writer to something similar to what you have, that's why I am wondering.

I agree it should be included in the output zarr metadata when we do the downsampling from scratch, I'll update this PR to include that. To my knowledge, that information isn't currently included with N5 pyramids created with BigStitcher or N5-Spark, and there isn't a standard for representing that information in the attributes.json, so it could be non-trivial to have a general solution for computing based on previous levels (especially for N5 datasets created by others).


Jenkins

@ubrals Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

Thanks for the suggestion. We don't need a Jenkinsfile in a getting started guide though.


Support Enum

Close #149
Orig:
#[savvy]
enum Foo {
    A,
    B,
}

Expanded:
pub enum Foo {
    A = 0,
    B = 1,
}

impl TryFrom<Foo> for savvy::Sexp {
    type Error = savvy::Error;
    fn try_from(value: Foo) -> savvy::Result<Self> {
        (value as i32).try_into()
    }
}

impl TryFrom<savvy::Sexp> for Foo {
    type Error = savvy::Error;
    fn try_from(value: savvy::Sexp) -> savvy::Result<Self> {
        let i = <i32>::try_from(value)?;
        match i {
            0 => Ok(Foo::A),
            1 => Ok(Foo::B),
            _ => Err("Unexpected enum variant".into()),
        }
    }
}

C:
(No C code)
R:
.savvy_wrap_Foo <- function(ptr) {
  e <- new.env(parent = emptyenv())
  e$.ptr <- ptr

  class(e) <- "Foo"
  e
}

Foo <- new.env(parent = emptyenv())
Foo$A <- .savvy_wrap_Foo(0L)
Foo$B <- .savvy_wrap_Foo(1L)

Note to future self:
I'm giving up this approach of treating an enum as integer. It works perfectly with the case of T, but when it's &T or &mut T, it's complicated and it seems it's hard to handle enum and struct transparently.


In Ivy, ng-container is preventing content children from being queried

Describe the bug
Previously, I was able to query content children that were nested inside ng-container. For example:
<hello-container>
    <ng-container>
        <hello [name]="'Tom'"></hello>
        <hello [name]="'Jerry'"></hello>
    </ng-container>
</hello-container>

@Component({
  selector: 'hello-container',
  template: `<ng-content></ng-content>`,
})
export class HelloContainerComponent  {
  @ContentChildren(HelloComponent) hellos: QueryList<HelloComponent>;

  ngAfterContentInit() {
    console.log(this.hellos.length);
  }
}

This code snippet above would log 2 like here reproduced in Angular 8: https://stackblitz.com/edit/ngfor-ngcontainer-ngif.
But in Ivy, it would log 0 as it's not querying/registering content children if they are nested with ng-container.
How to reproduce
We just need to enable ivy.
Reproduction in Ivy: https://ivy.ng-run.com/edit/83UoqeeHqo08KIenCTIM
Expected behavior
Should be able to register content children like it does in Angular 8.
Versions

Angular: 9.0.0-rc.5

Additional notes
I tried it with ngProjectAs as shown below:
<hello-container>
    <ng-container ngProjectAs="hello">
        <hello [name]="'Tom'"></hello>
        <hello [name]="'Jerry'"></hello>
    </ng-container>
</hello-container>

Unfortunately, this didn't help solve the issue either.

Oops I didn't mean to post it here.


Support custom plugin when clone material

If PBRMaterial or StandardMaterial has custom plugins, and when clone material, an error will be thrown.
Playground below can reproduce the issue
https://www.babylonjs-playground.com/#X55BQW
The reason for this problem is that the name of custom plugin doesnt start with BABYLON.,
so material._parsePlugins doesnt instantiate the custom plugin for cloned material.

and then in material._clonePlugins, pluginManager!.getPlugin(plugin.name) will return null

and because _clonePlugins doesnt support custom plugin, so add material.onClonedObservable, then
the user can decides how to clone a custom material plugin when clone event is triggered.
such as:
originMaterial.colorify = new ColorifyPluginMaterial(originMaterial);
originMaterial.colorify.color = BABYLON.Color3.Red();

// clone custom plugin
originMaterial.onClonedObservable.add((cloned) => {
       cloned.colorify = new ColorifyPluginMaterial(cloned);
       cloned.colorify.color = originMaterial.colorify.color.clone();
});

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14662/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14662/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14662/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14662/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14662/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14662/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14662/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14662/merge#BCU1XR#0

Let me close for now while I ll push a fix for the plugins.
About the observable I am not sure we should add it, may I ask in which use case you require it ?

@sebavan  the observable is used for helping clone custom plugin, associating to some material instance
for example：
the general usage of material plugin is as follows
// ColorifyMaterialPlugin.ts
class ColorifyMaterialPlugin extends MaterialPluginBase {
 color: BABYLON.Color3;
 //...
}

BABYLON.RegisterMaterialPlugin("Colorify", (material) => {
  material.colorify = new ColorifyMaterialPlugin(material);
});

If I want to clone a material and the material plugins associated with it, I can only write as follows
// module_need_clone_material.ts
const cloned = material.clone();
cloned.colorify.color = material.colorify.color;

On the one hand, as the number of material plugins increases, the code of material cloning above becomes more and more complex.
On the another hand,  I don't want to couple the code of material cloning with the implementation of various material plugins.
With onClonedobservable, I can put together code related to certain material plugin and handle cloning of a certain material plugin in one place, such as follows
 // ColorifyMaterialPlugin.ts
BABYLON.RegisterMaterialPlugin("Colorify", (material) => {
 material.colorify = new ColorifyMaterialPlugin(material);
 material.onClonedObservable.add((cloned) => {
     cloned.colorify.color = material.colorify.color;
 });
});

 // AnotherMaterialPlugin.ts
BABYLON.RegisterMaterialPlugin("Another", (material) => {
 material.another = new AnotherMaterialPlugin(material);
 material.onClonedObservable.add((cloned) => {
     cloned.another.prop1 = material.another.prop1;
     cloned.another.prop2 = material.another.prop2;
     cloned.another.prop3 = material.another.prop3;
 });
});

// module_need_clone_material.ts
const cloned = material.clone();

This is not the right approach, the plugin should be able to clone themselves without having to deal with it manually cc @Popov72  to see how we are handling it (if we do) at the moment

You must register your plugin class in the BABYLON class store by doing BABYLON.RegisterClass("BABYLON.PLUGINNAME", PluginClass);, with PLUGINNAME the name of your plugin (as returned by getClassName()) and PluginClass the name of your plugin class.
For eg:
https://playground.babylonjs.com/#22HT5Z#106
If you comment line 86, you will see you get a warning in the console log and the material of the ground won't be ok.

@Popov72 Thanks, I will try it.


[PoC] add executed sql to trace event in mysql datastore

Not intended to merge: just a placeholder for hacking executed SQL into the mysql datastore to help understand what actually happens during check requests. Perhaps useful to others trying to understand what check does. Perhaps useful to enable this behaviour behind a flag.
Eg:

Running with mysql and jaeger based on this example:
---
# This runs SpiceDB, using MySQL as the storage engine. SpiceDB will not have
# any schema written to it.
#
# Once the database service is running, the migrate service executes, running
# "spicedb migrate head" to migrate MySQL to the most current revision. After
# MySQL is migrated, the migrate service will stop.
#
# SpiceDB settings:
#   pre-shared key: foobar
#   dashboard address: http://localhost:8080
#   metrics address: http://localhost:9090
#   grpc address: http://localhost:50051
#
# MySQL settings:
#   user: root
#   password: secret
#   port: 3306

version: "3"

services:
  spicedb:
    # local build with tracing hack to include sql
    image: "7605cb00063266272110179212410142fce9736d428a6b2da3f9f5e15c0479c8"
    # image: "authzed/spicedb:v1.16.2"
    command: "serve --otel-sample-ratio=1.0"
    restart: "always"
    ports:
      - "8080:8080"
      - "9090:9090"
      - "50051:50051"
    environment:
      - "SPICEDB_GRPC_PRESHARED_KEY=insecure-shared-key"
      - "SPICEDB_DATASTORE_ENGINE=mysql"
      - "SPICEDB_DATASTORE_CONN_URI=root:secret@tcp(database:3306)/spicedb?parseTime=true"
      - "SPICEDB_EXPERIMENT_ENABLE_CAVEATS=true"
      - "SPICEDB_LOG_LEVEL=trace"
      - "OTEL_EXPORTER_OTLP_ENDPOINT=http://jaeger:4317"
      - "OTEL_SAMPLE_RATIO=1.0"
      - "SPICEDB_OTEL_PROVIDER=otlpgrpc"
    depends_on:
      - "migrate"
      - "jaeger"

  migrate:
    image: "authzed/spicedb"
    command: "migrate head"
    restart: "on-failure"
    environment:
      - "SPICEDB_DATASTORE_ENGINE=mysql"
      - "SPICEDB_DATASTORE_CONN_URI=root:secret@tcp(database:3306)/spicedb?parseTime=true"
    depends_on:
      - "database"

  database:
    image: "mysql"
    # enable sql logging
    # https://stackoverflow.com/questions/32938120/how-to-log-queries-to-stdout-on-mysql
    command:
      - /usr/local/bin/mysqld.sh
    ports:
      - "3306:3306"
    environment:
      - "MYSQL_ROOT_PASSWORD=secret"
      - "MYSQL_DATABASE=spicedb"
    volumes:
      # enabling logging, can be commented out
      - ./my.cnf:/etc/mysql/conf.d/my.cnf:ro
      - ./mysqld.sh:/usr/local/bin/mysqld.sh:ro

  jaeger:
    image: "jaegertracing/all-in-one:1.41"
    environment:
      - "COLLECTOR_ZIPKIN_HOST_PORT=:9411"
      - "COLLECTOR_OTLP_ENABLED=true"
    ports:
      # see https://www.jaegertracing.io/docs/1.42/getting-started/#all-in-one
      - "6831:6831/udp"
      - "6832:6832/udp"
      - "5778:5778"
      - "16686:16686" # jaeger web interface http://localhost:16686/
      - "4317:4317"
      - "4318:4318"
      - "14250:14250"
      - "14268:14268"
      - "14269:14269"
      - "9411:9411"

@wcurrie Do you think this is worth merging behind a flag?

@wcurrie Do you want to put this behind a flag or close out the PR?

@wcurrie Closing out; feel free to reopen if you'd like to rebase and continue working on this


chore: update sources type for vite 5

Description
In Vite 5, the map type is SourceMap | { mappings: '' } and the new { mappings: '' } type trips up TypeScript. This PR handles the typecheck fail from vite-ecosystem-ci

Disable whitespace when reviewing the git diff, as I'm only wrapping if ('sources' in map) { ... } between the affected code.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. If the feature is substantial or introduces breaking changes without a discussion, PR might be closed.
[ ] Ideally, include a test that fails without this PR but passes with it.
[ ] Please, don't make changes to pnpm-lock.yaml unless you introduce a new test example.

Tests

[ ] Run the tests with pnpm test:ci.

Documentation

[ ] If you introduce new functionality, document it. You can run documentation with pnpm run docs command.

Changesets

[ ] Changes in changelog are generated from PR name. Please, make sure that it explains your changes in an understandable manner. Please, prefix changeset messages with feat:, fix:, perf:, docs:, or chore:.

Is there a reason why it returns such a weird type? Looks unintentional to me to be honest.

That's the type Rollup documents here and some plugins during the transform hook can return that, which in turn affects transformRequest's return type.
An empty map like { mappings: '' } could happen in this Vite loadAndTransform flow.


inventory: Replace 'softlayer' machines with 'ibmcloud'

The SoftLayer branding is no longer used on IBM cloud. This PR makes two changes:

Changes the softlayer provider name to ibmcloud
Replaces all of our -softlayer- machines in the inventory with other ones with -ibmcloud- names that we have obtained and tested (This is independent of the rename but we had to do it for other reasons)

Signed-off-by: Stewart X Addison<EMAIL_ADDRESS>

Resolved. Merging


Add support for Solar/Powerwall

Is your feature request related to a problem? Please describe.
This feature request is not related to a problem, just a request for additional functionality.
Describe the solution you'd like
Would like the TeslaMate solution to support Tesla Solar and Powerwall integration. Provide the ability to query the Tesla API related to Solar and Powerwall features. This will allow for graphing of Solar and Powerwall related data graphing.
Describe alternatives you've considered
N/A
Additional context
N/A

Adding some resources:
Tesla Powerwall Local Gateway API Documentation:
https://github.com/vloschiavo/powerwall2
Tesla API Commands:
https://www.teslaapi.io/powerwalls/commands

@knewmania I would love to see this included as well.  TeslaMate is the best!


Feature upgrade to ES6

Thanks for working on this! I have a couple of observations so far:

I noticed that there is a dependency on webpack, but I don't see a config for it, nor do I see it referenced in any of the npm scripts. I'm assuming this is residual code?

Do we need the gulpfile anymore now that you added npm scripts?

Since the source code is not being transpiled by babel, I think it would be a good idea to specify the minimal version of node required by setting the "engine" field in package.json. For example, in rancher.js the let keyword is used, which is unsupported in versions like 4.4 LTS

@EvanSnapp There still seem to be packages such as babel and eslint that are in the dependencies section. Also, I'm getting errors when installing dependencies from a blank slate due to an unmet peer dependency for a module that doesn't seem to be used (babel-loader)
Any packages that are not required by the service itself (testing, etc) should be in devDependencies. Additionally, I think the Dockerfile should be updated so dev dependencies are not pulled into the image. This will keep it small.

I fixed those things in the package.json

thanks, I'll try and put some eyes on it again tonight 👍

@EvanSnapp - thanks for all the work you continue to put into this. Please see the notes about the typos and the question about introducing a second promise library. We can keep it as it is for now, if needed but I would like to see one of three things:

move to native promises only
move everything to when APIs
move everything to bluebird APIs
Being on Node 6 gives us options but I'd like to pick one of those and stick with it.
Thanks!

I moved the app from when to bluebird in all places except for db.js. I was using bluebird because of the coroutine feature it supports and how it greatly simplifies the program flow. I left when in db.js because db.js utilizes when/node of which there did not appear to be a direct equivalent in bluebird. It seemed like for program simplicity and readability it might be best to leave it in place. please let me know if you would like a different solution.

I agree with @arobson that we should only use one promise library. For the most part, they generally play nice with each other, but sometimes you can run into inconsistencies.
@EvanSnapp It looks like the usage of when in db.js is just converting callback based nedb functions to promises. This can probably be accomplished by using the promisify functions of in Bluebird.

Ohh yeah! Thanks! I have fully removed when now.

@EvanSnapp - I found one additional type but other than that, this is great.


Adding License and Contribution guidelines to Readme.

Pull Request Test Coverage Report for Build 1939

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

Totals

Change from base Build 1927:
0.0%

Covered Lines:
7022

Relevant Lines:
11168

💛  - Coveralls


[Bug] The KeyboardEvent.getModifierState function can be undefined.

We have been encountering a type error TypeError: t.getModifierState is not a function
Steps to Reproduce:
Do not have a way to reproduce the issue but my hunch lies with a autofill event that does not have the getModifierState gets triggered. Noticed in monaco-editor library but the file associated with the issue is in this library.

@samirzubi-db What browser do you use? According to https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent/getModifierState#browser_compatibility this should be supported since a couple of years everywhere

Closing since we could not reproduce.

@alexdima I have a reproduction for this issue in the monaco-editor
See https://github.com/microsoft/monaco-editor/issues/4325
It occurs in Edge and Chrome.

Send your details for help
On Mon, Jan 29, 2024, 5:50 PM justin-mellor @.***>
wrote:

@alexdima https://github.com/alexdima I have a reproduction for this
issue in the monaco-editor
See microsoft/monaco-editor#4325
https://github.com/microsoft/monaco-editor/issues/4325
It occurs in Edge and Chrome.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/198308#issuecomment-1915132854,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFT2KGQEVPLLFYUTXBGOSEDYQ7HPHAVCNFSM6AAAAAA7MRHQR2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJVGEZTEOBVGQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

This can be reproduces easily


fix flaky tests against time

Motivation
fix flaky tests
tested with
go clean -testcache && go test ./activation/ -v -run "^TestBuilder_waitForFirstATX" -count 2000
go clean -testcache && go test ./timesync/ -v -run "^TestClock_" -count 1000

bors try

bors try

bors merge

bors merge


[Enh]: Add Support For PySpark

We would like to learn about your use case. For example, if this feature is needed  to adopt Narwhals in an open source project, could you please enter the link to it below?
No response
Please describe the purpose of the new feature or describe the problem to solve.
PySpark is one of the most used dataframe processing frameworks in the big data space. There are whole companies built around it and it is commonplace in the Data Engineering realm.
One common pain point is that data scientists usually works with Pandas (or more recently Polars) and when integrating their code in big ETL processes, that code is usually converted to PySpark for efficiency and scalability.
I believe that is precisely the problem Narwhals tries to solve and it would be a great addition to the data ecosystem to support PySpark.
Suggest a solution if possible.
PySpark has two distinct APIs:

PySpark SQL - Identical API to Scala version of Spark
PySpark Pandas - Subset of Pandas API (Previously known as Koalas)

Given that PySpark Pandas has an API based on Pandas, I believe it should be relatively straightforward to re-use the code already written for the Pandas backend.
There is a PySpark SQL to PySpark Pandas conversion so in theory it should be possible to also add ad hoc support for PySpark SQL Dataframes and check the overhead. If it is too big it can be considered to also add a separate backend for that different API.
If you have tried alternatives, please describe them below.
No response
Additional information that may help us understand your needs.
I do have experience with the PySpark API and would like to contribute, I read the "How it works" section but would like some concrete direction on how to get started and if this is of interest to the maintainers

Thanks @ELC for your request! Yup, this definitely in scope and of interest!

I was literally coming on this channel to ask the same question - love it!!

Fantastic, thank you! I did use pyspark in a project back in 2019, but I think I've forgotten most of it by now 😄 From what I remember, it's a bit difficult to set up? Is there an easy way to set it up locally so we use it for testing to check that things work?

PySpark Pandas

It might be easiest to just start with this to be honest. Then, once we've got it working, we can remove a layer of indirection
In terms of contributing, I think if you take a look at narwhals._pandas_like, that's where the implementation for the pandas APIs is. There's a _implementation field which keeps track of which pandas-like library it is (cudf, modin, pandas). Maybe it's as simple as just adding pyspark to the list?

I had experience working with Azure Pipeline agents which I believe are the same VM Agents running on Github actions and they come with all relevant Java dependencies pre-installed so having the test run on the CICD should not be a problem.
As per local development, there are a couple of options:

Setting PySpark locally - Really troublesome
Using VS Code with Dev Containers - Requires docker desktop installation
Using Github's workspaces with a PySpark image - Easiest but relies on a working internet connection

For this contribution I will go with the third option as it is the fastest and easiest to set up. If you would like me to set up the necessary files for the second one, I can do that two on a separate issue
I will have a look at the _pandas_like and _implementation files to have a look at what's needed and will keep you posted on the progress.

Hey folks, I have been working on this feature on a local branch.
I have a few questions:

What is the best way to gradually develop and implement this?

Adding support for a whole DataFrame library is quite a big feature, and it is a little all or nothing. I think it would be useful if any changes/choices being made are picked up early on.
Have we considered the following for pyspark pandas?

pyspark pandas converts to pyarrow, then pandas. There is quite often type confusion somewhere in the process. There is also a significant, but often overlooked, overhead of these conversions. Performance is generally much better with straight pyspark.
pyspark pandas doesn't work well with the latest version of pandas, it is particularly unstable with pandas >2. I have had to limit the pandas version in dev-requirements in my local branch to pandas<2.1.0 because pandas pyspark still uses this deprecation.

The local branch I have been working on uses straight pyspark.
It would also be possible to add pyspark pandas as it's own pandas-like implementation AND the straight pyspark API and give users the choice.
This is perhaps something to be explored in a later feature, but if we add support for the pyspark API we could possibly have a root to much more library support through sqlframe. I think it is worth preferring pyspark over the pyspark pandas API.
I am not developing using pyspark pandas, but I still had to limit my pandas version to <2.1.0 on my local branch. Why is this?
Following the lead from dask lazyframe, I have been implementing the collect method as returing a pandas pandaslike DataFrame. This then needs to still use pandas pyspark, but this then has the deprecation issue >=2.1.0
I think there might be a better way. The Pyspark 4.0.0 preview adds a toArrow feature. The collect -> eager implementation can then return an arrow DataFrame. There is only one conversion here (pyspark -> arrow) rather than three (pyspark -> pandas arrow). Ritchie Vink has made a related comment about the two conversion being preferable in the past, back when there was no official way of going to arrow from spark. The challenge here is then this would make at least the eager implementation in narwhals incompatible with pyspark<4.0.0 (when spark 4.0.0 is actually released). Any thoughts on this?

hey @ELC I usually use SDKMAN when developing pyspark locally and it usually works pretty nicely in my experience (on WSL, set up on a few different laptops). It seems like a lot of pyspark developers aren't aware of this approach, because the tool is most used by java developers, and many pyspark developers don't in fact know java (at least I don't ☺ ).
SDKMAN allows you to, at least the way I think about, create the java equivalent of a virtual directory for your repo via a .sdkmanrc file at the root.
Right now I have the following:
java=21.0.4-tem
spark=3.5.1
hadoop=3.3.5

along with adding pyspark to the dev-requirements.txt, I was good to go. I do agree with you that some approach which uses containerization is likely to work best for the most people.

Nice one, thanks @TomBurdge ! would be great to have you on board as contributor here!
I think converting to Arrow might make sense in collect. If anyone wants to support earlier versions of spark and convert to pandas, they can always do that manually with something like
if is_spark_lazyframe(df):
    df = nw.from_native(nw.to_native(df).toPandas(), eager_only=True)
else:
    df = df.collect()

But I think that data science libraries which explicitly support Spark are quite rare unfortunately - well, fortunately for us, it means that backwards-compatibility is less of a concern, and so they likely wouldn't mind only support Spark 4.0+ too much 😄

Thanks Marco, yeah keen to get involved, spare time permitting!
I will try to make it to one of the community calls.
Question on intended behaviour:
For the rename method, the relevant spark function is: withColumnRenamed.
It has a major gotcha, which is that if the original column is not present, then it is a no-op (mentioned in the docs). This behaviour has caused me a lot of frustrations. 🙃
What is the narwhals intended behaviour? Strict?
The narwhals tests in tests/frame/rename_test.py aren't opinionated on strict/non-strict currently.

ah interesting, thanks @TomBurdge ! I think that, as Narwhals is mainly aimed at tool-builders, we'd be better-off using the strict behaviour

Hi @TomBurdge I completely missed these last replies and I have started implementing something for this. Do you also have something implemented? We can collaborate on this and merge what we have :)
I think we should aim for a "minimal" implementation now and then have follow-up PRs for the rest of the methods.

Just letting you know that I'm following this with great interest and could potentially be helpful with testing once you have something ready.

I am new to Spark, just for fun, I tried it locally with: https://github.com/jplane/pyspark-devcontainer

Given that DuckDB also has a PySpark API, maybe this would give us DuckDB support almost for free (we could make a _spark_like folder, like the current _pandas_like one) https://duckdb.org/docs/api/python/spark_api.html


PineconeHynridSearchRetriever missing several args and not returning ids of vectors when adding texts

System Info
There are a few differences between the PineconeHybridSearchRetriever and base Pinecone retriever making it difficult to switch to the former.
@hw
You can pass search_kwargs to pinecone index.as_retriver(search_kwargs:{filter: 'value'}) because it inherits from base VectorStore which has the as_retriever method with search_kwarg args.
But it is not clear from the documentation of PineconeHybridSearchRetriever, which inherits from BaseRetriever, how to pass those same arguments.
Also, the add_text method in Pinecone class returns the IDs, whereas in HybridSearch it does not.
`[docs]    def add_texts(
self,
texts: Iterable[str],
metadatas: Optional[List[dict]] = None,
ids: Optional[List[str]] = None,
namespace: Optional[str] = None,
batch_size: int = 32,
embedding_chunk_size: int = 1000,
**kwargs: Any,
....
....
for i in range(0, len(texts), embedding_chunk_size):
chunk_texts = texts[i : i + embedding_chunk_size]
chunk_ids = ids[i : i + embedding_chunk_size]
chunk_metadatas = metadatas[i : i + embedding_chunk_size]
embeddings = self._embed_documents(chunk_texts)
async_res = [
self._index.upsert(
vectors=batch,
namespace=namespace,
async_req=True,
**kwargs,
)
for batch in batch_iterate(
batch_size, zip(chunk_ids, embeddings, chunk_metadatas)
)
]
[res.get() for res in async_res]
    return ids`

Who can help?
No response
Information

[ ] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[X] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
Create pinecone index and use as_retriever with passing search_kwargs('filter':{key:value})
Vs
Create PineconeHybridSearchRetriever, there is no way to pass filters
Expected behavior
Pass search_kwargs in the class instantiation of PineconeHybridSearchRetriever so it can be used downstream in conversationalretrievalchain

Dear Support,
I'm facing same issue for search_kwargs and apply the advise of @dosubot @OmriNach have you found it solved?
Nevertheless the question now is how to declare "search_kwargs" into my PineconeHybridSearchRetriever , is there any property?
Thank you !!


cirq_google GridDevice: set JSON namespace to cirq.google

Preferred alternative over https://github.com/quantumlib/Cirq/pull/5476 (renaming GridDevice to GoogleGridDevice). This resolves the issue of JSON name collisions between cirq_google and elsewhere by including the cirq.google namespace.
Furthermore, by staying with GridDevice as the class name, we require the caller to include the cirq-google module name in cases when multiple GridDevice classes are used (for example in the server side code today).
@95-martin-orion @mpharrigan
cc @dstrain115 @tanujkhattar

Yes, using the json "namespacing" feature is much preferred to coming up with new names that don't match the class name. All objects would ideally be namespaced (with a carveout for top-level cirq. items)


latest update seems to slow stuff down?

Hi,
I have pulled the latest update (with the skip verification check option) and reinstalled. I think it updated but the version number is the same? anyway I am finding it a lot slower than it was before even with the -si flag enabled.
Has anyone else found the same? It could environmental (more people using JASMIN right now) but would be good it see if anyone is in a similar position as I was hoping it would be quicker!

Oops meant to add it was taking like 10 mins for a monthly mean of grid T (eORCA12) but is now more like an hour.


Error with name of 'op'?

Traceback (most recent call last):
  File "AI_control_4layer.py", line 218, in <module>
    writer)
  File "/home/longfei/Repository/drl_navigation/saved_neural_qlearning.py", line 61, in __init__
    self.create_variables()
  File "/home/longfei/Repository/drl_navigation/saved_neural_qlearning.py", line 148, in create_variables
    self.train_op = self.optimizer.apply_gradients(gradients)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/optimizer.py", line 412, in apply_gradients
    self._create_slots(var_list)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/rmsprop.py", line 101, in _create_slots
    self._get_or_make_slot(v, val_rms, "rms", self._name)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/optimizer.py", line 639, in _get_or_make_slot
    named_slots[var] = slot_creator.create_slot(var, val, op_name)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/slot_creator.py", line 101, in create_slot
    return _create_slot_var(primary, val, '')
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/slot_creator.py", line 55, in _create_slot_var
    slot = variable_scope.get_variable(scope, initializer=val, trainable=False)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variable_scope.py", line 988, in get_variable
    custom_getter=custom_getter)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variable_scope.py", line 890, in get_variable
    custom_getter=custom_getter)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variable_scope.py", line 348, in get_variable
    validate_shape=validate_shape)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variable_scope.py", line 333, in _true_getter
    caching_device=caching_device, validate_shape=validate_shape)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variable_scope.py", line 684, in _get_single_variable
    validate_shape=validate_shape)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variables.py", line 226, in __init__
    expected_shape=expected_shape)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variables.py", line 322, in _init_from_args
    name=name)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/state_ops.py", line 178, in variable_op_v2
    shared_name=shared_name)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gen_state_ops.py", line 708, in _variable_v2
    name=name)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/op_def_library.py", line 763, in apply_op
    op_def=op_def)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2398, in create_op
    self._add_op(ret)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2099, in _add_op
    "is already used" % op.name)
ValueError: cannot add op with name q_network/W_conv1/RMSProp as that name is already used

Please provide details about what platform you are using  (operating system, architecture). Also include your TensorFlow version and the code/steps necessary to reproduce your problem. We ask for this in the issue submission template, because    it is really difficult to help without that information. Thanks!

I think this problem is coursed by duplicated name of Variables.

@xiaop1987 Looks very likely, but hard to tell from the scant information in the original problem report.  It does seem unlikely to be a bug.

I've just encountered the same erro ,  so I was wondering did you fixed the bug? Thx!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Migrate to Gatsby

See #541 for context. Migrates site to Gatsby, positions us to convert the core to a Gatsby theme shared by this repo and headlesscms.org.
Closes #634
Closes #600
Closes #541
Closes #534
Closes #513
Closes #479
Closes #422
Closes #384

"About" and "Contributing" are displayed on the main page as if they were static generators, not special pages.

Argh good catch, thanks
Sent with GitHawk

Another thing: links are all wrong! For example, the card of "yst" actually points to https://www.staticgen.com/heckle

@dmbaturin both seem to be fixed now

@erquhart The issue with "contributing" and "about" having cards on the main page is fixed, but the issue with messed up links got more interesting.
The Nanoc card now points to Nikola and vice versa, for example. Many other links point to seemingly random wrong places.

I haven’t been able to reproduce, those two cards are linking correctly as far as I can tell. Are there steps to take?
Sent with GitHawk

@erquhart Ok, sorry, this is a bit different issue. When clicked, the cards take you to the right place. However, the tooltips are wrong when the page first loads.
Clicking on a card and going back fixes it somehow, but reloading the page makes it appear again.

Ah okay I’ll take a look
Sent with GitHawk


Problem with connect to selenium via Remmina

I try to connect but get connecting.. forever
Every set and up already. The only problem i can't connect to selenium chrome for test mftf
Already read docs about this and follow but not get any luck
https://docs.warden.dev/configuration/mftf.html
How to solve this! I have no idea to get debug on this
Used ubuntu 18.04 + with warden 0.2.2
SELENIUM_HOST=selenium-hub
BROWSER=chrome

Could you submit screenshot of your Remmina connection details?

@lbajsarowicz hi i have just uploaded screenshot

You haven't provided the Server name in basic tab:

Instead of magento2 you should put the value of WARDEN_ENV_NAME with latest Warden.

@lbajsarowicz tried like your screenshot but not get any luck. Same results

@lbajsarowicz @davidalger

Let me verify Documentation, as from 0.2.0 warden needs to have DEBUG setting on, to run Debug Docker image. Example:
WARDEN_SELENIUM=1
WARDEN_SELENIUM_DEBUG=1

Closing this issue for now due to inactivity.


Create Rooms by Polling Homebridge for Rooms

I know that there's some sort of endpoint in the HomeKit API that I am pretty sure Homebridge can access to sync the rooms set up in Homekit with this Config-UI-X package. I know this because the Philips Hue app syncs rooms between the Hue bridge and HomeKit.
It would be nice if homebridge-config-ui-x would mirror it's rooms with HomeKit this way.

That is technically not possible.  You can only setup rooms using the HomeKit API, which is available only to HomeKit apps running on iOS.  homebridge uses the HomeKit Accessory Protocol (HAP), which doesn’t provide access to rooms (nor scenes nor automations).

Thanks @ebaauw.

My bad. But that's also Interesting .. a homebridge helper app on iOS could help with this. Though, Apple may reject it due to the nature of homebridge not being "certified" or an official partner.


Order By Child-Parent-Child Yields "There can be only one input field named Child"

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Ordering requires a sequential narrative presently, such as "order by Last Name, then by First Name, then by Age".  Because of this, if a query needs to order by parent and child properties where the child's properties are not back-to-back, the dreaded "There can only be one input field named Child" will appear.
Steps to reproduce
I've created a repository to pull down and demonstrate the following example.

Ensure the root entity represented in the query has at least one child.
Order by a child property, then parent property, then child.
Attempt to execute the query

Sample query
query {
  people (order: {
    demographic: { gender: ASC }
    address: { state: ASC }
    demographic: { age: ASC }
  }) {
    lastName
    demographic {
      gender
      age
    }
    address {
      city
      state
    }
  }
}

Relevant log output
{
  "errors": [
    {
      "message": "There can be only one input field named `demographic`.",
      "locations": [
        {
          "line": 5,
          "column": 5
        }
      ],
      "path": [
        "people"
      ],
      "extensions": {
        "field": "demographic",
        "specifiedBy": "http://spec.graphql.org/October2021/#sec-Input-Object-Field-Uniqueness"
      }
    }
  ]
}

Additional Context?
Note that this only applies to order.  Filters don't care about what order they are entered and can be grouped by child objects.
A possible workaround would be to use the array option like the where's and and or can use:
query {
  people (order: [
    { demographic: { gender: ASC } }
    { address: { state: ASC } }
    { demographic: { age: ASC } }
  ]) {
    lastName
    demographic {
      gender
      age
    }
    address {
      city
      state
    }
  }
}

Product
Hot Chocolate
Version
12.5.0

Being as changing order from an object descriptor to an array, that would be a breaking change.  Perhaps adding a operation like the and operation in the filter convention would keep it from breaking existing functionality.  Something like all?
query {
  people (order: { all: [
    { demographic: { gender: ASC } }
    { address: { state: ASC } }
    { demographic: { age: ASC } }
  ]}) {
    lastName
    demographic {
      gender
      age
    }
    address {
      city
      state
    }
  }
}

order is already an array, so you can do
query {
  people(
    order: [
      { demographic: { gender: ASC } }
      { address: { state: ASC } }
      { demographic: { age: ASC } }
    ]
  ) {
    lastName
    demographic {
      gender
      age
    }
    address {
      city
      state
    }
  }
}


docs(web): change coverpage logo url to global path

Local path is resolved incorrectly.

Thank you!


Bug: Nested list Markdown ingestion

Lexical isn't handling nested markdown lists with whitespace properly. When passing a nested list in like this:
- foo

  - bar

Lexical creates two separate lists with a paragraph in between them. Though this is in fact legitimate syntax for nested lists in commonmark and other popular tools understand this.
Lexical version: 0.3.3
Note that Lexical doesn't emit lists like this, but when passing in existing markdown this occurs
Link to code example:
https://codesandbox.io/s/lexical-rich-text-example-forked-m96rm2?file=/src/Editor.js
The current behavior
Lexical creates two separate lists, separated by a paragraph
The expected behavior
Lexical should create a single list, with the child list properly nested within

Running .replace(/^(\n)(?=\s*[-*+])/gm, '') on editor content before passing it into Lexical fixes this issue for us, as it collapsed new lines between nested lists

If it helps, prosemirror-markdown uses markdown-it as a dependency to handle commonmark parsing: https://github.com/ProseMirror/prosemirror-markdown/blob/master/src/from_markdown.js. Lexical could adopt that instead of hand rolling it.

@fantactuka Any idea if this has been fixed already?

@fantactuka hello,
is this bug going to be fixed in any future releases?
Thanks :)


Ability to have a more complex logic for determining a singular filter

Currently the "singular filter" feature works on a per filter basis. In the old library the isSearchOne function existed on the adapter in which a more complex logic could be written, for example in the case of a composite primary key.
    protected function isSearchOne(Collection $filters): bool
    {
        if ($filters->has('userId') && $filters->has('clientId')) {
            return true;
        }

        return false;
    }

I'm not aware if/how the same thing can be achieved with this package.

Yep, true this will need to be added. Thanks for reporting.

This is implemented as the Schema::isSingular method.
For example:
class PostSchema extends Schema
{

    // ...

    public function isSingular(array $filters): bool
    {
        return isset($filters['userId'], $filters['clientId']);
    }

}


Pagination - explore page

Add pagination to bottom of explore page at certain number of items/rows
component: https://mui.com/material-ui/react-pagination/

Can you assign this to me?


Item constraint that requires an exact value match

With the introduction of #94 (Support dynamic item validation constraints), it is possible to specify a dynamically computed value for a property or element constraint (e.g. minimumValue: function(doc, oldDoc, value, oldValue) { ... }). In some cases, it may be useful to include a constraint that dynamically specifies a value that must be matched exactly by the property/element value.
For example (not to be considered final):
propertyValidators: {
  processorId: {
    type: 'string',
    required: true,
    mustEqual: function(doc, oldDoc, value, oldValue) {
      var docIdRegex = /^paymentProcessor\.([A-Za-z0-9_-]+)$/;
      var regexMatches = typeRegex.exec(doc._id);

      return (regexMatches.length > 1) ? regexMatches[1] : '';
    }
  }
}

Note that, while the primary use case may be as a dynamic item constraint, it should also accept a static value, as with all other item constraints, for the sake of consistency.
Also note that a constraint value of null should be treated literally. In other words, if a mustEqual constraint specifies null, then the property/element's value must be null as well. But a constraint of undefined means that the mustEqual constraint is not applicable.

Addressed by #125


[fix] Allow numeric as value of header

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[ ] This message body should clearly illustrate what problems it solves.
[x] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

In the current version of SvelteKit, if an endpoint return an header with a numeric value, an error is raise:

TypeError: value.forEach is not a function

The test on the value only assume that value can be either a string or an Array (the assumption is on the array).
https://github.com/sveltejs/kit/blob/89a97551f9e771e0e00064646a5196b943480ab1/packages/kit/src/utils/http.js#L10-L12
Per definition, a header value can only be a text (there is no typing in the HTTP protocol).
The type definition also define string and an array of string as only possibility
https://github.com/sveltejs/kit/blob/89a97551f9e771e0e00064646a5196b943480ab1/packages/kit/types/helper.d.ts#L7
But nothing prevent a number (integer or decimal) to be defined (if not typing check is done) and there are not, per say, wrong (for example content-length header is a numerical value)

Another approach can be to ignore all values that are not a string or an array of string
if (typeof value === 'string') { // Allow string value
	headers.set(key, value);
} else if (Array.isArray(value)) { // Allow array
	value
		.filter((value) => typeof value === 'string') // Ignore every value that are not string
		.forEach((value) => {
			headers.append(key, value);
		});
} // values that are not a string or an array are ignored

The fact that user should set correctly headers can also be a way to handle this case.
Then feel free to close the PR :wink:

thanks!


fix(auth, android): fix crash on Android where detaching from engine was not properly resetting the Pigeon handler

Description
See title
Related Issues
Fixes https://github.com/firebase/flutterfire/issues/9210
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[ ] All existing and new tests are passing.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] The analyzer (melos run analyze) does not report any problems on my PR.
[ ] I read and followed the Flutter Style Guide.
[ ] I signed the CLA.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[X] No, this is not a breaking change.

Any updates here?
It's a kinda urgent fix :D

What is the version for the fix?

What is the version for the fix?

It is not published yet. The will include it to 3.5.2, I guess

Does anyone know the right version mix to downgrade? 3.4.2 saying for Android build:
Unexpected element  (URI:"", lokal:"base-extension").
../../../.pub-cache/hosted/pub.dartlang.org/firebase_auth-3.4.2/lib/src/recaptcha_verifier.dart:57:27: Error: Required named parameter 'auth' must be provided.
      _factory.delegateFor(

In my pubspec.yaml: firebase_auth: '<=3.4.2'. Same for 3.4.1 and 3.3.20.

@timobaehr you can pin these versions:
dependencies:
  firebase_auth: 3.4.2
dependency_overrides:
  firebase_auth_platform_interface: 6.3.2

Seems like the  firebase_auth: ^3.6.1 is fixing that issue for me.   (3.6.0 contain the fix itself)
will keep testing...

What is the version for the fix?

The fix is published in version 3.6.0. You can see the changelog of this version here: https://pub.dev/packages/firebase_auth/changelog#360


Dealt with the rails5.1

Obvious credit goes to @metalels.
Closes #1.

@adamcooke Any chance of merging this? :-)


add helper to generate txt files

Description
Add helper to get the list of packages in txt format which still a compatible version with 4.9 sorted by name

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
18

Relevant Lines:
2115

💛  - Coveralls


Implement an RcStr (e.g. Rc<str>-alike)

That is: a thread-unsafe (e.g. !Send) equivalent to ArcStr. std::rc::Rc would be to RcStr as std::sync::Arc is to ArcStr). It would avoid needing to use atomic operations, but otherwise would support the same features.
 Neither is exactly ideal, but I've wanted a Rc version of this a few times.
I'm not sure if this would be better as a separate crate. (Ideally not, as it would allow sharing code, but this might take renaming the arcstr::literal! macro, and honestly the arcstr name implies, well, Arc-equivalence, so the name is a bit weird).

+1 on this!

Hm, yeah, it would be a nice addition.


Featured Projects for 2017

We need some new featured projects for 2017, looking for open source projects that are specifically very friendly to new users and will be able to actively help out with issues and pull requests on their repo during December.
Featured projects get shown on the homepage to all visitors and are randomly ordered to avoid any one project getting overwhelmed.
If you'd like to nominate a project drop a comment below mentioning why you think it should be featured and if you're not one of the maintainers, please @ one of them here too.

Nominating Tailwind CSS by @adamwathan and co.

The Beeware suite of Python tools are very consciously targetting first-time contributors:
https://pybee.org/contributing/how/
https://pybee.org/contributing/how/first-time/
Many of their repos have issues already tagged as "up-for-grabs" or "first-timers-only". e.g:
https://github.com/pybee/briefcase/issues?q=is%3Aissue+is%3Aopen+label%3Aup-for-grabs
https://github.com/pybee/batavia/issues?q=is%3Aissue+is%3Aopen+label%3Afirst-timers-only

I'm going to nominate @nodunayo's project Speakerline

Nominating @nemesisdesign's https://github.com/openwisp - Very helpful for newbies and regularly takes part in the google's summer of code.

I'd like to nominate Godot. They have beginner-friendly issues in their tracker, and also it'd be nice to have something gamedev related in this year's run.

Umbraco is a fantastic open-source .NET CMS with a wonderfully friendly community. It's easy to use, easy to get involved in, and everyone is incredibly helpful! :)

Another org with a mentoring program aimed at new contributors is OpenSUSE. see:
http://101.opensuse.org/
https://github.com/openSUSE/mentoring/issues
a lot of the projects/tasks where they are looking to mentor a new contributor are quite large though: Maybe not such a great fit if you're trying to encourage people to just crank out a PR every day.

Nominating

Magento https://github.com/magento/magento2 (Not sure who the primary maintainer)
Mautic https://github.com/mautic/mautic (Not sure who the primary maintainer)
Phergie https://github.com/phergie (@matthewtrask)
Joind.in https://github.com/joindin (@liam-wiltshire)
PHP Mentoring https://github.com/phpmentoring (@dragonmantank)

I nominate https://github.com/joomla/joomla-cms. Always been very friendly and open when I've got involved.

I'd like to nominate our open-source PHP project.
It's a middleware stack, following the PSR-7 guidelines, and is a very good entry-point for people trying to move away from fully fledged frameworks.
You can find the docs at dotkernel/dotkernel, but the main stack is at dotkernel/frontend

https://github.com/backdrop/backdrop

Issue queue: https://github.com/backdrop/backdrop-issues
Backdrop CMS is a Drupal fork dedicated to usability and backwards compatibility

https://github.com/lando/lando

Open source javascript local development environment, app dependency management using docker containers.  Supports dev stacks for PHP, Drupal, Backdrop, WordPress, Javascript, Ruby, Python, Go, .net

@Magento has a team of engineers and architects and programs dedicated to helping people to become involved in PRs. @maksek leads this effort.
Also, joind.in is a worthy cause, I think led by @lornajane.

thank you @jcampbell05 for mentioning @openwisp.
OpenWISP is an accepted mentoring organization of the Google Code-In 2017:
https://codein.withgoogle.com/organizations/openwisp/
We 15 to 20 people that will help out new contributors very actively between 28th of November and 17th of January.

@andrew when is cut-off date for submissions :) seems like we have a lot, when will we expect these to be on the website

@jcampbell05 no cut off date, they get cycled randomly so we shouldn't have a problem with too many of them

Also nominating The Laravel Framework. PHP biggest framework to the day :smile:

@andrew what do you think about @cdnjs?

@andrew Feel free to add https://github.com/nexB/scancode-toolkit and other projects at https://github.com/nexB as first PR friendly ;)  as it has always been our way

Nominating a repo which can only be completed by lots and lots of contributions ! :blush: :heart:

A repository which holds various edX-Institutions Logo.  :heart:

I have made edXLogoData, super-friendly, and added fun & easy tasks, just like Google Code-In does. :smiley:
Links Below :+1:  :-
Gitter Chat Channel
Github

I'll nominate https://github.com/thelounge/lounge
It's a modern IRC client and we're looking for new contributors.

Don't forget that you need to add the project you're suggesting before I can set it as "Featured" https://24pullrequests.com/projects/new

Thanks! @andrew :+1:
I got a long-long list of awesome projects!
Exploring! :heart:


Not update component on ternary

The components not change on the render view when use on conditional ternary operator
example:
variable = true
render () {
return (

{ variable ? <ComponentA /> : <ComponentB /> }

}

if you change the variable to false the render will continue to show the ComponentA

Hey @peeweesilva, did you mean to use this.variable?
And to reactively update it, would be:
this.variable = !this.variable

@peeweesilva @GuiDevloper i was there while the issue happened.
The test case was with "me" coming from context like:
render ({ me }) {
  return (

{ me? <ComponentA /> : <ComponentB /> }

    )
 }

@Mortaro @GuiDevloper sorry for the simple example
Here is one better and tested example of the  https://github.com/peeweesilva/nullstack-issue-137
The issue looks happening only when you use outer components

@Mortaro @GuiDevloper sorry for the simple example
Here is one better and tested example of the  https://github.com/peeweesilva/nullstack-issue-137
The issue looks happening only when you use outer components

@peeweesilva this example is awesome! I managed to reproduce the issue and will look into it this weekend. thanks!
@GuiDevloper tasukete! ಥ_ಥ

@peeweesilva this example is awesome! I managed to reproduce the issue and will look into it this weekend. thanks!
@GuiDevloper tasukete! ಥ_ಥ

[WIP]
After some test, seems like the problem is that instead of defining render ComponentD is defining renderComponentD and i didnt notice that before https://github.com/peeweesilva/nullstack-issue-137/blob/5a3266775a0c9f6ea96ccc09699948d9717658ac/src/components/ComponentD.njs#L4
However even after fixing that its not working as it should since its sharing instances with the same key. ill add the class name as a criteria to the key

[WIP]
After some test, seems like the problem is that instead of defining render ComponentD is defining renderComponentD and i didnt notice that before https://github.com/peeweesilva/nullstack-issue-137/blob/5a3266775a0c9f6ea96ccc09699948d9717658ac/src/components/ComponentD.njs#L4
However even after fixing that its not working as it should since its sharing instances with the same key. ill add the class name as a criteria to the key

what do you think to instead of use the class name, which can be the same, or do not have anything for fragments, always create an unique key for each component?
Does this will be overworking and/or overthinking?
Thanks
On Jan 1, 2022, 19:02 -0200, Christian Mortaro @.***>, wrote:

[WIP]
After some test, seems like the problem is that instead of defining render ComponentD is defining renderComponentD and i didnt notice that before https://github.com/peeweesilva/nullstack-issue-137/blob/5a3266775a0c9f6ea96ccc09699948d9717658ac/src/components/ComponentD.njs#L4
However even after fixing that its not working as it should since its sharing instances with the same key. ill add the class name as a criteria to the key
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were mentioned.Message ID: @.***>

what do you think to instead of use the class name, which can be the same, or do not have anything for fragments, always create an unique key for each component?
Does this will be overworking and/or overthinking?
Thanks
On Jan 1, 2022, 19:02 -0200, Christian Mortaro @.***>, wrote:

[WIP]
After some test, seems like the problem is that instead of defining render ComponentD is defining renderComponentD and i didnt notice that before https://github.com/peeweesilva/nullstack-issue-137/blob/5a3266775a0c9f6ea96ccc09699948d9717658ac/src/components/ComponentD.njs#L4
However even after fixing that its not working as it should since its sharing instances with the same key. ill add the class name as a criteria to the key
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were mentioned.Message ID: @.***>

Creating unique keys per component would break the persistent components (or make it heavier to compute), ideally we would have an object_id equivalent for the class itself (from ruby) but JavaScript does not and the overhead might be overkill compared to just warning the user if two components have same name and same depth that it would have weird behavior.
If anyone has a low cost solution for this we could still implement before merge
the issue is fixed on this PR https://github.com/nullstack/nullstack/pull/142/files but still missing tests
@peeweesilva or @GuiDevloper if one of you want to add tests it would be as simple as creating a ClassTernary.njs and a ClassTernary.test.js following the pattern from any other test

Creating unique keys per component would break the persistent components (or make it heavier to compute), ideally we would have an object_id equivalent for the class itself (from ruby) but JavaScript does not and the overhead might be overkill compared to just warning the user if two components have same name and same depth that it would have weird behavior.
If anyone has a low cost solution for this we could still implement before merge
the issue is fixed on this PR https://github.com/nullstack/nullstack/pull/142/files but still missing tests
@peeweesilva or @GuiDevloper if one of you want to add tests it would be as simple as creating a ClassTernary.njs and a ClassTernary.test.js following the pattern from any other test

This is fixed and currently merged into #next, but could use better test coverage (っ °Д °;)っ

Tests added, closing this issue, thanks everyone!


IssueID:1700:Workaround for ethernet spi hardware

[Detail]
Workaround for ethernet spi hardware to prevent dead loops.
[Verified Cases]
Build Pass: helloworld_demo@haas100
Test Pass: helloworld_demo@haas100

感谢提交 Pull Requests！
Thanks for submitting a pull request.

Security Check : 1b6793de2cd26be6cbb3df122afb162c504f191d

auto build success,number:199

autotest fail,number:1535


Create a script or function to remove timeseries

Mule has problems reading timeseries in fieldsfiles. Would be better to have a separate function to remove these?
Something similar to um_fields_subset.py or something mule_based (if the read error can be trapped and handled).

I am having a look at where using mule in perturbIC specifically would fail and the error could be caught for if someone doesn't remove the timeseries.
ifile is the rstart file
f = mule.FieldsFile.from_file(args.ifile) sm = STASHmaster.from_umfile(f) nlon = f.integer_constants.num_cols nlat = f.integer_constants.num_rows
I then plan to try to use STASHmaster to access the theta field with the Itemcode.
So far mule has not failed. Is this expected? We expect mule to fail when accessing this file?

Demo script https://github.com/ACCESS-NRI/umfile_utils/blob/7-remove-timeseries/src/remove_timeseries_demo.py
When run with  /g/data/vk83/configurations/inputs/access-esm1p5/modern/pre-industrial/restart/atmosphere/PI-02.astart-01010101 it removes the timeseries ok but writing the new file fails because of validation of the river fields #8


FancyZones layout resets to defaults despite showing my custom settings

Microsoft PowerToys version
"0.58.0"
Running as admin

[ ] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
Pick a layout, use your computer as normal. Put computer to sleep, disconnect devices. Reconnect devices, resume computer. FancyZones now uses the default "Priority Grid" with 3 zones, despite that not being what my settings show. Turning on and off the "Enable FancyZones" setting fixes the issue and I get my expected layout back.
✔ Expected Behavior
That my layout settings stay as I've set them, no matter what.
❌ Actual Behavior
A different (default) layout gets used instead, and I have to enable and disable the feature to get my configured layout back.
Other Software
No response

/bugreport

Of course, now that I've reported it, I can't get the behavior to trigger anymore. Do you still want the zip file, even while everything seems to be working as expected or should I wait until I'm in an "error state?"

Yes, please send us a report. I'm pretty sure that it's a bug with a monitor name retrieving, it's a known problem.
/needinfo

PowerToysReport_2022-05-30-08-51-51.zip


[Feature Request]: can I plot k33?

I assume it plots k32 files at the moment, how can I plot k33?

It's not supported at the moment. Check Issue: #25

It's not supported at the moment. Check Issue: #25

Thank you. The chia official says they will deprecate k32 in the next pool protocol release, so maybe k32 won't last long.

It's not supported at the moment. Check Issue: #25

Thank you. The chia official says they will deprecate k32 in the next pool protocol release, so maybe k32 won't last long.

Where did they say that? Do you have a source?

The chia official says they will deprecate k32 in the next pool protocol release

@winkee01  No, they did not - stop spreading that lie.

The chia official says they will deprecate k32 in the next pool protocol release

@winkee01 No, they did not - stop spreading that lie.

Guessed so - especially after this: https://github.com/Chia-Network/chia-blockchain/discussions/452#discussioncomment-842628

The chia official says they will deprecate k32 in the next pool protocol release

@winkee01 No, they did not - stop spreading that lie.

 How would you confirm what you said is not a lie?? you are so arrogant

The chia official says they will deprecate k32 in the next pool protocol release

@winkee01 No, they did not - stop spreading that lie.

The chia official says they will deprecate k32 in the next pool protocol release

@winkee01 No, they did not - stop spreading that lie.

 How would you confirm what you said is not a lie?? you are so arrogant

Huh????

k32 will surely be deprecated pretty soon. chia_plot change the whole game. period.

The chia official says they will deprecate k32 in the next pool protocol release

@winkee01 No, they did not - stop spreading that lie.

 How would you confirm what you said is not a lie?? you are so arrogant

Huh????

k32 will surely be deprecated pretty soon. chia_plot change the whole game. period.

I mean not really... it's impressive, sure, but it's not like "oh my god". Because it doesn't parallelise well: @madMAx43v3r can back me up on this - it can produce some stunning plot times, (30 minutes) but only when you have something like 32 cores and 350gb of ram dedicated to a single plot. As such, sure, you can get 30 min plot times, but think about this: 30 min plots = 2 plots/hr = 48 plots a day. With the official plotter, you could run 8 plots parallel with 4 cores per plot on an SSD. These plots would finish in about 9 hrs, so that's a bit under an average of 1 plot/hr therefore 24 plots/day. As such, this plotter produces a stunning doubling of plots/day on many CPUs, but often you need a ramdisk (at least 128gb of ram for a ram/ssd hybrid plot, ideally 512gb for a full ramdisk plot). So, the, this plotter does exactly what it says in the second sentence of the readme: it maxxes out your storage device, so you need less drives, and makes for faster plots, but it will not singlehandedly cause the downfall of k32.

correct

The concern on k size is with respect to generating solutions on the fly (less than 20 seconds), not the rate of filling hard drives. Still have a long way to go before k32 is deprecated.

I believe k32 is going to stale

The chia official says they will deprecate k32 in the next pool protocol release

@winkee01 No, they did not - stop spreading that lie.

 How would you confirm what you said is not a lie?? you are so arrogant

Huh????

k32 will surely be deprecated pretty soon. chia_plot change the whole game. period.

I mean not really... it's impressive, sure, but it's not like "oh my god". Because it doesn't parallelise well: @madMAx43v3r can back me up on this - it can produce some stunning plot times, (30 minutes) but only when you have something like 32 cores and 350gb of ram dedicated to a single plot. As such, sure, you can get 30 min plot times, but think about this: 30 min plots = 2 plots/hr = 48 plots a day. With the official plotter, you could run 8 plots parallel with 4 cores per plot on an SSD. These plots would finish in about 9 hrs, so that's a bit under an average of 1 plot/hr therefore 24 plots/day. As such, this plotter produces a stunning doubling of plots/day on many CPUs, but often you need a ramdisk (at least 128gb of ram for a ram/ssd hybrid plot, ideally 512gb for a full ramdisk plot). So, the, this plotter does exactly what it says in the second sentence of the readme: it maxxes out your storage device, so you need less drives, and makes for faster plots, but it will not singlehandedly cause the downfall of k32.

I get up to 52 plots per day (less than 28 min per plot) on a 500 dollars setup. 3900X, 32GB RAM and one 1 Tb 980 pro. Your assumptions are wrong.


Add Sentry configuration, README details on disabling. Also custom mailserver README.

@lognaturel looked over my shoulder as I approved those changes.