Question: can I change habit and reward names?

If I do, will the extension break / make new ones and ignore the og one?

Hi, please do not change the names of the rewards and habits created by the
extension, it might cause some issues with the connection to Habitica
On Fri, 3 Feb 2023, 10:40 writeblankspace, @.***> wrote:

If I do, will the extension break / make new ones and ignore the og one?
—
Reply to this email directly, view it on GitHub
https://github.com/ofekmiz/Habitica-Pomodoro-SiteKeeper/issues/116, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AJOEOFSZWYO5OPL6CSP3QG3WVS76FANCNFSM6AAAAAAUP7TC74
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

ok, thanks!

Is the default reward is 0 is there just something wrong with me?

Hi, if you mean the cost of the reward than yes.
The cost changes automatically according to the blocked website cost
On Mon, 30 Oct 2023, 20:57 Adrian English Lord Of The Manor, <
@.***> wrote:

Is the default reward is 0 is there just something wrong with me?
—
Reply to this email directly, view it on GitHub
https://github.com/ofekmiz/Habitica-Pomodoro-SiteKeeper/issues/116#issuecomment-1785856921,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJOEOFWWT7MSXJLIYQGHFFLYB72CTAVCNFSM6AAAAAAUP7TC76VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBVHA2TMOJSGE
.
You are receiving this because you commented.Message ID:
@.***>


Allow hitting global renderPending

We should support render methods returning undefined, which will then cause things to hit the global renderPending, as in base Found.
This should be the default when specifying only Component and no render.
This might require/work best with removing prerender and just passing in an initialRender property to render to signify initial renders (when throwing Found errors would be allowed).

This would accomplish something similar to #29.

Thanks for the quick response!

Depends on #45


PlatformException(FlutterSegmentException) is thrown on Segment.track in iOS

We were trying phyllo_connect: 0.3.2 version in our flutter project But it is throwing this error in iOS.
Our Flutter project depends on flutter_segment.
However, phyllo_connect: 0.3.1 version works perfectly fine.
On digging it a bit we figured out PhylloConnect with dependency 0.3.1 in Podfile is causing this issue in iOS

Changes between 0.3.1 & 0.3.0 (PhylloConnect): https://github.com/getphyllo/phyllo-connect-ios/compare/0.3.0...0.3.1

See there is a class with name SegAnalytics  which is identical to the class name in segment.

