[r2.6][PR] Cannot find real source of dependency No such file or directory  error when building from scratch

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04
TensorFlow installed from (source or binary): source
TensorFlow version: r2.6
Python version: 3.9.5
Installed using virtualenv? pip? conda?: miniconda
Bazel version (if compiling from source): 3.7.2
GCC/Compiler version (if compiling from source): 7.5.0

Describe the problem
I am currently working on a relatively large PR to introduce ZSTD support in TF for TFRecordWriter, which currently supports ZLIB and GZIP.
To introduce my changes, I am of course writing some tests, and as I am working my way up to the dependencies chain, I have stumbled upon this error:
(tensorflow) ubuntu@tensorflow-compression-build-1:~/Workspace/tensorflow$ bazel test //tensorflow/core/lib/io/zstd:zstd_test --test_filter=* --verbose_failures
INFO: Options provided by the client:
  Inherited 'common' options: --isatty=1 --terminal_columns=143
INFO: Reading rc options for 'test' from /home/ubuntu/Workspace/tensorflow/.bazelrc:
  Inherited 'common' options: --experimental_repo_remote_exec
INFO: Reading rc options for 'test' from /home/ubuntu/Workspace/tensorflow/.bazelrc:
  Inherited 'build' options: --define framework_shared_object=true --java_toolchain=@tf_toolchains//toolchains/java:tf_java_toolchain --host_java_toolchain=@tf_toolchains//toolchains/java:tf_java_toolchain --define=use_fast_cpp_protos=true --define=allow_oversize_protos=true --spawn_strategy=standalone -c opt --announce_rc --define=grpc_no_ares=true --noincompatible_remove_legacy_whole_archive --enable_platform_specific_config --define=with_xla_support=true --config=short_logs --config=v2 --define=no_aws_support=true --define=no_hdfs_support=true
INFO: Reading rc options for 'test' from /home/ubuntu/Workspace/tensorflow/.tf_configure.bazelrc:
  Inherited 'build' options: --action_env PYTHON_BIN_PATH=/home/ubuntu/miniconda3/envs/tensorflow/bin/python --action_env PYTHON_LIB_PATH=/home/ubuntu/miniconda3/envs/tensorflow/lib/python3.9/site-packages --python_path=/home/ubuntu/miniconda3/envs/tensorflow/bin/python
INFO: Reading rc options for 'test' from /home/ubuntu/Workspace/tensorflow/.tf_configure.bazelrc:
  'test' options: --flaky_test_attempts=3 --test_size_filters=small,medium
INFO: Found applicable config definition build:short_logs in file /home/ubuntu/Workspace/tensorflow/.bazelrc: --output_filter=DONT_MATCH_ANYTHING
INFO: Found applicable config definition build:v2 in file /home/ubuntu/Workspace/tensorflow/.bazelrc: --define=tf_api_version=2 --action_env=TF2_BEHAVIOR=1
INFO: Found applicable config definition test:v2 in file /home/ubuntu/Workspace/tensorflow/.tf_configure.bazelrc: --test_tag_filters=-benchmark-test,-no_oss,-gpu,-oss_serial,-v1only --build_tag_filters=-benchmark-test,-no_oss,-gpu,-v1only
INFO: Found applicable config definition build:linux in file /home/ubuntu/Workspace/tensorflow/.bazelrc: --copt=-w --host_copt=-w --define=PREFIX=/usr --define=LIBDIR=$(PREFIX)/lib --define=INCLUDEDIR=$(PREFIX)/include --define=PROTOBUF_INCLUDE_PATH=$(PREFIX)/include --cxxopt=-std=c++14 --host_cxxopt=-std=c++14 --config=dynamic_kernels --distinct_host_configuration=false
INFO: Found applicable config definition build:dynamic_kernels in file /home/ubuntu/Workspace/tensorflow/.bazelrc: --define=dynamic_loaded_kernels=true --copt=-DAUTOLOAD_DYNAMIC_KERNELS
DEBUG: /home/ubuntu/.cache/bazel/_bazel_ubuntu/ef409778dc6f6079679ade72bc957ee0/external/tf_runtime/third_party/cuda/dependencies.bzl:51:10: The following command will download NVIDIA proprietary software. By using the software you agree to comply with the terms of the license agreement that accompanies the software. If you do not agree to the terms of the license agreement, do not use the software.
INFO: Analyzed target //tensorflow/core/lib/io/zstd:zstd_test (1 packages loaded, 70 targets configured).
INFO: Found 1 test target...

Target //tensorflow/core/lib/io/zstd:zstd_test failed to build
INFO: Elapsed time: 0.926s, Critical Path: 0.28s
INFO: 5 processes: 5 internal.
FAILED: Build did NOT complete successfully
//tensorflow/core/lib/io/zstd:zstd_test                         FAILED TO BUILD

FAILED: Build did NOT complete successfully

And it did happen before, so I just went into the BUILD file that the error references, and look if there is somewhere I might have missed a dependency somehow. I have yet to find the real source of the dependency, and I have gone through every possible place where I should have put my dependencies.
Provide the exact sequence of commands / steps that you executed before running into the problem
Clone my fork and go to my branch, with commit sha 9d7b6a5e223c59d8d9687ce128dd1ebc5a6ab908 or build-issue-adrian-compression-r2.6:
git clone https://github.com/IAL32/tensorflow
git checkout 9d7b6a5e223c59d8d9687ce128dd1ebc5a6ab908

Build from source however you like, and execute the test:
bazel test //tensorflow/core/lib/io/zstd:zstd_test

You should now see the error above.
Finally, if you revert the last commit, which changes tensorflow/core/lib/io/record_writer.cc and tensorflow/core/lib/io/record_writer.h to accommodate my ZSTD class, launching the test again works fine.
An overview of the stuff I have changed can be found here: https://github.com/tensorflow/tensorflow/compare/r2.6...IAL32:build-issue-adrian-compression-r2.6?expand=1
I believe this can be solved by including some header as a dependency, but I have been struggling to understand where exactly.
Thanks for any help!

Hi @Saduf2019! Could you please look at this issue?

@IAL32
With respect to the error "" can you refer to this link, also please downgrade your python and let us know if this is still an issue.
Please refer to similar error issues:Link,link1,link2

Hi @Saduf2019 , thanks for the answer! Although, none of the links you have provided really answer my issue, or have much to do with it.
The "header missing" error does not come from the lack of the library, which is correctly provided and installed in third_party/zstd.BUILD (and of course is not supposed to be installed on the local machine).
What I am having issues with is not the procedure of building itself, but this specific behavior of the build system when a new dependency is introduced in reader_writer.cc.

I think I have fixed it, although I really don't know why.
I have just moved the declaration for the ZstdOutputBuffer from the header to the cpp file:
https://github.com/IAL32/tensorflow/commit/efef02f0026ec3b2a6e590337d15e88e859e9a20#diff-5ab5acb26dbf183efb7a3707d0b15f82287cc4d21720caa91ca08d215b140838
I will close the issue, although it is apparent that this should not be done, it serves my purposes for now.


Allow access to the currently active 'when clauses' from the extension api

Hi, I couldn't find a way to access the currently active 'when clauses' from an extension. This could be useful to allow the extension to behave in a different way if some clauses are active or not. I have seen it requested multiple times in Gitter.
An example of a use case: http://stackoverflow.com/questions/43372232/a-method-to-expose-the-vscode-active-when-clause-contexts

dupe of #10471


Summarized node execution

Prints out a summarized execution of the nodes as per the conditions. I guess it's also pulling in a bunch of the hundreds of commits made before to this branch.

@Ivy8127 Closing this as I believe all your changes related to summarising node execution are in the other PR


Counting workshops with memberships

I have seen some discrepancies in how workshops are being counted on the membership page. For example:
On a member view page like this we see that  they have completed 4 of 6 allowed workshops plus 1 self organized workshop.
Yet when I do SELECT * FROM workshops_event WHERE membership_id = 253  I get 8 workshops total (1 self organized and 7 centrally organized).
Where is this discrepancy coming from?

The membership ended Aug 05, which makes some of ASU events fall out of calculated numbers.
Here are the events and why they are not counted in:

name
reason

2020-12-07--agave-shell-intro
self-organized

2020-12-14-asu
tagged as stalled

2021-01-18-tempe
valid (even though tagged as unresponsive)

2021-03-08-asu-online
valid

2021-06-14-asu-online
valid

2021-07-06-asu-online
valid

2021-08-16-asu-online
outside membership agreement dates

2021-09-10-asu-online
outside membership agreement dates

As you can see in the table above, only 4 entries are valid and these are the 4 counted on the membership page.

Ok.  Now that we have the membership field in workshops_events  this should be used for assigning a workshop to a membership, not the membership valid dates.


adds to contibution

This is my contribution.

Sorry, this is an invalid PR.
Please take Hacktoberfest as opportunity to learn something new. Please try to learn a programming language or a tool. Just adding yourself to some lists does not level up your skills.

Hacktoberfest's value #2 Quantity is fun, quality is key


dart2js cps: Variables declared in finally block

Variables declared in a finally block cause an assertion error in the CPS builder, because one variable is added to the environment multiple times.
This is the assertion that fails:
  void extend(Local element, ir.Primitive value) {
    // Assert that the name is not already in the environment.  `null` is used
    // as the name of anonymous variables.
    assert(!variable2index.containsKey(element));
    if (element != null) variable2index[element] = index2variable.length;
    index2variable.add(element);
    index2value.add(value);
  }

There are a bunch of tests that fail in checked-mode because of this; here is a minimal one:
main() {
  try {
    print("fdg");
  } finally {
    var x = 12345678;
    print(x);
  }
}

The variable may be assigned different indices because of different environment sizes at entry points to the finally. I can't see an easy fix to this short of wrapping all variable elements in a "finally element" so different copies of the variable can be distinguished.
The issue should go away when we introduce first-class finally to the IR. I'm not sure if it's worth fixing in the current form.
@kmillikin

Obsolete

Issues are still marked with this number in pkg.status. Please investigate.

The status file sections were obsolete. Fixed in 2d6c90a6b3d6d1be5095e12a0a633305e1d5db2a.


Line/Area is clipped, Scatter is not

Bugs and Questions
Checklist

[X] This is a victory-native specific issue. (Issues that also appear in victory should be opened here)

[X] I have read through the FAQ and Guides before asking a question

[X] I am using the latest version of victory-native

[X] I have checked to make sure that my versions of react-native and react-native-svg are compatible with this version of victory-native. Read about version requirements

