s3Client->DeleteObject() outcome is the same (key present, key not present)

When s3Client->DeleteObject() used on a bucket+key that exists -or- not exists, the outcome is the same.
DeleteObjectRequest objectsRequest;
objectsRequest.WithBucket(bucket).WithKey(key);
auto outcome = s3Client->DeleteObject(objectsRequest);
if (outcome.IsSuccess()) {...

Repro 1
Bucket exists and key exists. Credentials are valid. No versioning.
Result via Outcome
result	{m_deleteMarker=false m_versionId="" m_requestCharged=NOT_SET (0) }	Aws::S3::Model::DeleteObjectResult
m_deleteMarker	false	bool
m_versionId	""	std::basic_string<char,std::char_traits<char>,std::allocator<char> >
m_requestCharged	NOT_SET (0)	Aws::S3::Model::RequestCharged
error	{m_errorType=INCOMPLETE_SIGNATURE (0) m_exceptionName="" m_message="" ...}	Aws::Client::AWSError<enum Aws::S3::S3Errors>
success	true	bool

Expected
This is expected result.
Repro 2
Bucket exists and key does not exists. Credentials are valid. No versioning.
Result via Outcome
result	{m_deleteMarker=false m_versionId="" m_requestCharged=NOT_SET (0) }	Aws::S3::Model::DeleteObjectResult
m_deleteMarker	false	bool
m_versionId	""	std::basic_string<char,std::char_traits<char>,std::allocator<char> >
m_requestCharged	NOT_SET (0)	Aws::S3::Model::RequestCharged
error	{m_errorType=INCOMPLETE_SIGNATURE (0) m_exceptionName="" m_message="" ...}	Aws::Client::AWSError<enum Aws::S3::S3Errors>
success	true	bool

Expected
Expected success = false and error to contain fields consistent with a 404 not found.
Notes

http://docs.aws.amazon.com/AmazonS3/latest/API/RESTObjectDELETE.html has no mention about deletes of keys that do not exist. Therefore, I would assume common behavior that operations on a resource that is not found results in a 404 not found error.
http://docs.aws.amazon.com/AmazonS3/latest/API/multiobjectdeleteapi.html has an explicit rule that "if the object specified in the request is not found, Amazon S3 returns the result as deleted". Personally, I think this is very odd behavior, but at least the rules are clear.
http://docs.aws.amazon.com/AmazonS3/latest/API/RESTObjectGET.html is in contrast. It will return 403 or 404 when a key does not exist.
I suspect this is an upstream design/issue. If so, is there doc/direction/logic on why this is intended?
It is likely possible to do a head or stat on the key-not-yet-deleted to see if it exists. This potentially doubles the network traffic -- undesired.

I suspect this is an upstream design/issue.

Yes. This is how S3 is designed to work. You'd get the same response if you hit the endpoint using curl from the cli.

is there doc/direction/logic on why this is intended?

If DELETE would fail with a 404 for non-existing keys, then it gets tricky on what the outcome should be for a batch delete. Imagine if 1 out of 10 keys does not exist. Do you fail the whole operation?
It's simpler to consider it a success, since the observed state of the bucket is the same after that operation regardless of whether that key existed or not.
Hope that helps.

Got it. Thank you.


Fix tuple definition

Motivation
Per this comment https://github.com/Microsoft/TypeScript/issues/28894#issuecomment-445370682 , Tuple type should be fixed width in Typescript 2.7. Which means the current document for Tuple type is outdated, My update is to fix this.
What I really did in this pull request

Modified the code samples for the Tuple type, add more comments, trying to give a more detailed declaration for Tuple Type. e.g. The first time I read the doc, I'm quite confused how Tuple type actually works. I was thinking it's actually used for declaring some key-value pair, but actually, it's for declaring an array which each of the element has a specific type.
As mentioned above, the Tuple definition is outdated, now access the index outside of the defined one in a Tuple is not allowed anymore, and this will resolve #975 and another issue in Typescript repo https://github.com/Microsoft/TypeScript/issues/28894#issue-388536852

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

There's a typo here https://github.com/microsoft/TypeScript-Handbook/pull/998/files#diff-b482036f3eac612448112805b2d12eaeR84

Thanks! I've merged a PR which covers this https://github.com/microsoft/TypeScript-Handbook/pull/1030 - so this can be closed <3


String parameter signed constants

Codecov Report

Merging #1 (89e9949) into main (0ae0033) will increase coverage by 0.50%.
The diff coverage is 59.09%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main       #1      +/-   ##
==========================================
+ Coverage   83.68%   84.19%   +0.50%
==========================================
  Files           8        8
  Lines         282      291       +9
==========================================
+ Hits          236      245       +9
  Misses         46       46

Flag
Coverage Δ

unittests
84.19% <59.09%> (+0.50%)
:arrow_up:

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

Impacted Files
Coverage Δ

src/VerilogConstructor.cpp
85.96% <ø> (+7.99%)
:arrow_up:

src/VerilogTypes.cpp
84.61% <ø> (ø)

src/VerilogParser.yy
85.26% <35.71%> (-7.60%)
:arrow_down:

src/VerilogScanner.ll
73.33% <100.00%> (+0.45%)
:arrow_up:

src/VerilogTypes.h
88.88% <100.00%> (-0.21%)
:arrow_down:

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add js sourcemaps to build process

fixes #16583
The cssmin task supports this as well but I figured that would be better in its own PR

Punting per my comment in #16583.


How to get paired devices in IOS through this library

I successfully pair the Bluetooth device through the app on the iPhone, then close the app, and open the second time I want to connect to the paired Bluetooth device. Is there any way to do it?

I am very eager for everyone to reply to me, thank you

@JonYun when you try connect a paired bluetooth in iOS return error too?


Add option to sort by starred experiments

2/4 main <- #2164 <- this <- #2170 <-#2171
This PR gives users the ability to sort experiments by their starred status.
Demo
https://user-images.githubusercontent.com/37993418/183797952-6c6b0dfa-08f4-410e-bfa0-9741126b8ab4.mov

Related to #1720

I am not able to reproduce this locally with this branch. Are there files missing?

Nvm, it's the checkpoints I cannot sort. I can sort the experiments.


Widget

Instead of having to open the app to view the upcoming departures, it would be nice to have a widget on the home screen.
Question
Should this only apply to a select StopAreas, StopPoints or Lines?

Show the 3 nearest StopAreas.

Android and iOS


toga-iOS: Fixed imageview to work with iOS. Leave frame to pack for now,

Signed-off-by: Moises Aranas<EMAIL_ADDRESS>This updates the iOS image view to behave more like the Cocoa version.
I made the default scaling behavior to aspect-fit, as that is typical for most
phone implementations, but I need some feedback on this.
I'm leaving alignment and size hinting to the container, so it uses the available
space unless the style's height/width values are set
This also updates Image so the image downloading works.
PR Checklist:

[x] All new features have been tested
[x] All new features have been documented
[x] I have read the CONTRIBUTING.md file
[x] I will abide by the code of conduct

related: #78

Thanks for checking! BTW should we create a separate toga-demo for iOS?

@maranas Ideally, we'd only have a single toga-demo. Toga-demo as currently defined obviously won't work with iOS, so the better approach would be to restructuring toga-demo so that it is a good example of what can/should be done with Toga's "capture the underlying motivation of the UX" philosophy.
Ideally, that would then be an app that we could push to the App Store.

@freakboy3742 got it. For this pull request i'm making a specific example for the widget, I don't think i should modify the toga-demo yet since most other targets don't support the imageview widget yet.

UIImages are usually initialized from a name in iOS because when images are built with the app, the images are copied to the root of the bundle - otherwise, you have to provide a full path to it. So it is either a name or a full path. But you have a good point, I shouldn't expose the app_dir in this case. I added this because our current briefcase template bundles everything into the app bundle, respecting the directory tree - so to get the full path to the image, I need to get the full path to the image by appending it to the app_dir path.
I have an idea though - since we will be doing this with iOS classes for other possible widgets (e.g. if we plan to load icons or fonts), we can just add a helper method to resolve relative paths (e.g. the 'resource' directory) to their full paths, hidden from the "public interface" but accessible to the widgets. What do you think?

@freakboy3742 I updated it based on the latest comments - now the paths should be treated the same way we do other platforms (so examples/imageview should also work e.g. in gtk). The relative path resolution is now done in the private interface, similar to how we do it in the Icon widget


Add audioroom plugin + list endpoint

fyi @kris-lab

lgtm. let's wait for build.

Works now :o Different commit helped?


[T2A2][T09-A4]Cao Wei

Ready for review

@LoneGunman001 while you have made a number of improvements to the code, notice that there is just one commit for the entire exercise. please ensure you commit after every change for better revision control.
on an unrelated note, please add a photo to your profile to help us identify you better :-)


Readd test_docker to .circleci/config.yml

Please select one of the following

[ ] I am removing an existing difference between facebook/react-native and microsoft/react-native-macos :thumbsup:
[ ] I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos :thumbsup:
[x] I am making a fix / change for the macOS implementation of react-native
[ ] I am making a change required for Microsoft usage of react-native

Summary
We're seeing some dependabot PR failures (#1167 and #1169) due to a missing job definition, so let's reintroduce that definition to make everyone happy.

I might be wrong, but I thought someone from @microsoft/rnandroid recently removed this because Android builds were failing from this?

I might be wrong, but I thought someone from @microsoft/rnandroid recently removed this because Android builds were failing from this?
Discussed offline, and we decided that this change should be fine.


[Logistic Regression] Support multiple classes

Support multiple classes in fit, transform, and benchmarking.

build

build

build

build


[COM-388] Implementation of Popover component

