Modified intergration/config files to fabric 2.0

Config files have been updated to fabric 2.0. However FPC Additions from 1 to 5 have been disabled (commented out)
TODO:
enable decorator , validation plugins and tlcc
Purpose of this PR:
This PR contains changes that make fabric/bin/ledger_init.sh and fabric/bin/ledger_shutdown.sh work with the new lifecycle.
In this PR ercc.go chaincode is installed using the new fabric 2.0 lifecycle.
To test use:
env FABRIC_LOGGING_SPEC=debug FABRIC_CFG_PATH=$(/bin/pwd)/integration/config/ ./fabric/bin/ledger_init.sh
env FABRIC_LOGGING_SPEC=debug FABRIC_CFG_PATH=$(/bin/pwd)/integration/config/ ./fabric/bin/ledger_shutdown.sh
Signed-off-by: Jordan Karaze<EMAIL_ADDRESS>
What this PR does / why we need it:
This PR addresses some of the components of PR #254 by upgrading config files

Do these networks run and get set up in travis? Or one of the make targets?

These networks are used by the integration test scripts in integration/.  As integration is currently disabled (it doesn't work before #254 is fixed) it is not yet invoked in travis. But certainly should be re-enabled in the future ....

As some reference, here is work @swetharepakula did to go from a 1.4 based setup with solo orderer to a 2.0 based setup with raft orderer. https://github.com/swetharepakula/Fabric101Workshop/commit/b8ad704e981b9dc7426f61826b7d0b11509a1c56

As some reference, here is work @swetharepakula did to go from a 1.4 based setup with solo orderer to a 2.0 based setup with raft orderer. swetharepakula/Fabric101Workshop@b8ad704

Glancing through it and seeing the references to tls/server.crt makes me remember that Swetha said that raft requires tls.  So i guess the main (only?) hurdle to cross from the error i got above is to create TLS certificates and put them in the right place.

Yes Raft requires TLS. Cryptogen will create the TLS certs automatically, so you only need to mount them and configure the peer, orderer, ca to use them. If you look at the commit @MHBauer linked above you should see all the necessary changes needed for the peer, orderer and the ca to enable tls.

@Jordy24 please re-request reviews once you are done with your changes and want us to have another look. Thanks :)

Yes Raft requires TLS. Cryptogen will create the TLS certs automatically, so you only need to mount them and configure the peer, orderer, ca to use them. If you look at the commit @MHBauer linked above you should see all the necessary changes needed for the peer, orderer and the ca to enable tls.
Note: this is not for the docker-compose setting, but the run-directly-on-host way the integration tests are run. So no mount and integration tests previously also didn't use any cryptogen ...

Thanks Jordan! Ledger_init and shutdown works in my test setting. Some minor changes still required.


Remove parsing of invited_by_id because it breaks and we don't use it, fixes #645

The Problem/Issue/Bug:
The invited_by_id field is apparently not always what we expect, and we don't use it. So stop parsing/using it.
This must not be pulled because it's an evil experimental edit of a vendor file. Real change will have to be in the go-pantheon repo.
This PR brings in v0.1.0 of go-pantheon, which just ignores the invited_by_id, https://github.com/drud/go-pantheon/pull/14
OP: #645
How this PR Solves The Problem:
Manual Testing Instructions:
Basically we only know one person who can reproduce this. He's tested it. It's probably good to go. We don't know what the bad json looks like.
Automated Testing Overview:

Related Issue Link(s):
Release/Deployment notes:

Given this was approved in the upstream repo, I would vote for a +1, but I think @beeradb would be more appropriate. Adding him to the reviewer field.


Work with Unity for Windows

Does it work with Unity for Windows? As opposed to Android/iOS

Greetings,
At this moment windows platform isn't supported.
We don't plan to include new platforms in the near future.

For anybody curious
A similar piece of software that supports Windows is Agora.io
https://github.com/AgoraIO/Video-Call-for-Mobile-Gaming/issues/8


Migration does not like hyphens in strings

When migrating, I am unable to use a custom expression to generate a UUID for a new column without adding my own quotes to the string:
import 'package:uuid/uuid.dart';

...

// works
final uuidExpr = CustomExpression("'${const Uuid().v4()}'");

// fails
final uuidExpr = CustomExpression(const Uuid().v4());

...

if (from < 3) {
  await m.alterTable(
    TableMigration(
      myTable,
      columnTransformer: {myTable.newUuidField: uuidExpr},
    ),
  );
}

As far as I'm aware, hyphens (-) do not need to be escaped in SQLite.
SqliteException(1): while executing, unrecognized token: "321d", SQL logic error (code 1)

     Causing statement: INSERT INTO tmp_for_copy_{{redacted}} ("id", {{redacted}}) SELECT "id", c5d96733-321d-48ec-b0a4-782926da0375, {{redacted}} FROM {{redacted}};,

Hello @MrTarantula . Referencing to the documentation:
Example:
   await m.alterTable(
                TableMigration(
                  usersAtV2,
                  columnTransformer: {
                    users.name: const Constant<String>('no name'),
                  },
                  newColumns: [usersAtV2.name],
                ),
              );

My suggestion is that you should do:
              m.alterTable(
                TableMigration(
                  myTable,
                  columnTransformer: {
                    myTable.newUuidField: Constant(Uuid().v4());
                  },
                ),
              );

Please take a look at the documentation. Also to the the examples:

I've gotten pretty familiar with the documentation. Although there is a page on expressions, I couldn't find anything on when to use Constant() vs Variable() vs CustomExpression() vs some other built-in expression.
I also see these expressions in the API docs, but the drift namespace contains quite a bit of things that aren't expressions, so it's difficult to know what's in there.
Thank you, all the same. I do appreciate the assistance.


NetworkImage occasionally does not load (continued)

 Issue by afandria
Sunday Oct 18, 2015 at 21:13 GMT
Originally opened as https://github.com/flutter/engine/issues/1664

Using flutter 0.0.4. The solution in #801 may not have been applicable to my app, after all. There aren't any Sky errors printed out to stdout (or adb logcat).
I am building the .flx file with all my image assets pointed to in the manifest.yaml. I start Sky with pub run sky_tools -v --very-verbose run_mojo (and other options including --enable-multiprocess).
Related #1072

 Comment by abarth
Monday Oct 19, 2015 at 16:24 GMT

Darn.  We'll keep working on it.

cc @chinmaygarde you fixed networking loading. Do you think we can close this?

Yeah. This is fixed.


i'm find it not work on my anroid mobile default browser

i'm find it not work on my anroid mobile default browser

Hi @webxmsj
Thanks for filing an issue.
Which example is not working?

Hi @webxmsj
Thanks for filing an issue.
Which example is not working?

about getusermedia

I don't think getUserMedia is supported in any version of Android Browser: caniuse.com/#feat=stream.
You'll need to use Chrome or another browser that supports getUserMedia.

Haven't heard any more, so closing this.
Feel free to reopen if necessary.

I am having the same issue. it seems Chrome wont give you the current value. it only gives you the value once it changes or has a battery event.
Does anyone know a way to instantly get it?


Gradle task packageExe fails on subtask unzipWix on WIndows

If from a clean project you execute the :packageExe task, it fails with an error, and as you can notice, it seems that the error is related with the ZIP name, since it tries to unzip something that is not a ZIP file.

Also, if I rename the real ZIP file to the expected name for the :unzipWix task, it fails anyway with another error, and this is a known error related with Gradle wrapper starting from some version, where something like duplicatesStrategy = DuplicatesStrategy.EXCLUDE is necessary to work.

The Project where I'm having this issue is https://github.com/sunlandx/chain-pass.
Windows 10
IntelliJ Community v2021.3
Gradle wrapper v7.4.2
Compose v1.1.1

Probably the same as #1250. There is a walkaround here.

Probably the same as #1250. There is a walkaround here.

Thank you, the workaround is working!

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Integrate directory in app

Fix Mirage to use the right user
Integrate Directory in App
Restyle the landing page

update copyright otherwise looks fine.


Build Up of Log Files

As the documentation notes 'The super workflow never deletes any log file or log archive.'  This is causing issues on machines with smaller drives since these logs compound and quickly fill them. Suggest a profile setting to allow archive logs to either backup off machine, or automatically delete when either certain size thresholds or even date thresholds are met.

The super log files shouldn't be that large... are you deploying with --verbose-mode enabled?

I expect our IT team has verbose mode set and I have queried them to check. In this particular instance the archived log files are 114 GB.

  So yeah... the --verbose-mode option is for troubleshooting and should not be deployed.

I think the issue is the archive feature is zipping up previous compressed archives each time its used

Is it worth just putting a bit of code to delete excess archives for now? 100-200GB is a bit much lol

I found the bug... it was a small typo... but the zip command is surprisingly aggressive! If you tell the zip command to compress a specific file... but it can't find it... it does the whole directory. That is why it's grabbing all the previous zips.
Also there is no such thing as "a bit of code to delete" when you are writing a script with root privileges.  In other words, short term I will fix the archive bug... but it will be a long-term feature to delete old archives.

Of course, you put a lot of effort into Super with the 8K lines of code :)

https://github.com/Macjutsu/super/releases/tag/v4.0.2


Adjust 'projectMacroImpl' to work for camelCased project names.

Currently project macro allows us to do this:
lazy val util = project // Project('util', file('util'))

This patch allows project macro to expands in backward compatible way allowing camelCased project names to be hyphenated like so without resorting to `backtick-workaround`:
lazy val projUtil = project // Project('proj-util', file('proj-util'))

Can one of the admins verify this patch?

Hi Indrajit. Nice seeing you here :)
This at the first this looks like an interesting PR, but I have a few concerns.

This breaks the semantics of project. In other words, if there were some build file, that relied on older behavior like val projA = project, this will break the build.
Even though people do sometimes use hyphen-concatenated lowercase, I don't think it's a strong enough convention for sbt 0.13. For instance, key names have started to migrate to camel case (to unify towards Scala syntax) in sbt 0.13. We frequently get criticism on sbt shell discrepancies from build.sbt, and we are likely to move towards test in Test instead of test:test.
In general, I am already not a big fan of project assuming things from its LHS because it feels like magic and because it needs to be explained to new users. Imagine writing documentation on this behavior. "project will take the name of val, and it turns it into hyphen-concatenated name."

@eed3si9n, great points!
My thoughts:

