Ability to rename ASC export resource group name

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
Is your feature request related to a problem?
No

Describe the solution you'd like
Discussed with Kevin to have the ability to rename ASC export resource groups names for a customer deployment, submitting this feature request per discussion

Additional context

Thank you @mbilalamjad
Will address this as part of the next release.

Adding this in an upcoming PR using an optional input under configure_management_resources.advanced, which allows us to set this value using the following configuration setting:
  configure_management_resources = {

    # ...

    advanced = {
      asc_export_resource_group_name = "${var.root_id}-security-logs-export"
    }
  }

This allows setting the value to whatever you want it to be in the managed Policy Assignment input parameter:

I've considered whether a more generic approach would add value elsewhere, but this is the only parameter value for policies currently managed by the module which makes sense to be able to customise in this way.


Android App - Tic Tac Toe Game Project

I would like to create a new directory named android-apps and would like to contribute my android apps to this open source projects.

@indrakishore Do review this request and assign me this issue, if you are okay with this.

@indrakishore Thanks I will make pr in few minutes


Security > Google reCAPTCHA Admin Panel | Adobe Commerce

Name
WildHorse
Additional Context
Security > Google reCAPTCHA Admin Panel | Adobe Commerce

Invitation sent to join the GitHub Organization. Welcome to Infraform 🎉Please check your mail to accept the invitation sent by GitHub! 🤖About usInfraform is an open source solution for serverless platforms, it is designed for developing solutions and enhancing creators to create their infrastructure 💻 Great to have you in our community ✨ Looking forward to your valuable contributions! 🤓 You can also check our community ✨ Github ✨ LinkedIn 💜 You can also share our community with other folks who want to learn about development, DevOps, tech, and contribute together and with us 👉💜👈


[BUG] Adding model to existing plugin, existing plugin instances disappear

Description
I have some simple plugins which are basically just templates to wrap child content, things like rows and columns. Usually they don't need a model. If the requirements of a plugin change and I do need a model for a plugin, adding a model will make all existing instances of a plugin disappear from the editor.

Steps to reproduce
`
    @plugin_pool.register_plugin
    class ColumnPlugin(CMSPluginBase):
        name = _("Column")
        # model = models.Column    <-- adding model to plugin that already exists
        render_template = "cmsplugin_column/cmsplugin_column.html"
        allow_children = True
        require_parent = True
        parent_classes = ["RowPlugin"]

        def render(self, context, instance, placeholder):
            context = super().render(context, instance, placeholder)
            return context

`

create the plugin above, initially without a model
create a couple instances of the plugin on the page
add the model to the plugin ( uncomment the commented line above )
plugins will disappear from the page

Expected behaviour
I would expect that either plugins show up, but without any settings, or that some error displays, indicating that some plugins dont have corresponding model isntances.
Actual behaviour
Plugins disappear without any warning or indication
Additional information (CMS/Python/Django versions)
django==3.2.14
django-cms==3.11.0
Do you want to help fix this issue?
Could this be solved via an manage.py command that creates default model instances for plugins that hove none? I could help creating a manage.py command if I had some guidance how to correctly query for this case.

[ ] Yes, I want to help fix this issue and I will join #workgroup-pr-review on Slack to confirm with the community that a PR is welcome.
[x ] No, I only want to report the issue.

Hey @alexgustafson ! Thanks for the precise issue report and taking the time to outline all details.
Indeed, Django's migration system does not recognize that by adding a model to a plugin a one-to-one relation is generated between that model and the CMSPlugin class. The correct solution is to create a migration manually.
I have not tested this but it could look something like this:
from django.db import connection, migrations

from cms.models.pluginmodel import CMSPlugin

from cms_plugins import MyCoolPlugin

def add_model_to_plugin(model):
    """Factory for model forward migration"""

    def forwards(apps, schema_editor):
        """ Adds instances for the new model.
        ATTENTION: All fields of the model must have a valid default value!"""
        for plugin in CMSPlugin.objects.filter(plugin_type=model.__name__):
            model(cmsplugin_ptr=plugin).save()
   return forwards

def remove_model_from_plugin(model):
    """Factory for model backwards migration"""

    def backwards(apps, schema_editor):
        """ Removes all instances of the new model from the database.
        ATTENTION: All plugin configurations are lost!"""
        ...

