fix: lf_inherited_tags null value

Description
I'm getting the following error when inherited tags is not set
pydantic.error_wrappers.ValidationError: 1 validation error for LfTagsConfig
inherited_tags
  none is not an allowed value (type=type_error.none.not_allowed)

Checklist

[x] You followed contributing section
[x] You kept your Pull Request small and focused on a single feature or bug fix.
[x] You added unit testing when necessary
[x] You added functional testing when necessary

@svdimchenko seems that the PR introduces lf_inherited_tags as part of lf_tags_config, is that correct? if so this is a breacking config change that we should communicate in the next release.

@svdimchenko seems that the PR introduces lf_inherited_tags as part of lf_tags_config, is that correct? if so this is a breacking config change that we should communicate in the next release.

to be honest I thought it would be as it's represented here
class LfTagsConfig(BaseModel):
    enabled: bool = False
    tags: Optional[Dict[str, str]] = None
    tags_columns: Optional[Dict[str, Dict[str, List[str]]]] = None
    inherited_tags: List[str] = []
    inherited_tags: Optional[List[str]] = None

and yes, we should mention that in release notes

In the last 1.6.4 release, this
lf_inherited_tags: domain, dataTier
lf_tags_config:
  enabled: true
  tags:
   	securityClassification: confidential

worked for me, also considering that we had a config.get('lf_inherited_tags)'... I like what you proposed better, because lf_inherited_tags is part of lf config.

@svdimchenko could you update the readme too? Specifically here, we mention this:
models:
  my_project:
    example:
      +lf_inherited_tags: [inherited-tag-1, inherited-tag-2]

but should be
models:
  my_project:
    example:
      +lf_tags_config:
          enabled: true
          lf_inherited_tags: [inherited-tag-1, inherited-tag-2]

@svdimchenko could you update the readme too? Specifically here, we mention this:
models:
  my_project:
    example:
      +lf_inherited_tags: [inherited-tag-1, inherited-tag-2]

but should be
models:
  my_project:
    example:
      +lf_tags_config:
          enabled: true
          lf_inherited_tags: [inherited-tag-1, inherited-tag-2]

sure, I've updated our readme file

@dacreify could you please take a look here as well ?

@nicor88 @svdimchenko Apologies just catching up here. I initially had lf_inherited_tags nested inside lf_tags_config but I ran into an issue: There was no way to set lf_inherited_tags at the project level and also apply specific tags to individual models. i.e. It didn't seem like lf_tags_config values from the project level were merged with ones at the model level. Is that not the case? I'm trying to avoid having to redundantly set lf_inherited_tags on every model I want to tag.

@dacreify I don't know how the setup looks like for you, but can't something like this?
models:
  my_project:
     models_to_tag:
       +lf_tags_config:
          enabled: true
          lf_inherited_tags: [inherited-tag-1, inherited-tag-2]

works for you? e.g. you put group of models with the same lf_inherited_tags under one folder?

@dacreify I don't know how the setup looks like for you, but can't something like this?
models:
  my_project:
     models_to_tag:
       +lf_tags_config:
          enabled: true
          lf_inherited_tags: [inherited-tag-1, inherited-tag-2]

works for you? e.g. you put group of models with the same lf_inherited_tags under one folder?

I use same setup and everything works OK. Are there any edge cases missed here @dacreify ?

@nicor88 @svdimchenko
Here's the scenario we're trying to accomplish:
dbt_project.yml
models:
  my_project:
     models_to_tag:
       +lf_tags_config:
          enabled: true
          lf_inherited_tags: [inherited-tag-1, inherited-tag-2]

models/models_to_tag/models_to_tag.yml
models:
  - name: my_model
    config:
      lf_tags_config:
        enabled: true
        tags:
          my_tag: my_value

Here the model-level lf_tags_config clobbers the project-level one and my lf_inherited_tags is lost. This is why we hoisted it to be a peer with lf_tags_config.
Per the DBT documentation:
https://docs.getdbt.com/reference/resource-configs/plus-prefix

Note: This use of the + prefix, in dbt_project.yml, is distinct from the use of + to control config merge behavior (clobber vs. add) in other config settings (specific resource .yml and .sql files). Currently, the only config which supports + for controlling config merge behavior is grants.

Looks like there's syntax to support merging instead of clobbering but it only applies to grants currently. :/

@nicor88 @svdimchenko Any thoughts here?☝ With potentially hundreds of models to tag repeating the lf_inherited_tags block for every single one could be onerous.

@dacreify I understand the reason why you want to have lf_inherited_tags outside the lf_tags_config, but again, can't you group your models with the same lf_tags_config together in order to use dbt project configs?
Also, in my current setup, lf_tags_config is setup by model basis, because we want to overwride only specific tag value for specific columns, specifically to default tags in case of PII data.

@nicor88 Your second point is exactly why grouping the models and applying things at the project level doesn't help: We tag many models at the column level and will need to repeat our lf_inherited_tags on each model under the current implementation.
It's too bad DBT hasn't generalized the model-level usage of + on grants to merge other kinds of configs like this. I just ran my project under 1.7.0 to confirm the clobbering behavior and got Failed to remove LF tags as expected.
Thanks for the discussion here! If we have to duplicate lf_inherited_tags on every model we want to tag at the column level at least it will fail loudly if someone forgets to do so. I understand DRY isn't always the highest principal. 🙂

@nicor88 Your second point is exactly why grouping the models and applying things at the project level doesn't help: We tag many models at the column level and will need to repeat our lf_inherited_tags on each model under the current implementation.
It's too bad DBT hasn't generalized the model-level usage of + on grants to merge other kinds of configs like this. I just ran my project under 1.7.0 to confirm the clobbering behavior and got Failed to remove LF tags as expected.
Thanks for the discussion here! If we have to duplicate lf_inherited_tags on every model we want to tag at the column level at least it will fail loudly if someone forgets to do so. I understand DRY isn't always the highest principal. 🙂

hi there @dacreify ! Sorry for the delay from my side. Just the previous implementation broke the pipelines at least for 2 community members including my setup, my bad I did not test all edge cases on the review stage, that's why needed to fix that rapidly.
WDYT if you add default_ lf_inherited_tags parameter to profile config and set it as default value for all models. And if the config for the model is not emply you can join lists of inherited tags. Will such approach work for you ?

@svdimchenko Apologies for missing a spot and causing that regression! Next time I'll do more manual testing on my end.
@nicor88 It sounds like what you're describing is going back to the 1.6.4 behavior and accounting for the None properly? I can make a draft PR to look at if so.
We are running fine on 1.6.4 for now.
Thanks!

@svdimchenko Apologies for missing a spot and causing that regression! Next time I'll do more manual testing on my end.
@nicor88 It sounds like what you're describing is going back to the 1.6.4 behavior and accounting for the None properly? I can make a draft PR to look at if so.
We are running fine on 1.6.4 for now.
Thanks!

Sure, we can add extra config like default_lf_inherited_tags for the model level and set it on top of dbt_project.yml file. In this case we should parse this config in all materializations as well.
On the other hand, we can have the default_lf_inherited_tags set in profile and pass it to all models as extra list. No need to parse it in materialisations, as lf_tags_config is already parsed there. So all we need is just to get default_lf_inherited_tags from credentials and pass the correct value to LfTagsManager class. Just seems more elegant to me.

Thanks @svdimchenko - what you proposed sounds good for me - you are right, having in profiles and referring the default_lf_inherited_tags from there can be really helpful and clean, but we can take both approaches - as we do for s3_data_dir for example. We can try to see if credentials.default_lf_inherited_tags is not none or we fallback to dbt_project or model config.
@dacreify sure give it a shot. - issue https://github.com/dbt-athena/dbt-athena/issues/518


[Event Request]Table 352 "Default Dimension" - OnUpdateCustomerTemplGlobalDimCodeOnCaseElse

Request a new event in procedure UpdateCustomerTemplGlobalDimCode in table 352 "Default Dimension":
    local procedure UpdateCustomerTemplGlobalDimCode(GlobalDimCodeNo: Integer; CustomerTemplCode: Code[20]; NewDimValue: Code[20])
    var
        CustomerTempl: Record "Customer Templ.";
    begin
        if CustomerTempl.Get(CustomerTemplCode) then begin
            case GlobalDimCodeNo of
                1:
                    CustomerTempl."Global Dimension 1 Code" := NewDimValue;
                2:
                    CustomerTempl."Global Dimension 2 Code" := NewDimValue;
                //<NEW>
                else
                    OnUpdateCustomerTemplGlobalDimCodeOnCaseElse(GlobalDimCodeNo, CustomerTemplCode, NewDimValue);
                //</NEW>
            end;
            CustomerTempl.Modify(true);
        end;
    end;

    //<NEW>
    [IntegrationEvent(false, false)]
    local procedure OnUpdateCustomerTemplGlobalDimCodeOnCaseElse(GlobalDimCodeNo: Integer; CustomerTemplCode: Code[20]; NewDimValue: Code[20])
    begin
    end;

Realized later that this procedure is only called for Global Dimensions 1 and 2.  By using the DimMgt.DefaultDimOn... calls, this request is unnecessary.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Issue 56

This refers to #56
There are cases when a Storable object is not in a valid JSON format. This checks that the format is valid before serializing it to JSON.

Thanks!


enhance hello-sidebar example

After our 4.0 release and the new flexibility allowed for sidebars to become panes and to re-orient themselves, we should update our sidebar example such that it can:

become both a pane and a sidebar
re-orient itself into both horizontal and vertical orientations

Review by end-user was successful.


Update main.js

Hey there! It seems you've submitted a pull request to include keyword suggestions for the Minecraft target selectors! Thanks a bunch for such an awesome idea!


[Android] Can't run project in flutter 3.19.0

Issue Summary
Launching lib/main.dart on sdk gphone64 arm64 in debug mode...
Building with Flutter multidex support enabled.
ERROR:~/.gradle/caches/transforms-3/00d9d634312ff5516499237d011ad1ac/transformed/core-1.14.0-alpha01-runtime.jar: D8: java.lang.NullPointerException

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:mergeExtDexDebug'.
> Could not resolve all files for configuration ':app:debugRuntimeClasspath'.
   > Failed to transform core-1.14.0-alpha01.aar (androidx.core:core:1.14.0-alpha01) to match attributes {artifactType=android-dex, asm-transformed-variant=NONE, dexing-enable-desugaring=true, dexing-enable-jacoco-instrumentation=false, dexing-is-debuggable=true, dexing-min-sdk=26, org.gradle.category=library, org.gradle.dependency.bundling=external, org.gradle.libraryelements=aar, org.gradle.status=release, org.gradle.usage=java-runtime}.
      > Execution failed for DexingNoClasspathTransform: ~/.gradle/caches/transforms-3/00d9d634312ff5516499237d011ad1ac/transformed/core-1.14.0-alpha01-runtime.jar.
         > Error while dexing.
   > Failed to transform core-1.14.0-alpha01.aar (androidx.core:core:1.14.0-alpha01) to match attributes {artifactType=android-dex, asm-transformed-variant=NONE, dexing-enable-desugaring=true, dexing-enable-jacoco-instrumentation=false, dexing-is-debuggable=true, dexing-min-sdk=26, org.gradle.status=release, org.gradle.usage=java-runtime}.
      > Execution failed for DexingNoClasspathTransform: ~/.gradle/caches/transforms-3/00d9d634312ff5516499237d011ad1ac/transformed/core-1.14.0-alpha01-runtime.jar.
         > Error while dexing.
   > Failed to transform core-1.14.0-alpha01.aar (androidx.core:core:1.14.0-alpha01) to match attributes {artifactType=android-dex, asm-transformed-variant=NONE, dexing-enable-desugaring=true, dexing-enable-jacoco-instrumentation=false, dexing-is-debuggable=true, dexing-min-sdk=26, org.gradle.status=release, org.gradle.usage=java-runtime}.
      > Execution failed for DexingNoClasspathTransform: ~/.gradle/caches/transforms-3/00d9d634312ff5516499237d011ad1ac/transformed/core-1.14.0-alpha01-runtime.jar.
         > Error while dexing.
   > Failed to transform core-1.14.0-alpha01.aar (androidx.core:core:1.14.0-alpha01) to match attributes {artifactType=android-dex, asm-transformed-variant=NONE, dexing-enable-desugaring=true, dexing-enable-jacoco-instrumentation=false, dexing-is-debuggable=true, dexing-min-sdk=26, org.gradle.status=release, org.gradle.usage=java-runtime}.
      > Execution failed for DexingNoClasspathTransform: ~/.gradle/caches/transforms-3/00d9d634312ff5516499237d011ad1ac/transformed/core-1.14.0-alpha01-runtime.jar.
         > Error while dexing.
   > Failed to transform core-1.14.0-alpha01.aar (androidx.core:core:1.14.0-alpha01) to match attributes {artifactType=android-dex, asm-transformed-variant=NONE, dexing-enable-desugaring=true, dexing-enable-jacoco-instrumentation=false, dexing-is-debuggable=true, dexing-min-sdk=26, org.gradle.status=release, org.gradle.usage=java-runtime}.
      > Execution failed for DexingNoClasspathTransform: ~/.gradle/caches/transforms-3/00d9d634312ff5516499237d011ad1ac/transformed/core-1.14.0-alpha01-runtime.jar.
         > Error while dexing.
   > Failed to transform core-1.14.0-alpha01.aar (androidx.core:core:1.14.0-alpha01) to match attributes {artifactType=android-dex, asm-transformed-variant=NONE, dexing-enable-desugaring=true, dexing-enable-jacoco-instrumentation=false, dexing-is-debuggable=true, dexing-min-sdk=26, org.gradle.status=release, org.gradle.usage=java-runtime}.
      > Execution failed for DexingNoClasspathTransform: ~/.gradle/caches/transforms-3/00d9d634312ff5516499237d011ad1ac/transformed/core-1.14.0-alpha01-runtime.jar.
         > Error while dexing.
   > Failed to transform core-1.14.0-alpha01.aar (androidx.core:core:1.14.0-alpha01) to match attributes {artifactType=android-dex, asm-transformed-variant=NONE, dexing-enable-desugaring=true, dexing-enable-jacoco-instrumentation=false, dexing-is-debuggable=true, dexing-min-sdk=26, org.gradle.status=release, org.gradle.usage=java-runtime}.
      > Execution failed for DexingNoClasspathTransform: ~/.gradle/caches/transforms-3/00d9d634312ff5516499237d011ad1ac/transformed/core-1.14.0-alpha01-runtime.jar.
         > Error while dexing.
   > Failed to transform core-1.14.0-alpha01.aar (androidx.core:core:1.14.0-alpha01) to match attributes {artifactType=android-dex, asm-transformed-variant=NONE, dexing-enable-desugaring=true, dexing-enable-jacoco-instrumentation=false, dexing-is-debuggable=true, dexing-min-sdk=26, org.gradle.status=release, org.gradle.usage=java-runtime}.
      > Execution failed for DexingNoClasspathTransform: ~/.gradle/caches/transforms-3/00d9d634312ff5516499237d011ad1ac/transformed/core-1.14.0-alpha01-runtime.jar.
         > Error while dexing.
   > Failed to transform core-1.14.0-alpha01.aar (androidx.core:core:1.14.0-alpha01) to match attributes {artifactType=android-dex, asm-transformed-variant=NONE, dexing-enable-desugaring=true, dexing-enable-jacoco-instrumentation=false, dexing-is-debuggable=true, dexing-min-sdk=26, org.gradle.status=release, org.gradle.usage=java-runtime}.
      > Execution failed for DexingNoClasspathTransform: ~/.gradle/caches/transforms-3/00d9d634312ff5516499237d011ad1ac/transformed/core-1.14.0-alpha01-runtime.jar.
         > Error while dexing.
   > Failed to transform core-1.14.0-alpha01.aar (androidx.core:core:1.14.0-alpha01) to match attributes {artifactType=android-dex, asm-transformed-variant=NONE, dexing-enable-desugaring=true, dexing-enable-jacoco-instrumentation=false, dexing-is-debuggable=true, dexing-min-sdk=26, org.gradle.status=release, org.gradle.usage=java-runtime}.
      > Execution failed for DexingNoClasspathTransform: ~/.gradle/caches/transforms-3/00d9d634312ff5516499237d011ad1ac/transformed/core-1.14.0-alpha01-runtime.jar.
         > Error while dexing.
   > Failed to transform core-1.14.0-alpha01.aar (androidx.core:core:1.14.0-alpha01) to match attributes {artifactType=android-dex, asm-transformed-variant=NONE, dexing-enable-desugaring=true, dexing-enable-jacoco-instrumentation=false, dexing-is-debuggable=true, dexing-min-sdk=26, org.gradle.status=release, org.gradle.usage=java-runtime}.
      > Execution failed for DexingNoClasspathTransform: ~/.gradle/caches/transforms-3/00d9d634312ff5516499237d011ad1ac/transformed/core-1.14.0-alpha01-runtime.jar.
         > Error while dexing.

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 6s
Error: Gradle task assembleDebug failed with exit code 1

Exited (1).

Steps to Reproduce

flutter pub add twilio_voice
flutter run lib/main.dart

This error occur only after adding this twilio_voice package to my dependency, if I remove it then I can run the project normally.
I think this happen because of this Breaking Changes.
Hopefully you can update this package and solve this problem.
Thank you

I solve the problem by removing implementation 'androidx.core:core-ktx:+' in android/build.gradle file as below:
dependencies {
    implementation(platform("org.jetbrains.kotlin:kotlin-bom:$kotlin_version"))
    implementation(platform("com.google.firebase:firebase-bom:32.2.2"))
    implementation("com.twilio:voice-android:6.3.2")
    implementation("com.google.firebase:firebase-messaging-ktx:23.2.1")
    implementation 'androidx.appcompat:appcompat:1.6.1'
    implementation 'androidx.lifecycle:lifecycle-process:2.6.1'
    // implementation 'androidx.core:core-ktx:+'
    runtimeOnly("androidx.lifecycle:lifecycle-extensions:2.2.0")
    runtimeOnly("androidx.constraintlayout:constraintlayout:2.1.4")
}

But, I am afraid this approach may affect some functionality in this package

@kkday-kevinhorasia  project is running, but the first run crashed and happened  exception when requesting permission from Twilio

Hi @yazanturk,
About requesting permission from Twilio, in my case it only work for the first time we installed and open the app.
When we close app, clear app, open it again and request the same permission again it will not request it again.
I don't know why this happened, maybe you can open a new issue regarding this problem.
And if you find the solution to the problem please let me know too :)

