Bot deployed in Azure is not reflecting in Skype

Bot Info  https://eatonfirstbot.azurewebsites.net

Bot handle:
App ID: 0084e56e-8431-4a0a-9266-f71ad00524fa
SDK Platform:  .NET
SDK Version:  4.7
Active Channels: Skype (Have not published but using via., link)
Deployment Environment: Azure App Service

Issue Description
I am hosting my bot in the Azure. I am able to access my bot through the "test console" in the "Developer Console". However, when I try to a Skype channel, the bot has stopped responding.
Code Example
Reproduction Steps

Expected Behavior

Actual Results

whats your bot handle so I can take a look?

The Bot Handle is "joebot_8126"
On Wed, Sep 6, 2017 at 10:24 PM, Jason Sowers<EMAIL_ADDRESS>wrote:

whats your bot handle so I can take a look?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotBuilder/issues/3429#issuecomment-327546913,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdQ5aYyi3Xiq6ELKgqlmwRuGYvkF2oecks5sfs5bgaJpZM4POdQN
.

--
Regards
Tushar Parmar
+91<PHONE_NUMBER>

Is it still not working for you?  I was able to talk to your bot

The issue is with Skype. The response is intermittent. You can try with
following commands to test my bot

Get me all incidents ( this will display a list of incident numbers)
Click on the incident for publish results.
Get me incident INC1234

type "back" to before testing each point on Skype channel.
On Wed, Sep 6, 2017 at 11:20 PM, Jason Sowers<EMAIL_ADDRESS>wrote:

Is it still not working for you? I was able to talk to your bot
[image: image]
https://user-images.githubusercontent.com/6301037/30126568-274d5280-92f1-11e7-9c74-54d7117ec35a.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotBuilder/issues/3429#issuecomment-327562181,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdQ5aXgVw6uQ7XD76hPU0GwRagmAfcZXks5sfttvgaJpZM4POdQN
.

--
Regards
Tushar Parmar
+91<PHONE_NUMBER>

Yes, that is correct. However,  I an unable to understand why the results
are intermittent on Skype. You can try  adding this BOT to your skype and
test. And you will realize, that the results are intermittent.
https://join.skype.com/bot/0084e56e-8431-4a0a-9266-f71ad00524fa
On Wed, Sep 6, 2017 at 11:27 PM, Jason Sowers<EMAIL_ADDRESS>wrote:

Looks like someone is building a serviceNow bot 👍
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotBuilder/issues/3429#issuecomment-327564179,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdQ5aXODeiUwNb5cKZfVyfzl33nK0Jqdks5sft0YgaJpZM4POdQN
.

--
Regards
Tushar Parmar
+91<PHONE_NUMBER>

Still works fine for me

@tgparmar75, I've just tested your bot and was able to successfully add it in Skype and then start a conversation with it.

What version of skype are you using?

You would have to converse more with it to know the real issue.  Please
type "Help" and follow various dialogs with the bot to experience
intermittent results.
I am using Skype on Android and web.skype.com on my laptop browser.
On 6 Sep 2017 23:20, "Jason Sowers"<EMAIL_ADDRESS>wrote:
Is it still not working for you? I was able to talk to your bot
[image: image]
https://user-images.githubusercontent.com/6301037/30126568-274d5280-92f1-11e7-9c74-54d7117ec35a.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotBuilder/issues/3429#issuecomment-327562181,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdQ5aXgVw6uQ7XD76hPU0GwRagmAfcZXks5sfttvgaJpZM4POdQN
.

Yes. The issue is when you converse more with rheumatoid bot on Skype.
Please type "help" to initiate various dialogs.
On 6 Sep 2017 23:52, "Steven Gum"<EMAIL_ADDRESS>wrote:

@tgparmar75 https://github.com/tgparmar75, I've just tested your bot
and was able to successfully add it in Skype and then start a conversation
with it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotBuilder/issues/3429#issuecomment-327571157,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdQ5aRDuThK31Sl0jKrrAFlkdnNhcMksks5sfuLPgaJpZM4POdQN
.

@tgparmar75  I spent about 20-30 minutes conversing with your bot and was unable to reproduce.  I used the "help" command many times as well as others.  I would absolutely love to help you solve your problem, but there is not much I can do with the information I have and not being able to reproduce.  Are you getting any error message?  I was using the desktop app, can you try to reproduce with the desktop app and give me the exact conversation so I can try once more to reproduce?

Hello
This morning I started conversing with the bot and all seems to be working
fine using the Skype channel. I have spend round 15 minutes with thmy bot
and is responding promptly.
Are there any tips and tricks that I must follow once I redeploy my code
into Azure to endure that the bot reflect my new changes?
On Thu, 7 Sep 2017 at 3:18 AM, Jason Sowers<EMAIL_ADDRESS>wrote:

@tgparmar75 https://github.com/tgparmar75 I spent about 20-30 minutes
conversing with your bot and was unable to reproduce. I used the "help"
command many times as well as others. I would absolutely love to help you
solve your problem, but there is not much I can do with the information I
have and not being able to reproduce. Are you getting any error message? I
was using the desktop app, can you try to reproduce with the desktop app
and give me the exact conversation so I can try once more to reproduce?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotBuilder/issues/3429#issuecomment-327610571,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdQ5ada8ooCP8r5G6Habx0oa24WxMgFYks5sfwcogaJpZM4POdQN
.

--
Regards
Tushar
+91<PHONE_NUMBER>

@tgparmar75 there is no special procedure for deploying your bot to Azure so there isn't really any tips or tricks to share.  It's just like any other web api application.   Going to close this issue for now, but can reopen if you are experiencing issues again

Thank you very much for your help and patience. I ran my bot all day on
Skype and it responded, very positively.
Thank you for your support.
On 7 Sep 2017 23:08, "Jason Sowers"<EMAIL_ADDRESS>wrote:

@tgparmar75 https://github.com/tgparmar75 there is no special procedure
for deploying your bot to Azure so there isn't really any tips or tricks to
share. It's just like any other web api application. Going to close this
issue for now, but can reopen if you are experiencing issues again
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotBuilder/issues/3429#issuecomment-327871299,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdQ5afTLIVKqSGTSCV6Sj0Gpycej7fmXks5sgCohgaJpZM4POdQN
.

My pleasure, Good luck


Implement User Attributes Reconciliation in Keycloak-Operator

The keycloak-operator needs enhancement to support the reconciliation of user attributes. This feature aims to ensure that user data within Keycloak remains consistent and up-to-date, specifically focusing on the following attributes:
- FirstName
- SecondName
- Email
- Email-Verified (optional)
Acceptance Criteria:

Attribute Reconciliation: The keycloak-operator must be able to reconcile user attributes including FirstName, SecondName, and Email.
Optional Email-Verified Field: The operator should support the reconciliation of the Email-Verified field, but this should be optional and configurable.
Consistency Check: Ensure that after reconciliation, the user attributes in Keycloak match the source of truth.
Automated Tests: Implement automated tests that verify the correct reconciliation of user attributes.
Documentation: Update the keycloak-operator documentation to include information on how to use and configure the new feature.

Done.


Create Stanislas Paris

https://www.stanislas.fr/
https://www.stanislas.fr/presentation-des-filieres-de-la-prepa-de-stanislas

@Timo064 lib/domains/fr/Stanislas: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@Timo064 What is your school email address pattern? Is it<EMAIL_ADDRESS>

Hello!
My school email address pattern is<EMAIL_ADDRESS>

@Timo064 lib/domains/fr/Stanislas: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@Timo064 Thank you for the clarification. I corrected the request and merged it. Thank you. You can start requesting free licenses in an hour.


About TTS resume

HI, I found that resume code of TTS is in
https://github.com/open-mmlab/Amphion/blob/main/models/tts/base/tts_trainer.py#L140
and
https://github.com/open-mmlab/Amphion/blob/main/models/tts/base/tts_trainer.py#L302
however, _accelerator_prepare is in
https://github.com/open-mmlab/Amphion/blob/main/models/tts/base/tts_trainer.py#L145
So when resume_type=="resume",  self. _check_resume function seems not to work.
Is there something which I missed?

For another issue, I am confused with the phon_id_collator.get_phone_id_sequence
when i run the infer process with
sh egs/tts/VITS/run.sh --stage 3 --gpu "0" \     --config ckpts/tts/vits_ljspeech/args.json \     --infer_expt_dir ckpts/tts/vits_ljspeech/ \     --infer_output_dir ckpts/tts/vits_ljspeech/result \     --infer_mode "single" \     --infer_text "This is a clip of generated speech with the given text from a TTS model."

HI, I found that resume code of TTS is in https://github.com/open-mmlab/Amphion/blob/main/models/tts/base/tts_trainer.py#L140 and https://github.com/open-mmlab/Amphion/blob/main/models/tts/base/tts_trainer.py#L302
however, _accelerator_prepare is in https://github.com/open-mmlab/Amphion/blob/main/models/tts/base/tts_trainer.py#L145
So when resume_type=="resume", self. _check_resume function seems not to work.
Is there something which I missed?

Thanks for your feedback. Please check this PR #108 .

HI, I found that resume code of TTS is in https://github.com/open-mmlab/Amphion/blob/main/models/tts/base/tts_trainer.py#L140 and https://github.com/open-mmlab/Amphion/blob/main/models/tts/base/tts_trainer.py#L302
however, _accelerator_prepare is in https://github.com/open-mmlab/Amphion/blob/main/models/tts/base/tts_trainer.py#L145
So when resume_type=="resume", self. _check_resume function seems not to work.
Is there something which I missed?

