ShiftLeft pipeline support request by Fuming Zhang: Sync Swagger to GitHub is not creating diff PR

Relevant post on API Service Toolset Teams channel.
I had calls with Ray Chen about it. Recordings available on https://www.microsoft365.com/mycontent
Recording 1 - about ShiftLeft in general
Recording 2 - debugging this issue specifically
Urgency:

[Tuesday 3/14/2023 7:55 PM] Fuming Zhang
got it, it takes 6 days before service is ready to perform swagger release, so we could wait a little bit

Tasks that didn't diff anything:

Sync TSP to GitHub
Sync Swagger to GitHub

Hey @konrad-jamrozik, may I ask how is the progress of this bug fix? We want to do swagger release as soon as possible.

@FumingZhang apologizes for the delay! I am going to work on this over the next 12 hours, expect an update from me in that time window.

I have advised @FumingZhang to create a PR directly to the azure-rest-api-specs repo. This is in violation of the ShiftLeft rules, as can be seen e.g. here: https://github.com/Azure/azure-rest-api-specs/pull/22309#issuecomment-1404107367
However, the rule assumes timely support if issues like this one arise, which is currently not possible, as I am new to the codebase and I am still working through it, making progress, albeit slow. Fuming confirmed he is working on submitting a PR to the main repo.

Manually created PR #23189.

I submitted a PR against ShiftLeft source adding diagnostic info to help diagnose the root-cause:
https://devdiv.visualstudio.com/DevDiv/_git/openapi-pipeline/pullrequest/459818

@FumingZhang I believe I found the root cause. In our call we had few days ago you gave as an example that the following directory should have been included in the diff PR:
/swagger/containerservice/resource-manager/Microsoft.ContainerService/aks/stable/2023-02-01
I ran another build from that branch with additional diagnostic output which you find here. In this build you can see:
CaldDirForArm='swagger/containerservice/fleets.management' CadlDirForDataplane=''
Executing Apply-PatchTsp src='/home/vsts/work/1/s/../Azure-azure-rest-api-specs-pr/swagger/containerservice/fleets.management' tar='specification' ServiceName='containerservice' tspFolder='fleets.management' targetTspFolder='specification/containerservice/fleets.management'
Executing Remove-DeletedTsp src='/home/vsts/work/1/s/../Azure-azure-rest-api-specs-pr/swagger/containerservice/fleets.management' tar='specification/containerservice/fleets.management'
removing whole folder specification/containerservice/fleets.management
Skipped processing CadlDirForDataplane as it is an empty string
Executing Push-Changes branch='containerservice/official/aks-api-release' commitMsg='[AutoSyncTsp] bae8054ad7e Merged PR 7768257: Revert "Merged PR 7726419: [API Release] 2023-03-02-preview Create Swagger ba...'
No tsp changes

Please note the:
CaldDirForArm='swagger/containerservice/fleets.management'
this is a different path than the example you gave, of:
/swagger/containerservice/resource-manager/Microsoft.ContainerService/aks/stable/2023-02-01
it differs by suffix: fleets.management vs resource-manager.
Hence it seems to me that if you would change the value of the CadlDirForArm variable, which currently is swagger/containerservice/fleets.management, the PR should be properly created.
Let me know if this solves the issue, thanks!

@konrad-jamrozik, there are 2 sub-services under the Microsoft.ContainerService namespace, the first one is aks (with path /swagger/containerservice/resource-manager/Microsoft.ContainerService/aks) developed in the traditional way (modify the json file directly), and the second one is fleet (with path /swagger/containerservice/resource-manager/Microsoft.ContainerService/fleet) developed with cadl.
The API changes of these two services will be released independently, so we defined the manifest file like
"official/aks-api-release": {
    "includedPaths": [
        "containerservice/resource-manager/Microsoft.ContainerService/aks/*"
    ]
},
"official/fleet-api-release": {
    "includedPaths": [
        "containerservice/resource-manager/Microsoft.ContainerService/fleet/*"
    ]
}

This time, I am trying to release aks related changes via branch official/aks-api-release and I think it should have nothing to do with fleet or cadl (not sure what Tsp is short for, but I guess it's related to cadl).

@FumingZhang apologies, my mistake. I somehow assumed the issue is that both Sync Swagger to GitHub and Sync Tsp to GitHub do not create any diff, but obviously you meant only Sync Swagger to GitHub. Hence, I will need to work a bit more on getting the root cause, but relevant code to which I need to add diagnostic logging is very similar, so expect a reply from me within next 24h (i.e. my upcoming business day).
I believe Tsp stands for TypeSpec which is the new name for Cadl.

@FumingZhang I submitted a PR with experimental bugfix, waiting for approval:
Pull Request 460255: Path matching bugfixes in SyncTspToGitHub.ps1 & SyncSwaggerToGitHub.ps1 for backslashes, and for case of non-existent excludedPaths.

Thanks for the update

@FumingZhang OK looks like the bug is finally fixed!
PR with the last bugfix
Successful build
Resulting PR, confirming the fix works
Now I am going to look into https://github.com/Azure/azure-sdk-tools/issues/5788
I am closing this issue as resolved, but let me know if something is still not right.

Cool, thanks @konrad-jamrozik.


how do i run this? both the documentation and example in git give different codes.

'NoneType' object has no attribute 'outputs' in trainer.build_model()
This is the code so far
import sys

#from pykg2vec.data.kgcontroller import KnowledgeGraph
from pykg2vec.utils.kgcontroller import KnowledgeGraph

from pykg2vec.config.config import Importer, KGEArgParser
#from pykg2vec.common import Importer, KGEArgParser
from pykg2vec.utils.trainer import Trainer

def main():
    # getting the customized configurations from the command-line arguments.
    args = KGEArgParser().get_args(sys.argv[1:])

    # Preparing data and cache the data for later usage
    #knowledge_graph = KnowledgeGraph(dataset=args.dataset_name, custom_dataset_path=args.dataset_path)

    knowledge_graph = KnowledgeGraph(dataset=args.dataset_name, negative_sample=args.sampling)

    knowledge_graph.prepare_data()

    # Extracting the corresponding model config and definition from Importer().
    config_def, model_def = Importer().import_model_config(args.model_name.lower())

    print('-----------------------------------')
    print(model_def)
    print(config_def)
    print('-----------------------------------')

    config = config_def(args=args)

    print(config)

    ####model = model_def(**config.__dict__)
    model = model_def(config)

    print(model)

    print('model done')

    #### Create, Compile and Train the model. While training, several evaluation will be performed.
    #trainer = Trainer(model=model, debug=True)
    #trainer = Trainer(model, config)
    trainer  = Trainer(model=model,debug=args.debug)

    trainer.build_model()
    trainer.train_model()

if __name__ == "__main__":
    main()

I run it with arguments as follows
-mn TransE -ds fb15k

Hello, sunil
yea previously the code in user document is kind of inconsistent with the repo. However, right now we have made them synced.
Can you try the one shown here again? https://pykg2vec.readthedocs.io/en/latest/auto_examples/train.html
Thanks!

thank you it ran but I see no figures generated


[teslamate] Update Chart.yaml to new version for Teslamate

I missed the requirement to update Chart.yaml for Teslamate

Description of the change

Benefits

Possible drawbacks

Applicable issues

fixes #

Additional information

Checklist

[ ] Chart version bumped in Chart.yaml according to semver.
[ ] Title of the PR starts with chart name (e.g. [home-assistant])
[ ] (optional) Chart is using our common library as a dependency.
[ ] (optional) Variables are documented in the README.md

You do not need to create a new PR for this, please update your other PR. Thanks


Android CI based on GitHub Image

Alternative to the Nix based setup of #702
Closes #622 and #606 as well but uses the default Ubuntu image from GitHub.

GitHub actually includes many relvant packages into their standard ubuntu images. See here for a list of versions: https://github.com/actions/runner-images/blob/main/images/linux/Ubuntu2204-Readme.md
Maybe we could just use the same versions as them and use directly their images...

android-ci.yml: https://github.com/louwers/maplibre-gl-native/actions/runs/3971563148
android-ci-pull.yml: https://github.com/louwers/maplibre-gl-native/actions/runs/3971584067

Woke up in the morning. Got time to review! :D
It sounded way cooler in my head.
Reviewing by EOD

@thehoneymad Thanks for your review.
We can either merge this PR or #702.
Oliver and Ovi will take a look if they can run Nix locally, since one of it's advantages is that you can re-create an exact copy of the CI environment. Which approach has your preference?

I am still in favour of merging this. And then reading the Nix PR on top of this. That allows us to have a stable build with pure GitHub Actions first.
And then if Nix pans out, we can use that too. :)

I agree. We should not limit ourselves on Nix. I'm almost sure I'll not use it 😉 (but I also do not plan to develop the Android SDK).


Automatically update GIR files

https://wiki.gnome.org/Apps/Nightly
Gir files could be taken from gnome nightly like gtk-rs does: https://github.com/gtk-rs/gir-files/blob/54ae87ae2ecec6cde7c6e7a7c50301f58fd2e72c/.github/workflows/regenerate_scheduled.yml#L1
can we use the stable runtime with stable gir files? See: https://docs.flatpak.org/en/latest/available-runtimes.html
https://gitlab.gnome.org/GNOME/gnome-build-meta/-/pipeline_schedules
Should we depend on gtk-rs instead?

Fixed with: https://github.com/gircore/gir-files/pull/2


Ctrl Shift S should be save all (default), not save as

