ECL merge "Miscellaneous optical flow use improvements" and update Firmware master

From https://github.com/PX4/Firmware/pull/9613 the ECL submodule is currently pointing at the ECL branch pr-ekfOptFlowFixes (https://github.com/PX4/ecl/pull/452). This needs to be merged to PX4/ECL master and updated PX4/Firmware side before the v1.8.0 release.

https://github.com/PX4/Firmware/pull/9628


Improve presentation of forms

Forms looking ugly right now, should be better. AC to be provided.
Forms:

[ ] Team (add and edit)
[ ] Profile (edit)

Don't forget that we're missing labels for form inputs


Fix : moving instance_name logging to log entry label

#1994  removing unsupported resource labels for standard, flex
Adding instance_name to log entries instead of resource for Flex.
@meltsufin

Changes Unknown when pulling 7d2d3ef3e449ac40a28bb1f925e5a23f3ea755a0 on jabubake:flex_logging_fix into ** on GoogleCloudPlatform:master**.

@gregw PTAL

@meltsufin I've build @jabubake's branch locally and run the https://github.com/GoogleCloudPlatform/jetty-runtime/pull/81 against it. The exception is gone and I can see the correct logs in the console - however the remote tests are failing as the logs are not being found by the API calls the test has made, so perhaps the filter is now wrong against the labels used.
tl;dr; It's fixed the #1994 issue but I need to check if other labels are missing.

Greg, Mike,
The trace id label is now just trace_id ,  I noticed the naming was done
more in the format of logging  v1 labels, so I moved them to v2 style with
URL not being in the label name. I also wanted to set it standardized to
not be tied to app engine in the URL.
Let me know if this is an issue
On Thu, Apr 27, 2017 at 9:22 AM Greg Wilkins<EMAIL_ADDRESS>wrote:

@meltsufin https://github.com/meltsufin I've build @jabubake
https://github.com/jabubake's branch locally and run the
GoogleCloudPlatform/jetty-runtime#81
https://github.com/GoogleCloudPlatform/jetty-runtime/pull/81 against
it. The exception is gone and I can see the correct logs in the console -
however the remote tests are failing as the logs are not being found by the
API calls the test has made, so perhaps the filter is now wrong against the
labels used.
tl;dr; It's fixed the #1994
https://github.com/GoogleCloudPlatform/google-cloud-java/issues/1994
issue but I need to check if other labels are missing.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-java/pull/2011#issuecomment-297766501,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWfpWyBNbKAVTji4Twy8W-iOQvEqWOhfks5r0MDggaJpZM4NJi-k
.

@jabubake Ah thanks.  Updating my tests and redeploying now...

@jabubake @meltsufin the branch works fine with my tests now that I have updated the traceId and instance_name checks.
However, I do not know if the shorter name will prevent the log console from matching the nginx log entries (which still use the long name) with the gae_log entries.  The matching is currently not working for other reasons so I cannot test.

@meltsufin , @gregw  : given the complexity, I'll update this for now to be in sync with what used to exist. We can take the rename to shorter v2 type versions in a separate CL.

@jabubake Sounds good, thanks!

@gregw @meltsufin  : let me know if this will do

Changes Unknown when pulling 51131e4d09c72bc8d7c10b0c1d32079b6387a8c8 on jabubake:flex_logging_fix into ** on GoogleCloudPlatform:master**.

@garrettjonesgoogle  : the changes address the bug fix to resource labels and also reverting to the  currently supported Stackdriver API name for trace_id. ptal.

@vam-google I added some comments - could you finish the review?

Changes Unknown when pulling 33bccbd7424ae27c3d3868d84d7ed494fe6e9d62 on jabubake:flex_logging_fix into ** on GoogleCloudPlatform:master**.

@vam-google ping :)

Changes Unknown when pulling 8936d807776c455ac19f005f23267f839b07ae6f on jabubake:flex_logging_fix into ** on GoogleCloudPlatform:master**.

Changes Unknown when pulling bf7b47b924833bef970aa33b86300dfa5c3fb860 on jabubake:flex_logging_fix into ** on GoogleCloudPlatform:master**.

Changes Unknown when pulling c2fc97a57d9a134dfdb6abe7af226d0d13c80d9c on jabubake:flex_logging_fix into ** on GoogleCloudPlatform:master**.


delete can't find the existing key

Have been testing the SMT delete operation and encountered unexpected behavior in the following case:
import { LocalStorageDB, Merkletree, str2Bytes } from "@iden3/js-merkletree";

import "mock-local-storage";

it("should remove elements from Merkle Tree correctly", async () => {
      const storage = new LocalStorageDB(str2Bytes(""));
      const localMerkleTree = new Merkletree(storage, true, 20);

      const keys = [7n, 1n, 5n];

      for (let key of keys) {
        await localMerkleTree.add(BigInt(key), key);
      }

      for (let key of keys) {
        await localMerkleTree.delete(BigInt(key));
        await localMerkleTree.add(BigInt(key), BigInt(key));
      }
});

The following code throws the error:
1) SparseMerkleTree
       Uint SMT
         should remove elements from Merkle Tree correctly:
     Error: the string "Key not found in the MerkleTree" was thrown, throw an Error :)

While the expected result is to succeed because I am just deleting and adding the same elements in place.

Package version: "@iden3/js-merkletree": "^1.2.0"
Node version: v18.18.0

Thanks for reporting the issue. We were able to reproduce this problem. It will take some time to fix it.

Hi @Arvolear. The latest commit to the main branch should fix the problem. Could you please check? Feel free to reopen the issue.

Will do. Thanks. If I encounter any divergence between this Solidity implementation and the one in iden3, will let you know :)


"/" baseurl changes to "" when deployed to GitHub Pages

Config used was baseurl: "/" and works fine locally. However, when deployed to GitHub Pages, the baseurl is changed to "" and href for navigation and footer will not have the "/" prepended.
With some research I stumbled on this issue https://github.com/github/pages-gem/issues/350 and a possible solution to this can be found here
Would like your thoughts on this solution as I've only tested on a user site.

Hi
I just tried deploying to Github Pages on a new site. It seems to work fine.
What did you set as your url in _config.yml?

Hi there,
when i set the url as "https://nicholaschuayunzhi.github.io" and base url "/" it works fine locally but when deployed to ghpages,  the href of my links don't have "/" prepended.

You can leave the base url as "". It should work on local as well as on Github Pages. Try it and let me know.
Thanks

Hi there!
I tried leaving baseurl as "" and it does not work locally. I've also tried using the "/" as suggested and works locally but does not work when deployed to Github Pages.
I've updated my repository. You can refer to the config file there. The site has also been deployed to gh pages as a user site.
The error arises when i try to use the side bar from my about page. https://nicholaschuayunzhi.github.io/about/
When i click about on the side bar it redirects me to https://nicholaschuayunzhi.github.io/about/about

How are you deploying the site to Github Pages?
I just tried deploying using travis as well as pushing contents of site-pages to the gh-pages branch of the repository and it worked fine.

I'm not too sure what you're asking here sorry - I'm a bit new to this. I'm building from master branch.
I read in the readme that i'm supposed to push the _site folder? but its under the gitignore. Am i doing something wrongly?

No worries.
If you installed the theme using method 1, you need to push the contents of the destination folder (mentioned in _config.yml. eg - destination: ../hanuman-pages/) to the gh-pages branch(if project page) or master branch(if user page) of your repository.
If you installed the theme using Ruby Gem method, you need to push the contents of _site folder.
I will make the instructions more clear.

Hmm, i tried the first method but i still get the same error.
Here are the steps i did:

$ git clone https://github.com/samanyougarg/hanuman.git
updated config.yml gist
push the destination folder to my repo (/hanuman-pages/) https://github.com/nicholaschuayunzhi/nicholaschuayunzhi.github.io

I still get the same error mentioned in https://github.com/samanyougarg/hanuman/issues/3#issuecomment-352199771

Would like to add the error i made before was:

$ git clone https://github.com/samanyougarg/hanuman.git
updated config.yml gist
$ bundle exec jekyll build
push the cloned repo into my repo


[BUG] An exception occurred while running bloxstrap

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
Bloxstrap.Exceptions.HttpResponseException: Could not connect to https://clientsettingscdn.roblox.com/v2/client-version/WindowsPlayer/channel/ZLIVE because it returned HTTP 401 (Unauthorized)
Bloxstrap_20231102T092329Z.log

just read wiki, please
https://github.com/pizzaboxer/bloxstrap/wiki/'HTTP-401-(Unauthorized)'-when-trying-to-launch-Roblox


Add pre release process

Oops, the types-check are not working, I approved before seeing this

checked with @bidoubiwa we merge this PR despite the type-check failing


Support GHC 9.12

[__5] rejecting: base-<IP_ADDRESS>/installed-cc73 (conflict: scientific => base>=4.15 && <4.21)

I was able to get it to compile and pass all tests with the diff:
diff --git a/cabal.project b/cabal.project
index e6fdbad..c97a946 100644
--- a/cabal.project
+++ b/cabal.project
@@ -1 +1,6 @@
 packages: .
+
+allow-newer:
+  , integer-logarithms:base
+  , hashable:base
+  , splitmix:base
diff --git a/scientific.cabal b/scientific.cabal
index aaabe13..989dd43 100644
--- a/scientific.cabal
+++ b/scientific.cabal
@@ -104,7 +104,7 @@ library
       , bytestring-builder  >=0.10.4 && <0.11

   else
-    build-depends: bytestring >=0.10.4 && <0.12
+    build-depends: bytestring >=0.10.4 && <0.13

   if impl(ghc >=9.0)
     build-depends: base >=4.15


python requirements

Hi!
I'm running Ubuntu 22.04 and having a hart time to fullfill all the python requirements defined in requirement.txt
Some modules doesn't seem to be available in the required version for Ubuntu 22.04
What distribution do you recommend to fullfill all the requirements?
Kamil

The requirements are not set true, there is many garbage requirement i will send a pr for this.


How to mark parameters as trainable or not?

Greetings!
I got custom Layers in equinox that look approximately like this.
class ProductLayer(InnerLayer):

    child_layers: List[Union[SumLayer, InputLayer]]
    edges: BCOO

class SumLayer(InnerLayer):

    log_weights: List[BCOO]
    child_layers: Union[List[[ProductLayer]], List[InputLayer]]

class ContinuousLayerWithFiniteSupport(ContinuousLayer, ABC):
    interval: jax.Array

I now want to exclude ProductLayer.edges from the parameters of a model since they cannot be adjusted by gradient descent.
Fruthermore, SumLayer.log_weights.indices can also not be adjusted. The ContinuousLayerWithFiniteSupport.interval can also not be adjusted using gradient descent. How can i best filter these out for the eqx.partition method?

See the FAQ: https://docs.kidger.site/equinox/faq/#how-to-mark-arrays-as-non-trainable-like-pytorchs-buffers

There is a risk to the suggested approach that should at least be highlighted in the docs: the parameters may still be punished by regularization.
import equinox as eqx
import jax
import jax.numpy as jnp
from jaxtyping import Array
from optax import adamw