Please delete checklist items and sections that are not relevant
Checklist

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Please check if the PR fulfills these requirements

[x] Tests for the changes have been added (for bug fixes / features)
[x] Storybook or docs have been added / updated (for bug fixes / features)

What is the ticket / issue associated with this change? (Link to the ticket or issue)
https://impulsumvc.atlassian.net/browse/COM-388
https://5eed577398d08d0022b52d76-acxsfjuplb.chromatic.com/?path=/story/components-primitives-popover--all

The close button doesn't have the correct design.
I think that for the correct implementation of this component you will need our custom close button since the icon is incorrect, but that PR is still under review , in the mean time I think you can pass the Icon as a children, since PopoverCloseButton returns a CloseButton and this in turn renders children if it was passed

Fixed: I've used the Composer CloseButton that should contain the correct icon SVG


Support for reading bare HFS image (e.g. no header/tags) into mac400/mac800 format

I'd like to add support for reading non-diskcopy images into mac400 and mac800 (usually ending in .dsk). For these, we can treat the tags as all zeros (that's what FloppyEmu does), and just used a fixed sector mapping.
However, it looks like the existing configuration options for the imgimagereader aren't flexible enough to describe the track layout needed. I was thinking of adding an extra "mac" flag to the imgimagereader, that when set, generates the appropriate track layout. Alternatively, I could expand the proto files to allow a whole table of track/sector assignments (though this feels overkill for the single image format that requires it).

What's the HFS format? One thing on my to-do list was to extend imgimagereader/writer to allow for different block orders (e.g. Acorn formats use all of side 0 and then all of side 1).

In terms of http://cowlark.com/fluxengine/doc/disk-macintosh.html it's just a bare image with 512 byte sectors, without holes (and with the 12 byte tag implied as 0 bytes). The block order is the same as in DiskCopy files, I believe.
(HFS is the filesystem - an image in this format can be loop mounted by Linux, for example)

Ah, I'd forgotten those pesky variable-sector disks...
So this can already be done, but it's pretty clunky. You need to specify the number of sectors in each track in the image_reader/image_writer sectiosn, using something like this:
trackdata {
  sector_range {
    start_sector: 0
  }
}
trackdata {
  track: 0
  up_to_track: 9
  sector_range {
    sector_count: 10
  }
}
trackdata {
  track: 10
  up_to_track: 19
  sector_range {
    sector_count: 15
  }
}

(although obviously with the right numbers substituted in). It assumes CHS order which should be correct for HFS logical block order.
Two disclaimers: I currently have a cold and my brain's undervolted, and this has also never been used by any formats and so could be buggy. The way the track data works is that all clauses which match the track and side predicates are merged, in order, which subsequent values overriding the prior.

mac dsk support is now merged, i tested it not with floppyemu but with a powerbook 180 and some 400k and 800k diskettes sucessfully

Oops, forgot to tag this bug. That was #449.

all good, i was sweeping thru any mac-related issues in case there was something else to test


Parity: NSCoding: NSOrderedSet

NSOrderedSet uses a subset of the way NSSet encodes things; encode using the existing methods.

We’re using convenience initializers to mimic “/* not inherited */“ factory initializers from ObjC. Just like those initializers, they aren’t actually inherited. We need to replicate all the convenience initializers of NSOrderedSet in NSMutableOrderedSet for source compatibility, and we do so here.

@swift-ci please test

Fixes https://bugs.swift.org/browse/SR-10404

@swift-ci please test and merge

@swift-ci please test and merge


Testing the correctness of scope_identity() and @@IDENTITY

Description
Engine changes - https://github.com/babelfish-for-postgresql/postgresql_modified_for_babelfish/pull/354
As part of engine commit https://github.com/babelfish-for-postgresql/postgresql_modified_for_babelfish/commit/ec9fda039c972f0ef7884d2eb176c988266cff64, we made some parsing level changes for quals which involves scope_identity() or @@IDENTITY in order to let optimiser choose index scan in certain situation.
With extension commit 80905a001c2e13a07f8c33e865401a3abce68c1d merged, we found that above engine commit has introduced correctness issue. So mentioned engine PR fixes that issue by reverting above mentioned changes. Please also note that this will not cause any performance regression as optimiser will now choose index scan with the extension commit 80905a001c2e13a07f8c33e865401a3abce68c1d.
This commit adds test cases to evaluate the correctness of scope_identity() and @@IDENTITY
Task: BABEL-4876
Signed-off-by: Dipesh Dhameliya<EMAIL_ADDRESS>Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 72.632%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
41578

Relevant Lines:
57245

💛  - Coveralls


Set styling for ClientGeoJsonSource polyline

Hello, in this issue #1655 I was advised to use ClientGeoJsonSource instead of Marker interface for showing polylines correctly. Unfortunately my app is already in production and many users have problems with displaying polylines when they are long. So I have decided to use ClientGeoJsonSource and now I need to set styling for these polylines. I can see that functionality for adding ClientGeoJsonSource is exposed by MapzenMap.addPolyline(). As I have digged deeper I have discovered that OverlayManager.addPolylineToPolylineMapData() has option to set properties, but I don't know how to use it correctly. I have tried hardcoding this for example but color for line does not changed:
private MapData addPolylineToPolylineMapData(List<LngLat> coordinates) {
  HashMap<String, String> properties = new HashMap<String, String>();
  properties.put("color",  "[1.0, 1.0, 0.0, 1.0]");
  MapData mapData = polylineMapData.addPolyline(coordinates, properties);
  mapController.requestRender();
  return mapData;
}

Could I ask you to provide me some information how to correctly style ClientGeoJsonSource polyline through code? Thank you a lot.

Hi @rwrx
properties for a MapData append to the properties of the tiled feature and has no direct affect on the rendering style.
To style a clientDataSource one would have to define apt draw group in the scene file. As an example refer this block in our bubble-wrap style.
    mz_route_line_transit:
        data: { source: mz_route_line_transit }
        draw:
            ux-transit-line-overlay:
                # each transit route segment could be a different "line" each with it's own color
                # but some transit lines don't define a color, in those cases default to blue
                # and since the color is coming from Transit.land they call it "color" instead of "colour"
                interactive: global.sdk_interactive
                color: function() { return feature.color || '#06a6d4'; }
                order: 500
                width: [[0,3.5px],[5,5px],[9,7px],[10,6px],[11,6px],[13,8px],[14,9px],[15,10px],[16,11px],[17,12px],[18,10px]]

Here all features added do the mz_route_line_transit client data source (from mapzen android or ios sdk), will be drawn with the style parameters in the draw group.
Similar to this you will have to define a styling layer which will use the clientdatasource name you are adding in the data yaml block.
With respect to using client data source properties, you can refer this block.
    mz_search_result:
        data: { source: mz_search_result }
        draw:
            ux-icons-overlay:
                interactive: global.sdk_interactive
                sprite: ux-search-active
                size: [36px,54px]
                collide: false
                anchor: top
                transition: { show: { time: 0s }, hide: { time: 0s } }
        inactive:
            filter: { state: inactive }
            draw:
                ux-icons-overlay:
                    sprite: ux-search-inactive

Here the state property is set in mapzen sdk which can then be used to filter features for different styling.
Closing this issue, please reopen if you still have more questions on this.
Thanks

Hi @tallytalwar. Thank you a lot for your answer. However what I need in final is to dynamically style polylines from code. Also I need for each polyline to have different color, width and order styling. Can this be somehow done through ClientDataSource? Thank you a lot.

Hi @tallytalwar, thank you a lot.

Hi @rwrx, you can use the feature properties to pass styling values. For your example this should work with:
  draw:
    lines:
      color: function() { return feature.color; }

Hi @hjanetzek, thank you a lot for your suggestion. I have already tried it and I have removed my previous comment.


[Bug]: bsdtar toolchain fails on macOS if host tar is gnutar

What happened?
bazel-lib doesnt provide any non-host bsdtar toolchains on macOS. This causes a hard-fail when the host tar binary is a gnutar implementation instead of bsdtar, as is the case when e.g. running inside a Nix shell.
It may be expected that in this case, one should provide their own bsdtar toolchain, in which case this is working-as-intended. It would definitely be more convenient though for bazel-lib to also provide that : )
Version
Development (host) and target OS/architectures: macOS arm64
Output of bazel --version: aspect 5.8.19
Version of the Aspect rules, or other relevant rules from your
WORKSPACE or MODULE.bazel file: 2.5.3
Language(s) and/or frameworks involved: Nix
How to reproduce
No response
Any other information?
No response

Also seeing this

We'll use https://github.com/dzbarsky/bsdtar/releases/tag/v3.7.2 which includes darwin binaries to fix this.


In wasm-bindgen 0.2.84 version, webpack cannot generate wasm files

Steps to Reproduce
I created a repo with the problem here
Step1:
npm run i
Step2:
wasm-pack build
Step3:
npm run build
A dist dir contain index.js and wasm file will be generated in the project root; It is expected behavior.
When i change the wasm-bindgen version to 0.2.84 and run
rm -r pkg && wasm-pack build && npm run build
In dist dir, webpack doesn't generate wasm files.

I've noticed this as well, additionally: setting the version of wasm-bindgen back to 83 isn't respected for some reason, and cargo continues building with 84 regardless.

I can reproduce this issue on my project too. Trying to use the WASM module in a development server throws a wasm is undefined exception.

This seems to be a bug in Webpack, triggered by #3152. As a workaround, switching to dynamically importing the wasm-bindgen shim fixes it for some reason.