Thanks for your feedback. Please check this PR #108 .

Thank you!
Besides, can you check the second issue I mentioned? I tried to add
phones  =" ".join(phone_seq) phones = "{"+phones"}" phone_seq=phone.split(" ")
after this line:
https://github.com/open-mmlab/Amphion/blob/a840088a9cc1d5c3afa3ed2f6c39db35c32d1f65/models/tts/vits/vits_dataset.py#L80
and retrain a new vits model.
also I change the same code of inference. However, the retrained model fail to synthesis human-understandable English.
It's so weird that I have debugged for several days and stil can't find out the reason now.

For another issue, I am confused with the phon_id_collator.get_phone_id_sequence when i run the infer process with sh egs/tts/VITS/run.sh --stage 3 --gpu "0" \ --config ckpts/tts/vits_ljspeech/args.json \ --infer_expt_dir ckpts/tts/vits_ljspeech/ \ --infer_output_dir ckpts/tts/vits_ljspeech/result \ --infer_mode "single" \ --infer_text "This is a clip of generated speech with the given text from a TTS model."

When cfg.preprocess.phone_extractor == "lexicon", we convert text to phone sequence based on the dictionary defined in https://raw.githubusercontent.com/open-mmlab/Amphion/main/text/lexicon/librispeech-lexicon.txt.
For the conversion from phone sequence to phone ID sequence, we currently uses the phoneme set from the https://github.com/HarryHe11/vc-dev/blob/main/text/symbols.py. However, it should use the phoneme set from the librispeech-lexicon.txt. I'll refactor this part. Thanks for your feedback.

Hi @arieszhang1994 , If you have any further questions about the TTS resume, feel free to re-open this issue. We are glad to follow up!


feat(tutorial): complete step 5 Hursley

https://carbon-tutorial-ptaunton.eu-gb.mybluemix.net

This PR went very wrong, please ignore


[iOS] - No visible @interface for 'AIRMap'

Is this a bug report?
I don't know, probably yes.

Have you read the Installation Instructions?
Yes.
Environment

"react": "^16.3.1",
"react-native": "^0.55.4",
"react-native-maps"<EMAIL_ADDRESS>
Steps to Reproduce

I just updated node_modules and the problem started. My last build was successfully before.
Expected Behavior

Compile.
Actual Behavior

No visible @interface for 'AIRMap' declares the selector 'showAnnotations:edgePadding:animated:'

I tried some things that people said here #2396 but nothing seems to work.
Android is fine.
iOS is not compiling. I'm not using Google Maps.

Same here I started receiving this issue two days ago, updated node modules and problem started.
I even tried replacing "github:react-community/react-native-maps" with "^0.21.0"
still the same issue

I changed for 0.21.0 and the issue was solved.
@gitpushdev try removing node_modules, npm install, remove ios/Pods then pod install. Worked here.
But we probably have a bug on last git version here.
xcode 9.4.1 here.

@vendramini thank you, I will try your steps

I don't use cocoapods, but i also have this issue with git version

I'm experiencing the same problem.  This just started a few days ago.  It was working perfectly fine.  I've tried removing node, reinstalling, removing pods and reinstalling.
Any other solutions?

@marcelkalveram Could You please check, iOS project can't be built using latest master branch and this problem is possibly introduced in this commit - https://github.com/react-community/react-native-maps/commit/c928c717afe5bc17046cfc04512d9843572ee081

@todorone thanks for the heads-up. There's indeed an issue with the function I modified.
Here's a patch
patch-package
--- a/node_modules/react-native-maps/lib/ios/AirGoogleMaps/AIRGoogleMapManager.m
+++ b/node_modules/react-native-maps/lib/ios/AirGoogleMaps/AIRGoogleMapManager.m
@@ -204,7 +204,7 @@ RCT_EXPORT_METHOD(fitToSuppliedMarkers:(nonnull NSNumber *)reactTag
       CGFloat bottom = [RCTConvert CGFloat:edgePadding[@"bottom"]];
       CGFloat left = [RCTConvert CGFloat:edgePadding[@"left"]];

-      [mapView animateWithCameraUpdate:[GMSCameraUpdate fitBounds:bounds withPadding:55.0f withEdgeInsets:UIEdgeInsetsMake(top, left, bottom, right)]];
+      [mapView animateWithCameraUpdate:[GMSCameraUpdate fitBounds:bounds withEdgeInsets:UIEdgeInsetsMake(top, left, bottom, right)]];
     }
   }];
 }
--- a/node_modules/react-native-maps/lib/ios/AirMaps/AIRMapManager.m
+++ b/node_modules/react-native-maps/lib/ios/AirMaps/AIRMapManager.m
@@ -245,12 +245,7 @@ RCT_EXPORT_METHOD(fitToSuppliedMarkers:(nonnull NSNumber *)reactTag

             NSArray *filteredMarkers = [mapView.annotations filteredArrayUsingPredicate:filterMarkers];

-            CGFloat top = [RCTConvert CGFloat:edgePadding[@"top"]];
-            CGFloat right = [RCTConvert CGFloat:edgePadding[@"right"]];
-            CGFloat bottom = [RCTConvert CGFloat:edgePadding[@"bottom"]];
-            CGFloat left = [RCTConvert CGFloat:edgePadding[@"left"]];
-
-            [mapView showAnnotations:filteredMarkers edgePadding:UIEdgeInsetsMake(top, left, bottom, right) animated:animated];
+            [mapView showAnnotations:filteredMarkers animated:animated];

         }
     }];

I'll now submit a PR based on this fix: https://github.com/marcelkalveram/react-native-maps/commit/cd285cfc43b13b73c6ec395004de0d5f86cdb3f6.
You can use this instead of the original repo, but npm also seems to work as it didn't include the bug yet.

@marcelkalveram Amazing, thanks a lot for the quick fix!

No problem. Again, apologies. This should never have gone into master.
It looks like I didn't properly rebuild when testing this, but instead was just testing the bundle changes.
Maybe failing builds should be caught by CI? @rborn @alvelig

Thank you for the quick fix! Awesome =)


added exemptOnConsensusThrottle property

Signed-off-by: Tanyu Dermendzhiev<EMAIL_ADDRESS>

Will be merged as part of https://github.com/hashgraph/hedera-services/pull/2494/files


No IDN encoder found (install the intl extension or the true/punycode package

Q
A

Bug report?
yes

Feature request?
no

RFC?
no

How used?
laravel

Swiftmailer version
6.1.0

PHP version
7.0.30

When upgrading swiftmailer version from 6.0.2 to 6.1.0 , I get the following error when trying to send an email:
exception: "Swift_SwiftException"
file:"vendor/swiftmailer/swiftmailer/lib/classes/Swift/AddressEncoder/IdnAddressEncoder.php"
line:64
message: "No IDN encoder found (install the intl extension or the true/punycode package"
This is the first time it happened after using this package for a long time.
I think the correct way to solve this is to make the php extension ext-intl or the package true/punycode a requirement, not only a suggestion, in composer.json
(When installing the true/punycode package the issue was solved)

/cc @c960657

Which email address are you trying to send to? Can you post the full stack trace?
The exception thrown by Swift_AddressEncoder_IdnAddressEncoder is a subclass of Swift_RfcComplianceException. Swift_RfcComplianceException is also thrown if you are trying to send to a malformed email address, so you probably want to catch it in any case, unless you have validated the addresses in advance.

All validations are done before sending, and I have been sending to the same email address I sent before with success.
I know this is meant to solve a problem with utf-8 characters in the email address, but I have none in my addresses.
Here is the stack trace:
{
    "message": "No IDN encoder found (install the intl extension or the true/punycode package",
    "exception": "Swift_SwiftException",
    "file": "vendor/swiftmailer/swiftmailer/lib/classes/Swift/AddressEncoder/IdnAddressEncoder.php",
    "line": 64,
    "trace": [
        {
            "file": "vendor/swiftmailer/swiftmailer/lib/classes/Swift/AddressEncoder/IdnAddressEncoder.php",
            "line": 43,
            "function": "idnToAscii",
            "class": "Swift_AddressEncoder_IdnAddressEncoder",
            "type": "->"
        },
        {
            "file": "vendor/swiftmailer/swiftmailer/lib/classes/Swift/Mime/Headers/IdentificationHeader.php",
            "line": 164,
            "function": "encodeString",
            "class": "Swift_AddressEncoder_IdnAddressEncoder",
            "type": "->"
        },
        {
            "file": "vendor/swiftmailer/swiftmailer/lib/classes/Swift/Mime/Headers/AbstractHeader.php",
            "line": 426,
            "function": "getFieldBody",
            "class": "Swift_Mime_Headers_IdentificationHeader",
            "type": "->"
        },
        {
            "file": "vendor/swiftmailer/swiftmailer/lib/classes/Swift/Mime/Headers/AbstractHeader.php",
            "line": 169,
            "function": "toTokens",
            "class": "Swift_Mime_Headers_AbstractHeader",
            "type": "->"
        },
        {
            "file": "vendor/swiftmailer/swiftmailer/lib/classes/Swift/Mime/SimpleHeaderSet.php",
            "line": 310,
            "function": "toString",
            "class": "Swift_Mime_Headers_AbstractHeader",
            "type": "->"
        },
        {
            "file": "vendor/swiftmailer/swiftmailer/lib/classes/Swift/Mime/SimpleMimeEntity.php",
            "line": 466,
            "function": "toString",
            "class": "Swift_Mime_SimpleHeaderSet",
            "type": "->"
        },
        {
            "file": "vendor/swiftmailer/swiftmailer/lib/classes/Swift/Mime/SimpleMessage.php",
            "line": 576,
            "function": "toString",
            "class": "Swift_Mime_SimpleMimeEntity",
            "type": "->"
        },
        {
            "file": "vendor/swiftmailer/swiftmailer/lib/classes/Swift/Message.php",
            "line": 142,
            "function": "toString",
            "class": "Swift_Mime_SimpleMessage",
            "type": "->"
        },
        {
            "file": "vendor/laravel/framework/src/Illuminate/Mail/Transport/MandrillTransport.php",
            "line": 48,
            "function": "toString",
            "class": "Swift_Message",
            "type": "->"
        },
        {
            "file": "vendor/swiftmailer/swiftmailer/lib/classes/Swift/Mailer.php",
            "line": 70,
            "function": "send",
            "class": "Illuminate\\Mail\\Transport\\MandrillTransport",
            "type": "->"
        },
        {
            "file": "vendor/laravel/framework/src/Illuminate/Mail/Mailer.php",
            "line": 451,
            "function": "send",
            "class": "Swift_Mailer",
            "type": "->"
        },
        {
            "file": "vendor/laravel/framework/src/Illuminate/Mail/Mailer.php",
            "line": 235,
            "function": "sendSwiftMessage",
            "class": "Illuminate\\Mail\\Mailer",
            "type": "->"
        },
        {
            "file": "vendor/laravel/framework/src/Illuminate/Notifications/Channels/MailChannel.php",
            "line": 64,
            "function": "send",
            "class": "Illuminate\\Mail\\Mailer",
            "type": "->"
        },
        {
            //the rest is app-specific
        }
    ]
}

Sorry, my bad. We do infact require an encoder to be present even for emails that do not contain UTF-8 characters. I'll make a fix later today.

I have the same issue. also laravel install.
After running composer update our testsuite fails.
Since the new version this error comes up:
 No IDN encoder found (install the intl extension or the true/punycode package at /vendor/swiftmailer/swiftmailer/lib/classes/Swift/AddressEncoder/IdnAddressEncoder.php:64)
We went from swiftmailer/swiftmailer (v6.0.2 => v6.1.0):

Same here after updating from 6.0.2 to 6.1.0. Thanks @c960657 for fixing that ;-)

