Show reconcile status as string

Enums render as int, this gives a more human readable output:
status:
status:
Citadel:
status: 2
statusString: HEALTHY
EgressGateway:
status: 2
statusString: HEALTHY
...

@ostromart: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e_operator
fd38faa691f880422fecd75ce51fa3c1c89377f9
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e_operator
fd38faa691f880422fecd75ce51fa3c1c89377f9
link
/test e2e_operator

gencheck_operator
6af8943181aba84b5a0834d11db7a67d9a73c48b
link
/test gencheck_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e_operator
fd38faa691f880422fecd75ce51fa3c1c89377f9
link
/test e2e_operator

gencheck_operator
6af8943181aba84b5a0834d11db7a67d9a73c48b
link
/test gencheck_operator

lint_operator
6af8943181aba84b5a0834d11db7a67d9a73c48b
link
/test lint_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_operator
6af8943181aba84b5a0834d11db7a67d9a73c48b
link
/test gencheck_operator

lint_operator
6af8943181aba84b5a0834d11db7a67d9a73c48b
link
/test lint_operator

e2e_operator
6af8943181aba84b5a0834d11db7a67d9a73c48b
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart would approve this PR although the checks are failing badly.

There's something messed up with lint...

@ostromart: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_operator
6af8943181aba84b5a0834d11db7a67d9a73c48b
link
/test lint_operator

e2e_operator
fbd6ee0eea6a4803c8880d7db953cec62179eedf
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

In response to a cherrypick label: #458 failed to apply on top of branch "release-1.4":
error: Failed to merge in the changes.
Using index info to reconstruct a base tree...
M	pkg/helmreconciler/reconciler.go
Falling back to patching base and 3-way merge...
Auto-merging pkg/helmreconciler/reconciler.go
CONFLICT (content): Merge conflict in pkg/helmreconciler/reconciler.go
Patch failed at 0001 Show reconcile status as string


Bug: current_iter check in Checkpoint callback in RL trainer

I have found that there might be a bug in the current_iter in Checkpoint.
According to the definition: since on_iter_end is called after the bump of current_iter, when the previous iteration is considered complete.
This check should be
trainer.current_iter % self.every_n_iters
instead of
(trainer.current_iter + 1) % self.every_n_iters
Otherwise, it will cause mismatch of the evaluation metrics and the corresponding checkpoint being saved if every_n_iters > 1.
Assuming every_n_iters = 10, you will get checkpoints saved at 9-th, 19-th, 29-th iteration.

Hi,could you mind sending a PR to fix this issue and become one of the contributors of Qlib? : )


Introduce Iooxa.dev

I have just released https://iooxa.dev which is a suite of components and layouts to make creating interactive and reactive writing on the web easier.
There are tangle-like components, common-inputs, basic reactive-SVG components, as well as layouts, equations, callouts etc.  for common articles templates.
Let me know if you want any other details, or check out the docs at https://iooxa.dev, hopefully it is a good fit for this page! :)
Not sure if the ordering is important - I put it close to Idyll, as it is pretty similar. See https://github.com/iooxa/article/issues/8 for a discussion!

Thanks!


Tab close buttons

This change adds close buttons to tabs - these are visible when more than one tab is open. A sample of the close buttons is shown in the following screenshot.

The close button changes on focus. See additional sample screenshot below.

@shockone I am not sure what you had in mind when it came to mouse/ui design for tabs. Please let me if this is close to something you expect or not. I am happy to make changes to get closer to what you wanted.

Thank you for the contribution, @mkenne11. Design-wise I'd like the close button to appear on the left side (as in Safari tabs), and also it would be cool it it appeared only on hover to have less visual noise.

@shockone I believe I made the changes you requested. Please let me know if any extra changes are required :)

Thanks, @mkenne11. I'll merge it. I think I know a better way to handle closing, but I can modify it by myself instead of bothering you. Also, it looks buggy that the tab number is not aligned properly, but we can fix that separately.
Two more things:

Please don't use color values, try to get them from the map. You'll find a lot of examples in the CSS file. It will help when we modify the theme or implement themes support.
How strong do you feel about having the top line? I thought that distinguishing tabs by color should be enough. Besides, it makes the number move down, which also looks buggy.

No worries @shockone. I can see your point about using colour values from the map - you have a standard colour palette and you'd like like to use that - definitely makes things more consistent.
I don't feel that strongly about having the top border, it was an idea I had but I'm not wedded to it. And can definetly see your point that moving the number down makes it look buggy.

@mkenne11, I've eliminated the need for the value attribute by binding the index to the handler as we iterate over them. https://github.com/shockone/black-screen/commit/fcd7184714e95eae1170b5286a5489fdd4166a39

Thanks @shockone that looks cleaner.

I've messed with CSS quite a bit. Now it looks closer to the original tabs design. https://github.com/shockone/black-screen/commit/23921e449266d942a146aae628fa1cf9c7569f40
On hover:

On icon hover:

Hopefully I haven't offended you by modifying your work.

No probs at all I'm not offended :) your the main author of the project and have a vision for how things should look and feel. Plus I'd expect you'd want the "cleanest" commits possible to minimise code maintenance over time.

Awesome, @mkenne11. I'm glad you're cool with it.


feature: openrouter compatibility

Claude Opus Openrouter returns an EndTurn token that was not being handled before.

Thanks for the PR. Can you fix the formatting issue, then the PR is ready to be merged.


Update XMonad.Actions.TopicSpace

Description
The XMonad.Actions.TopicSpace module is extremely useful, but some
small idiosyncrasies make it harder to use than necessary.  This pr
tries to fix that.  I'll provide a short description of the commits and
what they aim to achieve:

Makes switchTopic update the last focused topic properly.
Before, this functionality was solely used for pretty printing the
topics, making the documentation of e.g. maxTopicHistory very
confusing.
Adds a new function, switchNthLastFocusedExclude, which makes it
possible to exclude certain topics when querying the history.
Corrects several spelling mistakes and tries to make the overall
module documentation a little more palatable.  The whole config is
divided up into smaller chunks and some parts that have nothing to
do with the module as such (like a lot of myConfig) were removed.
Some more example configuration was also added in places where this
made sense.
Indents everything to 2 spaces (the majority of the module was
already indented as such, but there were some outliers).
Properly fixes maxTopicHistory by making setLastFocusedTopic
aware of the length (again, so far maxTopicHistory was only used
for pretty-printing).
Applies some hlint hint
Changes the order of filtering the topic history list and consing
the given topic onto it (because we respect maxTopicHistory now
this is much more convenient when e.g. preventing certain topics
from ever entering the history).
Adds switchTopicWith, which allows one to pass a custom
filtering function to setLastFocusedTopic
Adds a new function to XMonad.Hooks.WorkspaceHistory, in order to easily
modify the workspace history with a pure function, because
Does away with the hand-rolled history that TopicSpace was using and uses
the one provided by XMonad.Hooks.WorkspaceHistory instead.  This is not a
user-facing change, as something like getLastFocusedTopics has just been
aliased.
Adds new, screen-aware, functions getLastFocusedTopicsByScreen and
switchNthLastFocusedByScreen (that do what they say).
Updates the changelog

Checklist

[X] I've read CONTRIBUTING.md

[X] I tested my changes with xmonad-testing

[X] I updated the CHANGES.md file

[ ] I updated the XMonad.Doc.Extending file (if appropriate)

@byorgey I saw that you're using this module, would you care to review this pr?

@slotThe will do, thanks for the nudge!

Looks great to me!  @slotThe , thanks so much for all the improvements!


Update ICameraHandler.cs

PR for issue #155
The playerLocation will only be send if connected to the server (playerCount > 0), also added a check to make sure the camera object exist.
also changed the hook method to LateUpdate, this should help a little bit with peformance.

Code looks good, I just need to physically test it.


Order APIs

@loosebazooka @etanshaul fix #526
I cleaned up the names, alphabetized what was left, and updated some versions.

JSON parsers maintain the order of arrays in the data, not quite the same as enforcing sort order.


Increase font size

Can we across the board make the font size a bit bigger? Whatever the climate-change-walleye-bass size is using, I think that would be better.

I sure can

I like it. I pushed it to dev. The headers are now the same size as the paragraphs, and the bullet points are still small. That being said, I think it looks fine that way, so I'm going to close the issue. But, have a look and if you disagree, you can feel free to do another pull request with some more fancy css updates.


Record more complete information about contestEntry

Fixes #213. Need help testing.

You really want to flex? :wink:
Why is your approach to store the display completely different from #217 ?

It's the same? 👀 Some fields are not there for backwards compatibility reasons, like pokemon_id.

You really want to flex? 😉

What's the point of grinding if I can't flex? 😉

Can we get some poggers on this PR?

looks solid, you tested this?

Actually I need to finish RM support for gym defender first. Please standby. :)

Can be merged now!

{"total_entries":9,"last_update":1726506754,"contest_entries":[{"rank":1,"score":1127.381020222036,"pokemon_id":428,"form":0,"costume":0,"gender":1,"shiny":true,"temp_evolution":1,"temp_evolution_finish_ms":1721221702374,"alignment":0,"badge":0,"location_card":0},{"rank":2,"score":1110.9649792598755,"pokemon_id":531,"form":0,"costume":0,"gender":1,"shiny":false,"temp_evolution":0,"temp_evolution_finish_ms":0,"alignment":0,"badge":0,"location_card":0},{"rank":3,"score":1106.0886862076218,"pokemon_id":352,"form":0,"costume":0,"gender":2,"shiny":false,"temp_evolution":0,"temp_evolution_finish_ms":0,"alignment":0,"badge":0,"location_card":0}]}

@Mygod seems to be ns, not ms, right? :)

Seems like ms to me?

{"total_entries":9,"last_update":1726506754,"contest_entries":[{"rank":1,"score":1127.381020222036,"pokemon_id":428,"form":0,"costume":0,"gender":1,"shiny":true,"temp_evolution":1,"temp_evolution_finish_ms":1721221702374,"alignment":0,"badge":0,"location_card":0},{"rank":2,"score":1110.9649792598755,"pokemon_id":531,"form":0,"costume":0,"gender":1,"shiny":false,"temp_evolution":0,"temp_evolution_finish_ms":0,"alignment":0,"badge":0,"location_card":0},{"rank":3,"score":1106.0886862076218,"pokemon_id":352,"form":0,"costume":0,"gender":2,"shiny":false,"temp_evolution":0,"temp_evolution_finish_ms":0,"alignment":0,"badge":0,"location_card":0}]}

@Mygod seems to be ns, not ms, right? :)

damn you are right... too late here :D

By the way I just checked and it seems like dynamax pokemon can enter showcases but displays as normal Pokemon in the game. Should we add this information now or hold out for later when it matters?

hold out for later... right now even dynamax pokemon are not displayed as dynamax in gyms :D

