Introduce prettier workflow

Based on our slack discussion the goal of this is to introduce a workflow that runs prettier on each PR/push. I added an extensive prettierignore file, that we can gradually reduce by files that have been linted.

I agree with @TylerHelmuth that it should be easy for contributors to run Prettier on single files as well as over the entire repo (as is being done in the GH action).

I was so sure that I commented on this issue, anways, I just tried the following with another PR:
https://github.com/open-telemetry/opentelemetry.io/pull/2191/commits/b964a43550c4c296655ea238a4b73962491c75ed
Using that would allow us to force the use of prettier at some point, wdyt?

Using that would allow us to force the use of prettier at some point, wdyt?

Yes, prettier-ignore-* feature is just what we need! Note that sometimes there is a way to format markdown pages containing shortcodes so that Prettier plays well, but not always. This ignore feature will allow us to cover the cases where it doesn't play well.

I didn't create a .prettierignore file, because I wanted to have this seperated from people manually running npx prettier -w on their file.

Eventually we should only ignore files that we really don't want to have checked/reformatted. In which case, having the ignore file at the top level shouldn't prevent users from running it on valid markdown files, etc. My 2 cents.
This is a great start! I'd vote to keep it simple, but get something committed. We can then iteratively improve the checking.

For an alternative base config and setup, see the following PR that won't require a separate script:

#2286


Add more debugging information to RavenFactory.ravenInstance.

#183
Used to provide this:
[WARNING]
java.lang.reflect.InvocationTargetException
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.codehaus.mojo.exec.ExecJavaMojo$1.run(ExecJavaMojo.java:293)
	at java.lang.Thread.run(Thread.java:745)
Caused by: java.lang.IllegalStateException: Couldn't create a raven instance for 'Dsn{uri=http://localhost:9999/}'
	at com.getsentry.raven.RavenFactory.ravenInstance(RavenFactory.java:100)
	at com.getsentry.example.Application.main(Application.java:16)
	... 6 more

Now provides output like this:
[WARNING]
java.lang.reflect.InvocationTargetException
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.codehaus.mojo.exec.ExecJavaMojo$1.run(ExecJavaMojo.java:293)
	at java.lang.Thread.run(Thread.java:745)
Caused by: java.lang.IllegalStateException: Couldn't create a raven instance for: 'Dsn{uri=http://localhost:9999/}'; ravenFactoryName: com.getsentry.example.TestFactory; skipped factories: com.getsentry.raven.DefaultRavenFactory; tried factories: com.getsentry.example.TestFactory; cause contains exception thrown by the last factory tried.
	at com.getsentry.raven.RavenFactory.ravenInstance(RavenFactory.java:148)
	at com.getsentry.example.Application.main(Application.java:16)
	... 6 more
Caused by: java.lang.ArithmeticException: / by zero
	at com.getsentry.example.TestFactory.createRavenInstance(TestFactory.java:10)
	at com.getsentry.raven.RavenFactory.ravenInstance(RavenFactory.java:103)
	... 7 more

This change is

Reviewed 1 of 2 files at r1.
Review status: 1 of 2 files reviewed at latest revision, 2 unresolved discussions.

raven/src/main/java/com/getsentry/raven/RavenFactory.java, line 102 [r1] (raw file):

            logger.debug("Attempting to use '{}' as a Raven factory.", ravenFactory);
            try {
                triedFactories.add(name);

Does this need to be in the try block?

raven/src/main/java/com/getsentry/raven/RavenFactory.java, line 112 [r1] (raw file):

        }

        // Throw an IllegalStateException that attempts to be helpful.

:)

Comments from Reviewable

Reviewed 1 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: 1 of 2 files reviewed at latest revision, 1 unresolved discussion.

raven/src/main/java/com/getsentry/raven/RavenFactory.java, line 102 [r1] (raw file):
Previously, tkaemming (ted kaemming) wrote…
> Does this need to be in the `try` block?

Whoops, nope.

Comments from Reviewable

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks broke.

Comments from Reviewable


docs: update README.md for better explanation

Since I got people asking about SDK's usage, and I generally feel that the explanation provided by us (including Golang version and Website's docs) is quite ambiguous, I feel the necessity to add more explanations about the usage and working procedure of Casdoor.
Signed-off-by: ffyuanda<EMAIL_ADDRESS>

@ffyuanda can you also improve the docs at: https://casdoor.org/docs/overview ?

@hsluoyz sure I can do that

@ffyuanda great! Plz make PR to: https://github.com/casdoor/casdoor-website . Merged here.


Add fault injection with 1-byte reads

Closes #86

@vinipsmaker I made a patch that add a fault injection errinj_1byte_read that limits amount of data on every read() by a single byte. Could you check that fault injection works for you?

Could you check that fault injection works for you?

I think I can give it a try by next month.
Thanks for the patch. It should help one of my projects a lot.

I think I can give it a try by next month.

Good, I will wait for your feedback.

@vinipsmaker How to check the new fault injection with 1-byte read:
$ git clone https://github.com/ligurio/unreliablefs/
$ git checkout ligurio/gh-86
$ cd unreliablesfs
$ cmake .
$ make -j
$ mkdir source target
$ ./unreliablefs -basedir=./source target/
$ cat << EOF > target/unreliablefs.conf
[errinj_1byte_read]
op_regexp = .*
path_regexp = .*
probability = 80
EOF
$ umount source

It's not working at all.
The read() syscall returns -1 and errno is set to EINTR.
That's the log from unreliablefs:
errinj_1byte_read triggered on operation 'getattr', /home/vinipsmaker/Projetos/tabjson/test.BAK/jpat_changes.in
start of 1-byte read
errinj_1byte_read triggered on operation 'open', /home/vinipsmaker/Projetos/tabjson/test.BAK/jpat_changes.in
start of 1-byte read
errinj_1byte_read triggered on operation 'read', /home/vinipsmaker/Projetos/tabjson/test.BAK/jpat_changes.in
start of 1-byte read
errinj_1byte_read triggered on operation 'read', /home/vinipsmaker/Projetos/tabjson/test.BAK/jpat_changes.in
start of 1-byte read
errinj_1byte_read triggered on operation 'flush', /home/vinipsmaker/Projetos/tabjson/test.BAK/jpat_changes.in
start of 1-byte read
errinj_1byte_read triggered on operation 'lock', /home/vinipsmaker/Projetos/tabjson/test.BAK/jpat_changes.in
start of 1-byte read
errinj_1byte_read triggered on operation 'release', /home/vinipsmaker/Projetos/tabjson/test.BAK/jpat_changes.in
start of 1-byte read

Before it hits my call to read() the host for my plugin (gawk) performs some additional simple calls. My plugin controls the calls to read() so that's the part that matters to me.

It's not working at all.
The read() syscall returns -1 and errno is set to EINTR.

Thanks. I'll take a look.


Fetching files on requests cause timeout

Hi,
I started locally an instance and run a generation request from the documentation. However I receive a timeout that seems to be caused by the fact that the request timeout from the hugging face CDN.
Pulling 15 files take times, is it possible to warmup the server and download all this before? Or keep the download instead of dropping them.
Thanks!
Docker command
docker run -it --rm \
  --gpus all  \
  -p 3000:8000 \
  -e HF_AUTH_TOKEN="XXX" \
  -e AWS_ACCESS_KEY_ID="$AWS_ACCESS_KEY_ID" \
  -e AWS_SECRET_ACCESS_KEY="$AWS_SECRET_ACCESS_KEY" \
  -e AWS_DEFAULT_REGION="$AWS_DEFAULT_REGION" \
  -e AWS_S3_ENDPOINT_URL="$AWS_S3_ENDPOINT_URL" \
  -e AWS_S3_DEFAULT_BUCKET="$AWS_S3_DEFAULT_BUCKET" \
  -v ~/root-cache:/root/.cache \
  "$@" gadicc/diffusers-api:latest

Request
{
  "modelInputs": {
    "prompt": "Super dog",
    "num_inference_steps": 50,
    "guidance_scale": 7.5,
    "width": 512,
    "height": 512,
    "seed":<PHONE_NUMBER>
  },
  "callInputs": {
    "MODEL_ID": "runwayml/stable-diffusion-v1-5",
    "PIPELINE": "StableDiffusionPipeline",
    "SCHEDULER": "LMSDiscreteScheduler",
    "safety_checker": true
  }
}

Server log

{
  "modelInputs": {
    "prompt": "Super dog",
    "num_inference_steps": 50,
    "guidance_scale": 7.5,
    "width": 512,
    "height": 512,
    "seed":<PHONE_NUMBER>
  },
  "callInputs": {
    "MODEL_ID": "runwayml/stable-diffusion-v1-5",
    "PIPELINE": "StableDiffusionPipeline",
    "SCHEDULER": "LMSDiscreteScheduler",
    "safety_checker": true
  }
}
download_model {'model_url': None, 'model_id': 'runwayml/stable-diffusion-v1-5', 'model_revision': None, 'hf_model_id': None}
loadModel {'model_id': 'runwayml/stable-diffusion-v1-5', 'load': False, 'precision': None, 'revision': None}
Downloading model: runwayml/stable-diffusion-v1-5
Fetching 15 files:   0%|                                                                                | 0/15 [00:00<?, ?it/s]
Downloading (…)"model.safetensors";:   0%|                                                                                | 0.00/1.22G [00:00<?, ?B/s]
Downloading (…)_model.safetensors";:   0%|                                                                                | 0.00/335M [00:00<?, ?B/s]
Downloading (…)_model.safetensors";:   0%|                                                                                | 0.00/3.44G [00:00<?, ?B/s]
Downloading (…)"model.safetensors";:   0%|                                                                                | 0.00/492M [00:00<?, ?B/s]

Downloading (…)"model.safetensors";:   2%|██▎                                                                                | 21.0M/1.22G [02:04<1:56:43, 171kB/s]
Downloading (…)_model.safetensors";:   3%|████▎                                                                                | 10.5M/335M [01:49<47:32, 114kB/s]
Downloading (…)_model.safetensors";:   0%|▍                                                                                | 10.5M/3.44G [01:38<7:39:24, 124kB/s]

