"Acceptable" inputs are difficult to assert on (missing flags?)

Wycheproof defines a tri-state result value, with "acceptable" alongside "valid" and "invalid". It would be nice to programmatically dispatch on these so implementations can test that they accept or reject them as they expect.
It looks like the flags field could be used here, if that was not already the intent. I started looking at using them in BoringSSL, and it's almost what we'd need, but some tests are under-annotated. The model I'm thinking is that our GetWycheproofResult function would take a list of flags we consider acceptable. If any (all?) of a test case's flags fall in that set, we'd map "acceptable" to "valid", otherwise "invalid".
Is this close to what you all had in mind? In that case, these are the "acceptable" inputs that lack flags:

aes_eax_test.json - The "small IV size" and "IV size == 0" tests
aes_gcm_test.json - The "small IV sizes" tests
kw_test.json - All "acceptable" tests
kwp_test.json - Ditto
x25519_test.json - Most tests do indeed have flags now, but the "public key >= p" and "private key == 1 (mod order) on twist" tests are missing flags. Additionally, while most "public key with low order" tests have the "Small public key" annotation, some are flags-less. ("Small public key" is also a different style from the other flag names. "SmallPublicKey", perhaps?)

Would you accept a PR to x25519_test.json which marked any of the ones described as "public key with low order" as "Small public key"? Does the PR just get sent directly to that file?

@bleichen

I'm working on adding more flags.
Since flags are relatively new, not all the generation code has them.
Not sure yet, whether that status of some test vectors should be changed.
I.e., many of the test vectors with weak parameters (e.g. small IV sizes)
currently
have status "acceptable" because some libraries already reject weak
parameters.
It might be possible to ignore the fact that some parameters are weak and
simply
mark such test vectors are "valid" then let the tester decide if a failure
is due to
weak parameters based on the information in the header of the test group.
On Wed, Jul 11, 2018 at 1:36 AM, Thai Duong<EMAIL_ADDRESS>wrote:

@bleichen https://github.com/bleichen
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/wycheproof/issues/59#issuecomment-403998602,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYx_3lQN7Zjf5aR4H81wmxmTAivsiT6Pks5uFTptgaJpZM4VACSq
.

On Wed, Jul 11, 2018 at 7:26 PM, David Benjamin<EMAIL_ADDRESS>wrote:

For BoringSSL, I think we're interested in:

Excluding curves we don't support; the split up files are great,
thanks!

That was the idea. Also the mixed files are becoming too large, so they
will eventually be reduced to test cases not covered in other files.

X25519 should compute the right answer in all cases, but there is a
return value that corresponds to the all zero output. We're fine with the
other "acceptable" inputs. (Per the formulation in RFC7748.)

Not sure I understand this.

We don't allow explicit curve encodings in public keys. These are
forbidden by RFC 5480.

Others do however. So it is necessary to test with such encodings. E.g.
one case was BouncyCastle, where it was possible to encode a public EC key
with
an incorrect order. One of the schemes (ECDHC) would check that the curves
for public and private key are the same by checking that a,b,p,... match
but then
reduce the private key modulo the order passed in with the public key.
Since the main goal is to find bugs that are exploitable, such test vectors
typically contain the shared ECDH result computed over the curve from the
private key.
Getting a different value is the most concerning test result.

Our ASN.1 parsers are generally strict.

Any test case that is not DER encoded is at most "acceptable".
Making all BER encoded inputs invalid seems a bit premature, since there
are still a lot of libraries that are less strict.
Annotations are still weak, so that will be improved.

We supported compressed coordinates.

Compressed and uncompressed coordiantes should probably be mixed in the
same files.
The question whether an implementation is vulnerable to attacks with public
key points not on the curve,
depends what formats the receiver accepts and not what the sender
generates.

We are strict in the DigestInfo encoding in RSASSA-PKCS1-v1_5.

Well, everyone should.

We accept all specified IV lengths of AES-GCM

I hope not IV length 0.

and leave RSA key size limits for the caller to enforce. (The joys
of being a low-level library with existing users... 😢)

Which seems completely reasonable to me. I don't think it is the task of
the library maintainer to enforce new security limits.
Most libraries will have to support existing users with existing keys.
If restrictions are made then the key generation is a good place to start.
E.g., it isn't a problem if 1024 bit RSA keys are supported by a library.
But it is a problem if 1024 bits are the default
(or even any default).

Having weak parameters, at least for things like RSA keys, filtered out by
either looking at the value or just splitting into files makes sense. I
agree that weak things count more as "acceptable" than "valid", but I
expect different libraries to have different cutoffs and whatnot for this,
based on their needs, and so it may not be useful as a
programmatically-checked status code on the test. Though it makes sense as
something that could programmatically go either way; the main reason I'm
interested in programmatically forcing the other "acceptable" inputs one
way or another is the other cases, like bad ASN.1, are quite interesting to
assert on.

Finding good cut-off is indeed a problem. Since the parameter sizes are
also in the headers of the test groups it might be possible to drop this
distinction and just require that the tester
decides whether a failing test vector is due to weak parameters not
supported by the library.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/wycheproof/issues/59#issuecomment-404248803,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYx_3oY6C7o-CyOvLDzMQaK6niKZ3vkrks5uFjVYgaJpZM4VACSq
.

Others do however. So it is necessary to test with such encodings.

Oh, certainly! Sorry, that was probably unclear.

Chiming in late to this issue, but if I can selfishly add my own wish it would be to have a way to distinguish between test cases (for signatures) that are needed to achieve strong unforgeability vs those needed for existential unforgeability.
I am seeing some test failures for lax parsing of ASN.1 DER for RSA signatures in Java (in particular allowing a constructed/concatenated OctetString in the DigestInfo structure). While it would be good for the cryptography provider to fix this, from our point of view we don't need canonical/unique signatures so I don't think this presents an exploitable issue in our scenario. For example, we also support ECDSA signatures that are malleable anyway. (Please correct me if I am misunderstanding this issue!)


Automatic updates?

Hey can someone help with automatic updates? We just need to automate our current update script.

I have fixed the updater @zachthecoder14

This issue will be closed as we have fix it.


feat: allow for custom ids passed in through a config

This PR adds a pretty neat feature: the possibility to include multiple burner connectors at once.
Previously, we would pass in the connector like this:
const wallets = [metaMaskWallet, rainbowkitBurnerWallet];

Now, you can pass in a custom id parameter, allowing you to simultaneously have two burner wallets in the same app. For example:
const wallets = [
  metaMaskWallet.
  rainbowkitBurnerWallet, // backwards compatible!
  rainbowkitBurnerWallet({ id: "burner-1", name: "Burner Wallet #1" }),
  rainbowkitBurnerWallet({ id: "burner-2", name: "Burner Wallet #2", storageKey: "custom-localstorage-private-key" }),
]

The id parameter is used by wagmi to differentiate between different connectors, as well as for storing the private key (by default, it stores the key in localStorage at {id}.pk). The localStorage key can be overridden using the storageKey field.

Just thought of this: should we have something that tracks the registered IDs, to make sure there are no duplicates?


shaking effect working only for the first wrong submission FIXED

Associated Issue: #877
Summary of Changes

change 1:  I set animation: 'none' to restart shake animation

added sound effect to key pressed


Use deploy --out instead of stdout

The new param cds deploy --out <file> allows writing DDL statements to a separate file.
This is needed by CAP Java to replace their stdout redirection (cds deploy --dry > schema.sql). That one fails if debug traces are on as these mess up the output.
So,
cds deploy --to h2 --dry > schema.sql

can change to
cds deploy --to h2 --dry --out schema.sql

Also, one can now request the log independently of the dry mode to get an additional log:
cds deploy --to h2 --out schema.sql

BTW: we have also added a check in cds deploy that will error if the sap/cds version is too old to understand --out (versions < 8.3.0).  This way we detect mismatches between @sap/cds-dk and @sap/cds versions.


Improve nodeSelector formatting and logic if possible.

We added the ability to add a nodeSelector to the restic daemonset in the operator, but the formatting and logic is unintuitive. I will see if we can make improvements to this when time permits.

https://bugzilla.redhat.com/show_bug.cgi?id=1853534

https://bugzilla.redhat.com/show_bug.cgi?id=1853534

Instructions on how to use this parameter have been added, especially since removing selectors is counterintuitive.


Multiprocess issue on windows

On Windows, writing from multiple processes while simultaneously listening for notifications causes something to lock up preventing the Realm from refreshing/delivering notifications.
Here's a simple app that reproduces the issue after 3-4 instances of it are run:
using System;
using System.Linq;
using System.Threading.Tasks;
using Nito.AsyncEx;
using Realms;

namespace SimpleRealmWriter
{
    internal class Program
    {
        private const string RealmLocation = "C:\\Work\\example.realm";

        private static void Main(string[] args)
        {
            AsyncContext.Run(async () =>
            {
                var realm = Realm.GetInstance(RealmLocation);
                var counter = realm.All<MyCounter>().FirstOrDefault();
                if (counter == null)
                {
                    counter = new MyCounter();
                    realm.Write(() => realm.Add(counter));
                }
                counter.PropertyChanged += (s, e) =>
                {
                    Console.WriteLine($"Counter's {e.PropertyName} changed: {counter.Value}");
                };

                while (true)
                {
                    await Task.Delay(1000);
                    var tsr = ThreadSafeReference.Create(counter);
                    await Task.Run(() =>
                    {
                        using var bgRealm = Realm.GetInstance(RealmLocation);
                        var bgCounter = bgRealm.ResolveReference(tsr);
                        bgRealm.Write(() =>
                        {
                            bgCounter.Value++;
                        });
                    });

                    realm.Refresh(); // Locks up here

                    Console.WriteLine($"Counter: {counter.Value}");
                }
            });
        }
    }

    internal class MyCounter : RealmObject
    {
        public int Value { get; set; }
    }
}

@fealebenpae assigning you to do a sanity check of my code and confirm I'm not doing something obviously stupid. I don't think it's a super high priority issue, but just confirming that it's a bug would be a good start.

Hi, guys!
Thanks for looking into this bug. I'm currently stuck at this and it's important for my project that more than one client can work on the same local realm.
From my testing, the thread lock problem happens while more than two clients are trying to write, notify changes and/or quit. I hope they are all related to the same bug.
I'll be more than happy to test for different scenarios.
As a side note, I'm using Realm.dll (Local DB version), the version you get pulling off of Realm.Database NuGet. There is a reason why I'm using the local version instead of the networkable version. And I will really appreciate it if the fixes are made to both versions.
Cheers!