Thanks @c960657
Temporal fix (in Laravel, for example) :wrench:
{
    "require": {
        "laravel/framework": "5.6.*",
        "swiftmailer/swiftmailer": "6.0.*",
         "...": "..."
     },
     "...": "..."
}

Hi all,
I am having some trouble with the Swiftmailer package but I am unsure if it related to a known issue or something. Also, I am a complete beginner with this, so I may not be able to provide you the information you need if you to support, if you request using jargon :).
I downloaded the Windows Composer installer (from getcomposer.org) and I successfully installed in on my computer. I then ran the command (cd (to my project directory) composer require swiftmailer/swiftmailer @stable) to download swiftmailer, and after it finished, there were some things that are listed as missing - please see below:
swymfony/polyfill-intl-idn suggests installing ext-intl (For best performance)
egulias/email-validator suggests installing ext-intl (PHP Internationalization Libraries are required to use the SpoofChecking validation)
swiftmailer/swiftmailer suggests installing ext-intl (Needed to support internationalized email addresses)
swiftmailer/swiftmailer suggests installing true/punycode (Needed to support internationalized email addresses, if ext-intl is not installed)
I tried to run a test to check if the Swift class was available using an if statement, and I always get the response as true. However, when I try to instantiate a method to compose my email - such as Swift_Message::newInstance(), I get an error saying: Call to undefined method Swift_Message::newInstance().
In case this might be useful, please note that this has happened to me twice with 2 different devices: Dell Latitue E5470 running Windows 7 Enterprise & Surface Pro 4 running Windows 10
Can you please help me resolve this issue? Is there that I am doing wrong? Is there support somewhere that can show me how to correctly and efficiently use Swiftmailer? I need to learn how to create HTML5 emails, with attachments and be able to send those to more than one recipient.
Please let me know if there is any other information that I can provide that could help to resolve this issue.
Thank you in advance.


feat: update dotnet sdk package to 4.12.0-rc1, js sdk.schema to 4.12.0-rc3

Description

Update dotnet sdk package to 4.12.0-rc1.
Update JS runtime sdk.schema based on js 4.12.0-rc3

Task Item
#minor

Screenshots

Coverage remained the same at 53.906% when pulling da1bef0c7504ece3553504cba34c7f2192ff8bf8 on runtime/4.12rc1 into fdbc4181ee5cd64b786494a9c65ff56bcc007778 on main.

@cwhitten SendHandoffActivity was in the uischema of dotnet but not in JS due to it's not implemented yet.
For the other two $kinds, I was not aware of them, I need to create a PR towards dotnet SDK repo

How about we take this first, while at the same time, letting Ze follow up on dotnet repo with new ui schema @cwhitten  @srinaath ?


Modlist V1.4.2 - Unable to download several files (not listed in readme)

