Implementing Video Conferencing with ReactJS, NodeJS, and WebRTC

Implementing_Video_Conferencing_with_ReactJS,_Node.md
Article checklist
ATTENTION: In your PR - add the ISSUE (topic form) # that this PR corresponds to. You can do this by also adding a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
If you have not already please go over our Resources Page for more tips and tricks on how to improve our overall technical writing.
Preliminary Checks - Formatting and Structure

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.
[x] Does your article follow any of the suggested structure formats? - see example formats
[x] Is your article properly formatted in Markdown? - see Github Markdown guide
[x] Have you used the correct folder and file structure? - see Contribution Guidelines
[x] Is your article over 750 words? - Tool: Wordcounter.net
[x] Have you used our preferred heading structure? - H3 (###) and up
[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.
[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.
[ ] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images
[ ] In your PR - add a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid -  Grammarly
[x] Please place your article through a 3rd party plagiarism checker? We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[ ] Are your code snippets properly formatted for syntax highlighting - see Syntax guide

Contribution guidelines
For first-time contributors and for more details, see Contributing Guidelines

plagiarism check done

@Dawe-7
Revision done

I've seen the changes you've made but they are not as required. The following are the changes I need you to make:

I noticed that you identified the programming language for the programs but you need to place them inside the backticks for proper code formatting.
Remove your author details from the article. You should first create a new folder in the author's section of your repository. Afterward, create two files named index.md and avatar.jpg. Save your author details in the index.md file and a snapshot of you in the avatar.jpg file.
Add a hero image - This is the image that describes your article. It should be saved as hero.jpg file in the articles folder.

Take a look at this beginner article to understand how to implement the above. It will show you how you should save the different files (hero image and author details) and how programs are formatted.

Thank you,
I believe i have effected the changes.
On Sat, Aug 14, 2021, 5:24 PM Dawe Daniel @.***> wrote:

I've seen the changes you've made but they are not as required. The
following are the changes I need you to make:

I noticed that you identified the programming language for the
programs but you need to place them inside the backticks for proper code
formatting.
Remove your author details from the article. You should first
create a new folder in the author's section of your repository. Afterward,
create two files named index.md and avatar.jpg. Save your author
details in the index.md file and a snapshot of you in the avatar.jpg
file.
Add a hero image - This is the image that describes your article.
It should be saved as hero.jpg file in the articles folder.

Take a look at this beginner article
https://github.com/section-engineering-education/engineering-education/pull/1783/files
to understand how to implement the above. It will show you how you should
save the different files (hero image and author details) and how programs
are formatted.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/pull/3087#issuecomment-898914778,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQLFTAR4XMLCV5BIAHH24BTT42KFDANCNFSM5BJMCSXA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

@SamoraCletus an article of this caliber (quality) can NOT be published and may be closed. 👍

Permit me to ask what is wrong with the quality of the article?
On Tue, Aug 17, 2021, 8:08 PM hectorkambow @.***> wrote:

@SamoraCletus https://github.com/SamoraCletus an article of this
caliber (quality) can NOT be published and may be closed. 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/pull/3087#issuecomment-900560091,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQLFTAWYR7AV2WTXCYUVWIDT5KXT7ANCNFSM5BJMCSXA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Have not read through it throughly - I'm basing this off the level of revision needed in order to get it published 👍

It cannot possibly be a totally wasted effort. I'll make all the necessary
changes🙏
On Tue, Aug 17, 2021, 11:17 PM hectorkambow @.***>
wrote:

Have not read through it throughly - I'm basing this off the level of
revision needed in order to get it published 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/pull/3087#issuecomment-900668525,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQLFTAV6C3QBU46LIQC6TBDT5LNZLANCNFSM5BJMCSXA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

The article is well detailed and adequate. Besides I've gone through the
recommended revisions and they're all minor issues that could be fixed.
Thanks for reconsidering
On Tue, Aug 17, 2021, 11:17 PM hectorkambow @.***>
wrote:

Have not read through it throughly - I'm basing this off the level of
revision needed in order to get it published 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/pull/3087#issuecomment-900668525,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQLFTAV6C3QBU46LIQC6TBDT5LNZLANCNFSM5BJMCSXA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

@SamoraCletus an article of this caliber (quality) can NOT be published and may be closed.

@SamoraCletus, Besides the suggestions, there were major changes that i've made and if you go through my commits you can see those changes. It seems like you did not go through the pre-submission checklist therefore your article had some issues like code indetation, wrong format and improper file saving. Apart from that there were some grammar errors. To avoid article closure because of multiple rounds of revision make sure you implement the suggestions given and revisit your article. I will go through and explain the changes after review in order to avoid the same mistakes in your future articles.

Owing to the fact that this is my first article I made alot of errors, but
i believe subsequent articles will be better than this one. Is it still
possible that this article will published?
On Wed, Aug 18, 2021, 2:46 AM Dawe Daniel @.***> wrote:

@SamoraCletus https://github.com/SamoraCletus an article of this
caliber (quality) can NOT be published and may be closed.
@SamoraCletus https://github.com/SamoraCletus, Besides the suggestions,
there were major changes that i've made and if you go through my commits
you can see those changes. It seems like you did not go through the
pre-submission checklist therefore your article had some issues like code
indetation, wrong format and improper file saving. Apart from that there
were some grammar errors. To avoid article closure because of multiple
rounds of revision make sure you implement the suggestions given and
revisit your article. I will go through and explain the changes after
review in order to avoid the same mistakes in your future articles.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/pull/3087#issuecomment-900749058,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQLFTAQE4ICQD5YWS2D3FSDT5MGJFANCNFSM5BJMCSXA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

The article is well detailed and adequate. Besides I've gone through the recommended revisions and they're all minor issues that could be fixed. Thanks for reconsidering
…
On Tue, Aug 17, 2021, 11:17 PM hectorkambow @.***> wrote: Have not read through it throughly - I'm basing this off the level of revision needed in order to get it published 👍 — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#3087 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQLFTAV6C3QBU46LIQC6TBDT5LNZLANCNFSM5BJMCSXA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .

These "minor" issues should have been fixed before submitting the article.
We can only review articles that are well polished and ready to publish - as we do not provide revision services.
@Dawe-7 please review to see the changes were implemented and that no other changes are required 👍
thank you

Thank you Very Much,
There'll definitely be some major improvements to subsequent articles.
On Thu, Aug 19, 2021, 1:48 PM Dawe Daniel @.***> wrote:

@.**** approved this pull request.
@SamoraCletus https://github.com/SamoraCletus, Thanks for the changes,
the article looks good. Here are a few things you should note before
submitting your article for review and publishing:

Ensure your article is in the correct structure as specified here
https://github.com/section-engineering-education/engineering-education/blob/master/new_contributors/tutorial-template.md
.
Polish up your article - Grammar, readability, and substance are
all being reviewed. Some minor changes i implemented such as commas,
semicolons and full-stops also count and improve readability. They are
things we expect not to ask the author to place where necessary.
You should always place all your code between the backticks and
specify the language.
Avoid long sentences because they are hard to read and they reduce
the readability of your tutorial.

Please go over my changes to help improve the flow and format of your
future articles.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/pull/3087#pullrequestreview-733943035,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQLFTAXCQAG3SXO3MTD5X6LT5T4RRANCNFSM5BJMCSXA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

@SamoraCletus check out the minor edits I have made across the article to improve in the future. Thanks  cc @Dawe-7


feat: DA for terraform-ibm-icd-postgresql

Description
Refer Issue-7145
Release required?

[ ] No release
[ ] Patch release (x.x.X)
[x] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

DA (Deployable Architectures) for terraform-ibm-icd-postgresql
Added supporting documentation for complex object types in DA(Deployable Architectures).

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[x] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

Postgres version validation does not considers the latest version currently i.e. 16 . I have added this in the version validation block.

variables.tf
PostgreSQL instance creation

/run pipeline

Two tests are added for this TestRunStandardSolution has passed but the upgrade test i.e.TestRunStandardUpgradeSolution failed.

/run pipeline

Is the reference architecture svg correct? The blue box has Rd in it, which seems like it should be Pg (but I am not sure, it requires verification)

Thanks for pointing out, I have updated the right image i.e. Pg as provided by IBM Cloud Stencils.

Left a few comments. Its also missing the ibm_catalog.json and catalog-onboard-pipeline.yaml files + the update in .releaserc to trigger the pipeline Once they are added, I can add the offering to our catalog and share the offering ID

Hi @ocofaigh - I have included the changes required. Please review and provide the offering ID

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

@shemau - Should we also change the TF_VAR_existing_kms_instance_guid to TF_VAR_existing_kms_instance_crn in cra-config.yaml ?

@imprateeksh you should update the cra-config.yaml to point to the DA directory (its currently pointing to the fscloud example)

/run pipeline

/run pipeline

Please note the close with comment button was clicked by mistake due to some lag in UI. Hence reopened again.

/run pipeline

/run pipeline

Failure is seen due to default value of configuration.wal_level i.e. replica. I am checking with Joyce on this.

Failure is seen due to default value of configuration.wal_level i.e. replica. I am checking with Joyce on this.

Reverting the value back to hot_standby

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

This looks good. It is consistent with the recent redis DA. It is a little behind the elastic search DA, so an effort is required to standardize the three solutions. eg. the latest support for KMS in a different account is not here yet.

Sure @shemau Thank you for the review.
Can this PR be merged now and then work on a new issue related to latest support for KMS in a different account ?

/run pipeline

/run pipeline

/run pipeline

/run pipeline

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

GitHub release
v3.17.0

Your semantic-release bot :package::rocket:


is_placeholder() implemented

small part of #6928 and #7046
is_placeholder()
I use is_placeholder() to differentiate numpy inputs from tensorflow op inputs, which allows y_true and y_pred to be correctly assigned to the appropriate tf op, without this change the program crashes by trying to incorrectly pass an op to K.placeholder.
is_placeholder can also be useful when enumerating the layers, inputs, outputs, and transforming an arbitrary model. For example anything that walks the graph may or may not want to modify placeholders, such as an automated labeling model to segmentation model converter.
The implementation of is_placeholder was based on _uses_learning_phase and _keras_shape.

Can you explain the use of is_placeholder to me? I noticed its use in examples/variational_autoencoder.py and looked up where it is used in master, looked through the changes of your PR, but its still unclear what it does. Why is it necessary in that example and when should I use it?

is_placeholder = True makes sense for tensors but why would layers also have the option?

@PavlosMelissinos Could you confirm that I've addressed everything? Thanks for the review!

_is_placeholder is a Layer/Node attribute. Therefore K.is_placeholder accepts a tensor, a layer or a node.
a. I think the docstring should be updated to reflect this.
b. Since the tensor parameter can either be a tensor or a layer/node object, do you prefer it like this or do you think it should be a class method of Layer/Node? In the second case, the calls would become layer.is_placeholder(), so you'd get away with having to define the parameters and it would still work for any backend.
For which layers/nodes/tensors should _is_placeholder be set to True? It is False by default, so in case the user would like to enable it, the 'writing your own keras layers' page should be updated with a brief description of the role of placeholder.

I think the docstring and the parameter name should be updated to reflect this.

How about this?
Returns if a Keras Tensor, Layer, or Node contains a `K.placeholder()`.

do you think it should be a class method of Layer/Node?

I think keeping is_placeholder() separate lowers the cognitive load for users, they don't have to know about it unless they need it and putting it in the Layer API might be overly prominent.

For which layers/nodes/tensors should _is_placeholder be set to True?

It should be True when the entity has a K.placeholder() under the hood.

'writing your own keras layers' page should be updated with an example

Good idea, I'll add this when I have a chance.

How about this?
Returns if a Keras Tensor, Layer, or Node contains a K.placeholder().

Yeah, something like that or as a more complete keras docstring:

"""is_placeholder function.
# Arguments
x : Tensor, Layer or Node.
# Returns
True if x is a Keras Tensor, Layer or Node and contains a K.placeholder(), False otherwise.
"""

I think keeping is_placeholder() separate lowers the cognitive load for users, they don't have to know about it unless they need it and putting it in the Layer API might be overly prominent.

I'm not sure that's the case:

cognitive load seems the same to me, except for code completion maybe where there'll be one extra entry.
most users won't have to know about it since it's going to be a member method of an ancestor class (Layer/Node/Tensor).
I don't know about prominence, it's just a few extra lines.

My main objection is redundancy because it would have to be implemented three times instead of once.
I think I'm slightly in favor of turning it into a class member mostly due to encapsulation but redundancy is an issue so yeah, it's not a big deal, either solution sounds ok to me!

It should be True when the entity has a K.placeholder() under the hood.

Last (tangential and slightly off-topic) question: Would it make sense to determine this on the spot? Flags are fast (they don't require any computation) but they're more prone to bugs in general. I know the flag was there before the PR and this is probably out of scope but I'm just wondering if maybe there's a better way.

Based on a conversation with @fchollet there is no longer an independent _is_placeholder() function, since it can be substituted easily with getattr(keras_placeholder, 'is_placeholder', False). Therefore, the separate function has been removed, and all areas correcting where object.is_placeholder was not set have been merged to #7067. Closing this pull request accordingly.


MODEL_TO_ENCODING mapping does not work with Azure naming restrictions

Azure does not allow for . in the model name. tiktoken package is used by other packages e.g., langchain. Patching this naming restrictions in tiktoken seems like the best fix.
Problem:
GPT-3.5-turbo cannot be a valid name in model deployed in Azure

#72

Thanks for bringing this up!
I'm actually not too familiar with the Azure SKUs. Do you have a link to them handy? That way I could attempt to make sure they're covered in the model to encoding mapping in the future.

Basically langchain package uses tiktoken with encoding_for_model() method. So it expects to find a match in the model.py mapping. When deploying a OpenAI model via Azure, one can choose the model name (i.e., in order to match den standard OpenAI model names). However, these model names cannot include any dots in the name. Hence it is not possible to match the gpt-3.5-turbo name. This root cause of this is of course on the Azure side (or one could argue that langchain has the same issue), but I would assume that extending the mapping in tiktoken is the easiest fix for this.

@hauntsaninja Thank you for helping the community with maintaining this encoder package.
Have one thought here. In larger enterprises (on Azure) where you would need to tag and prefix Model Deployments for accounting purposes, having restrictions around or hardcoding model prefixes and model names may not always be a great idea.
I understand this is done so that we validate the pairing of the encoder and model variants.
Given that you have pretty good documentation around model variants and tokenizers used, may I please suggest making these configuration dictionaries parameters with sensible defaults?
Thanks much!

Just hit this same issue, had to manually patch models.py to allow the use of tiktoken for gpt-35-turbo.

@hauntsaninja Thank you for helping the community with maintaining this package.
Currently, when working with popular frameworks in conjunction with models deployed on Azure OpenAI Platform, we get an error message as follows.
KeyError: ‘Could not automatically map "<custom-prefix>-text-embedding-ada-002-<custom-suffix>" to a tokeniser. Please use `tiktok.get_encoding` to explicitly get the tokeniser you expect.'

This is due to the hardcoded dictionary for model name and encoding pairs (MODEL_TO_ENCODING)
Do you think we have an alternative here?
If not, in larger enterprises where you would need to add tag and/or prefix/suffix Model Deployments for accounting purposes, having restrictions on naming may not always be a great idea. As I understand it, this is done so that we validate the pairing of the encoder and model variants. The model names on Azure OpenAI Platform can almost be anything that the customer chooses.
Especially considering most frameworks including OpenAI package use tiktoken package, it may be easier to allow flexibility in this module rather than each of us forking a new repository for this tokeniser package.
Given that you have great documentation around model variants and tokenizers used, may I please suggest making these <model name, encoding> pair configurable with sensible defaults?
Thanks much!

Hi @jerrygb , I don't think model names on Azure OpenAI Platform can almost be anything that the customer chooses. because we only can do is change/create a Deployment model name.
Model name is not equal to Deployment name.
we can not change deployment name than fix this issue.
because there is model name not pair between "gpt-3.5-turbo" and "gpt-35-turbo"

@jerrygb sorry, you are right. I have confirm that we can change code to below, then will fix this issue:

This looks like its fixed with https://github.com/openai/tiktoken/commit/affbd6e50da3f15ea044ea10d71e63d2429b66d4 but not released yet. Do we know when the next release will be?

Thanks all, this was released in 0.5.0


Update the material design to current one ?

Hello ,
are you interested to update the material design of this library to be close to the current design ?
Thanks

Hello,
well that's the point of JFoenix, however MD keeps changing frequently. It's hard to keep up with a very limited resources.
Regards,

I will try to update the design in my free time and do pull request.

All contributions are appreciated :)


[API] Azure DevOps.NET API deployment

Proposal Submission
Proposed title of article
Azure DevOps.NET API deployment
Proposed article introduction
Workflow management, code collaboration, and app development and deployment are all made easier with Azure DevOps. Collaboration between developers, project managers, and other stakeholders is fostered through Azure DevOps.
We'll learn how to continually integrate and deploy.NET APIs using Azure DevOps.
Key takeaways

By the end of this instructional exercise we will have covered the following:

Setting up SQL Azure database
Set up an Azure Application.
Using Azure DevOps, create a Build and Release pipeline.
Update .NET app
Initiate the Pipeline

Article quality
By reading this article learners will understand how Azure DevOps simplifies the management of servers for application deployment. As well as how each machine communicates with Azure DevOps to coordinate app deployment. We will have a clear explanation with well illustrated screen shots to allow learners to understand every bit stated above.
References
N/A

Good afternoon and thank you for submitting your topic suggestion.
Your topic form has been entered into our queue and should be reviewed (for approval) as soon as a content moderator is finished reviewing the ones in the queue before it.

Thank you for submitting your topic suggestion. There seems to be many articles that already cover this topic. Can you tell me how yours will build upon them and be unique to add value to our readers?

@louisefindlay23 Hello From the content online most of the blogs are based on what is already covered in the official documentation and in some steps they are not fully bringing some critical issues of which I intend to provide a one of a kind article that will be Understood by developer at every level on top of that I will provide screen shots to every step to indeed prove that what will be explained actually works. Thanks

Could you tell me what these critical issues are?

@louisefindlay23,For example in this key takeaway  Create an Azure DevOps project and build and release pipelines for it. which is probably the interesting part of the article I have read a few of the blogs and most of them have failed to bring out the context in it among others. Thanks

Thanks, @fabulousDesigns. There seems to be many articles with that key takeaway included. Is your article a more complete package or does it have unique material?

@louisefindlay23 of course it will have more material because I believe knowledge cannot be exhausted I'll also go ahead and show learners the following how to create a SQL Server Database and how to get an Account on Azure DevOps Services and Azure Portal. Thanks


Error when turning on PowerToys Run

Microsoft PowerToys version
0.59.0
Running as admin

[ ] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce
I've been using PT for a number of versions and never used PT Run until today - I wanted to try it. I turned it on from settings and tried the Alt+Space shortcut with no luck. Tried in a number of applications and with desktop selected; most just gave me the context menu from the title bar. I tried checking 'Use centralized keyboard hook' - again, no luck; nothing happened when pressing the Alt+Space keys.  After closing the window, PT Run opens correctly without the centralized keyboard hook box ticked.
2022-08-22.txt
✔ Expected Behavior
PT Run displays the search box when pressing Alt+Space.
❌ Actual Behavior
PT Run does not display initially, but displays after scrolling down on the setting page and returning a crash message.
Other Software
No response

/dup https://github.com/microsoft/PowerToys/issues/14840


ETC,BLD,SEC: quoting to prevent CWE-78, and add branch-checkout-args

The (already-prvileged) github actions user controls most(?)/all of these parameters which need to be quoted before passing to a shell

It's easier to do shell/yaml code review when things are already shell-quoted; even if unnecessarily per shellcheck, It's easier to review when already shell-quoted.

"CWE-78: Improper Neutralization of Special Elements used in an OS Command ('OS Command Injection')"
https://cwe.mitre.org/data/definitions/78.html

Let's release a new version. Thanks for your contribution!

Thanks!
On Tue, Jan 23, 2024, 5:02 PM Diego Prada-Gracia @.***>
wrote:

Let's release a new version. Thanks for your contribution!
—
Reply to this email directly, view it on GitHub
https://github.com/uibcdf/action-sphinx-docs-to-gh-pages/pull/8#issuecomment-1906992729,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAMNS644GGB5STR4AV3PLTYQAXN5AVCNFSM6AAAAAA2NIBYP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBWHE4TENZSHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

https://github.com/ammaraskar/sphinx-action/issues/9
On Tue, Jan 23, 2024, 5:23 PM Wes Turner @.***> wrote:

Thanks!
On Tue, Jan 23, 2024, 5:02 PM Diego Prada-Gracia @.***>
wrote:

Let's release a new version. Thanks for your contribution!
—
Reply to this email directly, view it on GitHub
https://github.com/uibcdf/action-sphinx-docs-to-gh-pages/pull/8#issuecomment-1906992729,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAMNS644GGB5STR4AV3PLTYQAXN5AVCNFSM6AAAAAA2NIBYP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBWHE4TENZSHE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Capitalization of titles for List view of the Calendar

The two titles needed fixing are:
- "start Time" --Change to--> "Start Time"
- "end Time"  --Change to--> "End Time"
Not urgent or anything since it won't break the code in any way, but still this change should be made because it looks better that way : p

Pushed the changes to NewSignup_MemoryLeakTempFix:
https://github.com/HKN-UCSD/hkn-member-portal/commit/3e0ec75074b62e915f3015c37d13497df5783603


How do I reorder or remove items?

I want to remove the battery on a desktop, how?

This is fixed with the latest version :)


Deepspeech workload variants

Juhan thanks a ton for fixing up this PR!
I see some lint tests are failing but other than that LGTM.
I'll merge it in as soon as the lint tests pass.

Juhan thanks a ton for fixing up this PR! I see some lint tests are failing but other than that LGTM. I'll merge it in as soon as the lint tests pass.

Thank you! I fixed the lint tests :)

@chandramouli-sastry will make a PR into this branch with modeldiff fixes for the base model and add modeldiff tests for the 3 newly added deepspeech variants


Hydrate with additional information

Not all pulled data has actors, directors, etc. Is it worth the effort of trying to hydrate this data from external sources?
Should data be consistent across all cinemas?

This should now be in place for the latest release -- https://github.com/alistairjcbrown/hackney-cinema-calendar/commit/693fee0fdfe9f65f86477e0d74d4793f4ece3907
Schema has been updated for a new moviedb property, as well as year in the overview object -- using movie name and year is a good identifier for most movies. For cinemas which don't provide the year (e.g. Picture House), we're relying on The Movie DB to provide good results, or a cache hit from more specific searches done for previously run cinema pulls.


Autocorrect

Discord has regular spell check, but why not autocorrect?
Suggested by Maxbeebs, not me.

C'mon Joe still not labels added yet...
(Oh and at least github isnt blocked in my school) :joy:

Enable BD plugins in settings and get this plugin https://github.com/Metalloriff/BetterDiscordPlugins/blob/master/AutoCorrect.plugin.js should work ;)

Accepting the above comment as a solution, since I don't see this being added as an ED plugin anytime soon.


Issue 43135: attach material outputs to run from final provenance recording step

Rationale
In provenance recording steps, output materials from run were not getting source output application and successor run id that caused the following error in run graph:
java.lang.IllegalStateException: successorRunIdList not populated        at org.labkey.experiment.api.AbstractRunItemImpl.getSuccessorRuns(AbstractRunItemImpl.java:82)        at org.labkey.experiment.ExperimentRunGraph.generateSummaryGraph(ExperimentRunGraph.java:559)        at org.labkey.experiment.ExperimentRunGraph.generateRunGraph(ExperimentRunGraph.java:197)
Related Pull Requests

https://github.com/LabKey/provenance/pull/70

Changes

attach source output application to material outputs of run
add successor runId to output material of run

merging as the failing test is failing in develop and assigned to Matt.


[qtbase] Add asyncify feature

This adds support for building Qt with Emscripten's asyncify when targeting WebAssembly as per

https://doc.qt.io/qt-6/wasm.html#qt-configure-options-reference
https://github.com/qt/qtbase/blob/210f2004df/cmake/configure-cmake-mapping.md

Marking this as a draft for now, since I haven't figured out whether this is actually needed (just linking the downstream app with -sASYNCIFY seems to work without this too), though it looks like it forces the downstream app to use asyncify.
Feedback is of course appreciated.
Checklist

[x] Changes comply with the maintainer guide.
[x] SHA512s are updated for each updated download.
[x] The "supports" clause reflects platforms that may be fixed by this new version.
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

@fwcd Are you still continuing this PR? :)


