Add AWS CDK examples for creating and using a layer from the Serverless Application Repository

What were you initially searching for in the docs?
Using the AWS CDK, I wanted to deploy a Lambda layer from the Powertools Serverless Application Repository ARN for re-use within my CDK application.
The Lambda Layer documentation does not include how to do this.
Is this related to an existing part of the documentation? Please share a link
https://awslabs.github.io/aws-lambda-powertools-python/#lambda-layer
Describe how we could make it clearer
A code snippet and details could be provided to make it easier for a customer to get up and running faster.
If you have a proposed update, please share it here
Using similar language to the existing documentation, but some rewording could also used to provide the overall detail with SAM and CDK examples below it.
If using the AWS CDK, you can create include this SAR App and lock to a specific semantic version. The Layer can be used in the same CDK application. Once deployed, it'll be available across the account this is deployed to.
** insert a TypeScript CDK example **
# Create the AWS Lambda Powertools for Python layer
powertools_arn = 'arn:aws:serverlessrepo:eu-west-1:057560766410:applications/aws-lambda-powertools-python-layer'
powertools_application = aws_sam.CfnApplication(self, 'AWSLambdaPowertoolsApplication',
                                                location={'applicationId': powertools_arn,
                                                          'semanticVersion': '1.12.0'})
powertools_resource = cdk.CfnResource(powertools_application, 'AWSLambdaPowertoolsResource',
                                      type='AWS::Serverless::Application',
                                      properties={'Location': {
                                          'ApplicationId': powertools_arn,
                                          'SemanticVersion': '1.12.0'
                                      }})
powertools_layer = aws_lambda.LayerVersion.from_layer_version_arn(
    self, 'AWSLambdaPowertoolsLayer', powertools_resource.get_att("Outputs.LayerVersionArn").to_string())

# Reference the Layer in a Lambda definition
my_function = aws_lambda.Function(
    self, "MyFunction",
    layers=[powertools_layer],
    runtime=aws_lambda.Runtime.PYTHON_3_8,
)

Possibly a Python CDK example :P

The example I included was Python CDK. 😁

+1 for this, was on my todo list for a long time ;)

is there a way to always get the latest published version?

Hi Andrew,
Thanks for opening the issue and providing the sample code. I will find a good spot in the docs to add it.

is there a way to always get the latest published version?

sadly no, you would need to run CLI or some custom resource to fetch it and sort by version

is there a way to always get the latest published version?

sadly no, you would need to run CLI or some custom resource to fetch it and sort by version

For CDK, I like to load that kind of thing via configuration settings.

is there a way to always get the latest published version?

sadly no, you would need to run CLI or some custom resource to fetch it and sort by version

For CDK, I like to load that kind of thing via configuration settings.

I will need to write a function that takes the powertools version from my pipfile.lock file and set the layer version parameter. annoying, but will work.

@risenberg-cyberark  for docs i would just use the cdk.json  or runtime context to configure the version of powertools.

@risenberg-cyberark @austoonz  here is a full working example using Poetry:
Directory structure like below, with cdk folder for the AWS CDK code and hello-world with the lambda code.
.
├── cdk
│   ├── Makefile
│   ├── README.md
│   ├── app.py
│   ├── cdk.json
│   ├── lambda_stack
│   │   ├── __init__.py
│   │   └── lambda_stack.py
│   ├── poetry.lock
│   └── pyproject.toml
├── hello-world
│   └── app.py

Source for: cdk/pyproject.toml
[tool.poetry]
name = "cdk"
version = "0.1.0"
description = "CDK code for the hello world example"
authors = ["Michael Brewer<EMAIL_ADDRESS>
[tool.poetry.dependencies]
python = "^3.8"
"aws-cdk.core" = "^1.96.0"
"aws-cdk.aws-sam" = "^1.96.0"
"aws-cdk.aws-lambda" = "^1.96.0"

[tool.poetry.dev-dependencies]

[build-system]
requires = ["poetry-core>=1.0.0"]
build-backend = "poetry.core.masonry.api"

Source code for the main stack cdk/lambda_stack/lambda_stack.py
import pathlib

import aws_cdk.aws_lambda as aws_lambda
import aws_cdk.aws_sam as sam
from aws_cdk import core as cdk

class LambdaStack(cdk.Stack):
    def __init__(self, scope: cdk.Construct, construct_id: str, **kwargs) -> None:
        super().__init__(scope, construct_id, **kwargs)

        lambda_src_dir = str(pathlib.Path(__file__).parent.parent.parent) + "/hello-world/"

        # Reference the Layer in a Lambda definition
        aws_lambda.Function(
            self,
            "MyFunction",
            layers=[self.get_powertools_lambda_layer()],
            runtime=aws_lambda.Runtime.PYTHON_3_8,
            handler="app.lambda_handler",
            code=aws_lambda.Code.from_asset(lambda_src_dir),
        )

    def get_powertools_lambda_layer(self):
        powertools_version = self.node.try_get_context("powertools_version")
        powertools_arn = "arn:aws:serverlessrepo:eu-west-1:057560766410:applications/aws-lambda-powertools-python-layer"
        powertools_application = sam.CfnApplication(
            self,
            "AWSLambdaPowertoolsApplication",
            location={
                "applicationId": powertools_arn,
                "semanticVersion": powertools_version,
            },
        )
        powertools_resource = cdk.CfnResource(
            powertools_application,
            "AWSLambdaPowertoolsResource",
            type="AWS::Serverless::Application",
            properties={
                "Location": {
                    "ApplicationId": powertools_arn,
                    "SemanticVersion": powertools_version,
                }
            },
        )
        return aws_lambda.LayerVersion.from_layer_version_arn(
            self,
            "AWSLambdaPowertoolsLayer",
            powertools_resource.get_att("Outputs.LayerVersionArn").to_string(),
        )

@heitorlessa  @am29d ☝🏼  what do you think?

@michaelbrewer i got the same code in my project but we are using pipenv instead. I'm working on adding a code that will extract the powertools version from the pipenv.lock file and use that for the layer version is the cdk. that way when you upgrade your layer locally, it will also be used automatically for uploaded lambdas layer.

@michaelbrewer i got the same code in my project but we are using pipenv instead. I'm working on adding a code that will extract the powertools version from the pipenv.lock file and use that for the layer version is the cdk. that way when you upgrade your layer locally, it will also be used automatically for uploaded lambdas layer.

and for Poetry :)

Coz Python users use: requirements.txt, setup.py, poetry or pipenv for dependencies.

@risenberg-cyberark  i guess self.node.try_get_context("powertools_version") can be abstracted into various choices to get the powertools version

Hey everyone - two quick updates

@am29d is on PTO until next week so this will come after 1.14.0 release this week

We've decided to publish a public Lambda Layer from our accounts to ease consumption.

We agreed that there are too many customers having challenges with the SAR App approach. Despite providing semantic versioning, these deployment gymnastics to get sem versions for Lambda Layers cause more harm than good whether that is SAM, CDK, Serverless framework etc.
We'll take on these maintenance costs by

Publishing a new public Lambda Layer upon new releases
Create a dynamic GitHub Badge that shows the latest Lambda Layer available
Clean up old Lambda Layers every 6 months (we'll document this)
Continue to collect customers feedback to feed in to Lambda Product management team on the need for Lambda Layers Aliases to solve this problem in the long term

We'll start working on that once Alex is back.
Thank you

@heitorlessa , arent you already publishing a new layer every version?

@heitorlessa , arent you already publishing a new layer every version?

Yes, but it's different. We need additional infrastructure, a cadence for removing old versions without breaking anyone (SAR doesn't as it's local to your account), and a public API to fetch the latest Layer and by version(s).

@heitorlessa @michaelbrewer  @am29d
so after i finally got enough dev permissions to use SAR, i came to a realization that you dont need to deploy cdk.CfnResource in the cdk example.
`from aws_cdk import core, aws_sam, aws_lambda
class LambdaLayers(core.Construct):
POWER_TOOLS_VER = '1.13.0'
POWER_TOOLS_ARN = 'arn:aws:serverlessrepo:eu-west-1:057560766410:applications/aws-lambda-powertools-python-layer-extras'
POWERTOOLS_BASE_NAME = 'AWSLambdaPowertools'

# pylint: disable=redefined-builtin,invalid-name,no-value-for-parameter, too-many-arguments
def __init__(self, scope: core.Construct, id_: str) -> None:
    super().__init__(scope, id_)
    self.powertools_layer = self._create_powertools_layer()

def _create_powertools_layer(self: str) -> core.Resource:
    # Create the AWS Lambda Powertools for Python layer
    powertools_application = aws_sam.CfnApplication(
        self,
        f'{LambdaLayers.POWERTOOLS_BASE_NAME}Application',
        location={
            'applicationId': LambdaLayers.POWER_TOOLS_ARN,
            'semanticVersion': LambdaLayers.POWER_TOOLS_VER
        },
    )

    return aws_lambda.LayerVersion.from_layer_version_arn(
        self,
        f'{LambdaLayers.POWERTOOLS_BASE_NAME}WithExtras',
        powertools_application.get_att("Outputs.LayerVersionArn").to_string(),
    )

`

@heitorlessa should we be adding CDK examples along side the AWS Sam instructions?

@michaelbrewer I'd like to wait for more customers to raise before we do. Terraform is more widely used, so I'd like to add all of them as soon as we get our next biggest chunk of work done - Public Lambda Layers ARN by @am29d

Added both CDK and Serverless framework examples - This will be available in the next release. Closing this now and tagging to share when it's released.


Unable to get directories (or files) with GetListing

I'm unable to get the directories (or even files) with GetListing:
                FtpClient ftpConn = new FtpClient();
                ftpConn.Host = FtpServer;
                ftpConn.Port = FtpPort;
                ftpConn.Credentials = new System.Net.NetworkCredential(Username, Password);
                ftpConn.SslProtocols = System.Security.Authentication.SslProtocols.Tls12;
                ftpConn.EncryptionMode = FtpEncryptionMode.Implicit;
                ftpConn.ValidateCertificate += new FtpSslValidation(Client_ValidateCertificate);
                ftpConn.BulkListing = false;
                //ftpConn.DataConnectionType = FtpDataConnectionType.AutoPassive;
                ftpConn.Connect();

                FtpListItem[] FtpFolders = null;

                FtpFolders = ftpConn.GetListing(Folder);

I'm using the latest version. Any suggestions?

Same here. GetListing returns "Authentication failed because remote party has closed the transport stream" while downloading file using FTPS.
Problem starts from the past month, maybe a Windows Update about TLS is blocking the communication ?

Not sure, too many issues could be at play here...

We have moved temporarily to WinSCP.
We made no modification to the code what so ever. But I asume it's a Windows update because it still works on the production environment, but not no our development. Production is behind on updates.

I have the same problem. Any suggestions?

I have the same problem. Any suggestions? With Filezilla I can connect without any problems.

We also solved using the net assembly of winscp
https://winscp.net/eng/docs/library
It support FTPS even if in the documentation is'nt specified.


Osiris still work ?

I inject DLL file on the game for the third time and nothing work what i can do 😅
this is my laptop performance :
HP OMEN 15.6" Gaming Laptop 144Hz i7-8750H 16GB RAM 1TB HDD 128GB SSD RTX 2070 8GB Max-Q

try another injector it worked for me i use rando now

give me the injector ?
i use this one https://www.dllinjector.com/

braken #0133 ill send you

try this https://github.com/BohanRen/BobHCSGOInjector   work for trusted mod

Its working fine

give me the injector please?

give me the injector please?

https://github.com/BohanRen/BobHCSGOInjector work for trusted mod


Typo in AnimationFunction definitions

Hi,
Noticed what appears to be a typo, AnimationFunctions.Liner looks as though it should be AnimationFunctions.Linear.

Thanks for reporting this. Fixed in the master and will be included in the next update.


subpar:0.1.0

I am submitting

[x] a new package
[ ] an update for a package

Description: Provides easy to use sub figure environments.
I have read and followed the submission guidelines and, in particular, I

[x] selected a name that isn't the most obvious or canonical name for what the package does
[x] added a typst.toml file with all required keys
[x] added a README.md with documentation for my package
[x] have chosen a license and added a LICENSE file or linked one in my README.md
[x] tested my package locally on my system and it worked
[x] excluded PDFs or README images, if any, but not the LICENSE

I have marked this as a draft to wait for the fixes in #587.

I have upstreamed the fixes and updated the generated manual.

PS: make sure to use squash and merge.

Thank you :)


