Re-change the options for rebuild index batch number

I'm so sorry. the configuration file was wrong in the previous PR #540. Let's correct that in this PR.
:sweat_smile:

Hold on for one sec.

I discussed it with @critical27 , rebuild_index_batch_num is not very effective as the speed limit. added the new option rebuild_index_process_interval for speed limit.


Add "info" variant to the StatusIcon component

For use in places like https://github.com/konveyor/virt-ui/blob/master/src/app/Plans/components/Wizard/VMConcernsIcon.tsx#L25-L32 and https://github.com/konveyor/virt-ui/pull/163

@mturley, even if it's not used for aforementioned PR, I think it'd be good to have it.

@gildub yep I agree, there's at least one other place I want to use it. I think maybe we should also add a "loading" variant that puts a spinner in place of the icon and spaces the text next to it properly.


Add Solution in Swift Language

Solutions are welcomed for each problem in Swift language. Please check contributing Guidelines before starting.  You can register for Hacktoberfest here to receive free swags once you raise 4 PR.

[ ] Day 0 Assigned to @penelopearaujo
[ ] Day 1
[ ] Day 2
[ ] Day 3
[ ] Day 4
[ ] Day 5
[ ] Day 6
[ ] Day 7
[ ] Day 8
[ ] Day 9
[ ] Day 10
[ ] Day 11
[ ] Day 12
[ ] Day 13
[ ] Day 14
[ ] Day 15
[ ] Day 16
[ ] Day 17
[ ] Day 18
[ ] Day 19
[ ] Day 20
[ ] Day 21
[ ] Day 22
[ ] Day 23
[ ] Day 24
[ ] Day 25
[ ] Day 26
[ ] Day 27
[ ] Day 28
[ ] Day 29

I'd like to add solution for Day 1 & Day 2 :)

Right! I can do the next three - days 3, 4 and 5

I can work on days 6 and 7

@penelopearaujo @rogerthat39 @kristoft , assigned to you 👍

I'd like to continue working on this, could I work on days 8, 9 and 10?

sure @rogerthat39 go ahead 👍

I'd like to add the solutions for Day11 to Day14.

@ham118 go ahead 👍

@sagarchoudhary96 I'd like to provide solution for Day16 .

@Kuljeet-123 sure go ahead will assign to you.

@sagarchoudhary96 please review my PR and merge into your repo and my Pr id is #80 .

Done @Kuljeet-123  looking forward for more contributions.

@kristoft @penelopearaujo @ham118 @rogerthat39, we are halfway completing this task. would be great
if you guys would like to take up more questions to solve.

I could work on days 17-19?

@rogerthat39 sure go ahead.

@rogerthat39  any updates, is any one of 17-19 is completed? you can raise PR for individual solutions too.

@sagarchoudhary96 17 and 18 are now completed, but 19 is not available in Swift. If you like, I could do Day 20 instead?

@rogerthat39 sure go ahead

@kristoft @penelopearaujo @ham118 @rogerthat39 @Kuljeet-123  we are about to finish this.  would you guys like to take up more questions from available on?

@sagarchoudhary96  I would like to take Day 21 ..please assign me this.

@sagarchoudhary96  I would like to take Day 22 ..please assign me this.

@Kuljeet-123 @saurabh-7777  go ahead :)

@sagarchoudhary96  please review my PR and merge into your repo and my PR id is #92 .

@sagarchoudhary96  please I have added Day 21 solution please review it and merge into your repo and my PR id is #93 .

@sagarchoudhary96 I'd like to add a solution for Day23.

@ham118 go ahead👍

@sagarchoudhary96 Please review PR #96

@sagarchoudhary96 There is no option for a swift compiler on Day 27. Can i take Day 26 challenge "Nested Logic" ?

Yeah sure go ahead

@sagarchoudhary96 thank you, its done. If you can check my pr and test it on hackerrank.

Great. Sure will check in a while :)

@sagarchoudhary96 i also added day 28.


[Snyk] Security upgrade sphinx from 1.5.1 to 3.3.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

docs/requirements.txt

⚠ Warning
requests 2.20.1 has requirement urllib3<1.25,>=1.21.1, but you have urllib3 1.26.5.
Jinja2 2.11.3 requires MarkupSafe, which is not installed.

Vulnerabilities that will be fixed
By pinning:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

658/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 5.3
Regular Expression Denial of Service (ReDoS) SNYK-PYTHON-SPHINX-5811865
sphinx:  1.5.1 -> 3.3.0
No
Proof of Concept

658/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 5.3
Regular Expression Denial of Service (ReDoS) SNYK-PYTHON-SPHINX-5812109
sphinx:  1.5.1 -> 3.3.0
No
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Regular Expression Denial of Service (ReDoS)

Codecov Report
Patch and project coverage have no change.

Comparison is base (0aa947e) 89.24% compared to head (ada723d) 89.24%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff            @@
##             master      #19   +/-   ##
=========================================
  Coverage     89.24%   89.24%
  Complexity      379      379
=========================================
  Files            36       36
  Lines          1181     1181
=========================================
  Hits           1054     1054
  Misses          127      127

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


Website: Decrease left margin width for better presentation of long codelines

Before:

After:

Seems reasonable, I think we might just be better off at this point removing the left margin entirely - the React docs don't have any and look very good. Can you update? Will merge as soon as you have it.

@enaqx - haven't heard back from you so I made this change myself in 18a3d715cebcecec48634943ac631dd38d59b84b, thanks!

It's OK. I've removed original repo and couldn't make changes to commit. So I've thought about opening new PR anyway. Thanks for acceptance. :smiley:


Azure: Adding script for custom role creation

[!warning]
This is a public repository, ensure not to disclose:

[x] personal data beyond what is necessary for interacting with this pull request, nor
[x] business confidential information, such as customer names.

What kind of PR is this?
Required: Mark one of the following that is applicable:

[x] kind/feature
[ ] kind/improvement
[ ] kind/deprecation
[ ] kind/documentation
[ ] kind/clean-up
[ ] kind/bug
[ ] kind/other

Optional: Mark one or more of the following that are applicable:

[!important]
Breaking changes should be marked kind/admin-change or kind/dev-change depending on type
Critical security fixes should be marked with kind/security

[ ] kind/admin-change
[ ] kind/dev-change
[ ] kind/security
[ ] kind/adr

What does this PR do / why do we need this PR?

Added the script for the creation of custom roles needed for control node and worker node to access the Azure resources used by CCM, CSI etc. Default permissions gets cluster-wide permission and we don't want to give wide range of permissions needed.

Fixes  https://github.com/elastisys/mse-internal-docs/issues/200

Information to reviewers

Checklist

[x] Proper commit message prefix on all commits

Change checks:

[x] The change is transparent
[ ] The change is disruptive
[ ] The change requires no migration steps
[ ] The change requires migration steps
[ ] The change upgrades CRDs

Metrics checks:

[ ] The metrics are still exposed and present in Grafana after the change
[ ] The metrics names didn't change (Grafana dashboards and Prometheus alerts are not affected)
[ ] The metrics names did change (Grafana dashboards and Prometheus alerts were fixed)

Logs checks:

[ ] The logs do not show any errors after the change

Pod Security Policy checks:

[ ] Any changed pod is covered by Pod Security Admission
[ ] Any changed pod is covered by Gatekeeper Pod Security Policies
[ ] The change does not cause any pods to be blocked by Pod Security Admission or Policies

Network Policy checks:

[ ] Any changed pod is covered by Network Policies
[ ] The change does not cause any dropped packets in the NetworkPolicy Dashboard

Audit checks:

[ ] The change does not cause any unnecessary Kubernetes audit events
[ ] The change requires changes to Kubernetes audit policy

Falco checks:

[ ] The change does not cause any alerts to be generated by Falco

Bug checks:

[ ] The bug fix is covered by regression tests

Should these not be setup before the clusters? Why is this in apps?

Yeah true !! Do you think cluster API repo I can use ?
And yes in the documentation, we can add this step before the cluster as this role needed to be there and so MI can use it for role assignments.

Should these not be setup before the clusters? Why is this in apps?

Yeah true !! Do you think cluster API repo I can use ?
And yes in the documentation, we can add this step before the cluster as this role needed to be there and so MI can use it for role assignments.

Yes, let's move the discussion over there!

Closing this PR as created a new one in Cluster-API repo - https://github.com/elastisys/ck8s-cluster-api/pull/148


Error in the documentation

Hi,
Scenarios section : scenario 2 contain error 150 instead of 500, please review

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 06201faf-0037-3647-4e91-bb0af80d4812
Version Independent ID: 1b397d76-ecfe-046e-a4c4-0fe5293bceb3
Content: What is entitlement management? - Azure AD
Content Source: articles/active-directory/governance/entitlement-management-overview.md
Service: active-directory
Sub-service: compliance
GitHub Login: @barclayn
Microsoft Alias: barclayn

@googleirfan Thank you for your feedback . We will have this checked and update the thread.

@googleirfan I am checking this internally and provide an update soon.

Adding @ajburnle

@googleirfan This was checked and we found that the 150 users are the employees from the current company out of the total 2000 who requested access. The number 150 does not denote the guest users (external identities) rather in-company employees  . And a total of 10500 external identities requested the access as well . The external identities from contoso are looked at a ratio of 1:5 and that leaves 500 extra users as explained in the table as highlighted below.

Hope this clarifies your query . Should you still have any doubts , please let me know and we will be happy to help you .
Thank you .


Update to use bullet points

@simonaco,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

#sign-off

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/includes/storage-samples.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@simonaco @johnpapa Merged


Add and fix some AST Node parent types

I noticed that MappedTypeNode is not only a child of TypeAliasDeclaration but can also occur everywhere else as type.
While I'm already editing this file, I fixed another bug in VariableLikeDeclaration['name'] and added some more parent types.

@mhegazy This merge just broke master. The SpreadAssignment node type has two parent properties following the merge!

removing it.

should be fixed now by https://github.com/Microsoft/TypeScript/commit/c82881f36ef412cacfe62e3394284e34cd9c4388


Bump identity-apps version to 1.2.997

$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2541071050

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2541071050
Status: success


Theming Slider component like swicth or checkbox support

now slider theme can't change by user !
@FeodorFitsner

With the latest Flet release you can change colours of all switch elements and checkbox. Could you please elaborate what's missing?

Yeah, all switch elements and checkbox can, but  slider widget can't change theme

Oh, right, Slider! :) Sorry, I must be slow on Sunday!
Will implement that tomorrow.


Fix dlangui on Virtualbox and old graphics cards.

Add ability to fall back to software renderer. Libraries updated to 2.0.5 because 2.0.3 crashes on VirtualBox.