It turns out this is happening because wasm-pack sets "sideEffects": false for wasm-bindgen generated code, which is a Webpack-specific flag that tells Webpack it can remove any code whose only impact is a side effect. In #3152, I unwittingly made initialisation rely on side effects, causing this problem.
(So, it's not a Webpack bug, I was a bit quick to assume that.)
I've opened rustwasm/wasm-pack#1224 to fix this. @alexcrichton or @drager, could you take a look at that PR?

Ah sorry, I don't manage the wasm-pack repository myself.

It turns out this is happening because wasm-pack sets "sideEffects": false for wasm-bindgen generated code, which is a Webpack-specific flag that tells Webpack it can remove any code whose only impact is a side effect. In #3152, I unwittingly made initialisation rely on side effects, causing this problem.
(So, it's not a Webpack bug, I was a bit quick to assume that.)
I've opened rustwasm/wasm-pack#1224 to fix this. @alexcrichton or @drager, could you take a look at that PR?

Yeah, I will take a look soon. Thanks for looking into and fixing it!

I tried setting everything back to 0.2.83, by running these steps in this order (after some trial and error):
cargo update -p wasm-bindgen-test --precise 0.3.33
cargo update -p wasm-bindgen-futures --precise 0.4.33
cargo update -p web-sys --precise 0.3.60
cargo update -p js-sys --precise 0.3.60
cargo update -p wasm-bindgen --precise 0.2.83

And then built it with rustc 1.67.1, making sure that the old versions were indeed the ones that were compiled...
But it still didn't solve the issue I ran into in #3306.

I tried setting everything back to 0.2.83, by running these steps in this order (after some trial and error):
cargo update -p wasm-bindgen-test --precise 0.3.33
cargo update -p wasm-bindgen-futures --precise 0.4.33
cargo update -p web-sys --precise 0.3.60
cargo update -p js-sys --precise 0.3.60
cargo update -p wasm-bindgen --precise 0.2.83

And then built it with rustc 1.67.1, making sure that the old versions were indeed the ones that were compiled...
But it still didn't solve the issue I ran into in #3306.

I can't reproduce this. Could you give more detailed instructions?

I tried setting everything back to 0.2.83, by running these steps in this order (after some trial and error):
cargo update -p wasm-bindgen-test --precise 0.3.33
cargo update -p wasm-bindgen-futures --precise 0.4.33
cargo update -p web-sys --precise 0.3.60
cargo update -p js-sys --precise 0.3.60
cargo update -p wasm-bindgen --precise 0.2.83

And then built it with rustc 1.67.1, making sure that the old versions were indeed the ones that were compiled...
But it still didn't solve the issue I ran into in #3306.

I can't reproduce this. Could you give more detailed instructions?

I've updated and uploaded my MRE:
https://github.com/cryptoquick/wasm-bindgen-mre

I've updated and uploaded my MRE:
https://github.com/cryptoquick/wasm-bindgen-mre
I've made a few more changes, up to here: https://github.com/cryptoquick/wasm-bindgen-mre/commit/ee313c078b464d0d255e0158385e8f15c7fe879f
Still not working, same error as in README, not sure what else I can do...
I'm sure the cache is clearing too, since each time, I test with a different port on my local server.

That repo isn't demonstrating the original issue that you had in #3306, it's a completely separate issue which is occurring because you're not running await init() before using the wasm exports on --target web, as people already pointed out in #3306.
The original problem was with --target bundler, not --target web, and that's the issue that I believe is the same as this one and wasn't present in 0.2.83.

I have this exact issue, as soon as I upgrade to web_sys 0.3.61 from 0.3.60 any call to a Rust function from JavaScript results in an undefined error like:
bootstrap.js:2 Error importing `index.js`: TypeError: Cannot read properties of undefined (reading 'on_assembly_change')
    at Module.on_assembly_change (spcasm_web_bg.js:269:10)
    at eval (index.js:36:46)
    at __webpack_require__.a (bootstrap.js:357:13)
    at eval (index.js:1:21)
    at ./index.js (index_js.bootstrap.js:28:1)
    at __webpack_require__ (bootstrap.js:284:33)
    at Function.fn (bootstrap.js:592:21)

(on_assembly_change is the Rust function being called, which sort of has bindings generated, as you can see)

I have this exact issue, as soon as I upgrade to web_sys 0.3.61 from 0.3.60 any call to a Rust function from JavaScript results in an undefined error like:
bootstrap.js:2 Error importing `index.js`: TypeError: Cannot read properties of undefined (reading 'on_assembly_change')
    at Module.on_assembly_change (spcasm_web_bg.js:269:10)
    at eval (index.js:36:46)
    at __webpack_require__.a (bootstrap.js:357:13)
    at eval (index.js:1:21)
    at ./index.js (index_js.bootstrap.js:28:1)
    at __webpack_require__ (bootstrap.js:284:33)
    at Function.fn (bootstrap.js:592:21)

(on_assembly_change is the Rust function being called, which sort of has bindings generated, as you can see)

I get issues with my own functions, or some stack pointer calls, such as Uncaught (in promise) TypeError: Cannot read properties of undefined (reading '__wbindgen_add_to_stack_pointer')

That repo isn't demonstrating the original issue that you had in #3306, it's a completely separate issue which is occurring because you're not running await init() before using the wasm exports on --target web, as people already pointed out in #3306.
The original problem was with --target bundler, not --target web, and that's the issue that I believe is the same as this one and wasn't present in 0.2.83.

I actually updated the comment you responded to only a few minutes after I posted it. Please take another look.

This is now fixed in wasm-pack v0.11.0.


Add a cancel button when using orb of origin

When players right-click orb of origin, they can no longer cancel it. And then it will be consumed.
They probably just wanted to check out the effects of other races, but because they couldn't cancel it, they wasted an orb forever... That's sad...

Not on my end, sorry.


How to control whether to generate inline list/map or not?

My data classes:
data class MetadataDetailDto(
    val authors: List<String>,
    val collections: Map<String, Map<String, ChapterMetadataDto>>,
)
data class ChapterMetadataDto(
    val name: String,
    val title: String,
)

I would like to generate files like this:
authors = [ "author1",  "author2"]

[collections.collection1]
chapter1 = {name = "name", title = "title"}

But it generates this:
[[authors]]
 = "author1"

[[authors]]
 = "author2"

[collections]
collection1.chapter1.name = "name"
collection1.chapter1.title = "title"

So is there a way to control whether to generate inline list/map or not?

Currently this is not possible. I'm thinking a good API for this might be something like a generateInline<T>(Boolean) function in the config block to control this type by type, as well as a similar function to set the default generation style. Would this be a good solution?

There is a small problem with this solution: types like Map/List may sometimes need to be inlined and sometimes don't. But I think it's fine to let users handle these themselves.
Also,  I have some suggestions:

Use inline style by default for better readability.
Provide an interface to control whether to keep inline Map/List oneline.

By the way, thanks for your library. Without it, I would have to introduce a huge dependency just for TOML.

Separated the invalid TOML generation part into its own issue (#12), so this issue can focus on the configurability part.

c2678aa0a64d1bff732224a562b717e514062ac8 introduces an interface for configuring serialization. It doesn't yet allow per-type configuration yet, but otherwise it covers most aspects of table/inline/multiline syntax. What do you think?
It does not change the default to prefer inline syntax, mostly to avoid changing defaults people might have come to expect.


you need to update your crates package

The fix for this is not released!
https://github.com/polyverse/pagerduty-rs/issues/2

Looking now...

Done: https://crates.io/crates/pagerduty-rs


All in one Laravel 5.7?

Summary of problem or feature request
I try to install the package all in one and I do not allow it because the Laravel version is not compatible. How can I solve that? Thank you.
Code snippet of problem
USER@USER-PC c:\xampp72\htdocs\starcommerce-erp composer require yajra/laravel-datatables:^1.0
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for yajra/laravel-datatables 1.0 -> satisfiable by yajra/laravel-datatables[v1.0.0].
    - Conclusion: remove laravel/framework v5.7.9
    - Conclusion: don't install laravel/framework v5.7.9

System details

Windows 10 with Xampp
PHP 7.2.9
Laravel 5.7.9
yajra/laravel-datatables:^1.0

I tried using in composer.json:
"yajra/laravel-datatables-oracle": "^8.0",
"yajra/laravel-datatables-buttons": "^3.0",
"yajra/laravel-datatables-html": "^3.0"
And I get same errors.

Excellent! Thank you so much, with your suggest this issue was resolved!


Support Min/Max Threshold for Custom SQL Tests

Is your feature request related to a problem? Please describe.
Currently, custom SQL quality tests only supports row count <= threshold. It will be great if we can set max/min value instead of a single threshold.
Describe the solution you'd like
Change Threshold to Max Value
Add another filed for Min Value
Describe alternatives you've considered
Without this feature, if I hope to set a test which should return at least 10 rows, I can not set the threshold to 10. Insted, I have to write a case when expression to return 0 if it meets the expectation and 1 if it doesn't.
Additional context
N/A

this is something that should be achievable today just by tuning the returning rows of the custom SQL query. Adding more options that might be too complex for the majority of users


Use same styling as BelongsTo link + run npm

@rick-hidef done ;)

@Youniteus All merged up. Thanks for contributing!


Load 2W+label with primitives and it gets stuck when loading, Loading completed in 18-30 seconds

The text will load quickly if it is the same, and this phenomenon will occur when the loaded text changes every time
  this.labelCollection = viewer.scene.primitives.add(
      new Cesium.LabelCollection({
        show: true,
      }),
    );
    for (let i = 0; i < 23000; i++) {
      let feature = features[i];
      let longitude = feature.geometry.coordinates[0];
      let latitude = feature.geometry.coordinates[1];
      let text = feature.properties.MC;
      this.labelCollection?.add({
        position: Cesium.Cartesian3.fromDegrees(longitude, latitude),
        text: text,
        font: '16px Helvetica',
        HeightReference: Cesium.HeightReference.RELATIVE_TO_GROUND,
        disableDepthTestDistance: Number.POSITIVE_INFINITY,
        horizontalOrigin: Cesium.HorizontalOrigin.CENTER,
        verticalOrigin: Cesium.VerticalOrigin.CENTER,
        DistanceDisplayCondition: new Cesium.DistanceDisplayCondition(0.0, 900.0)
      });
    }
  }

Hi @juzhiqiang. I'm not sure I understand what the issue is here. Could you provide some additional description of the problem?
Would you be able to include a Sandcastle example that replicates the issue? That would help us give us more context to determine the issue. Thanks!

I'm closing this issue due to inactivity. If you believe this is still an issue, please feel free to re-open.
Thanks!


Escape Sequence error

I have an text file as given below
750011114|GM"SMEET
I am using load statement as given below
load hadoop using file url '/user/XYZ/test1'
with source properties ('field.delimiter'='|','ignore.extra.fields'='true','quote.char'='"')
into table xyz.Test1 overwrite;
I am not able to load the data after executing above query. Please help me in resolving the isse

Could you share your stream/job definition?


Added a new button

completed #836

Hi, there seems to be a merge conflict, please resolve it

@PoorviShetty  Hey,I resolved it.

Can you please check again?

Can you please check again?
There r no more conflicts now

Can you please check again?
There r no more conflicts now

Hey!! @PoorviShetty there seems to be no more conflicts now.


macOS .pkg installer for Enterprise Deployments

While the homebrew package is great for personal and developer use, it severely limits the options for Enterprise Deployments (say through Jamf or Microsoft Intune) for corporate owned and managed Macs.
Is there an ETA or plan to release a .pkg installer for the Azure CLI tools? The "catch all" bash script is far from ideal for macOS as well :/

@jiasli for awareness

happy to test out any early release .pkg installers! :) I have access to a wide range of Macs both M1 and Intel running various versions ranging from Big Sur to the latest Ventura dev builds. I can test out MDM/Enterprise Deployments as well in Jamf Pro or Munki