phyllo_connect (flutter): https://pub.dev/packages/phyllo_connect
flutter_segment: https://pub.dev/packages/flutter_segment
PlatformException(FlutterSegmentException, +[SEGAnalytics sharedAnalytics]: unrecognized selector sent to class 0x11197fd30, (
0 Runner 0x000000010c2f57c3 -[FlutterSegmentPlugin track:result:] + 519
1 Runner 0x000000010c2f4ce2 -[FlutterSegmentPlugin handleMethodCall:result:] + 513
2 Flutter 0x00000001177e6f81 __45-[FlutterMethodChannel setMethodCallHandler:]_block_invoke + 104
3 Flutter 0x0000000117225ef2 ___ZN7flutter25PlatformMessageHandlerIos21HandlePlatformMessageENSt3__110unique_ptrINS_15PlatformMessageENS1_14default_deleteIS3_EEEE_block_invoke + 86
4 libdispatch.dylib 0x00007ff80013b7fb _dispatch_call_block_and_release + 12
5 libdispatch.dylib 0x00007ff80013ca3a _dispatch_client_callout + 8
6 libdispatch.dylib 0x00007ff80014c32c _dispatch_main_queue_drain + 1338
7 libdispatch.dylib <…>

@himanshugarg08 Hey , thanks for reporting we are looking into this , will keep you update here .

@himanshugarg08 i have released beta version of ios sdk : 0.3.3-beta3 (pod 'PhylloConnect','0.3.3-beta3') . as well as i have used this beta version in flutter beta package here is version : phyllo_connect: ^0.3.4-beta.1
you can use "phyllo_connect: ^0.3.4-beta.1" for flutter.
Let me know how its working for you . please give me comment here . you can reach out me here if you have any query .


Multiple collections missing newlines

When there are multiple collections, there should be newlines between them.

I think I fixed this one.  Please check.


Overhaul the style

This PR overhauls the entire style-system to not ship github primer anymore. Thus reducing the bundle size.
Additionally we now added dark-mode support that automatically enables, based on the users preference. A nice side-effect of this overhaul is that we've now added support for the various VitePress markdown extensions, those together with GFM should ease the process of writing content (i.e. the "Note" or "Important" blocks from github are now available :tada:). Furthermore math-mode and footnotes are now supported.

[!NOTE]
Before approving, please re-check that the page looks fine with the new style.

Resolves #20

Change the light mod border color of the whole ad to: #ccd4db (This is the dark color used for dividers)
I guess you meant the --borderColor-default css variable? As there exists no color #ccd4db inside the entire styles.

Increase the border-radius to sth. from 0.35rem to 0.4rem so that it matches the border-radius of the button in the ad.
Both have the same border radius, 0.25rem.

Otherwise I've implemented the changes.

Regarding the colors, yes. That's what I meant.
You are completely right. The value of the border radius is the same. However, the inner radius should be less than the outer radius (see here: https://www.30secondsofcode.org/css/s/nested-border-radius/)


[HUDI-3654] Preparations for hudi metastore.

What is the purpose of the pull request
The preparations which changes existing modules for hudi metastore.
The following commits are in https://github.com/apache/hudi/pull/5064
Brief change log

Modify the  access modifier of HoodieActiveTimeline for inheritance.
Get properties when build hoodieMetaClient for get more info.
Add HoodieMetastoreConfig

Verify this pull request
This pull request is already covered by existing tests.
Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@vinothchandar @xiarixiaoyao @XuQianJin-Stars Please take a look

LGTM
@minihippo  just a few comments,
looking forward to the implementation of  HoodieTableMetastoreClient

rebase this pr for fix CI's error


update nan to v1.7.0

Update the nan dependency to v1.7.0, so gc-stats builds on current
node.js/io.js versions.
I tested this with node.js v0.12.0, v0.10.36 and io.js v1.5.1 on Linux,
seems to work fine there (apart from a few deprecation warnings during
the build). Actively using it in a project running on io.js (v1.3.0 at
the moment) without any issues so far.
Don't know if or how well it works on Windows or MacOS, as I currently
don't have suitable environments to test that.

thanks


kubernetes v1.13.5 driver name io.rancher.longhorn not found in the list of registered CSI drivers

When I upgraded from 1.16 to 1.13.5 ， it was wrong .
driver name io.rancher.longhorn not found in the list of registered CSI drivers

Which version of Kubernetes you're using before?
If it was 1.11 or less, you need to delete the driver deployer pod to allow redeploy of Longhorn CSI drivers, since the registration mechanism changed starting Kubernetes 1.12.
You can refer to https://github.com/rancher/longhorn/issues/425#issuecomment-468864486 and later comments.
For Longhorn v0.4.0 or before, you need to follow https://github.com/rancher/longhorn/issues/425#issuecomment-468893915 instead. We've fixed the deployer for changing kubernetes version in v0.4.1.


Create retry of a subscription.

Create a way to retry a subscription.
And also removed examples/preapproval/README.md, but I wonder if it is a good idea.

LGTM 😉


running make test fails

Running ./build-infer.sh java runs with no issues, but make test yields several errors, all with the following form:
g -O2 -stdlib=libc++ -std=c++11 -g -I/Users/Jacob/Documents/dev/infer/facebook-clang-plugins/clang/install/include -D_GNU_SOURCE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -fvisibility-inlines-hidden -fno-exceptions -fno-rtti -fno-common -Woverloaded-virtual -Wcast-qual -fno-strict-aliasing -pedantic -Wno-long-long -Wall -W -Wno-unused-parameter -Wwrite-strings -Wno-uninitialized  -Wno-missing-field-initializers -Wno-vla-extension -Wno-c99-extensions -O3 -DNDEBUG -c SimplePluginASTAction.cpp -o build/SimplePluginASTAction.o
bash: g: command not found
make[1]: [build/SimplePluginASTAction.o] Error 127 (ignored)

It looks like clang - is not being prepended to these commands, causing them to fail.
I am building Infer 0.10.0 on Mac OS 10.12.3

At the moment make test fails if you're building infer for Java only. This is because the Makefile does not exclude some tests that depend on the clang parts of infer, even though it should. Right now it tries to exclude them but the logic is incomplete.
We're working on adding this workflow to our CI so that we can fix it without having it regress soon after, but in the meantime the easiest solution is to build infer with the clang analysers enabled if you plan on  running the tests.


Fix CSS glitches

This fix some little issues:

Speakers on the home are now taking the full width
Mailing list form got some margin (it was too sticky to the footer before)
The sponsors logos are responsive

sweet +1 from me
@simonmcmanus?

Looks good to me.


[FEATURE REQ] make costmanagement compatible with last sdk version.

Is your feature request related to a problem? Please describe.
I am using<EMAIL_ADDRESS>and I need to fetch cost information.
I have been looking everywhere in the sdk until I found<EMAIL_ADDRESS>I tried using it but I get:
Caused by: java.lang.SecurityException: class "com.microsoft.azure.credentials.ApplicationTokenCredentials"'s signer information does not match signer information of other classes in the same package
Describe the solution you'd like
Could a compatible version be made?
Describe alternatives you've considered
Stop using de sdk and doing all by hand/rest.
Additional context
None
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ x] Description Added
[ x] Expected solution specified

@ChenTanyi  could you help publish the cost management to the latest runtime? Thanks. //cc @yungezz

@erzads Do you use any client-runtime package lower than 1.7.0? I've tested with these two packages successfully. The test code is https://github.com/tanyi-test/azure-sdk-for-java-8764.
PS: but the request got 404 which makes it raise a null pointer exception.

@ChenTanyi I couldn't find instructions on why dependencies I should add. I am using maven and I just included the dependency<EMAIL_ADDRESS>Then, when I couldn't find cost management, I added<EMAIL_ADDRESS>

@erzads No, you shouldn't add other package unless you know about it. You can see my test code, I just add these two package together, and they work fine. If you use any client-runtime package lower than 1.7.0, please remove them or upgrade to 1.7.*

@ChenTanyi  cost management package is quite old and it's runtime need updated.

@yaohaizh cost management doesn't include runtime-authentication for ApplicationTokenCredentials, so it work fine to use the credential in azure 1.31.1, we don't need to update every package runtime due to it.

Then, the costmanagement used the old codesign signature, thus we cannot have two different signature for runtime, it is not necessary depends on runtime-authentitcaion.

The root case is that there is a lower version authentication being included in the project. I guess there is a dependency in @erzads project:
    <dependency>
      <groupId>com.microsoft.azure</groupId>
      <artifactId>azure-client-authentication</artifactId>
      <version>1.6.*</version>
    </dependency>

@erzads if there is a dependency like it, please remove it. Otherwise, please give me your dependency in pom.xml or dependency:tree.
Besides, I've publish<EMAIL_ADDRESS>with latest dependency. You can try with it, but I don't think it works if you just change such dependency.

@erzads could you try the latest bits from @ChenTanyi  to see whether this issue is fixed. Thanks.

Sorry for the delay. I ended up doing all by rest.
I added the new dependency and my app can now startup with no errors.
Please, could you point me to the documentation on how to use the cost management sdk?
I would rather use it but I have no idea how to.

I admit I am lost here, but I am trying to list each resource group cost:
costManagementManager.operations().listAsync().toBlocking().first()
I get JsonParseException the first 3 times, and then null pointer exception after that.
java.lang.NullPointerException
at com.microsoft.azure.management.costmanagement.v2018_05_31.implementation.OperationsInner$4.call(OperationsInner.java:132)
at com.microsoft.azure.management.costmanagement.v2018_05_31.implementation.OperationsInner$4.call(OperationsInner.java:129)
at rx.internal.operators.OnSubscribeConcatMap$ConcatMapSubscriber.drain(OnSubscribeConcatMap.java:259)
at rx.internal.operators.OnSubscribeConcatMap$ConcatMapSubscriber.onNext(OnSubscribeConcatMap.java:144)
at rx.internal.operators.OperatorMerge$MergeSubscriber.emitScalar(OperatorMerge.java:511)
at rx.internal.operators.OperatorMerge$MergeSubscriber.tryEmit(OperatorMerge.java:466)
at rx.internal.operators.OperatorMerge$MergeSubscriber.onNext(OperatorMerge.java:244)
at rx.internal.operators.OperatorMerge$MergeSubscriber.onNext(OperatorMerge.java:148)
at rx.internal.operators.OnSubscribeMap$MapSubscriber.onNext(OnSubscribeMap.java:77)
at retrofit2.adapter.rxjava.CallArbiter.deliverResponse(CallArbiter.java:120)
at retrofit2.adapter.rxjava.CallArbiter.emitResponse(CallArbiter.java:102)
at retrofit2.adapter.rxjava.CallExecuteOnSubscribe.call(CallExecuteOnSubscribe.java:46)
at retrofit2.adapter.rxjava.CallExecuteOnSubscribe.call(CallExecuteOnSubscribe.java:24)
at rx.Observable.unsafeSubscribe(Observable.java:10327)
at rx.internal.operators.OnSubscribeMap.call(OnSubscribeMap.java:48)
at rx.internal.operators.OnSubscribeMap.call(OnSubscribeMap.java:33)
at rx.internal.operators.OnSubscribeLift.call(OnSubscribeLift.java:48)
at rx.internal.operators.OnSubscribeLift.call(OnSubscribeLift.java:30)
at rx.Observable.unsafeSubscribe(Observable.java:10327)
at rx.internal.operators.OnSubscribeConcatMap.call(OnSubscribeConcatMap.java:94)
at rx.internal.operators.OnSubscribeConcatMap.call(OnSubscribeConcatMap.java:42)
at rx.Observable.unsafeSubscribe(Observable.java:10327)
at rx.internal.operators.OnSubscribeMap.call(OnSubscribeMap.java:48)
at rx.internal.operators.OnSubscribeMap.call(OnSubscribeMap.java:33)
at rx.Observable.unsafeSubscribe(Observable.java:10327)
at rx.internal.operators.OnSubscribeFlattenIterable.call(OnSubscribeFlattenIterable.java:66)
at rx.internal.operators.OnSubscribeFlattenIterable.call(OnSubscribeFlattenIterable.java:38)
at rx.Observable.unsafeSubscribe(Observable.java:10327)
at rx.internal.operators.OnSubscribeMap.call(OnSubscribeMap.java:48)
at rx.internal.operators.OnSubscribeMap.call(OnSubscribeMap.java:33)
at rx.Observable.subscribe(Observable.java:10423)
at rx.Observable.subscribe(Observable.java:10390)
at rx.observables.BlockingObservable.forEach(BlockingObservable.java:111)

The NPE usually is due to the response status code is 404. You can use the the following configure to check http request is OK or not.
CostManagementManager
    .configure()
    .withLogLevel(LogLevel.BODY_AND_HEADERS)
    .authenticate...

After my quick test, it seems the service not supports this apiVersion.
URL: https://management.azure.com/providers/Microsoft.CostManagement/operations?api-version=2018-05-31
Response:
{
	"error": {
		"code": "InvalidResourceType",
		"message": "The resource type 'operations' could not be found in the namespace 'Microsoft.CostManagement' for api version '2018-05-31'. The supported api-versions are '2017-10-01-preview,2018-08-31,2019-01-01,2018-08-01-preview,2018-10-01,2019-10-01'."
	}
}

but actually the api was here: https://github.com/azure/azure-rest-api-specs/blob/1c3a60403422ca8c73df7eb7c65ae2a3c90bf152/specification/cost-management/resource-manager/Microsoft.CostManagement/stable/2018-05-31/costmanagement.json#L778
I'll add service attention for it and generate a new version SDK for test.

@erzads And also, I've released costmanagement v2019_11_01, you can try it later, if there is still service error.

@ChenTanyi, @erzads. Is this still an issue? can this item be closed?

Hello @erzads - apologies for the delay in our response. At this point, we are not sure if the issue still persists at your end. we are archiving the request.
Please feel free to reopen this issue if you are blocked and need our team assistance.
Thank you for reaching out to us.


Add graph

@yasmensarhan27 could please add a graph to our code? The output graph can also be placed in the documentation to comprehend our results more easily.

To generate it using a code ??

To add a plot use matplotlib. very useful and easy to use:
https://matplotlib.org/


Update FlutterLogoDecoration

We should change "normal" to "multiply" on this line:
https://github.com/flutter/flutter/blob/master/packages/flutter/lib/src/painting/flutter_logo.dart#L333
...and update the gradient colours to 0x661A237E at top 0 and 0x001A237E at stop 1.

Is this fixed? I see multiply at the linked location.

I think I meant change multiply to normal.

Hi, is anyone working on this issue?

After a quick look it looks like multiply is the correct blend mode (since it's for the shadow). As far as the gradient colours are concerned, I'm not too sure but they seem out of place compared to the other values.
If that's not the case I'd be happy to take another look and fix the issue.

@Hixie  Do I need to change multiply to normal also which color you want to change the shades of blue used..???
Please tell me I want to fix this issue..


Add socketio example

This PR adds an example using an express.js, socket.io to turn on and off a
built-in LED on an Arduino board from the browser.
Not really sure if this is what the issue was looking for, or if it is necessarily a
solution! Would love the feedback!
refs: https://github.com/EmergingTechnologyAdvisors/node-serialport/issues/1059

Codecov Report

Merging #1081 into master will increase coverage by 0.26%.
The diff coverage is 83.47%.

@@            Coverage Diff            @@
##           master   #1081      +/-   ##
=========================================
+ Coverage   89.73%     90%   +0.26%
=========================================
  Files          15      14       -1
  Lines         633     590      -43
  Branches      105     104       -1
=========================================
- Hits          568     531      -37
+ Misses         65      59       -6

Impacted Files
Coverage Δ

lib/parsers/delimiter.js
100% <ø> (ø)
:white_check_mark:

lib/parsers/byte-length.js
100% <ø> (ø)
:white_check_mark:

lib/parsers/index.js
100% <ø> (ø)
:white_check_mark:

lib/index.js
100% <100%> (ø)
:white_check_mark:

lib/util.js
100% <100%> (ø)
:white_check_mark:

lib/parsers/readline.js
100% <100%> (ø)
:white_check_mark:

lib/bindings/linux-list.js
97.91% <100%> (ø)
:white_check_mark:

lib/bindings/unix-read.js
13.04% <13.04%> (ø)
:white_check_mark:

lib/bindings/auto-detect.js
69.23% <69.23%> (-30.77%)
:x:

lib/bindings/darwin.js
72.91% <72.91%> (ø)
:white_check_mark:

... and 10 more

Continue to review full report at Codecov.

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

Hi .. my working branch was behind master so I had to do a merge .. in doing so I may have messed up the history of this PR. Not exactly sure how to fix ... heh. Any suggestions?

Nice! Thanks for the tip! Worked beautifully :)


Tab disappear after moving Document on another

Hi, I'm trying to make docks with different views, but I have a problem: when I move "Document", its tab disappears.
View models also different, but every one of 'em inherit from the same class.

I made little example app: https://github.com/AidarGiza/AvaloniaDockIssueExample
Can someone help me figure out what I'm doin' wrong?
Windows 10 Pro 21H1
using 0.10.12 version
.NET 5.0

Try using latest version


Announce GSoC 2023 projects

Call-to-action for Typelevel projects in Google Summer of Code. Feedback welcome! :)

Checked in with Arman, this is good to merge and then I'll post on socials :)


ENH: Add auto versioning

Version support for N4BiasFieldCorrection, and a "CHANGES" file for new releases' features.

@armaneshaghi This looks good.  :+1:  Only about 50 more programs to instrument :)

Yes I did this specifically because the options have changed over time and
we needed it for Nipype. Hopefully we will do this for others as well
On Friday, January 16, 2015, Hans Johnson<EMAIL_ADDRESS>wrote:

@armaneshaghi https://github.com/armaneshaghi This looks good. [image:
:+1:] Only about 50 more programs to instrument :)
--
Reply to this email directly or view it on GitHub
https://github.com/stnava/ANTs/pull/150#issuecomment-70190384.

and please keep this PR open , I will add new ones very soon. Hopefully before the next release.


Enable cargo-deadlinks in CI

Codecov Report

Merging #66 (490e185) into master (b986424) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #66   +/-   ##
=======================================
  Coverage   65.19%   65.19%
=======================================
  Files          22       22
  Lines        2718     2718
  Branches      114      114
=======================================
  Hits         1772     1772
  Misses        945      945
  Partials        1        1

