Add a .hasNext() assertion method for Iterator<?>

I'd have thought that this method already existed but it appears that in fact, no...
It'd be a nice-to-have; along with .doesNotHaveNext().
Comments?

I would prefer isExhausted

Yes indeed, that is a better name (or .hasNoMoreElements(); bah)

Why not use isNotEmpty? It does internally check for hasNext() and should semantically fit, should it?
Although the behavior is a little bit strange to me in the following code
        ArrayList arrayList = new ArrayList();
        arrayList.add("Luke");
        arrayList.add( "Leia");
        Iterator<String> it = arrayList.iterator();
        while (true) {
            assertThat(it).isNotEmpty();
            String next = it.next();
            System.out.println(next);
        }

java.util.NoSuchElementException

Your while condition is wrong, it should it.hasNext().
I think exhausted is the usual term for an Iterator that don't have any elements left.

I personally agree that the code extract of @AlexBischof should work.
Note that I haven't looked at the code of assertj in detail yet, but it looks to me that .isNotEmpty() just does something like this (and yes, I am quite sure that this is not how things work internally):
// "it" is the iterator here
public boolean isNotEmpty()
{
    try {
        underlyingIterator.next();
        return true;
    } catch (NoSuchElementException ignored) {
        return false;
    }
}

which in turn would trigger the NoSuchElementException in the code example.
Now, another important thing to bear in mind here is that an Iterable and and Iterator are not quite the same thing. An Iterable is supposed to produce a fresh Iterator each time you call its .iterator() method; but this is not quite the same thing as testing an Iterator in itself.
In other words, an Iterator produced by an Iterable is independent from any other Iterator produced by that same Iterable.
As such I believe that it is incorrect to "promote" the test of an Iterator to that of Iterable, and that dedicated assertions for Iterators are worth considering.
For instance:
assertThat(iterator).hasNext();
assertThat(iterator).nextElement().is*();
// or even chain

Comments?

Javadoc states clearly that assertions on an Iterator will consume it in most of the case. It was an easy solution to provide Iterator assertions.
Having said that, I agree that checking that an Iterator has elements left should not consume it.
@fge My gut feeling is that having specific Iterator assertions will require a lot of work if you want to support the richness of Iterable assertions.

+1 for a specific assertion because it would be a much cleaner integration.

I am interested in working on this. Is the initial analysis still valid after three years?

I think that @fge is right, the best solution would be to have Iterator assertions that only deals with the Iterator API including isExhausted(), hasNext(), ...
Navigational assertions like assertThat(iterator).nextElement().isEqualto(...); would be nice to have but are tricky to implement because of soft assertions proxying, I'd rather create a separate issue for them.

Yes, I would like to work on that.
It is very likely that there will be questions regarding the toterable(iterator) and asIterable() methods, but I will ask them here when I get there.

Okay, first question for my initial commit. I have some issues satisfying the generic constraints for the tests, especially the unchecked warning in AbstractIteratorAssert. Do I need something like ConcreteIterableAssert as an intermediate type?

@stovocor can you create a PR ? It makes it easier to review changes, thanks!

can you create a PR ? It makes it easier to review changes, thanks!

I do not think that the changes are ready to be merged, but I created the PR as you asked.

that's alright, it just to have a look at it and give you some feedback for the next steps

@joel-costigliola I just rebased this against the current master. Can you please tell me how I should continue with this pull request?

@stovocor I'll have alook this we and let you know (remind me if I fail to!)


set_radio, set_check

I'm not quite sure of the purpose of this check of the form method in the form_helper::set_radio and form_helper::set_checkbox.
What it has done for me is prevent it from working with GET forms. Seems odd that this check of the method is not also checked if the submitted value is an array.
if ($CI->input->method() === 'post')
{
	return ($input === $value) ? ' checked="checked"' : '';
}

I've overridden it with my own methods, but seems like it should be removed altogether?

Seems odd that this check of the method is not also checked if the submitted value is an array.

The value comes from $_POST, so if $_POST[$field] is an array, we know that this check would match.
But with anything other than arrays, an unchecked radio/checkbox will simply not be submitted, resulting in $value = null, which is the same thing as the form not being submitted at all, so the only way to check that is to look at the request method.


Remove configSet timeout in JedisCommandTestBase

It was added in https://github.com/xetorthio/jedis/commit/cd9e17a70976a8d5093703206da4b2ab0d1bb770#diff-fc3c922202222a7dce6ca86f63e2ef07R31 without any explanation.

@HeartSaVioR @marcosnils

Merged into master, 2.10, 2.9.


Limit test duration

Fixes ooni/probe#1280

Adds a migration function that

removes the url_limit option
adds two new keys websites_enable_max_runtime and websites_max_runtime

Use setTimeout to start a timer in node.js to kill the child_process
Cancel the timer when manually stopped in UI
Disable the numeric input when the first boolean option is turned off

Disabled

When max_runtime is enabled the Estimated time left should be that of the maxruntime and not that returned by probe-engine:

When max_runtime is enabled the Estimated time left should be that of the maxruntime and not that returned by probe-engine:

@hellais I have adressed the progress bar and ETA comments. I noticed a bug where the config context wasn't updated after onboarding. It seemed to happen only in that context. But the fix I added basically checks if the value in ConfigContext is null and fetches the config to refresh the context. This check happens whenever useConfig hook is called.

@hellais I have adressed the progress bar and ETA comments. I noticed a bug where the config context wasn't updated after onboarding. It seemed to happen only in that context. But the fix I added basically checks if the value in ConfigContext is null and fetches the config to refresh the context. This check happens whenever useConfig hook is called.

The progress bar and ETA are not working as expected.
The progress keeps flickering between the runtime given out by probe-engine and the overridden progress.
Moreover the status update messages are not being displayed anymore.
See at around 1m of the video:
https://user-images.githubusercontent.com/424620/104020438-8495a300-51bd-11eb-8a40-cb31f14fec19.mov

The progress bar and ETA are not working as expected.
The progress keeps flickering between the runtime given out by probe-engine and the overridden progress.
Moreover the status update messages are not being displayed anymore.
See at around 1m of the video:
https://user-images.githubusercontent.com/424620/104020438-8495a300-51bd-11eb-8a40-cb31f14fec19.mov


Hibernate Validator is great! But...

Could we have "this thing"?

@bot say something

@bot say-hello

@bot say-hello Yoann

@bot say something


Script editor autocompletion for "exit"

Autocompletion in the scritpt editor for the function "exit" has a small bug. See the image below.

Looking at the functions.html, it appears that this exit function is the only one that has multiple invocations separated by an or outside the parentheses.
@haesleinhuepf Our priority should be to make sure that functions.html works as written automatically without tweaking, right? So that we can easily keep it updated as time goes on by dropping in new versions? If so, we need to write some special-case parsing for this odd case, right?

I updated the file at
http://wsr.imagej.net/developer/macro/functions.html
so that exit and exit("error message") are separate entries.

exit

Terminates execution of the macro.

exit("error message")

Terminates execution of the macro and displays an error message.

@haesleinhuepf I tried to update the locally bundled functions.html resource to finish addressing this issue, but it seems there are many differences between the local copy and the one on the remote site. So I wasn’t sure what the process is for updating it? Did you make modifications manually? If so, do you have automation to remake them when things change? And can we commit that script to the repo? Or if not: would it make sense to create such a script?


⚠ ClamAV has degraded performance

In b0e6807, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 72 ms

Resolved: ClamAV performance has improved in e219d24.


Chapter 5 data extraction code does not work as Apache mailing list has changed its interface

The Apache mailing list has changed its interface, and it is not anymore mod_mbox of Apache HTTP, hence url like http://mail-archives.apache.org/mod_mbox/spark-dev/201911.mbox/ajax/thread?0 will cause the error because of /ajax part.

By removing /ajax, the url http://mail-archives.apache.org/mod_mbox/spark-dev/201911.mbox/thread?0 mailing list URL redirect to new interface<EMAIL_ADDRESS>November 2019 but it does not provide MBOX format listing, hence cannot extract the MBOX format elements such as FROM, TO, SUBJECT.
The thread ID pattern is now different too, e.g. https://lists.apache.org/thread/hg85hhvt270of8fdrmb62kfvm7rpl96p.

@oonisim Hi!
Is this the end of study for this book or any chances to get the data?

May be here is our mailing list
<EMAIL_ADDRESS>But how to get data in xml format?


dashboard: allow widget locking

dashboard 24.7 : allow widget locking

@fichtner That's part of the 24.7 release. I copied the dashboard.css a few days too early. Then came this improvement, which of course wasn't in my css file yet.

@opnsenseuser sure, I merged after building 24.1.9 so we don't need to modify versions again. Thanks!


Filter by specialisation

As an NUS computing student, I wish to filter my list of contacts on NetworkBook by specialisations, so that I can easily find and connect with individuals from a particular field of study.


Incompatibility in returned errors between 1.9.3 and 1.10.0

Before the update of the 'send' module the code pasted below returned the error (next(err)), but now it doesn't. With the old version of 'send' the 'err' object had no 'statusCode' property => so !(err.statusCode < 500) returned true. With the new version of 'send' err.statusCode returns some http code (for example 403) and  !(err.statusCode < 500) is false.
// forward errors
stream.on('error', function error(err) {
if (forwardError || !(err.statusCode < 500)) {
next(err)
return
}
  next()
})

Hi! I'm sorry, I don't completely understand the report. Can you provide a test case? Perhaps even a PR where you add a test that fails before your change and succeeds after your change"

Hi, I didn't notice the new 'fallthrough' property. It works fine when I set it to 'false'.


go module recommendations: prefer stretchr/testify/assert to  zeebo/assert?

In Go I use the library https://github.com/stretchr/testify for some minor ergonomics improvements for testing. VSCode always instead autosuggests this module: https://github.com/zeebo/assert . I have no reason to think there is anything wrong with that module, but it has two stars, one developer, no issues, and no development in over a year. In comparison stretchr/testify is extremely active and gets a lot of eyes.
zeebo/assert has over a thousand projects depending on it, including some big names. I assume in most cases that is due to VSCode autocompletion rather than conscious selection. Should the auto-recommendation behavior be changed?