Acknowledged! This didn't come to my mind. So this actually ends up being a breaking change after all -- that in itself trumps over any other point it might have had in favor.
Hmm, I thought we usually make the project id of the format proj-lib, proj-core etc. so that the name and thus the moduleName ends up being of that format (although I see that you don't follow that in your configs :smile:). In fact, I did come across quite a few cases where the developer resorted to `backtick` to achieve this and got me wondering how to persuade people to actually move towards camel-case. This is the most recent one I saw today that triggered me to create this pull request.
Point taken. The fact that just writing val x = project does lot more with the name x feels like too much magic anyway. Too much implicit convention and it becomes crazy magikz, too much explicit expression, and it becomes crazy verbose :)

The benefit is very marginal (and maybe cosmetic) and the breaking change might not be worth it. Let's kill this. Alternately, this can be moved under DefaultOptions in case people chose to opt-in.

Yeah, I'm in agreement with Eugene. We should strive for consistency here.  Translating CamelCase to database-case everywhere is inconsistent in sbt.   Additionally, in sbt 1.0 we may even deprecate the old key db-style syntax bridge we've been using and turn all keys into the camel-case variant....

I am closing this PR. This could be a fun plugin though.

Agreed, you guys got me convinced that this magic isn't a good idea -- even as a plugin.
Individual project builds might have this for convenience though.


LineFormatter::normalizeException doesn't include SoapFault properties

When NormalizerFormatter::normalizeException() encounters a \SoapFault it adds properties specific to that exception:
https://github.com/Seldaek/monolog/blob/334b8d8783a1262c3b8311d6599889d82e9cc58c/src/Monolog/Formatter/NormalizerFormatter.php#L171-L183
This information can be useful when debugging errors.
LineFormatter::normalizeException() overrides the parent method but does not include the \SoapFault properties - is there a reason for this?
I would happily submit a PR to include this data but I was wondering if the LineFormatter should just call parent::normalizeException() and json_encode the result?

I think I'd rather have the SoapFault special treatment there as well than calling the parent and json_encoding that, because the current formatting is quite different and I'd argue nicer than a simple json_encode would do.


Carousel does not automatically start cycling on pageload

Hi Folks,
Please help me. Page load Carousel cycling automatic how to stop. after prev or next click only it should be cycling.

I have the same issue, was you able to find the solution?

When you load your content async it won't start autoplay. I'm trying to make some kind of a hack right now, I guess solution will be to simulate click on prev or next button.

I have the same issue, @TarikTz did you find a solution ?

@nocerALE Yes I have, I've created timeout function which calls click triggerHandler on .carousel-next element. Here is a preview:
   vm.handleAsyncLoad = () => {         $timeout(function() {           angular.element('.carousel-next').triggerHandler('click');         }, 5000);       };
And I've called this functions after http get request return success.


check that dv metadata added after layer 1st created

May be sending update before metadata itself still being saved

Bug on Worldmap dev.  Dataverse often receives 500 error on 1st legit attempt to retrieve image.  Happens after layer 1st created
May be unresolved (on dev) until after geonode updates.

Metadata definitely being received


Training ILSVRC12 on inception v4 / inception_resnet

@soeaver have you by any chance done some training of ILSVRC12 on your inception v4 / inception_resnet2? I would be interested in an ILSVRC12 pre-trained model for both architectures, therefore I have started training inception_resnet2 a couple of days ago. But since this is taking quite some time I thought maybe you already have done some steps into that direction yourself and one could save some effort here / share?

I'm sorry to tell you I don't have the pre-train models.
To train the models for inception v4 or resnet2 will take a long time and must be very hard to close to the performance of papers.

@soeaver Thank you for your reply.
By now I have  trained a network on ILSVRC12 (validation on all 50k) on your inception_resnet_v2_train_test.prototxt (only modifications: batch_size: 16 in phase: TRAIN and num_output: 1000 on layer "classifier") for 20 epochs.
Data is full ILSVRC12 train and val rescaled to 328x328.
Solver is RMSprop with rms_decay: 0.9, delta: 1.0, gamma: 0.94, base_lr: 0.045 decreased every 2 epochs as in the publication. I did learning on 2 GPUs with iter_size: 2, so effective batch_size is 64.
In the publication http://arxiv.org/abs/1602.07261 Figure 24 shows a single crop Top-5 error on 50k validation set after 20 epochs of ~10% for inception-resnet-v2, Figure 26 a single crop Top-1 error of ~30%.
In my own experiment results after 20 epochs are unfortunately much worse: single crop Top-5 validation error: 45%, Top-1: 70%, so I guess there is a major bug somewhere in my setup.
I will now go back and review the inception_resnet_v2_train_test.prototxt and the solver.
I agree with you that coming up with a well trained model of similar accuracy as in the paper might be somewhat hard. But reaching similar accuracy after 20 epochs as stated in the paper should definitely not be impossible. If you (or anybody else) has got an idea as to the proper setup for training inception-resnet-v2 I am happy to hear.

I have restarted training of inception-resnet-v2 from scratch with small modifications of the solver: 1. reduced effective batch size to 32 and 2. removed small value of weight decay regularization which was present in my solver. The latter one seems to have a dramatic effect after first 2 epochs, i.e. now validation error seems to drop much quicker.

@revilokeb
I am so grateful to you for training inception-resnet-v2, and I think this is a very meaningful job for many people. In https://github.com/dmlc/mxnet-model-gallery/blob/master/imagenet-1k-inception-v3.md, maybe there is something useful to you, they crop 299 x 299 image from 384 x 384 image for training V3. This may improve some performance.
Now, my GPUs are used to train resnet-152 on ILSVRC2012, I'm very glad to share my work when the training is over. I also hope you can get a better performance on inception-resnet-v2.
Thank you again for your efforts!

