Firebase messaging caused iOS Unity app crash on startup (4.0.2, not always, rendering)

Hello, we installed Firebase 4.0.2 to our Unity game. And it works quite well. However, when we try to send our game for Apple review, the reviewer said our game crashed on startup. The call stack shows the app crashed somewhere in Unity Metal code, and we ended up forcing our game running on OpenGL ES3. Today we got it reproduced in house and it seems related to how Firebase invokes Unity rendering API and I would like to report it:
Unity version: 5.6.3f1 (macOS 10.12.5)
Device: iPhone5 A1429
Device log:
Aug 22 11:52:33 Sigono-de-iPhone heaven01(Foundation)[679] <Notice>: *** Assertion failure in -[UnityAppController renderingAPI], /Users/sigono/workspace/workspace/Heaven_iOS/Builds/iOS/Classes/UnityAppController+Rendering.mm:138
Aug 22 11:52:33 Sigono-de-iPhone heaven01[679] <Notice>: Uncaught exception: NSInternalInconsistencyException: [UnityAppController renderingAPI] called before [UnityAppController selectRenderingApi]

Symbolicated main thread call stack
Hardware Model:      iPhone5,2
OS Version:          iPhone OS 10.2 (14C92)
Report Version:      104

Exception Type:  EXC_CRASH (SIGABRT)
Exception Codes: 0x0000000000000000, 0x0000000000000000
Exception Note:  EXC_CORPSE_NOTIFY
Triggered by Thread:  0

Application Specific Information:
abort() called

Filtered syslog:
None found

Last Exception Backtrace:
0   CoreFoundation                	0x1cc5cdf2 __exceptionPreprocess + 126
1   libobjc.A.dylib               	0x1bebf072 objc_exception_throw + 34
2   CoreFoundation                	0x1cc5cccc +[NSException raise:format:arguments:] + 92
3   Foundation                    	0x1d55792c -[NSAssertionHandler handleFailureInMethod:object:file:lineNumber:description:] + 88
4   heaven01                      	0x000cdb04 -[UnityAppController(Rendering) renderingAPI] (UnityAppController+Rendering.mm:138)
5   heaven01                      	0x000df6b4 -[UnityAppController application:didFinishLaunchingWithOptions:] (UnityAppController.mm:247)
6   heaven01                      	0x015ed6ee firebase::messaging::AppDelegateApplicationDidFinishLaunchingWithOptions(objc_object*, objc_selector*, UIApplication*, NSDictionary*) + 398
7   UIKit                         	0x21d7a72e -[UIApplication _handleDelegateCallbacksWithOptions:isSuspended:restoreState:] + 394
8   UIKit                         	0x21f86e24 -[UIApplication _callInitializationDelegatesForMainScene:transitionContext:] + 3740
9   UIKit                         	0x21f8c442 -[UIApplication _runWithMainScene:transitionContext:completion:] + 1708
10  UIKit                         	0x21f9f12c __84-[UIApplication _handleApplicationActivationWithScene:transitionContext:completion:]_block_invoke.3141 + 36
11  UIKit                         	0x21f89828 -[UIApplication workspaceDidEndTransaction:] + 142
12  FrontBoardServices            	0x1e5529ae __FBSSERIALQUEUE_IS_CALLING_OUT_TO_A_BLOCK__ + 14
13  FrontBoardServices            	0x1e552868 -[FBSSerialQueue _performNext] + 216
14  FrontBoardServices            	0x1e552b52 -[FBSSerialQueue _performNextFromRunLoopSource] + 40
15  CoreFoundation                	0x1cc18716 __CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION__ + 8
16  CoreFoundation                	0x1cc18220 __CFRunLoopDoSources0 + 434
17  CoreFoundation                	0x1cc164f6 __CFRunLoopRun + 758
18  CoreFoundation                	0x1cb6552e CFRunLoopRunSpecific + 482
19  CoreFoundation                	0x1cb6533c CFRunLoopRunInMode + 100
20  UIKit                         	0x21d73ebc -[UIApplication _run] + 724
21  UIKit                         	0x21d6e54c UIApplicationMain + 146
22  heaven01                      	0x000c55ea main (main.mm:33)
23  libdyld.dylib                 	0x1c32f506 _dyld_process_info_notify_release + 24

Thank you very much for your help

Hi @sigono-chao3 we don't invoke Unity's rendering API directly, we do however swizzle the didFinishLaunchingWithOptions app delegate method present to setup messaging, dynamic links and invites in the call stack you've shared.
Any chance you have additional information to help us reproduce?
The assert does look very dodgy.
Aug 22 11:52:33 Sigono-de-iPhone heaven01[679] : Uncaught exception: NSInternalInconsistencyException: [UnityAppController renderingAPI] called before [UnityAppController selectRenderingApi]
Suggests that the call order is the problem here in Unity's framework.  Are you sure this isn't a known issue for Unity 5.6.3f1 on iOS?

@sigono-chao3 any luck finding the root cause of your issue?

@stewartmiles
Hello, sorry we don't have any good news here. It seems kind of random and we are still waiting for Unity about this issue. Please close this issue if you would like to. Thank you very much for your help.


DMagic/Universal Storage Modules

I noticed that the goo and science lab "wedges" for Universal Storage from the dmagic orbital science mod are in the original tech nodes. I assume this is not intentional (or hasn't been seen/fixed yet)?
Small fix for ScienceTweaks.cfg:
@PART[dmUSGoo]:FOR[Kerbalism]:NEEDS[DMagic]
{
  @TechRequired = basicScience
}
@PART[dmUSMat]:FOR[Kerbalism]:NEEDS[DMagic]
{
  @TechRequired = spaceExploration
}

Thanks, I've committed it. Will be included in next version.


Mock DB support

It could be useful if this tool could also spin up a mock API that's JS-only. This could be useful for:

experimenting with changes before affecting the real db
testing
offline development without needing to set up a local DB

Gonna close for now...this is far off.


Additional cmd args

Closes #58 and first half of #39.
Tested with Forerunner 310XT on Python 2.7.8.

Changed repair to pair.

Merged, thanks!

Thanks! I'm happy I can help :)


Expose methods for manipulating search params

Hi, do you have any plans to expose any methods for manipulating the search params that leverages the schema defined for the route? I'm thinking mainly for updating params to reflect state of a page (current page of a table, applied filters, etc).

I'm not sure I understand sorry
Could you provide an example?

Could it be just a wrapper on useRouter and $path?
The good thing about this would be that it will make sure people are using the library as intended.
It seems trivial so I am not sure if it is a good addition but I kind of understand why people could ask for it.

Just following up on this, I think it'd be an amazing addition to have this! Managing or reflecting state in the URL is a great way to create easily sharable / bookmarkable pages in applications where the user's interactions change the content of the page, think of an e-commerce page where the variant of a certain item is stored in the URL.


Possible small error in tutorial example code?

In the tutorial code,
https://github.com/quantumlib/OpenFermion/blob/master/docs/tutorials/intro_to_openfermion.ipynb, under the heading "InteractionOperator and InteractionRDM...", there is the following block of code,
`for p in range(n_orbitals):
for q in range(p + 1, n_orbitals):
kappa[p, q] = random_angles[index]
kappa[q, p] = -numpy.conjugate(random_angles[index])
index += 1
# Build the unitary rotation matrix.
difference_matrix = kappa + kappa.transpose()
rotation_matrix = scipy.linalg.expm(kappa)

# Apply the unitary.
molecular_hamiltonian.rotate_basis(rotation_matrix)`

It seems to me that the last two blocks should not be indented, i.e. should be outside the p-loop. The way it is, the bases get rotated n_orbital times, for which I don't see a reason, even though it essentially does no harm. Indeed, when I removed the indents (taking them out of the p-loop), the agreement on ground state energy gets slight better, i.e. "more" iso-spectral. Presumably repeated base rotations accumulated some error, albeit the error is still very small.
Anyway, I could be mistaken, but I thought I should ask in case no one has raised the issue.

You are right.  It shouldn't be indented.  The loops are for forming the generator matrix kappa.  Indeed we are rotating with some partial kappa every time which isn't the intended function based on the text.  Thanks for finding the issue.  Feel free to open a PR to change it.  If I don't hear back in a couple of days I'll get around to making the change over the weekend.

Glad to know I am not mistaken. As far as fixing it, I'm not hacker enough to be able to handle it (don't even know what "opening a PR" means) and will leave it to the pros. Thanks!


MediaViewer shift after pan or zoom

Not quite a bug but annoying: often when you zoom or pan a photo in the MediaViewer, there is a slight jerk in the position of the photo. This is usually easiest to replicate immediately after opening the photo, e.g.

Open the photo
Unpinch to zoom in immediately
Notice a slight shift in the position of the image immediately after zoom

Note sure if we can fix this, but worth trying.

I made a small change that I think cuts down on these shifts, but it certainly doesn't eliminate it. I'm not sure what else to do other than digging into the underlying ImageZoom library, which I'm not sure is worth it here.


Fix failing tests - [merged]

In GitLab by @wtbarnes on Feb 11, 2020, 21:43
Merges fix-the-tests -> master
This fixes a few tests that are astropy-units related. I suspect these cropped up because of the transition to astropy v4. Notably, the correction table is now a QTable with units where appropriate.

In GitLab by @markcheung on Feb 12, 2020, 10:48
approved this merge request

In GitLab by @wtbarnes on Feb 12, 2020, 11:11
added 1 commit
6a3d55cf - pin coverage below v5 to make helpers happy
Compare with previous version

In GitLab by @markcheung on Mar 23, 2020, 13:30
mentioned in commit 76839b7183c99a6b7012c5768e5f19feb0459c4a

In GitLab by @markcheung on Feb 12, 2020, 11:12
enabled an automatic merge when the pipeline for 6a3d55cfb1f54b4d4d3ccb59c0d763eb7f04dc12 succeeds

In GitLab by @markcheung on Feb 12, 2020, 11:23
merged

In GitLab by @markcheung on Feb 12, 2020, 11:12
approved this merge request

In GitLab by @markcheung on Feb 12, 2020, 11:23
mentioned in commit a03008c3a3855e9db71714a3fe7aab2abb172c5f


[FLINK-2956] [tests] Migrate integration tests for Table API.

Migrate integration tests of Table API from temp file to collect() as described in umbrella jira.

I would like to shepherd this PR. Thanks for opening pull request @gallenvara. I'll review soon.

Hi @gallenvara, I read your PR and have some comments.
First, since Flink 0.10, we dropped Java 6 support and can use omit types if the types can be inferred. So could you remove type description in that case? With removing, we can make codes more readable.
Second, to convert Scala sequence to Java list, using JavaConverters is better than JavaConversions. For example:
import scala.collection.JavaConverters._

~skipped~

val result = ds.collect()
val expected = "~~~"

TestBaseUtils.compareResultAsText(result.asJava, expected)

After importing scala.collection.JavaConverters._, we can convert Scala collection to Java collection with asJava method.

Hi, @chiwanpark . Thanks for your review. I'm not very clear about your first suggestion. Do you mean that we should remove type description when to create a new generics?
For example:
DataSource<Tuple2<Float, String>> input = env.fromElements(new Tuple2<Float, String>(1f, "Hello"))
-->
DataSource<Tuple2<Float, String>> input = env.fromElements(new Tuple2<>(1f, "Hello"))