VSCode Version: Code 1.12.1 (f6868fce3eeb16663840eb82123369dec6077a9b, 2017-05-04T21:26:50.689Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

csharp
ms-vscode
0.3.7

debugger-for-chrome
msjsdiag
0.4.8

debugger-for-edge
msjsdiag
0.4.1

Steps to Reproduce:

change text in editor (arbitrary file/files)
Ctrl+Shift+S (should NOT open a Save as DLG, it should just save all without prompt
This has been the WIndows standard forever, why change it to save as?

Duplicate https://github.com/Microsoft/vscode/issues/1760


Code fix for K8s fetch failure

Previous code was not working. A final fix to make this repo working and then we can freeze this repo. Request owner to merge this change as it fixes previous bugs.

Deprecated, please use the k8s modules that ship with Ansible


Cannot access dataflint UI through Knox gateway URL

Describe the bug
Currently, we are using Spark on AWS EMR. We are also using Knox Apache gateway as a gateway for our cluster, we will access spark history, livy, and dataflint through Knox gateway URL. But after installing successfully dataflint on our EMR, we cannot access it. After I checked it, I saw that data-flint is trying to call API directly without through gateway.
GET https:///api/v1/applications/<application_id/1/environment 404 (Not Found)
it should be call to: https:///gateway//sparkhistory/history/<application_id>/1/environment/
So I think it might be from this config: https://github.com/dataflint/spark/blob/main/spark-ui/src/services/SparkApi.tsx#L78-L80
Please help to check it.
Thanks in advance
Environemnt
spark verison: 3.4.1
platform: EMR
To Reproduce
Steps to reproduce the behavior: it's hard to reproduce, we need to install knox apache gateway
Expected behavior
Dataflint UI should be loaded successfully
Screenshots

Hi @anle20, really sorry for missing your issue. Did you manage to fix your problem?

Hi @menishmueli , Thanks for your response. Regarding this issue, we resolved it by rewriting Knox Gateway. But we've got other problems.
1/When we click "TO HISTORY SERVER", it cannot back to the spark history page.
for example:
spark history page:  https://knox.com/gateway/sparkhistory
dataflint: https://knox.com/gateway/sparkhistory/history/<application_id>/dataflint
-> When clicking to SPARK HISTORY, it will access to: https://knox.com/history instead of https://knox.com/gateway/sparkhistory
I think it is related to this code: spark history page
2/ The logo.png cannot load

@anle20 I have encountered the exact same problem. can you elaborate on what did you change in the knox gateway. I am trying to use the latest version(currently 0.2.5)

Hi @ef1236 , you need to rewrite your service.xml and rewrite.xml on your knox to route all requests to /api/v1 -> sparkhistory/api/v1.


ngx-monaco-editor : unable to find loader.js file from node_modules

I want to include ngx-monaco-editor in my project to create a calculator-like screen on the front-end which shows the text entered as the user clicks on the corresponding buttons.
I have installed ngx-monaco-editor (version 6.0.0) for the same.
The dependencies have been included in app.module.ts and I have added the glub to assets in the angular.json file.
app.component.hlml :
<ngx-monaco-editor [options]="editorOptions" [(ngModel)]="code">
app.component.ts :
editorOptions = {theme: 'vs-dark', language: 'javascript'};
code: string= 'function x() {\nconsole.log("Hello world!");\n}';
The editor is not getting rendered. I'm instead getting an error that my loader.js file is not getting found.
http://localhost:4200/assets/monaco/vs/loader.js net::ERR_ABORTED 404 (Not Found) @base-editor.js:65
The loader.js file is located at node_modules/ngx-monaco-editor/assets/monaco/vs/loader.js and I'm able to navigate to it.
The error is occuring at base-editor.js, line 65. The location for this file is node_modules/ngx-monaco-editor/base-editor.js
@atul-wizni I have followed the step-by-step setup for ngx-monaco-editor as demonstrated here https://www.npmjs.com/package/ngx-monaco-editor/v/6.0.0
And I have also referred the sample project https://github.com/atularen/ngx-monaco-editor/files/2016365/Archive.zip to the point.

Changing the input configuration in angular.json enabled the files to be copied over to the dist folder.
Changing the output configuration allowed the loader.json file to be found.

The loader.js file is located at node_modules/ngx-monaco-editor/assets/monaco/vs/loader.js and I'm able to navigate to it.
I dont have assets folder only inside ngx-monaco-editor at node modules.
Can anyone let me know, is there something wrong with the installation I did?


[spk] Update more bedrock projects to use the introspection tool

As a: developer in an organization
I want: all my pipelines of a Bedrock project sending data to a Spektate storage
So that: I can introspect my services
Describe the solution you'd like:
Right now, the only project being used by introspection tool is the prototype project for hello-bedrock and hello-spektate. Start bringing more real projects to use the spk introspection tool.

[ ] a working example end-end of private ADO repos working in a private ADO pipeline
[ ] service introspection tool dashboard and cli working

Does this require updates to documentation?: No

a working example end-end of private ADO repos working in a private ADO pipeline

@samiyaakhtar @sarath-p This sounds like an integration test environment.
We should combine it with the integration env we want to support service management scenarios. How are thinking of creating this environment?
Something that can be scripted out (e.g. create AzDO project, create repos, setup a sample application that pushes to ACR, Sets up AKS)?
^ This can affect the effort of this task. If it makes senses to combine this with other integration environments let's think about it.

service introspection tool dashboard and cli working

Given that the above is setup this is now just a simple test. I see these two task list items as being a 90/10 split in terms of effort. Thoughts?

@andrebriggs this was created as an effort to help on-board their projects to use the introspection tool after our Tuesday meeting. This isn’t for integration testing, I’m happy to close it if you don’t see it fit anymore, but I opened it because we had decided to do this task next week.

I think creating specific issues which are blocking the usage of the product would help to address gaps. We can brainstorm the existing gaps to come up with actionable issue list.

Closing this since we had a successful run of cli and dashboard on 10/24


Mode flash with right click

This area is reserved to developers.
If you are not a developer, you are in the wrong place and your issue will be closed.
For discussions, feedbacks, ideas and new fixtures, go to:
http://www.qlcplus.org/forum/index.php
and provide as many details as possible regarding your findings.
Before reporting new issue, please discuss it on the forum. Let's keep the issues tidy!
If you are a developer, you are supposed to have built QLC+ from sources, and started to debug the issue you have found.
Therefore, please provide a crash log or references to the code that are supposed to help the QLC+ developers to resolve the issue.
Also, if your issue concerns a specific platform, please indicate it in the issue title like [platform] description.

It is possible to implement flash mode by right clicking on the button (in virtual console).
Please, it's a function that I need a lot.
Sorry if this is not well located.

Please use the forums as indicated in the message you ignored.


[AutoPR] sql/resource-manager

Created to accumulate context: sql/resource-manager

(message created by the CI based on PR content)
Installation instruction
Package azure-mgmt-sql
You can install the package azure-mgmt-sql of this PR using the following command:
pip install "git+https://github.com/Azure/azure-sdk-for-python@restapi_auto_sql/resource-manager#egg=azure-mgmt-sql&subdirectory=azure-mgmt-sql"
You can build a wheel to distribute for test using the following command:
pip wheel --no-deps "git+https://github.com/Azure/azure-sdk-for-python@restapi_auto_sql/resource-manager#egg=azure-mgmt-sql&subdirectory=azure-mgmt-sql"
If you have a local clone of this repository, you can also do:

git checkout restapi_auto_sql/resource-manager
pip install -e ./azure-mgmt-sql

Or build a wheel file to distribute for testing:

git checkout restapi_auto_sql/resource-manager
pip wheel --no-deps ./azure-mgmt-sql

Direct download
Your files can be directly downloaded here:

azure-mgmt-sql-0.9.0.zip
azure_mgmt_sql-0.9.0-py2.py3-none-any.whl


digest: use github.com/minio/sha256-simd

https://github.com/minio/sha256-simd simdizes sha256 computation on amd64 and arm64.
Micro benchmark: ctr images pull docker.io/library/fedora:latest on Intel(R) Core(TM) i7-1068NG7 CPU @ 2.30GHz (with sha_ni), 4 cores
Before:
9.110s
9.172s
9.390s

After:
8.609s
8.321s
8.654s

Hopefully this is could be a temporary change and Go can find an upstream solution. The performance improvement is significant enough to pull in though.

Hopefully this is could be a temporary change and Go can find an upstream solution. The performance improvement is significant enough to pull in though.

Looks like it was merged in Go master; https://github.com/golang/go/issues/53084 - see https://github.com/golang/go/commit/f1b1557cf3c9ca5e3def849656f79f39041d0055 (not sure if more is needed after that>), which would mean this will be in go1.20?
If so, should we add a TODO comment to remove it after we switched to go1.20?

Looks like it was merged in Go master; golang/go#53084 - see golang/go@f1b1557 (not sure if more is needed after that>), which would mean this will be in go1.20?

crypto/sha256: add sha-ni implementation ( https://go-review.googlesource.com/c/go/+/408795/1 ) doesn't seem merged.
Not sure it will be merged before Go 1.20.

Added a code comment
func NewSHA256() hash.Hash {
        // An equivalent of sha256-simd is expected to be merged in Go 1.20 (1.21?): https://go-review.googlesource.com/c/go/+/408795/1
        return sha256simd.New()
}

Strange that github claims there are conflicts but doesn't list any file conflicts?

@AkihiroSuda Would it be relevant to have this new computation in BuildKit too?

Yes, I guess we can safely have this in BuildKit (and Moby, etc.)

This is pretty cool, but I wonder if this should be opt-in given that the performance advantage is fairly negligible unless you are doing a lot digesting.
This could even be a plugin that is disabled by default where the plugin init would register the hasher.

opt-in

Any drawback to enable SIMD by default?

This could even be a plugin that is disabled by default where the plugin init would register the hasher.

I'm not sure. The plugin will be NOP when https://go-review.googlesource.com/c/go/+/408795/1 is merged.

This is pretty cool, but I wonder if this should be opt-in given that the performance advantage is fairly negligible unless you are doing a lot digesting vs the breadth of code that relies on sha256.

Are you concerned mostly with bugs in this implementation or reliability of the CPU instructions or something else?

This could even be a plugin that is disabled by default where the plugin init would register the hasher.

Another alternative would be to make this a build tag.  Then there wouldn't be an extra plugin hanging around if this gets folded into Go.

Does the Go upstream have aarch64 support already?

Yes https://github.com/golang/go/blob/master/src/crypto/sha256/sha256block_arm64.s


Change HDMI input to PC when PC has already turned on instead of wake from sleep/hibernate or off (Feature)

Example from PS5:

Ps5 is ON/Hdmi 2, PC is on Hdmi 1, when TV is at Hdmi 1, press the PS button on ps5 controller, TV will switch to Hdmi 2 (Ps5)

Is this possible to do this for PC by let's say making a shortcut or exe for whenever triggering it, it will do the exact same behavior like on the Ps5 controller (Can just set such shortcut/exe to a button on the pc controller)
Thanks for your hard work!

Hi,
That is possible, by using the command line. The following will switch input for Device1 to HDMI1:
Create a shortcut/script/macro pointing at executing: \LGTV Companion.exe -sethdmi1 Device1
Full reference for the command line parameters can be found in the readme

Hi,
That is possible, by using the command line. The following will switch input for Device1 to HDMI1:
Create a shortcut/script/macro pointing at executing: c:\program files\LGTV Companion\LGTV Companion.exe -sethdmi1 Device1
Full reference for the command line parameters can be found in the readme

It works! Thank you so much!


Create sample data

This should be done using a plugin.
The homepage will be used to showcase all module positions

https://github.com/C-Lodder/lightning/commit/76542fe29b51ffe21dfd9c5008faabb48a972d7d


Error from Aggregate Query: Undefined property: Mongo_db::$read_preference

Hi,
When running an aggregate query, I am getting the following error. It is the second query i am running on the same page. I can't seem to get rid of this error. Any ideas?
A PHP Error was encountered
Severity: Notice
Message: Undefined property: Mongo_db::$read_preference
Filename: libraries/Mongo_db.php
Line Number: 3943
Thanks!
Brian

Hi, @bthart! Update library to the last version. It’s error was been already fixed!


Update Jackson Databind to Latest Version

Description
Updates jackson-databind to <IP_ADDRESS> to resolve CVE.

API change check
API changes are not detected in this pull request.


Add PWM dtbs

BB-PWM0-00A0.dts
BB-PWM1-00A0.dts
BB-PWM2-00A0.dts
Connects to user request.
 Front conversations

Fixed by https://github.com/resin-os/resin-beaglebone/pull/128

Fix merged. The requested overlays will be available in the next beaglebone release.
Closing this item


Documentation practices, the Trunk website, and the guide book

Before I say all this, I want to acknowledge the (often unthanked) work that the trunk team has done and continues to do to provide this tool for the community. Thank you! Open source maintenance is not easy.
The Problem
The issue I have is twofold:

As a new user of Trunk trying to do slightly more advanced things, it has been difficult for me to find the information I need.

As someone who wants to contribute new functionality to Trunk, I see no consistent practice dictating how I should document that functionality.

Problems as a new user

The website does not document all existing functionality of Trunk.

It is not obvious that there is also a guide, nor is it obvious why there is also a guide.

The guide contains more information than the website in some places, but also the website contains more information than the guide in some places.

It's easy for the not-so-careful reader (myself) to miss the link in https://trunkrs.dev/configuration/ that points to a complete example Trunk.toml, leaving them thinking that you can only configure a few things like a proxy.

On several occasions I have had to read the source code to figure out whether or not Trunk is at all capable of doing a particular thing.

Problems as an aspiring contributor

There is no guideline in CONTRIBUTING.md for when, where, or how to write documentation.

The existence of multiple different documentation sources makes me confused about where to put my new documentation

Next Steps
Here is what I would like to see as both a user and contributor:
1) Make it clear whether the website or the guide is the authoritative documentation resource.

If it's the website, then get rid of the guide and put all of its information in the website. This would be my preferred option.

If the guide is supposed to be the one authoritative source, then make that very clear on both GitHub and the website.

Alternative: generate the website contents from the guide, or vice versa.

2) Paste the example Trunk.toml into the authoritative documentation source's "Configuration" page.
I believe this is possible with mdbook. Then users do not have to travel to the GitHub to see what is possible to configure, and they have no chance of missing the GitHub link like I did either.
3) Make a policy for how to document new functionality, put it in CONTRIBUTING.md, and enforce it in PRs.
Developing such a policy does not happen overnight, and it can improve over time. But having anything at all will both lower the bar for contribution and improve the quality of the documentation.

