move FixedSizeList to array_fixed_size_list.rs

Signed-off-by: remzi<EMAIL_ADDRESS>Which issue does this PR close?

None. This is a subtask of #2217.
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Codecov Report

Merging #2250 (d98b2d4) into master (f41fb1c) will decrease coverage by 0.02%.
The diff coverage is 92.91%.

@@            Coverage Diff             @@
##           master    #2250      +/-   ##
==========================================
- Coverage   82.30%   82.27%   -0.03%
==========================================
  Files         241      244       +3
  Lines       62437    62443       +6
==========================================
- Hits        51389    51378      -11
- Misses      11048    11065      +17

Impacted Files
Coverage Δ

arrow/src/array/array_list.rs
95.83% <ø> (+0.72%)
:arrow_up:

arrow/src/array/mod.rs
100.00% <ø> (ø)

arrow/src/array/array_fixed_size_list.rs
92.91% <92.91%> (ø)

...rquet/src/arrow/record_reader/definition_levels.rs
84.81% <0.00%> (-2.54%)
:arrow_down:

arrow/src/array/builder/boolean_buffer_builder.rs
97.05% <0.00%> (-0.99%)
:arrow_down:

parquet/src/encodings/rle.rs
91.91% <0.00%> (-0.52%)
:arrow_down:

arrow/src/datatypes/datatype.rs
62.30% <0.00%> (-0.40%)
:arrow_down:

parquet/src/util/bit_util.rs
93.92% <0.00%> (-0.28%)
:arrow_down:

arrow/src/array/builder/decimal_builder.rs
86.09% <0.00%> (-0.19%)
:arrow_down:

parquet/src/encodings/encoding/dict_encoder.rs
91.22% <0.00%> (-0.16%)
:arrow_down:

... and 10 more

Help us with your feedback. Take ten seconds to tell us how you rate us.


MarkdownDB README with features, motivation, brief tutorial

New user: When coming to MarkdownDB I want to know (briefly) what it is, what features it has (existing or future) and a brief tutorial on use.
We could port material from https://datahub.io/notes/markdowndb especially the job stories (they could become features section).

Note one clarification re https://datahub.io/notes/markdowndb is that there is an ambiguity in use between markdowndb as the database itself and markdowndb as a library (index and API) for accessing a markdowndb. Our package is the latter and so for us the description of "markdowndb" is something like: "A javascript library for treating a directory of markdown files as a database" (or a better version thereof).

Emphasize: fine if this is a rough cut, we will rapidly iterate. better to get something down ...
Also find starting from a tutorial is something easier (a short example is worth a thousand descriptions) eg.
On disk you have:

```
my-blog-folder-of-markdown-files/
  blog-1.md
  blog-2.md
```

Then do ...

```
import db from markdowndb

db.create('path-to-folder')
db.query('all blogs written by x author'_
```

Behind the scenes we are creating an SQL(ite) database so you can do everything sql can! Here are some examples ...

etc

Acceptance

[x] Brief description of what markdowndb is and why you might use it (e.g. why we will use it in Flowershow etc)
[x] Brief description (e.g. bullet list) of features existing or forthcoming (emojis are a bonus!)
[x] Quick start tutorial ...

@olayway can you update acceptance with links to outputs


xcwd: 2019-05-09 -> 1.0

Description of changes

I updated xcwd from 2019-05-09 to 1.0. Note that 1.0 is the first release version of xcwd.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@ofborg build xcwd xcwd.passthru.tests


⚠ Bofrost.de has degraded performance

In 1450d44, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 878 ms

Resolved: Bofrost.de performance has improved in af564f6.


fix: security issues

Please describe the Improvement and/or Feature Request

Scope (please mark with X where applicable)

New Functionality              [ ]
Install                        [ ]
Gateway API - GatewayClass     [ ]
Gateway API - Gateway          [ ]
Gateway API - HTTPRoute        [ ]
Gateway API - TLSRoute         [ ]
Gateway API - TCPRoute         [ ]
Gateway API - UDPRoute         [ ]
Gateway API - PolicyReference  [ ]
Ingress                        [ ]
Egress                         [ ]
MultiCluster                   [ ]
PIPY Control Plane             [ ]
PIPY Script/Config             [ ]
CLI Tool                       [ ]
Metrics                        [ ]
Certificate Management         [ ]
Sidecar Injection              [ ]
Logging                        [ ]
Debugging                      [ ]
Tests                          [ ]
Demo                           [ ]
CI System                      [ ]
Project Release                [ ]

Possible use cases

/close


In the connect dataset modal when creating a map scrollbar is shown but not needed

https://github.com/CartoDB/cartodb/commit/93ed4da3070b7124f1e5e605242868a242be4cfe


BUG:  Horizontal scroll issue

Describe the bug
On horizontal scroll, it becomes stuck sometimes and then resumes to scroll again, attaching a video of the same
https://user-images.githubusercontent.com/40525668/180362825-0c2e21ab-a77f-4839-9e7e-167f5ec7f863.mov
Expected behavior
Should scroll smoothly

This is a trade-off issue between loading less data (UI responding faster, but possible blank spaces) vs loading more data (UI slower, less blank spaces):

When the view is near the end to the right the component loads and render more weeks to the right. This process takes time
If the user scrolls too often/fast, you see the behavior you're describing
We currently try to optimize this with some of these FlatList props. You can check the props pros and cons for the trade-off. For example we set windowSize=5 (5 weeks = 2 to the left + 1 central + 2 to the right)

Idea: we could expose props windowSize, updateCellsBatchingPeriod, maxToRenderPerBatch, so the user can tune them better. What do you think?

@pdpino  Just thinking out loud, If we give a window size as you mentioned, will it work well with goToDate function, because I've observed a similar behaviour when the user is moving to a specific date.

will it work well with goToDate

you are right, it won't work well. See this similar issue: #207 when navigating to a far date (or even to close dates)
We are trying to solve that separately from the flatlist optimizations. The process when swiping with the finger (moving one page by one) is slightly different than goToDate (navigating to an arbitrary date)

@ Got it, Sure then we can try to add widow size, Since you mentioned we already are discussing this already, are you planning to take this up?

@TheTushar2696 I sent a PR exposing the props. You can try different values to get better results, but I cannot assure you there will be better results
Some other comments:

I'm implementing other optimizations to be able to implement some other features. These optimizations could help
I think a better solution would be to block the scrolling for the user while the component is loading. I want to add this, but I need to do (1) first

Hi @pdpino,
Got it, Also one more thing I noticed is when I am populating large number of events it is showing this in console, any idea about it?

@TheTushar2696 I haven't seen that one with week-view
This mentions passing large objects to console log


Wishlist: !!/test-scan or !!/scan-test - non-reporting test of a post

Is your feature request related to a problem? Please describe.
When you !!/scan a message, it will also be reported if it triggers a reason. If that's not what you want, there is no way really to test whether the post will trigger, short of copying it into a local JSON representation and testing locally (which is harder than it needs to be, because different parts of Smoke Detector look at different representations of posts).
Describe the solution you'd like
It would be nice if there was an option to !!/scan, like the opposite of !!/scan-force, where you only want to get a report of the result.
In some ways, this is also similar to the !!/test command, so I'm thinking there should probably be an alias with that prefix.
Describe alternatives you've considered
I wanted to test a new expression I watched, but as outlined above, the only way to really know if it works as intended is to scan an actual post which contains the target phrase. Similarly, the only way to check against false positives is to scan posts which are not supposed to match (typically then posts which exhibit a corner case which you want to test you have covered adequately).
You can add a test case to tests/test_findspam.py but this again hinges on getting the representation of the post right in the test case. If you guess wrong what the API actually returns for the post you are targeting, the test proves nothing.
Additional context
https://chat.stackexchange.com/transcript/message/61838114#61838114 where I rescanned the wrong post, adding insult to injury.

hola!
please add hacktoberfest tag in the "Labels".

I would love to contribute to this.

We will be happy to add #hacktoberfest-approved to any PR we accept during October. Maybe a WIP pull request would be a good start, to let us comment early on your approach. If you need help setting up a testing instance, probably approach us in the chat room (you'll need a bit of reputation on one of the Stack Exchange sites to be able to participate in chat, but you can achieve that over a few days by submitting good suggested edits to old posts).


[BUG] BrowserType.name() returns the wrong name when channel is used

System info

Playwright Version: [v1.XX] 1.40
Operating System: [All, Windows 11, Ubuntu 20, macOS 13.2, etc.] mac
Browser: [All, Chromium, Firefox, WebKit] all
Other info:

Source code
Browser browser = playwright.chromium().launch(new BrowserType.LaunchOptions().setChannel("chrome"));
assertEquals(browser.browserType().name(), "chrome")

//org.opentest4j.AssertionFailedError:
//Expected :chrome
//Actual   :chromium

[x] I provided exact source code that allows reproducing the issue locally.

Expected
Expect BrowserType.name() to return the channel that was used.
Actual
returns the BrowserType that was used to launch ("chromium")

This is expected as per here. The underlying BrowserType did not change. You can manually store the settings you pass a your launch options to get the channel later on in your execution.


Add switch config messages

As per https://github.com/LIFX/public-protocol/blob/main/protocol.yml#L472-L541, LIFX supports some other messages which aren't documented in their API to do with getting and setting switch properties.
The most interesting here is setting the backlight color of the switch.
Addresses: https://github.com/frawau/aiolifx/issues/74

Will look into this this WE.


Duration is wrong

The formula in index.js line 102 is wrong
const genDuration = showDuration
? > Duration: ${Math.round(duration / 60)}s \n
: "";
The value from the json is displayed in milliseconds, so to end up with seconds, divide by 1000 instead

Great catch - how I could miss that...

Resolved: package updated also on npm, update package if needed :)
Thank you for your report.


ARA Source provenance standards -- July 7

In this ticket: https://github.com/NCATSTranslator/TranslatorArchitecture/issues/48
KPs will return to ARAs edges with attributes related to source provenance.
ARAS now need to return this information, along with adding to the edges  attributes describing the edge's return from the ARA.
If the ARA adds edges from other sources (such as NGD or other lit co-occurrence or calculated edges), then provenance for these new edges should also be added.
The details of what these properties look like is/will be described in the KP issue linked above.

Guide to implementation: https://docs.google.com/document/d/177sOmjTueIK4XKJ0GjxsARg909CaU71tReIehAp5DDo/edit#


Cannot find the paper Rethinking the Elementary function fusion for Single-Image Dehazing in the arxiv or other related  websites

Cannot find the paper Rethinking the Elementary function fusion for Single-Image Dehazing in the arxiv or other related  websites

This paper will release soon