Hi and a good new year Sovn,
I have several files unable to download. Tried it 3 times today.
Added the last lines of the wbbajack log file.
So I'm nut sure what the reason is. All other files were downloaded correctly.
Not using the Anniversary version of Skyrim.
Would be glad for a little help for this fantastic modlist.
2323.74 - Unable to download Data_ccBGSSSE025-AdvDSGS.bsa (GameFileSourceDownloader+State|SkyrimSpecialEdition|1.6.342.0|data\ccbgssse025-advdsgs.bsa)
2323.74 - Unable to download Data_ccBGSSSE025-AdvDSGS.esm (GameFileSourceDownloader+State|SkyrimSpecialEdition|1.6.342.0|data\ccbgssse025-advdsgs.esm)
2323.74 - Unable to download Aldrnari - Lods Output.7z (WabbajackCDNDownloader+State|https://authored-files.wabbajack.org/Aldrnari - Lods Output.7z_55bcadb1-7f52-4950-9c87-7e8f9faacb17)
2323.74 - Unable to download Data_Skyrim - Animations.bsa (GameFileSourceDownloader+State|SkyrimSpecialEdition|1.6.342.0|data\skyrim - animations.bsa)
2323.74 - Unable to download Data_Skyrim - Interface.bsa (GameFileSourceDownloader+State|SkyrimSpecialEdition|1.6.342.0|data\skyrim - interface.bsa)
2323.74 - Unable to download Data_Skyrim - Meshes0.bsa (GameFileSourceDownloader+State|SkyrimSpecialEdition|1.6.342.0|data\skyrim - meshes0.bsa)
2323.74 - Unable to download Data_Skyrim - Meshes1.bsa (GameFileSourceDownloader+State|SkyrimSpecialEdition|1.6.342.0|data\skyrim - meshes1.bsa)
2323.74 - Unable to download Data_Skyrim - Misc.bsa (GameFileSourceDownloader+State|SkyrimSpecialEdition|1.6.342.0|data\skyrim - misc.bsa)
2323.74 - Unable to download Data_Skyrim - Shaders.bsa (GameFileSourceDownloader+State|SkyrimSpecialEdition|1.6.342.0|data\skyrim - shaders.bsa)
2323.74 - Unable to download Data_Skyrim - Sounds.bsa (GameFileSourceDownloader+State|SkyrimSpecialEdition|1.6.342.0|data\skyrim - sounds.bsa)
2323.74 - Unable to download Data_Skyrim - Textures1.bsa (GameFileSourceDownloader+State|SkyrimSpecialEdition|1.6.342.0|data\skyrim - textures1.bsa)
2323.74 - Unable to download Data_Skyrim - Textures2.bsa (GameFileSourceDownloader+State|SkyrimSpecialEdition|1.6.342.0|data\skyrim - textures2.bsa)
2323.74 - Unable to download Data_Skyrim - Textures4.bsa (GameFileSourceDownloader+State|SkyrimSpecialEdition|1.6.342.0|data\skyrim - textures4.bsa)
2323.74 - Unable to download Data_Skyrim - Textures5.bsa (GameFileSourceDownloader+State|SkyrimSpecialEdition|1.6.342.0|data\skyrim - textures5.bsa)
2323.74 - Unable to download Data_Skyrim - Textures6.bsa (GameFileSourceDownloader+State|SkyrimSpecialEdition|1.6.342.0|data\skyrim - textures6.bsa)
2323.74 - Unable to download Data_Skyrim - Textures7.bsa (GameFileSourceDownloader+State|SkyrimSpecialEdition|1.6.342.0|data\skyrim - textures7.bsa)
2323.74 - Unable to download Data_Skyrim - Textures8.bsa (GameFileSourceDownloader+State|SkyrimSpecialEdition|1.6.342.0|data\skyrim - textures8.bsa)
2323.74 - Cannot continue, was unable to download one or more archives
2323.74 - File hash check (-42) b0d5dde21b778a964e8528c1a32014ebf3e3f3e83df667a708eeb474fe3ad52c
2323.74 - Encountered error, can't continue - System.Exception: Cannot continue, was unable to download one or more archives
at Wabbajack.Lib.AInstaller.Error(String msg) in C:\Wabbajack\splitwork\wabbajack\Wabbajack.Lib\AInstaller.cs:line 70
at Wabbajack.Lib.MO2Installer._Begin(CancellationToken cancel) in C:\Wabbajack\splitwork\wabbajack\Wabbajack.Lib\MO2Installer.cs:line 161
at Wabbajack.Lib.ABatchProcessor.b__66_0() in C:\Wabbajack\splitwork\wabbajack\Wabbajack.Lib\ABatchProcessor.cs:line 177
at Wabbajack.MO2InstallerVM.<>c__DisplayClass31_0.<b__0>d.MoveNext() in C:\Wabbajack\splitwork\wabbajack\Wabbajack\View Models\Installers\MO2InstallerVM.cs:line 181
--- End of stack trace from previous location ---
at Wabbajack.MO2InstallerVM.Install() in C:\Wabbajack\splitwork\wabbajack\Wabbajack\View Models\Installers\MO2InstallerVM.cs:line 175
at Wabbajack.InstallerVM.<>c__DisplayClass85_0.<<-ctor>b__75>d.MoveNext() in C:\Wabbajack\splitwork\wabbajack\Wabbajack\View Models\Installers\InstallerVM.cs:line 383
2324.15 - File hash check (-42) b0d5dde21b778a964e8528c1a32014ebf3e3f3e83df667a708eeb474fe3ad52c

After updating to Anniversary Update und verifying local files of Skyrim installation via steam client, the installation is continuing now with the step "Extracting modlist".
Maybe you can update the readme for a notice that the Anniversary update is necessary.
Thanks.

If only you'd read the documentation properly.


Make nextcloud php package customizable

Many nextcloud extensions like https://apps.nextcloud.com/apps/facerecognition need additional php extensions that are not part of the extension enabled for nextcloud.
It would be great to either:

Provide an option to override the php package nextcloud uses, i.e. services.nextcloud.php.package

Or:

Provide an option extraPHPExtensions, and make the current extension selection its default value

I'm opening this as an issue to get some feedback on what the better option is, and to get the naming right.
CC maintainers @schneefux @bachp @globin @fpletz @ma27

Meta: I'm not sure all labels I've applied make sense here. Feel free to remove inappropriate ones.
Also: There is no PHP label?

cc @NixOS/php

cc @NixOS/php


Feature/MN-273/native-demo-app

Technical description

[x] Create a new package named @monkvision/eslint-config-typescript-react-native for ESLint config that will be the one used everywhere in React Native project in MonkJs
[x] Create a blank NATIVE demo app to test the refacto components
[ ] PENDING - Run and test app in Android emulator

Tested on
Screenshots (optional)

Fixed all ESLint issues from the code...


Feat: Add focus modal

What

Add focus modal

How

Add absolute element on top of everything with modal-like properties

Testing

Add storybook stories Molecules/Focus-modal:

regular
overflow

@fPolic do you wanna take another read through this before we merge?

@fPolic do you wanna take another read through this before we merge?

@srindom LGTM!


Switch to rancher/shell based audit loggin

Issue:

Problem

Solution

Testing

Engineering Testing
Manual Testing

Automated Testing

Test types added/modified:

Unit
Integration (Go Framework)
Integration (v2prov Framework)
Validation (Go Framework)
Other - Explain: EXPLAIN
None
REMOVE NOT APPLICABLE BULLET POINTS ABOVE

If "None" - Reason: EXPLAIN THE REASON

If "None" - GH Issue/PR: LINK TO GH ISSUE/PR TO ADD TESTS

Summary: TODO
QA Testing Considerations

Regressions Considerations

TODO
Existing / newly added automated tests that provide evidence there are no regressions:

TODO

We'll hold off on an idea like this for down the line.


JDK 21 incompatible with Pinot

Hi. I have:

linux x86_64 machine
OpenJDK 21
Apache Pinot 1.0.0 binary

I tried to run controller with command:
bin/pinot-admin.sh StartController -zkAddress <IP_ADDRESS>:2181 -controllerHost <IP_ADDRESS> -controllerPort 9777 -dataDir /usr/local/pinot/data/controller -clusterName cluster-prod

And I saw next error:
/usr/local/pinot/bin/pinot-admin.sh: line 203: [: : integer expression expected

My java -version output:
$ java -version
openjdk version "21" 2023-09-19
OpenJDK Runtime Environment (build 21+35-2513)
OpenJDK 64-Bit Server VM (build 21+35-2513, mixed mode, sharing)

CC @gortiz  :-)

Hi Evgeny, thanks for you interest in Apache Pinot.
First of all, I have to say that Apache Pinot 1.0.0 should work in Java 21, but it is still not officially supported. Next release (1.1.0) will bring official support. Also, as a recommendation, it seems you are using OpenJDK 21, which is not recommended for production usage. Instead it is recommended (not by the Pinot team, but the Java community in general) to use other distributions like Eclipse Temurin, Microsoft JDK, Oracle JDK, Azul OpenJDK, Amazon Corretto, etc.
Anyway, as said, although we don't officially support Java 21, Apache Pinot should run in there. I've downloaded the binary from the web and executed in the same way you did in Linux and didn't have that issue. By the way, have you tried to execute Pinot with other JDK (specially Java 11)? did it work?
Given your bash reports an error in line 203 of pinot-admin, which is:
if [ "$(jdk_version)" -gt 11 ]; then

I would say the error is in jdk_version bash function, which tries to detect your jvm version in order to modify JAVA_OPTS accordingly.
That script is defined as
jdk_version() {
  IFS='
'
  # remove \r for Cygwin
  lines=$(java -Xms32M -Xmx32M -version 2>&1 | tr '\r' '\n')
  for line in $lines; do
    if test -z $result && echo "$line" | grep -q 'version "'
    then
      ver=$(echo $line | sed -e 's/.*version "\(.*\)"\(.*\)/\1/; 1q')
      # on macOS, sed doesn't support '?'
      if case $ver in "1."*) true;; *) false;; esac;
      then
        result=$(echo $ver | sed -e 's/1\.\([0-9]*\)\(.*\)/\1/; 1q')
      else
        result=$(echo $ver | sed -e 's/\([0-9]*\)\(.*\)/\1/; 1q')
      fi
    fi
  done
  unset IFS
  echo "$result"
}

Given you included your java --version in the report, I've modified the script to use your java --version and it correctly returned 21 for me. So I'm not sure where is the issue.
Can you modify your copy of bin/pinot-admin.sh to add set -x to the first line of jdk_version function? It should be something like:
jdk_version() {
  set -x
  IFS='
'
  # remove \r for Cygwin
  lines=$(java -Xms32M -Xmx32M -version 2>&1 | tr '\r' '\n')
  for line in $lines; do
    if test -z $result && echo "$line" | grep -q 'version "'
    then
      ver=$(echo $line | sed -e 's/.*version "\(.*\)"\(.*\)/\1/; 1q')
      # on macOS, sed doesn't support '?'
      if case $ver in "1."*) true;; *) false;; esac;
      then
        result=$(echo $ver | sed -e 's/1\.\([0-9]*\)\(.*\)/\1/; 1q')
      else
        result=$(echo $ver | sed -e 's/\([0-9]*\)\(.*\)/\1/; 1q')
      fi
    fi
  done
  unset IFS
  echo "$result"
}

Once you execute with that script you should see something like:
> bin/pinot-admin.sh StartController -zkAddress <IP_ADDRESS>:2181 -controllerHost <IP_ADDRESS> -controllerPort 9777 -dataDir /usr/local/pinot/data/controller -clusterName cluster-prod

++ IFS='
'
+++ java -Xms32M -Xmx32M -version
+++ tr '\r' '\n'
++ lines='openjdk version "21" 2023-09-19 LTS
OpenJDK Runtime Environment Temurin-21+35 (build 21+35-LTS)
OpenJDK 64-Bit Server VM Temurin-21+35 (build 21+35-LTS, mixed mode, sharing)'
++ for line in $lines
++ test -z
++ echo 'openjdk version "21" 2023-09-19 LTS'
++ grep -q 'version "'
+++ echo 'openjdk version "21" 2023-09-19 LTS'
+++ sed -e 's/.*version "\(.*\)"\(.*\)/\1/; 1q'
++ ver=21
++ case $ver in
++ false
+++ echo 21
+++ sed -e 's/\([0-9]*\)\(.*\)/\1/; 1q'
++ result=21
++ for line in $lines
++ test -z 21
++ for line in $lines
++ test -z 21
++ unset IFS
++ echo 21
SLF4J: Class path contains multiple SLF4J bindings.
...

Which should give us more information about the issue.

@gortiz thank you for help. debug logs help me understand that "java" can not be found in PATH :)


aria-haspopup should have boolean value, has string like "dialog"

Apologies, ARIA 1.0 now supports enumerated values.


feat(remote): VocabularyRemoteSelect component

Some proptypes issues