@soeaver thanks, anyone interested might check https://github.com/revilokeb/inception_resnetv2_caffe where I am posting how training of that inception-resnet-v2 architecture is going.
Would you mind me asking why you have decided to retrain resnet-152 on ILSVRC12 as this has been already released as pre-trained model in caffe (https://github.com/KaimingHe/deep-residual-networks)? I am sure it is a very valuable exercise but I assume also a lot of work and energy on your side...

@soeaver can I ask how your training of resnet-152 has been going and if you have reached similar perormance as He et al. or the FAIR guys?
With respect to your https://github.com/soeaver/caffe-model/blob/master/prototxts/inception_resnet_v2_train_test.prototxt, this has not been generated using your https://github.com/soeaver/caffe-model/blob/master/inception_resnet.py as the latter has got ReLUs after elementwise addition (residuals)
def eltwise_relu(bottom1, bottom2):
    residual_eltwise = L.Eltwise(bottom1, bottom2, eltwise_param=dict(operation=1))
    residual_eltwise_relu = L.ReLU(residual_eltwise, in_place=True)

whereas the former has not. I think the latter is more in line with the original Google publication.
Furthermore:

in https://github.com/soeaver/caffe-model/blob/master/inception_resnet.py  use_global_stats=False in BatchNorm layers is set but not lr_rate. Not 100% sure, but http://caffe.berkeleyvision.org/doxygen/classcaffe_1_1BatchNormLayer.html#details seems to indicate that lr_rate must be set when using use_global_stats?
in https://github.com/soeaver/caffe-model/blob/master/inception_resnet.py convolutional layers followed by BatchNorm do have a bias_term which seems to be superflous given the subsequent BatchNorm layer (however it might not do any harm).

Hi,
Do you have any news about these models ?
Thanks

@christopher5106 in case you are referring to inception-resnet-v2: I have run quite some number of experiments in addition (on Imagenet 299x299, 128x128 & CIFAR10; also made various adjustments) but so far have never been able to train my prototxt's of that architecture to any competitive accuracy. When wide residual nets came out I decided to give them I try and since then got stuck with them (https://github.com/revilokeb/wide_residual_nets_caffe). I might go back to inception-resnet-v2 when I have got some better intuition of wrn's. You might check https://github.com/lim0606/torch-inception-resnet-v2 for more succesfull training of inception-resnet-v2 and also a detailed account of their experiments in torch in the meantime.

@revilokeb thanks a lot. did you add any good news by chance on this topic ?

@soeaver   Where you able to train resnet-152 layer? If so can you share the Hyperparameter or prototxt files.   I am also trying the 152... But the accuracy remains flat...

I have recently train a model on imagenet, here is my experience. you may get bad val accuracy because the use_global_stats set in bn layers, during the training, bn layers is unstable, so if you want to test your model during training, you should set use_global_stats=false, and when the training is almost over, you can set this back to true.

@argman what accuracy did you get on imagenet ?

@christopher5106 I didnot train inception-v4, but another network.

@tyrosathish  I am very sorry to tell you I didn't train resnet152 of my prototxt. If you want the pre-train models of resnets you can fork these works:
https://github.com/KaimingHe/deep-residual-networks
https://github.com/tornadomeet/ResNet
https://github.com/facebook/fb.resnet.torch


[Feature Request] Scan from already taken image.

There should be solution to get the scan result from an image which is already taken like from gallery, etc.

I saw there is a closed issue for this same feature request to which you closed saying:

The focus of quickie is about an easy integration with the camera and therefore UI focused. Calling native filepicker and sending the file uri to the scanner would require this library to decouple the barcode scanner from CameraX (which is currently not the case).

But, what I want is just a way for the app to pass the image as ByteArray or URI or whatever you find easy, which the library will scan and give the scan result back.

Hi and thanks for this FR. As you already notices this is a duplicate of #32. The library is build around CameraX for setting up the camera and allowing live QR code scanning (thats what most of the code is about). Processing the image is basically just a one liner:
https://github.com/G00fY2/quickie/blob/a971f09a0e2feb5a4b0e8f15622a4d0d3989f9bf/quickie/src/main/kotlin/io/github/g00fy2/quickie/QRCodeAnalyzer.kt#L48
If you just need this functionality I encourage you to integrate ML Kit manually.


[GAIA-IR] fix gaia-test ci caused by invalid group type used in compiler

What do these changes do?
as titled

Related issue number

Fixes

Codecov Report

Merging #1979 (bc7084c) into main (48d3c31) will decrease coverage by 3.84%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1979      +/-   ##
==========================================
- Coverage   43.93%   40.08%   -3.85%
==========================================
  Files          89       89
  Lines        9820     9820
==========================================
- Hits         4314     3936     -378
- Misses       5506     5884     +378

Impacted Files
Coverage Δ

python/graphscope/tests/unittest/test_java_app.py
0.00% <0.00%> (-100.00%)
:arrow_down:

python/graphscope/deploy/hosts/cluster.py
29.41% <0.00%> (-58.83%)
:arrow_down:

python/graphscope/analytical/app/java_app.py
24.24% <0.00%> (-44.95%)
:arrow_down:

python/graphscope/framework/utils.py
40.10% <0.00%> (-14.85%)
:arrow_down:

python/graphscope/framework/app.py
69.95% <0.00%> (-10.33%)
:arrow_down:

python/graphscope/client/rpc.py
81.48% <0.00%> (-8.34%)
:arrow_down:

python/graphscope/framework/graph_schema.py
52.45% <0.00%> (-7.11%)
:arrow_down:

python/graphscope/tests/conftest.py
48.21% <0.00%> (-4.65%)
:arrow_down:

python/graphscope/client/session.py
63.75% <0.00%> (-3.22%)
:arrow_down:

python/graphscope/analytical/udf/utils.py
36.17% <0.00%> (-3.20%)
:arrow_down:

... and 3 more

Continue to review full report at Codecov.

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

Thanks for the quick fix!


Hotfix/campus cameras

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

byuSuite/Apps/CampusCameras/model/CameraStream.swift, line 25 at r1 (raw file):

        cameraStreamDescription = dict["description"] as? String
        refreshInterval = dict["refreshInterval"] as? Int ?? MIN_INTERVAL
        if refreshInterval < MIN_INTERVAL {

refreshInterval = max(refreshInterval, MIN_INTERVAL)

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion.

byuSuite/Apps/CampusCameras/model/CameraStream.swift, line 25 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
refreshInterval = max(refreshInterval, MIN_INTERVAL)

Done.

Comments from Reviewable

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion.

byuSuite/Apps/CampusCameras/model/CameraStream.swift, line 25 at r1 (raw file):
Previously, ecbrady (Erik Brady) wrote…
Done.

Should I inline this with the previous line?

Comments from Reviewable

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

byuSuite/Apps/CampusCameras/model/CameraStream.swift, line 25 at r1 (raw file):
Previously, ecbrady (Erik Brady) wrote…
Should I inline this with the previous line?

Yes. And default to 0

Comments from Reviewable

Review status: 0 of 1 files reviewed at latest revision, 1 unresolved discussion, all commit checks successful.

byuSuite/Apps/CampusCameras/model/CameraStream.swift, line 25 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Yes. And default to 0

Done.

Comments from Reviewable

Reviewed 1 of 1 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Anilist tracker unable to search proper series listing

Steps to reproduce
Attempting to add tracking for a series results in other series showing up that may or may not have slight relevance to the title. Currently attempting to add a series under it's romaji, furigana, and English name and only the romaji gets results and they're the wrong series
Step 1. Go to series listing on Mihon
Step 2. Hit Add Tracking
Step 3. Auto search based off name finds series shown in image
Step 4. Typing in name with Japanese characters (もう触れないキミ) results in blank results
Step 5. Typing English name results in blank results.
I also attempted to search through other extensions and same result occured. Confirmed that Anilist does have a listing on their site for the series, it just won't appear or let me search it through the tracker search.
Expected behavior
Result for series should appear https://anilist.co/manga/177828
Actual behavior
Blank results or other series that are not the correct one appear

Crash logs
No response
Mihon version
0.16.5
Android version
Android 14
Device
Samsung Galaxy S23U
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 0.16.5.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

My apologies then, through Mihon was the only time I was having issues finding series. I didn't realize it was an Anilist issue only!


[Feature]upgrade seatunnel version to 2.1.2

Purpose of this pull request
upgrade seatunnel version to 2.1.2

Brief change log
upgrade seatunnel version to 2.1.2

upgrade seatunnel version to 2.1.2

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If necessary, please update the documentation to describe the new feature.

Codecov Report

Merging #81 (2046626) into dev (7e1e61d) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##             dev     #81   +/-   ##
=====================================
  Coverage   0.00%   0.00%
=====================================
  Files        142     142
  Lines       3782    3781    -1
  Branches     230     230
=====================================
+ Misses      3782    3781    -1

Impacted Files
Coverage Δ

...ava/cn/sliew/scaleph/api/security/TokenFilter.java
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


[Question] How to use this mod?

Please excuse me, how to use this mod? xD

See https://github.com/comp500/Indium/issues/18#issuecomment-779832536


Clean up error pages

Error pages, such as 404 and other pages. need to be cleaned up so they are handled by styles and don't provide any information that may open security holes.

We just need to disable debug and all they'll see is a white screen saying error


Next Node.js Certified Developer Contributors Meeting

Per the meeting today (https://github.com/nodejs-certified-developer/certification/issues/7), we wanted to get another meeting on the Calendar to discuss next steps as a group.
I wanted to be sure that we included everyone in this process, including those that didn't have a chance to participate in the discussion today - I went ahead and created the @nodejs-certified-developer/contributors team to make sure everyone can be made aware as effectively as possible.
Mark has planned for the next meeting to be on March 22nd at 11:00am ET. We're trying to accommodate many time zones here, so hopefully that will work for almost everyone. If we need to tweak it for more individuals to be able to make it, we can definitely attempt that for the next meeting 👍

The call information is here:
Topic: Node.js Certification Meeting
Time: Mar 22, 2018 11:00 AM Eastern Time (US and Canada)
Join from PC, Mac, Linux, iOS or Android: https://zoom.us/j/601197698
Or iPhone one-tap :
US: +16465588656,,601197698#  or +16699006833,,601197698#
Or Telephone:
Dial(for higher quality, dial a number based on your current location):
US: +1<PHONE_NUMBER>  or +1<PHONE_NUMBER>  or +1<PHONE_NUMBER> (Toll Free) or +1<PHONE_NUMBER> (Toll Free)
Meeting ID: 601 197 698
International numbers available: https://zoom.us/zoomconference?m=0VdFdwxfdveMJeNogF1dmqbAbCNCA_qB


Use Fansible provisioning

@MaximeThoonsen would you like to update fansible and integrate it in here?

@benja-M-1 I can help if there are problem but I think the roles here are fine. Next bug update will be for the next Ubuntu LTS

I mean I would like we do not reinvent the wheel here and reuse what you've done on Fansible instead.

@benja-M-1 it's too late


feat: Add ExcludeMethod options with default

Changed

Add filtering logic based on method signatures
Add default method signatures for encoding/marshaling parts of standard libraries

https://github.com/raeperd/recvcheck/issues/7#issuecomment-2468624471

Add Config, NewAnalyzer function

referenced alingse/asasalint
user can pass extra signatures in form of UnmarshalJSON(data []byte, v any) error

Considerations

check only names not signature? name check can be added later, so check signature in this pr
runWithFilter is good pattern? tried to be as simple as possible without custom structs

next todos

Add check for simple method name check, so that user can simply add name of methods
Add README for options

Fixes #7

The signatures of the SQL elements are too "generic" (Value() (any, error)) and it can lead to false negatives (some methods will be ignored, but they should not).
I understand you are trying to create something strict and configurable, but I think the full signature approach is a bit overkill, at least as a first step.
For me, the option is a bit weird: either it is an exception, then it can be managed with the exclusions inside golangci-lint, or this a common interface, and then it should be integrated internally (if possible).
Also, the option can lead to false negatives: a signature can be the same between different methods but those methods are related to different structures.
For me, the first step could be to not provide options, and filter on method names internally.
names := []string{
	"MarshalText",
	"MarshalJSON",
	"MarshalXML",
	"MarshalYAML",
	"MarshalBinary",
	"GobEncode",
}

for _, name := range names {
	if funcDecl.Name.Name == name {
		return
	}
}

This will reduce the majority of false positives, and for the other cases, classic exclusions for the type inside golangci-lint can be enough.
issues:
  exclude-rules:
    - path: myfile.go
      text: 'the methods of "Foo" use pointer receiver and non-pointer receiver.'
      linters:
        - recvcheck

If you want to add an option, I think something simple can be used: struct name + method name (ex: Foo.Value).
This is less "generic" but easier to configure than the signature for a user and more strict.
typeName := recvTypeName(funcDecl.Recv.List[0].Type)

func recvTypeName(r ast.Expr) string {
	switch n := r.(type) {
	case *ast.Ident:
		return n.Name

	case *ast.StarExpr:
		return recvTypeName(n.X)
	}

	return ""
}

WDYT?

@ldez
Thanks for the detailed review!
I agree that checking the method signature is somewhat overkill for most users.
The idea of struct name and method name configuration is a good one. It’s much more usable than the full signature approach. (Maybe we can add this to the next PR.)
I’ll fix this PR with the following changes:

Check only the default method names in slices.
Remove the Setting.ExcludeMethod option and leave the NoBuiltinExcludeMethod option.
Remove the Value function from the default.

@ldez
Can we merge this now? lgtm


Provide option to customise snippet placeholder highlight style

Hello,
Unless I've accidentally missed the option from the colour theme reference, there doesn't appear to be a way to customise the highlight colour when filling out snippet placeholders.
The image I've attached should clarify the highlighting I'm referring to.

Colour themes, such as Nord pictured above, using a greyish tones for areas which snippets apply (e.g. comments) will suffer from bad contrast.
I've traced the relevant overlay CSS class to a file named snippetSession.css, where the highlight colour is being hard-coded to a semi-transparent shade of grey.
Exposing this style so that it can be configured would help general users and theme developers alike with better visual clarity.
(VS Code Version: 1.25.0)

/duplicate of #32484


Sinytra connector + respawning mechanic

Hi there,
I am a NeoForge 1.21.1 user who is really interested in trying out your mod. It seems incredibly intriguing, both conceptually and practically, filling an important niche. I see that your mod is a Fabric/Quilt one, which I respect. I was wondering whether you could make your mod working with the Sinytra connector? Right now, when I try to play with your mod with the connector, I receive the following error (and crash) on startup: https://mclo.gs/Mzh6Rbr
Secondarily, given that these birds you're adding are subject to change and there are so many planned in the future, have you considered a "respawning" mechanic? As in, the way animals work in bedrock edition. This could be a toggle in the config file, for instance. Here is the respawning amimals mod for proof of concept -- of course in your case it'd be limited to the mobs added by your mod. This would also really help me with playing on an existing world.
Thank you for reading this and for your time!

Hello! I'm glad you're interested in my mod, I'll try to fix the crash with sinytra connector as soon as possible. Hopefully I can implement a multi-loader setup and support NeoForge as well, but I'm unfamiliar with how to set that up at the moment.
Currently, songbirds like robins and cardinals are placed in the "Ambient" spawn group and should be able to respawn. I'll definitely make the other birds respawn, and I might also look into wandering traders selling eggs, once I add them.
Thanks for taking the time to make an issue!

After some investigation with using Sinytra connector, I've noticed a problem: even after removing the code that causes the crash from my mod, there is still a strange bug where non-aquatic entities can't seem to move at all when touching water (including the player). It's possible that there are even more problems that I haven't found.
I'm not sure what is causing this problem, as I don't believe any of the code I've written is particularly intrusive to Minecraft itself. I think the best option would be to add native NeoForge compatibility, although that may take a while.

I see, sorry to hear this is happening. From what I know, this inability to swim bug actually occurs with quite a few mods on the Sinytra connector, so I don't think it's exclusive to your code, for what it's worth.
If you do move forward with NeoForge compatibility, I would look greatly forward to it.
By the way, if you ever make a Discord server or anything like that, please do not hesitate to let me know. I would love to join and keep up to date with your work — it's impressive stuff.

Thank you for the kind words! I'm sorry that I cannot find a suitable solution for Sinytra connector compatibility. I'll look into multi-loader support, although I cannot guarantee a time frame for its implementation.

Discord sounds like a great idea :]

I just remembered, I believe I may have some information that might assist with reaching Sinytra connector compatibility, if you're still interested. Previously, I had used the Birds | Boids Addon mod, and had the same exact issue of non-aquatic entities (incl. player) being unable to move in water. Perhaps there is some sort of code or effect shared between the two mods that causes the issue? I hope this helps.

I've looked through their source code, but there doesn't seem to be anything special that might cause this problem. In fact, the Birds mod is even less intrusive than my mod, seeing as it only adds one mob. My guess is that this bug has something to do with Sinytra connector itself when loading these mods, and isn't something that can be fixed on my side.

Discord sounds like a great idea :]

Unfortunately, I don't have enough time to manage a discord server, or anything of the like. If you have any suggestions or questions, feel free to make a new issue. They don't all have to be bugs!

Speaking of questions, I do have one for you. For context, I am very eager to use your mod (I'm a huge ornithophile in real life) and do have the means for this. Assuming it would take you some time to ascertain NeoForge compat, is it permissible / would it help for me to commission a developer experienced with NeoForge to port it? I can ask them to PR you the result too, if you'd like.

I have no issue with ports of my mod to another mod loader. If you want to commission someone to port my mod, then feel free to do so! If they can PR me the result, and if it doesn't require anything extra, then that would be very helpful. Of course, please don't feel obligated to do so.
I do also want to point out that I might be able to get a multi-loader setup working within a week or two, so it may not be necessary to port it. If I encounter any difficulties doing so, or some other problem arises, I'll be sure to let you know.

Hey, I think I may have found a solution to the issue on the connector:
/attribute  neoforge:swim_speed base set 1.0
Running this command seems to have fixed the issue, even upon closing the world and reloading it. It works with other mods with the issue, at least. Apologies if you already knew this would fix the issue, this just proved to be a big revelation to me at least.

Actually, I may have been a little too eager. While this resolves the inability to swim issue for the player, it does not resolve it for all the other non-aquatic entities. Darn!

I'm afraid I've encountered some problems while working with the multiloader project. Without Fabric API, there are quite a few things that my mod can no longer rely on, like access wideners for certain classes. I haven't found any alternatives yet, meaning that this is likely going to delay the NeoForge port for a bit. Sorry about that!

No worries at all! I understand if the NeoForge port is not a viable solution for the time being.
I am happy to report though that I've learned from a friend of a method to fix the inability to swim issue for not only the character, but for all entities: /execute as @e run attribute @s neoforge:swim_speed base set 1.0
I wonder if you would be willing, for the next release of your mod, to change/remove the code causing the crash on startup with Sinytra connector (assuming that's still feasible as you could earlier)?
At least this way people like me can toy around with the mod, the inability to swim by default notwithstanding (an issue easily fixable with a command, as aforementioned). I understand again it's not ideal, but it would mean a lot!

I can certainly do that! I might instead release this version on Github, as it requires the removal of a mixin that is (somewhat) important to the functionality of the penguin. I can link it in this thread as well, if you want.

That works, though I was hoping to be able to keep up with the new releases of your mod as they come (again I’m a huge ornithophile!), and I can imagine it would be impossible to maintain the version without this particular mixin all the same. Apologies for my inexperience here (I’m speaking as a layman with minimal coding knowledge), but perhaps there could be a file in the jar of the mod which I could just delete, or a config setting for that mixin letting me disable it?

As a temporary solution, I think I'll just build a separate jar that has the mixin removed, which will then be uploaded to the Github releases. This should be a quick process, and I'll be able to do it for every release. So no need to worry about missing out on new updates!

Sounds like a great temporary solution. You are a legend, this means a lot to me and I'm sure many other NeoForge users. Thank you!
If you ever make a Patreon or anything like that down the line, know you will have my backing in a heartbeat!

Thank you so much for the kind words! It really means a lot!
You can download the new version at
https://github.com/aqariio/Fowl-Play/releases/tag/v0.6.1-alpha.sinytra

Hello guys. May i join your discussion
I really want to ask about 1.20.1 support for sinytra, even through Gihub. It is really important.
Im making rn pretty big modpack and i dont have enough "life" in my world. It will be wonderful to see your mod there.
Thanks!

Of course! Here's the file:
https://github.com/aqariio/Fowl-Play/releases/tag/v0.6.2-alpha.sinytra%2B1.20.1
It doesn't seem to have the same problem with swimming as the 1.21 version so it should work straight out of the box!

Thank you, i believe forge community will love this :)
here small video what i am working with
https://imgur.com/a/6wOijoh
I think birds fits amazing here!


Error on install when "wheels" is missing

My python has updated through brew and now I'm getting this:

I haven't tried fixing it yet but it's not a great message to show in the user's face.
According to https://github.com/allenai/document-qa/issues/47 I might have to install some dependencies globally but that doesn't sound like a good solution.

@rwols do you have any idea for how to solve this nicely. If we could avoid building at all, that would be nice since it's quite CPU-heavy process.

Nothing comes to mind :)

MacOS 11-specific bug and fixed 5 days ago in cpython...
https://github.com/python/cpython/commit/5291639e611dc3f55a34666036f2c3424648ba50

Cpython Version with the fix is already available so we can close this.


Agenda item: sunsetting the Rust Wasm WG

The Rust Wasm WG is part of the launching pad team, and has been effectively defunct for nearly five years now. It has no official members, and it's probably time we make it official that it's no longer active. I've filed an issue to make that change official here: https://github.com/rust-lang/team/pull/1489.
In https://github.com/rust-lang/team/pull/1489#issuecomment-2200286762 @rylev requested I file an issue with the council to discuss this in a meeting. I'd be more than happy to; I've been trying to get this in front of the council for nearly a year now, and I didn't realize I could file an issue here to put it on the council's agenda.
Proposed process
Procedurally speaking, the process I intend to follow here is the same process we've followed for the Rust Async Ecosystem WG I co-lead and subsequently sunset. Once we decide to the shut the working group down:

We officially archive the working group in the rust-lang/team repo.
We offer project owners the ability to migrate projects from the rustwasm org to an org or account of their choosing.
We proceed to officially archive the rustwasm github org.

This ensures that any projects which are still useful or have activity have a life beyond the working group. But at the same time, this allows us to clearly communicate that these projects are now being developed outside of the Rust governance structure. Which is pretty just a way of codifying the way things have been for a while now.

Closing this as completed as described in https://github.com/rust-lang/team/pull/1489#issuecomment-2203794452


add PerfBuf type and implement InitPerfBuf method to KABPFObject

@humancalico It's done. Thank you!


Added ability to name anonymous modules

When using

We could do this elsewhere?
import initPage from 'init-page';
console.log(initPage.foo);

And of course:
System.import('init-page').then(function (initPage) {
    console.log(initPage.foo);
});

In my case, I'm wanting the above functionality but assumed the name attribute makes more sense than src since it's an inline script.

Also, the script would show up in the dev tools sources panel as init-page instead of <Anonymous Module 1>.

The module system was changed a few months back to use URLs instead of special names. This was done to align with the rest of URLs instead of inventing all normalization operations.
There's still the ability to perform pre-normalization steps like mapping init-page into init-page.js before hitting normal URL normalization, but the js extension adding is being removed and the module table will store everything keyed entirely by URLs.
Do you not have production issues with using inline modules?

I might be doing something fundamentally wrong here, but suppose I have some pretty dynamic pages that initialize differently depending on the user, and I want to include these initialization parameters and methods immediately as inline script(s) rather than adding an unnecessary request.
In the following (relatively simple) example, this HTML might be generated based on something the user created.  SomethingDynamic would be whatever the user had associated with that page.
<!DOCTYPE html>

    <meta http-equiv='Content-type' content='text/html; charset=utf-8'>
    <title>Example</title>

And then suppose we had some other static module that could take a look at what exactly was created for the page via main:
SomeStaticModule.js:
import main from 'main';
console.log(main.App);
console.log(main.UserProps);

Unfortunately loading an ES6 transpiler in production is worse than an extra request here.
The options are:

Have some kind of HTML build process automatically compile this inline ES6 into the System.register format
Just reference a single main entry point
Just write in AMD or CommonJS for this inline code

For production optimization you need to bundle though anyway, so I'm not sure that that extra request costs you anything at all.
I'd strongly suggest the single main entry point bootstrapper approach.

This page is actually generated in a development environment and why I find it acceptable (and necessary).  The production flow will definitely be different, minimal everything.

I'll elaborate a little more.  The idea here is that developers can collaboratively build collaborate apps in real-time, which is where this development page and all my other recent issues are coming from.  :)

don't forget that the script type="module" thing is just temporary until <module> is implemented. But AFAIK no-one has even started on a spec for <module>.


cast interface not working with gnosis chain

Component
Cast
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
No response
What command(s) is the bug in?
cast interface
Operating System
macOS (Apple Silicon)
Describe the bug
When running cast interface --chain gnosis 0xf8D1677c8a0c961938bf2f9aDc3F3CFDA759A9d9 to generate the interface for the SDai token which is verified on gnosisscan, the command fails with the below error:
(Looks like the error is bubbled up from ethers-etherscan and something with the api format is not quite right).

Seems fixed after running foundryup again


CanvasKit Image downscaling is poor quality compared to HTML and Native

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
In CanvasKit when downscaling an image the quality is really poor compared to the output from the HTML renderer and Native, especially on 1080p and below screen resolution. I think this is different than the other related Canvaskit renderer bugs as it clearly shows the difference in the output.
I tried a number of different things around filters and nothing appears to help. Is there a setting I'm missing?
*Second observation, the aspect ratio / spacing is somehow different compared to the iPhone emulator?
Expected results
Smoothly downsized images that are legible
Actual results
Very jagged and faded detail, this is on a 3440x1440p monitor. In particular it is the bottom line size that I'm having difficulty with.
As you can see on the last line, the 30 is not legible for the Canvaskit renderer.
And for the smallest zipper image the details are smoothed nicely for the Native and HTML renderers but not for the Canvaskit
Code sample
Code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      home: const MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key});

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            Image.network(
              'https://raw.githubusercontent.com/zambetpentru/flutter_89037_video_exit_fullscreen_black_image/82d06d5a5c5004793c2711a4445a3249b85b4fc7/72420_010002_180_PA.jpg',
              width: 450,
            ),
            Image.network(
              'https://raw.githubusercontent.com/zambetpentru/flutter_89037_video_exit_fullscreen_black_image/82d06d5a5c5004793c2711a4445a3249b85b4fc7/72420_010002_180_PA.jpg',
              width: 250,
            ),
            Image.network(
              'https://raw.githubusercontent.com/zambetpentru/flutter_89037_video_exit_fullscreen_black_image/82d06d5a5c5004793c2711a4445a3249b85b4fc7/72420_010002_180_PA.jpg',
              width: 150,
            ),
            Image.network(
              'https://raw.githubusercontent.com/zambetpentru/flutter_89037_video_exit_fullscreen_black_image/82d06d5a5c5004793c2711a4445a3249b85b4fc7/72420_010002_180_PA.jpg',
              width: 100,
            ),
            Center(
              child: SizedBox(
                height: 64,
                child: Row(
                  mainAxisAlignment: MainAxisAlignment.center,
                  children: [
                    Image.network(
                      'https://raw.githubusercontent.com/zambetpentru/flutter_89037_video_exit_fullscreen_black_image/82d06d5a5c5004793c2711a4445a3249b85b4fc7/B1.jpg',
                    ),
                    Image.network(
                      'https://raw.githubusercontent.com/zambetpentru/flutter_89037_video_exit_fullscreen_black_image/82d06d5a5c5004793c2711a4445a3249b85b4fc7/W30.jpg',
                    ),
                    Image.network(
                      'https://raw.githubusercontent.com/zambetpentru/flutter_89037_video_exit_fullscreen_black_image/82d06d5a5c5004793c2711a4445a3249b85b4fc7/CNO.jpg',
                    ),
                  ],
                ),
              ),
            ),
            Center(
              child: SizedBox(
                height: 48,
                child: Row(
                  mainAxisAlignment: MainAxisAlignment.center,
                  children: [
                    Image.network(
                      'https://raw.githubusercontent.com/zambetpentru/flutter_89037_video_exit_fullscreen_black_image/82d06d5a5c5004793c2711a4445a3249b85b4fc7/B1.jpg',
                    ),
                    Image.network(
                      'https://raw.githubusercontent.com/zambetpentru/flutter_89037_video_exit_fullscreen_black_image/82d06d5a5c5004793c2711a4445a3249b85b4fc7/W30.jpg',
                    ),
                    Image.network(
                      'https://raw.githubusercontent.com/zambetpentru/flutter_89037_video_exit_fullscreen_black_image/82d06d5a5c5004793c2711a4445a3249b85b4fc7/CNO.jpg',
                    ),
                  ],
                ),
              ),
            ),
            Center(
              child: SizedBox(
                height: 24,
                child: Row(
                  mainAxisAlignment: MainAxisAlignment.center,
                  children: [
                    Image.network(
                      'https://raw.githubusercontent.com/zambetpentru/flutter_89037_video_exit_fullscreen_black_image/82d06d5a5c5004793c2711a4445a3249b85b4fc7/B1.jpg',
                    ),
                    Image.network(
                      'https://raw.githubusercontent.com/zambetpentru/flutter_89037_video_exit_fullscreen_black_image/82d06d5a5c5004793c2711a4445a3249b85b4fc7/W30.jpg',
                    ),
                    Image.network(
                      'https://raw.githubusercontent.com/zambetpentru/flutter_89037_video_exit_fullscreen_black_image/82d06d5a5c5004793c2711a4445a3249b85b4fc7/CNO.jpg',
                    ),
                  ],
                ),
              ),
            ),
            Center(
              child: SizedBox(
                height: 16,
                child: Row(
                  mainAxisAlignment: MainAxisAlignment.center,
                  children: [
                    Image.network(
                      'https://raw.githubusercontent.com/zambetpentru/flutter_89037_video_exit_fullscreen_black_image/82d06d5a5c5004793c2711a4445a3249b85b4fc7/B1.jpg',
                    ),
                    Image.network(
                      'https://raw.githubusercontent.com/zambetpentru/flutter_89037_video_exit_fullscreen_black_image/82d06d5a5c5004793c2711a4445a3249b85b4fc7/W30.jpg',
                    ),
                    Image.network(
                      'https://raw.githubusercontent.com/zambetpentru/flutter_89037_video_exit_fullscreen_black_image/82d06d5a5c5004793c2711a4445a3249b85b4fc7/CNO.jpg',
                    ),
                  ],
                ),
              ),
            ),
          ],
        ),
      ),
    );
  }
}

