Accept buttons not working on first click

I have noticed scenarious where clicking on "Accept all" or "Accept essentials" will not work (popup box remains open). In such cases, users have to click a second time for the popup box to close.
This issue was the placement of @cookieconsentscripts - if there are other script tags of a larger size above it, the cookie box will open without the actual laravel-cookie-consent javascript being loaded, so clicking on it will throw an undefined error:

The solution is to move @cookieconsentscripts above all other script tags to ensure it loads first.
However, I would suggest displaying the popup box after the event DOMContentLoaded is fired

@nlukic97 I tried your suggestion (script at the very top), but still no luck for me.
@toonvandenbos Any input on the proposition for "after the event DOMContentLoaded is fired"

I'm not against the idea of masking the DOM element until the document is fully loaded when JavaScript is enabled since the whole point of this package is to work when JS is not enabled too. If somebody has some time to work on a PR having these constraints in mind, that would be helpful! Thanks :)

Sure, i’ll start working on it
On Thu, 8 Aug 2024 at 19:08, Toon Van den Bos @.***>
wrote:

I'm not against the idea of masking the DOM element until the document is
fully loaded when JavaScript is enabled since the whole point of this
package is to work when JS is not enabled too. If somebody has some time to
work on a PR having these constraints in mind, that would be helpful!
Thanks :)
—
Reply to this email directly, view it on GitHub
https://github.com/whitecube/laravel-cookie-consent/issues/47#issuecomment-2276286864,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANSJ22J7L3UK7YRLGNME3HLZQOQXNAVCNFSM6AAAAABHJSDNKSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZWGI4DMOBWGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Same issue with 1.2.3 and 1.2.2
@cookieconsentscripts at the top of the head
@cookieconsentview bottom of the body
It is used in more projects they're working.
window.LaravelCookieConsent is undefined at the console.
In local the head try to loads like this:
<script src="http://localhost:8000:8000/cookie-consent/script?id=051b492e30fe8edd2aa2c23b8f8bb8ae" defer=""></script>
In .env the APP_URL is http://localhost:8000. In live with correct domain also not working.

@szotyi41 Is there a version without this issue?

In our older site it is working in the same way: https://mvs-systems.com/ (1.2.2)
In our new site it is not working (tried with 1.2.2 and 1.2.3)
Update:
In live mode the script is loaded and the server respond with 200 {"status": "ok"}
After I go to another page in the same site, the cookie bar appears again and again.


Adding error announcements to user form

This fixes https://github.com/CovidShield/portal/issues/7.
It uses the suggestion in that issue to connect the error messages to the inputs using aria

I forgot to mention, I'd suggest changing each .text-field__error-message containers from div to the more semantically correct small element.

Good catch. Updated with that. 👍

@dfmcphee Would it be possible to send focus to the first input in an error state? This would be helpful in conveying the current state.
And now that I think about it, there's no explicit "success" message other than loading a new page. I guess there was nothing in the design for a success state visual?

@dfmcphee Would it be possible to send focus to the first input in an error state? This would be helpful in conveying the current state.

For sure. Added that now. Will follow up about a success message in another PR.


Update index-6ljbsvac.js

merge

merge

don't worry. I will update the preview in couple of mins 😃

Updated demo 👍


Addition of inner type which doesn't pass through shutdown() calls

As per: https://users.rust-lang.org/t/stream-data-in-compress-and-stream-out/72521/6
At current, the stream writer implementation calls shutdown() on the compression writer when attempting to close the entry.
This is done as the upstream compression crate does not implement any way to finish the encoding without calling shutdown().
However, their implementation of the shutdown also polls the inner writer's shutdown, thus chaining the shutdown call up to the generic writer.
I don't see any way to avoid this shutdown call, but feel free to comment below if you do.
So for the time being, an inner type wrapping the generic writer should be implemented which ignores calls to poll_shutdown().
Likely want to be implemented within: https://github.com/Majored/rs-async-zip/blob/main/src/write/compressed_writer.rs

Hi @Majored, that was me! :)
Only reason why I didn't file this issue myself, is because one of the people who helped me identify this bug told me he was already working on a PR.
@Nemo157 author of async_compression, mentioned he's happy to provide the relevant API to solve this properly. Pinging him, in case.

I've gone ahead and added this inner type and your original code (without the customised DuplexStream) is now working correctly on my end.
I'll eventually get around to making a PR for async_compression with the API change needed for this (if no one else has), but I want to get familiar with the codebase before doing that given its heavy use of macros for code generation/abstraction.
Let me know if you still run into any issues with this.


chore: bdd-sbhg-1585753604 to 0.0.1

chore: Promote bdd-sbhg-1585753604 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


error[E0658]: the unsafe_op_in_unsafe_fn lint is unstable

kreyren@leonid ~/Repositories/guix [env]$ ./pre-inst-env guix build rust-cargo-make
...
   Compiling ansi_term v0.11.0
   Compiling git_info v0.1.2
   Compiling rust_info v0.3.2
error[E0658]: the `unsafe_op_in_unsafe_fn` lint is unstable
   --> /tmp/guix-build-rust-cargo-make-0.35.6.drv-0/cargo-make-0.35.6/guix-vendor/rust-rust-info-0.3.2.tar.gz/src/lib.rs:1:1
    |
1   | / #![deny(
2   | |     absolute_paths_not_starting_with_crate,
3   | |     ambiguous_associated_items,
4   | |     anonymous_parameters,
...   |
125 | |     useless_deprecated
126 | | )]
    | |__^
    |
    = note: see issue #71668 <https://github.com/rust-lang/rust/issues/71668> for more information

error: aborting due to previous error

For more information about this error, try `rustc --explain E0658`.
error: could not compile `rust_info`.

To learn more, run the command again with --verbose.
warning: build failed, waiting for other jobs to finish...
error: build failed
command "cargo" "build" "--release" failed with status 101
builder for `/gnu/store/9gi25bn6f4zn6c9qrz6mbhgw3k82yxcj-rust-cargo-make-0.35.6.drv' failed with exit code 1
build of /gnu/store/9gi25bn6f4zn6c9qrz6mbhgw3k82yxcj-rust-cargo-make-0.35.6.drv failed
View build log at '/var/log/guix/drvs/9g/i25bn6f4zn6c9qrz6mbhgw3k82yxcj-rust-cargo-make-0.35.6.drv.bz2'.
guix build: error: build of `/gnu/store/9gi25bn6f4zn6c9qrz6mbhgw3k82yxcj-rust-cargo-make-0.35.6.drv' failed

Describe The Bug

Fails to build on while defining cargo-make as a package for GNU Giox
To Reproduce
On GNU Guix:

Invoke guix import crate cargo-make -r >> path/to/guix/repo/gnu/packages/crates-io.scm
Use fix described in https://github.com/briansmith/webpki/issues/246
Experience the build failure

Theoretical

Build it on stable rustlang toolchain?

Relevants
R.1. https://github.com/rust-lang/rust/issues/71668

i think you need to update your rust version

yep, sorry didn't realize that guix is using 1.48 x.x


Email Scheduler

Description
Added a file that schedules email and send them via SMTP
Fixes #2196

Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] My documentation follows Template for README.md
[x] I have added the project metadata in the PR template.
[ ] I have created the requirements.txt file if needed.

Project Metadata
If there is no-file/nothing to fill the below fields with, then type: none
Example: If no requirements.txt is needed/present, then type none in Requirments.
Category:

[ ] Calculators
[ ] AI/ML
[ ] Scrappers
[ ] Social_Media
[ ] PDF
[ ] Image_Processing
[ ] Video_Processing
[ ] Games
[ ] Networking
[ ] OS_Utilities
[x] Automation
[ ] Cryptography
[ ] Computer_Vision
[ ] Fun
[ ] Others

Title: Email_scheduler.py
Folder: Email-Sender
Requirements: None
Script: Email_scheduler.py
Contributor: Juhibhojani
Description: Scheduling Email

I have made the necessary changes, please review it now. @avinashkranjan

Conflicts are still there.

I can't resolve these conflicts

Okay


BitmapDataChannel compiler errors on Flash when doc-gen is defined

<haxedef name="doc-gen" />

package;

import openfl.display.BitmapData;
import openfl.display.BitmapDataChannel;
import openfl.display.Sprite;

class Main extends Sprite
{
	public function new()
	{
		super();
		new BitmapData(1, 1).copyChannel(
			null, null, null, BitmapDataChannel.ALPHA, 0);
	}
}

src/Main.hx:13: characters 21-44 : openfl.display.BitmapDataChannel should be UInt
src/Main.hx:13: characters 21-44 : For function argument 'sourceChannel'

It compiles when to UInt is added to the extern definition of BitmapDataChannel, but I'm not sure if that's a proper fix.
This causes issues when generating API docs, as doc-gen should be defined for that.
Haxe 3.2.0, OpenFL 3.6.0, Lime 2.9.0.

https://github.com/openfl/openfl/blob/develop/extern/openfl/display/BitmapDataChannel.hx
Yeah, I think that is reasonable, or perhaps it could be an abstract over UInt to begin with, if the type in Flash Player is UInt anyway.
Feel free to commit this fix yourself if it resolves all the issues for you. Thank you! :wink:


Docz doesn't work with newly cloned project

After cloning react-rinse for a new project, docz doesn't seem to work.
System Specs:
$ node -v
10.16.3
$ yarn -v
1.19.0

Resolution: Tried upgrading webpack and everything seems to be working.
I can open up a PR for the same.

Fixed in #35


Deprecated Gradle features were used in this build, making it incompatible with Gradle 7.0.

Expected Behavior

./gradlew assemble

Current Behavior

Context

I want to compile gradle source. Compilation is complete but no binaries are generated. Specific information is as follows；
I want to know why
Steps to Reproduce

[11:28:34@root gradle_01]#./gradlew assemble
Partial virtual file system invalidation is an incubating feature.

Deprecated Gradle features were used in this build, making it incompatible with Gradle 7.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/6.1/userguide/command_line_interface.html#sec:command_line_warnings