use the analysis server binary from the SDK?

Should we switch over to using the analysis server binary from the SDK, or is it still important to track the source from HEAD?

Unfortunately we can't do this until at least the new task model having landed.

Fixed in https://github.com/dart-lang/dart-services/pull/257


REPO: Date range field on reports always shows an "unavailable" icon on hover.

Date range field on reports always shows an "unavailable" icon on hover.
Remove the misleading on hover style.

on what module does that happen

The issue is prefixed with the letters "REPO" to indicate that this is an issue on the Repository module.


address latest clippy lints (from rust-1.73)

Fixes lint failures like this:
warning: unneeded `return` statement with `?` operator
   --> geozero/src/mvt/mvt_reader.rs:261:13
    |
261 |             return Err(MvtError::GeometryFormat)?;
    |             ^^^^^^^ help: remove it

Hm... cargo +nightly check is failing against some of our dependencies.
It might be a legitimate problem or might just be some problem with nightly.
Do we really want to be running +nightly as part of our CI?

The nightly check was intended for something else -- to catch config errors like these examples. It is only available in nightly. We should probably disable any other linting warnings though to reduce the noise levels

I don't think the tradeoff for running cargo +nightly check is worth the value of linting typos in our cfgs, so I've proposed removing it: https://github.com/georust/geozero/pull/178/commits/c3f76cf0a6455109b43d0e11522d0164e18c9dc1`
What do other people think?
If someone else (@nyurik?) wants to fix the broken check to be more robust, I'd also be fine to keep it.

+1 to not running nightly in CI. Nightly breaks things regularly enough that's it's not worth the tradeoff IMHO

sounds good to me too - config check should probably be commented out until it becomes stable to remind us to re-enable it? Unless it will be part of the warnings by default(?)


Support default data type

Add default_data_type param to show_output_to_df
Add dataframe_assert_equal and its test

@josh803316: That sounds good. I'll see if @jdavenport can take a look, as he has contributed to the library before.


🪲 fixed "Error: TypeError: Cannot read properties of undefined (readi…

…ng 'split')" if failure element is empty
closes #159
closes #241

@uebelack, could you please fix all CI and I will merge it.
Thanks for your help

The bug with calling .split() on undefined object was fixed in #258.
We will add the new unit test from this PR, but the change in the getFirstNonEmptyLine() function is no longer necessary.

And the new report from jest in junit format test overwrites pulsar-test-results.md output file which is used by other test.


Region history

Is there a way to view history of a particular region of lines?
Emacs has this which uses git log output based on lines.
The longer way to do this is via git log/blame manually but I feel this would be pretty useful to quickly go back in time.

