Optimize combine operator to fully utilize threads

Currently in the combine operator all the segments are pre-assigned to the threads. If one thread finishes early, it won't be able to pick up more segments.
This PR makes the threads pick up segments once the previous one is finished

Good one.
cc @jasperjiaguo

Codecov Report

Merging #9387 (81e25f3) into master (45cc5dd) will decrease coverage by 54.40%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master    #9387       +/-   ##
=============================================
- Coverage     69.75%   15.35%   -54.41%
+ Complexity     5095      170     -4925
=============================================
  Files          1885     1832       -53
  Lines        100406    98146     -2260
  Branches      15277    15018      -259
=============================================
- Hits          70042    15073    -54969
- Misses        25414    81933    +56519
+ Partials       4950     1140     -3810

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
?

unittests2
15.35% <0.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...not/core/operator/combine/BaseCombineOperator.java
0.00% <0.00%> (-86.25%)
:arrow_down:

...perator/combine/GroupByOrderByCombineOperator.java
0.00% <0.00%> (-82.66%)
:arrow_down:

...nMaxValueBasedSelectionOrderByCombineOperator.java
0.00% <0.00%> (-72.52%)
:arrow_down:

...reaming/StreamingSelectionOnlyCombineOperator.java
0.00% <0.00%> (-66.00%)
:arrow_down:

...src/main/java/org/apache/pinot/sql/FilterKind.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ain/java/org/apache/pinot/core/data/table/Key.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...in/java/org/apache/pinot/spi/utils/BytesUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/java/org/apache/pinot/core/data/table/Table.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../java/org/apache/pinot/spi/utils/BooleanUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../java/org/apache/pinot/core/data/table/Record.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 1424 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

LGTM


Making standard easier to use

The vision of standard is to save you time in three ways:

No configuration
Automatically format code
Catch style issues & programmer errors early

I think we do a really good job on (3), enforcing style and catching errors. We do a pretty good job on (2) as well, thanks to ESLint's --fix functionality.
But we can do way better at Number 1.
We're pretty far away from the vision of "just do this and you're done" for lots of real-world projects.
$ npm install standard
$ standard

In fact our readme promises "no configuration" and then goes on to do a bit too much explanation about packages to install and flags to pass to get standard to work for your particular codebase. And it's not working so well right now.
Here are some very rough ideas that that might improve the situation. Feedback very much welcome!

Include babel-eslint by default

One less package for Babel users to install
If the user is already using Babel, then their .babelrc is recognized and used automatically
No noticeable difference for non-Babel users (Note: need to verify if performance is same)

Remove --verbose flag and just always include the rule name in the error message

I think users almost always want to know the rule name they can google it to learn more

ESList exposes features that we decided to remove/hide for simplicity. But then users sometimes need these features, so we created additional packages to install as an escape hatch to get those features back. I'm not sure that this is actually simpler than just adding a command line flag for these features. So here's a few ideas:

Build in snazzy so users can add do standard --report=pretty and get pretty output
Build in standard-json so users can do standard --report=json to get JSON output
Build in standard-tap so users can do standard --report=tap to get TAP output

ESLint already supports these internally so it feels kind of strange to have to maintain these packages which just parse output and convert it back.
Note: these flags are not allowing arbitrary rules to be added or changed, so they're not the kind of "configuration" that we want to avoid. These are just command line flags that help users accomplish their goals. I can't see teammates bikeshedding about whether to use --pretty or not, and especially not for --json.

Allow all non-confusing browser globals https://github.com/standard/standard/issues/1330

Eliminate the need to use a window. prefix or a /* global Blah */ comment
Code that uses window. is not usable in a service worker or in Node.js
globalThis is not pretty and not widely available yet anyway.
Confusing browser variables like top and self will still be treated as undefined by default

Build in prettier https://github.com/standard/standard/issues/811

Users who want deterministic formatting on every save can just do standard --format.
Users who want minimal changes to their code, can continue using standard --fix.
Eliminates the need for additional wrapper packages and users to install multiple tools.

Lint JavaScript in Markdown files and HTML files by default https://github.com/standard/standard/issues/1355

All my code examples in README.md files and docs/ folders should be standard, but I never remember to install standard-markdown. We can build this in by default.

The TypeScript experience is completely broken right now. Typescript users need to install an ESLint plugin and configure some rules in order to get standard working. The problem is that you can't configure rules in standard. Indeed, that's the point. So they have to switch to standardx, another escape hatch that we built to handle cases that we don't handle correctly in standard itself.
Users also have to do this to get set up:
npm install @typescript-eslint/parser @typescript-eslint/eslint-plugin --save-dev

Followed by
standard --parser @typescript-eslint/parser --plugin @typescript-eslint/eslint-plugin *.ts

Ugh. This looks like configuration to me. Here's a possible solution:

Support Typescript out of the box https://github.com/standard/standard/issues/1283

Detect .ts files and apply the necessary minimal rule changes so everything works.
Bundle the necessary parser and plugin so it just works.
Then typescript users can just do standard

If we support parsing Typescript and we default to the babel-eslint parser, then we can also:

Remove the --plugin and --parser flags.

No more need for users to understand this whole aspect of ESLint.
If you want to use plugins, different parsers, etc. then you should just use ESLint directly.

(An alternative is to just drop support for Typescript entirely if we can't do a good job of it)

Here's an idea that might be going too far, but if it worked would be really cool.

Support extensions https://github.com/standard/standard/pull/703

Users can do e.g. standard --react or standard --vue and get additional rules for their specific framework. There can be a very small list of possible choices, maintained as shareable configs in the @standard org.
We can provide helpful warnings and bug catching rules to users of e.g. React, etc. that we currently can't do because these rules would apply to everyone.
There is already a concept of an env or an ESLint environment with the --env flag. Maybe this can piggyback on that concept?
So users could do e.g standard --env react --env mocha if they want additional rules for React and globals defined for Mocha tests, as one hypothetical example.

If we make all the changes I've proposed here, we'd go from this help page:
Flags:
        --fix       Automatically fix problems
    -v, --verbose   Show rule names for errors (to ignore specific rules)
        --version   Show current version
    -h, --help      Show usage information

Flags (advanced):
        --stdin     Read file text from stdin
        --global    Declare global variable
        --plugin    Use custom eslint plugin
        --env       Use custom eslint environment
        --parser    Use custom js parser (e.g. babel-eslint)

to this:
Flags:
        --fix       Automatically fix problems
        --format    Aggressively format code for consistency
        --version   Show current version
    -h, --help      Show usage information

Flags (advanced):
        --stdin     Read file text from stdin
        --report    Use custom reporter (choose from: pretty, json, tap)
        --global    Declare global variable
        --env       Use custom environment (choose from: react, vue, mocha, etc.)

If I had to summarize the main idea here I'd say it feels like we've let a lot of the complexity of ESLint leak through, and we can do better. Lots of the details of my proposed solutions might not be the best, but I hope this will re-energize us and get us thinking about solutions.
Eager for feedback!

(An alternative is to just drop support for Typescript entirely if we can't do a good job of it)

Or, if you do decided not to add TypeScript support by default, you might want to consider supporting it as a plugin like you mentioned for react and vue instead?
Note: This should still be an "alternative," it'd be much better if standard supported TypeScript by default with no extra config/installs.

Or, if you decided not to add TypeScript support by default, you might want to consider supporting it as a plugin like you mentioned for react and vue instead?

Yeah, that's a good point. It would fit easily into that model since it's just a plugin, parser, and a few rule tweaks.

:+1: Babel by default.
:+1: Including rule name in message by default and removing --verbose.
:+1: Pretty output by default.
:+1: Built-in JSON and TAP reporters
#1330 seems too large to be yes/no in the context of this issue.
I have a feeling that bundling prettier is not a worthwhile investment. Perhaps making it officially supported and very easy to use along with standard would be wiser. I can imagine a future where ESLint fixes make prettier obsolete. You can say I'm a dreamer. I've never used prettier.
Formatting people's code goes beyond what we do here. If we decide to ever replace prettier with anything else, then the code breakages could be enormous. It's a can of worms, I tell ya :skull_and_crossbones:
:+1: #1355.
:+1: for #703 (I wrote it) but without the need for a CLI flag. Just having the extension package installed should activate it.

I have a feeling that bundling prettier is not a worthwhile investment[...]

Personally, this is one of the things I'm looking forward to the most! I've been in a lot of projects that used to use standard, but has switched to Prettier. It's always a hassle since Prettier is impossible to configure to 100% match standard, so in the end we'll end up adding a prettier config to match as closely as possible, and then either run eslint after (which preserves all the good "catch bug" linting) or just skip standard alltogheter.
If all we needed to do was to add a --format flag to the standard command, that would be amazing and a huge win IMHO 😍

How much larger will the install be with the babel stuff?

This mostly looks like a nice one!

Include babel-eslint by default

Including babel-eslint doesn't seem right for people that do not use Babel (hello, TypeScript...). Maybe just to allow standard check for current project dependencies, and if it sees babel-eslint automatically enable it, without the need to specify --plugin?
That means that the user still has to install babel plugin, but doesn't need to configure it.

Build in prettier #811

Adopting Prettier seems to be the right decision, as the war between eslint, standard and prettier preferences is getting quite annoying.

Lint JavaScript in Markdown files and HTML files by default

nice, but means even more default dependencies...

Support Typescript out of the box

sounds nice, but isn't that more dependencies for non-TypeScript users? Plugins (extensions) still feels better. Just npm install @standard/typescript and standard automatically should pick it up.

(An alternative is to just drop support for Typescript entirely if we can't do a good job of it)

oh no, that will sound like an end of the standard. I've already mentioned that problem — because of the "closed" philosophy, it's risky for anyone to adopt technology which suddenly can become unusable with newer techs just because the user has no way to adjust it to the pipeline. And by saying no to common nowadays techs, we'll show that tech is definitely not viable.

There can be a very small list of possible choices, maintained as shareable configs in the @standard org

The question is, will that list update frequently enough? If for some reason project will stagnate (as that happened a few times, because, well, we all lazy and rely on core maintainers...), it could quickly lose users. That partially happened because of the TypeScript, or Vue, which were unusable with standard for a very long period of time.
Would allowing extension of the standard with a 3rd-party extension be that bad? Technically, they are not that much configuration. Yeap, they might "shift"  defined standards, but I think that ultimately what usually extensions do. Some people suggested to disallow overriding of the current rules and allow the only extension, but there'll be too many times when it won't work. An example — TS plugin — it needs to disable some default standard rules to make it work.
It feels to me that it would be much better if standard would provide the community with means to quickly adapt and extend at times when it needed...

So users could do e.g standard --env react --env mocha if they want additional rules for React and globals defined for Mocha tests, as one hypothetical example.

That concept looks interesting.
It also means that certain features can be invoked ad hoc, in some files, right? For instance, standard can't know that particular .js  file is in fact a React file, but by specifying env comment in the beginning of the file that could be denoted and standard would apply some additional rules to that file. Sounds nice.

How much larger will the install be with the babel stuff?

We'd go from 287 dependencies to 321.
As for the performance of babel-eslint, I'm pleased and quite surprised to say that it actually appears to be faster than the built-in ESLint parser.
I ran time npm run test -- --offline and got 33.350 seconds with babel-eslint vs. 45.248 seconds with the built-in parser.
I say that we ship this for the performance improvement alone!

@feross Maybe also for ease of development you could move standard-engine to this repository?

Also I woudn't complain about adding other packages in monorepo style:
packages
├── ...
├── eslint-config-standard
└── standard-engine

Why? It's hard to coordinate changes in all repositories. For example to add some change in eslint-config-standard I'd need to make request to it, then update standard-engine, then update standard, very tedious. I'd like to help but the fragmentation of standard repositories makes me meh :)

I guess it would be possible to add the --format option without necessarily bundling prettier. I mean, the decision of adding this option shouldn't depend on the decision of bundling prettier. I'd be happy to have built-in prettier, but it would already help me a lot if --fix didn't clash with prettier so much.

For someone that's been using tslint, eslint & prettier separately for a long time I find standard not really easy to use.
I like defaults & presets but I do want the ability to easily customise them. Like dangling commas & semicolons, it's a personal preference but I like them. I found how to turn the default behaviour off that it removes them, but how do I enforce them?
Also is there any VSCode plugin that actually works (with standardx I guess, as I want to customise stuff)?

Could we have globalThis enabled by default without the need to set it in package.json as
"standard": {
  "globals": [ "globalThis" ]
}

@catalin-luntraru that seems like a good move, would you mind opening a separate issue for that, and maybe a PR as well?

@feross same issue, i want to. use standard with vue, but when i run command like standard --parser babel-eslint --plugin vue **/*.vue,vue files still prompt error

Could we have globalThis enabled by default without the need to set it in package.json as

Supporting --env browser would be a much better fix. Each file can currently have /* eslint-env browser */ tacked on at the top but that's not very elegant

@feross


AlreadyExistsError: File system for s3 already registered

System information

OS Platform and Distribution ): windows 10
TensorFlow installed from (source or binary):
TensorFlow version: 2.50
Python version: 3.8
Installed using: Conda

Describe the problem
** Not able to verify the scripts model_builder_tf2_test.py **
VERIFICATION_SCRIPT = os.path.join(paths['APIMODEL_PATH'], 'research', 'object_detection', 'builders', 'model_builder_tf2_test.py')
!python {VERIFICATION_SCRIPT}
tensorflow.python.framework.errors_impl.AlreadyExistsError: File system for s3 already registeredr
2021-11-15 00:16:54.612239: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'cudart64_110.dll'; dlerror: cudart64_110.dll not found
2021-11-15 00:16:54.636703: I tensorflow/stream_executor/cuda/cudart_stub.cc:29] Ignore above cudart dlerror if you do not have a GPU set up on your machine.
Traceback (most recent call last):
File "Tensorflow\models\research\object_detection\builders\model_builder_tf2_test.py", line 25, in
from object_detection.builders import model_builder
File "C:\Users\Sanjay\Anaconda3\envs\tensorflowEnv\lib\site-packages\object_detection-0.1-py3.8.egg\object_detection\builders\model_builder.py", line 37, in
from object_detection.meta_architectures import deepmac_meta_arch
File "C:\Users\sanjay\Anaconda3\envs\tensorflow\lib\site-packages\object_detection-0.1-py3.8.egg\object_detection\meta_architectures\deepmac_meta_arch.py", line 28, in
import tensorflow_io as tfio  # pylint:disable=g-import-not-at-top
File "C:\Users\sanjay\Anaconda3\envs\tensorflowEnv\lib\site-packages\tensorflow_io-0.22.0-py3.8-win-amd64.egg\tensorflow_io_init_.py", line 17, in
from tensorflow_io.python.api import *  # pylint: disable=wildcard-import
File "C:\Users\sanjay\Anaconda3\envs\tensorflowEnv\lib\site-packages\tensorflow_io-0.22.0-py3.8-win-amd64.egg\tensorflow_io\python\api_init_.py", line 19, in
from tensorflow_io.python.ops.io_dataset import IODataset
File "C:\Users\sanjay\Anaconda3\envs\tensorflowEnv\lib\site-packages\tensorflow_io-0.22.0-py3.8-win-amd64.egg\tensorflow_io\python\ops_init_.py", line 96, in
plugin_ops = load_library("libtensorflow_io_plugins.so", "fs")
File "C:\Users\sanjay\Anaconda3\envs\tensorflowEnv\lib\site-packages\tensorflow_io-0.22.0-py3.8-win-amd64.egg\tensorflow_io\python\ops_init.py", line 64, in load_library
l = load_fn(f)
File "C:\Users\sanjay\Anaconda3\envs\tensorflow\lib\site-packages\tensorflow_io-0.22.0-py3.8-win-amd64.egg\tensorflow_io\python\ops_init.py", line 56, in
load_fn = lambda f: tf.experimental.register_filesystem_plugin(f) is None
File "C:\Users\sanjay\Anaconda3\envs\tensorflowEnv\lib\site-packages\tensorflow\python\framework\load_library.py", line 218, in register_filesystem_plugin
py_tf.TF_RegisterFilesystemPlugin(plugin_location)
tensorflow.python.framework.errors_impl.AlreadyExistsError: File system for s3 already registered

@Sanjaygowda66
Could you please try with cuda 11.0 and cudnn 8.0 as i see you have the error.
please refer to similar isues:link,#43193,#50819,#44381.#44291

System information

OS Platform and Distribution ): windows 10
TensorFlow installed from (source or binary):
TensorFlow version: 2.50
Python version: 3.8
Installed using: Conda