class Model(eqx.Module):
    buffer: Array
    param: Array

    def __call__(self, x):
        return self.param * x + jax.lax.stop_gradient(self.buffer)

@eqx.filter_value_and_grad
def loss(model, x):
    return model(x)

model = Model(jnp.ones(()), jnp.ones(()))
loss, grad = loss(model, 2)
optimizer = adamw(1e-1)  # Optimizer with regularization
opt_state = optimizer.init(eqx.filter(model, eqx.is_inexact_array))
updates, opt_state = optimizer.update(grad, opt_state, eqx.filter(model, eqx.is_array))
model = eqx.apply_updates(model, updates)
assert model.buffer == jnp.ones(())  # Fails!

Unless I am missing a downside, the approach I think should be recommended is to use a wrapper class (NonTrainable) to wrap non-trainable nodes, and partitioning parameters e.g.  with:
params, static = eqx.partition(
        model,
        eqx.is_inexact_array,
        is_leaf=lambda leaf: isinstance(leaf, NonTrainable),
    )

Ah! That really isn't very good, you're right.
Hmm, I'm trying to figure out if there's a way to handle this ergonomically. The best I can come up with is to wrap the Optax calls (like we already do for eqx.apply_updates) with something that respects such a Nontrainable wrapper. This is just such an easy footgun!

FWIW I've landed on the optax wrapper approach. I have a trainable/non_trainable mask that I create early on and partition that way. I don't even bother with stop_grad most of the time and pray that XLA does the DCE for me (it seems to).
For things that are really constants (e.g. rotary embeddings) I just materialize those in the kernel with ensure_compile_time_eval

Ah, nice! Okay, I think I'm convinced.
I'd be happy to take a PR implementing this, then.


How can we send mmplayerView data to another Screen Using Push

Hey @MillmanY , when I pushing the view controller to another view controller, how can I pass the mmplayerView values to another controller so that same video will play to detail Screen. You have provide the example of present. How can we achieve using push or Segue

Before you call  self.navigationController.push......
1.set
self.mmPlayerTransition.push.pass { (_)in} and implement protocol
extension ViewController: MMPlayerFromProtocol {
var passPlayer: MMPlayerLayer {
get {
return self.playLayer
}
}
func transitionWillStart() {

}

func transitionCompleted() {
}
func backReplaceSuperView(original: UIView?) -> UIView? {
    self.view.layoutIfNeeded()
    currentCell?.imageView.isHidden = true
    return currentCell?.imageView
}

}
2. Second Viewcontroller
extension SecondViewController: MMPLayerToProtocol {
var containerView: UIView {
get{
return self.playerBaseView
}
}
func transitionCompleted(player: MMPlayerLayer) {
    if let p = player as? ETPlayerLayer {
        defaultPlayer = p
        defaultPlayer.player?.isMuted = false
        self.setDefaultPlayer()
        defaultPlayer.showCover(isShow: false)
    }
}

}

let me implement your code

I have one question how can I get the currentCell. Can you tell me about this?

In my case when I play data I remeber the location of index , and currentcell like this in my Viewcontroller
fileprivate var currentCell: BaseTVCell? {
        get {
          let index = self.presenter.playIndex
          if let cell = self.collectionView.cellForItem(at: IndexPath(row: index, section:0)) {
              return cell
          }
          return nil
    }


Disaster recovery could derive wrong sequence number

In prospect of supporting disaster recovery from locations with application-deleted events, local sequence number recovery from the number of recovered events alone is not sufficient. The corresponding entry in the version vector of the recovered log must be considered as well.

Will be replaced by a ticket that covers a complete re-implementation of disaster recovery.


Fix current ctx._matchedRoute

Fixes #516

@mdenisov this repo is abandoned. Maybe you'd be interested to contribute to the active repo instead: https://github.com/koajs/router/issues/34#issuecomment-651765289

@falkenhawk Yes of course


Apexcharts-card causing entire web browser to hand due to high resource utilization

Checklist

[x] I updated the card to the latest version available
[x] I cleared the cache of my browser
[x] I verified that I'm really running the lastest version in my browser console
[x] I checked if there is another issue opened with the same problem
https://github.com/RomRider/apexcharts-card/issues/210 <---[Closed with no solution]

Describe the bug
When I make apexcharts-cards using my HA sensors, it causes the entire web browser to become unresponsive; even on a relatively fast PC with lots of free resources.
NOTE:  This issue doesn't happen with other graph cards such as history-graph, etc.
Things I already tried
-Disabled web browser extensions/addons; including ad-blocking
-Changed update_interval to >1 minute
-Tried different web browsers on PC
Version of the card
Version: 2.0.4
To Reproduce
This is the configuration I used:
type: custom:apexcharts-card
graph_span: 1d
update_delay: 5s
update_interval: 1min
header:
  show: true
  title: ApexCharts-Card
  show_states: true
  colorize_states: true
all_series_config:
  stroke_width: 2
series:
  - entity: sensor.total_cpu_usage
  - entity: sensor.cpu_package_temperature
    type: column

Screenshots

NOTE:  Screenshot has a couple of extensions enabled.  However, disabling all of them still has the same result.
Expected behavior
Resource utilization should be comparable to other graph cards like history-graph, etc using the same sensors.
Desktop (please complete the following information):

Browser [Chrome, MS Edge]
Version [latest]

the same problem,+ incredible freezes while editing dashboard with apexcharts

I'm beginning to think this card was never meant to be used by sensors that update relatively frequently; regardless of specified update interval in the card.  My sensors update once every 6 seconds... which is completely fine for all other HA graph cards; just not Apexcharts-card.

I'm beginning to think this card was never meant to be used by sensors that update relatively frequently; regardless of specified update interval in the card. My sensors update once every 6 seconds... which is completely fine for all other HA graph cards; just not Apexcharts-card.

I set the refresh rate to 30 s per sensor, but this did not give the desired result, I even reduced the number of sensors with data to 10 per page, this also did not help. Until I have no other options

Are you sure the other chart graphs are not simply using statistics instead of raw data? Can you see a data point every x sec on the other cards?
You can enable statistics in the apex-charts graphs manually.

Are you sure the other chart graphs are not simply using statistics instead of raw data? Can you see a data point every x sec on the other cards? You can enable statistics in the apex-charts card manually.

yes, simple graph cards work without any freezes, but their visualization is not enough for me, it is worse than apex. I also understand that this is a bug, since my macbook pro freezes for a minute or two when (editing page) working with apex and starts to make a lot of noise. To temporarily transfer the apex to a simple graph card, I had to use a powerful PC on Windows))) it's funny to talk about it, but these are features, apparently, of a home assistant in such situations

I will add that all sensors leading to freezing work through esphom and through esp32:  mhz19, pmsx003, sgp30, bme280, etc
but there are aqara sensors that work on gw2 in apex work without friezes

Okay, good to know. I was just checking because I too am using apexcharts-card for my visualization and think that it freezes the system.
As soon as amount of data increases, the system displaying the card becomes extremely slow.
But it seems to be an issue on the displaying system, not the server.
It reminds me of applications that work with uncompressed data. They decompress the input data, hence need lota of memory, whereas other programs work with the compressed input data and hence need less memory. So the latter is much faster.
No idea what causes apexcharts-card to slow down the displaying system. Because if you force close the browser or companion app and reopen it, the system is fast again until you try to view the chart again.

Just a note... this issue was open for nearly 2.5 years.  Unfortunately, @RomRider wasn't able to reproduce this issue that everyone else was having on his Home Assistant setup.  He offered some general tips (below) which may help a little to reduce the load.  However, those tips didn't help me resolve the actual issue (presuming it's related to how ApexCharts interacts with Home Assistant's "frontend", Lovelace).  Maybe when @RomRider has time, he could open an issue here: https://github.com/home-assistant/frontend?
https://github.com/RomRider/apexcharts-card/issues/210#issuecomment-1118884848

