New App: Bgg Last Play

Simple app that displays the number of days since the last time the given user has recorded a play on board game geek; I eventually want to pretty it up a bit, but this is a good first iteration.

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

@DanDobrick, I'm excited to see this app get submitted! I just made my first one yesterday, a very simple 'N days since X' display. My idea was to make that app to get an idea of how to work with Pixlet and everything, and then move onto making exactly what you just submitted.
Now I'll see if I can learn from Henry So Jr's BGGHotness app and combine it with your effort here to display box art and title of the last game played on a user's BGG account.

@DanDobrick, I'm excited to see this app get submitted! I just made my first one yesterday, a very simple 'N days since X' display. My idea was to make that app to get an idea of how to work with Pixlet and everything, and then move onto making exactly what you just submitted.
Now I'll see if I can learn from Henry So Jr's BGGHotness app and combine it with your effort here to display box art and title of the last game played on a user's BGG account.

That's a great idea! I want to make this app look nicer anyway, so it may be good to collaborate and integrate whatever you come up with into this app instead of having multiple bgg "last play" apps

@DTSH That's a great idea! I want to make this app look nicer anyway, so it may be good to collaborate and integrate whatever you come up with into this app instead of having multiple bgg "last play" apps

Yeah, absolutely, I meant building on top of this, though I didn't articulate it. I'll start chipping away at it, once this PR is approved and if I've made any progress I'll let you know and potentially submit another PR.


Adding / removing reactions shouldn't refresh/re-render the whole page

Whenever I add or remove a reaction, the editor tab reloads.
It would be great if it didn't need to and I could just see the added reaction without re-rendering the whole page but just the specific component.

@Stelios123 / @JChrist can you please add an estimated size here?

Result :
https://github.com/cytechmobile/radicle-jetbrains-plugin/assets/31673488/f2e6f567-376e-4f7f-a251-7c3121b98d51


fix(hierarchy-list): reset page (as necessary) when pageSize changes

Closes #2918
Summary

Previously, if you paged through the results to a higher page, but then changed the pagination (or removed it) You could get stuck on the higher page and be unable to get back to the list items.

Change List (commits, features, bugs, etc)

add useEffect to check current page and max page and reset to page 1 if necessary.

Acceptance Test (how to verify the PR)

Go the HierarchyList with large number of items story
change pageSize to small
change to page 2
change pageSize to undefined
you should return to page 1 with all results