Binary protocol revamped

It's a bit dirty--for some reason GAT and TOUCH don't work on the non-local storage backends (which is weird because all the logic is [probably wrongly] in CacheImpl).  But for the BINARY protocol and the LOCAL_HASH storage backend, GAT and TOUCH work.

@anthonyalford, @stormehawke, @ragnor can you take a look?

looks good to me. Is there an approve button? Didn't see one.


Feature: Support to make download link easily

Hi 👋
I would like to use this library in production.
So I send a request to support making signed URLs which give time-limited access to a specific R2 resource. (Same as GCS feature one: https://cloud.google.com/storage/docs/access-control/signed-urls?hl=en)

Hi, @Code-Hex
Thank you for requesting!
I am not familiar with that function, so I apologize if I am misinterpreting it.
Surely it's useful to be able to set it up from HonoS3Storage.
I’d like to add this feature.
So, we have to discuss about the interface.
I have two suggestion of this implementation.
1 )
Add sign field to HonoS3Storage option and recieve signedUrl with vars.
const storage = new HonoS3Storage({
  key: (_, file) => `${file.originalname}-${new Date().getTime()}`,
  bucket: "[your-bucket-name]",
  client,
  sign: {
    expireIn: 3600, // add sign field (from @aws-sdk/s3-request-presigner)
  }
});