Regardless I'm in favor of keeping proto as is without doing any unit conversions. Those extra data might be useful for reverse engineering or something. We are not saving any storage/bandwidth anyways.

Umm... We already had this conversation 2 weeks ago...?

Regardless I'm in favor of keeping proto as is without doing any unit conversions. Those extra data might be useful for reverse engineering or something. We are not saving any storage/bandwidth anyways.

Umm... We already had this conversation 2 weeks ago...?

It's not about storage, just about consistency.. e.g. when RM wants to display it...  :)
But agree it's the same in gym display, so fine

Thank you. :)


Michael's todo before next meeting

[x] Research swift syntax
[x] Do a swift tutorial
[x] Discuss plans with Taige

Completed goals successfully


Android Training from Google Developer

https://developer.android.com/training/index.html

https://classroom.udacity.com/courses/ud853/lessons/1395568821/concepts/16438585680923

Setup gradle build
modified gradle version in this two files for Project Sunshine to run


Dask friendly check for nans in .weighted()

[ ] Closes #4541
[ ] Tests added
[ ] Passes isort . && black . && mypy . && flake8
[ ] User visible changes (including notable bug fixes) are documented in whats-new.rst
[ ] New functions/methods are listed in api.rst

Hello @jbusecke! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file xarray/core/weighted.py:

Line 102:1: W293 blank line contains whitespace
Line 111:1: W293 blank line contains whitespace

The Ci environments without dask are failing. Should I add some pytest skip logic, or what is the best way to handle this?

Yes, @requires_dask on the test

since the test you added requires dask it definitely should get a @requires_dask.
That won't fix all the failing tests, though: map_blocks is only valid if we operate on dask arrays. I guess you either have to apply the function directly on other arrays, or we have to make DataArray.map_blocks on non-dask arrays call to DataArray.map.

I think you need to do someting along the lines of:
if dask_duck_array(weights):
    import dask.array as dsa
    dsa.map_blocks(_weight_check, weights.data, dtype=weights.dtype)
else:
    _weight_check()

I did have to fiddle with this a bit. I did change .isnull() to np.isnan() and replace the data of weights,  otherwise the error would not be raised for the dask array at all.
This does not look terribly elegant to me, but passes tests locally. Waiting for the CI to see if the others pass aswell. Happy to make further changes, and thanks for all the input already.

Do you think this works or are further changes needed? Many thanks for the guidance so far!

I am getting some failures for py38-upstream (TestDataset.test_polyfit_warnings) that seem unrelated?

Similarly on MacOSX py38 this fails: TestDask.test_save_mfdataset_compute_false_roundtrip

the TestDask.test_save_mfdataset_compute_false_roundtrip failure should be #4539, and the failing upstream-dev CI shouldn't be related, either (my guess is that a dependency – most likely numpy – added a new warning). I'll investigate and open a new issue for that.

Ok I think this should be good to go. I have implemented all the requested changes. The remaining failures are related to other problems upstream (I think). Anything else I should add here?

I am not understanding why that MinimumVersionPolicy test is failing (or what it does haha). Is this something upstream, or should I fix?

this CI is sometimes flaky, it's usually enough to rerun it. I'll do that for you once the other CI finished.

Seems like all the other test are passing (minus the two upstream problems discussed before).

Thanks @jbusecke


Requirements

It is generally a best practice to include a requirements.txt file at the top level of the repository when 3rd party dependencies are used in the package.
Here the plotting part of dap-py says it requires numpy and matplotlib, so I would recommend adding these to a requirements.txt file. Anyone who clones this repository can then run pip install -r requirements.txt to install numpy and matplotlib. If you are creating a python package, then these can also be included in your setup.py file and when users run pip install dap_py they will also get these extra requirements.

Making extra requirements optional for the pip package is not too tricky either, if you are thinking that you'd rather not have users install extras by default: See python packaging extras / optional dependencies for a tutorial

Added in 3bc40c7b0d2bb6ac6353c2e93978ebc9c27366c1


Using HPC cluster as remote

Nice tool! I was wondering if it would be possible to include HPC cluster environment as a remote (For example using torque)

Great idea! Yes the expectation is that Guild will support remote types like torque. Slurm is another candidate. These remotes take a fair amount of work to support - and of course it helps to have access to an environment, which can be quite complex to setup and maintain over the course of development, not to mention the need to run regression tests for releases.
Short answer, yes in theory, but it's a big investment.
There might also be a more direct path to supporting what you want, short of full blown remote support.
If you're on the Slack workspace, ping me or start a topic in a channel and we can discuss in real time.


Tests are now being executed during gcc builds

This [re]enables tests during gcc builds.
Please note this PR shouldn't ve merged before https://github.com/envoyproxy/envoy-build-tools/pull/34.
I'm not sure if it's ok to use bazel.gcc target, or a new target should be added, decided to use bazel.gcc in this PR.

@dmitri-d merged envoyproxy/envoy-build-tools#34

@lizan: could you restart the tests please?

/azp run envoy-linux

There's a bunch of failures due to memory leaks:
*** WARNING: Cannot convert addresses to symbols in output below.
*** Reason: Cannot find 'pprof' (is PPROF_PATH set correctly?)
*** If you cannot fix this, try running pprof directly.
Leak of 8 bytes in 1 objects allocated from:
	@ 40ac9c
	@ b4b8ad

I tracked them down to static initialiser here: https://github.com/envoyproxy/envoy/blob/master/source/extensions/common/crypto/utility_impl.cc#L105. Not sure what to do about these other than to pass "HEAPCHECK=minimal" to tests.

ping

@dmitri-d you need to do the build image update as well to pick up gcc-9, so this can't be in until #10506 is merged.

@lizan: could you restart the tests pls?

@dmitri-d you need merge master.

merged in master

some QUIC test failed to build in gcc. cc @danzh2010

@lizan: the build fails due to https://github.com/envoyproxy/envoy/issues/10422.

@lizan: could you re-open the PR please? At least one of the quiche-related issues has been resolved.

@lizan -- not sure what to do with these failures, any thoughts? There are a lot of memory leaks reported, and I think I have a few of these tests passing locally just fine.

@dmitri-d seems its the heap profiler detected some leak, not sure whether its a false positive. If you think it's false positive, add --test_env=HEAPCHECK=?

@lizan: pprof points to https://github.com/envoyproxy/envoy/blob/master/test/fuzz/fuzz_runner.cc#L56 as the source of the leak. I can try disabling heapcheck for log_silencer, but it would require dragging in pprof as a dependency.

@dmitri-d don't even specify minimal I meant just do --test_env=HEAPCHECK= literally to disable it.

MacOS tests are failing today, and one shard of //test/integration:protocol_integration_test failed, which I think is a bit flaky? WDYT, @lizan?

merged in latest changes

Hitting https://github.com/envoyproxy/envoy/issues/7647 in dynamic_opentracing_driver_impl_test
In quic_platform_test, the stacktrace generated with -c opt doesn't include the test where the exception was thrown
stats_integration_test reports different values under gcc
MacOS tests continue to flake

