Why do you have to call flush all the time ?

First of all, great work! Second, why do I have to call flush to write to file ?
I`m testing this on Windows with the following code:
auto custom_logger = bl::logger::make_custom(
        "Test",                   // logger tag
        bl::level::info,             // log level filter
        bl::logger::default_pattern, // logger pattern
        false,                       // is asynchronous
        //bl::sink::make_stderr(),     // any number of sinks at the end
        //bl::sink::make_stdlog(),     // any number of sinks at the end
        //bl::sink::make_stdout(),     // any number of sinks at the end
        //bl::sink::make_syslog(),     // any number of sinks at the end
        //bl::sink::make_console(),
        bl::sink::make_file(
            test,                 // path to a directory where you want the log files to be stored
            bl::infinite,            // bytes per file
            bl::infinite,            // maximum log files
            true                     // should rotate logs?
        )
    );
    custom_logger->debug("{0}, {}!", "Hello", "debug");
    custom_logger->info("{0}, {}!", "Hello", "info");
    custom_logger->error("{0}, {}!", "Hello", "error");
    custom_logger->warning("{0}, {}!", "Hello", "warning");
    custom_logger->critical("{0}, {}!", "Hello", "critical");
    custom_logger->flush();

No matter what options you will use (even async) it will not write to file until you called flush();. The idea is to write to file whatever you output to console.
Is this expected ?
Further enhancements:

Add an option to customise the file name, eg with time/date ? The program might run 24/7 and you would need logs for each day (preferably)
Add an easier option to change the console output colour ?
Change the timestamp to include date/time/ms/ns ?
Resolve the full path to the desired logs folder automatically ? For example in my test without adding the full path it would not create the log file...

Also I have a couple of questions:

Is this TS ? I plan to use it from multiple threads/thread pool
Is this really cross platform ? Meaning, Linux, Mac etc ?

Thank you!

Hey! Appreciate the feedback.
Thanks for the enhancement suggestions, they all look interesting, and I would love to implement them some day, probably not any time soon tho because I am focused on something else right now.

No matter what options you will use (even async) it will not write to file until you called flush();

Flush is pretty expensive, especially for files, I would imagine it's OK to have it flush for the asynchronous logger every time, perhaps I should add that feature.

Is this TS ? I plan to use it from multiple threads/thread pool

Logging is completely TS, however, changing logger properties (e.g pattern) from multiple threads isn't.

Is this really cross platform ? Meaning, Linux, Mac etc ?

Confirmed on Windows and Linux, cannot say anything about Mac because I don't have one at home to test it on.

Flush is pretty expensive, especially for files, I would imagine it's OK to have it flush for the asynchronous logger every time, perhaps I should add that feature. (atm it does flush on destruction, and when a file is closed/switched)

Well I cannot imagine any logger that would not write to file... eventually ? Calling yourself when to write to file is a bit strange. Also when we say "async" it means immediately, so it would need to write immediately to file ? Or maybe I got it wrong how to use your logger ?
Nevertheless, if you use this for example in 50 threads at once and you have important logs, calling flush from each of them would cause issues ?

Logging is completely TS, however, changing logger properties (e.g pattern) from multiple threads isn't.

That sounds great to me!

Confirmed on Windows and Linux, cannot say anything about Mac because I don't have one at home to test it on.

Great stuff! S**** Apple :)

Well I cannot imagine any logger that would not write to file... eventually ?

Yeah it does write eventually like I said, upon destruction or switching files.

Also when we say "async" it means immediately, so it would need to write immediately to file ?

"async" means asynchronous, not immediate. but I do agree that the async version should flush by itself every time.

Nevertheless, if you use this for example in 50 threads at once and you have important logs, calling flush from each of them would cause issues ?

No, it shouldn't, it would simply call flush 50 times atomically.

Great stuff then, for now I will use it as is until you can update it for a bit 👍

I got a (dumb) question, but quite crucial... how would one use this across the whole application ? Say in threads, different classes etc ?
Would this work ?
std::shared_ptr<bl::logger> m_BLogger;

m_BLogger = bl::logger::make_custom....

Then just add in each class:
 std::shared_ptr<bl::logger> m_BLogger;
Is this safe/ok ?

Yeah, this should work. You could also make it static or something. Also, there is a typedef bl::logger::ptr I suggest you use that instead to be safe in case the actual type changes down the road.

Yeah, this should work. You could also make it static or something. Also, there is a typedef bl::logger::ptr I suggest you use that instead to be safe in case the actual type changes down the road.

Yeah well, my idea does not work and neither your bl::logger::ptr for some reason in another class. Weirdly, it was works in threads.
This is most likely because the class I was trying to use it in, was later instantiated using std::make_shared.
The solution for now was, to define bl::logger::ptr m_BLogger; in the main file, and then extern bl::logger::ptr m_BLogger; in the precompiled header :(


azurelinux-sysinfo: update version for 2.0

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

The earlier PR to backport this package from 3.0-dev used a macro azl that does not exist in 2.0. Updated the version to 2.0-1 for 2.0 branch.
Change Log

Change version to 2.0.

Does this affect the toolchain?

NO
Test Methodology

PR checks pass.
Buddy build: https://dev.azure.com/mariner-org/mariner/_build/results?buildId=561831&view=results

Auto cherry-pick results:

main :x: -> https://github.com/microsoft/azurelinux/pull/8997

Auto cherry-pick pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=561887&view=results


Issue #33: Resolved generation time error on Windows

Add an additional user specified check so example Windows application generated only when user specifies.

Thanks for your contribution! But it doesn't solve actual problem (even though I agree that examples should be off by default). Actual problem is that windows example always links to single header version while it is disabled by default. Can you update your commit and include in examples/win32-example/CMakeLists.txt following instead of direct link to single  header version:
if (${BUILD_SINGLE_HEADER})
    target_link_libraries(embedded_cli_win32 PRIVATE EmbeddedCLI::SingleHeader)
else ()
    target_link_libraries(embedded_cli_win32 PRIVATE EmbeddedCLI::EmbeddedCLI)
endif ()

I've adapted your changes and merged into master. Thanks again for your contribution!


in CK Editor How to Add Equation Editor

in CK Editor How to Add Equation Editor

This is possible as described here.


Connect-ExchangeOnline Fails To Work on SOME tenants

This is very strange and we have been unable to figure out whats different between tenants.  We have tried connecting to tenants with modern auth enabled/legacy auth disabled as well as both enabled.  There doesnt seem to be a connection but when it fails, we receive the following error message:  Import-PSSession : Data returned by the remote Get-FormatData command is not in the expected format.
We can open up 2 separate powershell instances and run Connect-ExchangeOnline against 2 different tenants.  One will work perfectly and the other throws the error above.  We have tried on multiple machines with the same results.  Support said there was nothing they could do and it would "just clear up in a few days."  We are unable to manage certain tenants because of this new condition.  It used to work great!  All modules are updated and even installed from scratch.

After further testing, I've been able to work around the issue.  Perhaps this is by design but it's not clear in the docs if that's the case.  And what's also strange is that it is not consistent across tenants.  When I use Connect-ExchangeOnline using a *.onmicrosoft.com global admin account, the cmdlet fails as per previous post.  However, if I run Connect-ExchangeOnline with a global admin with a *.domain.com global admin account, it works.  This is new behavior.  To work around the *.onmicrosoft.com global admin login, one can use the switch-DelegatedOrganization 'tenantname.onmicrosoft.com' and it will connect.   It's weird that it isn't the same on every tenant.  I can run the command using *.onmicrosoft.com credentials and it works.   It seems like things are in flux.  Where's the flux-capacitor when we need it.   Clarity on this would be super helpful.  Our powershell scripts are failing depending on the tenant.  Thanks guys.

@officedocsbot assign @yogkumgit

This appears to be resolved within the last 8 hours.  Thanks to the team for giving it attention.

@dawander  Thank you very much for the contribution and sharing this explanation. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Cannot delete a user account if it has related (observation) comments

I discovered this while trying to manually remove a user in the admin (see #179). Adjustments needed with on_delete on the models?

Now that #181 is fixed, I was able to delete accounts (after manually removing the two observation comments). It would still be better if that was done automatically, though.

I can very easily change the system so the comments are automatically deleted when we delete a user, but is it a desirable behaviour? We might lose some valuable contents there (Imagine there's an actual discussion taking place for an observation, and half of the comments disappear).
If we want to avoid that, I see two possible options:

Alter a the model (allow NULL in comment/user relationship) so when a user is deleted, is comment is kept (and displayed with something like posted by <deleted user> on ...)
take the habit to actually never delete users, but disable them instead (so they cannot login anymore). I am not sure if this is GDPR-compliant.

Thanks @niconoe to point this aspect.
First of all, I don't think long and interesting discussions will take place here. At least, it never happened and it will be unlikely to happen.
Second, I think that even your first option is not GDPR compliant. It's a right of any user to delete ALL kind of informations he/she provided via its own account. Just to give an idea, the same applies to iNaturalist accounts: see screenshot below taken from the delete webpage related to my account.

As you see, by deleting my account I would remove all my 110 comments as well.
Maybe such kind of page could be shown to the user willing to delete an account? Something more than just "are you sure"?

Update: the initial issue is now solved in #179. Keeping this open to keep the last suggestion (give explicit details to the users about what they're going to delete!)


ALB is not created in aws

I'm getting an error while running below command-
#kubectl logs -n kube-system $(kubectl get po -n kube-system | egrep -o "alb-ingress[a-zA-Z0-9-]+")**
1 :0] kubebuilder/controller "msg"="Reconciler error" "error"="failed to build LoadBalancer configuration due to retrieval of subnets failed to resolve 2 qualified subnets. Subnets must contain the kubernetes.io/cluster/\u003ccluster name\u003e tag with a value of shared or owned and the kubernetes.io/role/internal-elb tag signifying it should be used for ALBs Additionally, there must be at least 2 subnets with unique availability zones as required by ALBs. Either tag subnets to meet this requirement or use the subnets annotation on the ingress resource to explicitly call out what subnets to use for ALB creation. The subnets that did resolve were []"  "Controller"="alb-ingress-controller" "Request"={"Namespace":"2048-game","Name":"2048-ingress"}
#kubectl get ingress/2048-ingress -n 2048-game
NAME           HOSTS   ADDRESS   PORTS   AGE
2048-ingress   *                                     80      81s

Does you subnets have below tags for your subnet :
kubernetes.io/cluster/{cluster_name} : shared/owned
kubernetes.io/role/internal-elb   : 1 (or ``)  (for internal)
kubernetes.io/role/elb  : 1 (or ``) (for internet-facing)

You can use ingress annotation as well.
Reference : https://kubernetes-sigs.github.io/aws-alb-ingress-controller/guide/controller/config/#subnet-auto-discovery


Twitch stream notifications not working (as well as a few other problems)

Basically whenever I use ~tw TwitchUserHere and they start streaming, Nadeko Bot doesn't notify the channel. No idea why it's like that; ever since I updated the bot it's been acting pretty weird
Sometimes ignores some commands, sometimes runs them. It seems random whether it'll run or not
Anyone got a fix for it?
If I have to reinstall the bot from scratch, how do i keep my previous settings/preferences?

Twitch notifications have been fixed for 1.0 release, they broke due to a change in the twitch API.
Keep the /data/ folder and the credentials.json to keep all your settings

Twitch changed their API, it will be fixed in the next release.
Dammit, @fearnlj01 xd


[FLINK-16566][mesos] Change the log level of the launching command and dynamic properties from DEBUG to INFO in Mesos

What is the purpose of the change
Change the log level of the launching command and dynamic properties from DEBUG to INFO in Mesos
Brief change log

Change the log level of the launching command and dynamic properties from DEBUG to INFO in Mesos

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.

[ERROR] src/main/java/org/apache/flink/mesos/entrypoint/MesosTaskExecutorRunner.java:[44,8] (imports) UnusedImports: Unused import: java.util.Map.


Close #3 Create QuestionBox

closes #3

hey @chasm we committed the changes, can you have a look at it again?


Estimated Time like Mihon

Describe your suggested feature
Mihon has a feature of estimated time of every entries where it shows the estimated timing of when the new chapters would be released.  My request is to bring this amazing feature to Yokai too.
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an request for a feature in the original Tachiyomi, I am opening because multiple versions of Yōkai have been released without this feature.
[X] If this is an issue with an extension, or a request for an extension, I should be contacting the extensions repository's maintainer/support for help.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have checked through the app settings for my feature.
[X] I will fill out all of the requested information in this form.

Duplicate of GH-53, it will be added when smart update is added.

Duplicate of GH-53, it will be added when smart update is added as it depends on it.

So will you ppl add smart update?

It is planned


How to use secp256k1

please help me . thanks !!!

what you trying to do? sign message? generate key pairs?


JsonToArgument function fails with valid input

I'm not 100% sure what is going on here, but I have been debugging for quite a while - was hoping to be able to submit a PR, but I hit a deadend.
Using SetArgs(object o) function, the Json conversation works fine (even with Enum), but then when JsonToArgument runs, and the following error happens:
System.ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index

Based on the debugger, it seems that indexes has a Length(Count) of 1 here:
        if (jsonChar[i] == ':'){
            jsonChar[indexes[0]] = ' ';
            jsonChar[indexes[1]] = ' ';
        }

but it isn't clear exactly why. I added a check to make sure index[1] exists before running that second call to jsonChar[indexes[1]], but it results in a malformed query.
I made an isolated example on Repl.it if it is any help: https://repl.it/repls/AbleVerticalMaps
The input to the function is a direct copy of what is received after string json = JsonConvert.SerializeObject(inputObject, new EnumInputConverter());. In my schema I have an enum like AssetType:
enum AssetType {
  DRESSING
  GLOBAL
  MAP
  PLAYERHANDOUT
  TOKEN
  OTHER
}

and then in my local code:
enum AssetType {
  DRESSING
  GLOBAL
  MAP
  PLAYERHANDOUT
  TOKEN
  OTHER
}

I thought the enum might be the problem, because I have other queries that do work just fine. But I don't think AssetType enum is the problem here.
Code from Repl.it if it helps:
using System;
using System.Collections;
using System.Collections.Generic;

class MainClass {
  private static string JsonToArgument(string jsonInput){
    char[] jsonChar = jsonInput.ToCharArray();
    List<int> indexes = new List<int>();
    jsonChar[0] = ' ';
    jsonChar[jsonChar.Length - 1] = ' ';
    for (int i = 0; i < jsonChar.Length; i++){
        if (jsonChar[i] == '\"'){
            if (indexes.Count == 2)
                indexes = new List<int>();
            indexes.Add(i);
        }

        if (jsonChar[i] == ':'){
            jsonChar[indexes[0]] = ' ';
            jsonChar[indexes[1]] = ' ';
        }
    }

    string result = new string(jsonChar);
    return result;
  }

  public static void Main (string[] args) {
    JsonToArgument("{\"input\":{\"adventureId\":null,\"assetSubType\":null,\"assetType\":Map,\"campaignId\":null,\"contentPackageId\":null,\"description\":\"A wonderful place to go!\",\"encounterId\":null,\"name\":\"Abandoned House\",\"originalImageUrl\":\"https://i0.wp.com/www.fantasticmaps.com/wp-content/uploads/2010/06/Attachment-1.jpeg\",\"s3ImageUrl\":\"https://omitted.s3.amazonaws.com/08d7fa90-a716-4083-8fd6-41d21e317fe3/Attachment-1.47c282ef-8c42-4c29-8321-80360a2eaa36.jpeg\",\"userId\":\"08d7fa90-a716-4083-8fd6-41d21e317fe3\"}}");
  }
}

I believe that the issue is something to do with the URL strings. It seems if you remove the URL strings, the function will run correctly. Maybe the URL needs to be escaped in some why before being sent in? If you change it to:
    JsonToArgument("{\"input\":{\"description\":\"A wonderful place to die!\",\"name\":\"Abandoned House\",\"userId\":\"08d7fa90-a716-4083-8fd6-41d21e317fe3\"}}");

The function stops failing to parse indexes.

Oh, I see now. When value is like "http://" or "https://", it is catching that semi-colon and treating it as though it is part of the parseable input.
In my case, I can just omit the protocol (http/https) but I do think this function will need to be made more sophisticated to ensure that the ":" colon it is matching on is legitimate and not actually part of the string value.

hi, i apologize for just seeing your issue.
also, thank you for pointing it out to me and doing proper debugging. i understand the problem clearly because of you and it's obvious i didn't consider this edge case. are you working on a PR to fix it or should i take it into my own hands?

Hello, thanks for getting back to me.
I actually spent some time trying to solve this but I just couldn't quite figure out the internals of this parsing engine and how to fix it. I am not sure that it is a good long-term solution to do something like "Look for protocol matches when you see : like file://,http://,https://,ws:// and skip if you do" - this seems a bit amateur to me. I think the wiser option is to modify that function I mentioned above to do some different lexical parsing, like
WHEN find( colon, i.e. : ), parseUntilNext( next escaped open quote i.e. /" )
But if I am being honest, I am not really sure what your code there in that function is doing and C# is not my primary language (although admittedly, that particular function isn't really doing anything language-specific). If this is something you have a clear idea how to implement, it will be more reliable in your hands. Also not to sound pushy, but if you could grab a pot of coffee and sit down and just go through the code adding comments, I think it would go a long way to getting some contributions :) the only comments in the whole source are ToDos
Also, as a side note: I am trying to create a GUI-free update to this project because the GUI is just not reliable or intuitive (it is very good for free :D). I have this operational now but it is quite hacky. I am working tireless hours on a student project for my grad degree right now so I am struggling to make this a priority, but it is on my mind and I hope to get something together for this.

Any news on this ? I tried to use JsonB parameters and I have the same issue


Warning about Node version

Hi
I had a different extension but switched to yours since it's maintained but unfortunately I get the same warning in the pipeline for this extension.
Is this something that you would like to look into?
##[warning]Task 'Pull Request Comment' version 1 (PullRequestComment@1) is dependent on a Node version (10) that is end-of-life. Contact the extension owner for an updated version of the task. Task maintainers should review Node upgrade guidance: https://aka.ms/node-runner-guidance
Best Regards
Pontus

Well spotted. I'll increase the Node version in next release later this week.

@klamfeldt new version 1.3.0 of the task released with Node.js 16 & 20 support.


[feat] Allow $schema field in tauri.conf.json

Having a schema for tauri.conf.json would be very helpful.
I made my own schema file based on your documentation and I reference that schema using the $schema field. But your parser doesn't accept the field. As a workaround, I customized the CLI in config.rs and removed deny_unknown_fields to make it work.
I later found out that you do generate a schema.json file from config.rs code automatically. It doesn't have very good enum support right now IMO, but I recommend you make that schema file available and use it in the code examples.

We validate the schema on our CLI. Why do you need the $schema field? Are you performing the validation on your end?

My IDE (Webstorm) uses it, so I get intellisense/code-completion. I'm also used to these features when editing angular.json which is the main configuration file in Angular. At the top of the angular.json file we have this:
"$schema"<EMAIL_ADDRESS>

Ahh makes sense, we'll implement this :)

I think pointing to an URL would work too 🤔 having the schema locally would be nicer, but only helps people who use the cli from npm

so like, where is the url? lol

schema is released to GitHub releases on each release of tauri-build crate, here is one of them https://github.com/tauri-apps/tauri/releases/download/tauri-build-v2.0.0-beta.13/schema.json
schema is also distributed with the NPM cli package so you can use a local path to it.

Nice. Thanks!


Automated cherry pick of #4228: mcclient: proxy setting aware by using httputils' transport

Cherry pick of #4228 on release/2.12.
#4228: mcclient: proxy setting aware by using httputils' transport

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.12@e4d2d67). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             release/2.12   #4230   +/-   ##
==============================================
  Coverage                ?   6.86%
==============================================
  Files                   ?     501
  Lines                   ?   85073
  Branches                ?       0
==============================================
  Hits                    ?    5837
  Misses                  ?   78742
  Partials                ?     494

Flag
Coverage Δ

#aFlag
6.86% <ø> (?)

Impacted Files
Coverage Δ

pkg/util/httputils/httputils.go
40.17% <ø> (ø)

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 e4d2d67...8999868. Read the comment docs.

/lgtm

/approve


What's the status of this project?

Hi all,
what's the status of this project?
There haven't been any commits/merges since 2014 so I wonder is this still live project or you guy gave up on it?

Mocha & Chai

Looking at the log of issues, it is clear this project has been abandoned.

Jasmine BDD itself offers good node.js support now in my opinion

https://github.com/jasmine/jasmine-npm

I was added as a collaborator to take over maintenance for packages that really need it. I just documented the project status to be "maintenance mode". It is recommended to use jasmine or jasmine-npm package whenever possible. Closing now.


perlPackages.XMLParser: Work around cross-compilation regression

Since 2.44_01, the behaviour for check_lib in their Makefile.PL has
been "fixed" to fail when the assert_lib function fails to build the
test.

https://github.com/toddr/XML-Parser/commit/2bc1e90c04220a04a0b40d1047d464bc55f56210

Now, this wouldn't be so bad, since it's good to actually test what
stuff is being compiled against.
Except that something is wonky with the cross-compilation build-time
Perl.
Undefined subroutine &File::Temp::mktemp called at inc/Devel/CheckLib.pm line 236.

As far as I know, this is a built-in function from Perl.

https://perldoc.perl.org/File/Temp.html

Though, something else is wrong with Checklib.pm. Side-stepping the
issue by (eww) shelling out to mktemp, we get these errors:

/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-armv7l-unknown-linux-gnueabihf-binutils-2.31.1/bin/armv7l-unknown-linux-gnueabihf-ld: assertlib_src1_0.553056903257133: file not recognized: file truncated
collect2: error: ld returned 1 exit status
 -I/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-expat-2.2.8-armv7l-unknown-linux-gnueabihf-dev/include -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-expat-2.2.8-armv7l-unknown-linux-gnueabihf/lib -lexpat
/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-armv7l-unknown-linux-gnueabihf-binutils-2.31.1/bin/armv7l-unknown-linux-gnueabihf-ld: assertlib_src2_0.262169388446154: file not recognized: file truncated
collect2: error: ld returned 1 exit status
Can't link/include C library 'expat.h', 'expat', aborting.

Meanwhile, the actual build, while building the library, seemingly has
no issues building using those paths. ¯\_(ツ)_/¯

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @volth as de facto perl peep
cc anyone handy enough with perl :(
cc @matthewbauer @Ericson2314  who could have insight with cross-compilation
cc @flokli as he was doing cross-compilation and maybe had the same issue.

It is already "fixed" on staging by downgrading to 2.44 (https://github.com/NixOS/nixpkgs/issues/73011)

I think it would be best if we can go forward with the update. Though I would prefer not using a workaround and fixing the upstream software, but the issue is way above my knowledge on how to fix upstream. Additionally, it seems that the implementation of CheckLib.pm (a bundled-in external component) is likely to blame and likely needs a revamp.

We also have HTTP::Daemon stuck to 6.01 because newer versions fail to cross-compile.

this broke the build in staging


Bug

apparently the addon conflicts with another addon
Bagnon Boe.txt

This is the same bug as on our itemlevel plugin. They changed something in the backend on the most recent version, and it's affecting all the plugins it seems. I'll look into it.
It's not a conflict, btw. "WildPants" is just the name of the libraries Bagnon is using to run certain things.

It works like issue with Bagnon_ItemLevel #3 or you can use my lua

Fixed


feat: add kettle11/devserver

#18103 kettle11/devserver: HTTPS server for local development. Implemented in Rust
$ aqua g -i kettle11/devserver

How to confirm if this package works well
Reviewers aren't necessarily familiar with this package, so please describe how to confirm if this package works well.
Please confirm if this package works well yourself as much as possible.
Command and output
$ cat index.html
Hello World
$ devserver &
[1] 4756
$
Serving [/home/test/aqua-registry/test/] at [ https://localhost:8080 ] or [ http://localhost:8080 ]
Automatic reloading is enabled!
Stop with Ctrl+C

$ curl localhost:8080
Hello World
<script>
    // This code is inserted by devserver to enable reloading.
    const socket = new WebSocket("ws://" + window.location.hostname + ":8129");
    socket.addEventListener('open', function (event) { console.log("Reloading enabled!"); });
    socket.addEventListener('message', function (event) { location.reload(); });
$

If files such as configuration file are needed, please share them.
none.
Reference

https://github.com/kettle11/devserver/releases

Thank you!

v4.101.0 is out 🎉
https://github.com/aquaproj/aqua-registry/releases/tag/v4.101.0


Add isInstalled to Module

side note/not blocking: since we're starting to use this more, might be nice to lift this into Module as a helper function e.g. isInstalled() or isInstalled(module) and requireNotInstalled()

Originally posted by @holic in https://github.com/latticexyz/mud/pull/1789#discussion_r1378758582

done in https://github.com/latticexyz/mud/pull/2056


feat: schema loader

Adds SchemaLoader to load schema over HTTP, Fs Path
https://github.com/Permify/permify/assets/62795688/decb9cef-637f-4deb-af70-523efb425bba
/claim #840
/closes #840

Hello @neo773 , thank you for your contribution. Could you please write the unit tests as a final step?

@tolgaOzen
Added the unit tests, also you might wanna change the load from HTTP URL to a something owned by you currently it's pointing to a gist by me.

Can you also update the commit names according to conventional commits?

@tolgaOzen
Shall I make a new PR with all commits merged into one?
I'm not sure how to change the commit messages for multiple commits as I've never done it.

@tolgaOzen
Closing this as I made a new PR https://github.com/Permify/permify/pull/849
I failed miserably at trying to rename the commit messages


Setup for project

Hi @vegadelalyra how are you? could you explain more about this requirement:

webpack-admon init : automatically runs npm i "all typical dependencies for a typical website project (let's say @babel/core babel loader copy-webpack-plugin html-webpack-plugin webpack webpack-ci webpack-dev-server css-loader styles-loader, etc)" and creates || updates our webpack.config.js file with the typical configurations for each of these dependencies/loaders/plugins

What are the typical configurations for each dependency? Could you explain these configurations?

Greetings, Walter! I'm good on my end. Hope the same for you.
The main goal of this package would be to save time implementing webpack to
our projects.
We would define default configurations for a list of the most commonly used
dependencies in web development (we could start with five, in any order:
1-babel, 2-copy-plugin, 3-webpack-cli, 4-cssMinimizerPlugin 5-html-webpack,
for example).
How is each configuration of each dependency going to be? The most general
possible, capable to work with any project. We can just go to the
documentation of each package and read its usage section, just keep in mind
that we still want users to have a window to customize their webpacks.
For example:
For babel. If we google "how to use babel with webpack", the first page is
going to be the webpack-loader documentation
https://webpack.js.org/loaders/babel-loader/
There, we will get the install and usage process.
[image: image.png]
If we run, for example, npm-admon babel (or any similar command, I'd
like to tolerate a typo grade of 1 or 2, we could do that with  Levenshtein
https://www.npmjs.com/package/fast-levenshtein), then our package
webpack-admon would be running something like:
-Check if package.json exists.
If it doesn't, run the npm init -y command && npm i -D webpack && touch
webpack.config.js
if it does, disable this checker. Check if webpack is installed ? install :
disable checker.
-Check if babel is already installed
if it doesn't, run the...
npm install -D babel-loader @babel/core @babel/preset-env
...command and add to the webpack.config.dev this: the "typical
configuration" that I'm talking about, the basic one, the required for the
dependency to work, the exposed at the dependency's documentation. Then the
user can change whatever he wants.
module: {
rules: [
{
test: /.m?js$/,
exclude: /(node_modules|bower_components)/,
use: {
loader: 'babel-loader',
options: {
presets: @.***/preset-env']
}
}
}
]}
...And so on with other dependencies, we could start with the most common
ones, the ones that I cited or indicated at the beginning of the webpack
documentation.
If Babel is part of our webpack-admon custom dependencies list (which, by
default I would like it to be), then it's going to be installed with the
webpack-admon init command and the same is true for the rest of the "list".
webpack-admon would come with a default list, let's start with 3
dependencies and then we will see... User can customize his
webpack-admon list with webpack-admon custom command and the usar can
also define his own custom settings for any dependency with webpack-admon
config  (it would prompt a .json file vscode  window,
for example and we can work with templates) and all of the setting up
webpack process would be simply reduce to a few command lines (most of the
cases).
And that's my idea, pretty much.
Hope I make sense... I want to clarify that I'm starting off in all of this
programming stuff (and I'm loving it), so sorry if sometimes I sound rave
or vague.
And I really appreciate your interest !
El lun, 12 dic 2022 a la(s) 16:16, Walter Lensinas @.***)
escribió:

Hi @vegadelalyra https://github.com/vegadelalyra how are you? could you
explain more about this requirement:
webpack-admon init : automatically runs npm i "all typical dependencies
for a typical website project (let's say @babel/core babel loader
copy-webpack-plugin html-webpack-plugin webpack webpack-ci
webpack-dev-server css-loader styles-loader, etc)" and creates || updates
our webpack.config.js file with the typical configurations for each of
these dependencies/loaders/plugins
What are the typical configurations for each dependency? Could you explain
these configurations?
—
Reply to this email directly, view it on GitHub
https://github.com/vegadelalyra/webpack-admon-project/issues/1, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ASM42RD7HF3AFY6HXLLYX3DWM6I3PANCNFSM6AAAAAAS4NL3UQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Stratify schools

Stratify schools in Hanoi into different levels...
Some can afford the formal natural sciences schools and some have less to invest.
cf.  investment risk willingness

Now have public and private schools


Added respawn logic for player.gd

Added logic to respawn player on death. All code is in Player.gd, there is also commented code for testing. There is new loaded UI element called death_popup_instance which is popup with text "You died" and 10 seconds countdown timer. UI panel is called DeathPopup and is under GUI on player, it contains 2 labels for text, countdown time and a timer.
issue: https://github.com/jonathaneeckhout/jdungeon/issues/20
UI looks like this

@Noazs #63 is merged and should allow you to have some monsters (the tree guys) that will actually attack and eventually kill the player. For testing purposes, you may want to lower the player's max hp.

Tested it by linking an input event that hurts the player with 1000 damage.
Going to merge.
Thanks for your contribution!


RedisConnectionException: SlaveConnectionPool no available Redis entries. Master entry host

We are using the Redisson Client 3.20.1 to connect to Elastic Cache Cluster. When we performed the service update on the elastic cache then we started to face following error at redisson side.
We have our elastic cache cluster within VPC, so AWS makes sure that DNS and it's IP of nodes will not be changed.
https://docs.aws.amazon.com/AmazonElastiCache/latest/mem-ug/ClientConfig.DNS.html
Error:
o.r.c.RedisConnectionException: SlaveConnectionPool no available Redis entries. Master entry host:  Disconnected hosts:
at o.r.c.p.ConnectionPool.get(ConnectionPool.java:211)
at o.r.c.p.SlaveConnectionPool.get(SlaveConnectionPool.java:30)
at o.r.c.b.LoadBalancerManager.nextConnection(LoadBalancerManager.java:315)
at o.r.c.MasterSlaveEntry.connectionReadOp(MasterSlaveEntry.java:583)
at o.r.c.RedisExecutor.connectionReadOp(RedisExecutor.java:745)
at o.r.c.RedisExecutor.getConnection(RedisExecutor.java:669)
at o.r.c.RedisExecutor.execute(RedisExecutor.java:128)
at o.r.c.RedisExecutor$1.run(RedisExecutor.java:317)
at i.n.u.HashedWheelTimer$HashedWheelTimeout.run(HashedWheelTimer.java:715)
at i.n.u.c.ImmediateExecutor.execute(ImmediateExecutor.java:34)
at i.n.u.HashedWheelTimer$HashedWheelTimeout.expire(HashedWheelTimer.java:703)
at i.n.u.HashedWheelTimer$HashedWheelBucket.expireTimeouts(HashedWheelTimer.java:790)
at i.n.u.HashedWheelTimer$Worker.run(HashedWheelTimer.java:503)
at i.n.u.c.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)

@mrniko The above document I shared talks about Memcached but we are using redis so IP might get changed. What configuration changes do you recommend so whenever there is IP changes then Redisson will resolve it automatically.

I can see inside the logs that IP got changed from IP1 to IP2.

I can see the below logs where Redisson was able to resolve the new IP but was not able to get the data when calling get a call to Redis.
o.r.c.RedisConnectionException: SlaveConnectionPool no available Redis entries. Master entry host: point to new IP, Disconnected hosts: [point to older IP].
I assume the get call was using the same older connection not new one. So as per my understanding, the possible solution could be to catch this exception and rebuild the connection and use it again, or any way to set this up using the config itself?

try to increase retryAttempts parameter to overcome errors during failover.

@mrniko What is the default value for retryAttempts? And the exception we are facing says that connection was not available in the pool, so do you think increasing the retryattempts will resolve this connection issue?

At our application side, we manually set up the master connection and slave ones. So I am little surprised how the retryAttempts will resolve this issue. We faced this issue for a long time, so if retry could help then it would have been resolved by the time but restarting the application helped us in this case. Curious to know from your side.

@mrniko One question: If we use masterSlave config then does redisson automatically detect the change of primary or master node and underlying IP address of the node during the failover? Or should we go with replicated ones?

If we use masterSlave config then does redisson automatically detect the change of primary or master node and underlying IP address of the node during the failover?

It will detect ip change of master/slave nodes.

@mrniko More concisely, The client library did react to the updated topology but after 5-7 mins and then later ran into connection pooling limit errors.

I need logs with trace logging level for org.redisson package to understand the cause.

@vishaljain790  hey brother did you solve this issue ?
agar ho gaya ho to bata do bhaai, mera task kaafi din se pending hai, same error tha


Highlit extension breaks the Vue instance

Version
3.2.11
Reproduction link
https://sfc.vuejs.org/
Steps to reproduce
After trying it in Incognito mode, I realized that my issue seems to be with this Chrome extension.

Install the extension: https://chrome.google.com/webstore/detail/highlit/cooahmcpmajfpnanjkdfiejkffphknnm
Go to the SFC site and click the "Version" dropdown.
The following error appears:

TypeError: Cannot read properties of null (reading 'insertBefore')
    at insert (vendor.93237667.js:1)
    at C (vendor.93237667.js:1)
    at S (vendor.93237667.js:1)
    at m (vendor.93237667.js:1)
    at A (vendor.93237667.js:1)
    at $ (vendor.93237667.js:1)
    at q (vendor.93237667.js:1)
    at O (vendor.93237667.js:1)
    at m (vendor.93237667.js:1)
    at N (vendor.93237667.js:1)

What is expected?
I don't think the extension should be affecting the Vue instance. I've used the extension with Vue 2 and haven't had any problems.
What is actually happening?
It seems that the Highlit extension may be modifying the DOM in a way that Vue loses the parent element when updating its children.

Now that I know the problem behind the error, the workaround is to disable the extension while developing or use Incognito mode without the extension.

I don't think the extension should be affecting the Vue instance. I've used the extension with Vue 2 and haven't had any problems.

I reproduced your problem.  Indeed,  ‘highlit’ modified the dom element on the sfc page, causing the optional version of vue in the drop-down box to fail to load.  Vue2 does not have sfc, so it cannot be reproduced in the same scene.  No problem does not mean that ‘highlit’ will definitely not modify the dom in vue2.

It seems that the Highlit extension may be modifying the DOM in a way that Vue loses the parent element when updating its children.
Now that I know the problem behind the error, the workaround is to disable the extension while developing or use Incognito mode without the extension.


Update Contact Us Page || Issue 21 Solve

Title : Update Contact Us Page
Issue No. : #21
Code Stack : Same as Base Code Stack
Close #21

Video

https://github.com/user-attachments/assets/3331be2c-3e06-41fa-9ea9-a2160a86a3d3
Video Summary
->Before

Validation happens only at the submission that causes entering the data again from starting.
No error popup for submission of form with wrong values.
No prior check or error alert.
UI is not good at all.

->After

Validation happens on every key stroke that reduces the error at the input time.
Error popup for submission of form with wrong values.
Prior check or error alert.
UI is good at all.
validation sign (Right icon) on each field.

Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md if its a new page/tech stack
[X] I have gone through the  contributing.md file before contributing

Additional context (Mandatory )
Are you contributing under any Open-source programme?

[X] I'm a GSSOC-EXT contributor
[X] I'm a HACKTOBERFEST contributor

@jeevan10017 please also add hactoberfest label to this


runPCA: svd() function - infinite or missing values in 'x'

Hi there!
I am trying to analyse a WES dataset with the STAARpipeline and as part of that using FastSparseGRM.
First of all, thank you for providing this package.
I came across an issue however, which may very well be an issue with my data.
When running Step4: runPCA, I get the following error:
svd() function - infinite or missing values in 'x' (something similar)
For the code:
init<-svd(H,nv=0)$u
I read this was caused by NA values in the matrix. I therefore replaced all NAs with 0 for both the "H" matrix and "T".
The script worked and I got my PCs, however, all my files are either all "0" now or "inf".
Could someone perhaps let me know where I could have gone wrong? (please let me know what data I should provide)
Thank you very much!
Daniel

I just realised I didn't remove sex chromosomes, which is likely causing the issue. I will rerun and update if the issues persists/is solved. Sorry!

Hi, the problem is because of monomorphic SNPs in the BED file. Please filter the BED file to have only SNPs with MAC>=1, and the problem should be solved.

Hi Rounak,
thank you for your fast response! That helped! In theory monomorphic sites should've been removed as I filtered by MAF, however there were some stragglers. I ended up getting a new error using the WES data - something along the lines of "exceeding buffer size".
I ended up using a different dataset with common variants only from a GSA chip, which as Xihao Li pointed out makes more sense anyway, and everything worked fine.
All the best,
Daniel


Update power_utils.cpp

Added a curve to Heltec v3 voltage calculation, greatly improving accuracy - KI6PSP

can you write me over Telegram at my user (same as here in github) to talk about all the details of this PR ?


Which 14 languages does it support?

Which 14 languages does lunr support?

https://lunrjs.com/guides/language_support.html


audio-recorder: init at 1.7-5

The only issue with this package is that under Gnome 3, without installing any different icon themes but the standard, the icons will be missing. I also had some icons missing under the Arch Linux (AUR) build, so I assume they are rare icons, not provided by most icon themes.

@pSub Thanks for pointing those out to me!


gif

the gif or video in the readme isn't watchable

You're probaby missing the specific codec. It's a webm video.


Scriptable Assets don't show name of asset

I'm using a variation of a script on the Unity Wiki to create assets out of our scriptable objects. When i want to reference one of those assets in a script via the inspector, it shows the name of the scriptable object instead of the name of the asset. This makes it really hard to tell the difference between different references when you have a list of these objects on your class because they all have the same name.
Here is an image of what im seeing in the inspector:
https://s3.amazonaws.com/uploads.hipchat.com/143523/1038333/W12xlMjYMqkHfLp/upload.png
As you can see the Resources says "Resource" instead of the name of the asset (eg. Wood, Metal, ect)
I hope this is clear explanation. Let me know if you need more details.

For some reason this has started working. Not sure why... anyways, carry on.

Closing since it is now working.


[TextServer] Improve ligature cursor handling.

Fix mid-grapheme hit test.
Fix OpenType features property handling, add default features override option.
Enable mid-grapheme cursor by default.
Fixes #55100

There's still a visual issue with the ligature selection, and I'm not sure what's the best way to fix. Since ligature is one glyph, it's always drawn using a single color, and this is not working well with the selection text color:

In case ligatures are not desired for the default font, with this PR it's possible to disable them with the following patch:
diff --git a/scene/resources/default_theme/default_theme.cpp b/scene/resources/default_theme/default_theme.cpp
index a1d76ef352..b3bdbead1c 100644
--- a/scene/resources/default_theme/default_theme.cpp
+++ b/scene/resources/default_theme/default_theme.cpp
@@ -1035,9 +1035,13 @@ void make_default_theme(bool p_hidpi, Ref<Font> p_font) {
 		Ref<Font> dynamic_font;
 		dynamic_font.instantiate();

+		Dictionary ot_overrides;
+		ot_overrides["standard_ligatures"] = 0;
+
 		Ref<FontData> dynamic_font_data;
 		dynamic_font_data.instantiate();
 		dynamic_font_data->set_data_ptr(_font_OpenSans_SemiBold, _font_OpenSans_SemiBold_size);
+		dynamic_font_data->set_opentype_feature_overrides(ot_overrides);
 		dynamic_font->add_data(dynamic_font_data);

 		default_font = dynamic_font;

Thanks!


It is possible to search through Pocket items

What I'd love to see is a way that the pocket items would be added to a master pocket note (which already exists through the command, but not in an actual note).
If the note actually exists, we could use dataview to search for pocket items and even add them to other notes.

What do you mean by a master pocket note?

Are you looking for a way to collect all of your Pocket item notes so it can be addressed via a FROM clause? https://blacksmithgu.github.io/obsidian-dataview/query/queries/#from

I second the need for the ability to search the pocket list within obsidian. @kenjibailly's method is one way of doing it. Otherwise, a search bar above the article list, and maybe some filtering options, would be great!


invalid unordered_map<K, T> key

My ongamemodeinit !

I use latest version.

Try that.
https://github.com/urShadow/Pawn.CMD/issues/23

Nope, it's not work...

@urShadow

Try to register aliases via "alias" macro instead of manual calling PC_RegAlias.

It's work. Thanks you!

Nope, it's not work with alias. it's a same problem!

Can you attach server_log.txt?

Of course
server_log.txt

I think you try to register an alias for command that does not exist.

Yea.. my problem is solved. Thanks you @urShadow !


shifter_classic_components: add new upv roll control to arm_2jnt and leg_2jnt

The request is to have something similar to this example:
https://vimeo.com/258674376

Wondering how that behaves when animated. My guess is that you end up having less keys to deal with but way more back and forward while posing than usual systems. Nice to have.

@jdrese  It is a request from our animSup. Provably I will implementer first in a version 2.0 and later if  the addition works as expected I will consolidate it as an optional feature


Allow cross linking to scripting functions with :ref:func_{name}

Summary
This is a…

[ ] Correction
[x] Addition
[ ] Restructuring
[ ] Minor / simple change (like a typo)
[ ] Other

Reason for the Change
Various places in the documentation mention specific scripting functions. It should be possible to easily cross reference those to the function's documentation.
Description of the Change
Add func_{name} targets to all function documentations, allowing them to be referenced with e.g. :ref:`func_is_complete`
Additional Action Required
Something similar would be great for variables as well, but I'm unsure how to best do this without adding to much redundancy. A completely separate section per variable seems to much. So for now I restricted this to functions only.

Something similar would be great for variables as well, but I'm unsure how to best do this without adding to much redundancy. A completely separate section per variable seems to much.

I agree that a separate page per tag / variable would be excessive, but if you put the .. var_{variable}: bookmarks just ahead of each variable (in the existing files) doesn't the :ref:`var_{variable}`  point to an anchor in the page?


forms/libreoffice/convert Error

Image launch log:
A Docker-powered stateless API for PDF files.
Version: 7.5.2
[SYSTEM] modules: api chromium gc libreoffice logging pdfcpu pdfengines pdftk prometheus qpdf uno uno-pdfengine webhook
[SYSTEM] gc: application started
[SYSTEM] uno: long-running LibreOffice listener ready to start
[SYSTEM] pdfengines: pdfcpu pdftk qpdf uno-pdfengine
[SYSTEM] prometheus: collecting metrics
[SYSTEM] api: server listening on port 8088

When trying to make a request to remote Gotenberg docker:
curl --request POST 'http://<IP_ADDRESS>:8088/forms/libreoffice/convert' --form 'files=@"C:\Users\user\blank.docx"' -o my2.pdf
This happens:
{"level":"error","ts":1653572720.2345355,"logger":"api","msg":"code=404, message=Not Found","trace":"1ea0e53a-f27a-46e4-adec-cce84a7d6be5","remote_ip":"<IP_ADDRESS>","host":"<IP_ADDRESS>:8088","uri":"/forms/libreoffice/convert","method":"POST","path":"/forms/libreoffice/convert","referer":"","user_agent":"curl/7.81.0","status":404,"latency":65412,"latency_human":"65.412µs","bytes_in":13033,"bytes_out":9}
But same request works normaly on localhost:3000

This example works properly:
curl --request POST 'http://<IP_ADDRESS>:8088/forms/chromium/convert/url' --form 'url="https://sparksuite.github.io/simple-html-invoice-template/"' -o my.pdf
Maybe something wrong with libreoffice module?
Trying port 3000 doesn't help.

Hello @hanzzbriant,
Huh weird lol, the URL seems good. Maybe a hidden character?

Problem solved

Same here. @mvlach were you able to solve it?

Guys we were all pranked. The example from off docs has a clear flag to disable LibreOffice routes
My command looked like
docker run --rm -p 8081:80 gotenberg/gotenberg:8 gotenberg --api-port=80 --api-timeout=10s --libreoffice-disable-routes --log-level=debug
--libreoffice-disable-routes ....


toolchain: Fix warning about too few arguments on BUILD_ASSERT()

Commit c408fa88a36a595306e96d9b06422e8d1c051c57 introduced changes
to the BUILD_ASSERT() macro so the MSG argument is optional.
However, in include/toolchain/common.h, the BUILD_ASSERT()
definition has not made the MSG argument as optional which results
in build errors complaing about too few arguments. Fix by adding
some dots there.
This is observed when using XCC (based on Clang 3.9).
Fixes #24008
Signed-off-by: Daniel Leung<EMAIL_ADDRESS>

tested, it fixed #24008.


Moves authomaticallAdjustsScrolls to the base ARComponentVC

so it applies to all subclasses - should fix #370 and #368

👍

Grazie 👍


Test mixed-mode programs (with opted-in and not opted-in libraries)

This is now generally support in our testing systems.


Allow Disabling Actor Name Changes on Inport

Simple request, I'd love a per-world module setting that prevents the name of an actor being changed when using the GCS/GCA importer.
Players frequently list their character names differently in GCS and then they need to be fixed repeatedly on import. I also use special characters in names to force Foundry to not split names on space in the player list, so this feature would be a nice QoL improvement.

Done.   It will be in the next release.

Done.   It will be in the next release.


ValueError: invalid literal for int() with base 10

I keep hitting this error with several books. Any ideas what is wrong?
Using the command python main.py --tts edge
Running on Windows.
2024-05-01 23:14:24 [INFO] Converting chapter 7/22: THE_LIKEOMETER, characters: 48686
2024-05-01 23:14:54 [INFO] Converting chapter 8/22: ULTRASOCIALITY, characters: 30998
Traceback (most recent call last):
  File "C:\Users\dege1\Documents\GitHub\epub_to_audiobook\main.py", line 139, in <module>
    main()
  File "C:\Users\dege1\Documents\GitHub\epub_to_audiobook\main.py", line 135, in main
    AudiobookGenerator(config).run()
  File "C:\Users\dege1\Documents\GitHub\epub_to_audiobook\audiobook_generator\core\audiobook_generator.py", line 101, in run
    tts_provider.text_to_speech(
  File "C:\Users\dege1\Documents\GitHub\epub_to_audiobook\audiobook_generator\tts_providers\edge_tts_provider.py", line 158, in text_to_speech
    asyncio.run(
  File "C:\Users\dege1\AppData\Local\Programs\Python\Python311\Lib\asyncio\runners.py", line 190, in run
    return runner.run(main)
           ^^^^^^^^^^^^^^^^
  File "C:\Users\dege1\AppData\Local\Programs\Python\Python311\Lib\asyncio\runners.py", line 118, in run
    return self._loop.run_until_complete(task)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\dege1\AppData\Local\Programs\Python\Python311\Lib\asyncio\base_events.py", line 653, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "C:\Users\dege1\Documents\GitHub\epub_to_audiobook\audiobook_generator\tts_providers\edge_tts_provider.py", line 101, in save
    await self.chunkify()
  File "C:\Users\dege1\Documents\GitHub\epub_to_audiobook\audiobook_generator\tts_providers\edge_tts_provider.py", line 65, in chunkify
    for pause_time, content in self.parsed:
  File "C:\Users\dege1\Documents\GitHub\epub_to_audiobook\audiobook_generator\tts_providers\edge_tts_provider.py", line 58, in parse_text
    yield int(pause_time), content.strip()
          ^^^^^^^^^^^^^^^
ValueError: invalid literal for int() with base 10: '3 Reciprocity with a Vengeance Zigong asked: “Is there any single word that could guide one’s entire life?” The master said: “Should it not be reciprocity? What you do not wish for yourself, do not d

I also have these warnings before the processing happens, but some books do succeed so I don't think these warnings should actually matter.
C:\Users\dege1\Documents\GitHub\epub_to_audiobook\venv\Lib\site-packages\ebooklib\epub.py:1395: UserWarning: In the future version we will turn default option ignore_ncx to True.
  warnings.warn('In the future version we will turn default option ignore_ncx to True.')
C:\Users\dege1\Documents\GitHub\epub_to_audiobook\venv\Lib\site-packages\ebooklib\epub.py:1423: FutureWarning: This search incorrectly ignores the root element, and will be fixed in a future version.  If you rely on the current behaviour, change it to './/xmlns:rootfile[@media-type]'
  for root_file in tree.findall('//xmlns:rootfile[@media-type]', namespaces={'xmlns': NAMESPACES['CONTAINERNS']}):
C:\Users\dege1\Documents\GitHub\epub_to_audiobook\audiobook_generator\tts_providers\base_tts_provider.py:14: RuntimeWarning: coroutine 'EdgeTTSProvider.validate_config' was never awaited
  self.validate_config()
RuntimeWarning: Enable tracemalloc to get the object allocation traceback
C:\Users\dege1\Documents\GitHub\epub_to_audiobook\venv\Lib\site-packages\bs4\builder\__init__.py:545: XMLParsedAsHTMLWarning: It looks like you're parsing an XML document using an HTML parser. If this really is an HTML document (maybe it's XHTML?), you can ignore or filter this warning. If it's XML, you should know that using an XML parser will be more reliable. To parse this document as XML, make sure you have the lxml package installed, and pass the keyword argument `features="xml"` into the BeautifulSoup constructor.
  warnings.warn(

I found an copyright free example ebup that causes the error. Feel free to use this one to reproduce the issue, or if it is environmental, maybe you will not get the error.
famouspaintings.zip

I managed to fix it.
In the file:
epub_to_audiobook/audiobook_generator/tts_providers/edge_tts_provider.py
I added a try except block around the int parser, and now it doesn't crash:
    def parse_text(self):
        if not "[pause:" in self.text:
            return [(0, self.text)]

        parts = self.text.split("[pause:")
        for part in parts:
            if "]" in part:
                pause_time, content = part.split("]", 1)
                try:
                    yield int(pause_time), content.strip()
                except ValueError:
                    yield 0, content.strip()

            else:
                content = part
                yield 0, content.strip()

Thank you @LarsHLunde, haven't got chance to test dege13's doc. However, it worked perfectly on my books, which are failing before the fix.

The fix from @LarsHLunde works for me. I'm guessing it breaks when the text itself has a ] in it maybe? Something like this could be a fix? (not proper code just expressing the idea)
    match = re.search(r'\[pause: ([^\]]+)\]', self.text)
    if match:
        x_part = match.group(1)
        rest_of_string = s[match.end():]
        # recursively apply this to rest_of_string
    else:
        yield 0, s

Also I'm confused because the comment on the code says
    # This class uses edge_tts to generate text
    # but with pauses for example:- text: 'Hello
    # this is simple text. [pause: 2s] Paused 2s'

But that example would throw an error when it tries to do int('2s'). So maybe I am misunderstanding the code somewhere.

Just made a new release to address issues related with edge tts feature. https://github.com/p0n1/epub_to_audiobook/releases/tag/v0.6.0
It would be great if anyone in this issue to try it out and let me know if it's working.


[Bug] showPasswordVisibilityToggler option doesn't seem to work

Bug report
When I update the showPasswordVisibilityToggler option on config/backpack/theme-tabler.php file nothing happens.
What I did
I updated config/backpack/theme-tabler.php file and set the showPasswordVisibilityToggler to true
What I expected to happen
A show password toggle element should be visible.
What happened
Nothing happened.
What I've already tried to fix it
Checked and searched for showPasswordVisibilityToggler but it didn't appear on my project any other place other than config/backpack/theme-tabler.php file.
I checked on Safari Version 17.4.1 (196<IP_ADDRESS>.14) and Google Chrome Version 124.0.6367.62 (Official Build) (arm64)
Context to replicate the bug - Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
### PHP VERSION:
8.3.6

### PHP EXTENSIONS:
Core, date, libxml, openssl, pcre, sqlite3, zlib, bcmath, bz2, calendar, ctype, curl, dba, dom, hash, FFI, fileinfo, filter, ftp, gd, gmp, json, iconv, SPL, session, intl, standard, ldap, mbstring, mysqlnd, mysqli, pcntl, exif, imap, PDO, pdo_mysql, pdo_pgsql, pdo_sqlite, pgsql, Phar, posix, random, readline, redis, Reflection, imagick, shmop, SimpleXML, soap, sockets, sodium, sysvmsg, sysvsem, sysvshm, tokenizer, xml, xmlreader, xmlwriter, xsl, zip, Zend OPcache

### LARAVEL VERSION:
<IP_ADDRESS>

### BACKPACK PACKAGE VERSIONS:
backpack/activity-log: 2.0.3
backpack/basset: 1.3.2
backpack/crud: 6.7.7
backpack/generators: v4.0.5
backpack/logmanager: v5.0.1
backpack/pro: 2.1.12
backpack/settings: 3.1.1
backpack/theme-tabler: 1.2.8

Hey @altayevrim
It's highly possible that you have the form.blade.php file in your resources/views/vendor/backpack/theme-tabler/auth/login/inc folder, so you are not getting the updates we do on the themes files.
Here is where it lives in theme files: https://github.com/Laravel-Backpack/theme-tabler/blob/98bb65a5064930e372d7115d3463ffc44eb94cc0/resources/views/auth/login/inc/form.blade.php#L32 and there is also some Javascript needed at the top of that file.
I've just tested it and seems to be working as expected:

I will be closing this as I don't think there is any issue here form our side, if I am wrong, and I am a lot of times, please re-open or ping me.
Cheers

Hi, I fixed it. Somehow even though I created the project freshly 2-3 days ago. I needed to run composer update to get the update again. Also there is only menu_items.blade.php on the resources/views/vendor/backpack/* directory.
I thought the ** options.showPasswordVisibilityToggler** option would change all password fields, including the CRUD and profile. It seems it only changes the login form.


why is ghost faster ?

Is there a post which explains what Ghost does to make it faster than any other blogging stuff out there ?
what measures are taken to not load areas of the site that aren't even required for 90% of the page visitors ?

Hi @jpca999, the best place to ask these questions is over on our slack team. We use Github solely for bug tracking and on-going feature development so we avoid support request and general questions here to keep the noise down. Thanks!


Clicking hyphenated symbols, and 4 #'s in block comments

Problems this resolves:

"When I double click on a hyphenated symbol in alicorn source code, it highlights a single word out to the hyphens instead of the entire symbol, which makes the feature a lot less useful. I would like it to instead highlight out to spaces."
"ctrl-/ still thinks block comments start with # but the correct thing is ####"

Previously double-clicking bar in foo-bar-baz would only highlight bar. Now it selects the entire string.
Single clicking also highlights the symbol properly too.
Previously block comment with ctrl-/ did:
foo
bar

->
#\tfoo
\tbar

now it does:
->
####foo
\tbar

This does assume tab size is 4, If someone wanted to make a version to do:
####
\tfoo
\tbar

From the same input it would be a bit more complicated.
However if you select an additional empty line above foo it will behave like that! But you also have to select the #### to uncomment if you do that.

The select on click still isn't quite right, but it does work much better now, so I'm merging it so that I can get on with my work.


Fix ValidatesRunner_Apex #5604

Run Apex ValidatesRunner

Run Apex ValidatesRunner

Is there a reason why you wanted to add runners-core test scope instead of depending on hamcrest_core/hamcrest_library?

Similar issue for Dataflow, opened https://github.com/apache/beam/pull/5671


[SPARK-38416][Pandas API on Spark] Change day to month

What changes were proposed in this pull request?
Right now we have two functions that are testing the same thing.
Why are the changes needed?
To teste both day and mount
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Get the green light.

Can one of the admins verify this patch?

[SPARK-38416][PYTHON][TESTS] Change day to month
PR title need to be changed as above, otherwise LGTM.

Merged to master and branch-3.2.


Allow AwkwardExtensionArray to be generally slicable

The AwkwardExtensionArray.__getitem__ method is opinionated about the types that it gets; it would be more powerful if it passes any slicer through to the underlying Awkward Array and lets that code succeed or fail, depending on slicer type.
We want to be able to take a Pandas Series that contains an Awkward Array, series, and say
series.values["x"]

or
series.values[[2, 0, 0, 1]]

and have that pass the slice "x" or [2, 0, 0, 1] to the Awkward Array. (The
Since the __getitem__ checks types,
https://github.com/intake/awkward-pandas/blob/4c1020c4d88ebccc8c2bfaad69c3b01b0aba3bac/src/awkward_pandas/array.py#L60-L67
both of these fail. (The [2, 0, 0, 1] slice would succeed if we wrapped it as an np.ndarray.)
This came up in scikit-hep/uproot5#734, editing test_0033-more-interpretations-2.py#L82 to be result.values["x"].tolist(), where result is a Series.
The strategy of __getitem__ can be to pass whatever item it gets as a slice on self._data, then check self._data to see if it's an ak.Array, an ak.Record, or other, and wrapping ak.Array or ak.Record as a new AwkwardExtensionArray. The scalar output types ("other") can be returned as-is.
Cc: @kkothari2001

Thanks for raising this @jpivarski-- I'll start working on expanding __getitem__ support.

I'll just point out, though, that series[] will not be able to support this, since pandas handles the input before passing it down to the extension array. We already have series.values.ak[...], which is I think exactly what you are after.

series.values.ak[...] would be fine, if documented.

We already have series.values.ak[...]

I think you mean series.ak[...] :)

er, that might be it :)

Not sure if this needs to be a separate PR, but the AwkwardExtensionArray.__repr__, inherited from the superclass, is making as many Awkward Arrays as there are rows (or visible rows?) which would be expensive if it really is all rows, but is hard to read due to the type information in either case. If it's just a cosmetics issue and not a performance issue, printing the rows with str instead of repr would fix it.

https://github.com/intake/awkward-pandas/pull/16 was an attempt to deal with that, we are aware that we need something better.

Quick start docs which will hopefully make clear the difference between acting on the Series and the underlying array are starting to form here: https://awkward-pandas.readthedocs.io/en/latest/quickstart.html


Modify categories shown on the side to show all categories

Codecov Report

Merging #211 (7b26259) into master (bd41863) will decrease coverage by 0.12%.
The diff coverage is 0.00%.

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


insomnia-inso generate config with --type kubernetes produces Ingress Resources with deprecated apiVersion (extensions/v1beta1)

Describe the bug
The latest version of insomnia-inso produces a Kubernetes configuration file with Ingress resources with apiVersion "extensions/v1beta1". This apiVersion is deprecated as of k8s 1.16 https://kubernetes.io/blog/2019/07/18/api-deprecations-in-1-16/
inso generate config should instead produce Ingress resources with apiVersion "networking.k8s.io/v1"
To Reproduce
Steps to reproduce the behavior:
inso generate config --type kubernetes --output out.yaml

Resulting out.yaml contains Ingress resources with apiVersion extensions/v1beta1
Expected behavior
out.yaml should contain Ingress resources with apiVerion networking.k8s.io/v1
Desktop (please complete the following information):

OS: macOS 11.4
Installation Method: npm
App Version 2.3.0

I have the same issue

Hi @msmith93, thanks for reporting this!
This is indeed already fixed/updated, it was introduced in this PR: https://github.com/Kong/insomnia/pull/4667
And it exists since Insomnia version 2022.3.0 and up.


builtin: expose fmt.Sprint* as builtins

Is your feature request related to a problem? Please describe.
In order to leverage fmt-like behaviour, we currently have to drop down to the text/template package:
import "text/template"

i: 10
res: template.Execute(#"{{printf "%04d" . }}"#, i)

Playground: https://tip.cuelang.org/play/?id=STXBegHgGPs#cue@export@cue
Describe the solution you'd like
To be able to use a fmt-like builtin:
import "fmt"

i: 10
res: fmt.Sprintf("%04d", i)

Describe alternatives you've considered
As above, working around with text/template, or defining the fmt-like behaviour manually via strings and friends.
Additional context
n/a
cc @seh @verdverm

This is a duplicate of https://github.com/cue-lang/cue/issues/1463, I'm pretty sure :) There are more thumbs up reactions in this issue, but the other has input from Marcel and is older, so I think the other is the one to keep around.

Goodness me. That memory of mine again... and I created the issue!


Repackaging Solutions with Parser instructions Update

Required items, please complete
Change(s):

Update in instruction for parser in data connector instruction steps

Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Guidance <- remove section before submitting

Before submitting this PR please ensure that you have read the following sections and filled out the changes, reason for change and testing complete sections:
Thank you for your contribution to the Microsoft Sentinel Github repo.

Details of the code changes in your submitted PR.  Providing descriptions for pull requests ensures there is context to changes being made and greatly enhances the code review process.  Providing associated Issues that this resolves also easily connects the reason.

Change(s):

Updated syntax for XYZ.yaml

Reason for Change(s):

New schema used for XYZ.yaml
Resolves ISSUE #1234

Version updated:

Yes
Detections/Analytic Rule templates are required to have the version updated

The code should have been tested in a Microsoft Sentinel environment that does not have any custom parsers, functions or tables, so that you validate no incorrect syntax and execution functions properly.  If your submission requires a custom parser or function, it must be submitted with the PR.

Testing Completed:

Yes/No/Need Help

Note: If updating a detection, you must update the version field.

Before the submission has been made, please look at running the KQL and Yaml Validation Checks locally.
https://github.com/Azure/Azure-Sentinel#run-kql-validation-locally

Checked that the validations are passing and have addressed any issues that are present:

Yes/No/Need Help

Note: Let us know if you have tried fixing the validation error and need help.

References:

Guidance for Detection checks
General contribution guidance
PR validation troubleshooting

Hi @v-vdixit, I deployed the maintemplate and I could see that description for Infoblox NIOS data connector isn't proper, can you please check?

Hi @v-rbajaj I have checked and resolved the issue, thanks!


User got [object Object] error when uploading a video

The Issue
A user tried to upload a video and got an error message - link here: https://twitter.com/willkriski/status/935567877524029440
Steps to reproduce

User uploaded video, got error message

Expected behaviour
Video should be published
Actual behaviour
[object Object] error happened
System Configuration

LBRY Daemon version: Not sure, got the error info over Twitter
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

Most likely the same as https://github.com/lbryio/lbry-app/issues/748


Implement Selector (#574 part4)

In this part:

Use selector machinery errors for action selectors on construction
Test selector from function
Throw if selector returns anything except null, undefined or DOM node

\cc @VasilyStrelyaev @helen-dikareva @AlexanderMoskovkin

\r-

lgtm but rebase required

lgtm


Minor fix to CGCloud-in-a-nutshell section in docs

Remove bullet 3 in http://toil.readthedocs.io/en/latest/installation.html#cgcloud-in-a-nutshell
-core is a dependency of -toil. Installing is explicitly is unnecessary. Installing it in separate command is potentially racy.

Actually, it's not racy.


Use magnet or link to torrent instead of uploading

Hey man,
First of all thanks for this code, it helped me download a torrent for a dataset I was not able to download myself.
I wanted to ask you, that in your code it is necessary to upload a .torrent file first and then download it. Can we modify this code to accept Magnet links directly or get a torrent file from a link instead of uploading a .torrent file ourselves?

It uses libtorrent library and according to this stackoverflow question replacing these lines:
info = lt.torrent_info(list(uploaded.keys())[0])
h = ses.add_torrent({"ti": info, "save_path": "/content/drive/My Drive/Torrent"})

With this:
params = { 'save_path': '/content/drive/My Drive/Torrent'}
link = "your_magnet_link"
h = lt.add_magnet_uri(ses, link, params)

Should download magnet links. (Because I've sold my pc i stopped developing things so i haven't had a chance to try it but i think it should work if not let me now, if works close the issue)

I tried this, It worked cause my torrent files are being checked and the % counter has started rising.
I will close the issue just after I ask you this: Your readme.md states that this is useless, can you explain why do you think this is useless? I tried transmission-cli and that didn't work for me, are there other torrent clients for CLI that work in Google Colab?

Because you know most torrent clients allows you to select which files to download and loading files to google drive is useless. But actually in your case (downloading dataset to google colab) is might be useful because of downloading torrents in colab and not streaming them to drive.


Testing 1.2.0 on ci

Related issues
Refs: #none
(For autoclosure of issues when PR is merged use Fixes #<issue-number> syntax)
Describe the changes made in this pull request
This PR

fixes some merge errors in the schema
changed the test collection setup to limit itself to tests/1.0.3 and tests/1.1.0
adds tests/1.2.0/poc with a proof of concept data file
adds testing of the poc data file as part of the CI

Instructions to review the pull request
GitHubh Action should run tests for 1.0.3 and 1.1.0 data, then run the poc schema validation in a separate step
If I add some random keyname as part of an author, the test breaks (https://github.com/citation-file-format/citation-file-format/pull/196/checks?check_run_id=2720914662), albeit in an unexpected way (it complains about date-start for example). Not sure if that's something we should address.

If I add some random keyname as part of an author, the test breaks (https://github.com/citation-file-format/citation-file-format/pull/196/checks?check_run_id=2720914662), albeit in an unexpected way (it complains about date-start for example). Not sure if that's something we should address.

Interesting. I've had this before, if I remember correctly in cases where we have different combinations of required/non-required keys or checking against different patterns. I think in these cases jsonschema tries to (heuristically?) guess the target object type, but not reliably so.
In the case you link to I think it breaks due to the injected bad key (which is correct), but also tries to match against person (instead of entity), and so lists all of the keys it wouldn't expect for person. Could be a bug in jsonschema.

Awesomesauce :cookie: :cake:


Added facebook login

Thank you so much sir @shivahr  for mentoring and motivating me....It was an honor and great learning experience for me..


start defining ebml reader

partially resolves https://github.com/Matroska-Org/ebml-specification/issues/88

LGTM.
Suprising it was backticked but not defined.