Describe the problem
** Not able to verify the scripts model_builder_tf2_test.py **
VERIFICATION_SCRIPT = os.path.join(paths['APIMODEL_PATH'], 'research', 'object_detection', 'builders', 'model_builder_tf2_test.py') !python {VERIFICATION_SCRIPT}
tensorflow.python.framework.errors_impl.AlreadyExistsError: File system for s3 already registeredr
2021-11-15 00:16:54.612239: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'cudart64_110.dll'; dlerror: cudart64_110.dll not found 2021-11-15 00:16:54.636703: I tensorflow/stream_executor/cuda/cudart_stub.cc:29] Ignore above cudart dlerror if you do not have a GPU set up on your machine. Traceback (most recent call last): File "Tensorflow\models\research\object_detection\builders\model_builder_tf2_test.py", line 25, in from object_detection.builders import model_builder File "C:\Users\Sanjay\Anaconda3\envs\tensorflowEnv\lib\site-packages\object_detection-0.1-py3.8.egg\object_detection\builders\model_builder.py", line 37, in from object_detection.meta_architectures import deepmac_meta_arch File "C:\Users\sanjay\Anaconda3\envs\tensorflow\lib\site-packages\object_detection-0.1-py3.8.egg\object_detection\meta_architectures\deepmac_meta_arch.py", line 28, in import tensorflow_io as tfio # pylint:disable=g-import-not-at-top File "C:\Users\sanjay\Anaconda3\envs\tensorflowEnv\lib\site-packages\tensorflow_io-0.22.0-py3.8-win-amd64.egg\tensorflow_io__init__.py", line 17, in from tensorflow_io.python.api import * # pylint: disable=wildcard-import File "C:\Users\sanjay\Anaconda3\envs\tensorflowEnv\lib\site-packages\tensorflow_io-0.22.0-py3.8-win-amd64.egg\tensorflow_io\python\api__init__.py", line 19, in from tensorflow_io.python.ops.io_dataset import IODataset File "C:\Users\sanjay\Anaconda3\envs\tensorflowEnv\lib\site-packages\tensorflow_io-0.22.0-py3.8-win-amd64.egg\tensorflow_io\python\ops__init__.py", line 96, in plugin_ops = load_library("libtensorflow_io_plugins.so", "fs") File "C:\Users\sanjay\Anaconda3\envs\tensorflowEnv\lib\site-packages\tensorflow_io-0.22.0-py3.8-win-amd64.egg\tensorflow_io\python\ops__init_.py", line 64, in load_library l = load_fn(f) File "C:\Users\sanjay\Anaconda3\envs\tensorflow\lib\site-packages\tensorflow_io-0.22.0-py3.8-win-amd64.egg\tensorflow_io\python\ops__init_.py", line 56, in load_fn = lambda f: tf.experimental.register_filesystem_plugin(f) is None File "C:\Users\sanjay\Anaconda3\envs\tensorflowEnv\lib\site-packages\tensorflow\python\framework\load_library.py", line 218, in register_filesystem_plugin py_tf.TF_RegisterFilesystemPlugin(plugin_location) tensorflow.python.framework.errors_impl.AlreadyExistsError: File system for s3 already registered

@Sanjaygowda66 Could you please try with cuda 11.0 and cudnn 8.0 as i see you have the error. please refer to similar isues:link,#43193,#50819,#44381.#44291

