Azure DTL - The selected virtual machine does not support creating a custom image.

I am using a custom template to provision a VM with data disk in dev test lab. When i browse to the machine in DTL and click Create Custom Image from the blade, it shows me the following information message...
The selected virtual machine does not support creating a custom image.

Do machines creatd from custom images not support custom image functionality?

Can you elaborate on the scenario? Are you saying you created an Environment with a VM in it using your own ARM template? Any chance you can share the script with me, without any sensitive information, to help troubleshoot?

Leo take a simple custom template that provisions a network card, a data disk for a virtual machine using the Visual Studio Marketplace image. When you now try to create a custom image or formula u get the same message. If you are stuck I can share my template with you, if u have access to live site I can share the subscription and lab details to help investigate this.

Thank you. I wanted to ensure this was indeed a BYOA scenario. Unfortunately, in a BYOA scenario we do not support some lab features, like creating custom images, by design.

Leo I am using a data disk. Does the lab support creating a snap shot of the data disk and adding that to the custom image library? I know that the service runner gets certain elevated permission when provisioning in the lab, does this also include permissions on data disk? I ask because I am in an enterprise setup with full access to the Dev Test lab resource group but since each new VM gets provisioned in its own new resource group without the runner delegating more permissions it gets hard to use in the enterprise.

Just wanted to acknowledge that I understand this is by design but I see this as a very big limitation. Do you have this on the backlog? When is this forecasted for?

I'm trying to understand the scenario better. Is there a reason you are not creating the custom image using the normal DTL flow (i.e. not using BYOA)? If you create a lab VM and add a data disk, you can then create a custom image, which will include the data disk, as long as the custom image is generalized.

The simple single vm solution doesn't support is profile specification. Which is required if you want to set up an Internet proxy when the vm is being provisioned. This is a pre-requiste to be able to run the extensions such as download and execute powershell, bgInfo and Domain Join. The simple CM doesn't support this scenario and BYOT doesn't support custom image and formula creation, this use case really needs to be supported in my opinion.

Tarun, can you setup a call to do this faster. Please send me or Leo a meeting invite.
Meanwhile, if you want to create a custom image from a VM that you created via BYOA, you can follow this approach.

Create the VM in Lab via BYOA flow.
Apply sysprep artifact on the VM so that the machine is generalized.
Once the VM shutsdown, locate its backing VHD from the Compute VM blade Disks blade. If the VM is Managed, you will need to export the disk into a VHD.
Once you have the VHD, you can import it into a Lab and create a custom image.

This approach will not include any data disks in the custom image. However if you create a custom image from a VM created in the regular flow (non BYOA), and if its generalized, then the data disks get baked into the custom image.

@ajay-arora what's ur email address? What time zone are you in?

ajaroraatmicrosoftdotcom. I am in PST.

We will be closing this issue. Please reopen if you are still seeing this error.


Lazy csv file render

Gitea version (or commit ref): 1.16
Git version:
Operating system:

Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No

Log gist:

Description
Request lazy load feature for csv file render. Some large csv files will make the browser stuck.
Maybe it's better to render the first 100 rows at first, then show a load more button at the bottom to render the next 100 rows, and so on.

Wouldn't it be much easier to install a browser extension that supports full CSV rendering/editing/JSON-export etc on the raw CSV file?


Fix encoding issue in XMLReporter

The writeSuiteToFile method currently uses the writeFile method instead of writeUtf8File, resulting in encoding issues in the generated file for certain characters (for example, "en-dash", "em-dash", and the "section sign" character).
Refer to line 69 for an example of where writeUtf8File is already being used.

LGTM  :+1: Thanks.
Do you think you'll be able to add a little test ? Maybe by writing the file and then by looking for the expected value.

So I've spent most of the day attempting to add a test as requested, but unfortunately haven't had much luck.
To give some context - the problem has to do with the default system encoding that's set during a test run (more specifically I believe it gets set when the JVM is started and cannot be changed afterwards).  What I'm not sure of is how best to implement a test for this that will not be dependent on the settings of the system on which it is run.
To further explain, if you take a look at line 240 of Utils.java, basically what it does is check to see if the specified encoding is null or not.  If not, it uses that encoding in the writer; if it is, then it creates the writer without specifying the encoding, in which case the system default encoding is used.
What that means is in many cases it will work just fine; it's only for a select set of characters in somewhat less common scenarios that it becomes an issue.
My team encounters it frequently because we use TestNG (through Ant) to run Selenium tests on remote Windows VMs (has to be Windows so we can accurately match actual end user configurations) for an application that handles legal content (where these characters are very common).  I've tried many other ways of getting around this problem over the past couple of years without success (the number of variables involved makes it challenging to come up with a solution that works in all circumstances, both for our nightly regression test runs as well as running tests locally through Eclipse to debug issues); what I ultimately had to do in our internal test suites that leverage TestNG is basically create our own copy of the XMLReporter where I changed that one call from writeFile to writeUtf8File (which is how I know the fix works even if it's difficult to prove).
Sorry I can't do much more here; I'll keep investigating to see if I can find a way to reliably reproduce the issue this is meant to address in a way that is not dependent on the default system encoding, but it's not looking too hopeful.

@entryspace Thanks for the detailed report and giving this a try.
@juherr Are you okay with merging this even without a test?  (I am)

I am too. :+1:
Thanks for the effort.


Idea creates .iml files on multi module projects

I believe *.iml are also part of the Intellij configuration strategies, and should be ignored when present in a module directory.
Reasons for making this change:
TODO
Links to documentation supporting these rule changes:
TODO
If this is a new template:

Link to application or project’s homepage: TODO

@fotisp thanks for the contribution. We have an existing section which is commented out by default that feels relevant here:
https://github.com/github/gitignore/blob/105ce45ee9b3ac4a9de282752737269b0adc0b93/Global/JetBrains.gitignore#L27-L38
Does this make sense? Can this be improved at all?

Oh sorry, didn't see that, really sorry


Bug when loading a saved world

Bug when loading a saved world

did you load a pre 2.0 savegame?

Duplicates #49


Make VTK dependency optional

+10

This is already optional when running the worker, so I'm closing this. Related: #50


Read flags from a global file

It might be helpful to be able to read flags from both the user's ~/.config/chromium-flags.conf and a global file, possibly /etc/chromium-flags.conf.

A single file would be too limiting and a configuration directory would be too complicated and possibly an overkill. For single-user machines, the existing implementation suffices. For systems with many users, it's not clear that a global configuration would be appropriate for all of them.
Thank you for taking the time to suggest this, but the use case for a global file isn't interesting to me.

Thanks for the response! I guess I will implement this in my own fork then.


RET-2631 Update Appium and Chromedriver

This was committed but this pull request was messed up because it was originally based off of the update-nginx branch.


How to use this tool in typescript?

How to use this tool in typescript?

How could i import this module in typescript?

this could not work


KYLIN-4775 Use docker-compose to deploy Hadoop and Kylin

Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to Kylin?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have create an issue on Kylin's jira, and have described the bug/feature there in detail
[x] Commit messages in my PR start with the related jira ID, like "KYLIN-0000 Make Kylin project open-source"
[ ] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If this change need a document change, I will prepare another pr against the document branch
[ ] Any dependent changes have been merged

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

Pull Request Test Coverage Report for Build 6394

0 of 0   changed or added relevant lines in 0 files are covered.
14 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.006%) to 28.049%

Files with Coverage Reduction
New Missed Lines
%

tool/src/main/java/org/apache/kylin/tool/query/ProbabilityGenerator.java
1
78.95%

core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java
4
78.12%

core-metrics/src/main/java/org/apache/kylin/metrics/lib/impl/MetricsSystem.java
9
60.0%

Totals

Change from base Build 6388:
-0.006%

Covered Lines:
26242

Relevant Lines:
93558

💛  - Coveralls

如何启动一个集群
进入docker 目录：
cd apachekylin/kylin/docker

执行如下命令可以 构建docker镜像，并启动docker-compose
sh setup_cluster.sh --cluster_mode write-read --hadoop_version 2.8.5 --hive_version 1.2.2 --enable_hbase yes --hbase_version 1.1.2  --enable_ldap no

--cluster_mode 参数值可选项为[write,write-read]默认参数是 write，也就是会启动一个写集群，不做读写分离
在write 模式下会启动一个 hadoop集群包含hive，hbase，等。在write-read模式下会启动两个hadoop集群，一个作为计算集群，一个作为查询集群，并且hbase装在查询集群中
--hadoop_version 可以指定hadoop的版本，默认是 2.8.5
hadoop 集群包含 1个 namenode节点和3个datanode节点，以及1个resourcemanager和2个nodemanager，以及一个jobhistoryserver。
--hive_version 可以指定hive版本，默认是 1.2.2
hive 包含一个 hivemetastore 和一个 hiveserver2 节点
--hbase_version 可以指定hbase版本，默认是 1.1.2
--enable_hbase 是否启动habase服务，可选参数为[yes,no], 默认参数是yes
hbase 包含一个 hmaster 和两个 regionserver
--enable_ldap 是否启动ldap服务，可选参数为[yes,no], 默认参数是no
--enable_kafka 是否启动kafka broker，可选参数为[yes,no], 默认参数是no
--enable_kerberos 是否启动 kerberos 服务，可选参数为[yes,no], 默认参数是no。(目前尚为开发完成)

另外会启动一个kylin集群，包含一个all节点一个job节点和一个query节点。三个节点的kylin目录与 docker-compose/others/kylin/ 一一对应
其中。一般将打包好的kylin jar包解压到 对应的路径下，并修改相应的配置文件，即可实现部署和修改kylin配置的目的。
如果是第一次执行以上命令，那么会首先去构建和拉取相应的docker镜像，所以执行时间会很长，一般会在半个小时左右才能完成。
脚本执行完成后可以执行如下命令查询集群状态：
docker ps -a

等待所有的container都达到health状态即可执行kylin的测试工作。
关闭集群：
sh stop_cluster.sh

Hadoop Version
Hive Version
HBase Version

2.8
1.2
1.2

2.8
2.2
2.2

3.1
2.2
2.2

3.1
3.1
3.1

Content from cell 1 | Content from cell 2

