Will External Member B2B support with the PowerBI?

From the article below, it mentions that External member isn't supported in Power BI. Only External Guest will do.
May I know any reasons? Any workarounds and plan to allow External Member supported in Power BI?
https://learn.microsoft.com/en-us/azure/active-directory/external-identities/user-properties

Thanks for your comment, @simonkw2000. This issue has been assigned to the doc author for review.

Thanks for your comment, @simonkw2000.
This looks like product feedback or a product question, rather than documentation feedback. You can submit product feedback or feature ideas by going to Power BI Ideas.
You can also engage with the Power BI engineering team by being active in the Power BI Community.


sqlalchemy example should just use cockroachdb and not postgresql in the connection string

I was a little confused by this example in our sqlalchemy docs and saw someone in the community getting indirectly confused due to this.
if __name__ == '__main__':
    # For cockroach demo:
    # DATABASE_URL=postgresql://demo:<demo_password>@<IP_ADDRESS>:26257?sslmode=require
    # For CockroachCloud:
    # DATABASE_URL=postgresql://<username>:<password>@<globalhost>:26257/<cluster_name>.defaultdb?sslmode=verify-full&sslrootcert=<certs_dir>/<ca.crt>
    db_uri = os.environ['DATABASE_URL'].replace("postgresql://", "cockroachdb://")
    try:

I don’t see a good reason to do this string substitution here. We should just have:
if __name__ == '__main__':
    # For cockroach demo:
    # DATABASE_URL=cockroachdb://demo:<demo_password>@<IP_ADDRESS>:26257?sslmode=require
    # For CockroachCloud:
    # DATABASE_URL=cockroachdb://<username>:<password>@<globalhost>:26257/<cluster_name>.defaultdb?sslmode=verify-full&sslrootcert=<certs_dir>/<ca.crt>
    db_uri = os.environ['DATABASE_URL']
    try:

Because the UI already shows cockroachdb://

Also can we make it explicitly clear in a highlighted that when using SQLAlchemy the connection string should start with cockroachdb and not postgres.
See: https://github.com/cockroachdb/sqlalchemy-cockroachdb#install-and-usage

There are two reasons why we decided to modify the connection URL in the example.

SQL Alchemy requires a cockroachdb prefix on the URL to use the correct provider. You'll get errors if your connection string starts with postgresql.
All the connection URLs that the user can copy/paste, either in the Cloud Console or when starting with cockroach demo, are prefixed with postrgresql.

So we had the choice of either making the user hand-edit the copied connection string, or just handle it in code. For simplicity we handled it in code.

Yep that makes sense. For now as we discussed on Slack, let's use add a notice to make it more clear that cockroachdb should be use on the connection string on the doc.


Exit with an error if pulse audio connection fails

I had a hard time debugging, why scream-ivshmem-pulse would not start when run by a libvirt hook. Unable to get any output, I quickly checked the source and found it always exits with 0, thus not displaying an error.
Changed exit code on error to 1
Changed printf to fprintf to print to stderr
edit: just checked the pulseaudio receiver and did the same there.

Thanks :)

Thank you for this great piece of software :)


doc(contributing): Red label grammar adjustment

Description: This PR adjusts the grammar in the Red (C) label description. The phrase "an such issue" is reordered to "such an issue" in an effort to adhere to common English grammar.
BREAKING CHANGE: No
Related issue (if exists): No

We don't accept PRs like this

My apologies. I noticed several other pull requests and approved commits attempting to update any typos and grammar and assumed this pull request was acceptable. Here are a few examples:
https://github.com/swc-project/swc/commit/8905ea2cef0307bc49dbee9e6dadd3700e55f518
https://github.com/swc-project/swc/commit/32bc2f57be06997497d2ad43c2168efb3daaa435
https://github.com/swc-project/swc/commit/cd4a564eead8cb3a6b0b0780ab6b0ba1ee116346
https://github.com/swc-project/swc/commit/776bb91d9b839742aa9ae05d8fa5bc78ae57e33f
https://github.com/swc-project/swc/commit/65d376a91baf19f1f6ddf99e700a5f82b33c9e0c
Could you clarify what you mean by "like this" for me to understand how this pull request is different from the aforementioned?


Allow creating applications and libraries in separate folders for better project structure.

[ V ] Feature request
Currently both applications and libraries placed in same directory.
And there is only one config for this: newProjectRoot
Please add another one: newLibraryRoot - if set, libraries would be stored there.

It would be nice to define the directory/folder as a generate option:
ng g library foo -d libs/datelibs/
ng g library bar -d libs/layoutlibs/
ng g application -d projects/mysubapp

Just figured out that you could generate libraries with namespace.
For example:
ng g library @namespace/xxx
ng g library @namespace/yyy
Will generate two libraries with following structure

@  (at sign) is required for namespace.

For some reason it not mentioned in wiki.
https://github.com/angular/angular-cli/wiki/generate
@filipesilva Should be documented, don't you think ?

@filipesilva this should definitely be documented. can you maybe promote from your backlog?

still, it's open.


Clear button on individual pills/clear all button in multiselect ComboBox

Feature
We are using the ComboBox  in Pineapple and would like to request the ability to clear selected items without having to scroll the list to find them. If there are lots of options in the ComboBox, this can become cumbersome for the user to find/deselect the item.

Additionally, we'd like to request the ability to clear the ComboBox entirely in one click without having to deselect each item. I've put some examples on how react-select works that would be ideal for our use.
Examples
Include a close or "x" button on each "pill" that appears when multiple items are selected. This example is from react-select:

Also include a close or "x" button to the far right of the component that clears all selected elements. This example is also from react-select:

For the "remove" button on individual tags, you can track and 👍 this issue: #907
For the "clear" button in the combobox input, I think it totally makes sense and would be useful even for single-select ComboBox. You might be able to implement it yourself using InputWithDecorations but we should just have it built in.

Ah, sorry I should've double checked for that before I filed this. I will track that work item 😄  Thanks for your reply Mayank!


It works on .net framework 4.0, but minimal is 4.5

Do you have a good reason for still using .Net <4.5?

.net 4.0 is set by default for win 7 users. sometimes they are lazy to install 4.5 or higher.

windows 7 support ends on the 14 january 2020.
People should update their systems rather than using a framework released back in 2011.
I am sorry but you can compile the library yourself if you need 4.0 support.

This has been added while i added support for multiple .Net Frameworks and the .Net Standard 2.0 😄


Tally Compilation Units for Performance Monitoring

Something is much slower than need be

Tally count (bottom up) and time each compilation (top down).


high-performance multi-gpu video2tfrecord

We likely need to redo the code again to support multi-device streaming or use video2numpy, but for now, it works fine. The bottleneck currently is FFmpeg, not GPU or multiprocessing transfer. Using 30 FPS in FFmpeg, we can quantize over 1000 frames per second. However, at 1 FPS, this drops to 150 quantized frames per second.


Shorten PyPI Embed

Description
The PyPI embed can take up a lot of vertical space, which can be a little problematic in rapidly moving chat channels, such as python-general. We could maintain most of the embed's functionality while reducing the space it takes up.
Implementation
There are currently the following fields in the embed: Author, Summary, License, and Python Versions.
One possible implementation would be to limit the number of fields displayed to only include the most helpful ones. For example, we could leave only the description:

All Fields

Just Description

This minified version can be limited to certain channels, such as python general.
Another possible implementation would be to have inline fields. That may work, but we are also limited on horizontal space.

I like this new embed, and I also agree that the license and author is not necessary when providing info.
Do we want the "Summary" field as well, or do we just want the summary below the title?
Either way, I can yoink this one.

I think the latter sounds better. Feel free to grab the issue 😄.


Unit 6 part 2 - gym installation not working

It is not possible to install gym 0.21.0 (see error below, error: metadata-generation-failed).

If I manually install a newer version to avoid this error tied to version 0.21.0 then installation succeeds but the code won't run: env = gym.make(env_id) doesn't work (AttributeError: module 'gym' has no attribute 'GoalEnv').

It is then impossible to run part 2.

Some refs:
https://stackoverflow.com/questions/75833207/install-gym-on-google-coolab-metadata-generation-failed
https://github.com/qgallouedec/panda-gym/issues/19

Hey there we currently updating this notebook before the end of the week with gymnasium I keep you updated

+1, commenting to get an update when a fix is pushed!

+1, commenting to get an update when a fix is pushed!

Yes, I recently pushed two models from unit 6

This PR fixed the issue for me https://github.com/huggingface/deep-rl-class/pull/323

It has also worked for me:
!pip install setuptools==65.5.0 wheel==0.38.4
!pip install -r https://raw.githubusercontent.com/huggingface/deep-rl-class/main/notebooks/unit6/requirements-unit6.txt

Haven't pushed a model yet but it is training (it was impossible to get to this step earlier).

I tried all solutions mentioned here and I am getting this error on current unit 6 notebook.


Cannot initialize quibbler.

Cannot initiate quibbler with initialize_quibbler, got error from FileNotFoundError: Can't identify the notebook path. Can't identify the notebook path." from ipynbname.path().
Package version in Python listed from:
pkg.txt

I presume you are on Jupyter lab?
if so, pls try saving the notebook before running the script

Thanks! I tried on both jupyter notebook and jupyter lab and neither would work for me. I also tried saving the notebook before running without success.

can you pls send the entire exception traceback?

FileNotFoundError                         Traceback (most recent call last)
Cell In[1], line 11
9 import pyquibbler as qb
10 from pyquibbler import iquib
---> 11 qb.initialize_quibbler()
File ~/anaconda3/envs/quibbler/lib/python3.10/site-packages/pyquibbler/utilities/input_validation_utils.py:56, in validate_user_input.._decorator.._wrapper(*args, **kwargs)
54     if not isinstance(arg_values.get(var_name), expected_types):
55         raise InvalidArgumentTypeException(var_name=var_name, expected_type=expected_types)
---> 56 return func(*args, **kwargs)
File ~/anaconda3/envs/quibbler/lib/python3.10/site-packages/pyquibbler/function_overriding/override_all.py:91, in initialize_quibbler(draggable_plots, show_quibs_as_widgets)
88 if IS_QUIBBLER_INITIATED:
89     return
---> 91 within_jupyterlab = create_jupyter_project_if_in_jupyter_lab()
93 func_definitions = create_definitions_for_python_functions()
94 for func, func_definition in func_definitions.items():
File ~/anaconda3/envs/quibbler/lib/python3.10/site-packages/pyquibbler/project/jupyer_project/jupyter_project.py:260, in create_jupyter_project_if_in_jupyter_lab()
258 project = JupyterProject.get_or_create()
259 project.override_quib_persistence_functions()
--> 260 project.listen_for_events()
261 project.get_save_within_notebook_state()
262 project.set_undo_redo_buttons_enable_state()
File ~/anaconda3/envs/quibbler/lib/python3.10/site-packages/pyquibbler/project/jupyer_project/jupyter_project.py:203, in JupyterProject.listen_for_events(self)
195 """
196 Listen for all events from frontend- this will create a callback for any event coming from the frontend,
197 and will run the relevant method for the event.
198 It will then send BACK the response with the request_id specificed in the original event
199 """
201 from pyquibbler.optional_packages.get_IPython import Comm
--> 203 self._jupyter_notebook_path = os.environ.get("JUPYTER_NOTEBOOK", ipynbname.path())
204 self._comm = Comm(target_name='pyquibbler')
205 action_names_to_funcs = {
206     "undo": self.undo,
207     "redo": self.redo,
(...)
213     "cleanup": self._cleanup,
214 }
File ~/anaconda3/envs/quibbler/lib/python3.10/site-packages/ipynbname/init.py:93, in path()
91     root_dir = Path(srv.get('root_dir') or srv['notebook_dir'])
92     return root_dir / path
---> 93 raise FileNotFoundError(FILE_ERROR.format('path'))
FileNotFoundError: Can't identify the notebook path.