When I try to add a new item and then, I try to save the draft (this is in context of contributor for example) when press to save the draft I get a 500 and
self.resolve_and_store_authority_record(
File "/home/dusanst/Projects/oarepo-vocabularies/oarepo_vocabularies/authorities/components.py", line 76, in resolve_and_store_authority_record
raise InvalidRelationError(
oarepo_runtime.services.relations.errors.InvalidRelationError: 'External authority failed: '
Though it is possible I am maybe missing latest version of some dependency. Does it work for you?

Small issue with pagination on this pesky screen size

I have never seen this error before, though it does not seem to impact the functionality

Screencast from 2024-09-06 15-43-08.webm

I have never seen this error before, though it does not seem to impact the functionality
 Screencast from 2024-09-06 15-43-08.webm

This happens because search apps needs to get exchanged (unmount + mount) when switching from internal APIs to external (ROR) APIs, dunno how to exactly mitigate this yet

Works much better now. Is there some specific reason, why we display the ror:id to the user instead of some human readable thing please? For example, if someone opens a draft and goes to creators, he will see this

And it is not possible to know what those things are. Even if you open the modal, you still don't know, because you have not made any api calls to search i.e. the listing does not exist at this point.


feat: improve key creation/import options when creating identifiers

What issue is this PR fixing
At the moment, specifying key options when creating identifiers (via didManagerCreate) is fragile. Each provider has a slightly different format for options, and many omit the ability to provide meta for keys. This is a problem, as it means there is no nice way to create both an identifier and associated keys if you need to store metadata.
This PR proposes some non-breaking changes to each of the DID providers that unifies key option signature with a base type. It passes all key options from options.key to keyManagerCreate/keyManagerImport automatically.
If no options.key or options.keyRef is provided, each provider falls back to the currently support configuration signature, which has been deprecated.
What is being changed
createIdentifier methods on each DID provider now utilise a base options type which includes a keyRef option, allowing the user to pass in an existing key, or a key option specifying the details for the key to be created/imported.
export type ImportOrCreateKeyOptions<TKey extends TKeyType = TKeyType> = Omit<
  IKeyManagerCreateArgs & MinimalImportableKey,
  'kms' | 'type'
> & { type: TKey }

export type CreateIdentifierBaseOptions<T extends TKeyType = TKeyType> = {
  keyRef?: string;
  key?: ImportOrCreateKeyOptions<T>;
}

For did:peer, there is an additional agreementKey and agreementKeyRef option.
Providers

did-provider-ethr
did-provider-jwk
did-provider-key
did-provider-peer
did-provider-pkh
did-provider-web

This change does not affect did-provider-ion. The ION provider requires specific metadata in each of the keys, which requires some consideration on the best way to handle.
// Create with an existing key
agent.didManagerCreate({
  kms: 'local'.
  provider: 'did:peer',
  options: {
    keyRef: existingKey.kid
  }
})

// Create a new key with params
agent.didManagerCreate({
  kms: 'local'.
  provider: 'did:key',
  options: {
    key: {
      type: 'X25519',
      meta: { userId: 123 }
    }
  }
})

// Import a key
agent.didManagerCreate({
  kms: 'local'.
  provider: 'did:key',
  options: {
    key: {
      type: 'X25519',
      privateKeyHex: 'DEADBEEF',
      meta: { userId: 123 }
    }
  }
})

Quality
Check all that apply:

[X] I want these changes to be integrated
[X] I successfully ran pnpm i, pnpm build, pnpm test, pnpm test:browser locally.
[X] I allow my PR to be updated by the reviewers (to speed up the review process).
[X] I added unit tests.
[ ] I added integration tests.

I'd be in favour of keeping the option to do both. As currently implemented, it is non-breaking. I did consider a signature like key: { kid: string; }, but this also has ambiguity in whether you're looking up a key by KID or providing a KID for the key being created. keyRef has precedent in Veramo (e.g keyManagerSign({ keyRef: 'xyz' })), as opposed to a new signature

Agreed. Naming the internal property kid was a mistake as it's usually a public JWT header property confusing everyone.
We tried to migrate to keyRef at some point but not all the way through. I suppose a simple id property would have been better.. or alias would have worked as well.
Suggestions welcome!

Thanks for the review @mirceanis, should all be fixed

Codecov Report
Attention: Patch coverage is 82.81250% with 44 lines in your changes missing coverage. Please review.

Project coverage is 89.69%. Comparing base (2f0b06d) to head (46e4e72).
Report is 2 commits behind head on next.

Files with missing lines
Patch %
Lines

packages/utils/src/did-utils.ts
17.39%
19 Missing :warning:

...ackages/did-provider-peer/src/peer-did-provider.ts
86.95%
12 Missing :warning:

packages/did-provider-pkh/src/pkh-did-provider.ts
78.26%
5 Missing :warning:

...ackages/did-provider-ethr/src/ethr-did-provider.ts
90.00%
2 Missing :warning:

packages/did-provider-jwk/src/jwk-did-provider.ts
93.75%
2 Missing :warning:

packages/did-provider-key/src/key-did-provider.ts
94.28%
2 Missing :warning:

packages/did-provider-web/src/web-did-provider.ts
93.54%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             next    #1423      +/-   ##
==========================================
- Coverage   89.72%   89.69%   -0.03%
==========================================
  Files         177      177
  Lines       26818    26937     +119
  Branches     2220     2228       +8
==========================================
+ Hits        24062    24161      +99
- Misses       2756     2776      +20

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


style: rename params for clearity, update && added comments

Noteable Changes

Updated docs
Added some comments
Renamed params to make them clearer

Can we merge this in?


New iOS release zip file is 4 bytes (incomplete file)

Could you please try the latest one? Thanks.

Works now, thanks!

Thank you for your feedback :-)


IRC notifications are temporamental

This is likely an issue either with Helpmebot, or with the communication path between the tool and Helpmebot.
Marking this as "live-config" because it's specific to Labs, and "upstream" because it's likely to involve Helpmebot too.

I haven't seen this recently, so closing as stale.


BUG + Suggestion: researching body with telescope yields 0%

BUG: I researched a body with a telescope once, it was random, I didn't get to choose which body I wanted it to research, which was annoying. It added 10% onto Moho. I did it a 2nd time, and it added 0% onto nothing, or it possibly added 10% onto a body I hadn't discovered yet. But from what I can see, nothing was added to any bodies I currently know about. I am unsure how to send my game data so you'll have to help me with that.
Suggestion: the research of a body going up only 10% per every scan (90 days minimun) you do on it? I feel like that's a bit slow. I think it should go up 33% each time, basically only having to do it 3 times to complete the research. Each time, the planet would gain a significant amount more detail compared to the previous %.
Also, not sure if it is a bug or a feature, but please enable me to choose what body I want to research that I have unlocked, with the telescope.
I know I can start the research with the ground telescope, and it knocks it up by 10%, but it only seems to be a 1 time thing...

I think you hit the wrong mod.  This mod doesn't do any research at all, it's just an orbit calculation tool
You probably want ResearchBodies:
https://forum.kerbalspaceprogram.com/index.php?/topic/139473-18x-researchbodies-v1110-19th-oct-jun-2019/
https://github.com/JPLRepo/ResearchBodies

oh yeah, my bad haha, thanks


Web3Auth login flow failed

Login fails on iOS using firebase custom auth on version web3auth_flutter 3.0.1. Works on Android

we just released 3.1.0 with a bunch of fixes.
pls test and let us know

Hey @peerwaya
Please follow this working firebase example  as per the latest SDK.

Tried 3.1.0, it still does not work. Returns the error Web3Auth login flow failed

The issue is only happening on IOS. Android works fine.

I think this is specific to iOS and firebase verifier using JWT. I tried the example app and it worked well. Switching back to 2.0.1, everything works well.  I also tried logging the error on the swift side and was getting unknown error

I have fixed the issue here https://github.com/Web3Auth/web3auth-swift-sdk/pull/47


Implement exec builtin.

My only concern with this change is now the pre-fork and post-fork builtin api are a bit inconsistent.


👷 Build for Windows on publish

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 28.31%. Comparing base (27824a2) to head (acdb22a).

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #17   +/-   ##
=======================================
  Coverage   28.31%   28.31%
=======================================
  Files          22       22
  Lines         883      883
  Branches      883      883
=======================================
  Hits          250      250
  Misses        577      577
  Partials       56       56

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


user own profile page

[ ] check how we can implement it with using invenio-account-rest
[ ] user 'account' page mockup - if applicable
[ ] user account page implementation
[ ] ongoing loans list
[ ] past loans

to discuss about if we use /me of accounts rest


Switch from -Z to -w

The flag -Z (trailing space) is not available in all versions of diff, but -w (all space) is available.
When I run things on my macbook I hit:
e2e_test.go:107: diff: invalid option -- Z
diff: Try `diff --help' for more information.

Fixes: https://github.com/tektoncd/pipeline/issues/4177
/kind bug
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Docs included if any changes are user facing
[ ] Tests included if any functionality added or changed
[ ] Follows the commit message standard
[ ] Meets the Tekton contributor standards (including
functionality, content, code)
[ ] Release notes block below has been filled in or deleted (only if no user facing changes)

Release Notes
NONE

/assign @vdemeester

Not the one test I changed 🤣
/retest

/lgtm

/retest
Not the test I changed, so a flake.


Any chance we'll get audio modding features any time soon? #enhancement

Going to close this as a duplicate of #78 - support is planned at some point in the future, but I've been prioritising working on the tools that are bespoke to Alien: Isolation's engine for the time being, such as the script editor. The game uses Wwise for its sounds, so existing tools should work to import/export audio.
I've also made the sound exporter previously which correctly re-names all sounds for you: https://github.com/MattFiler/Alien-Isolation-Audio-Extractor


Add install instructions for MacPorts users

I maintain the kubectx port in MacPorts: https://www.macports.org/ports.php?by=name&substr=kubectx
This pull request adds instructions for MacPorts users to the kubectx README.md.

I signed the CLA.

Thanks a lot for maintaining the port!


Server on crash loop

Issue description:
Upon breaking and placing a controller, the server goes on a crash loop.
What you expected to happen:
The server not going on a crash loop, ideally not crashing at all.
Steps to reproduce:

Place a controller
Break a controller
Place the controller again
(As far as I can tell that's it, I don't know the specific setup)
...

Version (make sure you are on the latest version before reporting):

Minecraft: 1.12.2
Forge: 2836
Refined Storage: 1.6.14

Does this issue occur on a server? [yes/no]
Yes
If a (crash)log is relevant for this issue, link it here:
https://pastebin.com/jRtwN5PQ

same as #2219

@Darkere  Were you able to reproduce this?

I've broken and placed the controller multiple times. Those aren't the steps to reproduce, just the way it happened. I can't identify how it happens. I turned on delete tile entities on forge, and it doesn't crash when I do that, but it also doesn't delete the controller. When turning that feature off again, the server goes back into a crash loop. The only way it doesn't, is if I remove the controller from the player's setup. A setup that uses item conduits, rs conduits, and an interface. My operating theory is that it's an issue with the interface.

No not able to reproduce. Only thing I can think of is that at least part of the old TE is still there.

It's weird. The bug is quite serious and seems like it would happen a lot, yet there are only 2 reports about it.

Lots of pack authors turn on the delete erroring tile entities. And since the problematic block is easily identified server admins will just delete and replace the block.

RS issues are much worse than any other problem. They are a higher priority and can't be fixed by just deleting the issue. Server owners prioritize getting systems back up and hoping it doesn't break again. The crash report also indicates it's an issue in the mod's code, so it's well above people's heads (myself included) but it does seem like it's a misuse of a method.

It's obvious that this is an RS problem. The question is whether this bug is only reproduceable in certain conditions.

McJty on the issue:

This can happen if that tile entity implements an interface that has getWorld()
While TileEntity has a getWorld() too
Which gets obfuscated
However your getWorld() doesn't get obfuscated
And that causes problems outside of dev environment

@Darkere  I tested this in a devenv but couldn't reproduce. Could you test this on an actual obfuscated server for me please?

Not crashing. Any idea on what to specifically test?
Played around with conduits and replacing the controller a lot.
Looking through the reports doesn't seem like it's specific to servers either. #2219

Basically just starting up a regular, obfuscated, Forge server.
Then breaking/placing the controller.

@CritFlaw  Can you still repro this?

I wasn’t doing anything related to a Controller when I got in this state. It’s not specific to Controllers. I had just placed a bunch of Crafters (as reported in #2219).

This just happened on my server, I was adding a new crafting grid which didn't connect. By the time the server crashed the error was throughout the log.
I've narrowed down the pack until its refined storage and ender io, the error still happens even when its client only. It happens the moment the world starts to load. If I'm able to trim down the size of the world save I can send it over.
Running 1.12.2, forge <IP_ADDRESS>47, refined storage 1.6.15 and ender io 5.1.52.
Crash log: https://gist.github.com/Txuritan/a89ee98d1444e7de64ba3e28fcb79933

I took a look myself, first trying out the issue of conflicting getWorld implementations.
In INetworkNode I renamed the getWorld method to getNodeWorld (changing any references to it), which got past the first error but this lead to another error with getPos. I assumed it was the same problem as with getWorld, I renamed it to getNodePos (again changing any references to it).
Doing so allowed me to load my world that caused a consistent crash. I also tested it on my server and it fixed the issue there to. Not sure if this will fix the issue entirely or if its just buried it more.

The problem is that I can't rename it because it breaks compatibility with add-ons

I fiddled around with it some more, at first I thought it was due to the interface's abstract method overriding the abstract class's method, which is not the case. Turns out, getWorld and getPos is being obfuscated due to extending TileBase but in doing so removing the implementations required for INetworkNode. Foresty also had this issue.
The only ways I can think of solving this would be to rename the methods in INetworkNode (breaking compatibility) or make the obfuscator rename the method to the correct name (also breaking compatibility due to the obfuscated names).

Create new methods with non-breaking signatures, move method movies from old methods into new methods, call new methods in old methods, deprecate old methods with published date for removal. Not an instant fix but you can start calling new methods immediately and fix bug for you. Add-ons have path and time.

Should be fixed by #2490


Update test session content and document it's "error"

Codecov Report
Base: 89.82% // Head: 89.82% // No change to project coverage :thumbsup:

Coverage data is based on head (2f26518) compared to base (5d428be).
 Consider uploading reports for the commit 2deddc6 to get more accurate results

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #344   +/-   ##
========================================
  Coverage    89.82%   89.82%
========================================
  Files           46       46
  Lines         4385     4385
========================================
  Hits          3939     3939
  Misses         446      446

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Update nodeversion: fix node-gyp and build errors

Upgrades node-libcurl to v4 to resolve node-gyp install error
Workaround typescript build error TS18047: 'repository.owner' is possibly 'null'.

Feel free to squash or manually refactor :)

@mchelen , thanks for fixing. Would you be so kind to rebase/squash your commits into one or do you really need every single commit ("test,test")?

@thomasmerz Ok done, cleaned up a bit. You can also squash the commits when merging the PR if that helps.


Add Climb+ChargeJump as a lower blackroot access path

Suggested by AvengedRuler. You can grab the side of the wall and charge jump above the spikes

I realized that I have no idea how inline comments on your changes get surfaced, so I'm going to also mention here that we were also going to promote casual-core Bash Grenade to standard-core, and add a dbash Bash path. Do you mind adding these to this PR?


add getQueryString convenience method fixes #15

Add convenience method getQueryString(url) and assicoated tests

Argh, sorry i even mixed spaces and tabs. I blame sublime for this. Fixed

👍


Prettier, Linting, Core Options, Initial Context File (tsx)

This PR adds the following:

Prettier (resolves https://github.com/thetrevorharmon/gatsby-theme-shopify-core/issues/6)
Linting (resolves https://github.com/thetrevorharmon/gatsby-theme-shopify-core/issues/5)
useCoreOptions: this hook lets us get at the options passed into the plugin in the gatsby-config of the consumer
StoreContext.tsx: this is the initial typescript version of the context file. Right now it just sets up a shopify-buy client.

All of this work is happening in the theme folder, and nothing is inside the example folder. Maybe at some point we'll add linting there too, but for now we'll just focus on the theme.

After looking this again, I think that Linting and Prettier need to be on the workspace level, not on the individual project level.

This is meant to be reviewed after https://github.com/thetrevorharmon/gatsby-theme-shopify-core/pull/8


Bump collector to 0.90.1

I don't think so. It's a bugfix patch. Not another minor version of the collector


Nintendo Wii: Misspelled 'Nunchuck'

In the Expansions subsection of Next-gen controllers, the following paragraph misspelled Nunchuck as Nunchuk.
Nintendo shipped this system with another controller to be used on the opposite hand, the *Nunchuk*, this one comes with its own accelerometer, joystick and two buttons.

Thanks


[WIP][GWS] schedule enhancement

Coverage decreased (-0.1%) to 83.173% when pulling<PHONE_NUMBER>b1b547fffee97fe04258ab2b37486e on sunny4381:features/schedule-enhancement into 33da2a15cf2c5ee2b915bed38b2fcc4abe7cfd88 on shirasagi:dev/gws.

Coverage decreased (-0.1%) to 82.909% when pulling 684a329f6aee452e1dffcc4df18bd0874b4a0307 on sunny4381:features/schedule-enhancement into 9f92d04b42737d8edeffae07f675949328c02cb6 on shirasagi:dev/gws.

Coverage decreased (-0.09%) to 82.966% when pulling 29792a2bfa8fb551d824621a0ade8fe0ccfc7765 on sunny4381:features/schedule-enhancement into 9435d456013a21597ed7d41374d7c2ef34d281bb on shirasagi:dev/gws.


Suggestion: point out material.io docs have more API calls

Hi @stasson,
Since this is my first issue let me start by saying I believe this project is the one that holds most potential in bringing Material design to Vue. The adapter approach is a good one and doesn't try to reinvent the wheel. Thanks for doing this!
I've just discovered the hard way (as in I had to look at the source code 🙂 ) how to programatically change tabs on vue-mdc-adapter. I think this should be on the docs somewhere and in fact it's already documented on the material.io docs at https://material.io/components/web/catalog/tabs/#the-full-foundation-api-1.
So my suggestion is that for each component doc page a paragraph mentioning the Foundation object and a link to the material.io docs is added (EDIT: the links to material.io docs are already there, only the introductory paragraph is needed, IMHO). That would help newcomers understand the full potential of the adapter.

@dmarcelino
Ideally we should not need to do that as I should already have exposed and documented any foundation features in the adapter. Obviously I'way behind,  and your proposal is will help in the mean time ...
I'll try to add that for next release if I have a little bit of time.

Documenting foundation features on the vue-mdc-adapter is even better, good stuff!

outdated


Add cache-dependency-path

I noticed that on my  PR no caching is taking place.
🔧 Type of changes

[x] bugfix

✨ What's the context?
https://github.com/prebid/prebid-server-java/pull/3613/checks shows

Creating settings.xml with server-id: github
Writing to /home/runner/.m2/settings.xml
maven cache is not found

🧠 Rationale behind the change
Make PR tests faster

Seems to work now:


How do I save data using json_to_sheet to save formulas?

So I am creating a bunch of JSON objects I am saving to a sheet. One of the fields should actually be a forumla, specifically I am wanting to create a hyperlink (`=HYPERLINK("","")).
How does one do that exactly? My objects currently look like:
{
     "company": "google",
     "home": "=HYPERLINK('https://www.google.com','Google')",
}

