How to manually check if one position or row has correct masking?

Hi,
I wonder if we can manually verify attention mask patterns during testing. While I can visualize masks by printing them as strings, I'm looking to add proper test assertions.

How to assert if a specific position (i,j) has the correct True/False value
How to verify an entire row matches expected masking pattern

Current approach:

Can print mask as string for visual inspection

Would appreciate examples of how to properly test these mask values.

https://github.com/pytorch-labs/attention-gym/blob/36f8bd5ded5b3469f7892099590bb2405cc8f744/attn_gym/utils.py#L24
This function makes use of some private functions, and we dont provide BC guarantees on this but this should allow you to extract out the (potentially large) attn-scores matrix

Thanks @drisspg!
Will try!


[type:feat] add etcd exchanger client

Description

use config to create etcd client
etcd op : get / put / find / reomove

Related Issues
Closer : #17
New Behavior

create etcd client

put

get

find

remove

please sign cla

I have read the CLA Document and I hereby sign the CLA

I have read the CLA Document and I hereby sign the CLA

@zfkhhh pls check the ci failure

Codecov Report

Merging #33 (18a624e) into main (b935cc4) will decrease coverage by 6.74%.
The diff coverage is 48.83%.

@@            Coverage Diff             @@
##             main      #33      +/-   ##
==========================================
- Coverage   68.21%   61.46%   -6.75%
==========================================
  Files           6        7       +1
  Lines         129      218      +89
==========================================
+ Hits           88      134      +46
- Misses         36       68      +32
- Partials        5       16      +11

Impacted Files
Coverage Δ

internal/envcd/envcd.go
60.00% <0.00%> (ø)

internal/core/exchanger/etcd/etcd.go
51.68% <49.41%> (ø)

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 b935cc4...18a624e. Read the comment docs.


🐛 Bug: Mocha loads incomplete Promise polyfill

Prerequisites

[x] Checked that your issue hasn't already been filed by cross-referencing issues with the faq label
[x] Checked next-gen ES issues and syntax problems by using the same environment and/or transpiler configuration without Mocha to ensure it isn't just a feature that actually isn't supported in the environment in question or a bug in your code.
[x] 'Smoke tested' the code to be tested by running it outside the real test suite to get a better sense of whether the problem is in the code under test, your usage of Mocha, or Mocha itself
[x] Ensured that there is no discrepancy between the locally and globally installed versions of Mocha. You can find them with: node node_modules/.bin/mocha --version(Local) and mocha --version(Global). We recommend that you not install Mocha globally.

Description
I recently upgrade Mocha from v7 to latest and noticed that Mocha loads its own Promise polyfill. Unfortunately the polyfill is incomplete and does not implement Promise.finally and causes our tests to fail in IE11 (sorry, we can't drop support yet). We try to load es6-promise if we detect that Promise isn't already defined on window, but since Mocha loads first we detect that Promise exists and don't load our working polyfill.
I verified that the Promise polyfill is loaded starting in v8.1.0 and above.
With mocha on the page

Without mocha on the page

@straker is the first image from IE11? Showing "F12 Developer Tools" and "function Promise() { [native code] }?
Anyway I haven't reproduced your issue, because it's IE11 ...
In our Babel configuration we set useBuiltIns: 'usage', therefore only those polyfills are inserted which we are using in our Mocha code. And we haven't used Promise#finally() so far.
For testing I inserted a dummy finally() somewhere in runner.js and now the debug log of the build shows - besides es.promise - an additional es.promise.finally polyfill (both from core-js).
So I don't know how to proceed:

should Mocha add es.promise.finally, via a dummy finally() or evtl. via the include option?
could you check for window.Promise and window.Promise.finally and load your own polyfill?

see also #4446

Thanks for the information. Yes the first image is from IE11 when mocha is loaded on the page. I'm not sure why core-js adds Promise as [native code], but it seems to.
Would it be possible import the core-js Promise polyfill in the mocha code where it's used and then use the non-global import? That might get around the global pollute.
import Promise from 'core-js-pure/features/promise';

I don't know, but by fixing "global Promise" only, the root problem still persists. There are dozens of other pollyfills loaded by Mocha.
There is this babel-polyfills package, which seems to offer an option "method": "usage-pure" in order to prevent global pollution. I think it relies on core-js-pure.
I would need some help on this issue. Maybe you know somebody, who could struggle through this transpiling challenge. We also have some funds to spend.

There's no longer any code specific to Promises in rollup.config.js, nor core-js referenced anywhere in config files or project code. I think this was resolved by #4848. 🚀


Don't have default values when obtaining item from spinner

A spinner by definition has only one item selected, so if it is not one of those items an exception should be thrown. That's why this PR removes default values when getting the string from the spinner as this can easily be a source of bugs if a license is ever added to the spinner.

Looks good to me, but awaiting @maskaravivek 's input since he wrote that code.

I'm likely to update the preferences soon and call setDefaultValues in welcomeActivity - would it still be worth doing a null check?

If you call setDefaultValues in WelcomeActivity, why would obtaining the license ever be null?

I agree with removal of the default value, but not sure about the best way to handle the exception. Do we want to just throw the exception (which crashes the user), or can we handle it in a smoother manner?

We could log it as a "warn" and then recall setDefaultValues maybe?

It shouldn't ever crash for a user, it is only useful when debugging the app. It should only be one of the licenses for the user and if the user doesn't have a license, then we aren't in a state of the app which should be supported.
I'll probably convert the licenses to enums at some point so adding or removing a license should be as easy as removing an enum value.

@maskaravivek , what do you think?

Am scared about the idea of removing the null check after adding the default values in WelcomeActivity. That would add an assumption in our code base that SingleUploadFragment will always be initiated after WelcomeActivity. Lets keep the null check as it is even if we decide to set the default value else where.
+1 for adding licenses as enums.

I don't feel that we should protect ourselves if we have logical inconsistencies in our code. If we have logical inconsistencies, we should fix them instead of hiding behind a myriad of null checks. Even if we keep the null checks we have to define a state of the app if it is null, so we are effectively defining a state of the app when there is a logical inconsistency, which seems counterintuitive to me.

@whym or @nicolas-raoul , any thoughts on this?

Crashes can happen for tons of different reasons like OS bugs etc, so I am not against extra checks sometimes, but in this case if I understand correctly hiding the crash could very realistically prevent us from seeing a bug, so I guess I am with @veyndan.


Extend Lit version range to 3.0

Hi John, we're looking to move our projects to Lit 3.0 and would love to keep utilizing wc-responsive-grid.

Thanks so much John. Appreciate it!


Unhandled Exception when trying to start Hyper terminal on Windows 10

After running the Windows Installer Bootstrapper executable "hyper-Setup-1.3.1.exe", Hyper immediately trys to start it's main process. An unhandled exception is thrown immediately and the program exits.
Same happens when trying to start Hyper later on from the start menu.
Same happens when trying to uninstall the Package via Apps & Features
OS: Windows 10 Pro, 64 bit.
Version 1703 (OS Build 15063.138)

Try their repo instead: https://github.com/zeit/hyper
awesome-hyper is only about resources for Hyper, not Hyper itself.


Add github actions für build & deployment

I assumed that we are going to have a production branch called "scai".
Changed the node-version to 20.x, Angular's last version requires at least 18.13.0


Problems Getting TensorFlow to behave Deterministically

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug, a feature request, or a significant problem with documentation (for small docs fixes please send a PR instead).
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):

Yes, the code I am working with is company proprietary. If you need an example I will need to try to extract something that I can share.

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):

Mac OS 10.13.2

TensorFlow installed from (source or binary):

binary

TensorFlow version (use command below):

v1.3.0-rc2-20-g0787eee 1.3.0

Python version:

3.6.4

Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:

Running on CPU.

GPU model and memory:

Exact command to reproduce:

Please see description of problem. I don't currently have an example I can give you but I am hoping that the information I am providing is enough to get at least some idea about what is going on.
You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
I am having a major struggle trying to get TensorFlow to behave in a deterministic manner. I started out trying to compare two different versions of my source code, but the numeric differences from one execution to the next were large enough to make comparisons problematic.
So I went to trying to run exactly the same code in two different shells. And this is where things got weird.
I went through the code, and everywhere where TensorFlow is imported, everywhere I did operations using the form "with graph as default:", and prior to each call to run I added in a call to     tf.set_random_seed(42) [and yes, I used 42 in all of the calls].
I also placed breakpoints before anything in my code which was a tf call with 'random' in the name (I even searched the docs to see if there were random calls that didn't have random in the name, I found a couple but I am not using them). In all of the random calls that I am using I added in a seed=42 argument. So as far as I can tell I have covered all of the bases.
So what happens? I run the code in one shell, get a set of numbers. Run code in a second shell, get identical numbers. Good. Run the code a few more times, keep getting the same numbers. Better.
Run the code again, get completely different numbers!!! If I keep running the code in the different shells sometimes I get the same numbers from execution to execution. Sometimes I get different numbers. Sometimes it goes on to completely new numbers. Sometimes it goes back to an older set of numbers.
So I am getting something that is consistent enough that it feels like I haven't missed any places where I need to set the seed, but not consistent enough to say that it is deterministic.
I really want to be able to run the code over and over again and get exactly the same numeric result. Is this an unreasonable expectation?
Can anyone offer an explanation as to why the code would generate exactly the same result (and I'm looking down to 8 decimal places so this is fairly precise) five times in a row, and then switch to a completely different result and then repeat that result multiple times in a row?
I can certainly try to reduce this into an example I can share (the model I am using is a toy model we use for unit testing). I wanted to see if anyone had any insight into the general problem before I put the time into that.
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.
I will include two examples of the numeric output I am getting. This data is part of the returned result from Session.run().
Example A:
out_node_weights [[ 9.09113979  9.09934711  9.09535408  9.09768963  9.11262989  9.09756184
9.10322666  9.09527016  9.10811996  9.09029293]
[ 9.34178257  9.34915924  9.34216785  9.34740925  9.32617569  9.34794617
9.34459496  9.35203457  9.34202194  9.34494209]
[ 9.09113979  9.09934711  9.09535408  9.09768963  9.11262989  9.09756184
9.10322666  9.09527016  9.10811996  9.09029293]
[ 9.09113979  9.09934711  9.09535408  9.09768963  9.11262989  9.09756184
9.10322666  9.09527016  9.10811996  9.09029293]
[ 9.59115124  9.59841442  9.59205914  9.59892082  9.58935356  9.59801197
9.59655094  9.5970974   9.5989027   9.59136868]
[ 9.39457417  9.39968395  9.39309788  9.4028101   9.40468502  9.39697838
9.39671326  9.39805126  9.39696026  9.39565849]]
Example B:
out_node_weights [[ 9.04592514  9.05451775  9.05065632  9.05263042  9.06681919  9.05208111
9.05834484  9.05059719  9.06315136  9.04525948]
[ 9.25311852  9.26151276  9.25516319  9.25806046  9.23816681  9.26067734
9.25860214  9.26416397  9.25700569  9.25596619]
[ 9.27453995  9.28063679  9.27472687  9.28252983  9.28667641  9.27761269
9.27913475  9.27859116  9.28033924  9.27534485]
[ 9.27453995  9.28063679  9.27472687  9.28252983  9.28667641  9.27761269
9.27913475  9.27859116  9.28033924  9.27534485]
[ 9.50198936  9.51004601  9.50419044  9.50922203  9.50067139  9.50993729
9.50946426  9.5085659   9.51255226  9.50196838]
[ 9.27453995  9.28063679  9.27472687  9.28252983  9.28667641  9.27761269
9.27913475  9.27859116  9.28033924  9.27534485]]

Do you mind sharing the code you are running here, it is really difficult to help without that information. Thanks!

Thanks for taking a look. I can try to code up something that replicates the behaviour. Before I do that, can you please confirm whether or not Tensorflow is meant to be able to be used in a way where the numeric results are identical over an indefinite number of executions?
The best I have been able to get is within 4-5 decimal places.
I come from a computer graphics background and am used to being able to rerun very complicated renders or simulations and getting exactly the same result every time if I haven't changed any settings (that is, if I compare the resulting binary files they are identical). While the documentation seems to suggest that Tensorflow can provide this level of consistency, I haven't seen it.
If you expect that it is possible to run a graph today and get exactly the same result I got running the same graph last month, then I will try to put together an example which fails to achieve that goal. Then we can figure out if this is me failing to understand something, or a bug in Tensorflow.
Thanks!

@rundembear, I believe we do not guarantee this level of determinism with our current code. I would though suggest to use newer versions of tensorflow (I see you are using v1.3), which might give better results.
@ebrevdo do you have any comment on this, I see you are tracking a similar issue#14663.

Nagging Assignees @ebrevdo, @shivaniag: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 44 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Have you tried setting your inter_op_parallelism_threads to 1? Some core TensorFlow operations on multi-core CPU will be non-deterministic by default, due to parallel execution being enabled by default.
https://stackoverflow.com/questions/41233635/meaning-of-inter-op-parallelism-threads-and-intra-op-parallelism-threads

Nagging Assignee @ebrevdo: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 46 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 61 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

It would be great to be able to get a more deterministic behaviour, above all while using GPU training, as this would make hyper-parameters tuning much more useful and doable.
Nowadays it can really change quite much from one training to the next. In case that there is already a way to reduce that, I would appreciate it if you could provide some documentation.

Nagging Assignee @ebrevdo: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 30 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 45 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 60 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 74 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 89 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 105 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 120 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Have you tried setting your inter_op_parallelism_threads to 1? Some core TensorFlow operations on multi-core CPU will be non-deterministic by default, due to parallel execution being enabled by default.
https://stackoverflow.com/questions/41233635/meaning-of-inter-op-parallelism-threads-and-intra-op-parallelism-threads

@yoavz Why would inter-op make cause non-deterministic results instead of intra-op. For intra-op, I expect that you can get different order of arithmetic operations in every run because of how a tensor operation is broken up to be run on different threads resulting in different floating-point error. For inter-op, I don't understand how.

I understand that this issue, as it started out, is related to determinism when running on a CPU, but you might be interested in the work that I've been doing on determinism in TensorFlow focused on GPUs.
For more information, see: https://github.com/NVIDIA/tensorflow-determinism
@lucav76 The reproducibility on GPUs that you have been wanting should now be available.

@rundembear We are checking to see if you still need help on this issue, as you are using an older version of tensorflow(1.x) which is officially considered as end of life. We recommend that you upgrade to 2.6 which is latest stable version of TF and let us know if the issue still persists in newer versions. Thanks!


Feature/construct json

Coverage increased (+0.26%) to 98.7% when pulling 34825fed2e1b3a54c5e0a01b73d6b9a1a46c682c on feature/construct_json into 76d0fecd84ebe7f12b2e130054b734049d13e5d2 on master.


pipelineX Adjust local exchange plan rule

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Rename lib to phobos.

As discussed in a recent planning meeting, we're renaming lib to phobos, making it so that the root for Phobos v3 is phobos. So, that's what this commit does.
However, on top of that, I removed the existing package.d files (both lib/package.d and lib/sys/package.d) rather than fixing them, since I'm inclined to think that we should be very selective about when we use package.d, and I think that allowing something like import phobos; is a mistake.
It wouldn't surprise me if we get screaming about that later, and maybe we'll add it back at that point, but I'd rather not worry about maintaining it as modules get added, and if we can get away with it, I think that we should just not have it at all, since it's better practice in general to limit the symbols that you import to something close to what you're actually using, whereas import phobos; would be importing everything. And having a package.d at each level like lib/sys/package.d seems to be trying to do would complicate things even further.
We may very well end up with some package.d files in some cases, but I think that that should be handled on a case-by-case basis - and that the default should be to only have one module to import a symbol from.
So, with these changes, lib becomes phobos, and there are no longer any package.d files in it.

As this is just an administrative change and no code has actually changed I'm going to push this one through.


Docker compose error

I have docker compose installed yet the appimage says i do not. Is this error AppImage specific?

hey devs sorry I didn't read the previous issues, the app launched my browser to create an issue instead of opening the issues page. I see you have already been receiving this issue, again sorry.

I also just noticed that the issues others face is not with AppImage. which is what I am using. Any help is appreciated and thank you for taking time out of your day to read this, I understand we are all busy in life and I want you to know that I do  appreciate you. Thank you.


[UXD][AAP-22020] Minor updates to casing and text on EDA overview

https://issues.redhat.com/browse/AAP-22020
Minor updates to casing and text on the EDA overview page
[ x ]Update Projects card text link
[ x ]Update Rulebook Activations card text link
[ x ]Update Rulebook Activations card subtext
[ x ]Update Rule Audit card text link
[ x ]Updated Decision Environments card text link
[ x ]Update Decision Environments card subtext

@tiyiprh I will also need to change the overview tests on platform end, will open a separate PR for that and merge when this is merged

Closing in favor of https://github.com/ansible/ansible-ui/pull/2119


Mediaresource general attributes.

mediaResourceId
mediaResourceType
This will be covered by hasIdentifier and objectType?

Those as well:
resourceId
resourceIdType

Absolutely.


how to reduce cpu in arm?

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
D435i

Firmware Version
<IP_ADDRESS>

Operating System & Version
Linux (Ubuntu 14/16/17)

Kernel Version (Linux Only)
4.4.154

Platform
Arm v7 4 cores

SDK Version
2.39

Language
c++

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
I am using d435i testing in arm,when cross-compile realsense sdk2.39,i have turn off build_with_openmp,but using sdk to run program in arm, cpu is about 77 prectange in single core(capture 640*480 rgb and depth in 30fps).Do you have any suggestion to reduce cpu?
i have found some reasons, whether if convert yuy decoder cost more cpu resource.

Hi @Johnlan197  The SDK's CMake build configuration documentation confirms that having OpenMP enabled can result in greater CPU utilization when performing YUY to RGB conversion if OpenMP = true.
https://github.com/IntelRealSense/librealsense/wiki/Build-Configuration

In regard to ways to reduce CPU usage:
On computers equipped with an NVidia GPU, CUDA support can be enabled to optimize YUY to RGB conversion, and also reduce the CPU usage when performing depth to color alignment.  This is because CUDA offloads some of the CPU's work onto the GPU.
If you are building librealsense with CMake, you can speed up building by using a '-j' term in the make instruction to define how many cores the build process can make use of.  This may cause random crashes when used on low-end devices though.
The librealsense documentation for building from source with this multicore recommends '-j8', though with high-end processors with a large number of cores you may be able to use as high as '-j16'.  The j value represents the number of processor threads, so -j8 is representative of a quad-core processor: 4 (cores) x 2 (parallel threads per core) = total number of threads.

You may also be able to reduce CPU usage by making use of GLSL Processing Blocks when programming SDK scripts.  GLSL is an alternative to CUDA that also offloads CPU work to the GPU, and has the advantage that it is vendor-neutral (it should work with any GPU, though may not produce noticable improvement on low-end devices).
The link below has a very good pros and cons analysis of GLSL processing blocks and when it can be used.
https://github.com/IntelRealSense/librealsense/pull/3654
Adding GL support is straightforward, as it is usually a matter of putting GL:: int front of instructions that start with RS2::.  For example:
rs2::pointcloud,
becomes
rs2::gl::pointcloud

@MartyG-RealSense Thank you for reply, i have some questions

i have test realsense by AMCap tool, this tool can capture realsense data,the cpu cost is very low compared to use realsense sdk to capture, why using sdk cost more cpu?

The percentage of CPU usage in the SDK will be influenced by the operations being performed by a particular program.  For example, CPU usage may rise if depth to color alignment is being performed and the processing is not being offloaded from the CPU onto a graphics GPU.  So for some programs CPU usage wll be low and for some it will be high, depending on what the program is doing.  High CPU usage may also be caused by inefficiencies in a program.

Hi @Johnlan197   Do you require further assistance with this case, please?   Thanks!

Case closed due to no further comments received.


test modify teamcity regression pipline fe conf to 4G

Proposed changes
Issue Number: close None
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] No

Has unit tests been added:

[ ] No Need

Has document been added or modified:

[ ] No Need

Does it need to update dependencies:

[ ] No

Are there any changes that cannot be rolled back:

[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.07 seconds
load time: 442 seconds
storage size:<PHONE_NUMBER>7 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221125084745_clickbench_pr_52921.html


PHP 7 Session Pb (osc 2.4)

Warning: session_start(): Cannot send session cookie - headers already sent by (output started at /home/www/test/oscommerce2/catalog/includes/configure.php:31) in /home/www/test/oscommerce2/catalog/includes/functions/sessions.php on line 108
Warning: session_start(): Cannot send session cache limiter - headers already sent (output started at /home/www/test/oscommerce2/catalog/includes/configure.php:31) in /home/www/test/oscommerce2/catalog/includes/functions/sessions.php on line 108
see also : PHP7 REWRITE SESSION 2.3.x and 2.4 issue

There was probably extra whitespace in configure.php - this file no longer exists in the latest set of commits for v2.4.0b1.


[DynamicShape] Add dynamic shape support for GlobalLpPool

dynamic version of builder::opset1::get_monotonic_range
changed normalization builders to use Node

The PR replaces a static attribute with an input. That changes the API and would have backward compatibility issue. Any reason why you couldn't add additional API that supports dynamic shape instead of replacing the old one?

The PR replaces a static attribute with an input (Node) for builders. That changes the builder API and would have backward compatibility issue. Any reason why you couldn't add additional builder API that supports dynamic shape instead of replacing the old one? Or nobody is using the old builders?

@silee2
I am not aware of any user of this non-static mean builder API.
Due to the fact that it is NOT part of ngraph.hpp main header I've thought about it as internal helper.
However, it is not problem to keep both version (in separate files such as builder/mean and builder/dynamic_mean or in the same file???)
@diyessi What do you think about it?


Possible Grammatical typo in sentence

This process is called contextual typing because the context that the function occurred in informed what type it should have.

This sentence has a possible typo, where the part ... function occured in informed what type it should have ... is unclear. Maybe, ... function occured informs what type it should have is the intended sentence.

Thanks, yeah I agree it's a bit ambiguous, I've built on your recommendation and I think it makes more sense now


Issue with vBulletin integeration

Upgraded to the latest version of kleeja and did a vbulletin integration successfully but getting SQL error when login with arabic usernames only
Sorry , We encountered a MySQL error: Error In query
--[query]--------------------------
SELECT s FROM u WHERE username='علي علي' AND u != '8'
[1267 : Illegal mix of collations (latin1_swedish_ci,IMPLICIT) and (utf8_general_ci,COERCIBLE) for operation '=']

Also tried to change the Database char encoding in the VBulletin Intergration options to these values with no luck
latin1_swedish_ci
utf8

You should convert vb database to utf8. We dropped support of any other encoding.

You should convert vb database to utf8. We dropped support of any other encoding.

Thank you for replying, anyway UTF-8 is better.


squirrel: resolve .quit() issue with missing ../screen

As explained in: https://github.com/electron/electron/issues/8862#issuecomment-294303518
an issue with .quit() exists with a "Cannot find module '../screen'" issue,
while using the .exit() alternative avoids the issue

Validated on Windows with the same logic as #744 where the issue recently appeared

@AdrianDC sure! Thanks! Merging.


Chore: Remove SyncChannel

Description
Remove SyncChannel and SyncChannels methods to clean up the API a little.
We no longer use these functions and I don't think we'll be using them soon. If it turns out I'm wrong we can just revert this PR.

Checklist:
Code quality

[x] I have written clear commit messages
[x] I have performed a self-review of my own code
[x] This change does not have an unduly wide scope
[x] I have separated logic changes from refactor changes (formatting, renames, etc.)
[x] I have commented my code wherever necessary (can be 0)
[ ] I have added tests that prove my fix is effective or that my feature works, if necessary

Project management

[x] I have applied the appropriate labels
[ ] I have linked to all relevant issues (can be 0)
[x] I have linked to all dependent issues (can be 0)
[x] I have assigned myself to this PR
[x] I have chosen the appropriate pipeline on zenhub for the linked issue

Without looking at the code: wouldn't you still need to sync a running channel? syncObjective would not cover that case

Without looking at the code: wouldn't you still need to sync a running channel? syncObjective would not cover that case

That's a fair point! I assumed that everything would have an objective but that doesn't make sense when it's running.


pull request for e2e test at 11/11/2016 04:45:09

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


README update

HI @tjzo I think your merge request is out of date. The master branch of the repo has had this change in it for a while? Reopen this request if I'm wrong.


Default values for gauage component are always visible

Default values for amount and total (75 out of 100GB) are always visible. There was a requirement to hide those values because we dont have valid data to be displayed right now.

@vidyakhadsareibm, this issue deals with the carbon-addons-data-viz-react repo, so I'm going to move this issue over there.

Issue moved to carbon-design-system/carbon-addons-data-viz-react #38 via ZenHub


Better config for years

Context
I personally find for most of my use-cases that year configuration is not the most practical. In a majority of cases I would like to set asymmetrical min and max years either part of today's date (for example 1900 to 2023). I think this is currently not possible.
Suggestion
I would change the YearsConfig to something more flexible.
type YearsMode = 'decade' | 'fluid';

interface YearsConfig {
  mode: YearsMode,
  minYear : number,
  maxYear : number,
  step: number;
},

To prevent breaking changes, both configuration could be handled simultaneously with something like :
type YearsMode = 'decade' | 'fluid';

type NumberOfYearConfig =
    | { minYear: number; maxYear: number }
    | { numberOfYears: number };

type YearsConfig = NumberOfYearConfig & {
    mode: YearsMode;
    step: number;
};

This will require a change in create-years.ts and possibly elsewhere in the codebase

@AAS97, hi.
Thanks for reaching out with this question.
Min and max dates do exactly what you needed.
dates: {
  minDate: new Date(1900, 0, 1),
  maxDate: new Date(2023, 11, 31),
}

👆 with this configuration all years, months and days, before minDate and after the maxDate will be disabled.
LMK if this, doesn't help to solve your problem

@Feshchenko, thanks for the prompt answer. The solution works but, years is still populated with much more years than I need (and I need to filter them down on disabled), and I need to fill numberOfYears with a value twice as big as the span I want to display.

Do you need a way to generate the exact amount of years with the end of the range bp current year or maxDate year?
dates: {
  maxDate: new Date(2022, 0, 1),
},
yearns: {
  numberOfYears: 100,
}

Should produce an array of 100 years from 1923 - 2020.

dates: {
  maxDate: new Date(2020, 0, 1),
},
yearns: {
  numberOfYears: 100,
}

This does not work as expected. I get 100 years from 2019 to 2118.

Yea, I know that it not working as you describe.
I'm asking questions to clarify your need and that I get it right :)

Looks promising ! Well done in such a short time. When do you expect to release ?

@AAS97 please try 'exact' mode in v4.2.0
I hope it will help you


[Go Live][March 29]

Push to live

Docs Build status updates of commit 9f8930d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/icons/certificate-icon-16.png
:white_check_mark:Succeeded
View

msteams-platform/assets/icons/certificate-icon.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/dev-preview/devtools-right-click.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/dev-preview/mac-os-developer-tools.PNG
:white_check_mark:Succeeded
View

msteams-platform/concepts/build-and-test/teams-developer-portal.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/deploy-and-publish/appsource/prepare/teams-store-validation-guidelines.md
:white_check_mark:Succeeded
View

msteams-platform/tabs/how-to/developer-tools.md
:white_check_mark:Succeeded
View

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


This module is declared with using 'export =', and can only be used with a default import when using the 'allowSyntheticDefaultImports' flag.

Description
When I use import ReactSlider from 'react-slider' I get the following error (red-squiggly lines) in VSCode:

This module is declared with using 'export =', and can only be used with a default import when using the 'allowSyntheticDefaultImports' flag.

The way to overcome allowSyntheticDefaultImports is to use it as import * as ReactSlider from 'react-slider' so I tried that & my console was filled with errors & the app crashed:
popup.js:69479 Warning: React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: object.

Check the render method of `wrappedComponent`.
    at createFiberFromTypeAndProps (popup.js:27181)
    at createFiberFromElement (popup.js:27209)
    at createChild (popup.js:15574)
    at reconcileChildrenArray (popup.js:15847)
    at reconcileChildFibers (popup.js:16253)
    at reconcileChildren (popup.js:19118)
    at updateHostComponent (popup.js:19760)
    at beginWork (popup.js:21208)
    at HTMLUnknownElement.callCallback (popup.js:6073)
    at Object.invokeGuardedCallbackDev (popup.js:6122)

The app doesn't crash when I use import ReactSlider from 'react-slider' but I get red-squiggly lines which I always get rid of since I use TS.
Would love to find a solution to this?

Meanwhile I'm setting allowSyntheticDefaultImports to true in tsconfig.json :)

Meanwhile I'm setting allowSyntheticDefaultImports to true in tsconfig.json :)

Sorry, I don't use VSCode or TypeScript, so I'm not sure what issue you are having. Somebody contributed type support a while back -- is it as simple as installing that?
npm install --save-dev @types/react-slider

Sorry, I don't use VSCode or TypeScript, so I'm not sure what issue you are having. Somebody contributed type support a while back -- is it as simple as installing that?
npm install --save-dev @types/react-slider

I installed that already but TypeScript yelled at me. Doing this fixed the issue but maybe the types can be improved so we don't have to set that flag in tsconfig.json.
Idk TypeScript very well either. Just started using it so don't know as well what's wrong 🤔
The current fix works for me but maybe someone else who knows TS well can chime in :)

I installed that already but TypeScript yelled at me. Doing this fixed the issue but maybe the types can be improved so we don't have to set that flag in tsconfig.json.
Idk TypeScript very well either. Just started using it so don't know as well what's wrong 🤔
The current fix works for me but maybe someone else who knows TS well can chime in :)