Screenshots or Video

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[!] Flutter (Channel stable, 3.13.4, on macOS 13.3.1 22E261 darwin-arm64, locale en-GB)
    ! Warning: `flutter` on your path resolves to /Users/omega/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /users/omega/flutter.
      Consider adding /users/omega/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/omega/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /users/omega/flutter. Consider
      adding /users/omega/flutter/bin to the front of your path.
[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
[✓] Xcode - develop for iOS and macOS (Xcode 14.3)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.2)
[✓] VS Code (version 1.82.2)
[✓] Connected device (3 available)
[✓] Network resources

Hi @zambetpentru. Thanks for filing this. I can't seem to reproduce this on the latest version of flutter. Can you upgrade to the latest version of flutter to see if this issue still persists.

Canvaskit
HTML

Details

[✓] Flutter (Channel stable, 3.13.6, on macOS 13.0 22A380 darwin-arm64, locale
    en-NG)
    • Flutter version 3.13.6 on channel stable at
      /Users/damilolaalimi/sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ead455963c (30 hours ago), 2023-09-26 18:28:17 -0700
    • Engine revision a794cf2681
    • Dart version 3.1.3
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/damilolaalimi/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.82.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (5 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554                        •
      android-arm64  • Android 14 (API 34) (emulator)
    • iPhone 14 Pro (mobile)      • 6548B0D1-7674-4306-9852-198C465F1104 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • iPhone 14 Pro Max (mobile)  • BB55E997-7F31-462D-B3B1-6B177D9B40C7 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)
    • macOS (desktop)             • macos                                •
      darwin-arm64   • macOS 13.0 22A380 darwin-arm64
    • Chrome (web)                • chrome                               •
      web-javascript • Google Chrome 117.0.5938.132

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Hi,  what is your screen resolution?