Rebased :)

Merged, labeled :)
Thanks!


Adding Overview of Kubernetes Architecture to User-Guide.md File

This is a Bug Report

Problem Description:
Upon reviewing the "user-guide.md" document, I noticed that it dives straight into configuring and deploying a Windows container in Kubernetes. However, this approach may be challenging for newbies who lack prior knowledge of Kubernetes architecture.
To Reproduce:
To address this issue, I propose adding a new section titled "Overview of Kubernetes Architecture" to the user guide. This section will provide an overview of essential Kubernetes components, including control plane components, worker nodes, and the Kubernetes API. By incorporating this addition, newbies will gain a better understanding of the concepts and be able to establish meaningful connections.
Kindly assign me this issue. Thank you.😊

/assign

Thank you so much for your assistance.✨
Can I add the references in the document?

@dipesh-rawat Sir, I also require your assistance on the issue #40740. Kindly review and guide me in this regard.
Thank you.😊

I don't recommend that we accept this issue. I don't see a justification for making this change.
@KiranAminPanjwani, would you consider adding hyperlink instead?

I don't recommend that we accept this issue. I don't see a justification for making this change.
@KiranAminPanjwani, would you consider adding hyperlink instead?

Yes sir.

/remove-lifecycle stale


Upgrade bootstrap.

Upgrade Dependency of bootstrap from Version 3 to Version 4.
Fixes #70

Sorry but you also need to edit the layouts in the project view files.

@ozdemirburak I can do that if you want. Thanks!


Sort telescope results by recency

When I have a lot of files open, I usually end up navigating exclusively through Hbac Telescope. I don't know how feasible it is, but I would love to be able to see the results in it sorted by most recently viewed.
I don't know if nvim has some concept of when a buffer was last open and visibile, and I'm sure it would get a little complicated like if you have splits with multiple files open... https://github.com/nvim-telescope/telescope-frecency.nvim is something in that space, but it has to use a sqlite database to do it.

If by most recently viewed you mean within a session, then its totally doable.

Yep! That is exactly what I mean.
Thanks for the pointer on where to look for implementation.  May try my hand if I get annoyed enough 😁

@technicalpickles Hi! The features discussed here are now in the latest 2.0.0 release! Could you please update your config to use the master branch and check that everything works for you? If yes, we can close the issue.

Hi again 🙂! I am going to close this. You can reopen the issue or create a new one if you find any problems.


Duplicate content in architecture e-books

This issue is to track getting to a clean build in the architecture folder. Currently there are warnings for duplicated H1 headings and metadata titles in ~10 articles in this folder. To resolve the warnings, we could either differentiate the H1s and titles, or we could consolidate duplicated content (as suggested by #19932, #19933, #19934, #19935).
Related to #25322.

Closing this issue, justification noted here: https://github.com/dotnet/docs/pull/25630#issuecomment-900336524

Hi @gewarren - what PR? I think this issue is different than the clean up effort, which is captured in #25322

This is the PR I was referring to: https://github.com/dotnet/docs/pull/25392. It removes all the warnings from the architecture folder.
I'd created this issue as a separate issue because #25322 is assigned to me and I wasn't planning to do the work to clean the architecture folder.

Closed by #25740


SOUPS

@VKGL-Kwaliteit/veldnorm SOUPS part ready for review :)

@TLeenders @rhpvorderman  I think I addressed the comments.


Html - Bold text in ListItem

Great library, thank you for this. Just found one issue. I cannot add a bold text segment inside an unordered list.

Tap to open the phone settings and search for Device Care

this is rendering a list item but the text contain the bold tags. Is this on purpose? How can I achieve that the bold segments are rendered bold as well?

Thank you! I'm glad you like the library.
DailyTags does not support nested tags yet. You can find the full list of supported features here.
However, nested tags support will be added in future releases.
For now, the easiest way to get around this restriction would be to additionally parse the content to reach the nested tags.
    // render list first and retrieve the final string
    val renderedList = parser.parse(source, rules).render().toAnnotatedString().text
    // now render what is inside list (second level tags)
    val renderedBold = parser.parse(renderedList, rules).render().toAnnotatedString()

    Text(text = renderedBold)

Let me know if it works for you :)


Support void controllers

For example,
func (c *Controller) Create() {
  // do something
}

These functions should return 204 No Content if successful.

This seems to work for void controllers already. Changing the Show in the basic example results in a 204.
However, if the func returns a nil error, it is parsed as a 200. Not sure if it would make sense for void and nil error to parse the same.

Thanks for looking into this!

This seems to work for void controllers already.

Okay, so I think it's a bit more nuanced then. I saw an error when you only have one controller action and that controller action had no results. I think it's an "import is not used" error. Should be pretty easy to fix, just moving an imports.AddNamed to a different location in the controller loader.

if the func returns a nil error, it is parsed as a 200

Yep, I totally think it makes sense for func (c *Controller) Action(...) error { return nil } to also return a 204 No Content


[Request] External Storage Feature

Hoping for the ability to snapshot/partition the contents of the chest to the filter of the external storage feature. As well as the ability to upgrade the amount of slots in the filter of it just like in AE. Ender IO already has a lot of slots by default but having more slots through upgrade is fine too.

I'm very confused as to why suddenly an issue from 2016 (that has not even been tagged) comes up.
I think GitHub did something weird?


How does Eventmesh support a cluster of multiple eventmesh runtime instances

In production deployment of eventmesh in cloud environment, eventmesh cluster of multiple instances is required for high availability and fault tolerance.
From the code study, eventmesh runtime using local cache Concurrent Map to store client and session information. For example,

HTTP Server side: localConsumerGroupMapping and localClientInfoMapping
TCP Server side:  sessionTable and clientGroupMap

My question is when multiple eventmesh runtimes are running as a single cluster, how these local cache map each each instance are synchronized?
If these local cache are not synchronized, problem happens when incoming requests are load balanced into the cluster.
For example, a SUBSCRIBE request is routed to eventmesh runtime instance 1. then later UNSUBSCRIBE request is routed to to instance 2,  UNSUBSCRIBE request will failed.  (That's because the actual subscriber is in instance1)

@vongosling

we need to implement the eventmesh-registry-plugin for this.

Thanks @xwm1992 this is a good suggestion for eventmesh-registry-plugin


Fixed error in monsters.js where initiative was defined wrong

Every monster, except the first one, in the monsters list for Tome of Beasts and Volo's has their initiative value in the JSON as "int" instead of "init".

Good catch, thanks!


How do I get Electrolux Wellbeing to ask for credentials/API key again?

How do I get Electrolux Wellbeing to ask for credentials/API key again?
I am running version 1.2.3 but it says v0.0.0 in Integrations
I get this error in my logfile: [pyelectroluxgroup.auth] Failed to get access token: Not enough segments

Test this,
To use this integration you need to login in to https://developer.electrolux.one/ using your Electrolux account details and generate an API key, Access Token and Refresh Token that is needed in the configuration.

I have created an Api Key, an Access Token, and a Refresh Token, but the integration does not ask me to authenticate again.
Where is the previous authentication values for Electrolux Wellbeing saved? Can I delete them and it will ask me again?

It's saved in the config entry. Try remove the integration and add it again.

Removing and adding the integration again solved the problem, but as expected I lost all historical data.


Deleting a Post

Thought this might be useful to someone. You can delete a post from the rails console with:
Post.find(POST_ID).destroy
Replace POST_ID with the numerical ID of your post.
I was hoping there was a method to delete a tag in the API but I don't see it.

Should this be a soft delete? Maybe to have a kind of trashcan for deleted projects in case the user wanted to recover them

Soft delete would be a good idea

I agree. If we can archive posts, that'd be great! Please create a PR for this if anyone ends up implementing this 👍

@milezzz : Post.find(POST_ID).destroy we can use finder this way because of this gem friendly_id. Please refer this doc gem. @nearkingdom @kenny-hibino : Please review the pull request regarding this issue and provide your feedback and let me know If I miss any thing.


Modifying existing object in the cache.

Hello, I was wondering about the expected pattern for a scenario similar to this.
https://www.apollographql.com/docs/react/features/optimistic-ui.html#optimistic-advanced
In this scenario, a user adds a comment and has the comment pushed to the list in the cache without performing a refetch. For apollo-android, if I were to do something similar I'd imagine that would require me to access the ApolloStore and then call write with the updated data for that particular query that populates the list of comments.
How do you address the different models under each query and mutation. Since there could be a CommentsQuery.Comment and a AddCommentMutation.Comment. I have to reconstruct a CommentsQuery.Comment using the builders even though they use the same schema. For larger models this seems like it could get messy, especially in the case for nested data. I know one way this could be handled is to use a fragment shared between both the query and mutation to make it easier. Is there a pattern for this that we could be using that I'm not seeing? Or is there potential for support for easily rebuilding these classes into other model classes that are under the same Schema?
I hope this will help answer some questions for others around optimistic UI for Android. Thanks!

In this scenario, a user adds a comment and has the comment pushed to the list in the cache without performing a refetch. For apollo-android, if I were to do something similar I'd imagine that would require me to access the ApolloStore and then call write with the updated data for that particular query that populates the list of comments.

The tricky part about inserting into the list is that there is no easy way to resolve where this new item should go, what page? As normalized cache for lists based on the key that includes arguments how Apollo store can know if this new item belongs to the cached page or not, for example:
query HeroDetails {
  hero(episode: JEDI) {
    id
    name
    ...
  }
}

If we want to insert a new Character (hero) how do we know that this item belongs to this requested list of heroes for provided episode? Now to make it more complicated if pagination based on cursor, how store can find out that new item belongs to the list filtered by episode and to page with the provided cursor?
Apollo normalized cache can deal with updating models only, as it uses key-value store that shares the same data across queries. For example 2 queries that accessing the same type (like hero) will share the same cached data, of course if there is global unique identifier (hero.id) defined on this type.

How do you address the different models under each query and mutation. Since there could be a CommentsQuery.Comment and a AddCommentMutation.Comment. I have to reconstruct a CommentsQuery.Comment using the builders even though they use the same schema. For larger models this seems like it could get messy. I know one way this could be handled is to use a fragment shared between both the query and mutation to make it easier. Is there a pattern for this that we could be using that I'm not seeing?

That is a pattern, if you want to share models between queries you have to use fragments.

Closing due to inactivity.


[help] bulk smart reschedule

how do I reschedule multiple selected tasks to the date that smart scheduler suggests?
Is there any way to bypass the smart schedule popup so that I can just select, schedule, smart rescheduled? It works like that for a single task (x, then t and then s for smart reschedule)?
I trust AI-scheduler suggests a good date :)