Regression Test (how to make sure this PR doesn't break old functionality)

no adverse effects on other stories

Things to look for during review

[ ] Make sure all references to iot or bx class prefix is using the prefix variable
[ ] (React) All major areas have a data-testid attribute. New test ids should have test written to ensure they are not changed or removed.
[ ] UI should be checked in RTL mode to ensure the proper handling of layout and text.
[ ] All strings should be translatable.
[ ] The code should pass a11y scans (The storybook a11y knob should show no violations). New components should have a11y test written.
[ ] Unit test should be written and should have a coverage of 90% or higher in all areas.
[ ] All components should be passing visual regression test. For new styles or components either a visual regression test should be written for all permutations or the base image updated.
[ ] Changes or new components should either write new or update existing documentation.
[ ] PR should link and close out an existing issue

If you change the knob to page size xl then go to page 33 and then change the prop to lg the results go back to 0-9 but you remain on page 33. you are still able to go to page 32 and that seems to reset to the right number but def unexpected.

Nice work. Perhaps you want to add a unit test for this?

Yeah, I'm going to. These are just small issues I've come across adding the virtual list stuff. I'll add the unit tests in that PR when I open it to cover all the basis with both regular and virtual list content.

If you change the knob to page size xl then go to page 33 and then change the prop to lg the results go back to 0-9 but you remain on page 33. you are still able to go to page 32 and that seems to reset to the right number but def unexpected.

Adjusted this so that it stays on page 33 and updates the list to show the correct items for page 33.


BLE project compile fails with cmake (IDFGH-1623)

Currently swapping a project from original make system to the new cmake build system for esp-idf.
I have no issue when compiling and flashing the project with make but when I try to use idf.py I get a LoadProhibited guru error.
dump.txt
From my understanding the error seems to come from whenever I call any of bluetooth library functions like esp_bt_controller_mem_release or esp_bt_controller_init. Am I missing something with setup of menuconfig and not enabling something for BLE?

Can you provide 'elf' file with the log to help me identify the problem? thank you!

Hey @gengyuchao sorry for late reply here is the BIN and log:
Log.txt
SourceCode.zip

@igrr @projectgus Any thoughts?

Just to clarify: Both Make and CMake compiles the project successfully. The problem is that the binaries compiled with Make works while the binaries compiled with CMake does not (with the LoadProhibited panic)?
Is is possible for you to set the debug level to debug, rebuild and upload the logs and the new .elf again? Perhaps even the .map file.

Correct @renzbagaporo both compile without issue, but on running them the cmake binaries gives a LoadProhibited panic. I'll attach the .map and .elf below.
build.zip

Hi @lachlanleslie1995, in your sdkconfig what is the value of CONFIG_BT_RESERVE_DRAM?

Here is my sdkconfig
sdkconfig.zip

The config seems to be correct. That makes this issue really weird to me.
Some more clarification: You mention that the issue comes up whenever you call function from BLE library, yet the logs indicate the issue occurs on startup, particularly failing on this assertion:
assert(reserved[i+1].start > reserved[i].start);

I just want to verify that we are referring to the same issue.
Investigating the failed assertion mentioned, the issue seems to be the contents of reserved regions (from the map file it is from 0x3f41b288 to 0x3f41b2d0)
Taking a peek at the elf file at these addresses:
 3f41b280 bcf21240 e0f21240 e0e6fa3f 10fffa3f  ...@...@...?...?
 3f41b290 28dbfb3f 5cdbfb3f 0080fb3f 209afb3f  (..?\..?...? ..?
 3f41b2a0 0000fb3f 8863fb3f 00e0fa3f e0e6fa3f  ...?.c.?...?...?
 3f41b2b0 203ffe3f 5043fe3f 0000fe3f 4004fe3f<EMAIL_ADDRESS> 3f41b2c0 00800740 00000840 00000740 00800740<EMAIL_ADDRESS>
This region contains data of type soc_reserved_region_t, which has a start and end address members. The contents of this region, plus {&_data_start, &_static_data_end} and {&_iram_start, &_iram_end} is what's checked during startup (see function s_prepare_reserved_regions)
The issue seems to be that &_data_start and the start address at 0x3f41b2a0 in the memory region is the same, 0x3ffb0000. From the map file:
.dram0.data     0x3ffb0000     0x2958
                0x3ffb0000                _data_start = ABSOLUTE (.)
                0x3ffb0000                _bt_data_start = ABSOLUTE (.)

Ideally, _data_start should be 0x3ffb0000 + CONFIG_BT_RESERVE_DRAM so that the checks would pass. The case where these values would be the same is if CONFIG_BT_RESERVE_DRAM = 0. But your sdkconfig says otherwise.
Just to be sure, can you also upload a file called esp32_out.ld, which should be in the project's build folder. Also, may I get project_description.json?

Hey @renzbagaporo,
I have changed my code slightly since I originally created this issue as I've been continuing to develop with the old make system while this issue gets resolved but originally if I didn't call any BLE functions inside my btEnable func the system would run fine. This no longer seems to be the issue instead it's whenever I call my basic init function even with every other call inside it commented out it still seems to fail.
I have two esp32_out.ld's in my build folder one from esp-idf/esp32 folder inside:
esp32_out.zip|
and another from just esp32 folder:
esp32_out.zip
Here is the project_description
project_description.zip

Ah, the esp-idf/esp32 esp32_out.ld (used by CMake build system) exhibits the problem that I mentioned:
  dram0_0_seg (RW) : org = 0x3FFB0000 + 0x0 ,
                                     len = 0x2c200 - 0x0

0x0 is supposed to be CONFIG_BT_RESERVE_DRAM (0xdb5c based on you sdkconfig), but somehow the value is not being used here.
The presence of two esp32_out.ld suggest you are using the two build systems at the same time, which could account for the issue. Have you tried clearing the build folder and sdkconfig before building with CMake?

Ahh! That's done it, thanks a bunch @renzbagaporo . Must've not converted from the make to cmake system properly.


Ignore Sails Blueprint routes.

I can't find a way to ignore sails blueprint routes. Blueprints are disabled in the blueprint config file but they still appear in the swagger.json. I tried sails-swagger and it does ignore the blue print routes but it has a whole slew of other problems. So I'm curious if there is a way to not include the blueprint routes in the swagger.json.
For example, I have POST /api/0.1/user in my route.js file. I get that in the swagger.json but then i also get sails POST /user.
Thanks for any help.

oh ok,  would look into it now, to make it use the config instead to enable or disable default routes

Has this feature been removed since then? I seem to have the same problem with the current version and I can't immediately find the fix listed here in the current codebase

@gabeklavans What feature are you looking for specifically? This hook now listens to the router:bind events to determine routes to add - this means that if blueprint routes are disabled they will not appear in the Swagger. What behaviour are you observing?

@danielsharvey I have an action in my app named <controller>/remove which matches one of the sails blueprint actions, so I disabled blueprints in sails using rest = false and all that.
However, when I generate the swagger.json, it adds parameters I didn’t define to that route that throw “not found in path” errors, like “association” and “childId” (which I assume are due to the blueprint matching).
The workaround I used for this was to set this configuration
updateBlueprintActionTemplates: () => 'RndmString'
but this seems kinda hacky and I’m not sure if it’s doing exactly what I think it is. It does however prevent the generator from adding those parameters that don’t exist.
I’m not certain but it seems like the same problem @Kawirider636 had

@gabeklavans Thanks, I understand.
In your case, where you have disabled blueprints completely (rest = false) you can remove the blueprint templates as follows:
updateBlueprintActionTemplates: () => {}

It sounds like there is also a case for a per-action (or per-model) configuration like:
blueprint: false

which does skips application of the blueprint template for that specific item.


Create more accurate global allocation pool defintion

Currently the global allocator is allocated from a region that is found by the following algorithm:

Get the range of available addresses from the multiboot info
Get the range of excluded addresses (because they are occupied by other modules, elf sections, etc)
Allocate from the portion of the available addresses that is past the end of the maximum excluded address

This is obviously not correct. If, for example, there is an excluded address at the end of the available regions, this will fail. It is also not guaranteed to produce the largest available range.
See  https://github.com/mythril-hypervisor/mythril/blob/master/mythril_multiboot2/src/main.rs#L128

This probably relates to #85

When this is done, it would be good to develop a series of tests as well. It seems like that should be pretty doable.

When this is done, it would be good to develop a series of tests as well. It seems like that should be pretty doable.


Intellisense is not well supported in VUE

Does this issue occur when all extensions are disabled?:  Yes

VS Code Version: 1.59.1
OS Version: Windows_NT x64 10.0.18363

Steps to Reproduce:
Javascript intellisense works fine if it's in the same scope, but it stops working if the scope is different.
This happens even without the introduction of extensions.

.vue files are not supported by VS Code and you need to use extensions to get support for these custom files. E.g. Vetur

Vetur already exists in my extension, but its intellisence for this situation is still weak, maybe I should ask it for help, thank you.


feat: sort mempool txs

https://github.com/hirosystems/explorer/assets/97111756/3fe993c4-f9fe-4477-9229-b0ef01a2d697

:tada: This PR is included in version 1.143.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


quicksort.java Add Quicksort Algorithm

This adds QuickSort Algorithm which return the concatenation of the
quicksorted list of elements that are less than or equal to the pivot, the
pivot, and the quicksorted list of elements that are greater than the pivot.
Closes https://github.com/NITSkmOS/Algorithms/issues/150
By submitting this pull request I confirm I've read and complied with the
below declarations.

[X] I have read the Contribution guidelines and I am confident that my PR reflects them.
[X] I have followed the commit guidelines for this project.
[X] My code follows the standard code structure.
[X] This pull request has a descriptive title. For example, {Tag}: Add {Algorithm/DS name} [{Language}], not Update README.md or Added new code.
[X] This pull request will be closed if I fail to update it even once in a continuous time span of 7 days.
[X] This pull request shall only be reviewed and merged once all the checks passes. No maintainer or supporter shall be obliged to review it before this condition is met.
[X] I have mentioned the issue number correctly (with hyperlink) in this pull request description.

After you submit your pull request, DO NOT click the 'Update Branch' button.

Also, after amending your commit, rebase your branch from upstream branch

ack 28f4d61898f018be5ce143e9415f1023f33afeac

ack 52bd1ad8d76aa94bb16745d90c572282b05f820c

@sangamcse ff

Hey! I'm GitMate.io! This pull request is being fastforwarded automatically. Please DO NOT push while fastforward is in progress or your changes would be lost permanently :warning:

Automated fastforward with GitMate.io was successful! :tada:


NPM package releases

First we need to complete the following tasks:

[x] https://github.com/EpicGamesExt/PixelStreamingInfrastructure/issues/1
[ ] https://github.com/EpicGamesExt/PixelStreamingInfrastructure/issues/2

Once we have done this, we can further improve the user experience in using PixelStreamingInfrastructure by synthesising it's libraries into a single or multiple semantically versioned npm packages, generating these automatically as new changes make their way into this repository.

Moved from https://github.com/EpicGames/PixelStreamingInfrastructure/issues/14

Close as it is effectively duplicated by: https://github.com/EpicGamesExt/PixelStreamingInfrastructure/issues/2


Array representation: [nil]

i can print the pagedArray object content but when i do something like this pagedArray[0], the result is always nil and after debugging i get this : Array representation: [nil]

Hi! That sounds weird. I cannot reproduce this issue and there are several unit test which tests this case and passes. Can you show in a few lines of code how you can reproduce this?

self.pagedArray = PagedArray(count: (object as! Page).total!, pageSize: ((object as! Page).records_per_page)!)
// Set page items
self.pagedArray?.set(((object as! Page).records)!, forPage: self.pageNumber)
// Use array items
self.pagedArray?[0] -> nil
although when i print the pagedArray object i can see items at index 0


Make UnexpecTEDIntentPolicy compatible with E2E data

Proposed changes:

Closes https://github.com/RasaHQ/rasa/issues/9102
The approach taken here: Filter incoming training trackers to consist of only those trackers which have no UserUttered event with intent undefined or an ActionExecuted event with action name undefined.

Status (please check what you already did):

[x] added some tests for the functionality
[x] updated the documentation
[x] updated the changelog (please check changelog for instructions)
[x] reformat files using black (please check Readme for instructions)

When does this happen: ActionExecuted events with no action_name.
Is that because of an E2E prediction?

It's the debatable case that was discussed on the disentangling doc as well. When you create an E2E story like -
story:
  - user: "Hello, i'm looking for a play station"
  - bot: "Which version"

See the usage of bot instead of action. Whenever a training story like this is added, ActionExecuted events don't have an action_name but they have action_text.

@dakshvar22 This is beyond the scope of this PR but is it worth considering whether to hide E2E turns from UnexpecTEDIntentPolicy rather than removing them from training data? I'm worried that this current setup will cause UnexpecTEDIntentPolicy to do unpredictable things at run time after an E2E turn because it's input state features will include featurizations of user or bot utterances and it will not have seen those at training time or if these turns are hidden from the policy at run time, the states will still be unusual given the training data censoring. WDYT?

Let me first disable auto-merge 😅

@kedz I think you have raised a good point. Hiding E2E turns might be a good idea however I think we should think more about this problem and potential solutions when either of these features are promoted to first class features. Otherwise, we'll add a lot of code for interaction between two experimental features without being sure about either which does not seem like the best idea. I'll create a separate issue for this so that we don't lose track of it.

@dakshvar22 💯 Thanks for creating the issue!


Proposal: GenericTransformFloat

Transforms.Add(new GenericTransformFloat<MyComponent>(c,
    c => c.SomeProperty, // get
    (c, v) => c.SomeProperty = v); // set

We now have void TransformTo<TValue> which can be fed a custom Transform in the constructor. While it's not as flexible as providing the get/set directly as above, so far I think it's enough for our use-cases.
We can revisit this if providing custom one-use transforms becomes a more common thing.


Bug in the API Developer Portal list view

Description:
When clicking on the sorting arrows list doesn't get sorted

We cannot supporting sorting at the moment because the backend does not support it at the moment.
Therefore we will be removing sorting for the remaining 2 columns.
I have created this issue to track this: https://github.com/wso2/product-apim/issues/6503


[google_maps_flutter_web] Render custom Marker icons.

Description
This PR adds support for fromBytes BitmapDescriptors, so Markers can have programmatically-generated icons in web.

This also fixes an issue where custom Marker icons were not being rendered as part of the "initial geometry" (there was a TODO/question about it). Now they should work as expected.
(Tangentially, this also enables defaultMarkerWithHue, which can be implemented by computing the bytes of the rotated hue of a default Marker image, similar to the code on this jsfiddle).
Related Issues

Fixes https://github.com/flutter/flutter/issues/66622
Fixes https://github.com/flutter/flutter/issues/67048

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[x] No, this is not a breaking change.

This has been published. Please try: google_maps_flutter_web: ^0.1.0+8


sa/pos-messaging-schema-cleanup

Current dependencies on/for this PR:

feature/proof-of-stake

PR #570

PR #572   👈

This comment was auto-generated by Graphite.


Alpha matting settings don't do anything?

Hi,
Thank you for this contribution!
For some reason when I tick alpha matting setting on and try to make adjustments to the mask, they don't seem to do anything as they normally would with rembg
Am I missing something?

Thank you for bringing this up! I wasn't sure if it was normal or not, and I rarely use it.

Thanks for quick response!
None of the different models seem to affect the mask either
I know from using rembg they have huge differences
The image seems to reload and re-process, but nothing changes with the mask
Also, I've only tested the plugin on the Forge repo so far

I am confident the issue is in the extension (I'm on plain old A1111).

Reference for comarision:
https://github.com/AUTOMATIC1111/stable-diffusion-webui-rembg/blob/master/scripts%2Fpostprocessing_rembg.py
Sorry, I've been more busy than I thought I would be last week. I'll try to look into it this week.

Okay I found why it's not working!
Essentially, when you call rembg.remove with return_mask=True, it doesn't process the mask with alpha matting. It just returns it as-is. You would expect otherwise from how the function looks, but I guess that's just how it is.
The solution is to return the image without the background, and then resolve the mask manually from the result.
I'll see if I can actually resolve the mask or not from that.


Issue when run setup

I have tried running the setup file on both my personal computer and Google Colab, but both received error messages. Can you help me fix this?

It appears your torch install doesn't fit your cuda install. Look for the exact requirements of installing torch.


nixos/ipa: fix proper hostname handling

In my company nixos configuration have something like:
  networking = {
    domain = "vm.dev.company.local";
    hostName = "my-host-1";
  };
  security.ipa.domain = "company.local";
  security.ipa.realm = "COMPANY.LOCAL";

So final hostname will be my-host-1.vm.dev.comapny.local And this hostname should register into freeIPA.
But unfortunately it isn't working now like it should.
@s1341 You are the autor please check ;)
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] Fits CONTRIBUTING.md.

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

@patryk4815 have you tested this change? I cannot test as I do not have a system where hostname != fqdn.

@s1341 yes, this was tested on my internal freeipa server

@s1341 why there is requirement for ntp in this module?
    services.ntp.servers = singleton cfg.server;
    services.ntp.enable = true;

@patryk4815 I can't remember to be honest. Does it work without? If so, we can probably get rid of it.


Add Slack link to website left navigation bar

Link to the Submariner channel on the Kubernetes Slack workspace from
the section with similar links in the website's left bar.
Also slightly reorder links to reflect link weights.
Closes: #111
Signed-off-by: Daniel Farrell<EMAIL_ADDRESS>

Please check https://github.com/dfarrell07/submariner-website/pull/1


Feature/update infinite scroll

This PR adds 3 small things

waits 50 ms in between each scroll. It was 0 busy loop before which is weird.
adds an option to scroll up after each scroll down, required for https://github.com/apifytech/apify-js/issues/704 but there might be more sites where this helps
adds an option to click on buttons, needed for example at https://www.google.com/search?q=google+images&source=lnms&tbm=isch&sa=X&ved=2ahUKEwiilO6kidTpAhXMm6QKHabYCk8Q_AUoAXoECBYQAw&biw=1536&bih=792

I haven't changed the test suite because it would have to be tested on real pages. But I tested it on actual use-cases and it works.

I was wondering why the builds failed :) Now I realize that if I click through, it will show me the errors. Dummy Github editor :)