You can pass cell objects to json_to_sheet directly:
var cell = { t:"s", v:"Google" }; // string cell with value "Google"
var data = [
  { company: "google", home: cell } // the "home" key is the cell object
]
var ws = XLSX.utils.json_to_sheet(data, { header: ["company", "home"] } ); // worksheet

There are two ways to specify the hyperlink:
A) actual cell hyperlink: see https://docs.sheetjs.com/#hyperlinks for an explanation
var cell = {
  t: "s",
  v: "Google",
  l: { Target: "https://www.google.com", Tooltip: "Not SheetJS :(" } // Target is the URL, optionally specify Tooltip
};

B) HYPERLINK formula
Mirroring what you intended to do:
var cell = { t:"s", v:"Google", f: "HYPERLINK('https://www.google.com','Google')" }; // cell with formula, no = prefix

Hello @SheetJSDev , I was trying to add this formula TEXT(SUM(DATEVALUE("01/21/2020"),TIMEVALUE("09:20:05"),settings!B1/24),"DD/MM/YYYY hh:mm:ss") but my xls file show a blank cell.
const row = { A: { t: 's',  f: 'TEXT(SUM(DATEVALUE("01/21/2020"),TIMEVALUE("09:20:05"),settings!B1/24),"DD/MM/YYYY hh:mm:ss")' } }