app.get("/", storage.single("image"), (c) => c.text(c.vars.files.image.signedUrl))

2 )
Extend storage.single, .multiple, .fields, and enable to set sign to each field.
const storage = new HonoS3Storage({
  key: (_, file) => `${file.originalname}-${new Date().getTime()}`,
  bucket: "[your-bucket-name]",
  client,
});

app.get("/", storage.single("image", { expire: 3600 }), (c) => c.text(c.vars.files.image.signedUrl))
app.get("/", storage.fields([
  { name:"image", expireIn: 3600 },
  { name:"pdf", expireIn: 3600 * 24 }
]), (c) => c.json({
  image: c.vars.files.image.signedUrl,
  pdf: c.vars.files.pdf.signedUrl
}))

If you have any other good interfaces or suggestions for improvement, please suggest me!

@sor4chi Thanks for your suggestion. I like the second opinion. because It's clear that each file is associated with an expiration date.

@Code-Hex
Okay, then I will start with opening the custom fields in the core package! Thanks!
I plan to use @aws-sdk/s3-request-presigner internally, do you have any comments or requests regarding this?
https://developers.cloudflare.com/r2/examples/aws/aws-sdk-js-v3/#generate-presigned-urls

@sor4chi No I don't have!
I'm looking forward to implementing this feature!

Hi @sor4chi and @Code-Hex,
The logic may not be the same, but I've created the "SignedRequest Middleware" in my personal repo:
https://github.com/yusukebe/signed-request-middleware
And here's a Japanese description: https://zenn.dev/yusukebe/articles/647aa9ba8c1550#11.ホットリンク禁止---signedrequest編
I think it would be better to make it more general and have it as built-in middleware if there's potential for it to be used by many users. What do you think? However, I'm not fully understanding the methods of GCP and AWS, so it may not be a completely related thing.

Hi, @yusukebe
It is true that it would be more generic to use middleware, but in this case, I want to set a different sign for each field and get the signedUrl at the same time as the post to R2.

I'm actually trying to implement it now, but I may have to change the interface in some way since I haven't designed it in a way that allows for extending field.


Can not change  push  animation in the android platform？ I need a custom animation！

page transitions are not available in Xamarin forms nor in FreshMvvm. You will need to do that yourself, or use any third party nuget library.Jut google "page transitions xamarin forms" and you should plenty of resources to help you get started. I am closing this issue.

OK thank you ! I get it. buy the way, in freshMvvm should add a push model animation，I can push a PR in freshMvvm.

just can degsin to freshMvvmCore and freshMvvmEx two visions, haha

@liuzhenLz : I dont personally agree adding the animation to the core library but agree with the idea to have in freshmvvmextension

@liuzhenLz : Just so you are aware, there is another library called https://github.com/XAM-Consulting/FreshEssentials. Have a look, it has many utilities too


Use appraisal to test against multiple versions of Rails

Thanks Ricky!

Thanks a lot for merging the PR! Do you know if there's any possibility to get the changes in this fork back into rails/verification and have it released on rubygems.org?


Set-SPOTenant -UserVoiceForFeedbackEnabled:$false seems to have no effect on modern pages

I ran Set-SPOTenant -UserVoiceForFeedbackEnabled:$false on two tenants and nothing changes (checked after 24h).
I would expect the "Feedback" link in the lower right hand corner to disappear when setting this value to false.

I will check with @VesaJuvonen who we can check with on Product Team. I would expect the Feedback to disappear too.

I'm now looking for the internal owner so that we can make this fixed. It clearly seems to be a bug on the modern sites for ignoring this setting. Will come back asap.

