Update GoogleTest version

Currently we use 1.10.0 which is ~2 years old.
Maybe we can upgrade to 1.11.0.
https://github.com/google/googletest/tree/B4DE7B5F4F45390ABCFB887AFADA9303

Newer versions make trouble for builds with sanitizers (unresolved symbol when linking).

Can we solve it in our side, or is this a bug that needs to be reported to GoogleTest devs?
It can be postponed to post 5.0.0 release.

@stweil
Which linker errors occur exactly?
I had a project which also produced linker errors when upgrading to 1.11.0 but it was easily solved by removing GTEST_API_ in front of all my main functions calling RUN_ALL_TESTS.
My error(s) were: Compiler Error C2491: 'identifier' : definition of dllimport function not allowed
https://docs.microsoft.com/en-us/cpp/error-messages/compiler-errors-1/compiler-error-c2491?f1url=%3FappId%3DDev16IDEF1%26l%3DEN-US%26k%3Dk(C2491);%26rd%3Dtrue&view=msvc-170
Before: GTEST_API_ int main(int argc, char **argv)
After: int main(int argc, char **argv)
OS: Windows
Compiler: MSVC (14.1 = Visual Studio 2017)

I cannot reproduce the linker error and therefore now decided to upgrade GoogleTest, see commit a185c068d1ace62d91285d094a3858135cb0cefe.

I cannot reproduce the linker error and therefore now decided to upgrade GoogleTest, see commit a185c068d1ace62d91285d094a3858135cb0cefe.


[Web Components] Use Light DOM instead of Shadow DOM

Fix #280 and possibly #284
I would like to wait merging this until #290 is resolved.

@kojiishi PTAL

@kojiishi Could you review this PR when you have a chance?


Budget-App_Feature

Description
This pull request encompasses a series of fundamental changes aimed at establishing the initial structure and functionality of the Ruby on Rails application. Here is a breakdown of the modifications:
Setting Up
Created a new Rails application configured with a PostgreSQL database for development.
Integrated a linter and Rubocop for code styling consistency.
Established the necessary models and their associations.
Configured routes to define the application's navigation.
Authentication
Integrated Devise to handle authentication within the application.
Pages and Functionality
Introduced a splash page providing a welcoming interface for login and sign-up.
Implemented dedicated login and sign-up pages.
Developed the categories page.
Implemented the expenses page.
Testing
Implemented comprehensive integration tests and model tests to ensure the correctness of the application

I need help

Hi @Graciano1997,  kindly share the issues you are experiencing with the deployment on render, please this is a comment not a review 👍🏿

I have did all my best

I am losing the hope, time

Hi @Graciano1997, Kindly checkout this thread on how some student solved this issue, did you create a postgres database?

yes I created a Postgres db I am checking now even though I am without time

@prolajumokeoni  my review is pending what can I do ?

Hi @Graciano1997 , I was giving you some time to try to fix the build error

thanks I still doing

I will  drop a review while you fix the deployment issue, when I click on a category to add a transaction I see the error below, please take out time to fix these issues 👍🏿

Yes

Hello @Graciano1997 👋,
This is a manual review.
Good job making all the necessary changes 👍
Approved ✅
Keep rocking 🚀!


Create/ Edit Project Modal - Jobs created Modal

need to add a new field to the Project modal for Jobs Created.
will require a new field in the database for Projects

As a (type of user)
I want (goal)
so that (reason/ why/ value)
Acceptance Criteria
Scenario 1:
GIVEN ... (you or your condition)
WHEN ... (what you do)
THEN ... (what you see)
Scenario 2:
xxxxxx
Validation Rules?
(If yes, list here)
Screenshot
(Link above will always be the most up-to-date version.)

@maria-kuzmenko designs modification needed here

@Connor1892, what are jobs?


add SpellCheck CI

I have used this for roughly three weeks in SummationByPartsOperators.jl. It seems to work fine. We should wait until #1411 is merged to get some fixes before merging this. But this is a nice opportunity to see it in action when it fails.
If you agree, I can distribute this in the Trixi.jl framework.

