Download Error

General troubleshooting tips
Before logging an issue please check that:

[x] You have the latest version installed (There may be a beta version that fixes your issue), see here for the latest releases and bug fixes: https://github.com/JohnCoates/Aerial/releases
[ ] Your issue isn't already mentioned in our issues. You may find a workaround there or a similar request already made.
[ ] Your problem isn't mentioned in the troubleshooting section.

If none of this fixes your issue, tell us about the problem you are experiencing or the feature you'd like to request.
Required information
In order to help us sort your issue, we ask that you provide the following information:

[ ] Mac model: Macbook Pro 2017 Late
[ ] macOS version: 10.15.3 (19D76)
[ ] Monitor setup: 1440x900

If appropriate, please enable Debug mode and Log to disk in Advanced tab and replicate your bug, then attach the AerialLog.txt file (You can access this file through the Advanced tab).
Description of issue / Feature request
{{Replace this}}
11:36:53 : avInit .saver (0.0, 0.0, 296.0, 184.0) p: true o: true
11:36:53 : <AerialView: 0x7f9cdc509930> AerialView setup init (V1.8.0) preview: true
11:36:53 : Time Management initialized
11:36:53 : Display Detection initialized
11:36:53 : Display Detection
11:36:53 : 1 display(s) detected
11:36:53 : 1cm = 50.393701544239036 points
11:36:53 : npass: dict [__C.NSDeviceDescriptionKey(_rawValue: NSScreenNumber): 69733568, __C.NSDeviceDescriptionKey(_rawValue: NSDeviceIsScreen): YES, __C.NSDeviceDescriptionKey(_rawValue: NSDeviceSize): NSSize: {1440, 900}, __C.NSDeviceDescriptionKey(_rawValue: NSDeviceColorSpaceName): NSCalibratedRGBColorSpace, __C.NSDeviceDescriptionKey(_rawValue: NSDeviceResolution): NSSize: {144, 144}, __C.NSDeviceDescriptionKey(_rawValue: NSDeviceBitsPerSample): 8]
11:36:53 :        bottomLeftFrame (0.0, 0.0, 1440.0, 900.0)
11:36:53 : src orig : (0.0, 0.0)
11:36:53 : left 0.0 below 0.0
11:36:53 : [id=69733568, width=1440, height=900, bottomLeftFrame=(0.0, 0.0, 1440.0, 900.0), topRightCorner=(1440.0, 900.0), isMain=true, backingScaleFactor=2.0]
11:36:53 : (0.0, 0.0, 1440.0, 900.0)
11:36:53 : Display Detection Done
11:36:53 : Using : nil
11:36:53 : <AerialView: 0x7f9cdc509930> setting up player layer with bounds/frame: (0.0, 0.0, 296.0, 184.0) / (0.0, 0.0, 296.0, 184.0)
11:36:53 : Using dot workaround for video driver corruption
11:36:53 : Manifest init
11:36:53 : app support exists
11:36:53 : No customvideos.json at : /Users/username/Library/Containers/com.apple.ScreenSaver.Engine.legacyScreenSaver/Data/Library/Application Support/Aerial/customvideos.json
11:36:53 : isManifestCached 10 true
11:36:53 : isManifestCached 11 true
11:36:53 : isManifestCached 12 true
11:36:53 : isManifestCached 13 false
11:36:53 : 2020-02-20
11:36:53 : Interval : Optional(-41813.59520494938)
11:36:53 : No need to check for new videos
11:36:53 : Manifests files were not loaded in class
11:36:53 : Files were not already loaded in memory
11:36:53 : Fetching missing manifests online
11:36:53 : <AerialView: 0x7f9cdc509930> backing change 2.0 isDisabled: false frame: (0.0, 0.0, 296.0, 184.0) preview: true
11:36:54 : <AerialView: 0x7f9cdc509930> startAnimation frame (0.0, 0.0, 296.0, 184.0) bounds (0.0, 0.0, 296.0, 184.0)
11:36:54 : restarting playback
11:36:55 : <AerialView: 0x7f9cdc509930> stopAnimation
11:36:55 : pwc init2
11:36:55 : Aerial control panel V1.8.0
11:36:56 : Cache size : 0,31 GB
11:36:56 : No local or user cache exists, using ~/Library/Application Support/Aerial
11:36:56 : cache to be used : Optional("/Users/username/Library/Containers/com.apple.ScreenSaver.Engine.legacyScreenSaver/Data/Library/Application Support/Aerial")
11:36:56 : Poi Strings Provider initialized
11:36:56 : Preferred languages : ["tr-TR"]
11:36:56 : Best matched language : tr
11:36:56 : TVIdleScreenStrings13.bundle is missing, please remove entries.json in Cache folder to fix the issue
11:36:56 : Defaulting to bundle
11:36:56 : path : /Users/username/Library/Screen Savers/Aerial.saver/Contents/Resources/en.json
11:36:56 : Community JSON : 99 entries
11:36:56 : cvcinit2
11:36:56 : cvcawake
11:36:56 : wdl

Hi @salimcansatici
Can you clarify what is not downloading exactly ? It looks like you don't have the tvOS 13 bundle at the very least.
Can you try downloading this link to see if you don't have an issue with some sort of firewall ? This is where the file is : https://sylvan.apple.com/Aerials/resources-13.tar
Thanks

I can download videos, but the download speed is too slow, only 43 kB/s. Is there any other way to download videos?

I can download videos, but the download speed is too slow, only 43 kB/s. Is there any other way to download videos?

Hi @Andy0570
The videos are hosted by Apple on their servers, but I've been told they are blocked in China, so there's not much I can do. If you want to try to use other tools to download the files manually, you can check the instructions here (but know that Aerial uses 4 threads per file to download already) :  https://github.com/JohnCoates/Aerial/blob/master/Documentation/OfflineMode.md
Your best bet might be to try an alternate connexion (some form of VPN?). Sorry I can't help you more.

I can download videos, but the download speed is too slow, only 43 kB/s. Is there any other way to download videos?

Hi @Andy0570
The videos are hosted by Apple on their servers, but I've been told they are blocked in China, so there's not much I can do. If you want to try to use other tools to download the files manually, you can check the instructions here (but know that Aerial uses 4 threads per file to download already) : https://github.com/JohnCoates/Aerial/blob/master/Documentation/OfflineMode.md
Your best bet might be to try an alternate connexion (some form of VPN?). Sorry I can't help you more.

Hi,
Thank you very much this answer, my problem is solved.


Documentation - General Guidance for Plugin Development

Closes #437
Adding in Plugin Developer Documentation for Contributors outside of the limited team for AaC

[x] Contribution Points are documented

[x] Name
[x] Description of the content
[x] Description/Context of what the data is used for/the impact of it

[x] Describe the components of the auto-generated Plugin structure

[x] __init__.py
[x] implementation file
[x] Plugin architecture file
[x] Adding Jinja templates for generation plugins

[x] How to generate a plugin using the gen-plugin command
[x] How to build, unit test, and integration test the plugin
[x] How to install a plugin and test it with via the AaC CLI

Second to last AC I think is either complete or partially complete will need confirmation, unless I am missing something.

For the Jinja templates, they have to be manually created and added as we don't have any generation support for them. You can look at Gen-Protobuf and Gen-Design-Doc as examples. If I recall correctly they just need to be in a python directory (one with __init__.py)

@Coffee2Bits Let me know if there are any further typos or wording issues. The last commit should have addressed the current comments that are on this PR thus far

The error for the pipeline I am not sure how to handle. It is saying that a certain version of python is not available

The error for the pipeline I am not sure how to handle. It is saying that a certain version of python is not available

This is so weird. The error directs us to a link of versions: https://raw.githubusercontent.com/actions/python-versions/main/versions-manifest.json
Which has a 3.9.0 x64 binary:
    "version": "3.9.0",
    "stable": true,
    "release_url": "https://github.com/actions/python-versions/releases/tag/3.9.0-96741",
    "files": [
      {
        "filename": "python-3.9.0-darwin-x64.tar.gz",
        "arch": "x64",
        "platform": "darwin",
        "download_url": "https://github.com/actions/python-versions/releases/download/3.9.0-96741/python-3.9.0-darwin-x64.tar.gz"
      },
      {
        "filename": "python-3.9.0-linux-16.04-x64.tar.gz",
        "arch": "x64",
        "platform": "linux",
        "platform_version": "16.04",
        "download_url": "https://github.com/actions/python-versions/releases/download/3.9.0-96741/python-3.9.0-linux-16.04-x64.tar.gz"
      },
      {
        "filename": "python-3.9.0-linux-18.04-x64.tar.gz",
        "arch": "x64",
        "platform": "linux",
        "platform_version": "18.04",
        "download_url": "https://github.com/actions/python-versions/releases/download/3.9.0-96741/python-3.9.0-linux-18.04-x64.tar.gz"
      },
      {
        "filename": "python-3.9.0-linux-20.04-x64.tar.gz",
        "arch": "x64",
        "platform": "linux",
        "platform_version": "20.04",
        "download_url": "https://github.com/actions/python-versions/releases/download/3.9.0-96741/python-3.9.0-linux-20.04-x64.tar.gz"
      },
      {
        "filename": "python-3.9.0-win32-x64.zip",
        "arch": "x64",
        "platform": "win32",
        "download_url": "https://github.com/actions/python-versions/releases/download/3.9.0-96741/python-3.9.0-win32-x64.zip"
      },
      {
        "filename": "python-3.9.0-win32-x86.zip",
        "arch": "x86",
        "platform": "win32",
        "download_url": "https://github.com/actions/python-versions/releases/download/3.9.0-96741/python-3.9.0-win32-x86.zip"
      }
    ]
  },

@Coffee2Bits found the issue relating to the pipeline error and he pushed out a fix a shortwhile ago. Merged the lated main into this branch so hopefully the issue from the pipeline will be resolved now

Pipeline seems to be fine now!

Thanks guys! Merging now!


Chi diagnostics from Cherian et al 2024

feature request
Wondering if this could be merged into MITgcm master branch?

code @ https://github.com/MITgcm/MITgcm/compare/master...dcherian:MITgcm:chi-diags
corresponding to https://doi.org/10.1175/JPO-D-23-0135.1 (I believe)

@dcherian

Absolutely, I'd be really happy for this to be in the main branch. I hope I did not make a mistake hehe 🙈

Absolutely, I'd be really happy for this to be in the main branch. I hope I did not make a mistake hehe 🙈

Great! The dev team will be able to advise, either here or on PR you may open


Consider using chat instead of commands

Consider and document the pros and cons of using commands vs reading the message chat.

User's may chat in the channel. Don't want the bot responding to all chat messages


adds tools for running tests with @require

This adds some new functionality surrounding the @require decorator. If you set the environment variable IGNORE_REQUIRE to 'yes' or 'true', then @required tests will run as though there were no decorator. It also modifies @required tests with the attr decorator, so you can select them with nosetests -a required.
Unfortunately, they'll still be skipped if you don't set IGNORE_REQUIRE; to have these play together nicely would require digging into the behavior of plugins, which is out of scope right now.
@aboudreault and/or @shawnkumar to review. Asking for extra-careful review of the docstring changes -- can you think of a better way to document this?

@mshuler should have a look, too, to make sure this does what we need.

Sorry Jim, what is the goal of this? I thought @require were only used for no-ready feature, when waiting a patch to be merged. Why would we want to run only tests with @require? I have probably missed the discussion on hipchat.

@aboudreault I assume the goal is for when we annotate a test with @require(ticket) where the ticket is not a feature in development, but a bug-fix for the feature being tested.

