Rotated textures for spine atlases are wrong in both webgl and canvas

That issue is not related to pixi-spine, but to atlases spine is generating. Users who bought Spine dont have to buy TexturePacker because spine already has its features.
Thread at html5gamedevs
JsFiddle
Original images:

Packed images, first and second are rotated:

UVs in webgl are wrong

Both UVs and anchors in canvas are wrong

Classes:

TextureUVs

Sprite

Is that possible than spine atlas "rotate" differs from TexturePacker rotate?

Hmm.. looks like it may be the case?

However, TextureUVs and Sprite implementations are contradicting: TextureUVs use (frame.width, frame.height) while canvas uses (frame.height,frame.width) for dimensions. Im sure that there's a bug somewhere.

Cool, I did just push a fix for canvas rotated textures earlier this week. WebGL texture uvs are working ok though as far as i am aware? Let me know if you find anything!

Ok i see it: https://github.com/pixijs/pixi.js/commit/72b155ca34bbde4620b22ea3628ee0ca73e7a33f
But you didnt fix it in textureUVs.
TextureUvs.prototype.set = function (frame, baseFrame, rotate)
{
    var tw = baseFrame.width;
    var th = baseFrame.height;

    if(rotate)
    {
            this.x0 = frame.x / tw;
            this.y0 = (frame.y + frame.width) / th;

            this.x1 = frame.x / tw;
            this.y1 = frame.y / th;

            this.x2 = (frame.x + frame.height) / tw;
            this.y2 = frame.y / th;

            this.x3 = (frame.x + frame.height) / tw;
            this.y3 = (frame.y + frame.width) / th;
    }
    else
    {

        this.x0 = frame.x / tw;
        this.y0 = frame.y / th;

        this.x1 = (frame.x + frame.width) / tw;
        this.y1 = frame.y / th;

        this.x2 = (frame.x + frame.width) / tw;
        this.y2 = (frame.y + frame.height) / th;

        this.x3 = frame.x / tw;
        this.y3 = (frame.y + frame.height) / th;
    }
};

I see! The current version works well with texture packer as far as i am aware? do we have a texture packer example to test?

Yeah, now I see it, your version works, I was wrong. But for spine atlas we need to make new rotate value: -1. It uses other direction!

stoopid spine! we could refactor rotate to be a number?

that way we can cover all rotations

Yep , I think 0,1,2,3 will be awesome.

wicked, are you happy to take a look at that?

No, I just dont want to do my actual work right now :) its boring

lol, i know that feeling :)

I'll add mirroring too. Lets add all D4 group ! That will help so many projects, including my bomber :) Imagine that you can specify texture with rotate=6 instead of sprite.scale.x=-1

very nice! this will be ace!

Work in progress:

:+1:

Look at updated WIP.
Odd rotations were used only internally, they differ in canvas and webgl mode, but I think its cool effect :)

PR #2237 , demo

all fixed now thanks to @ivanpopelyshev  :+1:


add source link to footer

add page source link to the footer in the new template

@krisdavis : this will add the source link in the footer (I will merge when the tests have passed)


Refactor of Warning ScenarioReCalculation

A warning is logged when the recalculation of partitions different from the one imported with the file is performed.
Since this warning is needed for each importer that could retrigger Scenario Calculations, find a way to not repeat unnecessary code

This item requires some more clarification.
@amuolo @calderinisystemorph


CMake error

Running cmake ..  I'm getting this error:
CMake Error at CMakeLists.txt:57 (add_executable): Cannot find source file: ui_calculator.h
https://github.com/H4wk507/QtCalculator/blob/4c0870ced9a453f4b52668fd5086b6d4c8560114/CMakeLists.txt#L23

I forgot to add this file to git, but apparently, ui_calculator.h is not needed for compilation, because QT automatically creates this file, so I removed it from cmake altogether. Hope it works for you now, and thanks for raising an issue!


nixos/nvidia: optionally disable setup commands for reverse sync

Description of changes
This adds an option to disable the display manager setup commands for PRIME reverse sync.
In some cases, having the display manager set up the providers causes Xorg to crash when a user attempts to reconfigure the monitor layout using xrandr after logging in.
The option is enabled by default to preserve backwards compatibility.
When the option is disabled, PRIME reverse sync is purposely not fully configured; The providers are not set. A consequence of this is that the outputs connected to the NVIDIA GPU can't be used by the display manager. However, it allows users to set the provider and then configure the monitor layout, after logging in.

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

OK. I don't know the correct incantation that will please the checks.
I know the code works because I'm running a copy of it.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/when-i-boot-with-the-external-monitor-connected-xrandr-crashes-xorg/46619/2


Envoy heapprofiler cannot write to directory

istio-proxy@svc02-0v1-55f67fbbf6-wgfg4:/$ curl -X POST localhost:15000/heapprofiler?enable=y
Starting heap profiler
istio-proxy@svc02-0v1-55f67fbbf6-wgfg4:/$ curl -X POST localhost:15000/heapprofiler?enable=n
Heap profiler stopped and data written to /var/log/envoy/envoy.prof. See http://goog-perftools.sourceforge.net/doc/heap_profiler.html for details.
istio-proxy@svc02-0v1-55f67fbbf6-wgfg4:/$ cat /var/log/envoy/envoy.prof
cat: /var/log/envoy/envoy.prof: No such file or directory

I think we just need to either give write access to this folder, or update the bootstrap to configure it to write somewhere else.
cc @silentdai

What do you think if the bootstrap set the envoy.prof path to the directory where the bootstrap is?

What do you think if the bootstrap set the envoy.prof path to the directory where the bootstrap is?

Seems reasonable to me, especially if it outputs where it goes to. Running the heap profiler is not a day to day thing, so if the location it sends it to isn't intuitive or the best location I don't think its that big of a deal, as long as it goes somewhere.

@silentdai I made a custom bootstrap with:
apiVersion: v1
kind: ConfigMap
metadata:
  name: istio-custom-bootstrap-config
  namespace: default
data:
  custom_bootstrap.json: |
    {
      "admin": {
        "profilePath": "/var/lib/istio/envoy/envoy.prof",
        "accessLogPath": "/dev/null",
        "address": {
          "socketAddress": {
            "address": "<IP_ADDRESS>",
            "portValue": 15000
          }
        }
      }
    }

Still is readonly.. I don't get how we write the bootstrap templates though if it is readonly
Error I get:
Dumping heap profile to /var/lib/istio/envoy/envoy.prof.0001.heap (stop and dump)
Failed dumping heap profile to /var/lib/istio/envoy/envoy.prof.0001.heap

With a bit of sudo this works, instructions added to: https://github.com/istio/istio/wiki/Analyzing-Istio-Performance#memory-1


Format

- Summary
The text showed well in Github, but not in https://www.netlifycms.org/docs/authentication-backends/
- Test plan
None
- Description for the changelog
I understand that the problem should be solved with a blank line before the code block.

OK, that works as expected: compare
https://deploy-preview-1315--netlify-cms-www.netlify.com/docs/authentication-backends/
to the current one:
https://www.netlifycms.org/docs/authentication-backends/


double clicking active background thumbnail in list does not open finder

Hi again - we worked on another issue many months back :-).
Small issue I found on macOS Mojave v10.14.6:
So the issue seems to be when opening your app from the menu bar > scrolling down to the bottom > clicking the bottom most thumbnail (which is the active background) > it does NOT open finder and highlight the file.
However if I double click on the thumbnail right above the last entry, it will correctly open finder and go to that image. Then if I go back to the app and now try to double clicking on the bottom most/active background thumbnail, it does correctly change Finder to highlight that file.

Hi @smitty-codes, nice to talk to you again. :)
That behavior is a bit strange. I'll upgrade my system and will investigate it.
Thanks for bringing it to my notice; I'm glad that you are still using the app!

I have this same issue.  also I am not sure why it is listing other wallpapers above, they  are not recently used wallpapers, and I am on a single screen setup, so not sure what those 9 wallpapers above current one are supposed to be.  This is a great app and It is very useful to me. please update it, or is this project abandoned ?

Hi @Aladdinho !
The place where the app fetches the wallpapers from does not list the recency, IIRC. See also, the discussion at #2, esp. the last comment.
Those wallpapers are likely from an extra desktop which you had once created and then closed, or from such desktops from external monitors you may have connected.
I was unable to actually replicate that behavior myself, but now that you are also facing the same problem, I'll try it again.


crypto/rsa: add rand initialization for rsa.SignPSS

If nil as random source is being passed to rsa.SignPSS
this is going to lead to a nil pointer dereference and invalid memory
access. This commit intents to this fix via initializing
a secure random source with crypto/rand.Reader

Hi,
This is my first PR for Go, so please look twice over this PR.

The nil pointer dereference happens here: https://github.com/golang/go/blob/c2406259692427ade0be638e02e8d42a926d41d9/src/crypto/rsa/pss.go#L283
A possible stacktrace for it looks like this:
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
	panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0x49e5e8]

goroutine 19 [running]:
testing.tRunner.func1.1(0x5f4660, 0x7be6c0)
	/usr/lib/go/src/testing/testing.go:940 +0x2f5
testing.tRunner.func1(0xc00014e240)
	/usr/lib/go/src/testing/testing.go:943 +0x3f9
panic(0x5f4660, 0x7be6c0)
	/usr/lib/go/src/runtime/panic.go:969 +0x166
io.ReadAtLeast(0x0, 0x0, 0xc00014c640, 0x20, 0x20, 0x20, 0x736c616972657461, 0x616e222c7d7b3a22, 0x6f6f66223a22656d)
	/usr/lib/go/src/io/io.go:310 +0x58
io.ReadFull(...)
	/usr/lib/go/src/io/io.go:329
crypto/rsa.SignPSS(0x0, 0x0, 0xc000132fc0, 0x5, 0xc00014c620, 0x20, 0x20, 0xc00017dde0, 0x44fd58, 0xc000182228, ...)
	/usr/lib/go/src/crypto/rsa/pss.go:263 +0x105

@heschi can you provide some information why this has been closed? I still think that throwing panics is bad practice and initializing the random as fallback is a good way.

I closed old PRs to reduce load on the Gerrit importer (#50197), sorry for the trouble. I'll reopen the CL and PR.

@heschi no worries :) @FiloSottile I don't know if discussions around this should happen in gerrit or github, but can I get an answer on my latest comment or a final decision? I am not sad, if you reject this commit. I would just like to know if I should invest more time into this.