If you agree, I can distribute this in the Trixi.jl framework.

Sounds great, thanks!

We need to ignore the false positives since they made the previous CI run fail: https://github.com/trixi-framework/Trixi.jl/actions/runs/4816348133/jobs/8575876587#step:3:157

Ah, great, thanks, I didn't see that!


AWS region configuration issue - Fargate task defintion log group region

What happened?
I was following this tutorial: https://github.com/pulumi/examples/tree/master/aws-ts-containers, using an AWS profile.
In my ~.aws folder there is a credentials file and configuration file with a region defined for the profile.
Creation of Fargate service failed with timeout, root cause was failure of the task to execute as the log group region was "null"
Issue resolved once region was configured in Pulumi yaml file.
Expected Behavior
Either use region configured for the aws profile, or report an error.
Steps to reproduce

Using AWS CLI, create a profile, configure credentials and a region for the profile
Follow the tutorial https://github.com/pulumi/examples/tree/master/aws-ts-containers
Do not configure region in Pulumi yaml
Run pulumi up

Output of pulumi about
CLI
Version      3.79.0
Go Version   go1.21.0
Go Compiler  gc
Plugins
NAME        VERSION
aws         5.42.0
aws-native  0.8.0
awsx        1.0.5
docker      3.6.1
nodejs      unknown
Host
OS       darwin
Version  13.5.1
Arch     arm64
This project is written in nodejs: executable='/usr/local/bin/node' version='v18.13.0'
Current Stack: organization/observability-playground/observability-playground-1
Found no resources associated with observability-playground-1
Found no pending operations associated with observability-playground-1
Backend
Name           MacBook-Air-2.localdomain
URL            file://./data
User           ***
Organizations
Dependencies:
NAME                VERSION
@pulumi/aws-native  0.8.0
@pulumi/awsx        1.0.5
@pulumi/pulumi      3.79.0
@types/node         16.18.46
Pulumi locates its logs in /var/folders/mx/64p_g9kx79n58s0pm8718y3m0000gn/T/ by default
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

I was able to reproduce this issue locally, thank you @ygyg70 for the detailed description.
It's definitely an issue in our awsx implementation, not in the provider itself, so I will move the issue again. It looks like it's about the way we retrieve the default region here, or rather about the implementation of the getRegion function in utils.


[API DOC] REVAMP -> Use JSdoc to generate API reference

Description
Our documentation is constantly obsolete and very tough to navigate.
We want to enable developers to harness the full power of Dust, and for this we need a strong documentation.
Because of our current architecture, there's no good way to fully automate the generation of the doc. A good way is then do use JSDoc to describe each endpoint.
What this PR adds:

A few first JSDoc comments on top of conversation, messages and FF endpoints
A new script npm run docs to generate the OpenAPI file and validate it.
IN THE FUTURE:  All API changes should be reflected in the corresponding JSDoc comment above

We will then use a service than can interpret OpenAPI 3.0 files to generate a user friendly, live-testable API reference by pulling daily the current public/swagger.json file.
How the doc will end up looking like:

Risk
If moving forward, we will end up retiring the /docs repo.
Deploy Plan
The public/swagger.json file needs to be valid at all times. The npm run docs script takes care of validation, but a enw status check needs to be added on PRs running npx swagger-cli validate public/swagger.json

What's the plan with docs.dust.tt ? transition some of it to Notion and remove the rest? or the other way around?

Replaced by this entirely. Guides will be ported to readme.io solution that also embeds the API ref

Does that make sense?


Implement second round of contentdm mappers

Includes:

oai.contentdm.lapl
oai.contentdm.csudh
oai.contentdm.pepperdine

Ready for review. You only need to look at the last 3 comments.


REST Session Timeout not behaving as expected