[X] I've searched open issues to make sure I'm not opening a duplicate issue

The Problem
When I have a scatter and an area inside of a VictoryChart with padding, the line/area gets clipped, but the scatter does not.

I tried to apply clip padding (i.e. clipContainerComponent={<VictoryClipContainer clipPadding={{ left: 20, right: 20 }} />}), but it didn't change anything

This is expected behavior for Victory, since VictoryLine and VictoryArea use VIctoryClipContainer as their groupComponent. You can try switching out the groupComponent if you want different behavior

using groupComponent={<VictoryClipContainer />} worked, thanks!


Update to run 'devcontainer exec' without JSON parsing output

As discussed in #223, an upcoming change in devcontainer exec will mean that it no longer returns a JSON response object via stdout. In preparation for that change, this PR modifies the behaviour of the action/task to determine the outcome of devcontainer exec solely using the process exit code (which is valid currently as per this comment)

/test


Revisit UI/UX of playground

The idea here is to create a better experience for our users

According to our meeting

[ ] We'll add the deployment costs calculated to the right so the user can see the price updated without att details without scrolling.
[ ] Remove the balance from the nav and add it at the top of the deployment costs calculated (the new).
[ ] change the size of Connect your TFChain Wallet and Connect your TFchain Wallet pop-up (It is too wide)
[ ] add license checkbox at Connect your TFchain Wallet.
[ ] Change Pinck color to blue to match with new branding.
[ ] Show the contracts page in a better way.

@rkhamis @AhmedHanafy725 If there is something missing please update the issue

first edition

https://www.figma.com/file/A16wcW2xqLCWWxkj9NduRt/Playground-UI%2FUX?type=design&node-id=0-1&t=Zszk7gHTmjjeQOvA-0

what do you think if we add the deploy button to price details, it'll be clearer?

Also, this pop-up connects TF chain wallet


[SPARK-6431][Streaming][Kafka] Error message for partition metadata requ...

...ests
The original reported problem was misdiagnosed; the topic just didn't exist yet.  Agreed upon solution was to improve error handling / message

Can one of the admins verify this patch?

I am not sure if I can ask Jenkins to test a patch that isn't mine, but I will try and see if Jenkins likes me.

Jenkins, test this please

LGTM btw :-)

ok to test

Test build #30121 has started for   PR 5454 at commit 44300f8.

Test build #30121 has finished for   PR 5454 at commit 44300f8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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


BaseParser still defines quote as a property but doesn't work

Version: 7.0.2
Node 18
You can see here that quote: string is defined but if you look at the docs you need to use a string formatter. This is confusing especially because the base param doesn't work and the docs don't mention this.
https://github.com/juanjoDiaz/json2csv/blob/main/packages/plainjs/src/BaseParser.ts#L65
export interface Json2CSVBaseOptions<TRaw, T> {
  fields?: Array<string | FieldInfo<T, unknown>>;
  ndjson?: boolean;
  defaultValue?: string;
  quote?: string; // This should be removed or if its intentional its broken
  escapedQuote?: string;
  delimiter?: string;
  eol?: string;
  excelStrings?: boolean;
  header?: boolean;
  includeEmptyRows?: boolean;
  withBOM?: boolean;
  formatters?: FormattersOptions;
  transforms?:
    | []
    | [Transform<TRaw, T>]
    | [Transform<TRaw, any>, ...Array<Transform<any, any>>, Transform<any, T>];
}

const parser = new AsyncParser({
      delimiter: columnDelimiter,
      eol: recordDelimiter,
      header: withHeader,
      quote: "", // This doesn't work
      formatters: {
        string: stringFormatter({ quote: "" }) // This works
      }
    });

Good catch!
It's a typings error.
Will fix asap 😁

Fixed in v7.0.6


Nudging is broken

After the latest refactoring the nudging doesn't work anymore.
The ParameterGestureHandler is no longer calling onChange in the keyhandler, just onNudge, but Control is not applying the nudged value by calling onChange itself.

thanks, this should now be fixed in v0.5.2!


Feedback on potential README improvements

As a new user, the README instructions came across as quite confusing to me. Here are some suggestions:

What is the purpose of wdsub, and what is the general mechanism by which it operates (detail)?
The installation and compilation is a bit unclear, ie using sbt vs docker
Usage and functions could be improved -- what do they do, how, and why?
Examples of README instructions that may be helpful: QIIME2 Metabolomics plug-in, Configuration instructions for our Applied Bioinformatics course (this was updated after feedback from students)

That would be great!
Thanks a lot for using wdsub. For some reason, I missed the github notifications about your issue trying to install wdsub and also this answer.
We have been working these days on a sister project to wdsub called sparkwdsub which can also be used to create wikidata subsets but follows a different approach based on Apache Spark which allows to create the subsets using ShEx based graph traversal.


Fatal error: out of memory.

After successful 13 hours run with new release, memory is consumed fully and throwed fatal error.
Fatal error: out of memory.
Coda process exited with status code 2
2020-12-07 11:49:18 UTC [Info] Coda daemon is booting up; built with commit "[DIRTY]880882e45e48aa
f987aca191f488c7f65609c305" on branch "HEAD"
              total        used        free      shared  buff/cache   available
Mem:       16283524    12456216     1977732<PHONE_NUMBER>     5116360
Swap:       8182780     3384168     4798612

Global number of accounts:       1193
Block height:                    2595
Max observed block length:       2595
Local uptime:                    12h20m5s
Ledger Merkle root:              jwX2vNkcdaQb4mPuieN1LgNA9fZoVpxZ2edehWU573HDi5URRhH
Protocol state hash:             3NLJK8nJw8p4r5NDma5tPydpCpWPUKdapAQf1wDtASnMDcoTBtXQ
Chain id:                        ef95774fb2ed657da61bfc40a8a148a9c9a202476b539aaf915e6cd81d1ef268
Git SHA-1:                       [DIRTY]880882e45e48aaf987aca191f488c7f65609c305
Peers:                           148
Sync status:                     Synced
Block producers running:         1
Best tip consensus time:         epoch=1, slot=2055
Next block will be produced in:  in 5.993d
Consensus time now:              epoch=1, slot=205

After restart, catchup again but as you can see from below output, Block height is pretty old. I guess somehow cannot write sync data to disk.
Before out of memory.
Global number of accounts:       1201
Block height:                    2797
Max observed block length:       2797
Local uptime:                    5h28m57s
Ledger Merkle root:              jxcJSJ2QgoWXQ9MigzvSXJL1QwfY8kK3ad34ffUThZdTuJgNvmp
Protocol state hash:             3NLdGk188MSnYR8XHVz1GpW4DYvnvU5AFmVgkDf3Wc1iTiXECA1g
Chain id:                        ef95774fb2ed657da61bfc40a8a148a9c9a202476b539aaf915e6cd81d1ef268
Git SHA-1:                       [DIRTY]880882e45e48aaf987aca191f488c7f65609c305
Peers:                           140
Sync status:                     Synced
Block producers running:         1
Best tip consensus time:         epoch=1, slot=2613
Next block will be produced in:  in 4.827d
Consensus time now:              epoch=1, slot=2616
After out of memory.
Global number of accounts:       1189
Block height:                    2516
Max observed block length:       2808
Local uptime:                    39m41s
Ledger Merkle root:              jxy8Ua3r3WodrexUWZJ82HEAm9xunLDQ7zWnN1sgeDEV5pDu1N9
Protocol state hash:             3NK6u7MYSFuWMiNuFTNXKEZXe2jUtfaZGQhsQQXaHetHGY8mwxR6
Chain id:                        ef95774fb2ed657da61bfc40a8a148a9c9a202476b539aaf915e6cd81d1ef268
Git SHA-1:                       [DIRTY]880882e45e48aaf987aca191f488c7f65609c305
Peers:                           102
Sync status:                     Catchup
Block producers running:         1
Best tip consensus time:         epoch=1, slot=1856
Next block will be produced in:  in 4.763d
Consensus time now:              epoch=1, slot=2646

Hello,
The daemon also crash quite often because of out of memory issue.
The last log in entry before the crash was
2021-06-04 11:22:51 UTC [Info] Received a block from $sender
	sender: {
  "Remote": {
    "host": "<IP_ADDRESS>",
    "peer_id": "12D3KooWByaQ1ojuvEsJgFHvgsCth5Qtra6tk3298AiY41CTHWWW",
    "libp2p_port": 8302
  }
}

After that (maybe unrelated), it tried to allocate something like 91 GB ??
Attempted to allocate<PHONE_NUMBER>3 bytesFatal error: out of memory.
It runs in a docker container. The physical machine has 16GB of RAM + 16 GB of swap in case.
Version is
Git SHA-1:                                     a42bdeef6b0c15ee34616e4df76c882b0c5c7c2a