@lizan -- I think you had a fix here (https://github.com/envoyproxy/envoy/compare/master...lizan:dynamic_ot?expand=1) for the dynamic_opentracing_driver_impl_test failure? Not sure what effect it will have on clang/libc++ toolchain though. As an alternative, I can disable this test under gcc altogether.
Not sure what to do re: quic_platform_test, except disable it under gcc.
stats_integration_test: I can drop ENVOY_MEMORY_TEST_EXACT=true when running tests under gcc.

@dmitri-d

feel free to reuse my patch (better upstream to opentracing though). I don't think it affect clang/libc++.
for quick-platform-test, I'm fine with ifdef there too.
for stats_integration_test, yes let's drop that. they will never match.

dynamic_opentracing_driver_impl_test is passing now
disabled quic_platform_test
under gcc stats_integration_test uses Mode::Approximate even when ENVOY_MEMORY_TEST_EXACT=true

Tried using https://github.com/envoyproxy/envoy/compare/master...lizan:dynamic_ot?expand=1, but it breaks asan (libc++ and tcmalloc in general?) dynamic_ot:dynamic_opentracing_driver_impl_test and dynamic_ot:config_test tests with:
C++ exception with description "opentracing: failed to load dynamic library: /b/f/w/bazel-out/k8-dbg/bin/test/extensions/tracers/dynamic_ot/dynamic_opentracing_driver_impl_test.runfiles/io_opentracing_cpp/mocktracer/libmocktracer_plugin.so: cannot allocate memory in static TLS block" thrown in the test body.

As much as I dislike it, I think disabling these tests under gcc is the way forward.
@lizan

@dmitri-d sgtm

@lizan: the build is green after disabling dynamic_ot:dynamic_opentracing_driver_impl_test

Removed ENVOY_MEMORY_TEST_EXACT from the gcc target
merged in the latest master

@lizan: could you restart the build please? There were networking issues during the last run.

Looking at errors from gcc tests.

Refer to https://github.com/envoyproxy/nighthawk/issues/382#issuecomment-650179207 for explanation of problem.

Added 'override' to MockGrpcMux methods implementing the interface

@lizan: Not sure why asan failed (all tests passed), otherwise looks good.

resolved conflict, merged in latest changes

@lizan, the build is green now.

Thank you very much for the reviews and feedback, @lizan.


[Proposal] Consolidate string column handling

String column is currently represented as *array.String|*array.Binary . This is not consistent because most cases I have seen string((*array.Binary).Value(n)).
I'm proposing being consistent and use *array.String for string column. We can introduce binary column and use *array.Binary for it if it is necessary.
Only blocker I see is the Sample struct
type Sample struct {
	ExampleType string            `frostdb:"example_type,rle_dict,asc(0)"`
	Labels      map[string]string `frostdb:"labels,rle_dict,asc(1),null_first"`
	Stacktrace  []uuid.UUID       `frostdb:"stacktrace,rle_dict,asc(3),null_first"`
	Timestamp   int64             `frostdb:"timestamp,asc(2)"`
	Value       int64             `frostdb:"value"`
}

The Stacktrace field type  []uuid.UUID has  special handling that resolves to []byte, however this is not used  anywhere else apart from tests. I confirmed it is not in parca.
Why do we special handle []uuid.UUID ?  If it is not used anywhere else apart from tests, I suggest we get rid of it as well. Most of *array.Binary chunks revolve around this special case.

We used to utilize the uuid behavior on cloud. However that hasn't been the case for quite some time. I think it's acceptable to normalize the behavior and remove historical choices that haven't been used in a while.

We should check if this is still the case but I remember there being non zero overhead in the rest of the query engine and that’s why we wanted to user to be able to control this.

We should check if this is still the case but I remember there being non zero overhead in the rest of the query engine and that’s why we wanted to user to be able to control this.

What do you mean?  Are you talking about *array.Binary vs *array.String overhead during query time?

Closing this for now.

I don’t remember the exact details but this was because there was some overhead when using string columns in Parca’s code that happens after the query engine but ultimately the idea is to have the query engine be extensible and the work that Parca currently does on top will just be a query layer.


fix(build.in.yml): escape special characters in quotes

Followup to #14077 which was causing the build mathlib step to not actually run the build, e.g. see this log.
Zulip.

Please make a small change to a .lean file somewhere (e.g., a whitespace change) to ensure that it works. If it works, then please merge.
bors d+

Thanks!
As the test commit did build successfully, I have reverted it (canceling the rest of the steps), and will now merge:
bors r+


Fix symbols names on macOS aarch64

libSystem on macOS Big Sur on aarch64 exports these symbols without the $suffixes. See also: https://github.com/GuillaumeGomez/sysinfo/issues/372

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @JohnTitor (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

@bors r+

:pushpin: Commit ecffd09397ccf3855eb3e933d12bfc3907d6db8b has been approved by JohnTitor

:hourglass: Testing commit ecffd09397ccf3855eb3e933d12bfc3907d6db8b with merge c9e8bab1e0e7c482f6c1d3dd7bc473078453e74f...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: JohnTitor
Pushing c9e8bab1e0e7c482f6c1d3dd7bc473078453e74f to master...


Remove ACC tuning from 9th Gen Accord

Updated tuning to use the standard Honda tuning. No longer needed on both pedal and PCM control.

Thanks for the PR!


Angle terrain skirts

Terrain skirts should be angled under the terrain in order to further improve rendering fidelity.  This can help further reduce visibility of seams.

Agree.


ScrollView scroll to textbox not working when app loses focus

We are using a KeyboardAvoidingView inside a  ScrollView to allow our TextInputs to scroll above the keyboard when it's open. When functioning correctly, the ScrollView is automatically scrolled to the correct correct position to allow for the size of the keyboard.
<ScrollView>
    <KeyboardAvoidingView>
          {this.props.children}
    </KeyboardAvoidingView>
</ScrollView>

However, when the application loses focus and then re-gains focus, this auto-scroll functionality breaks. Instead, when you focus a TextInput, the keyboard opens and covers it and no auto-scroll happens, this happens for the remainder of the app session. A force close of the app fixes it.
React Native version: 0.37.0
Platform: Android (iOS untested)

Can you reproduce this on rnplay.org? Please provide a link.
I'm not sure but should it be the other way round? ScrollView inside a KeyboardAvoidingView?

@facebook-github-bot no-reply

Hi there! This issue is being closed because it has been inactive for a while. Maybe the issue has been fixed in a recent release, or perhaps it is not affecting a lot of people. Either way, we're automatically closing issues after a period of inactivity. Please do not take it personally!
If you think this issue should definitely remain open, please let us know. The following information is helpful when it comes to determining if the issue should be re-opened:

Does the issue still reproduce on the latest release candidate? Post a comment with the version you tested.
If so, is there any information missing from the bug report? Post a comment with all the information required by the issue template.
Is there a pull request that addresses this issue? Post a comment with the PR number so we can follow up.

If you would like to work on a patch to fix the issue, contributions are very welcome! Read through the contribution guide, and feel free to hop into #react-native if you need help planning your contribution.


Script for test user creation

Script that creates test users and optionally a clan or classroom for them.
This is useful for dev instances whenever a number of users is needed for developing / debugging / testing something.
As an example: #2975
The script takes 3 seconds to create 0-100 users and 6 seconds to create 1000 users in my dev vm.
It is well-structured and has a usage comment at the top and can serve as an inspiration for more test data scripts.
My motivation for this, copied from previous slack discussion:

from the wiki pages and issues I've read so far it seems a common workflow is "if you need data, let us create a dump from prod for you" - giving people the facilities to create the data by themselves means they don't need to wait for it, and means nobody has to go make a dump and sanitise it. Having a handful of (ideally well-maintained and integrated into automatic tests) scripts to generate some commonly-needed data also gives people something to push off of to adapt them to make their own for their own use cases.

It would be good to integrate this sort of generation logic with the server tests. They do a lot of data generation, and have a system for linking models together consistently. For example: user creation and classroom creation with users added as members. These two in action.
Some of the generation functions rely on calling endpoints with a global request object, which is suboptimal since you have to login as a certain user sometimes for the model creation to work (you have to login as a teacher to create a classroom, for example). What would be a way to organize these such that scripts, tests and the server can all use the same logic to create new models?


Mail server

So i just installed this template and used mailhog, in readme it says just run npm install and grun serve, but i already fail at npm install as there is no package.json? Am i doing this wrong or whats recommended to get this local mail testing up?

package.json and Gruntfile.js only is shipped on generated project if you select "use_grunt [n]: y"
if "use_grunt [n]: n", package.json and Gruntfile.js is removed by cookiecutter post-generate hook:
https://github.com/pydanny/cookiecutter-django/blob/master/hooks/post_gen_project.py#L180-L181

Oh forgot about that i fixed now but it does not work "from the box"?

Running "bgShell:runMailHog" (bgShell) task
Running "bgShell:runDjango" (bgShell) task
Running "watch" task
Waiting...

/bin/sh: ./mailhog: No such file or directory
Error: Command failed: ./mailhog
Running "sass:dev" (sass) task

Done, without errors.


Add token expired exception

This commit adds a subclass of HyperResource::ClientError for token
expires. So far, this is only intended to be used to shut Sentry up since
having a bunch of token expiry errors doesn't add much value, but at
the same time we do want to know about legitimate
HyperResource::ClientErrors.
cc @krallin

I was hoping this might be something we could in Sentry configuration with e.g. a filter (considering the issue appears to lie there); did it turns out not to be / were you able to look?

To be more precise, the reason I was asking this is that it might be easier to just filter those in aptible_api_shared, rather than have to update all the APIs to expose this new exception, and then still update aptible_api_shared in order to filter it out.
I think Sentry lets us add filters reasonably easily, so it might be possible to just drop this one there rather than have all consumers see a new exception that might not be completely relevant to them.
Hope this clarifies why I was asking :)

The Sentry config does have a should_capture method which takes a Proc as an argument. The filtering would look like something below:
config.should_capture = Proc.new { |e| true unless e.class == HyperResource::ClientError && e.body == 'token_expired' }

The reason why I took the subclassing approach is because I feel it's more clear what is happening, since it isn't obvious from the Sentry documentation what the semantics of mixing should_capture and excluded_exceptions are (I had to take a look in raven-ruby to see what actually happens).
I'm happy to take the Proc approach if you prefer it though.

Up to you; I'm mentioning this as an alternative (mainly because I'm a little concerned we're going to end up with a an arbitrary set of exceptions that are wrapped into their own subclass), but if you'd rather go with this approach, that's fine.


Getting 404 Not found.

I added the headers and body as per the format specified in the Documentation. Tried it using Postman.
Getting a 404 error response.

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

ID: 78411727-ea56-aef5-4f04-b016c25afba9
Version Independent ID: 8687b7bc-a196-6fd0-fa57-9fa5dd2d4545
Content: Send metrics to the Azure Monitor metric database using REST API - Azure Monitor
Content Source: articles/azure-monitor/platform/metrics-store-custom-rest-api.md
Service: azure-monitor
Sub-service: metrics
GitHub Login: @anirudhcavale
Microsoft Alias: ancav

Thank you for your feedback! Since this channel is for driving improvements towards MS Docs, could you provide the URL of the specific documentation that your feedback is applicable to? That way, we are able to make sure your feedback is addressed by the right team.

https://docs.microsoft.com/en-us/azure/azure-monitor/platform/metrics-store-custom-rest-api
This is the url of the docs

@sanketpandia can you tell me what region you are trying this in? And share the URL you are sending the data to?

hello @sanketpandia  - can you share the information requested by @anirudhcavale?

As we havent heard from you, we will now close this issue. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Implement yaas repo

LGTM :+1:

Let me know what you think about yaas generate.
Do the config to client change, or don't and we'll do another PR with the change in all the other files too.
 If you have a button on one place in the screen, it shouldn't affect something somewhere else on the page. Similarly, I don't think you should have to do yaas generate repo | yaas repo create HDP 2.3
I'll make the config->client change and merge.
That was a great SNL skit haha.

fwiw, I hadn't thought of yaas generate repo | yaas repo create HDP 2.3, but it kinda turns me on.

Also, generation logic is all related: it doesn't touch Ambari; it produces something that Ambari expects you to create on your own.


getting "Error occurred"

⚡[server]: Server is running at http://localhost:8888
Error occurred [ 'Response code 400 (Bad Request)' ]
my .env has my client id and my secret put in correctly but still wont work hmm

Response from: https://accounts.spotify.com/api/token [
{
statusCode: 400,
body: '{"error":"invalid_client","error_description":"Invalid client"}',
retryCount: 0
}
]
Error occurred [ 'Response code 400 (Bad Request)' ]

Response from: https://accounts.spotify.com/api/token [
{
statusCode: 400,
body: '{"error":"invalid_client","error_description":"Invalid client"}',
retryCount: 0
}
]
Error occurred [ 'Response code 400 (Bad Request)' ]

nvmd i got it working, for some reason it couldnt acsess the .env file


added registercanary header

We prefer small, well tested pull requests.
Please refer to Contributing to Spinnaker.
When filling out a pull request, please consider the following:

Follow the commit message conventions found here.
Provide a descriptive summary for your changes.
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Add inline code comments to changes that might not be obvious.
Squash your commits as you keep adding changes.
Add a comment to @spinnaker/reviewers for review if your issue has been outstanding for more than 3 days.

Note that we are unlikely to accept pull requests that add features without prior discussion. The best way to propose a feature is to open an issue first and discuss your ideas there before implementing them.
https://devopsmx.atlassian.net/browse/OP-7035

This PR  triggered this build in this execution, please merge to master only if this succeeds:  https://spin.prod.opsmx.com/#/applications/gitprtrigger/executions/details/01FAPZVBARAVRBV22HM4G1ZDWY


[Enhancement] Optimize the de-serialization of vector when reading from Doc Values

Description
In reference to the experiments done in here: https://github.com/opensearch-project/k-NN/issues/1049, I was found out that when we flip to exact search and use the doc values do the exact search we spend like 60-70% of the time in de-serialization of vectors from binary format to float array. Hence as part of that there was few solutions provided, capturing them as part of this issue. Experiment details:
Experiment 1 (Baseline to Find latency for doing distance calculation on the different size dataset with different dimensions)Below are the steps that are performed to run this experiment:For every dimension and every run, generate a dataset with required number of vectors in it. We choose 1k, 2k, 5k, 8k, 10k, 15k vectors count.After query vector is generated.Now distance calculation is done on the dataset for every run and every dimension. The time to do distance calculation is stored and this is what we see in the tables below. Lucene distance calculation functions are used to do the distance calculation. code ref The experiment then collects all the latencies and format them in a table.