class Migration(migrations.Migration):

    dependencies = [
        ("your package", "migration which creates the new model"),
    ]

    operations = [
        migrations.RunPython(add_model_to_plugin(MyCoolPlugin.model),
        remove_model_from_plugin(MyCoolPlugin.model)
    ]

Can you give it a try (on a development system with database backup, please)?
Also, if you get it to work, might it be useful to add those two factory functions and to cms.pluginmodel. Also, we would need to add the example migration to the docs. Wouldn't that be a great PR, @alexgustafson ?

Sorry coming back to this after such a long time.
I could not make this work then, and did not have the time to properly debug and write up what I was seeing.
Now I have a bit of time, and have the same problem on a new project.
Here is the code from my migration:

from cms.models.pluginmodel import CMSPlugin
from my_plugin.cms_plugins import MyPlugin
from my_plugin.models import MyPluginSettings

def add_model_to_plugin(model, plugin):
    """Factory for model forward migration"""

    def forwards(apps, schema_editor):
        """ Adds instances for the new model.
        ATTENTION: All fields of the model must have a valid default value!"""
        for plugin_instance in CMSPlugin.objects.filter(plugin_type=plugin.__name__):
            instance = model(pk=plugin_instance.pk, cmsplugin_ptr=plugin_instance)
            instance.save()
    return forwards

 class Migration(migrations.Migration):

    dependencies = [
        ('apartments', '0004_alter_apartment_options_apartmentplugin'),
    ]

    operations = [
        migrations.RunPython(add_model_to_plugin(MyPlugin.model, MyPlugin), remove_model_from_plugin(MyPlugin.model)),
    ]

The error is  NodeAlreadySaved: Attempted to add a tree node that is already in the database
I thtought this might be a problem related to the version of TreeBeard that I have installed. Version 4.4 seems to be a good version though.
It seems as if trying to save an instance on an already existing plugin, tries to save the plugin itself as a new instance, but then sees that the plugin already exists.

Having the same question. I tried to do it locally and had the same issue: treebeard.exceptions.NodeAlreadySaved: Attempted to add a tree node that is already in the database

Just adapted code from @fsbraun:
def add_model_to_plugin(model: ReferencePluginModel, plugin: ReferencePlugin):
    """Factory for model forward migration"""

    def forwards(apps, schema_editor):
        """ Adds instances for the new model.
        ATTENTION: All fields of the model must have a valid default value!"""
        from cms.api import add_plugin

        plugin_instances = CMSPlugin.objects.filter(plugin_type=plugin.__name__)
        for plugin_instance in plugin_instances:
            logger.info('Creating new model instance for plugin instance %s', plugin_instance.pk)
            obj = model()
            obj.pk = plugin_instance.pk
            obj.cmsplugin_ptr = plugin_instance
            obj.plugin_type = plugin.__name__
            obj.placeholder = plugin_instance.placeholder
            obj.parent = plugin_instance.parent
            obj.language = plugin_instance.language
            obj.position = plugin_instance.position
            obj.creation_date = plugin_instance.creation_date
            obj.depth = plugin_instance.depth
            obj.path = plugin_instance.path
            obj.numchild = plugin_instance.numchild
            obj.save()

And it works 🎉

Looking at my own example, I realize that it is bad practice to access models in migrations through direct import. It is recommended practice to use the apps parameter, e.g.:
CMSPlugin = apps.get_model("cms", "CMSPlugin")
new_model = apps.get_model("my_plugin", "MyPluginSettings")

I haven't been able to test with django-treebeard > 4.4 yet because of other dependencies.
I do have a workaround though. In the example below I have a GalleryPlugin that had no parameters. The client requested a "max-height" parameter, so I added a Gallery model to the GalleryPlugin. In order to bypass the Django checks I just create the initial Gallery instances for each plugin directly with SQL.
`def forwards_func(apps, schema_editor):
CMSPlugin = apps.get_model("cms", "CMSPlugin")
from django.db import connection
with connection.cursor() as cursor:
    for gallery_plugin in CMSPlugin.objects.filter(plugin_type="GalleryPlugin"):
        cursor.execute(
            f"INSERT INTO cmsplugin_gallery_gallery (max_height, cmsplugin_ptr_id) VALUES ('580px', {gallery_plugin.id})"
        )`

Not pretty, but simple enough.
For a more complex model I would recommend breaking the migration down into two steps. One to initialize the model instances (in SQL), the second migration would add the fields with proper defaults using "normal" migration techniques.


Roll/Pitch/Yaw setpoints not logged in STABILIZE mode

http://logs.uaventure.com/view/XyiLq9cLLKzhboT9rpZsmd
Firmware v1.4.4

@kd0aij
I was just looking at a recent log that had attitude and rate setpoints during FW stabilize mode. Can you confirm?

I have a logfile from 17 October with:
Hardware Version	PX4FMU_V4
Firmware Version	6be0eb76b6c9a42d67fb7965b17f477083b0f6f0
that has vehicle_attitude_setpoint_0.roll/pitch/yaw_body and vehicle_rates_setpoint logged in nav_state 0
So I'll close this


Test new codecov configuration

codecov does not seem to use the config from the PR. Let's see the result after merging it.
bors r+


Thread - fake member in thread for message in WS chat of Removed member

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to NewDot
Invite a member
Send a messages in WS chat
Create a thread for this message
Remove this member from workspace
WS chat is archived but thread stays in LHN
Go to thread chat and check members in settings

Expected Result:
no fake accounts should be displayed
Actual Result:
fake member in thread for message in WS chat of Removed member
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[x] Android / Chrome
[ ] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: yes
Reproducible in production?: yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/dfdf334c-79c2-4aeb-98d6-bc00d220435a
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0107070ca2f72b0d39
Upwork Job ID:<PHONE_NUMBER>786818048
Last Price Increase: 2023-06-06

@thienlnam This is looking Internal to me. Can you correct me and switch it to External if you think that's wrong?

I can't reproduce on main anymore - closing!


fixes #23 preventing yielded progress data from being swallowed

It feels a bit awkward the way, I'm forwarding the yields here. Would love a review from another rubyist. cc @sneal

@mwrock  I think we just need to pass along the block, see this branch https://github.com/WinRb/winrm-fs/tree/pass-along-blocks-issue-23

Cool. Thought I tried that but it clearly works and is cleaner. Just a couple rubocop violations to cleanup there.


P2 and P1 download trouble

Download with debug to P2,
unplug,
download to P1

Propeller Tool is stuck.

Fixed.
Problem was the communication thread was given P1 code to download and the FP2Debug was set.  This caused a P1 download to transition the CThread from downloading to debugging.
Fixed so that FP2Debug is set true only if Model is P2 and DebugMode is true.


Docker compose version 2 not supported.

Rancher Version:
Rancher	v1.1.4
Cattle	v<IP_ADDRESS>
User Interface	v1.1.14
Rancher Compose	v0.8.6
Docker Version:
Client:
Version:      1.12.3
API version:  1.24
Go version:   go1.6.3
Git commit:   6b644ec
Built:        Wed Oct 26 21:44:32 2016
OS/Arch:      linux/amd64
Server:
Version:      1.12.3
API version:  1.24
Go version:   go1.6.3
Git commit:   6b644ec
Built:        Wed Oct 26 21:44:32 2016
OS/Arch:      linux/amd64
OS and where are the hosts located? (cloud, bare metal, etc):
Bare metal / ubuntu 14.04 LTS
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
single node rancher
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
any version 2 docker-compose file, it breaks on the line version: '2'
Results:
in web ui: Activating (Unknown resolution for '2' using <config.RawService Value> at line 0, column 9)
Expected:
normal execution

This is a duplicate issue: https://github.com/rancher/rancher/issues/3973
Support for services and volumes will be added in 1.2.0.


New logo?

Hello sir.
You have a great app, unfortunately this app does not have a logo yet, may I donate a logo for your app?

Haha, I'd say why not, but I'm not part of the team. I want to see the issues tab showing the number 0 xD

Hi. Thank you for your offer, but we actually have a new logo. We just have not managed to put it up yet. However, I believe we should get to it soon.


Preview in realtime at editor

First of all, thanks for contributing us with this project, is really nice.
I am looking a way to show the preview on top of what is being written.
Is it possible?

sorry, I think this is impossible.

Please have a look into the ckeditor demo
https://ckeditor.com/docs/ckeditor5/latest/features/markdown.html
At the moment you are inserting text it previews the written format.

I think the ckeditor's demo is meaing you written richtext then it will give you the markdown output.
And this component is using ace editor not ckeditor.

Ok understood. Thanks for response, I very appreciate


Cassandra use

Can I disable it ? and will it impact my server performance ? am trying to minimise processes running on my machine

Theoretically cassaandra is required only when using database / documents REST API.
For Push Notification and IDP/IDM only PostgreSQL is required.
However, disabling cassandra without linking to an external cluster will lead to startup errors.

'#' cassandra['enable'] = true

cassandra['enable'] = false

Please open an issue request, at https://aerobase.atlassian.net

Support server startup when cassandra is down

Ok cool I will stop cassandra after startup till this issue is resolved.


Bot reports wrong user count.

General Troubleshooting

[x] I have checked for similar issues.
[x] I have updated to the latest promcord version
[x] I have checked the branches or the maintainers' PRs for upcoming bug fixes.

Bug Report

The Bot reports a wrong user count.

Expected Behavior

The bot should report the right user count.

Exception or Error

N/A

Propably a discord related issue. Cant reproduce.


Add a banner component

Fixes https://github.com/pulumi/marketing/issues/13.

My recommendation would be to use a cookie to track banner dismissal. For example, we could have a cookie, ex. pb (pulumi banner), that's value is set to the name/id of the banner when the banner is dismissed. This way when the component mounts we can check if the cookie exists and if it does then we won't show the banner. We can eliminate the need to plumb this stuff through the State and localStorage.

This is effectively exactly how this works now; the only difference is the storage mechanism. One reason cookies are not a good fit here is that this sort of data does not need to be sent to the server on every request. Local storage is actually quite suitable for this sort of thing, and using the store as an abstraction over localStorage is definitely by design.


Added a higher order update to Seq

Implements https://github.com/javaslang/javaslang/issues/1819

Great addition and well done! Thank you, Lőrinc.

Had to fix some ambiguous null tests, should be ok now :)

Current coverage is 97.08% (diff: 86.66%)

Merging #1821 into master will decrease coverage by 0.01%

@@             master      #1821   diff @@
==========================================
  Files            87         87
  Lines         11269      11277     +8
  Methods           0          0
  Messages          0          0
  Branches       1871       1869     -2
==========================================
+ Hits          10942      10948     +6
- Misses          205        207     +2
  Partials        122        122

Powered by Codecov. Last update 842e5f0...baf734d

Thanks!


About  code on calgary-campinas

Thanks for your amazing work, which has inspired me a lot！
I am currently trying to apply PromptMR to the calgary-campinas dataset and examine its effects. However, I am facing some difficulties in processing the data and adapting it to the format required by the model. If possible, could you please share the code for training PromptMR on this dataset?
I appreciate it very much ！

Hi @string-ellipses , thank you for your interest. However, I currently do not have the time to do this. The code will be released in the near future.

Hi @string-ellipses , thank you for your interest. However, I currently do not have the time to do this. The code will be released in the near future.

Thanks for your kind reply！


UnusedPrivateMember doesn't work with type resolution exactly like without it

Expected Behavior

UnusedPrivateMember (and probably other rules) should work with type resolution exactly like without it.
Observed Behavior

UnusedPrivateMember (and probably other rules) doesn't work with type resolution exactly like without it.
Steps to Reproduce

Reproduction.zip
Run ./gradlew detektAll, it should run successfully without any errors. Uncomment these lines in the project build.gradle
//    jvmTarget = "1.8"
//    classpath.setFrom(project.configurations.getByName("detekt"))

Run ./gradlew detektAll again, now it complains about UnusedPrivateMember, although it's excluded in the config for functions with Preview annotation.
Context

Your Environment