Basically, we discussed using @require a little more liberally, and this would give us a simple way to run with or only the tests with @require. This gives us a jenkins job like http://cassci.datastax.com/view/All_Jobs/job/scratch_mshuler-trunk_dtest-experimental/

High-level: what @mshuler said. More detailed: here's the workflow this supports:

A test is failing in one way or other -- in some cases, it'll be a flaky test.
we file a ticket and @require the test, pointing to the ticket

At this point, most test runs will run normally, skipping @required tests. However, we'll also have a job that runs daily (probably) that either includes @required tests, or only runs @required tests. This will help us keep an eye on flapping and failing tests without adding failure noise to dev branch builds.
I understand it's a little weird to @require a flapping test -- if flaky works out, then we won't have to use it that way. But for the moment, this will help us reduce failure noise in dev branches and normal builds.

Thanks Jim for the clarification. +1 for this -- I agree that the flaky plugin would be in favor in the future.

This also serves as an audit job, since there is a disconnect between fixing a JIRA bug and removing the @requires annotation in dtest. For example, a few tests passed in that run, why are we skipping them (assuming they were features/fixes that aren't just flakey tests)?

Thanks, all. Merging.


Vscode seems to default to dist/index.js version of this lib

Vscode seems to default to dist/index.js version of this lib. Not sure if this a bug or configuration issue. Would be nice to get the typings as I can see that this is developed with typescript.

ok, created a pr https://github.com/Turee/koa-yup-validator/pull/19

Merged, thanks @hixus .


@sourceloop/cli : no proper error when command run in wrong folder

Describe the bug
Very vague error description when sl microservice command is run in a wrong folder
To Reproduce
Steps to reproduce the behavior:

Run sl scaffold test
Dont change the working directory to test (cd test)
Run sl microservice
See error

Expected behavior
A clear and understandable error message should be shown
Screenshots

clear and understanble message is coming. so we are closing this issue


Jaspe_Company

We have written the needed data into your clipboard because it was too large to send. Please paste.

Hey, you maybe better off if you paste the Live Share logs

Closing this issue due to lack of enough information to investigate a fix for it. Thank you!


rollbar PHP memory error

Allowed memory size of<PHONE_NUMBER> bytes exhausted (tried to allocate 20480 bytes) in /rollbar/vendor/rollbar/rollbar/src/Scrubber.php on line 81
Allowed memory size of<PHONE_NUMBER> bytes exhausted (tried to allocate 20480 bytes) in /var/www/rollbar/vendor/rollbar/rollbar/src/DataBuilder.php on line 795

PHP can consume quite a bit of memory for nested structures. Of course, 1G is quite a bit!
At first blush, I'm not sure we'll be able to do much about this with the code as implemented now: we're passing the data directly into parse_str, which is a black box and consumes as much memory as it needs to parse the given data structure. However, we may be able to optimize this for your particular use case if we had some details. Could you post:

What PHP version and server OS you're using
The value of the data you were passing (scrubbing any sensitive values)

I wrote own error handler and for me this is not issue anymore

I wrote own error handler and for me this is not issue anymore

I wrote own error handler and for me this is not issue anymore

Some elaboration would be nice just to see how you handled your particular challenge for future readers. If you could still do this, seeing it has already been a while, that would be great.


Implementation of GivenDataComponent class (reporting component) that describes specified values of parameters

Write a new ReportComponent class, which displays a table in the report for given data for the dynamic system.

DONE
The component was added to mech.pl and mech.en module.


Improve admin delete account page and add ability for admins to rename deleted accounts

Depends on https://github.com/NuGet/NuGetGallery/pull/7471
Currently, we don't have a way for site admins to release the username of a deleted account. This PR adds that functionality to the admin delete account page and also improves the UI of the page.

This appears to have stalled. What needs to happen to move this toward a resolution? Given its age, is this still meeting a need? It's dependent on https://github.com/NuGet/NuGetGallery/pull/7471 which also wasn't merged. Is this something to proceed on or cleanup? /cc @scottbommarito @joelverhagen @agr @loic-sharma

Let's close it. We'll take this later if it needs attention.


Establish CASE on re-subscription

This adds support for re-establishing CASE on re-subscription as a default policy implementation, with the application having the ability to over-ride that if needed.
Fixes #16202
This works out-of-the-box if you call SendAutoResubscribeRequest.
This deletes the function-callback approach in ReadPrepareParams::mResubscribePolicy and uses instead the re-named, ReadClient::Callback::OnResubscriptionNeeded callback to effect the right re-subscription policy.
That callback now pivots from being just an informational callback to one that places the onus on the application (if it over-rides the default impl) to schedule the ensuing re-subscription by calling back into ReadClient::ScheduleResubscription.
Helper methods are provided to help applications pick and select elements of the default re-sub implementation as they desire.
Other notables:

Fixes missing clean-up of timers on ReadClient destruction initiated by the application.
Fixes missing error-handling of calls made during re-subscription to ensure they trigger further re-subscription.

Testing
Using the REPL, I setup the following scenarios:

Setup a subscription successfully, killed the server, and waited till it attempted to re-subscribe multiple times, before booting the server back up and ensuring re-sub succeeded. This validates re-subscription attempts failing, and ensuring the timers get correctly scheduled.
Same as above, but immediately rebooted back the server. This validates re-subscription passing in the first attempt.
Setup 2 independent subs, and validates that both failed, and attempted to re-sub multiple times, before passing. This validates releasing of OperationalDeviceProxy instances from both ReadClient instances and ensuring nothing broke.

FYI I did not update chip-tool since I didn't know whether you wanted the default behavior there as well, or if you wanted it to be hidden behind an option.
@bzbarsky-apple let me know if you'd like me to update chip-tool (which goes through the SendSubscribeRequestWithoutValidation API unfortunately, so that makes it a bit more work..)


[BUG] Parameter  is not working.

Describe the bug
When I am trying to run either mvn se.kth.castor:depclean-maven-plugin:2.0.1:depclean -DcreatePomDebloated=true or mvn se.kth.castor:depclean-maven-plugin:2.0.1:depclean -DcreatePomDebloated=true -DcreateResultJson=true then either of these is not producing a debloated-pom.xml in the root of the project (The later one is only producing a .json and .csv file but not .xml file) .
See the results of mvn se.kth.castor:depclean-maven-plugin:2.0.1:depclean -DcreatePomDebloated=true here
and result of mvn se.kth.castor:depclean-maven-plugin:2.0.1:depclean -DcreatePomDebloated=true -DcreateResultJson=true here
Expected behavior
A debloated-pom.xml file should be created in the root of the project.
I am unable to get what I am doing wrong? But if the problem is from my side only, then please guide me through this.
Thank you
Abhay

Hi @ABHAY0O7, thanks for reporting this.
It is caused by a typo in the property that defines the parameter, here https://github.com/castor-software/depclean/blob/master/depclean-maven-plugin/src/main/java/se/kth/depclean/DepCleanMojo.java#L102.
I noticed that you actually solved this in your PR https://github.com/castor-software/depclean/pull/85.
Can you check if the debloated-pom.xml is generated when using the (current) version 2.0.2-SNAPSHOT?

Hi @cesarsotovaler

It is caused by a typo in the property that defines the parameter, here https://github.com/castor-software/depclean/blob/master/depclean-maven-plugin/src/main/java/se/kth/depclean/DepCleanMojo.java#L102.

You are right, the whole problem is just because of the typo.

Can you check if the debloated-pom.xml is generated when using the (current) version 2.0.2-SNAPSHOT?

Currently, since my PR is not merged, that's why the debloated pom is generating with this parameter -DcreatPomDebloated=true not this -DcreatePomDebloated=true.
But, now I can continue my work and when I finished that then you can review that and merge accordingly.
Also, that PR will solve this issue too so if you are OK with this then I will close this issue when PR ( #85 ) got merged.
Thank you

Hi @ABHAY0O7, that's excellent. Please provide some unit tests as part of https://github.com/castor-software/depclean/pull/85, then I'll merge it!

Hi @cesarsotovalero,
I noticed that the command mvn se.kth.castor:depclean-maven-plugin:2.0.1:depclean -DcreatePomDebloated=true still not producing any debloated pom.
But this command is mvn se.kth.castor:depclean-maven-plugin:2.0.1:depclean -DcreatPomDebloated=true producing a debloated pom.
Can you please confirm that this issue is solved or not?
Thank you.

Hi @ABHAY0O7,
The comma-DcreatePomDebloated=true parameter is not working on version 2.0.1 of DepClean. However, it works on version 2.0.2-SNAPSHOT (the current version on master), thanks to your merged PR (https://github.com/castor-software/depclean/pull/85). Therefore, this issue is solved and you can close it.

Hi @cesarsotovalero
I got it, I was using the wrong command.
Correct one is mvn se.kth.castor:depclean-maven-plugin:2.0.2-SNAPSHOT:depclean -DcreatePomDebloated=true.
All LGTM, so I am closing this issue now.
Thank you


 But I have a lot of legacy apps, that I'm slowly migrating one by one into more rich JS stuff. I'd love to be able to have webpack hot reload working in a classic OWIN pipeline.

@wasabii sorry no plans at this time to support OWIN directly. If you wanted to take a crack at it, and have the time, you could consider porting the code yourself (and maybe even release it to the world!). By my understanding of the code, it's probably not a monumental effort, but it's just not something we can devote any time to given our other priorities. Thanks!


Add Chain[contractAddresses] property

⚠ Depends on https://github.com/safe-global/safe-config-service/pull/1175 from the Config Service.
Summary
When deploying the Safe infrastructure, it is currently necessary for a desired network to be present in our safe-deployments repo. as that is the source of truth for contract addresses.
In order to remove this hurdle, new (optional) fields for contract addresses to populate the ContractNetworksConfig of the protocol-kit for a given chain are to be added to the Config Service. This propagates the fields and validates them accordingly.
Changes

Add new ContractAddressesSchema, include it in ChainSchema and infer a new ContractAddresses entity from it
Add new contractAddressesBuilder and include it in chainBuilder
Add appropriate test coverage

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.005%) to 48.129%

Totals

Change from base Build<PHONE_NUMBER>:
0.005%

Covered Lines:
4245

Relevant Lines:
7079

💛  - Coveralls


Feat: Lets Update menu make them using Cards

Describe the feature
See currently We are seeing only list of food Item we should update it to cards so that in future we can add functionality where user can order food from your website itself. And for that first we will have to make it a card so In This

I will revamp the Ui Of Menu page
2)I will Update the backend of Canteen where Canteen will be able to Add Images And Description of Food item As well
and i will retrive that cards using api

Add ScreenShots
No screenshot required
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Ok do this!


[DOCS] use JSdoc to generate documentation

Description
There are already comments using the jsdoc syntax, generate the documentation
Screenshots
No response

I could work on this.

I think linting isn't proper in the codebase. When I run the jsdoc command it tells to put semi colons

I think linting isn't proper in the codebase. When I run the jsdoc command it tells to put semi colons

na, i really don't think it's that, hopefully

Here is what I did.

Added jsdoc as a dev dependency.
Added a config file for it

{
    "source" :{
        "include": ["src"],
        "includePattern":".ts$",
        "excludePattern":"(node_modules/|docs)"
    },
    "plugins":["plugins/markdown"],
    "templates":{
        "cleverLinks":true,
        "monospaceLinks":true
    },
    "opts":{
        "recurse":true,
        "destination": "./docs/"
    }
}

Added a script to run jsdoc with this config
Run the script and got this error.

Maybe I am doing something wrong could you take a look?

The lint throws these errors

I really don't know, I haven't really generated the docs before

Have you executed your lint command before?

Have you executed your lint command before?

I don't remember😂

Okay! I am unable to pin point the error. I think it will be better if someone else took over. Thanks!

Okay! I am unable to pin point the error. I think it will be better if someone else took over. Thanks!

Alright❤


Gem isn't ruby 2.3 compatible

That's what is defined in the gemspec, but it has ruby 2.6 syntax like this which makes the gem to fail in older versions (ruby 2.4 in my case), so it should be updated the gemspec or change that syntax to make it compatible with older ruby versions.

Hi @alagos , it sounds like your preference is for Ruby 2.4 to remain supported (let me know otherwise!).
I'll check with the team to see whether EOL Ruby versions should continue to be supported.

Well, if ruby 2.4 can be supported, that would be awesome. We're still in the process to upgrade to supported ruby versions, but in the meantime would be nice to have access to this gem.

Hi @alagos , I'm working on restoring comparability for Ruby 2.3 over at #201 , let me know if you have any feedback or comments!
Support for Ruby 2.4 will not be permanent and we will likely have a plan to gradually drop support for older versions of Ruby.
How long do you think your team requires Ruby 2.4 support?

@gchan wow, that's great.
We're in the process to upgrade our stack (a year ago we were with ruby 2.2), so hopefully a year from now.

Hi @alagos , we've published a new version of the gem which supports Ruby versions 2.3 and above.
https://github.com/buildkite/test-collector-ruby/releases/tag/v2.3.2
Please note support for EOL'ed versions of Ruby may be removed in the future.

Sure, I'll keep in mind.
Thanks for all


Revert "enable unnecessary_string_escapes and use_raw_strings"

Reverts flutter/flutter#68302
Reverting due to bot redness and tree closure.

looks like the failures was unrelated to my PR
but LGMT

looks like the failures were unrelated to my PR

Sorry about that. If you could hold off on the reland until tomorrow that would be helpful. We're trying to limit commits to things related to nnbd at the moment while some fixes make their way up from Dart to the engine into the framework.

If you could hold off on the reland until tomorrow

ok I'll wait
After rerun failing jobs all succeeded


10.0.17763.5122 not supported

Version 10.0.17763.5122 is not supported (Windows Server 2019)

DLL included
Thank you very much!
termsrv.zip

https://github.com/sebaxakerhtc/rdpwrap.ini/commit/f09d9648913958343d364a421a46dffba70f364c


Hover effects added in footer's icons

Pull Request
Title
Show Hover effects on footer's Github and Home icons.
Description
Added Hover effect in github and Home icons in footer.
Related Issues
issue no #164
Changes Made
In footer.tsx change add className so i can use that for give className.
In App.css added styles for that.
Checklist

[x] I have tested the changes locally
[x] Documentation has been updated (if necessary)
[x] Changes are backward-compatible

Screenshots (if applicable)

Additional Notes

@VaibhavArora314 Please check it and merge !

is it possible to implement this using tailwind? to maintain consistency throughout the project

@VaibhavArora314 Changes made in tailwind  please check it


Parsing filenames with trailing newlines

Hello!  I am playing with the online elvish shell, and tried testing the following commands:
touch "Hello World\n\n"
ls (find . -name "*\n")
This fails since, as the docs mention, the paren invocation strips a trailing newline.  The docs suggest slurp to avoid this behavior.  So I tried
ls (find . -name "*\n" | slurp)
but unfortunately, this also fails, although the newlines do pass properly to ls.
How can elvish parse filenames like the above properly?

Honestly, the easiest would be to avoid newline characters in filenames - they are all sorts of trouble in any shell.
Having said that, this is achievable by using slurp, but not by itself. The thing is that slurp swallows the whole output, including the final newline. You can see this if you run the command by itself, notice there are three \n at the end:
> find . -name "*\n" | slurp
▶ "./hello\n\n\n"

To overcome this, we can use find's -print0 option, which uses a NUL character as the separator. This will happen both between records, and at the end, as can be seen if we create a second file:
> touch "bye\n\n"
> ls
bye\n\n  hello\n\n
> find . -name "*\n" -print0 | slurp
▶ "./bye\n\n\x00./hello\n\n\x00"

So now we can work on this by splitting the string on the NULL character, and passing the results to ls. One last tweak: we remove the last character of the string returned by slurp to eliminate an empty element at the end:
> str:split "\x00" (find . -name "*\n" -print0 | slurp)
▶ "./bye\n\n"
▶ "./hello\n\n"
▶ ''
> str:split "\x00" (find . -name "*\n" -print0 | slurp)[0:-1]
▶ "./bye\n\n"
▶ "./hello\n\n"
> ls (str:split "\x00" (find . -name "*\n" -print0 | slurp)[0:-1])
./bye\n\n  ./hello\n\n

BTW I think this type of question is better suited for the forum (see "Join the Community" at https://elv.sh/) rather than GitHub issues.

Thank you for the information, it was very illuminating!

You're very welcome, and welcome to Elvish!

@ykonstant1: In addition to the excellent comments by @zzamboni I will point you to a proposal for disallowing newlines (and other "magic" characters) from file names on UNIX: https://dwheeler.com/essays/fixing-unix-linux-filenames.html. POSIX shells not only do not handle newlines correctly, at least not without a huge amount of effort, they also do not handle file names containing spaces or tabs without significant, careful, effort by the person writing the script. Elvish, at least, limits the catastrophic corner case to just one of three of the problematic chars in a POSIX shell :smile:

@krader1961 I have read the proposal, and I agree completely with the spirit.  However, file names in all major OSs are, tragically, also file manager display names.  It is what non-technical people see as "the file's title".
I am a reseacher and expositor.  In my field, filenames with special characters abound: a researcher downloads a paper from arxiv, say 4040.9592.pdf.  They open it, grab the title and authors (and thus  commas, periods, newlines, spaces, hyphens, tex code and high unicode) and paste it in the file name.  It is a widespread practice.  Same for writers, middle managers in non-tech companies and so on.
And crucially, from their perspective, it is absolutely the right thing to do!  They are modifying the display name of a document to make it easier to identify on their desktop or file manager.
Being prevented by the system, as in Wheeler's proposal, would be the best compromise in my opinion.  But it is still an unfortunate compromise resulting from desktop environments using the file name as their display name.
Sorry for hijacking the issue to write this, but I think it is an important and overlooked point by many developers.

The original question has been answered and the issue has been quiet for two weeks. Close it? Unless, of course, you have a proposal for how Elvish can handle the filename with an embedded newline case better than it does today. Alternatively, a suggestion for how the documentation could be improved.


Docker compose modularity improvements + kubernetes support

As per pull request: Refactor docker-compose configuration for modularity.

Improved docker compose overrides specialization.
Added an installation script with complete functionalities.
Updated README.md accordingly
Fixed .gitignore which was too generic.
Added support for kubernetes (kustomize and helm)

Note: the script "run-compose.sh" contains a function (for linux) to automatically detect the GPU installed on the host system, i'm not be able to test it on AMD and Intel GPUs cause i do not have any. On Nvidia works flawlessly.

Thank you so much for your PR, this is amazing stuff! I'll leave review comments in a bit, and will merge it to main afterwards. Thanks!

Again, thanks a lot for this PR. For the k8s part, if anyone in the community could review it, it will be greatly appreciated!

I struggle to test with multi-gpu environments, i don't currently have a multi-gpu cluster available.
Front-end scales successfully but ollama it's limited to 1 replica due to it's being stateful, i need to evaluate it's ability to manage multiple connections to the same model file by different ollama replicas (also history should be accessible from multiple sessions).

Hi, I'll take a look and try to merge this PR to main after #216! In the meantime, if you could help me with the testing of #216, I'd appreciate it a lot. Also, I wanted to express my sincere thanks for your sponsorship; I'm incredibly grateful for your support!

Hi, I'll take a look and try to merge this PR to main after #216! In the meantime, if you could help me with the testing of #216, I'd appreciate it a lot. Also, I wanted to express my sincere thanks for your sponsorship; I'm incredibly grateful for your support!

Ofc, i'll help testing out the #216! Np for support, i love the project! I think that in combination with ollama and a good model like Wizard-Vicuna-7B this will be an industry-changing project!

@dnviti minor tweak to make the webui Ingress optional: https://github.com/dnviti/ollama-webui/pull/1

Hey @dnviti,
I've made some updates to enable the selection of GPU nodePools in Cloud environments and added persistent storage for WebUI authentication.
I've tested these changes on GKE and they seem to be working well.
You can check out the details in the PR: dnviti#2.
As this is my first contribution to an open-source project, there might be some areas where I could improve.
If you spot anything, please leave a comment and I'll be more than happy to make the necessary adjustments.

Why building an image, if there is a prebuilt image available ?
Prebuilt images are more userfriendly and faster to deploy :)
image: ghcr.io/ollama-webui/ollama-webui:main

Because it could be useful for an advanced user to change something and then start it's own docker image locally.
Why not enabling that feature, being totally optional?
It will build only if passing the --build parameter to "docker compose up -d" command or to my script "./run-compose.sh --enable-gpu --build", as stated in the new README.md.

Again, thanks a lot for this PR. For the k8s part, if anyone in the community could review it, it will be greatly appreciated!

Review from @braveokafor merged succesfully @tjbck

Hi! Could you resolve the conflicts so that I can merge this PR to main? Thanks for all the hard work!

Hi! Could you resolve the conflicts so that I can merge this PR to main? Thanks for all the hard work!

Done

Facing Error:  kubernetes/helm/templates/ollama-statefulset.yaml: SyntaxError: All collection items must start at the same column (17:7) issue :/

Facing Error: kubernetes/helm/templates/ollama-statefulset.yaml: SyntaxError: All collection items must start at the same column (17:7) issue :/

i'm going to check it right now!

Facing Error: kubernetes/helm/templates/ollama-statefulset.yaml: SyntaxError: All collection items must start at the same column (17:7) issue :/

Helm and kubernetes are working as intended, it seems though that Prettier cannot consistently check and format helm template syntax.
Adding the kubernetes folder to .prettierignore fixes the pipeline errors.
Sorry, my bad, didn't try to format using prettier.

LGTM, Thanks for all the hard work and Happy New Year!

LGTM, Thanks for all the hard work and Happy New Year!

Happy new year for you too and i hope it will be a marvelous new year for open source LLMs and this project!

@dnviti, If you haven't joined our discord server yet, I'd greatly appreciate if you could join and help others troubleshooting run-compose.sh file. Thanks as always!

@dnviti, If you haven't joined our discord server yet, I'd greatly appreciate if you could join and help others troubleshooting run-compose.sh file. Thanks as always!

Of course! I'll be there as soon as possible!


Extend individual options

Rather than replicating the entire options object, this PR allows the user to only extend those options he/she wishes too.
Changes moved into this branch hence why: https://github.com/jkuri/ng-daterangepicker/pull/27 is closed

bump


Ray Executor

In addition to accelerator support (e.g. via #304), Cubed could benefit ML users by providing ray executor: https://docs.ray.io/en/latest/ray-core/walkthrough.html
Since Cubed is a serverless model, I bet it could get away with only using Tasks/remote functions.
From talking with @cromwellian a bit, my hope is that Cubed could provide memory bounds when trying to saturate GPUs during model training. I'm not totally sure exactly what a training loop with Cubed would look like. Here's how ray integrates with PyTorch, for example:  https://docs.ray.io/en/latest/train/api/doc/ray.train.torch.TorchTrainer.html#ray.train.torch.TorchTrainer
@shoyer pointed out to me once the idea that GPU OOM errors occur while taking the gradient of a function graph, not necessarily on the forward pass. I'm not totally sure right now if Cubed is in fact a good fit for tackling this problem, only that the potential is exciting.

Thanks for opening this issue @alxmrs! I think Ray would be a great runtime for Cubed, and should be relatively straightforward to write an executor for (maybe a bit like the Modal one?). Do you know what people generally run Ray on in production/at scale?

Hey Tom! Do you mean what does the userbase look like, or do I know specific people? On the former: Ray is the engine that OpenAI uses to train its GPT models; it's really popular in the ML world. On the latter: Ray, the person (cromwellian), uses Ray, the framework, at Roblox for model training. :)

should be relatively straightforward to write an executor for (maybe a bit like the Modal one?).

I agree, and it does look like it will be similar to Modal.

I meant usage of Anyscale vs KubeRay vs ?? I was wondering if there was a choice that most people use, or whether it's a bit of everything.

Ray, the person (cromwellian), uses Ray, the framework, at Roblox for model training. :)

Got it!

I added some notes on how to write a new executor in #498.


fix: fix for snakemake deployment for mamba > 2.0.0 #3108

Description
This PR removes the --no-default-packages flag if the dependency engine detected is determined to be mamba since this flag is deprecated in mamba >= 2.0.0.
QC
This change should be covered by the pre-existing test cases that exhibit conda dependencies. It also should not require any documentation changes

[x] The PR contains a test case for the changes or the changes are already covered by an existing test case.
[x] The documentation (docs/) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).