dimensions
1000_latency(ms)
2000_latency(ms)
5000_latency(ms)
8000_latency(ms)
10000_latency(ms)
15000_latency(ms)

64
0.0751
0.07755
0.18681
0.33033
0.40572
0.60473

128
0.11913
0.14973
0.38251
0.64307
0.92643
1.17756

256
0.21269
0.30679
0.78219
1.26894
1.8096
2.38399

512
0.38937
0.5992
1.50937
2.5473
3.08621
4.54336

768
0.60041
0.88308
2.35232
3.64441
4.63974
7.03523

968
0.5958
1.11113
2.90708
4.61883
5.92536
8.58447

1024
0.60106
1.20861
3.12053
4.83357
6.24293
9.18919

2048
1.16614
2.45611
6.04847
9.83145
12.46421
18.96537

4096
2.39995
4.86545
11.93488
19.87645
25.46671
40.70765

Experiment 2(Doing de-serialization on vectors and then doing distance calculation)
Below are the steps that are performed to run this experiment:

For every dimension and every run, generate a dataset with required number of vectors in it. We choose 1k, 2k, 5k, 8k, 10k, 15k vectors count. The vectors in this dataset are serialized using the same code we use in K-NN(Code ref).
After query vector is generated. We don’t need to serialize the query as this is not what happens in the exact search.
Now to do the distance calculation we first de-serialized the vectors in the dataset using same code used by k-NN(Code ref) and then performed the distance calculation using Lucene provided functions. This is done for different dimensions and dataset. The time taken in this full step de-serialization and distance calculation is what being provided in below table. Lucene distance calculation functions are used to do the distance calculation. code ref
The experiment then collects all the latencies and format them in a table.

So, the query latency include:

Time taken to de-serialize the vectors to floats.
 We can see that with 512 dimensions and 5K vectors we are taking around 2.6 ms for de-serialization which is around 63% of the time(4.18ms taken to compute the distances) and around 69% for lower dimensions like 128.

Time Spent In De-Serialization in %age
--

dimensions
1000_latency_diff
2000_latency_diff
5000_latency_diff
8000_latency_diff
10000_latency_diff
15000_latency_diff

64
85.99
74.51
75.99
74.20
77.58
74.38

128
72.38
72.64
69.37
66.69
62.66
67.15

256
63.50
71.15
62.22
65.69
63.08
72.64

512
51.16
63.89
63.91
61.87
63.82
70.14

768
48.01
59.90
58.61
61.04
61.45
69.99

968
58.08
61.07
62.45
67.63
60.43
74.09

1024
58.95
61.70
58.40
66.82
63.28
69.11

2048
60.83
64.19
63.41
60.35
58.69
75.98

4096
59.34
58.59
61.57
69.01
69.41
66.70

Experiment Conclusion

The above experiments showed us that its the De-Serialization that is taking up most of the time during the exact search.

Possible solutions:

Short Term: To improve the exact search time, we should enable a caching layer for this de-serialization so that we can improve the latencies for exact search. Along with this SIMD instructions set will help reducing the overall time.
Figure out better format to store the vectors. Like can we use .vec file which is used by lucene engine too.

Benefits
This will feature will not only boost the filtered search but will also help in reducing the latency for script score query for k-NN where we read these vectors to perform the exact search.
This will also provide us some gains during the merge operation of segments as we read the binary doc values there too.

Caching could be potentially tricky. I wonder if we should first evaluate lucene's .vec approach and potentially move towards that instead of first implementing caching. In the long run, I think we should convert our codec to KnnVectorFormat anyways.

Agree we should move to KnnVectorFormat in long term and piggy back on the improvements from Lucene.  Should we create dedicated GitHub issue to merge the codec to Vector classes used by Lucene?

@vamshin we can do that. But we need to make sure that we are not loosing focus on what we want to achieve. Plus BWC can be challenge here.

@navneet1v I think merging to KnnVectorFormat is one of the solution to get optimizations. If we think its a long term solution, I dont see having dedicated issue is a problem. I am fine if we can track it part of the same issue as well.

@vamshin I am not saying we shouldn’t create a separate issue. I just want to make sure that we are not loosing focus on why we want to merge the format.
Also, merging formats is a big change. Given de-serialization is a long poll should we consider other alternatives too. Also, will having same vector format solves the core problem? What if lucene also has the same issue? I think there are unknowns here hence I first want to make sure first that we have exhausted all the options

How do we get the time spent in deserialization?
Is there an API that exist already which provides this number?

@navneet1v @bugmakerrrrrr Id be interested to see if with #1573 the derserialization of vector values is faster or slower than our binarydocvalues
The above experiments i think would be hard to retry with vector values, but Im wondering if we could just run with script scoring instead of with faiss and check the latency diff.

@navneet1v @bugmakerrrrrr Id be interested to see if with #1573 the derserialization of vector values is faster or slower than our binarydocvalues
The above experiments i think would be hard to retry with vector values, but Im wondering if we could just run with script scoring instead of with faiss and check the latency diff.

+1 we can do that. We have added micro-benchmarks module in k-NN can we leverage that? @jmazanec15 what your thoughts? or we should go with script score query and do this on a cluster?

@navneet1v I think micro-benchmarks would be the right way to see the diff - I started trying this comparison here: https://github.com/jmazanec15/k-NN-1/blob/micro-benchmarks/micro-benchmarks/src/main/java/org/opensearch/knn/VectorSerdeBenchmarks.java, but Im not sure it is quite apples to apples comparison - so Im not pursuing this any further.
I think we would need to try to figure out how to properly setup this class to run the test: https://github.com/opensearch-project/k-NN/blob/main/src/main/java/org/opensearch/knn/index/KNNVectorDVLeafFieldData.java#L24-L28. It might get a little tricky initializing the LeafReader and everything though.

We could potentially re-use logic from the tests: https://github.com/opensearch-project/k-NN/blob/main/src/test/java/org/opensearch/knn/index/KNNVectorScriptDocValuesTests.java#L54

@jmazanec15 I think we can do something like this: https://github.com/opensearch-project/OpenSearch/blob/main/benchmarks/src/main/java/org/opensearch/benchmark/index/mapper/CustomBinaryDocValuesFieldBenchmark.java#L72-L78 to make the docValues for micro benchmarks.

@navneet1v ah nice that looks good

FieldDataCache in my understanding was an old concept of Lucene which is used before DocValues came in.
You can do the benchmarks but as per my knowledge its not used now.
I would say lets come up with another Cache layer where we can cache the vectors serialized from DocValues.

I decided to start from running some end to end benchmarks to get an initial sense of the perf difference between the two vector representations. I created a repo for running some of these experiments from docker: https://github.com/jmazanec15/opensearch-knn-rescore-experiments. The repo is very basic right now. But I ran some preliminary tests and the initial results suggest that the KnnVectorsFormat may perform better for script scoring:
10000 vectors, 1000 queries, k=100, dim=1024, space=l2
nmslib 1: 35.33668613433838 seconds to run queries
nmslib 2: 35.46980571746826 seconds to run queries
faiss 1: 35.574172258377075 seconds to run queries
faiss 2: 35.60338592529297 seconds to run queries
lucene 1: 20.01723551750183 seconds to run queries
lucene 2: 20.044759511947632 seconds to run queries

(ref)
Will iterate on these experiments and have some higher fidelity results in the next week or two.

@jmazanec15 thanks for sharing the initial results.
can we capture the flame graphs for these runs?


Implement 'Ord' for Token

I'd like to store the Token associated with my Evented types in a BTreeMap but cannot because Token doesn't implement Ord. I'm happy to open a pull request for this but wanted to open an issue first to make sure it made sense.

Fine with me!


' ' not implied with width

Doc says "Specifying width forces a fixed width format. Flag '#' is implied, ' ' is implied unless '+' is given"
but the ' ' doesn't seem to be implied, leaving the result not fixed width.

fixed.  this changes the formatted results in places.


uninitialized constant Devise::SessionsController

Environment

Ruby [3.2.1]
Rails [7.0]
Devise [4.9.2]
Activeadmin [2.13]

Current behavior
I have rails application that was built on ruby 2.7.1, rails 5.2, devise 4.7.2, and activeadmin 2.9.0. i want to update it to the current versions. when i update the ruby to 3.2.1, rails to 7.0, devise to 4.9.2, and activeadmin to 2.13, i got the  uninitialized constant Devise::SessionsController in gem lib file .rvm/gems/ruby-3.2.1/gems/activeadmin-2.13.1/lib/active_admin/devise.rb:54:in <module:Devise>': uninitialized constant Devise::SessionsController (NameError)
Expected behavior
Evertyting need to be work wihout error

I'd recommend doing smaller upgrades and not going all at once to the latest. You could for example upgrade Devise without upgrading everything else first.
It sounds like this error is coming from active admin though, not Devise, so I'd recommend reporting there. If you can provide a sample app showing the error I'd be happy to try looking into it more, otherwise there's not much else I can do without additional info, stack traces, and/or a sample app.

I have also faced the same issue. I had 2FA integration for active admin and it was working fine for me.  but I have to upgrade my ruby and rails version to ruby 3.2.1 and rails 7
When I updated this I faced the same issue it started giving
rake aborted!
NameError: uninitialized constant Devise::SessionsController (NameError)
class SessionsController < ::Devise::SessionsController

**Solution**
# app/controllers/admin/custom_sessions_controller.rb

`# app/controllers/admin/custom_sessions_controller.rb

module Admin
class CustomSessionsController < ActiveAdmin::Devise::SessionsController
include ::ActiveAdmin::Devise::Controller
def create
  super do |resource|
    # Generate and save OTP
    resource.generate_and_save_otp
    puts "Mobile Verification code for admin login is #{resource.mobile_otp}"
    if resource.mobile_otp.present?
      redirect_to admin_otp_verification_path(resource_id: resource.id) and return
    end
  end
end

end
end
`
routes.rb file
`custom_config = ActiveAdmin::Devise.config.merge(
controllers: { sessions: 'admin/custom_sessions' }
)
devise_for :admin_users, custom_config
ActiveAdmin.routes(self)`


T265 fail to restart after stop sensor without close on Windows

Required Info

Camera Model
T265

Firmware Version
<IP_ADDRESS>6

Operating System & Version
Win 10 x64 10.0.19041.928

Platform
PC

SDK Version
2.32.1

Language
C++

Issue Description
I have maintained a program using old SDK 2.30.0 that available to start and stop T265 sensor individually. Recently it is upgraded to latest version 2.44.0 for other features and the sensor fail to restart the data stream after stop. After test, the situation appears between 2.31.0 and 2.32.1.
The program will use on a tablet. Pause the T265 can better save the battery consumption. Reconnect each times is a method but the program designs to connect all devices before use, or block the usage. Will it pay attention on solving this problem? Or is there another way such that connect T265 but deactivate the data updating? Any recommend for last question?