@gallenvara Yes, exactly right.

Hi, @chiwanpark . I have modified the code and committed a new commit. Can you help me with review work?

Hi, @chiwanpark . I have modified the code and submitted a new commit. Can you help me with review work?

Okay I'll review soon. Thanks for update.

@gallenvara Sorry for late. I reviewed your changes. Except some minor comments, looks good to merge for me.

Hi, @chiwanpark . I have modified the code format and submitted a new commit.

@gallenvara Thanks for update! Looks good to merge for me. If there is no objection until tomorrow, I'll merge this.

Looks good, please go ahead @chiwanpark :+1:

Merging...

@gallenvara Because I forgot add close message in the commit, the PR should be closed manually. Could you close this PR? This change is merged. Sorry for inconvenience.


bazel: Assume default compiler is gcc

Commit Message:
It is impossible to turn on the "//bazel/gcc_build" config setting on Bazel 3.7.x, see:
https://github.com/bazelbuild/bazel/issues/12707
GCC 9, which is the default in Ubuntu 20.04, requires the -Wno-maybe-uninitialized for
@com_googlesource_quiche//:quic_core_framer_lib to compile successfully.
Mitigate by assuming the default "compiler" is GCC. Clang is properly recognized by Bazel, so it is not affected by this.
Signed-off-by: Jarno Rajahalme<EMAIL_ADDRESS>Additional Description: See https://github.com/bazelbuild/bazel/issues/12707
Risk Level: Medium, non-GCC compilers may not implement -Wno-maybe-uninitialized, and may start to fail if not recognized by Bazel.
Testing: Tested compilations with both GCC and clang, and observed that -Wno-maybe-uninitialized is present with GCC but not with clang.
Docs Changes:
Release Notes: Fix compilation with GCC by assuming it is the default compiler.

@envoyproxy/build-tools-maintainers for review of which target environments could be broken by assuming "compiler" is actually "gcc" by default. (Windows would be unaffected.)

@envoyproxy/build-tools-maintainers for review of which target environments could be broken by assuming "compiler" is actually "gcc" by default. (Windows would be unaffected.)

Mitigate by assuming the default "compiler" is GCC.

So when this is "fixed", our build is broken? Can we simply add a second condition for each of "gcc" and "compiler" so that a corrected bazel won't break us?

Mitigate by assuming the default "compiler" is GCC.

So when this is "fixed", our build is broken? Can we simply add a second condition for each of "gcc" and "compiler" so that a corrected bazel won't break us?

Mitigate by assuming the default "compiler" is GCC.

So when this is "fixed", our build is broken? Can we simply add a second condition for each of "gcc" and "compiler" so that a corrected bazel won't break us?

Don't know how that would work, might be simpler just to include -Wno-maybe-uninitialized in "//conditions:default": as well as in repository + "//bazel:gcc_build":? But then any further selects on gcc_build would need to do that as well until Bazel is fixed. So far there is only this one select on it, though.

Mitigate by assuming the default "compiler" is GCC.

So when this is "fixed", our build is broken? Can we simply add a second condition for each of "gcc" and "compiler" so that a corrected bazel won't break us?

Don't know how that would work, might be simpler just to include -Wno-maybe-uninitialized in "//conditions:default": as well as in repository + "//bazel:gcc_build":? But then any further selects on gcc_build would need to do that as well until Bazel is fixed. So far there is only this one select on it, though.

Don't know how that would work

Have the gcc_build a composite selects.config_setting_group ... of either ":compiler" or ":gcc"?

Don't know how that would work

Have the gcc_build a composite selects.config_setting_group ... of either ":compiler" or ":gcc"?

Don't know how that would work

Have the gcc_build a composite selects.config_setting_group ... of either ":compiler" or ":gcc"?

@wrowe I'm not versed enough in Bazel to know how to get from the following to what you suggest, could you elaborate a bit?
config_setting(
    name = "gcc_build",
    flag_values = {
         "@bazel_tools//tools/cpp:compiler": "compiler",
    },
)

Don't know how that would work

Have the gcc_build a composite selects.config_setting_group ... of either ":compiler" or ":gcc"?

@wrowe I'm not versed enough in Bazel to know how to get from the following to what you suggest, could you elaborate a bit?
config_setting(
    name = "gcc_build",
    flag_values = {
         "@bazel_tools//tools/cpp:compiler": "compiler",
    },
)

@wrowe Added a commit using selects.config_setting_group as requested.

Terrific, that looks right to me. Note the format failure, relevant log states;
2021-02-17T23:48:21.2730199Z Format check failed, try apply following patch to fix:
2021-02-17T23:48:21.3184164Z diff --git a/bazel/BUILD b/bazel/BUILD
2021-02-17T23:48:21.3184994Z index f0633a4cd..d349be15f 100644
2021-02-17T23:48:21.3185688Z --- a/bazel/BUILD
2021-02-17T23:48:21.3186175Z +++ b/bazel/BUILD
2021-02-17T23:48:21.3186827Z @@ -184,7 +184,10 @@ config_setting(
2021-02-17T23:48:21.3187313Z
2021-02-17T23:48:21.3187719Z  selects.config_setting_group(
2021-02-17T23:48:21.3188152Z      name = "gcc_build",
2021-02-17T23:48:21.3188859Z -    match_any = [":gcc_build_gcc", ":gcc_build_compiler"],
2021-02-17T23:48:21.3189413Z +    match_any = [
2021-02-17T23:48:21.3189841Z +        ":gcc_build_gcc",
2021-02-17T23:48:21.3190272Z +        ":gcc_build_compiler",
2021-02-17T23:48:21.3190684Z +    ],
2021-02-17T23:48:21.3191047Z  )
2021-02-17T23:48:21.3191401Z
2021-02-17T23:48:21.3191774Z  config_setting(

Appears macOS fails with this change:
error: unknown warning option '-Wno-maybe-uninitialized'; did you mean '-Wno-uninitialized'? [-Werror,-Wunknown-warning-option]

Fixed by only assuming GCC by default on linux.

This crash in looks like it is likely a flake (more common among integration tests, nothing this patch should have affected.) Will retest after verify examples is done. It looks good, @lizan or @moderation could you take a look?
2021-02-19T17:53:13.6206839Z //test/integration:cds_integration_test                                  FAILED in 26.3s
2021-02-19T17:53:13.6207625Z   /build/tmp/_bazel_envoybuild/b570b5ccd0454dc9af9f65ab1833764d/execroot/envoy/bazel-out/k8-fastbuild/testlogs/test/integration/cds_integration_test/test.log

2021-02-19T17:34:45.6440831Z [2021-02-19 17:34:35.113][20574][info][main] [source/server/drain_manager_impl.cc:70] shutting down parent after drain
2021-02-19T17:34:45.6441542Z [2021-02-19 17:34:38.025][20574][debug][main] [source/server/server.cc:202] flushing stats
2021-02-19T17:34:45.6442242Z [2021-02-19 17:34:43.029][20574][debug][main] [source/server/server.cc:202] flushing stats
2021-02-19T17:34:45.6443094Z [2021-02-19 17:34:43.317][14168][critical][assert] [test/integration/http_integration.cc:388] assert failure: 0. Details: Timed out waiting for new connection.
2021-02-19T17:34:45.6444119Z [2021-02-19 17:34:43.318][14168][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:104] Caught Aborted, suspect faulting address 0x3e900003758
2021-02-19T17:34:45.6444709Z #0 Envoy::SignalAction::sigHandler() [0x7f3f67f03d04]
2021-02-19T17:34:45.6445013Z #1 __restore_rt [0x7f3f4f320980]
2021-02-19T17:34:45.6445366Z #2 Envoy::HttpIntegrationTest::waitForNextUpstreamRequest() [0x7f3fcd82a7bd]
2021-02-19T17:34:45.6445792Z #3 Envoy::HttpIntegrationTest::waitForNextUpstreamRequest() [0x7f3fcd828eba]
2021-02-19T17:34:45.6446202Z #4 Envoy::HttpIntegrationTest::sendRequestAndWaitForResponse() [0x7f3fcd828d75]
2021-02-19T17:34:45.6446649Z #5 Envoy::HttpIntegrationTest::makeHeaderOnlyRequest() [0x7f3fcd82c613]
2021-02-19T17:34:45.6447109Z #6 Envoy::HttpIntegrationTest::testRouterHeaderOnlyRequestAndResponse() [0x7f3fcd82c7b7]
2021-02-19T17:34:45.6447441Z #7 [0x3b2962]
2021-02-19T17:34:45.6447771Z #8 testing::internal::HandleSehExceptionsInMethodIfSupported<>() [0x7f3f501adc66]
2021-02-19T17:34:45.6448194Z #9 testing::internal::HandleExceptionsInMethodIfSupported<>() [0x7f3f50192571]
2021-02-19T17:34:45.6448564Z #10 testing::Test::Run() [0x7f3f50179eb2]
2021-02-19T17:34:45.6448875Z #11 testing::TestInfo::Run() [0x7f3f5017a9c8]
2021-02-19T17:34:45.6449207Z #12 testing::TestSuite::Run() [0x7f3f5017b2a9]
2021-02-19T17:34:45.6449597Z #13 testing::internal::UnitTestImpl::RunAllTests() [0x7f3f50189056]
2021-02-19T17:34:45.6450035Z #14 testing::internal::HandleSehExceptionsInMethodIfSupported<>() [0x7f3f501b55a6]
2021-02-19T17:34:45.6450659Z #15 testing::internal::HandleExceptionsInMethodIfSupported<>() [0x7f3f501956c1]
2021-02-19T17:34:45.6451219Z #16 testing::UnitTest::Run() [0x7f3f50188aa0]
2021-02-19T17:34:45.6451697Z #17 RUN_ALL_TESTS() [0x7f3f7cedd485]
2021-02-19T17:34:45.6452175Z #18 Envoy::TestRunner::RunTests() [0x7f3f7cedc6bf]
2021-02-19T17:34:45.6452628Z #19 main [0x7f3f7ce705b7]
2021-02-19T17:34:45.6453007Z #20 __libc_start_main [0x7f3f4ef3ebf7]
2021-02-19T17:34:45.6453554Z external/bazel_tools/tools/test/collect_coverage.sh: line 131: 14168 Aborted                 (core dumped) "$@"
2021-02-19T17:34:45.6453960Z + TEST_STATUS=134
2021-02-19T17:34:45.6455150Z + touch /tmp/sandbox_base/bazel-sandbox.506fa982d9679d3c6eeab1b1d124a688b61c231e37f3485140dfb23bf122cc4f/processwrapper-sandbox/6928/execroot/envoy/bazel-out/k8-fastbuild/testlogs/test/integration/cds_integration_test/coverage.dat
2021-02-19T17:34:45.6456076Z + [[ 134 -ne 0 ]]
2021-02-19T17:34:45.6456472Z + echo --
2021-02-19T17:34:45.6456816Z --
2021-02-19T17:34:45.6457228Z + echo Coverage runner: Not collecting coverage for failed test.
2021-02-19T17:34:45.6457931Z Coverage runner: Not collecting coverage for failed test.

Handing this off to you @lizan, I don't have merge karma on this tree.


move link to separate paragraph

Same as previous PR.

Thanks Zack,
I have updated the code accordingly. :)


Feature - API additions

I am wondering if it would be possible to add to the api a hide-loading attribute with a default value of false.  This would allow for those who have their own loaders to use them.
Thanks!!

too bad, you dont like our spinner :)
kidding,
you always have the option of hiding the <rapi-doc> element itself,  use your own spinner, and show rapidoc after spec-loaded event.
We dont want to make our API surface too granular. Let me know if that helps