Summary by CodeRabbit

New Features

Improved handling of the --no-default-packages option when creating conda environments based on the selected frontend.

Bug Fixes

Enhanced clarity in logging messages related to environment creation.

mamba 1.x does not ignore .condarc  "create_default_packages" option. If specified, in .condarc, this might lead to unwanted results. Does mamba 2.0x ignore this?

ran into this error in my tests as well, working solution?:
"--no-default-packages" if (
  self.frontend != "mamba" or not self.mamba_version.startswith("2")
) else ""

and
    @property
    def mamba_version(self) -> str:
        if not is_mamba_available():
            return ""
        try:
            version = subprocess.check_output(["mamba", "--version"]).decode().strip()
            return version.split()[-1]
        except subprocess.CalledProcessError:
            logger.debug("Could not determine mamba version.")
            return ""

@jjjermiah thats a good call to version-check mamba to preserve the behavior of <2.0.0. I wonder if its more forward-facing to actually do the complement logic though (in anticipation of 3.0, etc)?
"--no-default-packages" if (
  self.frontend != "mamba" or self.mamba_version.startswith("1")

@aabiddanda looks like #3121 and the new release deprecate mamba anyways

wow useful to know this @jjjermiah - thanks for the heads up. This seems like a more effective solution than mine at the moment. I'll close this issue/PR since its superceded by #3121


Network Design for Multi-GPU support

https://www.reddit.com/r/MachineLearning/comments/fbse3i/discussion_how_to_support_multigpu_for_a_deep/

@silverdevelopper @ocebenzer I think we can continue with the current design. I found that we can parallelize the way of calculations. If it's possible, we don't need to make a change in the design. We can speed up the documentation and the other stuff.


Feature Request: Update helmchart targets to wildcards from autodiscovery

Is your feature request related to a problem?
I'm trying to setup helm autodiscovery to automatically create GitHub PRs when chart dependency updates are available.
Here is a simplified directory with multiple helm charts which each have their own dependencies:
├── README.md
├── apps
│   ├── external-dns
│   │   └── Chart.yaml
│   └── external-secrets
│       └── Chart.yaml
└── manifest.yaml

I initially tried with the following manifest but I get the note (from --debug) that a target is needed in order for an action to be run: DEBUG: no target found, skipping action

Autodiscovery manifest with no target
scms:
  default:
    kind: "github"
    spec:
      user: "GitHub Actions"
      email<EMAIL_ADDRESS>      username: "github-actions"
      branch: "main"
      owner: "foo"
      repository: "bar"
      token: "token"

autodiscovery:
  scmid: "default"
  crawlers:
    helm:
      versionincrement: "none"

actions:
  default:
    kind: github/pullrequest
    scmid: "default"
    spec:
      automerge: false
      draft: true
      labels:
        - "dependencies"

I then tried to add a target but it seems that the helmchart spec.name is required and doesn't allow wildcards. This means I need to specify each path as a target which seems to defeat some of the purpose of autodiscovery.
For example, this fails:

Autodiscovery manifest with wildard target (failing)
scms:
  default:
    kind: "github"
    spec:
      user: "GitHub Actions"
      email<EMAIL_ADDRESS>      username: "github-actions"
      branch: "main"
      owner: "foo"
      repository: "bar"
      token: "token"

autodiscovery:
  scmid: "default"
  crawlers:
    helm:
      versionincrement: "none"

targets:
  test:
    name: Generic target
    kind: helmchart
    spec:
      name: "apps/*"
      file: Chart.yaml
      key: "$.dependencies[0].version"
    scmid: default

# PR
actions:
  default:
    kind: github/pullrequest
    scmid: "default"
    spec:
      automerge: false
      draft: true
      labels:
        - "dependencies"

In order to get this to work, I need to specify the full path to the chart. Example:
targets:
  test:
    name: Generic target
    kind: helmchart
    spec:
      name: "apps/external-dns"
      file: Chart.yaml
      key: "$.dependencies[0].version"
    scmid: default

Solution you'd like
I'd like to avoid having to add a target for each chart directory.
It would be ideal if we could provide a wildcard path in the target that can match multiple charts in a repository.
Alternatives you've considered
Currently my workaround is to not use autodiscover at all and instead wrap updatecli in a script that iterates over each directory in apps/ then creates a custom values.yaml file which is fed into updatecli. While this works, it's more cumbersome than I'd like.
Anything else?
No response

Thanks for opening the issue, I am wondering what's wrong with your attempt to do the autodiscovery manifest because your example is almost good and you shouldn't need to specify target.
Your are just missing the actionid in the autodiscovery section such as
autodiscovery:
  scmid: "default"
  actionid: "default"
  crawlers:
    helm:
      versionincrement: "none"

The scmid specifies which git repository to use for fetching the helm files and the actionID specifies which action config to use when opening the pullrequest.
I am wondering if you didn't forget the --experimental flag such as updatecli diff --config manifest.yaml --experimental
The "autodiscovery" analyzes directories/files to generate "common" manifest, you can:

see what would be generated by Updatecli by running updatecli manifest show --experimental or updatecli manifest show --experimental --config manifest.yaml
see what would be changed by running updatecli diff --experimental
apply the changes by running updatecli apply --experimental --config yourManifest.yaml

You can find working examples on https://github.com/epinio/helm-charts/tree/main/updatecli/updatecli.d/autodiscovery
And the github action workflow https://github.com/epinio/helm-charts/blob/main/.github/workflows/updatecli.yml
One of the challenges when it comes to dependency management with change validation is to handle the load of change request.
I am experimentation with three main different approaches that work differently depending on the project
One pullrequest with all dependencies
---
name: "Autodiscovery"

autodiscovery:
  scmid: default
  actionid: default
  crawlers:
    helm:

scms:
  default:
    kind: "github"
    spec:
      user: "{{ .github.epinio.user }}"
      email: "{{ .github.epinio.email }}"
      owner: "{{ .github.epinio.owner }}"
      repository: "{{ .github.epinio.repository }}"
      token: '{{ requiredEnv .github.epinio.token }}'
      username: "{{ .github.epinio.username }}"
      branch: "{{ .github.epinio.branch }}"

actions:
  default:
    kind: "github/pullrequest"
    scmid: "default"
    spec:
      title: Bump all chart dependencies
      labels:
        - "dependencies"
        - "helm"

One pullrequest per dependencies
Only groupby: individual is changed here
---
name: "Autodiscovery"

autodiscovery:
  groupby: individual
  scmid: default
  actionid: default
  crawlers:
    helm:
scms:
  default:
    kind: "github"
    spec:
      user: "{{ .github.epinio.user }}"
      email: "{{ .github.epinio.email }}"
      owner: "{{ .github.epinio.owner }}"
      repository: "{{ .github.epinio.repository }}"
      token: '{{ requiredEnv .github.epinio.token }}'
      username: "{{ .github.epinio.username }}"
      branch: "{{ .github.epinio.branch }}"

actions:
  default:
    kind: "github/pullrequest"
    scmid: "default"
    spec:
      labels:
        - "dependencies"
        - "helm"

One pullrequest with all dependencies per chart
By using only we can filter down to a specific chart
---
name: "Autodiscovery"

autodiscovery:
  groupby: all
  scmid: default
  actionid: default
  crawlers:
    helm:
      versionfilter:
        kind: semver
      only:
        - path: "chart/epinio/*"

scms:
  default:
    kind: "github"
    spec:
      user: "{{ .github.epinio.user }}"
      email: "{{ .github.epinio.email }}"
      owner: "{{ .github.epinio.owner }}"
      repository: "{{ .github.epinio.repository }}"
      token: '{{ requiredEnv .github.epinio.token }}'
      username: "{{ .github.epinio.username }}"
      branch: "{{ .github.epinio.branch }}"

actions:
  default:
    kind: "github/pullrequest"
    scmid: "default"
    spec:
      title: Bump Epinio chart dependencies
      labels:
        - "dependencies"
        - "helm"

I am closing this issue as I think my answer provided the needed context but feel free to reopen if I am missing something


Cannot read from abcpath because it is incomplete

alluxio fs stat alluxio://alluxio:19998//xxx/yyy/zzz/dt=2020-12-07/6/000000_0
alluxio://alluxio:19998//xxx/yyy/zzz/dt=2020-12-07/6/000000_0 is a file path.
FileInfo{fileId=6098199248895, fileIdentifier=null, name=000000_0, path=//xxx/yyy/zzz/dt=2020-12-07/6/000000_0, ufsPath=hdfs://hdfs/hive/warehouse//xxx/yyy/zzz/dt=2020-12-07/6/000000_0, length=0, blockSizeBytes=134217728, creationTimeMs=1607910793716, completed=false, folder=false, pinned=false, pinnedlocation=[], cacheable=true, persisted=true, blockIds=[], inMemoryPercentage=100, lastModificationTimesMs=1607729256078, ttl=-1, lastAccessTimesMs=1607997847785, ttlAction=DELETE, owner=hadoop, group=supergroup, mode=493, persistenceState=PERSISTED, mountPoint=false, replicationMax=-1, replicationMin=0, fileBlockInfos=[], mountId=1, inAlluxioPercentage=100, ufsFingerprint=, acl=user::rwx,group::r-x,other::r-x, defaultAcl=}
This file does not contain any blocks.
but hdfs have this file and it is completed.
How should I fix it, please help me.
alluxio version:2.3.0

@xinip If the file is already in HDFS, you can re-sync that particular file. To do that you can run:
alluxio fs -Dalluxio.user.file.metadata.sync.interval=0 stat alluxio://alluxio:19998//xxx/yyy/zzz/dt=2020-12-07/6/000000_0

What this will do is force a sync with the HDFS, in order to retrieve the latest metadata for that file.

my config interval is 0.but still error.
cat alluxio-site.properties|grep "alluxio.user.file.metadata.sync.interval"
alluxio.user.file.metadata.sync.interval=0

@xinip Hrmmm, can you delete the incomplete data?
alluxio fs rm --alluxioOnly /xxx/yyy/zzz/dt=2020-12-07/6/000000_0

This will remove the metadata from only Alluxio, and then you can read it again with
alluxio fs stat /xxx/yyy/zzz/dt=2020-12-07/6/000000_0

@xinip Yes, I verified in the code that the sync will ignore incomplete files. This is because there is an assumption that incomplete files are in the process of being written to, so the sync should not disturb incomplete files. However, in our case, it seems like the writing was maybe interrupted, so now, you can force to remove the inconsistent file by alluxio fs rm --alluxioOnly and then sync it from UFS again with alluxio fs -Dalluxio.user.file.metadata.sync.interval=0 stat.

alluxio fs rm --alluxioOnly also report an error, it cannot be executed successfully，because our alluxio is read-only

@xinip Could you describe how you "created" this (incomplete) file, if the mount is read-only? What operations did you perform to create this file?

my app is presto, presto on alluxio, this my alluxio config:
alluxio.master.mount.table.root.ufs=hdfs://hdfs/hive/warehouse/
alluxio.master.mount.table.root.readonly=true

it's recommended to use local cache with presto. Will close it for now, feel free to reopen it and contact us if this is a strong request.


The Title and the Command are not matching

The title says Get-AksHciContainerStorage, while the cmdlet is Get-AksHciStorageContainer
The cmdlet that works is Get-AksHciStorageContainer
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 76441d3e-7523-e1de-97ad-ce50f1ac786f
Version Independent ID: b3f8f717-5696-6fa6-e5e9-3829ca907e89
Content: Get-AksHciStorageContainer for AKS hybrid - AKS hybrid
Content Source: AKS-Hybrid/reference/ps/get-akshcistoragecontainer.md
Service: azure-stack
Sub-service: aks-hci
GitHub Login: @sethmanheim
Microsoft Alias: sethm

@jaypaddy
Thanks for your feedback! We will investigate and update as appropriate.

@jaypaddy Thank you very much for catching this. We will fix this ASAP and it should be live later today. Thanks again for the contribution! #please-close


Use fourquare bio to link to freenode nick. See issue #1.

This probably has bugs- but it should give you a concept how this can be done.

@maxogden ....as I mentioned at NodeConf. 😺

MAKE NERDTRACKER GREAT AGAIN


make all links open a new window

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.006%) to 99.613%

Totals

Change from base Build<PHONE_NUMBER>:
0.006%

Covered Lines:
12462

Relevant Lines:
12464

💛  - Coveralls


Uninstallation Documentation

As a potential user of NKG
I want a document to describe the uninstallation process to me
So that I do not have to struggle to remove all relevant resources or reroll my entire cluster to remove NKG.
Acceptance

A document containing uninstallation instructions is created in the NKG repo.

I think this is blocked by #780


Update README URLs based on HTTP redirects

Created with https://github.com/dkhamsing/frankenstein
GitHub Corrected URLs

Was
Now

https://github.com/wtmoose/TLIndexPathTools
https://github.com/SwiftKickMobile/TLIndexPathTools

https://github.com/wtmoose/TLLayoutTransitioning
https://github.com/SwiftKickMobile/TLLayoutTransitioning

          1 Warning

  :warning:
  Too many changes (when adding, please keep it to one project per Pull Request)

  Generated by :no_entry_sign: danger


[python-package] running .predict() on a loaded model silently sets the feature_names attribute

When I load a model with bst.load_model() that was saved using bst.save_model(), and then call bst.predict(X) for the first time, the model silently is given the feature_names of X, which can be dangerous behavior if the X isn't actually the full size of the model. This is a consequence of the following code in Booster._validate_features(self, data):
if self.feature_names is None:
    self.feature_names = data.feature_names
    self.feature_types = data.feature_types

I know the documentation recommends pickling, but I think a warning should be displayed if _validate_features() is actually setting feature_names rather than validating.
Example:
In [1]: import xgboost as xgb

In [2]: import numpy as np

In [5]: X_train = np.random.rand(5,10)

In [8]: bst = xgb.XGBRegressor()

In [13]: y_train = np.random.rand(5,1)

In [14]: bst.fit(X_train, y_train)
[08:23:04] WARNING: src/objective/regression_obj.cu:152: reg:linear is now deprecated in favor of reg:squarederror.
Out[14]:
XGBRegressor(base_score=0.5, booster='gbtree', colsample_bylevel=1,
             colsample_bynode=1, colsample_bytree=1, gamma=0,
             importance_type='gain', learning_rate=0.1, max_delta_step=0,
             max_depth=3, min_child_weight=1, missing=None, n_estimators=100,
             n_jobs=1, nthread=None, objective='reg:linear', random_state=0,
             reg_alpha=0, reg_lambda=1, scale_pos_weight=1, seed=None,
             silent=None, subsample=1, verbosity=1)

In [17]: bst.save_model('model')

In [25]: bst_loaded = xgb.XGBRegressor()

In [26]: bst_loaded.load_model('model')
[08:26:24] WARNING: src/objective/regression_obj.cu:152: reg:linear is now deprecated in favor of reg:squarederror.

In [27]: X_pred_1 = np.random.rand(1, 9) # oops! wrong input shape

In [28]: bst_loaded.predict(X_pred_1) # still returns prediction
Out[28]: array([0.25939447], dtype=float32)

In [29]: bst_loaded.get_booster().feature_names # silently has defined `feature_names`
Out[29]: ['f0', 'f1', 'f2', 'f3', 'f4', 'f5', 'f6', 'f7', 'f8']

In [30]: X_pred_2 = np.random.rand(1, 10) # now try correct input shape

In [31]: bst_loaded.predict(X_pred_2)
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-31-7e193d6f83c4> in <module>
----> 1 bst_loaded.predict(X_pred_2)

~\AppData\Local\Continuum\miniconda3\envs\mpopp_infra\lib\site-packages\xgboost\sklearn.py in predict(self, data, output_margin, ntree_limit, validate_features)
    454                                           output_margin=output_margin,
    455                                           ntree_limit=ntree_limit,
--> 456                                           validate_features=validate_features)
    457
    458     def apply(self, X, ntree_limit=0):