It's easy for the not-so-careful reader (myself) to miss the link in https://trunkrs.dev/configuration/ that points to a complete example Trunk.toml

I do have to agree on that point. AFAICT this is the only place where all attributes are defined. Its fair enough to have that in the repo, where it can be tested, and where contributions can easily add new fields. But the documentation doesn't specifically say that its the only complete 'reference', and having it included inline in a paragraph, alongside stuff such as links to external projects etc, occasionally it took me a while to find even when I knew what I was looking for.
Re. Point 1. I think it would be good to declare an authoritative source, and ask contributors to add to in Contributing.md (Point 3). When it comes to unifying these, it would sure be great, but not required at this point, IMO.

I completely agree. The idea was to move away from the documentation in the website, and move to the guide instead, as it allows for a more complex documentation.
I'd like to avoid breaking existing links. So what that would require, are some redirects from previous pages to the guide. Not necessarily to the matching page, but at least to the guide in general.
I'd really appreciate any help on this.


[Suggestion] Group shared or linked containers

Ok not sure if already requested. But i would love it if there was an option to have players be able to access group storage such as a bag of holding or a keeps storage room etc together. Is it possible to have an item that can be linked to a "container group actor" of sorts? so that everyone that has the linked item in their sheet, can access it like a shared storage? would save so much time for people to not having to ask who has the bag of holding right now, is x item currently in there or does someone have it on their sheet etc.

Group actors and vehicles exist for this use case.

You can also use a document link to the shared actor in an item description or your character's biography to make it quicker to access.

These all sound like workarounds to me. Why open an item's description to get an actor link etc. what makes the current 5e containers so great is that they are at the top of inventory, icons you can click, and they open the corresponding bag immediately.
It would be great if that would be possible for a shared container. And if Group Actors are intended for this, then there should be a way for players to access them and treat them like bags/storage if they are designated as such, from within their inventory in a simple manner.
I know how often my players want to just take a quick look into a bag and see if they have the item they are looking for there, without having to ask the guy carrying the bag of holding if it's in there and then trade around. All that costs time and bogs down gameplay. Hence, why this is a suggestion for a new feature.

Originally the Group actor was created as a means to show a party's position during overland travel (that was the pitch anyway). It has grown in scope since then obviously.
It would be very nice if group actors that contain any given character actor were visible in an area near the containers on said character's sheet, and could be opened from there, and - ideally - would allow for easily adding or removing (i.e. not duplicating) an item between the two actors.
(The Group actor is an obvious choice for some sort of "trading" functionality to be put in the system in this way.)


Could not install Rb.Forms.Barcode 0.7.1

Hi,
I just wanted to install the nugets in PCL , Droid and iOS project and get that install error:

Could not install package 'Rb.Forms.Barcode 0.7.1'. You are trying to install this package into a project that targets '.NETPortable,Version=v4.5,Profile=Profile259', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.

Hope you can help my out.
thanks in advance
Eric

Do you try it to install from a windows system?
If that is the case we all not working with windows systems and only able to try it out on mac. So you could help us to get it working on a windows system
Here is the other issue from windows users
https://github.com/rebuy-de/rb-forms-barcode/issues/57

@fvermeulen Yes Visual Studio 2015 Enterprise including latest stable Nuget Package Manager, Xamarin SDK and SP. I guess the PCL profile has to be changes on my side. But to which one?

I digged through the source code and saw that your PCL i configured for that profile

TargetFrameworkProfile>Profile78

I will try to change my project, if no side effects popup!

@ericbrunner I use PCL-Profile7 and with that it works as well fine, but which one do you used?

@ I develop an Azure Mobile Apps Xamarin.Forms PCL App for Droid + iOS . The initial version was scaffolded by the Azure Portal with Profile=259.

@fvermeulen Well I tried to switc hto Profile7 but I get several compiler errors which is clear becasue the nugets are not compatible anymore. Ok I understand that you nuget is not compatible to Profile259 now.
Sorry, I will use ZXing.Net which is working.

yeah you have to retarget your packages. But that is possible. In Xamarin Studio you have to press a button for that. I don't know how that will be handled in Visual Studio.
I looked what is PCL259 for and PCL 7. In your case it makes more sense to use PCL 7, but anyway sorry that I cannot help you with your problem

@fvermeulen Could you send my a link or screenshot where I can do that migration in Xamarin Studio? Then I would copy my code temp. from TFS to MAC , switch to Profile7 and copy it back to my windows box.

@ericbrunner after changing your profile right-click on the Packages folder and select Retarget.

@rdoubleui Hi Thanks. I tried but I have no Retarget Command. I user the latest Xamarin Studio Community Edition. Do I have to install some Tools ?

The feature has been there from version 5.2, don't think that there's a different behaviour within the community edition. What nuget version are you using? I recall you need to have at least nuget 2.1

@ericbrunner were you able to solve the issue with XS?

@rdoubleui I currently try to retarget to Profile=7. I get back to you when I succeed.

Do we have any progress with this? I am having the same problem

So I changed it to profile 78 for adding the package, and then change it back to 111. I tested the demo the same UI from the sample on the repo and seems to be working just fine..
I dont know if changing it back to profile 111 is a good idea or no, hope someone can give an opinion on that.
Nice package, by the way, works pretty good!.

See the imports section below. This perfectly works for me using .net standard.
{
"dependencies": {
"Acr.UserDialogs": "6.5.1",
"Ansuria.XFGloss": "<IP_ADDRESS>",
"CarouselView.FormsPlugin": "4.4.6",
"FreshEssentials": "2.1.3",
"Humanizer": "2.2.0",
"libphonenumber-csharp": "8.8.4",
"Microcharts.Forms": "0.8.3-pre",
"Microsoft.AspNet.SignalR.Client": "2.2.2",
"Microsoft.Azure.Mobile.Analytics": "0.16.0",
"Microsoft.Azure.Mobile.Crashes": "0.16.0",
"Microsoft.Azure.Mobile.Push": "0.16.0",
"Microsoft.NETCore.Portable.Compatibility": "1.0.1",
"modernhttpclient.standard": "2.4.2",
"NETStandard.Library": "2.0.1",
"Newtonsoft.Json": "10.0.3",
"PCLStorage": "1.0.2",
"Plugin.Permissions": "2.1.0",
"Plugin.Share": "7.0.1",
"Plugin.VersionTracking": "2.0.1",
"Rb.Forms.Barcode": "0.7.1",
"Refractored.MvvmHelpers": "1.3.0",
"Rg.Plugins.Popup": "1.0.4",
"SkiaSharp": "1.59.1",
"SkiaSharp.Views.Forms": "<IP_ADDRESS>",
"Splat": "2.0.0",
"SQLite.Net.Async-PCL": "3.1.1",
"SQLite.Net.Core-PCL": "3.1.1",
"Toasts.Forms.Plugin-PCL": "3.2.0",
"Xam.Plugin.Connectivity": "3.0.3",
"Xam.Plugin.DeviceInfo": "3.0.2",
"Xam.Plugin.ExternalMaps": "4.0.1",
"Xam.Plugin.Geolocator": "4.1.2",
"Xam.Plugin.Hud": "1.0.5",
"Xam.Plugin.Media": "3.0.1",
"Xam.Plugins.Forms.ImageCircle": "1.8.1",
"Xam.Plugins.Messaging": "4.0.0",
"Xam.Plugins.Notifier": "2.1.0",
"Xam.Plugins.Settings": "3.1.1",
"Xamarin.Controls.SignaturePad.Forms": "2.2.0",
"Xamarin.FFImageLoading": "2.2.22",
"Xamarin.FFImageLoading.Forms": "2.2.22",
"Xamarin.FFImageLoading.Transformations": "2.2.22",
"Xamarin.Forms": "<IP_ADDRESS>2",
"Xamarin.Forms.Maps": "<IP_ADDRESS>2",
"Xamarin.Forms.Pages": "<IP_ADDRESS>2",
"Xamarin.Forms.Theme.Base": "<IP_ADDRESS>-pre1",
"Xamarin.Forms.Theme.Dark": "<IP_ADDRESS>-pre1",
"Xamarin.Forms.Theme.Light": "<IP_ADDRESS>-pre1"
},
"frameworks": {
"netstandard1.4": {
"imports": [
"portable-net45+win81+wpa81+wp8",
"portable-net45+win8+wp8"
]
}
}
}

In my case I changed my project to profile 78 and still the package cannot be added, still having the same problem. The NuGet package is getting installed on Android and iOS projects but not on PCL
Windows System
Visual Studio 2017
Xamarin Forms <IP_ADDRESS>635
Microcharts.Forms 0.7.1


[Bug]: Foundry cannot install Uniswap V4 periphery contracts like Hooks.sol

Describe the bug
Foundry cannot install Uniswap V4 periphery contracts for hook development.
Expected Behavior
Expected:
Should compile contracts using Uniswap V4 periphery contracts.
Actual:
Error:
Failed to resolve file: "/home/marcuswentz/github/ethnyuniswapv4hooks/lib/v4-periphery/lib/v4-core/contracts/libraries/Hooks.sol": No such file or directory (os error 2).
 Check configured remappings..
    --> "/home/marcuswentz/github/ethnyuniswapv4hooks/src/TradingDaysHook.sol"
<EMAIL_ADDRESS>
To Reproduce
Foundry install library Uniswap V4 library
forge install https://github.com/Uniswap/periphery-next --no-commit