Thank you for reporting the issue, I'll investigate this issue later the week.

@cybex-dev  Thanks

Hi @yazanturk,
About requesting permission from Twilio, in my case it only work for the first time we installed and open the app. When we close app, clear app, open it again and request the same permission again it will not request it again.
I don't know why this happened, maybe you can open a new issue regarding this problem. And if you find the solution to the problem please let me know too :)

the problem is solved check the package please ?

Hi @yazanturk,
About requesting permission from Twilio, in my case it only work for the first time we installed and open the app. When we close app, clear app, open it again and request the same permission again it will not request it again.
I don't know why this happened, maybe you can open a new issue regarding this problem. And if you find the solution to the problem please let me know too :)

the problem is solved check the package please ?

I think it is not a problem according to this docs. It is actually android permissions default behavior.


Customize SQL integrity constrains

How can i catch and customize spot orm SQL execution integrity violations or any other sql error?

With a try/catch or some kind of custom error handler, depends on what you want to achieve.

+1, you can do something like this:
// untested piece of code
$mapper = $spot->mapper('Entity');
try {
  $mapper->update($entity);
} catch (\Spot\Exception $spotException) {
  if (strpos($spotException->getMessage(), 'constraint violation') != false) {
    $response = new Forbidden;
  }
}

Closing this issue, because of the inactivity. Feel free to reopen this if you have more questions regarding this subject.


TypeError: object of type 'NoneType' has no len()

I executed the following command:
orangepi@orangepi5b:~/rknn_model_zoo/examples/yolov8/python$ python3 yolov8.py --model_path ../model/yolov8s.rknn --target rk3588 --img_show

After running the command, the output showed:
--> Init runtime environment
I RKNN: [18:43:51.326] RKNN Runtime Information: librknnrt version: 1.5.2 (c6b7b351a@2023-08-23T15:28:22)
I RKNN: [18:43:51.326] RKNN Driver Information: version: 0.9.3
I RKNN: [18:43:51.327] RKNN Model Information: version: 6, toolkit version: 1.5.2+b642f30c(compiler version: 1.5.2 (c6b7b351a@2023-08-23T15:34:44)), target: RKNPU v2, target platform: rk3588, framework name: ONNX, framework layout: NCHW, model inference type: static_shape
W RKNN: [18:43:51.375] query RKNN_QUERY_INPUT_DYNAMIC_RANGE error, rknn model is static shape type, please export rknn with dynamic_shapes
W Query dynamic range failed. Ret code: RKNN_ERR_MODEL_INVALID. (If it is a static shape RKNN model, please ignore the above warning message.)
done
Model-../model/yolov8s.rknn is rknn model, starting val
E Catch exception when setting inputs.
E Traceback (most recent call last):
  File "/home/orangepi/.local/lib/python3.10/site-packages/rknnlite/api/rknn_lite.py", line 200, in inference
    self.rknn_runtime.set_inputs(inputs, data_type, data_format, inputs_pass_through=inputs_pass_through)
  File "rknnlite/api/rknn_runtime.py", line 1067, in rknnlite.api.rknn_runtime.RKNNRuntime.set_inputs