@liuyonghengheng  Really appreacite your contribution! @zhangayqian will add your patchs in her PR(  https://github.com/apache/kylin/pull/1465) with some modification, so I will close this one .


GPUTreeSHAP crashes Google Colab

Hi. I am trying to utilize Google Colab GPUs to speed up some of my work with xgboost. By default, Colab machines are set up with Python 3.7 and Xgboost 0.90. Running !pip install --upgrade xgboost installs version 1.4. However, when I am trying to run booster.predict(dmat, pred_contribs=True)) in a loop (I am trying to compute SHAP values for different models on the same dataset), it runs for exactly 32 iterations and then crashes the Colab kernel.
I don’t think the crash is related to resource consumption as RAM and GPU memory loads look low. Also, my dataset and model are pretty small and xgboost predicts the SHAP values blazingly fast until it crashes.
I suspect this is some compatibility issue on the Colab end, but was wondering if there may be things I can try to resolve it, as I don’t have a GPU at home. Any advice would be greatly appreciated. Thanks!
UPDATE:
Here is the crash log from the Colab machine:
|Jul 8, 2021, 12:20:35 AM | WARNING | WARNING:root:kernel 1556ff02-ab27-4cf4-9743-414d48877806 restarted|
|---|---|---|
|Jul 8, 2021, 12:20:35 AM | INFO | KernelRestarter: restarting kernel (1/5), keep random ports|
|Jul 8, 2021, 12:20:32 AM | WARNING | what(): device free failed: an illegal memory access was encountered|
|Jul 8, 2021, 12:20:32 AM | WARNING | terminate called after throwing an instance of 'thrust::system::system_error'|

This is happening on Tesla P100 and V100 cards with Ubuntu LTS 18.04.

Thanks for the bug report. Do you have a python script with xgboost that reproduces the issue? Maybe something with randomly generated data? It's helpful if you can simplify the example as much as possible.

Thanks for the fast response. Appreciate it!
I did some experimentation and found out that it is the hyper-parameter tuning function I have in my code that is interacting with GPUTreeSHAP and causing the crash, here is the minimum working example:
import numpy as np
import pandas as pd
from sklearn.model_selection import StratifiedKFold
from xgboost import XGBClassifier, DMatrix

X = pd.DataFrame(np.random.rand(10**4, 10**2))
X_train = pd.DataFrame(np.random.rand(10**2, 10**2))
y_train = pd.Series(np.random.randint(0, 1, X_train.shape[0]))

shap_values = []
base_estimator = XGBClassifier(
    tree_method='gpu_hist', gpu_id=0,
    use_label_encoder=False, eval_metric='error', random_state=42
)

def tune_n_estimators(base_estimator, X_train, y_train, early_stopping_rounds=5):
    base_estimator_params = base_estimator.get_params()
    kfold = StratifiedKFold()
    best_iterations = []
    estimator = XGBClassifier(**base_estimator_params)
    for train_fold, cv_fold in kfold.split(X_train, y_train):
        train_index = X_train.index[train_fold]
        val_index = X_train.index[cv_fold]
        X_train_fold = X_train.loc[train_index]
        y_train_fold = y_train.loc[train_index]
        X_cv_fold = X_train.loc[val_index]
        y_cv_fold = y_train.loc[val_index]
        estimator.fit(
            X_train_fold, y_train_fold,
            early_stopping_rounds=early_stopping_rounds,
            eval_set=[(X_cv_fold, y_cv_fold)],
            eval_metric='auc',
            verbose=0
            )
        best_iterations.append(estimator.best_iteration)
    new_params = base_estimator_params.copy()
    new_params['n_estimators'] = int(np.mean(best_iterations))
    estimator = XGBClassifier(**new_params)
    return estimator

for i in range(100):
    estimator = tune_n_estimators(base_estimator, X_train, y_train)
    estimator.fit(X_train, y_train)
    booster = estimator.get_booster()
    booster.set_param({'predictor': 'gpu_predictor'})
    dmat = DMatrix(X)
    shap_values.append(booster.predict(dmat, pred_contribs=True))
    print(i)

As you can see, I am creating new xgboost instances within tune_n_estimators(), maybe that is causing the problem? Thank you!

Figured it out! The tuning function was returning 0 for n_estimators on some iterations. It looks like unexpected behavior to me for early_stopping but definitely, something that should be addressed upstream of GPUTreeSHAP, still, it is interesting that it crashes the kernel.

Thanks! I'll make a note of it.


Make hovering over name of longform review author clearly a clickable step to read the review

It is unclear that user can click on name to read the review. Maybe make it go italic or raise as a button when hovering over name?

Closing; obsolete after #321


Update (10 => 5 min) cwa-risk-assessment.md and risk_calculation_enf_v2_overview.pdf

Where to find the issue

https://github.com/corona-warn-app/cwa-documentation/blob/master/images/risk_calculation/risk_calculation_enf_v2_overview.pdf

cwa-risk-assessment.md

Describe the issue

Says 10 minutes
Suggested change

should say 5 minuts

Perhaps @tklingbeil can update his diagram to change the 10 minutes to 5 minutes?
risk_calculation_enf_v2_overview.pdf

@dsarkar
Regarding cwa-risk-assessment.md: this document describes risk calculation based on the v1 model. CWA now uses the v2 model, so the document cwa-risk-assessment.md is no longer correct at all. There is an open issue about this point https://github.com/corona-warn-app/cwa-documentation/issues/453.
It is not just a question of changing 10 minutes to 5 minutes in that document. In fact I couldn't see any mention of 10 minutes in that document.

@dsarkar
This issue has been fixed in the current document https://github.com/corona-warn-app/cwa-documentation/blob/main/solution_architecture.md#mobile-applications.
The document https://github.com/corona-warn-app/cwa-documentation/blob/main/cwa-risk-assessment.md is now deprecated.
You should be able to close your issue now.

@dsarkar Friendly reminder: Please close this issue.

@dsarkar

Your issue can be closed since it has been implemented through PR https://github.com/corona-warn-app/cwa-documentation/pull/822 on Jan 25, 2022.

See https://github.com/corona-warn-app/cwa-documentation/blob/main/images/solution_architecture/risk_calculation.svg linked to from document https://github.com/corona-warn-app/cwa-documentation/blob/main/solution_architecture.md#mobile-applications.


Linear Search | PHP

#hacktoberfest
Reference: https://www.geeksforgeeks.org/linear-search/

Please #hacktoberfest-accepted

#22
#hacktoberfest
#hacktoberfest-accepted
@

Added requested changes and some debugging


dnssd-browse only browses once at start

Hi,
thanks for building this useful tool for node-red.
I am missing one feature, browsing for a service regulary. For example, every 10 minutes, or at every input, which can be triggered.
The problem I am facing is, that my ipv6 addresses changes every 24 hour, and also all ip addresses in the thread network changes as well. Therefore, I have to browse again if the ip address changes, but there is no way of doing that.
What do you suggest? Do you have this use case in mind?
Best
Dongha

That is not a use case that I had in mind as my device IP addresses are fixed. But I've been thinking of adding the ability to trigger starting / restarting the browser as a way to recover from network errors, which I think might cover your requirement as well? Can't promise how soon I might get to it, but I'll keep your request in mind when I get back to this.

Added input message support to dnssd-browser node. Supports start, stop and list methods of dnssd.Browser. Browsers will no longer start automatically when flows start. Send {"command":"start"} in msg.payload to begin listening.

Thanks a lot!
I'll try it and give you a feedback.

I couldn't test it, because I don't find dnssd-browser on my node-red after updating package.


Fixing link for pre-registered organizations to check their status at admin.dataportal.se

Pull Request Description
Please include a summary of the change and which issue is fixed or added.
Please also include relevant motivation and context.
List any dependencies that are required for this change.
Fixes #(issue)
Checklist

[x] Changes are limited to a single goal (avoid scope creep)
[x] I confirm that I have read any Contribution and Development guidelines (CONTRIBUTING and DEVELOPMENT) and are following their suggestions.
[x] I confirm that I wrote and/or have the right to submit the contents of my Pull Request, by agreeing to the Developer Certificate of Origin, (adding a 'sign-off' to my commits).

Issue was fixed.


Tensoboard callback need to create different logs for each run

Tensoboard callback need to create different logs for each run, otherwise will create a graph in which each different run override the previous ones

I will create a different folder using current time


[WIP] Network view

Initial implementation of the network view.
It will follow the established UI convention of the details panel.

Hover over a "Networks" overlay button to show networks.
Clicking the "Networks" overlay button to show and pin all networks and reveal a legend w/ network names listed.
Hover over individual networks labels to fade out nodes that are not included in that network.
Click individual networks to open the details panel w/ network info and pin the faded/highlighted nodes states. v2
Clicking the nodes should overlay an additional details panel that can be popped to return the network details.

[x] Add coloured circle segments to a node that represents network membership.
[ ] click legend item to show details panel showing network details. v2
[x] style legend w/ colors
[x] Allow enable/disabled network-arc overlays (maybe by expanding the legend).
[x] highlight networks members on legend item hover (or fade others)
[x] include net-view state in url state

stretch

[x] add a collapsed state to the legend.

Design RC:

Report to test w/ tnscompose.json.zip

Visual:

The underlines are flush against "Networks" which looks a bit abrupt. Have you tried underlining "Networks" in neutral color?

Could we make this not overlap?

Could this be ameliorated?

Negligible edge case?
JS Code looks good.
My comment re data still applies:
       // TODO move this setting of networks as toplevel node field to backend,
       // to not rely on field IDs here. should be determined by topology implementer
       state = state.update('nodes', nodes => nodes.map(node => {
         if (node.has('metadata')) {
           const networks = node.get('metadata')
             .find(field => field.get('id') === 'docker_container_networks');
           if (networks) {
             return node.set('networks', fromJS(
               networks.get('value').split(', ').map(n => ({id: n, label: n, colorKey: n}))));
           }
         }
         return node;
       }));


iphone 11 16.1 Jailbreak failure when not connected

I was trying to jailbreak while offline and it failed multiple times. However as soon as i got online it worked l. Is this intentional?

I can’t reproduce with airplane mode only with no wifi to connect to.

placebo

Pretty sure its not but okay

Even if it's not, there would be no way I could do anything about it.


Fix bot git handling

This PR uses persist-credentials: false in the checkout@v2 action to avoid overriding the credentials we provide manually in the Github URL. Additionally, it uses shallow clones of the repository where possible to speed up jobs.

rebase!


map_blocks converts dask arrays kwargs to strings

Hi,
The following example causes me trouble:
import dask.array as da
import numpy as np

indices = da.arange(4, chunks=4)
data = da.arange(25, chunks=5).reshape(5, 5)

def get_samples(indices, data):
    print(data, indices)
    return data[:, indices]

da.map_blocks(get_samples, indices, data=data, new_axis=0, dtype=data.dtype).compute()

