Improved counting

Here's my version of a counting channel. It has built-in support for a -funinfo command which I will PR later. It keeps track of how many times users have counted.
Difference with the already-existing one is that mine doesn't use a role to prevent counting twice in a row. It's all done through the database and all efficient since it's using a single database entry.
It's very customisable using (a lot of) variables. It has a built-in resetcount command which resets the count with a warning as well, so you can't do it on accident. Apart from that the main part is the amount of feedback it gives.

[X] Code changes have been tested on an instance of YAGPDB, or there are no code changes
[X] I have read and followed the contribution guide

I do not see a benefit to the UX when not using a role - the reason most people count in the support server is exactly because of the (admittedly, hoisted) role. Maybe add an option to toggle this?
Further, we already have a very similar system - Where is the need to add more? How would an end user know which one to use? My experience is that most users who "just want a counting system" are already overwhelmed with the currently existing one...

I understand your point about why people count in the support server, but I've had this same cc for a few months in one of my servers with over 2k members (which are mostly kids). They are still as enthusiastic about it as when I first implemented it. That is without a special role or any other benefits.
I don't think we should question people's motives for wanting to count.
The biggest advantage with this would only really work if I already PR'd my -funinfo command which uses data from this.

If there really are too many variables (which I would totally understand), I could lower them by having come responses just coded in instead.

I will add that note, but I'm not going to write a whole paragraph on how they should do that.

Thanks!


Encountered unknown options during setup(): unknown option: progress.ignore_empty_message

Hello!
I was trying to write my configuration using lazy.nvim. I was getting a lot of "In progress... WARN" messages #295 haskell-tools and I was trying to fix it. When I used the following config, it did not do anything and instead gave me an error that there was an unknown option. Any help? Thank you!
Jason
plugins.lua
{ "j-hui/fidget.nvim", opts = { progress = { ignore_empty_message = true } } },

Can you please confirm that your Fidget is up to date?

I'm having a similar issue and I'm on the latest commit:
[WARN  Thu Nov 23 19:52:48 2023] /lua/fidget.lua:20: Encountered unknown options during setup():
[WARN  Thu Nov 23 19:52:48 2023] /lua/fidget.lua:22: - unknown option: window

I think a more recent update caused the issue because it stopped working after I did an update haven't changed my config in ages except for swapping blend for winblend since I noticed that API change during my troubleshooting:
fidget = require("fidget")
fidget.setup({
	window = {
		winblend = 0,
	},
})

@daneski13 , the version of Fidget you're using has been completely written as of https://github.com/j-hui/fidget.nvim/pull/143.
In the rewrite, the equivalent option is:
{ notification = { window = { winblend = 0 } } }

@TheJasonPlayz are you still encountering this issue? If not, I will close it.


Can't find extensions when api uses a different package name then "ro.fortsoft.pf4j.demo.api"

Hello!
I've checked out the demo_gradle project of the code base and tried to add a couple extensions.
Whenever I add a new ExtensionPoint, lets say "com.thyllian.Hallo" with the following content:
package ro.fortsoft.pf4j.demo.api;

import ro.fortsoft.pf4j.ExtensionPoint;

public interface Hallo extends ExtensionPoint {
    public String getHallo();
}

And add the newly created ExtensionPoint to "plugin1/WelcomeGreeting":
@Extension
public static class WelcomeGreeting implements Greeting, Hallo {
	@Override
    public String getGreeting() {
        return "Welcome";
    }
    @Override
    public String getHallo() {
        return "Hallo";
    }
}

Pf4j can't find WelcomeGreeting when executing pluginManager.getExtensions(Hallo.class);
Now however when I move the "at.lyze.api.Hallo" class to "ro.fortsoft.pf4j.demo.api.Hallo" and fix all imports it it works.
Log files:

ro.fortsoft.pf4j.demo.api: https://paste.ubuntu.com/24007186/
at.lyze.api: https://paste.ubuntu.com/24007188/

Log Diff: https://www.diffchecker.com/mDmlDaJw
MANIFEST.MF will get created properly in both cases:
# Generated by PF4J
ro.fortsoft.pf4j.demo.welcome.WelcomePlugin$WelcomeGreeting
ro.fortsoft.pf4j.demo.welcome.WelcomePlugin$WelcomeGreeting

Project Files:

at.lyze: https://transfer.sh/2FDX9/at.lyze.zip
ro.fortsoft.pf4j.demo: https://transfer.sh/jsvWo/ro.fortsoft.pf4j.demo.zip

First of all I want to say that it's probably one of the best issue report  which I have seen. :+1:
It's for the first time then I see in PF4J that someone tries to create an extension class (annotated with @Extension) that implements multiple extension points. An issue is that we shouldn't have three entries/lines in extensions.idx (for plugin2) with same content.
Can you test the same approach with Hallo in at.lyze.api package but with three separate extensions in plugin2 (one extension class for each extension point)?