Impacted Files
Coverage Δ

src/iterator/backend.rs
84.82% <ø> (ø)

src/lib.rs
51.49% <100.00%> (ø)


temp_directory cleanup and revisit its impl

From https://github.com/numba/numba/pull/4254#discussion_r300071354
The testsuite is not cleaning up temp_directory() immediately.  We should revisit its design.

There are some tricks using atexit, for an example implementation that may be useful in such cases see here:
https://github.com/Blosc/bloscpack/blob/master/bloscpack/testutil.py#L48-L67
In very rare circumstances when debugging, one would like to keep the temporary directory, it should be possible to implement this using environment variables if need be.

Thanks for pointing that out @esc, I wasn't aware of the atexit module!
Registering a shutil.rmtree call using atexit by default, while allowing an environment variable to disable the registration, seems like a reasonable way to cover the debugging case. If others are in favor of this approach, I'm happy to submit a PR


refactor(cdk): improve performance for resize service

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows Conventional Commits
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[x] Refactoring
[ ] Code style update
[ ] Build or CI related changes
[ ] Documentation content changes

What is the current behavior?
Many events are generated every second
Does this PR introduce a breaking change?

[ ] Yes
[x] No

@waterplea hello, please review me)


Current ROS Noetic installer's target is testing repository

The target of the installer added by https://github.com/Tiryoh/ros_setup_scripts_ubuntu/pull/7 was testing repository, which should be main repository.
testing repository: https://wiki.ros.org/TestingRepository

command to update the target to main from testing
sed -e "s/ros-testing/ros/g" -e "s/^#//g" /etc/apt/sources.list.d/ros-testing.list | sudo tee /etc/apt/sources.list.d/ros-latest.list
sudo sed -e -i "s/^deb/#deb/g" /etc/apt/sources.list.d/ros-testing.list


user settings: directly send password reset email

This sends a password reset link to the user's current email
instead of redirecting them to another page.
An option to resend the link is also added.
Fixes: #6342
GIFs or Screenshots:

@rishig can you please review this?

This is open for review :)

Thanks! One more place the password thing appears is under Show/Change your API key.

@rishig is this fine now?

could you send another screencast of the interaction in settings? The first one you sent was very helpful, thanks :).

@rishig here it is :)

Actually quality of this gif is not good due to its size.

Great, thanks! I do think it should be "Check your email (Resend)" rather than the other way around, but I'll let Tim decide that when merging.
There's a minor issue of there being a flash of something when you click "Reset password" in the API section; do you know what that is?

It's just text saying "sending..."

@shubhamdhama @pragatiagrawal31 can you help review this?

Got it. Let's just use the same inline loading indicator here as well.

Okay :))

I've tested this locally and it works fine.

@pragatiagrawal31 @shubhamdhama @rishig
I tried to de-duplicate code but it didn't work out and inline styling is fixed.
If any other changes are needed can you guys tell me exactly what to do.
Thanks :))


[TASK] Test documentation examples

Goal
Test the examples in the docs with the new type inference compiler.

Closed with #66


wrong IP address in url

With the latest version 1.10.0 the displayed ip address in the url (in my case <IP_ADDRESS>) is wrong:

Correct ip address should be <IP_ADDRESS>:

is there is only a single IP address and QR code printed by ezshare?
is the behaviour different in v1.9?

is there is only a single IP address and QR code printed by ezshare?

omg i'm such a fool :/
Scrolled up and there is indeed a second QR Code with the correct URL.
Any chance to implement a feature to choose which adapter to use?


jdk8_hs_sanity.external_x86-64_linux_tomee hung

Describe the bug
jdk8_hs_sanity.external_x86-64_linux_tomee hung Downloading JBoss repo
https://ci.adoptopenjdk.net/job/Test_openjdk8_hs_sanity.external_x86-64_linux_tomee/463/consoleFull
23:58:42  [INFO] Downloaded from central: https://repo.maven.apache.org/maven2/org/apache/maven/doxia/doxia-module-xdoc/1.0/doxia-module-xdoc-1.0.jar (28 kB at 78 kB/s)
23:58:42  [INFO] Downloaded from central: https://repo.maven.apache.org/maven2/org/jdom/jdom/1.1/jdom-1.1.jar (153 kB at 415 kB/s)
23:58:43  [INFO] Downloading from apache: https://repository.apache.org/content/repositories/releases/org/jboss/shrinkwrap/resolver/shrinkwrap-resolver-bom/3.1.4/shrinkwrap-resolver-bom-3.1.4.pom
23:58:43  [INFO] Downloading from JBoss Repository: https://repository.jboss.org/nexus/content/groups/public-jboss/org/jboss/shrinkwrap/resolver/shrinkwrap-resolver-bom/3.1.4/shrinkwrap-resolver-bom-3.1.4.pom
Cancelling nested steps due to timeout
09:55:57  Sending interrupt signal to process
09:56:16  /bin/sh: line 5: 23270 Terminated              /home/jenkins/workspace/Test_openjdk8_hs_sanity.external_x86-64_linux_tomee/openjdk-tests/TKG/../external/external.sh --run --tag nightly --version 8 --impl hotspot --dir tomee --reportdst "/home/jenkins/workspace/Test_openjdk8_hs_sanity.external_x86-64_linux_tomee/openjdk-tests/TKG/../TKG/test_output_16019386977872/tomee_test_0" --docker_args "-v /home/jenkins/workspace/Test_openjdk8_hs_sanity.external_x86-64_linux_tomee/openjdkbinary/j2sdk-image:/opt/java/openjdk"
09:56:16
09:56:16  tomee_test_0_FAILED

Looking at the job history the tomee testcase rarely completes within the 10hr Timeout

Close in favour of epic #2057


Aobut the interpretability

I use this model,and it makes a better effects,so i really wonder why it works on the tab-data.Do i have any methods?

Sorry for the late reply. Currently, we do not have a convincing answer to the question. It definitely requires more research to understand the core reasons of why FT-Transformer (which is "this model", right?) performs the way it does.


global ==> root

Please name a source for this change! https://github.com/webpack/webpack/blob/6afc397/lib/ExternalModule.js#L78-L95 contradicts the use of "root".

Closing... @jingsam feel free to re-open if you can name a source for your change as @simon04 mentioned.


StressChaos extension for more stress-ng options

What problem does this PR solve?
metions issue #144 , a experimental  intensify about how to make use of hundreds stress-ng options .

It's not well tested, be careful pls
chaos-mesh compile is a long job, be patient

What is changed and how does it work?
Check List
Tests

Unit test
E2E test
Manual test (add detailed scripts or steps below)
No code

Code changes

Has Go code change
Has CI related scripts change
Has Terraform scripts changeg

Side effects

Breaking backward compatibility

Related changes

Need to update the documentation

Does this PR introduce a user-facing change?:

NONE

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

@huangwei2013 Thanks for your contribution, please sign CLA first.

@cwen0 @fewdan please take a look at this CI

@huangwei2013
Sorry, the code you submitted caused the CI failed.  It seems that you have added a new "Options" field in stresschaos, but there is no such field in "stresschaos_types_test.go". So, can you modify "stresschaos_types_test.go" to make the code pass CI test? Thank you.

Codecov Report

Merging #515 into master will decrease coverage by 5.38%.
The diff coverage is 39.19%.

@@            Coverage Diff             @@
##           master     #515      +/-   ##
==========================================
- Coverage   61.39%   56.00%   -5.39%
==========================================
  Files          64       66       +2
  Lines        4069     4262     +193
==========================================
- Hits         2498     2387     -111
- Misses       1383     1705     +322
+ Partials      188      170      -18

Impacted Files
Coverage Δ

api/v1alpha1/kernelchaos_types.go
13.84% <0.00%> (+9.40%)
:arrow_up:

api/v1alpha1/timechaos_types.go
17.14% <0.00%> (+7.14%)
:arrow_up:

controllers/kernelchaos_controller.go
0.00% <0.00%> (ø)

controllers/podchaos/types.go
65.95% <0.00%> (ø)

controllers/stresschaos_controller.go
0.00% <0.00%> (ø)

pkg/chaosdaemon/netem_linux.go
26.66% <0.00%> (ø)

pkg/chaosdaemon/netlink_linux.go
0.00% <ø> (ø)

pkg/chaosdaemon/stress_server_linux.go
0.00% <0.00%> (ø)

pkg/utils/chaosdaemon.go
51.21% <ø> (ø)

pkg/webhook/config/config.go
17.14% <ø> (-72.86%)
:arrow_down:

... and 47 more

Continue to review full report at Codecov.

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

/merge


Environment Variables

Hello,
I'm currently implementing a web application with your framework. I'm also using the UI5 Tooling.
In my app, I need to access an authentication service and, since the credentials for that authentication service change whether I'm in a development or production environment, I need to use an environment variable in one of my controllers. However, I could not find any documentation about how I can use environment variables inside my app. Normaly, in any Node.js application, I would use dotenv, but since process is not defined in an OpenUI5 application, it's not possible for me to do so.
So, my question is this: how can I define and then use environment variables in my OpenUI5 application? I know I can define them in the shell I use to run ui5 serve, but how can I then access them in my application?
Thank you!
Math

Hi @mathlizee ,
thanks for opening this issue. This is not an issue of UI5 however I can give you a recommendation how you can achieve your scenario.
Typically, you have following setup:

The UI5 application where you have a data source maintained in the manifest.json pointing to a Service-URL. This Service-URL should be pointing to a location which is the same as the host of the UI5 application e.g. /myservice/service1.
A proxy which has two configurations:
2.1 Dev Setup: The proxy redirects the traffic for urls containing /myservice/service1 with user/credentials A
2.2 Prod Setup: The proxy redirects the traffic for urls containing /myservice/service1 with user/credentials B

When you start the application, you either start the proxy with devsetup or prod setup.
Further information:

https://github.com/petermuessig/ui5-ecosystem-showcase/tree/master/packages/ui5-middleware-simpleproxy
https://blogs.sap.com/2020/04/15/ui5ers-buzz-51-ui5-tooling-2.0-how-to-develop-and-run-sap-fiori-elements-locally/ (Step 5: Setting up a proxy to tunnel the Northwind OData Service)

Closing this

Closing this

Hi, is there no other way to pass variables without proxy? If it's not the issue of UI5 then where's the problem?