Basically, it looks like the data variable is converted to a string, hence the [:, indices] failing with a TypeError. Using a numpy array as data works as expected.
(btw, this is obviously just a toy example).
I understand that data needs to be computed before we can run map_blocks, but I don't want to do it just there.
So this looks like a but to me, but maybe I'm missing something ?
Traceback:
-> python test_dask.py                                                                                tmp/
('finalize-4b8aee4f1226703a4df0635fe70b6c86', array([0, 1, 2, 3]))
Traceback (most recent call last):
  File "test_dask.py", line 11, in <module>
    da.map_blocks(get_samples, indices, data=data, new_axis=0, dtype=data.dtype).compute()
  File "/home/a001673/.local/lib/python2.7/site-packages/dask/base.py", line 143, in compute
    (result,) = compute(self, traverse=False, **kwargs)
  File "/home/a001673/.local/lib/python2.7/site-packages/dask/base.py", line 392, in compute
    results = get(dsk, keys, **kwargs)
  File "/home/a001673/.local/lib/python2.7/site-packages/dask/threaded.py", line 75, in get
    pack_exception=pack_exception, **kwargs)
  File "/home/a001673/.local/lib/python2.7/site-packages/dask/local.py", line 521, in get_async
    raise_exception(exc, tb)
  File "/home/a001673/.local/lib/python2.7/site-packages/dask/local.py", line 290, in execute_task
    result = _execute_task(task, data)
  File "/home/a001673/.local/lib/python2.7/site-packages/dask/local.py", line 271, in _execute_task
    return func(*args2)
  File "test_dask.py", line 9, in get_samples
    return data[:, indices]
TypeError: string indices must be integers, not tuple

I believe that map_blocks currently only unpacks dask arguments from args, not from kwargs.  This could be changed if someone wants to submit a fix, but short term I would recommend including your dask array arguments as normal positional arguments.

@mrocklin I believe he intended to compute the entire data array and pass it to the function so it could be indexed. The oddness here is that the token of the dask array seems to get passed to the function and not the dask array.

@mrocklin yes, I understand that, and the intended purposed wasn't to unpack the data array, rather to index it. The strange thing is that map_block transform the data dask array into a string (looks like the token as @davidh-ssec says) that is then unusable as is in the get_samples function.

Also passing data as a positional argument exhibits the same problem:
da.map_blocks(get_samples, indices, data, new_axis=0, dtype=data.dtype).compute()
-> TypeError: string indices must be integers, not tuple

Using positional arguments I get the following error:
In [1]: import dask.array as da
   ...: import numpy as np
   ...:
   ...: indices = da.arange(4, chunks=4)
   ...: data = da.arange(25, chunks=5).reshape(5, 5)
   ...:
   ...: def get_samples(indices, data):
   ...:     print(data, indices)
   ...:     return data[:, indices]
   ...:
   ...: da.map_blocks(get_samples, indices, data, new_axis=0, dtype=data.dtype).compute()
   ...:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
/home/mrocklin/workspace/dask/dask/array/core.py in map_blocks(func, *args, **kwargs)
    759             try:
--> 760                 chunks2 = list(broadcast_chunks(*[a.chunks for a in arrs]))
    761             except Exception:

/home/mrocklin/workspace/dask/dask/array/core.py in broadcast_chunks(*chunkss)
    815         if len(set(step2)) != 1:
--> 816             raise ValueError("Chunks do not align: %s" % str(step2))
    817         result.append(step2[0])

ValueError: Chunks do not align: [(4,), (5,)]

During handling of the above exception, another exception occurred:

ValueError                                Traceback (most recent call last)
<ipython-input-1-ec53fee48272> in <module>()
      9     return data[:, indices]
     10
---> 11 da.map_blocks(get_samples, indices, data, new_axis=0, dtype=data.dtype).compute()

/home/mrocklin/workspace/dask/dask/array/core.py in map_blocks(func, *args, **kwargs)
    760                 chunks2 = list(broadcast_chunks(*[a.chunks for a in arrs]))
    761             except Exception:
--> 762                 raise ValueError("Arrays in `map_blocks` don't align, can't "
    763                                  "infer output chunks. Please provide "
    764                                  "`chunks` kwarg.")

ValueError: Arrays in `map_blocks` don't align, can't infer output chunks. Please provide `chunks` kwarg.

Yes, I get that error too with data as positional argument, sorry for that.
Anyway, the use case is when you want to create an array with the values of another (data) using the indices which has the shape of the array to create.
In that case we can't really use data as a positional argument. Is there a way to delay the evaluation of data until get_samples is executed, since passing data as a dask array in the kwargs just returns its token ?

I looked a bit further, and it looks like this function is responsible for replacing the data array with its token:
https://github.com/dask/dask/blob/69ea16e388c215b6c1a849d543a690334406d2e2/dask/delayed.py#L32-L37
So I see the delayed objects are replaced with their tokens, should it do the same for dask arrays ? Is there a way to retrieve the dask array from its token somehow ?

Small update on this: I was able to use atop to solve @mraspaud's problem. He had written some logic to get around dask arrays not supporting indexing by dask arrays and I used that to provide the necessary dimension information to atop.  For reference (although I don't know how readable this is out of context):
        def _my_index(index_arr, vii, data_arr, vii_slices=None, ia_slices=None, fill_value=np.nan):
            vii_slices = tuple(x if x is not None else vii.ravel()
                               for x in vii_slices)
            mask_slices = tuple(x if x is not None else (index_arr == -1)
                                for x in ia_slices)
            ia_slices = tuple(x if x is not None else index_arr for x in ia_slices)
            res = data_arr[vii_slices][ia_slices]
            res[mask_slices] = fill_value
            return res

        new_data = data.data.reshape(flat_src_shape)
        vii = vii.ravel()
        dst_adims = [dst_dim_to_ind[dim] for dim in dst_dims]
        ia_adims = [dst_dim_to_ind[dim] for dim in dst_geo_dims]
        res = da.atop(_my_index, dst_adims,
                      ia, ia_adims,
                      vii, flat_adim,
                      new_data, src_adims,
                      vii_slices=vii_slices, ia_slices=ia_slices,
                      fill_value=fill_value,
                      dtype=new_data.dtype, concatenate=True)

Where ia is an integer index array (dask), vii is a boolean array of "valid inputs", and the original input data is an xarray DataArray. Above this code is a big for loop to map source dimensions to output dimensions along with figuring out which dimensions are flattened to handle the boolean array indexing being used. The ugliest part of this is that I have to provide these _slices lists so that I ignore dimensions I don't care about and slicing the ones I do care about. Not sure there is an easier way to do this.
I guess it is up to you guys if this issue stays open since the original question is more generic than our use case.

I can run the code snippet without error with dask 2021.09.0, so I'm going to close this issue since it looks like it's been addressed in more recent versions of Dask.
Anyone still experiencing this should try upgrading their version of dask, and if that doesn't help then please open an issue.


Update action.yml

The descriptions refer to the unpack action, which creates a little bit of confusion.
Ref: https://github.com/microsoft/powerplatform-actions/blob/main/unpack-solution/action.yml

@GTRekter PR loop fails due to PAT not resolving for your run. I'm overriding to merge your fix. Thx!


Kult Divinity Lost: Improved styling and fixed minor layout bugs

Updated trait layout in line with the core rulebook.
Made the appearance, dark secret, gear and weapons textareas auto-resize.
Fixed bug where textareas didn't always resize quite right.
Less boxy relations layout.

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[x] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Added the auto-resize code to the appearance, dark secret, gear and weapons textareas.
Made the dramatic hooks textarea 100% width.
Made the background 100% width instead of cover so it doesn't get stretched on long sheets.
Removed superfluous columns in the grid layout and made the column gap larger.
Made all textareas have the box-sizing: border-box; style so they match the text inputs.
Restyled the traits: added coloured text and thick borders to match the layout of the core rulebook.
Restyled the relations: removed box outlines to make it a single block, with thick borders as dividers.
Took out various chunks of obsolete CSS.

Character Sheet Info Roll20 Internal Use only.


Can I reinclude your modification to the entity multithreading in a fork of mcmt fabric ?

Can I reinclude your modification to the entity multithreading in a fork of mcmt fabric ?

Yes, of course.


fix(sec): updating libraries to reduce number of vulnerabilities, had…

… to update a few imports from the new libraries
📲 What
Updating libraries
🤔 Why
To fix a number of known vulnerabilities in Stacks
🛠 How
More in-depth discussion of the change or implementation.
👀 Evidence
Screenshots / external resources / links / etc.
Link to documentation updated with changes impacted in the PR
🕵 How to test
Notes for QA
✅ Acceptance criteria Checklist

[ ] Code peer reviewed?
[ ] Documentation has been updated to reflect the changes?
[ ] Passing all automated tests, including a successful deployment?
[ ] Passing any exploratory testing?
[ ] Rebased/merged with latest changes from development and re-tested?
[ ] Meeting the Coding Standards?

Hey Paul, for Java currently two branches are maintained and the Pr would also need to be for the without cqrs feature branch please. (that branch isn't well maintained and doesn't have proper builds running on it).
Also once merged a PR needs to be open to stacka-webapp-template repo to update with the new shas. I can show you how to do this if you'd like! =)