happy to test out any early release .pkg installers! :) I have access to a wide range of Macs both M1 and Intel running various versions ranging from Big Sur to the latest Ventura dev builds. I can test out MDM/Enterprise Deployments as well in Jamf Pro or Munki

Any movement on this?
We have many devices we want to push this out to. Having it released as a pkg would allow IT to keep it updated so the end users don't have to worry about it.

Bumping this, would be nice to have for Enterprise that cant use brew and could be packaged and deployed easily via AutoPKG.


Use SystemTime rather than DateTime directly

Hey @jezzsantos, it can be useful to be able to control time in testing and staging. As SS already has SystemTime I've replaced direct use DateTime.

Hi George, this is very interesting. Thanks.
Kind of being looking for something like where we can control actual time in testing for a long time.
So, what's the idea?
If you use SystemTime in both the tests and the code, then somehow you can manipulate the underlying clock in a test, to fastforward/rewind the time that the code thinks it is?

That's right, in tests you can just set the resolver in the test.

Great stuff.
We should probably either then: agree to replace all use of DateTime everywhere in the code/tests with SystemTime, or continue using other time testing patterns. (Consistency is the most important thing here).

BTW: don't worry about the build failing, I am getting that resolved with AppVeyor. As soon as that is resolved, I'll sort out the PR's that are queuing up!


TimePicker: Adjust AM/PM width to fit other languages

We should update the width of our am/pm field in the date picker so it can work for other languages.
In this example, the characters should be side by side.

We should also test it out with other character languages. We want it to grow to fit other languages but not necessarily be larger for Roman alphabet.

Confirmed the time picker looks good for a variety of languages:


provide hashFunction parameter to DependencyTemplates

What kind of change does this PR introduce?

bugfix
Did you add tests for your changes?
no

Does this PR introduce a breaking change?
no

What needs to be documented once your changes are merged?
nothing

Thanks


Implement execLoadAsync

Modify execLoad to use execute_async and rename to execLoadAsync

Create an execLoad that calls it and waits for results for backwards compatibility. This isn't perfect since we're still doing async calls in parallel if we send multiple rows in data, but it minimises code duplication. We could create a true synchronous version easily enough, it will just be messier.

Optimise the string processing a bit

Thanks for this.  Happy to merge the code.
Will be interesting to do some more speed tests with other types of data. One question I have is that since the executeLoadAsync code doesn't wait, we presumably don't know that it's finished, so I'm not sure my end to end when it runs first time will produce the correct answer if I switch it to use executeLoadAsync (Cassandra may not have finished loading).

That's kind of the point. If you use executeLoadAsync you probably want to explicitly wait for it to complete somewhere, just not right away. That's why it needs to return the list of future objects to wait on.

Code is merged. Will create a new release of the code to PyPI.

Code has been pushed to PyPI as gkdbutils version 0.1.0.

Yeah - the slight issue I have is - what if Cassandra never completes the load? Are we giving ourselves a false idea of its load speed (especially of images) if we don't check that the futures objects have completed?  I guess we should store the futures objects somewhere and periodically check them to make sure they completed. We should maybe check then before loading the next batch of alerts...

It didn't occur to me to do any speed tests where I never waited for the operation to complete because, as you say, that would not be safe and you should not do that in real code. Assuming that it did in fact work without dropping anything I doubt whether it would be a lot faster - checking the result is a purely local operation and should be fast.


Tominthe toon patch 2

Added new 'Program Addresses' page and page reference in summary.

@DonDuala new PR with updated base. Hope this isn't causing you more hassle than worth!

@brittcyr are we missing any relevant PDAs / should we include all of these?


Can't import: ImportError: DLL load failed: %1 is not a valid Win32 application

When I run on Python27 import pptx I get the foolowing error:
>>> import pptx
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "c:\Python27\lib\site-packages\pptx\__init__.py", line 14, in <module>
    from pptx.api import Presentation  # noqa
  File "c:\Python27\lib\site-packages\pptx\api.py", line 15, in <module>
    from .package import Package
  File "c:\Python27\lib\site-packages\pptx\package.py", line 11, in <module>
    from .opc.package import OpcPackage
  File "c:\Python27\lib\site-packages\pptx\opc\package.py", line 13, in <module>
    from .oxml import CT_Relationships, serialize_part_xml
  File "c:\Python27\lib\site-packages\pptx\opc\oxml.py", line 12, in <module>
    from lxml import etree
ImportError: DLL load failed: %1 is not a valid Win32 application.

How can I fix it?

After installing Python 64 it was solved.


Babylon in an Ember component, Error: renderingGroup.prepare is not a function

Hi, I'm attempting to use Babylon within EmberJS, in a comoponent, and am getting the following error on scene.render():
renderingGroup.prepare is not a function

Here is the code for the component (works fine outside of Ember), any ideas?
import Ember from 'ember';

export
default Ember.Component.extend({
    buildScene: function() {
        var self = this;
        var canvas = document.getElementById("renderCanvas");
        var engine = new BABYLON.Engine(canvas, true);
        var scene = new BABYLON.Scene(engine);

        // This creates and positions a free camera (non-mesh)
        var camera = new BABYLON.FreeCamera("camera1", new BABYLON.Vector3(0, 5, -10), scene);

        // This targets the camera to scene origin
        camera.setTarget(BABYLON.Vector3.Zero());

        // This attaches the camera to the canvas
        camera.attachControl(canvas, true);

        // This creates a light, aiming 0,1,0 - to the sky (non-mesh)
        var light = new BABYLON.HemisphericLight("light1", new BABYLON.Vector3(0, 1, 0), scene);

        // Default intensity is 1. Let's dim the light a small amount
        light.intensity = 0.7;

        // Our built-in 'sphere' shape. Params: name, subdivs, size, scene
        var sphere = BABYLON.Mesh.CreateSphere("sphere1", 16, 2, scene);

        // Move the sphere upward 1/2 its height
        sphere.position.y = 1;

        // Our built-in 'ground' shape. Params: name, width, depth, subdivs, scene
        var ground = BABYLON.Mesh.CreateGround("ground1", 6, 6, 2, scene);
        engine.runRenderLoop(function() {
            scene.render();
        });
        window.addEventListener("resize", function() {
            engine.resize();
        });
    }.on('didInsertElement'),
});

Thanks,
Dan

Oh! I've already seen this. This is because ember redefine the array prototype :( You have to define array.prototype.prepare() to fix it

Thanks for the quick answer @deltakosh, where would I define it, overwrite the renderingGroup.prepare() function?

I think so, from what I can remember this is due to Ember that redefine array prototype like: array.prototype = foo

The (sort of) solution to this issue is to disable prototype extensions in Ember, you can do that by changing some environment variables in config/environment.js (Ember CLI):
EmberENV: {
    EXTEND_PROTOTYPES: false
}

This will disable all prototyping in Ember, which will force you to rewrite / change syntax on a bunch of stuff, see here:
http://guides.emberjs.com/v1.10.0/configuring-ember/disabling-prototype-extensions/
The only conflict specifically with Babylon.js are the Arrays, so you can also use:
EmberENV: {
    EXTEND_PROTOTYPES: {
        String: true,
        Array: false,
        Function: true
    }
}

Which will keep Ember's prototyping on Strings and Functions, but remove it for Arrays.  Then in your project, you'll need to find any Arrays you've declared as properties (you only need to do the ones you yourself declared) and convert them to Ember Arrays like so:
//Old Array
var myArray = [1, 2, 3, 4];

//New Array
var myArray = [1, 2, 3, 4];
Ember.A(myArray); // converts it to an Ember Array