Exception: The input[0] need 4dims input, but 3dims input buffer feed.

Traceback (most recent call last):
  File "/home/orangepi/rknn_model_zoo/examples/yolov8/python/yolov8.py", line 263, in <module>
    boxes, classes, scores = post_process(outputs)
  File "/home/orangepi/rknn_model_zoo/examples/yolov8/python/yolov8.py", line 120, in post_process
    pair_per_branch = len(input_data)//defualt_branch
TypeError: object of type 'NoneType' has no len()

Any assistance would be appreciated. Thank you!

https://github.com/airockchip/rknn_model_zoo/issues/105


[4.0] Update and re-enable the diagnostics for using unstable runtime names with NSCoding

Explanation: Foundation's standard serialization mechanism, NSKeyedArchiver, can end up storing mangled names into persistent archives, which isn't really a good idea. We're going to solve this with a run-time check that suggests changing the name, or at least explicitly confirming the current mangled name in use, but we can catch the obvious cases at compile-time as well.
Scope: Affects classes that conform directly to NSCoding that do not have "simple" mangled names.
Radar: rdar://problem/32314195 (and rdar://problem/32414557)
Reviewed by: @DougGregor, @itaiferber
Risk: Medium-low.
Testing: Passed updated compiler regression tests, confirmed that the problematic cases from #9809 will not cause issues now.

@swift-ci Please test

@swift-ci Please test source compatibility


How do you connect to Druid datasource today?

A lot of the documentation found on the net is referencing the old Superset that had a Druid Cluster menu item to configure Druid. I'm trying to connect Druid to the new modernize platform where Druid is not the sole database you try to connect to.
Two questions:

How do I connect to my druid cluster using the new interface?
How do I connect to a druid cluster that has TLS/SSL enabled?

Warm regards.

@MaxOrelus You can either use the sqlalchemy druid dialect and connect via a connection string, or you can enable the DRUID_IS_ACTIVE config flag in your superset config to use the legacy connector.

@nytai using DRUID_IS_ACTIVE did the trick for bringing back the druid cluster menu. Do you have any advice on how to connect to a TLS/SSL enabled druid? The port typically used is usually 8082, but since I'm using TLS it's 8282 and I'm not sure where I should be adding the client certificate, client key, and ca certificate.

are these any help? https://github.com/apache/incubator-superset/issues/7596 and https://github.com/apache/incubator-superset/pull/4480/

so ,How do I connect to my druid cluster using the superset of 1.1.0?
I set DRUID_IS_ACTIVE=True ,but this was not work

Same issue here, how to navigate to Druid datasources in Superset 1.1.0 ?

@wchaofan  and @colax94, have you tried running superset init again after enabling the DRUID_IS_ACTIVE flag? Found that in the Slack channel, it worked for me


#62 cli tool for printing file stats

This is based on the orc-statistics tool from the cpp repository.
I would like also to add tool for exporting data to css, but as a separate tool and PR.

Please run make fmt and make fmt-toml

Please run make fmt and make fmt-toml

done


Dimensions.addEventListener is not a function

Current Behavior
When i'm trying to use react-navigation in my project.
I'm having :

_reactNative.Dimensions.addEventListener is not a function
withOrientation.componentDidMount
with.Orientation.js:37:17

It worked until really recently when i tried to remove my node_modules and install them again.
Your Environment

software
version

react-navigation
"^1.0.0-beta.13",

react-native
"0.42.x"

node
v8.3.0

npm or yarn
4.6.1

Can you ensure you're using the latest release? 1.0.0-beta.15

tbh you are almost 8 versions of react native behind, and probably some of the other libraries/nested dependencies have been updated to be compliant to newer versions.
That said, why do you think it's related to react-navigation? Can you post the complete error log?

Still not working with "react-navigation": "^1.0.0-beta.15"
And for compatibility issue, i'm stuck with react 0.42.x

That said, why do you think it's related to react-navigation? Can you post the complete error log?