Hi @maaaaagda ,
using env variables means when you have the web server runtime under control. OpenUI5 does not provide a server runtime. All code is executed on the client side (browser runtime). With that it's not possible to use environment variables.
See https://github.com/SAP/openui5/issues/3051#issuecomment-714414402 whats the general best practice to achieve your scenario.

@flovogt But all I want to do is pass some variables during build time. Other building tools do not have such problem (webpack). I found out that in manifest.json I can define custom config and then reuse it. But I can't see have to pass it during ui5 build -a. Assuming that I have an app that needs only one flag/ build variable there's it's no point to create a proxy for it. It would be perfect if we could pass --custom-config option to this command and provide a json that would populate custom options in manifest file, so that I can use it in the application with this.getManifestEntry("/sap.ui5/config");

Hi @maaaaagda ,
the manifest.json is an artefact which is available on the runtime. Which also means all content is exposed to the public, so no security related content, like passwords etc. should be stored inside the manifest.json.
When you use the UI5 tooling there is another configuration artefact available, the ui5.yaml. This artefact can be used to store build specific configurations which are not available to the public during the runtime.
There are also many community projects available extending the UI5 tooling. See ui5-ecosystem-showcase.

@maaaaagda see for example the ui5-middleware-stringreplacer in the ecosystem repo that @flovogt pointed out.

@flovogt @codeworrior I think this string replacer as a task will work for me, thanks!


Client Closes only when creating Soul Shard on server.

Creating base and applying diamond creates Soul Shard in stand-alone forge world, but when doing same process in my hosted forge server, my client closes as soon as the diamond is placed. Multicraft server console does not so any errors logged, only stating Jokerz_Assassin left the game. Any ideas what I can fix?

Can you provide the log file and let us know what version of Minecraft you are playing

closing due to inactivity


Feedback : Call REST API with PUT request in ADF pipeline which involves dynamic passing of each source row to the REST API URL.

Document Enhancement Proposal: How to use REST API with PUT requests in ADF where source needs to be passed to the REST API URL.
Documentation:  https://learn.microsoft.com/en-us/azure/data-factory/connector-rest?tabs=data-factory
Overview
This request provides a step-by-step by explanation on using REST API with PUT requests where each source row needs to be passed to the REST API URL.
There is no proper documentation in ADF to use PUT request as sink in ADF which involves the dynamic passing of each source row to the REST API URL. Even though, we use Lookup and For-each loop as a workaround, this takes lot of time as it involves iterations.
Enhancement Details:
This guide will allow users to:
REST API with PUT requests if the source is from Azure IR.
Steps:

Prerequisites:
First, have a REST API with PUT request which involves dynamic passing of each source row to the REST API URL.

Steps to handle the above scenario:
We can use ADF Dataflows for this scenario. For sample, as the URL requires the  user_id  from the source, give the Base URL as  https://api.interfaceurl.com/users/  in the REST linked service.
In the dataflow source, give the source dataset. Convert the required  user_id  to string type using a derived column transformation and store in a new column as shown below.

Now, use  External call  transformation to call the REST API. In this transformation, give the REST API linked service with  PUT  method and set the above created column as  Row relative URL.

To not include this extra column, check the  Skip writing row relative URL  option. We can give any JSON dataset for the sink.
For each  user_id  in the source, the Relative URL from this column will be appended to the Base URL and REST API will be called with PUT method.

Conclusion
By following this guide, users can call REST API with PUT requests where source needs to be passed to the REST API URL.
Reference:
azure data factory - ADF REST API Put requests - Stack Overflow
This Enhancement request aims to improve user experience and security by providing clear documentation and example scripts to call REST API with PUT requests where source needs to be passed to the REST API URL.
title: Copy and transform data from and to a REST endpoint
author: jianleishen
ms.subservice: data-movement
ms.custom: synapse
ms.topic: conceptual
ms.date: 08/29/2024
ms.author: makromer

@GovindulaRakesh
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@pesalapavan we no longer work on Github issues on this repo. We need to pause on assigning them to content teams.


App Bar implementation

Description
This PR implements the App Bar for the napari hub. The styling is very similar to https://github.com/napari/napari.github.io/pull/122, using the same 2-column and 3-column layout for styling the app bar.
Changes

Minor fixes: ESlint, package.json scripts, and making the viewport responsive
Added new packages:

react-use: Library with a bunch of useful utility hooks
framer-motion: Library for React based animations

Make page scroll by default

The page scroll bar takes about 20px of space.
This will cause a visible layout shift when switching between pages with scrollable and non-scrollable content, so I added a scroll bar by default

Added components with unit and snapshot tests:

Link
SearchBar
Overlay
MenuDrawer
AppBar

Demos
Slide Out Menu
https://user-images.githubusercontent.com/2176050/114610333-9ff8ca00-9c54-11eb-878c-0632460d344e.mp4
Responsive App Bar
https://user-images.githubusercontent.com/2176050/114610328-9e2f0680-9c54-11eb-9f8b-679547d725cd.mp4
Demo Code

./src/pages/index.tsx
/* eslint-disable no-param-reassign */

import clsx from 'clsx';
import { useState } from 'react';

import styles from './index.module.scss';

const COLORS = ['red', 'blue', 'green', 'purple', 'cyan'];

function shuffle(nums: string[]) {
  const result = [...nums];
  let index = nums.length - 1;

  while (index > 0) {
    const randIndex = Math.floor(Math.random() * index);
    const tmp = result[randIndex];
    result[randIndex] = result[index];
    result[index] = tmp;
    index -= 1;
  }

  return result;
}

interface BarsProps {
  className?: string;
}

function Bars({ className }: BarsProps) {
  const [colors] = useState(shuffle(COLORS));

  return (

      {colors.map((color) => (

          Hello, World!

      ))}

  );
}

export default function Home() {
  return (

      <Bars className={styles.firstColumn} />
      <Bars />
      <Bars className={styles.thirdColumn} />

  );
}

./src/pages/about.tsx
export default function About() {
  return (

      About Page!

  );
}

./src/pages/index.module.scss
.container {
  @apply justify-center items-center;

  // Grid gap
  @apply gap-napari-sm md:gap-napari-lg;

  // Change to 2 column grid layout when (screen >= 2xl)
  @apply 2xl:grid 2xl:grid-cols-napari-2-col;

  // Use 3 column layout when (screen >= 3xl)
  @apply 3xl:grid-cols-napari-3-col;
}

.box {
  @apply flex flex-auto flex-col;

  h1 {
    @apply flex flex-auto items-center justify-center;
    height: 300px;
  }
}

.firstColumn {
  @apply hidden;
  @apply 2xl:flex;
}

.thirdColumn {
  @apply hidden;
  @apply 3xl:flex;
}

lolol I don't know how, but this PR jumped from #3 to #11 somehow 😆

lolol I don't know how, but this PR jumped from #3 to #11 somehow 😆

github tracks issues with prs together, I think the recently opened issues claimed those numbers

@justinelarsen @kne42 I made a change to the PR that moves almost all of the Tailwind utilities into the component class name. I did a comparison on the the CSS output, and doing it this way generates a smaller CSS file than having everything as its own class in a CSS module. I went ahead and removed most of the CSS module files, but left a couple since there's no equivalent in Tailwind.
I think going forward, we should try to use Tailwind class names as much as possible, and reserve custom CSS for things Tailwind can't do. For example, using the :not(:last-child) selector.

@justinelarsen @kne42 I made a change to the PR that moves almost all of the Tailwind utilities into the component class name. I did a comparison on the the CSS output, and doing it this way generates a smaller CSS file than having everything as its own class in a CSS module. I went ahead and removed most of the CSS module files, but left a couple since there's no equivalent in Tailwind.
I think going forward, we should try to use Tailwind class names as much as possible, and reserve custom CSS for things Tailwind can't do. For example, using the :not(:last-child) selector.

Fine with me if it makes the files smaller. Would love to what @kne42 thinks about it.

Another big reason is that the @apply approach is technically not using Tailwind properly, but I didn't realize that until now. When using the @apply directive, it's' actually just copying the CSS to the class, so it's no different from writing the CSS by hand. When using Tailwind in the class name, we're actually reusing the styles


Remote can crash host app when using ModuleFederationPlugin

Bug report

What is the current behavior?
An incorrectly configured remote can crash its host app
If the current behavior is a bug, please provide the steps to reproduce.
https://github.com/module-federation/module-federation-examples/blob/master/basic-host-remote
On this line change the value of publicPath to "/"
app2 still runs fine by itself
app1 throws a ChunkLoadError
Not only can the error not be caught in App.js, App.js and bootstrap.js don't even run

What is the expected behavior?
Remotes should never be able to crash their host apps. The dynamic import should return a promise that can be caught if needed, not throw strange errors that take down the whole app. If a team can deploy a broken remote that takes down every live app that consumes it, it means the whole federated network is unacceptably fragile.

Other relevant information:
webpack version:  5.4.0
Node.js version:  12
Operating System:  Mac

And what do you want? public path is wrong so you get the error

what I want is to be able to protect app1 from errors in its remotes. There doesn't seem to be any way to catch the ChunkLoadError and display some sort of fallback.

Where you place try/catch, we don't have magic for errors

Also ran into this. The dynamic import promise should be rejected, since the chunk error should happen within the bounds of that promise.

/cc @sokra

ping @ScriptedAlchemy

This is a webpack level problem of requiring wrong chunks. You'd need to wrap that in try catch to see if it can even get there. Or use the low level api in an attempt to find it ahead of time and call it.

I was trying to solve a similar issue where I wanted my container app to run even if a dependency was down.
I ended up using the import() function to import my dependency. My host app's index.js looks something like this:
import('./bootstrap').catch(error => console.log('The dependency is down'));

With that your host should survive any issues with loading up the dependent service.

If you use the low level api - you can dynamically try and require the module and allow it to fail gracefully, checking ahead of time if the get request is successful.
You can also use a promise / async external or other fallback pattern in remote modules I believe.

@Rinaldo , do you have a workaround for this issue? I am facing the same.

@ScriptedAlchemy , could you provide some sample code to overcome this issue?

@ramesh-murugan-at-philips
As @ScriptedAlchemy said, you can use the advanced api for dynamic remotes to have more control over script loading.
I adapted / simplified the following  from the dynamic-remotes example
const loadScope = (url, scope) => {
  const element = document.createElement('script');
  const promise = new Promise((resolve, reject) => {
    element.src = url
    element.type = 'text/javascript'
    element.async = true
    element.onload = () => resolve(window[scope])
    element.onerror = reject
  })
  document.head.appendChild(element)
  promise.finally(() => document.head.removeChild(element))
  return promise
}