The error

This module is declared with using 'export =', and can only be used with a default import when using the 'allowSyntheticDefaultImports' flag.

could be fixed by putting export default instead of export = in @types/react-slider.
But I don't know if this raises compatibility issues. Any input @axelboc ?

The error

This module is declared with using 'export =', and can only be used with a default import when using the 'allowSyntheticDefaultImports' flag.

could be fixed by putting export default instead of export = in @types/react-slider.
But I don't know if this raises compatibility issues. Any input @axelboc ?

You're right @loichuder, it should be export default, since that's how ReactSlider is exported in https://github.com/zillow/react-slider/blob/master/src/components/ReactSlider/ReactSlider.jsx.
Until this is fixed, it seems the best solution is actually to import the package with this TypeScript syntax: import ReactSlider = require('react-slider');, as explained here: https://www.typescriptlang.org/docs/handbook/modules.html#export--and-import--require

You're right @loichuder, it should be export default, since that's how ReactSlider is exported in https://github.com/zillow/react-slider/blob/master/src/components/ReactSlider/ReactSlider.jsx.
Until this is fixed, it seems the best solution is actually to import the package with this TypeScript syntax: import ReactSlider = require('react-slider');, as explained here: https://www.typescriptlang.org/docs/handbook/modules.html#export--and-import--require