Implement (or document how to use) other input formats

The Graph.fromdict method lets us accept data from several formats in addition to the current YAML.
I have had some motivation recently to explore TOML input for demes, and code like this below "just works":
# requires python >= 3.11 (part of std lib)
# tomllib does not WRITE to toml.
import tomllib

# to read/write toml
import toml

# A toml writer mentioned in tomllib docs
import tomli_w

import demes

with open("example.toml") as f:
    input = f.readlines()

input = "".join(input)

print(input, type(input))
parsed_with_toml = toml.loads(input)
parsed_with_tomllib = tomllib.loads(input)

print(f"parsed with toml = {parsed_with_toml}")
print(f"parsed with tomllib = {parsed_with_tomllib}")

graph_from_toml = demes.Graph.fromdict(parsed_with_toml)
graph_from_tomllib = demes.Graph.fromdict(parsed_with_tomllib)

print(f"demes.Graph from toml: \n{graph_from_toml}")
print(f"demes.Graph from tomllib: \n{graph_from_tomllib}")

assert graph_from_toml.isclose(graph_from_tomllib)

# NOTE: tomllib does NOT support WRITING toml...
graph_as_toml = toml.dumps(graph_from_toml.asdict())

print(f"graph back into toml = \n{graph_as_toml}")

# ... but tomli_w does

graph_as_toml_2 = tomli_w.dumps(graph_from_toml.asdict())

print(f"graph_from_toml via tomli_w = \n{graph_as_toml_2}")

The contents of the example are:
time_units = "years"
description = "a description"
generation_time = 25

[defaults]
[defaults.migration]
rate = 0.25
demes = ["A", "B"]

[[demes]]
name = "A"
[[demes.epochs]]
start_size = 100

[[demes]]
name = "B"
[[demes.epochs]]
start_size = 42

I also suspect that we get JSON support for free in a similar way.
We could add functions like demes.loads_toml (or toml_loads??), etc., which would be very thin wrappers around code like the above.
Or we can document somewhere that the asdict method is more useful than we've let on.
Thoughts?

Hey @molpopgen.
The Graph fromdict(), asdict() and asdict_simplified() methods are 100% intended for serialisation-format interoperability like this. TOML was mentioned as a candidate format in our initial discussions of Demes file formats, but I think the way of writing lists and doing nesting in TOML was seen to be more confusing than in YAML. From memory, RON (Rusty object notation) looked really approachable, but unfortunately lacked parsers outside of Rust. We could certainly include some example docs showing how to do serialisation to TOML or other things, as you've outlined above. We do already have JSON support built in (it's a proper subset of YAML after all, and the dump() functions accept a format="json" option too), but this is intended for applications where including a YAML parser would be a pain (e.g. reading a Demes file into SLiM).
In regard to including support for more formats natively in the library, I'd want to see the serialisation format used elsewhere for Demes files before considering it, to help justify the use case. I mean, we could add support for 20 additional formats with little code, but how useful would this be to users?

Thanks @grahamgower.
After a bit more exploring, I found that several of the demes-spec valid examples are not compatible with JSON or TOML as inputs.  For JSON, the infinite start time is the issue.  For TOML, the metadata example as foo: null which is out of spec. (For TOML, null values must be omitted.)  All of that can be handled but it takes some work.
I was actually finding the TOML way of doing lists easier of late.
My use case is for input for a program that I'm working on.  To support the current YAML input, it would be a string literal buried inside a file that is otherwise TOML.  Writing it "inline" as TOML is easier.
But it sounds like the thing to do is to just add some text to the fromdict docstring.


Tests for flutter test [some_directory]

Adding test coverage to be sure we can test a directory's contents instead of just its children.

Please ignore the codecov bot. It is correct, but still a little strict.

FYI: https://github.com/flutter/flutter/wiki/Tree-hygiene#squashing-commits
The commit message for this commit contains:
"""

Add a test for a directory instead of a single test.

Add test data to a child directory to test the command.

Add test data to a child directory to test the command.

Add test data to a child directory to test the command.

Correct test.
"""

Thanks Todd, I will keep this in mind going forward.


chore(main): release 0.4.11

:robot: I have created a release beep boop
0.4.11 (2024-04-08)
Performance Improvements

deps: Check dependabot daily (07d57c2)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/linrongbin16/cargo-ci-template.rs/releases/tag/v0.4.11 :sunflower:


web-push-libs/web-push-php + License

cc @Minishlink @marco-c
Just wondering how you both saw this repo vs the web-push-libs repo working together, looks like this repo has a couple more fixes than the web-push-libs version, @Minishlink would you be willing to move the project full time over to web-pushlibs?
You should have full admin rights over the repo and can lock down the master branch so only PR's you're happy with land etc.
Is there any chance you'd be willing to change the license to something like Apache 2, MIT or MPL? Main reason being it should enable some developers to clearly identify what the the restrictions are for the license.
Would either of you be able to enable issues on https://github.com/web-push-libs/web-push-php ?
Cheers,
Matt

I consider the web-push-libs/web-push-php repo as the "very stable" version. I'll merge as soon as VAPID works well. Currently, it works for me in Firefox but not Chrome, though others have it working with Chrome.
The license is MIT, I don't know why it doesn't appear in the GitHub header of the web-push-php repo.
Issues enabled. :)

I consider the web-push-libs/web-push-php repo as the "very stable" version. I'll merge as soon as VAPID works well. Currently, it works for me in Firefox but not Chrome, though others have it working with Chrome.

Perhaps it would be less confusing if there was just one repository, with a devel branch for the unstable version.
You would still be the owner of the repo, merge PRs, etc..
In https://github.com/Minishlink/web-push/issues/34#issuecomment-246789083, you said you wanted to keep it in your repositories to avoid problems with composer/packagist. I've just remembered that I've done this in the past (https://github.com/mozilla/wp-web-push/commit/fcfde518c6f5a557a63cf162b5a12a0940a8e5ae) and haven't run into problems.
Btw, you could test it and if it doesn't work revert the change.

Hey guys, I deleted the old "web-push-php" because the "minishlink/web-push" had the issues history, more watchers, stars, and forks. Looks like the transition worked well.

Awesome, thanks!


Improve ACF install error reporting

This fixes an issue where the authRc variable was shadowed in the ACF install path so it was not set to any meaningful value.
This improves the error code produced by the ACF install path.  The main observation is that multiple public keys are attempted and all but one of them will fail with CeLoginRc::RcBase::SignatureNotValid.  That is a valid error code, but errors from keys where the signature was valid should take precedence.  For example, consider when the second key is valid but the ACF is expired: the SignatureNotValid from the third key should not mask this more significant error code.
Tested: Yes
Scenarios: valid ACF installed, incorrect signature, expired ACF. The caller writes the error code into a journal entry.

This should be backported at least to FW1060.


fix bug with triggers encoded in triggers file

also to check oci terraform provider; and also to check that all existing triggers in wolfi are non-failing, as they will start to be executed for the first time.


TensorFlow not working in HA Docker 0.84.6

Home Assistant release with the issue: 0.84.6
Last working Home Assistant release (if known): never
Operating environment (Hass.io/Docker/Windows/etc.):
Docker
Component/platform:
https://www.home-assistant.io/components/image_processing.tensorflow/
Description of problem:
Tensorflow is not starting, hangs HA
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
---
platform: tensorflow
scan_interval: 20000
source:
  - entity_id: camera.front_hikvision
model:
  graph: /config/tensorflow/frozen_inference_graph.pb

Traceback (if applicable):
2018-12-31 16:49:38 INFO (MainThread) [homeassistant.setup] Setting up image_processing
2018-12-31 16:49:38 INFO (MainThread) [homeassistant.components.image_processing] Setting up image_processing.tensorflow
2018-12-31 16:49:38.667433: F tensorflow/core/platform/cpu_feature_guard.cc:37] The TensorFlow library was compiled to use AVX instructions, but these aren't available on your machine.

Additional information:
Ive followed this script
https://gist.github.com/det-peralta/5753f37f2bb46dce71b9fcc3cbef8f05

What CPU are you running?  Is your docker image being run directly on the host, or in a virtual machine?  See https://github.com/home-assistant/home-assistant/issues/18435 for a similar issue that was resolved when a user was running Docker inside a VM, but did not have the VM configured to use the Host CPU.

https://www.synology.com/en-global/products/DS918+#specs
CPU: Intel Celeron J3455
There is specify command/settings to use host CPU?
docker run -d --restart=always -v /pathconfig/:/config -e TZ=Europe/London --name ha -p 8123:8123 homeassistant/home-assistant

The Celeron CPU you are using uses the Goldmont microarchitecture, which does not have support for the AVX extension.  See here for TensorFlow hardware requirements.
If you want to run TensorFlow in Home Assistant on your Synology, you can extend the Home Assistant docker image and build TensorFlow from source.

I also though on the beginning that there's a problem with HA/Tensorflow in Docker, but then realized what hunterjm said - my Celeron does not support AVX.
@hunterjm - are you suggesting above that it is possible to build Tensorflow on Synology and still be able to use it even though these CPUs do nto support AVX? Could you elaborate a bit more?
I'm asking as I'm interesting exactly in this - to run Tensorflow on my Synology and use it with HA.
Thanks!

The pre-built TensorFlow pip packages are compiled to require AVX on the CPU.  We need to update the docs to reflect this, as we can not add support for all CPUs with a single docker image.


Missing comma

I find missing commas
Please confirm

Trailing commas are optional, and there doesn't seem to be consistency across the repo. I wonder what the preferred style is?

The preferred style is to have a trailing comma, however this PR has languished for quite sometime and because of the current sync process it would be much easier if someone could just hunt down all missing cases and fix them in a single PR, thanks though!


