whats_new plugin: error when packages get

When I try to add the flutter_whatsnew (v:0.3.0) package to my project (SKD version: 1.5.4) dependencies I receive this error:
PS myApp> flutter packages get
Invalid argument(s): Minimum version ("1.0.0") must be less than maximum ("1.0.0-0").
package:pub_semver/src/version_range.dart 67:7                            new VersionRange
package:pub/src/solver/reformat_ranges.dart 61:27                         _reformatTerm
package:pub/src/solver/reformat_ranges.dart 39:26                         reformatRanges.<fn>
dart:_internal                                                            ListIterable.toList
package:pub/src/solver/reformat_ranges.dart 40:12                         reformatRanges
package:pub/src/solver/reformat_ranges.dart 33:27                         reformatRanges
package:pub/src/solver/reformat_ranges.dart 33:27                         reformatRanges
package:pub/src/solver/reformat_ranges.dart 33:27                         reformatRanges
package:pub/src/solver/reformat_ranges.dart 33:27                         reformatRanges
package:pub/src/solver/reformat_ranges.dart 33:27                         reformatRanges
package:pub/src/solver/version_solver.dart 308:24                         VersionSolver._resolveConflict
package:pub/src/solver/version_solver.dart 131:27                         VersionSolver._propagate
package:pub/src/solver/version_solver.dart 96:9                           VersionSolver.solve
===== asynchronous gap ===========================
package:pub/src/solver.dart 35:10                                         resolveVersions.<fn>
package:pub/src/log.dart 378:18                                           progress
package:pub/src/solver.dart 32:10                                         resolveVersions
package:pub/src/entrypoint.dart 203:24                                    Entrypoint.acquireDependencies
===== asynchronous gap ===========================
package:pub/src/command/get.dart 41:23                                    GetCommand.run
package:args/command_runner.dart 194:27                                   CommandRunner.runCommand
===== asynchronous gap ===========================
package:pub/src/command_runner.dart 171:39                                PubCommandRunner.runCommand.<fn>
dart:async                                                                new Future.sync
package:pub/src/utils.dart 109:12                                         captureErrors.<fn>
dart:async                                                                runZoned
package:pub/src/utils.dart 126:5                                          captureErrors
package:pub/src/command_runner.dart 171:13                                PubCommandRunner.runCommand
===== asynchronous gap ===========================
package:pub/src/command_runner.dart 119:11                                PubCommandRunner.run
===== asynchronous gap ===========================
C:\b\s\w\ir\k\src\third_party\dart\third_party\pkg\pub\bin\pub.dart 8:22  main
This is an unexpected error. Please run
    pub --trace '--verbosity=warning' get --no-precompile
and include the logs in an issue on https://github.com/dart-lang/pub/issues/new
Running "flutter packages get" in mes-mobile-flutter...
pub get failed (1)

This is my pubspec.yaml:
name: myapp_flutter
description: My app

version: 0.4.0+1

environment:
  sdk: ">=2.1.0 <3.0.0"

dependencies:
  flutter:
    sdk: flutter

  cupertino_icons: ^0.1.2
  http: ^0.12.0+1
  intl: 0.15.7
  local_auth: 0.3.1
  random_string: ^0.0.2
  shared_preferences: ^0.5.1+1
  queries: ^0.1.9
  flutter_markdown: ^0.2.0
  xml: ^3.3.0
  #cached_network_image: ^0.7.0
  flutter_advanced_networkimage: ^0.4.15
  photo_view: ^0.3.1
  percent_indicator: ^1.0.16
  rxdart: ^0.22.0
  flutter_whatsnew: ^0.3.0
  package_info: ^0.4.0+3

dev_dependencies:
  flutter_launcher_icons: "^0.7.0"
  flutter_test:
    sdk: flutter

flutter_icons:

  ios: true
  android: "launcher_icon"
  image_path: "assets/launcher/icon.png"

flutter:

  uses-material-design: true

  assets:
    - assets/logo.png
    - assets/logo_white.png
    - assets/app_logo.png
    - assets/changelog.md

Please update to the latest version and test again! Let me know if it is still an issue

Solved! Thank you for your support


Make some attributes of AppStoreVersionLocalization optional

Encountered following error when trying to add new language support:

[2022-07-15 05:51:11.999] [app_store] [ERROR] Cannot deserialize '<class 'NoneType'>' to '<class 'str'>' for 'typing.List[asconnect.models.app_store_version_localizations.AppStoreVersionLocalization][1].attributes.description'

These attributes could be None when a language is newly added.

Thanks!


fix incorrect gutter padding to *-uncollapse classes

Here is a proposed fix to issue #7452. In addition, it now accepts a $gutter parameter so that columns with non-default gutter value can make use of this mixin.

In a project I'm working on in which there is a nested row with the *-uncollapse class applied, I still am seeing the unaltered (double) padding being applied (i.e., in my _settings where there is a 30px $grid-column-responsive-gutter for the medium breakpoint ... this value is being applied on my medium-uncollapse child columns instead of the rem_calc(30px)/2). Any ideas why this might happen?

I'm having the same issue.
See screenshot below. Default gutter set to 24px resulting in 0.75rem (12px) column gutters.
When using uncollapse, padding is restored using the full 24px gutter width.


Reworked Questions.

Syntax need to be the same in every questions. (Spaces, punctuation, uppercase, etc.)
Category are for now : Anime, Arts, Biology, Games, Geography, History, Literatures, Movies, Musics, Other, Riffles, Science, Sports.
Remaining questions is updated on my fork, but not merged.

Oh nice~

The comma after the last item should be removed

In left_questions.json, you forgot a lot of commas after the category:
{
    "Category": "Biology"
    "Question": "What wreath of flowers is used as a decoration?",
    "Answer": "Garland"
},

->
{
    "Category": "Biology",
    "Question": "What wreath of flowers is used as a decoration?",
    "Answer": "Garland"
},

Sounds good, the question is, will it parse? :D
I'll check it out tonight

