RPi 4: H.264 and HEVC hardware decoding

Hi, I moved my Kodi setup on my Raspberry Pi 4 from LibreELEC to NixOS. This repository with the hardware-specific settings was very helpful. Unfortunately, one important thing still doesn't work: Hardware video decoding acceleration. Currently, Kodi only software decodes videos, which is choppy at H.264 1080p at best.
I enabled hardware.raspberry-pi."4".fkms-3d.enable already, else Kodi wouldn't even start. If I enable DRM PRIME decoding in Kodi's settings, video output is broken and the log spits a lot of this line
ERROR <general>: CDVDVideoCodecDRMPRIME::AddData - send packet failed: Invalid data found when processing input (-1094995529)

The exact error is different depending on video codec (the above was H.264), but always in the class CDVDVideoCodecDRMPRIME.
The reason I am reporting this here: According to one forum poster, hardware decoding with this setup requires not firmware KMS but "kernel"/"real" KMS. I even tried to add my own dtoverlay based on vc4-kms-v3d-overlay and vc4-kms-v3d-pi4-overlay, but video output after reboot was always broken. The latest version of them is here: vc4-kms.nix, vc4-kms-pi4.nix.
Hence my core question: Has anyone successfully set up hardware video decoding on raspberry pi 4 and NixOS? I'm at a loss and thought one of you could help me. Would "real" KMS help?

I'm stuck with your exact same problem. I have already tried adding the vc4-kms-3d overlay to the config.txt on the firmware partition, but it seems to have no effect (changing config.txt was suggested here: https://nixos.org/guides/installing-nixos-on-a-raspberry-pi.html)

Did anyone get this working?

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/slow-video-playback-on-a-raspberry-pi/58101/2


Rewrote timestamping and examples. Multiple bug fixes.

Fixed up the examples, added a new timestamping feature.
ts is now reserved for manual timestamping using USO 8601 timestamps. You now use tsIsNow to add a timestamp at the current time.
Oh! There's also a .gitignore now.
Tabs have also been replaced with spaces, as per PEP 8.
EDIT:
Added a set_timestamp function.

Really good Pull, but you kind of wasted your time. This repo is dead.

I'm unfortunately aware, I was just making changes for my own use and decided it'd be good to try and try out my first PR. First contribution to open source!


Auth Type select hidden behind box in 1.25.26.

What happens?

What did you expect to happen?
Being able to select an auth type, if I click on the select.
OpenAPI Document
No response

Ah I don't think its hidden, I think it might be empty. Mind sharing the spec if its public, or a reproducible example spec?
also how should we handle empty dropdown @cameronrohani ?

@amritk Ah, okay. I think you could select all of them in the past, even if they were not in the openapi spec. I will have to look into this.

Yea I mean its still probably a bug that they aren't showing up!

@amritk I am not sure, how to define this.
https://github.com/scalar/scalar/blob/main/documentation/configuration.md
I am using elysiajs. I put this in my scalarConfig:
authentication: {
    preferredSecurityScheme: "BearerAuth",
}

and this in detail, at the endpoint:
detail: {
    security: [
	    {
		    BearerAuth: []
	    }
    ]
},

But I still do not get any auth types in the ui.
I am going back to 1.17.16 for now, because I used that before and it worked fine.

Do you think you could give me an example spec that shows this bug? Then I should be able to fix it pretty handily!

@amritk
import swagger from "@elysiajs/swagger";
import { Elysia } from "elysia";

const app = new Elysia()
  .use(
    swagger({
      scalarVersion: "1.25.26",
      path: "/api/docs",
      // doesn't generate anything in the openapi spec
      scalarConfig: {
        authentication: {
          preferredSecurityScheme: "BearerAuth",
        }
      },
    })
  )
  .get("/",
    () => {
      return "Hello Elysia"
    },
    {
      // removing this causes the select to be empty
      detail: {
        security: [
          {
            BearerAuth: []
          }
        ]
      }
    })
  .listen(3000);

console.log(
  `🦊 Elysia is running at ${app.server?.hostname}:${app.server?.port}`
);

{
openapi: "3.0.3",
info: {
title: "Elysia Documentation",
description: "Development documentation",
version: "0.0.0"
},
paths: {
/: {
get: {
operationId: "getIndex",
security: [
{
BearerAuth: [ ]
}
],
responses: {
200: { }
}
}
}
},
components: {
schemas: { }
}
}

Adding the security.BearerAuth property causes the auth tab to be completely empty. Not even a select is displayed in that case:

Ok yea so we need the securitySchemes in the components as well like this:
"components": {
    "securitySchemes": {
      "BearerAuth": {
        "type": "http",
        "scheme": "bearer"
      },
    },
  }

check in the openapi generation in the elysia plugin maybe?

@amritk
Where should I put it? Putting it here doesn't do much and above that throws a type error.
Object literal may only specify known properties, and 'components' does not exist in type 'ElysiaSwaggerConfig<"api">'.ts(2353)

scalarConfig: {
  components: {
	  securitySchemes: {
		  BearerAuth: {
			  type: "http",
			  scheme: "bearer"
		  },
	  },
  },
  authentication: {
	  preferredSecurityScheme: "BearerAuth",
  }
},

Result:
"components": {
  "schemas": {}
}

Alright I opened up a new ticket, you can follow it here!
https://github.com/elysiajs/elysia-swagger/issues/160

There should be a place to add it in elysia, I'm not quite sure where though

C# Bearer #4015


Add project_id option to query method of _WorkItem

Provides ability to call query on individual WI_TYPES
with default project

Fixes: https://github.com/RedHatQE/pylero/issues/11

Running autopep8 on whole project is cumbersome, let's use flake8 and autopep8 before raising a PR

# flake8 src/pylero/work_item.py
src/pylero/work_item.py:9:1: F401 'json' imported but unused
src/pylero/work_item.py:408:44: E225 missing whitespace around operator
src/pylero/work_item.py:527:40: E127 continuation line over-indented for visual indent
src/pylero/work_item.py:548:40: E127 continuation line over-indented for visual indent
src/pylero/work_item.py:571:40: E127 continuation line over-indented for visual indent
src/pylero/work_item.py:1387:40: E127 continuation line over-indented for visual indent
src/pylero/work_item.py:1394:40: E127 continuation line over-indented for visual indent
src/pylero/work_item.py:1415:17: E117 over-indented (comment)
src/pylero/work_item.py:1416:17: E117 over-indented (comment)
src/pylero/work_item.py:1417:17: E117 over-indented
src/pylero/work_item.py:1521:40: E127 continuation line over-indented for visual indent
src/pylero/work_item.py:1539:1: E305 expected 2 blank lines after class or function definition, found 1

# Not aggressive just confirming to PEP 8
# autopep8 -i src/pylero/work_item.py

# flake8 src/pylero/work_item.py

I'm unsure if we want to enforce a project_id on all queries, or make it optional. The question is would any of our users ever want to query the entire system, instead of just their project.

Note that this should bprobably be a decision across the entire library and not just in this one function

@simzacks ack,
taking a step back and reflecting upon the issue:

With WI_TYPE.query(query): Even if query argument doesn't consist project_id we are adding default_project, as WI_TYPE is created on fly from _SpecificWorkItem

With WI_TYPE.get_query_result_count(query): We are directly calling method on tracker_client without modifying query argument and so the difference in result

Based on above, we can either just document the behaviour or enforce get_query_result_count to be an internal only method.
What's your view?

I think the best way around this is to somehow both enable the users to use a default and at the same time enable them to somehow indicate that they want to query the whole system. Maybe add a BasePolarion  attribute query_entire_system, which will override the default project.


Incorrect guide on how to bridge from Stellar to TFChain

See https://library.threefold.me/info/manual/#/manual__dashboard_readme
Incorrect information.

new one should be on info_grid


push to trigger AP

trigger AP to test publish changes

https://github.com/checkdigit/eslint-config/pull/61

this is to fix a publishing issue with PR #61


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_23vjec6wsp/clone/recipe/meta.yaml:23: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_23vjec6wsp/clone/recipe/meta.yaml:40: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_23vjec6wsp/clone/recipe/meta.yaml:46: missing_license_family: The recipe is missing the about/license_family key.
WARNING: /tmp/abs_23vjec6wsp/clone/recipe/meta.yaml:46: missing_description: The recipe is missing a description
ERROR: /tmp/abs_23vjec6wsp/clone/recipe/meta.yaml:46: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_23vjec6wsp/clone/recipe/meta.yaml:46: missing_dev_url: The recipe is missing a dev_url
Errors were found


Add and update status on GitHub

Update the status of a commit on GitHub once we've triggered/run/completed a job

done


Blurb on requests

The number of request messages that would need to be sent per byte of payload byte received.

That doesn't seem true. Am I missing something? You always send requests for 16KiB-chunks, no matter the piece size.
In fact, the request usage should go down a tiny bit since endgame mode will be shorter.

yeah, it doesn't look right. I'm trying to understand what I meant by that. I think that maybe I meant "have messages" instead of "request messages", and was trying to distinguish between the "cost" of not coalescing the reports of which data one has. What makes me thing that is that I seem to deliberately not call out (4) as something this proposal is trying to address. But even changing it to have messages, it doesn't make much sense, nor is distinguished from (3).
I intend to write up the HAVE coalescing we talked about in a separate BEP.


Add information on backward compatibility

Add a section with information on backward compatibility.
Relates-To: OLPEDGE-1732
Signed-off-by: Halyna Dumych<EMAIL_ADDRESS>

I would request to change the wording from "Data Service" to "Data API" since that is the official customer facing name

@uweschul, changed.


exclude does not work when using project in config file

#Hi,
I have this configuration
project:
  file: Project.xcodeproj
  root: .
  target:
    name: Project
  exclude:
    - Project/Classes/Generated

Sourcery fails to run because it tries to find files under Project/Classes/Generated which have not yet been created.

you don't need to exclude files that are generated with Sourcery, it does that automatically

Hi, @haimwaking was the answer helpful? If so could you close the issue?

So that is the behaviour I observed when using sources but not project. I'm guessing the problem might be in the pbxproj file referencing those generated files, that they might not exist at first?