I have already submitted my paper to arXiv, but its status is currently on hold and it may take some more time for it to be processed.

Now, the paper can be viewed at https://arxiv.org/abs/2405.15817


Tab zoom is breaking Minimal theme css classes

I don't know if this is limited to Minimal theme only.
With new VT tab zoom version, cssclasses are over-ridden/ignored:

With VT disabled (or with VT 0.10 reinstalled):

"Table-wide" allows the table width to extend beyond the normal Readable Line Length setting (table-small sets a smaller font in the table).
I've reverted to v0.10 for now.

Thank you! I'll look into it ASAP.

Hi @Lotus49b ! After some inspections, it seems that Minimal theme does not follow Obsidian's default design and style structures. Since there is no CSS class or anchor for identifying the theme, I cannot apply patches to this specific theme. Or otherwise it will break other themes. Please consider opening a Plugin Support issue on their side. Thank you.
In the next release, tab zoom will only be applied if necessary. So, if you don't enable tab zoom, everything will work as usual.

Great, thanks!


Type of $default can be more than string

The docblock defines the second param $default to be string. But actually it can be null as well. By the way, couldn't that param be of any type, i.e. mixed?
https://github.com/slimphp/Slim-Http/blob/473a81173018e19e18c78e31d221e51973a6d199/src/ServerRequest.php#L530-L555

In what case could that param be anything but a string?

I guess I thought so, because the @return is mixed. Is it possible, that $default might be an array too?

I think it should return a string only. Parsed $_POST or $_GET params should always be strings.. unless there's some weird case where the value is overwritten by the user to be something else which would be weird.

Then this issue can be closed.


consider removing some association scopes added to the model

some association scopes defined for timescale tables have generic names, especially jobs, which can conflict with existing domain entity
I think that leaving only the hypertable method (which returns model that already has all the necessary scopes) will allow to avoid such conflicts and won't pollute already polluted model namespace
the method responsible for it is Timescaledb::ActsAsHypertable::Core::ClassMethods#define_association_scopes:
https://github.com/jonatas/timescaledb/blob/72041cd9426f687c0ac052169320a3ff57842de4/lib/timescaledb/acts_as_hypertable/core.rb#L17

Very good point! I think we can make it alternative and build a keyword for the acts_as_hypertable macro to include scopes and associations on the model.
I'll be thinking about it and come with a solution! Thanks for the feedback! I started guessing that it was useful but probably it can be useful only in certain scenarios.


v10.0 requests failing

I'm seeing the tag sending requests to https://graph.facebook.com/v10.0/[...]/events . These are returning a 400 response with the error message:
OAuth "Facebook Platform" "invalid_request" "(#2635) You are calling a deprecated version of the Ads API. Please update to the latest version: v11.0."
I've tried adding a fresh tag, same result. I know the template has been updated for v11 so I'm sure I shouldn't be seeing this problem.
Context: I'm sending events to the GA4 Measurement Protocol client. They're being received and triggering the FB tag. The body being sent on my requests to FB looks to be valid, carrying all the parameters I need and expect.
Grateful for any ideas, thanks.

Try to publish changes on server container if you don't.
Also as an option, you can try my FB template https://github.com/stape-io/facebook-tag

Great, thank you. I assume it was because I published the container, but the GTM interface showed that there were template updates available. The updated version is working, as expected.
Does anyone know if it's possible to force a check for new template updates? And/or can you recommend the best way to be sure an update isn't missed? Perhaps just watch the repo?


Clean up Hero file

Fixes #69

Checks:

[x]   the commented code are gone

[x] the unused code is removed

[x] the unused imports are removed

[x] tests still pass yarn test

Hi @VishalMyla! Thank you for your PR. This looks ready to merge! 🚀
One last thing before I can do so could you please sign your commits so your contribution gets recorded for #hacktoberfest like?! See this https://github.com/movesthatmatter/movex/blob/main/CONTRIBUTING.md#before-you-contribute

Hey @VishalMyla, did you have a chance to look into the commit signing process, so I can merge this?
If you don't sign it github won't be able to recognize your contribution for hacktoberfest :(

Hey @VishalMyla I'm going to merge this!
One last thing, could you give the repo a star when get a chance so we can bump it up in the hacktoberfest rankings! 🙌🙏


Update Hadoop version for Spark 1.6.0 (different Hadoop versions)

This PR simply modifies Hadoop versions for Spark 1.6.0.

Current coverage is 89.62%

Merging #79 into master will not affect coverage as of 70c25fc

@@            master     #79   diff @@
======================================
  Files           14      14
  Stmts          636     636
  Branches        85      85
  Methods          0       0
======================================
  Hit            570     570
  Partial          0       0
  Missed          66      66

Review entire Coverage Diff as of 70c25fc

Powered by Codecov. Updated on successful CI builds.

I am going to merge this.

Merged.


Infer Type issue when dtype passed is None

Description
 data = mx.sym.var("data")
 data2 = mx.sym.var("data2")
 out = mx.sym.elemwise_add(data, data2)
 arg_types, out_types, aux_types = out.infer_type_partial(data=None, data2=np.float32)

This will fail, because of this line : https://github.com/apache/incubator-mxnet/blob/master/python/mxnet/symbol/symbol.py#L1006
Makes it inconsistent to the documentation which says :
Unknown type can be marked as None.

https://github.com/apache/incubator-mxnet/blob/master/python/mxnet/symbol/symbol.py#L896
Also, there is no way to pass unknown types to infer_type_partial because of this bug.
This is because:
>>> np.dtype(None)
dtype('float64')

https://github.com/apache/incubator-mxnet/blob/master/python/mxnet/symbol/symbol.py#L1006
The fix should be simple, but this issue has always been there assuming that np.dype(None) to be converted to float64 was always the case. Will there be users depending on this buggy behavior and are we okay with breaking these user scripts ?
@ptrendx

@lanking520 assign @anirudh2290

un-assigning myself since  i am not working on this currently. if someone is interested, please go ahead.


Code cleanup for Kubernetes Client

org.apache.spark.deploy.kubernetes.Client is just over 800 lines long. Find a way to break it down into multiple classes that each handle a different aspect of submitting to Kubernetes. One aspect could be the watches and readiness, for example. Another could be configuring SSL.

@mccheah Do we want to do this before we cut the alpha release?

Don't think so - would be a pretty significant change. Let's release what we have as the alpha (or something close to it) and we can refactor + harden it after the fact.

Agreed. Marking this as beta.

Moving to alpha since we'd like this to get #110 into alpha as well.

This item is done I think
@mccheah

Sort of - I'm going to close this for now, it's a little vague and we can break it down more granularly, and the different cleanups can be done over time. This has everything I eventually want though, at least from my current perspective: https://github.com/apache-spark-on-k8s/spark/pull/184


Content pages created & v4

@vikkihowe Hey, This is the new manual content pages and v4 of Minimum functionalities

Looks good to me :)


Future

Disclaimer, my bad this is a huge PR and the CLI and remote function stuff got mangled together.
The goal of this PR is to get feedback and add basic remote function support to bolt python. This contribution only includes support for synchronous architecture and does not include the related documentation.
This also include very basic support for the slackcli along with its tests.
Here are the modifications to the python template I've done to use these changes
overview
cli stuff

slack_bolt/cli
tests/slack_bolt/cli

function stuff

Every file affected by changes excluding cli stuff

Testing
Test these changes manually
Setup
you will need a file structure that resembles:
.
├── bolt-python-starter-future         # https://github.com/slack-samples/bolt-python-starter-template/tree/future
└── bolt-python-future-fork            # https://github.com/WilliamBergamin/bolt-python/tree/future

helpful commands
git clone https://github.com/slack-samples/bolt-python-starter-template/tree/future
git clone https://github.com/WilliamBergamin/bolt-python/tree/future

Build
Bolt

Navigate to your bolt-python-future-fork folder
set up a python envbrew install pyenv
pyenv install 3.9.6
python -m venv env_3.9.6
source env_3.9.6/bin/activate

build the package using the script bash scripts/build_pypi_package.sh

application

Navigate to your bolt-python-starter-future folder
Set up a pyenv follow the same steps as Bolt step 2
ensure that this dependency is present in the requirements.txt file ../bolt-python-future-fork/dist/slack_bolt-1.14.3-py2.py3-none-any.whl

this tells pip where your local build of python-bolt is located
install the dependencies pip install -r requirements.txt

Run

With slack cli (else you need to install the app traditionally) slack install
 slack run

Add the app Bolt Template App (dev) to a channel in your workspace
Find and trigger the Reverse shortcut to start the function

Feed back
I would like to figure out if these changes yield the proper pythonic user experience.
example: Is passing complete_error and complete_success a correct python way of resolving a function
Category (place an x in each of the [ ])

[x] slack_bolt.App and/or its core components
[ ] slack_bolt.async_app.AsyncApp and/or its core components
[ ] Adapters in slack_bolt.adapter
[ ] Document pages under /docs
[ ] Others

Requirements (place an x in each [ ])
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've run ./scripts/install_all_and_run_tests.sh after making the changes.

Thanks for the feedback @srajiang, Ive added this information at the top, let me know if you have any issues setting up project locally or want more info 😄

Thanks for dropping in some of your wisdom @srajiang !!

get-manifest supports finding a manifest file outside of the project root. This is something I put in place for JS / TS because of the variety of ways you might set up your project folder (including in a src folder). Curious what your thoughts whether support for this is needed in Python.

This feature can be helpful for the user 💯  it is definitely something we can implement in python I just want to understand the scope better, should the script accept a folder path in an env variable (like SLACK_CLI_CUSTOM_FILE_PATH) or as a parameter? and when you mention including in a src folder this means searching through the subdirectory to find the file?

start supports customization of app entry-point path at the project level. I noticed you are honoring a potential SLACK_CLI_CUSTOM_FILE_PATH passed down through env (to date, CLI doesn't send this and I'm kind of on the side of the fence of removing this support and letting the dev set this at the project level). I'm curious is that something Python devs would expect to be able to do? In node, for example, we let them set their entrypoint via the package.json and the hook honors this entrypoint.

Maybe @seratch can peer review me on this, from my understanding pip is a simple dependency manager, unlike npm, gradle or maven, pip or python do not have knowledge surrounding the project. There isn't an equivalent to package.json file builtin to python. Thus the user needs to somehow provide the entrypoint file to execute the process, using env variable like SLACK_CLI_CUSTOM_FILE_PATH is a simple and scalable way to pass values to the script. But we can use other approaches.

This feature can be helpful for the user 💯 it is definitely something we can implement in python, I just want to understand the scope better, should the script accept a folder path in an env variable (like SLACK_CLI_CUSTOM_FILE_PATH) or as a parameter? and when you mention including in a src folder this means searching through the subdirectory to find the file?

