[clang] --target=thumbv7-eabi doesn't compile

Hi, passing --target=thumbv7-eabi to clang(all versions) compiler options cause that error:
clang-6.0: warning: argument unused during compilation: '--gcc-toolchain=/opt/compiler-explorer/gcc-7.2.0' [-Wunused-command-line-argument]
clang-6.0: error: unable to execute command: Executable "ld.lld" doesn't exist!
clang-6.0: error: ld.lld command failed with exit code 1 (use -v to see invocation)
Compiler returned: 1```

On Ubuntu I solved this issue by running sudo apt install lld-8.

@vesim987 we don't support linking, but if you disable the link (untick the 11010 button) it should work. @rehwinkel we don't use apt to manage linkers etc, everything is built from source. We'd need to work out how to get ld.lld built: the ld in the gcc toolchain isn't being used as it's not the right architecture. We don't have binutils built for all the various architectures.


[MAINT] fix pendingdeprecationwarning

Fixes issue related to https://github.com/mne-tools/mne-python/pull/10825 as in https://github.com/mcmtroffaes/sphinxcontrib-bibtex/pull/302.

Codecov Report

Merging #76 (fbf7916) into main (602160b) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #76      +/-   ##
==========================================
+ Coverage   88.98%   89.00%   +0.02%
==========================================
  Files          11       11
  Lines         917      919       +2
==========================================
+ Hits          816      818       +2
  Misses        101      101

Flag
Coverage Δ

pytests
89.00% <100.00%> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

sphinx_design/dropdown.py
95.83% <100.00%> (+0.05%)
:arrow_up:

sphinx_design/tabs.py
90.32% <100.00%> (+0.07%)
:arrow_up:

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 602160b...fbf7916. Read the comment docs.

Okay to merge? Thanks for a great extension 😄

Thanks, the only reason I didn't #74 though, was that I had a slight worry that, since findall is a generator, rather than a pre-created list, there might be an issue with node.replace_self, since it is altering the thing you are generating from.
I wanted to double-check that this would not end up with any unexpected behaviour

Are we still worried about unexpected behavior or is this okay to merge?

Bump again. I think my suggested change addresses the only worry you've so far expressed @chrisjsewell.  Can you find time to review?

Heya, thanks for the PR, but I've superseded it with #94, to use the already available "compat" function 😄


Support AWS pagination when fetching resources

If there are too many resources, they may be sent by bunches, which is currently not managed in awless.

I'd like to take a crack at this if you don't mind. I'd probably start with ListUsers since (selfishly) it is currently most relevant to my use of this tool. Any objections? Also, any aversion to using the SDK-provided paginators (ListUsersPages in this case) where available instead of explicitly handling the NextToken?

Actually in this case you might take a crack at it since this fetcher is not automatically generated Go code.
We have a convention where files containing generated Go code logic have the gen_ suffix (ex: awless/cloud/aws/gen_api.go)
Your modification would then be in awless/cloud/aws/api.go. The method fetch_all_user_graph is manually implemented since we have some custom logic around fetching users.
Using ListUserPages make sense for this one. You will be able to test if it works properly with you live data.
Out of curiosity, it seems to be a large number of users you have to require some pagination?

Thanks for the information, I'll check it out. The fetch_all_user_graph method is exactly where I was looking, so I'm glad to have confirmation.
And yes, plenty of users. I'm wondering if implementing a user-definable limit (via a maxresults or maxresults.<resource type> config setting perhaps) should accompany this change, since supporting pagination without limits can have unfortunate side effects (increased API requests with the risk of throttling, slower performance, unmanageable result sets, etc). One step at a time though!


Update templates and integrate with slack

This project uses semver, please check the scope of this pr:

[ ] #none# - documentation fixes and/or test additions
[X] #patch# - backwards-compatible bug fix
[ ] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

Closes: https://agile-jira.ciena.com/browse/FROST-1280 and https://agile-jira.ciena.com/browse/FROST-1319
CHANGELOG

Updated pull request template
Added issue template
Updated to pr-bumper version 3
Updated to node 8
Added slack integration
Updated ember-frost-test to ^4.0.1
Updated ember-test-utils to ^8.1.1
Added package-lock.json file

👍


Types: Add support for createHandler

Previously the returned handler was typed as an express request handler, even if the createHandler option had been used to return something different.
With these changes there is a type parameter for the handler exported by the handler creation function inside the compiler, and for the handler returned by createHandler. They both default to express handlers, so there is no breaking change.

Coverage remained the same at 100.0% when pulling ae7b5b5434f97ba9237b3fd69e44953ffed02d06 on hedgepigdaniel:types/createHandler into 1d4306c9efab722e4b12ea38752e4e6855e36816 on 60frames:master.

Any chance of coming back to this?


Emulator not properly closing ngrok processes

Version
4.2.1 on Windows10 1809
Describe the bug
When I close the main window, ngrok processes become orphans and are lying around until I close them manually
I tend to keep my emulator opened for several hours, sometimes days without closing it.
I also reconnect ngrok tunnels when asked from the notification pane.
To Reproduce
Steps to reproduce the behavior:

Open emulator
Enable ngrok
Leave it opened a few hours
Close the emulator
ngrok processes are still running

Expected behavior
ngrok processes opened by emulator should close upon closing the main window
Screenshots

You can see from process explorer that ngrok process are not attached to emulator anymore.

@justinwilaby will investigate if https://github.com/Microsoft/BotFramework-Emulator/pull/1291 fixes this.


Rename DucklingHTTPExtractor to DucklingEntityExtractor

The duckling component is called DucklingHTTPExtractor for legacy reasons - it's hard to remember it's name and the case sensitivity makes it even more confusing. We should rename it to DucklingEntityExtractor

我就是报错一直找不到相应的 class name，但是万万没想到也没有注意到会是这个情况。 嗐~


[Docs] API Reference is not up to date

For some reason, the API Reference section of the docs is missing a bunch of exported members like Session, Repository RepositoryConfig.

yeah which looks correct, everything is exported under icechunk. Very confusing

Oh weird, it only exports objects with doc strings! It ignored anything without a docstring which is not what i would assume. I would assume all functions and classes are shown regardless of if they have doc strings


Fixes for production

Codecov Report

Merging #22 (a4837ab) into main (36446b5) will increase coverage by 4.42%.
The diff coverage is 62.50%.

@@            Coverage Diff             @@
##             main      #22      +/-   ##
==========================================
+ Coverage   74.78%   79.20%   +4.42%
==========================================
  Files          23       17       -6
  Lines         456      351     -105
==========================================
- Hits          341      278      -63
+ Misses        115       73      -42

Impacted Files
Coverage Δ

src/events/guildMemberAdd.js
20.00% <0.00%> (ø)

src/slash_commands/utils.js
28.57% <25.00%> (-68.40%)
:arrow_down:

src/index.js
86.66% <50.00%> (+0.45%)
:arrow_up:

src/commands/student/courses.js
91.66% <100.00%> (ø)

src/commands/student/help.js
100.00% <100.00%> (+7.40%)
:arrow_up:

src/init.js
71.42% <100.00%> (-1.55%)
:arrow_down:

src/service.js
88.60% <100.00%> (+12.01%)
:arrow_up:

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 36446b5...a4837ab. Read the comment docs.


Ensure the kernel component don't call userspace API

Summary
Kernel component should prefer to use file_, psock_, nx_ API, let's replace the wrong usage in the code base.
Impact
Skip to the code which change errno
Testing

LGTM


adapt to spark 2.4 remote dependencies support

start from this issue https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/issues/404
and trying to get s3a:// dependencies work,
i think the problem is because the change of how dependency are retrieved in spark 2.4

The Spark driver running in a pod in a Kubernetes cluster no longer uses an init-container for downloading remote application dependencies, e.g., jars and files on remote HTTP servers, HDFS, AWS S3, or Google Cloud Storage. Instead, the driver uses spark-submit in client mode, which automatically fetches such remote dependencies in a Spark idiomatic way.

https://databricks.com/blog/2018/09/26/whats-new-for-apache-spark-on-kubernetes-in-the-upcoming-apache-spark-2-4-release.html

and this is confirmed by the removal of spark.kubernetes.mountDependencies.filesDownloadDir etc
https://spark.apache.org/docs/2.4.4/running-on-kubernetes.html#spark-properties
https://spark.apache.org/docs/2.3.4/running-on-kubernetes.html#spark-properties
this part might needs to change or have a compatible layer?
https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/blob/master/pkg/apis/sparkoperator.k8s.io/v1beta2/types.go#L362

Yes, some legacy fields like the ones you referred to should be deprecated and removed.

Legacy init-container related fields will be removed in #750.


Watch for file changes in /public and build to /dist

Clear and concise description of the problem
As a developer using Vite I wish to --watch for changes to project files in /public (and/or any other root subdirectory) and transpose them to /dist upon change. This would speed up dev time and create a more efficient workflow when building for multiple platforms (ie: cms/ecommerce theme development) from a single codebase.
Suggested solution
During --watch, add an option in vite.config.js to specify directories to monitor for changes and possibly specify an argument for what to do with those files (transpile, transpose, etc).
Alternative
Coming from webpack, this would be a simple thing to add to webpack.config.js -> https://webpack.js.org/configuration/watch/
Additional context
Possibly somewhat related use-case... https://github.com/vitejs/vite/issues/3233#issuecomment-842527710
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Adding this to plugins in vite.config.js works to solve this issue.
plugins: [
        {
            name: 'watch-external', // https://stackoverflow.com/questions/63373804/rollup-watch-include-directory/63548394#63548394
            async buildStart(){
                const files = await fg(['src/**/*','public/**/*']);
                for(let file of files){
                    this.addWatchFile(file);
                }
            }
        }
    ]

fg

what's fg?

It's https://github.com/mrmlnc/fast-glob
I think this is a good idea, but we probably have to use a custom chokidar instance to watch the public dir, and trigger watcher.emit('restart') at here, as the plugin approach won't detect new files.

For people like me, coming here to find out how to add file from a backend to the watch list, I want to mention, that I meanwhile found a plugin which which it is possible to add additional files and folders to watch, which will trigger a full reload on change.

Tried to implement this today but it's harder than it seems because watcher.emit('restart') doesn't actually restart because Rollup sees that nothing changed. The only way to copy and on new files is to close down the watcher and restart it, or create a temporary file for Rollup to watch and trigger a write to restart.
So with the above, I'm not sure if it's worth to implement this.

I think we should close this issue. This doesn't seem to be a pain point for most users. And the conceptual model is that files in public are not watched for HMR during dev either. So I think this isn't surprising. We can later re-evaluate if there is more demand for the feature, but it would be good at that point to revamp public to also have HMR during dev.

I am new to Vite but also did not find a way to watch for the updated files and build them to the dist
built in 19578ms.
built in 23381ms.

The performance is not good on a weak machine, It seems it rebuilds from scratch all files every time


cpu,stdlib: Fix Access Trace for Accessing Indices in SpatterGen

This change fixes the way indices are generated in a multi generator setup.
It changes it from all cores generating the same trace of indices for accessing the index array to each core generating an interleaved subset of indices.
For an example look below for traces (indices to index array) in a 2 core setup.
Before:
core_0: 0, 1, 2, 3, 4, 5, 6, 7, ...
core_1: 0, 1, 2, 3, 4, 5, 6, 7, ...
After:
core_0: 0, 1, 2, 3, 8, 9, 10, 11, ...
core_1: 4, 5, 6, 7, 12, 13, 14, 15, ...
Additionally, this change fixes the SpatterKernel class in the standard library to comply with the change in the SpatterGen source code.

I went ahead and removed the unused member variables.


Preventing offline behaviour on mutations

I am wondering whether there is a way of preventing the offline behaviour on mutations. It seems to just to not resolve/reject when calling them.

Hey @kirkness,
I don't quite understand what you mean by

It seems to just to not resolve/reject when calling them.

Could you please clarify where in the public API you expect promises to be returned? - Or have you noticed something not working properly internally?
You want your mutations to be executed once and then discarded if they fail, right?
If so, there currently is no way to selectively exclude certain mutations/queries from being retried. However, I'd be happy to add this feature in the next release.
Have a great evening!

Hey @MLPXBrachmann, thanks for getting back - sorry if I wasn't clear. Currently if I call a mutation like so:
try {
  this.setState({ isLoading: true });
  const data = await props.mutation();
  Alert.alert('Success');
} catch(e) {
  Alert.alert('Error');
} finally {
  this.setState({ isLoading: false });
}

Neither of the alerts will be called and it will remain loading, is this intended behaviour or am I doing something wrong? Ideally, as I say, id like to prevent the user from attempting mutations offline and just show a network error - so preventing the offline cache on select queries/mutations would be a very nice feature!
p.s. Thanks a lot for putting this together, I had the classic "Oh and the app should work offline" on the day of release - and this dramatically reduced my stress! 👍

@kirkness, thank you very much for the kind words!
Assuming this occurred while the app couldn't reach your backend: Yes, this is the intended behaviour.
Apollo-Offline does not resolve mutations until they've been delivered to the server or an irreconcilable error occurred. This is one of the cornerstones of the offline-first approach since it allows users to work offline and guarantees that all changes will be synced once they go back online. Before they do, there is now way to tell whether a mutation will ultimately succeed or fail.
I can, however, see the benefits of being able to selectively disable this behaviour. Expect that feature to be implemented by tomorrow.
It is likely going to be enabled by adding a __online__ field with a truthy value to your query/mutation variables. Doing this will allow you to revert back to Apollo's default behaviour for that specific query/mutation.
I hope I understood you correctly and the proposed feature will meet your requirements.
If not - or if you have any additional questions - feel free to just comment again!

Legend - thanks for clearing that up for me! __online__ is exactly what I'm after.

I added selective opt-out through the __online__ flag in b8f928a. It is available from v0.0.6 onwards (you may need to upgrade manually).
Thank you for helping to improve Apollo-Offline!

@kirkness May I close this issue?


[Dropdown] allowAdditions set to true, message "addResult" not appearing when typing in new value

Steps
The error is visible in the live docs:
https://semantic-ui.com/modules/dropdown.html#tagging-and-user-additions

Setup semantic dropdown with allowAdditions: true
Type in new tag
Dropdown displaying 'Add {term}' does not appear

Expected Result
Addition item menu should appear, displaying "text to add {term}."
Actual Result
No menu appears.
Version
2.4.1

Use hideAdditions:false;

That was the issue. Thank you.


Couldn't get the valid product image through amazon url

Describe the bug
Couldn't get the product image through amazon url.
To Reproduce
Steps to reproduce the behavior:
Go to 'https://www.amazon.in/Noise-Bluetooth-Truly-Wireless-Earbuds/dp/B09F2VK7FC/?encoding=UTF8&pd_rd_w=jGYxy&pf_rd_p=ee853eb9-cee5-4961-910b-2f169311a086&pf_rd_r=BADPW5Z8QXFEE5E9SM9J&pd_rd_r=dae04816-b1d3-4dbb-9241-0037760159cb&pd_rd_wg=Su60f&ref=pd_gw_ci_mcx_mr_hp_atf_m&th=1'
Expected behavior
Should get the valid product image. Product images are placed at  class '.a-dynamic-image', and has a attribute 'data-old-hires'.
Screenshots

Desktop (please complete the following information):

OS: [Macbook PRO]
Browser [chrome]

The library works with OpenGraph tags, if amazon has not placed them in the correct tag, it doesn't matter which class they have.
https://github.com/ospfranco/link-preview-js/blob/main/index.ts#L79
If you really want to try to parse specific classes/selectors you can submit a PR, but at the end of the day, there is nothing wrong, because trying to parse specific sites would take thousands of exceptions.


The slowest job is 11min. I think it was just waiting to get started for a long time.

The repo checks are fast but bors has gotten pretty slow.
On Tue, Nov 5, 2019, 12:25 PM Laurențiu Nicola<EMAIL_ADDRESS>wrote:

Looks like our builds have gotten quite slow 😅.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rust-analyzer/rust-analyzer/pull/2185?email_source=notifications&email_token=ABBACRFPQ66KWVZGJ5AUJFTQSGT7VA5CNFSM4JJEKJIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDDSAZA#issuecomment-549920868,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBACRHUCJKVLVSIWLFSD5LQSGT7VANCNFSM4JJEKJIA
.

Yeah, we need to check that our recent great split of hir didn't inflate CI. For from-scratch builds, cratification should be a net negative.
Also, I think we should get rid of indicatiff altogether, and do a very simple progress rendering manually (the code can be stolen from Cargo).

indicatif only brings in number_prefix and unicode-width, so it's probably not hugely expensive.

It's still 1.3k lines of code by itself, and I've also spend couple of
hours figuring out it's API
On Wed, 6 Nov 2019 at 13:08, Laurențiu Nicola<EMAIL_ADDRESS>wrote:

indicatif only brings in number_prefix and unicode-width, so it's
probably not hugely expensive.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rust-analyzer/rust-analyzer/pull/2185?email_source=notifications&email_token=AANB3M4WNRRRG5A2O4RPFITQSKJSTA5CNFSM4JJEKJIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDF75HQ#issuecomment-550239902,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANB3M54MP533F67X26M3F3QSKJSTANCNFSM4JJEKJIA
.


SW-10224 removed limitation on main shop for snippet export

This pull-request allows the export of all existing template snippets. This removes the limitation in shopware, that only snippets from the template used in the main shop are exported.

Thanks ben,
the constraint to shopId 1 seems indeed very random.
I think this PR is good to merge for the next release window.

Dear Contributor,
Thank you for your contribution to Shopware. We value your engagement.
Unfortunately we have to close all existing Pull Requests.
This is due to the new Shopware 5 Major Release and a restructuring of the branch names.
If your contribution is still relevant for Shopware 5, a new pull request is highly appreciated.
We are very sorry for this inconvenience.
Greetings from the Shopware Team.


swipe not working

Not sure is anyone is out there, but swiping doesn't seem to work. Just go to the demo page and try and swipe the image version. Almost like swiping it being interpreted as a  drag an drop.? Any ideas on how to get the swipe to work? Cheers

should have said, the swiping was on a browser running on a PC/Mac (swipe using mouse) and not on a touchscreen (e.g. ipad) device.


Getting Started fails: TypeError: code() takes at least 14 arguments (13 given)

Hi,
I'm trying to run an example from Getting Started from README and it seems there's a bug in code:
$ pip install kubeflow-kale
...
$ wget https://raw.githubusercontent.com/kubeflow-kale/examples/master/titanic-ml-dataset/titanic_dataset_ml.ipynb
...

$ kale --nb titanic_dataset_ml.ipynb
03-03 22:35 | kubeflow-kale |  INFO: Pipeline code saved at titanic-ml-dq7ht.kale.py
Traceback (most recent call last):
  File "/home/ay/venv/bin/kale", line 10, in <module>
    sys.exit(main())
  File "/home/ay/venv/lib/python3.8/site-packages/kale/command_line.py", line 65, in main
    pipeline_package_path = kfp_utils.compile_pipeline(script_path, kale.pipeline_metadata['pipeline_name'])
  File "/home/ay/venv/lib/python3.8/site-packages/kale/utils/kfp_utils.py", line 52, in compile_pipeline
    spec.loader.exec_module(foo)
  File "<frozen importlib._bootstrap_external>", line 783, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/tmp/tmpbaqy973a/pipeline_code.py", line 903, in <module>
    loaddata_op = comp.func_to_container_op(loaddata)
  File "/home/ay/venv/lib/python3.8/site-packages/kfp/components/_python_op.py", line 694, in func_to_container_op
    return _create_task_factory_from_component_spec(component_spec)
  File "/home/ay/venv/lib/python3.8/site-packages/kfp/components/_components.py", line 276, in _create_task_factory_from_component_spec
    task_factory = _dynamic.create_function_from_parameters(
  File "/home/ay/venv/lib/python3.8/site-packages/kfp/components/_dynamic.py", line 47, in create_function_from_parameters
    modified_code = types.CodeType(
TypeError: code() takes at least 14 arguments (13 given)

$ python --version
Python 3.8.1

$ pip freeze | grep kale
kubeflow-kale==0.4.0

Unfortunately, it fails cifar example too :(
https://github.com/kubeflow-kale/examples/blob/master/pytorch-classification/cifar10_classification.ipynb

Hello!
What's the output of pip freeze | grep kfp?


app-only authentication

Is it possible to use the library to login the application? For example: https://docs.microsoft.com/cs-cz/graph/tutorials/php?WT.mc_id=Portal-Microsoft_AAD_RegisteredApps&tabs=aad&tutorial-step=8
Thans.

There are a few differences from the basic procedure:

"client_credentials" must be supplemented with the "scope" parameter:
           $scope = "$baseGraphUri/.default";          $token = self::$provider->getAccessToken('client_credentials', ['scope' => $scope]);
[BUG] - public function getRootMicrosoftGraphUri($accessToken) function crashes on this token. There is a bug in field lookup and it is necessary to pass null instead of token.

            $tenant = array_key_exists('tid', $idTokenClaims) ? $idTokenClaims['tid'] : $this->tenant;             $version = array_key_exists('ver', $idTokenClaims) ? $idTokenClaims['ver'] : $this->defaultEndPointVersion;
If "$idTokenClaims" is null, which is a valid value, the array_key_exists function will throw an exception.

Hello, sorry for late reply.
I believe it should be possible - see sample here: https://github.com/TheNetworg/DreamSpark-SSO/blob/master/cron.php#L25
I understand that you are trying to use v2.0 endpoint with Microsoft Graph which requires scope, there might be some changes required to enable this behavior.

any updates on adding client_credentials grant flow?
As of now MSFT addded support for it.


Add ScrollReveal for images and canvas

I added ScrollReveal, which fades and transitions in specified content whenever it appears in the viewport. For the current pull request, I've added an array which includes all portfolio items, the canvas and the circle image towards the footer. I then use an each function to loop over the array with ScrollReveal.
Let me know what you think.

Hey @jacobproffer! I love the ScrollReveal, it's beautiful ✨
Feel free to work on anything else in this repo 😄


Added support for running the TCK in Arquillian

This provides a test class that can be run in Arquillian. I have tested this with Thorntail (using an Arquillian AuxillaryArchiveAppender to add in my own implementation), and it works.
I did originally try and make this work better with Arquillian, by trying to have it deploy each test to the container. I was able to work around the fact that the tests are created by a factory by writing my own custom container TestRunner that gets them from the factory. I was able to work around the fact that none of the tests extend the TestNG Arquillian class by extracting all its logic into a TestNG listener that was applied to the whole suite. And finally, I was able to work around the fact that none of the test classes has an @Deployment annotated method on it, by writing a custom DeploymentScenarioGenerator that extracted the deployment from a different class - the same class for each test.  This was the most annoying one because WildFly has its own custom one, so doing this in a generic way was impossible. So, in the end it worked, however, it was a lot of what felt like very fragile code, so my strategy in the end was simply to write one Arquillian test that has one test method, and then, in that test method, instantiate a TestNG runner that runs the entire suite for me, using a CDI injected ReactiveStreamsEngine. If you run this from the IDE, it says there's only one test, but the logging shows 1100 or so tests, and makes it clear which failed and what the failures are.
Also fixed two bugs in the TCK where the default engine was loaded instead of the passed in one.

@lordofthejars can you have a look to be sure it follows the recommendations?
@jroper the strategy looks good to me, but I don't understand how to run it. Adding a README would be very beneficial. I only used Arquillian with Junit so far.

It should be a matter of running the ReactiveStreamsArquillianTck. IntelliJ unfortunately doesn't let you run test classes from jars, so to work around that, create a class that extends it, and then run that.

@cescoffier Here's an example of using it. If you're testing against a container that has an implementation of mp rs ops built in, then you shouldn't need any of the Arquillian extension code, that's just to add an implementation to the container.
https://github.com/lightbend/microprofile-reactive-streams/pull/8

Documentation added in #85.

@jroper got it to work. I needed to add a producer creating the engine instance. The producer had to be in a specific package.

Merging as it was decided in the weekly hangout that this is the right approach.

Sorry for being late, I will do tomorrow, we can always improve in another
PR
El mar., 7 ago. 2018 21:07, James Roper<EMAIL_ADDRESS>escribió:

Merging as it was decided in the weekly hangout that this is the right
approach.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/eclipse/microprofile-reactive/pull/84#issuecomment-411167010,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABcmYVGzuEDJrD3KsFfVuWLmmHipb3gXks5uOeVcgaJpZM4Vrt6N
.


List.of() creates an immutable List instance, without a backing Array...

…List, saving one Object[] array creation and saving ~12 bytes for each component in the hierarchy.   This ends up being a few KB for each of our pages, resulting in meaningful memory savings, a little less GC pressure and a little more memory fitting into the processor caches.
This is a minor change.   I expect I will be sending along other small improvements like this one for the Wicket code base if there is an interest in receiving them.   FYI   We have been a Wicket user since ~2008.

I just mistakenly pushed a 2nd commit into this pull request.    I was going to make this a 2nd pull request, but it can go here too.
This 2nd commit greatly reduces the overhead of hashCode() computation in the rendering cycle as well.    These hashcode computations, which make calls to Objects.hash() create a temporary Object[] array, autobox the boolean and integer values, and are expensive due to this overhead.     I ran some tests on JDK17 and JDK19 using JMH and these Object[] creations are not elided in after inlining and after Escape Analysis.    One of our simplest Ajax callbacks results in 140 hashcode calls to the Abstract class.   In total, these changes avoid the creation of about 11KB of garbage memory in the server _per_mouse_click.     This code also runs in ~1ns, compared to ~15ns for the original code (on a M1 Ultra Macbook, JDK19).     On older JDKs, like JDK11, the original code is much more expensive by comparison.

@theigl I just saw your note.   I've been using git for some time, but new to guthub.   I'll have to take some time to figure out how to remove this 2nd commit and move it into a new PR.    That was my original intent, and didn't expect the 2nd commit to show up in this PR.

@astange1: There is some related code that might also benefit from using List.of.  E.g.
https://github.com/apache/wicket/blob/c6d18e697b699723ca2cc3788e64b56f5929445e/wicket-core/src/main/java/org/apache/wicket/markup/head/JavaScriptContentHeaderItem.java#L75
And all other occurrences of Arrays.asList in org.apache.wicket.markup.head.

I would be happy to proceed and make those needed edits.   I did just that one for two reasons:

it shows up as being a somewhat hot in some of my profiles.
I figured I would do that one to start.   If that simple PR was accepted then I would gladly do more, and if not then I hadn't wasted too much of my time.

Let me know how you would like to proceed.    I can add the changes in this PR, or submit a new one.

I figured I would do that one to start. If that simple PR was accepted then I would gladly do more, and if not then I hadn't wasted too much of my time.

That sounds good to me. Let's keep it simple. Also I just realized that List.of is not null-safe. So we probably can't use it as a universal replacement for header items anyway.


Unable to install the library

Hi,
I am trying to install the library using the pip and getting following error on Ubuntu 16
$pip install swarmlib

ERROR: Could not find a version that satisfies the requirement swarmlib (from versions: none)
ERROR: No matching distribution found for swarmlib

can you help?Thanks

Could it be that you use python2‘s pip? Please try pip3 install swarmlib

Hi,
I am getting the same error with pip3
$ sudo pip3 install swarmlib
/usr/local/lib/python2.7/dist-packages/pip/_vendor/requests/__init__.py:83: RequestsDependencyWarning: Old version of cryptography ([1, 2, 3]) may cause slowdown.
  warnings.warn(warning, RequestsDependencyWarning)
DEPRECATION: Python 2.7 reached the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 is no longer maintained. pip 21.0 will drop support for Python 2.7 in January 2021. More details about Python 2 support in pip, can be found at https://pip.pypa.io/en/latest/development/release-process/#python-2-support
ERROR: Could not find a version that satisfies the requirement swarmlib (from versions: none)
ERROR: No matching distribution found for swarmlib

Seems like also your pip3 command refers to python2's pip. This package requires minimum python 3.5. Ensure you have installed python ≥ 3.5 and then run python3 -m pip install swarmlib.
You can check your python's version by running python3 --version.


⚠ Bofrost.lu has degraded performance

In ff10495, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 910 ms

Resolved: Bofrost.lu performance has improved in af791a4.


Android - Add more android keycode support

What does the pull request do?
Adds more key mappings for common keycodes on android.
What is the current behavior?

What is the updated/expected behavior with this PR?

How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues
Fixes #11372

You can test this PR using the following package version. 11.0.999-cibuild0035200-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]


add missing export variable

Add CLASS_LOADER_PUBLIC to ClassLoader::has_unmananged_instance_been_created_ or it won't build on Windows

@johnsonshih do you have an example where this is needed? or a reproducible example of this not building ?
I'm not fully familiar with visibility macros for Windows but it seems surprising to me that private attributes need to be declared public.
The code on the ros2 branch (that has the same set of visibility macros) seems to build fine on Windows 10 (VS2015 and VS2017).

The build will fail when building shared_ptr_test.cpp with unresolved external symbol class_loader::ClassLoader::has_unmananged_instance_been_created_ if we don't add the declaration.  The name of CLASS_LOADER_PUBLIC macro is misleading, it's not to declare a entity public but specify the storage-class attribute when it's built on Windows (or visibility on non-Windows environment).  The storage-class attribute (dllexport or dllimport)is used to tell the compiler if the current built component need to provide the entity instance.  If the current built dll provide the entity, need to set to dllexport or nothing.  If the current built dll consumes the entity, the attribute need to set to dllimport.  has_unmananged_instance_been_created_ is a static value, and is provided by Class_Loader.dll, so we need to use CLASS_LOADER_PUBLIC to specify the attribute so compiler know how to handle this value.

hi @nuclearsandwich, just a friendly ping so this pull request won't be lost in the mail box

hi @nuclearsandwich, just a friendly ping so this pull request won't be lost in the mail box

Thanks @kejxu. As an update, the prerelease job is not behaving stably. Which is why these PRs are still in flight. I don't think any one of them is the cause, but I am still trying to get the pre-release workspace to build, let alone run tests.

get the pre-release workspace to build, let alone run tests

My apologies for the spam, I thought the pre-release check was referring to the CI checks. Please take your time with the build =)

@nuclearsandwich ping! Just in case this got lost in your backlogs. 😃


Running multiple listener instances requires partitions assigned?

Hi, just experimenting with Phobos at the moment (thanks, it's cool!) and running into an issue. This possibly stems from not fully understanding Kafka, and not so much Phobos.
I setup a handler, and then in 2 separate shells start separate instances.  This results in "Listener crashed, waiting 0.86s (No partitions assigned!)" messages.
Any docs/examples as to how to deal with this? Or is this not the intended use? I'm not totally clear on the concurrency of the handlers, I believe I'm thinking in sidekiq terms where you set a thread concurrency so that a process running waiting on say, network io, doesn't block handling of other requests.
Thanks for your time!
EDIT: I'm thinking max_concurrency in the config might be part of this story.  I'm thinking though of instances where you might want to run multiple consumers of the same topic because of memory concerns per instance. I'm not sure if this works with an 'at least once' consumption setup though, if each listener would consume the entry (perhaps Phobos DB Checkpoint deals with it?), or because they are part of the same consumer group that is balanced out. Again, probably need to read up more on Kafka.

Hi @bemurphy, you can use the Phobos::Executor to start and stop your listeners easily. About the error, when you let kafka create the topic automatically it usually creates a topic with a single partition and the way you parallelize work in kafka is consuming different partitions concurrently. The error No partitions assigned! means that kafka couldn't assign a different partition to this listener.
You can take a look here https://github.com/klarna/phobos/blob/master/utils/create-topic.sh
on how to create a topic with more partitions.

@tulios great! Thanks so much for the pointer to the example in the shell script, that will be useful in experimenting with this some more. I'll close this issue for now as I suspect I can run with that from here.  Thank you again, I appreciate it.

👌


PageSpeed Optimized CSS and CORS Headers

How can I add CORS headers to pagespeed optimized css files? I have the following location block, and it is not matching the optimized files
location ~*.*\.(?:woff|woff2|ttf|otf|eot|js|css)$ {
    add_header 'Timing-Allow-Origin' "*";
    add_header 'Access-Control-Allow-Origin' "*";
    add_header "Access-Control-Allow-Methods" "GET";
    expires 12M;
    try_files $uri @backend;
}

I also tried adding the following config and it too did not work
location ~ "\.pagespeed\.(?:[a-z]\.)?[a-z]{2}\.[^.]{10}\.css" {
    add_header 'Timing-Allow-Origin' "*";
    add_header "Access-Control-Allow-Origin" "*";
    add_header "Access-Control-Allow-Methods" "GET";
}

Is it possible to add custom headers to CORS headers to pagespeed optimized resources in Nginx?

I think we're going to need to add something to let people set these.  Right now PageSpeed is careful to keep add_header commands from affecting the headers it sets on optimized resources because most add_header commands that are already in someone's config aren't the ones you'd want for our resources.  But there needs to be a way to say "I'm specifically trying to control the headers on .pagespeed. resources, set this please".

Actually, as long as you're not using LoadFromFile or outlining filters this should work.  If you set a non-caching header on foo.js, then when pagespeed creates foo.js.pagespeed.ce.HdziXmtLIV.js it should include the header.  You would need to flush your http cache, though, to pick up the new header if the input resource had a long cache lifetime.

(And if you have combiners than only headers present on all the input resources will be included on the output resource.)

@jeffkaufman there's also https://github.com/pagespeed/ngx_pagespeed/pull/906

@jeffkaufman Thank you for the reply. I am using LoadFromFile, outlining and combiners.
It would be great if there was a new directive, say pagespeed_add_header for adding headers to optimized resources. Or maybe PageSpeed could maintain a blacklist of headers that should not be applied on a optimized resource, and allow all other headers. The first option would provide much more control, IMHO. Is it possible to do something like this?

It looks like we don't need pagespeed_add_header because we're already maintaining a blacklist as you would like us to.  Basically, we blacklist caching related headers, and so any other headers you set on the original resource will get included on the pagespeed-optimized resource.

But then how can we target resources loaded using LoadFromFile?

Sorry, yes.  We do need a directive like this for LoadFromFile and also for (far less common) outlined resources.

Thank you for the clarification. So, can this issue be accepted as a feature request? Or do I have to file one elsewhere?

Added to https://github.com/pagespeed/mod_pagespeed/wiki/Accepted-Feature-Requests

One important use case here would be no-transform headers.

I am unable to use the new crossorigin="anonymous" attribute for script tag, because of the missing headers. Firefox will not evaluate the scripts without the correct CORS headers. Is there an ETA for this feature?

No one is currently working on this, sorry!

You're right though, that this should be higher priority.  I've pushed it up higher on https://github.com/pagespeed/mod_pagespeed/wiki/BugFix-Prioritization

Is there any work around for this?

There might be a workaround, this patch moves ngx_pagespeed's module position: https://gist.github.com/oschaaf/f9f71f493da928f07dba
Patching in that change moves ngx_pagespeed in the module order so that you can override headers for pagespeed resources like this:
location ~ "\.pagespeed\.([a-z]\.)?[a-z]{2}\.[^.]{10}\.[^.]+" {
    add_header 'Timing-Allow-Origin' '*';
}

trunk-tracking has a change that allows you to position ngx_pagespeed like this through an option. Over there, this change passes all system tests except SSI. I just manually tried the linked patch on the latest release and it worked for me.

@oschaaf It failed for me with the following message

[root@mars ngx_pagespeed-release-<IP_ADDRESS>-beta]# patch < nps_pos_aux.diff
patching file config
Hunk #1 FAILED at 193.
Hunk #2 succeeded at 201 with fuzz 1 (offset -10 lines).
1 out of 2 hunks FAILED -- saving rejects to file config.rej

@JoyceBabu Sorry, I pasted the wrong patch, that one was for the master branch.
I updated it to work for ngx_pagespeed-release-<IP_ADDRESS>-beta, could you try again?

@oschaaf Thank you. I have manually edited the config file and applied the changes. It is now working correctly.
Should I try again with the new patch?

@JoyceBabu If the new patch looks correct when you compare it to your changes in your config file you don't need to try again as far as I'm concerned.
Are you able to set the headers now?

I tried the new patch and it also throwing an error, even though the patching itself is successful

patching file config
patch unexpectedly ends in middle of line
Hunk #1 succeeded at 193 with fuzz 1.

Looks like it is just a warning
http://unix.stackexchange.com/questions/1395/what-does-patch-unexpectedly-ends-in-middle-of-line-mean

The cross origin headers are being duplicated for pagespeed optimized resources. For example, I am getting Access-Control-Allow-Origin: *, * instead of Access-Control-Allow-Origin: *. For original file, the header is correct.

@JoyceBabu Do you have headers-more available? If you use more_set_headers instead of add_header I think that the duplication will be gone.

Is the new option available in release v<IP_ADDRESS>?

@JoyceBabu changes are pending that have been reviewed but have not yet been merged, so they are not yet available in a release.

@oschaaf I was trying to cherry pick the fix, and found out that the patch is for mod_pagespeed. Is it not available for ngx_pagespeed?

@JoyceBabu You would need to build ngx_pagespeed against a PSOL version that includes this. No changes required for ngx_pagespeed.

Would it be a terribly bad idea to run PSOL master in production? Also, should the version of PSOL match with the version of ngx_pagespeed? Is it necessary to run PSOL master with ngx_pagespeed master?

ngx_pagespeed:master builds against the latest PSOL release.  The main reason to do this would be if you wanted to apply a patch to PSOL, but not get full updates.  This is relatively safe, especially if your patch is simple, small, and well understood.
ngx_pagespeed:trunk-tracking builds against mod_pagespeed:master.  This is what you'd do if you wanted to run at the complete bleeding edge.  In general I wouldn't recommend this for production unless you canaried it first and didn't see any issues.  (If you did see issues, though, your finding them early would be really useful!)

I meeet something like this:

Failed to load https://a.com/tips_v3.7.json: The 'Access-Control-Allow-Origin' header contains multiple values '*, *', but only one is allowed. Origin 'https://a.com' is therefore not allowed access.

And this is my Nginx configuration:
        .......
        pagespeed on;
	# Default
	# Needs to exist and be writable by nginx.  Use tmpfs for best performance.
	pagespeed FileCachePath /var/ngx_pagespeed_cache;

	# Ensure requests for pagespeed optimized resources go to the pagespeed handler
	# and no extraneous headers get set.
	location ~ "\.pagespeed\.([a-z]\.)?[a-z]{2}\.[^.]{10}\.[^.]+" {
	  add_header "" "";
	}
	location ~ "^/pagespeed_static/" { }
	location ~ "^/ngx_pagespeed_beacon$" { }

	location ~* \.(json)$ {
	    if ($http_referer = "") {   return 403;  }
            add_header 'Access-Control-Allow-Origin' *;
            add_header 'Access-Control-Allow-Methods' 'GET, POST, OPTIONS';
            add_header 'Access-Control-Allow-Credentials' 'true';
            add_header 'Access-Control-Allow-Headers' 'User-Agent,Keep-Alive,Content-Type';

            expires 7d;
	    log_not_found off;
	}

I tried to remove add_header 'Access-Control-Allow-Origin' *; from location ~* \.(json)${} block, and rm -rf /var/ngx_pagespeed_cache/*,  but seems it it still not work.
The JSON file worked fine before enabling ngx_pagespeed module...

I got these headers:

So may I guess problem is this?
location ~ "\.pagespeed\.([a-z]\.)?[a-z]{2}\.[^.]{10}\.[^.]+" {
	  add_header "" "";
}

How can I fix it? 😭

I also tried to clean cache, still not work 😢


sstarcher/sensu:latest container has rebuilt 14 times and counting  in the last 24 hours

Is there something that is causing the latest image to rebuild 14 so many times in the last day?

It was linked to the debian repo which must of pushed out alot of updates.  Now that we are building off of ruby I unlinked it from debian.
Thanks


added Happy Valley-Goose Bay, NL, CAN

For posterity, this was fetched from http://maps.happyvalley-goosebay.com/layers/CivicAddresses2.js, which Justin found at http://maps.happyvalley-goosebay.com/.

Thx!  I'll add a note to the source.


Test Git - Ability to clean up diff editors

Refs: https://github.com/microsoft/vscode/issues/141475

[ ] anyOS @alexdima
[x] anyOS @karrtikr

Complexity: 3
Create Issue

This milestone we have made some improvements in order to address a paper cut of having to manually close the diff editors. We have added both a new command, Git: Close All Diff Editors, as well as a new setting, git.closeDiffOnOperation, that would automatically close the diff editors after particular git operations.
Test plan (command)
Prerequisites

Open settings and make sure that the git.closeDiffOnOperation is not enabled.

Steps

Open a folder/workspace that contain one of more git repositories
Make changes to multiple files
Switch to the SCM viewlet and stage some of the changes
Click on changes, staged files to open multiple diff editors (working tree, and index)
Invoke the Git: Close All Diff Editors command from the command palette

Confirm that all diff editors (working tree, and index) have been closed

Test plan (setting)
Prerequisites

Open settings and make sure that the git.closeDiffOnOperation is enabled.

Stash changes

Open a folder/workspace that contain one of more git repositories
Make changes to multiple files, and switch to the SCM viewlet
Click on the changes to open multiple diff editors (working tree)
Invoke the command to create a new stash based on the changes

Confirm that all diff editors (working tree) have been automatically closed

Discard changes

Make changes to multiple files, and switch to the SCM viewlet
Click on the changes to open multiple diff editors (working tree)
Click on the button to discard the changes for one/more files

Confirm that the diff editors (working tree) for the particular files have been automatically closed

Stage changes

Make changes to multiple files, and switch to the SCM viewlet
Click on the changes to open multiple diff editors (working tree)
Click on the button to stage one/more changes

Confirm that the diff editors (working tee) for the staged files have been automatically closed

Unstage changes

Click on one/more of the staged files to open one/more diff editors (index)
Click on the button to unstage one/more files

Confirm that the diff editors (index) for the unstaged files have been automatically closed

Commit changes

Click on one/more of the changes to open one/more diff editors (working tree)
Click on the button to commit the changes

Confirm that the diff editors (working tree) for the committed files have been automatically closed

It looks like this needs to be retested with a new Insiders because there was proposed API breakage -- https://github.com/microsoft/vscode/issues/143746#issuecomment-1048897416

@alexdima another fix went in just now, so the experience should be fixed in the next Insiders release. Thanks @lramos15!

Works well today!


extra semicolon after functino

hi
when i use predefined function name an extra semicolon added at the end of function that really bad.
i think if it will delete will be better
Thanks

Which snippet are you referring to?

Close due to inactivity


Make contexts reactive

Currently, It's not possible to listen to camera changes or pointer position changes.
The proposed changes would change the ThrelteContext to this:
type ThrelteContext = {
  size: Writable<{ width: number; height: number }>
  pointer?: Writable<Vector2>
  clock: Clock
  camera?: Writable<Camera>
  scene: Scene
  renderer?: WebGLRenderer
  composer?: EffectComposer
  invalidate: (reason?: string) => void
}

So it's not the whole context that's a store but rather single properties. This makes it possible to destructure the result of useThrelte like const { invalidate } = useThrelte().
But listening to camera changes would make HUD visibility and general UI work easier and more performant.

Same with  url for model. It's not updated when model url is changed.


How do I populate more than one group of forms?

I get API information via @Input, groups are split into tabs, and each group wants popular with such information.

@MuriloEduardo

How do I populate more than one group of forms?

Your use case is beyond the scope of the lib. Sorry.


How to add icon for submenu in sidebar?

This must be a very newbies question. But I have been trying for two days on my own without success.
E.g., I want to add the icon "flashlight in entypo" onto submenu of "Charts" in "Statistics".
Could anybody please give me some hint? Thanks so much!

HI @LucasMeier !
We haven't implemented it, we could add it in our next milestone, what do you think?

That will be great! Thank you!

@smartapant That will be great! Thank you!

Added docs


[NO-TICKET] Update retest description with dependency on PtaaS Tier

@ana-dashuk-cobalt as I go through our plan to convert Zendesk articles, I realized that, for "retesting," we do not include any reference to the PtaaS tier.

@mjang-cobalt Actually, we have https://zombie.atlassian.net/browse/DOCS-190 for the Free Retesting feature that will be updated in Q3. The expected launch date is September 30. I'm keeping track of this issue.
Once the functionality is updated, we can add something like:
The duration of free retesting for your findings depends on your {{% ptaas-tier % }}.


Fix parameter types int -> integer changes

[x] documentation is added or updated
[ ] tests are added or updated

BTW, why is 'int' changed to 'integer;? For non-casualness purposes?

We use 'integer' rather than 'int' when describing OSSL_PARAMs with the data type OSSL_PARAM_INTEGER, so as not to have the reader confuse that with the C type int.

For parameters we use integer and unsigned integer because they aren't related to the C type int.

This pull request is ready to merge

Merged to master
2c3f1a6cad300e3812aee34dab98f23a329296f4 Fix parameter types int -> integer changes


Terminal: copy under Windows should insert Windows line delimiter character

Currently only \n is inserted.

Upstream: https://github.com/sourcelair/xterm.js/issues/737


kiali doesn't work out of box in istio 1.3

Unable to reach Kiali UI:
istioctl dashboard kiali is pointing to http address of kiali (http://localhost:59157/kiali) which failed with Client sent an HTTP request to an HTTPS server.
same error if i use port-forwarding - http://localhost:20001/kiali/console
Kiali team suggested we should move up kiali to 1.3 or 1.4.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[x ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ x] User Experience
[ ] Developer Infrastructure
Expected behavior
Steps to reproduce the bug
Version (include the output of istioctl version --remote and kubectl version)
How was Istio installed?
Environment where bug was observed (cloud vendor, OS, etc)
Additionally, please consider attaching a cluster state archive by attaching
the dump file to this issue.

In kiali 1.4 http still fails, you need https it seems

Does anyone have kiali logs? http is supported. you are not required to use https. But seeing logs would help determine the problem

In order to tell Kiali to use http, you must define "identity.cert_file" with a value of empty string in the Kiali CR. If you are not using the Kiali Operator, that value is in the kiali configmap.
See: https://github.com/kiali/kiali/blob/v1.4/operator/deploy/kiali/kiali_cr.yaml#L344-L359

Kiali logs with 1.4:
I0910 18:42:23.107942       1 kiali.go:66] Kiali: Version: v1.4.0, Commit: a4925d87c630755e68ca5fc4f7845b3c37e517e1
I0910 18:42:23.108806       1 kiali.go:210] Using authentication strategy [anonymous]
W0910 18:42:23.108833       1 kiali.go:218] Kiali auth strategy is configured for anonymous access - users will not be authenticated.
I0910 18:42:23.108884       1 kiali.go:87] Kiali: Console version: 1.4.0
I0910 18:42:23.108903       1 kiali.go:272] Updating base URL in index.html with [/kiali]
I0910 18:42:23.109379       1 kiali.go:253] Generating env.js from config
I0910 18:42:23.141464       1 server.go:55] Server endpoint will start at [:20001/kiali]
I0910 18:42:23.141578       1 server.go:56] Server endpoint will serve static content from [/opt/kiali/console]
I0910 18:42:23.141659       1 metrics_server.go:18] Starting Metrics Server on [:9090]
I0910 18:42:23.141889       1 server.go:61] Server endpoint will require https
2019/09/10 18:44:02 http: TLS handshake error from <IP_ADDRESS>:54696: remote error: tls: unknown certificate
2019/09/10 18:44:02 http: TLS handshake error from <IP_ADDRESS>:54700: remote error: tls: unknown certificate
2019/09/10 18:44:08 http: TLS handshake error from <IP_ADDRESS>:54710: remote error: tls: unknown certificate
2019/09/10 18:44:08 http: TLS handshake error from <IP_ADDRESS>:54712: remote error: tls: unknown certificate
E0910 18:44:09.886931       1 grafana.go:35] grafana URL is not set in Kiali configuration
E0910 18:44:09.897064       1 jaeger.go:27] Jaeger URL is not set in Kiali configuration
W0910 18:44:10.228166       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
2019/09/10 18:44:10 http: TLS handshake error from <IP_ADDRESS>:54732: remote error: tls: unknown certificate
W0910 18:44:21.372777       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:44:25.303199       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:44:26.571887       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:44:30.270940       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:45:09.935074       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:45:26.297039       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:45:41.029331       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:45:56.312873       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:46:11.623966       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:46:26.995575       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:46:42.292108       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:46:57.386704       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:47:12.571837       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:47:27.725521       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:47:42.921717       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:47:58.083039       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:48:13.305680       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:48:28.482979       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:48:43.716719       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:48:58.880784       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:49:14.160733       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:49:29.441945       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:49:44.548488       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:49:59.717782       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:50:14.930049       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:50:30.580827       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:50:45.770223       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:51:00.909524       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:51:16.480996       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:51:32.460629       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:51:48.477629       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope
W0910 18:52:04.468383       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope

Request to kiali return: Client sent an HTTP request to an HTTPS server.
I can send https traffic though

Yes, with https enabled, it will refuse http traffic for security purposes.
As for this:
W0910 18:51:32.460629       1 tls.go:56] GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope

That is easily corrected by modifying the role.yaml in the kiali helm chart to give it that permission.

I suspect this permission is missing in the Istio helm chart: https://github.com/kiali/kiali/blob/master/operator/deploy/role.yaml#L238

This exists in the current master of Istio installer: https://github.com/istio/installer/blob/master/istio-telemetry/kiali/templates/clusterrole.yaml#L98
and it is in the 1.3 istio installer: https://github.com/istio/installer/blob/release-1.3/istio-telemetry/kiali/templates/clusterrole.yaml#L98
So I'm not sure why kiali was not given these permissions. How is Kiali being installed here? I was assuming this new Istio installer is being used, but maybe not?

Kiali works with Istio 1.3 out of the box if you configure it by passing --set kiali.security.enabled=false. Someone change the default to have security enabled (which sets the identity.cert_file) but we haven't updated our docs/istioctl to ues https so we should have that set.

Although I am still seeing the server could not find the requested resource (get meshpolicies.authentication.istio.io) even though the clusterrole has this...
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"rbac.authorization.k8s.io/v1","kind":"ClusterRole","metadata":{"annotations":{},"labels":{"app":"kiali","chart":"kiali","heritage":"Tiller","release":"istio"},"name":"kiali"},"rules":[{"apiGroups":[""],"resources":["configmaps","endpoints","namespaces","nodes","pods","pods/log","replicationcontrollers","services"],"verbs":["get","list","watch"]},{"apiGroups":["extensions","apps"],"resources":["deployments","replicasets","statefulsets"],"verbs":["get","list","watch"]},{"apiGroups":["autoscaling"],"resources":["horizontalpodautoscalers"],"verbs":["get","list","watch"]},{"apiGroups":["batch"],"resources":["cronjobs","jobs"],"verbs":["get","list","watch"]},{"apiGroups":["config.istio.io"],"resources":["adapters","apikeys","bypasses","authorizations","checknothings","circonuses","cloudwatches","deniers","dogstatsds","edges","fluentds","handlers","instances","kubernetesenvs","kuberneteses","listcheckers","listentries","logentries","memquotas","metrics","noops","opas","prometheuses","quotas","quotaspecbindings","quotaspecs","rbacs","redisquotas","reportnothings","rules","signalfxs","solarwindses","stackdrivers","statsds","stdios","templates","tracespans","zipkins"],"verbs":["create","delete","get","list","patch","watch"]},{"apiGroups":["networking.istio.io"],"resources":["destinationrules","gateways","serviceentries","sidecars","virtualservices"],"verbs":["create","delete","get","list","patch","watch"]},{"apiGroups":["authentication.istio.io"],"resources":["meshpolicies","policies"],"verbs":["create","delete","get","list","patch","watch"]},{"apiGroups":["rbac.istio.io"],"resources":["clusterrbacconfigs","rbacconfigs","servicerolebindings","serviceroles"],"verbs":["create","delete","get","list","patch","watch"]},{"apiGroups":["monitoring.kiali.io"],"resources":["monitoringdashboards"],"verbs":["get","list"]}]}
  creationTimestamp: "2019-09-10T21:35:28Z"
  labels:
    app: kiali
    chart: kiali
    heritage: Tiller
    release: istio
  name: kiali
  resourceVersion: "975"
  selfLink: /apis/rbac.authorization.k8s.io/v1/clusterroles/kiali
  uid: 7e32a40e-157f-4e40-a645-eabb22d35d4a
rules:
- apiGroups:
  - ""
  resources:
  - configmaps
  - endpoints
  - namespaces
  - nodes
  - pods
  - pods/log
  - replicationcontrollers
  - services
  verbs:
  - get
  - list
  - watch
- apiGroups:
  - extensions
  - apps
  resources:
  - deployments
  - replicasets
  - statefulsets
  verbs:
  - get
  - list
  - watch
- apiGroups:
  - autoscaling
  resources:
  - horizontalpodautoscalers
  verbs:
  - get
  - list
  - watch
- apiGroups:
  - batch
  resources:
  - cronjobs
  - jobs
  verbs:
  - get
  - list
  - watch
- apiGroups:
  - config.istio.io
  resources:
  - adapters
  - apikeys
  - bypasses
  - authorizations
  - checknothings
  - circonuses
  - cloudwatches
  - deniers
  - dogstatsds
  - edges
  - fluentds
  - handlers
  - instances
  - kubernetesenvs
  - kuberneteses
  - listcheckers
  - listentries
  - logentries
  - memquotas
  - metrics
  - noops
  - opas
  - prometheuses
  - quotas
  - quotaspecbindings
  - quotaspecs
  - rbacs
  - redisquotas
  - reportnothings
  - rules
  - signalfxs
  - solarwindses
  - stackdrivers
  - statsds
  - stdios
  - templates
  - tracespans
  - zipkins
  verbs:
  - create
  - delete
  - get
  - list
  - patch
  - watch
- apiGroups:
  - networking.istio.io
  resources:
  - destinationrules
  - gateways
  - serviceentries
  - sidecars
  - virtualservices
  verbs:
  - create
  - delete
  - get
  - list
  - patch
  - watch
- apiGroups:
  - authentication.istio.io
  resources:
  - meshpolicies
  - policies
  verbs:
  - create
  - delete
  - get
  - list
  - patch
  - watch
- apiGroups:
  - rbac.istio.io
  resources:
  - clusterrbacconfigs
  - rbacconfigs
  - servicerolebindings
  - serviceroles
  verbs:
  - create
  - delete
  - get
  - list
  - patch
  - watch
- apiGroups:
  - monitoring.kiali.io
  resources:
  - monitoringdashboards
  verbs:
  - get
  - list

@howardjohn - I think having http for demo profile is okay for user to get started.  In your case, I assume the workload graph would not work with this the server could not find the requested resource (get meshpolicies.authentication.istio.io)?

In your case, I assume the workload graph would not work with this the server could not find the requested resource (get meshpolicies.authentication.istio.io)?

It seems to work, at least partially. I am not sure why I get this error though, as it should have permissions

MeshPolicy issue is due to the kubernetes vulnerabilities that have been patched. If you run unpatched kubernetes it works :woman_shrugging: see https://github.com/kiali/kiali/issues/1337

Fixes:

Turn off security https://github.com/istio/istio/pull/16991
Update to 1.4.0 https://github.com/istio/istio/pull/16993

There is one issue for the meshpolicies.authentication.istio.io problem - https://github.com/istio/istio/issues/16103

Fixed in 1.3.0

Should this be fixed in the kiali operator as well? I have the same problem, on bleeding edge versions (istio 1.3, kiali 1.4.2) from the kiali operator.

The problem is (I think) this istioctl command. I think it is starting a proxy that isn't able to pass through https traffic. I am not sure about that - but I think the problem originates in that istioctl dashboard kiali command.

@xeor if you are using the Kiali Operator, in your Kiali CR, set "identity.cert_file" and "identity.private_key_file" both to empty string. That turns off https and should then work for you. See: https://github.com/kiali/kiali/blob/master/operator/deploy/kiali/kiali_cr.yaml#L344-L359
This should work around what I think is a problem in the istioctl dashboard kiali command - whatever it is doing, it is only allowing http traffic.

I had cert_file="", but was missing private_key_file="".. Still looks like I get the error tho...
GetMeshPolicies failed during a TLS validation. Probably Kiali doesn't have cluster permissions. Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope :(

Error: meshpolicies.authentication.istio.io is forbidden: User "system:serviceaccount:istio-system:kiali-service-account" cannot list resource "meshpolicies" in API group "authentication.istio.io" at the cluster scope

That is a different issue than the http vs. https thing. That was the bug fix that was put in recently: https://github.com/kiali/kiali/issues/1337


[Question] Questions about zero_embed and weights

Thanks for this great work. I'm recently interested in using diffusion model to generate images iteratively. I found your script cfg_sample.py was a nice implementation and I decided to learn from it. However, because I'm new in this field, I've encountered some problems quite hard to understand for me. It'd be great if some hints/suggestions are provided. Thank you!!
My questions are listed below. They're about the script cfg_sample.py.

I noticed in the codes, we've used zero_embed as one of the features for conditioning. What is the purpose of using it? Is it designed to allow the case of no prompt for input?
I also notice that the weight of zero_embed is computed as 1 - sum(weights), I think the 1 is to make them sum to one, but actually the weight of zero_embed could be a negative number, should weights be normalized before all the intermediate noise maps are weighted?

Thanks very much!!

It is for classifier-free diffusion guidance. The zero embedding means "no prompt", generate unconditionally. (The model was trained with 20% of the input embeddings replaced at random with the zero embedding.) We can take the difference between the unconditional distribution and the conditional distribution and strengthen it and generate from the artificially "super-conditional" distribution, thus producing images that fit the prompt better than you would get from just inputting the prompt's embedding.
The weight of the zero embedding is supposed to be negative, say you want to generate with a cond_scale of 3. Then your prompt's weight should be 3 and the zero embedding's weight should be -2 (they have to sum to 1 but do not otherwise have restrictions). This is because of the formula for classifier-free guidance with a single prompt:
v = v_uncond + (v_cond - v_uncond) * cond_scale.
If cond_scale is 3 then the first term is (1, 0) for uncond and cond and the second is (-3, 3). Add them and you get the weights (-2, 3).

Thanks for the quick reply! These are very helpful! If you don't mind, may I ask some other questions here?
After learning about the diffusion model, I am actually quite curious about whether it is possible to use the diffusion process to modify some images directly on the pixel space without manually adding noises or converting back to the noisy states?
As far as I know, diffusion models have been applied to many fields such as image generation, image inpainting and image editing, but they all start from gaussian noises and gradually denoise the images to recover. I also find that you also implement a reverse sampling process which allows to obtain the noise maps from the original images. I find that nearly all of them require some noisy images to start their denosing processes. Is it really necessary to start from a noisy image? Is it possible to directly operate on a clean image (without any noise) via iterative updates?
About this, I've conducted some toy experiments. They show that the initial images will not change too much, their shapes only blur a bit, and some high-frequency details are lost after denosing.
Thanks again!

Really appreciate for the detailed reply!! But I think my doubt hasn't been fully resolved.
You did mentioned a possible solution of reversing a clear high-SNR image back to a relative noisy low-SNR image. It seems that in order to utilize a diffusion model, we may have to convert our input clear image back to the noisy one that the model can tackle and denoise.
Is it right that a diffusion model must starts from some noisy images (i.e. images with low SNR with noisy property, and look like a noisy image)? If it starts from a clear image (i.e. not reverse back to a noisy one), is it likely to fail to modify the image contents?


update: Update anchore/sbom-action to v0.15.1

Rerunning pipeline twice also fixed the issue with sbom


[Admin] Publish

Docs Build status updates of commit 26b4365:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/concepts/correlated-objects-pattern.md
:white_check_mark:Succeeded
View

docs/concepts/privacy-and-security.md
:white_check_mark:Succeeded
View

docs/design/contextual-tabs.md
:white_check_mark:Succeeded
View

docs/design/disable-add-in-commands.md
:white_check_mark:Succeeded
View

docs/develop/add-ins-with-angular2.md
:white_check_mark:Succeeded
View

docs/develop/asynchronous-programming-in-office-add-ins.md
:white_check_mark:Succeeded
View

docs/develop/bind-to-regions-in-a-document-or-spreadsheet.md
:white_check_mark:Succeeded
View

docs/develop/create-addin-commands.md
:white_check_mark:Succeeded
View

docs/develop/create-sso-office-add-ins-aspnet.md
:white_check_mark:Succeeded
View

docs/develop/create-sso-office-add-ins-nodejs.md
:white_check_mark:Succeeded
View

docs/develop/debug-office-add-ins-in-visual-studio.md
:white_check_mark:Succeeded
View

docs/develop/dialog-api-in-office-add-ins.md
:white_check_mark:Succeeded
View

docs/develop/dialog-best-practices.md
:white_check_mark:Succeeded
View

docs/develop/initialize-add-in.md
:white_check_mark:Succeeded
View

docs/develop/loading-the-dom-and-runtime-environment.md
:white_check_mark:Succeeded
View

docs/develop/office-javascript-api-object-model.md
:white_check_mark:Succeeded
View

docs/develop/read-and-write-data-to-the-active-selection-in-a-document-or-spreadsheet.md
:white_check_mark:Succeeded
View

docs/develop/run-code-on-document-open.md
:white_check_mark:Succeeded
View

docs/develop/show-hide-add-in.md
:white_check_mark:Succeeded
View

docs/excel/call-excel-apis-from-custom-function.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-authentication.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-batching.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-errors.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-json.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-json-autogeneration.md
:white_check_mark:Succeeded
View

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Whitespace fixes and multiboot2 module support

Fixes #7
I think I'd also like to add a ModuleIter but for that I'd have to change some other things with TagIter and such. Sorry for taking so long but it took me awhile to get a module working that I could test with.

Thank you very much!

@phil-opp, how would you suggest I go about implementing a ModuleIter?

What's the problem with the current TagIter?
The problem is that people depend directly on this git repo, since I never pushed it to crates.io. I just fixed this (multiboot2 is now on crates.io) and I try to update the blog_os post in the next days. However I still think we should avoid breaking changes in the near term.

The only problem that I have with TagIter is that there it is possible to get only one Tag of each type. There can be  multiple tags of the type module, and it is only possible to get the first one at the moment. This can be solved by either making TagIter public and having public methods on it, or create a ModuleIter struct.

Ah, thanks for the explanation. Are there any other tags with this property (multiple tags with same type)? If the module tag is the only tag with multiple copies, I would prefer the ModuleIter option.
In either case, I think we should rename the module_tag function to module_tags and return some kind of Iterator<&'static ModuleTag> from it.


Add completion generation for nushell, fig and carapace

Since this is called clap_allgen, it should probably generate all possible things, right? At least fig and nushell are upstream, but intentionally split into separate crates, with the carapace thing being a downstream thing from a clap perspective, as it comes from carapace themselves.

Since this is called clap_allgen, it should probably generate all possible things, right?

Anything less and you'd be eligible for a money back!
Thanks for this great PR 🙇♂


[FLINK-31297][Runtime] Use processResourceRequirements to allocate sk manager in FineGrainedSlotManagerTest

What is the purpose of the change
We use TaskManagerTracker.addPendingTaskManager to allocate task manager in FineGrainedSlotManagerTest. This make the resource requirements different between FineGrainedSlotManager and TaskManagerTracker. And the FineGrainedSlotManager may trigger resource recycling after the requirementCheckDelay.
So we use processResourceRequirements to allocate task manager. And set the requirementCheckDelay to 0 to stabilize this test.
Brief change log
(for example:)

use processResourceRequirements to allocate task manager
set the requirementCheckDelay to 0 to stabilize the test

Verifying this change

this test ran successfully 200 times

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)

@huwh Did you accidentally misplace the comment? This pr was created by yourself.

@JunRuiLee sorry，I mixed up the tabs


Hello.

i hope you can view this. I got Error: listen EACCES: permission denied C:\tmp\13078626-ce72-46d2-a63c-323798a253e0.sock error when i type /save 10 single

Ah I know this one. I bumped the version of a dependency. Could you try it again with the latest version?

i did edit this bot a bit. Like, i did add drive.google support. Which is able to get voice record unlimited. Thank you for your reply.

Anyway, to resolve the issue you probably need to bump the dependency @kirdock/discordjs-voice-recorder to version 1.0.4


[RESOLVED] High ICMP ping response time

Problem started at 08:14:29 on 2023.04.27
Problem name: High ICMP ping response time
Host: HP_iLO
Severity: Warning
Operational data: Value: 549ms
Original problem ID: 20015
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23268&eventid=20015

Problem has been resolved in 3m 0s at 08:17:29 on 2023.04.27
Problem name: High ICMP ping response time
Host: HP_iLO
Severity: Warning
Original problem ID: 20015


Travis / Appveyor CI

added AppVeyor and Travis CI config files
fixed build on Linux systems

AppVeyor build configurations:

Visual Studio 2017, Qt 5.13.2, 64-bit
MinGW 7.3.0, Qt 5.13.2, 64-bit

Travis CI build configurations:

Ubuntu 18.04, GCC 7, Qt 5.9.5
macOS 10.14, Xcode 11.3, Qt 5.13.2

Closes #182

I think it could be beneficial to stay on a LTS branch of Qt. This would be 5.12.x for the moment.
Just some time ago we had massive problems with incompatible changes in newer Qt versions (see #160). Some other incompatibilities exist for automatic definition update and Qt version.

I agreed that it is better to stick to LTS, but in case of public build servers it is not always possible at least in easy way...
for example, Qt 5.12.x is included only in "Visual Studio 2019" AppVeyor image (details), but Qt itself is build for MSVC2017. in general, this is not good idea to mix code compiled using different compiler versions, especially for Windows and especially for Qt. it can even don't link. or in worst case lead to unexpected and strange results on runtime.
moreover, only few Linux distributions have Qt 5.12.x (or above), and all of them have short lifetime (like Ubuntu 19.10) or updating very frequently and don't stick on specific version for a long time (like Arch). even Debian 10 doesn't have Qt 5.12.x, it uses Qt 5.11.x. only starting from Debian testing Qt 5.12.x is available in repos.
for macOS situation is even worst: there is only one easy way to get some library as binary - use Homebrew. this is some kind of try to invent something like Linux repos replacement for macOS. binaries from there can be used for building/testing purposes, but they must be used very carefully when distributing app for end users, just because the way how Homebrew works and libraries was build a lot of unexpected dependencies may appear or software "may think" that it works on some Linux system, not macOS. moreover, according to Homebrew' database, Qt 5.14.0 must be installed, why Qt 5.13.2 was used on build server is unknown to me... looks like uncontrolled environment, again. even more, it is a big question "is it possible to install previous library version using Homebrew?". at glance Homebrew provides only latest versions, and when its DB was updated, it is impossible to install previous version.
What I made supposed to be used just to ensure that everything can be built on any of supported OS. not including any runtime testing, just build, nothing more.
if you want something which will provide end user distributions, build custom Qt binaries for any required OS, upload it somewhere and use these binaries on public build servers instead of preinstalled ones. I can help with Qt build process for any OS (Win/macOS/Linux), in any configuration you want.

P.S> I saw debian folder in repo, so I assume you build deb packages. due to different Qt versions in various distros, I think code must be ready for it. I didn't analyze affected code very well, but it looks like you need to create some file with unique name. if this file must exist permanently, and before open its name must be calculated based on some known info, it is better to have own "conversion" (known info -> filename) function, which implementation is fully controlled by you.
using library generic functions like qHash() for such purposes is not a good idea, because they intended to be used only on runtime, not for permanent data. from version to verson algorithm of such functions may be changed, and data produced by previous function will not be equal to data produced by current. that is happened in your case as I understood.

You are right with your observations for different Qt versions across  the different operation systems. That means we have to sort out much details before automatic builds are working flawless.
For the releases we normally did static builds to circumvent use of library versions provided by the OS.
Minutor should work with all Qt versions from the last couple of years. Only 5.12 was a bigger problem. And you are right, we should replace qHash() with the current Qt implementation copied into put own code to prevent this error to happen again.

let's split this pull request into 2 independent:

first one adds CI config files (this one)
another fixes Linux build - https://github.com/mrkite/minutor/pull/206


SPARK-10878 Fix race condition when multiple clients resolves artifacts at the same time

What changes were proposed in this pull request?
When multiple clients attempt to resolve artifacts via the "--packages" parameter, they could run into race condition when they each attempt to modify the dummy "org.apache.spark-spark-submit-parent-default.xml" file created in the default ivy cache dir.
This patch changes the behavior to encode timestamp in the dummy module descriptor so each client will operate on a different resolution file in the ivy cache dir. In addition, this patch changes the behavior of when and which resolution files are cleaned to prevent accumulation of resolution files in the default ivy cache dir.
How was this patch tested?
Unit test added in SparkSubmitUtilsSuite

Can one of the admins verify this patch?

Update the title:
[SPARK-10878][Core] Fix race condition when multiple clients resolves artifacts at the same time

cc @JoshRosen @cloud-fan

ok to test

Test build #80324 has started for PR 18801 at commit 1ace5cc.

Test build #80324 has finished for PR 18801 at commit 1ace5cc.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80324/
Test FAILed.

retest this please

Test build #80351 has started for PR 18801 at commit 1ace5cc.

Test build #80351 has finished for PR 18801 at commit 1ace5cc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80351/
Test PASSed.

Your test case don't reflect the change you made to support multiple clients resolves artifacts at the same time, could you add a new test case or manually test that? Thanks!

@jiangxb1987 @kiszk
Thanks for the review.
Will update the PR shortly.

@Victsm kindly ping

gentle ping @Victsm

gentle ping @Victsm

Can one of the admins verify this patch?

+1 on using UUID and having a better test case

Can one of the admins verify this patch?

Anyone can take over this PR and submit it again?

I can take over this PR.

@kiszk Thank you!

@Victsm you can close this PR now.


Added markdown and reStructuredText filters

and I moved the filters into their own module and stuff in hopes of making it easier to add/extend them.
I also reformatted some small parts of the code to make it a bit easier for me to read (I got the pylint score up from 3 to 6 for nodes.py), so apologies for that cluttering up the diff's;  I hope you don't mind too much :)

Thanks a lot for your work!
I like much more the clean code and the filters you have added :)

Your modify of nodes.py is not working at all in your commit, this is caused by a malformation of the equilibrate_parenthesis function.
Is fixed in the new commits :)

oh sorry, I just checked to make sure that all the template tests in test.py worked, but other than that, I didn't go too deep into it because I wasn't sure of what everything did. Glad it was a quick fix though.


Add "User Templates" option

Hi there,
I'd like to see an option to create  com.apple.dock.plist files in the user templates folder.
That way we could run 1 line that would update all current users docks & all templates docks, so new users get the changes.
Current workaround is to deploy a blank dock & then run each command twice:
The home templates are updated, but not all homes.. so presumably i need to run two lines of script everytime i wish to add an item:

For User Templates:
sudo /usr/bin/dockutil --add '/path to some app' --homeloc '/System/Library/User Template/'
For All Existing Users:
sudo /usr/bin/dockutil --add '/path to some app' --allhomes

Also, as (in @ least 10.8) the templates folders are empty.. create a blank plist OR copy the template from: /System/Library/CoreServices/Dock.app/Contents/Resources/.lproj/default.plist.
Thanks.

@diwanicki @elvisizer @macmule @kcrawford
Add my vote for this feature. In fact I came here specifically looking for a solution to manage the dock plist for the User Template.
Note that Catalina now users /Library/User Template/ as the location for the User Template folder as /System/Library/User Template is protected by SIP.
I did a sudo defaults read /Library/User Template/English.lproj/Library/Preferences/com.apple.dock.plist (and ones in other languages) and none of them seem to contain entries for dock icons like they used to in Mojave. Even after logging in as a user thereby triggering the creation of a home directory a users own ~/Library/Preferences/com.apple.dock.plist does not list such entries either.
Are they stored elsewhere now?

For what it's worth, in Catalina it does not look like any of the com.apple.dock.plist files in /Library/User Template/ actually contain any dock entries.
I can only presume that the content is being generated dynamically when a new user account is created along with its home directory.
I would still like a solution to being able to customize the dock so it applies to new user accounts. Unless someone can identify from where the information is being generated then the only solution I can see would be as follows.

Install dockutil
Install outset
Configure outset to run a login-once script to clean up a users dock, as this would only be run once it would remove (in my case) the broken icons but unlike settings via MDM would allow a user to later install the missing apps and re-add them to the dock

That’s what I do.
Kyle
Sent from Mobile

On Oct 29, 2019, at 5:47 AM, jelockwood<EMAIL_ADDRESS>wrote:

For what it's worth, in Catalina it does not look like any of the com.apple.dock.plist files in /Library/User Template/ actually contain any dock entries.
I can only presume that the content is being generated dynamically when a new user account is created along with its home directory.
I would still like a solution to being able to customize the dock so it applies to new user accounts. Unless someone can identify from where the information is being generated then the only solution I can see would be as follows.
Install dockutil
Install outset
Configure outset to run a login-once script to clean up a users dock, as this would only be run once it would remove (in my case) the broken icons but unlike settings via MDM would allow a user to later install the missing apps and re-add them to the dock
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Wow.. old issue, but do the similar.. closing off thanks


Store ReadOnly commands and query these as bitmask

Redis Cluster and Master/Slave store ReadOnly commands as array and Intent resolution iterates through the array. Using a bitmask reduces computational intensity to arithmetic operations.
Current intent resolution for all known commands:
# Warmup Iteration   1: 2282,007 ns/op
# Warmup Iteration   2: 2441,771 ns/op
# Warmup Iteration   3: 2240,678 ns/op
# Warmup Iteration   4: 2345,982 ns/op
# Warmup Iteration   5: 2312,569 ns/op
Iteration   1: 2296,601 ns/op
Iteration   2: 2284,608 ns/op
Iteration   3: 2338,286 ns/op
Iteration   4: 2351,286 ns/op
Iteration   5: 2304,565 ns/op

Optimized approach
# Warmup Iteration   1: 194,052 ns/op
# Warmup Iteration   2: 177,044 ns/op
# Warmup Iteration   3: 174,675 ns/op
# Warmup Iteration   4: 174,695 ns/op
# Warmup Iteration   5: 173,879 ns/op
Iteration   1: 173,907 ns/op
Iteration   2: 172,596 ns/op
Iteration   3: 171,966 ns/op
Iteration   4: 175,460 ns/op
Iteration   5: 180,235 ns/op

Well, use EnumSet.

Enum sets are represented internally as bit vectors.