So it seems that ipynbname fails identifying the name of the Jupyter notebook.
(see several related ipynbname issues: https://github.com/msm1089/ipynbname/issues)
I merged a patch that allows Quibbler to run even if ipynbname fails.
In this case, saving quibs to notebook is disabled.  To still enable saving to notebook, Quibbler will further suggest setting the notebook file path manually using:
qb.get_project().set_jupyter_notebook_path(...)
Commit: cfd20edc0f0287a874ed7411d19f010c31e22ffd

Thanks, now I'm able to explore Quibbler!

Happy quibbling


[BEAM-2052] Allow dynamic sharding in windowed file sinks

This is a slightly modified and rearranged version of @reuvenlax 's #2647 .
My concerns about it are:

In direct runner, the integration tests of dynamic sharding are vacuous, because direct runner replaces unspecified sharding with fixed sharding at https://github.com/apache/beam/blob/master/runners/direct-java/src/main/java/org/apache/beam/runners/direct/WriteWithShardingFactory.java (applied at https://github.com/apache/beam/blob/master/runners/direct-java/src/main/java/org/apache/beam/runners/direct/DirectRunner.java#L217). However, this is a testing-only concern: other runners don't have this override, so overall the testing is non-vacuous, this is just hard to test against direct runner and I suspect that we probably want these tests to be non-vacuous in direct runner too.

When I removed that override for testing purposes, I noticed that there's a very large number of files being written - primarily, I guess, because the bundles are very small. So large a number of files that the test time for batch with dynamic sharding grows from 21 seconds to 5 minutes. In particular, we write many, many files for each window/pane - presumably because in streaming runners and in direct runner, there's at least 1 bundle per key, and we create at least 1 file per bundle in WriteFiles.Write(Windowed,Unwindowed)Bundles.

Reuven, can you please comment on whether this "at least 1 file per key" is expected behavior in a streaming runner? I suspect that it's not, but then I'm not sure how to fix the PR semantically.
CC: @reuvenlax @davorbonaci @dhalperi

retest this please

This is true for the non-windowed case as well. Fixing this might
require some work in the DirectRunner - @tgroh can comment.

1 bundle per key is true of the Dataflow streaming runner, but is in
general runner dependent. This is true of unwindowed FileBasedSinks as
well, this is not new in the windowing support.

I see two regressions in your PR from mine (one of which was also caught by
a test). Added comments.
On Tue, May 9, 2017 at 4:40 PM, Eugene Kirpichov<EMAIL_ADDRESS>wrote:

This is a slightly modified and rearranged version of @reuvenlax
https://github.com/reuvenlax 's #2647
https://github.com/apache/beam/pull/2647 .
My concerns about it are:

In direct runner, the integration tests of dynamic sharding are
vacuous, because direct runner replaces unspecified sharding with fixed
sharding at https://github.com/apache/beam/blob/master/runners/
direct-java/src/main/java/org/apache/beam/runners/direct/
WriteWithShardingFactory.java
https://github.com/apache/beam/blob/master/runners/direct-java/src/main/java/org/apache/beam/runners/direct/WriteWithShardingFactory.java
(applied at https://github.com/apache/beam/blob/master/runners/
direct-java/src/main/java/org/apache/beam/runners/direct/
DirectRunner.java#L217
https://github.com/apache/beam/blob/master/runners/direct-java/src/main/java/org/apache/beam/runners/direct/DirectRunner.java#L217).
However, this is a testing-only concern: other runners don't have this
override, so overall the testing is non-vacuous, this is just hard to test
against direct runner and I suspect that we probably want these tests to be
non-vacuous in direct runner too.
2.
When I removed that override for testing purposes, I noticed that
there's a very large number of files being written - primarily, I guess,
because the bundles are very small. So large a number of files that the
test time for batch with dynamic sharding grows from 21 seconds to 5
minutes. In particular, we write many, many files for each window/pane -
presumably because in streaming runners and in direct runner, there's at
least 1 bundle per key, and we create at least 1 file per bundle in
WriteFiles.Write(Windowed,Unwindowed)Bundles.
Reuven, can you please comment on whether this "at least 1 file per key"
is expected behavior in a streaming runner? I suspect that it's not, but
then I'm not sure how to fix the PR semantically.
CC: @reuvenlax https://github.com/reuvenlax @davorbonaci
https://github.com/davorbonaci @dhalperi https://github.com/dhalperi
You can view, comment on, or merge this pull request online at:
https://github.com/apache/beam/pull/3023
Commit Summary

Implement dynamic-sharding for windowed file outputs, and add an
integration test.
Renames FileBasedSink inner classes
Simpler code for setting shard numbers on results in FileBasedSink
Splits WriteBundles into windowed/unwindowed versions

File Changes

M examples/java/src/main/java/org/apache/beam/examples/
WindowedWordCount.java
https://github.com/apache/beam/pull/3023/files#diff-0 (7)
M examples/java/src/main/java/org/apache/beam/examples/
common/WriteOneFilePerWindow.java
https://github.com/apache/beam/pull/3023/files#diff-1 (24)
M examples/java/src/test/java/org/apache/beam/examples/
WindowedWordCountIT.java
https://github.com/apache/beam/pull/3023/files#diff-2 (29)
M runners/core-construction-java/src/test/java/org/apache/
beam/runners/core/construction/PTransformMatchersTest.java
https://github.com/apache/beam/pull/3023/files#diff-3 (2)
M runners/direct-java/src/test/java/org/apache/beam/runners/direct/
WriteWithShardingFactoryTest.java
https://github.com/apache/beam/pull/3023/files#diff-4 (2)
M runners/google-cloud-dataflow-java/src/main/java/org/apache/
beam/runners/dataflow/util/DefaultCoderCloudObjectTransla
torRegistrar.java
https://github.com/apache/beam/pull/3023/files#diff-5 (2)
M sdks/java/core/src/main/java/org/apache/beam/sdk/io/AvroSink.java
https://github.com/apache/beam/pull/3023/files#diff-6 (14)
M sdks/java/core/src/main/java/org/apache/beam/sdk/io/
FileBasedSink.java
https://github.com/apache/beam/pull/3023/files#diff-7 (316)
M sdks/java/core/src/main/java/org/apache/beam/sdk/io/
TFRecordIO.java https://github.com/apache/beam/pull/3023/files#diff-8
(16)
M sdks/java/core/src/main/java/org/apache/beam/sdk/io/TextSink.java
https://github.com/apache/beam/pull/3023/files#diff-9 (14)
M sdks/java/core/src/main/java/org/apache/beam/sdk/io/
WriteFiles.java
https://github.com/apache/beam/pull/3023/files#diff-10 (259)
M sdks/java/core/src/test/java/org/apache/beam/sdk/io/
FileBasedSinkTest.java
https://github.com/apache/beam/pull/3023/files#diff-11 (162)
M sdks/java/core/src/test/java/org/apache/beam/sdk/io/
SimpleSink.java
https://github.com/apache/beam/pull/3023/files#diff-12 (4)
M sdks/java/io/xml/src/main/java/org/apache/beam/sdk/io/
xml/XmlSink.java
https://github.com/apache/beam/pull/3023/files#diff-13 (8)

Patch Links:

https://github.com/apache/beam/pull/3023.patch
https://github.com/apache/beam/pull/3023.diff

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/3023, or mute the thread
https://github.com/notifications/unsubscribe-auth/AUGE1ef1ECr_2_E3BIuSK-oH13NV2a1Lks5r4Pl5gaJpZM4NWAz-
.

Run Spark ValidatesRunner

Run Flink ValidatesRunner

Run Dataflow ValidatesRunner

I merged this after manually running WindowedWordCountIT with Flink runner.
Looking at the dependency error - it looks pretty clearly unrelated to my PR though, the diff doesn't even mention "findbugs"...

 Will send a fix PR right away.

DO NOT cherrypick without https://github.com/apache/beam/pull/3059


Add http2 service on VCR

Added http2 service on VCR server.
Added tests.

Codecov Report

Merging #1984 (51702be) into master (70dceb2) will decrease coverage by 11.87%.
The diff coverage is 88.00%.

@@              Coverage Diff              @@
##             master    #1984       +/-   ##
=============================================
- Coverage     73.82%   61.94%   -11.88%
+ Complexity     9863     8082     -1781
=============================================
  Files           706      706
  Lines         54144    54179       +35
  Branches       6729     6735        +6
=============================================
- Hits          39970    33561     -6409
- Misses        12048    18474     +6426
- Partials       2126     2144       +18

Impacted Files
Coverage Δ

...rc/main/java/com/github/ambry/cloud/VcrServer.java
83.21% <88.00%> (-1.88%)
:arrow_down:

...rc/main/java/com/github/ambry/store/TimeRange.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...java/com/github/ambry/notification/UpdateType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/java/com/github/ambry/store/CostBenefitInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...m/github/ambry/accountstats/AccountStatsStore.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...om/github/ambry/store/CompactAllPolicyFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...thub/ambry/notification/BlobReplicaSourceType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...github/ambry/store/CompactionPolicySwitchInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...github/ambry/store/StatsBasedCompactionPolicy.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...hub/ambry/store/HybridCompactionPolicyFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 150 more

Continue to review full report at Codecov.

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

@ankagrawal @Arun-LinkedIn   this PR is ready. Hopefully we can merge it this week.


make clean fails

It appears that make clean fails when run on the Warehouse repository on my machine.
Here is the error message I got:
$ make clean
rm -rf warehouse/static/components
rm -rf warehouse/static/dist
rm: cannot remove 'warehouse/static/dist/fonts/fontawesome-webfont.a7bbeb58.woff': Permission denied
rm: cannot remove 'warehouse/static/dist/fonts/fontawesome-webfont.1fb8a705.eot.gz': Permission denied
rm: cannot remove 'warehouse/static/dist/fonts/fontawesome-webfont.e1dd9adb.ttf.br': Permission denied
rm: cannot remove 'warehouse/static/dist/fonts/fontawesome-webfont.1fb8a705.eot.br': Permission denied
rm: cannot remove 'warehouse/static/dist/fonts/FontAwesome.b5744b64.otf.gz': Permission denied
rm: cannot remove 'warehouse/static/dist/fonts/FontAwesome.b5744b64.otf': Permission denied
rm: cannot remove 'warehouse/static/dist/fonts/fontawesome-webfont.912ec66d.svg.gz': Permission denied
rm: cannot remove 'warehouse/static/dist/fonts/fontawesome-webfont.e1dd9adb.ttf.gz': Permission denied
rm: cannot remove 'warehouse/static/dist/fonts/fontawesome-webfont.912ec66d.svg': Permission denied
rm: cannot remove 'warehouse/static/dist/fonts/fontawesome-webfont.68560c51.woff2': Permission denied
rm: cannot remove 'warehouse/static/dist/fonts/fontawesome-webfont.a7bbeb58.woff.gz': Permission denied
rm: cannot remove 'warehouse/static/dist/fonts/FontAwesome.b5744b64.otf.br': Permission denied
rm: cannot remove 'warehouse/static/dist/fonts/fontawesome-webfont.e1dd9adb.ttf': Permission denied
rm: cannot remove 'warehouse/static/dist/fonts/fontawesome-webfont.1fb8a705.eot': Permission denied
rm: cannot remove 'warehouse/static/dist/fonts/fontawesome-webfont.912ec66d.svg.br': Permission denied
rm: cannot remove 'warehouse/static/dist/manifest.json.br': Permission denied
rm: cannot remove 'warehouse/static/dist/images/favicon.6a76275d.ico': Permission denied
rm: cannot remove 'warehouse/static/dist/images/blue-cube.e6165d35.svg': Permission denied
rm: cannot remove 'warehouse/static/dist/images/logo-small.972290d5.svg.gz': Permission denied
rm: cannot remove 'warehouse/static/dist/images/favicon.6a76275d.ico.gz': Permission denied
rm: cannot remove 'warehouse/static/dist/images/logo-small.972290d5.svg': Permission denied
rm: cannot remove 'warehouse/static/dist/images/white-cube.2bc660f6.png': Permission denied
rm: cannot remove 'warehouse/static/dist/images/white-cube-small.cb1d84fb.png': Permission denied
rm: cannot remove 'warehouse/static/dist/images/logo-large.87e8a3ef.svg.br': Permission denied
rm: cannot remove 'warehouse/static/dist/images/blue-cube.e6165d35.svg.br': Permission denied
rm: cannot remove 'warehouse/static/dist/images/blue-cube-small.72d3cc02.png': Permission denied
rm: cannot remove 'warehouse/static/dist/images/white-cube.8c3a6fe9.svg.br': Permission denied
rm: cannot remove 'warehouse/static/dist/images/favicon.6a76275d.ico.br': Permission denied
rm: cannot remove 'warehouse/static/dist/images/logo-large.87e8a3ef.svg': Permission denied
rm: cannot remove 'warehouse/static/dist/images/blue-cube.e6165d35.svg.gz': Permission denied
rm: cannot remove 'warehouse/static/dist/images/history-line.ded09064.png': Permission denied
rm: cannot remove 'warehouse/static/dist/images/white-cube.8c3a6fe9.svg.gz': Permission denied
rm: cannot remove 'warehouse/static/dist/images/logo-small.972290d5.svg.br': Permission denied
rm: cannot remove 'warehouse/static/dist/images/logo-large.87e8a3ef.svg.gz': Permission denied
rm: cannot remove 'warehouse/static/dist/images/white-cube.8c3a6fe9.svg': Permission denied
rm: cannot remove 'warehouse/static/dist/css/font-awesome.css.4741d860.map.gz': Permission denied
rm: cannot remove 'warehouse/static/dist/css/warehouse.css.b3ca61fa.map.gz': Permission denied
rm: cannot remove 'warehouse/static/dist/css/warehouse.css.b3ca61fa.map.br': Permission denied
rm: cannot remove 'warehouse/static/dist/css/warehouse.8aff0405.css.gz': Permission denied
rm: cannot remove 'warehouse/static/dist/css/font-awesome.css.4741d860.map.br': Permission denied
rm: cannot remove 'warehouse/static/dist/css/font-awesome.628aab96.css.gz': Permission denied
rm: cannot remove 'warehouse/static/dist/css/font-awesome.css.4741d860.map': Permission denied
rm: cannot remove 'warehouse/static/dist/css/warehouse.css.b3ca61fa.map': Permission denied
rm: cannot remove 'warehouse/static/dist/css/font-awesome.628aab96.css.br': Permission denied
rm: cannot remove 'warehouse/static/dist/css/warehouse.8aff0405.css': Permission denied
rm: cannot remove 'warehouse/static/dist/css/warehouse.8aff0405.css.br': Permission denied
rm: cannot remove 'warehouse/static/dist/css/font-awesome.628aab96.css': Permission denied
rm: cannot remove 'warehouse/static/dist/manifest.json': Permission denied
rm: cannot remove 'warehouse/static/dist/manifest.json.gz': Permission denied
rm: cannot remove 'warehouse/static/dist/js/warehouse.80c5a887.js': Permission denied
rm: cannot remove 'warehouse/static/dist/js/warehouse.80c5a887.js.br': Permission denied
rm: cannot remove 'warehouse/static/dist/js/warehouse.js.2932ed70.map': Permission denied
rm: cannot remove 'warehouse/static/dist/js/warehouse.js.2932ed70.map.br': Permission denied
rm: cannot remove 'warehouse/static/dist/js/vendor/zxcvbn.js.gz': Permission denied
rm: cannot remove 'warehouse/static/dist/js/vendor/zxcvbn.js': Permission denied
rm: cannot remove 'warehouse/static/dist/js/warehouse.80c5a887.js.gz': Permission denied
rm: cannot remove 'warehouse/static/dist/js/warehouse.js.2932ed70.map.gz': Permission denied
Makefile:156: recipe for target 'clean' failed
make: *** [clean] Error 1

@lgh2 what does ls -lah warehouse/static give you when run from the same location?

This is the output:
$ ls -lah
total 596K
drwxr-xr-x  14 user user 4.0K Feb 14 16:28 .
drwxr-xr-x   6 user user 4.0K Feb 12 16:34 ..
-rw-r--r--   1 user user   25 Jan  8 11:18 .babelrc
drwxr-xr-x   2 user user 4.0K Jan 15 17:45 bin
-rw-r--r--   1 user user  118 Nov  5 15:08 codecov.yml
-rw-r--r--   1 user user  406 Nov  5 15:08 CONTRIBUTING.rst
-rw-r--r--   1 user user  891 Nov  5 15:08 .coveragerc
drwxr-xr-x   2 user user 4.0K Feb 14 12:59 dev
-rw-r--r--   1 user user 2.4K Jan 15 17:45 docker-compose.yml
-rw-r--r--   1 user user 6.6K Jan 15 17:45 Dockerfile
-rw-r--r--   1 user user  954 Jan 15 17:45 Dockerfile.static
-rw-r--r--   1 user user   65 Jan  8 11:18 .dockerignore
drwxr-xr-x   5 user user 4.0K Feb 14 16:28 docs
drwxr-xr-x   8 user user 4.0K Feb 14 16:34 .git
-rw-r--r--   1 user user  271 Dec 19 15:21 .gitignore
-rw-r--r--   1 user user 8.2K Feb 14 16:28 Gulpfile.babel.js
drwxr-xr-x   2 root root  12K Feb  9 15:48 htmlcov
-rw-r--r--   1 user user 9.5K Nov  5 15:08 LICENSE
-rw-r--r--   1 user user 5.6K Feb 14 16:28 Makefile
drwxr-xr-x 147 user user 4.0K Nov 28 18:03 node_modules
-rw-r--r--   1 user user 1.7K Jan 15 17:45 package.json
-rw-r--r--   1 user user 426K Jan 15 17:45 package-lock.json
drwxr-xr-x   2 user user 4.0K Nov  5 15:08 policies
-rw-r--r--   1 user user  208 Dec 19 15:21 Procfile
-rw-r--r--   1 user user 1.5K Jan 29 13:26 README.rst
drwxr-xr-x   2 user user 4.0K Feb 14 16:28 requirements
-rw-r--r--   1 user user   52 Nov  5 15:08 requirements.txt
-rw-r--r--   1 user user   13 Dec 19 15:21 runtime.txt
-rw-r--r--   1 user user 1.2K Feb  9 15:41 .sass-lint.yml
-rw-r--r--   1 user user  332 Nov  5 15:08 setup.cfg
drwxr-xr-x   3 user user 4.0K Feb  9 15:52 .state
drwxr-xr-x   6 user user 4.0K Feb  9 15:41 tests
-rw-r--r--   1 user user 1.9K Jan  8 11:18 .travis.yml
drwxr-xr-x   2 user user 4.0K Jan 23 13:27 vcl
drwxr-xr-x  21 user user 4.0K Feb 14 16:28 warehouse

@lgh2 That's the output of ls -lah, can you try ls -lah warehouse/static?

I apologize! Here it is:
$ ls -lah warehouse/static
total 24K
drwxr-xr-x  6 user user 4.0K Feb 14 13:21 .
drwxr-xr-x 21 user user 4.0K Feb 14 16:28 ..
drwxr-xr-x  6 root root 4.0K Feb 14 13:21 dist
drwxr-xr-x  2 user user 4.0K Dec 19 15:21 images
drwxr-xr-x  3 user user 4.0K Feb 14 16:28 js
drwxr-xr-x  8 user user 4.0K Feb 12 14:14 sass

@lgh2 It's a little odd that a) you have a dist directory here and b) that it's owned by root. Did you sudo gulp dist or something similar at one point? (Shouldn't be necessary, with or without sudo).
You should be able to sudo rm -rf warehouse/static/dist to remove that directory, and then make clean and make purge should work as expected.

No, I don't think I've ever run sudo gulp dist .
Thank you for the fix -- it worked!

Hmm, maybe a sudo make serve or a sudo docker-compose build static or something could have done it?
At any rate, developers shouldn't be running any of those commands as root, so I don't see this being a reoccurring issue.

I'm agreeing with @di and closing this issue.
None of our documentation tells people to run those commands with sudo. If one of us runs into the make clean issue again then we'll be in a better position to look at our bash histories and figure out why we inappropriately used sudo or what other command got us into this situation.
Thanks all.

We ran into this problem again during a sprint night last week but I didn't think to capture people's bash histories.
https://github.com/brainwane/warehouse/tree/make-clean-issue

I think this could be resolved by fixing https://github.com/pypa/warehouse/issues/1536, which would remove the need to ever write or remove static files to/from the user's filesystem.

I was hoping that #5084 fixed this, but just now I ran into the issue again. I got Permission denied errors when running make clean. I ran ls -lah warehouse/static and the dist line said:

drwxr-xr-x  6 root    root    4.0K Jun  3 14:28 dist

After I ran sudo rm -rf warehouse/static/dist I was able to make clean and make purge without difficulty. And I know -- because this is a fresh machine and I looked through my bash history -- that I
never ran sudo make serve or sudo docker-compose build static, nor ran sudo with any Make commands or with any Warehouse-specific docker commands (I did run sudo docker run hello-world when checking that Docker had installed correctly).

Just ran into this again.

The issue here is that we mount the warehouse directory into the container: https://github.com/pypa/warehouse/blob/6721a8aba73e13129f4400a64ff273099ce52850/docker-compose.yml#L65
When our static assets are compiled, this produces the warehouse/static/dist directory, which is then written onto the host file system.
Originally make clean was trying to remove this directory. This is unnecessary now as everything happens within the container. I removed this in #8063.
I tried to exclude these directories from being synced back to the host as well, but the only way to do this in docker-compose causes other permissions issues when attempting to modify these directories within the container.
I suspect there may still be issues on certain platforms when gulp attempts to remove a directory that was created in the container and was synced to the host, but for now since this issue is about make clean, I'm going to consider this resolved.


Update iothub.json

Update details on fallback properties
Choose a PR Template
Switch to "Preview" on this description then select one of the choices below.
Click here to open a PR for a Data Plane API.
Click here to open a PR for a Control Plane (ARM) API.

@rkmanda @lirenhe  Can you let us know what the process is for getting this PR reviewed and merged? Thanks

/azp run

The model validation errors are not related to this PR.


AM rest services update

@Bobfrat , @birdage , if you have a moment, please take a peak at the clean up of the assets GET endpoint and let me know if removing any of those keys will mess up your stuff.
I checked around a lot, and didn't see that it would - but I'd feel better if you had a look.
@DanielJMaher , This is to support all that AM work for the UI.

Dang, this is likely going to fail - need to update some tests


Install error

This is for a dxd/es snapshot downloaded today, 13-Jul-15.
(pypyenv)[cyli6s85]/scratch/dxr-es-based> dxr-es/peep.py install -r dxr-es/requirements.txt
Oh no! Peep had a problem while trying to do stuff. Please write up a bug report
with the specifics so we can fix it:
https://github.com/erikrose/peep/issues/new
Here are some particulars you can copy and paste into the bug report:

peep: (2, 4, 1)
python: '2.7.5 (default, Jun 17 2014, 18:11:42) \n[GCC 4.8.2 20140120 (Red Hat 4.8.2-16)]'
pip: '7.0.1'
Command line:  ['dxr-es/peep.py', 'install', '-r', 'dxr-es/requirements.txt']
Traceback (most recent call last):
File "dxr-es/peep.py", line 901, in
exit(main())
File "dxr-es/peep.py", line 873, in main
return commandsargv[1]
File "dxr-es/peep.py", line 833, in peep_install
for path in req_paths))
File "dxr-es/peep.py", line 833, in
for path in req_paths))
File "dxr-es/peep.py", line 794, in downloaded_reqs_from_path
finder = package_finder(argv)
File "dxr-es/peep.py", line 335, in package_finder
return PackageFinder(index_urls=index_urls, **kwargs)
TypeError: init() got an unexpected keyword argument 'use_wheel'