@VesaJuvonen feel free to alert the owner of the "Get the mobile app" link as well - wouldn't hurt to hide it.

@VesaJuvonen any news?

I'm following this as well. Changing the value doesn't seem to have any effect.

Just pinged again internal threads around this so that we could get this moving. Stay tuned.

Fix for this is currently rolling out worldwide and should be completely out within week or two. Thanks for reporting and let's close this issue when we have full confirmation that the fix is properly working.

Is this setting (Set-SPOTenant -UserVoiceForFeedbackEnabled:$false) expected to hide Feedback button in delve as well ?

I can confirm this is working in several tenants on modern pages. @gurubn, it will only work on modern pages right now, not system pages, like Delve.

It work for me as well but as with others, only on standard pages.  Can we also have a way to hide the "Get the mobile app" button as this only addresses half the issue.  It shows an incredible lack of foresight not to have this sorted when modern was released let alone to still not have a proper solution many months down the line, when clearly upwards of 80% of people don't want this feature.  And whilst we are at it, please can we have a proper web UI setting for this instead of having to resort to PowerShell for everything.

Confirmed working in both targeted and standard release tenants, at least on site pages. As per above posters, there are some pages (like certain application pages) where the button still appears.

@clkmsc was there a delay in how long it took to update? Does it apply to all sites, modern and classic? I've attempted the same but not seeing the Feedback button disappear on my tenant.

Tested it here as well and the Feedback link is indeed gone now from modern site pages. I'd say for consistency and to avoid confusion this issue is not resolved until it also disappears from i.e. the site contents as well. This is currently not the case yet.

@wobba Thank you for this feedback
From my understanding this has been corrected and verrifed that it works now. I am going to close this issue. Please reopen if this issue is not resolved.

Created a new issue to address the inconsistency as mentioned above: https://github.com/MicrosoftDocs/office-docs-powershell/issues/1752

@KoenZomers Thank you for opening a new issue for that so we can keep these separate and track issues to completion. Closing again.

It should be working fine now on all pages

@victorbutuza, I can confirm from testing on multiple tenants that it's now consistently gone on both modern site pages as well as modern site pages. Superb job!

@wobba It looks like this issue has been resolved. I am going to close this issue. If the issue is not resolved for you please re-open the issue.

Just FYI, our official documentation has now also been updated to reflect this change in the behavior of the UserVoiceForFeedbackEnabled option in Set-SPOTenant:
https://docs.microsoft.com/en-us/powershell/module/sharepoint-online/set-spotenant?view=sharepoint-ps#optional-parameters

Don't know how to reopen but no, the issue has not been solved as described in the link above provided by KoenZomers.
The updated documentation for UserVoiceForFeedbackEnabled says that: "When set to $false, both of these links will not be shown anymore." That is not true. The case regards, and has always been about, the feedback button. The "Get the mobile app" button is STILL showing om each and every page.
I wish for the possibility to also have the "Get the mobile app" button deselected. I also wish for the possibility to deselect the feedback and mobile buttons on ALL O365 pages and not only the modern pages!

@inorde, during my tests it did consistently also remove the "Get the mobile app" button when I fiddled the setting, although it did take some time before it was applied (several hours). I've received feedback that this is also how the PG has changed things. Same for the "Feedback" button. It has been removed from ALL modern SPO pages now with the sole exception of the modern SPO Admin pages. So what you're seeing does not seem right and fit with the patch that has been rolled out. Curious to hear if more people are running into this as I can't reproduce this on my tenants.

@KoenZomers, oddly enough it doesn't remove the "Get the mobile app" button from the top page in a site collection (see image), but it seems like it doesn't show up on new pages created under the top one. I will run  run Set-SPOTenant -UserVoiceForFeedbackEnabled:$false again and see if it changes, I'll get back later with the result.

Also a bit confusing that sharepoint admin closed Optionally disable feedback button with the comment NOTE: this only applies to the feedback button; there is no way to disable the “get mobile app” button at this time.

Also a bit confusing that sharepoint admin closed Optionally disable feedback button with the comment NOTE: this only applies to the feedback button; there is no way to disable the “get mobile app” button at this time.

Noticed that one too. Bit of inconsistency going on. Let me try to address that.

Also a bit confusing that sharepoint admin closed Optionally disable feedback button with the comment NOTE: this only applies to the feedback button; there is no way to disable the “get mobile app” button at this time.

Noticed that one too. Bit of inconsistency going on. Let me try to address that.

Any update on this? Customer would like to also hide the mobile app button, but it is still showing up after making the change with PowerShell.

On my tenants the "Get the mobile app" also have returned now. Not sure what's going on. Still trying to get feedback on this.

Until Microsoft get this properly sorted out you can try out my App
Customizer solution
https://kaboodlesoftware.com/products/framework-solutions-for-sharepoint-modern/kaboodle-branding-package/
which adds CCS to the page to hide these pesky buttons.
On Fri, Oct 26, 2018 at 9:44 PM Koen Zomers<EMAIL_ADDRESS>wrote:

On my tenants the "Get the mobile app" also have returned now. Not sure
what's going on. Still trying to get feedback on this.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/office-docs-powershell/issues/508#issuecomment-433374258,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADh00tiOp_ZvRAOYaz9p_XOYVM4Wh6S0ks5uou6qgaJpZM4S-eLX
.

I was able to run this and it worked for me - set-SPOTenant -UserVoiceForFeedbackEnabled $false

Can anyone else confirm this is resolved? The POSH works in multiple tenants from my testing & from what @rachoksh reported...

For me it also works for the "Feedback" link. There was a slight confusion before that it would also impact the "Get the mobile app" link, but it doesn't and wasn't intended to impact it after all.

Ran this command, waited a full day and the button is still there.  Looks like this has been an issue for a number of years now :)

Tested. It seems the "feedback" button got removed after some time on all modern sites. The "get the mobile app" seems to show on my hub site, and not my hub connected sites. This is all very confusing. Please just let us remove them easily and consistently.

Any update on this?, after using Set-SPOTenant -UserVoiceForFeedbackEnabled $false we still have the "Get the mobile app".

@trent-green who is the PM for SharePoint Online.. Trent any thoughts?

@yogkumgit - you can reach out to @trent-green on feedback. He is the PM that owns SPO. I am happy to help however I can but am not a SME on this particular area. Feel free to loop me in with Trent so we can get this figured out. This issue has been open a very, very long time.

@wobba Thank you for reporting this feedback here. This list is focused on content issues and I want to make sure your immediate product issue does not get lost in this list. If you would please open a service ticket the support engineers will be able to identify if there is a bug (highly likely) and escalate to the appropriate product team.

Is there some reason you can’t just recreate the issue and forward it to
support? We’ve been waiting a long time and obviously it’s a bug.
Thanks,
Jake
On Wed, Feb 19, 2020 at 7:27 AM Yogesh Kumar<EMAIL_ADDRESS>wrote:

Closed #508
https://github.com/MicrosoftDocs/office-docs-powershell/issues/508.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/office-docs-powershell/issues/508?email_source=notifications&email_token=ABPBOAHKYXJX7PQYSBKZQZTRDUXTNA5CNFSM4EXZ4LL2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOWXS6PNY#event-3051743159,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPBOAE3DT7XEDJ27OWXBI3RDUXTNANCNFSM4EXZ4LLQ
.

@JakeMadren - I completely agree with you and wish there was a way to do that. The official channel is that the support engineers identify a bug, versus something unique to a tenant that can be fixed with other means, and then they have a process to escalate (with high visibility) to the relevant product teams and chains of leadership. If everyone that is experiencing the issue opens a support request then those will get to the right leadership chains in the product team and prioritized and added to their work queue. With all that being said though, @trent-green , is the PM that owns SharePoint PowerShell and I had a meeting with him and he is aware of these that have been brought up in the documentation feedback and he is working on it. I will keep bringing up this one in meetings too and any official bug reports from support engineering will help drive it home. I know it isn't the best answer and I will do all I can to increase the volume from the documentation side. And I am very much looking forward to updating the documentation when this is resolved.


Snow loss reported incorrectly

Hourly DC snow loss value is incorrectly reported. This issue appears to have been introduced in SAM version 2018.11.11 (SSC 198).
For example for default PV/No Financials case using Fargo, ND MTS 1 weather file with snow enabled,
SAM 2017.9.5 reports total snow loss of 11%/yr, and SAM 2018.11.11 r3 reports 0.82%.
See snow_issue.zip for sample files and details.

Fixed in patch 6b322b36b6081449cde23f83b87093f2cea53400 and develop 6e6bc3c6b5726886c9eebac093a7aec0abc29f16 branches of ssc commits and have 11% per year and match with 2017.9.5 snow loss


Fix tests

puppet-rspec isn't pinned to a version, so we pick up the latest from Rubygems at build time. This, in turn, dictates the version of RSpec we use, which in this case, becomes 2.99.0. This version uses a different syntax around expect. Use Transpec to pick up a lot of them, and then make a few manual changes to fix the rest.

puppet-rspec isn't pinned to a version, so we pick up the latest from Rubygems at build time. This, in turn, dictates the version of RSpec we use, which in this case, becomes 2.99.0.

It would be better to pin the versions.  2.99 isn't a version that's intended to be used other than in the process of migrating to rspec 3.

@alext I see. I just pinned to whatever we were using. I'll pin to 3.0.


Sync gdbclientutils with upstream

Should hopefully bring the 4.2 lldb swift bot green again.

@swift-ci test and merge


ModuleNotFoundError: No module named 'pandas'