Hey guys, any update on this? To me, locking is a critical bug. Thanks for taking a look.

Hi guys, any progress being made on this issue?
Thanks.

Happy new year guys!
I'm still waiting if this has been fixed in the upcoming release. Please let me know as this is a show stopper for me.
Thanks

Hey @chrisk414, unfortunately we haven't been able to address this issue so far. It's something we've seen happen for a while but we haven't been able to come up with a fix.

Hi, Is there anyone who is actively looking at this? This is easily reproducable for anyone using it and I think it's pretty critical. Thanks.

I'd like to clarify that this is not very high priority because a very small percentage of our users are affected by it. The majority of our users ship Android/iOS apps where the issue doesn't exist. Then, for those that ship windows apps, they'll have to write to the same realm from multiple processes which is not a common scenario. So while I understand it's critical for you and your use case, the unfortunate reality is that we have tasks that will benefit far more people that we'd like to address first.

Right, thanks for making clear after a couple of months of the wait. Perhaps you should make it clear about the warning that it's not for .Net then.  Is that going to be your stance in the future too? Regardless of how many users are using .Net, the severity of this problem is severe and there is no workaround.  I see a similar bug about the deadlock above and I'm not alone.

I already mentioned it's not a super high priority issue in my comment immediately after filing it. As I said, the number of users affected by this issue is very small compared to the number of people using the .NET SDK (as most of them write Xamarin apps and this issue is windows-specific). Our stance will always be that we'll prioritize issues based on the number of people affected or the severity of the issue. If we suddenly get swarmed by people complaining about this problem, we'll obviously bump the priority.

Thanks. I'll wait for it then and I hope it lands before the final 6.0 release. Cheers!

This is a pretty major blocker for our potential uptake of Realm as a replacement for EF-core. The only way to avoid this bug is to not use notification subscription at all, which severely limits how we can interact with realm in an asynchronous way.
Any chance of this getting attention within the near future? Thanks!

I tried running the repro case with the fix in https://github.com/realm/realm-object-store/pull/1142. The bad news are that it doesn't appear to resolve the problem. I distilled a simpler test case and ran that:

Simpler test case
using System;
using System.Linq;
using System.Threading.Tasks;
using Nito.AsyncEx;
using Realms;

namespace SimpleRealmWriter
{
    internal class Program
    {
        private const string RealmLocation = "C:\\Work\\example.realm";

        private static void Main(string[] args)
        {
            AsyncContext.Run(async () =>
            {
                var realm = Realm.GetInstance(RealmLocation);
                var counter = realm.All<MyCounter>().FirstOrDefault();
                if (counter == null)
                {
                    counter = new MyCounter();
                    realm.Write(() => realm.Add(counter));
                }
                counter.PropertyChanged += (s, e) =>
                {
                    Console.WriteLine($"Counter's {e.PropertyName} changed: {counter.Value}");
                };

                while (true)
                {
                    await Task.Delay(1000);
                    realm.Write(() =>
                    {
                        counter.Value++;
                    });

                    //realm.Refresh(); // Locks up here

                    Console.WriteLine($"Counter: {counter.Value}");
                }
            });
        }
    }

    internal class MyCounter : RealmObject
    {
        public int Value { get; set; }
    }
}

Here are some findings:

This test deadlocks with the latest public nuget (10.0.0-beta.2). It doesn't deadlock when using latest OS + https://github.com/realm/realm-object-store/pull/1142.
If I uncomment the realm.Refresh() line, it deadlocks both with the public nuget and the one based on https://github.com/realm/realm-object-store/pull/1142.
When I run multiple instances of the app with the fix in https://github.com/realm/realm-object-store/pull/1142, everything works fine, until I terminating them. There appears to be a condition where closing a running process would cause the other processes to deadlock.

I speculate if you remove the PropertyChanged event handler the dead lock will be gone. In Realm JS it deadlocks on a test that has notification listeners. The test is called testAddListener https://github.com/realm/realm-js/blob/master/tests/js/results-tests.js#L453

Yes, this only happens when we have notifications in combination with multiprocess.

https://github.com/realm/realm-core/issues/4189


C4GT Community: Basic Details of Ansh Garg

Ticket Contents
Description
Please provide the following basic details to help us understand your background and context better.

**Name:Ansh Garg
**GitHub Username:anshgarg030404
**LinkedIn Profile:https://www.linkedin.com/in/ansh-garg-777ab7258/
**College/University:USAR, GGSIPU
If you feel comfortable, you can also provide additional information such as your area of expertise or any specific information you'd like to share.

Goals

[My goal is to become full stack web developer with great command in JavaScript and in Python as well ] Adding your basic details