I think the problem of memory allocation does still exist in version 1.2.0beta1-c856692-mainnet
Possibly more hidden than before
2021-07-02 10:28:28 UTC [Error] Possible reason for signal: "Process killed because out of memory"
Log extract
2021-07-02 10:27:31 UTC [Warn] RPC call error for "get_transition_chain_proof"
2021-07-02 10:27:35 UTC [Error] error sending message on stream 25703: $error
error: {
"commit_id": "c856692fddc525a673ba075f714811b5c50bd3a7",
"string":
"RPC #46101 failed: "only wrote 0 out of 9 bytes error: libp2p error error: closed stream""
}
2021-07-02 10:27:35 UTC [Warn] RPC call error for "get_transition_chain"
2021-07-02 10:27:41 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] Not rebroadcasting block $state_hash because it was received "1 slots too late"
state_hash: "3NLex6ZoBK5FxhxRwPwmEBHLVEycVBf2kWULC9gTcDwPq5wskHQz"
2021-07-02 10:27:51 UTC [Info] Saw block with state hash $state_hash
state_hash: "3NLex6ZoBK5FxhxRwPwmEBHLVEycVBf2kWULC9gTcDwPq5wskHQz"
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:51 UTC [Warn] initial_validate: disconnected chain
2021-07-02 10:27:54 UTC [Warn] RPC call error for "get_transition_chain_proof"
2021-07-02 10:27:55 UTC [Info] Received a block from $sender
sender: {
"Remote": {
"host": "<IP_ADDRESS>",
"peer_id": "12D3KooWAbpbSc9WbfkrJE8FQNLebzL9A7WGUVaMFaEjexW4MPmU",
"libp2p_port": 8302
}
}
2021-07-02 10:28:05 UTC [Fatal] Unhandled top-level exception: $exn
Generating crash report
exn: {
"commit_id": "c856692fddc525a673ba075f714811b5c50bd3a7",
"sexp": [
"monitor.ml.Error",
"Cached item has already been finalized",
[
"Raised at file "src/error.ml" (inlined), line 9, characters 14-30",
"Called from file "src/lib/cache_lib/impl.ml", line 173, characters 30-51",
"Called from file "src/lib/cache_lib/impl.ml", line 199, characters 6-69",
"Called from file "src/lib/ledger_catchup/super_catchup.ml", line 928, characters 25-58",
"Called from file "src/deferred1.ml", line 17, characters 40-45",
"Called from file "src/job_queue.ml" (inlined), line 131, characters 2-5",
"Called from file "src/job_queue.ml", line 171, characters 6-47",
"Caught by monitor coda"
]
],
"backtrace": [
"Raised at file "format.ml" (inlined), line 242, characters 35-52",
"Called from file "format.ml", line 469, characters 8-33",
"Called from file "format.ml", line 484, characters 6-24"
]
}
2021-07-02 10:28:05 UTC [Info] Updating new available work took 21.804094314575195 ms
2021-07-02 10:28:28 UTC [Error] Daemon child process 97 terminated after receiving signal "sigkill"
2021-07-02 10:28:28 UTC [Error] Possible reason for signal: "Process killed because out of memory"
2021-07-02 10:28:28 UTC [Error] Child process of kind "Prover" with pid 97 has unexpectedly terminated
2021-07-02 10:28:28 UTC [Fatal] Unhandled top-level exception: $exn
Generating crash report
exn: {
"commit_id": "c856692fddc525a673ba075f714811b5c50bd3a7",
"sexp": [
"monitor.ml.Error",
[ "Failure", "Child process of kind Prover has unexpectedly terminated" ],
[
"Raised at file "stdlib.ml", line 33, characters 17-33",
"Called from file "src/app/cli/src/cli_entrypoint/mina_cli_entrypoint.ml", line 579, characters 10-94",
"Called from file "src/deferred0.ml", line 56, characters 64-69",
"Called from file "src/job_queue.ml" (inlined), line 131, characters 2-5",
"Called from file "src/job_queue.ml", line 171, characters 6-47",
"Caught by monitor coda"
]
],
"backtrace": [
"Raised by primitive operation at file "src/signal.ml", line 162, characters 6-61"
]
}
2021-07-02 10:28:28 UTC [Error] verifier terminated unexpectedly
2021-07-02 10:28:28 UTC [Info] Starting a new verifier process
2021-07-02 10:28:28 UTC [Info] verifier successfully stopped
2021-07-02 10:28:28 UTC [Info] Rebroadcasting $state_hash
state_hash: "3NLnPyR37jEecT8RdmJ6m7HKnNxicr8gj1kKA4x5iZC4nueGevJ4"
2021-07-02 10:28:28 UTC [Fatal] libp2p_helper process died unexpectedly: "died after receiving sigkill (signal number 9)"
2021-07-02 10:28:28 UTC [Error] error during validationComplete, ignoring and continuing: $error
error: {
"commit_id": "c856692fddc525a673ba075f714811b5c50bd3a7",
"string":
"helper process already exited (doing RPC {"seqno":25735,"is_valid":"accept"})"
}

@EmreNOP @hsq125 do you think it is still valid issue and relevant to the most recent Daemon versions?

I'll close this issue as Stale. Please feel free to open another one should you experience any issues in future.


Fix some doc links

/run-acceptance-tests

Thank you for the catch! I think synchronizing the python README is fine; it does not get regenerated during CI.


Add new flux command sleep #495

Resolves #495

Basic idea looks good. Some issues:

PR description should reference the issue using closing keywords
I'd copy an existing module, e.g. ObjectExceptionCatcher to get the types right (current build: does not compile cannot find symbol [...] class T)
Formatting is off (comments), would also be avoided by copying an existing module
I think the flux command should be just sleep
The flux command needs to be added in src/main/resources/flux-commands.properties
The Thread.sleep needs to be wrapped in a try-catch-block, handling InterruptedException, see e.g. in the OERSI SitemapReader

Basic idea looks good. Some issues:
* PR description should reference the issue using closing keywords

* I'd copy an existing module, e.g. ObjectExceptionCatcher to get the types right (current build: does not compile `cannot find symbol [...] class T`)

* Formatting is off (comments), would also be avoided by copying an existing module

* I think the flux command should be just `sleep`

* The flux command needs to be added in `src/main/resources/flux-commands.properties`