During development of PyU4V features breakpoints were added to Unisphere to investigate server side actions during calls.
It was noticed that instead of timing out after 120 seconds by default as set here and returning a requests.Timeout exception, an erroneous PyU4V.utils.exception.VolumeBackendAPIException was raised instead with a very misleading error message:
PyU4V.utils.exception.VolumeBackendAPIException: Bad or unexpected response from the storage volume backend API: Error GET Array. The status code received is None and the message is None.
This should be fixed asap, we gracefully handle timeouts and respond with a suitable message here
It may be PyU4V related issue, it could be a requests related issue, but given the extent to which requests is used globally I would assume that its something to do with PyU4V.

Thanks Michael.  We will take a look at this.


How to dynamically add/remove tasks in runtime?

It works fine as like djceley's DatabaseScheule did. But the only problem is that it doesn't seem to send tasks that were added in runtime, then i restart the celery-beat, the tasks that were added before will be sent successfully.
So, is it possible to dynamically add/remove tasks without restarting celery-beat?
Thanks.

It is possible, but the sync interval between beat's internal state and the database is several minutes by default. This of course means the task you added will not be picked up instantly. You can change the DatabaseScheduler.sync_every value to a faster one.


New error possibly related to new options in 3.0

Getting the following error:
Potentially unhandled rejection [5] TypeError: Cannot match against 'undefined' or 'null'.
  at module.exports.postcss.plugin (/Users/ben/Dev/Git/PROJECT/node_modules/postcss-pseudo-class-enter/index.js:6:63)
  at creator (/Users/ben/Dev/Git/PROJECT/node_modules/postcss/lib/postcss.js:150:35)
  at Function.get (/Users/ben/Dev/Git/PROJECT/node_modules/postcss/lib/postcss.js:159:27)
  at Processor.normalize (/Users/ben/Dev/Git/PROJECT/node_modules/postcss/lib/processor.js:136:12)
  at new Processor (/Users/ben/Dev/Git/PROJECT/node_modules/postcss/lib/processor.js:51:25)
  at PostCSS.postcss [as engine] (/Users/ben/Dev/Git/PROJECT/node_modules/postcss/lib/postcss.js:73:10)
  at PostCSS._render (/Users/ben/Dev/Git/PROJECT/node_modules/accord/lib/adapters/postcss/4.x - 5.x.js:35:24)
  at PostCSS.Adapter.render (/Users/ben/Dev/Git/PROJECT/node_modules/accord/lib/adapter_base.js:122:19)

