A pressed effect with inverted-inner shadows

Nice to have neumorphism on Xamarin. Have you considered to add a pressed effect with inverted-inner shadows? (Probably need SkiaSHarp then)
A Simple sample goes like this dribbble pic.
Btw, more "want" than "need" for this request. Thanks.

I have the same need... +1
A full neuromorphism UI implementation will need to manage shadows "outside" (elevation positive) and shadows "inside" (elevation negative) for different views (Entry/Button/Frame...) exactly like @shawyunz shared image.
Is there a possibility/ehancement for doing it with your "Sharpnado.Shadow" implementation ?
I tried to put negative offset, but it doesn't work (I didn't expected this to work, but I tried ;-) )
I'm dreaming of a single boolean bindable property (IsShadowInside) to achieve that like this :
<sh:Shadows Shades="{sh:NeumorphismShades}"
            IsShadowInside="{Binding IsPushed}">
        <Frame BackgroundColor="#F1F1F1"
               CornerRadius="10"
               HasShadow="False">
            <StackLayout>
                <Label Style="{StaticResource TextHeadline}" Text="Welcome to the HomePage !" />
                <Label Style="{StaticResource TextBody}" Text="Need to customize your page here..." />
            </StackLayout>
        </Frame>
</sh:Shadows>

I'll check your sourcecode to find if it can be a possible ehancement... feel free to guide me (or suggest me other ways)

Inner Shadows are definitely something interesting and I will probably give it a try :)
Right now your can already have some kind of push effect by swapping the colors of the 2 neumorphic shades on the Tap or click event in the code-behind.
Or I think you can even achieve it with VisualState by setting shades with NeumorphismShades in a "Idle" state and create a pair of shades like this (offset between white and dark shades are inverted) on a "Pressed" state:
        <sh:ImmutableShades>
                <sh:Shade BlurRadius="10"
                          Opacity="1"
                          Offset="10,10"
                          Color="White" />
                <sh:Shade BlurRadius="10"
                          Opacity="1"
                          Offset="-6, -6"
                          Color="#19000000" />

On the topic of inverted shadows with neumorphism, I feel like they are sort of a bad idea in certain scenarios as they dont have much of a contrast to the unpressed look (also it just looks sort of weird to me). I was reading an article providing a better alternative which was using an outline that you could just darken and lighten to indicate pressed and unpressed states.

i really need it too.


Incremental build for Swift

As part of the work avoidance scenario, incremental compilation, and linking are a good candidate by only recompiling the changed files and relinking only when dependencies change.
Expected Behaviors
Gradle is expected to take advantage as much as possible the incremental feature of swiftc.
Test Cases
The test cases to support should be close to the one found in AbstractNativeLanguageIncrementalBuildIntegrationTest, and AbstractNativeLanguageIncrementalCompileIntegrationTest.

Initial PR: https://github.com/gradle/gradle/pull/4226
Follow-up to fix flaky tests: https://github.com/gradle/gradle/pull/4228
Additional test coverage: https://github.com/gradle/gradle/pull/4239
@ghale would you like to take a look?

@ghale I've addressed your comments here: https://github.com/gradle/gradle/pull/4256


[Tracking] Angular 6

Branch https://github.com/NativeScript/nativescript-angular/tree/sis0k0/ng6.

[x] Update nativescript-dev-webpack to support webpack 4. https://github.com/NativeScript/nativescript-dev-webpack/issues/492
[x] Enable strictMetadataEmit when https://github.com/angular/angular/issues/23310 is resolved. There's a PR in the Angular repo with the fix: https://github.com/angular/angular/pull/23349.
[x]  ActionBarExtension e2e test is failing when running webpack with uglify without specifying mangleExcludes. More specifically, the issue is reproducible when the name ActionBar is changed.
[x] Verify that the unit tests are passing.
[x] Update apps to Angular 6, rxjs 6 and TypeScript 2.7.2.

[x] sampleGroceries
[x] nativescript-sdk-examples
[x] tests-app-ng
[x] renderer/router e2e apps
[x] templates

Watching this eagerly :)

Any news?

@RainingNight @mr-White the support for Angular 6 is in development but you can already test it with nativescript-angular@next (and then updating all Angular related dependencies with executing ./node_modules/.bin/update-app-ng-deps) - test with POC app here

Thank you @NickIliev ! I'll have to check it out this weekend :)
To anyone who hasn't subscribed to the NativeScript newsletter, in a recent issue, it mentioned that Angular 6 support is included in upcoming NativeScript 4.1 release.
Here is an except from that Newsletter:

What’s Coming in NativeScript 4.1? 🧐
Even though NativeScript 4.0 was just released, our engineering team has taken off the brakes and is steadily working towards the 4.1 release! Truth be told, this is bigger than most point releases, as we will be providing features and improvements such as:
V8 upgrade to latest version and performance improvements on Android;
Documentation revamp - complete revamp of the current cookbook, new table of contents, new user experience;
Various fixes on debugging and LiveSync; LiveSync will work with sockets to provide more stable and predictable experience;
Support for Angular 6 and Webpack 4;
Extended CSS support with gradients, box shadow, and media queries;
Debugging on multiple iOS simulators;
New setup scripts for Windows and Mac.
Stay tuned as 4.1 is due to drop at the very end of May or early June! As a reminder, register for our upcoming webinar to learn all about the 4.1 release.

/cc @RainingNight

@sis0k0  Regarding RXJS6 &  Angular 6 , there is a problem when importing operators in the new way.
aka
`import {map} from 'rxjs/operators'
(from 'rxjs/operators/map' is not supported anymore)
In short : all operators (from rxjs/opearators) will be imported regardless their usage. ( for dev and for bundling with webpack also)
Webpack configuration must be added with code in order not to import all unused operators
Already reported this here :  https://github.com/NativeScript/nativescript-dev-webpack/issues/550


Enable CI builds

Also the PR fixes build failures reported in #552
closes #552

@rojkov do you know when the yottaci will finish and light up green? It does not seem like it would do so within the last 24 hours. Is this expected?

@bulwahn yes, it's expected for the first build (when SSTATE is empty). Hopefully it finishes in several hours.
Though there are issues with outdated hash-sums for some tarballs generated on fly by github. I can't find all of them at once locally due to my corporate proxy perhaps, so I fix them as they break builds in CI.

Finally CI is green again

Okay, this looks good to me. I am still unsure about tying the recipes to the git revisions instead of the archive files, as this potentially has a few drawbacks as well, but we can discuss this more in #552. For now, I will just merge it for a working CI.


fixing deprecting time unit warning

warning: deprecated time unit: :microseconds. A time unit should be :second, :millisecond, :microsecond, :nanosecond, or a positive integer

Thanks for the pull request @mkaravaev.


runtime: add jumpoptions to vim syntax file

Hi there 👋🏼
While testing the new jumpoptions option, I noticed it's not highlighted correctly in my .vimrc file. Looks like the option was missing in runtime/syntax/vim.vim.

thanks!


Added common-utils to 1.2.3 manifest.

Signed-off-by: dblock<EMAIL_ADDRESS>Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

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

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

Codecov Report

Merging #1374 (f4f7c1c) into main (0faa8b0) will decrease coverage by 19.94%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main    #1374       +/-   ##
=============================================
- Coverage     94.02%   74.07%   -19.95%
  Complexity       11       11
=============================================
  Files           136       17      -119
  Lines          3011       81     -2930
  Branches          8        8
=============================================
- Hits           2831       60     -2771
+ Misses          172       13      -159
  Partials          8        8

Impacted Files
Coverage Δ

src/test_workflow/test_args_path_validator.py

src/build_workflow/build_args.py

src/system/working_directory.py

...workflow/integ_test/integ_test_start_properties.py

src/paths/script_finder.py

src/manifests_workflow/component_opensearch_min.py

src/run_perf_test.py

src/ci_workflow/ci_check_gradle_properties.py

src/build_workflow/build_artifact_checks.py

src/ci_workflow/ci_check_lists.py

... and 109 more

Continue to review full report at Codecov.

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

Add Job Scheduler too?
It's now merged.

@saratvemulapalli Job Scheduler doesn't have a dependency on common utils?  - we will get it in the next pass

Apparently job-scheduler doesn't depend on common-utils!


Hotfix updatedforcleanliness

Needed to update the Contributing.md to include some reminders for Tree hygiene as this is something that we've been having issues with lately.

There should be a dedicated PR for merging stable into dev. This update should only be for updating contributing.md.


Add a configuration option to change ConsoleLogger output

Similar to #483, but we would like to change the way console output is logged. For example, we would like console lines to be one line only. Currently a line break is hard codec into the console logger, which produces a lot of noise and complexity at the log ingestion stage.
Consider adding a "log line format" property, that allows for easy configuration of the log lines, without having to rewrite the console logger of pull in custom logging frameworks.

We are not planning on adding more console logger formatting options.

Could you please elaborate? I have a hard time understanding the current best practice from the aspnet team's POV.
I am using .NET core on Docker/k8s, and I want to log to stdout to handle log management externally, same as for any other container. This means I won't log to a file, and definitely not the event source logger, which seems to be a Windows-only thing.
The only built-in solution I found is the ConsoleLogger, which looks a bit difficult to subclass (the virtual methods access private fields, etc). For me, it looks like the ConsoleLogger is not really meant for production use (and especially after reading #483). For example, if I use the Azure log ingestion service, I would basically add one redundant garbage entry for each row logged.
Which means I am left with two options: write my own logger from scratch, or evaluate third parties to find which ones are any good and does what I want. Both tasks are time consuming and must be repeated for every team that wants to take a ASP.Net core into production. Unless I am mistaken..?

We decided to go with serilog for now. It is vastly overkill for our needs, but we did not have time to evaluate all options, and it gets the job done.


UI/UX - Scroll Bar glitch

I want to

[x] I have read the Contributing Guidelines 🔐
[x] I have read the Setup guidelines 🔐

Notice ⛔
Please make sure in the .css files you are making/using , the styles should be applied to Unique Class names
If there are images in your .jsx files  give them unique classnames like **registrationImage , clubLoginImage, **, then in the Css files make sure the styles are applied to those unique classnames. Else things are getting messed up.

FIXES 🛠
Take a look at this video
https://user-images.githubusercontent.com/72851613/152650591-1ad01039-d93f-4a91-aea8-5d147ba3e62f.mp4
Notice how the scrollbar is getting stuck a bit while scrolling ?
I would want that to  be fixed

Completed / Doubts ?  🚀
Once you are done with the code , please tag me in #milan with the Screenshot of whatever you made so that we can review that there together.
If you have any doubts, queries please tag me in #milan so that we can discuss the issue there together.
Cheers 🥂

I would like to work on this issue.please assign it to me.

I would like to work on this issue.please assign it to me.

These are only for JWOC participants.

Go ahead.
Make sure you've read the guidelines.
Incase of doubts do contact me.
Assigning to @Devmindtk

Do I have to join any discord channel to give the updates?

Do I have to join any discord channel to give the updates?

Umm yes , JWOC has a discord channel. Sent via email now.

scrollbar is getting stuck only on clubs page right?

https://user-images.githubusercontent.com/91221069/152932346-4a00b201-18be-456f-8521-120174fd4b94.mp4
@IAmTamal  check this one is it fine now?

@Devmindtk looks good to me. Is it totally smooth now in the club page ? (on scrolling thorugh the mouse wheel)

yeah its working smooth.

@Devmindtk looks good to me. Is it totally smooth now in the club page ? (on scrolling thorugh the mouse wheel)

yeah its working smooth.

Great !!
You can make a PR

@Devmindtk looks good to me. Is it totally smooth now in the club page ? (on scrolling thorugh the mouse wheel)

yeah its working smooth.

Great !! You can make a PR

error: src refspec main does not match any
facing this error in pushing my code to origin branch

@IAmTamal after writing command git pull it shows this how to fixed it?
fatal: not a git repository (or any of the parent directories): .git

Make sure you are in the right directory.

NOT HERE ⚠


Request: Include operation arguments in callbacks

It would be super helpful in Resource callbacks to get some context on what the operation is...
For example, I am adding a before_remove_to_many_link callback that will halt the removal if the parent object is in a specific state (e.g., you cannot remove OrderLineItems from an Order that is "finalized").
The remove_to_many_link method receives the relationship_type and key arguments from the corresponding *Operation, but those arguments are not passed through to callbacks.
Since my Order resource has more than one relationship, it would be useful to know, in my callback, which relationship we are trying to remove a link from.
Same goes for other callbacks, especially replace_fields.

I agree that would be useful. We're using ActiveSupport callbacks. I know I looked when we initially set this up, but I don't know if it's possible to pass parameters to the callback methods.
This isn't exactly what you are asking for, but if you create your own OperationsProcessor you will have access to the full stack of operations (@operations) and the current operation (@operation). Using those I added a method on all my resources called approve_operation` and in the custom operations processor I call this before every operation:
class MyOperationsProcessor < ActiveRecordOperationsProcessor
  before_operation :approve_operation

  def approve_operation