Hey Paul, for Java currently two branches are maintained and the Pr would also need to be for the without cqrs feature branch please. (that branch isn't well maintained and doesn't have proper builds running on it. EDIT: There needs to be a proper discussion on whether it's worth maintaining a second without cqrs branch or not and how we properly run tests etc against it).
Also once merged a PR needs to be open to stacks-webapp-template repo to update with the new shas. I can show you how to do this if you'd like! =)

I've added the raised a PR for the no-cqrs branch.
Yes that would be great if you could show me the sha updates.


How to play dfp playlist ad from vmap

Hi @shawnbuso
Kindly check the below link,
https://goo.gl/s4bTCj

The link below generate vMAP list with Vast ads
Click here
Video sample on my website
Please your advice?
Regards
Hany

Hi there,
You can have the plugin serve canned ads by specifying the VAST or VMAP directly in the options via adsResponse:
var options = {
  fluid: true,
  id: 'content_video',
  adsResponse: '<vmap:VMAP xmlns:vmap="http://www.iab.net/videosuite/vmap" version="1.0">' +
'<vmap:AdBreak timeOffset="start" breakType="linear" breakId="preroll">' +
'<vmap:AdSource id="preroll-463" allowMultipleAds="false" followRedirects="true">' +
'<vmap:AdTagURI templateType="vast3">' +
'<![CDATA[ ]]>' +
'</vmap:AdTagURI>' +
'</vmap:AdSource>' +
'</vmap:AdBreak>' +
'</vmap:VMAP>',
};

Note that the VMAP you provided doesn't have any ad tags in it, so you'd still need to modify it with your ad tag URI.

Closing this out as there hasn't been any activity for a while. Feel free to reopen.


owner permissions still appearing [sanity-tets]

Issue
Despite it being known that corso cannot handle restoring one-drive owner permission, we keep failing sanity test checks on that expectation.  Either the onedrive code needs to be made more resilient here, or the sanity tests need to identify and ignore this case.

PR - https://github.com/alcionai/corso/pull/3103


Adding script that consolidates our existing install instructions

/cc @achandmsft @sptramer
This consolidates the instructions that are currently found here into a single script. The idea is that we can tell Debian/Ubuntu users to just execute sudo curl -sL https://aka.ms/InstallAzureCLIDeb | bash.
Because they will need to use sudo to do this, I've added a conspicuous warning that they should try to refrain from sudo executing scripts from sources other than well trusted entities.

@marstr Which versions of Ubuntu/Debian is this script verified to work on?

Excellect question, @sptramer. I have confirmed this with the following:

Distro
Versions

Ubuntu
bionic, xenial, cosmic

Debian
stretch, jessie

By end of week, I expect to also support ubuntu:disco

@marstr, just a question, will this script also work on WSL?

@yugangw-msft says:
@marstr, just a question, will this script also work on WSL?

Yes! This should work on WSL. I'll have @kristapratico try this now, assume no news is good news on that front.

Closing and opening to attempt to kick DevOps into paying attention.

/AzurePipelines run

For anybody who wants to try this, the command to use is this:
curl -sL https://aka.ms/InstallAzureCLIDeb > az_install_apt.sh && sudo bash ./az_install_apt.sh

@achandmsft @marstr, this change looks good to me. My only question is that the script doesn't include the prerequisite configuration e.g. sudo apt-get install curl apt-transport-https lsb-release gnupg. I am not very familiar with Linux community, but if those packages are common enough then skipping this step is fine. My worry is mainly on the debian distros which might not even have the curl. Ubuntu should be fine since it has preloaded lots of common stuff.

I've only tested in container environments where those tools might not be present for minification purposes. You raise a good point though, I'm creating a VM based on debian:9 to check.

Good call, @yugangw-msft. Those pre-requistes are now installed at the beginning of the script.


[WIP] Display hash table at start

Proposed changes
Display hash table for each available component in a project
Fixes
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to agents-aea?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist
Put an x in the boxes that apply.

[ ] I have read the CONTRIBUTING doc
[ ] I am making a pull request against the develop branch (left side). Also you should start your branch off our develop.
[ ] Lint and unit tests pass locally with my changes and CI passes too
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that code coverage does not decrease.
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Codecov Report

Merging #78 (35dfbdd) into main (1894b3b) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #78      +/-   ##
==========================================
- Coverage   96.99%   96.99%   -0.01%
==========================================
  Files         307      307
  Lines       24164    24196      +32
==========================================
+ Hits        23437    23468      +31
- Misses        727      728       +1

Flag
Coverage Δ

unittests
96.99% <100.00%> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

aea/cli/run.py
88.52% <100.00%> (+3.90%)
:arrow_up:

aea/configurations/constants.py
100.00% <100.00%> (ø)

aea/test_tools/test_contract.py
98.82% <0.00%> (-1.18%)
:arrow_down:

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 1894b3b...35dfbdd. Read the comment docs.


Add ale_go_gofmt_options variable

Fix #1817

I tried to add #1817 variable.
Sorry, I don't have docker environment (because WSL debian now) so couldn't run ./run-tests script.
And also don't have vador environment.
But I think it's OK because it seems CI checks them.

Now all checks are passed, rebased it.

oops, PR title and commit message is wrong... fixed it. s/gofmt/govet/

Cheers! :beers:

Thanks! :tada:


Allow module references to be public JSON template URIs

Closing the issue since we will add support for referencing modules available in public module registries with anonymous authentication, so it may not be necessary to implement public URI references. If you think this is still a requirement, feel free to reopen the issue.


plugin doesn't work with EAP 2021.1 / "Please ensure JavaScript is enabled."

Hi folks,
Trying this plugin in latest version of WebStorm (EAP) but when I try to open a diagram it says:
Page could not be loaded. Please try refreshing.
Please ensure JavaScript is enabled.

I cannot find any information about how to get this to work Am I missing something?
Cheers,
Danny

interesting. It should just work out of the box. Let me install webstorm and give it a try.

what I did:

installed latest version of webstorm
started it as free trial
installed asciidoctor plugin (is this maybe the problem?)
installed diagrams.net plugin
created test.adoc file
typed image::test.dio.svg[]
alt+enter to create non existing test.dio.svg
editor opened without problem

what did I do different from your setup?

I created a diagrams.net document online in Chrome and stored it locally. Then installed the plugin and opened it. I am running the EAP version:

It is probably an issue with the 2021.1 EAP versions, because I am encountering the same issue in PyCharm 2021.1 EAP Build #PY-211.6085.15, built on February 18, 2021, however it works fine with the very same files in Pycharm 2020.3.3 Build #PY-203.7148.72, built on January 27, 2021.

EAP could be the reason but would be surprising to me. I'll try to check this out

I updated the IntelliJ Marketplace that the plugin supports 2020.3.* at maximum.

I can confirm that it now works in WebStorm. Thanks all.


[Castanets] Training Version 3 (Test)

Process Description
Detailed description of the process.
Payload

version: 3
issue_id: 3

Castanets process has been started.
Name | MLOps Example
Description | Best practice using Castanets in MLOps Field.

Click, clack. We are moving forward!
Review stage changed from Start to Data Processing. Workflow was triggered for this stage.

    flowchart LR
        data_processing[Data Processing]:::Running --> training[Training]:::Pending --> Deployment[Deployment]:::Pending

    classDef Done fill:#2da44e,stroke:#fff,color:white
    classDef Running fill:#bf8700,stroke:#fff,color:white
    classDef Pending fill:#888,stroke:#fff,color:white

Stage Description
📣 New version of dataset released.

Name: Iris Dataset
Version: 3

You can check the dataset here: Iris Dataset Link
Required Reviews
At least 1 approvals are needed to move this process forward.
Requested reviewers:

@FYLSunghwan (MUST REVIEW)
@harrydrippin

/approve

Click, clack. We are moving forward!
Review stage changed from Data Processing to Training. Workflow was triggered for this stage.

    flowchart LR
        data_processing[Data Processing]:::Done --> training[Training]:::Running --> Deployment[Deployment]:::Pending

    classDef Done fill:#2da44e,stroke:#fff,color:white
    classDef Running fill:#bf8700,stroke:#fff,color:white
    classDef Pending fill:#888,stroke:#fff,color:white

Stage Description
👩🏫 ** Training with Iris Dataset v3 is triggered!**

Model: Ssimple SVM Classifier Model

You can check the training job here: Azure ML Studio
Required Reviews
At least 2 approvals are needed to move this process forward.
Requested reviewers:

@FYLSunghwan (MUST REVIEW)
@harrydrippin

Azure ML Studio Alert
👩🏫 Training Result
Run Details:
Experiment/Pipeline ID: mlops-example-master_1654494943_c7a26532

Run ID
Parameter
Value

mlops-example-master_1654494943_c7a26532
Kernel type
linear

mlops-example-master_1654494943_c7a26532
Penalty
1.0

mlops-example-master_1654494943_c7a26532
confusion_matrix_normalized
aml://artifactId/ExperimentRun/dcid.mlops-example-master_1654494943_c7a26532/confusion_matrix_normalized_1654495125.png

mlops-example-master_1654494943_c7a26532
Accuracy
0.933

mlops-example-master_1654494943_c7a26532
precision
0.933

mlops-example-master_1654494943_c7a26532
recall
0.933

mlops-example-master_1654494943_c7a26532
f1-score
0.933

mlops-example-master_1654494943_c7a26532
confusion_matrix
aml://artifactId/ExperimentRun/dcid.mlops-example-master_1654494943_c7a26532/confusion_matrix

mlops-example-master_1654494943_c7a26532
confusion_matrix_unnormalized
aml://artifactId/ExperimentRun/dcid.mlops-example-master_1654494943_c7a26532/confusion_matrix_unnormalized_1654495125.png

Model Pushed to:

mlops-example-master:3


chore(CI): Test plugin on RC Gradle version

Codecov Report

Merging #132 into master will decrease coverage by 2.35%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #132      +/-   ##
============================================
- Coverage     89.65%   87.30%   -2.36%
+ Complexity       69       67       -2
============================================
  Files            34       34
  Lines           638      638
  Branches         53       53
============================================
- Hits            572      557      -15
- Misses           41       57      +16
+ Partials         25       24       -1

Impacted Files
Coverage Δ
Complexity Δ

...tarter/modules/plugins/AndroidApplicationPlugin.kt
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-2.00%)

...ect/starter/modules/internal/AndroidPluginUtils.kt
78.37% <0.00%> (-13.52%)
0.00% <0.00%> (ø%)

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 4f4c216...9c4ed17. Read the comment docs.


correcting editUrl to include /blob/main for editing on GitHub

fixes #28

Preview: https://deploy-preview-29--dexidp.netlify.app/docs/openid-connect/

Approve, and can you (once again) PR this upstream to cncf/hugo-netlify-starter as well?

Will do!


BUG: Deactivating non-existant inbound-number returns 500 internal server error

Description
Deactivating non-existant inbound-number returns 500 internal server error.

[ ] Ticket is understood, and QA has been contacted (if the ticket has a QA label).

When using the POST /inbound-number/:inbound-number-id/offroute to deactivate an inbound number (sets the active property to false), the service returns a 500 Internal Server Error, when we know what the problem is and should return a 400 Bad Request status and appropriate error message
Steps to Reproduce

Use Postman to POST /inbound-number/:inbound-number-id/off where :inbound-number-id is a UUID of an inbound-number that does not exist.
Observe the 500 Internal Server Error and corresponsing stacktrace in the logs.

Workaround
Is there something we can do to work around this issue in the meantime?

no

Impact/Urgency

Medium
Expected Behavior

When I try to deactivate a non-existant inbound number
Then I should receive a 400 Bad Request and an appropriate error response
QA Considerations
For QA to populate. Leave blank if QA is not applicable on this ticket.

[ ] Tests added to regression to exercise this route
[ ] Regression passes