BUILD SUCCESSFUL in 14s
697 actionable tasks: 91 executed, 606 up-to-date
[11:28:53@root gradle_01]#
[11:29:10@root gradle_01]#
[11:29:10@root gradle_01]#
[11:29:10@root gradle_01]#
[11:29:10@root gradle_01]#
[11:29:11@root gradle_01]#
[11:37:26@root gradle_01]#
[11:37:26@root gradle_01]#
[11:37:26@root gradle_01]#tree build
build
├── build-receipt.properties
├── determined-commit-id.txt
└── kotlin
    └── sessions

2 directories, 2 files
[11:37:35@root gradle_01]#
[11:37:43@root gradle_01]#./gradlew -v

------------------------------------------------------------
Gradle 6.1
------------------------------------------------------------

Build time:   2020-01-15 23:56:46 UTC
Revision:     539d277fdba571ebcc9617a34329c83d7d2b259e

Kotlin:       1.3.61
Groovy:       2.5.8
Ant:          Apache Ant(TM) version 1.10.7 compiled on September 1 2019
JVM:          11.0.2 (Oracle Corporation 11.0.2+9)
OS:           Linux 4.15.0-76-generic amd64

[11:37:51@root gradle_01]#

Your Environment

Build scan URL:
[11:38:17@root gradle_01]#java --version
openjdk 11.0.2 2019-01-15
OpenJDK Runtime Environment 18.9 (build 11.0.2+9)
OpenJDK 64-Bit Server VM 18.9 (build 11.0.2+9, mixed mode)
[11:38:27@root gradle_01]#

Can you please take a look at the documentation before building from source?
https://github.com/gradle/gradle/blob/master/CONTRIBUTING.md

Can you please take a look at the documentation before building from source?
https://github.com/gradle/gradle/blob/master/CONTRIBUTING.md

Thank you for your relpy！
From the documentation, your gradle source does not support command line compilation. Is that right?
I am going to compile with eclipse 2018. Is it OK?

Can you please take a look at the documentation before building from source?
https://github.com/gradle/gradle/blob/master/CONTRIBUTING.md

I compiled using intelliJ. The compilation results are the same.
Compilation is complete but no files are generated.
> Task :signing:compileJava

> Task :signing:compileGroovy NO-SOURCE
注: /home/loongson/gradle/gradle_src/gradle_01/subprojects/signing/src/main/java/org/gradle/plugins/signing/SigningExtension.java使用了未经检查或不安全的操作。
注: 有关详细信息, 请使用 -Xlint:unchecked 重新编译。
> Task :signing:parameterNamesIndex UP-TO-DATE
> Task :signing:classes
> Task :signing:jar UP-TO-DATE

> Task :ideNative:compileJava
注: 某些输入文件使用了未经检查或不安全的操作。
注: 有关详细信息, 请使用 -Xlint:unchecked 重新编译。

> Task :toolingNative:compileJava
> Task :toolingNative:compileGroovy NO-SOURCE
> Task :toolingNative:parameterNamesIndex UP-TO-DATE
> Task :toolingNative:classes
> Task :toolingNative:jar UP-TO-DATE
> Task :scala:assemble
> Task :signing:assemble
> Task :testingNative:assemble
> Task :toolingNative:assemble

> Task :jacoco:compileJava
注: 某些输入文件使用了未经检查或不安全的操作。
注: 有关详细信息, 请使用 -Xlint:unchecked 重新编译。

> Task :jacoco:compileGroovy NO-SOURCE
> Task :jacoco:parameterNamesIndex UP-TO-DATE
> Task :jacoco:classes
> Task :jacoco:jar UP-TO-DATE
> Task :jacoco:assemble

> Task :ideNative:compileGroovy
注: /home/loongson/gradle/gradle_src/gradle_01/subprojects/ide-native/src/main/groovy/org/gradle/ide/visualstudio/plugins/VisualStudioPlugin.java使用了未经检查或不安全的操作。
注: 有关详细信息, 请使用 -Xlint:unchecked 重新编译。

> Task :ideNative:parameterNamesIndex UP-TO-DATE
> Task :ideNative:classes
> Task :ideNative:jar UP-TO-DATE
> Task :ideNative:assemble
> Task :runtimeApiInfo:generateGradleApiPackageList UP-TO-DATE
> Task :runtimeApiInfo:jar UP-TO-DATE
> Task :runtimeApiInfo:assemble

> Task :internalPerformanceTesting:compileGroovy
注: 某些输入文件使用或覆盖了已过时的 API。
注: 有关详细信息, 请使用 -Xlint:deprecation 重新编译。
注: 某些输入文件使用了未经检查或不安全的操作。
注: 有关详细信息, 请使用 -Xlint:unchecked 重新编译。

> Task :internalPerformanceTesting:classes
> Task :internalPerformanceTesting:jar UP-TO-DATE
> Task :internalPerformanceTesting:assemble UP-TO-DATE

Deprecated Gradle features were used in this build, making it incompatible with Gradle 7.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/6.1/userguide/command_line_interface.html#sec:command_line_warnings

BUILD SUCCESSFUL in 4m 55s
697 actionable tasks: 284 executed, 413 up-to-date
下午4:25:27: Task execution finished.

It's documented in https://github.com/gradle/gradle/blob/master/CONTRIBUTING.md#development-workflow

To try out a change in behavior manually, install Gradle locally and use it. Install: ./gradlew install -Pgradle_installPath=/any/path. Use: /any/path/bin/gradle taskName

Please make sure you read the doc before doing anything.

./gradlew install -Pgradle_installPath=/any/path

What I want to know is that this installed gradle is the gradle I compiled? How to prove? What I need now is to modify the gradle source code to make it take effect.


Stopping ffmpeg / wrapper remotely

Hi,
I use this library as an RTSP client to record IP camera streams. I can start the streams but I could not figure out how to stop recoding remotely, since ffmpeg waits for q button to stop and terminate.
What can be the proper way of stopping ffmpeg / wrapper, without killing the sub-process ?

That is not a use case I designed for. Other that killing the sub process I
don't know of a clean way. I'll leave this bug open as a reminder to add
better support for sending input to ffmpeg while it is running.

I may propose a solution until you find a better one. What I've done is :

get StdIn of the process as you get StdOut
make FFmpeg wrapper instance as global variable in my code
Generate two threads, one thread starts FFmpeg wrapper instance, other thread gets StdIn using getProcessIn() and stops it, both methods using same global wrapper instance

we should add following code fragment to FFmpeg.java
to imports:
...
import java.io.BufferedWriter;
import java.io.OutputStreamWriter;
....
to members:
....
private BufferedWriter processIn;
....
to methods:
....
private BufferedWriter wrapOutWriter(Process p) {
return new BufferedWriter (new OutputStreamWriter(p.getOutputStream()));
}
public BufferedWriter getProcessIn() {
return processIn;
}
....
and finally altered run method:
....
public void run(List args) throws IOException {
List newArgs = ImmutableList.builder().add(path).addAll(args).build();
Process p = runFunc.run(newArgs);
try {

  processIn = wrapOutWriter(p);

  // Now block reading ffmpeg's stdout. We are effectively throwing away the output.
  IOUtils.copy(wrapInReader(p), System.out); // TODO Should I be outputting to stdout?

  FFmpegUtils.throwOnError("ffmpeg", p);

} finally {
  p.destroy();
}

}
....

I will have the same need for my application Podcast-Server, which is a downloader with GUI for different source. In the GUI, I should be able to start / stop / pause a download.
Right now, the download is made in plain Java with InputStream, but this lead to corrupted files... so I would like to switch to ffmpeg which is simpler, has a good wrapper and is simpler to use. But, for some case (like rtmp), I had to play with rtmpdumpcommand and process...
It will be the same here for ffmpeg, so, in order to solve the of @turmalin and let user play with the process, we should be able to access the process.
I'll try to do some proposition about this 😄
Thanks

Hello,
any news on this issue? I am also running into the problem that I have no means to stop a recording job that is running. Is there no way to stop recording an infinite stream to file?

I've handle it in my application with a ProcessListener. You can see the code here 👍
https://github.com/davinkevin/Podcast-Server/tree/master/Backend/src/main/java/lan/dk/podcastserver/utils/custom/ffmpeg
https://github.com/davinkevin/Podcast-Server/blob/master/Backend/src/main/java/lan/dk/podcastserver/service/FfmpegService.java

I've handle it in my application with a ProcessListener. You can see the code here 👍
https://github.com/davinkevin/Podcast-Server/tree/master/Backend/src/main/java/lan/dk/podcastserver/utils/custom/ffmpeg
https://github.com/davinkevin/Podcast-Server/blob/master/Backend/src/main/java/lan/dk/podcastserver/service/FfmpegService.java

Hi @davinkevin  The above link does not exist anymore. Do you mind giving the new link?

https://gitlab.com/davinkevin/Podcast-Server/tree/aeb34992ebf0262eb063589b67597811e0b09a30/backend/src/main/kotlin/com/github/davinkevin/podcastserver/utils/custom/ffmpeg
https://gitlab.com/davinkevin/Podcast-Server/blob/aeb34992ebf0262eb063589b67597811e0b09a30/backend/src/main/kotlin/com/github/davinkevin/podcastserver/service/FfmpegService.kt
Sorry for that 😅.
I move my project to youtube-dl instead of doing the download by myself then encoding it with ffmpeg, so I will remove those files in few iterations I think.


[tensorflow] Remove tensorflow packet on Windows

Tensorflow on Windows is unusable, without users manually hacking the produced libraries.
https://github.com/tensorflow/tensorflow/issues/23542
https://stackoverflow.com/questions/62954634/tensorflow-2-3-unresolved-external-symbols-in-machine-generated-files-when-build
This is confusing since "ok" installed packets fail during link phase because missing exported symbols.
Regardless of "who should fix it/whose fault it is" fact is that having a broken package in vcpkg is bad for user perspective of vcpkg quality.
As I presume that it is unlikely somebody will write a magic fix for this since it does not seem trivial so I think proper thing is to remove the tensorflow packet for Windows.
If it matters broken version I tried is 2.4#2, but I presume all of them are broken.
note: build(installing tensorflow_cc with vcpkg) works fine, I get .lib and .dll (in both Release and Debug variants), but if you actually try to actually write some TF code that uses those libs then you get LNK2019.
as for repro:
install tensorflow_cc and try to actually use to build some toy TF project. Some of missing symbols are:
LNK2019: unresolved external symbol "public: __cdecl tensorflow::SessionOptions::SessionOptions(void)”
LNK2019: unresolved external symbol "class tensorflow::Status __cdecl tensorflow::NewSession(struct tensorflow::SessionOptions const &,class tensorflow::Session * *)"