Outdated documentation for Partner API Create Referral

Hello,
Can you please review the example to create a new Referral as partners report that the example in the documentation is out of date: partner-rest/partner-rest/develop/create-a-referral.md

Hello, we are archiving this repo and moving its content to the central Partner Center developer content location at https://learn.microsoft.com/en-us/partner-center/develop/create-a-referral. This feedback has been logged internally and we will use it to update the content in the future.


[BUG]: Cannot download Master quality

Which tool
tidal-gui and tidal-dl
Version
<IP_ADDRESS>
Platform
windows11
Describe the bug
When the sound quality option is set to master and the song has master resources, a "cannot get the streamUrl" error occurs. Both Tidal-GUI and Tidal-dl have this issue.
Relevant log output
DL Track[Trio Sonata No. 6 in G Major, BWV 530: I. Vivace (Arr. for Mandolin, Cello, and Double Bass)] failed.Can't get the streamUrl, type is application/dash+xml

Add screenshots
No response

I guess it is because tidal removed MQA from their whole library.
You can read more here: https://support.tidal.com/hc/en-us/articles/25876825185425-Audio-Format-Updates

Elsewhere in this Issues List is published a technique to acquire an AccessToken that can be used to enable MAX HiRes downloads.
This is successful, but not much of the Tidal content appears to be in 192khz 24-bit encoding yet.
The current gui app does not distinquish between which search results are regular or Max hi-res.
The [M] MQA indicator is no longer applicable.
In the gui version you'll often see multiple listings of the same album.  Each usually represents a different encoding.
By trial-and-error you can find one that is Max HiRes (24-bit and such).
Tidal will withhold offering the MAX versions if it thinks the device is not capable of playing that format.
Getting an Auth Token from the official Tidal App allows for that access.

I guess it is because tidal removed MQA from their whole library. You can read more here: https://support.tidal.com/hc/en-us/articles/25876825185425-Audio-Format-Updates
My guess is, if there is no MQA anymore, then the GUI cant get the stream for it.

Yes change the download quality at the bottom of the search page from master to HI-FI

Any news or FIX on how to get Hi Res working after the Tidal changes ?

Any news or FIX on how to get Hi Res working after the Tidal changes ?

I sadly have no idea when it will work again. This project seems very abandoned...
In case someone makes a tutorial / fixed .exe of the GUI i would like to be informed too...

Th GUI app CAN download Hi_Res... up to 192khz 24bit encoding.
However the app does not show which files are "lossless" vs "hi res lossless".
Maybe someone has modified a version to fix these limitations... check the forks.. I don't think Yaronzz is working on this version though.
The Hi_res download can be enabled in the GUI app by manually getting an AccessToken as described elsewhere here.
It works for about 24 hrs, then you need to get a new token.
I do like the way this gui app names and organizes the downloads and sets the metadata in the files.
There are other DL apps out there that show more indicators of which particular download options are Lossless (44khz 16-bit)  vs HiRes lossless (usually 24-bit, up to 192khz sampling).

"The Hi_res download can be enabled in the GUI app by manually getting an AccessToken as described elsewhere here."
"There are other DL apps out there that show more indicators of which particular download options are"
I did try to find the AccessToken using the Charles App - but it just displays encoded stuff, not anything in English so the search for the token was fruitless.
Any other Apps (links) you can recommend that work at this time?

"The Hi_res download can be enabled in the GUI app by manually getting an AccessToken as described elsewhere here." "There are other DL apps out there that show more indicators of which particular download options are"
I did try to find the AccessToken using the Charles App - but it just displays encoded stuff, not anything in English so the search for the token was fruitless. Any other Apps (links) you can recommend that work at this time?

Look for the instructions on FIDDLER app.  it works for me

Look for the instructions on FIDDLER app. it works for me

I am trying Fiddler now but again can't find "Authorization" anywhere.
Am I doing something wrong?

I don't have any problems in downloading master files

I had an issue within an album. Some tracks download without any issue and some tracks downloaded with "Decrypt failed" error.

Im not a coder anymore (no python experience) but the Decrypt error is possibly due to a memory handling bug.
The "decrypt failure" error seems to happen on the much larger 24-bit /192khz files that are over ~ 250MB in size.
If you have multiple threads enabaled,  switch to downloading a single thread (1 track) at a time.
If you're downloading entire album,  and 1 super large track fails, delete the failed track and then go back and download that 1 track by itself.
Yes it's messy... and Ive found that TIDAL-GUI usually cant EVER successfuly decrypt anything > 450MB.
Code just needs some tweaking to resolve this issue.
I'm pretty sure the original design didn't account for flac files of this size.

@Neuralized-III can you please upload your "tidal-ini" file without actual accesstoken, since tidal-gui.exe is single file app I've put tidal-ini file in the same dir with only one line accesstoken=xxxxxxxxx, should it be quoted? does the tidal-ini need .ini extension as well? do I need any other config in it?
thanks


UVC support?

May support uvc lib? just like: https://github.com/saki4510t/UVCCamera/

I have no plan to do it. Anyway I want develop a custom option for render a surface with your own video data.
You can do it with a little modification. This is really similar to camera2 builder

OK!