There are different options for doing this that aren't mutually exclusive.
One way is to require developers to supply the manifest path as environment variables (in that case, perhaps it doesn't make sense to use SLACK_CLI_CUSTOM_FILE_PATH, since SLACK_CLI_* preface indicates it's being passed along from the CLI context). You'd want to name it something different I think.
AND / OR you could make the hook do a little more by doing a search through the project and looking for the manifest. That's what I've opted for with Bolt JS, mostly just to avoid giving developers another hurdle in setup. The thinking here is that having a manifest file in your project in the first place is a CLI / 2.0 requirement. If you only had the env variable supported then it'd become a requirement for the dev to set that up first thing. I think that's less of a good DX for a first time user.
Re: entrypoint path in the env That's something I do think makes sense for devs to want to configure or setup based on other needs.

@srajiang
As @WilliamBergamin mentioned, pip is just a dependency resolver. This does not manage anything about your Python project. Also, even when you manage your Python project using more sophisticated tools such as Poetry, those tools are only interested in which Python runtime is used and where to load source files.
As long as the CLI-related paths never affect where to place source files and which runtime to use, you can use any paths for CLI regardless of how to load the paths (env variables or whatever).

source files and which runtime to use, you can use any paths for CLI regardless of how to load the paths

We shouldn't be using SLACK_CLI_* preface for variables that the developer is expected to set, since that is the prefix the CLI uses when it sets environment variables.

@srajiang Ive modified the get_manifest hook to search in subfolders of the working directory for the manifest.json, One important thing to address is that I am attempting to ignore certain Folders (hidden folders, generated folders, dependencies, tests)
This is the new file in question let me know what you think of this solution
As for the SLACK_CLI_* env variables I was under the impression these were set by the cli before executing the hook, I will take this off line to get more info.


Broken links

"Related" section is broken.
Relevant URLs
https://docs.influxdata.com/influxdb/cloud/reference/flux/stdlib/built-in/transformations/keyvalues/
https://docs.influxdata.com/influxdb/v2.0/reference/flux/stdlib/built-in/transformations/keyvalues/
BTW: Finding this I was looking for a method to query with a filter like
Give me _value of _field == A if _value < 20 of _field == B
I've done it with a join but it's very inefficient and takes forever with big sets of data.

Thanks for reporting this @cgfoed. I'm looking into fixing the related links.
In regards to the query you're after, you need to query both field A and B, and then pivot the results so a column for each field exists in each row. You can then use the kind of logic you're hoping to use:
from(bucket: "example-bucket")
  |> range(start: -1d)
  |> filter(fn: (r) => r._field == "A" or r._field == "B")
  |> pivot(rowKey:["_time"], columnKey: ["_field"], valueColumn: "_value")
  |> filter(fn: (r) => r.B < 20)
  |> rename(columns: {A: "_value"})

Thanks for reporting this @cgfoed. I'm looking into fixing the related links.
In regards to the query you're after, you need to query both field A and B, and then pivot the results so a column for each field exists in each row. You can then use the kind of logic you're hoping to use:
from(bucket: "example-bucket")
  |> range(start: -1d)
  |> filter(fn: (r) => r._field == "A" or r._field == "B")
  |> pivot(rowKey:["_time"], columnKey: ["_field"], valueColumn: "_value")
  |> filter(fn: (r) => r.B < 20)
  |> rename(columns: {A: "_value"})


Docs add dependencies tab option

Coverage decreased (-0.04%) to 99.956% when pulling 80b863919b8792512e69bd2f561f04eee8535cdc on pavolloffay:doc-disable-dep-tab into 77fd3d32d9f1072307261f80339eb1bfbc26cd6e on jaegertracing:master.

Coverage remained the same at 100.0% when pulling d97ba5ef97e75826b790afd6019e030394dda268 on pavolloffay:doc-disable-dep-tab into 972b7957fee24294458596039ecc4b784ac39fd1 on jaegertracing:master.

Since this is not code submission, I am going to merge without the DCO.


Add pco_python to workspace

Closes #128
This is incomplete, but I'm opening the PR to get some help and feedback.
What I did

created a new package called pco_python and added it to the workspace
the package is set up according the Maturin pure Rust project recommended layout
wrote a very naive wrapper which satisfies the numcodecs interface (see discussion in #128)
blindly added the Maturin github action

What's working
I can build the package using maturin and run some python tests to to verify it works. :tada:
maturin develop -m pco_python/Cargo.toml
pytest -v

What's not working

cargo build fails. I don't understand how to properly integrate the maturin build process within the workspace. There are some suggestions online that this is not possible, e.g. https://github.com/PyO3/maturin/issues/291#issuecomment-1305018097. Overall, there doesn't seem to be a clearly documented best practice for how to lay out a project this way.
The github action is untested and almost certainly will fail

There are also numerous potential improvements to be made to the code itself. I'd recommend we get the packaging and CI sorted out, then address that later.

Thanks for getting this started - I'll take a closer look some time in the next few days

I think this was the reason you couldn't build (same as me, a Mac/Maturin thing): https://stackoverflow.com/a/77382609/6100177. Rather unfortunate that this is necessary.

Not to derail my own PR, but I just recently discovered the cramjam project. It looks like the goal is to bring a bunch of compression algorithms together all in one place and making them accessible via both Rust and Python.
Before putting more work in here, perhaps worth considering whether it would make more sense to add pco do that project.
AFAICT the main blocker seems to be that those codecs all operate on raw bytes, rather than typed numerical arrays.

@rabernat Let's not use cramjam and stick with the original idea:

It seems unlikely pco would fit into its API easily
I think there are pco configurations and low-level API stuff I'll eventually want to expose in FFI, and those definitely won't fit into a general-purpose wrapper
cramjam has even less adoption than pco itself (which I have hardly even mentioned to the outside world, announcements to come soon)

Sounds good! I'll carry on with this then.
As you can probably tell, this is currently my "nights and weekends" project to learn Rust a bit better. 😆 I'm looking forward to getting back to it over the holidays. In the meantime, don't hesitate to push changes yourself if you want to move faster.

Thanks for your work, @rabernat !

Seems like I got the workflows right! We now have https://pypi.org/project/pcodec/

This is great! Thanks Martin for getting this over the finish line!


Use LLD to link programs

We can now use LLD to link Cortex-M programs!. This change moves over to using lld in .cargo/config. I haven't tested this with any hardware yet so don't merge just yet.

@therealprof this is working fine for me, can you give it a test and merge in if it all looks good?

Updated from master, merge at your leisure :)


TODOs for rootless mode