I have the following setup now:
api: at.lyze.api.Hallo:
public interface Hallo extends ExtensionPoint {
     public String getHallo();
}

plugin1: ro.fortsoft.pf4j.demo.welcome.WelcomePlugin:
@Extension
public static class WelcomeGreeting implements Greeting {
	@Override
    public String getGreeting() {
        return "Welcome";
    }
}

@Extension
public static class WelcomeHallo implements Hallo {
    @Override
    public String getHallo() {
        return "Hallo";
    }
}

I think the code should step into the if but it doesn't:

When I move the "at.lyze.api.Hallo" class back to "ro.fortsoft.pf4j.demo.welcome.Hallo" it works.

I don't see in first screenshot the value for type (on line type.isAssignableFrom(extensionClass)).

Ah I'm sorry, here's a better screenshot with hopefully all variables.

WelcomeHallo implements Hallo and in normal situation Hallo.isAssignableFrom(WlecomeHallo) returns true but the problem (my feeling) is that you have two classes from different class loaders and from this reason that method returns true.
Can you verify to see if the class loader of type (Hallo) is the same with class loader of extension (WelcomeHallo)?

For some reason it's not the same.

If the response is that extension and extension point are loaded with different class loader. then please see https://github.com/decebals/pf4j/issues/110#issuecomment-244309598 and this line in pom.xml of Maven.

Ah thank you, works now when I remove the api jar!
I should have read the Troubleshooting page more clearly, sorry.

No problem. You are welcome.


Rawyaml issue Argocd 2.4 version fix

Description
Fixes the issue where the yaml inserted in rawyaml section of devtron deployment-template created issue after argocd version upgrade to 2.4.0
Fixes # (issue)
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[ ] The title of the PR states what changed and the related issues number (used for the release note).
[ ] Does this PR require documentation updates?
[ ] I've updated documentation as required by this PR.
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have tested it for all user roles
[ ] I have added all the required unit/api test cases

Already done


Increase JRS Duration from 2 digits to 3 digits

Jobs support Duration of 3 digits, but JRS's are set to 2 digits.

Fixed in release 3.79


[TASK] Refactor GridTableBuilder

This PR is  a preparation to make #955 easier to solve. It should not change functionality
releases: main, 1.0

@jaapio as I mainly refactored your code here, please have a look at this

Sorry I didn't have the headspace to review this in detail. Gridtables are a real pain. My original idea with the tables was that all tables have the same structure. They do have rows, and columns that exist of cells. Once we have the cells we can use the producers to create the content nodes of the cells. Where they mostly have just InlineNodes, and sometimes some more complex nodes like lists, or directives.
To make this work I had the intention to split everything into normal sized cells as there are no rowspans and colspans. And then start merging those in a second iteration. And the last step of parsing would be parsing the cell contents. However I faced some issues in this approach. Colspan content is longer that a column, so words are cut in parts when they stretch over multiple columns, whitespaces are an issue ect.
I do not remember exactly how I solve this, and if I did solve it anyway or just moved on with my focus to some other pain point back then. I wouldn't be surprised if there were more issues with the parsing of tables as the original code was even more of a mess than what we have right now.
My suggestion of this refactoring is to add loads of tests that cover the table parsing and rendering and start improving things. step by step. We already have quite a few test covering the current behavior but I'm not sure we cover everything as there are way to many options create a table. This is a system on it's own.
On the other hand... as long the tests are green, it's ok to proceed the refactoring process. I have no strong opinion on how this should look like.


Please add connection to editor.view.connectionProducer params (or bind it)

Because I would really like to change the curve depending on node (connections -> output & input). And product different line styles for different connections.

Thank you ;). Do you accept PR's?

Yes, if they satisfy the code style and have a simple solution

@Ni55aN please check PR :)


[BUG]: programs.plasma.workspace.cursor.theme doesn't work

I've set up a custom theme and set it like so:
  programs.plasma = {
    workspace = {
      lookAndFeel = "Catppuccin-Mocha-Mauve";
      cursor = {
        theme = "Breeze";
      };
    };
  };

The theme gets set but the cursor theme doesn't. It still uses the default set by the theme set in lookAndFeel instead of Breeze. Or, sometimes, it works for Wayland apps but not for Xorg apps and it's still the theme's default in the settings.

Should've used breeze_cursors instead of Breeze. It just didn't occur to me until I made this as the other commands output like this:
$ plasma-apply-lookandfeel --list
Catppuccin-Mocha-Mauve
org.kde.breezedark.desktop
org.kde.breezetwilight.desktop
org.kde.breeze.desktop

and you can just use whatever is first on the line you want.

@jiriks74 doesn't work for me. Cursor theme is inconsistent between Plasma bottom panel (white cursor) and i.e. Chromium (black breeze cursor). Any suggestions?

Any suggestions?

Post the relevant config and the output of plasma-apply-cursortheme --list-themes.

This could be a GTK issue or a Wayland/XWayland issue. Did you try other apps?