I wasn't aware of it before, but i think command gs_show_current_file is very related to this.
I also worked out a very rudimentary plugin for myself: [see here](https://github.com/dufferzafar/sublime-text-3/blob/master/Packages/Git Region History/git_region.py#L1)

This has been implemented actually in 2.26.0


Use Qt5 for melodic

Tested on Ubuntu 18.04 and ROS Melodic.

see also https://github.com/gareth-cross/rviz_satellite/pull/43 which fixes same issue for lunar AND melodic.


[13.0.x] ISPN-13625 Scaling up 8.3 cluster with payload is slower compared to 8.2

https://issues.redhat.com/browse/ISPN-13625
Increase xmit_timeout in TCP stacks to 200ms
Increasing xmit_timeout reduces the number of attempts to resend
LEAVE_RSP when the destination is no longer reachable.
TCP already retransmits messages, so the UNICAST3 retransmission
is only needed when the connection is unstable or the destination's
thread pool is full.
ISPN-13625 Reduce UNICAST3.conn_close_timeout to 5000ms
Reducing conn_close_timeout reduces the number of attempts to resend
LEAVE_RSP when the destination is no longer reachable.

Backport of #9799, just waiting on CI before integrating.

Integrated into 13.0.x!


Bump plugin versions

@Shredder121 @johnktims Is this good to merge?

Looks fine to me. I'll merge it.


Add LCM, IPA, and Scene Lora in Video Inference.

LCM
New LCM acceleration added. When producing images of 512x512 resolution, totaling 16 frames, with all models preheated, the generation time is approximately 100 seconds.

IPA
Added IPA control that can improve likeness. After usage, the similarity of portraits is mostly above 0.65.

Scene Lora
Added a new scene, Lora, which is used in the same way as text2photo.

commit aff26f0a0 has bug

should fix  @bubbliiiing

commit aff26f0 has bug
should fix @bubbliiiing

已经修复。
这是因为在scripts/easyphoto_utils/animatediff/animatediff_infv2v.py的125行，获取了control.control_model.uncond_image_emb[context]，但uncond_image_emb事实上长度只有1，并没有拓展到可以获取的长度；
而与之对应的control.control_model.image_emb的长度是生成视频帧数的2倍，使用context可以正常获取。
如图为controlnet中的定义，具体为何为1还需debug，因为看起来在controlnet定义中，正负两个的长度应该一样：

768*512 16frame + superresolution on A10
W/O LCM ： 250s
W LCM ：155s


Feature request: Option to hide AQI value over picture

It is possible to add an option to hide the brown card with AQI value over picture?

I would also like this feature to further compact the card, e.g:

I would also like this feature to further compact the card, e.g. to look like this:

For this, i think you can try
compact_view: true

I would also like this feature to further compact the card, e.g. to look like this:

For this, i think you can try compact_view: true

Compact view only gets you to this view (e.g. AQI or the text box still in place):

I would also like to hide AQI as well.  Compact Mode doesn't hide it.

I'm using a little bit trick is that using lovelace card mod and add code as below which not have to modify file and receive update as normally.
card_mod:
  style: |
    ha-card > div.preview > div.controls > div >div {
      visibility: hidden

Thanks for the advice

I'm using a little bit trick is that using lovelace card mod and add code as below which not have to modify file and receive update as normally.
card_mod:
  style: |
    ha-card > div.preview > div.controls > div >div {
      visibility: hidden

Doesn't seem to have worked for me.

I'm using a little bit trick is that using lovelace card mod and add code as below which not have to modify file and receive update as normally.
card_mod:
  style: |
    ha-card > div.preview > div.controls > div >div {
      visibility: hidden

Doesn't seem to have worked for me.
Here is how I add them

Thanks for that. I realise now that it only works if compact mode is set to false. What I am really trying to do is remove that panel altogether as with my edited pic on Jan 7th.

Thanks for that. I realise now that it only works if compact mode is set to false. What I am really trying to do is remove that panel altogether as with my edited pic on Jan 7th.

Have you found a solution to hide it?

You can use https://github.com/thomasloven/lovelace-card-mod with the above CSS, confirmed it works.

Thanks for that. I realise now that it only works if compact mode is set to false. What I am really trying to do is remove that panel altogether as with my edited pic on Jan 7th.

With card-mod you can hide every part of data. First you must conifgure card-mod (https://github.com/thomasloven/lovelace-card-mod). I hide PM2,5 and slider with this code:

card_mod:
style: |
ha-card > div.preview > div.controls > div > div {
visibility: hidden}
ha-card > div.preview > div.controls > div.slider > round-slider {
visibility: hidden}

With card-mod you can hide every part of data. First you must conifgure card-mod (https://github.com/thomasloven/lovelace-card-mod). I hide PM2,5 and slider with this code:

As with my previous post, this works fine in normal mode, but not in compact mode which is what I am trying to run.
WIth compact mode set to true, the card-mod hiding does not work - the AQI remains.

With card-mod you can hide every part of data. First you must conifgure card-mod (https://github.com/thomasloven/lovelace-card-mod). I hide PM2,5 and slider with this code:

As with my previous post, this works fine in normal mode, but not in compact mode which is what I am trying to run. WIth compact mode set to true, the card-mod hiding does not work - the AQI remains.

In compact mode also works.

ha-card > div.preview > div.controls > div.current-aqi {
visibility: hidden}

Yes that works to hide AQI for me (thanks!!) but does not remove the dead space unfortunately. Don't suppose that is something card_mod can handle?

card_mod:
style: |
ha-card > div.preview > div.controls > div > div {
visibility: hidden}

I use it like this to minimize the gap
card_mod:
  style: |
    ha-card > div.preview > div.controls {
    visibility: hidden; height: 0px}

I use it like this to minimize the gap
card_mod:
  style: |
    ha-card > div.preview > div.controls {
    visibility: hidden; height: 0px}

This is the solution for me - thanks so much!


Nightly?

I believe it's possible to use stable rust together with riscv and riscv-rt crates. With the latest Rust release everything should work just fine.

Hello, the issue I have with stable is that it doesn't like me using features, such as inline assembly or panic info messages.

For assembly you can use binary trick, which is widely used in the rust-embedded org. For already "implemented" instructions like wfi you can use riscv::asm::wfi(). With the latest Rust release it's even possible to inline such "pre-compiled" instructions with ThinLTO.

I appreciate the help! I'm trying to make this more of a language learning experience, but eventually I will switch over to Rust's libraries and make heavy use of core and alloc. For now, I would like the "raw" experience. But, I would invite your help for the second half where we start cleaning up the code and force as much as possible into Rust.
Thanks again.


Generic map

Now that type parameters are coming to Go, it might be an optimization opportunity to reduce usages of interface{} with generics.

Indeed it would be a good enhancement. #29 might be needed to get rid of interface{} for keys, but otherwise, the migration should be straightforward. If there is any demand for generics support, I'm going to implement it. Contributions are also welcome.


Align error-constructor-expr with spec

Purpose
Fixes #26341
Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Shall we change the base to https://github.com/ballerina-platform/ballerina-lang/tree/lang-sl-alpha?

Shall we change the base to https://github.com/ballerina-platform/ballerina-lang/tree/lang-sl-alpha?


Crashing

My game is crashing when i join to the server if the script is ensured! Fix pls

u re funny ^^ ... just an advice, NOBODY will fix anything if u dont give more informations. artifacts version ? framework ? esx? which version ? any logs... or screens (at best both) maybe any conflicts with other scripts ? in short: WHAT ARE U USING?! Issue #73 is not an issue until u give informations, somebody can work with. Regards

Bro dont be so angry, im using latest esx and artifacts, it shouldnt conflict with any script

Same issue for me, same error latest artifact on esx


Wrong file generated

If I run:
java -jar openvalidation.jar --culture en --language java
--rule "your age HAS to be greater than 22" --schema "{ age: 21 }" --output age_check.js
The file age_check.java is generated, therefore the output directive is not precisely respected, which is confusing to me

Yes, Federico, once again you're right. This parameter is not entirely clear.
The parameter --output(-o) actually defines the output directory. However, there is the parameter --single-file (-f) over it you can get all the generated code, which actually consists of 2 files: Implementation(Rule Set) and HUMLFramework in one file. If the -f switch is set, then you can specify the name of this single file with -o. I know that it is not really cleanly implemented at this point. And we should definitely change it.
https://docs.openvalidation.io/openvalidation-cli#single-file
https://docs.openvalidation.io/openvalidation-cli#output
My suggestion is:

That we use the --output (-o) parameter to enter only the output directory. (it is not allowed to give file names with extension. we can validate it.)
the parameter --single-file (-f) should no longer be a switch(boolean), but a string parameter to specify the output name of the single file. If you do not specify anything, the default name (HUMLValidator.xxx) will be used.

what do you think about this?

it makes sense to me


Use SENSU_API_URL env variable to define the API endpoint location

I think it could be useful to provide only an environment variable over a JSON file to specify the API url.

Can you update your PR to resolve the conflicts? Then I'll merge it, thanks!

@analytically, i've rebased my PR on sensu/master


Security Administrator is misleading

Right below the description for the security administrator role there is a table showing "In" and "Can do" columns. In one example it says:
"In: " Azure Security Center
"Can do": Can view security policies, view security states, edit security policies, view alerts and recommendations, dismiss alerts and recommendations
However, this does not seem to be accurate as long as the user doesn't have the appropriate RBAC role on a subscription or management group level. Neither can I see security alerts, nor recommendations, nor can I view or edit the security policies.
I tested this on 2 completely separate tenants.
I might be misinterpreting this table but to me it seems confusing as the permissions mentioned for PIM work flawlessly.

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

ID: 0767aac4-fcd9-eaac-2c30-8f17d8eadd92
Version Independent ID: a380a828-03f9-e037-3a62-e4165fda7cb6
Content: Administrator role descriptions and permissions - Azure Active Directory
Content Source: articles/active-directory/users-groups-roles/directory-assign-admin-roles.md
Service: active-directory
Sub-service: users-groups-roles
GitHub Login: @curtand
Microsoft Alias: curtand

@Oechiih Thank you for your feedback! We will review and provide an update as appropriate.

@Oechiih Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

Hi @Oechiih
I don't know the answer to this, so I have to investigate. The roles PM is out of the office on paternity leave, so I put this in my work item backlog and will continue to pursue. ETA is still TBD. Thanks for the update!
https://identitydivision.visualstudio.com/Technical Content/_workitems/edit/810551

Hi @Oechiih
Haven't forgotten  this.

hi @Oechiih
You might be confusing Azure RBAC roles that grant permissions on Azure subscriptions and resources with Azure AD administrative roles, which grant permissions on Azure Active Directory resources such as users, groups, and so on. He says “Security Administrator does not have access to Azure subscriptions. This is correct and by design. Roles in Azure AD do not grant access in Azure.”
Thanks for your input!
@SaurabhSharma-MSFT
#please-close

Hi @curtand
As it has been a while since I opened this issue I don't remember exactly what I tested. But I'm not confusing AAD roles and RBAC roles. My point is the documentation on the Security Administrator AAD Role is misleading as (IIRC) it doesn't enable you to do the actions mentioned in Azure Security Center.
Also I think the fact that there is a 'Security Administrator' AAD role and a 'Security Admin' RBAC role as well as a 'Security Reader' role in both is even more misleading, but that should probably be a separate issue. I had to explain the difference between those many times.
Thank you for your efforts
Cheers


Incorrect validation for Bytes32 cast

There is an incorrect validation for Bytes32 and Bytes32 arrays. The following link addresses the issue: https://github.com/JaimeSilver/hedera-sdk-js/tree/Bytes32-Cast

Can you elaborate on what you mean by "incorrect validation". I looked your last commit in the link your referenced and all you did was remove error handling. I'm wondering why you did that. addBytes32() and its array counterpart addBytes32Array() expect the input to be exactly 32 bytes long because the Solidity type being encoded is bytes32, not bytes${param.length}.

Hello Daniel,
In Solidiy (and Remix) a Bytes32 is submitted in Hex, so at the point of validation the string is not 32 in length, but 66 (64 plus the 2 of the 0x prefix). We were using Composer with Pablo and the validation was stopping the conversion from our input variable.
Example: “TESTNET-OA-TEST002” 0x544553544e45542d4f412d544553543030320000000000000000000000000000
I suspect the 8 arrays you have now might work since every variable is managed as byte string. But that is the next thing to check. Pablo is testing that repo and we will let you know if it solved the problem.
Jaime Plata
Co-founder, All Over Chain
<EMAIL_ADDRESS><EMAIL_ADDRESS>WhatsApp/Mobile : +1<PHONE_NUMBER>
This email may contain material that is confidential, privileged and/or attorney work product for the sole use of the intended recipient.  Any review, reliance or distribution by others or forwarding without express permission is strictly prohibited.  If you are not the intended recipient, please contact the sender and delete all copies.
From: Daniel Akhterov<EMAIL_ADDRESS>Sent: Friday, May 22, 2020 8:43 PM
To: hashgraph/hedera-sdk-js<EMAIL_ADDRESS>Cc: AOChain<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [hashgraph/hedera-sdk-js] Incorrect validation for Bytes32 cast (#248)
Can you elaborate on what you mean by "incorrect validation". I looked your last commit in the link your referenced and all you did was remove error handling. I'm wondering why you did that. addBytes32() and its array counterpart addBytes32Array() expect the input to be exactly 32 bytes long because the Solidity type being encoded is bytes32, not bytes${param.length}.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/hashgraph/hedera-sdk-js/issues/248#issuecomment-632956694 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKRIKUZRQZZOP3L5YE7XHN3RS4L2FANCNFSM4NIFMKKA .

@JaimeSilver The parameter to addBytes32 is not a string, but instead an Uint8Array meaning you should decode your string before you pass it to the method.

Alright then it should happen in Composer, I guess.

On May 22, 2020, at 9:27 PM, Daniel Akhterov<EMAIL_ADDRESS>wrote:
@JaimeSilver The parameter to addBytes32 is not a string, but instead an Uint8Array meaning you should decode your string before you pass it to the method.


Add telemetry to install command

This add's new style snowplow telemetry to meltano install.

If I try to install a plugin that doesnt exist meltano install extractor tap-foo-bar-baz in my project I get the log Installing 0 plugins... and started/completed events + struct event. Is this expected? It seems like its not considered a failure, which is fine but wanted to double check.

Correct, thats not a failure. The command didn't do anything, but it didn't do anything successfully 😁

If I try to install an extractor thats mis-configured i.e. pip_url is missing I get Extractor 'tap-something-invalid' is not known to Meltano and no events are fired at all. Would we expect a started/aborted set here?

There's probably just a exception happening somewhere I don't know about let me poke around. We should be able to fire a started/failed pair for those.

@pnadolny13 found it.

@pnadolny13 found it.

@pandemicsyn I'm still not seeing any events come through when I try to meltano install extractor tap-something-invalid for:
plugins:
  extractors:
  - name: tap-something-invalid
    variant: meltanolabs
    pip_url: git+https://github.com/foo/tap-something-invalid.git

I acknowledge that this is a weird edge case, so I'm totally fine if we merge without fixing this.

@pnadolny13 im gonna go ahead and merge but I'll pop an issue to follow up tomorrow if I have time. fwiw - testing it locally with your example yaml - i do get two events including 1 aborted:
https://gist.github.com/pandemicsyn/df635c72e707b95c2c8034ec10192a62


Flash message translations

Hi everyone, I post that message do discuss about the better way to translate flash messages. In my case, I use symfony/demo as the place were better practices are grouped in a project.

I  wonder if it can be interesting to call the t() function in the Controller to have the extractor ?
The second case allow to auto-extract the messages in XLIFF, but add complexity in the Controller.
Can it be a good idea to use a cusom domain ?
Is it better to use reusable keyword ? or like actually localized keywords ?

These are questions I am asking myself right now to implement tramslation in a project. I know that questions are not totally related with the demo, it's more best practices.

Id' say that (2) we should not use a custom domain (the complication is not worth it) and that (3) we should use keywords as recommended in Symfony best practices (see https://symfony.com/doc/current/best_practices.html#use-keys-for-translations-instead-of-content-strings).
About (1) I'm not sure what should we do. Let's read other opinions. Thanks!


Scope flows bug #309

Fixes the issue #309 where redirecting stdin to scope flows throws an error.

The problem is that terminal size cannot be detected when using redirected std input. In this PR, instead of throwing an error when we cannot get terminal size, we will now assume a default terminal size of 160, greater than the threshold for displaying all fields.
@coccyx we also grab terminal size in cmd/events, cmd/metrics and cmd/dash. do you want me to make this change in those files also? i.e. are we likely to pipe something into scope events, scope metrics or scope dash?

Makes sense to make the same change in all the commands. They could also have their output piped.


feat:Support regex on the Sidecar::IstioEgressListener::hosts

Signed-off-by: cobb-tx<EMAIL_ADDRESS>Please provide a description of this PR:
Fixes https://github.com/istio/istio/issues/40767
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ x ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Hi @cobb-tx. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Documentation (README.md) error

The README.md file contains two references to "next_release_version" as the ENV variable name.  That did not work!  However in the code I found "release_tag" which does!
P.S. Thanks for this "action" it is just what I needed!

Oh, thanks for pointing this out. Since this was for my side project, I didnt update the Readme after some releases. Will fix the documentation, thanks.


NS2B3 fitness view issues

[ ] fix correct setup for NS2B3 - currently DMS sequence and protein sequence don't match
[ ] install hard check that ensures DMS and protein sequence are aligned

a PR for this can also be used to enable a fitness view for all targets, even if they don't have fitness data. In those cases the whole surface would just show up as blue.


Allow access to Swagger UI endpoint for AAD-enabled in-proc functions

Describe the issue
Prior to securing our function with AAD, we were able to access the Swagger UI endpoint at /api/swagger/ui. Since securing the function with AAD, the swagger endpoint now also requires authentication. We want to allow access to this endpoint.
To Reproduce
Steps to reproduce the behavior:

Enable AAD for the Function App using the Authentication blade
Visit the Swagger UI endpoint at https://{function_app}/api/swagger/ui
The page returns a 401 unauthorized.

Expected behavior
I'd like to understand how to intercept the routing for the OpenAPI extenstion so I can override this authorization behavior and allow anonymous access to the swagger UI. I've seen examples of similar middleware behavior using OpenApiHttpTriggerAuthorization but have not been able to get this anonymous behavior to work.
I believe this would be a common need for many functions and so would like to see an example of how to go about it in the documentation. If I've missed it, I'd appreciate a pointer to it.

After some more research I discovered that I can exclude certain paths from authentication in the globalValidation section of the authSettingsV2 page for the Function App in the resource manager. The section initially looks like this:
"globalValidation": {
"requireAuthentication": true,
"unauthenticatedClientAction": "Return401"
},
Which forces all paths to be authenticated. Changing it like below opens up access to the swagger endpoints:
"globalValidation": {
"requireAuthentication": true,
"unauthenticatedClientAction": "Return401",
"excludedPaths": [
"/api/swagger/ui",
"/api/swagger.json"
]
},
I hope this helps anyone who might run into this issue with a Function App in the future.

Closing this issue as it is not really an OpenAPI extension concern as I first thought. My solution to the issue is in the comments.


Fix restarting causing the locationWidth and locationHeight to reset to their defaults.

Since moving the pygame-related code into the Apex class, this is no longer happening.


weathermap disabled - Uncaught Error: Class "index" not found

Hello,
I'm running the following:

Ubuntu 22.04.1 LTS (jammy)
PHP 8.1.2 (apt stock)
NGINX 1.18 (apt stock)
Cacti 1.3.0(git clone -> dev eaefda08)

I installed weathermap from git clone (https://github.com/Cacti/plugin_weathermap.git), changed permissions, and I enabled/install the plugin. Once I start trying to add a map Weathermap crashes and the cacti.log shows:
2023-01-17 14:40:07 - ERROR PHP WARNING in  Plugin 'weathermap': Cannot modify header information - headers already sent by (output started at /var/www/html/cacti/lib/functions.php:1543) in file: /var/www/html/cacti/plugins/weathermap/lib/datasources/index.php  on line: 42
2023-01-17 14:40:07 - CMDPHP PHP ERROR WARNING Backtrace:  (/poller.php[863]:api_plugin_hook(), /lib/plugins.php[89]:api_plugin_run_plugin_hook(), /lib/plugins.php[237]:weathermap_poller_bottom(), /plugins/weathermap/setup.php[1111]:weathermap_run_maps(), /plugins/weathermap/lib/poller-common.php[360]:WeatherMap->__construct(), /plugins/weathermap/lib/WeatherMap.class.php[372]:WeatherMap->Reset(), /plugins/weathermap/lib/WeatherMap.class.php[529]:WeatherMap->LoadPlugins(), /plugins/weathermap/lib/WeatherMap.class.php[822]:include_once(), /plugins/weathermap/lib/datasources/index.php[42]:header(), CactiErrorHandler())
2023-01-17 14:40:07 - ERROR PHP ERROR in  Plugin 'weathermap': Uncaught Error: Class "index" not found in /var/www/html/cacti/plugins/weathermap/lib/WeatherMap.class.php:841 Stack trace: #0 /var/www/html/cacti/plugins/weathermap/lib/WeatherMap.class.php(529): WeatherMap->LoadPlugins() #1 /var/www/html/cacti/plugins/weathermap/lib/WeatherMap.class.php(372): WeatherMap->Reset() #2 /var/www/html/cacti/plugins/weathermap/lib/poller-common.php(360): WeatherMap->__construct() #3 /var/www/html/cacti/plugins/weathermap/setup.php(1111): weathermap_run_maps() #4 /var/www/html/cacti/lib/plugins.php(237): weathermap_poller_bottom() #5 /var/www/html/cacti/lib/plugins.php(89): api_plugin_run_plugin_hook() #6 /var/www/html/cacti/poller.php(863): api_plugin_hook() #7 {main} thrown in file: /var/www/html/cacti/plugins/weathermap/lib/WeatherMap.class.php on line: 841
2023-01-17 14:40:07 - CMDPHP PHP ERROR Backtrace:  (CactiShutdownHandler())
2023-01-17 14:40:07 - CMDPHP ERRORS DETECTED - DISABLING PLUGIN 'weathermap'
What else I can provide to help troubleshoot this?
Thank you!
eric

Did you install this over a pre-existing install of Weathermap?  If so, move the lib folder out of the way, and copy the lib folder fresh from the cacti repo version.

Hello,
This was a fresh install of Cacti and weathermap.

.

Okay, now I know why you are experiencing this as well as others fixing now.  Helps reading the tickets on a PC and not one your phone.  Old age and no glasses ;)

Hello again,
I applied the patch and it worked without any errors or reverting to disabled plugin.
Thank you for your help on this!
eric

Updated the formatting in the original post, the solution looks good.


upgrade grafana dependencies

Upgrade the grafana related dependencies so that dependabot alerts can be resolved.
fixes #185

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Fixed several issues and added a couple of additions

Dog Command

I fixed multiple typos I made before. Oops!

Windows UI -> ScrollingFrame

:GenerateList() supports RichText. NOTE: RichText can be activated with parameters RichText = true

Fixed FreeCam and UnFreeCam command

Transferred the free cam GUI from Roblox Studio and implemented it to Adonis
Supported KeyCode F to toggle free cam (This doesn't replace the old hotkey Shift+P. It's a secondary and way faster to toggle)
Prevented the command to stack Freecam ScreenGuis when found an existing freecam in the player's PlayerGui
Players using free cam no longer have their character anchored. The anchoring wasn't quite necessary and useless, especially that players stopping their free cam would have an issue of it happening.
Removed the unnecessary wait when destroying the freecam gui.

Added ToggleFreeCam command

ToggleFreeCam command toggles players' freecam.

Fixed Rocket command

This command kept exploding targeted players' characters after respawning to a new one. It has been patched to prevent new characters from being exploded while the old characters was targeted.

Added a few songs to Variables.MusicList

Rickroll, Crab Rave (Oof Version), Alone, and Deathbed

P.S. I should've created this pr last week. 😄

Added more changelogs to the Freecam commands.

Added more changelogs to the Freecam commands.

If you're attempting to run the FreeCam command on Roblox studio, it may ignore because there's a Freecam ScreenGui added on your PlayerGui on studio. Studio players have this GUI loaded once they run the game,

If you're attempting to run the FreeCam command on Roblox studio, it may ignore because there's a Freecam ScreenGui added on your PlayerGui on studio. Studio players have this GUI loaded once they run the game,


Click handler for search in navigation header

It looks like at some point GitHub added a click area in the blank part of the header, right in the middle, that triggers the search opening.
It is pretty annoying to me but I wanted to open a discussion first to see if anyone else is bothered by it before doing more.

GIF or Video of what you are referring to?

There you go
https://user-images.githubusercontent.com/75278/122133772-a3b7f100-ce3d-11eb-9e90-3c63558fcaff.mp4

This does not happen to me in chrome on windows

Same on Safari, nothing happens when clicking outside

That is weird, I can repro without any browser extensions even on Chrome 92.0.4515.51. I will test with more browsers.

I managed to repro consistently in a fresh Chrome 91 profile (having login to github)
by rightclicking on the middle of the blank area of the header, then immediately leftclicking
(the mouse has to be in a specific vertical height, otherwise it doesn't occur)
and finally pressing Esc, and repeat:

Please report this to GitHub


How to use Class C

Hello, I'm trying to use the Class C mode
Is there something special to do other than calling the AT+CLASS=2 command ?
I don't see anything on the serial lien sending data from the network to the device ...
Do you have any sample code ?
Paul

hum ... apparently the problem is upstream ... no radio emission from the gateway and it respond with an error message to the chirpstack submission ... so i'll back later with my question

The network server needs to know that it can enqueue the downlink to the device immediately and also which gateway can reach the device. Make sure the network server knows the device is class C capable. Also, the device should have recently sent a Join, a LinkCheck, or an uplink message so that the network server can  learn which gateway can send downlinks to the device.
If you switch the device to class C without any prior uplink communication, the network server may not know where to enqueue downlinks to the device.


Newsletter-70: translate into Chinese

I really do not know what happend so that the make production can not link to the page.

May be the en and es version is OK.

So should I just pull request the linking page Bitcoin Optech Schnorr Taproot Workshop first? Or what should I do to fix this?

@bitschmidty Thank you for your efforts and I learned what is hugo conversion.
But it seemed still did not slove the problem yet?

@bitschmidty
It seemed like something almost there since it shows the link to en version has promblem on the zh file.

@bittomhan can you rebase on master which has the changes from https://github.com/bitcoinops/bitcoinops.github.io/pull/1902 regarding these pages?

@bittomhan can you rebase on master which has the changes from #1902 regarding these pages?

It works! Thank you very much!


Are simultaneous speedtest sessions possible?

I have set up a cron job to run speedtest-cli against a few servers of interest, including a mini server I set up in my Amazon Web Services virtual private cloud.  I have found that if I fork a process for each speedtest session the output that I get seems broken, i.e., I don't get clean, independent output for each process.  So, I'm running the speedtest-cli calls in sequence with no process forking, which seems to work.  I was wondering if I am doing something wrong in trying to fork three or four sub-processes to run speedtest-cli simultaneously with each process utilizing a different server.

If you run multiple speedtest processes simultaneously, you are obviously going to run into problems, because each process is trying to consume the most bandwidth possible.
I do this a lot in testing, and generally configure cron to run in offsets


iOS 14.2 native player does not show captions

Hello everyone
On iOS 14.2 (iPhone specifically), native player (full screen mode) does not show captions/subtitles.
I am using SubtitlesOctopus with the latest video.js
Standard subtitles .vtt of video.js, work well in full screen mode, but needed .ass format.
Anyone have some idea how to fix it?
versions:
videojs
Latest
browsers
Safari
OSes
iOS 14.2

this. project. dead. butterbiscut.

@Alexander-Kane Am I understanding correctly that this is only when the native HTML-video player is used, and the player gets full-screened? If so, then this is expected behaviour and not a bug; it is impossible to display subtitles over a native HTML-cideo when fullscreened, see https://github.com/libass/JavascriptSubtitlesOctopus/issues/1#issuecomment-271834400 .

Lacking a response from OP, I'll assume the guess from previous post was right and close the issue.


feat: v1 support diagnostic packages

Proposed Change

potential implementation of diagnostic packages (v1 agent)

Checklist

[ ] Changes are tested
[ ] CI has passed

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Create PWSTR and PSTR from literals

Right now literals need to be copied into new buffers every time one is passed into a function that expects a PWSTR. Instead we should provide some type of macro convenience for creating PSTR and PWSTR from literals. These would encode the literals properly and add a trailing \0 at the end all at compile time. Bonus points if we can make macro that works for either kind of string pointer.

Do we need to use a macro? It would be nice if we could (also) write a const function to do this so you can write something like this:
let s = PSTR::from("hello");

It would somehow have to distinguish between a string literal and a regular string reference. I'm not sure if that's possible.

Any function that takes a &str, &[u8], &[u16] etc with the goal of null-terminating it or transcoding it (ie. UTF-8 to UTF-16) would have to use another buffer to store the result, because the value can't be modified in-place. To get around that, the idea here is to make a new literal string that gets stored in the program's static storage at compile-time.
Functions cannot distinguish literals in their input (although you can get close with &'static str), and furthermore, they cannot create new literals (even if they're const). Literals can only be created by using literal syntax, so they have to be written in the code, either directly or by a macro.
For example, to convert a literal b"abcd" to one that is null-terminated, we can write a macro to rewrite the literal, syntactically, to b"abcd\0". That's the only way to do so at compile-time.

Yep, I'd recommend using a crate like widestring. I use widestring::U16CString which automatically converts strings to UTF-16 and  null-terminates them.

The null terminator is no doubt part of it but actually it looks like the trait constraints of the way some functions are now defined don't allow for null pointers when the API expects them in some cases, which is where I was actually stuck (the compiler gave an ambiguous message that made me think I was using the wrong type at first.) Namely this:
https://docs.microsoft.com/en-us/windows/win32/api/adshlp/nf-adshlp-adsopenobject
Where in the example they pass a null pointer to lpzusername and lpzpassword. But in the windows-rs crate it MUST be PWSTR per the type constraints:
https://microsoft.github.io/windows-docs-rs/doc/bindings/Windows/Win32/ActiveDirectory/fn.ADsOpenObject.html
So my previous way of simply passing a ptr::null_mut() doesn't work with the newest version of windows-rs when I want passive authentication. Is there another way of doing that?
Sorry may be off topic, I can create a separate issue if required.

Here's an example (the last parameter is effectively a null pointer value).
https://github.com/microsoft/windows-rs/blob/87f32117f613e3a1666e3dc7a379ba596789a4dd/examples/event/src/main.rs#L10

Ah I figured that was more analogous to an empty string than a null pointer, but at first glance it seems to work, thanks!

Might be late to the party, but I got around it using my own impl:
impl SystemServices::PWSTR {
    fn from(text: &str) -> Self {
        Self(text.encode_utf16().chain(::std::iter::once(0)).collect::<Vec<u16>>().as_mut_ptr())
    }
}

Then used it like so:
let class_name = SystemServices::PWSTR::from("Hello world?");

Might be late to the party, but I got around it using my own impl:
impl SystemServices::PWSTR {
    fn from(text: &'static str) -> Self {
        Self(text.encode_utf16().chain(::std::iter::once(0)).collect::<Vec<u16>>().as_mut_ptr())
    }
}

That creates a Vec<u16> and get a raw pointer .as_mut_ptr() to the vec's internal data.  The Vec will be freed and that pointer will not be valid.

Playing with an implementation of this here: https://github.com/microsoft/windows-rs/compare/utf16?expand=1

#1891 introduced string literal macros that address this issue.


TypeError: Float cannot represent non numeric value: Invalid Date

I'm getting this error and I have no idea why. I've been upgrading my graphql packages to keep up with the ecosystem and now getting this error on code that used to work.
Relevant package.json:
    "graphql": "0.10.5",
    "graphql-date": "^1.0.3",
    "graphql-loader": "^1.2.1",
    "graphql-log": "^0.1.3",
    "graphql-redis-subscriptions": "^1.2.0",
    "graphql-server-express": "^1.0.4",
    "graphql-subscriptions": "^0.4.4",
    "graphql-tools": "^1.0.0",
    "graphql-type-json": "^0.1.4",

cool I will check thes out

can you send me a stack trace atleast or some repro code?

This is the stack trace my server is logging to console:
W20170726-11:40:38.496(-4)? (STDERR) TypeError: Float cannot represent non numeric value: Invalid Date
W20170726-11:40:38.497(-4)? (STDERR)     at coerceFloat (/Users/jcursi/Sites/joncursi/redbird-web/node_modules/graphql/type/scalars.js:73:9)
W20170726-11:40:38.497(-4)? (STDERR)     at GraphQLScalarType.serialize (/Users/jcursi/Sites/joncursi/redbird-web/node_modules/graphql/type/definition.js:210:12)
W20170726-11:40:38.497(-4)? (STDERR)     at completeLeafValue (/Users/jcursi/Sites/joncursi/redbird-web/node_modules/graphql/execution/execute.js:668:37)
W20170726-11:40:38.498(-4)? (STDERR)     at completeValue (/Users/jcursi/Sites/joncursi/redbird-web/node_modules/graphql/execution/execute.js:617:12)
W20170726-11:40:38.498(-4)? (STDERR)     at completeValue (/Users/jcursi/Sites/joncursi/redbird-web/node_modules/graphql/execution/execute.js:597:21)
W20170726-11:40:38.499(-4)? (STDERR)     at completeValueWithLocatedError (/Users/jcursi/Sites/joncursi/redbird-web/node_modules/graphql/execution/execute.js:546:21)
W20170726-11:40:38.499(-4)? (STDERR)     at completeValueCatchingError (/Users/jcursi/Sites/joncursi/redbird-web/node_modules/graphql/execution/execute.js:515:12)
W20170726-11:40:38.500(-4)? (STDERR)     at resolveField (/Users/jcursi/Sites/joncursi/redbird-web/node_modules/graphql/execution/execute.js:467:10)
W20170726-11:40:38.500(-4)? (STDERR)     at /Users/jcursi/Sites/joncursi/redbird-web/node_modules/graphql/execution/execute.js:314:18
W20170726-11:40:38.506(-4)? (STDERR)     at Array.reduce (native)

This appears to be an issue with the Meteor framework: https://github.com/orionsoft/meteor-apollo-accounts/issues/70


API's document link in Devportal opens a blank page

Description
Documents link in Devportal displays a blank page in the latest U2 level.
Steps to Reproduce

Update to wso2am-<IP_ADDRESS> (or latest as of now <IP_ADDRESS>).
Navigate to the publisher and create an API or select an existing API.
Navigate to the 'Documents' section and add a document.
Save changes and navigate to the Devportal.
Try to view the uploaded document.
A blank page will appear.

Affected Component
APIM
Version
wso2am-4.1.0
Environment Details (with versions)
No response
Relevant Log Output
No response
Related Issues
No response
Suggested Labels
No response

Fixed with https://github.com/chanaka3d/apim-apps-1/commit/620bc9a953fd59f5ef37ed8254955ad1902ebeb4

No public fix is required as the issue was with U2 system.


CSRF validation blocks django clients

CSRF validation needs to be disabled by default for djnago_view.
This can be done using the @csrf_exempt view function decorator
from django.views.decorators.csrf import csrf_exempt
...
    @csrf_exempt
    def django_view(self, request):
...

Solved in version 0.1.7


fix: time not implemented on wasm

Motivation and Context
Related to https://github.com/awslabs/smithy-rs/issues/2087 and https://github.com/awslabs/aws-sdk-rust/discussions/679#discussioncomment-4357224.
Whenever targeting a platform that the std::time::SystemTime::now() or std::time::Instant::now() is not supported, it will compile successfully but panic during runtime with the following message:

time not implemented on this platform

Description
In order to prevent that from happening, we will use the time library with the wasm-bindgen feature flag enabled. The crate aws-smithy-types will have that feature enabled by default and we will consume that function to get the current time for the system.
Testing
Unit tests already in place
Tested in the browser based on example here: https://github.com/awsdocs/aws-doc-sdk-examples/pull/4325
Checklist

[ ] I have updated CHANGELOG.next.toml if I made changes to the smithy-rs codegen or runtime crates
[ ] I have updated CHANGELOG.next.toml if I made changes to the AWS SDK, generated SDK code, or SDK runtime crates

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I think we will probably want to figure out a way to do this without pulling in the wasm-bindgen — js-sys is a fairly meaty crate that I don't think we want to pull in for all users in all cases.
My guess is we'll want to refactor https://github.com/awslabs/smithy-rs/blob/main/aws/rust-runtime/aws-credential-types/src/time_source.rs into a dyn-safe trait like we have to AsyncSleep, pull it into aws-smithy-types, then make a WASM compatible implementation.
Probably worth an RFC to get everyone aligned on the design

I agree with you with pulling js-sys to all consumers is not a good idea (that was my poor implementation because we should be able to set that conditionally based on target). There is a big difference a see between AsyncSleep (as you can see it had to be reimplemented for the browser here), because this error can simply break you application during runtime silently and it can happen even when a dependency uses that function from std library.
I don't think we want to every time we need to fetch the current time to always go through the TimeSource (that was passed during instantiation or default), do you? I think this will somewhat become a burden while developing it. In the ideal scenario, we would want the standard library itself to not throw this error. This might happen when there is a standard way of dealing with of the host environment in a WebAssembly context (possibly WebAssembly component model will solve this).

@rcoh , I convert the PR back to draft. Let me know if you still think it is worth going on the route of this PR, otherwise I will park this for now.

I had a discussion with @rcoh this morning about this. We want to take the following approach:

Add a TimeSource trait to aws-smithy-types
Make TimeSource configurable the exact same way as AsyncSleep/sleep_impl everywhere in smithy-rs
Refactor all places that consume time to use the TimeSource instead
Have a default TimeSource implementation that takes the time from Tokio's Instant to make testing easier since we will be able to use Tokio's time travel testing capabilities. This default would be wired up the same way as the default async sleep implementation.

You're welcome to implement this if you want.
On the fallibility of AsyncSleep, that browser use case you provided is interesting. Do you know why Delay is fallible in wasm-timer? From what I could tell from the source code, the timer can disappear, but when would that happen?

I had a discussion with @rcoh this morning about this. We want to take the following approach:

Add a TimeSource trait to aws-smithy-types
Make TimeSource configurable the exact same way as AsyncSleep/sleep_impl everywhere in smithy-rs
Refactor all places that consume time to use the TimeSource instead
Have a default TimeSource implementation that takes the time from Tokio's Instant to make testing easier since we will be able to use Tokio's time travel testing capabilities. This default would be wired up the same way as the default async sleep implementation.

You're welcome to implement this if you want, and I can provide more details and answer questions if you're interested.

Based on this comment from the wasm-timer maintainer, the approach you guys are proposing seems a good idea. Unfortunately, I will not be able to tackle this anytime soon.


Added Page For Archived User

Issue

[ ] #575

What is the change?
Created archived card for archived user
Is Development Tested?

[ ] Yes
[ ] No

video
https://github.com/Real-Dev-Squad/website-www/assets/111434418/70b70fd4-815b-4860-9829-8d0035259d9e
In the middle of the video I have changed archived to false

Hello @vinit717 where you handle this case: when user is archieved and when we go to /join route then it will redirect to /archived route and when user is not archieved and when we go to /archived then it redirect to /join route

Because currently that approach is not take cape of every edge case


CLI with --lr_scheduler does not work if optimizer is configured via configure_optimizers

Bug description
Per the example code snippet in the docs, you should be able to configure an optimizer in configure_optimizers and then specify a LR scheduler via cli argument --lr_scheduler.
The code snippet is
# main.py
import torch
from lightning.pytorch.cli import LightningCLI
from lightning.pytorch.demos.boring_classes import DemoModel, BoringDataModule

class LitLRScheduler(torch.optim.lr_scheduler.CosineAnnealingLR):
    def step(self):
        print("⚡", "using LitLRScheduler", "⚡")
        super().step()

cli = LightningCLI(DemoModel, BoringDataModule)

DemoModel returns an optimizer via configure_optimizers.
The custom LitLRScheduler does not get run via python main.py fit --lr_scheduler LitLRScheduler --lr_scheduler.T_max 100  --trainer.accelerator cpu as implied by the docs. ("⚡", "using LitLRScheduler", "⚡" is never printed).
Note I had to include --lr_scheduler.T_max because T_max is a required parameter of CosineAnnealingLR, so the example wouldn't even run without error out of the box.
What version are you seeing the problem on?
v2.0
How to reproduce the bug
No response
Error messages and logs
# Error messages and logs here please

Environment

Current environment
#- Lightning Component (e.g. Trainer, LightningModule, LightningApp, LightningWork, LightningFlow): LightningModule, CLI
#- PyTorch Lightning Version (e.g., 1.5.0): commit dea1ff6 -- I have to pin this version because I need the behavior of bf16-mixed w/ fsdp which changed after this commit, and bf16-true is not available yet for fsdp.
#- Lightning App Version (e.g., 0.5.2):
#- PyTorch Version (e.g., 2.0): 2.0.1
#- Python version (e.g., 3.9): 3.10.10
#- OS (e.g., Linux): linux
#- CUDA/cuDNN version:
#- GPU models and configuration:
#- How you installed Lightning(`conda`, `pip`, source): pip
#- Running environment of LightningApp (e.g. local, cloud):

More info
No response
cc @carmocca @mauvilsa

We recently updated the docs about this (https://github.com/Lightning-AI/lightning/pull/17552). It requires that you also specify the --optimizer

In that case, passing an optimizer or optimizer and scheduler will override the model's configure_optimizers

@lendle Were you able to make it work?


Layers that modify transforms

Currently, push_layer() doesn't modify the transforms of drawing operations in that layer, but there are a couple places in my project where that would be useful. I could append() the fragment that I'd like to transform, but that involves a copy and it may be very large, so I'd rather avoid that if possible.
I think there's two obvious options to change the API to allow this. Either, add an Option<Transform> parameter to push_layer() that gets applied to everything within that layer, or add a separate function called something like push_transform_layer() that only effects transforms. In the second case it might make sense to rename push_layer() to push_clip_layer(), but then you'd have two different types of layers to pop, which could be confusing and error prone.
If this is something the project is willing to include, I'm happy to implement whatever API gets settled on! The scene encoding doesn't seem too hard to work with... (knock on wood)

The main difficulty here is that currently transforms are just inserted as-is, and this would require a second book-keeping layer of transform stacks, which would only be used for this API. Would you consider creating your own version of this for external use. e.g.
pub struct TransformedScene {
    scene: &mut Scene,
    transform: Affine
}
impl TransformedScene {
    pub fn fill(...){self.scene.fill(..., self.transform.then(transform))}
}

I suppose that would work, but I'm exposing the scene to client code in a GUI framework, so it would be much cleaner to have it built-in rather than creating a wrapper.
The two situations where it would be useful for me are 1) Scaling an entire scene to transparently account for HiDPI, and 2) Transforming the contents of UI nodes to the origin, so client code doesn't have to account for their final laid-out position while painting.
I imagine these two use cases are pretty useful for other GUI frameworks as well, so I would argue that it's worth an extra stack of transforms.
Again, I'm happy to implement it!

At least in the second case, wouldn't you want the ability for scenes to be retained for a widget if it didn't need to be repainted?
That is the reason that masonry uses append for this case, which already handles this properly.
And then you can also apply the HiDPI support at the same time (although note that you probably want some of the code to be aware of the scale factor anyway, to e.g. avoid bugs like in this article).
But if you think it being handled in your paint equivalent is needed, the way to do that would is to have it handled externally. I don't see how it's any less clean to use a wrapper struct - the external API you expose would stay the same.
The implementation work isn't the problem, I just don't see that it fits with how Vello currently works.
Maybe I'm misunderstanding though. Do you have parts of our extant example code which could be written more clearly using this capability?

If a widget wants to cache its drawing commands, there's no reason it can't store a Scene internally and append it to the main scene when it's time to draw. But for widgets that are highly dynamic, writing commands to an intermediate scene and then immediately appending to the main scene is wasteful when they could just write those commands to the main scene directly. So, in my framework I've chosen to leave that choice up to individual widgets.
The majority of my frame time is spent building the scene, which is why this added bit of flexibility is so important to me.
I would much prefer to re-export the actual Scene type from Vello, rather than a thin wrapper with an almost identical API. It would be an extra thing to keep in sync with Vello, it could be confusing to users, but also: a wrapper type wouldn't have any compatibility with the larger ecosystem of crates built on top of Vello. So that's what I mean when I say it would be less clean.
I really don't think this would be a significant departure from the API as it stands. Right now, the docs for push_layer() even emphasize that "transforms are not saved or modified by the layer stack" which I think is evidence that some users may already expect it to work this way - I know I did at first. All I'm asking for is the option.

I'm not likely to approve your proposed design, but I'm not going to block it either.
Ultimately, the cleanness of your API isn't our responsibility - from my perspective this is much less clean, as it is adding extra-bookkeeping for every operation on a Scene, to slightly improve the ergonomics of a very niche case.
Feel free to make a PR implementing this to continue the discussion, and a different maintainer can approve if they think the ergonomics gain for your use case is worth the unavoidable encoding performance cost.

+1 to Daniel’s comments.
In addition to the extra bookkeeping, this request makes the assumption that a layer and transform group are the same which is not universally true. The color font spec, for example, does not merge these concepts and implementing that with this proposed change would be exceedingly painful.
For this reason and others, I’m strongly opposed to making this change. However, we’ve had several people ask for some form of state tracking API and it would be reasonable to add such a thing to vello. This could mimic the original piet RenderContext or the web Canvas. The semantics should be well defined and considerations should be made for conversion to/from the current scene type.
If you’re interested in working on this, I’d recommend opening a new issue or starting a zulip thread for design discussion before submitting a PR.

Ok, so you're saying that one way forward would be to create an alternate API for constructing an Encoding? I imagine it would be built on top of Scene to reduce code duplication... Are there other discussions about that you could point me to?
How would either of you feel about exposing an &mut Encoding from Scene? I know that's probably a separate discussion, but that would also give me the flexibility to change the scale of a scene without copying the whole thing.
Sorry if at any point I came across as snarky, that wasn't my intention! I really appreciate the work you're doing. :)


Present custom edit menu on long press

https://github.com/goatboyzz/swift-toolkit/assets/15877903/fa6262d3-0fd7-43d4-a800-f63cf89e913d
Was not able to customize the edit menu for text edit but was able to customize the edit menu for long press.
To reproduce, you can highlight a text and long press on it
Reference: https://stackoverflow.com/questions/73712955/using-uieditmenuinteraction-with-uitextview

We cannot access the highlighted text with this method, so closing


[PR][CLOSED] Feature: V8, Support a build facility for Yocto-based SMP platform - @open sesame 07/18 17:27

 Issue by geunsik-lim
Thursday Jul 12, 2018 at 10:37 GMT
Originally opened as https://github.sec.samsung.net/STAR/TAOS-CI/pull/125

Feature: V8, Support a build facility for Yocto-based SMP platform

Resolves: #106

This commit is to add a SMP build facility to support SMP platform as well as
Ubuntu and Tizen. To build source code for SMP platform, We compile a github
source code with devtool command.
After this PR, TAOS-CI supports triple OSs such as Tizen, Ubuntu, and Yocto as a portable
automated CI system solution.
devtool of OpenEmbedded(OE):

https://www.yoctoproject.org/software-item/devtool/

Changes proposed in this PR:

Version 8:

Added variable setting via configuration file for build mode
a. Tizen (packaging/.spec): If a maintainer done the gbs based build process, you may change builde mode from 99 to 0.
b. Ubuntu (debian/.rule)  : If a maintainer done the pdebuild based build process, you may change builde mode from 99 to 0.
c. Yocto (CMakeLists.txt)  : If a maintainer done the devtool based build process, you may change builde mode from 99 to 0.

Build modes: BUILD_MODE_TIZEN=99, BUILD_MODE_UBUNTU=99, and BUILD_MODE_YOCTO= 99

Version 7:

Added language set to avoid the execution error of "devtool add/build" command.
Archived log files for debugging three steps of devtool (e.g., reset -> add -> build)
Fixed pontential issues that can be generated due to introduction of this commit
Added annotation why the BUILD_MOD=99 statement always has to be set by default

Version 6:

Added debug messages to handle unexpected operations.
Replaced SMP/smp with YOCTO/yocto to open TAOS-CI into github.com (as Apache license)
Declared all build modules (e.g., tizen, ubuntu, and yocto) with BUILD_MODE=99 by default
Unified cibot_rest_api.sh and inspect_dependency.sh for maintenance

Version 5:

Fixed incorrect existing if statement after adding -${input_commit} style.
Activated three functions to run SMP build process really.

pr-audit-build-smp-trigger-queue()
pr-audit-build-smp-trigger-run()
pr-audit-build-smp()

Version 4:

Fixed typos
Updated annotation for maintenance
replaced $result with $build_result for readability

Version 3:

Enabled BUILD_MODE=99 in case of SMP because sTArOS SMP edition is not still enabled.

Version 2:

Added SMP build module in plugins-base folder
Added annotation limitation(@todo) on un-completed isolation environment
Fixed typos

Version 1:

Added configuration setting for SMP package build
Updated build-tizen and build-ubuntu facility for BUILD_MODE=99

Self assessment:
we have evaluated this PR by creating the below repository that can be built with cmake.
cmake is the official build process in Yocto/SMP.

https://github.sec.samsung.net/geunsik-lim/smp-hello-world

Signed-off-by: Geunsik Lim<EMAIL_ADDRESS>
geunsik-lim included the following code: https://github.sec.samsung.net/STAR/TAOS-CI/pull/125/commits

:octocat: cibot: Thank you for posting issue #149. The person in charge will reply soon.

 Comment by git-bot-sec
Thursday Jul 12, 2018 at 10:37 GMT

Thank you for submitting PR #125. Your PR has to pass all verificiation processes of cibot before getting a review process from reviewers. We recommend that you read instruction manual in . If you want to monitor the progress status of your PR in detail, visit http://aaci.mooo.com/.

 Comment by geunsik-lim
Friday Jul 13, 2018 at 01:44 GMT

The aging test is started to avoid unexpected regression.

 Comment by git-bot-sec
Wednesday Jul 18, 2018 at 04:48 GMT

:octocat: cibot: @geunsik-lim, Oooops. A SMP builder checker could not be completed. To get a hint, please go to http://aaci.mooo.com/TAOS-CI/ci/repo-workers/pr-audit/201807181347550.75168299674988-125-f8e362ecef37f2913b3bca0916d47de9396246fd/.

 Comment by git-bot-sec
Wednesday Jul 18, 2018 at 05:30 GMT

:octocat: cibot: @geunsik-lim, Oooops. A SMP builder checker could not be completed. To get a hint, please go to http://aaci.mooo.com/TAOS-CI/ci/repo-workers/pr-audit/201807181430440.31754398345947-125-4eb9a639a4e9c13ce3ec21c9ff0dfd5c2b96c0ce/.

 Comment by geunsik-lim
Wednesday Jul 18, 2018 at 06:15 GMT

Started the integration test and aging test.

 Comment by geunsik-lim
Wednesday Jul 18, 2018 at 06:15 GMT

Changes proposed in this PR:

Version 6:

Added debug messages to handle unexpected operations.
Replaced SMP/smp with YOCTO/yocto to open TAOS-CI into github.com (as Apache license)

Version 5:

Fixed incorrect existing if statement after adding -${input_commit} style.
Activated three functions to run SMP build process really.

pr-audit-build-smp-trigger-queue()
pr-audit-build-smp-trigger-run()
pr-audit-build-smp()

Version 4:

Fixed typos
Updated annotation for maintenance
replaced $result with $build_result for readability

Version 3:

Enabled BUILD_MODE=99 in case of SMP because sTArOS SMP edition is not still enabled.

Version 2:

Added SMP build module in plugins-base folder
Added annotation limitation(@todo) on un-completed isolation environment
Fixed typos

Version 1:

Added configuration setting for SMP package build
Updated build-tizen and build-ubuntu facility for BUILD_MODE=99

 Comment by git-bot-sec
Wednesday Jul 18, 2018 at 06:42 GMT

:octocat: cibot: @geunsik-lim, One of the format checkers is failed. If you want to get a hint to fix this issue, please go to https://github.sec.samsung.net/STAR/TAOS-CI/wiki/.

 Comment by git-bot-sec
Wednesday Jul 18, 2018 at 07:36 GMT

:octocat: cibot: @geunsik-lim, Oooops. A YOCTO builder checker could not be completed. To get a hint, please go to http://aaci.mooo.com/TAOS-CI/ci/repo-workers/pr-audit/201807181636100.35979008674622-125-97a884e32b87001a3195780282ac358f8001f1a7/.

 Comment by git-bot-sec
Wednesday Jul 18, 2018 at 07:52 GMT

:octocat: cibot: @geunsik-lim, Oooops. A YOCTO builder checker could not be completed. To get a hint, please go to http://aaci.mooo.com/TAOS-CI/ci/repo-workers/pr-audit/201807181652520.81073594093323-125-97a884e32b87001a3195780282ac358f8001f1a7/.

 Comment by git-bot-sec
Wednesday Jul 18, 2018 at 07:55 GMT

:octocat: cibot: @geunsik-lim, Oooops. A YOCTO builder checker could not be completed. To get a hint, please go to http://aaci.mooo.com/TAOS-CI/ci/repo-workers/pr-audit/201807181655040.8360059261322-125-97a884e32b87001a3195780282ac358f8001f1a7/.

 Comment by git-bot-sec
Wednesday Jul 18, 2018 at 08:06 GMT

:octocat: cibot: @geunsik-lim, Oooops. A YOCTO builder checker could not be completed. To get a hint, please go to http://aaci.mooo.com/TAOS-CI/ci/repo-workers/pr-audit/201807181705560.67504906654358-125-97a884e32b87001a3195780282ac358f8001f1a7/.

 Comment by git-bot-sec
Wednesday Jul 18, 2018 at 08:18 GMT

:octocat: cibot: @geunsik-lim, Oooops. A YOCTO builder checker could not be completed. To get a hint, please go to http://aaci.mooo.com/TAOS-CI/ci/repo-workers/pr-audit/201807181718380.87053203582764-125-97a884e32b87001a3195780282ac358f8001f1a7/.

 Comment by git-bot-sec
Wednesday Jul 18, 2018 at 08:24 GMT

:octocat: cibot: @geunsik-lim, Oooops. A YOCTO builder checker could not be completed. To get a hint, please go to http://aaci.mooo.com/TAOS-CI/ci/repo-workers/pr-audit/201807181724150.75446891784668-125-97a884e32b87001a3195780282ac358f8001f1a7/.

 Comment by git-bot-sec
Wednesday Jul 18, 2018 at 08:26 GMT

:octocat: cibot: @geunsik-lim, Oooops. A YOCTO builder checker could not be completed. To get a hint, please go to http://aaci.mooo.com/TAOS-CI/ci/repo-workers/pr-audit/201807181725420.28200888633728-125-97a884e32b87001a3195780282ac358f8001f1a7/.

 Comment by git-bot-sec
Wednesday Jul 18, 2018 at 08:28 GMT

:octocat: cibot: @geunsik-lim, Oooops. A YOCTO builder checker could not be completed. To get a hint, please go to http://aaci.mooo.com/TAOS-CI/ci/repo-workers/pr-audit/201807181727380.96935296058655-125-97a884e32b87001a3195780282ac358f8001f1a7/.

 Comment by geunsik-lim
Wednesday Jul 18, 2018 at 09:04 GMT

Changes proposed in this PR:

Version 7:

Added language set to avoid the execution error of "devtool add/build" command.

Version 6:

Added debug messages to handle unexpected operations.
Replaced SMP/smp with YOCTO/yocto to open TAOS-CI into github.com (as Apache license)
Declared all build modules (e.g., tizen, ubuntu, and yocto) with BUILD_MODE=99 by default
Unified cibot_rest_api.sh and inspect_dependency.sh for maintenance

Version 5:

Fixed incorrect existing if statement after adding -${input_commit} style.
Activated three functions to run SMP build process really.

pr-audit-build-smp-trigger-queue()
pr-audit-build-smp-trigger-run()
pr-audit-build-smp()

Version 4:

Fixed typos
Updated annotation for maintenance
replaced $result with $build_result for readability

Version 3:

Enabled BUILD_MODE=99 in case of SMP because sTArOS SMP edition is not still enabled.

Version 2:

Added SMP build module in plugins-base folder
Added annotation limitation(@todo) on un-completed isolation environment
Fixed typos

Version 1:

Added configuration setting for SMP package build
Updated build-tizen and build-ubuntu facility for BUILD_MODE=99

 Comment by geunsik-lim
Wednesday Jul 18, 2018 at 09:05 GMT

The aging test is done. This PR is ready to be merged.

 Comment by geunsik-lim
Wednesday Jul 18, 2018 at 23:51 GMT

We have updated the existing changelog as following:
Changes proposed in this PR:

Version 7:

Added language set to avoid the execution error of "devtool add/build" command.
Archived log files for debugging three steps of devtool (e.g., reset -> add -> build)
Fixed pontential issues that can be generated due to introduction of this commit
Added annotation why the BUILD_MOD=99 statement always has to be set by default

Version 6:

Added debug messages to handle unexpected operations.
Replaced SMP/smp with YOCTO/yocto to open TAOS-CI into github.com (as Apache license)
Declared all build modules (e.g., tizen, ubuntu, and yocto) with BUILD_MODE=99 by default
Unified cibot_rest_api.sh and inspect_dependency.sh for maintenance

Version 5:

Fixed incorrect existing if statement after adding -${input_commit} style.
Activated three functions to run SMP build process really.

pr-audit-build-smp-trigger-queue()
pr-audit-build-smp-trigger-run()
pr-audit-build-smp()

Version 4:

Fixed typos
Updated annotation for maintenance
replaced $result with $build_result for readability

Version 3:

Enabled BUILD_MODE=99 in case of SMP because sTArOS SMP edition is not still enabled.

Version 2:

Added SMP build module in plugins-base folder
Added annotation limitation(@todo) on un-completed isolation environment
Fixed typos

Version 1:

Added configuration setting for SMP package build
Updated build-tizen and build-ubuntu facility for BUILD_MODE=99

 Comment by geunsik-lim
Wednesday Jul 18, 2018 at 23:53 GMT

To Reviewers,
The current status is still "Do-Not-Merge".
Currently, We have been doing the aging test with https://github.sec.samsung.net/geunsik-lim/smp-hello-world/pull/2 as a final step before merging this PR to avoid unexpected regressions.

 Comment by geunsik-lim
Thursday Jul 19, 2018 at 02:36 GMT

The configurable setting of build mode is added in Version 8.
(the comment from @myungjoo-ham)

Version 8:

1. Added variable setting via configuration file for build mode
   a. Tizen (packaging/*.spec): If a maintainer done the gbs based build process, you may change builde mode from 99 to 0.
   b. Ubuntu (debian/*.rule)  : If a maintainer done the pdebuild based build process, you may change builde mode from 99 to 0.
   c. Yocto (CMakeLists.txt)  : If a maintainer done the devtool based build process, you may change builde mode from 99 to 0.
   - Build modes: BUILD_MODE_TIZEN=99, BUILD_MODE_UBUNTU=99, and BUILD_MODE_YOCTO= 99


Validate uses of AbstractString and string indices

I noticed that at least a few of your functions that take AbstractString can't actually deal with AbstractString, and/or does not handle non-ASCII strings, for example:
julia> parse(Identifier, Test.GenericString("a"))
ERROR: ArgumentError: regex matching is only available for the String type; use String(s) to convert

julia> DataToolkitBase.stringdist("ÆaÆb", "ÆaÆb")
ERROR: StringIndexError: invalid index [2], valid nearby indices [1]=>'Æ', [3]=>'a'

In general, I've found that

You rarely need to index into strings. If you do, use nextind, prevind etc.
Any time you need a pointer to a string (e.g. when matching a Regex to it), you need it to be Union{String, SubString{String}}
If you test your functions with Test.GenericString containing non-ASCII characters, you catch most of these bugs.

Hi Jakob, thanks for taking a look and making this issue. This project is beginning to move onto the testing/documenting/tweaking phase, so your comments are particularly appreciated. I'll see what I can do to make those functions better behaved.
If you might have any other comments, please do share them. I'd really like this project to be as well-constructed as I can make it, and so far I'm the only one who's really looked into the code, so I'm very keen on feedback.

I really just noticed it when I copy-pasted your stringdist function to test in the context of a comment of yours. I don't use your package, just wanted to let you know

Yea, I'm pretty sure at this stage I'm basically the only user of this package :stuck_out_tongue:, I really want to avoid locking in bad design decisions though, which is why I'm desperate keen to get more eyes on the code.
Just mentioning this in case anything might come of it, but I understand this is a drive-by issue :slightly_smiling_face:

The utils.jl string functions should be multi-codepoint-safe now.


[Enhancement] Support automatic partition in FE part 3

Signed-off-by: Astralidea<EMAIL_ADDRESS>What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #11444
Problem Summary(Required) ：

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

[FE PR Coverage Check]
:disappointed: fail : 32 / 66 (48.48%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/OlapTable.java
0
1
00.00%
[1710]

:large_blue_circle:
com/starrocks/sql/analyzer/AnalyzerUtils.java
7
15
46.67%
[644, 645, 646, 647, 659, 660, 661, 664]

:large_blue_circle:
com/starrocks/service/FrontendServiceImpl.java
25
50
50.00%
[1582, 1588, 1596, 1602, 1608, 1618, 1619, 1620, 1624, 1625, 1626, 1633, 1634, 1635, 1639, 1640, 1641, 1645, 1646, 1647, 1654, 1655, 1656, 1660, 1682]

run starrocks_admit_test


Items in queue wrong .

When i add items to queue two items , but i call previous or next two times player can changed item.

i mean to previous function wrong

Fixed on master.
Thanks for reporting this.

thx for support :)

Send a new release?

Done! 1.1.0 is up

There is still a problem, previous function is wrong, hasPrevious is wrong...

Yes, I can confirm, the issue is "open" again.

I met same problem when it comes to 2 items in queue, is there a work around?

when it comes to one item in queue, hasPrevious is wrong

Reopened the issue. I'm happy to review a PR if anyone is down to contribute otherwise I'll try to fix it soon.

    var hasPreviousItem: Bool {
        if !queue.isEmpty &&
            (nextPosition - 1 > 0 || mode.contains(.repeat) || mode.contains(.repeatAll)) {
            return true
        }
        return false
    }

maybe should this.


2MB limit

Thanks for your tools!  you did a great job! And I would like to ask if it is possible to break the 2MB limit?
Another strange thing is that the game has a limit on the number of songs (500 in the latest version, the game will only play songs with [uniquId] between 0-499), (in earlier versions this number was 200), I don't know how to try to break it, it's unlikely that the file in the datatable directory determines this number. I think it might be in the main file which is in the exefs directory
Have you do any research in this? I'm not very good at decompiling, I tried to change this number limit but failed ....T_T

Hello! Conceptually speaking, increasing the datatable filesize limit seems very possible.
There are two static byte array[0x200000] in the .bss section of the ELF executable (which is where statically allocated variables are stored) that are used for temporarily decrypting/decompressing the datatable file content into before being parsed.
I have the strong intuition that it should be possible to increase the size of this section, update the code to point to the (now enlarged) end part and update the corresponding array sizes to match (which are loaded as constants directly into registers).
Theoretically this would allow increasing the size to another larger (but still fixed size) limit without too much work.
Problem is, I'm not actually sure how one would pull this off in practice. I'm assuming the section sizes are defined somewhere in the ELF header and increasing the allocated size would be as simple as editing it there but... shrug, I usually deal with windows EXEs
A more robust solution would of course be to dynamically allocate the right amount of memory in place and roll with that.
That would require a bit of ARM assembly but otherwise shouldn't be too bad.
Except another problem with this is that there isn't really any space to actually inject new code into.
One workaround I can think of (and that I have done before somewhere else, although for x86) is to abuse the fact that functions are always aligned to 16 byte boundaries which leaves the padding bytes inbetween as free executable space.
So one way to inject new code would be to insert a JMP (or rather a "branch" in this case) from the original function into the padding of another function, write parts of the new code there then jump directly to the next padding and so on until eventually returning to the original function.
Doing it this way would be incredibly tedious but at least theoretically possible (and of course would also have to be repeated for every game region/update).
Another important thing to note also is that an easy way to reduce the json file size would be to remove all whitespace (minifying it) and (I haven't tried this) but also try to remove all of the fields that are either 0s or empty strings "" since presumably (I would hope) those would be default initialized in case they are omitted making them redundant.
I don't know what the current status on the ID limit for the switch release is (I believe at least for some of the other taiko games it has been somewhat solved)
but in case you don't already know, there's a taiko modding discord server so feel free to also try your luck there if you'd like.
I don't really do too much modding myself directly and mostly only work on the tooling side of things

Thank you very much for your reply, according to your suggestion, I deleted some unnecessary information and empty strings, and successfully compressed the json to less than 2MB
About the limitation of the number of songs, I am going to go to the discord channel to know the current status, thank you for your suggestion and wish you all the best~