Our temporary solution was to have a hard-coded script to touch the generated files before running sourcery, which won't scale in the future.

SourceryTest.zip
@haimwaking check out the project I've attached to the comment. It seems it has a configuration file with project but it doesn't fail and I don't need any touch command as a workaround.

Hi @Antondomashnev, thanks for looking into this. That project does not reference the generated files though, which seems to be a problem. Try this one, it will repro the problem above:
SourceryTest.zip

you don't need to exclude files that are generated with Sourcery, it does that automatically

Hi @krzysztofzablocki is it somehow possible to disable this? Or to generate file without
// Generated using Sourcery Major.Minor.Patch — https://github.com/krzysztofzablocki/Sourcery
--
2 | // DO NOT EDIT

I'm generating DTO classes for CoreData entities and I would like them to implement AutoEquatable. Currently i'm running sourcery script twice, one will auto generate DTOs and second should add AutoEquatable but sourcery is ignoring that because of that comment.
Thank you

@beretis your use-case will be possible when #437 gets merged

@beretis #437 is now merged

@lucabartoletti thank you :). Now I just have to wait for new version release, or to build Sourcery from source right?.

@beretis we had a release since then so the newest one should work

Hey @krzysztofzablocki looks like you closed this based on @beretis ' issue but I don't think it addressed the original one raised by @haimwaking which is that when specifying an XCode project file, sourcery will fail if it's own files are included by the project file. This is hurting our build times a lot as we have to touch each output file each build before running sourcery which causes XCode to do a lot of recompiling.

@samskiter what you describe seems like a different issue to me. Even if included to the projects' target (sourcery does not scan just files included into project, it only scans files included in the specified target's build phase) sourcery should skip files which have sourcery header comment. Can you please create another issue with example project that demonstrates it?
But looking at the code I agree that original issue have not been addressed, so I'll reopen this.

Sorry, I'm obviously doing a bad job of re-describing the original issue. I work with @haimwaking and we are still seeing this issue.

Can you provide error description?

Thanks @ilyapuchka, if I'm reading it correctly the last PR you created might fix it. I described the error in the very first comment and also attached a sample project in one of my follow ups.


[garden] Added  to

➡ Forward port #985
Forward port of https://github.com/gazebosim/sdformat/pull/985
Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

I believe this was already merged forward in #1024


Http server connect issue

I tried with the steps, and found the http server demo can not work properly:

Then send request will failed:
coderscat@MININT-ODP168:~/dapr-wasm$ curl -d "WasmEdge" -X POST http://<IP_ADDRESS>:8000
curl: (7) Failed to connect to <IP_ADDRESS> port 8000: Connection refused

If we try to telnet, it will crash with an error:

~/wasmedge-quickjs/example_js$ wasmedge --dir .:. ../target/wasm32-wasi/release/wasmedge_quickjs.wasm http_server_demo.js
listen on <IP_ADDRESS>:8000
InternalError: Error(TrackableError { kind: InvalidInput, cause: Some(Cause("assertion failed: `left == right`; assertion failed: `(left == right)` (left: `'\\r'`, right: `' '`)")), history: History([Location { module_path: "httpcodec::method", file: "/home/coderscat/.c
    at accept (native)
    at <anonymous> (http_server_demo.js:15)

Could the Dapr check sends a "proper" http request with a header? The telnet request is technically not http.

I know the reason now.
We must run the command nohup，otherwise we can not connect to server successfully.
cat@MININT-ODP168:~/code/wasmedge-quickjs/example_js$ nohup wasmedge --dir .:. ../target/wasm32-wasi/release/wasmedge_quickjs.wasm http_server_demo.js &
[1] 28385

I don't know the root cause, why do we have this limitation？
wasmedge --dir .:. ../target/wasm32-wasi/release/wasmedge_quickjs.wasm http_server_demo.js

can not start the server properly.

Hmm, without nohup, you can still connect via curl or other HTTP clients. But in order to connect via Telnet, we must use nohup. Is that correct?

Just to make sure -- if you do not use nohup, you will need to start the server in one terminal and run the HTTP client in another terminal window.

Just to make sure -- if you do not use nohup, you will need to start the server in one terminal and run the HTTP client in another terminal window.

I used multiple terminals to test, I guess there is a potential issue we haven't understood :)

@chenyukang Both curl and telnet fails on your machine if you do not have nohup?

@chenyukang Do both curl and telnet fail on your machine if you do not have nohup?

Yes


freerdp crash on autoreconnect

Connecting to a windows 10 after some minutes (or also more time, like one hour) disconnect (for issue server side seems) and on autoreconnect it crash.
Here the terminal output, latest lines probably can be useful, and backtrace: https://paste.debian.net/1232894/
I don't know if can be related but I spotted this trying to solve/workaround/debug another issue that make me going crazy recently: occasional disconnections from the windows 2019 terminal servers, it happens only on some servers (and also on clients with 2 servers "twins on one happens and on one does not"). No crash in that case and no useful info client side. Any help to debug is appreciated.
Application details

FreeRDP version 2.6.0
Command line used: launched from remmina
OS version connecting to (server side): windows 10 pro v2004 build 19041.1415
custom build based on debian packages: http://debomatic-armhf.debian.net/distribution#dpr-tc-buster-backports https://github.com/M2Rbiz/freerdp2-pkg https://github.com/M2Rbiz/remmina-pkg

Environment (please complete the following information):

OS: linux
Version/Distribution: Debian 10
Architecture: armhf
hardware: raspberry 4b

This looks like you´ve encountered a bug in rdpsnd channel cleanup code.
I´m surprised this was not discovered sooner, it is there for ages now (2.1.0)

The dynamic channel version creates a background thread decoding data with the events CHANNEL_EVENT_INITIALIZED and cleans them up with CHANNEL_EVENT_TERMINATED
The context variables are created in CHANNEL_EVENT_CONNECTED and cleaned up in CHANNEL_EVENT_DISCONNECTED This is a recipe for headaches.
There is also a static channel version which shares the context. Cleanup is done in a different function. This needs proper guards and variable invalidation, which is missing/only partly there.

@Fantu I´ve created a backport pull request #7698 which should address this. Can you confirm?


OLS-624: Disable additional properties + basic REST API tests with attachments

Description
Disable additional properties + basic REST API tests with attachments
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)
[x] e2e tests

Related Tickets & Documents

Related Issue #OLS-624
Closes #

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

Project coverage is 95.26%. Comparing base (de85e97) to head (f0bc72c).
Report is 35 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1030      +/-   ##
==========================================
- Coverage   95.34%   95.26%   -0.08%
==========================================
  Files          54       54
  Lines        2211     2259      +48
==========================================
+ Hits         2108     2152      +44
- Misses        103      107       +4

Files
Coverage Δ

ols/app/models/models.py
99.14% <ø> (+0.05%)
:arrow_up:

/lgtm

/approve


KVO support when updating property on background thread

Goals
I would like to get informed via KVO when a property has changed on a Realm Object even when this property has been changed on the equivalent Object on a background thread.
Expected Results
I would have expected
func observeValue(forKeyPath keyPath: String?, of object: Any?, change: [NSKeyValueChangeKey : Any]?, context: UnsafeMutableRawPointer?)
to be called on the observer when the property changed on the background object. The property has been changed on the main thread object, but KVO didn't trigger.
Actual Results
KVO does not trigger even though the property has been changed.
Steps to Reproduce

Create a Realm Object on the main thread
KVO a property
Create a background thread and fetch the object created in step 1
Update the observed property
KVO is not triggered

Code Sample
    let realm = try! Realm()
    let user = User()
    user.id = "1234"
    user.name = "John"
    try! realm.write {
        realm.add(user)
    }
    user.addObserver(self, forKeyPath: "name", options: [], context: nil)

    DispatchQueue.global(qos: .background).async {
        let realm = try! Realm()
        let user = realm.object(ofType: User.self, forPrimaryKey: "1234")!
        try! realm.write {
            user.name = "Jorn"
        }
    }`

Version of Realm and Tooling
ProductName:	Mac OS X
ProductVersion:	10.12.1
BuildVersion:	16B2555

/Applications/Xcode.app/Contents/Developer
Xcode 8.1
Build version 8B62

/Users/jornvandijk/.rvm/rubies/ruby-2.3.1/bin/pod
1.1.1
(not in use here)

/bin/bash
GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin16)

carthage not found
(not in use here)

/usr/local/bin/git
git version 2.10.2

Realm version:

Realm (2.1.1):

Realm/Headers (= 2.1.1)
Realm/Headers (2.1.1)
RealmSwift (2.1.1):

Realm (= 2.1.1)

@jvdijk Realm's KVO should notify when updating a property on a background thread. I believe that the reason is observing object user has been deallocated before changes happen. Could you try to keep user as an instance property instead a local variable?

@kishikawakatsumi You are absolutely right in the example I gave. I'm unable to reproduce it my project today… Thanks for the reply.


Fix labeling for 3Ds

The labeling of inline/xline numbers is off for 3Ds.

I think v3 fixes this.


Chewie Issue

Hey
Great work. And I was trying to run locally.
I get this error.
/E:/flutter_stable/flutter/.pub-cache/hosted/pub.dartlang.org/chewie-0.9.10/lib/src/chewie_player.dart:83:7: Error: No named parameter with the name 'resizeToAvoidBottomPadding'.
resizeToAvoidBottomPadding: false,
^^^^^^^^^^^^^^^^^^^^^^^^^^
/E:/flutter_stable/flutter/packages/flutter/lib/src/material/scaffold.dart:1451:9: Context: Found this candidate, but the arguments don't match.
const Scaffold({
^^^^^^^^
/E:/flutter_stable/flutter/.pub-cache/hosted/pub.dartlang.org/chewie-0.9.10/lib/src/chewie_player.dart:276:17: Error: The method 'inheritFromWidgetOfExactType' isn't defined for the class 'BuildContext'.

'BuildContext' is from 'package:flutter/src/widgets/framework.dart' ('/E:/flutter_stable/flutter/packages/flutter/lib/src/widgets/framework.dart').
Try correcting the name to the name of an existing method, or defining a method named 'inheritFromWidgetOfExactType'.
context.inheritFromWidgetOfExactType(_ChewieControllerProvider)

Here is my flutter and Dart version. There are some breaking changes that happened in Flutter and Dart which might be the cause of this issue.
√] Flutter (Channel stable, 2.0.3, on Microsoft Windows [Version 10.0.19041.867], locale en-US)
• Flutter version 2.0.3 at E:\flutter_stable\flutter
• Framework revision 4d7946a68d (28 hours ago), 2021-03-18 17:24:33 -0700
• Engine revision 3459eb2436
• Dart version 2.12.2

I'm Using Flutter 1.22.6 • channel stable
i'll update to  new version and commit after
meanwhile try  using this  version

yea. Had to downgrade to figure it out. Thank you for the response.


Migrate from Dependabot Preview to Dependabot

Dependabot is moving natively into GitHub.
We should migrate at some point:
https://docs.github.com/en/github/administering-a-repository/enabling-and-disabling-version-updates#enabling-dependabot-version-updates

Bringing this up here since we might end up losing the conversation in the PR otherwise: Do we need the rest of the rules?  In the case of the chromedriver rule I'm pretty sure we don't - we shouldn't be depending on a (potentially old) version of the driver, right?
The Paella version can also likely go, since we're already past the 6.4 limit in the depdendabot config.  I would also assume that the Paella devs would keep us up to date as things move forward.
Typescript.... that I'm less sure about.  @LukasKalbertodt @Arnei you're either working on stuff with typescript, or assigned to PRs reviewing them.  Should dependabot be filing PRs to automatically upgrade things?

I'm pretty sure we don't need any, and I've introduced most (all) of them in the first place


Keyboard Shortcuts and buttons don't work together.

BUG:
If I select Bold and Italic

and proceed to type something, and use the keyboard shortcuts Ctrl+B and/or Ctrl+I it does switch from Bold and Italic back to default formatting, but the buttons remain blue.
If I click the buttons, they turn grey again, but then Bold and Italic formatting get enabled.
It's rather confusing.

Will look into it soon.


allow adding class attribute to some elements

Allows class attribute in mj-text, mj-section, mj-container, mj-column, mj-image.
Resolves #480

Hi @adekbadek and thanks for this PR !
We're discussing about this internally and the best way to add this into MJML.
Something I have in mind is the class attribute name. We already have mj-class what do you guys thinks about prefixing by css-.
I'm not sure about mj-container getting additional css classes, you can only have one per email so I don't think this add much.

I vote for mj-class
When we add the attribute, it will add a class, and if some stuff is in the head relating to this class, it will added too. This way, you keep it simple.

I agree. I think we need to have 2 classes to avoid miss reading. mj-class for mj-elements as we use it actually and the other could could css-class for managing mediaqueries ie. as you suggested.

Hi @adekbadek
We want to merge this for next 3.3.1 but it still need a little work on it. So as we discussed, we want to change class to css-class :  it should avoid confusion about mj-class and regular css classes.
I think we need to add this to he MJMLElement decorator root td for every elements that lives inside a column can now support css-class. CSS classes should be applied on root tag of each elements, and add the same class with a -outlook suffix on potential outlook conditionnal comments.
What do you think about it, do you have a little time to work on this ?
Thanks a lot for your work !

Hi @iRyusa!
I changed class to css-class.
However I'm not so familiar with the project to understand what changes does MJMLElement  decorator need (perhaps you could point me to a readme or something?).
I'll look into it, but in the meantime if someone wants to complete this PR they're welcome

Hey guys! Thanks for your work! Any updates on this?

We're working on improving this PR to ship it with 3.3.2 ! Sorry for not providing enough feedback on this :)

Lol no sorries. You're giving a high-quality service like this is a paid software. Nice work, man! Thanks for your response.

Hi guys,
Just wondering ; so the goal here would also to have a check on the attributes passed to the mj element ? So that we can have errors displayed when we try to apply a class with non-supported attributes ?

@RDeluxe, no, no check on the attributes. The idea is to pass a class name from a MJML element to the first HTML element generated from that MJML element.
Example: if you add css-class="myClass" to a mj-image, the first HTML element generated from mj-image (tr) will bear class="myClass". This will enable you to target your image in a custom CSS style more easily, especially useful for media queries!

Hi @adekbadek
I'm merging this in a branch of MJML repo to work on this. I will ping you when this will be ready to release ! Thanks for you work on this 👍


Automator: update istio/client-go@master dependency in istio/istio@master

Generated by Automator - 2024-12-16T20:12:41+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
e5acfb9925c0b0e231dab93c0dc2fe2972c7bf94
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.


⚠ Bofrost.be has degraded performance

In 66e7340, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2435 ms

Resolved: Bofrost.be performance has improved in d335c48.


Convenience function for splitting weights and biases

Given a vector of parameters p to be used with a SimpleChain sc, it could sometimes be useful to figure out which of the elements of p will be used for weights and which will be used as biases.
I'm currently missing it a bit myself when I'm defining a regularised loss function to be used with an external optimiser.

This is a duplicate of https://github.com/PumasAI/SimpleChains.jl/issues/24


[AutoRelease] t2-loadtesting-2022-12-29-88205(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/3610
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=3474
BuildTargetingString
azure-mgmt-loadtesting
Skip.CreateApiReview
true

issue link:https://github.com/Azure/sdk-release-request/issues/3610


snapshot.changedAttributes() is broken with fragments and Ember Data > 3.1

In Ember Data they have removed the usage of Ember.copy so when you use snapshot.changedAtributes() it expects that this._changedAttributes[key] is always an Array (now they are copying an element with slice instead of using Ember.copy).
The problem is with this code of EDMF: it returns a Boolean instead of an Array. What's the utility of this change?
I'm going to remove the changedAttribute method from this model.reopen to see what happens.

I have already removed the code but i have a test failing, don't know why it happens :thinking:
Source:
TypeError: (0 , _emberMetal.get)(...).getURL is not a function
    at Class.<anonymous> (http://localhost:7357/assets/vendor.js:46173:53)
    at ComputedProperty.get (http://localhost:7357/assets/vendor.js:41212:36)
    at Class.CPGETTER_FUNCTION [as url] (http://localhost:7357/assets/vendor.js:39456:31)
    at Object.object (http://localhost:7357/assets/test-support.js:3391:17)
    at Object.parse (http://localhost:7357/assets/test-support.js:3263:20)
    at Object.object (http://localhost:7357/assets/test-support.js:3392:54)
    at Object.parse (http://localhost:7357/assets/test-support.js:3263:20)
    at Object.object (http://localhost:7357/assets/test-support.js:3392:54)
    at Object.parse (http://localhost:7357/assets/test-support.js:3263:20)
    at Object.array (http://localhost:7357/assets/test-support.js:3225:20)


config: online change raftstore config

What have you changed?
This pr is part of online change config.
This pr add support for online change raftstore config, each thread of store pool and apply pool check incoming config update after handled a batch of fsm, when there is no update  it cost a read of AtomicU64.
What is the type of the changes?

New feature (a change which adds functionality)

How is the PR tested?

Unit test
Integration test

Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No
Does this PR affect tidb-ansible?
No

/run-unit-test

/run-all-test

/run-all-tests

/run-all-tests

/rebuild

/run-unit-test

/run-unit-test

/rebuild

/rebuild

/rebuild


[DIST-1057] Update dependencies

Supersedes #269 #265 #264 #263 #250

:tada: This PR is included in version @typeform/embed-v1.10.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Debugging through USB on physical iPhone often gets HttpException (when phone and desktop are connected to same wifi network)

When start debugging on my iPhone (connected to the computer through USB), I regularly get this error message:
Error connecting to the service protocol: HttpException: , uri = http://<IP_ADDRESS>:1041/5cHcFx711Kk=/ws
The error message shows up when running flutter run or with the debugging tool of VS code.
> edwin$ flutter run
> Launching lib/main.dart on Wenchao Liu’s iPhone in debug mode...
> Automatically signing iOS for device deployment using specified development team in Xcode project: 3S867Y73K3
> Running Xcode build...
>
>  ├─Assembling Flutter resources...                           6.8s
>  └─Compiling, linking and signing...                         4.6s
> Xcode build done.                                           74.4s
> Configuring the default Firebase app...
> 6.8.1 - [Firebase/Core][I-COR000004] App with name __FIRAPP_DEFAULT does not exist.
> Configured the default Firebase app __FIRAPP_DEFAULT.
> 6.8.1 - [Firebase/Analytics][I-ACS023007] Analytics v.60101000 started
> 6.8.1 - [Firebase/Analytics][I-ACS023008] To enable debug logging set the following application argument: -FIRAnalyticsDebugEnabled (see http://goo.gl/RfcP7r)
> Installing and launching...                                        15.7s
> Error connecting to the service protocol: HttpException: , uri = http://<IP_ADDRESS>:1041/5cHcFx711Kk=/ws

Despite the error, the app starts like normal, only the debugger stopped working (hot reloading too).
I have identified that this issue is wifi-related. To be clear, I did not attempt wireless debugging and the phone and desktop are always connected through the same USB cable. I tested under the following scenarios:

iPhone on wifi, desktop on same wifi - the problem often occurs(but not always)
iPhone on wifi, desktop on different wifi from the same router (2.4GHz and 5GHz networks) - same as above
iPhone on cellular data, desktop on wifi - never any issue
iPhone on wifi, desktop on a separate wifi network - never any issue
iPhone on wifi, desktop disconnected from wifi - never any issue
iPhone on wifi, desktop disconnected from wifi, start the debugger, then reconnect to wifi - often get a slightly different error message when doing a hot reload: Could not update files on device: HttpException: , uri = http://<IP_ADDRESS>:1033/CX2EYMGiynw=/

Unplugging and replugging the phone sometimes (not always) solve the problem for a short period of time.

Hello @wliumelb !
Thank you for posting an issue.
Can you please also attach the flutter doctor -v command output?
Thank you.

Hi @janmoppel
Here is the output.

[✓] Flutter (Channel stable, v1.9.1+hotfix.2, on Mac OS X 10.14.6 18G95, locale en-AU)
• Flutter version 1.9.1+hotfix.2 at /Users/edwin/development/flutter
• Framework revision 2d2a1ffec9 (3 weeks ago), 2019-09-06 18:39:49 -0700
• Engine revision b863200c37
• Dart version 2.5.0
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at /Users/edwin/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-29, build-tools 29.0.2
• Java binary at: /Applications/Android
Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.0)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.0, Build version 11A420a
• CocoaPods version 1.7.5
[✓] Android Studio (version 3.5)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 39.0.3
• Dart plugin version 191.8423
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
[✓] VS Code (version 1.38.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.4.1
[✓] Connected device (1 available)
• Wenchao Liu’s iPhone • aceee8140c056e8ecb8ee4de3d244454f7c492dc • ios • iOS 12.4.1
• No issues found!

Also, this issue is probably not related to firebase. I just started a new project with flutter create and got the same error.

the same

Same here...
I think I will go back to native development. This problem is making me waste a lot of time!

I can confirm this issue happens when connected on the same wifi network. I debuggued all day with my Mac connected on my 5.8Ghz network and the iPhone connected on the 2.4Ghz one and no problema at all

I upgraded to 1.12.13+hotfix.7 and this is still an issue

This has not happened for a while. Guess it is somehow fixed already.


If node.js is installed on a path with a space (i.e. C:\Program Files) the powershell scripts fail.

I'll be putting in a pull request with a change that fixed it for me. :)

Oh well, it won't let me push my branch for the PR, but it looks like you already got someone to address this. I did mine a little differently if you're ever interested.


Incorrect pledge(2) signature

The current signature for pledge(2) in https://github.com/aperezdc/signify/blob/master/compat.h#L36 seems to match that of OpenBSD 6.1: https://man.openbsd.org/OpenBSD-6.1/pledge.2 but it got changed in 6.3: https://man.openbsd.org/OpenBSD-6.3/pledge.2 It's listed in the changelog here: https://www.openbsd.org/63.html

Good catch, thanks @saghul for reporting this.

That was quick! Cheers!


Bug Report or Feature Request (mark with an x)
- [x] bug report
- [ ] feature request

Area
- [x] devkit
- [ ] schematics

Versions
node: v8.11.2
npm: 5.6.0
Windows: 10
Angular CLI:  6.0.8
Repro steps
I updated the project from 5.0.1 to version 6. Everything seems to be working fine except for HMR. Configurations for HMR in the project are as follow:
angular.json
    "build": {
              "configurations": {
                "hmr": {
                  "fileReplacements": [
                    {
                      "replace": "src/environments/environment.ts",
                      "with": "src/environments/environment.hmr.ts"
                    }
                  ]
                }
              }
            },
 "serve": {
          "configurations": {
            "hmr": {
              "hmr": true,
              "browserTarget": "appHit:build:hmr"
            },
          }
        },

hmr.js
import { NgModuleRef, ApplicationRef } from '@angular/core';
import { createNewHosts } from '@angularclass/hmr';

export const hmrBootstrap = (module: any, bootstrap: () => Promise<NgModuleRef<any>>) => {
  let ngModule: NgModuleRef<any>;
  module.hot.accept();
  bootstrap().then(mod => ngModule = mod);
  module.hot.dispose(() => {
    const appRef: ApplicationRef = ngModule.injector.get(ApplicationRef);
    const elements = appRef.components.map(c => c.location.nativeElement);
    const makeVisible = createNewHosts(elements);
    ngModule.destroy();
    makeVisible();
  });
};

main.ts
import { enableProdMode } from '@angular/core';
import { platformBrowserDynamic } from '@angular/platform-browser-dynamic';

import { AppModule } from './app/app.module';
import { environment } from './environments/environment';

import { hmrBootstrap } from './hmr';

if (environment.production) {
  enableProdMode();
}

const bootstrap = () => platformBrowserDynamic().bootstrapModule(AppModule);

if (environment.hmr) {
  if (module[ 'hot' ]) {
    hmrBootstrap(module, bootstrap);
  } else {
    console.error('HMR is not enabled for webpack-dev-server!');
    console.log('Are you using the --hmr flag for ng serve?');
  }
} else {
    bootstrap().catch(err => console.log(err));
}

Commands used:
ng serve --hmr
ng serve --hmr --configuration hmr
ng serve --configuration hmr
The log given by the failure
In Terminal, I can see the app being watched and getting compiled on changes to the file. In browser, I get the following logged in console:
[WDS] App updated. Recompiling...
[WDS] App hot update...
But after that nothing happens. The app is kinda stuck on the App hot update.
Desired functionality
Components getting hot reloaded in the browser.
Mention any other details that might be useful
package.json
  "devDependencies": {
    "@angular-devkit/build-angular": "~0.6.8",
    "@angular/cli": "^6.0.8",
    "@angular/compiler-cli": "^6.0.4",
    "@angularclass/hmr": "^2.1.3",
    "@angularclass/hmr-loader": "^3.0.4",
    "@types/core-js": "^2.5.0",
    "@types/hammerjs": "^2.0.34",
    "@types/jasmine": "^2.8.8",
    "@types/jquery": "^3.3.2",
    "@types/node": "^10.1.4",
    "add-asset-html-webpack-plugin": "^2.1.3",
    "angular-router-loader": "^0.8.5",
    "angular2-template-loader": "^0.6.2",
    "copy-webpack-plugin": "^4.5.1",
    "css-loader": "^0.28.11",
    "extract-text-webpack-plugin": "^3.0.2",
    "file-loader": "^1.1.11",
    "html-webpack-plugin": "^3.2.0",
    "ts-helpers": "^1.1.2",
    "ts-node": "^6.1.0",
    "tslib": "^1.9.2",
    "tslint": "^5.10.0",
    "tslint-loader": "^3.6.0",
    "typescript": "^2.7.2",
    "webpack": "^4.12.0",
    "webpack-dev-server": "~3.1.4",
    "webpack-dll-bundles-plugin": "^1.0.0-beta.5",
    "webpack-merge": "^4.1.2"

Looks like this issue is happening only on my end?

I have the same problem.

I have same issue, seems the problem is that fileReplacements is not working.
If you change environment.ts and add hmr: true then everything works
To check it add a console.log('env:', environment); in main.ts

i got this problem too because i wanted to fix a dependency warning for webpack not explicitly installed for webpack-cli. so i added latest webpack and suddenly hmr was not working.
for now i decided to live with the warning and removed webpack from my devDeps cause angular-devkit seems to install it.
but would be great to hear about a solution for this
"devDependencies": {         "@angular-devkit/build-angular": "~0.6.8",         "@angular/cli": "^6.1.0",         "@angular/compiler-cli": "6.1.0",         "@angular/language-service": "6.1.0",         "@angularclass/hmr": "^2.1.3",         "@compodoc/compodoc": "^1.1.3",         "@types/d3": "^5.0.0",         "@types/hammerjs": "^2.0.35",         "@types/jasmine": "~2.8.6",         "@types/jasminewd2": "~2.0.3",         "@types/node": "~9.4.6",         "codelyzer": "~4.2.1",         "jasmine-core": "~3.1.0",         "jasmine-spec-reporter": "~4.2.1",         "karma": "~2.0.0",         "karma-chrome-launcher": "~2.2.0",         "karma-cli": "~1.0.1",         "karma-coverage-istanbul-reporter": "^1.4.1",         "karma-jasmine": "~1.1.1",         "karma-jasmine-html-reporter": "^0.2.2",         "nodemon": "^1.17.1",         "npm-run-all": "^4.1.2",         "protractor": "~5.3.0",         "ts-loader": "^4.3.0",         "ts-node": "~6.0.3",         "tslint": "~5.9.1",         "typescript": "2.7.2",         "webpack-bundle-analyzer": "^2.13.1",         "webpack-cli": "^3.0.3",         "webpack-node-externals": "^1.6.0"     }

The same issue with Angular CLI: 6.1.2 and
"@angular-devkit/build-angular": "~0.7.2",

Jep, same here.

Same here

Same here on Angular 8, with no additional Webpack dependency

@trumbitta 	I removed the webpack from devDependencies and works :)

@TheAifam5 as I wrote above, I don't have any Webpack dependency

@TheAifam5 as I wrote above, I don't have any Webpack dependency

@TheAifam5 Same here on Angular 8, with no additional Webpack dependency

Yup same Issue after updating from Angular 7 to 8

Guys, rather spamming that does not work, provide your configuration. The Author of this post have the webpack in the package.json which makes that issue.

As of my most recent tests I was able to use HMR on a plain CLI project, whereas the same configurations don't work in Nx 8 (they worked in Nx 7).
So for me the problem seem to be in Nx, not in @angular/cli.
But since @angular/cli uses Webpack, direct support in @angular/cli for HMR without third-party modules and all the configuration drama would be much appreciated.

Update on my end: it works.
I don't know what I exactly updated to make it  work, but I generally updated Nx and Angular to the latest versions.
Also, very important, remember to delete package-lock.json as well as node_modules and reinstall after every update.

Update on my end: it works.
I don't know what I exactly updated to make it work, but I generally updated Nx and Angular to the latest versions.
Also, very important, remember to delete package-lock.json as well as node_modules and reinstall after every update.

After upgrading from Angular 7 to 8, had the same issue. Removing node_modules and package-lock.json resolved my issue.

Closing as this seems to be solvable by removing node_modules and package-lock.json.
If the problem persists, please open a new issue, provide a simple repository reproducing the problem, and describe the difference between the expected and current behavior.


on ARM -t xfce,xfce-desktop gives network-manager error, doesn't complete install

will retry with just -t xfce
this is a new chroot
screenshot:

after the above, I did a sudo enter-chroot, and was prompted if I wanted to complete the install, typed Y, but gave a no uid 1000 error, tried to sudo su and adduser/useradd a user, but password lock file error.
deleted chroot, reinstalling now with just -t xfce

OK, so just -t xfce works fine, prompts for user and password for UID 1000, and startxfce4 works fine.
I'm not sure why the xfce,xfce-desktop didn't complete, but being on ARM/precise, perhaps the xorg message meant something, not sure about that suggestion for network-manager either, but just in case it was expecting wi-fi, I turned that on (main connection is USB to ethernet wired adapter).
After installing xfce, I installed these missing packages, most of which are on the ubuntu live cds by default, and some only take a few kb, or mb: nano, traceroute, dnstools, vim, emacs, nmon, atop, man, terminator, whois, and ssh. I realize some of those wouldn't be on a stock 12.04 live cd distro like terminator and nmon, but certainly ssh (client), traceroute, vim (full, not tiny), man, whois. I'm glad to see that gcc and cpp are in there now, they used to not be, or were installed, then removed.
I did a -d -f boot but how do I use the boot debbootstrap file for precise/xfce to reduce the next time I want to create a chroot?

the boot file didn't write, I think I need to give it a path to a writable dir:
I: Extracting tzdata...
I: Extracting udev...
I: Extracting upstart...
I: Extracting util-linux...
I: Extracting xz-utils...
I: Extracting zlib1g...
Compressing bootstrap files...
tar (child): boot: Cannot open: Read-only file system
tar (child): Error is not recoverable: exiting now
chronos@localhost

@tedm,
I don't think you need to specify both xfce and xfce-desktop, just one or the other but that's beside the point.
I ran into a similar situation on my Cr-48 with a trusty / gnome-desktop install where it at first failed due to 'network-manager' not being installed but, after entering the chroot and installing it, I re-ran the setup script and it completed.
If you're in your home directory or ~/Downloads or you provide a path to a rw folder, the bootstrap file shouldn't have any problem with permissions.
Once you have a bootstrap file downloaded, just use a similar install command but, instead of specifying the release with '-r', use '-f bootstrapfile' instead.
-DennisL

@DennisLfromGA Thanks! I will write the deboostrap files properly later tonight or tomorrow, and then specify them as you noted, to create a new test chroot.
I wonder if it would be a good idea to save them the first time to say ~/Downloads/Crouton/tmp and prompt the user just before prompting for username if they want to save them as a file so if/when they next create a chroot, it will go faster? And upon crouton creating a new chroot, it looks for a specific place and file to use an existing deboostrap, prompting, if necessary?

No, we generally try to avoid trashing the filesystem with large unnecessary files in the common use cases.

I have a number of them saved to an SD Card, that way I don't trash my system ;-)

I'm pretty sure that's what ~/Downloads is for ;) If all Crouton users stored these locally, we could probably increase the effective speed of the internet by 10x and not have these lags with Netflix, etc. :)

@DennisLfromGA It may be a little off topic, but how did you manage to "after entering the chroot and installing it, I re-ran the setup script and it completed"?
I encountered the same 'network-manager not being installed' problem when trying to install trusty + gnome-desktop on my samsung chromebook2. I can't find a way to enter the chroot first and then install network-manager as you said. Could you please tell me how to do this?

@zhao-gang,
When you run 'sudo enter-chroot [-n chrootname]', it will tell you a setup script still exists and asks if you want to finish the setup (Y/n/d)?
At this point, select 'n' for no and you'll be dropped into a chroot shell where you can then run 'sudo apt-get network-manager'.
Once that succeeds, you can exit the chroot and then either re-enter it and this time when it prompts to run the setup script hit 'Enter' or 'y' to let it finish - or - you can re-run your crouton command to install gnome-desktop.
The above procedure is from memory which may be flawed ;). If that doesn't work, please let me know and I'll try to help you out.
-DennisL

@DennisLfromGA I completed the installation process with your suggestion.
Unfortunately, sudo startgnome just showed me a black screen, the related error log is: /usr/bin/xinit: XFree86_VT property unexpectedly has 0 items instead of 1, maybe I should report this in another issue.

@zhao-gang - If you just re-entered the chroot via 'sudo enter-chroot ...' and finished the setup, then I would exit the chroot and do a crouton update (sudo sh ~/Download/crouton -u [-n chrootname]) to see if that fixes it. That works in most cases - for me at least.

@DennisLfromGA Would your above tips work on the ARM and the latest stable update to try targets xfce,xfce-desktop (or just xfce-desktop)? I don't mind the process above about manually installing network-manager, but I don't think it's a great idea to have network-manager in the chroot by default because of the possible conflicts, but I think that is noted elsewhere. My goal is to see if xfce-desktop brings in a lot of the packages that target xfce leaves off.

@ted,
I don't have an ARM Chromebook but the above procedure should work I believe.
I get around the errors/failure by first installing the vanilla target (e.g.) gnome, then entering the chroot via 'sudo enter-chroot' and running 'apt-get install network-manager', then exiting the chroot and installing the desktop target (e.g.) gnome-desktop. It's a little convoluted but it works in the interim.
The crouton gurus are aware of this https://github.com/dnschneid/crouton/issues/1408 and are feverishly ;) working on a fix via a 'fake' package.

@DennisLfromGA Thanks Dennis, yes, #1408 addresses this and I'll post over there, where I have some ARM xfce info, and keep the things specific to xfce-desktop here. So are we in agreement that ideally, when the network-manager and related script conflict issues (.xscreensaver?) are resolved, that a user should be able to specify multiple targets on a new chroot, and have the script complete, even if it's the base, and the *-desktop, like :
sh ~/Downloads/crouton -t xfce,xfce-desktop
and be good to go with a new precise chroot with xfce and whatever extras xfce-desktop add?
The hack you do above by telling the crouton script to not continue, entering chroot, apt-get install network-manager, exit, enter-chroot, answer Yes to continue, is the ultimate definition of "hack" ;)

@tedm,
Yes, I believe they are striving for a fix that would obviate the 'hack'. ;)
Keep in mind that when you specify the full desktop targets (e.g.) xfce-desktop, there's no need to specify the xfce target too since the full desktop targets pull in the vanilla targets.

@DennisLfromGA  @tedm Just for your information. On my ARM chromebook(which is a samsung chromebook2), ubuntu trusty + xfce works, and there's no need to manually install network-manager or other packages. But trusty + gnome still doesn't work, even after I updated the chroot.

@zhao-gang -
When you say 'trusty + xfce' works & 'trusty + gnome' doesn't work, are you talking about the non-desktop targets or 'xfce-desktop' and 'gnome-desktop'?
If they are the no frills non-desktop targets, then that is indeed a problem and may necesitate a new issue.
If they are the meta-package desktop targets, then they may not completely install until the fix (https://github.com/dnschneid/crouton/issues/1408) is in place.

@DennisLfromGA Yeah, I think I am a little misleading. When I refer to xfce, I mean xfce, but when I refer to gnome, I mean gnome-desktop. So I think the problem just happens in -desktop target. Thanks for your efforts on this problem.

@zhao-gang - No problem, I think this'll get figured out pretty soon.

Merging into #1408


expired credentials

Admin:~/environment $ pcluster update -c al2-config.ini hpc-cluster-al6
Retrieving configuration from CloudFormation for cluster hpc-cluster-al6...
ERROR: Failed when retrieving cluster config version from DynamoDB with error An error occurred (UnrecognizedClientException) when calling the GetItem operation: The security token included in the request is invalid

Note that ~/.aws/credentials exists on Cloud9 and contains:
# Do not modify this file, if this file is modified it will not be updated. If the file is deleted, it will be recreated on Mon Oct 05 2020 23:48:15 GMT+0000 (Coordinated Universal Time).
# [...] #

[default]
aws_access_key_id=[...]
aws_secret_access_key=[...]
aws_session_token=[...]
region=us-east-1

We've seen this before, Cloud9 creds are short lived (5 mins) and it looks like you got unlucky on this one. We should probably setup an instance profile.
We did this in the labs to solve this issue, see https://www.hpcworkshops.com/05-aws-batch/00-environment.html

Cloud9 refreshes the credentials and access is retained. Long running tasks using the API are the only thing impacted. Aside from pcluster status, I have not encountered other issues.


Upgrade to Gradle 6 with minimum required versions of plugins

Coverage increased (+0.04%) to 92.667% when pulling a9479a87c4e86b83473afcece8455618afd1f6d1 on chali:Gradle6 into 4b3ce28c0c09cb963978f1e2dffc8c596fe78c40 on Netflix:master.

@tgianos npm-shrinkwrap.json has now its original content.


enhance: lsp hover  lit str attr

->

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y #1244

2. What is the scope of this PR (e.g. component or file name):
kclvm/sema/src/ty/mod.rs

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[x] Other

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

5 of 6 (83.33%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.001%) to 71.213%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

kclvm/tools/src/LSP/src/hover.rs
5
6
83.33%

Totals

Change from base Build<PHONE_NUMBER>:
0.001%

Covered Lines:
54319

Relevant Lines:
76277

💛  - Coveralls


TCMalloc crashes on Windows 11 when opening a file dialog

Here's a crash that consistently reproduces when the following conditions are met

The host machine is running Windows 11 with a build number between [10.0.22000.0, 10.0.22621.900)
The user has OneDrive Desktop syncing enabled
The application which is TCMalloc enabled tries to open a file dialog

Here's a full breakdown by TBBMalloc contributors.
Separate discussion for mimalloc
The TLDR is there is a malloc/HeapFree mismatch in the Windows explorer DLL.
This issue affects TCMalloc because while we replace calls to malloc, for HeapFree we just do a pass-through for instrumentation.
Here's a workaround that should work for TCMalloc https://github.com/lennoxho/gperftools/commit/18171db63b79f94ec15511109c9e813fe2551702
While MSFT has since issued a fix, Windows 11 10.0.22000 aka 21H2 won't be EOL until October 2024.
With that in mind, do you think this is still worth addressing in TCMalloc?

Added a brief comment and referenced this issue.
Created PR https://github.com/gperftools/gperftools/pull/1491


Snippet that creates a recurring event

Added a new snippet that will create a calendar event that occurs every Tuesday and Thursday from 1PM to 2PM.

@davechmsft I'd be happy to take a look at this. Sound good, @Ricalo?

Please do...

Passing back to @davechmsft to check my amends

@iambmelt the formatting changes look good.


Add :match() selector

This selector matches based on regex against text content of subtree. Note that upstream would not like this, probably, since it's lxml etree specific.

sorry about this, was supposed to be against my fork. Closing

Indeed, I’d rather not have non-standard selectors in this library. But there could be a mechanism for users to support their own pseudo-classes.


Instance: Fix a couple of file descriptor leaks in LXC driver

And improve logging.

jenkins: test this please

jenkins: test this please

Closing as all commits were merged through other PRs.


Resolves remaining suggestions

Links and the figcaptions

@brianebeling

This PR was a bit difficult to work with because the branch name brifemu/science-blog-6 is the same as the one used in https://github.com/cuisines/cwa-website/tree/brifemu/science-blog-6. It's better to have a different name e.g. science-blog-6-2 to allow both branches to coexist without confusion in the same clone. (I use e.g. git remote add cuisines to have them side-by-side instead of having a complete separate clone of each fork.)

Also I tripped up on the master branch of cuisines and your clone being out of date. npm ci fails on the master branch with the current Node.js 18 LTS version. It would be a good idea (but not essential for this PR) for that branch to get updated. https://github.com/corona-warn-app/cwa-website/blob/master/README.md#development specifies using Node.js 18 LTS. In addition there are a lot of old branches in the cuisines fork. Perhaps they could be removed?

The link [Datenspende der Corona-Warn-App](../2021-10-15-science-blog-4/#3-privacy-preserving-analytics) is still broken. One way to solve it is just to remove ../.

Fixed, thanks!

Brian-J. Ebeling

This PR was a bit difficult to work with because the branch name brifemu/science-blog-6 is the same as the one used in brifemu/science-blog-6. It's better to have a different name e.g. science-blog-6-2 to allow both branches to coexist without confusion in the same clone. (I use e.g. git remote add cuisines to have them side-by-side instead of having a complete separate clone of each fork.)

Ok. I will try to work with an appended number to the name next time. Usually I only work with branches and was admittedly a bit
clueless as how to work with the fork correctly, so I treated it more like a "new" repository. Adding another remote is a painfully obvious solution, I didn't consider. Thanks!

Brian-J. Ebeling

Also I tripped up on the master branch of cuisines and your clone being out of date. npm ci fails on the master branch with the current Node.js 18 LTS version. It would be a good idea (but not essential for this PR) for that branch to get updated<EMAIL_ADDRESS>specifies using Node.js 18 LTS. In addition there are a lot of old branches in the cuisines fork. Perhaps they could be removed?

I will add it to the backlog, unfortunately we got more important issues right now and I want to avoid working with forks. Using branches in the main repo feels much more efficient. Nonetheless this is a good suggestion.

@brianebeling

I want to avoid working with forks.

I fully agree. It would be great if all contributors employed directly by the project i.e. SAP and any partners, subcontractors, paid freelancers. etc. worked directly without forks. For unpaid voluntary community members (like me) probably there is no alternative to using forks.

Brian-J. Ebeling

I want to avoid working with forks.

I fully agree. It would be great if all contributors employed directly by the project i.e. SAP and any partners, subcontractors, paid freelancers. etc. worked directly without forks. For unpaid voluntary community members (like me) probably there is no alternative to using forks.

Speaking of voluntary contributions, I just wanted to say thank you. You are a great great help with this whole project. 😄


Enable fwd and varlen_fwd on AMD

Enable flash_attn_func using amd's triton flash attention kernel.
This enables flash_attn_func. There will be follow up prs to enable the forward pass of the other flash attention apis.
we skip building the cuda kernels and use the triton kernel on AMD. This is fine for our fork but upstream will probably require a way to have one wheel.

@vgokhale Can I merge this?


feat(aws-eventbridge-kinesisstrems, aws-events-rule-kinesisstreams, aws-eventbridge-kinesisfirehose-s3, aws-events-rule-kinesisfirehose-s3): support for custom EventBus

Issue #, if available:
Description of changes:
Following constructs are enhanced to support custom EventBus

aws-eventbridge-kinesisstrems
aws-events-rule-kinesisstreams
aws-eventbridge-kinesisfirehose-s3
aws-events-rule-kinesisfirehose-s3

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

AWS CodeBuild CI Report

CodeBuild project: codebuildgithubautobuildPro-fkVQbXRiQi6A
Commit ID: 74521b13e015b708f74e85bde44ff11ae3e18abb
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: codebuildgithubautobuildPro-fkVQbXRiQi6A
Commit ID: 0947e7b0835a3067c1880c7e86952dc5be0c4871
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: codebuildgithubautobuildPro-fkVQbXRiQi6A
Commit ID:<PHONE_NUMBER>c502af4a4831761fa963c9c937d826
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Removed mvnw

mvnw is not functional in this directory, and since newer mavens will delegate to mvnw if present, this is even more harmful.
The simplest solution is to remove it or, alternatively, add a complete distribution of mvnw.

Thank you Roland! Good catch. We will discuss this update at our next team meeting next week. Thanks!  Scott

Thanks again! The team appreciates your help. We are merging your contributions.


github.com%2Fajstarks%2Fdeck/generate/v0.0.0-20210309230005-c3f852c02e19

Type: Incorrect
Summary:
github.com%2Fajstarks%2Fdeck/generate/v0.0.0-20210309230005-c3f852c02e19
Details:
Add CC-BY-4.0
Resolution:
https://github.com/ajstarks/deck/blob/c3f852c02e19235a0b00a692c533fc1c4a4852ca/LICENSE
Affected definitions:

generate v0.0.0-20210309230005-c3f852c02e19

Agree with CC-BY-4.0. 1.	That is the top level license and there's a closed issue from 2014 where the author states the license is “CC by attribution.”


Suggestion: Support setting proxy

You need to add a proxy to access the internal and external networks of the enterprise. It is recommended that the client either supports setting up the proxy by itself, or synchronizes with the system proxy. Now the client cannot download the driver after running it.

Duplicate of #1401


[Web] App violates Content Security Policy

Steps to Reproduce

Create a template app with flutter create bug.
flutter build web --csp
Run built app on a server with SSL/TLS configured

Expected results:
App to run normally.
Actual results:
Browser throws error

Uncaught EvalError: Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "default-src 'self' http: https: data: blob: 'unsafe-inline'".

The inline string indicated is
"try {return this===window;}catch(e){ return false;}"

Note
Building with the flutter build web --csp --web-renderer html mitigates this issue.

flutter doctor
[✓] Flutter (Channel beta, 2.1.0-12.2.pre, on macOS 11.2.3 20D91 darwin-x64, locale en-GB)
    • Flutter version 2.1.0-12.2.pre at /Users/ytt/Library/Flutter
    • Framework revision 5bedb7b1d5 (4 weeks ago), 2021-03-17 17:06:30 -0700
    • Engine revision 711ab3fda0
    • Dart version 2.13.0 (build 2.13.0-116.0.dev)

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/ytt/Library/Android/sdk
    • Platform android-29, build-tools 29.0.2
    • ANDROID_HOME = /Users/ytt/Library/Android/sdk
    • Java binary at: /Users/ytt/Library/Application
      Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7188722/Android Studio 4.2
      Preview.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to
        your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To install see https://guides.cocoapods.org/using/getting-started.html#installation for
      instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio
    • Android Studio at /Users/ytt/Library/Application
      Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7188722/Android Studio 4.2
      Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Ultimate Edition (version 2020.3.3)
    • IntelliJ at /Users/ytt/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.53.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 89.0.4389.114

! Doctor found issues in 2 categories.

Hi @alexttyip
Can you please upgrade to the latest  master and try again
flutter channel master
flutter upgrade --force
flutter doctor -v

Hi @TahaTesser
Completed upgrade and built with flutter build web --csp, no change.

flutter doctor
[✓] Flutter (Channel master, 2.1.0-13.0.pre.574, on macOS 11.2.3 20D91 darwin-x64, locale en-GB)
    • Flutter version 2.1.0-13.0.pre.574 at /Users/ytt/Library/Flutter
    • Framework revision 02efffc134 (2 days ago), 2021-04-10 03:49:01 -0400
    • Engine revision 8863afff16
    • Dart version 2.13.0 (build 2.13.0-222.0.dev)

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/ytt/Library/Android/sdk
    • Platform android-29, build-tools 29.0.2
    • ANDROID_HOME = /Users/ytt/Library/Android/sdk
    • Java binary at: /Users/ytt/Library/Application
      Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7188722/Android Studio 4.2
      Preview.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to
        your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To install see https://guides.cocoapods.org/using/getting-started.html#installation for
      instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio
    • Android Studio at /Users/ytt/Library/Application
      Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7188722/Android Studio 4.2
      Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Ultimate Edition (version 2020.3.3)
    • IntelliJ at /Users/ytt/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.53.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 89.0.4389.114

! Doctor found issues in 2 categories.

May be related to https://github.com/emscripten-core/emscripten/issues/12910

/cc @kjlubick this may require a change in CanvasKit.

That's a little snippet of code to determine if we are on nodejs or not.
https://github.com/google/skia/blob/375e1f6a6486a1e423f61d221bc39d81a2aaf6a0/modules/canvaskit/helper.js#L212
I'll bet we can find an alternative.

Or, delete that function, its only caller saveBytesToFile and replace its only caller Picture.saveAsFile() to return a Uint8Array and let the client deal with making a file.
I like that approach.

Should be fixed in https://skia-review.googlesource.com/c/skia/+/397289 and will roll out in CanvasKit 0.26.0

AFAIK CanvasKit 0.26.0 rolled out last week and Flutter 2.2.0-10.2.pre is still using CanvasKit 0.25.1. Is there a timeline as to when Flutter will upgrade its CanvasKit version?
Cheers.

Latest master grabs CanvasKit 0.27 but it still errors out with unsafe-eval. Curious

@danReynolds Do you know if it's the same bit of code that's being eval'd or is it something else?

I can confirm it when hosting firebase using html renderer too
triage-79c7c.web.app/:33 Refused to execute inline script because it violates the following Content Security Policy directive: "script-src 'self'". Either the 'unsafe-inline' keyword, a hash ('sha256-VgQf9Uorw0AD0VsOCdKg+JAbJGp880Hay+VmShtH6RM='), or a nonce ('nonce-...') is required to enable inline execution.

stable
master

❌
❌

Check flutter doctor -v outputs for each channel below

flutter doctor -v
[✓] Flutter (Channel stable, 2.2.3, on Microsoft Windows [Version 10.0.22000.71], locale en-US)
    • Flutter version 2.2.3 at C:\Users\Taha\Code\flutter_stable
    • Framework revision f4abaa0735 (3 weeks ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\Taha\Code\android-sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_SDK_ROOT = C:\Users\Taha\Code\android-sdk
    • Java binary at: C:\Users\Taha\Code\android-studio-4.1\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.10.3)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.10.31424.327
    • Windows 10 SDK version 10.0.19041.0

[✓] Android Studio (version 4.1.0)
    • Android Studio at C:\Users\Taha\Code\android-studio-4.1
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[✓] IntelliJ IDEA Community Edition (version 2021.1)
    • IntelliJ at C:\Users\Taha\Code\intellij
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.58.2)
    • VS Code at C:\Users\Taha\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.24.0

[✓] Connected device (4 available)
    • Redmi K20 Pro (mobile) • def0ad20 • android-arm64  • Android 11 (API 30)
    • Windows (desktop)      • windows  • windows-x64    • Microsoft Windows [Version 10.0.22000.71]
    • Chrome (web)           • chrome   • web-javascript • Google Chrome 91.0.4472.124
    • Edge (web)             • edge     • web-javascript • Microsoft Edge 91.0.864.67

• No issues found!

[✓] Flutter (Channel master, 2.4.0-5.0.pre.87, on Microsoft Windows [Version 10.0.22000.71], locale en-US)
    • Flutter version 2.4.0-5.0.pre.87 at C:\Users\Taha\Code\flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision e084e9d946 (2 days ago), 2021-07-17 12:00:21 -0700
    • Engine revision 14a9e9a50e
    • Dart version 2.14.0 (build 2.14.0-321.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\Taha\Code\android-sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_SDK_ROOT = C:\Users\Taha\Code\android-sdk
    • Java binary at: C:\Users\Taha\Code\android-studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.10.3)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.10.31424.327
    • Windows 10 SDK version 10.0.19041.0

[✓] Android Studio (version 4.1.0)
    • Android Studio at C:\Users\Taha\Code\android-studio-4.1
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[✓] Android Studio (version 4.2.0)
    • Android Studio at C:\Users\Taha\Code\android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)

[✓] IntelliJ IDEA Community Edition (version 2021.1)
    • IntelliJ at C:\Users\Taha\Code\intellij
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.58.2)
    • VS Code at C:\Users\Taha\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.24.0

[✓] Connected device (5 available)
    • Redmi K20 Pro (mobile)  • def0ad20 • android-arm64   • Android 11 (API 30)
    • Windows (desktop)       • windows  • windows-x64     • Microsoft Windows [Version 10.0.22000.71]
    • Windows (UWP) (desktop) • winuwp   • windows-uwp-x64 •
    • Chrome (web)            • chrome   • web-javascript  • Google Chrome 91.0.4472.124
    • Edge (web)              • edge     • web-javascript  • Microsoft Edge 91.0.864.67

• No issues found!

✅ : No Issue ❌: Issue reproduced

With the HTML renderer style-src is also an issue: Refused to apply inline style because it violates the following Content Security Policy directive: "style-src 'self'". See #86553 for details.

https://bugs.chromium.org/p/chromium/issues/detail?id=946838
=>
https://bugs.chromium.org/p/chromium/issues/detail?id=948834
=>
https://github.com/w3c/webappsec-csp/pull/293
Since this ends in a W3C appsec bug, this might not be fixed for a while.

@kjlubick This is specific only to the canvaskit renderer right?

Right, I believe only the CanvasKit renderer uses WebAssembly

Maybe it's better to split the CSP issues between CanvasKit and the HTML renderer into separate issues. And make this issue an epic for the functional case (ie CSP does not work). Because they are from a technical perspective completely different problems. The HTML case may be simpeler to fix for example because more code is owned by the Flutter team?
I originally made an issue for the HTML renderer and CSP (#86553 ) but it was closed and html labels were added to this issue to make it a general issue for CSP related stuff. Not sure if that is the best way to go though.

Steps to Reproduce

Create a template app with flutter create bug.
flutter build web --csp
Run built app on a server with SSL/TLS configured

Expected results:
App to run normally.
Actual results:
Browser throws error

Uncaught EvalError: Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "default-src 'self' http: https: data: blob: 'unsafe-inline'".

The inline string indicated is
"try {return this===window;}catch(e){ return false;}"

Note
Building with the flutter build web --csp --web-renderer html mitigates this issue.
flutter doctor

The web-renderer has issues with style-src: self:

This is the code causing the issue:

When building with --profile the CSP error is caused somewhere in file html_dart2js.dart

Any progress on this? We really want to transition to Flutter but this is really a blocking issue as we can not go back to a less secure CSP for our application.

Dynamic function creation has been removed from CanvasKit. However, we haven't yet thought of CSP support holistically. For example, the --csp option only applies to the JS output produced by dart2js, but now that dart2js outputs CSP code by default that option has no effect (we should remove it). There could be other things that violate CSP though. To make this work well, we'll need a test harness that enforces CSP and a suite of test cases that run in it.

@yjbanov Is there an eta on this or any indication? Sounds like a very long term plan

I'm having this issue as well. I was able to move inline JavaScript to its own file and remove 'unsafe-inline' for 'script-src', but have to set the following in my CSP to get it to work:
script-src: 'unsafe-eval'; style-src: 'unsafe-inline';
script-src: 'unsafe-eval' is needed for Canvaskit Wasm to load, and style-src: 'unsafe-inline' is cause the CSS is being minified and inlined during the build step.
Using flutter build web --csp --renderer html doesn't fix it for me.

@fahali : Hello, how did you manage to get flutter build to alter the line
// The value below is injected by flutter build, do not touch.
var serviceWorkerVersion = null;

in it's own file ? At least flutter build should alter this line in another file like main.js (the file containing the initialization script)...

Exactly what I needed. Thanks a lot.
Yet, it should be in the default web build. It is not normal that flutter builds a non CSP compliant web app.

Have an issue with Firebase and --csp flag not stopping it from injecting a script DOM.
I am building for chrome extension, forced to use CSP and can't get around it.

This is the code causing the issue:

Have nearly identical build.
Am looking into manually deleting these lines and moving them into a script hardcoded :( :(
Would greatly appreciate a fix!

@ditman – are we better here now?

We modified some stuff to work with TrustedTypes, but TIL about --csp

@passsy thanks for the pointer! I've seen the same recommendation made when people wanted their ManifestV3 Chrome Extension to be able to use the firebase JS libs (I guess ManifestV3 is very stringent in what JS is allowed into an extension)

@passsy could you explain to a beginner how to do this? i'm trying to solve this error:
Refused to load the script<EMAIL_ADDRESS>because it violates the following Content Security Policy directive: "script-src 'self'". Note that 'script-src-elem' was not explicitly set, so 'script-src' is used as a fallback.

If you're willing to wrap your Flutter app in a docker container it's easiest to use dockerize_sidekick_plugin which handles most CPS rules for you.

@passsy Will docker work for a chrome extension?
also the issue seems to be: https://github.com/Dart-Code/Dart-Code/blob/bca09132c5f8afd58c740f652edf8c0ef3b8fc79/src/extension/sdk/dev_tools/embedded_view.ts
this.panel.webview.html =

;
This line:
<meta http-equiv="Content-Security-Policy" content="default-src 'self' 'nonce-${scriptNonce}' 'nonce-${cssNonce}' http://${vs.Uri.parse(devToolsUri).authority};">
introduces all the issues.
Might there be a fix for this because, Chrome extensions no longer accept these CSP directives.

However you implement it, you need to set CSP headers on the server side. It might be in a docker container or with the tools your static hoster provides (e.g. firebase static hosting).
Chrome Extensions are unique because they no longer allow using inline scripts, meaning you can't use a nonce. But you can extract the inline script into a separate file and load that.
This doesn't work:
<html>
<head>
  <script nonce="sha256-asdfasdf">
   var serviceWorkerVersion = '4242841849';
  </script>
</head>
...
</html>

This works:
<html>
<head>
  <script src="init_flutter.js"></script>
</head>
...
</html>

// init_flutter.js

var serviceWorkerVersion = '4242841849';

@passsy im a total beginner but can you explain exactly how i would do that?

Any update regarding this issue?

yes. The issue is caused by firebase. If your chrome extension can do without firebase then removing it solves your issues. As to a proper fix, I dont know.

Did anyone figure this out?

Is it fair to assume that the use case that triggers this issue mainly is "attempting to use Flutter Web to develop a Chrome Extension with manifest v3", or are there more use cases that I've missed?

@ditman

Is it fair to assume that the use case that triggers this issue mainly is "attempting to use Flutter Web to develop a Chrome Extension with manifest v3", or are there more use cases that I've missed?

Our case is "attempting to use Flutter Web to develop a web application hosted on a server with strict CSP, accessed with Edge, Chrome and Firefox". See our original issue https://github.com/flutter/flutter/issues/86553 which was merged into this issue and then closed.

Our case is "attempting to use Flutter Web to develop a web application hosted on a server with strict CSP, accessed with Edge, Chrome and Firefox". See our original issue #86553 which was merged into this issue and then closed

Me too, this is a blocker in production sites because it forces to use html web renderer instead of canvaskit, which is the one that gives us better results and better performance. Solving this is a must in order to create and publish professional Flutter Web apps

The solution is there, as described in https://github.com/firebase/flutterfire/discussions/9222
You need to bundle firebase yourself with webpack.

The solution is there, as described in firebase/flutterfire#9222

@passsy Please check the other scenario (https://github.com/flutter/flutter/issues/80221#issuecomment-1564062744). We don't use Firebase in our context at all.

Right, just tested it. canvaskit still required the use of unsafe-eval. I raised a new issue for this https://github.com/flutter/flutter/issues/127658

@henry2man @passsy (more details in #127658, but TL;DR): When enabling CSP you need the 'wasm-unsafe-eval' source directive for WASM to work.

@codemasterover9000 more details in #86553, but TL;DR: Your CSP configuration for certain directives is too strict and that breaks the way Flutter Web works. I've proposed a few things that you could do to relax your CSP, maintain a level of extra security, but allow Flutter Web to run. I hope we can find some middle ground there!

For everybody else on the thread; I've tested this a little bit, and I don't think that any of the Flutter dependencies are "breaking CSP" as described above anymore.
With the way Flutter Web currently works, we require some CSP directives to have values other than 'self', but in practice there's not much we can do about it.
Here's an (annotated) minimal policy that allows the Flutter example app work:
script-src 'self' 'nonce-flutter-init' 'wasm-unsafe-eval' https://www.gstatic.com/flutter-canvaskit/; font-src https://fonts.gstatic.com; style-src 'unsafe-inline';

If we pick it apart, it reads like this:

script-src: specifies valid sources for JavaScript.

'self': the domain in which the app is running from. This is needed to load flutter.js, for example.
'nonce-flutter-init': a "nonce" for the inline scripts in the index.html, (add nonce="flutter-init" to them!) [^1]
'wasm-unsafe-eval': enables WASM. [^2]
https://www.gstatic.com/flutter-canvaskit/ : The source of canvaskit.js and .wasm from the CDN [^3]

font-src: specifies valid sources for fonts loaded using @font-face.

https://fonts.gstatic.com : The source of fallback fonts for Flutter.

style-src: valid sources for stylesheets.

'unsafe-inline' : Flutter needs to set some DOM stylesheets inline to operate correctly.

For slightly more complex apps (for example one that used images from unsplash.com, or the Google Maps JS SDK or videos from youtube...) the above configuration is already too strict, and the CSP would need to be relaxed even further.
[^1]: the flutter-init part should ideally be randomly generated by the server. Read more about nonce. Alternatively, 'unsafe-inline' can be used, or move the initialization scripts to a separate JS file.
[^2]: Only needed if using --web-renderer canvaskit. See #127658.
[^3]: Optional, this value may change if the canvasKitBaseUrl configuration is set when initializing flutter. Initializing the engine.

Should we keep this issue open?
(In case this stays open, and for future new reports, please include the CSP that you're attempting to set on your Flutter Web app, in addition to the reproduction code, as @codemasterover9000 did in #86553. There isn't a "singular CSP"; CSP has a ton of configuration options that restrict standard browser behavior in many different ways.)

Having to use unsafe- settings permanently defeats the purpose of CSP. These only exist in the standard to allow an insecure site to function while gradually improving code and styling until the unsafe settings aren't needed anymore.

I'm not sure that we can remove all the inline styling that we do, but something that we can do is to allow programmers to pass a nonce value into flutter through the initialization procedure, so we can "sign" all of flutter's inline stuff with it.
Something like:
  _flutter.loader.loadEntrypoint({
    ...
    onEntrypointLoaded: async function(engineInitializer) {
      // Initialize the Flutter engine
      let appRunner = await engineInitializer.initializeEngine({
        nonce: 'Some_Random_Nonce_Value_Computed_From_The_Server',
      });
      // Run the app
      await appRunner.runApp();
    }

font-src: specifies valid sources for fonts loaded using @font-face.

https://fonts.gstatic.com : The source of fallback fonts for Flutter.

Note that this requirement can be prevented by using work-arounds like https://github.com/flutter/flutter/issues/77580#issuecomment-1112333700.

For me, After I met this issue. I asked my devops and he told me that the path to flutter.js was wrong.
The sever requirement is https://domain/path_to_flutterjs/flutter.js
Currently it is: https://domain/flutter.js
After some research, I change my base path inside html to " and it works
Or you can modify the command build the web to flutter build web --base-href "/path/"
Hope this work for you guys


Updated slogan on system.site.yml file like doc

Following this guide https://pantheon.io/docs/guides/build-tools/pr-workflow/ to make pull request and create a new branch.

Trying to remove pantheoncse multidev as per this guide:


rmcdecouverte.bfmtv.com extractor is broken

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.12.17. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.12.17

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
youtube-dl -v https://rmcdecouverte.bfmtv.com/le-mont-saint-michel/program_3587/
[debug] System config: ['--prefer-free-formats']
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://rmcdecouverte.bfmtv.com/le-mont-saint-michel/program_3587/']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.12.17
[debug] Python version 3.7.1 (CPython) - Linux-4.19.8-300.fc29.x86_64-x86_64-with-fedora-29-Twenty_Nine
[debug] exe versions: ffmpeg 4.0.3, ffprobe 4.0.3
[debug] Proxy map: {}
[generic] program_3587: Requesting header
WARNING: Falling back on generic information extractor.
[generic] program_3587: Downloading webpage
[generic] program_3587: Extracting information
ERROR: Unsupported URL: https://rmcdecouverte.bfmtv.com/le-mont-saint-michel/program_3587/
Traceback (most recent call last):
  File "/usr/lib/python3.7/site-packages/youtube_dl/YoutubeDL.py", line 791, in extract_info
    ie_result = ie.extract(url)
  File "/usr/lib/python3.7/site-packages/youtube_dl/extractor/common.py", line 508, in extract
    ie_result = self._real_extract(url)
  File "/usr/lib/python3.7/site-packages/youtube_dl/extractor/generic.py", line 3298, in _real_extract
    raise UnsupportedError(url)
youtube_dl.utils.UnsupportedError: Unsupported URL: https://rmcdecouverte.bfmtv.com/le-mont-saint-michel/program_3587/

Description of your issue, suggested solution and other information
The RMC Découverte website was redesigned a few days ago and since then, the extractor no longer works.

I confirm the problem and i've found a way to download the file.
Display the source code and serach for data-video-id. You will find data-video-id="5980028585001"
The<PHONE_NUMBER>001 is the video ID on the brightcove platform so if you use the brighcove url it will works.
youtube-dl http://players.brightcove.net/1969646226001/default_default/index.html?videoId=5980028585001
The rmcdecouverte.py extractor has to be rewrited. I've tried to have a look but it's a complicated framework to learn for me and I miss times.
I'm pretty shure that the _VALID_URL variable has to change, and youtube-dl can recognize the site.

This is not a fix of youtube-dl but meanwhile javascript can easily grab the video id and generate the url automatically using @moimadmax method (mentioned here also #24862 ).
Here is a simple script you can copy in the javascript console (F12 > console in firefox) to get the url and youtube-dl command to get your video (check my project here for a js bookmarklet and shell implementation to use directly in the shell until youtubedl gets fixed)

	let iddecomptea = document.querySelectorAll('[data-account]')[0];
	if (iddecomptea === undefined){
        var iddecompte = document.querySelectorAll('[accountid]')[0].getAttribute("accountid");
    } else {
        var iddecompte = iddecomptea.getAttribute("data-account");
    }
    let iddevideo = document
        .querySelectorAll('[data-video-id]')[0]
        .getAttribute("data-video-id");
    let urldevideo = "http://players.brightcove.net/" + iddecompte + "/default_default/index.html?videoId=" + iddevideo;
    alert('youtube-dl --write-info-json ' + urldevideo + '\r\n\r\nURL:\r' + urldevideo);

This is not a fix of youtube-dl but meanwhile javascript can easily grab the video id and generate the url automatically using @moimadmax method (mentioned here also #24862 ).
Here is a simple script you can copy in the javascript console (F12 > console in firefox) to get the url and youtube-dl command to get your video (check my project here for a js bookmarklet and shell implementation to use directly in the shell until youtubedl gets fixed)

	let iddecomptea = document.querySelectorAll('[data-account]')[0];
	if (iddecomptea === undefined){
        var iddecompte = document.querySelectorAll('[accountid]')[0].getAttribute("accountid");
    } else {
        var iddecompte = iddecomptea.getAttribute("data-account");
    }
    let iddevideo = document
        .querySelectorAll('[data-video-id]')[0]
        .getAttribute("data-video-id");
    let urldevideo = "http://players.brightcove.net/" + iddecompte + "/default_default/index.html?videoId=" + iddevideo;
    alert('youtube-dl --write-info-json ' + urldevideo + '\r\n\r\nURL:\r' + urldevideo);


[front] feature: Use dust-app for parsing table headers

fixes: https://github.com/dust-tt/dust/issues/6782
Description

Risk

Deploy Plan

Just to make sure I undertsand. This would be called on all table upserts right?

Let's start by running the app without awaiting it, then compare outputs and log differences + latency to introspect results as a starter. Maybe behind a flag.
If results look encouraging, cost is confirmed and latency is OK we can move forward from here :+1:

Let's start by running the app without awaiting it, then compare outputs and log differences + latency to introspect results as a starter. Maybe behind a flag.
If results look encouraging, cost is confirmed and latency is OK we can move forward from here 👍

Ok, will continue on the pr and do more tests


add user subodh101 as ci viewer

What?
As per the title.
Why?
For the ci debugging purposes.

/assign @jlewi

/ok-to-test

You might as well go ahead and join the GitHub org in this PR as well so you can trigger tests.

Thanks! made the changes.

Let's get this in if everything looks good.

/lgtm
/approve
apologies for the delay

Note the group sync is not automatic. It will probably be a couple days as I'm working on automating the process.


chore(pkg-cleanup): update dev packages, better feedback in a11y audit

This PR upgrades some of the development dependencies which solve some of the errors reported when performing a fresh install.
Taking us from here:

To here:

This also makes a small tweak to the a11y script so that the happy path of the audit still reports the current and threshold number of errors, for example;

Instead of only showing this information when the audit fails. This should encourage people to decrease the threshold in the config when possible.

Deploy preview for pf-next ready!
Built with commit 6871dc59b2bd005b0919ed0b1b47258b3efac305
https://deploy-preview-1677--pf-next.netlify.com

:tada: This PR is included in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Port asyncfuture to use Qt 6.3

Now that Qt 6.3 supports whenAll (https://doc-snapshots.qt.io/qt6-6.3/qtfuture.html#whenAll) we should be able to port asyncfuture's backend to use Qt 6.3 API. That's probably performance benefits since Qt QFuture api doesn't use QObject to combine futures.

This has been ported on https://github.com/vpicaver/asyncfuture


Break a blocky Game!

Whenever I break a block its crashes!

maybe you have a mod activated that has function destroyBlock on it

Which scripts, which version of MCPE, which device, which texture pack?

0.13.1 :P