Foundry compile contracts
forge compile

Additional context
No response

Working with
https://github.com/saucepoint/v4-template

Might be an outdated remapping issue.

Try:
Deleting the remappings:
rm -rf remappings.txt

Then automatically create remappings:
forge remappings > remappings.txt

Also for file
.gitmodules

I have now
[submodule "lib/v4-periphery"]
	path = lib/v4-periphery
	url = https://github.com/uniswap/v4-periphery


feat(*): Add SMI CRD checker

Signed-off-by: Shalier Xia<EMAIL_ADDRESS>
Description:
Adds SMI CRD existence checker so that users know when they're missing a required SMI CRD (currently that's HTTPRouteGroup, TCPRoute, TrafficSplit, TrafficTarget)
If a SMI CRD is missing or the version is not the required version, the pod will no longer be ready. But it will not restart, so that the logs telling the user what the issue is aren't cleared.

The user can check the logs to see which SMI CRD is missing and why. Example log
Missing SMI CRD: TCPRoute. To manually install TCPRoute, do `kubectl apply -f https://raw.githubusercontent.com/openservicemesh/osm/v0.8.0-rc.2/charts/osm/crds/specs.yaml`

Once the user has added the missing SMI CRD the pod will become ready.

and resolves #2850

Affected area:

New Functionality      [X ]
Documentation          [ ]
Install                [ ]
Control Plane          [ ]
CLI Tool               [ ]
Certificate Management [ ]
Networking             [ ]
Metrics                [ ]
SMI Policy             [ ]
Security               [ ]
Tests                  [ ]
CI System              [ ]
Demo                   [ ]
Performance            [ ]
Other                  [ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?
No

Codecov Report

Merging #3191 (1bbe463) into main (2bf5974) will decrease coverage by 0.62%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #3191      +/-   ##
==========================================
- Coverage   66.70%   66.08%   -0.63%
==========================================
  Files         155      162       +7
  Lines        7155     7434     +279
==========================================
+ Hits         4773     4913     +140
- Misses       2357     2494     +137
- Partials       25       27       +2

Flag
Coverage Δ

unittests
66.08% <0.00%> (-0.63%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

cmd/osm-controller/osm-controller.go
16.53% <ø> (-0.27%)
:arrow_down:

pkg/smi/health.go
0.00% <0.00%> (ø)

pkg/envoy/types.go
33.33% <0.00%> (-66.67%)
:arrow_down:

pkg/catalog/dispatcher.go
32.69% <0.00%> (-28.85%)
:arrow_down:

pkg/catalog/debugger.go
88.88% <0.00%> (-8.26%)
:arrow_down:

pkg/debugger/namespace.go
80.00% <0.00%> (-5.72%)
:arrow_down:

pkg/identity/types.go
95.45% <0.00%> (-4.55%)
:arrow_down:

pkg/envoy/route/route_config.go
95.00% <0.00%> (-4.21%)
:arrow_down:

pkg/envoy/ads/profile.go
75.00% <0.00%> (-3.95%)
:arrow_down:

pkg/envoy/proxy.go
96.55% <0.00%> (-3.45%)
:arrow_down:

... and 56 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2bf5974...1bbe463. Read the comment docs.


Remove some external command dependencies

The epm module depends unnecessarily on several external commands; e.g., cat and test.  Remove those dependencies. It also has a bug in its -first-upper function due to using the obsolete colon slice index syntax rather than the double period (..) syntax. Replace that function with the str:title builtin.
Related: 1661

@zzamboni, Please review this change.

Note that there are other possible improvements to the epm module. For example, when PR#1679 (https://github.com/elves/elvish/pull/1679) is merged the sole use of the rm command can be replaced by path:remove. Similarly, I intend to introduce a path:mkdir command which will allow replacing the existing dependency on the external mkdir command with a builtin. Since the epm module has had few changes since it was added there are probably other improvements possible but those are outside the scope of this series of changes.


Cookbook: Example output incorrect?

In the first Meter example:
>>> import pronouncing
>>> phones_list = pronouncing.phones_for_word("snappiest")
>>> pronouncing.stresses(phones_list[0])
u'0102'

But when I try that code I get this:
>>> import pronouncing
>>> phones_list = pronouncing.phones_for_word("snappiest")
>>> pronouncing.stresses(phones_list[0])
u'102'

cmudict has SNAPPIEST  S N AE1 P IH0 EH2 S T so u'102' makes sense.
PR to fix this: https://github.com/aparrish/pronouncingpy/pull/5

yep, this was a typo (I probably had a different code example initially and forgot to copy over the resulting output when I changed the example).


location of fieldkeys

ENCRYPTED_FIELDS_KEYDIR = '/path/to/fieldkeys'
Is this relative to my project folder?
so / would be considered the top of my project folder.

It is not relative to your project folder. However, you can make it so thusly.
import os

PROJECT_DIR =os.path.dirname(os.path.dirname(__file__)) # assuming $PROJECT_DIR/somename/settings.py

ENCRYPTED_FIELDS_KEYDIR = os.path.join(PROJECT_DIR, 'mycoolpath/keys')


Fix issue 4411 (intermittent ci fail on cypress 'search Cluster Deployment & ping k8s cluster' integration test)

Listen for 'uncaught:exception' without trailing ' (setting 'onopen')' & move cy.on to listen to 'uncaught:exception' in beforeEach (so intermittent failure doesn't occur if more tests are added later)
Signed-off-by: Mario Arriaga<EMAIL_ADDRESS>Description
This PR fixes #4411
Notes for Reviewers
As per these failures, uncaught exception message is 'Cannot set properties of null' without trailing string ' (setting 'onopen')':
https://github.com/meshery/meshery/pull/4410/checks?check_run_id=3884206500
https://github.com/meshery/meshery/pull/4405/checks?check_run_id=3881006604
https://github.com/meshery/meshery/pull/4408/checks?check_run_id=3883590474
Signed commits

[ x] Yes, I signed my commits.

Codecov Report

Merging #4568 (4325b4f) into master (c398f64) will decrease coverage by 7.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4568      +/-   ##
==========================================
- Coverage   33.16%   25.99%   -7.18%
==========================================
  Files          60       60
  Lines        5044     5044
==========================================
- Hits         1673     1311     -362
- Misses       2935     3409     +474
+ Partials      436      324     -112

Flag
Coverage Δ

gointegrationtests
?

unittests
25.99% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/system/start.go
1.11% <0.00%> (-58.52%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
2.50% <0.00%> (-47.50%)
:arrow_down:

mesheryctl/internal/cli/root/system/status.go
0.00% <0.00%> (-47.44%)
:arrow_down:

mesheryctl/internal/cli/root/system/update.go
1.05% <0.00%> (-34.74%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
0.86% <0.00%> (-27.59%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.39% <0.00%> (-22.38%)
:arrow_down:

mesheryctl/internal/cli/root/system/logs.go
3.29% <0.00%> (-20.88%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9c1e007...4325b4f. Read the comment docs.

Thank you @MarioArriaga92 for this PR

Glad to help @warunicorn19, FYI @ramrodo @piyushsingariya I've posted about Meshery / UI end-to-end tests PR failure here, looks like its unrelated to changes introduced in current PR:
https://layer5io.slack.com/archives/C011VPREG2Z/p1636131781031700

Do we know if the Cypress changes take affect in the build checks that run in this PR or will we only know if the build checks are passing only after these changes are merged?

Let's merge and see...


Add event handler for window resizing.

Description of the Change
The Issue
Atom window state is not being saved. This means that after closing and reopening Atom, the window size is not the same it was before closing. However, blurring the window saves its dimensions.
Requirements

Source code files
The window dimensions are being saved (.atomEnvironment.storeWindowDimensions()) on handleWindowBlur() and handleWindowBeforeunload() functions in atom/src/window-event-handler.js. However, there should be an event handler function responsible for handling window resizing. If that event handler existed, then it would save the window dimensions every time they were modified.
Design of the fix

AtomEnvironment is a class used to deal with packages, themes, menus and the window. This class contains the functions to save and get the current window dimensions.
WindowEventHandler is the class containing all the handlers for the window. The handlers worth checking for this issue are handleWindowBlur() and handleWindowBeforeUnload(). These are the handlers where the window dimensions are saved.
In the diagram above we colored the function that we added in green. This function is a handler that activates whenever the user changes the window dimensions.
Fix source code
We first created the new event handler by adding this line of code:
this.addEventListener(this.window, 'resize', this.handleWindowResize)

Then, we had to bind the handler:
this.handleWindowResize = this.handleWindowResize.bind(this)

Finally inside the handler, we placed the function that saves the window dimensions, like this:
handleWindowResize () {
    this.atomEnvironment.storeWindowDimensions()
  }

With this fix, we make sure that the window dimensions are always saved whenever the window is resized.
Alternate Designs
We considered using setInterval to routinely call the function that stores the window dimensions but this method was discarded as it wasn't event driven and slightly lowered atom's performance.

Why Should This Be In Core?
People expect atom to save the window size everytime they resize it so that when they open atom the previously saved size is restored.
Benefits
Explained above.
Possible Drawbacks
No relevant drawbacks.
Applicable Issues
Window close does not trigger save window state #13909

I like the way you describe you PR, +1

I was able to test these changes and it looks good to me.  @xfontes42 any chance you can add test to this?  There's another PR (https://github.com/atom/atom/pull/15514) that tries to do the solve the same issue but I think this solution actually solves it for all the cases as I tried closing with the red-x, cmd-q and windows:close and the size is saved.
The other PR has a test, could you possibly move that over to this PR and add any others you feel would be useful to add?
/cc @leroix  for code review

Assuming the tests pass, it looks good to me 👍


Name vs Id issues with instance destruction in GCP?

https://github.com/iterative/cml/issues/754
https://github.com/iterative/cml/issues/754#issuecomment-928240276
Possibly related:
https://github.com/iterative/cml/pull/742
https://github.com/iterative/cml/pull/738
https://github.com/iterative/terraform-provider-iterative/issues/178
Shortly after Github started Clubbing the runner name env the workaround of setting the name empty resulted in successful workflows from @0x2b3bfa0 suggestion of RUNNER_NAME=  then it seems with the release of CML 0.7.4 the destroy stopped working. From my activity logs on GCP when the workflows were successful the instances were named with by the Id generation found here: https://github.com/iterative/terraform-provider-iterative/blob/732d0919577867d40b7ff635dd0667caf19d3cb9/iterative/utils/helpers.go#L19
and in total it sets the name if it is empty to equal the Id:
https://github.com/iterative/terraform-provider-iterative/blob/732d0919577867d40b7ff635dd0667caf19d3cb9/iterative/utils/helpers.go#L17-L25
The code for the destroy pulls the Id https://github.com/iterative/terraform-provider-iterative/blob/732d0919577867d40b7ff635dd0667caf19d3cb9/iterative/gcp/provider.go#L273-L275
But the Golang documentation has some not exactly clear wording that makes me think this should be the name and not the Id...
https://pkg.go.dev/google.golang.org/api/compute/v1#InstancesService.Delete
From looking at the Name vs ID in my GCP activity logs it seems the delete VM has only succeed when the Name and Id were the same.

Happy to provide any logs from GCP that may be of help upon request.


Support custom cloudtrail namespaces

We already ingest cloudtrail data to an elasticsearch cluster, but we have the data indexed in a nested dictionary structure for namespacing reasons.
For example, if a cloudtrail json blob looks roughly like:
{
  "eventVersion": "1.0",
  "userIdentity": {
    "type": "IAMUser",
    "principalId": "EX_PRINCIPAL_ID",
    "arn": "arn:aws:iam::123456789012:user/Alice",
    "accessKeyId": "EXAMPLE_KEY_ID",
    "accountId": "123456789012",
    "userName": "Alice"
  },
  "eventTime": "2014-03-06T21:22:54Z",
  "eventSource": "ec2.amazonaws.com",
  "eventName": "StartInstances",
  "awsRegion": "us-east-2",
  "sourceIPAddress": "<IP_ADDRESS>",
  "userAgent": "ec2-api-tools <IP_ADDRESS>",
  "requestParameters": {
    "instancesSet": {
      "items": [
        {
          "instanceId": "i-ebeaf9e2"
        }
      ]
    }
  },
  "responseElements": {
    "instancesSet": {
      "items": [
        {
          "instanceId": "i-ebeaf9e2",
          "currentState": {
            "code": 0,
            "name": "pending"
          },
          "previousState": {
            "code": 80,
            "name": "stopped"
          }
        }
      ]
    }
  }
}

It would actually look like:
{
  "the_cloudtrail_data": {
    "eventVersion": "1.0",
    "userIdentity": {
      "type": "IAMUser",
      "principalId": "EX_PRINCIPAL_ID",
      "arn": "arn:aws:iam::123456789012:user/Alice",
      "accessKeyId": "EXAMPLE_KEY_ID",
      "accountId": "123456789012",
      "userName": "Alice"
    },
    "eventTime": "2014-03-06T21:22:54Z",
    "eventSource": "ec2.amazonaws.com",
    "eventName": "StartInstances",
    "awsRegion": "us-east-2",
    "sourceIPAddress": "<IP_ADDRESS>",
    "userAgent": "ec2-api-tools <IP_ADDRESS>",
    "requestParameters": {
      "instancesSet": {
        "items": [
          {
            "instanceId": "i-ebeaf9e2"
          }
        ]
      }
    },
    "responseElements": {
      "instancesSet": {
        "items": [
          {
            "instanceId": "i-ebeaf9e2",
            "currentState": {
              "code": 0,
              "name": "pending"
            },
            "previousState": {
              "code": 80,
              "name": "stopped"
            }
          }
        ]
      }
    }
  }
}

I think cloudtracker could expose the concept of a key_prefix or key_namespace or something, to account for this type of setup

Makes sense.  I need to make the index configurable, as right now it is hard-coded at: https://github.com/duo-labs/cloudtracker/blob/07dcc7697e39876e6acdaf2b35a69f61448b0cdc/cloudtracker/datasources/es.py#L22
And then I think I could support the key_prefix by just prefixing the various query fields, for example this: https://github.com/duo-labs/cloudtracker/blob/07dcc7697e39876e6acdaf2b35a69f61448b0cdc/cloudtracker/datasources/es.py#L52

Ack; that's exactly what I had in mind. I'm eager to run this against my cloudtrail data set, so I'll hopefully be adding this shortly :)

Actually there is at least one issue. You are not considering key_prefix for the filter_errors field name. You should do something like
$ git diff -U10
diff --git a/cloudtracker/datasources/es.py b/cloudtracker/datasources/es.py
index 9b58e48..9cc8795 100644
--- a/cloudtracker/datasources/es.py
+++ b/cloudtracker/datasources/es.py
@@ -29,21 +29,21 @@ class ElasticSearch(object):
     def __init__(self, config, start, end):
         # Open connection to ElasticSearch
         self.es = Elasticsearch([config], timeout=900)
         self.searchfilter = {}
         self.index = config.get('index', 'cloudtrail')
         self.key_prefix = config.get('key_prefix', '')
         if self.key_prefix != "":
             self.key_prefix += "."

         # Filter errors
-        self.searchfilter['filter_errors'] = ~Q('exists', field='errorCode')
+        self.searchfilter['filter_errors'] = ~Q('exists', field='{0}errorCode'.format(self.key_prefix))

         # Filter dates
         if start:
             self.searchfilter['start_date_filter'] = Q('range', eventTime={'gte': start})
         if end:
             self.searchfilter['end_date_filter'] = Q('range', eventTime={'lte': end})

     def get_field_name(self, field):
         return self.key_prefix + field + ".keyword"

hmmmm there's another problem. The start_date and end_date range queries are not using the key_prefix, which is more annoying to address because eventTime is being passed as a kwarg (although, on second thought, this may actually be a different issue in that the user should probably be able to specify which which field is being used as the index's timestamp (in my case, it's not eventTime, and in fact, it's not even the_cloudtrail_data.eventTime))

@danielpops Is this resolved now with the work you did for #10?

Yup. This appears to be working as expected. Thanks alot!


typofixes

Cherry-picked your typo fixes into main. Thanks! https://github.com/hodgesmr/mastodon_digest/commit/c7b61b2738988bce4910ac5f7988f5a5753bee8c


Set correct _NET_WM_WINDOW_TYPE for dialog windows

For dialogs and popups, this solution should work on all tiling window managers.
Fixes #55415
Remake pull request (original one #55442 needed to move pull to another branch and at same time updated to newest so easier just completely remake patch)

@bruvzg @Riteo Could you give this another look?

Yeah I agree that is hacky solution, but it would require quite lot of refactoring to get better solution, and my C/Cpp is from before 2010, don't have touched that much so my expertise for refactoring this is out of my skill level / scope.
This solution works also in Wayland to extend, don't have seen any jarring behavior.


adding new .github/test-spec.yml

Signed-off-by: Sebastian Wezel<EMAIL_ADDRESS>This PR is meant to be a starting point to create the .github/test-spec.yml
Which will work similar to the sdk-nrf/.github/test-spec.yml
You can add changes to the test-spec branch. I'll combine them to a single commit on Friday (18.03)

I squashed all the commits to one, and added default entries for missing labels.
Also added CODEOWNERS entry


Fix typedoc generation

This is independent of the other docs stuff I'm working on, so making this a separate PR. Please focus review on package/-lock.json.

Use --ignoreCompilerErrors option to typedoc because it complains about the following compiler error, even though it doesn't show with tsc:

Using TypeScript 3.1.1 from /Users/keanulee/Code/Polymer/lit-html/node_modules/typescript/lib
Error: Option 'declarationMap' cannot be specified without specifying option 'declaration' or option 'composite'.

Add Google Analytics ID

Ping?

The compile error is weird, since we do have declaration: true in the tsconfig. I assume that tsdoc is generating its own tsconfig from ours and not copying both settings.

Updated docs now deployed to https://lit-html.polymer-project.org/api/index.html


hi ,could you open source the datasets of  extract information  on your finetune stage?

Hello, thank you for your attention. Our information extraction dataset will be made public after the CCKS competition ends.

Hi, our datasets for instruction tuning is available at https://github.com/zjunlp/KnowLM/blob/main/README_ZH.md


Linter Tests - CSS and HTML

My last pull request closed when I was trying to squash my commits because the feature branch was deleted. I'll ask around to see if there is a solution first before making a second PR next time.
For GCI

Old PR: https://github.com/zulip/zulip/pull/7586

@Symt: Hey, thanks for working on this! I have a couple of comments:

The commit message could be more informational. Maybe something like linter: Add good/bad lines for HTML and CSS linter rules. Try to describe what the commit does and not what you did when creating the commit. Another thing to keep in mind is that the general convention within the Zulip project is that commit messages should end in a period. I would suggest amending the commit message like I suggested and making sure that it ends in a period too. :)

Also, even if you feel like you deleted a branch by mistake or something went wrong, feel free to ask for help on chat.zulip.org before creating a new PR. We are always happy to answer your questions! :)

That's all I have for now! Perhaps, @timabbott and @roberthoenig might have more suggestions! (Also the tests pass locally, so I am not sure why Travis is complaining!)
Submitting this for more work on GCI! :)
Thanks!

There is one thing I'd like to address.

@Symt did you accidentally closed the PR?

Yes

Yes

I'm going to see what I can do to fix it, if anything before opening another PR


Migrate to the arm architecture

Hi there
I have two questions that I hope can be answered.

Is it possible to start Linux by running this directly on bare metal? Or should I start zephyr first and then run the FSBL application to start Linux?
If I want to migrate to the arm architecture, what modifications should I make?

Thx

If you want to run Linux on RISC-V you have to use some software that provides SBI interface (typically OpenSBI). In order to run OpenSBI some hardware must be initialised (e.g. RAM memory where OpenSBI is loaded by default). This is usually done by some type of FSBL software. You can read more about the boot flow on RISC-V in OpenSBI and U-Boot documentation.
The application is RISC-V specific. It is intended to prepare a platform in the first place to run OpenSBI.
If you’re thinking about running this on arm32, then initialising the hardware and jumping into Linux is all that is needed. On arm64 you might need to load ATF (arm trusted firmware) first.


feature: create loss_function parameter

Option to use different loss functions is added.
For regression, all default loss functions in pytorch are working.
For classification only 2 loss functions work (CrossEntropyLoss and NLLLoss). This likely has something to do with formatting of pred and target that is non-uniform across loss functions. I raised a new issue (#318) to solve this, as it will require some careful consideration.
Also: I added 'BA' as a default target type for regression tasks, and fixed a mistake in test_dataset.py

I've run the tests locally but I got many errors, even in tests that do not user class_weights parameter. Did you notice that as well? Solved those tests, I have an idea about how to move stuff without reinserting the loaders in the init.

Yeah, I think I added another bug last evening when I was working late/tired. Those should be fixed now and locally I only have the one with weights is failing now.

Tests fixing, then I can fix the one about the weights

done.


Message decode offset error

I found two issues in the project on 2014/11/24. Now one has been fixed, but another was ignored by me.
In this file "\src\main\java\org\ice4j\message\Message.java", the founktion "decode" line 1154 is "offset += (4 - (att.getDataLength() % 4));".
I think it should be like this "offset += (4 - (att.getDataLength() % 4)) % 4;".
Because when the attribute's lenth is 0, the result was 4. but we need the offset += 0.
the ver. is SHA-1: 8b70299567f3062238a77877a01617e47e36d0de on 2015/12/3

Line 1154 isn't executed because 0%4==0 on line 1152.

I was too careless, because the other that caused an exception error in my program, it would not look carefully at the context. sorry!


Support websockets

API gateway doesn't support websockets, but if you have the power behind the curtain it would be lovely to enable websocket support somehow -- the api-gateway itself could expose the open sockets with IP addresses and expose a way to send to them through the context?

Lambda does not currently have the ability to support web sockets so there isn't any way to accomplish this currently. What use case would you have for web sockets that would each be timed out in an unknown amount of time and unable to communicate with other clients?

Amazon API Gateway supports WebSocket APIs today.

@lis186 related issue https://github.com/aws/chalice/issues/1017 is open.


OpenSource : 2.2 Learn open source basics

Description :

Learn open source basics 👇

Every work has the best practices and standards which were
initiated by experienced people and followed by many in order to
maintain the quality of work. It applies to open source as well. It is
better to go through things like best practices for commit
messages, semantic versioning, open-source licenses, contribution
guidelines, and the code of conduct for open-source projects.

Note :

Changes should be made inside the Open_Source/ directory & Opensource branch.
Issue only for GWOC'21 contributors.
Issue will be assigned on a first come first serve basis, 1 Issue  ==  1 PR.

Must Follow : Contributing Guidelines & Code of Conduct before start Contributing.
Happy Contributing 🙌

Can I be assigned this issue?

how to start @Aryamanz29

@nayakaj  Any updates on this?

Can I try it

@Aryamanz29  i submit by today

@Aryamanz29

plz look into this when im pushing it it is asking git password and username

@Aryamanz29  completed

@Aryamanz29 plz check

@Aryamanz29 plz check

Complete requested changes

If No one is working on this issue, I would like to work on it, Please Assign it to me.

@mayankkuthar  Sure


Getting yellow border around app

Just tried out FluentWPF and noticed I get a yellow border around my application window. I tried using AcrylicWindowStyle="None" but that doesn't help. Looked for anyone else describing the same situation on here but couldn't find anyone explicity talking about a yellow border.
Here's my code:
<fw:AcrylicWindow x:Class="LockscreenUI.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
        xmlns:local="clr-namespace:LockscreenUI"
        xmlns:fw="clr-namespace:SourceChord.FluentWPF;assembly=FluentWPF"
        mc:Ignorable="d"
        Title=""
        Height="1440"
        Width="5120"
        WindowStyle="None"
        Topmost="True"
        AllowsTransparency="True"
        Left="0"
        Top="0"
        ResizeMode="NoResize"
        FontFamily="Segoe UI Light"
        AcrylicWindowStyle="None">
    <Grid
        Background="#70000000">

        <Grid.ColumnDefinitions>

            <ColumnDefinition Width="*"/>
            <ColumnDefinition Width="*"/>

        </Grid.ColumnDefinitions>

        <Grid.RowDefinitions>

            <RowDefinition Height="100"/>
            <RowDefinition Height="*"/>
            <RowDefinition Height="90"/>
            <RowDefinition Height="10"/>

        </Grid.RowDefinitions>

        <!-- Left display -->

        <Border Background="#70000000"
                Grid.Column="0"
                Grid.Row="0"/>

        <Border Grid.Column="0"
                Grid.Row="1"
                CornerRadius="20"
                Margin="200"
                BorderThickness="10">

            <Border.Background>

                <ImageBrush Stretch="Fill" ImageSource="/Assets/Images/1.jpeg"/>

            </Border.Background>

        </Border>

        <!-- Right display -->

        <Border Background="#70000000"
                Grid.Column="1"
                Grid.Row="0"/>

        <Border Grid.Column="1"
                Grid.Row="1"
                CornerRadius="20"
                Margin="200"
                BorderThickness="10">

            <Border.Background>

                <ImageBrush Stretch="Fill" ImageSource="/Assets/Images/1.jpeg"/>

            </Border.Background>

        </Border>

        <Button Content=""
                Background="Transparent"
                Foreground="Transparent"
                Grid.Row="3"
                Grid.ColumnSpan="3"
                BorderThickness="0"
                Click="ExitButton_Click"/>

    </Grid>
</fw:AcrylicWindow>

I solved it by setting window border thickness to zero)  although my had blue color brder))

I solved it by setting window border thickness to zero) although my had blue color brder))