~\AppData\Local\Continuum\miniconda3\envs\mpopp_infra\lib\site-packages\xgboost\core.py in predict(self, data, output_margin, ntree_limit, pred_leaf, pred_contribs, approx_contribs, pred_interactions, validate_features)
   1282
   1283         if validate_features:
-> 1284             self._validate_features(data)
   1285
   1286         length = c_bst_ulong()

~\AppData\Local\Continuum\miniconda3\envs\mpopp_infra\lib\site-packages\xgboost\core.py in _validate_features(self, data)
   1688
   1689                 raise ValueError(msg.format(self.feature_names,
-> 1690                                             data.feature_names))
   1691
   1692     def get_split_value_histogram(self, feature, fmap='', bins=None, as_pandas=True):

ValueError: feature_names mismatch: ['f0', 'f1', 'f2', 'f3', 'f4', 'f5', 'f6', 'f7', 'f8'] ['f0', 'f1', 'f2', 'f3', 'f4', 'f5', 'f6', 'f7', 'f8', 'f9']
training data did not have the following fields: f9

Happy to submit a PR that issues a warning when Booster._validate_features(self, data) is called with self.feature_names is None.

Happy to submit a PR that issues a warning when Booster._validate_features(self, data) is called with self.feature_names is None.

Yes, a pull request is welcome.