That could definitely work, had not thought of using the spec-loaded in that way.  Thanks for the help, and all the work on this project.

Can the loader be a .svg file that can be overwritten easily ?
I believe this is how Redoc does as mentioned in this issue https://github.com/Redocly/redoc/issues/1187
Not, suggesting that RapiDoc should follow what Redoc is doing, It just a reference point on how others are dealing in this space. Hiding the entire element seems like an overkill


[PE-D][Tester A] Inconsistency in command format: addGrade and removeGrade

Description: inconsistency in stating the command formats for addGrade and removeGrade.
Help Page's addGrade and removeGrade command format:

UG's addGrade and removeGrade command format:

There is also discrepancy in the UG's format and shortcut, and the example command provided which may confuse readers.

Labels: severity.Medium type.DocumentationBug
original: jessica2828/ped#7

Can perhaps update the user guide to be more descriptive on what the shortcut means. The help command is meant to serve as a quick reference. If the user does not know how the command works, they should clearly read the user guide instead.

I think in the format section, n/ should be changed to name/ as well

In the examples, asgn should be changed to a/


Display Light toggle makes mini-split beep but does not change display light state

Describe the bug
In Home Assistant, all of my mini-splits have a Display Light toggle. When I click it, it toggles the state inside Home Assistant, and the mini-split beeps as it does whenever you make any change, but the display light's state is unchanged.  If it was on, it remains on.
Additionally, the state of the display light toggle in Home Assistant has no correlation with the actual state of the display light on the units.
Here's the JSON for one of my mini-splits (it's huge so I didn't want to paste it inline)
Expected behavior

The toggle should turn the unit's display light  on or off as appropriate
Ideally the state of the toggle in Home Assistant should reflect the actual state of the light on the devices

Screenshots

Environment details:

Environment: VM running on Hyper-V under Windows 11
Home Assistant version installed:

Component version installed: Not sure how to check this, but I downloaded it yesterday (Jan 8th 2024) by cloning the master branch of this repo
Last know working version: n/a (I just set up HomeAssistant for the first time and have no historical context)
LG device type and model with issue: All of the following mini-splits:

6 x - LMN079HVT
1 x - LSN090HSV5
1 x - LSN120HSV5
1 x - LSN181HSV5

LG devices connected (list): compressors:

2 x - LMU300HHV
1 x - LMU240HHV

Output of HA logs
Paste the relavant output of the HA log here.
No relevant entries found in the logs (settings -> system -> logs). Is there somewhere else I should check for logs?
Additional context
Add any other context about the problem here.

This has been discussed on many other threads: some device do not support this function also if it is exposed in model_info and so is not possible to determinate when switch should not be exposed.
Just disable the control in HA to not show it.


Issue in running example of city traffic

Hello, I am using you git hub and trying to simulate smart city traffic example. however, I am getting no power model for texi can you explain it what wrong with this.

similarly for traficlights i get same error as well.

Hi! Thanks a lot for your bug report and the trace.
This case should now be correctly handled in version 0.4.1 which I just published - let me know if this works for you!

Thanks yes its working now. However, 1 more issue in while using create plots. The CSV file is only saved in result/fog_0 folder. For other configuration there are no CSV file which give error when creating plots using create plots.

Oh great thanks 👍


Can the RTMP block be used inside the sites-enabled directory?

My RTMP block works perfectly inside the /etc/nginx/nginx.conf but as soon as I add it into a file under /etc/nginx/sites-enabled directory I get error messages. My simple question can RTMP blocks be read from the sites-enabled directory? If so how do I get it to work please?

rtmp block must be put in root context.
If you put the configuration inside sites-enabled, it is included in http {} context, which will not work.
see https://github.com/arut/nginx-rtmp-module/wiki/Directives#rtmp

rtmp block must be put in root context.
If you put the configuration inside sites-enabled, it is included in http {} context, which will not work.
see https://github.com/arut/nginx-rtmp-module/wiki/Directives#rtmp

Can the sites-enabled be excluded from the HTTP block and still be picked by the nginx.conf still, or will it not work that way?