Thanks, I'll try it out!

Hi @originalodz
AcrylicWindowStyle="None" is not for modify window border color.
Please try BorderThickness="0" as @TMS-Namespace says.

I'll close this issue.
If you have any problems, please reopen this issue.


K8s Proxy Attempt 3

This will be edited to line up with Backstage PR standards

Something that could go a long way here is giving a decent indication of how frontend code could consume this endpoint. I'm imagining extending the KubernetesApi interface to include a method for getting an arbitrary kubernetes API client, something like
diff --git a/plugins/kubernetes/src/api/types.ts b/plugins/kubernetes/src/api/types.ts
index 7c5943a6eb..491387fc62 100644
--- a/plugins/kubernetes/src/api/types.ts
+++ b/plugins/kubernetes/src/api/types.ts
@@ -19,6 +19,7 @@ import {
   ObjectsByEntityResponse,
 } from '@backstage/plugin-kubernetes-common';
 import { createApiRef } from '@backstage/core-plugin-api';
+import type { ApiConstructor, ApiType } from '@kubernetes/client-node';

 export const kubernetesApiRef = createApiRef<KubernetesApi>({
   id: 'plugin.kubernetes.service',
@@ -35,4 +36,8 @@ export interface KubernetesApi {
       oidcTokenProvider?: string | undefined;
     }[]
   >;
+  proxyApiClient<T extends ApiType>(
+    clusters: { [key: string]: string },
+    apiClientType: ApiConstructor<T>,
+  ): T;
 }

which could then be called like
import { CustomObjectsApi } from '@kubernetes/client-node';

const kubernetesApi = useApi(kubernetesApiRef);
const clusterTokenMap = {}; // figure out how to get tokens for each cluster of interest
const customObjects = kubernetesApi.proxyApiClient(
  clusterTokenMap,
  CustomObjectsApi,
);
const response = await customObjects.listClusterCustomObject(
  'kappctrl.k14s.io',
  'v1alpha1',
  'apps',
);

Even better (but probably not required for the PR to be mergeable) would be to have a sample plugin which does this in the contrib folder.

There's a bit of a JSON bias here -- some core k8s APIs (like logs, port-forward and exec obviously) do not speak JSON so this implementation can't support those, and I'm pretty confident that clusters can contain API aggregation layer services which I think are super flexible and not necessarily required to speak JSON.


Trinity filesystem utils

What was wrong?
There are some filesystem utility functions that are needed in trinity.
How was it fixed?
Separated them out into their own pull request to reduce overall size of https://github.com/ethereum/py-evm/pull/310
Cute Animal Picture

This should be against the trinity branch.


Feat/opcode sar

LGTM.
todo:

i leave one comment
fix ci

moved to https://github.com/scroll-tech/zkevm-circuits/pull/190


Partial views logic

First off, congratulations for making this amazing project. I hope it gets the recognition and the credit it deserves.
My question is this: how do you go about loading partial views for states' templateUrl?
On a very simple CRUD example, and in pseudo-code:

Data model "staff" has a detail view and a create/edit view.
state "app.staff.create" needs to load "form.html" partial.
state "app.staff.show" needs to load "detail.html" partial
getView() method for templateUrl will look for a folder called "staff", and a file "staff.html", but I'd like to keep all of my partials within the "staff" feature folder, "staff.html" being the container for most of them.

Now I can simply not use the getView() helper, but I'm wondering if you had some kind of logic for loading partials in mind (based on all of your other clever solutions :))
Thanks!

Hi @AArmin
Thank you for your kind words :smile:
How about using getView('staff/detail')?
Let me know if it works for you

@AArmin Remember that ui-router is quite powerful so don't feel restricted by the example in the starter project. If you have not looked at the ui-router wiki it's very worthwhile.
To start with check out this and this which I think will be of use to you.
Also if the getView() method does not meet you requirement you can always add some parameters to add more functionality maybe like so (Not tested):
var getView = function(viewName, viewPath, fileName) {
    // If no viewPath is passed set a default.
    var path = viewPath ||  './views/app/';

    // If no fileName is passed use the viewName.
    var file = fileName || viewName

    return path  + viewName + '/' + file+ '.html';
};

This works the same but has some optional parameters you can use. Leave them out and it works as it originally did.

Amazing answer @Bodom78
Closing this for now - @AArmin feel free to comment if this works for you or if you have any question


Visual Studio for mac not supported

Apparently  Visual Studio for Mac doesn't Support C# 7.3 thus not able to build a blazor app.
Dotnet 2.1.300 installed (and possible to build/run blazor apps on mac via CLI), but Visual Studio For Mac is not.
Error: CSC : error CS1617: Invalid option '7.3' for /langversion. Use '/langversion:?' to list supported values.

Thanks for posting.
We're tracking VS for Mac support in #336.


Javascript binding examples

The blob storage javascript/typescript input examples here are not showing any useful details. How are these supposed to be used correctly? When I generated a function using the VS Code extension, it made a function like this:
const blobTrigger: AzureFunction = async function(context: Context, myBlob: any): Promise<void> {
    context.log('Blob trigger function processed blob \n Name:', context.bindingData.name, '\n Blob Size:', myBlob.length, 'Bytes');
};

The problem here is what is myBlob? It is typed as any, which isn't helpful. Will I get a string of the blob contents? Is it an object containing information? I will run this and find out shortly, but was hoping the documentation would at least tell me what to expect.

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

ID: cd428773-db08-e2cb-8320-2943be82ffb7
Version Independent ID: b8f4e7f0-21a6-1c25-3fd4-7bd3229c4e3a
Content: Azure Blob storage input binding for Azure Functions
Content Source: articles/azure-functions/functions-bindings-storage-blob-input.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@ocdi Thank you for your feedback! We will review and update as appropriate.

@ocdi Yes I agree that more information could be included for the JavaScript example. We are discussing to add samples and list of supported types under the "Usage" section. I am looping the content author @craigshoemaker in it.


Make competition endpoint function available globally

This PR makes the function that gathers all data for a single settlement from the competition endpoint available globally. It also renames the configuration.py file to the more appropriate helper_functions.py

The test errors for me, but I am not sure why.

The test errors for me, but I am not sure why.

But is this due to this PR? Or does it fail even in the main branch?

It does not fail on the main branch.


Autocomplete not working for javascript on Mac

Issue Type: Bug
The autocompletion of methods and properties for objects has stopped working.
If I start typing eg. con the editor suggests console and const as word completions, but if I select console and then type a period I don't get any of the methods of the console object.
Similarly if I create an array and then type a period I don't get any of the methods or properties for the array.
If I hit CTRL + Space after typing console I get "No suggestions".
I've tried disabling extensions and restarting VS Code, and restarting my machine but nothing is working.
VS Code version: Code 1.38.1 (b37e54c98e1a74ba89e03073e5a3761284e3ffb0, 2019-09-11T13:31:32.854Z)
OS version: Darwin x64 18.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz (4 x 2300)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 4

Memory (System)
16.00GB (4.23GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

vscode-eslint
dba
1.9.1

flow-for-vscode
flo
1.5.0

/needsMoreInfo
Do you see "Initializing JS/TS language features" in status bar when you open JS file first time? Does it stops to spin?
Is it for any workspace or there are some specifics? Do you have any jsconfig.json or tsconfig.json files?
Are there any error in DevTools console if you open Help > Toggle Developer Tools ?

Hi @IllusionMH thanks for the input. I'm afraid I actually got impatient with it and did a complete uninstall/reinstall. It's working now. I will take note of what you said and if the problem re-occurs hopefully I'll be able to provide more information. Thanks again.

A bit radical way, but I'm glad that it worked for you.


[Question] Any support for cooperative rebalancing protocol?

Our use case implements a large number of consumers running in K8s. We are seeing a lot of "rebalancing storms" and I believe cooperative rebalancing might help. Is there a way to achieve this, or perhaps a workaround?
Thanks in advance.

this will be in v1.6.0, which should be out in about a week. it's in 1.6.0-RC2, which is on nuget, but note that there are some known issues in that version.

this will be in v1.6.0, which should be out in about a week. it's in 1.6.0-RC2, which is on nuget, but note that there are some known issues in that version.

I think you could close this issue.
Cooperative sticky strategy appeared in version 1.6.0.


New Governance Metric - OSC Governance/Security TWG

Per the Security TWG: Please update/add Security.md
Please refer to: https://github.com/IntersectMBO/Open-Source-Office/blob/main/SECURITY.md
The initial deadline for these changes to be made is 30 September 2024. Thank you.
Ref: Issue #482

@Ranchhand87 PR #497 add SECURITY.md. Could you please review that PR


PLAT-200: split old logicrunner into multiple components

- What I did
- How I did it
- How to verify it
- Description for the changelog

Rebased


Fix refs in vars in projects being rendered (#1047)

In #1035 I added logic that uses jinja to render all fields with a context containing vars/env_vars. This was a mistake, since apparently you can ref() models in there, and rendering will always fail if you do that.
I also added a test that reproduces the issue.
An alternative to consider: should nothing under models/seeds get rendered until execution time?

Fixes #1047


Improve example of usage

Coverage remained the same at 100.0% when pulling 687ad35e3845711926a13ffc2aa2369f06b8c6b0 on funkyboy:patch-1 into 016c12bf402d074806ede592ef7ad48484d7108c on scour:master.

Coverage remained the same at 100.0% when pulling 687ad35e3845711926a13ffc2aa2369f06b8c6b0 on funkyboy:patch-1 into 016c12bf402d074806ede592ef7ad48484d7108c on scour:master.


Work out circular version of our team logo

Follow up of #1 .

Combining our existing marker icon with a radar (idea from Bo)

Such that it fits a more circular design. A first iteration is shown below:

But this needs a bit more finetuning

A new version has been made and is open for feedback

Think we have made our decision!


Visualisation on node with error doesn't show on spacebar press

Discord username
No response
What type of issue is this?
Permanent – Occurring repeatably
Is this issue blocking you from using Enso?

[ ] Yes, I can't use Enso because of this issue.

Is this a regression?

[ ] Yes, previous version of Enso did not have this issue.

What issue are you facing?
While having node selected space bar doesn't show visualisation of error.
https://user-images.githubusercontent.com/108271897/235462244-782200ef-ce0f-4be2-bd16-3631c6d3de2a.mp4
Visualisation of error is only visible while hovering on the output port of the node.
Expected behaviour
Space bar pressed while node with error is selected shows the error message (warnings viz)
How we can reproduce it?
on node with error
select it
press spacebar
Screenshots or screencasts
No response
Enso Version
2023.5.1 nightly
Browser or standalone distribution
Standalone distribution
Browser Version or standalone distribution
standalone
Operating System
Windows
Operating System Version
Win11pro 22H2 22621.1555
Hardware you are using
12th Gen Intel(R) Core(TM) i9-12900HK / RTX3060 Laptop / Nvidia Drivers 531.68

Probably a duplicate of https://github.com/enso-org/enso/issues/6454

Actually this is an error, not a warning (there is type error message at some point) - nodes with errors does not have the visualization as they are not producing result (unlike the nodes with warnings). The boarder of the node is red, not yellow like for a warning. We should have an option to show error message permanently but this is not a valid bug.

@sylwiabr visualization should work no matter what it is - error or warning. After pressing space I should get the same visualisation as on hover over output.

The warnings showing in the visualization is a temporary hack that should be revisited and improved (there is an issue for this) so it should be fixed by fixing warnings displaying correctly, not going with merging errors into visualizations. @xvcgreg


Extension page version number styling

Issue Type: Bug
https://github.com/microsoft/vscode/issues/94123
Tthe italicized version number at the top of an extension's page looks out of place to me, I think it would look more consistent for it to not be in italics, and perhaps say "Version" in front of it.

Just a suggestion, feel free to close.
VS Code version: Code - Insiders 1.45.0-insider (ece7aaee861d7261a728d52ce436c667030ce17d, 2020-05-01T09:44:59.826Z)
OS version: Darwin x64 18.7.0

No hard feelings. Changed as per your suggestion


APIGW Authoriser definition not supported

cfn-lint version: cfn-lint 0.16.0
Hi, there!
Thanks for this great tool! It looks really promising!  :)
I think this is yet another SAM issue. The following piece of template works in Cloudformation, but  cfn-lint think otherwise :P
  API:
    Type: "AWS::Serverless::Api"
    Properties:
      StageName: Prod
      Auth:
        DefaultAuthorizer: LambdaRequestAuth
        Authorizers:
          LambdaRequestAuth:
            FunctionPayloadType: REQUEST
            FunctionArn: !Ref AuthoriserArnParameter
            Identity:
              Headers:
                - authorization
              ValidationExpression: "Bearer\ .*"
              ReauthorizeEvery: 0

The error message is:
2019-03-14 15:18:18,767 - cfnlint - DEBUG - Setting AWS_DEFAULT_REGION to us-east-1
2019-03-14 15:18:18,854 - cfnlint - DEBUG - Completed linting of file: ../../server/template.yaml
E0001 Error transforming template: Resource with id [API] is invalid. Auth works only with inline Swagger specified in 'DefinitionBody' property
../../server/template.yaml:1:1

This is an error we are getting back from samtranslator
I believe this is the issue that corresponds to the problem you are having.
https://github.com/awslabs/serverless-application-model/issues/756

Cheers mate, thank you for pointing me to the right direction! Please feel free to close this.

I'm going to leave this open for a little bit.  We have seen a spike in SAM issues in the last week or two.  Since the template deploys I want to see if they can fix this or if we will have to "fix" it.   Kind of like what we are talking about for #700

We should have a work around for this in the next release as a result of #778

Released.


Increase Kafka ebs since the storage is full cause kafka to reject connection from all source

Already increase Kafka ebs from 20gb to 200gb on prod env.


Created Update Secure Configuration Task

I don't exactly know how to test this, but I took a stab at creating the task I offered to build.
https://github.com/WaelHamze/xrm-ci-framework/issues/42
Since this is my first attempt, I'm all ears for what I've missed and how I can do this better.  Thanks for considering my contribution.

So I just noticed that the pull request changes the sln file to target VS 2017.  Sorry about that.  Other than that, I tried to make sure there weren't any other side effects.  If the other parts look good, then I can fix the sln file.  Otherwise, I will fix it when I make any changes base on your feedback.

Thanks a lot for creating the pull request.
Testing is not very difficult actually. I can show you the few steps.
1- Rebuild the solution
2- Make you updated this to include the task PackageFramework.ps1. Run this script it will copy all the task to the bin directory
3- Yo will need a VSTS instance that you can test this on. I suggest you use a different account than the one where the extension is installed
4- Install the TFX command line tools.
5- tfx login --service-url https://[account].visualstudio.com/defaultcollection --auth-type 'pat' --token '[token]'
(You can get a token from your user profile in VSTS)
6- upload the task from the bin directory
7- tfx build tasks upload --task-path .\MSCRMBackupOnlineInstance
Alternatively I will merge the pull request when I get a chance and release a new version that you can try.

Thanks!  I'll give this a shot.  Any suggestions on how to handle the fact that I'm using VS2017 and it changes the sln file?

The testing instructions were very helpful.  I found some issues with my code, fixed those issues, and tested everything is working as expected.  The latest commit should reflect a working task.  Please let me know your feedback.

Thanks. Will package it soon once I finish the task. Will let you know once it is live.


dark themes error

I'm getting an error when trying to use a dark theme.
Tried all "build-in" dark themes: dark, mr_robot and apogee. All give me the same error.
This is the error I get:
"partials/functions/parse_theme.html" at <$theme.light.background>: can't evaluate field background in type interface {}

If I set the theme setting light = true in any of the dark themes, the site is compiled.
Any idea?
I've the latest version of academic-kickstart (6dfec63) and I'm running Hugo version 0.74.3.
Christian

Thanks for reporting this! It should be fixed now in the latest master version of Academic.


Merge development to master

Perform a weekly merge of development to master.

@mananwason @championswimmer Any idea why Travis build isn't getting triggered for this?

@the-dagger Don't know why it's not showing up here but the build passed https://travis-ci.org/fossasia/open-event-android/builds/146976236


i am having some error at server console

my code is like below
{{>SimpleChatWindow roomId=roomID username=name}}
name and roomID is helpers returning string.
name : Meteor.user().username;
roomID : random generated 10 character string
and in configuration file this configuratoin saved
SimpleChat.configure ({
limit: 10,
beep: false,
showViewed: true,
showReceived: true,
showJoined: true,
publishChats: function(roomId, limi){
return false;
}
});
when sending message it gives error on console.
I20160309-12:48:36.188(5.5)? hi OCHNZH9aPa gyanAdmin null gyanAdmin
I20160309-12:48:36.973(5.5)? Exception while invoking method 'SimpleChat.messageViewed' TypeError: Cannot read property 'usernames' of undefined
I20160309-12:48:36.974(5.5)?     at [object Object].Meteor.methods.SimpleChat.messageViewed (methods_server.js:78:39)
I20160309-12:48:36.974(5.5)?     at maybeAuditArgumentChecks (livedata_server.js:1698:12)
I20160309-12:48:36.980(5.5)?     at livedata_server.js:708:19
I20160309-12:48:36.980(5.5)?     at [object Object]..extend.withValue (packages/meteor/dynamics_nodejs.js:56:1)
I20160309-12:48:36.980(5.5)?     at livedata_server.js:706:40
I20160309-12:48:36.981(5.5)?     at [object Object]..extend.withValue (packages/meteor/dynamics_nodejs.js:56:1)
I20160309-12:48:36.981(5.5)?     at livedata_server.js:704:46
I20160309-12:48:36.982(5.5)?     at tryCallTwo (C:\Users\rorschach217\AppData\Local.meteor\packages\promise\0.5.1\npm\node_modules\meteor-promise\node_modules\promise\lib\core.js:45:5)
I20160309-12:48:36.982(5.5)?     at doResolve (C:\Users\rorschach217\AppData\Local.meteor\packages\promise\0.5.1\npm\node_modules\meteor-promise\node_modules\promise\lib\core.js:171:13)
I20160309-12:48:36.982(5.5)?     at new Promise (C:\Users\rorschach217\AppData\Local.meteor\packages\promise\0.5.1\npm\node_modules\meteor-promise\node_modules\promise\lib\core.js:65:3)
I20160309-12:48:37.019(5.5)? received
I20160309-12:48:37.830(5.5)? Exception while invoking method 'SimpleChat.messageReceived' TypeError: Cannot read property 'usernames' of undefined
I20160309-12:48:37.831(5.5)?     at [object Object].Meteor.methods.SimpleChat.messageReceived (methods_server.js:25:41)
I20160309-12:48:37.831(5.5)?     at maybeAuditArgumentChecks (livedata_server.js:1698:12)
I20160309-12:48:37.832(5.5)?     at livedata_server.js:708:19
I20160309-12:48:37.832(5.5)?     at [object Object]..extend.withValue (packages/meteor/dynamics_nodejs.js:56:1)
I20160309-12:48:37.833(5.5)?     at livedata_server.js:706:40
I20160309-12:48:37.833(5.5)?     at [object Object]..extend.withValue (packages/meteor/dynamics_nodejs.js:56:1)
I20160309-12:48:37.833(5.5)?     at livedata_server.js:704:46
I20160309-12:48:37.834(5.5)?     at tryCallTwo (C:\Users\rorschach217\AppData\Local.meteor\packages\promise\0.5.1\npm\node_modules\meteor-promise\node_modules\promise\lib\core.js:45:5)
I20160309-12:48:37.834(5.5)?     at doResolve (C:\Users\rorschach217\AppData\Local.meteor\packages\promise\0.5.1\npm\node_modules\meteor-promise\node_modules\promise\lib\core.js:171:13)
I20160309-12:48:37.835(5.5)?     at new Promise (C:\Users\rorschach217\AppData\Local.meteor\packages\promise\0.5.1\npm\node_modules\meteor-promise\node_modules\promise\lib\core.js:65:3)

Hi, I can't copy the issue, but I see in your code:
publishChats: function (roomId, limit) {
return false;
},
and have to be:
publishChats: function (roomId, limit) {
return true;
},
make the fix, try again, and if the error continue, please send a gist o repo where i can reproduce the issue
bye

Hi, any notice?

no info,feel free to reopen

no info,feel free to reopen


Color formatting does not work on Windows Command Prompt

Issue:
When using commands like kubens and kubens newnamespace in a windows cmd terminal, the output is shown but the part that is supposed to represent the color setting is not being sent correctly to the terminal and is instead being rendered out as text.
Command Prompt Output
C:\Users\username>kubens
default
username
kube-node-lease
kube-public
kube-system

Command Prompt Screenshot

Steps to Reproduce:

Use Windows 10 (Version is 10.0.18363.815 as reported by the command prompt)
Open Command Prompt (normal or Admin)
Run kubens

Expected Result
It appears it is trying to color the username namespace as green to represent which one is the default namespace.

This is already fixed (see recently closed issues) but I need to make a release available. Thanks.

Sorry about that, I didn't mean to double report.


Add Metal to user-guide/09-targets.md

Partially resolves #4262
The description of "Parameter passing" part for Metal is related to PR #4022.

You need to update the table of contents by running docs/build-toc.ps1

it seems that adding "Partially resolves https://github.com/shader-slang/slang/issues/4262" to the PR description may cause https://github.com/shader-slang/slang/issues/4262 to close automatically upon this PR being merged


Understanding the thread safety claim - random sum every program invocation

Describe the bug
I'd like to understand a little bit more about the thread-safety claim, since a test program I wrote (inspired from https://www.yegor256.com/2018/11/06/ruby-threads.html) is returning a random sum on every program invocation, while without threads the result is stable.
Given this code:
class Program
  extend Dry::Configurable

  setting :counter

  def self.call; new.call; end

  def call
    self.class.config.counter = 1
    2.times.map do
      Thread.new do
        50.times do
          number_value = self.class.config.counter
          sleep(0.001)
          self.class.config.counter = self.class.config.counter + number_value
        end
      end
    end.each(&:join)
    self.class.config.counter
  end
end

To Reproduce
Program.call
Return value should be<PHONE_NUMBER>228229401496703205376, which is what you get when you comment the Thread.new do and end.each(&:join) lines above, so no Threads are involved.
Expected behavior
Return is random:
Program.call
=><PHONE_NUMBER>93078999176
Program.call
=><PHONE_NUMBER>009943038933
Program.call
=><PHONE_NUMBER>93078999176
Program.call
=><PHONE_NUMBER>766319185501
Program.call
=><PHONE_NUMBER>174159114038

My environment

Affects my production application: NO
Ruby version: 2.7.2
OS: Mac OS X 10.15

Please forward questions to out forward, github is for bug tracking or feature requests previously discussed on the forum. In a nutshell, your code lacks synchronization, dry-configurable cannot do it for you.


Fix nudge and textbox creation

Closes #669 and #660

Just a little note for future reference, GitHub doesn't parse the "and" in your PR message so it only recognizes the first bug number for the "Closes" command. You have to write out "closes" for each number. I wish they would fix this!


How do I use this tool?

Excuse me for asking what´s probably a really dumb question, but I never use github and I can´t figure out where or how to use this tool. Is there a .exe file somewhere? Or is it something else? I´d really appreciate some help, and I apologize for the inconvenience

Sorry about this. This is a web app that I never got around to hosting on GitHub. I've really been meaning to but again, haven't gotten to it, yet.

Ah I see. Thanks for the quick reply! I hope you find time to host it soon

Alright, the site's up at https://njs-guy.github.io/bst-graph/
Really should have done this earlier lol.

Ooh just saw this! Thats awesome! Currently the little badges showing the types dont seem to be working right? They look ok in Dark Mode, but they´re impossible to read in Light Mode

That's weird. What browser are you using? Works for me in Chrome and Firefox.

Odd.. I´m using Google Chrome as well :/


Fix typo in GattConnectionState documentation

Documentation was not matching the appropriate states

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


chore(build): Update ndm dockerfiles to use ndm build base image

Build ndm images using the ndm build base image
Why is this PR required? What issue does it fix?:
The ndm docker images built during CI install dependencies (openSeachest, libudev, libblkid) again and again for each build. This increases the duration of the CI run. This PR aims to make the CI runs faster.
What this PR does?:
Uses a custom base image for ndm builds with all the build dependencies pre-installed.
Checklist:

[x] Fixes #628
[x] PR Title follows the convention of  <type>(<scope>): <subject>
[ ] Has the change log section been updated? (NA)
[ ] Commit has unit tests (NA)
[ ] Commit has integration tests (NA)

Codecov Report

Merging #655 (2ab5aaa) into develop (669c5f6) will increase coverage by 0.27%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #655      +/-   ##
===========================================
+ Coverage    44.97%   45.25%   +0.27%
===========================================
  Files           79       79
  Lines         3964     3964
===========================================
+ Hits          1783     1794      +11
+ Misses        2014     2001      -13
- Partials       167      169       +2

Impacted Files
Coverage Δ

cmd/ndm_daemonset/probe/udevprobe.go
49.01% <0.00%> (+0.79%)
:arrow_up:

cmd/ndm_daemonset/probe/addhandler.go
70.91% <0.00%> (+2.94%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 669c5f6...2ab5aaa. Read the comment docs.


[Grid] Typescript Grid dynamically spacing warning

[x ] This is not a v0.x issue.
[x ] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔
No error in CLI!
Current Behavior 😯
Typescript give me an error about spacing value:
TS2322: Type 'number' is not assignable to type 'boolean | 7 | 2 | 10 | 1 | 3 | 4 | 5 | 6 | 8 | "auto" | 9 | 11 | 12'.
No lint errors found
Version: typescript 3.5.2, tslint 5.14.0

Steps to Reproduce 🕹
Iterate on an array and set a xl, lg, .. and so on  prop dinamically:
<Grid container spacing={2}>
  {row.map(resumeItem =>
     <Grid key={resumeItem.id} item xs={12} lg={6} xl={resumeItem.space}>
       ...
     </Grid>
   )}
</Grid>

Of course space property is a number!
Context 🔦

Your Environment 🌎

Tech
Version

Material-UI
v4.1.1

React
16.8.6

Browser
Chrome

TypeScript
3.5.2

Thank you!

Look for "type widening" regarding TypeScript.


Only one client can be connected at one time.

I can't connect two devices to one ble_controller at the same time is this a limitation or a bug?
I have one esp32 that is the ble_controller and two other connecting to it.
I Just wanted to ask if it is limitation as the "normal" ble_server supports 3 connections?

That is an interesting question. I googled a little bit and found this discussion. It seems technically possible. But the lower-level APIs, which my component is based on, do not support it AFAIK. So, I will consider it rather a limitation than a bug.

How much changing does the library need to work with multiple devices

I do not know. In the other thread the guy says, he got it to work, but he also got some errors afterwards. So I am not sure if he really has a solution. ;-)

Oh, and what do you mean by „normal“ ble_server?

The one from esphome directly

Oh, that‘s interesting. Let’s reopen then.

You could use that directly instead of a 3rd party one if it allows for the full featureset

Have you found a way on how to use the ble_server because i would really like to have a ble network in my home so it even works with no wifi

Have you found a way to connect more than one device ?

Sorry, currently I’m away from my computer. Maybe I can check next month.

Have you tested if it would be possible?

Hey no problem if you don't want to implement it yourself. Have you researched it already? Would you accept a PR?


ldc: 1.30.0 -> 1.31.0, bump bootstrap ldc to 1.30.0, bump LLVM to 14

Description of changes

ldc: 1.30.0 -> 1.31.0, changelog available here. Also update the bootstrap LDC's version to 1.30.0 and build against LLVM 14 instead of LLVM 11.

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 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

[x] Fits CONTRIBUTING.md.

Oops, sorry for the hash mismatch. I think I got confused with which bootstrap version I was using. I just double checked all the hashes. Thanks!

I was not able to run nixpkgs-review (using rev HEAD or pr 218905) because it hangs on the first command: git -c fetch.prune=false fetch --no-tags --force https://github.com/NixOS/nixpkgs master:refs/nixpkgs-review/0. I shallow cloned the nixpkgs with depth 1. Could this be the issue?

So it does need a full clone... Successfully ran nixpkgs-review.

ping @ThomasMader @lionello
I think Thomas wants to update DMD and LDC in lockstep. However, I just created [another MR] regarding DMD which should make a matter of only a hash update.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/934

it would be nice to set doCheck = true, it fails some tests and the current setup -- having checkPhase without doCheck (which defaults to false) -- hides the problem


Cargo --list group installed and alias

Problem
When running cargo without a command it shows a --list option. This lists "all commands", however the text that shows implies that they are installed. This can be confusing and less helpful.
the --list command should show the different types in groups, this data is already dealt with in the function that writes too console, so it should be quick and easy, in fact i have a pr ready too submit but i'm kind of new too rust so i'm not sure if this is the best way too do this and would like some input
Proposed Solution
https://github.com/Hellzbellz123/cargo/tree/fix-list
cargo --list should group aliases, built in commands and installed commands separately
output of cargo --list should look as below
❯ cargo run -- --list
Finished dev [unoptimized + debuginfo] target(s) in 0.12s
Running target/debug/cargo --list
Configured Aliases:
t                    alias: alias: test
bump-check           alias: run --package xtask-bump-check --
....
Built-In Commands:
yank                 Remove a pushed crate from the index
add                  Add dependencies to a Cargo.toml manifest file
....
Installed Commands:
zigbuild
xwin
.....
Notes
No response

Seems like pretty much similar to #12114 and effectively the same as #12093?

See also https://github.com/rust-lang/cargo/pull/12196#pullrequestreview-1448339868

I searched for  --list and list but missed those particular issues, closing as duplicate
My apologies