Your version of pip is too new to be used with peep. We're working on it. In the meantime, you can either downgrade to pip <7.0, or you can throw security to the wind and call pip instead of peep.


[Feature Request]: support Mistral embeddings

Describe the problem
Chroma doesn't provide an embedding function for Mistral.
Describe the proposed solution
Chroma should provide an embedding function for Mistral.
Alternatives considered
No response
Importance
nice to have
Additional Information
No response

Adding refs:

https://docs.mistral.ai/capabilities/embeddings/#mistral-embeddings-api
https://pypi.org/project/mistralai/
Example EF to guide the impl - https://github.com/chroma-core/chroma/blob/e04a71e5465f2bba94b1fc16f14b74979366cee7/chromadb/utils/embedding_functions.py#L229

I want to work on this issue.


Thread-safety problem in RoundRobinJedisPool

First of all, issue raised up from io.codis.jodis:0.4.1. We have frequently ran into exception Pool not open which threw by RedisPool

RoundRobinJedisPool.resetPools is not thread-safety. Although RoundRobinJedisPool#pools is carefully declared volatile and ImmutableList, the inner pool (thus, JedisPool) can still be modified. At the end of resetPools, it starts closing pool(s) that not exists under zookeeper path:
        for (PooledObject pool: addr2Pool.values()) {
            LOG.info("Remove proxy: " + pool.addr);
            pool.pool.close();
        }