Please do not do this kind of mixing files, they are different kind, it will make things complicated and error prone.
http {
  include sites-enabled/*.conf;
}

rtmp {
  include rtmp-enabled/*.conf;
}

Sadly appending an extension to the file that holds my RTMP block and then modifying the nginx.conf file still doesn't work.
nginx: [emerg] "rtmp" directive is not allowed here in /etc/nginx/sites-enabled/livestream.rtmp:3

@firewire10000 You are redefining rtmp directive inside rtmp, that's why the error happen.
If your /etc/nginx/sites-enabled/*.rtmp files contains the following:
rtmp {
    server {
        listen 1935;
        application live {
            live on;
        }
    }
}

You should remove the rtmp directive:
    server {
        listen 1935;
        application live {
            live on;
        }
    }


Add dot product support for quantized convolution.

High level description of the submission
We added two new intrinsics in: topi/arm_cpu/tensor_intrin.py, namely

mmla4x4: compute a matrix multiplication between tile A(4,4) and tile
B(4,4)
mmla16x4: compute a matrix multiplication between tile A(rows,4) and tile
B(4,16)
Then we used those intrinsics in two separate strategies. We added the
strategies in topi/arm_cpu/conv2d_int8.py and implemented the schedules
in topi/arm_cpu/conv2d_gemm.py. In particular:
schedule_conv2d_gemm, when accelerated, packs matrix A, compute GEMM,
and unpack the resulting matrix. This uses the mmla4x4 intrinsic
schedule_conv2d_gemm_hybrid doesn't do any packing on A and C which
are in native form.  This uses the mmla16x4 intrinsic

Please note that for the limitations of tensorize we need to pad
matrix A in both cases (when dimensions are not multiple of the tiling
shape)
RFC
This PR is based on the following RFC: https://discuss.tvm.apache.org/t/rfc-accelerate-quantized-convolution-through-dot-product/7873
Change-Id: Id0d818d84ffc458c6dad7983fd350a0f3d5db395

@anijain2305 @FrozenGene Would you please have a look at this?

Can you see why ci is failing @giuseros ?

@u99127 , I am on it. It is strange that the command I ran locally didn't catch this.

Mmmm the last failure seems like a tolerance issue. Let me retrigger the CI

Hi @mbaret ,
Thanks for the review!
I addressed the comments and added compilation tests to verify the compilation flow with dot-product.

Hi @mbaret ,
Thank you for the careful review!
@FrozenGene , @anijain2305 should we merge this in?

Thanks everyone. Merged now.

Thanks @FrozenGene !


network: Fix bugs introduced in 2293e2

Fix a reference to cnci-agent in README.md
Fix $cnci-agent var in generate_cnci_cloud_image.sh

Signed-off-by: Alberto Murillo<EMAIL_ADDRESS>

LGTM: Will merge once travis checks pass.

Coverage remained the same at 54.514% when pulling 1b966ca7c79e7a09ad42547516d4423cffffb4e5 on albertomurillo:issue55 into 2293e2e8d62307896108db50f76243756a3d0e97 on 01org:master.

Coverage decreased (-0.02%) to 54.491% when pulling 68c90c776e3861a06ae6351a02ba121fa53e1f6b on albertomurillo:issue55 into 2293e2e8d62307896108db50f76243756a3d0e97 on 01org:master.


Fix documentation links to work in both source repo and documentation builds

Description

Replace symlinks to markdown files under examples/ with stub files using include directives.
Replace existing https to relative urls
Add stub file for the devcontainer instructions

Closes #1783
By Submitting this PR I confirm:

I am familiar with the Contributing Guidelines.
When the PR is ready for review, new or existing tests cover these changes.
When the PR is ready for review, the documentation is up to date with these changes.

/merge


Test to allow custom Config values

Description
This fixes #1618
Also multiple reading of the config file is mitigated to just once when db-sync starts.
We can now read a config json file once in the test class that as an init and update it with any custom values required for testing. A single example of it working is:
https://github.com/IntersectMBO/cardano-db-sync/blob/eecc3ee307d425ca29d507b81b4d692615bf85f2/cardano-chain-gen/test/Test/Cardano/Db/Mock/Unit/Conway/CommandLineArg/MigrateConsumedPruneTxOut.hs#L38-L65
Checklist

[ ] Commit sequence broadly makes sense
[ ] Commits have useful messages
[ ] New tests are added if needed and existing tests are updated
[ ] Any changes are noted in the changelog
[ ] Code is formatted with fourmolu on version <IP_ADDRESS> (which can be run with scripts/fourmolize.sh)
[ ] Self-reviewed the diff

Migrations

[ ] The pr causes a breaking change of type a,b or c
[ ] If there is a breaking change, the pr includes a database migration and/or a fix process for old values, so that upgrade is possible
[ ] Resyncing and running the migrations provided will result in the same database semantically

If there is a breaking change, especially a big one, please add a justification here. Please elaborate
more what the migration achieves, what it cannot achieve or why a migration is not possible.

LGTM, but I worry that the list of arguments (runDbSync and withFullConfig and friends) is getting a bit too long

@sgillespie I tried to pop in SyncNodeParams but that has a Show instance which is required by Logging.Configuration, so I made a placeholder instance. But then ran into the issue that SyncNodeParams is populated via Parser monad in pRunDbSyncNode so to also add the reading of the json file in that step I couldn't work out how to do.


[$250] Mentions - @ Here icon is black

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.17-1
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause-Internal Team
Slack conversation: @
Action Performed:

Go to any chat.
Type @ here.

Expected Result:
Here icon is green.
Actual Result:
Here icon is black.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~014f2330f066b567e3
Upwork Job ID:<PHONE_NUMBER>587080192
Last Price Increase: 2023-12-26
Automatic offers:

paultsimura | Contributor | 28070446

Proposal
Please re-state the problem that we are trying to solve in this issue.
In mention list for here the icon is black
What is the root cause of that problem?
In this PR we have removed the fill property for the avatar.
https://github.com/Expensify/App/pull/30905/files#diff-3e66159ec7e545cdb519881ff0194d3e2151d3d3c9a963bf2b82e46b3119a92cL75
What changes do you think we should make in order to solve the problem?
We can add the fill property with fill={theme.success} again here to make it a green icon.
https://github.com/Expensify/App/blob/de5a9435f97ca5fee0ead16fd38bdfc71da6582a/src/components/MentionSuggestions.tsx#L68-L74

This was missed and should be fixed as a follow-up of the https://github.com/Expensify/App/issues/29067

Decreasing the price as this should be very simple fix from the regression PR.
@Pujan92 can you raise the PR quickly?

If @Pujan92 is unavailable, I can make the quick PR.
Didn't bother posting a full-scale proposal because I thought it would be handled by the original PR author.

Yeah they are ooo though so I have decreased the reward.
I think that given this is a deploy blocker and you are available, I am going to assign you. This is not common so sorry to @Pujan92 that this was not handled earlier when they were on but we need to treat this with urgency

@paultsimura Can you raise the PR quickly please?

Yes, I'm on it

The PR is ready, adding the screenshots now: https://github.com/Expensify/App/pull/33634

Yeah they are ooo though so I have decreased the reward.
I think that given this is a deploy blocker and you are available, I am going to assign you. This is not common so sorry to @Pujan92 that this was not handled earlier when they were on but we need to treat this with urgency

No issues.. :)

This caused a regression: https://github.com/Expensify/App/issues/33657
Creating a follow-up PR now.

This was a deploy blocker caused by a big change in how we handle images, using expo-image package. I think that has been stated already and there was bunch of other small issues so no need to dig more into that, the checklist can be omitted in this case

@mountiny Am I eligible for partial compensation here for providing the proposal which is used?

Agree, my bad. Sorry for the ask.

no problem, one has to try

@bfitzexpensify this should be $250 to @paultsimura  please

I can step in for Ben!

paid.

Good to close?

I think so thanks!


CFE_ES_CDSRegDumpRec_t has implicit padding

Is your feature request related to a problem? Please describe.
Related to #664 - implicit padding should be avoided
Describe the solution you'd like
Eventually pack definitions will come from a common definition, with explicit padding
Describe alternatives you've considered
None
Additional context
Observed in testing Bootes
Requester Info
Jacob Hageman - NASA/GSFC
@wmoleski

Actually I noticed this as part of #909 because this structure is used for data files (and therefore should be included in the msg structs as it is part of the interface) and fixed it as part of that change.

Checked again - there is one more thing needed here.  Previous change fixed the name length to use mission-scope definition which is a multiple of 4, fixing that part of the problem, but there is still a padding due to the use of a bool value (1 byte) - so the alignment does need adjustment.


Shell integration: Explore how to handle sub-shells

I don't think we would be able to support sub-shells without a manual shell integration installation. Perhaps this should be an option for advanced users to be cwd/command aware even in sub-shells? There are other implications to this like knowing when the sub-shell starts and ends if we want to track the "tree". This would probably involve spelling out the shell ID (or SHLVL?) on every command.

The current thinking here is to document how to inject the script manually on the website to solve this more advanced case. The goal being users get a nice out of the box experience when shell integration is enabled and if users miss shell integration in sub-shells they can do the advanced setup which I don't think is possible without touching people's scripts which we'd rather not do.

Closing in favor of https://github.com/microsoft/vscode/issues/151933 and https://github.com/microsoft/vscode-docs/issues/5219 as manual install is the approach we'll recommend for sub-shell in the docs.


fix: dependency error

📑 Description

Summary by CodeRabbit

New Features

Expanded service management capabilities with new methods for creating various service instances (e.g., authentication, caching, logging).
Introduced methods for binding and registering shared instances.

Bug Fixes

Improved clarity and consistency in import aliases for contract interfaces.

Chores

Removed the Container interface, streamlining dependency management.
Updated mock variable references in test files for better naming consistency.
Deleted outdated test file related to service provider functionality.

Conflicts

Solved


KAFKA-8187 Bugfix for branch 2.0

Fix KAFKA-8187: State store record loss across multiple reassignments when using standby tasks.

Do not let the thread to transit to RUNNING until all tasks (including standby tasks) are ready.

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

retest this please

Test with JDK 10 and Scala 2.12  can't pass, the error log is as follow
ERROR: No tool found matching GRADLE_4_10_2_HOME

ERROR: No tool found matching GRADLE_4_10_2_HOME

[kafka-pr-jdk10-scala2.12] $ /bin/bash -xe /tmp/jenkins5680363984934145053.sh

+ rm -rf /home/jenkins/jenkins-slave/workspace/kafka-pr-jdk10-scala2.12/.gradle

+ /bin/gradle

/tmp/jenkins5680363984934145053.sh: line 4: /bin/gradle: No such file or directory

Build step 'Execute shell' marked build as failure

[FINDBUGS] Skipping publisher since build result is FAILURE

Recording test results

ERROR: No tool found matching GRADLE_4_10_2_HOME

ERROR: Step ?Publish JUnit test result report? failed: No test report files were found. Configuration error?

ERROR: No tool found matching GRADLE_4_10_2_HOME

ERROR: No tool found matching GRADLE_4_10_2_HOME

Adding one-line test results to commit status...

ERROR: No tool found matching GRADLE_4_10_2_HOME

ERROR: No tool found matching GRADLE_4_10_2_HOME

@bbejeck maybe there is something wrong with the test environment, do you know how can I fix it ?
Thanks.

retest this please

@hustclf let's re-run the tests one more time and if there is an issue, we'll file a ticket with Apache INFRA

created ticket with Apache INFRA https://issues.apache.org/jira/browse/INFRA-18548

retest this please

retest this please

retest this please

retest this please

retest this please

Closing this PR due to inactivity. The issue is fixed in newer version already and it's unlikely that we do a 2.0.2 release.


Fixes Cannot read property 'some' of undefined

Closes #842

Oddly, master currently has 28 failing tests. I checked that after this change there are sill 28 failing tests.

hmm, I only get 1 failing test locally on master (which I'm fixing momentarily)

@alexgorbatchev would you mind rebasing on top of latest master?

Rebased. I think the tests are failing on Windows. Ran it OSX and it was as you said, just one. No failures on OSX after the rebase.

Coverage increased (+0.002%) to 96.032% when pulling c246b9e0dfce95fd7a8c3fc08f94a856e42e9fbb on alexgorbatchev:alexgorbatchev/issue-842 into 1958e0b4f889f752b7b3d579b1d81c50b5f8be7a on benmosher:master.

Coverage increased (+0.002%) to 96.032% when pulling c246b9e0dfce95fd7a8c3fc08f94a856e42e9fbb on alexgorbatchev:alexgorbatchev/issue-842 into 1958e0b4f889f752b7b3d579b1d81c50b5f8be7a on benmosher:master.

Coverage increased (+0.002%) to 96.032% when pulling c246b9e0dfce95fd7a8c3fc08f94a856e42e9fbb on alexgorbatchev:alexgorbatchev/issue-842 into 1958e0b4f889f752b7b3d579b1d81c50b5f8be7a on benmosher:master.

We'll have to deal with the windows failures later I guess.

Could you please publish this to NPM when you get a chance?


Points vanishing when disabling some lines via label click

Bug report
Codepen
No Codepen needed, also occurs in the demo at https://apexcharts.com/javascript-chart-demos/line-charts/null-values/
Explanation
On first load, with all lines being displayed, points show up fine on hover for all lines.
Click the "Peter" label to hide the blue line. Hover a point on the yellow line, e.g. at x value 7:

The point on the yellow line is not showing, only the one on the cyan line.
Show "Peter" again, hide "Jonny":

Points are shown on both lines (though with their colors swapped, see #1427).
If you show only either the "Peter" or "David" line, there will be no points shown on hover. However, for the "Jonny" line, the points will always work.

The issue vanishes when setting the markers option to a value > 0.

Which version of apexcharts are you using? I cannot reproduce this with the latest version.

Latest version from npm, but as I said it also occurs in the demo (tested with Firefox latest, Windows).

Ah! I was checking in chrome.
It will be addressed soon.

Fixed with this commit


Run command as vim operator

Hi,
is it possible to use the run command as a operator on text objects? Often I would like to run the inner paragraph for example or the same indent (with indent textobject plugin).
Also, when opening the ipython buffer, could you set syntax=python automatically?
Thanks a lot!

Implemented it in my fork https://github.com/mightymicha/nvim-ipy

@mightymicha Nice! I might integrate the operatorfunc part, in the weekend or so.

@bfredl did you integrate the operatorfunc part? I would be keen on this. Am going to look at @rattletat 's fork

I cannot find the fork now: https://github.com/mightymicha/nvim-ipy 404 error :(

@JohnReid Hi yes sorry, it seems I deleted it. However, I switched to another plugin which uses a more universal approach and which work quite well:
https://github.com/kassio/neoterm

Thx for the reply. I'll check it out. I do like nvim-ipy though.


feat: add unplugin transform hook doc

Summary
Related Links

Checklist

[ ] Tests updated (or not required).
[x] Documentation updated (or not required).

 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.qianziyang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks!


Suggest using PUT instead of POST when creating list members

There is an issue with how MailChimp handles deletion of list members, in that they don't delete them from the list, they change the status to "". If you want to add that person back to the list, you have to use the PUT operation to update the list member (PATCH apparently doesn't work).
http://developer.mailchimp.com/documentation/mailchimp/reference/lists/members/#
Since the PUT operation allows add or update, it might make sense to use that consistently instead of POST for creating a list member?

Issue #24 is similar to this problem and has been resolved in the latest pull

Ok, I just close the issue then


API - Get credential by Id

@tmarkovski @StevenTCramer
For new API support, existing implementation of getting single credential record for a given credentialId is as shown below.
Existing implementation:
{
  IAgentContext agentContext = await AgentProvider.GetContextAsync();
  List<CredentialRecord> credentialRecords = await CredentialService.ListAsync(agentContext);
  CredentialRecord credentialRecord =
    credentialRecords
      .FirstOrDefault(aCredentialRecord => aCredentialRecord.CredentialId == aGetCredentialRequest.CredentialId);

  var response = new GetCredentialResponse(aGetCredentialRequest.CorrelationId, credentialRecord);
  return response;

}
There already exists a method which directly fetches the single record for a given credentialId. Can we not use GetAsync() method instead of ListAsync() method?
Proposed implementation:
{
  IAgentContext agentContext = await AgentProvider.GetContextAsync();
  CredentialRecord credentialRecord = await CredentialService.GetAsync(agentContext, aGetCredentialRequest.CredentialId);
  var response = new GetCredentialResponse(aGetCredentialRequest.CorrelationId, credentialRecord);
  return response;

}
I have implemented and tested it. Below is the link for the above changes. Kindly verify and assist so that we can merge this change if required and valid.
https://github.com/RinkalBhojani/aries-framework-dotnet/commit/c8446eb3add2e9e6974d72c59f1404fb3d9782b2

@RinkalBhojani  my first impression is you are correct.  If you would like to do a PR that would be great.


Update to use latest Apache RocketMQ

RocketMQ has been incubating on Apache for a while and new version has been released thereafter. This PR is to update the plugin in order to be compatible with the latest release of Apache RocketMQ.

Please fix the build failure.

@lizhanhui, the error seems to be an unused import.
[ERROR] /home/travis/build/druid-io/druid/extensions-contrib/druid-rocketmq/src/main/java/io/druid/firehose/rocketmq/RocketMQFirehoseFactory.java:38:8: Unused import - java.util.Iterator. [UnusedImports]

@lizhanhui


Otalk cann't connect to XMPP server

I tested with stanza.io demo, client can log in XMPP server. But with Otalk, client can't log in. I think there's bug in public/js/login.js. Following to demo of stanza.io, password shouldn't be nested in credentials. Is it right ?
localStorage.config = JSON.stringify({
        jid: jid,
        server: jid.slice(jid.indexOf('@') + 1),
        wsURL: wsURL,
        boshURL: boshURL,
        transport: transport,
        credentials: {
            password: password
        }
    });

I have the same problem!!
I have the same issue with ejabberd server (V 15.10). This is my jabbed configuration:
hosts:
  - "myserver.com"
listen:
  -
    port: 5280
    module: ejabberd_http
    certfile: "/Applications/ejabberd-15.10/conf/myserver.pem"
    request_handlers:
      "/websocket": ejabberd_http_ws
    web_admin: true
    http_bind: true
    captcha: false


add david-dm npm package dependencies status badges

Hi Peter,
Thanks for your pull request.
I don't think it neccessary to putting dependency status in our readme file. A page with too many badges could be a little bit messy.

No problem :smile:
Don't know if you know it'll show warning when it found that at least one of the dependencies has security issue, may help make safer program!

ping @ottomao

 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.


Replace PRNG with crypto/rand

In the file signer.go (link) there is an usage of a custom PRNG built using the private key, without an apparent reason.
The documentation for GenerateKey states that, instead, crypto/rand.Reader is the reccomended option, and here we are using our own PRNG instead.
Maybe should we remove it at all and just rely on crypto/rand library?

I talked with the creator and he told me that the PRNG is present to always generate the same certificates for the same host


metrics: wrap metrics in gateway as well

Codecov Report

Merging #1189 (447218c) into main (b85dd05) will increase coverage by 0.03%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1189      +/-   ##
==========================================
+ Coverage   56.92%   56.96%   +0.03%
==========================================
  Files         127      127
  Lines       21350    21351       +1
==========================================
+ Hits        12153    12162       +9
+ Misses       7726     7718       -8
  Partials     1471     1471

Impacted Files
Coverage Δ

cmd/gateway.go
30.11% <0.00%> (-0.18%)
:arrow_down:

pkg/meta/sql.go
68.14% <0.00%> (+0.45%)
:arrow_up:

Continue to review full report at Codecov.

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


Fixed typo in README.md

donwloading edited to downloading

Copy of https://github.com/TheCherno/Hazel/pull/462


Add Workspace section

Also added hyprkool because I'm impatient, so #62 is obsolete now

Thx

thanks :)


cannot import name 'activation_ops' from 'vllm'

We discovered some issues while experimenting with the framework. Some packages are missing in the vllm directory：

ImportError: cannot import name 'cuda_utils' from partially initialized module 'vllm' (most likely due to a circular import)
ImportError: cannot import name 'activation_ops' from 'vllm'

Our run command is：python3 -m vllm.entrypoints.api_server

Could you run vLLM outside the root directory of the repo?

No, we did it in the vllm-main

Thank you for your reply. We successfully solved the problem using the method you mentioned.


How to create complex route transitions?

By complex animation I mean is that it not only transits to a new route, but also does something on an old route. For example, in every app (both android and iOS) it is considered a good practice to have a transition, which pushes new route from right and old route goes a bit to left and dims.
Your dev guides and API docs seem to be not touching this topic at all, googling also gave me no answers and that badly surprised me because it's such a basic thing in almost every native application.
Stackoverflow question

Possible solution for lists posted here

Animating both the old and new routes during a transition is covered here: https://api.flutter.dev/flutter/widgets/ModalRoute/buildTransitions.html. I agree that you'd be hard-pressed to find this if you looked in most of the obvious places where'd you'd expect to at least find a link.  Will leave this bug open until doc links have been added.

Animating both the old and new routes during a transition is covered here: https://api.flutter.dev/flutter/widgets/ModalRoute/buildTransitions.html.

Ok, but firstly: what if I need different secondary animations for different route pushes?
Simple example: in my app a have a route that slides from down and the one that comes from right. For right slide I want to have dim effect, for down one I don't.
Secondly, how do I apply this to a home page of MaterialApp?

It sounds like you want to create custom routes for your app; the route object (or its RouteSettings) would inform the transition animations. Presumably all routes would be constructed with onGenerateRoute, like https://api.flutter.dev/flutter/material/MaterialApp/onGenerateRoute.html. That would enable creating a custom route for the home page as well.

Hey, why don't you add something like this into you library? https://github.com/nt4f04uNd/flutter_music_player/blob/master/lib/components/route_transitions.dart

Why does secondaryAnimation played only for top level pop when multiple routes are in stack on top of origin?

i'm not sure if this is still relevant, but i just answered my SO question, where provided a few explanations on how to properly do this and given an example of custom route transition https://stackoverflow.com/a/58369240/9710294

a good improvement here would be to change this cookbook to include example of secondaryAnimation with  https://flutter.dev/docs/cookbook/animation/page-route-animation


OptionStrategies for Iron Butterfly

Description

Add OptionStrategies abstraction helper method for Iron Butterfly.

Margin for long Iron Butterfly (short iron condor with same strike for middle strikes):

Expected: 0; actual: 0

Margin for short Iron Butterfly (long iron condor with same strike for middle strikes):

Expected: 1000; actual: 1000 (+10 for 1% commission)

Add Short version of Iron Butterfly and Iron Condor

Related Issue
NA
Motivation and Context
Missing feature
Requires Documentation Change
Add helper method usage in

Trading and Orders -> Option Strategies -> Iron Butterfly
Trading and Orders -> Option Strategies -> Short Iron Butterfly
Trading and Orders -> Option Strategies -> Short Iron Condor

How Has This Been Tested?

Added unit tests
Added regression test

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Refactor (non-breaking change which improves implementation)
[ ] Performance (non-breaking change which improves performance. Please add associated performance test and results)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Non-functional change (xml comments/documentation/etc)

Checklist:

[X] My code follows the code style of this project.
[X] I have read the CONTRIBUTING document.
[X] I have added tests to cover my changes.
[X] All new and existing tests passed.
[X] My branch follows the naming convention bug-<issue#>-<description> or feature-<issue#>-<description>

Please rebase from master, the regression test interface was changed & a unit test CI bug was shipped


getArbTransactions() is not returning any transactions

Using the sample-code-php/RecurringBilling/get-subscription.php
This line of code always returns an empty array.
$transactions = $response->getSubscription()->getArbTransactions();
Note that the other details of the subscription are being returned and I've verified that there are transactions under 2 different ARB ID's, both in the sandbox and production.

Are you passing the includeTransactions as true in the request, similar to line 25 of the sample code?

a. How old are the subscriptions?
b. What is the status of the transactions in the sandbox/production?

Can you share the contents of your phplog file which will have the full xml response from the API. You can find it in the same folder as your PHP SDK.

Can you also share the code snippet for traversing the arbTransactions array?

#1 was what I missed.
Thank you very much for your help.


Added IES Sa Blanca Dona

IES B. Dona have a special class for IT and programming.
https://aulavirtual.caib.es/c07006299/course/index.php?categoryid=4

@albertomario Pull request merged. Thank you!


Add support for 'all protocols and ports' firewall

Tested by creating such a firewall in GCE and verifying that it creates a firewall with allowed.IPProtocol=='all'

{
"kind": "compute#firewall",
"id": "5013197503613505035",
"creationTimestamp": "2017-07-19T12:20:04.239-07:00",
"name": "fw-wildcard",
"description": "",
"network": "https://www.googleapis.com/compute/v1/projects/iap-1-174217/global/networks/default",
"sourceTags": [
"wildcard"
],
"allowed": [
{
"IPProtocol": "all"
}
],
"selfLink": "https://www.googleapis.com/compute/v1/projects/iap-1-174217/global/firewalls/fw-wildcard"
}

Adding cla: yes label, as the PR author is a googler at the time of this PR was created.


[DSS] Resource path text area is not editable

When defining a resource in a data service, cannot add a resource path. Text area is not editable.
However, figured out that when you click here and there we can get it.
Reporting the issue, as it is not user friendly.

Fixed in https://github.com/wso2/devstudio-tooling-dss/pull/90


Update h2 to 1.4.200

Updates com.h2database:h2 from 1.4.196 to 1.4.200.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.h2database", artifactId = "h2" } ]

labels: test-library-update, semver-patch

Codecov Report

Merging #280 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #280   +/-   ##
======================================
  Coverage    89.9%   89.9%
======================================
  Files         134     134
  Lines        2902    2902
  Branches      132     132
======================================
  Hits         2609    2609
  Misses        293     293

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 a7dac04...a0b18b7. Read the comment docs.


Unable to add Embedded Ansible Repositories

I'm unable to add github ansible repositories in manageiq. Any idea how to resolve this?
The option to add is greyed out. I'm using the latest github manageIQ repo to build manageIQ docker instance.

Figured it out. I had to enable Embedded Ansible in Application Settings.


Encode authToken before sending it from client

Part of #303, encodes authToken in request headers in client side and decodes it on the server side middleware.

A few naive questions/comments:

what issue does this solve? Is it in case the shop's name had some non-ascii characters?
also maybe it's just me but the name "authToken" always confuses me. As far as I understand, this is not a credential but just a way to identify which shop to operate on. maybe we should consider renaming to shopIdentifier or equivalent?
why can't we just use the shop id? Wouldn't that be simpler?
do we need to worry about this breaking existing shops Front-End until the merchant publishes them again?

what issue does this solve? Is it in case the shop's name had some non-ascii characters?

Yes, some shop name's had emojis and some shop name's had non-Ascii characters. The request header had to be encoded before it can be sent.

also maybe it's just me but the name "authToken" always confuses me. As far as I understand, this is not a credential but just a way to identify which shop to operate on. maybe we should consider renaming to shopIdentifier or equivalent?

Yep, we should change that. I'm not sure why it is called authToken either. What do you think @nick?

why can't we just use the shop id? Wouldn't that be simpler?

I'm not sure why the name is being used to identify the shop, and yeah I don't see a reason to use authToken over id. Perhaps @nick can shed some lights here.

do we need to worry about this breaking existing shops Front-End until the merchant publishes them again?

No, it won't break. decoding ASCII characters should return whatever the input is, as-is.


feat: add es and zh translation to the feedback form

Closing issues
closes #202 202
...
Description
Adds feedback form translations to es and zh.

...
Changes

Import i18next to the feedback component.
Created a new translation grupo into rightSidebar for es and zh
Replaced the hardcoded texts with the t function.

@dghelm I'm creating a new PR here https://github.com/scroll-tech/scroll-documentation/pull/329, got some issues when updating my branch. Please let me know if we should do it differently.


Cannot read property '0' of undefined - ts source

Large typescript source gives the error: Cannot read property '0' of undefined Unfortunately there is no line number. I can try to reduce the test case if I can have some clues :)
Input can be found at https://gist.github.com/krk/2c9f8aa8ecef1b4e05dd4c8b60d279f2

Thank you!    What I'd do to find the bug is run quicktype in the debugger.  Do you want me to write something up on how to do that?

That bug is fixed now, but another bug prevents that file from working: https://github.com/YousefED/typescript-json-schema/issues/198

I fixed the stack overflow, too, in PR #809 .  It's STILL not working, because of this type:
interface IDBArrayKey extends Array<number | string | Date | IDBArrayKey> {
}

On the one hand, this is another bug in typescript-json-schema (https://github.com/YousefED/typescript-json-schema/issues/200), but even if it generated the correct Schema here, quicktype couldn't handle it, because of #564.
Defining this type in a way that's not a recursive array will produce a result.  If you have time to look at how correct the resulting types are, I'd be very interested.  Please file bugs for anything that's not as it should be.
This is as far as I'm willing to take this one at this point.  The recursive array issue isn't a big priority right now I think.


Tap on iOS10 mobile embed jumps to page bottom

Context
One tap over an embed on mobile iOS10 jumps the browser (Safari) until the page bottom. If you scroll back to the embed, the normal double tap zoom interaction works as expected. This is not happening with Editor embeds.
Comes from: https://github.com/CartoDB/support/issues/660
Steps to Reproduce

Open this on iOS10 Safari.
Tap on the second embed, generated with Builder

Current Result
Browser jumps to the bottom
Expected result
Normal zoom double tap

Moving to its right place.

Issue moved to CartoDB/cartodb.js #1652 via ZenHub


Docs: Removed underline from anchor

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] New feature or enhancement
[ ] UI change (please include screenshot!)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[x] Documentation content changes
[ ] Internationalization and localization
[ ] Other (please describe):

Preview: https://github.com/0x5bfa/DevToys/blob/baf30b96b17a91a8af302dd4e36f50f816547bfe/README.md

Good catch! Thank you for this!


Python Interface on Windows

There seems to be some issues with the Python interface on Windows.
Reports are that it stems from differences in the asyncio implementation on Windows.

Reports of this error:
 File "C:\python-3.7\lib\asyncio\base_events.py", line 463, in _make_subprocess_transport
    raise NotImplementedError

I think this should be fixed since we moved away from asyncio in cdcf4cd8776ff21121a1a5111be21d158fb20e22.
We don't have any Windows CI, but we will close this for now as we stuck to only platform-safe parts of subprocess.


[1.21] Terminals don't go into curio slots.

Tag seems to be wrong.

There are like half a million curio forks for 1.21 right now, if those forks are not implementing the curio spec correctly, that is on them.
I believe there is some work to get a unified curio api into neoforge, if that gets released I'll look into fixing the curio related bugs.

i meant as in, you have it as curios/tags/items while it should be curios/tags/item
and from what i quickly saw in the code, you use AE2s curio compat for yours, right?

AE2s curio compat works fine.

ah right, minecraft renamed the folders, I'll have to fix that


Changed about page background and added amFOSS to that

Changed the old background which is not so clear and changed foss@amrita to amFOSS

This is the old one

After changing this is the new one

@abhishekbvs, please fix the conflicts.


Virtual text only for insert mode and floating window for normal mode

Greetings, I have noticed that this plugin override the default vim.lsp.buf.signature_help() and now I wonder if there is any way to have virtual text only while I am typing and trigger the floating window without typing (with keystrokes)?

I do not think it is possible ATM.
Do you accept virtual text only in insert mode, and in normal mode, when you use your keymap to invoke vim.lsp.buf.signature_help() showing the floating window?

That would be great

It should been fixed in new version. Please update your plugin and give it a try. Thanks!

Yes, it's perfect, it also works in insert mode. Thanks


Bundle sqlite

~Allow bundling~ bundle the sqlite3 library, rather than relying on dynamic linking to a system sqlite3 library, which may be out-of-date or nonexistent.

@gx0r how would you go about this? Is this just a feature flag on diesel/r2d2 or something?

ah darn I thought this was an issue for a second haha
thanks for making this PR! I'll take a look

I believe it works from my own testing, but please validate/double check if you have time to. libsqlite3-sys is a transitive dependency by deisel that pulls the actual library, and libsqlite3-sys has an available "bulded" feature ( https://crates.io/crates/libsqlite3-sys )

makes sense -- what do you think about making this the default?

<0.26.0 -- wouldn't that be 0.25?

Probably, what do you think? In my experience, common distributions in docker tend to have 1+ year old SQLite so you have to compile your own or something like that to get a "reasonable" version. Plus it can be a hurdle to install on dev machines and Windows. So perhaps it's a good default.

<0.26.0 -- wouldn't that be 0.25?

Right. Though it seems to work specifying our own version so maybe Diesel is just too restrictive there.

yeah let's make this the default; could you make these changes please?
##
## Database
##

# sqlite dependencies
libsqlite3-sys = { version = "0.25", optional = true, features = ["bundled"] }

database_sqlite = ["diesel/sqlite", "libsqlite3-sys"]

I remember now more about why I ran into this. The Debian SQLite version is still about 1 year old and is just before SQLite supported "returning" clause, so the todo backend would not actually work without later version compiled in.

@gx0r oh man, that makes sense! I actually moved to arch from debian because of similar experiences like this lol

Grant, thanks for this, do you want me to add you as a collaborator?

Grant, thanks for this, do you want me to add you as a collaborator?

You're welcome, appreciate the merges, sure that would be nice.

Done, check your email! Feel free to suggest more changes :)


[spike] add assets to GH releases

description
Prerequirement for #3632 , would like to be able to test a couple things with assets before moving forward. 🙇
Testing

Trade-offs

Dependencies

Screenshots

Package-lock conflict.

resolved 👍

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


dbeaver/pro#3620 Introduce JUL handler

I don't like that you're using INFO logging level in the driver. They bloat the Error Log view which is used for tracking issues like warnings and errors. Can it be fixed within a separate ticket?

Error Log view has Filters actually

Verified

Verified


Aspect_Sentiment_Classification.ipynb error

When running the notebook, the following error shows up

Which version and optimizer do you use?

Thanks for your reply, wich versions of torch and transformers do you use?

I use torch 1.7.1 and transformers is 4.25.1

The v2.0 introduces a maximize option, which is only available in torch 1.12 or 1.13.


Upgrade geotools to newer version

Look into using a more recent version of geotools, preferably the latest version, which is currently 12.1.

Changed Geootools version to 12.1


Allow API to set a custom ID for server creation.

Useful when dealing with billing systems such as WHMCS

So this is going to have to be slightly different than the code used for adding users with unique IDs because we pass all fields in. What I would do is do 'custom' => 'sometimes|required|numeric|unique:servers,id' which would then only check it if the field existed.
Your current method requires passing unique_id to all requests, which isn't necessarily desirable. Then you could just do an isset() check on the field for if the ID should be set.

Ohhh i see, that does make sense. Will make those changes to my code.
Thank you for the feedback!


Find full GitHub project link in Dockerfile lock functionality

Describe the Feature
There is a pending PR for issue #509 that will append the git project name and sha in a comment for Dockerfile lock functionality. This is a good start but, ideally, we would be able to append the full https:// URL to the comment. In many cases the repository will be hosted on GitHub under the 'origin' label (i.e. when you list git remote -v) but this is not always the case. The user may have re-named the label from origin to something else or the code may not even be hosted on GitHub. There may also be multiple repos listed.
Use Cases
This will enhance the information supplied to the user of Tern's Dockerfile Lock functionality.
Implementation Changes
Run git remote -v at the top level of the git directory that is being ADDed in the Dockerfile. Significant parsing will need to take place to the output to determine the repo URL. Most changes will take place in tern/analyze/docker/dockerfile.py in the find_git_info() function although you may also want to create a new function to determine the git project URL.

Maybe i can work on this.

@ForgetMe17 yes, let's get your other PR merged first and then you can build upon it for this issue :) Thanks!

I think I can work on this issue.

A PR for this issue was merged to master after the 2.0.0 release. Closing this.


about separation ideas.

Hello，thanks a lot for your sample code and tutorial ,and I have questions to ask you .In the first paragraph of your tutorial ,you say "We want a clear separation of concerns so that the UITableViewCell isn’t acting as the data source or delegate for the UICollectionView (because that would be very, very bad)."
About clear separation,why not let the UITableViewCell act as the data source or delegate for the UICollectionView,I would like to make the UITableViewCell act as the data source or delegate for the UICollectionView usually if I haven't read your tutorial.There are my questions :
1.What are the shortcomings?
2. When to make the UITableViewCell act as the data source or delegate for the UICollectionView? When not do that?Could you talk about your ideas?
thanks a lot .

Hi there! That's a good question. I've actually gotten this question before, and wrote a blog post about it: https://ashfurrow.com/blog/challenging-established-dogma/ Hopefully that helps clear some things up. Cheers!

Hi,
Thank you so much for your detailed ideas yourself.
I think I learned a better way of thinking than technology, and I will regard issues from the point of dialectics .Thank you very much for sharing !


[New Feature] Addressing Data Contamination in Evaluation Benchmarks

Data contamination is an urgent crisis in the evaluation community. There are many concerns about contamination issues in LLMs' evaluations from people in the media and reviewers on OpenReview.
I have developed a tool to purify existing benchmarks and assess the true capability of LLMs.
More specifically, I categorise test samples into three subsets with search engine:

clean
input contaminated: the exactly same question appears online.
input-and-label contaminated: both question and answer appear online.

By testing LLMs on the clean set, we shall be able to avoid the impact of memorisation in the evaluation.
And by comparing the performance on the clean and contaminated set, we can have an impression on how serious existing evaluation was affected by data contamination.
Check the performance of Llama-2 70B as an example:

Dataset
Condition
Llama-2 70B

MMLU
Clean
.6763

MMLU
Input-only contaminated
.6667 ↓

MMLU
Input-and-label contaminated
.7093 ↑

Hellaswag
Clean
.7726

Hellaswag
Input-only contaminated
.8348 ↑

Hellaswag
Input-and-label contaminated
.8455 ↑

ARC
Clean
.4555

ARC
Input-only contaminated
.5632 ↑

ARC
Input-and-label contaminated
.5667 ↑

Average
Clean
.6348

Average
Input-only contaminated
.6882 ↑

Average
Input-and-label contaminated
.7072 ↑

We saw large model like Llama-2 70B is very good at utilise contaminated test samples to achieve higher metric.
For more info, check this repo and this pre-print.

Closing because as discussed on Github, the best way to integrate with this tool would be for it to upload a dataset cleaned/contaminated copy to HF hub and then one can run the task pointing to that dataset copy.
With our --use_cache feature this should not expand inference runtimes at all.


Meteor app works great on browsers but not Cordova app.

I have a Meteor app using semantic-ui-react. It works perfectly on all browsers as a web app. But when I compile and run for iOS, I get no styling whatsoever. All components are plain, and absolutely no indication of Semantic UI anywhere. I have not modified the Xcode project that was generated by Meteor.
Browsers I've tested where it works: Chrome, Safari (desktop and iOS), Firefox. I've checked issues and done exhaustive web searches to no avail.
Thanks for any insight you might have!!
Meteor 1.9.2 with:
packages
<EMAIL_ADDRESS>           # Packages every Meteor app needs to have
<EMAIL_ADDRESS>     # Packages for a great mobile UX
<EMAIL_ADDRESS>                 # The database Meteor supports right now
<EMAIL_ADDRESS>          # Reactive variable for tracker

<EMAIL_ADDRESS>  # JS minifier run for production mode
<EMAIL_ADDRESS>              # ECMAScript 5 compatibility for older browsers
<EMAIL_ADDRESS>            # Enable ECMAScript2015+ syntax in app code
<EMAIL_ADDRESS>          # Server-side component of the `meteor shell` command

static-html             # Define static page content in .html files
react-meteor-data       # React higher-order component for reactively tracking Meteor data
std:accounts-ui
<EMAIL_ADDRESS>alanning:roles
jquery
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>aldeed:collection2
<EMAIL_ADDRESS>sakulstra:aggregate
less
apollo
swydo:graphql

package.json
{
  "name": "ExsoMed",
  "private": true,
  "scripts": {
    "start": "meteor run --settings settings.json",
    "test": "meteor test --once --driver-package meteortesting:mocha",
    "test-app": "TEST_WATCH=1 meteor test --full-app --driver-package meteortesting:mocha",
    "visualize": "meteor --production --extra-packages bundle-visualizer"
  },
  "dependencies": {
    "@apollo/react-common": "^3.1.3",
    "@apollo/react-components": "^3.1.3",
    "@apollo/react-hoc": "^3.1.3",
    "@apollo/react-hooks": "^3.1.3",
    "@babel/runtime": "^7.8.4",
    "@types/react": "^16.9.23",
    "ag-grid-community": "^22.1.1",
    "ag-grid-react": "^22.1.1",
    "apollo-cache-inmemory": "^1.6.5",
    "apollo-client": "^2.6.8",
    "apollo-link": "^1.2.13",
    "apollo-link-ddp": "^3.0.0",
    "apollo-link-http": "^1.5.16",
    "apollo-server-express": "^2.10.1",
    "bcrypt": "^4.0.0",
    "cron": "^1.8.2",
    "csvtojson": "^2.0.10",
    "graphql": "^14.6.0",
    "graphql-tag": "^2.10.3",
    "graphql-tools": "^4.0.6",
    "merge-graphql-schemas": "^1.7.6",
    "meteor-node-stubs": "^1.0.0",
    "mobx": "^5.15.4",
    "mobx-react": "^6.1.8",
    "moment": "^2.24.0",
    "moment-timezone": "^0.5.28",
    "node-quickbooks": "^2.0.33",
    "react": "^16.13.0",
    "react-apollo": "^3.1.3",
    "react-compose": "^2.0.0",
    "react-dom": "^16.13.0",
    "react-modal": "^3.11.2",
    "react-phone-number-input": "^3.0.23",
    "react-router": "^5.1.2",
    "react-router-dom": "^5.1.2",
    "react-sortable-hoc": "^1.11.0",
    "react-spinners": "^0.8.0",
    "semantic-ui-css": "^2.4.1",
    "semantic-ui-react": "^0.88.2",
    "shortid": "^2.2.15",
    "simpl-schema": "^1.5.7",
    "simplecheck": "^0.1.3",
    "ssh2-sftp-client": "^5.0.2",
    "uniforms": "^2.6.6",
    "uniforms-bridge-simple-schema-2": "^2.6.6",
    "uniforms-semantic": "^2.6.6",
    "uniforms-unstyled": "^2.6.6",
    "uploadcare-widget": "^3.8.3"
  },
  "meteor": {
    "mainModule": {
      "client": "client/main.jsx",
      "server": "server/main.js"
    },
    "testModule": "tests/main.js"
  },
  "devDependencies": {
    "autoprefixer": "^9.7.4",
    "postcss": "^7.0.27",
    "postcss-easy-import": "^3.0.0",
    "postcss-load-config": "^2.1.0"
  },
  "postcss": {
    "plugins": {
      "autoprefixer": {
        "browsers": [
          "last 2 versions"
        ]
      }
    }
  }
}

@KalalauEnterprises all of the styles are not contained in the SUIR project, so this is not related to SUIR.
I do however have some Cordova based applications that run with SUIR and SUI styling in them. The solution to get this working is typically taking a look at the build process being run. This usually means having a separate environment config for Cordova builds. Path resolution can differ.
It sounds like your app is running okay, but however your styling is not showing because when running in Cordova, your web wrapper cannot find the styles in their normal path resolution. I'd start debugging efforts there to determine where the application is trying to get the CSS files, are they there or not, and if not, where are they. If I recall, I think there may also be issues with relative paths not working as expected.
Closing as this is not related to SUIR, but hopefully those suggestions point you down the right debugging path in your application builds.


fix: svg image transparent background should be preserved

Close https://github.com/Brooooooklyn/canvas/issues/428

@Brooooooklyn what's the timeline for merging this?
ps. ty for this wonderful package


[ARRDT] Add usePaginationFragment

I would ideally switch the conversations and messages connections in pagination direction, so to resemble how we actually use them in Teams.

Ready for merging back into the larger feature branch.


Prevent multiple instances of light to be discovered for one device, …

Description:
Prevent multiple entities to be created for 1 device with multiple indexes in light component.
Some small refactor of strings.
Add debug for polling
Related issue (if applicable): fixes #
Possibly #4159

Code looks fine with me.

Seems neat.

Related to https://github.com/OpenZWave/open-zwave/issues/1003 ?

Closing this, as an other fix will in a different PR will be provided.


markBands in gisticChromPlot has no effect

Looking through the code, it seems it is not yet implemented.

Hi,
Yes that is correct. I am still working on it, will update it soon.

Hi,
Sorry for the delay. I have updated the code, it should be working now.


Defining global object in vite config breaks any imports with global in their name

Describe the bug
If I add something like this in my Vite config:
// vite.config.js
export default {
  define: {
    global: {},
  },
};

And also have some imports with global in the name like:
import { globalReducer } from './global-reducer'

The build process fails with this error message:
Could not resolve "./{}-reducer" from "main.js"

This only happens in vite build, not during development.
Reproduction
https://stackblitz.com/edit/vitejs-vite-dzocfw?file=main.js
Steps to reproduce
Run npm run build in the terminal in the stackblitz URL.
System Info
System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 7.17.0 - /usr/local/bin/npm
  npmPackages:
    vite: ^4.3.0-beta.2 => 4.3.0-beta.4

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Further digging, I guess this is warned against in the docs... https://vitejs.dev/config/shared-options.html#define


[WIP] Move Foreman managers into ExtManagementSystems.

This is needed because so many other things rely on EMSes that having
them separate makes the calling so much more complex.  This includes
things like workers, reporting, REST API, validations, scheduling, etc.
This commit may break reporting and/or introduce inconsistencies in
reporting, which will be fixed in follow up work.

@kbrock @gmcculloug @brandondunne Please review.
Also note that I am marking this WIP only so that I can verify that building out the other things like workers actually works on top of this...if I find problems with that I will update here.

We were concerned about the extra business logic in the base ExtManagementSystem. But I guess it will not cause too many problems.
Lets consider renaming belongs_to :configuration_manager to belongs_to :manager

We were concerned about the extra business logic in the base ExtManagementSystem. But I guess it will not cause too many problems.

So I originally started down this path and put it on the back burner so I could actually get workers and validations and stuff done for this Sprint.  It mostly just affects reporting.  My other branch is introducing a new class in the hierarchy called VirtManager.  All of the cloud/infra specific stuff (like, hosts, resource_pools, etc) in the base class would move here.  Then I just gotta get reporting working correctly.
ExtManagementSystem  # Has "virt" stuff like :hosts, :resource_pools, etc.
  VirtManager  # Introduce the new layer and move everything "virt" from EMS here
    EmsInfra  # This is currently directly deriving from EMS
      EmsOpenstackInfra
      EmsMicrosoft
      EmsRedhat
      EmsVmware
    EmsCloud  # This is currently directly deriving from EMS
      EmsAmazon
      EmsOpenstack
  EmsContainer
    EmsKubernetes
  ConfigurationManager
    ConfigurationManagerForeman
  ProvisioningManager
    ProvisioningManagerForeman

Lets consider renaming belongs_to :configuration_manager to belongs_to :manager

Cool, yeah I debated this, but didn't want to mess with callers right now.  Same with :provisioning_manager.  You think I should do this now, or wait until separate PR?  Should I just rename the configuration_manager_id to manager_id?

Fixed the Travis issues..mostly had to do with supported_subclasses not seeing the new subclasses, and also missing factories for the new classes.

This is good to go...the workers are working out locally.


Change how "from" email is constructed from service name

Currently, the email address that emails are sent from is constructed using:

the service name
the service email_from (or default email_from after #123)
the service sending_domain or default NOTIFY_EMAIL_DOMAIN

from_address = '"{}" <{}@{}>'.format(service.name, service.email_from, sending_domain)

For email providers other than Govdelivery, the service name will appear as the email sender. (Govdelivery will just remove it and send from the email address, with no name)
Is this desired? If so, what should the service names be if they're gonna be visible to the recipients?

Value Statement:
As
I want to
So that
Acceptance Criteria
GIVEN
WHEN
THEN
Checklist:

[ ] Checklist item here

Assumptions:

Additional Info/Resources:

Out of Scope:

Open Questions:

Priority will depend on SES speed.


request help: After removing apisix-ingress-controller it will not start

Issue description
I deleted a route created by apisix-ingress-controller on the dashbord dashboard, executed kubectl delete pod apisix-ingress-controller and it doesn't start anymore
Environment

your apisix-ingress-controller version (1.4.1):
your Kubernetes cluster version (v1.24.2+rke2r1):
if you run apisix-ingress-controller in Bare-metal environment, also show your OS version ( 5.15.0-43-generic Ubuntu):
apisix version (2.13.1-alpine)
apisix dashbord version (2.11-alpine)

apisix-ingress-controller  error log

In fact there are only two apisixroutes in the cluster

please show me your kubectl get pods apisix-ingress-controller command's result. thanks.

please show me your kubectl get pods apisix-ingress-controller command's result. thanks.

I did an uninstall and deleted the data in etcd and redeploy also couldn't get it to work

I saw the pod status is CrashLoopBackOff, In this scenario, you need to view its log and debug it.
In addition, when using APISIX Ingress, it is not recommended that you use APISIX Dashboard for data manipulation.

我看到pod状态是CrashLoopBackOff，在这种情况下，你需要查看它的日志并调试它。
此外，在使用 APISIX Ingress 时，不建议您使用 APISIX Dashboard 进行数据操作。

Its log is as follows, I don't have upstreams appearing in the log, and it doesn't help after I redeploy, how can I fix it

Solved, my k8s cluster node network is not connected, and the health check fails.


LetsEncrypt SSL

Free LetsEncrypt SSL anyone?
https://letsencrypt.org/

Would need a domain name.


@stdlib/string: startcase function does not work when the input string is fully uppercase

Description
Encountered an unexpected result when trying to change the case of an enumerator key
The string is obtained using the logic: Enumerator[Enumerator.EnumeratorKey]
Related Issues
No related issues found (yet)
Questions
No.
Demo
https://codesandbox.io/s/agitated-hill-v37msg?file=/src/App.tsx
Reproduction
- Define/import an enumerator with uppercase string keys
- Use the enumerator value to get the enumerator key (`startcase(Enumerator[Enumerator.ENUMERATOR_KEY])`)

Expected Results
Execute `startcase(StatusCodes[StatusCodes.UNAUTHORIZED])` and obtain `'Unauthorized'`

Actual Results
// The enumerator is exported by the `http-status-codes` library

enum StatusCodes { // Enumerator structure for reference
  ACCEPTED = 202,
  BAD_GATEWAY = 502,
  // ...
  UNAUTHORIZED = 401,
  // ...
};

startcase(StatusCodes[StatusCodes.UNAUTHORIZED]); // -> UNAUTHORIZED

startcase(StatusCodes[StatusCodes.UNAUTHORIZED].toString()) // -> UNAUTHORIZED

startcase(`${StatusCodes[StatusCodes.UNAUTHORIZED]}`); // -> UNAUTHORIZED

startcase("UNAUTHORIZED"); // -> UNAUTHORIZED

The workaround is:
startcase(StatusCodes[StatusCodes.UNAUTHORIZED].toLowerCase()); // -> Unauthorized

Version
"@stdlib/string": "^0.0.13"
Environments
Firefox, Chrome, Microsoft Edge
Browser Version
No response
Node.js / npm Version
<EMAIL_ADDRESS>/<EMAIL_ADDRESS><EMAIL_ADDRESS>Platform
No response
Checklist

[X] Read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.

Thank you for filing this issue and providing a detailed description!
The current behavior is intentional and matches our other *case functions such as camelcase and pascalcase in that it leaves uppercase character appearing in the middle of a word intact
(e.g., see [1], [2]).
We tried to convey this via the package description, which is:

Capitalize the first letter of each word in a string.

and have a unit test included to that effect.
We suggest you stick to first lowercasing the string before passing it into the startcase function in order to achieve the desired behavior.


Add Border on all four sides of Menu Container

I want to add border to all four sides of menu view, also there I want to have some margin from left and right, can anyone help how I can do that.
Thanks

Hi! You can archive this by subclassing PagingView and overriding the setupConstraints method to position the menu (self.collectionView) the way you prefer. You could also apply borders on the collection view here. Here's a similar example: https://github.com/rechsteiner/Parchment/blob/main/Example/Examples/Images/UnsplashViewController.swift#L24. Does that work for you?


Bump Tailscale to 1.38.2

Replaces PR #127
Fixes #124
PR #130, #131 and #132 precedes this PR
Proposed Changes
Bump Tailscale to 1.38.2

@lmagyar I think you need to add a label to the PR in for the GitHub Actions to pass.

@lmagyar I think you need to add a label to the PR in for the GitHub Actions to pass.

Only the repo owners can add labels.

Only the repo owners can add labels.

Ah that's unfortunate, there's not much activity.

Dear @frenck and @jeffothy . I think this pull request seems fine and I'd love to get the newest version of Tailscale running on my HA instance. Can you please have a look at this PR?

Hi @frenck and/or @jeffothy.  Can you please have a look at this PR?

Hi, what is the problem with accepting this pull request?
People are contributing, discussing, but no answers from "owners".
lmagyar is so patient with continuously updating PR and answering questions.
What is wrong with this PR? Is there anything to improve?
@frenck @jeffothy please speak up.

@lmagyar I think you need to add a label to the PR in for the GitHub Actions to pass.

Only the repo owners can add labels.

Hi @frenck and/or @jeffothy. Can you please have a look at this PR?

Hi, what is the problem with accepting this pull request? People are contributing, discussing, but no answers from "owners". lmagyar is so patient with continuously updating PR and answering questions. What is wrong with this PR? Is there anything to improve? @frenck @jeffothy please speak up.

Could it be that it is not in their interest to support tailscale officially, as it directly undermines one of the main selling points of Home Assistant Cloud?

Unfortunately, I am not a codeowner here nor a maintainer on this repository.
However, I did contribute the most recently merged upgrade bump PR https://github.com/hassio-addons/addon-tailscale/pull/108
Based on that PR and the review of a related dep bump PR https://github.com/hassio-addons/addon-tailscale/pull/109 by Franck, perhaps I can make a couple suggestions here:

Based on a historical review comment, I would say the preference is to only make required and related changes in the Upgrade PR. I suggest the removal of the unnecessary changes here in the PR:

Are the base image bumps required in this PR? (can they be delivered separately?)
What about the docs and scripts minor modifications?

The fact that there is a vulnerability in the current addon version provides some good justification for the PR, and should probably be mentioned in the description.

Are the base image bumps required in this PR? (can they be delivered separately?)
What about the docs and scripts minor modifications?

You're right.
As I remember, due to some version changes the current master branch 6be6bff0fb894fd9f8adc441663c086c0ba4f4b6 can't be built, so some Dockerfile and base image changes required before doing anything, so that's why some PR precedes this. I thought this way I keep this PR at least build-able.
And those required commits has their own PR, so my plan was to rebase this PR after those PRs are merged.

Would like to see the PR got merged as the current add on is pretty out of date

Please merge. There are even newer versions available now so this merge is getting a little outdated.

It's not outdated, I regularly push new commits to this PR with the latest Tailscale releases when I update my fork.

I asked @frenck on Discord about and he said "I get to it when I get to it".
@lmagyar is there a way for me to use your branch in my Home Assistant Operating System deployment?

is there a way for me to use your branch in my Home Assistant Operating System deployment?

Yes, use my forks (they are fully functional drop-in replacements, images pre-built and stored by Github):
https://github.com/lmagyar/homeassistant-addon-tailscale
https://github.com/lmagyar/homeassistant-addon-tailscale-funnel (it has additional support for the new Tailscale Funnel feature (deatils))

It's not outdated, I regularly push new commits to this PR with the latest Tailscale releases when I update my fork.

Not stale in that sense. You cannot keep chasing your tail when things are not getting merged. I assume you've been trying to merge these updates for some time.

It's not outdated, I regularly push new commits to this PR with the latest Tailscale releases when I update my fork.

Not stale in that sense. You cannot keep chasing your tail when things are not getting merged. I assume you've been trying to merge these updates for some time.

@frenck

superseded by #152


Modified on date is overwritten after canceling from editing a page

Problem
If on a page, I click Edit then click Cancel then click History, the modified on most recent entry gets updated and displays that the last user modified the bag a few seconds ago. Even though I had canceled out of the operation, it is updating the modified on date.
Steps to replicate

Go to a wiki page, click Edit.
Click cancel to leave the page editor.
Click history. The latest entry in history "modified on" date has been updated due to going into editor and clicking cancel.

Another problem scenario

When you click Back from the browser instead of clicking cancel.

Additional Information
It may be that simply clicking Edit is causing the modified on date to change for the latest entry.

I found another problem which may or may not be related. It appears that if the user chooses to save the page, it is overwriting the created by user with the most recent user.
Steps to replicate:

Go to a page that wasn't created by your username, edit, and save changes.
Click on pages up at top to display page history, sort order by modified date. The page that you just modified updated the created by user with your username.

As of 8/10/2016, these issues appear to have been fixed. Thanks!


How to print cell outputs when running sphinx-build in notebooks

I need to debug a notebook and looking for a way to print all output to console. I tried to increase verbosity in sphinx-build but this did not help. Although the exception is captured and printed to console, nothing before/after (e.g. in a try-except-finally) is printed.

I don't quite understand what you mean.
Can you please provide a minimal example?

Assuming you have already set up a Sphinx project with nbsphinx add a cell with print('hello'), then run sphinx-build -v... My expectation was to see hello printed to stdout.

OK, I see.
There is currently no way to do that.
Can nbconvert --execute do that?

Can nbconvert --execute do that?

Running jupyter nbconvert --execute <notebook> does not print outputs. So I guess that this is probably related to https://github.com/jupyter/nbconvert/issues/648.

Yes, it looks like that has been requested for nbconvert but there has been no solution yet, but there have been suggestions to use alternative notebook runners.
So once nbconvert can do this, nbsphinx should be able to implement it, too.
I'm also open to suggestions for using an alternative notebook runner, but I'm not planning on working on that myself.


exptime to exposures table

Added a 4th column to the metadata on the exposures table: EXPTIME. Solves #50 .

Looks good; thanks.


browser: in all tags dialog, reset to first page on clear text

When the clear-text button is clicked in the all-tags filter input box:

reset all-tags dialog pagination to the first page
clear the all-tags input box
initiate a new search (to reset tag content in all-tags dialog)

Closes #518

Merged as 1ada8180d1866f0c5d2cb30e436732184c743fa2
Please remember to get  full build done and commit the java/static files, too! :)

@laurelnaiad Thanks for merging!
Just want to make sure I understand, were there java/static files missing in this PR? I see the following in my commit:
appserver/java-spring/static/app/dialogs/allTags.html
appserver/java-spring/static/app/dialogs/allTags.js

   * Clears text input and resets dialog.

was in allTags.js in the static build.

   * Clears text input and resets search.

was in the source code.

It means I have to amend the commit and then manually close the PR, just a bit of extra work for me.

Got it. Thanks for pointing that out and the reminder!

Sure. I've been seeing it a bit lately...


misc cleanup of mysql module

@jeremyrickard similar cleanup to what was in #308

Thanks for the review @jeremyrickard.


KeyErrors on nickname and firstname

Pytest test cases are failing on the keyerrors like nickname and first_name may be those are not correctly passed to the request data to be parsed by the method.
FAILED tests/test_schemas/test_user_schemas.py::test_user_base_valid - KeyError: 'nickname'
FAILED tests/test_schemas/test_user_schemas.py::test_user_create_valid - KeyError: 'nickname'
FAILED tests/test_schemas/test_user_schemas.py::test_user_update_valid - KeyError: 'first_name'

Added the nickname and firstname in the request data that is being passed to the create and update data.
Adding this had fixed the errors and test cases are successfully passed.


Scala mode throwing exceptions

The scala mode is throwing exceptions trying to parse this piece of code:
val out = (1 until x.length).map(x(_)).fold(x(0)){(z,f) => minOftwo(z,f)}
This seems to be a new issues. I'm pretty sure it's caused by 14dae96542c63c155adf6e7dfce79df1b0a43218

Actually, it looks like the scala demo page is throwing the same exception https://codemirror.net/mode/clike/scala.html


Extensive caching

Hi, this looks like a neat little DB for embedded development. I wonder whether you made any performance measurements? From the description it seems there is not much caching which could harm performance.
Do you (plan to) support user-adjustable aggresive caching techniques to lower latency and increase throughput?

Hi @dumblob ! No, I did not any performance measurements. And there is no cache, only write on copy actions. If you have a proposal for such feature we can discuss. The amount of required space is not big in general as the ideia is to use small flash sections for data storing but it can be a prolbem for microcontrollers with low RAM.

Yep, caching shall be optional but shall be the default IMHO.
I think before diving into caching strategies, we should measure the performance (and have a tiny "standardized" repeatable measurement process).
Intuitively I would cache a few last written words ("write cache") and maybe even a few lastly read words (if they were not present in the "write cache"). Some bit tagging could make this into one or two dereferences making it still much faster than true reads from the flash.


Fix Basic page

Amazing repo! While checking out some pages, I thought I'd fix some minor formatting bugs first but then I ended up making more changes, mostly around consistency. Let me know your thoughts. More PRs are coming.

Thank you!