Or use allowSyntheticDefaultImports in tsconfig.json which is much cleaner imo :)

Or use allowSyntheticDefaultImports in tsconfig.json which is much cleaner imo :)

Thanks all for investigating! Going to close this since the problem is external.

Thanks all for investigating! Going to close this since the problem is external.

This should now be fixed by https://github.com/DefinitelyTyped/DefinitelyTyped/pull/50819


Error connecting to kind registry

Can't seem to get ctlptl to connect kind to the local registry. Have tried with the registry image already running, registry stopped, and registry rm'd.
$ ctlptl create cluster kind --registry=ctlptl-registry
Creating registry "ctlptl-registry"...
Creating cluster "kind" ...
 ✓ Ensuring node image (kindest/node:v1.17.0) 🖼
 ✓ Preparing nodes 📦
 ✓ Writing configuration 📜
 ✓ Starting control-plane 🕹
 ✓ Installing CNI 🔌
 ✓ Installing StorageClass 💾
Set kubectl context to "kind-kind"
You can now use your cluster with:

kubectl cluster-info --context kind-kind

Not sure what to do next? 😅 Check out https://kind.sigs.k8s.io/docs/user/quick-start/
   Connecting kind to registry ctlptl-registry
connecting registry: exit status 1

(Additionally, a more obvious/detailed error message would be helpful!)

does upgrading kind to v0.10 help?


Add cyclone client timeout and make func execution timeouts configurable

Description
This PR adds a timeout wrapper when executing functions via the cyclone client in veritech. In addition, the default timeout value can be overriden as well as the recently added timeout value for cyclone server's lang server process.
As a result of this commit and recent commits, the following values are now configurable:

cyclone client timeout

dynamically configurable via veritech args and/or config file

cyclone server timeout

statically configurable with cyclone args and/or config file

lang-js function timeout

statically configurable with cyclone args and/or config file
dynamically configurable with lang-js args, but they are also statically configured due to how cyclone is invoked via firecracker (see the rootfs builder script for more information)

This PR also renames some functions for clarity, fixes some doc comments and does some cleanup.

All tests pass locally before we posted this PR... going to see if we missed something due to the merge queue failure.

Fixed. We needed to change the dal test config since we slightly changed how the defaults worked.


How to get gaze direction from heatmap, especially for out of image traget?

the model only output heatmap and in/out confidence, is there a way to convert this to the gaze direction, for example a vector or yaw,pitch angle (especially if the gaze target not in the image/looking out).
Also any plan to release training code?

Our model is not designed for the task of 3D gaze direction prediction (pitch & yaw), which is a separate task. There are several existing training datasets / benchmarks for this task such as Gaze360, ETH-XGaze, Eyediap, MPII-Gaze, etc. so for your use case, I would recommend looking into models trained on those datasets. Here is one such codebase: https://github.com/zgchen33/MCGaze
We will consider releasing training code.


Is there a way to load existing files?

Hi
I am using blueimp jquery upload in oneupuploaderbundle. I was wondering if there is a way to load existing files like the basic plus ui in http://blueimp.github.io/jQuery-File-Upload/?
Say I have an interface to upload the files. After I upload the files and gets into the interface again all the existing files will be listed.
Or is this something we need to write by our own?

Is there an exemple of how to create own logic to list existing files ? Even for Fine Uploader ?

You need to create a 'session' endpoint which returns all current files.
More info: http://docs.fineuploader.com/branch/develop/features/session.html


First Companion Table Ingest

First ingest into Companions table

[x] ingest reference not in Simple
[x] ingest source not in Simple
[x] import google sheets
[ ] write ingest function

Looking good!!


test-branch/execute-job/fork-branch/19a92a94-2db4-4586-8f86-3f76c3596225

PR for testing 555d3fd7ab9d0b977a46606f6cd23e57c1394652

padding /canonical/self-hosted-runners/run-workflows 555d3fd7ab9d0b977a46606f6cd23e57c1394652 padding


feat: add server-side search, filter and sorter for all pages

Signed-off-by: Yixiang Zhao<EMAIL_ADDRESS>#370
The demo link is http://<IP_ADDRESS>:8000/.
I referenced components-table-demo-ajax and components-table-demo-custom-filter-panel. I I extracted the common components and created BaseListPage.js which each of list pages needs to inherit. The code looks like a lot, but many of the operations are similar. Please test in the demo link.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@Steve0x2a plz review


NIH

Have you considered https://github.com/hapijs/joi as a solution? May it will be better not to invent something completely new... :)

First of all, because I can :)
Actually, I needed a lightweight, declarative and dependency-free library for fast check objects.
Maybe it's NIH, but Joi is a huge combine, while PropChecker is a simple validator.

OK, got it.
Anyway, good first issue for a JS library ;)


Write "getting started" guide

See original issue on GitLab
In GitLab by [Gitlab user @jjardon] on Mar 8, 2018, 14:39
This should go in https://buildstream.gitlab.io/buildstream/, probably as part of the installation guide
This should cover:

what it's for in a couple of sentencess.
how to install it (via a link if it's complex)
how to do 'hello-world' within 10 minutes, with words people with no bst experience can understand
where to dig deeper

As a reference, GNOME people wrote this to replace its jhbuild guide when they migrated to bst: https://wiki.gnome.org/Newcomers/BuildSystemComponent

In GitLab by [Gitlab user @jjardon] on Mar 8, 2018, 14:40
changed the description

In GitLab by [Gitlab user @devcurmudgeon] on Mar 8, 2018, 14:43
Crucial thing IMO is that a new user should be able to get a hello-world style minimal success in less than 10 minutes (and no need for them to understand gnome etc)

In GitLab by [Gitlab user @samthursfield] on Mar 8, 2018, 16:10
I raised this on the mailing list recently: https://mail.gnome.org/archives/buildstream-list/2018-February/msg00014.html

In GitLab by [Gitlab user @tristanvb] on Mar 8, 2018, 16:36
If this is a duplicate of #103, please close this and keep the discussion in one issue.

In GitLab by [Gitlab user @devcurmudgeon] on Mar 8, 2018, 16:59
[Gitlab user @samthursfield] your doc is pretty good from my pov.
I've done an attempt at a Hello World, am happy to write some text around it which I think would go after your 'what is buildstream' and before 'concepts'.

In GitLab by [Gitlab user @jjardon] on Mar 15, 2018, 21:40
Close in favour of #103

In GitLab by [Gitlab user @jjardon] on Mar 15, 2018, 21:40
closed


Colors on opposite sides don't match on REPLACE and VISUAL modes

I'm not sure if this is a bug or something on my configuration, but the colors on the opposite sides of my airline don't match when I'm on REPLACE and VISUAL modes.
I'm using molokai vim colorscheme with powerlineish airline theme.

I guess it was supposed to look like this:

Sorry for necroposting, but would anyone care to explain how to go about tweaking this theme to better resemble vanilla Powerline (changing the status line from dark green on light green to black on light gray, tweaking the font colors, etc.)? Hopefully this should be possible if powerlineish indeed does not mirror.

@ebkalderon this is the plugin i use when i build themes: https://github.com/guns/xterm-color-table.vim, give that a try!


Allow options to be a builder function

By allowing the plugin options to be a function that has access to the output options, we can generate unique configurations on a per-output basis.
This is particularly important when integrating with Vite's plugin-legacy to enable creating distinct stats files for the legacy and modern outputs.
But why allow access to outputOptions?
I'm using Vite and @vite/plugin-legacy to create modern and legacy output in a single build. I'd like to use rollup-plugin-webpack-stats stats for both the modern and legacy bundles.
How @vite/plugin-legacy works is by essentially adding a new rollup output that uses format: 'system' to your existing build, copying the plugins and other options of any existing outputs.
By making the options be a function that can use the current output's options, I can add webpackStats to my Vite config like so, using the contents of the output options to infer if I'm producing legacy or modern outputs:
// vite.config.js - using plugin-legacy, and generating a stats file
// for the the modern and legacy outputs
import { defineConfig } from 'vite';
import legacy from '@vitejs/plugin-legacy';
import { webpackStats } from 'rollup-plugin-webpack-stats';

export default defineConfig((env) => ({
  build: {
    rollupOptions: {
      output: {
        plugins: [
          // Output webpack-stats-modern.json file for the modern build
          // Output webpack-stats-legacy.json file for the legacy build
          // Stats are an output plugin, as plugin-legacy works by injecting
          // an addtional output, which duplicates the plugins configured here
           statsPlugin((outputOptions) => {
            const isLegacy = outputOptions.format === 'system';
            return {
              fileName: `webpack-stats${isLegacy ? '-legacy' : '-modern'}.json`,
            };
          }),
        ],
      },
    },
  },
  plugins: [
    legacy({
      /* Your legacy config here */
    }),
  ],
}));

This ability to expose the outputOptions is inspired by rollup-plugin-visualizer

Thanks for the review, I've updated the README

Published on v0.3.0.
Thanks again for the PR!
Also, would really appreciate it if you could share how you are using the package. The package was created primarily to allow users to use bundle-stats and @relative-ci/agent with vite/rollup projects, if we manage to gather more info about the use cases, will help us when prioritizing and adding new features.

Also, would really appreciate it if you could share how you are using the package. The package was created primarily to allow users to use bundle-stats and @relative-ci/agent with vite/rollup projects, if we manage to gather more info about the use cases, will help us when prioritizing and adding new features.

Using this package with Vite and using bundle-stats for analysis is exactly my team's use-case.
We're in the process of migrating our webpack application build process to use Vite. We currently perform webpack bundle analysis on a somewhat ad-hoc basis using statoscope, but we're hoping to get better at automating this analysis and surfacing impact on PRs. I've been generally somewhat surprised at the lack of comparable bundle analysis tools for Rollup/Vite, so I was relieved to find rollup-plugin-webpack-stats and bundle-stats.
rollup-plugin-webpack-stats feels like a great solution to this problem space - porting data into a webpack-style stats file means being easily able to leverage existing analysis tools such as bundle-stats and statoscope - though I appreciate that focusing on making bundle-stats better is going to a much higher priority for you than stats file completeness to enable usage for other tools analysis tools.
A few questions that we tend to want to answer when doing build analysis:
Single builds:

Where in our bundle is the majority of our file size?
Are there any particular egregiously sized assets?
Why is a file in a particular chunk? (what was a file's import chain)

Build comparisons:

What files size impact did this change have, and in what chunk(s)? Ideally we'd be able to state something saying "Loading this URL results in loading the entry asset and these async assets" so we can get an idea of "amount of js/css required to load some particular URL - which is a sum of many assets.
Did this PR cause files to move between chunks and if so, why?
Did this PR introduce new package dependencies into the build?

Thank you for the detailed response, very insightful!

though I appreciate that focusing on making bundle-stats better is going to a much higher priority for you than stats file completeness to enable usage for other tools analysis tools.

We didn't aim for complete parity, just to extract the data needed by bundle-stats. I think this is the case for most of the tools, they rely on a small percentage of the stats data.
If it helps extract more data for other cases or tools, we are happy to accept PRs or implement the changes when we can ;)


Fix crash when calling screenpos() with a negative column

can the same happen for a negative row?

That's already covered by the pos->lnum >= wp->w_topline check.

thanks!


fix: Fix validation of raw values onBlur.

When validate() is provided as an option to a raw() parameter, it is always called with undefined as the value when a blur event is fired.  This fixes it so it uses the current form state value instead.

Pull Request Test Coverage Report for Build 326

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.322%

Totals

Change from base Build 323:
0.0%

Covered Lines:
182

Relevant Lines:
183

💛  - Coveralls

Oh, I see someone else has run across this, too.   :) https://github.com/wsmd/react-use-form-state/pull/73

Thanks for the PR, @jwalton. Could you please update this branch with master?

Rebased.


please consider current source DAC LTC2662 (ADI)

please consider current source DAC LTC2662 (ADI) with fast sweep.
https://www.analog.com/en/products/ltc2662.html
Thank you!

The manufacturer seems to be literally "ahead of what is possible". Although it is officially in production, I could not find LTC2662 on Farnell, Mouser, TME nor whole Ebay. My policy is to keep all extensions tested experimentally, so I cannot help you as of yet.
Thanks for the tip, anyway. Its parameters are nice and price is also fine.

Dear @caizikun, if I implement rudimentary SPI messaging, would you still be interested in testing it on a LTC2662?
Very cheap chips I buy without hesitation just for testing; the latter is unfortunately not one of them. Currently I am not going to buy it, so I would have to rely on collaboration with you.

@FilipDominec  Sorry, I just leave my last job and have no suitable debugging environment. But as i know, multi-channels current source would be more popular for DAQ card application. Maybe you can try to prepare the register setting API. If I have debugging environment, i will tell you.

OK, thanks.
Meanwhile I decided to demonstrate DAC capabilities with TDA1543. Though far from being high-end, it receives data over I2S present in Pi Pico; it has 2×16 bit outputs, comes in convenient DIP package and costs near to nothing. Of course other I2S devices should be compatible as well.
Further DAC or other peripheral tips are welcome.


Build(deps-dev): Bump eslint-plugin-react from 7.30.0 to 7.30.1 (#3676)

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

Merged in 71a25a806f0ee4dd1bfb5387c676d9732935de57


c_style_strlen instead of str_length

As str_length is gone after beta 045, we now use c_style_strlen for comparison of the incoming strings from postgres. This also works on beta 044 at least.

Thanks!

Thanks!


Fixing the error message when guestfs digest and tag are missing and …

Align error message with the guestfs logic when neither Digest nor Tag could be determined
Signed-off-by: Arnaud Aubert<EMAIL_ADDRESS>
What this PR does / why we need it:
Clarifies error message for guestfs command
Special notes for your reviewer:
I simply encountered this misleading error message and wanted to clarify it
Release note:

NONE

/lgtm

/retest

/ok-to-test

/retest

/remove-lifecycle rotten

/retest


Project encoding performance improvements

Improve project encoding performance 📝
For my test project: xcodeproject.write(path: path) was 57 seconds, and it became 8 seconds.
Improvements
Obtain PBXBuildFile from PBXBuildPhase instead searching PBXBuildPhase in PBXObjects
The main blocker was obtaining build phase name for PBXBuildFile encoding from PBXObjects -plistKeyAndValue. For each file, we ran through all the build phases and searched for a file with the same identifier. Of course, it took a lot of time.
Instead, I added an utility class that contains the file and build phase from which this file is. I am building this class from the build phase files.
This speed up for encoding on 46 seconds. ( 11 seconds instead of 57 ).
Replaced the append in the string to append into the array + join.
Append into a typed array faster than into a string. This gave about 3 seconds. ( 8 seconds instead of 11 ).
Also this preparation for the subsequent changes. For an array, we can pre-set the size. This will allow non-allocation of memory several times. And we can prepare the data for recording in parallel.
Now this code slows down the most:
        elements.forEach { (element, multiline) in
            write(dictionaryKey: element.key, dictionaryValue: element.value, multiline: multiline)
        }

We can pass an inout array for each thread. I'm working on it now.

@pepibumur I fixed all issues. And added one more optimization - concurrent decoding. Performance boost from 8 second to ~3.


Use OAuth state to forward the environment to the handler

Feature Suggestion
The current EnvironmentHandler implementation uses the env parameter in the callback URL. This can cause a bit of confusion as it needs to be part of the configured callback URL of the OAuth clients. We should be able to simplify the configuration by forwarding the environment via state instead.
Possible Implementation
Use a querystring encoded state, something like nonce=835v7t83&env=development. Since the state param is specific to OAuth we also need to figure out how to do the refactor while still supporting SAML and other non-oauth flows. I think it would be reasonable to simply remove support for separate environment configs from SAML, at least until the implementation of that is more mature and we know how to do it and if it's needed.
Context
https://discordapp.com/channels/687207715902193673/687235481154617364/738061190214713444

@Rugvip I would like to help with this and take a shot at this. Can you assign it to me ?

@GoWind Awesome, thanks! 😁

@GoWind @Rugvip is this done?

Yep! Fixed by #1812


[mini.completion] Missing function / variable definition (result.detail) from popup docs for languages like TypeScript

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md
[X] I have updated 'mini.nvim' to latest version

Module(s)
mini.completion
Description
For languages like TypeScript I was struggling with mini.completions because the popup docs for each completion item was not showing the function / variable definition (i.e., function signature / variable type). As you can see here, for the JSON.stringify() builtin function, it only includes the documentation "annotations" explaining the function but not the function definition itself.

But as you can see in the LSP response, the detail property holds this information. So we need to concatenate result.detail with documentation.value.

Compared to a language like Lua, the function definition is included in the documentation.value prop in the markdown.

Now the one caveat is that for a language like Lua, it has function in the result.detail prop so if you concatenated it for Lua, it would be a little redundant since the full function definition is also included at the start of the documentation.value prop.
Neovim version
Neovim nightly
Steps to reproduce
Explained in detail in description...
Expected behavior
Explained in detail in description...
Actual behavior
Explained in detail in description...

As an additional item which is related, but more of a "nice to have" than above which is pretty important to the core functionality of completions, there is also the labelDetails.description value which tells you the library an import statement will be pulling from in JS / TS.

In nvim-cmp they add this so that you can see what library will be pulled in when an auto import statement is generated from the completion.

This is super nice, especially for ambiguous names that may have options from multiple libraries... BUT this I could do without, as opposed to the original issue above which I consider vital as props on an object will not even show you which type they are (no docs popup comes up at all for variable props).

Thanks for the suggestion!
I've only now found out about the detail field in CompletionItem. Judging by the description, it is something that is meant to be put beside completion item in the initial popup menu. However, I mostly prefer it it to not be wide.
Same goes for labelDetails.
I'll take some time to think about this, but currently I am skeptical that this will be part of 'mini.completion'.

I'll take some time to think about this, but currently I am skeptical that this will be part of 'mini.completion'.

Thanks for taking a look so quickly. This is not meant to be "aggressive" but just to clarify understanding of the impact, without having the detail I really don't think mini.completion is usable for me as a typescript dev. The function definitions is one thing, but the main deal breaker is that for an object properties, it shows no docs pop up at all because it is only in the detail field.
For example, if I have this object:
FooObj {
foo: string;
bar: number;
baz: Date;
}
And if I have to supply it as a prop to a function like: fooFunc(fooObj)
When I do:
fooFunc({
baz| (autocomplete shows no "doc" popup so I do not know the type of the prop is a Date)
})
Hopefully this makes sense. I would agree this likely is something the typescript LSP could populate things differently, but from experience I know getting the LSP to change is not an option 😉).