this operation may close pool(s) already inited by other thread (and successfully assigned to RoundRobinJedisPool#pools) by mistake. For example:

pools holds JedisPool: a, b, c
Thread A reads a, b, c from zookeeper
Thread B reads a, b from zookeeper
Thread B build new list instance, then assign to pools (anyway it loses the competition)
Thread A build new list instance (note that reference of a, b, c were copied), then assign to pools
Thread B start closing resources c

This makes pools contains a closed JedisPool c.

Is it possible that two threads want to call resetPools concurrently? IIRC, the curator will always uses a single threaded thread pool to execute the callbacks. If not, then we need to add a synchronized modifier to the resetPools method.

@Apache9 THANKS for your reply.
We doubt that the ctor of RoundRobinJedisPool may lead to multi-access to resetPools.
In our case, we call the build() to create an instance (auto wired by spring actually).
    pool = RoundRobinJedisPool.create()
        .curatorClient(zkServer, zkTimeout)
        .zkProxyDir(zkCodisProxyDir)
        .build();

There will be two thread finally reach resetPools which is beyond our control. To illustrate,
private RoundRobinJedisPool(CuratorFramework curatorClient, boolean closeCurator,
            String zkProxyDir, JedisPoolConfig poolConfig, int connectionTimeoutMs, int soTimeoutMs,
            String password, int database, String clientName) {
        ···
        watcher = new PathChildrenCache(curatorClient, zkProxyDir, true);
        watcher.getListenable().addListener(new PathChildrenCacheListener() {

            private void logEvent(PathChildrenCacheEvent event) {
                ···
                LOG.info(msg.toString());
            }

            @Override
            public void childEvent(CuratorFramework client, PathChildrenCacheEvent event)
                    throws Exception {
                logEvent(event);
                if (RESET_TYPES.contains(event.getType())) {
                    resetPools();                                        // NOTE this calling
                }
            }
        });
        try {
            watcher.start(BUILD_INITIAL_CACHE);      //Thread B
        } catch (Exception e) {
            close();
            throw new JedisException(e);
        }
        resetPools();                                                    //Thread A
    }

During our application start-up, childEvent comes at the same time (we can see the log event).
Or else, is there any misunderstanding about the mechanism?

我正好碰到这个问题，启动的时候, 主线程resetpool， 然后listener线程听到node add消息也resetpool，造成两倍的连接数，pools的连接池里还是对的，但有多余的pool生成了没被删除
2018-07-26 15:53:37,883 [main-EventThread] INFO  o.a.c.f.s.ConnectionStateManager - State change: CONNECTED
2018-07-26 15:53:38,023 [Curator-PathChildrenCache-0] INFO  c.s.i.i.jodis.RoundRobinJedisPool - Receive child event: type=CONNECTION_RECONNECTED, no data
2018-07-26 15:53:38,057 [Curator-PathChildrenCache-0] INFO  c.s.i.i.jodis.RoundRobinJedisPool - Receive child event: type=CHILD_ADDED, path=/jodis/codis-iptv/proxy-72f4013ead5c6a213432478a03047397, stat=8069,8069,1532499605339,1532499605339,0,0,0,100077600475054269,201,0,8069
, bytes length=201
2018-07-26 15:53:38,207 [Curator-PathChildrenCache-0] INFO  c.s.i.i.jodis.RoundRobinJedisPool - Add new proxy: <IP_ADDRESS>:29000
2018-07-26 15:53:38,208 [Curator-PathChildrenCache-0] INFO  c.s.i.i.jodis.RoundRobinJedisPool - Add new proxy: <IP_ADDRESS>:29000
2018-07-26 15:53:38,210 [Curator-PathChildrenCache-0] INFO  c.s.i.i.jodis.RoundRobinJedisPool - Receive child event: type=CHILD_ADDED, path=/jodis/codis-iptv/proxy-adb4cbd6c40c8705d99eaf4409588067, stat=8091,8091,1532581151277,1532581151277,0,0,0,100077600475054274,201,0,8091
, bytes length=201
2018-07-26 15:53:38,210 [main] INFO  c.s.i.i.jodis.RoundRobinJedisPool - Add new proxy: <IP_ADDRESS>:29000
2018-07-26 15:53:38,213 [main] INFO  c.s.i.i.jodis.RoundRobinJedisPool - Add new proxy: <IP_ADDRESS>:29000

OK, so the problem is the resetPools during start up. Let me prepare a fix.

a simple solution: first start the cache and resetpools, then add the listenner. to avoid the small possibility of concurrency, sleep  a while between the two action.

@weilinqwe Maybe just add a synchonized modifier to resetPools will be more effective. Since the curator will exactly use single thread executor to process child event.
public class PathChildrenCache implements Closeable
{
    private final CloseableExecutorService executorService;
    ···
    public PathChildrenCache(CuratorFramework client, String path, boolean cacheData)
    {
        this(client, path, cacheData, false, new CloseableExecutorService(Executors.newSingleThreadExecutor(defaultThreadFactory), true));
    }
    ···
}

So, synchonized will avoid the currency during instantiation with little drawback.

understood. startup is the only chance involving concurrency.

Has it already been fixed? @Apache9

Please try 0.5.1.


Earn guide has the wallet theme

Not sure why the Earn guide needed up with the theme for Wallets
https://lightsats.com/guide/claim

@juangb87 this should be fixed now!


r2 implementation

Aims to close https://github.com/cloudflare/miniflare/issues/276
TODO:

[x] Bucket testing.
[x] Check that all error responses match.
[x] Check that all implementations are the same.
[x] Get windows tests working

Since I've sent you the ListObjects implementation, we've got a report of another issue & I have a couple of bug fixes pending. We should probably make sure to update the Miniflare implementation to match.
Also, we've added range in the returned object on get if a range is specified (the actual range encountered is returned). Range logic is very tricky and probably needs to be compared against what R2 is actually doing to make sure things are right (I haven't checked that part).
I'm also in the process of plumbing through startAfter as a list option - probably should be rolled into here (but again - double-check against the latest version of list in R2).
It's probably a good idea for whoever is reviewing this on the cloudflare side (not me - I don't have the cycles aside from providing on-demand consultation) to look at the real R2 implementation and compare/contrast:

ListObjects implementation
Conditional handling (conditional puts + general conditionals)
Range handling

Also on-deck for being implemented (& might be worth prototyping the bindings for it in Miniflare):

Multipart
Copy object
Move object

I have some of the shape of the APIs sketched out already (multipart in particular) so please coordinate with me on Discord whenever someone takes that on.


[Compute] 2020-06-01 API version

https://github.com/Azure/azure-rest-api-specs/pull/9167
https://github.com/Azure/azure-rest-api-specs/pull/9937
https://github.com/Azure/azure-rest-api-specs/pull/9962

@isra-fel Could you review and merge this PR?


TypeError: Cannot read property 'array' of undefined

Description
Invariant Violation: "main" has not been registered. This can happen if:

Metro (the local dev server) is run from the wrong folder. Check if Metro is running, stop it and restart it in the current project.
A module failed to load due to an error and AppRegistry.registerComponent wasn't called., js engine: hermes

React Native Version
0.72.4
Output of npx react-native info
info Fetching system and libraries information...
System:
OS: macOS 10.15.7
CPU: (4) x64 Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz
Memory: 33.98 MB / 8.00 GB
Shell:
version: 5.7.1
path: /bin/zsh
Binaries:
Node:
version: 18.17.1
path: /usr/local/bin/node
Yarn: Not Found
npm:
version: 9.8.1
path: /usr/local/bin/npm
Watchman: Not Found
Managers:
CocoaPods:
version: 1.12.1
path: /Users/javascript/.gem/ruby/3.2.0/bin/pod
SDKs:
iOS SDK:
Platforms:
- iOS 14.4
- DriverKit 20.2
- macOS 11.1
- tvOS 14.3
- watchOS 7.2
Android SDK:
API Levels:
- "29"
- "30"
- "31"
- "33"
Build Tools:
- 27.0.3
- 28.0.2
- 28.0.3
- 29.0.2
- 30.0.2
- 30.0.3
- 31.0.0
- 33.0.0
System Images:
- android-29 | Intel x86 Atom_64
- android-30 | Google APIs Intel x86 Atom
- android-30 | Google Play Intel x86 Atom
- android-30 | Google Play Intel x86 Atom_64
Android NDK: Not Found
IDEs:
Android Studio: 4.1 AI-201.87<IP_ADDRESS>53283
Xcode:
version: 12.4/12D4e
path: /usr/bin/xcodebuild
Languages:
Java:
version: 11.0.10
path: /usr/bin/javac
Ruby:
version: 3.2.0
path: /Users/javascript/.rubies/ruby-3.2.0/bin/ruby
npmPackages:
"@react-native-community/cli": Not Found
react:
installed: 18.2.0
wanted: 18.2.0
react-native:
installed: 0.72.4
wanted: 0.72.4
react-native-macos: Not Found
npmGlobalPackages:
"react-native": Not Found
Android:
hermesEnabled: Not found
newArchEnabled: Not found
iOS:
hermesEnabled: Not found
newArchEnabled: Not found
Steps to reproduce
Fix the ViewPropTypes problem or remove it if not useful anymore, so it won't get any error in the future.
Snack, screenshot, or link to a repository

cannot be investigated without code, but it looks llike you miss importing:
import PropTypes from 'prop-types'

Description
Invariant Violation: "main" has not been registered. This can happen if:

Metro (the local dev server) is run from the wrong folder. Check if Metro is running, stop it and restart it in the current project.
A module failed to load due to an error and AppRegistry.registerComponent wasn't called., js engine: hermes

React Native Version
0.72.4
Output of npx react-native info
info Fetching system and libraries information... System: OS: macOS 10.15.7 CPU: (4) x64 Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz Memory: 33.98 MB / 8.00 GB Shell: version: 5.7.1 path: /bin/zsh Binaries: Node: version: 18.17.1 path: /usr/local/bin/node Yarn: Not Found npm: version: 9.8.1 path: /usr/local/bin/npm Watchman: Not Found Managers: CocoaPods: version: 1.12.1 path: /Users/javascript/.gem/ruby/3.2.0/bin/pod SDKs: iOS SDK: Platforms: - iOS 14.4 - DriverKit 20.2 - macOS 11.1 - tvOS 14.3 - watchOS 7.2 Android SDK: API Levels: - "29" - "30" - "31" - "33" Build Tools: - 27.0.3 - 28.0.2 - 28.0.3 - 29.0.2 - 30.0.2 - 30.0.3 - 31.0.0 - 33.0.0 System Images: - android-29 | Intel x86 Atom_64 - android-30 | Google APIs Intel x86 Atom - android-30 | Google Play Intel x86 Atom - android-30 | Google Play Intel x86 Atom_64 Android NDK: Not Found IDEs: Android Studio: 4.1 AI-201.87<IP_ADDRESS>53283 Xcode: version: 12.4/12D4e path: /usr/bin/xcodebuild Languages: Java: version: 11.0.10 path: /usr/bin/javac Ruby: version: 3.2.0 path: /Users/javascript/.rubies/ruby-3.2.0/bin/ruby npmPackages: "@react-native-community/cli": Not Found react: installed: 18.2.0 wanted: 18.2.0 react-native: installed: 0.72.4 wanted: 0.72.4 react-native-macos: Not Found npmGlobalPackages: "react-native": Not Found Android: hermesEnabled: Not found newArchEnabled: Not found iOS: hermesEnabled: Not found newArchEnabled: Not found
Steps to reproduce
Fix the ViewPropTypes problem or remove it if not useful anymore, so it won't get any error in the future.
Snack, screenshot, or link to a repository

@PrinceSP did you find a solution for this ?
Even i  am having same issue


installation problem

Hi,
I got the following error during installation
mkdir /home/chuan/torch/install/lib/luarocks/rocks/display/scm-0/lua/display && cp -a static plugins /home/chuan/torch/install/lib/luarocks/rocks/display/scm-0/lua/display
cp: cannot stat ‘plugins’: No such file or directory
It seems to be related to the recent update in display-scm-0.rockspec
Any idea how to fix this? Thanks.

Sorry about that, I've been working on adding a framework for plugins. Fixed.


Allow more flexibility for the template

It would be nice if we could have a template option that allows a function argument. Some context should be available as function arguments. E.g.:
opts = {
  default = {
    template = function(context)
      return "Cool: " + context.file_path
    end
  },
},

The function return value should be a possibly parameterized template string. Thus, it can contain the placeholder values such as $CURSOR or similar. And the context argument should have the relevant context (more or less the placeholder variables, but perhaps there are some more relevant stuff).
Personally, I would see myself using this in some specific cases where I want to put the captured images into a custom directory. But even though the text I'm writing is in a file located elsewhere, I still want the link to be fully relative to the media path. I can use the $FILE_NAME for this, but then I have to keep things flat.

@lervag Does PR #28 work as intended for you?

No, not quite. The main motivation from my side was to have $FILE_PATH available such that I could do e.g. vim.fn.fnamemodify(...) or similar on the result.

I looked at the code, and it seems like what I'm asking might require some larger refactoring. It seems you are evaluating the config function when initializing the plugin. My suggestion would only be relevant if the config value is evaluated as a function only when it is used (and every time it is used.
No, scratch that, I'm partly wrong. But: Instead of evaluating the template function in .get_option, you should evaluate the function the .insert_markup function only after the placeholders are ready to be interpolated, here:
https://github.com/HakonHarnes/img-clip.nvim/blob/bfb7d05cec1a52abff9ac3b4f3108a574a3bf2e0/lua/img-clip/markup.lua#L89-L92
That is, above these lines, you should check if template is a function, and if it is, then you replace it with the result of the function where you pass a context object that contains the placeholder values.
I hope I'm being more clear on what I'm thinking now?

So, what I wanted was to have the values in placeholders interpolated already at this point, before you call val(placeholders). And in this case, I believe it is better to call the object "context" or similar instead of placeholders, to avoid confusing the placeholder names with the interpolated values. (From PR)

That makes sense to me. I understand now. This certainly has more value than simply returning the placeholder strings.

That is, above these lines, you should check if template is a function, and if it is, then you replace it with the result of the function where you pass a context object that contains the placeholder values.

That's probably the easiest solution with the current state of the code. I'll look into it.

Cool, thanks!


how would you run with the production flag?

Ahh, okay - you just need to set NODE_ENV to production; closing
    webpack : {
      options : require('./webpack.config'),
      development: {
        plugins: [
          new webpack.optimize.DedupePlugin()
        ]
      },
      production: {
        plugins: [
          new webpack.DefinePlugin({
            "process.env": {
              "NODE_ENV": JSON.stringify("production")
            }
          }),
          new webpack.optimize.DedupePlugin()
        ]
      }
    },


Add support for getting contacts for group

The API supports getting all the contacts of a group, the client should implement this.

@rudigiesler / @hodgestar - can I have a review please and if it's acceptable, a release?

Could we get a test for calling group_contacts with no contacts in the group? (so that we expect [])

Finished reviewing, looks good, just that one comment.

@rudigielser - made the test changes, another quick pass?

Just those few comments. Picked up something with the way that we create groups in these tests that I missed on the first review.

@rudigiesler done

:+1:


Reland "Flutter Driver - Create widget finders from serialized finders extensions"