Seem to have hit an odd error I can't figure out by digging through the source.  I have a function that processes a csv file on s3 and places the result on s3. When invoking the function through pywren by mapping it over batches of input file names I'm getting:
Traceback (most recent call last):
  File "process_files.py", line 121, in <module>
    df_results = process_csv(file_name)
  File "process_files.py", line 107, in process_csv
    results = [f.result() for f in futures]
  File "process_files.py", line 107, in <listcomp>
    results = [f.result() for f in futures]
  File "/venv/site-packages/pywren/future.py", line 202, in result
    reraise(*self._traceback)
  File "/venv/site-packages/six.py", line 692, in reraise
    raise value.with_traceback(tb)
  File "/var/task/jobrunner.py", line 29, in <module>
  File "/tmp/pymodules/pywren/serialize/cloudpickle/cloudpickle.py", line 718, in subimport
ModuleNotFoundError: No module named 'pandas'

Am in interpreting the docs right in that pywren uses anaconda as execution environment? Is it possible that my local python venv is pickling something incorrectly?
A more general question might be: what are the best practices for using third party libraries with pywren? Should they be separate lambdas called from a coordinating pywren execution? Should I be using stand alone mode?

Hi Chris ! We do in fact use anaconda as an execution environment, but behind the scenes we download a pre-packaged runtime where we've already installed a bunch of packages. Our serialization layer does its best to import/marshal any other libraries your code might depend on, but it can't serialize c-based modules like pandas.
@craig_wilkins on twitter had this problem with pandas, so we tried building him a custom python 3.6 runtime. https://twitter.com/stochastician/status/907264445763567616 is the exchange.
Right now, the way runtimes are built is a little complicated, but we're working on improving it. Can I ask -- how familiar are you with the python / anaconda ecosystem? What about tools like docker? I'd love to figure out the right way of making this work for you.

We solved it two ways:

swapped pandas for stdlib stuff (hammer approach)
Created lambdas with guaranteed runtime environments via chalice and then used pywren to call them in parallel as a coordinator. This means pywren just calls a URL (api gateway) or does a direct lambda invoke.

Right now, the way runtimes are built is a little complicated, but we're working on improving it. Can I ask -- how familiar are you with the python / anaconda ecosystem? What about tools like docker? I'd love to figure out the right way of making this work for you.

I am also interested in packing my own Python runtime. Either Conda or Docker would be good for me. Do you have any instructions for how to do this? Thanks!
In particular, I'd like to use the xarray and NetCDF4 libraries. With Anaconda, the installation is simply:
conda install xarray netcdf4

For your reference, xarray is a multi-dimensional extension to pandas (so it relies on pandas). It reads and writes NetCDF format, which is the most widely used data format in Earth science. I want to see if it is possible to combine them with pywren to process Public Earth science datasets on AWS.

I suspect this issue is related to this https://github.com/pywren/pywren/issues/253. Also I tried pywren with lxml module too.I get same issue. Pywren could not serialise some 3rd party modules.As pointed in this twitter link  https://twitter.com/stochastician/status/907264445763567616, I edited pywren.runtime json configuration to include my modules. But still no success

Right now, pywren is unable to serialize certain third party modules that
are c extensions; for these we necessitate a separate runtime. We're
working on making this better but it's complicated :)
On Mon, Jun 4, 2018 at 10:14 PM, Ratha<EMAIL_ADDRESS>wrote:

I suspect this issue is related to this #253
https://github.com/pywren/pywren/issues/253. Also I tried pywren with
lxml module too.I get same issue. Pywren could not serialise some 3rd party
modules.As pointed in this twitter link https://twitter.com/
stochastician/status/907264445763567616, I edited pywren.runtime json
configuration to include my modules. But still no success
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pywren/pywren/issues/219#issuecomment-394584108, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACPEqhH3jN8Sjv28CLtsz1iSON3v47qks5t5hPJgaJpZM4QCJ1Y
.

Thanks Eric. So, Do you have any plan to include this feature in your near future release?
Looks this is a major blocker to use Pywren library in a production environment.

Yes, although it's still an active research area. I think it's unlikely we'll ever be able to reliably ship c extensions over the wire, because you might be on a mac/windows (common use cases!) and linux can't handle a shared lib. So we're working on two things in parallel:

Make bigger more inclusive runtimes, that are more likely to contain the kitchen sink of what people want (this requires #2 below)
the ability for a user to take an anaconda env and package it up as an environment
the ability to do some crazy stuff behind the scenes to handle the fact that this environment is probably too large for /tmp on lambda

It's just complex and has a lot of moving pieces. But we're hoping to get it in by the end of the summer.

Thanks Eric

@Vaishaal, @ericmjonas that's great - thanks for the link

@ericmjonas and yes, would love to see the beta code if that's different

@cwharland thanks for the pointers - very useful.
Meanwhile I've made a start with building a runtime here (warning, very brittle code): https://github.com/tomwhite/runtimes/tree/custom


Enable interruptible invokeAll

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8106/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8108/
Test PASSed.

alluxio-bot, merge this please


Feat: Description of deals must including rating and prizes.

Describe the feature
I thing we must include ratings , prizes , discount on every deal . As it will make it different and far more attractive than previous.
Add ScreenShots

Record

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

I really wanna to work on this issue , please assign this to me.

thank you sir

i want to solve this issue please assign it to me sir


fix(ui): Fix inconsistent severity colors and icons on dashboard

Description
Problems

Low severity is gray on main dashboard but blue everywhere else
High/important severity is orange-300 on dashboard but orange-200 everywhere else.
SecurityIcon for critical and important on dashboard but CriticalRiskIcon and AngleDoubleUpIcon everywhere else.

Analysis
Find in Files severityColors has 10 results in 5 files

src/constants/visuals/colors.ts
export const severityColors: Record<PolicySeverity, string> = {
    LOW_SEVERITY: 'var(--color-severity-low)',
    MEDIUM_SEVERITY: 'var(--color-severity-medium)',
    HIGH_SEVERITY: 'var(--color-severity-important)',
    CRITICAL_SEVERITY: 'var(--color-severity-critical)',
};

src/Containers/Dashboard/Widgets/ImagesAtMostRiskTable.tsx vulnerability severity
<SecurityIcon
    className="pf-u-display-inline pf-u-mr-xs"
    color={severityColors.CRITICAL_SEVERITY}
/>

<SecurityIcon
    className="pf-u-display-inline pf-u-mr-xs"
    color={severityColors.HIGH_SEVERITY}
/>

src/Containers/Dashboard/Widgets/MostRecentViolations.tsx policy severity
<SecurityIcon
    className="pf-u-display-inline"
    color={severityColors[policy.severity]}
/>

src/Containers/Dashboard/Widgets/PolicyViolationTiles.tsx policy severity
<Button
    style={{ '--pf-severity-tile-color': severityColors[severity] } as CSSProperties}
    className="pf-severity-tile pf-u-w-100 pf-u-px-md pf-u-py-sm pf-u-align-items-center"
    key={severity}
    variant={ButtonVariant.link}
    component={LinkShim}
    href={link}
>
    <Stack>
        <StackItem className="pf-u-font-weight-bold pf-u-font-size-xl">
            {violationCount}
        </StackItem>
        <StackItem>{severityLabels[severity]}</StackItem>
    </Stack>
</Button>

src/utils/chartUtils.ts
export const severityColorScale = Object.values(severityColors);

Solution

src/constants/visuals/colors.ts

Delete unused variables.

src/Containers/Dashboard/Widgets/ImagesAtMostRiskTable.tsx vulnerability severity

Replace SecurityIcon and CRITICAL_SEVERITY with CriticalSeverityIcon
Replace SecurityIcon and HIGH_SEVERITY with ImportantSeverityIcon

src/Containers/Dashboard/Widgets/MostRecentViolations.tsx policy severity

Replace SecurityIcon and severityColors with policySeverityIconMap

src/Containers/Dashboard/Widgets/PolicyViolationTiles.tsx policy severity

Replace severityColors with policySeverityColorMap

src/utils/chartUtils.ts

Replace severityColors with policySeverityColorMap

src/Components/PatternFly/SeverityIcons.tsx

Export noViolationsColor from fadedTextColor in WorkflowCVEs container
Export individual icons
Export policySeverityIconMap

src/css/acs.css

Delete variables for severityColors

Residue

Move colors that are used only in VerticalBarChart and VerticalClusterBar into Compliance container.
Move colors.ts from visuals subfolder to src/constants folder.
Rename some export variables for clarity and consistency.

Checklist

[ ] Investigated and inspected CI test results
[x] ~Unit test and regression tests added~

Testing Performed

yarn lint in ui
yarn build in ui and then compute branch - master for the following

wc build/static/css/*.css
main.css: -244 = 797740 - 797984
total: -244 = 1641178 - 1641422
ls -al build/static/css/*.css | wc -l
0 = 29 - 29 files
wc build/static/js/*.js
main.css: 0 = 3758534 - 3758534
total: 3059 = 9904953 - 9901894
ls -al build/static/js/*.js | wc -l
0 = 80 - 80 files

yarn start in ui

Manual testing

Visit /main/dashboard

TODO

/test gke-ui-e2e-tests

Compliance by standard reminds me

On one side of the coin, thankfully a different blue from low severity.
On other side of the coin, click View all to see severity colors for <50% >50% >70% 100%

widgets do respond to a 1930px+ screen width and expand to a 3x2 layout

If the breakpoint is under our control, it seems like 3x2 could begin somewhere between 1440px (my work laptop) and 1920px (my work panel).

we specifically restricted it to not expand to 4 columns by request

Sure, I get that.


Added support for showManageSubscriptions on Storekit 2

This PR adds support for showManageSubscriptions: https://developer.apple.com/documentation/storekit/appstore/3803198-showmanagesubscriptions

This panel is very useful for testing where one can cancel the subscription and avoid waiting for expire and additional renewals.
This panel is very user friendly allowing users to cancel or change their subscription without going to the App Store or Settings app to find their subscription.

I was looking around for this functionality. If I'm not mistaken this PR does not actually expose this function in iosSk2.js ?
With that being the case, it looks like people can just achieve this at the moment by doing
   const { RNIapIosSk2 } = NativeModules

   RNIapIosSk2.showManageSubscriptions().catch(() => {
      Linking.openURL('https://apps.apple.com/account/subscriptions')
    })

Addressed your request on https://github.com/dooboolab/react-native-iap/pull/2376 @kyle-ssg

released as 12.10.2

Amazing


Review attribute schema: mandatory association with SyncopeUser / SyncopeRole

From chicchiricco on June 21, 2010 11:14:21
Review attribute schema: mandatory association with SyncopeUser / SyncopeRole
Original issue: http://code.google.com/p/syncope/issues/detail?id=11

From chicchiricco on June 21, 2010 09:45:34
This issue was closed by revision r272 .
Status: Fixed


Consider relaxing aiobotocore requirements

The aiobotocore dependency was updated to 2.7.0, which is great because it relaxes the dependency spread of botocore. However it became a semi-pinned dependency while no API changes impacting s3fs happened on the aiobotocore side. Given the fact that s3fs and its parent fsspec are often used are infrastructural dependency (e.g. part of pandas, arrow, airflow, iceberg etc) it might be worth considering relaxing the dependency requirement. This would then allow more flexibility on the side of the project using s3fs.
Given that aiobotocore provides semantic versioning maybe a better approach is to set the dependency on aiobotocore to
aiobotocore>=2.5.3,<3.0.0

cc: @lou-k

+1 would save me some headaches

Ever since being burned by a change in aiobotocore which broke us on a point release...

Understood and that was an expected answer. However, that was two years ago from what I can see in the logs. Is that choice still the right one now?

Is that choice still the right one now?

We do release pretty quickly when a new aiobotocore appears (except the one that didn't build); the problem is more with botocore/boto3 moving ahead of aiobotocore, and there's nothing we can do about that.

Bumping the issue here.  AWS just announced the support for MWAA Airflow 2.7.2
Right now, the only compatible s3fs version that can be installed on airflow 2.7.2 is 0.6.0, released over 3 years ago!.
The reason being Airflow constrains aiobotocore==2.6.0  , and not a single s3fs release supports this version. (there is a commit where this aiobotocore version was used but never made it to its own release.

@martindurant is there any way we can make s3fs work with airflow 2.7.2 requirements?

From airflow 2.8 onwards we will include fsspec with a more recent version. I suggest pushing the MWAA team to upgrade quickly. They seem reasonably on top nowadays.

@bolkedebruin thats good to know. They are faster nowadays, but given how they just announced 2.7.2 last week I  imagine it would take months until MWAA supports 2.8 or higher.

I think https://raw.githubusercontent.com/apache/airflow/constraints-2.7.3/constraints-3.11.txt is the relevant place: this already has aiobotocore==2.7.0 (the release is new).
Your point stands though, maybe we can relax our pin to X.Y.* and be prepared to yank some future release should a breakage happen again sometime.

@martindurant that is for 2.7.3. MWAA supports 2.7.2 which has aiobotocore==2.6.0 (here https://raw.githubusercontent.com/apache/airflow/constraints-2.7.2/constraints-3.11.txt)

Understood: so the problem is MWAA, not airflow itself.

By the way, for future folks facing the same issue on MWAA.
Im checking the difference between aiobotocore versions between s3fs (2.5.4) and airflow 2.7.2 (2.6.0), seems the only commit difference just has minor updates  (only relevant update is aiohttp from  'aiohttp>=3.3.1,<4.0.0', to 'aiohttp>=3.7.4.post0,<4.0.0' .
For now we are going to just have a custom constraints file with aiobotocore==2.5.4  (instead of the original aiobotocore==2.6.0).
That way we can use s3fs==2023.9.2 which is a pretty recent version instead of 0.6.0 (very old).
Would still love to see the minor pinning strategy ( X.Y.*) you mentioned implemented @martindurant , seems to me like a more robust solution given how s3fs by itself is a building block for other packages.

OK, you have convinced me - and also because the conda-forge feedstock once again needs fixing.

I can do this.
And yes this would help very very much. Yesterday a synced release was done by aiobotocore, botocore and boto3 and we were, at airflow, chasing why our builds were failing. S3fs was tying itself to aiobotocore 2.7 - together with pip backtracking (we have complex dependencies) and breaking the builds.
Synced releases are the intention now I believe so this change (and maybe aligning with those Synced releases) would be extremely welcome.
🙏🙏

OK, you have convinced me - and also because the conda-forge feedstock once again needs fixing.

To track - raised a PR to fix the conda-forge PR - https://github.com/conda-forge/aiobotocore-feedstock/pull/73


docs: Big Mendable Improvements

Higher accuracy on the responses
New redesigned UI
Pretty Sources: display the sources by title / sub-section instead of long URL.
Fixed Reset Button bugs and some other UI issues
Other tweaks

@hwchase17 will dm soon with a report and things we have seen that might significantly improve the docs!

thanks @nickscamara!


Include burnt_tokens in balance proof.

Include burnt_tokens in balance proof.
To send BP with burnt_tokens during settlement, you need to use settleChannel2, as adding more parameters to settleChannel is not possible without exceeding the stack size.
This builds on https://github.com/raiden-network/raiden-contracts/pull/1402, so that commits from that PR are included until that is merged. First commit that belongs to this PR is c62ccd3.

is the goal directory of this PR correct? You want to merge into Raiddit, right?

is the goal directory of this PR correct? You want to merge into Raiddit, right?

You're right, I set the target to raiddit.


Clarify how to enter multiple recipient addresses

Just a small clarification to indicate how to properly enter multiple email recipients so they are handled properly.
Related to #16.

Thanks Norman, much appreciated!


use usersite for getting currently installed packages

Why?
There's a bug in our Python module setup where removing a package using poetry will remove it from pyproject.toml but not .pythonlibs. It happens because while .pythonlibs is listed as a user site package directory and is considered a standard lib directory, it is not searched as a lib directory for gathering the list of already installed packages.
There's a secondary problem: a bug in poetry where for a "GenericEnv", getting the set of paths never includes the user site package directory because the shell out command to python includes the -I which means isolate and implies
-s     : don't add user site directory to sys.path; also PYTHONNOUSERSITE.
I'd like to upstream this change in the future, but that would require a larger change (passing a config parameter to some functions and adding them to various call sites.
Changes

Added the user site directory to the paths searched for installed packages if POETRY_USE_USER_SITE is on
Use the POETRY_USE_USER_SITE flag for the other place too, that way we can opt-in to this feature
Disable the -I flag for when getting paths for the generic environment

a bug in poetry where for a "GenericEnv", getting the set of paths never includes the user site package directory because the shell out command to python includes the -I which means isolate and implies
-s     : don't add user site directory to sys.path; also PYTHONNOUSERSITE.
I'd like to upstream this change in the future, but that would require a larger change (passing a config parameter to some functions and adding them to various call sites.

Can we at least file the bug upstream and link here? maybe somebody else will pick up the issue, and you can point them to this PR for inspiration

a bug in poetry where for a "GenericEnv", getting the set of paths never includes the user site package directory because the shell out command to python includes the -I which means isolate and implies
-s     : don't add user site directory to sys.path; also PYTHONNOUSERSITE.
I'd like to upstream this change in the future, but that would require a larger change (passing a config parameter to some functions and adding them to various call sites.

Can we at least file the bug upstream and link here? maybe somebody else will pick up the issue, and you can point them to this PR for inspiration

It might be hard to step a reproducible test case for them since they don't have the other 2 changes in this PR. I plan to try to upstream it to their 1.6 alpha properly soon. Just want to get some prod-level testing in first.


Space key not pause the video

Reproduction Link

https://surmon-china.github.io/vue-video-player/

Steps to reproduce

Play the video
Go fullscreen
Press space key
It exits fullscreen

What is Expected?

Video pauses

What is actually happening?

It exits fullscreen

this might help:
https://github.com/videojs/video.js/pull/3032

Thanks for feedback, i am trying to fix it.

After testing, you can use the videojs-hotkeys plugin to implement hotkey listening, it looks good work, the case code can be seen here,
Hotkey requires the player to be in the focus state to take effect, and wish you a happy one!

The above answers helped me solve the control problem, but after testing, the problem still exists in Firefox. My fix is to add an event handler to the space key. If you have any questions, you can correct me.
CN: 上面的解答帮助我解决了控件问题，但是经过测试，在火狐浏览器下，依然存在问题，我的修复方法是在空格按键上增加一个事件处理。就像下面这样，看起来没啥毛病。
playerIsReady(player) {
      player.hotkeys({
        volumeStep: 0.1,
        seekStep: 5,
        enableModifiersForNumbers: false,
        // 强制使用热键来控制，即便当前空间获得焦点
        alwaysCaptureHotkeys: true,
        playPauseKey: function(event, player) {
          // FIXED: 在FF中，如果当前播放器全屏，防止空格键切换全屏
          // FIXED: In FF, if the current player is full screen, prevent the space bar from switching to full screen
          if (player.isFullscreen()) {
            player.el().focus()
          }
          // Space bar or MediaPlayPause
          return event.which === 32 || event.which === 179
        },

        fullscreenKey: function(event) {
          // override fullscreen to trigger when pressing the F key or Ctrl+Space
          return event.which === 70
        }
      })
    }


Manage swap partition as a hard requirement

Add swap partition management into the bootstrap role
The swap file/partition management logic is carried over from the existing EDPM pre-adoption implementation, but moved out of the nova compute specific scope into generic tasks for EDPM.
Closes: OSPRH-133

The molecule testing is blocked as we cannot use swapon from podman containers, we'd need a delegated driver for that, likely. Any hints to make this unblocked, @raukadah perchance?

the rdo-project job is failed, probably by unrelated reason, because I can see the swap task completed there
https://logserver.rdoproject.org/56/156/814b11000ae0869a6e1559bee243ec11f332ff28/github-check/edpm-ansible-github-rdo-integration-centos-8-crc-singlenode-centos-9-external-compute/79eb0cc/controller/controller/pod/deploy-external-dataplane-compute-n4zc8-logs.txt

/test rdoproject.org/github-check

check-rdo

check-rdo

Just to clarify. The molecule in podman seems to work with this patch after all https://github.com/openstack-k8s-operators/edpm-ansible/actions/runs/5014407281/jobs/8988616358?pr=156 . Do you still want to replace it with delegated driver?

Ci failure looks unrelated https://github.com/openstack-k8s-operators/edpm-ansible/actions/runs/5199776741/jobs/9377637527#step:6:249

/recheck

@kajinamit PTAL

recheck


Closes #1109

Confirmation message is no longer in the corner. At some point a more elegant solution will replace these inline overrides.

Closes #1109


V11 v12 errmsg fix for frozen config

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

https://github.com/starcoinorg/starcoin-framework/pull/251 merge to branch v11_branch


make it compatible with windows (nmake)

I saw this tweet saying that this library didn't compile in windows, so I made it compatible with windows now.

Thank you @cocoa-xu! I dropped one comment.
(btw, the precompiled is still on my backlog, but i will be busy until i shop elixir v1.14-rc this or next week).

Thank you @cocoa-xu! I dropped one comment.
(btw, the precompiled is still on my backlog, but i will be busy until i shop elixir v1.14-rc this or next week).

@josevalim @jonatanklosko thank you for reviewing and suggestions
@josevalim don't worry, I'm still working on the precompiled thing. I need to write a precompilation guide for the elixir_make (for precompiler developers and for users)

@cocoa-xu I have approved the PR, it is up to you to decide if we need the additional notations on mix.exs or not. :) THank you!

@josevalim supporting mingw32 on windows is too much pain, I gave up... have no idea why mingw32-gcc failed to parse erl headers, https://github.com/elixir-nx/stb_image/runs/7577239240?check_suite_focus=true#step:4:43


feat: Create Resources For Mentorship

Is your feature request related to a problem? Please describe.
No, Resource for mentorship
Describe the solution you'd like
Describe alternatives you've considered
Additional context

Assigned to @goelsonali

Assigned to @nverbic


Helper Function $fetchSanctum

How can we use useSanctumClient as a utility / helper in the same way we use $fetch in Nuxt now?

Hey @SDIjeremy, have you checked this page in docs? Usage of useSanctumClient

Yes, my question is is there a way to use useSanctumClient as a global helper without defining it in every place? Similar to how you use $fetch with Nuxt.

@SDIjeremy
Oh, now I see what you mean 😄
Yes, it is definitely possible by creating additional util file, here  you can find more details. You can create sanctum.ts file in the utils directory with the following content (or similar)
export default const $fetchSanctum = useSanctumClient();

It should be auto-imported by your Nuxt application and available in other files as $fetchSanctum, so to make a request you just need to call it like in your example
await $fetchSanctum('/api/endpoint');

Let me try that, I thought I did that already and was getting an error, but let me give it a try again.

I'm getting this error, which is what I was getting before:
Uncaught (in promise) Error: $fetchSanctum is not a function
const { data, error }: { data: Ref<Account.AccountResponse>, error: Ref } = await useAsyncData(`accounts`, async () => {
    return await $fetchSanctum(`/api/accounts`);
});

Not sure why you got this error, but it looks like it won't work anyway, since I tried my recommendation with the util file and got this
A composable that requires access to the Nuxt instance was called outside of a plugin, Nuxt hook, Nuxt middleware, or Vue setup function

I believe that utils import works outside of those scopes and cannot access the defined Nuxt plugin. Since $fetch utility just provides an interface to ofetch library, we cannot do it in the same way with Sanctum plugin.

is there any way we can replace $fetch entirely or maybe instead just force $fetch/ofetch to use the CSRF cookie headers from the authentication?

No, this is actual reason why this package was created, there is no easy way to change the global behavior of Nuxt / ofetch tools. Moreover, it is a bad-practice to use $fetch helper everywhere anyway since it is not always compatible with SSR.


Change statusbar color for each screen

I want to set status bar color for each screen.
E.g. in screen A status bar color is red and  in screen B is blue.
I tried many solutions. But each one has it's own problem.
like:
AnnotatedRegion(
	value: SystemUiOverlayStyle(
		systemNavigationBarColor: navigationBarColor,
		statusBarColor: statusBarColor,
	),
	child: child,
);

or this before return in build method:
SystemChrome.setSystemUIOverlayStyle(SystemUiOverlayStyle(
  statusBarColor: Colors.white
));

My final try :
  static void pushNamed({
    BuildContext context,
    String route,
    Object arguments,
    Color statusBarColor,
    Color navigationBarColor = Colors.black,
  }) async {
    if (statusBarColor != null) {
      await FlutterStatusbarcolor.setStatusBarColor(statusBarColor);
    }
    await Navigator.pushNamed(
      context,
      route,
      arguments: arguments,
    );
  }

but sometimes fails.
and other problem of this solution is when use back button, color not changed.(I write my code for back button in appbar).
and even when in hot reload color change to previous.
How can I handle this problem?

Hi @Hamidrezana
This platform is not meant for assistance on personal code.
Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you


⚠ HDBits has degraded performance

In 4a38f71, HDBits ($HDB) experienced degraded performance:

HTTP code: 200
Response time: 2540 ms

Resolved: HDBits performance has improved in ecbac83 after 19 minutes.


LG Heat Pump temp of water set

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
A possibility to set the temperature of the water
So my question is, how can I change the temp. of Heater Pump water please ?
Thanks
Denis

This is duplicated of #332. Please move to that thread.


Add increase/decrease allowance functions for ERC20

To prevent well known ERC20 approval issues, non-standard functions increaseAllowance and decreaseAllowance need to be implemented.

Fixed in #29

Fixed in #29


Can't remove default value once set

 Issue by marktani
Monday Jul 31, 2017 at 09:51 GMT
Originally opened as https://github.com/graphcool/api-bugs/issues/213

What is the current behavior?
If you remove the default value from the schema file and resubmit it, the default value will be returned in the final schema.
Please share the relevant part of your GraphQL schema and all functions, permissions or other project settings for easier reproduction
Start with this:
type A implements Node {
  id: ID! @isUnique
  title: String @defaultValue(value: "a")
  createdAt: DateTime!
  updatedAt: DateTime!
}

Note that title has a defaultValue. Now, remove the defaultValue and submit:
type A implements Node {
  id: ID! @isUnique
  title: String
  createdAt: DateTime!
  updatedAt: DateTime!
}

The returned schema will still contain the default value.
What is the expected behavior?
The returned schema should not have the default value anymore.

 Comment by marktani
Wednesday Aug 02, 2017 at 08:51 GMT

Also see https://github.com/graphcool/console/issues/1250

@do4gr Did we fix this with the recent change to how we handle null values?

Current behavior: Schema change is not applied, no warning or error. You can only do 'reset changes', which puts the default value back in...

We have a fix ready for this and will soon deploy it.

This is fixed 🙂


Adding a front end

Hi. Would it be possible to add a wiki page or some pointers in the readme on some ideas about how to add a front end. I am a bit overwhelmed with all the different files.
I am guessing it has something to do with adding a front end file to server folder and making some adjustments to the manifest.js?

Hi @andersea, there are many options/routes to adding a front end, however ideally a front end project would be completely separate from the appy server.  For example, you might write an angular app in a separate project.  appy simply provides an api that the front end would communicate with for services such as logging in, authenticating requests, and accessing data.
I'm currently working on a front-end portion for appy. I'm not sure when it will be complete but I'm expecting it will make it much more clear how the appy system should function as a whole.

Very useful advice. Thanks!


fix: top nav z-index

z-index of the top nave in page was more than Modal.

closing since modified in another commit.


Add Carousel to Competition

Please add the following text:
"Win Tickets to Townlands Carnival Festival!"
A Pair of weekend camping tickets to the 3 day music and arts extravaganza are up for grabs for this new festival in Macroom, Co. Cork. Enter now!"

Sorry the link to click through to is:
http://festivals.ie/competition/show?code=townlandscarnival


fix: add supportsPartialDelete property to registry

What does this PR do?
Adds a supportsPartialDelete property to registry entries to denote that a metadata type should do a deploy when only part of the files that make up the metadata are deleted, rather than a delete.
What issues does this PR fix or reference?
@W-11385865@
@W-11828599@

we should add a note in the adding metadata section to describe what this new field is, what it denotes, and some examples of metadata types that use it


Right-click comment/uncomment

I would like to have the following feature: Mouse right-click comment and uncomment code menu for a line and multiple lines, at least for python or c/cpp.
Thank you,
(Revisiting this request from last year.)

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Setup Consistent Nightly Versions for Pip and Conda

See https://github.com/rapidsai/rmm/pull/1347

/merge


Implement template search

Is your feature request related to a problem? Please describe.
The scope of the issue is to implement a search of templates by name here.
Describe the solution you'd like
You can check out the pod search in node details and implement it similarly.

I would like to solve it

sure thing @kom-senapati


RDS returns nothing - EC2 works fine

Regardless of the time period and length, rds just never returns anything
`discovery:
exportedTagsOnMetrics:
rds:
- Name
jobs:

type: rds
enableMetricData: true
regions:

af-south-1
metrics:
name: CPUUtilization
statistics:

'Maximum'
period: 300
length: 1200

`

I am facing the same/similar issue on RDS DB instances metrics, they are not visible through YACE when there is no tag applied on the AWS RDS DB instance level.
After I am adding a tag on RDS DB instance in AWS: something like: "Name: db-name", YACE started scraping this DB instance metrics.
So question is: is a minimum of one tag needs to be applied on AWS RDS DB instance in-order for YACE to scrape it's metrics?
YACE config looks like below:
discovery:
jobs:

type: rds
roleArns:

"arn:aws:iam::111111111111:role/iam-role"
regions:
us-west-2
period: 300
length: 900
delay: 120
additionalDimensions:
DBClusterIdentifier
metrics:
name: CPUUtilization
statistics: ['Average', 'Minimum', 'Maximum']
nilToZero: true

Any input on this is really appreciated! if this is the defect, can it be please addressed?

@rsvas I ran into this same issue with a similar config to yours.
The default dimension is "DBInstanceIdentifier", so when I leave out the dimensions conifg part out, metrics work for that dimension only. I also tried to specify "DBClusterIdentifier", "Role" and even "EngineName" but no luck getting additional dimensions to work for rds.
I believe there might be something missing in the logic for the rds "additionalDimensions" config option.

Looking more into it, it appears to be a known issue:
https://github.com/ivx/yet-another-cloudwatch-exporter/issues/113

This should be fixed by newest release. Please comment if not. Closing it.

I am seeing the behaviour described in the ticket with v0.28.0-alpha. RDS instances without tags are ignored, but adding any tag to the instance causes it to appear in the metrics.


latte-dock: 0.10.4 -> unstable-2022-09-06

Description of changes
Version bump.
I also remade the autostart patch, since the old one didn't apply properly anymore.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

bump?

Thank you for this work. The homepage URL seems to be outdated? I guess the project source repo has been moved

It might've. Given that it's been a while since the last release, I suppose it would make sense to switch to master branch instead.

Ok, I bumped it to the latest git code. It seems to have new Plasma features support, like accent colour, as well as Wayland thingies. I haven't tested this with Wayland, but it works with X11 just fine.

Please change version (in src, commit and PR title) to unstable-XXXX-XX-XX

I'm going to remove myself from the maintainers list for this package - I haven't used latte-dock for quite a while so I'm not really useful for testing it.

Please change version (in src, commit and PR title) to unstable-XXXX-XX-XX

Done.

Ok, I bumped it to the latest git code. It seems to have new Plasma features support, like accent colour, as well as Wayland thingies. I haven't tested this with Wayland, but it works with X11 just fine.

I'm using latte-dock 0.10.4, right click the dock and choose Add widget crashes latte-dock, and I can't use left-click to open a new application instance.
I wonder if these problems still exist on the latest version？

Sorry for noticing this so late but I guess latte-dock has moved to the KDE source repository https://invent.kde.org/plasma/latte-dock
We should prefer this source since I guess GitHub is only a mirror. Further there are newer stable tagged release 10.4.8. Would be really cool if you can upgrade to this :)

Sorry for noticing this so late but I guess latte-dock has moved to the KDE source repository https://invent.kde.org/plasma/latte-dock We should prefer this source since I guess GitHub is only a mirror.

Done.

Further there are newer stable tagged release 10.4.8. Would be really cool if you can upgrade to this :)

Do you mean v0.10.8, perchance? It's already 7 months old and doesn't include the wayland changes and plasma 5.25 support. It would be more wise to stick to the latest git code until a new release arrives.

bump?

more bump?

Result of nixpkgs-review pr 176895 run on x86_64-linux 1

1 package built:

latte-dock

Thank you, really good update :)


Manually add or remove items from the list after awesomplete is created and populated

I need to be able to add or remove single items to the suggestion list after I've already assigned the awesomplete its initial data. This seems like something that should be really easy to do, but somehow I can't find any explanation of how to do it.
I tried just pushing an item on top of the ._list object and it does add it to the dropdown options; however, I cannot select it from the list afterwards. When I try to select it, the suggestion list closes but nothing gets put in the input field.
Is there an established way to add list items one by one programmatically?

My mistake. Pushing a single item on top of the ._list object did add it appropriately to the list of suggestions. It was not populating in the input field for a different reason.


Android SDK notification handling package

This PR refactors the notification handling code for Android into a package bundled with the breez_sdk maven dependency.
C-Breez PR https://github.com/breez/c-breez/pull/794
The breez_sdk_notification package is used to define a foreground service and jobs needed to be executed for each notification type. The package has a NotificationHelper to define notification channels and trigger notifications. A ResourceHelper to load resources via resource ID or meta-data, falling back to a default resource when one is not defined by the application.
To sdk-flutter we add a flutter_secure_storage package which includes a FlutterSecuredStorageHelper to read and decode secured values. This can be used then separately from the breez_sdk_notifcation package in flutter based apps.
The application can simply extend the breez_sdk_notification.ForegroundService and override getConnectRequest:
class ExampleForegroundService : ForegroundService() {
    companion object {
        private const val TAG = "ExampleForegroundService"
    }

    override fun onCreate() {
        super.onCreate()
        registerNotificationChannels(applicationContext)
    }

    override fun getConnectRequest(): ConnectRequest? {
        val apiKey = applicationContext.getString(R.string.breezApiKey)
        val glNodeConf = GreenlightNodeConfig(null, null)
        val nodeConf = NodeConfig.Greenlight(glNodeConf)
        val config = defaultConfig(EnvironmentType.PRODUCTION, apiKey, nodeConf)

        config.workingDir = PathUtils.getDataDirectory(applicationContext)

        return readSecuredValue(
            applicationContext,
            "${ELEMENT_PREFERENCES_KEY_PREFIX}_${ACCOUNT_MNEMONIC}"
        )
            ?.let { mnemonic ->
                ConnectRequest(config, mnemonicToSeed(mnemonic))
            }
    }
}

Extend the breez_sdk_notification.MessagingService with your preferred messaging handler and override startForegroundService:
class ExampleFcmService : MessagingService, FirebaseMessagingService() {
    companion object {
        private const val TAG = "ExampleFcmService"
    }

    override fun onMessageReceived(remoteMessage: RemoteMessage) {
        super.onMessageReceived(remoteMessage)
        configureLogger(applicationContext)
        Logger.tag(TAG).debug { "FCM message received!" }

        if (remoteMessage.priority == RemoteMessage.PRIORITY_HIGH) {
            remoteMessage.asMessage()?.also { message -> startServiceIfNeeded(applicationContext, message) }
        } else {
            Logger.tag(TAG).debug { "Ignoring FCM message" }
        }
    }

    private fun RemoteMessage.asMessage(): Message? {
        return data[Constants.MESSAGE_DATA_TYPE]?.let {
            Message(
                priority,
                data[Constants.MESSAGE_DATA_TYPE],
                data[Constants.MESSAGE_DATA_PAYLOAD]
            )
        }
    }

    override fun startForegroundService(message: Message) {
        Logger.tag(TAG).debug { "Starting ExampleForegroundService w/ message ${message.type}: ${message.payload}" }
        val intent = Intent(applicationContext, ExampleForegroundService::class.java)
        intent.putExtra(Constants.EXTRA_REMOTE_MESSAGE, message)
        ContextCompat.startForegroundService(applicationContext, intent)
    }
}

To override a resource simply add a resource with the same id or add meta-data to set it:
<?xml version="1.0" encoding="utf-8"?>
<resources>
    <string name="lnurl_pay_metadata_plain_text">Pay to Example App</string>
</resources>

or
<?xml version="1.0" encoding="utf-8"?>
<manifest xmlns:android="http://schemas.android.com/apk/res/android" package="com.example.app">
    <application android:label="Example App" android:name="${applicationName}">
        <meta-data
            android:name="lnurl_pay_metadata_plain_text"
            android:value="Pay to Example App" />
    </application>
</manifest>

@dangeross that looks good to me. Do you think we can hold the merge untill this is merged: https://github.com/breez/c-breez/pull/783
Then we can add the swaps notification job to the sdk as well?

@dangeross Do you think we can hold the merge untill this is merged: breez/c-breez#783 Then we can add the swaps notification job to the sdk as well?

Yes, makes sense


Fix watcher sample CR

Currently it is missing the minimal databaseInstance field so it is failing to deploy.
I'm also renaming it to watcher.

For reference to the error in CI:
https://logserver.rdoproject.org/15/15/7c87b769e9eac4e5c37ae8e5b15293147faca040/github-check/watcher-operator-validation/30d58f4/controller/ci-framework-data/logs/openstack-k8s-operators-openstack-must-gather/namespaces/openstack/crs/watchers.watcher.openstack.org/watcher-sample.yaml

/lgtm

/approve


How to suppress repaint of hidden Views?

My excuse beforehand as it is not a bug, but i have no idea where to mention it.
When i go to another View and current one is being hidden it continues rebuild itself (why???) and affects performance despite of invisibility. So maybe there is some workaround or callback to process this situation?
Here is a code:
class PerformanceView extends StatelessWidget {
  @override
  Widget build(context) {
    final Controller c = Get.put(Controller());
    return Scaffold(
      appBar: AppBar(title: Obx(() => Text("Clicks: ${c.count}"))),
      body: Center(
        child: RaisedButton(
          child: Text("Go to Other"),
          onPressed: () => Get.to(
            OtherView(),
          ),
        ),
      ),
    );
  }
}

class OtherView extends StatelessWidget {
  final Controller c = Get.find();
  @override
  Widget build(context) {
    return Scaffold(body: Center(child: Text("${c.count}"),),);
  }
}

class Controller extends GetxController {
  Controller() {
    Timer.periodic(Duration(milliseconds: 100), (timer) {
      count.value = timer.tick;
    });
  }

  var count = 0.obs;
}

This is the expected behavior in a stack framework (like Flutter).
It is precisely because the views below are being rebuilt that you can view the updated data when you pop it, since only the push reconstructs the screen, the pop does not call the build again.
This does not degrade performance, this is precisely a Flutter performance feature, since we will not have an entire screen being reconstructed when the last route leaves the stack.

It sounds good but screenshots with performance not so positive ))

There are alternatives in the mobile world if you don't like working with Stacks, like the native android (I say this because the behavior mentioned is not related to Get, nor to Flutter, but to stack frameworks).
However, analyzing the graph I also don't see any performance problem in the screenshots, the project is running in debug (that only the VM alone already spends more than 45mb, and consumes much more CPU and GPU than in profile / mode release), I am sure that if it is run in profile / release mode, this would not be a problem even on a 2015 device. Also in the example you are updating the same variable in widgets of the same hierarchy, a single Obx in the Column widget would be appropriate for that situation.