@pedroSG94
I wrote a demo here :
https://github.com/jp1017/UVCCamera/blob/master/usbCameraCommon/src/main/java/com/serenegiant/encoder/MediaEncoder.java#L632,
but it results some warns:
java.lang.IndexOutOfBoundsException: offset=13, count=-3798, length=1300
    at java.nio.Buffer.checkGetBounds(Buffer.java:173)
    at java.nio.DirectByteBuffer.get(DirectByteBuffer.java:107)
    at com.pedro.rtsp.rtp.packets.H264Packet.createAndSendPacket(H264Packe
    at com.pedro.rtsp.rtsp.RtspClient.sendVideo(RtspClient.java:553)
    at com.serenegiant.encoder.MediaEncoder.drain(MediaEncoder.java:632)

This makes the APP just run for less than one minutes, can you review my code and give some advice? :+1: :smile: :100:

Set bytebuffer position to 0 before send the packet to rtsp module.
You can try too overriding buferinfo.size with bytebuffer.remaining and discard frames with bufferinfo.size < 0
If you can tell me line crash in rtsp module maybe i can help you more.

3G network results in this problem and WIFI is OK.


Relay Subscription on edges giving error.

Describe the bug
Have a schema USER which has a field addressess. The field addressess is a list of address[type address] associated with User. I have created a subscription, whenever a new user is created i should be able to get the new user details along with the address and display it. However I am getting the below error
Cannot query field 'edges' on type 'Address'. (line 7, column 9):↵        edges {↵        ^"
Note the field addressess in USER is of types AddressConnection. Therefore the relay compiler is not giving any error.
The subscription schema is as below:
subscription userSubscription {
  User(filter: {mutation_in: [CREATED]}) {
    mutation
    node {
      name
      addresses {
        edges {
          node {
            pincode
            id
          }
        }
      }
      id
    }
  }
}

To Reproduce
Steps to reproduce the behavior:

Go to https://gitlab.com/ankit5174/relay-subscription-example and clone this app
Install and start the project using npm start
Go to chrome =>localhost:3000 => f12 => network =>WS => Frames
See error in payload as
Cannot query field 'edges' on type 'Address'. (line 7, column 9):↵        edges {↵        ^"

Expected behavior
A proper Connection of Web socket should be established
Screenshots

**Versions

OS: ``Windows 10`

Hi @ankit5174
Sorry for a very late reply.
Looks like you are using the query improperly, for relational stuff you don't any relay styled pagination structure. You can directly query them like so:

So use the following query:
subscription userSubscription {
  User(filter: {mutation_in: [CREATED]}) {
    mutation
    node {
      name
      addresses {
        pincode
        id
      }
      id
    }
  }
}


limit reset/persistence to legacy items

following the change in https://github.com/apache/brooklyn-server/pull/814,
legacy compatibility caused persistence of registered types from bundles, unnecessarily (and error-prone on rebind where it expects a real CatalogItemDto)

Note this fixes https://issues.apache.org/jira/browse/BROOKLYN-539

Thanks @ahgittin - confirmed this fixes it for me in Karaf. Merging now.


Support automatic selection of whether to load kprobe

Signed-off-by: jundizhou<EMAIL_ADDRESS>
Description

Related Issue

Motivation and Context

How Has This Been Tested?

I review the code of agent-libs, the name SCAP_UNKNOW_KPROBE cannot be used for the program type of tracepoint (scap_bpf.c, line: 649, 837). I think some name like SCAP_LOAD_FAILED may be better.
Besides, UNKNOW should be UNKNOWN.
Others: format for scap_bpf.c:839: redundant blank before delimiter.
For this commit, I think the update aims to tolerate some errors of loading kprobe programs, changing the title may be better. Besides, I wonder if the program will output error info when skip loading kprobe programs

我会修改unknow->unkown，代码逻辑只有kprobe会返回这个类型。

我会修改unknow->unkown，代码逻辑只有kprobe会返回这个类型。

looks like tracepoint may reach this branch. scap_bpf.c

Please update the changelog. I suggest this as an enhancement.


Improve Code Quality by Resolving Linting Issues

Title: Improve Code Quality by Resolving Linting Issues
Description:
This GitHub issue is dedicated to enhancing code quality across our server and all packages within the lib folder by addressing linting issues. Linting issues can impact code readability and maintainability, and resolving them is crucial for maintaining a consistent and high-quality codebase.
Tasks:

Linting Analysis:

[ ] Conduct a comprehensive linting analysis of the server code and all packages within the lib folder.
[ ] Identify and document all linting issues, including code style violations, formatting errors, and potential improvements.

Linting Resolution:

[ ] Address and resolve the identified linting issues in the server code.
[ ] Address and resolve the identified linting issues in all packages within the lib folder.
[ ] Ensure that code style, formatting, and best practices are consistent and follow established guidelines.

Code Quality Improvements:

[ ] Make improvements to the codebase beyond just resolving linting issues where necessary.
[ ] Enhance code readability, maintainability, and overall quality during the process.

Acceptance Criteria:

All identified linting issues are resolved in the server code and all packages within the lib folder.
Code style, formatting, and best practices are consistent and meet established guidelines.
The codebase exhibits improvements in code quality, readability, and maintainability.

Additional Information:

Resolving linting issues contributes to a more consistent and high-quality codebase.
Improvements made during this process enhance the development experience for the team.
Rigorous testing should be conducted to ensure the code remains functional and reliable after linting improvements.

I would like to work on this issue.

@Sarveshk76 Awesome


bug: can't use extension -> black screen after opening

Summary
I can't use extension.
Black screen after opening a log file.
Steps To Reproduce
Apex Log -> Right Clic -> Log: Show Apex Log Analysis
Actual result

Additional information
VS Code Version: 1.84.2
Log Analyzer Extension Version: 1.10.3 (same issue since 1.7.0) -> v1.6.0 works
OS and version: Windows 11 23H2

@Arkin69 Thanks for reporting. To get this fixed can you help out with some more details?

Open dev tools and share any errors you see there (chrome menu -> more tools -> developer tools)
Check and share errors in the extension output (in vscode -> view -> output -> change the option in the picklist to "Lana")

Ok i can see in dev tools an error.
I think i have found the issue my user in windows has a space between my firstname and lastname

Lana output :
Registered command 'Lana: Retrieve Log'
Registered command 'Lana: Show Log'

Thank you for the error message, I have managed to track down the bug.
It turned out to only apply when the code was minified in the final release version.
The issue appears when the path to the extension included a space (like you saw with you user name)
I should have this resolved and a patch released by the end of the week, maybe today or tomorrow if I find the time.

Fixed and a patch released. This should appear on marketplace soon.


Not all types from chrome present in the package

I tried to replace each chrome with browser and this code didn't compile:
import TabActiveInfo = browser.tabs.TabActiveInfo;

Not sure which type definitions you had before, but there is no type TabActiveInfo in web-extensions. Neither in the MDN documentation nor in the schema.json files from Mozilla.
DefinitelyTyped seems to have it, so I'm guessing you've been using this. A lot of these types are not defined in the schema files, so they are autogenerated in webextension-polyfill-ts. DefinitelyTyped wrote this by hand, so they could fine-tune it.
In this case, I'm assuming you are interested in the type for the onActivated parameter. For webextension-polyfill-ts, this type is automatically as OnActivatedActiveInfoType (see onActivated.
I know, not as nicely named as TabActiveInfo. It's not as easy to avoid when auto-generating names, which are not present in the schemas (and there are a lot of them). It would be nice to have a map of autogenerated names
Long story short: You can import these types like this:
import { Tabs } from "webextension-polyfill-ts";

function onTabActivated(activeInfo: Tabs.OnActivatedActiveInfoType) {
    // ...
}

Or, if you want to keep using the identifier TabActiveInfo:
import { Tabs } from "webextension-polyfill-ts";
type TabActiveInfo = Tabs.OnActivatedActiveInfoType;

function onTabActivated(activeInfo: TabActiveInfo ) {
    // ...
}

Hope this helps :)

Everything is correct.
Too bad the switch from DefinitelyType is not as smooth as expected.
I guess there is nothing to do but to maintain the mapping.
Many people would use this project if the transition was smooth.


Add a minimal .gitignore file

In git, ignore all build* directories by default. This is particularly handy if we want to use the standard mkdir build & cd build & cmake .. commands.
Also, some IDEs (e.g. VSCode) do similar things.

Every developer uses different names for his build directories, so this patch won't match everybody and the .gitignore file needs to be updated for new developers. Therefore I'll reject this PR.
I suggest adding a global .gitignore file covering the needs for your build infrastructure, that will be used for all of your git projects automatically.


Update GitHub Actions workflow to use new features

Run tests on Linux, Windows, and macOS.
Show debug logs during tests.

Fixes #26.

Looks like this exposed a bug in how Content-Length is handled, just on Windows. That's why it's a good idea to run tests across many platforms!


feat: also copy over CNAME from root

Copy the CNAME file from the root directory as well. Allows for deployment to GitHub pages with a custom domain
closes #1

Thanks, the contribution is good.
It's just failing the check for code formatting with "black".

Should be addressed now.


Jesse: Watch (?-i:3BXGHXLPwCw)(?#YouTube)

Jesse requests the watch of the watch_keyword (?-i:3BXGHXLPwCw)(?#YouTube). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by cigien in Charcoal HQ


Test adapter crash

Dev 15
I don't have IronPython installed.
I also only installed Data Science components.
I had a flask/jade project open.
Test window had been opened but was now closed.
New Project, IronPython WPF application.
Tried to open IronPython WPF app immediately after startup..  Once it crashed, the other time it didn't.
Need to retest with newer build.
Crash:
Application: devenv.exe
Framework Version: v4.0.30319
Description: The process was terminated due to an unhandled exception.
Exception Info: System.NullReferenceException
   at Microsoft.PythonTools.TestAdapter.TestAnalyzer.GetTestCases(System.String)
   at Microsoft.PythonTools.TestAdapter.TestContainerDiscoverer+ProjectInfo+<AnalysisComplete>d__5.MoveNext()
   at System.Runtime.CompilerServices.AsyncMethodBuilderCore+<>c.<ThrowAsync>b__6_0(System.Object)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(System.Delegate, System.Object, Int32)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(System.Object, System.Delegate, System.Object, Int32, System.Delegate)
   at System.Windows.Threading.DispatcherOperation.InvokeImpl()
   at System.Windows.Threading.DispatcherOperation.InvokeInSecurityContext(System.Object)
   at MS.Internal.CulturePreservingExecutionContext.CallbackWrapper(System.Object)
   at System.Threading.ExecutionContext.RunInternal(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object, Boolean)
   at System.Threading.ExecutionContext.Run(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object, Boolean)
   at System.Threading.ExecutionContext.Run(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object)
   at MS.Internal.CulturePreservingExecutionContext.Run(MS.Internal.CulturePreservingExecutionContext, System.Threading.ContextCallback, System.Object)
   at System.Windows.Threading.DispatcherOperation.Invoke()
   at System.Windows.Threading.Dispatcher.ProcessQueue()
   at System.Windows.Threading.Dispatcher.WndProcHook(IntPtr, Int32, IntPtr, IntPtr, Boolean ByRef)
   at MS.Win32.HwndWrapper.WndProc(IntPtr, Int32, IntPtr, IntPtr, Boolean ByRef)
   at MS.Win32.HwndSubclass.DispatcherCallbackOperation(System.Object)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(System.Delegate, System.Object, Int32)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(System.Object, System.Delegate, System.Object, Int32, System.Delegate)
   at System.Windows.Threading.Dispatcher.LegacyInvokeImpl(System.Windows.Threading.DispatcherPriority, System.TimeSpan, System.Delegate, System.Object, Int32)
   at MS.Win32.HwndSubclass.SubclassWndProc(IntPtr, Int32, IntPtr, IntPtr)

It just crashed in a solution with non-IronPython projects.  It happened right after I renamed a file using F2.


Use Setuptools build system, and Pip for install

Distributing applications via Zip archive is not reliable with current Python packaging and execution practices (relative import becomes infeasible).
The Setuptools library allows for a standard build system for Python applications, that can be managed with the standard Pip package manager.

I have implemented changes to allow Pip to read a Setuptools build system for this application. See the branch wip/feature/setuptools in my personal fork repository.

I have no problem adding pip support, but I like having a single file build option.
I've reworked the zip process to include a stub that lets me import things absolutely.  Can you explain more about what you mean by this causing a problem with "current Python packaging and execution practices"?
Like I said, I can add pip support as well, but aside from that, is there a problem with an end user just carting around a xyppy.py file in the way that the zip file is being made as of this latest commit?
7386843ae2dbade014d8463547ef831e1f5b9f86

I know this is an old issue but and what I'm asking here is tangential to that discussed above, but how would the build script provided be modified if there were other directories? For example, the build script now works fine if the assumption is that all files are in the xyppy directory, as they are.
But if you wanted to scale this and have directories within xyppy the build script would not generate a usable file because module imports would no longer resolve.
If that's not possible to do with a build script like this, that would be one major argument I could see for going the pip and build system approach, since it would let you scale the project.

That's because I'm using os.listdir instead of os.walk, so files deeper than the first layer aren't zipped up.
I did that because I occasionally had uncommitted test code in deeper folders that I didn't want added.
I just did a quick sanity check on that and using os.walk works fine for as deep a structure as you want.

Interesting. I'll have to practice with that. The problem I was having was imports into a directories was not working. But I wasn't using os.walk. It looks like when you do, you have to change up the join() a bit. But I appreciate the insight in os.walk. I'll play around with that.

The executable zip didn't even have those directories in it, hence the error messages.
os.listdir doesn't recurse down into subfolders so nothing below the first layer was added.
But os.walk does exactly that kind of recursion.
And here's my sanity check code for reference.
Hope all this helps. Cheers.

I was just going to mention the same thing.

pip install xyppy
Nothing happens.

look
It is pitch black. You are likely to be eaten by a grue.

lamp on
I see no lamp here.

I'll try @bignose-debian 's solution and see what happens.

merged #7 which allows for pip install from the repo.


MatMenu: full size menu

Bug, feature request, or proposal:
It would be nice to have a full-size menu option for mobile or maybe the ability to adjust the x and y locations and size (height/width) of the overlay.
The overlay is being set by the component and forces the size of the menu and the only way to change this is to extend the mdMenu to allow for specific heights and widths of the menu as well as specific positioning.  Or in my case create a component from scratch to do this.
What is the use-case or motivation for changing an existing behavior?
My main concern with this request is that it's not in the material spec, however the angularjs version of the menu does have width options.
Is there anything else we should know?
If this isn't a good feature for the menu, what would be the best approach to creating a full-size mobile menu?

It would also be nice if we could set it to attach to another element.
eg. have the "Open Menu" trigger at the bottom of the page but have the acctual  menu open at top.

(revisiting old issues)
I think this is actually something we should eventually explore for all overlay-based components (menu, select, etc.). We already support it in the datepicker with the touchUi option. That said, I don't see us getting to this any time in the near future.

(revisiting old issues)
I think this is actually something we should eventually explore for all overlay-based components (menu, select, etc.). We already support it in the datepicker with the touchUi option. That said, I don't see us getting to this any time in the near future.

(revisiting old issues)
I think this is actually something we should eventually explore for all overlay-based components (menu, select, etc.). We already support it in the datepicker with the touchUi option. That said, I don't see us getting to this any time in the near future.

That's the first thing I was googleing for: matmenu touchui


Sensor streaming issue and Enabling Librealsense Logs Unity

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
{ L515 }

Firmware Version
<IP_ADDRESS>

Operating System & Version
{Win (11) }

Kernel Version (Linux Only)

Platform
PC

SDK Version
{ legacy / 2.0 }

Language
{c#/unity }

Segment
{ }

Issue Description
I have a question.
I am running L515 sensors using Realsense unity package. When I start and stream through a particular sensor multiple times, the sensor is not able to receive frames after some time.
What I am doing is

start a sensor
start streaming
stop sensor

when repeated multiple times after some iterations no frames are being received from sensor.
And the interesting thing I am noticing is when I start the above process again the same sensors starts receiving frames.
So I wanted to check the realsense logs generated for sensors to debug.
but unable to find a way to get those.
I tried the below link to enable the logs but unable to find the logs. And still don't have any idea if the logs are being generated or not.
https://github.com/IntelRealSense/librealsense/wiki/Troubleshooting-Q%26A#q-how-do-i-enable-librealsense-logs

Hi @KIL299  The path of the folder on Windows where logs are stored is C: > Users >  Documents
For example, if your Windows username was KIL299 then the path to the log file location would be C:\Users\KIL299\Documents

Hi @MartyG-RealSense
I have a attached a log file below
here's what I am doing

Starting 8 sensors
starting stream for those 8 sensors
stopping those 8 sensors

repeat the above steps multiple times
In my case it runs for about 3-4 times then suddenly we couldn't receive frame points from device
LibrealsenseLogs_.txt.zip

If you are enabling 8 L515 cameras simultaneously then doing so without problems would require a powerful computer, as the more cameras that are active simultaneously, the more of the computer's resources that are consumed.  For just 4 cameras, a computer with a processor chip equal to Intel Core i7 is recommendable.
A way around this processing burden is to use hardware sync.  When used with the L515 camera model, it cycles through the multiple attached cameras, activating capture at different instants so that all cameras are not active simultaneously.  Intel provide a guide about this at the link below.
https://dev.intelrealsense.com/docs/lidar-camera-l515-multi-camera-setup
In Unity, hardware sync can be enabled by adding a RealSense wrapper component called RsDeviceInspector to your RsDevice object using the 'Add Component' button at the bottom of the Inspector and typing 'rsdeviceinspector' into the Add Component interface's search box.
The settings of RsDeviceInspector are initially hidden but once a scene is played and the camera begins streaming, the settings become accessible and a tick can be placed in a tick-box beside the InterCamSyncMode option to enable that camera as a hardware-synced one.

Alternatively, you could use a more powerful computer to enable 8 cameras simultaneously without using hardware sync.  A processot equivalent to an i9 may be able to handle this processing burden.

@MartyG-RealSense
we are currently using processor chip equal to Intel Core i7.
will try the intercamsync mode as you suggested and will get back to you.

Thanks very much for the update.  I look forward to your next report.  Good luck!

Hi @MartyG-RealSense
I have uploaded below file that was generated when device was not able to capture streams on starting streams.
and i would like to specify that I have more than 3 setups available.
out of these setups I am getting the issue in one of the setup.
Do you have any idea that why would this happen?
Only one particular sensor is causing the issue.
RealSenseLogs.txt

Have you tried unplugging the micro-sized end of the USB cable from the base of the camera, reversing the connector's orientation and plugging it back into the L515 camera, please (USB-C cables are two-way insertion at the micro-sized end).  A characteristic of the L515 camera model is that it works best when the connector is in one particular orientation of the two insertion directions available.

Regarding the error: GetNativeMediaType(sIndex, k, &pMediaType.p) returned: HResult 0xc00d36b3: "The stream number provided was invalid." - this is a highly rare error that has only appeared a couple of times in past cases, and none with L515.
It is likely to be an issue with the camera rather than the computer or the SDK installation, as your other L515 work with it.
You could try downgrading your L515's firmware version from <IP_ADDRESS> to <IP_ADDRESS> to eliminate the possibility that the firmware driver is damaged on that particular camera.
https://dev.intelrealsense.com/docs/firmware-releases-l500

Okay will look into and get back to you.

Hi @KIL299  Do you have an update about this case that you can provide, please?  Thanks!

Hi @MartyG-RealSense
I tried the connection disconnection but not improvements were found
so need to check by swapping the sensors
Will get back to you once it's tested.

Hi @KIL299  Do you have an update about this case that you can provide, please?  Thanks!

Hi @KIL299  Do you require further assistance with this case, please?  Thanks!

Yes, it is totally fine to close this case and return to it at a future date if you need to.


Epic 7453: Implement Error Codes for HL7 Validation

This PR Implements the ErrorCodes feature as described in the linked epic https://github.com/cdcgov/prime-reportstream/issues/7453.
This feature was implemented in such a way that it will not affect the universal pipeline. The universal pipeline (UP) has complete discretion over what errorCodes they want to use, if any, and where/how they want to use them. The algorithms implemented in this PR ONLY effect the old pipeline, and specifically hl7Serializer. It should be mentioned that this implementation keeps the use of two fields in actionMessages: errorCode and fieldMapping so messages returned by UP will contain these, even if unused by the pipeline.
Test Steps:

Run Unit Tests
Run Integration Tests

Changes

Functionally everything stays the same (A prototype version of error codes already existed prior to this ticket). When returning validation errors/warnings, the JSON will still have the errorCode parameter and no additional parameters were added. What changed was how we calculate those errorCodes, which is all done on the back end.
New integration tests to verify the correct error codes get thrown when certain HL7 errors occur.
Front end code had a mapping of errorCodes to messages, so I updated that since I renamed the existing errorCodes.

In the screenshot below, you can see what the JSON error/warning response looks like and what it looks like in the validation UI. Its important to note the fields relevant to this discussion:

errorCode - the error code returned by the back end
field - the HL7 "field" or CSV "column" or FHIR "path"
message - the message calculated by the backend. The front end can display this if they want, but can also create their own message based off the errorCode/field combination. The back end does not make any guarantees that the "message" will be presentable to an end-user.

Checklist
Testing

[x] Tested locally?
[x] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
[x] (For Changes to /frontend-react/...) Ran npm run lint:write?
[x] Added tests: Yes, Integration tests to verify error codes and unit tests for high-risk functionality.

Process

[ ] Are there licensing issues with any new dependencies introduced?
[ ] Includes a summary of what a code reviewer should test/verify?
[ ] Updated the release notes?
[ ] Database changes are submitted as a separate PR?
[ ] DevOps team has been notified if PR requires ops support?

Linked Issues
The following are in order of completion (if that helps to logically understand the feature)

Fixes #7526
Fixes #7525
Fixes #7524
Fixes #7709
Fixes #7523

To Be Done

Notify Experience team that they can start using the errorCodes, as documented in https://github.com/CDCgov/prime-reportstream/blob/6ca83c6e4381c7a3e9772509e3d193c2bd4aee8e/prime-router/docs/proposals/0017-serializer-error-codes.md. The experience team will need tickets to add support for any additional errorCodes they may want to have a custom message for, as well as for localization.

I'm ready to approve otherwise.

Jim,
I looked into the issue you found and here are my findings:

I was catching the wrong exception in fun getErrorCode, should be catching an IllegalArgumentException. That is why the page was "erroring" and not displaying anything. Will fix that.
Now, let's look at root cause of this error because it is interesting. You messed with field SFT-6(0), WHICH DOES NOT APPEAR IN THE SCHEMA, and HAPI chokes on it. Because it does not appear in our schema, we could not find a "Type" for it. So, with my pending changes, the error code will be INVALID_MSG_PARSE_UNKNOWN Instead of the more helpful INVALID_MSG_PARSE_DATE. This means that currently, since the front end is not mapping INVALID_MSG_PARSE_UNKNOWN, the validation page will display the backend message by default (see screenshot below).

@arnejduranovic , Re SFT-6(0):
So it appears that HAPI knows enough about SFT-6 to be, ahem, unhapi with the format of it, but then our code doesn't track a type for that field.  So the solution for now is for our code to throw up its hands and mark it _UNKNOWN.   This seems fine - a quick google will confirm that SFT-6 is a datetime, and then you fix it.
I think the vast majority or maybe even all of the fields likely to cause issues are tracked in our schemas.
So I think this is a 'won't fix'.


Return student statistics after accepting solution.

Coverage remained the same at 67.054% when pulling 29e8f818f0b3d28026bee4d2b522610aee99990f on bugfixes into a649757ea8f62cd1eaee7ffa5b189b7d048e3fc8 on master.


Mitigations for potential SNAT port exhaustion

There's a possibility (to be confirmed) that our flakiness with E2E tests may be caused by SNAT port exhaustion in the Actions runners. Runners are Azure VMs behind a load balancer, so SNAT port exhaustion may be a possibility if we're making a lot of outbound requests.
This commit includes improvements with recommendations that could mitigate that:

Ensure that HTTP clients are re-used and not recreated every time
Ensure that responses' Body streams are always closed properly, so that the client is not idle

Without the ability to control the Actions infrastructure, these application-level changes are our best option, assuming our issues are indeed caused by SNAT.

/ok-to-test

/ok-to-test

/ok-to-perf

/ok-to-test

/ok-to-perf

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-perf

/ok-to-perf

/ok-to-test

Closing as the experiment was successful. Will create a PR with a squash of all commits


[WebSites] SiteLoadBalancing enum members need detailed descriptions

In web app's SiteConfig.loadBalancing enum
Please explain internal working implementation of all Possible values include: 'WeightedRoundRobin', 'LeastRequests', 'LeastResponseTime', 'WeightedTotalTraffic', 'RequestHash'
and how this could change behaviour of Azure internal Routing to Instance.
As user has no way to route request to Instances, this property is very important to understand.

Routing to Service team for detailed explanations.

@nirnaymsft Apologies for the delayed response. I see that this issue is opened long time ago and no further activity had taken place. So wanted to check if you are still looking for assistance on this query? Please let us know .

Can we open again this issue? The explanation is really important


Resetting properties of instanced scene

Godot version:
e619727
OS/device including version:
Ubuntu Linux 17.10 64bit
Issue description:
I have a scene (let's call it A) with a Panel node and a few buttons with signals and whatnot.
That scene is instanced in another scene, (let's call it B), but when I make changes to the base Panel node in scene A, specifically its rect size, the changes not only don't fully reflect in B, but the reset button won't let me get the size of A as set in its scene file. Changing child element sizes in A seems to reflect fine in B, though.
Other Control-inheriting nodes seem to display the same behavior as well.
While this is just a papercut UX kinda sort of bug, and it's not too bad when it's just a few elements changing in size, or elements aligning to a grid, but can be a bit annoying when doing those changes often or during more complete overhauls. So it'd be nice if the reset button could work in those situations to help expediting the process.
Steps to reproduce:

Create a scene with a Panel node and some childs like buttons or something for scene A.
Create a second scene and instance scene A for scene B.
In scene A, alter the size rect of the buttons and the base Panel. Save.
When switching to scene B, the changes in rect for the buttons are present, but the panel size won't change, and pressing the reset button does nothing.

I can't reproduce this in 3.0 stable.

Oh, panel size is overriden if instanced as a child node? Well, my mistake, that could explain it.
Still, the reset button not working is the important issue here.

Oh, panel size is overriden if instanced as a child node?
Only if you put it as child of a Container node, like VBoxContainer or GridContainer ;)

Oh, that's not the case, then

The Rect value in not really a property of the node, it's a value dependant on both the node's anchors and margin values, and the parent's size.
Could you check if the margins and anchors values are kept correctly ?

I never really modify those, I prefer manual placement where available.

Well maybe, but that's how control works. If the margin and anchors values stay the same when inheriting this is an intended behavior. The rect is not guaranteed to stay the same since it depends on the parent's size.

That's a tiny bit annoying, but should be all fine if the reset button is made to simply take the "original" size rect.
Maybe I wrote the issue wrong (I struggle a bit with English at times, it's not my main language, sorry for that), but the reset button for size is the actual thing here. It'd be kinda ideal if it'd reflect automatically, but pressing one button and repositioning is sure faster than manually resizing and repositioning, specially as the number of elements grows.
As for the margin and anchor values, as I said I never used them, unless... are they being set automatically "behind the scenes" when you instance a control as a child of another control? That'd explain it. Note that none of the custom UI I use make use of the scale property, in case that'd affect things.

When you modify the Control's Rect, position or size, it in facts modifies the values of the anchors and margins. Only the anchors and margins values are actually recorded when saving the scene.
The rect/position/size properties are in fact helpers to set the anchors and margins, but they depend on the parent. Thus when you instance the scene, the rect changes since the parent is not the same anymore. Thus the rect changing is not a problem.
That being said, the fact that the button is displayed next to the Rect property is misleading, I agree that it should be removed, but it might need to wait the rework of the inspector before.

Hm, I see... strange I didn't see mention of it anywhere else. Specially since using the pos and size rects via code seems to work as expected.
That said, I'd really prefer if it could get the instance's "default" size since it'd be a nice time-saver.
I'd like to emphasize the "as the number of elements grows" in my previous reply. To have a control be the size you want in the final scene after you change it, you must go to the scene, open the rects sections in the inspector, check the size, go back to the final scene, put the size in, then reposition. With the button working, it's just press reset and reposition.
On a single control it's a few seconds, when it's several similar ones, it's enough time and tedium to get your mind away from whatever you were doing. Please consider that before removing it entirely.

If you need the size and position kept when you instance a Control scene, just set all the anchors to 0. This will remove the impact of the parent's rect on the node's when you instance it.

That's the thing, as I said I never touch them. To test it out, I made a quick menu and instanced it elsewhere, then raised its height by 10 pixels by the grid.

Now the instance is still the same size it was when going back to the other scene. Anchors are still 0.

What's the instanced scene parent's in the tree ?

Ah, I edited my reply but was too slow :grin:
Yeah every single control I have has all anchors set to zero. As said I never touch them.

Can you provide the full screenshot with both the anchor + margin values and the scene tree in both scenes ?

Please don't laugh at the crappiness of this test scene.

The second has a few margin values, although I didn't set them manually.

The change in size doesn't reflect when running the scene either. Menu slides in and out, so its position changes at runtime, still keeps the old size in the test scene instead of the new.
Resetting margin values in the test scene results in this

When you instance the scene, what is the parent node that you child it to?

Oh my bad. But the Panel that is the direct parent of battlecontrols is still having anchors at 0.

Ok I think understood wrongly the problem, the only unexpected behaviour here is the reset's button one. The size is not change when you instance the scene am I right ?

Nope, when freshly instanced it's an 1:1 copy.

Oh, ok thank. The issue is perfectly valid then, sorry I was a little bit confused by what you described.

I apologize, and thanks for the explanation of how controls actually work. Coming from hacking UI elements in libsdl and C by throwing math and pointers at the text editor, concepts such as anchors and margins are a little bit alien to me.

I cannot try 3.2.2b4 at the moment, but in 3.2.1 it requires a weird dance of resetting the sizes, then the margins, then the size again until it kind of works.

Can't reproduce this in 4.3, closing.
Please comment if you can still reproduce and provide a minimal reproduction project.


Wrong flow value name

In last step of advanced kotlin coroutines you can see strange growZoneChannel in the text few times. Actually it must be growZoneFlow. And in code snippets you should write growZoneFlow instead of growZone

The problem is in 14'th step


[dtor] method is gone from the Rust bindings

https://github.com/bytecodealliance/wit-bindgen/commit/a0c7383542183d0d9bb60d370477961f8118224a#diff-665b1964a32aa70c9fcedf3844e4c6ba399636be51bcb4d4d3f62faaa280a0a4L1900 removed the generation of the destructor for guest exported resources without a mention in the ChangeLog.
I wonder whether this way of removing exported resources on the guest was intentionally removed or is a bug.
I only found this by accident when creating a new binary with a newer wit-bindgen resulted in missing exported symbols.

Ah that's a mistake, I meant to add that to the export! macro but I forgot to do so. I'll work on fixing that.

without a mention in the ChangeLog

To clarify, there is no "ChangeLog" file in this repository. I basically do not have time to add that to the things I do as maintenance here.

without a mention in the ChangeLog

To clarify, there is no "ChangeLog" file in this repository. I basically do not have time to add that to the things I do as maintenance here.

Sorry, of course I meant the commit message … which was detailed enough to facilitate this mistake :wink:


Re-implement strict encoding for arrays with const generics once MSRV > 1.50

Hi, i don't see the array implementation of strict encoding

Here they are: https://github.com/LNP-BP/client_side_validation/blob/master/strict_encoding/src/pointers.rs#L36-L94
Basically what we need is to have just a single one in form of impl<Len> StrictEncode/Decode for [u8; Len]

Closed in #45


Add a way to filter for extensions which will actually work on github.dev

So on the extensions tab on github.dev, if I filter through for a linter, it'll show a bunch of linters which won't work on github.dev. Please add another filter so I can see extensions which support github.dev.

/duplicate

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Suboptimal formatting of let-bound chains of || with == inside them.

Input
                    let assign_to_field =
                        context == PlaceContext::MutatingUse(MutatingUseContext::Store)
                        || context == PlaceContext::MutatingUse(MutatingUseContext::Drop)
                        || context == PlaceContext::MutatingUse(MutatingUseContext::AsmOutput);

Output
                    let assign_to_field = context
                        == PlaceContext::MutatingUse(MutatingUseContext::Store)
                        || context == PlaceContext::MutatingUse(MutatingUseContext::Drop)
                        || context == PlaceContext::MutatingUse(MutatingUseContext::AsmOutput);

Expected output
                    let assign_to_field =
                        context == PlaceContext::MutatingUse(MutatingUseContext::Store)
                        || context == PlaceContext::MutatingUse(MutatingUseContext::Drop)
                        || context == PlaceContext::MutatingUse(MutatingUseContext::AsmOutput);

The reformatting actually makes the code look worse (linebreak in the middle of one of the comparisons) without a need (such as a line being too long).
Meta

This is via ./x.py fmt in current rustc

Though I understand how this formatting isn't ideal, I do believe it's correct given then Style Guide prescriptions for this case that govern how rustfmt formats as well as the max width constraints.
Due to the indentation levels it's not possible to fit the lhs and first rhs line on the same line (let assign_to_field = context == PlaceContext::MutatingUse(MutatingUseContext::Store would exceed max width when starting at that position). And even if the entire rhs was kept together on its own line as indicated in your expected output snippet, the subsequent lines would have to be indented again, which would result in both of those lines exceeding the default max width of 100
                            let assign_to_field =
                                context == PlaceContext::MutatingUse(MutatingUseContext::Store)
                                    || context == PlaceContext::MutatingUse(MutatingUseContext::Drop)
                                    || context == PlaceContext::MutatingUse(MutatingUseContext::AsmOutput);

This does seem to me like formatting we could do somewhat better with, and it doesn't seem impossible to handle. In general, we don't do a great job of detecting multi-line things that are similar and should get line-broken to accent that similarity.
I wonder if this could be improved in rustfmt 2.0?

In general, we don't do a great job of detecting multi-line things that are similar and should get line-broken to accent that similarity.

Agreed -- there are also several issues related to match formatting that boil down to this high-level point.

Let me know if I'm missing something obvious, but I still don't see a way in which this could be formatted any differently without either exceeding max_width, or contradicting the rules for let statements and/or the rules for binary expressions.
While it wouldn't necessarily be the way I'd write it instinctively, it seems completely aligned to the rules dictated in the guide.
I fear there's often an underlying theme where folks want rustfmt to ignore the width wrapping rules in specific cases where it might "look better" to ignore the width rule. Although I can understand that perspective in certain circumstances, I worry about the practicalities of something like that which I believe ends up inevitably more subjective than objective.

I wonder if this could be improved in rustfmt 2.0?

I wouldn't count on rustfmt 2.0 ever happening, absent a rustc 2.0 release. The original rfc for rustfmt included text for major/breaking versions, but when we proposed doing such not too long ago it was quite unequivocally rejected by both the dev tools team and a coule folks from the core team as well (though they were speaking for themselves individually)

The original example at the top of this issue is within the width both in the actual and expected formatting. I am not sure which rules are violated by what I called the expected formatting, but in my opinion that just goes to shows the rules still need to be tweaked. It is pretty hard to come up with general rules that always look good, so if we discover unintended consequences of those rules I don't think we should be afraid to improve the rules.

I am not sure which rules are violated by what I called the expected formatting

As per this rule:

If line-breaking, put the operator on a new line and block indent. Put each sub-expression on its own line. E.g.

Your expected output is missing that mandatory block indentation, and would instead have to be this:
context == PlaceContext::MutatingUse(MutatingUseContext::Store)
    || context == PlaceContext::MutatingUse(MutatingUseContext::Drop)
    || context == PlaceContext::MutatingUse(MutatingUseContext::AsmOutput);

With the extra, mandatory, indentation levels resulting in the width excess.

so if we discover unintended consequences of those rules I don't think we should be afraid to improve the rules.

I don't think fear is a factor.
I completely agree with you that it's not feasible to come up with generalized rules that always look good, which is why I don't think that's ever a realistic goal with pretty printer style formatters; it's more of a "good enough in most cases" with an implicit acknowledgement there will be some non-ideal cases.
We still have the stability/idempotence problem where we're simultaneously being asked for breaking formatting changes but  prohibited from doing so intentionally. However, if someone's able to articulate a new rule/rule adjustment that's both deterministic and consistent with all other relevant rules then that's certainly a path that can be explored (though that's more of a style guide topic vs. rustfmt topic). In the meantime though, I think it's important to point out when an ask contradicts the existing rules that dictate rustfmt's behavior.


Bridging header with swift

Is it possible to do bridging header with this plugin? Thanks

I thought Xcode would automatically provide one for frameworks?

What do you mean provide one for frameworks?

What exactly are you trying to do?

Because my project using swift. I need to do bridging header.

You should be able to import octokit in your project's bridging header.


Alt attributes added

Related Issue
Closes #552

Description

Added meaningful alt attributes to all the img tags of the project
Checklist

[x] My code adheres to the established style guidelines of the project.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

@Avinash905 please resolve the conflicts


[Update] May 9th of 2022 Applied Changes and New Contents

There's a new content in Master Duel like New Solo Modes, New Packs, New Cards, New Structure Deck, New Banlist and more that is not being applied in the YgoMaster
We're not rushing you here 😅😅, We can wait anytime for the update. 🥰😍😘

Maybe just added those new card id to ~\Data\CardList.json

v1.7 adds these https://github.com/pixeltris/YgoMaster/releases/tag/v1.7


Tuple vs arrays casting mismatch

TypeScript Version:  2.2.1 / nightly (2.2.0-dev.201xxxxx)
Code
http://www.typescriptlang.org/play/#src=export interface Vector {
    x%3Anumber%3B
    y%3Anumber%3B
}

type Point %3D [number%2C number]%3B

const Vectors%3A Vector[] %3D []%3B
const points%3AArray<Point> %3D Vectors.map((Vector) %3D> {
        return [Vector.x%2C Vector.y]%3B
})%3B%0A%0A%2F%2F%20This%20is%20working%0Aconst%20pointsOk%3AArray%3CPoint%3E%20%3D%20Vectors.map((Vector)%20%3D%3E%20%7B%0A%20%20%20%20%20%20%20%20return%20%3CPoint%3E%5BVector.x%2C%20Vector.y%5D%3B%0A%7D)%3B%0A%0A
export interface Vector {
    x:number;
    y:number;
}

type Point = [number, number];

const Vectors: Vector[] = [];
// Error in the affectation
const points:Array<Point> = Vectors.map((Vector) => {
        return [Vector.x, Vector.y];
});

// This is working
const pointsOk:Array<Point> = Vectors.map((Vector) => {
        return <Point>[Vector.x, Vector.y];
});

Expected behavior:
The compiler should understand I am returning a tuple and not an array of values
Actual behavior:
I get an error and need to cast the conversion to get the right behaviour

I have just encountered this same issue myself.
I think the compiler should infer [1, 2] as [number, number], and allow [number, number] to be assigned to number[]. The same should hold for [1, 'a'] being inferred as [number, string] which should be allowed to be assigned to (number | string)[].

See https://github.com/Microsoft/TypeScript/issues/11924#issuecomment-257029046  / #10195


makes interstitials requests hit the same router

Changes proposed in this PR

makes interstitials requests in the integration test hit the same router

Why are we making these changes?
Avoids flakiness in the test

Why does sending requests to the same router make interstitials less flaky?

https://github.com/twosigma/waiter/pull/280/files#diff-777b28cad6b2c2c5bca3dbc6beec28a7R759 relies on the interstital promise being already initialized on the router from a previous request (from one of the other threads). If the request in this thread ends up being the first request to hit that router, then it will be redirected to the interstitial and the test will fail.


Keys Not Assigning to the Correct Level in Level Order Randomizer

At some point between 0.1.2 and 0.2, level keys no longer get assigned to the correct level for Level Order Randomizer.
What is currently happening:
Level 1 = Random Key
Level 2 = Random Key
etc
What should happen:
Level 1 = Key 1
Level 2 = Key 2
etc
Python logic in the randomize function look exactly the same for this part when comparing 0.1.2 and 0.3. This will need more investigation.

Fixed with #31


Update Get-ADComputer.md

corrected formatting and added example 6 and 7

Docs Build status updates of commit ab573c2:
:clock10: Pending: waiting for processors (16 builds ahead of you)
:warning: Docs Build is busy, currently there are 16 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit ab573c2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/addsadministration/Get-ADComputer.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Issue#https://github.com/MicrosoftDocs/windows-powershell-docs/issues/1617

@JohanFreelancer9 Minor style checks needed for this PR. Thanks.

@dariomws306 @get-itips @tseward
Peer review needed for this PR. Thanks.

One other edit, can you please change "User01" and "User02" in the cmdlet example to Computer01 and Computer02? This is to prevent confusion on what we're querying. Approved after that.

One other edit, can you please change "User01" and "User02" in the cmdlet example to Computer01 and Computer02? This is to prevent confusion on what we're querying. Approved after that.

Thank you @tseward  I changed the value for "User01" and "User02" in the cmdlet example to Computer01 and Computer02.

Docs Build status updates of commit ab856f0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/addsadministration/Get-ADComputer.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit c29e392:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/addsadministration/Get-ADComputer.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

I think you should also change user01 for computer01 on example 7. Also the output should be computer01 and not computer02, I think.

cc @msbemba

I think you should also change user01 for computer01 on example 7. Also the output should be computer01 and not computer02, I think.

cc @msbemba

I have made these changes now.
Thanks
Sri

Docs Build status updates of commit 9a52b48:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/addsadministration/Get-ADComputer.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Dump: ITAB_DUPLICATE_KEY

Similar to #1457

Scenario is create a new subpackage, move a clas from main package to sub package, commit the changes and then switching branches.
Steps to reproduce with demo repo:

clone master of https://gitlab.com/christianguenter/test_move_package.git
switch to branch new_package

Maybe also related to #1275

problem solved. Thanks @larshp


feature/helm-chart

I added Helm chart to help quickly starting with msi-acrpull on Kubernetes with the following changelog
chore - add helm index
docs: removed helm 2 step
chore - Added helm chart for msi-acrpull

@samkreter @yangl900 Could you help to review on this?

Any updates on this review? @weng271190436

I actually don't have permission to merge this anymore. @paulgmiller wondering who should merge this now

I think @samkreter @gambtho  and isreal are the only admins. Seeing if I can join aks-overlay group.

Any updates on this? @paulgmiller

Superseded by #80


Reuse existing tab after :make error

My Vim/Makefile development workflow depends heavily on Vim's response to a complier error when I invoke :make.
MacVim's response is to open a new tab on the offendimg source file, even if there is already a tab open on that file.  Thus, every tim I run :make and get an error I must close atleast one tab, which seems unnecessary.
What would be a real help is for MacVim to find an existing tab to switch to before deciding to open a new tab.
Even better would be for someone to point me to a correct usage pattern to achieve the same goal.  For now I'll just shelve MacVim in favor of plain old Vim.

You mean you are calling :make within MacVim? The behavior of where the new file is opened is controlled entirely within Vim via switchbuf, so there should be no difference in behavior. Can you describe in more details how you are configuring it exactly, and what your settings for switchbuf are? You at least share the same .vimrc between MacVim and Vim right?

Srttin switchbuf to usetab had exactly the desired effect.  Thank you!


[Feat] Gpu data filtering - 1

Step. 1 GPU Data filtering feature (https://github.com/uber/kepler.gl/issues/328)

Add gpu property to filter.
Implement logic to manage which filter is gpu enabled
Remove filtered data array from filter result (not used anywhere)
Implement set function to immutably edit reducer state

renamed branch to gpu-data-filter-1
https://github.com/uber/kepler.gl/pull/451


"Insert Media" seems to insert images with relative path that breaks when rendered

It seems like the "insert media" action in the insert menu, though appearing to work nicely in the WYSIWYG editor, inserts an image with a  relative src that's obviously doesn't point to anything when the page renders.  What's more, it seems like when I try to fix the problem by manually adding a leading slash (in the source code or in the edit image dialog) it strips them and I'm back to the same issue.

newgeekorder's comment is an enhancement.  scogle's comment is a bug I'm also experiencing.  I use "Insert media", pick a photo I've uploaded using the Mediamanager, and it inserts:

It looks to be TinyMCE's doing.  Cockpit's code is correctly changing site: to the base path but then the editor is making it a relative URL.
Solution: add "relative_urls: false" to TinyMCE's init options.
http://www.tinymce.com/wiki.php/Configuration:relative_urls