I don't have GPU, I'm just loading pre-trained model checkpoints.
I ran the below-mentioned code twice after restarting kernel on jupyter notebook Since then I'm getting S3 already registered error whenever I ran verifying model_builder_tf2_test.py script
And this is how I installed Tensorflow Object detection
if os.name=='nt':
    url="https://github.com/protocolbuffers/protobuf/releases/download/v3.15.6/protoc-3.15.6-win64.zip"
    wget.download(url)
    !move protoc-3.15.6-win64.zip {paths['PROTOC_PATH']}
    !cd {paths['PROTOC_PATH']} && tar -xf protoc-3.15.6-win64.zip
    os.environ['PATH'] += os.pathsep + os.path.abspath(os.path.join(paths['PROTOC_PATH'], 'bin'))
    !cd Tensorflow/models/research && protoc object_detection/protos/*.proto --python_out=. && copy object_detection\\packages\\tf2\\setup.py setup.py && python setup.py build && python setup.py install
    !cd Tensorflow/models/research/slim && pip install -e .

I have similar problem.
OS Platform and Distribution ): windows 10
TensorFlow version: TensorFlow-gpu=2.4.1
Python version: 3.8
Installed using: Conda
cudatoolkit = 11.0.221
cuDNN = 8.0.4 for CUDA 11.0
python object_detection/builders/model_builder_tf2_test.py
2021-11-16 21:55:05.642660: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library cudart64_110.dll
Traceback (most recent call last):
File "object_detection/builders/model_builder_tf2_test.py", line 25, in
from object_detection.builders import model_builder
File "D:\tf_train\models\research\object_detection\builders\model_builder.py", line 37, in
from object_detection.meta_architectures import deepmac_meta_arch
File "D:\tf_train\models\research\object_detection\meta_architectures\deepmac_meta_arch.py", line 28, in
import tensorflow_io as tfio  # pylint:disable=g-import-not-at-top
File "C:\Users\jim\anaconda3\envs\tf2_gpu\lib\site-packages\tensorflow_io_init_.py", line 17, in
from tensorflow_io.python.api import *  # pylint: disable=wildcard-import
File "C:\Users\jim\anaconda3\envs\tf2_gpu\lib\site-packages\tensorflow_io\python\api_init_.py", line 19, in
from tensorflow_io.python.ops.io_dataset import IODataset
File "C:\Users\jim\anaconda3\envs\tf2_gpu\lib\site-packages\tensorflow_io\python\ops_init_.py", line 96, in
plugin_ops = load_library("libtensorflow_io_plugins.so", "fs")
File "C:\Users\jim\anaconda3\envs\tf2_gpu\lib\site-packages\tensorflow_io\python\ops_init.py", line 64, in load_library
l = load_fn(f)
File "C:\Users\jim\anaconda3\envs\tf2_gpu\lib\site-packages\tensorflow_io\python\ops_init.py", line 56, in
load_fn = lambda f: tf.experimental.register_filesystem_plugin(f) is None
File "C:\Users\jim\anaconda3\envs\tf2_gpu\lib\site-packages\tensorflow\python\framework\load_library.py", line 178, in register_filesystem_plugin
py_tf.TF_RegisterFilesystemPlugin(plugin_location)
tensorflow.python.framework.errors_impl.AlreadyExistsError: File system for s3 already registered

@Sanjaygowda66 ,
Can you please try to test your code in latest stable v2.7 and let us know if the issue still persists.Please check this link for tested build configurations.

@Sanjaygowda66 , Can you please try to test your code in latest stable v2.7 and let us know if the issue still persists.Please check this link for tested build configurations.

Thanks for your help, It's working completely fine with v2.7

@Sanjaygowda66 , Can you please try to test your code in latest stable v2.7 and let us know if the issue still persists.Please check this link for tested build configurations.

Thanks for your help, It's working completely fine with v2.7

How did you upgrade it?
I tried doing !pip install --upgrade tensorflow==2.7.0 but its showing version upto 2.6.2 only.
ERROR: No matching distribution found for tensorflow==2.7.0
Current specifications:
tensorflow              2.5.0.dev2021032900
Python                  3.6.5 :: Anaconda
ipykernel               5.5.6
protobuf                3.19.1
tensorflow-addons       0.12.1
tf-estimator-nightly       2.5.0.dev2021032501
pip                                 21.3.1
Keras-Preprocessing           1.1.2
jupyter-client          7.1.0
jupyter-core            4.9.1
Cython                  3.0a6

@Sanjaygowda66 , Can you please try to test your code in latest stable v2.7 and let us know if the issue still persists.Please check this link for tested build configurations.

Thanks for your help, It's working completely fine with v2.7

How did you upgrade it? I tried doing !pip install --upgrade tensorflow==2.7.0 but its showing version upto 2.6.2 only. ERROR: No matching distribution found for tensorflow==2.7.0
Current specifications: tensorflow 2.5.0.dev2021032900 Python 3.6.5 :: Anaconda ipykernel 5.5.6 protobuf 3.19.1 tensorflow-addons 0.12.1 tf-estimator-nightly 2.5.0.dev2021032501 pip 21.3.1 Keras-Preprocessing 1.1.2 jupyter-client 7.1.0 jupyter-core 4.9.1 Cython 3.0a6

Just use !pip install --upgrade tensorflow, it will automatically upgrade to a newer version.

I tried that but it is upgrading to 2.6.2 not 2.7

I tried that but it is upgrading to 2.6.2 not 2.7

Maybe you should upgrade your python >3.7, mine was 3.9 or try reinstalling TensorFlow.

I tried that but it is upgrading to 2.6.2 not 2.7

Maybe you should upgrade your python >3.7, mine was 3.9 or try reinstalling TensorFlow.

Ok.

I tried that but it is upgrading to 2.6.2 not 2.7

Maybe you should upgrade your python >3.7, mine was 3.9 or try reinstalling TensorFlow.

Ok.

System requirements
Python 3.7–3.9
Python 3.9 support requires TensorFlow 2.5 or later.
Python 3.8 support requires TensorFlow 2.2 or later.

I have similar problem. OS Platform and Distribution ): windows 10 TensorFlow version: TensorFlow-gpu=2.4.1 Python version: 3.8 Installed using: Conda cudatoolkit = 11.0.221 cuDNN = 8.0.4 for CUDA 11.0
python object_detection/builders/model_builder_tf2_test.py 2021-11-16 21:55:05.642660: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library cudart64_110.dll Traceback (most recent call last): File "object_detection/builders/model_builder_tf2_test.py", line 25, in from object_detection.builders import model_builder File "D:\tf_train\models\research\object_detection\builders\model_builder.py", line 37, in from object_detection.meta_architectures import deepmac_meta_arch File "D:\tf_train\models\research\object_detection\meta_architectures\deepmac_meta_arch.py", line 28, in import tensorflow_io as tfio # pylint:disable=g-import-not-at-top File "C:\Users\jim\anaconda3\envs\tf2_gpu\lib\site-packages\tensorflow_io__init__.py", line 17, in from tensorflow_io.python.api import * # pylint: disable=wildcard-import File "C:\Users\jim\anaconda3\envs\tf2_gpu\lib\site-packages\tensorflow_io\python\api__init__.py", line 19, in from tensorflow_io.python.ops.io_dataset import IODataset File "C:\Users\jim\anaconda3\envs\tf2_gpu\lib\site-packages\tensorflow_io\python\ops__init__.py", line 96, in plugin_ops = load_library("libtensorflow_io_plugins.so", "fs") File "C:\Users\jim\anaconda3\envs\tf2_gpu\lib\site-packages\tensorflow_io\python\ops__init_.py", line 64, in load_library l = load_fn(f) File "C:\Users\jim\anaconda3\envs\tf2_gpu\lib\site-packages\tensorflow_io\python\ops__init_.py", line 56, in load_fn = lambda f: tf.experimental.register_filesystem_plugin(f) is None File "C:\Users\jim\anaconda3\envs\tf2_gpu\lib\site-packages\tensorflow\python\framework\load_library.py", line 178, in register_filesystem_plugin py_tf.TF_RegisterFilesystemPlugin(plugin_location) tensorflow.python.framework.errors_impl.AlreadyExistsError: File system for s3 already registered

use gpu==2.5.1 i think it works.


Load custom document implementation

Hi @paulocoutinhox I was working on this a while.
There was an error with the callback function signature. Additionally, we need to allocate memory to pass the struct to WebAssembly (WASM). Aside from that, I commented out the document closing because the document needs to remain open, and we also need to expose the removeFunction call. This call should be postponed until we no longer need the function. I'm not very familiar with the sample code, so I'm sure there is a better solution than commenting it out. Good news, it works!
Do you think you can merge the changes exposing addFunction, setValue and removeFunction? Thanks!

Ok, nice, i will make tests and will refactor something if need. Thanks.


mptcpd 0.13

mptcpd now supports ELL 0.68.

Documentation was improved for the "check_route" address
notification flag.

Listening socket creation is now optional for users space path
manager plugins.  Plugins may now call
mptcpd_pm_add_addr_no_listener() instead of mptcpd_pm_add_addr() to
announce a new address without creating a new listener socket.

The script mptcp-get-debug' was added to help simplify information collection for MPTCP related bug reports in general, not just for mptcpd alone.  It is installed in the libexec' directory for a
given Linux distribution, such as `/usr/local/libexec'.  Run it with
the '--help' command line argument to list available options.

A crash (seg fault) that occured if the `/etc/protocols' file does
not exist was fixed.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 64.075%

Totals

Change from base Build<PHONE_NUMBER>6:
0.0%

Covered Lines:
1409

Relevant Lines:
2199

💛  - Coveralls


breaking change when upgrading to argocd-operator:v0.7.0 ("managed-by" not working anymore in newly created namespaces - openshift/okd)

Describe the bug
We upgraded our ArgoCD-operator from v0.5.0 to v0.7.0 and noticed that new namespaces are not able to be managed by the correct ArgoCD-instance anymore when adding
oc label namespace <namespace> \
argocd.argoproj.io/managed-by=<namespace>

ref. this upstream argocd operator documentation and this upstream openshift/okd documentation
example of error message:

Namespace "renovate-bot" for ConfigMap "renovate-config" is not managed

expected behavior:
the following rolebindings should get created automatically when adding the above mentioned label:(this does not happen in 0.7.0)
Role/argocd-application-controller
Role/argocd-grafana
Role/argocd-redis
Role/argocd-server

Screenshots
New namespaces don't show up automatically under "NAMESPACES" when adding the above mentioned label in different namespaces.

Additional info
OKD-version: 4.13.0-0.okd-2023-08-04-164726
k8s-version: v1.26.6+73ac561
When rolling back to v0.5.0, it immediately works again.

@jonasbartho its a label not an annotation

@jonasbartho its a label not an annotation
Sorry, small typo, I fixed the description now. :)

Additional info:
We are running the default "namespace scoped instance" of ArgoCD, we are not running a Cluster Scoped Instance.

Hi @jaideepr97
After some digging we found out the following permission issue in v0.7.0(oc logs -f -c manager <argocd operator pod> -n openshift-operators):
cannot list resource "prometheusrules" in API group "monitoring.coreos.com" at the cluster scope

When we add a custom Clusterrole that remediates the permission issue, it immediately starts working again! :)
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
  name: argocd-operator-promrules-fix
rules:
  - apiGroups:
      - monitoring.coreos.com
    resources:
      - prometheusrules
    verbs:
      - list
      - watch

Shall I or someone else create a PR on this?
Suggested fix is to add
  - apiGroups:
      - monitoring.coreos.com
    resources:
      - prometheusrules
    verbs:
      - list
      - watch

under argocd-operator/config/rbac/role.yaml

This is probably linked to https://github.com/argoproj-labs/argocd-operator/pull/925 ?


Multi colors svg are not included in font.

List of incompatible svg:

Kubernetes
npm
Postman
Flutter
GatsbyJS
Slack

Fixed.


Does this only work with WebPack?

I'm getting an error with the default svelte rollup template and it looks like this packages deps are all WebPack based.

@nemo-omen, looks like rollup and webpack are working with UMD-modules in different ways.
I suggest you trying this, it worked for me:
// router.js

import _Router from 'svelte-easyroute'
const Router = _Router.default

const router = new Router(...)

I will try to fix this in next releases, or add a note in documentation.
Thank you!

@nemo-omen
v2.1.3 is here ( #15), now you can seamlessly use Easyroute with rollup.
Thank you!


ENH: update package version

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #52.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #52

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ noarch: python recipes should usually follow the syntax in our documentation for specifying the Python version.

For the host section of the recipe, you should usually use python {{ python_min }} for the python entry.
For the run section of the recipe, you should usually use python >={{ python_min }} for the python entry.
For the test.requires section of the recipe, you should usually use python {{ python_min }} for the python entry.
If the package requires a newer Python version than the currently supported minimum version on conda-forge, you can override the python_min variable by adding a Jinja2 set statement at the top of your recipe (or using an equivalent context variable for v1 recipes).

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12073699643. Examine the logs at this URL for more detail.


Rendering object at specific react angle or bounding box.

I want to put my object in a bounding box that is taken at run time.

If you have a bounding box in which you'd like your model to fit, the best way to achieve this is to get the collision shape from the ModelRenderble using getCollisionShape.
Next you should be able to cast this to a Box and find the model size from the bounds of the box.
Finally, adjust the scale of the model accordingly so that the collision box fits in the bounding box you are targetting.

For advice and general questions, please use the Sceneform tag on
Stack Overflow: https://stackoverflow.com/questions/tagged/Sceneform
In addition, please see the Sceneform SDK developer documentation and API reference.


Correct all the English in the codebase

Correct all the spellings and English in the codebase - especially around error messages and command line help information

Working on this right now

wow some very interesting English and strange wording in there! Great job correcting it!

There definitely is. Very interesting challenge.

multi language support would be great via i18n - maybe a languages directory with a set of yaml files for each locale - and it defaults to english but you can choose your language as a command line option - sounds like it should be raised as a new issue - I'll raise it

That was exactly my thought. This shard might be our best bet.

Looks good :)

done now


bugfix: exclude sample and system-tests modules from module name check also if edc is loaded as submodule

What this PR changes/adds
Changes how the ModuleNamePlugin excludes samples ans system-tests
Why it does that
To permit EDC to be loaded as submodule
Further notes

Tried to add unit tests, but doing it into the buildSrc folder seems not so straightforward

Linked Issue(s)
Closes #1232
Checklist

[ ] added appropriate tests?
[ ] performed checkstyle check locally?
[ ] added/updated copyright headers?
[ ] documented public classes/methods?
[ ] added/updated relevant documentation?
[ ] added relevant details to the changelog? (skip with label no-changelog)
[ ] formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

Codecov Report

Merging #1234 (0dcb94e) into main (9d0ba10) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #1234   +/-   ##
=========================================
  Coverage     59.14%   59.14%
  Complexity     2824     2824
=========================================
  Files           715      715
  Lines         15691    15691
  Branches       1036     1036
=========================================
  Hits           9281     9281
  Misses         5990     5990
  Partials        420      420

Continue to review full report at Codecov.

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


Uncaught TypeError: editor.getPath is not a function

[Enter steps to reproduce below:]

...
...

Atom Version: 2.3.3
Electron Version: 1.3.13
System: Mac OS X 10.11.6
Thrown From: learn-ide-tree package, v1.0.21
Stack Trace
Uncaught TypeError: editor.getPath is not a function
At /Applications/Learn IDE.app/Contents/Resources/app.asar/node_modules/learn-ide-tree/lib/nsync/initializer.js:69

TypeError: editor.getPath is not a function
    at atom-workspace.onSave (/Applications/Learn IDE.app/Contents/Resources/app.asar/node_modules/learn-ide-tree/lib/nsync/initializer.js:69:19)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Learn IDE.app/Contents/Resources/app.asar/src/command-registry.js:259:29)
    at /Applications/Learn IDE.app/Contents/Resources/app.asar/src/command-registry.js:3:59
    at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/Applications/Learn IDE.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:587:16)
    at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/Applications/Learn IDE.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:382:22)
    at WindowEventHandler.module.exports.WindowEventHandler.handleDocumentKeyEvent (/Applications/Learn IDE.app/Contents/Resources/app.asar/src/window-event-handler.js:106:36)
    at HTMLDocument.<anonymous> (/Applications/Learn IDE.app/Contents/Resources/app.asar/src/window-event-handler.js:3:59)

Commands
     -7:17.5.0 core:move-right (input.hidden-input)
     -7:16.4.0 editor:newline (input.hidden-input)
     -7:09.5.0 core:move-right (input.hidden-input)
     -7:09.2.0 editor:newline (input.hidden-input)
  3x -6:57.3.0 core:backspace (input.hidden-input)
     -6:55 learn-ide:save (input.hidden-input)
     -6:46.9.0 core:copy (input.hidden-input)
     -6:45.6.0 learn-ide:paste (div.terminal)
     -2:48.6.0 editor:newline (input.hidden-input)
     -2:39.8.0 learn-ide:save (input.hidden-input)
  2x -1:46.6.0 editor:newline (input.hidden-input)
     -1:42.6.0 autocomplete-plus:confirm (input.hidden-input)
     -1:41.8.0 learn-ide:save (input.hidden-input)
     -0:47.6.0 core:backspace (input.hidden-input)
     -0:47.1.0 editor:newline (input.hidden-input)
  2x -0:45.7.0 learn-ide:save (input.hidden-input)

Config
{
  "learn-ide-tree": {}
}

Installed Packages
# User
atom-dark-syntax, v0.28.0 (inactive)
atom-dark-ui, v0.53.0 (inactive)
atom-light-syntax, v0.29.0 (inactive)
atom-light-ui, v0.46.0 (inactive)
base16-tomorrow-dark-theme, v1.4.0 (inactive)
base16-tomorrow-light-theme, v1.4.0 (inactive)
one-dark-ui, v1.8.2 (active)
one-light-ui, v1.8.2 (inactive)
one-dark-syntax, v1.6.0 (active)
one-light-syntax, v1.6.0 (inactive)
solarized-dark-syntax, v1.1.1 (inactive)
solarized-light-syntax, v1.1.1 (inactive)
about, v1.7.2 (active)
archive-view, v0.62.0 (active)
autocomplete-atom-api, v0.10.0 (active)
autocomplete-css, v0.14.1 (active)
autocomplete-html, v0.7.2 (active)
autocomplete-plus, v2.33.1 (active)
autocomplete-snippets, v1.11.0 (active)
autoflow, v0.29.0 (inactive)
autosave, v0.23.2 (active)
background-tips, v0.26.1 (active)
bookmarks, v0.43.2 (active)
bracket-matcher, v0.85.1 (active)
command-palette, v0.39.1 (inactive)
deprecation-cop, v0.55.1 (active)
dev-live-reload, v0.47.0 (active)
encoding-selector, v0.22.0 (active)
exception-reporting, v0.40.0 (active)
find-and-replace, v0.204.5 (inactive)
fuzzy-finder, v1.4.0 (active)
git-diff, v1.2.0 (active)
go-to-line, v0.31.2 (inactive)
grammar-selector, v0.48.2 (active)
image-view, v0.60.0 (active)
incompatible-packages, v0.26.1 (active)
keybinding-resolver, v0.35.0 (active)
line-ending-selector, v0.5.1 (active)
link, v0.31.2 (inactive)
markdown-preview, v0.159.1 (active)
metrics, v1.1.2 (active)
notifications, v0.65.1 (active)
open-on-github, v1.2.1 (inactive)
package-generator, v1.0.2 (inactive)
settings-view, v0.244.0 (active)
snippets, v1.0.4 (active)
spell-check, v0.68.5 (active)
status-bar, v1.6.0 (active)
styleguide, v0.48.0 (active)
symbols-view, v0.113.1 (inactive)
tabs, v0.103.1 (active)
timecop, v0.33.2 (active)
update-package-dependencies, v0.10.0 (active)
whitespace, v0.35.0 (active)
wrap-guide, v0.39.0 (active)
language-c, v0.54.0 (active)
language-clojure, v0.22.1 (active)
language-coffee-script, v0.48.1 (active)
language-csharp, v0.13.0 (active)
language-css, v0.40.1 (active)
language-gfm, v0.88.0 (active)
language-git, v0.15.0 (active)
language-go, v0.43.0 (active)
language-html, v0.47.1 (active)
language-hyperlink, v0.16.1 (active)
language-java, v0.24.0 (active)
language-javascript, v0.122.0 (active)
language-json, v0.18.3 (active)
language-less, v0.29.6 (active)
language-make, v0.22.2 (active)
language-mustache, v0.13.0 (active)
language-objective-c, v0.15.1 (active)
language-perl, v0.37.0 (active)
language-php, v0.37.3 (active)
language-property-list, v0.8.0 (active)
language-python, v0.45.1 (active)
language-ruby, v0.70.2 (active)
language-ruby-on-rails, v0.25.1 (active)
language-sass, v0.57.0 (active)
language-shellscript, v0.23.0 (active)
language-source, v0.9.0 (active)
language-sql, v0.25.0 (active)
language-text, v0.7.1 (active)
language-todo, v0.29.1 (active)
language-toml, v0.18.1 (active)
language-xml, v0.34.12 (active)
language-yaml, v0.27.1 (active)
learn-ide, v2.3.3 (active)
learn-ide-tree, v1.0.21 (active)

# Dev
No dev packages

My test would not save and when I try to put learn test, it would give me errors on all of the console.

typeerror: editor.getPath is not a function    at atom-workspace.onSave (C:\Users\chona\AppData\Local\learn_ide\app-2.4.2\resources\app.asar\node_modules\learn-ide-tree\lib\nsync\initializer.js:71:19)    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (C:\Users\chona\AppData\Local\learn_ide\app-2.4.2\resources\app.asar\src\command-registry.js:259:29)    at CommandRegistry.handleCommandEvent (C:\Users\chona\AppData\Local\learn_ide\app-2.4.2\resources\app.asar\src\command-registry.js:3:59)    at CommandRegistry.module.exports.CommandRegistry.dispatch (C:\Users\chona\AppData\Local\learn_ide\app-2.4.2\resources\app.asar\src\command-registry.js:160:19)    at AtomEnvironment.module.exports.AtomEnvironment.dispatchApplicationMenuCommand (C:\Users\chona\AppData\Local\learn_ide\app-2.4.2\resources\app.asar\src\atom-environment.js:1164:28)    at EventEmitter.outerCallback (C:\Users\chona\AppData\Local\learn_ide\app-2.4.2\resources\app.asar\src\application-delegate.js:320:25)    at emitThree (events.js:116:13)    at EventEmitter.emit (events.js:194:7)

My test would not save and gave me an error.

Experiencing the same problem.

http://help.learn.co/the-learn-ide/unable-to-save-on-ide <----- try this


Versioning and multiple users

Hi Ron,
loving that you've moved this across to HACS - got a questions.  I have two accounts set up, one for me, and one for my wife.  I can successfully retrieve data for one account at a time, but not when both are configured.  Not sure if (or how to do it) there needs to be a check for the integration running multiple retrievals consecutively, and offset them by a minute if there are?

Do you still have this issue? Sometimes after login failure it corrects itself.

Do you still have this issue? Sometimes after login failure it corrects itself.

Sometimes it works straight after a restart, but sometimes it can take hours or days to sort itself out, or even not at all.
I'm wondering if you're making multiple calls for different accounts at the same time, based on the log entries indicating multiple logins from the same IP and then denied by Garmin ...

Please try with latest version and open ticket if you still have issues.


Are there any updates to this document?

This information appears to be somewhat dated. Are there any updates to the procedures and process?

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

ID: 7acd6be9-dfb2-9569-ff34-cad061c47cb4
Version Independent ID: 6e0fcc47-fd74-45b2-f8be-d61fb270f186
Content: How to containerize your Azure Service Fabric microservices (preview)
Content Source: articles/service-fabric/service-fabric-services-inside-containers.md
Service: service-fabric
GitHub Login: @anmolah
Microsoft Alias: anmola

@larrywa Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.


Find definition of word in WikDict

I think using the database generated for wikdict-web, querying translation_block, is the way to go. This does not seem to be available in the databases that are provided on the download page.

https://github.com/karlb/wikdict-gen/issues/17

From karlb/wikdict-gen#17:
:

They already are available at https://download.wikdict.com/dictionaries/wdweb/. Be aware that the format of these files is considered an implementation detail and I will not try to keep it stable across different releases.

The code from src/run.py in wikdict-gen seems simpler: https://github.com/karlb/wikdict-gen/blob/master/src/run.py#L10

DBInterface.execute(
	db,
	"SELECT written_rep, trans_list, sense_list
	FROM (
			SELECT DISTINCT written_rep
			FROM search_trans
			WHERE form MATCH :term
		)
		JOIN translation USING (written_rep)
	ORDER BY
		lower(written_rep) LIKE '%'|| lower(:term) ||'%' DESC, length(written_rep),
		lexentry, coalesce(min_sense_num, '99'), importance * translation_score DESC
	LIMIT 5",
	[query]
) |> DataFrame

This (from src/run.py) works reasonably well. I have to manually split sense_list and trans_list, which are delimited by |


Add 'archive' cli option

a researcher may want to take a snapshot of a db at a certain time to preserve the data in whatever state it is in for reference later during a publication. Even if it's slightly wrong, it's useful for reproducibility.
the library should provide an archive command, which allows the user the ability to extract :all: tables from a study and save them in a zip file. The user will have to assume responsibility for storing this file in an appropriate location, and we should warn them as such and ask them to confirm before downloading.

#183


Setup.ps1 operation fails on Installing C++ Build Tools if already installed.

Describe the bug
Setup.ps1 operation fails on Installing C++ Build Tools if already installed.

To Reproduce
Steps to reproduce the behavior:

Run the Setup script '.\scripts\Setup.ps1'
See the failure

Expected behavior
Should be successful.
Additional context
The Visual Studio Enterprise 2022 is already installed on my machine. I recalled that the first time I run the script, a VC Build Tool was installed successfully (It showed in Visual Studio Installer with Visual Studio Enterprise). But I manually deleted the tool as it is duplicated than the Visual Studio Enterprise. I hit the issue after then.

@niuxw-msft could you take a look at this issue?

Had some trouble with this and fixed it. Will be putting up a PR.


EPUB import confformation dialog is too wide (MacOS)

When the window is resized manually, its dimensions snap to fit the dialog's width (and cannot be reduced further)

Related issue (same cause?): https://github.com/readium/readium-desktop/issues/358

It works on Mac os on my side. Have you still the same issue ?

Yes, the issue persists. To reproduce: close Thorium, delete the JSON file that saves the application's window dimensions (/Users/YOU_USERNAME/Library/Application Support/EDRLab.ThoriumReader/windowRectangle.json), and start the app. You will see that the window is too narrow for the import dialog (as per the screenshot I posted in the issue description).

you're right, I invert height and width per default


How to create doctor here in mongo db?

No Data showing when selecting doctors

The front end is for the patient side of the app only. You need to connect the back-end to your own Mongodb database by adding your DATABASE_URL to a config.env file in a similar format bellow.
<EMAIL_ADDRESS>
Then you can create new doctors:

Get their id:

Then user their id to add available appointment times:

Once you've added the doctor and available appointment times to your Mongodb database the patient would then be able to book their appointments.

thnx but it is very basic, my need is we want payment from client while booking appointment

There is no payment gateway set up in the project, so I don't think this is what you're looking for.

can't you add it now?


Doesn't work with the BetterTTV Extensions on Chrome

I noticed it didn't work for me and found out it doesn't work if I have the BetterTTV extension on, just so you know.
(I use Chrome)

could you describe exactly what is not working?(like button not showing, not inverting, etc...) I just added the BetterTTV extension along with this and it works fine for me.

Interesting, It works fine for me. If anyone else has this issue ill reopen it.


[docs] : Missing package manager instructions in CONTRIBUTING.md file.

No explicit instructions about what the package manager should be used for contributing to the project.
There should be instructions on how one should install project dependencies in the root directory before starting the project.

@oliviertassinari please have a look at this issue.


This dependency is unused within the Angel code

I noticed that you have already removed this dependency in your 29-logger-zombies branch, but I cannot do a fresh build without removing it.  If others are having the same problem, then they would also benefit from this change.

The travis build seems to be failing with a dependency conflict that is the same as the one that I was getting before removing the dependency from angel.cabal.  Is there something to force travis CI to refresh the dependency list or have I missed a travis configuration in the project itself?

I don't know what's going on but I just merged that branch since it needed doing anyways. Build is green so I'll be releasing it as 0.5.1 now.

Excellent, thanks.


Move to new barretenberg.js for circuit proofs

After we refactored out BarretenbergWasm, we did some changes with how circuits could do proofs. As it is not currently, we have a TODO to resolve the features needed for actual proofs

This feels really stale, while generally true this isn't useful anymore as we are tracking our gap for real proofs in many other ways


add support for database mirroring sessions

Add support for database mirroring sessions.
Add two new connection string parameters, failoverpartner and failoverport. These new parameters are only used if a partner hasn't already been associated with the connection string.
Similarly to how the Microsoft drivers work, cache the partner of the first successful connection that provides a partner token. (see https://msdn.microsoft.com/en-us/library/ms175484.aspx#InitialConnection)
When a connection to the intial server fails, try the cached partner. If a partner has not yet been cached, use the failoverpartner and failoverport values.
The database mirror partner values provided in the ENVCHANGE token do not include port information, so append an instance name to the partner name before caching the partner so connecting to the partner will trigger service discovery.
This fixes the third scenario on #86.

In future it would be good to implement connection retry mechanism similar to: https://msdn.microsoft.com/en-us/library/ms175484.aspx#RetryAlgorithm

Thanks for merging this. I considered implementing that retry algorithm, but since retries and pooling is managed by database/sql, it seemed like it might not be idiomatic. I could certainly add that, though, if you'd like.

Yeah, I think you are right. Thanks!


Proxy reload Issue

When All proxies are banned it could not reLoad it http://prntscr.com/n7rn1f
I have this option selected also
http://prntscr.com/n7rofx
but still got  http://prntscr.com/n7rn1f  the same issue

I notice this problem only arise when "Max uses per proxy" is selected http://prntscr.com/n7t0qt ...
which never have arised on BlackBullet ver.....

Thank you I will look into it

Fixed here https://github.com/openbullet/openbullet/commit/6ae7c461820739a241c8febd7279130842c16962


ifconfig is deprecated and should be avoided

Some Linux distributions have deprecated the use of ifconfig and route in favor of the software suite iproute2. The command ip should be used instead of ifconfig.

Excellent point! Can we fix headers so CI passes and we can merge?

Sorry for the ignorance, but exactly what's not passing? :(

There is a conflict that needs resolved.. and the CI job failed:
You can click on details next to the travis icon and see that it failed compile because of the damn bootstrap script again.. I think a rebase should do it.. and we are working on #21 and #132 to solve this issue moving forward

Perfect thank you so much


feat(rust): add delete and list commands for kafka-outlet

Included in https://github.com/build-trust/ockam/pull/8220


Fix problem that may appear when including Windows.h before clipper headers

Resolves #601

This PR fixes a Microsoft-specific problem that generally speaking does not exist on other compilers. However, basically the "fix" isn't harmful on the other platforms, so it may be a good idea to incorporate it.
Alternatively, the problem could be addressed by moving the relevant code from headers to the corresponding .cpp files.


 They sync up when an instance's bindings are requested. When binding functions are registered after creating an instance's bindings, the instance's bindings are out of sync until requested again. If they're never requested, they're never synced.
unregister_binding_functions indexes into binding_data, but only checks that its safe to index into binding_functions. When they're out of sync, indexing fails.
This revision checks that it's safe to index into binding_data.
Bugsquad edit: Fixes #28923

Talking about "instance bindings" eventually sounds like doubletalk.

Oh, this fixes #28923.

Thanks!

Cherry-picked for 3.1.2.


Fix #14: Change license to GNU Justified Public License

Resolves #14
License choice was made after thorough review of all aesthetic and legal considerations.

lgtm


Remove argument max_credits from execute and assemble

Summary
In #7409, the parameter max_credits started its deprecation cycle for execute and assemble.
Details and comments
The deprecation of the parameter max_credits for the functions qiskit.compiler.assembler.assemble and qiskit.execute_function.execute raise a warning since 0.20 (released in 2022-03-31). Time to remove it.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.004%) to 84.639%

Files with Coverage Reduction
New Missed Lines
%

qiskit/qobj/pulse_qobj.py
1
76.45%

Totals

Change from base Build<PHONE_NUMBER>:
-0.004%

Covered Lines:
64155

Relevant Lines:
75798

💛  - Coveralls

Elena: it was probably my fault - I likely just lost the PR instead of revisiting it after the last commit.  I've fixed up the merge conflicts now; if you want to take a look to check you're happy I've done that correctly, I'll be able to merge it.


Adds a density operator

Is this pull request associated with an issue(s)?
No
Description
We had a class which represents the physical entity which is a density, but we did not have a density operator. It should be noted we need both. The density operator is needed to form the
physical density via a BraKet object.
TODOs
None. Should be r2g if it passes CI and I remembered to test everything.

🚀 [bumpr] Bumped! New version:v1.3.4 Changes:v1.3.3...v1.3.4


feat(en): Make snapshots applier resilient and process storage log chunks in parallel

What ❔

Modifies DB recovery logic so that it retries on failures.
Parallelizes DB recovery using a semaphore (i.e., similarly to snapshot recovery in some components).

Why ❔

Retries on transient errors reduce the chance of the recovery getting stuck.
Parallelizing DB recovery can speed it up.

Checklist

[x] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[x] Tests for the changes have been added / updated.
[x] Documentation comments have been added / updated.
[x] Code has been formatted via zk fmt and zk lint.
[x] Spellcheck has been run via zk spellcheck.
[x] Linkcheck has been run via zk linkcheck.

@popzxc I've updated the description and linked the PR to a Linear task.

@slowli thank you! Had a second look on the PR and it does seem to be inline with the description.


Weights for brain extraction

Hi!, very nice work!
I'm trying to reproduce the preprocessing steps, but I don't seem to find the weights for the brain extraction: '../weights/brain_extraction_model.pth.tar'. Are they in the repo? Would it be possible to include them in the release?
Thanks a lot!
Simon

Hi thanks for your comment,
Unless @alanqrwang has changed it, the legacy weights for the brain extractor is found in
https://github.com/alanqrwang/keymorph/tree/legacy#decompressing-trained-weights
Best,
Evan

Great! that works, thanks!


Multiprocess not working [Bug]

Describe the bug
If Multiprocessing is enabled the app crashes on iOS.
I think there got something lost while you refactored the library in several steps.
Or is it just the documentation missing an important bit?!
To Reproduce
add MMKV = MMKV.setProcessingMode(ProcessingModes.MULTI_PROCESS)
Expected behavior
Multiprocess should work
Platform Information:

OS: iOS
React Native Version: latest
Library Version: latest

Additional context
<libMMKV.mm:180::+[MMKV mmkvWithID:cryptKey:rootPath:mode:]> Getting a multi-process MMKV [settings] without setting groupDir makes no sense

@IZSkiSurfer Multi process works. See https://github.com/ammarahm-ed/react-native-mmkv-storage/issues/131
It's useful when you want to share same storage instance between Share Extension & App.

According to your documentation you simply call MMKV.setProcessingMode(ProcessingModes.MULTI_PROCESS) [https://rnmmkv.vercel.app/#/loaderclass?id=setprocessingmode]
But if I do so I get that error message. Somehow I need to set that groupDir but I don't find any hint in the documentation nor any way in the current sourcecode to set it - at least not from within the react-native code.
The issue #131 is kinda old and references an older version I guess.

The solution there applies to the latest version. Check the comments.


Cannot load simulation: KeyError

When I want to load the simulation in the session_00 directory, I receive the key error which may be related to info_solver.xml file.
from snek5000 import load; sim = load()
[11/03/21 09:49:03] WARNING  The .../cbox_run_32x32x8_V1.x1.x1._20 __init__.py:98
                             21-11-02_21-02-30/session_00/info_solver.xml file is missing!
---------------------------------------------------------------------------
KeyError                                  Traceback (most recent call last)
/.fsnet/data/legi/calcul9/home/khoubani8a/Dev/fluidsim/lib/fluidsim_core/loader.py in import_module_solver(key, entrypoint_grp)
     41     try:
---> 42         solver = solvers[key].load()
     43     except KeyError:

KeyError: 'session'

During handling of the above exception, another exception occurred:

ValueError                                Traceback (most recent call last)
<ipython-input-1-9bf224db26da> in <module>
----> 1 from snek5000 import load; sim = load()

/.fsnet/data/legi/calcul9/home/khoubani8a/Dev/snek5000/src/snek5000/__init__.py in load_simul(path_dir)
     94     from snek5000.solvers import import_cls_simul, get_solver_short_name
     95
---> 96     Simul = import_cls_simul(get_solver_short_name(path_dir))
     97
     98     # Load parameters

/.fsnet/data/legi/calcul9/home/khoubani8a/Dev/fluidsim/lib/fluidsim_core/loader.py in import_cls_simul(key, entrypoint_grp)
     66         key = key[len(entrypoint_grp) + 1:]
     67
---> 68     solver = import_module_solver(key, entrypoint_grp)
     69     return solver.Simul

/.fsnet/data/legi/calcul9/home/khoubani8a/Dev/fluidsim/lib/fluidsim_core/loader.py in import_module_solver(key, entrypoint_grp)
     42         solver = solvers[key].load()
     43     except KeyError:
---> 44         raise ValueError(
     45             "You have to give a proper solver key. Given: "
     46             f"{key}. Expected one of: {list(solvers)}"

ValueError: You have to give a proper solver key. Given: session. Expected one of: ['kth', 'nek', 'cbox']```

If I move info_solver.xml to the session_00 directory, I receive this error:
ValueError                                Traceback (most recent call last)
<ipython-input-1-9bf224db26da> in <module>
----> 1 from snek5000 import load; sim = load()

/.fsnet/data/legi/calcul9/home/khoubani8a/Dev/snek5000/src/snek5000/__init__.py in load_simul(path_dir)
     99     from snek5000.params import Parameters
    100
--> 101     params = Parameters._load_params_simul(path_dir)
    102
    103     # try:

/.fsnet/data/legi/calcul9/home/khoubani8a/Dev/fluidsim/lib/fluidsim_core/params.py in _load_params_simul(cls, path, only_mpi_rank0)
    107                         params = cls(hdf5_object=h5file["/info_simul/params"])
    108                 else:
--> 109                     raise ValueError
    110
    111         if mpi.nb_proc > 1 and not only_mpi_rank0:

ValueError:

If I move params_simul.xml and info_solver.xml to the session_00 directory, it can load the simulation.

I was trying to load it in session_00 directory. it works in the directory of simualtion.

Feedback is always welcome. Point out if something is unclear. The session mechanism is quite new and some clarification might be needed in the docs.

By the way, shouldn't it be allowed to load a simulation from the session_00 directory?
Also for a directory with more than 1 session, is there a way to load a particular session? Does it make sense?


Add lint checks

links
spelling

Addressed https://github.com/availproject/availproject.github.io/actions/workflows/build.yml


kodi doesn't start and logs [kodi  /usr/bin/kodi: 1: /usr/bin/kodi: fbset: not found]

The following is logged in the kodi log file (restarting results in the same log message)
15.08.19 16:07:52 (+0200)  kodi  /usr/bin/kodi: 1: /usr/bin/kodi: fbset: not found

rebooting through balenaCloud dashboard and manually powering on/off doesn't fix it.

apparently fbset is no longer installed by default.
you can install it by adding fbset in the Docker.template file to the list of packages that must be installed using ...apt-get install -y ...

So installing fbset alone didn't fix the issue.
In the end I manage to fix this issue by

using an older image (=balenalib/%%BALENA_MACHINE_NAME%%-node:12.1-stretch-run-20190511) in Dockerfile.template
removed  apt-get upgrade -y in Dockerfile.template
I also installed fbset but it is not clear if this is really needed.

So it really looks like that the latest image and/or the apt-get upgrade broke the kodi installation.

seems very related to https://forums.balena.io/t/tvservice-command-hangs-on-all-newly-built-images/25446

The final fix was applying what is suggested in the forum link (see previous comment).
In other words I added the below line to my Dockerfile.template
libraspberrypi-bin=1.20180328-1~nokernel1 libraspberrypi0=1.20180328-1~nokernel1 --allow-downgrades -y \


start daemon posh script with RemoteSigned execution policy

the default effective policy is Restricted which prevents the hidden_daemon.ps1 script from running which is started from the scheduled task to run the  daemon
this fixes #3812

... however, the state of the task is now not consistent.


Add Gremlin language syntax

A graph language that would be nice to see in colour.

@PhilStollery
I rewrite what @ldct said in other issues:

We can review and merge PRs with language implementations but won't be able to actually implement all the requested languages.
Please see http://highlightjs.readthedocs.io/en/latest/language-requests.html for more details.

Asked and answered, thanks @gusbemacbe.
Rather than leave these issues open to languish forever (as we have in the past) we're going to start closing them (to clean up the issues list).
I've moved this request to the wish list:
https://github.com/highlightjs/highlight.js/wiki/Grammar-Requests


Fail to install shyaml 0.5.2 on python 3.6 and pip 9.0.1

Hello,
I have an error when I try to install the lastest version:
~/sources/shyaml ((0.5.2))$ python setup.py install --root=pkg --optimize=1
Missing version information: running './autogen.sh'...

Installed /home/sanpi/sources/shyaml/.eggs/d2to1-0.2.12.post1-py3.6.egg
error in setup command: Error parsing /home/sanpi/sources/shyaml/setup.cfg: RecursionError: maximum recursion depth exceeded while calling a Python object
~/sources/shyaml ((0.5.2))$ python setup.py install --root=pkg --optimize=1
running install
running build
running build_scripts
creating build
creating build/scripts-3.6
copying and adjusting shyaml -> build/scripts-3.6
changing mode of build/scripts-3.6/shyaml from 644 to 755
running install_egg_info
running egg_info
creating shyaml.egg-info
writing shyaml.egg-info/PKG-INFO
writing dependency_links to shyaml.egg-info/dependency_links.txt
writing requirements to shyaml.egg-info/requires.txt
writing top-level names to shyaml.egg-info/top_level.txt
writing manifest file 'shyaml.egg-info/SOURCES.txt'
[d2to1] running patched manifest_maker command with extra_files support
reading manifest file 'shyaml.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
warning: no files found matching '*.py' under directory 'src'
writing manifest file 'shyaml.egg-info/SOURCES.txt'
Copying shyaml.egg-info to pkg/usr/lib/python3.6/site-packages/shyaml-0.5.2-py3.6.egg-info
running install_scripts
creating pkg/usr/bin
copying build/scripts-3.6/shyaml -> pkg/usr/bin
changing mode of pkg/usr/bin/shyaml to 755

Running the command twice works :thinking:

Try using pip. It is the much preferred to handle Python code.
BTW shyaml is a single-file software!

OK! I am a diehard Debian user installing python code with pip. I think OS packaging has long release cycles for python code.
Try stealing ideas from the source of https://pypi.org/project/stdeb/

Even pip fails:
$ pip3 install --user https://github.com/0k/shyaml/archive/master.zip
Collecting https://github.com/0k/shyaml/archive/master.zip
  Downloading https://github.com/0k/shyaml/archive/master.zip
     \ 20kB 305kB/s
    Complete output from command python setup.py egg_info:
    autogen.sh: error: Didn't find a git repository (or no tags found).  ``./autogen.sh`` uses git to create changelog and version information.
    It seems that ``./autogen.sh`` couldn't do its job as expected.
    Please try to launch ``./autogen.sh`` manualy, and send the results to the
    maintainer of this package.
    Package will not be installed !
    Missing version information: running './autogen.sh'...

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-rma95y21-build/

But
$ pip3 install --user git+https://github.com/0k/shyaml.git
Collecting git+https://github.com/0k/shyaml.git
  Cloning https://github.com/0k/shyaml.git to /tmp/pip-9fhldnat-build
Collecting pyyaml (from shyaml==0.5.2.dev201804250410)
  Downloading https://files.pythonhosted.org/packages/4a/85/db5a2df477072b2902b0eb892feb37d88ac635d36245a72a6a69b23b383a/PyYAML-3.12.tar.gz (253kB)
    100% |████████████████████████████████| 256kB 4.1MB/s
Building wheels for collected packages: pyyaml
  Running setup.py bdist_wheel for pyyaml ... done
  Stored in directory: /home/viktor/.cache/pip/wheels/03/05/65/bdc14f2c6e09e82ae3e0f13d021e1b6b2481437ea2f207df3f
Successfully built pyyaml
Installing collected packages: pyyaml, shyaml
  Running setup.py install for shyaml ... done
Successfully installed pyyaml-3.12 shyaml-0.5.2.dev201804250410

I have the same error with pip:
$ pip3 install --user git+https://github.com/0k/shyaml.git
Collecting git+https://github.com/0k/shyaml.git
  Cloning https://github.com/0k/shyaml.git to /tmp/pip-tmhdfq3n-build
    Complete output from command python setup.py egg_info:

    Installed /tmp/pip-tmhdfq3n-build/.eggs/d2to1-0.2.12.post1-py3.6.egg
    error in setup command: Error parsing /tmp/pip-tmhdfq3n-build/setup.cfg: RecursionError: maximum recursion depth exceeded while calling a Python object
    Missing version information: running './autogen.sh'...

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-tmhdfq3n-build/
You are using pip version 9.0.1, however version 10.0.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.

archlinux is archlinux territory for me.

It’s just another linux distribution :smiley:
$ python3 --version
Python 3.6.5
$ pip3 --version
pip 9.0.1 from /usr/lib/python3.6/site-packages (python 3.6)

BTW, for now, you should avoid using the zip releases provided by github. They are not functional, and I would be glad to remove them if possible. You can still use direct installation of master branch via:
pip install git+https://github.com/0k/shyaml@master

As a note, I'd like to leverage the release mecanism of github in a hopefully not so distant future to provide safe zips that are effectively usable for installation. This is tracked in #27

BTW, can you confirm me actually that you can't install also older version of shyaml (0.5.1, 0.5.0) with your current toolset ?

Yes I have the same error with shyaml 0.5.0 (I don’t found the 0.5.1 release).

There is a well known issue already identified when combining Python 3.6 and old libraries like setuptools 9.0.1 ... This seems to be your issue.

Ok, we will wait the update of python-pip package.

Is that something you can do ?

I used a virtual env to build the package and that works fine.
Thank you.

Thanks for your confirmation and your feedback.


chore(Data/Matrix): Clean up some API

This adds some missing Matrix.of typecasts, and uses Matrix.submatrix in places where this is more reasonable than adding of.
Matrix.det_permute' is a new lemma that fills the obvious gap that these submatrix restatements emphasize.

Could you retitle this PR to something more specific like "insert missing explicit typecasts"?
LGTM otherwise.
bors d+

bors merge


Updating the readme

Hello,
Would you be open to me submitting a PR for the readme. At the moment, the info spread throughout the issues & multiple pages. I'd like to make it a single page.
Let me know 😄

+1

Feel free :)


math: Add exp and log2 / log10 functions

Hugo already has the natural log function. So it seems like a gap missing out on its inverse, the exp function.
We also add a log2 (binary logarithm) and log10 (decimal logarithm) function which both can be very useful at times. By including explicit functions, template authors do not have to implement change of base themselves. (We do not do change of base but rather use functions provided by the golang math lib.)

I'm closing this. PRs with new template functions/features needs to come via an accepted proposal. You could argue that this somehow "completes" earlier work in this area, but that argument can certainly be used for just about every new feature, and then we have a never ending story.

I don't know what you have in mind with an "accepted proposal" for pure and simple functionality like this ... the proposal is the PR ...


GAROU: MARK OF THE WOLVES [PCSE00958]

M

App summary

App name: ?
App serial: ?
App version: ?

Vita3K summary

Version: v?
Build number: ?
Commit hash: https://github.com/vita3k/vita3k/commit/[commit]
CPU backend:
GPU backend:

Test environment summary

Tested by: ?
OS: Windows 10/macOS/Linux Distro, Kernel Version?
CPU: AMD/Intel?
GPU: AMD/NVIDIA/Intel?
RAM: ? GB

Issues

Screenshots

Log
Recommended labels

A?
B?
C?

Funciona perfectamente

please follow the format and share screenshots of the game working "perfectamente"


There is no license listed

Thanks for the release!  Under what license are you releasing your model?  Unrestricted?  MIT?

I have the same question about https://github.com/shicai/DenseNet-Caffe
Thanks!

They are both released under BSD license.
Thank you.

Thanks for clarifying!


How to upload image in S3 using hono and cloudfareworkers

here is my implementation of code
import { Hono } from "hono";
import { S3Client } from "@aws-sdk/client-s3";
import { HonoS3Storage } from "@hono-storage/s3";
const client = (accessKeyId: string, secretAccessKey: string) =>
new S3Client({
region: "ap-southeast-2",
credentials: {
accessKeyId,
secretAccessKey,
},
});
const storage = new HonoS3Storage({
key: (_, file) =>
${file.originalname}-${new Date().getTime()}.${file.extension},
bucket: "myblogiumbk1",
client: (c) => client(c.env.AWS_ACCESS_KEY_ID, c.env.AWS_SECRET_ACCESS_KEY),
});
userRoute.post('/api/upload', storage.single("file"), async (c) => {
try {
const { file } = await c.req.parseBody();
// Check if file exists and has a name
if (!file ) {
return c.json({ error: "Missing file or filename in request" }, 400);
}
// Upload the file using storage
// ... (your upload logic)
return c.text("Image uploaded successfully!");
} catch (e) {
console.error(e);
return c.json({ error: "Internal server error" }, 500);
}
});
can anyone tell me how to change my code so that I can upload image from when user inputs it and store it in my bucket

Hi, @BandiDhruv
Thank you for raising the issue.
Your code is actually already set up regarding uploading.
The actual uploading process is written in the storage.single middleware, so it actually works on its own.
import { S3Client } from "@aws-sdk/client-s3";
import { HonoS3Storage } from "@hono-storage/s3";
import { Hono } from "hono";

const client = (accessKeyId: string, secretAccessKey: string) =>
  new S3Client({
    region: "ap-southeast-2",
    credentials: {
      accessKeyId,
      secretAccessKey,
    },
  });

const storage = new HonoS3Storage({
  key: (_, file) =>
    `${file.originalname}-${new Date().getTime()}.${file.extension}`,
  bucket: "myblogiumbk1",
  client: (c) => client(c.env.AWS_ACCESS_KEY_ID, c.env.AWS_SECRET_ACCESS_KEY),
});

const userRoute = new Hono();
userRoute.post("/api/upload", storage.single("file"), async (c) => {
  return c.text("Image uploaded successfully!");
});

export default userRoute;


refactor(embed): generate font subset for embed

I generated a subset of our font for the embed tool to minimize the file size and get independent from Google. What do you think @andrew-secret? Right now, the generation of the font is a manual step. If we need to do it more often, then we could also automate it.
Replacing the fonts on the website is out of scope for this pull request.
I also enabled bundle optimization, so we minify our code. The result:
Before:

After:

Ah foo, but now the example project is also minfied :D

Looks good to me expect of one little thing.
the bundle optimazation bundles the css example too 😅

Yeah, need to fix that 😂

Okay so I kicked the optimization, but I would love to introduce some form of optimization in another pull request.


session: remove CFG_WEBSESSION_DIFFERENTIATE_BETWEEN_GUESTS

## CFG_WEBSESSION_DIFFERENTIATE_BETWEEN_GUESTS -- when set to 1, the session
## system allocates the same uid=0 to all guests users regardless of where they
## come from. 0 allocate a unique uid to each guest.
CFG_WEBSESSION_DIFFERENTIATE_BETWEEN_GUESTS = 0

In modern Flask time, if one wants to store some session related data for a guest user, should better do it in a cookie. This functionality had been implemented in the past in order to showcase certain Invenio functionalities (baskets, alerts) to guests users, without requiring them to login.
In modern time it is socially acceptable to require authentication to use user oriented functionalities of an online service.
So this RFC is to propose the removal of such configuration piece (thus simplifying the session/authentication/authorization stack).

The number of user login attemp could be stored in the cookie?

@chokribr cookie is not a safe place. Better solution would be to use Flask-Ratelimiter and use the nickname/email as a key.

:+1:
IIRC INSPIRE was the only installation using this feature (for guest submissions and paper claiming) in recent years.


Headless rendering on ubuntu machine

I'm trying to get visual observations from a headless Ubuntu machine with MUJOCO_GL=egl, and got the following error report:
    AttributeError: 'GLXPlatform' object has no attribute 'EGL'

The same code can run on a desktop ubuntu with monitor without error report. It seems like this is related with the pyopengl package but I'm not sure how to solve this.

Solved. Solution: install PyOpenGL 3.1.4 and PyOpenGL_accelerate 3.1.4 and export MUJOCO_GL=osmesa and export PYOPENGL_PLATFORM=osmesa

You shouldn't have to export PYOPENGL_PLATFORM - this environment variable gets set automatically when you import either the OSMesa or EGL rendering backends (e.g. see here). One possible reason why you might encounter that AttributeError is if PYOPENGL_PLATFORM and MUJOCO_GL are set to mismatching values.

You shouldn't need to set PYOPENGL_PLATFORM explicitly - this environment variable gets set automatically when either of the OSMesa or EGL rendering backends is imported (e.g. see here).

Are you importing modules from OpenGL elsewhere in your code by any chance? That error message suggests that GLXPlatform (which is the default on Linux) has already been imported. My guess is that somewhere in your code you're importing from OpenGL before we've had a chance to set PYOPENGL_PLATFORM=egl inside egl_renderer.py.

For anyone who might run into this issue and find this issue from Googling: make sure you have libegl1 (or libegl-dev) packages installed.

Solved. Solution: install PyOpenGL 3.1.4 and PyOpenGL_accelerate 3.1.4 and export MUJOCO_GL=osmesa and export PYOPENGL_PLATFORM=osmesa
hi , if my code must use pyopengl3.1.0，and I have added "os.environ["PYOPENGL_PLATFORM"] = "osmesa"" before "import pyrender" ，but I get an error that “'GLXPlatform' object has no attribute 'OSMesa'”?could you give me some advice to solve this problem?

hi ， I have tried "os.environ['PYOPENGL_PLATFORM'] = 'egl' ", but get an error "GLError(
err = 12297,
baseOperation = eglCreateContext,
cArguments = (
<OpenGL._opaque.EGLDisplay_pointer object at 0x7f77438fc510>,
<OpenGL._opaque.EGLConfig_pointer object at 0x7f77438fc400>,
<OpenGL._opaque.EGLContext_pointer object at 0x7f7743b5a1e0>,
<OpenGL.arrays.lists.c_int_Array_7 object at 0x7f77438fc950>,
),
result = <OpenGL._opaque.EGLContext_pointer object at 0x7f77438fce18>
)"， could you give me more advice to solve this problem?

Are you importing modules from OpenGL elsewhere in your code by any chance? That error message suggests that GLXPlatform (which is the default on Linux) has already been imported. My guess is that somewhere in your code you're importing from OpenGL before we've had a chance to set PYOPENGL_PLATFORM=egl inside egl_renderer.py.

Thanks! export PYOPENGL_PLATFORM=egl works for me!


Continuous Alchemy poison has the opposite effect

In Continuous Alchemy, if I drink the poison, I loose health.  If an enemy drinks it, he becomes a headhunter and kills me (which he's only supposed to do if he drinks water).  So the level can't be won

Hello, I looked into this and have fixed it. The problem was Omarn was throwing out a second potion when you said "Poison!".
Try it again and let us know if it still doesn't work after a day (so the caching can catch up).

Yup, works for me :)


Multiplicative additive sobo objectives

A mix of multiplicative and additive SOBO strategies for combining >1 objectives:

Structure of torch_utils.py changed: included utility functions to gather objective callables, etc. for additive / multiplicative / and mixed objective
A notbook illustrates the differences between additive, multiplicative and the new, mixed strategy (merging_objectives.ipynb)
SOBO tests for additive / multiplicative and new methods included

Changes on the weight calculation of multiplicative- and the new AdditiveMultiplicative objective:

weights were calculated as exponents: $f = f_1^{w_1} * f_2^{w_2} * ...$
This caused inf/nan errors for negative objectives (e.g. MinimizeObjectives)
The weights from these classes are therefore transformed from [0, 1] to [1, inf). Errors are thrown for weights <1e-8 for these strategies
Extra option on callables for torch_utils.py adapt_weights_to_1_inf is used for this scaling

Still some issues with the test... I take care

The failing xgb test is really strange, I do not know if this really due to your PR or due to some update dependencies. Need to ivestigate it.

It is definitely an issue not related to the PR, I was running a test pipeline again for something branch which was merged in yesterday and it is now failing with exactly the same issue ...

The error stems from a new sklearn version which was released on the 9th of December. Something is going wrong there with XGB ..., have to dig into it. I will review the PR the latest over the weekend. I hope this is fine for you. One comment already, if we use doc strings what we should do everywhere (note to myself), we use google type docstrings. You can configure your vscode by using the autoDocstring plugin to automatically provide you a scaffold for every function that you write.

Thanks Johannes :)
I was wondering about xgboost, too. There is one failed test, besides xgboost. I`ll look into it

ok, now the docstrings are adapted, and only the xgboost is failing.


Surge does not run on Ubuntu 20.04-LTS desktop

Describe the bug
It seems surge need higher version of glibc 2.32 than Ubuntu 20.04-LTS (2.31)
./surge
./surge: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by ./surge)
./apt install libc6
Reading package lists... Done
Building dependency tree
Reading state information... Done
libc6 is already the newest version (2.31-0ubuntu9.1).
libc6 set to manually installed.
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.

I just updated the linux executable on our release page - could you please redownload the file and try again?

I just updated the linux executable on our release page - could you please redownload the file and try again?

Tested the latest Linux executable on Ubuntu 20.04-LTS desktop and it works properly. Issue closed.

Tested the latest Linux executable on Ubuntu 20.04-LTS desktop and it works properly. Issue closed.


Include new set of 2024.1 images of commit 636b66d for codeflare-sdk

Include new set of 2024.1 images of commit 636b66d for codeflare-sdk

set the annotation for the codeflare-sdk 0.21

Follow-up: https://github.com/red-hat-data-services/notebooks/pull/406
Related-to: JIRA: https://issues.redhat.com/browse/RHOAIENG-14201
This is required for 2.14 , so the codeflare SDK 0.21 version is included in 2024.1 image set.

/approve
going to merge this for fixing the blocker.
Please take a look at the issue , when you all get some time.

/cherry-pick rhoai-2.14


How to debug "Invalid HTTP upgrade" issue

let socket = WebSocket(url: URL(string: "wss://api.lionshare.capital")!)
Have tested on HTML it connects correctly, but when I use Starscream, I get the Invalid HTTP upgrade error. Do you have any suggestions on how to debug the issue? Or are there any iOS specific settings / formats it needs to adhere too?
Using Starscream:2.0.3 XCode:8.2.1.

Invalid upgrade comes from the server when it fails to upgrade the HTTP/TCP connection to a WebSocket connection. I would double check your url, your headers and such, so the server has the correct info to properly upgrade the connection. If that looks correct, I would suggest a packet capture to see what the differences, so you can make the needed code changes to match. Also be sure to check your Origin header, as most WebSocket servers will reject any connection that doesn't match.


feat: ADR-010: Posts module

Description
This PR implements the x/posts module as per ADR-010.
Implementation checklist

[x] Types
[x] Keeper
[x] CLI
[x] Simulation tests
[x] Invariants
[ ] spec folder

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[ ] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[ ] followed the guidelines for building modules
[ ] included the necessary unit and integration tests
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed ! in the type prefix if API or client breaking change
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

Thanks @bragaz. I've added the automerge label. As soon as @dadamu confirms it's ready, it will be merged


can not add new apps

I am experiencing the same issue - deleted my Slack app as it would not function anymore and attempted to readd the app, but got the above error message

Same here.

Same here.

Fixed in 2.1.0


Make tokio dependency optional if possible

I tried to use reqwest for making https requests, which means a transitive dependency on hyper, and then this lib for TLS. However despite reqwest's and hyper's tokio dep being optional (I think?), this library appears to unconditionally depend on tokio with default features.
Unfortunately tokio with default features compiles to a binary so big, it will not fit on my embedded device. This is unfortunate, since without it (and using a simpler executor), everything else is pretty lightweight.
I haven't dug very deep into it, but a cursory look suggests that the tokio dependency could be removed or feature gated. Would this be possible?

does reqwest = { version = "0.11.12", default-features = false, features = ["native-tls-crate", "blocking", "json"] } work?
default-features = false remove hyper-tls dependency which seem dead project

Does reqwest = { version = "0.11.12", default-features = false, features = ["native-tls-crate", "blocking", "json"] } work?
Note: remove hyper-tls dependency using default-features = false


Dockerhub login and private repositories

Hi there,
is it possible to login to the docker hub to get private repositories?
Regards,
Sven

This feature is planned, and discussed in https://github.com/kitematic/kitematic/issues/232


Add CLA-Assistant badge

This PR adds the CLA-Assistant badge to the README file.  The CLA-Assistant has (hopefully) been configured for this repo so that all PRs will ensure that the authors have signed the CLA.

 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.

:tada: This PR is included in version 0.8.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


SPFx compatibility for Node.js (LTS) v18

What type of issue is this?
Question
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Target SharePoint environment
SharePoint Online
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[ ] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

browser version
SPFx version
Node.js version
etc

Issue description
Node.js LTS v16 will soon reach end of lifetime (maintenance) when will SPFx be compatibility with Node.js LTS v18?

@matsla - this is one of our top priorities.
Unfortunately, we're a bit behind of schedule but working on the version bump.


[Automation] Manually starting docker if it doesn't come up after kill

Making kill docker plugin failsafe. starting docker manually if it doesn't come up automatically.
Testing done locally.
=== RUN   Test
2017/05/24 22:36:26 START: basic-test.TestVolumeIsolation
2017/05/24 22:36:26 Creating volume [basic_test_volume_1495665386] on VM [<IP_ADDRESS>]
2017/05/24 22:36:33 Checking volume [basic_test_volume_1495665386] availability from VM [<IP_ADDRESS>]
2017/05/24 22:36:33 Checking volume [basic_test_volume_1495665386] availability from VM [<IP_ADDRESS>]
2017/05/24 22:36:34 Destroying volume [basic_test_volume_1495665386]
2017/05/24 22:36:35 END: basic-test.TestVolumeIsolation
2017/05/24 22:36:35 START: basic-test.TestVolumeLifecycle
2017/05/24 22:36:35 Creating volume [basic_test_volume_1495665395] on VM [<IP_ADDRESS>]
2017/05/24 22:36:42 Checking volume [basic_test_volume_1495665395] availability from VM [<IP_ADDRESS>]
2017/05/24 22:36:42 Attaching volume [basic_test_volume_1495665395] on VM [<IP_ADDRESS>]
2017/05/24 22:36:45 Confirming attached status for volume [basic_test_volume_1495665395]
2017/05/24 22:36:48 Removing container [basic_test_container_1495665395] on VM [<IP_ADDRESS>]
2017/05/24 22:36:50 Confirming detached status for volume [basic_test_volume_1495665395]
2017/05/24 22:36:54 Destroying volume [basic_test_volume_1495665395]
2017/05/24 22:36:56 Checking volume [basic_test_volume_1495665395] availability from VM [<IP_ADDRESS>]
2017/05/24 22:36:56 Creating volume [basic_test_volume_1495665395] on VM [<IP_ADDRESS>]
2017/05/24 22:37:03 Checking volume [basic_test_volume_1495665395] availability from VM [<IP_ADDRESS>]
2017/05/24 22:37:03 Attaching volume [basic_test_volume_1495665395] on VM [<IP_ADDRESS>]
2017/05/24 22:37:06 Confirming attached status for volume [basic_test_volume_1495665395]
2017/05/24 22:37:10 Removing container [basic_test_container_1495665395] on VM [<IP_ADDRESS>]
2017/05/24 22:37:11 Confirming detached status for volume [basic_test_volume_1495665395]
2017/05/24 22:37:16 Destroying volume [basic_test_volume_1495665395]
2017/05/24 22:37:17 Checking volume [basic_test_volume_1495665395] availability from VM [<IP_ADDRESS>]
2017/05/24 22:37:18 END: basic-test.TestVolumeLifecycle
2017/05/24 22:37:18 Creating volume [restart_test_volume_1495665438] on VM [<IP_ADDRESS>]
2017/05/24 22:37:24 START: restart_test.TestPluginKill
2017/05/24 22:37:24 Attaching volume [restart_test_volume_1495665438] on VM[<IP_ADDRESS>]
2017/05/24 22:37:27 Confirming attached status for volume [restart_test_volume_1495665438]
2017/05/24 22:37:30 Killing vDVS plugin on VM [<IP_ADDRESS>]
2017/05/24 22:37:32 Sleep for 2 seconds
2017/05/24 22:37:34 Stopping container [restart_test_container_1495665438] on VM [<IP_ADDRESS>]
2017/05/24 22:37:46 Starting container [restart_test_container_1495665438] on VM [<IP_ADDRESS>]
2017/05/24 22:37:48 Stopping container [restart_test_container_1495665438] on VM [<IP_ADDRESS>]
2017/05/24 22:38:00 Confirming detached status for volume [restart_test_volume_1495665438]
2017/05/24 22:38:04 END: restart_test.TestPluginKill
2017/05/24 22:38:04 Removing container [restart_test_container_1495665438] on VM [<IP_ADDRESS>]
2017/05/24 22:38:05 Destroying volume [restart_test_volume_1495665438]
2017/05/24 22:38:06 Creating volume [restart_test_volume_1495665486] on VM [<IP_ADDRESS>]
2017/05/24 22:38:12 START: restart_test.TestVolumeStaleMount
2017/05/24 22:38:12 Attaching volume [restart_test_volume_1495665486] on VM [<IP_ADDRESS>]
2017/05/24 22:38:15 Confirming attached status for volume [restart_test_volume_1495665486]
2017/05/24 22:38:19 Killing docker on VM [<IP_ADDRESS>]
2017/05/24 22:38:20 Sleep for 2 seconds
2017/05/24 22:38:52 Sleep for 2 seconds
2017/05/24 22:38:54 Confirming detached status for volume [restart_test_volume_1495665486]
2017/05/24 22:39:07 END: restart_test.TestVolumeStaleMount
2017/05/24 22:39:07 Removing container [restart_test_container_1495665486] on VM [<IP_ADDRESS>]
2017/05/24 22:39:08 Destroying volume [restart_test_volume_1495665486]
2017/05/24 22:39:09 Finding VM name from IP Address [<IP_ADDRESS>]
2017/05/24 22:39:10 START: Test vmdkops service restart + kill vm process
2017/05/24 22:39:10 Creating volume [vmlistener_test_volume_1495665550] on VM [<IP_ADDRESS>]
2017/05/24 22:39:16 Attaching volume [vmlistener_test_volume_1495665550] on VM [<IP_ADDRESS>]
2017/05/24 22:39:19 Confirming attached status for volume [vmlistener_test_volume_1495665550]
2017/05/24 22:39:23 Restarting vmdkops service on ESX host [<IP_ADDRESS>]
2017/05/24 22:39:27 Confirming attached status for volume [vmlistener_test_volume_1495665550]
2017/05/24 22:39:32 Retrieving VM power state for [ubuntu]
2017/05/24 22:39:33 VM[ubuntu]'s current power state is [poweredOn]
2017/05/24 22:39:33 Retrieving World ID for VM [ubuntu] from ESX [<IP_ADDRESS>]
2017/05/24 22:39:34 VM's process ID is: 3780454
2017/05/24 22:39:34 Killing VM from ESX [<IP_ADDRESS>]
2017/05/24 22:39:35 Sleep for 10 seconds
2017/05/24 22:39:45 Retrieving VM power state for [ubuntu]
2017/05/24 22:39:46 VM[ubuntu]'s current power state is [poweredOff]
2017/05/24 22:39:49 Powering on VM [ubuntu]
2017/05/24 22:39:51 Sleep for 35 seconds
2017/05/24 22:40:26 Retrieving VM power state for [ubuntu]
2017/05/24 22:40:27 VM[ubuntu]'s current power state is [poweredOn]
2017/05/24 22:40:27 Confirming detached status for volume [vmlistener_test_volume_1495665550]
2017/05/24 22:40:36 END: Test vmdkops service restart + kill vm process
2017/05/24 22:40:36 Destroying volume [vmlistener_test_volume_1495665550]
--- PASS: Test (251.17s)

// CC @shuklanirdesh82

@shuklanirdesh82 ran into this on his local testbed. I verified the issue.

Yeah I am experiencing the issue.
Get Outlook for iOShttps://aka.ms/o0ukef
On Wed, May 24, 2017 at 4:31 PM -0700, "Sam Chen" <notifications<EMAIL_ADDRESS>wrote:
@shaominchen commented on this pull request.
Have we really run into this scenario, i.e. docker doesn't come up after kill, or we are making this change just to be safe?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_vmware_docker-2Dvolume-2Dvsphere_pull_1293-23pullrequestreview-2D40176329&d=DwMCaQ&c=uilaK90D4TOVoH58JNXRgQ&r=RjGBQIdeDa3xg2DqcCaG27fjbAl8yq3UmvzPcj6TUig&m=lGb9nwgLooI-6qDh_W3RnwOlfGa0vfF9Eg6ebiF9VT4&s=fekttu8E3ghgdSywBFn5nJWC2D47PRffYcs3YwqTQ6Y&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AV7Hul4ls-2Dpi4SeqK9-2DdCjgX-5FuILiZXZks5r9L27gaJpZM4NlxDj&d=DwMCaQ&c=uilaK90D4TOVoH58JNXRgQ&r=RjGBQIdeDa3xg2DqcCaG27fjbAl8yq3UmvzPcj6TUig&m=lGb9nwgLooI-6qDh_W3RnwOlfGa0vfF9Eg6ebiF9VT4&s=7iprZio6RxDtWR8IPI-ksn2OW1mdAiEd6y1CB8C7pXg&e=.