Oh, yes, my bad x) I did reorganize questions, and forgot to remove the last coma.
The "left_question.json" is not "ready to use", so it missing lot of thing. (In true, i didn't want to merge it, but, since I'm new in Github, I think I've done something wrong...)
I'll focus on my studies, so I'll not work on questions until my exams are done.
I apologize for this inconvenience.

@Bakaba how much of the original questions are converted?

Hm, approximately 700 questions.

Out of?

~5000...

PR dev branch from now on


error when trying to force download

Hi mate!
first, thanks for the app.
I got it running via docker and the app starts, I can update the settings and add streamers, so far so good. But it seems when a stream start the app does not record it and when I try to force record I get the following error:
Slim Application Error
The application could not run because of the following error:

Details
Type: Error
Code: 0
Message: Call to a member function getTimestamp() on bool
File: /var/www/twitchautomator/src/TwitchVOD.php
Line: 991
Trace
#0 /var/www/twitchautomator/src/TwitchVOD.php(192): App\TwitchVOD->parseChapters(Array)
#1 /var/www/twitchautomator/src/TwitchVOD.php(255): App\TwitchVOD->load('/var/www/twitch...')
#2 /var/www/twitchautomator/src/TwitchAutomator.php(578): App\TwitchVOD->refreshJSON()
#3 /var/www/twitchautomator/src/TwitchAutomator.php(365): App\TwitchAutomator->capture(Array)
#4 /var/www/twitchautomator/src/TwitchAutomator.php(205): App\TwitchAutomator->download(Array)
#5 /var/www/twitchautomator/src/Routes/routes.php(80): App\TwitchAutomator->handle(Array)
#6 /var/www/twitchautomator/vendor/slim/slim/Slim/Handlers/Strategies/RequestResponse.php(43): Closure->{closure}(Object(Slim\Psr7\Request), Object(Slim\Psr7\Response), Array)
#7 /var/www/twitchautomator/vendor/slim/slim/Slim/Routing/Route.php(381): Slim\Handlers\Strategies\RequestResponse->__invoke(Object(Closure), Object(Slim\Psr7\Request), Object(Slim\Psr7\Response), Array)
#8 /var/www/twitchautomator/vendor/slim/slim/Slim/MiddlewareDispatcher.php(81): Slim\Routing\Route->handle(Object(Slim\Psr7\Request))
#9 /var/www/twitchautomator/vendor/slim/slim/Slim/MiddlewareDispatcher.php(81): Slim\MiddlewareDispatcher->handle(Object(Slim\Psr7\Request))
#10 /var/www/twitchautomator/vendor/slim/slim/Slim/Routing/Route.php(341): Slim\MiddlewareDispatcher->handle(Object(Slim\Psr7\Request))
#11 /var/www/twitchautomator/vendor/slim/slim/Slim/Routing/RouteRunner.php(84): Slim\Routing\Route->run(Object(Slim\Psr7\Request))
#12 /var/www/twitchautomator/vendor/slim/slim/Slim/Middleware/RoutingMiddleware.php(60): Slim\Routing\RouteRunner->handle(Object(Slim\Psr7\Request))
#13 /var/www/twitchautomator/vendor/slim/slim/Slim/MiddlewareDispatcher.php(140): Slim\Middleware\RoutingMiddleware->process(Object(Slim\Psr7\Request), Object(Slim\Routing\RouteRunner))
#14 /var/www/twitchautomator/vendor/slim/twig-view/src/TwigMiddleware.php(125): class@anonymous->handle(Object(Slim\Psr7\Request))
#15 /var/www/twitchautomator/vendor/slim/slim/Slim/MiddlewareDispatcher.php(140): Slim\Views\TwigMiddleware->process(Object(Slim\Psr7\Request), Object(class@anonymous))
#16 /var/www/twitchautomator/vendor/slim/slim/Slim/Middleware/ErrorMiddleware.php(107): class@anonymous->handle(Object(Slim\Psr7\Request))
#17 /var/www/twitchautomator/vendor/slim/slim/Slim/MiddlewareDispatcher.php(140): Slim\Middleware\ErrorMiddleware->process(Object(Slim\Psr7\Request), Object(class@anonymous))
#18 /var/www/twitchautomator/vendor/slim/slim/Slim/MiddlewareDispatcher.php(81): class@anonymous->handle(Object(Slim\Psr7\Request))
#19 /var/www/twitchautomator/vendor/slim/slim/Slim/App.php(215): Slim\MiddlewareDispatcher->handle(Object(Slim\Psr7\Request))
#20 /var/www/twitchautomator/vendor/slim/slim/Slim/App.php(199): Slim\App->handle(Object(Slim\Psr7\Request))
#21 /var/www/twitchautomator/public/index.php(136): Slim\App->run()
#22 {main}

Only after stopping the container and deleting the created json file ind the folder of the streamer the dashboard works again. Any help would be much appreciated!
edit: It just caught my eye that the image on dockerhub doesn't seem to be up2date (last updated 1 month ago even though this repo has commits to it 2 days ago). Maybe this has already been fixed and updating the image would be enough. Also using semantic versioning in addition to the latest tag would be nice on dockerhub :)

yeah pushing docker images was just too much for me, you should be able to build the image manually though. but you're gonna have to add cronjobs manually too, since i couldn't find a solution for that either. basically, docker is just annoying for this

Hi, thanks for answering so fast.
As for building and pushing docker images: It's super easy, you just need 3 commands in your terminal.
docker login --username=<dockerhub username>
docker build -t mrbrax/twitchautomator:latest -f Dockerfile .
docker push mrbrax/twitchautomator:latest

Since most people will run this on a NAS or something similar and nowadays everyone just runs docker images because it doesn't get more comfortable than that.
As for the cron jobs: Can't really help you with that but there are enough docker images of applications around that do scheduled tasks. Don't know how they do it exactly but it can be done.

i've checked out docker a bit more, it seems that the alpine linux image doesn't have php 7.4 yet, so i'll have to wait for that

Hi, sorry for the late reply.
Looked at the Dockerfile: You could just use alpine as the base image and install nginx + php 7.4. I'm sure there are other Dockerfiles around where people do exactly that. Could just look how they do it but it shouldn't really that complicated.

Fixed in 5f80469d53964a8d4df733bd5dafd4875bf2e537


fix citation in README

Very minor, but the Flash Attention 2 citation was missing a closing brace on the title (resulting in an error when copy/pasting into a paper 🙂 )

Thanks!


Separate Integration Testing

Motivation

Split all CIs for each integration module #30
Description of the changes

Integration-Specific Test Modules

@DanielAvdar Thank you for this PR, but as this PR involves a lot of changes in CI, our team decided to solve this issue by ourselves.
Sorry for the inconvenience, we should have mentioned this in the issue description, but we are always welcome to see any PRs for the issues with the contribution welcome label.


Add an option to use battery design capacity

This little PR implements a new option design-capacity to the battery module with default value false. If set to true, Waybar will use the design capacity of the battery instead of the actual full capacity which may be lower due to deterioration of the battery. With this option, ti should report the same battery percentage than i3status' default behavior.

I will edit the wiki as soon as you merge this PR.

Hi, thanks for the change.
Just a typo change and i'll merge this PR!

Hi, thanks for the change.
Just a typo change and i'll merge this PR!

Sorry this change wasn't intended.
I changed it, the extra commit is because I forgot to exclude my .cache folder

Sorry this change wasn't intended.
I changed it, the extra commit is because I forgot to exclude my .cache folder

LGTM, thx, could you update github wiki?

LGTM, thx, could you update github wiki?

Done,
thanks for merging

Done,
thanks for merging


login with 2fa  not working

Hi,
I've enabled TFA in config
auth_totp=true
but after typing username, password and OTP Key I got "Invalid username and/or password, please try again!"

same here. how can I provide more debugging infos?

To verify, you are able to log in with this user to the Proxmox web
interface?
On Sat, Jan 27, 2024 at 1:12 PM Robin Loose @.***>
wrote:

same here. how can I provide more debugging infos?
—
Reply to this email directly, view it on GitHub
https://github.com/joshpatten/PVE-VDIClient/issues/86#issuecomment-1913299323,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABIP64TWBZNKRUPPQVVAZOLYQVGSNAVCNFSM6AAAAABAAXYXROVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJTGI4TSMZSGM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

correct, doing this alsmost every day. TOTP works fine with realm pve and pam wie web interface

Proxmox changed the API, proxmoxer code is not changed... There is an issue where ive posted the way Proxmox does the OTA with an Bash example... But no response there for weeks... Sadly since its the only Wrapper that is usable (well maybe not anymore)
Thinking of writing my own instead, but have to many projects right now... :-/

I'm going to see if I can make some progress on this today. I didn't
realize that the API changed.
On Sat, Feb 17, 2024, 3:47 PM Adde Lovein @.***> wrote:

Proxmox changed the API, proxmoxer code is not changed... There is an
issue where ive posted the way Proxmox does the OTA with an Bash example...
But no response there for weeks... Sadly since its the only Wrapper that is
usable (well maybe not anymore)
Thinking of writing my own instead, but have to many projects right now...
:-/
—
Reply to this email directly, view it on GitHub
https://github.com/joshpatten/PVE-VDIClient/issues/86#issuecomment-1950391312,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABIP64V4RD7TYEGGAU3M6TLYUEQNLAVCNFSM6AAAAABAAXYXROVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJQGM4TCMZRGI
.
You are receiving this because you commented.Message ID:
@.***>

Since you use Proxmoxer API its in their repo you will need to do the change.
It is the Proxmox API that changed and proxmoxer hasnt updated... Just to be clear. ;-)

I submitted a PR: https://github.com/proxmoxer/proxmoxer/pull/158 to the proxmoxer library. I verified this worked with the VDI client.


K8s views are always present

There is an "internet" node in the services view, which I think is causing them to display.
Seems like the unconnected internet node is not being filtered, but showing up when showing system pods/services.
The report contains no pods or services.
But http://<IP_ADDRESS>:4040/api/topology/pods?system=true shows:
{
   "nodes" : {
      "out-theinternet" : {
         "label_minor" : "Outbound connections",
         "control_node" : "",
         "controls" : {},
         "children" : [],
         "sets" : {},
         "shape" : "cloud",
         "adjacency" : null,
         "latest" : {},
         "stack" : false,
         "edges" : {},
         "pseudo" : true,
         "id" : "out-theinternet",
         "parents" : {},
         "rank" : "",
         "counters" : {},
         "metadata" : {},
         "label_major" : "The Internet"
      }
   }
}

might be that the "is_connected" value of the internet pseudonode is being calculated before the nodes are converted to pods (and dropped).


[Feature Request] Support config.js for dynamic configs

Affected Packages

I think pkg:cli and/or pkg:config
Problem

I do not want to manually update .changesets/config.json. I want to be able to use a config.js file out of the box so that I can write logic to dynamically set properties in my config.
Proposed solution

Nothing specific at the moment. I see that this was supported in v1 but was removed -likely for good reason- but I'd like to know more if possible. I was unable to find anything looking through the changelogs.
Many other tools support both .json and .js files, so I would look there to get a possible solution if this is something that is worth supporting.

Static configs are usually better when it comes to caching and stuff but we are not concerned about this here. I think that I would be OK with adding this. I would like to double-check with @mitchellhamilton though, to hear out his opinion.
Also note that with ESM in node this might be slightly more complicated than implementing require(path.join(projectPath, '.changeset', config.js))

The reasoning behind static configs was primarily to allow tools like https://github.com/changesets/bot to exist and be able to say things like "these are the packages that will be released with these versions" without having to execute user code.
I'd mostly say, what do you want to be dynamic in the config?

Thanks for the reply! We wanted a way to add linked or fixed packages dynamically. Turns out we currently don’t need either, but we may in the future.

Is it an option to run a script before Changesets to regenerate the config? That way you could still have it dynamic while Changesets could see it as static.

Is it an option to run a script before Changesets to regenerate the config?
That's a fair solution that should work for us. Thank you for the input!


Can I set alarms for different locations of the same item in gather windows?

Hi Ottermandias,Thanks for your plugin, it helped me a lot,but i have some problems and need your help
Yes, I want to display multiple Dark Matter Cluster alarms at the same time.
The main reason is that when multiple dark matter clusters enter the active state,gather windows will only display one collection point.
I hope that the same item at different collection points can be displayed separately (if possible)

and can i colsed this one ?I seldom use this function, if I can, I hope it can be folded like gatherbale

Thanks again and be well.

p.s.:The above content is translated by Google, it may be a little strange, please forgive

I've actually had this come up myself as well for items that have many locations where they can be gathered.
For example, the Dark Matter Clusters above, I'd love to be able to track different nodes' uptimes because I'm working towards achievements for gathering a certain number of items in certain zones. Specifically harvesting in Thanalan -- I'd love to go grab this Botany node when it spawns in Drybone, but I don't need clusters badly enough that I want to know about the uptimes on the 20-ish different places it can be gathered.
If I need Lightning Crystals, I'd want to be teleported directed to The Clutch where I'm currently gathering them for achievement credit, but I'm beholden to whatever the list thinks is the "Best Node" with no way to tell it otherwise.


image preview doesn't work in normal image mode while photoswipe is enable

While I enable directive photoswipe in file config.toml, image preview doesn't work in normal image mode
![This is an image in `static/image` folder.](/image/example.jpg)

But images can be previewed via {{
}} shortcode.
{{</* figure src="/image/example.jpg" title="figure image with title" */>}}

Does theme jane doesn't support image preview in normal image mode any more?

This problem is caused by load-photoswipe.js.
If we want to solve it, maybe the best way is:
convert

  <img src="/hugo-theme-jane/image/example.jpg" alt="This is an image in `static/image` folder.">

to

    <img itemprop="thumbnail" src="http://localhost:1313/hugo-theme-jane/image/example.jpg" alt="This is an image in `static/image`">

This is an image in `static/image`

In other words, unified the html code of the image.
What do you think? @Zebradil

Was it supported before we migrated to photoswipe? If we broke something, it's better to fix it. If it worked in the same before the migration, we can leave it as is.

I have fixed this issue.


ADDED CONTRIBUTERS Section

ADDED CONTRIBUTERS Section
fixes #1771
AFTER :

GLANCE at CONTRIBUTERS Section :

Pull Request Checklist