Then things should work.  I'm not sure if turning Babylon into AMD (#62) would fix this altogether.  Also / alternatively, it'd be great if you could set the extend properties in Ember on a per component basis.


Add node for cosmos connections

Like this (except for just the cosmos node for now):

Things that still need to be decided:

The name of the node
The icon for the node

This needs to be added under a feature flag. See the appService.enableRemoteDebugging setting as an example. https://github.com/Microsoft/vscode-azureappservice#preview-features

What do you think about "Connected Utilities" or "Linked Services"? At least for the beginning?

Personally I vote for "Connections"


Is application containerized ?

@Tejas1510 Is this project containerized ?
If not I would like to containerize it

/assign

@Tejas1510 Can you assign me this ?

Go ahead @Jassi10000

@AmeyaJain-25 Yes, actually 80% stuff is done ,
but the issue is that application is breaking at some points , so struck in that only from last few days

Actually that's because of the singing in and out issues.
Currently you can comment the private routes and work with the application


ambiguous error message when refs are missing from a git dependency

This is related to, but not quite the same as, #3897.
When you run bundle install in a project that has a dependency on a git ref, and that ref no longer exists, you get an error message like
Git error: command `git reset --hard 0d4f9b7dcc50fe5235abd5035a49c7075ae514d6` in directory /home/doy/tmp/bundler-test/vendor/bundle/ruby/2.1.0/bundler/gems/bundler-test-0d4f9b7dcc50 has
failed.
If this error persists you could try removing the cache directory '/home/doy/tmp/bundler-test/vendor/bundle/ruby/2.1.0/cache/bundler/git/bundler-test-a59c53842b1d2c30f68c5498ce0cca55e39b6e20'

This is pretty unhelpful, since removing the cache directory isn't actually going to fix anything in this case. A much more helpful response here would be for bundler to automatically remove the cache directory and try again, and if it fails the second time, give an error message indicating that the remote ref actually doesn't exist.

What version of bundler are you running? I think we might have improved this in the 1.11 beta. If not, a pull request doing so would be much appreciated :)

This was on 1.10.6 - I'll give it a try, thanks!


Support for "/etc/hosts" file dns entry

Sometimes I need to alter the DNS name being used for testing due to a virtual host based registration system (Kazoo & Sipwise).
From what I can see there baresip uses a direct connection to the dns server to query the IP. Is it possible to have an override in the config or in the account definition to allow a different IP for a domain name?

baresip is using the DNS-module from libre, which implements its own DNS client talking
directly to the configured DNS nameservers(s).
I guess we could add this functionality in libre, by looking up A queries in /etc/hosts
before the DNS request is sent out.

It is kind of what I figured.. just helps a lot when debugging servers...

is it possible for you to use a local DNS proxy/cache that also reads entries
from /etc/hosts ?
at the moment I do not have any plans to add support for this in libre's resolver.
you are welcome to create a patch for this, and send it to us for review.
if there is no more activities then feel free to close this issue.


AdminClient throws StackoverflowException when passing in configuration and attaching event handler

Description
A stackoverflow exception is caused on the AdminClient when attempting to attach to any of the logging events. This scenario occurs when you pass in the configuration to the client.
How to reproduce

Create Configuration
Pass configuration into AdminClient (eg new AdminClient(new Dictionary....))
Attempt to attach to any logging event handler (eg: OnLog)
Process will crash with a StackoverflowException

Workaround
using (var producer = new Producer<byte[], byte[]>(config))
using (var adminClient = new AdminClient(producer.Handle))
{
    adminClient.OnLog += (sender, message) => {};
}

Notes

No log messages seem to be produced either (even with debug: all)
Transport failures occur when trying to get metadata using: adminClient.GetMetadata(TimeSpan.FromSeconds(5))

Checklist
Please provide the following information:

[x] Confluent.Kafka nuget version: 1.0.0-beta2
[x] Apache Kafka version: n/a
[x] Client configuration:
[x] Operating system:
[x] Provide logs (with "debug" : "..." as necessary in configuration)
[x] Provide broker log excerpts
[x] Critical issue

thanks for the report. In #716, i'm proposing changing how clients are instantiated, which as a side effect resolves this issue. I just added an integration test for this specific case to that PR: https://github.com/confluentinc/confluent-kafka-dotnet/commit/f1812165767dcc3881ac89c9538c648f46e74543

closing since this has been addressed (though isn't in a release yet).

see als #670.

Thanks for the response @mhowlett - any idea of when we can expect the new bits to be available (even as a beta package) so I can carry on? Happy to beta test, since we are doing a lot of testing of new functionality as part of a migration anyway.


[Bug]: remix-architect hard codes URL scheme in requests to https, even for dev / sandbox server where requests are made using http

Which Remix packages are impacted?

@remix-run/architect

What version of Remix are you using?
1.1.1
What version of Node are you using? Minimum supported version is 14.
14.18.2
Steps to Reproduce

Start an architect project using create-remix.
Start arc sandbox.
Start npm run dev.
Retrieve request.url from any loader.

Expected Behavior
The URL scheme should be http://.
Actual Behavior
The URL will start with http://, even though the actual request was made using http://, because the URL scheme is hardcoded in: https://github.com/remix-run/remix/blob/776cb79b97bdfba42f802c2f583b1ec5d4dcb785/packages/remix-architect/server.ts#L116

https://github.com/remix-run/remix/pull/1537 was merged to dev, will be out next week

Indeed, it is fixed now. Thank you!


DCOS-14255: Clean up advanced package install review

This PR fixes a bug where the advanced package install modal's review stage wasn't rendering properly in Firefox. The modal's body wasn't shrinking to allow for the modal footer to be visible.
I also reduced the complexity of the displayed configuration by simply passing it to HashMapDisplay which improves the consistency of this view.
Before:

After:

Checklist

[x] Did you add a JIRA issue in a commit message or as part of the branch name?
[x] Did you add new unit tests?
[ ] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

run integration tests

Thanks for the fix and the quick turnaround! Let's ⛵ this thing!

retest this please

@jfurrow test is failing - ReviewConfig-test.js

Thanks @MatApple, I accidentally committed an autoMockOff() and removing it killed the tests — fixing now!

@MatApple Fixed!


feat(tz): add pytz as a requirement to get better timezone data (Olsen DB)

https://docs.djangoproject.com/en/1.9/topics/i18n/timezones/ recommended

Sounds good--I see we already have USE_TZ = True in settings.py.


Switch to find & remove globbing for directories

Having over 20k directories would result in 'argument list too long' as the glob would expand them on the command line. Switched this over to find type directory.

Thank you.


--version format change?

https://updater.factorio.com/get-available-versions?apiVersion=2
Unable to determine source version. Please provide either a starting version (with --for-version) or a Factorio binary (with --apply-to).
Did the --version output format change? Seems broken with 0.16.34

Fixed by #41, thanks for the report!
The format did change, but only in that a couple of extra lines started showing up before the --version output:
   0.000  Current open file limits: 1024, 1048576
   0.000  Setting open file limits: 1024, 1048576
Version: 0.16.33 (build 36193, linux64, headless)
Binary version: 64
Map input version: 0.13.1-0
Map output version: 0.16.33-0

I actually hadn't realized I was doing a "must start with" match, but I guess I had been.

Awesome, thanks for responding to the issue so quickly!
On Sat, Mar 24, 2018 at 11:12 AM, narc0tiq<EMAIL_ADDRESS>wrote:

Fixed by #41 https://github.com/narc0tiq/factorio-updater/pull/41,
thanks for the report!
The format did change, but only in that a couple of extra lines started
showing up before the --version output:
0.000  Current open file limits: 1024, 1048576
0.000  Setting open file limits: 1024, 1048576
Version: 0.16.33 (build 36193, linux64, headless)
Binary version: 64
Map input version: 0.13.1-0
Map output version: 0.16.33-0
I actually hadn't realized I was doing a "must start with" match, but I
guess I had been.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/narc0tiq/factorio-updater/issues/40#issuecomment-375909093,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Af3G9uQ9oiJdtT6w3TqPNLH1qHUitff8ks5thn6YgaJpZM4S5oW2
.


Fixes required to pass updated pylint checks

Pylint has been updated and is picking up a couple of new issues, hence all test builds will fail. This PR fixes the issues.

Coverage remained the same at 57.713% when pulling 42fd7b9c73d950f96d3ccf31ce8f355923781034 on pwt:pylint_fixes into 989f6af060affecd498f7f111d06f56d7002f918 on SoCo:master.

Thanks, merged


modify direqual add filter

i check all everywhere when they call this method, they are all need to use hiddenfilefilter,its realy need a new method with an new argumen?

@hn5092 - Can you elaborate on the advantages of doing this and re-open the PR when ready?


grammar error fixer in CONTRIBUTING.md

grammer error fixed in CONTRIBUTING.md
Screenshots - If Any (Optional)
Issue ticket number and link
Checklist before requesting a review

[ ✓] My code follows the style guidelines of this project
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have performed a self-review of my code
[ ] If it is a core feature, I have added thorough tests.
[ ] New and existing unit tests pass locally with my changes

Please open the PR against the dev branch (Unless the PR contains only documentation changes)

@slashben Please take a look


Composer's OAuth card not rendering in Teams

Hi,
I have setup a chatbot exposed on webchannel and Teams with a dialog that displays an Oauth card (that uses AAD v2 as identity provider). I have already setup app registrations and the connection name for the bot and put it in the card property in Composer.
This Oauth card works fine when chatting with the bot via webchat (on Azure portal).
I then have deployed my bot as an App on Teams, and tried to test the same OAuth dialog/card.
However, I'm having trouble with the Oauth card, which does not render in Teams (I hope it works after I can fix the visualization problem, but we'll see).
My Teams app manifest looks like this:
{
  "$schema": "https://developer.microsoft.com/en-us/json-schemas/teams/v1.7/MicrosoftTeams.schema.json",
  "manifestVersion": "1.7",
  "version": "1.0.0",
  "id": "BOT-ID-REDACTED",
  "packageName": "com.microsoft.teams.samples",
  "developer": {
    "name": "Microsoft",
    "websiteUrl": "https://example.azurewebsites.net",
    "privacyUrl": "https://example.azurewebsites.net/privacy",
    "termsOfUseUrl": "https://example.azurewebsites.net/termsofuse"
  },
  "icons": {
    "color": "icon-color.png",
    "outline": "icon-outline.png"
  },
  "name": {
    "short": "chatbot",
    "full": "chatbot"
  },
  "description": {
    "short": "short description",
    "full": "Test version of chatbot"
  },
  "accentColor": "#FFFFFF",
  "bots": [
    {
      "botId": "BOT-ID-REDACTED",
      "scopes": [
        "personal",
        "team",
        "groupchat"
      ],
      "supportsFiles": false,
      "isNotificationOnly": false
    }
  ],
  "webApplicationInfo": {
    "id": "AAD-AppRegistration-ID-REDACTED",
    "resource": "api://botid-BOT-ID-REDACTED"
  },
  "permissions": [
    "identity",
    "messageTeamMembers"
  ],
  "validDomains": [
    "token.botframework.com"
  ]
}