@echasnovski curious your thoughts on a hook that provides the docs, details and maybe the labelDetails as args for the hook and then the user can choose how to populate the docs popup?
If the docs popup is correct then I am not too worried about the initial completion item list. Although in a perfect world a hook for the completion menu items would be amazing. Thoughts?

I suppose would also need to provide the lsp name as an arg so you could do conditional logic based on the lsp/language.

@echasnovski curious your thoughts on a hook that provides the docs.value, details and maybe the labelDetails as args for the hook and then the user can choose how to populate the docs popup?
If the docs popup is correct then I am not too worried about the initial completion item list. Although in a perfect world a hook for the completion menu items would be amazing. Thoughts?

There is config.lsp_completion.process_items. Users can manipulate raw items to populate their documentation.value and label the way they like.
Also, after looking at code, the detail value of completion item should be shown beside it but not in the info window (that is what menu field is for in completion items).

@echasnovski I'll do a little PR as proof of concept and make sure it could work out simple enough. No pressure to integrate it as I'm fine with just using my fork for the time being.

@echasnovski I'll do a little PR as proof of concept and make sure it could work out simple enough. No pressure to integrate it as I'm fine with just using my fork for the time being.

@echasnovski I'll do a little PR as proof of concept and make sure it could work out simple enough. No pressure to integrate it as I'm fine with just using my fork for the time being.

Would you mind only making a change in your fork and post the link to the branch/commit? I'd like to have PR count as low as possible and I have doubts there will be changes in 'mini.completion' regarding this issue.

@echasnovski I'll do a little PR as proof of concept and make sure it could work out simple enough. No pressure to integrate it as I'm fine with just using my fork for the time being.

Would you mind only making a change in your fork and post the link to the branch/commit? I'd like to have PR count as low as possible and I have doubts there will be changes in 'mini.completion' regarding this issue.

No problem. And see my latest message above. I did not see your other message before I sent that. I will dig into the processitems config and see if it gives what is needed but last time I checked it did not. Part of it may be that the docs come through after the initial request and only when you ctrl-n/p select an item and it gets the secondary response / request from the LSP in your code. Not at a computer right now so explaining off memory. Will take a look when back at computer tonight likely after I put the kids to bed.

Part of it may be that the docs come through after the initial request and only when you ctrl-n/p select an item and it gets the secondary response / request from the LSP

@echasnovski so I think this may be the issue. See the lines from your code below. Correct me if I'm wrong, but this "secondary" request when there is no documentation returned sends completionItem/resolve after "cycling" through over items in the completion menu (<C-n/p>) and then that response updates the documentation popup window. I am not seeing it come through in items with the process_items hook.
https://github.com/echasnovski/mini.nvim/blob/61e5d46fc0cf71306c51275383767d996f559a60/lua/mini/completion.lua#L1034-L1052
Does what I am saying make sense? I am still somewhat new to diving into the LSP completions stuff so if I am not describing it very well let me know and I can try to clarify.

Also, after looking at code, the detail value of completion item should be shown beside it but not in the info window (that is what menu field is for in completion items).

Are you sure you aren't mixing up labelDetails with detail? Not saying just because nvim-cmp does something, it is correct, but they put to the side (in menu) the labelDetails.

Here is where nvim-cmp sets menu to labelDetails (not detail).
https://github.com/hrsh7th/nvim-cmp/blob/ae644feb7b67bf1ce4260c231d1d4300b19c6f30/lua/cmp/entry.lua#L262-L272

And here is where nvim-cmp handles the detail that I was referring to in my origin post above where the full function definition is and should be pre-pended to the documentation popup. It looks like that is exactly what nvim-cmp does here too:
https://github.com/hrsh7th/nvim-cmp/blob/ae644feb7b67bf1ce4260c231d1d4300b19c6f30/lua/cmp/entry.lua#L448-L459

Yes, using labelDetails in the menu seems to be more idiomatic than details. Maybe preferring it with fallback to currently used detail is a good decision.
I think the reason labelDetails is not  used in 'mini.completion' is that 3.17 version was still in development when 'mini.completion' was created.

I think the reason labelDetails is not  used in 'mini.completion' is that 3.17 version was still in development when 'mini.completion' was created.

Yeah that makes sense! labelDetails I recall only showed up in nvim-cmp more recently (in the last few months) so likely similar reason.
The problem with detail though is I don't believe it actually belongs in menu. As you said, it would make the menu way too wide. But also judging by the descriptions in LSP, it starts with "Human readable..." exactly like the docs property does. Point being, I still believe detail belongs in the docs popup window prepended ahead of docs.value.
Additionally the reason I think this is true is because detail doesn't even show up in initial response from LSP. it seems to typically only be returned by LSP after you <c-n/p> cycle over a choice in the completions menu, so it can't be part of the menu itself because it won't show up until you select an item, hence why I believe it goes in the docs popup.

As well, label and labelDetails are next to each other in the lsp docs.

And then detail appears next to documentation and as mentioned above, start with a similar description "human readable..."

TLDR: label / labelDetails should be together in the completions menu. And documentation / detail should go together in the docs popup.

Additionally the reason I think this is true is because detail doesn't even show up in initial response from LSP. it seems to typically only be returned by LSP after you <c-n/p> cycle over a choice in the completions menu, so it can't be part of the menu itself because it won't show up until you select an item, hence why I believe it goes in the docs popup.

This depends on the LSP server. clangd, for example, includes it in the initial response and contains things like char *, enum <anonymous>, etc.
So, I am afraid, it is not as black and white. I'll take a look later.

So, I am afraid, it is not as black and white.

Should be the tag line for LSP generally 🤣

@echasnovski to one of my earlier points, a lot of this could be solved by the user if the process_items hook also processed the additional LSP response when items are selected. Or a second hook. Because none of that passes through it so users have no way to customize that part (which for typescript is where all the detail and docs comes through).
See here: https://github.com/echasnovski/mini.nvim/issues/1258#issuecomment-2395215176

@echasnovski to one of my earlier points, a lot of this could be solved by the user if the process_items hook also processed the additional LSP response when items are selected. Or a second hook. Because none of that passes through it so users have no way to customize that part (which for typescript is where all the detail and docs comes through).
See here: #1258 (comment)

Looks like what I am referring to (for clarity) is exactly what you called out in your docs / notes here:
https://github.com/echasnovski/mini.nvim/blob/61e5d46fc0cf71306c51275383767d996f559a60/lua/mini/completion.lua#L99-L106
And also similarly here:
https://github.com/echasnovski/mini.nvim/blob/61e5d46fc0cf71306c51275383767d996f559a60/lua/mini/completion.lua#L930-L934

While I am digging into it, documenting for future references.
LUA
textDocument/completion

completionItem/resolve

TypeScript (vtsls / tsserver)
textDocument/completion

completionItem/resolve

@echasnovski here is my hacky working fork for completions for everything discussed above: https://github.com/GitMurf/mini.nvim/commit/cc8101f390acb7dd0d68c17a973b3c54ca0f9418
One thing that is not ideal is for languages like lua, where the LSP provides details as well as includes function signature / variable type info in the documentation.value, there is a bit of redundancy in the docs popup (prepends "function" to top of docs popup). Could conditionally program around that but I didn't care enough to go any more complex than this for POC.


pull request for e2e test at 12/02/2016 08:15:11

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Logout and redirect process does not work correctly when token expires

Logout and redirect process does not work correctly when token expires.

resolved


RC / RT should not be readable for the victim

If you send out a link for phishing this link should consists of a domain and an ID at max:
http://my-site.com/ServiceLogin?id=1234567890
The RC and RT should be mapped in the backend to an uniq ID and not presented to the victim as this is minimizing the success rate drastically if you see an url like this:
http://my-site.com/ServiceLogin?rc=http://my-real-site.com/login&rt=Session

I agree.
It is an issue to implement though as parameters are handled via LuaJIT in nginx's  site config file. There is no way to access any database backend from there that I know of.
It would be enough to just base64 encode the parameters. If you can find out how to do it with LuaJIT, I would greatly appreciate it.

Fixed in version 1.0.0. Added base64 encoding for redirection parameters.


resource change to me/events but failed

hi, I can build the project based on gradle, but when I try to change resource to me/events, I got Unauthorized; Reason: The tenant for tenant guid does not exist.

Hey @kangqiwang
As outlined here the delegated context is not yet supported by this sample.
This means any subscription with a resource path starting with 'me' won't work because there's no notion of a current user in an application context. You need to specify users/id/events instead.
https://github.com/microsoftgraph/java-spring-webhooks-sample/issues/3

thank you, I try it /users/user_id/events instead, but it still won't work and show the same error  The tenant for tenant guid does not exist. however, I am pretty sure the tenant id is correct

Can you copy/paste the configuration you have in NotificationController.java here please? (without the secrets)

private String clientId  = "ca809fbf-a6cd-4fa3-8bed-471475e02243";
private String clientSecret = "CLIENT_SECRET";
private String tenantId = "71e8bd49-7319-479c-85d1-ea483993eef0";
private String publicUrl = "https://88ceb5cc47b0.ngrok.io";
private String resource = "users/209af94f359a5a39/events";
private String changeType = "created";