const ws = XLSX.utils.json_to_sheet([row], { header: ['A'] })
const ws2 = XLSX.utils.json_to_sheet(
    [{ A: 'timezone offset', B: -timezone }],
    { skipHeader: true, header: ['A', 'B'] }
  )

XLSX.utils.book_append_sheet(book, ws, SHEET_NAME) // book already exists
XLSX.utils.book_append_sheet(book, ws2, SETTINGS_SHEET_NAME)

XLS / XLSB do not support writing formulae at the moment.

Hi, @SheetJSDev is it planned to add support for writing formulae in XLSB, in the near future?


Deprecate Base#with_variant

Summary
Deprecates Base#with_variant. Variants are an often misunderstood feature that use the Action Pack variant information present in the request (i.e. phone, tablet, etc) to render a component differently depending on the requesting device. Normally one passes in a list of acceptable variants in an Action View template, eg:
render(MyComponent.new, variants: [:phone, :tablet])

The variant can be overridden by calling #with_variant on the component, eg:
class MyComponent < ViewComponent::Base
  def call
    render(OtherComponent.new.with_variant(:tablet))
  end
end

It's important to note that these are two distinct render functions (the second does not accept a :variants option). Since components don't have access to the request object, it doesn't really make sense for them to decide which variant to render (that's probably why :variants isn't available in the first place). The consensus on the team was that overriding the variant is something that should really only be done in a test environment and not supported otherwise.

We are using variants, right now, to define two different templates for the same component.
Noticing the Deprecation Warning we realised that we are abusing the variant concept.
The idea is simple: the same component renders something in a page, and something completely different in another page.
We keep the same component, and just define two variants. How do we achieve that otherwise?

Thanks for the input.
Complexity does not matter too much in my opinion: the point is that we need to pass an additional parameter mode, which basically decides the template to render. We then have to use a if...else somewhere to decide which template to render. All of this was already possible with with_variant which was very convenient and allowing a component to have multiple templates. That's not possible anymore, but needs to be implemented "manually". It would be nice to have an alternative 😄

@coorasse I think that's a good idea for a feature proposal then! https://github.com/github/view_component/pull/1096 has some overlap with what you're looking to do, perhaps you could check that out and provide commentary there?


feat: Feature/improve price chart

Removed the Echart component
Put full width for the Linechart
Connect the Linechart to the API

Works for me! I see no conflicts with the watchlist either. I'm just wondering where the left chart went?

@DannyVogel thanks for the review! If every thing is ok can you approve the PR?


"Translated from EN to PT-BR - Guide > Security Overview"

Translated Guide > Security Overview from EN to PT-BR to website Quarkus.io

After translating, I removed some fuzzy words on the Guide > Security Overview.


Adding CommandType to IBotCommand, editing CommandsCommand descriptio…

…n text and changing database #109