fix ColladaExporter

Remove old colladamax format, fixed up bad coordinates, added new collada format, valid against the COLLADA 1.4.1 schema, added to load textures and UVs.
Test it!  T3D_ColladaExporter (15MB) Thanks!
Open Issue: https://github.com/GarageGames/Torque3D/issues/467

Features:

UV
material
texture
Valid collada 1.4.1 schema
Compability to Autodesk Collada
Compability to COLLADAMax
Compability to COLLADAMaya
Atlas texture

3D app Tested:

Blender 2.78a
3ds Max 2016
SketchUp 2017
Maya 2016
MeshLab
open3mod (Assimp Viewer)
Sweet Home 3D

3D App Failed:

Winds3D

Feature missing:

Animation

Need to fix:

Collision

failed schema validation:

wrong time format, need atomic type (xs:dateTime) (minor issue)


catalog-model: fix examples/all.yaml name

Oops 😶

This doesn't need a changeset because the next release is mainline? We should probably jump out of next mode then also

Doesn't need a changeset because it's not part of the published package

Good call tho :grin: https://github.com/backstage/backstage/pull/11481

Ah sorry, I mistakenly though that the examples folder was in this: https://github.com/backstage/backstage/blob/master/packages/catalog-model/package.json#L52-L55


Fix RefResolutionError when building swagger schema

Building a swagger schema (i.e. GET requesting /swagger.json) would lead
to a RefResolutionError if the swagger.json contains $refs
The spec.resolver.resolving context manager resolves the ref url and pushes it to the resolver's resolution_scope: https://github.com/Julian/jsonschema/blob/master/jsonschema/validators.py#L318
This causes the spec.resolver.resolve(url) call in resolve_ref to get a RefResolutionError since it's trying to resolve the same ref after the ref url has been pushed to resolution_scope. Getting rid of the spec.resolver.resolve and getting the spec_dict directly from the context manager yield seems to fix this issue
What's Happening:
spec.resolver.resolution_scope = {base_path}
url = {ref_path}
Calling spec.resolver.resolve(url) returns {base_path}/{ref_path}
After with spec.resolver.resolve(url):
spec.resolver.resolution_scope = {base_path}/{ref_path}
Calling spec.resolver.resolve(url) returns {base_path}/{ref_path}/{ref_path} leading to a RefResolutionError

Coverage decreased (-0.001%) to 99.417% when pulling b6cbf91b06445dfa2e7c51cecf9780052102361f on ATRAN2:master into a7886c1c41d61dc6141ddc29c0a0999a04e29495 on striglia:master.

I think this has some pretty strong overlap with https://github.com/striglia/pyramid_swagger/pull/164, which is solving this issue (refs across files) and some others. Does that look like it will solve this issue?
I'll make a note in that PR that we should add a test for serving swagger.json cleanly like you've checked here. Surprising that we don't have one yet!

Hopefully solved by #164. Closing for now, let's re-open if that doesn't cover this issue.


Initial support for Accounts

This adds initial support for accounts a concept meant to signify a customer, a department or any other entity that somehow owns or manages one or more domains.
Meant to solve #267 but using an approach that allows for greater control and easier management.
The purpose is to be able to assign an account to any number of domains, making it easy to track who owns or manages a domain, significantly improving manageability in setups with a large number of domains.
An account consists of a mandatory, unique name and optional description, contact name and mail address. The account name is stripped of spaces and symbols, and lower cased before getting stored in the database and in PowerDNS, to help ensure some type of predictability and uniqueness in the database.
The term account is actually taken from the PowerDNS database, where the domains.account column is used to store the account relationship, in in the form of the account name.
The link to a domain in PowerDNS-Admin is done through the domain.account_id FOREIGN KEY, that is linked to the account.id PRIMARY KEY.
Important notices
Database usage
As we utilize the domains.account column of PowerDNS to properly persist the account relationship in a domain, people using this column for other purposes should be careful. PowerDNS-Admin did not previously use this column at all, so the chance of this being an issue is probably relatively small.
Account associations
If an account is deleted, the account association is actively removed from all it's domains, but the domains themselves will not be modified or deleted in any way. Re-adding an account with the same name later, WILL NOT restore the old associations associations.
However, if the PowerDNS-Admin database gets re-initialized for whatever reason, adding a new account with a name previously assigned to a domain WILL restore the association. This helps simplify recovery in case of data loss.
Account access
In setups where multiple groups of people are managing a lot of domains, managing access to domains through a simple account relationship will save a lot of time and help ensure access rights are correct.
The account concept has grown the possibility of managing domain access permissions through accounts to provide this exact feature. When creating or editing an account, it's possible to add users to the account, through the same familiar multi select control, used on the domains. Users added to an account, automatically gains access to every domain that is tied to that account. Conversely, removing a user from an account, will revoke his access to all domains tied to the account.
Note: This process does not interfere with per-domain access at all, so a user might still have permission to a domain through per-domain permissions after removing him from an account.

Almost ready with the "allow users to be associated with an account an gain domain access through that" code and expect to be able to push it here within 24 hours. I'll update this PR.

There - Account permissions added

@ngoduykhanh having done development on sqlite3, there are a few changes needed to make this work on MySQL/MariaDB. Hang on...

With the latest commit, this works well on my system... The db_update stuff is not included yet - I need to look into how that works.

@tmuncks great work! You have an mysql alter table command so I can test this functionality?

@rodehoed absolutely...
Something like this should work - pastebin'ed to keep markdown from messing with it too much :)
https://pastebin.com/sn57y0Cz
Depending on your DB backend, you may need to tweak it a bit, but it should be pretty clear what it's trying to achieve.
Please let me know if you stumble on anything.
FWIW, we're working on a good solution for the DB migration, so that part is on it's way.

After the discussion with @tmuncks, we are OK with the current implementation and workaround to fix the SQLite migration issue related to ALTER TABLE.
FYI: We are going to switch from current migration scripts to flask-migrate soon.

The new feature in this PR looks good to me. Merging it now. Thanks @tmuncks for the contribution.


add additional test coverage

Description
Adding some additional testing coverage to help prevent future issues
Issues Resolved
None I am aware of
Check List

[x] All tests pass. See https://github.com/chef-brigade/sysctl/blob/master/TESTING.md
[x] New functionality includes testing.
[x] New functionality has been documented in the README if applicable

sysctl::default
  on Ubuntu 14.04
    should create the .d files in the specified location
  on Ubuntu 16.04
    should create the .d files in the specified location
  on Debian 7.11
    should create the .d files in the specified location
  on Debian 8.8
    should create the .d files in the specified location
  on Fedora 25
    should create the .d files in the specified location
  on Redhat 6.9
    should create the .d files in the specified location
  on Redhat 7.3
    should create the .d files in the specified location
  on Centos 6.9
    should create the .d files in the specified location
  on Centos 7.3.1611
    should create the .d files in the specified location
  on Freebsd 10.3
    should create the .d files in the specified location
  on Freebsd 11.0
    should create the .d files in the specified location
  on Suse 12.2
    should create the .d files in the specified location

sysctl::default
  on Ubuntu 14.04
    should not restart procps if restart_procps is false
    should start procps if restart_procps is true
  on Ubuntu 16.04
    should not restart procps if restart_procps is false
    should start procps if restart_procps is true
  on Debian 7.11
    should not restart procps if restart_procps is false
    should start procps if restart_procps is true
  on Debian 8.8
    should not restart procps if restart_procps is false
    should start procps if restart_procps is true
  on Fedora 25
    should not restart procps if restart_procps is false
    should start procps if restart_procps is true
  on Redhat 6.9
    should not restart procps if restart_procps is false
    should start procps if restart_procps is true
  on Redhat 7.3
    should not restart procps if restart_procps is false
    should start procps if restart_procps is true
  on Centos 6.9
    should not restart procps if restart_procps is false
    should start procps if restart_procps is true
  on Centos 7.3.1611
    should not restart procps if restart_procps is false
    should start procps if restart_procps is true
  on Freebsd 10.3
    should not restart procps if restart_procps is false
    should start procps if restart_procps is true
  on Freebsd 11.0
    should not restart procps if restart_procps is false
    should start procps if restart_procps is true
  on Suse 12.2
    should not restart procps if restart_procps is false
    should start procps if restart_procps is true

sysctl_test::attributes
  testing apply action
    apply sysctl_param[vm.swappiness]

sysctl_test::resources
  testing apply action
    apply sysctl_param[vm.swappiness]

Finished in 3.82 seconds (files took 6 seconds to load)
38 examples, 0 failures

I can restructure if needed. Short of doing a full blown test of everything in the LWRP this should provide some quick wins on coverage

@tas50 or whoever. Any issues with this PR?


Terminal Glasses add component missing function

I feel like this function should be added.. its a simple one, but i feel its needed.. unless i am wrong.
after you add a component (addBox, addText, addIcon, etc.) it returns an OBJECT of functions related to that component it generates an ID and it is never the same (as far as i can tell after you restart the computer it gets a different one) so i feel like there should be a getID function included in the returned object.
i feel like its a simple function that should have been added in from the start, but im sure that there are reasons it is not in there, but i have not found anyone else that has thought about this or has came up with a way of doing this.. only reason i found it was because im trying to make a stupidly complex terminal glasses program.

Have you tried turning it off and on again?

Just to clarify, what would the purpose of having the ID be?
Since after a computer/server restart the UI will need to be rebuilt, the ID doesn't provide any persistence.

Also, an aside, the correct place to report this issue is https://github.com/OpenMods/OpenPeripheral-Addons/.
Don't worry about moving the issue, I can cross reference it from the other project. But just for future. Cheers.

sorry about that... and the ID would be used to detect clicking events for an object so that you can just make buttons and not have to use the X and the Y to determine it. it does not even have to be a function per-say, just being able to get the ID when the component is created would be enough. i just this it would add more possibility to terminal glasses programs, and also be able to simplify some code.
my example of the use would basically be scroll bar buttons. to scroll though a list of items on the screen. and the code would be super simple if you could get the component ID, since it gives you what component you clicked on from the event pulled.

Okay, it's been 2 years since I've touched OPA, can you tell me what values are given to you in a pullEvent?

