Refactor RenameDriver to use derived record type

fixes #590
In adding support for #590, this also refactors RenameDriver to be significantly lazier. It no longer needs to traverse the entire parameters input, just the portion actually accessed in the expression. This is done by leveraging the new lazy type support to wrap the parameter record and lazily replace the display name providers on demand.

:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Connectors.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Core.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Interpreter.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.LanguageServerProtocol.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Transport.Attributes.dll

:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Connectors.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Core.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Interpreter.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.LanguageServerProtocol.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Transport.Attributes.dll

:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Connectors.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Core.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Interpreter.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.LanguageServerProtocol.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Transport.Attributes.dll

:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Connectors.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Core.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Interpreter.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.LanguageServerProtocol.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Transport.Attributes.dll

:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Connectors.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Core.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Interpreter.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.LanguageServerProtocol.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Transport.Attributes.dll

:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Connectors.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Core.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Interpreter.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.LanguageServerProtocol.dll
:heavy_check_mark: No Binary Compatibility issues for Microsoft.PowerFx.Transport.Attributes.dll


calculator in python

made an easy calculator in python where we can add,substract,divide and multiply

hacktoberfest 2024


Memory usage regression in pulsing-rewards tag

OS
Linux
Versions
cardano-db-sync 12.0.1 - linux-x86_64 - ghc-8.10
git revision 30c0afb11bf7e4114487339a9341208ac5388205

Run method

full Mainnet sync from a clean state

Additional context
Add any other context about the problem here.
Problem Report
RAM usage regression for the same epochs in the new tag:

mainnet_1 = last released version (12.0.2)
mainnet_1 = pulsing-rewards tag

While db-sync needs to maintain its own copy of ledger state, there is no solution to this.

Closing this as newer tags have fixed the issue


Add versioning

This should be deployed only when /v1 is also prepended in front end.

@valonhaliti  Did you contact frontend teams?


 Comparison ignoring whitespace

I've now found the editor setting to stop vscode putting whitespace everywhere so hopefully this will be the end of it!


Missing section Resources and theme browser

The section ressources has somehow disappeared recently. I've removed the links to it and added the theme browser link back – you can remove it again if you feel it doesn't add value.
All the best!

Checklist for submitting an awesome plugin, theme, or resource:

[ ] The title for my package or theme uses its npm title (hyper-plugin-example)
[ ] The link for my package or theme uses the npmjs.com link (https://www.npmjs.com/package/hyper-plugin-example)
[ ] There is a visual representation of what my plugin or theme does in the repo. (Plugin: screenshot or gif || Theme: screenshot of the theme applied to Hyper)
[ ] Put your awesome item at the BOTTOM of the correct (plugin, theme, or resource) list.
[ ] VERY IMPORTANT: I've written a short (one sentence) description for my package or theme as the description of the awesome plugin, theme, or resource in the README.md file I'm submitting in the PR.

Re-added the Resources section the other day. Thanks for letting us know, and submitting a fix! 🙏
I have added the themes gallery in the resources section in https://github.com/bnb/awesome-hyper/commit/f70a19210eccfb48a0490e2557fcc0e3393a642e, too! Would do it through this commit, but not sure how to execute that best.


Is there a way to generate global labels from labels in a repo?

I've set up my labels in one repo and would like to copy them to global labels

There isn't possible to modify global labels using the GitLab API as I can see here: https://docs.gitlab.com/ee/api/labels.html#labels-api
I recommend you setup the labels globally, then copy them to your repo.


Specify a commit instead of tag with tool.uv.sources

On https://docs.astral.sh/uv/concepts/dependencies/, it gives an option to specify a git repository with a tag, but not a specific commit. Is this possible?
e.g., I would like to do
[tool.uv.sources]
wonderwords = { git = "https://github.com/mrmaxguns/wonderwordsmodule", commit = "5dfa455554e7c225a59c865d706c4293ca2c132e" }

Because this package https://github.com/mrmaxguns/wonderwordsmodule has made a "release" but not made a tag nor a release to PyPI

You can use rev.
[tool.uv.sources]
wonderwords = { git = "https://github.com/mrmaxguns/wonderwordsmodule", rev = "5dfa455554e7c225a59c865d706c4293ca2c132e" }


getting error during file downloading

Getting error   "java.io.IOException: The current offset on block-info isn't update correct, 1048577 != 8010121 on 2"
Download URL: http://<IP_ADDRESS>:1337/parse/files/QzxAOPDw8p3VmLecPW5b8oyKLYhIr1btsqRzw4fd/d575e21c9fc7558841da949ccec740e7_V07-NetballSG-13Mar-31Mar.mp4
Can you please rectify the issue

REASON
This URL response header can't meet HTTP 1.1 RFC 2616 and HTTP 1.1 RFC 7233 define, and violate it very far.
I debug okdownload on the sample project SingleActivity with replacing with your provide URL, and get the following log:

As you can see, the request and response to the trial connection before split several blocks to downloading:

Request header: {range=[bytes=0-0]}
Response header: code[206]{accept-ranges=[bytes], access-control-allow-headers=[X-Parse-Master-Key, X-Parse-REST-API-Key, X-Parse-Javascript-Key, X-Parse-Application-Id, X-Parse-Client-Version, X-Parse-Session-Token, X-Requested-With, X-Parse-Revocable-Session, Content-Type], access-control-allow-methods=[GET,PUT,POST,DELETE,OPTIONS], access-control-allow-origin=[*], connection=[keep-alive], content-length=[1048577], content-range=[bytes 0-1048576/24030363], content-type=[video/mp4], date=[Sat, 07 Apr 2018 01:41:54 GMT], x-powered-by=[Express]}

So as page 5 of RFC 7233 said:
The first and last bytes only (bytes 0 and 9999):

        bytes=0-0,-1

So, when the request with {range=[bytes=0-0]}, the response would with content-length=[1] what is only the first-byte length, so in here we can't get the instance length from content-length because on the define with the request it is not the instance length, But on the response of your provide URL the response header is content-length=[1048577] what is very wrong, and we can't use it, as define it is not the instance length. Then as the RFC 2616 defined, the content-range would tell us the right instance length with content-range=[unit range/instant-length], and with your URL we got the instance length from content-range=[bytes 0-1048576/24030363] is 24030363, and then we use this instance length to split into several blocks to download this resources, but on the block two the input-stream from response body only provide 1048577 bytes data, so you occur such problem.
As I so guess, the right response to your URL would contain content-length=[1], content-range=[bytes 0-0/1048576], but your response is content-length=[1048577], content-range=[bytes 0-1048576/24030363] , as you can see: none of them are correct, and why don't we use content-length of it directly? because as the RFC define, it is not the instance length, even if you don't provide content-range field we also need use HEAD method request to get the certain content-length as the instance length.
Appendix
Why need to download with split several blocks?
Because of the most resource, one connection doesn't provide enough speed input-stream for it, so we split several blocks of it and use each independent connection to touch each block input-stream simultaneously to enhance the download speed.
Why there is a trial connection before split several blocks to downloading?
Because of before split-block, we need to know how many blocks do we need to split, then we need to know whether the resource support split block or not and the instance length of resource, and use GET method request with {range=0-0} or HEAD method request for the trial connection is just for avoiding cache useless data on TCP receive window.

Thanks for your response.
URL is not in my control. Which API should I use from your library?

I think this resource has something special, it is like chunked resource but it response content-length and content-range without transfer-encoding=[chunked].
I tried the Url without {range=0-0} and just raise an HTTP HEAD request( without any addition request header), the response is: content-length=[24030363]. So the instance-length is 24030363.
REAL PROBLEM
As the Request-Range define on the  RFC 7233:
   Additional examples, assuming a representation of length 10000:

   o  The final 500 bytes (byte offsets 9500-9999, inclusive):

        bytes=-500

   Or:

        bytes=9500-

We think such as 9500- is just means from 9500 to end, but for the URL you provide it means  9500-1058075 (9500+1048576-1), so I tried using especially range even for the last block the download is correct.
So consider this case we will add exactly range even for the last block to fix such problem on 1.0.1.

Now, You can use v1.0.1-SNAPSHOT which handled this issue, and final release would be on v1.0.1.

Thank you very much. Its working perfectly now.


LibraryMirror.libraryDependencies fails in dart2

The method LibraryMirror.libraryDependencies is declared to return a List<LibraryDependencyMirror>.
List<LibraryDependencyMirror> get libraryDependencies;

This call is failing when invoked, w/ the error:
type 'List<dynamic>' is not a subtype of type 'List<LibraryDependencyMirror>' where
    List is from dart:core
    List is from dart:core
    LibraryDependencyMirror is from dart:mirrors

I believe that the native code backing this method creates a plain, unparameterized list, not a List of LibraryDependencyMirror.
https://github.com/dart-lang/sdk/blob/master/runtime/lib/mirrors.cc#L440

Also from spot checking where a client library fails - MethodMirror.parameters (which should return a List<ParameterMirror>).

The mustache package hits the MethodMirror.parameters one.

@a-siva, can you triage this / assign this?

When I've helped @askeksa-google debug his issue, I made a patch which should fix this:
diff --git a/runtime/lib/mirrors_impl.dart b/runtime/lib/mirrors_impl.dart
index 6ee28f12c0..90258de5f1 100644
--- a/runtime/lib/mirrors_impl.dart
+++ b/runtime/lib/mirrors_impl.dart
@@ -1262,7 +1262,8 @@ class _LocalLibraryMirror extends _LocalObjectMirror implements LibraryMirror {
   var _cachedLibraryDependencies;
   get libraryDependencies {
     if (_cachedLibraryDependencies == null) {
-      _cachedLibraryDependencies = _libraryDependencies(_reflectee);
+      _cachedLibraryDependencies =
+            _libraryDependencies(_reflectee).cast<LibraryDependencyMirror>().toList();
     }
     return _cachedLibraryDependencies;
   }
@@ -1299,7 +1300,11 @@ class _LocalLibraryDependencyMirror extends _LocalMirror
       this.isDeferred,
       unwrappedMetadata)
       : prefix = _s(prefixString),
-        metadata = new UnmodifiableListView(unwrappedMetadata.map(reflect));
+
+        metadata =
+          new UnmodifiableListView<InstanceMirror>(
+              unwrappedMetadata.map<InstanceMirror>(reflect));

   bool get isExport => !isImport

I haven't landed this, because I think it would be better to fix the non-generic allocation in C++ runtime code.


Differential scan

It would be useful if gometalinter could perform a scan on 2 Git branches, and report only the warnings/errors that exist on one branch but not the other branch.

This is useful functionality, but is definitely outside the scope of gometalinter. I imagine there are existing tools that provide this functionality.

I use and wrote https://github.com/bradleyfalzon/revgrep for this exact purpose, works better with gometalinter as the output is standardised.

@alecthomas, it can be used as a library, I use it in GopherCI, it could be integrated if there's value. But I just pipe gometalinters output into it, which works fine.

Nice! That's exactly what I was thinking of.

It would actually be nice to have this as a FAQ entry for gometalinter.

Sure, I can PR something for review.


OrganizationsService.GetPackage escapes packagName, but UserService.GetPackage does not

It seems that the OrganizationsService does escape  package names by calling url.PathEscape, but the UserService  does not.
Is this a bug or is there a reason behind escaping one of the APIs but not the other?
If this is a simple bug, I'll be happy to submit a fix.

This looks like a bug to me.  Sounds great... thank you, @jhrozek !
Please remember to add unit tests that demonstrate at least one package name that requires escaping.

Also, our CONTRIBUTING.md guide has useful information in it.

hi! @gmlewis  I'm interested in the problem, apparently the boy didn't do anything about it.

OK, thank you, @DiegoDev2 - it's yours.

while I was testing some of the unit tests it gave me FAIL:
 go test -v orgs_packages_test.go
# command-line-arguments [command-line-arguments.test]
./orgs_packages_test.go:18:30: undefined: setup
./orgs_packages_test.go:22:3: undefined: testMethod
./orgs_packages_test.go:50:20: undefined: referenceTimeStr
./orgs_packages_test.go:51:20: undefined: referenceTimeStr
./orgs_packages_test.go:61:67: undefined: PackageListOptions
./orgs_packages_test.go:66:13: undefined: Package
./orgs_packages_test.go:67:17: undefined: Int64
./orgs_packages_test.go:68:17: undefined: String
./orgs_packages_test.go:69:17: undefined: String
./orgs_packages_test.go:70:17: undefined: Int64
./orgs_packages_test.go:70:17: too many errors
FAIL    command-line-arguments [build failed]
FAIL

does this not matter? Or should the test go well?

@DiegoDev2 - That is not how you run unit tests in Go.
Try this instead: go test -v ./...
Please read CONTRIBUTING.md for more information.


Workflow for distributing wheels for multiple CUDA versions.

❓ Questions & Help
Hi, I'm in the process of packaging my code for distribution and I would like to build wheels for multiple CUDA versions. I see you've built PyTorch Geometric wheels for different CUDA/Python/OS specifications and I was wondering what your workflow is?
For a bit of context, my project relies on PyTorch Geometric, so I would like to do something similar to what you have, i.e. one example distribution of my project would be my_project-latest+cu102-cp38-cp38-linux_x86_64.whl which would rely on the underlying torch_sparse, torch_cluster and torch_scatter wheels with the same specifications (i.e. wheels ending in cu102-cp38-cp38-linux_x86_64).
Do you know if this is possible, or should I have an install process that essentially asks the user to do
$ pip install torch-sparse==latest+${CUDA} -f https://pytorch-geometric.com/whl/torch-1.6.0.html
$ pip install torch-cluster==latest+${CUDA} -f https://pytorch-geometric.com/whl/torch-1.6.0.html
$ pip install torch-spline-conv==latest+${CUDA} -f https://pytorch-geometric.com/whl/torch-1.6.0.html
$ pip install torch-geometric

first, followed by a pip install of my package?
Thanks!

The PyG workflow for building wheels for different CUDA versions can be found in the travis config.
Regarding the installation process: The only way I know of how to group those packages into a single installation command is via conda. I'm fairly certain that this cannot be done via pip, and that's the reason users need to install the external packages in advance for PyG.
As such, your suggested solution should work fine (even if it is a bit cumbersome), but I will work on conda support in the future.

Great, thanks!


Metadata field not returning duplicateLocation.

I have two Google businesses, one of which is a duplicate of the other. In Google Business Profile Manager it shows as a duplicate. When getting location data via the python client, we get data under the metadata field in the location object but the duplicateLocation field is missing for some reason. Not sure why. Perhaps it is a bug or error on my part. Please advise.

Hi @zacharyinserra-lovingly, please provide more details about your issue. I is not clear which libraries you are using and which specific requests you are making (service, input, output messages). Also, if you can, please use the new GAPIC client libraries instead of google-api-python-client. This repository is a legacy (and de-facto deprecated) one and is in maintenance mode.

Hello Vadym,
We are using the google-api-python-client library. Here is a snippet of our
code that fetches the location data.
info = discovery.build('mybusinessbusinessinformation', 'v1',
credentials=creds)
keys =
('name,title,serviceArea,storeCode,labels,openInfo,metadata,storefront_address,phone_numbers,'
'specialHours,regularHours,serviceItems,websiteUri,categories,latlng,profile,relationshipData,'
'adWordsLocationExtensions,languageCode')
locs = info.accounts().locations().list(parent=account,
readMask=keys).execute()
Thanks!
Zac
On Fri, Aug 26, 2022 at 5:01 PM Vadym Matsishevskyi <
@.***> wrote:

Hi @zacharyinserra-lovingly https://github.com/zacharyinserra-lovingly,
please provide more details about your issue. I is not clear which
libraries you are using and which specific requests you are making
(service, input, output messages). Also, if you can, please use the new GAPIC
client libraries https://github.com/googleapis/google-cloud-python
instead of google-api-python-client. This repository is a legacy (and
de-facto deprecated) one and is in maintenance mode.
—
Reply to this email directly, view it on GitHub
https://github.com/googleapis/google-api-python-client/issues/1856#issuecomment-1228943257,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUJ5TE7UC4F5UIM7I5EKGX3V3EWA7ANCNFSM546NTCQQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


HIVE-13873 Column pruning for nested fields

@wagnermarkd @sunchao Thank you for your reviews.
I'd like to add an option to disable nested column pruning by default because we may not support all data types in a single PR. To limit the impacts on other data format, let's make it configurable. And we may remove it in the future. Any thoughts about it?

@winningsix Yes, I think adding a configuration sounds good to me.

@winningsix It's up to the SerDe whether or not to leverage the information provided, right? What's the concern in placing the additional info in the configuration? Parquet can leverage it immediately, while other file formats can be updated to use it (or not) at their own pace.
Defaulting to off means that it will get less test coverage and there's more risk that it could be inadvertently reverted.

It's up to the SerDe whether or not to leverage the information provided, right?

Current solution did some changes on NeededCols. It will filter them if one of its subtype is included in groupPath.

Defaulting to off means that it will get less test coverage and there's more risk that it could be inadvertently reverted.

OK, I will try to update the code working in this way.


labhub.py: Block gci issue assignment

Block assignment of issues labeled as
'initiatives/gci'.
Closes https://github.com/coala/corobo/issues/501
Reviewers Checklist

[ ] Appropriate logging is done.
[ ] Appropriate error responses.
[ ] Handle every possible exception.
[ ] Make sure there is a docstring in the command functions. Hint: Lookout for
botcmd and re_botcmd decorators.
[ ] See that 100% coverage is there.
[ ] See to it that mocking is not done where it is not necessary.

@sils Pls review. :sweat_smile:

@andrewda  Pls review.

@Makman2 Review request.

unack d4b726df8069a885f097a3ad2fdc880eab286a15

@Makman2 :+1:

Build fails for no reason whatsoever. I keep getting AttributeError, even when the only change that I make is a comment.
Pls look into it.... @meetmangukiya @jayvdb @Makman2 @andrewda @sils
https://semaphoreci.com/coala/corobo/branches/pull-request-503/builds/25

@EverWinter23 , I think the error might be occurring because your fork is a few commits behind the master branch of coala/corobo and I think it's worth checking out this source of error.

@random-access No my fork is not behind the master branch.

A dependency repository is causing the build to fail. Waiting for it to be fixed...

--> https://github.com/coala/corobo/pull/503/commits/c2be92c719da887632179a049318230ce88769ee#r175265738

@Vamshi99 ->A dependency repository is causing the build to fail.
Waiting for it to be fixed...

@EverWinter23 It is fixed, rebase

@Makman2 Build still failing. IDK why?

@EverWinter23 Your branch is still not same as corobo master. Please rebase

@javydb
Workaround for #306 as it still hasn't been solved and is difficult to implement. This will serve as a short-term solution to the problem.
The solution being:
gci issues will not be assigned to any contributors using corobo.
gci members can still claim the issues by commenting on the issue thread.

@Makman2 status update?

@EverWinter23 see @jayvdb 's comments ;) Mainly the phrasing inside docs is important :+1:

The workaround blocking those issues for all is okay :+1:

@Makman2 Got it. Will Update.

@Makman Review request.

Please fix all old comments before requesting a review :+1:

@Makman2 :sweat_smile: I think I have taken care of everything this time, hopefully.

Travis tests have failed
Hey @EverWinter23,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
1st Build

     python -m pytest

============================= test session starts ==============================
platform linux -- Python 3.6.3, pytest-3.3.0, py-1.5.2, pluggy-0.6.0
rootdir: /home/travis/build/coala/corobo, inifile: setup.cfg
plugins: requests-mock-1.5.0, cov-2.5.1
collected 33 items

tests/answer_test.py .                                                   [  3%]
tests/coala_lowercase_c_test.py ..                                       [  9%]
tests/coatils_test.py ...........                                        [ 42%]
tests/deprecate_bot_prefixes_test.py .                                   [ 45%]
tests/explain_test.py .                                                  [ 48%]
tests/ghetto_test.py .                                                   [ 51%]
tests/git_stats_test.py .                                                [ 54%]
tests/labhub_test.py .F......                                            [ 78%]
tests/lmgtfy_test.py .                                                   [ 81%]
tests/nevermind_test.py .                                                [ 84%]
tests/pitchfork_test.py .                                                [ 87%]
tests/searchdocs_test.py .                                               [ 90%]
tests/ship_it_test.py .                                                  [ 93%]
tests/the_rules_test.py .                                                [ 96%]
tests/wolfram_alpha_test.py .                                            [100%]

----------- coverage: platform linux, python 3.6.3-final-0 -----------
Name                                Stmts   Miss  Cover   Missing
-----------------------------------------------------------------
plugins/__init__.py                     0      0   100%
plugins/answer.py                      25      0   100%
plugins/coala_lowercase_c.py           12      0   100%
plugins/coatils.py                    120      0   100%
plugins/constants.py                    5      0   100%
plugins/deprecate_bot_prefixes.py       6      0   100%
plugins/explain.py                     21      0   100%
plugins/ghetto.py                      10      0   100%
plugins/git_stats.py                   51      0   100%
plugins/labhub.py                     194     12    94%   263-265, 295-301, 321-322, 336-339
plugins/lmgtfy.py                       4      0   100%
plugins/nevermind.py                    5      0   100%
plugins/pitchfork.py                   12      0   100%
plugins/searchdocs.py                  16      0   100%
plugins/ship_it.py                      7      0   100%
plugins/the_rules.py                    5      0   100%
plugins/wolfram_alpha.py               18      0   100%
-----------------------------------------------------------------
TOTAL                                 511     12    98%

FAIL Required test coverage of 100% not reached.
------------------------------ Captured log call -------------------------------
test.py                    371 DEBUG    Merging {} to the bot config.
--------------------------- Captured stdout teardown ---------------------------
INFO     errbot.backends.test

MESSAGE:
- You must be a member of the coala org to be assigned an issue. If you are not a member yet, just type Hello World and corobo will invite you.
- A newcomer cannot be assigned to an issue with a difficulty level higher than newcomer or low difficulty.
- A newcomer cannot be assigned to unlabelled issues.
- corobo cannot assign issues labelled with 'initiatives/gci'.

--------------------------- Captured stderr teardown ---------------------------
2018-06-06 04:21:33,622 INFO     errbot.backends.test

MESSAGE:
- You must be a member of the coala org to be assigned an issue. If you are not a member yet, just type Hello World and corobo will invite you.
- A newcomer cannot be assigned to an issue with a difficulty level higher than newcomer or low difficulty.
- A newcomer cannot be assigned to unlabelled issues.
- corobo cannot assign issues labelled with 'initiatives/gci'.

=============================== warnings summary ===============================
tests/answer_test.py::TestBot
  cannot collect test class 'TestBot' because it has a __init__ constructor

None
  Module already imported so can not be re-written: errbot.backends.test

tests/coatils_test.py::TestBot
  cannot collect test class 'TestBot' because it has a __init__ constructor

tests/labhub_test.py::TestBot
  cannot collect test class 'TestBot' because it has a __init__ constructor

tests/wolfram_alpha_test.py::TestBot
  cannot collect test class 'TestBot' because it has a __init__ constructor

-- Docs: http://doc.pytest.org/en/latest/warnings.html
============== 1 failed, 32 passed, 5 warnings in 101.23 seconds ===============

Travis tests have failed
Hey @EverWinter23,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
1st Build

     python -m pytest

============================= test session starts ==============================
platform linux -- Python 3.6.3, pytest-3.3.0, py-1.5.2, pluggy-0.6.0
rootdir: /home/travis/build/coala/corobo, inifile: setup.cfg
plugins: requests-mock-1.5.0, cov-2.5.1
collected 33 items

tests/answer_test.py .                                                   [  3%]
tests/coala_lowercase_c_test.py ..                                       [  9%]
tests/coatils_test.py ...........                                        [ 42%]
tests/deprecate_bot_prefixes_test.py .                                   [ 45%]
tests/explain_test.py .                                                  [ 48%]
tests/ghetto_test.py .                                                   [ 51%]
tests/git_stats_test.py .                                                [ 54%]
tests/labhub_test.py .F......                                            [ 78%]
tests/lmgtfy_test.py .                                                   [ 81%]
tests/nevermind_test.py .                                                [ 84%]
tests/pitchfork_test.py .                                                [ 87%]
tests/searchdocs_test.py .                                               [ 90%]
tests/ship_it_test.py .                                                  [ 93%]
tests/the_rules_test.py .                                                [ 96%]
tests/wolfram_alpha_test.py .                                            [100%]

----------- coverage: platform linux, python 3.6.3-final-0 -----------
Name                                Stmts   Miss  Cover   Missing
-----------------------------------------------------------------
plugins/__init__.py                     0      0   100%
plugins/answer.py                      25      0   100%
plugins/coala_lowercase_c.py           12      0   100%
plugins/coatils.py                    120      0   100%
plugins/constants.py                    5      0   100%
plugins/deprecate_bot_prefixes.py       6      0   100%
plugins/explain.py                     21      0   100%
plugins/ghetto.py                      10      0   100%
plugins/git_stats.py                   51      0   100%
plugins/labhub.py                     194     12    94%   263-265, 295-301, 321-322, 336-339
plugins/lmgtfy.py                       4      0   100%
plugins/nevermind.py                    5      0   100%
plugins/pitchfork.py                   12      0   100%
plugins/searchdocs.py                  16      0   100%
plugins/ship_it.py                      7      0   100%
plugins/the_rules.py                    5      0   100%
plugins/wolfram_alpha.py               18      0   100%
-----------------------------------------------------------------
TOTAL                                 511     12    98%

FAIL Required test coverage of 100% not reached.
------------------------------ Captured log call -------------------------------
test.py                    371 DEBUG    Merging {} to the bot config.
=============================== warnings summary ===============================
tests/answer_test.py::TestBot
  cannot collect test class 'TestBot' because it has a __init__ constructor

None
  Module already imported so can not be re-written: errbot.backends.test

tests/coatils_test.py::TestBot
  cannot collect test class 'TestBot' because it has a __init__ constructor

tests/labhub_test.py::TestBot
  cannot collect test class 'TestBot' because it has a __init__ constructor

tests/wolfram_alpha_test.py::TestBot
  cannot collect test class 'TestBot' because it has a __init__ constructor

-- Docs: http://doc.pytest.org/en/latest/warnings.html
============== 1 failed, 32 passed, 5 warnings in 115.20 seconds ===============

@Makman2 @nvzard All done.

@jayvdb Done.

ack https://github.com/coala/corobo/pull/503/commits/0f8a0c0f89c7f14edb14bf2c15a132b5476bb841

@gitmate-bot ff


X-Forwarded-Proto

Hi,
Seems like X-Forwarded-Proto is set to https,http, when configuring the tool to access my http server.
Is this intentional or expected behaviour?
My application validates the header, expecting it to be either blank, http or https. Otherwise it presents an error page.
By reading this documentation, I would expect X-Forwarded-Proto to be set to https:
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-Forwarded-Proto
Thanks,
Juan

Good question! I believe that you can have multiple values in the X-Forwarded-Proto header. Each proxy hop between the client and the final destination server has a https or http appended to the list.
e.g.  X-Forwarded-Proto: https, http (list can go on and on)
which translates to:
, , [, ...]

@TheBoroer , so it is normal multiple protocols are appended to the list?
I'm asking because I have not encountered this issue while testing other tools like ngrok.
Thanks!


Error after upgrading

Steps to Reproduce

do flutter upgrade
wait until error after upgrading

Logs
command
flutter --no-color --no-version-check precache
exception
FileSystemException: FileSystemException: Deletion failed, path = 'C:\flutter\bin\cache\flutter_web_sdk' (OS Error: Папка не пуста.
, errno = 145)

Logs

#0      _Directory._deleteSync (dart:io/directory_impl.dart:201:7)
#1      FileSystemEntity.deleteSync (dart:io/file_system_entity.dart:408:7)
#2      ForwardingFileSystemEntity.deleteSync (package:file/src/forwarding/forwarding_file_system_entity.dart:70:16)
#3      ErrorHandlingDirectory.deleteSync.<anonymous closure> (package:flutter_tools/src/base/error_handling_io.dart:459:22)
#4      _runSync (package:flutter_tools/src/base/error_handling_io.dart:556:14)
#5      ErrorHandlingDirectory.deleteSync (package:flutter_tools/src/base/error_handling_io.dart:458:12)
#6      FlutterWebSdk.updateInner (package:flutter_tools/src/cache.dart:838:16)
#7      CachedArtifact.update (package:flutter_tools/src/cache.dart:692:11)
#8      Cache.updateAll (package:flutter_tools/src/cache.dart:562:24)
<asynchronous suspension>
#9      PrecacheCommand.runCommand (package:flutter_tools/src/commands/precache.dart:167:7)
<asynchronous suspension>
#10     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1157:12)
<asynchronous suspension>
#11     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1009:27)
<asynchronous suspension>
#12     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#13     AppContext.run (package:flutter_tools/src/base/context.dart:149:12)
<asynchronous suspension>
#14     CommandRunner.runCommand (package:args/command_runner.dart:197:13)
<asynchronous suspension>
#15     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:278:9)
<asynchronous suspension>
#16     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#17     AppContext.run (package:flutter_tools/src/base/context.dart:149:12)
<asynchronous suspension>
#18     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:234:5)
<asynchronous suspension>
#19     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:64:9)
<asynchronous suspension>
#20     run.<anonymous closure> (package:flutter_tools/runner.dart:62:12)
<asynchronous suspension>
#21     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#22     AppContext.run (package:flutter_tools/src/base/context.dart:149:12)
<asynchronous suspension>
#23     runInContext (package:flutter_tools/src/context_runner.dart:73:10)
<asynchronous suspension>
#24     main (package:flutter_tools/executable.dart:90:3)
<asynchronous suspension>

Downloading package sky_engine...                                  859ms
Downloading flutter_patched_sdk tools...                         1 947ms
Downloading flutter_patched_sdk_product tools...                 1 848ms
Downloading windows-x64 tools...                                    4,8s
Downloading windows-x64/font-subset tools...                       485ms
[√] Flutter (Channel stable, 2.0.4, on Microsoft Windows [Version 10.0.17763.1817], locale ru-RU)
    • Flutter version 2.0.4 at C:\flutter
    • Framework revision b1395592de (11 days ago), 2021-04-01 14:25:01 -0700
    • Engine revision 2dce47073a
    • Dart version 2.12.2

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\emakey\AppData\Local\Android\sdk
    • Platform android-30, build-tools 30.0.3
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    X Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Android Studio (version 4.1.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] VS Code (version 1.55.1)
    • VS Code at C:\Users\emakey\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension can be installed from:
       https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[√] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 89.0.4389.114

! Doctor found issues in 1 category.

Closing as duplicate of https://github.com/flutter/flutter/issues/64410. Please subscribe on that issue for updates.
Thank you

Closing as duplicate of #64410. Please subscribe on that issue for updates.
Thank you

It's true. But maybe important to see 'PATH'? Maybe this issue happens only with Web SDK?


1.20.1

@ItsProfessional Just put the mod into your mods folder, despite it only being supported for 1.19. I have tried it in 1.20.1, and Minecraft loads fine, and the mod functions properly too.

The render distance change is broken in 1.20.4 though, at least with VulkanMod installed it do nothing with the actual (server) render distance. You can reduce the visible area, but you can't increase it.


Add brief Iris course

Add short Iris course, based on the existing Iris course. The intention is that this course takes less than one day to teach without leaving out a significant amount of the content of the existing Iris course.

Ping @sophieallan @benclarke18

Replaced by #100


Missing score_util in live_qa_annotator

Forwarding a user's bug report:

Hi Yifan, there seems to be code error in the helm main repo. I cant really run the live_qa  scenarios because there is no score_util file in the codebase although it is being imported here: https://github.com/stanford-crfm/helm/blob/318c3d9f0a8ba6cb31f700b84ea2320a051977ac/src/helm/benchmark/annotation/live_qa_annotator.py#L8C32-L8C42

@farzaank could you take a look?


Replicate: ZipFile writing fails on first run (including retries) but succeeds with --restart

First run:
 File "/home/ubuntu/rna-seq_pipeline.py", line 598, in consolidate_output
with z.ZipFile(qc_zip, 'r') as z_out:
File "/usr/lib/python2.7/zipfile.py", line 770, in __init__self._RealGetContents()
 File "/usr/lib/python2.7/zipfile.py", line 811, in _RealGetContents
  raise BadZipfile, "File is not a zip file"
 BadZipfile: File is not a zip file

retrying with --restart results in completion of pipeline with no errors and produces the expected .zip file.

Replicated.  I can't explain how this happens, but since ZIP has other limitations I'll be switching to using the tarfile module.


⚠ GitLab Server has degraded performance

In 45bab6f, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11514 ms

Resolved: GitLab Server performance has improved in 1c4de5e.


Update UserGuide.md

Added the parts I completed to the user guide. Will make a PR for the toggle-list-mode command after #139 is merged.

Codecov Report
Base: 65.30% // Head: 65.30% // No change to project coverage :thumbsup:

Coverage data is based on head (f5e82f4) compared to base (183eb0e).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #145   +/-   ##
=========================================
  Coverage     65.30%   65.30%
  Complexity      538      538
=========================================
  Files            88       88
  Lines          1963     1963
  Branches        247      247
=========================================
  Hits           1282     1282
  Misses          571      571
  Partials        110      110

Impacted Files
Coverage Δ

...a/seedu/address/logic/parser/AddCommandParser.java
100.00% <ø> (ø)

.../java/seedu/address/logic/commands/AddCommand.java
60.86% <100.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


locale days

in my locale days didn't show I am from Egypt

Hi @eslamwael74,
the problem is that in the library after taking the short weekdays from DateFormatSymbols, for each day I take the first char with nameOfDay.substring(0, 1) and it returns this ا.
So to fix this i can just skip this step and print the whole string (e.g. الاثنين).
I'll expose a method to make the user decide how to format the day name.

okaiii am waiting!!
Thank you.

ok, should be done, check v0.1.3.
Let me know if you like it.

how to use it?

As I wrote in the README.mb,
there are 2 new methods in the ‘ViewInteractor’ interface:

String formatWeekDayName(String nameOfDay);
Here you can format the name of the week day.
This method is called after injectViewInteractor method of WeekDaysView.
You should return the formatted name of the week day.

boolean hasImplementedWeekDayNameFormat()
Here you should return true if you have implemented the above method (formatWeekDayName).
This method is called after injectViewInteractor method of WeekDaysView.


Load models as assets v2

Note: this won't work for us as is, since the assets are redownloaded when the app restarts. The benefit of this approach is that it automatically uploads the assets to an AWS instance/CDN. The problem is that the bertqa and wav2vec models are too big to be committed to github as is.


Why evaluation response does not return a boolean value?

Is there any specific reason evaluation response does not return a boolean flag, and we get a string message that too deeply nested?
{
  "evalContext": {
    "enableDebug": true,
    "entityContext": {
      "email": "jhk"
    },
    "entityID": "a1234",
    "entityType": "report",
    "flagID": 1,
    "flagKey": "kmmcd1nsd6ze56chh"
  },
  "evalDebugLog": {
    "segmentDebugLogs": [
      {
        "msg": "matched all constraints. rollout no. there's no distribution set",
        "segmentID": 2
      }
    ]
  },
  "flagID": 1,
  "flagKey": "kmmcd1nsd6ze56chh",
  "flagSnapshotID": 12,
  "segmentID": 2,
  "timestamp": "2021-11-12T23:58:16Z"
}

you can just check the 'variantKey' or 'variantID' from the response, if they are there, which means that the flag evaluation is good.


Use twine to check generated package and readme

The setup.py check command is deprecated. Use twine check instead.
Rename the tox environment to packaging, it now checks the package.

Thanks!


GeoJSON 404 on published docs

Oy! Indeed, on Chrome:

Do you think this is a browser update that broke this? I swear this one was working fine on chrome a week ago..

Originally posted by @andrewpbray in https://github.com/qmd-lab/closeread/issues/101#issuecomment-2424357157

This is actually working locally, and on the published docs the console is showing a 404 for the GeoJSON file. So I'm thinking maybe we've just forgotten to add it to the site resources somehow.

The GeoJSON file actually appears correctly in _site when I render locally even if I don't specify it in the resources key. I thought maybe there could be a difference in how site resources are handled between Quarto release (in the GHA) and my local prerelease, but the GeoJSON file still seems to make it even when I check out Quarto v.1.5.57 and render 🤔
so maybe the logic for including default site resources has been tightened up between the Quarto latest release (which it looks like the container uses?) and the Quarto prerelease (which I'm running locally)

Actually this is fine on the preview docs! It only occurs on the main docs. Not sure if there's a Quarto version difference between them or a file difference...

Even weirder — no difference in the OJS demo's index.qmd or  the project _quarto.yml between main and dev branches 🤔
$ git diff dev main -- docs/gallery/demos/ojs-variables/index.qmd | cat
$ git diff dev main -- docs/_quarto.yml | cat

It looks like closeread.dev (nee closeread.netlify.app) hasn't been updated since mid August, so. I'm on a pre-release and it works fine, so I'll publish from local using the same command as the action: quarto publish netlify docs --id 2a052dac-9a3d-43c1-9f3d-e37e3cb55af2.

@andrewpbray Could this have been the same issue as the missing image when we visited a page on Netlify without the trailing space?

Might well be!


Updating docs to include IKS in various places

Changes include:

Updating product name
Adding IKS to table of solutions
Adding the IBM Cloud Container Registry to available registries
Adding IKS to the cloud provider page

@k8s-ci-robot new message from CNCF:

Contribute to CNCF as an employee
You are already authorized to contribute code to this project through your membership with CNCF - IBM.

/lgtm

/assign @bradtopol

Great job!
/lgtm
/approve


Fixes for someone doing changes in assert :(

assert is not supposed to be used with code that has side effects. Some compilers when in release mode replace assert with void(0) which is a no-operation. This can cause lost funds when a change address is generated and not saved because the assert method removed the save.

Sorry for the delay I will look shortly and merge
On Fri, May 28, 2021, 11:17 PM Shawn M. @.***> wrote:

Merged #33 https://github.com/ldoge/LDOGE/pull/33 into master.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/ldoge/LDOGE/pull/33#event-4815562105, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJKNCDJGZLG3GS7W3TVAKHLTQBTFLANCNFSM45XUF62Q
.


Sticky header not working with responsive table

Sticky header not working with the responsive table. On scrolling table, when the sticky header has comes. the table header goes left side of the container. The same issue has in your demo page.
Demo Page
Pls check the table example in the page

Is there any update?


RFC: Expect "encryptionKey" to be base64 encoded in Cloud Storage

The current implementation is a little confusing, as the $encryptionKey is required to be a raw byte string:
$key = openssl_random_pseudo_bytes(32); // Make sure to remember your key.
$options = [
    'encryptionKey' => $key
    'encryptionKeySHA256' => hash('SHA256', $key, true)
];

Outputting the random bytes will be ASCII nonsense, and the key will be lost.
var_dump($key);
// string(32) "?}?
//               {?׿
// ? ??r??z??????J??"

Additionally, the passing in of encryptionKeySHA256 seems best handled by the client library. So I propose a parameter which expects the user to handle base64_encode of their random bytes like so:
// Make sure to remember your key.
$key = base64_encode(openssl_random_pseudo_bytes(32));

$options = [
    'encryptionKey' => $key,
];

This is simpler and more intuitive. It's also consistent with the x-goog-encryption-key header, which is base64 encoded.
Are there any downsides to doing it this way that I am missing? I imagine there was a reason behind the current implementation, but I am not a fan of it.

@tmatsuo has brought up the performance implications of always having the client library calculate the sha. To address this, the encryptionKeySHA256 can be an optional parameter, with docs conveying it's purpose is to increase performance.

Thanks for the feedback @bshaffer!
The original thought was to provide a way for users to handle the SHA so it didn't need to be recalculated on each call.
Asking that the parameters are base64 encoded and allowing the SHA to be optional sound like solid suggestions to me. I should be able to get the changes in this week.


Use System.Text.Json instead of Json.Net in Operator Framework

Hello folks!
Was looking at the Operator Framework and realized it is using Json.Net.
Is there any particular reason we can't go with System.Text.Json?
I could make a PR if that is acceptable.
Thanks!

The operator framework is an internal library that we have taken (temporary) custody of from another team that had developed it but ended up going a different direction. Its therefore not had the level of scrutiny that we'd do for code that is in .NET.
There are internal discussions about where it should live and which team should own it, so knowing who wants to use it and in what scope (as part of YARP or other projects) is definitely useful information that can help inform decisions.
Thankyou for filing an issue rather than just submitting a PR - as we don't yet know the direction we'll go with the library, I think we'll be a bit more cautious in what PRs we take to it - but switching from Json.Net to System.Text.JSON is pretty uncontroversial so go for it.

Tagging @jkotalik for FYI

We'd definitely be willing to swap to System.Text.Json. We'd gladly accept a contribution.

Triage: Yes, we would take the contribution. Are you still interested? Or should we unassign for now?

#1642 removes direct use of Json.Net package from src modules, with some minor cleanup remaining in Yarp.Kubernetes.Test module (that should be straightforward to remove).
However, the Yarp.Kubernetes.Controller module indirectly pulls in the Newtonsoft library via:
KubernetesClient -> Microsoft.Rest.ClientRuntime -> Newtonsoft.Json.
The good news is that the latest version of the KubernetesClient package removes the dependency on Microsoft.Rest.ClientRuntime, so once we update the KubernetesClient version we should be good.


ocamlPackages.bisect_ppx: 2.6.1 -> 2.8.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://opam.ocaml.org/packages/bisect_ppx/bisect_ppx.2.8.1.
meta.description for ocamlPackages.bisect_ppx is: "Bisect_ppx is a code coverage tool for OCaml and Reason. It helps you test thoroughly by showing what's not tested."
meta.homepage for ocamlPackages.bisect_ppx is: "https://github.com/aantron/bisect_ppx"
meta.changelog for ocamlPackages.bisect_ppx is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
Warning: no invocation of /nix/store/k790czwifx23r0m1v7irxb5r7mxg0pas-ocaml4.13.1-bisect_ppx-2.8.1/bin/bisect-ppx-report had a zero exit code or showed the expected version
0 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 2.8.1 in filename of file in /nix/store/k790czwifx23r0m1v7irxb5r7mxg0pas-ocaml4.13.1-bisect_ppx-2.8.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

0 x86_64-linux rebuild(s)
0 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
ocamlPackages.bisect_ppx

Instructions to test this update (click to expand)

Build yourself:
nix-build -A ocamlPackages.bisect_ppx https://github.com/StridBot/nixpkgs/archive/420f225b3df65a458f93445f9df2968805d5bdc0.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/k790czwifx23r0m1v7irxb5r7mxg0pas-ocaml4.13.1-bisect_ppx-2.8.1
ls -la /nix/store/k790czwifx23r0m1v7irxb5r7mxg0pas-ocaml4.13.1-bisect_ppx-2.8.1/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings

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).
Result of nixpkgs-review pr 165612 run on x86_64-linux 1

1 package built:

ligo


fix(#2921): Changing locale on ColorArea doesn't change direction thumb moves with home/end key

Removes the special RTL behavior so that reversing the direction of the gradient actually reverses the direction in which Home or End keys adjust the value.
Closes #2921
✅ Pull Request Checklist:

[x] Included link to corresponding React Spectrum GitHub Issue: #2921
[x] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[x] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[x] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

Open HSB xChannel="saturation", yChannel="brightness"
Set the Locale to a RTL language, either Arabic or Hebrew.
Tab to move keyboard focus to the ColorThumb within the ColorArea in the Storybook example.
Press Home (fn + ArrowLeft on a Mac) key to decrease the horizontal channel value, in this case "saturation", by 10%. The thumb should move to the right.
Press End (fn + ArrowRight on a Mac) key to increase the horizontal channel value, in this case "brightness", by 10%. The thumb should move to the left.
Press Shift + ArrowRight keys to decrease the horizontal channel value, in this case "saturation", by 10%. The thumb should move to the right.
Press Shift + ArrowLeft keys to increase the horizontal channel value, in this case "saturation", by 10%. The thumb should move to the right.

Does this seem like the correct behavior, where Home decreases value and End increases value, or should the Arrow keys used to trigger Home or End match the orientation of the ColorArea?
🧢 Your Project:
Adobe/Accessibility

Just wanna make sure, has this behavior been vetted by the internationalization team?
(tbh, this feels weird to me but I'm no expert haha)

@rgeraghty, Could you take a look at how Home/End keys behave for the ColorArea component using a RTL language, following the test instructions able. The question is whether this seems like the correct behavior, where Home decreases value and End increases value, or should the Arrow keys used to trigger Home or End match the orientation of the ColorArea?

@majornista it would be better to match orientation for RTL as works with arrows keys as well (which are not just mac specific, windows as well for smaller form keyboards)

@majornista it would be better to match orientation for RTL as works with arrows keys as well (which are not just mac specific, windows as well for smaller form keyboards)

Thanks for your feedback. In that case, I think I can close this to keep the behavior as originally implemented.


CLI installer for examples (npx create-liveblocks-app)

This PR adds a CLI tool that allows for cloning and installing Liveblocks examples with a single command. It can be tested by opening the package's directory and running:
npm run demo

Then check inside the .demo directory. This will simulate running:
npx create-liveblocks-app nextjs-live-avatars

Closing this because I've moved it to a separate repo: https://github.com/liveblocks/create-liveblocks-app


ElixirLS should detect . as the project directory and raise an error

elixirLS.projectDir having the value . causes the extension to not work. We should either ignore . or raise an issue
Results in an error like Skipping build because cwd changed from /home/jason/dev/gen_tracker/. to /home/jason/dev/gen_tracker in the output log

Hi :wave: I'd like to take a stab at this issue. I see your comment here https://github.com/elixir-lsp/elixir-ls/issues/427#issuecomment-738175152

I agree that a warning would be a good improvement. Should we go even further and ignore a value of . replacing it with ""? (this would be in addition to the warning).

That would be done here correct? Where we'll check if the rootPath is . and replace it with ""

@fmterrorf I would not pollute this module with settings validation. Let's do it in server.ex in set_settings.


gtk:x86-windows build failure

Host Environment

OS: Windows

To Reproduce
vcpkg install gtk:x86-windows
Failure logs
Package: gtk:x86-windows
Vcpkg version: 2020.02.04-nohash
build.log
Additional context
Prior to installing gtk I uninstalled tiff:x64-windows and wxwidgets:x86-windows because of database corruption.

Please unzip this file, put it to VCPKG_PATH/downloads then rebuild gtk.
Thanks.

Thank you, it worked!


insufficient funds for deposit error

The Issue
i had a user report they tried to make deposit of 10lbc and they only had 7lbc without realizing it, but the error message said [object object] instead of proper insufficient funds readout
Steps to reproduce

go to publish
enter deposit with more than balance
throws error [object object]

Expected behaviour
it should throw proper errors
Actual behaviour
looks like some json parsing issue
System Configuration

LBRY Daemon version: v.0.18.0
LBRY App version: v.0.18.0
LBRY Installation ID: 5kkPJX88NgHEhRihDHwJZSrxqissMNZ67xUUDxVKBcxWFKzPHRHmtTiMd9QccCqnL6
Operating system: windows 10

Anything Else

Screenshots

would also help if changed the default from 10lbc

Probably the same as https://github.com/lbryio/lbry-app/issues/427 but this is really a daemon side issue per that thread.

ahhh ok i think we should still switch the default amount to the lowest possible just so users now thats its the lowest and they shouldnt run into this issue if they dont change the default amount

even if they do they at that point they would have most likely noticed they didnt have that much but its easily overlooked by new users on first publishes

Interesting, @seanyesmunt what do you think about this? Certainly there are some issues on the app end in terms of displaying the proper error message (as @MSFTserver pointed out, probably an issue reading json) and explaining/modifying the default behavior.
i'm going to assign this to you Sean, obviously feel free to unassign it

https://github.com/lbryio/lbry/issues/855 let's see what Alex says. Can we still capture the daemon output even with the exception? I'm sure we can but we'd wanna know what the daemon will return exactly.
I'm all for making this a priority, comes up alot.

@kauffj Is there any reason we can't just check the users wallet balance app side for this? It would be a quick fix while we figure out a better solution for the daemon interaction. Added to to the channel creation to see what it would look like here https://github.com/lbryio/lbry-app/pull/794/files

@seanyesmunt yes an app side check could be added. But we should still figure out what's going on here, since other errors could likely generate this same outcome (I'm guessing we're rendering an Error object rather than printing it's message, but haven't looked).

👍
Yeah we will definitely need to fix the daemon error handling. But it will
be nice to have a somewhat better error presented for now.
On Tue, Nov 28, 2017 at 4:25 PM Jeremy Kauffman<EMAIL_ADDRESS>wrote:

@seanyesmunt https://github.com/seanyesmunt yes an app side check could
be added. But we should still figure out what's going on here, since other
errors could likely generate this same outcome (I'm guessing we're
rendering an Error object rather than printing it's message, but haven't
looked).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lbryio/lbry-app/issues/748#issuecomment-347668674,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQGcjqw0EwbPyXE1b2hN83nH9Lb6AvFsks5s7HpXgaJpZM4QeX0S
.

Fixed in #794

Once the daemon side is fixed(the exception, plus the update balance issue), this will need to be reverted because the daemon needs to take into account already claimed LBC when updating claims.


Issue 92

Closes #92
Extended the interface for 2d histograms.  Can be tested with the example file.
The show() method behaves in a similar way as before, but there are line breaks for each y-bin.
The numpy method returns the 2d array for the non-under/overflow bins with bin edges for x and y.
Please let me know if you have any feedback -- I tried to match the existing style and such, but if something isn't correct I'm happy to modify it.

This looks fantastic, thank you! You did match the surrounding style well. Just a few requests from the API police:

No capital letters in method names, like numXbins and numYbins; it impedes speed typing.
Since this is the first two-dimensional object, you're setting a convention about where to put the x and y. I'd be happy with putting them at the beginnings of words (e.g. xlow, yhigh), except that when this combines with the num convention, you get numxbins and numybins. When navigating with move-forward-word and move-backward-word keystrokes, that important letter is buried in the middles of the words, making it hard to change quickly. However, religiously putting x and y at the ends of words leads to gross words like indexx (below). How about this? x and y go at the beginnings of words and this convention trumps num: xnumbins and ynumbins.
underflows and overflows returning tuples: could these be four functions instead: xunderflow (singular, even though it's an array), yunderflow, xoverflow, and yoverflow?
Same for xinterval and yinterval. I don't like a switch based on a string because the string can take on infinitely many values, one of which is mapped to "x" and all the others are mapped to "y". It's too much freedom for something that has only two states. And anyway, everything else is functions with names that end in "x" or "y", so it would be more consistent if this were, too. If you want to share code between the implementations, you can introduce an internal method (starting with "_").
Same for xindex and yindex.

It looks like this is going to merge into my "issue-92" branch, rather than "master," so I'll go ahead and merge it and make the suggested changes myself. Just wanted you to know why I'm making them.
Thanks for this work!


Join forces with OpenMQTTGateway

There is another MQTT-Gateway project under active development.
https://github.com/1technophile/OpenMQTTGateway
I believe that both projects address the same idea.
They both developed in different directions
Differences to the MQTT433Gateway

Supports officially ESP32 and ESP8266
Supports other hardware, BLE, IR, direct Sensors, RFM69, GSM, etc.
Programming structure keeps it open for other Protocols as well (Zigbee, Zwave, Enocean)

Differences of this project to the OpenMQTTGateway

User interface for configuration (now very newbie friendly)
Uses the an own version of the pilight lib and hence supports many 433 MHz devices

I think both projects would benefit from each other and we might have at the end an ultimate Foo2MQTT-Gateway system, which ideally is even newbie friendly enough for non-programmers.

As far as I know the RC-switch library used in that project is not as good as ESPilight, it also does not support the newer KoKo protocols.

Therefore, I would propose to join forces here... why not using the epsilight lib of this project but using the BLE, RFM69 and other modules of the OpenMQTT project. That would bring us the best of both projects

All the magic towards parsing the rf signals will be performed with ESPiLight which based on the protocol implementations of pilight.
ESPiLight is intentionally a library by it's own. Every project is free to use this library and I will happily support this.
But keep in mind that pilight was not developed with micrcocontoller in mind and thus is relativly bulky and memory hungry.
Btw.  I would like to see ESPiLight support in esphomelib.

Hello,
Thanks for your library @puu, for your information we integrated it into OMG a few weeks ago:
Of course it will not work on an uno but we can use it with ESP.

@1technophile you beat me, as I was just going to post this.
Now, in my opinion we have a project with a nice fancy GUI for config (even after flashing) and platformio support (which I find much better rather then the arduino gui, if only for the dependency checks).
And we have a project which goes much more further and above 433 MHz devices most important being BLE.
Maybe there are even more ways to join both projects?

@1technophile cool, ESPiLight is already integrated into OpenMQTTGateway, this is amazing!
I had a short look into the integration. As I understood, you need to select between rcswitch and ESPiLight but can not run both with the same receiver/transmitter? I think, it would be interesting to receive/transmit with one implementation and parse with both. Unfortunately (as far as I understand), the raw data formats are different and rcswitch does not support impulse trains as implemented in ESPiLight. The pulse trains (array of pulses in µs) are at a lower abstraction level, then the rcswitch data formats. Thus, to provide interoperability between ESPiLight and rcswitch, pulse train support must be implemented into rcswitch.
If you interested in a web frontend, please have a look to this project or blinkenhat. @janLo did a great job by implementing a lightweight rest api in combination with a feature full javascript web frontend.


Stop trying to sync when there are PG::Errors

When a sync gets a PG error, there's no way that running the sync again is going to succeed the second or nth time. This PR revises the REVERT state to a FAILED state (lolz). The expected behavior is that the PG errors will be returned by DataWarehouse::Result, but instead of REVERT moving the sync back to FINISHED, it moves it to ERRORS. The message to slack is then dispatched and the failed sync will not be picked up again until a dev investigates and changes the state.
Fixes #216

Cool, good call! I'd like to suggest the following change:
self.when(FAIL, SYNCING => FAILED)

I think this more closely matches the existing naming pattern.


Add ability to test standalone hive jobs

Example
Join table a table b insert to c
Test this job via maestro
Any hive query

@kfish we already got support for testing hive queries as part of a load. The missing piece is a nice way to create and populate existing hive tables as part of the test set up.
If you use the Hive monad from ebenezer then your code needs to go into ebenezer to avoid circular dependencies.

Working on this in the hive-standalone branch on ebenezer

See https://github.com/CommBank/ebenezer/pull/100

This has been merged to ebenezer and util.etl


Same File Have different ETag on from different server response with Play 1.2.10,

Hi Team,
I am working on old play 1.2.10 application , where we use activator start/run to launch the web application. I have load balance server as apache and two server instance of application. I want to generate same ETag from both server. Cloud you please help me , where I can find and fix this. I am new to play and we are using java controller.
I tried to disable using  below conf parameter in application.conf, but does not works
Disable the Etag Creation By Play Server
http.useETag=false
I need help to over-ride the Assets functionality using java.
Assets built in Note :Etag support
The Assets controller automatically manages ETag HTTP Headers. The ETag value is generated from the digest (if sbt-digest is being used in the asset pipeline) or otherwise the resource name and the file’s last modification date. If the resource file is embedded into a file, the JAR file’s last modification date is used.
When a web browser makes a request specifying this Etag then the server can respond with 304 NotModified.
Thank you in Advance.

What is Play 1.2.10? I'm not 100% sure what you're asking in relation to Express.js. Are you trying to get your Express.js app to match the same ETag format from Apache or something? Please provide some additional information in what your issue with Express.js is.

Thank you Dougwilson!
Its not related to express. Its Scala Play framework. Where we launch server using activator run command. I commented  here because it same problem, which I have.   here is the play link.
https://github.com/playframework/playframework/issues/3657

Hi @krishtyagi I have never used the Play framework and until now never even heard of it before. Not sure if anyone here would be able to help, though. This is an issue ("bug") tracker for the Express.js project. If you are having an issue with Express.js let us know! If you're asking a general question, you may want to try out a forum like Stackoverflow.

Thank you Dougwilson!


'wrap' phase of the compiler pipeline is only called on the first bundle

we're using the wrap phase of the pipeline to add a semicolon as @substack recommended, but it only gets called on the first bundle. not after watched file changes. should i not expect it to be called every time?

This should work as of<EMAIL_ADDRESS>and<EMAIL_ADDRESS>


Update documentation on how to update with scoop on Windows

Feedback from user:

Kan man opgradere via scoop? f.x.:
scoop upgrade git-phlow
Det kunne man (med update):
phlow specification
sha             : aa0b5a9ba306a5d725c0cfd9a09a0dea4aeae844
version         : 3.0.6
date            : 21-08-2017

måske god idé at tilføje "Upgrading git-phlow" til README.md

Yes.. Scoop can be updated. After a scoop update which updates the manifests for the CLI's the command scoop update * will update all tools with updates. scoop update git-phlow should update git phlow

I will also add how to install a pre-release from the AWS bucket


Create a new table to hold the metadata for the UP metadata

User Story
As a report stream developer, I would like the ability to store metadata for reports flowing through the universal pipeline
Description/Use Case
Similar to the covid pipeline, having access to the metadata for reports going through the UP will be necessary to unlock reporting features.
Risks/Impacts/Considerations
Dev Notes

Link to design doc

Acceptance Criteria

[ ] A new table called metadata exists with the following schema:

[ ] metadata_id (UUID, primary key)
[ ] sender_id (UUID, foreign key to setting)
[ ] report_id (UUID, foreign key to report_file)
[ ] report_index (Int)
[ ] message_id (UUID, indexed, foreign key to message)
[ ] metadata (JSONB)
[ ] created_at (Datetime, indexed)
Indices

[ ] (report_id, report_index)

Hey team! Please add your planning poker estimate with Zenhub @arnejduranovic @jack-h-wang @JessicaWNava @jimmyfagan @luis-pabon-tf @mkalish @victor-chaparro


Restart the server when  child process is killed

Context
I have a typescript node server which I'm starting using ts-node. I want to improve the dev experience by using HMR.
I followed this nestJs example to configure webpack.
Here is my config:
const config: Configuration = {
  mode: 'development',
  watch: true,
  entry: ['webpack/hot/poll?100', './src/index.ts'],
  output: {
    path: path.resolve(__dirname, 'dist'),
    filename: 'server.js',
  },
  externals: [
    nodeExternals({
      modulesFromFile: true,
      allowlist: ['webpack/hot/poll?100'],
    }) as any,
  ],
  target: 'node',
  externalsPresets: { node: true }, // in order to ignore built-in modules like path, fs, etc.
  module: {
    rules: [
      {
        test: /.tsx?$/,
        include: path.resolve(__dirname, 'src'),
        use: [
          {
            loader: 'ts-loader',
            options: { transpileOnly: true },
          },
        ],
      },
    ],
  },
  resolve: {
    extensions: ['.tsx', '.ts', '.js'],
  },
  plugins: [
    new webpack.CleanPlugin(),
    new webpack.DefinePlugin({
      'process.env': {
        ...dotenv.config().parsed,
        ...dotenv.config({ path: './.env.local' }).parsed,
        NODE_ENV: JSON.stringify('development'),
      },
    }),
    new webpack.HotModuleReplacementPlugin(),
    new ForkTsCheckerWebpackPlugin(),
    new RunScriptWebpackPlugin({
      name: 'server.js',
      signal: true,
      restartable: true,
    }),
  ],
  stats: { errorDetails: true },
};

Problem
The source code is well compiled by webpack, but it does not trigger a restart of the server, hence I do not see my changes in live.
Even when enabling the signal option, the child process is killed but none is spawned.
Solution
Either I have misunderstood the config or the purpose of this plugin, either we could always call this.startServer() when afterEmit hook is called. (tested on my config and it worked well. I dont know if it breaks any other usage)

Hello, @AlexGeb
I'll look through this PR on monday, right now not in a proper condition to do it. Thanks in advance!

Thinking about it, I think I misunderstood what HMR means.
When using HMR, you don't need to restart the server because the code is "replaced" during runtime.
However, I did not managed to get it work on my application: when I make some changes on a route, the built code is updated, but the server still send me the old response.
Until I fix it, I will stick to a more classic approach: hot reloading (restart the server when compiled files are updated).
I will cose this PR in favor of #6


sniffnet-glow added to the AUR

Is there an existing issue for this?

[X] I have searched the existing issues.

Describe the solution you'd like
As Arch Linux user on older hardware I had to build Sniffnet from the glow-renderer branch. I've added sniffnet-glow to the AUR so other people can more easily find it.
This is a FYI, the app works fine on my aging laptop. Maybe it can go into the readme, but that's up to you. Let's hope a future iced release obliterates the need for all this :).
Thank you for creating and offering Sniffnet to the world!
Is your feature request related to a problem?
No response

Thank you very much!
I will take care of distributing the information to users who will incur in issues on Arch.
I didn't add the download links of other packages built with glow, so I'll follow the same logic.
And yeah... good news from iced actually: they are about to release a new version which include a new software renderer to be used as fallback in case wgpu fails (glow will be discarded completely from what I've understood).
I'll ping you when it's ready so that you can test it if you want.

@all-contributors please add @glitsj16 for platform.

Hi @glitsj16! I'm coming back to you since recently Iced has finally overcome the limitations due to the graphical renderers and is now able to use a different renderer at runtime if the default one crashes.
The alternative renderer is tiny-skia, a CPU-only renderer that should run everywhere.
You can find the version of Sniffnet based on the latest Iced release on this branch.
It'd be awesome if could kindly try it and let me know if everything is correctly displayed 🤞

Hi @GyulyVGC, I've done a build from the iced-0.10 branch and it works very well. Feels very smooth on the old laptop, both on X11 and Wayland. All in all an impressive achievement. I really appreciate your work and I'll retire the sniffnet-glow package when this tiny-skia makes it into a new sniffnet release.
Regards!

Thanks for having tried, and for the kind words... means a lot to me 🙌

FYI, the updated sniffnet v1.2.2 hit the Arch Linux extra repository. That effectively makes my sniffnet-glow obsolete and I've filed a deletion request.
Regards!

Thank you again!


feat: modalbottomsheet menu

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

3 of 4 (75.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 77.574%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/vibrant-components/src/lib/ModalBottomSheet/ModalBottomSheetMenu.tsx
1
2
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
4128

Relevant Lines:
5178

💛  - Coveralls


Fix recursive path can not be created bug when init multipart upload

What changes are proposed in this pull request?
Support recursive path when using s3 client api to init multipart upload.
Why are the changes needed?
When using the s3 client api to init multipart upload, if the path is recursive, a 404 error will be returned. To fix this, add recursive option when create directory.

@maobaolong Please take a look at this PR.

@apc999 Please take a look at this PR.

thanks @jffree for fixing this issue!

Codecov Report

Merging #14067 (b7970f6) into master (18c2d7e) will decrease coverage by 25.89%.
The diff coverage is 41.46%.

@@              Coverage Diff              @@
##             master   #14067       +/-   ##
=============================================
- Coverage     43.23%   17.33%   -25.90%
+ Complexity     9205     2696     -6509
=============================================
  Files          1423     1423
  Lines         82517    82543       +26
  Branches       9986     9991        +5
=============================================
- Hits          35676    14310    -21366
- Misses        43878    66983    +23105
+ Partials       2963     1250     -1713

Impacted Files
Coverage Δ

...ava/alluxio/client/block/stream/BlockInStream.java
0.00% <0.00%> (-62.71%)
:arrow_down:

...uxio/client/file/cache/LocalCacheFileInStream.java
0.00% <0.00%> (-75.49%)
:arrow_down:

...a/alluxio/client/file/cache/LocalCacheManager.java
0.00% <0.00%> (-88.59%)
:arrow_down:

...alluxio/master/journal/raft/RaftJournalSystem.java
0.00% <0.00%> (-56.49%)
:arrow_down:

...in/java/alluxio/proxy/s3/S3RestServiceHandler.java
0.00% <0.00%> (-3.11%)
:arrow_down:

...n/java/alluxio/master/AlluxioJobMasterProcess.java
0.00% <0.00%> (ø)

...ommon/src/main/java/alluxio/metrics/MetricKey.java
97.43% <100.00%> (-0.39%)
:arrow_down:

...n/src/main/java/alluxio/metrics/MetricsSystem.java
52.03% <100.00%> (-9.38%)
:arrow_down:

.../server/common/src/main/java/alluxio/RpcUtils.java
32.00% <100.00%> (+2.83%)
:arrow_up:

...mon/src/main/java/alluxio/shell/CommandReturn.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 678 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 6a0c01d...b7970f6. Read the comment docs.

thanks @jffree for fixing this issue!

Benefit from alluxio and give back to alluxio.

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.6 please


Fix action leak in mario example

Previously each key press was creating a new loop of Time actions.

🍜


Fix cornercase of to_csv with skipHidden enabled

When running xlsx.stream.to_csv(sheet, { skipHidden: true }) on the only sheet in this XLS-file from the Swedish Pension Authority the stream never got to on('end'). I think it's because all the rows in the last _read call was set to hidden, leaving the stream kept in limbo. This seems to have fixed it and makes sense to me but I'm not very familiar with node streams.

Thanks for sharing. Given how the function is currently implemented, your diagnosis sounds right.  The hidden logic is not used in to_html / to_json, so can you confirm that both streams properly terminate when processing the file?

Nice, to_html and to_json terminated properly for the file yes


Unable to disable access logging for specific NS using Telemetry API

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
Installed Istio. Installed bookinfo. Tried disabling access logging just on the bookinfo-backends NameSpace with Telemetry API following an example from the docs
apiVersion: telemetry.istio.io/v1alpha1
kind: Telemetry
metadata:
  name: bookinfo-backends-no-log
  namespace: bookinfo-backends
spec:
  # no selector specified, applies to all workloads in the namespace
  accessLogging:
  - disabled: true

Even after the creation of the above Telemetry object, when I sent sample traffic access logs were getting printed.
Steps taken to test-

Creaated kind cluster-

kind create cluster

Installed istio 1.20.2 using istioctl

istioctl install --set meshConfig.accessLogFile=/dev/stdout

Deployed bookinfo

kubectl create ns bookinfo-frontends
kubectl create ns bookinfo-backends

curl "https://raw.githubusercontent.com/istio/istio/release-1.20/samples/bookinfo/platform/kube/bookinfo.yaml" > bookinfo.yaml
kubectl label namespace bookinfo-frontends istio-injection=enabled
kubectl label namespace bookinfo-backends istio-injection=enabled
# deploy the frontend bookinfo service in the bookinfo-frontends namespace
kubectl -n bookinfo-frontends apply -f bookinfo.yaml -l 'account in (productpage)'
kubectl -n bookinfo-frontends apply -f bookinfo.yaml -l 'app in (productpage)'
# deploy the backend bookinfo services in the bookinfo-backends namespace for all versions
kubectl -n bookinfo-backends apply -f bookinfo.yaml -l 'account in (reviews,ratings,details)'
kubectl -n bookinfo-backends apply -f bookinfo.yaml -l 'app in (reviews,ratings,details)'
# Update the productpage deployment to set the environment variables to define where the backend services are running
kubectl -n bookinfo-frontends set env deploy/productpage-v1 DETAILS_HOSTNAME=details.bookinfo-backends.svc.cluster.local
kubectl -n bookinfo-frontends set env deploy/productpage-v1 REVIEWS_HOSTNAME=reviews.bookinfo-backends.svc.cluster.local

Created Gateway and VirtualService

kubectl apply -f- <<EOF
apiVersion: networking.istio.io/v1beta1
kind: Gateway
metadata:
  name: ingress-to-productpage-gw
  namespace: istio-system
spec:
  selector:
    istio: ingressgateway
  servers:
  - hosts:
    - productpage-bookinfo.myorg.net
    port:
      name: http-80
      number: 80
      protocol: HTTP
---
apiVersion: networking.istio.io/v1beta1
kind: VirtualService
metadata:
  name: ingress-to-productpage-vs
  namespace: istio-system
spec:
  exportTo:
  - .
  gateways:
  - ingress-to-productpage-gw
  hosts:
  - productpage-bookinfo.myorg.net
  http:
  - match:
    - sourceLabels:
        istio: ingressgateway
      uri:
        prefix: /productpage
    name: ingress-to-bookinfo-route
    route:
    - destination:
        host: "productpage.bookinfo-frontends.svc.cluster.local"
        port:
          number: 9080
EOF

Port forwarded istio ingressgateway

kubectl -n istio-system port-forward service/istio-ingressgateway 8080:80

Sent some sample traffic via the ingress gateway

curl -s -H "Host: productpage-bookinfo.myorg.net" "http://localhost:8080/productpage"

Verified that access logs are getting printed everywhere in the beginning

kubectl -n istio-system logs deploy/istio-ingressgateway | tail -1

[2024-01-31T21:23:03.928Z] "GET /productpage HTTP/1.1" 200 - via_upstream - "-" 0 4294 5300 5299 "<IP_ADDRESS>" "curl/8.4.0" "1a5f956f-e840-460f-b3e9-7c9f792e3981" "productpage-bookinfo.myorg.net" "<IP_ADDRESS>:9080" outbound|9080||productpage.bookinfo-frontends.svc.cluster.local <IP_ADDRESS>:54444 <IP_ADDRESS>:8080 <IP_ADDRESS>:60900 - ingress-to-bookinfo-route

kubectl -n bookinfo-backends logs deploy/details-v1 -c istio-proxy | tail -1

[2024-01-31T21:23:08.964Z] "GET /details/0 HTTP/1.1" 200 - via_upstream - "-" 0 178 3 2 "-" "curl/8.4.0" "1a5f956f-e840-460f-b3e9-7c9f792e3981" "details.bookinfo-backends.svc.cluster.local:9080" "<IP_ADDRESS>:9080" inbound|9080|| <IP_ADDRESS>:60621 <IP_ADDRESS>:9080 <IP_ADDRESS>:35316 outbound_.9080_._.details.bookinfo-backends.svc.cluster.local default

kubectl -n bookinfo-frontends logs deploy/productpage-v1 -c istio-proxy | tail -1

[2024-01-31T21:23:03.940Z] "GET /productpage HTTP/1.1" 200 - via_upstream - "-" 0 4294 5286 5286 "<IP_ADDRESS>" "curl/8.4.0" "1a5f956f-e840-460f-b3e9-7c9f792e3981" "productpage-bookinfo.myorg.net" "<IP_ADDRESS>:9080" inbound|9080|| <IP_ADDRESS>:60689 <IP_ADDRESS>:9080 <IP_ADDRESS>:0 outbound_.9080_._.productpage.bookinfo-frontends.svc.cluster.local default

Started following istiod logs before creating the Telemetry object

kubectl -n istio-system logs -f deploy/istiod

Created the Telemetry object

kubectl apply -f - <<EOF
apiVersion: telemetry.istio.io/v1alpha1
kind: Telemetry
metadata:
  name: bookinfo-backends-no-log
  namespace: bookinfo-backends
spec:
  # no selector specified, applies to all workloads in the namespace
  accessLogging:
  - disabled: true
EOF

Logs in istiod

2024-01-31T21:30:38.301275Z	info	ads	Push debounce stable[18] 1 for config Telemetry/bookinfo-backends/bookinfo-backends-no-log: 100.119292ms since last change, 100.119083ms since last push, full=true
2024-01-31T21:30:38.301440Z	info	ads	XDS: Pushing Services:8 ConnectedEndpoints:7 Version:2024-01-31T21:30:38Z/8
2024-01-31T21:30:38.302299Z	info	ads	LDS: PUSH for node:istio-ingressgateway-554657d945-k9xn6.istio-system resources:1 size:3.4kB
2024-01-31T21:30:38.303062Z	info	ads	LDS: PUSH for node:ratings-v1-686ccfb5d8-p2l4f.bookinfo-backends resources:13 size:47.9kB
2024-01-31T21:30:38.303416Z	info	ads	LDS: PUSH for node:details-v1-5f4d584748-g9p9s.bookinfo-backends resources:13 size:47.9kB
2024-01-31T21:30:38.303790Z	info	ads	LDS: PUSH for node:reviews-v3-5c5cc7b6d-x7fcb.bookinfo-backends resources:13 size:47.9kB
2024-01-31T21:30:38.303968Z	info	ads	LDS: PUSH for node:reviews-v2-b7dcd98fb-x25ps.bookinfo-backends resources:13 size:47.9kB
2024-01-31T21:30:38.304258Z	info	ads	LDS: PUSH for node:productpage-v1-5894d88c56-rgnwk.bookinfo-frontends resources:13 size:48.0kB
2024-01-31T21:30:38.304744Z	info	ads	LDS: PUSH for node:reviews-v1-86896b7648-tcfqp.bookinfo-backends resources:13 size:47.9kB

Started following istio-proxy logs of details in the bookinfo-backends.

kubectl -n bookinfo-backends logs -f deploy/details-v1 -c istio-proxy

Sent sample traffic

curl -s -H "Host: productpage-bookinfo.myorg.net" "http://localhost:8080/productpage"

Expectation
Access log lines will not be printed
Observation
Access log line was printed
[2024-01-31T21:32:17.985Z] "GET /details/0 HTTP/1.1" 200 - via_upstream - "-" 0 178 6 4 "-" "curl/8.4.0" "1ae24d97-8dd0-4bce-b57f-6c9d7adfad97" "details.bookinfo-backends.svc.cluster.local:9080" "<IP_ADDRESS>:9080" inbound|9080|| <IP_ADDRESS>:39979 <IP_ADDRESS>:9080 <IP_ADDRESS>:40286 outbound_.9080_._.details.bookinfo-backends.svc.cluster.local default

Version
$ istioctl version
client version: 1.20.2
control plane version: 1.20.2
data plane version: 1.20.2 (7 proxies)

$ kubectl version
Client Version: v1.29.0
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.27.3
WARNING: version difference between client (1.29) and server (1.27) exceeds the supported minor version skew of +/-1

$ helm version --short
v3.13.3+gc8b9489

Additional Information
istioctl bug-report output attached
bug-report.tar.gz

can you show the output of kubctl get telemetry -A -oyaml, don't forget to remove sensitive information.

Sure. Here is the output-
apiVersion: v1
items:
- apiVersion: telemetry.istio.io/v1alpha1
  kind: Telemetry
  metadata:
    annotations:
      kubectl.kubernetes.io/last-applied-configuration: |
        {"apiVersion":"telemetry.istio.io/v1alpha1","kind":"Telemetry","metadata":{"annotations":{},"name":"bookinfo-backends-no-log","namespace":"bookinfo-backends"},"spec":{"accessLogging":[{"disabled":true}]}}
    creationTimestamp: "2024-01-31T21:30:38Z"
    generation: 1
    name: bookinfo-backends-no-log
    namespace: bookinfo-backends
    resourceVersion: "4505"
    uid: 0643c2cf-6605-4df6-9d8f-045d8abddb83
  spec:
    accessLogging:
    - disabled: true
kind: List
metadata:
  resourceVersion: ""

can you share the meshconfig by running kubectl get cm istio -n istio-system -oyaml

Here it is:
apiVersion: v1
data:
  mesh: |-
    accessLogFile: /dev/stdout
    defaultConfig:
      discoveryAddress: istiod.istio-system.svc:15012
      proxyMetadata: {}
      tracing:
        zipkin:
          address: zipkin.istio-system:9411
    defaultProviders:
      metrics:
      - prometheus
    enablePrometheusMerge: true
    rootNamespace: istio-system
    trustDomain: cluster.local
  meshNetworks: 'networks: {}'
kind: ConfigMap
metadata:
  creationTimestamp: "2024-01-31T21:09:29Z"
  labels:
    install.operator.istio.io/owning-resource: installed-state
    install.operator.istio.io/owning-resource-namespace: istio-system
    istio.io/rev: default
    operator.istio.io/component: Pilot
    operator.istio.io/managed: Reconcile
    operator.istio.io/version: 1.20.2
    release: istio
  name: istio
  namespace: istio-system
  resourceVersion: "1711"
  uid: 5cbf803f-3eca-433a-a066-9aa9f2dd0a54

The istioctl bug-report output is also attached to the description of the issue for reference.

there's no log providers, so I think is fallback to the legacy way, read from meshconfig. accessLogFile.

Ok, I just created the default envoy access log provider using the Telemetry API (followed this documentation)-
kubectl apply -f- <<EOF
apiVersion: telemetry.istio.io/v1alpha1
kind: Telemetry
metadata:
  name: mesh-default
  namespace: istio-system
spec:
  accessLogging:
    - providers:
      - name: envoy
EOF

Since I already had the following in my cluster-
apiVersion: telemetry.istio.io/v1alpha1
kind: Telemetry
metadata:
  name: bookinfo-backends-no-log
  namespace: bookinfo-backends
spec:
  accessLogging:
  - disabled: true

Now access logs are not getting printed for apps in the bookinfo-backends NS. Access logs are geting printed on the bookinfo-frontends NS. This matches the expectation.
Followup question
Does this mean that if someone wants to use the following-
spec:
  accessLogging:
  - disabled: true

they must also explicitly set the envoy provider first ?
  accessLogging:
    - providers:
      - name: envoy

Optional. Name of provider(s) to use for span reporting. If a provider is not specified, the default tracing provider will be used. NOTE: At the moment, only a single provider can be specified in a given Tracing rule.

In your case, it means no providers. Istio will fallback to meshconfig. accessLogFile.
https://github.com/istio/istio/blob/f33730b153a6b315eb7cdf351c4a6b9dacbc3ea8/pilot/pkg/networking/core/v1alpha3/accesslog.go#L85

At the time of reporting the issue, I was installing istio using-
istioctl install --set meshConfig.accessLogFile=/dev/stdout

After the above suggestion, I am installing using the following (explicitly mentioning the defaultProviders value)-
cat <<EOF > ./my-config.yaml
apiVersion: install.istio.io/v1alpha1
kind: IstioOperator
spec:
  meshConfig:
    accessLogFile: /dev/stdout
    defaultProviders:
      accessLogging:
      - envoy
EOF
istioctl install -f my-config.yaml

After I deployed bookinfo apps, Gateway, VirtualService and created the following Telemetry object in bookinfo-backends NS, when I sent some sample traffic, I see that access logging is disabled in the bookinfo-backends NameSpace but it is enabled (as expected) in bookinfo-frontends NS.
apiVersion: telemetry.istio.io/v1alpha1
kind: Telemetry
metadata:
  name: bookinfo-backends-no-log
  namespace: bookinfo-backends
spec:
  accessLogging:
  - disabled: true

Thanks @zirain !


XACML REST API call fails

Affected version - 5.9.0
Create a new XACML policy using the 'Basic Policy Editor', with below details with First applicable rule combining algorithm
The policy is based on a resource 'user'
Add a 'Permit' rule based on below
Resource is equal 'user'
User's role is equal 'Manage'
Action Name is equal to 'add_user'
Add a Deny rule as below
Resource is equal 'user'
User's role is equal 'admin'
Action Name is equal to 'add_user'
Publish the policy.
Create a user named david and assign him to a role name 'Manager'.
Use the below curl command. It fails with 'Indeterminate' state. (xacml-request.json) is given at the bottom.
curl -k --user admin:admin --header "Content-Type:application/json" https://localhost:9443/api/identity/entitlement/decision/pdp -d @xacml-request.json
{"Response":[{"Decision":"Indeterminate","Status":{"StatusMessage":"Couldn\u0027t find AttributeDesignator attribute","StatusCode":{"Value":"urn:oasis:names:tc:xacml:1.0:status:processing-error"}}}]}
This is working fine in IS 5.8.0. The UI try it tool also intermittently fails.
<!-- xacml-request.json contents --> {"Request":{"Action":{"Attribute":[{"AttributeId":"urn:oasis:names:tc:xacml:1.0:action:action-id","Value":"add_user"}]},"Resource":{"Attribute":[{"AttributeId":"urn:oasis:names:tc:xacml:1.0:resource:resource-id","Value":"user"}]},"AccessSubject":{"Attribute":[{"AttributeId":"urn:oasis:names:tc:xacml:1.0:subject:subject-id","Value":"david"}]},"Environment":{"Attribute":[{"AttributeId":"urn:oasis:names:tc:xacml:1.0:environment:environment-id","Value":"m"}]}}}

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Feat: Allow users to view a different index by a click

Also add a feature to enable viewing the map of different indicators

Also, allow the users to toggle between the mapping data and indices


" isn't being escaped in resource_link shortcode

Steps to Reproduce

Open a page in a course like https://ocw-studio-rc.odl.mit.edu/sites/peters-test-2021-09-29-1/type/page/edit/23355cd0-d1bb-473d-a2d0-820b9df7e1e2/
Put a double quote in the link text
Publish

Expected Behavior

The link should publish, with double quotes

Actual Behavior

The double-quote isn't escaped
so the link text is truncated at the double-quote

See https://ocw-draft-qa.global.ssl.fastly.net/courses/peters-test-2021-09-29-1/pages/ocw-studioxxx/

Ooops. No it wasn't.

Now it is out. #1264


Mislabeled graph

Pretty sure the #5 is meant to be labeled "User access" or something to that effect. Admin access appears twice as it is.

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

ID: 8f6e327d-b280-a35c-2588-87a19ac2a4ba
Version Independent ID: fa9b0713-771f-4792-979a-2f7a4f4445a2
Content: Microservices with AKS - Azure Solution Ideas
Content Source: docs/solution-ideas/articles/microservices-with-aks.yml
Service: architecture-center
Sub-service: solution-idea
GitHub Login: @doodlemania2
Microsoft Alias: pnp

Thanks for reporting - this issue is under review. This is a Microsoft Internal DevOps Bug ID # 414272

Thanks for reporting this issue. It has been corrected.


fastlane finished with errors

[11:26:06]: --- Step: cocoapods ---
[11:26:06]: -----------------------
[11:26:06]: $ bundle exec pod install
[11:26:06]: ▸ /usr/bin/bundle:23:in load': cannot load such file -- /Library/Ruby/Site/exe/bundle (LoadError) [11:26:06]: ▸ from /usr/bin/bundle:23:in '
[11:26:06]: Exit status of command 'bundle exec pod install' was 1 instead of 0.
/usr/bin/bundle:23:in load': cannot load such file -- /Library/Ruby/Site/exe/bundle (LoadError) 	from /usr/bin/bundle:23:in '
+------------------+-----------+
|         Lane Context         |
+------------------+-----------+
| DEFAULT_PLATFORM | ios       |
| PLATFORM_NAME    | ios       |
| LANE_NAME        | ios setup |
+------------------+-----------+
[11:26:06]: /usr/bin/bundle:23:in load': cannot load such file -- /Library/Ruby/Site/exe/bundle (LoadError) 	from /usr/bin/bundle:23:in '
+------+-----------------------+-------------+
|              fastlane summary              |
+------+-----------------------+-------------+
| Step | Action                | Time (in s) |
+------+-----------------------+-------------+
| 1    | default_platform      | 0           |
| 2    | npm install -g apollo | 15          |
| 💥   | cocoapods             | 0           |
+------+-----------------------+-------------+
[11:26:06]: fastlane finished with errors
Looking for related GitHub issues on fastlane/fastlane...
➡  Gym errors: error 14 and exit status 70.
https://github.com/fastlane/fastlane/issues/5358 [closed] 16 💬
26 Feb 2017
🔗  You can ⌘ + double-click on links to open them directly in your browser.
bundler: failed to load command: fastlane (/usr/local/bin/fastlane)
FastlaneCore::Interface::FastlaneShellError: [!] /usr/bin/bundle:23:in load': cannot load such file -- /Library/Ruby/Site/exe/bundle (LoadError) 	from /usr/bin/bundle:23:in '

pod --version
1.9.1