Basic Authentication Failure

I have a new dedicated server I am configuring that has ISPConfig and RainLoop installed, no other mods. Squirrelmail is not installed and I setup Rainloop using this guide: https://www.howtoforge.com/installation-and-configuratio-of-rainloop-webmail-client-with-nginx-on-ubuntu-14.04
I then set it up to forward from the nginx to apache using:
VirtualHost *:80>
ProxyPass / http://localhost:8087/ connectiontimeout=5 timeout=30
ProxyPassReverse  /  http://localhost:8087/
ServerName mail.DOMAIN.com
ErrorLog ${APACHE_LOG_DIR}/DOMAIN_error_log
CustomLog ${APACHE_LOG_DIR}/DOMAIN_access_log common
/VirtualHost>
I have the basic admin user setup during Ubuntu installation, and root enabled. I was able to log into the admin interface. The server I have setup is going to be managing DNS and many domains, so I may end up scripting edits to application.ini, but I am confused about the actual user accounts.
If I have:
Domain1.com
Domain2.com
I tried adding Domain1.com to the "Domains" tab, then did useradd<EMAIL_ADDRESS>from a shell prompt. I got an authentication error response.
I tried adding Domain1.com as the "Default Domain" and leaving the extension off when logging in (only using "user1").
I tried setting up the domain to go to localhost as well, but no success. There isn't any errors in the apache logs, what would be my next step?

I discovered that ISPConfig had overwritten the default settings in dovecot to use a mysql table to admin users, once I used ISPConfig to create the user I could log in. I haven't been able to send / receive yet, but that may come soon :)


chore: Adds identifier assertions for xhr and fetch adapters

Partially solves #244
Will work on the other adapters soon.

@jasonmit this looks great! Thanks for taking this up 😄.

Awesome, I'll wrap this up over the weekend!


[SPARK-37328][SQL] OptimizeSkewedJoin should work for complex queries with multiple joins

What changes were proposed in this pull request?

In https://github.com/apache/spark/pull/32816 , we moved the rule OptimizeSkewedJoin from queryStageOptimizerRules to queryStagePreparationRules. This means that the input plan of OptimizeSkewedJoin will be the entire query plan, while before the input plan was a shuffle stage which is usually a small part of the query plan.
However, to simplify the implementation, OptimizeSkewedJoin has a check that it will be skipped if the input plan has more than 2 shuffle stages. This is obviously problematic now, as the input plan is the entire query plan and is very likely to have more than 2 shuffles.
This PR proposes to remove the check from OptimizeSkewedJoin completely, as it's no longer needed.

We can and should optimize all the skewed joins in the query plan.
If it introduces extra shuffles, we can return the original input plan, or accept it if the force-apply config is true.

Why are the changes needed?

Fix a performance regression in the master branch (not released yet)
Does this PR introduce any user-facing change?

no
How was this patch tested?

a new test

cc @zhengruifeng @ulysses-you

Test build #146441 has started for PR 34974 at commit 844077f.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50916/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50916/

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

Test build #146441 has finished for PR 34974 at commit 844077f.

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

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

thanks for the review, merging to master!


Exclude zend_test and dl_test

Extensions zend_test and dl_test are internal to PHP and should be excluded from stubs.

Fixed: https://github.com/phpstan/php-8-stubs/commit/baf8188ea45cf528de62211c5b5efd5675b114d2 + https://github.com/phpstan/php-8-stubs/commit/945d7554e8bc083afb3d13ae4431017aaf786f52


Global option for allowing only admins to add networks or DenyAddNetwork flag

This is often requested/asked feature on #znc but I wasn't bale to see this at GitHub or I am searching with wrong words.

It's worth noting that the PanicBNC team seems to have done plenty of work in this area: https://github.com/PanicBNC-Dev/ZNC so their changes may be able to be reviewed and imported.  Other changes they've made may also be worth adding, but I haven't reveiwed it/them in length.

You already can limit number of networks per user, and it's even the default...
If user reached the limit, they can't add any more networks.


File managers and bedrock access rights

Submit a feature request or bug report

✓ I've read the guidelines for Contributing to Roots Projects
[maybe] This is a feature request
[maybe] This is a bug report
✓ This request isn't a duplicate of an existing issue
✓ I've read the docs and followed them (if applicable)

Replace any X with your information.

What is the current behavior?
When i install any plugin that works with filesystem it will only show the contents of bedrock/web/wp and stick to that directory. I can see lower-level directories, but not the app directory of the project (bedrock/web/app).
What is the expected or desired behavior?
Plugins that work with wordpress files can access only bedrock/web/app and NOT bedrock/web/wp

Bug report
(delete this section if not applicable)
Please provide steps to reproduce, including full log output:

install bedrock
install any filemanager plugin (https://wordpress.org/plugins/file-manager/)
go to the plugin main page

Please describe your local environment:
Bedrock version: 1.8.2
WordPress version: 4.9.1
PHP version: 7.0.12
OS: Windows 10 Pro / Ubuntu Linux 16.04
Where did the bug happen? Development or remote servers?
Both - development (Windows 10 Pro) and remote (Ubuntu Linux 16.04)
Is there a related Discourse thread or were any utilized (please link them)?
Could not find anything.

Other relevant information:
This problem could also affect backup plugins (such as BackUpWordpress). They all are broken due to this problem.

the nice thing about bedrock is that if a plugin isn't functioning properly then there's likely something wrong on the plugin's end. please reach out to the plugin author about them properly supporting wordpress installed in a subdirectory with a custom content directory.


typing-extensions required for older python versions

For Python < 3.7 typing-extensions should be documented as a dependency. Maybe this is something that can be solved via https://www.python.org/dev/peps/pep-0508/

Sounds good.   I will try to make it happen.

Does #767 do it?


Error from server: request to convert CR from an invalid group/version: acme.cert-manager.io/v1alpha2

kubectl get Issuers,ClusterIssuers,Certificates,CertificateRequests,Orders,Challenges --all-namespaces
NAMESPACE   NAME                                                READY   AGE
clusterissuer.cert-manager.io/letsencrypt-prod      True    106m
clusterissuer.cert-manager.io/letsencrypt-staging   True    106m
NAMESPACE   NAME                                                         READY   SECRET                           AGE
dev    certificate.cert-manager.io/web-app-tls          False   web-app-tls          107m
NAMESPACE   NAME                                                                      APPROVED   DENIED   READY   ISSUER             REQUESTOR                                         AGE
dev    certificaterequest.cert-manager.io/web-app-tls-zr2p9          True                False   letsencrypt-prod   system:serviceaccount:cert-manager:cert-manager   107m
NAMESPACE   NAME                                                                         STATE   AGE
dev    order.acme.cert-manager.io/web-app-tls-zr2p9-1807584986                  106m
Error from server: request to convert CR from an invalid group/version: acme.cert-manager.io/v1alpha2
Environment details::

Kubernetes version:1.22.11
cert-manager version: 1.7.3
Install method: e.g. static

change to cert-manager version: 1.7.3 , it work fine

S:ee: https://cert-manager.io/docs/installation/api-compatibility/#alpha--beta-api-versions
In cert-manager v1.7 all alpha and beta API versions prior to v1 were removed.
NB: The Kubernetes deprecation policy notes that API removal introduces an issue with objects stored at the removed versions. To fix this, we wrote a custom tool that users could run once to migrate their resources.

background: https://github.com/cert-manager/cert-manager/issues/4057

Please use the v1 API instead of v1alpha2.
Feel free to re-open if not resolved.


[WIP] Better regex for movie match

CHANGES
Change the default movie regex to use negative check [^\(\[]* instead of .* for group Names. This prevents regex from going over the actual year and hitting something like 1080.
Example movie name:
1. Die Hard (1988) BDRip 1080p H.265 [2xUKR_ENG].mkv
PROPOSED
I would prefer having regexes encapsulated into a separate class so they could be tested with a set of possible cases to make sure that any changes to regexes don't break anything. Thinking about including into this PR.

This looks promising, ill merge it and we'll see if its better! thanks


do not use set of int literal in tiff

The set {8} generates a value with a size of 8192 bytes. Changed to use a simple equals check instead.

Feel free to patch yourself


SliceableDataset

This is a new dataset class that supports partial fetch. With this feature, users can avoid calling heavy functions when they are not needed. For example, in the case of LabeledImageDataset, users can calculate the statistics of labels without loading any images.
class MyDataset(sliceable.GetterDataset):
    def __init__(self):
        super().__init__()
        # add getters
        self.add_getter('img', self.get_image)
        self.add_getter('label', self.get_label)

    # usually this function is slow because it loads images
    def get_image(self, i):
        ...
        return img

    # usually fast
    def get_label(self, i):
        ...
        return label

dataset = MyDataset()
dataset[0]  # returns (img, label) using both get_image and get_label

label_dataset = dataset.slice[:, 'label']
label_dataset[0]  # returns label using only get_label (get_image is not called)

# compute statistics of labels
stats = collections.Counter(label_dataset)

# get a subset with label = 0, 1, 2
indices = [i for i, label in enumerate(label_dataset) if label in {0, 1, 2}]
dataset_012 = dataset.slice[indices]

I think the name of the class implies only subset of the features it supports.
Since it universally handles all functionalities of SubDataset, ConcatenatedDataset, and PickableDataset, how about naming it "UniversalDataset"? :smile:
By the way, I think there are other possible names.

To make connection with https://github.com/chainer/chainer/pull/3343 clear,
I will add an example that does same thing done in the example of #3343 with @Hakuyume 's dataset.
Note that this dataset does not unnecessary load data that are not picked.
>>> from chainer.datasets import TupleDataset
>>> tuple_dataset = TupleDataset([0, 1, 2], [0, 1, 4])
>>> print(tuple_dataset.features[:, 1])
[0, 1, 4]
>>> print(tuple_dataset.features[1:, 1])
[1, 4]

from chainer.datasets import PickableDataset
class MyDataset(PickableDataset):

    def __init__(self):
        super(MyDataset, self).__init__()
        self.keys = ('feat0', 'feat1')
        self.add_getter(self.get_feat0, 'feat0')
        self.add_getter(self.get_feat1, 'feat1')

    def __len__(self):
        return 3

    def get_feat0(self, i):
        return [0, 1, 2][i]

    def get_feat1(self, i):
        return [0, 1, 4][i]

dataset = MyDataset()
print(dataset.pick(keys='feat1')[:])
# [0, 1, 4]
print(dataset.sub(1, None).pick(keys='feat1')[:])
# [1, 4]
print(dataset.pick(keys='feat1').sub(1, None)[:])
# [1, 4]

On transform, the current design is not clear about whether it returns a dataset or the dataset calling it gets changed.
Similarly, concatenate has this ambiguity.
Also, having concatenate as a method is not standard. Usually, it is declared at module level np.concatenate.
How about declaring methods apply_transform and concat_datasets?

@yuyu2172 Thank you. I added a new module chainer.datasets.sliceable and renamed classes.
SliceableDataset: An abstract class that supports slice. Since concatenate and transform become functions (not methods), this class supports only slice.
SliceableGetterDataset: A class to construct a SliceableDataset by getter(s). Users usually use this class.

I added sliceable version of TupleDataset. With this class,  @yuyu2172 's example can be written as follows.
dataset = TupleDataset(('feat0', [0, 1, 2]), ('feat1', [0, 1, 4]))
print(dataset.slice[:, 'feat1'][:])
# [0, 1, 4]
print(dataset.slice[1:, 'feat1'][:])
# [1, 4]

An example of LabeledImageDataset.
Without sliceable datasets,
dataset = LabeledImageDataset('list.txt')

# compute statistics of labels
# images are loaded unnecessarily
stats = collections.Counter(int(dataset[i][1]) for i in range(len(dataset)))

# get a subset with label = 0, 1, 2
include_indices = []
exclude_indices = []
for i in range(len(dataset)):
    # images are loaded unnecessarily
    _, label = dataset[i][1]
    if label in {0, 1, 2}:
        include_indices.append(i)
    else:
        exclude_indices.append(i)
order = include_indices + exclude_indices
dataset_012 = SubDataset(dataset, 0, len(include_indices), order)

With sliceable datasets,
dataset = SliceableLabeledImageDataset('list.txt')

# compute statistics of labels
# no images are loaded
stats = collections.Counter(map(int, dataset.slice[:, 'label']))

# get a subset with label = 0, 1, 2
# no images are loaded
indices = [i for i, label in enumerate(
    dataset.slice[:, 'label']) if label in {0, 1, 2}]
dataset_012 = dataset.slice[indices]

SliceableLabeldImageDataset can be implemented using GetterDataset.
class SliceableLabeledImageDataset(sliceable.GetterDataset):
    def __init__(self, pairs, root='.'):
        super().__init__()

        with open(pairs) as f:
            self._pairs = [l.split() for l in f]
        self._root = root

        self.add_getter('image', self.get_image)
        self.add_getter('label', self.get_label)

    def __len__(self):
        return len(self._pairs)

    def get_image(self, i):
        path, _ = self._pairs[i]
        return read_image(os.path.join(self._root, path))

    def get_label(self, i):
        _, label = self._pairs[i]
        return np.int32(label)

@hvy (or other reviewers) Could you check the concept of sliceable datasets?  I'm planning to use these sliceable datasets in ChainerCV. I'd like to know whether they can be merged in Chainer because I think these datasets are not specific to vision tasks.

I think this class is beneficial to make sub dataset for cross validation easy as well.
For example, when we want to split dataset based on "group" (TimeSeries data etc.) I use GroupKFold, which is not supported by chainer's dataset split functionality.
Now I guess I can write as follows:
from sklearn.model_selection import GroupKFold
for train_idx, valid_idx in GroupKFold(n_splits=5).split(X, y, groups):
    # make subset dataset class easily by `slice` indexer
    train = dataset.slice[train_idx]
    val = dataset.slice[valid_idx]

If chainer team goes the direction that this PR to merge, I may discard the existing PR https://github.com/chainer/chainer/pull/3343 (even though part of functionality is already supported in chainer chemistry ...)
BTW, I have one question & suggestion.
Are you going to introduce the same name dataset class TupleDataset, ConcatenatedDataset and TransformDataset under sliceable module?
I guess sliceable.ConcatenatedDataset and sliceable.TransformDataset __init__ interfaces are same with existing class, then how about to keep slicealbe.TupleDataset __init__ API to be same as original TupleDataset as well?
Concretely, how about allowing key to be int instead of str and automatically increment key value for sliceable.TupleDataset?
So I am suggesting this kind of behavior,
from chainer.datasets.sliceable import TupleDataset

# no need to specify `key` name for dataset, key is automatically assigned to 0, 1, 2...
dataset = TupleDataset([0, 1, 2], [0, 1, 4])
print(dataset.slice[:, 0][:])
# [0, 1, 2]
dataset_more = TupleDataset(dataset, [0, 1, 8])
print(dataset_more.slice[:, [1, 2]][:])
# [(0, 0), (1, 1), (4, 8)]

For the purpose of explicitly naming the key,  chainer.datasets.sliceable.DictDataset can be introduced to expect following behavior
# need to additionally implement this class
from chainer.datasets.sliceable import DictDataset

# `key` can be passed as `kwargs` with DictDataset
x0 = [0, 1, 2]
x1 = [0, 1, 4]
dataset = DictDataset(feat0=x0, feat1=x1)
# this is also same.
# dataset = DictDataset(**{'feat0': x0, 'feat1': x1})
print(dataset.slice[:, 'feat0'][:])
# [0, 1, 2]

I suggested this behavior to follow the existing API so that user can switch to use sliceable.TupleDataset from existing TupleDataset without addtional re-implementation cost, and explicitly naming might be a burden.
What do you think of this behavior?

@corochann Thank you for comments!

Are you going to introduce the same name dataset class TupleDataset, ConcatenatedDataset and TransformDataset under sliceable module?

Yes. These classes are very similar to existing classes, but somehow different from them. Another solution is using a prefix Sliceable***, i.e. SliceableTupleDataset, SliceableConcatenatedDataset, and SliceableTransfromDataset.

I guess sliceable.ConcatenatedDataset and sliceable.TransformDataset init interfaces are same with existing class, then how about to keep slicealbe.TupleDataset init API to be same as original TupleDataset as well?

When sliceable datasets are passed to sliceable.TupleDataset, the API is similar to TupleDataset. Users don't need to specify keys for sliceable datasets because they have their own keys. In this situation, I think the APIs are similar. However, as you pointed out, this name is somehow confusing. Perhaps another name is better. DictDataset is an option but the API is no exactly same.
Please note that the API of sliceable.TransformDataset is not exactly same as that of TransfromDataset. sliceable.TransformDataset requires names (keys) of the transformed data.

@corochann

I suggested this behavior to follow the existing API so that user can switch to use sliceable.TupleDataset from existing TupleDataset without addtional re-implementation cost, and explicitly naming key might be a burden.

I agree with you that explicit naming is sometimes a burden. I updated sliceable datasets to accept both name and index. I also implemented auto naming for TupleDataset.
Now, we can write as follows.
from chainer.datasets.sliceable import TupleDataset

# omit keys
dataset = TupleDataset([0, 1, 2], [0, 1, 4])
# keys are automatically named
print(dataset.keys)  # ('key_0', 'key_1')
print(dataset.slice[:, 0][:])  # [0, 1, 2]

dataset_more = TupleDataset(dataset, [0, 1, 8])
print(dataset_more.keys)  # ('key_0', 'key_1', 'key_2')
print(dataset_more.slice[:, [1, 2]][:])  # [(0, 0), (1, 1), (4, 8)]

# specify the name of keys
named_dataset = TupleDataset(('feat0', [0, 1, 2]), [0, 1, 4])
# the unspecified key is automatically named
print(named_dataset.keys)  # ('feat0', 'key_1')
# slice takes both key and index (or their mixture)
print(named_dataset.slice[:, ['feat0', 1]][:])  # [(0, 0), (1, 1), (2, 4)]

How about this API?

Thank you for your very quick action!
I think the above API is ok.
I hope PR review proceeds!!! @Hakuyume @mitmul

I updated TupleDataset to use None instead of key_***.
from chainer.datasets.sliceable import TupleDataset

# omit keys
dataset = TupleDataset([0, 1, 2], [0, 1, 4])
print(dataset.keys)  # (None, None)
print(dataset.slice[:, 0][:])  # [0, 1, 2]

dataset_more = TupleDataset(dataset, [0, 1, 8])
print(dataset_more.keys)  # (None, None, None)
print(dataset_more.slice[:, [1, 2]][:])  # [(0, 0), (1, 1), (4, 8)]

# specify the name of a key
named_dataset = TupleDataset(('feat0', [0, 1, 2]), [0, 1, 4])
print(named_dataset.keys)  # ('feat0', None)
# slice takes both key and index (or their mixture)
print(named_dataset.slice[:, ['feat0', 1]][:])  # [(0, 0), (1, 1), (2, 4)]

I also updated TransformDataset and GetterDataset to support the number of keys. Now, users can use all sliceable datasets without explicit naming.

@Crissman Any advice on names of these new datasets? All are enough intuitive?

Jenkins, test this please.

@Hakuyume Please add those dataset classes to the documentation. And please also write a brief explanation (i.e., motivation) of the sliceable module and its usage in docs/source/reference/datasets.rst. Then, please add example code to the docstrings of those dataset classes. Many of the above discussion can be used in the example code, I suppose.

@Hakuyume Sorry, let us consider to provide these functions in another way. So please ignore my above request temporarily.

It seems that the Chainer team will consider another way to provide this functionality.


Fix typos

Checklist

[ ] Updated CHANGELOG.md describing pertinent changes.
[ ] Updated README.md with pertinent info (may not always apply).
[ ] Updated site content with pertinent info (may not always apply).
[x] Squash down commits to one or two logical commits which clearly describe the work you've done. If you don't, then Dodd will 🤓.

:partying_face: :partying_face: :partying_face: :partying_face: :partying_face:


Creating windows credentials times out

Hi again,
I have tried to change RDP  password from RDCMan, and it throws an error after a while with a "timeout".
However, it looks like in the serial logs that the operation was successful:
11/13/2019 9:27:51 AM UTC: Creating a new user account for xxx.
11/13/2019 9:27:51 AM UTC: Adding user...
11/13/2019 9:27:51 AM UTC: User accounts updated.
11/13/2019 9:29:23 AM UTC: Creating a new user account for xxx.
11/13/2019 9:29:23 AM UTC: Username already exists. Changing password...
11/13/2019 9:29:23 AM UTC: User accounts updated.
Nothing urgent, just to let you know.
-Johan

Hi Johan,
a few questions:

are you seeing the error message Timeout waiting for password reset or a different error message?
does the error occur every time or only intermittently?
does viewing the serial port (Show serial port output in context menu) work for you?

Johannes

Hi Johannes,
Yes I see that timout message.
So far it has happened every time, but only the first time it was logged in the serial log, the other attempts appear to have been unsuccessful. Connecting to the server works fine but also the serial log although it displays it does seem to be old, possibly a cached version (?), not sure how that works.
Johan

I am having trouble reproducing this issue or figuring out what might be causing this. A few more questions:

Is this only happening with a single VM or multiple VMs?
Which IAM role(s) does the user have on the GCP project
When you open cmd on a VM and run echo test>COM1, does that show up in the serial port output window?

Hi have tried again with a server where I hadn't tried before:
In the console it looks like this:
11/29/2019 1:29:46 PM UTC: Creating a new user account for xxx.
11/29/2019 1:29:46 PM UTC: Username already exists. Changing password...
11/29/2019 1:29:46 PM UTC: User accounts updated.
Is it possible it has anything to do with me changing username (it proposes my windows account name)?
Another possibility is that this is agian based on a pretty old image. But I have tried also with a brand new n2 image, and I got the same issue. This was in version 1.1.22. Not sure if I could build the plugin and hook up the debugger? I downloaded the source code but I haven't seen how to build/publish.

and yes, the echo test thing works fine

and the user is an editor of the project (or actually, the owner)

Would you mind testing if https://github.com/GoogleCloudPlatform/iap-windows-rdc-plugin/releases/tag/1.1.32 fixes the issue? Thanks.

sorry after installing this the plugin is no longer loaded in RDCMan...

ok yeah I have installed 1.1.32 but there is the same issue. The password is changed, but the response never gets to the client so it times out.
The serial log looks good:
12/12/2019 10:51:45 AM UTC: Creating a new user account for xxx.
12/12/2019 10:51:45 AM UTC: Adding user...
12/12/2019 10:51:45 AM UTC: User accounts updated.

Thank you for verifying. I am sorry that fix was not sufficient to fix the error.
I have a few more questions:

When you look at serial port 4 of the respective VM in the Cloud Console, can you see that a line like {"encryptedPassword":"EmXznt+5flyLqxgt9LSdGr ...  is printed after generating a new password? Somehow the plugin fails to read that output.
Do you have the Cloud SDK installed on the same machine? If so, does gcloud compute reset-windows-password [instance] work reliably?
Do you use a proxy or anything that could have an impact?

In Cloud SDK on the same machine it looks like this
Resetting and retrieving password for [x] on [y]
No zone specified. Using zone [asia-south1-a] for instance: [y].
Updated [https://www.googleapis.com/compute/v1/projects/xxxx/zones/asia-south1-a/instances/y].
ip_address: x.x.x.x
password:
username:   x
In the serial log 4 there is the "encrypted password" e.g.
{"encryptedPassword":"EDsdfsdfsdfQ==","passwordFound":true,"userName":"x"}

One more thing that might or might not help (I have never used serial logs). If I list the log from the console, I get all the password resets for that server in the log. At the bottom it says
Specify --start=3510 in the next get-serial-port-output invocation to get only the new output starting from here.
Not sure if this is normal.

After some more testing, I realized that the JSON response is sometimes read in two pieces, which causes the operation to time out. It only happened to me in maybe 1% of the cases, so I am not sure if it is really the same issue as you are seeing. I fixed that issue in https://github.com/GoogleCloudPlatform/iap-windows-rdc-plugin/releases/tag/1.1.44.
If the issue persists, you can now also enable tracing:

In the Settings dialog, set Tracing to Verbose
Restart RDCMan
Run DbvView to observe trace output

Note that tracing adds a good amount of overhead to tunneling, so you should only turn that on temporarily.

Ok thanks for trying, I have restarted using the updated version, without success. It looks like it assumes the response is waiting for more bytes to come, but nothing comes.
I have attached the log file.
rdcman updatepassword.LOG

Does this problem still apply to the latest version of IAP Desktop?
I've made several fixes and optimization to the password-reset logic over the course of the past releases, so I wonder whether that might have addressed this issue?

Hi,
Last time I tried it was working fine with completely new images but didn’t work (sometimes, not sure why) when I tried on an old (2-3 years old) image. So for me the problem is less and less of an issue.
Thanks for your support,
Best regards
Johan
From: J. Passing<EMAIL_ADDRESS>Sent: 07 October 2020 13:30
To: GoogleCloudPlatform/iap-desktop<EMAIL_ADDRESS>Cc: Johan Johansson<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [GoogleCloudPlatform/iap-desktop] Creating windows credentials times out (#27)
Does this problem still apply to the latest version of IAP Desktop?
I've made several fixes and optimization to the password-reset logic over the course of the past releases, so I wonder whether that might have addressed this issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/GoogleCloudPlatform/iap-desktop/issues/27#issuecomment-704902089, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AC3MCR5FRTIMZEMUCZDORWLSJRNM5ANCNFSM4JMW6JPA.

Thanks for following up. As it turns out, the fact that you used an old image was an important hint and I was able to reproduce the issue with the (rather ancient) projects/windows-cloud/global/images/windows-server-2012-r2-dc-v20160112 image.


How to use it on vlp16 and generate training datasets？

How to use it on vlp16 and generate training datasets？What parameters need to change?

How to use it on vlp16 and generate training datasets？What parameters need to change?


fix *With functions

This is meant to be a fix for an issue brought up in #466, where changing toQueue to only offer a single Take.End blocks joinWith and zipWith. I had an idea to fix joinWith originally, but it only works for zipWith. A similar fix can't be done for joinWith, since the loop can't observe the elements being taken off the queue (it's up to the user to decide whether they want to pop it off the queue).
Several solutions I can identify:

Revert toQueue.
Introduce another toQueue variant that pushes Take.End forever.
Change the signature of joinWith. What is the intended purpose for this function? The comment doesn't give much detail.

@iravid The signature of joinWith:
def joinWith[E1 >: E, B, C](that: Stream[E1, B], lc: Int = 1, rc: Int = 1)(
    f0: (IO[E1, Option[A]], IO[E1, Option[B]]) => IO[E1, Option[C]]
  ): Stream[E1, C]

f0 accepts two IO values, which are substituted with queue.take from the left and right queues respectively. This gives a caller the ability to choose whether or not an element should be removed from the queues. Not sure whether this is a useful property or not, but my imagination is limited.
As a consequence, the implementation of joinWith has no way to observe the value of those queue.take calls, so it can't determine whether a stream has ended. We could change the definition to pass in IO.now(left) and IO.now(right) (so we can pull left and right out) to "fit" the signature, but we might as well go ahead and redefine the function as:
def joinWith[E1 >: E, B, C](that: Stream[E1, B], lc: Int = 1, rc: Int = 1)(
    f0: (Option[A], Option[B]) => IO[E1, Option[C]]
  ): Stream[E1, C]

How about we delete joinWith? It's not powerful enough to do much beyond the others. If you want really powerful things, you have to convert both sides to queues, anyway.

@RaasAhsan By the way, I know the originally intent was for None to signal end of stream for join.

@RaasAhsan Right, thanks. I think you can still track the queue ending by doing some dirty tricks with a promise - e.g. pass a modified IO value to f0 which will complete a promise if the queue take resulted in an end.
However I don't have a concrete usecase for joinWith in mind currently so I agree with just getting rid of it for now.
Sorry I'm not very responsive, btw - busy times here...


Create put.txt

Allow *.put.edu.pl , currently only *.put.poznan.pl is handled.

@Adiqq Pull request merged. Thank you!


add yum notice to install guide

Signed-off-by: n1zyz<EMAIL_ADDRESS>

@rbuysse ?

@dcmiddle Approving - we should make sure this feature gets into planning!


Try -O1 instead of -O0 for verilator cpp compiles - #329, #331.

O0 seems to cause too many changes to the generated .{h,cpp} files, breaking the test harness.
Once #331 is addressed, this commit can be reverted.

I don't think this is the right way to do this. I've been looking into the chisel-testers RealGCD2 failure and as far as I can tell, I think it's purely coincidental that the test has been working at all.
I looked at the VRealGCD.h generated with O0 vs O1 and here's the important part of the diff
With O1
VL_MODULE(VRealGCD2) {
  public:
    // CELLS
    // Public to allow access to /*verilator_public*/ items;
    // otherwise the application code can consider these internals.

    // PORTS
    // The application code writes and reads these signals to
    // propagate new values into/out from the Verilated model.
    VL_IN8(clock,0,0);
    VL_IN8(reset,0,0);
    VL_OUT8(io_in_ready,0,0);
    VL_IN8(io_in_valid,0,0);
    VL_OUT8(io_out_valid,0,0);
    //char  __VpadToAlign5[1];
    VL_IN16(io_in_bits_a,15,0);
    VL_IN16(io_in_bits_b,15,0);
    VL_OUT16(io_out_bits,15,0);

    // LOCAL SIGNALS
    // Internals; generally not touched by application code
    VL_SIG8(RealGCD2__DOT__p,0,0);
    VL_SIG8(RealGCD2__DOT__T_44,0,0);
    VL_SIG8(RealGCD2__DOT__T_46,0,0);
    //char  __VpadToAlign19[1];
    VL_SIG16(RealGCD2__DOT__x,15,0);
    VL_SIG16(RealGCD2__DOT__y,15,0);
    VL_SIG16(RealGCD2__DOT__ti,15,0);
    VL_SIG16(RealGCD2__DOT__T_39,15,0);
    VL_SIG16(RealGCD2__DOT__GEN_0,15,0);
    VL_SIG16(RealGCD2__DOT__GEN_1,15,0);
    VL_SIG(RealGCD2__DOT__GEN_9,31,0);
    VL_SIG(RealGCD2__DOT__GEN_10,31,0);
    VL_SIG(RealGCD2__DOT__GEN_11,31,0);
    VL_SIG(RealGCD2__DOT__GEN_12,31,0);
    VL_SIG(RealGCD2__DOT__T_49,16,0);

with O0
VL_MODULE(VRealGCD2) {
  public:
    // CELLS
    // Public to allow access to /*verilator_public*/ items;
    // otherwise the application code can consider these internals.
    VRealGCD2_RealGCD2* __PVT__RealGCD2;

    // PORTS
    // The application code writes and reads these signals to
    // propagate new values into/out from the Verilated model.
    VL_IN8(clock,0,0);
    VL_IN8(reset,0,0);
    VL_OUT8(io_in_ready,0,0);
    VL_IN8(io_in_valid,0,0);
    VL_OUT8(io_out_valid,0,0);
    //char  __VpadToAlign5[1];
    VL_IN16(io_in_bits_a,15,0);
    VL_IN16(io_in_bits_b,15,0);
    VL_OUT16(io_out_bits,15,0);

    // LOCAL SIGNALS
    // Internals; generally not touched by application code

See that the LOCAL SIGNALS are missing in O0, but also note the comment "// Internals; generally not touched by application code"
I started poking around Verilator and it seems that if you want internal signals to be public, you're supposed to indicate as much in the Verilog code with /*verilator public*/. See http://www.veripool.org/projects/verilator/wiki/Faq. Grepping through chisel-testers does not show that this is being done so unless I'm missing something we're not actually marking any of the internal signals as public like we're supposed to. Also note that both of the header file snippets above have a section where /*verilator public*/ signals should show up and the signals aren't there.
Verilator does have a --public option (http://www.veripool.org/projects/verilator/wiki/manual-verilator) but it is for "historical debug use only" so I guess we can't easily make all of the internal signals public.

Closed by ucb-bar/chisel-testers#51


Added a MetroFlatToggleButton style like MetroFlatButton

Added a MetroFlatToggleButton style like MetroFlatButton including

Added the Style to MahApps.Metro/Styles/Controls.Buttons.xaml
Added a FlatToggleButton.xaml
Added Samples to samples/MetroDemo/ExampleViews/ButtonsExample.xaml

@Koopakiller Welcome to the contributors list :+1:


Added logic for resending registration for new-user

What does this PR (Pull Request) do?

[x] This PR fixes #5340
Add the ability to resend a user registration from the user screen

Which Traffic Control components are affected by this PR?

Traffic Portal

What is the best way to verify this PR?

Run TP's API test cases. Added 6 new test cases for user section.
If this is a bug fix, what versions of Traffic Control are affected?

master (07554d8)

The following criteria are ALL met by this PR

[x] This PR includes tests
[x] This PR does not includes documentation
[x] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[x] This PR includes any and all required license headers
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Additional Information

you'll also want to add a "Registration Sent" column to all the "users" tables found in common/modules/table...

you'll also want to add a "Registration Sent" column to all the "users" tables found in common/modules/table...


How to increase article sidebar, index?

Sorry my English is very bad !
How to add a side index bar in the article,  toc
this    https://github.com/ghiculescu/jekyll-table-of-contents
thank !!

Well, it is explained on the link. If you want to customize the theme it is up to you.
Basic Usage
The script requires jQuery. First, reference toc.js in templates where you would like to add the table of content. Then, create an HTML element wherever you want your table of contents to appear:

Finally, call the .toc() function when the DOM is ready:

Here I have made an example:
https://sylhare.github.io/Type-on-Strap/2017/12/12/toc.js-for-table-of-content.html
with that, you are all set ;)


Editing previous route tables

The doc mentions that for preexisting routes, to use new route table capabilities, delete any pre-existing routes from Routing section for the hub in Azure portal.  Currently, the preexisting routes in the vWAN are greyed out, so they're un-selectable, un-viewable, and un-deletable.
How, or when, can we view/delete these routes made previously?

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

ID: fac72373-5539-de97-0acd-5f217279536e
Version Independent ID: 58a01edf-3fba-871d-bb7b-5044adf95794
Content: About virtual hub routing - Azure Virtual WAN
Content Source: articles/virtual-wan/about-virtual-hub-routing.md
Service: virtual-wan
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@InfiniteQ2 , Thanks for reaching out. We will investigate and update this thread.

@InfiniteQ2 , Appreciate your patience. Could you please mention which SKU(Basic/Standard) of VWAN was deployed in your environment?

@SubhashVasarapu-MSFT -- Standard

@InfiniteQ2 , Appreciate your patience. We request you to wait till this weekend as there is a portal roll out underway which will let you use previous routes.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Fixed code execution bug with safe_load()

📊 Metadata *
Please enter the direct URL for this bounty on huntr.dev. This is compulsory and will help us process your bounty submission quicker.
Bounty URL:
⚙ Description *
Describe your fix clearly and concisely - imagine you are describing it to a non-technical friend.
💻 Technical Description *
Describe in-depth, the technical implementation of the proposed security fix. Imagine you are describing it to a NASA engineer.
🐛 Proof of Concept (PoC) *
Provide the vulnerability exploit to show the security issue you're fixing.
🔥 Proof of Fix (PoF) *
Replay the vulnerability exploit to show the successful fix and mitigation of the vulnerability.
👍 User Acceptance Testing (UAT)
Run a unit test or a legitimate use case to prove that your fix does not introduce breaking changes.
🔗 Relates to...
Provide the URL of the PR for the disclosure that this fix relates to.

❌ Oops, @Asjidkalam - it looks like you haven't submitted this PR with a valid bounty URL. If this was intentional, please ignore. Otherwise, please close this PR and open a new PR with the following markdown template, and a valid bounty URL included. A valid template and bounty URL ensures that we can reward you for your fix.
If you have any questions, don't hestitate to join our Discord and a member of the team will be happy to help! 👋

❌ Oops, @Asjidkalam - it looks like you haven't submitted this PR with a valid bounty URL. If this was intentional, please ignore. Otherwise, please close this PR and open a new PR with the following markdown template, and a valid bounty URL included. A valid template and bounty URL ensures that we can reward you for your fix.
If you have any questions, don't hestitate to join our Discord and a member of the team will be happy to help! 👋


[WIP]: add vyper compiler support

[ ] check filename .sol || .vy
[ ] compile on condition

Ref: https://github.com/tintinweb/vscode-vyper


parse-hipaa-parse-1 exited with code 1

I'm running docker-compose up in the forked repo on snapcat branch, but am getting the following error. Any help would be grand!
parse-hipaa-parse-1      | Postgres is up - executing command: node index.js
parse-hipaa-parse-1      | warn: DeprecationWarning: The Parse Server option 'encodeParseObjectInCloudFunction' default will change to 'true' in a future version.
parse-hipaa-parse-1      | node:internal/process/promises:288
parse-hipaa-parse-1      |             triggerUncaughtException(err, true /* fromPromise */);
parse-hipaa-parse-1      |             ^
parse-hipaa-parse-1      |
parse-hipaa-parse-1      | [UnhandledPromiseRejection: This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason "fileUpload.fileExtensions must be an array.".] {
parse-hipaa-parse-1      |   code: 'ERR_UNHANDLED_REJECTION'
parse-hipaa-parse-1      | }
parse-hipaa-parse-1      |
parse-hipaa-parse-1      | Node.js v18.17.1
parse-hipaa-parse-1 exited with code 1

It worked when I downloaded the zip file as opposed to cloning the repo

False positive - it works on the main branch, not the SnapCat branch - same error

Thanks for reporting, the SnapCat branch has been fixed here: https://github.com/netreconlab/parse-hipaa/commit/3efc56969a7b3afbbbf8601e2e3fe60b20063237


Send IDictionary<string, object> to server, the server received the dictionary but modify the key: Change the first letter to lower case.

Versions of Server-Side NuGet Packages: 2.1.2
Versions of Client-Side NuGet/NPM Packages: 1.1.0-preview2-35157
Are you using the C# client or the JavaScript client: C# client
The Server you are using (Kestrel/HttpSysServer/IIS/IIS Express/Azure Web App/etc.):
The Operating System on the Server (Windows/Linux/macOS): Windows
The Operating System on the Client (Windows/Linux/macOS): Windows

The hub protocol is json.
Send IDictionary<string, object> to server, the server received the dictionary but modify the key: Change the first letter to lower case.
For example: Key => key
How to keep the key unchanged?

We configure JSON.NET to convert property names to camelCase by default because it is more familiar on the JavaScript side. However, we have documentation on how to disable this in our documentation JSON/MessagePack serialization options
. In fact, the example on that document describes exactly what you are trying to do: preserve the casing of your JSON property names.

I see. Thank you.
But if I set hub protocol = messagepack, the convert property names are not changed by default.
Therefore, If I change hub protocol while using default configurations, I have to care about all my dictionaries' property names... That's inconvenient.

Correct, each serialization library has it's own way of handling property names.

Okay


WIP:add disc format

Change content and motivation

qcow2 disc_format is potentially incompatible with some cloud providers

What does this MR do?

Adds disc_format as an option

What is there to be completed

[ ] testing
[ ] establishing a sensible default.. perhaps it should be left as qcow2 default format.
[ ] the aws server does not have formats other than qcow2

An alternative approach which I sort of like due to the long wait time to dowload the kubenow image is to add the kubenow image to the docker image... At least then it doesn't have to be downloaded on every time kn apply runs the docker container. I started some work on that alternative approach here

@stephenmoloney thanks for opening this PR. I think that the image gets downloaded only the first time, and then there should be a check that skips the download if the image is already present in the tenancy. So, I'd rather keep this current approach, as we should try to keep the Docker image as light as possible. Please let me know when you have something working, so I can review and enable the CI.
@carmat88, @andersla any ideas on this?

If the image gets downloaded every time it might be due to recent breaking developments with security updates for images. @carmat88 do I need to merge anything to make this not happening?

@mcapuccini No, the currently queued PR in the image repo is not meant to fix any breakings. Also we do not have any cron-job running for the time being. Though I've noticed various PRs for Kubenow, so I wonder if there was anything there meant to be a fix?

@carmat88 Yep, exactly. @andersla merged a fix recently that solves the issue of image re-download. I will make a 0.3.2 stable that will sort things out.

I can confirm that the image doesn't keep downloading provided it has been successfully uploaded... sorry if that is a bit misleading.... I think not putting the image inside the docker image is reasonable to keep the docker image small..
But if the server does not accept a qcow2 disc_format, I'm not sure best approach. Perhaps, making a raw image and a qcow2 image available is an option?

closing in favour of this PR instead


Feature/verification expansion

Working on #1454

Note to self: encountering rate limit for quay.io

Coverage decreased (-10.3%) to 60.077% when pulling ce859e34e786990c17e0a05d56b517b27f54858c on feature/verification_expansion into 43619832d9f7dbececba1fb0e6c221c2c8f50549 on develop.

Not sure what's going on with coverage.
Changed files seems to include changes from Feature/divide integration tests (#1465)  which is odd.

can you mention which commits are relevant/not relevant?

@garyluu  the cherry-picked commits are
3402c18 and b53bd7a


Flyway Gradle plugin should not add JAR(s) from a project's source sets for scanning if resolved artifact file does actually not exist.

What version of Flyway are you using?
4.2.0
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin, SBT plugin, ANT tasks)
Gradle 3.5.0
What database are you using (type & version)?
irrelevant to this issue
What operating system are you using?
irrelevant to this issue
What did you do?
In a multi-project Gradle build, consider a sub project, say s_1, that applies the Gradle Flyway plugin. In addition, assume that s_1 defines a project dependency onto another sub project, say s_2. If the locations property contains at least one classpath: reference, the Flyway plugin will add resolved artifacts from the source sets specified by s_2 to the Java-based migrations scanner (besides those of s_1). This is the case even if there are actually no Java-based migrations in any of the source sets defined by s_2. Next, run any of the Flyway Gradle tasks on s_1; e.g.
gradle :s_1:flywayInfo
Note that in this example, it was intended to run flywayInfo just for s_1. As long as flywayInfo does not depend on some task of s_2 (e.g. the build task), no task of s_2 will run. If s_2 is "clean" at this point, no artifacts of source sets specified by s_2 do exist; but flywayInfo will still add them for scanning (since Gradle resolves them at configuration phase).
One possible solution to this behavior is to let :s_1:flywayInfo depend on :s_2:build. That, however, would be a rather poor solution since building s_2 in order to be able to execute :s_1:flywayInfo is not what one wants; in particular, if it is known that s_2 does not contribute any Java-based migrations.
What did you expect to see?
The build of s_1 should complete without errors. Considering the example, it should print information about the database migration state (taking into account migrations found according to how locations have been configured).
What did you see instead?
The build ends by throwing an exception (private informations have been removed/modified):
org.gradle.api.tasks.TaskExecutionException: Execution failed for task ':s_1:flywayInfo'.
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeActions(ExecuteActionsTaskExecuter.java:98)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.execute(ExecuteActionsTaskExecuter.java:68)
        at org.gradle.api.internal.tasks.execution.SkipUpToDateTaskExecuter.execute(SkipUpToDateTaskExecuter.java:62)
        at org.gradle.api.internal.tasks.execution.ValidatingTaskExecuter.execute(ValidatingTaskExecuter.java:58)
        at org.gradle.api.internal.tasks.execution.SkipEmptySourceFilesTaskExecuter.execute(SkipEmptySourceFilesTaskExecuter.java:88)
        at org.gradle.api.internal.tasks.execution.ResolveTaskArtifactStateTaskExecuter.execute(ResolveTaskArtifactStateTaskExecuter.java:46)
        at org.gradle.api.internal.tasks.execution.SkipTaskWithNoActionsExecuter.execute(SkipTaskWithNoActionsExecuter.java:51)
        at org.gradle.api.internal.tasks.execution.SkipOnlyIfTaskExecuter.execute(SkipOnlyIfTaskExecuter.java:54)
        at org.gradle.api.internal.tasks.execution.ExecuteAtMostOnceTaskExecuter.execute(ExecuteAtMostOnceTaskExecuter.java:43)
        at org.gradle.api.internal.tasks.execution.CatchExceptionTaskExecuter.execute(CatchExceptionTaskExecuter.java:34)
        at org.gradle.execution.taskgraph.DefaultTaskGraphExecuter$EventFiringTaskWorker$1.execute(DefaultTaskGraphExecuter.java:236)
        at org.gradle.execution.taskgraph.DefaultTaskGraphExecuter$EventFiringTaskWorker$1.execute(DefaultTaskGraphExecuter.java:228)
        at org.gradle.internal.Transformers$4.transform(Transformers.java:169)
        at org.gradle.internal.progress.DefaultBuildOperationExecutor.run(DefaultBuildOperationExecutor.java:106)
        at org.gradle.internal.progress.DefaultBuildOperationExecutor.run(DefaultBuildOperationExecutor.java:61)
        at org.gradle.execution.taskgraph.DefaultTaskGraphExecuter$EventFiringTaskWorker.execute(DefaultTaskGraphExecuter.java:228)
        at org.gradle.execution.taskgraph.DefaultTaskGraphExecuter$EventFiringTaskWorker.execute(DefaultTaskGraphExecuter.java:215)
        at org.gradle.execution.taskgraph.AbstractTaskPlanExecutor$TaskExecutorWorker.processTask(AbstractTaskPlanExecutor.java:77)
        at org.gradle.execution.taskgraph.AbstractTaskPlanExecutor$TaskExecutorWorker.run(AbstractTaskPlanExecutor.java:58)
        at org.gradle.execution.taskgraph.DefaultTaskPlanExecutor.process(DefaultTaskPlanExecutor.java:32)
        at org.gradle.execution.taskgraph.DefaultTaskGraphExecuter.execute(DefaultTaskGraphExecuter.java:113)
        at org.gradle.execution.SelectedTaskExecutionAction.execute(SelectedTaskExecutionAction.java:37)
        at org.gradle.execution.DefaultBuildExecuter.execute(DefaultBuildExecuter.java:37)
        at org.gradle.execution.DefaultBuildExecuter.access$000(DefaultBuildExecuter.java:23)
        at org.gradle.execution.DefaultBuildExecuter$1.proceed(DefaultBuildExecuter.java:43)
        at org.gradle.execution.DryRunBuildExecutionAction.execute(DryRunBuildExecutionAction.java:32)
        at org.gradle.execution.DefaultBuildExecuter.execute(DefaultBuildExecuter.java:37)
        at org.gradle.execution.DefaultBuildExecuter.execute(DefaultBuildExecuter.java:30)
        at org.gradle.initialization.DefaultGradleLauncher$RunTasksAction.execute(DefaultGradleLauncher.java:230)
        at org.gradle.initialization.DefaultGradleLauncher$RunTasksAction.execute(DefaultGradleLauncher.java:227)
        at org.gradle.internal.Transformers$4.transform(Transformers.java:169)
        at org.gradle.internal.progress.DefaultBuildOperationExecutor.run(DefaultBuildOperationExecutor.java:106)
        at org.gradle.internal.progress.DefaultBuildOperationExecutor.run(DefaultBuildOperationExecutor.java:56)
        at org.gradle.initialization.DefaultGradleLauncher.doBuildStages(DefaultGradleLauncher.java:161)
        at org.gradle.initialization.DefaultGradleLauncher.doBuild(DefaultGradleLauncher.java:112)
        at org.gradle.initialization.DefaultGradleLauncher.run(DefaultGradleLauncher.java:95)
        at org.gradle.launcher.exec.GradleBuildController.run(GradleBuildController.java:66)
        at org.gradle.tooling.internal.provider.ExecuteBuildActionRunner.run(ExecuteBuildActionRunner.java:28)
        at org.gradle.launcher.exec.ChainingBuildActionRunner.run(ChainingBuildActionRunner.java:35)
        at org.gradle.launcher.exec.InProcessBuildActionExecuter.execute(InProcessBuildActionExecuter.java:41)
        at org.gradle.launcher.exec.InProcessBuildActionExecuter.execute(InProcessBuildActionExecuter.java:26)
        at org.gradle.tooling.internal.provider.ContinuousBuildActionExecuter.execute(ContinuousBuildActionExecuter.java:75)
        at org.gradle.tooling.internal.provider.ContinuousBuildActionExecuter.execute(ContinuousBuildActionExecuter.java:49)
        at org.gradle.tooling.internal.provider.ServicesSetupBuildActionExecuter.execute(ServicesSetupBuildActionExecuter.java:49)
        at org.gradle.tooling.internal.provider.ServicesSetupBuildActionExecuter.execute(ServicesSetupBuildActionExecuter.java:31)
        at org.gradle.launcher.daemon.server.exec.ExecuteBuild.doBuild(ExecuteBuild.java:67)
        at org.gradle.launcher.daemon.server.exec.BuildCommandOnly.execute(BuildCommandOnly.java:36)
        at org.gradle.launcher.daemon.server.api.DaemonCommandExecution.proceed(DaemonCommandExecution.java:120)
        at org.gradle.launcher.daemon.server.exec.WatchForDisconnection.execute(WatchForDisconnection.java:37)
        at org.gradle.launcher.daemon.server.api.DaemonCommandExecution.proceed(DaemonCommandExecution.java:120)
        at org.gradle.launcher.daemon.server.exec.ResetDeprecationLogger.execute(ResetDeprecationLogger.java:26)
        at org.gradle.launcher.daemon.server.api.DaemonCommandExecution.proceed(DaemonCommandExecution.java:120)
        at org.gradle.launcher.daemon.server.exec.RequestStopIfSingleUsedDaemon.execute(RequestStopIfSingleUsedDaemon.java:34)
        at org.gradle.launcher.daemon.server.api.DaemonCommandExecution.proceed(DaemonCommandExecution.java:120)
        at org.gradle.launcher.daemon.server.exec.ForwardClientInput$2.call(ForwardClientInput.java:74)
        at org.gradle.launcher.daemon.server.exec.ForwardClientInput$2.call(ForwardClientInput.java:72)
        at org.gradle.util.Swapper.swap(Swapper.java:38)
        at org.gradle.launcher.daemon.server.exec.ForwardClientInput.execute(ForwardClientInput.java:72)
        at org.gradle.launcher.daemon.server.api.DaemonCommandExecution.proceed(DaemonCommandExecution.java:120)
        at org.gradle.launcher.daemon.server.exec.LogAndCheckHealth.execute(LogAndCheckHealth.java:55)
        at org.gradle.launcher.daemon.server.api.DaemonCommandExecution.proceed(DaemonCommandExecution.java:120)
        at org.gradle.launcher.daemon.server.exec.LogToClient.doBuild(LogToClient.java:60)
        at org.gradle.launcher.daemon.server.exec.BuildCommandOnly.execute(BuildCommandOnly.java:36)
        at org.gradle.launcher.daemon.server.api.DaemonCommandExecution.proceed(DaemonCommandExecution.java:120)
        at org.gradle.launcher.daemon.server.exec.EstablishBuildEnvironment.doBuild(EstablishBuildEnvironment.java:72)
        at org.gradle.launcher.daemon.server.exec.BuildCommandOnly.execute(BuildCommandOnly.java:36)
        at org.gradle.launcher.daemon.server.api.DaemonCommandExecution.proceed(DaemonCommandExecution.java:120)
        at org.gradle.launcher.daemon.server.exec.StartBuildOrRespondWithBusy$1.run(StartBuildOrRespondWithBusy.java:50)
        at org.gradle.launcher.daemon.server.DaemonStateCoordinator$1.run(DaemonStateCoordinator.java:297)
        at org.gradle.internal.concurrent.ExecutorPolicy$CatchAndRecordFailures.onExecute(ExecutorPolicy.java:63)
        at org.gradle.internal.concurrent.StoppableExecutorImpl$1.run(StoppableExecutorImpl.java:46)
Caused by: org.flywaydb.core.api.FlywayException: Error occurred while executing flywayInfo
Unable to scan for SQL migrations in location: classpath:a/b/c/datamigration
/.../s_2-3.0.0.jar (No such file or directory)
        at org.flywaydb.gradle.task.AbstractFlywayTask.handleException(AbstractFlywayTask.java:419)
        at org.flywaydb.gradle.task.AbstractFlywayTask.runTask(AbstractFlywayTask.java:321)
        at org.gradle.internal.reflect.JavaMethod.invoke(JavaMethod.java:73)
        at org.gradle.api.internal.project.taskfactory.DefaultTaskClassInfoStore$StandardTaskAction.doExecute(DefaultTaskClassInfoStore.java:141)
        at org.gradle.api.internal.project.taskfactory.DefaultTaskClassInfoStore$StandardTaskAction.execute(DefaultTaskClassInfoStore.java:134)
        at org.gradle.api.internal.project.taskfactory.DefaultTaskClassInfoStore$StandardTaskAction.execute(DefaultTaskClassInfoStore.java:123)
        at org.gradle.api.internal.AbstractTask$TaskActionWrapper.execute(AbstractTask.java:692)
        at org.gradle.api.internal.AbstractTask$TaskActionWrapper.execute(AbstractTask.java:675)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter$1.execute(ExecuteActionsTaskExecuter.java:115)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter$1.execute(ExecuteActionsTaskExecuter.java:109)
        at org.gradle.internal.Transformers$4.transform(Transformers.java:169)
        at org.gradle.internal.progress.DefaultBuildOperationExecutor.run(DefaultBuildOperationExecutor.java:106)
        at org.gradle.internal.progress.DefaultBuildOperationExecutor.run(DefaultBuildOperationExecutor.java:56)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeAction(ExecuteActionsTaskExecuter.java:109)
        at org.gradle.api.internal.tasks.execution.ExecuteActionsTaskExecuter.executeActions(ExecuteActionsTaskExecuter.java:90)
        ... 70 more
Caused by: org.flywaydb.core.api.FlywayException: Unable to scan for SQL migrations in location: classpath:a/b/c/datamigration
        at org.flywaydb.core.internal.util.scanner.Scanner.scanForResources(Scanner.java:59)
        at org.flywaydb.core.internal.resolver.sql.SqlMigrationResolver.scanForMigrations(SqlMigrationResolver.java:105)
        at org.flywaydb.core.internal.resolver.sql.SqlMigrationResolver.resolveMigrations(SqlMigrationResolver.java:96)
        at org.flywaydb.core.internal.resolver.sql.SqlMigrationResolver.resolveMigrations(SqlMigrationResolver.java:48)
        at org.flywaydb.core.internal.resolver.CompositeMigrationResolver.collectMigrations(CompositeMigrationResolver.java:123)
        at org.flywaydb.core.internal.resolver.CompositeMigrationResolver.doFindAvailableMigrations(CompositeMigrationResolver.java:105)
        at org.flywaydb.core.internal.resolver.CompositeMigrationResolver.resolveMigrations(CompositeMigrationResolver.java:91)
        at org.flywaydb.core.internal.resolver.CompositeMigrationResolver.resolveMigrations(CompositeMigrationResolver.java:44)
        at org.flywaydb.core.internal.info.MigrationInfoServiceImpl.refresh(MigrationInfoServiceImpl.java:112)
        at org.flywaydb.core.Flyway$4.execute(Flyway.java:1115)
        at org.flywaydb.core.Flyway$4.execute(Flyway.java:1097)
        at org.flywaydb.core.Flyway.execute(Flyway.java:1464)
        at org.flywaydb.core.Flyway.info(Flyway.java:1097)
        at org.flywaydb.gradle.task.FlywayInfoTask.run(FlywayInfoTask.java:29)
        at org.flywaydb.gradle.task.AbstractFlywayTask.runTask(AbstractFlywayTask.java:319)
        ... 83 more
Caused by: java.io.FileNotFoundException: /.../s_2-3.0.0.jar (No such file or directory)
        at org.flywaydb.core.internal.util.scanner.classpath.ClassPathScanner.findResourceNames(ClassPathScanner.java:198)
        at org.flywaydb.core.internal.util.scanner.classpath.ClassPathScanner.scanForResources(ClassPathScanner.java:87)
        at org.flywaydb.core.internal.util.scanner.Scanner.scanForResources(Scanner.java:57)
        ... 97 more

Actually the more I think about this, the more I feel the current behavior is correct. Flyway has no way of knowing that s_2 will not be contributing Java or SQL migrations, and there should try to scan that.
So yes, adding a build dependency is the right solution here.


Added missing plugins

I just simply dropped all the missing plugin files from the ckeditor repo at
https://github.com/ckeditor/ckeditor-dev/tree/master/plugins
into the ckeditor/static/ckeditor/ckeditor/plugins directory.
I tested this with the embed plugin and that works. I'm not sure how to go about testing for the other plugins.

Oh, and I already did the same ;)


Mutable reference for non refcounted records

Found possible problem with Boxed macro:
gtk_widget_path_iter_get_siblings return *const GtkWidgetPath and your from_glib_full don't accept it (allow only *mut)

Current stage allow to compile gtk with records.
Override the constness will go to other PR as I not sure how it better do.
Other possible problem is unneeded adding use std::mem for records (also not sure how do to fix it)

gtk_widget_path_iter_get_siblings return *const GtkWidgetPath and your from_glib_full don't accept it (allow only *mut)

It might make sense to add a generic implementation for *const T bounded on existing *mut T implementation. This implementation would have to leave from_glib_borrow and from_glib_full unimplemented!() though. (from_glib_borrow currently assumes mutable borrow and transfer: full would not be placed on a const pointer).

Override the constness will go to other PR as I not sure how it better do.

I've been thinking: there already is overriding return types nullability. A generalization of both needs could allow assigning quirks to any parameter or return value of a function, something like
[[object]]
name = "Gtk.TreeModel"
[[object.function]]
name = "get_path"
[object.function.parameter]
name = "iter"
const = true

[[object]]
name = "Gtk.EntryBuffer"
[[object.function]]
name = "get_text"
[object.function.return]
nullable = false

Some clumsiness is a consequence of the chosen config style. An alternative could look like
[Gtk.TreeModel.fn.get_path.parameter.iter]
const = true

[Gtk.EntryBuffer.fn.get_text.return]
nullable = false

Now that this from_glib_borrow deficiency has become apparent I might remove it and take a different approach: make Borrow<T> and BorrowMut<T> structs, that would deref to T and forget it when dropped. This would achieve the same goal of not touching ref/unref or free when borrowing and handing out references to T in such cases.
This
            f(FFIWidget::wrap_widget(this as *mut _), &mut from_glib_borrow(iter),
                TreePath::wrap_pointer(path));

would then change to
            f(FFIWidget::wrap_widget(this as *mut _), &mut BorrowMut::from(iter),
                TreePath::wrap_pointer(path));

Are you sure that last comment belong this PR?
I can not estimate your decision, as not studied the current realization of signals.

Just had to put this somewhere so it wouldn't be a complete surprise and this is relevant to FromGlibPtr impls discussion. :) Callbacks (in particular, signals) are the main use case of the borrow mode, where we get a pointer that outlives the scope.

Ping

Apologies, I haven't had much time lately but still hope to finish the reform this year.
The Borrow idea from above doesn't feel quite right, so I'm not implementing it for now.
@homu r+

:pushpin: Commit a14824c has been approved by gkoz

:zap: Test exempted - status

No problem, I also had no free time, so I don't even tried do my next PRs (about constness overriding and use rework)


[mgmt] nginx release

https://github.com/Azure/sdk-release-request/issues/5012

API change check
APIView has identified API level changes in this PR and created following API reviews.
@azure/arm-nginx

need regen this package after https://github.com/Azure/azure-rest-api-specs/pull/27220 merged into main


Improve/Fix using values in Condition and SetValue

In both SetValue and Condition I think you are supposed to be able to use [Values] to add or compare values.
This seems to not work as expected. Not sure if it's just not implemented.
I will test this further in the future and update this report.

Will work on this now.

Alright, looked at it a bit more. It's not as broken as I thought, but not ideal nontheless
Right now

in the Condition event you have to enter the name of the value definition without brackets

this means if you have a value named "something" you cannot compare to the string "something"

there is no way to use the path in the Condition event

this means if you have multiple value named "something" you cannot specify to which you want to compare

in the SetValue event, using values is just not supported

I think I would like

people to have to specify whether they use a value or a string

this could be by enclosing the value with square brackets or by enabeling a selector with a toggle button

to make specifying a specific value possible

either by making the use of full paths possible or by adding a selector (and thus using the ids instead of the names)

to add similar ability to the SetValue event

Tell me what you think, and I can try to implement my suggestions

Hmm maybe using the full path can work? I'm thinking what it would make more sense for a Godot user to do but not really compromising the novice experience.
Feel free to implement it as you would like it to be 👍🏻


ServiceEnricher should not magically map ports

Description
At the moment the DefaultServiceEnricher maps the first exposed port from the image which is either 8080 or 9090 to the port 80 of the generated service, whereas all other exposed ports are mapped to the same service port. This is dangerous e.g. when in the image configuration the order of the ports changes (the order actually doesn't matter for the docker image), then a different port can be mapped to 80.
Also the selection of 8080 and especially 9090 is a bit random (why not also 8181, 8282, 8000, 8001, ....?
I suggest to get rid of this implicit mapping and allow a mapping in the enricher configuration. This could be
<enricher>
  <config>
    <fmp-service>
       <ports>80=8080,81=9090</ports>
     ....

This could be shortened to <port>80</port> if only one port is exposed.
If there are no objections, I would implement it that way, and would add this mapping to all quickstarts / devops / .. images which currently have used this implicit service mapping (I would scrap in the generated yaml files).

I suggest to touch with the @jstrachan @rawlingsj @jimmidyson and others. I personally love that fmp can be used OOTB with no configuration with basic microservices that has a HTTP service exposed.
Having to add boilerplate XML configuration to the  is a bit downer. Its fine you can override and configure this. But OOTB we should have conventions for 8080 and 8181 for 80/81 IMHO.
Just look at your XML sample above, you need 4 xml tags just to get down to where  are defined. Nobody can really remember that.

Why do we need to map the ports at all by default? Mapping arbitrary ports probably fails the principle of least surprise. Using the same port for service & pod seems most consistent. The port is only important inside the cluster, from outside either ingress, route or node port will be used which are unaffected by the service port.

That's also my point. Why is a service port of 80 so much better than a 8080 ? If you don't map it to a different number its so much easier to understand that an application which exports at 8080 is also reachable at 8080.

@davsclaus Its not the purpose to add the sample configuration above by default, only that you are able to provide a mapping in 1% of the use cases.

Ah yeah it makes okay sense to use the same ports, eg 8080->8080 etc by default. The expose controller will expose it as a nice external url with port 80, wont it @rawlingsj ?

@davsclaus @rhuss the exposecontroller for Ingress creates a rule that maps a hostname to a service name + port so what you propose should work.
I'm not 100% sure that's how it is for OpenShift Routes though.  I have something in the back of my mind that says we had to use a Service port of 80 for Routes to work.  Easily testable to be sure but there was probably a reason the automatic mapping was added in the first place.

No: services for OpenShift routes can be any port. If a service has multiple ports, then you can specify the service port to use, but if it's a single port service then it maps to that port without cofig required.

+1 @jimmidyson -  sometimes automatic port addition to my resources kind of confuses the users. I had a scenario where I need to expose Eureka default port of 8761 but i was not able to enable it via my k8s manifests as the f8-m-p adds 8080 and maps to name http.. though i can do workaround of it .. but i don't want container ports for the ones which I dont define.

Update now to the DefaultServiceEnricher to allow to remove the legacy mapping 8080 / 9090 --> 80.
It's still enabled because I don't know what apps it is going to affect as it is not backwards compatible.
Two new config options has been added:

legacyPortMapping : Enable the old mapping 8080 / 9090 --> 80 (switched on by default but would like to turn if off soon)
port : Service port to use for the first pod port to map, that's the first exposed port from the first configured image for a pod.

If a build relies on the old, implicit mapping, the fix is to add:
<configuration>
   <enricher>
      <config>
          <fmp-service>
              <legacyPortMapping>true</true>
               <!-- Alternatively: -->
               <port>80</port>
          </fmp-service>
      </config>
   </enricher>
</configuration>

as alternative, the property fabric8.enricher.fmp-service.legacyPortMapping=true can be set (that's true btw for every generator / ernicher config, and I plan to make it even more flexible to allow to us enricher.fmp-service.legacyPortMapping, fmp-service.legacyPortMapping and legacyPortMapping, too. at least as sytem properties when given from the outside.
The question is, when should we switch over the mapping ? I suggest to make at least a 3.3.0 release to indicate this change.
@davsclaus @jstrachan @rawlingsj @jimmidyson @kameshsampath Any opinions on this ?

Disabled now the legacy mapping so that the next release should be 3.3.0. That's a good fit as we change the default base image, too (from alpine to centos).
In cases where a mapping would apply the following warning is printed:
 log.warn("Implicit service port mapping to port 80 has been disabled for the used port %d. " +
    "To get back the old behaviour either use set the config port = 80 or use legacyPortMapping = true. " +
    "See https://maven.fabric8.io/#fmp-service** for details.", podPort);

I'm pretty sure that there will be some issue with downstream builds. @rawlingsj if there is anything breaking downstream when 3.3.0 is released (would grep for 'legacyPortMapping'), I'm more than happy to help.

For 4.0 we should remove the legacyPortMapping completely.


Input files

Hi, I‘m a little confused about the input files.
How can I get the inter/intra expect file ? Can I use the HiC-Pro's results which have been  normalized by ICE normalization algorithm ?
Thanks!
Chen

How to make The Intra/Inter expect files for other species?Thanks you!

Hi Chen,
In terms of the expectation files for other species, we don't quite have those yet. What we do to make them is to combine Hi-C matrices from multiple diverse cell types in a given species, so the basic requirement is data from diverse cell types. This may be most straightforward to generate for mouse at this point. If you are interesting in getting the files for mice I can work on generating them. Alternatively if there are other species you are interested in and can point me to what may be a good set of reference (normal) cells to use to make the files I can try to do that as well.
-Jesse

Hi, Jesse.
I have read your work for NG. It's very nice. Then I downloaded your data to repeat your results, and found that the Intra/Inter expect files can't be downloaded from the https://salkinstitute.box.com/s/m8oyv2ypf8o3kcdsybzcmrpg032xnrgx, which can't be opened. Can you update the link? And do you have Intra/Inter expect data for other species, such as Arabidopsis. If you don't have, can you give me some advice about how to create these files or upload the scripts involved in creating Intra/Inter expect files to github? Looking forward to your reply.
Best,
Yu

Hey Dixonlab,
For the input bam files, I would like to make use of bam files generated from bowtie in which every two lines correspond to a mapped pair of reads. Is there a good way to convert this type of file to the desired output of Breakfinder instead of re-running the alignment for my samples with the provided scripts?
Thanks,
Sonali


#244 - add a tenant parameter to each MessageStore method

we should not allow anyone to manipulate the entire MessageStore, especially things that they do not have control over
requiring a tenant for each operation ensures that MessageStore will only operate on that "slice" of the data
for MessageStoreLevel, this is implemented via AbstractLevel.prototype.sublevel

Also I noticed the workflow (rebase whatnot?) that you are using is making the PR reviewers' life more difficult: ie. the very useful feature of just looking at diffs between iterations of code after each review is lost every time you push, which means the reviewer is forced to look over the entire PR again to look for the changes corresponding to the comments being addressed and new changes introduced. Not a big deal for small PRs but definitely a toll for larger PRs.

FYI there should be a "Compare" button next to the commit each time I push that'll show the changes in the push, even if it was a force push


Feature request: Make use of rust conditional compilation [feature]

When using flake-info as a dependency, lots of dependencies are included in the lib that the user might not want. Most notably, uploading to elasticsearch is unnecessary for me and has a lot of dependencies.
Rust has the possibility to use conditional compilation using {features = [...] } in the Cargo.toml (see doc).

I'm aware of conditional compilation.
It should be relatively straight forward to cfg! the eleastic part out of the pipeline.
I'm happy to review if you give a shot at it or look at it myself when i find time for it

Are you mainly interested in using the lib only?

@LoveIsGrief does this solve your issue?

That was faster than I expected!  Indeed, I can now compile without having gcc or openssl-dev installed. The other build errors I can blame on my bad code :yum:  Thank you!
It was necessary to pin it to a commit though since there are no tags and the version number stayed the same. Is that intentional?

This project was never prepared to be released as a library hence we don't have proper versioning for the library in place yet.
Now there seems to be at least some interest, so maybe we have to rethink that.

That would be a nice addition indeed :+1:


Fix typos

@mjul : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@mjul : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Typo fixes look good; thanks.

Typo fixes look good; thanks.


Install SBT on self-hosted gha runners V2

Downloads the sbt tar file to $RUNNER_TEMP/_sbt
Extracts the sbt files to $RUNNER_TOOL_CACHE/sbt/version
Check the tool cache for sbt before bothering to pull it from the cache or download it.
Check the actions/cache if its not in the tool cache.

As the logic here gets more complex, it may be less error prone to write an action Javascript and use (and not reimplement) the tool-cache library: https://docs.github.com/en/actions/sharing-automations/creating-actions/developing-a-third-party-cli-action

I would like to hold off on using JavaScript as much as possible because I don't understand it.

Thank you!

@zarthross I pushed https://github.com/sbt/setup-sbt/releases/tag/v1.1.2. Let us know if it worked in your environment.

Worked perfectly, thanks for your help and feedback!


New App Submission - Krystal Wallet

App Logo (Required)

App Details (Required)
Type: Wallet
Name: Krystal
Description: Krystal is a secure and simple way to access the best DeFi services on a single platform
Homepage Url: https://krystal.app/
Chains supported: Ethereum, Binance Smart Chain, Polygon (Matic), Avalanche
Versions supported:
App Url (Required - at least one)
Browser Url: https://krystal.app/
iOS Url: https://apps.apple.com/vn/app/krystal-one-platform-all-defi/id1558105691
Android Url: https://play.google.com/store/apps/details?id=com.kyrd.krystal
Mac Url:
Windows Url:
Linux Url:
Mobile Linking (Optional)
Deep Link: krystalWallet:
Universal Link:
Desktop Linking (Optional)
Deep Link:
Universal Link:
Metadata (Optional)
Short Name:
Primary Color:
Secondary Color:

Added on commit 7bbe312bc9f997bb58be43b23bb0e0b5cd0c14aa


Is CNT2 regulating or part of cGAS/STING pathway

Review:
CNT2
PMID:34965418

As CNT2 bypasses cGAS, will label this as positive regulation. If more examples of c-di-GMP transport and pathogenic activation of STING come up, might have to redefine cGAS/STING


FEATURE - enable typescript in an existing project

As a developer,
I want to enable typescript support in an existing JS project,
so that I also write modern code.
As a consumer of the the open-ux-tools,
I want to be able check if a project has typescript enabled,
so that I can offer additional functionality
Description
This is a follow up to #621
The ui5-application-writer needs to be extended to offer functionality allowing to check if an existing project has typescript enabled and needs to allow to enable it for existing projects
Technical Design
Typescript is enabled if the ui5.yaml files have the correct middlewares configured, therefore, the ui5-config module needs to be enhanced to allow checking for specific tasks or middlewares and then this functionality needs to be consumed by the ui5-application-writer and an isTypescriptEnabled as well as enableTypescript function needs to be exposed.
Acceptance Criteria
GivenWhenThen format (https://martinfowler.com/bliki/GivenWhenThen.html)
Given I have a valid UI5 project
when I execute isTypescriptEnabled
then I will know whether the project supports typescript coding or not.
Given I have I have UI5 project that doesn't support typescript yet
when I execute enableTypescript
then I will be able to start using typescript in my project
Notes
Tasks

[x] enhance ui5-config
[x] enhance ui5-application-writer
[x] Unit Tests added
[x] Test Cases defined

Implemented with #667


Update surfer.rst

Removed  lucid section

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (91fab54) 76.02% compared to head (9d4e749) 76.02%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #667   +/-   ##
=======================================
  Coverage   76.02%   76.02%
=======================================
  Files          32       32
  Lines        2273     2273
=======================================
  Hits         1728     1728
  Misses        545      545

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Rename CfP to Call for Speakers in organiser view sidebar

For better clarity, instead of using the acronym "CfP" we can instead use "Call for Speakers" directly.
fixes: #9

Checklist

[ ] I have added tests to cover my changes.
[ ] I have updated the documentation.
[x] My change is listed in the doc/changelog.rst.

Didn't find any documentation changes to do. Please check if the changelog is done properly, if so will apply similarly to my other PRs.
Thanks!

Done. Is it okay now? If so I will do the same for the other PR as well. Thanks!

This is not a feature. @Aryan20, moving forward let's not make an entry on the changelog for every single change we made, plz.

This is not a feature. @Aryan20, moving forward let's not make an entry on the changelog for every single change we made, plz.

Sure, I did it only because I was specifically told so in another PR and I did ask through comment on this one.
Also, if that is the case then we should update the issue template as that is contradictory and gives the message that it is required.
Won't do for other PRs.


TST: Calculate coverage

This should be pretty simple by adding pytest-cov to the tests extra, and codecov to CircleCI to upload. Probably a little jiggering in codecov.yml to fix the paths, but plenty of examples in the nipy projects.

Looks like we do?
https://codecov.io/gh/poldracklab/niworkflows

Ups

Not sure why it doesn't get commented to PRs, though.

I believe it should https://docs.codecov.io/docs/pull-request-comments

Agreed.

Pretty sure the reason it's failing is because CircleCI is running under our users, not poldracklab. Commits to master succeed, and outside contributions as well.


Rollup+TypeScript+Jimp: Error: 'default' is not exported by node_modules/jimp/es/index.js, imported by src/index.ts

I am not sure if this issue is in Rollup, a rollup plugin, or Jimp.
So, if this is (for sure) not a Jimp issue, please close this issue and point me to the right direction.
I have also opened an issue over at rollup and will provide a link.
Expected Behavior
Compile and Bundle src/index.ts
Current Behavior
$ yarn rollup
yarn run v1.22.5
$ rollup --config

./src/index.ts → ./dist...
[!] Error: 'default' is not exported by node_modules/jimp/es/index.js, imported by src/index.ts
https://rollupjs.org/guide/en/#error-name-is-not-exported-by-module
src/index.ts (1:7)
1: import Jimp from 'jimp/es';
          ^
2:
3: // open a file called "lenna.png"
Error: 'default' is not exported by node_modules/jimp/es/index.js, imported by src/index.ts
    at error (/home/mogoh/src/github.com/mogoh/rollup-typescript-jimp/node_modules/rollup/dist/shared/rollup.js:5265:30)
    at Module.error (/home/mogoh/src/github.com/mogoh/rollup-typescript-jimp/node_modules/rollup/dist/shared/rollup.js:9835:16)
    at handleMissingExport (/home/mogoh/src/github.com/mogoh/rollup-typescript-jimp/node_modules/rollup/dist/shared/rollup.js:9724:28)
    at Module.traceVariable (/home/mogoh/src/github.com/mogoh/rollup-typescript-jimp/node_modules/rollup/dist/shared/rollup.js:10228:24)
    at ModuleScope.findVariable (/home/mogoh/src/github.com/mogoh/rollup-typescript-jimp/node_modules/rollup/dist/shared/rollup.js:8747:39)
    at MemberExpression.bind (/home/mogoh/src/github.com/mogoh/rollup-typescript-jimp/node_modules/rollup/dist/shared/rollup.js:6451:49)
    at CallExpression$1.bind (/home/mogoh/src/github.com/mogoh/rollup-typescript-jimp/node_modules/rollup/dist/shared/rollup.js:2868:23)
    at CallExpression$1.bind (/home/mogoh/src/github.com/mogoh/rollup-typescript-jimp/node_modules/rollup/dist/shared/rollup.js:6644:15)
    at MemberExpression.bind (/home/mogoh/src/github.com/mogoh/rollup-typescript-jimp/node_modules/rollup/dist/shared/rollup.js:2868:23)
    at MemberExpression.bind (/home/mogoh/src/github.com/mogoh/rollup-typescript-jimp/node_modules/rollup/dist/shared/rollup.js:6469:19)

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Failure Information (for bugs)

Steps to Reproduce

Jimp Version: 0.16.1
Operating System (or Browser): Ubunut 20.10
Node Version (if applicable): 15.3.0

Failure Logs
see above

Here is the issue at rollup: https://github.com/rollup/rollup/issues/3900

Fix module build. If you have more problems feel free to open new issues #1149


Relocate account balance and transactions endpoints to external

Fixes PT-156367049.

It appears it makes some tests fail - I am taking the liberty of marking as WIP.

Thanks for the PR, AK!
It looks good but you should add some changes in the tests, as well. They currently are still expecting to find those APIs on the internal port
please change this to
    Host = external_address(),

Same stands true for the account transactions as well.
Also it looks like you will have to do the same change here and here
You'll also have to move the account transactions test and the balances tests to the external_endpoints group.
You can run tests locally using make test target but this will run all tests. In order to run only the APIs' tests you can use
SUITE=apps/aehttp/test/aehttp_integration make test

@lucafavatella wrt. API usage update: aeternity/protocol#59

Merged too https://github.com/aeternity/protocol/pull/59


User should see an error message when a request to MP did not go through

finishes https://artsyproduct.atlassian.net/browse/PURCHASE-188
This is a simple change for displaying an error message when an network error occurs. I'm not sure what the app should take the user to when the CONTINUE button is pressed, but that could be addressed as a separate task.
I tested this by disabling WiFi and hitting the PLACE BID button. The actual device may behave differently depending on the network condition.
Screenshot:

Fails

:no_entry_sign:

No CHANGELOG added.

  Generated by :no_entry_sign: dangerJS


Use test-all command in the e2e test CI

in .github/workflows/e2e-test.yml a few commands are listed to run scenarios using the test-zeekoe.py script. We've added a test-all command to run different scenarios, we should use this command instead of the separate manual scenarios

Solved in original PR


Uncaught TypeError: n.addEdge is not a function

I have an issue when load data before render()
In the object graph the data is loading and at n node it stops and i have this error
Uncaught TypeError: n.addEdge is not a function

Which version are you using?
Are there Number type of nodes' 'id's in your data? In G6, String type is required for 'id's. So as to 'source's and 'target's for edges.

@Yanyan-Wang I am having the same issue as well, I changed all the id's to string but I am still getting the same error.
Using version 3.3.6

The ID must be unique than source and target.

I fixed the issue by removing the id field I had one the edges. @mkarimv @Yanyan-Wang

Is the id you removed not unique?

@Yanyan-Wang Yes it was, I actually added it back but instead of just returning the integer as a string I appended 'e_' on it for edge, I also do the same for nodes ('n_') and groups ('g_').

Could you pls provide an online demo? With codepen like this: https://codepen.io/Yanyan-Wang/pen/mdbYZvZ , or any other platform you like.

I also meet this problem.
In my case, the id is not string (but a object like special-int),
then the edge cause error.

if no edge, then no error; any one edge will case error again.

The id of a node and an edge MUST be UNIQUE STRINGs.

The id of a node and an edge MUST be UNIQUE STRINGs.

Is it possible to add number type to id？


Session 4 - WES setup and workflow execution

Steps to setup WES and make workflow run requests. Steps include

deploy WES instance (part of docker compose)
Service info endpoint
Run/monitor "hello world" workflows in both Nextflow and WDL
Run/monitor CNest workflow in both Nextflow and WDL

Hi @jb-adams , Where can I find the CNest nextflow and wdl scripts?

@yash-puligundla - this is what I've worked from https://github.com/tf2/CNest

Added my CNest WES Script. See 2/notebooks/SBCGC Running CNest.ipynb


Rule "ibm-required-array-properties-in-response" throws errors for schemas with 'allOf' keyword

We are seeing several errors for this rule, and each error references a schema that already contains the field in the required list.
I'll use the below schema SchedulerJobsResult as an example:
---
additionalProperties: false
allOf:
  - $ref: "../../index.yaml#/components/schemas/CommonViewResult"
  - additionalProperties: false
    properties:
      jobs:
        description: "Array of replication job objects."
        items:
          $ref: "../../index.yaml#/components/schemas/SchedulerJob"
        maxItems:<PHONE_NUMBER>
        minItems: 0
        type: "array"
    type: "object"
required:
  - "jobs"
  - "offset"
  - "total_rows"
description: "Schema for a listing of replication scheduler jobs."

It throws the error:
  Message :   In a response body, an array field MUST NOT be optional
  Rule    :   ibm-required-array-properties-in-response
  Path    :   components.schemas.SchedulerJobsResult.allOf.1.properties.jobs

If I remove the allOf and just have one set of properties, I don't see the error.
This was tested with ibm-openapi-validator version 1.25.0.

Yep, I can see why the rule is complaining about "jobs".  Specifically, the rule is evaluating the second allOf element and finds that "jobs" is optional since - within the context of just that schema - no "required" field is found.

The main "bug" is what I mentioned in my previous comment.  I've opened a PR that relaxes the rule slightly for an allOf element that does not include the "required" field.  This is simply a heuristic for whether or not we should bypass checks within an allOf element, and we'll see if it survives a code review :)

:tada: This issue has been resolved in version 1.23.3 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.25.1 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

YW...  @dpopp07 helped out also by suggesting some improvements to the rule's logic.

@emlaver no problem! Thanks for reporting the issue

:tada: This issue has been resolved in version 1.5.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Better guidance on semantic conventions for database client call span names in case of missing information

What are you trying to achieve?
When tracing an activity from Npgsql (.NET Database provider for PostgreSQL) we want to set a valid span name according to the semantic conventions for database client calls without having to parse the database statement on the client side and without having to guess (or rather infer from our knowledge on the current default behavior of PostgreSQL) information that we don't have when issuing a database client call.
What did you expect to see?
Guidance on creating a valid span name in cases where all three of db.operation, db.name and db.sql.table are not available to the database client.
Per discussion following https://github.com/npgsql/npgsql/issues/4757#issuecomment-1316724458 we think that it is probably pretty safe to infer the database name from our knowledge on PostgreSQL's default behavior but we may still want to research/discuss what good alternatives could be.
Additional context.
https://opentelemetry.io/docs/reference/specification/trace/semantic_conventions/database/
https://github.com/npgsql/npgsql/issues/4757#issuecomment-1316599424

When it’s otherwise impossible to get any meaningful span name, db.name or the tech-specific database name MAY be used.

Looking back at where this language was introduced (https://github.com/open-telemetry/opentelemetry-specification/pull/575), I think "tech-specific database name" meant:

db.cassandra.keyspace
db.hbase.namespace

which were alternatives to db.name until https://github.com/open-telemetry/opentelemetry-specification/pull/1973 removed them and consolidated on db.name.
Recommending falling back to db.system (which is what @jsuereth suggests above) sounds like it could be a good option.


Support for Paths resolve for SASS files inside .vue component

Current behavior:

tsconfig.json
    "paths": {
      "@/*": [
        "src/*"
      ],

Folder /app/shared contains shared.sass etc.
Inside component:
<style lang='sass' scoped>
@use 'sass:math'
@import '@/app/shared/shared'

Error:
Syntax Error: SassError: Can't find stylesheet to import.
  ╷
3 │ @import '@/app/shared/shared'

That tsconfig.json path works correctly for Typescript files. Doesnt work for SASS imports.. tried to solve this via custom configure-webpack.js however it looks resolve / alias is being ignored?

it works when extension is added (doesnt work without extension)


Feat/notification center

Description:
ClickUp links: https://app.clickup.com/t/86byxnvnw
What are the acceptance criteria? What are the steps to test that this code is working?

I have implemented the notification centre component. It shows the revealable questions. If there are questions to be revealed red dot on notification bell image appears.

@boranfurkan looks like a conflict popped up. I'll test this PR after conflict fixed.

@boranfurkan looks like a conflict popped up. I'll test this PR after conflict fixed.

fixing it right now

@boranfurkan looks like a conflict popped up. I'll test this PR after conflict fixed.

Resolved


Circular object values crash  update

Vue.js version
2.2.1
Reproduction Link
https://jsfiddle.net/Tamschi/6q4Ljk0j/
Steps to reproduce
Create a vue-backed <select> element that satisfies the following conditions:

The selected value is an object.
At least one <option> has an object value.
At least one of these instances leads to a circular reference and does not appear only after a value that loosely equals the selected value in the option list.

What is Expected?
The Vue.js app should select the right option and avoid a crash.
What is actually happening?
The app crashes with
Uncaught TypeError: Converting circular structure to JSON
    at JSON.stringify (<anonymous>)
    at looseEqual (vue@2.2.1:225)
    at setSelected (vue@2.2.1:6674)
    at cb (vue@2.2.1:6616)
    at inserted (vue@2.2.1:6618)
    at callHook$1 (vue@2.2.1:5102)
    at callInsert (vue@2.2.1:5045)
    at wrappedHook (vue@2.2.1:1690)
    at Object.invoker [as insert] (vue@2.2.1:1637)
    at invokeInsertHook (vue@2.2.1:4828)

looseEqual	@	vue@2.2.1:225
setSelected	@	vue@2.2.1:6674
cb	@	vue@2.2.1:6616
inserted	@	vue@2.2.1:6618
callHook$1	@	vue@2.2.1:5102
callInsert	@	vue@2.2.1:5045
wrappedHook	@	vue@2.2.1:1690
invoker	@	vue@2.2.1:1637
invokeInsertHook	@	vue@2.2.1:4828
patch	@	vue@2.2.1:4992
Vue._update	@	vue@2.2.1:1994
updateComponent	@	vue@2.2.1:2104
get	@	vue@2.2.1:2417
Watcher	@	vue@2.2.1:2400
mountComponent	@	vue@2.2.1:2114
Vue$3.$mount	@	vue@2.2.1:7153
Vue$3.$mount	@	vue@2.2.1:9154
Vue._init	@	vue@2.2.1:3652
Vue$3	@	vue@2.2.1:3733
window.onload	@	(index):48

in Chrome and
TypeError: cyclic object value 1 vue:225:12
	looseEqual https://unpkg.com/vue:225:12
	setSelected https://unpkg.com/vue:6674:11
	inserted/cb https://unpkg.com/vue:6616:9
	inserted https://unpkg.com/vue:6618:7
	callHook$1 https://unpkg.com/vue:5102:5
	_update/callInsert https://unpkg.com/vue:5045:9
	wrappedHook https://unpkg.com/vue:1690:5
	invoker https://unpkg.com/vue:1637:9
	invokeInsertHook https://unpkg.com/vue:4828:9
	patch https://unpkg.com/vue:4992:5
	lifecycleMixin/Vue.prototype._update https://unpkg.com/vue:1994:16
	mountComponent/updateComponent https://unpkg.com/vue:2104:7
	get https://unpkg.com/vue:2417:13
	Watcher https://unpkg.com/vue:2400:7
	mountComponent https://unpkg.com/vue:2114:17
	Vue$3.prototype.$mount https://unpkg.com/vue:7153:10
	Vue$3.prototype.$mount https://unpkg.com/vue:9154:10
	initMixin/Vue.prototype._init https://unpkg.com/vue:3652:7
	Vue$3 https://unpkg.com/vue:3733:3
	window.onload https://fiddle.jshell.net/Tamschi/6q4Ljk0j/show/:48:1

in Firefox. (I thought I'd include both traces since they contain slightly different information.)

Why do you need a circular reference as the value of an option

@posva The data in question models a game world with interdependent entities. It's easiest to make those relations configurable by binding them directly to option values.
@yyx990803 Thanks for the fast resolution!
At least for now, I'm still going with a custom <strict-select> component since my entities can appear the same to JSON.stringify without being functionally identical. I'm very new to web development though, so I'm not sure how correct my current implementation is.


Shard download via HTTP

As discussed in #2561, please also allow downloading shards via HTTP, not only via HTTPS. All data is hashed and verifiable anyways.

This feature is completed - https://github.com/ripple/rippled/pull/3578


Chef-automate ssh

chef-automate ssh command implementation
:nut_and_bolt: Description: What code changed, and why?
As a part of command mapping, chef-automate ssh was not working as aspected, it was keep waiting for user input,
we have resolved this issue in this PR.

:chains: Related Resources
https://app.zenhub.com/workspaces/the-delta-quadrant-5fc7c4c3922dc10021323fa6/issues/chef/automate/5975
:+1: Definition of Done
Dev Done, Dev Tested
:athletic_shoe: How to Build and Test the Change
rebuild component/automate-cli
:white_check_mark: Checklist
All PRs must tick these:

[x] I have read the CONTRIBUTING document.
[x] All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

[x] Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
[x] Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
[x] Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
[x] Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
[x] Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

[ ] Tests added/updated? (all new code needs new tests)
[ ] Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.
:camera: Screenshots, if applicable

@kalroy @vivek-yadav  Please review this PR, It is waiting in review for long.


feat(meru AB): try assistant retractable drawer

Description
fixes https://github.com/dust-tt/dust/issues/3778
Phone UI is unchanged:

Preview works even with a "blank" new assistant

Convo is cleared & preview is reloaded when assitant changes

It's hard to cover everything in the description, but this should be good enough for v1.
If naming isn't done yet, assistant is called "preview" in the try drawer
Risk

Deploy Plan

Weird scrollbar appearing. Might only be linux; but if it's windows too, we should fix

I notice the autoscroll is working fine :)

Nit: typing in instructions then opening preview => a reload occurs even if first time opening

Thanks @philipperolet ! Trying to fix both these things now