Implemented!

When scheduling multiple items, s now opens the smart
scheduler. Within the smart scheduler, enter applies the updates,
and escape cancels.

works like a charm. Many thanks!


Enable text tile test.

Pull Request Test Coverage Report for Build 1903

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

Totals

Change from base Build 1901:
0.0%

Covered Lines:

Relevant Lines:
0

💛  - Coveralls


Ensembler: trectools integration

There should be an ensembler that implements an interface for  trectools  rank fusion techniques.

Dismissed.


spin plugin list showing two of the same version of cloud plugin installed

I'm currently making changes to cloud plugin and so things are a little odd on my dev machine but it seems like this shouldn't be allowed regardless:
$ spin plugin list
check-for-update 0.0.1
check-for-update 0.1.0
cloud 0.1.0
cloud 0.1.1
cloud 0.1.2
cloud 0.2.0 [installed]
cloud 0.2.0 [installed]

That does seem weird.  What's in your plugins directory?

@itowlson Embarrassingly, I don't remember how to access my plugins directory. Where is it located?

@michelleN On Linux, the plugins directory is at ~/.local/share/spin/plugins.  On Mac, I think it would be at ~/Library/Application Support/spin/plugins but I might be wrong!

Thanks! It ended up being /Users/<username>/Library/Application\ Support/spin/plugins

/Users/michelle/Library/Application Support/spin/plugins/
├── cloud
│   └── cloud
├── js2wasm
│   ├── js2wasm
│   └── js2wasm.license
├── manifests
│   ├── cloud.json
│   ├── js2wasm.json
│   ├── pluginify.json
│   └── py2wasm.json
├── pluginify
│   └── pluginify
└── py2wasm
    ├── py2wasm
    └── py2wasm.license

The plugins directory also contains a hidden .spin-plugins directory which contains a snapshot of the registry.  My guess is that 0.2.0 exists in both that snapshot and in your installed plugins (manifests) directory, but has different manifests (because of cloud plugin local install).  And then spin plugins list sees that 0.2.0 is installed and marks all the 0.2.0s installed maybe?
I'm still a bit surprised it can get into this state though.  Usually when you do a spin plugin install and the version number is already installed it says "already installed" and nopes out.  This is why spin pluginify --install does an uninstall-reinstall cycle.  Can you remember how you installed the cloud plugin that you were working on?

Does this have anything to do with this? https://github.com/fermyon/cloud-plugin/pull/111

Does this have anything to do with this? https://github.com/fermyon/cloud-plugin/pull/111

Ah good call @karthik2804 -- that's probably it.

I'm a bit sceptical.  spin plugins list doesn't use the binary version, only the version in the plugin manifest, which was at 0.3 at that release.

I'm now thinking it is an effect from using the pluginify plugin to install the... plugin.
When I install from the plugin manifest, all appears well, but when I install from a local build via spin pluginify --install, I see the duplicated entries:
$ spin plugins install --url https://github.com/fermyon/cloud-plugin/releases/download/v0.3.0/cloud.json
Are you sure you want to install plugin 'cloud' with license Apache-2.0 from https://github.com/fermyon/cloud-plugin/releases/download/v0.3.0/cloud-v0.3.0-macos-aarch64.tar.gz? yes

Plugin 'cloud' was installed successfully!

Description:
	Commands for publishing applications to the Fermyon Cloud.

Homepage:
	https://github.com/fermyon/cloud-plugin

$ spin plugin list --filter cloud
cloud 0.1.0
cloud 0.1.1
cloud 0.1.2
cloud 0.2.0
cloud 0.3.0 [installed]

$ spin pluginify -i
Plugin 'cloud' was installed successfully!

Description:
	Commands for publishing applications to the Fermyon Cloud.

Homepage:
	https://github.com/fermyon/cloud-plugin

$ spin plugin list --filter cloud
cloud 0.1.0
cloud 0.1.1
cloud 0.1.2
cloud 0.2.0
cloud 0.3.0 [installed]
cloud 0.3.0 [installed]

Wild.  pluginify --install uninstalls any existing installed plugin before installing the dev one.  And all it does is call spin plugin uninstall followed by spin plugin install -f - it's not going behind the system's back.
But yeah... I see it.  And something is odd because you're seeing 0.3.0 whereas spin-pluginify.toml still says 0.2.0 https://github.com/fermyon/cloud-plugin/blob/main/spin-pluginify.toml
Anyway now I can see it I can take a look.  Thanks for the detective work @vdice!

But yeah... I see it. And something is odd because you're seeing 0.3.0 whereas spin-pluginify.toml still says 0.2.0 https://github.com/fermyon/cloud-plugin/blob/main/spin-pluginify.toml

Oh sorry, I was futzing with the version locally when testing.  You're right, it was still 0.2.0 in source.  (Will be updated in https://github.com/fermyon/cloud-plugin/pull/117)

Okay, what is happening is this:

There are two versions both called 0.2.0: the dev version (installed from file) and the released version (not installed, but known about from the registry).
When we run spin plugins list, it enumerates both installed and registry plugins.  This means we get two cloud 0.2.0 plugins, one on the installed list and one on the registry list.
As part of enumerating the registry plugins, we try to determine which are installed.  (Originally, spin plugins list was written to be registry only, but an influential Danish user found it surprising that a plugin appeared on the installed list but not on the "full" list.)  This is done by looking to see if there is one in the installation store with a matching name and version.  So the registry 0.2.0 gets marked as installed because it's matched by the installed dev 0.2.0.
We then reconcile the installed and registry lists by matching on manifest equality.  So the registry 0.2.0 and installed 0.2.0 are correctly identified as distinct.

I am guessing the correct outcome is that we print cloud 0.2.0 twice, once as installed and once not?  If so, it should be a relatively simple fix to make the "is it installed" check look at manifest equality.
If we prefer to print cloud 0.2.0 only once (and as installed), we instead change reconciliation to look at only name and version.  Let me know if you prefer this, otherwise I'll do per the previous para.
https://github.com/fermyon/spin/pull/1613


ntr: dietary counseling, ntr: medical nutrition therapy

parent: counseling
label: dietary counseling
exact synonym: nutritional counseling, nutrition counseling
definition: Nutrition intervention for nutrition counseling is a supportive process, characterized by a collaborative counselor–patient/client relationship to establish food, nutrition and physical activity priorities, goals, and individualized action plans that acknowledge and foster responsibility for self-care to treat an existing condition and promote health.
Reference: Academy of Nutrition and Dietetics. Nutrition Terminology Reference Manual (eNCPT): Dietetics Language for Nutrition Care 2014, Nutrition Intervention/Terms and Definitions/Nutrition Counseling (C), page-064, http://ncpt.webauthor.com
parent: medical action
label: medical nutrition therapy
exact synonym: MNT
definition: Medical nutrition therapy (MNT) is an evidence-based application of the Nutrition Care Process. The provision of MNT (to a patient/client) may include one or more of the following: nutrition assessment/re-assessment, nutrition diagnosis, nutrition intervention and nutrition monitoring and evaluation that typically results in the prevention, delay or management of diseases and/or conditions.
Reference: Academy of Nutrition and Dietetics. Nutrition Terminology Reference Manual (eNCPT): Dietetics Language for Nutrition Care 2014. http://ncpt.webauthor.com.
orcid: 0000-0002-7463-6306

If we like MNT as a new class, we can also add subclasses such as

nutrition counseling, and subsequent specific counseling e.g. heart healthy diet counseling, DM counseling etc.
nutrition support (total parenteral nutrition, enteral nutrition)
oral supplement use, (Ensure, Boost etc.)

@LCCarmody quick question, how does approval/addition of a NTR work? There hasn't been any discussion about these ones, so I wasn't sure if I should move ahead and put in a pull request or just let this one hang around until we've finished pulling literature terms here soon.

It is less clear in this ontology because we aren't established yet.  That said, by adding a pull request, you are allowing me (or @pnrobinson ) to review your changes. We can decide to keep your new term or deny it.  To keep from doing too much work, it may be easier to start a new term request and wait for someone to respond.  You can follow up or specifically tag us for a quicker response.
I think dietary counseling shoudl be a term, but I'd like @pnrobinson to weigh in on medical nutrtion therapy.  Also, could you get ISBN # or other sources besides websites?  Websites change, so it is better to get other sources, if possible. Pubmed would be ideal, for this ontology, it will be difficult.
Also, I think you may have to wait for responses. Sorry, we are busy!  Hope you have a couple of additional projects?

Sounds good, I've got other stuff to plug away at, just trying to get a feel for this work schedule/process!
For updated citations, both of these terms are available within the newest edition of the Nutrition Care Process Terminology Reference Manual,
ISBN-13:<PHONE_NUMBER>692
ISBN-10:<PHONE_NUMBER>

Following up on this.  Nutritional counseling should be a term, but I do not think the child term of 'subsequent specific counseling' is necessary.  I do not know what DM counseling is.
Total parenteral nutrition (etc) have already been added.
I am not sure if oral supplementation should be added.  Can you give a specific use case from a rare disease? Also, is there a label that is more specific than 'oral supplementation' because that is vague.  If so, open a separate ticket. Thanks.

Hi! Looking back at this, 'nutrition counseling' I think is a valid class. The other child classes I was initially considering were 'heart healthy diet counseling' or 'diabetes mellitus counseling' but at this time, I'm not certain those would be a priority to add since they're more common disease focused.
As for oral supplementation, it's definitely a little vague. From my initial NTR request, I think I was more considering a term that represent additional/supplemental calorie consumption through an oral formula.
i.e. for individuals with maple syrup urine disease, they can use high calorie formula that is low in branched chain amino acids (BCAAs) to provide adequate energy without excess BCAAs.
A similar scenario is seen with Isolated methylmalonic acidemia in which low propiogenic amino acid formulas are given to provide supplemental calories.
I think a term regarding supplemental oral formulas for extra calorie provision could be widely used (not sure what exact term we would like, maybe 'oral calorie supplementation'?), but we can also make more specific terms that describe oral calorie supplementation through formula with other specific parameters i.e. low BCAAs if that feels more appropriate.


Colors not working in 3.1.3

Hello, fantastic extension! I downloaded the extension and the example app. However, the colors do not seem to be showing up at all for the custom coloring - not even in the example app. It only shows black. Any chance you get get this working?

Hello dustinbaxa, just (re)tested and everything works fine with 3.1.3. Possibly it's external to the extension. Have you tried to remove everything and re-install? Also, no need to put an '=' before the color expression and be sure a string is returned with the color code, including the '#'. Something like:
if(IsOwnerFlag=1,'#6bb344','#5C91BD')