Hi @zambetpentru. It's a macbook 13.3-inch (2560 × 1600). Please do check on the latest version and if it persists, you may share your screen resolution as well.

Hi,
This is still with 3.14.4 and on the internal screen of my M1 Macbook Pro 14" (3024 × 1964).

Thanks for checking this @zambetpentru. This seems to vary with the resolution of the screen's device. I'd label this for more insights.
Thank you.

Technically this is not a web-only problem. It should be the same on all platforms. However, the web is more frequently affected because many desktop monitors have low pixel density.

Hello @yjbanov, is there any update on this issue? I am still facing the same problem, and it is kind of annoying that canvaskit does that.

A few releases ago the framework switched to FilterQuality.medium by default. This implies mipmap generation and is required to have good image quality while downsampling to less than half the original resolution.
I wonder what part of canvaskit guarantees that mip generation has happened? is it possible we're asking for linear sampling + mipmaps, but if the texture has no mip levels, then canvaskit is skipping it?


Revert "Don't explicitly call _onUnregistered in unregisterApp()"

Reverts UnifiedPush/flutter-connector#115
Ping @emersion

I was a little too quick to accept PR #115, so I open this one to discuss a bit what is the best behavior.
On the android lib, the connector set a onUnregistered method in a broadcast listener in case the distributor unregister it.
And if the app sends an unregister request, the connector removes directly the connection token, so the onUnregistered after it is ignored.
On flutter, calling unregister does the same thing, but for ease of use it directly call _onUnregistered.
By removing the explicit call to _onUnregistered, the app need to write the unregistration logic after the unregister request. To illustrate:
With the explicit call to _onUnregistered:
[...]
void onUnregistered(String _instance) {
  myFunc(_instance);
}
[...]
UnifiedPush.unregister(instance);
// nothing
[...]

Without the explicit call:
[...]
void onUnregistered(String _instance) {
  myFunc(_instance);
}
[...]
UnifiedPush.unregister(instance);
myFunc(instance);
[...]

I think the flutter one is more efficient but it gives less liberty to the developers.
@emersion : What do you think of it ?

PS: Fortunately, the modification of this behavior seems to disturb only the example :)

At least ntfy will call onUnregistered asynchronously after it's done handling the unregister intent... Are you using a distributor which doesn't?
IOW, this explicit call shouldn't be needed for the onUnregistered callback to be called.

This is how it works atm:
Application sends unregister

Step
application
Distributor

1
call unregister  function

2
unregister function store the token in a local var

2
unregister function removes the token

3
send UNREGISTER to the distributor with the token

4

Store the token in a local var

5

Remove the connection and the token

6

Send UNREGISTERED with the token

7
Ignore since the token is not known

Distributor unregisters an application

Step
application
Distributor

1

Store the token in a local var

2

Remove the connection and the token

3

Send UNREGISTERED with the token

4
Remove the token

5
call onUnregistered

PS: I was waiting, thinking I sent this comment ^^
PPS: The more I think about it, the more I think it makes sense to ask the app to explicitly make its logic after the unregister request is sent, like it is done on android. I will probably just close that PR then

Are you using a distributor which doesn't?

Thinking out loud, @p1gp1g would be able to answer this better, but what if a distributor is uninstalled or force stopped or just plain buggy? The app should still move on with its life.
Apps should call their internal unregister logic (delete the endpoint from the server) after both an UNREGISTER request and UNREGISTERED notice. So, I like @emersion's commit, however, if an app runs its unregistered logic (delete endpoint from server) only inside onUnregistered, it would break. Therefore, this should be a major version.