Version of detekt used: 1.20.0-RC1
Version of Gradle used (if applicable): Gradle 7.2 (Revision a773786b58bb28710e3dc96c4d1a7063628952ad)
Operating System and version: Fedora 35
Link to your project (if it's a public repository): Reproduction.zip

🤔 Ok, so the problem here is that ignoreAnnotated isn't working properly... Thanks for the reproducer. I'm going to check what's going on here.

I take a deep dive on your example and I don't see any issue on detekt. A few things I found:

The way you configured detekt is strange, I'm not sure if it is completely correct or not. But I changed it to use the one that is in the documentation and the ignoreAnnotated start to work properly.
It's normal that some rules find nothing without type solving and find lots of issues with type solving. That's completely the point of type solving, enable more powerful rules.

I'm closing this because all is working as expected. If you think that I'm missing something, please, don't hestiate to answer.


[Monitoring] Failure to generate a temporary token too many times

Should be closed when https://github.com/cds-snc/forms-terraform/pull/176 is merged


Replace icons

Replace the CEISL icon in the navbar with color, and add a CEISL lockup in the footer and replace the IUPUI lockup with the full resolution original (did not scale well).

Removed the IUPUI icon, not worth the effort, especially since I like the non-transparent dark midnight color anyway.


Footer isn't positioned on the bottom of the page

Describe the bug
On sites where the page content doesn't fill the viewport height, the header is not fixed to the bottom of the page.
Expected behavior
Regardless of page content height, there should be no space between the bottom of the screen and the footer.
Additional context
I tried incorrectly to fix this before.
Try one of these solutions: Sticky footers

Fixed with #176


[Doc update][Teams-support-feedback]

updated table.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/feedback.md
80
18
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 557f782:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/feedback.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/feedback.md
81
18
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 298caba:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/community-feedback-edit.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/community-feedback-this-page.png
:white_check_mark:Succeeded
View

msteams-platform/feedback.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/feedback.md
81
18
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit d527606:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/community-feedback-edit.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/community-feedback-this-page.png
:white_check_mark:Succeeded
View

msteams-platform/feedback.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/feedback.md
81
18
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 3ac49d8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/community-feedback-edit.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/community-feedback-this-page.png
:white_check_mark:Succeeded
View

msteams-platform/feedback.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/feedback.md
81
18
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 15ec23d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/community-feedback-edit.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/community-feedback-this-page.png
:white_check_mark:Succeeded
View

msteams-platform/feedback.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/feedback.md
81
18
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 049706f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/community-feedback-edit.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/community-feedback-this-page.png
:white_check_mark:Succeeded
View

msteams-platform/feedback.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/feedback.md
96
6
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit d8cf250:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/community-feedback-edit.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/community-feedback-this-page.png
:white_check_mark:Succeeded
View

msteams-platform/feedback.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/feedback.md
96
6
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit c27db63:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/community-feedback-edit.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/community-feedback-this-page.png
:white_check_mark:Succeeded
View

msteams-platform/feedback.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/feedback.md
96
6
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit abe613d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/community-feedback-edit.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/community-feedback-this-page.png
:white_check_mark:Succeeded
View

msteams-platform/feedback.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/feedback.md
96
6
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 9d47bb2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/icons/GitHub-icon.png
:white_check_mark:Succeeded
View

msteams-platform/assets/icons/teams-icon.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/community-feedback-edit.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/community-feedback-this-page.png
:white_check_mark:Succeeded
View

msteams-platform/feedback.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Wraper for main mothulity script

Proposed change
Create bash wraper for main mothulity python script, remove shebang edition from installation.

I am somewhat reluctant as this is a working solution that passes the CI tests. I have never encountered any problems with it.
Though we are using the conda envs, there's a piece from the virtualenv documentation that says directly that it is ok to use such a shebang.


[RFE] Support credential files for Gradle projects

When adding Gradle as a supported project management system for analysis in Konveyor, the dependencies defined in the build.gradle file may be hosted in private repositories or repositories that do not allow anonymous access.
Currently, it is possible to upload a settings.xml file to obtain such dependencies in Maven projects, however, there is no option to do so for Gradle projects.
Without this feature, it would not be possible to use Konveyor to analyze Gradle projects with dependencies hosted in private repositories.

I am not sure if we need an enhancement for this. The solution is simply for us to add a new type of credential named "Gradle properties". This file can be placed directly in the project directory, and it will contain the passwords set by the user. The user will have to include references in the build.gradle file as explained here. The gradle.properties file will contain the passwords.


[WIP] Update rsyslog to also listen on TCP port 515

Right now long messages are getting truncated.
I still need to update st2.conf to utilize the TCP transport.

Where should I put ini_setting directive which configures st2.conf to use TCP transport + port 514, st2server.pp?

https://github.com/StackStorm/st2workroom/blob/master/modules/profile/manifests/st2server.pp
In there somewhere.

Tested the changes, seem to be working fine.

lgtm


missing default in flake outputs

Please add this or add instructions on how to run

This is not how this is supposed to be used.

How is it supposed to be used?

Thank you for asking, mr 714, ser

@lemmon-714 just use Agora
@lemmon-714 ok thanks I will do that, thumbs up


Submit an bioconda recipe?

A very good tool! But would you like to submit a bioconda recipe? So that people can install the tool by a simple command "conda install mummer2circos".

Hi @liaochenlanruo
Mummer2circos is now available on bioconda!


Fix HeaderHelper listed headers parsing (#15)

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Fixed issues
#15 #16

Add HeaderHelper tests;
Fix test case from #16

👍


Unrecognized event: MESSAGE_REACTION_REMOVE_EMOJI

General Troubleshooting

[x] I have checked for similar issues.
[x] I have updated to the latest JDA version.
[x] I have checked the branches or the maintainers' PRs for upcoming bug fixes.

Bug Report

After calling Message#clearReactions(String unicode) with the emote encoded as Unicode characters, I receive this error message:
2021-09-30 22:58:59.691 DEBUG 7756 --- [inWS-ReadThread] n.d.j.internal.requests.WebSocketClient  : Unrecognized event:
{"op":0,"s":40,"t":"MESSAGE_REACTION_REMOVE_EMOJI","d":{"emoji":{"name":"⬇","id":null},"guild_id":"254236409706053632","message_id":"893240468891119636","channel_id":"259021952767754250"}}

Since the event MESSAGE_REACTION_REMOVE_EMOTE is registered here, I'd assume Discord is inconsistent with it's event naming and MESSAGE_REACTION_REMOVE_EMOJI should be handled just like MESSAGE_REACTION_REMOVE_EMOTE. Therefore both events should be registered for processing by the same handler implementation (or this is considered a bug on discord's side).
Expected Behavior

No error should occur and the eventhandlers should be called accordingly.
Code Example or Reproduction Steps

fun main() {
    val jda = JDABuilder
        .createDefault(token)
        .setActivity(Activity.of(Activity.ActivityType.DEFAULT, "..."))
        .setAudioPool(Executors.newScheduledThreadPool(4))
        .setAutoReconnect(true)
        .setEnableShutdownHook(true)
        .setStatus(OnlineStatus.ONLINE)
        .addEventListeners(listener)
        .build()

    jda.awaitReady()
    val slightSmileEmoji = "\uD83D\uDE42"
    val msg = jda
        .getGuildById("") // Any guild
        ?.getTextChannelById("") // Any text Channel
        ?.sendMessage("example")
        ?.complete()
    msg?.clearReactions(slightSmileEmoji)?.complete()
}

val listener = object : ListenerAdapter() {
    override fun onMessageReactionRemove(event: MessageReactionRemoveEvent) {
        LoggerFactory.getLogger("test").info("removed")
    }
}

For reproducing the method Message#clearReactions has to be called (just removing your own reaction does not work).
The listener is in this example is never called.
Code for JDABuilder or DefaultShardManagerBuilder Used

JDABuilder
            .createDefault(token)
            .setActivity(Activity.of(Activity.ActivityType.DEFAULT, "..."))
            .setAudioPool(Executors.newScheduledThreadPool(4))
            .setAutoReconnect(true)
            .setEnableShutdownHook(true)
            .setStatus(OnlineStatus.ONLINE)
            .addEventListeners(*listeners.toTypedArray())
            .build()

Exception or Error

2021-09-30 22:58:59.691 DEBUG 7756 --- [inWS-ReadThread] n.d.j.internal.requests.WebSocketClient  : Unrecognized event:
{"op":0,"s":40,"t":"MESSAGE_REACTION_REMOVE_EMOJI","d":{"emoji":{"name":"⬇","id":null},"guild_id":"254236409706053632","message_id":"893240468891119636","channel_id":"259021952767754250"}}

So this is actually a typo/bug in JDA.
This should be _EMOJI. _EMOTE is not an actual event.
The rest of the code in JDA knows this, it just happens to be that we typo'd this. Kinda crazy that no one has noticed this almost 3 years.
Feel free to open a PR against development please!

Addressed by #1853

Thank you! The event is now processed correctly, but another error occurs:
2021-10-03 03:40:07.503 ERROR 11164 --- [inWS-ReadThread] n.d.j.internal.requests.WebSocketClient  : Got an unexpected error. Please redirect the following message to the devs:
	JDA 4.3.0_332
	MESSAGE_REACTION_REMOVE_EMOJI -> {"emoji":{"name":"⬇","id":null},"guild_id":"317670715652308995","message_id":"894036059753680986","channel_id":"317670715652308995"}

java.lang.NumberFormatException: String value<PHONE_NUMBER>53680986 exceeds range of unsigned int.
	at java.base/java.lang.Integer.parseUnsignedInt(Integer.java:839) ~[na:na]
	at java.base/java.lang.Integer.parseUnsignedInt(Integer.java:928) ~[na:na]
	at net.dv8tion.jda.api.utils.data.DataObject.get(DataObject.java:830) ~[JDA-4.3.0_332.jar:4.3.0_332]
	at net.dv8tion.jda.api.utils.data.DataObject.getUnsignedInt(DataObject.java:586) ~[JDA-4.3.0_332.jar:4.3.0_332]
	at net.dv8tion.jda.internal.handle.MessageReactionClearEmoteHandler.handleInternally(MessageReactionClearEmoteHandler.java:59) ~[JDA-4.3.0_332.jar:4.3.0_332]
	at net.dv8tion.jda.internal.handle.SocketHandler.handle(SocketHandler.java:36) ~[JDA-4.3.0_332.jar:4.3.0_332]
	at net.dv8tion.jda.internal.requests.WebSocketClient.onDispatch(WebSocketClient.java:952) ~[JDA-4.3.0_332.jar:4.3.0_332]
	at net.dv8tion.jda.internal.requests.WebSocketClient.onEvent(WebSocketClient.java:839) ~[JDA-4.3.0_332.jar:4.3.0_332]
	at net.dv8tion.jda.internal.requests.WebSocketClient.handleEvent(WebSocketClient.java:817) ~[JDA-4.3.0_332.jar:4.3.0_332]
	at net.dv8tion.jda.internal.requests.WebSocketClient.onBinaryMessage(WebSocketClient.java:991) ~[JDA-4.3.0_332.jar:4.3.0_332]
	at com.neovisionaries.ws.client.ListenerManager.callOnBinaryMessage(ListenerManager.java:385) ~[nv-websocket-client-2.14.jar:na]
	at com.neovisionaries.ws.client.ReadingThread.callOnBinaryMessage(ReadingThread.java:276) ~[nv-websocket-client-2.14.jar:na]
	at com.neovisionaries.ws.client.ReadingThread.handleBinaryFrame(ReadingThread.java:996) ~[nv-websocket-client-2.14.jar:na]
	at com.neovisionaries.ws.client.ReadingThread.handleFrame(ReadingThread.java:755) ~[nv-websocket-client-2.14.jar:na]
	at com.neovisionaries.ws.client.ReadingThread.main(ReadingThread.java:108) ~[nv-websocket-client-2.14.jar:na]
	at com.neovisionaries.ws.client.ReadingThread.runMain(ReadingThread.java:64) ~[nv-websocket-client-2.14.jar:na]
	at com.neovisionaries.ws.client.WebSocketThread.run(WebSocketThread.java:45) ~[nv-websocket-client-2.14.jar:na]

I presume this is a typo here.
I can commit a bugfix myself, but that would only happen tomorrow (since it's already quite late for me).

Yeah, should be content.getUnsignedLong("message_id") instead.

I agree with @DManstrator. Try making a PR for it :)

Issue fixed with version 4.3.0_333


Image tagging for local build

@robinbobbitt I couldn't find a way to tag the image directly in the bonfire config file, however the --set-image-tag in the bonfire deploy command seems to work well. So our make deploy-app target should correctly tag the image as "latest".

@robinbobbitt Thanks for linking me to this Slack conversation. I've added the image and tag as parameters in the bonfire config file.

/lgtm


commas not needed in pip install command

Hi Gediminas,
I don't know if it depends on PIP version but commas are not accepted on the shell command line as they mean something different. Removing them will make the install command work.

Hello Florido. You are completely right. Commas are a mistake.


Error with Quick Start

What happened?
It got the following output trying to get the Quick Start code to work.  I made some minor changes to the prompt because it was generating fake data rather than trying to pull original data.
The prompt I used was pretty close to.
Create a plot of NVDA vs TSLA stock returns YTD from 2024-01-01. Download real world data. Don't use sudo in the command line.
I also made a modification to run this in an async function.  I don't think this affected the error, because it ran properly once before.  However, I'll include the full function just for thoroughness.
async def main():
    async with DockerCommandLineCodeExecutor(work_dir=work_dir) as executor:  # type: ignore[syntax]
        # Register the assistant and executor agents by providing
        # their agent types, the factory functions for creating instances and subscriptions.
        await Assistant.register(
            runtime,
            "assistant",
            lambda: Assistant(
                OpenAIChatCompletionClient(
                    model="gpt-4o",
                    api_key=os.getenv("OPENAI_API_KEY")
                )
            ),
        )
        await Executor.register(runtime, "executor", lambda: Executor(executor))

        # Start the runtime and publish a message to the assistant.
        runtime.start()
        await runtime.publish_message(
            Message(instruction), DefaultTopicId()
        )
        await runtime.stop_when_idle()

# Run the async main function
asyncio.run(main())

--------------------------------------------------------------------------------
Assistant:
The `!pip install` command is specific to Jupyter notebooks and similar interactive environments. Since we are in a standard script environment, let's try a different approach.

You can install the `yfinance` package by running the following command directly in your terminal or command prompt (not in the Python script):

```
pip install yfinance
```

After installing `yfinance`, rerun the Python script. Here's the script again for your convenience:

```python
import yfinance as yf
import pandas as pd
import matplotlib.pyplot as plt

# Define the start and end dates
start_date = "2024-01-01"
end_date = pd.Timestamp.today().strftime('%Y-%m-%d')  # Today's date

# Define the stock tickers
tickers = ["NVDA", "TSLA"]

# Download the stock data
data = yf.download(tickers, start=start_date, end=end_date)

# Calculate daily returns
returns = data['Adj Close'].pct_change().dropna()

# Plot the returns
plt.figure(figsize=(10, 6))
for ticker in tickers:
    plt.plot(returns.index, returns[ticker], label=ticker)

plt.title('YTD Stock Returns for NVDA and TSLA (2024)')
plt.xlabel('Date')
plt.ylabel('Daily Returns')
plt.legend()
plt.savefig('nvda_vs_tsla_2024_returns.png')
```
Error processing publish message
Traceback (most recent call last):
  File "/path/to/folder/.venv/lib/python3.12/site-packages/autogen_core/application/_single_threaded_agent_runtime.py", line 385, in _process_publish
    await asyncio.gather(*responses)
  File "/path/to/folder/.venv/lib/python3.12/site-packages/autogen_core/application/_single_threaded_agent_runtime.py", line 377, in _on_message
    return await agent.on_message(
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "/path/to/folder/.venv/lib/python3.12/site-packages/autogen_core/components/_routed_agent.py", line 468, in on_message
    return await h(self, message, ctx)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/path/to/folder/.venv/lib/python3.12/site-packages/autogen_core/components/_routed_agent.py", line 148, in wrapper
    return_value = await func(self, message, ctx)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/path/to/folder/autogen_agents.py", line 52, in handle_message
    result = await self._code_executor.execute_code_blocks(
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/path/to/folder/.venv/lib/python3.12/site-packages/autogen_ext/code_executors/_docker_code_executor.py", line 284, in execute_code_blocks
    return await self._execute_code_dont_check_setup(code_blocks, cancellation_token)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/path/to/folder/.venv/lib/python3.12/site-packages/autogen_ext/code_executors/_docker_code_executor.py", line 249, in _execute_code_dont_check_setup
    command = ["timeout", str(self._timeout), lang_to_cmd(lang), filename]
                                              ^^^^^^^^^^^^^^^^^
  File "/path/to/folder/.venv/lib/python3.12/site-packages/autogen_core/components/code_executor/_impl/utils.py", line 78, in lang_to_cmd
    raise ValueError(f"Unsupported language: {lang}")
ValueError: Unsupported language:

What did you expect to happen?
The message to be processed as usual.
How can we reproduce it (as minimally and precisely as possible)?
Explained in the beginning.
AutoGen version
0.4.0.dev3
Which package was this bug in
Core
Model used
gpt-4o
Python version
python3.12
Operating system
Ubuntu 24.04
Any additional info you think would be helpful for fixing this bug
No response

Modified the issue description to use "````" quote for the output block.
You can see this markdown code block in the output:
```
pip install yfinance
```

It doesn't have a language annotation. So, the language annotation becomes an empty string. The Executor's code executor cannot understand it.
To fix this we can throw a value error in the Executor if the language is not detected.

In the given sample outputs, the code executor gets 2 code blocks, the first of which fails to execute because it has no language tagged on it. Realistically, this should be tagged as sh. I wonder if we can just add a section to the code executor's description, basically adding
f"All code blocks must be tagged with one of the following languages: {_executor.SUPPORTED_LANGUAGES}"
this would get around cases where we execute only some of the code blocks and potentially break the execution environment

Yep. Though I was thinking if we can even remove the quick start -- it is not quick but rather complex, and I the Framework Guide already does a job to take the reader through the basic concepts.
We can move the quick start example to a multi-agent design pattern page.

I'm not actually able to reproduce this issue, every run I've tried has chatgpt correctly label the code block as bash (both the dev branch and the dev3 release branch).
@CJFEdu can you try running the example again on the latest release (dev11)? In the meantime I'll add additional prompting to try and ensure an LLM correctly labels their code blocks

Resolved as the quick start is updated.


TypeError: OneAI is not a constructor

file:///D:/programming/projects/actives/javascript/video-maker/robots/text.js:10
const oneai = new OneAI(apiKeys.oneai)
              ^

TypeError: OneAI is not a constructor
    at file:///D:/programming/projects/actives/javascript/video-maker/robots/text.js:10:15
    at ModuleJob.run (node:internal/modules/esm/module_job:193:25)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:530:24)
    at async loadESM (node:internal/process/esm_loader:91:5)
    at async handleMainPromise (node:internal/modules/run_main:65:12)

Node.js v18.12.1

import OneAI from "oneai";

const oneai = new OneAI(apiKeys.oneai)
const pipeline = new oneai.Pipeline(oneai.skills.keywords())
const output = pipeline.run("Hi I'm Michael Jackson and I like doing the moonwalk dance move.")

console.log(output)

Hi Marcuth!
what SDK version are you using?

Use
import { OneAI } from "oneai";

Updated docs to reflect


fix(source): Add check for http status code for importing schema from https

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
PLEASE DO NOT LEAVE THIS EMPTY !!!
Please explain IN DETAIL what the changes are in this PR and why they are needed:

Add check for http status code for importing schema from https

Checklist
~- [] I have written necessary rustdoc comments~
~- [ ] I have added necessary unit tests and integration tests~

[x] All checks passed in ./risedev check (or alias, ./risedev c)

Please fill the pr form about the change


Intel Power Gadget doesn’t work with release 0.6.4

Strangely it works after updating to macOS 11.1 (same IPG ver 3.7.0)

I can confirm that Intel Power Gadget works on 11.0.1 as well (for both 0.6.3 and 0.6.4).


Set --config for htmlhint

I'm trying to get htmlhint working with my htmlhint config file. I have it created and named .htmlhintrc and it is in the root of my project. While htmlhint is working in Ale it seem to be using the default config for the lint rules. No matter what I change in my .htmlhintrc file it doesn't take. According to https://github.com/yaniswang/HTMLHint/wiki/Usage .htmlhintrc is the correct name and it should be in the root dir. Running htmlhint from the command line works just fine.
Doing this approach works great with stylelint. I was hoping the same would be for htmlhint. Is this a real issue, is it something wrong with htmlhint, or is it just me doing something dumb? Thanks
Current Filetype: html
Available Linters: ['htmlhint', 'proselint', 'tidy']
Enabled Linters: ['htmlhint']
Linter Variables:
let g:ale_html_htmlhint_executable = 'htmlhint'
let g:ale_html_htmlhint_options = '--format=unix'
let g:ale_html_htmlhint_use_global = 0
Global Variables:
let g:ale_echo_cursor = 1
let g:ale_echo_msg_error_str = 'Error'
let g:ale_echo_msg_format = '%s'
let g:ale_echo_msg_warning_str = 'Warning'
let g:ale_enabled = 1
let g:ale_fix_on_save = 0
let g:ale_fixers = {}
let g:ale_keep_list_window_open = 0
let g:ale_lint_delay = 200
let g:ale_lint_on_enter = 1
let g:ale_lint_on_save = 1
let g:ale_lint_on_text_changed = 'always'
let g:ale_linter_aliases = {}
let g:ale_linters = {'html': ['htmlhint'], 'javascript': ['eslint'], 'css': ['stylelint']}
let g:ale_open_list = 0
let g:ale_set_highlights = 1
let g:ale_set_loclist = 1
let g:ale_set_quickfix = 1
let g:ale_set_signs = 1
let g:ale_sign_column_always = 1
let g:ale_sign_error = '>>'
let g:ale_sign_offset = 1000000
let g:ale_sign_warning = '--'
let g:ale_statusline_format = ['%d error(s)', '%d warning(s)', 'OK']
let g:ale_warn_about_trailing_whitespace = 1
Command History:
(executable check - success) htmlhint
(started) ['/bin/zsh', '-c', 'htmlhint --format=unix ''/var/folders/7m/5rzsq7915692mlnd1p2m4ny40000gn/T/nvimLAOmxp/4/acuity.html''']
(finished - exit code 1) ['/bin/zsh', '-c', 'htmlhint --format=unix ''/var/folders/7m/5rzsq7915692mlnd1p2m4ny40000gn/T/nvimLAOmxp/7/acuity.html''']
<<>>
/var/folders/7m/5rzsq7915692mlnd1p2m4ny40000gn/T/nvimLAOmxp/7/acuity.html:13:45: The attribute name of [ viewBox ] must be in lowercase. [error/attr-lowercase]
^[[31m1 problems^[[39m
<<>>

The linter will need to be modified to set --config with the filename manually, so it can both check a temporary file and use whatever configuration file.

See the jshint linter for an example. Several other linters do that.

Thanks for the response and for your work on Ale.
I'm confused about next steps. Is this something I can configure in my init.vim where I have all my vim configuration or is there work that needs to be done on Ale to get this going?

I did try putting this in my init.vim
let g:ale_html_htmlhint_options = '--config=~/.htmlhintrc'
with no luck.

Nah, someone needs to modify the VimL code so it automatically sets the configuration file. That's the most important thing.

Good to know. I wish I had more knowledge about this. I have no idea where to start or I'd give it a try. I'm a html/css guy and get lots with this complicated stuff.


Dataset upload not appearing until refresh after dataset upload

The 'my collections' page is not displaying a pending dataset after it is successfully selected from Dropbox. The dataset is not visible until after a refresh occurs, which the expected behavior is that the dataset appears async, once the API recognizes it.
It should be noted that the current main/prod branches display aberrant behavior after seemingly-successful dataset selection as well, with the dataset sometimes not appearing until after navigating away and coming back. With this particular code branch, however, the dataset never appears until after refresh.

There is a whole class of UX rust  bugs requiring manual refresh as changes have been made that need to be revalidated and investigated. See curation ux + bug.

Ah yes, thanks for adding, Brian!

@NoopDog - would it be safe to assume that this problem is now resolved with the recent changes?

Closing per this morning's call ...


Make pegjs AMD-compatible

Here's a start.
The pegjs command-line tool still needs to be updated to spit out AMD-compatible code.

Let me know if this would be accepted. If so, I'll finish the command-line tool and write tests and such.

I have only vague and superficial knowledge of AMD and its typical usage, so I have few questions to understand the situation better:

Are there any other widespread libraries that support it directly (in a similar way to what you suggest)?
Is there a method to convert e.g. Node.js modules or generic code automatically to AMD?
If not, what do developers typically do when they need to integrate a library that does not support AMD directly to their AMD-based application?
Is this the current specification?

Well, that's an interesting question. At the time I wrote this patch, underscore.js would have been the first module that came to mind (and I based this implementation on their implementation)… But they have recently removed AMD support. As of version 1.7, jQuery has support for AMD (see exports.js), and I would guess that a bunch of the other "cool" JavaScript libraries support it… But I can't say for sure, because I don't use them.

A Node.js module that doesn't use require() can be replaced by an AMD compatible module by wrapping it in:
define(function() {
var module = {};
// … Node module code …
return module.exports;
});

(of course, this assumes that the node code uses module.exports instead of exports… But supporting that should be obvious.)

I don't think this case is covered by the AMD spec (it has been a while since I read it), but basically the loader (ex, require.js) will load the plain JavaScript file as normal, and it will be up to the developer to get access to its exports (ex, using global variables, or something similar).

I think so, yes.

Since I wrote this patch, I have become a bit more ambivalent towards AMD. It's useful in some situations, and there seem to be a fair number of people who like it, but I haven't found the available tooling to be great, and I don't think I'll be using it on new projects (in favour of Closure's dependency management). I will be supporting AMD on my projects, but I think it would also be reasonable to go the route of underscore.js and drop support.

Thanks for your detailed explanation. I am too ambivalent towards AMD and it seems I am not alone. Let's close this request.


chore(main): release 1.0.4

:robot: I have created a release beep boop
1.0.4 (2024-12-17)
Bug Fixes

add stability logo (264ed73)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.4
:sunflower:


Update how-to-connect-sync-service-manager-ui.md

Updating tab links

@dcbrown16 : Thanks for your contribution! The author, @billmath, has been notified to review your proposed change.


GH-1311 config templates for native/memory + rdfs and lucene

This PR addresses GitHub issue: #1311  .
Briefly describe the changes proposed in this PR:

added config templates for native store and memory store with lucene and rdfs entailment

@rdf4j-bot run tests


Refactor RHESSIVisibility.from_fits as it is too complex

Reduce nested loops remove or replace with functions
Extract internal functions to the class
As the visibility are ordered by time / energy can probably improve on current implementation

Closed in #32


Add non-functional requirements into the developer guide

Close the 2 issues:

https://github.com/AY2425S1-CS2103-F10-3/tp/issues/78
https://github.com/AY2425S1-CS2103-F10-3/tp/issues/93

For NFR 6 I am just talking about in the event of a save file corruption or mission only. If this happens it should be handled accordingly


the celery worker indefinitely not processing any tasks after airflow-redis pod restarted

Checks

[X] I have checked for existing issues.
[X] This report is about the User-Community Airflow Helm Chart.

Chart Version
8.6.0
Kubernetes Version
v1.20.0

Helm Version
Helm version 3

Description
as part of deployment, the worker pod deployed first after that airflow-redis-master pod recreated. in airflow worker pod, we can see the redis disconnect message and after few retries connected to redis but missing heartbeat from worker node and worker node indefinitely not processing any tasks. The tasks will be in queued status until worker pod restarted.
[2022-05-30 17:53:06,186: ERROR/MainProcess] consumer: Cannot connect to redis://:**@airflow-redis-master.-XX-XXX--env1.svc.cluster.local:6379/1: Error 111 connecting to airflow-redis-master-XX-XX-env1.svc.cluster.local:6379. Connection refused..
Trying again in 12.00 seconds... (6/100)
Relevant Logs
[2022-05-30 17:53:18,213: INFO/MainProcess] Connected to redis://:**@airflow-redis-master-XX-XXX-env1.svc.cluster.local:6379/1
[2022-05-30 17:53:18,228: INFO/MainProcess] mingle: searching for neighbors
[2022-05-30 17:53:19,239: INFO/MainProcess] mingle: all alone
[2022-05-30 17:53:24,246: INFO/MainProcess] missed heartbeat from celery@airflow-worker-0

Custom Helm Values
celery:
celery_app_name = airflow.executors.celery_executor
worker_concurrency = 16
worker_umask = 0o077
broker_url = redis://:airflow@airflow-redis-master.airflow2-env1.svc.cluster.local:6379/1
result_backend = db+postgresql://airflow_airflow2_x:5432/airflow_db
flower_host = <IP_ADDRESS>
flower_url_prefix = /flower
flower_port = 5555
flower_basic_auth = admin:admin,admin1:admin1
sync_parallelism = 0
celery_config_options = airflow.config_templates.default_celery.DEFAULT_CELERY_CONFIG
ssl_active = False
ssl_key =
ssl_cert =
ssl_cacert =
pool = prefork
operation_timeout = 1.0
task_track_started = True
task_adoption_timeout = 600
task_publish_max_retries = 3
worker_precheck = False
worker_log_server_port = 8793

@thesuperzapper , Thanks for responding. We didn't define custom-values.
I have resolved this issue with configuring liveness checks for Worker pod as per below .
https://github.com/apache/airflow/pull/22378
The only problem here is, The worker containers restart happening if both of the workers heartbeat missed and not processing any messages.
command:
{{- if .Values.workers.livenessProbe.command }}
{{ toYaml .Values.workers.livenessProbe.command | nindent 16 }}
{{- else}}
- sh
- -c
- exec celery --app airflow.executors.celery_executor.app inspect ping

@anu251989 we now have a liveness probe for the workers as of chart version 8.8.0.
It will only restart the specific worker that is not responding, and was implemented with PR https://github.com/airflow-helm/charts/pull/766.


stdin detection fails to detect pipes vs. redirected stdin in various situations

Hello! Great tool!
I've come across a couple of situations where the stdin detection breaks.
The first situation is: gobuster works using < filename style IO redirection:
▶ gobuster -u "https://www.google.com/" -q < wordlist.txt
/robots.txt (Status: 200)

But not if the input comes from a pipe:
▶ cat wordlist.txt | gobuster -u "https://www.google.com/" -q
1 error occurred:

* [!] WordList (-w): Must be specified

The second situation is kind of the reverse. When in a bash while loop fed using IO redirection, and a wordlist is specified with -w, gobuster erroneously attempts to read the wordlist from the file being fed to the while loop and fails:
▶ while read url; do gobuster -u "$url" -q -w wordlist.txt; done < urls.txt
1 error occurred:

* [!] Wordlist (-w) specified with pipe from stdin. Can't have both!

This doesn't happen when the while loop is being fed with a pipe (because the size of the stdin device is zero, which is checked for here):
▶ cat urls.txt | while read url; do gobuster -u "$url" -q -w wordlist.txt; done
/robots.txt (Status: 200)

I think this is a difficult problem to tackle because of the various different permutations in which the tool can be run. One option is to avoid doing stdin detection at all, and allow people to specify -w - to force reading the wordlist from stdin; it's not something that is without precedent, but it does add a little something extra for the user.
Another option is to always use a wordlist if one is specified with -w, and only attempt to read from stdin if none is specified. This option has the downside that the command will hang if the user forgets the -w option but provides no words on stdin.

@tomnomnom I encountered the same behaviour before and changed it in this PR: https://github.com/OJ/gobuster/pull/79
It's not merged yet, but the wordlist parameter now takes a - as a value to read from stdin. If you want to try the updated version there are test instructions in the pull request

Ah super! I'll close this issue in that case :) Thanks!


refactor(swc/cli): alias plugin build with wasm target instead of

override

Description:

PR replaces plugin scaffolding config to not override build to specific target, instead introduce aliased one. This is due to overriding target defeats all other commands (i.e cargo test doesn't work out of the box).
Name of new alias will likely change.
BREAKING CHANGE:
New plugin template created via cli now need to specify target for cargo build or use new alias.

Related issue (if exists):

Yes, probably.


Need to make sure tools are disabled when there is no game to edit them with

Need to make sure tools are disabled when there is no game to edit them with

This was fixed.


how to connect safepal wallet

Safepal wallet does not support metamask, so if we want to migrate the coins in safepal wallet address there is no way to connect to the DAPP,
how to connect safepal wallet to this dapp and migrate the tokens?

The pw-up tool already supported safepal: https://github.com/homura/pw-up/pull/19
Please try again.
https://pw-up.vercel.app/


Polyp Segmentation in Endoscopy Image

I propose to add a feature for Polyp Segmentation using an endoscopy image dataset. This feature aims to improve the detection and segmentation of polyps in endoscopic images, which is crucial for early diagnosis and treatment of colorectal cancer.
Kindly assign me the issue under the label of GSSoC'24.
Dataset  : it contains various images by taking endoscoy.
Link : https://polyp.grand-challenge.org/CVCClinicDB/
Model : U - Net model is used for segmentation with using tensorflow library.

Hi @Kaushal-11 can you implement at least 3 models for this project?

Yes, i can implement different models also like DeepLab, FCN, ResNet.

Cool, issue assigned to you @Kaushal-11

Hey, @abhisheks008
UNet model's training accuracy has reached 0.96, and the validation accuracy is at 0.93

Hey, @abhisheks008 UNet model's training accuracy has reached 0.96, and the validation accuracy is at 0.93

What about the rest of the models?

I am working on it

Hey, @abhisheks008
2nd Model ResUnet model's training accuracy has reached 0.98, and the validation accuracy is at 0.95
And 3rd model is under working

Hey, @abhisheks008
I have completed my work on all models.
ResUnet model give accuracy 0.95,
DeepLabV3+ model give accuracy 0.94

I have done my task with 3 model,
so can you suggest me how to add this stuff in repository ?

I have done my task with 3 model,
so can you suggest me how to add this stuff in repository ?

Clone/fork this repo, make the changes in the forked repo and push your code by a PR.

okay, I have done it.
Now check all the stuffs,
If you see any areas that need improvement please inform me.

@abhisheks008
Hey @SrijanShovit,
Thank you for your guidance throughout this project.


Adds flake8-copyright checker

Added flake8-copyright plugin which checks for copyright notices in all python files
Fixed errors

Jenkins please retry a build

Jenkins please retry a build

Jenkins please retry a build


VS addin for adding guard clauses

Should not only support explicit addition invoked by user but should also provide methods of fixing CA violations, including any raised by #62 or #64.

Should be a Roslyn analyzer warning and code fix instead.


feat(console): add Jupiter toggle for api creation & edition

Issue
https://github.com/gravitee-io/issues/issues/7784
Description
Add Jupiter toggle for api creation & edition
Additional context

🚀 CI was able to deploy the build of this PR, so you can now try it directly here
Notes: The deployed app is linked to the management API of APIM master. (Same login and password as APIM master)

I think we should target prototype-reactive branch no?

yes maybe, I've changed


Add CheckM for genome completeness calculation

I propose adding CheckM for genome completeness calculations.

CheckM2 https://www.nature.com/articles/s41592-023-01940-w

Agreed -- I'm OK with using CheckM2 to calculate completeness/contamination.

Completed by https://github.com/jmtsuji/rotary/pull/76


Fix/issue 647 + add snail

fix bug: delete all pets with the same name
fix bug: add snail to UI
I enjoy this extension and want to contribute to it ^^!

fix bug: delete all pets with the same name #647
add snail to UI

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

Project coverage is 17.04%. Comparing base (cd3b4af) to head (d6a74d9).

Files with missing lines
Patch %
Lines

src/panel/pets.ts
0.00%
5 Missing :warning:

src/panel/main.ts
0.00%
3 Missing :warning:

src/extension/extension.ts
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #648      +/-   ##
==========================================
- Coverage   17.06%   17.04%   -0.02%
==========================================
  Files          25       25
  Lines        1260     1261       +1
  Branches      167      167
==========================================
  Hits          215      215
- Misses       1039     1040       +1
  Partials        6        6

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

Thanks for the fix!


Raw HTML is not escaped in the web output

If I store raw HTML in Mongo, it is not escaped in Humongous UI. For example, scripts are executed.

Good point. But due to other commitments, I may not be able to look into this sooner. Request you to fix and raise a pull request I will be happy merge

I think you are confusing open source Humongous, a mongodb database browser with service https://app.humongous.io/. That is paid service and has nothing to do with this open-source free project


Feature/session renew

This renews the session on_before_render if a user is logged in. This prevents sessions and session cookies from expiring after the default lifetime.
Resolves #3995

I'm a little confused by all of this functionality and honestly why we need it. Back in the olden days of concrete5 the session just stayed open until the browser was closed. If you checked the Remember Me checkbox the session was still killed when you closed the browser, but the cookie allowed it to be seamlessly restarted when you came back to the site, so you never knew that you had a new session. Why do we need all of this session refreshing, etc... ? Why can't we just set the session to not expire? Can't we just do all of this with settings?
http://stackoverflow.com/a/19597247/2679577
Why can't we just make the default cookie lifetime 0 (like it always was) – and obviously if someone wants their own defaults they can override them. Then, we make the gc_maxlifetime something like 2 hours, so that if the user is truly inactive for 2 hours their session will get cleaned up. This means that if a user is inactive in their browser for 2 hours they'll get logged out, but if they're active all day they'll never get logged out until they close their browser – which seems right to me.

We needed this because when the cookie lifetime is 0, the remaining time is impossible to see. It's fine with me if we reset the default to 0, shouldn't we still refresh the cookie on page load?

If someone configures their cookie lifetime away from 0, this change should still be around since otherwise they will get the strange logouts we've been seeing.

I honestly don't think it's necessary – if we set this back to 0, we could
simply check the remaining time (for that session tracker add-on) by
checking the garbage collection lifetime value. That way it's assumed that
the moment you load a page, the remaining session lifetime is
gc_maxlifetime.
I just would rather err on the side of less code than more.
Secondarily, this all might be moot, unless we store sessions in the
database
http://stackoverflow.com/questions/25402331/symfony2-sessions-not-working-as-expected-session-keeps-timing-out
On Wed, Aug 24, 2016 at 7:41 AM, Korvin Szanto<EMAIL_ADDRESS>wrote:

We needed this because when the cookie lifetime is 0, the remaining time
is impossible to see. It's fine with me if we reset the default to 0,
shouldn't we still refresh the cookie on page load?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/concrete5/concrete5/pull/4235#issuecomment-242088666,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAgNwZ5fUwVwSJXsMk8pbdWIeRyuSHgZks5qjFgIgaJpZM4Jo9tO
.

@aembler the symfony session doesn't report the time remaining on the existing session unless the cookie has a lifetime.
I'm fine with setting it back to 0, they have their cookie_lifetime overridden anyway, the problem is that if someone has their cookie_lifetime set to anything other than 0, they will be logged out at the end of that cookie lifetime regardless of their activity on the site. Ignoring the fact that we have to determine the remaining length of the session, we still need to support limited cookie sessions properly by refreshing the cookie on page load.

With this code we'd be regenerating a completely new session ID on every
request. That doesn't seem right to me.
On Wed, Aug 24, 2016 at 8:21 AM, Andrew Embler<EMAIL_ADDRESS>wrote:

I honestly don't think it's necessary – if we set this back to 0, we could
simply check the remaining time (for that session tracker add-on) by
checking the garbage collection lifetime value. That way it's assumed that
the moment you load a page, the remaining session lifetime is
gc_maxlifetime.
I just would rather err on the side of less code than more.
Secondarily, this all might be moot, unless we store sessions in the
database
http://stackoverflow.com/questions/25402331/symfony2-
sessions-not-working-as-expected-session-keeps-timing-out
On Wed, Aug 24, 2016 at 7:41 AM, Korvin Szanto<EMAIL_ADDRESS>wrote:

We needed this because when the cookie lifetime is 0, the remaining time
is impossible to see. It's fine with me if we reset the default to 0,
shouldn't we still refresh the cookie on page load?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/concrete5/concrete5/pull/4235#issuecomment-242088666,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAgNwZ5fUwVwSJXsMk8pbdWIeRyuSHgZks5qjFgIgaJpZM4Jo9tO
.

Okay, well lets revisit this later. Here's symfony's issue for this: https://github.com/symfony/symfony/issues/2171

Makes sense..
Do we still want this line ?
https://github.com/concrete5/concrete5/blob/develop/concrete/src/Session/SessionFactory.php#L77
I'm asking because if we want the default session to never expire as long as the browser is open, it looks like this is resetting that value to whatever the max_lifetime

No, we want to make sure that get changed, I'm planning to open a pull today.

Thanks!


[of_flow_manager] Improve README

Add examples of the API utilization to the readme file.

Update: method insert_flow has no documentation, and the format of the request/data is not clear. Can't add flows without this information.

@renanrodrigo please add examples to a separated file, like EXAMPLES.rst


Unable to join a consumer group with the same client

Versions
Please specify real version numbers or git SHAs, not just "Latest" since that changes fairly regularly.
Sarama Version: v1.12.0
Kafka Version: <IP_ADDRESS>
Go Version: 1.9
Problem Description
Hi, I am the author of https://github.com/bsm/sarama-cluster and recently received a bug ticket about users being unable to run multiple consumer instances while (re-)using the connections of the  same underlying sarama.Client instance. After some debugging, I noticed that it's impossible to call broker.JoinGroup concurrently from multiple goroutines (when the client is shared).
 I tried to hack sarama and vary the broker's correlationID but that didn't work either. Kafka seems to be actively blocking the broker connection which means that I need a dedicated broker connection per consumer to make this work but I am unable to obtain multiple Broker instances through the existing client.Coordinator API. Any hints/ideas on what I can do?
The full code:
package main

import (
	"context"
	"log"

	"github.com/Shopify/sarama"
	"golang.org/x/sync/errgroup"
)

const groupID = "my-group"

func main() {
	if err := run(); err != nil {
		log.Fatalln(err)
	}
}

func run() error {
	config := sarama.NewConfig()
	config.Version = sarama.V0_10_0_0

	client, err := sarama.NewClient([]string{"<IP_ADDRESS>:9092"}, config)
	if err != nil {
		return err
	}
	defer client.Close()

	ctx, cancel := context.WithCancel(context.Background())
	defer cancel()

	eg, _ := errgroup.WithContext(ctx)
	eg.Go(func() error { return joinGroup("A", client) })
	eg.Go(func() error { return joinGroup("B", client) })
	return eg.Wait()
}

func joinGroup(name string, client sarama.Client) error {
	req := &sarama.JoinGroupRequest{
		GroupId:        groupID,
		SessionTimeout: 7 * 1000,
		ProtocolType:   "consumer",
	}
	if err := req.AddGroupProtocolMetadata("range", &sarama.ConsumerGroupMemberMetadata{
		Version: 1,
		Topics:  []string{"topic-a"},
	}); err != nil {
		return err
	}

	broker, err := client.Coordinator(groupID)
	if err != nil {
		return err
	}

	resp, err := broker.JoinGroup(req)
	if err != nil {
		return err
	} else if resp.Err != sarama.ErrNoError {
		return resp.Err
	}

	role := "follower"
	if resp.MemberId == resp.LeaderId {
		role = "leader"
	}
	log.Printf("Consumer %q joined group %q as %s/%d (as %s, %d members)", name, groupID, resp.MemberId, resp.GenerationId, role, len(resp.Members))
	return nil
}

The output is:
2017/09/26 13:19:38 Consumer "B" joined group "my-group" as sarama-269af852-e5a8-4521-a902-ac46f1972a61/10 (as leader, 1 members)
2017/09/26 13:19:45 Consumer "A" joined group "my-group" as sarama-1c10fcbe-acda-474b-ba5e-8296f3ea3b92/11 (as leader, 1 members)

Hi, I am the author of https://github.com/bsm/sarama-cluster

I know who you are :)

When I run my example (see below), one of the consumers hangs until the other times out.

Kafka processes requests on a single connection in a strictly serial fashion; until the first message has been processed, the second will not be considered. See https://cwiki.apache.org/confluence/display/KAFKA/A+Guide+To+The+Kafka+Protocol#AGuideToTheKafkaProtocol-Network. Given this, it isn't surprising that the second JoinGroupRequest ends up waiting on the first when using only a single connection.
What is surprising to me is that the first JoinGroupRequest takes more than a few-hundred milliseconds to complete, based on my reading of the spec there's no blocking or slow processing involved there. I wouldn't have expected it to cause this kind of problem.

I am unable to obtain multiple Broker instances through the existing client.Coordinator API

That is for simplicity since in general you should never need multiple connections to the same broker. You can manually clone a broker if you have to by calling NewBroker(oldBroker.Addr()); it won't have the right broker-ID and thus might confuse some metrics but it should get you a second TCP connection.

unable to run multiple consumer instances while (re-)using the connections of the same underlying sarama.Client instance

I'm not super-familiar with sarama-cluster's capabilities in this respect, but Kafka's networking behaviour makes this a tricky proposition, as you've discovered. It is currently impractical to run multiple pure-sarama consumers or producers off of a single client because they will fight over the broker connection in this fashion (particularly consumers on low-volume topics when the request/response cycle can be slow if there are insufficient messages). This isn't really a problem because a single producer or consumer can handle arbitrary topic/partitions so there's usually never a need to have multiple instances.

Hmm, that's what I thought, thanks for clarifying.


Initial structure and unit tests for PMR

Context
This is the first step for implementing the common module that handles a PMR https://github.com/canonical/github-profiles-automator/issues/8
We discussed for the first iteration to include the code from the common module as part of the GitHub Automator charm and later on move it to be its own repo and PyPi module. So the first step is to submit the code alongside the charm code, and in an easy way to later on decouple from the charm.
What needs to get done

Initialise with charmcraft init the code of the charm
Have the code for the module to live inside the src dir, next to the charm.py
Implement the class that creates a PMR
Have some simple unit tests
Add poetry
Ensure the basic tox environments (lint, format, unit, integration` are working

Then Plan A is to have dedicated tox and poetry environments for unit-library and integration-library that will encapsulate the testing of the common module.
Then the unit and integration tests for the whole charm will also include the tests for the common module
Definition of Done

The repo has an OK README
The existing CI is passing
The common modules are tested alongside the simple charm

While working on this issue, I used poetry for handling the dependencies, and tox for running the different environments.
As a north start I used the kafka-k8s-operator repo. Some critical points to expose:

No depends was used from tox
All environments and dependencies were defined as groups in poetry
The tox environments always install the corresponding poetry group's dependencies

By using charmcraft init the code initialised the repo and in the pyproject.toml file it explicitly defined line-length: 99 in ruff, that is used for linting.
To align with this, I updated the line-length in black to also be 99


fix(curriculum): Added introduction to meta tag

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #47118

I could be wrong about this, but I seem to recall someone mentioning that "we" should be avoided in the description.

The meta element is used to add additional information to the page. Like the img element, the meta element uses a self-closing tag. To help the browser display the text on the page properly, add a meta element as a child of the head element and set its charset attribute to UTF-8.

@bbsmooth I've changed the phrasing up and have a draft regex. I still need to test the regex and see if I have to do a full regex or not.

I appreciate the effort, but I still think the description needs work. I think it's too repetitive and verbose. And we aren't requiring forward slashes at the end self-closing tags any more so I would not include one in the description. And according to MDN, the only valid value for charset is UTF-8, so it's not just the most common encoding scheme, it's the only one. And the W3C will throw an error if you use any other encoding scheme. In fact, I believe you don't need to set the charset at all as all browsers will automatically default to UTF-8 and the W3C validator doesn't complain if you leave it out. So perhaps this step ought to just be removed?

I've simplified the description but haven't removed the charset step yet. should we wait for some other input?

Ya, removing a step in a course is not something the two of us are going to decide :smile: There may be other legit reasons for keeping this step.

Yea, I think the placement of the step should be closer to the start. Hopefully we get some more opinions on it.

Hmm, it looks like the charset used to take valid options like ascii and utf-16, but doesn't in HTML5?

the test still accept meta closing tag

@Sboonny the PR seems to be coming together. Are there any other things to fix for the description?


Decision on paid github license in response to project(s) requests

Projects are already asking for paid github for additional functionality. See pricing.
Formulate a policy on this and make a decision.

What functionality is needed? Team or Enterprise?

For one of these projects mentioned that only mirrors code to minbzk: the Team plan would be sufficient, as its primarily about the branch protection feature.
Enterprise is mostly about SSO, having an SLA and advanced auditing features which you might want to use in the future.


[FEATURE] Add toggle functionality to Save Post

Description
I would like to add toggle functionality to Save button.  If the post is already in your saved list, on clicking it should remove and vice versa.
Also, add a color to the Save icon when the post is already in your Favorites list so that the end user knows the post is already saved.
Screenshots

Additional information
No response

Description
I would like to add toggle functionality to Save button. If the post is already in your saved list, on clicking it should remove and vice versa.
Also, add a color to the Save icon when the post is already in your Favorites list so that the end user knows the post is already saved.
Screenshots

Additional information
No response

you can go ahead!

@narayan954, created PR. Kindly check.


Add support for create / update / delete file

I plan to open a PR for this endpoint:
https://developer.github.com/v3/repos/contents/#create-or-update-a-file
and this one:
https://developer.github.com/v3/repos/contents/#delete-a-file

Would like feedback from maintainers on a preferred type for the file contents on update and delete. I coded it as a String, since that's the literal JSON value and passed responsibility for base64 encoding back to the caller (which probably ought to be changed).
Should we just accept base64 encoding responsibility and change the input type? I feel like an Array[Byte] or fs2.Stream[F,Byte] would be preferred.

I think it's fine for now to leave base64 encoding to the caller and transmit to the API what we receive without modifications.

@BenFradet just saw your comment.
I found that API annoying during testing, so I converted the API to use Array[Byte] in commit 02f1cb4.
Since the base64 requirement is due to GitHub's implementation requirement, I can see an argument for the lib performing it for us.
We can revert that commit if you prefer it the other way.

I think usually we try to be as transparent as possible and stick to what the underlying API says. However, if you find that your approach is motivated (seems like it'll be easier on the user in this case) then it's fine 👍 .
I'll review the PR later today.


Default Font Not Found in Workbench Files, Reinistall VS Code

Hello,
the extension used to work well but recently stopped, now setting a custom font from the "Change UI Font" command does nothing but returning this message:

Default Font Not Found in Workbench Files, Reinistall VS

Already tried to reinstall but no luck.

can you provide your vs code version details?

Sure:
Version: 1.85.2 (user setup) Commit: 8b3775030ed1a69b13e4f4c628c612102e30a681 Date: 2024-01-18T06:40:10.514Z Electron: 25.9.7 ElectronBuildId: 26354273 Chromium: 114.0.5735.289 Node.js: 18.15.0 V8: <IP_ADDRESS>-electron.0 OS: Windows_NT x64 10.0.19045

check the source code: the vscode path is hardcoded.

I have the same issue. My vscode version is 1.95.3. I tried different fonts.


point: allow compressed and uncompressed serialization

This PR is a second iteration of a previous attempt to serialize points in both compressed and uncompressed forms.
In this new iteration:

I changed the semantics of the serialization methods to have similar meanings as master. That is: Bytes() and SetBytes() still refer to compressed form, and there're new ones for uncompressed.
I created a new method to transform uncompressed to compressed, which is needed now in go-verkle for HashedNode node types.

For now, I'll keep it as a draft since this branch will be needed for separate go-verkle and go-ethereum PRs.
We're doing further testing and benchmarks to confirm benchmark improvements; then, I will add some comments and open the PR.
All this work is part of improving the replay benchmark in go-ethereum.

I'll close this for now since doesn't seem to be in our roadmap; if for whatever reason this work is useful we can refloat it later.


Prevent LBs from being renamed

The keepers on the LB's "random_id" are obsolete and cause the LBs to be
renamed after the infra hosts are deployed.
The IP's are managed via Puppet so redeploying the hosts is not
necessary.

Checklist

[x] Keep pull requests small so they can be easily reviewed.
[x] Categorize the PR by setting a good title and adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog
[x] Link this PR to related issues.

Succesfully tested locally, hence mergin


Ensure k8s version matches the target version after upgrades

closed by https://github.com/replicatedhq/embedded-cluster/pull/894