ref: https://github.com/containerd/containerd/issues/2246 https://github.com/containerd/containerd/pull/2006
TLDR
On runc side, we only need  https://github.com/opencontainers/runc/pull/1688 to be merged. Other runc-side PRs are optional.
On containerd side, we only need to add a helper function to oci/spec_opts_unix.go (https://github.com/containerd/containerd/pull/2006).
TODOs on runc side
Mandatory:

[ ] https://github.com/opencontainers/runc/pull/1688: main: support rootless mode in userns

Optional:

[ ] https://github.com/opencontainers/runc/pull/1692: rootless: optional support for generating config with subuid map
[ ] https://github.com/opencontainers/runc/pull/1693: libcontainer: allow setgroup in rootless mode
[ ] https://github.com/opencontainers/runc/pull/1759: rootless: cgroup: treat EROFS as a skippable error  rootless-containers

TODOs on containerd side
No change is likely to be needed on the daemon part (except CRI plugin).
Client part:

[ ] Add WithRootless() to oci/spec_opts_unix.go, which converts the runtime config via libcontainer/specconv.ToRootless().  https://github.com/containerd/containerd/pull/2006/files
[ ] If we fail to get  https://github.com/opencontainers/runc/pull/1759 merged, we would need to set CgroupsPath to an empty value in WithRootless() depending on the permission configuration.
[ ] Add documentation. https://twitter.com/AkihiroSuda/status/953231819008180224 https://github.com/AkihiroSuda/containerd/blob/demo-rootless.20180116-1/docs/rootless.md

How it is expected to work

The user needs to unshare an user namespace and a mount namespace (unshare -U -m), and configure /proc/PID/[ug]id_map via new[ug]idmap.

If depending on new[ug]idmap is problematic, the user could use some (slow/unstable) ptrace hack to emulate uid/gid. https://github.com/AkihiroSuda/runrootless

To unshare a dedicated network namespace with NAT, the user would need to use either a SUID binary (e.g. lxc-user-net) or some usermode NAT stack with TAP (e.g. slirp: https://github.com/AkihiroSuda/runrootless/tree/master/misc/vde)

The user can launch containerd after nsenter-ing to these newly unshared namespaces.

I do not like the idea of a user required to create the user/mount namespace, this is not scalable in a CI pipeline, I would like the tool to do that. I an open to suggestions/reasons why that is required.

Let me clarify the question, why can't containerd/img/buildkit or whatever the tool used create the namespace, am I missing something here? :thinking:

mount(2) requires mapped root in userns (and mountns of course).
http://man7.org/linux/man-pages/man1/unshare.1.html

It should be noted that unsharing namespaces wouldn't necessarily be necessary with umoci-like emulation of the relevant image unpacking code (which would reduce the need for the rootless-in-rootless runc changes that you've linked.
But of course I totally understand that containerd doesn't want to have to deal with all of the fun that comes with emulating VFS in userspace as an unprivileged user. I might be one of the few people crazy enough to use that as a solution to the privilege problem. :wink:

@cyphar The naive snapshotter should already support "the magic", if you will. Image changesets can be calculated from arbitrary directories.
@AkihiroSuda We also may need to make some changes in the differ. Specifically, it takes mounts that need to be performed by root. For the naive snapshotter, these mounts don't actually need to be performed, so that snapshotter can emit mounts that are really just filesystem paths.

We also may need to make some changes in the differ. Specifically, it takes mounts that need to be performed by root.

This is covered by unshare -m -U (and uid_map).

@cyphar I'm not sure what you mean. It looks like it changes the permissions and then reapplies them (based on https://github.com/openSUSE/umoci/blob/master/pkg/unpriv/unpriv.go#L34). If there is missing functionality in continuity, please let us know, but that is not where the implementation is. I would think this would belong in snapshotters.
However, that concerns me, because if that implementation is interrupted, the metadata will get corrupted.

If there is missing functionality in continuity, please let us know, but that is not where the implementation is. I would think this would belong in snapshotters.

I spent quite a while trying to figure out where the actual code that does snapshotting resides, and I thought I figured out that it was in github.com/containerd/continuity/fs. Am I mistaken about that? Is it somewhere else? I was finding it a bit hard to go through the different levels of interfaces since I'm not familiar with them.

However, that concerns me, because if that implementation is interrupted, the metadata will get corrupted.

This is true, but there is currently no other way of doing it without using user namespaces (which causes the sorts of issues you've seen here -- not to mention problems with older kernels or distros that disable CLONE_NEWUSER). If you're running as a privileged user (with CAP_DAC_OVERRIDE) you don't need to do those tricks (and umoci doesn't). Also it's implemented it so that it's only done if absolutely necessary (which is rarely the case).
My point was that if you wanted to make unprivileged containerd work without unshare -U you would need to implement something like the unpriv package I pointed to (or just use it since it already exists) in order for snapshots to work for most distribution images. But since the plan is to use unshare -U it shouldn't really matter.

Reopened https://github.com/containerd/containerd/pull/2006

@AkihiroSuda any progress?

Still waiting for the PR to be merged.

PR: https://github.com/containerd/containerd/pull/2766


Change default data amount to 1200 points- about 3 years.

I think we should merge this first, to close the issue, rather than be dependent on bitflux adding features. This can be customized later as a separate feature.
Performance is good (can't see a difference).

Would it be difficult for you to make a change to BitFlux to add this to their API and PR that in? @jleft

I think expanding API would be ideal. This could go together with the API for setting and getting indicators.

We have an open PR to add this to the BitFlux API, it's a fairly straightforward change to make.
https://github.com/ScottLogic/BitFlux/pull/660

Given the PR for BitFlux has now been merged, and this PR changes our version to use the latest dev anyway, can you re-work this PR to use the BitFlux repo and access the new API method?

This will be reworked using new api


integ-security-k8s tests very flaky

https://testgrid.k8s.io/istio-istio-postsubmit#integ-security-k8s-tests_istio_postsubmit&exclude-non-failed-tests=&width=20
We can see failure rate is ~75%

Some more examples:
https://prow.k8s.io/view/gcs/istio-prow/logs/integ-k8s-112_istio_postsubmit/71
https://prow.k8s.io/view/gcs/istio-prow/logs/integ-security-k8s-tests_istio_postsubmit/87
https://prow.k8s.io/view/gcs/istio-prow/logs/integ-security-k8s-tests_istio_postsubmit/76

Some more examples:
https://prow.k8s.io/view/gcs/istio-prow/logs/integ-k8s-112_istio_postsubmit/71
https://prow.k8s.io/view/gcs/istio-prow/logs/integ-security-k8s-tests_istio_postsubmit/87
https://prow.k8s.io/view/gcs/istio-prow/logs/integ-security-k8s-tests_istio_postsubmit/76

From https://testgrid.k8s.io/istio-istio-postsubmit#integ-security-k8s-tests_istio_postsubmit&exclude-non-failed-tests=&width=20, it seems the JWT and RBAC test failed only once on 09/19 and 09/18 which doesn't seem flaky.
The sds_ingress/multiple_mtls_gateway.TestMtlsGateways seems failing more frequently. cc @JimmyCYJ who added those tests.

If RBAC fails once in production I think a user would be pretty disappointed :slightly_smiling_face:. The reason I am concerned is these tests used to pass 100% of the time -- I want to make sure we don't have a real regression which bypasses RBAC.
Here are some more failures:
https://prow.k8s.io/view/gcs/istio-prow/pr-logs/directory/integ-security-k8s-tests_istio/647
https://prow.k8s.io/view/gcs/istio-prow/pr-logs/directory/integ-security-k8s-tests_istio/723
https://prow.k8s.io/view/gcs/istio-prow/pr-logs/directory/integ-security-k8s-tests_istio/749
https://prow.k8s.io/view/gcs/istio-prow/pr-logs/directory/integ-security-k8s-tests_istio/750

If RBAC fails once in production I think a user would be pretty disappointed . The reason I am concerned is these tests used to pass 100% of the time -- I want to make sure we don't have a real regression which bypasses RBAC.
Here are some more failures:
https://prow.k8s.io/view/gcs/istio-prow/pr-logs/directory/integ-security-k8s-tests_istio/647
https://prow.k8s.io/view/gcs/istio-prow/pr-logs/directory/integ-security-k8s-tests_istio/723
https://prow.k8s.io/view/gcs/istio-prow/pr-logs/directory/integ-security-k8s-tests_istio/749
https://prow.k8s.io/view/gcs/istio-prow/pr-logs/directory/integ-security-k8s-tests_istio/750

Are these in pre-submit? For the RBAC test case that failed in post-submit, it looks more like some transient error in Envoy or the testing infrastructure,

https://prow.istio.io/view/gcs/istio-prow/pr-logs/pull/istio_istio/17515/integ-security-k8s-tests_istio/1229

https://prow.k8s.io/view/gcs/istio-prow/logs/integ-security-k8s-tests_istio_postsubmit/203

https://prow.istio.io/view/gcs/istio-prow/pr-logs/pull/istio_istio/17616/integ-security-k8s-tests_istio/1494

https://prow.istio.io/view/gcs/istio-prow/pr-logs/pull/istio_istio/17625/integ-security-k8s-tests_istio/1542

Can someone take a look at this? I'm pretty concerned this is a real bug we are ignoring as a "flake", which is something we do a lot. These tests used to pass 100% of the time and now fail very often.

https://prow.istio.io/view/gcs/istio-prow/pr-logs/pull/istio_istio/17625/integ-security-k8s-tests_istio/1542
I am pretty sure RBAC is not working properly on master..

@istio/wg-security-maintainers I am bumping this to a release blocker for 1.4. I don't feel comfortable shipping with RBAC is failing

I will take a look at the RBAC test failure today.

https://prow.k8s.io/view/gcs/istio-prow/logs/integ-k8s-114_istio_postsubmit/279
https://prow.k8s.io/view/gcs/istio-prow/logs/integ-k8s-114_istio_postsubmit/278
show TestNamespaceTargeting is having trouble as well

The last 20 postsubmits have failed: https://testgrid.k8s.io/istio_istio_postsubmit#integ-security-k8s-tests_istio_postsubmit&width=20

(might worth splitting into different issues...?)
Namespace targeting test are all failed for those expect true case, where the secrets should be generated, TestNamespaceTargeting/no_relevant_namespace_labels, TestNamespaceTargeting/matching_env_label,_no_override_label.
My hutch is that extend the timeout might mitigate the problem. I'll take a look of the previous history later to confirm though.

FYI, the post-submit is one run per test, we'll have to look at the Git SHA from the build log and then find the corresponding PR
+ [[ -n prow ]]
+ '[' -z '' ']'
+ export GIT_SHA=0b83609b03930bc1ee590fb0f3c46241d20c0c73

More historical test run test might be available through some gsutil ls ...

I was looking at https://prow.istio.io/view/gcs/istio-prow/pr-logs/pull/istio_istio/17625/integ-security-k8s-tests_istio/1542 where TestV1beta1_OverrideV1alpha1 failed. But my local run of this test always passes. Looking at testgrid, the test is showing green.
It seems that the test that is failing the most is sds_ingress/multi_tls_gateway_invalid_secret.TestMultiTlsGateway_InvalidSecret.

These tests are still failing consistently @liminw @JimmyCYJ have we identified what caused the regression?

https://prow.istio.io/view/gcs/istio-prow/pr-logs/pull/istio_istio/17625/integ-security-k8s-tests_istio/1542
I am pretty sure RBAC is not working properly on master..

@howardjohn The RBAC tests doesn't seem flaky: https://testgrid.k8s.io/istio_istio_postsubmit#integ-security-k8s-tests_istio_postsubmit&width=20, could you clarify a little bit what makes you think RBAC is not working properly on master?

RBAC does still seem to fail:

https://prow.k8s.io/view/gcs/istio-prow/logs/integ-security-k8s-tests_istio_postsubmit/307
https://prow.k8s.io/view/gcs/istio-prow/logs/integ-security-k8s-tests_istio_postsubmit/323
https://prow.k8s.io/view/gcs/istio-prow/pr-logs/directory/integ-security-k8s-tests_istio/2310
https://prow.k8s.io/view/gcs/istio-prow/pr-logs/directory/integ-security-k8s-tests_istio/2326

It is not the most frequent failure, but it is still concerning. Unlikely networking stuff, RBAC only has to fail a single time for potential exploits.
This issue is not just about RBAC though, there are still other security tests failing 100% of the time that used to pass.

It's not clear whether it's caused by real issues in the code or by some delays in the config distribution due to transient networking error or any other delays inside Envoy. It's also not clear whether this is happening only after config update or it's happening randomly in a cluster.
I will try to increase the timeout in the test to see if it helps to reduce the occasionally failures.

I have a feeling that it is due to the latency of config distribution.
Afaik, E2E tests share the same cluster, so it could cause some unexpected
delay when multiple tests are running, right? We may need to find a more
robust way to check when the test setup are ready to run test query.
On Wed, Oct 16, 2019 at 11:21 AM Yangmin Zhu<EMAIL_ADDRESS>wrote:

It's not clear whether it's caused by real issues in the code or by some
delays in the config distribution due to transient networking error or any
other delays inside Envoy. It's also not clear whether this is happening
only after config update or it's happening randomly in a cluster.
I will try to increase the timeout in the test to see if it helps to
reduce the occasionally failures.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/16998?email_source=notifications&email_token=AF7X24KKT64H7VQ5ZYP3Z4LQO5LS3A5CNFSM4IVPCA32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBNOV3Q#issuecomment-542829294,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF7X24JKNDSOHRLZSUFMOBTQO5LS3ANCNFSM4IVPCA3Q
.

--
Diem Vu | Software Engineer |<EMAIL_ADDRESS>| +1<PHONE_NUMBER>

https://prow.k8s.io/view/gcs/istio-prow/logs/integ-k8s-116_istio_release-1.4_postsubmit/15

@JimmyCYJ @rlenglet TestMultiTlsGateway_InvalidSecret is still failing 100% of the time. I have been trying to understand for over a month if this is a regression or a bad test. Please take a look ASAP -- this is marked as a release blocker for 1.4 and we will not ship until this is answered.
cc @jwendell @ericvn

@howardjohn TestMultiTlsGateway_InvalidSecret is a test only failure. The expected error message is connection refused, but got connection reset by peer. I am going to update the test.

RBAC still fails occasionallyhttps://prow.istio.io/view/gcs/istio-prow/pr-logs/pull/istio_istio/18317/integ-security-k8s-tests_istio_release-1.4/133

RBAC still fails occasionally https://prow.istio.io/view/gcs/istio-prow/pr-logs/pull/istio_istio/18317/integ-security-k8s-tests_istio_release-1.4/133

The fix https://github.com/istio/istio/pull/17935 is in master and the test seems no longer fail in master: https://k8s-testgrid.appspot.com/istio_istio_postsubmit#integ-security-k8s-tests_istio_postsubmit&width=5, I will cherrypick that to 1.4 as well.

Thanks!
On Fri, Oct 25, 2019 at 10:10 AM Yangmin Zhu<EMAIL_ADDRESS>wrote:

RBAC still fails occasionally
https://prow.istio.io/view/gcs/istio-prow/pr-logs/pull/istio_istio/18317/integ-security-k8s-tests_istio_release-1.4/133
The fix #17935 https://github.com/istio/istio/pull/17935 is in master
and the test seems no longer fail in master:
https://k8s-testgrid.appspot.com/istio_istio_postsubmit#integ-security-k8s-tests_istio_postsubmit&width=5,
I will cherrypick that to 1.4 as well.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/16998?email_source=notifications&email_token=AAEYGXLQ2VEUNM7OAAWR3Q3QQMR7XA5CNFSM4IVPCA32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECI7BMI#issuecomment-546435249,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXMMOZPHVCDLIBQRYI3QQMR7XANCNFSM4IVPCA3Q
.

Can we close this issue then?

Should drop down the priority now, but probably keep open as we are still seeing ~30% failure rates on various tests. If someone on security wants to break out individual failures we could close this though? ref https://testgrid.k8s.io/istio_istio_postsubmit#integ-security-k8s-tests_istio_postsubmit

@howardjohn I have created https://github.com/istio/istio/issues/18651 to trace the flakiness of ingress SDS tests. Given that other tests are not flaky now, I am going to close this issue. Please feel free to reopen.


PRBuilder.md: List needed python modules

Changes in the new patch version:

Say that installation instructions are just an example
Include instructions for NixOS
Add "requests" to the list

We should really make tools like this properly support nix, that should also take care of python dependencies.
One simple example script here: https://github.com/tiiuae/ci-public/tree/PRbuilder/csvdiff with its python dependencies declared in default.nix here: https://github.com/tiiuae/ci-public/blob/daaa2102a2adaa9a258424a6e5cb791025595a86/csvdiff/default.nix#L17
Which itself is by no means a perfect example, but we should aim something along those lines with all or our tooling (on longer term).


Inconsistent SHA1 between deployment/logsearch-deployment.yml and bosh.io

We just started using the deployment/logsearch-deployment.yml manifest and noticed that the SHA1 for the release is not the same as the one from bosh.io. The same goes for the logsearch-for-cloudfoundry release.
We have a pipeline that pulls all required release from bosh.io and transfers these files to our air-gapped network. However I now need to add an ops file to modify the SHA.
Wondering why the releases are different and which one should be use: the one from the manifest of the one from bosh.io?

Hi @mymasse.
When we cut a release, we build the tarball locally, then put it into S3 bucket. You can find a link to tarball in the manifest, together with SHA sum.
As for bosh.io, they has it own scripts to build all releases. They build the release by they own, so SHA1 is differ.
You can use whatever release you like.

OK, weird didn't think bosh.io was building releases, I thought they were just referencing the blob final configs?

So I think I found the difference, downloading from bosh.io I get logsearch-boshrelease-<version>.tgz and from your this repo I get logsearch-<version>tgz. Wouldn't that explain the difference in SHA1 (since the tar file in the tgz uses a diferrent name). I think bosh.io builds the release using the repo name, hence the boshrelease part in the name. To minimize confusion should the one you build be change to match the same name?

I don't think so, because filename is not part of the file itself, so shouldn't affect SHA sum.

But the tgz contains a tar called logsearch-.tar, so wouldn't the tgz sha1 be affected by it

logsearch-for-cloudfoundry tarball has the same name as on bosh.io, but SHA sum is differ :-(


Fixed broken references across the documentation

In many places in the documentation, we have used references
of the form:
:meth:`~cms.models.pluginmodel.CMSPlugin.copy_relations()`

to the referent:
..  class:: cms.models.pluginmodel.CMSPlugin

    [...]

    ..  method:: copy_relations()

Often, this has not been done correctly, with either the
directive or the role that links to it incorrectly formatted, and
the directive often simply missing.
This patch fixes 176 warnings across the documentation.
It's a first step to improving the reference documentation more generally. The next step will be to rationalise and reorganise the /reference section.

Addresses https://github.com/divio/django-cms/issues/4785

LGTM!

Please rebase

Don't merge until I can figure out how to get rid of the unnecessary 5876d2f commit from this.

Coverage increased (+0.6%) to 89.528% when pulling 9ea40a1e08b25ce8f210bfbbe35aa8d3e1638069 on evildmp:docs-fix-references into 7632db76ab346bebc094d55dabbb4cc686b0b15d on divio:release/3.3.x.

Still LGTM

@czpython thanks for these notes, but I this PR is mainly to fix the errors; I have another one coming with mainly reference updates https://github.com/divio/django-cms/pull/5453 (in progress), so I will add those to that instead.

@evildmp Fine with me. Ok to merge.


Add ability to cache person avatar

It's more polite than hot-linking.

Depends on #16 since we needs to think about how we split author urls and images away from a static value to multiple values.

Note: very few podcasts seem to include this element so it's not a super high priority.


[Bug] Could not find artifact xxx in central/apache.snapshots during "mvn clear install -DskipTests"

Search before asking

[X] I searched in the issues and found nothing similar.

Paimon version
Master Branch
Commit id: 41997b2170dcfa32aa867bbc4573989911f05f61
Compute Engine
No Compute Engine involved
Minimal reproduce step
CPU: M1 Pro
OS: macOs Ventura
IDE: IntelliJ IDEA
JDK Version: Amazon Corretto Version 1.8.0 (aarm64)
Maven Version: Maven 3.8.1 Bundled (default setting provided by IntelliJ IDEA)
git clone git@github.com:apache/incubator-paimon.git
Open the repositories with IntelliJ IDEA, run Maven command "mvn clean install -DskipTests" with the following configurations:

What doesn't meet your expectations?
Got lots of errors related to dependency issues, for example:
Could not find artifact com.google.code.findbugs:jsr305:jar:tests:1.3.9 in central (https://repo.maven.apache.org/maven2)
Could not find artifact org.apache.paimon:paimon-parent:jar:tests:0.5-SNAPSHOT in apache.snapshots (https://repository.apache.org/snapshots)
Screenshot:

Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

@YIHONG-JIN
Check your network? Among them, the Apache repository is on a foreign server, and it is usually very slow to download the jar, please try a few more times.

I am physically in the United States so I believe that should not be the cause. But thanks for your suggestions.

@YIHONG-JIN Check your network? Among them, the Apache repository is on a foreign server, and it is usually very slow to download the jar, please try a few more times.
I am physically in the United States so I believe that should not be the cause. But thanks for your suggestions.

@YIHONG-JIN Check your network? Among them, the Apache repository is on a foreign server, and it is usually very slow to download the jar, please try a few more times.

I am physically in the United States so I believe that should not be the cause. But thanks for your suggestions.

@YIHONG-JIN Check your network? Among them, the Apache repository is on a foreign server, and it is usually very slow to download the jar, please try a few more times.

I am physically in the United States so I believe that should not be the cause. But thanks for your suggestions. Is there a suggested maven configuration for this project?

No special configuration of maven is required. I can develop locally normally.

@YIHONG-JIN Check your network? Among them, the Apache repository is on a foreign server, and it is usually very slow to download the jar, please try a few more times.

I am physically in the United States so I believe that should not be the cause. But thanks for your suggestions. Is there a suggested maven configuration for this project?

No special configuration of maven is required. I can develop locally normally.

I retried for several times. It looks like it is not because maven could not find the dependencies theirselves. Due to the log, maven is trying to find the "tests" of these dependencies during compilation, which is the reason why the log looks like "Could not find artifact com.google.code.findbugs:jsr305:jar:tests:1.3.9 in central (https://repo.maven.apache.org/maven2)". However, there is not such jar in central repo.
In addition, I have also seen warnings like:
"[WARNING] Could not get tests for com.google.code.findbugs:jsr305:jar:1.3.9:compile
[WARNING] Could not get tests for org.apache.paimon:paimon-parent:pom:0.5-SNAPSHOT"
I would be thankful if someone could help me root the cause.

Update: When I try to build this project through terminal, it success. However, there are still warnings like "[WARNING] Could not get tests for org.glassfish.jersey.core:jersey-client:jar:2.30:compile", which means the cmd version Maven is also trying to find tests for dependencies, which does not exist. There is no error reported in terminal despite it is reported as an error inside IntelliJ.

Though I am not an expert of Maven, I believe it should be possible to prevent Maven from finding tests for dependencies by modifying pom.xml structures.


Remove unique Pet name validation

Pets can have the same name! So let's remove this validation
https://github.com/rubyforgood/pet-rescue/blob/a1605ebcbd5dbfb36c26b075cc0d401853448998/app/models/pet.rb#L37-L37

Resolved by #58


Cannot create folder/files in mounted s3 bucket

Not sure if it is an issue or I do something wrong.
I am logged in as a normal user (raztud), and I mount the s3 bucket like this:
goofys --region eu-west-1 bucketname /home/raztud/mywebsite.org/public_html/wp-content/s3

and it seems all the permissions are okay and I can browse through files.
# ls -lh ~/mywebsite.org/public_html/wp-content
...
drwxr-xr-x 2 raztud raztud 4.0K Mar 20 08:22 s3
...
# ls  -l s3/photos/mypics/DSC09463.JPG
-rw-r--r-- 1 raztud raztud 85614 Jul 14  2011 s3/photos/mypics/DSC09463.JPG

But:

I cannot create anything inside the s3 folder

# mkdir s3/asd
mkdir: cannot create directory ‘s3/asd’: Permission denied
# touch s3/blah
touch: failed to close 's3/blah': Permission denied

nginx cannot access the files:

# curl -I http://mywebsite.org/wp-content/s3/photos/mypics/DSC09463.JPG
HTTP/1.1 404 Not Found

and the nginx log:
2017/03/20 08:27:33 [error] 7733#7733: *3126 open() "/home/raztud/mywebsite.org/public_html/wp-content/s3/photos/mypics/DSC09463.JPG" failed (13: Permission denied), client: <IP_ADDRESS>, server: mywebsite.org, request: "GET /wp-content/s3/photos/mypics/DSC09463.JPG HTTP/1.1", host: "mywebsite.org"

There is any trick how can I mount the bucket to be able to access the files from nginx and create files/folders inside the bucket?
Thx.

If the bucket is public-read, goofys by default uses that instead of the configured credential. To overwrite that, use --profile default (or replace default with the correct profile).
Is nginx running under the user raztud? If not, you need to use -o allow_other and possibly --file-mode/dir-mode options

I am running into the same issue.  I tried --profile default with no success

could you post the output of --debug_s3?

@raztud I am trying goofys for the first time, and I had the same problem. My issue was the IAM role for the VM I was using was too restrictive. If you are using IAM to guard S3, then try looking in your policies to see if that is where the denial is coming from.

closing for now, please reopen if the solution by @mrapczynski doesn't work

I am having the same issue and I am not running in a VM.  I am using MacOS with homebrew.  When I try and create a directory I get
iosbuild$ mkdir folder1
mkdir: folder1: Input/output error
Syslog contains
/usr/local/Cellar/goofys/0.0.9/bin/goofys[99199]: s3.ERROR code=A header you provided implies functionality that is not implemented msg=501 request=
I have a feeling that it has to do with the content length being 0, because the same thing occurs if I try and 'touch' a file.  Yet I can copy files.
iosbuild$ touch file2
touch: file2: Input/output error

@dlhoward Please run again and attach output from --debug_s3.

@dlhoward you are using go1.8 and no released version of goofys is compatible with it. Build from source in master

Just my 2 cents:
Using goofys to mount an s3 bucket in my ubuntu server filesystem via an IAM user attached to a similar policy to
{
  "Version": "2012-10-17",
  "Statement": [
    {
      "Effect": "Allow",
      "Action": "s3:*",
      "Resource": "example"   <<< replace example with ARN
    }
  ]
}

That caused the inability to create files.
Changing it to the managed policy of AmazonS3FullAccess works. I guess something wrong with my policy. Possible related issue I had noted earlier here

I am confused, could you give a more concrete example? If you only allow access to "example", what files do you expect to be able to create?

I mean that when I mounted an s3 bucket with goofys, I got an error about being unable to create files/folders. It turned out not to be because of goofys nor because of local file permissions, but because of the policy attached to the IAM user that I was using. The policy that did not work was my custom policy listed above. Replacing the policy with the one readily available from amazon (s3 full access) solved the problem.

Did the error come about when you mount or when you ran other commands?

It came about after I mount when I tried cp test.txt /mnt/my-s3/

can you paste a more realistic example of "example"? Are you only allowing access to a path or is it a wildcard? See https://github.com/kahing/goofys/issues/136 for an example of a working policy

My exact "example" was "arn:aws:s3:::shadi-torrents", where "shadi-torrents" was the name of the bucket. I don't specify neither path nor wildcard.


Add option to add a reason to ForbiddenImport

I'm really happy to be able to introduce this feature :). Thanks @marschwar for creating the necesary tooling to be able to implement it.
Important: This introduces a breaking change in ForbiddenImport. It no longer allows comma-separated imports as a String. It should use yml lists. I'm targetting this to 1.22.0 because on 1.21.0 we added make that all those usages will be a warning. Tell me is we should be more conservative.
The reason that this is a draft is just to avoid accidental merges. But the code should be ready to merge.
Related with #3501

There is no documentation for this?

Probably it is not. Do you want to create a PR to help improve the documentation?


Updated XML string to inform user that MSA auth is not supported for …

…requests currently.

@stefuhnee,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Builds on master result in high CPU use for the Watcher

CPU utilisation is consistently around 1.5 vCPU (1574m)

Dropping a note for future reference here. After the reset of the devel child chain, the issue still persists in that environment, I've noticed it affects bee013a, 5602254 and 55abe27, load around 1.5 vCPU
At the same time, the Watcher idling consumes 0.05 CPU for me locally.

This was addressed in https://github.com/omisego/elixir-omg/commit/535552bc2e3d72591c0bbb320e93ab6490481acf


Changed Firebase Project Settings - unable to obtain API-Key

In step 6. of chapter "Create a Firebase project and enable Firebase Cloud Messaging" there's no "Server Key" displayed in the "Cloud Messaging" section of the Firebase project settings.
How can the required API Key needed for step 2. of chapter "Configure GCM/FCM settings for the notification hub" be obtained?

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

ID: 2a03ab5f-3dda-f469-33e0-9504c90d468c
Version Independent ID: 4f8e4309-cd06-f09f-00f0-d978b3d16777
Content: Send push notifications to Xamarin.Android apps using Azure Notification Hubs
Content Source: articles/notification-hubs/xamarin-notification-hubs-push-notifications-android-gcm.md
Service: notification-hubs
GitHub Login: @sethmanheim
Microsoft Alias: matthewp

@pauljereb
Thanks for your feedback! We will investigate and update as appropriate.

As mentioned, we have created a new issue with the author to update the doc. We believe the above suggestion should help to serve as a workaround.


docs-bug(stepper): stepper errors not working in documentation

Documentation Feedback
The following example is not showing the messages in the example code: https://material.angular.io/components/stepper/overview#error-state
This could be a general bug as I can't get errors to show in my own project either.
Affected documentation page: https://material.angular.io/components/stepper/overview#error-state

The example is mostly set up correctly, the problem is that the stepper is marked as linear which means that you can't move off of the current step in order to see the error. I've submitted a PR to make it non-linear and expand the example description.


[Question]: A usecase for numpy processing with frequent IO communication

I am trying to use aiomultiprocess in my project which the simplified idea of current implementation is
# pip install gym
import gym
import numpy as np
from multiprocessing import Process, Pipe

def worker(master_conn, worker_conn):
    master_conn.close()

    env = gym.make('Pendulum-v0')
    env.reset()

    while True:
        cmd, data = worker_conn.recv()

        if cmd == 'close':
            worker_conn.close()
            break
        elif cmd == 'step':
            results = [env.step(data) for _ in range(1000)]
            worker_conn.send(results)

class Master(object):
    def __init__(self):
        self.master_conns, self.worker_conns = zip(*[Pipe() for _ in range(10)])
        self.list_process = [Process(target=worker, args=[master_conn, worker_conn], daemon=True)
                             for master_conn, worker_conn in zip(self.master_conns, self.worker_conns)]
        [p.start() for p in self.list_process]
        [worker_conn.close() for worker_conn in self.worker_conns]

    def go(self, actions):
        [master_conn.send(['step', action]) for master_conn, action in zip(self.master_conns, actions)]
        results = [master_conn.recv() for master_conn in self.master_conns]

        return results

    def close(self):
        [master_conn.send(['close', None]) for master_conn in self.master_conns]
        [p.join() for p in self.list_process]

master = Master()
master.go(np.random.rand(10, 1))

It has a lot of IO communication through Pipes, I am wondering how could I speed it up with aiomultiprocess

Sorry I missed this.  Given your example code, where the individual work items appear to be CPU bound, I'm not sure aiomultiprocess would help you much.  aiomultiprocess depends on serializing and deserializing data between parent/child processes, which still takes the GIL in both processes.  If your workload instead was itself IO bound, such as making or waiting on network requests, then aiomultiprocess might be worth considering.


Pause chat keybind, disables clicking

If you use a hotkey to stop the chat from scrolling, you can no longer select users/use commands on users.

https://streamable.com/lxpd7
Chatterino Nightly 2.1.5 (commit 367b1970f, 22.09.2019)

Description is inaccurate. Clicking still works but the cursor does not change. Also it is unrelated to pausing.

Now I noticed that it's a duplicate of https://github.com/Chatterino/chatterino2/issues/1292


update pytest requirement from ^7.4.3 to ^8.0.0

i forked and added a dependabot.
it reported these recommendations

update pytest requirement from ^7.4.3 to ^8.0.0

Updates the requirements on pytest to permit the latest version.

Release notes
Changelog
Commits

pyproject.toml
change:
pytest = "^7.4.3"
to:
pytest = "^8.0.0"

(See #10 for more info)


[WIP] Python

This is an ongoing effort to wrap go-git with Python.
Internals

go build -o libgogit.so -buildmode=c-shared github.com/src-d/go-git/cshared makes a shared library for general usage
Python uses cffi <- libgogit.h
Go and Python interfaces must be written manually

Goals

Python 2.7 and 3.3+ support
Cover 95% of go-git API
100% test coverage

What currently works
See README.python.md
Basically, what's already wrapped should work. I am not using TDD here for several reasons so tests will be written in the end.

[Current coverage][cc-pull] is 70.32%

Merging [#53][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master        #53   diff @@
==========================================
  Files            31         31
  Lines          1732       1732
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1218       1218
  Misses          514        514
  Partials          0          0

Powered by Codecov. Last updated by [e9bce55...d525126][cc-compare]
[cc-base-branch]: https://codecov.io/gh/src-d/go-git/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/src-d/go-git/compare/e9bce553cf38f50633bef54ed9d4a9a37bf842ea...d525126bbe5692153819a3ab167ad91a55886379
[cc-pull]: https://codecov.io/gh/src-d/go-git/pull/53?src=pr

Please make the PR to https://github.com/src-d/gypogit


screen orientation support for TWAs

The previous PR is having too many unnecessary files so I made a new branch with the implementation of screenOrientation support.
This one should be clean and right to get reviews. PTAL @PEConn @andreban @glennhartmann @EiraGe
Thanks very much in advance!

I do not know why this PR has many commits in it.

Compared to wrong branch. Created a new PR to the alpha branch here #127.


[Bug] Table prefix added to database name when using database.table

Laravel Version: 5.5.#
PHP Version:7.1
Database Driver & Version:

Description:
Detail in #21295
I am sure it is a database bug, but how can we fix it ?
Steps To Reproduce:
DB::connection()->getGrammar()->setTablePrefix('prefix_');
$sql = DB::table('public.user')->toSql();

result：
select * from "prefix_prefix_public"."user"

@themsaid

Why do you set the database name? Every query connection has a single database.

Why we have the test case in Illuminate\Tests\Database\DatabaseQueryBuilderTest, line 126:
public function testBasicTableWrapping()
    {
        $builder = $this->getBuilder();
        $builder->select('*')->from('public.users');
        $this->assertEquals('select * from "public"."users"', $builder->toSql());
    }

?

I don't see how that is related. That test is teting if a from() or selects work with table wrapping

@Dylan-DPC You should see, DB::table() will call function from():
class Connection implements ConnectionInterface
{
    public function table($table)
   {
      return $this->query()->from($table);
   }
}

Please answer my question to help us understand better:

Why do you set the database name? Every query connection has a single database.

When we have two databases in one machine or host, we can join different table on different database, sql
select a.id, b.id from db1.a as a left join db2.b as b on a.b_id = b.id

I think there are some reasons that the test case exists in file Illuminate\Tests\Database\DatabaseQueryBuilderTest, above is just one of them. If you think the test case is unnecessary, I can take a PR to delete it, and close this issue. thanks.

Why don't you define multiple connections for this, that way you're able to better show what connection you need to use for the query.

Sometime multiple connections can not resolve our problems. For example, we can not use multiple connections to paginate, sql
select a.id, b.id where a.country='UK' and b.country='US' from db1.a as a left join db2.b as b on a.b_id = b.id  limit 30 offset 2

how can we use multiple connections to achieve this sql?
In SQL SERVER, there is function sp_addlinkedsrvlogin that helps us build remote connection to other databases. It is proved that it is necessary even in distributed environment.

For suggestions and feature requests please use the https://github.com/laravel/internals repo, closing this here since it's not considered as a bug with the current assumption laravel makes that each query hits a single database connection.


Just wondering how to reset the FCB data

Basically, I was messing about with the Shades, hue's and other colour values, trying to get some specific colours, and I changes a value by 1 too many digits, which causes the game to freeze and crash on loading...
Any Fixes for this, as I've tried re-installing forge, deleting the mod countless times, I've tried everything I can think of, It would be a great help if there is some form of way to fix this
Thanks

Delete flatcoloredblocks.cfg file in config folder.


save video in sd

hello,
i want to save video to sd card, how can i do it?

Can not save video through MediaPlayer.
You can  only download video file from url.

thanks


Constructing terragrunt cache is extremely slow for local modules

Our project is structured similar to this: https://github.com/tfutils/tfscaffold
For example:
├─ components/
│  └─ a-component/
│     └─ main.tf
└─ modules/
│  └─ a-module/
│  │  └─ main.tf
│  └─ b-module/
│     └─ main.tf
└─ config/
   └─ target/
      └─ terragrunt.hcl

In components/a-component/main.tf
module "aws_vpc_cni" {
  source = "../../modules/a-module"
}

In config/target/terragrunt.hcl
terraform {
  source = "../..//components/a-component"
}

All works nicely, but we've noticed that the entire project gets copied into the terragrunt-cache for local modules, so even though b-module is not used by a-component it is still copied into the cache. For smaller projects this is fine, but as a project grows things can get extremely slow on apply (seems to take minutes while it copies into the cache) and disk usage can quickly grow up to many gigabytes in size.

This is true, but I'm not sure what a good solution would be? Terragrunt copies more than just the module you're using due to the double-slash: e.g., xxx//components/a-component tells Terragrunt to copy the entire xxx folder so that any relative paths within a-component will work correctly. If you don't need relative paths, you can perhaps reduce the amount copied by removing double-slashes, but we don't usually recommend that, as it can lead to confusing errors...

Ah, I see what you mean about the relative paths. So if component-a uses module-a and module-a uses module-b then there is no guarantee that the path to module-b would be correct! Is it possible to calculate the local dependencies and file paths before the cache is copied and rewrite them to work within the cache?
We did have the double slash removed for a while, but it just confuses the IDE amongst other things!

Is it possible to calculate the local dependencies and file paths before the cache is copied and rewrite them to work within the cache?

I suspect it's quite tricky. It would require parsing all your Terraform code and knowing all the places where a relative path could be. Examples:

source URLs in module { ... } blocks.
Paths passed to the file function.
Paths passed to the local_file resource.
Paths passed to the fileset function.
Paths passed to the templatefile function.
Paths passed to the abspath function.
Probably a dozen other functions I'm not aware of... Plus all the ones added in the future.
Paths used with provider blocks.
Paths used with specific resources (e.g., aws_s3_object).
And so on.

I've had some success in using this to decrease the time as for me, it mostly came from re-downloading the provider plugins for multiple times.
terraform {
  before_hook "before_cache" {
    commands     = [get_terraform_command()]
    execute      = ["mkdir", "-p", abspath("${get_parent_terragrunt_dir()}/../build/terraform")]
  }
  extra_arguments "cache" {
    commands = [get_terraform_command()]
    env_vars = {
      TF_PLUGIN_CACHE_DIR = abspath("${get_parent_terragrunt_dir()}/../build/terraform")
    }
  }
}

Closing this as the original question has been answered, and there isn't anything actionable for terragrunt side. If there are any follow ups, please open a new ticket with updated context. Thanks!


docs: fix a few mistakes in ORM getting-started.md document

The documentation requires proper maintenance.

I plan to browse through the documentation and correct any errors that I encounter. Happy to be a part of this project.

Great, thanks!

Hey @zzpwestlife  We created a private channel in Discord for contributors. You can click the link and DM me (@Bowen) to join it if you are interested.
Here you can:

Send PR link or review others' PRs;
Discuss new features before creating an issue;
Vote for something;
etc.


Support for the out-of-band and did-exchange protocols

Summary of changes:

Two new cases to test both main flows in the out-of-band protocol.

The sender flow --  I create an invitation and you use it
The receiver flow -- You create an invitation and I use it

Support for the out-of-band handshake reuse protocol.

This includes a new message handler to accept and respond  appropriately

Support for the did-exchange protocol.

You will now be able to complete a did-exchange flow between the AUT and the suite.
There is no new test case that explicitly completes this flow, but it is used within the OOB protocol test if specified in the AUT's handshake_protocol list.

Support for sending messages with https://didcomm.org/ in their @type field instead of did:sov:BzCbsNYhMrjHiqZDTUASHg;spec/

Receiving a message with did:sov:BzCbsNYhMrjHiqZDTUASHg;spec/ is still supported however.

NOTES:

Request attach handling is not supported quite yet.
During the receiver flow, we only handle the first supported handshake protocol. If that one fails, we don’t retry with the next one.

@smithbk With regard to your trust ping comments -- I think the did-exchange complete would be sufficient to test the connection, but I'm wondering what we should do in the case that the AUT doesn't want to use did-exchange and chooses to use the connections protocol instead. I think in this case, without something like a trust ping there'd be no real way to verify the connection. Open to ideas for this, but I had originally put in the trust pings following how it's done by the current code. I've removed them for now, but I think we definitely need a method to verify the connections. Also I've re-read the OOB spec and it doesn't seem there's an explicit way to discover features in it.

@smithbk With regard to the message prefixes. I've centralized the prefix definitions into the Suite class. When the time comes we can make these types such that the Suite.TYPE_PREFIX and Suite.ALT_TYPE_PREFIX to be the same. This effectively makes the suite only support the one prefix.

@Tangelo1 IIRC, the connection protocol has an ACK message which would be equivalent to the DID exchange complete message.  WRT the current code, yeh, I think an early version of the connection protocol may have said or recommended using the trust ping protocol to complete the connection.  Also, the trust ping protocol used to be part of Aries Profile V1 but was removed.  Obviously APTS wasn't updated appropriately.
Yeh, I was misremembering about OOB having features.  If you wanted to do a trust ping, I think the proper way is that the trust ping request is added to the OOB message as an attachment.  The receiver of the OOB message would optionally process the attachment and send the trust ping response.   That said, I don't think it is necessary.  Given that we could use the ACK message for the connection protocol and the complete message for the DID exchange protocol, I think it is sufficient to verify that the connection was successful.

@smithbk Yep, you're correct. I forgot about the ACK. I agree that should be sufficient to test the connection.


Adds default button classes to template.

Using the default value for buttonClass would lead to a flash when rendered on the server. This updates the template so that buttonClass is handled the same as arrowClass.

LGTM

Just started reviewing :) :octocat: Sent from GH.

Just started reviewing :) :octocat: Sent from GH.


Requests with exceptions on Application Insights have Sucess = "True" instead of "False"?

I'm using IHttpResponseHandler to handle the execeptions using CreateResponseFromException:
public async Task<IActionResult> CreateResponseFromException<TCommand>(TCommand command, Exception exception)
	where TCommand : ICommand
{
	this.logger.LogError(exception.InnerException, exception.InnerException.Message);
	IActionResult response = new ContentResult()
	{
		StatusCode = (int)HttpStatusCode.InternalServerError
	};
	return response;
}

I'm logging here the exceptions to Application Insights using the Microsoft ILogger abstraction.
When an exception occurs is properly log, but the request is assumed as "Sucess" instead of "Failed", because the exception was handled by Function Monkey (I think). Is possible to set the request to "Failed"?

I think to set the request to failed you literally need to fail the function - which means throwing an exception. In the code above you could throw an exception immediately under your call to the logger and that should return a 500 status code and fail the function.
Let me know how you get on. Will leave this open for now.

Yes if I do that it works, but the above code was just a simple sample to understand the issue.
In my real scenario, when people call the Http API, I need to provide a structured ErrorContent with the exceptions/errors and still have the "Sucess = False". If I throw an Exception I'm not able to provide that.
So as I understand this is one limitation of Azure Functions/ Application Insights? Since the only way to write "Sucess = False" is the function to fail with an exception.

Yes that's correct. I'm not sure there is anything I can do to turn this off. I may be misunderstanding?

Interesting. That could well do it. And I don't think anythng is lost from a FM / REST point of view. I'll have an experiment with that.

My custom ResonseHandler just throws an original exception and it works well (Failure in App Insights as expected). The problem with the handler is that it hides the origin of the exception by re-throwing via "throw ex". Using just "throw" in the catch area would solve it.


Add requirements.txt and progress bar by using progressbar2.

acc should be 100.100.(correct / total)  not just correct/total.


I can't launch MiEx

Well, i don't have any idea why i can't launch it. The info below may help.
OS: Windows 11
Java: I have jdk 17 and java 8

What happens if you run java -jar MiEx.jar

Thanks ! I ran D:\Arc>MiEx.jar and it worked. Tho i still can't launch it with double click


New badges

@hendrikmuhs
added badges for:

supported python versions
supported python implementations
PyPI latest version

hmm, the badge links do not work for me, do they work for you?

Coverage remained the same at 86.143% when pulling 15f524727c7048d3f8cefb4ec3f65168ee49af68 on narekgharibyan:badges into 2094e0237eb23324a4169ec9f3cde20a244f949f on cliqz-oss:master.

they did.
looks like at the moment the moment shields.io is serving requests very slowly :/


Add tip in console to install wallet-connect if you try to use connectAlgoWallet and dont have the lib installed via npm

never mind! we include this as a dependency.
but perhaps algonaut.js should handle the checking of the local storage var walletconnect which that lib sets after a successful connect. that way algonaut and walletconnect can be in sync without having to make the project handle that.
something like this?:
const wcLocal = localStorage.getItem('walletconnect');
let savedState;
if (wcLocal) {
    savedState = JSON.parse(wcLocal);
    algonaut.account = savedState.accounts[0];
}

@spencercap @youraerials @lanidelrey How elaborate do we want to get with our support of WalletConnect? It's kind of a lot for our small team to support. Is there anything wrong with providing minimal support (ie. ceasing development of WC features) and primarily telling people to use Inkey?

i'm fine w this move – let's tell people to use inkey!
when wallet connect v2 has an Algorand provider like it does for eth (https://docs.walletconnect.com/2.0/javascript/providers/ethereum) then we can revisit

i'd like to keep minimal support and revisit when they have V2 up and running
... which i CANNOT BELIEVE is not done yet on the algo side it's like, such a blocker

using any-wallet resolves this


IMA sdk doesn't compile on MacBook Pro with M1 chip

After adding pod reference to my podfile as following
pod 'GoogleAds-IMA-iOS-SDK'
and launching a pod install command, my app doesn't compile anymore on my MacBook Pro M1 using any iOS Simulator target facing the following compile time error:
Could not find module 'AnyModuleName' for target 'x86_64-apple-ios-simulator'; found: arm64, arm64-apple-ios-simulator, at: /Users/.../.../Debug-iphonesimulator/AnyModuleName.swiftmodule

Same project with the same podfile works without any problems on an Intel MacBook Pro (2,6 GHz 6-Core Intel Core i7).

They haven't updated this to build an .xcframework yet. A lot of the google sdks don't.  @google-admin can you give an ETA on when you intend to support this?  While you're at it could you also support SPM? This is literally a 1-2 day job.

The same issue here. @google-admin can you give us an ETA when you plan to support M1 and when will it support SPM?

Is this fixed now? it what version

@madmet I was unable to replicate this issue. Here is what I did.

ran pod install on Swift/BasicExample
Opened the workspace
Ran the project

Please let me know if you have alternative steps.

The IMA SDK for iOS has been packaged as an SDK since version v3.16.3. Without confirmed reproducibility on a recent version I'm closing this as resolved for now


Gradient checking fails for non-cuDNN GPU convolution wgrad

There are so many hypothesis configs for this test that it may take several tries before this error appears. Or you can increase the robustness of the test with @hypothesis.settings(max_examples=50).
$ pytest -sv caffe2/python/operator_test/conv_test.py::TestConvolution::test_1d_convolution_nchw
...
Failed. [idx, grad, grad_estimate] are:
[[ 0.         -0.04264414  0.25350109]
 [ 1.          0.00933766 -0.21303922]
 [ 2.          0.10659322  0.00899091]]
...
AssertionError: Gradient check failed for input w

$ pytest -sv caffe2/python/operator_test/conv_test.py::TestConvolution::test_3d_convolution_nchw
...
Failed. [idx, grad, grad_estimate] are:
[[  7.           0.70774764   0.33553407]
 [ 14.           0.20726326  -0.16495049]
 [ 15.           0.35344854   0.26257008]]
...
AssertionError: Gradient check failed for input w

1d convolution still fails with the same error.
3d convolution is possibly worse?
$ pytest -sv caffe2/python/operator_test/conv_test.py::TestConvolution::test_3d_convolution_nchw
Trying example: test_3d_convolution_nchw(self=<conv_test.TestConvolution testMethod=test_3d_convolution_nchw>, input_channels=2, output_channels=1, batch_size=1, stride=1, size=4, kernel=2, dilation=2, pad=0, u$e_bias=True, gc=<caffe2.proto.caffe2_pb2.DeviceOption at 0x7fcc28dfeb90>, dc=[<caffe2.proto.caffe2_pb2.DeviceOption at 0x7fcc28dfeb18>,
 <caffe2.proto.caffe2_pb2.DeviceOption at 0x7fcc28dfeb90>])
F0830 23:07:05.518461  7324 context_gpu.cu:357] Error at: /caffe2/caffe2/core/context_gpu.cu:357: an illegal memory access was encountered

Aborted (core dumped)

@lukeyeager Its a bug in im2col_nd_gpu_kernel. I tried to track it down but I wasn't able to get cuda-gdb to work with caffe2 which makes debugging the device code very hard.


Improvement for say command

Contains
A nicer solution for transform String[] to String with separator in between : Arrays.stream(words).collect(joining(sep));
Thanks for advice @skaldarnar
How to test
Please see #2805

Hooray Jenkins reported success with all tests good!

thanks for the quick fix 👍


Throws error "Unknown method: 09/30/2018"

I tried one of the basic examples. However, every browser throws me this error. I have included all the libraries of course.
The countdown and plugin js are in a local folder.
It happens no matter what I use for the time.
 Scripts -->
    <script src="https://code.jquery.com/jquery-3.3.1.js" integrity="sha256-2Kok7MbOyxpgUVvAk/HJ2jigOSYS2auK4Pfzbm7uH60=" crossorigin="anonymous"></script>
    <script src="./js/jquery.plugin.js"></script>
    <script src="./js/jquery.countdown.min.js"></script>
    <script>
    $('#clock').countdown("09/30/2018", function(event) {
      $(this).html(event.strftime('%D days %H:%M:%S'));
    });
  </script>
  </body>

</html>

i have the same error


Feat / Webhook Notifications

[x] Add new NotificationType GraphQL object type, based on the Notification model.
[x] Return notifications with a Collective. Makes it possible to query and edit it on the Edit Collective page.
[x] Add webhook notification type.
[x] Implement editWebhooks mutation.
[x] Implement subscribe and unsubscribe endpoints for Zapier to subscribe to webhooks.

Ref: opencollective/opencollective#1839

Thank you all. It's time to merge !

Nice!


Upload File window opens twice & disabled doesn't work in Next.js

When the package is imported and run in Next.js, the file upload window opens twice when the component is clicked. Also, the file upload is still clickable & will open a window even when disabled is set to true. I haven't been able to replicate this in regular React apps.

@room9design  Thanks for your contribution.
This one is solved now on v.2.3.6


Fixed ansible_role vars in zuul.yaml

@sbstnnmnn Always use -s to Sign-off commits.

Please fix operator_group, operator_user in
https://github.com/osism/ansible-collection-commons/blob/main/molecule/delegated/vars/configuration.yml
https://github.com/osism/ansible-collection-commons/blob/main/molecule/delegated/vars/sosreport.yml
https://github.com/osism/ansible-collection-commons/blob/main/molecule/delegated/vars/sshconfig.yml

Please install ufw in
https://github.com/osism/ansible-collection-commons/blob/main/molecule/delegated/prepare/firewall.yml


Implement JWT Authentication in the server

As a user I want to be able to complete Middleware, Login Route and Protect API Routes

I've finished this already. Issue was cover with the server creation


discovery plus not working anymore

Hello, i wanted to download as usual from Discovery plus but now it says "Unsupported URL"
the url is "https://www.discoveryplus.com/it/video/i-signori-della-neve/stagione-2-episodio-1-i-preparativi"
Can anyone kindly help?

Duplicate of #29144; see #29294. I believe this is implemented in https://github.com/yt-dlp/yt-dlp/releases, if you can run that.

Hello, i wanted to download as usual from Discovery plus but now it says "Unsupported URL" the url is "https://www.discoveryplus.com/it/video/i-signori-della-neve/stagione-2-episodio-1-i-preparativi" Can anyone kindly help?

try again using the cookies option

it still works
D:\ytdl>yt-dlp.exe -F https://www.discoveryplus.com/it/video/i-signori-della-neve/stagione-2-episodio-1-i-preparativi
[DiscoveryPlus] i-signori-della-neve/stagione-2-episodio-1-i-preparativi: Downloading JSON metadata
ERROR: [DiscoveryPlus] i-signori-della-neve/stagione-2-episodio-1-i-preparativi: This video is only available for registered users. You may want to use --cookies.

yt-dlp includes the fix for new (as of earlier this year) Discovery VOD sites. yt-dl doesn't yet.

Hello, i tried yt-dlp but unfortunately i still cannot understand how to get the video. BTW, this video is available for free and not for registered users only as yt-dlp is stating... how can i solve this??

Best to raise this at https://github.com/yt-dlp/yt-dlp/issues


ci: Footprint tracking workflow is failing (aws not found)

Describe the bug
As part of addressing #56613, it looks like the footprint tracking actions broke. It's been failing since mid-April with aws not found error message.
First failed run seems to be https://github.com/zephyrproject-rtos/zephyr/actions/runs/4695626273/jobs/8324962702. The two changes introduced vs. the previous successful builds are:

https://github.com/zephyrproject-rtos/zephyr/commit/b1e4116d404e35817abab532096a9cbc7eb83006
https://github.com/zephyrproject-rtos/zephyr/commit/4691359596621693002ee0a82328c26c7b5ea362

To Reproduce
https://github.com/zephyrproject-rtos/zephyr/actions/workflows/footprint-tracking.yml
Expected behavior
Job doesn't fail and footprint tracking results are properly uploaded to s3
Impact
Footprint tracking is unavailable https://stats.zephyrproject.org/testing/d/SSxr5P6Gk/footprint-tracking?orgId=1
Additional context
Related: since this issue went unnoticed until now, I wonder if there is anything that could be improved process-wise so that footprint tracking is something that's made more prominent and a bit more "in the way" of reviewers' and maintainers' workflow?
There might (or might not!) have been commits made over the past month that may have increased the footprint in ways that people didn't catch when reviewing the PRs, and I believe part of the intention behind the footprint tracking workflow would be to have been able to flag these in time so that an informed decision is made when merging code that's increasing e.g. the kernel or BT stack footprint .
@nashif @stephanosio

Related: since this issue went unnoticed until now, I wonder if there is anything that could be improved process-wise so that footprint tracking is something that's made more prominent and a bit more "in the way" of reviewers' and maintainers' workflow?

We used to have a working group for this with regularly scheduled meetings -- not sure what happened to that and if anyone "owns" this now.

I've confirmed that footprint reports are properly uploaded again ^1, thank you @stephanosio for the prompt fix!


App Header should adjust alignment when used with vertical nav

Select one ...  (check one with "x")
[ ] bug
[x] feature request
[ ] enhancement

Expected behavior
When developing an application that uses both the application header and the vertical navigation component, the elements should not cause alignment discontinuities.  In particular, a collapsed vertical navigation bar should align with some element of the app header.
Actual behavior
The collapsed vertical nav bar aligns with something approximating 2/3 of the app header icons width, which looks weird and like the components were built in isolation rather than as part of cohesive design system.

Environment details

Angular version: 4.3.6

Clarity version: 0.10.5

I agree with this design choice.  The icons should align.

cc: @reddolan

@reddolan ^


typo in src/tree/updater_approx.cc line 55

found a typo src/tree/updater_approx.cc at line 55.
The word "GloablApproxBuilder " should be "GlobalApproxBuilder ".

Good catch, will submit a fix.


No RTL support (Right To Left)

this control does not support RTL flow like arabic language.

Thanks for the problem. I'm currently in the process of relocating, I'll do it when I have free time