const loadModule = async (url, scope, module) => {
  try {
    const container = await loadScope(url, scope)
    await __webpack_init_sharing__('default')
    await container.init(__webpack_share_scopes__.default)
    const factory = await container.get(module)
    return factory()
  } catch (error) {
    console.error('Error loading module:', error)
    throw error
  }
}

const RemoteButton = React.lazy(() => loadModule(
  'http://localhost:3002/remoteEntry.js',
  'app2',
  './Button'
))

Try using it in the basic-host-remote example (app1/src/App.js). It catches the ChunkLoadError I was having issues with.
I still don't think we should have to drop down to the low level api to get proper error handling though.

@Rinaldo, The above works with lazily loading React components.  How would you deal with importing a utility functions to use in a react component from a federated module?

@kevinschaffter, the loadModule function returns a promise for a module, and a module is just an object with keys corresponding to the defined exports.
If you have a utils file in a remote like this:
export const sayHello = () => console.log("hello")

export default () => console.log("I'm the default")

Then you can load and run those functions like so:
loadModule(
    'http://localhost:3002/remoteEntry.js',
    'app2',
    './utils'
)
.then(module => {
    module.sayHello() // "hello"
    module.default() // "I'm the default"
})

For using the utils in a React component you could stick the above in an onClick but you probably want to use useState + useEffect to fetch the module once when the component mounts and then be able to use the functions synchronously

So what is the final outcome of this problem?

@Rinaldo  Thanks a lot for your workaround suggestion. I tried it for our case and it works great.
I agree this should be built into the core API of the Module Federation plugin. Not having discoverability and custom error handling is a huge disadvantage for enterprise apps. Our product should still be usable even if a remote is completely down or fails to load correctly. One of our remotes is also in an earlier development stage and therefore the remoteEntry.js doesn't even exist on some environments.
One slight addition: I modified the loadScope method to return the promise returned by finally. Otherwise any errors in the script loading lead to an uncaught promise rejection (leading to the CRA error overlay). However we want to handle this error gracefully and just log it to the console.
const loadScope = (url, scope) => {
  …
  document.head.appendChild(element)
  return promise.finally(() => document.head.removeChild(element))
}

@ramesh-murugan-at-philips
As @ScriptedAlchemy said, you can use the advanced api for dynamic remotes to have more control over script loading.
I adapted / simplified the following from the dynamic-remotes example
const loadScope = (url, scope) => {
  const element = document.createElement('script');
  const promise = new Promise((resolve, reject) => {
    element.src = url
    element.type = 'text/javascript'
    element.async = true
    element.onload = () => resolve(window[scope])
    element.onerror = reject
  })
  document.head.appendChild(element)
  promise.finally(() => document.head.removeChild(element))
  return promise
}

const loadModule = async (url, scope, module) => {
  try {
    const container = await loadScope(url, scope)
    await __webpack_init_sharing__('default')
    await container.init(__webpack_share_scopes__.default)
    const factory = await container.get(module)
    return factory()
  } catch (error) {
    console.error('Error loading module:', error)
    throw error
  }
}

const RemoteButton = React.lazy(() => loadModule(
  'http://localhost:3002/remoteEntry.js',
  'app2',
  './Button'
))

Try using it in the basic-host-remote example (app1/src/App.js). It catches the ChunkLoadError I was having issues with.
I still don't think we should have to drop down to the low level api to get proper error handling though.

How would this be used in angular?


Existing GetIterator usages disturbed

The spec proposal takes away the previous form of GetIterator which takes a method as input. However, the main spec uses this in a few cases, e.g., IterableToList. Maybe make the kind a mandatory second argument, and preserve the third optional argument?

I believe my thinking was that the optional method argument to GetIterator was strange and that I could improve things with some refactoring of the call sites. That may have been unnecessarily ambitious.

Sorry, filed too quickly; I see there's actually a note explaining all of this. I don't think it's too ambitious; this is a small helper with not too many callsites. It's fine either way.


refactor: ModuleProgram uses unique pointers

Change ModuleProgram so its Program members are unique pointers instead of shared, and are accessed as a raw pointer from the outside.

Task linked: CU-865bpj35x Refactor: unique + raw pointers in ModuleProgram and Program


Ignore active livestreams

Please follow the guide below

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

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

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

Before submitting an issue make sure you have:

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

What is the purpose of your issue?

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

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
I believe a parameter option to ignore active livestreams could be a useful feature. For example when I'm downloading a whole YouTube channel, youtube-dl would start downloading the active livestream and be stuck on it until it the broadcast has completed, which might be many hours. An ignore option for those livestreams could be useful in the cases where the user feels it's sufficient to download the livestream recording sometime later.
Since the output template option is_live exists, it might only be a matter of utilising information which is already available within youtube-dl. Meaning if is_live is True, that video is skipped/ignored if that parameter is given.

Is this issue still up for grabs? I am a student at the University of Michigan looking to contribute to an open source project for a class assignment. We are expected to work on open issues which are not being tackled at the moment. If someone is pursuing this issue, I would appreciate it if you could let me know

https://github.com/ytdl-org/youtube-dl/issues/12946#issuecomment-298299093