Description
As null safety feature for Flutter Driver was merged (#67570), we'd need to reland the extensions feature. It should be compliant now, no lint nor build errors are reported.
Related Issues
#67570
#67456
Tests

packages/flutter_driver/test/src/real_tests/extension_test.dart

Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[X] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[X] I signed the CLA.
[X] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[X] I read the Tree Hygiene wiki page, which explains my responsibilities.
[X] I updated/added relevant documentation (doc comments with ///).
[X] All existing and new tests are passing.
[X] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[X] I am willing to follow-up on review comments in a timely manner.

Breaking Change

[X] No, no existing tests failed, so this is not a breaking change.

TBR @dnfield

Fixes applied @jonahwilliams

@jonahwilliams I've made a mistake during rebase. Fixed.

@dnfield @jonahwilliams would you mind merging this?

It should get merged when the tree becomes green again.  Thanks for updating it.

It should get merged when the tree becomes green again. Thanks for updating it.

Exactly. Thank you for the review and cooperation.

@dnfield it seems Mac builds were failing on the master, seems fine now. In the meantime the bot has removed the auto-merge label. May I ask for re-adding waiting for tree to go green?

It's passing now.  @jonahwilliams - AFAICT the google internal failures are related to null-safety not being rolled into google3. I'm going to land this on that assumption. We can revert if I'm wrong.


Run sessions via gateway, revise session API (BREAKING)

based on #5297
Now that we're able to stream arbitrary progress to the frontend, it's possible to run services via the gateway instead.
There were a few tricky parts to this.
Problem A: It doesn't seem possible to take an arbitrary Container and run it via the Gateway instead. The Container already "bakes in" an LLB definition that refers to the result of running the service. The only thing you can really do is try to Solve() it.
Solution A: change the API such that Services are no longer just Containers. Instead, you call Container.Service which takes the exact same params as Container.WithExec, only it returns a Service instead of a Container. Note: we now have to maintain two different ways to run containers (Container.WithExec + Service.Start).
The new API is as follows (this is a BREAKING CHANGE):
type Container {
  # REMOVED
  # hostname: String!

  # REMOVED
  # endpoint(...): String!

  withServiceBinding(
    alias: String!
    service: ServiceID! # BREAKING: used to be ContainerID
  ): Container!

  # NEW: return a Service that will run the given command
  #
  # all params are the exact same as withExec
  service(
    args: [String!]!
    skipEntrypoint: Boolean
    stdin: String
    redirectStdout: String
    redirectStderr: String
    experimentalPrivilegedNesting: Boolean
    insecureRootCapabilities: Boolean
  ): Service!
}

extend type Query {
  service(id: ServiceID!): Service!
}

scalar ServiceID

type Service {
  id: ServiceID!
  hostname: String!
  endpoint(
    port: Int
    scheme: String
  ): String!
}

Problem B: it's harder to generate service hostnames. Previously we could just take the digest that WithExec baked in, but now that we're not doing that, we have to come up with our own content-addressing scheme. The naive thing to try is taking a digest of the Service, akin to a ContainerID, but ContainerIDs aren't stable because *pb.Definition contains fields that can change order while maintaining the same semantics.
Solution B: I implemented a helper to take a stable digest of an arbitrary struct. This works by recursing through structs, maps, slices, etc. and digesting types, with a special-case for *pb.Definition which was the source of unstable hashing as its inner Def field is topographically ordered.
Problem C: bizarre behavior with running the same service across Dagger sessions (i.e. parallel test runs), causing each session to start the same service + hostname, and some of them to fail when resolving the host to an IP.
Solution C: dedupe services within a session, but not across sessions. Maintain a global map of all services, keeping track of their starting/running state to handle parallelism, and injecting a "session ID" into each Service so that equivalent containers don't share hostnames across Dagger sessions.
TODO:

[ ] Complete the session's vertex when it exits
[ ] Try out the TUI DX for running services

Current status: https://github.com/dagger/dagger/pull/5322/commits/9456f3461741a8c98e07ff97f104338ffbac3e10 removed ExperimentalServiceHost from the Git and HTTP APIs in favor of a smaller change to support the Services tests (adding lazy: true to Container.Directory/File). But I later found we're dependent on Git service support for the new Project tests, and it's not just an implementation detail like the Service tests.
One option is to change the Project tests to use an external test repo instead of a self-served one. We would just need an external repo fixture.
But instead of just ripping those APIs out and calling it a day, I'm going to try supporting them via upstream changes to Buildkit. More cross-pollination between our APIs makes for richer possibilities, rather than saying "services are for containers only."
So far I've added ExtraHosts support to llb.HTTP and llb.Git, the only catch being we need to start the dependent services at API call time rather than Directory/File usage time, because we need an IP to pass to the llb.HTTP or llb.Git op. Not a huge deal, except that it's hard to know when we can "detach" the service. Maybe we just leave it running and document that as a caveat to this likely-rare API use case? Going further than this would mean adding programmable host resolution to Buildkit, similar to secrets, which sounds very cool but I don't know if it would be accepted upstream. Maybe I'll experiment with it anyway, if it's a small change.
Anyway it seems to be working for the most part, and now I'm just chasing down a handful of failing tests (13 by last count but there's some double-counting involved). Haven't pushed anything yet since I'm still hacking on things via local dependency bind-mounts/etc. to iterate more quickly.

Update for today: ok, I think DNS is pretty hard to avoid with all of the crazy nesting things we do. The project tests in particular need to be able to fetch from a git:// URL that ultimately gets passed to the child.
I pushed the old work to a branch and brought DNS back for this PR, with a twist: using search domains to namespace services.
Each service container now has a hostname like d3prnq0oeduqm.m31h3p7msle94, where the first segment is the service's actual hostname as returned by the API, and the second segment is a session-wide domain. I had to change Buildkit to support configuring search domains in llb.HTTP and llb.Git, and I modified the shim to retrieve the search domain as a secret (to avoid cache busting) and inject it in the container's /etc/resolv.conf. Putting it all together, we now have session-local container-to-container traffic with session-independent caching.
Still a lot of cleanup to do, and TBD whether Buildkit will be OK merging these changes, but all the services tests are passing now.
I'm not 100% sure how this will all work with nesting, but I have a rough idea to just propagate search domains from the parent to the child (with the child's domain taking precedence) if needed.

Current status: all tests passing! :tada: The linter is yelling at me for Service.Start cyclomatic complexity - and rightly so.
The last batch of changes add the aforementioned nesting behavior whereby the search domain from the outer session is propagated to the child, which appends it to its search domain list. I also had to make it selectively add the search domain for llb.Git because using the new search domains feature was breaking Dockerfiles that use a frontend that doesn't know about the new source.git.searchdomains capability.
I'm going to try to do this a different way in Buildkit, though. Rather than passing SearchDomains to llb.Git and llb.HTTP, I want to try configuring it as a session.Attachable. That way the domain value never has to enter into LLB. It's easy to make the HTTP and Git sources not count it towards the CacheKey, but there are other parts of Dagger that would be busted, namely service hostnames and ContainerIDs, because it will still result in a different LLB definition each time.

Buildkit session.Attachable went smoothly, upstream PR is here: https://github.com/moby/buildkit/pull/3960
Now we have a non-hacky way to pass dynamic DNS search domains to Exec, Git, and HTTP without busting the cache. No more abusing secrets. 🎉


Add referencial integrity checks to component collections

Due to the auto-recovery behavior of RobustWebElement objects, it's possible to have the items in an existing component collection silently fall out of sync with the actual page content they represent.
All of the solutions I've thought about so far will either impose overhead on every collection access or fire too late to be bulletproof.
This issue is complicated by the prospect of nested collections. When an element goes stale, all of the collections that contain the element must be rebuilt. This will require logic in the reference refresh implementation to check for collection associations. Ultimately, it may prove to be impossible to implement bulletproof auto-recovery for nested collections.

To clarify, the issue here is that collections can represent elements that no longer exist, or that collections will fail to represent elements that have appeared since the collection was initially built.

It may be sufficient to rebuild collections whenever element references they contain go stale.

The methods used to retrieve elements from collections would need to perform the check for referential integrity, rebuilding the collection if a stale reference is detected. Each item in the collection (list or map) includes an element reference. When a client retrieves an item from the collection, the element reference must be tested:

Grab the time the reference was acquired: element.aquiredAt()
Perform a low-cost operation that uses the reference: element.getTagName()
Grab the element acquisition time again and compare it to the previous value.
If the times don't match, this indicates that the element had gone stale and had to be refreshed. The collection must be rebuilt if the reference goes stale.

Open question: How should we handle "optional" elements? I think that collections should explicitly prohibit the use of "optional" elements as item containers, because there's no way to build the collection without know what's in it.

I don't know for certain, but I think the code needed to rebuild collections would just need to be extracted from the constructors for ContainerList and ContainerMap. These would be declared as new protected methods, so that subclasses could perform additional operations as required.

Another thing to consider is how to handle map entries that no longer exist. These should probably throw StaleElementReferenceException for entries that once existed but no longer do after the collection is rebuilt.p

For component lists, I should be able to compare the item index to the container element index on the switchTo call. However, there's currently no explicit indication that a component is an item in a list. The same is true to map elements.
To resolve this part of the issue, I'll need to define two new interfaces - one for list items and one for map elements. The API currently verifies collectability, but I need to add the ability for components to retain specific collection association:

component is a member of a collection
collection to which a component belongs
list item index or map element key

Currently, I retain the parent container and locator used to acquire the container elements for the members of the collection. However, collections aren't currently linked into the page model hierarchy. This is a bit sticky, because collections aren't search contexts, and the current structure consists entirely of nested search contexts. You don't switch to a collection; you can only switch to search contexts.
I could add the collection membership interface/implementation during the "enhancement" process. This is a bit sneaky, but the client code shouldn't need to be aware of this implementation detail. For list items, the membership information will retain a reference to the list and the item index. For map elements, the membership information will retain a reference to the map and the element key.
Once the association is established between component and collection, I need to add code to the switchTo method to verify that the container element is still valid.

If the size of the list or the composition of the map changes, the response of the API would be to publish notifications to registered listeners. To be useful, these notifications will need to include an indication of which collection has changed. These notifications could be used for synchronization.

There's no generic recovery strategy in the event that the size of a list changes. I can introduce a facility for clients to define model-specific handlers for this event, allowing implementers to determine how to respond.
If the composition of a map changes, but the target element still exists, the API could potentially perform a silent recovery as it would for a StaleElementReferenceException. However, it's probably still preferable to invoke model-specific handling.


gpg 2.1 doesn't depend on curl

I think we can remove this dependency, it seems all TLS usage has been shifted to gnutls.

Yeah, looks like it glancing through upstream's configure script. Doesn't even check for it anymore, and Debian have dropped it from the deps list. We should be good to cut it.


Auto compress when right clicking 'Copy SVG Code'

Great plugin!
I noticed right clicking Copy SVG Code shows the original uncompressed code, however it would be great to override this somehow. The flexibility of having this as a quick 'SVG copy' with compression would be really useful.

1 on this. The first thing I checked was if it also worked for 'Copy SVG Code' !

I used the 'Copy SVG Code' as an easy way to compare uncompressed code with the resulting compressed code, so I guess I forgot the most obvious use case 😅
I'll see what I can do, thanks for filing!

thanks @bomberstudios that is epic!

👍
Any update on this? This would come in super handy.

Yep, I could really use this to

Same should be the case when dragging (as opposed to hitting export) the SVG to, say, desktop.

I'd love this - I didn't realise that there was another way to export to SVG, so couldn't figure out why "the plugin didn't work".

I just wrote a plugin here https://github.com/chenxsan/copy-optimized-svg-code, you can copy optimized svg code with it :)


[Ballista] Enable ApproxPercentileWithWeight in Ballista and fill UT

Which issue does this PR close?

Closes #2193.
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

@houqp @yjshen plz take a look.

I think we should add the function in https://github.com/apache/arrow-datafusion/blob/28a6da3d2d175eb9d2f4ff8a6ea58e7c22dae97c/ballista/rust/core/proto/datafusion.proto#L203 proto

I think we should add the function in
https://github.com/apache/arrow-datafusion/blob/28a6da3d2d175eb9d2f4ff8a6ea58e7c22dae97c/ballista/rust/core/proto/datafusion.proto#L203
proto

There is some issue in #2221

There is some issue in https://github.com/apache/arrow-datafusion/issues/2221