use the last version (and make sure it's not using and old cached version).
use update_interval (it's now fixed).
use group_by to reduce the number of data points displayed
don't try to display 10th of thousand of datapoints on a chart (eg: a sensor updating every 5 sec displayed over 2 days is 35000 datapoints unless you use group_by)
make sure there are no other custom cards interfering

uaaauaaa comment is EXACTLY what happens on my system....

Might want to try the latest beta version, I have seen some performance improvements as it's now using the latest apexcharts.js library


Implement doesCommandExist for Posix instead of linux

See https://stackoverflow.com/questions/762631/find-out-if-a-command-exists-on-posix-system
for reference.

thanks a lot ... it is not really critical for me, but good to have it also working on os x ...


Clickable issue numbers

This commit turns issue numbers on the downloads page into clickable GitHub Issues links.
This is done by passing the commit comment to a new parseIssues function, which looks for exact matches of issue numbers. If a match is found, the issue number is replaced with a clickable link.

Clickable issue numbers have been re-implemented in PHP:
https://github.com/pl3xgaming/PurpurWebsite/pull/6/commits/e1938d3b06ba7d5b5c49e268de5e064a8642f63d
I've also now included clickable MC-###### issue numbers.


feat: Add uptime in status_api

Closes issue #7 of `kytos_stats
Summary
This PR augments the core endpoint GET /status to include when the APIServer started and how much time has elapsed.
Local Tests
End-to-End Tests
N/A

Thank you so much, @viniarck.


Add cluster-mode to Dataproc Qualification

Signed-off-by: Ahmed Hussein (amahussein<EMAIL_ADDRESS>Contributes to #182

Add cluster mode to Dataproc Qulification by adding new argument exec_cluster which is the cluster name on which the tool is supposed to run. The cluster has to be running Spark3.x.
Some few changes were needed to the Qualification class to support the new execution mode
Implementation for Databricks/EMR was left blank
Most of the issues were due to gsutil API and make storage-driver works on both S3 and GS

gsutil does not have the notion of directory/file which causes bugs copying from/to remote storage
gsutil will lead to nested directories compared to S3

Added a quick section for dataproc Cluster-deployment mode

Thanks @amahussein for adding cluster mode  to dataproc qual tool. Overall looks good. Just have one nit/question.


feat(array-methods): add forEach exercise

Migrated from https://codepen.io/neuefische/pen/LYOezjB

I applied the changes and rewrote the exercise. Might be easier now and as far as I know, can be merged. 🚢


[$500]  LHN - Web - Offline created chat on LHN disappear and reappear when come back online

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

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

Disable the internet connection in the device
Click on the green + button
Click on the "Send Message" option
Search for a user with a valid email that you don't have any messages with and open the chat
Verify the user is navigated to the conversation
Verify the conversation is grayed out indicating is pending to be created
Send some messages
Navigate to another chat
Come back online

Expected Result:
Chat that was created in offline mode should be stable on LHN
Actual Result:
Offline created chat on LHN disappear and reappear when come back online
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

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

Upwork Job URL: https://www.upwork.com/jobs/~01573f4bf2c4908020
Upwork Job ID:<PHONE_NUMBER>837231104
Last Price Increase: 2024-01-29
Automatic offers:

tienifr | Contributor | 28130631

Issue OwnerCurrent Issue Owner: @arosiclair

This is expected if the user's priority mode is #Focus.
Could you provide some screenshots or videos?
Thanks 😄

Taking over!

Hi @tienifr!
Thanks for your proposal.
Thanks

@allroundexperts Sure!
Here they are: Before: https://github.com/Expensify/App/assets/113963320/a08f89a9-7245-4544-873c-7109856ec5d9
After: https://github.com/Expensify/App/assets/113963320/53b200fd-71bc-46e4-80ee-d909e6e67eec

Cool
Is this issue solved ?
please close it and others also !

Thanks for the screen recordings @tienifr. Your proposal works good. Let's go for it.
🎀 👀 🎀 C+ reviewed

No BZ got assigned for some reason. Trying again

The default notificationPreference was set to hidden to fix empty chats displaying as "unread" in focus mode. I'm asking about this internally here.

Hey @arosiclair what's the latest with this one, did you get the info you needed?

Yup sorry for the late update. I agree with @marcaaron's assessment:

A correct solution would be something like combining dependent requests into a single request to handle any side effects. But it’s just not how the offline architecture works right now. So, the behavior is “expected”.

This behavior is somewhat expected since there are intermediary steps between the offline chat being created and the offline messages being added. Moreover, the default hidden preference is correct until comments are added.
@tienifr's frontend changes still make sense to me, but it sounds like they do not fix the issue on their own. Is that correct?

Bump @tienifr ^

Hey @tienifr can you weigh in on Andrew's question above?

Alright if frontend changes don't fix the issue on their own, then we should just do nothing here. Closing this out.

Summarizing payment on this issue:

Contributor: @tienifr $500 via Upwork (PAID)
Contributor+: @allroundexperts $500 via Upwork (offer extended, please accept!)

Upwork job is here: https://www.upwork.com/jobs/~01573f4bf2c4908020

OH, thanks @allroundexperts. All set!

$500 approved for @allroundexperts based on summary above.


Stacked mode implementation

@koukibadr  As per #102 #82  . This feature allows the developer to push multiple notifications in stacked mode given the same key, it is highly customizable and I think it covers most of the cases discussed in these 2 threads

@Pana-g there's conflict in this PR after merging this PR

@koukibadr Conflicts have been resolved.

Thanks @Pana-g for your contribution I'll publish these new updates in the next version


EnumMap: don't construct globally.

Objects with non-trivial construction should not be
initialized at global scope.
Signed-off-by: Henner Zeller<EMAIL_ADDRESS>

Codecov Report

Merging #943 (e8f3180) into master (7a0f35d) will decrease coverage by 0.00%.
The diff coverage is 92.00%.

@@            Coverage Diff             @@
##           master     #943      +/-   ##
==========================================
- Coverage   92.76%   92.76%   -0.01%
==========================================
  Files         316      316
  Lines       20701    20731      +30
==========================================
+ Hits        19204    19231      +27
- Misses       1497     1500       +3

Impacted Files
Coverage Δ

verilog/analysis/verilog_linter.h
100.00% <ø> (ø)

verilog/formatting/tree_unwrapper.cc
97.22% <0.00%> (-0.33%)
:arrow_down:

common/formatting/align.cc
90.99% <100.00%> (+0.04%)
:arrow_up:

common/formatting/basic_format_style.cc
100.00% <100.00%> (ø)

verilog/analysis/symbol_table.cc
98.80% <100.00%> (+<0.01%)
:arrow_up:

verilog/analysis/verilog_equivalence.cc
94.91% <100.00%> (+0.13%)
:arrow_up:

verilog/analysis/verilog_linter_configuration.cc
90.44% <100.00%> (+0.16%)
:arrow_up:

verilog/tools/diff/verilog_diff.cc
100.00% <100.00%> (ø)

verilog/tools/kythe/verilog_kythe_extractor.cc
87.09% <100.00%> (+0.65%)
:arrow_up:

verilog/tools/lint/verilog_lint.cc
98.73% <100.00%> (+0.04%)
:arrow_up:

... and 9 more

Continue to review full report at Codecov.

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


adding partnercenter

CLI extension for the Partner Center, v0.2.6-alpha
Currently, experimentally supporting Marketplace management commands, commissioned by Global Partner Services US division.
Maintainers
@kevinhillinger
@bobjac
Source
Type: External repository
URL: https://github.com/Azure/partnercenter-cli-extension

Thank you for your contribution! We will review the pull request and get back to you soon.

Hi @bobjac, Please write the description of changes which can be perceived by customers into HISTORY.rst. If you want to release a new extension version, please update the version in setup.py as well.

I should have updated the VERSION in version.py to 0.2.6.  I will update the HISTORY.rst and commit a merge to main.  Would you like me to create a new release for version 0.2.7 that contains the updates to the HISTORY.rst or would an update of the main branch be sufficient?

Hi @jsntcy - Is there anything else that I need to do on my end?

@jsntcy @yonzhan @AllyW  - I just wanted to follow-up and see if there is anything else needed on my end.

@jsntcy @yonzhan @AllyW - Are there any updates on this?


ScrollView cannot be cast to com.melnykov.fab.ObservableScrollView

compileSdkVersion 21
minSdkVersion 15
targetSdkVersion 21
What happen?

ObservableScrollView extends ScrollView. You can cast from ObservableScrollView to ScrollView, but not vise versa. See casting objects at http://docs.oracle.com/javase/tutorial/java/IandI/subclasses.html

@IlyaEremin is right. Closing this issue.


dispatched LoadData action on adding new data

dispatched LoadData action on adding new data to load the newly added data

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Bose seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

No this is not perfect ,please use CRUD operations multiple times then you will find the error

Closed as it is already included in #22


system-libs default [.NET 10]

Currently the vmr defaults to build with bundled libs.
To fix an issue in .NET 9 we switched to use system brotli in https://github.com/dotnet/runtime/pull/107225 and in https://github.com/dotnet/runtime/pull/107166 we're switching back to bundled for .NET 10.
If there is a set of libraries that maintainers will likely prefer to use from the system, it would be nice if the vmr would default to those for "non-portable, source-only" builds.
I assume that brotli is one of those because there was zero objection to switch to the system brotli in https://github.com/dotnet/runtime/issues/107020.
cc @MichaelSimons @omajid @jkotas @dotnet/distro-maintainers

It would be the default value of the argument that is settable on the vmr build script.
For convenience, it would be nice if we make runtime use the same system libs by default when building it non-portable.

For convenience, it would be nice if we make runtime use the same system libs by default when building it non-portable.

It would mean to hard this default in dotnet/runtime repo. If the default is hardcoded in dotnet/runtime repo, is there anything else needed?

No. The runtime can use the default when nothing is set explicitly.
A user will have to look in the runtime sources to know what the default means.

[Triage] If the default is to use the system-libs, we need to support scenarios where a distro doesn't have all of the required system libs (e.g. there has to be a granular opt out mechanism).  I would like more feedback from @dotnet/distro-maintainers on this.  What behavior would they like to see?

On Ubuntu, the default is to always use archive dependencies whenever possible. Having the VMR default to system libraries then is handy because it would follow our behavior and we would avoid having to maintain a --with-system-libs for all our builds. Also, in a scenario where a new dependency is taken, we don't inadvertently upload a build with a bundled library before either adding a new archive dependency to the package itself or explicitly stating that the bundled version is to be used.
So, IMO I believe that inverting the flag logic from --with-system-libs to --with-bundled-libs would make more sense. We do have corner cases where we use a bundled library because the archive one won't work (such as how we are currently using the bundled libunwind for non-x64 builds), and having an explicit stance for that in the build parameters, instead of the opposite, makes more sense from a package maintenance standpoint, especially since it's much easier to visualize.
Also, I think it would be quite helpful if the help message for the flag in the ./build.sh --help output stated exactly which libraries/entries are supported.


Update getting-started-overview-new.md

Don't need this for now

🚀 Deployed on https://65775198294e942744d968fe--cozy-fox-0defec.netlify.app


fix bug in MinTSOScheduler that estimated_thread_usage/waiting_tasks_count/active_tasks_count are not 0 even if there is no queries running

Signed-off-by: xufei<EMAIL_ADDRESS>What problem does this PR solve?
Issue Number: close #5556
Problem Summary:
What is changed and how it works?
Fix some bugs that active tasks count/ estimated thread usage may be repeat added, and type_waiting_tasks_count may lost update when schedule meet error.
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/merge


Convert Option to Kotlin

@gino-m Any ideas why the @Serializable annotation doesn't seem to be working properly?

@gino-m Any ideas why the @Serializable annotation doesn't seem to be working properly?

Actually I think I was able to resolve this after following the setup instructions here: https://github.com/Kotlin/kotlinx.serialization#setup

@gino-m Any ideas why the @Serializable annotation doesn't seem to be working properly?

I was just looking into this, I see you found a fix!

Codecov Report

Merging #1201 (29f0ec9) into master (c9add97) will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1201      +/-   ##
============================================
+ Coverage     29.57%   29.58%   +0.01%
  Complexity      785      785
============================================
  Files           292      292
  Lines          7612     7608       -4
  Branches        347      345       -2
============================================
  Hits           2251     2251
+ Misses         5252     5248       -4
  Partials        109      109

Impacted Files
Coverage Δ

...nd/persistence/local/room/entity/OptionEntity.java
0.00% <0.00%> (ø)

...istence/remote/firestore/schema/OptionConverter.kt
0.00% <0.00%> (ø)

...d/com/google/android/ground/model/task/Option.java

...und/com/google/android/ground/model/task/Option.kt
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Develop ISA for Data Upload Manager to enable SBN-PSI to deliver data to the cloud

💡 Description

Underway.

Status: Document is ongoing with hopefully deliver to SAs and SBN team by end of week

@jordanpadams is answering the comments.

Decided not to pursue a specific ISA for SBN-PSI, and will just create a detailed ISA for all the nodes. https://github.com/NASA-PDS/software-issues-repo/issues/73


⚠ Bofrost.de has degraded performance

In 24dfd01, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1441 ms

Resolved: Bofrost.de performance has improved in 19746f1.


iOS runtime name is no longer consistent

Is there an existing issue for this?

[X] I have searched the existing infra issues

Type of Request
bug
Infrastructure Environment
LUCI
What is happening?
iOS runtime dmgs are uploaded separately from Xcode. As such, the dmg file retains the file name it had when uploaded.
For example: iOS_17_Simulator_Runtime.dmg
https://logs.chromium.org/logs/flutter/led/vashworth_google.com/aa2fb5b8a22a660d5ff0abb028b89b49aa8b651d5c8149e14e3421afb30fb331/+/u/install_runtimes/list_xcode_runtime_dmg_ios-17-0_15a240d/listdir
In our current recipes, we generate the name of the dmg using the version. However, this is no longer reliable since the name may not be the same each time. We should update recipes to find the dmg from the files downloaded rather than by name.
Steps to reproduce
No response
Expected results
No response

Fixed in https://flutter-review.googlesource.com/c/recipes/+/52900.


F.temp optimizer and sort order merged (DONT MERGE, but Hannah may test it)

There is a newer version of this "One for everything TEMP pr, close this one.


Build(deps): Bump history from 5.2.0 to 5.3.0 (#2838)

Pull request opened by the merge tool on behalf of #2838

Merged in 25cf0cce9008d292989d482bd337ebbeefbddf1c


[Question]: Can RAGFlow be deployed locally?

Describe your problem
使用bash entrypoint.sh之后，用的是T4GPU ,感觉特别慢，离线部署的时候，
embd_mdl = LLMBundle(r["tenant_id"], LLMType.EMBEDDING, llm_name=r["embd_id"], lang=r["language"])这行代码和
paddle layouts: 0.1166270489920862这个日志之间间隔差不多10分钟。不知道是哪一步占用了时间。
我查看nvidia-smi之后，发现有启动的进程好像有使用GPU，但就算不使用GPU，速度也不会有10分钟左右吧

We intend to create an international community, so we encourage using English for discussion. Thank you.

It's probably downloading embedding model.
GPU can only accelerate embedding procedure. And you need to change docker-compose..yaml a little bit.

It's probably downloading embedding model. GPU can only accelerate embedding procedure. And you need to change docker-compose..yaml a little bit.

That's a complete offline state. I feel that it is time to ask the external network timeout, but I don't know which part of the code.


Internal authentication DB username spec inconsistency

While developing a custom kibana plugin for a client looking to deploy an ODfE cluster, my team has been asked to configure user access using the internal authentication database.
We've developed functionality to add users to the internal auth db via the REST API without an issue, initially.
While updating some of our test users via the Kibana security plugin rather than the REST API, we've been presented with the following error trying to save the modified user:
Username must not contain '*', '.' or curly brackets
However, the REST API enforces no such requirements on usernames for newly created or modified users. Users created via the REST API using an email address as a username are able to log in as we had expected without issue.
Could someone address this inconsistency and (preferably) remove the restriction on . characters in usernames via the kibana security plugin, or alternatively (and definitely not preferred) add the restriction to the REST API for consistency?

Can we get some feedback on this so we know whether or not we need to switch to a different solution?

Still nothing?

Thanks for raising this. We will look into this.

I currently hit this issue also. We have several FQDN of servers used as username which worked very well (I am not sure if this restiricton was already in ODFE 1.3.0 and we just did not reognize it) but now with ODFE 1.9.0 I get this error.
Please remove the dots-restriction as it would be difficult to get non-conflicting usernames otherwise

@DJSundog @WennySoft This appears to be fixed in latest Opensearch version 1.2.0, perhaps you might want to think about upgrading if you are still using old odfe versions.

We are doing some "spring cleaning in the fall", and to make sure we focus our energies on the right issues and we get a better picture of the state of the repo, we are closing all issues that we are carrying over from the ODFE era (ODFE is no longer supported/maintained, see post here).
If you believe this issue should still be considered for current versions of OpenSearch, apologies! Please let us know by re-opening it.
Thanks!


Alacritty is fuzzy after screen lock

Which operating system does the issue occur on?

Fedora 31 with alacritty built from git (v0.3.3-98-gb3afb97)

If on linux, are you using X11 or Wayland?

Wayland with fractional scaling enabled on GNOME 3.34
gsettings set org.gnome.mutter experimental-features "['scale-monitor-framebuffer']"

and then set scaling factor to 150% in GNOME settings.
Here is how a terminal looks when I start it:

and here is how it looks after the screen locks and shuts off, then comes back after unlocking:

You have to open those full-size and you'll see the difference in the text rendering at the top left of the terminal.
Also the terminal originally had 75 lines and after unlocking the screen only has 73 lines...

I'd assume this gets fixed when the window is resized in any way?

I'd assume this gets fixed when the window is resized in any way?

Sometimes this fixes it, sometimes not.

Does this happen without fractional scrolling enabled?

Not sure. I disabled fractional scaling, and it hasn't happened yet.
However here's another case where it happens, without fractional scaling:

start alacritty with scaling set to 100%
change scaling to 200%
alacritty is fuzzy, and resizing doesn't fix it

This sounds a bit like https://github.com/jwilm/alacritty/issues/2886 but for Wayland.
Can you share the output of alacritty -v when trying to change the DPI while it's running?

This is a known upstream bug https://github.com/Smithay/client-toolkit/issues/81

probably dup of #1801


feat: cmd to list provide addrs

@dignifiedquire this also includes a quick and dirty test for the IPs. Think this is more stable & sufficient compared to the netsim integration tests.
@flub I used pieces from your PR on https://github.com/n0-computer/iroh/pull/820 and made a utility function. Potentially you can reuse (or I depending who merges first).
Closes https://github.com/n0-computer/iroh/issues/858

@dignifiedquire


Update UPGRADE_FROM_FACTORY_GIRL.md

At my OS X Yosemite 10.10.5 the grep commands does not work. I have updated them with the versions working for me.

Thanks for this PR! The example we gave was really intended to be an example, not necessarily a perfect solution that would work for everyone. We have had lots of PRs suggesting alternatives (#1070, #1075, #1084, #1095, #1102), but to reduce churn on this file I have decided to close these issues. I appreciate your help!


The product price should be correct after exporting to excel.

CRM Version: V1.2.0
Steps To reproduce--
1. Log in as Admin/User.
2. Click on the product inside the category tab.
3. Try to export the products.
4. View the product's excel file.
Actual result: The product price is not correct after exporting to excel. Please find the below screenshot.
Expected result: The product price should be correct after exporting to excel.

@Sanjaybhattwebkul did you verified this with the master branch?

@Sanjaybhattwebkul did you verified this with the master branch?

@ghermans   Yes I did


Colab notebook has error, numpy array used instead of torch

while True:
    user_message = input("\nYour message: ")
    messages.append(dict(
        role="user",
        content=user_message
    ))

    out = model.generate(input_ids=input_ids, max_length=2000, temperature=0.9, top_p=0.7, eos_token_id=tokenizer.eos_token_id)

    decoded = tokenizer.batch_decode(out)
    messages.append(dict(
        role="assistant",
        content=decoded[0].split("<|assistant|>\n")[-1])
    )

    print("Model:", decoded[0].split("<|assistant|>\n")[-1])

Error:
AttributeError                            Traceback (most recent call last)
[<ipython-input-6-8618df48d4c0>](https://localhost:8080/#) in <cell line: 2>()
      9     input_ids = tokenizer.apply_chat_template(messages, return_tensors="pt", add_generation_prompt=True).to("cuda")
     10
---> 11     out = model.generate(input_ids=input_ids, max_length=2000, temperature=0.9, top_p=0.7, eos_token_id=tokenizer.eos_token_id)
     12
     13     decoded = tokenizer.batch_decode(out)

18 frames
[/usr/local/lib/python3.10/dist-packages/causal_conv1d/causal_conv1d_interface.py](https://localhost:8080/#) in forward(ctx, x, weight, bias, seq_idx, activation)
     17             x = x.contiguous()
     18         bias = bias.contiguous() if bias is not None else None
---> 19         seq_idx = seq_idx.contiguous() if seq_idx is not None else None
     20         ctx.save_for_backward(x, weight, bias, seq_idx)
     21         ctx.activation = activation in ["silu", "swish"]

AttributeError: 'str' object has no attribute 'contiguous'

Line most likely causing error, torch conversion did not happen, numpy arrays do not have any contigous method. Fix this to explicitly cast as torch tensor:
input_ids = tokenizer.apply_chat_template(messages, return_tensors="pt", add_generation_prompt=True).to("cuda")

So like what is the fix now? The input_ids is indeed of type tensors

@microcoder-py Have you modified the code in the notebook in some way? The return_tensors='pt' argument is already present in the code if I'm not mistaken

The code is run as it is from the colab @justusmattern27  and still it gives the above error

@LuciAkirami got it, thanks - I ran into it myself now, something that should be a tensor seems to be a string here, which is pretty weird. The notebook worked a few days ago, so I guess it could be due to updates in the dependencies. Will look into it later

It was indeed due to wrong versions of the dependencies, I pinned them now.
@LuciAkirami @microcoder-py This should be fixed now, feel free to let me know if you run into the error again. Otherwise, I'll close the issue soon :)

Cooool @justusmattern27 , now it's working

nice!


Issue when using INFRACOST_TERRAFORM_CLOUD_TOKEN and multiple projects in a config file

I have a very simple project file:
version: 0.1

projects:
  - path: production
  - path: staging

And when using the env variable to set the TF CLOUD credentials, the first project will run but the second project fails.
The output is:
time="2021-04-23T11:31:19-04:00" level=info msg="Detected Terraform directory at production"
time="2021-04-23T11:31:20-04:00" level=warning msg="Can't sync usage file as it's not specified"
time="2021-04-23T11:31:20-04:00" level=debug msg="Creating temporary config file for Terraform credentials"
time="2021-04-23T11:31:20-04:00" level=debug msg="Writing Terraform credentials to temporary config file /tmp/945025377"
time="2021-04-23T11:31:20-04:00" level=info msg="starting: Running terraform plan"
time="2021-04-23T11:31:20-04:00" level=info msg="Running command: /home/rridley/.asdf/shims/terraform plan -input=false -lock=false -no-color -out=/tmp/tfplan326797644"
....
time="2021-04-23T11:32:04-04:00" level=info msg="Detected Terraform directory at staging"
time="2021-04-23T11:32:04-04:00" level=warning msg="Can't sync usage file as it's not specified"
time="2021-04-23T11:32:04-04:00" level=debug msg="Creating temporary config file for Terraform credentials"
time="2021-04-23T11:32:04-04:00" level=debug msg="TF_CLI_CONFIG_FILE is set, copying existing config from /tmp/945025377 to config to temporary config file /tmp/333311547"

I snipped the logs that didn't seem relevant
I see the file 333311547 get created, but ultimately it's empty when I check it after it fails.

Thanks @keslerm. I was able to reproduce and have fixed in https://github.com/infracost/infracost/pull/610.

@aliscott excellent! thank for the quick response, this is an fantastic tool!

@keslerm this has been fixed in v0.8.5


Refreshed Connection Solution Problem

I installed the RefreshedConnectionsUI_1_1_managed.zip file and then navigated to the Connections.htm page.  Using the debugger I see that there are exceptions preventing the page from functioning.
I have a screen shot of the page and the debugger but I can't seem to add it to the issue.
The errors are:

Unspecified error. (Connections.htm(1,1))
Unable to get property Entity of undefined or null reference (ClientUI.js(48,242))
Unable to get the property 'majorVersion' of undefined or null reference (ClientUI.js(39,47))

I have installed SparkleXrm 7.3.0 on the server.
Any ideas?

This is because 7.3.0 changed the namespace - I'll recompile and upload the new version.

I get an error when trying to import it: The solution file is invalid. The compressed file must contain the following files at its root: solution.xml, customizations.xml, and [Content_Types].xml. Customization files exported from previous versions of Microsoft Dynamics 365 are not supported.


New article translated and translation corrections

Translated:

WebGL2 from WebGL1
Corrected:
All articles that have already been sent

I wasn't sure if you planned to make this not have conflicts or you want me to do it.
It would nice if you could at least clean up the bad commits. For example I see "delete npm-debug.log". What that means is npm-debug.log will be added to the repo which is a waste of space.
I can fix this on my side but I'm guessing you just need some git tips because if I fix it on my side I suspect your next pull request will have similar conflicts.
The simplest way to resolve all of this i think is to rebase your changes on top of this version
    # downloads the files from this repo and puts them in the database in your .git folder
    git fetch https://github.com/greggman/webgl2-fundamentals.git master

    # apply your changes on top of the current version from this repo (the version you just fetched)
    git rebase FETCH_HEAD master

you'll likely get conflict errors. For each file that has a conflict you need to load it into your text
editor and look for groups of lines that look like
   <<<<<<
   some lines
   from upstream-master (this repo)
   ========
   some lines
   from master (your repo)
   >>>>>>

You need to edit them so they're whatever they need to be from now on in the process deleting
the <<<<< ====== >>>>>> parts as well
You then need to git add them. For example if the file is webgl/lessons/foo.md that has a conflict
then after fixing the conflict in your text editor and saving the file you'd type
    git add webgl/lessons/foo.md

Once you've done this for all the files with conflicts you'd type
    git rebase --continue

If you get more conflicts repeat the process.
If at anytime you make a mistake you can type git rebase --abort
Finally you squash your commits. To do that this should work
  # reset git's idea of the current version to the same as this repo
  git reset 7b9f269

  # check in all your files
  git add webgl/lessons/pt-br/webgl1-to-webgl2.md
  git commit -a -m "New article translated and translation corrections"

Then upload that to github
  git push --force origin master

If you don't want to do this then I can do it on my side but you need to reset your git after I'm done so there's no more conflicts.
    git fetch https://github.com/greggman/webgl2-fundamentals.git master
    git reset --hard FETCH_HEAD

Thanks for your tips and sorry for my mistakes. I'm still new at all these git stuff and I accidentally added npm-debug.log to my fork, so I needed to delete it. But don't worry, I'll fix as soon as possible and I'll create a pull request again.
Thanks for your pattience.

FYI, in the future you don't need to close a pull request to fix it. If you updated your repo and then pushed with --force this pull request would have automatically been updated with your changes.


Docs are out of date

The docs are out of date.

Found the reason why. Since we moved to container-based testing, the part pushing the docs (which used sudo) was not longer working. I've updated it to use a --user install, and it's updating now.


Twitter DMs returning unparsable Unicode

Environment: Twitter gem v5.15.0. Rails 4.2.8, Ruby 2.3.2.
I understand there's a chance this is patched under a more recent version, but I haven't been able to find an issue or PR related.
In a request for retrieving DMs, I'm getting a parsing error that appears to be from Twitter returning potentially invalid Unicode. Of course, ideally Twitter would never return invalid unicode, but we don't control them.
In the return body of a DMs retrieval request (I can't post the entire thing, because it belongs to a client). The vast majority (99.99%) of DM retrieval works in my codebase, so it isn't a code error otherwise on my end.
"text":"Hello! \ud83d\ude2d\ud83d... "

It appears to be an issue from the Twitter:REST:Response#parse call to JSON.parse(body, :symbolize_names => true). I'm calling the twitter_rest_client_instance.direct_messages(since: last_message_id, count: 200) method. Below is my stack trace:
JSON::ParserError: invalid escaped character at line 1, column 20187 [parse.c:259]
  from twitter/rest/response/parse_json.rb:15:in `parse'
  from twitter/rest/response/parse_json.rb:15:in `parse'
  from twitter/rest/response/parse_json.rb:20:in `on_complete'
  from faraday/response.rb:9:in `block in call'
  from faraday/response.rb:57:in `on_complete'
  from faraday/response.rb:8:in `call'
  from faraday/response.rb:8:in `call'
  from faraday/request/url_encoded.rb:15:in `call'
  from faraday/request/multipart.rb:14:in `call'
  from twitter/rest/request/multipart_with_file.rb:19:in `call'
  from faraday/rack_builder.rb:139:in `build_response'
  from faraday/connection.rb:377:in `run_request'
  from faraday/connection.rb:140:in `get'
  from twitter/rest/request.rb:33:in `perform'
  from twitter/rest/utils.rb:50:in `perform_request'
  from twitter/rest/utils.rb:95:in `perform_request_with_objects'
  from twitter/rest/utils.rb:80:in `perform_get_with_objects'
  from twitter/rest/direct_messages.rb:27:in `direct_messages_received'
  from twitter/rest/direct_messages.rb:90:in `direct_messages'
  from lib/twitter_api_service.rb:66:in `block in direct_messages'
  from lib/hyper/utils.rb:23:in `social_network_request'
  from lib/twitter_api_service.rb:65:in `direct_messages'
  from app/workers/twitter_direct_message_retrieval_worker.rb:8:in `perform'
  from sidekiq/processor.rb:158:in `execute_job'
  from sidekiq/processor.rb:138:in `block (4 levels) in process'
  from sidekiq.rb:36:in `block in <module:Sidekiq>'
  from sidekiq/processor.rb:133:in `block (3 levels) in process'
  from sidekiq/middleware/chain.rb:128:in `block in invoke'
  from new_relic/agent/instrumentation/sidekiq.rb:33:in `block in call'
  from new_relic/agent/instrumentation/controller_instrumentation.rb:363:in `perform_action_with_newrelic_trace'
  from new_relic/agent/instrumentation/sidekiq.rb:29:in `call'
  from sidekiq/middleware/chain.rb:130:in `block in invoke'
  from sidekiq/middleware/server/active_record.rb:6:in `call'
  from sidekiq/middleware/chain.rb:130:in `block in invoke'
  from raven/integrations/sidekiq.rb:7:in `call'
  from sidekiq/middleware/chain.rb:130:in `block in invoke'
  from sidekiq/batch/middleware.rb:33:in `call'
  from sidekiq/middleware/chain.rb:130:in `block in invoke'
  from sidekiq-ent/limiter/middleware.rb:38:in `call'
  from sidekiq/middleware/chain.rb:130:in `block in invoke'
  from sidekiq/middleware/server/retry_jobs.rb:74:in `call'
  from sidekiq/middleware/chain.rb:130:in `block in invoke'
  from sidekiq/failures/middleware.rb:9:in `call'
  from sidekiq/middleware/chain.rb:130:in `block in invoke'
  from sidekiq/middleware/server/logging.rb:11:in `block in call'
  from sidekiq/logging.rb:32:in `with_context'
  from sidekiq/middleware/server/logging.rb:7:in `call'
  from sidekiq/middleware/chain.rb:130:in `block in invoke'
  from sidekiq/middleware/chain.rb:133:in `invoke'
  from sidekiq/processor.rb:132:in `block (2 levels) in process'
  from sidekiq/processor.rb:174:in `stats'
  from sidekiq/processor.rb:131:in `block in process'
  from sidekiq.rb:35:in `block in <module:Sidekiq>'
  from sidekiq/processor.rb:126:in `process'
  from sidekiq/processor.rb:82:in `process_one'
  from sidekiq/processor.rb:70:in `run'
  from sidekiq/util.rb:17:in `watchdog'
  from sidekiq/util.rb:26:in `block in safe_thread'
Twitter::Error: invalid escaped character at line 1, column 20187 [parse.c:259]
  from twitter/rest/request.rb:37:in `rescue in perform'
  from twitter/rest/request.rb:32:in `perform'
  from twitter/rest/utils.rb:50:in `perform_request'
  from twitter/rest/utils.rb:95:in `perform_request_with_objects'
  from twitter/rest/utils.rb:80:in `perform_get_with_objects'
  from twitter/rest/direct_messages.rb:27:in `direct_messages_received'
  from twitter/rest/direct_messages.rb:90:in `direct_messages'
  from lib/twitter_api_service.rb:66:in `block in direct_messages'

I can attempt to write a failing test for it. Since it deals with something pretty deep inside Ruby's JSON parsing, I'm not sure what we can do about it though.

There's now a $100 bug bounty on this:
https://www.bountysource.com/issues/45140196-twitter-dms-returning-unparsable-unicode

hey @tibbon,
JSON parsing is ultimately left to the json library so it seems that this issue is most likely upstream. I see that you opened an issue on flori/json but it doesn't look like it has been addressed yet.
I think the best approach is to create a failing test there and see if that can get things headed toward resolution. I'm not a C programmer, but I did attempt to locate the code that raises the error only to find that parse.c does not exist in the code base. 😕

Just a bump here on this. We're still getting this frequently, and I assume others are too.


Fix for GRPC frame too large and context closed warning

Fixes #2216 and tries to fix #2218

Lint is failing


[ci] Fix bazel query bug to run a larger set of tests

#12611 had a bug in the bazel query that did not request the full set of tests to be run. This PR should fix that.

What was the bug? Which tests did it miss?

The bazel query I guess wasn't being formed correctly. It only ran the tests I explicitly listed.

Oh yeah bazel isn't installed on the CI servers anymore, so by moving it to bazelisk it should be fixed.


Registers with long names are tructated with elipsis

there is no way to display the full name.

Yes, this is possible by executing the set command through GDBDriver. I will try to add this in the next release.


Fixed node cpu stats …

…so that the percentage value is divided by the number of reported cores, since elasticsearch is reporting summarized stats across all (HT-)cores.

makes sense, thanks :+1:

@jannemann hum, actually this won't work exactly like this. Will have to take a look, as some of the data from os returned by ES was removed.

They removed sigar und now using jmx through reflection.
See changes here: https://github.com/elastic/elasticsearch/commits/e95af69fb71d67c3fb2a2806da24214dc4805140/core/src/main/java/org/elasticsearch/monitor/os
So it is possible to gather cpu stats for a node in "os.cpu.usage". Just like marvel does.
I checked it by using stress on a linux machine and it reflects exactly what htop shows.
Should I create another pull request?

Sure :)
Just make sure to do it both on the dist and also on the source file. I can fix the tests later as this is not really well documented and still kind of manual(regarding changes of the elastic response format).
thanks again :)

@jannemann just had some time to look into this, and I cannot really see where "os.cpu.usage" is coming from.
This used to be available through sigar, and since sigar was removed the best you could get was cpu from the jvm process.
Thought maybe they had added this back somehow, but looking at the code, and also at the output of ES 2.0 just downloaded from their site, I can't see it anywhere.
Have to leave now, if you have time to look at that on your end great, otherwise hopefully tomorrow I can look into it again.
I would only wait for this, and cut the 2.0 release, together with ES :)


add fuzz target for ring

https://github.com/briansmith/ring
there's potential for many fuzz targets here. those can be separate PRs

also, it might be look into turning on sanitization when compiling crates with C code


liquibase changelog isuue-- unable to generate liquibase script.

Liquibase Version: 4.14.0
Liquibase Integration & Version: <Pick one: CLI, maven, gradle, spring boot, servlet, etc.> CLI- 4.14.0
Liquibase Extension(s) & Version:
Database Vendor & Version: cloudera(hive)-3.1.3<IP_ADDRESS>.0-551
Operating System Type & Version:
Infrastructure Type/Provider: <AWC, GCS, Azure, VM, etc>
Description
I tried to execute a liquibase script to create a table in a hive database.
Getting error 'errorMessage:Error while compiling statement: FAILED: SemanticException [Error 10099]: DATETIME type isn’t supported yet. Please use DATE or TIMESTAMP instead), Query: CREATE TABLE DATABASECHANGELOGLOCK (ID INT NOT NULL, LOCKED BOOLEAN NOT NULL, LOCKGRANTED datetime, LOCKEDBY VARCHAR(255), CONSTRAINT PK_DATABASECHANGELOGLOCK PRIMARY KEY (ID)).'
Seems the internal tables of liquibase are attempting to use an invalid datatype.
I tried to create DATABASECHANGELOGLOCK and DATABASECHANGELOG manually (replacing with hive valid datatype).
Nonetheless, it tries to recreate those tables and fails with the same error.

Hi! We already had reports for this kind of issues before. The suggestion I can give for you at this moment is on this issue and create a new database class. This is probably the fastest solution and we will very happy to recive any PR from the community! I hope this helps.

Hi! Im closing this issue for now. You can always re-open it if you want to adress it on the future! Thanks


More integration test

Codecov Report

Merging #495 into master will increase coverage by 4.35%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #495      +/-   ##
============================================
+ Coverage     72.54%   76.89%   +4.35%
- Complexity     1237     1312      +75
============================================
  Files           102      102
  Lines          4469     4471       +2
  Branches        645      644       -1
============================================
+ Hits           3242     3438     +196
+ Misses         1026      841     -185
+ Partials        201      192       -9

Flag
Coverage Δ
Complexity Δ

#fixedclientenvironment
20.86% <55.55%> (+14.02%)
341 <2> (+224)
:arrow_up:

#integration
49.63% <27.77%> (+1.43%)
734 <0> (+19)
:arrow_up:

#livelocal
7.51% <5.55%> (+0.73%)
127 <0> (+11)
:arrow_up:

#unit
51.28% <50%> (+8.5%)
954 <1> (+133)
:arrow_up:

Impacted Files
Coverage Δ
Complexity Δ

...r/xenon/adaptors/schedulers/JobQueueScheduler.java
94.87% <ø> (+19.48%)
67 <0> (+21)
:arrow_up:

...xenon/adaptors/schedulers/RemoteCommandRunner.java
87.09% <100%> (ø)
8 <0> (ø)
:arrow_down:

...xenon/adaptors/schedulers/local/CommandRunner.java
80.85% <100%> (+80.85%)
12 <2> (+12)
:arrow_up:

...center/xenon/adaptors/schedulers/OutputReader.java
91.66% <100%> (+14.39%)
12 <1> (+3)
:arrow_up:

...ecenter/xenon/adaptors/schedulers/InputWriter.java
88.23% <100%> (+16.36%)
9 <0> (+2)
:arrow_up:

...aptors/schedulers/local/LocalSchedulerAdaptor.java
90.9% <100%> (+24.24%)
11 <0> (+5)
:arrow_up:

...ors/schedulers/gridengine/GridEngineScheduler.java
79.02% <0%> (-1.4%)
43% <0%> (-1%)

...on/adaptors/schedulers/torque/TorqueScheduler.java
74.85% <0%> (+0.57%)
48% <0%> (+1%)
:arrow_up:

...ecenter/xenon/adaptors/shared/local/LocalUtil.java
33.54% <0%> (+1.29%)
18% <0%> (+3%)
:arrow_up:

.../adaptors/filesystems/webdav/WebdavFileSystem.java
67.21% <0%> (+2.73%)
32% <0%> (+1%)
:arrow_up:

... and 15 more

Continue to review full report at Codecov.

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


Update libraries

Update our libraries to their latest versions (as long as they are backwards compatible)

Closing as this is a duplicate of #355


Laser issue

When laser hit entity, laser stops, but anyway can activate catcher behind entity

Fixed in 8ef482c


ADD ACCOUNT_CONFIG_UPDATE

PR Details

Description

Related Issue

Motivation and Context

How Has This Been Tested

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the
CONTRIBUTING
document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

2021-04-09 20:35:59,784 /root/anaconda3/envs/bot1/lib/python3.8/site-packages/unicorn_fy/unicorn_fy.py unicorn_fy.py Line:557 [DEBUG   ] binance_futures_websocket 24816<PHONE_NUMBER>51936 unicorn_fy: UnicornFy->binance_websocket({"e":"ACCOUNT_CONFIG_UPDATE","T":1617971759717,"E":1617971759721,"ac":{"s":"BTCUSDT","l":19}})

Is the conversion of ACCOUNT_CONFIG_UPDATE missing?

ye,and event-margin-call

Thank you!!

there was a conflict in the merge and i am not sure if everything is as it should be.
Could you please take a final look over the merged lib?
Thanks!

I also need a raw data of margin call please!


feat(sqllab): SPA migration

SUMMARY
Part 4-8 of SQLLab SPA migration work: detail link

Change the url from /superset/sqllab to /sqllab
Update frontend routes
Update unit tests and cypress e2e tests
Update redirectSQLLab logic

For the redirect logic, it will replace the SupersetClient.post redirection by react-router redirection with the location state for the post payload data. This means the redundant form_data serialize/deserialize logic has been discarded (which is also another performance gain).
FYI: This commit does not include the delete work (i.e. old entrypoint related files, redirect legacy endpoint to new endpoint). This work will be handled in the next PR to minimize the regression rollback.
TESTING INSTRUCTIONS
Make sure the SQL Lab page works as before but in the context of SPA (does not reload the page when navigating)
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

FYI: This commit does not include the delete work (i.e. old entrypoint related files, redirect legacy endpoint to new endpoint). This work will be handled in the next PR to minimize the regression rollback.
@justinpark are you removing any old api endpoints in the next PR? If so we may want to wait until 4.0. I think there may be some integrations outside of the application that use it, so it could be a breaking change. cc @betodealmeida for the CLI that I think uses this.

FYI: This commit does not include the delete work (i.e. old entrypoint related files, redirect legacy endpoint to new endpoint). This work will be handled in the next PR to minimize the regression rollback. @justinpark are you removing any old api endpoints in the next PR? If so we may want to wait until 4.0. I think there may be some integrations outside of the application that use it, so it could be a breaking change. cc @betodealmeida for the CLI that I think uses this.

After discussed with other folk, I decided to include the delete work in the same PR. I will post the commit for the delete work soon.

Hi @eschutho. Just to add a little more context. I asked @justinpark to include the cleanup of old code in the same PR given that:

Some features might work because of old code and they shouldn't. We should validate SPA considering that the old code has no influence in the project.
The PR is big and requires a lot of testing and validation. It's good if we do this only once. If we do the cleanup work later, we'll need to repeat the same extensive test process.

Just to be clear, removing old code shouldn't mean introducing breaking changes. Let's make sure that's the case when reviewing the PR.
Once code review is completed, we'll create a test environment to make sure everything is working. It would be great if @sadpandajoe and @jinghua-qa could help us out.

@michael-s-molina I just changed all addressed issues.

Code looks good! Great to see the old obsolete code gone

/testenv up

@justinpark and @michael-s-molina This all looks good to me. I didn't see any breaking changes in the PR, as the current endpoints have all the redirects in place as @justinpark mentioned. It's only the deleting of the endpoints that would raise any concerns for breaking changes. 👍

@sadpandajoe @jinghua-qa Could you help QA test for SPA migration work?

@sadpandajoe @jinghua-qa Could you help QA test for SPA migration work?

Sure @justinpark. Bring up an ephemeral. Also what have you tested so far so we don't duplicate testing.

Sure justinpark. Bring up an ephemeral. Also what have you tested so far so we don't duplicate testing.

@sadpandajoe Ephemeral environment already set up at http://<IP_ADDRESS>:8080/.
We completed the basic page transition (from chart to sqllab. etc) so I hope you test running queries and make a bunch of page transition and page reload that make sure no data lost or error thrown.

@sadpandajoe Do you have any updates in your side?
We're planning to merge it this Wednesday if you don't have any inputs by then.

@sadpandajoe Do you have any updates in your side? We're planning to merge it this Wednesday if you don't have any inputs by then.

@justinpark I haven't had a chance to really look at it yet since I got stuck doing other things. I think things I was going to look at for testing include:

Saved queries links still work
The different navigations back to sql lab like on a chart where you can view query in sql lab
Stopping queries that are running

But probably won't have time to get to it until the later half of this week as I need to finish some other stuff. I would say merge and if I find anything I'll just create new issues and ping you on them.

I did a quick test on the ephemeral, and it's very snappy! 🙌

But probably won't have time to get to it until the later half of this week as I need to finish some other stuff. I would say merge and if I find anything I'll just create new issues and ping you on them.

Sounds good! We've completed reasonable tests so I'll take cake in the following bug fixes for your reported bugs.

@justinpark code LGTM.
Not sure if all of these are related, actually this might all be unrelated, but for reference here is what I found from manual testing and messing up with SQL Lab

The duplicate tab does not recognize tab name and goes with "Untitled". Also, when going back to previous tab the result set is emptied

https://github.com/apache/superset/assets/60598000/26d873f1-54cb-452e-8415-682eab80f4cb

When from saved queries to SQL Lab but the menu is still highlighting Saved Queries. Same happens with query history

Again, this is probably unrelated but when I have multiple tabs, if I select "Query History" in one tab it will move all the other tabs to "Query History". I think this is related to the video above. For instance, if I am viewing the definition of a table and then switch to a different query it will go empty and it will make the table definition disappear in the originating tab

https://github.com/apache/superset/assets/60598000/f8f908cc-f927-40bd-878e-c83a27d0caae

Not sure how this is happening but I found queries in Query history with no title, which will correspond to a tab named "Copy of" in SQL Lab.

Found that you can name a query with an empty space (100% unrelated)

I saw that we lost the ability to forcefully open the query in a new browser tab from Explore -> Run in SQL Lab. I think the user should still be able to choose whether to open it in a new browser tab or not. Same happens in Saved Queries but not in Query History

Thank you for all the tests and comments @geido!
@justinpark Given that this PR contains 54 changed files, I highly recommend we fix the bugs that are related to this PR, merge the PR, and then open follow-ups for bugs that already exist on master. The reason for this suggestion is to facilitate the review process given the size of the PR.

Not sure if all of these are related, actually this might all be unrelated, but for reference here is what I found from manual testing and messing up with SQL Lab

The duplicate tab does not recognize tab name and goes with "Untitled". Also, when going back to previous tab the result set is emptied

This is an existing bug and will follow up in the separate PR.

Superset.mp4
2. Wnet from saved queries to SQL Lab but the menu is still highlighting Saved Queries. Same happens with query history

I think you navigate back/forth by the browser navigation (not menu). This is same issue on existing tabs (went from Saved Queries to Query History) and will follow up in the separate PR.

Again, this is probably unrelated but when I have multiple tabs, if I select "Query History" in one tab it will move all the other tabs to "Query History". I think this is related to the video above. For instance, if I am viewing the definition of a table and then switch to a different query it will go empty and it will make the table definition disappear in the originating tab

This is how SQL Lab designed before (because activeSouthPaneTab not belong to each tab but belong to the entire sqllab). Should follow up in the separate PR.

I found queries in Query history with no title, which will correspond to a tab named "Copy of" in SQL Lab.

Same as no. 1

Realized that you can name a query with an empty space (100% unrelated)

This is an existing bug and will follow up in the separate PR.

I saw that we lost the ability to forcefully open the query in a new browser tab from Explore -> Run in SQL Lab. I think the user should still be able to choose whether to open it in a new browser tab or not. Same happens in Saved Queries but not in Query History

The basically it should support cmd + click that can open the link in a new browser tab but some links(which you mentioned) require a post data to retrieve the sql context so hard to maintain in SPA approach. I hope this is not a blocker of running SPA and want to discuss in the next thread(PR) for the support of opening new tab.

@rusackas @eschutho @geido  could you help to stamp the PR to unblock the reviewer req?

Oh, and THANK YOU for making this huge stride in the SPA effort. This is a big deal, and we're almost at the end of the tunnel! :D


plugin: scaffolder dotnet

.NET Scaffolder backend actions
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

fix: DCO

Would be great if you add this with the other actions here: https://backstage.io/docs/features/software-templates/writing-custom-actions#list-of-custom-action-packages


Additional Region Support for select locales

Provide a general summary of the feature here
We would like to support some additional locales for region support, in same way as exists for French (Canada)
Meaning....

Only added regional support, no new translations required

Example...

French Canadian

In case of French (Canada) we display translations using fr-FR (French France), but use region of Canada.

New locales requested...

es-419 (Spanish, Latin America)
es-MS (Spanish, Mexico)
en-AU (English, Australian)
en-IL (English, Israel)

🤔 Expected Behavior?

es-419: displays es-ES translations, but uses es-419 region
es-MX: displays es-MX translations, but uses es-MX region
en-AU: display en-US translations, but uses en-AU region
en-IL: display en-US translations, but uses en-IL region

😯 Current Behavior
These will all revert to en-US only
💁 Possible Solution
From looking at the code, looks like it might be just updating language tables:
https://github.com/adobe/react-spectrum/blob/main/.chromatic-fc/constants.js
https://github.com/adobe/react-spectrum/blob/main/packages/dev/docs/pages/react-aria/home/I18n.tsx
https://github.com/adobe/react-spectrum/blob/main/packages/%40internationalized/message/src/MessageDictionary.ts
https://github.com/adobe/react-spectrum/blob/main/packages/%40internationalized/string/src/LocalizedStringDictionary.ts
and some related files like storybook and tests:
https://github.com/adobe/react-spectrum/blob/main/.storybook/constants.js
https://github.com/adobe/react-spectrum/blob/main/.storybook-s2/constants.js
https://github.com/adobe/react-spectrum/blob/main/packages/%40internationalized/date/tests/DateFormatter.test.js
https://github.com/adobe/react-spectrum/blob/main/packages/%40internationalized/date/tests/queries.test.js
MessageDictinary looks to already support this functionality, with just adding language codes to support. es-419 might need extra handling as not xx-XX format.
🔦 Context
Match other Adobe areas (pls contact rgeraght, lreis any question)
💻 Examples
No response
🧢 Your Company/Team
Adobe
🕷 Tracking Issue
No response

Should en-AU: display en-BG translations, but use en-AU region?? @rgeraghty

@dannify en-AU would fall back to en-US strings (as no en-AU strings present, like en-GB falls back to en-US) but use the en-AU region (https://github.com/adobe/react-spectrum/blob/main/packages/%40internationalized/message/src/MessageDictionary.ts#L59)
We have option at any time to add strings for en-AU, en-GB, fr-CA, es-MX etc.... if feel its needed, but not required for this this issue request, which is just add additional locales to get the region support.


a cp command would be nice

Output of restic version
...

What should restic do differently? Which functionality do you think we should add?
restic already supports ls to list files but "only" restore to restore a complete path or dump to output a file.
an additional command cp would be nice (like linux or docker)
cp latest:/home/foo/bar.txt /tmp/
cp latest:/home/foo/bar.txt /tmp/foobar.txt
cp -r latest:/home/foo/ /foo/
cp -r latest:/etc/* /tmp/
...

latest could also be a snapshot-id of course

What are you trying to do? What problem would this solve?
copy files/folders from snapshots to a given folder/file.

Did restic help you today? Did it make you happy in any way?
makes me always happy because commads are easily to remember 🙂

I don't get it. Using restore you can restore individual/specific files. So what is this suggested command needed for?

better to remember
posibility to restore a single/file to a specific folder (without to create the absolute path insode the target dir)
workaround to issue with unc-paths (#2807)

current
$ /opt/restic/restic --repository-file /opt/restic/.restic_repository --password-file /opt/restic/.restic_password restore latest --target test --include /etc/passwd
repository 66fd5f9c opened (version 2, compression level auto)
restoring <Snapshot 26936de4 of [/etc /home /opt /var/spool] at 2023-01-25 01:00:01.379619277 +0100 CET by root@nuc> to test
$ find test/
test/
test/etc
test/etc/passwd

expected
$ /opt/restic/restic --repository-file /opt/restic/.restic_repository --password-file /opt/restic/.restic_password cp latest:/etc/passwd test/
repository 66fd5f9c opened (version 2, compression level auto)
$ find test/
test/
test/passwd

It would probably be better to improve existing commands instead of starting to spread similar functionality out over several commands..

imho adding new arguments like
restore latest --target test --include /etc/passwd --strip-path /etc/ --recursive
would add the functionality but is not easy to remember
maybe something like this?
restore latest:/etc/passwd /tmp/test/
restore --recursive latest:/etc/ /tmp/test/
it's the same as above only with restore as command instead of cp
maybe cp ans restore havee different usecases
restore with target-dir and includes to "restore" the complete tree (with absolute path) according to the includes
and cp to copy files/subtrees to a target dir (as the cp command on unix)

What is your --recursive supposed to do? Restoring a directory will restore it and whatever is in/under it as well.

I was thinking about the -r of the cp command
$ cp /etc/* /tmp/test/
cp: -r wurde nicht angegeben, daher wird das Verzeichnis '/etc/alternatives' ausgelassen
cp: -r wurde nicht angegeben, daher wird das Verzeichnis '/etc/audit' ausgelassen
...

this copies all files from /etc/ to /tmp/test/
one could argue that the copy of a directory should always be recursive but on th other hand having the same syntax as known commands would be nice.
but on the other other hand 🙂 -r may not work because it's already used for the repo. 🤔
to be honest I always copy files with cp x y and directories with cp -r x y because cp needs it - I've never needed to copy directories wihout -r. 🤔
so omitting -r it may be ok

This has already been discussed in https://github.com/restic/restic/issues/4027#issuecomment-1374652405 . The way forward is to add the : syntax from https://github.com/restic/restic/issues/3871 to the restore command. There is absolutely no need to introduce yet another nearly identical command.


Return the proto from Submap::ToProto instead of passing pointer.

Submap::ToProto now returns the proto.
This PR makes the interfaces for serialization more consistent: In mapping_state_serialization.cc all ToProto methods return the proto except Submap::ToProto.

@schwoere FYI: CI indicates you have more call sites to fix.

@wally-the-cartographer merge

Merge requested by authorized user wohe. Merge queue now has a length of 2.


More cache friendly storage of the World grid

The World Slots are currently stored in a flat 3D array where only moving in the X dimension visits memory adjacent to the currently loaded one, and moving in Y or Z dimensions visits possibly faraway locations.
WFC propagations are depth-first and crawl the grid topologically.
We could be more cache friendly by changing the storage layout to chunks. We could store the World in NxNxN chunks (with N being a small integer) where Slots within the same chunk are stored close to each other in memory. This will mostly help to pre-warm the cache for the current chunk and we probably won't be able to fit an entire chunk into a cache line, because the current 256-bit bit vectors are already too large (32 bytes) and only two of them fit in an x64 cache line. However, even pre-warming the cache could increase the likelihood of cache hits.
As a quick experiment, we could time the algorithm with a changed search order. Currently it first tries searching left and right, which is the most cache friendly direction. By changing this to up and down, we should see a drop in speed. If we don't, we are unlikely to see an increase in speed by changing the storage architecture.

Morton codes could help here https://en.wikipedia.org/wiki/Z-order_curve, but it looks like we'd have to make figure a few things out.
As they currently are, they can only index into squares/cubes, but not rects/boxes. We'd have to make modifications so that we don't have to densely allocate all the space they can index into, but instead snap into powers of two, e.g. if the user requests a 100x100x3 grid, we only want to allocate 128x128x4, etc.
We'd also need to see if we can distribute the bits irregularly, e.g. if we have a budget of 32 bits for one index, can we do 15+15+2?

Prototype in branch https://github.com/subdgtl/WFC/tree/morton.
Results (comparing against branch morton-compare):
ruleset: pipes
module count: 5
rule count: 39

world size: 64 64 64

old indexing: 0m5.271s
new indexing: 0m3.722s    (improvement of 29.3%)

ruleset: mondieu
module count: 46
rule count: 3288          (improvement of 10.2%)

world size: 32 32 32

old indexing: 0m10.413s
z-curve indexing:  0m9.342s

Looks promising, if we can resolve questions from above comment.


screw you gdn

not merging right now, when that mod will be on the index we will reevaluate whether this is a right move and merge/close the pr.


So sorry to reach out through here!

Unfortunately github doesn't have messages so I hope this is okay.
I am looking for anyone who has shown recent interest in PyQt5.
Me and a small group of others are working on a productivity tool using PyQt5. Our goal is to create a tool that can help others in any industry. Specifically we are creating a feature heavy productivity tool. I think this could be a good opportunity to add on to your portfolio, learn something new and work in a team setting. We need any help we can get :)
https://github.com/derto42/FlowBuddy
Please add the discord if interested: https://discord.gg/ahvpBbXG

Thank you for the offer, but I currently don't have the time to dedicate to this. Have a good day and good luck with your project!
Note: Not a problem about issue openning :)


Disable EPEL and use only RDO repos

While installing with this playbook i encountered errors when installing
test-requirements with pip. The error was related to pyparsing that had to
be updated to at least 2.2.0 for satisfying the dependencies of
python-keystoneclient (oslo.utils depends on pyparsing).
Centos provides 1.5.x version of pyparsing, instead RDO repos provides
an updated version.
Additionally using RDO repos and EPEL is discouraged, due to updates in
EPEL breaking backwards compatibility.
See: https://docs.openstack.org/ocata/install-guide-rdo/environment-packages.html#prerequisites

thanks for the PR, while testing I ran into a another problem[0] but that should be fixed in a separate patch.
[0] - https://bugs.centos.org/view.php?id=14786


lavapack,browserify,perf/deps<EMAIL_ADDRESS>

lavapack,browserify,perf/deps<EMAIL_ADDRESS>browserify/deps<EMAIL_ADDRESS>

@legobeat commitlint failed but commits look fine. It might help to rebase on main.

I tried both rebasing and merging this but it's too stale to do anything. I'm tempted to redo package.json changes manually and let all other stuff regenerate correctly. Merging this has high potential of reintroducing moved/deleted files and that kind of stuff.

These updates have all been applied via other means


in code: wait for, instead of create, pilot CRDs (as those are created when pilot is installed already (yaml))

@linsun commented on Fri Sep 29 2017
We have code in pilot to create CRDs and we also added code in istio.yaml to create these same resources via istio installation, mainly for the purpose to delete them easily during istio uninstallation via kubectl delete -f istio.yaml.   We should look into an optimized approach.
open this for discussion and resolution of this.  slack chat histories:
Lin Sun [7:58 PM]
oh i was not referring to the the template in pilot.  i’m talking about this code:  https://github.com/istio/pilot/blob/master/adapter/config/crd/client.go#L147
GitHub
istio/pilot
pilot - Istio Pilot implementation
Andra Cismaru [8:21 PM]
that code is fine, it does not require changes, @kuat can confirm
Lin Sun [8:43 PM]
i’d appreciate if @kuat can chime in.  I thought the purpose of doing the resource install via yaml is we don’t need to do it in the code.
kuat
[9:07 PM]
i don't think code change is necessary since the code won't fail if CRDs are present
Lin Sun [9:10 PM]
that was my worry, would it not fail (i assume you’ll get XX already exist warning) plus it is unnecessary code.
kuat
[9:10 PM]
@andra's definitions are derived from the programmatic creation, i think it'll be easier to keep it in sync when CRDs change
[9:11]
mixer does similar thing, we install a snapshot of 0.2 CRDs but these CRDs might change
Lin Sun
[9:21 PM]
oh, i thought the registerResource() only runs when discovery service starts…  no?   i don’t understand your comment about keep in sync when CRDs change…. is that when user edits the CRD?
Laurent Demailly [9:30 PM]
I don't think we have a good process to keep those yaml and the code in sync, something for 0.3 also agree with lin that we probably don't need to try to create the crds if we expect them to be created at install time - on the other hand it may be more resilient to create them in the code and maybe have a cleanup
----- Today September 29th, 2017 -----
Andra Cismaru [12:08 AM]
because we do not control the timeing of the application of resources, CRDs and pilot/mixer svc, we still have to create them in the code. Their presence in the yaml files is required if we want to do a clean cleanup.
Laurent Demailly [1:03 AM]
As the CRD is in the same file as pilot there is no reason (except to mask typos between the code and the file) that I can see (happy to hear one though) to create them again in the code, is there ? Just need to wait that they are all available to process the CRs (edited)
Lin Sun
[9:13 AM]
@andra were your definition in the yml file generated programmatically from pilot code?
Andra Cismaru [12:05 PM]
To recap, since it seems to have gotten lost in slack, the purpose of adding the CRDs to the yaml file was that they are cleaned during un-install, so the user does not need to run a fancy command to clean them. It was not to simplify or remove that code from pilot. They are not programmatically generated, that would be overkill for little benefit. If you look at the code more carefully, there is no error if the resource already exists. Yes, the yaml files have to be kept in sync with the code, this is true for all other resources in the yaml, secrets, configMaps etc. The worst thing that can happen in case of a typo is they would be left in the system after uninstall, not serious enough to justify a machinery to generate based on code or other complexity. (edited)
Laurent Demailly
[12:09 PM]
I think we are just asking why keep re creating in code / I don't think you answered that (my question above)
Andra Cismaru [12:15 PM]
because we do not control the timing of the start, I commented about that in the mixer channel. Imagine the CRDs creation and mixer/pilot start can be in any order .
[12:16]
this is why we still need that in the code
[12:17]
though mixer and pilot may be different here, mixer does not create them, indeed, but it has some logic to wait for their creation. It would be indeed good to re-design this part to behave the same @kuat and @mjog own this code
[12:19]
we should probably move this discussion to github, please open an issue in pilot and assign it to kuat
Lin Sun [12:27 PM]
ok the unable to control timing makes sense to me now.   @kuat do you think it is possible to delete the resource in pilot code like how the resource is created in discovery init?
[12:29]
not sure if k8s has a delete pod/deployment hookpoint
kuat [12:34 PM]
@linsun yes, but it's a programmatic call, not just kubectl command. registering/deregistering CRDs can be a multi-minute process
Laurent Demailly
[12:35 PM]
That's why I said "wait" above, not create
Lin Sun [12:38 PM]
i see, thanks @kuat!  yea, it’d be consistent if we do it either all in yaml which we need to wait, or all in code.  we are hybird now :slightly_smiling_face:

@ldemailly commented on Thu Oct 05 2017
another reason to not create crd in code: https://groups.google.com/forum/#!msg/istio-users/NrQ6iABjcqo/jlTemfoEBQAJ
cc @rkpagadala

This seems pretty stale.

I don't think anything has been done about it: https://github.com/istio/istio/blob/master/pilot/pkg/config/kube/crd/client.go#L221

CRD are now installed via explicit YAML. Installing them from Pilot isn't necessary. It's also lead to subtle bugs where the YAML and Pilot CRD definition diverge, e.g. Namespace vs. Cluster scoped RBACConfig. We should consider removing self-installing CRD from Pilot by default with --disable-install-crds=true for 1.1 release. This aligns with the longer term plan to remove direct k8s code dependencies from Pilot for 1.2+.
https://github.com/istio/istio/blob/5cf84e5e31b8425154ce524044cf5a3f05ddd67d/pilot/cmd/pilot-discovery/main.go#L115


RuntimeError: Error in faiss::FileIOReader::FileIOReader(const char*) at /project/faiss/faiss/impl/io.cpp:67: Error: 'f' failed: could not open vectorstore/db_faiss/index.faiss for reading: No such file or directory

Traceback (most recent call last):
File "/home/hdd/MLProjects/yellow-paper-physics-bot/models.py", line 61, in
f=final_result("Define Weight")
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/hdd/MLProjects/yellow-paper-physics-bot/models.py", line 57, in final_result
qa_result = qa_bot()
^^^^^^^^
File "/home/hdd/MLProjects/yellow-paper-physics-bot/models.py", line 49, in qa_bot
db = FAISS.load_local(DB_FAISS_PATH, embeddings)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/varun/.local/lib/python3.11/site-packages/langchain/vectorstores/faiss.py", line 696, in load_local
index = faiss.read_index(
^^^^^^^^^^^^^^^^^
File "/home/varun/.local/lib/python3.11/site-packages/faiss/swigfaiss_avx2.py", line 10206, in read_index
return _swigfaiss_avx2.read_index(args)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
RuntimeError: Error in faiss::FileIOReader::FileIOReader(const char) at /project/faiss/faiss/impl/io.cpp:67: Error: 'f' failed: could not open vectorstore/db_faiss/index.faiss for reading: No such file or directory

I am also getting this error

you might've mistaken the name of vectorstore as vectorstores, just check it. Close this issue, if your issue is resolved.

Yep! the issue resolved!!!


runtime: “lock ordering problem” between trace and wbufSpans

2022-05-08T17:28:26-1b86ef4/linux-amd64-staticlockranking:
37363  ======
0 : trace 28 0x8485a0
1 : wbufSpans 39 0x840770
fatal error: lock ordering problem

(attn @golang/runtime; see previously #41021)

If I understand the log correctly, this would mean that lockrank.go would not allow write barriers when an M is holding trace.lock. The call to gcWriteBarrierCX comes from https://github.com/golang/go/blob/1b86ef47de/src/runtime/trace.go#L410, setting trace.strings = nil. But that line also exists in go1.18.1, still within the section that holds trace.lock https://github.com/golang/go/blob/go1.18.1/src/runtime/trace.go#L368.
Another "missing, but valid, edge on the wbufSpans lock"?
I don't see lockRankTrace in the list for lockRankMHeap https://github.com/golang/go/blob/1b86ef47de/src/runtime/lockrank.go#L244 , but I also don't see anywhere in runtime.StopTrace that allocates .. only write barriers.

@rhysh Yeah, this just looks like a missing edge to me. The write barrier path can acquire the mheap lock in order to allocate a new write barrier buffer.


Add process for adding secrets in 1password

OpenDAL's behavior tests depend on the secrets stored in 1password. We have an internal process for adding secrets, such as sending emails to<EMAIL_ADDRESS>with the subject line Request to Add Secrets for <service>. It is better for us to document them publicly.

Hey @suyanhanx, you've handled most of the requests. Do you want to create a document in the "community/committers" section for this?

I'll add this~


sap.m.Input in a dialog is not reachable

Hello colleagues,
Describe the bug
We have an issue trying to interact with sap.m.Input located in a dialog. The control is simply cannot be found via the following selector:
const selector: wdi5Selector = {
    selector: {
      controlType: "sap.m.Input",
      viewName: "hcng.view.Projects",
      labelFor: {
        text: "Project Name"
      }
    }
  }

Tried the same approach on a different application and it works just fine. No obvious differences between control tree structure of two applications weren't noticed. Both have simple sap.m.Inputs in dialog popup. More weird we have Project Description Textarea next to the Project Name input . And similar selector works fine:
const selector: wdi5Selector = {
  selector: {
    controlType: "sap.m.TextArea",
    viewName: "hcng.view.Projects",
    labelFor: {
      text: "Project Description"
    }
  }
};

Adding searchOpenDialogs: true, forceSelect: true and idSuffix: 'inner' didn't help. The only way we have now is to use the control id.
Logs/Console Output
Error: No DOM element found using the control selector
{"viewName":"hcng.view.Projects","labelFor":{"text":"Project Name"},"searchOpenDialogs":true,"sOriginalControlType":"sap.m.Input"}

Screenshots

Runtime Env (please complete the following information):

wdi5/wdio-ui5-service-version: 1.5.2
UI5 version: 1.115.0
wdio-version (output of wdio --version): 7.8.0
node-version (output of node --version): 18.16.1
OS: Windows 11
Browser + Version: Chrome 116.0.5845

Missed an obvious detail. labelFor property was simply missing in the Label control.
Closing


The user id used to export HashRatchet is LE

The implementation and specifications are different.
The specifications say BE, but the implementation is LE.
specifications: https://daveprotocol.com/#sender-key-derivation
implementation: https://github.com/discord/libdave/blob/7b15f1fc16f159da0478aa6be909e38f1e957833/cpp/mls/session.cpp#L671

Nice catch here, this has been updated in the protocol whitepaper: https://github.com/discord/dave-protocol/commit/fc916f48b3e04403f36315673d4935bba6a39ae3


No tests for AnalysisOptionsWarningCode.INCLUDED_FILE_WARNING

There are no tests for this warning.

https://dart-review.googlesource.com/c/sdk/+/278906