x/net/http2: serverConn.processHeaders has two error paths that don't clean up

I continue to hunt a stream accounting bug in either the Go HTTP/2 transport and/or server. Reading code again now, I noticed:
(*serverConn).processHeaders does:
st := sc.newStream(id, 0, initialState)

... where that newStream call is what's responsible for incrementing the stream count:
sc.curClientStreams++

(that's what's ultimately causing the return sc.countError("over_max_streams", streamError(id, ErrCodeProtocol)) I'm hitting)
But just after that newStream call are two error exit paths:
st := sc.newStream(id, 0, initialState)

if f.HasPriority() {
        if err := sc.checkPriority(f.StreamID, f.Priority); err != nil {
                return err
        }
        sc.writeSched.AdjustStream(st.id, f.Priority)
}

rw, req, err := sc.newWriterAndRequest(st, f)
if err != nil {
        return err
}
// ... [no returns]
go sc.runHandler(rw, req, handler)
return nil

Those two return err errors are suspect. If those happen, then curClientStreams can be left incremented, never to be decremented again.
The decrement happens in (*serverConn).closeStream, which is called from:

processResetStream, reading a RSTStream frame from the client (e.g. they canceled the request)
wroteFrame, called after a frame write is successful, including handler panics. But if we return err above before starting the handler, we can't ever write anything, as the handler never runs.

I'm not sure whether this is what we're hitting yet, but it looks suspicious.
/cc @neild

I still think this is a problem, but I don't think it's the over_max_streams problem I'm hitting. I checked all the exit paths and they all have error counter metrics on them, except for one (https://golang.org/cl/353031), which I doubt we're hitting, as CONNECT requests should be unable to make it to our backend.


Fix the collector spec now that we strip null refs

After https://github.com/ManageIQ/topological_inventory-providers-common/pull/4 we need to update the collector spec to not look for nil references
Fixes https://travis-ci.com/ManageIQ/topological_inventory-azure/jobs/248332352#L329-L346

Thanks @slemrmartin, yeah travis runs daily so that helps us catch most things.  We can always use https://github.com/ManageIQ/manageiq-cross_repo-tests to test all of the collectors with a providers-common gem change before merging also.


Save derivative description

BEP 003 requires derivative datasets to have dataset_description.json:

Derived dataset and pipeline description
<dataset>/derivatives/<pipeline_name>/dataset_description.json
Keys:

HowToAcknowledge
PipelineDescription

Name
Version
CodeURL
DockerHubContainerTag
SingularityContainerURL
SingularityContainerMD5

SourceDatasetsURLs - a list of URLs to the source dataset(s) (mandatory)
SourceDatasetsVersions - a list of versions of the source dataset(s) (optional only if SourceDatasetsURLs specify the version unequivocally)
License

It would be good to go ahead and generate this, as pybids is starting to validate the presence of this file.
While we're at it, we should consider whether we think any other fields ought to be added to the spec, such as CreationDate, ExecutionParameters, etc.
We should
Related: poldracklab/fitlins#29

Related #1190


Add notebook demonstrating dea_coastaltools funcs

Proposed changes
This PR revises an older notebook that demonstrated how to combine tidal modelling with dea data. It shows how to use the new tidal_tag and tidal_stats functions that aim to make it easier to quickly tag satellite data with tidal heights, and evaluate whether the satellite is observing a biased portion of the tidal range.

Checklist

[x] Notebook created using the DEA-notebooks template
[x] Remove any unused Python packages from Load packages
[x] Remove any unused/empty code cells
[x] Remove any guidance cells (e.g. General advice)
[x] Ensure that all code cells follow the PEP8 standard for code. The jupyterlab_code_formatter tool can be used to format code cells to a consistent style: select each code cell, then click Edit and then one of the Apply X Formatter options (YAPF or Black are recommended).
[x] Include relevant tags in the final notebook cell (refer to the DEA Tags Index, and re-use tags if possible)
[x] Clear all outputs, run notebook from start to finish, and save the notebook in the state where all cells have been sequentially evaluated
[x] Test notebook on both the NCI and DEA Sandbox (flag if not working as part of PR and ask for help to solve if needed)
[x] If applicable, update the Notebook currently compatible with the NCI|DEA Sandbox environment only line below the notebook title to reflect the environments the notebook is compatible with

@erialC-P Just fixed up everything except the display_map issue! (see #486 )


disable FileType qf for nvim ≥ 0.4

Fixes #352 (for nvim).

I can't accept this patch, because detecting feature availability using has-patch is consistent way in Vim, and sooner or later correct in NeoVim as well. It is almost 2 years and a half passed since Vim 8.2 is released, and it is responsibility of NeoVim devs to include the changes of Vim. I think there're only a few patches to bump the v:version in NeoVim, so I would recommend to help them finish their works.


Add placeholder drawer that opens and closes on app language button press in settings pane

We have not discussed how this will look in landscape. But I imagine the default is to fullscreen and not allow drag down, but "done" button press works or maybe slide from side. Not sure, curious to test.

I did something explicit for this - it shows the modal still but with higher height and a fixed width in the middle.

The landscape modal is exactly what I envisioned, great foresight there.
It might be better to set a higher height on portrait as well, or otherwise indicate a list with a scrollbar? Because it's not evident that a user can scroll up and see more options.

"panjābī" is being cut off (perhaps wrong line-height on title items)
"hindī" is being just barely clipped at the top (same as second)
once scrolled up on portrait (or just default open on landscape) there is a white bar at the bottom or a lack of background? not sure how to explain, here is a screenshot

I'm also able to break the modal, not sure if that falls under this issue. Reproduction:

Swipe down most of the way (but do not dismiss, keep finger on screen)
Rapidly tap the background (outside dimmed area)
You do not have to let go of first finger to reproduce

You will get the following:
Note:

You cannot tap app language button in landscape (it shows presses, but not working)
You cannot tap done to close (it shows presses in portrait, but not working)
Gets hidden in landscape and removes dimming (but I don't think you can even swipe out of settings)

Lastly, I believe the border-radius is too high for iOS (see "Apple Pay" implementation when paying). But I definitely don't mind either way and this border-radius issue doesn't seem to be a part of the ACs.

Initial setting is not being properly set for "English (UK, Canada, India)" etc. The app fallsback to whatever is below that. In the example below, if I reorder Hindi above French, then the app will use Hindi. As it is currently shown below, first install of the app defaults to French.


Optimize computation of DEEP queries in recursive verifier

Using better stack management, optimizes the computation of the DEEP queries by about 6%. This is less than the 10% we got in the signature verifier as the VM trace is wider and hence the impact of the optimizations is smaller.

Thanks for the review!

This is ready to be merged

I have also included the changes from  #1596 as it was the easiest way to get commits in that one signed


Add DREQ getter methods for all peripherals (that are missing them)

In the spirit of static inline uint pio_get_dreq(PIO pio, uint sm, bool is_tx)

merged into develop


[redis-plus-plus] Update to 1.2.1 and support Windows

Is your feature request related to a problem? Please describe.
I cannot build redis-plus-plus on vcpkg in Windows. Redis-plus-plus version 1.2.1 supports building in Windows.
Proposed solution
Update to version 1.2.1 and enable building support for Windows.
Additional context
See commit that supports building on Windows: https://github.com/sewenew/redis-plus-plus/commit/c48d79bf62a212d86a7b6b71b08aa16411eda861

I tried pushing my branch with a proposed solution, but seems I'm missing some credentials. Anyway here's a patch if someone wants to continue with this issue.
From 21a805c1f54b52d467e7fb0b7846492cc65fb50e Mon Sep 17 00:00:00 2001
From: Daniel Lundborg<EMAIL_ADDRESS>Date: Thu, 22 Oct 2020 10:33:17 +0200
Subject: [PATCH] [redis-plus-plus] Update to 1.2.1

Also enable Windows support

Signed-off-by: Daniel Lundborg<EMAIL_ADDRESS>---
 ports/redis-plus-plus/CONTROL                 |  2 +-
 .../fix-static-compile-windows.patch          | 16 ++++++++++++++++
 .../fix-ws2-linking-windows.patch             | 12 ++++++++++++
 ports/redis-plus-plus/portfile.cmake          | 19 ++++++++++++++-----
 4 files changed, 43 insertions(+), 6 deletions(-)
 create mode 100644 ports/redis-plus-plus/fix-static-compile-windows.patch
 create mode 100644 ports/redis-plus-plus/fix-ws2-linking-windows.patch

diff --git a/ports/redis-plus-plus/CONTROL b/ports/redis-plus-plus/CONTROL
index 77a355f40..d3b824f60 100644
--- a/ports/redis-plus-plus/CONTROL
+++ b/ports/redis-plus-plus/CONTROL
@@ -1,5 +1,5 @@
 Source: redis-plus-plus
-Version: 1.1.2
+Version: 1.2.1
 Homepage: https://github.com/sewenew/redis-plus-plus
 Description: This is a C++ client for Redis. It's based on hiredis, and written in C++ 11.
 Build-Depends: hiredis
\ No newline at end of file
diff --git a/ports/redis-plus-plus/fix-static-compile-windows.patch b/ports/redis-plus-plus/fix-static-compile-windows.patch
new file mode 100644
index 000000000..8be821a6c
--- /dev/null
+++ b/ports/redis-plus-plus/fix-static-compile-windows.patch
@@ -0,0 +1,16 @@
+diff --git a/CMakeLists.txt b/CMakeLists.txt
+index eb3ff66..d8184b6 100644
+--- a/CMakeLists.txt
++++ b/CMakeLists.txt
+@@ -54,6 +54,11 @@ if (REDIS_PLUS_PLUS_BUILD_STATIC)
+         target_compile_definitions(${STATIC_LIB} PRIVATE NOMINMAX)
+         set_target_properties(${STATIC_LIB} PROPERTIES CXX_STANDARD ${REDIS_PLUS_PLUS_CXX_STANDARD} CXX_EXTENSIONS OFF)
+         set_target_properties(${STATIC_LIB} PROPERTIES OUTPUT_NAME ${PROJECT_NAME}_static)
++        target_compile_options(${STATIC_LIB} PUBLIC
++            "$<$<AND:$<BOOL:${MSVC}>,$<CONFIG:>>:/MT>"
++            "$<$<AND:$<BOOL:${MSVC}>,$<CONFIG:Debug>>:/MTd>"
++            "$<$<AND:$<BOOL:${MSVC}>,$<CONFIG:Release>>:/MT>"
++        )
+     else()
+         set_target_properties(${STATIC_LIB} PROPERTIES OUTPUT_NAME ${PROJECT_NAME})
+     endif()
diff --git a/ports/redis-plus-plus/fix-ws2-linking-windows.patch b/ports/redis-plus-plus/fix-ws2-linking-windows.patch
new file mode 100644
index 000000000..9171a38bc
--- /dev/null
+++ b/ports/redis-plus-plus/fix-ws2-linking-windows.patch
@@ -0,0 +1,12 @@
+diff --git a/CMakeLists.txt b/CMakeLists.txt
+index eb3ff66..1309213 100644
+--- a/CMakeLists.txt
++++ b/CMakeLists.txt
+@@ -77,6 +77,7 @@ if (REDIS_PLUS_PLUS_BUILD_SHARED)
+         target_compile_definitions(${SHARED_LIB} PRIVATE NOMINMAX)
+         set_target_properties(${SHARED_LIB} PROPERTIES CXX_STANDARD ${REDIS_PLUS_PLUS_CXX_STANDARD} CXX_EXTENSIONS OFF)
+         set_target_properties(${SHARED_LIB} PROPERTIES WINDOWS_EXPORT_ALL_SYMBOLS TRUE)
++        target_link_libraries(${SHARED_LIB} Ws2_32.lib)
+     endif()
+
+     target_link_libraries(${SHARED_LIB} ${HIREDIS_LIB})
diff --git a/ports/redis-plus-plus/portfile.cmake b/ports/redis-plus-plus/portfile.cmake
index 5878671e4..e4c520794 100644
--- a/ports/redis-plus-plus/portfile.cmake
+++ b/ports/redis-plus-plus/portfile.cmake
@@ -1,18 +1,27 @@
-#Blocked by ninja: error: build.ninja:348: multiple rules generate lib/redis++.lib [-w dupbuild=err]
-vcpkg_fail_port_install(ON_TARGET "Windows")
-
 vcpkg_from_github(
     OUT_SOURCE_PATH SOURCE_PATH
     REPO sewenew/redis-plus-plus
-    REF b08b36a9a91de00636e583307610f49af7876f50 # 1.1.2
-    SHA512 6dcead9fca9e7082ace28dcd72a1b325e229297080eea3e5a28ef5e6b9e4a7d1bcb3568997a5e7a031d7937a025a017ed92d7869db5829ba6113783c84bc5a68
+    REF a9f9c301f8de1c181e6d45c573b5d1fe7b8200b1 # 1.2.1
+    SHA512 038641a91a2d62f8b07b548fb16f03e87a014fc3a53e59d95e9d31d707e800de4922838bc1bc181d642bb6ac23eae877e0a8724548d9d8f113ebc8d744384abf
     HEAD_REF master
+    PATCHES fix-static-compile-windows.patch fix-ws2-linking-windows.patch
 )

+if (VCPKG_CRT_LINKAGE STREQUAL static)
+    set(REDIS_PLUS_PLUS_BUILD_STATIC ON)
+    set(REDIS_PLUS_PLUS_BUILD_SHARED OFF)
+else()
+    set(REDIS_PLUS_PLUS_BUILD_STATIC OFF)
+    set(REDIS_PLUS_PLUS_BUILD_SHARED ON)
+endif()
+
 vcpkg_configure_cmake(
     SOURCE_PATH ${SOURCE_PATH}
     PREFER_NINJA
     OPTIONS
+        -DREDIS_PLUS_PLUS_USE_TLS=OFF
+        -DREDIS_PLUS_PLUS_BUILD_STATIC=${REDIS_PLUS_PLUS_BUILD_STATIC}
+        -DREDIS_PLUS_PLUS_BUILD_SHARED=${REDIS_PLUS_PLUS_BUILD_SHARED}
         -DREDIS_PLUS_PLUS_BUILD_TEST=OFF
 )

--
2.27.0.windows.1


[iOS] Returns a specific error message when app has no access to Photos

If user cancel the UIImagePickerController and the app has no authorization to access to the Photos, an error with the message "has no access to assets" is returned as the plugin result.

Hi, sorry to resurrect an old thread but i'm having trouble with this modification.
When users click cancel in camera interface, we get a has no access to assets error instead of no image selected.
@m-revetria do you have more information about your use case ?
I fail to see how this test is pertinent here.
If user cancels action then it's just a cancellation, not a auth error.

Hi, Nicolas.
The use case was show an alert message to the user indicating:

If he canceled the selection of an image that he must select an image
in order to continue.
If he canceled due to the app has no access to the photos
library/camera, then show an alert message to let the user knows the
app wasn't granted to access that resource.

That error could happen in the case where the user previously denied access
to camera roll and later he opens the camera to take a picture but he
cancels. In that case [ALAssetsLibrary authorizationStatus] != ALAuthorizationStatusAuthorizedand the error "has no access to assets" is
returned wrongly.
I have solved this problem in commit:
https://github.com/m-revetria/cordova-plugin-camera/commit/bbdafb08640cafc8e98ed0cbd5d558ad9b6dc150
.
I didn't create a PR with this change yet.
Thanks!
[image: xmartlabs website] http://xmartlabs.com/
MIGUEL REVETRIA
iOS Engineer

https://www.linkedin.com/in/mrevetria https://github.com/m-revetria
On Thu, Dec 3, 2015 at 1:22 PM, Nicolas Leclerc<EMAIL_ADDRESS>wrote:

Hi, sorry to resurrect an old thread but i'm having trouble with this
modification.
When users click cancel in camera interface, we get a has no access to
assets error instead of no image selected.
@m-revetria https://github.com/m-revetria do you have more information
about your use case ?
I fail to see how this test is pertinent here.
If user cancels action then it's just a cancellation, not a auth error.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/cordova-plugin-camera/pull/24#issuecomment-161702527
.

Thanks for the answer.
Actually i was a bit confused because i came over the PR from another
commit, hence my question.
This new commit seems very nice, i hope it can be merged quickly ! :-)
On Fri, 4 Dec 2015 at 15:08 Miguel Revetria<EMAIL_ADDRESS>wrote:

Hi, Nicolas.
The use case was show an alert message to the user indicating:

If he canceled the selection of an image that he must select an image
in order to continue.
If he canceled due to the app has no access to the photos
library/camera, then show an alert message to let the user knows the
app wasn't granted to access that resource.

That error could happen in the case where the user previously denied access
to camera roll and later he opens the camera to take a picture but he
cancels. In that case [ALAssetsLibrary authorizationStatus] != ALAuthorizationStatusAuthorizedand the error "has no access to assets" is
returned wrongly.
I have solved this problem in commit:
https://github.com/m-revetria/cordova-plugin-camera/commit/bbdafb08640cafc8e98ed0cbd5d558ad9b6dc150
.
I didn't create a PR with this change yet.
Thanks!
[image: xmartlabs website] http://xmartlabs.com/
MIGUEL REVETRIA
iOS Engineer

https://www.linkedin.com/in/mrevetria https://github.com/m-revetria
On Thu, Dec 3, 2015 at 1:22 PM, Nicolas Leclerc<EMAIL_ADDRESS>wrote:

Hi, sorry to resurrect an old thread but i'm having trouble with this
modification.
When users click cancel in camera interface, we get a has no access to
assets error instead of no image selected.
@m-revetria https://github.com/m-revetria do you have more information
about your use case ?
I fail to see how this test is pertinent here.
If user cancels action then it's just a cancellation, not a auth error.
—
Reply to this email directly or view it on GitHub
<
https://github.com/apache/cordova-plugin-camera/pull/24#issuecomment-161702527
.

—
Reply to this email directly or view it on GitHub
https://github.com/apache/cordova-plugin-camera/pull/24#issuecomment-161976293
.


Some typos in the Readme.md file

<-- Bird is however lacking an API to allow IXPs to provide thise same looking glass type tools. More over, this also affects an IXP's ability to monitor these routing daemons and member sessions to them.
--> Bird is, however, lacking an API to allow IXPs to provide these same looking glass type tools. Moreover, this also affects an IXP's ability to monitor these routing daemons and member sessions to them.

Main one is "thise" --> "these" plus some missing commas around however and "Moreover" is a single word.

fixed in 4f24ee5


Cycle detection in moduleDeps and compileModuleDeps

The problem: We currently don't detect cycles in transitive moduleDeps and compileModuleDeps. This is results in a StackOverflowError.
To solve this, we first need to detect cycles. This is the newly added Util.recursive method. Instead of moduleDeps we use the moduleDepsChecked in some places to ensure we checked for cycles at least once.
Unfortunately, we can't easily report the detected cycle at evaluation time, as moduleDeps needs to be evaluated before we can create a task graph (via T.traverse), hence we need an alternative way to propagate the exception. The current approach throws an BuildScriptException which itself derives from the also new MillException.
In MillMain, our central entry point for Mill, we catch MillException and report the error to the user.

Fix https://github.com/com-lihaoyi/mill/issues/2341

I don't think we have any choice but to do it lazily. Scala objects are already lazily instantiated, so any error will get thrown during the resolveTasks stage, as we collect the necessary tasks, but before evaluation actually starts. Throwing and catching seems like the only way

Hm, some tests run way too long and got cancelled. Probably unrelated, restarting them again.

Superseded by #2790


Make connections work!

It will be great if we can type check this with only auto? (Currently RedPRL is quite slow and looks buggy on this, not to mention proving it.)
Thm Connection(#A) : [
  a/type : #A type >>
     (a : #A)
  -> (b : #A)
  -> (p : paths({_}.#A; a; b))
  -> paths({i}.(paths({_}.#A; a; p@i)); <_>a; p)
] by [
  {
  lam a. lam b. lam p.
    <i> <j> 'hcom{i=0,i=1,j=0,j=1;0~>1}(#A; ,a;
     {k}.hcom{k=0,k=1;1~>j}(#A; hcom{k=0,k=1;1~>0}(#A; ,p @ k; {_}.,a; {j}.,p @ j); {_}.,a; {_}.,a);
     {k}.hcom{k=0,k=1;1~>j}(#A; ,p @ k; {_}.,a; {j}.,p @ j);
     {k}.hcom{k=0,k=1;1~>i}(#A; hcom{k=0,k=1;1~>0}(#A; ,p @ k; {_}.,a; {i}.,p @ i); {_}.,a; {_}.,a);
     {k}.hcom{k=0,k=1;1~>i}(#A; ,p @ k; {_}.,a; {j}.,p @ j))
  };
].

@favonia I experimented with this and there is just a blowup of proof obligations.

And there must be some bugs somewhere too, because I am seeing incorrect subgoals arising. It is possible that either the bug is in the rules, or in the refinement apparatus (the latter is probably too complex for you to debug, but I can look into it.)

I have at least mostly fixed the performance problems which made this kind of impossible to test (the blow-up of subgoals was partially caused by using a "parallel" version of tactic sequencing, in which substitutions don't propagate along the subgoals telescope until after all subgoals have been refined.)
I still get goals now which look either non-trivial or untrue. But at least we can try and figure out what is going on.

Fixed a bug in the example. (Finally it's RedPRL helping me debugging.) However there are still four goals left unsolved.

@favonia Awesome! And once I get around to adding refinement rules for compositions (so that you don't have to use '/exact), it will be easier to write correct compositions anyway.
Regardless, I think that RedPRL should (in a moral sense) be able to automatically "typecheck" this example, so we can use it as a target: let's keep fixing things and adding rules if necessary until this example is automatically discharged.

@jonsterling You can update the code in OP. I think we really have to solve the naming issues now. We need to find a good name for these rules: @cangiuli

p@e = P_e
coe^{r~r}
hcom{r~r}
hcom{r=r}

The three cases are due to the lack of the symmetry of CapEq. Is there a way to sequence two tactics, like (Symmetry >> CapEq)? Should we have a special version to handle the cases where both sides are hcom? If so, what should be its name?

@favonia

OK, I'll address the naming issue prior to merging.
There is a way to sequence rules, but let's not do that for now; currently, doing so may cause unexpected behavior with respect to the name store, so I want to keep things as they are for now. I have a plan to address that issue, after which we can remove the "two-step" things and replace them with tactic compositions.

As I told @favonia today, I don't have any good names for these rules. Proof-theoretically, the first one is a kind of elimination rule: the three hypotheses of path-intro are that you have a path and that its endpoints agree with the type, so the three elimination rules correspond to dimension application and recovering the two endpoints.
The other three equations I currently just refer to as the second, third, and fifth Kan conditions, but you are welcome to make up names.


Medigun names show up when streamer mode is active and player is being healed

See title: Medigun names show in the overlay that pops up when one is getting healed by a Medic.

This is a known issue, but unfortunately I'm not really able to fix it because nuking names would also nuke things like the level of engineer buildings when you look at them, as well as just being able to tell what medigun someone is using in general (since they're all tied to the same element)

Just checked to make sure and it appears I left a comment about it in the file it applies to: https://github.com/rbjaxter/budhud/blob/master/stream/resource/ui/targetid.res#L8-L18
That said, if you don't care about those names being missing you can always uncomment this section, and then it will also remove those medigun names


two new more advanced API use case examples: FASTA and GFF export

Current coverage is 59.81%

Merging #102 into develop will not affect coverage as of 585ca67

@@            develop    #102   diff @@
=======================================
  Files            15      15
  Stmts          2322    2322
  Branches          0       0
  Methods           0       0
=======================================
  Hit            1389    1389
  Partial           0       0
  Missed          933     933

Review entire Coverage Diff as of 585ca67

Powered by Codecov. Updated on successful CI builds.

Ok, so do you want to work on getting the tests to pass in Travis CI from this PR or do you want to move forward with what Dan G. did?

@realmarcin I can't merge this until the Travis CI tests pass.


EKF2_AID_MASK: rotate external vision in 1.7.3

Hello,
Just want to ask for some information about the newly added option rotate_external_vision(6) for EKF2_AID_MASK.
I wonder if I have a SLAM measurement in NED, but not in true-north, do I need to enable this? and automatically getting the position in true-north by fusing the magnetometer?
If that is the case, will it be problematic if I also enable vision_yaw_fusion (4)? Since from what I understand, enabling vision_yaw_fusion will disable magnetometer fusion.
If that is not the case, what is the purpose rotate_external_vision option?
Thank you in advance.
Chang

hello?

You cannot enable both fusion flags at once. They are mutually exclusively in the current implementation. Enabling rotate_external_vision will estimate the offset between the vision frame and true NED. The position estimate will be in true NED. The EKF thus keeps using the compass for yaw estimation, and this is not ideal, but we are working on a better solution.

Do we need an additional metadata mechanism to explicitly block invalid combinations? The other quick thing we could do is put a check in EKF2 updateParams() to log an error.

Thank you for the clarification @mhkabir.
Great! That's one step forward! Despite the magnetic interference, at least we can always fuse GPS + Vision together! (that is what's happening when enables use_GPS + vision_position_fusion + rotate_external_vision, right?)
Great work!


sections is not defined on Theme Editor unload event with Sections JS

Problem
Using Slate with Shopify/starter-theme, the theme editor throws a sections is not defined error on the unload event inside the sections.cjs.js file. This also prevents scripts inside the load event from running.
Replication steps

Using Shopify starter theme, add a console log inside the initalization of product.js.

/**
 * Product section constructor. Runs on page load as well as Theme Editor
 * `section:load` events.
 * @param {string} container - selector for the section container DOM element
 */

sections.register('product', {
  onLoad() {
    console.log('product section');

Inside the theme editor, navigate to product page and uncheck Show social sharing buttons setting to trigger an unload then load of the product section.

More information
Unsure if this is a Slate issue or a Shopify Starter Theme issue. But since error is getting thrown from @shopify/theme-sections I opened up the issue here.

I've managed to solve this by changing the ref to the sections variable from sections to __sections__ which matches the export default window.__sections__; in sections.js
sections.cjs.js
function _onSectionUnload(event) {
  if (this.id !== event.detail.sectionId) {
    return;
  }

  event.type = "section_unload";
  this.trigger(event);

  this.off(this.namespace);
  __sections__.off(this.namespace);
  (0, _jquery2.default)(document).off(this.namespace);
  (0, _jquery2.default)(window).off(this.namespace);

  (0, _remove2.default)(__sections__.instances, { id: this.id });
}

I realise this may not be related to Slate and more appropriate for Shopify/theme-scripts. If so I'll close this issue and open up a PR there.

Ah, found out @t-kelly is one step ahead. I'll close this.


[FEATURE] Improve the UI of login page

Description
Hey @narayansoni I want to improve the UI of the login page. The image on the right side looks distorted. I also wish to add the official colors of Google and Facebook on the login buttons. Please assign me this issue. Thank you!Google
Screenshots

Additional information
No response

Description
Hey @narayansoni I want to improve the UI of the login page. The image on the right side looks distorted. I also wish to add the official colors of Google and Facebook on the login buttons. Please assign me this issue. Thank you!Google
Screenshots

Additional information
No response

I think icons have been added, can you fix other aspects you see fit?

Yeah sure @narayan954


Missing "by Spotify" playlists from my library in  MA

What version of Music Assistant has the issue?
2.3.3
What version of the Home Assistant Integration have you got installed?
2024.12.3
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
When connecting my Spotify account, the playlists created by Spotify which are saved in my Library do not appear in MA.
My personal playlist as well as other public/shared lists sync ok, only the "created by Spotify" ones do not (neither general thematic lists such as https://open.spotify.com/playlist/37i9dQZF1DX0Yxoavh5qJV, nor personalized such as daily mixes...)
How to reproduce
Happened on first synchronisation.
Sync or reload didn't help
Music Providers
Spotify
Player Providers
Google Cast
Full log output
No response
Additional information
adding A Spotify Dev API key didn't help.
No error in log:
2024-12-14 10:05:15.667 INFO (MainThread) [music_assistant.spotify] Successfully logged in to Spotify as User
2024-12-14 10:05:15.668 INFO (MainThread) [music_assistant] Loaded music provider Spotify
2024-12-14 10:06:03.763 INFO (MainThread) [music_assistant.music] Sync task for Spotify: User completed

What version of Home Assistant Core are your running
2024.12.3
What type of installation are you running?
Home Assistant Container
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

This is not an issue it is a feature request which already exists https://github.com/orgs/music-assistant/discussions/1797


incompatiblity wih p-mysql >= 32

latest version of p-mysql disallow users to LOCK TABLES.
see https://github.com/cloudfoundry/cf-mysql-release/releases/tag/v32
Should adjust db-dumper script génération to remove lock tables clause (mysql --skip-add-locks)

Thanks for the report, it's been fixed on this commit: https://github.com/orange-cloudfoundry/db-dumper-service/commit/1d260ba0441514e249dd97815a9a8abaac926e55


[Demo & docs] Clears demo site cache on startup

Pull Request
📖 Description
Clears the browser cache when the demo site is started.
🎫 Issues
Old versions of the markdown are retained between sessions. So when a new version of the site is released, new markdown content will not be downloaded if an older version is cached.
For example, the WhatsNew page continues to render previous version.
👩💻 Reviewer Notes
I've made the CacheStorageAccessor a singleton and provided a flag to check if the cache has been previously cleared.
I would have preferred being to tag the cache but was not sure if the extra effort was necessary. That way, the cache would only be cleared if there is a new version or build.  Thoughts?
📑 Test Plan
✅ Checklist
General

[ ] I have added tests for my changes.
[x] I have tested my changes.
[ ] I have updated the project documentation to reflect my changes.
[X] I have read the CONTRIBUTING documentation and followed the standards for this project.

⏭ Next Steps

In the CacheStorageAccessor.js script, I 'put' the responses with a cache duration (7 days) and chechk the validity on the 'get' action. Wouldn't this change make that part a bit useless then? As everything gets cleared out when the site is restarted anyway.
Maybe we should just shorthen the cache duration (to 2 or 3 days)?

In the CacheStorageAccessor.js script, I 'put' the responses with a cache duration (7 days) and chechk the validity on the 'get' action. Wouldn't this change make that part a bit useless then? As everything gets cleared out when the site is restarted anyway. Maybe we should just shorthen the cache duration (to 2 or 3 days)?

Shortening the cache time will still result in the user getting old information for several days after a new release. I'm thinking in the case of breaking changes.
The best solution would be to tag the cache with the version and build information (Version: 4.6.1+ff23edc6). Check to see if there is a match and only flush the cache if they dont' match.
The plan would be to implement a new service that returns the current build information and inject it into CacheStorageAccessor. Thoughts?

Yes, that sounds like a good plan. Tagging each item is the way to go, I think

@vnbaaij
I put a file in the cache ('appVersion') with the version corresponding to the files cached. I used the same version string as that shown in the footer of the demo site. When this version is updated, all files cached are considered stale and removed.

Changing status back to draft. I accidentally reverted something. 😒


Sound Mac -> CC with delay

Hello,
while using this awesome app to cast the sound from a MBPr Early 2015 i noticed a delay between two and three seconds. Even between the volume changes. So for example:
Hearing music via Spotify -> Pause -> 2-3 seconds still playing -> Stops
It not the worst thing ever happened but is that normal? Or still an issue caused out of development reasons? I'm an IT student so i would expect that there is some delay through using different frameworks to get it working, but a confirmation would be nice :)
Best regards

FYI: https://github.com/andresgottlieb/soundcast/issues/11


Fix properties in base inventory classes.

Change property access modifier to private.
Add get methods.
Fix property initialization.

I guess I have caused some confusion. As I understand Properties do not need any other separate getters or setters, as they have {get; set;}. You can check out a video on the Properties and their getters/setters (https://www.youtube.com/watch?v=EbW-1fPhXQE&ab_channel=MikeDane). Properties should be public.


Fix and enhance info reporter

The info reporter in the original version contains various bugs (for example,
skipping can lead to crashes) and the reporter is not as fancy as it could be.
For example, the new info reporter gives summaries for each context
and omits showing nested contexts if they are skipped.
(Note that we are using this version of the info reporter for over two years now,
so I only cherry-picked the development commits.)
[WIP] because I am currently writing the long awaited tests for the reporter and this will lead
to some further behavior-changing commits. I only started the pull request
to allow a review of the non-WIP commits. (I am used to GitLab where [WIP]
makes the pull request unmergeable. I hope this works here, too.)

This is not [WIP] any longer.
Note that appveyor also fails on master, so this is unrelated.


[Enhancement] Pass some context to renderView

Problem context
When rendering my view, I had an issue with the space occupied by the scrollbar. I do not want the scrollbar to hover my content, but I don't want to leave a blank space when there is no scrollbar.
Suggested solution
The renderView property is a simple react component React.StatelessComponent<any>.
If you add a property for context it could open some rooms for several improvements :
interface ScrollbarsViewContext = {   hasScrollbar: boolean;   fullViewHeight: number;   scrollbarWidth: number;   scrollbarHeight: number; }
The first one clearly solves my problem, the others are just ideas from the top of my head.

Fair enough. I don't have a lot of time, but if you are happy to work up a PR for this I'll take a look.
Depending on your browser requirements though a slider that sits outside the container may be easier and v.possible with native browser css requirements

I had the same issue, and I wish to share my solution.
I created a new wrapper component. Hopefully this will help others.
There is a bug with getValues() method, it will always return the same values for scrollHeight and clientHeight on the first render. So to fix this, I've added an *init state value, which updates via the useEffect hook. This triggers the next  useEffect hook which contains out value checking. If the scrollHeight is larger than the clientHeight then we set isScrollable to true.
Inside the renderView function, I'm adding the pr-4 class (this is a TailwindCSS class) to push the view away from the track.
This code contains customisation to the tracks. It also introduces 'pointer-events-none' (again, a TailwindCSS class) so that when the scrolling is active, the mouse pointer doesn't trigger any hover states on the content. Pretty nifty I think.

import {Scrollbars} from 'react-custom-scrollbars-2';
import classNames from 'classnames';

    const Scroll = (props) => {
        const vertTrackWidthChosen = '16px'; // Original is 6px
        const horzTrackHeightChosen = '16px'; // Original is 6px

        const [isSelectable, setIsSelectable] = useState(true);
        const [isScrollable, setIsScrollable] = useState(false);

        const scrollbars = useRef();

        const [init, setInit] = useState(false);

        useEffect(() => {
            const values = scrollbars.current.getValues();
            if (values?.scrollHeight >= values?.clientHeight) setIsScrollable(true);
        }, [init]);

        useEffect(() => {
            setInit(true);
        }, []);

        return (
            <Scrollbars
                ref={scrollbars}
                hideTracksWhenNotNeeded
                onScrollStart={() => setIsSelectable(false)}
                onScrollStop={() => setIsSelectable(true)}
                {...props}
                renderThumbVertical={(props) => <div {...props} className="bg-green-300 rounded" />}
                renderView={({style, ...props}) => {
                    return (
                        <div
                            className={classNames('', {
                                'pr-4': isScrollable,
                            })}
                            style={style}
                            {...props}
                        />
                    );
                }}
                renderTrackHorizontal={({style, ...props}) => {
                    return <div {...props} className="" style={{...style, height: horzTrackHeightChosen}} />;
                }}
                renderTrackVertical={({style, ...props}) => {
                    return (
                        <div
                            {...props}
                            className="ml-16 top-0 bottom-0 right-0 p-0.5 bg-gray-900 bg-opacity-20"
                            style={{...style, width: vertTrackWidthChosen}}
                        />
                    );
                }}
            >
                <div
                    className={classNames('', {
                        'pointer-events-none': !isSelectable,
                    })}
                >
                    {props.children}
                </div>
            </Scrollbars>
        );
    };

Use it like this
<Scroll thumbSize={30}>
    _your content_
</Scroll>


online demo not working without www. (wrong cert)

without www it apparently seems to load the wrong cert (issued to "*.azurewebsites.net"), this is even more frustrating when some users dont actually enter www (like me) or browser just ax the www from the URLbar.

This is now finally fixed. Gave up figuring out the LE + Azure setup (https://github.com/sjkp/letsencrypt-azure/pull/9) and just placed passwordless.dev behind cloudflare.


scalable randomization

add random_no INT column to items table and index it.  Set it to Math.round(Math.random()*ROUGHLY_TABLE_SIZE)) when creating a record (and consider updating it in a beforeUpdate hook - or whatever the node version of that is..)
That way we can select random items via:
var rnd = Math.random()
var pivot = rnd * ROUGHLY_TABLE_SIZE
var direction = rnd > 0.5 ? '>=' : '<'
var sql = `SELECT * FROM items WHERE random_no ${dir} ${pivot} LIMIT ${LIMIT}`

.. something like that..

Thanks! Let's do this once we include more collections to Surveyor.


generate bitcode for libgd

hello,
I want to use this tool for generating bitcode for libgd.
my llvm version in 10.
$ls -l /usr/bin/clang
lrwxrwxrwx 1 root root 17 Mar 27 04:16 /usr/bin/clang -> /usr/bin/clang-10

and I used these commands:
git clone https://github.com/libgd/libgd.git
cd libgd
export LLVM_COMPILER=clang
CC=wllvm ./configure
make
cd src
extract-bc gd2togif

ERROR:File "/libgd/src/gd2togif" of type UNKNOWN cannot be used

can you please help me?

If you just run make and look at those intermediate results,gd2togif is a shell script. Look at the executables, or try running install to get libtool to finish processing everything.

Thank you,
yes, executables are in src/.libs
I try this but again there is a problem!
.libs$ extract-bc gd2togif

ERROR:WLLVM Failed to execute: ['llvm-link',
 '-o=/libgd/src/.libs/gd2togif.bc',
 '/libgd/src/.gd2togif.o.bc']
ERROR:Your llvm-link does not seem to be easy to find.
Either install it or use the -l llvmLinker option.

extract-bc needs access to the llvm-link command in order to combine bitcode. If it is not in your PATH, you need to pass extract-bc the -l argument to tell it where to find your copy of llvm-link (possibly suffixed with a version number)

what is the difference between using ./configure; make; and using cmake for generating executable files for this project?
why generated .bc files are different?
I used these commands to generate
export LLVM_COMPILER=clang

mkdir bld
cd bld
CC=wllvm CXX=wllvm++ cmake ../
make

extract-bc Bin/libgd.so.3.0.10

export LLVM_COMPILER=clang

CC=wllvm ./configure
make

extract-bc src/.libs/libgd.so.3.0.10

Thank you


Error for any table operations (refresh, change column order, etc)

Description
A popup error message appear each time i try to do some operation on table/column using the gui tools provided.
For example, when i try to re-order a column:

Or if i refresh the table tab:

Tipically the error does not affect the success the operation.
The problem occurs with every table, the one of the images is for example only
logs:
eclipse.buildId=unknown
java.version=17.0.6
java.vendor=Eclipse Adoptium
BootLoader constants: OS=linux, ARCH=x86_64, WS=gtk, NL=en_US
Framework arguments:  -eclipse.keyring /home/ale/.local/share/DBeaverData/secure/secure_storage
Command-line arguments:  -os linux -ws gtk -arch x86_64
This is a continuation of log file /home/ale/[omitted]/dbeaver/sei/.metadata/.bak_1.log
Created Time: 2024-04-26 10:35:39.749
org.jkiss.dbeaver.model
Error
Fri Apr 26 10:53:07 CEST 2024
You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '= 'event_queue'' at line 1
java.sql.SQLException: You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '= 'event_queue'' at line 1
at org.mariadb.jdbc.internal.protocol.AbstractQueryProtocol.readErrorPacket(AbstractQueryProtocol.java:1594)
at org.mariadb.jdbc.internal.protocol.AbstractQueryProtocol.readPacket(AbstractQueryProtocol.java:1453)
at org.mariadb.jdbc.internal.protocol.AbstractQueryProtocol.getResult(AbstractQueryProtocol.java:1415)
at org.mariadb.jdbc.internal.protocol.AbstractQueryProtocol.executeQuery(AbstractQueryProtocol.java:289)
at org.mariadb.jdbc.ClientSidePreparedStatement.executeInternal(ClientSidePreparedStatement.java:221)
at org.mariadb.jdbc.ClientSidePreparedStatement.execute(ClientSidePreparedStatement.java:157)
at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCPreparedStatementImpl.execute(JDBCPreparedStatementImpl.java:261)
at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCPreparedStatementImpl.executeStatement(JDBCPreparedStatementImpl.java:205)
at org.jkiss.dbeaver.model.impl.jdbc.cache.JDBCStructLookupCache.reloadObject(JDBCStructLookupCache.java:115)
at org.jkiss.dbeaver.model.impl.jdbc.cache.JDBCStructLookupCache.refreshObject(JDBCStructLookupCache.java:85)
at org.jkiss.dbeaver.ext.mysql.model.MySQLTableBase.refreshObject(MySQLTableBase.java:139)
at org.jkiss.dbeaver.ext.mysql.model.MySQLTable.refreshObject(MySQLTable.java:699)
at org.jkiss.dbeaver.model.navigator.DBNDatabaseNode.lambda$0(DBNDatabaseNode.java:362)
at org.jkiss.dbeaver.model.exec.DBExecUtils.tryExecuteRecover(DBExecUtils.java:194)
at org.jkiss.dbeaver.model.navigator.DBNDatabaseNode.refreshNode(DBNDatabaseNode.java:360)
at org.jkiss.dbeaver.ui.navigator.actions.NavigatorHandlerRefresh$1.run(NavigatorHandlerRefresh.java:189)
at org.jkiss.dbeaver.model.runtime.AbstractJob.run(AbstractJob.java:115)
at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)
DBeaver Version
24.0.3
Operating System
Ubuntu 20.04
Database and driver
mariadb 3.3.2
Steps to reproduce

have a database
have a table and columns
some actions that cause the error:
3a. change the datatype of some column and save
3b. change the position of columns (using icons) and save
3c. refresh the table (F5 or refresh button)

Additional context
No response

Does your database name contain non-latin characters?

Does your database name contain non-latin characters?

No

Please provide driver and server version. You can get this information by clicking Test connection... button in the Connection configuration dialog.

@a-locatelli Hi, could you open the Query Manager and give me information on the screenshot from it after the failed query?
https://dbeaver.com/docs/dbeaver/Query-Manager/

Please provide driver and server version. You can get this information by clicking Test connection... button in the Connection configuration dialog.

Hi, here the screenshot of 'Test Connection' dialog:

@a-locatelli Hi, could you open the Query Manager and give me information on the screenshot from it after the failed query? https://dbeaver.com/docs/dbeaver/Query-Manager/

Hi,
i didn't know the Query Manager, thank you.
Here the screenshot. Is taken after a refresh on Table view (Properties tab)

'smart3' is the schema name

Thank you for providing more information

Please try turning off this setting

Please try turning off this setting

My flag was already unchecked.
I've tried enabling it and now the error does not appear anymore....


Starting guide

Please , write starting guide for replace the Auth0 for custom auth by API rest

Thanks, this is on the roadmap. I made some initial notes about custom backend login functionality at https://github.com/futurice/pepperoni-app-kit/issues/92#issuecomment-238091047.
This will be fleshed out, and we are considering providing a Auth0-less build out of the box. I'll close this issue now, thanks for the suggestion! 👍


ImportLayoutStyle#ImportPackage always calls Pattern.compile(packageWildcard).

            public ImportPackage(Boolean statik, String packageWildcard, boolean withSubpackages) {
                this.statik = statik;
                this.packageWildcard = Pattern.compile(packageWildcard
                        .replace(".", "\\.")
                        .replace("*", withSubpackages ? ".+" : "[^.]+"));
            }

Serialization and deserialization replace the "." characters with escaped characters.
The Deserializer calls builder.importPackage(block) on a pattern like java\..+, which creates a new ImportPackage.
And generates java\.\.+., so the package wildcard fail to recognize the pattern and everything is grouped into allOtherImports.

Fixed with 2a77145f649c648a672c30c3c7bf004fdc1f8f91


Default name to ID.

Codecov Report

Merging #166 into master will decrease coverage by 0.04%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #166      +/-   ##
=========================================
- Coverage   66.94%   66.9%   -0.05%
=========================================
  Files          30      30
  Lines        2959    2961       +2
=========================================
  Hits         1981    1981
- Misses        791     793       +2
  Partials      187     187

Impacted Files
Coverage Δ

api/properties_util.go
50% <0%> (-3.34%)
:arrow_down:

Continue to review full report at Codecov.

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


Get property name from call stack for error message

I considered something similar, determining the function name using inspect instead of hard-coding the names as strings, but, after browsing SO, thought it looked too convoluted; the way you've implemented it looks good, however.


Does Jupyter notebook in VS Code insiders have "Variable explorer and data viewer" function.

In VS Code we can use the "Variable explorer and data viewer" function, and the Jupyter notebook in VS Code insiders has better "IntelliSense" and other functions.
Is there a "Variable explorer and data viewer" function in Jupyter notebook in VS Code insiders.
For example:

Thanks a lot!

Hi @Jill-Cheng , this is on the way! Planning to have this in Feb

Hi @Jill-Cheng , this is on the way! Planning to have this in Feb


Warn about multiple imports of the same module

I request that ruff warn of multiple imports of the same module, for example
import numpy as np
print(np.random.normal(size=3))
import numpy as np
print(np.random.normal(size=5))

Thanks very much for ruff.

Ruff's F811 already warns you about redefinition if the first definition is unused
import numpy as np
import numpy as np

print(np.random.normal(size=3))

print(np.random.normal(size=5))

I could see this as a possible extension of F401 that caches unused imports, assuming that re-importing can generally be considered a no-op, which I'm not sure is given. In that case it could make sense to have a separate rule (with a lower severity)

Unlike with flake8 the F811 rule in ruff doesn't trigger for me in a case like this:
import sys

if __name__ == "__main__":
    import sys
    sys.exit(0)

I also think it would be useful to get a new or extended rule for the original posters case.


Fix display of result background...

CSS change so background gradient fills result box for results with long
descriptions. Fixes part B of #16 (part A seemingly already fixed)

Closing as we are redoing the UI completely.


Use go 1.17 in Hugo module

Fixes a static web apps build failure.

@sumedhb1995 @sdeshpande3 Can one of you verify that your PR builds succeed with this change? I'd like a little more validation. :)

@sumedhb1995 @sdeshpande3 Can one of you verify that your PR builds succeed with this change? I'd like a little more validation. :)

Yep! I'll validate this one

@sumedhb1995 @sdeshpande3 Can one of you verify that your PR builds succeed with this change? I'd like a little more validation. :)

Yep! I'll validate this one

Build succeed!!


CUDA out of memory error while Fine-Tuning the DreamPose

Hi,
While I try to fine-tune the pre-trained DreamPose model with the demo data given with the project, I'm facing OOM error. I tried to play with batch sizes, and resolution but the result is the same.

I have 2x3090 in my system.
I can run the demo.py with 16~17 GB vRAM
I have used accelerate launch finetune-unet.py --pretrained_model_name_or_path="CompVis/stable-diffusion-v1-4" --instance_data_dir=demo/sample/train --output_dir=demo/custom-chkpts --resolution=512 --train_batch_size=1 --gradient_accumulation_steps=1 --learning_rate=1e-5 --num_train_epochs=500 --dropout_rate=0.0 --custom_chkpt=checkpoints/unet_epoch_20.pth --revision "ebb811dd71cdc38a204ecbdd6ac5d580f529fd8c"  command and changed the resolution to the powers of 2, to 4 but didn't work either.
I have tried to do distributed training with the config of accelerate but it didn't work either.
I have watched the GPU usage with watch nvidia-smi, and it seems that without using all of the GPU, the code generates an error.

The error is:
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 800.00 MiB (GPU 0; 23.70 GiB total capacity; 16.55 GiB already allocated; 407.56 MiB free; 17.43 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF
Steps:   0%|                                                                   | 0/5 [00:01<?, ?it/s]
WARNING:torch.distributed.elastic.multiprocessing.api:Sending process 18804 closing signal SIGTERM> ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 1) local_rank: 0 (pid: 18803) of binary: /anaconda3/envs/

Just find out there is an argument "use_8bit_adam" in the finetune-unet.py while debugging. Adding this parameter solved the problem.

"use_8bit_adam" in the finetune-unet.py while debugging   , how much memory it need ?
@emreaniloguz

"use_8bit_adam" in the finetune-unet.py while debugging , how much memory it need ?
@emreaniloguz

In the comments it says it should be activated for 16 GB GPUs but it uses ~20 GB


TitleElement on TextBox Alignment

The Title alignment on a TextBox seems to be unaligned with the control

Could we fix it so the title lines up with the edge of the textbox

No, that's how it's designed

May I ask why? It looks weird in my opinion

the title will lines up to content or placeholder, at least our designers think it looks better, It's just a matter of design.

One could extent the title placement with another attached property, e.g. a margin, to support other designs more easily.
Of course this could be taken way further generalizing more areas by allowing data templates or styles to be provided by the user.

Configuring everything equal to configuring nothing, we should keep a balance between set meals and buffet. Then again, HC is just a lightweight library.


fix: Fix transitionEnd emulation

Fixes #70

probably can close a few issues in a few places if we ship this


Implement ^ operator in expr macro

Implement binary operator ^ which given n^m computes n to the power of m. Consider using BigFloatNumber::powi for unsigned integer m, BigFloatNumber::powsi  for signed integer m, and BigFloatNumber::pow in other cases.

Since ^ has lower precedence than other arithmetic operators for the sake of simplicity ^ will be left unimplemented.


dimension error

I don't know why it's giving me dimension error, during prediction?
Traceback (most recent call last):
File "predict.py", line 225, in
predict_model(image_crops, image_crops_name)
File "predict.py", line 54, in predict_model
model = create_model(False, True, input_shape, len(class_names), optimizer, learn_rate, decay, momentum, activation, dropout_rate)
File "/home/prashant/DeepFashion/model.py", line 67, in create_model
model.load_weights(top_model_weights_path_load, by_name=True)
File "/usr/local/lib/python2.7/dist-packages/keras/engine/topology.py", line 2570, in load_weights
load_weights_from_hdf5_group_by_name(f, self.layers)
File "/usr/local/lib/python2.7/dist-packages/keras/engine/topology.py", line 3111, in load_weights_from_hdf5_group_by_name
K.batch_set_value(weight_value_tuples)
File "/usr/local/lib/python2.7/dist-packages/keras/backend/tensorflow_backend.py", line 2183, in batch_set_value
assign_op = x.assign(assign_placeholder)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variables.py", line 594, in assign
return state_ops.assign(self._variable, value, use_locking=use_locking)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/state_ops.py", line 276, in assign
validate_shape=validate_shape)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gen_state_ops.py", line 59, in assign
use_locking=use_locking, name=name)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
op_def=op_def)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 3162, in create_op
compute_device=compute_device)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 3208, in _create_op_helper
set_shapes_for_outputs(op)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2427, in set_shapes_for_outputs
return _set_shapes_for_outputs(op)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2400, in _set_shapes_for_outputs
shapes = shape_func(op)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2330, in call_with_requiring
return call_cpp_shape_fn(op, require_shape_fn=True)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/common_shapes.py", line 627, in call_cpp_shape_fn
require_shape_fn)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/common_shapes.py", line 691, in _call_cpp_shape_fn_impl
raise ValueError(err.message)
ValueError: Dimension 1 in both shapes must be equal, but are 19 and 3. Shapes are [256,19] and [256,3]. for 'Assign_32' (op: 'Assign') with input shapes: [256,19], [256,3].

Hi ! Did you solve this problem? I also encountered the same situation during prediction
“ Dimension 1 in both shapes must be equal, but are 19 and 3 for 'Assign_32' (op: 'Assign') with input shapes: [256,19], [256,3]. ”

Hi i have the same problem, did anyone find the solution

perhaps this is a mismatch in the number of  output classes. Check the output classes of the checkpoint and the base model.

@hoanganh928 is right. dataset_download.sh is failure. you should download data in Google drive and just set variable category_name_generate in dataset_create.py as line 49. and add fashion_dataset_path.

predict.py line 28 and above, change class_names = ['Coat', 'Kaftan', 'Robe']
Because your dataset_create.py creates this three classes.


Fibonacci.py

@Priyanshu729 Please delete python/palindrome_checker.py and make a commit

Hello Can You Help Me With Your Issue In This Please

Did you delete all the contents of python/palindrome_checker.py?

Can I add Fibonacci using recursion too as its very helpful for people to hop and understand from the repository easily?

@rokibulislaam Check the conflicting file python/palindrome_checker.py.


Add isSaved HologramData option

This is a nice API feature other hologram plugins have - ability to have "in-memory" holograms that do not get saved, so they are kinda like "temporary" holograms.

That is true

You can already create holograms that are only in-memory, by not registering it in the HologramManager?

That is true, but then I have to track created hologram myself. With this solution, I can add it as a part of existing hologram manager and retrieve later. If you believe this is not necessary, we can close this PR.

Adding a note here, if you don't register a hologram then when players join after it's been crated, they won't see it


Corrected formatting on the blocks page.

I read through the blocks page and there were a few formatting errors. I decided to fix those before looking in detail at the words/code as any changes I did add to in that regard would get lost in the formatting changes.

I suppose a quick discussion about line breaks might be in order as well. My screen is quite small so viewing the markdown and preview side by side makes the markdown look really janky. I removed the line breaks in this file so that I could read them better in markdown. But I guess it comes down to preference.
What are your thoughts here?

I use pandoc which unfortunately adds line breaks.  I was bothered by them at first but then when I edit the markdown in vscode and other "smart" markdown editors, they seem to ignore it.  I'm totally ok nuking them.


Command failed: git checkout add-devnet-quorum

It appears the latest update broke the install. See my command output below.
Specifically: error: pathspec 'add-devnet-quorum' did not match any file(s) known to git.
ubuntu@fe4eacf6771c:~/mynode$ dashcore-node install @dashevo/insight-api
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142
npm ERR! code 1
npm ERR! Command failed: git checkout add-devnet-quorum
npm ERR! error: pathspec 'add-devnet-quorum' did not match any file(s) known to git.
npm ERR!

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/ubuntu/.npm/_logs/2020-10-26T16_54_46_126Z-debug.log
/home/ubuntu/.nvm/versions/node/v12.19.0/lib/node_modules/@dashevo/dashcore-node/lib/cli/main.js:87
          throw err;
          ^

Error: There was an error installing service: @dashevo/insight-api
    at ChildProcess.<anonymous> (/home/ubuntu/.nvm/versions/node/v12.19.0/lib/node_modules/@dashevo/dashcore-node/lib/scaffold/add.js:56:19)
    at ChildProcess.emit (events.js:314:20)
    at maybeClose (internal/child_process.js:1021:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:286:5)

Looks like this branch is no longer there. https://github.com/dashevo/insight-api/blob/3a938c417225b0c1d03549971b157a2c53ea0fe4/package.json#L63

Yeah, it seems we dropped the branch in the dashcore-lib repo. Please try to use dashcore-lib 0.18.14 in package.json instead of the branch.


human readable format of history file

This is essentially only relevant when hint mode with configurable coordinate sets gets implemented and should stay here  as a reminder until the decision has been taken:
Assuming the above feature gets implemented, the configurable coordinate sets should be saved in some simple human readable format which the history file should use as well. This way, it would also be trivial to build a coordinate set by simply copying (parts of) an appropriate history file.

when hint mode with configurable coordinate sets gets implemented

This is now implemented in the form of --query.

Assuming the above feature gets implemented, the configurable coordinate sets should be saved in some simple human readable format which the history file should use as well.

I don't agree with this. The history file is for warpd's internal use only (akin to vim swap files). The fact that it is not human readable is a feature, not a bug.

This way, it would also be trivial to build a coordinate set by simply copying (parts of) an appropriate history file.

I've introduced a 'print' key which should make it fairly trivial to generate files which can be consumed by --query.


Option to disable stream api checking in Nginx Plus

Add feature to disable stream Nginx Plus API if no stream in Nginx configuration
With the current integration version, we get a lot of errors in Nginx Plus for calling Stream endpoints stats that does not exist if no Stream defined in Nginx configuration.
Now we will have the ability to disable stream checking in the Datadog integration configuration file.
When use_plus_api: True then we need now set also use_plus_api_stream: True to have all covered. This is aligned to values defined for endpoints in the Nginx integration code.
    ## @param use_plus_api_stream - boolean - optional - default: false
    ## If you are using the commercial version of Nginx, for releases R13 and above,
    ## there is a new API to replace the extended status API.
    ## See https://www.nginx.com/blog/nginx-plus-r13-released/
    ## When no Stream servers used we can disable this part to remove errors in Nginx Plus API calls.
    ## To use it set `use_plus_api_stream` to true
    #
    # use_plus_api_stream: true

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

File conf.yaml.example needs to be synced

@hithwen how can I sync config file once again ??

@szibis you need to update to the latest version of ddev

Something wrong these config files changes are in master one hour ago and after sync all files sync shows
ddev validate config -s
Validating default configuration files...
All 230 configuration files are valid!

latest ddev version installed
datadog-checks-dev==3.19.0

Still failing on files with info that files should be synced.

I'm very sorry, yesterday we removed some trailing whitespaces from the base templates so you need to rebase with master before syncing (or try applying the suggestion I've left)


Patch 4

Added a bit of technical details.

@ryanteck can you move this PR to be against the shawaj:patch-4 branch?


Detect renamed rules in configuration resolver

Following recomendation of #5880 from @keradus .
I improved the detection rules suggestion with the following table:
https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/3.0/UPGRADE-v3.md#renamed-rules
From:
In ConfigurationResolver.php line 717:

  The rules contain unknown fixers: "lowercase_constants" (did you mean "lowercase_cast"?).

To:
In ConfigurationResolver.php line 811:

  The rules contain unknown fixers: "lowercase_constants" is a renamed rule,  did you mean "constant_case"? note: use configuration ['case' => 'lower']

  For more info see: https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/3.0/UPGRADE-v3.md#renamed-rules.

Coverage decreased (-0.03%) to 92.277% when pulling 743b810d994afe22081d9fe7c1c49b3563db1717 on shakaran:detect_renamed_rules into e9aa4c8c1d71de791c605cab0f6e358686dab38d on FriendsOfPHP:3.0.

Thank you @shakaran.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-k8s-23-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 906

Questions ?
Please refer to the Backport tool documentation


SDK7: Removing a temporary message display can cause another ui element to move

Background is provided in a comment
Steps to reproduce:

Clone the demo scene at Cloning the scene in https://gitlab.com/dcl-tech/demo-uis
Be sure that the useGeneralizedCompositing variable near the top of DemoUs.tsx is set to true
Run it in preview
If you don’t see the 2D Uis appear automatically, click on the green box to open them
.   // You may have to click a second time, see issue #936
Click the button on Ui_2 to maximize Ui_2 and see a second button.
Click that second button
Immediately, before the Announcement goes away, click the first/original/right button
// Expected: the UI will minimize in place
// Actual: it minimizes, but jumps to the top left corner of the Announcement panel, and might disappear at the end of the announcement, or remain there for the rest of the scene session

This is really ugly, please fix asap.

Background: I needed a way to manage multiple unrelated 2D Uis being added and removed independently.
Why?

Because one can only call ReactEcsRenderer once in a scene,
Because my scenes sometimes need more than one 2D Ui
Because those Uis may be completely unrelated, in their own reusable modules, for example
a. BuilderHUD
b. CinemaControls
c. AdmissionTicket
d. Dance button
e. Stage Manager Controls
This linked scene demonstrates such a mechanism,
but also demonstrates some issues with SDK7 2D UIs, which I’ll report in decentraland/sdk,
including interfering with (cross-talking with) the properties of other UiEntities.


Do not remove previous sid from database

close #718

I think there is a problem here, it's good that we keep the hash so we can always access but we are loosing the information that this hash is linked to a sid.
Maybe that will be done in another PR but we should have an array of hashes for the sid
Something like that (of course with error management ;))
tx.Put([]byte(sidKeyPrefix+s.Sid), []byte(s.Hash), nil)

into
tx.Put([]byte(sidKeyPrefix+s.Sid), append(tx.Get(s.Sid), []byte(s.Hash)), nil)

I think there is a problem here, it's good that we keep the hash so we can always access but we are loosing the information that this hash is linked to a sid.

In that case, we don't lose it because we have it in the service struct, what we lose here is the order of added hashes (but we never had :)) so this will be done in next PR.
Also, I was thinking (we should have it i guess) about adding for example * in command service list to show the last sid.


[stable/jenkins] Add cpu and memory limit for jenkins master

What this PR does / why we need it:

This PR makes it possible to configure cpu and memory limits for jenkins master. In some environments it is not possible to launch jenkins without these values provided.
This PR bumps version to 0.16.4 and appVersion to reflect the current LTS version of Jenkins (see https://jenkins.io/changelog-stable).

PR #6077 does the same, and IMO it is more future proof: if Kubernetes adds a new key to resources, this chart will need another PR to use them while with PR #6077 it would work out of the box.

That's fine for me. Haven't seen the PR before.

Closing this pull request in favour of #6077.


Drop Down menu

Hi guy, thanks a lot fot this beautiful site theme I'm planning to use \o/
Actually this is not really an issue but more a request to extend that theme: could you add drop down menu? :-) I mean, something like this: http://codepen.io/philhoyt/pen/ujHzd
As you say pull requests are welcome, I can try to do it by myself if you tell me which file I should put new code in: I don't feel very comfortable yet with scss.

How many levels of submenu do you want?

Thanks for your reply!
Ha ha, basically the general solution would be « n > 1, with automatic positioning », but actually no, just 1 sublevel will be perfect. If you just give me hint, I can try to do it by myself -- scss is easier to get started with then expected :-)

@piotr-yuxuan I am working on it. But if you want, you can update scss styles in css/styles.scss and add sub menu in _includes/header.html .

Ok, I've got a proposal here: https://piotr-yuxuan.github.io/
It's only a proposal because -- you'll very quickly understand why but I'm not that good in CSS and I believe you could correct that bug easily. I've don't either know how to give an automatic width to sub-items. Last remark, it disappears when mobile mode, but I think it's not that bad -- it would take some time to make a proper mobile-adapted sub-menu.
You can notice I put links and your name in he About section, so we know who thie theme comes from ~

I am adding support for single level submenu in my next commit.

great, thanks!

Heya :-) just a short message to know when you'll commit it. As you can see in my proposal, there is something wrong I haven't managed to fix.

unresolved, moving to another theme.


bazel: Migrate py_proto_library to @com_github_grpc_grpc

py_proto_library provided by @com_google_protobuf//:protobuf.bzl has been deprecated for a while now:

This is provided for backwards compatibility only.  Bazel 5.3 will
introduce support for py_proto_library, which should be used instead.
https://github.com/protocolbuffers/protobuf/blob/32af7d211b85f71920acdfa9b796db008f8c2a45/protobuf.bzl#L642-L644

However, native py_proto_library has never been  provided, see https://github.com/bazelbuild/bazel/issues/3935. Instead @rules_python//python:proto.bzl is recommended. I attempted switching to this library, but it's not compatible with @com_google_googleapis's py_proto_library targets, see https://github.com/cncf/xds/pull/69. I found a hacky workaround by using cc_proto_library to generate python targets, but downstream integration into Envoy failed (https://github.com/envoyproxy/envoy/pull/30159).
This PR migrates py_proto_library implementation to to @com_github_grpc_grpc. This implementation is used by  @com_google_googleapis's, and, more importantly, uses bazel aspects. Which decouples cncf/and xds's dependencies from concrete upstream py_proto_library implementations.
This resulted in a significant code improvement of bazel/api_build_system.bzl:

No more custom @com_google_googleapis dependency mapping needed via py_proto_library rules override.
No more hardcoded dependencies _xds_py_proto_library - proto dependency tree is determined from proto_library definitions via Basel aspects.
No more EXTERNAL_PROTO_PY_BAZEL_DEP_MAP dependecy map needed - for the same reason.

Similar work in Envoy: https://github.com/envoyproxy/envoy/pull/30834.

@adisuissa @kyessenov this is ready for review.
@htuch - FYI this addresses a bunch of TODO's assigned to you.

Overall I like this approach a lot. @phlax @moderation are there any concerns here about taking the dependency from https://github.com/grpc/grpc? It's already a build dependency, and these are Bazel rules, so we should still be able to compile Envoy without linking Google gRPC, but just wanted to check.

overall looks fine, and appears address a number of issues - not sure iiuc re grpc dep, but from what i do seems ok

@adisuissa yes - I have a PR in progress: https://github.com/envoyproxy/envoy/pull/30834.
Was out yesterday, planning to wrap it up today.

LGTM

Looks like to make it work nicely in Envoy, we'd have to merge this first. Otherwise, bazel complains about missing _genproto targets (generated by @com_google_protobuf//:protobuf.bzl's py_proto_library), because this cncf/xds used rules_override for googleapis before this PR.
ERROR: /private/var/tmp/_bazel_sergiitk/e60ee3f6402a793527d720007beb00cb/external/com_github_cncf_xds/xds/annotations/v3/BUILD:5:18: no such target '@com_google_googleapis//google/api:httpbody_py_proto_genproto': target 'httpbody_py_proto_genproto' not declared in package 'google/api' defined by /private/var/tmp/_bazel_sergiitk/e60ee3f6402a793527d720007beb00cb/external/com_google_googleapis/google/api/BUILD.bazel (Tip: use `query "@com_google_googleapis//google/api:*"` to see all the targets in that package) and referenced by '@com_github_cncf_xds//xds/annotations/v3:pkg_py_proto_genproto'

It might be possible to solve this in Envoy without merging this, but it'd be a throw-away work. Let's get this one in first.


linux:add the watchdog support for linux image

TI has removed the device node from k3-am65-mcu.dtsi
we are lacking of watchdog support,so we need adding the function back
the device node refer to ti sdk 06_03_00
Signed-off-by: Chao Zeng<EMAIL_ADDRESS>

There is https://github.com/torvalds/linux/commit/6674a90bb141496f56ce840c22f2f2ddccc5d397 - not compatible with the TI kernel?
Also, did you check if the downstream wdt driver is up to speed with the mainline driver? There were several changes in upstream in order to make the essential pick-up of an already running watchdog (started by U-Boot) work.

And TI_SCI_PD_EXCLUSIVE breaks over https://github.com/siemens/k3-rti-wdt.

There is torvalds/linux@6674a90 - not compatible with the TI kernel?
Also, did you check if the downstream wdt driver is up to speed with the mainline driver? There were several changes in upstream in order to make the essential pick-up of an already running watchdog (started by U-Boot) work.

for the kernel 4.19.94, there is no this dts node.
downstream wdt driver has fall behind serval commit with upstream wdt driver.
is it necessary to pick up upstream wdt driver to the currently transitional kernel?

Depends on what purpose the watchdog should fulfill. But if it shall serve e.g. #115, a clear yes.

Depends on what purpose the watchdog should fulfill. But if it shall serve e.g. #115, a clear yes.
In general: What is in upstream has settled. What is in the downstream kernel is still under development or even experimental. Even worse, if updating of a downstream branch was stopped.

for transitional kernel, as lacking of the wdt fucntion,just to add the fucntion back and keep the same with Released image.
so for this function, I think there is no need to pick up the difference from upstream.

by the way, for SR2 soc, TI has solved the silicon bug for watchdog.
if so, does it mean it is not necessary rti-wdt firmware on r5f core to support watchdog?

OK if this is just about restoring the (limitedly useful) support in the first release as-is. The transition kernel will not live very long, and the next kernel will be upstream based, code-wise aligned.
Regarding SR2 WDT: I strongly assume that we no longer need that firmware on the R5 core, but I didn't have the time yet to validate this. U-Boot will likely need some runtime detection, to avoid loading the firmware when that's not needed.

OK if this is just about restoring the (limitedly useful) support in the first release as-is. The transition kernel will not live very long, and the next kernel will be upstream based, code-wise aligned.
Regarding SR2 WDT: I strongly assume that we no longer need that firmware on the R5 core, but I didn't have the time yet to validate this. U-Boot will likely need some runtime detection, to avoid loading the firmware when that's not needed.

As far as I know, WDT function has not ready for SR2,so currently we still need R5F core firmware until TI has completed the hardware WDT fucntion

Echo to Jan, we have the watchdog enabled in upstream u-boot, does this conflict with that one, any test performed on the sysfw 3.0 firmware?

We do not start the watchdog in U-Boot by default, but we prepare that our users could do that. In that case, the kernel must be ready for it. That's not yet true, at least due to the power settings. In any case, it should be tested.

We do not start the watchdog in U-Boot by default, but we prepare that our users could do that. In that case, the kernel must be ready for it. That's not yet true, at least due to the power settings. In any case, it should be tested.

for sr2 kernel, TI has removed the process loading the  rti_dwwdtest firmware to the R5F core on the kernel side
can we started the watchdog under uboot and kernel just follow it?
if not, we need to  pick up  the rti_dwwdtest firmware loading process on the kernel side.

rti_dwwdtest was a test application. https://github.com/siemens/k3-rti-wdt is what we use today (maybe time to clean up the image). If that is loaded there, Linux cannot load its own thing anymore.

if we like to load the  rti_dwwdtest on the uboot, so it not necessary to load the application on the linux?
so in order to implement the watchdog function on the PG2, we need to start the watchdog on the uboot firstly?
Am I right?

Again, forget about rti_dwwdtest - that was never intended to be used for what we used it. And was missing a lot of protection needed for a real watchdog reset mechanism.
If the RTI watchdog does not trigger a hardware reset without R5 help (and I still need to check that on PG2), yes, we need to load that l4-rti-wdt binary via rproc. That has to happen before the watchdog is started. If U-Boot starts it (required for SWUpdate scenarios), it will happen there. If only Linux starts the watchdog, the firmware must be loaded via Linux' rproc means. If Linux tries to load it while the watchdog is already running, it will get an error (firmwalls are up, blocking the R5 access).

Then if R5 WDG is started in u-boot, kernel will never able to use that watchdog in linux?

No, why should that be the case? The watchdog register interface is not affected by the R5 code working around the missing reset wiring.
All what fails (with a warning) is loading R5 code. But that is not done by the Linux watchdog driver anyway - in contrast to the U-Boot watchdog driver with something like<EMAIL_ADDRESS>

Sure, we will test the kernel watchdog with u-boot watchdog loaded.
Meanwhile, I will merge this PR cause this one is urgent for next image release, also it will not make things more worse.
Then we need continue the discussion of the long term solutions.

The patch is not correct as I explained.


Can't connect to DB during script in .yml

Hello again,
My build is passing but another error is appearing in the test stage.
I'm using this in .gitlab-ci.yml
variables:
  MYSQL_ROOT_PASSWORD: secret
  MYSQL_DATABASE: testing
  MYSQL_USER: user
  MYSQL_PASSWORD: secret

test:
  stage: test
  services:
    - mysql:5.7

  image: chilio/laravel-dusk-ci:stable
  script:
    - cp .env.dusk.local .env
    - cd ${CI_PROJECT_DIR}
    - chmod -R 775 storage
    - chmod 775 bootstrap/cache
    - chown -R www-data ./
    - php artisan key:generate
    - php artisan migrate:fresh
    - php artisan db:seed -n
    - chrome-system-check
    - sleep 10s
    - start-nginx-ci-project
    - php artisan dusk --colors --debug

But during the CI, the php artisan migrate:fresh is faliing with this error
$ php artisan migrate:fresh

In Connection.php line 664:

  SQLSTATE[HY000] [2002] Connection refused (SQL: SHOW FULL TABLES WHERE tabl
  e_type = 'BASE TABLE')

In Connector.php line 68:

  SQLSTATE[HY000] [2002] Connection refused

But if I connect manually to the docker container. And I run manually the command, everything is working fine concerning the migration & seeding.
Have you any idea why it is not working?
More info
When I connnect to my docker container, I'm also able to run mysql like this
musql -u user -p -h mysql
My database testing exist

cd project dir should be before cp. env....

I changed it but no difference.
My .env was present in both situation.
Why is it working manually but not within the script? Do you have any idea how can I debug this?

@Raccoon5031 what do you mean by running manually?
Do you have properly set port for mysql connection (env and db config) ?

By manually I mean
In my .gitlab-ci.yml I added a ``sleep` like this
script:
    - cp .env.dusk.local .env
    - cd ${CI_PROJECT_DIR}
    - chmod -R 775 storage
    - chmod 775 bootstrap/cache
    - chown -R www-data ./
    - sleep 1h

So the container is not destroyed if I run my CI.
And then I connect to the container by using this (on my machine running docker)
 docker exec -it [container_id] /bin/bash
And then I cd into my laravel project (/build/project/my_project)
And If I type by myself the following, it is working
php artisan key:generate
php artisan migrate:fresh
php artisan db:seed -n

The correct database is selected, the tables created and filled by my seeds. No errors.
But the script section is doing exactly the same but is failing. Maybe I should put a timer? (Like you did after chrome-system-check)

@Raccoon5031 ok good, you are right, it is strange. Unless ....
manually you might be calling some different db  like for example the one configured in .env and not the one defined in .env.dusk.local
Try to add to variables:
DB_HOST: mysql
DB_CONNECTION: mysql

BTW Did you try with timer?
Which runner executor did you choose while registering runner BTW?

@chilio In the script in copy the .env.dusk.local into .env
I tried to add the variables and timer after the migration & seeds but this is failing like before.
I also tried this in the script section
    ...
    - php artisan key:generate
    - cat .env
    - sleep 5h

And the result contain the following information
DB_CONNECTION=mysql
DB_HOST=mysql
DB_PORT=3306
DB_DATABASE=testing
DB_USERNAME=user
DB_PASSWORD=secret

So everything looks fine. But the job is still failing.
Which runner executor did you choose while registering runner BTW?
--> I don't really understand the question but I guess the answer is docker
I don't know if it is useful but ... my runner config looks like this
[[runners]]
  name = "my-web-runner"
  url = "http://gitlab.local/"    #This is a bare-metal machine, accessible from internal network
  token = "0axxxxxxxxxxxxxxxxxxxxad"
  executor = "docker"
  [runners.docker]
    tls_verify = false
    image = "chilio/laravel-dusk-ci:stable"
    privileged = true
    disable_cache = false
    volumes = ["/cache"]
    shm_size = 0
  [runners.cache]

@Raccoon5031 just to double check I would disable cache in runner and  also try calling service in this way:
services:
    - name: mysql:5.7
      alias: mysql-test

and then in variables section:
variables:
  ...
  DB_HOST: mysql-test

To make sure there is no problem with mysql name...
Have you tried to cache config? If so php artisan config:clear should help in script.
Finally I would check if there are any custom modifications in config/database.php which could break the flow for example regarding APP_ENV

If I add the alias and change the variable shouldI also change the ´DB_HOST´ of my .env from mysql to mysql-test?
Here is my config/database.php, nothing really special here
'mysql' => [
            'driver' => 'mysql',
            'host' => env('DB_HOST', '<IP_ADDRESS>'),
            'port' => env('DB_PORT', '3306'),
            'database' => env('DB_DATABASE', 'forge'),
            'username' => env('DB_USERNAME', 'forge'),
            'password' => env('DB_PASSWORD', ''),
            'unix_socket' => env('DB_SOCKET', ''),
            'charset' => 'utf8mb4',
            'collation' => 'utf8mb4_unicode_ci',
            'prefix' => '',
            'strict' => true,
            'modes' => [
                'STRICT_TRANS_TABLES',
                'NO_ZERO_IN_DATE',
                'NO_ZERO_DATE',
                'ERROR_FOR_DIVISION_BY_ZERO',
                'NO_AUTO_CREATE_USER',
                'NO_ENGINE_SUBSTITUTION'
            ],
            'engine' => "InnoDB",
        ],

@Raccoon5031 mystic things are happening here...
Could you post your complete .gitlab-ci.yml?
What versions of gitlab, runner, docker, host os are you running?

I removed the MySQL service from the build part
stages:
  - build
  - test

variables:
  MYSQL_ROOT_PASSWORD: secret
  MYSQL_DATABASE: testing
  MYSQL_USER: user
  MYSQL_PASSWORD: secret
  DB_HOST: mysql
  DB_CONNECTION: mysql

build:
  stage: build

  image: chilio/laravel-dusk-ci:stable
  script:
    - composer install --prefer-dist --no-ansi --no-interaction --no-progress --no-scripts
    - npm install
    - npm run dev
  cache:
    key: ${CI_BUILD_REF_NAME}
    paths:
      - vendor
      - node_modules

test:
  stage: test
  services:
    - mysql:5.7

  image: chilio/laravel-dusk-ci:stable
  script:
    - cd ${CI_PROJECT_DIR}
    - cp .env.dusk.local .env
    - composer update
    - chmod -R 775 storage
    - chmod 775 bootstrap/cache
    - chown -R www-data ./
    - php artisan key:generate
    - php artisan config:clear
    - php artisan migrate:fresh
    - php artisan db:seed -n
    - chrome-system-check
    - sleep 10s
    - start-nginx-ci-project
    - sleep 4h
    - php artisan dusk --colors --debug
  cache:
    key: ${CI_BUILD_REF_NAME}
    paths:
      - vendor
      - node_modules
    policy: pull

And now the migration and seeding are fine. That's strange but my DB is not necesarry for the builing part so it's not a big deal for me.
I tried to retry everything from a fresh new machine (re-install docker and so on) but I still have the same problem.
But I'll close this issue anyway, thanks again for your time and patience!


Running several adapters on the same input

Feature request
I'd like to be able to run several adapters on the same input.
For example, being able to call the python client function with several adapters instead of a single one and retrieve the answers of all the adapters at the same time.
Motivation
There are many cases where we want to run several distinct adapters on a user input. Being able to list all the adapters in a single function call would save on sending the user input several times, and we may be able to share some computing across adapters because we know the input is the same.

I'd like to +1 this use case, which is particularly useful when each adapter is actually a classifier and we want each to run over the same input.

+1


sync idl file from server

sync idl file from server

Coverage increased (+0.05%) to 65.509% when pulling f0784f528de4db852e2cdc31b3eae584b81a477a on yiminc:sync_idl_file into c93ab4f261dfbe7c95d8f705565e3b8c1f2a893a on uber-go:master.


RichTextLabel spacing issue on small resolutions

Godot version:
v3.1.beta2.official
OS/device including version:
Windows 10 pro x64
Issue description:
This weird spacing skip or something only happens when the game is running, on editor it looks just fine (like on the pic).
Also the project works fine on Godot 3.0, since I'm testing it on 3.1.
I also tweaked with all possible options for both the RichTextLabel and the Dynamic font but no fix.

Steps to reproduce:
Any RichTextLabel with a Dynamic font of size 16 using the file bellow.
pixel_text.zip
Game resolution: 224x126
Stretch:

Mode: Viewport
Aspect: Ignore

I have the same problem and also noticed that, if "Extra Spacing / Char" is increased, spacing is not respected after formatted text:

Te bbcode used in the screenshot is:
Has conseguido el [color=red]Palo Legendario[/color]!! Legendario!!

Label seems to work fine, at least in my case:

The first is a RichTextLabel, de second a simple Label. They both are using the same DynamicFont.
It seems like RichTextLabel is ignoring the width of the chars in the ttf file, removing the empty spaces, and only taking into account the "drawn area" of them.
As Wagnerfs said it seems to work properly in the editor, but not when running the game.
Changing from GLES2 to GLES3 does not affect at all.

Demo Project: Test.zip

I'll close this issue as a duplicate in favor of https://github.com/godotengine/godot/issues/24570. Thanks for confirming it :slightly_smiling_face:

Ups, my last comment in the other issue was supposed to be posted here… I've edited my comments in that issue to link the demo Project attached to this one!
Thank you Calinou, and sorry for the mess.


Handle catastrophic JSoup errors

Add the Internet permission at the library level since any user of the library will need it
Add the Google maven repository so that the Android testing libraries can be pulled down
Bump the minSdkVersion so that the Google testing library can be used
Add the Android Testing library and JUnit
Refactor some methods in TextCrawler to facilitate testing
Add a test for TextCrawler to replicate what occurs when JSoup has a catastrophic error

@jdonmoyer Thanks.


dyld: Library not loaded

When I run the example code, I got this error below.
Any idea?

dyld: Library not loaded: /usr/local/BerkeleyDB.4.8/lib/libdb-4.8.dylib
Referenced from: /users/gracelee/documents/metamap/public_mm/bin/metamap14.BINARY.Darwin
Reason: image not found
/users/gracelee/documents/metamap/public_mm/bin/metamap2014.TEMPLATE: line 146: 95257 Abort trap: 6           $COMMAND

I have not seen that error before. It looks like it may be a problem with your metamap installation. Can you run metamap via command line? Also, make sure you have started the SKR/Medpost Part-of-Speech Tagger Server and the Word Sense Disambiguation (WSD) Server.

When I try an example of MetaMap, it gives the same error.
For the last few hours I googled for the solution but still I haven't solved it.
Btw thank you for the prompt response.

If there is an issue with your metamap installation, then I probably won't
be much help. Are you using a mac or linux machine?
Anthony
On Thu, Oct 19, 2017 at 10:11 AM, GL<EMAIL_ADDRESS>wrote:

When I try an example of MetaMap, it gives the same error.
For the last few hours I googled for the solution but still I haven't
solved it.
Btw thank you for the prompt response.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/AnthonyMRios/pymetamap/issues/21#issuecomment-337920510,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA7Vsu19egRH15HLVbcyojIN6qo212fvks5st1imgaJpZM4P_Ks7
.

Hi Anthony
I am using Mac Sierra. I will fix MetaMap problem first and then come back to use the pymetamap!
I am closing the issue.
-G


feat(tests): verify routing to every backend in e2e tests

What this PR does / why we need it:
This PR proposes enhanced approach for testing KIC e2e tests

Kind clusters have configuration --max-endpoints-per-slice 2 to make possible to test KIC against Services with multiple EndpointSlices (introduced in #4091)
basic Ingress deployment and its verification is enhanced to ensure that traffic is routed to every backend by Kong gateway (IPs are in many multiple EndpointSlices)

Rationale - the proper discovery of IP addresses of all backends and setting routings in Kong gateway are the core feature of KIC, thus it should be covered with tests that verify it. Having K8s Kind cluster with such config and enhanced tests allows ensure that KIC with Kong gateway do it properly.

Which issue this PR fixes:
closes #4026, it's the last from the series of PRs #4088, #4091 - all of them surpass to big initial PR https://github.com/Kong/kubernetes-ingress-controller/pull/4084 which presented the approach

Special notes for your reviewer:

I've tested it locally in the loop for the test TestDeployAllInOneDBLESSLegacy$ and I haven't observed flakiness
Measured time of this test (locally) after the change is around 100s the same as before
I run e2e tests for this PR by adding label ci/run-e2e
Maybe there are other places with should be tested in terms of dealing with multiple EndpointSlices? Let's create separate issues for them.

PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[ ] the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

e2e tests failure for family TestDeployAndUpgradeAllInOnePostgres - is a known issue https://github.com/Kong/kubernetes-ingress-controller/issues/4097
so it's good to go 🟢