Additional Info & Resources

Hey team! Please add your planning poker estimate with Zenhub @cris-oddball @edDocMe360 @EvanParish @k-macmillan @kalbfled

Keeping.


Add recent queries

My thought is that you can add a backward and forward arrow to the queries and 20 (or whatever) recent queries can be stored in localstorage.

Closed by https://github.com/NYCPlanning/postgis-preview/commit/46091dfb0875cea2baf9b8056f5cf4d669676b9c


Update base image used in ko build

The default base image, cgr.dev/chainguard/static, no longer provides all the platforms the Chains controller claims support for.

Changes

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs included if any changes are user facing
[ ] Has Tests included if any functionality added or changed
[ ] Follows the commit message standard
[ ] Meets the Tekton contributor standards (including
functionality, content, code)
[ ] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
Use `ghcr.io/wolfi-dev/static:alpine` as the base image for the controller. This restores the platforms supported by the Chains controller: `linux/amd64`, `linux/arm64`, `linux/ppc64le`, `linux/s390x`.

/test pull-tekton-chains-integration-tests


Server-side: Function type prop’s default function is undefined when rendering server-side

Vue.js version
2.1.10
Reproduction Link
https://github.com/shentao/vue-hackernews-2.0
Steps to reproduce

Run the app

npm run dev

What is Expected?
<my-comp></my-comp> should render to
Default function result: value

What is actually happening?
<my-comp></my-comp> throws error:
Vue warn]: Invalid prop: type check failed for prop "functionProp". Expected Function, got String.
(found in component <my-comp> at /Users/damiandulisz/code/private/vue-hackernews-2.0/src/MyComp.vue)
Default function result: undefined
[Vue warn]: Error when rendering component <my-comp> at /Users/damiandulisz/code/private/vue-hackernews-2.0/src/MyComp.vue:
error during render : /top
TypeError: _vm.functionProp is not a function
    at Proxy.render (__vue_ssr_bundle__:1605:48)
    at VueComponent.Vue._render (/Users/damiandulisz/code/private/vue-hackernews-2.0/node_modules/vue/dist/vue.runtime.common.js:2216:22)
    at renderComponent (/Users/damiandulisz/code/private/vue-hackernews-2.0/node_modules/vue-server-renderer/build.js:6078:25)
    at renderNode (/Users/damiandulisz/code/private/vue-hackernews-2.0/node_modules/vue-server-renderer/build.js:6061:7)
    at next (/Users/damiandulisz/code/private/vue-hackernews-2.0/node_modules/vue-server-renderer/build.js:6197:9)
    at next (/Users/damiandulisz/code/private/vue-hackernews-2.0/node_modules/vue-server-renderer/build.js:6189:7)
    at Object.cachedWrite [as write] (/Users/damiandulisz/code/private/vue-hackernews-2.0/node_modules/vue-server-renderer/build.js:36:9)
    at next (/Users/damiandulisz/code/private/vue-hackernews-2.0/node_modules/vue-server-renderer/build.js:6199:17)
    at cachedWrite (/Users/damiandulisz/code/private/vue-hackernews-2.0/node_modules/vue-server-renderer/build.js:36:9)
    at renderNode (/Users/damiandulisz/code/private/vue-hackernews-2.0/node_modules/vue-server-renderer/build.js:6069:7)

I cannot reproduce it in jsfiddle. https://jsfiddle.net/ko6pwtLu/
Is this ssr only issue?

Ah, the problem is how we render server bundle. Vue-ssr will render bundle in a new context, so two Function constant is different. I need more digging here. But it seems fixable.