Closing in favor of #6520.


SetRole when dying from tesla

When, being Scp0492, you die on tesla, scp0492 makes you again.
The problem is here, when you die from Tesla, in ev.Killer.Role = your role
https://github.com/DGvagabond/SCP-008-X/blob/55e293e1672702a773162c9b9f0927ca4ee02c3b/Handlers/Player.cs#L55

I've gone ahead and added a new if check at the beginning of the Dying event that'll allow the target to die if they've died to Tesla, decontamination, falling into a pit, or from the pocket dimension. The code is as follows:
 if (DamageTypes.FromIndex(ev.HitInformation.Tool).name == "DECONT" ||                    DamageTypes.FromIndex(ev.HitInformation.Tool).name == "TESLA" ||                    DamageTypes.FromIndex(ev.HitInformation.Tool).name == "FALLDOWN" ||                    ev.Target.ReferenceHub.playerEffectsController.GetEffect<Corroding>().Enabled)             {                 ev.IsAllowed = true;                 return;             }
If this does not fix the issue, feel free to ping me in the Discord. I've updated release v1.1.2 to include this change.

Everything is working fine so far. Could you in the future transfer all broadcast/hints to the config so that they could be translated.
https://github.com/DGvagabond/SCP-008-X/blob/c28a701386ddeebd94e84d59c06c336503dc2112/Handlers/Player.cs#L84
Also add the file you forgot to the github Properties\AssemblyInfo.cs


Suggestion: add pysnooper.snoop decorator to add failing function's variables' types/values/updates to error message context

pysnooper.snoop() is a decorator that helps automate printf style debugging: example
import pysnooper

@pysnooper.snoop()
def number_to_bits(number):
    if number:
        bits = []
        while number:
            number, remainder = divmod(number, 2)
            bits.insert(0, remainder)
        return bits
    else:
        return [0]

number_to_bits(6)

I think this will help focus/improve GPT-4's debugging ability - https://github.com/cool-RR/PySnooper
There's also torchsnooper for even better snoop insight into pytorch

Cool idea. As it increases the number of tokens it increases costs and reduces the maximal length of processed code as far as I know. Could be an additional option.

Does vectorizing the Python code make it easier for the LLM to understand the syntax and semantics of the code, and potentially provide better suggestions for error fixes?

thanks for the tip, I will look into PySnooper!

Does vectorizing the Python code make it easier for the LLM to understand the syntax and semantics of the code, and potentially provide better suggestions for error fixes?

GPT-4 is pretty good at understanding syntax, loops, etc. It's possible it could help, feel free to experiment and let us know!


txn: Add txn_heart_beat API

What have you changed?
Please explain in detail what the changes are in this PR and why they are needed:
This PR adds txn_heart_beat API to TiKV. This feature is required for supporting Large Transaction.
txn_heart_beat API should be invoked on primary lock on a transaction and try to enlarge the lock's TTL to the specified value.
What is the type of the changes?

New feature (a change which adds functionality)

How is the PR tested?
By unit tests. It's better that we do some further tests.
Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No.
Does this PR affect tidb-ansible?
No.
Refer to a related PR or issue link (optional)
Corresponding PR in TiDB: https://github.com/pingcap/tidb/pull/11979

/run-integration-common-test

@youjiali1995 @AndreMouche @tiancaiamao @coocood PTAL

/run-integration-common-test

LGTM

Need to update gen_command_lock?

@coocood Yes you are right. I forgot it again. We should avoid using wildcards _ in this kind of code to avoid forgotting adding new invariants to these matches.

LGTM

@sticnarf @youjiali1995 PTAL

@coocood Yes you are right. I forgot it again. We should avoid using wildcards _ in this kind of code to avoid forgotting adding new invariants to these matches.

https://github.com/tikv/tikv/pull/4614 doesn't help?

It helps and panicked in TiDB's integration test.
I just noticed that I forgot to add tests for the storage. I'll add it. Wait a minute.

@youjiali1995 PTAL again thanks


VSCode is undefined in debug console when debug extension

Version info:
Version: 1.63.0-insider (Universal)
Commit: d09289a2b6dd0e56b2a264a8e9380685498e10d6
Date: 2021-12-04T02:02:08.723Z
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Darwin x64 21.1.0
ms-vscode.js-debug-nightly: v2021.11.1217

Steps to Reproduce:

clone repository: https://github.com/tjx666/path-autocomplete
add breakpoint in following screenshot position
input vs in debug console