<EMAIL_ADDRESS>@operation)
  end
end

class MyResource < JSONAPI::Resource
...
def approve_operation(context, operation)
  case operation
    when JSONAPI::RemoveToManyRelationshipOperation
      #your code here
    else
  end

It's not the most elegant solution, but it works fairly well.


Let Kotlin DSL source path calculation properly look into platforms/

Fixes https://github.com/gradle/gradle/issues/27405

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle merge

Your PR is queued. See the queue page for details.

I've triggered a build for you. Click here to see all failures if there's any.


bug with painting mentions over PDF?

I was playing around and noticed this issue, seems like the mentions are being painted on the wrong page, perhaps?  or maybe the underlying PDF changed?
https://cloud.science-miner.com/software_kb/frontend/document.html?id=611a11d8e8d4855847028e44
Hopefully this screenshot shows the issue (This is with Firefox 109.0.1 on Mac OS btw)

Indeed, in the current version it can happen when the underlying paper has changed. The GUI fetches the Open Access PDF at URL given by a call to the Unpaywall API, but if the version of the PDF has changed between the time the annotations was produced and stored in the KB and now, we have position "mismatch" for the annotations.
For example, the annotations produced in 2021 was made with the preprint, and now Unpaywall points to the Gold version of the article.
I store a hash for the PDF originally annotated, so I could detect normally if the fetched PDF is still the same of not - but this is not yet implemented. Then if the PDF mismatch is detected we could imagine a fallback solution, like trying alternative URL to get other version of the PDF or re-annotating on the fly.

Makes sense.  Maybe we should store something that enables us to hit the same version (if at all possible)? That's hard without storing the actual article (which we can't do), but perhaps re-trying the URL (or following the metadata to the same version).
But also highlight that unpaywall has a new version?


Project page