* The `Thread.sleep` needs to be wrapped in a try-catch-block, handling `InterruptedException`, see e.g. in the OERSI [SitemapReader](https://gitlab.com/oersi/oersi-etl/-/blob/master/src/main/java/oersi/SitemapReader.java?ref_type=heads#L65)

Thanks I adjusted as suggested.
It is missing java tests. I am not sure how to add them for this specific function.

Would it make sense to enable this command to also support the Fix function use case? Just add another setter for the time unit and extract the sleep "action" into a dedicated method. Then we could easily reuse this class to implement the corresponding Fix function (see Catmandu).

Would it make sense to enable this command to also support the Fix function use case? Just add another setter for the time unit and extract the sleep "action" into a dedicated method. Then we could easily reuse this class to implement the corresponding Fix function (see Catmandu).

I think this is a good idea, could you provide the code for this?

@fsteeg can you review the additions that @blackwinter introduced. Also can you tell me how I would write a unit test for this? Not sure since it does change the result in a transformation.

Also can you tell me how I would write a unit test for this?

Initially, I thought we should just skip the test for this. But since you brought it up: You could verify that the process takes at least the amount of time that it was configured to sleep. It's not pretty, but it's something.

Not sure since it does change the result in a transformation.

You mean "it does not change the result", right?

Also can you tell me how I would write a unit test for this?

Initially, I thought we should just skip the test for this. But since you brought it up: You could verify that the process takes at least the amount of time that it was configured to sleep. It's not pretty, but it's something.

Okay, I will have a look at this.

Not sure since it does change the result in a transformation.

You mean "it does not change the result", right?

Correct, it does not change the result

Also can you tell me how I would write a unit test for this?

Initially, I thought we should just skip the test for this. But since you brought it up: You could verify that the process takes at least the amount of time that it was configured to sleep. It's not pretty, but it's something.

Not sure since it does change the result in a transformation.

You mean "it does not change the result", right?

@blackwinter is this the kind of test you had in mind?

Yes, thanks. However, I've modified your test a little (74bc15d).
Stylistic changes:

Switch from Instant/Duration to nanoTime().
Avoid static import statements.
Declare local variables as final.
Provide failure message if assertion fails.
Clean up formatting/indentation.

Functional changes:

Verify that sleep time doesn't exceed expected time by too much (otherwise you could make all tests pass by always sleeping for a very long time).
Add tests with explicit time unit.


SPSA manual parameter naming problem

Issue Description
What did you expect to see happen?
In the manual, some of the parameters are named with a different name then what it is in the  code.
For example,
gainGrowthFactor is expected by the code, but the manual says we shoould input gainGrowthSize
Do you have a suggested fix for the development team?
Fix
Please attach the input file(s) that generate this error.  The simpler the input, the faster we can find the issue.

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[x] 1. Is it tagged with a type: defect or improvement?
[x] 2. Is it tagged with a priority: critical, normal or minor?
[x] 3. If it will impact requirements or requirements tests, is it tagged with requirements?
[x] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
[x] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[x] 1. If the issue is a defect, is the defect fixed?
[x] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[x] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[x] 4. If the issue is a defect, does it impact the latest stable branch? If yes, is there any issue tagged with stable (create if needed)?
[x] 5. If the issue is being closed without a merge request, has an explanation of why it is being closed been provided?

I think this will also affect the stable branch. @alfoa

This issue only requires manual update, and I changed the tag to improvement. However, an email should be sent to the users regarding the fix for this issue.

Checklist is satisfied, and an email is proposed in PR#673


[FAB-17508] Removing viper library from viperutil

Type of change
Improvement (improvement to code by avoiding viper library that is not
needed in this context)
Description
Avoiding the overlap in the implementation of getKeysRecursively and
viper.Get() function. The sub-keys needed by getKeysRecursively is
readily available and the usage of viper.Get() can be avoided.
The reference to viper library is removed from the following modules

orderer/common/localconfig
internal/configtxgen/genesisconfig
common/viperutil

A ConfigParser class is created in viperutil/config_utils.go to achieve
the needed funtionality.
Additional details
Unit tests done are the following
go test -race -count 1 -v -run Test github.com/hyperledger/fabric/orderer/common/localconfig | tail -2
PASS
ok      github.com/hyperledger/fabric/orderer/common/localconfig
1.260s
go test -race -count 1 -v -run Test github.com/hyperledger/fabric/internal/configtxgen/genesisconfig | tail -2
PASS
ok      github.com/hyperledger/fabric/internal/configtxgen/genesisconfig
1.115s
go test -race -count 1 -v -run Test github.com/hyperledger/fabric/common/viperutil | tail -2
PASS
ok      github.com/hyperledger/fabric/common/viperutil  0.793s
go test -race -count 1 -v -run Test github.com/hyperledger/fabric/orderer/consensus/etcdraft | tail -2
PASS
ok      github.com/hyperledger/fabric/orderer/consensus/etcdraft
52.528s
go test -race -count 1 -v -run Test github.com/hyperledger/fabric/orderer/consensus/etcdraft | grep SUCCESS!
SUCCESS! -- 107 Passed | 0 Failed | 0 Pending | 0 Skipped
Related issues
https://jira.hyperledger.org/browse/FAB-17508
Signed-off-by: NicholasB<EMAIL_ADDRESS>

Type of change

Improvement (improvement to code, performance, etc)

Description
Avoiding the overlap in the implementation of getKeysRecursively and
viper.Get() function.
Additional details
Unit Tests Done
Related issues
https://jira.hyperledger.org/browse/FAB-17508

@sykesm apologies as I had to create a separate new PR.  Prior to doing a squash in the previous PR, I synced fabric/master to bug-fix branch and inadvertently during squash failed to drop other commits. Further tried git rebase --onto option and messed up my bug-fix branch. I tried a few things but was unable to recover, so created a new branch.
Unfortunately the review history is not here in this PR, but available at https://github.com/hyperledger/fabric/pull/1392. I have double checked and made sure that every review comment is addressed here.


Connecting to cortex wss over external ip

Hi,
In the examples, I see that connection ip is always "wss://localhost:6868" which implies that any app should be running locally. Is there a way to connect to cortex client service through global ip i.e. lan ip? For e.g. my machine has an ip address of <IP_ADDRESS> but connecting to "wss://<IP_ADDRESS>:6868" via cortex api fails. Any help or pointers would be appreciated.
Regards,
Nitesh

@niteshbhatia008 , that is good idea.
we will consider support it in the future.


Temp fix for UI scaling on Android

I think this may be a common problem with small screens or high DPI displays, but as you've said, we can look into it later.

Thanks!


Add smc_run_get_method

I have in the C# library I've been referencing, the ability to call smc.runGetMethod against liteclients and then interrogate the returned Bag of Cells.
Is it possible to add this to your implementation? It's much more featured, but missing this vital piece.
smc.runGetMethod id:int53 method:smc.MethodId stack:vector<tvm.StackEntry> = smc.RunResult

Or does it already exist and I can't find it?

Found it on smc_load:
        let (connection, id) = self.clone().ton_client.smc_load(&address.to_string()).await?;
        connection.smc_run_get_method(id, TonMethodId::Name("METHOD"))


Misc changes

Allow for easier debugging of PETs
Add ability to fetch hosts for a region without overriding the global preferred region -> great for credential exports
Adopt new function signatures
Add function to easily create standalone credentials

@labaiv63 please review & merge once you're ok with this.


refactor: move async key value source into libsinsp

Signed-off-by: Jason Dellaluce<EMAIL_ADDRESS>What type of PR is this?
/kind cleanup
/kind design
Any specific area of the project related to this PR?
/area libsinsp
What this PR does / why we need it:
The async_key_value_source class used to reside in its own async base level directory. However, such a class is only used in the scope of libsinsp, and having an extra directory inside userspace makes the overall design of the libraries repo more confusing.
This PR cleans this up by moving the async directory inside userspace/libsinsp and by putting the async_key_value_source class under the libsinsp namespace.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
refactor: move async key value source into libsinsp

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: jasondellaluce
To complete the pull request process, please assign mstemm after the PR has been reviewed.
You can assign the PR to them by writing /assign @mstemm in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

LGTM label has been added.  Git tree hash: 8eb43c665bb2518117396b53d06d5a15ec7827c6

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

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


Fullcontrol path requests has waypoints with wrong yaw values

When x coordinates of consecutive waypoints are decreasing the yaw value is always 0.0 but it needs to face the other direction (3.14). The same goes for decreasing y coordinates where it is 1.57 but needs to be -1.57. I also notice robot spins to 0.0 degrees infront of the doors. What node is providing the yaw values in the robot_path_requests messages?
path:
- t:
    sec: 1851
    nanosec: 745000000
  x: 48.25251388549805
  y: -19.186559677124023
  yaw: 1.5216608047485352
  obey_approach_speed_limit: false
  approach_speed_limit: 0.0
  level_name: L1
  index: 0
- t:
    sec: 1852
    nanosec: 42178250
  x: 48.25251388549805
  y: -19.186559677124023
  yaw: 1.5658183097839355
  obey_approach_speed_limit: false
  approach_speed_limit: 0.0
  level_name: L1
  index: 0
- t:
    sec: 1856
    nanosec: 579068139
  x: 48.242881774902344
  y: -21.121646881103516
  yaw: 1.5658183097839355
  obey_approach_speed_limit: false
  approach_speed_limit: 0.0
  level_name: L1
  index: 0
- t:
    sec: 1859
    nanosec: 465962349
  x: 48.242881774902344
  y: -21.121646881103516
  yaw: 0.013681734912097454
  obey_approach_speed_limit: false
  approach_speed_limit: 0.0
  level_name: L1
  index: 0
- t:
    sec: 1869
    nanosec: 817861630
  x: 43.40037155151367
  y: -21.15367317199707
  yaw: 0.013681734912097454
  obey_approach_speed_limit: false
  approach_speed_limit: 0.0
  level_name: L1
  index: 0
- t:
    sec: 1870
    nanosec: 62518684
  x: 43.40037155151367
  y: -21.15367317199707
  yaw: -0.016246803104877472
  obey_approach_speed_limit: false
  approach_speed_limit: 0.0
  level_name: L1
  index: 0
task_id: '1473'

The robot appears to  spin randomly at turning points and in front of doors. We are using ROS2 free_fleet and Nav2. Can someone help to fix this behaviour? Thanks.

If your robot is designed to generally not drive in reverse, then be sure to set the reversible parameter to false. By default that parameter is true, so the robot will prefer to drive in reverse if the alternative would be to spend time turning 180-degrees.
The default was set to true because our earliest testing platforms for RMF were agnostic towards driving forwards or backwards, but I've been thinking about changing that default to false considering how many robots (especially commercial robots) seem to only want to drive forwards :sweat_smile:.

Thank you, changing the parameter fixed it.


UI: Wait confirmations is not respected in the UI

When waiting for a certain number of confirmations after a tx, the UI sometimes shows that all confirmations are met even when it is not the case.
https://user-images.githubusercontent.com/9441295/156392444-75ab0d7c-7dcd-4919-a11d-cfb8eb071f67.mov

Closing since confirmations are no longer shown there and that component be replaced by account history


Remove MergeParts internal telemetry device

The Merge Parts telemetry device -- which relies on a specific logging configuration via rally-teams allowing Lucene index writer trace logging -- is present for historical reasons and can be removed.
This ticket tracks:

[ ] Removal of MergeParts telemetry device and all references to it in the code base (e.g. in reporter.py, launcher.py, telemetry_test.py and possibly other places).
[ ] Remove references in docs

IMPORTANT NOTE: This ticket needs to be dealt with after https://github.com/elastic/rally/pull/743 has been merged.


Ansible issue with Flannel

I believe we resolved this one before. What was the fix again?
TASK: [flannel | configure flannel in etcd] ***********************************
failed: [k8s-master] => {"changed": false, "cmd": ["curl", "-L", "http://k8s-master:4001/v2/keys/cluster.local/network/config", "-XPUT", "--data-urlencode"<EMAIL_ADDRESS>"delta": "0:00:00.235268", "end": "2015-08-02 18:58:00.020974", "rc": 7, "start": "2015-08-02 18:57:59.785706", "warnings": ["Consider using get_url module rather than running curl"]}
stderr:   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0curl: (7) Failed connect to k8s-master:4001; Connection refused
FATAL: all hosts have already failed -- aborting

Strange, really. Not seen after the last time we fixed it. Do you tried second run?

If your hostname does not match the inventory, this will happen. We can create a check in the future.


inconsistency on projects with --no-lockfile

This line is resolving PM wrongly as it first assumes that projects contain lockfile
https://github.com/antfu-collective/package-manager-detector/blob/2c335cd32dbacc01e89216e9eda40b13e616b942/src/index.ts#L25

Can you provide more details? Does it crash or just provide wrong answer? What's happening vs what do you expect to happen?

After a short debugging I think the issue is that the working folder does not contain the lockfile and when using the findUp function it ends up resolving the package manager from parent directories which in my case is not related to the current project.

I'm not sure this is a bug. What would you expect to happen instead?
At least for some package managers like pnpm, the lockfile is contained in an ancestor directory. What package manager are you using?

@benmccann I would expect this library to first exhausted all the strategies (lockfile, packageManager field on package.json) on the working directory and if no luck find on parent directories...


correct RFC3484 address selection caveat

and move it to a more prominent location
Fixes #1710

cc @abuehrle

LGTM

LGTM

This PR was created against master instead of 1.6. I've fixed that by merging the branch into 1.6 and that into master.


JIS (Japanese) Keyboard Layout: Define Keybinding popup window shows wrong key

VSCode Version: 1.7.2
OS Version: macOS Sierra 10.12.1
Keyboard layout: JIS (Japanese)

Steps to Reproduce:

Open keybindings.json
Press ⌘K ⌘K
Some of key combinations are shown as wrong key

Example of wrong combinations:

Pressed
Expected
Actual

@
@
[

Shift+1
!
Shift+1

Shift+:
*
Shift+'

Shift+Cmd+:
Cmd+*
Shift+Cmd+;

@ypresto Starting with today's insiders version (i.e. after PR #22894 aka #17521) we have changed how keybinding dispatching works on OSX from using e.keyCode to using e.code (i.e. scan codes).
Can you please download the insider's version and check if we work now correctly (i.e. labels reflect what you press).
In trying to reproduce, I have installed the Japanese keyboard layout on OS X and tried with both Hiragana and Katakana, but for me, the Keyboard Viewer always seem to reflect the US keys, and things appear to be working correctly. Can you perhaps explain in greater detail what steps I need to take to get this OS X system in the same state as yours. e.g. of the Keyboard Viewer:

My example of Keyboard Viewer:

It reproduces only when I use Japanese IME (even if alphabetical (英字 / eisuu) input is used).
But it does not reproduce when I use U.S. IME.

Using Keyboard Viewer or hardware key does not change result in my env.

@ypresto Thanks to the help of @landonepps (through PR https://github.com/Microsoft/node-native-keymap/pull/8), I believe we now support Japanese IME on JIS keyboards on OSX a lot better.
Sorry for the constant noise, could you please try again using a more recent VS Code version (I think the fix is also in stable 1.12, but best would be if you could check using the Insiders build)
The reason I am asking is because I only have access to ANSI and ISO keyboards, and not to a JIS keyboard, and OSX has very different behaviours on the same input method if different physical keyboard types are used.

@alexandrudima That's great news. I'm currently on vacation without a computer now, but I'll check if it works with a hardware keyboard when I get back.

I just tested all the listed combinations, and I believe this issue can now be closed.
I'm not sure why @ypresto listed the expected value of shift+1 as ! and shift+: as *. The currently displayed values shift+1 and shift+[Quote] match the respective physical key combination, which I think is logical.
@ displays as [BracketLeft] which, while not ideal, does indicate beneath and in a tooltip that for the current keyboard layout the user should press @.

Thank you @landonepps
Indeed, when a key is ambiguous (i.e. differs from the US keyboard layout), we generate scan code based keybindings, so this is expected.

I tried insider version:

Wow! It is working like a charm..!! Thank you for fix!


Live demo and API Documentation into own section.

The live demo and API documentation links get lost at the top of the document. The first expectations in most GitHub repositories is an explanation not links somewhere else.
Also these are important links and not easy to find when scanning the page. Putting them in their own section improves findability.

This change is

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

Thanks for the PR. We generally follow the same concepts as Polymer elements, see e.g.
https://github.com/PolymerElements/paper-button
I agree that description could be the first line, and the links could go after it, but not in the separate section (as it would be eventually more difficult to discover).
Also, please take the following into account when contributing:

Any changes to readme, like this one, should be first submitted to the skeleton repo:
https://github.com/vaadin/vaadin-element-skeleton
Once we discuss it there and decide if we should merge or decline, we will apply change to components.

Any changes to readme and demos should have [skip ci] in the end of the commit message in order to prevent triggering Travis build for nothing.

Closing in favour of vaadin/vaadin-element-skeleton#186


Add Kronecker Product support to sparse dia qarrays

This PR adds the __and__ dunder with 2 methods to calculate the kronecker product between a SparseQArray matrix and a another SparseQArray matrix and between a SparseQArray matrix and a Dense matrix.

Also @pierreguilmin, which type should be output for the dense & sparse operation? Dense? Sparse? Leave it as an option?

Sorry just saw this. Mmmm. Let's do dense by default to move fast, and write smwh to think about it 😋

@gautierronan These tests seem to be failing because when the test matrices are generated, "ghost" values remain in the sparse.diags (ie values that do not exist in the matrix but exist in the diagonals because they are manually initialized.) I would recommend changing _setup from:
def _setup(self):
        N = 10
        num_diags = 3
        diags = jnp.arange(num_diags * N).reshape(num_diags, N)
        offsets = tuple(range(-(num_diags // 2), (num_diags // 2) + 1))

        self.sparseA = dq.SparseDIAQArray(diags=diags, offsets=offsets, dims=(N, N))
        self.sparseB = dq.SparseDIAQArray(diags=diags, offsets=offsets, dims=(N, N))

        self.matrixA = dq.to_dense(self.sparseA)
        self.matrixB = dq.to_dense(self.sparseB)

to
def _setup(self):
        N = 10
        num_diags = 3
        diags = jnp.arange(num_diags * N).reshape(num_diags, N)
        offsets = tuple(range(-(num_diags // 2), (num_diags // 2) + 1))

        self.sparseA = dq.SparseDIAQArray(diags=diags, offsets=offsets, dims=(N, N))
        self.sparseB = dq.SparseDIAQArray(diags=diags, offsets=offsets, dims=(N, N))

        self.matrixA = dq.to_dense(self.sparseA)
        self.matrixB = dq.to_dense(self.sparseB)

        self.sparseA = dq.to_sparse_dia(self.matrixA)
        self.sparseB = dq.to_sparse_dia(self.matrixB)

to wipe out these ghost values.


reversed axis with EDGES gets  *** NOTE: Edges "" do not enclose point on axis Z

Reported by ansley b manke on 7 Jul 2009 18:51 UTC
This is a long-standing bug (going back to the first use of CVS). If the coordinates are reversed, then when storing them, we invert them:
cd_get_1_axis.F
	    IF ( reversed(iaxis) ) THEN
	       tmp = line_delta(iaxis)
	       line_delta(iaxis) =
     .           (line_start(iaxis)-line_delta(iaxis))/(npts-1)
	       line_start(iaxis) = tmp

but then later, rather than the analogous reversing of the ordering of edges (or bounds) values, we just multiply by -1  (!)
* if the coordinates were reversed then the box bounds must be, too
	 IF ( first_delta .LT. 0.0 ) THEN
	    DO 450 i=ipt1+npts-1,(ipt1+npts-1)+npts
 450	    line_mem(i) = -1. * line_mem(i)
	 ENDIF

Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1676

Comment by ansley.b.manke on 10 Jul 2009 19:24 UTC
Fixed in fmt/src/cd_get_1_axis.F


Bug in AddChange button

The AddChange button balances without taking deposits into account now. In ledger mode.

It is not detected.


validates title presence and uniqueness

Going to close this since presence validation was added. We need to handle uniqueness at some point on the server-side and client-side (while people are typing) so if you want to tackle this feel free to update this PR and re-open it.


[Feature Request] Adding subfolder keyword arg to import_from function to allow import from other subfolders than utils

[User story]
One wants to factorize the body of its solvers (resp. datasets) using class inheritance as they all share the same structure and one wants to avoid code duplications.
Thanks to the recent PR #473, one can add the files template_solver.py (resp. template_dataset.py) in the solvers (resp. datasets) folders, in which one might write:
from benchopt import BaseSolver, safe_import_context
with safe_import_context as import_ctx():
    ...

class SolverTemplate(BaseSolver):
    def my_shared_method(self, ...):
    def set_objective(self, ...):
    def run(self, n_iter):

Then in the solvers (resp. datasets) folders one would like to inherit this class to define different solvers:
from benchopt import safe_import_context
with safe_import_context as import_ctx():
    # importing SolverTemplate from solvers folder is required

class Solver(SolverTemplate):
    def specific_method(self, ...):

However, as written in the comment, one needs to first import SolverTemplate. Ideally one would like to use the safe_import_context utils to seamlessly import the template class like:
with safe_import_context as import_ctx():
    TemplateSolver = import_ctx.import_from(
            'template_solver', 'TemplateSolver')
    # fails when template_solver.py is not in utils

However this import works only from the utils folder, which defeats the purpose of filtering over template_ names in solvers (resp. datasets) folders.
Therefore for the moment, if one wants to keep template classes in solvers (resp. datasets) folders, one has to manually use BENCHMARK_DIR variable and copy most of the code from the import_from function (or hack sys.path (!)).
with safe_import_context as import_ctx():
    import sys
    sys.path.append(__file__)
    from template_solver import TemplateSolver

One quick fix would be to allow to specify the subfolder to append to BENCHMARK_DIR in the import_from function (instead of always utils).
Aka:
TemplateSolver = import_ctx.import_from(
        'template_solver', 'TemplateSolver', subfolder='solvers')

TemplateDataset = import_ctx.import_from(
        'template_dataset', 'TemplateDataset', subfolder='datasets')

What do you think ? I can do a quick PR and his would be a backward compatible change with utils as a default argument.
This would allow for numerous other use-cases as well.

Hi @jeandut, as of #472  instead of import_from you can use a regular Python module that is named benchmark_utils, accessible under this name from all solvers and datasets. The folder must be named exactly like that and located in the same folder as objective.py. See here in the template benchmark for an example
We should deprecate the import_from syntax

Thanks @mathurinm I'll make those modifications in my benchmarks that when I have time. One small thing that I regret with this approach is that it is slightly disappointing to not be able to have abstract solver/dataset classes inside the solvers/datasets folders but it is indeed cleaner.

It is true that it is a very frequent pattern to have a abstract solver/dataset class (did it for the last 2 benchmarks I did with deep learning).
I think it is not too complex to do such thing, as we already have the mechanism to create dynamic modules at import.
We would simply need to skip them in the dataset/solver lists (as we do we templates) to have this running.
The important questions are:

Is it really much more convenient than putting this in benchmark_utils. For discovery purpose, I think it might be so I would be +1 on this.
What is the right API/naming for this? base_solver.py/base_dataset.py would probably work? Then from which module name do we import the object? we need to serialize this to make it easy to do.
How to document this well? As we add more features, this makes it not so straightforward to find the right modules.

It seems the original issue is addressed, so can we close this and maybe open a new one for the continuation of #473 (import from a template_solver.py in the solvers/ folder)?
But for simplicity, I lean more on putting everything in the benchmark_utils module


Fixed broken layout on contributors page

What's changed, or what was fixed?

fixed broken layout on contributors page

Fixes: N/A
Target Live Date: N/A

[ ] This has been reviewed and approved by (NAME)
[X] I have run gulp test locally and all tests pass.
[ ] I have added the appropriate type-something label.
[ ] I've staged the site and manually verified that my content displays correctly.

CC: @petele

:tada: This has been pushed live to https://developers.google.com/web/


Improve compatibility with value objects and entity relations

#52 introduced the ability to use an object as default data. However this goes wrong in several cases:
If the returned value is null
Probably detection for this and converting it to an empty string is enough.
If the returned value is a DateTime object
Convert it to a standard 'Y-m-d H:i:s' string?
If the returned value is an object (value object other entity)
Maybe add something like how Symfony handles forms. Set the name to something like 'methodName[method]'. e.g. <input type="text" name="address[street]" />.  This would then execute $defaultData->getAddress()->getStreet();`

Or maybe just revert #52 since it makes it a lot more complicated.

I would suggest rejecting objects completely: it's out of scope for the library, IMO.

Well it prevents a lot of typing. But I already stumbled upon some issues and I'm pretty sure more issues will be reported once others are going to use this library.
I think I'll take your advice and keep it simple.

@xtreamwayz that's the bit handled by hydrators & co - I wouldn't rewrite it. Keep it all arrays for now, and you'll be fine.

Thanx


Data files are missing

Hi!
There is a lot of data files that are missing in the book. I haven't finished it yet but so far these are missing:

C03 data/campaign_data.csv
C05. ../data/superstore.csv
C06 ./data/anomaly_data.csv
C07 '../data/retail_rfm.csv'
C07 ("retail_segmentation.csv")

Hey @gdberrio ! Could you please help us on this?

Hi @scisterna, apologies for the delay. I've pushed a new commit with the data files inside the correct chapter folders.


Duplicate pipeable-operators.md

Description Of The Issue
I've noticed there's pipeable-operators.md file twice in this repo that are almost the same:

docs_app/content/guide/v6/pipeable-operators.md
doc/pipeable-operators.md

I'm not sure however which one is the correct one. Maybe the second one because it was updated more recently.

I see there are actually much more duplicate docs, so this is probably in process of moving from doc/ to docs_app/.

For further clarification: you are absolutely right. There are some duplicates in the doc folder, which used to be used for the old docs.


Correct C++ related makefile variables

FINAL_CXXFLAGS should not inherit CFLAGS because they can contain C
specific options unsupported by C++.
REDIS_CXX should point to the C++ compiler rather than C.

Hi @arkamar
The reason for the original behavior is because there are a bunch of scripts that only set CFLAGS. However this is the right approach.
Let me do a review of CFLAGS before merging.

@arkamar There are quite a few places where only CFLAGS is set and not CXXFLAGS.  I'll merge this and fix in unstable.

Change cda8d42a has additional fixes.

Thanks!


Path.Join missing (string path1, string path2) for .net3.1 same as #774

Hi, love the project, it just gave me a headache this afternoon!
issue #774 still applies for .net Core 3.1
This is no longer a significant issue for me as I have just updated to .net5.0 to workaround the problem but it might occur for others so I thought I should raise an issue:
The fix appears to only apply to FEATURE_PATH_JOIN_WITH_PARAMS which supports the following targets:
Condition="'$(TargetFramework)' == 'net6.0' OR '$(TargetFramework)' == 'net5.0'"
I'm having this issue with .net Core 3.1 but you can see from the link below that it should support Join(string path1, string path2)
https://docs.microsoft.com/en-us/dotnet/api/system.io.path.join?view=netcore-3.1#system-io-path-join(system-string-system-string)
I've raised this as a new issue as I wasn't sure if my comment on the closed issue would get seen, sorry for the duplicate.

Thanks for raising the issue @foz1284 and sorry for the headache 😀
#774 was a different problem where missing methods and smart overload resolution made the library act a bit weird.
The reason why the methods are missing in your case is the following: This library does not provide an explicit target for .NET Core 3.1. When a .NET Core 3.1 project references a library NuGet will pick then "next-best" target, which in our case is .NET Standard 2.0. This framework in turn does not expose the Path.Join overloads, causing the errors you are seeing.
So, even if I understand your confusion, this is actually working as expected. I'll therefore close the issue. Feel free to @ me if you have further questions!

@fgreinacher  thanks for the explanation!


What should happen when logging in a slack channel?

Scenario: In a slack channel, a user initiates a conversation with the AzureBot and, currently, the login link is sent to the channel. Since all joined users receives all messages, a different user could use the link and share their credentials. While this could be useful it is not the expected behavior.
Proposals:

Send Login link in a new private conversation to the initiating user (we need to check if it's doable)

DM'ing people the auth link would be a great option if possible

We have a fix for this which utilizes a browser code method to address it.  Will implement in future version of the AzureBot.


Setup application enforced restrictions for unmanaged devices

It would be useful to add instructions or link on how to actually enable the restrictions on SharePoint and Exchange.

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

ID: 7748fc00-fbc7-df50-de8e-f04839e09a11
Version Independent ID: b3a4e4e4-f291-99d8-c7e5-3c273828a8bb
Content: Conditional Access - Use application enforced restrictions for unmanaged devices - Microsoft Entra
Content Source: articles/active-directory/conditional-access/howto-policy-app-enforced-restriction.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@JMCA2
Thanks for your feedback! We will investigate and update as appropriate.

Hi @JMCA2 As per this document, its already mention Block or limit access to SharePoint, OneDrive, and Exchange content from unmanaged devices.
Please you can contribution same on the feedback. Please add your ideas in below link, so our production team can review it and update the same.
feedback · Community (azure.com)


In Spyder 3.1.3, Ubuntu 16.04, and Python 2.X editor is not working

Description
Hi, I just upgraded Spyder to 3.1.3 through pip.  Now after the upgrade I cannot move my cursor or type anything with my keyboard in the editor.  The Python console is fine
What steps will reproduce the problem?

What is the expected output? What do you see instead?
Please provide any additional information below
Version and main components

Spyder Version: 3.1.3
Python Version: 2.7.12
Qt Versions: 4.8.7, PyQt4 (API v2) 4.11.4 on Linux

Dependencies
pyflakes >=0.5.0 :  1.5.0 (OK)
pep8 >=0.6       :  1.7.0 (OK)
pygments >=2.0   :  2.2.0 (OK)
qtconsole >=4.2.0:  4.2.1 (OK)
nbconvert >=4.0  :  5.1.1 (OK)
pandas >=0.13.1  :  0.19.2 (OK)
numpy >=1.7      :  1.12.0 (OK)
sphinx >=0.6.6   :  1.5.2 (OK)
rope >=0.9.4     :  0.10.3 (OK)
jedi =0.9.0      :  0.9.0 (OK)
psutil >=0.3     :  5.1.3 (OK)
matplotlib >=1.0 :  2.0.0 (OK)
sympy >=0.7.3    :  1.0 (OK)
pylint >=0.25    :  1.6.5 (OK)

Please use Qt/PyQt 5 instead. This clearly seems a Qt bug instead of a Spyder one.
The problem is Qt4 is not maintained anymore, and no new releases are going to be made for it by the company that creates Qt.

Packages for Qt5 are available in Ubuntu since 14.04, I think, so you should be fine.

I am sorry for the lame question.  Actually, I did not have permissions to
write on the files because it was under the 'root' for some reasons.  So,
no problems with Spyder at all.  Sorry for the trouble.
Srini
On Mon, Feb 27, 2017 at 10:50 AM, Carlos Cordoba<EMAIL_ADDRESS>wrote:

Packages for Qt5 are available in Ubuntu since 14.04, I think, so you
should be fine.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/4200#issuecomment-282759036,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKwmg0NocstumJgACDTBBeOednE-JqK_ks5rgvDGgaJpZM4ML3iF
.

Ohh, I see. No worries then ;-)


FFM-7317: adds target expiry section

The only addition is the small section How targets expire, lines 117 - 127 in add-targets.md.
The rest is just some opportunistic cleanup for style/clarity.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Operator crashlooping after upgrade from 0.19.2 to 0.19.3

What steps did you take and what happened:

Upgraded trivy-operator using the helm chart to the latest version (by uninstalling, deleting all crds and installing again)
Operator pod keeps crashlooping

{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"Stopping and waiting for non leader election runnables"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"Stopping and waiting for leader election runnables"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"Shutdown signal received, waiting for all workers to finish","controller":"replicationcontroller","controllerGroup":"","controllerKind":"ReplicationController"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"All workers finished","controller":"replicationcontroller","controllerGroup":"","controllerKind":"ReplicationController"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"Shutdown signal received, waiting for all workers to finish","controller":"vulnerabilityreport","controllerGroup":"aquasecurity.github.io","controllerKind":"VulnerabilityReport"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"All workers finished","controller":"vulnerabilityreport","controllerGroup":"aquasecurity.github.io","controllerKind":"VulnerabilityReport"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"Shutdown signal received, waiting for all workers to finish","controller":"pod","controllerGroup":"","controllerKind":"Pod"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"All workers finished","controller":"pod","controllerGroup":"","controllerKind":"Pod"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"Shutdown signal received, waiting for all workers to finish","controller":"exposedsecretreport","controllerGroup":"aquasecurity.github.io","controllerKind":"ExposedSecretReport"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"All workers finished","controller":"exposedsecretreport","controllerGroup":"aquasecurity.github.io","controllerKind":"ExposedSecretReport"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"Shutdown signal received, waiting for all workers to finish","controller":"job","controllerGroup":"batch","controllerKind":"Job"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"All workers finished","controller":"job","controllerGroup":"batch","controllerKind":"Job"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"Shutdown signal received, waiting for all workers to finish","controller":"statefulset","controllerGroup":"apps","controllerKind":"StatefulSet"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"All workers finished","controller":"statefulset","controllerGroup":"apps","controllerKind":"StatefulSet"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"Shutdown signal received, waiting for all workers to finish","controller":"daemonset","controllerGroup":"apps","controllerKind":"DaemonSet"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"All workers finished","controller":"daemonset","controllerGroup":"apps","controllerKind":"DaemonSet"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"Shutdown signal received, waiting for all workers to finish","controller":"cronjob","controllerGroup":"batch","controllerKind":"CronJob"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"All workers finished","controller":"cronjob","controllerGroup":"batch","controllerKind":"CronJob"}
{"level":"info","ts":"2024-04-04T10:29:02Z","logger":"metrics","msg":"Unregistering resources metrics collector"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"Shutdown signal received, waiting for all workers to finish","controller":"job","controllerGroup":"batch","controllerKind":"Job"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"All workers finished","controller":"job","controllerGroup":"batch","controllerKind":"Job"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"Shutdown signal received, waiting for all workers to finish","controller":"replicaset","controllerGroup":"apps","controllerKind":"ReplicaSet"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"All workers finished","controller":"replicaset","controllerGroup":"apps","controllerKind":"ReplicaSet"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"Stopping and waiting for caches"}
W0404 10:29:02.850172       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.ConfigMap ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from su
cceeding
W0404 10:29:02.850225       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.Deployment ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from s
ucceeding
W0404 10:29:02.850336       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1alpha1.ExposedSecretReport ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the
 request from succeeding
W0404 10:29:02.850374       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1alpha1.SbomReport ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request
from succeeding
W0404 10:29:02.850497       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1alpha1.VulnerabilityReport ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the
 request from succeeding
W0404 10:29:02.850725       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.ReplicationController ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the req
uest from succeeding
W0404 10:29:02.850762       1 reflector.go:462] pkg/mod/k8s.io/client-go@v0.29.3/tools/cache/reflector.go:229: watch of *v1.DaemonSet ended with: an error on the server ("unable to decode an event from the watch stream: context canceled") has prevented the request from su
cceeding
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"Stopping and waiting for webhooks"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"Stopping and waiting for HTTP servers"}
{"level":"info","ts":"2024-04-04T10:29:02Z","logger":"controller-runtime.metrics","msg":"Shutting down metrics server with timeout of 1 minute"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"shutting down server","kind":"health probe","addr":"[::]:9090"}
{"level":"info","ts":"2024-04-04T10:29:02Z","msg":"Wait completed, proceeding to shutdown the manager"}

Events:
  Type     Reason     Age                  From               Message
  ----     ------     ----                 ----               -------
  Normal   Scheduled  11m                  default-scheduler  Successfully assigned mgmt/trivy-operator-7444d5b8cd-92n5v
  Normal   Pulling    11m                  kubelet            Pulling image "ghcr.io/aquasecurity/trivy-operator:0.19.3"
  Normal   Pulled     11m                  kubelet            Successfully pulled image "ghcr.io/aquasecurity/trivy-operator:0.19.3" in 2.533s (2.533s including waiting)
  Warning  Unhealthy  11m                  kubelet            Liveness probe failed: Get "http://<IP_ADDRESS>:9090/healthz/": context deadline exceeded (Client.Timeout exceeded while awaiting headers)
  Warning  Unhealthy  11m                  kubelet            Readiness probe failed: Get "http://<IP_ADDRESS>:9090/readyz/": context deadline exceeded (Client.Timeout exceeded while awaiting headers)
  Warning  Unhealthy  47s                  kubelet            Liveness probe failed: Get "http://<IP_ADDRESS>:9090/healthz/": dial tcp <IP_ADDRESS>:9090: connect: connection refused
  Warning  Unhealthy  47s                  kubelet            Readiness probe failed: Get "http://<IP_ADDRESS>:9090/readyz/": dial tcp <IP_ADDRESS>:9090: connect: connection refused
  Warning  BackOff    37s (x5 over 6m10s)  kubelet            Back-off restarting failed container trivy-operator in pod trivy-operator-7444d5b8cd-92n5v_mgmt(f74c21b0-42f8-41d6-86ab-ebc10ad434bb)

What did you expect to happen:

operator starts and runs like before

Environment:

Trivy-Operator version (use trivy-operator version):

0.19.3

Kubernetes version (use kubectl version):

Client Version: v1.28.4
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.28.3

@szEvEz thanks for reporting creating a fix now


[Checkbox] Special case for  tag to allow clicks

For example:

     <input type="checkbox" ng-model="accept">
     Accept rules

Checkbox's event are blocking url. This should be fixed.

Agreed, that's no good.

Am I missing something or is this not reproducible?
http://jsfiddle.net/efp8z6Ln/175/

@tcigrand You forgot about JS: http://jsfiddle.net/efp8z6Ln/176/

Gotcha, thanks, sorry to muddle up the issue.

Fixed in next


Add ImageResizer to MSIX installer and make the code MSIX-compatible

Summary of the Pull Request
This PR adds ImageResizer to the MSIX package and makes code refactoring to implement the IExplorerCommand interface which is required for MSIX Context menu handlers. There is no perf issue for ImageResizer (as in #1197 ) when PowerRename is commented out in the appmanifest.xml.

PR Checklist

[X] Applies to #53
[X] CLA signed. If not, go over here and sign the CLA
[X] Tests passed

Detailed Description of the Pull Request / Additional comments

Added ImageResizer files and dlls to MSIX packaging layout
Added ImageResizerExt.dll as SurrogateServer (ExeServer can't be used as there is no C++ exe for it).
Added a file explorer context menu entries to the appmanifest for *
Removed the registry key which stores the ImageResizer.exe file location from the MSI installer and made changes to the code so that the location of ImageResizer.exe is identified using the location of the module ImageResizerExt.dll.
Implemented IExplorerCommand interface methods which are required for MSIX.
Modified ResizePictures() method to be reusable for both MSI and MSIX builds.

Observed issues and workarounds

On adding the SurrogateServer, the icon failed to load. This happened because the current module in ContextMenuHandler.cpp is Dllhost.exe, and in the GetIcon function the icon was being retrieved from the current module. This was resolved by pointing to the module HINSTANCE of ImageResizerExt.dll instead.
Adding the ImageResizer fileExplorerContextMenus handler to a particular file type like .png instead of * resulted in very weird menu entry positioning, and it caused the "Open" entry to move down for some reason. A workaround was to set it as * instead of particular file types and filter for image file types within the GetState function in the code.
.png handler

* handler

Open Issues

The drag and drop handler does not appear on MSIX install.
The Resize pictures option does not appear with Rotate left/right (as in #1110 ). Unlikely to be fixable based on investigation for #1038 .

Validation Steps Performed

Verified with multiple MSIX install/uninstalls.
All tests pass
Checked the context menu with png and jpg files and non-image files.
Checked enabling/disabling ImageResizer

@yuyoyuppe @enricogior I have used a SurrogateServer for ImageResizer since there isn't a C++ executable like PowerRename has, and there was relatively less refactoring required for ImageResizer to use with SurrogateServer than ImageResizer. I noticed that I don't have the perf issue (#1197) for ImageResizer (if I comment out the PowerRename lines in the appmanifest). I'm not sure if it is because of using SurrogateServer or if there are any other major differences in how the two extensions work. ImageResizer launches a process with the ImageResizer.exe(.NET) only in the Invoke/InvokeCommand functions. Can you review this PR to make sure there isn't anything missing for MSIX?
I was also able to solve the missing icon problem when using SurrogateServer (as mentioned here in #1197 ). I've described the fix in the PR description under observed issues and workarounds.


Readme references wrong format of pull request title

Reporter: Alex Levenson / @isnotinvain
Assignee: Alex Levenson / @isnotinvain
Note: This issue was originally created as PARQUET-338. Please see the migration documentation for further details.

Alex Levenson / @isnotinvain:
Issue resolved by pull request 244
https://github.com/apache/parquet-mr/pull/244


Pydantic validation error

Hi there,
everytime I run the initial command : python generate.py --base_model='llama' --prompt_type=llama2 --score_model=None --langchain_mode='UserData' --user_path=user_path --model_path_llama=https://huggingface.co/TheBloke/Llama-2-7b-Chat-GGUF/resolve/main/llama-2-7b-chat.Q6_K.gguf --max_seq_len=4096
I'm getting this error :
Using Model llama
0it [00:00, ?it/s]
0it [00:00, ?it/s]
Loaded 0 sources for potentially adding to UserData
Starting get_model: llama
Starting get_model: llama
Starting get_model: llama
Starting get_model: llama
Traceback (most recent call last):
File "/Users/admin/AI/h2ogpt/h2ogpt/generate.py", line 16, in
entrypoint_main()
File "/Users/admin/AI/h2ogpt/h2ogpt/generate.py", line 12, in entrypoint_main
H2O_Fire(main)
File "/Users/admin/AI/h2ogpt/h2ogpt/src/utils.py", line 64, in H2O_Fire
fire.Fire(component=component, command=args)
File "/Users/admin/miniforge3/envs/h2ogpt/lib/python3.10/site-packages/fire/core.py", line 141, in Fire
component_trace = _Fire(component, args, parsed_flag_args, context, name)
File "/Users/admin/miniforge3/envs/h2ogpt/lib/python3.10/site-packages/fire/core.py", line 475, in _Fire
component, remaining_args = _CallAndUpdateTrace(
File "/Users/admin/miniforge3/envs/h2ogpt/lib/python3.10/site-packages/fire/core.py", line 691, in _CallAndUpdateTrace
component = fn(*varargs, **kwargs)
File "/Users/admin/AI/h2ogpt/h2ogpt/src/gen.py", line 1417, in main
model0, tokenizer0, device = get_model_retry(reward_type=False,
File "/Users/admin/AI/h2ogpt/h2ogpt/src/gen.py", line 1765, in get_model_retry
model1, tokenizer1, device1 = get_model(**kwargs)
File "/Users/admin/AI/h2ogpt/h2ogpt/src/gen.py", line 1996, in get_model
model, tokenizer, device = get_model_tokenizer_gpt4all(base_model,
File "/Users/admin/AI/h2ogpt/h2ogpt/src/gpt4all_llm.py", line 27, in get_model_tokenizer_gpt4all
model, tokenizer, redo, max_seq_len = get_llm_gpt4all(**llama_kwargs)
File "/Users/admin/AI/h2ogpt/h2ogpt/src/gpt4all_llm.py", line 175, in get_llm_gpt4all
llm = cls(**model_kwargs)
File "/Users/admin/miniforge3/envs/h2ogpt/lib/python3.10/site-packages/langchain/load/serializable.py", line 97, in init
super().init(**kwargs)
File "pydantic/main.py", line 341, in pydantic.main.BaseModel.init
pydantic.error_wrappers.ValidationError: 1 validation error for H2OLlamaCpp
root
Could not load Llama model from path: https://huggingface.co/TheBloke/Llama-2-7b-Chat-GGUF/resolve/main/llama-2-7b-chat.Q6_K.gguf. Received error Model path does not exist: https://huggingface.co/TheBloke/Llama-2-7b-Chat-GGUF/resolve/main/llama-2-7b-chat.Q6_K.gguf (type=value_error)
The client could be started with python generate.py , but when I want to load model in the interface I get the same error as when initializing. I've tried to completely uninstall a reinstall Homebrew, Conda with envs, the h2opt, pydantic etc., but still getting this error. Any idea what could case the issue?
P.S. The same error occurs with different --model_path_llama values, either HF links, and local paths.
Thanks a lot and have a nice sunday!
T.

Thanks.  I can run your command without error.
If you only had issues with the URL, I'd suspect this block is causing you issues:
https://github.com/h2oai/h2ogpt/blob/f51ffa7079ecfb86ed9ba0fbc47be69b11bfb8b0/src/gpt4all_llm.py#L152-L162
maybe url_alive is not detecting the link as valid, even though it is..
For local paths, I don't have a good idea of what could be wrong.
@Mathanraj-Sharma any chance you can try on your mac?

Thanks a lot for your reply and the whole awesome project. After another complete reinstall of the repo and clean conda environment, it has started to work properly! I suppose there was some collision of the installed packages and / or envs. Sorry for bothering you with my mistake. Take care!


fix: SiderMenu collapse for breakpoint change and make SiderMenu transition works.

The Sider menu still has many problems.  includes the openKeys didn't be cache when collapsed and didn't restore when expanded.I'm also a little busy at my work, but i will try to help to resolve these.

The Sider menu still has many problems.  includes the openKeys didn't be cache when collapsed and didn't restore when expanded.I'm also a little busy at my work, but i will try to help to resolve these.

BaseMenu refactor render functional, this PR has expired
Thank your PR.


hello ravindran accept our pull request

We changed the color of the font size in what is oss team and we changed event description as event description and followed by
our names... Now the website is looking much better than previous

It's a good practice for Pull Request Titles to be descriptive of the issue the pull request solves.


Right killing and quitting doesn't quit Spotify

Just as it sounds. Right clicking on the tray icon on KDE Plasma (using the KDE Systray) doesn't actually kill spotify, spotify-tray. Music continues to play as the process spotify is still running.

Upon closely examining the app's code, it seems somewhere in the code you're forking the process or making a new thread but that thread is never exiting when the main function returns.

I have tested this. If you just start spotify-tray and let it launch spotify everything is correct. The behavior you are describing is if we try to hijack a running Spotify window. Please do not do this. Simply start spotify-tray

That's not true. At least, in my case it isn't. I closed all instances of spotify, then I started spotify-tray and started playing a song. Then I right clicked on the spotify icon in the system tray and chose Quit Spotify. It did not work.

Something weird I noticed is if you run the program from the terminal and then follow all the steps above and then instead of closing from the system tray, you just press Ctrl+C in Terminal, it does properly stop spotify and the music.

I am sorry, but I cannot reproduce behavior you are describing. Everything closes correctly on my computer.


TextBox doesn't bring typed character into view when overflowing

On Windows 10, using .NET Core 3.0, TextBox does not properly scroll the most recently typed character into view when the contents are too long. Pressing <Right Arrow> does not affect anything, and continuing to type will cause the second to last character to come into view. Pressing <Left Arrow> will cause the end of the text to pop back into view.
Update: I tested on Ubuntu 19.04 and it has the same issue there as well, so it does not appear to be platform-specific.
Steps to reproduce:

Create a new project with dotnet new avalonia.app -o aui-bug
Open MainWindow.xaml and replace the default text with <TextBox Height="26" Width="100" />
Run the program
Type in the TextBox until the text reaches the end
Continue typing

Expected result:
When a character is typed, the contents of the TextBox should scroll to the left, with the newly typed character and the carat in view.
Actual result:
When a character is typed right as the contents are about to overflow, the text does not move, the carat moves out of view and the newly typed character is entered "off screen". Subsequent characters cause the text to scroll to the left, but only enough for the second-to-last character to be visible.

I don't think that the TextBox should scroll if you reach the maximum width and height. Only when the text is measured with infinite space you get the expected result.

@gillibald i don't understand how that relates to the issue. Could you explain?

The text layout is limited by the constraint passed to it. When you set the width explicitly the text isn't measured with infinite width and therefore isn't rendered properly. We could ignore the constraint when we have no wrapping.

I only set the width and height in my example to make it easier to reproduce because otherwise the TextBox expands to fill the entire window and you have to enter more text to reach the right side, but the behavior is the same without it.
Standard text box behavior is to keep the caret in view.

The issue is that the constraint is cached. So FormattedText can't expand during it's lifetime. Will have a look how to remove this behavior.

Possible duplicate of  #2723

@Gillibald No, that one is talking about auto-sizing text boxes not resizing properly.

This is definitely a regression. Taking a look.

Problem seems to be here: https://github.com/AvaloniaUI/Avalonia/blob/04e2e460d2887c5262319556e5571fb9a9d59521/src/Avalonia.Controls/Presenters/TextPresenter.cs#L241
The inner HitTest/BringInToView runs before the measure has actually taken place.

Here we go: https://github.com/AvaloniaUI/Avalonia/pull/3075
Simple fix :)


Add healthcheck

Hi, would be nice to have some health endpoint, implemented in https://github.com/open-fresh/avalanche/pull/15

closing, as merged


fsck tool issues

https://github.com/zrlio/crail/blob/master/client/src/main/java/com/ibm/crail/tools/CrailFsck.java#L181

The usage show getLocation whereas the check is on getLocations

https://github.com/zrlio/crail/blob/master/client/src/main/java/com/ibm/crail/tools/CrailFsck.java#L176

Please show more details to tell the user what is a wrong option. Just exiting is non-intuitive and less helpful to a user who might not know what went wrong with the system.

the -r feature is undocumented.

https://github.com/zrlio/crail/blob/master/client/src/main/java/com/ibm/crail/tools/CrailFsck.java#L49

A bit more explanation of commands would be helpful too (as I keep forgetting which one to use or what is the difference between getLocation and blockStatistics).

Fixed in b52952ed3b59752dc4cb27f890009f0131a0b87d


Added Pagination in the footer section of the page

title: Added Pagination in the footer section of the page
issue: #140

I have added a pagination bar of width 7 at the bottom of the page.
It also includes Previous and Next buttons to go to previous and next pages respectively.

@BarmenduC  @Rahul-Palahwat  Please review and merge my PR for Hactoberfest-2022

Hi there is a merge conflict. Please resolve it and then start a pr.

@Raunak173 I have resolved the conflict. Please review and merge it for Hactoberfest-2022

Resolved now Ig


Numerical instability with auc_mu validation and large datasets

How you are using LightGBM?

LightGBM component: Python package
Environment info

Operating System: Fedora 29
CPU/GPU model: Intel(R) Xeon(R) Gold 5120 CPU
C++ compiler version: g++ (GCC) 8.3.1 20190223 (Red Hat 8.3.1-2)
CMake version: 3.14.5
Java version:  OpenJDK Runtime Environment (build 1.8.0_232-b09)
Python version: Python 3.6.10 -- on conda
R version: N/A
Other:

LightGBM version or commit hash: 2.3.2 / 2e2757f183a17c2998c1002531fbbf27d0010783
Error message and / or logs

When using auc_mu metric with a large dataset, validation score varies wildly outside the range [0, 1].
Reported validation metrics on cv were floating around -27, or 4.5 in some cases (atleast for my data).
Running the example described below, this is the first output:
[1]     cv_agg's auc_mu: 14.9488 + 0.0245611
Reproducible example(s)

It only takes two lines:
data = lgb.Dataset(np.random.randn(3000000, 15), np.random.randint(0, 9, 3000000))
lgb.cv({'objective': 'multiclass', 'num_class': 9, 'metric': 'auc_mu'}, data, verbose_eval=True)
Interestingly, running the same code with 10x less data gives the expected result, giving 0.5 auc:
data = lgb.Dataset(np.random.randn(300000, 15), np.random.randint(0, 9, 300000))
lgb.cv({'objective': 'multiclass', 'num_class': 9, 'metric': 'auc_mu'}, data, verbose_eval=True)
yields [1]     cv_agg's auc_mu: 0.499315 + 0.000975809

@btrotta Can you please take a look?

@thadeuluiz Thanks for reporting this and providing a reproducible example. I'll look into it.


Health check startup status lazyness

Currently, the HealthCheckService is started at it's instantiation, that's during the inititalize phase.
Doing so, the first execution of the query runner will always update the cache with a "unhealthy" state, and that's correct, because during the initialization the connector is not yet started.
Unfortunately, also after all the extensions startup we have to wait the whole "startupStatus" period to have that checks updated (by default 60 seconds), until then the connector will be marked as "unhealty"
There's the possibility to reduce that time in configuration, but I think we need a more general approach.
A solution that comes to mind could be to have a method on the HealthCheckService that permits to force the cache update, that could be called by the runtime after completing the startup.
@paullatzelsperger what do you think?

Sounds reasonable. I agree, overloading the query[Liveness|StartupStatus|Readiness] methods would be the lowest hanging fruit here.
Another more invasive way would be to have some sort of listener/observer concept on the *providers, which allows them to initiate a cache update.
Maybe do the first approach first and keep the second approach in mind for when we do the telemetry+metrics stuff (not formally planned yet)?

Yes, let's go with the first approach that will solve the issue for the moment. PR opened


Roll Flutter Engine from b5c52563bbb6 to 6c353718e6d6 (2 revisions)

https://github.com/flutter/engine/compare/b5c52563bbb6...6c353718e6d6
2022-10-28<EMAIL_ADDRESS>Roll Dart SDK from 669d49fc13ea to 32d5fe76c3dc (2 revisions) (flutter/engine#37096)
2022-10-27<EMAIL_ADDRESS>[Impeller] Use the fast rrect blur path for blurred circles/circular shadows (flutter/engine#37094)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

AutoRoller is stopped; closing the active roll.


[FloatingActionButton] Fix Unknown prop 'disabledColor' warning

[ ] PR has tests / docs demo, and is linted.
[ ] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[ ] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

I saw Unknown prop 'disabledColor' warning which this issue talks about https://github.com/callemall/material-ui/issues/4897. I tried to catch that in enhancedButton. It could have been caught in FloatingActionButton also but since it is not being used in the return function, it gave me lint errors. I decided to handle it in enhancedButton by putting disabledColor as the value of backgroundColor in mergedStyle object. This property disabledColor can be found in floatingActionButton. It can also be found in RaisedButton also for reference. This makes the disabledColor property work just fine and removes the warning.

Will make a new PR to fix this issue.


Update tutorial for RBAC

Updated tutorial to reference RBAC permission support for Auth policy admin management of SSPR.

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

@tilarso: I'm sorry - only the author of this article, @justinha, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@Justinha,
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge. Thanks.

#sign-off sorry to miss this!


/ex fakespawn PLAYER

Command:
/ex fakespawn PLAYER <player.location> players: <player>
The output of /version on my server is:
This server is running Paper version git-Paper-379 (MC: 1.18.2) (Implementing API version 1.18.2-R0.1-SNAPSHOT) (Git: 276d830)
The output of /version denizen on my server is:
Denizen version 1.2.5-SNAPSHOT (build 6335-DEV)
A link to a paste of a script demonstrating the issue:
https://paste.denizenscript.com/View/98692
The issue is:
When i execute this command in the game or using script, the answer is the https://paste.denizenscript.com/View/98692

Yet again: support is on Discord @ https://discord.gg/Q6pZGSR


[SPARK-43760][SQL] Nullability of scalar subquery results

What changes were proposed in this pull request?

Backport of https://github.com/apache/spark/pull/41287.
Makes sure that the results of scalar subqueries are declared as nullable.
Why are the changes needed?

This is an existing correctness bug, see https://issues.apache.org/jira/browse/SPARK-43760
Does this PR introduce any user-facing change?

Fixes a correctness issue, so it is user-facing.
How was this patch tested?

Query tests.

@cloud-fan

thanks, merging ton 3.4!

Thank you, @agubichev and @cloud-fan !


The SDK 'Microsoft.NET.Sdk.Web' specified could not be found.

Hi,
I am using Ubuntu 18.04. After a recent update of Visual Studio Code I get: "The SDK 'Microsoft.NET.Sdk.Web' specified could not be found.". How can I resolve this problem, as it prevents Omnisharp from loading the project file correctly.

I had the same  issue, it seems that there is an update to omnisharp that this extension haven't yet publish or using. As a workaround I had to configure omnisharp to use the legacy SDK resolver. There should be a hidden folder on your HOME folder; named ".omnisharp", inside create or modify a file named omnisharp.json and include the following:
{
"MSBuild": {
"UseLegacySdkResolver": true
}
}
This worked for me in the meantime the extension is updated to use the latest omnisharp.

I see the problem was resolved in the latest update. Many thanks to the OmniSharrp team!


After authenticating one user, everyone has access event without providing a token

Hello!

}

Sending the same request without header, from a different device, in a different network, with a different remote ip, just gives the same result.
If I delete the token, the result is right:
$ http POST https://droag.annorath-game.com/interface/request/HTTP/1.1 401 Unauthorized
Allow: OPTIONS, GET
Connection: keep-alive
Content-Type: application/json
Date: Mon, 20 Feb 2017 22:24:56 GMT
Server: nginx/1.10.2
Transfer-Encoding: chunked
WWW-Authenticate: Token

{
    "detail": "Authentication credentials were not provided."
}

What can be the cause of this? I don't get it currently.

Never mind, after I tried to debug it, I got some more strange things happening.  Maybe a note would be good for safety, that caching must be disabled in any cases.