[x] I have added screenshots and videos to show before and after the working of my code.
[x] I have ensured that the screen size is set to 100% while making the video.
[x] I have synced the latest fork with my local repository and resolved any conflicts.
[x] I have mentioned the issue number which I created before making this PR .(format to mention issue number is : fixes #(issue number) )
[x] I understand that if any the above conditions are not met , my PR will not be MERGED .

@ANSHIKA-26 Please Review My Changes and Merge it by Assigning Labels to it !!!

@PatelHarsh2006  you did not wait for the issue to be assigned to you , i have closed several issues related to contributors section in wordwise , we had one earlier and did not like it ,it doesnt go with out overall functionality and hence i do not wish to implement this , KINDLY wait for the issue to be assigned before starting to work on it !

https://github.com/ANSHIKA-26/WordWise/issues/435
please read this


[libyuv] update to latest commit

Fix #17288
Update libyuv to the latest commit 2021-04-15
Note: no feature need to test

Perf, thanks @JonLiu1993


Feature request: i18n

👋 Hi there.
Nice and polished library you've created here, kudos!
One feature I think it's currently lacking is the ability to internationalize texts.
That's in my opinion the main thing that will prevent someone like myself from relying on it on future projects.
I'd also stay with english localization as defaults.
What do you think?

Hello @arnaud
This is a great suggestion!
If you have the time and can develop this feature, it would be great!
Otherwise, as soon as we have some time here we will arrange for the development of this functionality! 👍

I just saw that @robert already sent a PR #2  solving this and adding the prop "labels".
This property can now be informed, changing the field labels.
It is not standardized in English, but for now I think it solves. We can reopen in the future if applicable =)


Segmentation Faults in pachd and Pipeline storage Containers

What happened?:
I have a single pipeline deployed for performance testing. The glob pattern is "/", so the entire input repo is treated as a single datum.  The code in the pipeline simply copies all of the files from the input repo to the output repo. My performance test puts 10,000 files at a time into the input repo using "pachctl put file", waits for the pipeline to complete, then puts another 10,000 files into the input repo. It does this until a total of 200,000 files exist in the repo, at which point it stops. Therefore, the pipeline will run a total of 20 times, each time with more files than the previous run.
I started a run of these tests last night, and checked the results this morning. I discovered that both pachd and my pipeline's storage container had triggered restarts due to segmentation faults, not just once, but multiple times:
{pach 2.0.0-beta.1 : aws} / {k8s 1.16.13-dispatcher : aws} / {git 2.24.3 : pachyderm/master}
~/Documents/Work/Repos/pachyderm -->> kubectl get pods
NAME                           READY   STATUS    RESTARTS   AGE
etcd-0                         1/1     Running   0          12h
pachd-6ffcddcfbc-7cm6v         1/1     Running   13         12h
pipeline-benchtests-v3-bd945   2/2     Running   11         11h
postgres-0                     1/1     Running   0          12h

From the end of the previous pachd container log:
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x10 pc=0x1dd8c77]

goroutine 162 [running]:
github.com/pachyderm/pachyderm/v2/src/internal/metrics.(*Reporter).internalMetrics.func1(0xc0003ad680, 0x295b178, 0xc001972b00, 0xc000713dd8, 0xc001972cc0, 0x0)
	/Users/nitin/Code/Pachyderm/pachyderm/src/internal/metrics/metrics.go:264 +0x157
github.com/pachyderm/pachyderm/v2/src/internal/metrics.(*Reporter).internalMetrics(0xc0003ad680, 0xc000713dd8)
	/Users/nitin/Code/Pachyderm/pachyderm/src/internal/metrics/metrics.go:364 +0x148
github.com/pachyderm/pachyderm/v2/src/internal/metrics.(*Reporter).reportClusterMetrics(0xc0003ad680)
	/Users/nitin/Code/Pachyderm/pachyderm/src/internal/metrics/metrics.go:149 +0x8e
created by github.com/pachyderm/pachyderm/v2/src/internal/metrics.NewReporter
	/Users/nitin/Code/Pachyderm/pachyderm/src/internal/metrics/metrics.go:46 +0x15b

From the end of the previous pipeline storage container log:
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x10 pc=0x1dd8c77]

goroutine 103 [running]:
github.com/pachyderm/pachyderm/v2/src/internal/metrics.(*Reporter).internalMetrics.func1(0xc000561c80, 0x295b178, 0xc007031140, 0xc000461dd8, 0xc007031300, 0x0)
	/Users/nitin/Code/Pachyderm/pachyderm/src/internal/metrics/metrics.go:264 +0x157
github.com/pachyderm/pachyderm/v2/src/internal/metrics.(*Reporter).internalMetrics(0xc000561c80, 0xc000461dd8)
	/Users/nitin/Code/Pachyderm/pachyderm/src/internal/metrics/metrics.go:364 +0x148
github.com/pachyderm/pachyderm/v2/src/internal/metrics.(*Reporter).reportClusterMetrics(0xc000561c80)
	/Users/nitin/Code/Pachyderm/pachyderm/src/internal/metrics/metrics.go:149 +0x8e
created by github.com/pachyderm/pachyderm/v2/src/internal/metrics.NewReporter
	/Users/nitin/Code/Pachyderm/pachyderm/src/internal/metrics/metrics.go:46 +0x15b

Now, I did discover a flaw in the way that my testing waits for the pipeline to complete, as a result of which it occasionally proceeds to upload the next set of files while the pipeline is still running. Obviously, that's an issue with my test, but a side effect of that is that the pipeline was sometimes running multiple jobs concurrently. In fact, that's the state I found things in this morning when I did a "list job":
ID                               PIPELINE   STARTED      DURATION           RESTART PROGRESS  DL       UL       STATE
3ba46c8bed964514a244cf5cfbf0f260 benchtests 2 hours ago  -                  0       0 + 0 / 0 0B       0B       running
138883d672594ad88589dfcbe8f37807 benchtests 2 hours ago  -                  2       0 + 0 / 0 0B       0B       running
ae05806f782847aca7132db89eea674c benchtests 2 hours ago  2 minutes          0       1 + 0 / 1 995.6MiB 995.6MiB success
7203a7ef9af34984b5e85e8c05ab12a8 benchtests 3 hours ago  31 minutes         0       1 + 0 / 1 928.7MiB 928.7MiB success
34495b8303524d7abdfff6baafa417c8 benchtests 4 hours ago  2 minutes          0       1 + 0 / 1 862.3MiB 862.3MiB success
8d861a01bfdc4da6a9e38c73718a9982 benchtests 11 hours ago About an hour      2       1 + 0 / 1 795.9MiB 795.9MiB success
fd4e68a4da884a899e1051a5897644b6 benchtests 11 hours ago About a minute     0       1 + 0 / 1 729.5MiB 729.5MiB success
d4d235cc130f4e8ab3e25333410bc987 benchtests 11 hours ago About a minute     0       1 + 0 / 1 663.1MiB 663.1MiB success
74d034c621eb48f6b9e60b03fe4753eb benchtests 11 hours ago About a minute     0       1 + 0 / 1 596.7MiB 596.7MiB success
759fc9ac2b6f4a6cb2646425c7bb7a43 benchtests 11 hours ago About a minute     0       1 + 0 / 1 530.3MiB 530.3MiB success
506e8274b62d4a50a244d4ae2a520ce4 benchtests 11 hours ago 56 seconds         0       1 + 0 / 1 464MiB   464MiB   success
d4fb7105044d4faabb9a3198a78527f0 benchtests 11 hours ago 49 seconds         0       1 + 0 / 1 397.9MiB 397.9MiB success
255dd9bce5c6466e8891a34444779a85 benchtests 11 hours ago 42 seconds         0       1 + 0 / 1 331.6MiB 331.6MiB success
4a0a46c539c94768b33133287be222ea benchtests 11 hours ago 33 seconds         0       1 + 0 / 1 265.2MiB 265.2MiB success
d666cf61a82248b48483693b3517d1d5 benchtests 11 hours ago 25 seconds         0       1 + 0 / 1 199MiB   199MiB   success
460bc41366f54af49fbfa55239d050e9 benchtests 11 hours ago 16 seconds         0       1 + 0 / 1 132.7MiB 132.7MiB success
8843010dad01438496b1d8e41257d126 benchtests 11 hours ago 1 second           0       0 + 1 / 1 0B       0B       success
a8fce725128841baa68e8aa14000b208 benchtests 11 hours ago 10 seconds         0       1 + 0 / 1 66.39MiB 66.39MiB success
382f882b8db8421eaf973498849983a5 benchtests 11 hours ago 10 seconds         0       1 + 0 / 1 53.07MiB 53.07MiB success
cedccad7e69649008a09cadce249ee25 benchtests 11 hours ago 8 seconds          0       1 + 0 / 1 39.77MiB 39.77MiB success
4ad45f493b5d4eff8d00e6d785cbe200 benchtests 11 hours ago 6 seconds          0       1 + 0 / 1 26.58MiB 26.58MiB success
33839a874e134bcab204a2626dea07ce benchtests 11 hours ago 4 seconds          0       1 + 0 / 1 13.3MiB  13.3MiB  success

As you'll have noticed from my "get pods" listing earlier, I only have one pipeline pod running, so it's not clear to me how multiple jobs of the same pipeline could even be running in parallel at all, but I bring this up in case it might be a contributing factor.
The other thing which may (or may not) be contributing: I didn't do anything special with respect to resource requests, so by default my pipeline containers only requested 64 MiB. However, they also didn't specify any limit, I didn't see any OOM errors, and the pachd container requested 2 GiB and still produced segmentation faults, so perhaps this isn't a contributing factor after all.
Environment?:

Kubernetes version (use kubectl version): v1.20.4-eks-6b7464
Pachyderm CLI and pachd server version (use pachctl version): 2.0.0-beta.1
Cloud provider (e.g. aws, azure, gke) or local deployment (e.g. minikube vs dockerized k8s): AWS
OS (e.g. from /etc/os-release):
Others:

The core issue appears to be a pretty simple issue with the metrics collection which is doing a list pipeline without collecting the details and attempting to access the details in the resulting pipeline infos. As far as the parallel job stuff is concerned, I think it is more likely that we didn't terminate the prior job properly (I see a place where that can happen if the pachd sidecar goes down). Some other weird things are the two jobs that took much longer than the others and the one job that looks like it skipped the datum. We should probably look into this run / workload more to make sure that there is not a separate issue there.

Duplicate of https://github.com/pachyderm/pachyderm/issues/6485