Sorry for late, can you please provide a sample project to reproduce this bug?

@JackBoosY
5 days including weekend is what I consider fast.
So now back to how to reproduce:
As I wrote: install the TF packet, integrate with VS(to make it easy to use it), write some toy program that uses TF and it will fail with linker errors.
In particular try to use
SessionOptions::SessionOptions or any other items patched in medium link bellow.
You will get linker errors.
This is expected and it matches this blog post:
https://medium.com/vitrox-publication/deep-learning-frameworks-tensorflow-build-from-source-on-windows-python-c-cpu-gpu-d3aa4d0772d8
(TF does NOT export symbols so that medium posts suggest manually editing TF source to add TF_EXPORT in front of important stuff).
note: I am 99+% certain in what I am saying because I have TF working (built from source using above method), but I wanted to switch to using package manager since I do not like to maintain such a huge dependency.
Additionally I have done DUMPBIN / EXPORTS on "my" dll(it contains the symbols missing), also it works fine when i use it in my program. DUMPBIN /EXPORTS on vcpkg dll does not contain necessary symbols.
here is the search results in Notepad++ for .dll built using above medium post:

here is the search results in Notepad++ for .dll installed using vcpkg

Sorry for a long reply, so recap:

I have TF working(built manually using medium post that shows where to stick TF_EXPORT), my application that uses TF links fine.
I wanted to move to vcpkg package TF
That failed since TF in vcpkg does not export necessary symbols, so my application build fails at link stage.
This is hard to fix since you would need to modify TF source to get it to work, not sure if vcpkg team wants to commit to that kind of maintenance burden.
Since I think it is hard to fix this I suggest you remove the package from vcpkg, but if you are willing to make the patches to TF sources to get it working that would be great.

P.S. If you wonder why TF does not export those symbols by default:  I have no idea... you can check the 3 y old issue I linked in my first comment.

Just double checked, unfortunately I made a mistake, in my original post i said I am using 2.4 package, I was using 2.3 package.
2.3 package is indeed missing exported symbols, TF2.4 seems to be fine.
Apologies for false alarm.

Ah I was right originally, vcpkg tensorflow is still broken.
So reopening.
For context(not that it matters I feel) I am using vcpkg without integration, just having this in my FindTensorflow file:
include(D:/vcpkg/installed/x64-windows/share/tensorflow-cc/tensorflow-cc-config.cmake)
As for version here it is:
./vcpkg.exe list
tensorflow-cc:x64-windows                          2.4.1            Library for computation using data flow graphs f...
tensorflow-common:x64-windows                      2.4.1#2          This meta package holds common files for the C [...

@libboze  Can you please provide a sample project (contains some code) to reproduce this issue?

cc @jgehw do you know why?

@libboze Can you please provide a sample project (contains some code) to reproduce this issue?

My failures were in "large real" project so I can not just easily reduce it to the example project. I believe @jgehw will agree with me that problem is in missing exported symbols.
As mentioned before my problems exactly match the problems mentioned in medium link ( Step 11: Build and Test Shared DLL Library for C++ API (CPU & GPU) chapter) so I am 99+% certain that is the issue.
If @jgehw does not see this or he is too busy to reply I will try to find some time in 4-5 days(weekend) to make a simple C++ program that fails.

problem is in missing exported symbols.

I checked the source code, the upstream doesn't export them either, so is it by design?

problem is in missing exported symbols.

I checked the source code, the upstream doesn't export them either, so is it by design?

It is probably by "bug", not by design, from what I saw TF does not have any CI for Debug Windows build.
As mentioned in first comment I am not the first to notice this. https://github.com/tensorflow/tensorflow/issues/23542

I wasn't aware of these C++ linking issues because I'm only using the C API on Windows, which is working fine, and because another vcpkg user, I was in contact with, apparently used the C++ API via static linking.
According to the upstream thread @libbooze mentioned, there doesn't seem to be an easy fix.
I strongly disagree with removal of the vcpkg tensorflow Windows ports because the C version [tensorflow] is working fine both as DLL and with static linkage, and because the C++ version [tensorflow-cc] is reported to work with static linkage (x64-windows-static triplet).
@JackBoosY Maybe it makes sense to issue message(FATAL_ERROR "TensorFlow doesn't support the C++ API via dynamic linkage on Windows. Please consider using static linkage (x64-windows-static triplet).") for the [tensorflow-cc] port and dynamic linkage on Windows?

I wasn't aware of these C++ linking issues because I'm only using the C API on Windows, which is working fine, and because another vcpkg user, I was in contact with, apparently used the C++ API via static linking.
According to the upstream thread @libbooze mentioned, there doesn't seem to be an easy fix.
I strongly disagree with removal of the vcpkg tensorflow Windows ports because the C version [tensorflow] is working fine both as DLL and with static linkage, and because the C++ version [tensorflow-cc] is reported to work with static linkage (x64-windows-static triplet).

My apologies, I had no idea that vcpkg supports static linking... or C libraries.
As for static linking: don't you then again run into the problem for Debug builds since MSVC can not handle pdbs over around 6GB? Or the mentioned vcpkg user only uses TF in Release builds?
In any case I agree the issue exists for current dll ports and something needs to be done to make cause of this clear.

As for static linking: don't you then again run into the problem for Debug builds since MSVC can not handle pdbs over around 6GB? Or the mentioned vcpkg user only uses TF in Release builds?

Microsoft sadly indeed didn't remove the 4GB limit on x64 Windows from the internals of .pdb files, and even more annoying, from .lib files. While there was an easy work-around for the .pdb files (I reduced the size of the .pdb file below 4GB by limiting some debug features using /DEBUG:FASTLINK instead of /DEBUG:FULL), it was more painful to work around the 4GB limit on .lib files: I needed to split TensorFlow into several .lib files (cf. convert_lib_params_windows.py and generate_static_link_cmd_windows.py).

@jgehw  thank you for making the pull request.
I wish TF people fixed this, but will probably not happen.
Also not sure if you are right person to report this to, but I think 2.5 is incompatible with Eigen 3.3.9 (at least IIRC on my machine it was broken since both Eigen and TF specialized std::hash for same type), so if there are any plans to update TF version in vcpkg this could be another issue that could hit users. TF 2.5 works with custom git commit that they use internally, but it is not really fun experience for users since they just picked random commit from Eigen, not a tagged release.
Again this is for TF 2.5, current version in vcpkg is 2.4, so this only applies for future.

@libbooze thanks for reporting the TF 2.5 / Eigen 3.3.9 incompatibility. Do you know if this is fixed in TF 2.6(-rc*)? (My plan was to skip 2.5 and update to 2.6 soon.)

@libbooze thanks for reporting the TF 2.5 / Eigen 3.3.9 incompatibility. Do you know if this is fixed in TF 2.6(-rc*)? (My plan was to skip 2.5 and update to 2.6 soon.)