Did you end up finding a solution?


CFE should not allow redirection from const to non-const constructor

CFE version of https://github.com/dart-lang/sdk/issues/27617
Caught by language_2/regress_27617_test.

The same goes for factory constructors. See co19_2/Language/Classes/Constructors/Factories/const_modifier_t01 and co19_2/Language/Classes/Constructors/Factories/const_modifier_t02.


mergeSegmentsUsingPreset Error 11820

I receive this error occasionally (when recording a video in my app 1 out of 20 times) when using mergeSegmentsUsingPreset:
Error Domain=AVFoundationErrorDomain Code=-11820 "Cannot Complete Export" UserInfo=0x1748662c0 {NSLocalizedDescription=Cannot Complete Export, NSLocalizedRecoverySuggestion=Try exporting again.} {
NSLocalizedDescription = "Cannot Complete Export";
NSLocalizedRecoverySuggestion = "Try exporting again.";
}
When it happens, it breaks all AVPlayers in the app.  None of them work and the only way to make the AVPlayers work again is to send the app to the background and bring it back to the foreground.
Any ideas of what could be happening?

@lapat can you post your export code? I was getting this error a while back and it was something to do with the way the export session was getting getting configured.
Another thing you can try is using the AVAssetExportSession and see if you get the same error.

@mjgaylord I started to use my own AVAssetExportSession as soon as I started getting the error but still get the same error occasionally.  The error will come usually after calling the export session several times (15+).  Then, once I get the error once, it never goes away and I have to kill the app and restart it.  I thought maybe it was related to old videos taking up space but I delete all the old videos and still get the error.
    AVAsset *asset = recorder.session.assetRepresentingSegments;
    AVAssetExportSession *exportSession = [[AVAssetExportSession alloc] initWithAsset:asset presetName:AVAssetExportPresetMediumQuality];
    exportSession.outputFileType=AVFileTypeQuickTimeMovie;
    NSError *error;
    [FCFileManager removeItemAtPath:[NSTemporaryDirectory<EMAIL_ADDRESS>error:&error];
    if (error){
        NSLog(@"File delete error %@", error);
    }
    exportSession.outputURL = [NSURL fileURLWithPath:[NSTemporaryDirectory<EMAIL_ADDRESS>    [exportSession exportAsynchronouslyWithCompletionHandler:^{
        switch ([exportSession status]) {
            case AVAssetExportSessionStatusFailed
           handleFailure...
        }
      case AVAssetExportSessionStatusCompleted:
            {
           handleSuccess...
           }
    }];

Here's my initialization calls:
recorder = [SCRecorder recorder];
recorder.session = [SCRecordSession recordSession];
recorder.captureSessionPreset = AVCaptureSessionPresetHigh;
recorder.device = AVCaptureDevicePositionBack;
recorder.maxRecordDuration = CMTimeMake(30, 1);
recorder.delegate = self;
recorder.initializeSessionLazily = NO;
videoConfiguration = recorder.videoConfiguration;
videoConfiguration.scalingMode = AVVideoScalingModeResizeAspectFill;
videoConfiguration.sizeAsSquare = YES;
[recorder startRunning];


Pod library?

Can you add to cocoapods EasySocial?

I definitely want to create a pod library.
I had a quick look at the cocoapods website and it looked very confusing. I will need some spare time in the near future to properly read the information and do the bare minimum steps and create the bare minimum files to create it and put it in the cocoapods repository.
If anyone has done it before (and only if they have done it before), feel free to write down the steps below. I'll have it done within a month.

pod 'EasySocial', '~> 3'


Fixing the simple libertem API.ipynb

I had fixed the Libertem API.ipynb but while doing it I had to replace the original file with my file, so pls check it while merging. By the way, I had fully tested it except the location for the raw file and some minor changes other things are as it is.

Thanks, this looks good!

I had to replace the original file with my file, so pls check it while merging

Yes, this should be fine. I used nbdime to compare the two versions, and it looks fine!

By the way, I had fully tested it except the location for the raw file and some minor changes other things are as it is.

I also ran the notebook locally, and with your changes, it runs without errors. I'm going to go ahead and merge this now.

Thanks, @sk1p


Fix different results over three backends for ResNet50 and MobileNet

As discussed in #9457, there are the discrepancies in ResNet50 and MobileNet. The tests scripts are:
import numpy as np
from keras.preprocessing import image
from keras.applications.mobilenet import decode_predictions

from keras.applications.mobilenet import MobileNet
from keras.applications.mobilenet import preprocess_input as p1
from keras.applications.resnet50 import ResNet50
from keras.applications.resnet50 import preprocess_input as p2

model1 = MobileNet(weights='imagenet')
model2 = ResNet50(weights='imagenet')

img = image.load_img('cat.png', target_size=(224, 224))
x = image.img_to_array(img)
x = np.expand_dims(x, axis=0)
x1 = p1(x.copy())
x2 = p2(x.copy())

print('Predicted:', decode_predictions(model1.predict(x1), top=3)[0])
print('Predicted:', decode_predictions(model2.predict(x2), top=3)[0])

The outputs of CNTK and Theano are:
 $ KERAS_BACKEND=cntk python test_mobilenet.py
Using CNTK backend
Selected GPU[0] Tesla P100-PCIE-16GB as the process wide default device.
('Predicted:', [(u'n02123159', u'tiger_cat', 0.25886607), (u'n02124075', u'Egyptian_cat', 0.14433344), (u'n02123045', u'tabby', 0.06473238)])
('Predicted:', [(u'n02124075', u'Egyptian_cat', 0.17055313), (u'n02123597', u'Siamese_cat', 0.11803201), (u'n03482405', u'hamper', 0.055440076)])

 $ KERAS_BACKEND=theano python test_mobilenet.py
Using Theano backend.
('Predicted:', [(u'n02123159', u'tiger_cat', 0.2602814), (u'n02124075', u'Egyptian_cat', 0.14234008), (u'n02123045', u'tabby', 0.06521355)])
('Predicted:', [(u'n02124075', u'Egyptian_cat', 0.17173238), (u'n02123597', u'Siamese_cat', 0.10258683), (u'n02130308', u'cheetah', 0.0565808)])

The problem is that TensorFlow differs as follows:
 $ KERAS_BACKEND=tensorflow python test_mobilenet.py
Using TensorFlow backend.
('Predicted:', [(u'n03482405', u'hamper', 0.21035317), (u'n02123159', u'tiger_cat', 0.113607496), (u'n04074963', u'remote_control', 0.09508652)])
('Predicted:', [(u'n02124075', u'Egyptian_cat', 0.116303556), (u'n02123597', u'Siamese_cat', 0.075559795), (u'n02130308', u'cheetah', 0.048775144)])

The main problem is from different padding behaviour of Conv2D(filters, kernel, padding='same', strides=(2, 2)). With explicit ZeroPadding2D, TensorFlow is able to show small numerical differences.
 $ git checkout fix_resnet_mobilenet
Switched to branch 'fix_resnet_mobilenet'
 $ KERAS_BACKEND=tensorflow python test_mobilenet.py
Using TensorFlow backend.
('Predicted:', [(u'n02123159', u'tiger_cat', 0.2602828), (u'n02124075', u'Egyptian_cat', 0.14233977), (u'n02123045', u'tabby', 0.0652139)])
('Predicted:', [(u'n02124075', u'Egyptian_cat', 0.17173262), (u'n02123597', u'Siamese_cat', 0.102586426), (u'n02130308', u'cheetah', 0.056580666)])

@taehoonlee, since you already understand the issue (I don't), could you, please, document the difference in behavior of Keras Conv2D(padding='same') and friends, across backends. Maybe this will answer my question why we synchronize this way (instead of, say "fixing" Theano/CNTK Conv2D and friends).
@fchollet

@ozabluda, Here is a toy example:
import numpy as np

from keras.layers import Conv2D
from keras.models import Sequential
from keras import backend as K

w = [np.arange(3*3*5*2).reshape(3, 3, 5, 2)]
if K.backend() == 'theano':
    w = [np.flip(np.flip(w[0], 0), 1)]

model = Sequential()
model.add(Conv2D(2, kernel_size=3, strides=2, input_shape=(6, 6, 5),
                 padding='SAME', use_bias=False))
model.set_weights(w)
model.predict(np.ones((1, 6, 6, 5)))

The results are:
 $ KERAS_BACKEND=theano python test.py
array([[[[1280., 1300.],
         [1770., 1800.],
         [1770., 1800.]],

        [[1470., 1500.],
         [1980., 2025.],
         [1980., 2025.]],

        [[1470., 1500.],
         [1980., 2025.],
         [1980., 2025.]]]], dtype=float32)
 $ KERAS_BACKEND=cntk python test.py
array([[[[1280., 1300.],
         [1770., 1800.],
         [1770., 1800.]],

        [[1470., 1500.],
         [1980., 2025.],
         [1980., 2025.]],

        [[1470., 1500.],
         [1980., 2025.],
         [1980., 2025.]]]], dtype=float32)
 $ KERAS_BACKEND=tensorflow python test.py
array([[[[1980., 2025.],
         [1980., 2025.],
         [1170., 1200.]],

        [[1980., 2025.],
         [1980., 2025.],
         [1170., 1200.]],

        [[ 870.,  900.],
         [ 870.,  900.],
         [ 480.,  500.]]]], dtype=float32)

The difference comes from the difference in padding behavior when padding='same', strides=2. Here's a little more expansion:

Theano
CNTK
TensorFlow

odd kernel / odd input

odd kernel / even input

differs

even kernel / odd input
differs

even kernel / even input
differs

I'm getting same output for 1by1 kernel with different padding:
`conv5 = Conv2D(16,kernel_size=(1,1),strides=1,padding='same')
conv6 = Conv2D(16,kernel_size=(1,1),strides=1,padding='valid')
conv7 = Conv2D(16,kernel_size=(1,1),strides=2,padding='same')
conv8 = Conv2D(16,kernel_size=(1,1),strides=2,padding='valid')
x10=inputs
xx5 = conv5(x10)
xx6 = conv6(x10)
xx7 = conv7(x10)
xx8 = conv8(x10)
print("Input shape is:",inputs.shape)
print("kernel_size=(1,1),strides=1,padding='same'",xx5.shape)
print("kernel_size=(1,1),strides=1,padding='valid'",xx6.shape)
print("kernel_size=(1,1),strides=2,padding='same'",xx7.shape)
print("kernel_size=(1,1),strides=2,padding='valid'",xx8.shape)
('Input shape is:', TensorShape([Dimension(None), Dimension(32), Dimension(32), Dimension(3)]))
("kernel_size=(1,1),strides=1,padding='same'", TensorShape([Dimension(None), Dimension(32), Dimension(32), Dimension(16)]))
("kernel_size=(1,1),strides=1,padding='valid'", TensorShape([Dimension(None), Dimension(32), Dimension(32), Dimension(16)]))
("kernel_size=(1,1),strides=2,padding='same'", TensorShape([Dimension(None), Dimension(16), Dimension(16), Dimension(16)]))
("kernel_size=(1,1),strides=2,padding='valid'", TensorShape([Dimension(None), Dimension(16), Dimension(16), Dimension(16)]))
`
In the above results, xx5 and xx6 have same shape (xx7 and xx8 also ) even though different padding is used for convolution.

It is really amazing to me that padding which should be an integer has been left to be interpreted from some strange set of strings (for ease of use?). When ease of use destroys clarity, it is time for some upgrade :-)

Hi! Is/Has anybody working/worked on making Conv2D(padding='same') consistent for the Tensorflow backend? If no, can I help with that issue? I'm a first time contributor, so please let me know if I have any misunderstanding. @fchollet


Not working for Sublime Text 3

Installed this via Packages Control ( Also tried manually ) but it always says "It appears Tern has not been installed...".
I think it's because how ST3 handles packages now.

I have the same problem with Sublime text 3 and windows 7 x64

@mladenuzelac add (adjusting path) "tern_command":["node", "C:/Users/blabla/AppData/Roaming/npm/node_modules/tern/bin/tern", "--no-port-file"] to your Preferences -> Settings - User file.

Have/had the same problem as @mladenuzelac did what @SAnDAnGE said and it "worked". Now I get no errors and it seems that Tern starts. But it won't auto-complete anything.
It says: Starting TernJS server for empty with 1 libs and 0 files.

Is it worth adding a step for Mac users to install https://github.com/int3h/SublimeFixMacPath in the README? It's just caught me out again, reinstalling.

Attached patch adds a short note about this.

:+1: SublimeFixMacPath fixes it for me. Cheers

Still having trouble making autocomplete work (I'm using Linux Mint 17.2 Cinnamon and ST3 Build 3114).
I tried installing it both manually and via the Package Manager. I added "auto_complete_triggers": [ {"selector": "text.html", "characters": "<"}, {"selector": "source.js", "characters": "."} ] to my Preferences file. I set:
"tern_argument_hints",
"tern_argument_completion",
"auto_complete"

to true. I followed @wtw's advice by adding a .tern-port file all to no avail.
I also supposedly cleaned npm's cache and then updated both npm and minimatch. However, after that when I tried to install tern, I was still getting the following warnings:
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue

Maybe the problem stems from there?

By the way I think this fixes the path issue for those not on Mac


youtube slider

Is there an example for using embla as a youtube video slider?

Hi @ramiwi,
Thank you for your question. No, not at the time of writing. But it should be easy to achieve because Embla Carousel doesn't care about what you put inside the slide elements? Have you tried it yourself?
Kindly,
David

Yes, its tricky to keep the aspect ratio right for youtube iframe slides. Actually not specific to Embla but I wondered if someone had already created an embla youtube slider.

I see. Are you using the VanillaJS or ReactJS version?
Kindly,
David

Vanilla js

Allright @ramiwi. I'll create a CodeSandbox that demonstrates this when I have time. Any other requirements you had in mind?

This will be great!
For other requirements. a video slider usually displays multiple slides, as in the CodeSandbox you created for issue #69 (https://codesandbox.io/s/bppew) and preferably keeps the video frame proportions without cropping its content.

Hi @ramiwi,
Thank you for the additional details. This is what I came up with:

Demo
CodeSandbox

In order to maintain correct aspect ratio throughout different screen sizes, which is 16:9 for Youtube videos, I use the padding-bottom-hack. You can read more about in this article.
Note that I intentionally added draggable: false to the Embla options like so:
const embla = EmblaCarousel(viewPort, {
  align: "start",
  containScroll: "trimSnaps",
  draggable: false // here
});

The reason for this is that Youtube-Iframes will "hijack" pointer events and use them to trigger events related to their video players. So when you drag with your pointer inside an iframe Embla won't receive any events.
One way to solve this issue is to use preview images inside the slides together with a play icon instead of iframes. And when a user clicks on a slide, it triggers a modal with an iframe.
Let me know if it helps.
Kindly,
David

It works great, thanks!

Glad to hear that @ramiwi. Yes, you can use any desired width for your slides by changing min-width 👍🏻.
Closing this issue as resolved. Enjoy 🎉


Java debug error

i cloned the app and followed the steps stated but i run into this error. Any help will be appreciated
Task :app:compileAnalyticsDebugJavaWithJavac FAILED

Hi @elishaakyaw can you try setting the app flavour :- like this:

then do a full rebuild:

If this doesn't work can you let me know the full error message, I will try to diagnose with you.

Thanks alot.  Went out to get some groceries so will do that when I get
home and give you my feedback
On Tue, Oct 18, 2022, 4:08 PM James Brown @.***> wrote:

Hi @elishaakyaw https://github.com/elishaakyaw can you try setting the
app flavour :- like this:
[image: image]
https://user-images.githubusercontent.com/12689544/196482764-924e64fc-211c-4ae6-8545-eab5fe8486dc.png
then do a full rebuild:
[image: image]
https://user-images.githubusercontent.com/12689544/196484372-f5db39bf-9dd9-4f90-a7bc-dca1555e6750.png
If this doesn't work can you let me know the full error message, I will
try to diagnose with you.
—
Reply to this email directly, view it on GitHub
https://github.com/AlphaWallet/alpha-wallet-android/issues/2876#issuecomment-1282643227,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMS7FV62HJDV6ZYKTJ5KOZDWD3DPPANCNFSM6AAAAAARH53UNI
.
You are receiving this because you were mentioned.Message ID:
@.***>

i did what you recommended. ended up with the above errors.

Ok I think I know what the problem is:
can you do:

I think it's the Java JDK version. We're using verion 11. Try changing your Java version. Please let me know if you get it working I'll ensure the docs are up to date.

ok sure. will try that now. thanks

is it necessary to build in android studio. Is it possible to do this in vs code. running into a lot of errors.

i also get this window often, i just click yes without clicking anything

i also get this window often, i just click yes without typing anything


Extract registerable domain returns empty

I have a loop that extracts registerable domains. However, despite most of them running fine, a still have a lot of empty responses. What could be the cause that it does not find a registerable domain for the following examples :
mail-sor-f69.google.com  (you would expect google.com)
static.vnpt.vn  (you would expect vnpt.vn)
mx1.sub5.homie.mail.dreamhost.com  (you would expect dreamhost.com)
and there are many , many, many more.

Please provide code that you run.
tld_extract('mail-sor-f69.google.com')->getRegistrableDomain(); // returns 'google.com'

Please find below
		 $ptr = 'mail-sor-f69.google.com'; $tldresult = $extract->parse($ptr); $tld = $tldresult->getRegistrableDomain();

 $extract = new \LayerShifter\TLDExtract\Extract();
 $ptr = 'mail-sor-f69.google.com';
 $tldresult = $extract->parse($ptr);
 $tld = $tldresult->getRegistrableDomain();

 var_dump($tld);

  string(10) "google.com"

I'm almost sure that there are no problems with package. Try remove vendors folder and install depencies again. Also check that you're using latest version of TLDExtract.
Did you touched TLDDatabase? What will return code below?
$store = new \LayerShifter\TLDDatabase\Store();
$store->isExists('com');
$store->isExists('vn');


Fix uneven trailing whitespace in strings

Found as part of JuliaLang/julia#46372 - the reference parser treats lines of uneven whitespace in a weird inconsistent way so there's some differences in whitespace in the final deindented string. This change removes those differences.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
84

Relevant Lines:
101

💛  - Coveralls


Miscellaneous 0.6.0 stuff

I'll use this thread to document various minor changes.

I'll deduplicate Shift<T> since there's no need to differentiate between Shift<T> and Shift<T.Magnitude>. Shift<T.Magnitude> is enough. Any effect this has on performance will go away once Shift is pointer-bit.

The systems integer Shift/inverse() constraint will be lifted as a result of the count approach (#31) (#33).

InfiniInt's bit counting methods will be hoisted to [Mutable]DataInt as a result of the count approach (#31) (#33).


[DEVOPS-10]  OS X local build automation

This completes the Daedalus local build automation (the first half of the effort was on Windows):

The build sequence is extracted into a stand-alone script, that is both invoked by CI and available for developer use.
A convenience script that allows a single-launch build of a fresh Daedalus version on OS X, that checks out a fresh version from git in a clean directory, installs Nix and invokes the build.
The README.md is revised in an attempt to provide better structure.

Known issues:

[ ] The --upload-s3 option to the build script doesn't work without pre-obtained Amazon credentials
[x] The Linux build on the CI currently bombs out due to a failure of deployment -- which shouldn't even happen, because there's no installer to deploy on Linux yet -- the build is just a stub.  Clearly, something started to trigger this unwanted deployment.. couldn't figure out what did, so far.

cc @jmitchell

@domenkozar, I'm now guessing that --upload-s3 cannot be made to work, without leaking this secret.
But also -- I'm beginning to thing that maybe only CI should upload to S3 anyway..

Yes, I'd prefer only CI to be able to upload. We can have a script for manual testing though, not integrated into the CI.

@jmitchell, sorry, I don't quite see what you are referring to..

windows: successfully connects to blockchain: https://ci.appveyor.com/api/buildjobs/fg0qcy4otc9vhftp/artifacts/installers%2Fdaedalus-win64-0.3.1507.0- installer.exe

OS X: successfully connects to the blockchain: http://s3.eu-central-1.amazonaws.com/daedalus-travis/Daedalus-installer-0.3.1380.pkg

Verifying that restarting Daedalus doesn't lead to an unkillable process..

It doesn't.


[Bug]Cannot drag and move the window, if there is an modal of tips to save the change when you click close

Describe the bug
Cannot move the window, if there is an modal of tips to save the change. If the modal is not visible, you can only use the escape key to close the modal box, and then drag and move the window, then close the windows and click "Discard & Exit".
To Reproduce
Just open notepads and type some word, then click close.
Expected behavior
The window should can be dragged as another UWP app like "Paint 3D" when it exists a modal.
Screenshots
(a 10M gif screenshots, playback in browser may be slow)

Desktop:

OS: Windows 10 2004 19041.487
Version v<IP_ADDRESS>

Additional context
This issue may like #678

As fas as I know, Notepads use the ContentDialog to show popups like "Discard & Exit". The ContentDialog is a modal dialog, whence why you can't drag and move the main window. On the other hand, it's interesting how Paint3D have what it looks like a modeless ContentDialog.  There is also a relevant issue on the WinUI repo.
A question: would be enough for your scenario to have a kind of light dismiss behavior so that when you click on the title bar to drag and move the main window, it will automatically close any popup? I think this might be easier to implement in the meantime until a non-modal ContentDialog or similar control is available.

A question: would be enough for your scenario to have a kind of light dismiss behavior so that when you click on the title bar to drag and move the main window, it will automatically close any popup? I think this might be easier to implement in the meantime until a non-modal ContentDialog or similar control is available.

The dismiss behavior is acceptable, although it is not as intuitive as moving directly. Of course, it would be nicer if it can be moved directly.

This issue (the same as #701) is not the same as light dismissing. This is definitely something that should be brought up with WinUI. @AHBICJ @Kuuchuu could one of you make an issue there?

Btw, this area is still draggable and you can try:

Btw, this area is still draggable and you can try:


Add USB LED support - Mfg: ThingM, Model Blink(1)

The USB LED made by ThingM is very popular and a simple USB LED status LED.
Possibly add direct support for controlling this LED.
Company web site is: ThingM.com
The Blink(1) LED can be controlled via IFTTT, but direct API would simplify setup.

Did you try using  the Custom API? I do no have resources to build a custom solution for this light.

Ok, no problem- I will look into that, the project is an interesting design and will be something I can dig into.
BTW- is there any way to set something in the Presence Light (code?) to monitor the status of other contact than myself - which I can see status of in Teams, but really I wanted a Boss/Supervisor Alert was really my goal with this (jokingly referred to as DEFCON Status among peers).
https://en.wikipedia.org/wiki/DEFCON
Thanks for the great work, I did find that as-is I can see a better status of what I am posting, than what is shown in teams, I'll look into the extended API for the USB-LED and let you know if I get it to work.
-James
On Monday, March 20, 2023 at 01:39:08 PM PDT, Isaac Levin<EMAIL_ADDRESS>wrote:

Did you try using the Custom API? I do no have resources to build a custom solution for this light.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Thx for quick reply- I will look at the "graphing API"- seems like that is where many get the status from.
On Tuesday, March 21, 2023 at 09:06:53 AM PDT, Isaac Levin<EMAIL_ADDRESS>wrote:

It would be a considerable effort to add something like that. Not sure if I would need to add additional scopes to PresenceLight in graph.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Add Snowflake dialect to PyPIKA

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/444
Originally created by @mbergeron on 2019-03-04 15:20:31

Problem to solve
We are currently using PyPIKA to generate the SQL queries for the Meltano UI. As we want to use Snowflake as a data warehouse, we need to make sure our SQL generation engine supports it. We'll need to add a Snowflake dialect to PyPIKA, which might involve sending back as a contribution or as a separate package. Once the dialect in, PyPIKA should be able to generate queries compatible with Snowflake specifics.
Target audience
Data Analyst
Further details
Without this, the generated SQL might cause an error when queried.
Proposal

[ ] Investigate Snowflake SQL support
[ ] Investigate PyPIKA dialect handling
[ ] Investigate PyPIKA dialect integration
[ ] Build the dialect

What does success look like, and how can we measure that?
Our SQL generation engine can output Snowflake compatible SQL.
Links / references
Related to #428
https://github.com/kayak/pypika/blob/master/pypika/dialects.py

View 3 previous comments from the original issue on GitLab


Add !commands

!commands should do one or both of the following:

Display the commands in chat.
Display a link to a web page where the commands are shown with a long description. (Nightbot behaves this way)

fixed the streamer.bot action.


Is it possible to hide the disclaimer for packages from Microsoft?

Description of the new feature / enhancement

Currently, when a user installs a package from the winget source, winget will prompt something like this:

This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.

It makes sense when the user installs an application from a third-party developer. But if the target application is from Microsoft, things get a little confusing.
So, in my opinion, it will be better if winget hides the prompt or replaces it with the link pointing to https://www.microsoft.com/legal/terms-of-use when installing a Microsoft application.
Proposed technical implementation details
No response

Duplicate of #288


Pass Parent module name to included xml files for publishing sensor reading

It would be very helpful if there was example on how to pass the vehicle name variable to included sensor xml files for better sensor publisher name.

It was actually not possible to do that until now, but after the latest commit it is! See the new tutorial files in 05c26a4d0ffeeed49b291a22d508a136f56928ac
Closed via 05c26a4d0ffeeed49b291a22d508a136f56928ac


Cannot read a nc file that contains the diacritic mark tilde (á) in the path

What is your issue?
When I use xarray to open a nc file that contains the diacritic mark tilde (á) in the path, I get an error
import xarray as xr
import numpy as np

# Create a new dataset using xarray
data = xr.Dataset(
    {
        "temperature": (("x", "y"), np.random.rand(100, 100))
    },
    coords={
        "x": np.linspace(0, 1, 100),
        "y": np.linspace(0, 1, 100)
    }
)

# I don't know if there are any errors in this step,
# because the actual Chinese text I generated here "example_xarray_谩.nc"
data.to_netcdf('example_xarray_á.nc' )

Running the following one will cause an error
dataset = xr.open_dataset('example_xarray_á.nc')

FileNotFoundError: [Errno 2] No such file or directory: 'C:\\Users\DELL\\Desktop\\example_xarray_á.nc'

@gitxiaobai667 As @max-sixty already told you twice to fill out the bug template properly, please do so. We still miss essential information (eg. OS, version of dependencies). We all have limited bandwidth and rather dedicate our time to well written bug-reports.
This might be an issue with windows system encoding and or netcdf-c/hdf5:

https://github.com/pydata/xarray/issues/3974
https://github.com/pydata/xarray/issues/5666
https://github.com/Unidata/netcdf4-python/issues/941
https://github.com/Unidata/netcdf4-python/issues/997

Check your issue with recent versions of xarray/netcdf-c/hdf5/netcdf4-python in your environment and see if the issue persists.
I do not close this issue yet. But if you do not provide the full information according to the bug template some other core dev might do so in the next days.

I think the miscommunication here is that they always chose our "issue" template while the template with the more detailed forms is the "bug report" template (so they never saw the template and are likely confused about what you are asking for). Should we rename the "issue" template? What do you think about "general issue", or maybe something not involving the word "issue" might be better?

@keewis Oh, ha, that's painful. Sorry, I'll be more careful on that next time. Thanks for making this obvious.
I'm not sure how to improve this to be more specific The question is what do we understand as General issue, that's not a bug report.?

Note that that's just how I interpret this situation, I can't be certain without hearing back from them. However, based on my own experience with foreign languages these things can happen (like, you know what an issue is but not what "bug report" means), and I was wondering how we can make things a bit more obvious. Though in the end, all of this depends on working communication, which is not easy if the language proficiency differs.
Maybe move the blank template below the feature request and change the help text/name to something along the lines of "use only if it doesn't fit the other templates (mostly intended for development organization)"? (since that's how I understood it when we added that template)

I think the misunderstanding here is that they always chose our "issue" template while the template with the more detailed forms is the "bug report" template

The first issue was the bug report template! https://github.com/pydata/xarray/issues/8529 — they just skipped past most of the fields.
My guess is that they're newer to this, don't have much context, and wanted to get help. Maybe someone suggested they open an issue. I don't think we need to believe people are badly intentioned to hold high standards for issues
I'm very open to feedback on better ways to have done this — I certainly don't revel in closing issues (feedback.maxroos.com !)

More generally, I'm holding there are two reasonable equilibria:

GH Issues are only used for bug reports / feature requests, and we set clear expectations. Currently at xarray, we follow this model.
GH Issues are also for "user support", and there are people who enjoy answering these questions or think it's beneficial enough for the project. (At PRQL we follow this model)

A bad approach is to have expectations of user support but then to not get around it — people write things that don't get responses, issues stack up, we miss out on the high quality bug reports that could have made the library better. So hence me being quite strict here. There's then a gap in user support — to some extent that's inevitable given the relative supply & demand[^1].

@gitxiaobai667 if you see this — what could have made this experience better for you, aside from having your question answered? Were the instructions unclear? Did you get what you wanted from @kmuehlbauer 's message. We appreciate you at least attempting to engage...
[^1]: Though maybe there's a also a business to be started in allowing maintainers of open-source projects to offer a couple of hours of support — I would be a buyer in some other projects!


Getting error with code: 120 ("App store failed to load")

Getting following error when ad view is tapped, loader is shown for some time and then following error is received.
Error Details:
Error description: "App store failed to load"
Error Domain : flurryAds
Error Code = 120
AdError : FLURRY_AD_ERROR_CLICK_ACTION_FAILED

@Ankit-Aggarwal can we move this to<EMAIL_ADDRESS>That's the best avenue for support issues like this.


[v4.0.0-beta.12] sc-ion-* style class added to ion-elements in Firefox only => difficult to style across browsers

Bug Report
Ionic Info
Ionic Angular v4-beta.12
Describe the Bug
sc-ion-* style class are added to ion-elements in Firefox only. This has for effect that the cascaded CSS are then different only in Firefox which makes it difficult to style the app across browsers
Related Code
For example, Chrome and Safari rendered navbar:
<ion-header class="header header-md hydrated">
   <ion-toolbar class="hydrated">

Firefox rendered navbar:
<ion-heaver class="header header-md hydrated">
   <ion-toolbar class="sc-ion-toolbar-md-h hydrated">

Expected Behavior
I would except a consistent CSS style class behavior across browsers. In best case I would be happy that style class would behave for Firefox like for Safari and Chrome because I would be happy to not restyle again my app ;)
Additional Context
Chrome:

Safari:

Firefox:

No sure why firefox having scope classes make it harder, you should be using CSS variables to customize the app, which should work the same in all browsers.
Theses classes are just scoped CSS, because native shadow-dom is not available yet in firefox.

@manucorporat
let's say I want to style the ion-toolbar background color, in Safari and Chrome the following works well:
ion-header ion-toolbar {
  --background: red;
}

but doesn't work in Firefox because Firefox give more priority to .sc-ion-toolbar-md-h than to ion-header ion-toolbar
.sc-ion-toolbar-md-h {
   --background: var(--ion-toolbar-background-color, #f8f8f8);
}

ion-header ion-toolbar {
  --background: red;
}

so yes of course I could set --ion-toolbar-background-color for that particular case but that's not the only one aka it means that I should and will always have go thru the all app "just" for Firefox to double check because it doesn't render like Safari and Chrome


Interactive Window: Variable Explorer issues

Using #%% in .py files, I've had issues with the interactive window variable explorer.
Since the new update: the variables do not load consistently. I often see "no variables defined" after running my code.
Also, I dislike that the variable explorer tab now opens in the bottom panel. And it seems I can only view one variable at a time now.

Thanks! This is a dupe of #6993


[Bug] There is error when upgrading ohif to version 3

Hi @ALL !
I checkout the ohif version 3 at:
https://github.com/OHIF/Viewers/tree/v3-stable
Then I integrated my ohif ( running on  v4.9.17) into version 3.
There is following error:

Please check and help me!
Thanks guys!

What happens if you do a fresh yarn install for v3-stable?


Build a docker container automatically after the CI

Use the build and push docker images action to build and push the container with the app to dockerhub.

feature provided


Route and sidebar management clean up #122

Closed in favour of

https://github.com/EnAccess/micropowermanager/pull/132
https://github.com/EnAccess/micropowermanager/pull/138


[TF2.0][FR] Skip TFRecord files with 'DataLossError: corrupted record at'

System information

TensorFlow version (you are using): 2.0.0-dev20190319

Describe the feature and the current behavior/state.
In this version of TF and prior versions, TFRecord files get corrupted or are detected as corrupt spontaneously during training. This is a problem already heavily discussed in e.g https://github.com/tensorflow/tensorflow/issues/13463 . Unfortunately, no one came up with an solution or a script triggering the problem, yet. This makes debugging what causes the files to change(?) during training quite hard.
Thus, it would be nice to be able to indicate to Tensorflow that such files should be ignored and only a warning is printed instead of canceling the whole training process.
An exemplary error message:
Traceback (most recent call last):
[...]
    obj = distributed_train(summary_writer)
  File "/home/meyerjo/code/kaggle/nuclei-detection-tf2/nucleidetection/train.py", line 297, in distributed_train
    lambda x: train_step(x, summary_writer), train_iterator)
  File "/home/meyerjo/code/testfield-tf-nightly/tf-nightly/lib/python3.5/site-packages/tensorflow/python/distribute/distribute_lib.py", line 470, in experimental_run
    args = (input_iterator.get_next(),) if input_iterator is not None else ()
  File "/home/meyerjo/code/testfield-tf-nightly/tf-nightly/lib/python3.5/site-packages/tensorflow/python/distribute/input_lib.py", line 145, in get_next
    self._iterators[i].get_next_as_list(new_name))
  File "/home/meyerjo/code/testfield-tf-nightly/tf-nightly/lib/python3.5/site-packages/tensorflow/python/distribute/input_lib.py", line 415, in get_next_as_list
    data_list = self._iterator.get_next_as_optional()
  File "/home/meyerjo/code/testfield-tf-nightly/tf-nightly/lib/python3.5/site-packages/tensorflow/python/data/ops/multi_device_iterator_ops.py", line 311, in get_next_as_optional
    self._device_iterators[i]))
  File "/home/meyerjo/code/testfield-tf-nightly/tf-nightly/lib/python3.5/site-packages/tensorflow/python/data/ops/iterator_ops.py", line 689, in get_next_as_optional
    output_shapes=iterator._element_structure._flat_shapes),
  File "/home/meyerjo/code/testfield-tf-nightly/tf-nightly/lib/python3.5/site-packages/tensorflow/python/ops/gen_dataset_ops.py", line 1858, in iterator_get_next_as_optional
    _six.raise_from(_core._status_to_exception(e.code, message), None)
  File "<string>", line 3, in raise_from
tensorflow.python.framework.errors_impl.DataLossError: corrupted record at 852968336
	 [[{{node MultiDeviceIteratorGetNextFromShard}}]]
	 [[RemoteCall]] [Op:IteratorGetNextAsOptional]

Reassigning to @jsimsa for triage (but maybe tf.data.experimental.ignore_errors() would solve the problem here?).

@meyerjo as per @mrry's suggestion, have you tried using ignore_errors?

I think that has went under my radar. As in the other issues discussed it seems to work around the error message. Thanks!
May I leave this open, as TFRecord Files still get spontaneously corrupted during training? As training should be readonly regarding those files, this seems to be a bug.

@meyerjo thank you for your response. Unless you can provide us with an example that can be used to reproduce this issue, there is little value in keeping this issue open (as no further action will be taken).

Ok. Then I will close the issue


AlertText type error

Description
Since updating to @gluestack/themed v1.1.22 I've started to the following typescript error when using the AlertText component:
Type '{ children: string | undefined; }' has no properties in common with type 'IntrinsicAttributes & RefAttributes<StyledComponentProps<StyleProp<TextStyle>, unknown, StyledComponentProps<StyleProp<TextStyle>, unknown, TextProps, "Text"> & RefAttributes<...>, "AlertText"> & RefAttributes<...>>'.

CodeSandbox/Snack link
No response
Steps to reproduce
Use the AlertText component and try and compile TS
gluestack-ui Version
1.1.22
Platform

[ ] Expo
[ ] React Native CLI
[ ] Next
[ ] Web
[ ] Android
[ ] iOS

Other Platform
No response
Additional Information
No response

Same here. As a workaround I downgraded to "@gluestack-ui/themed": "1.1.18" and it is working here.

Same here - bump

@jordmccord Thanks for reporting the issue.
It has been resolved here.
Version bump of @gluestack-ui/alert to 0.1.14
The type error won't come.

Thanks for fixing @DaminiPandey


Add Graphy package

The Graphy library is useful and can be added as a package. Would be helpful if added and a new debug console command created to hide/show at runtime.

Merged with v0.5.0


short circuit before the version_compare call

combined with the other PRs this will make most of the
subclasses never even call version_compare now.
the subclasses that need allow_downgrade as a feature
should implement both the property and implement
version_compare now.

it is possible now that version_compare in the superclass won't ever be called an we could raise an error that it must be implemented now...
IDK feel that aggressive tonight though.


DK: Version 1.3.0 QR codes

The PNG files should be in the folder 1.3.0 and there should be a README.md instead of the json files.

Android 1.0.7.-acc : all certs are valid

@chrloch fixed

Android 1.0.7.-acc : all 4 certs are valid

iOS 1.0.5 all 4 codes are valid


internal/Restore-DBFromFilteredArray.ps1:149 missing -Message param on Write-Message

Bug Report

General Troubleshooting steps

[x] Verified running latest release of dbatools?
[x] Verified errors are not related to permissions?
[x] Can duplicate in new/fresh PowerShell.exe session?

Version Information

Operating System (Name|Version):
Windows Server 2012 R2 Standard- Microsoft Windows [Version 6.3.9600]
PowerShell Version:

Tried remotely on version below:
PSVersion                      5.1.14409.1012
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14409.1012
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
Attempted locally using the version below:
PSVersion                      4.0
WSManStackVersion              3.0
SerializationVersion           <IP_ADDRESS>
CLRVersion                     4.0.30319.42000
BuildVersion                   6.3.9600.18728
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0}
PSRemotingProtocolVersion      2.2

SQL Server (Edition|Version):
Destination: 2016 Ent, but tried on 2012 Ent also.

Steps to Reproduce

[x] Attach any screenshots (if possible/allowed)
[ ] Attach logs generated by dbatools (Get-DbatoolsLog)
[x] Attach output from PowerShell console (if possible/allowed)

Steps

On a brand new VM with SQL 2012/2016 (version doesn't seem to matter) installed.
Create a new folder C:\SQLBackups
Grant Everyone read permissions to the folder (just to be sure, i realise this isn't good security practice!)
Copy a small backup into the folder and rename it to 'RandomSmallBackup.bak'
open PowerShell ISE (locally) and run the following (note against the local SQL instance):

.'C:\dbatools\install.ps1'

cls

get-module dbatools;

'C:\SQLBackups\RandomSmallBackup.bak' | Restore-DbaDatabase -SqlInstance localhost -ReuseSourceFolderStructure -WhatIf

Additional information
Problem seems to exist within internal/Restore-DBFromFilteredArray.ps1:149
https://github.com/sqlcollaborative/dbatools/blob/master/internal/Restore-DBFromFilteredArray.ps1#L149
Note that Write-Message is missing the -Message param on line 149, so "File doesn't exist, check for parent folder" is expected to be a positional parameter which matches the error message.
Line 151/157/161 as a comparison has the -Message parameter
Should be an easy fix, but i'm not sure how to contribute just yet...sorry! It's on the to-do to learn!!:)
Credit to - Marcin Gminski on #dbatools Slack who spotted the missing -Message param :)

I've checked the rest of the file, and there's another missing -Message param at line 119, same file. Checked every other Write-Message line and they're all good otherwise, just Lines 119 & 149

Thanks so much for the report! This was an easy fix -thanks for the follow up Slack messages. I'll merge the fixes once the tests are complete.


Move plugin certs to better location.

Move plugin certs from install/kubernetes to samples/certs.

Please remember to update the doc accordingly (in master branch)

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: wattli
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

samples/OWNERS [wattli]
security/OWNERS [wattli]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Sure. I'm working on it.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: wattli
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

samples/OWNERS [wattli]
security/OWNERS [wattli]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[master] Bump Mesos to nightly master e6b15b1

High-level description
This is a routine bump to the latest Mesos and mesos-modules.
Related JIRA Issues
Checklist for all PRs

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Changelog:
https://github.com/apache/mesos/compare/beaf0cd844f3658bfccb86049f7181036b0e6ae4...e6b15b1e968649c794d67bb97961e8f0216e2ed8

[ ] Test Results: [link to CI job test results for component]

[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label Ready For Review

@mesosphere-mergebot override-status
teamcity/dcos/test/terraform/aws/onprem/static/group4 https://jira.mesosphere.com/browse/D2IQ-66711

@mesosphere-mergebot label Holding

@mesosphere-mergebot label Holding

@mesosphere-mergebot label Ready For Review

@mesosphere-mergebot changelog-not-required this is a regular Mesos bump


feat: add telemetry to package manager

Description
Tracks:

package installed (package, version, isUpdate)
package uninstalled
pacakges searched (term)
package conflict found when installed
package conflict resolved
package uninstall failed
package install failed
add feed
delete feed

Task Item
fixes #5947

Coverage remained the same at 53.422% when pulling 03dec3b6c9752163c5f6a468a047cbc4b75927f5 on benbrown/pkgtelem into 71b25eb52aa58dd5ec6a2159ae3997f257b50f75 on main.


Loading the pretrained weights into pytorch

Hi,
I'm trying to use the pretrained weights provided in the readme, but am having some trouble. I loaded model_final.pth as a state_dict, and then tried to load it into a new instance of KeypointDetector, but this hit an error because the expected keys for a KeypointDetector state_dict and the given dictionary keys do not match up. What am I doing wrong?
Thanks,
Josh

It seems like the codebase may have changed since the pretrained weights were saved.  I found the following (manually changed) key names gets things to work:
    pretrained = torch.load('/mnt/drive/MyDrive/Datasets/SMOKE/model_final.pth')['model']
    for k1, k2 in zip(sorted(pretrained.keys()), sorted(model.state_dict().keys())):
        assert(k1.replace('module.', '') == k2)
        model.state_dict()[k2][:] = pretrained[k1]