[#119736241] Log 5xx responses with ERROR loglevel

Adds a check for the response_status and sets the log level to ERROR for 5xx responses.

Nice :+1:


Open to search tab on initial page load

Overview
When visiting the OAR map page, have the search tab visible instead of the facilities tab
Is your feature request related to a problem? Please describe.
Currently, the page loads with the first tab (the facilities tab) visible and it is not immediately obvious what the user should do. Loading the search tab immediately showcases the search functionality.
Describe the solution you'd like
The search tab should be visible when accessing the main openapparel.org page. Consult @designmatty in regards to whether the tabs should be reordered so Search is first or if just opening to the Search tab is sufficient.
Additional Context
Currently, the number of total facilities/search results is at the bottom of the Search tab. Consult with @designmatty on where it could be placed higher up on the page so it is more visible.

Please switch the order of the tabs, Search first, then Facilities

@designmatty Do you have thoughts on where we can place the number of total facilities/search results higher on the page so it is more visible?

I think we could do one of the following:
Option A

Option B

@dboyer any preference?

We have discussed the possibility of removing that big blue bar, so I would vot for option A

@dboyer @jwalgran Currently, the facilities count shown on the search tab is the count of total facilities (NOT the count of search results), but the count on the facilities tab is the count of search results. Which number do we want to show in the tab bar? And do we want to remove the total count (on search tab) and/or the search results count (on facilities tab)?

I think we get a 2-for-1 here. It is important to engagement/marketing efforts that we prominantly show the total number of facilities when you arrive at https://openapparel.org/. I think we should connect the count in the tab bar to the current number of search results since that will also be the full count when arriving on the site with no filters. The separation between the full count and the filter count is a leftover from when we were capping search results at 500.


Have release the images(jdk8) to fix  CVE-2023-0286

See https://github.com/docker-library/faq#why-does-my-security-scanner-show-that-an-image-has-cves -> this will likely be fixed as soon as https://github.com/docker-library/official-images/pull/14094 is sorted.

How long the images(jdk8) for fix CVE-2023-0286 will release


Remove deprecated get input descriptor and get output descriptor methods from Node

Remove all deprecated descriptor access methods from node
Remove the const versions of get_input_descriptor(n) and get_output_descriptor(n)
Make all remaining descriptor methods in Node private since they are only used within Node

@rkimballn1 Mac Ut has six failed cases with "C++ exception with description "node output index is out of range" thrown in the test body."


[Task] Move proof in IotaDocument outside of metadata

Description
Move the proof section in IotaDocument from inside its metadata to the root level.
Current JSON:
"doc": {},
"meta": {
  "proof": {},
  ...
}

Proposed JSON:
"doc": {},
"meta": {},
"proof": {}

Motivation
This change would remove the special case for signing DID Documents, which stands in contrast to all other structs which have their proof at the root level, such as Credential and Presentation.
While in Rust this is handled by the SetSignature trait, this discrepancy has caused problems in the Wasm bindings and will likely continue to cause problems. This also causes problems when verifying data, because we need to know what type of data we're dealing with in order to know where its signature is located.
The simplest solution is to move the DID Document proof to align with other objects.
To-do list

[ ] Move proof in IotaDocument.
[ ] Update the IOTA DID Method Specification.

Change checklist

[ ] The feature or fix is implemented in Rust and across all bindings whereas possible.
[ ] The feature or fix has sufficient testing coverage
[ ] All tests and examples build and run locally as expected
[ ] Every piece of code has been document according to the documentation guidelines.
[ ] If conceptual documentation (mdbook) and examples highlighting the feature exist, they are properly updated.
[ ] If the feature is not currently documented, a documentation task Issue has been opened to address this.

We need to verify this change against the explorer application. It expects only the the doc and meta root keys. I assume ATM you would not be able to inspect the proof in the UI with this change.

I believe the proof field was manually removed from the JSON in the initial release of the explorer, if I'm not mistaken. Was it added back in the refactor for 0.5? Needs investigating either way, thanks for the heads-up.


[MM-7854] Revoke all sessions from all users (front)

Summary
Add a button that enables a system administrator to revoke all sessions in the system.
Ticket Link
Github: https://github.com/mattermost/mattermost-server/issues/7811
Jira: https://mattermost.atlassian.net/browse/MM-7854
Related Pull Requests

Has server changes: https://github.com/mattermost/mattermost-server/pull/11200
Has redux changes: https://github.com/mattermost/mattermost-redux/pull/866

@Willyfrog Would you please merge master into your branch and update the redux commit?

hi @hanzei what do you mean by "update the redux commit"? is there some other PR that affects this? I haven't seen any

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0cc843fc8adca350c.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-0cc843fc8adca350c

1 - Fixed.
2 - I put it into black as the original description asked, let me know if you want it back to red.
3 - changed text on english and spanish

Spinmint test server destroyed

How can I get it to be recreated?

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0a2bee0db0b83417d.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-0a2bee0db0b83417d

Thanks!
1 - Looks like the button is now cut off the screen

3 - Looks like the text changed it also for the dialog when you try to deactivate someone. Ideally this would stay as "Deactivate"

PS: You don't actually need the translations in es.json, they will be handled separately via https://translate.mattermost.com

I've already fixed number 3 on my local, sorry about that, I'm dumb and forgot I started from the deactivate modal.
Number 1 is going to take me a little longer, as css is acting weird and content goes over the wrapper for some reason, pushing everything down.

@jasonblais Is there a confirmation that a user receives after clicking the button? If so, i'm fine having it black, and then in the modal, it can be red.

There is a modal asking for confirmation, and that is red. So I guess you are fine with it being black. Thanks!

@jasonblais button should be better placed, I changed the scss which I feel is unique for this component within the admin section.

Just to verify, is this PR related to https://mattermost.atlassian.net/browse/MM-12440 (which was split off from MM-7854)?

Creating a new SpinWick test server using Mattermost Cloud.

Spinmint test server destroyed

Creating a new SpinWick test server using Mattermost Cloud.

@lindalumitchell that's correct 👍

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-3033.test.mattermost.cloud
Test Admin Account: Username: sysadmin | Password: Sys@dmin123
Test User Account: Username: user-1 | Password: User-1@123

New commit detected. SpinWick upgrade will occur after the build is successful.

Spinmint test server destroyed

Hi @hmhealey, since the PR instance is removed and the code is still not merged, I am kind of stuck here as I am trying to complete the test automation for this story. Can I go ahead and bring up the instance again until the code is merged into master?

Creating a new SpinWick test server using Mattermost Cloud.

Hey @srkgupta,
Sorry, I didn't know you still needed the test server. I didn't mean to disturb your work flow.
FYI: You can set up a test server yourself by adding the Setup Cloud Test Server label.

Thanks @hanzei. I wasn't sure about the process and the permissions. Good to know about it.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-3033.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

Spinmint test server destroyed

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-3033.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

@srkgupta What are the next steps for this PR? Should this be merged together with the tests or can this be merged now?

Hi @hanzei,
This PR is ready to be merged along with the rest referenced.

@amyblais Would you be fine including this into 5.14? The server and redux PR have already be merged and a cypress test is on his way.

I'm sorry, we are past feature complete so I would prefer not to merge this for v5.14. If the server/redux PRs will cause issues in v5.14 if this PR isn't merged, then it is ok to include this in v5.14. @hanzei

Spinmint test server destroyed


Dependabot merges PRs that do not pass CI

Looks like the CI errors noted in https://github.com/ruby-syntax-tree/syntax_tree-haml/pull/32 were introduced by https://github.com/ruby-syntax-tree/syntax_tree-haml/pull/31 being auto-merged by https://github.com/ruby-syntax-tree/syntax_tree-haml/blob/8dac2207ae3d24143b735ccfe23d8d0974c74fb6/.github/workflows/main.yml#L28-L41 despite some checks not passing:

For some reason the same CI was run twice, but once as a no-op (which passed) and once with the actual tests (which failed). I think the GitHub Actions configuration needs to be updated somehow to prevent that.

Looks like this is (partially) because pull_request_target is the wrong trigger to use. Per docs, it fully ignores the head of the PR and only runs against the base branch, so, of course will pass despite bad code in PR.
I think what we want is a trigger (or action) that will run against a synthesized merge commit instead.

Ugh... I kind of knew this might be wrong and didn't know how to fix it. Thanks for catching, I'll see what I can find. If you know how to fix this I would appreciate the help as well!

Fixed in https://github.com/ruby-syntax-tree/syntax_tree-haml/pull/38.


MKDocs automation

Before submitting a pull request, Please make sure the following:
Description
Automation of GitHub Pages release
Link to design (if applicable)
Checklist

[X] Code compiles correctly
[N/A] Relevant tests were added (unit / contract / integration)
[N/A] Relevant logs were added
[N/A] Formating and linting run locally successfully
[X] All tests passing
[N/A] UA review
[N/A] Design is documented
[N/A] Extended the README / documentation, if necessary
[N/A] Open source is approved

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


Focus Stealing

Keybase started updating automatically, the Electron app window popped up, I was wrenched from my window of choice and yanked out of my space and thrown into another desktop so that I could see your electron app window with the "Keybase is loading".
Suffice to say the thing I was doing at the time was completely ruined, and you're application is to blame.
https://en.wikipedia.org/wiki/Focus_stealing

In computing, focus stealing is a mode error produced when a program not in focus (e.g. minimized or operating in background) places a window in the foreground and redirects all keyboard input to that window. This is considered to be an annoyance or hazard to some users because the program may steal the focus while their attention is not on the computer screen, such as when typing while reading copy to the side.

The Keybase Electron application is guilty of focus stealing. Stop it.
You don't need to pop up the Keybase main window when you do an update, you've got a perfectly good menu icon. Show it up in the background and make it bounce in the dock, or highlight the menu bar icon and tell me there's an update to install just don't steal my focus. I can't even turn auto-update off, at least ask me!

OS: MacOS Mojave 10.14.1
Keybase version after update: Version 3.0.0-20190205183613+6394d03e6c

+1. This could really ruin someone's day if they were gaming. Or worse doing something like online poker or trading financial assets. If the user was in a poorly designed application when focus was robbed, it could result in said application being unusable, or even crashing, resulting in data loss.
I don't believe focus stealing is nessesary or wise. Please fix.

Further investigation suggests that this is the default behaviour of apps opening on MacOS, and that the root cause is the full electron app window opening.
Only opening it if it was already open would resolve this and prevent focus stealing

I forgot to mention that I'm running Windows 10 Pro x64, so this isn't strictly a MacOS issue.

Got hit with the Keybase update notification taking frontmost while I was watching a movie fullscreen. I poked around to see if there's a good AppKit API for detecting if something is fullscreen, but didn't find much. Bouncing the dock icon or only showing update notifications when Keybase is focused would be way preferable to focus stealing.

Other open issues also reference this behavior:
https://github.com/keybase/client/issues/3493
https://github.com/keybase/client/issues/3504


Where does gen-hie come from

Hello
Maybe I am a dummy,
but I have no idea how to use gen-hie.
I am working in a virtual box. The os in the virtual box is ubuntu.
An explanation how to make gen-hie available would be nice.
gen-hie > hie.yaml
gen-hie: command not found

Best Regards

don't worry, you need to install the executable first with cabal install implicit-hie or stack install implicit-hie


Enhancement cluster RBAC: RBAC in the explore cluster can only be limited to Cluster members

Rancher version: master-head 54d4afc

Create an rke2 node driver 1 node all roles cluster
From the explore cluster --> RBAC , we have resources like ClusterRoleBindings, ClusterRoles, RoleBindings, Roles.
These resources are also present under explore cluster --> More Resources --> Rancher --> RBAC.
The resources ClusterRoleBindings, ClusterRoles, RoleBindings, Roles can be removed from under cluster and can be maintained only under explore cluster --> More Resources --> Rancher --> RBAC.
explore cluster --> RBAC can only have Cluster members for easier access.

Are you asking us to remove ClusterRoleBindings, ClusterRoles, RoleBindings, Roles from RBAC but leave them under More Resources?

@codyrancher yes theres another RBAC under more resources which also includes these options. So, the ClusterRoleBindings, ClusterRoles, RoleBindings, Roles can be left under more resources and removed from the RBAC with cluster members to avoid redundancy :

Verified on master-head rancher server 3e44415
ClusterRoleBindings, ClusterRoles, RoleBindings, Roles are removed from cluster --> RBAC and only cluster members is available in RBAC


Stop using feature "libc/std"

Fixes #852. When using the old cargo tree resolver and writing no_std code which depends on libc, updating to 1.0.80 breaks your code.
@thomcc

If you make a PR updating the version to 1.0.83, I'll merge and cut a release.


[RFC] Mainline withClockAndReset and friends out of experimental

Type of issue: feature request

Impact: API addition (no impact on existing code)

Development Phase: proposal
Other information

Currently Chisel's multi-clock support is contained in chisel3 experimental and has been there for almost 2 years now since #509. It appears that users are using this API, including in rocket-chip. Mainlining this API into chisel3._ may make this feature more discoverable and official since the API appears to be relatively stable and usable. Thoughts?

Put in mainline, deprecate aliases in experimental. Everyone in agreement at Chisel meeting.


Pdelong42/cleanup

This is my first pass at making this code work again.  It isn't as polished as it could be, and there will certainly be more refinements later.  But I've gotten it to a point where I can render graphs again, and that's a milestone.
Naturally, use your judgement on whether you think this is worthy of merging (or if you'd rather wait until further improvements from me - but I make no promises on how quickly I can get to them).

Great work.
I guess I'm happy to hand off this project to someone as I'm not a Clojure user.
What license are you thinking to apply? If the new license is fine for the Clojure community and my copyright remains with limited years (2010-2015. You can add yourself with a new copyright starting from 2015), I can transfer the ownership to you.
What do you think?

That sounds great to me.  I would be happy to take on ownership of the
project.
The copyright idea sounds good.  If that commit comes from you, after you
merge the pull-request, but before you transfer ownership, then I think
that makes the most sense.
I'm not sure whether I would want to even retain copyright for myself, from
this point forward.  But I would need to look into what makes the most
sense, regarding what copyright law says.  I guess it can't hurt for me to
claim copyright for at-least the 2016 period.
As for the license, I was thinking of using the Eclipse Public license,
since that's what most of the rest of the Clojure community seems to be
using.  But I'm fine with keeping the New BSD license.  I'm not terribly
picky, as long as it's something that is fairly permissive and encourages
contributions from others.
Thanks again for the time and energy you've put into this already.  You
gave me something to build on, and learn from.  Just let me know what the
next steps are when you can.
Thanks,
Paul DeLong
On Sat, Dec 19, 2015 at 1:34 AM, Mitsutoshi Aoe<EMAIL_ADDRESS>wrote:

Great work.
I guess I'm happy to hand off this project to someone as I'm not a Clojure
user.
What license are you thinking to apply? If the new license is fine for the
Clojure community and my copyright remains with limited years (2010-2015.
You can add yourself with a new copyright starting from 2015), I can
transfer the ownership to you.
What do you think?
—
Reply to this email directly or view it on GitHub
https://github.com/maoe/rrd4clj/pull/2#issuecomment-165954976.

Let me know if you're still amenable to transferring ownership.  I'm still willing to take over responsibility for the project.  If you've been busy, then no worries.  I'm in no particular hurry, just give me a quick heads-up.
I've never done an ownership transfer of a repository.  Do you know how this works?  Do I need to delete my fork before you transfer ownership to me (to avoid a name collision)?

Sorry for the late response. I think I can work on this during the new year holidays. Basically I'm going to merge this PR then transfer the repository to you. This is my first transfer so I'm not sure about the questions you have.

Then the process is new to both of us. :-)
When I get a moment, I should look and see if they answer my questions in a FAQ list somewhere.  If I learn anything, I'll let you know.
By the way, sorry about my noisy commit history.  I only just yesterday learned how to squash commits (and now I see why that's such a useful feature).

It's okay.  I think it would be too much trouble at this point.  Plus I'm not crazy about modifying commit history once it's been pushed to someplace public.  So the merge can happen any time you're able.

It's okay.  I think it would be too much trouble at this point.  Plus I'm not crazy about modifying commit history once it's been pushed to someplace public.  So the merge can happen any time you're able.

I've just merged this PR and updated the copyright in the README.
Regarding the ownership transfer, the target account must not have a repository with the same name, or a fork in the same network. So you need to delete the fork first. Let me know when you've done it.

Okay, I've deleted my fork.
It will interesting to see how this goes, since I've never transferred a
repository before.

Alright. I'll give it a shot now.

The status line on the Settings page says:

This repository is being transferred to @pdelong42.

Let's see how it goes.


request help: Better documentation for standalone mode configuration

Issue description
The documentation for standalone mode is very few, it doesn't cover many scenarios, like host match, plugins, people can only make trials with lots of failures. It is time-consuming. Please complete the document for this.
Environment

apisix version (cmd: apisix version):
OS:

@apache/apisix-committers Anyone wants to do this?

Just waiting for this


Save, user changed, view distance in db

If we change distance in game via /view X it gets applied only for current session. Allow us to save that value in db.

Maybe could be stored in some other kind of preferences/settings file (or would user settings also go in the same database?)


Control heater from ECU

Allow ECU to enable/disable heater control

A strategy to do this without ECU commands (e.g. "standalone mode") would also be worthwhile.
Can we measure ESR and estimate EGT - does it have any resolution <565*C?

I can test what the ESR looks like with the heater disconnected.

"standalone mode" tracked in #19


[chore] fix some test flakiness

waitForTimeout is inherently flaky and we should avoid using it as much as possible
page.textContent is discouraged: https://playwright.dev/docs/api/class-page#page-text-content
as is page.click: https://playwright.dev/docs/api/class-page#page-click
The window.now cache invalidation basically didn't work at all. We were just accidentally triggering cache invalidation a lot of the time because the tests ran slowly. If you had a bigger cache invalidation window the test would fail somewhat consistently

There's something funny going on that I haven't been able to figure out yet. I put this in draft and added tons of debug to reproduce it on the CI. I wouldn't bother taking a look at it yet. If I can figure out the issue I might just send the fix in a new PR as I'm making a bit of a mess of this one


Have simple code GPIO input/output led ?

Hello
i try codeing gpio input/output for STC8G1K08A
but i can't send output port like arduino
#include "fw_hal.h"

void GPIO_Init()
{
    GPIO_P3_SetMode(GPIO_Pin_0, GPIO_Mode_Output_PP);
}

void main()
{
    GPIO_Init();
    SYS_SetClock();
    while (1)
    {
        SYS_Delay(10);
    }
}

can add simple basic code GPIO input/output led ?

thankyou

It's similar while the syntax/methods are different, in you code
void main()
{
    GPIO_Init();
    SYS_SetClock(); // Not needed if you just want to drive a led, wrong trim values may break the MCU from running.
    while (1)
    {
        // You need to add some code here to set the pin to output high/low voltage level to make the led blink, e.g. P30 = SET
        SYS_Delay(10);
    }
}

I have added a simple led-blink example that run on stc8g1k08a sop8.


feat: export raw kubeconfig from cluster resource

Resolves #14

Current dependencies on/for this PR:

main

PR #16   👈

This comment was auto-generated by Graphite.

I've built and tested it locally, works like intended 👍

🙏🏻


Added rollla bundle files

Hey @samuelxyz, I opened up this PR to try to add some files for easier set up for Rollla. Let me know if you'd like to include them or not and if you'd like for me to also update the README for usage (or not).

Yo thanks for this! I think it'd be great to include these.
I'm not really familiar with the different setup methods, so I'll defer to you to add instructions. I'd prefer they be in a dedicated readme in the rollla folder, to keep the main readme focused on just the pure layout properties.
ROLLLA ROLLLA ROLLLA

Makes sense to me! I'll push up some readme changes inside the rollla folder and have you review them prior to merge.
ROLLLA ROLLLA ROLLLA


Update nav and other misc tweaks

This PR renames About Studio to Core Concepts in the nav and closes https://github.com/livepeer/livepeer-studio-docs/issues/131, https://github.com/livepeer/livepeer-studio-docs/issues/132, https://github.com/livepeer/livepeer-studio-docs/issues/133, https://github.com/livepeer/livepeer-studio-docs/issues/134

Closing in favor of #138


migrator: update type name in constructor call if the type becomes nested. rdar://40042252

@swift-ci please smoke test and merge


Crash Report in Engineers Life

OTG Plugin or Mod

[x ] OTG Mod for Forge
[ ] OTG Plugin for Spigot

OTG Version

[x ] 1.12.2_v8_r1
[ ] 1.12.2_v7_DEV_r26
[ ] 1.12.2-v6
[ ] 1.12.2-v5
[ ] 1.11.2-v8
[ ] 1.11.2-v9
[ ] 1.10.2-v21
[ ] Other - specify here

Forge or Spigot Versions
forge-<IP_ADDRESS>47
Server/Client Log
VanillaFix: https://paste.dimdev.org/ojiwosigig.mccrash
What other plugins are you running?
EngineersLife EL 1.8
Description
Minecraft Crashes Upon first generating, VanillaFix catches it

My guess is you have some non-standard characters in your world name

Hey guys, thanks for the feedback and sorry for the late reply. Unfortunately I can't open the link, feel free to reopen this issue if you find it's a technical problem/bug, thanks!


[AUTOPATCHER] Patches CVE-2022-0443 for vim

        Patches CVE-2022-0443 for vim.
        This patch and PR was automatically generated and needs human verification before merging.

https://github.com/microsoft/CBL-Mariner/pull/2127


[WIP] Convert AST to use Erlang records and Elixir structs

Mainly for comparison/discussion with @bruce about PR #18

Some initial thoughts:
I'd put the AST-related modules/structs under a common module name (eg, GraphQL.AST) to more clearly break them away from the rest of the package, and probably remove the kind field (since the struct knows what it is).
I don't see a benefit to using records; Elixir structs could be created in build_ast_node via, eg:
%% Expecting `Node` to be a map, as in #18
build_ast_node(Type, Node, Loc) ->
  Node#{'__struct__' => list_to_atom("Elixir.GraphQL.AST." ++ atom_to_list(Type)), loc => Loc}.

(I'm experimenting with these ideas in another implementation, too, that's a more direct port of the reference implementation -- since I need to work through the concepts quickly for a project, and that's the most direct path. My hope is to get that to a point where I can open it that up sometime this weekend and we can do some more cross-polination -- in a concrete vs abstract manner.)

Agreed @bruce would love to chat about project structure, have been pondering the best way to go with this, but don't have enough of a sense of the Elixir conventions yet. My thinking was to try and follow the graphql-js reference implementation structure.

The kind field was there only for compatibility with the reference impl.
Also the plan is to convert the maps to structs like you describe, took a while to realise that the Elixir. prefix was required for the __struct__ trick to work.
Look forward to seeing what you come up with!

Closing this as plain maps are just fine


version 0.0.14: Implements ToolBar, relevant ToolbarItems, and MacosSearchField

Adds a new Toolbar widget, along with toolbar-appropriate versions of current widgets (ToolbarItems), and a new MacosSearchField widget.
ToolBar
A ToolBar is now provided as a toolbar property of a MacosScaffold. A toolbar provides convenient access to frequently used commands and features (toolbar items). Routes can have different toolbars.
Live demo:
https://user-images.githubusercontent.com/9117427/165895933-89c09952-1221-47ff-86fc-095bffa3f524.mp4

The toolbar appears below the title bar (if present) of the macOS app or integrates with it, by using its title property.
Breaking change: the title bar (TitleBar) should now be set via the titlebar property of MacosWindow (was previously a property of MacosScaffold). Reasoning behind this here. If a title bar is present, user needs to do a small modification to the Swift code of the app, documented in the README > Modern Window Look.
ToolbarItems
Toolbar items include ToolBarIconButton, ToolBarPulldownButton, and ToolBarSpacer widgets, and are provided via the items property. These are essentially extensions of current widgets that are styled appropriately for use in a toolbar.
A simple example:
ToolBar(
  title: const Text('Untitled Document'),
  titleWidth: 200.0,
  leading: MacosBackButton(
    onPressed: () => debugPrint('click'),
    fillColor: Colors.transparent,
  ),
  actions: [
    ToolBarIconButton(
      label: "Add",
      icon: const MacosIcon(
        CupertinoIcons.add_circled,
      ),
      onPressed: () => debugPrint("Add..."),
      showLabel: true,
    ),
    const ToolBarSpacer(),
    ToolBarIconButton(
      label: "Delete",
      icon: const MacosIcon(
        CupertinoIcons.trash,
      ),
      onPressed: () => debugPrint("Delete"),
      showLabel: false,
    ),
    ToolBarPullDownButton(
      label: "Actions",
      icon: CupertinoIcons.ellipsis_circle,
      items: [
        MacosPulldownMenuItem(
          label: "New Folder",
          title: const Text("New Folder"),
          onTap: () => debugPrint("Creating new folder..."),
        ),
        MacosPulldownMenuItem(
          label: "Open",
          title: const Text("Open"),
          onTap: () => debugPrint("Opening..."),
        ),
      ],
    ),
  ]
),

, which builds this toolbar:

When the toolbar items exceed the available toolbar width, a double chevron (>>) ToolbarOverflowButton appears, that opens a submenu with the overflowed actions. This submenu is right-aligned to the window (can't exceed window width due to Flutter limitation). If a pulldown button is included here, it also opens a second submenu with its actions on hover:

This was the most difficult part of the implementation, making sure the menus appear in the correct positions, the overflow button shows up when resizing, mouse events are followed properly, etc. A lot of credit to @bdlukaa, as much of this was derived from fluent_ui's CommandBar implementation 🙌.
Toolbar examples:
It is now possible to use this to build toolbars similar to the ones found on other macOS apps. Some examples:

With icon buttons (no labels):

With icon buttons and labels:

With a pulldown button open:

With title bar above:

Without title and leading widgets:

Further development will include:

Implementing ToolbarToggleButton, ToolbarPopupButton, ToolbarSearchField (which can also be hidden behind a search icon button).
Adding a ToolbarTheme, to enable more user customizations.
In the overflow menu, fix moving the mouse from a normal overflow item to a overflow submenu item.
Add tests.

Will be opening issues for the above to track them.
MacosSearchField
A search field is a style of text field optimized for performing text-based searches in a large collection of values.

Here's an example of how to create a search field:
MacosSearchField(
  placeholder: 'Search...',
)

It is essentially a wrapper around the MacosTextField. I will be extending this to make it available for the toolbar as well, where it is usually found.
Misc
Minor fixes and extensions:

Fixed issues with the MacosPulldownTheme and MacosPopupTheme.
Added a new iconColor property to MacosPulldownTheme.
Added the ability to right-align the MacosPulldownMenu via a flag.
Updated the example pages with a new Toolbar page and a slightly changed Buttons page.

Pre-launch Checklist

[✅] I have run dartfmt on all changed files
[✅] I have incremented the package version as appropriate and updated CHANGELOG.md with my changes
[✅] I have added/updated relevant documentation
[✅] I have run "optimize/organize imports" on all changed files
[✅] I have addressed all analyzer warnings as best I could

@whiplashoo does this have any impact on #84? From browsing the component library in XCode, there doesn't appear to be a NavigationBar.

I also don't see a Navigation bar in the Sketch library too. The Figma design can be easily replicated with the current ToolBar features (with a smaller height and a few changes), so I guess we don't need a separate widget.

I also don't see a Navigation bar in the Sketch library too. The Figma design can be easily replicated with the current ToolBar features (with a smaller height and a few changes), so I guess we don't need a separate widget.

In that case I will close that issue.

LGTM


Fix travis-ci

All tests time out and thereby fail.
https://travis-ci.org/elnappo/ansible-role-prosody/builds

There is a problem with ansibles service module. Temporary fixed by disabling the idempotence test.


Clearify licence

I'm looking into publishing this to the AUR.
can you specify what licence this is released under.

Thanks, that sounds great!
I've just released 0.2.0 and added a LICENSE (MIT).
I'll close this, but please re-open or make a new issue if you run into any problems.


TestScheduler can miss tests when flushing

Bug Report
Current Behavior
Calling expectObservable adds tests to flushTests field, and calling toBe on the expect, marks those tests as ready. Then calling flush should go over flushTests, and execute and remove those that are ready at this point. The logic of removing is broken, and different tests get removed than should be. As a result, some tests might never be checked, and some are checked multiple times.
The problem is that TestScheduler.flush is broken. When removing items from flushTests, it uses the index of the item from before removal, so for example if there are 3 tests, [ready1, ready2, notReady], then flush() will execute ready1, then remove at index 0, so ready1. Then execute ready2, then remove at index 1, but now it's notReady that is present at index 1. So after flush(), flushTests should be [notReady], but in fact it is [ready2].
Reproduction
https://stackblitz.com/edit/rxjs-testscheduler-flush-bug
  expectObservable(cold('-x')).toBe('-x');
  expectObservable(cold('-y')).toBe('-y');
  const expectation=expectObservable(cold('-z')).toBe.bind(null,'-q');
  flush(); // Causes the check z vs q not to be checked at all.
  expectation();

Expected behavior
The expectation in the code above should be executed, and the test should fail, because 'z' is not equal to 'q'. The test of 'y' should be executed once, while it is executed twice.
Environment

RxJS version: 6.2.1

Possible Solution
Change the relevant part of TestScheduler.flush to:
  this.flushTests = this.flushTests.filter(test => {
    if (test.ready) {
      this.assertDeepEqual(test.actual, test.expected);
      return false;
    }
    return true;
  });

Additional context
I'm testing a service in an Angular application, calling also a material dialog. For some reason, I need to call flush() at some point, between subscribing to an observable (by calling expectObservable) and actually comparing its result with the expectation.

@cartant 👍 to close?

@ladyleet This is the issue for https://github.com/ReactiveX/rxjs/pull/3901. It'll be closed automatically when that's merged. However, I guess there's no reason it couldn't be closed right now.


Issues using simonpasquier/klog-gokit 2.1.0 with k8s.io/klog/v2

The README seems indicates that this should be compatible with k8s.io/klog/v2. However, when I attempt to substitute this in go.mod, I see:
vendor/istio.io/pkg/log/config.go:418:32: cannot use NewLogrAdapter(KlogScope) (type logr.Logger) as type "github.com/go-kit/kit/log".Logger in argument to klog.SetLogger:
	logr.Logger does not implement "github.com/go-kit/kit/log".Logger (missing Log method)

Removing the require override from go.mod causes the builder failure to go away.

https://github.com/kubernetes/klog/blob/v2.0.0/klog.go uses "github.com/go-logr/logr", while
https://github.com/simonpasquier/klog-gokit/blob/v2.1.0/klog.go uses "github.com/go-kit/log"
github.com/go-kit/log requires implementation of this interface:
type Logger interface {
	Log(keyvals ...interface{}) error
}

which klog-gokit does not implement.  This causes issues when using dependencies that expect klog, but have it replaced with klog-gokit


Fix JSMA bug for odd-length inputs.

When running JSMA, fix a bug where if the set of changable features
is odd, then it will crash if no solution is found instead of
returning failure.
This is triggered because JSMA assumes there are at least two pixels
that can be changed at any point in time. If there is only one, things
go bad.

I think this might not get reviewed until next week. I don't use this particular part of the codebase. @npapernot would be a good reviewer but he is on vacation until Monday.

Thanks for spotting this @carlini, there is indeed a bug in this implementation. The search domain should have at least two pixels left for the while loop to iterate.
Merging now


Performance: investigate ctx.willReadFrequently

just add ctx.willReadFrequently = true; in ComputeProbeValue_gs() seem to produce a minor speed improvement... but maybe it's now CPU-bound? maybe a context should be passed instead of an image/canvas if possible?
https://caniuse.com/?search=willReadFrequently

https://developer.chrome.com/blog/offscreen-canvas/

Note to self: tested again today, the offscreenCanvas commit is the largest performance boost; a 137x zoom image when from ~50ms to ~15ms per row in the SEM live view.

Commit where offscreenCanvas is being used: 7737a73e4ab4afcb0990a0af29d9cdf61de2ea92
Commit just before it: 8ddaf9f711995f5cb6e2d78cf6c14a549cf2dbd5
newest commit presently:<PHONE_NUMBER>3072b930b2214ae2290e2a9bf01388

1 & 3 performed the same, 2 was slowed and eventually crashes the tab when switching away and back.


Fix/viewer errors

Problem
The test bed viewer had a lot of errors, and it loaded files different than the web app, ie you had to click "load".
Solution
I loaded everything from the local files, and changed the "on change" function to just load the file immediately.
Type of change
Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)

some of this is already in #341


Oh the biomes weve gone compat

I have both mods installed but the stairs and slabs and stuff from obwg dont seem to work? ive tried adding them to the config but the config reset everytime i modify it
Neoforge-21.1.73
additionalplacements-NeoForge-1.21-2.0.8
Oh-The-Biomes-Weve-Gone-NeoForge-2.2.2

it also seems to not work with macaws? am i just doing something wrong or?

If the config is resetting it means you didn't edit it correctly, and it ended up having errors that made it un-parsable.
If a mod's blocks don't work, it means that they aren't using the vanilla base classes, and adding them to the config won't change anything.


Feature Request: Document when/if support of Azure China Cloud is planned

Is your feature request related to a problem? Please describe.
There is no roadmap / information regarding support of azure container apps  in China
Describe the solution you'd like.
Please provide an outlook on support of azure container apps in China
Describe alternatives you've considered.
n.a.
Additional context.
n.a.

China cloud is on our roadmap and planned for Q4 2022

Hello,
Is there any progress regarding container apps in China?
Thank you.

We also need container apps in China! Can you give us the current status? We must soon decide which way to go.
Thanks!

Hello again.
I just wanted to ask again if you plan to support container apps in China in near future.
Thank you.

This is done now 🎉


chore(main): release taiko-client 0.32.0

:robot: I have created a release beep boop
0.32.0 (2024-08-27)
Features

taiko-client: add optional l1.private (#17962) (9274f2d)
taiko-client: optimising statistics on proof request times (#17976) (791f44f)

Bug Fixes

taiko-client: fix some issues about calculateBaseFee (#17978) (b010efe)
taiko-client: prints logs when using privateTxMgr (#17980) (a0c3388)

Chores

taiko-client: fix lint errors (#17969) (eedec99)
taiko-client: keep env vars same with the flag name (#17964) (d08a1de)

Tests

taiko-client: introduce multi nodes hive test (#17981) (9910863)
taiko-client: introduce reorg hive test (#17965) (ab601ee)
taiko-client: open container logs and close build image logs (#17959) (b541201)
taiko-client: update HIVE test configurations (#17950) (4818274)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

taiko-client-v0.32.0
:sunflower:


clean file process to terminate if no files in a session can be matched against schema

Currently, we have a loop in pipeline.py where we loop over all files in the incoming folder and we:

match them against a file type
enrich them
degrade them
write them out as cleaned

In the process of doing this we write log messages to the system saying "enriching...blah blah blah", "degrading...blah blah blah".
However, if we have a situation where none of the files in incoming can be matched, we shouldn't continue the other steps in our loop - there's nothing to enrich, nothing to degrade - as the logs that are then written are misleading. They make it appear like steps are occurring that are not actually happening.
Can we close this loop earlier (figuratively, only potentially literally) so that if all files are duds, we don't send logs about enriching/degrading/outputting?

Conduct as part of GM pipelines


Don't save undo/redo history when aborting transactions

For now solved with a simple split of undo/redo into a "forgettable" version and non-forgettable.
Fixes #1659

!build

Redo history is still wiped out (as before), but now it doesn't put the wrong history state into the redo history. E.g. previously, you could drag a shape upwards and abort, and that upward location where you aborted would be stored in one step of redo history. Now it simply loses all redo history, which is a separate issue.


Update README.md

removes unnecessary spaces + list for code blocks (- point things)
oh and i might submit another pull request that adds a wiki. this readme is a mess lol...

This is the only place where any of the changes are visible in a side by side comparison, and the extra bullets are un-necessary in this place; it's cleaner as-is.