Implementation Details
To implement the changes in the Community.md file on the GitHub repository (https://github.com/Code4GovTech/C4GT/blob/main/Community.md), follow these steps:

Fork the Repository:

Visit the repository (https://github.com/Code4GovTech/C4GT).
Click the "Fork" button in the upper right corner to create a fork of the repository in your GitHub account.

Clone Your Fork:

Open a terminal or command prompt.
Run the following command to clone your fork to your local machine:

     git clone https://github.com/your-username/C4GT.git

Navigate to the Repository:

Change into the repository directory:

    cd C4GT

Edit the Community.md File:

Open the Community.md file in a text editor. You can use any code editor or a simple text editor.
Locate the existing table and replace the details with your own:

        Name
        Your Name

        GitHub
        https://github.com/your-username

        LinkedIn
        https://www.linkedin.com/in/your-linkedin

        College
        Your College/University

Save Changes:

Save the changes to the Community.md file.

Click on the "New pull request" button.
Ensure the base repository is Code4GovTech/C4GT and the base branch is main.
Ensure the head repository is your fork and the compare branch is main.
Click "Create pull request" and provide a meaningful title and description.
Click "Create pull request" again to submit the pull request.
Your changes will now be submitted as a pull request to the original repository, and the maintainers can review and merge them.

Product Name
C4GT
Organisation Name
C4GT
Tech Skills Needed
Markdown
Mentor(s)
@Shruti3004
Complexity
Beginner
Category
Documentation

When we have to do this, when we solve our first issues or i have to do this now to register me in the community?


Rebase on Postgres v16

Currently PGlite uses a fork or pg v15, we should rebase on v16

@samwillis there seem to be short instructions for rebuilding with the currently supported v15 version. Apart from rebasing and fixing any potential conflicts, are there any other things to do? I seem to be finding lots of stability issues and it would be good to see if an upgrade makes them go away (almost certainly not but hey!).
v17 is also just around the corner, so if there were an easyish procedure then that would allow for motivated souls to get some initial testing out of the way...

Hey @AntonOfTheWoods
The new version we are building is based on v16.2, I'm pull it should be ready in about two weeks. The intention is to have a patches ready for v17 on release too.
The new version is a significant refactor that unlocks dynamically runtime loaded extensions and has much better error handling.


RAMDisk Sample lost the project "package"

The folder "Windows-driver-samples/storage/ramdisk/" contains only project "WdfRamdisk", but ReadMe says:
"In Solution Explorer, you can see one solution that has two projects. There is a driver project named WdfRamdisk and a package project named package (lower case).
Project "package" is not found.

You can create a package project in Visual Studio.
And you see othr Package samples in "Windows-driver-samples",such as "Windows-driver-samples\usb\umdf_filter_umdf”


Fix: when only config custom evaluators, the keys(input_key,prediction_key,reference_key) config will be lost

Description:  When only use  custom evaluators, Will meet  Could not map run {which} with multiple keys {source} \n Please manually specify a {which}_key  ERROR from ChainStringRunMapper.   Because  the configuration of the keys(input_key,prediction_key,reference_key) are initialized only when config.evaluators are configured. So ChainStringRunMapper can't get the keys configuration.
Issue: None
Dependencies: None
Maintainer:  @hinthornw @baskaryan
Twitter handle: @_LiuHu

@hinthornw  Hi, could you give some suggestions

the same as #9763


Photo resize if Pillow installed

Is there a way to have django-ckeditor automatically create a resized photo upon upload? For instance, someone uploads a 3000x2000 px image, obviously too big for the web. Can we hood into Pillow somehow and set a max width/height or square thumbnail function?

Hello, I have done this feature (set max size for image upload) in a fork.
Take a look: https://github.com/appli-intramuros/django-ckeditor
You just have to set CKEDITOR_IMAGE_MAX_WIDTH and/or CKEDITOR_IMAGE_MAX_HEIGHT.
This feature only works with pillow backend.


Questions about group enbale

Hi, I'm implementing some functionality with the API you developed. I'm using V1742, with four groups of channels, I've added two parameters to enable_channels method, the code is like below and I've set the four channels to True. however, I've output event.GrPresent[:] in the get_waveforms method, and it outputs 1,1,0,0, and only the first two groups are enabled. Is there something wrong with my writing? I would be very grateful if you could help me! Thank you very much.
def enable_channels(self, group_1:bool, group_2:bool,group_3:bool,group_4:bool):
mask = 0
for i,group in enumerate([group_1,group_2,group_3,group_4]):
mask |= (1 if group else 0) << i
code = libCAENDigitizer.CAEN_DGTZ_SetGroupEnableMask(
self._get_handle(),
c_uint32(mask),
)
check_error_code(code)

Hi, according to the documentation of the CAENDigitizer library, your code should work. Note, however, that my function get_waveforms has hardcoded all the channels numbers for the 16 channel digitizer. If you are using a 32 channel unit, you should change this as well. This should not be too much complicated.


mupx logs not works on custom mup config

I don't know why, but after I setup two servers with different environments I can't run mupx logs with custom --config file, like:
mupx logs --tail=100 --config=mup.production.json --settings=settings.production.json
or
mupx logs --tail=100 --config=mup.staging.json --settings=settings.staging.json
the logs only shows:
Meteor Up: Production Quality Meteor Deployments
------------------------------------------------
Using custom configuration/settings file
Configuration file : mup.production.json
Settings file      : settings.production.json

[server-address] flag provided but not defined: --config
[server-address] See 'docker logs --help'.

Anybody can help me?

There is no --config option.
Just mupx logs --tail=100
On Sat, Jun 6, 2015 at 11:30 PM Caio Ribeiro Pereira <
<EMAIL_ADDRESS>wrote:

I don't know why, but after I setup two servers with different
environments I can't run mupx logs with custom --config file, like:
mupx logs --tail=100 --config=mup.production.json
--settings=settings.production.json
or
mupx logs --tail=100 --config=mup.staging.json
--settings=settings.staging.json
the logs only shows:
Meteor Up: Production Quality Meteor Deployments
Using custom configuration/settings file
Configuration file : mup.production.json
Settings file      : settings.production.json
[server-address] flag provided but not defined: --config
[server-address] See 'docker logs --help'.
Any helps?
—
Reply to this email directly or view it on GitHub
https://github.com/arunoda/meteor-up/issues/466.

But how can I see logs from a custom environment server which is from a
mup.json?
Att,
Caio Ribeiro Pereira
Founder & CTO at 99folks.com
http://crpwebdev.com
On Jun 6, 2015 3:46 PM, "Arunoda Susiripala"<EMAIL_ADDRESS>wrote:

There is no --config option.
Just mupx logs --tail=100
On Sat, Jun 6, 2015 at 11:30 PM Caio Ribeiro Pereira <
<EMAIL_ADDRESS>wrote:

I don't know why, but after I setup two servers with different
environments I can't run mupx logs with custom --config file, like:
mupx logs --tail=100 --config=mup.production.json
--settings=settings.production.json
or
mupx logs --tail=100 --config=mup.staging.json
--settings=settings.staging.json
the logs only shows:
Meteor Up: Production Quality Meteor Deployments
Using custom configuration/settings file
Configuration file : mup.production.json
Settings file : settings.production.json
[server-address] flag provided but not defined: --config
[server-address] See 'docker logs --help'.
Any helps?
—
Reply to this email directly or view it on GitHub
https://github.com/arunoda/meteor-up/issues/466.

—
Reply to this email directly or view it on GitHub
https://github.com/arunoda/meteor-up/issues/466#issuecomment-109633737.

I didn't try.
But, try to add --config before logs like,
mup --config xxx logs --tail=100
On Sun, Jun 7, 2015 at 12:19 AM Caio Ribeiro Pereira <
<EMAIL_ADDRESS>wrote:

But how can I see logs from a custom environment server which is from a
mup.json?
Att,
Caio Ribeiro Pereira
Founder & CTO at 99folks.com
http://crpwebdev.com
On Jun 6, 2015 3:46 PM, "Arunoda Susiripala"<EMAIL_ADDRESS>wrote:

There is no --config option.
Just mupx logs --tail=100
On Sat, Jun 6, 2015 at 11:30 PM Caio Ribeiro Pereira <
<EMAIL_ADDRESS>wrote:

I don't know why, but after I setup two servers with different
environments I can't run mupx logs with custom --config file, like:
mupx logs --tail=100 --config=mup.production.json
--settings=settings.production.json
or
mupx logs --tail=100 --config=mup.staging.json
--settings=settings.staging.json
the logs only shows:
Meteor Up: Production Quality Meteor Deployments
Using custom configuration/settings file
Configuration file : mup.production.json
Settings file : settings.production.json
[server-address] flag provided but not defined: --config
[server-address] See 'docker logs --help'.
Any helps?
—
Reply to this email directly or view it on GitHub
https://github.com/arunoda/meteor-up/issues/466.

—
Reply to this email directly or view it on GitHub
<https://github.com/arunoda/meteor-up/issues/466#issuecomment-109633737
.

—
Reply to this email directly or view it on GitHub
https://github.com/arunoda/meteor-up/issues/466#issuecomment-109634780.

didnt work too =/

I think we haven't implemented this for testing.
Lets see.
On Sun, Jun 7, 2015 at 12:22 AM Caio Ribeiro Pereira <
<EMAIL_ADDRESS>wrote:

didnt work too =/
—
Reply to this email directly or view it on GitHub
https://github.com/arunoda/meteor-up/issues/466#issuecomment-109636733.

Any update on that ?

+1

+1

Since providing the config file does not work for logs, does any one know where the log files are located in the server?
I could ssh and tail on the server

This is the solution I came up with, and answering @jonathanmv question
mupx deploys things through docker, so you can read the log through docker on your server. (Make sure you have docker installed on your server machine to use the command line docker utilities)
docker ps -a
Find the docker container's id
docker logs containerId

+1

+1

+1

This version is depreciated and "stable" as in not going to change.
Suggests and PRs are welcome at the new version being ported at kadirahq/meteor-up

@ruler88 Thanks :)

darn, this is the only meteor-up that works with 1.2 meteor :(


Ldanzinger/path

@nandinirao last bit of cleanup :) casing issue found while testing on Linux

:+1:


ClinVar Landscape does not show any likely pathogenic variants

Describe the bug
No likely pathogenic variants are shown (none are returned by the annonars API). It is not completely clear whether this is a bug in clinvar-this or annonars.
To Reproduce
Steps to reproduce the behavior:

Go to https://reev.cubi.bihealth.org/gene/HGNC:20324/grch37
Scroll down to clinvar landscape
See no orange/likely pathogenic variants.

Expected behavior
There should be likely pathogenic / orange variants.
Screenshots

Additional context
N/A

Resolved by fixing the data.


mouse cc genes option should be added to documentation

A .tsv with mouse cc genes is provided on the git and it's presence is acknowledged in the documentation in one place.
However, might be of added value to mention the way to use it in the documentation in two other locations:

ingestion yaml
mouse ingestion tutorial

Hi @giuliaelgarcia , do you have an update on this? thank you!


Use hooks for Action Button

This is an adaptation of @sibelius' changes. It converts over to hooks and eliminates the yellowbox warning. I had trouble with #329, so made this to address #328.

thanks

@sibelius can you provide a timeline on when this will be released?

only @mastermoo can do a new release

@mastermoo Any chance we can get a new release?

+1

@mastermoo would be nice to release a new version, in the meantime I'm installing this PR branch instead.

Please new release, maybe add additional maintainers to help you release this?

+1

@mastermoo would be nice to release a new version, in the meantime I'm installing master branch instead.

I'd use the latest commit as opposed to master, otherwise you may start getting random updates.
npm install https://github.com/mastermoo/react-native-action-button\#ad8c7e5d482296ab4f92aebcf9dcfd28481d7e15


Equip powerBi server

User Story
As a modernisation platform engineer
I need to create a new EC2 for an upgrade to powerbi
So that the equip team can continue with there upgrade work
Value / Purpose
This server requires external access to the internet specifications as bellow
Useful Contacts
No response
Additional Information
Powerbi server
2019
8 gig
8 core
Relatively fast storage
20gb D
Outbound internet connection
GW suffix
Proposal / Unknowns
Hypothesis If we... [do a thing] Then... [this will happ]
Proposal A proposal that is something testable, don't worry whether it works or not, it's a place for ideas.
Unknowns Potential pitfalls that could cause the story to expand beyond its original scope. Ideally this section will remain blank.
Definition of Done
Example

[ ] New resource created
[ ] Another team member has reviewed
[ ] Tests are green

Simon to confirm this is production only


typo fix language zh-CH

Proposed changes
fix language in multi-catalog/iceberg docs
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[x] No
[ ] No Need

Has document been added or modified:

[x] Yes
[x] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

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

LGTM


Add Option: show_message

Some fields maybe don't want to show a message, this change adds the show_message parameter to be false if you don't want to. True by default.

Sorry this has been sitting for so long, i'm seeking new maintainers, let me know here if you're interested: https://github.com/HenrikJoreteg/Happy.js/issues/72


Update timeout for .github/workflows/spec.yml

This is an automated pull request to update default timeouts for all jobs in .github/workflows/spec.yml that do not currently contain a default timeout property.
Please review these changes closely to ensure that all jobs have sufficient time to run. If a job needs more time, please add a timeout property to that job to override the organization default.
Additionally, please be attentive to unintended changes to the workflow YAML file, as the parser is not perfect.
Lastly, note that it is up to the reviewer to merge this pull request and delete the PR branch once they are satisfied with the changes.

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


Allow for node-local params to be set dynamically.

#190

@iwasaki-kenta udpated and left comment

@rkeene @hasyimibhar @diamondburned guys, please, review


Extract Java or any other "language/framework" version from image

What would you like to be added: I would like Syft to be able to extract the Java version (or really any of the "large" language/framework versions like Java, Dotnet, Python etc.) from the image that is scanned when using a base-image
Why is this needed: Since these versions are pretty core to the image, it makes sense to have them in the generated SBOM as well
Additional context:
Asked this question in the syft-help Slack channel, and was asked to file an issue, https://anchorecommunity.slack.com/archives/C019BUXV7R6/p1632984127013300
I'm currently using the adoptopenjdk/openjdk11:alpine-jre base-image and Syft to extract packages from our custom image, which works great, but the Java version is not currently being extracted which is quite important in our SBOMs.
At the moment I extract it by running the image and parsing the output from java -version into a PURL which we then convert into a cyclonedx format component, but ideally we could avoid that slightly cumbersome step.
$ docker run adoptopenjdk/openjdk11:alpine-jre java -version 2>&1 | grep version | sed 's/ version / /;s/\"//g' | awk -F" " '{print "pkg:generic/oracle/"$1"@"$2}'
Which produces<EMAIL_ADDRESS>As mentioned, it would be great if there could be support for not just Java but others like Dotnet and Python as well, though I understand that it might be difficult to support everything and anything.

Interesting problem. If I think of npm or pip for a moment, syft will find those libraries but I don't think syft can assert which runtime those libraries use or which runtime will any arbitrary program that calls those libraries will use.
If the interpreter/runtime is in the image as a Linux package, it will know that (possibly also for openjdk in the alpine case you mention above, but less reliably so in other scenarios such as "distroless") but it's no guarantee that's the stack that will end up being used at runtime, seen venv and friends.
I do think there's value for example in having a bag of semantically relevant annotations in the syft output. For example, in a system with nodejs, I would like to know the version of npm and the hashes of that binary. Possibly the same for maven, cpan, etc., since those seem to be relevant to the supply chain security/dependency acquisition of the system being evaluated. I'd be particularly interested in those annotations in cases where the hash of the binary does not match the hash expected in, say, a package manifest.


Normalize the size bar

I guess the size bar currently shows  the percentage of the file relative to the whole folder. It would be better to set the largest one to 100%, and the others change proportionally, as ncdu does.

Why do you think it would be better? I think the current way is more logical. You would loose the graphical reference between the size of the whole folder and its files. You wouldn't see at the first sight if the greatest file consumed all the space or not.

My thought is to maximize the usage of the bar's space. There is already total size in the status bar, so it is easy to know if the largest file is taking up the most space. Also, if so, you will see one size bar much longer than any other ones anyway.
In my case, my home folder has a few folders with similar size, now I am seeing all the size bars only have two #s or less, while the space could show 10:
   35.6 GiB [##        ] /VM
   34.8 GiB [##        ] /Music
   27.9 GiB [#         ] /Documents
   22.0 GiB [#         ] /Videos
   20.0 GiB [#         ] /Downloads
   10.1 GiB [          ] /Pictures
... ...

What ncdu shows:
   35.6 GiB [###########] /VM
   34.8 GiB [########## ] /Music
   27.9 GiB [########   ] /Documents
   22.0 GiB [######     ] /Videos
   20.0 GiB [######     ] /Downloads
   10.1 GiB [###        ] /Pictures
... ...

It's a matter of whether the size bar should prioritize showing the absolute or relative size, then.

I am not sure for now if the usage of the bar's space is so important. I see it as two approches: ncdu shows comparison of sizes of the items. Gdu (and dua) shows disk usage of the items in the context of the current directory.
The example of /usr/share/icons/Adwaita/256x256/legacy/ is interesting thanks for it.
My idea: you can have two scenarios which would look the same in ncdu but are different in fact:
You will have one big file and many many small files in one directory.

the big file will occupy most of the space used by the whole directory
the big file is much larger then the small files, but they are so many that in the end the big file will occupy only some part of the whole usage

Both cases will seem the same in ncdu, but will be shown differently in gdu. It seems to be that the gdu/dua approach is more correct in the context of disk usage.

Yes, now we see, the two ways both have their own flaws, and serve different purposes. So I agree to disagree.

Ok, thanks. Closing for now.


Task to convert photos from heic to png and to upload them to the aws

On the add note screen the task is to add the photo either via gallery or taking a photo and then setup a route which allows the photo to be sent to an s3 bucket
this bucket will then generate a link which should be fetched on the mobile side and sent along with the note

This issue was resolved by the annotations branch. S3Proxy is still down but for now its behind hosted on a Windows server from @izakrobles home. AWS uploads work for PNG, HEIC, JPEG, and ultimately get sent as a JPEG.


Fix RDF::List#== comparisons to RDF::Values

Ruby 2.3.0 changed the behavior of Comparable#== to avoid hiding errors. This led to NoMethodErrors for comparisons that return false for previous Rubies.
This introduces a custom RDF::List#== implementation for explicitly supported comparisons between RDF::Value classes. We return false immediately for RDF::Values which are not #list?; falling back on Comparable for other types.
Further, RDF::Lists with three elements that happen to coincide with the terms of an RDF::Statement would previously return true. E.g.: RDF::List[:s, :p, :o] == RDF::Statement(:s, :p, :o). This unusual edge case is patched by way of the changes described above.
A similar change is made to RDF::Statement#== to preserve symmetry.
Closes #304

Coverage increased (+0.006%) to 90.961% when pulling 9bd9cadaf480252bf619df58bf726e3232833704 on feature/list-value-eq into e990dedbe6e56334dc162337834497aef8c5e156 on develop.


Simply the initialization of the integration test harness

Goal

Testing

Release Notes

WHAT:
WHY:
WHO:

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#716  👈
#715
#714
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @bidetofevil and the rest of your teammates on  Graphite


iPad - Text scrolling delayed

I' using Cufon for Headlines in an overflowed div using jScrollpane. On the iPad (iOS 5) I'm experiencing that parts of the headline are staying on their initial position for a few seconds before snapping to the position I've scrolled to.
Is that a performance issue or could it be Cufon-specific?
Best,
Juergen

Hi,
Things like this can usually be solved by adding position: relative to a suitable ancestor. For example, if the heading is inside a column, you might want to try positioning the column. Does it feel unnecessary? Yes it does. But does it work? Most of the time it does :) If it doesn't work right away, just keep playing with adding position: relative to other ancestors as well. If it STILL does not work, try doing everything again, but this time also add z-index: 1. It should work. If not, let me know.
Simo

Thanks for your proposals!
Unfortunately they don't seem to solve my issues. I've tried to position the surrounding elements as you said and also defined a z-index, still the headlines get "fragmented" when scrolling. Take a look: http://ralf-schmitz.de/aktuell/
Juergen

Hi,
I don't have an iPad, but I do have an iPhone so I tried with that. Unfortunately the error was nowhere to be seen :( You may be on your own here.. I would suggest just very aggressively adding different CSS properties to perhaps even all the elements (with the star selector) and seeing what helps. Your layout may break in other ways but when you find out what helps, you should be able to narrow it down.
I really expected the positioning to work, though. Could you perhaps also try:
cufon { position: relative; z-index: 1 }

Simo


Improve code coverage for DerivedAsyncCache

It's currently only 59% tested, see https://coveralls.io/builds/13267639/source?filename=src%2Fmain%2Fscala%2Fcom%2Fgilt%2Fgfc%2Fcache%2FDerivedAsyncCache.scala
Code coverage should be >95%

Wrong repo 🤦♂


Enhancing Analyzer Accuracy for String Parsing and Special Characters Handling

The current analyzer employs a fundamental string parsing logic (regular expressions, string splitting ..), which means that it is not guaranteed to be 100% accurate. This is because some characters, such as double quotes, can be interpreted as special characters by the analyzer.
For example, the following Python code:
from django.urls import path
from . import views

urlpatterns = [
    path("example\"'route", views.app2_index, name="index"),
]

will not be parsed correctly by the analyzer because the double quotes(") are interpreted as part of the path variable.
Similarly, the following Go code:
e.GET("/pet,comma", func(c echo.Context) error {
    return c.String(http.StatusOK, "Hello, Pet!")
})

will also not be parsed correctly because the comma(,) is interpreted as a delimiter.
As this doesn't represent a universal scenario, I'm not sure whether to keep as a known issue or implement a shared lexer and parser to handle these cases more comprehensively.

You're right; guaranteeing perfection is challenging due to our tool's reliance on regular expressions and string matching for analysis. Creating a Lexer/Parser would involve abstracting the code, considering each language's syntax, and identifying endpoints. This would necessitate significant changes to our current structure.
While I agree it's the right long-term direction, taking the first step is proving to be tough 😨


Video not working on Safari 11 (from Mac OSX Sierra)

You are using 0.7.X. Please update to 0.8.0.


Changing a Module Type has unforeseen consequences.

Describe the bug
Not all module types properly initialize the area of their selection.
Application: Brain Simulator
Module/Network: new network
To Reproduce
Steps to reproduce the behavior:

Create a ModuleCamera module.
Right-click it and change the type to ModuleColorComponent.
See the mix of two types of module cells.
What is the resulting module?

Notice that depending on the types chosen, results differ....
Expected behavior
Best outcome: a change in module type will properly set up and initialize the ENTIRE module
Second best: once a module is created, changing its type should be a non-option.
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: Windows 10 Pro  20H2
Version 2021-04-19 master

Add any other context about the problem here.

Looks fine now, area is cleared before module creation and changing a module type is no longer possible, but #112 is still a problem.


Integrate tqperf model under Jenkins CI

https://github.com/nrnhines/tqperf is good spike exchange test as it stresses MPI communication. We should integrate it under Jenkins Pipeline.

The instructions  to run the model are here.

~Note that we are using pramodk/tqperf fork as we are waiting for https://github.com/nrnhines/tqperf/pull/2 to be merged.~

Where should I include it?
New job or existing one?

It should be in existing Jenkinsfile as a new stage. @iomaganaris can help / clarify if anything needed.

#288

Done


mod rewrite enabled, but navigation still links to /index.php/...

Hey,
i purchased a webserver (and this one is only a webserver with no access to the cli) and installed pagekit on that server.
First links like http://myDomain.de/home didn't work, because i had to define this line in my .htaccess file:  RewriteBase / After i did this, links like http://myDomain.de/home finally worked.
Problem
Now the problem is, that the navigation links are still pointing to http://myDomain.de/index.php/myPage
Technical Details

Pagekit version: 1.0.3
Webserver: Apache
Database: MySQL 5.5.49-0+deb7u1-log
PHP Version: 5.5.36

Can you review https://github.com/pagekit/pagekit/issues/642 to see if you have the same server setup (Apache, but running with FastCGI and therefor Pagekit doesn't detect mod_rewrite). Also, there is a quick fix linked in the issue, although that fix isn't upgrade-safe.

That fixed it!
Apache is running with cgi-fcgi.
But now on the Dashboard, pagekit tells me that my pagekit version is outdated and i should update to pagekit 1.0.5? That message wasn't there before. And on other pagekit installations i manage, this message isn't shown, too.

I wouldn't close that issue...
this should be fixed in pagekit.. Wordpress doesn't have that issue.
Changing that file after every pagekit update can't be a solution.

You're right, there should be a better option than the dirty fix that is listed in the issue I linked. However, I think this is a duplicate of #505 which is still open. :)


Can't install ifuse

Installation grabs the dependencies then fails with
==> ./autogen.sh
checking whether clang understands -c and -o together... yes
checking for style of include used by make... GNU
checking dependency style of clang... gcc3
./configure: line 3899: syntax error near unexpected token `libimobiledevice,'
./configure: line 3899: `PKG_CHECK_MODULES(libimobiledevice, libimobiledevice-1.0 >= 1.0.0)'

Hmm, I just tried on my work laptop and it installed like a charm. I'm not sure what the difference is. Both are running Yosemite.

Can you brew gist-logs ifuse on the problem machine, Thanks!

I'm not familiar with gist-logs. Is this output useful?:
$ brew gist-logs homebrew/fuse/ifuse
Error: HTTP 404 Not Found (expected 201)

$ brew gist-logs ifuse
Error: HTTP 404 Not Found (expected 201)

That's unexpected, and unhelpful :smile:. Does brew doctor output anything interesting? brew config may be useful on this as well.

$ brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry and just ignore them. Thanks!

Warning: Python is installed at /Library/Frameworks/Python.framework

Homebrew only supports building against the System-provided Python or a
brewed Python. In particular, Pythons installed to /Library can interfere
with other software installs.

Warning: Unbrewed header files were found in /usr/local/include.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae, and may need to be deleted.

Unexpected header files:
    /usr/local/include/mkdio.h

Warning: Unbrewed static libraries were found in /usr/local/lib.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae, and may need to be deleted.

Unexpected static libraries:
    /usr/local/lib/libmarkdown.a

I think those are left over from when I installed Python from its website and the "discount" C-based markdown library.
$ brew config
HOMEBREW_VERSION: 0.9.5
ORIGIN: https://github.com/Homebrew/homebrew.git
HEAD: 3a35a9fc9a1329f2153dff2b905634f38c30d1fb
Last commit: 32 minutes ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_REPOSITORY: /usr/local
HOMEBREW_CELLAR: /usr/local/Cellar
HOMEBREW_BOTTLE_DOMAIN: https://homebrew.bintray.com
CPU: quad-core 64-bit haswell
OS X: 10.10.5-x86_64
Xcode: 6.4
CLT: <IP_ADDRESS>.1.1435007323
Clang: 6.1 build 602
X11: N/A
System Ruby: 2.0.0-p481
Perl: /usr/bin/perl
Python: /usr/local/bin/python => /usr/local/Cellar/python/2.7.10_2/Frameworks/Python.framework/Versions/2.7/bin/python2.7
Ruby: /usr/local/bin/ruby => /usr/local/Cellar/ruby/2.2.3/bin/ruby
Java: 1.6.0_65

I can't get this locally, but I'm not really sure why you are, unless it's an obscure race condition of some sort.
checking whether clang understands -c and -o together... yes
checking for style of include used by make... GNU
checking dependency style of clang... gcc3
checking for pkg-config... /usr/local/opt/pkg-config/bin/pkg-config
checking pkg-config is at least version 0.9.0... yes
checking for libimobiledevice... yes
...

==> Summary
🍺  /usr/local/Cellar/ifuse/1.1.3: 7 files, 76K, built in 56 seconds

Perhaps worth trying brew reinstall libimobiledevice just in case there's an issue there.

FTR: Tried brew reinstall libimobiledevice, didn't help. :-(

This did it:
$ brew search pkg-config
pkg-config (installed)

$ brew upgrade pkg-config
Error: pkg-config 0.28 already installed

$ brew reinstall pkg-config
==> Reinstalling pkg-config
==> Downloading https://homebrew.bintray.com/bottles/pkg-config-0.28.yosemite.bottle.2.tar.gz
######################################################################## 100.0%
==> Pouring pkg-config-0.28.yosemite.bottle.2.tar.gz
🍺  /usr/local/Cellar/pkg-config/0.28: 10 files, 612K

$ brew install ifuse
==> Installing ifuse from homebrew/homebrew-fuse
==> Downloading https://github.com/libimobiledevice/ifuse/archive/1.1.3.tar.gz
Already downloaded: /Library/Caches/Homebrew/ifuse-1.1.3.tar.gz
==> ./autogen.sh
==> ./configure --prefix=/usr/local/Cellar/ifuse/1.1.3
==> make install
🍺  /usr/local/Cellar/ifuse/1.1.3: 7 files, 76K, built in 15 seconds

Thanks!

Don't think I've seen pkg-config break much before, that's new :smile:. Glad all is working as hoped now, thanks for closing the issue up!


feat: implement transaction identifiers

Description
Enable users to read transactions identifiers on table and allows writing of transactions identifiers for all operations.
Related Issue(s)

closes #2130

@roeap Please provide some feedback on how I'm interacting with the log replay.
Transaction Identifiers are at the DeltaTableState level which encapsulates a EagerSnapshot however EagerSnapshot is responsible for using LogReplayStream for reading checkpoint and commit files. Either I move the application transcription into the eager snapshot, read the same files twice, or implement some hook for when the replay is being performed.
I implemented the hook with a visitor trait called ReplayVisitor which can be specified whenever a new EagerSnapshot is being created or whenever states are being updated merged. This will allow information from checkpoints & commits to flow upwards depending on what feature are being used.

@Blajda - very sorry for the long delay! Will do a review now.


Lecturio Error: ERROR: unable to open for writing: [Errno 2] No such file or directory

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.12.09. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x ] I've verified and I assure that I'm running youtube-dl 2018.12.09

Before submitting an issue make sure you have:

[x ] At least skimmed through the README, most notably the FAQ and BUGS sections
[x ] Searched the bugtracker for similar issues including closed ones
[x ] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
youtube-dl -v --cookies cookies.txt https://app.lecturio.com/medical-courses/sympathetic-nervous-system-hypothalamus-as-sympathetic-output-driver-and-horners-syndrome.lecture
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '--cookies', 'cookies.txt', 'https://app.lecturio.com/medical-courses/sympathetic-nervous-system-hypothalamus-as-sympathetic-output-driver-and-horners-syndrome.lecture']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2018.12.09
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.17763
[debug] exe versions: none
[debug] Proxy map: {}
[Lecturio] sympathetic-nervous-system-hypothalamus-as-sympathetic-output-driver-and-horners-syndrome: Downloading webpage
[Lecturio] sympathetic-nervous-system-hypothalamus-as-sympathetic-output-driver-and-horners-syndrome: Downloading JSON metadata
[debug] Default format spec: best/bestvideo+bestaudio
[debug] Invoking downloader on 'https://d31o3fxohn22zc.cloudfront.net/1606232300-120216-o-20160608_01_04_Neuroanatomy_Canby_Craig-v1-st1.mp4?Expires=1544654246&Key-Pair-Id=APKAJXIJCNEVQLT2FFDQ&Signature=YWzTuD8mNGw68iB-QbOoTD9KBqucYYpqxAqmTZWEdTgk51qxFF5QkHKEus6cYZAklaRvWbZwpa29i0KOW1AdxS6fANZTAMoWlKGsOg8bv3iAkRwkf99aXh1HZTGPJ11L00WAsIdgw5akp~PwkuAHm4dQafsfBsQ~Z-LGRZBhMNE_'
ERROR: unable to open for writing: [Errno 2] No such file or directory: 'Sympathetic Nervous System (SNS) - Hypothalamus as Sympathetic Output Driver and Horners Syndrome-24454.mp4.part'
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpg8efod20\build\youtube_dl\downloader\http.py", line 250, in download
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpg8efod20\build\youtube_dl\utils.py", line 458, in sanitize_open
FileNotFoundError: [Errno 2] No such file or directory: 'Sympathetic Nervous System (SNS) - Hypothalamus as Sympathetic Output Driver and Horners Syndrome-24454.mp4.part'

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.

The error message is self-explanatory.

I was intrigued to get this particular error (ERROR: unable to open for writing: [Errno 2] No such file or directory) while running youtube-dl from a specific directory, but not from another. It turned out that the movie had a long name and, along with the first directory name, led to a path that was longer than the maximum path length allowed in Windows, which is 260 characters:

In the Windows API (with some exceptions discussed in the following paragraphs), the maximum length for a path is MAX_PATH, which is defined as 260 characters. A local path is structured in the following order: drive letter, colon, backslash, name components separated by backslashes, and a terminating null character.

See #29912.


Bridge not found

I have HA-Bridge running back home in the UK - no problem in a Raspberry Pi.
I have a new installation here in Spain - I have two Alexa units here working perfectly (well, other than they won't report the weather here!)
I've set up Ha-Bridge as normal on port 82.  It works. I've given it two devices (exe - running mosquitto_publish) and I've tested them in Ha-Bridge with the on-off controls - all works perfectly.
However - when I tell Alexa to find devices - it finds one device (a customised SONOFF device) - but will NOT find these devices.
In my router, UPNP is ticked - and in the Bridge on the Pi, it refers to the correct IP address and port.
Help.

Are all your Echo units on one account?

Anything here?

I do hope so I’m getting no-where with the new HA-Bridge – my alexa is just not seeing it.
From: BWS Systems<EMAIL_ADDRESS>Sent: 03 May 2017 17:53
To: bwssytems/ha-bridge<EMAIL_ADDRESS>Cc: Peter Scargill<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [bwssytems/ha-bridge] Bridge not found (#609)
Anything here?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/bwssytems/ha-bridge/issues/609#issuecomment-298953617 , or mute the thread https://github.com/notifications/unsubscribe-auth/ABzUgz5cBwXbiwdWOOTlf8BV2lIi1dl2ks5r2KLSgaJpZM4NHtMy .

So, I assume you are running a second bridge in the Spain location?

In the UK location I had 2… and it seemed to pick everything up just fine…
In Spain… let me see….Right, with one device attached – I told Alexa to find devices and it found the one device I’ve created…. (2 devices as there is also an ESP8266 device which works with Alexa).  Yup – sure enough I can turn the Pi device on and off.
So I turned on the second Pi with Habridge on that… told Alexa to find devices and… it is still working – HOW ON EARTH is that – it wasn’t working before.
Dooooohhhh….
From: BWS Systems<EMAIL_ADDRESS>Sent: 04 May 2017 15:40
To: bwssytems/ha-bridge<EMAIL_ADDRESS>Cc: Peter Scargill<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [bwssytems/ha-bridge] Bridge not found (#609)
So, I assume you are running a second bridge in the Spain location?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/bwssytems/ha-bridge/issues/609#issuecomment-299187949 , or mute the thread https://github.com/notifications/unsubscribe-auth/ABzUg2trYM-Ohb5BIp1gAi5GV4O2jgH5ks5r2dUzgaJpZM4NHtMy .

Sometimes, it happens.... glad you got it solved.


Feature/rds add engine_mode and global_cluster_identifier

Needed this for an aurora global database setup that I'm working on.

v1.11.0 has been released.
Thanks for this addition, Hugo!


Move Schemas to APIs/schema folder

See https://specs.amwa.tv/bcp-007-01/branches/1.0-AF-edits/APIs/schemas/ (correct at time of writing)


Rus Translation

Issue from modhub:

Naron79 - 16 hours ago
Hydrogen gas not translated to russian

No response after a month. Closed.


Text width or wrap

Is it possible to have a multline Text?

Hi @kier70,
Unfortunately multiline is not supported by Text and you need to do it manually.

thanks @ystrot , I have implemented this myself.


baseline relative path

I think that the program searches for ./models/baseline.pth, so if you run python3 vocal-remover/inference.py --input ... it won't work.

Download the zip from releases folder. baseline.pth is included there.

I think that the program searches for ./models/baseline.pth

@hykilpikonna what I mean is that you have to run the script in the same folder

if you run python3 vocal-remover/inference.py --input ... it won't work

The problem lies on the torch load (line 127), as it's a relative path.
Changing it for something like this should work: model.load_state_dict(torch.load( os.path.join(os.path.dirname(os.path.abspath(__file__)), args.pretrained_model) , map_location=device))

我认为该程序搜索。/ models/baseline.pth，所以如果你运行python3 vocal-remover/inference.py --input ...，它将不起作用。

所以运行的时候报这个错误应该怎么解决？

Download the zip from releases folder. baseline.pth is included there.从releases文件夹下载zip。baseline.pth包含在其中。

where is release folder？

Here it is.
https://github.com/tsurumeso/vocal-remover/releases

@tsurumeso I think this issue has been misleading... I'm not complaining about "not being able to find the .pth"; as others said, you have to download it from the release. I'm complaining about "the program can't find the .pth, if it's not launched from the same base folder".
You can try to go one directory up, and run python3 vocal-remover/inference.py --input ... you'll get an issue. The PR https://github.com/tsurumeso/vocal-remover/pull/138 should fix that issue.


Append does not work if key=0

It's the 0 = False trap ;-). Replacing the test of key with
    if type(key) is str and key=='':
        key= str(id(value))

seems  to work

Hi Ken, thank you for reporting. Now should be fixed.


feat: add directory index fallback

Description

fixes #14506 (NOT YET this is a wip rn)
so far:

todo:

add fs.allow/fs.deny stuff
add human readable file size (show "1.2 kb" not "1200 bytes")
only activate middleware on DEV MODE and when some option (tbd) like server.directoryIndex = true (default false?)
sort the directories first, then the files like chrome does in file:/// pages

notes to self:
https://vitejs.dev/config/server-options.html#server-fs-allow
Additional context

idk really how to make the fs.allow and fs.deny stuff work in a dir listing. right now the shown dir list is UNSORTED and IRRESPECTIVE of any fs.allow/fs.deny stuff. i confess that I've never personally  used fs.allow/fs.deny so i dont have any personal experience with it. looking for help/suggestions + advice on that.
@bluwy

What is the purpose of this pull request?

[ ] Bug fix
[x] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[ ] Read the Contributing Guidelines.
[ ] Read the Pull Request Guidelines and follow the PR Title Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

 Submitted with StackBlitz Codeflow.

a better way to do this might be to delegate/rewrite the /@fs/ route? like just rewrite /hello/ to /@fs/home/user/workspace/hello/ instead? idk if the /@fs/ router is configured to already show directory listings. it doesn't look like it so that might be a no-go.

works!
i hope i got the fs.allow/fs.deny logic right 😬


Documentation on Migration of old A2HA  to new A2HA

Description
Below items will be covered as part of wiki documentation:

Migration doc for old A2ha to new a2ha

Acceptance Criteria
The WIki page should be ready along with KT given to the documentation team

https://github.com/chef/a2ha-docs/wiki/Migration-A2HA-to-Automate-HA

@dkumaras what information you need from us, this ticket was send back to us.


tests script for board

Mainly added a python script for running tests on board. Merged the original tests script into it as well.
Currently it's slow to run the nistkat test on board, due to the bottleneck of IO. Checking the checksum on board would be better.
☺  tests --help
Usage: tests [OPTIONS] COMMAND [ARGS]...

Options:
  --help  Show this message and exit.

Commands:
  func     Run functional tests
  nistkat  Run nistkat tests
  run      Run for the specified platform and hex file without parsing the
           output
  speed    Run speed tests
  stack    Run stack tests

☺  tests func --help
Usage: tests func [OPTIONS]

Options:
  -cfg, --platform-cfg PATH  Configuration file of the specified platform
                             [default: hal/stm32f4discovery.cfg]
  -v, --verbose              Show verbose output or not
  -e, --emulate              Emulate on the QEMU or not
  -t, --ntests INTEGER       Number of tests  [default: 1]
  --help                     Show this message and exit.

add marker for tests when running on board
add python script for running tests on board
use the python tests script in ci instead
clean up the original ci test script

[ ] Does this PR change the input/output behaviour of a cryptographic algorithm (i.e., does it change known answer test values)?  (If so, a version bump will be required from x.y.z to x.(y+1).0.)

Updated the README, but not sure if it's clear enough...


small changes to UI and some aspects of user guide

updated user guide help page and find images
changed help page UI
small changes to UI

Codecov Report

Merging #113 (b2abaef) into master (f6c30a2) will increase coverage by 0.08%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #113      +/-   ##
============================================
+ Coverage     65.28%   65.36%   +0.08%
  Complexity      620      620
============================================
  Files           104      104
  Lines          2336     2333       -3
  Branches        254      253       -1
============================================
  Hits           1525     1525
+ Misses          717      714       -3
  Partials         94       94

Impacted Files
Coverage Δ

...va/seedu/address/logic/commands/RemoveCommand.java
8.33% <ø> (ø)

...in/java/seedu/address/logic/parser/ParserUtil.java
92.45% <ø> (+6.48%)
:arrow_up:

...eedu/address/logic/parser/RemoveCommandParser.java
23.80% <0.00%> (ø)

src/main/java/seedu/address/ui/HelpWindow.java
0.00% <0.00%> (ø)

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


Add geocod.io

They have a pay-as-you-go option that allows you to start at no cost without needing to provide payment info: https://www.geocod.io/pricing/.
I've started creating a new geocoder class for this and can open a PR when it's ready.

Merged in #468


Update sample to reflect .NET Core 2.1 out of the box

Fixes #8558.
Problem
If I'm correct, I think the issue here is that the .NET Core 2.1 OOTB _Layout.cshtml is a little bit different now, and ApplicationInsights is not included out of the box. Therefore, the error was received after copying/pasting.
Fix Steps

I ran through the steps in the tutorial using .NET Core 2.1 on my local until I hit that step.
I copied the _Layout.cshtml file that my .NET Core 2.1 app came out with.
I pasted that into the example file, overwriting it.
I updated the title line and the line referring to the movies controller
I updated the highlights

Remaining Work

[x] Update _Layout.cshtml sample from an out of the box .NET Core 2.1 webapp.
[x] Keep the changed lines for title & Moves Controller reference
[x] Update the line highlighting in the actual example

@SeanKilleen much appreciated.
I deleted _CookieConsentPartial  as you suggested.
We'll soon be unifying the 3 tutorials so it should be much easier to updated them (the includes will be gone and there will be just one file).


Add local-echo functionality for reactions.

The goal here is to make it so that when I click a reaction, it immediately shows up, without having to wait for the server.  If the reaction gets rejected server-side, or times out, we'll want to deal with it accordingly.  A lost reaction is generally not the end of the world, so our emphasis here should be on a good experience for the normal case.

@zulipbot claim


Add support for ticket prices

Hi,
I did some research on how to retrieve ticket prices using the DB api. In order to make the information available to others, I created a small sample project to demonstrate the required requests:
https://github.com/TheRealMurmel/py-bahn-api
derhuerst suggested in public-transport/hafas-client/issues/291 to integrate this into pyhafas.
I could give it a try, if you are interested in this feature.

I would be interested in that feature as well. Could offer some help in implementation and/or testing :)


[FEATURE] Support encrypted RWX volumes

Is your feature request related to a problem? Please describe.
We support PV encryption for RWO volumes via the CSI mechanism.
For RWX volumes we need to add support to the share manager to support encrypted RWX volumes.
This required some changes on the share-manager to use host mounts instead of the inside pod mounts.
Additional context
PV encryption is implemented in https://github.com/longhorn/longhorn/issues/1859

@meldafrawi same testing as for RWO global-crypto and a per PVC test, since then the secret is stored in the PVC namespace and the longhorn share-manager controller will retrieve it and use it. Just need to make sure permissions work correctly for that one.
The per pvc secret, uses a template of the PVC name and will look up a secret for that name. The PVC will remain in pending state till that secret can be retrieved then the create volume call will execute.

Validation: PASSED
Setup:

Create a kubernetes cluster with 3 worker nodes, and 1 controller node.
Deploy Longhorn (master branch)
Create a secret to be used for encryption: Ref: secret-crypto-global.yaml

Test(1): Deploy workload using Encrypted RWX PV with ext4 filesystem. (PASSED)

Create a Longhorn volume, and mark check Encrypted: true and Access Mode: to ReadWriteMany
Create a PV/PVC, and set Secret Namespace and Secret Name values as specified in secret-crypto-global.yaml and check File System: ext4
Deploy a workload using the created PVC.
Check read/write operation into the volume.
write some data and compute it's checksum

Test(2): Deploy workload using Encrypted RWX PV with xfs filesystem. (PASSED)

Create a Longhorn volume, and mark check Encrypted: true and Access Mode: to ReadWriteMany
Create a PV/PVC, and set Secret Namespace and Secret Name values as specified in secret-crypto-global.yaml and check File System: xfs
Deploy a workload using the created PVC.
Check read/write operation into the volume.
write some data and compute it's checksum

Test(3): Provision a volume though creating a PVC from a storage class that support PV encryption  (PASSED)

Create a storage class that support PV encryption. Ref: crypto secret
storage class

Use the following manifest to create a PVC and provision a volume.

apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: volume-00
  namespace: default
spec:
  accessModes:
  - ReadWriteMany
  resources:
    requests:
      storage: 2Gi
  storageClassName: longhorn-crypto-global
  volumeMode: Filesystem

Deploy a workload to consume the PVC
make sure you can read/write into the volume.

Test(4): Provision a volume though creating a PVC from a storage class that support PV encryption per volume with  detecated namespace (PASSED)

Create a storage class that support PV encryption. Ref: crypto secret
storage class

Use the following manifest to create a PVC and provision a volume.

apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: volume-01
  namespace: default
spec:
  accessModes:
  - ReadWriteMany
  resources:
    requests:
      storage: 2Gi
  storageClassName: storageclass-crypto-global
  volumeMode: Filesystem

PVC will be in pending state

Create a secret with the same PVC name, in longhorn-system namespace, the secret use the same data keys as secret-crypto-global
Deploy a workload to consume the PVC
Make sure you can read/write into the volume.

Test(5): Provision a volume though creating a PVC from a storage class that support PV encryption per volume with general namespace (PASSED)

Create a storage class that support PV encryption. Ref: crypto secret
storage class

Use the following manifest to create a PVC and provision a volume.

apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: volume-02
  namespace: default
spec:
  accessModes:
  - ReadWriteMany
  resources:
    requests:
      storage: 2Gi
  storageClassName: storageclass-crypto-per-volume
  volumeMode: Filesystem

PVC will be in pending state

Create a secret with the same PVC name, and in the same PVC namespace, the secret use the same data keys as secret-crypto-global
Deploy a workload to consume the PVC
Make sure you can read/write into the volume.


fix(coral): Fix broken error message handling in some Form elements

Detailed bug report
https://github.com/aiven/klaw/issues/448
This change

updates the formSchema objects used in tests. They mostly used name="name", this covered a bug in error handling.
update parseFieldErrorMessage in Form to use errors[name] (e.g. errors[myFieldName]) instead of errors.name`
remove usage of const error = get(errors, name)?.message as string; that now is not needed anymore
updated MultiInput in Form to show error messages in some cases:

error is shown and field marked as invalid when user wants to submit an empty MultiInput that is required
error is shown when user adds something to MultiInput that may not be valid and tabs out
error is shown (for an required MI) when user adds something and removes it again

Screenshots for this implementation

Resolves: #448

@programmiri unfortunately not in a separate commit, sorry :/

@mathieu-anderson, not a problem, I copied it!
Should the validation work onBlur? 🤔  For me it only works when I add something something and remove it again 🤔

@programmiri When you say onBlur, you mean in the flow like this?

focus field
unfocus field
should trigger "cannot be empty" error

I think this doesn't work indeed. It only works onChange, I believe, so you need to have entered things in the input before it starts validating. You can see the behaviour in the ideo posted on the ACL request form PR https://github.com/aiven/klaw/pull/398

@mathieu-anderson  - we answered in parallel :D I just updated my post above yours.
I updated the PR, maybe you can do an 👀 again and if it's ok 👍  it and 🚢  it! (well, merge it, but there's no emoji for that)


Consolidate packages (part two)

(This continues the package consolidation from https://github.com/kubernetes-sigs/k8s-container-image-promoter/pull/262.)

/assign @listx @justinsb
This is ready for review.

Thanks!
/approve
/lgtm


Fix git metrics for languages that don't support entity parsing

Summary
Check that currentGraph is defined addGitMetricToFileNodes before attempting to update git metrics.
Languages that don't support entity parsing attempted to update the graph with git metrics before currentGraph was defined on initial rendering.
Issue
To reproduce the issue (before this change), you can run the following in this repo:
python3 emerge.py -c py-template.yaml

using the following py-template.yaml (default python config with git_directory and git-metrics added):
---
project_name: py-example-project
loglevel: info
analyses:
- analysis_name: py check
  source_directory: emerge
  git_directory: ./.git
  only_permit_languages:
  - py
  only_permit_file_extensions:
  - .py
  file_scan:
  - git_metrics
  - number_of_methods
  - source_lines_of_code
  - dependency_graph
  - louvain_modularity
  - fan_in_out
  - tfidf
  export:
  - directory: emerge-output
  - graphml
  - json
  - tabular_file
  - tabular_console_overall
  - d3

Opening the output html (i.e. emerge-output/html/emerge.html) won't display a graph due to the error described above.

Tested it with a Go repo :+1: Thanks!

@tonysyu Thanks for this fix 🚀, @greenled thanks for testing 💪.


Changed hosting port

changed port from 5000 to 80 PS needs to be run in sudo or higher permissions for this to work

Hi, thanks for the PR, I never worked with custom servers before, will those servers exclusively require port number 80? I know it's a standard port for HTTP, but when I have worked with Heroku, it used random ports for the server. If that's the case I will merge this PR.


Documentation incorrectly describes how to use the icons property

This applies to both the Accordion and Calendar documenation.
As discussed in Slack #general.
The documentation incorrectly describes how to use the icons property.  E.g.
accordion.icons.collapse
The icon to use when the panel is expanded.

However, using the icon this way leads to the JS Error:
Error
Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: object.

Check the render method of `Accordion`

Instead, it should be "UpIcon", not "".

See PR #194.

See PR #194.


Suggest a Repository name on the "Create Repository" page.

Feature Description
GitHub has had this feature for quite a while now and i find it really useful, especially for creating test repositorys. Basically, when creating a new repository, GH suggests a unique repo name that consists of three random words, and when clicking on that suggestion, it fills in automatically. That would be nice to have for Gitea.
Screenshots
Mock up:

Really? I find that to be rather annoying:
Instead of something useful, I always have to read through this irrelevant section that does nothing else but waste my time.
Especially, even if you have to create test repos (which already happens basically never or at most once), you still cannot use the suggested name: You will never remember that that repo is a test repo with no other purpose because there is no indication in its name.
Additionally, it doesn't take much longer to navigate to the text field and input test (or something similar).

Yes, really. I don't understand what you mean with "irrelevant section", this proposal would add five words to the already existing text. Also, labeling test repos with test1, test2, test3, ... isn't that helpful either, when it comes to finding your test repo from five months ago.


[HTTP] Remove setting of default headers in outbound request from transport to ballerina

Description:
The transport sets certain default headers to the client connector when it sends a message to the backend. This is not configurable and these headers should not be set in a pass through scenario. Hence setting of default headers should be done by ballerina and not by the HTTP-transport for more flexibility.

To be specific it is only the Accept-Encoding header. We have remove that behaviour from the transport with merged PR.


How to refetch 'atomWithQuery' ?

Hello!
How I can refetch 'atomWithQuery' ?
Example:
I got that assync GET atom which is constructed to get category dynamicly from showcaseCurrentActiveCategoryAtom  :
`
   export const showcaseCurrentActiveCategoryAtom = atom({
      class: 'showcase-categories__item--all',
      metaTitle: 'Fruits',
    })

  const getRandomOffersByCategoryAtom = api.offers.randomByCategory.atomWithQuery((get) => {
          return {
                       count: 25,
                       category: get(showcaseCurrentActiveCategoryAtom).metaTitle,
                      }
  })

`
I want to fetch what query on page, and in some moment to refetch it again after changing showcaseCurrentActiveCategoryAtom.
How can i do that?
If i do like that in click event it did't tefetch - it just make query with default value:
`
  const [randomOffersByCategory] = useAtom(getRandomOffersByCategoryAtom)

  const onCategoryClick = async () => {

   setShowcaseCurrentActiveCategory({ class: item.class, metaTitle: item.title })

   setShowcaseOffers(randomOffersByCategory.data)
  }

`

Think I got it - just put 'howcaseCurrentActiveCategory' in useEffect deps and call 'randomOffersByCategory' there

Yeah, jotai-trpc itself doesn't have a capability of imperative refetching.
We can do atomWithRefresh pattern.
https://jotai.org/docs/recipes/atom-creators#atomwithrefresh


casual-caller - incorrects logs when pool stickies are enabled and no sticky is present

Incorrect logs like Failed to call service=<someservice> on stickied pool=null, falling through to normal flow. happen. No service call has failed and pool=null is a give-away that something incorrect is happening
It happens here https://github.com/casualcore/casual-java/blob/dev/casual/casual-caller/src/main/java/se/laz/casual/connection/caller/FailoverAlgorithm.java#L190 if no sticky is set, but in this case an earlier check if the current transaction is actually stickied should be done to return without any form or logs that could indicate problems

Moved to https://github.com/casualcore/casual-caller/issues/3


Rejected transaction with type: EntityType(0x414C)

HashLockTransaction entity code has changed and this has not been reflected in the SDK.
After sending a HashLockTransaction, catapult returns the following error:

api-node-0_1              | 2019-02-07 13:45:06.497553 0x00007f1ebd869700:  (model::Transaction.cpp@31) rejected transaction with type: EntityType(0x414C)
api-node-0_1              | 2019-02-07 13:45:06.498598 0x00007f1ebd869700:  (ionet::PacketPayloadParser.h@34) entity (header size = 120) has failed validity check with size 176
api-node-0_1              | 2019-02-07 13:45:06.499150 0x00007f1ebd869700:  (handlers::HandlerUtils.h@39) rejecting empty range: packet Push_Transactions with size 184

HashLockTransaction entity code has changed and this has not been reflected in the SDK.
Previous type: 0x414C
New type: 0x4148
Related files:

https://github.com/nemtech/nem2-library-js/blob/master/src/transactions/HashLockTransaction.js#L35
https://github.com/nemtech/nem2-sdk-typescript-javascript/search?utf8=✓&q=0x414C&type=

review all of them: https://github.com/nemtech/catapult-rest/commit/d81a7417d5c5d6f4b129c555e856144ef8c849ba

I modified the changed types which were hashLock, secretLock and secretProof but also included some of the others to hold their uppercase representation, this is just to keep consistent on how we handle hexadecimal notation.


What's the best default percentile to display? (95th vs 99th)

I did some testing (please ignore the 95th % in the formatter, that's 99th my fault):
run.exs:
Name                  ips        average  deviation         median         95th %
flat_map           2.20 K      454.02 μs    ±10.18%         444 μs         758 μs
map.flatten        1.28 K      784.11 μs    ±22.11%         798 μs     1292.72 μs

Comparison:
flat_map           2.20 K
map.flatten        1.28 K - 1.73x slower

flat_map 95th: 482 μs vs. 99th: 758 μs vs. maximum: 1.54 ms
map.flatten 95th: 1.15 ms vs. 99th: 1.29 ms vs. maximum: 1.70 ms

multiple_map.exs:
Name                               ips        average  deviation         median         95th %
stdlib map                     63.66 K       15.71 μs    ±49.29%          15 μs          27 μs
bodyrecusrive map              63.45 K       15.76 μs    ±55.69%          15 μs          31 μs
map tco no reverse             61.32 K       16.31 μs    ±54.31%          16 μs          26 μs
map with TCO and reverse       55.24 K       18.10 μs    ±58.72%          17 μs          34 μs
map with TCO and ++             0.99 K     1007.89 μs    ±24.49%         911 μs     1525.45 μs

Comparison:
stdlib map                     63.66 K
bodyrecusrive map              63.45 K - 1.00x slower
map tco no reverse             61.32 K - 1.04x slower
map with TCO and reverse       55.24 K - 1.15x slower
map with TCO and ++             0.99 K - 64.17x slower

stdlib map 95th: 17 μs vs. 99th: 27 μs vs. maximum: 2.79 ms
map with TCO and reverse 95th: 21 μs vs. 99th: 34 μs vs. maximum: 3.57 ms
map with TCO and ++ 95th: 1.50 ms vs. 99th: 1.53 ms vs. maximum: 1.87 ms
bodyrecusrive map 95th: 17 μs vs. 99th: 31 μs vs. maximum: 2.96 ms
map tco no reverse 95th: 16 μs vs. 99th: 26 μs vs. maximum: 3.09 ms

multiple_map with a 100k list instead of 1k:
Name                               ips        average  deviation         median         95th %
stdlib map                      565.72        1.77 ms    ±10.09%        1.86 ms        2.21 ms
map tco no reverse              554.48        1.80 ms    ±10.09%        1.87 ms        2.40 ms
bodyrecusrive map               530.35        1.89 ms    ±11.10%        1.95 ms        2.56 ms
map with TCO and reverse        454.36        2.20 ms    ±12.77%        2.25 ms        2.79 ms
map with TCO and ++             0.0502    19929.24 ms     ±0.00%    19929.24 ms    19929.24 ms

Comparison:
stdlib map                      565.72
map tco no reverse              554.48 - 1.02x slower
bodyrecusrive map               530.35 - 1.07x slower
map with TCO and reverse        454.36 - 1.25x slower
map with TCO and ++             0.0502 - 11274.28x slower

stdlib map 95th: 2.00 ms vs. 99th: 2.21 ms vs. maximum: 3.11 ms
map with TCO and reverse 95th: 2.57 ms vs. 99th: 2.79 ms vs. maximum: 3.98 ms
map with TCO and ++ 95th: 19.93 s vs. 99th: 19.93 s vs. maximum: 19.93 s
bodyrecusrive map 95th: 2.17 ms vs. 99th: 2.56 ms vs. maximum: 3.12 ms
map tco no reverse 95th: 2.00 ms vs. 99th: 2.40 ms vs. maximum: 3.22 ms

My formatter code:
fn(suite) ->
    Enum.each suite.scenarios, fn(scenario) ->
      stats = scenario.run_time_statistics
      alias Benchee.Conversion.Duration
      IO.puts "#{scenario.job_name} 95th: #{Duration.format(stats.percentiles[95])} vs. 99th: #{Duration.format(stats.percentiles[99])} vs. maximum: #{Duration.format stats.maximum}"
    end
  end

My observations:

at least for big enough sample sizes 99th percentile seems to be different enough from maximum to be valuable
especially for fast functions they really show nicely the difference to hardcore outliers from maximum (e.g. multiple_map, both 95th and 99th are within ~2x of the average whereas maximum is ~200x)
sometimes 95th gets rather close to average/median but still far away enough/different enough

in the end: :woman_shrugging:
Sort of depends of how many values we think usually are outliers. Skylight uses 95th - but their context is different as they usually deal with a wide array of inputs and hence different response times while for us (usually) the inputs are the same and hence run time should be more stable and outliers should mostly be caused by GC/busy system.
Opinions highly welcome :+1:

No input so far :man_shrugging:
I guess 99th is good enough for now. wrk2 also seems to go with the 99th percentile so it's good enough I guess, we can still make the others configurable so people can have and use them.


Validation fixes

Fixed validation messages on the BE
Fixed missing return in validation messages component
Added missing validation in contacts section
Refactored code in both FE and BE
Beautified code

Thanks for the PR!
Any successful deployments (not always required) will be available below.
Backend available
Frontend available
Legacy available
Once merged, code will be promoted and handed off to following workflow run.
Main Merge Workflow

Some of the changes were too simple and unnecessary (like inverting the variable and the null on if checks) and some refactoring like extracting a repeating string into a variable instead of a constant was ok but too small to justify a new pull request.
The addition of new validators was the factor that really contributed to the addition of this new PR.
Also, keep in mind that if a variable is not required, it is always better to return the result of an operation instead of saving it into a variable before returning it.

The 1st comment is as per our code reviews standards and to avoid null pointer exceptions.
It was as per SonarCloud's recommendation and because it is easier to refactor 1 variable instead of multiple strings. i can explain this in more detail in a conversation.


New instance of javacomplete launched for each new java file

Actual behavior (Required!)
When we edit a file, the server is launched, but it is not stopped when we closed the file and exit vim.
Here, I open 4 files, with different names, and 4 instance are launched.
Expected behavior (Required!)
Server instance killed when exit vim
The steps to reproduce actual behavior (Required!)

Open multiple java files and use htop

Environment (Required!)

OS: Linux 4.4.5
Neovim version: Last production version. (EDIT: development version!)

Screenshot (Optional)

#217 same issue?

Many times this behavior was described. The better way I think is set a default value for g:JavaComplete_ServerAutoShutdownTime on the plugin, so we can avoid that.
@artur-shaik @wsdjeg What you guys think?

@wsdjeg @adelarsq This variable kills the server after x seconds, not when vim exits.

@dannywillems can you call :JCserverTerminate after server started up, and check is server still running or not.
Just can not to catch where this issue happens.

@artur-shaik Yes, the server still running.

@dannywillems, can you please, switch to this branch https://github.com/artur-shaik/vim-javacomplete2/tree/issue_234 and try the same again

I've got this error message when I first open a java file. Server still running after exit vim. (I didn't change the initial configuration, just cloned)

@dannywillems
you can try with this file
test.vim
set rtp+=/path/to/javacomplete2
autocmd FileType java setlocal omnifunc=javacomplete#Complete

and open vim by vim -u test.vim
then run command :e Foo.java,and :JCserverTerminate
and see if the java server still running

@dannywillems strange, cannot reproduce this issues. You can try, as @wsdjeg suggest, with minimal configuration. And will be nice if you could debug this issue...

@dannywillems just notice you are using neovim,can you give me the output of this script?I think maybe there is something wrong with you neovim's python config.
https://github.com/tweekmonster/nvim-python-doctor
also you can try with vim,I think it should works well

@wsdjeg: Here is the output
## Neovim Python Diagnostic

- Neovim Version: NVIM v0.1.4-195-g3d7a6e4

## Python versions visible to Neovim

### 'python' info from /usr/local/bin/nvim

WARN: 'g:python_host_prog' is not set
WARN: Fallback to '/usr/bin/python'
**Python Version**: `Python 2.7.6`
**Neovim Package Version**: `neovim (0.1.7)`

### 'python3' info from /usr/local/bin/nvim

WARN: 'g:python3_host_prog' is not set
WARN: Fallback to '/usr/bin/python3'
**Python Version**: `Python 3.4.3`
ERR: `/usr/bin/python3: No module named pip`

## Python versions visible in the current shell

- **python** version: `Python 2.7.6`
  - **neovim** version: `neovim (0.1.7)`
- **python3** version: `Python 3.4.3`
  - **neovim** version: `Not installed`

I use the neovim development version, not the last production (sorry, I edited my first comment).

@wsdjeg I have the same issue with vim. java kg.ash.javavi.Javavi ... still running.

as the script output shown,no issues about your config.
do you have this in your vimrc?
" python host for neovim
let g:python_host_prog = '/usr/bin/python2'
let g:python3_host_prog = '/usr/bin/python3'

@artur-shaik I think maybe #217 also do not be solved,but we do not know where the issue happend.

@wsdjeg I added it but the issue still present.

stange,but I can not catch this bug now,so sorry

@wsdjeg yeah, that was just a workaround. Will leave this issue open and hope that somebody can reproduce it.

Still no solution ? Very disturbing and boring to kill manually.

@dannywillems can you test issue_234 branch one time more?

You don't need now to switch to another branch, changes are in master now. Please try again.

The issue still present. Am I the only one who has this issue ?

@pvrs12 has this issue too. He made PR to fix it, seems like it works for him: https://github.com/artur-shaik/vim-javacomplete2/pull/240

Here maybe another issue:

I open a java file. The server starts and when I type ´´´import java.``` I have some autocomplete (I use YouCompleteMe).
I kill the server with JCserverTerminate -> The server is killed.
I type another import java. and the server restart and I have some autocomplete.

Is it normal?

Yeah, script autostart server if it is not running on completion trigger.

I guess if I kill the server with JCserverTerminate -> The server is killed. so issue is solved for you?

I must do it manually. It's not very convenient.

@dannywillems are you using neovim? have you tried in vim?

@artur-shaik I think this issue has be fixed,as I know @dannywillems is using YCM,and when he type into import java. the omnifunc will be auto started by YCM.so the server will auto restart.
and I think this issue is caused by
 let g:ycm_semantic_triggers =  {
                        \   'c' : ['->', '.'],
                        \   'objc' : ['->', '.'],
                        \   'ocaml' : ['.', '#'],
                        \   'cpp,objcpp' : ['->', '.', '::'],
                        \   'perl' : ['->'],
                        \   'php' : ['->', '::'],
                        \   'cs,javascript,d,python,perl6,scala,vb,elixir,go' : ['.'],
                        \   'java,jsp' : ['.'],
                        \   'vim' : ['re![_a-zA-Z]+[_\w]*\.'],
                        \   'ruby' : ['.', '::'],
                        \   'lua' : ['.', ':'],
                        \   'erlang' : [':'],
                        \ }

you need remove  \   'java,jsp' : ['.'],

@wsdjeg as I understand server still work after vim exit.

@artur-shaik @wsdjeg Yes, the server still working after exiting vim.

@wsdjeg Yes, same issue with vim. I think I will try without any other plugins and see if there is a conflict with another one. I don't have time now but I will try in some days. I will give you a feedback.

Actually that's strange, if JCserverTerminate works for, so must be do and on exit, too. Maybe you need bind JCserverTerminate manually, to run it on vim exit. This is how it runs by script:
autocmd VimLeave * call javacomplete#server#Terminate()

By the way, as writted in vim help, you can check previous exit code of vim, by run :echo v:dying and see if it equals to 0 or 1, thus if it's bigger so VimLeave not triggered on exit.

Any news?

@artur-shaik Don't have time to try something new for the moment, sorry.


When build settings are not selected in the pipeline configuration, the option 'Delete old builds' is still checked in Jenkins.

What is version of KubeSphere DevOps has the issue?
v3.4.1
How did you install the Kubernetes? Or what is the Kubernetes distribution?
kubekey
What happened?

Relevant log output
No response
Additional information
No response

/close