Looks more like Go extension behavior or something else, not built in VS Code one (code search doesn't show zeebo) in Thai repo.
Where do you get that suggestion from?

@IllusionMH You may be right about it being the go extension. If I disable that the behavior disappears. However I also lose the recommendations, which I want to keep. Anyway got a bit more info:

If I blow away all traces of zeebo from ~/go, it is no longer recommended. So the recommendations are coming from my filesystem.
In a fresh project with no dependencies, the recommendations are nondeterministic. I get variously pointed at stretchr/testify/assert, alecthomas/assert, zeebo/assert (when it was on my fs), etc. I imagine these are all very similar libs with similar function signatures.
In my projects that import stretchr/testify/assert and none of the others, I still am recommended to use the alternatives. I think maybe this is where the opportunity for improvement is - it seems like the recommendations should bias toward modules already imported by the project, or possibly weigh the module popularity.

I can follow this up in vscode-go if that seems like the appropriate place.

/causedByExtension
It was just simplest step to check that it's not VS Code built-in suggestion.
You can try Ctrl+. (or something close, can't check ATM) to see which extension provides that suggestion.

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines. If you don't know which extension is causing the problem, you can run Help: Start extension bisect from the command palette (F1) to help identify the problem extension.
Happy Coding!


Handle missing files after cargo rustc

Some flags to the compiler could cause it to stop early or not emit some files
altogether (or perhaps emit files in different locations even). Currently cargo
expects a few outputs of the compiler after cargo rustc is run, but this
commit alters cargo to know that when cargo rustc is being run that the
outputs may not exist and that's ok.
Closes #1675

r? @huonw
(rust_highfive has picked a reviewer for you, use r? to override)

r? @brson

@bors r+

:pushpin: Commit 8230f1f has been approved by brson

:hourglass: Testing commit 8230f1f with merge f98100f...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64


Prepare release 4 2

Overview
This PR is to prepare release 4.2.

Updated release howto (we dropped develop branch).
Updated changelog.

Related Issue / Discussion
See pywps-dev mailing list and gitter chat.
Additional Information
Infos for changelog taken from:

milestone 4.2
https://wiki.osgeo.org/wiki/Annual_General_Meeting_2018

Contribution Agreement
(as per https://github.com/geopython/pywps/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

[x ] I'd like to contribute [feature X|bugfix Y|docs|something else] to PyWPS. I confirm that my contributions to PyWPS will be compatible with the PyWPS license guidelines at the time of contribution.
[x ] I have already previously agreed to the PyWPS Contributions and Licensing Guidelines

@huard maybe you like to add something to the changelog.

... test failures due to opendap service downtime:
http://test.opendap.org
http://test.opendap.org/opendap/

Coverage remained the same at 74.864% when pulling ad0654ea2817a03cc87d8e5f11028ddadfbb3515 on bird-house:prepare-release-4_2 into dccec7d0f29285185f914ae3c44e068d7c871e46 on geopython:master.

Coverage remained the same at 74.864% when pulling ad0654ea2817a03cc87d8e5f11028ddadfbb3515 on bird-house:prepare-release-4_2 into dccec7d0f29285185f914ae3c44e068d7c871e46 on geopython:master.

thank you for picking it


Coding style

I think some thought needs to be had about coding style.
Currently local variable to instance variable adds a variable directly under the last instance variable with a single blank line below.
This often leads to multiple blank lines added below if one already exists, this is a simple bug to fix (I'll do that one shortly).
My issue however is I like a blank line above as my instance variables generate have a docblock so I like my definitions like so:
/**
 * @var string
 */
private $name;

/**
 * @var int
 */
private $age

However I'm guess it currently goes directly underneath as whoever implemented that feature styles there code that way?  I'm guessing like so:
private $name;
private $age;

Neither of these are right or wrong but it's got me thinking that maybe we need an approach to generate the refactors in different coding styles? Anyone have any thoughts on this? Maybe having a config file which defines your preferred style rules?

Sounds like a good idea.


Write tests for the transforms

in src/transform.rs

[ ] av1_fwd_txfm2d_4x4

[ ] av1_fwd_txfm2d_8x8

[ ] av1_fwd_txfm2d_16x16

[ ] av1_fwd_txfm2d_32x32

[ ] av1_inv_txfm2d_add_4x4

[ ] av1_inv_txfm2d_add_8x8

[ ] av1_inv_txfm2d_add_16x16

[ ] av1_inv_txfm2d_add_32x32

@lu-zero would something equivalent to these make sense?
https://aomedia.googlesource.com/aom/+/067e984254e658863b8c6dc335e26f92ecc32c1c/test/transform_test_base.h#47
Namely: accuracy tests and bit exactness tests.

It feels relatively overkill for an easy task.
We can have it with proptest, but it is harder than just do what we have for get_sad.

Solved by #697

Closing as resolved.


Update README.md of silk-icons-fa module to fix a typo

Update README.md of the silk-icons-fa module to fix a typo in the div class

Image from @bitspittle

Thanks!


Image NOT Selected on New File Upload

When uploading a new image, the new image is NOT selected by default.
This causes the user to have to search for the file which can be difficult on a large site with many files.
Is there a setting that will make sure the uploaded image is selected by default when the upload process completes?

We have found the same problem in DNN 9.2.2. On sites with many images you have to look up the image again with paging, which takes a lot of time and actions. Actually, e.e.a. is not workable!

Is it possible for the editor to recognize when only a single file has been uploaded and "select" the file by default?

Good morning @w8tcha .  Would you be able to provide some input on this topic?  I think it is a valid concern affecting many users.  Thank you in advance for the quality module.  You are appreciated.

It was working before. I will look in to it. But at the moment i dont have time for it

Hi @w8tcha, as i post before. We have a large site and it's for our users not fine to work with he editor.
Is there a SettingValue in the CKE_Settings file we'll have to change? What's the SettingName? Thanks!

The problem is that you can uploaded multiple images. and then selecting all wouldnt work. I dont think there will be a solution for this problem.

Ingo, wouldn't it be possible to check, whether just a single file has been uploaded?
or always picking the first one just being uploaded (from the upload string)?
IMO that would be helpful!

This would work but this would not solve the problem when paging is enabled

This would be very helpful to allow if paging is not enabled.

When uploading one file it's not done to select the file after uploading. On large sites this action takes i lot of time and you cannot search for the file! When paging is enabled you don't know on which page the file is been displayed. So i agree with the smbcreativegroup, automatic select
tbe upload file when paging is not enabled! Thank you on behalf of many users.

@w8tcha are you able to implement some feature to accommodate this issue?
Either selecting the image if a single file is uploaded.  Or select the first image if multiple images are uploaded and the community knows that this will only work if PAGING is Disabled.
Thank you for any update on this issue.

issue is fixed with https://github.com/w8tcha/dnnckeditor/releases/tag/v02.01.26

Thank you for adding this to the new release!


⚠ Develop API Endpoint has degraded performance

In 67b6ffb, Develop API Endpoint (http://develop.roblox.com/) experienced degraded performance:

HTTP code: 200
Response time: 2566 ms

Resolved: Develop API Endpoint performance has improved in b8a04ea after 8 minutes.


⚠ Processors has degraded performance

In 6479ff9, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 69 ms

Resolved: Processors performance has improved in 19cb9a9.


Category tree component displays wrong values

The category tree does not accurately reflect the state of selected components. The problem probably only appears after the selection has been saved once.
In the screenshot the categories “Toys, Movies, Books” and “Outdoors Jewelry” are selected and can be seen in the select field, however the tree component only marks the categories above as selected but not the actually selected categories.
Expected: The categories “Toys, Movies, Books” and “Outdoors Jewelry” should have a blue select box with a white checkmark in front of them.
!image-20240815-115502.png|width=1195,height=912,alt="image-20240815-115502.png"!

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-37718.
[created from NEXT-37718, comment 487670]


Issues opening a remote file when there is a swap file already in place

When using netman to open a remote file, if a swap file already exists, netman will fail the open with the following error
2023-02-25T08:55:48 Neo-tree  ERROR Error opening file: Vim(append):Error executing lua callback: /home/miversen/git/netman.nvim/lua/netman/init.lua:62: Vim:E325: ATTENTION
stack traceback:
        [C]: in function 'nvim_buf_set_name'
        /home/miversen/git/netman.nvim/lua/netman/init.lua:62: in function 'read'
        /home/miversen/git/netman.nvim/lua/netman/api.lua:265: in function </home/miversen/git/netman.nvim/lua/netman/api.lua:261>
        [C]: at 0x55e172ee3fc1
        [C]: in function 'pcall'
        ...cal/share/nvim/lazy/neo-tree.nvim/lua/neo-tree/utils.lua:510: in function 'open_file'
        ...miversen/git/netman.nvim/lua/netman/ui/neo-tree/init.lua:920: in function 'generate_node_children'
        ...miversen/git/netman.nvim/lua/netman/ui/neo-tree/init.lua:1003: in function 'navigate'
        ...rsen/git/netman.nvim/lua/netman/ui/neo-tree/commands.lua:16: in function 'func'
        ...are/nvim/lazy/neo-tree.nvim/lua/neo-tree/ui/renderer.lua:780: in function <...are/nvim/lazy/neo-tree.nvim/lua/neo-tree/ui/renderer.lua:778>

Not sure the best approach here, but we shouldn't be dying if the user tells vim to open the file even if it has as swap file already.

We need to follow up on this, I am having trouble reliably replicating this outside of netman. Which is... frustrating.

Neovim#23145 was closed with :h SwapExist as the answer.
From what I can tell, this is an event that is fired when a swap file is found that exists for a requested buffer. It would be wise to see if we can hook this and "handle" the swap event somehow (since Netman has a way to keep track of which files it currently has open, we might be able to steal the swap file or whatever).

We don't actually need to use SwapExist for this. There was a bug in how we were setting the buffer before loading it. I am not quite sure why this bug existed but it has been resolved with this commit


Failing API with update in OpenSSL 3.0.x

I have updated OpenSSL in the existing project of iOS. It takes a P12 file as input and then with the help of OpenSSL APIS it gives the data as Data. Then further this Data object is passed to Apple API SecPKCS12Import to import the detail of the certificate.
Since my P12 file using the old API mentioned in this URL: (https://www.openssl.org/docs/man3.0/man1/openssl-pkcs12.html#:~:text=The default encryption,provider-path option), I am using the legacy provider.
This flow was working fine with the older version of OpenSSL 1.x but with the update in OpenSSL 3.x an Apple API SecPKCS12Import is failing with the following error:
Error code: OSStatus -25293
Message: SecPKCS12Import Optional(Error Domain=NSOSStatusErrorDomain Code=-25293 "The user name or passphrase you entered is not correct." UserInfo={NSLocalizedDescription=The user name or passphrase you entered is not correct.})
In Investigation, I found the difference in data bytes after converting P12 using OpenSSL 1.x and OpenSSL 3.x in the same P12 file.
STR:

Find the attached P12 file or Create the p12 certificate using the following URLs:
https://www.akadia.com/services/ssh_test_certificate.html
Or https://stackoverflow.com/questions/21141215/creating-a-p12-file
Use the following code :

BIO* createPKCS12fromPKCS12(const unsigned char* data, long dataLength, char* originalPassphrase, char*
newPassphrase) {
OSSL_PROVIDER *legacy = OSSL_PROVIDER_try_load(NULL, "legacy", 1);
OSSL_PROVIDER *defaultProvider = OSSL_PROVIDER_try_load(NULL, "default", 1);
BIO* bp = BIO_new_mem_buf(data, (int)dataLength);
PKCS12* originalPKCS12 = NULL;
d2i_PKCS12_bio(bp, &originalPKCS12);
BIO_free(bp);

EVP_PKEY* privateKey;
X509* x509;
STACK_OF(X509)* caChain = NULL;
PKCS12_parse(originalPKCS12, originalPassphrase, &privateKey, &x509, &caChain);
PKCS12_free(originalPKCS12);

PKCS12* newPKCS12 = PKCS12_create(newPassphrase, "SAP Identity", privateKey, x509, caChain, 0, 0, 0, 0, 0);
EVP_PKEY_free(privateKey);
X509_free(x509);
sk_X509_free(caChain);

BIO* mem = NULL;
if (newPKCS12 != NULL) {
    mem = BIO_new(BIO_s_mem());
    i2d_PKCS12_bio(mem, newPKCS12);
    PKCS12_free(newPKCS12);

}

return mem;

}
+(NSData*)NSDataFromBIO:(BIO*)mem {
NSData* data = nil;
if (mem != NULL) {
char* ptr = NULL;
size_t size = BIO_get_mem_data(mem, &ptr);
data = [NSData dataWithBytes:ptr length:size];
BIO_free(mem);
}
return data;
}
let contentsOfFile = try Data(contentsOf: fileURL) //fileURL: file URL of the PKCS #12 file
let mem = createPKCS12fromPKCS12Data(data, passphraseOriginal: passphrase, passphraseNew: "")
let pkcs12Data = NSDataFromBIO(mem)
Then do the following:
SecPKCS12Import(pkcs12Data as CFData, query as CFDictionary, &items)
Technology used:
Xcode: 14.0.1
OpenSSL: 3.0.5
iOS: 16
The result expected: SecPKCS12Import pass with no errors
Issue: SecPKCS12Import is failing with data conversion after an update in OpenSSL 3.x

If you create a pkcs12 file with the openssl pkcs12 -legacy command, can you successfully import it with SecPKCS12Import?

Hi @t8m, I tried the certificate created with the following command, but it didn't work for me.
openssl pkcs12 -legacy -export -out certificate.p12 -inkey key.pem -in certificate.pem

Have you tried looking at the data using

openssl pkcs12 -info -in  myfilename.p12

The defaults were changed in 3.0 to use
PBKDF2, AES and SHA-256, with a MAC iteration count of PKCS12_DEFAULT_ITER.
Try using the -legacy option and then look at the output also to compare with the old 1.1 version.
Maybe SecPKCS12Import only handles AES not 3DES

Hi @slontis
I have compared the data of my certificate with the legacy command (OpenSSL 3.0.5) and with (OpenSSL 1.x). There is a minor difference in the data printed, but the certificate is the same.
Also, I have a certificate with AES and one with 3DES. SecPKCS12Import works for both with OpenSSL 1.x but SecPKCS12Import fails for both with OpenSSL 3.x with the same error.
Data of certificate with OpenSSL1.x:
Enter Import Password:
MAC Iteration 2048
MAC verified OK
PKCS7 Encrypted data: Certificate bag
Bag Attributes
    localKeyID: D3 CB 52 6F 97 47 3D 2E 10 DA 6F C0 7F B9 22 46 32 2D BE 4C
subject=/C=IN/ST=hg/L=hgvgh/O=hvhg/OU='cgg/CN=gvgh/emailAddress=cgg
issuer=/C=IN/ST=hg/L=hgvgh/O=hvhg/OU='cgg/CN=gvgh/emailAddress=cgg
-----BEGIN CERTIFICATE-----
<...>
-----END CERTIFICATE-----
PKCS7 Data
Shrouded Keybag: Bag Attributes
    localKeyID: D3 CB 52 6F 97 47 3D 2E 10 DA 6F C0 7F B9 22 46 32 2D BE 4C
Key Attributes: <No Attributes>
Enter PEM pass phrase:
Verifying - Enter PEM pass phrase:
-----BEGIN ENCRYPTED PRIVATE KEY-----
<...>
-----END ENCRYPTED PRIVATE KEY-----

Data of certificate with OpenSSL3.x:
openssl pkcs12 -legacy -info -in MAFTEST.p12
Enter Import Password:
MAC: sha256, Iteration 2048
MAC length: 32, salt length: 8
PKCS7 Encrypted data: PBES2, PBKDF2, AES-256-CBC, Iteration 2048, PRF hmacWithSHA256
Certificate bag
Bag Attributes
    localKeyID: D3 CB 52 6F 97 47 3D 2E 10 DA 6F C0 7F B9 22 46 32 2D BE 4C
subject=C = IN, ST = hg, L = hgvgh, O = hvhg, OU = 'cgg, CN = gvgh, emailAddress = cgg
issuer=C = IN, ST = hg, L = hgvgh, O = hvhg, OU = 'cgg, CN = gvgh, emailAddress = cgg
-----BEGIN CERTIFICATE-----
<...>
-----END CERTIFICATE-----
PKCS7 Data
Shrouded Keybag: PBES2, PBKDF2, AES-256-CBC, Iteration 2048, PRF hmacWithSHA256
Bag Attributes
    localKeyID: D3 CB 52 6F 97 47 3D 2E 10 DA 6F C0 7F B9 22 46 32 2D BE 4C
Key Attributes: <No Attributes>
Enter PEM pass phrase:
Verifying - Enter PEM pass phrase:
-----BEGIN ENCRYPTED PRIVATE KEY-----
<...>
-----END ENCRYPTED PRIVATE KEY-----

The p12 file you show above is the new default format.. Try printing out the old -legacy one...

I tried the same thing with my old p12 file
for OpenSSL 1.x
openssl pkcs12 -info -in MAFTESTOldest.p12
Enter Import Password:
MAC Iteration 100000
MAC verified OK
PKCS7 Data
Shrouded Keybag: pbeWithSHA1And3-KeyTripleDES-CBC, Iteration 50000
Bag Attributes
    friendlyName: maftest
    localKeyID: 54 69 6D 65 20 31 36 32 39 37 37 34 38 33 39 37 35 36
Key Attributes: <No Attributes>
Enter PEM pass phrase:
Verifying - Enter PEM pass phrase:
-----BEGIN ENCRYPTED PRIVATE KEY-----
<...>
-----END ENCRYPTED PRIVATE KEY-----
PKCS7 Encrypted data: pbeWithSHA1And40BitRC2-CBC, Iteration 50000
Certificate bag
Bag Attributes
    friendlyName: maftest
    localKeyID: 54 69 6D 65 20 31 36 32 39 37 37 34 38 33 39 37 35 36
subject=/C=DE/CN=MAFTEST
issuer=/C=DE/O=SAP-AG/CN=DEV-SMPSDK-AFARIA7-CA
-----BEGIN CERTIFICATE-----
<...>
-----END CERTIFICATE-----
Certificate bag
Bag Attributes
    friendlyName: CN=DEV-SMPSDK-AFARIA7-CA,O=SAP-AG,C=DE
subject=/C=DE/O=SAP-AG/CN=DEV-SMPSDK-AFARIA7-CA
issuer=/C=DE/O=SAP-AG/CN=DEV-SMPSDK-AFARIA7-CA
-----BEGIN CERTIFICATE-----
<...>
-----END CERTIFICATE-----

for OpenSSL 3.x
openssl pkcs12 -legacy -info -in MAFTESTOldest.p12
Enter Import Password:
MAC: sha1, Iteration 100000
MAC length: 20, salt length: 20
PKCS7 Data
Shrouded Keybag: pbeWithSHA1And3-KeyTripleDES-CBC, Iteration 50000
Bag Attributes
    friendlyName: maftest
    localKeyID: 54 69 6D 65 20 31 36 32 39 37 37 34 38 33 39 37 35 36
Key Attributes: <No Attributes>
Enter PEM pass phrase:
Verifying - Enter PEM pass phrase:
-----BEGIN ENCRYPTED PRIVATE KEY-----
<...>
-----END ENCRYPTED PRIVATE KEY-----
PKCS7 Encrypted data: pbeWithSHA1And40BitRC2-CBC, Iteration 50000
Certificate bag
Bag Attributes
    friendlyName: maftest
    localKeyID: 54 69 6D 65 20 31 36 32 39 37 37 34 38 33 39 37 35 36
subject=C = DE, CN = MAFTEST
issuer=C = DE, O = SAP-AG, CN = DEV-SMPSDK-AFARIA7-CA
-----BEGIN CERTIFICATE-----
<...>
-----END CERTIFICATE-----
Certificate bag
Bag Attributes
    friendlyName: CN=DEV-SMPSDK-AFARIA7-CA,O=SAP-AG,C=DE
subject=C = DE, O = SAP-AG, CN = DEV-SMPSDK-AFARIA7-CA
issuer=C = DE, O = SAP-AG, CN = DEV-SMPSDK-AFARIA7-CA
-----BEGIN CERTIFICATE-----
<...>
-----END CERTIFICATE-----

The above certificate works in OpenSSL (1.x) including SecPKCS12Import but SecPKCS12Import does not work in OpenSSL 3.x (with legacy).

I cant see any difference from those outputs, maybe look at the raw data to see where they are different...

didn't get the meaning of raw data here. how to get raw data from the certificate

You could use either openssl asn1parse OR
dumpasn1 my.p12 to see what is different. (dumpasn1 gives better output).

You can get it working with using the -legacy -certpbe pbeWithSHA1And40BitRC2-CBC parameters with the openssl pkcs12 -export command with the OpenSSL 3.x.x Version.

When generating the .p12 file with the command:
openssl pkcs12 -export -out certificate.p12 -in certificate.pem -inkey key.pem -password "pass:${myPassword}"
The double quotes around the password caused them to be included as part of the actual password.
This resulted in SecPKCS12Import failing with the error -25293 (incorrect password).
Removing the double quotes from the password in the OpenSSL command resolved the error without requiring the use of the -legacy flag.

Marked as inactive to be closed at the end of 3.5 development period.


Enhancement: Normalize composer.json

This PR

[x] normalizes composer.json

💁♂ For reference, see https://github.com/localheinz/composer-normalize.

@localheinz
I think, you should create a suggestion at zendframework/maintainers, because your plugin would be a help for all components.

@froschdesign

I think, you should create a suggestion at zendframework/maintainers, because your plugin would be a help for all components.

Thank you! What do you suggest, opening an issue, or a pull request?

We actually already have a tool in the maintainers repo for normalization. However, instead of putting the keys in alphabetical order, we have been grouping them based on functionality (e.g., require, require-dev, conflict, and suggest are grouped, in that order). IIRC, we already did that in this repo, so I'm going to close this issue; we can revisit if we decide to order them differently later.


Provide functionality for named arguments

I like the distinction between opt and arg, but opt allows naming them when passing them and arg does not. Could we add that for arg as well. Thank you

Are you looking for this: https://github.com/backuity/clist/blob/master/tests/src/test/scala/org/backuity/clist/SingleCommandParsingTest.scala#L142 ?

That wouldn't allow me to pass the argument using --name as it's done in
opt would it? It would just give it a name in the usage string . I'd
like to be able to say --arg for args as well
On 20 Apr 2016 12:19, "Bruno Bieth"<EMAIL_ADDRESS>wrote:
Are you looking for this:
https://github.com/backuity/clist/blob/master/tests/src/test/scala/org/backuity/clist/SingleCommandParsingTest.scala#L142
?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/backuity/clist/issues/11#issuecomment-212385715

Oh I see.

Should be soon available on maven central as 2.1.0.
See test cases:

https://github.com/backuity/clist/blob/master/tests/src/test/scala/org/backuity/clist/SingleCommandParsingTest.scala#L30
https://github.com/backuity/clist/blob/master/tests/src/test/scala/org/backuity/clist/SingleCommandParsingTest.scala#L37

Thanks !!


add japan academic agency

@ryusuke-j This domain does not belong to a particular school. It seems to belong to an umbrella institution (such as Ministry of Education or similar government body). We cannot add umbrella institutions domain to this repository. Please apply for free educational license with an official document that proves your student status. You can do it using "OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students.
Please send an email to<EMAIL_ADDRESS>if you need a license to install to a number of computers in a classroom. Please email<EMAIL_ADDRESS>if you need an academic discount for an academic institution other than a university or a college. Thank you for your understanding!


Remove duplicate null check

Removed because hasText() contains a null check.

Codecov Report

Merging #1730 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1730      +/-   ##
==========================================
+ Coverage   75.16%   75.18%   +0.01%
==========================================
  Files         199      199
  Lines        6205     6205
  Branches      777      777
==========================================
+ Hits         4664     4665       +1
  Misses       1200     1200
+ Partials      341      340       -1

Impacted Files
Coverage Δ

...cloud/netflix/zuul/filters/SimpleRouteLocator.java
81.52% <100%> (+1.08%)
:white_check_mark:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1d5a68c...cb9d9bd. Read the comment docs.


rebasing take action from production

@cgettings thank you! that was gonna be a leave-it-for-monday thing for me so i appreciate you. yeah let's replicate your setup.


Unable to install coursier

Hi,
First of all, thanks for building this GitHub action, this is great!
I've been having problems when running it though. I'm literally copying and pasting the example from https://github.com/scala-steward-org/scala-steward-action#usage and the action is failing with: Unable to install coursier
Here is the output of the step:
Launch scala-steward-org/scala-steward-action@v2
  with:
    github-token: ***
    scala-steward-version: 0.5.0-385-e5e4789c-SNAPSHOT
    ignore-opts-files: true
    sign-commits: false
✓ Connected to Maven Central
##[error] ✕ Unable to install coursier

Also tried fixing the version to v2.1.0 and v2.0.0 and it also fails.
Is anyone else experiencing the same?

Hey @marcelocarlos, thanks for reporting this and I'm glad you find it useful :)
Can you enable debug mode in your Github Actions? This can be done by adding a secret to your repository named ACTIONS_STEP_DEBUG with the value true. And then re-run the action so we can get more information on why it cannot install Coursier?
Thank you!!

Of course, here it is the output:
##[debug]Evaluating condition for step: 'Launch Scala Steward'
##[debug]Evaluating: success()
##[debug]Evaluating success:
##[debug]=> true
##[debug]Result: true
##[debug]Starting: Launch Scala Steward
##[debug]Loading inputs
##[debug]Evaluating: secrets.ADMIN_GITHUB_TOKEN
##[debug]Evaluating Index:
##[debug]..Evaluating secrets:
##[debug]..=> Object
##[debug]..Evaluating String:
##[debug]..=> 'ADMIN_GITHUB_TOKEN'
##[debug]=> '***'
##[debug]Result: '***'
##[debug]Loading env
Run scala-steward-org/scala-steward-action@v2
✓ Connected to Maven Central
##[debug]Downloading https://git.io/coursier-cli-linux
##[debug]Destination /home/runner/work/_temp/bbe1660a-10e9-49e6-bafc-38e02ca81513
##[debug]download complete
##[debug]EXDEV: cross-device link not permitted, rename '/home/runner/work/_temp/bbe1660a-10e9-49e6-bafc-38e02ca81513' -> '/home/runner/bin/cs'
##[error] ✕ Unable to install coursier
##[debug]Node Action run completed with exit code 1
##[debug]Finishing: Launch Scala Steward

Is this being run on Github Action's runner or in a self-hosted runner? Also, is the action using ubuntu-latest as the running system or other OS?

Is this being run on Github Action's runner or in a self-hosted runner? Also, is the action using ubuntu-latest as the running system or other OS?

Yes, tried ubuntu-latest (as in the example of the README) and ubuntu-18.04 as well. Both cases running on Github Action's runner.

I've created a simple test repository: https://github.com/alejandrohdezma/test-scala-steward-action, executed the action and it run fine.
Can you check if there is something different between your workflow and this one?

Thanks! I'm getting a 404, is that a private repo?

Ups! Sorry! I forgot gh creates them as private by default. Should be public now

No worries. My workflow is exactly the same, except for the trigger (mine is on push while I'm testing it). Tried in a different repo, it also fails there.
It seems related with this line? https://github.com/scala-steward-org/scala-steward-action/blob/77cbae442ec3b1b259affee3c6b00a985f6a9d4f/src/coursier.ts#L20

Yeah, that's what I think too. I've been trying to replicate this issue on different repositories (public/private, creating a new Github user...) without luck :(
I'll try to dig tomorrow into the actual error you posted from the debug logs to see I can get at least the root cause

I'm getting the same error as well: https://github.com/bpg/ss-gh-test/runs/695346873?check_suite_focus=true
I'm very new to GH Actions, but let me explain my train of thought, if it might help in investigation.
I think the difference in behavour between repos/builds could be due to slightly different GH Actions build agents, the way how project workspace is mounted inside the agent's container, etc... I'm a total nub in nodejs, but i think the .mv call is actually trying to rename the file. So if the source and target are on different volumes inside the container, the rename call will fails. According to the debug log above this is most likely what happens there.
So if we can switch from the "lame" .mv to actual move command (i.e. copy+delete, or just copy -- who cares what's left there :p ) that would be great!

Thank you very much @bpg for that tip, I think that can be the solution! 🎉
I just uploaded a branch with the fix @bpg suggested.
You can use it by setting your "version" to commit hash:
uses: scala-steward-org/scala-steward-action@07b63bf19f36bd08a077967ce592e872ee9ea171

I tried it in my own test repo and it is working.
Can you both try with that version to check if that fixes the issue? Thanks! 😸

Thanks @alejandrohdezma
I've just tried the new version, it is still failing unfortunately:
Run scala-steward-org/scala-steward-action@07b63bf19f36bd08a077967ce592e872ee9ea171
✓ Connected to Maven Central
##[debug]Downloading https://git.io/coursier-cli-linux
##[debug]Destination /home/runner/work/_temp/4e4e32f6-1342-4c09-9d4c-f8c7ff7b075e
##[debug]download complete
##[debug]EXDEV: cross-device link not permitted, rename '/home/runner/work/_temp/4e4e32f6-1342-4c09-9d4c-f8c7ff7b075e' -> '/home/runner/bin/cs'
##[error] ✕ Unable to install coursier

As an alternative, you could extract to a location and add that location to the PATH, so you don't need to move it to /home/runner/bin. It looks like this is what this action does: https://github.com/GoogleCloudPlatform/github-actions/blob/master/setup-gcloud/src/install-util.ts#L41

My bad, I totally forgot to execute npm run build to re-generate the action'
s binaries. This is the actual hash for the cp + rm fix:
uses: scala-steward-org/scala-steward-action@1dbeccd09b252b37a31750ad1cae96e13fa1a5ef

Great! That worked 🎉
Thanks a lot!

Thanks @alejandrohdezma, that worked for me too!

I'm really happy that worked :)
I just send a PR with the change, once tests passes I'll merge and re-publish v2.
Thank you both very much! 😸

v2 release has been updated to v2.1.1


[RFC] Introduce Response Content Matcher and Response Loader

I decided to extract some logic from api test case for future integration.

@lchrusciel what about namespace ? I was not sure where i should place those services.

@Arminek I add comments with my proposal. Haven't seen it before :dash:

:+1:

@Arminek are you willing to rebase and apply above comments?


Properly support "between" operator for valueSource: "field" in default ValueEditor

The reason it doesn't support that is because RQB would need to become more opinionated, and I don't really want to go down that path

Ahh got it, thanks for bearing with me, makes total sense. This… actually leaves me a bit uncomfortable? In that, the UI allows users to build something that is silently ignored.

Originally posted by @fasiha in https://github.com/react-querybuilder/react-querybuilder/issues/215#issuecomment-1146446614
Requirements:

ValueEditor should display two drop-down lists when operator is "between" or "notBetween" and valueSource is "field".
The value should be stored as a comma-separated string (e.g. "f1,f2").

Questions:

Should the default ValueEditor include a separator of some kind between the two drop-downs, like "and" or ","?

Should it be translatable?
Should it be stylable (have standard classnames)?
What would the default styles be?

When a value (i.e. field) is selected in one drop-down list, should it not be available in the other list? This would prevent rules like "f1 between f2 and f2". Maybe we could handle that with validation, or skip those rules in formatQuery.
What to do if there are less than two qualifying fields to populate the drop-downs? If one, maybe we just allow it. If zero...(what happens when there are no qualifying fields for other operators?)
Should valueSource: "value" rules also have two inputs? Respecting inputType: "number", etc.?

Whew, definitely one of those things that sound easy but are fractally complicated—as usual your attention to detail stands out!
I don't want to discourage from spending the time needed to find the global optimum in terms of features/behaviors but given that the library currently doesn't work (appears to accept between for fields but outputs no rules), I wonder if doing the minimum to fix it—while also keeping with the standard set by the rest of the library—might be advisable. If so, that'd mean:

no separator between the two dropdowns (but, low-hanging fruit: do give the dropdowns unique classnames so CSS can add before and after content?)
no other validation on the content of the dropdowns
valueSource:'value' continues behaving as is

And see how that makes you feel?

That's excellent feedback. Thanks! TBH I think I'm just going to implement your suggestions as written. I'll throw a PR together soon.

OK so this ballooned a little but I think it's for the best. See PR #337.
When I started working on handling the "between" operator for valueSource: "field", I realized RQB could handle "between" rules with valueEditorType: "select" (regardless of the valueSource) with the same code. So that was a nice surprise.
I also added an optional boolean prop to <QueryBuilder /> called listsAsArrays. This works similarly to the parse* option of the same name. When true, rules that have operators which take a list of values ("between", "in", and their "not" counterparts) will update the value property with an actual array, like this:
const query = {
  combinator: "and",
  rules: [
    {
      field: "f1",
      operator: "between",
      value: ["f2", "f3"], // <--
      valueSource: "field",
    },
  ],
};

The default behavior will stay the same, using comma-separated strings:
const query = {
  combinator: "and",
  rules: [
    {
      field: "f1",
      operator: "between",
      value: "f2,f3", // <--
      valueSource: "field",
    },
  ],
};

Strings can now handle escaped characters, too, like commas. For example, RQB will ignore the escaped comma when splitting a value of "this\, that, the other", splitting it into ["this, that", "the other"]. Joining will do the same in reverse.
As for styling: when two or more inputs are displayed by ValueEditor (currently only applicable for "between" and "notBetween"), they will each have the class "rule-value-list-item". The default CSS will add a margin-left of the standard spacing amount, 0.5rem, to the second input (and any subsequent ones, if we eventually support "in"/"notIn" in the same way).
I’d love to see what you think of the changes. #337 has links to codesandboxes that implement the new features so you can try them out without cloning/installing/etc.


arch/arm/src/stm32f7/stm32_i2c.c: Round up stm32_i2c_toticks return v…

…alue
When sending small number of bytes with larger CONFIG_USEC_PER_TICK this function should return at least 1. Solve this by rounding up the result.
Signed-off-by: Jukka Laitinen<EMAIL_ADDRESS>Summary
This function should always return at least 1. For example with values
bytecount == 1
CONFIG_USEC_PER_TICK == 10000 (default)
CONFIG_STM32F7_I2C_DYNTIMEO_USECPERBYTE = 500
Since USEC2TICK does rounding to nearest, the calculation in ticks would result in (((500 * 1) + (10000 / 2)) / 10000) == ( 5500 / 10000 ) == 0
Solve this by rounding it up (adding CONFIG_USEC_PER_TICK / 2 - 1), which always results 1 tick at 1 byte regardless of the selected USEC_PER_TICK or STM32F7_I2C_DYNTIMEO_USECPERBYTE values.
Impact
Fixes the i2c on stm32f7 with default USEC_PER_TICK setting
Testing
Tested on stm32f7 based board

@xiaoxiang781216 I realized what caused the issue in the first place. I am not sure what is the best way forward so I created an issue about it. see https://github.com/apache/nuttx/issues/8346

Sure, let's discuss there.


Power Automate integration for Powertoys Run

Description of the new feature / enhancement
A plugin for powertoys run that allows you to run flows created in power automate

Scenario when this would be used?
Power Automate is a useful utility that lets you automate repetitive tasks, but currently to run any 'flow' one has to open the app and do it from there; It would be useful if you could do that right from powertoys run, the utility already has plugins for other apps like OneNote and VS code and this would make an excellent addition to the tool's capabilities.
Supporting information
No response

Can try to work on this! Shall I go ahead?


LRO basics from CADL

First level of support for LRO in CADL, is to consider the presence of @Azure.Core.pollingOperation as a boolean switch to generate LRO code, like x-ms-long-running-operation: true was doing in Swagger:
  @Azure.Core.pollingOperation(Authoring.createProjectStatusMonitor)
  createProject is Azure.Core.LongRunningResourceCreateWithServiceProvidedName<Project>;

(full example)

ref https://github.com/Azure/autorest.java/issues/1684

@lmazuel
Java might additional like to have the type information on polling result, as Java API response is of PollerFlux<IntermediateModel, FinalResultModel>.
Wondering whether there is actually
createProjectStatusMonitor is CustomCore.PollingOperation<ProjectMonitorModel>;

that the response of monitor API is the ProjectMonitorModel, instead of the Project as final result.

Also SDK need to hide "createProjectStatusMonitor" operation as it is included in polling.

basically this task is done as DPG 1.0 fashion.
the remaining question is whether SDK should do special handling for the status monitor API.

autorest https://github.com/Azure/azure-rest-api-specs/pull/20808/files#diff-1848b2ecf27a48dfecaaefb41cb4cf78b172f44b97d66911337f3a8442b571ad

I believe we should consider the status api as any method, and we may add a decorator to say "this method private only please" ar something like that. Thoughts?

I believe we should consider the status api as any method, and we may add a decorator to say "this method private only please" ar something like that. Thoughts?

Decorator will surely do!
We keep all API at same state, till we got feedback from service or arch that something needs special handling.

completed via https://github.com/Azure/autorest.java/pull/1715


Fix params/typehints for Challenge

Domain should be a Domain object, subdomains an array. Enfore using typehints.
Also set subdomains empty array by default.

Thanks!


docs: remove alpine installation related warning

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/gravitee-io/keeper?version=dev:06b5f2509e2cfe4996a00a41bb73eb641ee8f956

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/gravitee-io/keeper?version=dev:8750036000b6462166bf37bcc7c0d061914e1532


sql: fix the handling of tuples in distsql

PR: https://github.com/cockroachdb/cockroach/pull/28143
From release notes:

CockroachDB now supports empty tuples with the syntax (), 1-valued tuples with the syntax (x,) in addition to row(x), and the ability to use IN with an empty tuple as right operand. This is a CockroachDB extension. [#28143][#28143] {% comment %}doc{% endcomment %}

Don't think docs are needed for this


Custom text for marker label

Hello, i'd like to set marks on chart with custom text, for example - "A", "B", "C" etc. Is it possible?

Hello! You can use a MarkerLabelFormatter for this purpose. MarkerComponent has a labelFormatter field.

Could you provide a sample code on how you would go about that? The formatter only provides basic x and y values as far as I'm concerned. I'm missing an index value or something similar.

Please see here, @BlazeCodeDev.

Please see here, @BlazeCodeDev.

Page Not Found :((

Hello, @jalalaghazadeh. Based on other threads, it looks like you're using Vico 2, so please see here and here.


Add option to save exact versions

Fixes #90

@dylang any updates?

@pgilad Thanks for doing this! I'm reviewing it now. Sorry for lack communication on my part.

Looks great, merging it in now! Thanks again!

Thanks for the project! Cheers


checking on value instead of key

Should be checking on params[k] for None values rather than the k.

sorry this needs to be -
for k in list(param.keys()):
    if params[k] is None:
        del(params[k])

else you'll get an error like this-
--> 265         for k in params:
    266             if params[k] is None:
    267                 del(params[k])

RuntimeError: dictionary changed size during iteration


[Feature] Scrape Results From Barnivore

Requesting scraping results from Barnivore as well.
There are no FLOSS apps on F-Droid that scrape Barnivore.
https://www.barnivore.com/

Moving to the server. But we don't scrape. It would need an API, or a compatible data licence.


INTG-28: Document new features

New PR: https://github.com/alfameCom/mule-bpm-module/pull/66
This will be closed because of broken commit history


ci: general refresh of github workflows, update gha versions and let dependabot do it, etc.

Just passing by trying to keep this repo up to date with various practices across the JupyterHub organization, fixing details such as ensuring the use of a pip cache is robust.
This should not have any influence on the actual function of repo2docker, just the CI system.

@minrk I've validated the tests to complete successfully after adding the commit 11f2d2d
I'm going for a self merge and get another PR open on top of this with some more optimizations to shave off ~10 minutes of wait for developers. Currently we wait ~50 minutes for these tests to pass.


lock whisper files during carbon-sync and whisper-fill

as discussed on #47, the option isn't symmetric between sync and fill (config file vs command line) however fill doesn't take a config file anyway. Also I noticed whisper-fill is included in graphite's distribution too so that will need updating as well.

also note this is missing some prose about the new option and its implication when syncing running/live clusters, I can take a stab at that but I'm no native speakr, @jssjr you'd do a much better job I guess :)

updated to reflect latest master, thoughts?

@jssjr thoughts on this? I think it'd be important to have and avoid multiple writers corrupt whisper files

Nah, maybe I nitpicking.

Looks like more test need to be fixed

Fixed in https://github.com/graphite-project/carbonate/pull/71


Need help

Hi guys, could you please provide me example of Sortable > Multiple containers > Vertical grid. I'll be very grateful, thanks!

https://github.com/clauderic/dnd-kit/blob/master/stories/2 - Presets/Sortable/MultipleContainers.tsx


bosh login - copy button is broke and getting authentication failure with UAA

What version of UAA are you running?
Version: 74.5.4
How are you deploying the UAA?

concourse pipeline

What did you do?
When using bosh login and getting redirected to the cloud foundry site to get the one time passcode, we get the following when we click the copy button:
copy button.pdf
15:25 $ bosh login
Using environment '<REDACTED>'

Email ():
Password ():
Temporary Authentication Code ( Get one at https://<REDACTED>:8443/passcode ) ():

Failed to authenticate with UAA

Email ():
Asking for text:
  EOF

Exit code 1

After looking at UAA logs, restarting UAA processes via monit, my teammate selected the 1 time passcode and copied it / pasted it into the terminal and things work fine.  The copy button is broke
We tried:

Chrome (regular & incognito)
Safari
Firefox

I also checked with our InfoSec team to see if they deployed any kind of new network policies, they did not.
What did you expect to see? What goal are you trying to achieve with the UAA?
I expected the copy button to copy, and paste into the terminal when I paste.  The copy button does not copy at all.
What did you see instead?
Failed to authenticate with UAA

Email ():
Asking for text:
  EOF

Exit code 1

Here is some info from the browser dev console:

Refused to execute inline event handler because it violates the following Content Security Policy directive: "script-src 'self'". Either the 'unsafe-inline' keyword, a hash ('sha256-...'), or a nonce ('nonce-...') is required to enable inline execution. Note that hashes do not apply to event handlers, style attributes and javascript: navigations unless the 'unsafe-hashes' keyword is present.

Workaround
We do have a workaround in place, however it would be nice if the copy button working again.

Thanks for reporting. The content security policy prevents now that java script is executed...
@pabloarodas @bruce-ricard  Can we invest here to extract the script into a file so that we can call the copy or should we add an exclusion next to where saml and implicit is excluded ?

I will schedule this fix.

Ok, I played with local UAA and therefore created now https://github.com/cloudfoundry/uaa/pull/2034 which should fix it

@adampeklay In the issue you mentioned that the bug existed in Version 74.5.4, which I don't think has the copy button (or the Content-Security-Policy header that would break the button). Did you mean version for 75 or above?

@peterhaochen47 This issue is about a month old, I'm sure I was accurate with the version number.  Feel free to go through the release to answer your question https://github.com/cloudfoundry/uaa-release/releases?page=1


The path '/browse/{authorSlug}' does not work

The FAQ says the following:

How can I keep track of new or updated test cases made by a specific user?
Every author has its own Atom feed, located at http://jsperf.com/browse/author-name.atom. Omit the .atom suffix to get a clickable list instead. If you’re identified on jsPerf (i.e. if you’ve commented, or created/edited a test case), a “My tests” link will appear in the navigation.

The source code appears to support this feature.
However these URLs just redirect to the home page

https://jsperf.com/browse
https://jsperf.com/browse/styfle
https://jsperf.com/browse/styfle.atom

I recently created a test under the name "styfle" in case my other tests were deleted, however still no luck.

jsperf.com is still running the old PHP version

This has been disabled on the old version. When we switch to v2 it will magically start working again.

When is the switch to v2 happening?


[Bug] Animation does not work very well whe driven by watch in a vue application

1. Describe the bug
I'm using motion one in a Vue project where I want to do the following:

A gsap timeline drives the animation of a value
I pass the value to a component as a prop
The value should drive a motion one animation

When I'm trying to call the animate() function in a Vue watcher with requestAnimationFrame it does not work (Example 1 in Codesandbox). If instead use a requestAnimationFrame loop it does (Example 2 in Codesandbox). Please comment in/out the code accordingly.
2. IMPORTANT: Provide a CodeSandbox reproduction of the bug
https://codesandbox.io/s/confident-mendeleev-pvgqmc?file=/src/components/Animation.vue
3. Steps to reproduce
Steps to reproduce the behavior:

Open Codesandbox Link
Example does not work
Comment out Example 1 and comment in Example 2
Example does work

4. Expected behavior
I'm not sure if its a limitation of the library or if I'm doing something wrong. My expectation would be that I can drive the animation with the watcher. This for example works as expected when using gsap: https://codesandbox.io/s/charming-easley-4gw4gp?file=/src/components/Animation.vue

Is there a reason why you try and animate these values in both situations rather than just setting them via style
element.style.setProperty("--motion-x") = props.x + "px"

Likewise these sandboxes aren't equivalent. Here's one mixing rAF with GSAP and it also stutters for me https://codesandbox.io/s/elastic-shtern-5n62vy?file=/src/components/Animation.vue

@mattgperry Yeah you're right, the second sandbox didn't use rAF, my bad. I believe, I did that because gsap uses requestAnimationFrame internally anyway.
However with your sandbox example at least some kind of motion is happening:
https://github.com/motiondivision/motionone/assets/227756/73a8ad0d-b379-4fcb-b159-852970b65afa
Whereas with the first sandbox from my initial post , there's no motion at all:
https://github.com/motiondivision/motionone/assets/227756/9c064e16-b72b-4161-a84c-27569f0f1a40
The video screencasts are recorded with Firefox 117 — I just noticed that Chrome it works fine in Chrome 🤷 Now it's just the question if it's a bug in the Browser or in Motion or Vue? 😅
The reason why I'm using animate instead of setting the values directly is that I want to have some drag, think of a element following the cursor, similar to your example in this issue: https://github.com/motiondivision/motionone/issues/43#issuecomment-1030835933


Add an option to prioritise download chapters based on some criteria

The idea
When downloading a volume, if there are multiple uploads of the same chapter by different groups, there is currently no way to decide which chapter takes priority unless you blacklist specific groups. The --group option only allows you to choose one group to download from.
If there are many chapters uploaded by different groups in the same language, it would be nice to have options for the following:

Prioritise one group over all others but still download from other groups if no chapter is available from the priority group.
Prioritise the most recently uploaded chapter - which may contain fixes or improvements over the older upload(s).
Prioritise the largest chapter (in terms of file size) for the highest scan quality available or prioritise the smallest chapter, for saving storage space.

Example usage: --priority {GROUP_ID,newest,oldest,largest,smallest}
Why this feature should be added to the app ?
It will give us greater control over what we download. At the moment, I have to download all of the chapters using --group "all" and then manually delete the chapters I do not want. Without specifying --group "all, mangadex-dl seems to randomly decide which upload to download if there are multiple uploads of the same chapter in the same language. Giving the user control over which chapter mangadex-dl prioritises would be a great improvement to the user experience.

For your information, --group option does support multiple groups or users and this behaviour is exists since version 2.6.0. I just forgot to update the documentation so it only show usage for single group (Writing documentation is pain).
mangadex-dl "mangadex_url_here" --save-as cbz-volume --group "group_id_1" --group "group_id_2" --group "group_id_3"

v2.6.0 changelog
The implementation
However, i don't deny your ideas to prioritise chapters and it might be useful for people who want total control for their downloads. I will add this on v3.0.0 to-do list
But uhhhhhh, i may don't have time to do code (especially doing documentation) because college times is crazy these days. So don't expect this feature is implemented earlier.

Automation is great, but even the program asking for feedback would be workable when doing volumes, like printing the options and asking the user to select:
ex.
"Volume. 5 Chapter 19 has (2) groups:
1. Group 1 name
2. Group 2 name
Please enter the number you wish to download into the volume:

Automation is great, but even the program asking for feedback would be workable when doing volumes, like printing the options and asking the user to select:

I may agree on program asking for feedback on user. But here's the problem, you don't wanna program asking for a question in a middle of download, the user will be mad if program acted like that. Most of the time, they just leave the program and do anything else.
Based on your example it's possible to ask feedback like that when users downloading a individual chapter, the question will be asked before download begin. However when user downloading a manga or a MangaDex list, the program can't just ask question like that, the program will constantly asking each chapter to user and they get annoyed.
So yeah, i agree to this idea when it's implemented for downloading a chapter. But it's not on my priority list since the program already has --group option. You might want to create new feature request about this, so other people can help adding this feature including me.

Hello, this feature has been added in commit https://github.com/mansuf/mangadex-downloader/commit/bd25b3ceaa74a98f43f8131c5646fa9e8a120b43 https://github.com/mansuf/mangadex-downloader/commit/32203fa171fb5620c855d6df9cb84f7c6a947a4e. You can test it by installing development version.
Installation with Git & Pip:
pip uninstall mangadex-downloader
pip install -U git+https://github.com/mansuf/mangadex-downloader.git@e7e58653649c803f8961ce4a9a7e1541a62e7837

Installation from Github Artifact CI (bundled executable):
You can download it from here in "artifacts" section

I didn't add --priority option, but rather adding 2 options --order and --group-nomatch-behaviour
--order option is used to change chapter ordering based on when the chapter released and can be read (newest or oldest).
--group-nomatch-behaviour is for change behaviour chapters group filtering (--group). The supported values are fallback and ignore. Default behaviour is ignore which is ignoring the chapter group that doesn't match with --option. If you set it to fallback the app will try to find another chapters if none of any group matching from --group option.
Usage
# Change order chapters to the newest released chapters
mangadex-dl "insert mangadex url" --order newest

# Find another chapters if any of group filtering didn't match
mangadex-dl "insert mangadex url" --group "group_1" --group "group_2" --group-nomatch-behaviour fallback

If you have problem or bugs, let me know 👍.


Updates to New/Set-MigrationBatch

Per Issue #1642

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/move-and-migration/New-MigrationBatch.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/move-and-migration/Set-MigrationBatch.md
:white_check_mark:Succeeded
View (exchange-ps)

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


Add Unit Tests and TS definitions for new applyMiddleware functionality

See #124

Also, we should clean up the applyMiddleware import (see #124 discussion).


Merge new ui into dev.

Coverage increased (+9.3%) to 63.982% when pulling a7b83c80e4ecb7454076f78bf7d8bfaf2be82713 on dev-new-ui into d7327a70d224ad0ef4c53478e79033ee01f0c312 on dev.

Coverage decreased (-2.9%) to 51.704% when pulling e20a5e62643ad48ecc6795e65fdfd4ce918ed482 on dev-new-ui into d7327a70d224ad0ef4c53478e79033ee01f0c312 on dev.

Coverage decreased (-3.002%) to 51.704% when pulling 6c337bbee1d8227365605703e11b1ad823725e6a on dev-new-ui into 37ea4273e15a616627da1581e377508d7736aa04 on dev.


I found below issue when trying to import tensorflow-hub - have no clue with this issue and appreciate your help.

I found below issue when trying to import tensorflow-hub - have no clue with this issue and appreciate your help.
ImportError Traceback (most recent call last)
in ()
1 import tensorflow as tf
----> 2 import tensorflow_hub as hub
C:\Program Files\Anaconda3\envs\tensorflow_sessions\lib\site-packages\tensorflow_hub_init_.py in ()
27 # error message is thrown instead of an obscure error of missing
28 # symbols at executing the imports.
---> 29 from tensorflow_hub.estimator import LatestModuleExporter
30 from tensorflow_hub.estimator import register_module_for_export
31 from tensorflow_hub.feature_column import image_embedding_column
C:\Program Files\Anaconda3\envs\tensorflow_sessions\lib\site-packages\tensorflow_hub\estimator.py in ()
23 from absl import logging
24 import tensorflow as tf
---> 25 from tensorflow_hub import tf_utils
26 from tensorflow_hub import tf_v1
27
C:\Program Files\Anaconda3\envs\tensorflow_sessions\lib\site-packages\tensorflow_hub\tf_utils.py in ()
31 # depending on TensorFlow internal implementation details.
32 # pylint: disable=g-direct-tensorflow-import
---> 33 from tensorflow.python.feature_column import feature_column_v2
34 # pylint: enable=g-direct-tensorflow-import

What environment are you importing, what is the version of Tensorflow_hub? Also can you give us the exact steps you followed for us to reproduce the issue. Thanks!

@venkatamaguluri Are you still facing the same issue?

its a Anaconda windows environment - Tensorflow 1.10.0 and Tensorflow_hub 0.7.0
Thanks - Venkata

@venkatamaguluri I am also running into this error using Tensorflow 1.10.0 and 1.11. Please start using  versions of Tensorflow that are 1.12 or beyond that.
I would strongly recommend you to use Tensorflow 1.15 or Tensorflow 2.x.

@venkatamaguluri Can you please let me know if the above comment helps. Thanks!

i donot find mentioned versions in annaconda, how can find mentioned versions?

@venkatamaguluri, First open anaconda navigator then select your environment and then search for tensorflow package. You will find 1.15.0 version.

some how i donot see it in my environment

@venkatamaguluri , select All in place of Installed or try to search after updating index.

@venkatamaguluri I can see that this issue is related to your environment not TF Hub itself. Can you please post it in conda/conda and close the issue here. Thanks!

Closing this issue as it has been awaiting response for more than 2 weeks. Please add additional comments and we can open this issue again. Thanks!


how declare dff without reset ??

Type of issue: bug report | feature request | documentation | other enhancement

Impact: no functional change | API addition (no impact on existing code) | API modification | unknown

Development Phase: request | proposal
Other information

If the current behavior is a bug, please provide the steps to reproduce the problem:
What is the current behavior?
What is the expected behavior?
Please tell us about your environment:

What is the use case for changing the behavior?

Type of issue: feature request

Type of issue: feature request

https://scastie.scala-lang.org/t0tsPfXGSFmJAmiOYc1kkA

https://scastie.scala-lang.org/t0tsPfXGSFmJAmiOYc1kkA

Why not just:
RegNext(input)

On 15 Jan, 2021, at 7:41, Jiuyang Liu<EMAIL_ADDRESS>wrote:
https://scastie.scala-lang.org/t0tsPfXGSFmJAmiOYc1kkA https://scastie.scala-lang.org/t0tsPfXGSFmJAmiOYc1kkA
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/chipsalliance/chisel3/issues/1734#issuecomment-760688146, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAE63GCAYVIWPLDUGTFFXQTSZ7PRNANCNFSM4WDSUUBA.

Why not just:
RegNext(input)

On 15 Jan, 2021, at 7:41, Jiuyang Liu<EMAIL_ADDRESS>wrote:
https://scastie.scala-lang.org/t0tsPfXGSFmJAmiOYc1kkA https://scastie.scala-lang.org/t0tsPfXGSFmJAmiOYc1kkA
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/chipsalliance/chisel3/issues/1734#issuecomment-760688146, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAE63GCAYVIWPLDUGTFFXQTSZ7PRNANCNFSM4WDSUUBA.


hash filter on load active state

going to a page with hash + filter in url loads up fine and filters fine, but I'm trying to get the active state to show as well...  this is only an issue when the url comes from another page, once on the page clicking filters creates active states no problem...
I tried the following with hard coded hash (since I wasn't sure if i was passing the hash correctly) but even hard coded hash don't seem to work.
link: https://goo.gl/NGwfBA
//changes the top center text
$('.mixitup-control[data-filter=".doctorofeducation"]').trigger('click');
//bolds the filter link
$('.mixitup-control[data-filter=".doctorofeducation"]').addClass('mixitup-control-active');

You can see from the source code I tried them inside a window.onload function, the getSelectorFromHash function and the getFilterFromHash function...
I'm not sure what else to try...
thanks in advance!

Hello, apologies for the late reply. Did you manage to fix this? I checked out the link and it appears to be working fine. What do you mean by the "active state"?
FYI you might want to use history.replaceState() instead of history.pushState() so that a browser history item isn't created each time you filter. There's a comment in the demo code where you can make this change.
I will close for now, but let me know if you want me to reopen.


Cannot read properties of undefined (reading 'uri')

TypeError: Cannot read properties of undefined (reading 'uri')
at <anonymous> in /linkedin.linkedin-multiproduct-vscode-plugin-pack-0.1.122/dist/extension.js:7:611021
at f.invoke in src/vs/base/common/event.ts:701:17
at g.deliver in src/vs/base/common/event.ts:908:22
at d.fire in src/vs/base/common/event.ts:869:24
at <anonymous> in src/vs/workbench/api/common/extHostDocuments.ts:47:28
at f.invoke in src/vs/base/common/event.ts:701:17
at g.deliver in src/vs/base/common/event.ts:908:22
at d.fire in src/vs/base/common/event.ts:869:24
at g.acceptDocumentsAndEditorsDelta in src/vs/workbench/api/common/extHostDocumentsAndEditors.ts:162:28
at g.$acceptDocumentsAndEditorsDelta in src/vs/workbench/api/common/extHostDocumentsAndEditors.ts:67:8
at l.N in src/vs/workbench/services/extensions/common/rpcProtocol.ts:455:17
at l.M in src/vs/workbench/services/extensions/common/rpcProtocol.ts:440:32
at l.H in src/vs/workbench/services/extensions/common/rpcProtocol.ts:370:19
at l.G in src/vs/workbench/services/extensions/common/rpcProtocol.ts:296:10
at <anonymous> in src/vs/workbench/services/extensions/common/rpcProtocol.ts:161:42
at f.invoke in src/vs/base/common/event.ts:701:17
at g.deliver in src/vs/base/common/event.ts:908:22
at d.fire in src/vs/base/common/event.ts:869:24
at p.fire in src/vs/base/parts/ipc/common/ipc.net.ts:671:19
at <anonymous> in src/vs/workbench/api/node/extensionHostProcess.ts:247:22
at f.invoke in src/vs/base/common/event.ts:701:17
at g.deliver in src/vs/base/common/event.ts:908:22
at d.fire in src/vs/base/common/event.ts:869:24
at p.fire in src/vs/base/parts/ipc/common/ipc.net.ts:671:19
at MessagePortMain.<anonymous> in src/vs/workbench/api/node/extensionHostProcess.ts:126:41
at MessagePortMain.emit (node:events:526:28)
at Object.MessagePortMain._internalPort.emit (node:<REDACTED: user-file-path>:5:364)

Go to Errors Site

This seems to be an error thrown from an extension linkedin.linkedin-multiproduct-vscode-plugin-pack.
Given our apparent push for reducing telemetry for errors in areas where we cannot act on, here I suggest to drop any telemetry event where the error originates from an extension.

This seems to be an error thrown from an extension linkedin.linkedin-multiproduct-vscode-plugin-pack.
Given our apparent push for reducing telemetry for errors in areas where we cannot act on, here I suggest to drop any telemetry event where the error originates from an extension.

I believe we're already supposed to be doing this, but @sbatten and I will look into what's going wrong here.


[Core] Support FileSourceScanExec driver metrics for spark3.4

What changes were proposed in this pull request?
Driver metrics in FileSourceScanExece are missed for spark 3.4.
After Spark 3.4 driverMetrics is changed to Map.empty[String, SQLMetric], metrics are still updated to it, but it's not added to the object FileSourceScanExecTransformer.metrics. So they are missed from UI.
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
Before:

After:

@JkSelf  and @PHILO-HE , could you help to review? Thanks.

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query
log/native_4848_time.csv
log/native_master_03_03_2024_6b5ee6971_time.csv
difference
percentage

q1
35.69
36.61
0.918
102.57%

q2
24.24
24.50
0.260
101.07%

q3
37.41
39.42
2.010
105.37%

q4
39.47
38.53
-0.936
97.63%

q5
68.78
70.78
1.995
102.90%

q6
6.99
7.30
0.313
104.48%

q7
84.81
84.97
0.158
100.19%

q8
85.13
84.52
-0.616
99.28%

q9
126.83
121.76
-5.069
96.00%

q10
44.02
44.77
0.746
101.69%

q11
22.49
20.74
-1.746
92.24%

q12
27.13
29.46
2.329
108.58%

q13
44.98
45.96
0.981
102.18%

q14
20.90
21.41
0.508
102.43%

q15
31.12
27.97
-3.147
89.89%

q16
13.21
13.22
0.016
100.12%

q17
102.83
103.13
0.292
100.28%

q18
142.61
146.21
3.606
102.53%

q19
12.52
13.66
1.139
109.09%

q20
26.04
26.57
0.533
102.05%

q21
227.27
225.15
-2.126
99.06%

q22
13.74
13.79
0.044
100.32%

total
1238.21
1240.41
2.207
100.18%

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query
log/native_master_03_06_2024_time.csv
log/native_master_03_03_2024_6b5ee6971_time.csv
difference
percentage

q1
35.69
36.61
0.916
102.57%

q2
24.09
24.50
0.404
101.68%

q3
37.17
39.42
2.248
106.05%

q4
37.31
38.53
1.221
103.27%

q5
70.29
70.78
0.485
100.69%

q6
7.07
7.30
0.229
103.24%

q7
84.52
84.97
0.447
100.53%

q8
86.24
84.52
-1.719
98.01%

q9
125.77
121.76
-4.015
96.81%

q10
42.91
44.77
1.861
104.34%

q11
20.70
20.74
0.040
100.20%

q12
29.30
29.46
0.159
100.54%

q13
45.63
45.96
0.332
100.73%

q14
20.55
21.41
0.860
104.19%

q15
29.07
27.97
-1.096
96.23%

q16
13.92
13.22
-0.701
94.97%

q17
102.68
103.13
0.445
100.43%

q18
145.05
146.21
1.166
100.80%

q19
12.61
13.66
1.052
108.34%

q20
25.67
26.57
0.899
103.50%

q21
224.29
225.15
0.860
100.38%

q22
13.74
13.79
0.044
100.32%

total
1234.28
1240.41
6.138
100.50%


SW-1901 Enable Sticky Header

change thead to TableHead
removing TableContainer

@charlie-tf could you test if drag and drop columns work in accessions table?

@charlie-tf could you test if drag and drop columns work in accessions table?

@constanzauanini Both d-n-d and sorting work:

@charlie-tf could you test if drag and drop columns work in accessions table?

@constanzauanini Both d-n-d and sorting work:

Excellent! Thank you!


Making view profile page [HACKTOBERFEST]

Describe the feature
when someone clicks on view profile button it will show a profile page of that freelancer
cab
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[ ] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@Ketanop321 Issue assigned to you

pls give me this issue


Update document to add wamr-rust-sdk introduction

Guess we don't need .github/workflows/rust_linter.yml and language-bindings/rust/.gitignore


Cosmetic changes for demo

@stevenmburns , this is ready for review. Converted select info messages to debug.


Not sound in iOS

How are you?
Audio Sound does not work in iOS. (working well in Android).
it seems like play. (i can get the total duration and position changed event).
but i am not able to listen the sound in iOS.
i added this to the info.plist file.
	<key>NSAppTransportSecurity</key> 	<dict> 		<key>NSAllowsArbitraryLoads</key> 		<true/> 	</dict>
Could you please let me know what i am missing?
Thank you.

Have a look at #1194


wireframe crashes.

To reproduce:

Open VAPOR, load data /little_glade/shaomeng/synthetic/sphere/sphere512nc, create a wireframe renderer, enable it.
Delete the current renderer.
Click "New Renderer" button.
Vapor crashes.

This gets fixed with PR #968.  I'll close once it gets merged.


Exclude specific models when caching is enabled in BaseModel

Somewhat self-explanatory, but in short, we have hundreds of models in play. We are currently the BaseModel/use Cachable; approach. It works great! However, there are a handful of models that we don't want to cache. Instead of having to remove the caching trait from the BaseModel and having to add it into hundreds of models, it would be great if we could somehow exclude certain models from being cached.
Could be via some other trait (use NotCacheable) or perhaps an exclusionary array of model names in the config file. Thoughts?

Hi @sburkett, thanks for writing in! I believe you should be able to add the following protected property on those models:
protected $isCachable = false;

Let me know how that works for you.

Works like a champ! Thanks!

That's awesome :)


[Change Request]: Research aria attributes on components

What type of enhancement/feature is this?
Behaviour Changes to existing UI Components, Documentation Improvements, Other
Description
Investigate using aria attributes on existing web components.
Currently the web components developed does not have full accessibility support. This is required for future proofing the components to cater for accessibility.
The following should be confirmed
How the current testing implementation of the components can be streamlined.
What accessibility attributes will be required for specific components (Labels vs Inputs).
Use Case/Reasoning for enhancement/feature.
Simplify the tests written for components.
Provide accessibility compliant components.
Understand which aria roles and aria attributes will be required for each component.
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I have searched for existing issues and to the best of my knowledge this is not a duplicate
[X] This is a single enhancement/feature

Note the following test case the aria-label attribute was added to the text field component a comparison using the browser extension lighthouse was used to compare the results of the same view on the Omni-components doc site.
Before aria-label added

After aria-label added

The following aria attributes should be considered (Note this comment will have edits as more attributes are confirmed)
aria-label
aria-disabled
aria-checked
aria-error


3.4.1 breaks existing tests/API

Describe the bug
When running our test suite with the latest version of enzyme we get:

When we run tests with enzyme 3.3.0 all the tests pass. When we run with 3.4.1 most of our tests break with the error message above.
To Reproduce
Steps to reproduce the behavior:

git clone https://github.com/DigitalRiver/react-atlas
npm install
npm install<EMAIL_ADDRESS>npm run test

Expected behavior
Tests pass and don't return error in screenshot above.
Desktop (please complete the following information):

OS: [Windows 7]
Browser [FF 61.0.1]

@ljharb If enzyme tests passed before with out strings it should continue to work without strings otherwise your breaking a public API. It doesn't matter if it was a "bug fix" or not updating minor versions shouldn't break existing tests. And if you are going to be making breaking changes to enzyme it needs to be in a new major version release, not a minor version release.

Tests pass with strings on 3.4.1 and they passed before on 3.3.0 with components so it seems like enzyme 3.3.0 allowed for both and 3.4.1 only allows for strings, which is a Public API breakage in a minor release.  Shouldn't there be a deprecation warning so users know about this?

Every bug fix is a "breaking change" if you're relying on the bug; we'd never be able to ship bugfixes as a patch version if the criteria was as simple as "every passing test must remain passing".
Again, that the tests passed for you doesn't indicate that they were actually testing what you think they were.
The exception you're receiving is the deprecation warning. I suppose I could convert the exception to a console.warn, but hopefully most enzyme users have console warnings/errors set to fail tests anyways, so that propType warnings fail tests.
If you convert them from elements to component constructors (not strings), do your tests pass in both v3.3 and v3.4?

I'm going to close this, on the assumption that the fact that these tests passed previously was a bug, which is now fixed.
Happy to reopen it if this is incorrect.


Develop to next post 2.1.0

Closes: #
What's Changed

TODO

[ ] Explanation of Change(s)
[ ] Added change log fragment(s) (for more information see the documentation)
[ ] Attached Screenshots, Payload Example
[ ] Unit, Integration Tests
[ ] Documentation Updates (when adding/changing features)
[ ] Example Plugin Updates (when adding/changing features)
[ ] Outline Remaining Work, Constraints from Design

@glennmatthews can you merge in the latest changes from develop to bring in the docker build fix?


05/28/2018 18:18 : Create Pull Request for branch master -> prTesting1_2018052818

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon1.md
:white_check_mark:Succeeded

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


Panic in the buddy_allocator

Hey, currently working on integrating of redb into our code, and got this error on CI.
The environment is cross with target aarch64-linux-android.
thread 'sync::tests::test_replica_sync_fs' panicked at
'assertion failed: !self.get_order_allocated(order).get(page_number)',
/root/.cargo/registry/src/index.crates.io-6f17d22bba15001f/redb-1.0.5/src/tree_store/page_store/buddy_allocator.rs:450:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

How can I reproduce the panic?

You can find the full CI run here https://github.com/n0-computer/iroh/actions/runs/5738430197/job/15552011144?pr=1315
The command being executed is
cross test --all --target aarch64-linux-android -- --test-threads=12

on this branch: https://github.com/n0-computer/iroh/tree/sync-db.
I am still working on reproducing it locally myself.

On that branch, it reproduces locally with the above command, minimizing the execution to the relevant test
> RUST_BACKTRACE=full cross test -p iroh-sync test_replica_sync_fs --target aarch64-linux-android
    Finished test [unoptimized + debuginfo] target(s) in 0.25s
     Running unittests src/lib.rs (/target/aarch64-linux-android/debug/deps/iroh_sync-f89f428cffa29e08)

running 1 test
test sync::tests::test_replica_sync_fs ... FAILED

failures:

---- sync::tests::test_replica_sync_fs stdout ----
thread 'sync::tests::test_replica_sync_fs' panicked at 'assertion failed: !self.get_order_allocated(order).get(page_number)', /cargo/registry/src/gith
ub.com-1ecc6299db9ec823/redb-1.0.5/src/tree_store/page_store/buddy_allocator.rs:450:13
stack backtrace:
   0:       0x55003edaf8 - <unknown>
   1:       0x550040b868 - <unknown>
   2:       0x55003ea4ac - <unknown>
   3:       0x55003ed8fc - <unknown>
   4:       0x55003ef420 - <unknown>
   5:       0x55003ef094 - <unknown>
   6:       0x55001809d8 - <unknown>
   7:       0x55003efafc - <unknown>
   8:       0x55003ef848 - <unknown>
   9:       0x55003ee00c - <unknown>
  10:       0x55003ef5d8 - <unknown>
  11:       0x5500037d2c - <unknown>
  ...

I got this, but unfortunately as you can see the backtrace is unusable

@dignifiedquire surprisingly it looks like this is a compiler bug, or maybe an issue with the cross tool you're using (I have no idea how it works). Try with this branch https://github.com/cberner/redb/tree/iroh and you'll see a different panic, which definitely should not be possible since it comes from this single line change which asserts that a zero initialized Vec is in fact all zeros: https://github.com/cberner/redb/commit/42444df61e0b2587f87c2cde25ade72cea0e9977

Looks like this does change the panic:
RUST_BACKTRACE=full cross test -p iroh-sync test_replica_sync_fs --target aarch64-linux-android
   Compiling redb v1.0.5 (https://github.com/cberner/redb?branch=iroh#42444df6)
   Compiling iroh-sync v0.1.0 (/project/iroh-sync)
    Finished test [unoptimized + debuginfo] target(s) in 5.87s
     Running unittests src/lib.rs (/target/aarch64-linux-android/debug/deps/iroh_sync-2600d34f0c7ac5a1)

running 1 test
test sync::tests::test_replica_sync_fs ... FAILED

failures:

---- sync::tests::test_replica_sync_fs stdout ----
thread 'sync::tests::test_replica_sync_fs' panicked at 'assertion failed: data.iter().all(|x| *x == 0)', /cargo/git/checkouts/redb-24e44532b0b35edd/42444df/src/tree_store/page_store/bitmap.rs:297:9
stack backtrace:
   0:       0x55003eca50 - <unknown>
   1:       0x550040a7c0 - <unknown>
   2:       0x55003e9404 - <unknown>
   3:       0x55003ec854 - <unknown>
   4:       0x55003ee378 - <unknown>
   5:       0x55003edfec - <unknown>
   6:       0x55001808c8 - <unknown>
   7:       0x55003eea54 - <unknown>
   8:       0x55003ee7a0 - <unknown>
   9:       0x55003ecf64 - <unknown>
  10:       0x55003ee530 - <unknown>
  11:       0x5500037c1c - <unknown>
  12:       0x5500037cb0 - <unknown>
  13:       0x55002a0740 - <unknown>
  14:       0x550027ab34 - <unknown>
  15:       0x550026e2c8 - <unknown>
  16:       0x550025f924 - <unknown>
  17:       0x550026024c - <unknown>
  18:       0x55002a20c4 - <unknown>
  19:       0x55002797d4 - <unknown>
  20:       0x550003948c - <unknown>
  21:       0x5500039584 - <unknown>
  22:       0x5500087a84 - <unknown>
  23:       0x55000df81c - <unknown>
  24:       0x55000f82d8 - <unknown>
  25:       0x550012e290 - <unknown>
  26:       0x5500184568 - <unknown>
  27:       0x550018367c - <unknown>
  28:       0x550015a448 - <unknown>
  29:       0x550015f638 - <unknown>
  30:       0x55003f22a0 - <unknown>
  31:       0x550208193c - <unknown>
  32:       0x550202347c - <unknown>
  33:                0x0 - <unknown>

failures:
    sync::tests::test_replica_sync_fs

Some additional debugging with printing out the size of the created vec and its content when it fails. It looks like it works most of the time, until it doesn't..
new_empty: 2
new_empty: 1                                                                                new_empty: 1                                                                                new_empty: 1                                                                                new_empty: 1                                                                                new_empty: 1                                                                                new_empty: 1
new_empty: 1
new_empty: 16384
new_empty: 8192
new_empty: 4096
new_empty: 2048
new_empty: 1024
new_empty: 512
new_empty: 256
new_empty: 128
new_empty: 64
new_empty: 32
new_empty: 16
new_empty: 8
new_empty: 4
new_empty: 2
new_empty: 1
new_empty: 1
new_empty: 1
new_empty: 1
new_empty: 1
new_empty: 1
new_empty: 1
new_empty: 16384
new_empty: 8192
new_empty: 4096
new_empty: 2048
new_empty: 1024
new_empty: 512
new_empty: 256
new_empty: 128
new_empty: 64
new_empty: 32
new_empty: 16
new_empty: 8
new_empty: 4
new_empty: 2
new_empty: 1
new_empty: 1
new_empty: 1
new_empty: 1
new_empty: 1
new_empty: 1
new_empty: 1
new_empty: 16384
thread 'sync::tests::test_replica_sync_fs' panicked at '[18446744073709551615,<PHONE_NUMBER>3709551615,<PHONE_NUMBER>3709551615,<PHONE_NUMBER>3709551615,<PHONE_NUMBER>3709551615,<PHONE_NUMBER>37095
51615,<PHONE_NUMBER>3709551615,<PHONE_NUMBER>3709551615,<PHONE_NUMBER>3709551615,
<all the same value>

It seems more related to cross and rustc than to the architecture, got the same error now on armv7 as well: https://github.com/n0-computer/iroh/actions/runs/5738430197/job/15591730496

Reproduced the issue on redbs tests:
> cross test --target aarch64-linux-android
failures:

---- db::test::small_pages stdout ----
thread 'db::test::small_pages' panicked at '16384: [0]', src/tree_store/page_store/bitmap.rs:298:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

---- db::test::small_pages2 stdout ----
thread 'db::test::small_pages2' panicked at '16384: [0]', src/tree_store/page_store/bitmap.rs:298:9

---- db::test::small_pages3 stdout ----
thread 'db::test::small_pages3' panicked at '16384: [18446744073709551615]', src/tree_store/page_store/bitmap.rs:298:9

---- db::test::small_pages4 stdout ----
thread 'db::test::small_pages4' panicked at '16384: [27584547717644288]', src/tree_store/page_store/bitmap.rs:298:9

---- transactions::test::transaction_id_persistence stdout ----
thread 'transactions::test::transaction_id_persistence' panicked at '16384: [18446744073709551615]', src/tree_store/page_store/bitmap.rs:298:9

---- tree_store::page_store::buddy_allocator::test::serialized_size stdout ----
thread 'tree_store::page_store::buddy_allocator::test::serialized_size' panicked at '16384: [19855633815385]', src/tree_store/page_store/bitmap.rs:298:9

---- tree_store::page_store::header::test::repair_allocator_checksums stdout ----
thread 'tree_store::page_store::header::test::repair_allocator_checksums' panicked at '16384: [29836347531329536]', src/tree_store/page_store/bitmap.rs:298:9

---- tree_store::page_store::header::test::repair_empty stdout ----
thread 'tree_store::page_store::header::test::repair_empty' panicked at '16384: [18446744069414592765]', src/tree_store/page_store/bitmap.rs:298:9

---- tree_store::page_store::header::test::repair_insert_reserve_regression stdout ----
thread 'tree_store::page_store::header::test::repair_insert_reserve_regression' panicked at '16384: [18446744073709551615]', src/tree_store/page_store/bitmap.rs:298:9

failures:
    db::test::small_pages
    db::test::small_pages2
    db::test::small_pages3
    db::test::small_pages4
    transactions::test::transaction_id_persistence
    tree_store::page_store::buddy_allocator::test::serialized_size
    tree_store::page_store::header::test::repair_allocator_checksums
    tree_store::page_store::header::test::repair_empty
    tree_store::page_store::header::test::repair_insert_reserve_regression

But it gets more interesting, if I replace the init code in bitmap.rs with this
pub fn new_empty(len: u32, capacity: u32) -> Self {
        let cap = Self::required_words(capacity);
        // let data = vec![0; cap];
        let data: Vec<u64> = (0..cap).map(|_| 0).collect();
        assert!(data.iter().all(|x| *x == 0), "{}: {:?}", cap, &data[..1]);
        Self { len, data }
}

I get the following failures only
> cross test --target aarch64-linux-android

running 30 tests
test db::test::crash_regression3 ... ok
test db::test::crash_regression4 ... ok
test db::test::dynamic_shrink ... ok
test db::test::small_pages ... ok
test db::test::small_pages2 ... ok
test db::test::small_pages3 ... ok
test db::test::small_pages4 ... ok
test transactions::test::transaction_id_persistence ... ok
test tree_store::page_store::base::test::last_page ... ok
test tree_store::page_store::bitmap::test::all_space_used ... ok
test tree_store::page_store::bitmap::test::alloc ... ok
test tree_store::page_store::bitmap::test::find_free ... ok
test tree_store::page_store::bitmap::test::free ... ok
test tree_store::page_store::bitmap::test::iter ... ok
test tree_store::page_store::bitmap::test::random_pattern ... ok
test tree_store::page_store::bitmap::test::record_alloc ... ok
test tree_store::page_store::bitmap::test::reuse_lowest ... ok
test tree_store::page_store::buddy_allocator::test::alloc_large ... ok
test tree_store::page_store::buddy_allocator::test::buddy_merge ... ok
test tree_store::page_store::buddy_allocator::test::record_alloc_buddy ... ok
test tree_store::page_store::buddy_allocator::test::serialized_size ... ok
test tree_store::page_store::header::test::magic_number ... ok
test tree_store::page_store::header::test::repair_allocator_checksums ... ok
test tree_store::page_store::header::test::repair_empty ... ok
test tree_store::page_store::header::test::repair_insert_reserve_regression ... ok
test tree_store::page_store::layout::test::full_layout ... ok
test tree_store::page_store::page_manager::test::out_of_regions ... ok
test tree_store::page_store::xxh3::test::test_empty ... ok
test tree_store::table_tree::test::round_trip ... ok
test tuple_types::test::width ... ok

test result: ok. 30 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 18.37s

     Running tests/backward_compatibility.rs (/target/aarch64-linux-android/debug/deps/backward_compatibility-ec8da7951aa53207)

running 3 tests
test container_types ... ok
test mixed_width ... FAILED
test primitive_types ... FAILED

failures:

---- mixed_width stdout ----
thread 'mixed_width' panicked at 'assertion failed: !self.get_order_allocated(order).get(page_number)', /cargo/registry/src/github.com-1ecc6299db9ec823/redb-1.0.5/src/tree_store/page_store/buddy_allocator.rs:450:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

---- primitive_types stdout ----
thread 'primitive_types' panicked at 'assertion failed: !self.get_order_allocated(order).get(page_number)', /cargo/registry/src/github.com-1ecc6299db9ec823/redb-1.0.5/src/tree_store/page_store/buddy_allocator.rs:450:13

failures:
    mixed_width
    primitive_types

Going to close this since it doesn't seem to be a redb bug. I'd report it to either rustc or cross, depending on which you think is broken