I think we should add the function in
https://github.com/apache/arrow-datafusion/blob/28a6da3d2d175eb9d2f4ff8a6ea58e7c22dae97c/ballista/rust/core/proto/datafusion.proto#L203
proto

Yes , but there is some issue in #2221

I think we should add the function in
https://github.com/apache/arrow-datafusion/blob/28a6da3d2d175eb9d2f4ff8a6ea58e7c22dae97c/ballista/rust/core/proto/datafusion.proto#L203
proto

Yes , but there is some issue in #2221 but in df it just rewrite APPROX_MEDIAN to APPROX_PERCENTILE(0.5)

My thought is that we should add
APPROX_PERCENTILE_CONT_WITH_WEIGHT = 16;

behind the  APPROX_MEDIAN=15;  in the proto file.


[BUG] RedM fails to authenticate when attempting to assign menu keybinds.

When logging in to RedM the below error is displayed and sometimes this causes the menu authentication to fail.

Executing /txAdmin-reauth doesn't fix the problem either, since this will also attempt to assign the menu keybinds.
So i fixed this by adding a check if the menu is running in FiveM or RedM

this has already been added. you need to wait for new update
https://github.com/tabarra/txAdmin/commit/4d388f82f32a4f6ec2e889272d5ecd55e93a2200

Thanks for the contribution @AleksanderEvensen, but as outsider said, this one was actually fixed in #946.
I tend to only merge requests when I'm ready to release new updates to prevent this from happening, but this time some plans changed last minute so here we are :P
But thanks again!

Aaah. My bad I didn't check the other PRs to see if it was already fixed.


More semantic promise usage

Redis::connect() now always returns a promise.
Only throw on user input errors.
Cleanup parser and Redis state upon connection close/severing so the same instance can be reused upon reconnection.
Change references to "future" to "promisor." Amp libs really should be using "promisor" and "promise" as these are the actual interfaces. Future is really just a one-size-fits-all implementation of both the Promisor and Promise interfaces. I'd prefer if everyone referenced these things by the appropriate terms so it's easier to understand what's happening in async code.

I think @staabm already covered all my concerns. :+1:

Woooah woops, just accidentally pushed to master. Was trying to push to my fork. Fixing ...


Surrealdb integration example

Hello,
Where there any surrealdb integration examples?
Specifically, this line does not compile:
        .layer(AuthSessionLayer::<User, i64, SessionPgPool, PgPool>::new(Some(poll)).with_config(auth_config));

When swapped out for surreal session pool and surreal client.

what version are you currently using and with what Axum? Also what is the error message?

I have added an Example for surreal to the example folder.

what version are you currently using and with what Axum? Also what is the error message?

I was using axum 0.6.1, the problem was the tower service implementation..
I will give it a shot. Thank you.

the latest version of axum_session does not use axum 0.6 anymore we have moved to axum 0.7 since 0.10.0 if you need to use axum 0.6 use axum_session 0.9.0

Hello,
This works as intended. Thank you.


Support Python 3.12

Blocked by https://github.com/cupy/cupy/pull/7941.

/test cuda122

/test cuda122

Thanks, LGTM!

@kmaehashi Failed to backport automatically.

Not cleanly cherry-picked


refactor!: remove multifile_strategy (this is now supported by Neovim natively)

Highlights across multiple buffers are now cleared by Neovim automatically so providing a separate multifile option is no longer necessary: https://github.com/neovim/neovim/pull/19360
Will be merged in four weeks from now (September 16) once tested and people have had a chance to upgrade their nightly version.

@RaafatTurki Could you test-drive this branch for a while (if you already use the latest Nvim version)? :) I have already been using  it and there don't seem to be any issues.

Hey again ;)
This branch has introduced a variety of issues within various lsp servers, the issues are most likely byproducts of a couple of bugs
gopls and clang

the highlight sticks around after renaming
pressing esc doesn't abort renaming
renaming across files is broken

gopls

the error: [inc-rename] Error while renaming: old and new names are the same: shows even when the old and new names are different.

clang

the error: [inc-rename] Error while renaming: Cannot rename symbol: new name is the same as the old name shows even when the old and new names are different.

sumneko_lua

pressing esc doesn't abort renaming

I've tested everything with the dressing.nvim implementation.
nvim version: NVIM v0.8.0-dev-900-g0d0a336c5
I've built it from source today.

Damn, thanks for the great feedback as always, that really helps a lot! Since I have mostly used the :IncRename command, I probably wouldn't have noticed these issues... I have started working on a rewrite of the dressing.nvim implementation because the current one has issues and we seem to have missed them until #7 was reported😅 that requires a change to dressing.nvim so it may take a while until that's done. Hopefully that will also fix the issues you reported.


[BULK] DocuTune - Rebranding

Global effort for the Microsoft Learn platform rebrand
The Microsoft team is applying changes to content for the Microsoft Learn platform rebrand. Changes include modifications to current Microsoft Learn and Microsoft Docs terminology as well as associated domain name and content URL path changes.
Please review this draft PR and provide any necessary comments as soon as possible. Because these fixes are tightly scoped, we have approval to automatically merge these changes to the default branch for publication.
This PR was generated via DocuTune. It includes only targeted fixes and minor formatting adjustments.
CorrelationId: d8c4c3ac-d49a-4c95-9ddc-9822ca297907
InitiativeId: docutune-rebranding-2022-09
PointOfContact: Alex Buck

#docutune-draft

#docutune-ready

Please do not merge until our team has a chance to review.

Hi @jehchow -
Please merge, thanks!
#docutune-request-approval


providers/kv - Wrap errors

Description:
Issues affected:

Relates to #308

This change is

Reviewed 7 of 7 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


New package: PQPolygonSDK v0.1.1

Registering package: PQPolygonSDK
Repository: https://github.com/Paliquant/PQPolygonSDK.jl
Created by: @Paliquant
Version: v0.1.1
Commit: 59594c229533525c2259e3695f684ce5589dfe26
Git reference: HEAD
Description: Software Development Kit for Polygon.io

Superseded by #51519, I presume


Use default/docker when starting with systemd

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "patch-1" git@github.com:jderusse/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

@jderusse systemd service file can handle that in your personal setup if you need to allow backward compat with other init systems but it's adviced from systemd itself to configure options directly in service file, so why should we add this to contrib? new installation with systemd will use service file to configure docker (or better is better to do so)

@runcom sorry, I'm not confortable with systemd. How can I configure such settings ?
The docker's documentation (https://docs.docker.com/articles/systemd/) talk about editing the file  /etc/sysconfig/docker when the EnvironmentFile is used (which is not the case)
Or directly editing the file docker.service but this file may be erased when running the script https://test.docker.io/

You can configure those settings directly in .service file, if you really need to support /etc/default you can do as you did here but that won't be compatible with os that use /etc/sysconfig instead because without the minus sign it prevents docker from running.
Given this, I think you can do like you did here to prevent service file to be overriden  from test.docker  but I think this change should not go into contrib here in the repo, my thought eh, maybe someone doesn't agree with this :)

@jderusse there's an updated version of that documentation that will be published soon. you can view it already here http://docs.master.dockerproject.org/articles/systemd/
HTH

@vbatts can you review this one?

See also #12134, #13415, #7220, and especially https://github.com/docker/docker/pull/5141#issuecomment-40940481, #3307, and #3627.

closing with the comments referenced above

It is still broken in Ubuntu Vivid 15.04:
https://bugs.launchpad.net/ubuntu/+source/docker.io/+bug/1487936
And since the apt package pulled in when creating a docker-swarm is from apt.dockerproject.org it cannot be fixed Ubuntu upstream. One has to docker-machine ssh new-swarm-node and fix docker.service and /etc/default/docker for each new node with a hacky ansible role or the like :/


CSIP101 Clarify structMap requirements for File division