[ ] Metadata , and stats (numbers of datasets, models, etc
[ ] Recent activities
[ ] Overview context ~ Description/abstract
[ ] Contributions/Sharing settings
[ ] #44
[ ] #45
[ ] #46

@dgauldie should we close this issue, as most of those features are not needed as of now.


uv-resolver: make source=git more structured

This splits out the git fields into an inline table instead of smushing
them into a single URL.
I'm unsure about this change, because it seems like we are using the git
URLs in other user facing ways. For example, some of the snapshot diffs
look like this:
     Installed 2 packages in [TIME]
      - project==0.1.0 (from file://[TEMP_DIR]/)
      + project==0.1.0 (from file://[TEMP_DIR]/)
-     + uv-public-pypackage==0.1.0 (from git+https://github.com/astral-test/uv-public-pypackage@0dacfd662c64cb4ceb16e6cf65a157a8b715b979?rev=0.0.1#0dacfd662c64cb4ceb16e6cf65a157a8b715b979)
+     + uv-public-pypackage==0.1.0 (from git+https://github.com/astral-test/uv-public-pypackage@0dacfd662c64cb4ceb16e6cf65a157a8b715b979)

which seems non-ideal. It should be possible to make the lock file use
structured fields while keeping the URL for terser output as above, but
I'm not sure that it's worth it.
This builds on top of #4627

I can own this one.

@charliermarsh OK, I've updated this PR to include the revision in the lock file so that doesn't get dropped.
I think the main issue remaining here is that some of the snapshot diffs for uv pip compile output look around due to the URLs changing. I think the root cause is that, since the lock file no longer uses a single URL string to represent all of the git information, things external to the lock data structure can't reuse it. So we might need a step that flattens the new structured representation to the URL format for uv pip compile?
We could also just keep the URL representation in the lock file and not use structured data. I wouldn't mind that personally.


format SQL introduce blank before underscore

System information:

Operating system (distribution) and version: Linux 5.16.20-200.fc35.x86_64
DBeaver version: community Version <IP_ADDRESS>204170718

Connection specification:
Not relevant
Describe the problem you're observing:
If a query as a database with a name starting with a number followed by an underscore
select * from 2015_autovetture.tblmarca t;
the "Format SQL" produce a wrong query due to an extra blank inserted after the number before the underscore
select
	*
from
	2015 _autovetture.tblmarca t;

Steps to reproduce, if exist:

new SQL script
write select * from 2015_autovetture.tblmarca t;
apply "Format SQL"
a blank is inserted before the underscore ...2015 _autovetture.tblmarca t;

Hello @Marco-InForge
Thanks for the bug report.
This ticket is duplicate of https://github.com/dbeaver/dbeaver/issues/15605
You can read discussion about this problem there.


Make toggles consistent on Navigation - UmbNav data type

Going down the list of toggleable features in the data type, it can get a little confusing as to which toggles disable something and which ones enable something. It would be great if all toggles behaved consistently, instead of having some toggles Allow something and others Hide something.

I agree, I will see what I can do with these to make them more consistent :)

Ok so I've had a play with this.
It would be a pretty crucial change, as the configuration items are set in a c# class rather than in the model, and modifying the value doesn't work.
Instead I have re-ordered the items so that they are more consistent.

I can imagine it would be a big effort. Thanks for looking into it.


[SPARK-32105][SQL]Refactor current ScriptTransformationExec code

What changes were proposed in this pull request?

Renamed  hive transform scrip class hive/execution/ScriptTransformationExec to hive/execution/HiveScriptTransformationExec (don't rename file)
Extract class BaseScriptTransformationExec  about common code used across SparkScriptTransformationExec(next pr add this) and HiveScriptTransformationExec
Extract class BaseScriptTransformationWriterThread of writing data thread across SparkScriptTransformationWriterThread(added next for support transform in sql/core )  and  HiveScriptTransformationWriterThread ,
HiveScriptTransformationWriterThread additionally supports Hive serde format
Rename current Script strategies in hive module to HiveScript, in next pr will add SparkScript strategies for support transform in sql/core.

Todo List;

Support transform in sql/core base on BaseScriptTransformationExec, which would run script operator in SQL mode (without Hive).
The output of script would be read as a string and column values are extracted by using a delimiter (default : tab character)
For Hive, by default only serde's must be used, and without hive we can run without serde
Cleanup past hacks that are observed (and people suggest / report)， such as
- Solve string value error about Date/Timestamp in ScriptTransform
- support use transform with aggregation
- support array/map as transform's input
Use code-gen projection to serialize rows to output stream()

Why are the changes needed?
Support run transform in SQL mode without hive
Does this PR introduce any user-facing change?
Yes
How was this patch tested?
Added UT

@gatorsmile @tejasapatil @wangyum
I have restart implement ScriptTransform in sql/core now,
Rewrite origin pr in current code. Hope for your review and suggestion

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Hi, @AngersZhuuuu . This is too big as a follow-up. Please create a new JIRA issue for this.

ok to test

Test build #120696 has started for PR 27983 at commit 0d9c437.

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-K8s/25395/
Test PASSed.

Test build #120696 has finished for PR 27983 at commit 0d9c437.

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/120696/
Test FAILed.

Hi, @AngersZhuuuu . This is too big as a follow-up. Please create a new JIRA issue for this.

Seems not need to create new jira? just delete follow-up?

@dongjoon-hyun
In latest test,
org.apache.spark.sql.execution.script.ScriptTransformationSuite.SPARK-14400 script transformation should fail for bad script command

Seem sometimes it won't catch the subprocess's exception.
I tried a lot of times, sometimes failed, sometimes success. May have problem about catch subproc's exception.

Test build #122194 has started for PR 27983 at commit 6fcb1af.

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-K8s/26866/
Test PASSed.

Test build #122194 has finished for PR 27983 at commit 6fcb1af.

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

Merged build finished. Test FAILed.

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/122194/
Test FAILed.

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-K8s/26885/
Test PASSed.

Test build #122214 has started for PR 27983 at commit 1e7c3df.

Test build #122214 has finished for PR 27983 at commit 1e7c3df.

This patch fails from timeout after a configured wait of 400m.
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/122214/
Test FAILed.

Test build #122637 has started for PR 27983 at commit aea128b.

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-K8s/27299/
Test PASSed.

cc @cloud-fan  Can you take a look?

Test build #122637 has finished for PR 27983 at commit aea128b.

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/122637/
Test FAILed.

Test build #122637 has finished for PR 27983 at commit aea128b.

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

@dongjoon-hyun For the latest failed test, Did I miss something?

Retest this please

Test build #123517 has started for PR 27983 at commit aea128b.

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-K8s/28140/
Test PASSed.

Test build #123517 has finished for PR 27983 at commit aea128b.

This patch fails due to an unknown error code, -9.
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/123517/
Test FAILed.

retest this please

Test build #123525 has started for PR 27983 at commit aea128b.

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-K8s/28148/
Test PASSed.

Test build #123525 has finished for PR 27983 at commit aea128b.

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/123525/
Test PASSed.

cc @viirya @maropu @cloud-fan @dongjoon-hyun @wangyum @gatorsmile @HyukjinKwon

Hi, @AngersZhuuuu . This is too big as a follow-up. Please create a new JIRA issue for this.

I have make new jira and make corresponding sub-tasks.

why do we need to keep HiveScriptTransformationExec? What's the behavior difference between the native and hive script transformation?

why do we need to keep HiveScriptTransformationExec? What's the behavior difference between the native and hive script transformation?

Implement ScriptTransformBase support Hive Serder

I'm not very familiar with script transformation. I think it's just: Spark sends data to the script, run the script, the script returns result to Spark. The serde (data exchange) should be invisible to end users?

I'm not very familiar with script transformation. I think it's just: Spark sends data to the script, run the script, the script returns result to Spark. The serde (data exchange) should be invisible to end users?

To be honest,   in Spark side, we transfer data inside with DF, we don't need to support hive serde.
But for script, it can support spark data write in/out with serde format.  Although this is a strange usage in my eyes, there is no guarantee that there will not be such a user.   If we get rid of that case, we can just support non-serde mode

it can support spark data write in/out with serde format

What do you mean? Is there a special script transformation SQL syntax to specify the hive serde?

it can support spark data write in/out with serde format

What do you mean? Is there a special script transformation SQL syntax to specify the hive serde?

In antlr4, transformClause is defined as below:
transformClause
    : (SELECT kind=TRANSFORM '(' namedExpressionSeq ')'
            | kind=MAP namedExpressionSeq
            | kind=REDUCE namedExpressionSeq)
      inRowFormat=rowFormat?
      (RECORDWRITER recordWriter=STRING)?
      USING script=STRING
      (AS (identifierSeq | colTypeList | ('(' (identifierSeq | colTypeList) ')')))?
      outRowFormat=rowFormat?
      (RECORDREADER recordReader=STRING)?
    ;

I have not looked at details yet. Generally I think this PR includes both refactoring Hive ScriptTransformation and adding ScriptTransformation in sql/core. Maybe we can refactor it first and add ScriptTransformation in sql/core later.

There are some problem in current ScriptTransformation:

support use transform with aggregation SPARK-28227
support array/map as transform's input SPARK-22435
Handle DateType/TimestampType un correctly.

In pr https://issues.apache.org/jira/browse/SPARK-28227 , I was suggest to implement in sql/core first.  In my opinion,  both way is clear if we design this part well.

Test build #123933 has started for PR 27983 at commit 2030935.

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-K8s/28555/
Test PASSed.

Test build #123933 has finished for PR 27983 at commit 2030935.

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/123933/
Test PASSed.

Is there anything need to update ？

@AngersZhuuuu, I think what @viirya suggested is just refactor it first without new codes, and add ScriptTransformationExec.scala in a separate PR because it's difficult to review.

Test build #124549 has started for PR 27983 at commit bb6676b.

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-K8s/29170/
Test PASSed.

@AngersZhuuuu, I think what @viirya suggested is just refactor it first without new codes, and add ScriptTransformationExec.scala in a separate PR because it's difficult to review.

I split this pr, make this one only refactor current code. and will add ScriptTransformationExec in pr  of https://issues.apache.org/jira/browse/SPARK-32105
In this issue split each step
https://issues.apache.org/jira/browse/SPARK-31936

Test build #124549 has finished for PR 27983 at commit bb6676b.

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/124549/
Test FAILed.

Test build #124550 has started for PR 27983 at commit f8142f1.

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-K8s/29171/
Test PASSed.

Test build #124550 has finished for PR 27983 at commit f8142f1.

This patch fails to build.
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/124550/
Test FAILed.

Test build #124551 has started for PR 27983 at commit 0e21f18.

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-K8s/29172/
Test PASSed.

Test build #124551 has finished for PR 27983 at commit 0e21f18.

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/124551/
Test PASSed.

Let's keep the PR description and title up-to-date.

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-K8s/29255/
Test PASSed.

Test build #124637 has started for PR 27983 at commit 4a40514.

Test build #124641 has started for PR 27983 at commit c33e0fb.

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-K8s/29260/
Test PASSed.

Test build #124637 has finished for PR 27983 at commit 4a40514.

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/124637/
Test PASSed.

retest this please

Test build #124728 has started for PR 27983 at commit c33e0fb.

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-K8s/29339/
Test PASSed.

Test build #124728 has finished for PR 27983 at commit c33e0fb.

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/124728/
Test FAILed.

retest this please

Test build #124731 has started for PR 27983 at commit c33e0fb.

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-K8s/29342/
Test PASSed.

Test build #124731 has finished for PR 27983 at commit c33e0fb.

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/124731/
Test FAILed.

retest this please

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-K8s/29353/
Test PASSed.

Test build #124743 has started for PR 27983 at commit c33e0fb.

Test build #124743 has finished for PR 27983 at commit c33e0fb.

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/124743/
Test FAILed.

@HyukjinKwon latest jenkins test failed seems not my change related?

retest this please

Test build #124781 has started for PR 27983 at commit c33e0fb.

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-K8s/29392/
Test PASSed.

Test build #124781 has finished for PR 27983 at commit c33e0fb.

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/124781/
Test FAILed.

retest this please

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-K8s/29477/
Test PASSed.

Test build #124871 has started for PR 27983 at commit c33e0fb.

Added ScriptTransformationExec which would run script operator in SQL mode (without Hive).

Can you update the PR description to match the patch itself?

Test build #124889 has started for PR 27983 at commit c49ed51.

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-K8s/29495/
Test PASSed.

Test build #124871 has finished for PR 27983 at commit c33e0fb.

This patch fails PySpark 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/124871/
Test FAILed.

Test build #124889 has finished for PR 27983 at commit c49ed51.

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/124889/
Test PASSed.

retest this please

retest this please

retest this please

Test build #124954 has started for PR 27983 at commit 697dc81.

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-K8s/29560/
Test PASSed.

Test build #125002 has started for PR 27983 at commit 68736e9.

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-K8s/29609/
Test PASSed.

Test build #125002 has finished for PR 27983 at commit 68736e9.

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/125002/
Test FAILed.

Test build #125003 has started for PR 27983 at commit f798acc.

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-K8s/29610/
Test PASSed.

Test build #125003 has finished for PR 27983 at commit f798acc.

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/125003/
Test FAILed.

Test build #125009 has started for PR 27983 at commit 8880931.

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-K8s/29617/
Test PASSed.

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-K8s/29620/
Test PASSed.

Test build #125013 has started for PR 27983 at commit f52f376.

Test build #125009 has finished for PR 27983 at commit 8880931.

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/125009/
Test FAILed.

Test build #124954 has finished for PR 27983 at commit 697dc81.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
abstract class ScriptTransformIOSchemaBase extends Serializable

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/124954/
Test FAILed.

Test build #125013 has finished for PR 27983 at commit f52f376.

This patch fails due to an unknown error code, -9.
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/125013/
Test FAILed.

retest this please

Test build #125055 has started for PR 27983 at commit f52f376.

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-K8s/29662/
Test PASSed.

Test build #125055 has finished for PR 27983 at commit f52f376.

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/125055/
Test FAILed.

retest this please

Test build #125070 has started for PR 27983 at commit f52f376.

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-K8s/29678/
Test PASSed.

Test build #125070 has finished for PR 27983 at commit f52f376.

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/125070/
Test FAILed.

retest this please

@dongjoon-hyun do you know why test failed a lot of times of https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/125070/testReport/
In my local it's ok to run.

Test build #125291 has started for PR 27983 at commit f52f376.

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-K8s/29902/
Test PASSed.

Test build #125291 has finished for PR 27983 at commit f52f376.

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/125291/
Test FAILed.

retest this please

Test build #125294 has started for PR 27983 at commit f52f376.

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-K8s/29905/
Test PASSed.

Test build #125294 has finished for PR 27983 at commit f52f376.

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/125294/
Test FAILed.

retest this please

Test build #125305 has started for PR 27983 at commit f52f376.

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-K8s/29916/
Test PASSed.

Test build #125305 has finished for PR 27983 at commit f52f376.

This patch fails due to an unknown error code, -9.
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/125305/
Test FAILed.

retest this please

Test build #125324 has started for PR 27983 at commit f52f376.

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-K8s/29937/
Test PASSed.

Test build #125324 has finished for PR 27983 at commit f52f376.

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/125324/
Test FAILed.

Could you update the PR description, too?

Could you update the PR description, too?

Done, sorry for always forgot to do that....

Test build #125424 has started for PR 27983 at commit 28cb9ae.

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-K8s/30036/
Test PASSed.

Test build #125424 has finished for PR 27983 at commit 28cb9ae.

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/125424/
Test FAILed.

Could you check the comment, too? https://github.com/apache/spark/pull/27983#discussion_r451514820

Could you update the PR description, too?

Done, sorry for always forgot to do that....

Renamed hive transform scrip class hive/execution/ScriptTransformationExec to hive/execution/script/HiveScriptTransformationExec (don't rename file)

/script/?

Test build #125428 has started for PR 27983 at commit e5cd3d8.

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-K8s/30041/
Test PASSed.

Test build #125431 has started for PR 27983 at commit fcb0957.

Test build #125431 has finished for PR 27983 at commit fcb0957.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Test build #125428 has finished for PR 27983 at commit e5cd3d8.

This patch fails due to an unknown error code, -9.
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/125431/
Test FAILed.

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/125428/
Test FAILed.

retest this please

Test build #125452 has started for PR 27983 at commit fcb0957.

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-K8s/30064/
Test PASSed.

Test build #125452 has finished for PR 27983 at commit fcb0957.

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/125452/
Test FAILed.

retest this please

Test build #125461 has started for PR 27983 at commit fcb0957.

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-K8s/30074/
Test PASSed.

Test build #125461 has finished for PR 27983 at commit fcb0957.

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/125461/
Test FAILed.

retest this please

Test build #125492 has started for PR 27983 at commit fcb0957.

Merged build finished. Test PASSed.


Error: curl_multi_socket_action: 8

To reproduce
using Downloads
function main()
           url = "https://julialang-s3.julialang.org/bin/linux/x64/0.6/julia-0.6.0-linux-x86_64.tar.gz"
           infos = [url, url]
           Downloads.download(url)
               @sync for info in infos
                   @async try
                       Downloads.download(info)
                   catch err
                       @warn "Failed to download binary" exception=(err, catch_backtrace())
                   end
               end

       end
main()

└ @ Downloads.Curl /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.6/Downloads/src/Curl/utils.jl:36
┌ Error: curl_multi_socket_action: 8
└ @ Downloads.Curl /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.6/Downloads/src/Curl/utils.jl:36
┌ Error: curl_multi_socket_action: 8
└ @ Downloads.Curl /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.6/Downloads/src/Curl/utils.jl:36
┌ Error: curl_multi_socket_action: 8
└ @ Downloads.Curl /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.6/Downloads/src/Curl/utils.jl:36
┌ Error: curl_multi_socket_action: 8
└ @ Downloads.Curl /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.6/Downloads/src/Curl/utils.jl:36
┌ Error: curl_multi_socket_action: 8

julia version info:
Julia Version 1.6.5
Commit 9058264a69 (2021-12-19 12:30 UTC)
Platform Info:
  OS: Linux (x86_64-pc-linux-gnu)
  CPU: Intel(R) Xeon(R) CPU E5-2686 v4 @ 2.30GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-11.0.1 (ORCJIT, broadwell)
Environment:
  JULIA_GPG = 3673DF529D9049477F76B37566E3C7DC03D6E495
  JULIATEAM_OS = UBUNTU
  JULIAHUB_MODE = false
  JULIA_VERSION = 1.6.5
  JULIA_DEPOT_PATH = /juliateam/depot
  JULIA_PATH = /usr/local/julia
  JULIA_PKG_SERVER = http://juliateamwebsrvr:9999

Dup of https://github.com/JuliaLang/Downloads.jl/issues/110. That issue's title isn't correct since it applies to downloads on multiple tasks as well.

Yeah, known issue on 1.6 and the changes to make it threadsafe are too significant to backport (unless we decide that it's ok to backport significant features in stdlibs to the LTS version of Julia, which would certainly make my life easier).


Compatibility check for 1.0.0 & <IP_ADDRESS>

Hi,
I'm about to use your tool for checks towards Kafka 1.0.0, noticed it was not in the list.
Tried your awesome compatibility test and it passed :)

Anyway we can get this merged in?

Thank you for taking your time to create & submit this PR! I've just now merged PR  https://github.com/andreas-schroeder/kafka-health-check/pull/22 which already covers these versions.


updates to support blanc data release

For the record:
Fitting results using this branch and based on the deep coadds are in /global/cfs/cdirs/desi/spectro/fastspecfit/blanc/tiles.
The merged catalogs (fits to the photometry and, separately, the spectroscopy) are here--
/global/cfs/cdirs/desi/spectro/fastspecfit/blanc/tiles/merged/fastphot-blanc-deep.fits
/global/cfs/cdirs/desi/spectro/fastspecfit/blanc/tiles/merged/fastspec-blanc-deep.fits

with the data model here--
https://fastspecfit.readthedocs.io/en/latest/fastphot.html
https://fastspecfit.readthedocs.io/en/latest/fastspec.html


Deployment/usage interactive map

We should try to collect data for (we already have some in our education listing) , and display on the website, an interactive map that shows, as a starter:

Uses of Jupyter in education.
Deployments of JupyterHub.

For each of these, I'm imagining a map marker (probably different colors), which upon hover/click, pops up a small info note with a short description and a link, if relevant, to e.g. the course website or relevant github repo.
If a given deployment uses both Hub and is for education, we can use a different marker.
We were discussing today with @SylvainCorlay whether using the ipyleaflet widget might be a good solution for this...

It seems like a perfect usecase for the static widgets.

Love the idea of using the leaftlet widget - we could provide a link to the
notebook that generated it.
On Tue, May 31, 2016 at 4:29 PM, Sylvain Corlay<EMAIL_ADDRESS>wrote:

It seems like a perfect usecase for the static widgets.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jupyter/jupyter.github.io/issues/96#issuecomment-222851458,
or mute the thread
https://github.com/notifications/unsubscribe/AABr0CGy7q8IeJGpTm5-MPAEwRmKVWGWks5qHMRMgaJpZM4IrBKf
.

--
Brian E. Granger
Associate Professor of Physics and Data Science
Cal Poly State University, San Luis Obispo
@ellisonbg on Twitter and GitHub
<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

We are currently thinking of making a page for each of the Jupyter subproducts (including JupyterHub). I think the JupyterHub page would be a perfect place to showcase this!

Since this ticket has been inactive for four years, I'm going to close as stale. If you'd like to continue the discussion, feel free to chime in.


Adds the ability to remove and add tracks at will.

This allows people to disable their video or mute their audio. In my use case I needed to be able to let the user turn off their video, and mute themselves.

It also removes lots of trailing whitespace.


Fix unclosed paths for TrueType glyphs.

TrueType glyph outlines were not being closed properly; only a single closePath command was being issued for each glyph.  This fix ensures that a closePath command is issued for each contour.

Looks good, thanks!


How do I access the plugin?

I want to disable the @mention module on discord whenever I reply to someone.

Once installed, you'll see a Better Replies tab in your settings where you'll be able to configure the plugin and disable mentions by default for outgoing replies (along with additional features).

Thank you.
Sent from my iPhone

On 17 Sep 2021, at 17:15, Cynthia @.***> wrote:

Once installed, you'll see a Better Replies tab in your settings where you'll be able to configure the plugin and disable mentions by default for outgoing replies (along with additional features).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Add example and CLI to docs

Closes #34.
Changes proposed:

Copy one of the docstring examples to a Jupyter notebook in the docs.

There are a lot of log messages and formatting in it.

Add CLI to the docs.

I broke some stuff, so I'll need to fix it before I can ask for a review.

good with me
merge at will
thanks @tsalo


ft: update versioning doc to reflect changes

Will need @nicolas2bert's help to figure out how to render the documents for preview on the readthedocs :smile:
See rendered changes here: http://s3-server.readthedocs.io/en/doc-adddoc/ARCHITECTURE/#versioning

When https://github.com/scality/Arsenal/pull/262 gets merged, it will change a bit how versioning code behaves:
The "Metadata Versioning Options" should be updated to reflect this: when versioning and versionId are both set, instead of saying that it returns an error, it could say that it's only used for internal needs (e.g. CRR). What is does is create or update a specific version + the master only if the master is older than the written version. I'm not convinced we need to explain this in the doc though.
I guess the easiest is to update the doc in this PR directly.

@jonathan-gramain does the update look OK for when your pr is merged?

@jonathan-gramain updated again - if you can proof sometime, can you confirm you agree with the description?

@electrachong Looks good to me!


mixer wrong dimension and green

hey 🤚  great project!! ❤
having the following problem.
when i have a couple of sources (uri, mp4) - and i cut_to_source them on the mixer1
on the first run it looks like:
(wrong color, and i think it is the wrong dimensions)

clicking on the eye  deleting it from the mixer, and cut_to_source again, fixes it

if i add an output directly to the source, it is stable/normal from the beginning. so i suspect some problem in the mixer, do you have any idea, where i could start looking at?

once all sources have been toggled atleast once, the cut_to_source works flawless 👍

Hello  @hjanuschka.
We've not seen any issue like this before. What type of uri sources have you been testing with?
I'm not a 100% sure but it could be some issues with the caps not being parsed correctly by underlying gstreamer.

it seemed to be a issue with gst 1.17-HEAD - rolled back to 1.16 and it was working.


Enable swap in sysprep specialize

If the swap is disabled, the issues at boot time with not enough
free disk space is solved, but the swap must be re enabled to not
be an issue if the RAM is too low.

Tested on Win2012R2 and Win2008R2 and it's good.


Any way to save an URL to favourites or play an m3u file?

Hi! I'm using MX Player on my Android TV to show an rtsp stream, cause it works much better than VLC. I need to open the stream at every startup without the typing it from zero, but in the program no way to create a favourite or to play an m3u file (that works with VLC).
Any suggestion how to solve this? Thnak you a lot!

I hope you may answer me, I wrote you here, email, facebook, xda forum, no answers..

Hi, please help


Syntax for Fragments

Right now we have a impl with varargs but that is a bit messy and buggy to work with. Would an openarray/seq-based approach be viable instead?
Examples:
# current approach:
fragment(@[fadeIn], @[highlightRed], @[fadeOut])

# seq-based approach:
fragment(@[@[fadeIn], @[highlightRed], @[fadeOut]])

The second approach is considerably uglier to me but easier to work with. Most importantly it would allow us to pass in more parameters to fragment, like end-animations that are applied at the end of a block.
I think using an all-seq implementation internally is needed, but then we can add sugary varargs functions for ease of use for common usecases. How should the distinction between them be API-wise then? Should the sugar or the low-level functions have the fragment name? (Both can't have it as varags and seq functions will clash with each other). So either we have fragmentSugar + fragment or fragment + fragmentLowLevel.
A third option would be to write a macro, but I'd rather not...

After some thought, I think I will use the naming scheme fragmentCore for the low-level seq-based function and fragment for the higher-level ones.

fragmentCore is the seq-based fundation that all varargs sugar is built on top of.


multiarch-latest fails to run

Hi,
2021.20.0-multiarch-latest is running fine, after update the server does not run anymore:
Downloading minecraft_server.[0.003s][error][cds]_Unable_to_map_CDS_archive_--_os::vm_allocation_granularity()_expected:_4096_actual:_65536
Any ideas what is going on?
Best regards,
hansnull

I'm not sure what that error means at all. From some of the keywords I'm guessing it related to the memory conditions of the docker host.
BTW, please switch to one of the java* image tags since those "multiarch" ones are deprecated.


raw data for dexcom

Great watch face...any chance to get raw data in? thank you

I was thinking raw data as a single-pixel-width line would work fairly well in the UI.  I suspect the biggest barrier to raw data support is the size of the message required to transmit raw data.

At the least, we can show the most recent raw readings and noise in the status bar. I haven't had a chance to understand how the current Pebble app does it (I use Share), but that feature is coming soon.
Graphing raw data is a bit more complicated, but it's not out of the question. There are some other updates that are higher priority before I'll have a chance to look into that.

Raw data is pretty simple.
From /api/v1/entries/cal.json:
{
_id: "5663509a658c460aec52fbcd",
device: "xDrip-DexcomShare",
type: "cal",
date:<PHONE_NUMBER>568,
dateString: "2015-12-05T14:59:51.568-0600",
slope: 825,
intercept: 30205,
scale: 0.9883771940276049
},

And from /api/v1/entries/sgv.json:
{
_id: "5663c24b658c460aec52fc2d",
device: "xDrip-DexcomShare",
date:<PHONE_NUMBER>264,
dateString: "2015-12-05T23:05:50.264-0600",
sgv: 132,
direction: "Flat",
type: "sgv",
filtered: 155000,
unfiltered: 156000,
rssi: 100,
noise: 1
},

This is the code from my dex library that applies a calibration to a raw value from the receiver:
interface Calibration {
    val slope: Double
    val intercept: Double
    val scale: Double
    val decay: Double
    fun apply(bgl: RawGlucoseValue): Double? {
        val mgdl = bgl.mgdl
        val filtered = bgl.filtered
        val unfiltered = bgl.unfiltered
        return if (unfiltered != null) {
            if ((mgdl != null) && (mgdl >= 40.0) && (mgdl <= 400.0) && filtered != null) {
                val ratio = scale * (filtered - intercept) / slope / mgdl
                scale * (unfiltered - intercept) / slope / ratio
            } else
                scale * (unfiltered - intercept) / slope
        } else
            null
    }
}

If the sgv field is unavailable (recently restarted, noise, etc), it uses the unfiltered value.  Otherwise it scales the weight of the filtered/unfiltered values dynamically.
Algorithmically it is the same as NightScout's implementation, only somewhat more compact.

I went ahead and implemented it in https://github.com/mddub/nightscout-graph-pebble/pull/5

Thank you i will give it a try. Many thanks again

@mgranberry, thanks for implementing this! Some thoughts:

Would it be worthwhile to drop "Raw" and include "Cln/Lgt/Med/Hvy" instead?
I think we should reverse the order of the numbers so that they go from older to newer (I think the graph biases towards that order). What do you think?
My sense is that if people have raw, they also have a rig battery level. Is that right? Should there be a status bar option that shows both?
Does cgm-pebble show the last 2 raw readings, or present the last value using both unfiltered and filtered? Not saying we should do the same thing, but if it's different, it might be worth calling the option "Raw Dexcom readings (last 2)" to avoid confusion.

I'm comfortable making these changes, but wanted some input since I don't use raw or xDrip.

Imho i think raw are useful. A great gap between raw and BG is of course clue of noise. I can't use it on pebble time however. Settings menu is not accessible

yes, I think so.  "raw" contains no information after a user is comfortable with it.
I've been thinking the same thing after using it for a day.
Probably.
it shows the three most-recent readings, but it does it without any consistency.  It didn't look like three would fit with enough space to be readily interpretable.  Nothing shows just filtered/unfiltered, but in most cases filtered is just unfiltered delayed by a few minutes.  The nightscout "scaled" display does a pretty good job of showing filtered when flat and unfiltered when BG is changing rapidly.  If nothing else it's what's expected because that's what nightscout and the cgm-pebble watchface do.

When i load it on pebble time i go to settings but it freezes on loading settings and then it says app unsuccesfully installed. Sometimes pebble time app crashes instead trying to load it

Tried also on pebble classic app. No settings access.

It's a problem of last commit because the pbw linked on readme.MD works

Another thing missing is the time ago indication imho. However i'm using it since this morning and it seems very stabile and full working.

Awesome! Maybe you were compiling it in a way that made the config page inaccessible from your phone. That would explain the app being unable to install. The watchface shouldn't crash in the absence of communication with the phone on startup, but I haven't actually tested that on a Time.
24 hour time format and showing noise and rig battery with raw will definitely be part of the next release. And yeah, I've also found myself wanting vertical gridlines... so they're coming Soon.
To show time since last update, Jason had a great suggestion of a pie chart icon that fills up (or empties) as it gets closer to the time of the next update. It could go in any corner of the screen like the battery icon. The fewer numbers, the better :)

Excellent. It will be the best watchface ever! Last thing missing: iob informations....

Made the changes to raw data I suggested above, plus support for mmol:
https://github.com/mddub/nightscout-graph-pebble/commit/e1fe03
https://github.com/mddub/nightscout-graph-pebble/commit/a0846d

@francesc0-cgm, by IOB, do you mean IOB calculated from CarePortal treatments? The one displayed in the pill on Nightscout web?

Yep thanks

But i will considerate it for an optional adding.

A bug i found is when you go away from the phone it will not connect automatically again. It shows bluetooth symbol with a line on and the time ago of disconnection but you have to restart the WF to let it works again

@francesc0-cgm, alarms are absolutely coming (eventually). IOB from NS would be straightforward to add by pulling it from the /pebble endpoint. Not sure 3 raw points would fit with the rig battery and noise (37% Cln 107 109 113), but I'll consider exposing number of raw points as an option (especially for when the status bar is taller).
For the connection issue, can you give steps to reproduce? On my iPhone, this is what I tried:

disable Bluetooth on iPhone
wait 15 minutes
see "no bluetooth" indicator + "15" on the watch
enable Bluetooth on iPhone
watchface gets new data

It could be an Android-only problem, or perhaps there is a difference between disabling Bluetooth and going out of range of the phone. Either way, being able to reproduce it is the first step.

Disconnection issue never happened again. It could have been a BT problem of my phone. 3 raw will be awesome. Maybe you can reduce size of pebble battery icon and put it next to the time indication on the upper part, 3 raw, noise indication and rig battery will fit on the bottom bar. Iob could be inserted on the left of time imdication.

I am running 3 points with both battery indicators on the bottom row now
and it is a very tight fit.  Some of the text is drawn over the battery
icon if BG is stable in the 140s.  At most other times it is close but not
obscured.
On Mon, Dec 14, 2015, 2:01 AM francesc0-cgm<EMAIL_ADDRESS>wrote:

Disconnection issue never happened again. It could have been a BT problem
of my phone. 3 raw will be awesome. Maybe you can reduce size of pebble
battery icon and put it next to the time indication on the upper part, 3
raw, noise indication and rig battery will fit on the bottom bar. Iob could
be inserted on the left of time imdication.
—
Reply to this email directly or view it on GitHub
https://github.com/mddub/urchin-cgm/issues/2#issuecomment-164372650.

@mddub could you move away the pebble battery meter and reduce it. I
appreciate editability of this watchface but for a reason of development
will be better to set a standard Aspect with most part of datas on screen?
Inviato con AquaMail per Android
http://www.aqua-mail.com
Il 15 dicembre 2015 03:44:08 AM Matthias Granberry
<EMAIL_ADDRESS>ha scritto:

I am running 3 points with both battery indicators on the bottom row now
and it is a very tight fit.  Some of the text is drawn over the battery
icon if BG is stable in the 140s.  At most other times it is close but not
obscured.
On Mon, Dec 14, 2015, 2:01 AM francesc0-cgm<EMAIL_ADDRESS>wrote:

Disconnection issue never happened again. It could have been a BT problem
of my phone. 3 raw will be awesome. Maybe you can reduce size of pebble
battery icon and put it next to the time indication on the upper part, 3
raw, noise indication and rig battery will fit on the bottom bar. Iob could
be inserted on the left of time imdication.
—
Reply to this email directly or view it on GitHub
https://github.com/mddub/urchin-cgm/issues/2#issuecomment-164372650.

Reply to this email directly or view it on GitHub:
https://github.com/mddub/urchin-cgm/issues/2#issuecomment-164626250

I agree it's worth optimizing the display for the most common data scenarios, but it's still a work-in-progress. At the moment I'm focusing on making the layout configurable from the phone, which will also enable having several preset layouts. Some layouts will inevitably be more optimal than others for certain types of data. In the meantime, you can configure the battery icon to be in the time area (and move the time to the left or right), or remove the battery indicator. And of course you could remove the right-padding from the battery.
It's probably worth standardizing the size of icons like battery, but I'm waiting until there's a second icon to do that. If you want to make a smaller battery icon, pull requests are welcome :)

I made the number of raw points configurable in https://github.com/mddub/urchin-cgm/commit/2b0eda. This will be more useful when combined with a configurable layout (so someone could, say, make the status bar twice as tall and view the last ~8 raw points), but you can use it to show 3 points now.
Thanks so much for all your testing and feedback. Since the raw feature has been implemented, I'm closing this issue. Feel free to open another issue for other feature suggestions.


Add Feature - Categories rearrange

Now you can re-arrange categories based on your preference

@sameersyd PR merged 🚀. Thanks for bringing up this 🙌


expired feed warning  on feeds with valid calendar ranges

It appears that feeds with valid calendar.txt ranges but no future valid calendar_dates.txt data are marked as expired. I assume this is because  the calendar_dates range trumps calendar.txt.

Worse than that, I'm afraid; calendars are not being considered at all (!)

No, I take that back.


Automatically search in all clients directories for games on scan/backup actions

Currently Ludusavi uses expects the default locations of where games are installed to detect the saves automatically. This can be an issue where games are not installed in the default location and use <path-to-game> as the save directory. The current solution is to add the location of all extra non-default library directories as a root.
The proposal is to automatically scan for all the library locations of all clients automatically.
Steam

Scan the SteamPath registry key in HKEY_CURRENT_USER\SOFTWARE\Valve\Steam to find steam installation directory.

Read the <SteamPath>\steamapps\libraryfolders.vdf in to get all the extra steam library directories.

Epic

Scan the ModSdkMetadataDir registry key in HKEY_CURRENT_USER\SOFTWARE\Epic Games\EOS to find steam installation directory.

Read the *.item manifests that are in that directory to get the games install locations

Still need to look what can be done in other clients like Uplay, Origin, GOG Galaxy or itch.io. A possible solution to this, but that could only be used in Ludusavi-Playnite would be to use the InstallDirectory property of the game.

I've implemented this with a "find roots" button:

https://user-images.githubusercontent.com/4934192/179449870-32461c89-faf8-4001-95bc-7d1f21e0d024.mp4


chore(rename.env.variable): prefix with FABRIC8 to match convention"

@xcoulon as discussed rename variable to FABRIC8_REALM.
I've aslo rename it in planner in: https://github.com/fabric8io/fabric8-planner/pull/1913/commits/039401db666c5214a2844660030aabda19437515
Is it ok to merge?

There were the following issues with your Pull Request

Commit: 92a53462c78e415ca73668e7f97b0010ae95317d
Your subject line is longer than 50 characters

Guidelines are available at https://github.com/fabric8io/fabric8-ui

This message was auto-generated by https://gitcop.com

@corinnekrych snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-1376-fabric8-ui.int.rdu2c.fabric8.io

Checked prod and prod-preview deployments, we don't use any env variable to set the realm.
We allow the defaults to be used - so this PR being merged shouldn't impact anything when elevated all the way to production.

Don't forget to update the Stack Recommender as they added the Realm too.
https://github.com/fabric8-ui/fabric8-recommender/pull/214

@joshuawilson I've rename it it there too https://github.com/fabric8-ui/fabric8-recommender/pull/216
Although keeping old name won't make the build/runtime fails but agreed, to align naming is better.
cc @xcoulon

There were the following issues with your Pull Request

Commit: 92a53462c78e415ca73668e7f97b0010ae95317d
Your subject line is longer than 50 characters

Guidelines are available at https://github.com/fabric8io/fabric8-ui

This message was auto-generated by https://gitcop.com

@corinnekrych snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-1376-fabric8-ui.int.rdu2c.fabric8.io


Remove reference to global maps

Try to fix https://github.com/dfinity/motoko-playground/issues/148

I can confirm this indeed fixes the problem.


enh(CPlusPlus): memory usage on Dijksta algorithm

https://github.com/MakeContributions/DSA/issues/1058 - try to fix this problem
I optimize the Dijkstra algorithm in C++
The Dijkstra algorithm uses vector<vector<pair<int,int>>> graph(V+1,vector<pair<int,int>>()) as a matrix. This is very, very bad for memory, because the vector in the vector in memory is not stored linearly, and can be scattered in memory very much. As an alternative, I propose to use a regular array (data = new T [rows * columns]) allocated on the heap and get its elements like this : *(_data + row * _rows + column).

Can you check one more time?


Add zoneinfo Library from Python 3.12.6

In this PR I have added zoneinfo with the corresponding test suite.

Thank you so much for contributing! Just one more thing. Could you check which Python version did you brought the library from?

Good afternoon,
This library was introduced as is in 3.11, but it is compatible with 3.12 and so far with what has been developed for python 3.13.0rc2

Do you mean you copied them from 3.13.0rc2? That's for record purpose. We usually leave version information the copied version for later.

Do you mean you copied them from 3.13.0rc2? That's for record purpose. We usually leave version information the copied version for later.

Hello,
I prefer to say that it is taken from python 3.12.6.

Thank you for confirming!
And welcome to the project.


Correction for raw affiliation Department of Haematology; Hospices Civils de Lyon; Groupement Hospitalier Sud; Pierre Bénite France; UMR 5239; Université Lyon 1; Lyon France

Correction needed for raw affiliation Department of Haematology; Hospices Civils de Lyon; Groupement Hospitalier Sud; Pierre Bénite France; UMR 5239; Université Lyon 1; Lyon France
raw_affiliation_name: Department of Haematology; Hospices Civils de Lyon; Groupement Hospitalier Sud; Pierre Bénite France; UMR 5239; Université Lyon 1; Lyon France
new_rors: 023xgd207;01502ca60;029brtt94;01bj4fd12
previous_rors: 023xgd207;01502ca60;029brtt94
works_examples: W2073744627
contact: 966f2abd50b520551c4429c0ec087e89:c11bf91992e855d3bfb815f2e79b @ ens-lyon.fr

This issue was accepted and ingested by the OpenAlex team on 2024-11-01. The new affiliations should be visible within the next 7 days.


Error: HyperExpress: Server/Router.use() -> Wildcard "*" & ":parameter" prefixed paths are not allowed when binding middlewares or routers using this method.

Hello, I'm new to Hyper-Express and trying to rewrite my Express auth system in it. Right now I'm trying to forward my /posts/ route to be usable from the comments route so I can do something like post to /api/posts/:id/comment buy I'm getting the error that I can't use wildcards when binding to a middleware.
I'm doing it the same way I do in express as I couldn't seem to find documentation on how to do it in Hyper-Express, so it looks like this from my posts route:
const commentRouter = require('./comments'); const commentRouter = require('./comments');
My entire comments route looks like this:
`
const HyperExpress = require('hyper-express');
const {
getComments,
getComment,
addComment,
updateComment,
deleteComment
} = require('../controllers/comments');
const Comment = require('../models/Comment');
const router = new HyperExpress.Router();
const advancedResults = require('../middleware/advancedResults');
//add protect ability; can now protect any route that needs to be logged in
const { protect, authorize } = require('../middleware/auth');
router.route('/')
.get(advancedResults(Comment, {
path: 'post',
select: 'name description'
}), getComments)
.post(protect, addComment);
router.route('/:id')
.get(getComment)
.put(protect, updateComment)
.delete(protect, deleteComment);
module.exports = router;
`
and the routes are mounted like so to the server.js
app.use('/api/posts', posts); app.use('/api/comments', comments);
Is there a proper method for forwarding routes so I can use the same name scheme I used with Express?

Hello, yeah so the error you are running into is that HyperExpress fundamentally treats middlewares separately from routes.
See middlewares can only run on a certain route pattern and are "wildcard" by default.
For example:
server.use((req, res, next) => {}); // This will run for ALL requests since it has no pattern hence a global match
server.use('/api', (req, res, next) => {}); // This will run for `/api` and ALL `/api/*` requests since it narrows to `/api` pattern

Please keep in mind that HyperExpress is mostly compatible with Express.js with a very familiar API but it is not 100% compatible and the middleware system is currently one of the more major differences. Feel free to read up on our Middleware docs for more information on usage and behavior.


Kick Command not working.

/ob kick 'playername' is not working does not show any error or success message and also does not kick the player from your island.
Repro Steps:
Create you own island.
Invite a player to your island.
Kick the invited player using : /ob kick 'playername'
The above command does nothing.

leave command is the same

fixed 1.2.2


even spacing of products as an option

Option to evenly space products, rather than randomly spacing products.

This has a new implementation but needs to be hooked up to the UI


Doesn't work on a fresh zshrc

Using oh my zsh's zshrc seems to work however.
Something else gets sourced in oh my zsh which makes this plugin work

Seems to be lib/key-bindings.zsh which fixes it. You could add that to the plugin so that it works outside of oh my zsh

Seems to be lib/key-bindings.zsh which fixes it.

What is this file, how does it fix it?
This plugin is only a single .zsh file, it’s up to you how/where you source it. It doesn’t depend on Oh My Zsh in any way, I don’t even use it.

Nevermind, looks like it's working again even without this omz plugin. I have no idea what caused it, but when I was remaking my zshrc this plugin refused to work.


File corrupted

I was running a .py file and it was corrupted when I was accidentally clicked a key, so it went empty but i didn't save it. Now it's still showing the same error even though I have downloaded the new file from the backup. Anyone know what we can do in this type of situation. If we mistakenly click any key and if it went empty what we need to do to undo it. Please someone suggest.

Hey @jeneeshr47, thanks for reporting and sorry to hear that you had this problem with Spyder. You said:

Now it's still showing the same error even though I have downloaded the new file from the backup

Unfortunately, if your backup file is showing the same problem, there's not much you can do to recover its previous contents. We offer autosave functionality to recover a file is Spyder crashes while you're working on it. But this doesn't seem to be the case, so Spyder can't help you to recover it.
Sorry for the bad news, but please let us know if we can help you with something else.

Hey @jeneeshr47, thanks for reporting and sorry to hear that you had this problem with Spyder. You said:

Now it's still showing the same error even though I have downloaded the new file from the backup

Unfortunately, if your backup file is showing the same problem, there's not much you can do to recover its previous contents. We offer autosave functionality to recover a file is Spyder crashes while you're working on it. But this doesn't seem to be the case, so Spyder can't help you to recover it.
Sorry for the bad news, but please let us know if we can help you with something else.

Closing due to lack of response.

Closing due to lack of response.


fix tests for yak_tween and yak_widgets

yak_tween/lib/src/box_constrains.dart

partially fixed in https://github.com/iapicca/yak_packages/pull/124


Bug Report: not very optimized

Please check for duplicates or similar issues, as well performing simple troubleshooting steps (such as clearing cookies, clearing AppData, trying another browser) before submitting an issue.
If you are playing a downloaded version of the game, what operating system are you using?
Windows (x86), Windows (x86_64), Linux, or macOS? Specify below.
Windows 11
What version of the game are you using? Look in the bottom left corner of the main menu.
The latest version

While I play a song, the FPS fall to 30. I am on a PC with a Intel I5 (integrated graphics card) and 8 GB of RAM. I have also a laptop with very similar specs and with 16 GB of RAM (and a SSD but IDK if this information is userful).

i will provide images when i am at home btw

Which chart are you playing

The normal Bopeebo chart.

On the latest commit:

On the latest release downloaded from the GitHub "Releases" tab:

@nebulazorua I am gonna try to fix that in the next week, if I have time and don't forgot.

aight if you wanna
I've been working on mods and optimizations in said mods
These will come to base andromeda at some point when I'm not busy

I discovered that the lags were caused by the raymarching shader. Disabling the shader fix them.

ok sorry but i didn't find a optimized raymarching shader. i will try to do that in the next two weeks maybe.


Download CATH4.3 returns accessdenied

Hello,
When I am downloading the CATH4.3 dataset following instructions in your script, it runs normally to download the chain_set.jsonl file while returns an accessdenied error in the following link https://dl.fbaipublicfiles.com/fair-esm/data/cath4.3_topologysplit_202206/split.jsonl. Would you mind checking the issue or providing an alternative download source?
Much appreciation in advance.

Hi @realfenston, thank you for reporting this issue. I'll check the original url later.
I have uploaded CATH data to google drive at this: cath data. Please have a try.
Best,
Zaxiang

Thanks a lot for your reply!
On Wed, Aug 23, 2023 at 4:04 PM zhengzx-nlp @.***>
wrote:

Hi @realfenston https://github.com/realfenston, thank you for reporting
this issue. I'll check the original url later.
I have uploaded CATH data to google drive at this: cath data
https://drive.google.com/file/d/1zFMpx-m9QTi-ykBJgavZZCMvynsF5E1D/view?usp=sharing.
Please have a try.
Best,
Zaxiang
—
Reply to this email directly, view it on GitHub
https://github.com/BytedProtein/ByProt/issues/5#issuecomment-1689480744,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY6QVZG4QVCYD5SQCNQEF5TXWW2PPANCNFSM6AAAAAA3XIJE7Y
.
You are receiving this because you were mentioned.Message ID:
@.***>


[manila-csi-plugin] e2e fix WIP

e2e fix WIP
NONE

/test openstack-cloud-csi-manila-e2e-test

Hi @gman0,
Are you still working on this?

Hi @pierreprinetti, following @gouthampacha's comment in https://github.com/kubernetes/cloud-provider-openstack/issues/2142#issuecomment-1563664268, there is an issue in Manila's NFS driver.
We can wait until this is fixed or switch to another backend for the moment.


Amazon.AWSCLI version 2.17.35

Checklist for Pull Requests

[ ] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

2024-08-21T23:35:42.8786342Z Manifest Validation Failed
2024-08-21T23:35:47.9276021Z

(Automated response - build 888.)

Close with reason: Duplicate of #168585;


Convert "accountid_other" to community id

I wanted to know if there was a way of converting the "accountid_other" parameter of a trade offer to a community id.
A quick google search threw nothing up, so any help would be appreciated.
Many thanks

Realised I could dive into index.js and find it myself. For anyone who is interested:
Long(parseInt(accountId, 10), 0x1100001).toString()

You are looking for steamid_other property which added automatically to trade offers in responses of .getOffer and .getOffers.


changing the shape of Raycast for better understanding

Note that this pull request was superseded by https://github.com/godotengine/godot/pull/46529.

Much better than my idea of implementation @Calinou ! I'll delete this branch tho


toImageSync should use kPremul_SkAlphaType

Right now it's using kUnpremul_SkAlphaType, which is resulting in darker anti-aliasing around edges, e.g. compare the square on bottom to the one on top:

They're made form the same picture drawn like this:
Future<List<ui.Image>> drawImage() async {
  var recorder = ui.PictureRecorder();
  var canvas = ui.Canvas(recorder);
  canvas.drawRect(ui.Rect.fromLTWH(20, 20, 100, 100),
      ui.Paint()..color = ui.Color(0xFFFF6D00));
  var picture = recorder.endRecording();
  var toImageImage = await picture.toImage(200, 200);
  var toImageSyncImage = picture.toImageSync(200, 200);
  return [toImageImage, toImageSyncImage];
}

The toImage image is on top, the toImageSync image is on bottom. Changing https://github.com/flutter/engine/blob/5f92d3668c7a172be23df13eebd24ab7e7df29ee/lib/ui/painting/picture.cc#L71 to kPremul_SkAlphaType seems to fix this, but probably deserves to be tested to make sure it works - Im' pretty sure I tried unpremul at one point to try to get Linux to be happy, before finally just putting Linux on the software path for now.
@jonahwilliams fyi. If you don't get to this first, I'll pick it up when I come back next week. We may want a golden test that asserts the bytes are the same between these calls.

Thank you Dan!


activationStrategy and determineActivationStrategy have no impact

I am attempting to get a VM to be replaced when the same VM is being navigated to.  however, adding either activationStrategy to the route in the configurations map or implementing a determineActivationStrategy method have no impact.  In-fact, the determineActivationStrategy method within the VM does not appear to be even called.

Here is an example VM that does not work:
import { activationStrategy } from "aurelia-router";

export class TestVM {
    constructor() {
        console.log("in constructor");
    }

    determineActivationStrategy(params: any, routeConfig: RouteConfig, navigationInstruction: NavigationInstruction): string {
        console.log("in determineActivationStrategy");
        return activationStrategy.invokeLifecycle;
    }
}

and the route is configured as:
{ route: ["testvm"], name: "testvm", moduleId: "testvm/index", nav: true, title: this.appService.i18n.tr("common:navigation.testvm"), generationUsesHref: true, href: "testvm" }

This is odd. I just built an app this week that uses this and had no issues. Can you double check to make sure that all your libraries are up to date?

@EisenbergEffect I can certainly do that when I get to my coffee on the morning. Any library in particular?

Definitely check router, templating-router, history-browser and route-recognizer. Those are all the pieces typically related to the routing infrastructure. You might want to check templating and binding as well. If that doesn't fix it, then I'll see if I can repro it in a fresh project. Let me know.

@EisenbergEffect here are the versions of my libraries:
router: 1.3.0
templating-router: 1.1.0
history-browser: 1.0.0
route-recognizer: 1.1.0
templating: 1.4.2
binding: 1.2.1
I have even deleted my node_modules and installed everything again, but no joy :(

@stuartbloom If you prepare a repro I can take a look - preferably a github repo.

thanks @StrahilKazlachev that is exactly what I am doing :)

@StrahilKazlachev @EisenbergEffect I have greated a repo here  it only has three views and models.  a very simple navigation.  click on any of the routes and they work fine; clicking on the route that is currently selected and nothing changes and no output from the determineActivationStrategy method.
Thanks for your assistance

The thing is that the VM won't be replaced when the url isn't changed. You are navigating to the same route without changing any parameters, which will have no effect.
A possible solution might be having the replace option available on the route-href attribute?

Also, are there any difference in behaviour when using push-state and hash in this case?

@stsje I think you may have just given me a hint.  i have updated the repo, and now I do see the determineActivationStrategy get called and then the constructor for the VM. I will leave this open until I move the code to my actual project, but thanks for the hint :)

@stsje Thank you so very much. I now have it working :)
in the nav, i have the following:
${row.title.toUpperCase()}
and then in the navs vm I have
    private goToRoute(route) {
        this.router.navigateToRoute(route, {}, { replace: true });
    }

Now  the determineActivationStrategy does get called and my VMs get reset.
Thank you :)
@EisenbergEffect @StrahilKazlachev Thank you also for looking into this guys; most appreciated :)


No test case: parameterization of exceptions via pytest.param, passing in 'marks='

As I was going through the tests within this project, I noticed there are no tests for parameterization of exceptions via pytest.param, passing in 'marks='.
A little background:
In the past, it has been an acceptable practice to pass pytest.mark.raises directly into your parametrize list if you want to parametrize a test that raises an expected exception.  But now, if you are using a newer version of pytest, you will in fact get a deprecation warning.
RemovedInPytest4Warning: Applying marks directly to parameters is deprecated, please use pytest.param(..., marks=...) instead.

Here is an example of the idiom to which one should probably migrate:
import pytest

class SomeException(Exception):
    pass

@pytest.mark.parametrize(
    'error',
    [None,
     pytest.param(SomeException('the message'),
                  marks=pytest.mark.raises(SomeException(),
                                           exception=SomeException)),
     ])
def test_mark_raises(error):
    if error:
        raise error

And although a bit more verbose, it conforms to the newer pytest, and the warnings go away.
To be clear, pytest-raises version 0.10 does support this and it is working fine for me.  I just didn't see a pytest which documents and verifies this specific feature within the project.

Unfortunately, with the release of pytest 4, this is now a breaking issue :sob:
I've opened an issue with them though - we'll see what they say: https://github.com/pytest-dev/pytest/issues/4395

I'm totally up for changing the tests, but, unfortunately, I don't really have time right now to do this.  I will gladly work with anyone who wants to create a pr for this issue though.

  Sorry, but it will take even more time for somebody who does not know your code and/or pytest framework development internals to achieve the same goal.


Country Request: Palau

Can you add this please?

Hi @herriotto. Palau MapWithAI roads have already been added. You can look at it here.


Add testing of chart using chart-testing

Utilize something like https://github.com/helm/chart-testing

Fixed by #11


Fixed GraphQL Get

This PR fixes #288

thank you for your PR!
Could you add a test? Best would be an integration test that includes references

To avoid any confusion in the future about your contribution to Weaviate, we work with a Contributor License Agreement. If you agree, you can simply add a comment to this PR that you agree with the CLA so that we can merge.beep boop - the Weaviate bot 👋🤖PS:Are you already a member of the Weaviate Slack channel?

I agree with the Contributor License Agreement.

What should be the behavior in case a class_name does not have any non-reference dataType properties ?

What should be the behavior in case a class_name does not have any non-reference dataType properties ?

I'm not 100% sure if I understood that. Do you mean if the class returned by the schema call does not contain a datatype?

@dirkkul, if the below schema is used :
[
  {
     "class": "Person",
     "description": "A person such as humans or personality known through culture",
     "properties": [
       {
         "name": "name",
         "description": "The name of this person",
         "dataType": [
           "string"
         ]
       }
     ]
   },
   {
     "class": "Group",
     "description": "A set of persons who are associated with each other over some common properties",
     "properties": [
       {
         "name": "members",
         "description": "The persons that are part of this group",
         "dataType": [
           "Person"
         ]
       }
     ]
   },
 ]

and then we do client.query.get("Group"), what should be a returned value ? (If you see group has got just one property that is a reference property and no non-reference property)

ahh, that is a good question. Currently it is run without any properties and will fail, right?

ahh, that is a good question. Currently it is run without any properties and will fail, right?

Yes exactly

I think this is ok.
It failed before, too. So I think it still is an improvement over the previous state

@dirkkul Since the PR has added a new parameter schema to be passed to GetBuilder, we actually need to update the older calls made to the constructor of this class i.e. tests for GetBuilder need to be updated. One such occurence is here : Link

Hi, you're right about adding it in the unit-tests. I think just adding a mock there would be fine

I noticed one more thing:
we only want to return all properties if there are no additional properties.
query.get("Class", None) => returns all properties
query.get("Class", None).with_additional("id") => returns only _additional(id) and no other properties

I think the correct place for this would be to move the "empty check" to here https://github.com/weaviate/weaviate-python-client/blob/1f1f80984a43ca9de2bf58ff9fe3b4a5daa8fb94/weaviate/gql/get.py#L1055

@dirkkul, I have made all the corrections since the last review. You can please proceed with adding comments about any further improvements that might be needed.

closing this as the tests are still failing and there hasn't been any activity in almost a month


Backport 2.7: Fix fragility issues in all.sh

Fix some issues that don't break all.sh on a pristine checkout, but make it fragile in a tree where you've been doing development.

In keep-going mode, don't hard-fail on some auxiliary script or test.
Fix #1691 #1713: check-files.py complaining about files in .git or in third-party subtrees.
Fix make apidoc looking for files in unintended subdirectories.

Backport of #2031. Differences with the original:

“Fix make WINDOWS_BUILD=1 clean on non-Windows hosts.” is not applicable since it fixes an issue introduced by #930. It should be fixed in the backport of #930.
The ALSR issue does not apply since we don't use gdb in the 2.7 tests.

@Patater @sbutcher-arm I rewrote the commit that modifies check-files.py to exclude other directories, including a fix for #1691. The original history is in https://github.com/gilles-peskine-arm/mbedtls/tree/all.sh-fragility-2.7.6-1. Please re-review.

New run of all.sh at https://jenkins-internal.mbed.com/job/mbedtls-release/728/

retest

retest

Jenkins passes, so ready for merge!


documented get_bmi function with Roxygen2 syntax

We documented the get_bmi function and made a suggestion to incorporate an error message for zero value numbers.

Resolves #13

Resolves #13

Resolves #6