Thinking out loud, @p1gp1g would be able to answer this better, but what if a distributor is uninstalled or force stopped or just plain buggy? The app should still move on with its life.

That is exactly why we have implemented that way
For what I've seen, there isn't any app that should suffer from the change ATM (there aren't a lot of flutter app using it ^^). Nevertheless, I agree with the major bump and I'm ok to do it

Closing. The _onUnregistered change has been released in a major release.


Fix zotero item sync counter

Rather than increment by page start index we set counter to start index. This resolves issue for me and now all my library syncs through.

Integration test failure is unrelated, merging.


Add confirmation modal when trying to create new invoice while form is partially or fully filled

Issue
Implement a confirmation modal when users attempt to create a new invoice, especially when the form is partially or fully filled. This confirmation step is meant to help prevent accidental loss of data or incomplete submissions.
Context
Currently, when users create a new invoice, there is no prompt or confirmation step. This can lead to data loss or inadvertent submissions when users have already entered information in the form. To enhance user experience and prevent potential errors, adding a confirmation modal is proposed.
Proposed Solution
When a user clicks on the "Create New Invoice" button while the form is partially or fully filled, a confirmation modal should appear. This modal should include a message such as, "You have unsaved changes. Are you sure you want to create a new invoice?" The modal should provide two options: "Create New Invoice" and "Cancel."

If the user selects "Create New Invoice," the new invoice is created, and any unsaved changes are discarded.
If the user selects "Cancel," they return to the form to continue editing.

Tasks

[x] Implement the confirmation modal in the create new invoice flow.
[x] Ensure the modal appears when the form is partially or fully filled.
[x] Include a clear message in the modal to inform users about unsaved changes.
[x] Provide "Create New Invoice" and "Cancel" options in the modal.
[x] Handle user actions accordingly based on their choice in the modal.
[x] Test the feature thoroughly to ensure it functions as expected.

Additional Information

This enhancement will help improve the user experience by preventing data loss and unintended submissions.
It's important to ensure the modal design aligns with the overall user interface and provides clear and concise messaging.

The feature has been successfully implemented. You can review the changes in commit 68c2cd1.


Add lore for other status pages

Previously, our lore only included information on the BonnyCI Status
Screenboard. Now, it includes brief summaries of the Zuul Status,
and Build Status pages.
Related-Issues: BoonyCI/projman#123, BonnyCI/projman#125
Signed-off-by: Matt Langbehn<EMAIL_ADDRESS>

I agree that we should rename the Build Status title to include Travis-CI, and soon we should be able to remove travis tests entirely.


RichTextLabel does not translate strings placed in-between tags

Godot version
v4.1.1.stable.mono.official [bd6af8e0e]
System information
Godot v4.1.1.stable.mono - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3060 Ti (NVIDIA; <IP_ADDRESS>17) - AMD Ryzen 9 3900X 12-Core Processor (24 Threads)
Issue description
RichTextLabel doesn't seem to translate keys placed in-between tags.
Example of the issue here (and how to reproduce it):

Also happens if the tag is closed:

With no tags, the key is translated:

Translation file:

Steps to reproduce

Import a strings csv file for localization with a few strings.
Create a RichTextLabel control.
Enable bbcode and add a tag to the text followed by a translation key.

Minimal reproduction project
N/A

You should use tr function for example:
bbcode_text = "[center]%s[/center]" % tr("%s" % [available_options.keys()[selected_option_idx])

I don't think it's a bug, just misuse of the API.

You should use tr function for example:
bbcode_text = "[center]%s[/center]" % tr("%s" % [available_options.keys()[selected_option_idx])

I don't think it's a bug, just misuse of the API.

That's what I was doing initially, but I ran into an issue where if I changed the locale during runtime, the text wouldn't get updated immediately, because by doing this the translated string is in the bbcode itself, not the translation key.
I could work around that issue too by calling this update function when changing the locale, but I don't think I should have to do that when I don't have to for regular labels or RichTextLabels that don't use tags.
I could also include the tag itself in the csv file, but I don't like this approach either, because I wouldn't be able to use this translation key for other regular labels, for example:


Stargazer Blue/Black Pill Card Design

This issue is similar to #4, however I would also like to have cards that have a low profile socket for the Blue Pill and Black Pill boards.
These can be low cost alternatives to using Adafruit Feather modules.
Edit:
I wanted to add a couple notes.

I'd like to be able to order these boards relatively cheaply through a service like JLCPCB or OshPark
I'd like these boards to be mostly through-hole compatible, so that they are easily solderable as kits, OR to be orderable fully populated (e.g. with JLCPCB's assembly service)
I am willing to fund the first couple hardware spins (PCBs, Components) for this. Please get confirmation from me directly BEFORE ORDERING if you would like me to pay for this or reimburse you.

Note: The Blue/Black pills do have different pinouts unfortunately, meaning that different designs would be needed for each (unless you think it could be handled with jumpers or similar).


cmake: Check if CMP0077 is available (Fix #742)

Since CMP0077 was introduced in CMake 3.13, I should have first detect if CMP0077 is available before set it to NEW.
FYI only policies >= 66 need this since they have been introduced after CMake 3.5 and may not be available, see https://cmake.org/cmake/help/latest/manual/cmake-policies.7.html

I already submitted a fix for this. It will be exported soon.

@derekmauro Thanks ! commented your CL...


Examples ~megathread~

This is a ~megathread~ for tracking the addition of examples.

Tasks will be listed here:

[ ] Exact diagonalization
[ ] vuMPS
[ ] TEBD (MPS)
[ ] TEBD (iPEPS)
[ ] MERA
[ ] Boundary MERA
[ ] TRG
[ ] TNR

Tasks will be listed here:

[ ] Exact diagonalization
[ ] vuMPS
[ ] TEBD (MPS)
[ ] TEBD (iPEPS)
[ ] MERA
[ ] Boundary MERA
[ ] TRG
[ ] TNR


Extend integer types for large file support on 32-bit systems

Disclaimer: from code inspection only.  Effect not verified.
"When your code mixes different types in arithmetic code, Cython must
know about the correct signedness and the approximate longness in
order to infer the appropriate result type.
...
If the type is declared too small and Cython considers it smaller
than other types it is used together with, Cython may infer the wrong
type for an expression and may end up generating incorrect coercion
code. You may or may not get a warning."
Some types get extra long for large file support (_FILE_OFFSET_BITS).
https://web.archive.org/web/20000903062829/http://ftp.sas.com/standards/large.file/x_open.20Mar96.html#2.2.2
time_t could also be 64-bit.  OpenBSD changed it already.

Is there a reason this is needed? Generally, Cython considers a typedef X to be sizeof(X) + epsilon, so these definitions would work whether the underlying type is long or long long. Given the choice, I'd rather leave them all as long.

I didn't test whether it's needed (and admitted that :).

Just to be sure, is "epsilon" either sizeof(X), or one notch up the C type ladder?  That would boost them all from 32 to 64-bits (for coercion in arithmetic).  In that case it will work fine.  Nice hack :).

Could we have a comment in types.pxd, for the benefit of readers like myself?
We can declare off_t as "long", because cython effectively raises the typedef to long long.
(If cython types are over-sized, they won't break anything).

Coming from the C side, I found "incorrect coercion code" pretty scary. I'm looking at code that wants largefile support on 32-bit. I.e. long is 32-bit, off_t is 64-bit. So I checked how Cython picks that up, and it spooked me.
A comment might be useful if you're struggling to narrow down a bug in your Cython program.

While we're here. what's the opinion on moving to unsigned ino_t?  Pretending it's signed will generate the wrong values when you call python code with it, right?

While I was looking - I think the Cython user guide misses a warning here: "[int] will work okay whatever the actual size of a word is".  (It won't do if the "word" type is long long).  http://docs.cython.org/src/userguide/external_C_code.html

[I half-wondered why boosting the size never causes a correctness problem.  But my impression is  the size of a corresponding Cython type doesn't matter very much.  Where it does, Cython errs on the side of safety.  If it slows down fast-path code then the programmer can always tweak their code to avoid it.]

n Fri, Aug 28, 2015 at 11:48 AM, Alan Jenkins<EMAIL_ADDRESS>wrote:

I didn't test whether it's needed (and admitted that :).

Just to be sure, is "epsilon" either sizeof(X), or one notch up the C
type ladder? That would boost them all from 32 to 64-bits (for coercion in
arithmetic). In that case it will work fine. Nice hack :).

Yeah, it's the best we can do.

Could we have a comment in types.pxd, for the benefit of readers like
myself?

We can declare off_t as "long", because cython effectively raises the typedef to long long.
(If cython types are over-sized, they won't break anything).
Coming from the C side, I found "incorrect coercion code" pretty scary.
I'm looking at code that wants largefile support on 32-bit. I.e. long is
32-bit, off_t is 64-bit. So I checked how Cython picks that up, and it
spooked me.
A comment might be useful if you're struggling to narrow down a bug in
your Cython program.

https://github.com/cython/cython/commit/6dd9fd22ecc8d37504870e390485d72dc6c8f654

While we're here. what's the opinion on moving to unsigned ino_t?
Pretending it's signed will generate the wrong values when you call python
code with it, right?

Nope, we detect signedness as well. But made it unsigned so it behaves
correctly wrt arithmetic with unsigned long.

While I was looking - I think the Cython user guide misses a warning
here: "[int] will work okay whatever the actual size of a word is". (It
won't do if the "word" type is long long).
http://docs.cython.org/src/userguide/external_C_code.html

[I half-wondered why boosting the size never causes a correctness problem.
But my impression is the size of a corresponding Cython type doesn't matter
very much. Where it does, Cython errs on the side of safety. If it slows
down fast-path code then the programmer can always tweak their code to
avoid it.]

Actually, what we do is generate custom conversion code for each type that
any self-respecting C compiler will optimize down to the correct branch:
https://github.com/cython/cython/blob/6dd9fd22ecc8d37504870e390485d72dc6c8f654/Cython/Utility/TypeConversion.c#L595

https://github.com/cython/cython/commit/6dd9fd22ecc8d37504870e390485d72dc6c8f654

Thanks for writing this up! That's even better targeted than I thought, from your initial description.


couldn't round with precision = 0

the convert function expects the 3rd parameters is the number of decimal to round
public function convert($from = null, $to, $int, $round = null, $date = null) {...}
but it doesn't work if we want to round the result with precision = 0
pls have a look at this line
https://github.com/danielme85/Laravel-CConverter/blob/master/src/Currency.php#L347
            if ($round) {                 $result = round($result, $round);             }
the condition checking will lead to fail if the $round parameter passed to the convert function is 0

You are absolutely right, should be fixed in master now ( c4919be ): null = no rounding, 0 = round to 0 decimals.
Thanks @longnd !


Fixes #593 Hide Master is Down message on startup

Added "display:none" to table column - attempted to use bootstrap .hide but was attributed with !important so could not be overwritten.

@hkeebler Thanks for the contribution. if you haven't already been added, and would like to be added to our people page at https://accumulo.apache.org/people/ you can open a pull request against the page here: https://github.com/apache/accumulo-website/edit/master/pages/people.md


Race condition

Fixes so a global role cannot overwrite a local

Deletion of local role case should be handled: A commit with a delete of a local role and an unmodified global role of same name results in deletion of role from server. Should upload global role in this case


Keywords in comments trigger end of line errors

Because keywords in comments are treated as normal sourcecode they trigger end of line errors like shown in the following screenshot.

Can be closed due to merged pull request #23.


Linked the formatting setting file in DEVELOPER_GUIDE

Signed-off-by: Owais Kazi<EMAIL_ADDRESS>Description
Linked formatterConfig.xml to use for IDE code style (formatting) settings
Issues Resolved
https://github.com/opensearch-project/OpenSearch/issues/975
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

start gradle check

* What went wrong:
Execution failed for task ':qa:rolling-upgrade:v1.3.0#oneThirdUpgradedTest'.

start gradle check

start gradle check

start grade check

❌   Gradle Check failure c858542 Log 1833
Reports 1833

Looks like a flaky test to me.
rg.opensearch.cluster.coordination.JoinTaskExecutorTests.testPreventJoinClusterWithUnsupportedNodeVersions


Design studio content

@jmheaster I updated the "Team members can write..." sentence. I don't see the odd rendering of the numbered list. Strange - perhaps we merge and see if it shows up?

@jenharris Ok let's give it a shot!


Fix SledStorage::new

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.367%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
55284

Relevant Lines:
56202

💛  - Coveralls


when buffering, it cannot be paused

If the video is buffering, it cannot be paused.

Sorry, can you provide more details?

Can't reproduce and no update. Closing.


When will the code be released?

Hello, many thanks for your attention to our work!
The code will be available as soon as possible (possibly this month). We are checking the codes and the checkpoints. Since we want to release our 10k-pretrained codes, it may take a few more days. Sorry for that.

Dear researchers, our work is now available at Large-Scale-Medical, if you are still interested in this topic. Thank you very much for your attention to our work, it does encourage me a lot!


Custom file extensions

Hi, I really like this plugin, but I needed support for templates with other extensions than php, just like in standard File Templates. So here is PR for such a functionality.
There are many instances when this might come in handy, in my case I wanted to define a test case template for nette/tester that by convention uses phpt extension.
What this PR does:

You can define File Template with extension class.*.
When you select a template in PHP Class From Template dialog, the selected file extension is automatically changed based on the extension of the selected File Template, e.g. class.php -> php, class.phpt -> phpt.
The change of file extension in PHP Class From Template dialog does not interfere with standard PHP Class dialog.

Thank you for your pull request, before I could merge could you merge your commits into single one?

done

@aurimasniekis ping


Sentinel Search in Dart

💥 Proposal
The script will show the implementation of Sentinel Search in Dart programming language.
Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign


fatal error: immintrin.h: No such file or directory

Environment and target

Python: 3.8, 3.9, 3.10, 3.11
TFX: 1.10

Error message
This error raised at #15 [tfx-build 10/15] RUN chmod +x replace_commits.sh && ./replace_commits.s... step.
#15 237.6 ERROR: /root/.cache/bazel/_bazel_root/69d415f69592c20e8fa96228da1bfb1a/external/arrow/BUILD.bazel:36:11: Compiling cpp/src/arrow/compute/exec/key_map_avx2.cc failed: (Exit 1): gcc failed: error executing command /usr/bin/gcc -U_FORTIFY_SOURCE -fstack-protector -Wall -Wunused-but-set-parameter -Wno-free-nonheap-object -fno-omit-frame-pointer -g0 -O2 '-D_FORTIFY_SOURCE=1' -DNDEBUG -ffunction-sections ... (remaining 39 argument(s) skipped)
#15 237.6
#15 237.6 Use --sandbox_debug to see verbose messages from the sandbox
#15 237.6 external/arrow/cpp/src/arrow/compute/exec/key_map_avx2.cc:18:10: fatal error: immintrin.h: No such file or directory
#15 237.6    18 | #include <immintrin.h>
#15 237.6       |          ^~~~~~~~~~~~~
#15 237.6 compilation terminated.
#15 238.0 Target //tfx_bsl:move_generated_files failed to build

Full log

tfx1100_py38.txt
tfx1100_py39.txt
tfx1100_py310.txt
tfx1100_py311.txt

@AseiSugiyama Please take a look at this PR and verify it works on your machine https://github.com/diyor28/tf-aarch64/pull/11. After that let me know if we can close this issue

Okay, let me check this.

Great! I tested #11 and it works!
Let me explain the way to test the TFDV builds.
Setup
I changed the locations of the artifacts (logs and wheel files) to avoid disabling the security feature of macOS. I used this workaround.
Build result
Passed

Python: 3.8, 3.9
TFX: 1.9.0, 1.10.0 (also, 1.10.x)

Failed

Python: 3.10
TFX: 1.9.0, 1.10.0

This is another issue we should solve. Let's skip it now.
Test wheel file
Setup container
I used python:3.8 for testing environment.
TFDV=tensorflow_data_validation-1.10.0-cp38-cp38-linux_aarch64.whl
TFX_BSL=tfx_bsl-1.10.1-cp38-cp38-linux_aarch64.whl
docker run -v /tmp/tf_aarch64/volumes/builds/$TFDV:/$TFDV -v /tmp/tf_aarch64/volumes/builds/$TFX_BSL:/$TFX_BSL -it python:3.8 bash

Then, I installed wheel files as followings;
pip install -U pip
pip install tfx_bsl-1.10.1-cp38-cp38-linux_aarch64.whl
pip install tensorflow_data_validation-1.10.0-cp38-cp38-linux_aarch64.whl

Also, I update numpy because we use numpy with C API version 0x10 in the build environment. C API 0x10 means version 1.23+ (currently)
pip install numpy~=1.23

We can check the relationship between numpy versions and C API versions of numpy from here. For more detail, check this comment.
Note: I cannot use wheel files of Python 3.8 and TFX 1.10 because of the dependency. (TFDV 1.10 requires tfx-bsl 1.10.1+)
Test file and test methods
I used a test dataset (subset of taxi dataset) and tfdv.generate_statistics_from_csv() to test the wheel file as followings.
curl https://gist.githubusercontent.com/kennysong/b9f3ffe1299d331cdde92c46504a5ea1/raw/832b6a958dec86ce313d5be3033e281a87c21847/taxi_2020_test_converted.csv > taxi.csv
python -c "import tensorflow_data_validation as tfdv; print(tfdv.generate_statistics_from_csv('taxi.csv'))"

@AseiSugiyama Great to hear! I will update tfdv1.10.0 build to use tfx1.10.1 and numpy 1.23+


How to fit Swift3.0

New Issue Checklist
Latest version upgrade to 3.0, after a lot of mistakes.
🚫 If this template is not filled out your issue will be closed with no comment. 🚫

[ ] I have looked at the Documentation
[ ] I have read the F.A.Q.
[ ] I have filled out this issue template.

Issue Info

Info
Value

Platform
e.g. ios/osx/tvos

Platform Version
e.g. 8.0

SnapKit Version
e.g. 0.19.0

Integration Method
e.g. carthage/cocoapods/manually

Issue Description
⚠ Replace this with the description of your issue. ⚠

Closing as this issue does not conform to the issue template and does not contain enough information.


[MAINT]: npm run build is failing

Describe the need
see https://github.com/octokit/auth-callback.js/actions/runs/9470231822/job/26090632310
@wolfy1339 I think you did the update to the build scripts across @octokit. Does that one ring any bells?
SDK Version
No response
API Version
No response
Relevant log output
Run npm run build

><EMAIL_ADDRESS>build
> node scripts/build.mjs && tsc -p tsconfig.json

Error: src/auth.ts(3,1): error TS1287: A top-level 'export' modifier cannot be used on value declarations in a CommonJS module when 'verbatimModuleSyntax' is enabled.
Error: src/hook.ts(11,10): error TS1286: ESM syntax is not allowed in a CommonJS module when 'verbatimModuleSyntax' is enabled.
Error: src/hook.ts(13,1): error TS1287: A top-level 'export' modifier cannot be used on value declarations in a CommonJS module when 'verbatimModuleSyntax' is enabled.
Error: src/index.ts(1,10): error TS1286: ESM syntax is not allowed in a CommonJS module when 'verbatimModuleSyntax' is enabled.
Error: src/index.ts(2,10): error TS1286: ESM syntax is not allowed in a CommonJS module when 'verbatimModuleSyntax' is enabled.
Error: src/index.ts(15,1): error TS1287: A top-level 'export' modifier cannot be used on value declarations in a CommonJS module when 'verbatimModuleSyntax' is enabled.
Error: src/types.ts(1,13): error TS1286: ESM syntax is not allowed in a CommonJS module when 'verbatimModuleSyntax' is enabled.
Error: src/with-authorization-prefix.ts(6,1): error TS1287: A top-level 'export' modifier cannot be used on value declarations in a CommonJS module when 'verbatimModuleSyntax' is enabled.
Error: Process completed with exit code 2.

Code of Conduct

[x] I agree to follow this project's Code of Conduct

Ah, yes, this package wasn't migrated to ESM, and the @octokit/tsconfig version was bumped to the one for ESM


[Feature]: Suite2p multiple optical channels

What would you like to see added to NeuroConv?
Some datasets might have multiple optical channels, for example a second channel with certain genetic tags. The F.npy and Fneu.py would have additional files called F_chan2.npy and Fneu_chan2.npy. No spks_chan2.npy as far as I know.
Here's an example sample dataset: https://gitlab.com/fleischmann-lab/calcium-imaging/calimag/-/tree/master/data/v2021_05-2p-Simon/suite2p
A "trimmed" version can be found here: https://gitlab.com/fleischmann-lab/calcium-imaging/calimag/-/tree/master/data/v2021_05-2p-Simon/suite2p_trim
suite2p
├── plane0
│   ├── F.npy
│   ├── F_chan2.npy
│   ├── Fneu.npy
│   ├── Fneu_chan2.npy
│   ├── iscell.npy
│   ├── ops.npy
│   ├── redcell.npy
│   ├── spks.npy
│   └── stat.npy
├── plane1
│   ├── F.npy
│   ├── F_chan2.npy
│   ├── Fneu.npy
│   ├── Fneu_chan2.npy
│   ├── iscell.npy
│   ├── ops.npy
│   ├── redcell.npy
│   ├── spks.npy
│   └── stat.npy
└── plane2
    ├── F.npy
    ├── F_chan2.npy
    ├── Fneu.npy
    ├── Fneu_chan2.npy
    ├── iscell.npy
    ├── ops.npy
    ├── redcell.npy
    ├── spks.npy
    └── stat.npy

Is your feature request related to a problem?
No response
Do you have any interest in helping implement the feature?
Yes, but I would need guidance.
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] Have you ensured this bug was not already reported?

transfered to https://github.com/catalystneuro/roiextractors/issues/234

@tuanpham96 I'll keep this one open - we'd also like to check the NeuroConv metadata looks good for this feature after ROIExtractors is able to read them

@weiglszonja @pauladkisson @alessandratrapani What do you all think?
Can we close this issue?


Bugfix/flag replacement

Bug Fixes

Fixed incorrectly python_variable->cli_option translated names. Addresses #110.

Breaking Changes/Features
The following outlines incredibly-minor breaking changes:

Introduces a new exception: UnknownOptionError.
Change negative-flag-assignment-exception from ValidationError to generic CycloptsError.

e.g. --no-flag=True now raises CycloptsError instead of ValidationError.
This may only impact people who were specifically catching a ValidationError for this in a meta-app. But this is such an esoteric scenario that I imagine it doesn't exist in the wild.

Change exception for unknown option token from ValidationError to UnknownOptionError.

I also imagine this is a rare currently-in-the-wild scenario; anyone attempting to catch this exception in a meta-app is probably catching a CycloptsError instead of a ValidationError.

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (9ee9559) 95.80% compared to head (ae8fa62) 95.41%.

Files
Patch %
Lines

cyclopts/exceptions.py
75.00%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #111      +/-   ##
==========================================
- Coverage   95.80%   95.41%   -0.39%
==========================================
  Files          16       16
  Lines        1739     1745       +6
  Branches      439      440       +1
==========================================
- Hits         1666     1665       -1
- Misses         37       45       +8
+ Partials       36       35       -1

Flag
Coverage Δ

unittests
95.30% <80.00%> (-0.33%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Auth doesn't work with non default local.userEndpoint

Auth doesn't work when using a custom userEndpoint (app.configure(authentication({local: {userEndpoint: '/api/users'}}))). Confirmed in 2 cases:

with a custom idField on feather-memory service
with feathers-nedb (simply app.use('users', nedbService({Model: db})))
didn't test other DBs

A gist with feathers-memory: https://gist.github.com/juodumas/67793c57ebbbde0e5e606b43675dea5e
When using feathers-rest, server log shows:
TypeError: Cannot read property 'get' of undefined
    at <...>/feathers-authentication/lib/hooks/populate-user.js:30:45

When using sockets, app.authenticate() is rejected and returns the hook object.

@juodumas you need to specify your idField in your authentication config.
app.configure(authentication({
  idField: 'id',
  userEndpoint: '/api/users'
}

You can find all the config options here: http://docs.feathersjs.com/authentication/readme.html. Also make sure you are using v0.7 as some of those options had changed from v0.5.

No, that's not it. Just try setting up feathers-nedb auth with any custom userEndpoint.

@juodumas ok well if you are using nedb then your idField needs to be _id. You said you were using feathers-memory.

I think in order to help out I'd need to see more code. If you want to fire a link to the app or add a gist that would be great.

The key was putting userEndpoint at the top level of auth options, just as you said initially. Missed it, sorry.
Would it be possible for feathers to be more verbose about such errors to help later newcomers like me? Right now when I incorrectly set local: {userEndpoint: '/api/users'}, app.authenticate() got rejected and returned the following object over sockets, but otherwise feathers was silent. Only later when testing did I add feathers-rest and saw an exception in the server log. Why is an exception thrown when using rest and not thrown when using sockets?
{
  "hook": {
    "data": {
      "id": "admin",
      "email": "admin",
      "password": "$2a$10$eWUbRA52A2H/899dnNPPru2EpMf8cFqNiw8KK5I24SrJaz6.9MBfK",
      "_id": "2e42YoJFgqRHo6AB"
    },
    "params": {
      "data": {
        "id": "admin",
        "email": "admin",
        "password": "$2a$10$eWUbRA52A2H/899dnNPPru2EpMf8cFqNiw8KK5I24SrJaz6.9MBfK",
        "_id": "2e42YoJFgqRHo6AB"
      }
    },
    "method": "create",
    "type": "after",
    "result": {
      "_id": "2e42YoJFgqRHo6AB",
      "token": "eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJfaWQiOiIyZTQyWW9KRmdxUkhvNkFCIiwiaWF0IjoxNDYwNzA1MTk2LCJleHAiOjE0NjA3OTE1OTYsImlzcyI6ImZlYXRoZXJzIn0.u6-2eu4FHfwAyP4_4_QAOudhMJf3hsBGWUj1b03EIvc"
    }
  }
}

Thanks for the replies.


Show error message next to the field with invalid input instead of at the bottom

When inputing in the top fields, such as thumbnail URL, user might not be aware of the error message until they scroll down, and even then they might not see it.

Thanks @WrathfulSpatula! 🎉


[Android] Fix the failed android tests on M49.

For OnReceivedErrorTest#testCacheMiss, the related resource id was
removed in
https://github.com/crosswalk-project/crosswalk/commit/29838edb08b71e49c43eff580ac02f4709d1cfaa, this leads to the description is null.
For SetInitialScaleTest#testSetInitialScale1, the clobber initial scale
should works in quirks mode, please refer to https://codereview.chromium.org/67193002,
but crosswalk has no API to set quirks mode. Meanwhile the previous
CriteriaHelper.poll* did not check the result inside function, it was fixed in
https://codereview.chromium.org/1491063002, then initial scale issue was
exposed.
For OnPageFinishedTest#testOnPageFinishedNotCalledForJavaScriptUrl, it
was caused by https://codereview.chromium.org/1304373007, the return
value of GetLoadingUrl() was reset to data url instead of unreachable
url.
BUG=XWALK-6533

@rakuco @lincsoon @mrunalk @axinging @fujunwei PTAL, thanks.

LGTM

LGTM

For the record, the testOnPageFinishedNotCalledForJavaScriptUrl bit was a regression in M49 that was fixed by https://codereview.chromium.org/1806853005 (part of the new M49 stable release I'm working on), so I had to revert the check in OnPageFinishedTest.java to the previous one.


(Temporary?) fix for failing tests...

This PR is for review.. it fixes the failing tests on 6.3.x but IDK if the fix just masks the issue :(
@ederign @porcelli Any opinions?

I don't want to fiddle with the (possible) rogue commit: https://github.com/droolsjbpm/kie-uberfire-extensions/commit/17bcc2019582cc7e0a3454539abd51088e59c1c6 as it does "special" things for WAS...

@manstis I thought about that yesterday, but I'm a little bit afraid of masking the real issue because the error is not a Social error but a "File system is closed error" when this tests creates a social event that will be persisted.

Closed as this masks the issue.

@manstis  the question is: is that a FS issue or a error in test FS setup?

@manstis one think that I've noticed yesterday is that they use:
fs.forceAsDefault(); -> For a .SimpleFileSystemProvider but in the FS is closed is a JGit one.
(investigating).


Extract common / repetitive code from instrumentations into a new common library

Which component is this bug for?
All Packages
📜 Description
Common things we have:

[ ] OpenTelemetry Metrics creation
[ ] Error reporting
[ ] Checking if prompt logging is allowed

Are you willing to submit PR?
None

Hey, can I work on this issue ?

Yes definitely @ishaanagw!

Can you help me with one example of the code that is repetitive ?

This one for example
https://github.com/traceloop/openllmetry/pull/1957#discussion_r1767317525

So would you want one library for all the common the common code or a different library for OpenTelemetry Metrics creation and a different one for  Checking if prompt logging is allowed, and so on ?

One library to rule them all. Maybe we can even put it in the ai semconv library we already have

okay, got it. Thanks

is it still open for contribution?


Use juju secrets to exchange the JWT and MUNGE keys between Slurm nodes

Currently the Slurm charms are exchanging both the JWT and MUNGE keys through Juju relations, but as of Juju v3, we can now use Juju Secrets to exchange sensitive information between applications rather than as plain text through relation data bags. We should use Juju's secrets implementation to exchange the JWT and MUNGE keys between our Slurm applications as it provides several benefits over our current implementation.
Juju Secrets offer the ability to control which applications are allowed to access specific secrets, and it is easier to rotate out stale secrets using SecretRotateEvent. You can also set expiration dates for secrets which enables us to determine how long a MUNGE key should be in use. Juju Secrets also enable us to more securely handle these keys within the charms.
The only downside to Juju Secrets is that anyone who has a Slurm cluster on Juju v2 will be unable to automatically refresh to the secrets-enabled Slurm charms. They will need to redeploy their Charmed HPC cluster.

100% behind this!


i18n(ru): update some guides

Update:

ru/guides/pages.mdx
ru/guides/css-and-tailwind.mdx

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

ru
guides/css-and-tailwind.mdx
Localization changed, will be marked as complete.

ru
guides/pages.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


confirmation page for recordset looks broken/unfinished after multi-record insert

...at least in some circumstances perhaps due to the use of pseudo-columns in the compact context of the table on which the insert is being performed. But this is far from a "corner" case as many of our useful tables are annotated to include pseudo-columns.

In the above example, the

columns "Experiment Type" and "Protocol" are outbound foreign key references to denormalize the user readable "name" or "title" of the referenced entity -- they remain blank even though "Experiment Type" had been selected (Protocol had not been)
columns "Species" through "Genotype" are pseudo-columns that make two outbound foreign key references to get to the user readable "name" for the corresponding terms -- at least "Species" should have a value (in the reference table) but all 4 of these columns remain blank and have infinite spinners
"Local Id" may be ignored as it was a regular column that was left blank.

This is a duplicate of the newly created #2017 issue. Since the newer one has more information I am going to close this one.