Can you please answers to those two questions?
That said, this is a library in beta - as probably many more you have been using in your project - so we cannot help you with such an old version of the main library :(

This is the 2 errors i have in the console of the React Native Debugger
ExceptionsManager.js:63 _reactNative.Dimensions.addEventListener is not a function
   handleException @ ExceptionsManager.js:63
   handleError @ InitializeCore.js:114
   reportFatalError @ error-guard.js:44
   guard @ MessageQueue.js:48
   callFunctionReturnFlushedQueue @ MessageQueue.js:107
   (anonymous) @ debuggerWorker.js:71
Unhandled JS Exception: _reactNative.Dimensions.addEventListener is not a function
   reactConsoleErrorHandler @ ExceptionsManager.js:71
   console.error @ YellowBox.js:62
   logIfNoNativeHook @ RCTLog.js:38
   __callFunction @ MessageQueue.js:242
   (anonymous) @ MessageQueue.js:108
   guard @ MessageQueue.js:46
   callFunctionReturnFlushedQueue @ MessageQueue.js:107
   (anonymous) @ debuggerWorker.js:71

@kelset As i said, It was working before. I've just got this issue since in reinstalled my dependencies

I've just got this issue since in reinstalled my dependencies

Then it's 100% related to some nested dependency of any of the libraries you are using in your project. So I don't understand why it should be react-navigation causing the issue (moreover, as you said, it was working before).
Unluckly in the react-native world you have to keep up with newer versions of the all the libraries you are using because even the main library is still in beta. So tbh you have no choice but to update to newer versions.

I don't know what you are trying to obtain @LouisJS by opening a clone issue (#2830).
Unless:

you can provide a reliable test project that reproduces the issue
you have definitive proof that it's caused by react-navigation - since the error log you posted states that it's caused by react-native

I won't re-open this issue. Again, you are using a really old version of the react-native, you should update.

I'm also having this issue today. My RN version is 0.42.0, node v8.1.2, npm v4.6.1 and react-navigation 1.0.0-beta.11.
I'm using a TabNavigator that when removed the error doesn't show. This can be a react-native issue since it refers to Dimensions but this only occurred when I did a fresh npm install.

Was able to fix this by updating a react-navigation dependency, react-native-tab-view to version 0.0.67.
Hope this helps.

Our team had the issue with react-navigation v1.0.0-beta.15. Downgrading to .11 resolved the issue.

I have the same issue eith both react-navigation v1.0.0-beta.15 and  .11

The causes of this is the addition of withOrientation which helps the navbar work in a more consistent manner (though it will be replaced later). @swl367  @yusisi it seems this issue is related to having a vastly outdated version of React Native. What version are you running on?
If you're on 0.42 or below you'll run into issues because Dimensions.addEventListener wasn't added until 0.43. 0.43 came out in April.

Makes sense. I’m on 0.42.3
On Sat, Oct 28, 2017 at 10:22 AM Spencer Carli<EMAIL_ADDRESS>wrote:

The causes of this is the addition of withOrientation which helps the
navbar work in a more consistent manner (though it will be replaced later).
@swl367 https://github.com/swl367 @yusisi https://github.com/yusisi
it seems this issue is related to having a vastly outdated version of React
Native. What version are you running on?
If you're on 0.42 or below you'll run into issues because
Dimensions.addEventListener wasn't added until 0.43. 0.43 came out in April.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/react-community/react-navigation/issues/2824#issuecomment-340198722,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA_qRqcQtVlyKkWeAjgqsS1D_l-kxvznks5sw0bMgaJpZM4QGSne
.

TypeError: addListener is not a function

This error is located at:
    in CardStack (at CardStackTransitioner.js:66)
    in RCTView (at View.js:71)
    in View (at Transitioner.js:151)
    in Transitioner (at CardStackTransitioner.js:19)
    in CardStackTransitioner (at StackNavigator.js:38)

i face simiar issue too
my packages were:
"react-native": "0.52.0",
"react-navigation": "^1.0.0-beta.29",

I had to downgrade react-navigation to 1.0.0-beta.21 to fix this issue

Same, downgrade to :

"1.0.0-beta.27"

@adnanaliarshad thank you so much ,I've been stuck with this problem these two days.

  "react-native": "0.42.3",
   "react-navigation": "^1.1.2"

downgrade react-navigation to:

1.0.0-beta.13

works fine 😃

I suddenly started having the same error after deleting node_modules and running "yarn install".
TypeError: addListener is not a function
ExceptionsManager.js:65
This error is located at:
    in CardStack (at CardStackTransitioner.js:67)
    in RCTView (at View.js:71)
    in View (at Transitioner.js:142)
    in Transitioner (at CardStackTransitioner.js:19)
    in CardStackTransitioner (at StackNavigator.js:39)
    in Unknown (at createNavigator.js:13)
    in Navigator (at createNavigationContainer.js:226)
    in NavigationContainer (at Root.js:52)
    in AppWithNavigationState (created by Connect(AppWithNavigationState))
    in Connect(AppWithNavigationState) (at Root.js:74)
    in PersistGate (at Root.js:65)
    in Provider (at Root.js:64)
    in Root (at App.js:25)
    in App (at renderApplication.js:35)
    in RCTView (at View.js:71)
    in View (at AppContainer.js:102)
    in RCTView (at View.js:71)
    in View (at AppContainer.js:122)
    in AppContainer (at renderApplication.js:34)
Warning: Can only update a mounted or mounting component. This usually means you called setState, replaceState, or forceUpdate on an unmounted component. This is a no-op.
ExceptionsManager.js:73
Please check the code for the PersistGate component.

Environment:
OS: macOS High Sierra 10.13.3
Node: 8.4.0
Yarn: 1.3.2
npm: 5.6.0
Watchman: 4.9.0
Xcode: Xcode 9.2 Build version 9C40b
Android Studio: Not Found
Packages: (wanted => installed)
react: 16.2.0 => 16.2.0
react-native: 0.53.3 => 0.53.3
"react-navigation": "^1.2.0"
I followed what @javahuang did and the error disappeared.
  render() {
    return (
      <AppNavigator
        navigation={addNavigationHelpers({
          dispatch: this.props.dispatch,
          state: this.props.nav,
          addListener: () => {}                <-- Add this line
        })}
      />
    );
  }


Updated labs 191 and 201 - K8S Version 1.20

Description of changes:
Updated labs 191 and 201. Tested both on versions 1.19 and 1.20
Lab 191 - Encrypting Secrets with KMS:

Fixed typos / rephrased sentences
Updated reasons to use custom key store to match the documentation

Lab 201 - Resource Management:

Fixed typos / rephrased sentences
Reordered pod creation to make it easier to follow
Fixed output rendering error
Added output for Resource Quotas

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

Fixed typos, added pricing information and updated last review date

Added a separate page for the Metric Server. Renamed cleaning.md to cleanup.md


Structured logging

It'd be nice to have some logging (structured, if implemented).
Perhaps using klog

https://kubernetes.io/blog/2020/09/04/kubernetes-1-19-introducing-structured-logs/


List for level-spawns and ratings for difficulty as my interpretation of it is messy

List of levels and their spawn
TutorialLevel1(12, 320, 1) - Hallway - needs more decorations
TutorialLevel2(12, 80, 1) - Dropguide to Solution - needs more decorations
TutorialLevel3(12, 297, 1) - Jump Over the Box - needs more decorations
TutorialLevel4(12, 200, 1) - Mind the Gap - move the text & more decorations
EasyLevel1(12, 292, 1) - Candles - maybe needs more decorations
EasyLevel2(132, 282, 1) - Mouse - I think it's good - would be nice if we had a sfx play if the player collides with the mouse nose
EasyLevel3(12, 12, 1) - Block Maze 5 - moved to another location or had as a failed level concept
EasyLevel4(37, 489, 1) - Letters - I think this is good - might more coloring with white blocks
EasyLevel5(30, 518, 1) - Staggers - - [ ] would be nice if I could add some blue and white and yellow boxes in the level to show so lights or something
EasyLevel6(21, 21, 1) - Spiral - made for testing purposes only. Needs more polish for it to be a playable level for the public.
--------NEEDS MORE TESTING__
EasyLevel7(37, 527, 1) - Burger - needs more dull coloring to avoid confusion
EasyLevel8(21, 258, 1) - Tip-toe - boxes need readjustments and better positioning
EasyLevel9(37, 524, 1) - You're not alone - boxes need readjustments and better positioning, more decorations for people and "platforms"
EasyLevel10(48, 53, 1) - Ritual or (keys) - boxes need readjustments and better positioning, more text, and renamed to Hardlevel1 sonn...
EasyLevel11(32, 338, 1) - Reverse Lamp - I think this one is solid and good
EasyLevel12(37, 47, 1) - Propaganda - This is a hard one and needs more detailed block as well towards the bottom
EasyLevel13(23, 259, 1) - Split Path - this one is eh
EasyLevel14(69, 263, 1) - Uneven platforms - could use more decorations and movements for improvements
EasyLevel15(6, 272, 1) - Lava drip - lvl just needs more testing and adjusting
EasyLevel16(33, 44, 1) - Ashtray - we good here
EasyLevel17(37, 47, 1) - ribbons - we good
EasyLevel18(37, 47, 1) - Choose Bait - better decorations at the bottom
EasyLevel19(26, 62, 1) - Upper Gates - more jumping and decorations needed
EasyLevel20(48, 53, 1) - na
EasyLevel21(32, 338, 1) - na
EasyLevel22(37, 47, 1) - na
EasyLevel23(23, 259, 1) - na
EasyLevel24(69, 263, 1) - na
EasyLevel25(6, 272, 1) - na
EasyLevel26(33, 44, 1) - na
EasyLevel27(37, 527, 1) - na
EasyLevel28(21, 258, 1) - na
EasyLevel29(37, 524, 1) - na
EasyLevel30(48, 53, 1) - na
EasyLevel31(32, 338, 1) - na
EasyLevel32(37, 47, 1) - na
EasyLevel33(23, 259, 1) - na
EasyLevel34(69, 263, 1) - na
EasyLevel35(6, 272, 1) - na
EasyLevel36(33, 44, 1) - na
____Unorgainzed levels
TutorialLevel14(33, 44, 1) - daz maze- no known spawn, figure it out
TutorialLevel24(0, 300, 1) - red floor- we good here
TutorialLevel5(10, 100, 1) - sandwich- no known spawn, figure it out
TutorialLevel6(0, 140, 1) - Not the same - _more blocks here needed _
TutorialLevel7(0, 120, 1) - Jump under the platform - we good here
HardLevel1(33, 44, 1) - Small Steps - figoure spawn out and difficulty of level
Note to self and others who might read this.
This issue shouldn't be resolved until all the levels have been tested and approved for functionality, difficulty, and decorations.
This list will be updated and modified when improvements are made with it.

Levels are names are going to be renamed - easylevel16(33,44,1) --> Ashtray(33,44,1)
I think this is better and easier also for whatever reason there are walls |:( so yea

I think changing the name is fine, since we can still keep track of level order with level ID (or with their order in the text file). Also if it's a list for level names, I believe we should move it to the discussions section. For the walls, I've been thinking to completely remove, as platforms have the exact same collision. I will open up a separate issue for it and we should change levels.txt. 👍

If you want to, we can revert the changes I've made.

nah its fine this way, I'll change it soon in the future


Update the links on the repo

All the websites links are not working that are the readme and the side bar.

Check my comment on latest issue

Check my comment on latest issue

I mean update them


Add support for image decoder options

closes #939
Release Notes
Can now have options for the underlying image codecs

:rocket: PR was released in v1.6.0 :rocket:


Make it clear that you need to create an .env file when setting up Notify

What
When you set up notify in the prototype - https://prototype-kit.service.gov.uk/docs/using-notify part of the instruction is to add the Notify Api key at end of the .env file.
We need to make it clear that you need to create this file.
Why
A user asked for support, as they could not find the .env file and from the guidance they thought the file would already exist.
Who needs to work on this
Content Designer
Done when
The guide explains that you need to create a file called .env in the root of your prototype (the same folder as app is in) and the kit will pick it up next time you start running it.

https://github.com/alphagov/govuk-prototype-kit-docs/pull/197


Fix use of case class

I also created pull requests to fix examples (https://github.com/apache/incubator-predictionio/pull/408) and document (https://github.com/apache/incubator-predictionio/pull/409) in the incubator-predictionio repository.
@shimamoto Could you check these fix?

LGTM!


Make commands, hooks and timers dynamically adjustable

It would be nice if plugins had a way of changing (adding, removing, maybe temporarily disabling) commands, hooks and timers at runtime.

dynamic commands, hooks and timers need to survive bot restarts.
Therefore, they might need to be distinguishable from static ones.

dynamic commands, hooks and timers need to survive bot restarts.
Therefore, they might need to be distinguishable from static ones.

closed by https://github.com/alturiak/nio-smith/commit/488dd292f8aefb4614812d36f030f462c319c5f6

closed by https://github.com/alturiak/nio-smith/commit/488dd292f8aefb4614812d36f030f462c319c5f6


bug: IonTextarea with autoGrow set will collapse/disappear in the presence of an IonButton

Bug Report
Ionic version: 5.1.1
Current behavior:
When using an <IonTextarea autoGrow . . . on the same page as an IonButton, the textarea is defaults to a height of 0, which almost completely hides it (there is some overflow). In order to get it to display correctly, you must focus on the textarea and type a character, in which case the autoGrow behavior kicks in and sets the correct height.
Expected behavior:
<IonTextarea autoGrow . . . should be displayed correctly no matter what else is on the page.
Steps to reproduce: / Related code:

Go to the sandbox example
Observe how in Firefox, there is only a button displayed. In Chrome, you can see one line of the textarea text on top of the button.
Move your cursor to the top of the button. You will notice a text selection cursor. Click to focus on the textarea that's triggering the cursor change.
Type a character. You will see the textarea grow to its correct size.

Something to note is that this only seems to happen on page load.  If you remove the IonButton from the page (pages/Home.tsx) and let the hot reload (or whatever Code Sandbox is doing) do its thing, the bug doesn't get fixed. However, if you refresh the preview browser after removing the IonButton, the textarea works as expected.
In my actual application, I'm running into the same issues with IonMenuButton and IonMenu. Or at least I think so; it's a bit difficult to rule out every possibility, but it's a pretty big issue which I can't seem to find a workaround for.
One last thing to note is that I first thought it was an issue with IonTextarea, so I tried using both the react-textarea-autosize and react-autosize-textarea packages, but the same thing happened with those.
Any help would be extremely appreciated.
An example gif

Ionic info:
Ionic:
   Ionic CLI       : 6.9.1 (/Users/grardb/.config/yarn/global/node_modules/@ionic/cli)
   Ionic Framework : @ionic/react 5.1.1

Capacitor:
   Capacitor CLI   : 2.1.0
   @capacitor/core : 2.1.0

Utility:
   cordova-res : not installed
   native-run  : not installed

System:
   NodeJS : v11.14.0 (/Users/grardb/.nvm/versions/node/v11.14.0/bin/node)
   npm    : 6.7.0
   OS     : macOS High Sierra

Thanks for the issue. I am going to close this as a duplicate of https://github.com/ionic-team/ionic/issues/21242 (The particular issue is different, but the underlying bug in Ionic Framework looks to be the same).


Initiate Arabic localization team

This issue is to initiate a new l10n Team
Please only use this template for submitting a new l10n team initiation requests.
See https://github.com/cncf/glossary/blob/main/LOCALIZATION.md for basic requirements for a new l10n team.

(1) The language and language code for l10n
Find localization's two-letter language code from ISO 639-1 standard.

Language name : Arabic
Language two-letter code : ar

(2) List of volunteers
Name and GitHub Username for each l10n volunteer. Add [A] to make them approvers.

[A] Tarek M. Sayed (@tarekmsayed)
[A] Sameh Ammar (@same7ammar)
[A] Ahmed Shabana (@AShabana)
[A] Osama Nabil (@Hacktron95)

(Note) Next actions from maintainers

Create a new development branch for the l10n. (ex: dev-ar)
Update codeowners to add l10n contents approvers.
Add a new label for the language.

Hello @same7ammar  :)
Thank you for the initiation of l10n for Arabic !!
We are going to proceed following tasks

Add a new label for the language.
Update codeowners to add l10n contents approvers.
Create a new development branch for the l10n. (dev-ar)

But we are testing some permission issues for new branches. So, these tasks will take some time.

Hello @same7ammar ! I'd like to inform you following progress.

Added a new label for the language.
lang/ar https://github.com/cncf/glossary/issues/labels
Created a new development branch for the l10n.
https://github.com/cncf/glossary/tree/dev-ar
Update permission to add l10n contents approvers.
Working on it (It takes time to reach consensus on updated permission strategy among administrators)

Arabic team can start site configuration (ex: PR #291) using the branch even though the team does not have a permission for approving PR yet.
You can refer https://github.com/cncf/glossary/blob/38bf7501dc9aea02696a1a6e8d48b74074c670c1/LOCALIZATION.md#2-configure-a-new-language-to-the-site for details (Note: it is a draft of LOCALIZATION.md from PR #341)

Hi @seokho-son .
Thank you so much 👍 .


Topic/cc create cq fixes

@miked-mellanox and @Artemy-Mellanox please review.
FYI:
@yosefe, @alex-mikheev, @vasilyMellanox and @igor-ivanov

Test PASSed.
See http://bgate.mellanox.com/jenkins/job/gh-ibverbs-tests-pr/21/ for details. (Mellanox internal link)

Test PASSed.
See http://bgate.mellanox.com/jenkins/job/gh-ibverbs-tests-pr/23/ for details. (Mellanox internal link)

:+1:


Spin up VMs on multiple libvirt providers

Hi,
On localhost, I am able to deploy and manage multiple VMs this way and its working fine. I have object list of VMs in 'myvms' variable. Using for_each I am planning to deploy VMs listed in 'myvms'.  I have a module 'singlevm' which accepts VM attributes, and spins up a single VM. Here is the code, and this is working perfect for me!
provider "libvirt" {
  uri = "qemu:///system"
}

# For each VM declared in terraform.tfvars, deploy a VM
module "kvm_instances" {
  source = "./modules/singlevm"

  for_each = var.myvms
  vmname   = each.value.hostname
  disksize = each.value.osdisk_gb
  datadisksize = each.value.datadisk_gb
  osimage  = each.value.Image
  memory   = each.value.RAM
  vcpu     = each.value.CPU_Count
}

Now, I am trying to extend above code to deploy and manage VMs on multiple libvirt providers. I attempting to have separate modules for deploying VMs on specific host. For each host I have separate variable which holds the list of VMs.  What is the best way to set 'provider for each of these modules? (The following code was my attempt, and it doesnt work. It throws up a syntax errror!!)
provider "libvirt" {
  uri = "qemu:///system"
}

provider "libvirt" {
  alias = "remotehost1"
  uri   =<EMAIL_ADDRESS>}

provider "libvirt" {
  alias = "remotehost2"
  uri   =<EMAIL_ADDRESS>}

# For each VM declared in terraform.tfvars, deploy a VM
module "kvm_instances_host1" {
  source = "./modules/singlevm"

  for_each = var.vm_list_on_host1
  vmname   = each.value.hostname
  disksize = each.value.osdisk_gb
  datadisksize = each.value.datadisk_gb
  osimage  = each.value.Image
  memory   = each.value.RAM
  vcpu     = each.value.CPU_Count
  providers = {
    libvirt = libvirt.remotehost1
  }
}

module "kvm_instances_host2" {
  source = "./modules/singlevm"

  for_each = var.vm_list_on_host2
  vmname   = each.value.hostname
  disksize = each.value.osdisk_gb
  datadisksize = each.value.datadisk_gb
  osimage  = each.value.Image
  memory   = each.value.RAM
  vcpu     = each.value.CPU_Count
  providers = {
    libvirt = libvirt.remotehost2
  }
}

This is the error i get! Not very much helpful. But I see, it doesnt like the 'providers' json within each module!
Error: virError(Code=38, Domain=7, Message='End of file while reading data: Ncat: No such file or directory.: Input/output error')

  on main.tf line 5, in provider "libvirt":
   5: provider "libvirt" {

Please recommend the best approach

Hello,

Ncat: No such file or directory

Have you installed netcat on the target hypervisors ?
On CentOS/RHEL/Fedora:
sudo dnf install nmap-ncat

Hope it helps.
Nicolas

@nmasse-itix  I see I have those packages already installed on all hypervisors.

@arunnalpet  hi thx for issue.
here is an example of the feature, as I can tell the syntax is bit different there
https://github.com/dmacvicar/terraform-provider-libvirt/blob/master/examples/v0.13/multiple/main.tf#L28
can you give it try?
If it is failing, you could add
TF_LOG=DEBUG terraform apply and attach somewhere the logs
Thank you :sun_with_face:

Hi @arunnalpet
is this issue solved? If so, could you close it?


Link to REST API documentation in "Help" page

The "help" page on classeur.io does not link to any documentation of the REST API. It should; the API should have an article link or a link on the main help page.
I'm viewing the help in English, so it may be present in other languages.

Thank you @zbentley!
For better understanding:
https://classeur.zendesk.com/hc/en-us should provide a link on its page to http://classeur.io/help/rest-api/?
-Robert

Yes please!

@benweet,
I think that's something to post under Enterprise. I don't have access to this section.
Can you add a link there, please?
Thankfully,
-Robert


fixing the join us button issue

🛠 Fixes Issue

👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

Don't mind . This is not correct code..
Try to correct it.

saurava , join us button still not working
shall i correct it..


Update CONTRIBUTING.md

Changed the slack site to discord

Closes #

Changes

Flags

Screenshots or Video

Related Issues

Issue #
Pull Request #

Author Checklist

[ ] Ensure you provide a DCO sign-off for your commits using the --signoff option of git commit.
[ ] Vital features and changes captured in unit and/or integration tests
[ ] Commits messages follow AP format
[ ] Extend the documentation, if necessary
[ ] Merging to master from fork:branchname
[ ] Manual accessibility test performed

[ ] Keyboard-only access, including forms
[ ] Contrast at least WCAG Level A
[ ] Appropriate labels, alt text, and instructions

Signed-off-by: Avyay M C<EMAIL_ADDRESS>


Chore: Allow node 14+ in published libs

This PR changes the allowed Node-versions in package.json files to allow all Node-versions above 14.19 (and not only allow 14 and 16).
This applies only to the libraries that are published and used by external apps.
My reasoning for this is that I think that we should not hold back other libraries by disallowing the use of newer versions of Node

Codecov Report
Base: 68.10% // Head: 72.56% // Increases project coverage by +4.45% :tada:

Coverage data is based on head (415ba28) compared to base (6c3fe03).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@              Coverage Diff              @@
##           release47     #860      +/-   ##
=============================================
+ Coverage      68.10%   72.56%   +4.45%
=============================================
  Files            280       98     -182
  Lines          34894    24905    -9989
  Branches        4656     2390    -2266
=============================================
- Hits           23766    18073    -5693
+ Misses         10677     6795    -3882
+ Partials         451       37     -414

Impacted Files
Coverage Δ

meteor/server/api/ingest/rundownInput.ts

meteor/lib/collections/RundownLayouts.ts

meteor/client/utils/positions.ts

meteor/lib/collections/Snapshots.ts

meteor/server/api/ExternalMessageQueue.ts

...teor/client/lib/reactiveData/reactiveDataHelper.ts

meteor/server/security/rundownPlaylist.ts

meteor/server/migration/databaseMigration.ts

meteor/server/api/profiler.ts

meteor/server/api/snapshot.ts

... and 172 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Does anyone remember why we needed ^16.14 specifically? Generally, I agree that there's no point in limiting consumers to a ceiling.

Perhaps it was inherited from a dependency? I can't think what that would be though


Calling Play or Dispose causes freeze when RTSP stream end was reached

When RTSP stream was ended (by timeout for example) and EndReached event was raised, it is impossible to restore streaming by calling Play method. The application will freeze if you try.
Disposing whole object for recreation also causes freeze.
Leaving VlcControl without dispose and creating new object couses memory leaks.
In other words: when RTSP stream is interrupted there is no way to get it back without restarting the application. It's not right. I assume it's a bug.
PS. Using Vlc ActiveX control I can achieve that with no problem.

I'm experiencing a similar thing with local files played using VLC in C#. What I'm doing is creating a series of vlc controls that get placed on a panel in a media sequencer. It then swaps the controls out and plays each one. I then try to dispose all the controls by first removing them from the Controls collection, then running the Dispose method. It ONLY freezes on the most recently played control. What I'm about to try is to create a local video file that is one second long and is blank, switching to it after the media sequence is done (leaving it basically a static control that always gets called and played) then disposing all the controls in the media sequencer, creating the new one, etc. See if that lets me dispose all the controls and clears the memory to boot.

And... it's causing other problems now... or my logic is... not sure... either way, I can't get an accurate read on whether or not memory is being freed.

Several issues reported freezes when disposing. We need to reproduce this issue and see what's wrong.
If you want to dig into the code and see what's wrong at disposal, that would really help me. Thanks

I suspect that your issue is caused by this : https://github.com/ZeBobo5/Vlc.DotNet/wiki/Vlc.DotNet-freezes-(don't-call-Vlc.DotNet-from-a-Vlc.DotNet-callback)
Please reopen if you still have an issue.

That sounds very likely the cause. Time was a factor, though, so I dumped everything and rewrote the app as a WPF using the native video panel.


Add Dynamic Contributors Section to README

Problem / Issue No.
Adding a "Contributors" section that dynamically displays profile photos of all contributors
Describe Problem / Root Cause
This change promotes visibility for all contributors, recognizing their efforts and encouraging more community involvement in the project.
Solution proposed
This pull request enhances the README file by adding a "Contributors" section that dynamically displays profile photos of all contributors using contrib.rocks.
Additional Information
Integrated a contributors section at the end of the README, which showcases all contributors with profile images linked to their GitHub profiles. This section is generated automatically, ensuring it's up-to-date as new contributors join the project.
Screenshots

Original Screenshot (Problem/Issue)

Updated Screenshot (Fixes/Solution)

@andoriyaprashant kindly check and merge please

give me label also of gssco24


NPM analisys: add package license information (enhancement)

We are using depcheck also to inventory open source licenses used in our application, very powerfull features for maven/java component!
For npm depcheck analisys (v. 3.2.1) is still missing any component license info in the reports, also for package.json with the corret tag compiled (SPDX expressions):
(from https://docs.npmjs.com/files/package.json)
{ "license": "ISC" }

{ "license": "(MIT OR Apache-2.0)" }

{ "license" : "SEE LICENSE IN <filename>" }

This maybe due to an overall issue with the NSP and NPM analyzers that was resolved in PR #1361. If you look at the code in the AbstractNpmAnalyzer the license information should be being collected.
I will hopefully be pushing a new version within the week and I would ask you to re-test then.


Liberms/#36: add delete cmd

Overview
Add shore delete [--dry-run | -d] command that deletes the pipeline and it children pipelines in the application generated by the shore project. Expose some of the APIs used to implement the command.
Github Issue: #36
Summary (required always)

delete command async implementation that utilizes the bellow exposed APIs (added to Backend interface):

add DeletePipeline() - deletes nested pipeline in spinnaker backend and displays all the pipelines deleted in the application - operates upon pipeline rendred with Render() (as Save())
add GetPipelinesNamesAndApplicaiton()  - returns the list of one or more pipelines names and application name in the pipeline configured.

spinnaker package also exposes DeletePipelines() without requiring this API on Backend interface - this allows to implement custom logic to delete multiple pipelines under spinnaker application not bound to shore project rendering.

UT/integration supplied

--dry-run | -d option makes delete command to only print out application and the list of pipelines to be deleted and exits. Doesn't make any RPC calls and doesn't initialize spin client.

Output of full command

Output of dry-run

Notes

Checklist

[x] My pull request title follows the format <username>/<gh-issue-#number>:<short-description>
[x] My code passes existing unit tests
[x] My code follows the code style set for the project
[x] I have added at least one reviewer for this PR

I have read the CLA Document and I hereby sign the CLA


Make edits available

I cannot edit a poll after submission. It could be frustrating when one cannot correct a typo. :sweat_smile:
For integrity, the post could contain a section (inserted to the end of it after user submission), which would warning users that it was changed, and would contain a version history section.

With current model this can be tricky, but with improvements each user will have secret key to make an edits.


fix attribute trailing comma

Missing multi-line attribute values was causing trouble with syntax highlighting. It seems to work perfectly now.
I added two tests to make sure trailing comma and missing both resulted in the same tree.

I fixed the CI. It was missing a package: node-addon-api.
I also upgraded to the latest LTS just because it felt right.
Proof it's working: https://github.com/angaz/tree-sitter-templ/actions/runs/8526421137

Thanks!


improve compatibility with different Invocable implementations

Nice one!

@dougmencken -- would you mind having a rebase and try again? It seems like you did a bit of re-indentation which caused this now to conflict in TestBshScriptEngine.java

Pushed an updated version. The only change was addition of "package bsh;" on top of that file


Fix publish and cache removal

Fixes #12 , for some reason, after the "exists" check in a package publish, the stream is already read, added a pause and resume to avoid this. Tested it locally and it worked, there wasn't a noticeably overhead.
Also proposes change on #13 , deleted any reference to a memory cache. As I said in that issue, there's no point in adding caching for the DB. Maybe a configuration parameter could be in order, but if anyone would want to cache in memory, they should be using just local storage.

Fix #12 & #13


Enable grafana explore feature for the anonymous access

This project uses GF_AUTH_ANONYMOUS_ENABLED=true env variable for the grafana to allow anonymous users to view SCS monitoring metrics and logs.
We also want to grant access to grafana explore feature for the viewers.
One way how to do it is to use viewers_can_edit setting. In our case, set GF_USERS_VIEWERS_CAN_EDIT=true

After adding this env variable GF_USERS_VIEWERS_CAN_EDIT: true into values, anonymous users are able to view logs. But also they can now edit dashboards, etc. and I think this is not what we want.

After adding this env variable GF_USERS_VIEWERS_CAN_EDIT: true into values, anonymous users are able to view logs. But also they can now edit dashboards, etc. and I think this is not what we want.

Users can edit panels and queries but are not able to save them or create new dashboards.

After adding this env variable GF_USERS_VIEWERS_CAN_EDIT: true into values, anonymous users are able to view logs. But also they can now edit dashboards, etc. and I think this is not what we want.

Users can edit panels and queries but are not able to save them or create new dashboards.

Yes, after testing it, users can only hit apply button. For saving, they need to have additional permissions.


fix(core): fix dependency collection for computed properties (fix #7960)

This fix addresses a regression introduced in https://github.com/vuejs/vue/commit/653aac2c57d15f0e93a2c1cc7e6fad156658df19, by altering the order of watcher.evaluate and watcher.depend, thus preventing the Vue.sets in a computed property getter from
accidentally clearing the deps array of the computed watcher.

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
[x] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

Closing as this is reverted in 6b1d431a


IP core extension

Hi, Dshekhalev
I'm interested in IP core extension. Can you give me your email for further communication？
Thanks

Sorry for waiting, I will be out for three weeks and can't be in touch. My e-mail for communication<EMAIL_ADDRESS>


Random responses incl 500 Internal Server error from Get rights/delegation/offered

Description of the bug
Getting offered right delegations for test organization as it's DAGL gives random responses.
200 OK with delegations
200 OK without delegations
500 Internal Server Error
Steps To Reproduce
Bruno request to reproduce can be found in linked branch
Additional Information
No response

Fixed by #611
Closing issue


Sync code with latest rs-fsrs

The main branch has been updated to FSRS-5 after merging #15.

oh, ok, I'm a little bit late


[FEATURES] Refactor getAvaliablePort

What feature you'd like to add:
func (e *AlluxioEngine) getAvaliablePort() (allocatedPorts []int, err error)

Change getAvaliablePort to global utils package which can be reused by other runtime rather than alluxio runtime
Change PORT_NUM from constant to parameter

Why is this feature needed:

closed by #727


The Calendar should be shown dark in the dark theme.

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
master
Issue Description
The Calendar should be shown dark in the dark theme.
Preconditions

Admin should be logged in
Admin needs to enable dark theme.

Steps to reproduce

Go to 'Admin panel'
Click on the 'Dark theme button'
Click on the store state Calendar
See the error

Expected Result
Should be shown the calendar in Dark.
Actual Result
It shows the calendar the same as a light theme. Here
Screenshots
No response
Additional context
No response

Hi can you please tell how to replicate this for myself, I opened the admin panel and the calendar dark mode is already working.

Hi @udeet27, The issue is replicated in the Bagisto 2.x. you are using Bagisto version 1.x.

Checked and Found that the issue has been fixed, please check the video for the reference.
Video
https://github.com/bagisto/bagisto/assets/92721837/ef939d78-8eae-4fc2-be0f-9dc747d610c5

Checked and Found that the PR has been merged and issue has been fixed. Hence I am closing the issue.
PR - https://github.com/bagisto/bagisto/pull/9046


textarea has wrong height after set text

if text is set to via JS, textarea has wrong height. Text should has a lot of enters( \r\n)
example

I agree that this is an issue.
At the very least, if the auto-resize must remain without option to turn it off, should there be some way of indicating that there is a lot of text in the text area? (maybe that's a better way to approach this issue?)

There is a note in the documentation saying if you dynamically set the text in a textarea you have to manually trigger the resize.
$('#textarea1').val('New Text');
$('#textarea1').trigger('autoresize');


Added support to instant run

This was a headache to all of us using the new Android Studio with Instant Run feature
Basically all I did was reading dex files in the newly added folder "instant-run" when using the Instant Run feature.

Coverage decreased (-0.06%) to 67.433% when pulling 4b75e4d6c6cd4ab77a334077be88ece6c3b162b2 on firasalkafri:master into 2358dae45b36a9e19e58a66566727ccd71792604 on satyan:master.

@firasalkafri I will take a look tomorrow (Monday), thanks

@firasalkafri could you solve the conflict? I will take a look to your code if solved this.

@JonatanSalas yes it's solved, as you can see in my changes. I only added a few statements to read dex names and paths in the newly created instant-run directory when using the Instant Run feature.

@JonatanSalas I don't see any conflicts :/

@JonatanSalas now I see it, resolved.

Coverage decreased (-0.06%) to 69.685% when pulling 9108e79ded710efba008780e746b3422c96d25b5 on firasalkafri:master into fbd74750ed15cbfe5f80e5f785a7c78919f7be10 on satyan:master.

@firasalkafri could you rebase your commits?

Coverage decreased (-0.06%) to 69.685% when pulling e974625638d4b6f4b4a4047f5add19f636076606 on firasalkafri:master into fbd74750ed15cbfe5f80e5f785a7c78919f7be10 on satyan:master.

@sibeliusseraphini I think everything is alright now

Coverage remained the same at 69.747% when pulling 77828dc4ba5b99616e1b5f336cfc75f3c006a4a1 on firasalkafri:master into fbd74750ed15cbfe5f80e5f785a7c78919f7be10 on satyan:master.


Header.SetExtension does not add an extension with id > 14 as a two-byte extension

Your environment.

Version: <=v1.8.13
Browser: n/a

What did you do?
Add an extension with payload with size <= 16 and extension map ID > 14.
What did you expect?
Header's ExtensionProfile should be set to ExtensionProfileTwoByte.
What happened?
It is set to ExtensionProfileOneByte. Consequently, the receiver (browsers) ignores the extension because its ID is invalid.

I will create a PR that would fix this (#255) and #249 together.


Use shorthand-ternary operator where possible

A minor improvement for code clarity, with the shorthand ternary operator.

\Illuminate\Routing\UrlGenerator::asset
\Illuminate\Database\Eloquent\Relations\Concerns\InteractsWithPivotTable::getCurrentlyAttachedPivots

Both these instances contain a ternary operator that uses the same value on its truthy expression, and the elvis/shorthand-ternary operator can shave off some code and improve readability.

In the article you have given as an example, it does not matter how the incoming data came from. However, for assetRoot, a data must be received. If the data does not come, the URL generation will be wrong.

Thanks for your review @selcukcukur. I'm afraid I'm probably misreading your first, comment, but I'm only converting the ternary operator to use the shorthand. Unless fetching $this->using or $this-assertRoot changes the state (which it does not, as far as I see), it is safe to change the ternary to ternary-shorthand.

Thanks for your review @selcukcukur. I'm afraid I'm probably misreading your first, comment, but I'm only converting the ternary operator to use the shorthand. Unless fetching $this->using or $this-assertRoot changes the state (which it does not, as far as I see), it is safe to change the ternary to ternary-shorthand.

@Ayesh I can understand you're using it for triple abbreviations. But this abbreviation changes the situation as follows.

Use $this->assetRoot or $this->using if not empty. If empty, use condition 3.

But this should be like this.

"assetRoot" and "using" should not be empty and the correct value should come. Otherwise, it may send an incorrect answer with a null value. I am sorry I cannot speak very well.

Thanks for the follow-up @selcukcukur .
This PR does not really change the way the call behaves.

Use $this->assetRoot or $this->using if not empty. If empty, use condition 3.

This is the same way it works even before this PR. What I'm proposing is purely a syntax change to improve the code clarity. With the shorthand ternary, PHP automatically returns the $this->assetRoot and $this->using values themselves (not true or false, and not null) if they are true-ish. It does not send a null value. This is the same before this PR too, but is now more cleaner without repeating $this->assetRoot and $this->using.


Use a configuration first method (modifiers and states) for ModalBox

Relates directly back to #273

After talking with @mcarrano and @kybaker we want to move the close icon closer to the right edge of the modal and aligned to the top of the text in the modal. And the close icon should have it's own "column" in the layout so that text doesn't wrap under it. Similar to this


Audit events

I can't find info in the audit events table about "User failed to start the registration for Authenticator App with Notification and Code" is it missing?
Service: Authentication Methods
Activity Type: User started security info registration

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

ID: 209406f7-3ef9-7424-a11d-46146b232906
Version Independent ID: 53f76c21-5d6d-54d2-8166-d6c3e0d2d508
Content: Troubleshoot combined registration for Azure AD SSPR and Multi-Factor Authentication (preview) - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-registration-mfa-sspr-combined-troubleshoot.md
Service: active-directory
Sub-service: authentication
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@bartvermeersch Thanks for the feedback. We are investigating into the issue and will update you shortly.

@MicrosoftGuyJFlo Could you please have the above audit event added to the page after review with engineering ?

Emailed the PG for guidance.

#please-close

@MicrosoftGuyJFlo can you share the feedback from the PG?


TreeView: Add support for external styling

Description
We are using TreeView in the NestedListView and we needed to override some styling to match the design needs for using NestedListView for SubIssues.
Currently, we are using css selectors to override some of the styles (which seems a little hacky) and thought it would be helpful we could pass styles using a prop.
Example PRs:

https://github.com/github/github/pull/320799
https://github.com/github/github/pull/320836

Hi! Thanks for taking the time!

Currently, we are using css selectors to override some of the styles (which seems a little hacky) and thought it would be helpful we could pass styles using a prop.

I see TreeView doesn't support sx for all the child items. This was partially intentional at the time of building it so that the design does not deviate too much. The use cases however are many and varied now (like NestedListView!)
We want to phase out styling through a prop (sx) over time, so I would recommend using classNames. However, TreeView does not have reliable selectors that you can target yet. Would that be a useful change for you?

@siddharthkp I think a good solution would be to support custom styling on TreeView and TreeView.Item. That could be via the sx prop or className prop (or maybe both for this in-between time).

However, TreeView does not have reliable selectors that you can target yet.

I think we should probably avoid getting into the habit of exposing targetable selectors, because it makes things difficult to change. Long term, it will make it harder for PRC to move to CSS Modules because the class names would need to remain static.
If there's specific things we want to support styling internally, I think we should either add props for the required variants, like hideNestingIndicators, or we should add support for more externally defined classnames rather than defining new internal classnames, like nestingIndicatorClassName.
Or, one more option might be to set data- attributes for selection instead of classes, which preserves encapsulation. But I think we'd want to document those and personally I'd prefer some props-based approach that can be typechecked vs an approach that requires more complex CSS selectors.

Closed by https://github.com/primer/react/pull/4512


<EMAIL_ADDRESS>hash check failed

New version of explorer++ is out so the hashes aren't matching
Updating 2 outdated apps:
Updating 'explorerplusplus' (<IP_ADDRESS>1 -> <IP_ADDRESS>3)
Downloading new version
explorerpp_x64.zip (2.0 MB) [=================================================================================] 100%
Checking hash of explorerpp_x64.zip ... ERROR Hash check failed!
App:         epower/explorerplusplus
URL:         https://ci.appveyor.com/api/projects/derceg/explorerplusplus/artifacts/explorerpp_x64.zip?branch=master&pr=false&job=Platform%3A%20x64
First bytes: 50 4B 03 04 14 00 00 00
Expected:    7b0067e8a1a7e48b10cfaec463eb5085b860e99c627efdc86e898a104c29f9c5
Actual:      73c45ec487cccb6c1e94c25099a19d1b6fe25a505d2c254ca587d5c7eab422c1

Please update to version <IP_ADDRESS>2, this application updated frequently recently.


discover crash

2015-12-06 11:21 CRITICAL task          tv-shows-find   BUG: Unhandled error in plugin discover: 'NoneType' object has no attribute 'read'
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/flexget/task.py", line 438, in __run_plugin
    return method(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/event.py", line 22, in __call__
    return self.func(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/input/discover.py", line 268, in on_task_input
    entries = self.estimated(entries)
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/input/discover.py", line 190, in estimated
    est_date = estimator.estimate(entry)
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/estimators/est_released.py", line 28, in estimate
    estimate = estimator(entry)
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/estimators/est_release_series_tvmaze.py", line 52, in estimate
    episode = lookup(**kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/utils/database.py", line 30, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/api_tvmaze.py", line 397, in episode_lookup
    episode = TVMazeEpisodes(pytvmaze_episode, series.tvmaze_id)
  File "<string>", line 4, in __init__
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/state.py", line 306, in _initialize_instance
    manager.dispatch.init_failure(self, args, kwargs)
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/util/langhelpers.py", line 60, in __exit__
    compat.reraise(exc_type, exc_value, exc_tb)
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/state.py", line 303, in _initialize_instance
    return manager.original_init(*mixed[1:], **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/api_tvmaze.py", line 159, in __init__
    self.update(episode)
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/api_tvmaze.py", line 176, in update
    self.airstamp = parser.parse(episode.airstamp)
  File "/usr/local/lib/python2.7/dist-packages/dateutil/parser.py", line 1008, in parse
    return DEFAULTPARSER.parse(timestr, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/dateutil/parser.py", line 392, in parse
    res = self._parse(timestr, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/dateutil/parser.py", line 492, in _parse
    l = _timelex.split(timestr)         # Splits the timestr into tokens
  File "/usr/local/lib/python2.7/dist-packages/dateutil/parser.py", line 174, in split
    return list(cls(s))
  File "/usr/local/lib/python2.7/dist-packages/dateutil/parser.py", line 171, in next
    return self.__next__()  # Python 2.x support
  File "/usr/local/lib/python2.7/dist-packages/dateutil/parser.py", line 164, in __next__
    token = self.get_token()
  File "/usr/local/lib/python2.7/dist-packages/dateutil/parser.py", line 82, in get_token
    nextchar = self.instream.read(1)
AttributeError: 'NoneType' object has no attribute 'read'

Let me know if you need any more information!

Thinking this is probably fixed with SHA:4c0498b

Actually with https://github.com/Flexget/Flexget/commit/527d5fc6c3cb7b4f5623175f96eff45acb90d33b
Should be fixed with next release.
In the mean time you can disable estimators. Look in discover wiki for details.

This should be fixed now, please try again and report back.

Tried with 1.2.402 and got the same error:
2015-12-07 20:40 CRITICAL task          tv-shows-find   BUG: Unhandled error in plugin discover: 'NoneType' object has no attribute 'read'
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/flexget/task.py", line 438, in __run_plugin
    return method(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/event.py", line 22, in __call__
    return self.func(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/input/discover.py", line 268, in on_task_input
    entries = self.estimated(entries)
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/input/discover.py", line 190, in estimated
    est_date = estimator.estimate(entry)
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/estimators/est_released.py", line 28, in estimate
    estimate = estimator(entry)
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/estimators/est_release_series_tvmaze.py", line 52, in estimate
    episode = lookup(**kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/utils/database.py", line 30, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/api_tvmaze.py", line 406, in episode_lookup
    episode = TVMazeEpisodes(pytvmaze_episode, series.tvmaze_id)
  File "<string>", line 4, in __init__
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/state.py", line 306, in _initialize_instance
    manager.dispatch.init_failure(self, args, kwargs)
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/util/langhelpers.py", line 60, in __exit__
    compat.reraise(exc_type, exc_value, exc_tb)
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/state.py", line 303, in _initialize_instance
    return manager.original_init(*mixed[1:], **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/api_tvmaze.py", line 159, in __init__
    self.update(episode)
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/api_tvmaze.py", line 176, in update
    self.airstamp = parser.parse(episode.airstamp)
  File "/usr/local/lib/python2.7/dist-packages/dateutil/parser.py", line 1008, in parse
    return DEFAULTPARSER.parse(timestr, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/dateutil/parser.py", line 392, in parse
    res = self._parse(timestr, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/dateutil/parser.py", line 492, in _parse
    l = _timelex.split(timestr)         # Splits the timestr into tokens
  File "/usr/local/lib/python2.7/dist-packages/dateutil/parser.py", line 174, in split
    return list(cls(s))
  File "/usr/local/lib/python2.7/dist-packages/dateutil/parser.py", line 171, in next
    return self.__next__()  # Python 2.x support
  File "/usr/local/lib/python2.7/dist-packages/dateutil/parser.py", line 164, in __next__
    token = self.get_token()
  File "/usr/local/lib/python2.7/dist-packages/dateutil/parser.py", line 82, in get_token
    nextchar = self.instream.read(1)
AttributeError: 'NoneType' object has no attribute 'read'

Luckily the workaround works!

@liiight check to to make sure variables are there before parsing. especially air_dates because it might not have aired yet. https://github.com/Flexget/Flexget/blob/develop/flexget/plugins/api_tvmaze.py#L176

Right, my bad. I'll push a fix asap

I can't seem to reproduce this. Even though this should be a simple fix I want to reproduce it before pushing a fix. Is is possible you send entire debug log and not justcrash?


Node.js message routing update

[x] Is numbered and named correctly and following the spec
--
[ ] All files have been spell checked
[ ] All style cop/ linting issues have been addressed
[x] Include all DCR changes
[x] Explicit state save
[x] Method name changes begin, end etc
[x] Does not use mainDialog unless that file actually extends Dialog/ ComponentDialog
[x] Compiles with latest packages
[x] Runs per instructions in README
[x] Add commented out code to use blob storage
[x] Update to look for production/ local (bot configuration)
[ ] onTurnError at adapter
[x] Include .env or appsettings.json or web.config with botFilePath and botFileSecret
[x] Deployment scripts folder has been cleaned up and only includes msbot export files
[ ] README to include
[x] msbot clone steps
[ ] Tools install/ uninstall
[x] Concise set of steps to run the sample
[ ] C# sample -- Include .bot file in the build output
[ ] All samples include the latest botfile (no "secretKey" field; replaced with "padLock"). Use latest MSBOT off the tools MyGet feed to generate the bot file
[ ] remove and sort usings
[x] remove azuredeploy.* under deploymentscripts folder (if any)
[x] Welcoming users is consistent & correct. See 13.basic bot for nodejs & 3. welcome user for C#

@vishwacsena  I was planning on putting moving it in another PR, not sure if I should move to samples in this pr.


fix: Update codebase to use newer django syntax

all commands of django_extensions now inherit from a base command _BaseDjangoExtensionsCommand.
this helps in defining constant at a single place.(The _ is prefixed intentionally to indicate the class as an internal API.)
handle django41 deprecation warnings.
update some test code to use upgraded version of django code.

I have had to make some changes to workaround mypy which might make the diff a bit more than it should have.
As of now, it seems that mypy doesn't necessarily work well with inherited class attributes, hence had to ignore it at certain places as a workaround.
class A:
    a: List[]

class B:
    a = []  # <- mypy tends to complain about this

I have created there as well https://github.com/python/mypy/issues/10375, so for the time being, I think we can use the type: ignore thing as a workaround.

Looks like a big change and the codebase has changed a lot since it was first proposed. I'm closing this. Thank you.


Subchunkable constructor takes too long (~2 seconds)

@nkemnitz is running advanced scripts, each of which is scheduling hundreds of subchunkable runs, and it takes many minutes to just construct all of the subchunkable flows. Link generation is already disabled.
Can we make subchunkable constructor faster?

@dodamih is this fixed? We narrowed the cause down to naive loki_logging, right? If that's the case, should we disable loki until #149 is addressed?

Yeah - does anyone actually use loki at the moment?

@dodamih this is resolved, right?

Yes - resolved by commenting out loki, so subsumed by #149.


[Nintendo DS Arch] Slight differences with the DSi

I don't know how different the DSi was in terms of architecture, but it has a few clear differences with the DS that might be useful to add:
Cartridges could be inserted at any time in the home screen, and would show up in the launcher which could play it without resetting.
Additionally, the system firmware was user-updatable, with Nintendo issuing multiple updates and homebrew software modifying the system firmware NAND as part of the installation.
It also doesn't have a GBA slot, though I believe can still run GBA games through homebrew the same way the original DS does with the slot-1.
This might also be completely useless if the DSi isn't relevant to the article... Hope this helps anyways!

Thanks for the info, I consider the DSi a separate console, so I'll keep this info for future reference.


View your camera stream in Home Assistant using HLS.

How can I check this?
In homeassitant I have camera entities. In lovelase I can view them. But the link still doesn't work

in go2rtc a have error

Run the script by yourself, catch the URL and try playing it in VLC.
Does it work there?
What integration provides this camera?
What kind of URL do you get?

I run script in terminal

I run script on VScode . I get URL ? its works fine

That's great news. It means this integration is working just fine and it's correctly passing the stream URL to go2rtc.
However, go2rtc is not being able to play it well.
Search go2rtc documentation and existing issues, and if there is none open already, I suggest you open one issue there.

Greetings to you Felipe.
Could you please help me a little. Oh, I don't know python at all. According to the description
https://github.com/AlexxIT/go2rtc?tab=readme-ov-file#source-echo
I need to add "fmmpeg:" before the link.
I can't imagine how I need to do this

You can copy the get_stream.sh script and make the following change at the end:
#!/usr/bin/env bash

set -eu

SUPERVISOR_TOKEN="${SUPERVISOR_TOKEN:?"SUPERVISOR_TOKEN is not set, you need to run this script from an add-on."}"
entity_id="${1}"

- exec curl -fsSL -H "Authorization: Bearer ${SUPERVISOR_TOKEN}" "http://supervisor/core/api/camera_stream_source/${entity_id}"
+ stream_url=$(curl -fsSL -H "Authorization: Bearer ${SUPERVISOR_TOKEN}" "http://supervisor/core/api/camera_stream_source/${entity_id}")
+
+ echo -n "ffmpeg:${stream_url}#audio=copy#video=h264#hardware"

No need for a Python script.

Sorry, I forgot to say thank you.
Everything worked fine for me.
Thank you very much for your efforts.
Unfortunately, I can’t send you a cup of coffee from Russia

It's alright, don't worry. I'm happy it worked for you!


DomParticle

DomParticle specialization as a simplified facade for particle writers. Implements a template interpolation model (aka Xenon).

Probably could use squashing if that's an option.

Oops, I need to fix some test failures, my bad.

Manually rebasing not worth the trouble, closing this one in favor of a squashed PR


Laravel Framework 7.16.1 is not supported?

Laravel Version: 7.16.1
PHP Version: 7.4.7
Laravel-admin: #.#.#

Description:
When I hit the following command with docker, I get an error reading the json file and cannot install . Unsupported?
docker-compose exec myApp composer require encore/laravel-admin
Here's the errors.
[Composer\Downloader\TransportException]
  The "https://repo.packagist.org/p/provider-2018%244c49a1daaa85c182fe3e399404805dab0f4ca0a260de6d95e0c988
  a14cec2b29.json" file could not be downloaded: failed to open stream: Connection refused

Steps To Reproduce:

Internet problem


110 problem in simulationinitialize pressure

Description
added second part to test_initialize_pressure.py: setting the analytic solution and checking whether pressure_poisson actually sticks with it or arrives at a different (worse?) solution
@McBs does this close #110 or do we ignore it?
Checklist

[ ] This pull request is associated to an issue
[ ] This PR contains a description
[ ] Add someone else as reviewer and wait for approval before merging.

Hello @PhiSpel! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file tests/flow/test_initialize_pressure.py:

Line 12:76: W291 trailing whitespace
Line 55:5: E303 too many blank lines (2)
Line 55:76: W291 trailing whitespace


Remove redundant pooling modules

pooling_2d.py is duplicated after functions directory is reorganized. This PR removes redundant pooling_2d.py.


unable to load a simple model file

    model = mujoco.MjModel.from_xml_string('/home/mujoco/model/humanoid/humanoid.xml', ASSETS)
ValueError: XML parse error 8:
Error=XML_ERROR_PARSING_TEXT ErrorID=8 (0x8) Line number=1

the model file are come from here:

my code:

ASSETS=dict()

model = mujoco.MjModel.from_xml_string('/home/mujoco/model/humanoid/humanoid.xml', ASSETS)
data = mujoco.MjData(model)

while data.time < 1:
    mujoco.mj_step(model, data)
    print(data.geom_xpos)

The error says that there's something wrong with the XML file. You need to open up '/home/mujoco/model/humanoid/humanoid.xml' and see what's in it.

@saran-t Hi, it was simple this file: https://github.com/deepmind/mujoco/blob/main/model/humanoid/humanoid.xml
anything wrong with the file you provided?

from_xml_string takes the actual XML string, not the filename. Try loading the string yourself or using the from_xml_path constructor.

@yuvaltassa OH......!

@yuvaltassa BTW, how to make Mujoco able to promot on vscode? I can not get any code jump with mujoco...(python)

Sorry, "promot"?

@yuvaltassa yes. code suggestions in IDE.


Rewrite client in Go with proper unit tests

The Python client is a nuisance to maintain and would be much better as a Go-based client in the vein of deisctl. We should rewrite it in Go with a test-driven approach so we have proper unit test coverage.
Replaces #2392, #2885

this could also solve #2680 using https://github.com/docker/docker/blob/master/graph/push.go#L497

Go client is now on par, feature wise, with the python client.


Fix IntelliJ inspection error

Description
Change by #8535 causes TeamCity inspection error in CI (although it does not show the error in the local IDE).

This PR has:

[x] been self-reviewed.

Example failure: https://teamcity.jetbrains.com/viewLog.html?buildId=2535403&tab=Inspection&buildTypeId=OpenSourceProjects_Druid_InspectionsPullRequests


Remote storage adaptor reads influxDB data only from the default retention policy

What did you do?
I have been playing around with the remote storage adaptor to see how the export of data into influxDB works. I have created 2 retention policies "low_density" and "high_density" in influxDB, with the "high_density" policy set as default.
name         duration  shardGroupDuration replicaN default
----         --------  ------------------ -------- -------
autogen      0s        168h0m0s           1        false
low_density  8736h0m0s 168h0m0s           1        false
high_density 1h0m0s    1h0m0s             1        true

I start the remote storage adaptor with the parameter
 -influxdb.retention-policy=low_density
What did you expect to see?
I expect that the adaptor will write/read to/from the low_density retention policy
What did you see instead? Under which circumstances?
The adaptor writes data to the low_density retention policy as expected but reads data from the default high_density retention policy
Below is influxDB log:
[httpd] <IP_ADDRESS> - - [28/May/2017:02:43:31 +0000] "POST /write?consistency=&db=prometheus&precision=ms&rp=low_density HTTP/1.1" 204 0 "-" "InfluxDBClient" 6fbe3de0-434f-11e7-809f-000000000000 2360
[I] 2017-05-28T02:43:34Z SELECT value FROM prometheus.high_density.scrape_duration_seconds WHERE time >=<PHONE_NUMBER>357ms AND time <=<PHONE_NUMBER>357ms GROUP BY * service=query
[httpd] <IP_ADDRESS> - - [28/May/2017:02:43:34 +0000] "POST /query?db=prometheus&epoch=ms&params=%7B%7D&q=SELECT+value+FROM+%22scrape_duration_seconds%22+WHERE+time+%3E%3D+1496039758357ms+AND+time+%3C%3D+1496040058357ms+GROUP+BY+%2A HTTP/1.1" 200 62 "-" "InfluxDBClient" 715e8cbd-434f-11e7-80a0-000000000000 5006

Environment
OS X 10.11.4
Prometheus and influxDB are run as docker containers, remote storage adapter as a go application on os, go version 1.8.1

System information:

Darwin 15.4.0 x86_64

Prometheus version:

prometheus, version 1.6.1 (branch: master, revision: 4666df502c0e239ed4aa1d80abbbfb54f61b23c3)
  build user:       root@7e45fa0366a7
  build date:       20170419-14:32:22
  go version:       go1.8.1

Alertmanager version:
insert output of alertmanager -version here (if relevant to the issue)

Prometheus configuration file:

global:
  scrape_interval:     1s # By default, scrape targets every 15 seconds.
  evaluation_interval: 1s # By default, scrape targets every 15 seconds.
scrape_configs:
  - job_name: "test_service"
    scrape_interval: "1s"
    scheme: "http"
    metrics_path: "/mgmt/prometheus"
    static_configs:
      - targets: ['<IP_ADDRESS>:8081']

# Remote write configuration (for Graphite, OpenTSDB, or InfluxDB).
remote_write:
  - url: "http://<IP_ADDRESS>:9201/write"

# Remote read configuration (for InfluxDB only at the moment).
remote_read:
  - url: "http://<IP_ADDRESS>:9201/read"

@sesto Thank you for reporting this! https://github.com/prometheus/prometheus/pull/2781 should fix this, can you confirm?


Customize Filter view

What I'm trying to achieve
…
I want to customize filters on the category page to show minimal icons too for eg in case of quantity I can show different size of bottle icons
Describe a proposed solution
...
Other solutions I've tried and won't work
…
I cant seem . to find the react component to edit this filter section

something like this I want.
Can someone point me to the component file of the filter section where i can customize it.

@dominik-zeglen / @benekex2 What files would you recommend to start looking to add such icons?

maybe start and try with a png.


Fix soundness hole in the public API

Closes #24

Note that this assumes that nobody outside this crate actually needs to call or override is_u8. With some more work, we can make this method safe to call from outside of the crate.


new release

There have been a lot of changes since, can we get a release so I am not building from master anymore?

that's a great idea. I will prepare a release in a day or two and update this thread.

Took longer than I thought @wilfredcho The new release is now up on PyPI/pip.

Thanks!


#23: The list and its ancestors are now made visible while calcula…

…ting its height in order to get the list's correct height. This fixes the issue that causes infinite requests to be sent if the list is invisible.

Pull Request Test Coverage Report for Build 142

13 of 16 (81.25%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.2%) to 93.59%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/lazyloader.js
13
16
81.25%

Totals

Change from base Build 139:
-2.2%

Covered Lines:
108

Relevant Lines:
112

💛  - Coveralls

lazyloader.js

386: Mistype the element that are hidden -> that is hidden

Can be merged afterwise.