[LANG-1585] refine performance of StringUtils.normalizeSpace

as title.

performance test:
D:\workspace\commons-lang>mvn -test -Pbenchmark
[ERROR] Error executing Maven.
[ERROR] The specified user toolchains file does not exist: D:\workspace\commons-lang\est

D:\workspace\commons-lang>mvn test -Pbenchmark
[INFO] Scanning for projects...

[INFO] Will parse SCM ignores for exclusions...
[INFO] Parsing exclusions from D:\workspace\commons-lang\.gitignore
[INFO] Finished adding exclusions from SCM ignore files.
[INFO] 73 implicit excludes (use -debug for more details).
[INFO] 14 explicit excludes (use -debug for more details).
[INFO] 434 resources included (use -debug for more details)
[INFO] Rat check: Summary over all files. Unapproved: 0, unknown: 0, generated: 0, approved: 429 licenses.
[INFO]
[INFO] --- build-helper-maven-plugin:3.1.0:parse-version (parse-version) @ commons-lang3 ---
[INFO]
[INFO] --- maven-antrun-plugin:1.8:run (javadoc.resources) @ commons-lang3 ---
[INFO] Executing tasks

main:
[INFO] Executed tasks
[INFO]
[INFO] --- maven-remote-resources-plugin:1.7.0:process (process-resource-bundles) @ commons-lang3 ---
[INFO] Skipping remote resources execution.
[INFO]
[INFO] --- buildnumber-maven-plugin:1.4:create (default) @ commons-lang3 ---
[INFO] Executing: cmd.exe /X /C "git rev-parse --verify HEAD"
[INFO] Working directory: D:\workspace\commons-lang
[INFO] Storing buildNumber: 8bdc0147b17432607900eae83d36ec15f715f203 at timestamp:<PHONE_NUMBER>461
[INFO] Storing buildScmBranch: refine_normalizeSpace
[INFO]
[INFO] --- maven-resources-plugin:3.1.0:resources (default-resources) @ commons-lang3 ---
[INFO] Using 'ISO-8859-1' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory D:\workspace\commons-lang\src\main\resources
[INFO] Copying 2 resources to META-INF
[INFO]
[INFO] --- maven-compiler-plugin:3.8.1:compile (default-compile) @ commons-lang3 ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 210 source files to D:\workspace\commons-lang\target\classes
[INFO] /D:/workspace/commons-lang/src/main/java/org/apache/commons/lang3/builder/ToStringStyle.java: Some input files use or ove
rride a deprecated API.
[INFO] /D:/workspace/commons-lang/src/main/java/org/apache/commons/lang3/builder/ToStringStyle.java: Recompile with -Xlint:depre
cation for details.
[INFO] /D:/workspace/commons-lang/src/main/java/org/apache/commons/lang3/function/FailablePredicate.java: Some input files use u
nchecked or unsafe operations.
[INFO] /D:/workspace/commons-lang/src/main/java/org/apache/commons/lang3/function/FailablePredicate.java: Recompile with -Xlint:
unchecked for details.
[INFO]
[INFO] --- maven-bundle-plugin:4.2.1:manifest (bundle-manifest) @ commons-lang3 ---
[INFO]
[INFO] --- animal-sniffer-maven-plugin:1.18:check (checkAPIcompatibility) @ commons-lang3 ---
[INFO] Checking unresolved references to org.codehaus.mojo.signature:java18:1.0
[INFO]
[INFO] --- maven-resources-plugin:3.1.0:testResources (default-testResources) @ commons-lang3 ---
[INFO] Using 'ISO-8859-1' encoding to copy filtered resources.
[INFO] Copying 2 resources
[INFO] Copying 2 resources to META-INF
[INFO]
[INFO] --- maven-compiler-plugin:3.8.1:testCompile (default-testCompile) @ commons-lang3 ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 183 source files to D:\workspace\commons-lang\target\test-classes
[INFO] /D:/workspace/commons-lang/src/test/java/org/apache/commons/lang3/StringEscapeUtilsTest.java: Some input files use or ove
rride a deprecated API.
[INFO] /D:/workspace/commons-lang/src/test/java/org/apache/commons/lang3/StringEscapeUtilsTest.java: Recompile with -Xlint:depre
cation for details.
[INFO] /D:/workspace/commons-lang/src/test/java/org/apache/commons/lang3/builder/ReflectionDiffBuilderTest.java: Some input file
s use unchecked or unsafe operations.
[INFO] /D:/workspace/commons-lang/src/test/java/org/apache/commons/lang3/builder/ReflectionDiffBuilderTest.java: Recompile with
-Xlint:unchecked for details.
[INFO]
[INFO] --- jacoco-maven-plugin:0.8.5:prepare-agent (prepare-agent) @ commons-lang3 ---
[INFO] Skipping JaCoCo execution because property jacoco.skip is set.
[INFO]
[INFO] --- maven-surefire-plugin:3.0.0-M5:test (default-test) @ commons-lang3 ---
[INFO] Tests are skipped.
 Total time: 00:25:07

REMEMBER: The numbers below are just data. To gain reusable insights, you need to follow up on
why the numbers are the way they are. Use profilers (see -prof, -lprof), design factorial
experiments, perform baseline and negative tests that provide experimental control, make sure
the benchmarking environment is safe on JVM/OS/HW level, ask for reviews from the domain experts.
Do not assume the numbers tell you what you want them to tell.

Benchmark                               Mode  Cnt     Score    Error  Units
StringUtilsNormalizeSpaceTest.testNew   avgt   25  1810.545 ?34.015  ns/op

StringUtilsNormalizeSpaceTest.testNew2  avgt   25   484.364 ?13.339  ns/op

StringUtilsNormalizeSpaceTest.testOld   avgt   25   631.630 ?25.964  ns/op

Benchmark result is saved to target/jmh-result.org.apache.json
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  25:38 min
[INFO] Finished at: 2020-06-29T20:19:28+08:00
[INFO] ------------------------------------------------------------------------

Coverage decreased (-0.005%) to 94.674% when pulling b60d309999bbb3a4e3db5726da2ae8eeeb566a0a on XenoAmess:refine_normalizeSpace into da0f6a8051cc60d1bebe703b4e390ce3035497db on apache:master.

refined the tests.
full jmh test at: https://pastebin.ubuntu.com/p/FhWRhHx6v6/
in short:
Benchmark                               Mode  Cnt     Score     Error  Units
StringUtilsNormalizeSpaceTest.testNew   avgt    5  1732.831 ?127.293  ns/op

StringUtilsNormalizeSpaceTest.testNew2  avgt    5   437.049 ? 29.495  ns/op

StringUtilsNormalizeSpaceTest.testOld   avgt    5   574.871 ? 46.116  ns/op

indicates:
New, New2, Old is 3 version of a same function.
we of course should choose the fastest one, means New2 here.

@garydgregory rebased. please find some time to review. thanks.


Some translation fixes

Changes
Introduced Italian translations of core parts of the astro blog template.
Docs
/

great! Merged


How to remove the FileLAyer control on map?

I tried to remove the created fileLayer control by using fileLayer.removeFrom(map), but it doesn't work! Any idea?

You are right, it looks like the remove function were not implemented :)
Feel free to open a pull-request to add those! It should not be extremely complex, but don't hesitate to submit a draft so that we can help you

Hi @mahmoudmy ! Thank you for the feedback.
I'm cleaning and improving a bit this plugin. Is this issue still true ?
I've just conducted some tests and in my case, it seems working just fine.
    ...
    var control = L.Control.fileLayerLoad(options).addTo(map);
    setTimeout(function () {
        control.removeFrom(map);
    }, 2000);

Could you please confirm ?

Hi @SBats , Thanks.
Yes, it's OK!

Ok great ! Thanks for answering !


Fix typo: AdmissionTLs -> AdmissionTLS

Cases don't make sense that way ;) ... oversaw that one until I used it in a project

But then it should at least bei AdmissionTls, nicht AdmissionTLs
Making TLS all capital is probably something I got from Go.
Either way is fine, but not TLs

Am 20.05.2021 um 14:05 schrieb Kevin Flansburg @.***>:

While I agree that it’s not correct English, the partially lowercase acronym is idiomatic Rust. If you run clippy on your branch, it should complain about the all uppercase acronym.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

@kesselborn im not sure where you see it as AdmissionTLs.If you look at the code more closely the L is lowercases to follow Rust’s convention of AdmissionTls. Please re-review more closely :)
From https://doc.rust-lang.org/1.0.0/style/style/naming/README.html:

In CamelCase, acronyms count as one word: use Uuid rather than UUID. In snake_case, acronyms are lower-cased: is_xid_start.

This follows Rust’s convention, so the current casing is correct. Because of that I’m going ahead and closing this as the current styling confirms to standard Rust style convention.

Uhm 🙈,
super sorry: I should sleep more 😀
I was 100% convinced that it was wrong but looking a the change proves me kind of wrong 😃
Sorry for the noise, don’t know what was wrong with me this morning.
Cheers,
Daniel

Am 20.05.2021 um 15:43 schrieb Matthew Fisher @.***>:

@kesselborn im not sure where you see it as AdmissionTLs. If you look at the code more closely the L is lowercased to follow Rust’s convention of AdmissionTls. Please re-review more closely :)
From https://doc.rust-lang.org/1.0.0/style/style/naming/README.html:
In CamelCase, acronyms count as one word: use Uuid rather than UUID. In snake_case, acronyms are lower-cased: is_xid_start.
This follows Rust’s convention, so the current casing is correct. Because of that I’m going ahead and closing this as the current styling confirms to standard Rust style convention.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Update rubocop & haml_lint and fix offenses

Coverage remained the same at 99.367% when pulling 08c544008c9a6c1088e3d537647af87465aa8423 on update-rubocop into e58d9ec8b4cfc6be78fe4b55002d45b35a66ea20 on master.


change type to hide implementation