@cudatuda Stylix sometimes conflicts with plasma-manager, I would recommend you to set themes in stylix if you're using it in conjuction with plasma-manager


GroovyClassLoader fileReallyExists super inefficient

fileReallyExists was doing a case sensitive test by listing all files in the directory. this is heavy on the filesystem API and can be done with simply using getCanonicalPath comparison.

Merged, thanks!

Merged, thanks!


AsyncDDGS returns different number of results everytime it's executed.

Let's say I search for the term "amazon", sometimes I receive 30 results, sometimes 70, sometimes 100. It's never inconsistent, what's wrong here?
How is this package working with DDG to retrieve search results and does it relates to this issue?
Code:
from duckduckgo_search import AsyncDDGS

results = await AsyncDDGS(proxies=None).news(
    keywords="amazon", region="wt-wt", safesearch="off", timelimit="d",
    max_results=100
)
print(len(results))

Show me the code.


use new vs code solidity plugin from the hardhat team

We ready to make the leap? 👀

I've been using that initially, switched back to Juan's because it definitely was causing issues I can't remember anymore.

So, I'm pretty sure that on my box the hardhat extension isn't resolving the libraries through remappings. Think that worked on my hackathon sample, though. Restarted 3 times. Juan Blanco's extension is disabled.

got the same result after a fresh clone / checkout / install. Hardhat plugin is enabled, Juan's extension is disabled, yarn && forge i. Looks like my screenshot. For reproduction reasons: could you do a fresh clone on your box & try again, @schmackofant ? Can you confirm that you've disabled Juan's Solidity extension as well?

Yes Juan's solidity extension is disabled.
Just did a fresh git clone and forge install, it still works. I get no module errors

No, the issue is that my vsc doesn't recognize the library source links (I can't click them). It only highlights the first occurrence it cannot discover. So, you can click on the ERC1155Upgradeable token and it opens files from lib/openzeppelin... for you correctly? Wdym with "hardhat packages"?

so, I tried a lot now to pinpoint that issue. It indeed seems that the hardhat.config file in side the foundry project. On my machine I can "fix" it by

removing hardhat.config.js
shutting down the IDE
restart the IDE, click 5 times on a dependency.

On my web3 template repo there's not hardhat config, there it works out of the box.
At the moment I'm hesitant to give this an approval because it would definitely break my workflow. Right now, foundry is also (still?) referring to Juan's Solidity extension btw -> https://book.getfoundry.sh/config/vscode. Reason that it works on your box could be that the Mac FS is enumerating file entries differently (and picks up the project config in a different order than on my machine).
Think, I'm going to post that on the hh extension's issues first so they can reproduce that. Maybe @DorianWilhelm would like to try it on his machine, too? Right now merging this definitely would make my life harder.

https://github.com/NomicFoundation/hardhat-vscode/issues/365#issuecomment-1377939072 think we can finalize this soon :)

fixed by #55 and #56


feat: add Firebase as an alternative to Mongodb

first of all, firebase is very fast you cloud use firebase directly in the front end without interacting with the backend.
Even though, We cloud scrap data on the local machine without deploying anywhere.
Apart from that, firebase also provides ML and analytics kits which may help us in the future. We cloud also do lots of cool stuff with firebase like cloud functions that can run automatically at a certain time.

first of all, firebase is very fast you cloud use firebase directly in the front end without interacting with the backend. Even though, We cloud scrap data on the local machine without deploying anywhere.
Apart from that, firebase also provides ML and analytics kits which may help us in the future. We cloud also do lots of cool stuff with firebase like cloud functions that can run automatically at a certain time.

Thanks for replying @Zain-ul-din! I am curious to know if it provides secure, restricted APIs (read-only) that we can provide directly to the user without explicitly adding our server between them.

firebase is very curious about security. you cloud write you're own security rules.

firebase is very curious about security. you cloud write you're own security rules.

Okay! Let's add this then. :+1:

sure


How is it possible to redraw the chart in Ember?

I have tried all manner of redrawing functions. The only thing I can get to working is to make a toggle that hides and reshows a component.
I want to update the data and redraw. I can't seem to get this to work.
The javascript examples don't translate directly to the ember implementation it seems.
I've tried code like this:
      let stage = anychart.graphics.create();       this.chart.container(stage);       this.chart.draw();
I've been able to use removeAllSeries() but what if I wanted to change a label or something?

@wonderful123
The chart calls redraw function automatically when something in appearance is changed. All you need is to change any visual setting or set new data. Here you can find a pure JS sample of adding a new point and changing chart label text Absolutely the same approach you can use in the integration.


DBZ-853 Fix kafka database history storage miscount attemp number even if there are more records to consume.

Fix kafka database history storage miscount attemp number even if there are more records to consume.
https://issues.jboss.org/browse/DBZ-853

Good catch, thanks! Can you rebase this PR onto the latest master, so it just contains the actual commit but no merge commits? Thanks!

@ooooorz Spot on, thanks a lot!