Improve test coverage for PackageJson parsing

TODO: contributors, private, main, bin, directories (hash), repository, config. Also verify that field ordering doesn't make a difference.
related to #320, as we should have caught this in our unit tests

Closing as too open ended.
Any changes the touch parser code should introduce pinning tests around the behavior being changed plus, if the change is a fix, tests to verify that whatever bug has been fixed. This way we build up a library of tests for this code over time, hopefully without accumulating too many unnecessary tests.


Preparing for release 13.13.1 (automated).

Fixes

#2318: Only show Clear search link if there's a search
#2321: Update hint for create page
#2317: Using internal GOV.UK Frontend more widely - this makes error pages, password pages etc. more reliable.

Looks good to me.


Port changes of [#17325] to branch-2.8

What changes are proposed in this pull request?
bin/alluxio fsadmin report capacity
Capacity information for all workers:
    Total Capacity: 10.67GB
        Tier: MEM  Size: 10.67GB
    Used Capacity: 0B
        Tier: MEM  Size: 0B
    Used Percentage: 0%
    Free Percentage: 100%
Format is short: true
Format is %-16s %-15s %-16s %-13s %s %-16s %-40s

Worker Name      State           Last Heartbeat   Storage       MEM              Version          Revision
<IP_ADDRESS>      ACTIVE           166              capacity      10.67GB          2.10.0-SNAPSHOT  ffcb706497bf47e43d5b2efc90f664c7a3e7014e
                                                  used          0B (0%)

What changes are proposed in this pull request?
Please outline the changes and how this PR fixes the issue.
Why are the changes needed?
Please clarify why the changes are needed. For instance,

If you propose a new API, clarify the use case for a new API.
If you fix a bug, describe the bug.

Does this PR introduce any user facing changes?
Please list the user-facing changes introduced by your change, including

change in user-facing APIs
addition or removal of property keys
webui

failed checkstyle
++++ readlink ./dev/scripts/check-docs.sh
++++ echo ./dev/scripts/check-docs.sh
+++ dirname ./dev/scripts/check-docs.sh
++ cd ./dev/scripts
++ pwd
+ SCRIPT_DIR=/usr/src/alluxio/dev/scripts
+ cd /usr/src/alluxio/dev/scripts/src/alluxio.org/
+ GO111MODULE=on
+ go build -o checkdocs check-docs/main.go
go<EMAIL_ADDRESS>unrecognized import path "gopkg.in/yaml.v3" (parse https://gopkg.in/yaml.v3?go-get=1: no go-import meta tags ())
go: error loading module requirements
Error: Process completed with exit code 1.

failed test
Error: 0.148 [ERROR] Errors:
Error: 0.149 [ERROR] alluxio.client.cli.job.DistributedCommandsStatsTest.testAsyncPersistCancelStats
Error: 0.149 [ERROR]   Run 1: DistributedCommandsStatsTest.testAsyncPersistCancelStats:221 » Timeout Timed o...
Error: 0.149 [ERROR]   Run 2: DistributedCommandsStatsTest.testAsyncPersistCancelStats:221 » Timeout Timed o...

alluxio-bot, force-merge this please


remove .idea IDE directory

This directory seems to be specific to IntelliJ IDEA and probably shouldn't be part of the repo. It's already listed in .gitignore, but this way it'll be removed completely.

@Pro75357 Is there a reason to keep .idea directory and not add it to .gitignore? It just seems to add metadata that is useless to anyone who doesn't lose IntelliJ IDEA.


Using Custom Pygments Styles, Take 2 #1121

Problem
The "solution" that was mentioned in Issue (https://github.com/readthedocs/sphinx_rtd_theme/issues/1121) cites looking at the pygments passage of the RTD theme configuration docs for the answer. That entire segment in the documentation currently reads:

The style name to use for Pygments highlighting of source code. If not set, either the theme's default style or 'sphinx' is
selected for HTML output.
Changed in version 0.3: If the value is a fully-qualified name of a custom Pygments style class, this is then used as custom style.

So, if I look only at these brief statements, I could infer that all I need to do to include a custom Pygments style is to add it to pygments_style = ['some_file_name.css'] in the Sphinx project'sconf.py using "a fully-qualified name of a custom Pygments style class".
However, this answer is clearly inadequate, not only because it invalidates my own experience trying to get this to work, but also by negating the ridiculous number of contrasting questions and opinions about this very issue throughout the entire Internet and even within the issues that have been presented on this repo. I had opened this issue initially in hopes of beginning a conversation and continue them on the previous issue, but since that was closed, I will try to ask as many as I can here (some may be entirely irrelevant, though, which could have been ruled out had we been able to have a conversation in that issue):

Is the Pygments style “class” in that description the same as the file name? (In CSS, usually a “class” is one tiny portion of a css file, so this is confusing. If there is somewhere that “class” is defined specifically for the RTD theme and/or Sphinx itself, it might help to either link to that statement within your pygments_style explanation, or change the statement to include something like “. . . of a custom Pygments style class, as defined by “class” here where here represents a link to the definition.
. . . On the other hand, if “class” here means the file name of the css file, then state that in the statement instead of “class”, like “. . . of a custom Pygments style by its filename, including the file extension” or “excluding the file extension”, depending on which is actually what the RTD theme or Sphinx expects.

What is the definition of “fully-qualified” in this statement? For instance, does it just mean that l need to specify a custom Pygments css file in relationship to the html_static_path settings like the ones used by thehtml_css_files = ['css/custom1.css','css/custom2.css',... etc]setting? For example, would this work: pygments_style = ['css/some_file_name.css']? If not, please either provide an example of what “fully-qualified”means in this situation, or explain it more clearly. “Fully qualified” could mean the actual path on the system it’s installed on (i.e, /Volumes/Sphinx_installation_path/, or include the domain name if Sphinx docs are being hosted on a web server such as Jekyll or Hugo or NGINX. There is really no way to know from the existing statement.
For the sake of easy consumption and clarity, these situations should probably be addressed — either individually to clarify how to set the path to custom Pygments themes — and/or the process of adding the custom theme needs to be simplified to work like the other options, i.e., in relationship to the html_static_path.

Does the filename of a custom Pygments style need to be exactly pygments.ext or can it have any filename?
NOTE: I used .ext because in #4, I ask what file types and languages are supported as Pygments styles in RTD/Sphinx — some questions about adding custom Pygments styles on the Internet say that they have found success by installing the pygments_style.py like any other Python package and then just declaring it in the conf.py. This may work in some situations, as it clearly it has for some people — or they are experiencing some odd bug that seems to work properly. Either way, this is confusing and when and why this might work should be addressed.

If I have a Pygments some_file_name.py (or other format) file, does it need to be converted to a css file using something like pygmentize -S [style] -f html -a .parent > [style].css? (Pygments itself does not use css files natively, which is why this question comes up:

(Philosophy: the styles aren’t written in CSS syntax since this way they can be used for a variety of formatters.)
Source: https://pygments.org/docs/styles/)

Stating the format the Pygments file should be in would be helpful information, even if it is obvious to RTD developers.

If RTD does support *.py (or other theme file formats/languages) Pygments styles natively, can they also be placed in a custom folder and specified like in #2 above? For example, would this work: pygments_style = [‘scripts/some_file_name.py‘]? (. . . or however “fully qualified” ends up being defined — the question posed in #2 above.)

Will a custom Pygments style completely replace the default style being used in the RTD Sphinx theme, or overlap it? For instance, if not all styles and classes in a custom Pygments theme are used compared to the default RTD theme pygments style, will those styles shine through? This could potentially affect any aspect of fonts that are displayed, etc, — or any other part of what a pygments style covers. If this is the case, it should be addressed in your description of how to add custom Pygments styles.
One RTD Issue (Choosing pygments monokai style #166) where this arises mentions needing to use a source/_templates/layout.html file with specifications like:
{# layout.html #}
{# Import the theme's layout. #}
{% extends "!layout.html" %}

{% set css_files = css_files + ['_static/pygments.css'] %}

Before specifying the pygments_style = ['css/pygments.css'], and then the custom theme will work.
But another person in that issue then states that no, it will only overwrite parts of the default style, and to fix it, to do something like this:

A not-so-nice workaround:

templates/layout.html
{% extends "!layout.html" %}

{% set css_files = css_files + ['_static/css/reset-syntax.css', '_static/pygments.css'] %}

Then they add a reset like this:
static/css/reset-syntax.css
.highlight * {
   color: black;
   font-weight: normal;
   font-style: normal;
   background-color: transparent;
}

. . . And then that person states that this will supposedly allow a custom Pygments style to actually overwrite the RTD default.
If this is the case, though, and custom Pygments styles require special setup in order to present properly and not allow the default RTD Pygments styles to bleed through, this needs to be mentioned, as does and the officially recommended method of implementation.

How to enable inline code highlighting for a custom Pygments style when used with the RTD theme? Or must these be addressed in a custom css file as mentioned in the Stack Overflow question here: Sphinx RTD theme: Change color of inline code? Or is that only an issue if the custom Pygments style doesn’t include styling  like code.literal {color: #[color] !important; background-color: #[color] !important;}?
That’s all I can think of at the moment. Hopefully a more productive conversation will come from this issue, and anything not mentioned or requiring clarification will be requested. Thank you for your time.

Reproducible Project
N/A
Error Logs/Results
The issues that arise from the lack of clarity in the official docs concerning this issue are numerous and easily located with a simple Google search for problems adding custom Pygments styles to the RTD theme or by performing an Issue search on this repo.
Expected Results
It should be made very clear in the official RTD documentation: a) how to add custom Pygments styles, including inline highlighting options, to the RTD Sphinx theme, and b) whether or not the custom Pygments style is looked at instead of the default RTD Pygments style or as an overlay.
Environment Info

Python Version: v3.9.3
Sphinx Version: v3.5.3
RTD Theme Version: v0.5.2

Again not sure why you are asking here, this is not a theme issue but an issue for Sphinx
The Sphinx documentation reference to a Python class, this is inferred because the sphinx documentation is documenting a python configuration file, and nowhere does it mention 'CSS'.
This should be as simple as the following, however, I think there is a bug where pygments_style thinks it should be a string when it could be a class. You may want to create an issue with sphinx to figure out what the proper way to create a new style. It would be nice if sphinx added Sphinx.add_pygments_style() to its API.
from pygments.style import Style
from pygments.token import Keyword, Name, Comment, String, Error, Number, Operator, Generic

class YourStyle(Style):
    default_style = ""
    styles = {
        Comment:                'italic #888',
        Keyword:                'bold #005',
        Name:                   '#f00',
        Name.Function:          '#0f0',
        Name.Class:             'bold #0f0',
        String:                 'bg:#eee #111'
    }

pygments_style = YourStyle

Aha, thank you for letting me know that this is a Sphinx issue and not a RTD theme issue; I will open a ticket over there. As for inline syntax highlighting, it is currently possible, though there seem to be just as many issues out there about how to get that to work, too. I will bridge that topic with them, as well. I also appreciate your comment about how to get custom pygments styles working. I will give it a shot!

Closing since an issue was opened upstream.


Quests Journal specific requirements in addition to custom descriptions

What does your feature do?
Right now if you add a custom description for a stage or quest it over-writes the criteria-based descriptions, like coordinates to reach or blocks to acquire. It would be great to have BOTH - a page explaining some of the details and story of the quest but then also a page showing how far they've progressed along (talked to 2 of these 3 NPC's etc).
How would people use it?
Probably a config option to allow custom objectives in addition to standard objectives, instead of in place of.

You can already over-write objectives. Inside of Stages use List Override option. And it would appear in Journal and /quest
As what you write in there

As Deadly implied, stage detail overrides intentionally hide the progress. Lore of the quest can be defined separately under start/complete messages. If you're not satisfied with how this applies to the Quests Journal, please consider https://github.com/PikaMug/Quests/wiki/Casual-‐-Modules#want-support-for-other-plugins-or-features


Validation that port used for receiving autentication token from brow…

WHY are these changes introduced?
Fixes #176
Along the authorization process the CLI opens an http server to listen to the browser response with the identity token. If for any reason the fixed port used for the server '3456' is already in use, the CLI gets stuck waiting for the response so the process never finishes.
WHAT is this pull request doing?

Adds a validation before opening the browser to get the identity token. In case the port is in use the CLI displays an error message pointing the user to unblock the port before executing the command again

How to test your changes?

Logout to ensure the needing of executing the authorization process yarn shopify auth logout
Run the dev command yarn shopify app dev
Suspend the process (CTRL + Z) when the CLI is waiting for the browser response
Execute again the dev command yarn shopify app dev

In MacOS to kill the process that listen to a specific port use follow these steps:

lsof -i -P | grep LISTEN | grep 3456
kill -9 <pid_from_lsof>

Since the port 3456 is a port that's hardcoded in our codebase, what if we randomize that port and eliminate this scenario altogether? It's a bit odd that an error tells me to look up the process that's using a port. How do I do that?

There is any kind of validation on the redirect_uri made by the identity project.

Don't you think it is a bit risky to let the user finishing a process (that maybe it's not the CLI) with so little information about it?

I don't think so. We can include additional information in the prompt to help developers make an informed decision. I think that's a better experience than asking users to figure out the process themselves. If I got that question I'd go straight to Google and I'd probably land on a Stackoverflow issue with a Unix command to run on my environment. If that's a yes/no question from the workflow, we save developers a lot of time.

Right now the CLI can only use port 3456 for auth because it is the only one included in the internal configuration, but maybe we could add backup ports there if we want and try with those before failing.

@isaacroldan I guess that this scenario is not very frequent and as we will need a verification on the blocked port/s maybe it does not worth to ask for the rights to add other supported redirection ports
I'm go with @pepicrft suggestion about the prompt. I will use the text proposed for the implementation and then @MeredithCastile could review and change it.

@richard-shopify @isaacroldan @alvaro-shopify Modified the original behaviour that aborts the authorization process if the port is blocked.
Now, in case the port is blocked the user will be prompted to terminate it

Authentication requires a port (23456) that's unavailable because it's running another process (NAME_OF_PROCESS_HERE). Terminate that process?

Yes,  terminate that process in order to log in now
No, cancel <name of CLI command that triggered the auth> and try later

Thank you for your feedback @MeredithCastile . We have two problems about gathering the information to change the message to the one you have proposed:

(NAME_OF_PROCESS_HERE) - It is difficult to obtain the name of the process that is currently listening to the port 3456. It is very dependant on the operating system and until now I haven't found a library cross platform that offers this feature
name of CLI command that triggered the auth - Authentication process is very nested within the executions of the commands. We will have to do a big refactor to carry the command type to the authentication process.
What do you think about leaving the copy with a more generic text right now and create an issue to achieve those two goals?

Based completely on your proposal the temporary message could be:

I'd say it's impossible to know the process or the cli command, we should just make the copy a bit more generic.

I'd say it's impossible to know the process or the cli command, we should just make the copy a bit more generic.

I have finally found a library find-process to inform the user about the process name that is blocking the port. I have updated the PR description with a new prompt screenshot
On the other hand, as @isaacroldan has commented, accessing at this point to the command executed could be out of scope of the issue. I have left that part of the message using a generic command word.
Please @MeredithCastile , could you confirm the message proposed is valid for now?


Sollet web pop-up has been blocked on Safari

This seems like an issue with Safari blocking popups (presumably for Serum, etc). It doesn't seem like an issue with WalletAdapter exactly. But if you still think it is, please open a new issue and fill out the bug report template.

Thinking about this more, the issue is probably this: https://github.com/solana-labs/wallet-adapter/blob/e15287221b90f8d7bee60726d251a5b4c76d243a/packages/wallets/sollet/src/base.ts#L105
The library Sollet uses is loaded dynamically now, which means there is a delay between the click event and the popup opening. Because of this delay, the browser sees the user clicking and the popup opening as unrelated events.
However, I don't think we should fix this. As a rule, we don't load external third party libraries for any of the adapters without the user attempting to connect to them, so there's no easy fix.
The only way around this would be to open a popup window immediately, then have Sollet update the URL for the open window instead of opening a new one.
Given that Sollet is deprecated and not well maintained, I don't see us implementing a workaround. Users can allow popups and it's mostly okay.
The paradigm Sollet uses of opening a browser window is subject to phishing attacks and we shouldn't do anything special to enable it.


get top donated givbacks eligible projects from 2024

From comms issue https://www.notion.so/giveth/EOY-12-Days-of-GIVing-Campaign-14a3ab28d48c8012a989cd964e87c72b?pvs=4
Changed scope again based on results of last queries.
Fetch top 12 most donated projects by USD value in the year 2024, projects must be:

givbacks eligible
NOT the matching pool

SELECT
    p.id AS "project_id",
    p.title AS "projectTitle",
    COALESCE(SUM(d."valueUsd"), 0) AS "total_donations_usd"
FROM
    public.project as p
LEFT JOIN
    donation d ON p.id = d."projectId"
WHERE
    d."createdAt" >= '2024-01-01' AND d."createdAt" <= '2024-12-31' and p.title != 'Giveth Matching Pool' and p.verified = true and p."isGivbackEligible" = true
GROUP BY
    p.id, p.title
ORDER BY
    "total_donations_usd" DESC
LIMIT 20;

Sent results by Dm


refac: Clean up logging packages

Description
See the release notes of the package.

Codecov Report

Merging #239 (d768306) into main (7429e93) will decrease coverage by 0.52%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

:exclamation: Current head d768306 differs from pull request most recent head c890e30. Consider uploading reports for the commit c890e30 to get more accurate results

@@            Coverage Diff             @@
##             main     #239      +/-   ##
==========================================
- Coverage   71.22%   70.71%   -0.52%
==========================================
  Files         199      190       -9
  Lines        3566     3316     -250
  Branches      441      409      -32
==========================================
- Hits         2540     2345     -195
+ Misses        881      835      -46
+ Partials      145      136       -9

Flag
Coverage Δ

logging
61.29% <ø> (-14.87%)
:arrow_down:

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

Files Changed
Coverage Δ

source/Logging/source/Logging/RootLoggingScope.cs
75.00% <ø> (ø)

...dleware/Internal/FunctionLoggingScopeMiddleware.cs
100.00% <ø> (ø)

...gMiddleware/Internal/HttpLoggingScopeMiddleware.cs
100.00% <ø> (ø)

...e/Logging/source/LoggingMiddleware/Registration.cs
0.00% <ø> (ø)

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


Digital twin secret for deploying IRS

Hi everyone!
I would like to receive a digital twin secret as instructed here:
https://eclipse-tractusx.github.io/docs/kits/Data Chain Kit/Operation View/
Thanks!

Thanks for your comment.


Add multi-line flag to semantics

Description
This flag is currently only used in the web engine to decide whether to create an <input> or <textarea> element.
This PR depends on the engine PR: https://github.com/flutter/engine/pull/9850
Related Issues
Fixes https://github.com/flutter/flutter/issues/34265
Tests
I added the following tests:

test/widgets/editable_text_test.dart: "EditableText sets multi-line flag in semantics"

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
[x] No, this is not a breaking change.

You'll also need to update: https://github.com/flutter/flutter/blob/master/packages/flutter_test/lib/src/matchers.dart#L397


DM-313 swu proposal resource

Back-end resource and db routines for SWU proposals

@dhruvio this is ready for another review


Delete the first line problem

There is a problem with swap the first row to delete, which continuously triggers onSwipeValueChange
onSwipeValueChange = (swipeData) => {
    const { key, value, direction } = swipeData;
    console.log(key);
    if (direction === 'right' && value < -width && !this.animationIsRunning) {
        this.animationIsRunning = true;
        Animated.timing(this.rowTranslateAnimatedValues[key], { toValue: 0, duration: 200 }).start(() => {
            const newData = [...this.state.tasks];
            const prevIndex = this.state.tasks.findIndex(item => item.key === key);
            if (prevIndex > -1) {
              newData.splice(prevIndex, 1);
              this.setState({tasks: newData});
            }
            this.animationIsRunning = false;
        });
    }
  }

Hey @stoneWeb I cannot reproduce your issue, I'm really not sure what's going on. But clearly your onSwipeValueChange function is being called with the wrong key. Perhaps you could try keeping track of the key of the row you're deleting and don't let the animation run if it's being called with the wrong key?
I might be able to help more if you post more \code, or better yet a fully reproducible example :)

Closing as inactive


Anyway to get user agent for a request?

I was trying out using the spa template with material-ui. Material-UI server side rendering requires userAgent. However, it doesn't seem the header property is provided through BootFuncParams. Is there anyway to get it from a request?

Answering my own question. Apparently there is a way to pass data from C# to server side javascript code.
So I just added asp-prerender-data attribute in my index.cshtml like this:

Loading...

Then I can simply use params.data.userAgent in my createServerRenderer function.


Add generation for free dynamic emails

Add generation for dynamic and free emails on: maildrop.cc, yopmail.com, mailinator.com.
Will allow emails sent for created inboxes can be read facilitating testing and preventing spam.
Generated emails has no signups, no passwords, is designed for no security, is designed for little to no privacy, to offers the ability to give out a quick e-mail address to your site or app, helps to stop your inbox from getting flooded with tests from that one time you are testing your application.
You can read emails accessing these links:
https://maildrop.cc/inbox/<generated_email>
http://www.yopmail.com/<generated_email>
https://www.mailinator.com/inbox2.jsp?public_to=<generated_email>

PS: generated email without<EMAIL_ADDRESS>
@fzaninotto, what do you think about?

Thanks, but I'll pass on this one. First, the formatter name isn't great ; also, this greatly depends on third party services ; and last, it's so basic that I don't see the point of making a custom formatter for that.

Ok.  I have a fork with this feature and help me more than freeEmail method, because it prevent-me to send spams. And yes, it is basic but helpfully. :smile:

@hedcler you should consider deploying https://github.com/djfarrelly/MailDev in your development environment, that's a more structural solution to this problem.


typo - English for developers - 4th course, Dialog 1, Task2 - work not workin

Describe the Issue
The word 'in' is double
I like to workin in the morning.
Affected Page
https://www.freecodecamp.org/learn/a2-english-for-developers/learn-how-to-talk-about-a-typical-workday-and-tasks/task-2
Steps to Reproduce
-Proceed to the 4th course 'Learn How to Talk About a Typical Workday and Tasks'
-Proceed to Task 2
-Observe the third sentence 'I like to workin in the morning.
Recommended fix or suggestions
I like to work in the morning.
Screenshots

Hi, I will try to fix it if that's ok!


by default select one option in dropdown

Hello , I want to by default select one option in drop-down and when i change the options it should be change is it possible.. ):

http://nya.io/nya-bootstrap-select/#!/examples/basic-usage


Choose between Divgent or Delayed

From Vicky Morley,
The tool includes a divergent net zero and disorderly scenario under the disorderly scenario, which may be too much to include both within the one report

Does not fit with NGFS scenrio structure


161766223 two up display

Add a PopulationSpace component that displays 2 side-by-side panels which will eventually house the population sim on the right and a graph/data view on the left.  Remove the now obsolete InvestigationPanel component.  Display the PopulationSpace component when the user clicks on the population button in the investigation dialog.  Update the UI model to contain population graph state (this will need to be reworked to have greater complexity.  We could use an enum that reflects the current space that is visible.  This is probably best done in its own story and branch).

:+1: Looks good.


How to add attachments CSVs for e.g.

you mean add a file upload button and feed them to messages. you can add a button on the sidebar!
I didn't implement this feature sry!

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
natsume777, @sunnygurm, @lahey08, @faddat, @nicolas-hbt, @yash-srivastava19, @laszlovandenhoek, @timthedevel0per, @devCaiquedePaula, @FelliYang, @EBazarov, @iam-Shashank, @hailieqh, @tonydavis629, @mikepapadim, @mpurnyn, @jerinzech, @Jason-cs18, @benmagnifico, @theonlyladybug

@jw782cn can you delete the phishing attempt above? I got pinged because I got tagged (at random, I guess), but I don't have anything to do with this! I reported it to GitHub too but you can't be too careful.

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
tsukumijima, @jesenzhang, @nauyan, @radu-gheorghiu, @ebeyabraham, @zjcanjux, @zizhanovo, @auniquesun, @VasanthBalakrishnan, @FLOPS-FLOPS, @veb-101, @willhsieh28, @nayutaha, @aravinthragav, @kikirizki, @Alxmrphi, @zhenweiding, @lg-code-repo, @netapy, @lkincel


Update paging.jsp to take a sort to the next page.

If you select a sort in the renderer and klick to the next page, the sort was resettet before. Now the next page is also sorted.

Hi Tobias,
Thanks for this valuable patch ! Merged for v1.5.13.


Use lofty-rs for reading metadata instead of audiotags

Lofty-rs has support for many more filetypes than audiotags. That way metadata-god can also support reading ogg,opus and wav metadata.
https://github.com/Serial-ATA/lofty-rs

That's a great library. Thanks!


[SPARK-15784][ML]:Add Power Iteration Clustering to spark.ml

What changes were proposed in this pull request?
As we discssed in the JIRA, PowerIterationClustering is added as a Transformer. The featureCol is vector type. In the transform method, it calls MLlibPowerIterationClustering().run(rdd) method and transforms the return value assignments (the Kmeans output of the pseudo-eigenvector) as a Dataframe (id: LongType, cluster: IntegerType).
How was this patch tested?
Add new unit tests similar to MLlibPowerIterationClustering.

Test build #68148 has started for PR 15770 at commit 8dd3ca2.

Test build #68150 has started for PR 15770 at commit e29a73e.

Test build #68148 has finished for PR 15770 at commit 8dd3ca2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68148/
Test PASSed.

Test build #68150 has finished for PR 15770 at commit e29a73e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68150/
Test PASSed.

Merged build finished. Test PASSed.

ping @yanboliang

@wangmiao1981 Thanks for working on this. Since we start QA for 2.1, I will review this after the release.

Test build #71020 has started for PR 15770 at commit 754112f.

Test build #72313 has started for PR 15770 at commit d40a6d5.

Test build #72313 has finished for PR 15770 at commit d40a6d5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72313/
Test PASSed.

@thunterdb Thanks for your review! I will address the comments soon.

@wangmiao1981 thanks a lot! I would be very happy to see that PR in Spark 2.2 and I will gladly help you for that.

Test build #73079 has started for PR 15770 at commit cb6c0bd.

Test build #73079 has finished for PR 15770 at commit cb6c0bd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73079/
Test PASSed.

Test build #73238 has started for PR 15770 at commit f53765b.

You are right, I had forgotten that for this algorithm, the input is the edges, and the output is the label for each of the vertices.
This is a tricky algorithm to put as a transformer, since it does not follow the usual convention that data should only be appended to the dataframe. I suggest we follow the same example as ALS the mllib implementation of PIC:

let's make it an estimator that returns a model: the model contains the labels for each of the points in a dataframe (the current output of transform)
the model's transform method now takes points with an id, and joins it with the models to append a column of labels. This is the same as ALS.

If we do not follow this pattern, then the model selection algorithms are not going to work. What do you think?

@thunterdb Thanks for your response. In the original JIRA, we have discussed why we want it to be a transformer. Let me find it and post it here.

Joseph K. Bradley added a comment - 31/Oct/16 18:14
Miao Wang Sorry for the slow response here. I do want us to add PIC to spark.ml, but we should discuss the design before the PR. Could you please close the PR for now but save the branch to re-open after discussion?
Let's have a design discussion first.
I agree that the big issue is that there isn't a clear way to make predictions on new data points. In fact, I've never heard of people trying to do so. Has anyone else?
Assuming that prediction is not meaningful for PIC, then I don't think the algorithm fits within the Pipeline framework, though it's debatable. I see a few options:
Put PIC in Pipelines as a Transformer, not an Estimator. We would just need to document that it is a very expensive Transformer.
Put PIC in spark.ml as a static method. We may have to do this anyways to support all of spark.mllib's Statistics.
Put PIC in GraphFrames (and push harder for GraphFrames to be merged back into Spark, which will include a much longer set of improvements).

My top choice is PIC as a Transformer. What do you think?
CC Yanbo Liang Seth Hendrickson Nick Pentreath opinions?
sethah Seth Hendrickson added a comment - 31/Oct/16 22:40
This seems like it fits the framework of a feature transformer. We could generate a real-valued feature column using PIC algorithm where the values are just the components of the pseudo-eigenvector. Alternatively we could pipeline a KMeans clustering on the end, but I think it makes more sense to let users do that themselves - but that's up for debate.

Yanbo Liang added a comment - 02/Nov/16 09:30 - edited
I'm prefer to #1 and #3, but it looks like we can achieve both goals.
Graph can be represented by GraphX/GraphFrame or DataFrame/RDD. PIC model can be trained on both of them, but we use GraphX operators in the internal implementation which means input data should be converted to GraphX representation if it's RDD of tuples. So it's straight forward to make PIC as one of the algorithms in GraphX(or GraphFrame when it is merged back into Spark). However, users may load their graph as DataFrame/RDD and transform via ML Pipeline which should also be supported, so it's better we can wrap PIC of GraphX/GraphFrame as an Pipeline stage and then ML users can use it as well.
For some historical reasons(we don't want to add new features to GraphX), I propose to split this task into the following step:
Put PIC in Pipeline as a Transformer, use the GraphX operators in the implementation (This is consistent with Joseph K. Bradley's proposal).
Add PIC algorithms to GraphFrames when it is merged into Spark.
Make the ML PIC as a wrapper to call the GraphFrames PIC implementation.

I think this scenario should be better for different users(ML users and GraphFrames users), but still open to hear your thoughts. Thanks.

I am checking ALS out to understand your suggestions. Thanks!

Test build #73238 has finished for PR 15770 at commit f53765b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73238/
Test PASSed.

@wangmiao1981 yes I had seen the discussions there. I believe that eventually PIC should be moved into graphframes, but we can have a simple API in spark.ml for the time being.

@thunterdb Per discussion with Yanbo, there is one concern of making it an Estimator. For every transform, there is an additional data shuffle. cc @yanboliang @jkbradley Thanks!

@thunterdb I have two concerns about making PIC as an Estimator:

As your suggestion, model's transform will introduce extra data shuffle compared with the original implementation.
It can not fit into the pipeline well, since both the estimator fit and model transform method should work on the dataset with same schema, like:

val model = new ALS().fit(dataset)
model.transform(dataset)

But in your suggestion, the input of PIC estimator is the edges or RDD[(Long, Long, Double)], the input of PIC model is the vertices ID.
I totally agree making PIC as a Transformer is tricky, but I did not find a better way. Thanks.

@thunterdb @yanboliang Do we reach an agreement on whether to make it a transformer or an estimator now?

Sorry for my absence from recent conversation!
I agree there is no clear answer for handling input and output schema.  Some options:

Option 1: same as RDD/GraphX-based API

Input: Each Row is 1 edge.  DataFrame represents a graph.
Output: [node ID, cluster ID]

Option 2: take adjacency list

Input: Each Row has [node ID, list of neighbor IDs, list of neighbor edge weights].  DataFrame represents a graph.
Output: Append column for "cluster ID"

Option 3: take feature vectors --> This is another use of PIC from the original paper.  Given feature vectors, compute a similarity matrix using some distance, and then run PIC as we do now.  We won't do this now, but we could add it as an option in the future.

Input: Each Row has [node ID, feature vector].  DataFrame does not represent a graph.
Output: Append columns for "cluster ID" and "adjacency list"

It sounds like Option 1 is the main one being considered right now.  What do you think of Option 2 though to keep it as a row->row Transformer?

@jkbradley Thanks for your reply! I quickly go through your suggestions. If I understand correctly, you prefer making it a Transformer, as we previously discussed, but changing the input data to fit into to the pipeline. Right? Let me think about details and evaluate each options before making the changes. Thanks!

Yep, that's correct.  Everyone, please let me know if you disagree.
Also, if we do go with Option 2 above, then the input schema could be a few possible things:

list of (neighbor ID, weight)

pro: ensures # neighbors = # weights
con: nesting can be annoying to work with

(list of neighbor IDs, list of weights)

pro: Easier to support unweighted graphs by omitting list of weights.  Avoid nesting.
con: could have invalid input with mismatched numbers of neighbors, weights

Vector (of length # neighbors)

pro: Familiar data type.  Ensures # neighbors = # weights.
con: If we later support passing raw features which are used to compute a similarity matrix within PIC, then this could confuse users.

I'm ambivalent about these schema options.  What do you think?

Note that any of these formats would cause trouble for a graph with high centrality (lady gaga in the twitter graph). That being said, I do not have a strong opinion as to which option we pick, in order to move things along.

Option 2 doesn't break our pipeline scheme which only appends the result column to the input dataframe. Besides the discussions above, the graph is undirected and the weight list will appear twice for two nodes, while Option 1 only stores the weights once.
Let me try Option 2 today.
Thanks!

Update:
Sorry for the delay. I am working on some other items.
Now, I am using  (node-id, list of neighbor IDs, list of weights) by adding two additional columns.
I will push soon.

Test build #74567 has started for PR 15770 at commit 675763e.

@jkbradley I changed the input data format by using (list of neighbor IDs, list of weights), which are two columns of the input dataset. For the result, I appended the predicted cluster ids to the input dataset by joining the assignment dataset by col id.
This is an initial implementation. Please take a look. Thanks!

Test build #74567 has finished for PR 15770 at commit 675763e.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74567/
Test FAILed.

Merged build finished. Test FAILed.

It passed at local. I will fix the issue.

Test build #74622 has started for PR 15770 at commit c938758.

Test build #74622 has finished for PR 15770 at commit c938758.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74622/
Test PASSed.

Test build #74681 has started for PR 15770 at commit 28ec1af.

Test build #74681 has finished for PR 15770 at commit 28ec1af.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74681/
Test PASSed.

@jkbradley can you take a look? Thanks!

gentle ping @jkbradley @yanboliang @thunterdb

Any update on this?

@yanboliang Can you take a look first? Thanks!

@WeichenXu123 Thanks for reviewing! I will address the comments soon.

Test build #80698 has started for PR 15770 at commit 6f22e97.

Test build #80698 has finished for PR 15770 at commit 6f22e97.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80698/
Test FAILed.

weird. Local style test passed. Anyway, I changed the order as required by Jenkins.

Test build #80699 has started for PR 15770 at commit 4c5fe72.

Test build #80699 has finished for PR 15770 at commit 4c5fe72.

This patch fails to generate documentation.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80699/
Test FAILed.

Jenkins, retest please

retest please

Test build #80702 has started for PR 15770 at commit de7c878.

Test build #80702 has finished for PR 15770 at commit de7c878.

This patch fails to generate documentation.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80702/
Test FAILed.

Test build #80704 has started for PR 15770 at commit 006dad3.

Test build #80704 has finished for PR 15770 at commit 006dad3.

This patch fails to generate documentation.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80704/
Test FAILed.

info] Main Scala API documentation successful.
[error] (spark/javaunidoc:doc) javadoc returned nonzero exit code
[error] Total time: 95 s, completed Aug 15, 2017 4:59:59 PM
[error] running /home/jenkins/workspace/SparkPullRequestBuilder/build/sbt -Phadoop-2.6 -Pmesos -Pkinesis-asl -Pyarn -Phive-thriftserver -Phive unidoc ; received return code 1
It seems irrelevant.

Jenkins, retest this please.

Test build #80745 has started for PR 15770 at commit 006dad3.

Test build #80745 has finished for PR 15770 at commit 006dad3.

This patch fails to generate documentation.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80745/
Test FAILed.

Test build #80755 has started for PR 15770 at commit e35fe54.

Test build #80755 has finished for PR 15770 at commit e35fe54.

This patch fails to generate documentation.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80755/
Test FAILed.

Test build #80792 has started for PR 15770 at commit 73485d8.

Test build #80792 has finished for PR 15770 at commit 73485d8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80792/
Test PASSed.

Merged build finished. Test PASSed.

@WeichenXu123 I have made changes based on your comments. Thanks!

ping @WeichenXu123

ping @WeichenXu123

@wangmiao1981 Sorry for delay, I will take a look later, thanks!

I will address the review comments soon. Thanks! @WeichenXu123

Test build #83064 has started for PR 15770 at commit 752b685.

Test build #83064 has finished for PR 15770 at commit 752b685.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83064/
Test FAILed.

Test build #83230 has started for PR 15770 at commit cfa18af.

Test build #83230 has finished for PR 15770 at commit cfa18af.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/83230/
Test PASSed.

@WeichenXu123 , for the graph helper, the Mllib has a version takes Graph[Double, Double] as a parameter for training. In ML, do we have to provide DataSet of Graph? Can you specify the requirement? I have addressed your other comments. Thanks!

@wangmiao1981 oh, not a big deal, what I thought is that, user is possible to use graphx package to get the Graph[Double, Double], and in ml package it cannot accept this format, require user to convert it into dataframe (because mllib API is deprecate maybe user do not want to use it).
But, this can be discussed in separated JIRA.

@WeichenXu123 Thanks for your review and reply! I agree with you that the helper can be discussed later for potential enhancement.

@weichenXu123 Any other comments? Thanks!

LGTM. ping @yanboliang

ping @yanboliang

@wangmiao1981  Do let me know if you're too busy now to resume this; I know it's been a long time.  Thanks!

I don't mind; I'll take it.  But I'll mark @wangmiao1981 as the main contributor for the PR.  Would you mind closing this issue @wangmiao1981  and I'll reopen a new PR under the same JIRA?

@jkbradley Sorry for missing your comments. Anyway, I will close it now. I will choose another one to work on. Thanks!

@jkbradley I close this one now. Thanks!

OK sorry to push @wangmiao1981 !  I just want to make sure this gets in before I no longer have bandwidth for it.  If you have the time, would you mind checking the updates I made in the new PR?   Thanks!


rviz can't visualize /voxblox_node/mesh

I found this unanswered question online.  I'm in the same exact situation.
I can run roslaunch voxblox_ros cow_and_lady_dataset.launch
but then I can't visualize the mesh published by voxblox with rviz.

I wasn't sourcing the corresponding setup.bash in the terminal that runs rviz

A little info as to why you need to source your workspace as the doc's were out of date.
The mesh is actually of type voxblox_msgs::MeshBlock. Voxblox use to output a standard MarkerArray, however this form of visualization means you have to output the entire mesh every time any part of it changes. This meant when running on a robot and visualizing on a laptop we were saturating our wifi connection to display even relatively small areas.
To correct this issue we made a custom message that supports partial updates only changing the mesh where needed. To use this custom message you have to build voxblox_rviz_plugin and have it in your workspace when running rviz.


fix: cleanup removes jobs only directly related to scheduler

Codecov Report

Merging #55 (f23f49f) into master (7b8ebd2) will decrease coverage by 0.47%.
The diff coverage is 50.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master      #55      +/-   ##
==========================================
- Coverage   56.09%   55.63%   -0.47%
==========================================
  Files           2        2
  Lines         287      293       +6
==========================================
+ Hits          161      163       +2
- Misses        103      106       +3
- Partials       23       24       +1

Impacted Files
Coverage Δ

scheduler.go
52.38% <50.00%> (-0.43%)
:arrow_down:


Crashes on hypixel server

Opening up the inventory when on the Hypixel server causes the entire game to crash
crash-2024-02-07_21.01.45-client.txt

Caused by an incompatibility between polymer and legacy4j

Caused by an incompatibility between polymer and legacy4j

Exactly, not related to Hypixel server

Fixed in version 1.6.4, thanks for reporting.


HUB75HAT v1.0: Space around I/O headers insufficient for shrouded header

The space around the I/O headers is insufficient for shrouded header. In v1.0 this has been solved by applying colored headers showing the functions of the pins. In V1.1 the space should allow for shrouded header:

The jumpers for power to the connector should be moved down;
The decoupling capacitors on the power rail should be either moved or flipped to the underside of the board;
The resistor networks should be either moved or replaced with their through-hole counterpart.

Going to be solved in #3

Solved in release [v1. 1] (src/HUB75HAT/releases/v1.1).


set_keepalive reports EINVAL on Linux

Linux is Ubuntu with 3.13.0-63-generic kernel, x86_64.
Error debug message is
Error { repr: Os { code: 22, message: "Invalid argument" } }

The same code runs well on OS X 10.10.5.
Call from the mio::tcp::TcpStream::set_keepalive with argument Some(300).

Perhaps the socket is in an invalid state? Do you have some example code I could try out?

Just call set_keep_live immediately after you accepted a new TcpStream.

Modified from MIO's official example
extern crate mio;

use mio::*;
use mio::tcp::{TcpListener, TcpStream};

// Setup some tokens to allow us to identify which event is
// for which socket.
const SERVER: Token = Token(0);
const CLIENT: Token = Token(1);

fn main() {
    let addr = "<IP_ADDRESS>:3000".parse().unwrap();

    // Setup the server socket
    let server = TcpListener::bind(&addr).unwrap();

    // Create an event loop
    let mut event_loop = EventLoop::new().unwrap();

    // Start listening for incoming connections
    event_loop.register(&server, SERVER, EventSet::readable(),
                        PollOpt::edge()).unwrap();

    // Setup the client socket
    let sock = TcpStream::connect(&addr).unwrap();

    // Register the socket
    event_loop.register(&sock, CLIENT, EventSet::readable(),
                        PollOpt::edge()).unwrap();

    // Define a handler to process the events
    struct MyHandler(TcpListener);

    impl Handler for MyHandler {
        type Timeout = ();
        type Message = ();

        fn ready(&mut self, event_loop: &mut EventLoop<MyHandler>, token: Token, _: EventSet) {
            match token {
                SERVER => {
                    let MyHandler(ref mut server) = *self;

                    println!("Got Readable event from TcpListener");
                    match server.accept() {
                        Ok(None) => {},
                        Ok(Some((stream, addr))) => {
                            println!("Got connection from {:?}", addr);
                            // Call set_keepalive
                            if let Err(err) = stream.set_keepalive(Some(300)) {
                                println!("Error occurs while set_keepalive: {:?}", err);
                            }
                        },
                        Err(err) => {
                            panic!("Error occurs while accepting: {:?}", err);
                        }
                    }
                }
                CLIENT => {
                    // The server just shuts down the socket, let's just
                    // shutdown the event loop
                    event_loop.shutdown();
                }
                _ => panic!("unexpected token"),
            }
        }
    }

    // Start handling events
    event_loop.run(&mut MyHandler(server)).unwrap();
}

The output is
Got Readable event from TcpListener
Got connection from V4(<IP_ADDRESS>:39653)
Error occurs while set_keepalive: Error { repr: Os { code: 22, message: "Invalid argument" } }

I believe the error here is that set_keepalive in mio is specified in seconds, where set_keepalive_ms on net2 is specified in milliseconds. Using a keepalive timeout of 300s apparently is too high for linux? Going down to 300ms (is that the intention?) works, however.
Regardless though I think everything's working as expected in this library, it may just be surprising behavior coming out of linux!

The man page describes EINVAL as

optlen invalid in setsockopt(). In some cases this error can also occur for an invalid value in optval (e.g., for the IP_ADD_MEMBERSHIP option described in ip(7)).

Yes, I want to set 300ms!

Actually, even I modified Some(300) to Some(60), it still gets EINVAL.

According to this line of code: https://github.com/alexcrichton/net2-rs/blob/master/src/ext.rs#L590
In man tcp, it saids
       TCP_KEEPIDLE (since Linux 2.4)
              The time (in seconds) the connection needs to remain idle before TCP starts sending  keepalive  probes,
              if  the socket option SO_KEEPALIVE has been set on this socket.  This option should not be used in code
              intended to be portable.

So the argument of TCP_KEEPIDLE should be seconds instead of milliseconds.
I think this issue should be re-open.

Aha, thanks for the sharp eye! I updated the impl here and published 0.2.15, could you try that out?

@alexcrichton Well, it works fine. Thanks.


Any plans on support for react native?

Hello, I just spent the last couple of days hacking my way around using the hosted ui with React Native in an Expo app. Recently, the Amplify library has added support for web react to use the Hosted UI using this library (amazon-cognito-auth-js). Since it appears that Amplify is relying on this package for usage of the Hosted UI, in order to add support for using the Hosted UI with React Native in Amplify, it seems that it would be necessary that some support is added for React Native in this library. Namely:

Storage options. Right now this library will use localStorage if available, otherwise it will default to memory storage. In order for full functionality with React Native, we would need to add an option for AsyncStorage much like the amazon-cognito-identity-js library does (see StorageHelper-rn.js and package.json for that project to see how they implement it). At the very least, we should allow Amplify to provide a storage class on the constructor in order to override the storage helper that comes with this package, much like amazon-cognito-identity-js does.
Using window.location. This one is a bit trickier because your implementation of how to do this depends on your app. For me, I use the AuthSession class provided by Expo in order to call out to the oauth endpoints and it handles the callbacks. This has turned out to be a great solution for me, but obviously will only work for those using Expo (and many people do). It would be nice if we could allow the developer to overwrite the implementation for making the oauth calls. This topic definitely needs discussion.

I would love to offer my assistance if this is something that can be supported here. If this library does not plan on supporting React Native, then it is necessary that all of this logic be duplicated and redone for Amplify, which to me seems like a waste of everyone's time since it is all done here.

Help me please:
https://github.com/aws/amazon-cognito-auth-js/issues/134


Cannot get external processing to work?

I want to use a separate processing server for refile.
So I created a small experimental processing server and started it with puma:
puma -t 1:4 -p 9292
But I only get this error when I try to get images from the server.
<IP_ADDRESS> - - [06/Jul/2015:09:39:40 +0200] "GET /attachments/store/joken/320/320/true/true/e56f00d76ba50d6bf7cf0e95a8d209e867f34fea9a887f94fd5cfda4614c/cover_image.png HTTP/1.1" 404 9 0.0048
E, [2015-07-06T09:39:42.993539 #1837] ERROR -- : Refile::App: Could not find processor: store
I, [2015-07-06T09:39:42.993823 #1837]  INFO -- : Refile::App: [2015-07-06 09:39:42 +0200] GET "/attachments/store/joken/320/320/true/true/e56f00d76ba50d6bf7cf0e95a8d209e867f34fea9a887f94fd5cfda4614c/cover_image.png" 404 0.5ms

I have added:
Refile.cache = Refile::S3.new(prefix: 'cache', **aws)
Refile.store = Refile::S3.new(prefix: 'store', **aws)

It really doesn't work!
So after that I tried with adding a secret key(it might be that's what is missing) but only get this error:
Refile.secret_key = '123'

error:
! Unable to load application: NoMethodError: undefined method `secret_key=' for Refile:Module

Need some help! I dunno where to go from here? :watermelon:

It it possible you have a version mismatch between the server (running master probably) and the client (running 0.5.5)? That's what the error sounds like to me.

I made sure both was on 0.5.5 but it resulted in the same error.
So I switched to master and it resulted in almost identical error:
E, [2015-07-06T14:57:18.383036 #1486] ERROR -- : Refile::App: Could not find processor: store
I, [2015-07-06T14:57:18.383262 #1486]  INFO -- : Refile::App: [2015-07-06 14:57:18 +0200] GET "/attachments/b40590d7161ced19aa5521f983115b5a23784b22/store/joken/320/320/true/true/e56f00d76ba50d6bf7cf0e95a8d209e867f34fea9a887f94fd5cfda4614c/cover_image.png" 404 0.5ms

<IP_ADDRESS> - - [06/Jul/2015:14:57:18 +0200] "GET /attachments/b40590d7161ced19aa5521f983115b5a23784b22/store/joken/320/320/true/true/e56f00d76ba50d6bf7cf0e95a8d209e867f34fea9a887f94fd5cfda4614c/cover_image.png HTTP/1.1" 404 9 0.0006

The only way I think this could happen is if you mounted the Refile app on / instead of on /attachments. Then Refile thinks that attachments is the name of the backend, and proceeds further with the notion that store is a processor.
In Rails it could have happened if you set Refile.automount = false, and then accidentally mounted it on "/" instead of Refile.mount_point.

Closing this issue. @janko-m's explanation sounds plausible, and we haven't heard back from OP.


metadata: Support gzip cloud-config from metadata

Implements https://cloudinit.readthedocs.org/en/latest/topics/format.html#gzip-compressed-content for metadata services.

I'm running into the problem on AWS EC2 that my cloud-config files to bootstrap my coreos instances become larger then 16kb: a hard limit on AWS for uploading userdata files. gzipping the userdata would solve that problem, so incorporating this fix would be much appreciated.

Superceded by https://github.com/coreos/coreos-cloudinit/pull/400.


Reverting Sonar plugin config changes

Reverting changes to the Sonar plugin as this breaks the existing Java cartridge

Closing this PR as the revert is no longer required as this has been fixed in this release: https://github.com/Accenture/adop-jenkins/releases/tag/0.3.3


Getting started with Blog Posts into Facebook Instant Articles

Topic Suggestion
Getting started with Blog Posts for Facebook Instant Articles
Proposed article introduction
Facebook Instant Articles is a mobile-optimized format that allows publishers to distribute interactive articles that load quickly on the Facebook app and share their information.
The good thing about these articles is that no plug-ins are required or integrations. All we require is a text editor to work with our HTML source code.
It will be a lively event because we will be interacting with instant generated output from our session and because the majority of people are social media netheads.
Key takeaways

Obtaining a Banner Image
Obtaining Metadata.
Taking a Copy of the Body Content.
Adding the tracker ChartBeat
Launch your Instant Article

Article quality
This article will help novices who have always wanted to share their business model or any other news with their peers, and in return, generate revenue. I will be pleased to explain the ins and outs of Facebook Instant Articles and how to put them up on your site in this article.
References
N/A
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

This is out of our preferred topics-area. Feel free to suggest another topic to explore. @cathy-254


[SPARK-47422][SQL] Support collated strings in array operations

What changes were proposed in this pull request?

Collations need to be properly supported in following array operations but currently yield unexpected results: ArraysOverlap, ArrayDistinct, ArrayUnion, ArrayIntersect, ArrayExcept. Example query:
select array_contains(array('aaa' collate utf8_binary_lcase), 'AAA' collate utf8_binary_lcase)

We would expect the result of query to be true.
Why are the changes needed?

To enable collation support in array operations.
Does this PR introduce any user-facing change?

Yes, with these changes listed array operations return expected results when applied on arrays of collated strings.
How was this patch tested?

Added test to collections suite and updated collations golden files.
Was this patch authored or co-authored using generative AI tooling?

No.

thanks, merging to master!