as discussed in #96
(i couldn't change the constructor - i was confused / wrong about what was possible).

Changing the constructor isn't that bad.  I does require one or two internal constructors in order to work (I just tried it--ask me if you want details).
I'd like to get @lindahua's comments before merging this.  The benefit of the current implementation (if any) is that you aren't required to use a Deque as a backing for Queues or Stacks, as long as whatever you use implements isempty, length, front, back, push!, pop! and shift!.
It should be possible to support both your desire to hide this, and the ability to change Deque to something else, but it might not be worth the trouble.

cleaning out old push requests i am no longer interested in.


Tasker

I just tried the "Tasker" in the new update, but for some reason it keeps counting every kill as 2. Am i doing something wrong, or is it just a little bug?

Interesting.  Did you have a prey bonus on the creature you were hunting?


Tearsheet disappers after executing in Terminal

I have successfully installed qstrader by following the instructions from the Readme. But unfortunately, whenever I try to execute 'python buy_and_hold_backtest.py'
The file executes, I can see the Tearsheet but it disappears within a second and I don't know where it gets saved. Hence, if anyone can help me out regarding the same, it would be great. Thanks

Hi nyk96,
In the commit on 2017-04-13 a change was made to the qstrader/statistics/tearsheet.py modele which changed:
plt.show()

to:
plt.show(block=False)

This has the effect of showing the tearsheet briefly but immediately closing the plot window.  This is a useful behaviour if you are running multiple backtests from a script such as when you want to optimize parameters.
To get the plot.show() function to block instead so you can see the tearsheet just change /home/user/venv/qstraderp3/lib/python3.5/site-packages/qstrader/statistics/tearsheet.py around line 637 back to “plt.show()” or “plt.show(block=True)”.  When you exit the plot viewer, the blocked backtest will continue and complete.
The path above is where the tearship.py module is installed by convention on a Linux/virtualenv install.  With a Windows/Anaconda or OS.X/Anaconda there will be a slightly different path but you can probably work out where the tearsheet.py file is with a search.
I expect there will eventually be a better way to override this behaviour but its a little beyond my skills to make the required changes.
Cheers,
Dieter.
From: nyk96
Sent: Monday, July 3, 2017 3:53 AM
To: mhallsmoore/qstrader
Cc: Subscribed
Subject: [mhallsmoore/qstrader] Tearsheet disappers after executing in Terminal (#215)
I have successfully installed qstrader by following the instructions from the Readme. But unfortunately, whenever I try to execute 'python buy_and_hold_backtest.py'
The file executes, I can see the Tearsheet but it disappears within a second and I don't know where it gets saved. Hence, if anyone can help me out regarding the same, it would be great. Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

How about to save result figure (tearsheet) to png/jpg file in out directory?


(test) UT: Add unit test for getting k8s version

update .gitignore
add unit test for ketting k8s version
Signed-off-by: Shovan Maity<EMAIL_ADDRESS>

Thanks for taking part in Hacktoberfest 2019. Please fill up this form to claim your swag https://hubs.ly/H0l5ldm0


Showing response body on panic?

Hey there,
Thanks for the great library. Really enjoying it!
One small pain point has been TestRequest::expect_success. I like the convenience of this function, but the problem is that when it fails, you get a very generic message like this:
Expect status code within 2xx range, got 422 (Unprocessable Entity), for request PUT http://localhost/api/whatever

Whenever I see this, I always end up modifying my test to use assert_text(""), which will then show me something like:
assertion failed: `(left == right)`

Diff < left / right > :
>{"metadata":{"status":"Failure","reason":"Sorry, you violated a unique constraint!","timestamp":{"secs_since_epoch":1729860582,"nanos_since_epoch":389126565}},"body":null}

Is there any way to combine these two pieces of functionality? Ideally, whenever expect_success fails, I'd like to just see the response body (as text) printed to the console along with the response status.
Thanks!

Hey @jkaye2012 , that is a great idea and I have ran into similar issues myself.
I'm working on a PR right now and should have it out over the next few days that will print the output within these assertions.

Awesome  - let me know if I can help in any way!

Hey @jkaye2012 , you should now find that change released as version 16.3.0. Given the size of the change I've gone with a full bump just incase it's a surprise for people.
Please try it out and let me know if that works well for you. I've added pretty printing for Json responses which should help.
Enjoy!

Looks great, thank you!
On Fri, Oct 25, 2024 at 5:00 PM Joseph Lenton @.***>
wrote:

Hey @jkaye2012 https://github.com/jkaye2012 , you should now find that
change released as version 16.3.0
https://crates.io/crates/axum-test/16.3.0. Given the size of the change
I've gone with a full bump just incase it's a surprise for people.
Please try it out and let me know if that works well for you. I've added
pretty printing for Json responses which should help.
Enjoy!
—
Reply to this email directly, view it on GitHub
https://github.com/JosephLenton/axum-test/issues/123#issuecomment-2439014585,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7FL7GFMALPHYI6KGFNVRDZ5LER3AVCNFSM6AAAAABQTHOMNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZZGAYTINJYGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

No worries. Let me know if you have any more ideas!


Run selection

Harlequin with latest duckdb on fedora 38.
Is it possible to run a selection from an sql scirpt file rather than the whole file?
Apologies if I am missing somehting very obvious.

Here's how I'd like to do this:
when the selection changes, have duckdb parse the selection (see #131 ). If it's a valid query, change the "Run Query" button to say "Run Selection" and on click or ctrl+enter, run the selection instead. Shouldn't be hard to manage the state with a few reactive vars.

#34 is kind of related -- with that we'd only execute the query at the cursor, and not execute other queries (separated by semicolons) in the editor.

This has been released in v0.0.23


Appstudio update test-component-pac-ngua

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-ngua-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
20 seconds
clone-repository

✅ Succeeded
1 minute
sast-snyk-check

✅ Succeeded
33 seconds
build-container

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
29 seconds
inspect-image

✅ Succeeded
30 seconds
deprecated-base-image-check

✅ Succeeded
46 seconds
clamav-scan

✅ Succeeded
9 seconds
sbom-json-check

✅ Succeeded
8 seconds
show-sbom

✅ Succeeded
8 seconds
show-summary

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-component-pac-ngua-on-pull-request-
creating pipelinerun test-component-pac-ngua-on-pull-request- in namespace build-e2e-iivx-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI/test-component-pac-ngua-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
24 seconds
clone-repository

✅ Succeeded
41 seconds
build-container

✅ Succeeded
25 seconds
sast-snyk-check

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
29 seconds
inspect-image

✅ Succeeded
28 seconds
deprecated-base-image-check

✅ Succeeded
8 seconds
clair-scan

✅ Succeeded
35 seconds
clamav-scan

✅ Succeeded
6 seconds
show-sbom

✅ Succeeded
6 seconds
show-summary


As an end user, I want the private cryptographic keys my browser creates to be managed in web cryptography

Background
Currently, XSS can steal delegated keys that are have full authorization for user actions.
Notes:

Currently a master key (in cold storage) is used to sign delegation statements for other keys. The other keys all have full authority to represent the user.
It is very dangerous if XSS can steal a user’s delegated key because that gives them same authority as user until the user deauthorizes the key.

Exploitability:

90% chance of canister made by DFINITY will be exploited using XSS over 12 months, and therefore complete and persistent compromise of user’s authority to access canisters. (Until user deauthorizes delegated keys)
almost 100% chance of a canister made by a third party will be exploited via XSS over 12 months.

Impact magnitude:

90% chance of >$1e7 total damage to our mission. Network will be unusable for anything of importance.

Recommendations:

Use web crypto to store keys. This will protect against XSS being able to steal a user’s delegated keys.

Post-review: If we use properly scoped delegations (which did not exist at the time of the review), the risk is significantly lower.
Story
In addition to plain Ed25519 and web authentication, add a web cryptography-based key management to the Javascript agent.
See SECFIND-11

Handled under https://agent-js.icp.xyz/identity/classes/ECDSAKeyIdentity.html


Search Page | Links | don't expand

Is it possible to not have each video within the video-search page results be full-wide clickable links?
e.g.
http://s18.postimg.org/693d40pbt/Untitled_1.png
This is becuase it's so hard to copy/paste text and you'll most likely end up clicking the video instead. So by disabling the full-wide clickable links it'll be easier to highlight text.

That's being forced by YouTube in the Search results and My Subscriptions pages, not by YTC. A simple way to disable this behavior is by removing a specific trigger class from the elements themselves, in this case it's being activated by the class .yt-uix-tile.
This simple function - or other with the same result - will do what you asked for, in case YePpHa decides to add your feature request:
while(document.querySelector('.yt-uix-tile')) {
    document.querySelector('.yt-uix-tile').classList.remove('yt-uix-tile')
}

Also a more clear title would be very helpful for future reference and for searching if others have the same or similar request, so whenever you can do change it to something more clear like Make search results not fully clickable.

There is a setting in YouTube Center that can modify the width of the search results. You can find it in YouTube Center Settings -> UI -> Search and enable row width limit. You will probably have to tweak the setting a little bit.
However, this will only limit the width and will not do the same as removing the .yt-uix-tile.

@YePpHa [SOLVED]

YouTube Center's UI option to modify row width limit fixed the issue.


IndexError when indexing Sequence of Array2D with None values

Describe the bug
Having None values in a Sequence of ArrayND fails.
Steps to reproduce the bug
from datasets import Array2D, Dataset, Features, Sequence

data = [
    [
        [[0]],
        None,
        None,
    ]
]
feature = Sequence(Array2D((1, 1), dtype="int64"))
dataset = Dataset.from_dict({"a": data}, features=Features({"a": feature}))

dataset[0]  # error raised only when indexing

Traceback (most recent call last):
  File "/Users/quentingallouedec/gia/c.py", line 13, in <module>
    dataset[0]  # error raised only when indexing
  File "/Users/quentingallouedec/gia/env/lib/python3.10/site-packages/datasets/arrow_dataset.py", line 2658, in __getitem__
    return self._getitem(key)
  File "/Users/quentingallouedec/gia/env/lib/python3.10/site-packages/datasets/arrow_dataset.py", line 2643, in _getitem
    formatted_output = format_table(
  File "/Users/quentingallouedec/gia/env/lib/python3.10/site-packages/datasets/formatting/formatting.py", line 634, in format_table
    return formatter(pa_table, query_type=query_type)
  File "/Users/quentingallouedec/gia/env/lib/python3.10/site-packages/datasets/formatting/formatting.py", line 406, in __call__
    return self.format_row(pa_table)
  File "/Users/quentingallouedec/gia/env/lib/python3.10/site-packages/datasets/formatting/formatting.py", line 441, in format_row
    row = self.python_arrow_extractor().extract_row(pa_table)
  File "/Users/quentingallouedec/gia/env/lib/python3.10/site-packages/datasets/formatting/formatting.py", line 144, in extract_row
    return _unnest(pa_table.to_pydict())
  File "pyarrow/table.pxi", line 4146, in pyarrow.lib.Table.to_pydict
  File "pyarrow/table.pxi", line 1312, in pyarrow.lib.ChunkedArray.to_pylist
  File "pyarrow/array.pxi", line 1521, in pyarrow.lib.Array.to_pylist
  File "pyarrow/scalar.pxi", line 675, in pyarrow.lib.ListScalar.as_py
  File "/Users/quentingallouedec/gia/env/lib/python3.10/site-packages/datasets/features/features.py", line 760, in to_pylist
    return self.to_numpy(zero_copy_only=zero_copy_only).tolist()
  File "/Users/quentingallouedec/gia/env/lib/python3.10/site-packages/datasets/features/features.py", line 725, in to_numpy
    numpy_arr = np.insert(numpy_arr.astype(np.float64), null_indices, np.nan, axis=0)
  File "<__array_function__ internals>", line 200, in insert
  File "/Users/quentingallouedec/gia/env/lib/python3.10/site-packages/numpy/lib/function_base.py", line 5426, in insert
    old_mask[indices] = False
IndexError: index 3 is out of bounds for axis 0 with size 3

AFAIK, the problem only occurs when you use a Sequence of ArrayND.
I strongly suspect that the problem comes from this line, or np.insert is misused:
https://github.com/huggingface/datasets/blob/02ee418831aba68d0be93227bce8b3f42ef8980f/src/datasets/features/features.py#L729
To put t simply, you want something that do that:
import numpy as np
numpy_arr = np.zeros((1, 1, 1))
null_indices = np.array([1, 2])
np.insert(numpy_arr, null_indices, np.nan, axis=0)
# raise an error, instead of outputting
# array([[[ 0.]],
#        [[nan]],
#        [[nan]]])

Expected behavior
The previous code should not raise an error.
Environment info

Python 3.10.11
datasets 2.10.0
pyarrow 12.0.0

Easy fix would be to add:
null_indices -= np.arange(len(null_indices))

before L279, but I'm not sure it's the most intuitive way to fix it.

Same issue here:
https://github.com/qgallouedec/datasets/blob/7fcbe5b1575c8d162b65b9397b3dfda995a4e048/src/datasets/features/features.py#L1398
Fixed in #5948


consider adding GH Action to auto-add PRs and issues to our PACTA board

based on https://github.com/RMI-PACTA/pacta.portfolio.allocate/blob/main/.github/workflows/add-prs-and-issues-to-project.yml

closed by #81


Random oppai failures

Some calls to oppai fail with "invalid .osu file", this map in particular.
The same command from a regular shell works though, so it might be something to do with subprocess.

Seems that the response text contains some random junk, here's the first line of a file downloaded with requests from Python:
ï»¿osu file format v14

And with curl:
osu file format v14


checkpoint conversion to tf2.0 question

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
ubuntu19.10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
source
TensorFlow version (use command below):
tf2.1.0
Python version:
python 3.7.4
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
GCC 7.3.0
CUDA/cuDNN version:
10.1/7.6.5
GPU model and memory:
RTX2060S 8GB

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
Hi,
This is just a general question
I have a model which is written in tf1.x and the model is like this:
https://github.com/CR-Ko/MegaDepth_Tensorflow/blob/master/hourglass_mega_tf_resize_bilinear_tflayer_prepost.py
It has checkpoint as well in tf1.x style. However, I have to rewrite it in TF2.1. For the rewrited 2.1 model, can I still use the TF1.X checkpoint? Or do I need use converter for the checkpoint?
https://github.com/tensorflow/estimator/blob/master/tensorflow_estimator/python/estimator/tools/checkpoint_converter.py
Also, about naming layers in rewrited 2.1 model, how should I do it to make sure correspondance between pretrain TF1.X checkpoint and rewrited TF2.1 model?

There's a mention at the end of tf.train.Checkpoint.restore's docstring: https://www.tensorflow.org/api_docs/python/tf/train/Checkpoint#restore
Checkpoints that came from Saver are matched using names, so you need to make sure Variable.name matches when loading one of those. It can be tricky to do, so the recommendation is to do it once and then save with Checkpoint.save so you get object-based matching in the future.
For more see the guide: https://www.tensorflow.org/guide/checkpoint


Download Again / Install Manually when run with the Kiosk cordova plugin

Hello Vaenow and team. Let me start: this plugin is awesome. Works like a charm.
I can just see 1 issue when run as well with the kiosk plugin.
and this case and only this case, I get a Download Again / Install Manually popup that is stuck on the screen.
From similar issues, I've seen that a way to fix this was to add
uses-permission android:name="android.permission.REQUEST_INSTALL_PACKAGES"  in the AndroidManifest.xml
But I've checked, and yes, I have already this line...
uses-permission
android:name="android.permission.READ_EXTERNAL_STORAGE"
uses-permission
android:name="android.permission.WRITE_EXTERNAL_STORAGE"
uses-permission
android:name="android.permission.REQUEST_INSTALL_PACKAGES"
uses-permission
android:name="android.permission.MOUNT_UNMOUNT_FILESYSTEMS"
uses-permission
android:name="android.permission.REORDER_TASKS"
uses-permission
android:name="android.permission.SYSTEM_ALERT_WINDOW"
Omitted the tags since interpreted by the editor.
My Tablet runs android 6.0.1
Thanks for help, I really need both plugin to work together. Thanks a lot in advance, Cheers, Denys

Hi @sealover06 I also use both those plugins in an app, on Android 6.0.
I have seen that it is necessary to disable temporarily the kiosk mode to let the  plugin-app-update do its work (stop, update, restart). The risk is that the user could be asked to confirm again that the app must be in kiosk mode (primary launch application).
Another way could be to force stopping the app, so the update can proceed.
It happend to me that the callback event after app-update does not work, so I added a timeout to force stopping the app  one minute after finding out that the app needs update.

Thank you so much @carlo318, let me test this asap.  Stay tuned !
From: carlo318<EMAIL_ADDRESS>Sent: samedi 20 juillet 2019 19:16
To: vaenow/cordova-plugin-app-update<EMAIL_ADDRESS>Cc: sealover06<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [vaenow/cordova-plugin-app-update] Download Again / Install Manually when run with the Kiosk cordova plugin (#136)
Hi @sealover06 https://github.com/sealover06  I also use both those plugins in an app, on Android 6.0.
I have seen that it is necessary to disable temporarily the kiosk mode to let the plugin-app-update do its work (stop, update, restart). The risk is that the user could be asked to confirm again that the app must be in kiosk mode (primary launch application).
Another way could be to force stopping the app, so the update can proceed.
It happend to me that the callback event after app-update does not work, so I added a timeout to force stopping the app one minute after finding out that the app needs update.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/vaenow/cordova-plugin-app-update/issues/136?email_source=notifications&email_token=ALKVYJRQWU5THRVPRMIDQFLQANB4NA5CNFSM4IFJDJM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2NSMKQ#issuecomment-513484330 , or mute the thread https://github.com/notifications/unsubscribe-auth/ALKVYJWGVPUZ573QE3XEFHDQANB4NANCNFSM4IFJDJMQ .  https://github.com/notifications/beacon/ALKVYJQK2ROZO2NLN62GX7DQANB4NA5CNFSM4IFJDJM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2NSMKQ.gif

Well, I'm not sure that I can deal with cordova-plugin-kiosk.  the KioskPlugin.exitKiosk(); function leads to a popup "Select destination Launcher3 / myApp"  I'm looking for full automation here with no user click...
@vaenow, please any thoughts on the compatibility of your great app-update plugin with a kiosk plugin ?  thank you so much in advance.


net: TCP2: connect() returns 0 without waiting for handshake completion

Describe the bug
When using CONFIG_NET_TCP2=y:
A call to BSD connect() or zsock_connect() returns 0 immediately after sending SYN, without waiting for TCP handshake completion. Specifically it does not wait for [SYN,ACK] reply from server.
That leads to client assuming connection is established. Immediately sending data after connect(), results with an error -128 (ENOTCONN).
To Reproduce
The bug can be reproduced on MQTT sample at path: samples/net/mqtt_publisher.
The sample uses mqtt_connect(), which calls connect()  followed immediately by writing to the socket (send()).
This works with CONFIG_NET_TCP1=y, but fails with CONFIG_NET_TCP2=y.
The bug was reproduced on the following boards:

stm32f746g_disco (using the on board ethernet)
nucleo_l432kc (using SLIP over usart1)

Expected behavior
connect() should return with 0 only after completing the TCP handshake, i.e. receiving [SYN,ACK].
Impact
This is a showstopper. You should be able to send immediately after connect() returns successfully.
Environment (please complete the following information):

OS: Linux, Ubuntu 18.04
Toolchain: Zephyr SDK

Additional remark
connect() succeeds (returns 0) even when there is no listening server.

Check with this PR https://github.com/zephyrproject-rtos/zephyr/pull/26098

Check with this PR #26098

Great! This is what we need.

Great! This is what we need.

Can you confirm it fixes this issue?

Great! This is what we need.

Can you confirm it fixes this issue?

Yes, it does.


Enhancement: Implement Profile Photo Management Functionality

As a user, I want to be able to manage my profile photo easily within the application. This includes being able to upload a new photo, delete my current photo if I want to change it, and update it with a new one.

For now I will write services that uses the local storage and not azure cloud


Fixed multimappers bug in bamToFastq

When bamToFastq encountered multi-mapping reads it would throw many misleading and spurious warnings indicating that a paired-end read was not adjacent to it's mate as well as including the read in the resulting fastq every time a second mate read in encountered. This is a bug in the bamToFastq code as mentioned in issues #312 and #109 .
This issue is easily solved since the bam file is already required to be namesorted, all of the reads are adjacent in the file. Thus the file must be read until the next read name is encountered or the file ends, saving only the most recent first and second mate records.

Thank you!


Beta Roadmap

add distinct track per url
choose song
add ability to pass to multiple people at once?
add record capability
Add upload capability
Add Social Media

Followup with slam:

On the "Are you ready to sing?" page, indicate the song title more explicitly. "Song: Amazing Grace" and perhaps in larger font.
For the countdown before you start singing, it isn't clear when the music will start. Could there be more visual cue leading up to the moment you hear the first sound? Like a visual pulse? The countdown numbers change slowly so it's hard to feel the beat/count.
Will you assume people know what the pitch of the starting note is? (i.e. what note they should start singing?)
The soundcheck page is extremely sensitive. Even if I do not say anything or try not to make any noise, it appears to detect my mic. Should it be less sensitive?

add ios support

lower sensitivity on mic to 0.3 maybe

adjust the lag adjuster to be more clear

help

privacy

mediarecorder polyfill

in case can't hear, mention device name connected

gracefully handle the person not singing (can't hear them singing)

FLAC encoder

handle internet outage

safari 13.1 does not support https://caniuse.com/#search=copytochannel. report it to mdn


Dev #621 -  Migrate dataset/tab matching markers from hardcoded into database

Migrate dataset/tab matching markers from hardcoded into database
The current version of the F&E tool is hardcoded to only consider a fixed subset of NPD datasets.
The real scope of the tool is wider, and the admin users should have the ability to import datasets that are not currently included in the fixed subset of imported NPD datasets.
To allow that, we need to remove the hardcoded logic and make the system more flexible.
Testing
There is nothing new on the frontend side, this is just a reworking of the backend.
The only possible test is to re-check that the NPD Data Tables import keeps working as it did before.

Thanks @cacquani, happy with this.


Sample Response  json has a difference

🛑 IMPORTANT: You can get your feedback addressed faster if you use the comment section for the article in which you encountered a problem.
Link to article:
https://docs.microsoft.com/ja-jp/rest/api/appservice/certificates/list
Problem:
The json of the Sample Response of this article is different.
Specifically, there is a value [] immediately under json of the response,
There is no value [] in the actual response.
Attach the capture of the response.

Whether the current state is a phenomenon peculiar to my environment,
Is it an overall bug occurring,
Would you please let me know the situation such as the article is wrong?
Thank you.

@dend Can you take a look at this feedback?

Assigning to @cephalin for follow-up.

Since List of other RestAPI has value []
Would you modify the implementation of RestAPI?
Your answer thank you.

This is Microsoft Azure support Japan. Since it will be a question from the customer, thank you for your answer.

@panchagnula @yiliaomsft Can you please help here? We have no control over this auto-generated doc

Navy, I believe there was some change in the API recently where the JSON result does not return value anymore. Can you help with this please?
https://docs.microsoft.com/ja-jp/rest/api/appservice/certificates/list
Thanks
Sisira
[https://docs.microsoft.com/en-us/media/logos/logo-ms-social.png]https://docs.microsoft.com/ja-jp/rest/api/appservice/certificates/list
Certificates - List (Azure App Service)https://docs.microsoft.com/ja-jp/rest/api/appservice/certificates/list
docs.microsoft.com
Get all certificates for a subscription.
Sent from Outlookhttp://aka.ms/weboutlook

From: Cephas Lin<EMAIL_ADDRESS>Sent: Monday, February 11, 2019 17:05
To: MicrosoftDocs/feedback
Cc: Sisira Panchagnula; Mention
Subject: Re: [MicrosoftDocs/feedback] Sample Response json has a difference (#897)
@panchagnulahttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpanchagnula&data=02|01|sisirap%40microsoft.com|18002f66d250459efe5808d690861f65|72f988bf86f141af91ab2d7cd011db47|1|0|636855303057469601&sdata=1lywV1PiX4FAJ3h0wRogM0Au%2FdCH3Y3gNJeV4K0n3uE%3D&reserved=0 @yiliaomsfthttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fyiliaomsft&data=02|01|sisirap%40microsoft.com|18002f66d250459efe5808d690861f65|72f988bf86f141af91ab2d7cd011db47|1|0|636855303057479609&sdata=xMIPHg97SRUcQ%2Bz%2FEzs2Wx09Swj2XbLIRKP4wuXxgKE%3D&reserved=0 Can you please help here? We have no control over this auto-generated doc
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Ffeedback%2Fissues%2F897%23issuecomment-462565447&data=02|01|sisirap%40microsoft.com|18002f66d250459efe5808d690861f65|72f988bf86f141af91ab2d7cd011db47|1|0|636855303057479609&sdata=7TCM7yL03RwBqMjXEiavWSDjU397y0riPJE1FewZEQg%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANMsDGzIrmhr3ExOM1NyZ2e54_DFkc6Sks5vMhM_gaJpZM4Y06Hg&data=02|01|sisirap%40microsoft.com|18002f66d250459efe5808d690861f65|72f988bf86f141af91ab2d7cd011db47|1|0|636855303057489613&sdata=kSqwlXPjzjC6LI4lBWdJZFVGARYCuYYDcmQu9mjOtF0%3D&reserved=0.

@SatoruTaniya what is the api-version the customer is using, the referenced article uses 2016-03-01

@panchagnula  Thank you for your response.　The API version customer use is 2018-02-01.In the verification I made, the same phenomenon occurs even with API version 2016-03-01

What customer want to know is whether to change the implementation of RestAPI or plan to change the document.
Thank you for your answer.

For example, when calling the following API, value exists as documented
https://docs.microsoft.com/ja-jp/rest/api/appservice/appserviceplans/list

Since customer already have this API used
Please notify something when changing the implementation of RestAPI.

talked to Navy, this is a bug that is fixed with the latest version ask the customer to use 2018-11-01, this is a REST API level that had to fixed with new API version only. Hope this helps, sorry this is a one-off case & we will make sure to document/communicate this better to the customers in the future.

@panchagnula Can you reference the bug or confirm if it's already fixed? That way we can close this issue.

Awesome thanks for getting to the bottom of this folks. @SatoruTaniya Sounds like this issue has been fixed. If you notice any more discrepencies, please don't hesitate to open another bug. Thank you for your help! #please-close

Hi @panchagnula
Same customer reports that the structure of the REST API's result changed to correct structure suddenly by using old api version 2018-02-01 since last May.
Could you please check if there is any update at that time?
Though the customer thinks that it's no problem to use latest API version, it's weird that the result/behavior changes suddenly.

@chibionos and/or @naveedaz can answer this question


Document Current Hardware/Code Base

What type of code license considerations might we need to follow for this project? See also: https://choosealicense.com/

Major documentation items can also be added via the wiki – https://github.com/andrewsng/assistive-technology/wiki


Java 8 required for selenium server

Latest selenium server complains that it requires Java 8 to start but default-jre package for ubuntu/trusty64 provides Java 7. Start script should be updated to install Java 8.

I managed to fix this here


Suggestion for Pypi information

It does an amazing job for Python point cloud analysis! You know that almost 90% of PCL python library depends on C++ PCL which is really really really difficult to install and hard to pack them in the product (GUI application etc.). Your library solved this problem, awesome!
I just missed this good library when I search in Pypi. I think it may be a good idea if you strengthen this is a pure python library without C++ dependency and PCL library in the Pypi introduction!
Thanks again for your great contribution! Hoping in the future, this library can include more useful funcions.

I'm sorry that I have to change the library back to C++ implementation because there are tooooo many existing functions to reimplement! And I definitely won't have that much time to spend on this. I hope the cython wrapping will offer a smooth experience over existing C++ codes.
In terms of PCL installation, it's easy to install both on Linux and Windows now. On linux you can just sudo apt install libpcl and on windows you can use vcpkg. If you have other installation requirements, I can offer python wheels.


doorbell pressed event not displayed

First off, sincere thank you for development here. Really cool integration and as I build more with it, I'll submit some pull requests.
In attempting to add a simple automation that triggers from the doorbell press, I cannot seem to find the doorbell_pressed event. Is there something I am missing here?

@treehousechampion doorbell_pressed is a binary_sensor entity of type OCCUPANCY. So in your case, you want to: Do something when Front Door Doorbell became occupied -> This translates to somebody just pressed the doorbell. The binary_sensor will remain true while the doorbell is ringing and become false when the doorbell is either answered or cancelled. Hope this helps

Thanks, super helpful. I am admittedly new to HA and still familiarizing with the YAML structures.


Integrate spam number database

Hi,
It would be great if the app can also optionally add spam number databases and block/silent numbers present there.
Eg database:
https://www.ftc.gov/policy-notices/open-government/data-sets/do-not-call-data

The other top Foss alternatives for spam blocking do use existing databases as well. I think finding one or more good databases and integrating them will make this a great feature to add.
https://f-droid.org/packages/us.spotco.carrion/
https://f-droid.org/packages/dummydomain.yetanothercallblocker/

There are a couple of reasons that I don't want to integrate databases.

They target different audiences:

Exclusive, for people who are more inclined to reject unknown calls, since they block all unknown numbers, a database would be redundant.
Inclusive, for people like salesmen who need to pick unknown calls to get new clients, they would prefer the database solution, and they don't need features in SpamBlocker.

In my opinion, SpamBlocker and database actually conflict.

Too many databases in different countries, their API may change from time to time, the maintenance would be an endless job.

There is now a feature "import blacklist from file", you can download the csv and import it manually, it'll be available in the next release.

Actually I think google should support chain of call screening, for example, when a call passes SpamBlocker, it's then checked again by some other blockers that based on database, a call would only pass when it passes all screening apps. So all different screening apps can work together with their functionality kept simple.

There is a plan now: https://github.com/aj3423/SpamBlocker/issues/140


Nullable reference types support

Feel free to apply changes from previous PR.

One more interesting point. R# automatically removes its own CanBeNull markup when adding "?" to a type. @NN---

I paid attention to that:)
As I said my concern is old ReSharper versions without good NRT support.

@andrewvk So, do we agree that all ReSharper nullability attributes will be replaced with .NET nullability attributes ?

@andrewvk So, do we agree that all ReSharper nullability attributes will be replaced with .NET nullability attributes ?

@andrewvk Can we merge what we have now ?

@andrewvk Can we merge what we have now ?

There is some issue with nullability and autogenerated code. By default, C# compiler treat all generated code as #nullable disabled, regardless of project settings.
In that case you should specify #nullable enabled explicitly. I fixed this in base T4 template, but did not rerun all templates.
So, after I did it we still have hundreds of nullability warnings and errors (see last commit).

There is some issue with nullability and autogenerated code. By default, C# compiler treat all generated code as #nullable disabled, regardless of project settings.
In that case you should specify #nullable enabled explicitly. I fixed this in base T4 template, but did not rerun all templates.
So, after I did it we still have hundreds of nullability warnings and errors (see last commit).

Ok. I can fix that too.
There is some problem with build time limit .
Is there any solution for this ?

Ok. I can fix that too.
There is some problem with build time limit .
Is there any solution for this ?

If there is one, I don't know it :(

If there is one, I don't know it :(

I think GitHub actions can handle this. Need to try.

I think GitHub actions can handle this. Need to try.

@andrewvk in my project I added NAssert.NotNull which tells compiler to make variable non nullable and avoid useless ! .
Do you think it worth adding it?

@andrewvk in my project I added NAssert.NotNull which tells compiler to make variable non nullable and avoid useless ! .
Do you think it worth adding it?

Can you explain in more detail what you mean by that? In any case, using ! in most cases is definitely not the best option.

Can you explain in more detail what you mean by that? In any case, using ! in most cases is definitely not the best option.

This assertion makes this code compiling
string? f();
void g(string) {}

string? a = f();
NAssert.NotNull(a);

// a here is not null
g(a); // OK

using System.Diagnostics.CodeAnalysis;
using NUnit.Framework;

namespace Tests
{
    public static class NAssert
    {
#pragma warning disable CS8777 // Parameter must have a non-null value when exiting.
        // The warning is due to fact we do not check for null and throw directly.

        #region NotNull

        /// <summary>
        /// Verifies that the object that is passed in is not equal to <see langword="null" />. Returns without throwing an
        /// exception when inside a multiple assert block.
        /// </summary>
        /// <param name="anObject">The object that is to be tested</param>
        /// <param name="message">The message to display in case of failure</param>
        /// <param name="args">Array of objects to be used in formatting the message</param>
        public static void NotNull([NotNull] object? anObject, string message, params object?[]? args)
        {
            Assert.NotNull(anObject, message, args);
        }

        /// <summary>
        /// Verifies that the object that is passed in is not equal to <see langword="null" />. Returns without throwing an
        /// exception when inside a multiple assert block.
        /// </summary>
        /// <param name="anObject">The object that is to be tested</param>
        public static void NotNull([NotNull] object? anObject)
        {
            Assert.NotNull(anObject);
        }

        #endregion

#pragma warning restore CS8777 // Parameter must have a non-null value when exiting.
    }
}

This assertion makes this code compiling
string? f();
void g(string) {}

string? a = f();
NAssert.NotNull(a);

// a here is not null
g(a); // OK

using System.Diagnostics.CodeAnalysis;
using NUnit.Framework;

namespace Tests
{
    public static class NAssert
    {
#pragma warning disable CS8777 // Parameter must have a non-null value when exiting.
        // The warning is due to fact we do not check for null and throw directly.

        #region NotNull

        /// <summary>
        /// Verifies that the object that is passed in is not equal to <see langword="null" />. Returns without throwing an
        /// exception when inside a multiple assert block.
        /// </summary>
        /// <param name="anObject">The object that is to be tested</param>
        /// <param name="message">The message to display in case of failure</param>
        /// <param name="args">Array of objects to be used in formatting the message</param>
        public static void NotNull([NotNull] object? anObject, string message, params object?[]? args)
        {
            Assert.NotNull(anObject, message, args);
        }

        /// <summary>
        /// Verifies that the object that is passed in is not equal to <see langword="null" />. Returns without throwing an
        /// exception when inside a multiple assert block.
        /// </summary>
        /// <param name="anObject">The object that is to be tested</param>
        public static void NotNull([NotNull] object? anObject)
        {
            Assert.NotNull(anObject);
        }

        #endregion

#pragma warning restore CS8777 // Parameter must have a non-null value when exiting.
    }
}

Btw, IDictionary doesn't require "TKey : notnull" according to sources.
https://github.com/dotnet/runtime/blob/master/src/libraries/System.Private.CoreLib/src/System/Collections/Generic/IDictionary.cs
It means adding this constraint is not correct.

Btw, IDictionary doesn't require "TKey : notnull" according to sources.
https://github.com/dotnet/runtime/blob/master/src/libraries/System.Private.CoreLib/src/System/Collections/Generic/IDictionary.cs
It means adding this constraint is not correct.

You mean that NUnit still doesn't have a NotNull attribute markup? Yeah, that's pretty weird. Of course, in this case it's worth adding your own variant.

You mean that NUnit still doesn't have a NotNull attribute markup? Yeah, that's pretty weird. Of course, in this case it's worth adding your own variant.

Yeah. I tried to reproduce this in a separate project but everything compiled.
Probably it requires this constraint only for specific framework.
Currently everything compiles but not in appveyor:(
I’ll check how to improve it.

Yeah. I tried to reproduce this in a separate project but everything compiled.
Probably it requires this constraint only for specific framework.
Currently everything compiles but not in appveyor:(
I’ll check how to improve it.

Core 3.0 library

Core 5.0 library

Shall we try to hide it under a compiler directive?

Core 3.0 library

Core 5.0 library

Shall we try to hide it under a compiler directive?

#if NETCOREAPP3_1 			where TKey : notnull #endif
Build OK

#if NETCOREAPP3_1 			where TKey : notnull #endif
Build OK

It is better to not introduce additional constraint if possble

It is better to not introduce additional constraint if possble

Fixed recursion and now the build works :)
@andrewvk Is there anything left in PR ?

Fixed recursion and now the build works :)
@andrewvk Is there anything left in PR ?


Revert "pegging ferryman version to 2.1.1"

Reverts blendededge/graphql-component-oih#11

The version of ferryman wasn't actually the issue. Reverting the previous pull request.


Deadlock/program stuck while calling function from parallelStream

Describe the bug
Program will stuck and never finish, if function called from parallelStream. There is no problem when function called from stream.
How To Reproduce
	public void Error() {
		List<String> s = new ArrayList<String>();
		for(int i = 0;i<4;i++) {
			s.add(Integer.toString(i));
		}

		s.parallelStream().forEach(k->{
			try {
				IntersectionUtils.dummy();
				System.out.println(k);
			} catch (TornadoExecutionPlanException e) {
				// TODO Auto-generated catch block
				//e.printStackTrace();
			}
		});
		System.out.println();
	}

Here is the class with required functions.
IntersectionUtils.zip
Computing system setup (please complete the following information):

OS:Windows 10
compute capability of 8.9, cuda 12.5
commit id 1.0.9-dev-3c0de66

Additional context
Add any other context about the problem here.

We pushed a fix: https://github.com/beehive-lab/TornadoVM/commit/0d4425264ffe0633ea79c8aba91233591059d3fd
You can check latest master branch. Please let us know if you encounter more issues.

Thanks I will try the new patch, I fixed this issue few days ago. The program works fine after I've changed function call for function body.


⚠ Bofrost.at has degraded performance

In be26e7e, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1032 ms

Resolved: Bofrost.at performance has improved in 467d9b5 after 305 days, 16 hours, 58 minutes.


Texture misalignment when using move tool and subdivisions

Not sure if I misunderstand how things should work, but the texture behaviour when using move tool+subdivs seems weird.
Texture moves along with the block. If i instead use the select face tool, followed by A to select all, I get the expected behaviour.
https://github.com/blackears/cyclopsLevelBuilder/assets/98854/21462001-bd00-4ed0-98a1-c716f7d80716
godot 4.2.1 or 4.3 dev with cyclops 1.0.2
Update:
Seems to be due to this block being commented out. Uncommenting makes things work. Not sure why it's disabled...
https://github.com/blackears/cyclopsLevelBuilder/blob/f3168e51008cf9fcf3d7ac9b62dd0336e63d3dd6/godot/addons/cyclops_level_builder/commands/cmd_transform_blocks.gd#L63

I think I disabled that when I was changing how commands were being called.  I've restored it now.


[T5A3][F10-B2]Ning Yu

@ningyuuu : Accepting this PR without a review because another of your PRs for this tutorial was reviewed in detail.


Enhance Support for Readonly Schemas

Hi, I have a question about the schema and readonly.
The action serves the exact purpose. The problem is the need
to write it for every property. I want to mention again that
I think it's great that this exists.
In the application I'm working on,
I want to ensure that data can only be modified by a dedicated process,
and all consumers receive a readonly dataset.
I also know from the modular approach that you have to be very
careful about simply adding things.
I tried to implement it using a DeepReadonly from typing,
but the schema didn't want to work with it, which wasn't unexpected.
However, this is basically what I'm trying to achieve.
I'm not really aiming to freeze objects but rather to improve the DX.
Additionally, I didn't find anything that valibot offers in this regard.
Now, to my key points :)
1: Is there already a solution that I'm just not finding?
2: Is there, or would it be an idea, to have a function in the generics
that can specify that only schemas of type readonly are allowed?
Example here:
const getReadonlySchema = <TSchema extends ReadonlyBaseSchema>(schema: TSchema){
  // do anything
}

Then the developer would still have to define it for each one,
but at least they would be blessed with the feedback.
3: Is there, or would it be an idea, to have a function similar
to the pick function that takes a schema and converts it into a readonly schema?
Example here:
import * as v from 'valibot';

const Schema = v.object({
  email: v.pipe(v.string(), v.email()),
  password: v.pipe(v.string(), v.minLength(8)),
});

// any prop is readonly now
const readonlySchema = v.readonly(Schema);

// or i think better
const readonlySchema = v.deepReadonly(Schema);

I included both examples because I believe readonly would
already be somewhat reserved by the action, and deepReadonly would
better match the term used in the TypeScript world.
4: Is there, or would it be an idea, to specify something
like this in the configuration of parse?

import * as v from 'valibot';

const Schema = v.object({
  email: v.pipe(v.string(), v.email()),
  password: v.pipe(v.string(), v.minLength(8)),
});

const result = v.safeParse(Schema, {
  email<EMAIL_ADDRESS>  password: '12345678',
}, {
  readonly: true
  // or
  deepReadonly: true
});

console.log(result);

Then the schema would generally remain open,
but the parse would produce readonly output. However,
I imagine that could be quite difficult to implement.
I hope I don't sound completely crazy, and that the comments make sense.
Sorry for the amount of text.

As a final note... I believe that one or the other mechanism would definitely be something that could help other libraries that want to use Valibot, especially when it comes to Flux solutions or other single point of truth implementations.

I'm more than ready to answer any questions and provide feedback.

We have to distinguish between the schema object (what a schema function returns) and the parsing output. Valibot's schema objects are already readonly by default. So there is nothing extra to do on your side. Parsing output is readonly if you use our readonly action in the pipe of a schema. See this API reference for more details and examples: https://valibot.dev/api/readonly/

I understood that so far. My question is rather whether it's
possible to define a schema without readonly and then make this
schema readonly afterwards, following the principle:
import * as v from 'valibot';

const Schema = v.object({
  entry: v.object({
    entry2: v.object({
      ...
    })
  })
});

const ReadonlySchema = deepReadonly(Schema);

The result would be that all entries and subentries would then
have readonly, making both the schema and the output marked
as readonly in TypeScript.
The current action does this only based on the element
specified by the developer. My question is whether a function
exists or could exist that does this as well, so that
libraries can automatically make an external schema readonly.
So instead of:
import * as v from 'valibot';

const Schema = v.pipe(v.object({
  entry: v.pipe(v.object({
    entry2: v.pipe(v.object({
      ...
    }), v.readonly())
  }), v.readonly())
}), v.readonly());

Or alternatively, that you can specify in the generic
that only readonly schemas are allowed.

const anyMethod = <TSchema extends ReadonlyBaseSchema>(schema: TSchema){
  ...
}

// error in typescript Schema is not of type ReadonlyBaseSchema
anyMethod(Schema)

// Works all is fine
anyMethod(ReadonlySchema)

This would inform the schema creator via TypeScript
to make everything readonly, otherwise it won't work.
I'll try again to illustrate roughly what I mean with
the two mentioned examples using a small sample:
Example 1 with deepReadonly:

// Method to create as example a Store with a Schema.
const store = <TSchema extends BaseSchema<unknown, unknown, BaseIssue<unknown>>>(schema: TSchema){
  const observers = [];
  return {
    dispatch: (value: InferOutput<TSchema>) => {
        // schema converts into readonly schema validation
        // value returns any given prop as readonly
        const result = safeParse(deepReadonly(schema), value);
        if(result.success){
          observers.forEach(observer => observer(result.output));
        }
    }
    observe: (observer) => {
      observers.push(observer);
    }
  }
}

const productStore = store(object({
  name: string(),
  price: object({
    currencyIso: string(),
    value: number()
  })
}))

productStore.observe((value) => {

  // typescript mark error for currencyIso as Readonly Props
  value.price.currencyIso = "Huhu"
});

By using the function, the incorrectly defined schema is
converted into a readonly, making the output readonly in TypeScript.
Example 2 with the type:

// Method to create as example a Store with a Schema.
// Name of extends Type is a Sample
const store = <TSchema extends ReadonlyBaseSchema<unknown, unknown, BaseIssue<unknown>>>(schema: TSchema){
  const observers = [];
  return {
    dispatch: (value: InferOutput<TSchema>) => {
        // schema converts into readonly schema validation
        // value returns any given prop as readonly
        const result = safeParse(schema, value);
        if(result.success){
          observers.forEach(observer => observer(result.output));
        }
    }
    observe: (observer) => {
      observers.push(observer);
    }
  }
}

// Typescript error while not all props are readonly
const productStore = store(object({
  name: string(),
  price: object({
    currencyIso: string(),
    value: pipe(number(), readonly())
  })
}))

productStore.observe((value) => {

  // typescript mark error for currencyIso as Readonly Props
  value.price.currencyIso = "Huhu"
});

I hope this explains my idea better.
The goal is to determine whether the schema is
fully readonly or to make it so if it isn't.
Currently, this is only possible by applying readonly()
to each schema or each property within the schema.

My question is rather whether it's possible to define a schema without readonly and then make this schema readonly afterwards

Yes, this works with pipe and readonly. But we don't have a deepReadonly action at the moment.
import * as v from 'valibot';

const ArraySchema = v.array(v.string());

const ReadonlyArraySchema = v.pipe(ArraySchema, v.readonly());

Or alternatively, that you can specify in the generic that only readonly schemas are allowed.

Unfortunately, I am not sure if this is technically possible with our current implementation, but you could recursively check at runtime if every schema contains a pipe with a readonly action and throw an error if not.

I would change your workaround to:
import { DeepReadonly } from "ts-essentials";
import * as v from 'valibot';

const Schema = v.object({ foo: v.string() });

const output = v.parse(Schema, ['a', 'b', 'c']) as DeepReadonly<
  v.InferOutput<typeof Schema>
>;

Yes, you're right. That's something one can live with much better :) Thank you very much.


publish nightly jar to apache nexus

Currently blocked by https://github.com/apache/incubator-pekko-grpc/pull/37#issuecomment-1448997163


Backport CMake Config installation to 0.17.x

https://github.com/intel/mkl-dnn/pull/358 introduced the CMakeConfig that is installed for consumption by CMake projects. This is very useful for consumers but is not included in any current release.
Can this be backported to a current release please? It is quite hard to use the library as the dependencies are not known (e.g. mkl, openmp, ...) when you only want to use this.

copied from related issue 408:

The CMake config in #358 is incomplete: It does not contain the required include paths in the install tree so programs using this fail to compile unless the includes happen to be in a standard location.
Furthermore it excludes all libraries that are required to link. This is especially fatal for static builds of MKL-DNN.
However this is a more general problem: As a user what libraries do I need to link against? The readme states that no linking is required for dynamic builds but nothing about static builds. But knowledge from the build process is required:

Do I need to use lib/libmklml_gnu.so, lib/libmklml_intel.so or neither?
Do I need to use lib/libiomp5.so? Getting this wrong results in undefined behaviour, but how would I know if I didn't build it myself?

We have 0.18rc targeted for release today or tomorrow. #358 is included. Will this suffice?

Unfortunately not due to #408. I opened a MR making it better. Any chance to get that included?

Well if the new code is released then close this. I wouldn't mix up the 2 issues though.

We have 0.18rc targeted for release today or tomorrow. #358 is included. Will this suffice?

So technically yes: It does include the config and it is usable IF mkldnn is installed into a system directory whose include subfolder is in the default include path, e.g. /usr/local.
However I strongly encourage to resolve #408 before the actual release or at least include #409

I think it makes sense to backport this to 0.17.x if the effort is low because 0.18 breaks some things and it may take a while until people adapt.

We decided to promote all the CMake related changes to v0.18 release and not backport to v0.17. Please let me know if you have any concerns with that.

The changes are now in rls-v0.18 branch and will be a part of the v0.18 release.


Recreate Storage class

Storage class parameters are inmutable, removing the SC to be re-created with the new or the same parameters.
That is the recommended practice, it does not affect the already provisioned volumes

The SC creation is not failing if already exists. The issue with SRIOV is not related to this change but happened because there cluster already had a CSV for SRIOV installed.


Document or update why 'engines' is limited to 'node: >18.0.0'

In testing of other modules, node 16 seems to work. Is there some RE that relies on post 16 features? If so, what? If not, reduce the engines.

Work for this issue will begin begin on branch work-liquid-labs/regex-repo/88.


Upgrade asciidoctorj-diagram to v2.2.1 to enable latest feature

Fixes https://github.com/confluence-publisher/confluence-publisher/issues/336

Thank you @jtama for your contribution!


Legend shows -inf when using lowest in UserDefined

When using the lowest parameter to create user defined classes, the plot shows -inf.
mapclassify: 2.5.0
geopandas: 0.12.2
# read in some polygons
data = gpd.read_file('https://www2.census.gov/geo/tiger/TIGER2018/STATE/tl_2018_us_state.zip')
data = data.loc[~data.NAME.isin(['Alaska','Hawaii','Puerto Rico','United States Virgin Islands','Commonwealth of the Northern Mariana Islands','Guam','American Samoa']),]

# Create some fake percentages without high or low values
data['mid_vals'] = np.random.uniform(0.3, 0.7, data.shape[0])

# Create user defined bins at 10% intervals
ten_pct_bins = mapclassify.UserDefined(data.mid_vals,
                                    [0.1, 0.2, 0.3, 0.4, 0.5, 0.6, 0.7, 0.8, 0.9, 1.0],
                                    lowest=0)

UserDefined

  Interval     Count
--------------------
[0.00, 0.10] |     0
(0.10, 0.20] |     0
(0.20, 0.30] |     0
(0.30, 0.40] |    15
(0.40, 0.50] |     5
(0.50, 0.60] |    13
(0.60, 0.70] |    16
(0.70, 0.80] |     0
(0.80, 0.90] |     0
(0.90, 1.00] |     0

# Plot the map
ten_pct_bins.plot(data, legend=True);

Multiple issues here actually :D.
That TypeError in @sjsrey's post is an issue here. We need to pass lowest to UserDefined here https://github.com/pysal/mapclassify/blob/34341a22db968b42bcc21b52d4bcf32a0a3e7098/mapclassify/_classify_API.py#L190-L191.
But the reported issue is maybe a different one because it does not use geopandas directly but via our MapClassifier.plot method, so it may be something there. In here https://github.com/pysal/mapclassify/blob/34341a22db968b42bcc21b52d4bcf32a0a3e7098/mapclassify/classifiers.py#L2452-L2461 we need to pass lowest in classification_kwds (for which we need the first issue fixed). Then it should work. Haven't tested it but since geopandas retrives the legend labels via MapClassifier.get_legend_classes, this should be completely fixable on our side here.

@sjsrey Was this issue completely closed with #177?


Migration steps for nextjs inbuilt routing

Are there any easy migration step to migrate to inbuilt nextjs routing

You can use the next js rewrites in your next.config.json for this.
// next.config.json

// Existing routes configured with next-routes

const routes = require("./your-next-routes")

// Map the routes to the below shape
// routes = [{ source: '/about', destination: '/' }]

module.exports = {
  async rewrites() {
    return [...routes]
  }
}

In your server.js file delete routes import and use app.getRequestHandler()
// server.js

const handle = app.getRequestHandler();

server.get("*", (req, res) => {
    return handle(req, res);
});

Might have to delete the .next folder before giving it a go.

Thank you so much for your answer, @asjadanis ! It worked for me with Next.js v9.5.5


Cart::search() must be an instance of Closure, array given

So what I understand you can't search with an array parameter like it said in the example?

No, the documentation is out of date. You will have to pass the search method a closure, just like the search method on a regular Collection

Yes, i found it. Thank you

just wondering how to pass the search method a closure?
as ive been stuck on the cart search method and it gives me an error "must be an instance of Closure, array given,"
any help is highly appreciated.


Fix typo in Variables concept

Summary
This fixes a typo in the Variables concept text.
Checklist

[x] If docs where changed, run ./bin/configlet generate to ensure all documents are properly generated.
[ ] CI is green

Thanks for the PR!
The concepts/variables/about.md file has similar text and can use a similar fix.


[Question] Understanding the name

Hi! This looks like a great project. I'm curious to know why the package is called playwright-controller. I'm new to the Fluent pattern, and I'm probably missing something about the role of "controller"

Hi @arjun27, thank you for your interest in this project.
I admit that the name of this project is questionable. If you think it could have a better name, please, propose another name.
This name came from TestCafe which exposes a fluent API.
In TestCafe,  a typical test looks like:
await t
 .navigateTo(url)
 .hover(selector)
 .click(selector)
 ... etc ...

The t object that gives access to all chainable actions and assertions is called the TestCafe controller.

Ah, thanks for the explanation. I was trying to figure out how "controller" fits in the fluent ecosystem and that explains it. However, it might be worth evaluating if "controller" is an implementation detail chosen by Testcafe. I was wondering if playwright-fluent might better convey the intent of the project.

playwright-fluent looks like a great name 👍 🥇 .
Thanks a lot for this suggestion.
Please make some noise on this thread to see if this name fits more than the actual one.
I propose you to wait some days before changing the name of the project and of the npm package in favor of your suggestion.

Hi @arjun27 , thank you for your feedback!


Completed version of vertex-gsoc-2024-assessment

This pull request encompasses a comprehensive enhancement of the entire project, including the addition of unit test for BackendVerticle.

Looking forward to receiving feedback on my performance regarding this task. Thank you

Thank you for reviewing my assessment. I addressed all the mentioned issues and suggestion and made a pull request from master branch. Thank you!

I'll keep this PR open

Does this PR incorporate all your changes? Or are there changes in #6 that you need to apply here?

No this PR incorporate all my changes


Error: "/Volumes/Macintosh: Not a directory"

Installing ruby-2.2.2...
mkdir: /Volumes/Macintosh: Not a directory

##BUILD FAILED (OS X 10.10.3 using ruby-build 20150413)

###Inspect or clean up the working tree at /var/folders/58/fg59q9n577j9mfctgy63ylm00000gn/T/ruby-build.20150422063937.25340
Results logged to /var/folders/58/fg59q9n577j9mfctgy63ylm00000gn/T/ruby-build.20150422063937.25340.log

### Last 10 log lines:
Last-Modified: Mon, 13 Apr 2015 20:10:53 GMT
ETag: "326e99ddc75381c7b50c85f7089f3260"
Accept-Ranges: bytes
Server: AmazonS3
Age: 65639
X-Cache: Hit from cloudfront
Via: 1.1 0a17b6e7205e842c4e078772a70bf8c4.cloudfront.net (CloudFront)
X-Amz-Cf-Id: eo64Oxam2DxYbi3A20nu6FIyZYEVooU08ssb8yxy5gQrTLmHDX-4NA==

/var/folders/58/fg59q9n577j9mfctgy63ylm00000gn/T/ruby-build.20150422063937.25340/ruby-2.2.2 /var/folders/58/fg59q9n577j9mfctgy63ylm00000gn/T/ruby-build.20150422063937.25340 ~

I think my problem may be similar to this perso's below
smace commented on Nov 20, 2014
I have found what was the problem, I'm using a SSD and a HDD in my MBP, / is mounted on my SSD and /user/home on my HDD, rbenv default directory is /user/home/.rbenv moving it to /usr/local/var/rbenv allowed to build ruby!
can you provide specific command line fix here as maybe this will help others in future

Thanks for reporting. I'm guessing this is because of a space in /Volumes/Macintosh HD/ path that the installer is supposed to work with. See #385 and https://github.com/sstephenson/rbenv/issues/558

Is your rbenv root at default location? The default location is ~/.rbenv
Can you tell us how your drives are mounted? df

~
~
❯ cd ~/.rbenv
~/.rbenv
❯ ls
shims    versions
~/.rbenv
❯ cd ..
~
❯ df
Filesystem                                            Size   Used  Avail Capacity   iused     ifree %iused  Mounted on
/dev/disk2                                           111Gi   80Gi   30Gi    73%  21039717   7964953   73%   /
devfs                                                192Ki  192Ki    0Bi   100%       663         0  100%   /dev
/dev/disk1s2                                         698Gi  180Gi  518Gi    26%  47246327 135687415   26%   /Volumes/Macintosh HD
map -hosts                                             0Bi    0Bi    0Bi   100%         0         0  100%   /net
map auto_home                                          0Bi    0Bi    0Bi   100%         0         0  100%   /home
map -fstab                                             0Bi    0Bi    0Bi   100%         0         0  100%   /Network/Servers
//;AUTH=No User<EMAIL_ADDRESS>1.8Ti  565Gi  1.2Ti    32% 148155667 321655423   32%   /Volumes/Public-1

Tried changing names of hard drives and now can not even log in to my user profile
###F********
ed
My data was structures so that I would launch into the SSD drive and had OSX installed on both al my application data was transferred onto SSD although I think technically my home folder was located on the HDD "MIcrosoft HD" and for some reason I've had Rbenv working correctly before maybe 2 months ago.
Not sure what to do now, but if anyone can help I'd appreciate it.

Update : logged in under guest and corrected path to my designated "login folder" with the new HDD name without spaces.
##Root path to my login account is:
/Volumes/OSX_HD/users/#{login_name}

I don't suggest you make any system changes. Instead, install rubies in a location that is not in your home folder, to avoid installing to "Macintosh HD". E.g.:
sudo mkdir -p /opt/rubies
sudo chown $USER:staff /opt/rubies
rm -rf ~/.rbenv/versions
ln -s /opt/rubies ~/.rbenv/versions

Now install Ruby version to /opt/rubies and they will be available as rbenv versions:
$ ruby-build 2.1.5 /opt/rubies/2.1.5
# (`rbenv install 2.1.5` might work as well; not sure)

$ rbenv global 2.1.5

Bad Substation error
Any suggestions as to what I can do from this point?

Result
When I attempted with Ruby-Build...

http://cl.ly/al5d/o

Result with rbenv install
##http://cl.ly/alMd

I disliked using RVM cause it was invasive on my path variables is there any hope for rbenv and Ruby-Build?

Saved -- Thank You!

Sorry you had trouble with my syntax (valid bash) in zsh. Maybe this would be better (in the future):
chown "${USER}:staff" /opt/rubies

Thank you!
That's working for me too! :stuck_out_tongue_winking_eye:
Ruby-build 2.2.2, 10.10.3 OSX.

Installed ruby-2.2.2 to /opt/rubies/2.2.2

I've added this caveat to our wiki so now I feel I can close this. Thanks everyone for participation.

Yes that problem was awful, similar issue that should be added to Wiki is when the file .rbenv.shim gets in the way of the executable path. It has been an issue prior. But I think that should be added into the documenation as well.


⚠ Estudiantes antiguos has degraded performance

In b46d0cb, Estudiantes antiguos (https://os.ar.dhamma.org) experienced degraded performance:

HTTP code: 200
Response time: 18660 ms

Resolved: Estudiantes antiguos performance has improved in d13d529 after 13 minutes.


BC-8 # include "lib" directory in npm package

fixes a bug where the npm package doesn't actually include all the necessary files

lgtm :+1:


Is this boilerplate maintained?

Readme suggests run npm start after installation.
But missing script: start indicated there is no start script in package.json
Any tips?

Getting the same, guess it's been abandoned

Same here. Bugger.

Hey sorry guys. This was a spike and not actively maintained. I'm moving this over to git.habd.as and may circle back to update it at some point but, to me, react apps require active maintenance and most of my time is spent developing sites that don't.


operator [R] project-quay (3.10.0-nightly.20230913 3.9.0-nightly.20230711)

@dmage this should fix the issue causing the failure in #3372

/merge possible

/merge possible


dat-node evaluation

Want to make sure the team is happy using dat-node in CLI and moving forward there.
I'd like to release the dat-node 1.0 beta before/alongside this being released as dat@next.

I think we can wait to release dat-node 1.0 until we evaluate dat-next and make sure we have eberything we need in the API.


Update MockInstance.md

Just some minor typos is stumbled over while reading

Hi @BantramFidian.
Thanks for contributing!


[PE-D][Tester A] Phone Number Validation

Based on the user guide it did not specify who the car workshop tailors its service to thus I will assume anyone can some and service their car. It is also possible for someone to  have a international number, therefore I feel that the input should accept the "+" symbol since there might be a case where +6598765432 and<PHONE_NUMBER> might be used by 2 different people

Labels: type.FunctionalityBug severity.Low
original: Nicholas-Cheng-De-Fei/ped#2

its a feature flaw


TestCase author cannot be updated.

Hi there,
Seems the TestCase author cannot be updated and no error message is printed. Is it by design? Thanks!

@yuxisun1217 if possible can you provide a small snippet of what you have done and what did you expect?
We need to overwrite whole objects generally but I can't be sure whether you are referring to the same or not without looking at an example.

Hi @leelavg ,

Like the "Author" in this snapshot. I know it cannot be edit through UI, but I'm not sure is it possible to modify it through API?
Thanks!

I know it cannot be edit through UI

Ack, then the chances of editing it via API is slim, nevertheless I'm currently afk, will get back at the earliest.

@yuxisun1217 I tried various ways but it's not possible to update author field via API.


Add package: khal

Had to wait until vdirsyncer was merged.
Will test ASAP.

Is the build fail because of my changes?

Can someone tell me what went wrong here and how to fix?

From what I can see from the travis log, you seem to be missing the dependency docopt.
Couldn't find index page for 'docopt' (maybe misspelled?)

Updated the package to 0.4.0, which removes the "docopt" dependency.
Just a quick question besides: I tested the installation via nix-env -iA pkgs.khal -f . from the nixpkgs root directory, but surprisingly I cannot remove the package afterwards (neither nix-env -e khal nor nix-env -e khal -f . work) - How to do this?

Hm. Now things get weird. I managed to build and install the package here on my machine, so why is there a build error?

Sometimes travis fails randomly. Do you have building with chroot enabled?

Nope, I will try.

Am I right with this:
nix-build --option build-use-chroot true -A pkgs.khal -I .

from the nixpkgs root directory?
It just prints the output path of the derivation:
/nix/store/<hash>-python3.4-khal-0.4.0

?

@pSub, @geerds ping.


daniel hasn't made any updates in 2 days?

I am just wondering if you are working on a new feature that's it, I don't care if you release updates regularly or not
ps: quality over quantity

if he is you will see it in commits when it's done

if he is you will see it in commits when it's done

commas are important. i read it as "if he is you" initially. pls fix
Daniel mades very little profit off of Osiris. it is free and all money recieved is from donations. He probably has a job and can't dedicate all his time on the project.
^ he also has to update GOESP, another project he is working on.

if he is you will see it in commits when it's done

commas are important. i read it as "if he is you" initially. ~pls fix~
Daniel mades very little profit off of Osiris. it is free and all money recieved is from donations. He probably has a job and can't dedicate all his time on the project.
^ he also has to update GOESP, another project he is working on.

I didn't know

if he is you will see it in commits when it's done

commas are important. i read it as "if he is you" initially. ~pls fix~
Daniel mades very little profit off of Osiris. it is free and all money recieved is from donations. He probably has a job and can't dedicate all his time on the project.
^ he also has to update GOESP, another project he is working on.

added a comma, also according to daniel's github profile he works for epic games (or is at least in their org on github)


Fix Ventoy not launching on Windows

Fixes #1
This works on my end
Note: Ventoy Toybox must be run with admin privileges.

Confirmed, It does work, Thanks!

Solves 2nd Error in Issue #1


Add creation of kueue resources in e2e tests

Issue link
RHOAIENG-11047
What changes have been made
Since we are adjusting e2e tests to run with AMD gpu, thus

Adding creation of kueue resources as part of test to make it compatible with gpuresources (which is covered in next PR)
Removing creation of kueue resources  from setup script
Adjust methods with updated Ray cluster client

Verification steps
Run test manually
go test -timeout 30m -v ./test/e2e -run TestMnistPyTorchAppWrapperCpu
Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[X] Manual tests
[ ] Testing is not required for this change

@sutaakar @astefanutti
Could someone please approve the PR ?

We may want to think about a strategy for the creation of cluster resources like ClusterQueues in the context of concurrent execution of tests in the future.

Current strategy is to provide generated name for cluster resources.


[vale] Review docs Route PLanner

This PR fixes the Vale linting warnings and/or suggestions.

Closing this as I am unable to trigger a test_build without modifying things and also #61 has not beed merged in. So I'm going to remove this branch and close the PR. We can revisit this work at another time after #61 has been successfully merged into main.


(fix) Submit form changes with submit() instead of click()

Resolves #167.

Thanks for the pull request, let me look into this

Looking good! Thanks for your contribution, @davidparsson


WIP: change withSasl api passing string arguments + api changes

Hey @rtuin and @nickpoulos i would really appreciate your review/feedback here.
What do you think about the withSasl api: should it accepts a Junges\Kafka\Config\Sasl instance or nullable strings to set the password, username, mechanisms and securityProtocol and, if null, it falls back to default sasl configuration?

@mateusjunges I’m having a busy weekend, will look at this asap!


"No media data found" in UI dataset import

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

I want to export a dataset, edit the dataset externaly and upload the dataset again via Import in the UI.
The dataset was created via file share data.
If i import the dataset again via CVAT or COCO format, i receive a visual error

Could not import dataset to the project 7
Error: Request failed with status code 500. "ValueError: No media data found\n".

Expected Behaviour
The dataset should imported and replace / extend the current dataset on the server.
Current Behaviour
The error above appear.
Possible Solution
I think there is a error in the shared files implementation.
Steps to Reproduce (for bugs)

Create a dataset which includes at least one task with images in /shared/folderX/folderY/img.jpg
Export the dataset in the CVAT fileformat via UI
Import the dataset in the CVAT fileformat via UI

Context
I want to export a dataset, edit the dataset externaly and upload the dataset again via Import in the UI.
 No
Operating System and version (e.g. Linux, Windows, MacOS): Debian
Code example or link to GitHub repo or gist to reproduce problem: No
Other diagnostic information / logs:

django@c58e8b630737:~$ cat logs/cvat_server.log
....
[2022-11-03 18:44:25,540] INFO cvat.server.project_7: The project '2022-08-xx' is exported as 'CVAT for images 1.1' at '/home/django/data/projects/7/export_cache/annotations_cvat-for-images-11.ZIP' and available for downloading for the next 3:20:00. Export cache cleaning job is enqueued, id '2fd48142-d81c-44b8-80c2-9fea218200e2'
[2022-11-03 18:45:52,344] INFO cvat.server: create task #116
[2022-11-03 18:45:53,763] ERROR django.request: Internal Server Error: /api/projects/7/dataset

Ive also tested this with the unittest images from https://github.com/opencv/cvat/tree/develop/tests/mounted_file_share/images and it yields the same error.
Maybe the shared folder causes a problem? This issue might be related to https://github.com/opencv/cvat/issues/5230
Export and import a TASK on the same data on a shared folder works.

i tried the proposed changes in the mnt shared from https://github.com/opencv/cvat/issues/5228#issuecomment-1303320245 but it didnt help me on my problem.

Hi, I failed to reproduce the error. Please check that you're exporting with the "Save images" option is enabled.

I have disabled this option. I need to disable this due to huge size of the dataset.
Could you please try to also disable to save images option to reproduce this?

If the imported archive doesn't contain images, it's impossible to import it as a dataset. Consider creating tasks with images and importing annotations to these tasks insted.

Thx for clarifying this, this was not obvious for me from the GUI.
Usecase is

I want to export a dataset, edit the dataset externaly and upload the dataset again via Import in the UI.

As every dataset has quite some tasks (about 30 for our appliocation) its quite some manual work to export , edit the dataset externaly and import each task again.

Ok, please check if you can use our SDK and CLI to automate these operations.

I want to export a dataset(without image files), edit the dataset externaly (add annotations, tags etc. via custom scripts) and upload the dataset again via Import in the UI.

In the exported datasets, do you expect that only annotations are going to change or the image set / the images themselves are going to change too? In the first case, you can just export only annotations and mount the images from the shared directory where they are needed then. In the second case, you will need to recreate tasks manually (maybe, using SDK or CLI) specifying image list from the share in the creation parameters each time and then import annotations if you want to save the disk space.

I expect to not change the images for our usecase. The annotation-info i want to add is already generated externally, so i dont need the images to generate/extend that info.
But due to this bug or missing feature from this issue at least this usecase seems to not work.

This problem is not a bug. The datasets are expected to be self-contained. If you have a set of annotations for an existing task, please use "Upload annotations" as I described above.

This problem is not a bug. The datasets are expected to be self-contained. If you have a set of annotations for an existing task, please use "Upload annotations" as I described above.

Thank you for this info. I though that its still expected to get imported again even if there is no image data included in the export.


stats: Move stats impl discussion from thread_local_store.h into source/doc/stats/md

Description: per @mattklein123 's suggestion on #4711 , adds a stats design doc. In this rev it only pulls in material from a comment in thread_local_store.h. A few TBD sections were added to suggest structure, but the intent is not to fill them out in this PR, which just moves the existing material and adds an overview for context. Future stats-related PRs should include edits to this file to keep it up-to-date.
This is one step toward resolving #4196.
Risk Level: low
Testing: none
Docs Changes: this is a doc change.
Release Notes: n/a

Please pull #4727 for the fix to v6 tests.


add delay to presets animation

Is there a way to add delay to the preset animation ? The doc didn't mention this, if you plan to add this in the future, I think this syntax would look good:
<div v-motion-slide-right  :delay="500" />
The feature I wanted to add to my app is to v-for loop through an array and give each item a different delay based on their index, if this is possible with the current version please let me know how, thanks

The feature I wanted to add to my app is to v-for loop through an array and give each item a different delay based on their index, if this is possible with the current version please let me know how, thanks

This is essentially staggered animation, and although the way you describe may be possible with a convenient preset-adjustment we're all talking about, I think that staggered-anim is still better served by using custom motions triggered by Vue animation hooks, all wrapped in a <TransitionGroup>. This is exactly what I've done using this library for staggered list animations.

I think this should be considered a bug, it's very confusing that the :delay is applied to all motion directives within a v-for and not to individual elements:
<div v-for="i in 5" :key="i">
	<div
		v-motion-slide-visible-left
		:delay="100 * i"
	/>
</div>


Replaced Array2D with swift 2d array

Nice find! 😄
Would you be able to update the playground as well?

Done :)
Commit URL

Thanks!


Release Workflow: Two Actions

The way the current release workflow is set up still doesn't quite work, as we don't create pull requests to bump the actual version tracked in __about__.py in the codebase. The version bumps are done locally in the github action server and only the tags are pushed.
Instead, I think we need to split up the release process into two parts:
GH Action "Tag Release"
This action is triggered manually by someone who wants to release a new version of DJ. They will select either client or server for the library they want to release and select a version bump type like alpha, beta etc. This action will then:

Bump the DJ <server|client> version with hatch version <bump-type>.
Automatically create a pull request with this version bump and wait for manual approval + merge.

GH Action "Release"
This action is triggered when a version bump pull request (from the above action) is merged. It will then do the following:

Create tags for the new version and push them.
Build the new version with hatch build
Publish the new version to PyPi with hatch publish
Create a GH release for this newly released version.

I noticed this when trying to do a release recently and I ended up doing a server and client release manually. It was actually quite easy to do with hatch, but having it be automated and integrated into Github Releases seems nicer 😄


Update Apache-Directory-Studio URL

Corrected a URL within the package

Thank you for the contribution. It was merged directly as e47930b7d7d3fc0c482e47c17da07ca17d2bd86a to keep commit history cleaner. Your contribution is still credited to you.


Rust project updates

Add .gitignore.
Ignore target folder after compiling.
Add the link to rifle in the README for easy install.
Add Cargo.lock to the repository as explained here.
Fix typo

Merged, thanks.


Spyse Integration

It would be really cool to integrate JARM with spyse.com.

domain no longer active


Extract a TaxonPresenter out of the taxon route

Makes the route implementation simpler. It focuses on request-handling
logic while the presenter assembles an object containing everything
needed to render the views.

Note to reviewers: given it's a fairly large change, worth pulling the branch down and having a click around to see if anything's broken.


Update link to Hero library

Summary:
We've moved the Hero project to a GitHub organization so I'm updating the link

          1 Warning

:warning:
Found 2 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

3036
301
http://nshipster.com redirects tohttps://nshipster.com/

3049
301
https://code.facebook.com/ios/ redirects tohttps://code.fb.com/category/ios

  Generated by :no_entry_sign: Danger

Thanks!


Task based refactor

Why are these changes needed?

Refactor to facilitate improvement of time-series fitting.
Delegate task specific logic to a Task sub-class, simplifying the AutoML class.
Introduce a TimeSeriesDataset class for encapsulation of time-series relevant data operations.
Introduce a TimeSeriesEstimator parent class to standardise time-series models.

Related issue number
N/A
Checks

[X] I've used pre-commit to lint the changes in this PR, or I've made sure lint with flake8 output is two 0s.
[X] I've included any doc changes needed for https://microsoft.github.io/FLAML/. See https://microsoft.github.io/FLAML/docs/Contribute#documentation to build and test documentation locally.
[X] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[X] I've made sure all auto checks have passed.

Was looking at the failed build and noticed that there were some import issues. Python 3.6 doesn't include data classes, but there is a backport. Adding that to the python 3.6 build script would solve that issue. Also saw that importing holidays fails. That's not part of the [test] requirements in setup.py (see https://github.com/markharley/FLAML/blob/fafd6524c9466d8ab9c9049bb6102c34cd5819ec/setup.py#L91).

Was looking at the failed build and noticed that there were some import issues.

Python 3.6 doesn't include data classes, but there is a backport that could be added to the build script (and setup).
Also saw that importing holidays fails. That's not part of the [test] requirements in setup.py (see https://github.com/markharley/FLAML/blob/fafd6524c9466d8ab9c9049bb6102c34cd5819ec/setup.py#L91).
It sounds like new tests were added for time series tasks? Should the requirements in [ts_forecast] and [forecast] also be included in [test] in that case?

The holidays are only used in time series forecasting which is optional. Make sure the import statement is invoked only under the time series environment. See https://github.com/microsoft/FLAML/blob/main/flaml/automl.py#L1096 as an example for nlp (another optional environment).

Was looking at the failed build and noticed that there were some import issues.

Python 3.6 doesn't include data classes, but there is a backport that could be added to the build script (and setup).
Also saw that importing holidays fails. That's not part of the [test] requirements in setup.py (see https://github.com/markharley/FLAML/blob/fafd6524c9466d8ab9c9049bb6102c34cd5819ec/setup.py#L91).
It sounds like new tests were added for time series tasks? Should the requirements in [ts_forecast] and [forecast] also be included in [test] in that case?

Hey! Thanks for the review. I've added dataclasses as a Python version dependent install requirement, thanks for spotting it! Holidays turned out not to be used outside of the tests at present so I've removed the unnecessary test in test_forecast

Was looking at the failed build and noticed that there were some import issues.

Python 3.6 doesn't include data classes, but there is a backport that could be added to the build script (and setup).
Also saw that importing holidays fails. That's not part of the [test] requirements in setup.py (see https://github.com/markharley/FLAML/blob/fafd6524c9466d8ab9c9049bb6102c34cd5819ec/setup.py#L91).
It sounds like new tests were added for time series tasks? Should the requirements in [ts_forecast] and [forecast] also be included in [test] in that case?

The holidays are only used in time series forecasting which is optional. Make sure the import statement is invoked only under the time series environment. See https://github.com/microsoft/FLAML/blob/main/flaml/automl.py#L1096 as an example for nlp (another optional environment).

Hey! Thanks for the review @liususan091219. I've applied your relative import suggestions. On holidays, it turns out that it wasn't called outside of tests at present and so I've removed the offending test and redundant code paths

Would it be more reasonable to put the "nlp" and "time_series" folders into the "automl" folder?

Hey! I think we ended up with this layout to avoid some circular imports, but I'll have another try at refactoring these into the automl subpackage :smile:

Some checks failed. I wonder if it will be easier to break the PR down to smaller PRs. For example, the first PR to make is to just create the automl subpackage and move files to corresponding locations. Otherwise this PR would take a long time to merge and you will get conflicts often.

Thank you @markharley! We indeed need to re-organize the whole structure of the flaml folder considering the need of adding an automl folder. I am attaching a proposal for the new structure (considering all content in flaml, not just the changes involved in this PR).
In this PR, perhaps you can just make .py files about automl and the time_series folder in the right place. We can come up with a plan with the other changes (and perhaps also discuss this proposed structure plan in the maintainer meeting on 10/10).

automl

nlp
time_series
automl.py
data.py
ml.py
model.py
train_log.py
config.py
[and other files need to be added in the automl folder]

tune

searcher
scheduler
[and existing files in the tune foler]

onlineml
default


HPA configuraation

While exploring hpa config, I found that current HPA configuration for each components except ingester has incorrect target ref name "openobserve". It should be "openobserve-component_name " .

actually, multiple references were wrong for HPA. fixed it.

@prabhatsharma thanks  for the speedy fix! 🙌