@weinand

TypeScript needs to implement names mapping for imports in order to fix this: https://github.com/microsoft/TypeScript/issues/9627

I had the same issue for imports with vscode too.
Shame Typescript doesn't translate esm named imports to a equivalent left-hand destructor in cjs.
Instead it translates
// esm
import { workspace } from 'vscode'`
const codeLensEnabled = vscode.workspace.getConfiguration().get('editor.codeLens')

into
// cjs debug evaluate fails :'(
const vscode_1 = require("vscode");
const codeLensEnabled = vscode_1.workspace.getConfiguration().get('editor.codeLens')

A workaround I found is to trick the compiler by using import * as vscode from 'vscode'
// esm
import * as vscode from 'vscode'
const codeLensEnabled = vscode.workspace.getConfiguration().get('editor.codeLens');

translates into
// cjs debug evaluate works!
const vscode = require("vscode");
const codeLensEnabled = vscode.workspace.getConfiguration().get('editor.codeLens');


Create new versions of documentation website

Info
We need to be able to support the latest version up to our major upgrade, and the versions after our upcoming major upgrade (after the following story #1379), on our documentation. As we will have to support the two versions simultaneously for a period of 3 months.
This will be done by implementing a system similar to Bootstrap's documentation, where they have the version # in the URL, and that separates which version the documentation is for.
Acceptance Criteria:

We have two different URLs for component documentation

How we decide on what the URLs should be is still left to be figured out. We have an upgrade from v3 to v4 for Angular, and from v4 to v5 for React.
Figma design for framework and version switcher

We will have to write all new documentation for Angular and React for the new version being supported.

Can't be done until #2035 and #2053 are complete


Use ExoPlayer as the built-in player

Describe your suggested feature
Use ExoPlayer as the built-in player instead of the current player.
Other details
Well, I've liked every feature of this app except the player. also, exoplayer is built by gogle.
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I will fill out all of the requested information in this form.

aniyomi has always used exoplayer 🤦

doesnt look like it.

🗿


Sword Blocking Combat (= Golden Age Combat) integration

Golden Age Combat
Broadly comes down to two things (afaict):

Allow shield effectiveness to be applied to ItemSword when Sword Blocking Combat is installed
Integrate DDD features into its blocking system.

To me: Additional info about this was discussed in #60 if needed

If Golden Age Compat allows swords to block similar to shields, then adding ItemSword to shieldClasses in the compat section of DDD's config in 1.7 should get this working. If it doesn't work, please reopen this issue.


$message is undefined when form was created with id set

Description
Maybe I'm missing something obvious here but:
setting await superValidate(schema, { id: 'foo' }); will result in $message on the client always being undefined.
https://stackblitz.com/edit/sveltekit-superforms-crud-4mxznc?file=src/routes/+page.server.ts
This is the CRUD example with id being set. You can observe that the message user created will no longer show.

You need to use the same id when you call superValidate in the form action as well, so the updated data can be sent to the correct form. So it's a simple fix on line 31 in +page.server.ts:
const form = await superValidate(data, schema, { id: 'foo' });

🤦♂ That makes a lot of sense. Thanks

No problem, easy to overlook. :)


[WIP] Making cross compile independent of gox vender package

What is the purpose of this change? What does it change?

Was the change discussed in an issue?
https://github.com/openshift/odo/pull/1998#discussion_r315745452

How to test changes?

make cross

You can make this cleaner by doing it in a shell script and calling it here i think

.PHONY: cross
cross:
    @for platform in linux darwin windows ; do \
        if [ $$platform == "windows" ]; then \
        IS_EXE := .exe; \
        fi \
        go build -o dist/bin/$$platform-amd64/odo$(IS_EXE) $(BUILD_FLAGS) ./cmd/odo/ \
    done

I prefer this over the change i made. But unfortunately the syntax fix took longer time without success. If anybody fix the error please comment cc_ @mohammedzee1000 @kadel

.PHONY: cross
cross:
    @for platform in linux darwin windows ; do \
        if [ $$platform == "windows" ]; then \
        IS_EXE := .exe; \
        fi \
        go build -o dist/bin/$$platform-amd64/odo$(IS_EXE) $(BUILD_FLAGS) ./cmd/odo/ \
    done

I prefer this over the change i made. But unfortunately the syntax fix took longer time without success. If anybody fix the error please comment cc_ @mohammedzee1000 @kadel

Finally able to fix the syntax error.

Please remove gox from makefile and from other places if it is mentioned somewhere

It works, although as stated by @kadel remove remaining references

Please remove gox from makefile and from other places if it is mentioned somewhere

Done

[odo]  ✗  Failed To Update Config To Component Deployed
[odo]  ✗  Applying configuration [55s]

/test integration

/refresh

• Failure [156.587 seconds]
odoJavaE2e
/go/src/github.com/openshift/odo/tests/integration/java_test.go:13
  odo component creation
  /go/src/github.com/openshift/odo/tests/integration/java_test.go:41
    Should be able to deploy a spring boot uberjar file using openjdk [It]
    /go/src/github.com/openshift/odo/tests/integration/java_test.go:94
     Failed after 90 retries. Content in http://uberjaropenjdk-app-phppkfrfmn.apps.ci-op-dr740dig-f09f4.origin-ci-int-aws.dev.rhcloud.com doesn't include 'HTTP Booster'.
     /go/src/github.com/openshift/odo/tests/helper/helper_http.go:41
------------------------------
 Summarizing 1 Failure:
 [Fail] odoJavaE2e odo component creation [It] Should be able to deploy a spring boot uberjar file using openjdk
/go/src/github.com/openshift/odo/tests/helper/helper_http.go:41
 Ran 4 of 97 Specs in 315.104 seconds
FAIL! -- 3 Passed | 1 Failed | 0 Pending | 93 Skipped

Something up with springboot

/retest

/test integration

/lgtm

/retest


Some issue in start time tracker in project list

Describe the bug

set timer button name should be start timer
close button name should be cancel
project box should be a project name
current status label should be a company name

Screenshots

compare old module


Clicking outside popover toggle popover

components: Popover
reactstrap version ^8.4.1
import method es
react version ^16.13.0
bootstrap version ^4.4.1

What is happening?
toggle prop does not dismiss popover when clicked outside.
What should be happening?
toggle props= dismisses a Modal. I guess the behavior should be consistent with popover?
Bootstrap uses trigger: 'focus'. Can we match this somehow?
Code
https://stackblitz.com/edit/reactstrap-ckciq5?file=Example.js

I tried using a popover last night and have this same problem.  Can't figure it out.
my code is the same as your's but I'm using a function component.
reactstrap version : 8.1.1
react version: 16.12.0
bootstrap version: 4.3.1

Did you try to set trigger="focus", you are missing the trigger param from the popover.
Example with use-case
Although this is just one of the options.
Reactstrap already supports setting a trigger, see the example on reacstrap here:
Popover.propTypes = {
  // space separated list of triggers (e.g. "click hover focus")
  trigger: PropTypes.string,
  //....
}

Ahh yes, he is missing that. I thought I saw it in his code before.  Once you add that his example works as it should. But unfortunately in my case, it does not.  I can copy and paste some code but visually it is not much different. What other information should I provide?

`const CustomOrderNav = (props) => {
const [slabTooltipOpen, setSlabTooltip] = useState(false);

 const toggleSlabTooltip = () => setSlabTooltip(!slabTooltipOpen);

 const [seriesTooltipOpen, setSeriesTooltip] = useState(false);

const toggleSeriesTooltip = () => setSeriesTooltip(!seriesTooltipOpen);

return(
    <React.Fragment>

                Series

            {/*

                Slabs

 */}

                Slabs

            {/*

                Cut

 */}

                Cut

                Finish

                Hardware

        <div className = "custom-order-progress-bar active"

            style = {{transform: `translate(${props.translateAmount})`}}
            >

            <Tooltip trigger = "focus" isOpen = {slabTooltipOpen}   target = "step2-select-slab" >

You Must Select Slab First

            </Tooltip>

            {/* <Tooltip  trigger = "focus"  isOpen = {seriesTooltipOpen}    target="step1-select-series">

You Must Select Serires First

            </Tooltip> */}

            <Popover trigger = 'focus' isOpen = {seriesTooltipOpen} target="step1-select-series">

You Must Select Serires First

            </Popover>

    </React.Fragment>
);

[}`]

I did some playing around and noticed that I have the tooltip/popover triggered by the onClick event inside the div. I looked back over the examples and noticed that it isn't done that way. I don't think there's an issue with the way tooltip/popover is working now with trigger = 'focus'. There's a problem with my code. I replaced the divs with buttons and got rid of the conditional onClick methods and the behavior works as expected but I will have to explore more solutions as this doesn't quite give me the solution I am looking for. Thanks for your time

Did you try to set trigger="focus"? you are missing the trigger prop from the popover
Example with use-case
Although this is just one of the options.
Reactstrap already supports setting a trigger, see the example on reacstrap here:
Popover.propTypes = {
  // space separated list of triggers (e.g. "click hover focus")
  trigger: PropTypes.string,
  //....
}

Example works in Chrome but not Safari (popover doesn't open). So is this a bug?


Wrong color for center location icon FAB on API 21 after fragment change

On API21, on first load of the map fragment the color is correct. After switching to another fragment and returning to the map fragment the icon is white instead of black. Probably some issue with the underlying button state definition.

@cbalster check plz https://github.com/criticalmaps/criticalmaps-android/pull/310

fixed by #310


Batch Template Wizard - First Changes

Critical Changes
Changes

Progress Indicator
Step 1: Template Overview
Step 2: Select Fields

Issues Closed
New Metadata
Deleted Metadata

##lurch: attach W-026943

##lurch: attach W-026901

##lurch: attach W-026900

Note to self -- Before merging, must test 3 tickets to A/Cs:
W-026900 [PASS]
W-026901 [PASS]
W-026943 [PASS]


Feature Request:  is there is any way to make our own Custome storage system.

I have an application that uses Bunnystreem storage and I want to migrate from laravel to Pocetbase but I have this problem with storage because Bunneystreem is not supported how we can make our storage driver

At least for v1 there are no plans for other storage implementations as it will be difficult to maintain.
Support for general file hooks will be added with the ongoing refactoring which eventually to some extend could allow integration with custom storage apis.


Extract instrumentation package

Closes #75

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls


Adding IBM Cloud Private Helm Chart Repository

The IBM/charts repository provides helm charts for IBM and Third Party middleware.

/lgtm


Doesn't work on other environements

When I tried this bot on windows, Linux, and kali I got errors on all 3. On windows it was a failed to create process(windows 10 64 bit, python 3.9.1). Linux was could not find specified file while I was in directory. Kali had issue with python. Should I try using the windows 10 python 3.7.4 environment to see if that works?

@aaqibmehrban

When I tried this bot on windows, Linux, and kali I got errors on all 3. On windows it was a failed to create process(windows 10 64 bit, python 3.9.1). Linux was could not find specified file while I was in directory. Kali had issue with python. Should I try using the windows 10 python 3.7.4 environment to see if that works?

can you share exactly where you getting the issue and when?
do share screenshot.

Hi there,
I ran the script too on Kali Linux and followed the installation steps but appear to be having some problems. Below is my terminal output:

┌──(kali㉿kali)-[~/Desktop/MASCALOGICS-Automatic-Reddit-text-to-Video-Generator-and-youtube-uploader]
└─$ pip install -r requirements.txt
Defaulting to user installation because normal site-packages is not writeable
Collecting absl-py==0.9.0
  Downloading absl-py-0.9.0.tar.gz (104 kB)
     |████████████████████████████████| 104 kB 3.4 MB/s
Collecting Adyen==2.2.0
  Downloading Adyen-2.2.0.tar.gz (12 kB)
Requirement already satisfied: aiodns==2.0.0 in /usr/lib/python3/dist-packages (from -r requirements.txt (line 3)) (2.0.0)
Collecting aiohttp==3.6.2
  Downloading aiohttp-3.6.2-py3-none-any.whl (441 kB)
     |████████████████████████████████| 441 kB 33.7 MB/s
Collecting aiohttp-socks==0.4.1
  Downloading aiohttp_socks-0.4.1-py3-none-any.whl (17 kB)
Collecting alembic==1.3.2
  Downloading alembic-1.3.2.tar.gz (1.1 MB)
     |████████████████████████████████| 1.1 MB 4.3 MB/s
Collecting altgraph==0.16.1
  Downloading altgraph-0.16.1-py2.py3-none-any.whl (20 kB)
Collecting anticaptchaofficial==1.0.23
  Downloading anticaptchaofficial-1.0.23-py3-none-any.whl (8.4 kB)
Collecting app==0.0.1
  Downloading app-0.0.1.zip (2.2 kB)
Collecting appdirs==1.4.3
  Downloading appdirs-1.4.3-py2.py3-none-any.whl (12 kB)
Collecting arabic-reshaper==2.1.0
  Downloading arabic_reshaper-2.1.0.tar.gz (18 kB)
Collecting asgiref==3.2.3
  Downloading asgiref-3.2.3-py2.py3-none-any.whl (18 kB)
Collecting asn1crypto==0.24.0
  Downloading asn1crypto-0.24.0-py2.py3-none-any.whl (101 kB)
     |████████████████████████████████| 101 kB 3.6 MB/s
Collecting astor==0.8.1
  Downloading astor-0.8.1-py2.py3-none-any.whl (27 kB)
Requirement already satisfied: async-timeout==3.0.1 in /usr/lib/python3/dist-packages (from -r requirements.txt (line 15)) (3.0.1)
Collecting atomicwrites==1.3.0
  Downloading atomicwrites-1.3.0-py2.py3-none-any.whl (5.9 kB)
Collecting attrs==19.3.0
  Downloading attrs-19.3.0-py2.py3-none-any.whl (39 kB)
Collecting auto-py-to-exe==2.6.5
  Downloading auto_py_to_exe-2.6.5-py2.py3-none-any.whl (64 kB)
     |████████████████████████████████| 64 kB 2.7 MB/s
Collecting Automat==0.8.0
  Downloading Automat-0.8.0-py2.py3-none-any.whl (31 kB)
Collecting backcall==0.1.0
  Downloading backcall-0.1.0.zip (11 kB)
Collecting beautifulsoup4==4.8.2
  Downloading beautifulsoup4-4.8.2-py3-none-any.whl (106 kB)
     |████████████████████████████████| 106 kB 5.1 MB/s
Collecting bleach==3.1.0
  Downloading bleach-3.1.0-py2.py3-none-any.whl (157 kB)
     |████████████████████████████████| 157 kB 4.6 MB/s
Collecting bottle==0.12.17
  Downloading bottle-0.12.17-py3-none-any.whl (89 kB)
     |████████████████████████████████| 89 kB 3.6 MB/s
Collecting bottle-websocket==0.2.9
  Downloading bottle-websocket-0.2.9.tar.gz (2.0 kB)
Collecting browser-cookie3==0.11.1
  Downloading browser-cookie3-0.11.1.tar.gz (7.6 kB)
Collecting browsercookie==0.7.6
  Downloading browsercookie-0.7.6.tar.gz (6.0 kB)
Requirement already satisfied: bs4==0.0.1 in /home/kali/.local/lib/python3.9/site-packages (from -r requirements.txt (line 27)) (0.0.1)
ERROR: Could not find a version that satisfies the requirement build==1.0.2 (from versions: 0.0.1, 0.0.2, <IP_ADDRESS>, 0.0.4, 0.1.0, 0.2.0, 0.2.1, 0.3.0, 0.3.1, 0.3.1.post1, 0.4.0, 0.5.0, 0.5.1, 0.6.0, 0.6.1)
ERROR: No matching distribution found for build==1.0.2

┌──(kali㉿kali)-[~/Desktop/MASCALOGICS-Automatic-Reddit-text-to-Video-Generator-and-youtube-uploader]
└─$ python3 run.py                                                                                1 ⨯
Traceback (most recent call last):
  File "/home/kali/Desktop/MASCALOGICS-Automatic-Reddit-text-to-Video-Generator-and-youtube-uploader/run.py", line 3, in <module>
    import pyttsx3
ModuleNotFoundError: No module named 'pyttsx3'

┌──(kali㉿kali)-[~/Desktop/MASCALOGICS-Automatic-Reddit-text-to-Video-Generator-and-youtube-uploader]
└─$ pip install pyttsx3                                                                                1 ⨯
Defaulting to user installation because normal site-packages is not writeable
Collecting pyttsx3
  Downloading pyttsx3-2.90-py3-none-any.whl (39 kB)
Installing collected packages: pyttsx3
Successfully installed pyttsx3-2.90

┌──(kali㉿kali)-[~/Desktop/MASCALOGICS-Automatic-Reddit-text-to-Video-Generator-and-youtube-uploader]
└─$ python3 run.py
Traceback (most recent call last):
  File "/home/kali/Desktop/MASCALOGICS-Automatic-Reddit-text-to-Video-Generator-and-youtube-uploader/run.py", line 4, in <module>
    import comtypes.client
ModuleNotFoundError: No module named 'comtypes'

┌──(kali㉿kali)-[~/Desktop/MASCALOGICS-Automatic-Reddit-text-to-Video-Generator-and-youtube-uploader]
└─$ pip install comtypes                                                                                1 ⨯
Defaulting to user installation because normal site-packages is not writeable
WARNING: Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<pip._vendor.urllib3.connection.HTTPSConnection object at 0x7f04745d1b80>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution')': /simple/comtypes/
Collecting comtypes
  Downloading comtypes-1.1.10.tar.gz (145 kB)
     |████████████████████████████████| 145 kB 2.7 MB/s
Building wheels for collected packages: comtypes
  Building wheel for comtypes (setup.py) ... done
  Created wheel for comtypes: filename=comtypes-1.1.10-py3-none-any.whl size=164937 sha256=05d961dc35183d203639a1f1a892698a1a3290c4f031ee73c84c1d3d62e45e82
  Stored in directory: /home/kali/.cache/pip/wheels/76/c4/bb/651111ab6c5e31ad6a1ff3022bfdbef5a4e88bfe568150870c
Successfully built comtypes
Installing collected packages: comtypes
Successfully installed comtypes-1.1.10

┌──(kali㉿kali)-[~/Desktop/MASCALOGICS-Automatic-Reddit-text-to-Video-Generator-and-youtube-uploader]
└─$ python3 run.py
Traceback (most recent call last):
  File "/home/kali/Desktop/MASCALOGICS-Automatic-Reddit-text-to-Video-Generator-and-youtube-uploader/run.py", line 4, in <module>
    import comtypes.client
  File "/home/kali/.local/lib/python3.9/site-packages/comtypes/__init__.py", line 23, in <module>
    from _ctypes import COMError
ImportError: cannot import name 'COMError' from '_ctypes' (/usr/lib/python3.9/lib-dynload/_ctypes.cpython-39-x86_64-linux-gnu.so)

┌──(kali㉿kali)-[~/Desktop/MASCALOGICS-Automatic-Reddit-text-to-Video-Generator-and-youtube-uploader]
└─$ python3 run.py                                                                                1 ⨯
Traceback (most recent call last):
  File "/home/kali/Desktop/MASCALOGICS-Automatic-Reddit-text-to-Video-Generator-and-youtube-uploader/run.py", line 4, in <module>
    import comtypes.client
  File "/home/kali/.local/lib/python3.9/site-packages/comtypes/__init__.py", line 23, in <module>
    from _ctypes import COMError
ImportError: cannot import name 'COMError' from '_ctypes' (/usr/lib/python3.9/lib-dynload/_ctypes.cpython-39-x86_64-linux-gnu.so)

┌──(kali㉿kali)-[~/Desktop/MASCALOGICS-Automatic-Reddit-text-to-Video-Generator-and-youtube-uploader]
└─$

Would be grateful for any advice on how to get this up and running.


name is not defined

For v1.2.0, I'm getting this error in my parcel project when I try to build (see logs below).
Repo in question: https://github.com/dashed/albertoleal
However, it works when I try to downgrade to v1.0.3.
This LOC seems to be the issue: https://github.com/sintloer/parcel-plugin-shebang/blob/5a3443e81b3c92ba090ee9f13bd137cafb737926/utils.js#L81

$ yarn build
yarn run v1.12.3
$ parcel build source/cli.js --target=node --out-dir=bin/ --bundle-node-modules --no-source-maps
✨  Built in 2.40s.

bin/cli.js                 181.73 KB    2.27s
bin/avatar.f4ca419f.png     51.03 KB    174ms
🚨  name is not defined
    at getBundles (/path/to/albertoleal/node_modules/parcel-plugin-shebang/utils.js:81:3)
    at Bundler.bundler.on.bundle (/path/to/albertoleal/node_modules/parcel-plugin-shebang/index.js:21:21)
    at emitOne (events.js:116:13)
    at Bundler.emit (events.js:211:7)
    at Bundler.bundle (/path/to/albertoleal/node_modules/parcel-bundler/src/Bundler.js:324:12)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:189:7)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

@dashed, I published 1.2.2 version. The above error should no longer occur.

@sintloer thanks for looking into this. I've tested it out on my project (https://github.com/dashed/albertoleal), but it doesn't seem to work as expected.
It seems that this shebang regex is not reliable: https://github.com/sintloer/parcel-plugin-shebang/blob/1410fc1cefeb35806e09784e80954f524d3a5062/utils.js#L8
since this sequence of characters, #!, can occur in the bundled source code.

@sintloer thanks for looking into this. I've tested it out on my project (https://github.com/dashed/albertoleal), but it doesn't seem to work as expected.
It seems that this shebang regex is not reliable:
parcel-plugin-shebang/utils.js
Line 8 in 1410fc1
const SHEBANG_REGEX = /#!(.) (.)\n/;
since this sequence of characters, #!, can occur in the bundled source code.

@dashed, try with 1.2.3 version.


Unable to connect to world.

I setup my server on port 19137:
package com.tsunderebug.cpe

import net.marfgamer.jraknet.identifier.{Identifier, MCPEIdentifier}
import net.marfgamer.jraknet.server.{RakNetServer, RakNetServerListener}
import net.marfgamer.jraknet.session.RakNetClientSession

import scala.util.Random

object Main {

  var onlineCount = 0
  val uid: Long = new Random().nextLong()

  def main(args: Array[String]): Unit = {
    val server = new RakNetServer(19137, 10, mcpeIdentifier)
    server.setListener(new RakNetServerListener {

      override def onClientConnect(session: RakNetClientSession): Unit = {
        onlineCount += 1
        server.setIdentifier(mcpeIdentifier)
        println("cl")
      }

      override def onClientDisconnect(session: RakNetClientSession, reason: String): Unit = {
        onlineCount -= 1
        server.setIdentifier(mcpeIdentifier)
      }

    })

    server.start()
  }

  def mcpeIdentifier: Identifier = new MCPEIdentifier("Memeoculos", 101, "1.0.3", onlineCount, 10, uid, "Meme World", "Creative")

}

but when I go to the Friends tab, it doesn't show up, and when I go to Servers and add <IP_ADDRESS>, 19137, it doesn't get info from it and when trying to join shows that error.

First off, JRakNet is not an MCPE server, it is only a protocol implementation of the underlying networking library. Second, the reason your server isn't showing up is due to a bug on the Windows 10 edition where it is unable to connect to servers on the same machine. I am not sure what causes this, but it is not related to JRakNet. To see what I'm talking about, launch the server you created and open Minecraft on another device (XBOX One, Android, iOS, another Windows 10 computer than the one the server is running on, etc.)
If you look at the README.md file, it clearly says that JRakNet is a protocol implementation of RakNet, however it was made so people using Java could easily get things up and running to create the server and start implementing MCPE's protocol from there.

The example code doesn't show much and the documentation is minimal.

Have you looked at all in the src/main/test or src/main/test/example folders at all? If so, I apologize that the examples and test code do not provide enough information. Is there anything you would like me to add in terms of example code?
Also, by documentation, do you mean JavaDocs?

I'd like it to be explained that this is not a full fledged MCPE server and a brief guide on how to make one/link to vgwiki's article on the MCPE RakNet protocol. And the JDoc is very simple, often lacking an example.

vgwiki's MCPE documentation is severly outdated

@TsundereBug Like I said before, the README file clearly states that this is for multiplayer software using the RakNet protocol (However, it was originally made for Minecraft so it is optimized for it). On top of that, wikiVG is outdated by over 3 years and should not even be touched. At the moment, we don't even know what the protocol is for 1.2 (Protocol 137) because Microsoft is playing favorites and only handing out the protocol to their partners.
However I will look into the JavaDoc issue and will add more examples as I agree that there are not many examples in the documentation.

hey @MarfGamer I am using your library in my DragonProxy project, I updated to latest version even tried snapshots from jitpack but I can't get a server ping and unable to connect at all. Server handler seems useless because it didn't call the method which I added print-out stuffs.
   public RaknetInterface(DragonProxy proxy, String ip, int port) {
        this.proxy = proxy;
        rakServer = new RakNetServer(port, Integer.MAX_VALUE);
        rakServer.addListener(this);
        sessions = this.proxy.getSessionRegister();
        rakServer.startThreaded();
    }

    public void setBroadcastName(String serverName, int players, int maxPlayers) {
        rakServer.setIdentifier(new MinecraftIdentifier(serverName, Versioning.MINECRAFT_PE_PROTOCOL, Versioning.MINECRAFT_PE_VERSION, players, maxPlayers, new Random().nextLong(), "DragonProxy", "Survival"));
    }


[Feature][ui] Add form-create plug-in into shell node page

Purpose of the pull request

Add form-create plug-in into shell node template
#3682
#3683

Brief change log

Refactor the shell node template: dolphinscheduler-ui/src/js/conf/home/pages/dag/_source/formModel/tasks/shell.vue

Verify this pull request
This pull request is code cleanup without any test coverage.

Codecov Report

Merging #5991 (8cfa14d) into dev (f7e0e9f) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                dev    #5991   +/-   ##
=========================================
  Coverage     46.75%   46.76%
  Complexity     3743     3743
=========================================
  Files           604      604
  Lines         24820    24820
  Branches       2832     2832
=========================================
+ Hits          11605    11607    +2
+ Misses        12074    12070    -4
- Partials       1141     1143    +2

Impacted Files
Coverage Δ

...eduler/server/worker/runner/TaskExecuteThread.java
59.50% <0.00%> (ø)

...ache/dolphinscheduler/server/log/LoggerServer.java
86.95% <0.00%> (+8.69%)
:arrow_up:

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 f7e0e9f...8cfa14d. Read the comment docs.

@break60 PTAL,Thx

@break60 PTAL,Thx
hi, can you review for your students?


Errors in HA with new pre-release

I am getting 2 errors which appear to be coming up frequently (see # of occurrences):
This error originated from a custom integration.

Logger: custom_components.openevse
Source: custom_components/openevse/__init__.py:225
Integration: OpenEVSE
First occurred: 6:06:40 AM (1276 occurrences)
Last logged: 11:19:01 AM

Could not update status for available_current
Could not update status for smoothed_available_current

AND
This error originated from a custom integration.

Logger: custom_components.openevse
Source: custom_components/openevse/__init__.py:241
Integration: OpenEVSE
First occurred: 6:06:40 AM (638 occurrences)
Last logged: 11:19:01 AM

Could not update status for divert_active

Those are technically not errors, I'll adjust the logging on those messages down to info level.

Should be sorted in 2.0.12

I also need to take a look at these, because those fields appear in the documentation of the OpenEVSE Wifi Module but somehow it looks like that they are not being published. I need to understand if these only get published if the divert mode is enabled and active.

Should be sorted in 2.0.12

still getting the divert_active error in 2.0.12:
This error originated from a custom integration.

Logger: custom_components.openevse
Source: custom_components/openevse/__init__.py:241
Integration: OpenEVSE
First occurred: 12:29:36 PM (3 occurrences)
Last logged: 12:30:02 PM

Could not update status for divert_active

Keep in mind that these fields do not exist in the firmware v3 and lower.
You can hide the message by setting the logging for openevse to warning or error.

Keep in mind that these fields do not exist in the firmware v3 and lower.
You can hide the message by setting the logging for openevse to warning or error.

I am on firmware v4.1.0. how does one change logging level for openevse?

I am on firmware v4.1.0. how does one change logging level for openevse?

In your configuration.yaml you'll need something along these settings for the logger:
logger:
  default: warning
  logs:
    custom_components.mail_and_packages: error

In your configuration.yaml you'll need something along these settings for the logger:
logger:
  default: warning
  logs:
    custom_components.mail_and_packages: error

Sorry I didn't notice the binary sensors were still reporting on warning level as well, 2.0.13 should fix this fully now.

@firstof9 I'm getting on v<IP_ADDRESS>P with WiFi version 4.1.2.dev:
2022-05-02 21:00:16 INFO (MainThread) [custom_components.openevse] Could not update status for divert_active
2022-05-02 21:00:16 INFO (MainThread) [custom_components.openevse.binary_sensor] binary_sensor [divert_active] not supported.
2022-05-02 21:00:30 INFO (MainThread) [custom_components.openevse] Could not update status for divert_active
2022-05-02 21:00:30 INFO (MainThread) [custom_components.openevse.binary_sensor] binary_sensor [divert_active] not supported.
2022-05-02 21:00:46 INFO (MainThread) [custom_components.openevse] Could not update status for divert_active
2022-05-02 21:00:46 INFO (MainThread) [custom_components.openevse.binary_sensor] binary_sensor [divert_active] not supported.

Is it supposed to be the case?

yup those are just info, not errors.

Would it be possible to change the logging level to debug? If it's not something I can act one and since it's logged every few seconds, I think it would create a better experience.

I can do this, but I do question why you logging on such a low level?
I believe the default logging for Home Assistant is error

I believe there is actually an issue -- do you want me to open an new issue to track this?
v<IP_ADDRESS>P with WiFi version 4.1.2.dev:
2022-05-12 23:48:00 ERROR (MainThread) [openevsehttp] Invalid value for divertmode: normal
2022-05-12 23:48:00 WARNING (MainThread) [custom_components.openevse.select] Could not set status for openevse Divert Mode error:

Yes please.
Thank you.


Upgrades from v0.4.5 to v0.5.0 not working

RancherOS Version: (ros os version)
Custom build of v0.5.0 branch (os3)
Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.)
Virtual Box
Attempt to upgrade from v0.4.5 to v0.5.0 branch causes panic.

Verified working with custom build of v0.5.0 branch (os6)


enhanceSelectElement always sets autoselect to true

opts.autoselect = opts.autoselect || true
should be
if (opts.autoselect === undefined) opts.autoselect = true
I think

Yep, good catch!

just looking into this - why are we setting autoselect to true in enhanceSelectElement?

@joelanman it makes it more similar to a real <select> in that it doesn't allow you to get away as easily without picking an option.

not sure about that - most selects would have '-choose an option-' first value. Also this means you get different behaviour just because you're enhancing a select - thats a bit unexpected.

That's a fair point and I do agree with it, @edwardhorsford may remember the exact reasoning (we were also considering making autocomplete: true the real default at one point)

Reasoning:
We're setting the default to the most sensible option in each case. For enhanced inputs we need users to be able to type anything, and not have to pick from the list. For selects, users must pick from the list. So the most sensible option is different for each.
The challenge here is to have consistent options, but also an easy way for us to provide 'recommended' sets of options. So far we've done this by having defaults that offer our recommended options.
My concern with changing this default is that we'll see most users of this component not using autoselect- a feature we've found to be very beneficial. I'm very open to alternate suggestions that achieve this though, and agree the inconsistency is awkward.

this was fixed by #167


MenuItemImpl implements implements Serializable and MenuItem, but the last already extends Serializable

I think the title self-explains it.

Does not seem to be a bug


Added Tailwindcsss ,Vue and Svelte structured properly

Added 3 Courses (locale: BN)
Add 10 Bangla Courses | Remove   1 Book   Because that website does not exist right now
Bangla resources
Description
Add 10 Bangla Courses

Vue.JS
Tailwindcsss
Svelte

Why is this valuable (or not)?
In Bengal resources, that's not enough, so I added the 2most helpful free resources And
How do we know it's really free?
These resources are from YouTube for its totally free
Checklist:

[x] Read our contributing guidelines
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction)

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

This is a judgment call. We could use "Learn With Sumit" or "Sumit Saha". Depends on how the creator uses the name. If for example, emails from the creator are signed "Learn with Sumit", then that's an indication of how the creator wants to be identified, especially if "Learn With Sumit" is more than just Sumit. If all the videos start with "Hello I'm Sumit Suha", then clearly the creator is "Sumit Suha". If a user needs to work to discover the "Suha" part, then "Learn with Sumit" more clearly identifies the creator for potential students. I doubt the verbose "Learn with Sumit - LWS - Bangladesh" is an appropriate thing to list as the creator. So @Mdkawsarislam2002 please consider the suggestions and use the phrase that you think best identifies the creator, and thank you for bearing with us!

Thanks.
Now it's ok, I think?

Checking the CONTRIBUTING file
https://github.com/EbookFoundation/free-programming-books/blob/a7558f56b8ffcd27571a8ff3fa4d6d11538a0755/docs/CONTRIBUTING.md#L163
and course listings there are no short youtube links (youtu.be)
I think it could be discussed whether to accept official short links or not.

Conflicts are with indentation made in #6988 merge

Some Vue & Svelte was lost. Maybe due to some bad merge. I'll review it later as last step previous to merge this PR

Some initial Vue & Svelte was lost. Maybe due to some bad merge in commits. I'll review it later as last step previous to merge this PR

This resources was removed due to haven't the minimum length of 1h


chore: patch std4#89

[x] depends on: leanprover/std4#89

@JamesGallicchio, I incorporated this patch into https://github.com/leanprover-community/mathlib4/pull/8888 which updates mathlib. Happy to use either patch, just did not want to hijack your PR.

@JamesGallicchio I updated this PR to mathlib master, after https://github.com/leanprover/std4/pull/89 has been merged into std4. Following @eric-wieser's advice of keeping PRs small, we probably should merge this PR as it covers exactly one std4 PR. Mathlib has been updated up to this PR, so this is the next PR in line for updating std4. From my perspective, this is ready to review, but I leave this to @JamesGallicchio who owns this PR.

If someone else wants to take this over, I'm going to be unavailable this weekend.

bors merge

bors merge