Twitch chat
Casrou: !commands
devchatterbot: These are the commands that Casrou is allowed to run: !RockPaperScissors, !Uptime, !Give, !Coins, !Bonus, !Streams, !ShoutOut, !Quote, !AddQuote, !Alias, !Help, !Commands, !AddCommand, !RemoveCommand, !discord, !github, !emotes, !lurk, *!Hangman. (*Only subscribers can start these games. However, all viewers in chat can join a game once it's started.)

The CommandType of !RockPaperScissors could later be changed to "Game", if we want to only let subscribers start a game.
A !Games command would also be possible to add in the future to list all games.

Going to close this one down, @casrou . Just make a new PR if you go that other route, and let me know in Discord if you need help setting it up.


Github To Lagrange Tool to auto sync repo  & swan chain on docker

why the PR not approved yat ?

thanks for your contribution, once it is verified, I will merge it


Improve the allowedStatusCode option override

Avoid having a big array with all the status code and see if we can do some filtering based on strings, eg: 5xx

I'm using Set like allowedStatusCodes.has(reply.statusCode) to improve performance.

:tada: This issue has been resolved in version 5.0.0-next.1 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Percentage chart not matching boardings

Describe the bug
The percentage chart on the agency report page does not match the boardings vs. violations.
Screenshot

hey, can I solve this bug?

Hi! This is a Charts limitation. As per the docs, embedding filters apply after the query. In our query, we $group the data in such a way that the ts field used in our embedding filter no longer exists. Hence, this filters out all of our data.
Charts will be adding functionality at a future time that lets us address this.
Until then, the workaround is to not use a chart at all, and calculate the information in the app itself.
The compliance rate is calculated as:
(# of boardings that have empty or nonexistent inspection.summary.violations values) / (# boardings) * 100

Hello, thanks for offering to work on this! I have assigned it to you.


how to parse twelve to ten?

Or twelve past a half
Ten o'clock?
Online try can not work

More generally though, it would be great to have support for wordy hours-minutes formats like: quarter past 5, half past 6, quarter to 4, twenty to 3, 20 to 3; etc.


Disable recording for certain windows (e.g. password managers)

It would be cool if I could configure clipmenu to not store text copied from windows with a certain title, to avoid storing passwords copied from a password manager.
This should be fairly straightforward to implement using xdotool getactivewindow getwindowname   to get the title of the currently active window and checking it against a regex before storing the clipboard.
CopyQ supports this:
https://copyq.readthedocs.io/en/latest/faq.html#how-to-omit-storing-text-copied-from-specific-windows-like-a-password-manager

I'd accept a patch doing this. :-)

I'll give it a go.

This is implemented in #89. Can you please have a look?


Zap to Char push the first character to copy buffer

Not sure if this was the desired function but I'm pretty sure in emacs zap-to-char will push the entire zapped region to the copy buffer. Now it just pushes the first character behind the cursor with the zap-to-char implementation. I guess ideally for me it would push the whole zapped region but I think it makes sense to have a default either don't push anything or push the whole thing to the copy buffer. Then maybe an option to do the other one.

this can be fixed by changing this one line in jove.py as it's missing the brackets []:
2225:  kill_ring.add([self.view.substr(region)], True, False)

Well spotted.
I will fix this soon. I am in the middle of redoing some improvements to i-search that I want to make sure are good.
JP

On 5 Apr 2016, at 13:56, excetara2<EMAIL_ADDRESS>wrote:
this can be fixed by changing this one line in jove.py as it's missing the brackets []:
2225: kill_ring.add([self.view.substr(region)], True, False)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub https://github.com/grundprinzip/sublemacspro/issues/131#issuecomment-205789698

Awesome no worries. I've fixed in my code base for now.
On Wed, Apr 6, 2016 at 1:45 AM, Jonathan Payne<EMAIL_ADDRESS>wrote:

Well spotted.
I will fix this soon. I am in the middle of redoing some improvements to
i-search that I want to make sure are good.
JP

On 5 Apr 2016, at 13:56, excetara2<EMAIL_ADDRESS>wrote:
this can be fixed by changing this one line in jove.py as it's missing
the brackets []:
2225: kill_ring.add([self.view.substr(region)], True, False)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub <
https://github.com/grundprinzip/sublemacspro/issues/131#issuecomment-205789698

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/grundprinzip/sublemacspro/issues/131#issuecomment-205864351

I've released your fix.

Ok thanks!
On Sat, Apr 16, 2016 at 9:48 PM Jonathan Payne<EMAIL_ADDRESS>wrote:

Closed #131 https://github.com/grundprinzip/sublemacspro/issues/131.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/grundprinzip/sublemacspro/issues/131#event-629885691

I have rewritten the zap-to-char function to my multi-cursor aware. It's actually implemented in terms of another new multi-cursor function called "jump-to-char".
Do you want to try them out before I release?
Also, does your zap to char eat the character zapped to OR stop just before that character? I want to know what to set for the default with the new one.
It works well: if you call zap to char multiple times in a row, with multiple cursors, they are all accumulated into the same cursor-aware kill-ring entry.

About to have a lot of travel time so if ou give it until Wednesday I
should be able to have a go at it and test if you want.
On Sun, Dec 11, 2016, 23:14 Jonathan Payne<EMAIL_ADDRESS>wrote:

I have rewritten the zap-to-char function to my multi-cursor aware. It's
actually implemented in terms of another new multi-cursor function called
"jump-to-char".
Do you want to try them out before I release?
Also, does your zap to char eat the character zapped to OR stop just
before that character? I want to know what to set for the default with the
new one.
It works well: if you call zap to char multiple times in a row, with
multiple cursors, they are all accumulated into the same cursor-aware
kill-ring entry.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/grundprinzip/sublemacspro/issues/131#issuecomment-266278628,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAtO1rspaTTOY7xR7Posn8licPnd7vNGks5rG-kogaJpZM4IAAE5
.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-28-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4357

Questions ?
Please refer to the Backport tool documentation


Update README.md

Description

Code sample

Closing as stale, also we're only considering security fixes at this point.


How to count objects?

Hi,
I would like to compute git count-objects for a remote git repo. How can I do that?
Thanks.

Repository.Objects just iterate and count, doesn't exists a dedicated method


Writing policies

Revisit the "writing policies" page

Explain what kubewarden policies are
Provide a low level overview of the communication protocol
used to communicate with policies. This is useful when new languages
are going to be added, new SDKs are going to be written,...

This PR is built on top of https://github.com/kubewarden/docs/pull/9

This PR addresses https://github.com/kubewarden/docs/issues/1

This got closed by the rename of the master branch. Reopening.


Extract raw PHMSA distribution and start of transmission data (Table A-D, H, I)

In this PR, adapt the generic Excel extractor to work on PHMSA datasets. The goal of this PR is to extract raw PHMSA distribution and selected transmission (tables A-D, H, I) data and create corresponding raw assets in PUDL.
This includes:

specifying column maps, footers, skiprows and file names for all years of PHMSA distribution and selected transmission data (tables A-D, H, I)
adapting the generic Excel extractor to work with multi-year zip files with a list of years partitions
adding PHMSA settings to the PUDL datapackage
fixing the missing keywords issue in pudl_datastore (in concert with PR#167 in the pudl-archiver repo)
drops columns that are mapped to other tables during extraction, to prevent split up years of data from getting appended to different datasets multiple times (using process_final_page)
creating a notebook to help diagnose column mapping errors before extraction

Still to do:

[x] Rename Zenodo partition as form not dataset to avoid variable clashes and add listed years partitions in https://github.com/catalyst-cooperative/pudl-archiver/pull/200
[x] Adapt extraction to handle multi-file zips with two partitions
[ ] Add PDFs to data_sources
[ ] Add PHMSA to release notes
[ ] Add PHMSA to docs
[ ] Add PHMSA to README

Out of scope:

Sometimes data spills over into unnamed columns. This happens between 2004-2009 for distribution data, when the comments section overflows into the last four columns and beyond. The data lost is the preparer's email, fax, and name, along with portions of the comment section. As this is a relatively minor issue and affects preparer metadata only, I think we can ignore this issue for now. These columns are dropped during extraction due to the changes made to process_final_page, and are logged as a warning.

PR Checklist

[x] Merge the most recent version of the branch you are merging into (probably dev).
[ ] All CI checks are passing. Run tests locally to debug failures
[x] Make sure you've included good docstrings.
[ ] For major data coverage & analysis changes, run data validation tests
[ ] Include unit tests for new functions and classes.
[x] Defensive data quality/sanity checks in analyses & data processing functions.
[ ] Update the release notes and reference reference the PR and related issues.
[x] Do your own explanatory review of the PR to help the reviewer understand what's going on and identify issues preemptively.

@e-belfer  i added some functionality into the standard excel extractor which checks for but does not require a new page_part_map. using this we pass in the additional part into the partitions for grabbing the zipfile (or it just returns the given partition of the page_part_map file didn't exist). This might be a case of preemptive generalization, but it was simple enough so I figured it was okay.
this is as far as i got testing wise (I also tests a few other excel extracted datasets to be sure i didn't junk them up):
from pudl.extract.phmsagas import Extractor
from pudl.workspace.datastore import Datastore

ds = Datastore(local_cache_path=pudl.workspace.setup.PudlPaths().pudl_input)
extractor_phmsagas = Extractor(ds=ds)
extractor_phmsagas.load_excel_file(page="distribution", years=2020)

something I noticed that might trip this up a little when we are trying to use the raw_df_factory / year_extractor_factory / extract_single_year tooling that most of the other datasets that use the excel extractor use is that extract_single_year assumes year is the partition name where the phmsa archive uses years. I know it literally is a list of years instead of just one year but i would be tempted to change it to year. I think this would be consistent with the rest of the list of parts and honestly feels like it makes more sense to say give me year=2020 instead of years=2020 in the extraction.

Doc updates can be viewed here: https://catalystcoop-pudl--2932.org.readthedocs.build/en/2932/data_sources/phmsagas.html
@aesharpe


build: set to buildoptimized by default like previously

On a side note, the readme should said to install meson with pip on Ubuntu, as the latest release on 21.04 for meson is Version: 0.57.0+really0.56.2-0.1 . So that doesn't pass the meson version requirement.

On a side note, the readme should said to install meson with pip on Ubuntu, as the latest release on 21.04 for meson is Version: 0.57.0+really0.56.2-0.1 . So that doesn't pass the meson version requirement.

shouldn't this meet the 0.57 requirement?

also, the real requirement is 0.55. It's just that I couldn't test it anywhere with such an older version. If you can test it with ubuntu's version of meson, can you add a patch to reduce the required config?

 Then launch ArduPilot SITL : Tools/autotest/sim_vehicle.py -v ArduCopter --console and another mavproxy instance : mavproxy.py --master=udp:<IP_ADDRESS>:14651 --console
Working fine. So the meson version could probably set Ubuntu 20.04 one's as minimal : v0.53.2
On Ubuntu 21.04 is it working too with meson 0.57.0+really0.56.2-0.1 that is detected as v0.56.2 and not v0.57

0.53 doesn't work for cross compilation. See 15bf443c622e ("ci: migrate to meson")

update with the comments addressed and the version change into another PR.


ClemBot.Site dashboard layout sidebar doesnt work on mobile correctly

We should have it be a flyout menu leveraging buefy and bulmas mobile css class options

Cc @MayorMonty

Can I take this issue?

Also do you mind if I take on the design inconsistency fixes too?

im certainly not opposed to it, what design inconsistencies are you seeing?

well the card sizes and layout in the landing page is one. i’ll try to compile a list of what needs fixing tomorrow morning (with photos)

For sure, css/html is not what I'm good at and it's fairly hacked together

Do you mind if I take more time doing this issue? I lowkey need to rewrite the components from scratch instead of just using bulma. I need more control over the styles and how the HTML is structured so...
Also as promised, here's a few screenshots of what I've fixed so far. (dont worry about the home button, it's temporary. i just need to fix the logo being there)

In comparison to what's in the prod version

The sponsor card is inconsistent from the other cards too.

Footer is somewhat... eh....

Colors are somewhat all over the place too
The typography system is somewhat inconsistent
The border radius values as well

For reference, here's a complete CSS audit: https://cssstats.com/stats/?url=https%3A%2F%2Fclembot.io%2F

So those are the things that needs fixing/changing

Those screenshots look good. I like the border emboss and shadow. However this many changes do need some more in depth discussion for sure. Join our discord here and ping me in the #clem-bot-discussion channel. https://discord.gg/wewgg4RQKw


Fix PipelineStage invocation inside DataPipeline.

Minor fix to resolve issue with nested DataPipelines. If DataPipeline is passed to FluidWrapper (assuming it was used in place of some IterableDataset) it will cause an error since DataPipeline is a PipelineStage, but doesn't have run method. Switching checks order helps to deal with it.

Thanks. I have patched this separately (since there was a merge conflict and that was easier).


Build Popup

Conveying momentary information is important to any UI. We need a popup that supports an arbitrary amount of content and flexes in both height and width depending on the size of the content and remains centered on the viewing area.
General thoughts:
We can probably assume that we'll want each popup to include a close button Button with Icon and a title Divider.

Have an agate/Popup: https://github.com/enactjs/agate/pull/24


[SPARK-27166][SQL] Improve printSchema to print up to the given level

What changes were proposed in this pull request?
This PR aims to improve printSchema to be able to print up to the given level of the schema.
scala> val df = Seq((1,(2,(3,4)))).toDF
df: org.apache.spark.sql.DataFrame = [_1: int, _2: struct<_1: int, _2: struct<_1: int, _2: int>>]

scala> df.printSchema
root
|-- _1: integer (nullable = false)
|-- _2: struct (nullable = true)
| |-- _1: integer (nullable = false)
| |-- _2: struct (nullable = true)
| | |-- _1: integer (nullable = false)
| | |-- _2: integer (nullable = false)

scala> df.printSchema(1)
root
|-- _1: integer (nullable = false)
|-- _2: struct (nullable = true)

scala> df.printSchema(2)
root
|-- _1: integer (nullable = false)
|-- _2: struct (nullable = true)
| |-- _1: integer (nullable = false)
| |-- _2: struct (nullable = true)

scala> df.printSchema(3)
root
|-- _1: integer (nullable = false)
|-- _2: struct (nullable = true)
| |-- _1: integer (nullable = false)
| |-- _2: struct (nullable = true)
| | |-- _1: integer (nullable = false)
| | |-- _2: integer (nullable = false)

How was this patch tested?
Pass the Jenkins with the newly added test case.

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/testing-k8s-prb-make-spark-distribution-unified/8933/
Test PASSed.

Test build #103522 has started for PR 24098 at commit 9263218.

LGTM. Thanks!

Thank you for review and approval, @dbtsai !

Test build #103522 has finished for PR 24098 at commit 9263218.

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

Merged to master.

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