I attach two screens of what I see via Webchat and what I (don't) see when chatting via Teams

Finally, I checked via f12 in Teams (on a browser) what is logged in console and all I could see is a permission needed.

What am I missing here? Does anyone know what should I do to grant such permission or to configure correctly the OAuth card for Teams?
Thank you very much in advance for your assistance.

@stevkan Ping.

@Erry91, can you specify what version of the Composer you are on? Also, does the issue prevail after fixing the visualization problem? Let me attempt a repro on my end and update the thread with my findings.

Hi @anusharr, I can tell you that the Composer's version we're using is the 2.1.1
I can't tell you exactly what would happen if we were able to visualize the card in Teams. For sure the card appears in webchat and the oath connection works fine in that channel. Is there any particular approach in composer if the card is to be used in Teams?
Thanks, Enrico.

@Erry91, I tried running a sample on my end and the OAuth card worked just fine on teams. Can you elaborate more on the steps you followed to setup the OAuth bot on Composer ? Are you trying to run an already existing sample? Can you share screenshots of the configuration in Composer? Also, do you get any additional error on the Teams channels health tab on Azure?

Hi @anusharr and thank you again for your assistance.
I am putting the screens of my Composer setup and try to document for you as much as possible what I have in the configuration on the azure portal as well if it can help.
This is the trigger that will call the oauth dialog, isolated like this for testing purposes.

On the azure portal instead, on the chatbot teams health screen I don't see any issue related (I triggered the dialog again to be sure before taking the screen):

I share with you the screen of the bot configuration, please bear with me with the fact that I have to redact it. But I indicated with:
X. is the bot ID and it's the same indicated in the app and in the token exchange url in the form "api://botid-BOTID/access_as_user"
O. It's the app registration ID of the app that I use for Identity provider.

If you think it could be easier, I can also find availability for a screen sharing session in case you think we could find a solution by doing live some attempts.
Enrico


the [DRAFT] badge should disappear when editing front matter

Currently editing draft: true to draft: false in the page front matter is not reflected on the preview (and vice versa).

related #240 #660


Skip the config if not in interactive mode

The title is self-explanatory, but I will elaborate a bit: this will speed up plugins making use of subshells, for example tide (prompt item functions run in subshell) and fzf.fish (fzf's preview run in subshell).
I'm not sure if this change will break some things potentially, but as far as I can tell, not likely.

Thank you @kidonng 👍


Tidy & reword “Configure a Pod to Use a Projected Volume for Storage”

Tidy & reword “Configure a Pod to Use a Projected Volume for Storage”

reference the glossary entry for Secret
write Volume in initial-caps
write container in lowercase
fix sample output marked as Bourne shell code

/kind cleanup

People may have different opinions about whether we should use Container or container. But that doesn't sounds like a blocking issue.
/lgtm
/approve


The class definition for eventLog1 is missing

The sample code for "Add features to the service" looks to be missing the EventLog class when creating eventLog1.

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

ID: 75152e30-b486-3ba0-c606-25ada7347b22
Version Independent ID: e797ddfe-a9d3-93a4-61bd-fd6dc9b8d6b4
Content: Tutorial: Create a Windows service app
Content Source: docs/framework/windows-services/walkthrough-creating-a-windows-service-application-in-the-component-designer.md
Product: dotnet-framework
GitHub Login: @ghogen
Microsoft Alias: dotnetcontent

That variable is created by adding the EventLog component in the Visual Designer. This is in step 2 under Add custom event log functionality.
Hope this helps.


MediaPlayerElement from Microsoft.Toolkit.Wpf.UI.Controls not working properly in in .net core 3.0.0

Describe the bug

Tried implementing UWP MediaPlayerElement in .net core 3.0.0  WPF application but Video is not playing sometime.
Steps to reproduce the bug

Steps to reproduce the behavior:

Add MediaPlayerElement from Microsoft.Toolkit.Wpf.UI.Controls
Give a video source file in xaml code and set AutoPlay property to true.
When playing a single video, it is playing it fine. When adding two MediaPlayerElement controls and trying playing them simultaneously then it is not playing.
Now even the single Video will not get played until the computer is restarted.

Expected behavior

Video should play as and when project is run.
Screenshots

Version Info

NuGet package version:

Microsoft.EntityFrameworkCore Version="3.0.0"
Microsoft.Toolkit.Wpf.UI.Controls  Version="6.0.0-preview7.1"
Microsoft.VCRTForwarders.140 Version="1.0.2-rc"

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

May 2019 Update (18362)
Yes

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Mobile

Xbox

Surface Hub

IoT

Additional context

@marb2000 - are you able to look into this to see if you can repro?

Hi,
Have we come to any conclusion regarding this issue?


[BUG] SKXamlCanvas reports only half for ActualWidth and ActualHeight on iOs

Description
If I draw something using the ActualWidth and ActualHeight of the SKXamlCanvas on iOs, it reports half of the size the canvas actually is. If e.g. I draw a rectangle like this:
e.Surface.Canvas.DrawRect(new SKRect(0, 0, (float)ActualWidth, (float)ActualHeight), new SKPaint(SKColors.Red) });
it only occupies one quarter of the whole canvas. I'm not sure if there is any Scaling factor in place or some DPI-issue. It might be related to this issue, where the underlying Bitmap is also rendered smaller on iOs and Android then it should be.
Is there any scaling/DPI-setup happening in SkiaSharp?
I may provide a sample if necessary - just running out of time for the moment.
Expected Behavior
The ActualSize is correct

Actual Behavior
The ActualSize is reported wrong.

@jeromelaban Do you have an idea on this one? The size reported from SKXamlCanvas on iOs is wrong - it is only 1/2 of the real size.

This is likely expected if your screen is using a scale of 200%. ActualWith and ActualHeight are DPI independent. You'll need to use the canvas size to do your computations.

Depending on what Uno does, you can either use the XamlRoot.RasterizationScale or the DisplayInformation.GetForCurrentView().LogicalDpi / 96.0.
That is what I do for UWP/WinUI things: https://github.com/mono/SkiaSharp/blob/main/source/SkiaSharp.Views/SkiaSharp.Views.UWP/SKXamlCanvas.cs#L100-L113

Thanks! That did the trick. I've used the Canvas-Size for the basic calculations and used the Dpi-Value to adjust touch points.
Thank you both!


RIAK Node does not Start after changing IP

I am in the process of setting up a Riak Cluster on Raspberry Pis. Unfortunately I get the following error message after changing the IP address. Versions I used: Debian Jessie (Raspberry PI) riak (Github Clone Mar2017) riak-cs2.1.1 stanchion-2.1.1, OTP_SRC_R16B02-basho10 (Erlang)
Using this guide I tried to change the IP addresses in the various .conf files. https://docs.riak.com/riak/kv/latest/using/cluster-operations/changing-cluster-info/index.html
Works on <IP_ADDRESS>:
$ ~/riak/rel/riak/bin/riak-admin test
Successfully completed 1 read/write cycle to<EMAIL_ADDRESS>Error Message (after changing IP:<IP_ADDRESS>):
sudo ./riak console
config is OK
-config /home/pi/neu/riak/rel/riak/data/generated.configs/app.2<IP_ADDRESS>.37.52.config -args_file /home/pi/neu/riak/rel/riak/data/generated.configs/vm.2<IP_ADDRESS>.37.52.args -vm_args /home/pi/neu/riak/rel/riak/data/generated.configs/vm.2<IP_ADDRESS>.37.52.args
Exec:  /home/pi/neu/riak/rel/riak/bin/../erts-5.10.3/bin/erlexec -boot /home/pi/neu/riak/rel/riak/bin/../releases/2.2.3/riak               -config /home/pi/neu/riak/rel/riak/data/generated.configs/app.2<IP_ADDRESS>.37.52.config -args_file /home/pi/neu/riak/rel/riak/data/generated.configs/vm.2<IP_ADDRESS>.37.52.args -vm_args /home/pi/neu/riak/rel/riak/data/generated.configs/vm.2<IP_ADDRESS>.37.52.args              -pa /home/pi/neu/riak/rel/riak/bin/../lib/basho-patches -- console
Root: /home/pi/neu/riak/rel/riak/bin/..
Erlang R16B02_basho10 (erts-5.10.3) [source] [smp:4:4] [async-threads:64] [hipe] [kernel-poll:true] [frame-pointer]
[os_mon] memory supervisor port (memsup): Erlang has closed
[os_mon] cpu supervisor port (cpu_sup): Erlang has closed
{"Kernel pid terminated",application_controller,"{application_start_failure,riak_core,{bad_return,{{riak_core_app,start,[normal,[]]},{'EXIT',{{function_clause,[{orddict,fetch,['riak@<IP_ADDRESS>',[]],[{file,"orddict.erl"},{line,72}]},{riak_core_capability,renegotiate_capabilities,1,[{file,"src/riak_core_capability.erl"},{line,441}]},{riak_core_capability,handle_call,3,[{file,"src/riak_core_capability.erl"},{line,213}]},{gen_server,handle_msg,5,[{file,"gen_server.erl"},{line,585}]},{proc_lib,init_p_do_apply,3,[{file,"proc_lib.erl"},{line,239}]}]},{gen_server,call,[riak_core_capability,{register,{riak_core,vnode_routing},{capability,[proxy,legacy],legacy,{riak_core,legacy_vnode_routing,[{true,legacy},{false,proxy}]}}},infinity]}}}}}}"}
Crash dump was written to: ./log/erl_crash.dump

I might expect to see this if you hadn't done the step of either renaming (or deleting the contents of) the ring directory.  Did you do this?
Also can you confirm if you're in the single-node or multi-node renaming scenario?

Many thanks for the quick feedback!
The error has been solved after I deleted the ring directory files.
After that there were some more errors, but I could solve them and now I could create a cluster of 4 pis.

================================= Membership ==================================
Status     Ring    Pending    Node
valid      25.0%      --<EMAIL_ADDRESS>valid      25.0%      --<EMAIL_ADDRESS>valid      25.0%      --<EMAIL_ADDRESS>valid      25.0%      --<EMAIL_ADDRESS>Valid:4 / Leaving:0 / Exiting:0 / Joining:0 / Down:0


US97068 Save last search

When a user clicks a tab, we save that as the most recent search. This behaviour exists in both the widget view and the all courses view - an attempt to combine the behaviours was made, but it ended up being a significantly larger change for very little benefit.

@jlai-d2l pointed out on Slack - I've got this wired up so that All Courses opens at the same tab that you've currently got selected in the widget, but not the opposite (i.e. selecting the tab in the widget that you were on in All Courses, when the latter is closed). Going to look into that - if I can figure out a way to do it easily/quickly, I'll update this, but otherwise I'll merge this and create a DE for it.

@ryantmer I think that would be worth logging as a DE and following up on, yeah.

Hokay, DE30403 created for that situation - in that case, I'll merge and deploy this.


SqlStreamStore.MsSql does not support netcoreapp1.1/netstandard 1.6

I'm developing a dotnet core netcoreapp1.1 web application and am unable to add the SqlStreamStore.MsSql library as it is not compatible.

I took a quick look at converting the projects to dotnet after first opening this and ran into the issue of Nito.AsyncEx not supporting the netstandard. I dug more into it today, and it looks like the owner has split the project up into smaller projects. The Nito.AsyncEx.Cooridination package appears to have the two types being used AsynceAutoResetEvent and TaskCompletionSource. I'll see if I can get this building for dotnet core with the updated dependency and provide an update.

NetStandard support on merged to master. CI builds on https://www.myget.org/F/cedar/api/v3/index.json shortly so you can test the packages.


There is a method missed in LanenetCluster class

In test_lanenet_instance.py got error:
Traceback (most recent call last):
  File "tools/test_lanenet_instance.py", line 107, in <module>
    test_net(args.image_path, args.weights_path, args.net)
  File "tools/test_lanenet_instance.py", line 85, in test_net
    predict_color = cluster.get_instance_masks_image(ori_image=ori_image,
AttributeError: 'LaneNetCluster' object has no attribute 'get_instance_masks_image'

The method get_instance_masks_image is missed?

@MaybeShewill-CV Hi, may I ask the instance label, how to set the pixel value of each lane, just random using value between 1-255 or using the class index 1, 2, 3, 4, 5...?

@jinfagang I use 30 50 70 90 110 to label different instance. You can any other color index to label different instance lane line.


Strange bold text on OCP details page

Describe the bug
unused/unrequested capacity text is bold on the OCP details page when grouping by cluster
To Reproduce
Steps to reproduce the behavior:

Go to 'Openshift details'
Click on 'group by cluster'
Scroll down to cluster and expand row
See the cluster capacity text

Expected behavior
Text should match other text in UI
Screenshots

Desktop (please complete the following information):

OS: Fedora
Browser: Firefox
Version: 70

This styling comes from the PatternFly TextList component. You can see this with the PatternFly data list example https://patternfly-react.surge.sh/documentation/react/components/text#examples
Note that this also matches the mock https://marvelapp.com/1jj511j6/screen/64401384
Mock snapshot:


HIVE-24169: HiveServer2 UDF cache

What changes were proposed in this pull request?
This feature adds a HiveServer2 level UDF cache that can be shared across sessions.
Why are the changes needed?
Without this feature, on S3 based system, describe function and select udf() will be localized each and every time, taking 2 minutes to download 300mb.
Does this PR introduce any user-facing change?
no
How was this patch tested?
manual

Can HDFS be used cache? In case where multiple HS2 are running, this will just cache the udf on one of the HS2. If user gets connected to other HS2, again UDF needs to be downloaded and cached.

Can HDFS be used cache? In case where multiple HS2 are running, this will just cache the udf on one of the HS2. If user gets connected to other HS2, again UDF needs to be downloaded and cached.
We don't have an HDFS in our use-case. We have on-prem s3 storage and an efs file system in hs2. Can you create a seperate jira so that we can track the proposed optimization in that one?


[flutter_tools] refactor build_system and targets to be context-free

Description
Refactors the build_system/target classes to be context free. Unlike other services in the flutter tool, the target classes need to be const and have the same API - we can't use constructor injection or setters to provide the additional dependencies. To work around this I expanded the Environment object to contain the ProcessManager, Logger, FileSystem, and Artifacts objects. I don't love this, since it is essentially the DI container I said we should never do.

the target classes need to be const and have the same API - we can't use constructor injection or setters to provide the additional dependencies.

Can you add named constructors for testing dependency injection only?

Could you give an example of what you mean?

Another approach I could take, is to stop relying on const instances and give each class a meaningful constructor. I could put a different const identifier as a field and use that instead.

That ended up being a huge change. :9

@jmagman I think I could gradually move towards that model. As is, it's a big change.


what

[Enter feedback here]

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

ID: cf5bf65b-0725-ce75-22a3-1b902008187e
Version Independent ID: fc6b8570-983c-b25d-a8fd-29685bc17153
Content: date
Content Source: WindowsServerDocs/administration/windows-commands/date.md
Product: windows-server
Technology: windows-commands
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

#assign:xelu86

User feedback is undeterminable. Closing thread.
#please-close


feat(proposal): Dogfooding of trestle in the oscal-compass community

This is a proposal to use trestle within the proposals process.
The intent that this should be debated on two levels:

The validity of doing so

The document structure of the proposal, such that it can be used as the actual template if approved.

@butler54 I completely agree that using trestle here is a good opportunity. I want to link the CNCF design proposal example. I think we could tailor this for the template. What you have already lines up well. I would recommend adding ~Goals~, ~Non-Goals~, and Impact/Key Questions.
Editing my recommendation: You already have Definition of Done, so maybe adding Scope to communicate Non-Goals. WDYT?

@jpower432 / @jflowers  - I've made an incremental change, however, looking at the CNCF design proposal template. I think we should faithfully recreate it. Why: It gives us a bit of a springboard to a long term goal: trestle, by default, is expected by the CNCF to be use used to enforce templates (maybe I am being ambitious).
if we agree this is a good idea moving forward I'll translate it into the proposal template @jpower432's referenced with a few tweaks (I would like to see the metadata stored as a a yaml header).

if we agree this is a good idea moving forward I'll translate it into the proposal template @jpower432's referenced with a few tweaks (I would like to see the metadata stored as a a yaml header).

@butler54 I am on board with this. The template we have is almost as comprehensive as the CNCF design proposal template so we might as well align on the naming of the sections for consistency/resuse. Another example of a good comprehensive template is Kubernetes Enhancements.

I like it! I think it's a good ambition.

Okay this should be good to review now / ready to roll.
As mentioned in the document the project will need to be updated to mandate trestle author docs validate passing.

It's good to be merged now based on your feedback.


transformations inside of init

Hi! At one point, I thought the code would be clearer if I applied the right transformations to the layers immediately instead of on the forward pass. For example, my encoder which expect 2D batch (just an example):
class Encoder(eqx.Module):
    encoder: nn.Linear

    def __init__(self, input_dim, *, key):
        self.encoder = eqx.filter_vmap(nn.Linear(input_dim, 256, key=key))

    def __call__(self, obs):
        # obs: [batch_size, input_dim]
        emb = self.encoder(obs)
        return emb

Although the normal apply_updates works (at least it doesn't throw errors), in RL it is often necessary to update weights in this way:
@eqx.filter_jit
def example_soft_update(model):
    params, static = eqx.filter(model, eqx.is_array)
    new_params = optax.incremental_update(params, params, 0.1)   # usually there will be (params, target_params), not same weights
    return eqx.combine(new_params, static)

And with module with vmaped layer inside this will raise an error. Should I filter params more rigorously or this is just a bad approach in general?

I'm assuming your first example is meant to use self.encoder = eqx.filter_vmap(nn.Linear)(input_dim, 256, key=key)? In which case, are you really wanting to use a different encoder for each batch element? That'd be very unusual.
In your second code snippet, you probably meant to write params, static = eqx.partition(...) rather than eqx.filter(...). That's probably the source of the error.
There's nothing wrong with using filter_vmap'd modules! You just need to be careful where you use them.

No! Actually I want same encoder to each element in the batch (I was a little too lazy to write in the call vmap every time on the batches and that's how I stumbled upon this error)

This is standalone example which I use:
import jax
import equinox as eqx
import equinox.nn as nn
import optax

class TestModel(eqx.Module):
    encoder: nn.Linear

    def __init__(self, in_channels, *, key):
        keys = jax.random.split(key, 3)
        self.encoder = eqx.filter_vmap(nn.Linear(in_channels, 265, key=keys[0]))

    def __call__(self, obs):
        emb = self.encoder(obs)
        return emb

@eqx.filter_jit
def update(critic, target_critic):
    params, static = eqx.partition(critic, eqx.is_array)
    target_params, target_static = eqx.partition(target_critic, eqx.is_array)

    new_target_params = optax.incremental_update(params, target_params, 0.1)
    new_target_criitc = eqx.combine(new_target_params, target_static)

    return new_target_criitc

key = jax.random.PRNGKey(0)
critic = TestModel(3, key=key)
target_critic = TestModel(3, key=key)

# this will raise an error
new_critic = update(critic, target_critic)

I think I got it....
eqx.filter_vmap(nn.Linear)(3, 256, key=key)  # this is same linear vmaped on full batch

eqx.filter_vmap(nn.Linear(3, 256, key=key)) # this is different linear for each example in the batch

However, I cannot use first variant as keyword arguments cannot be used with functions wrapped with filter_vmap for the key=key and key always should be keyword argument...
Well, I guess we should always do vmap in call

eqx.filter_vmap(nn.Linear)(3, 256, key=key)  # this is same linear vmaped on full batch

eqx.filter_vmap(nn.Linear(3, 256, key=key)) # this is different linear for each example in the batch

Other way around! The first one creates a batch of Linear layers; the second one batches the single Linear layer that is created.

However, I cannot use first variant as keyword arguments cannot be used with functions wrapped with filter_vmap for the key=key and key always should be keyword argument...

Ah, that's annoying. That can be worked around by doing  eqx.filter_vmap(lambda *args, key: nn.Linear(*args, key=key))(3, 256, key) instead, but I can see that that's quite annoying. Maybe we should do the same thing JAX does and just unconditionally set in_axes=0 for all keyword arguments.

This is standalone example which I use:

Thanks for the repro! I've ended up tracking this down to a incompatbility in the three-way combination of dataclass and property and __wrapped__. (All Python built-ins.) I'll have a think about what a possible fix might be.

Alright, this should be fixed in #381!


Add reactor uc

Initial draft (scripts still don't work)
make all in root should work after lfcg was run

@LasseRosenow, will you tag me for a review again after making modifications as discussed in today's meeting?

@erlingrj Ready for review again

This looks great. I have given you write access to the repo so you can push PRs directly to this instead of a fork


Missing Project File /Server

After node run
I get 'Couldn't find a project to run. Ensure a project exists in .\Server"

If you're run npm run build command, ensure that your options has cwd configured:
const options = {
	cwd: path.resolve(__dirname, './server/'), //add this line
	stdio: ['ignore', 'pipe', 'inherit']
};


Kubernetes variant

Hello @flaviostutz
Thank you so much for your work, just to let you know we made the Kubernetes implementation of your work and hosted it here: https://github.com/kerberos-io/virtual-rtsp.
Thanks for the great work done!
Cedric

Hi, @cedricve! Great work there as well! I am honored by your care in getting this back to me. Thanks!


Add Metering Ansible Operator

Though this passed CI, this is unlikely not work out of the box.
Issue:

Dependencies Needed
Ansible Version 2.6.0 and up
Ansible-Runner 1.1.0 and up
Ansible-Runner-Http 1.0.0 and up

Think I need to remove size from roles/meteringconfig/defaults/main.yml

Image names are hardcorded into deploy/operator.yaml

Operator SDK usage also we need to think about what Namespace

/test metering-e2e-aws

1 minor thing unresolved, otherwise LGTM

I'll just make that minor change myself so I can be unblocked on this.
/approve
/lgtm


example run fail

assets xml or json file has slider, but when run , error log:
E/flutter (11857): [ERROR:flutter/lib/ui/ui_dart_state.cc(157)] Unhandled Exception: Exception: No parser for element name slider found
E/flutter (11857): #0      FormParserService.parseElement (package:dynamic_forms/src/parser/form_parser_service.dart:28:7)
E/flutter (11857): #1      JsonParserNode.getChildren. (package:dynamic_forms/src/parser/json/json_parser_node.dart:51:31)
E/flutter (11857): #2      MappedListIterable.elementAt (dart:_internal/iterable.dart:417:29)
E/flutter (11857): #3      ListIterator.moveNext (dart:_internal/iterable.dart:346:26)
E/flutter (11857): #4      CastIterator.moveNext (dart:_internal/cast.dart:63:30)
E/flutter (11857): #5      new List.from (dart:core-patch/array_patch.dart:35:29)
E/flutter (11857): #6      Iterable.toList (dart:core/iterable.dart:400:5)
E/flutter (11857): #7      JsonParserNode.getChildren (package:dynamic_forms/src/parser/json/json_parser_node.dart:53:14)
E/flutter (11857): #8      FormGroupParser.parse (package:flutter_dynamic_forms_components/src/components/form_group/form_group_parser.g.dart:18:28)
E/flutter (11857): #9      FormParserService.parseElement (package:dynamic_forms/src/parser/form_parser_service.dart:30:19)
E/flutter (11857): #10     JsonParserNode.getChildren. (package:dynamic_forms/src/parser/json/json_parser_node.dart:51:31)
E/flutter (11857): #11     MappedListIterable.elementAt (dart:_internal/iterable.dart:417:29)
E/flutter (11857): #12     ListIterator.moveNext (dart:_internal/iterable.dart:346:26)
E/flutter (11857): #13     CastIterator.moveNext (dart:_internal/cast.dart:63:30)
E/flutter (11857): #14     new List.from (dart:core-patch/array_patch.dart:35:29)
E/flutter (11857): #15     Iterable.toList (dart:core/iterable.dart:400:5)
E/flutter (11857): #16     JsonParserNode.getChildren (package:dynamic_forms/src/parser/json/json_parser_node.dart:53:14)
E/flutter (11857): #17     FormParser.parse (package:flutter_dynamic_forms_components/src/components/form/form_parser.g.dart:18:28)
E/flutter (11857): #18     FormParserService.parseElement (package:dynamic_forms/src/parser/form_parser_service.dart:30:19)
E/flutter (11857): #19     FormParserService.parse (package:dynamic_forms/src/parser/form_parser_service.dart:19:12)
E/flutter (11857): #20     FormManagerBuilder.build (package:dynamic_forms/src/form_manager/form_manager_builder.dart:15:34)
E/flutter (11857): #21     _SimpleFormScreenState._buildForm (package:example/simple_form/simple_form_screen.dart:52:39)

Hi @muziling,
unfortunately, I am not able to reproduce this issue.
Based on the stack trace it looks like you are running the example project in the flutter_dynamic_forms_components package. It also looks like there is a missing parser object for the slider component. Because the slider component was added recently to the flutter_dynamic_forms_components package I guess the issue might be that you pulled the new changes but you are building your project against the old dependency where there was no slider parser.
Please try calling
flutter packages upgrade to refresh dependencies and flutter clean before building the project.
Or remove those lines from the pubspec,yaml:
flutter_dynamic_forms_components:
    path: ../

save the file, add it back again and save it one more time.
Please let me know if this helps.
Thanks!

Seems flutter_dynamic_forms_components  version 0.7.2 not support  slide,  remove slide from json file temporary ,My app no need slide now

Oh, now I see!
I forgot to publish the new version to the pub.
It is published now so you can just update it to 0.7.3.

Thanks ,  close.


Day56

Resolves #123.

Pull Request Test Coverage Report for Build 264

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 262:
0.0%

Covered Lines:
24

Relevant Lines:
24

💛  - Coveralls


Usage of GPIO16/D0

Hello, I need to modify my version of this device to use GPIO16 in exchange for the pin used for "OUT or ALARM", as that GPIO by default is not accessible.
I have scrubbed the source and am having a hard time finding where that mapping is.
Does anyone know where I could be able to find this?
Thanks!

Found it, it's done by the software configuring the device. Nice
https://github.com/home-assistant/core/blob/dev/homeassistant/components/konnected/const.py#L42


Remove debug assertions due to invalid syntax in generators transform

This removes the debug assertions in the generators transform that occur when we are unable to find an associated target for a break or continue statement. These resulted in debug assertions when we should be reporting syntax errors.
Fixes #17875

We need to port this to release-2.5

This is now in release-2.5


Bestiary

Add enemy info page. Make enemies drop some item to "research" each enemy and unlock more info.
Ideally 1 item > Name and image
5 items > Stats
20 items > Full info?

Just unlock entries, not pieces.

Donezo.


Update versions for github actions

Updating setup-miniconda from 2 to 3
Update checkout from 3 to 4

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

Comparison is base (f41d2fa) 75.48% compared to head (66f230e) 34.54%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #161       +/-   ##
===========================================
- Coverage   75.48%   34.54%   -40.95%
===========================================
  Files          27       27
  Lines        1844     1844
===========================================
- Hits         1392      637      -755
- Misses        452     1207      +755

Flag
Coverage Δ

unittests
34.54% <ø> (-40.95%)
:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.