Fetching 15 files:  20%|███████████████████████████████▊                                                                                | 3/15 [02:59<11:58, 59.84s/it]
Downloading (…)"model.safetensors";:   2%|██▎                                                                                | 21.0M/1.22G [02:58<2:49:42, 117kB/s]
Downloading (…)"model.safetensors";:   2%|██▊                                                                                | 10.5M/492M [02:57<2:16:15, 58.9kB/s]
Downloading (…)_model.safetensors";:   0%|▍                                                                                | 10.5M/3.44G [02:58<16:10:00, 58.9kB/s]
Downloading (…)_model.safetensors";:   6%|████████▌                                                                                | 21.0M/335M [02:58<44:24, 118kB/s]
[2023-01-31 16:17:45 +0000] - (sanic.access)[INFO][<IP_ADDRESS>:59646]: POST http://localhost:8000/  200 5470

Postman side
{
    "$error": {
        "code": "APP_INFERENCE_ERROR",
        "name": "ConnectionError",
        "message": "HTTPSConnectionPool(host='cdn-lfs.huggingface.co', port=443): Read timed out.",
        "stack": "Traceback (most recent call last):\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/urllib3/response.py\", line 444, in _error_catcher\n    yield\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/urllib3/response.py\", line 567, in read\n    data = self._fp_read(amt) if not fp_closed else b\"\"\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/urllib3/response.py\", line 533, in _fp_read\n    return self._fp.read(amt) if amt is not None else self._fp.read()\n  File \"/opt/conda/envs/xformers/lib/python3.9/http/client.py\", line 463, in read\n    n = self.readinto(b)\n  File \"/opt/conda/envs/xformers/lib/python3.9/http/client.py\", line 507, in readinto\n    n = self.fp.readinto(b)\n  File \"/opt/conda/envs/xformers/lib/python3.9/socket.py\", line 704, in readinto\n    return self._sock.recv_into(b)\n  File \"/opt/conda/envs/xformers/lib/python3.9/ssl.py\", line 1242, in recv_into\n    return self.read(nbytes, buffer)\n  File \"/opt/conda/envs/xformers/lib/python3.9/ssl.py\", line 1100, in read\n    return self._sslobj.read(len, buffer)\nsocket.timeout: The read operation timed out\n\nDuring handling of the above exception, another exception occurred:\n\nTraceback (most recent call last):\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/requests/models.py\", line 816, in generate\n    yield from self.raw.stream(chunk_size, decode_content=True)\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/urllib3/response.py\", line 628, in stream\n    data = self.read(amt=amt, decode_content=decode_content)\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/urllib3/response.py\", line 593, in read\n    raise IncompleteRead(self._fp_bytes_read, self.length_remaining)\n  File \"/opt/conda/envs/xformers/lib/python3.9/contextlib.py\", line 137, in __exit__\n    self.gen.throw(typ, value, traceback)\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/urllib3/response.py\", line 449, in _error_catcher\n    raise ReadTimeoutError(self._pool, None, \"Read timed out.\")\nurllib3.exceptions.ReadTimeoutError: HTTPSConnectionPool(host='cdn-lfs.huggingface.co', port=443): Read timed out.\n\nDuring handling of the above exception, another exception occurred:\n\nTraceback (most recent call last):\n  File \"/api/server.py\", line 39, in inference\n    output = user_src.inference(model_inputs)\n  File \"/api/app.py\", line 178, in inference\n    download_model(\n  File \"/api/download.py\", line 148, in download_model\n    loadModel(\n  File \"/api/loadModel.py\", line 59, in loadModel\n    model = pipeline.from_pretrained(\n  File \"/api/diffusers/src/diffusers/pipelines/pipeline_utils.py\", line 524, in from_pretrained\n    cached_folder = snapshot_download(\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/huggingface_hub/utils/_validators.py\", line 124, in _inner_fn\n    return fn(*args, **kwargs)\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/huggingface_hub/_snapshot_download.py\", line 215, in snapshot_download\n    thread_map(\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/tqdm/contrib/concurrent.py\", line 94, in thread_map\n    return _executor_map(ThreadPoolExecutor, fn, *iterables, **tqdm_kwargs)\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/tqdm/contrib/concurrent.py\", line 76, in _executor_map\n    return list(tqdm_class(ex.map(fn, *iterables, **map_args), **kwargs))\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/tqdm/std.py\", line 1195, in __iter__\n    for obj in iterable:\n  File \"/opt/conda/envs/xformers/lib/python3.9/concurrent/futures/_base.py\", line 609, in result_iterator\n    yield fs.pop().result()\n  File \"/opt/conda/envs/xformers/lib/python3.9/concurrent/futures/_base.py\", line 446, in result\n    return self.__get_result()\n  File \"/opt/conda/envs/xformers/lib/python3.9/concurrent/futures/_base.py\", line 391, in __get_result\n    raise self._exception\n  File \"/opt/conda/envs/xformers/lib/python3.9/concurrent/futures/thread.py\", line 58, in run\n    result = self.fn(*self.args, **self.kwargs)\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/huggingface_hub/_snapshot_download.py\", line 194, in _inner_hf_hub_download\n    return hf_hub_download(\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/huggingface_hub/utils/_validators.py\", line 124, in _inner_fn\n    return fn(*args, **kwargs)\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/huggingface_hub/file_download.py\", line 1282, in hf_hub_download\n    http_get(\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/huggingface_hub/file_download.py\", line 530, in http_get\n    for chunk in r.iter_content(chunk_size=10 * 1024 * 1024):\n  File \"/opt/conda/envs/xformers/lib/python3.9/site-packages/requests/models.py\", line 822, in generate\n    raise ConnectionError(e)\nrequests.exceptions.ConnectionError: HTTPSConnectionPool(host='cdn-lfs.huggingface.co', port=443): Read timed out.\n"
    }
}

Hey @AllanElleuch, thanks for the detailed report.
Yes, the way you have it there with the volume setup (-v ~/root-cache:/root/.cache), it will only download the model once (and you'll be able to see them in ~/root-cache on the host system).  Not sure what's up with HuggingFace, maybe they were / are having some temporary issues.
It's also possible to bundle the model with the container (as in https://github.com/kiri-art/docker-diffusers-api-build-download), but that's usually only useful for serverless, since you're able to persist the volume on your local machine.
But either way, you still have to download the model once (but only once).
If HuggingFace is playing up, you could try add the following callInput:
MODEL_URL="https://pub-bdad4fdd97ac4830945e90ed16298864.r2.dev/diffusers/models--runwayml--stable-diffusion-v1-5--fp16.tar.zst"
and it should download from kiri's public R2 storage (rate-limited).
Let me know how that goes.

Thanks Gadicc for the detailed feedback.
It seems that the issue was an unstable connection on my side. Now it is working perfectly fine and I was able to download all the files without an issue!

Fantastic, thanks for reporting back, @AllanElleuch.  Happy Diffusing 🧨 :tada:


Catalog manager register_schema should supports create_if_not_exists

What type of enhancement is this?
API improvement
What does the enhancement do?
The register_schema function in CatalogManager always returns true even when the schema already exists.
I think it should support create_if_not_exists option too, because create database if not exists [db] statement need it.
Implementation challenges
No response

I'm interested in this issues and do some tests!
mysql> create database test_db_schema;
Query OK, 1 row affected (26.31 sec)

mysql> create database test_db_schema;
ERROR 1815 (HY000): Failed to execute sql statement, source: Failed to register a new schema, source: Schema test_db_schema already exists

The tests showed that register_schema function in CatalogManager returns SchemaExistsSnafu when the schema already exists.

I'm interested in this issues and do some tests!

Sorry it's my mistake 🙇♂ @MichaelScofield is working on this, and we forget to assign it to you after your reply. I'll read notifications more carefully. Thanks for your enthusiasm and wish you still enjoy developing with us.


Support for environment variables for mutant runs

Summary
Feature Request: I'd like to support adding environment variables for the mutant runners to use.  For my specific situation, I needed to add the variable NODE_PATH on my command line and I tried to do that by running NODE_PATH=... stryker run.
When I used the command to execute my tests, the initial test run worked exactly as expected, but during the mutant runs ALL of my tests failed because the path wasn't set correctly in the runners.  The result was that stryker thought ALL of my mutants were killed and I got 100% coverage.  This is not true at all.  I proved this by hardcoding path resolution into my test suite to not rely on the NODE_PATH and got much more realistic mutation results.   I can't leave the hardcoded paths because it needs to be dynamic in my use case.
Uncovered Bug:  I would expect the mutation runners to be working in the same environment as the initial test run is in.  In order to have a correct base line, all conditions need to be the same, including the environment variables.

Stryker config

module.exports = function (config) {
    config.set({
        files: [
            { pattern: 'XXX/**/*.js', mutated: true, included: false },
            { pattern: 'test/**/*.js', included: true, mutated: false },
            { pattern: 'XXX/**/*.json', included: false, mutated: false },
            { pattern: 'test/**/*.json', included: false, mutated: false },
        ],
        testRunner: 'mocha',
        mutator: 'javascript',
        transpilers: [],
        reporter: ['html', 'clear-text', 'progress'],
        testFramework: 'mocha',
        coverageAnalysis: 'all',
        thresholds: { high: 80, low: 60, break: 60 },
        htmlReporter: { baseDir: 'build/mutation' }
    });
};

Stryker environment

+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>+--<EMAIL_ADDRESS>
Your Environment

software
version(s)

node
v6.11.3

npm
5.8.0

Operating System
MacOS 10.12.6

@CrazyBS Thanks for making us aware of this! I find it quite strange because Stryker runs all test runs in the same type of sandbox so the environment should be the same. So I would expect Stryker to either always fail or never fail.
Adding environment variables to the stryker config sounds like a good idea. What are your thoughts on this @stryker-mutator/stryker-core ?

When I used the command to execute my tests, the initial test run worked exactly as expected, but during the mutant runs ALL of my tests failed because the path wasn't set correctly in the runners.

That would be an actual bug. The path should be the same in all test runner processes.

I would expect the mutation runners to be working in the same environment as the initial test run is in. In order to have a correct base line, all conditions need to be the same, including the environment variables.

I totally agree. Apart from code coverage and source-map settings, everything should be the same.

@CrazyBS I've tried to reproduce it (first I had to read up on the NODE_PATH environment variable, I was clueless you could do something like that).
See this repository: https://github.com/nicojs/stryker-node_path-test I don't have any issue with NODE_PATH. When using NODE_PATH=./shared stryker run everything works fine.
git clone git@github.com:nicojs/stryker-node_path-test.git
cd stryker-node_path-test
npm i
npm t

Running the tests with mocha and running stryker with the mocha test runner both work as expected. Could you try to reproduce your issues in this repo? Thanks a lot!

@CrazyBS in order to keep our issues tidy, we will close this in 2 weeks if we don't hear from you.

We're closing it for now. Please create a new issue if the problem persists.


Code Release Please

Hello! Waiting for your code for so long, when are you planning to release it ?

+1

Sorry for all , due to some reasons, I promise that the code and model will be released before 2024.02.08.

Sorry for all , due to some reasons, I promise that the code and model will be released before 2024.02.08.

Thanks, waiting for it.

@TXH-mercury thank you. Very glad to see your great work being published.

@TXH-mercury Another dll flag is comming :), but happy new year! Look forward to your release!