Sorry, did not try.
Here is where I think they specify eigen versions:
https://github.com/tensorflow/tensorflow/commits/master/third_party/eigen3/workspace.bzl
I am not an really following Eigen macros policy, but is seems TF went from unreleased 3.4 to unreleased 3.3 (https://github.com/tensorflow/tensorflow/commits/master/third_party/eigen3/workspace.bzl)
this is what they used in March:
https://gitlab.com/libeigen/eigen/-/blob/f612df273689a19d25b45ca4f8269463207c4fee/Eigen/src/Core/util/Macros.h
this is what they use now:
https://gitlab.com/libeigen/eigen/-/blob/12e8d57108c50d8a63605c6eb0144c838c128337/Eigen/src/Core/util/Macros.h
If 2.6 works with Eigen version in vcpkg: I have no idea.... did not try it.
Also it seems they used to patch Eigen, but thankfully I can not find the patch file in 2.6(I hope it was not just moved).
https://github.com/tensorflow/tensorflow/blob/r2.4/third_party/eigen3/gpu_packet_math.patch


feat: add step that automatically lint & fix all files to avoid problems from raw template strings

This PR:

adds automatic eslint formatting & fixing of the generated project, to avoid problems from raw templates
adds CI steps to:

lint the --ci output test project
typecheck the --ci output test project
run all of the --ci checks for the package managers: pnpm, npm

-> fails if npm install is not run

-> fails if npm install is not run

resolved by not adding automatic linting and instead fixing the source of the issue: bad templates. The CI pipeline will catch these in the future now


n_positions vs n_ctx inconsistency

Thanks for providing the code. It made replication very smooth.
I find the use of n_positions vs n_ctx inconsistent and I am wondering whether this is a bug. What was the motivation to have these two variables? Can we drop one of these variables and just use either one?
Having one variable would make it consistent with the GPT-2 implementation.

The two separate variables were an artifact of the GPT-2 implementation that this was adapted from. In theory they are different (n_positions is the amount of positions that have a positional embedding while n_ctx is the maximum sequence that can be fed into the coarse transformer). There are some applications where you could use the differences to generate sequences that are longer than n_ctx by feeding the most recent n_ctx - 1 elements back into the model while continuing to iterate the position indices. However, our experiments don't make use of these capabilities so the variables could be combined (likely dropping n_positions and replacing the few uses of that with n_ctx).


thorium policies point to chrome registry keys

wouldnt the policies not apply since its set to policies\google\chrome instead of policies\thorium or similar?
that's what keeping me from using the policies archive

Thorium recognizes policies from:
HKEY_LOCAL_MACHINE\SOFTWARE\Policies\Chromium

then the policy files provided are wrong, correct?
wouldn't it be better for thorium policies to be under its own name?

the policy files provided

Where did you find these files?

the policy files provided

Where did you find these files?

its in the latest release, its just google policy definition files

its in the latest release, its just google policy definition files

Found them.
They are under a previous release.

@Juesto @nicolaasjan The latest policies in the latest release are incorrect. Next release will have this corrected.


Compatibility array

DOCUMENTATION ENHANCEMENT
What would you like to be added:
It would be awesome to have a compatibility array listing OS and kernel versions  compatible with pre-built kernel modules availables in the s3 bucket.
URL of the bucket : https://s3.amazonaws.com/download.draios.com/stable/sysdig-probe-binaries/
Why is this needed:
This information is needed to have the choice to change the kernel version of our host instead of trying to build Falco from source.

Until an offical array is provided, you have it HERE

Okay Thanks !


subclassing from UPath

Not sure if I am doing this properly, but subclassing UPath is challenging, in part due to this check in the __new__ method:
https://github.com/Quansight/universal_pathlib/blob/dbc57f7ec4c8bb8b2e22975f6b62e25b2b82ecff/upath/core.py#L12
I am curious if this could be changed to:
        if issubclass(cls, UPath):

This would return True even for issubclass(UPath, UPath).
Happy to submit a PR if you are ok with this change.
My surprisingly challenging attempt to subclass UPath to add some convenience methods has really grown my appreciation for what this project has accomplished.
On a related note, after digging deeper into the code, and exploring the pathlib implementation, I am wondering if UPath would benefit from having something like a _URI_Flavour derived from the pathlib._Flavour class.  This might make it easier to have consistent path handling across the fsspec based implementations.

It it can pass all the tests, I don't see any reason why that couldn't be merged. Please do submit a PR.
I'm interested in the _URI_Flavour idea too, but should probably be in a different issue. Would you mind opening another for that?


Build 14361:  Memory leak / issue in phantomjs

One of my untiTests seemed to get stuck in phantomJS.  Task manager showed a consistent increase in memory usage:

I left it continue until it nearly hit 100% and killed it via ctrl+c in the bash shell:

Steps to reproduce:
(install nodejs 6.2.1)
mkdir ./someDir
cd ./someDir
git clone https://github.com/hpcc-systems/Visualization.git
cd Visualization
git checkout candidate-1.10.x
npm install -g gulp
npm install -g bower
npm install -g jshint
npm install
bower install
sudo apt-get install phantomjs
npm test

Restarting that step reproduced the issue - but at a different test:
gulp unitTestBuildNonAMD

@GordonSmith - Thanks for reporting this and the detailed analysis. I tried to repro this on 14361, but the "npm test" command is failing with the error below. Any idea why?

Looks like your npm install failed?

Yes, thanks. Had to install it with the -g option and it seems to work now. Onto the repro now.

@sunilmut Any update on this investigation?


chore: choose if init scale when no LastActiveTime

Provide a description of what has been changed
We can now disable the Scale to MinReplicaCount / IdleReplicaCount when LastActiveTime is nil by just providing the allowInitScale parameter to false. On such case, the Scale will wait for the CoolDown period
Checklist

[ ] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[ ] Tests have been added
[ ] A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
[ ] A PR is opened to update the documentation on (repo) (if applicable)
[ ] Changelog has been updated and is aligned with our changelog requirements

Fixes #

/run-e2e

Hey,
After trying this and more on my cluster I've understand it will be much more complicated to do what I want because currently it doesn't work with all the logic behind the code and here is why :
On the case the ScaledObject is not Active, it will try to Scale Down to Zero or Idle the Deployment, and it will only update the LastActiveTime when it is Active or when it will Scale Up from Zero or Idle.
What  I tried to do just created an infinite loop when the system tried to scale down to zero because minReplicaCount is 0 but then it stay at replica=1 for instance, because it won't be enough to make the ScaleObject Active..
From what I understand, to be Active it needs to clearly scale down to zero or idle currently in order then to add Hpa and activate the trigger only if the event is present to Scale, and then we are at the state I'm interested:

Hpa created with minReplica, desiredReplica,
CurrentReplica = 1 for example

ScaledObject minReplicaCount=0

ScaledObject Active so that on the case we don't have events anymore, it will scale down to 0 after the Cooldown period

I'm not sure it's possible without breaking all the code logic, make you have some ideas here to be able to be at this state without the first Scale Down?

@JordanGoasdoue the more I think about what you're trying to do, the more it seems like you just want minReplicaCount to be set to 1. Is the only difference that you want to scale down to zero after a delay when the ScaledObject is first created?

Yes exactly.

Got it. In this case, it may make sense to add a new status so you can set a timer off of that.

@JordanGoasdoue Any thought if we can close this PR soon? We are planning to release in 2 weeks.

Thanks for the update @JordanGoasdoue!


Cannot attach files

When you click "attach" after choosing the file to attach, you get some kind of crash icon and can't even save the message.

Sorry, there's nothing we can do about actual incompatibilities. The only thing this plugin does is tell the OWA server that the browser is a different browser by sending a different user agent (UA) string.

Leaving open to let other people know.

This seems to have been fixed. I used to have this issue but in the last 6 months or so it works fine. Either an update to OWA itself or Chromium have fixed this issue and files can be attached as normal.
There is a minor bug with URLs where it will add an extra hyperlink if you paste or type a URL into an email. It affects me a fair amount so I'll try to track it down and add a PR.


Create measurement

See new PR with all features implemented in measurement api: https://github.com/tarent/gomulocity/pull/9


[topic-branch PR] Replace oldtagset with tagset

This is a rather literal replacement of the old tagset functionality (HashedTags, HashingTagsetAccumulator) with the new APIs in ./pkg/tagset.
The intention is that the diff should have high locality: each hunk should be reviewable on its own, more or less.  One notable exception to this is ./pkg/tagger/tagstore.EntityTags#computeCache, where the rewrite was a bit more substantial to use the SlicingBuilder, and involves storing a few more slices that were previously computed on the fly.
Subsequent commits to the topic branch will refactor to improve performance and address the TODO's.  You can see work-in-progress on those subsequent commits in https://github.com/DataDog/datadog-agent/compare/dustin.mitchell/pkg-tagset-oldtagset.

Since this is merging to a topic branch, I'll merge once the fixes suggested above are green.

hm, I think I need to rebase the topic branch!


Update the lxml version requirement to 4.9.1

Requirements failed to install due to stale lxml version; updating to 4.9.1.

thanks @wdamon-intel ! question, why do we need 4.9.1? latest available in Ubuntu from official repo is 4.5.0
https://packages.ubuntu.com/search?keywords=python-lxml
while on suse 15.4 is 4.7.1
https://software.opensuse.org/package/python-lxml
do we need something specific from 4.9.1 or is just that you used latest from https://lxml.de/installation.html?

We do not need anything specific from 4.9.1; I just used the latest version listed on the lxml website. Note, this seemed to work correctly on Ubuntu 22.04 LTS.
From: Jaime Arteaga @.>
Sent: Wednesday, October 26, 2022 2:35 PM
To: oneapi-src/level-zero-spec @.>
Cc: Damon, Will @.>; Mention @.>
Subject: Re: [oneapi-src/level-zero-spec] Update the lxml version requirement to 4.9.1 (PR #31)
thanks @wdamon-intelhttps://github.com/wdamon-intel ! question, why do we need 4.9.1? latest available in Ubuntu from official repo is 4.5.0
https://packages.ubuntu.com/search?keywords=python-lxml
while on suse 15.4 is 4.7.1
https://software.opensuse.org/package/python-lxml
do we need something specific from 4.9.1 or is just that you used latest from https://lxml.de/installation.html?
—
Reply to this email directly, view it on GitHubhttps://github.com/oneapi-src/level-zero-spec/pull/31#issuecomment-1292442203, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A25YXTLSWPSDLPHVRRWKA4TWFF2TRANCNFSM6AAAAAARPIQRAI.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

thanks @wdamon-intel . Do you think it would be possible to post the PR with 4.5.0, just to require what is already available in the official repos?

thanks @wdamon-intel . Which error do you see, could you share?
4.9.1 is latest release from source, but not from available official repos, so I'm just trying to see how to minimize burden on users forcing them to install dependencies that are not coming in the official repos.

@jandres742 Here is an example from Windows, though Linux issues a similar error for a different file (libxml/xmlversion.h).
C:\Users\<username>\AppData\Local\Temp\xmlXPathInitsrt1nyno.c(1): fatal error C1083: Cannot open include file: 'libxml/xpath.h': No such file or directory
      error: command 'C:\\Program Files (x86)\\Microsoft Visual Studio\\2019\\Professional\\VC\\Tools\\MSVC\\14.29.30133\\bin\\HostX86\\x64\\cl.exe' failed with exit code 2
      *********************************************************************************
      Could not find function xmlCheckVersion in library libxml2. Is libxml2 installed?
      *********************************************************************************
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: legacy-install-failure

× Encountered error while trying to install package.
╰─> lxml

note: This is an issue with the package mentioned above, not pip.

thanks @wdamon-intel . ok, let's merge it. I tested it on my side on Linux w/o any issues.


Use composer to autoload classes

Let composer handle the require'ing of the Tokens and Util classes.
If this is merged, composer dump-autoload needs to be run once so it regenerates the autoload file.

Looks like a useful shorthand indeed.


Store simulator Backend in FakeBackend instance

Summary
This commit updates the FakeBackend and FakeBackendV2 classes
(FakeLegacyBackend is being removed in #7886) to only build a noise
model from the stored properties one. This is done by initializing the
internal simulator object only once at storing it as an instance
attribute. The noise model is then constructed once at object
initialization and used to update the backend at the same time.
Details and comments
Fixes #7500
Fixes #7911

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

56 of 65 (86.15%)  changed or added relevant lines in 1 file are covered.
16 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.01%) to 83.929%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/test/mock/fake_backend.py
56
65
86.15%

Files with Coverage Reduction
New Missed Lines
%

qiskit/test/mock/fake_backend.py
2
80.56%

qiskit/test/mock/utils/backend_converter.py
14
76.53%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
54207

Relevant Lines:
64587

💛  - Coveralls


Allow specifying mime type on upload

Useful for files without extension, as auto-detection doesn't work in that case.

Thanks @MartinKolarik for the PR.
Merging

Thank you.


webdev runs as http only

Need to be able to run webdev as https as well as http to test ssl stuff.

In vue.config.js, set:
    https: {
      key: fs.readFileSync('/srv/maverick/data/security/ssl/web/mavweb.key'),
      cert: fs.readFileSync('/srv/maverick/data/security/ssl/web/mavweb.crt'),
      ca: fs.readFileSync('/srv/maverick/data/security/ssl/ca/mavCA.pem'),
    },


CI Failed

https://github.com/lean-dojo/LeanCopilot/actions/runs/8420847569/job/23057242430

Thanks, it's probably because x86-64_MacOS is tested but we no longer generate binaries for that architecture. Let me adjust the CI to fix it.

The original problem should have been fixed in Commit b983b89. Another problem is revealed when running the test for using LeanCopilot in an external project on ubuntu-latest, where lake build fails with no helpful log even in the verbose mode. The problem is pretty weird as testing on exactly the same platform with the same steps leads to no errors at all. I'm looking more into it.

Looks like it's working now.

Yeah it works now. PR #65 resolves the last few warnings.


Update to<EMAIL_ADDRESS>and handle jquery-less usage

Would be great to get this merged and get rid of these nasty deprecations: ```
DEPRECATION: ember-cli-babel 5.x has been deprecated. Please upgrade to at least ember-cli-babel 6.6. Version 5.2.8 located: clientmate -> ember-brokermate-components -> ember-notify -> ember-cli-babel
DEPRECATION: ember-cli-babel 5.x has been deprecated. Please upgrade to at least ember-cli-babel 6.6. Version 5.2.8 located: clientmate -> ember-brokermate-components -> ember-notify -> ember-string-ishtmlsafe-polyfill -> ember-cli-babel

Can someone restart the Travis build. I think this should pass on beta now.


Using TCACoordinators within Lists

Hi, thank you for taking the time to create this library.  I have been reading through the documentation however, it's not clear to me how to achieve an example of using a List view type that has tappable rows (that aren't buttons).  Normally, using a NavigationLink inside of a List type affords some extra functionality such as highlighting the row, selection and having the chevron icon to indicate that the row is tappable and will push the next screen.
Looking at your examples, there are Button view types nested in a List to achieve this functionality.  I might be missing something if you can clarify this use case for me.

Hi @silverfoxlabs, thanks for raising this issue. Sadly NavigationLink doesn't offer a way to override its action, and Button doesn't offer a way to style it like a NavigationLink, so the best I can offer are workarounds.
The following gets you most of the styling, but the row background selection state doesn't persist:
public struct NavigationButton<Label: View>: View {
  let action: () -> Void
  let label: Label

  public init(action: @escaping () -> Void, @ViewBuilder label: () -> Label) {
    self.action = action
    self.label = label()
  }

  public var body: some View {
    ZStack {
      NavigationLink(isActive: .constant(false), destination: { EmptyView() }, label: { label })
      Button(action: action, label: { EmptyView() })
    }
  }
}

public extension NavigationButton where Label == Text {
  init(_ titleKey: LocalizedStringKey, action: @escaping () -> Void) {
    self.init(action: action, label: { Text(titleKey) })
  }
  init<S>(_ title: S, action: @escaping () -> Void) where S : StringProtocol {
    self.init(action: action, label: { Text(title) })
  }
}

There's some interesting discussion of these shortcomings and possible workarounds in this TCA discussion, including this gist from @tgrapperon which offers a more faithful recreation of NavigationLink's list style.
I'd be interested to hear how you find these workarounds and if you think I should add something like that to this library.


Session-specific caching mechainsm

https://github.com/whitphx/streamlit-webrtc/pull/747/files#diff-568470d013cd12e4f388206520da39ab9a4e4c3c6b95846cbc281abc1ba3c959R303

Already asked at https://discuss.streamlit.io/t/session-specific-caching/271

This can be done with session_state?
-> Are the values in session_state cleared when the session closes?

https://discuss.streamlit.io/t/revisit-session-specific-cache/27017


[Feature Request] Automove unfinished tasks to the next day.

[Feature Request] Automove unfinished tasks to the next day.

Second this.  I have so far unable to rollover finished or unfinished tasks to next day note.  Is that working for you, so next day you have a new Day Planner file with all unfinished tasks?

Closing this as stale. If you're still interested in this feature, please reopen the issue with a comment.


[CARBONDATA-1345] Update tablemeta cache after table schema has been changed

Scenario
Steps to reproduce
Start 2 spark-beeline as two different sessions, do the following steps in corresponding session:
(SESSION1)

create table T_Carbn01(Active_status String,Item_type_cd INT,Qty_day_avg INT,Qty_total INT,Sell_price BIGINT,Sell_pricep DOUBLE,Discount_price DOUBLE,Profit DECIMAL(3,2),Item_code String,Item_name String,Outlet_name String,Update_time TIMESTAMP,Create_date String)STORED BY 'org.apache.carbondata.format' TBLPROPERTIES('table_blocksize'='128');

LOAD DATA INPATH 'hdfs://hacluster/user/Ram/T_Hive1.csv' INTO table T_Carbn01 options ('DELIMITER'=',', 'QUOTECHAR'='','BAD_RECORDS_LOGGER_ENABLE'='true', 'BAD_RECORDS_ACTION'='REDIRECT', 'FILEHEADER'='Active_status,Item_type_cd,Qty_day_avg,Qty_total,Sell_price,Sell_pricep,Discount_price,Profit,Item_code,Item_name,Outlet_name,Update_time,Create_date');

(SESSION2):

update t_carbn01 set(Active_status) = ('TRUE') where Item_type_cd = 41;

(SESSION1):

Drop table t_carbn01;

create table T_Carbn01(Active_status String,Item_type_cd INT,Qty_day_avg INT,Qty_total INT,Sell_price BIGINT,Sell_pricep DOUBLE,Discount_price DOUBLE,Profit DECIMAL(3,2),Item_code String,Item_name String,Outlet_name String,Update_time TIMESTAMP,Create_date String)STORED BY 'org.apache.carbondata.format' TBLPROPERTIES('table_blocksize'='128');

LOAD DATA INPATH 'hdfs://hacluster/user/Ram/T_Hive1.csv' INTO table T_Carbn01 options ('DELIMITER'=',', 'QUOTECHAR'='','BAD_RECORDS_LOGGER_ENABLE'='true', 'BAD_RECORDS_ACTION'='REDIRECT', 'FILEHEADER'='Active_status,Item_type_cd,Qty_day_avg,Qty_total,Sell_price,Sell_pricep,Discount_price,Profit,Item_code,Item_name,Outlet_name,Update_time,Create_date');

(SESSION2):

update t_carbn01 set(Active_status) = ('TRUE') where Item_type_cd = 41;

Outputs
message are as below:
Error: java.lang.RuntimeException: Update operation failed. Job aborted due to stage failure: Task 0 in stage 14.0 failed 4 times, most recent failure: Lost task 0.3 in stage 14.0 (TID 29, master, executor 2): java.io.IOException: java.io.IOException: Dictionary file does not exist: hdfs://user/hive/warehouse/carbon.store/default/t_carbn01/Metadata/ddfb3bc8-2fea-41fe-a4ff-18588df41aec.dictmeta
    at org.apache.carbondata.core.cache.dictionary.ForwardDictionaryCache.getAll(ForwardDictionaryCache.java:146)
    at org.apache.spark.sql.DictionaryLoader.loadDictionary(CarbonDictionaryDecoder.scala:686)
    at org.apache.spark.sql.DictionaryLoader.getDictionary(CarbonDictionaryDecoder.scala:703)
    at org.apache.spark.sql.ForwardDictionaryWrapper.getDictionaryValueForKeyInBytes(CarbonDictionaryDecoder.scala:654)
    at org.apache.spark.sql.catalyst.expressions.GeneratedClass$GeneratedIterator.processNext(Unknown Source)
    at org.apache.spark.sql.execution.BufferedRowIterator.hasNext(BufferedRowIterator.java:43)
    at org.apache.spark.sql.execution.WholeStageCodegenExec$$anonfun$8$$anon$1.hasNext(WholeStageCodegenExec.scala:378)
    at org.apache.spark.sql.execution.columnar.InMemoryRelation$$anonfun$1$$anon$1.hasNext(InMemoryRelation.scala:132)
    at org.apache.spark.storage.memory.MemoryStore.putIteratorAsValues(MemoryStore.scala:215)
    at org.apache.spark.storage.BlockManager$$anonfun$doPutIterator$1.apply(BlockManager.scala:1041)
    at org.apache.spark.storage.BlockManager$$anonfun$doPutIterator$1.apply(BlockManager.scala:1032)
    at org.apache.spark.storage.BlockManager.doPut(BlockManager.scala:972)
    at org.apache.spark.storage.BlockManager.doPutIterator(BlockManager.scala:1032)
    at org.apache.spark.storage.BlockManager.getOrElseUpdate(BlockManager.scala:715)
    at org.apache.spark.rdd.RDD.getOrCompute(RDD.scala:335)
    at org.apache.spark.rdd.RDD.iterator(RDD.scala:286)
    at org.apache.spark.rdd.MapPartitionsRDD.compute(MapPartitionsRDD.scala:38)

Input data
sample for input data：
TRUE,2,423,3046340,200000000003454300, 121.5,4.99,2.44,SE3423ee,asfdsffdfg,EtryTRWT,2012-01-12 03:14:05.123456729,2012-01-20
TRUE,3,453,3003445,200000000000003450, 121.5,4.99,2.44,SE3423ee,asfdsffdfg,ERTEerWT,2012-01-13 03:24:05.123456739,2012-01-20
TRUE,4,4350,3044364,200000000000000000, 121.5,4.99,2.44,SE3423ee,asfdsffdfg,ERTtryWT,2012-01-14 23:03:05.123456749,2012-01-20
TRUE,114,4520,30000430,200000000004300000, 121.5,4.99,2.44,RE3423ee,asfdsffdfg,4RTETRWT,2012-01-01 23:02:05.123456819,2012-01-20
FALSE,123,454,30000040,200000000000000000, 121.5,4.99,2.44,RE3423ee,asfrewerfg,6RTETRWT,2012-01-02 23:04:05.123456829,2012-01-20
TRUE,11,4530,3000040,200000000000000000, 121.5,4.99,2.44,SE3423ee,asfdsffder,TRTETRWT,2012-01-03 05:04:05.123456839,2012-01-20
TRUE,14,4590,3000400,200000000000000000, 121.5,4.99,2.44,ASD423ee,asfertfdfg,HRTETRWT,2012-01-04 05:06:05.123456849,2012-01-20
FALSE,41,4250,00000,200000000000000000, 121.5,4.99,2.44,SAD423ee,asrtsffdfg,HRTETRWT,2012-01-05 05:07:05.123456859,2012-01-20
TRUE,13,4510,30400,200000000000000000, 121.5,4.99,2.44,DE3423ee,asfrtffdfg,YHTETRWT,2012-01-06 06:08:05.123456869,2012-01-20

Analyze
In the error message, it says the dictmeta doesnot exist.
Actually this file is generated during the first load operation in SESSION1,And the tablemeta is cached in SESSION2 when doing update operation in SESSION2. After DELETE-LOAD operation in SESSION1, old dictionary files has been deleted and new dictionary files are generated in SESSION1. But in SESSION2, when doing update operation, we still use the outdated tablemeta from cache which refers to the dictmeta that were outdated, thus causing the error.
To solve this problem, we should refresh the cache for tableMeta when the corresponding data schema has been updated.
Solution
Refresh the tablemeta cache when table schema has been changed.
Since HiveSessionState.lookupRelation is slow(especially in concurrent query scenario), dont call this method when table schema has not been changed.
Notes
I've tested the scenario in my environment and it is OK.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@sraghunandan pls review this PR

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3283/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/688/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/689/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3284/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3316/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/720/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/721/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3317/

LGTM


Proposal for extending post transaction api

Extend POST transaction api to include query parameter mode which would enable the follows,

asynchoronously post the transaction
post the transaction and return after its validated
post the transaction and return after its commited

Codecov Report

Merging #1951 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1951   +/-   ##
=======================================
  Coverage   98.55%   98.55%
=======================================
  Files          60       60
  Lines        3040     3040
=======================================
  Hits         2996     2996
  Misses         44       44

I think this is a nice proposal and a good way to expose that Tendermint API.
I wonder what the failure modes are, and how we'll handle them.
For example, I suppose it's possible someone to POST a transaction in the sync mode but then the CheckTx takes a long time to happen (or maybe never happens). Is there some timeout (implicit or explicit)? If so, what happens when it's reached?
I wonder about other failure modes.

@ttmc that is valid concern. The tendermint documentation doesn't provide much details regarding this. This aspect needs to properly explored before we decide to implement this feature.


ru_ru.json lang.

Ooh fancy! ... We're about to do a giant refactor though, so I would hold on a bit and rebase once we merge multiloader into main.

Awesome!


IP

How do you get the ip?

The Minecraft server will automatically bind to <IP_ADDRESS>, so it is accessible on all IP addresses for your computer. That includes <IP_ADDRESS> and any other addresses in ipconfig or ifconfig. Of course outside users will need to be allowed via any firewalls that may exist on your computer/network.


Make the memory soft cap configurable and make Settings non-exhaustive

This is the continuation of https://github.com/paritytech/ws-rs/pull/6

Looks good. Regarding #6 (comment), I think it's possible to change
https://github.com/paritytech/ws-rs/blob/401d8c1e85933a7f003f89319cd8ebd7796b6358/src/circular_buffer.rs#L144
to
if self.remaining() > limit / 2 || self.current_capacity() < 2 * limit {

and then it would safe to apply in all cases, not just when self.is_empty().

Hmm... so only shrink it if the amount of data remaining in the buffer is at most half of the limit and the current capacity is at least double the limit? But couldn't you still do something like this then?

Write limit * 0.5 bytes. Assume the capacity is also limit * 0.5 bytes big.
Write 2 bytes. The buffer contains limit * 0.5 + 2 bytes of data. Capacity grows to limit * 2. The second condition is fulfilled.
Read 2 bytes. The buffer contains limit * 0.5 bytes. The first condition is fulfilled.
The buffer is reallocated. limit * 0.5 bytes is copied to the new buffer at the cost of sending only 2 bytes by the attacker. The new capacity is again limit * 0.5 bytes.
Go to (2).

Did I get that right?
(Of course whenever this is actually feasible is another matter altogether, since there would be a need to maintain a constant limit * 0.5 bytes of data in the buffer, and we're the ones doing the reads, not the attacker.)
A possible fix for this would be to change the second condition to require at least double the limit plus 1 of capacity, I think? (So change < 2 * limit to <= 2 * limit.)
Also, the second condition would prevent the buffer to be freed even when the buffer is empty and it's over the soft cap but still under the double of the soft cap (so the soft cap would effectively become a half soft cap then). I think special-casing this when the buffer is empty so that it continues to still work as it does now would be a good idea.

The buffer contains limit * 0.5 + 2 bytes of data. Capacity grows to limit * 2

Why does capacity grow to 2 * limit in this case? It grows that large when there is at limit of data. The idea here is that the user will have to write/read the amount of bytes proportional to the reallocation.
I've done amortized analysis on this during my university times. Basically, if you want to implement shrinking operation on a Vec, you should shrink capacity by 2 when capacity / size >= 4. This guarantees O(1) amortized complexity in combination with 2x capacity growth.

The buffer contains limit * 0.5 + 2 bytes of data. Capacity grows to limit * 2

Why does capacity grow to 2 * limit in this case?

Ah, sorry, yeah, you're right of course. That was a brainfart on my part. I had a feeling I was probably wrong, considering how late it is here. (:

No worries :) Glad we're on the same page now. Feel free to merge and publish w/ or w/o the suggested improvement.

@ordian Okay, I've added a new commit with your suggestion! It'd be great if you could take one final look at it. (:

LGTM, thanks!

Thanks!
I'd be great if you could tag and release a new version when you have a moment. (I'm not sure whenever I can do it myself; probably not?) I'll take care of bumping this in polkadot/substrate later.

I'd be great if you could tag and release a new version when you have a moment.

Published.

I'm not sure whenever I can do it myself; probably not?

Once you're added to the https://github.com/orgs/paritytech/teams/core-devs/members.

I'll take care of bumping this in polkadot/substrate later.

That might be not so easy, because substrate still uses jsonrpc v15. But there is no rush.


Generic camera unaccessible

Home Assistant release (hass --version):
0.38.3
Python release (python3 --version):
3.4.2
Component/platform:
camera:
  - platform: generic
    still_image_url: http://<IP_ADDRESS>/snapshot.cgi?user=user&pwd=password&count=3

Description of problem:
The camera image is not displayed. (see log) : "Error loading image"
Expected:
Camera image displayed
Problem-relevant configuration.yaml entries and steps to reproduce:
camera:
  - platform: generic
    still_image_url: http://<IP_ADDRESS>/snapshot.cgi?user=user&pwd=password&count=3

Traceback (if applicable):
17-02-16 16:00:06 ERROR (MainThread) [aiohttp.server] Error handling request
Traceback (most recent call last):
  File "/home/homeassistant/.homeassistant/deps/aiohttp/web_server.py", line 62, in handle_request
    resp = yield from self._handler(request)
  File "/home/homeassistant/.homeassistant/deps/aiohttp/web.py", line 270, in _handle
    resp = yield from handler(request)
  File "/usr/lib/python3.4/asyncio/coroutines.py", line 143, in coro
    res = yield from res
  File "/usr/lib/python3.4/asyncio/coroutines.py", line 143, in coro
    res = yield from res
  File "/srv/homeassistant/homeassistant_venv/lib/python3.4/site-packages/homeassistant/components/http/__init__.py", line 427, in handle
    result = yield from result
  File "/srv/homeassistant/homeassistant_venv/lib/python3.4/site-packages/homeassistant/components/camera/__init__.py", line 253, in get
    response = yield from self.handle(request, camera)
  File "/srv/homeassistant/homeassistant_venv/lib/python3.4/site-packages/homeassistant/components/camera/__init__.py", line 272, in handle
    image = yield from camera.async_camera_image()
  File "/srv/homeassistant/homeassistant_venv/lib/python3.4/site-packages/homeassistant/components/camera/generic.py", line 115, in async_camera_image
    url, auth=self._auth)
  File "/home/homeassistant/.homeassistant/deps/aiohttp/client.py", line 577, in __iter__
    resp = yield from self._coro
  File "/home/homeassistant/.homeassistant/deps/aiohttp/client.py", line 213, in _request
    proxy=proxy, proxy_auth=proxy_auth, timer=timer)
  File "/home/homeassistant/.homeassistant/deps/aiohttp/client_reqrep.py", line 73, in __init__
    assert isinstance(url, URL), url
AssertionError: http://<IP_ADDRESS>/snapshot.cgi?user=user&pwd=password&count=3

Additional info:
The URL http://<IP_ADDRESS>/snapshot.cgi?user=user&pwd=password&count=3 work fine in my explorer.
It was working on HA 0.37

See https://github.com/home-assistant/home-assistant/issues/6015


Can we have file pattern support in addition/instead of multiple file inputs

In enterprise projects where the coverage results' files are huge and dynamic in nature, can we pass on a 'FILE_PATTERN'  instead of supplying individual inputs?

Hello, are there any updates? seems like usefull feature? or maybe anyone could recommend alternative solutions?

Hello, are there any updates? seems like usefull feature? or maybe anyone could recommend alternative solutions?

A workaround:
npx cobertura-merge -p -o output.xml $(ls coverage/**/cobertura-coverage.xml | awk -v ORS=" " -v OFS="" '{ print "package", NR, "=", $0 }')

A workaround:
npx cobertura-merge -p -o output.xml $(ls coverage/**/cobertura-coverage.xml | awk -v ORS=" " -v OFS="" '{ print "package", NR, "=", $0 }')

working script for me
npx cobertura-merge -p -o output.xml $(ls coverage/**/cobertura-coverage.xml | awk -v ORS=" " -v OFS="" '{ print "package", NR, "=", $0 }')


Help with realtime resampling

I am trying to resample from 44100 to 16000 wav audio bytes in realtime using NWaves but the resulting audio is noisy and broken
Could someone provide an example on how to resample audio from a stream in realtime using NWaves please?
Below is a snippet of my code

_continuousSpeechStream is a Stream that keeps growing with wav audio bytes without header as the connection is on.
the bytes are read from the stream to be  resampled before adding header and processing.

This is what I have so far:
var _chunkSize = 35192 - 44 - 2; // size: 8192 could be used here
var buffer = new byte[checked((uint)Math.Min(_chunkSize, (int)_continuousSpeechStream.Length))];
int bytesRead = 0;

while (_continuousSpeechStream.CanRead &&
          !waitTask.Wait(300) && _websocketClient.IsConnected)
{
            bytesRead = await _continuousSpeechStream.ReadAsync(buffer, 0, buffer.Length);

            if (bytesRead > 0)
            {
                         // ----------- START RESAMPLE
                         var sizeInFloats = _chunkSize / sizeof(short);  // for Pcm 16bit

                         var f = new float[1][];
                         f[0] = new float[sizeInFloats];

                         ByteConverter.ToFloats16Bit(buffer, f);

                         var signal = new DiscreteSignal(44100, f[0]);
                         var resampledSignal = Operation.Resample(signal, 16000);

                         var resampledBytes = new byte[resampledSignal.Length * 2];
                         ByteConverter.FromFloats16Bit(new float[][] { resampledSignal.Samples }, resampledBytes);
                         // ---------- END RESAMPLE

                         // .... process resampled bytes
                         var tempStream = new MemoryStream();

                         // Write WAVE header the first time ONLY
                         if (sendRIFFHeader)
                         {
                                tempStream.WriteWaveHeader(channelCount.Value, sampleRate.Value, bitsPerSample.Value, 0);

                                // the RIFF header is only written in the first time
                                sendRIFFHeader = false;
                         }

                          // write the bytesRead to send to the websocket service
                          await tempStream.WriteAsync(resampledBytes, 0, resampledBytes.Length);

                         // DO STUFF
            }
}

Hi, realtime resampling is not supported. I've been planning to revise and improve resampling functions in NWaves since long ago, but unfortunately I can't find enough time for it

Ok =/. Thanks for your answer

the new ARM based laptops are fast, and low power,  like macs M chips but for windows laptops. and they have FPGA based chips and you can use Hastlayer and c# and burn Ai , or  real DSP algorithms, with  , fixed point filters.  kernels, and FFTs TDFFT whatever.    low power, low latency and not needing processor affinity or worked about blocking GC during voice or record.  you can make FIR kernels , EQ and do hotword detectors on it.   mid 2024.   I looked at SDL and XNA guta ( monogame) and a i think it puts effects trough the DSP via COM and c# wrapper and or Asio or SDL..  If we succeed to get reverb to work i can see if its can be fixed, because a pop or crackles on a speaker is a super high F noise..  i didnt know that Intell has DSP programmable by C and that ARM has similar, its been for years. but now they are taking enum types, and posit and fixed point.  floating points are just not periodic or discreet for ffts but ok enough for images, so waiting to see what come out of the Ai and chip companies might help  you decide if ifs worth the trouble.


want to ignore https CA ，but no effect，how to set？

Package
dio
Version
5.1.1
Output of flutter doctor -v
dio 5.1.1，want to ignore https CA，setting ：

(_dio.httpClientAdapter as DefaultHttpClientAdapter).onHttpClientCreate =
        (HttpClient client) {
      client.badCertificateCallback =
          (X509Certificate cert, String host, int port) => true;
    };

but no effect

If I want to forcibly ignore https certificate verification, what do I need to do what setting to make https requests？

Dart Version
2.19.5
Steps to Reproduce
setting
Expected Result
success
Actual Result
fail

fix


[Table - MultiSelectMode | Horizon High Contrast White Mode ]: On Table Selection in safari browser the row color is turning black & all the content becomes unreadable

OpenUI5 version: 1.121.0
Browser/version (+device/version): Safari
Any other tested browsers/devices(OK/FAIL): Chrome its Working Fine
URL (minimal example if possible): https://ui5.sap.com/#/entity/sap.m.Table/sample/sap.m.sample.TableMultiSelectMode
User/password (if required and possible - do not post any confidential information here):
Steps to reproduce the problem:
Open in Safari Browser

Select Responsive Table - MultiSelectMode
Click on the settings button which is next to Download button
Change the Settings to Horizon High Contrast White & Compact.
Select the table row & the background become black & all the content becomes un readable.

What is the expected result?
Ideally the background color must be #5c93ff for Horizon High Contrast White mode when we select a row for the table.
What happens instead?
The table row background become black & all the row content becomes unreadable.
Any other information? (attach screenshot if possible)

Hello @achuth2912 ,
Thank you for sharing this finding. I've created an internal incident DINC0088613. The status of the issue will be updated here in GitHub.
Regards,
Iliana

Tthanks for raising the issue @achuth2912.
The issue should now be fixed in the upcoming UI5 version as well as in 1.120, 1.108, 1.96 and 1.84


[Discussion] Strategy with fixed values.

In order to accommodate multiple strategies for different tables/indexes I am proposing the following strategy schema update for fixed values:
ReadCapacity.Fixed: Number
WriteCapacity.Fixed: Number
The Provisioner.js file would have to be adapted, since there is quite some conditional logic on Read/WriteCapacity.Increment and Read/WriteCapacity.Increment.When.
Currently the following strategy would have to be set when using fixed values for specific tables/indexes without causing any errors in Provisioner.js:
{
  "ReadCapacity": {
    "Min": 1,
    "Max": 1,
    "Increment":{
      "To": {
        "Units": 1
      },
      "When":{
      }
    },
    "Decrement":{
      "To": {
        "Units": 1
      },
      "When":{
      }
    }
  },
  "WriteCapacity": {
    "Min": 1,
    "Max": 1,
    "Increment":{
      "To": {
        "Units": 1
      },
      "When":{
      }
    },
    "Decrement":{
      "To": {
        "Units": 1
      },
      "When":{
      }
    }
  }
}

I'd be happy to add a pull request should this functionality be approved by more people/maintainer.

You should just be able to specify the min and max to the same value and that's it, if that's not working then that would be a bug.  I don't think I would change it to fixed as it would only save a single line in that case... Am I understand you correctly?

Hi Tom,
When you use the min and max values alone, no increment or decrements to that value are performed due to the following code blocks such as in the isReadCapacityIncrementRequired function in the Provisioner.js file line 50:
if (config.ReadCapacity.Increment == null) {
     return false;
}

When adding an "Increment" or "Decrement" attrbute you also seem to need at least a "To" attribute and an empty "When" block. Otherwise either no change is performed or "not defined errors" occur.

Small update, the minimum required configuration for a currently working fixed value currently is:
{
  "ReadCapacity": {
    "Min": 1,
    "Max": 1,
    "Increment":{
      "When":{}
    },
    "Decrement":{
      "When":{}
    }
  },
  "WriteCapacity": {
    "Min": 1,
    "Max": 1,
    "Increment":{
      "When":{}
    },
    "Decrement":{
      "When":{}
    }
  }
}

Ok that's a bug then, I'm making some other changes at the moment and I'll put a fix in for this also.  Thanks

Just checked a fix, should work now


multiple columns IN operation is not support

Hi :
i used JSqlParser - version 1.4
for example:
// mysql
select * from a where (c1,c2) IN ((1,2),(3,4))

I think this is standard syntax and should be supported

Do you want to contribute? PRs are welcome.

@wumpz thanks for your great project first! I also encounter this issue and I tried define a function contains to solve it but seems also not working.
The SQL I tried is select a, b, c, d from table where contains(((1, 2, 3, 4), (5, 6, 7, 8)), (a, b, c, d))
The version is 1.3
We have our own AST so there are spaces for us to perform. Do you have any advices to solve this? For example a specific syntax and so on.

I have solved this by define a tuple function, so the SQL looks like
select a, b, c, d from table where contains(tuple(tuple(1, 2, 3, 4), tuple(5, 6, 7, 8)), tuple(a, b, c, d))

we also encountered

I think @wumpz 's guess was correct, the problem starts at the where (:
CCJSqlParserUtil.parse("select a,b from foo where (a,b) in ((1,2),(3,4))");

Call:	Statement
  Call:	SingleStatement
    Call:	Select
      Call:	SelectBody
        Call:	SetOperationList
          Call:	PlainSelect
            Consumed token: <<K_SELECT>: "select" at line 1 column 1>
            Call:	getOracleHint
            Return: getOracleHint
            Call:	SelectItemsList
              Call:	SelectItem
                Call:	SelectExpressionItem
                  Call:	SimpleExpression
                    Call:	ConcatExpression
                      Call:	BitwiseAndOr
                        Call:	AdditiveExpression
                          Call:	MultiplicativeExpression
                            Call:	BitwiseXor
                              Call:	PrimaryExpression
                                Call:	Column
                                  Call:	RelObjectNameList
                                    Call:	RelObjectNameExt
                                      Call:	RelObjectName
                                        Call:	RelObjectNameWithoutValue
                                          Consumed token: <<S_IDENTIFIER>: "a" at line 1 column 8>
                                        Return: RelObjectNameWithoutValue
                                      Return: RelObjectName
                                    Return: RelObjectNameExt
                                  Return: RelObjectNameList
                                Return: Column
                              Return: PrimaryExpression
                            Return: BitwiseXor
                          Return: MultiplicativeExpression
                        Return: AdditiveExpression
                      Return: BitwiseAndOr
                    Return: ConcatExpression
                  Return: SimpleExpression
                Return: SelectExpressionItem
              Return: SelectItem
              Consumed token: <"," at line 1 column 9>
              Call:	SelectItem
                Call:	SelectExpressionItem
                  Call:	SimpleExpression
                    Call:	ConcatExpression
                      Call:	BitwiseAndOr
                        Call:	AdditiveExpression
                          Call:	MultiplicativeExpression
                            Call:	BitwiseXor
                              Call:	PrimaryExpression
                                Call:	Column
                                  Call:	RelObjectNameList
                                    Call:	RelObjectNameExt
                                      Call:	RelObjectName
                                        Call:	RelObjectNameWithoutValue
                                          Consumed token: <<S_IDENTIFIER>: "b" at line 1 column 10>
                                        Return: RelObjectNameWithoutValue
                                      Return: RelObjectName
                                    Return: RelObjectNameExt
                                  Return: RelObjectNameList
                                Return: Column
                              Return: PrimaryExpression
                            Return: BitwiseXor
                          Return: MultiplicativeExpression
                        Return: AdditiveExpression
                      Return: BitwiseAndOr
                    Return: ConcatExpression
                  Return: SimpleExpression
                Return: SelectExpressionItem
              Return: SelectItem
            Return: SelectItemsList
            Consumed token: <"FROM": "from" at line 1 column 12>
            Call:	FromItem
              Call:	Table
                Call:	RelObjectNameList
                  Call:	RelObjectNameExt
                    Call:	RelObjectName
                      Call:	RelObjectNameWithoutValue
                        Consumed token: <<S_IDENTIFIER>: "foo" at line 1 column 17>
                      Return: RelObjectNameWithoutValue
                    Return: RelObjectName
                  Return: RelObjectNameExt
                Return: RelObjectNameList
              Return: Table
            Return: FromItem
            Call:	JoinsList
            Return: JoinsList
            Call:	WhereClause
              Consumed token: <"WHERE": "where" at line 1 column 21>
              Call:	Expression
                Call:	OrExpression
                  Call:	AndExpression
                    Consumed token: <"(" at line 1 column 27>
                    Call:	OrExpression
                      Call:	AndExpression
                        Call:	Condition
                          Call:	SimpleExpression
                            Call:	ConcatExpression
                              Call:	BitwiseAndOr
                                Call:	AdditiveExpression
                                  Call:	MultiplicativeExpression
                                    Call:	BitwiseXor
                                      Call:	PrimaryExpression
                                        Call:	Column
                                          Call:	RelObjectNameList
                                            Call:	RelObjectNameExt
                                              Call:	RelObjectName
                                                Call:	RelObjectNameWithoutValue
                                                  Consumed token: <<S_IDENTIFIER>: "a" at line 1 column 28>
                                                Return: RelObjectNameWithoutValue
                                              Return: RelObjectName
                                            Return: RelObjectNameExt
                                          Return: RelObjectNameList
                                        Return: Column
                                      Return: PrimaryExpression
                                    Return: BitwiseXor
                                  Return: MultiplicativeExpression
                                Return: AdditiveExpression
                              Return: BitwiseAndOr
                            Return: ConcatExpression
                          Return: SimpleExpression
                        Return: Condition
                      Return: AndExpression
                    Return: OrExpression
                  Return: AndExpression
                Return: OrExpression
              Return: Expression
            Return: WhereClause
          Return: PlainSelect
        Return: SetOperationList
      Return: SelectBody
    Return: Select
  Return: SingleStatement
Return: Statement
Exception in thread "main" net.sf.jsqlparser.JSQLParserException
	at net.sf.jsqlparser.parser.CCJSqlParserUtil.parse(CCJSqlParserUtil.java:65)
	at net.sf.jsqlparser.parser.CCJSqlParserUtil.parse(CCJSqlParserUtil.java:40)
	at net.sf.jsqlparser.Visualizer.main(Visualizer.java:9)
Caused by: net.sf.jsqlparser.parser.ParseException: Encountered unexpected token: "," ","
    at line 1, column 29.

Comparing it with a standard a in (1,2), I noticed the path goes through a SimpleExpression:
                          Return: SimpleExpressionList
                          Consumed token: <")" at line 1 column 36>
                        Return: InExpression

I noticed that there's a PIVOT option that supports Multiple IN items, so we'd probably have to extend at least the IN to support multiple expression list items?
        String sql = "SELECT * FROM ( select a,b from foo) PIVOT ( func(column2) for column2 in ( (1,2), (3,4)) )";
        Statement statement = CCJSqlParserUtil.parse(sql);

@AnEmortalKid 👍

Took a preliminary stab at this on the weekend:
CCJSqlParserUtil.parse("select a,b from foo where (a,b) in (1,2)");
CCJSqlParserUtil.parse("select a,b from foo where (a,b) = (1,2)");

Parses as a SimpleExpressionList (on the right).
The tricky bit is with the right part, at least:
((1,2),(3,4))

I think for that we need to have at least a (3) look ahead and look for something like:
"("
  "(" , SimpleExpressionList() ")" ("," "(" SimpleExpressionList() ")")*
")"

Got some conflict productions while trying that but i might be on the right track.


Reliably and easily submit a form?

The Form class does not have a submit().
Why not?

Thanks for the suggestion. New Form.submit() method could be used to submit html forms. I am going to release this feature with version 2.1.0. You could checkout and build ui4j from master branch to use submit method.

Hi, version 2.1.0 is released and published to maven central repository.

thx


Consider adding build tools like Gradle

Hello.
Your work is fantastic!
Do you have any plans to introduce build tools like Gradle to this repository?

Hey @enkatsu
Thanks :) hopefully it is helping you with your projects.
At this stage I have no plans to update this repository. It currently serves its purpose for me, which is purely to be able to interface SimpleBLE with a Java application on MacOS.
Just for my own understanding, what does introducing build tools / Gradle add to the repo in terms of usefulness? I might use it as a learning experience in the future if I get time!

Hello @L4m4L,
Thank you for your reply!
Introducing Gradle would make it easier to adopt this project in other Java projects. Additionally, while I haven't tried it myself, we might be able to describe the procedure for downloading native libraries.


Add NetMHC 4.0 to mhctools

No more NetMHC class.
Confusingly, test code relies on netMHC = 4.0 and netMHC-3.4 = 3.4. Happy to consider less confusing alternatives.

Other than the link, looks good to me.
I guess now I have to fix Topiary to use NetMHC4

@iskandr two new additions:

I changed the prediction_method_name from netmhc to netmhc3 and netmhc4; any reason this would break things?
I added a parser test.

@tavinathanson
I think the prediction_method_name change should be OK. :pray:
LGTM (modulo tests)

Hey,
I think the change from netmhc to netmhc3 and netmhc4 may have broken varlens-variants. I'll file this as a separate issue in that repository.

@JPFinnigan Yikes, it's not clear to me why it would have, but I'll look into it!

Hey @tavinathanson,
Here's what I experienced. Tried to run varlens-variants w/:
JPF-MBP:varlens johnfinnigan$ varlens-variants \
> --include-read-evidence \
> --reads ~/Desktop/TEMP/Results/RNA/Tumor_B16.F10/ISMMS/Tumor_B16.F10_0810.127A/BAM/HISAT2/Tumor_B16.F10_0810.127A.HISAT2.sorted.bam \
> --reads ~/Desktop/TEMP/Results/RNA/Tumor_B16.F10/ISMMS/Tumor_B16.F10_0810.131B/BAM/HISAT2/Tumor_B16.F10_0810.131B.HISAT2.sorted.bam \
> --variants ~/Desktop/TEMP/Results/WES/Tumor_B16.F10_0810/ISMMS/VCF/MuTect/Tumor_B16_F10_0810.mutect.targets.pass.vcf \
> --variants ~/Desktop/TEMP/Results/WES/Tumor_B16.F10_0810/ISMMS/VCF/Strelka/results/passed.somatic.snvs.vcf \
> --out ~/Desktop/Tumor.B16.F10.0810.mutect.targets.pass.vcf_Tumor.B16.F10.0810.strelka.passed.snvs.vcf_Tumor.B16.F10.0810.127A.HISAT2.sorted.bam_Tumor.B16.F10.0810.131B.HISAT2.sorted.bam_varlens.csv

Received the following error message:
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/2.7/bin/varlens-variants", line 9, in <module>
    load_entry_point('varlens', 'console_scripts', 'varlens-variants')()
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/pkg_resources/__init__.py", line 547, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2720, in load_entry_point
    return ep.load()
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2380, in load
    return self.resolve()
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2386, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/Users/johnfinnigan/Desktop/Utilities/Varlens/varlens/varlens/commands/variants.py", line 22, in <module>
    from .. import variant_includes
  File "/Users/johnfinnigan/Desktop/Utilities/Varlens/varlens/varlens/variant_includes.py", line 12, in <module>
    from . import mhc_binding
  File "/Users/johnfinnigan/Desktop/Utilities/Varlens/varlens/varlens/mhc_binding.py", line 19, in <module>
    import topiary
  File "/Users/johnfinnigan/Desktop/Utilities/Topiary/topiary/topiary/__init__.py", line 7, in <module>
    from .predict_epitopes import (
  File "/Users/johnfinnigan/Desktop/Utilities/Topiary/topiary/topiary/predict_epitopes.py", line 20, in <module>
    from .commandline_args import (
  File "/Users/johnfinnigan/Desktop/Utilities/Topiary/topiary/topiary/commandline_args.py", line 185, in <module>
    "netmhc": mhctools.NetMHC,
AttributeError: 'module' object has no attribute 'NetMHC'

I'm thinking this means that varlens is looking for, but not finding NetMHC. Presumably, this is because mhctools was just updated to go from NetMHC to NetMHC3 NetMHC4. Also, this brings up a f/u question--are you sure you want to include MHC binding predictions by default in varlens-variants?

Ah, yeah: the issue is that topiary is broken (see https://github.com/hammerlab/topiary/issues/44), and varlens uses topiary. I'll put out either a fix or revert today. I'll defer the varlens question to @timodonnell.

Thanks @tavinathanson. Sorry about that, I didn't mean to bother you guys on a Sunday afternoon.
Take care!

No worries, I didn't mean to break software you were using :)

@JPFinnigan I think your problem should be resolved if you use the very latest version of mhctools (2.1); I just introduced NetMHC, which automatically determines whether the software is NetMHC 3 or 4, and that should hopefully fix that varlens/topiary issue.