private List scopes = Arrays.asList("https://graph.microsoft.com/.default");

ClientCredentialProvider authProvider = new ClientCredentialProvider(
            this.clientId,
            this.scopes,
            this.clientSecret,
            this.tenantId,
            this.endpoint);

        IGraphServiceClient graphClient = GraphServiceClient
                            .builder()
                            .authenticationProvider(authProvider)
                            .buildClient();

        graphClient.setServiceRoot("https://graph.microsoft.com/beta");
        Subscription subscription = new Subscription();
        subscription.changeType = this.changeType;
        subscription.notificationUrl = this.publicUrl + "/notification";
        subscription.resource = this.resource;
        subscription.expirationDateTime = Calendar.getInstance();
        subscription.clientState = "secretClientValue";
        subscription.expirationDateTime.add(Calendar.HOUR, 2);
        if (this.resource.startsWith("teams")) {
            subscription.additionalDataManager().put("includeResourceData", new JsonPrimitive(true));
            subscription.additionalDataManager().put("encryptionCertificate",   new JsonPrimitive(GetBase64EncodedCertificate()));
            subscription.additionalDataManager().put("encryptionCertificateId", new JsonPrimitive(this.alias));
            LOGGER.warn("encoded cert");
            LOGGER.info(GetBase64EncodedCertificate());
        }
        if (this.resource.startsWith("teams")) {
            subscription.additionalDataManager().put("includeResourceData", new JsonPrimitive(true));
            subscription.additionalDataManager().put("encryptionCertificate",   new JsonPrimitive(GetBase64EncodedCertificate()));
            subscription.additionalDataManager().put("encryptionCertificateId", new JsonPrimitive(this.alias));
        }

        subscription = graphClient.subscriptions()
            .buildRequest()
            .post(subscription);

those are your code and should return subscription id to listen to the user's event.
but once I change the resource to team, I will get unAuthentication problem. It even happened,once I delete this.resource.startsWith("teams") if condition.

did you redact the user id/name? it doesn't look valid

yes, I can run query in graph explorer by https://graph.microsoft.com/v1.0/users/209af94f359a5a39/events
I got result as well

I think the problem comes from ClientCredentialProvider or IGraphServiceClient  class, because we need to get Authorization : Bearer before we sent a request, I should get Authorization: Bearer ID to send any request. I also create a request to create an event, also I got the same problem
        ClientCredentialProvider authProvider = new ClientCredentialProvider(
            this.clientId,
            this.scopes,
            this.clientSecret,
            this.tenantId,
			this.endpoint);

        IGraphServiceClient graphClient = GraphServiceClient
                            .builder()
                            .authenticationProvider(authProvider)
							.buildClient();
		graphClient.setServiceRoot("https://graph.microsoft.com/beta");
		Event event = new Event();
		event.subject = "Let's go for lunch";
		ItemBody body = new ItemBody();
		body.contentType = BodyType.HTML;
		body.content = "Does late morning work for you?";
		event.body = body;
		DateTimeTimeZone start = new DateTimeTimeZone();
		start.dateTime = "2020-05-28T16:00:00";
		start.timeZone = "Pacific Standard Time";
		event.start = start;
		DateTimeTimeZone end = new DateTimeTimeZone();
		end.dateTime = "2020-05-28T17:00:00";
		end.timeZone = "Pacific Standard Time";
		event.end = end;
		Location location = new Location();
		location.displayName = "Harry's Bar";
		event.location = location;

		graphClient.me().events()
			.buildRequest()
			.post(event);

I confused about getting the Authorization: Bearer ID, I thought we need getAcccessToken to get Authoriaztion Bearer Id first and then send request to create calendar or subscription.
let me know if I am wrong

this is what the authProvider is supposed to do for us. Whenever we send a request that provider is supposed to request the token and attach it to the request before sending it.

if authProvider work fine, I should get different Bearer Id I think, but I can only return Authorization : Bearer eyJ0eXAiOiJKV1QiLCJub25jZSI[...] which is same every time. I hope it is fine, I just missing some small bugs.

the tokens are cached in memory AFAIK. but the cache should clear whenever you restart the application


Fix broken test_batch_code_upload test case

Description
Fix broken test_batch_code_upload test, caused by the addition of the plugins/types.py file to the std module.
Self Check:

[ ] Attached issue to pull request
[x] Changelog entry
[x] Type annotations are present
[x] Code is clear and sufficiently documented
[x] No (preventable) type errors (check using make mypy or make mypy-diff)
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation
[ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

Processing this pull request

Merged into branches master in c36e331e43d866036ec688e55bf173a3fe0993f8

Processing #6758.


Removed support for MRI ruby versions lower than 2.2

ditto

Coverage remained the same at 97.29% when pulling da897125a204eed086b12cc80625dd16c8925878 on AlfonsoUceda:master into d8d35c0152f4c71924f78b31b9bcb473e59f6d73 on lotus:master.

@AlfonsoUceda Thanks for this PR, this looks good! :+1:

We should hold before to merge this and keep merging master here, until we're close to the release.

ok ;)

Coverage remained the same at 97.29% when pulling b817e64ea943ddabbb6399c3a2415fc7b28880b0 on AlfonsoUceda:master into d8d35c0152f4c71924f78b31b9bcb473e59f6d73 on lotus:master.

Coverage remained the same at 97.29% when pulling b817e64ea943ddabbb6399c3a2415fc7b28880b0 on AlfonsoUceda:master into d8d35c0152f4c71924f78b31b9bcb473e59f6d73 on lotus:master.


How to combine separate repos into a single frontend build?

** Please describe which feature you have a question about? **
My website is separated into three repositories. I'd like the first to show up at web path /, the second to show up at path /forum/ and the third to show up at path /game/.
Is there a way I can have all the build artifacts go into the same website?

You could host the 3 separate sites on 3 different amplify apps and have a main application which handles the redirection to the other two namely /forum/ and /game/.
If you are interested in combining the 3 repositories under distinct folders (monorepo) we have support for that.  Check this out for more information about monorepos.
Closing for now - if you have any additional question please feel free to re-open the ticket.


Link to setup page about Kind

Following on from PR #17860, hyperlink from /docs/setup/ to /docs/setup/learning-environment/kind/ now that that target page exists.

/lgtm
/approve


'SSL_connect SYSCALL returned=5 errno=0 state=unknown state', type: IaasConfigurationVerifier

Support Question
Hi guys:
I get this error when I try to define a vCenter server in the BOSH Director for vSphere
'SSL_connect SYSCALL returned=5 errno=0 state=unknown state', type: IaasConfigurationVerifier
I've changed my vCenter to accept TLS 1.0 but the problem remains.
It happen with different versions of OpsManager :
pcf-vsphere-2.1-build.304.ova
pcf-vsphere-2.3-build.188.ova
pcf-vsphere-2.3-build.184.ova
I was googling but only get the same error associated to another systems.
Any help will be appreciated.
Thanks in advance
I forget to mention vCenter version 6.5 U2
You can also reach out to us via:

Slack: http://cloudfoundry.slack.com/messages/bosh-cpi-dev

Could you try reach out to opsMan team. It is not related to CPI. vSphere
CPI does not have any concept of verifiers.
More on verifiers:
https://docs.pivotal.io/pivotalcf/2-3/customizing/granular-verifier-control.html
On Tue, Nov 20, 2018, 2:03 AM agiorgi<EMAIL_ADDRESS>wrote:

Support Question
Hi guys:
I get this error when I try to define a vCenter server in the BOSH
Director for vSphere
'SSL_connect SYSCALL returned=5 errno=0 state=unknown state', type:
IaasConfigurationVerifier
I've changed my vCenter to accept TLS 1.0 but the problem remains.
It happen with different versions of OpsManager :
pcf-vsphere-2.1-build.304.ova
pcf-vsphere-2.3-build.188.ova
pcf-vsphere-2.3-build.184.ova
I was googling but only get the same error associated to another systems.
Any help will be appreciated.
Thanks in advance
You can also reach out to us via:

Slack: http://cloudfoundry.slack.com/messages/bosh-cpi-dev

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cloudfoundry/bosh-vsphere-cpi-release/issues/173, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGNfKjTCuZzZ_FezaBE3oB06UDHNos-Fks5uw9ONgaJpZM4Yqwqp
.


MSTest: Constructor runs for every test (rather than once for the class) during Debug

If you Debug an MSTest with one or more test methods, the constructor runs once for every test (rather than once for the entire class).  This does not seem to occur when running (not debugging).  The following code demonstrates the issue - failing when debugging (successful when not debugging):
`using System;
using Microsoft.VisualStudio.TestTools.UnitTesting;
namespace UnitTestProject1
{
[TestClass]
public class TestBase
{
}

[TestClass]
public class UnitTestSample
{
    static public int ConstructorCount { get; set; }

    public UnitTestSample()
    {
        ConstructorCount++;
    }

    [TestMethod]
    public void TestMethod1()
    {
    }

    [TestMethod]
    public void TestMethod2()
    {
    }

    [ClassCleanup]
    static public void ClassCleanup()
    {
        Assert.AreEqual<int>(1, ConstructorCount);
    }
}

}
`
@jinujoseph @GrantErickson @morrisjoe

@dotnet/vstest can you take a look?

This issue was moved to Microsoft/vstest#360


#300 - Implement return type annotation on lambdas

Not sure if the formatTests are sufficient enough.

Looks good to me, thanks!

Awesome!


Separate control messages cause complexity and bugs

The first integration trials of the "subscribe intent" in the prototype client failed when the source started at group 100, object 0. They did succeed when starting at group 0, object 0. The unit tests for starting with group 1, object 0 also succeeded. This is probably due to an issue with the state machines implemented in clients, relays and origin. A synchronization issue of some kind caused the relay to either not receive or not send the "start point" message that announced the change of origin. The clients then was waiting for object 0/0, that never came.
The immediate cause is probably a missed synchronization point in some condition. But the root cause is complexity. Publishing a media requires three messages:

POST message, to which the server replies with ACCEPT
START POINT message, sent after receiving ACCEPT when the transmission does not start at 0/0
CACHE POLICY message, sent after receiving ACCEPT when real time cache pruning is required.

Those three messages must be received and forwarded by relays, which creates complexity. For example, a client that subscribe to the relay may receive data after the POST is accepted by the relay, but may receive the START POINT message later or not at all. This complexity must be reduced.
The proposal is to merge the START POINT and CACHE POLICY message into the POST message, so the relays will receive a single event instead of three. The cache policy and the initial group ID would be passed as source properties in the publish source API. The change could be combine with a fix to issue #73, merging the "publish" and "post" API on the publisher.

+1 on merging these things

Should be solved by PR #101. Will need to reopen if the issue repeats.


Create security_zone with vni_id and fix incorrect assign pool

Detailed Description:

adding the ability to create sz with vni_id
missing comma "," for group_path will cause that pool will be assign incorrectly -> which cause that on GUI you will not be able to open /staged/virtual/routing-zones (will show blank screen - tested with Apstra 4.0.2)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

This change looks good. Will you please rebase off the latest main and push into a new branch? Create a PR off that branch and insure github-actions tox test is passing.

We cannot test this or run it through our CI unless these changes are pushed into a new branch. Please cancel this PR, push your changes into a new branch and create a PR from that branch. When the PR is approved the branch will be closed.


Replicate the 'Add a single price' view from the Open Prices web app

What

Replicate the 'Add a single price' view from the Open Prices mobile app (https://prices.openfoodfacts.org/app/add)

Figma mockups

https://www.figma.com/file/nFMjewFAOa8c4ahtob7CAB/Mobile-App-Design-(Quentin)?type=design&node-id=5816-22697&mode=design&t=0TyVkM5o0K1vKE0r-0

Part of

#5072

@teolemon Preview of location search:

empty
results

@monsieurtanuki you need a currency dropdown

proof is compulsory
=> already is
login is compulsory
=> ok
currency dropdown
=> it was in my todo list, as well as the proof type (price tag or receipt)

Shouldn't we also define the source (e.g. web app or Smoothie or whatever)?

the source is indeed a good idea, not sure openprices supports that @raphodn

the source is indeed a good idea, not sure openprices supports that @raphodn

Yes good idea ! Though it could easily be forged, no ? Do you already have a similar mecanism in OFF ?

In off-dart we use the user agent:
OpenFoodAPIConfiguration.userAgent = UserAgent(
  name: '<Name of your app>',
);

@raphodn To be more specific, our user agent data will be used with the following parameters for most GET requests:

app_name (the only mandatory parameter)
app_version
app_uuid
app_platform
comment

It's mostly used to debug API implementation issues, and identify if newer versions of the clients still have the issue

Ok I'll look at this today, to add a price.source field (and also a proof.source ?), that stores the user agent's app_name


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/opstest2/docs-archive-test-source.git

Docs Build status updates of commit 183ff58:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: RepositoryUnauthorizedWithSourceCommit] Git data access to 'https://github.com/opstest2/docs-archive-test-target' (branch: Release_Archive_main_2022-05-30-03-31-33) is not authorized.. It looks like this might be the first time user: v-alji submitted a pull request to this repo. Before user can run builds in a pull request, he/she can enable 'v-alji' GitHub account in Docs by visiting Docs Portal (https://ops.microsoft.com). After enabling his/her account (v-alji), close and then reopen this pull request to start another build.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


⚠ AGEG website (prod) has degraded performance

In d517f73, AGEG website (prod) (https://www.ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 2147 ms

Resolved: AGEG website (prod) performance has improved in 1f70031.


litmus chaos pod dns error experiment Failed to create Interceptor

What happened:
when executing the pod dns error experiment the helper pod errors with the following errors
k logs pod-dns-error-helper-flamem -n pgo
W0723 18:43:14.454777   68180 client_config.go:541] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
time="2021-07-23T18:43:14Z" level=info msg="Helper Name: dns-chaos"
time="2021-07-23T18:43:14Z" level=info msg="[PreReq]: Getting the ENV variables"
time="2021-07-23T18:43:14Z" level=info msg="container ID of database container, containerID: 95dbf9281725c57a4808fee3b856f0f626735e3a65592da22b100e23bdcd02a5"
time="2021-07-23T18:43:14Z" level=info msg="Container ID: 95dbf9281725c57a4808fee3b856f0f626735e3a65592da22b100e23bdcd02a5"
time="2021-07-23T18:43:14Z" level=info msg="[Info]: Container ID=95dbf9281725c57a4808fee3b856f0f626735e3a65592da22b100e23bdcd02a5 has process PID=60814"
time="2021-07-23T18:43:14Z" level=info msg="/bin/bash -c sudo TARGET_PID=60814 CHAOS_TYPE=error SPOOF_MAP='' TARGET_HOSTNAMES='' CHAOS_DURATION=60 MATCH_SCHEME=exact nsutil -p -n -t 60814 -- dns_interceptor"
time="2021-07-23T18:43:14Z" level=info msg="DNS Interceptor Port" port=53
time="2021-07-23T18:43:14Z" level=info msg="Upstream DNS Server" server="<IP_ADDRESS>:53"
time="2021-07-23T18:43:14Z" level=info msg="Chaos Error Targets" targets="[]"
time="2021-07-23T18:43:14Z" level=info msg="Chaos Spoof Map" spoof_map="map[]"
time="2021-07-23T18:43:14Z" level=info msg="Chaos type" chaos-type=error
time="2021-07-23T18:43:14Z" level=info msg="Target match scheme" match-scheme=exact
time="2021-07-23T18:43:14Z" level=fatal msg="Failed to create Interceptor" error="failed to inject interceptor in resolv.conf : open /etc/resolv.conf: read-only file system"
time="2021-07-23T18:43:14Z" level=fatal msg="Failed to run command" cmd=/usr/local/bin/dns_interceptor error="exit status 1"
time="2021-07-23T18:43:14Z" level=fatal msg="dns interceptor failed : exit status 1"

the permissions on the /etc/resolv.conf file in the container are:
-rw-r--r-- 1 root     root        145 Jul 23 18:34 resolv.conf

is there a way to run this experiment where it wouldn't need to edit the resolv.cof?  I've tried changing the permissions on the file but was unable to, any suggestions would be appreciated.
What you expected to happen:
I expected the pod dns error to complete and not error with a permissions error
How to reproduce it (as minimally and precisely as possible):
Create pod with the container docker image as centos8 and execute the pod dns error experiment.
Anything else we need to know?:

Assigning this to @uditgaurav & @ksatchit.
Val, feel free to join the slack community for community resolutions.
To join the slack please follow the following steps!
Step 1: Join the Kubernetes slack using the following link: https://slack.k8s.io/
Step 2: Join the #litmus channel on the Kubernetes slack or use this link after joining the Kubernetes slack: https://slack.litmuschaos.io/

thank you i'm asking on the slack channel now


Implementing Function strstr CPP

No description provided.

please use this file format nd then update your PR
i.e:-  ImplementStrstr/CPP/ImplementFunctionStrstr.c++

No description provided.

please use this file format nd then update your PR
i.e:- ImplementStrstr/CPP/ImplementFunctionStrstr.c++

@Palak0519 read this

Sure, will do that!

No description provided.

please use this file format nd then update your PR
i.e:- ImplementStrstr/CPP/ImplementFunctionStrstr.c++

@Palak0519 read this

Made changes in #179 PR. Kindly review.


Automator: update proxy@master in istio/istio@master

Generated by Automator - 2024-12-22T02:17:28+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
89058187342fdc002b7c18748649488b73606d59
link
true
/test unit-tests-arm64

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

/retest


[Bug]: lifecycle_hooks_exclude does not work with pnpm.neverBuiltDependencies

What happened?
I think this could be considered a feature request, but given what I needed to do to work around it, it feels more like a bug, or maybe missed requirement.
When using lifecycle_hooks_exclude with npm_translate_lock, it gets converted to the equivalent lifecycle_hooks definition under the hood. That's nice, but with pnpm v9, I now need to add pnpm.onlyBuiltDependencies. However, since this is an allow list, I now need to add every other package that has install scripts, even when I just want to exclude a few.
Pnpm has a nicer alternative when you just want to exclude and that's pnpm.neverBuiltDependencies. Rules_js doesn't check for this though, and still enforces you add onlyBuiltDependencies, which now requires us to maintain an ongoing allow list of all packages with install scripts, just so we can exclude a few.
Version
Development (host) and target OS/architectures:
Output of bazel --version: 6.5
Version of the Aspect rules, or other relevant rules from your
WORKSPACE or MODULE.bazel file: WORKSPACE
Language(s) and/or frameworks involved: JavaScript
How to reproduce
Create a JavaScript project with pnpm v9 or higher, add a package to `lifecycle_hooks_exclude` and add `pnpm.neverBuiltDependencies` with the same packages. Rules_js fails saying you must use `pnpm.onlyBuiltDependencies`.

Any other information?
No response

The problem is that pnpm 9+ no longer specifies what packages require a build, the requiresBuild attribute was removed.
So I think pnpm.neverBuiltDependencies is useless when we don't know which packages require a build in the first place.


Can't download from imagetwist.com

Example:
http://www.planetsuzy.org/showpost.php?p=11474030&postcount=9

This one worked Ok
http://www.planetsuzy.org/showpost.php?p=22010793&postcount=58

Hi!
This is weird, it works fine for me with the first page you mentioned.
Maybe Image Twist was having some maintenance on servers the day you tried it?
Can you confirm it works now?

It does work now. But at the time of the report it in some cases did and in some cases didn't.


Missing attrs in image list response

Image list API should send information about:

[ ] image is whitelisted
[ ] image has scan
[ ] scan result

These informations are indicated in deployment related context.