Happy new year! Is the code-release in progress? Looking forward!

When will it be released?

@dragen1860 @pribadihcr @lzhangbj @YajieW99
Sorry for the late update!
Overcoming all restrictions, VAST are successfully released now! Please leave message if you meet any problems when using this codebase. Thank you

Hi. Thanks for releasing the code. Can you please share the link to weights using google drive?
Thanks

Hi, google drive links have been updated now.


Version 2.x doesn't read gradle.properties file

On version 1.9.9 this works just fine. when i upgrade to 2.1.0 on task execution (not only build) it cannot read gradle.properties file, making unrecognizable other commands such .\gradlew gm --name update
Version 2.1.0 and 2.0.0:
> Task :gm FAILED
Clean Architecture plugin version: 2.1.0
Model Name: Update

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':gm'.
> java.io.IOException: No parameter language in gradle.properties file

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

Version 1.9.9:
> Task :gm
Clean Architecture plugin version: 1.9.9
Model Name: Update
No parameter lombok in gradle.properties file
WARN: variable lombok not present, if your project use lombok please add lombok=true to gradle.properties and relaunch this task
Applying changes
Changes successfully applied

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

You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.

See https://docs.gradle.org/7.1.1/userguide/command_line_interface.html#sec:command_line_warnings

BUILD SUCCESSFUL in 2s
1 actionable task: 1 executed

The problem is something different if you create the project with some version least than 2.x the file gradle.properties doesnot contain the languaje property, so you can solve the issue append this property in your project.
.
In a later release we try to improve the task update Project to append the missing properties


Eyedropper

I would love if pickr had the option of displaying and using an eyedropper as well. This could work very similar to the one within Google Chrome's Developer Tools:

I guess the pixel grid preview is a bonus and not necessary.

Ok, maybe I was a bit naive because I thought it might be possible to get colors below the mouse cursor via JS – but apparently it's not or at least not straight forward at all. If you like keep this as a reference in case anybody else has the same idea 😂


codegen: replace downcast_clone with more efficient mem::take

In fact, this PR removes the Dynamic::downcast_clone method entirely.
I wrote it just for my macro code, and (based on other recent API changes) it seems to add little value.

I just realized that I used std::mem::take in my code without thinking, yet the no_std build passed. That tells me there is a testing hole.
If you don't mind, I'd like to fix that in a separate PR. I'd like to use this as a bug it could catch.


Mounting CDFS

Filesystem ImprovementsAllow mounting of UNC pathsEnable CDFS support in drvfsCorrectly handle permissions for network file systems in drvfsAdd support for remote drives to drvfsEnable FAT support in drvfs
Okay I'll bite. At considerable risk of demonstrating my overlooking the patently obvious, how does one mount CDFS on 16176? I put a random ISO 9660 disk in my drive and it lights up as D: in Windows natch, but no sign in WSL /mnt/d. I have restarted the WSL instance. Can't mount it by the usual Linux block device means, which isn't how I would expect it to work on WSL anyway. So, ...?
And more generally, how are you managing disks coming and going? Which is really what has my curiosity piqued, as opposed to actually desiring to read anything off a plastic disk.

@therealkenc - good question. @SvenGroot has a blog post that will be releasing soon talking about the feature. Currently removable drives and network shares have to be mounted manually. Automatically mounting and unmounting drives as they appear in Windows is being considered in a future change. In the meantime here's some of the things you can do:
Mounting DrvFs (including USB and CD)
sudo mount -t drvfs D: /mnt/d
Mounting network locations
sudo mount -t drvfs '\\server\share' /mnt/share
Volumes mounted on empty NTFS folders
sudo mount -t drvfs 'C:\mountpoint' /mnt/myvolume

Awesome Ben; thank-you! [Everybody's Working for the Weekend 🎶.] Works as perfectly as advertised. I suppose re-purpose this question to feature request: Add support for NETLINK_KOBJECT_UEVENT. I mean, how hard can it be? 😀

Looks like udfs.sys is also wired up, even though it isn't explicitly mentioned in the release notes. Very nicely played, @SvenGroot 💯.
Unfortunately mrxdav.sys does not seem to take. Which spoiled my Inception experiment.

Closing as asked and answered. See the blog post for anyone who might land here in a search.


Improve tests of position_utils library.

Pull Request type

Testing
Other: Refactoring of the update_open_interest() function

What is the current behavior?
The testing module position_utils should be improved.
What is the new behavior?
New tests implemented, and they are all passing. The tests implemented are the following:

test_is_position_liquiditable_below_min_collateral_leverage
test_is_position_liquiditable_valid_position
test_is_position_liquiditable_below_min_collateral
test_is_position_liquiditable_negative_remaining_collateral_usd
test_update_open_interest
test_handle_referral
test_update_total_borrowing

Other information
Testing is still needed to be done on the following functions:

will_collateral_be_sufficient()
update_funding_and_borrowing_state()

Hey Zarboq, sorry for the late answer... Will update it!

Great! Left some comments. Are you planning to complete the missing tests?

Hey Zarboq, sorry for the late answer... Will update it! I don't have time to complete the missing tests... Do you want me to create an issue?

No worries :) Yes you can create an issue


Question about the function "compute_pixel_indices"

Hi Peter,
Thank you for sharing the source code which is well written and self-explanatory. However, could you explain the following lines, 278-279? I have pasted it below for your convenience. I know it would output the indices for the patch location corresponding to the original input x.   However, could you explain the logic behind the following lines of code?
idx_pat = (c * H * W).view(C, 1, 1).expand([C, O, O]) + (o * W).view(1, O, 1).expand([C, O, O]) + o.view(1, 1, O).expand([C, O, O])
idx_loc = b * W * H + y * W * S + x * S
idx = idx_loc.view(-1, 1, 1, 1).expand([n, C, O, O]) + idx_pat.view(1, C, O, O).expand([n, C, O, O])
Kind regards,

Using a 1D array as an example. Imagine if the input is
[0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11]

and we want to split into 4-element patches. We only want the first and the third patch, so the result supposed to be:
[[0, 1,  2,  3],
 [8, 9, 10, 11]]

idx_pat is the index of a single patch. In this case, it would be
idx_pat = [0, 1, 2, 3]

idx_loc is the index of locations. In this case, it would be
idx_loc = [0, 8]

Therefore, to get the final indices:
idx = idx_loc + idx_pat

You need to use .view() and .expand to change the dimension. But this is the basic idea.
In the actual code, it is a lot complicated when you need to consider 2D inputs with multiple channels. Getting the index is quite tricky. I spent a lot of time to figure it out. It involved a lot of drawing on paper as well as trials and errors.

Using a 1D array as an example. Imagine if the input is
[0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11]

and we want to split into 4-element patches. We only want the first and the third patch, so the result supposed to be:
[[0, 1,  2,  3],
 [8, 9, 10, 11]]

idx_pat is the index of a single patch. In this case, it would be
idx_pat = [0, 1, 2, 3]

idx_loc is the index of locations. In this case, it would be
idx_loc = [0, 8]

Therefore, to get the final indices:
idx = idx_loc + idx_pat

You need to use .view() and .expand to change the dimension. But this is the basic idea.
In the actual code, it is a lot complicated when you need to consider 2D inputs with multiple channels. Getting the index is quite tricky. I spent a lot of time to figure it out. It involved a lot of drawing on paper as well as trials and errors.

Hi Peter,
Thank you for the response and detailed explanation. This clarifies many things.
Cheers!

Hi Peter,
Thank you for the response and detailed explanation. This clarifies many things.
Cheers!


docs(adr): Update ADR for ODS-Only storage/RSMount

ADR needs to be updated but not for this - instead for new storage system


Long term memory not working in chatapp sample app