The wording of CSIP101 structMap/div/div is still fuzzy (we experienced this when trying to strictly follow all the rules to create minimal valid IPs, see https://github.com/DILCISBoard/eark-ip-test-corpus/pull/211 and https://github.com/DILCISBoard/eark-ip-test-corpus/pull/212).
<requirement ID="CSIP101" REQLEVEL="SHOULD" EXAMPLES="structMapExample1">
  <description>
    File division

The transferred files referenced in the file section file group are described in the structural map in a single sub division.

When the transfer comprises of only data and no representations there should still be a single empty representation div present.

METS XPath

structMap/div/div

Cardinality

0..1

  </description>
</requirement>

I propose three improvements.
1. Explanation of "File division"
The term "File division" appears suddenly in CSIP101 and is only used in the definitions of CSIP rules 101-104. Before that, there have already been the terms "File section" and "File group", which have been properly explained in the intro of Section 5.3.5. Use of the METS file section (element fileSec). Thus, I propose adding an introductory paragraph to CSIP101 description:
"If the IP contains actual content files (as opposed to an IP that only has metadata), the structural relationships of the content files are described in a structMap/div/div called "File division.""
Note: the exact wording should reflect the purpose of structMap and this div, see #426.
2. "File division" vs "representation division"
Both "File division" and "representation" are used here for the same thing. "Representation division" is a bit confusing, as here it means the root for all representations, but it could also be read as the div for a specific representation. As the representations folder became optional, I would ditch "representation div" here and leave only "File division" ("representation div" could then be used for the divisions of individual representations).
Actually, "Content division" or "Content file division" would be even better, as "File division" is ambiguous - both content files and metadata files are files.
3. "Transferred files" vs "content files"
"Transfer" implies SIP or DIP, but CSIP is also designed for long term preservation AIPs, so I propose using "content files" or "payload files" or similar (we should agree on one and use it consistently throughout the specification).
PS I used https://github.com/DILCISBoard/E-ARK-CSIP/blob/rel/v2.0-draft/profile/CSIP.xml text with the assumption that this is the right source for requirement texts and the requirements.md files e.g. https://github.com/DILCISBoard/E-ARK-CSIP/blob/rel/v2.0-draft/specification/implementation/metadata/mets/structmap/requirements.md are autogenerated from it.

Hi @koit I agree with these points and find myself struggling with point 2 at times in particular. Myself and @karinbredenberg are going to review the supporting text for these concepts and ensure that it exists, and that it's as concise and clear as we can make it. We'll get a chance to review the changes in the A3 group.

An example of the fuzziness: reading the requirements for File division and Representations division (CSIP101, CSIP103, CSIP105, CSIP107) I derived another possible interpretation:
"If the IP has only one representation, it can be regarded as not a representation but as "only data." In that case, there can be a strucMap/div/div for the content, with @LABEL = "Representations", but content files directly in this div, not having their own "rep1" div."
I doubt this was the intent, as the folder structure representations/rep1/data used to be mandatory.
In sum, File division and Representations division (CSIP101, CSIP103, CSIP105, CSIP107) have to be considered together to form a coherent whole.


misc: optimize gas in EnumerableSet and EnumerableMap

Updates
Main idea: use unchecked to avoid duplicate underflow check on keyIndex - 1
Gas report
according to the gas report, this saves an average of 100~ gas on involved methods. Here are some examples:

Contract
Method
Avg. gas used before
Avg. gas used after

ERC1155EnumerableMock
__burn
53484
53060

ERC1155EnumerableMock
safeTransferFrom
136119
135589

ERC1155Mock
__burn
53517
53093

ERC1155Mock
safeBatchTransferFrom
139048
138518

@antoncoding Thanks, merging.  Will release a new version soon.


Fix rust version rocket

Rocket could not compile on nightly, I've fix rustc to nightly-2018-04-28
Fix #204
@SergioBenitez will rocket target stable rust release in the future ?

SergioBenitez/Rocket#19 is tracking the requirements to build on stable rust. Hopefully even by the end of 2018!


python3Packages.pytorch: 1.7.1 -> 1.8.0

Motivation for this change
Changelog:
https://github.com/pytorch/pytorch/releases/tag/v1.8.0
Tested CUDA 10.2 build and CPU build (Python 3.8).
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@ofborg build python37Packages.pytorch python38Packages.pytorch python39Packages.pytorch

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 115173 run on x86_64-linux 1

2 packages marked as broken and skipped:

python37Packages.skorch
python38Packages.skorch

2 packages failed to build and already failed to build on hydra master:

python37Packages.shap: log was empty
python38Packages.shap: log was empty

15 packages built:

python37Packages.ignite
python37Packages.pywick
python37Packages.rising
python37Packages.slicer
python37Packages.tensorboardx
python37Packages.test-tube
python38Packages.ignite
python38Packages.pywick
python38Packages.rising
python38Packages.slicer
python38Packages.tensorboardx
python38Packages.test-tube
python39Packages.slicer
python39Packages.tensorboardx
python39Packages.test-tube

Please test ocamlPackages.torch in the future as well which is the outlier under packages that require pytorch (not in pythonPackages).


[Event Request] Codeunit 12 Gen. Jnl.-Post Line - OnAfterPostDeferralPostBuffer

We need a new event in Codeunit 12 "Gen. Jnl.-Post Line" named OnAfterPostDeferralPostBuffer:
[IntegrationEvent(false, false)]
local procedure OnAfterPostDeferralPostBuffer(VAR GenJournalLine: Record "Gen. Journal Line")
begin
end;
called from:
local procedure PostDeferralPostBuffer(GenJournalLine: Record "Gen. Journal Line")
var
DeferralPostBuffer: Record "Deferral Posting Buffer";
GLEntry: Record "G/L Entry";
PostDate: Date;
begin
with GenJournalLine do begin
if "Source Type" = "Source Type"::Customer then
DeferralDocType := DeferralDocType::Sales
else
DeferralDocType := DeferralDocType::Purchase;
......
......
......
until DeferralPostBuffer.Next() = 0;
DeferralPostBuffer.DeleteAll();
end;
end;
//INIT INSERT
OnAfterPostDeferralPostBuffer(GenJournalLine);
//END INSERT
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Crash ios

Hello, I can't compress video on ios only !
video_compress: ^2.1.1
Flutter 1.20.4
No logs, my app closing just
Any idea ?
thx 😇

On Xcode

me too
Did you solve it?

Did you guys find any solution ?

Yes all videos capture with my iphone 11

Exactly same crash error here 😭

Same error, no solution yet !

In my case that error caused by missing meta data of picked video.
I picked video file from AssetEntity.file and it not contain the meta data of video file.
When i change AssetEntity.file =>  AssetEntity.originFile to make it contain EXIF information, the error is solved.
So that, make sure your picked video contain meta data of video.

In my case that error caused by missing meta data of picked video.
I picked video file from AssetEntity.file and it not contain the meta data of video file.
When i change AssetEntity.file => AssetEntity.originFile to make it contain EXIF information, the error is solved.
So that, make sure your picked video contain meta data of video.

This also worked for me when using videos from the iOS library but when I am using a file from the camera, which is an XFile type it is still throwing the error.

Have you guys found a solution for this?


Error during apollo launch on AWS

I'm using the aws/public-cloud startup scripts.  On the last step, I'm seeing this error:

TASK: [weave | deploy weave service] ******************************************
fatal: [apollo-mesos-slave-2] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_default_ipv4'", 'failed': True}
fatal: [apollo-mesos-slave-3] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_default_ipv4'", 'failed': True}
fatal: [apollo-mesos-slave-1] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_default_ipv4'", 'failed': True}
fatal: [apollo-mesos-slave-2] => {'msg': 'One or more items failed.', 'failed': True, 'changed': False, 'results': [{'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_default_ipv4'", 'failed': True}]}
fatal: [apollo-mesos-slave-3] => {'msg': 'One or more items failed.', 'failed': True, 'changed': False, 'results': [{'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_default_ipv4'", 'failed': True}]}
fatal: [apollo-mesos-slave-1] => {'msg': 'One or more items failed.', 'failed': True, 'changed': False, 'results': [{'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_default_ipv4'", 'failed': True}]}
fatal: [apollo-mesos-master-1] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_default_ipv4'", 'failed': True}
fatal: [apollo-mesos-master-1] => {'msg': 'One or more items failed.', 'failed': True, 'changed': False, 'results': [{'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_default_ipv4'", 'failed': True}]}
fatal: [apollo-mesos-slave-0] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_default_ipv4'", 'failed': True}
fatal: [apollo-mesos-slave-0] => {'msg': 'One or more items failed.', 'failed': True, 'changed': False, 'results': [{'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_default_ipv4'", 'failed': True}]}
fatal: [apollo-mesos-master-2] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_default_ipv4'", 'failed': True}
fatal: [apollo-mesos-master-2] => {'msg': 'One or more items failed.', 'failed': True, 'changed': False, 'results': [{'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_default_ipv4'", 'failed': True}]}

FATAL: all hosts have already failed -- aborting

That was with the trunk.  I also ran into a similar issue with the 0.3.1 version and the aws-public provider.
TASK: [weave | upload weave template service] *********************************
fatal: [apollo-mesos-master-0] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_eth0'", 'failed': True}
fatal: [apollo-mesos-master-0] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_eth0'", 'failed': True}
fatal: [apollo-mesos-slave-1] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_eth0'", 'failed': True}
fatal: [apollo-mesos-slave-1] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_eth0'", 'failed': True}
fatal: [apollo-mesos-master-1] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_eth0'", 'failed': True}
fatal: [apollo-mesos-master-1] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_eth0'", 'failed': True}
fatal: [apollo-mesos-master-2] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_eth0'", 'failed': True}
fatal: [apollo-mesos-master-2] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_eth0'", 'failed': True}
fatal: [apollo-mesos-slave-2] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_eth0'", 'failed': True}
fatal: [apollo-mesos-slave-2] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_eth0'", 'failed': True}
fatal: [apollo-mesos-slave-3] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_eth0'", 'failed': True}
fatal: [apollo-mesos-slave-3] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'ansible_eth0'", 'failed': True}

FATAL: all hosts have already failed -- aborting

Anyone have any ideas about this error?

After some trial and error, I was able to get things working again.  I think the first run always fails, but if I keep running the launch script over and over, I'm able to finally get it to succeed.  Closing this issue for now since I've got it working.


Adding sidecars to validating webhook configuration

Addresses issue #12193

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: mbanikazemi
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: geeknoid
If they are not already assigned, you can assign the PR to them by writing /assign @geeknoid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/kubernetes/helm/istio/charts/galley/OWNERS

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

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
b780071141be1a0eb5725ff6271838677e1794f0
link
/test istio-pilot-multicluster-e2e

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

Does the backing validation code working in Pilot?
Can you please add a test to e2e-galley?
https://github.com/istio/istio/tree/master/tests/e2e/tests/galley/testdata
https://github.com/istio/istio/blob/5ecaf7557d1b3c7448db25d82313a8744f9df2e7/tests/e2e/tests/galley/galley_test.go#L104

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
b780071141be1a0eb5725ff6271838677e1794f0
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
b780071141be1a0eb5725ff6271838677e1794f0
link
/test istio-integ-k8s-tests

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

/lgtm

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

Needs approval from an approver in each of these files:

install/kubernetes/helm/istio/charts/galley/OWNERS [ymesika]

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

@wenchenglu Can we get this fix in?

@ozevren  yes, the validation code in Pilot works. Will add the tests you have requested tomorrow.


GEOMESA-686 DelimitedTextConverter fails to process records bigger than buffer size properly

Separate PipedReader and PipedWriter into multiple threads to avoid
blocking when writing records larger than PipedReader's buffer size.

6fa3e458eed56f062c804ea3d111786fd8a210df


RR 1-30-24 post created

Added default image used from last week.

Rotated to a different default image


Make it easy to test analyzer and code fixes on real open source projects

Right now we depend on independent testing and user feedback to know when we have a problem in an analyzer. We are able to create tests, but not anticipate every single possible problem an analyzer can have. There should be a an easy way to run an analyzer/codefixes project against real OSS projects, and get feedback if something threw.
This could be guidelines, tools, scripts, or something else. The end result should be something that runs without user interaction and that we can run after our unit tests run.
We could even build a suite of ready-to-run OSS projects, that could be run by a command line script. That should be easily used by any analyzer/code fix project author.
We have already started something like this on CodeCracker. We do it with Cecil. You can see the script here. That is a crafted script, for that specific library, but could be easily adaptable to other analyzer projects. We could create a standard for such scripts, what they produce, what parameters they take, etc, and submit them to a repo somewhere. That is an idea for the problem, not the definitive end solution, we should discuss other ways to solve the problem and pick the best one.

We've expanded our automation surrounding this functionality:
DotNetAnalyzers/StyleCopAnalyzers#1970

We did the same thing with powershell. You can see how we test CodeCracker against Cecil here. This is really manual.
What I think we should be aiming for is something really pluggable, where one could simply point an analyzer library to a set of existing, known to work, projects, and get a result if we got any error.


Updating doc links to TypeSpec.io & few broken anchor links

All changed packages have been documented.

You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/352/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/352/


CHORE: Add GitHub Actions Integration

Forgot I needed to have the game installed in the runners, oops! Skipping for now


Feature/p6 requirenments

Added PortCall Event Tests as well as a TransportCall Post test

This looks good to me - don't mind that eBL does not validate - we (@nt-as and I) will fix that part. I am a little worried that OVS fails...

Guess thats an chicken-egg situation, as the OVS is not merged yet, so the API-Validator currently tries an endpoint that is not reachable by now. How should we handle this? As one of them will alwas fail

I have admin rights and can merge - let me know if I should merge this. If you have any good suggestions as how to solve this in the future please let us know...


Include support for AMD/CommonJS environments

Add Universal Module Definition to head of file.

:+1:

@FrDH are you a maintainer here? ping!

The PR currently has conflicts, could you create a new PR from the latest branch?
Thanks.