Remove html directory and it content(src/html/*)

Related to: https://github.com/ijemmao/igbo_api/issues/186

@ezesundayeze looking good! Could you use the keywords Closes or Fixes instead of Related to so that once this PR is merged that issue gets closed


Generating image using Labels

I have the StyleGAN3 network trained on the labeled dataset.
I want to know how and where to pass a label to generate images while editing?
My labels are in numeric form like 1 and 2.

For generating images with label, we added below lines in edit_synthetic.py file under the function get_random_image:
label = torch.zeros([1, G.c_dim], device=device)
# Label 1
label[:, 1] = 1

label = torch.zeros([1, G.c_dim], device=device)
# Label 2
label[:, 2] = 1

Thank you for your help.


refact(device): refactor log messages and variable names

refact variable names for better readability
add verbose level for repeating log messages

@praveengt A thought on refactoring. Shall we combine the partsUsed and partsFree to a single struct
type Partition struct {
	// Path of the disk, eg: /dev/sda
	DiskPath string
	// partition number, Valid only if Free=false
	// eg: 1,2,3
	PartNum uint32
	// Name of the partition, Vaild if Free=false
	// eg: 12345-4212-123
	Name string
	// Path to partition, eg: loop0p1, sda1, nvme0n1p1
	PartitionPath string
	SizeMiB uint64
	StartMiB uint64
	EndMiB uint64
	// Tells whether this is a partition or free slot. If free = true, PartNum, Name, PartitionPath are invalid
	Free bool
}

@praveengt Pushed the changes.

@praveengt @iyashu Please take a look

Done
On 23-Jul-2021 at 12:29:12 PM, Akhil Mohan @.***> wrote:

@praveengt https://github.com/praveengt @iyashu
https://github.com/iyashu Please take a look
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openebs/device-localpv/pull/26#issuecomment-885440315,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEOK42LN6XBHJYFU3GJCXWTTZEHMBANCNFSM5ALP5PVA
.

--
-----------------------------------------------------------------------------------------
This email and any files transmitted with it are confidential and
intended solely for the use of the individual or entity to whom they are
addressed. If you have received this email in error, please notify the
system manager. This message contains confidential information and is
intended only for the individual named. If you are not the named addressee,
you should not disseminate, distribute or copy this email. Please notify
the sender immediately by email if you have received this email by mistake
and delete this email from your system. If you are not the intended
recipient, you are notified that disclosing, copying, distributing or
taking any action in reliance on the contents of this information is
strictly prohibited.****
****
Any views or opinions presented in this
email are solely those of the author and do not necessarily represent those
of the organization. Any information on shares, debentures or similar
instruments, recommended product pricing, valuations and the like are for
information purposes only. It is not meant to be an instruction or
recommendation, as the case may be, to buy or to sell securities, products,
services nor an offer to buy or sell securities, products or services
unless specifically stated to be so on behalf of the Flipkart group.
Employees of the Flipkart group of companies are expressly required not to
make defamatory statements and not to infringe or authorise any
infringement of copyright or any other legal right by email communications.
Any such communication is contrary to organizational policy and outside the
scope of the employment of the individual concerned. The organization will
not accept any liability in respect of such communication, and the employee
responsible will be personally liable for any damages or other liability
arising.****
****
Our organization accepts no liability for the
content of this email, or for the consequences of any actions taken on the
basis of the information provided, unless that information is
subsequently confirmed in writing. If you are not the intended recipient,
you are notified that disclosing, copying, distributing or taking any
action in reliance on the contents of this information is strictly
prohibited.
-----------------------------------------------------------------------------------------

@praveengt what is the issue you have hit with the parted command.


Check node version at project init

Description :
If the node version is not high enough the error message is : "Cannot download template from repository. Make sure that your connection is ok or that the specified branch exists"
Expected behavior :
The message should explain that the project can't be init because of the node version.

I think it comes from the CLI, I'm moving that issue to the correct repository.


bump client-go v0.26.4

issue https://github.com/containerd/containerd/issues/8941

/retest

Am I understanding this right that there is no functional change for us by bumping this version? We can merge to appease the scanners, I'm just curious if keeping an old version has downstream effects on importers that may use more of these libraries.


DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated

Issue Type

Bug report

Molecule and Ansible details
ansible --version && molecule --version

ansible 2.7.9
  config file = None
  configured module search path = ['/Users/mattia/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/lib/python3.7/site-packages/ansible
  executable location = /usr/local/bin/ansible
  python version = 3.7.2 (default, Mar 18 2019, 16:34:41) [Clang 10.0.0 (clang-1<IP_ADDRESS>)]
molecule, version 2.19.0

Molecule installation method (one of):

pip

Ansible installation method (one of):

brew

Detail any linters or test runners used:
All defaults:
dependency:
  name: galaxy
driver:
  name: docker
lint:
  name: yamllint
platforms:
  - name: instance
    image: centos:7
provisioner:
  name: ansible
  lint:
    name: ansible-lint
scenario:
  name: default
verifier:
  name: testinfra
  lint:
    name: flake8

Actual Behaviour
Running molecule test gives the following warning messages:
    ============================= test session starts ==============================
    platform darwin -- Python 3.7.2, pytest-3.9.3, py-1.8.0, pluggy-0.9.0
    rootdir: /Users/mattia/Projects/ansible-java-role/molecule/default, inifile:
    plugins: testinfra-1.16.0
collected 2 items

    tests/test_default.py ..                                                 [100%]

    =============================== warnings summary ===============================
    Package fixture is deprecated. Use host fixture and get Package module with host.package
    TestinfraBackend fixture is deprecated. Use host fixture and get backend with host.backend

    /usr/local/lib/python3.7/site-packages/jinja2/utils.py:485
      /usr/local/lib/python3.7/site-packages/jinja2/utils.py:485: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
        from collections import MutableMapping

    /usr/local/lib/python3.7/site-packages/jinja2/runtime.py:318
      /usr/local/lib/python3.7/site-packages/jinja2/runtime.py:318: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
        from collections import Mapping

    /usr/local/lib/python3.7/site-packages/ansible/errors/__init__.py:22
      /usr/local/lib/python3.7/site-packages/ansible/errors/__init__.py:22: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
        from collections import Sequence

    /usr/local/lib/python3.7/site-packages/yaml/constructor.py:126
      /usr/local/lib/python3.7/site-packages/yaml/constructor.py:126: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
        if not isinstance(key, collections.Hashable):

    /usr/local/lib/python3.7/site-packages/ansible/utils/unsafe_proxy.py:56
      /usr/local/lib/python3.7/site-packages/ansible/utils/unsafe_proxy.py:56: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
        from collections import Mapping, MutableSequence, Set
      /usr/local/lib/python3.7/site-packages/ansible/utils/unsafe_proxy.py:56: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
        from collections import Mapping, MutableSequence, Set

    /usr/local/lib/python3.7/site-packages/ansible/utils/plugin_docs.py:23
      /usr/local/lib/python3.7/site-packages/ansible/utils/plugin_docs.py:23: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
        from collections import MutableMapping, MutableSet, MutableSequence

    /usr/local/lib/python3.7/site-packages/ansible/utils/listify.py:22
      /usr/local/lib/python3.7/site-packages/ansible/utils/listify.py:22: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
        from collections import Iterable

    unknown file:0
      unknown file:0: RemovedInPytest4Warning: config.warn has been deprecated, use warnings.warn instead
      unknown file:0: RemovedInPytest4Warning: config.warn has been deprecated, use warnings.warn instead

    tests/test_default.py::test_hosts_file[ansible://instance]
      /usr/local/lib/python3.7/site-packages/cerberus/schema.py:3: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
        from collections import (Callable, Hashable, Iterable, Mapping,

    -- Docs: https://docs.pytest.org/en/latest/warnings.html
    ==================== 2 passed, 13 warnings in 12.79 seconds ====================
Verifier completed successfully.
--> Scenario: 'default'
--> Action: 'destroy'

> unknown file:0
>       unknown file:0: RemovedInPytest4Warning: config.warn has been deprecated, use warnings.warn instead
>       unknown file:0: RemovedInPytest4Warning: config.warn has been deprecated, use warnings.warn instead

^ This one is hard to tell, because they're coming out of unknown file. Maybe that's coming from test_default.py, but I don't see anything in the template that directly lines up with those warnings. My assumption is that these are coming out of testinfra.
The rest of these warnings are coming out of libraries, which as far as I can tell are up to date in molecule requirements for 2.19:

jinja2

required: 2.10
latest on pypi: 2.10

ansible

installed: 2.7.9
latest on pypi: 2.7.9

PyYAML

required: 3.13
latest on pypi: 5.1

cerberus

required: 1.2
latest on pypi: 1.2

testinfra

required: 1.16.0
latest on pypi: 2.0.0

PyYAML and testinfra are notable standouts.
For PyYAML, 3.13 was the current release for molecule 2.19's release, and the update to 5.1 is currently in discussion in #1806 and its corresponding PR, #1808.
testinfra has been updated to 1.19.0 for molecule 2.20, which includes a fix about the pytest config.warn vs. warnings.warn deprecation.
We should evaluate upgrading to testinfra 2, but otherwise there is nothing here that I can see for the molecule project to do.

which as far as I can tell are up to date in molecule requirements for 2.19

^ I foolishly wrote that before I found the testinfra commit and saw that testinfra was updated. :)
#1849 tracks upgrading from testinfra 1.19 to 2.0.

Righto! Thanks for clearing this up.
Are we covered for this one with #1849 and #1808?

:fire:


[15pt] Styling and UI/UX

Design the user interface using CSS frameworks or custom styles
Make it responsive (mobile-first)

Close with PR:
https://github.com/smunoz1988/vecs-school-frontend/pull/9
https://github.com/smunoz1988/vecs-school-frontend/pull/10


resolve issue when root middleware aren't used at all

Could you please explain the issue and the case it occurs in?

Could you please explain the issue and the case it occurs in?

When you provide any middleware to the router function, previously it just threw an error, since you're not binding app.use to app, so I changed that, and moved the middleware above the routers cause if they're called afterware you can't do anything.
router({
  middleware: [() => {}] // this errors without this fix.
});

Great, thanks!


Avoiding FPGA - possible simplification

Hello,
very nice project and thank you for sharing!
Did you already try indoor navigation using the IMU array and are do you expect 100 Hz to be enough for a e.g. vacuum cleaner rover?
I was wondering if the FPGA can be eliminated: My idea is to use a raspberry pi pico and interfacing the I2C interfaces of the IMUs using the PIO/DMA. The clock line can be shared, allowing for 27 IMUs (since the pico has 28 gpio lines), if all IMUs are set to the same I2C address, otherwise 54 IMUs. What's your opinion on this?
Best regards
Aaron

So originally I did think about using RP2040 but because of the lack of GPIO (I want to do 32 IMU) and some bad timing (RP2350 released a couple weeks after I got the board back), I end up going with a FPGA, but it is not that expensive though.
Issue with I2C is speed and complexity compare to SPI bus, you are limited to 1Mhz while SPI in this project is running at 20Mhz.
Overall if I have a second revision in the future I'll go with RP2350 + SPI to the IMUs, I think that is the ultimate solution.

Hey, thanks for the great project! I'm interested in porting this to an MCU - do you have any other thoughts? (I'd happily send over a PR, though likely makes sense to keep two separate projects as it's a bit of a redesign...) I'd like to run some slightly more advanced algorithms: interference suppression (to remove resonances from closely mounted IMUs), an outlier filter (to discard any out-of-tolerance IMUs), and a Kalman filter for position estimation (as well as something to calculate and cancel out the earth's precession!). So I'm considering either the RP2350, or something a little bit more powerful where the SPI I/O can run on one core and the more sophisticated algorithms can run on a fixed 100Hz loop on the other core...
As an aside: why did you choose the ICM-42688-P? It has the same stated random walk (2.8mdps/sqrt(Hz)) as the ST LSM6DSV IMU, which is half the price, so I'm curious if you've tested/know it to be better or if there's any other reason!

I believe the reference work uses the ICM ( https://www.youtube.com/watch?v=7fgcxDRti5s ). As for the MCU, I believe the RP2040 would be sufficient (since the RP2350 is not yet available). One GPIO for MOSI, one for SCK, two for CS (16 IMUs + 16 IMUs), 16 for MISO. So the idea is to read out 16 IMUs in parallel, then the next 16. This can be done using PIO and DMA.

The reason at least for me to use these ICM IMUs is because they have a very unique feature - You can send a 30~40Khz clock through the pins and clock its internal ASIC, which guarantees synchronization among all the sensor and MCU and you don't have to deal with traditional FSYNC (Or any external sync signal) that only gives you the time delta between last measurements and your external event and you will need to do linear interpolation.
Sidenote here: So after this project bring-up phase and having more familiar with this IMU I've added it to my camera boards and uses FSYNC function with Vsync output from the camera sensors, so now I do have both experience (CLKIN vs FSYNC). I'm so glad that I've got this IMU and uses the CLKIN in this IMU array project, otherwise it is going to be a pain to do synchronization.
As for the algorithm, I haven't finish writing the report yet but I'm able to replicate reference 6 (Aligning the Forces—Eliminating the Misalignments in IMU Arrays - by John-Olof Nilsson et. al) to calibrate the IMU array and run the calibration with RP2040 @ 240Mhz with 500Hz IMU data output. Yes I did use both CPU cores with one deliciated to IMU + calibration/array fusion processing and the other for WiFi/GPS ... etc, but I'm still surprised with the performance I can get with RP2040. (The calibration is even running in floating point lol).

The thing that you need to keep in mind is synchronization: Not just IMU array itself but with other systems too.
So the pin count may be good enough on paper but having a few more to do clock adjustment will be much better. At least this is what I think right now.
Feel free to just grab the files here and starts your own repo though, git isn't good for HW design anyways lol. Also, if you have any good papers on the filtering stuff please do post some links here : )

Sony is using ST also, see reference 2 for details. (H. Kamata, M. Kimishima, T. Sawada, Y. Suga, H. Takeda, K. Yamashita, and S. Mitani, MEMS Gyro Array Employing Array Signal Processing for Interference and Outlier Suppression, IEEE Inertial 2020, Hiroshima, Japan)

Hello，
I had tried IMU array with 2 imu, but it
don't solve the problem of IMU YAW axis drift. So how did you solve this problem？Can I solve this problem by increase the quantity of IMU?

Hello， I had tried IMU array with 2 imu, but it don't solve the problem of IMU YAW axis drift. So how did you solve this problem？Can I solve this problem by increase the quantity of IMU?

Hey, author of
Purely Inertial Navigation with a Low-Cost MEMS Sensor Array by Lukas Blocher et. al
https://ieeexplore.ieee.org/document/9430468
here. To deal with the yaw drift is not an easy task. You will gain a decrease of noise of sqrt(n) with n IMUs. But since you lack the reference it is hard to deal with the drifting offsets. On x and y you can use gravity as a reference, for the Z-axis this would require an additional sensor. But when choosing a high quality sensor you can keep up the yaw angle over 10min or longer and still stay within a 1deg range of precision.
What are your requirements exactly?
In my experience calibrating offsets, sensitivity and misalignment of all IMUs within the array is crucial.

Hi @will127534, thanks for your reply - seems like there are about 3 threads going on in parallel right now. 😅
All the papers I've read have already been linked in this thread. The production version uses a CY8C6247BZI MCU, it seems (with 8 I2C/SPI peripherals). The linked video has some interesting insights - at 20:06 they imply the IMUs aren't synchronised and the FPGA is presumably handling synchronisation. I assume there's a good reason they've done this, but perhaps it's just a tradeoff of a bit of development time vs being able to double the number of IMUs at the same cost.
On another ST thread someone suggested you could start the IMU measurement by simultaneously toggling the CS pin low. I wonder if this would work, and why they've chosen that MCU - with 16 sensors you could read them in 2 sets of 8, sequentially: https://community.st.com/t5/mems-sensors/how-to-configure-and-synchronize-lsm6dso-to-raspeberry-pi/m-p/189524
That said, linear interpolation isn't massively complex either. I need to do a bit of digging to figure out how easy it is to get reliable timestamps from the data...

@gmaxwell to your issue, even if your gyro is in a fixed location/orientation, the problem is still intractable: yaw drift in the plane perpendicular to gravity is impossible to calculate unless you have an external reference.
This does bring up a more interesting issue generally though: how reliable/consistent is the drift on each gyro? Is it random noise or could you calibrate / bin the IMUs to find the best one?
Looking at the graph from the paper linked above, it looks like some individual IMUs perform better than the calculated average of all 16 (there are a couple of purple lines below the blue average trace at the bottom right of the graph). Rather than going to all the effort of combining sensors etc, could you just instead leave them all on for a day, and select the best one, and discard all other results?

@nippoo I'm pretty much free in terms of where discussion be for my git reops/projects, typically it will be better to start a discussion but most people just asking questions in the issue anyways. Either way works for me, it is not as crazy long as some other issue I've seen on Github.
For the synchronisation I was reading ST's datasheet but I'm very much confused about how the heck do people use it without synchronisation signals, I did not see even the FSYNC functions exists in those IMUs. If I must guess maybe they are using the Data Ready (DRDY) and timestamp it in the FPGA against reference clock, and align all the IMUs with linear interpolation?
I'm not saying that I'm against linear interpolation, it just that if there is a easier way to synchronisation why bother. I think most people looking at this repo probably overestimate the work this project has done, the FPGA code here is merely a data aggregator, while the synchronisation is handled by a relatively niche function in the ICM-42688P that not a lot of people is aware of.
Start the IMU measurement by simultaneously toggling the CS pin won't work because their internal clock is still not aligned among the IMUs, and IMU sampling is based on its internal clock. You might able to start measurements at the same time but as time goes on it will still drift because IMUs can't align/adjust its internal clock just using the CS pin.
Finally, IMU sampling is not a one-shot procedure like how some ADC could be, because the sample have to pass through digital filter which needs a few samples to get started.

By the way I have some analyzer code in the analyzer branch with the data I've collected, I've also added some code/HW on the data processer board.

https://www.willwhang.dev/Project-IMU-Array/

@will127534 Thank you for the dev blog writeup.

@nippoo any updates on the mcu port? I could also contribute


Update marble

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

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


Rewrite net/http.Request.{URL.Path,RequestURI} consistently

@tomwilkie PTAL

LGTM; what broke?

what broke?

Nothing, I need it for logging.


Included a tweet function in campfire adapter to display messages

Hey @qix,
This is actually automatically interpretted by the campfire API and unnecessary, I believe.
If an explicit type is omitted, it will be inferred from the content (e.g., if the message contains new line characters, it will be considered a paste).

http://developer.37signals.com/campfire/messages


Spellflash when procc at Actionbar  (bug?) | bar without function in mythic plus dungeons

Hi there,
I have a question cause I have no addon for this and I don't think that this bug is caused from vanilla ui.
see screenshot:
https://picload.org/image/rlodilia/wowscrnshot_030517_101933.jpg
when I get a procc like gettin mana back from healing tuch (see screenshot) the wrong spells starts to flash at my bar (in this case solar wrath).
Do you know if there is some kind of spellflash integrated in your UI? Otherwise I have to start looking at other addons.
and by the way: as you can see in the middle of the screen there is a grey bar without any function. It always appears in mythic + dungeons... Is there any future purpose for this? Is this a known issue? I can make a separate ticket for that

fixed in 1.1.32

Bug with procc and starsurge: http://de.wowhead.com/spell=78674/
when using starsurge and gettin 2x buffs (Solar Empowerment and Lunar Empowerment http://de.wowhead.com/spell=164545
http://de.wowhead.com/spell=164547
When using Lunar strike with both buffs active the spellflash animation for Solar wrath vanishes altough the  solar empowerment buffs still exists.
When using solar wrath with both buffs active the spellflash animation for solar wrath stays altough there is no buff for solar wrath, just for lunar strike.


Correction for raw affiliation Dept. of Statistics and Actuarial Science, Simon Fraser Univ. 1 , 8888 Univ. Dr., Burnaby, BC V5A 1S6, CA

Correction needed for raw affiliation Dept. of Statistics and Actuarial Science, Simon Fraser Univ. 1 , 8888 Univ. Dr., Burnaby, BC V5A 1S6, CA
raw_affiliation_name: Dept. of Statistics and Actuarial Science, Simon Fraser Univ. 1 , 8888 Univ. Dr., Burnaby, BC V5A 1S6, CA
new_rors: 0213rcc28
previous_rors:
works_examples: W2565786589
contact: 61b1dff83b32c61d64166e5d1d95aa8c:0cb54c44 @ ourresearch.org

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


++ache has wrong default, is identical to ++each

> *(ache $a $b)
[%.y p=%a]
> *(each $a $b)
[%.y p=%a]

Strangely, the fork order is only lost when the mold is computed. Bunting the relevant literal directly gives the expected result:
> *$%({$| p/$b} {$& p/$a})
[%.n p=%b]
> *$%({$& p/$a} {$| p/$b})
[%.y p=%a]

Compare to a trivially-computed mold:
> *|-($%({$| p/$b} {$& p/$a}))
[%.y p=%a]

This may be the offending section
https://github.com/urbit/arvo/blob/master/arvo/hoon.hoon#L7004-L7014
Seems that ++whip:al for tagged forks produces a multiple-if, checking for the cases in order and defaulting to the last case. (Bunting directly produces a twig with ++bunt:al instead)

Yup.  I'd been meaning to look into this...
Sent from my iPhone

On Jul 18, 2016, at 11:18 PM, Max<EMAIL_ADDRESS>wrote:
This may be the offending section
https://github.com/urbit/arvo/blob/master/arvo/hoon.hoon#L7004-L7014
Seems that ++whip:al for tagged forks produces a multiple-if, checking for the cases in order and defaulting to the last case. (Bunting directly produces a twig with ++bunt:al instead)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Clam missing a warning about an unsafe assertion

Hi,
I recently encountered the following bug:
If I execute clam.py  --crab-check=assert test.c  with the following code I see no warning about the assertion:
extern void __VERIFIER_assume(int);
extern void __CRAB_assert(int);
extern int __VERIFIER_nondet_int(void);

int main() {
  const int MAX = __VERIFIER_nondet_int();
  __VERIFIER_assume(MAX <= 1);
  for (int i = MAX - 1; i >= 0; i--) {
  }
  __CRAB_assert(MAX <= 0);
}

However the assertion is unsafe for MAX = 1. I'm working with Clam version b41adfd16fb6e6c74081113f04597e7c5c73f972

Fixed by commit e32aa83


NameObfuscation issue with MC Clients.

Bug Report
Expected Behaviour
I simply wanted to obfuscate my client with this. I wanted to make sure I had NameObfuscation along with it.
Actual Behaviour
This ONLY happens with NameObfuscation enabled. The program reaches an error causing it to stop the obfuscation. I've tried adding unobf versions of minecraft to the classpath to fix it, but I had no luck.
How to reproduce the behaviour
Just take any minecraft client you have and try obfuscating it with NameObfuscation enabled (exempting class files doesn't fix it either.).

obfConfig.txt

Same is the issue with java clients as well


GitHub automation - deployment workflow cancelled because other higher prio workloads, a problem

We can't cancel the entire deployment workflow, as each workflow may deploy something different. For example, a support chart deployment workflow run could get cancelled because a hub deployment pr is merged - this logic can't be aware about this.

https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/configuring-pull-request-merges/managing-a-merge-queue maybe a solution

I looked into the merge queue thing.
It seems to handle a situation when you want to merge for example 10 PRs, and want tests to rerun tests just before merge so they pass against an updated branch. I think however it won't help us with this issue =/

Closing #2652 as a duplicate of this.


Refactor common tasks in the Asible Nagios playbooks

The Ansible playbooks for setting up Linux machines to be monitored by Nagios (here) contain common tasks.
Refactor these playbooks by moving the common tasks into main.yml

FYI @gdams @bblondin.


propose a PEP for TrumpScript?

Wonder if there's interest in defining/creating a PEP for TrumpScript?

Yeah it seems like a good suggestion for Trumpscript.I guess PEP8 would be a very good choice for Trumpscript.
Here is a sample of violation of pep8 in the code
E402:3:1:module level import not at top of file
E501:67:80:line too long (104 > 79 characters)
E501:86:80:line too long (145 > 79 characters)
E265:120:17:block comment should start with '# '
E265:121:17:block comment should start with '# '
E265:122:17:block comment should start with '# '
E265:125:17:block comment should start with '# '
E261:139:38:at least two spaces before inline comment
E262:139:39:inline comment should start with '# '
E501:139:80:line too long (103 > 79 characters)
E203:147:44:whitespace before ':'
E203:154:43:whitespace before ':'
E265:155:13:block comment should start with '# '
E231:160:24:missing whitespace after ','
E501:166:80:line too long (83 > 79 characters)
E501:167:80:line too long (88 > 79 characters)
E501:170:80:line too long (83 > 79 characters)
E501:171:80:line too long (88 > 79 characters)
E265:172:9:block comment should start with '# '
E501:176:80:line too long (80 > 79 characters)
E203:179:39:whitespace before ':'
E265:180:13:block comment should start with '# '
E501:181:80:line too long (99 > 79 characters)
E501:184:80:line too long (98 > 79 characters)
E501:196:80:line too long (97 > 79 characters)
E501:198:80:line too long (80 > 79 characters)
E501:200:80:line too long (98 > 79 characters)
E501:257:80:line too long (88 > 79 characters)
E501:266:80:line too long (90 > 79 characters)
E301:276:5:expected 1 blank line, found 0
E231:280:38:missing whitespace after ','
E231:291:35:missing whitespace after ','
E501:302:80:line too long (98 > 79 characters)
E231:326:32:missing whitespace after ','
E501:330:80:line too long (87 > 79 characters)
E501:340:80:line too long (83 > 79 characters)
E231:369:44:missing whitespace after ','
E301:371:5:expected 1 blank line, found 0
E501:372:80:line too long (87 > 79 characters)
E231:381:47:missing whitespace after ','
E501:381:80:line too long (82 > 79 characters)
E501:387:80:line too long (88 > 79 characters)
E501:398:80:line too long (83 > 79 characters)
E501:408:80:line too long (83 > 79 characters)
W292:409:49:no newline at end of file

See a76f078


Add unit testing foundation to SFML

Description
This PR lays the foundation with some very basic test cases. The goal here isn't to get 100% coverage, but to start somewhere and build unit tests over time. So over time, new features will generate new unit tests and unit tests will help shape features.
See the Unit tests discussion and the SFML Test Strategy thread on the forum.
Tasks

[x] Find a solution for dynamic library not being found for the runtests target
[x] Tested on Linux
[x] Tested on Windows
[x] Tested on macOS
[x] Tested on iOS
[x] Tested on Android

How to test this PR?
Test the tests! As in build the test step and run the tests. Would be especially interesting to hear how it runs on macOS and Linux.

Strangely, on macOS I get:
Undefined symbols for architecture x86_64:
  "sf::String::toAnsiString(std::locale const&) const", referenced from:
      Catch::toString(sf::String const&) in UnitTests.o
ld: symbol(s) not found for architecture x86_64

Given that the function isn't used I can just comment it out, and everything else works fine, but I can't see why it would be undefined, and nothing else. Would be interested to see what happens on linux

Successfully tested on Arch Linux. Btw, if tests were enabled in Buildbot, most platforms would have been tested already automatically.
Run tests
Test project /home/stsch/projects/sfml/build
    Start 1: System-module-tests
1/3 Test #1: System-module-tests ..............   Passed    0.00 sec
    Start 2: Window-module-tests
2/3 Test #2: Window-module-tests ..............   Passed    0.01 sec
    Start 3: Graphics-module-tests
3/3 Test #3: Graphics-module-tests ............   Passed    0.01 sec

100% tests passed, 0 tests failed out of 3

Total Test time (real) =   0.02 sec

Except the wouldn't as buildbot would fail, because it can't find the shared libraries. Check the status of previous commits. 😉

@eXpl0it3r I remember the talk about RPATH etc, and whether Windows and OSX have a similar feature.

You can set the working directory for add_test, if that helps

Works fine for me still, no local changes

You can tick off macOS testing now. Up to you if you want to tick off iOS or not. It’s basically disabled on iOS currently until someone works out a way to run iOS unit tests (and I presume android will have the same problem?)
You can also tick off a solution for dlls, as that’s covered now, although Laurent suggested an alternative solution of building all tests and examples in the same folder as the libs. Personally I would prefer not to mix up the libs and the other guff, and don’t want to  change behaviour on all platforms when only Windows is a problem, which is why I did it the way I did, but I’m not particularly precious so feel free to change it if it’s decided that the other way is better.
I’d say it’s best not to worry too much about iOS and Android tests right now as I’ve no idea how long that might take to set up (and also they aren’t “officially” supported yet) so let’s just make a decision on the dll situation and get this moving

Laurent suggested an alternative solution of building all tests and examples in the same folder as the libs. Personally I would prefer not to mix up the libs and the other guff, and don’t want to change behaviour on all platforms when only Windows is a problem

No, I said to tell CMake to put all binaries into a "bin" folder, as well as libraries in "lib". This would result in:

.dll and .exe in "bin", .a / .lib in "lib" on Windows
executables in "bin" , ".so" in "lib" on Linux

So it's not platform specific, CMake already differentiates correctly files types on all platforms. And I think it's quite common to do it this way (we do it for all our projects at work).

Sorry for the misunderstanding - my preference still stands for the Same reasons, but like I said I’m not precious about it

my preference still stands for the Same reasons

So, what reasons? As I said, there's no mix between libraries and runtime files, and it doesn't change anything in behaviour compared to the current version.

Mixing the "SFML Library" files with examples/tests

Depends what you call "libraries". CMake separates "runtime" files (.dll on Windows, none on Linux) from "library" files (.a/.lib import files on Windows, .so on Linux) from "archive" files (.a/.lib static libraries on Windows, .a static libraries on Linux) -- I'm not talking about MacOS since my knownledge of this OS is too limited.
But in the end, you have all "run-time" files in "bin", and all "link-time" files (libraries) in "lib". That's pretty close to what you get in final packages anyway.

changing the process for all platforms when only windows is the issue

We don't change anything for other OSes. We just set global rules that:

change something on Windows, where there was an issue,
change nothing on Linux, where things were already ok
again, I leave MacOS aside but I guess that similar conclusions can be drawn

So, can you be a litte more specific about what exactly bothers you with this solution?

Not really.
What’s left to get this merged then?

Looks like we still have some issues with DLLs with VS builds

What issues? Now that the DLLs are copied the tests run fine.

You can check the Travis build log. The error code indicates that it's missing a DLL, but we don't really know which one. To be honest it reads like a Travis issue.
I tried to figure out how to run the Travis VS system on my local machine, but couldn't find any documentation on this.
I'll look into it again some time this weekend.

Is buildbot set up to run the tests too?

To be honest it reads like a Travis issue.

Probably. Today I tried a fresh checkout of the this branch, with VS2015 / dynamic, and all built and ran fine.

Looks like it's still a Travis issue. There are two steps I want to get done before merging this:

Disable the tests on the failing Travis instance
Activate tests on our CI system


[TH2-5179] migrated to gradle 8.7 and added toml to share dependency versions between projects

@OptimumCode could you please review again, I have fixed flicker test problem


Exclude traffic to services outside the cluster from App Mesh.

This is to (hopefully) prevent the intermittent TCP failures that we've been seeing occasionally on connections to MongoDB. It's a hard-to-reproduce problem so it makes sense to just exclude all this traffic from App Mesh preemptively, given that there's no benefit to having it go through Envoy. We don't plan to use any of the App Mesh features for this traffic anyway so there's no reason for it to.
See https://trello.com/c/6yn0REmI/368.

On second thoughts we don't want to merge this to main for a bit - makes sense to run this on a workspace for a while and see how it differs from main.

Rather than put time and effort into reproducing this tricky, intermittent problem, let's just exclude all the traffic to external services now and then see if the problem reoccurs. If the App Mesh / Envoy / iptables stuff was the culprit then it won't, and if it does then we know it warrants further investigation.


Does not download extensions when switching to official user installation

🐛 Describe the bug
The original official installation of VSCode was system wide. Since then, Microsoft has released a user-only installation. After uninstalling the system-wide VSCode, and reinstalling the (latest) user-only VSCode, code-settings-sync is not capable of restoring backed up extensions.
🌴 Visual Studio Code Version : 1.29
🌴 Code Settings Sync Version : [ 3.2.0 ]
🌴 Standard or Insiders : [ Standard ]
🌴 OSS or Portable : [ No ]
🌴 Operating System : [ Win 10 1803 ]
🌴 Occurs On: [ Download ]
🌴 Proxy Enabled: [ No ]
🌴 Gist Id: [ 6922d55b455e32e9619195fd8aed0ccc ]
📰 To Reproduce
Steps to reproduce the behavior:

Install older version of VSCode which is a system-wide installation
Install any extension
Back up to gist using sync
Uninstall VSCode, install latest version which is a user-only installation
Restore from gist using sync
Observe "No extensions installed" status message.

💪 Expected behavior
Backed up extensions should be installed.
📺 Console Error Log
  ERR ENOENT: no such file or directory, open 'C:\Users\Tom\AppData\Local\Programs\Microsoft VS Code\extension-output-#6': Error: ENOENT: no such file or directory, open 'C:\Users\Tom\AppData\Local\Programs\Microsoft VS Code\extension-output-#6'
    at Object.fs.openSync (fs.js:646:18)
    at Object.module.(anonymous function) [as openSync] (ELECTRON_ASAR.js:166:20)
    at Object.fs.readFileSync (fs.js:551:33)
    at Object.fs.readFileSync (ELECTRON_ASAR.js:538:29)
    at TrailingSpaces.freezeLastVersion (C:\Users\Tom\.vscode\extensions\shardulm94.trailing-spaces-0.2.11\out\src\trailing-spaces\trailing-spaces.js:266:51)
    at C:\Users\Tom\.vscode\extensions\shardulm94.trailing-spaces-0.2.11\out\src\trailing-spaces\trailing-spaces.js:56:19
    at e.fire (c:\Users\Tom\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:105:344)
    at c:\Users\Tom\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:585:804
    at e.fire (c:\Users\Tom\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:105:344)
    at e.$acceptDocumentsAndEditorsDelta (c:\Users\Tom\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:496:483)
    at t._doInvokeHandler (c:\Users\Tom\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:609:757)
    at t._invokeHandler (c:\Users\Tom\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:609:423)
    at t._receiveRequest (c:\Users\Tom\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:608:15)
    at t._receiveOneMessage (c:\Users\Tom\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:606:957)
    at c:\Users\Tom\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:604:773
    at c:\Users\Tom\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:103:886
    at e.fire (c:\Users\Tom\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:105:344)
    at a (c:\Users\Tom\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:156:881)
    at Socket._socketDataListener (c:\Users\Tom\AppData\Local\Programs\Microsoft VS Code\resources\app\out\vs\workbench\node\extensionHostProcess.js:157:95)
    at emitOne (events.js:116:13)
    at Socket.emit (events.js:211:7)
    at addChunk (_stream_readable.js:263:12)
    at readableAddChunk (_stream_readable.js:250:11)
    at Socket.Readable.push (_stream_readable.js:208:10)
    at Pipe.onread (net.js:594:20)

📺  Additional context
Add any other context about the problem here.

I've seen a slightly different behavior; everything seems to work during the Download of the settings, even the installation of the extension. However the aren't installed, and don't get visible after a restart of Code as well.

The problem is Code team doenst provide API for which we ( authors ) can get the required things.
If you look into the code base, all the paths are Hard Coded.
Why you want to use older version of Code. is there any scenario for which you need older version. Its better to use latest versions.

Hi Shanalikhan,
I have experienced this exact same issue. The plugin was working fine until I switched to the user version of Visual Studio Code. Now it no longer works. I don't think Microsoft is distributing a system wide version of Code any more, so this is an issue that will need to be fixed if the plugin is going to continue offering value to the community.
Thanks

It is fixed now. Let me know if you find some problem.


IE11 support broken

When this addon is included in a Vaadin 14 project, compiled in production mode and browsed with IE11, the following error shows in the browser console:
ES5 bundle cannot be loaded:

SCRIPT5007: No se puede establecer la propiedad 'X2JS' de referencia nula o sin definir
vaadin-bundle.es5-726f6b861ab7362ac53f.cache.js (712,342762)

Removing RssItems from the project fixes the issue, which indicates there is some issue when RssItems is transpiled to ES5.
Indeed RssItems has a dependency with X2JS
https://github.com/FlowingCode/RssItemsAddon/blob/fd2c2a655d028f47953a459ff50234a62038150a/src/main/resources/META-INF/frontend/rss-items.js#L41
which makes this issue similar to x2js/x2js#61
Investigate isomorphic-xml2js as a possible workaround.

Back to triage for reconsideration


Add --x-check flag to wrangler types

What this PR solves / how to test
This PR adds support for type checking within the context of the wrangler types command:
wrangler types --x-check
wrangler types --x-check /path/to/types.d.ts
wrangler types --x-check --x-runtime
wrangler types --x-check --x-runtime=/path/to/runtime.d.ts
wrangler types --x-check /path/to/types.d.ts --x-runtime=/path/to/runtime.d.ts

Building on the wrangler check draft PR, it also does some light refactoring to make the type command handler cleaner and more readable by extracting the Env type generation to its own file.
Author has addressed the following

Tests

[ ] TODO (before merge)
[x] Included
[ ] Not necessary because:

E2E Tests CI Job required? (Use "e2e" label or ask maintainer to run separately)

[ ] I don't know
[x] Required / Maybe required
[ ] Not required because:

Changeset (Changeset guidelines)

[ ] TODO (before merge)
[ ] Included
[ ] Not necessary because:

Public documentation

[ ] TODO (before merge)
[x] Cloudflare docs PR(s): https://github.com/cloudflare/cloudflare-docs/pull/15750
[ ] Not necessary because:

Closing this as the work is no longer planned.