Describe the bug
Due to token issue, the response from text completion will not be valid json.
SemanticChatMemory memory = SemanticChatMemory.FromJson(result.ToString());
internal static async Task<SemanticChatMemory> ExtractCognitiveMemoryAsync(

{"items": [{"label": "workspaceName", "details": "SoP_High_Memory"}, {"label": "datasetName", "details": "SoP_High_Memory_0718"}, {"label": "columnName", "details": "AvgMemoryConsumption"}, {"label": "tableName", "details": "WabiAllHighRefresh0718"}, {"label...
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. Windows]
IDE: [e.g. Visual Studio, VS Code]
NuGet Package Version [e.g. 0.1.0]

Additional context
Add any other context about the problem here.

Hello!
Thanks for reporting the issue!
Could you provide more details as to what you mean by token issue? Did the prompt exceed the token limit?
Please note that the models will not always generate valid Json blocks.

warn: Microsoft.SemanticKernel.Connectors.Memory.Qdrant.QdrantMemoryStore[0]
No vectors were found.
warn: Microsoft.SemanticKernel.Connectors.Memory.Qdrant.QdrantMemoryStore[0]
No vectors were found.
warn: Microsoft.SemanticKernel.Connectors.Memory.Qdrant.QdrantMemoryStore[0]
No vectors were found.
warn: Microsoft.SemanticKernel.Connectors.Memory.Qdrant.QdrantMemoryStore[0]
No vectors were found.
warn: Microsoft.SemanticKernel.Connectors.Memory.Qdrant.QdrantMemoryStore[0]
No vectors were found.
warn: Microsoft.SemanticKernel.Connectors.Memory.Qdrant.QdrantMemoryStore[0]
Vectors not found
warn: Microsoft.SemanticKernel.Connectors.Memory.Qdrant.QdrantMemoryStore[0]
Vectors not found
warn: Microsoft.SemanticKernel.Connectors.Memory.Qdrant.QdrantMemoryStore[0]
No vectors were found.
warn: Microsoft.SemanticKernel.Connectors.Memory.Qdrant.QdrantMemoryStore[0]
Vectors not found
warn: Microsoft.SemanticKernel.Connectors.Memory.Qdrant.QdrantMemoryStore[0]
Vectors not found
  I'm suspecting there is some issue in memory setting.

Hi Powe!
Could you provide more details on how you encountered the errors/warnings? Logs or screenshots will help greatly.

This may have been caused by a bug that was impacting the Qdrant connector. This was resolved in #1313
@poweihuang0817 if you have a spare moment to try with the latest code from the repository (I don't think the fix has made it to nuget yet) please let us know if the issue persists

Hello craig and tao, I've mixed two issues up. The bigger question I want to report is that the long term memory would not be json format. GPT will not response as I requested. And then there will be json parsing error. I manage to reduce that by reducing the token.

Closing now as it appears this issue has been resolved. Please reopen as needed.


Update v1.41 api from upstream swagger.yaml (v1.42)

Rebase this one on #71 (relates to https://github.com/moby/moby/issues/39131).


Remove standalone recipe

It doesn't make sense to have a standalone recipe for a "cluster" cookbook. We have the chef-server to setup a standalone Chef Server 12 system.

I sort of disagree with this. In our infrastructure we have standalone master servers that we upload policy to and then replicate to servers in HA topology. It's preferable to only need to deal with single top level cookbook for deployments. The cluster cookbook could just depend on chef-server cookbook though to run its standalone recipe.

There's some background here. The standalone recipe predates the use of not only Chef Provisioning in this cookbook, but also (obviously?) the chef-server cookbook refactor to support Chef Server 12. As it predates the use of Chef Provisioning, there isn't a recipe to use Chef Provisioning to create a standalone machine resource, nor do we want to add such, as it increases the complexity in maintaining this cookbook.
Separate cookbooks draws a clear delineation of purpose. Depending on the chef-server cookbook is definitely something we don't want to do just for the use case of having a standalone recipe.
If you're replicating to multiple Chef Servers, have you looked into Chef Replication? Working with that add-on is something we want to support with this cookbook in the longer term - though we obviously need HA support before that :smile_cat:.

I did dig deeply into Chef Replication the last week or so with the original goal of syncing from Standalone chef server to an HA chef server. This is why was sort mentioning you may end up needed a modified standalone recipe that runs ec_sync_server.
But anyway the current state of Chef Replication unfortunately doesn't really cover our needs for High Availability. It is also disappointing that currently only focusses on syncing policy and not any RBAC information.
I have open ZenDesk #3136 ticket about chef replication.

(chef-server)[https://supermarket.chef.io/cookbooks/chef-server] doesn't provide any of the installation for add-ons. The nice thing about the standalone recipe is that you also got the add-ons installed.

I left a comment on https://github.com/chef-cookbooks/chef-server/pull/77.


New GH CLI wrapper

Alright, so I gave the plugin a first try since you now started to implement this feature. Pretty hyped. :grimacing:
Unfortunately I came not far. I used the ListIssues command. This works great in combination with telescope. But when I want to open an issue, I get an empty buffer and this error message: stdout { '{"message":"Not Found","documentation_url":"https://docs.github.com/rest/reference/issues#get-an-issue"}' }.
I can confirm that I'm logged in. The output of gh auth status is:
github.com
  ✓ Logged in to github.com as weilbith (~/.config/gh/hosts.yml)
  ✓ Git operations for github.com configured to use ssh protocol.
  ✓ Token: *******************

uhm weird, what is the name of the buffer opened? what is the repo/issue you are trying to open?

The buffer is named like the number of the issue. The repo is just one from work (just because it has thousands of issues :shrug: ). I could also just test it with this one. But it is open source. So theoretically the authentication is theoretically not even necessary. At least not for showing.

The buffer is named like the number of the issue.

Is that the output from bufname or how its listed by ls!?
El El dom, 6 dic 2020 a las 0:39, Thore Strassburg<EMAIL_ADDRESS>escribió:

The buffer is named like the number of the issue. The repo is just one
from work (just because it has thousands of issues 🤷 ). I could also
just test it with this one. But it is open source. So theoretically the
authentication is theoretically not even necessary. At least not for
showing.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pwntester/octo.nvim/pull/6#issuecomment-739431175,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA6WBIVOSQ7S4AZMOEW2N3STLAEJANCNFSM4UOBMVJQ
.

Sorry that was just the named displayed in my tabline. So the relevant output from :ls! should be that:
 11u     "term://~/projects/brainbot-projects/repositories/raiden/light-client/raiden-dapp//792503:GIT_PAGER=less gh issue view 2435" line 1
 12 %a   "github://weilbith/light-client/2435" line 1

The buffer name seems correct, what is the output of gh api repos/weilbith/light-client/issues/2435 in the command line?

Oh wait. I think I know what the issue might be related to. The get remote called origin is my fork. The "original" repo is called upstream. On origin there are no issues. But why does the listing of the issues work correctly?

Thats weird, the only difference I can think of is that in the api call you are explicitly setting the repo while in gh issue list the repo is inferred. Can you also try the following to force api to do a similar infer:
gh api repos/:owner/:repo/issues/2435

Please try :ListIssues repo=weilbith/light-client and then open the issue to avoid using the upstream repo implicitly

Merging since seems to work ok. I made some changes to make commands more consistent. Use :ListIssues weilbith/light-client instead


feat: Add resolver class

Codecov Report
Attention: Patch coverage is 12.90323% with 27 lines in your changes missing coverage. Please review.

Project coverage is 54.17%. Comparing base (5bd3d91) to head (027221e).

Files
Patch %
Lines

src/common/resolver-ao.ts
10.00%
27 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##            alpha     #134      +/-   ##
==========================================
- Coverage   55.45%   54.17%   -1.29%
==========================================
  Files          28       29       +1
  Lines         999     1030      +31
  Branches      183      194      +11
==========================================
+ Hits          554      558       +4
- Misses        445      472      +27

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

what's the maturity of the process this is using?


Freiburg im Breisgau, Germany

Hey, I started to add wikidata to streets in Freiburg. Please add my repository to the project: https://github.com/Sequynth/equalstreetnames-freiburg-im-breisgau

Welcome on board @Sequynth !
I've sent you an invite to @EqualStreetNames/freiburg-im-breisgau ; once you've accepted the invite you should be able to transfer your repository.

I cant find the invite (full disclosure: I also dont know where to look)

You should have received an email.
You can also try https://github.com/orgs/EqualStreetNames/invitation

I'll take care of the next steps!
Meanwhile could you add your MAPBOX_TOKEN in the repo secrets (see https://github.com/EqualStreetNames/equalstreetnames-template#integrate-your-city-to-the-project)

You can follow the progress of the deployment here: #306

🚀 https://freiburg-im-breisgau.equalstreetnames.eu/


[WIP / Spike] Discussion starter for a CompiledPipeline using delegates

Discussion PR.
DO NOT Merge! Code is far from workable jet.
See https://github.com/jbogard/MediatR/discussions/886

TOO LATE IT'S MERGED

TOO LATE IT'S MERGED

:-) I was looking.
But seriously this code is NOT ready and is targeted as a discussion starter / reference. It has lots of flaws both in design and code because it was for me a quick workout.
I know now that MediatR already uses a delegate but maybe it can be used more in order to speed things up...?


Introduce patchMode for pod annotations

Description
When patching pod annotations, some clients desire different behaviour. The CNI Plugin wishes to patch only the CNI annotations, and since it does not currently retry the CreateOrUpdate operation, it requires the revision to be set to "".
Other clients wish to patch different annotations, and to set the revision properly and handle retries.
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note
None required

/merge-when-ready squash-commits delete-branch

/hold-merge

/merge-when-ready


[WIP] Start testing the javascript

Based on some quick research, it seems like Jasmine and Teaspoon are pretty good choices for Rails Javascript testing (mostly unit and integrational, respectively).
I've added the bare-bones (quickly) to have it run these test frameworks and will look into adding some actual tests shortly.
Addresses more of #57.

I've now added an actual test! This ensures that the global.js ready handler definitely sets the localStorage key from the gon data.

Prefer to test this with integrational tests via (probably) Capybara instead. We could investigate mocha if we want to test the javascript by itself but it's a bit pointless when most of it is 80%+ dependent on the page.


Update Tumblr logo

Updates the tumblr logo to use the one that was introduced mid 2016.

Looks like you beat me to it.

:-) Thanks!


Support int16 and int64 in retries header

Some messages in borges returned the number of retries as an int64 number and this was generating errors.
It also rejects messages with malformed headers. If not the messages will be unacked and eventually block the channel.

There's a problem with rabbitmq in windows. I've restarted the job in my appveyor account a couple of times and was not able to start the server (connect to the port). I'm merging as the changes should not be affecting windows.


when train on multiface dataset, eyes can't close compared with MeshTalk

I try to train on the multiface dataset which has a lot of details in the upper face, but the driving results cannot close eyes when talking. While the meshtalk can close the eyes when talking.
The reconstruction stage works well when closing eyes?
Any suggestion?

Sorry that I didn't train our model on the multiface dataset, I may not provide useful suggestions.  But I think the model should support the motions in eyelids in principle. As the motions in eyelids can also be encoded into discrete motion representations and mapped from the audio signal somehow, although the relationship between the above two could be loose. I guess the reason that MeshTalk can easily drive eyelids is the tailored design in the loss function?

In the reconstruction stage, you use the mulitple facial component to quantize the code, while in the regression stage multiple facial component is predicted simultaneously for a frame. Compared to meshtalk, they autoregressive predict category for a single frame.


Await completion of future in example

For a first time user, it's nice to have a "happy moment" when using a new library for the first time. While running the example code from the README, I was a bit confused why I couldn't see my data in InfluxDB. Of course the solution was to simply wait for the future to complete.
Do you think it makes sense to add this final line to the example to write the data to InfluxDB when the code is executed?

was to simply wait for the future to complete.

That's actually not guaranteed to show the data in Influx. Influx only acknowledges that it has received your request, not that it's indexed it. When it's under heavy load, this becomes more obvious.

The content of the changes looks OK to me. Were this my repository, I'd want the two commits squashed, but I'll leave such decisions up to @stephanbuys , as well as the merge itself.

Can't commits be squashed on merge with Github? Just curious.

Can't commits be squashed on merge with Github?

That's certainly something that the GH UI allows; I've never done it myself.

Trying "squash and merge" from GH GUI... Apologies I'm out of town for a small while, so may not be very responsive.


PoC of checking available kernel features to determine what bpf programs to load

See samples/feature-detect/README.md
Note that the intention isn't to merge this into libbpfgo as is, I would remove reference to tracee before that, but it would be good to have as a sample. This is just a convenient way of discussing the PoC.

Closing this in favor of https://github.com/aquasecurity/tracee/pull/1846


Why deprecate tf.layers.dropout?

In the tensorflow 1.13 document of tf.layers.dropout, there is a warning:
Warning: THIS FUNCTION IS DEPRECATED. It will be removed in a future version. Instructions for updating: Use keras.layers.dropout instead.
It is ridiculous to deprecate tf.layers.dropout, because it is widely used, and very easy to use.
Do you really care about the requirements of TF developers?
Do you really care about the requirements of TF developers?
Do you really care about the requirements of TF developers?

Duplicate#26151


Can we extend email protocols other than sendmail and smtp to Mail APIs

Is there any way to extend email class's protocol other than sendmail and SMTP?
So we can change protocol to MailGun or SendGrid APIs and send the mail via APIs

This is a bug tracker, we don't answer such questions here.
Please post on our forums instead.


code: Cleanups for v0.1

fix #89

Codecov Report

Merging #90 (1d4f5b9) into main (1522a7d) will decrease coverage by 0.09%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #90      +/-   ##
==========================================
- Coverage   27.42%   27.33%   -0.10%
==========================================
  Files          50       50
  Lines        1721     1727       +6
==========================================
  Hits          472      472
- Misses       1249     1255       +6

Impacted Files
Coverage Δ

ui/src/service/forms.js
9.17% <0.00%> (-0.45%)
:arrow_down:

ui/src/shared/resources.js
34.54% <0.00%> (-0.64%)
:arrow_down:

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 1522a7d...1d4f5b9. Read the comment docs.


Load data for TCGA use case

Loading TCGA data for one of the tumor types is important because it gives a bigger dataset to play with during development. Furthermore, this will also be the dataset that our collaborators will use to provide feedback.

@sgratzl, what is the easiest way to achieve this? Add the hdf data connector and put a TCGA hd5 package in the data folder?

exactly

ok, great. can you please add this to the product so that the larger TCGA dataset appears in the daily build version?

Please also let the rest of the mothertable team know where to the TCGA hd5 data file can be found.

cd mothertable_workspace
mkdir _data
cd _data
wget https://s3.eu-central-1.amazonaws.com/phovea-data-packages/TCGA_GBM_2014_04_16.h5
cd ..
git clone https://github.com/phovea/phovea_data_hdf.git
yo phovea:update
docker-compose build api
docker-compose up -d

Great, thanks!

@mijar, @katush, @bikramkawan, please add the data in your local dev environment, too.

I am getting a wget: command not found and was not yet able to find how to install it...I am going to the defense now and will continue looking for it afterwards, if you would not be faster and help me.:-) thanks

wget is just a command to download a file. you can also download the file via your browser and put it in the _data folder.

If you are getting wget command not found you can simply download and place the file in _data folder. It worked for me.

Oh I see. Thank you.


Add an option to not add closing half of the pair when it's next to a none whitespace character

Autoclose.nvim is convinient when I freshly type things like below.
   cursor
     |
     V
(word|)

However I actually edit more than write I often find myself doing very inefficient editing like following.
; move the cursor to a existing word
|word

; type (
(|)word

; delete ) - this is a multiple key strokes
(|word

; put ) at the end
(word)|

I came up with a simple idea. Perhaps not adding a closing half of the pair when the cursor is "touching" the word.
So it would be nice if this plugin can offer an option like that.

This options is added in 8ed364d39a7e09265f283d5ec0ce88e3a6806f41. Thanks for your feedback.

Works perfectly, thank you!


Improve docs by removing typos and grammatical errors

Fixes #28390
Why:
Closes

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline.

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[ ] For content changes, I have completed the self-review checklist.

Yep, I noticed that too and have been coordinating with our engineering team to figure out why the test keeping timing out. We'll get this merged as soon as we can.


Carriage Return added to cell value incorrectly during import

Issue
During clipboard import using a minimal dataset to reproduce the issue found in a larger datasets with sometimes voided sensor data from oceanographic buoys.  When cells are null, not all cells are detected as null, and if a null value is found at the end of a row, it is getting incorrectly replaced as a carriage return (CR) for the cell value, rather than the carriage return being treated as a row delineation and the cells' value stored correctly as a null.
To Reproduce
Steps to reproduce the behavior:

Add minimal dataset to the Clipboard importer and Create Project with default settings checked.
Then, click Edit on Column C, row 4 and inspect its value which has an embedded char(13) carriage return.

Current Results
char(13) is hidden but selectable and can be copied.  Also shown in between a <span> </span> of HTML.
Expected Behavior
I expected that the 4th row currently being imported it seems as:
null,null,CR

would be parsed and stored as
null,null,null

Screenshots

Versions

Operating System: Windows 11 Pro 64GB
Browser Version: Edge latest
JRE or JDK Version: embedded
OpenRefine: 3.8.1

Datasets

"A","B","C"
"1","2","3"
"a","a","a"
"","",""
,,
'','',''

Optional - The last row 6 in the CSV are wrapped single quotes and were added just to help me verify during Java debugging and allow comparison to CSV row 4 and 5 for the CsvParser and the TabularImportingParserBase debugging.
But it doesn't matter, because the problem is exhibited regardless if quotes are mixed or consistent double quotes as in the following dataset which also results in char(13) carriage return in Column C values when there are null valued rows:
"A","B","C"
"1","2","3"
"a","a","a"
"","",""
,,
,,
,,
"z","z","z"

Additional context
When sensor row data lacks values then it is voided (connection issues, low battery, etc.) with null values.
For instance, there indeed would sometimes be rows looking like the minimal dataset where no value exists between comma , separators.  I.E. null values (not empty strings wrapped in double quotes).
When sensor row data has values, then they are typically wrapped in double quoted strings, sometimes with certain buoy model versions the sensors produce empty strings because the value is that of something akin to "nothing detected", so that is represented as "".

The reason CR is being saved is because the check in ExpressionUtil -> isNonBlankData() considers the value to be a non empty string. We can update the logic in the TabularImportingParserBase to handle this case.

@wetneb Can you assign this issue to me

I haven't looked into the problem, but I am not sure if ExpressionUtil -> isNonBlankData()  is the root cause of the problem here, because intuitively the CR character should already be just parsed by the CSV parser as marker for the end of the line. But maybe it's easier to discuss with a PR at hand.

@wetneb  I tested with the production release 3.8.1 when the last cell/col is empty CR is being saved in the cell

This is a Windows specific problem caused by us hardwiring the CSV line separator when Windows uses CRLF instead of LF. It isn't clear to me whether this will only affect the Clipboard or any CSV file.
As @wetneb implies, the fix needs to be applied when the initial parsing is done, not downstream. I've created a PR that tweaks the importer configuration to turn on automatic line separator detection for both TSVs and CSVs (the CSV case also covers other custom field delimiters).

This is also interesting but might be unrelated.... during build I spotted this:
[INFO] Compiling 496 source files with javac [debug release 11] to target\classes
[WARNING] /E:/IntelliJ_IDEA_Projects/OpenRefine/main/src/com/google/refine/importers/SeparatorBasedImporter.java:[189,66] non-varargs call of varargs method with inexact argument type for last parameter;
  cast to java.lang.Object for a varargs call
  cast to java.lang.Object[] for a non-varargs call and to suppress this warning

Works correctly now with fixes from https://github.com/tfmorris/OpenRefine/commit/e953dc63e97116b75e425a72c9d15b3a5e9bcd50


Create directories for output if they don't exist yet

Running wp makepot theme path/to/theme path/to/theme/languages/foo.pot when languages doesn't exist yet produces the following output:
PHP Warning:  file_put_contents(languages/foo.pot): failed to open stream: No such file or directory in /.wp-cli/packages/vendor/gettext/gettext/src/Generators/Generator.php on line 16

Warning: file_put_contents(languages/foo.pot): failed to open stream: No such file or directory in /.wp-cli/packages/vendor/gettext/gettext/src/Generators/Generator.php on line 16
Error: Could not generate a POT file!

This should be fixed now after 1c6e5b3d807d12b2b4715c5181e2e54dd312d821


Material Design Preloader not showm

In material design preloader not shown although in ios design works well. How can I fix it

Same problem here (Framework7 + vue.js + webpack - Template)

I can also confirm this problem.
On android the css style color definition it's missing the svg (while it's present on the ios counterpart) ie the definition:
.preloader.color-gray,
.preloader.preloader-gray {
background-image: url("data:image/svg+xml;charset=utf-8,%3Csvg%20viewBox%3D'0%200%20120%20120'%20xmlns%3D'http%3A%2F%2Fwww.w3.org%2F2000%2Fsvg'%20xmlns%3Axlink%3D'http%3A%2F%2Fwww.w3.org%2F1999%2Fxlink'%3E%3Cdefs%3E%3Cline%20id%3D'l'%20x1%3D'60'%20x2%3D'60'%20y1%3D'7'%20y2%3D'27'%20stroke%3D'%238e8e93'%20stroke-width%3D'11'%20stroke-linecap%3D'round'%2F%3E%3C%2Fdefs%3E%3Cg%3E%3Cuse%20xlink%3Ahref%3D'%23l'%20opacity%3D'.27'%2F%3E%3Cuse%20xlink%3Ahref%3D'%23l'%20opacity%3D'.27'%20transform%3D'rotate(30%2060%2C60)'%2F%3E%3Cuse%20xlink%3Ahref%3D'%23l'%20opacity%3D'.27'%20transform%3D'rotate(60%2060%2C60)'%2F%3E%3Cuse%20xlink%3Ahref%3D'%23l'%20opacity%3D'.27'%20transform%3D'rotate(90%2060%2C60)'%2F%3E%3Cuse%20xlink%3Ahref%3D'%23l'%20opacity%3D'.27'%20transform%3D'rotate(120%2060%2C60)'%2F%3E%3Cuse%20xlink%3Ahref%3D'%23l'%20opacity%3D'.27'%20transform%3D'rotate(150%2060%2C60)'%2F%3E%3Cuse%20xlink%3Ahref%3D'%23l'%20opacity%3D'.37'%20transform%3D'rotate(180%2060%2C60)'%2F%3E%3Cuse%20xlink%3Ahref%3D'%23l'%20opacity%3D'.46'%20transform%3D'rotate(210%2060%2C60)'%2F%3E%3Cuse%20xlink%3Ahref%3D'%23l'%20opacity%3D'.56'%20transform%3D'rotate(240%2060%2C60)'%2F%3E%3Cuse%20xlink%3Ahref%3D'%23l'%20opacity%3D'.66'%20transform%3D'rotate(270%2060%2C60)'%2F%3E%3Cuse%20xlink%3Ahref%3D'%23l'%20opacity%3D'.75'%20transform%3D'rotate(300%2060%2C60)'%2F%3E%3Cuse%20xlink%3Ahref%3D'%23l'%20opacity%3D'.85'%20transform%3D'rotate(330%2060%2C60)'%2F%3E%3C%2Fg%3E%3C%2Fsvg%3E");
}
It's not present on the framework7.material.colors.css file, adding any of the color definition with embedded svg will correctly show the preloader.
Thanks
Rob

Issue is closed because of outdated/irrelevant/not actual
If this issue is still actual and reproducible for latest version of Framework7, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


Generalise AbstractLoggingRewriter

We currently have
AbstractLoggingRewriter extends AbstractNodeVisitor<Node>
On the other hand, we never assume that the visit() methods return a Node object. To provide greater flexibility to subclasses of the AbstractLoggingRewriter we should change this to
AbstractLoggingRewriter<T> extends AbstractNodeVisitor<T>

I think we should apply the rule "components before inheritance" here and avoid having another abstract class.

This is overcome by #375


Cannot create property 'filename' on string 'a4.pdf'

Run example code, get the following result as bellow. In chrome, and jsPdf 1.4.1.
var doc = new jsPDF();

doc.text('Hello world!', 10, 10);
doc.save('a4.pdf');

jspdf.min.js:31 Error in function Object.It (http://localhost:7643/Scripts/jspdf.min.js:31:22222): Cannot create property 'filename' on string 'a4.pdf' TypeError: Cannot create property 'filename' on string 'a4.pdf'
    at Object.It (jspdf.min.js:31)
    at Object.Gt.foo [as output] (jspdf.min.js:31)
    at Object.W.save (jspdf.min.js:31)

Works in my chrome without issues and I think I already fixed it for 1.4.2.

Where is the 1.4.2, it‘s not release yet, is it ?

not yet released, correct.

Is there na update on this?

@ekatzenstein
It was an issue of jspdf v 1.4.1. It is fixed in v. 1.5.2. It is resolved. This should be left closed.


Set use_https correctly for password reset email

Overview
Adjust the use_https setting for the password reset email to False
in development and True in production. This effectively changes the
protocol setting used in the base PasswordResetForm's save method:
https://github.com/django/django/blob/master/django/contrib/auth/forms.py#L290
Connects #231
Connects #97
Demo
Using 'use_https': settings.ENVIRONMENT != 'Development',:

Using 'use_https': settings.ENVIRONMENT == 'Development',:

Testing Instructions

serve this branch then use the password reset form to send a reset email and verify that it uses http
adjust the use_https line updated in this changeset to be

'use_https': settings.ENVIRONMENT == 'Development',

... then reload the server and send another password reset email to verify that the email now uses https

Cool, thanks for taking a look!


Frog jump

hey @ayu8  , I want to add this solution in CPP into the DP folder, please assign this to me and add this under hacktoberfest label
problem link:https://www.codingninjas.com/codestudio/problems/frog-jump_3621012

Go ahead
Make sure to go through CONTRIBUTING.md once


colexec: add bool_or/bool_and operators

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

The vectorized execution engine now supports bool_and/bool_or builtin aggregation functions. [#44164][#44164] {% comment %}doc{% endcomment %}

vectorized execution https://airtable.com/tblD3oZPLJgGhCmch/viwNg2fK3NU3DUY6Z/recagjuRqKN5Jxh1n


fix: upload to scheduler

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 4 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.004%) to 92.109%

Totals

Change from base Build<PHONE_NUMBER>:
-0.004%

Covered Lines:
8183

Relevant Lines:
8884

💛  - Coveralls


Improve pre-commit config

Change the bot schedule from weekly to quarterly, to reduce noisy PRs.
Change the black target language version to 3.7, since we support 3.7.
Add prettier as a pre-commit hook, to auto-format YAML files.
Add a bunch of cheap, useful hooks from https://github.com/pre-commit/pre-commit-hooks. trailing-whitespace, end-of-file-fixer and mixed-line-ending are all autofix hooks. check-yaml, check-toml and check-merge-conflict are not, but are useful hooks for detecting problems early.

Not sure why pre-commit is failing to autofix the GitHub Actions workflow files -- I found this issue, which suggested that it might be because we haven't given the pre-commit.ci app the necessary permissions to autofix workflow files: https://github.com/pre-commit-ci/issues/issues/14

I don't see a way to adjust this in https://github.com/PyCQA/flake8-pyi/settings/installations, I think it might require org-level changes. Maybe just fix the formatting yourself locally?

I think it might require org-level changes.

Hmm. In that case, I think I'll just leave prettier out. I'm not a fan of formatters if they don't apply autofixes; that sounds more annoying than useful.


How to use BranchPythonOperator with task failure?

Correct me if I'm misunderstanding how these are used.
I'm attempting to use the BranchPythonOperator using the previous task's state as the condition.
After the previous task has run, I use on_success_callback or on_failure_callback to write a file that contains the task_id that should be used.
When running the BranchPythonOperator, I check the task_id recorded in the file in order to select which branch the worker should select.
So I have a few questions:

Is there an easier way to check the previous task's state?
I know there is an upstream_failed state that allows the scheduler to continue with the rest of the DAG. How would I use this to make the BranchPythonOperator (and the rest of the DAG) run? Or am I misunderstanding its usage?

Thank you very much.

Airflow wasn't really designed for "conditional pathing" originally, but there are ways to do this now. Note that anything downstream any non-successful task won't run as it is today. I'm thinking about adding a OnErrorOperator that would be triggered and succeed when any directly upstream task is in error.
In the meantime your can use a disconnected ExternalTaskSensor which would wait on a task in the same DAG, and a specific state. Keep in mind that it can't have the task you are targeting as a parent if you want it to trigger.
http://pythonhosted.org/airflow/code.html#airflow.operators.ExternalTaskSensor
I'm curious, what is your use case? What does the task that might fail do, and what do you do downstream if it fails?

#232 should solve your first question. Hopefully you'll be able to do something like
def branch_op_callable(**kwargs):
    state = kwargs['ti'].xcom_pull(from_tasks=upstream_task_id)
    if state == 'foo':
        return 'branch 1'
    else:
        return 'branch 2'

One of the motivations is to avoid having to write state to files to use it downstream. Would be great to learn about your use case to make sure it handles it!

We're currently using cron to schedule our jobs and would like to migrate to a new system (Airflow is very promising!).
The jobs run scripts that are run daily. The scripts are configured to print to stdout to log error messages that cron then e-mails out. The rest of the job continues normally in order to finish as much work as possible.
It wouldn't be so hard to view the task's log in the UI, except there are over 26 of them. It would be neat to use the new Slack Operator in order to notify us of a task failure. The proposed solution would be continue writing to stdout but also track whether or not an error was logged. At the end of the script it would return zero or a non-zero exit code.
Thanks for the replies!

I'm thinking about adding a OnErrorOperator that would be triggered and succeed when any directly upstream task is in error.
Maxime did you still thinking about this functionality ? It would be ewry helpful ...Thank u !

@zstan :
http://pythonhosted.org/airflow/concepts.html#trigger-rules
Also if you want something to run atomically with the task, there's a BaseOperator.on_failure_callback
http://pythonhosted.org/airflow/code.html#baseoperator


Error after upgrading to Node 8.0.0

Error after upgrading to Node 8.0.0
warning gulp-uncss > uncss > request ><EMAIL_ADDRESS>ReDoS vulnerability parsing Set-Cookie https://nodesecurity.io/advisories/130
warning browser-sync > localtunnel > request ><EMAIL_ADDRESS>Use uuid module instead
...
...
'javascript' errored after 560 ms
'build' errored after 569 ms
'default' errored after 571 ms
The following tasks did not complete: pages, sass, images, copy
Did you forget to signal async completion?

This could be related to the deprecation of node-uuid which is not compatible with 8.0.0.
How to reproduce this bug:
Install Node v >= 8.0.0
foundation new
web site
zurb template
yarn
foundation watch
What should happen:
Server should boot on localhost:8000
After switching to Node 7.10.0 using NVM I was able to build successfully.
What happened instead:
Above error message.

First of all, thanks for letting me know ... that Node 8.0 has arrived ... wasn't aware ... :)
On the issue, cc @kball !

A quick lookthrough makes me believe this is coming through the sassdoc dependency... will try updating that on Monday, unless someone wants to get to it before me

@IamManchanda @kball  Yes, and NPM 5 is also released! Thanks for the quick reply... Enjoy the weekend.

I tried to reproduce and could not with the latest version of node (8.1.2). Is this still a problem @michaelfriedman?

I'm with @colin-marshall, I've tested using Node 8.1.2 and unable to reproduce the issue.
Maybe node-uuid dependency has been updated in the package that depends on it?

Sounds like this was fixed downstream. Closing.


Change from Postgres to Postgis in Openshift

Describe the task
A task to change from the current Postgres installation in DEV environment to Postgis
Acceptance Criteria

[ ] first
[ ] second
[ ] third

Additional context

Add any other context about the task here.
Or here

After meeting with Om, using CrunchyData for PostGIS


chore(release): v1.3.2 🎉

:robot: I have created a release *beep* *boop*
1.3.2 (2022-11-25)
Bug Fixes

deps: upgrade depenedency on type fns (7201a41)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ehmpathy/with-remote-state-caching/releases/tag/v1.3.2 :sunflower:


[import/no-internal-modules] path normalisation makes preventing package internal imports impossible

It feels like the this rule ought to be support my use case of preventing an import from deep inside any package, e.g. somepackage/some/internal/path. There doesn't appear to me to be any way to achieve that, due to the normalisation that takes place.
So I set an "allow" config for "allow": ["./**"], so that reaching is prevented except for relative paths; however, the normalisation that happens inside the rule scrubs all . and .. segments away for some reason. There doesn't therefore be any possibility of distinguishing between relative and package imports.
How is this use case supposed to be achieved?

Why would you want to prevent that tho? Deep imports are the objectively superior way to get at subcomponents of a package; they obviate the need for treeshaking and result in much smaller bundle sizes and programs.

Cool, thank you for sharing your subjective opinion with me.
I need it because VS code auto-import messes up and imports stuff from deep inside packages sometimes, especially in monorepos, and I'd like to enforce that it shouldn't happen—a lot (most?) of NPM packages are written with barrel export files and the implicit assumption that anything not in the barrel is internal and therefore subject to change.
Typescript is also going through a phase (in the process of being fixed) of preferring to select auto-imports from a barrel file in the current package, meaning you end up with imports like from '../../..', which I'd like to prevent.

It’s not subjective; it’s objective. Anything that’s deep importable should always be deep imported.
Barrel files are an antipattern.

Opinions are subjective by definition. You don't define "deep importable" here, but if you're suggesting that a package consumer should expect to be able to import any file they please from deep inside a package, and therefore package maintainers should make guarantees (via version numbering, say) about the internals of a package, then I don't agree at all. I doubt that many package maintainers are bumping major versions when they make these kind of changes, which they would have to if the file structure formed part of the package contract.
I can't find anything setting out why barrel files are an anti-pattern as you suggest, and you don't provide any sources or clue to your thinking. My observation is that they are pretty common in the ecosystem.

because, the first is an external library that should not be used like that, and the latter is a lot of code that I want warnings for while we fix instead of 300 changes...
  11:30  error    Reaching to "xstate/lib/types" is not allowed                                                import/no-internal-modules
  14:31  error    Reaching to "./workflows/create-action" is not allowed                                       import/no-internal-modules

@IAmNatch it's a common use case, but it's still always a bad idea.

perhaps a better solution, at least in my case would be to have an [import/no-internal-from-external-modules] rule... which could potentially use the rule that defines what's internal

bad, no argument, but progressive migration ftw.

My workflow is to not allow imports too far from current file, eg import { smth} from '../../../ui/src/package';
Usually it means that you are trying to import file which is outside of boundaries in your project.
Usually it happens due to VSCode/WebStorm  imports.
Unfortunately, the following code does not work
"import/no-internal-modules": [
         "error",
         {
           "forbid": ["../../../**/*"]
         }
       ],

Finally I solved my issue with another rule
"no-restricted-imports": [
          "error",
            {
              "patterns": [
                "../../../*",
                "libs/**/*"
              ]
            }
        ]

@IAmNatch it's a common use case, but it's still always a bad idea.

The name of the rule is

no-internal-modules

And the rule description says

Use this rule to prevent importing the submodules of other modules.

Doesn't that mean this rule is literally for preventing deep imports from other modules?
If preventing deep imports is such a bad idea with no good use case what-so-ever, then the rule shouldn't have existed in the first place.
In the current form, the path normalization makes the rule effectively

Use this rule to prevent importing the submodules of other modules.

For those who are interested, here's a workaround for the problem.
Note that this will make your eslint significantly slower in large projects.
// The internal files of a package (any folder with a index.ts?(x) file)
// shall not be directly imported by other packages.
// This helps reducing the surface of a package.
'import/no-restricted-paths': [
  'error',
  {
    basePath: './src',
    zones: glob
      .globSync(['**/index.ts', '**/index.tsx'], {
        cwd: 'src',
        withFileTypes: true,
      })
      .map((p) => {
        const parent = p.parent.relative();
        const parentSegs = parent.split('/');

        // The following code is basically trying to construct a pattern
        // that is equivalent to `!${parent}/**/*`. Unfortunately, we cannot
        // use negated patterns. The plugin will try to resolve the pattern
        // like a path, making it invalid (e.g. `!dir/**` will become
        // `home/user/src/!dir/**`).

        // Nothing in the inner most directory shall be matched.
        let pattern = '{}';
        while (parentSegs.length !== 0) {
          const seg = parentSegs.pop();

          // Matches
          // 1. anything under `${seg}/` that matches the previous pattern,
          // 2. anything not under `${seg}`,
          // 3. anything not under a directory.
          //
          // Effectively, in each iteration, we expand the pattern to
          // include siblings in the parent directory.
          pattern = `{${seg}/${pattern},!(${seg})/**,*}`;
        }

        return {
          target: pattern,
          from: parent,
          except: [p.name],
        };
      }),
  },
],


go/types: some small problems in docs

In the docs of the selections:
Selector    Kind          Recv    Obj    Type               Index     Indirect

p.x         FieldVal      T       x      int                {0}       true
p.m         MethodVal     *T      m      func (e *T) m()    {1, 0}    true
T.m         MethodExpr    T       m      func m(_ T)        {1, 0}    fals

Should the Type columns for the second and third line be func() and func(_ T)?

And in the [func (*Scope) LookupParent] docs:
...  (see Insert, below) ...

The Insert method docs is shown above the LookupParent docs in fact.

The Insert method docs is shown above the LookupParent docs in fact.

It's below in source code, but above in documentation (since godoc displays identifiers in alphabetical order). We can fix this by making it say just "see Insert"; the reader should be able to find Insert easily in either medium.

Should the Type columns for the second and third line be func() and func(_ T)?

That does seem more fitting to me too. It's also what https://github.com/golang/example/tree/master/gotypes#selections uses. The current text seems to be close to what types.SelectionString may produce, but that is the string form of the entire selector expression, not just its type.
@griesemer Do you agree it would be an improvement, or are we missing some context?
/cc @dominikh

Agreed. CL forthcoming.


Inconsistent reporting of the number of compactions

I am doing some testing on a branch and I noticed an inconsistency in how we report the number of compactions. The Monitor reports 4 compactions active. The fate print command shows 1 FATE transaction. And the listcompactions command shows a compaction per each tablet compacted.

root@uno> listcompactions
2022-02-25T13:20:25,037 [Shell.audit] INFO : root@uno> listcompactions
 SERVER               | AGE       | TYPE  | REASON | READ  | WROTE | TABLE           | TABLET                                   | INPUT | OUTPUT                              | ITERATORS | ITERATOR OPTIONS
 ip-10-113-12-25:9997 |    33m19s |  FULL |   USER |  388K |  388K |              ci | 2;59999999999999a2;533333333333333b      |     1 |        /2/t-00001by/A0000214.rf_tmp |        [] | {}
 ip-10-113-12-25:9997 |    33m13s |  FULL |   USER |  387K |  387K |              ci | 2;533333333333333b;4cccccccccccccd4      |     1 |        /2/t-00001bx/A0000215.rf_tmp |        [] | {}
 ip-10-113-12-25:9997 |    23m44s |  FULL |   USER |  276K |  276K |              ci | 2<;79999999999999a5                      |     1 |   /2/default_tablet/A000021e.rf_tmp |        [] | {}
 ip-10-113-12-25:9997 |    23m44s |  FULL |   USER |  276K |  276K |              ci | 2;6000000000000009;59999999999999a2      |     1 |        /2/t-00001bz/A000021f.rf_tmp |        [] | {}
ip-10-113-12-25:10000 |    33m17s |  FULL |   USER |  388K |  388K |              ci | 2;6cccccccccccccd7;666666666666667       |     1 |        /2/t-00001c1/A0000250.rf_tmp |        [] | {}
ip-10-113-12-25:10000 |    33m15s |  FULL |   USER |  388K |  388K |              ci | 2;666666666666667;6000000000000009       |     1 |        /2/t-00001c0/A0000251.rf_tmp |        [] | {}
ip-10-113-12-25:10000 |    23m43s |  FULL |   USER |  276K |  276K |              ci | 2;79999999999999a5;733333333333333e      |     1 |        /2/t-00001c3/A0000254.rf_tmp |        [] | {}
ip-10-113-12-25:10000 |    23m38s |  FULL |   USER |  275K |  275K |              ci | 2;733333333333333e;6cccccccccccccd7      |     1 |        /2/t-00001c2/A0000255.rf_tmp |        [] | {}
root@uno> fate print
2022-02-25T13:20:29,504 [Shell.audit] INFO : root@uno> fate print
2022-02-25T13:20:29,506 [conf.SiteConfiguration] INFO : Found Accumulo configuration on classpath at /home/mike/workspace/uno/install/accumulo-2.1.0-SNAPSHOT/conf/accumulo.properties
2022-02-25T13:20:29,520 [zookeeper.ZooSession] DEBUG: Connecting to localhost:2181 with timeout 30000 with auth
txid: 73ecbda03cea6dc3  status: IN_PROGRESS         op: CompactRange     locked: [R:+default, R:2] locking: []              top: org.apache.accumulo.manager.tableOps.TraceRepo@61126fdf created: 2022-02-25T17:13:59.413Z
 1 transactions

I think the reason the Monitor is showing me 4 for that tserver is because its displaying per-tablet. The other tserver is also displaying 4 running compactions, which equals the 8 total printed in listcompactions.

Fate is listing transactions though, not compactions.

I think the reason the Monitor is showing me 4 for that tserver is because its displaying per-tablet. The other tserver is also displaying 4 running compactions, which equals the 8 total printed in listcompactions.

Do you have a screenshot? Which page of the monitor? If we're in a /tserver page, and the page clearly identifies itself as reporting stuff going on with that specific tserver, then that makes sense.

I think the reason the Monitor is showing me 4 for that tserver is because its displaying per-tablet. The other tserver is also displaying 4 running compactions, which equals the 8 total printed in listcompactions.

Do you have a screenshot? Which page of the monitor? If we're in a /tserver page, and the page clearly identifies itself as reporting stuff going on with that specific tserver rather than a total number of compactions, then that makes sense.

No. I think it was the tserver page.

No. I think it was the tserver page.

If it's a tserver page, then it makes sense that it's only showing those for that specific tserver. I'm not sure any changes are needed for this. Everything sounds like it's working as expected. Maybe the column on the monitor page could have hover text that explains it's only the totals for that tserver?

I don't have any ideas how to make this better.


True fullscreen in Opera

According to this topic:
http://my.opera.com/community/forums/topic.dml?id=1621502&t=1360668065&page=1
It should work if the code is changed to not specifically detect layout engines, but to a more proper feature detect. Something like that.
On a sidenote, it also doesn't work in either IE9 or IE10, but knowing those browsers, that probably is their fault.

@thany Is this still an issue?

Thanks for the followup :)
Not really an issue anymore, since Opera-from-back-then is effectively dead. If it works in Chrome, it'll work in Opera now.


Picks the most recently started agent's data when multiple agents with the same agent id sent stat data simultaneouly

When multiple agents are registered under a single agent id, stat data from different agents are mixed up and the stat charts shows unpredictable data.
This change makes it so that in cases where there are multiple stat data to choose from in a single timeslot, it chooses the stat data from the most recently started agent.
resolves #2345

Current coverage is 40.25% (diff: 96.00%)

Merging #2407 into master will increase coverage by 0.05%

@@             master      #2407   diff @@
==========================================
  Files          1629       1630     +1
  Lines         57084      57134    +50
  Methods       11103      11109     +6
  Messages          0          0
  Branches       7869       7873     +4
==========================================
+ Hits          22945      22998    +53
+ Misses        34139      34136     -3
  Partials          0          0

Powered by Codecov. Last update 698042e...8742207


Add WebXR support

The engine currently supports WebVR. This is being removed from Chrome in version 80. WebXR is being enabled (not behind a flag!) in Chrome 79.
Question:
Should the engine retain WebVR support for now and add WebXR support? The engine would then attempt to obtain access to the WebXR API first and fall back to WebVR if not present. Alternatively, WebVR support could be stripped out and WebXR added in its place. The former feels like the correct choice.

We could also remove first class WebVR support and leverage something like webxr-polyfill to handle the WebVR fallback.

How much code does WebVR comprise in the engine?

@halvves Good point!
@slimbuck A few hundred lines maybe.

Chrome has been released, are there any plans for XR related features to start?

Yes, there are plans to implement in-engine support for WebXR.


fix/psr4-autoload-exceptions-compliance

This is a suggestion to fix the #33 autoload issue

Another possibility is to wrap exceptions in a "Exceptions" folder under Cronitor\Exceptions namespace


RuntimeError: dimension specified as 0 but tensor has no dimensions.

I want to fine-tuning from your model faster_rcnn_1_10_14657.pth. I run the following command
python trainval_net.py --dataset coco \
 --nw 5 --bs 15 \
--net res101  --cuda \
--r True --checkepoch 10 --checkpoint 14657 --mGPUs

I get the error:
Traceback (most recent call last):
  File "trainval_net.py", line 320, in <module>
    rois_label = fasterRCNN(im_data, im_info, gt_boxes, num_boxes)
  File "/home/dell/anaconda3/lib/python3.6/site-packages/torch/nn/modules/module.py", line 491, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/dell/anaconda3/lib/python3.6/site-packages/torch/nn/parallel/data_parallel.py", line 115, in forward
    return self.gather(outputs, self.output_device)
  File "/home/dell/anaconda3/lib/python3.6/site-packages/torch/nn/parallel/data_parallel.py", line 127, in gather
    return gather(outputs, output_device, dim=self.dim)
  File "/home/dell/anaconda3/lib/python3.6/site-packages/torch/nn/parallel/scatter_gather.py", line 68, in gather
    return gather_map(outputs)
  File "/home/dell/anaconda3/lib/python3.6/site-packages/torch/nn/parallel/scatter_gather.py", line 63, in gather_map
    return type(out)(map(gather_map, zip(*outputs)))
  File "/home/dell/anaconda3/lib/python3.6/site-packages/torch/nn/parallel/scatter_gather.py", line 55, in gather_map
    return Gather.apply(target_device, dim, *outputs)
  File "/home/dell/anaconda3/lib/python3.6/site-packages/torch/nn/parallel/_functions.py", line 54, in forward
    ctx.input_sizes = tuple(map(lambda i: i.size(ctx.dim), inputs))
  File "/home/dell/anaconda3/lib/python3.6/site-packages/torch/nn/parallel/_functions.py", line 54, in <lambda>
    ctx.input_sizes = tuple(map(lambda i: i.size(ctx.dim), inputs))
RuntimeError: dimension specified as 0 but tensor has no dimensions

My pytorch version is 0.4.0.

@JerryLauzzz were you able to test this model?

Yes. wizardliu<EMAIL_ADDRESS>签名由 网易邮箱大师 定制 On 09/07/2018 01:45, Jianwei Yang wrote: @JerryLauzzz were you able to test this model? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

I suffer from same error.

see issue #226

I tried the solution. Another error occured:
Traceback (most recent call last):
  File "trainval_net.py", line 331, in <module>
    optimizer.step()
  File "/home/dell/anaconda3/lib/python3.6/site-packages/torch/optim/sgd.py", line 101, in step
    buf.mul_(momentum).add_(1 - dampening, d_p)
RuntimeError: The expanded size of the tensor (512) must match the existing size (1024) at non-singleton dimension 1

Maybe I should open a new issue?

@JerryLauzzz
My environment is
python=3.6.5
pytorch=0.4.0 with cuda-9.0 and cudnn-v7' It seems a bug of pytorch 0.4.0 using multiple GPUs. But this solution solved my problem RuntimeError: dimension specified as 0 but tensor has no dimensions`

Yes, the solution solved the titled problem.
I think it's a bug either... If there is no update about this problem, I will close the issue.

Hi, @JerryLauzzz
Does multiple GPUs speed up the training process? I have tried the solution, but the speed is the same as the speed with only one GPU. And my GPU util is only 0%-2%. Are you the same as me? Thank you!


phantomjs is stopped sending new requests

Hi, all
Recently, i am use protractor + phantomjs to do e2e testing.
First,  i use chromedriver to test and passd.
But when i transfer to use phantomjs and found it always hanged at some specific url and not sending request any more. I researched  google and phantom documents and try to find what happened inside phantomjs, but it's really frustrate me when i try to debug to internal.
So my question is how can i fix this type of problems? Can you guys figure out a way for me to quickly solve this type of problems?
Thanks!

Thank you for submitting this ticket!
This issue tracker primarily serves as the tool to report and track bugs, defects, and development tasks.
For general question, code review request, or other related discussion, please post it to either:

the official PhantomJS users mailing-list
Stack Overflow (with phantomjs tag)

If you have not read it yet, please check also a few tips on Effective Q&A.


fix NextClear out of range problem

when we come to the last uint64 digit in NextClear,  maybe we should be aware that not all bits of this one are included.
For example, v:= bitset.New(100), only the first 36 bits of the last uint64 should be available. Otherwise, after
   for i := uint(0); i != 100; i++ {
           v.Set(i)
   }

would be somehow wired to return the 101th bit as an unset bit since the user only created a 100 length bitset.

Looks good to me.

Coverage increased (+0.5%) to 84.272% when pulling f8f4dd578d3bbc809545173e461c1c1d62581e68 on liaojie:master into 5c3c0fce48842b2c0bbaa99b4e61b0175d84b47c on willf:master.


PolymorphicAuthValueFactoryProvider issue when upgrading to v2

I am getting the following error after upgrading to v2
final AbstractBinder binder = new PolymorphicAuthValueFactoryProvider.Binder<>(                                                                                ^     reason: no instance(s) of type variable(s) T,E exist so that Binder<T> conforms to AbstractBinder   where T,E are type-variables:     T extends Principal declared in class Binder     E extends Object declared in method <E>of(E,E)
The code is pretty standard and was working before the upgrade.
final PolymorphicAuthDynamicFeature<?> feature = new PolymorphicAuthDynamicFeature<>(
            ImmutableMap
                .of(MyPrincipal.class, myfilter,
                    MyPrincipal2.class, myfilter2));

        final AbstractBinder binder = new PolymorphicAuthValueFactoryProvider.Binder<>(
            ImmutableSet.of(MyPrincipal.class, MyPrincipal2.class));

        environment.jersey().register(feature);
        environment.jersey().register(binder);

Please help.

Do your custom principal classes both implement Principal?

Yes

Okay. I was just curious of the error message related to the type unification.

I don't think it would have worked with the previous version if those classes did not implement Principal

Yes, you're correct. The error message was just a bit misleading.
I figured out, what your problem is. Since Jersey now uses its own InjectionManager for DI and doesn't rely on HK2's ServiceLocator any more, your AbstractBinder import should be incorrect now. The proper import for Dropwizard 2 is now org.glassfish.jersey.internal.inject.AbstractBinder.

@liriarte did this solve it for you?

@zUniQueX yes it did. many thanks!


Cleanup leftovers after moving PSP doc to concepts/policy

In #3215 doc was moved but YAML file was copied. This PR removes the old YAML file to reduce duplication and simplify maintenance.
PTAL @pweil- @steveperry-53
CC @simo5

This change is

PTAL @chenopis