Fix logic of test_fail_deploy_account.

This change is

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (aaef67a) 70.93% compared to head (7faf566) 70.93%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1277   +/-   ##
=======================================
  Coverage   70.93%   70.93%
=======================================
  Files          57       57
  Lines        7154     7154
  Branches     7154     7154
=======================================
  Hits         5075     5075
  Misses       1666     1666
  Partials      413      413

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

crates/blockifier/src/transaction/account_transactions_test.rs line 508 at r1 (raw file):

        contract_address!(0_u32), // Dummy value.
        contract_address_salt,
    );

TY @noaov1.
Create the tx with MAX_FEE != 0.
Code quote:
    let deploy_account_tx = create_account_tx_for_validate_test(
        TransactionType::DeployAccount,
        INVALID,
        None,
        &mut NonceManager::default(),
        faulty_account_feature_contract,
        contract_address!(0_u32), // Dummy value.
        contract_address_salt,
    );

crates/blockifier/src/transaction/account_transactions_test.rs line 508 at r1 (raw file):
Previously, ArniStarkware (Arnon Hod) wrote…
TY @noaov1.
Create the tx with MAX_FEE != 0.

Done.

This PR might conflict with #1283.


Add support for associating a notebook with a project (#r project)

The question comes up occasionally about whether a project can be referenced from a notebook. The use cases vary, so a discussion would be useful to understand the kinds of things people might want to use such a mechanism for.
Example scenarios:

Initializing a notebook's NuGet dependencies
Defining methods and classes that can be used from within the notebook

Some questions:

Is it desirable to have an "automatic" association of some kind so that code from the project is invoked automatically based on some convention (e.g. the presence of a project with the same name in the notebook's folder)?
What's the workflow for seeing changes in the associated project reflected back in the notebook?
What kinds of projects should be supported?

Personally I see this feature being useful for the following:

I have a large domain project (csproj) that isnt a nuget package, but I want to reference it to generate stats, charts, reports etc from the domain (which accesses a database).
Building code that can be shared by multiple notebooks.

It would be great if an active notebook reloaded references when the project was rebuilt, but I think the user could just manually re-run the referencing cell as well

Also was looking for this so I could experiment with some of our internal libs along side of projects and provide "live documentation" for them.
It would also be great to see these notebooks integrated with the UnitTest runner, so I could interactively write testing code and turn them into unit tests by having cells do Asserts/Expects.

Related: https://github.com/dotnet/roslyn/issues/51480

Agreeing with previous comments : this feature would be a great way of creating interactive demos for library code.

I really think this would turn this project from a 'toy' with limited usefulness into an integral tool in my development toolkit.
With code reuse being so limited today it just isn't there.

I think that this feature would be great for REPL style workflows when developing. For example in a large project it isn't always possible or desirable to start the entire app and get the "State" of the app to where you want it to be and to start experimenting with code that manipulates the state. A notebook would be perfect for setting up a "state" of the app / lib / proj etc. and then being able to try out code in a self-contained way.  To answer the initial questions here is my response:

Is it desirable to have an "automatic" association of some kind so that code from the project is invoked automatically based on some convention (e.g. the presence of a project with the same name in the notebook's folder)? - Yes it would be awesome to be able to create a "notebook" from a project, for example a web API, that had auto generated cells that a) Referenced all the dlls used and b) loaded the default / environment config and could get the app running with some kind of initial "State".

What's the workflow for seeing changes in the associated project reflected back in the notebook? Be great if "hot-reload" worked in your notebook. Say you start up your notebook, it has some initial "state" from config etc. then you execute a function, for example load up a List into a var in your notebook. You want to write code that manipulates your loaded list, but it doesn't have an "included" foreign key, say List, so you update the code that loads the customers in the project to also get their order, and hot-reload your notebook.

Also would be awesome to be able to "bind" a function two-way to a notebook cell. Click a button that generates a new notebook with the config, refs etc. and the original source for your function is in a cell of your notebook. You manipulate the function in your cell and when you are happy - push it back to the project and update the original.

What kinds of projects should be supported? I suppose anything that can run without a GUI would be a great start. Might be tricky with WPF interactions etc. Maybe having a way to "jack-in" to a running process would be great also. Say start up your Xamarin Forms project / app and then have your notebook be able to attach to the running process and execute code and retrieve returned values / state from the process.

Just as a side-note I did a bunch of Clojure work and the work-flow is heavily geared to a REPL. Being able to modify any function or state in a running process and being able to return / visualize the state in a meaningful way is development GOLD! For .Net folks it is hard to imagine how much more productive a great REPL experience can make you. .Net 6 hot-reload is a step in the right direction however returning and visualizing state is not quite there yet. Hot-reload is also "all" saved changes in the dll however with a REPL you can "push" just a single desired code update or state change, that would be nice too ;-)

My use cases

Developing a domain logic and presenting its capability and usage guidance to ...

interested parties during presentations
playable developer documentation
experimentation

Data driven unit test development: I create a notebook, create some data there, throw it in an algorithm, see the outcome. Now I invoke a method to persist the triplet of input, algorithm and output into a data set which then extends the data set for data driven testing. Basically, while exploring a bug or developing the feature creating a test data set.

My use case:
Something I do fairly often when building proofs of concept is create a simple console application to drive whatever class library or other assembly I might be working on. In that case I'd have a project reference in the console app's .csproj to the other assembly. This is great because I can switch back and forth very quickly, intellisense / omnisharp are aware of my changes almost instantly without having to rebuild and restore, and when I dotnet run or debug my console app it will automatically rebuild and restore (if needed). It's also nice being able to (eg) "Go To Definition" from the console app.
So I guess what I'd love to see are all of those things, so that I could replace the console app driver in my proofing phase of development with interactive notebooks. These are the only big drawbacks to the current state of interactive notebooks in my opinion, for my specific use case. So to wrap it up neatly:

Automatic rebuild/restore of referenced projects when changes are made.
(near) Real-time analysis for intellisense including code changes that haven't been rebuilt
Enable Go To Definition/Implementation etc. from within the notebook

In terms of what types of projects, I think C# class libraries are at the top of my list, personally.

+1 for living, testable/debuggable documentation with Notebooks, and to a fast inner-loop by using Roslyn to dynamically compile the notebook in-memory based on project source code changes. Right now, I have to close Notebooks so I can build my .dlls, re-open Notebooks to continue work. Combine with not getting the automatic transient dependency loading you'd expect, and the workflow gets quite tedious. Being able to compile and run cells alongside my integration tests would be next level.

+1 I've just landed here searching for the exact same solution.

Is this progressing?

Hooray! The lack of Notebooks discussion at BUILD had me worried that the project wasn't moving forward. Can't wait to see how this gets shaped. For me, it could really be the next level of living documentation for our APIs—in our projects, in our GitHub README.md(s), on our websites...

Hopefully notebooks arent completely ignored come november .NET 7 release goodness...

I was initially skeptical about referencing the project in a notebook and experimented only with plain NuGet references.
But I think I have found a use case that is blocked now and it may be solved via the project reference in the notebook cell.
I mean the F# -> C# interop here.
F# has type providers  and they can be exposed somehow to C# via separated project/package.
Many type providers are based on static parameters that are resolved in a compilation phase.
So when I create  F# package and expose it to C# in .NET Interactive, a lot of parametrization possibilities are gone.
Having the possibility to create a type from a project reference on notebook cell run will unblock it and enable huge F#/C# metaprogramming capabilities/interop.
So can the below be possible to do?
C# cell:
#r "project:projectname"
F# cell:
#r "nuget:projectname"

Skeptical? Interesting.. I could see adding notebooks to basically every C# project to demonstrate certain features of a library or make interactive utilities.

I was initially skeptical about referencing the project in a notebook

I was initially skeptical about referencing the project in a notebook, not about pairing notebooks with projects.
I'm one of the biggest advocates for using notebooks next to projects and even creating full solutions that are hosted on notebooks & .NET Interactive

I simulated the project reference functionality in a cell with CliWrap that runs dotnet build/pack for a project located in the current directory.
It generates a new nuget package on each cell run with an autoincremented version number.
In the next cell it loads that nuget however it is always the same first version of the package.

I know it is by design and the same for every package even if I provide the version explicitly:
.
@jonsequitur  Is there a way to tell .NET Interactive to obey that and load the latest version with each run of #r "nuget: ..." ?
I know that generating a package on each run may not be the perfect way to work as it will pollute the storage badly; however, I have no experience/idea how to achieve it differently (in memory, embedded, whatever,...)

Adding a +1 to this. My use case is that I'm integrating a 3rd party API. I have some DTOS in my project that help me interact with the API. Would love to be able to quickly prototype an API service class in the notebook, then transfer that over to my project. This would be transformative in the way I could explore integrating other services with my codebase.

This would make it much more practical to use notebooks as documentation. One specific use case is having notebooks that presents high-level functionality in a project, while making it easy to explore details in the relevant source files.

+1 here too!

In case it's useful, notebook automation such as directly running and testing notebooks is supported by dotnet-repl.
This doesn't address the need to copy and paste code between the notebook and your project. The ability to reference project code (using classic C#, not C# script) within runnable documentation was implemented in dotnet try and is being updated and brought into .NET Interactive via CSharpProjectKernel as part of the Try .NET update currently in progress. This should eventually allow dotnet try-style Markdown or other formats to be editable and runnable in Polyglot Notebooks or other frontends.

Super useful, thanks @jonsequitur! I had no idea [dotnet-repl](https://github.com/jonsequitur/dotnet-repl#-run-a-notebook-script-or-code-file-and-then-exit) existed, well done! Great news about the Try .NET update as well - glad to hear MS is making these important investments! Bright future for living/executable documentation. :)

I agree with @dcuccia  I think this would be perfect for the documentation of https://github.com/mahomedalid/dotnet-llm-eval-samples as  most of ML people are familiar with jupyter notebooks.

I hope to see something on these grounds in the near future. I have multi-project solutions and have scenarios where I need to mix code from two different projects that don't have a shared dependent but do have shared dependencies, and having to list out not just the two projects I'm pulling into a notebook, but also their own dependencies (both other projects and NuGet packages) leads to a kind of DLL hell. It's not uncommon for me to end up with a FileLoadException in a cell later down in a notebook because the kernel tried to load the same assembly from a different directory because of this forked project dependency path.

I see this is no longer available in the visual studio marketplace.  Are there plans to release it again?

@zlsmith86 It's here:
https://marketplace.visualstudio.com/items?itemName=ms-dotnettools.dotnet-interactive-vscode


Rewrite visualization module for Awpy 2

We need to rewrite the visualization module (we could call it vis for convenience). The absolute basic set of features we would need are:

[ ] position transform utility functions (this is probably not going to change much)
[ ] plot game events
[ ] plot round in animated fashion
[ ] plot grenade trajectories
[ ] heatmap

This is not something we need for an Awpy 2.0 release. Maybe 2.1.

Hi! I would like to make one feature request. Is it possible to add LEGEND on the side, in the vis for CT,T, bombs and weapons?

Hi, yes, of course! I will keep that in mind.


Please replace hardcoded URLs

In Step3.cshtml and Step5.cshtml there are several places that use hardcoded URLs to your blobstorage. When someone tries to get the code up and running for himself, this will result in the app not showing up selected images or its predicted output.

Hey I'd like to do this as a first issue. Could anyone give me any pointers please as to what I should be replacing it with?

@Mephisztoe Is this still an issue and can you provide some pointers to the files in question?


Replace instances of update_attributes with update to avoid deprecation warnings in Rails 6.

Fixes #170.

Are there any blockers to get this merged?

Are there any blockers to get this merged?


Error when using two tooltips with same ID on same page

If you have multiple tooltips on the same page, if any of them have the same IDs, only the first one will work. Tested with the following code.
Armadyl Godsword
Armadyl Godsword

I came up with an idea how to fix this issue. The tooltip bug is caused by the issue that two HTML elements have the same id on the same page - which goes against the usage of the id property.
Solution: Add a random string to each tooltip id after the requested item ID. For example: `id="11802-7Gb8dS". Then you could easily strip the random number suffix. Pseduocode:
item_id = id.split("-")[0]

This could be a optional addition, and the tooltip code could be written to determine if a dash (-) is found in the id element. One drawback... the random string insertion into the id would be a manual entry that a user/web developer would have to add.

I took a swing at this issue (#16). The proposed method allows the change to be backwards compatible so existing usage doesn't need to add a uuid to each element.


Cannot read property 'destroy' of undefined

Good job on wrapping HighCharts, it is a pain to work with it otherwise!
I have version 1.3.1, and I get an error even with the simplest example in the demos. It seems that somewhere the chart reference is deleted before the BaseChart component's componentWillUnmount event is called, thus the error.
You can investigate why this occurs, but for now a simple solution is to check the existence of the chart before calling destroy().
So BaseChart.js componentWillUnmount should be
componentWillUnmount () {

 if(this.chart){
       this.chart.destroy();
  }
  this.chart.__destroyed = true;
}

This might result in some memory leakage, but it might be a good quick fix for now.

Hmm I'm not sure why the reference would be deleted. I've never seen that before.
What version of React are you using?

I am not at work currently so I can't check, but since I upgraded our dependencies just few days ago it should be 15.6.1.

Ok thanks, by any chance is the component mounted, then unmounted almost immediately? I can't think of many reasons why this.chart would be undefined.

I didn't really spend a lot of time debugging, I just added an additional check so I can continue with development, but I can have a look if it helps.

I found the cause of it. I have wrapped the chart in another component, that only renders its children when the status is not loading. It seems React instantiates the children before they are rendered, and then immediately unmounts them, before the this.chart object is created.
<StatusWrapper
          isLoading={isAnalysisDataFetching}
          errorMessage={analysisDataError}
          <HighGmsChart height={450} seriesData={seriesData} />
</StatusWrapper>

Render method of StatusWrapper:
...
<div style={style}>
      {Spinner || ErrorMessage || children}
</div>

Interesting, I didn't know about that React behaviour.
Yep I'll wrap the componentDidUnmount in an if statement and push out a new release.

Published as 1.3.2, thanks for the heads up and debugging.


bug: Run Validator > Keystore Location has the wrong dir structure

Instructions here https://docs.gnosischain.com/node/guide/validator/generate-keys/cli/, i.e. the CLI tool creates validator keystores under keystores/validator_keys
Subsequent instructions here: https://docs.gnosischain.com/node/guide/validator/run/lighthouse#4-keystore-location reflect a different file structure, leading to users possibly copying/moving their keystores up 1 directory, which is hairy.
 The documentation is already really good. This section just gave me a bit of pause.

Related issue: https://github.com/gnosischain/documentation/issues/238

Will be deployed today.


Additional docstrings

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable


Update contributing.md

[Insert URL to the list here]
N/A
[Explain what this list is about and why it should be included here]
list guidelines-->list of guidelines
Fixed the typo.
By submitting this pull request I confirm I've read and complied with the below requirements 🖖
Requirements for your pull request

[x] Don't open a Draft / WIP pull request while you work on the guidelines. A pull request should be 100% ready and should adhere to all the guidelines when you open it. Instead use #2242 for incubation visibility.
[x] Don't waste my time. Do a good job, adhere to all the guidelines, and be responsive.
[x] You have to review at least 2 other open pull requests.
Try to prioritize unreviewed PRs, but you can also add more comments to reviewed PRs. Go through the below list when reviewing. This requirement is meant to help make the Awesome project self-sustaining. Comment here which PRs you reviewed. You're expected to put a good effort into this and to be thorough. Look at previous PR reviews for inspiration. Just commenting “looks good” or simply marking the pull request as approved does not count! You have to actually point out mistakes or improvement suggestions. Comments pointing out lint violation are allowed, but does not count as a review.
[x] You have read and understood the instructions for creating a list.

All the below section wouldnt apply because this is a typo fix

"List guidelines" was imo referring to all the guidelines of the list, which also include awesome.md and contributing.md. "List of guidelines" changes the meaning to only the guidelines as stated in the pr template. In my opinion therefore this change was unnecessary, but I might be wrong because english is not my native language.


[REQUEST]: Open AIS documentation

please provide documentation on Open AIS (open-ais.html)
+please also update open-ais in _data/software-applications.md

@marc-portier @jorgecunhamendes is this still relevant? or can this be closed?


setVisibility Method dose not work

i can not gone this view from my layout

@mohsenzamani22 Mine is really fine.


[RuleDocGenerator] Ensure rule file path has / prefix in generation

refs https://github.com/symplify/symplify/pull/2983#discussion_r579727391

All checks have passed 🎉 @TomasVotruba it is ready for review.

Thank you :+1:


Sample for retrieving related work items

Existing samples show many example of working with work items. However, the libraries don't support a way to retrieve linked work items, i.e. Parent, Child, Related, etc.
WorkItem.Relations gives us a list of WorkItemRelation. A work item relation gives us only the relation type, and the title for the relation. No id is given for the work item. The closest we have is the Url to work item.
An sample showing how to retrieve a related work item would be most beneficial.

We added a sample for you here


Add README.md

Add the proper README.md
Take the reference https://github.com/amfoss/TempleApp

@jaindiv26 can you please work on this issue

Can I take this

Yeah, but please study project properly I gave one reference link please go on that way

@sriramdasavage Go ahead.

@sriramdasavage are you still working on this

As there is no response from @sriramdasavage @jaindiv26 can you work on this please

@Chromicle Yeah, I'll work on this.

@Chromicle Any updates ?


AQCU-1667: add 'or earlier' to method start date that span entire report period r…

…equested

Coverage increased (+0.0006%) to 97.111% when pulling 050fc462ccacc13f4b5bd932a793dccb1159339a on mhines-usgs:AQCU-1667 into 85822eb39ba82a5a345ce5e65c1cd5b9f5ee5c9e on USGS-R:master.


ci(SPV-1114): gh action for manual build of docker images

Pull Request Checklist

[ ] 📖 I created my PR using provided  : CODE_STANDARDS
[ ] 📖 I have read the short Code of Conduct: CODE_OF_CONDUCT
[ ] 🏠 I tested my changes locally.
[ ] ✅ I have provided tests for my changes.
[ ] 📝 I have used conventional commits.
[ ] 📗 I have updated any related documentation.
[ ] 💾 PR was issued based on the Github or Jira issue.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 46.60%. Comparing base (8d44c38) to head (3518cc3).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #742   +/-   ##
=======================================
  Coverage   46.60%   46.60%
=======================================
  Files         328      328
  Lines       13378    13378
=======================================
  Hits         6235     6235
  Misses       6569     6569
  Partials      574      574

Flag
Coverage Δ

unittests
46.60% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report in Codecov by Sentry.

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


Feature: provide redbox functionality in dev mode.

Problem: I've sat in on a number of in-person training / teaching sessions / gitter - newer developers (whether to JS or Angular) don't have the immediate reflex to go check the console when their app fails to boot. This manifests with questions like "why does it just say Loading...?" or "it just doesn't work"
React Native (and React, recently) implement a feature called the "Red Box of Death" - if the app throws an exception, it displays a massive red box containing the error - in effect, it surfaces the errors out of the console. See https://www.npmjs.com/package/redbox-react for example.
We should implement a similar feature to be available in devmode - this is obviously not something you want to ship to production, but as a default-on option for new users it would solve a lot of confusion.
(perhaps this should be implemented in Zones or CLI? discuss?)

Maybe this should be implemented outside the cli.  The cli and seeds can then include this functionality at will.
Maybe a WebPack plugin to only include it in dev, rather than including it in prod and disabling it with the environment config.
Would a try/catch around the bootstrap method catch all exceptions thrown?

I tried by creating custom ErrorHandler and did sweetalert the error, but not all exceptions can be caught. Simple case like when element is not known didn't get caught by ErrorHandler
The same one when using try/catch block, cannot get all exceptions.

@filipesilva could be closed now that #4513 has been merged ?

@cexbrayat right-o, I forgot that!


Tracer shows lots of warnings

Currently, the tracer shows a warning when a file-affecting syscall that is not handled is used by the experiment. However this means that a lot of warning can be printed when everything is actually fine (and warnings are scary!)

Maybe only show this on verbose mode? Or add this to a log file?
On Mon, Jul 13, 2015 at 12:28 PM, Remi Rampin<EMAIL_ADDRESS>wrote:

Currently, the tracer shows a warning when a file-affecting syscall that
is not handled is used by the experiment. However this means that a lot of
warning can be printed when everything is actually fine (and warnings are
scary!)
—
Reply to this email directly or view it on GitHub
https://github.com/ViDA-NYU/reprozip/issues/128.

--
Fernando Chirigati
Research Assistant and Doctoral Candidate at NYU
http://bigdata.poly.edu/~fchirigati/ http://vgc.poly.edu/~fchirigati/

Yes, some of these might be moved from WARNING to INFO, but it's not entirely clear which.


Conflict with Easy villagers and WTHIT

Minecraft Version: 1.17.1
Forge Version: 37.0.17
Mod Version: 1.0.0
Single Player or Server: Single
Describe problem (what you were doing; what happened; what should have happened):
There's some sort of conflict between Ocean Floor Control, WTHIT and Easy Villagers.
Log file link: Crashlog
Video/images/gifs (direct upload or link):

This has nothing to do with Ocean Floor Control. This is Easy villagers which uses the API from Jade, which isn't compatible with WTHIT's implementation

See https://github.com/henkelmax/easy-villagers/issues/79


License question

Hi @cschaefer26, I am confused because your repo's license appears to be a MIT license, but also states:
Copyright (c) 2020 Axel Springer AI. All rights reserved.

Can I use your code in my project? Are redistributions of code allowed?

Hi, the license is MIT, so redistribution is warranted as stated. So you can use the code no problems!

Awesome, thanks for your contributions to open source!


Support for 3D inputs

How to visualize of feature maps obtained from 3D CNN's using lucid api's ?

Hey @koriavinash1, we do not have support for 3D inputs built in. Support would consist of a 3D parameterization and robustness transforms.
You could look into lucid.optvis.param for how we construct parameterizations and build your own for the 3D input your model requires. Unfortunately we don't yet have any guides for this. The hard part would likely not just be the parameterization, but finding robustness transformations that work iwell n 3D… future research! Please let us know if you implement feature visualization in 3D!


[BUG] bloxstrap is unable to connect to roblox

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[ ] I did not answer truthfully to all the above checkboxes.

What problem did you encounter?

i tried to reinstall but when i try to open the installer it says that too
Bloxstrap Log
N/A

That's something I haven't seen before. Get Bloxstrap's latest log file from %localappdata%\Bloxstrap\Logs, and send it here.

That's something I haven't seen before. Get Bloxstrap's latest log file from %localappdata%\Bloxstrap\Logs, and send it here.

there is no log bc i was trying to reinstall it so i deleted it and i couldnt reinstall it bc that was showing up

i fixed it i had to delete roblox then let bloxstrap install it


TypeChecker#getSymbolsInScope returns undefined symbol

This may be more a question and maybe not be a bug... why does the following code return the undefined symbol? Is there something special about it? I noticed this when providing (ts.SymbolFlags as any)["All"] as the meaning argument to TypeChecker#getSymbolsInScope... seems to have the undefined symbol in every case I've tried.
Side question: do you think there's a sensible default for meaning in getSymbolsInScope? Maybe ts.SymbolFlags.Value? Or is filtering by meaning always useful? What do you think about making meaning an optional parameter that defaults internally to ts.SymbolFlags.All? Thanks!
TypeScript Version:  3.5.0-dev.20190518
Search Terms: getSymbolsInScope undefined
Code
import * as ts from "typescript";

console.log(ts.version);

const testFilePath = "/file.ts";
const testFileText = `let t: number;`;
const testSourceFile = ts.createSourceFile(testFilePath, testFileText, ts.ScriptTarget.Latest, true);
const program = getProgram();
const typeChecker = program.getTypeChecker();

console.log(typeChecker.getSymbolsInScope(testSourceFile, ts.SymbolFlags.Property));

function getProgram() {
    const options: ts.CompilerOptions = { target: ts.ScriptTarget.ES5 };
    const host: ts.CompilerHost = {
        fileExists: filePath => filePath === testFilePath,
        directoryExists: dirPath => dirPath === "/",
        getCurrentDirectory: () => "/",
        getDirectories: () => [],
        getCanonicalFileName: fileName => fileName,
        getNewLine: () => "\n",
        getDefaultLibFileName: () => "",
        getSourceFile: filePath => filePath === testFilePath ? testSourceFile : undefined,
        readFile: filePath => filePath === testFilePath ? testFileText : undefined,
        useCaseSensitiveFileNames: () => true,
        writeFile: () => {}
    };
    return ts.createProgram({
        options,
        rootNames: [testFilePath],
        host
    });
}

Expected behavior: Outputs empty array?
Actual behavior:
Outputs:
[ SymbolObject {
    flags: 33554436,
    escapedName: 'undefined',
    checkFlags: 0,
    declarations: [],
    type:
     TypeObject {
       checker: [Object],
       flags: 32768,
       id: 7,
       intrinsicName: 'undefined',
       objectFlags: 131072 } } ]

undefined can be legally shadowed (😲) so it needs to go into a special global symbol table

Hah, I did not know that!
(function () {
    const undefined = 5;
    console.log(undefined);
})();

Somewhat reminds me of these: https://twitter.com/jaredpar/status/981604285036208128 https://twitter.com/jaredpar/status/1115312773083189248?lang=en
Ok, thanks @RyanCavanaugh!

Ah, ok... so undefined is a property of the global object (https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/undefined#Description) and that's why it shows up when doing ts.SymbolFlags.Property (I was still wondering about that point).


Clicking skip on ads doesn't work

It doesn't seem to register the click. It shows you a skip button after a certain number of seconds like YouTube.

Skipping works for me. Maybe a bug with that particular ad on soundcloud.com?

+1. It has actually never worked for me on any ad. I don't regularly use Safari as my system browser, so not sure what might be going on.

I'm still experiencing this, @salomvary do you have any idea what might be going on? Anything else I can provide to further help debug?


Dont import hilite_me and insert_line_numbers_in_html with from manim import *

https://github.com/ManimCommunity/manim/blob/199424d713c77fca6f59de58ba0f8eb3955bfab1/manim/mobject/svg/code_mobject.py#L6
The functions hilite_me and insert_line_numbers_in_html are currently imported with from manim import * , and I think that is not necessary.
Further the question:
Why are they defined outside the Code class?

Why are they defined outside the Code class?

Maybe a mistake.
Marking this as a "good first issue" as it is as simple as removing those two lines from __all__.


Bdd

passed bdd, corrected logic of delete wishlist

IBM Cloud toolchain: Delivery Pipeline deployed wishlists to dev, including commit 56a48b7dd61715337af0c1255f51e52bd0b5b416


[mesa] Dependencies and CI updates

Revises dependencies and configuration.
Add feature for controlling vulkan support.
Enable osx CI.
Prepare for linux CI.
Intentionally not updating to mesa 33.3.3: It raises some requirements. It is better to that update in isolation, maybe after enabling linux CI.

/azp run

@BillyONeal et al. Artifact uploads seems to broken,  error: curl failed to put file to ... everywhere, https://dev.azure.com/vcpkg/public/_build/results?buildId=99766&view=logs&j=7922e5c4-0103-5f8f-ad17-45ce9bb98e80.

@dg0yt Are there plans to complete these changes? I'm interested in updating Mesa to the latest build to support LLVM 18.

Are there plans to complete these changes? I'm interested in updating Mesa to the latest build to support LLVM 18. See #37599.

@yurybura Yes. The delay is from getting additional system packages applied to the CI images, to enable CI for linux and osx. This happens once in a month... I would do a mesa update once CI works for these platforms.
That being said, I consider llvm significant enough to accept parallel mesa update work. It should perhaps be limited to minimal changes within the current CI coverage.

@dg0yt Is there any progress in this PR?

Not so soon.
On x64-linux, there is now an LLVM issue:
/usr/bin/ld: /mnt/vcpkg-ci/installed/x64-linux/debug/lib/libLLVMSupport.a(Process.cpp.o): in function `terminalHasColors(int)':
/mnt/vcpkg-ci/b/llvm/src/org-17.0.2-5fdca9a361.clean/llvm/lib/Support/Unix/Process.inc:354: undefined reference to `set_curterm'
/usr/bin/ld: /mnt/vcpkg-ci/b/llvm/src/org-17.0.2-5fdca9a361.clean/llvm/lib/Support/Unix/Process.inc:356: undefined reference to `setupterm'
/usr/bin/ld: /mnt/vcpkg-ci/b/llvm/src/org-17.0.2-5fdca9a361.clean/llvm/lib/Support/Unix/Process.inc:374: undefined reference to `tigetnum'
/usr/bin/ld: /mnt/vcpkg-ci/b/llvm/src/org-17.0.2-5fdca9a361.clean/llvm/lib/Support/Unix/Process.inc:380: undefined reference to `set_curterm'
/usr/bin/ld: /mnt/vcpkg-ci/b/llvm/src/org-17.0.2-5fdca9a361.clean/llvm/lib/Support/Unix/Process.inc:381: undefined reference to `del_curterm'

I think I have been there before, some kind of installation order issue. But I can't rebuild LLVM so many time as I would need to.

I'm trying to support at least mesa:x64-windows. The meson tool needs to be updated and patched to support LLVM 18.1.x (see https://github.com/mesonbuild/meson/commit/dec93ce9a52b234ed4cb25646eaf13c503ba739a).

As I said, go ahead with llvm as you like. Just keep changes to mesa minimal.


windows (mingwX64Test) failure: ModuleVersionNotFoundException: Could not find :kotlin-native-prebuilt-windows-x86_64:1.7.2

I'm not sure why this workflow is failing - it looks okay
https://github.com/Kantis/ks3/actions/runs/3330228810/jobs/5508472601#step:6:265
https://github.com/Kantis/ks3/actions/runs/3330228810/jobs/5508472601#step:6:542
Might be related to the centralised repo definitions, or perhaps Gradle is timing out when downloading the large file?

I think the centralised repo definitions is most likely candidate. Timeout shouldn't happen when an active transfer is in progress

It's strange because I think the centralised repos have worked before...
https://github.com/Kantis/ks3/actions/runs/3316468369/jobs/5478306674


Cannot mark input values into trans block as safe in auto escaped environment

Hi there,
I just noticed that I wasn't able to mark a value in a trans block as safe in an auto escaped environment.
For example: {% trans foo='<foo>'|safe %}bar{{foo}}bar{% endtrans %} results in bar<foo>bar instead of in bar<foo>bar as I would have expected.
wrapping the whole block in an {% autoescape false %} block allowed me to work around this issue for now, but it seems wrong, as the whole block is not something I want to remove from auto escaping, for example because it could contain other variables which should be escaped.
So, is this a bug in jinja2 or am I somehow using it wrong?

I can't reproduce this issue. Marking a variable as safe in the tag seems to work for both standard and new-style gettext.


Practice bot config

review this please

Additional tasks:

update Constants.java with the names of the new robots
update RobotContainer.java such that the new robot constants are handled

More complicated additional tasks:

The practice bot will have a different gear ratio for the swerve drive wheels. We don't want to just change the value in SwerveModuleConstants since that will break Nova. Instead, we need to add a new method to RobotConfig to query the swerve drive gear ratio and then implement that method in each config subclass. We then need to ensure that the constants in the SwerveModuleConstants file are never referenced and the RobotConfig singleton is used instead.


Show the actual implementation of Linear Regression (What is inside this Linear Regression black box and test it on salary dataset][GSSOC'23]

I want to show the actual implementation of linear regression without importing it from sklearn and then test it on small data-set

Hi, please link the dataset.

@Pritamstudent - you can go ahead! We are assigning you 21 days for this project, after which it will be assigned to someone else if not completed. All the best!
Name the file as: algorithm_dataset.ipynb and link it in the readme of the labeled directory as algorithm - dataset.


docs: add example "Multi Format Schema Object Example with Avro"

title: "docs: add example Multi Format Schema Object Example with Avro"
Make it clear on how to use Avro and other non-JSON Schema conform formats inline.
Discussed with @smoya
Related issue(s):

https://github.com/asyncapi/spec/issues/1015

/rtm


Add fileIncludePath option

Addresses #19

@tkrille - I just updated this PR with documentation of the new interface for includes.

Wow, thanks again for your awesome work @jklukas!

When do you need a release for this?

I don't have any immediate plans for using this in a project, so no hard deadline. It would be great to see this out in the next week or two.

Alright, I'll prepare everything.

Done: https://github.com/tkrille/dropwizard-template-config/releases/tag/v1.4.0


Update AnnotatedTransformer.ipynb

The Predict tensor has zeros in some positions and these result in -inf when log() is applied. Hence softmax_log() is proposed instead of log() - in methods example_label_smoothing() and loss()

The log() applied to the predict tensor is creating issue with LabelSmoothing(). Hence I propose to use softmax_log() instead of log()

On the same: #109, #115, #117, https://github.com/harvardnlp/annotated-transformer/pull/115#issuecomment-1729723671.


TCP Connection will auto close

Python Code:
from labview_automation import LabVIEW
vi_path = 'C:\Users\Administrator\Desktop\add.vi'
lv = LabVIEW()
lv.start() # Launches the active LabVIEW with the listener VI
with lv.client() as c:
control_values = {
"a": 2,
"b": 5
}
indicators = c.run_vi_synchronous(
vi_path, control_values)
print(indicators['c'])
#error_message = c.describe_error(indicators['Error Out'])
#lv.kill() # Stop LabVIEW
LVAutomation Listener:
[2019/3/29 9:17:52] (GUID - 5c9cad606465322594000008) Command received.
Message: {
"run_options" : 0,
"indicator_names" : [],
"control_values" : {
"a" : 2,
"b" : 5
},
"vi_path" : "C:\Users\Administrator\Desktop\add.vi",
"command" : "run_vi",
"open_frontpanel" : false
}
[2019/3/29 9:17:52] (GUID - 5c9cad606465322594000008) Command sent successfully.
Message: {
"c" : 7.0000000000,
"RunVIState_Status" : false,
"RunVIState_Code" : 0,
"RunVIState_Source" : ""
}
[2019/3/29 9:17:52] (TCP Connection-5c9cad606465322594000008) Lost
Python Shell:

lv.client().run_vi_synchronous(vi_path, control_values)

Traceback (most recent call last):
File "<pyshell#0>", line 1, in
lv.client().run_vi_synchronous(vi_path, control_values)
File "D:\Python\Python27\lib\site-packages\labview_automation\labview.py", line 175, in client
raise NotStartedError("You must start LabVIEW to get talk to it.")
NotStartedError: You must start LabVIEW to get talk to it.

I was having this issue as well. By specifying 32bit with lv=LabVIEW(bitness='x86') I was able to keep the connection open long enough to run my VI. Does this work for you? It still does seem to auto close after code completes for me. Also you may need to be running both 32 bit labview and python for this to work smoothly. That's just a guess though since there doesn't seem to be much documentation on this that I can find.

Did some more testing and I still have this issue actually, but only on the first run. When I run for the first time and the listener opens then it will close before running the command and throw this error. However, if I then run my script a second time leaving the listener window open from the first run, then it will run the command successfully.


chore(release): v0.10.0

:robot: I have created a release beep boop
0.10.0 (2024-12-06)
Features

docs search form add showRankingScore switch. gh-176 (f6bb8a1)

Bug Fixes

add offset & limit number value limit in search docs filter. gh-173 gh-174 (0032c63)
copyable component not work. gh-170 (c20b6ee)
document image url value insensitive gh-169. (63da243)
remove useless scrollbar in docs list. gh-175 (9d903a1)

Performance Improvements

index setting json editor height use vh unit. gh-168 (49efc50)
maintain list type in docs list page gh-172. (effbc9b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/riccox/meilisearch-ui/releases/tag/v0.10.0 :sunflower:


C#: Renames to follow .NET naming conventions

Renamed C# types and members to use PascalCase and follow .NET naming conventions.

Closes https://github.com/godotengine/godot/issues/28748

Renamed Godot.Object to GodotObject to avoid ambiguity with System.Object.
Renamed Godot.Thread to GodotThread to avoid ambiguity with System.Threading.Thread.
Renamed Godot.System to Godot.System_ to avoid ambiguity with the System namespace. We don't define a class named System but third-party modules could.

Fixes https://github.com/GodotECS/godex/issues/187

Renamed Godot structs

Original name
New name

AABB
Aabb

Rect2i
Rect2I

RID
Rid

Vector2i
Vector2I

Vector3i
Vector3I

Vector4i
Vector4I

Renamed Godot classes

Original name
New name

AESContext
AesContext

AudioStreamWAV
AudioStreamWav

CPUParticles2D
CpuParticles2D

CPUParticles3D
CpuParticles3D

CSGBox3D
CsgBox3D

CSGCombiner3D
CsgCombiner3D

CSGCylinder3D
CsgCylinder3D

CSGMesh3D
CsgMesh3D

CSGPolygon3D
CsgPolygon3D

CSGPrimitive3D
CsgPrimitive3D

CSGShape3D
CsgShape3D

CSGSphere3D
CsgSphere3D

CSGTorus3D
CsgTorus3D

CurveXYZTexture
CurveXyzTexture

DTLSServer
DtlsServer

EditorSceneFormatImporterFBX
EditorSceneFormatImporterFbx

EditorSceneFormatImporterGLTF
EditorSceneFormatImporterGltf

EditorVCSInterface
EditorVcsInterface

EncodedObjectAsID
EncodedObjectAsId

Generic6DOFJoint3D
Generic6DofJoint3D

GLTFAccessor
GltfAccessor

GLTFAnimation
GltfAnimation

GLTFBufferView
GltfBufferView

GLTFCamera
GltfCamera

GLTFDocument
GltfDocument

GLTFDocumentExtension
GltfDocumentExtension

GLTFDocumentExtensionConvertImporterMesh
GltfDocumentExtensionConvertImporterMesh

GLTFLight
GltfLight

GLTFMesh
GltfMesh

GLTFNode
GltfNode

GLTFSkeleton
GltfSkeleton

GLTFSkin
GltfSkin

GLTFSpecGloss
GltfSpecGloss

GLTFState
GltfState

GLTFTexture
GltfTexture

GLTFTextureSampler
GltfTextureSampler

GPUParticles2D
GpuParticles2D

GPUParticles3D
GpuParticles3D

GPUParticlesAttractor3D
GpuParticlesAttractor3D

GPUParticlesAttractorBox3D
GpuParticlesAttractorBox3D

GPUParticlesAttractorSphere3D
GpuParticlesAttractorSphere3D

GPUParticlesAttractorVectorField3D
GpuParticlesAttractorVectorField3D

GPUParticlesCollision3D
GpuParticlesCollision3D

GPUParticlesCollisionBox3D
GpuParticlesCollisionBox3D

GPUParticlesCollisionHeightField3D
GpuParticlesCollisionHeightField3D

GPUParticlesCollisionSDF3D
GpuParticlesCollisionSdf3D

GPUParticlesCollisionSphere3D
GpuParticlesCollisionSphere3D

HMACContext
HmacContext

HTTPClient
HttpClient

HTTPRequest
HttpRequest

InputEventMIDI
InputEventMidi

JNISingleton
JniSingleton

JSON
Json

JSONRPC
JsonRpc

MultiplayerAPI
MultiplayerApi

MultiplayerAPIExtension
MultiplayerApiExtension

Object
GodotObject

ORMMaterial3D
OrmMaterial3D

PacketPeerDTLS
PacketPeerDtls

PacketPeerUDP
PacketPeerUdp

PCKPacker
PckPacker

RDShaderSPIRV
RDShaderSpirV

ResourceUID
ResourceUid

SkeletonModification2DCCDIK
SkeletonModification2DCcdik

SkeletonModification2DFABRIK
SkeletonModification2DFabrik

SkeletonModification3DCCDIK
SkeletonModification3DCcdik

SkeletonModification3DFABRIK
SkeletonModification3DFabrik

StreamPeerGZIP
StreamPeerGZip

StreamPeerTCP
StreamPeerTcp

StreamPeerTLS
StreamPeerTls

TCPServer
TcpServer

Thread
GodotThread

UDPServer
UdpServer

UPNP
Upnp

UPNPDevice
UpnpDevice

VisualShaderNodeCurveXYZTexture
VisualShaderNodeCurveXyzTexture

VisualShaderNodeScreenUVToSDF
VisualShaderNodeScreenUVToSdf

VisualShaderNodeSDFRaymarch
VisualShaderNodeSdfRaymarch

VisualShaderNodeSDFToScreenUV
VisualShaderNodeSdfToScreenUV

VisualShaderNodeTextureSDF
VisualShaderNodeTextureSdf

VisualShaderNodeTextureSDFNormal
VisualShaderNodeTextureSdfNormal

WebRTCDataChannel
WebRtcDataChannel

WebRTCDataChannelExtension
WebRtcDataChannelExtension

WebRTCMultiplayerPeer
WebRtcMultiplayerPeer

WebRTCPeerConnection
WebRtcPeerConnection

WebRTCPeerConnectionExtension
WebRtcPeerConnectionExtension

XMLParser
XmlParser

ZIPPacker
ZipPacker

ZIPReader
ZipReader

@akien-mga This will need a special warning in the release blog post, as these renames are likely to affect most projects. I would also suggest deleting the res://godot/dotnet/ folder if the editor crashes before they can rebuild (as would be the case with exceptions thrown from static constructors, which we cannot catch).

Sounds good, I can include this in the release blog post.
Needs a rebase after a bunch of other C# changes I just merged.

Thanks!

Renamed Godot.Object to GodotObject to avoid ambiguity with System.Object.
Renamed Godot.Thread to GodotThread to avoid ambiguity with System.Threading.Thread.

@raulsntos
It seems a little redundant to me to prefix anything in the Godot namespace with Godot just for the sake of avoiding ambiguity.
Perhaps a "using alias directive" would be better suited for avoiding ambiguities like this?
While I would likely prefer to use Godot.Object and System.Object in full personally, a person wishing to rename/alias Godot.Object to avoid ambiguity with System.Object could simply:
using Godot;
using System;
using GodotObject = Godot.Object;

Alternatively, I believe this would also work:
using Godot;
using System;
using Object = Godot.Object;

@Zhersi Adding those aliases for each file is too annoying. And if you happen to forget it, you could be using the wrong class by accident. Additionally, it's about readability as well (it's hard to know which Object class is being used).
Because of all that, often in our own code I tend to explicitly write Godot.Object, for which I need to add a comment to disable ReSharper warnings about redundancy:
https://github.com/godotengine/godot/blob/47ca2f2e092b985e6c56a70781ac1a18972245a1/modules/mono/glue/GodotSharp/GodotSharp/Core/NativeInterop/VariantUtils.cs#L618-L619

@neikeq
Thanks for sharing your thoughts!
I agree that having to use aliases for each file can be annoying, which is why I also prefer to explicitly write Godot.Object and System.Object, however I wanted to share the possibility of using aliases for people who want different names for these classes.
My main issue is that I find it needlessly verbose to prefix for example Object with Godot when it already belongs to the Godot namespace (Godot.GodotObject).
I would much rather the Godot prefix be removed, in favor of letting users figure out how to resolve any potential ambiguities on their own.

I would like to throw my voice into this as well by agreeing with @Zhersi on the matter of redundancy-
One potential solution to having to define an alias for each file would be to make use of Global Using Directives. This allows one to define the alias in a single file that will then be used throughout the entire project without having to worry about the redundancy or aliasing:

Another potential solution would be to rename Object to something like GObject in order to still have a slight deviation from Object itself without having the need to add redundant naming schemes resulting in something like Godot.GodotObject

@Kyniverse
Thanks for introducing me to "global using directives"!
That would indeed prevent someone from having to create an alias for every file.
In regards to renaming GodotObject to GObject, while it is less verbose, I still find it slightly redundant for the same reasons as pointed out in my previous comment.

GObject makes perfect sense to me, since people who're reading to code definitely knows G stands for Godot. Just like Unreal's UObject.

I didn't go with GObject to avoid confusion because that name is commonly associated with the GLib project. It could be GDObect but I don't like the 3 capital letters.
I have at least the following issues with global using directives:

The semantics of a source file can change due to factors unrelated to the file itself or the namespaces it uses. This was already the case before, but it wasn't as problematic in practice. With global usings, a source file (which could have been copied from somewhere else) which was meant to reference the System.Object class, would now be referencing Godot.Object.
A global using alias cannot be overridden (in the same namespace) by a specific file (no using Object = System.Object;), as it counts as a duplicate.
It can easily cause confusion, as Object is commonly associated with System.Object, and the only disambiguation to be found is in another file.

Honestly, messing with global aliases in someone else's project just sounds wrong to me.
At the end of the day, the redundant naming in Godot.GodotObject is but a slight annoyance that's worth it to solve a real problem. I'm open to consider other naming options, but it's unlikely that we return to Object.

Thanks for taking the time to respond again @neikeq!
I really hope I am not making too much trouble over naming conventions/policies.
I cannot help but ask if something should be renamed just to avoid it having an identical name to something in another namespace. Is not some of the point of namespaces to allow for exactly this to be the case in the first place?
Renaming Object to GodotObject does solve the ambiguity between System.Object and Godot.Object.
However, should this really be the policy for resolving ambiguities of this kind?
What about other ambiguities?:

System
Godot

System.Array
Godot.Collections.Array

System.Drawing.Color
Godot.Color

System.Numerics.Vector2
Godot.Vector2

System.Numerics.Vector3
Godot.Vector3

Should these also be renamed (potentially prefixed "Godot")?
My personal opinion is that it would be better to revert the names of GodotObject and GodotThread to Godot and Thread respectively, and let users handle ambiguities like this (be it by aliases or by explicitly stating the namespace, ex. Godot.Object),

The .NET naming guidelines recommend avoiding type name conflicts (e.g.: 2 types with the same name in different namespaces that are often used together).
See the Namespaces and Type Name Conflicts section of the guidelines:

❌ DO NOT give the same name to types in namespaces within a single application model.
For example, do not add a type named Page to the System.Web.UI.Adapters namespace, because the System.Web.UI namespace already contains a type named Page.
❌ DO NOT give types names that would conflict with any type in the Core namespaces.
For example, never use Stream as a type name. It would conflict with System.IO.Stream, a very commonly used type.
❌ DO NOT assign type names that would conflict with other types within a single technology.
❌ DO NOT introduce type name conflicts between types in technology namespaces and an application model namespace (unless the technology is not intended to be used with the application model).

Since the System namespace is part of the Core namespaces, we should avoid names that conflict with names of types defined in that namespaces (e.g.: System.Object, System.Array, etc.)
We should also avoid names of types in namespace that are often used together with the Godot namespace. I don't think the System.Drawing namespace is often used together with the Godot namespace so we should be safe there.
For the System.Numerics namespace, I think it's a bit more complicated. I can see use cases for using the namespace in a Godot game to take advantage of some of their types, in order to use hardware acceleration for some math operations, but I would say these operations would be in some utilities file that would convert the types and most of the game code would not mix vector types. So I think we're still safe in this regard.
The only types that I agree should probably also be renamed are Godot.Collections.Array and Godot.Collections.Dictionary since they conflict with names of types in Core namespaces and they are very likely to be used together. In fact, I would say these conflicts are more common than GodotObject because I don't think most users would reference the GodotObject type directly.

About the Godot prefix, I don't think we should prefix every class with Godot but for GodotObject it seems to me like the best choice. GObject and GDObject don't really follow the guidelines.
See the .NET General Naming guidelines:

❌ DO NOT use abbreviations or contractions as part of identifier names.
For example, use GetWindow rather than GetWin.
❌ DO NOT use any acronyms that are not widely accepted, and even if they are, only when necessary.

Therefore, I think since the type represents a Godot Object and not any other kind of Object, it makes sense to me to call it GodotObject. If we were to use acronyms it would likely have to be GDObject since there's already precedent in the GD class, but again the guidelines recommend doing this only when necessary and I personally agree with @neikeq that 3 capital letters look ugly.

Thanks for taking the time to clarify @raulsntos!

@raulsntos @neikeq renaming fields like Vector2.x to Vector2.X is not consistent with .NET Naming conventions, and is a huge breaking change to boot.   Please consider partial reverting this.

my bad...  sorry I was wrong :(   (deleted old post)   System Numerics uses Pascal fields too.


Uniswap-Forks - Added and improved comments all throughout code.

The comments are nice. Thanks for adding them!


Documentation

I want to change the documentation of website, please assign me

@Akanksha-Sen what documentation do u want to change?

I'd change some text and improve the content quality.

@Akanksha-Sen if u want to change something in readme then there is nothing to improve and if u want to change something in the website then u should clearly mention what u would like to change. And u can dm in the discord what u would like to change.

@Akanksha-Sen can u explain?

I'd improve the text part by enhancing the english words used. Sorry for the delay I had health issues.

@Akanksha-Sen u can work


MeshStandardMaterial

A material with a standardized set of properties and behavior. See https://threejs.org/docs/#api/en/materials/MeshStandardMaterial

Implemented in #324


Some commands are missing the original English description

VSCode Version: 1.45.1
OS Version: Windows_NT x64 10.0.17763

Steps to Reproduce:

press F1 to show all commands
type English words to find the command

I used to ues next terminal or next to find the command , and focus to the next terminal. (v1.44.2)

after I update vscode to 1.45.1, I can't find these command by typing English words

But If i change vscode display language to English.
these commands can show English description correctly.

Does this issue occur when all extensions are disabled?: Yes

Since this works in general it doesn't look like a problem with the NLS infrastructure.
@Tyriar can you please check that you provide both strings for the command.


wrong_login_is_not_password_reset

🍰 Pullrequest

an incorrect login should always result in an error not in a request to reset the password
Issues

None

Todo

[X] None

If the user has not activate his email he gets redirected to:


No-Undef on global variable to be used in production only (injected by Android Webview)

I'm developing a webpage that will be viewed inside an Android webview environment.  That webview will inject a global variable which is a Javascript interface for the Android App.
Let's say the variable is AndroidClient.
I have the following code inside a helper function:
function signin() {
  if (typeof AndroidClient !== 'undefined') {
    AndroidClient.signIn();
  else {
    console.log('signing in');
  }
};

I'm getting the following build errors when I run the dev server, and I updated eslintrc to only warn about 'no-undef' error yet I'm still getting a build error.
./src/helpers.js
  Line 6:  'AndroidClient' is not defined  no-undef

How can I resolve this within a create react app / react-scripts environment?

Have you tried accessing AndroidClient through the global scope window
window.AndroidClient
Or you can wrap the code with

/*eslint-disable no-undef*/
...
/*eslint-enable no-undef*/

Yes, accessing it turning window is the recommended approach.

You can write this code in you .eslintrc file
{
  "globals": {
    "AndroidClient": true
  }
}


fix: shrink queued bodies and headers when drained or split

This fixes an OOM in the bodies stage and reduces overall memory allocations in the downloaders, by shrinking vectors whenever we split or drain them. Previously the headers downloader would effectively clone queued_validated_headers in terms of allocations, because split_off does not change the original vec's capacity. Because queued_validated_headers could be on the order of gigabytes, this caused an OOM.
Fixes #3175

supportive as discussed - any way of testing this?

Just added some checks in the existing tests that make sure len == capacity for downloaded values, when the stream batch size is lower than the number of values that should be produced.
also added another test for one-by-one download with a larger request limit, so queued_validated_headers is hopefully larger than the batch size in that test, which should trigger the necessary condition.

@mattsse added calls to shrink_to_fit whenever buffered responses are popped in 781d786e8d2f2de9b9e97e0e3748482d1815ec1e

thks for resolving this.


http: add comment for explaining the new http cli design and usage

What problem does this PR solve?

Issue Number: close https://github.com/pingcap/tiproxy/issues/602
Problem Summary:
What is changed and how it works: add some comment
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Notable changes

[ ] Has configuration change
[ ] Has HTTP API interfaces change
[ ] Has tiproxyctl change
[ ] Other user behavior changes

Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

@djshow832 PTAL~

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (main@22e2d51). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #608   +/-   ##
=======================================
  Coverage        ?   68.60%
=======================================
  Files           ?       83
  Lines           ?     7769
  Branches        ?        0
=======================================
  Hits            ?     5330
  Misses          ?     2066
  Partials        ?      373

Flag
Coverage Δ

unit
68.60% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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