I think this is to do with the new options you have added in 3.0 (which you don't mention in the changelog – caught me off guard!). I don't fully understand these options. Do you think you could elaborate a little in the readme? I don't understand what prefix is, or why the default is '-', and I would like to know exactly what the outline option will output and why it is a string option instead of a boolean.
Thanks for all your help! I have been happily using this package for months so if I can help in any way, let me know.

@Leeds-eBooks This error is happening because the updated plugin code assumes that an options object of some kind will be passed in. The code uses destructuring to extract the options out:
module.exports = postcss.plugin('postcss-pseudo-class-enter', ({
	outline = false,
	prefix = ''
}) => {

The problem is, if you don't pass an options object in, the function receives undefined, so what gets evaluated is something like the following:
const { outline = false, prefix = '' } = undefined

The quick fix is to just pass in an empty object for the options - require('postcss-pseudo-class-enter')({}).

Ah ok, thanks. What about:
module.exports = postcss.plugin('postcss-pseudo-class-enter', ({
	outline = false,
	prefix = ''
} = {}) => {

I am an hour or so from updating all my PostCSS plugins with safer destructuring. I finished updating all the plugins in need of this change last weekend, and I finished the postcss-partial-import update last night, and I'm making one more error handling improvement to postcss-tape. Once that's done, I can update all the dependencies, push the updates, wait 15 to be sure there were no errors from Travis (I run the tests locally first, too). When that all goes well, big ol' npm publish (on 20 odd plugins this would effect?)

Updates complete. Thank you, everyone!


Python: Azure Cosmos DB NoSQL Vector Store & Collection implementation

Motivation and Context

Description

Contribution Checklist

[ ] The code builds clean without any errors or warnings
[ ] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[ ] All unit tests pass, and I have added new tests where possible
[ ] I didn't break anyone :smile:

Python Unit Test Overview

Tests
Skipped
Failures
Errors
Time

2635
4 :zzz:
0 :x:
0 :fire:
1m 4s :stopwatch:


Steamroller challenges.   i did it right . but it does not let me go through it.

Challenge Name

Steamroller
Issue Description

do not know why it does not let me through this challenge .
Browser Information

every result is right .

Browser Name, Version:  chrome .
Operating System:  mac os
Mobile, Desktop, or Tablet:  Desktop

Your Code

function steamrollArray(arr) {
  // I'm a steamroller, baby

  for(var i=0;i<arr.length;i++){
    if(!Array.isArray(arr[i])){
      retArr.push(arr[i]);
    }else{
      retArr.concat(steamrollArray(arr[i]));
    }
  }
  return retArr;
}

var retArr = [];

steamrollArray([1, [2], [3, [[4]]]]);

Screenshot

@xiaomingfuckeasylife thanks for the issue. This is very similarly related to https://github.com/FreeCodeCamp/freeCodeCamp/issues/9199. Closing as duplicate. To complete the challenge, do not use global variable because the tests call your function over and over, which will modify any global variables you may have. Happy coding!


Merge to publish

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

File
Status
Preview URL
Details

docs/actionable-messages/identity-linking.md
:white_check_mark:Succeeded
View

docs/actionable-messages/security-requirements.md
:white_check_mark:Succeeded
View

docs/rest/get-started.md
:white_check_mark:Succeeded
View

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.


Upgrade v4 host dotnet sdk runtime image from preview to release

v4 host still references the preview (https://github.com/Azure/azure-functions-docker/blob/dev/host/4/bullseye/amd64/dotnet/dotnet-isolated/dotnet-isolated.Dockerfile#:~:text=FROM mcr.microsoft.com/dotnet/sdk%3A6.0.100-rc.2 AS runtime-image) version of the sdk - are there plans to get on the released version?
Production functions deployed w/ the v4 runtime are listing the RC image in log streams - this is generally confusing since the sdk and other services are GA.

Tagging @pragnagopa / @balag0  ,  for more insights

Thank you for making us aware of this.  We are rolling out a new version (v4.1.1) that will no longer log the RC image in log streams. Images that we use by default in production are -composite.template files such as https://github.com/Azure/azure-functions-docker/blob/dev/host/4/bullseye/amd64/dotnet/dotnet-isolated/dotnet-isolated-composite.template#L4
When the current release is rolled out to your application, it will no longer report rc image runtimes.

Sounds great thanks for following up

The above removes nightly from non-production images.  You should still receive the fix as the release rolls out.  :)


Fix unused import

Modulus Pull Request
Description
Removes unused import. This was pointed out during internal QA testing.
Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[ ] The documentation is up to date with these changes.
[ ] The CHANGELOG.md is up to date with these changes.
[x] An issue is linked to this pull request.

Dependencies

/blossom-ci


Code for "Writing a basic Plutus app in the Plutus Playground" doesn't compile on online Plutus Playground

The code in the "Writing a basic Plutus app in the Plutus Playground" tutorial doesn't compile in the online playground.
The full code is:
import Playground.Contract

import           Control.Monad          (void)
import           Data.Aeson             (FromJSON, ToJSON)
import qualified Data.Text              as T
import           GHC.Generics           (Generic)
import           Ledger
import qualified Ledger.Ada             as Ada
import qualified Ledger.Constraints     as Constraints
import qualified Ledger.Typed.Scripts   as Scripts
import           Plutus.Contract
import qualified PlutusTx               as PlutusTx
import           PlutusTx.Prelude
import qualified Prelude                as Haskell
import           Schema
import           Wallet.Emulator.Wallet

data SplitData =
    SplitData
        { recipient1 :: PubKeyHash -- ^ First recipient of the funds
        , recipient2 :: PubKeyHash -- ^ Second recipient of the funds
        , amount     :: Ada -- ^ How much Ada we want to lock
        }
    deriving stock (Haskell.Show, Generic)

-- For a 'real' application use 'makeIsDataIndexed' to ensure the output is stable over time
PlutusTx.unstableMakeIsData ''SplitData
PlutusTx.makeLift ''SplitData

validateSplit :: SplitData -> () -> ScriptContext -> Bool
validateSplit SplitData{recipient1, recipient2, amount} _ ScriptContext{scriptContextTxInfo} =
    let half = Ada.divide amount 2 in
    Ada.fromValue (valuePaidTo scriptContextTxInfo recipient1) >= half &&
    Ada.fromValue (valuePaidTo scriptContextTxInfo recipient2) >= (amount - half)

data Split
instance Scripts.ValidatorTypes Split where
    type instance RedeemerType Split = ()
    type instance DatumType Split = SplitData

splitValidator :: Scripts.TypedValidator Split
splitValidator = Scripts.mkTypedValidator @Split
    $$(PlutusTx.compile [|| validateSplit ||])
    $$(PlutusTx.compile [|| wrap ||]) where
        wrap = Scripts.wrapValidator @SplitData @()

data LockArgs =
        LockArgs
            { recipient1Wallet :: Wallet
            , recipient2Wallet :: Wallet
            , totalAda         :: Ada
            }
    deriving stock (Haskell.Show, Generic)
    deriving anyclass (ToJSON, FromJSON, ToSchema)

type SplitSchema =
        Endpoint "lock" LockArgs
        .\/ Endpoint "unlock" LockArgs

lock :: Promise () SplitSchema T.Text ()
lock = endpoint @"lock" (lockFunds . mkSplitData)

unlock :: Promise () SplitSchema T.Text ()
unlock = endpoint @"unlock" (unlockFunds . mkSplitData)

mkSplitData :: LockArgs -> SplitData
mkSplitData LockArgs{recipient1Wallet, recipient2Wallet, totalAda} =
    SplitData
        { recipient1 = walletPubKeyHash recipient1Wallet
        , recipient2 = walletPubKeyHash recipient2Wallet
        , amount = totalAda
        }

lockFunds :: SplitData -> Contract () SplitSchema T.Text ()
lockFunds s@SplitData{amount} = do
    logInfo $ "Locking " <> Haskell.show amount
    let tx = Constraints.mustPayToTheScript s (Ada.toValue amount)
    void $ submitTxConstraints splitValidator tx

tx = Constraints.mustPayToTheScript s (Ada.toValue amount)

unlockFunds :: SplitData -> Contract () SplitSchema T.Text ()
unlockFunds SplitData{recipient1, recipient2, amount} = do
    let contractAddress = Scripts.validatorAddress splitValidator
    utxos <- utxosAt contractAddress
    let half = Ada.divide amount 2
        tx =
            collectFromScript utxos ()
            <> Constraints.mustPayToPubKey recipient1 (Ada.toValue half)
            <> Constraints.mustPayToPubKey recipient2 (Ada.toValue $ amount - half)
    void $ submitTxConstraintsSpending splitValidator utxos tx

endpoints :: Contract () SplitSchema T.Text ()
endpoints = selectList [lock, unlock]

mkSchemaDefinitions ''SplitSchema
$(mkKnownCurrencies [])

And results in three compilation errors:
Compilation Error, Line 68, Column 24 (jump)
 error:
    * Variable not in scope: walletPubKeyHash :: Wallet -> PubKeyHash
    * Perhaps you meant one of these:
        `walletPubKeyHashes' (imported from Wallet.Emulator.Wallet),
        `getPubKeyHash' (imported from Ledger),
        `walletPubKey' (imported from Wallet.Emulator.Wallet)
   |
68 |         { recipient1 = walletPubKeyHash recipient1Wallet
   |                        ^^^^^^^^^^^^^^^^
Compilation Error, Line 69, Column 24 (jump)
 error:
    * Variable not in scope: walletPubKeyHash :: Wallet -> PubKeyHash
    * Perhaps you meant one of these:
        `walletPubKeyHashes' (imported from Wallet.Emulator.Wallet),
        `getPubKeyHash' (imported from Ledger),
        `walletPubKey' (imported from Wallet.Emulator.Wallet)
   |
69 |         , recipient2 = walletPubKeyHash recipient2Wallet
   |                        ^^^^^^^^^^^^^^^^
Compilation Error, Line 79, Column 37 (jump)
 error:
    Variable not in scope: s
   |
79 | tx = Constraints.mustPayToTheScript s (Ada.toValue amount)
   |                                     ^

I ran into the same issue, and I was able to overcome up with the changes I proposed in the PR above.
Specifically:
Replace walletPubKeyHash with (pubKeyHash . walletPubKey).
Also, the wallet id you need to pass should be 128 characters long.
so instead of just 1 you need to pass<PHONE_NUMBER>0000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000001


passing a class into the image tag

is there a way in which i can pass a css class for the image in here??
Editor.insertEmbed(cursorLocation, 'image', url);
help any one Thanks

Hello,
I have the same problem, did you find any solution for that?

This issue is stale because it has been open for 60 days with no activity.

This issue was closed because it has been inactive for 7 days since being marked as stale.


Show indicator card on click

Hi, thanks for your custom card!
My request is: Clicking on an indicator should show that indicator's card (e.g. history graph for a sensor entity, as implemented in custom:simple-thermostat).

Hello! Cool idea! will do it


bug: Ion-Toggle still works with PopUp on Screen

Type: bug
Ionic Version: 1.x
Platform: all

After a popup stays active, the ion-toggle continues to function normally.
I Just don't know why.
Even the pure backdrop component works normally on locking ion-toggle.

Hello! While we appreciate you opening an issue with us, I am going to close this issue and ask that you open a new one that has the issue template filled out please. Thanks for using Ionic!


basics

WHEN I load the portfolio
THEN I am presented with a page containing a header, a section for content, and a footer

done


Update nats-streaming server to actual version

Update nat-streaming server to 0.14.1 (not latest version).
Also add nats/config.conf to Docker-image cyberway/cyberway and use it from docker-compose-events.yml. Remove download nats/config.conf from hooks on testnet.

PR #810


Support delete option for NICs and Disks for VMs in Azure CLI

Resource Provider
CRP
Description of Feature or Work Requested
User experience:

In text:
az vm create
-n MyVm
-g MyResourceGroup
--public-ip-address-dns-name MyUniqueDnsName
--image ubuntults
--size Standard_DS2_v2
--generate-ssh-keys
--data-disk-sizes-gb 10 20
--data-disk-delete-option 0=Detach 1=Delete /*Use a singular value to apply on all data disks, or use = to configure the delete behavior for individual disks
--os-disk-delete-option Delete
--nic-delete-option = /*Use a singular value to apply on all nics, or use = to configure the delete behavior for individual NICs
The new switches do the following
--nic-delete-behavior configures the deleteOption field for the NIC created for the VM
vm.properties.networkProfile.networkInterfaces[0].properties.deleteOption = "Delete"
--os-disk-delete-option Delete configures delete option for the OS Disk
cstorageProfile.dataDisks[*].osDisk= "Delete"
--data-disk-delete-option configures the delete option for data disks as specified
vm.properties.storageProfile.dataDisks[*].deleteOption = "Delete"
Swagger PR containing the deleteOption for VM NICs: https://github.com/Azure/azure-rest-api-specs/pull/13449
Swagger PR for disk deleteOption: https://github.com/Azure/azure-rest-api-specs/pull/13657
Minimum API Version Required
2021-03-01
Swagger Link

Target Date
5/25

hi @szimmerMicrosoft there's no concrete ETA on when 2021-03-01 Swagger will be avaiable, there's high risk to catch 5/25 timeline. Based on info I got from Adam, Swagger will be ready for review at week of 4/26. With it, I dont' think features in 2021-03-01 could catch 5/25 release.

Hi @szimmerMicrosoft, it is only supported for vm create not including vmss create, right?

disk deleteOption is currently only supported for vm, not for VMSS.
We do have the ability to configure it for VMSS on NICs. We could support that now, or we could wait for deleteOption support for both disks and NICs to be possible on VMSS.
Setting deleteOption on VMSS flex for NICs is probably a small minority choice so not a hard requirement now.

The only option for NICs parameter looking here https://github.com/Azure/azure-cli/blob/1ca212af5ec9e41847a628d1df27e87e411b7345/src/azure-cli/azure/cli/command_modules/vm/custom.py#L873
Appears to be resourceIds.
vmName is used as the singular NIC name if nic_type == new.
Using the name as resourceId if the user provides resource id makes sense.
If the NIC being created is implicit I don't see any value in letting the user specify the VM name for the deleteOption, there is only one NIC, so let it just be deleteOption = X

The only option for NICs parameter looking here
https://github.com/Azure/azure-cli/blob/1ca212af5ec9e41847a628d1df27e87e411b7345/src/azure-cli/azure/cli/command_modules/vm/custom.py#L873
Appears to be resourceIds.
vmName is used as the singular NIC name if nic_type == new.
Using the name as resourceId if the user provides resource id makes sense.
If the NIC being created is implicit I don't see any value in letting the user specify the VM name for the deleteOption, there is only one NIC, so let it just be deleteOption = X
Let me know if I am misunderstanding how the create command works.

We also could use existing nics and set deleteOption for them. Both name and resource id format are allowed for existing nics. If users provide name, we will format it to resource id:
https://github.com/Azure/azure-cli/blob/bc8fca8df0b9087e44381def0afc8fee7768083e/src/azure-cli/azure/cli/command_modules/vm/_validators.py#L960-L964

Oh great,so identifying those via name makes sense.
However the user specifies the NICs, using that as the key I think is sensible.


CLI: Cleanup duplicate entries for operator ID and Key

There are duplicate entries for operator/treasury account 0.0.2 and key. We should probably eliminate duplicates for consistency: https://github.com/hashgraph/full-stack-testing/blob/main/solo/src/core/constants.mjs#L76-L80

Need to check which terminology we should use for account 0.0.2. Should it be treasury or operator?
Please close this ticket if both terminologies are required.

The treasury account is a special account fixed as 0.0.2.  And, we want to use the treasury account to fund all of the account key updates so they are no longer defaulted to the genesis key.
The operator is the account we want to use to perform/fund/sign Hedera operations on a wallet or NodeClient.  The operator could be any account, and it can be overridden with environment variables.  They are two separate things.  Both terminologies are required.


Google Maps map with Circle with a center of (NaN, NaN) crashes the app with no error message

Steps to Reproduce

Run flutter create bug.
Install the Google Maps package
Create Circles with a center of (NaN, NaN) and a radius of 0.0
See the app crash with no error message

Expected results:
Obviously this is a bug on the user's end, but the app crashes without an error message, so it is quite hard to debug. It would be nice to throw an error that can be caught by the IDE's debugger
Actual results:
The app crashes with no error message and the debug session ends.

Hi @thib92,
Can you provide flutter doctor -v and a minimal code sample that shows the issue ?
I tried with a demo code, but was able to see the map open properly without any issues. Maybe I am missing something in the demo code.

code sample
import 'package:flutter/material.dart';
import "package:flutter/cupertino.dart";
import 'package:google_maps_flutter/google_maps_flutter.dart';

num lt = 0/0;
num ln = 0/0;

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      home: TrialMaps(),
    );
  }
}

class TrialMaps extends StatefulWidget {

  @override
  _TrialMapsState createState() => _TrialMapsState();
}

class _TrialMapsState extends State<TrialMaps> {

  LatLng firstLocation = LatLng(lt, ln);

  GoogleMapController _mapController;

  @override
  Widget build(BuildContext context) {
    print(lt.isNaN);
    print(ln.isNaN);

    Set<Circle> circles = Set.from([Circle(
      circleId: CircleId("my id"),
      center: LatLng(lt, ln),
      radius: 0,
    )]);
    return GoogleMap(
      mapType: MapType.normal,
      myLocationEnabled: true,
      circles: circles,
      onCameraMove: null,
      initialCameraPosition: CameraPosition(
        target: firstLocation,
        zoom: 10,
      ),
      onMapCreated: (GoogleMapController controller) {
        setState(() {
          _mapController = controller;
        }
        );
      },
    );
  }
}

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


ate core project has the xsd not found issue on windows eclipse env.

Not sure why windows eclipse env. couldn't find the xsd.

org.xml.sax.SAXParseException: schema_reference.4: Failed to read schema document 'http://www.bigtester.org/schema/ate/ate-2.0.xsd', because 1) could not find the document; 2) the document could not be read; 3) the root element of the document is not xsd:schema. at org.apache.xerces.util.ErrorHandlerWrapper.createSAXParseException(Unknown Source)

this issue only exists on win xp eclipse env.


Problem: hare-{repair,rebalance} don't use private IP addresses when communicating to hax

As of #1301 hax doesn't listen to <IP_ADDRESS> and thus it is not accessible via localhost.
hare-repair and hare-rebalance must learn the virtual address of the current node first to use it in their CURL invocations.
Suggestion: re-implement the mentioned scripts in Python and leverage hax/util.py from hax module. Motivation:

There is get_hax_ip_address() function which does all the magic
There is a auto-retry mechanism for redundant and fault-tolerant communication to Consul.

@knekrasov is this issue still valid?

@vaibhavparatwar In fact, yes. SNS operations were implemented but not officially supported by current version of CORTX. Once we get back to this feature, this problem will have to be resolved somehow.

closing this issue as the SNS feature has been backlogged per this comment ->
https://github.com/Seagate/cortx-hare/issues/1258#issuecomment-959677011

Vaibhav Prakash Paratwar commented in Jira Server:

Its addressed recently in https://jts.seagate.com/browse/CORTX-29486

Vaibhav Prakash Paratwar commented in Jira Server:

.

Vaibhav Prakash Paratwar commented in Jira Server:

.


fix: accelerate not found

accelerate may not bye available in some envs in sd-webui,   and in most cases training a lora model only need on gpu, so remove accelerate


KConMapperProperty issue when using fromClasses

Hi I am currently facing an issue with generating the right Mapper with the models I have. Please see below. It generates correctly for TestRepo.toTestRemote() but doesn't for TestRemote.toTestRepo. I currently can't move the kConMapper to TestRemote as it lives in another module and doesn't have access to TestRepo.
Test classes:
data class TestRemote(
    val uid: String,
    val name: String
)

@KConMapper(fromClasses = [TestRemote::class], toClasses = [TestRemote::class])
data class TestRepo(
    @KConMapperProperty(["uid"])
    val userUid: String,
    @KConMapperProperty(["name"])
    val fullName: String
)

which generates
fun TestRemote.toTestRepo(
   userUid: String,
   name: String
) = TestRepo(
   userUid = userUid,
   name = name
)

fun TestRepo.toTestRemote() = TestRemote(
   uid = this.userUid,
   fullName = this.name,
)

Could you please verify if the bug still exists in your project with KConMapper version 1.0.0-alpha04?

still exists, doesn't map the propertyName for TestRemote.toTestRepo()


dismissible.dart code cleanup

While I was working on #149966, I noticed a couple of ways that _DismissibleState could be tweaked to be easier to follow.
Changes are described in comments below, and we're currently awaiting a test exemption.

I was curious why _isActive is written that way, when it seems redundant given how it's used. From git log -p -G isActive --follow packages/flutter/lib/src/widgets/dismissible.dart, it looks like the story is that it used to have a more complicated definition, and also to be used in more contexts; then commit a78d2c9e8ba8a673b6257c35098eed626af3e403 (from pre-GitHub, in 2016) made some simplifications which opened the way for this one.

I was curious about that as well—good to know!
(I should probably brush up on the various git command options 🙂)

Indeed, there's a lot of useful stuff there. Here's my own short writeup of tips for git log and reading Git history:
https://github.com/zulip/zulip-mobile/blob/e352f563e/docs/howto/git.md#reading-history

test-exempt: code refactor with no semantic change