https://gist.github.com/boq/9a098b89ee102cbf1b2b
there is a list :)

Thanks, it looks like the Drawable object already exposes a getId() function, right next to the delete() function.
Have you tried this?

hmm.. its not in the list of objects returned.. i'll try that now and edit this post with the results..

It's not a property, it's a ScriptCallable function, it might not be visible in the metadata of the object. Please check it and let me know.

ok, im semi smart when it comes to computercraft.. but i still have no idea what you just said. or how i would go about testing that.

Just call getId() on the object returned by addBox (:

ok haha thats what i thought and it returns nil :(

You can call delete on the object?

nope

i should note that im on version 0.3.1 more than likely not the most updated one.

It was foolish of me not to ask for version information first. One moment I'll verify it works at my end.

yea its also my bad that i didnt check to see what the latest version was, i cant seem to find a change log on it either...

Do you have a date? Because it needs to be newer than Feb 5th this year
https://github.com/OpenMods/OpenPeripheral-Addons/commit/6c54fe360ff8e71bc684fcc2a2f7304e5c5030e2

Build 162 or newer

sadly i dont have a build number (using a pre made pack) but seems as though it all got added :D so thank you for your help, i'll just update the mods and then mess around with the new version!

Let me know if updating does not work for you, between now and such time I should be able to actually test OPA. I only started developing again yesterday so no test environment for OP(A/I) yet.

will do!

Yep methods are all there and work :D AWESOME thanks for the help, here is the updated list of methods from addBox if anyone wants it...
http://pastebin.com/gFRGUvNr


show something better when player gets one on a dice, currently i am using alert box

I would like to work on it

I would like to work on it

go ahead

I would like to work on it

clone it again please

Yeah, I'll do changes on a new branch and send in a pull request


Update toc.yml

Thank you for contributing to Microsoft Fabric documentation
Fill out these items before submitting your pull request:
If you are working internally at Microsoft:
Provide a link to an Azure DevOps Boards work item that tracks this feature/update.

Who is your primary Skilling team contact? @mention them individually tag them and let them review the PR before signing off.

For internal Microsoft contributors, check off these quality control items as you go

[ ] 1. Check the Acrolinx report: Make sure your Acrolinx Total score is above 80 minimum (higher is better) and with no spelling issues. Acrolinx ensures we are providing consistent terminology and using an appropriate voice and tone, and helps with localization.

[ ] 2. Successful build with no warnings or suggestions: Review the build status to make sure all files are green (Succeeded).

[ ] 3. Preview the pages:: Click each Preview URL link to view the rendered HTML pages on the review.learn.microsoft.com site to check the formatting and alignment of the page. Scan the page for overall formatting, and look at the parts you edited in detail.

[ ] 4. Check the Table of Contents: If you are adding a new markdown file, make sure it is linked from the table of contents.

[ ] 5. #sign-off to request PR review and merge: Once the pull request is finalized and ready to be merged, indicate so by typing #sign-off in a new comment in the Pull Request. If you need to cancel that sign-off, type #hold-off instead. Signing off means the document can be published at any time. Note, this is a formatting and standards review, not a technical review.

Merge and publish

After you #sign-off, there is a separate PR Review team that will review the PR and describe any necessary feedback before merging.
The review team will use the comments section in the PR to provide feedback if changes are needed. Address any blocking issues and sign off again to request another review.
Once all feedback is resolved, you can #sign-off again. The PR Review team reviews and merges the pull request into the specified branch (usually the main branch or a release- branch).
From the main branch, the change is merged into the live branch several times a day to publish it to the public learn.microsoft.com site.

@SnehaGunda - you should really be using the private repo.  Can you either sign off / merge ... or close this and reopen it in the fabric-docs-pr repo? thanks
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Correct the config option name corresponding to --binstubs

This matches my experimental results at least.

Nice catch!
@bundlerbot r+

@bundlerbot retry


project tags test fails in all travis runs

test_15_project_tag in component/test_tags fails consistently in travis jobv 17
ISSUE TYPE

Bug Report
Other - test failure

COMPONENT NAME

CLOUDSTACK VERSION

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

STEPS TO REPRODUCE

go to the travis test job page and then to job 17. restart it if needed.

EXPECTED RESULTS

job passes

ACTUAL RESULTS

the mentioned test lists tags which return NONE insted of the epected list of one.

@DaanHoogland
in test_tags.py, projectid needs to be added.
diff --git a/test/integration/component/test_tags.py b/test/integration/component/test_tags.py
index 9180e7b..be4c6e2 100644
--- a/test/integration/component/test_tags.py
+++ b/test/integration/component/test_tags.py
@@ -1756,6 +1756,7 @@ class TestResourceTags(cloudstackTestCase):
             listall=True,
             resourceType='project',
             resourceIds=project.id,
+            projectid=project.id,
             key=tag_key,
         )
         self.debug("tags = %s" % tags)

Fixed in https://github.com/apache/cloudstack/pull/3476:
==== Marvin Init Successful ====
=== TestName: test_15_project_tag | Status : SUCCESS ===

Closing as this is part of general stabilising effort on master, not an explicit bug.


Where can I record a voice message?

I downloaded this because I saw the"voice record" feature on the website but later I found out that the feature I was looking forward was not even there. :(

It looks like messenger.com doesn't have the feature either. You may be thinking of Messenger for Mobile devices.

but :(

Looks like that needs removing from the site.

https://www.messenger.com/features#voicemessages
heres the url of the voice message

Yes, but that's on Messenger mobile (the mobile app).

New job with southern Cali apparel manufacturer
Here's a 5$ coupon. http://shop.pe/iD1nT
If I sell $500 dollars by 5pm x-apple-data-detectors://2 I keep my lights
on!
We have really cool stuff!
Have a good weekend!
On Saturday, August 8, 2015, crisjansayson<EMAIL_ADDRESS>wrote:

I downloaded this because I saw the"voice record" feature on the website
but later I found out that the feature I was looking forward was not even
there. :(
—
Reply to this email directly or view it on GitHub
https://github.com/Aluxian/Facebook-Messenger-Desktop/issues/215.

--
Rob Estes
Ipad_1
<EMAIL_ADDRESS>602-319-4052

That would be a nice feature, but not top priority right now.
Meanwhile please see the beta if you are still interested in the app: https://github.com/Aluxian/Facebook-Messenger-Desktop/issues/458


Withdrawal receipts aren't always generated

They were generated, but not displayed due to Quasar default for QTable and the fact that pagination controls where hidden
Fixed by https://github.com/ilmercatinodellibro/ilmercatinodellibro/commit/1af0b76b30cc48d1b3f60c6c0e0c06a762676e60


Enable the possibility to import definitions from one plugin to another and expose to other .gradle.kts files

Please enable the possibility to import the definitions from one plugin to another.
Maybe this could be considered a bug, since we could solve this problem in a more easy way in older verions of Gradle. Please see the description below.
Expected Behavior

For example,
plugin_a.gradle.kts
val myVariable = 10

data class MyClass(myProperty1: String, myProperty2: Int) {}

plugin_b.gradle.kts
import plugin_a.gradle.kts // Something like this should work

println(myVariable) // Do some stuff with the properties

val myObject = MyClass("myText", 1)

println(myObject.myProperty2) // Do some stuff with the classes

Current Behavior

It is not possible to import the definitions from one plugin to one another.
We need to bundle everything together and apply to the project.
After the changes from Gradle 6, we cannot read the classes from buildSrc anymore in settings.gradle.kts,
and because of that it is not possible to use the classes, variables, and objects that
we have there. Also, without this, our plugins are very limited and if we need to share some
logic it is not possible.
Context

In my case I needed to try a big change in the project and bundle every thing in one single
.gradle.kts file, which is not ideal.

We can use this to separate the logic from our plugins, like we do using buildSrc. But having something like this for plugins, will help us to have some logic to unify all configurations, like settings.gradle.kts, buildSrc/build.gradle.kts, build.gradle.kts, my-module/build.gradle.kts.
I'm needing to use some hack like this (which is very limited):
plugin/bundle.gradle.kts:
// Unfortunately I cannot use the definitions of the Object Properties outside on a easy way:

extra["getProperty"] = { keys: List<String> -> Properties.get(keys) }
extra["getPropertyOrNull"] = { keys: List<String> -> Properties.getOrNull(keys) }
extra["getPropertyOrDefault"] = { keys: List<String>, default: String -> Properties.getOrDefault(keys, default) }

settings.gradle.kts:
apply("plugin/bundle.gradle.kts")

@Suppress("unchecked_cast", "nothing_to_inline")
fun <T> uncheckedCast(target: Any?): T = target as T
val getProperty = uncheckedCast<(keys: List<String>) -> String>(extra["getProperty"])
val getPropertyOrNull = uncheckedCast<(keys: List<String>) -> String?>(extra["getPropertyOrNull"])
val getPropertyOrDefault = uncheckedCast<(keys: List<String>, default: String) -> String>(extra["getPropertyOrDefault"])

val projectGroup = getProperty(listOf("project.group"))
val projectVersion = getProperty(listOf("project.version"))

The decision to evaluate settings.gradle(.kts) before buildSrc is not something that is going to change again.
It was done to have a more consistent evaluation model and make buildSrc less special, as it caused all kind of issues in the context of composite builds.
For the rest of your question, could you share a reproducer showing what you were doing and what you have to do now? Excluding the buildSrc <-> settings part

Closing as the requested information was not provided.


MEIs (ClusterId and AttributeId) have a invalid pattern instead of a single invalid value, need to add separate

functions for checking if we have received valid values. TODO: Wildcard cluster id with non-global attributes or wildcard attribute paths should be rejected.

https://github.com/project-chip/connectedhomeip/blob/f0f5de0d79ae1c51d7fddbf70900ebac0acd899a/src/app/ReadHandler.cpp#L357-L362

This issue was generated by todo-issue based on a TODO comment in f0f5de0d79ae1c51d7fddbf70900ebac0acd899a.

@mrjerryjohns @bzbarsky-apple ?


Yeelight add support for MiBedsideLamp2 (MJCTD02YL)

Home Assistant release with the issue:
0.93.1
Last working Home Assistant release (if known):
n/a
Operating environment (Hass.io/Docker/Windows/etc.):
docker
Component/platform:
https://www.home-assistant.io/components/homekit_controller/
Description of problem:
Unsupported device
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
discovery:

Traceback (if applicable):
2019-05-21 17:59:08 INFO (MainThread) [homeassistant.components.discovery] Found new service: yeelight {'host': '<IP_ADDRESS>', 'port': 54321, 'hostname': 'yeelink-light-color2_miio133256187.local.', 'properties': {'epoch': '1', 'mac': '04cf8ca4c329'}, 'mac_address': '04cf8ca4c329', 'device_type': 'color2'}
2019-05-21 17:59:08 INFO (MainThread) [homeassistant.components.discovery] Unknown service discovered: homekit {'host': '<IP_ADDRESS>', 'port': 80, 'hostname': 'MiBedsideLamp2-F468.local.', 'properties': {'sh': '9m3WYQ==', 'ci': '5', 'sf': '1', 's#': '1', 'pv': '1.1', 'md': 'MJCTD02YL', 'id': '2C:4A:AE:98:0D:0C', 'ff': '1', 'c#': '1'}, 'name': 'MiBedsideLamp2-F468'}
2019-05-21 17:59:08 INFO (MainThread) [homeassistant.components.discovery] Unknown service discovered: homekit {'host': '<IP_ADDRESS>', 'port': 80, 'hostname': 'MiBedsideLamp2-BACF.local.', 'properties': {'sh': 'XQlSBg==', 'ci': '5', 'sf': '1', 's#': '1', 'pv': '1.1', 'md': 'MJCTD02YL', 'id': '23:AA:BB:A3:0E:E8', 'ff': '1', 'c#': '1'}, 'name': 'MiBedsideLamp2-BACF'}

Additional information:
The new Bedside Lamp 2 is supported in the Xiaomi Home and Google Home app.

What happens now ? Looks like its detected, what doesn't work ? Try declaring it as color2 or ceiling4, it might work.

The lamp isn't added to HA.
I just did
discovery:
  enable:
    - homekit

2019-05-21 18:21:55 INFO (MainThread) [homeassistant.components.discovery] Found new service: homekit {'host': '<IP_ADDRESS>', 'port': 80, 'hostname': 'MiBedsideLamp2-BACF.local.', 'properties': {'sh': 'XQlSBg==', 'ci': '5', 'sf': '1', 's#': '1', 'pv': '1.1', 'md': 'MJCTD02YL', 'id': '23:AA:BB:A3:0E:E8', 'ff': '1', 'c#': '1'}, 'name': 'MiBedsideLamp2-BACF'}
2019-05-21 18:21:55 INFO (MainThread) [homeassistant.components.discovery] Found new service: homekit {'host': '<IP_ADDRESS>', 'port': 80, 'hostname': 'MiBedsideLamp2-F468.local.', 'properties': {'sh': '9m3WYQ==', 'ci': '5', 'sf': '1', 's#': '1', 'pv': '1.1', 'md': 'MJCTD02YL', 'id': '2C:4A:AE:98:0D:0C', 'ff': '1', 'c#': '1'}, 'name': 'MiBedsideLamp2-F468'}

Then I got a notification that I can enter the PIN for both devices which is displayed on the bottom of the lamp and that worked.

Does it work if you define it manually as yeelight? https://www.home-assistant.io/components/yeelight/ has the documentation for manual configuration.
Maybe it advertises homekit support, so that's why homekit integration picks it up, unfortunately I have no idea about its details.

Does it work if you define it manually as yeelight? https://www.home-assistant.io/components/yeelight/ has the documentation for manual configuration.
Maybe it advertises homekit support, so that's why homekit integration picks it up, unfortunately I have no idea about its details.

Don't know for sure anymore since I enabled homekit support manually in the config as you can see and finally was able to configure it within HA with the right PIN. The Yeelight app can control the BedSide2 lamps without a problem. It advertises homekit support indeed but doesn't 'load' correctly within HA with 'standard' autodiscovery. I think personally that it is a homekit since auto discovery is giving the message and not yeelight.

Hi!
From a HomeKit point of view everything looks like it is working as expected. On 0.94 the homekit discovery will be enabled by default so it wouldn't have needed enabling manually. 0.94 will switch over to config entries too, and not depend on discovery at start up (theres a 30s delay after restarts on < 0.94 thats a bit annoying).
It's normal for some devices to have multiple control APIs - e.g. we have a few climate devices I have been looking at that can either be controlled via their cloud API or via homekit and homekit is the only viable "local" internetless API. So that bit is normal as well.
It's weird that the native client hasn't detected it but unless there are features not exposed to the HomeKit API I guess its fine to use the HomeKit API.

Hi!
From a HomeKit point of view everything looks like it is working as expected. On 0.94 the homekit discovery will be enabled by default so it wouldn't have needed enabling manually. 0.94 will switch over to config entries too, and not depend on discovery at start up (theres a 30s delay after restarts on < 0.94 thats a bit annoying).
It's normal for some devices to have multiple control APIs - e.g. we have a few climate devices I have been looking at that can either be controlled via their cloud API or via homekit and homekit is the only viable "local" internetless API. So that bit is normal as well.
It's weird that the native client hasn't detected it but unless there are features not exposed to the HomeKit API I guess its fine to use the HomeKit API.

In that case I'll wait for that update and remove the discovery entry for homekit. Hopefully it will detect it then automatically and add it to HA.

Cool - we are still working out some of the finer details of the switchover but if it doesnt work the things to check will be - is the zeroconf component active and is homekit not listed at all under discovery. If both of those are true it should work. (I believe that is the "stock" setup).

Hi @pascalsaul  and @Jc2k, I finally got mine lamp working after manually forcing homekit to discover but now the only feature controllable it's the brightness, (NOT RGB color) as it should be, so does you know how to handle other configs? Should I create a custom sensor with yeelight?

I think it might be same error as described here: https://github.com/home-assistant/home-assistant/issues/23871
Try this code from pending PR, it might fix it:
https://github.com/home-assistant/home-assistant/issues/23871#issuecomment-494466386
https://github.com/home-assistant/home-assistant/issues/23871#issuecomment-496381139

@zewelor thanks, fixed!

I don't know what the update to 0.94 break but now are both lights not detected at all...

You need to make sure the zeroconf component is turned on, it’s a breaking change listed in the release notes.

Ah I replied before I saw your edit

Have you managed to add Bedside Lamp 2 to HA somehow guys?
Trying to manual connection via Yeelight component with no luck.


Fix split layout and nav item style

Fix Nav::Item::Component width.
Fix SplitLayout::Component resize handler z-index to be over Source panel.

Thanks @mdrbohlav! Just working through your PRs now. Really appreciate the contributions 👍


JFXDatePicker disable date

Is it possible to disable dates in JFXDatePicker  ? I find nothing about this
Thanks for your help

Hello,
Currently you can't. Feel free to add your own contribution to JFoenix.
Regards,


multicall error

I want to use multicall to reduce the number of rpc requests, but I can't get the data needed for eth_call properly, thanks for your help.
solidity code:
https://etherscan.io/address/0x5ba1e12693dc8f9c48aad8770482f4739beed696#code
php code:
<?php

require __DIR__ . '/vendor/autoload.php';

use Web3\Web3;
use Web3\Contract;
use Web3\Providers\HttpProvider;
use Web3\RequestManagers\HttpRequestManager;

$rpc_url = 'https://rpc.ankr.com/eth';
$web3 = new Web3(new HttpProvider(new HttpRequestManager($rpc_url, 30)));

$abiJson = '[{"inputs":[{"components":[{"internalType":"address","name":"target","type":"address"},{"internalType":"bytes","name":"callData","type":"bytes"}],"internalType":"struct Multicall2.Call[]","name":"calls","type":"tuple[]"}],"name":"aggregate","outputs":[{"internalType":"uint256","name":"blockNumber","type":"uint256"},{"internalType":"bytes[]","name":"returnData","type":"bytes[]"}],"stateMutability":"nonpayable","type":"function"},{"inputs":[{"components":[{"internalType":"address","name":"target","type":"address"},{"internalType":"bytes","name":"callData","type":"bytes"}],"internalType":"struct Multicall2.Call[]","name":"calls","type":"tuple[]"}],"name":"blockAndAggregate","outputs":[{"internalType":"uint256","name":"blockNumber","type":"uint256"},{"internalType":"bytes32","name":"blockHash","type":"bytes32"},{"components":[{"internalType":"bool","name":"success","type":"bool"},{"internalType":"bytes","name":"returnData","type":"bytes"}],"internalType":"struct Multicall2.Result[]","name":"returnData","type":"tuple[]"}],"stateMutability":"nonpayable","type":"function"},{"inputs":[{"internalType":"uint256","name":"blockNumber","type":"uint256"}],"name":"getBlockHash","outputs":[{"internalType":"bytes32","name":"blockHash","type":"bytes32"}],"stateMutability":"view","type":"function"},{"inputs":[],"name":"getBlockNumber","outputs":[{"internalType":"uint256","name":"blockNumber","type":"uint256"}],"stateMutability":"view","type":"function"},{"inputs":[],"name":"getCurrentBlockCoinbase","outputs":[{"internalType":"address","name":"coinbase","type":"address"}],"stateMutability":"view","type":"function"},{"inputs":[],"name":"getCurrentBlockDifficulty","outputs":[{"internalType":"uint256","name":"difficulty","type":"uint256"}],"stateMutability":"view","type":"function"},{"inputs":[],"name":"getCurrentBlockGasLimit","outputs":[{"internalType":"uint256","name":"gaslimit","type":"uint256"}],"stateMutability":"view","type":"function"},{"inputs":[],"name":"getCurrentBlockTimestamp","outputs":[{"internalType":"uint256","name":"timestamp","type":"uint256"}],"stateMutability":"view","type":"function"},{"inputs":[{"internalType":"address","name":"addr","type":"address"}],"name":"getEthBalance","outputs":[{"internalType":"uint256","name":"balance","type":"uint256"}],"stateMutability":"view","type":"function"},{"inputs":[],"name":"getLastBlockHash","outputs":[{"internalType":"bytes32","name":"blockHash","type":"bytes32"}],"stateMutability":"view","type":"function"},{"inputs":[{"internalType":"bool","name":"requireSuccess","type":"bool"},{"components":[{"internalType":"address","name":"target","type":"address"},{"internalType":"bytes","name":"callData","type":"bytes"}],"internalType":"struct Multicall2.Call[]","name":"calls","type":"tuple[]"}],"name":"tryAggregate","outputs":[{"components":[{"internalType":"bool","name":"success","type":"bool"},{"internalType":"bytes","name":"returnData","type":"bytes"}],"internalType":"struct Multicall2.Result[]","name":"returnData","type":"tuple[]"}],"stateMutability":"nonpayable","type":"function"},{"inputs":[{"internalType":"bool","name":"requireSuccess","type":"bool"},{"components":[{"internalType":"address","name":"target","type":"address"},{"internalType":"bytes","name":"callData","type":"bytes"}],"internalType":"struct Multicall2.Call[]","name":"calls","type":"tuple[]"}],"name":"tryBlockAndAggregate","outputs":[{"internalType":"uint256","name":"blockNumber","type":"uint256"},{"internalType":"bytes32","name":"blockHash","type":"bytes32"},{"components":[{"internalType":"bool","name":"success","type":"bool"},{"internalType":"bytes","name":"returnData","type":"bytes"}],"internalType":"struct Multicall2.Result[]","name":"returnData","type":"tuple[]"}],"stateMutability":"nonpayable","type":"function"}]';
$abi = json_decode($abiJson, true);
$contractAddress = '0x5BA1e12693Dc8F9c48aAD8770482f4739bEeD696';

$contract = new Contract($web3->provider, $abi);

$data = $contract->at($contractAddress)->getData('aggregate', [["0xdAC17F958D2ee523a2206206994597C13D831ec7","0x70a0823100000000000000000000000028c6c06298d514db089934071355e5743bf21d60"],["0xdAC17F958D2ee523a2206206994597C13D831ec7","0x70a08231000000000000000000000000d6216fc19db775df9774a6e33526131da7d19a2c"]]);
echo $data;

// The correct data should be: 252dba4200000000000000000000000000000000000000000000000000000000000000200000000000000000000000000000000000000000000000000000000000000002000000000000000000000000000000000000000000000000000000000000004000000000000000000000000000000000000000000000000000000000000000e0000000000000000000000000dac17f958d2ee523a2206206994597c13d831ec70000000000000000000000000000000000000000000000000000000000000040000000000000000000000000000000000000000000000000000000000000002470a0823100000000000000000000000028c6c06298d514db089934071355e5743bf21d6000000000000000000000000000000000000000000000000000000000000000000000000000000000dac17f958d2ee523a2206206994597c13d831ec70000000000000000000000000000000000000000000000000000000000000040000000000000000000000000000000000000000000000000000000000000002470a08231000000000000000000000000d6216fc19db775df9774a6e33526131da7d19a2c00000000000000000000000000000000000000000000000000000000

php error:
Uncaught TypeError: call_user_func(): Argument #1 ($callback) must be a valid callback, first array member is not a valid class name or object in /xxx/vendor/web3p/web3.php/src/Contracts/Ethabi.php:162

Also interesting


Revert "Revert "[Enhancement] Optimize hash join when hash table is empty" (#9701)"

What type of PR is this：

[x] bug
[ ] feature
[x] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #9701  #901
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

@mergify branch-2.4

@mergify backport branch-2.4


Choosing Best model

I couldn't find a mechanism to choose the best model. Can you point me to it?

@xialeiliu Can you look into this?


Not allowed to load local resource -> ModuleBuildError: Module build failed -> TypeError: Failed to fetch dynamically imported module

Describe the bug
When using next-yak 0.2.1 on StackBlitz, the compilation fails due to a security error preventing the loading of a local resource.
Chrome/Brave:
Not allowed to load local resource: file:///home/projects/stackblitz-starters-bp97wq/node_modules/next-yak/dist/loaders/tsloader.js

Firefox:
Security error: Content on https://stackblitzstartersbp97wq-hlff.w-corp-staticblitz.com/iframe.70dbe416.html is not allowed to load or link file:///home/projects/stackblitz-starters-bp97wq/node_modules/next-yak/dist/loaders/tsloader.js

Safari:
[Error] Cannot load file:///home/projects/stackblitz-starters-bp97wq/node_modules/next-yak/dist/loaders/tsloader.js due to access control checks.
[Error] Failed to load resource: Cross origin requests are only supported for HTTP. (tsloader.js, line 0)

The problem arises because next-yak integrates into the Next.js config and injects a webpack loader using an absolute path (e.g., /home/projects/stackblitz-starters-bp97wq/node_modules/next-yak/dist/loaders/tsloader.js).
As soon as Next.js triggers the compilation and attempts to execute the loader, it fails due to the security restrictions in StackBlitz's environment
Link to the blitz that caused the error
https://stackblitz.com/edit/stackblitz-starters-bp97wq
Steps to reproduce

Open this StackBlitz project: https://stackblitz.com/edit/stackblitz-starters-bp97wq
The error occurs immediately on initial page load as the compilation is triggered.

Expected behavior
The project should compile and run successfully, as it does in a local environment
Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots

Platform
Browser name  = Chrome
Full version  = <IP_ADDRESS>
Major version = 126
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 273394398,
  "usedJSHeapSize": 226013986,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = 70dbe416

Additional context
Worked before with next-yak 0.2.0:
https://stackblitz.com/edit/stackblitz-starters-dfykqy?file=app%2Fpage.tsx

Hey @jantimon 👋 ! Thanks for reaching out. This looks like a bug indeed. I'll create a ticket and see what's causing this.


how to run ?

i don't want to run this code on google cloud i just want it till ("Extract the data and split it into shards")
but i don't know how to do it can someone explain me how to run this commands
this one
PROJECT="your-google-cloud-project"
DATADIR="gs://${BUCKET?}/opensubtitles/$(date +"%Y%m%d")"
python opensubtitles/create_data.py
--output_dir ${DATADIR?}
--sentence_files gs://${BUCKET?}/opensubtitles/raw/lines/lines-*
--runner DataflowRunner
--temp_location ${DATADIR?}/temp
--staging_location ${DATADIR?}/staging
--project ${PROJECT?}
--dataset_format TF

create account in google cloud,
create a project and enable bigquery, google cloud storage.
These commands should be ran on Google clould console

hello dyoungkstn i am asking i want  to run this whole project on vs code (on my local machine ) but things showing error
at first it was showing tensorflow error ,then apache baem and right now assertion error can you tell me how to run this project locally


HHH-17466 Fixed local variable names (2 variables)

Fixed local variable names (2 variables)
Reference: https://hibernate.atlassian.net/browse/HHH-17466?focusedCommentId=113718

Thanks

This one was fun - half a minute to refactor variable names and hour and half to find out how to fix my fork which had previous commit from other JIRA ticket. LOL
Now I know for next time.


Fix typo and CRDs

Jenkins job istio.github.io/presubmit passed

Jenkins job istio.github.io/presubmit passed


Parental rating restrictions on playout

DVB-I client is configured with a maximum unrated age value (maxUnratedAge). If program rating (programAge) exceeds maxUnratedAge then unlock PIN is requested. On program boundary, if programAge is greater than maxUnratedAge and current (new) program genre is different from previous program genre then unlock PIN is requested.
Test assets

Test stream with alternating rated and non-rated content (1 minute each) and applicable EPG metadata. Each time the rated content arrives, an unlocking PIN is requested. If not provided the playout is blocked.

Rated programs specified with <ProgramInformation><BasicDescription><ParentalGuidance><mpeg7:MinimumAge>18

Non rated programs specified with either

no <ParentalGuidance> or

<ProgramInformation><BasicDescription><ParentalGuidance><mpeg7:MinimumAge>4

Test stream with rated programs denoted with <ProgramInformation><BasicDescription><ParentalGuidance><mpeg7:MinimumAge>18 (alternating every 1 minute). DVB-I client should seek PIN code when program changes with same or higher rating.

Implemented in  https://github.com/DVBProject/DVB-I-Reference-Client/releases/tag/2.28


discouraged-comment-location improvements (and question)

Hey there folks! I'm trying to disable this rule because its annoying as s***.
In addition, the formatter won't auto-fix this. Please add this option to the formatter! ideally before making the rule standard.. I think its too late for that though.
QUESTION - I can't seem to disable this rule.. it impacts other rules which breaks the tool. Could you help me here?
Here is the error:
  - Rule with id 'RuleId(value=standard:if-else-wrapping)' requires rule with id 'RuleId(value=standard:discouraged-comment-location)' to be loaded

Here is my config:
[*.{kt,kts}]
ktlint_standard_discouraged-comment-location = disabled
ktlint_standard_if-else-wrapping = disabled

Expected Behavior

Rule that impacts huge areas of codebases that's newly standardized also releases formatter support as a code standard.
Easy way to disable it that's documented in a sensible and easy to find location.

Current Behavior

Additional information

Current version of ktlint: 4.1.0

For background see https://github.com/pinterest/ktlint/issues/2338 which basically handles same problem in combination with other rules. So I am closing this one as duplicate.
TL/DR: handling comments in all kind of different places just cost too much time and energy.


Move end to end test to 6-gce

Since this is now the last MVM step, e2e test makes more sense in here.

LGTM.


Generated code is not compliant with noUnusedParameters

struct MyResponse { }
generates the following code:
export class MyResponse {     constructor(args?: IMyResponseArgs) {     } }
which with noUnusedParameters causes errors

Yeah, we don't document that it will work with that flag, just noUnusedLocals. I would like to add this support. Not sure when we will have the bandwidth.


Added Android custom icon support

You can now supply the name of custom icons for play, pause, prev, next, close, and the general notification icon (in place of the play/pause icon).
E.g. in options:
{
...,
playIcon: 'media_play',
...
}

Hello! This looks pretty good, can you rebase into a single commit?  Thanks :)

Alright, I believe that's done. Full disclosure, I've never actually done a rebase to merge commits into one before.

I'll pull it down and see if It's registering correctly. It looks like there's new commits but that could just be github being weird.

closing in favor of using #99 (I was able to squash the commits)


Documentation typo.

Typo on page.
seperated should be separated.

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

ID: 27df8083-fc6e-b73d-9172-d91329bc9391
Version Independent ID: 504e3f3e-650f-3c41-041f-bd3112a1b749
Content: How to run a reindex job in FHIR service - Azure Health Data Services
Content Source: articles/healthcare-apis/fhir/how-to-run-a-reindex.md
Service: healthcare-apis
Sub-service: fhir
GitHub Login: @EXPEkesheth
Microsoft Alias: kesheth

@clarotech
Thanks for your feedback! We will investigate and update as appropriate.


(docs)(fix): Small Flyway naming convention fix

Hi! Thanks @geirolz for this wrapper, I started testing it today and it's working great :)
I'm creating this small PR to fix a small typo regarding Flyway naming convention!
Please let me know if anything!

Hi @zant, thanks for appreciating this library and thanks for contributing, I really appreciated!
The PR LGTM, I'm gonna merge it 👍

FYI I've also update this article thanks to this PR
https://dev.to/geirolz/database-migrations-in-scala-2lcl


⚠ Panel - Hydeos has degraded performance

In c2b3a56, Panel - Hydeos (https://pterodactyl.hydeos.fr) experienced degraded performance:

HTTP code: 200
Response time: 1879 ms

Resolved: Panel - Hydeos performance has improved in 30b3afa after 12 minutes.


Integrate IDS serializer, closes #236

Signed-off-by: Dominik Pinsel<EMAIL_ADDRESS>What this PR changes/adds:
This pull request integrates the IDS Serializer from the Fraunhofer repository. As the serializer adds message validation it was necessary to update/fix the creation of most IDS message.
Why it does that
The IDS serializer is necessary to send and receive IDS compliant messages.
Further notes
The biggest change happened for the negotiation, where the whole negotiation process was not based on IDS message ids.  As correlation IDs for most negotiation messages became mandatory in the IDS serializer the negotation managers where updated, to use them as well. To make this possible the ContractOffer got a properties map, to track corresponding message IDs.
Linked Issue(s)
Closes #236
Checklist

[x] added appropriate tests?
[ ] performed checkstyle check locally?
[ ] added/updated copyright headers?
[ ] documented public classes/methods?
[ ] added/updated relevant documentation?

Dominik Pinsel<EMAIL_ADDRESS>Daimler TSS GmbH, legal info/Impressum

Hi @DominikPinsel, after talking to @ronjaquensel and @tmberthold, I want to highlight the following issue we always had with the IDS Infomodel and Serializer libraries: Setting the serializer version as fixed, forces the component to only ensure the compatibility to one Infomodel version, as we do not have any compatibility table. This means an incoming message with 4.2.7 could be deserialized with serializer 4.2.8, but maybe not with 4.2.6 or 5.0.0, although the handling of the IDS objects stays the same. This implies that we need to reject messages of another version in order to prevent exceptions. This raises some issues regarding the compatibility to other IDS components both central services and connectors. Maybe we could find a way to support multiple serializer versions? In this context, the EDC could recognize what version the incoming message has and take the appropriate serializer version to deserialize objects. This would solve syntax problems. Nevertheless, we also have to keep in mind and implement ways of handling missing properties. So if our message processing depends on an attribute that a newer version of the Infomodel may not cover, we need to check and implement workarounds. Thus, every attribute from an IDS input has to be checked for existence before processing it.

Before code is written I would highly recommend we discuss the requirements and technical approach that should be taken. @juliapampus brings up a number of issues we should discuss upfront.

Before code is written I would highly recommend we discuss the requirements and technical approach that should be taken. @juliapampus brings up a number of issues we should discuss upfront.

Yea sure. Maybe we can do this in this PR, so that it's transparent for everyone.
For the discussion I'd assume these two points are given

to be IDS compliant our IDS messages must be JSON-LD
for the serialization and deserialization we will use the IDS-Serializer, which is forked in our EclipseDataspaceConnector organization, too.

Do you agree?

Before code is written I would highly recommend we discuss the requirements and technical approach that should be taken. @juliapampus brings up a number of issues we should discuss upfront.

@jimmarino Yea sure. Maybe we can do this in this PR, so that it's transparent for everyone.
For the discussion I'd assume these two points are given

to be IDS compliant our IDS messages must be JSON-LD
for the serialization and deserialization we will use the IDS-Serializer, which is forked in our EclipseDataspaceConnector organization

Do you agree?

That's not my point and this type of work should not begin with a PR.

@DominikPinsel @jimmarino I apologize for starting a discussion here, that was not my intention. We can turn my comment into a discussion if you like. I think adding one serializer at all (without major structural changes) also fits directly into a PR like this or not? This is actually just a change of the object mapper and adding mandatory attributes.

@DominikPinsel @jimmarino I apologize for starting a discussion here, that was not my intention. We can turn my comment into a discussion if you like. I think adding one serializer at all (without major structural changes) also fits directly into a PR like this or not? This is actually just a change of the object mapper and adding mandatory attributes.

@juliapampus that's exactly aligned with my thoughts. Adding a serializer should be a simple change that fits into a defined issue and focused PR. This PR currently includes gratuitous changes to 75 files all over the codebase and its intention is unclear. I think it may be more efficient to open an Issue (isn't there already one for the serializer work?) and then create a focused PR to address it.

@DominikPinsel @jimmarino I apologize for starting a discussion here, that was not my intention. We can turn my comment into a discussion if you like. I think adding one serializer at all (without major structural changes) also fits directly into a PR like this or not? This is actually just a change of the object mapper and adding mandatory attributes.

@juliapampus that's exactly aligned with my thoughts. Adding a serializer should be a simple change that fits into a defined issue and focused PR. This PR currently includes gratuitous changes to 75 files all over the codebase and its intention is unclear. I think it may be more efficient to open an Issue (isn't there already one for the serializer work?) and then create a focused PR to address it.

I will try to come up with a list of smaller issues, that all need to be resolved before it adding a serializer becomes a "simple change". Like already said, most message that were created by the code are currently not  IDS compliant (which is validated by the serializer). As these information is most of the time not next to the message creating code, many files needed to be updated, with the most complex changed in the contract negotiation.

I opened a discussion for the multi-serializer topic: https://github.com/eclipse-dataspaceconnector/DataSpaceConnector/discussions/767


Orca is too chatty when switching tabs due to aria-label being redundant to title

Step to Reproduce:

Switch between open documents with Alt+n (where n is the index of the tab)

Orca's presentation for a file named "hello world" is:
hello world. page tab /path/to/hello world

The reason why is that Orca by default presents (at least) the following information for a widget:

accessible name (e.g. the value of aria-label)
accessible role (e.g. the localized value of ARIA's role)
accessible description (e.g. the value of aria-description, aria-describedby, or the title)

From the inspector, I see:
<div ... role="tab" class="tab ... aria-label="hello world" title="/home/jd/hello world" ...>

I get that knowing the path is of value and makes sense as something which appears when the user hovers the mouse over the tab. By the same token, I don't think this amount of verbosity is great for the screen reader user.
And yes, I believe I said in some other issue that putting the path where it would be exposed as the accessible description made sense because users could interrupt speech, etc. But in this particular case, user interruption might not be ideal because what Orca says immediately after is relevant to the user, namely: That the editor has focus and the current line.
I'd be interested in hearing what users (@jvesouza, @webczat, et al.) and @isidorn think.

For me that is not really a problem. I probably have some issues with things being too chatty, for example when pressing numpad enter and I want to read current line of text I am on... but not when switching editors.

I think reading the path to a file when a tab gets focused is too verbose. There are other ways to figure out the path of the file and imho it should not be read out every time a user focuses on a file.
I just tried VoiceOver and it nicely ignores the title attribute and only reads the file name.
imho in general when aria-label is present screen readers should ignore title as the aria-label should containt the fully description. title should only be used when aria-label is not present as a substitute. Though you would know better, this just makes sense from my standpoint.

@isidorn Fair enough. But here's the thing:

accessible name != accessible description. See under https://w3c.github.io/aria/#terms. As you'll see there:

An accessible description provides additional information, related to an interface element, that complements the accessible name.

For most elements when there is an aria-label and a title attribute (and no other source of name/description info), the value of aria-label gets exposed to platforms as the accessible name; the title attribute gets exposed to platforms as the accessible description. Given the definition/explanation I quoted above, screen readers that include the accessible description by default are operating under the assumption that the accessible description is providing additional information rather than redundant information.

Screen readers using their accessibility APIs to get the accessible name and the accessible description don't necessarily know what authoring caused those accessible properties to be there. If the description was put there by, for instance, the new aria-description property, it would look the same to me as if it came from the title.

I suppose I can add a heuristic to Orca to handle this particular case (description looks like a file path and the filename seems to be in the accessible name)....

Ok thanks for clarifying. I oversimplified things.
So for me this more feels like a screen reader verbosity setting. If the screen reader is set to be very concise it should only read aria-description when explicitly asked for by the user.
If the screen reader is in a very verbose mode than it should always first read aria-label then aria-description.
And after reading the docs I take back what I said that the aria-label is a substitue for the title. That is incorrect, however aria-description is a substitue, thus if aria-description is set I think you should ignore the title.
That heuristic you propose is imho a bit too specific and sounds like a hack.
A heuristic which imho makes sense is to do some string matching is see how similar the name and description are. But even then this feels a bit magical. I still think this shuold be user configured via verbostiy setting and there shuold be an explicit action to get the description being read.

Orca already has a verbosity setting for description so it can be disabled. Orca also has a command to get detailed information for the current object. But the user is only very briefly on the page tab because focus immediately goes to the editor.
And I agree about sounds like a hack. Orca already has simple comparisons for name and description, but it's not yet catching this particular case.

Ok makes perfect sense.
Then I think the only way to fix this if you want is to fine tune that simple comparision which you already do.
💡 or even better, vscode sets the aria-description to be an empty string "". aria-descirption takes precendence over title and orca does not read anything 💡

To be honest, I'm not 100% sure that would work. Just to be sure, I asked in the associated PR.

@joanmarie ok thanks! Let me know what they say and if we should do the aria-descirption change on the vscode side.
For me it feels like the client should have some way to tell the screen reader that the title should be ignored.

Just tried out setting aria-description to an empty string and it does not work. Orca is still too chatty, but in theory if that is the right thing to do, we can do that change and you can fix it on the Orca side. So let's wait for the reply on the PR. Thanks!

So the comparison stuff I have locally (not yet committed) solves the original problem.
BUT it seems like now I'm sometimes (but not always) seeing a selection event for the tab after the editor claims focus. This is a new development and leads to different chattiness (which sounds very similar to the chattiness I reported here).
I tried to filter out this new chattiness, but because sometimes the events are missing the filtering resulted in Orca saying nothing. I'll get to the bottom of that in a bit (hopefully!) and then open a new issue if appropriate.

So Aaron just responded. We should get an empty description. I can confirm that's the case by loading this simple test in chromium built locally: data:text/html,<input aria-label="enter stuff here" title="tooltip!" aria-description=""> So I guess it's just a matter of VSCode / Electron using a more recent version of Chromium.

@joanmarie fantastic. I have just pushed a change that we set the aria-description on the tab area to empty.
Next milestone we should update to Chrome 80 so hopefully it should just work then.
We should have this aria-description in mind in case more situations appear where Orca is reading a title and it should not.


Fix infinite loop in LogLevel detection

While converting the logLevel into the right const value, calling $container->resolveEnvPlaceholders($level, true); does not always throw an exception and returns the value provided in INPUT, leading to an infinite loop
This PR fixes  https://github.com/symfony/monolog-bundle/issues/367#issuecomment-723988352 BUT is not related to the issue https://github.com/symfony/monolog-bundle/issues/367#issue-718712384

You were fast on this one, thanks Jérémy.


Why Urql converts Array to Object ?

I want to fix this strange behavior, Urql converts some array to object.
The property nodes is an array wich contains tuples of arrays, for example :
nodes : [[0,5],[10,12],[14,18]]

But after retrive data of Urql query, Urql returns objects

But, like you can see, the response of graphql and database is arrays, not objects.

To be sure, same verification with GraphQLi

My opinion is that Urql converts the array to object, is it possible ?
urql version & exchanges<EMAIL_ADDRESS>Steps to reproduce
The Urql query retrives objects instead arrays.
Expected behavior
Urql query returns arrays
Actual behavior
Urql query returns objects

Can you post a reproduction here, like queries, data samples, or a minimal reproduction sandbox? That'll really help speed up how long it'll take for us to look into this, since otherwise we'll have to start from scratch. Screenshots are fine for illustration purposes, but they're static and it's easier for us to have the queries on hand ❤
Also, we'll need steps to reproduce this and information about your environment. For instance, I'm assuming you must be using @urql/exchange-graphcache?

When you have spoken about @urql/exchange-graphcache, I have commented it and that resolves the problem.
  const urql = createClient({
    url: url,
    fetchOptions: {credentials: 'include'},
    exchanges: [
      dedupExchange,
      // cacheExchange, => disabled this resolve the problem.
      authExchange({//...}),
      ssr,
      fetchExchange
    ],
    requestPolicy: 'network-only',
    maskTypename : true
  });

Same console.log()

If you need yet more information , I'll add it later.
Thanks

Do you have any configuration on the cacheExchange from @urql/exchange-graphcache?

I don't have installed @urql/exchange-graphcache
I only activate the cacheExchange like in documentaiton, no particular configuration.

All right, that's a good detail to have that means it's in our document-cache we have this bug

If it can help, here is my package.json
{
  "name": "app",
  "version": "1.0.0",
  "private": true,
  "dependencies": {
    "@ciruxx/joi-extension-objectid": "^1.0.2",
    "@nuxtjs/axios": "^5.12.5",
    "@nuxtjs/proxy": "^2.1.0",
    "@riophae/vue-treeselect": "^0.4.0",
    "@shopify/draggable": "^1.0.0-beta.12",
    "@urql/core": "^2.1.4",
    "@urql/exchange-auth": "^0.1.3",
    "@urql/introspection": "^0.2.0",
    "bootstrap": "^5.0.2",
    "bricks.js": "^1.8.0",
    "column-resizer": "^1.3.6",
    "cookie-universal-nuxt": "^2.1.5",
    "core-js": "^3.15.1",
    "dayjs": "^1.10.4",
    "element-ui": "^2.15.6",
    "express": "^4.17.1",
    "graphql": "^15.5.1",
    "graphql-combine-query": "^1.2.3",
    "graphql-tag": "^2.12.5",
    "http-proxy-middleware": "^2.0.0",
    "interactjs": "^1.10.2",
    "isomorphic-unfetch": "^3.1.0",
    "joi": "^17.4.2",
    "json-to-graphql-query": "^2.0.0",
    "lodash": "^4.17.20",
    "masonry-layout": "^4.2.2",
    "nuxt": "^2.13.3",
    "pluralize": "^8.0.0",
    "v-calendar": "^2.1.1",
    "vue-material-design-icons": "^4.12.1",
    "vue-tippy": "^4.10.2",
    "vuex-map-fields": "^1.4.1"
  },
  "devDependencies": {
    "@nuxtjs/router-extras": "^1.1.1",
    "babel-plugin-component": "^1.1.1",
    "fibers": "^5.0.0",
    "sass": "^1.35.1",
    "sass-loader": "^10.1.0"
  }
}


@MartyG-RealSense That did the trick.

@MartyG-RealSense That did the trick.
For future reference, here is a complete list of steps I took to get the IntelRealsense Camara D455 to work with python3 on the Jetson Nano.

Update CMake using this method
(You need to have curl installed for this to work. )

Download the zip file from https://github.com/IntelRealSense/librealsense/releases/. I am using version 2.38.1

Extract the file, cd into the extracted file .

Create a dir called build and cd into it.

Run the CMake command to test to see if the build will work.
cmake ../ -DFORCE_RSUSB_BACKEND=ON -DBUILD_PYTHON_BINDINGS:bool=true - DPYTHON_EXECUTABLE=/usr/bin/python3.6 -DCMAKE_BUILD_TYPE=release -DBUILD_EXAMPLES=true -DBUILD_GRAPHICAL_EXAMPLES=true -DBUILD_WITH_CUDA:bool=true

If you have issues with the above step, check out this, this and this

Still in the build dir. Run make -j4 and then sudo make install.

Add these to the end of your .bashrc file

export PATH=$PATH:~/.local/bin
export PYTHONPATH=$PYTHONPATH:/usr/local/lib
export PYTHONPATH=$PYTHONPATH:/usr/local/lib/python3.6/pyrealsense2

source your .bashrc file

Deal with package restructuring between python2 and python3 using this method

You should now be able to import pyrealsense2 using python3
@MartyG-RealSense thank you once again for your help 💯
Update: I can confirm that this works for the Jetson Xavier NX as well.
Originally posted by @Drkstr in https://github.com/IntelRealSense/librealsense/issues/6964#issuecomment-707501049

I've followed all these steps and it still isn't recognizing pyrealsense2 in the python import. I got up to number 8.
In /usr/lib/python3.6/site-packages there is no pyrealsense2 folder.
I'll try doing the cmake and make commands and seeing how it goes but wanted to post this here in case someone is running into the same issue. This experiment is being done on the original jetson nano running on the latest jetpack and ubuntu 20. I have python3.8 also installed and sure that might be an issue but idk its late.

Hi @josh-github1  May I begin by asking whether librealsense and the RealSense Viewer program are working correctly after installation has completed?
Also, is the information in the link below by another Nano user of this installation guide helpful to your problem?
https://github.com/IntelRealSense/librealsense/issues/7775#issuecomment-730685749

There was a Jetson case involving AttributeError: module 'pyrealsense2' has no attribute 'pipeline' where the solution in that particular case was found to be to copy .so files to the same directory that the user's project script was in.
#7078 (comment)
This same approach of copying the .so files into the same directory as the user's project script also worked in another case:
https://support.intelrealsense.com/hc/en-us/community/posts/360048396854/comments/360012486393

There was a Jetson case involving AttributeError: module 'pyrealsense2' has no attribute 'pipeline' where the solution in that particular case was found to be to copy .so files to the same directory that the user's project script was in.
#7078 (comment)
This same approach of copying the .so files into the same directory as the user's project script also worked in another case:
https://support.intelrealsense.com/hc/en-us/community/posts/360048396854/comments/360012486393

Excellent news that you found a method that worked for you.  Thanks for the update!

Hi @josh-github1   Do you require further assistance with this case, please?   Thanks!

Case closed due to no further comments received.


Target the Parent of an Element Using jQuery Bug

In the page, it only says Undefined
What should i do

Click Reset.

Happy Coding.


Swap available in UI but not in SDK

A swap on Avalanche between axlUSDC<>USDC can be seen in official curve UI but the curve sdk returns no route for this.
Assets are:
https://snowtrace.io/token/0xfab550568c688d5d8a52c7d794cb93edc26ec0ec
https://snowtrace.io/address/0xb97ef9ef8734c71904d8002f8b6bc66dd9c48a6e

Share code snippet please


Fix issue preventing import.git from syncing due to divergent branches

PR Description
Consider this situation:

import.git is configured to use branch A.
Alloy is then stopped, and its new configuration is configured to use branch B.
If branches A and B are divergent, import.git will fail to switch to branch B because it's going to try to pull A into B.

The user will get a "failed to update repository \"git@github.com:org/repo.git\": non-fast-forward update" error.
I suppose the same thing could also happen if a force push is done to the branch Alloy is monitoring.
Since Alloy is only reading from the repo, but not writing, there is no point in trying to do a pull. It's much easier to just do a fetch and checkout.
PR Checklist

[x] CHANGELOG.md updated
[ ] Documentation added
[x] Tests updated
[ ] Config converters updated

I think this was my original approach but it had issues with switching branches and accessing HEAD. Hence the test failing.

I think this was my original approach but it had issues with switching branches and accessing HEAD. Hence the test failing.

@mattdurham  Thank you, I had a look at the Agent PR which introduced pull just now. I think the default value for "revision" shouldn't be something like "HEAD". HEAD refers to the tip of the current branch. But what is the current branch in import.git? :) There is no config attribute to set it, so I'm not sure how it gets chosen. Maybe the remote tells it what the default branch is.
I think we should simply not allow values such as "HEAD", "FETCH_HEAD", etc to be the "revision". I updated my PR accordingly - would you like to take a look please? If you agree with it, I can update the docs and I'll also list it as a breaking change in the changelog.
If a user is able to set "revision" to something like "HEAD", "FETCH_HEAD" etc, then they'd be relying on some internal quirk of how import.git works, which wouldn't be correct.

HEAD isn't a quirk of import.git but fundamental feature for git. That being said I am not against saying you cant use the pointer style aliases and instead have to use a non alias value.


Fix tests (testing Travis CI)

Woo-hoo! It looks like Travis works correctly and the tests (only two that we have now) are green.


Migrate pierreccesario.com to Terraform IaC

pierreccesario.com's Cloudflare and S3 infrastructure was provisioned before Terraform was used.
Attempt to migrate its resources over to Terraform.
If migration is not possible, then manually remove S3 buckets and Cloudflare records before applying Terraform configuration.

Successfully imported resources

References:

Article
Official Terraform import instructions
Cloudflare API

Forgot to create a new branch for PR, will close with this comment ._.


docker run dart cmd crash

dart info:
docker run -it --rm dart:3.2.0 dart --version
Dart SDK version: 3.2.0 (stable) (Tue Nov 14 18:26:59 2023 +0000) on "linux_x64"

system info:
linux
docker 19.03.14

error:
run
docker run -it --rm dart:3.2.0 dart pub get
get crash
../../runtime/bin/eventhandler_linux.cc: 420: error: Failed to start event handler thread 1

===== CRASH =====
si_signo=Segmentation fault(11), si_code=1, si_addr=0x10
Aborting reentrant request for stack trace.

The latest version of docker has no problem. It seems that there is something wrong with the old version of 19.
If you use dart:3.0.6, there is no problem

Given that this works on newer Docker I would assume some sort of Docker problem (other possibility is resource limits
/ resource exhaustion?) so I am going to close this.
The code in question did not change for 10+ years - so it's unclear why 3.0.6 vs 3.1.0 would make a difference.
If you really want to know what failed you can strace dart under Docker and see if we can identify the failed syscall.

I have found the cause of the problem, and there is nothing that Dart can do to make older docker versions (20.10.9 and earlier) work again.
The crash is in glibc version 2.34 or higher, which started using the clone3 syscall.  A workaround is to start the docker dart  container with the --security-opt seccomp=unconfined option.  This makes the clone3 syscall return unsupported rather than failed, so the fallback to clone works.
I don't think it is practical to start building Dart with a much older version of glibc, especially with this workaround available.
Details of the underlying failure are at https://pascalroeleven.nl/2021/09/09/ubuntu-21-10-and-fedora-35-in-docker/

The libc library is dynamically linked to the dart executable, so the failing higher glibc version is coming from the sysroot (libraries) in the docker image, not the Dart files in that docker image.
Copying the just the dart directories from the official Dart docker image into an older Ubuntu docker image might also be a workaround.

The libc library is dynamically linked to the dart executable, so the failing higher glibc version is coming from the sysroot (libraries) in the docker image, not the Dart files in that docker image.
Copying the just the dart directories from the official Dart docker image into an older Ubuntu docker image might also be a workaround.

Thank you very much. I'm already using the latest version of docker.


Accordion Component

Here is the design documentation link: https://www.figma.com/file/RrcLH0mWpIUy4vwuTlDeKN/Baklava-Design-Guide?type=design&node-id=15548%3A20910&mode=design&t=25ieOea9LEuP8Yzk-1

Native accordion
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/details
https://open-ui.org/components/accordion.explainer
https://12daysofweb.dev/2021/details-summary/

@buseselvi, is the title any hover action?

@fatihhayri yes, I added hover in the accordion list variant at the bottom of the document.

I made accordion component with native HTML element <details> and <summary>.
https://codepen.io/fatihhayri/pen/bGzYYWv?editors=1100
Note: name attribute only Chrome 120, Safari 17.2. support. (this time)

@Enes5519 do we have progress on this issue?

Great work, Enes! Here are some suggestions for the ADR:

The 'accordion' prefix is not needed for the event name bl-accordion-toggle.
I believe we can omit the variant for bl-accordion-group. In instances of standalone usage, there doesn't seem to be a need to enclose accordions within the accordion group. We can simply use bl-accordion-group for lists and bl-accordion for standalone scenarios.
The option to make an accordion either standalone or within a list may be marked as disabled, although this detail is not reflected in the Figma. We can explore this further during the design meeting with @buseselvi.
We could consider renaming auto-collapse to a shorter alternative like multiplefor simplicity.

I added the disabled versions to the design guide. Thank you @AykutSarac 🙏🏻

Thx for review @AykutSarac

Done ✅
I added it in case you want auto-collapse in your accordions and also want them to stand apart. We can change the name of the standalone variant if it creates confusion.
Done ✅  (Thx @buseselvi )
multiple does not explain what it does, I can update it if there is an alternative.

Thx for review @AykutSarac
2. I added it in case you want auto-collapse in your accordions and also want them to stand apart. We can change the name of the standalone variant if it creates confusion.
4. multiple does not explain what it does, I can update it if there is an alternative.

I'm not sure if it makes sense to auto-collapse if they are standing apart, I mean they should be using the list in this case. So, I offered to use bl-accordion-group only for lists without the variant. We can discuss this further at the weekly update.
Agreed, couldn't think of anything better. auto-collapse sounds ok.

Separate accordions will need to write their own for auto-collapse. We talked with @buseselvi, but we can also bring it to the weekly agenda.
thx

I updated it as we discussed at the weekly meeting.

Remove standalone variant from bl-accordion-group.

Since auto-collapse is enabled by default, it is not possible to turn it off as an attribute, so I will change the name to keep-open. @buseselvi @AykutSarac


Unable to set a maximum width and height as initial config or onResizeStop

So,
I am adding widgets to my dashboard. each widget has different settings and currently is working as I intented it to do it, with one only exception.
The only issue I have is that I am unable to set a maximum width and height for each widget.
I tried to block it setting a max-width and max-height with css but once the widget reach the maximum allowed with the css, the inline style overflow those settings and the resize button stops working.
I am trying to catch the event with the method onResizeStop, but I am unable (or I don't know how) to establish the width or height that I required as maximum.
Something like this:
HTML:
<section class="blotter-component"
  [(ngGridItem)]="itemGridConfig"
  [attr.widgetid]="widgetId"
  (onChangeStop)="resizing($event)"
>
 ...
</section>

Component:
resizing(event: NgGridItemEvent) {
    if (event.width > this.limits.maxW) {
      event.width = this.limits.maxW;
      this.itemGridConfig.sizex= 19;
    }
    if (event.height > this.limits.maxH) {
      event.height = this.limits.maxH;
      this.itemGridConfig.sizey = 17;
    }
  }

this.itemGridConfig is the gridItemConfiguration for each widget
this.limits contains the limits for height and width.
Thank you

Currently, the only way to limit the maximum sizes of items in the grid is to set the max_cols/max_rows properties on the grid of the maxCols/maxRows properties on the items. It looks like that's basically what you're trying to achieve by setting sizex=19 and sizey=17 in your event handler, no?
Can I ask what the rest of your config looks like?

 Didn't read that part on the documentation. That's why it wasn't working.
I added the maxCols, maxRows / minCols, minRows and is working perfectly now.
My settings for the grid are this:
private gridConfig: NgGridConfig = <NgGridConfig>{
    draggable: true,
    resizable: true,
    max_cols: 39,
    max_rows: 0,
    visible_cols: 0,
    visible_rows: 0,
    min_cols: 1,
    min_rows: 1,
    col_width: 50,
    row_height: 50,
    cascade: 'left',
    min_width: 30,
    min_height: 50,
    fix_to_grid: true,
    auto_style: false,
    auto_resize: false,
    maintain_ratio: false,
    prefer_new: true,
    zoom_on_drag: false,
    limit_to_screen: true,
    margins: [5]
  };

For the items:
private gridDefault: NgGridItemConfig = {
    borderSize: 30,
    col: 1,
    dragHandle: ".dragHandler",
    draggable: true,
    fixed: false,
    height: 530,
    left: 5,
    maxCols: 15,
    maxRows: 15,
    minCols: 11,
    minRows: 8,
    payload: 4,
    resizable: true,
    resizeHandle: ".resizeHandler",
    row: 1,
    sizex: 11,
    sizey: 9,
    top: 5,
    width: 650,
  };

Thank you for an awesome library. I searched for several days  a library that could work with dynamic components without luck, but this one worked like a charm.
Cheers!


Visibility fix in api method macro

Something that triggers clippy with unused methods

My bad, we did not need this


Allow specifying phrases that should be "read out"

A news organization might be used to writing abbreviations for the web that don't make sense when spoken.
For example, "DOT" in a news story should probably be spoken as "Department of Transportation," not "dot."
It would be nice to provide an interface that allows sites to list words or phrases that shouldn't be read as-is and to specify how those should be replaced.
The plugin might also be able to provide some starter replacements, like "in the morning" for "a.m.", although these should be removable if provided.

Thoughts on implementation:

Add a field in the main Alexa Skill Settings screen to define a list of words that get pronounced differently, maybe a format such as:
a.m. | in the morning, DOT | Department of Transportation
open to better methods than a pipe character separator
2.) on save_post, filter the content and insert the  SSML for any matches, so: DOT

That way it only runs once, and in conjunction with #32 allows for reading out specific phrases in a user defined way


Added lifecycle configuration support in bucketspace_fs

Explain the changes

added set\get\delete for lifecycle configuration

refactored bucket_lifecycle_configuration in common_api

added lifecycle_configuration_rules in nsfs_bucket_schema.js

A separate commit with beautify changes

Testing Instructions:

set\get\delete lifecycle policy using S3 API

[ ] Doc added/updated
[X] Tests added

@dannyzaken LGTM! Can you please just add it to the S3ops.md docs? we have there a list of supported S3 bucket operations


Feedback on request

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az ad sp list
Errors:
400 Client Error: Bad Request for url: http://localhost:50342/oauth2/token
Traceback (most recent call last):
python3.6/site-packages/knack/cli.py, ln 215, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 654, in execute
    raise ex
cli/core/commands/__init__.py, ln 718, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 709, in _run_job
    cmd_copy.exception_handler(ex)
...
python3.6/site-packages/msrestazure/azure_active_directory.py, ln 486, in get_msi_token
    result.raise_for_status()
python3.6/site-packages/requests/models.py, ln 940, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: http://localhost:50342/oauth2/token

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az ad sp list --output {}

Expected Behavior
Environment Summary
Linux-4.15.0-1095-azure-x86_64-with-debian-stretch-sid (Cloud Shell)
Python 3.6.10
Installer: DEB

azure-cli 2.12.0

Extensions:
ai-examples 0.2.3

Additional Context

add to S176