Unit tests aren't discovered in sub-folders

Environment data

VS Code version: 1.45.1
Extension version (available under the Extensions sidebar): 2020.5.80290
OS and version: Arch Linux (kernel: linux 5.6.14.arch1-1)
Python version (& distribution if applicable, e.g. Anaconda): Python 3.8.3 64-bit
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): pipenv
Relevant/affected Python packages and their versions: unittest
Relevant/affected Python-related VS Code extensions and their versions: None
Jedi or Language Server? (i.e. what is "python.jediEnabled" set to; more info #3977): True
Value of the python.languageServer setting: Microsoft

Actual behaviour
In main tests/ directory are some tests and some other files needed. Those work fine until a subdirectory is created, which contains 2 tests named test_decorators.py and test_pagination.py, this subdirectory also contains the __init__.py so that the tests can be recognized. After this subdirectory is created, the tests from main directory (tests/) disappear and the ones from subdirectory are not displayed either, a popup saying that no tests were discovered and asking me to configure the test framework (It is configured properly)
Once the __init__.py file is added and I rediscover the tests, this box pops up.

Expected behaviour
When I add the subdirectory to main tests/ directory and search for the tests, both the tests from my main directory and the subdirectory should be found.
Steps to reproduce:

Create directory for tests in root of your project tests/
Configure the framework to use unittests and recognise files labeled: test_*.py
Add some tests in it and let the framework discover them [To this point everything is as expected]
Add a subdirectory in your tests/ directory and place some other tests in it
Run test discovery again
No tests were found

Logs

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help; turn on source maps to make any tracebacks be useful by running Enable source map support for extension debugging)

[Extension Host] Info Python Extension: 2020-05-24 16:13:27: getActivatedEnvironmentVariables, Class name = E, completed in 1ms, has a truthy return value, Arg 1: <Uri:/home/itsdrike/Programming/Python/Projects/Discord-Bot>, Arg 2: undefined, Arg 3: undefined

[Extension Host] Info Python Extension: 2020-05-24 16:13:27: > ~/.local/share/virtualenvs/Discord-Bot-j5RKKR3Y/bin/python -c "import unittestloader = unittest.TestLoader()suites = loader.discover("./tests", pattern="test_*.py")print("start") #Don't remove this linefor suite in suites._tests:    for cls in suite._tests:        try:            for m in cls._tests:                print(m.id())        except:            pass"

[Extension Host] Info Python Extension: 2020-05-24 16:13:27: cwd: ~/Programming/Python/Projects/Discord-Bot

Any ideas why this is happening or what ca I do to prevent it?

@ItsDrike, thanks for the report.  Test discovery definitely shouldn't be behaving this way.
What happens when you run ~/.local/share/virtualenvs/Discord-Bot-j5RKKR3Y/bin/python -m unittest discover -p 'test_*.py' -t '.' -s './tests'?
(roughly equivalent to the command from the log, ~/.local/share/virtualenvs/Discord-Bot-j5RKKR3Y/bin/python -c 'import unittest; loader = unittest.TestLoader(); suites = loader.discover("./tests", pattern="test_*.py"); print("start"); for suite in suites._tests:  for cls in suite._tests:    for m in cls._tests: print(m.id())')

When I run the first command (~/.local/share/virtualenvs/Discord-Bot-j5RKKR3Y/bin/python -m unittest discover -p 'test_*.py' -t '.' -s './tests') all tests are found as seen in gif below (you can ignore that log message)

but your second command has raised a SyntaxError:

Yeah, that second command (from the log file) is actually multi-line:
~/.local/share/virtualenvs/Discord-Bot-j5RKKR3Y/bin/python -c '
import unittest
loader = unittest.TestLoader()
suites = loader.discover("./tests", pattern="test_*.py")
print("start")
for suite in suites._tests:
    for cls in suite._tests:
        for m in cls._tests:
            print(m.id())
'

The big question is if the two command actually are equivalent.  I'm thinking they actually aren't.  The "top_level_dir" argument to loader.discover() wasn't passed in the logged script.  It defaults to the start dir ("./tests"), which I expect makes the tests fail.  The equivalent command would be ~/.local/share/virtualenvs/Discord-Bot-j5RKKR3Y/bin/python -m unittest discover -p 'test_*.py' -s './tests' (note the missing -t arg).  What happens when you run that?

So is it a bug or should I do something to fix it on my machine?

Right.  Those failures are what I would expect  You should see similar output in the "Python test log" output panel.  What do you see there?

The initial output was without init, with __init__.py there, It only really says start

Check the console log:  In the "Help" menu click "Toggle Developer Tools"; in that panel click on "Console"; look for related messages in that log.

Check the console log: In the "Help" menu click "Toggle Developer Tools"; in that panel click on "Console"; look for related messages in that log.

For some reason there was an error in the console when the file __init__.py itself was created (before I clicked on reload). I'll provide a gif and some static pictures of the console log for better readability.
GIF:

Pictures of the log:

(The highlighted log message in the last picture seems to be related.)

Thanks.  That's helpful information.  I don't see any reference to the Python extension's files in those stack traces.  However it is hard to be sure because the paths are shortened.  Would you mind pasting here the actual text of the two above error entries.  That should give us the full paths.  (You may also need to widen the developer tools panel first.)
FWIW, the above stack trace refers to a "...kite-extension.js" where the problem happens.  What other extensions do you have installed?

Thanks.  Do you have the problem when if you disable all the other extensions?

Yes the problem happens even with all extension (except Python) disabled:

Any progress on this?

Hi, I have the same problem with you. Exactly the same, in fact.
I assume the bug is due to that the "-t" option of unittest is not passed properly, just as you mentioned before.
I made a simple example project. The test_1 is stand-alone and can be discoverd, but the test_2 will raise an exception during import, which will cause itself undiscovered.
unittest_bug_example_project.zip
For now, I guess we have to code absolute imports (as the top directory was forced to ".") to use this function.

Any progress on this?

I have came back to this page for about ten times, but nothing changed.......
This bug makes the unittest function of this plugin almost useless. I have already disable it in my projects........

Any progress on this?

I have came back to this page for about ten times, but nothing changed.......
This bug makes the unittest function of this plugin almost useless. I have already disable it in my projects........

Yeah unit-testing doesn't work at all, I think this should have higher priority than it does now since testing is a very important part of every bigger python project and it's quite rare to have all of the tests in the same directory, not detecting this automatically is a pretty big issue, why is it taking so long to fix?

Is this issue still happening with the latest version of the extension?

@AtomicGu I used your sample repo but tests are discovered just fine.

I see the following in Python Test log output channel,
start
test_1.Test_NoImport.test_1_1
test_1.Test_NoImport.test_1_2
test_1_1 (test_1.Test_NoImport) ... ok
test_1_2 (test_1.Test_NoImport) ... ok
test_2 (unittest.loader._FailedTest) ... ERROR
NoneType: None

======================================================================
ERROR: test_2 (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: test_2
Traceback (most recent call last):
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.9_3.9.1776.0_x64__qbz5n2kfra8p0\lib\unittest\loader.py", line 436, in _find_test_path
    module = self._get_module_from_name(name)
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.9_3.9.1776.0_x64__qbz5n2kfra8p0\lib\unittest\loader.py", line 377, in _get_module_from_name
    __import__(name)
  File "c:\Users\karraj\OneDrive - Microsoft\Desktop\folders\unittest_bug_example_project\package\tests\test_2.py", line 3, in <module>
    from .. import app
ImportError: attempted relative import with no known parent package

----------------------------------------------------------------------
Ran 3 tests in 0.010s

FAILED (errors=1)

Can you send the output?

Note we're rewriting the whole testing code with https://github.com/microsoft/vscode-python/pull/16769, this will probably be fixed.

Should be fixed with the new testing work.


Provide button to remove items from an offer

This PR provides a simple button to remove items from an offer.

 Provide functionality to remove selected items from the offer


Wrong scaling when scaling > 1

Hyprland Version
Hyprland, built from branch main at commit c416880cf96aee783b6a00f356512e5d758e6056 dirty (shadow: correctly scale boxes). Date: Fri Dec 22 10:54:31 2023 Tag: v0.32.3-200-gc416880c  flags: (if any) debug debug
Bug or Regression?
Regression
Description
hyprctl monitors shows wrong scaling when using fractional scaling
Furthermore, the cursor cannot reach the right and bottom edges of the screen
git bisect says https://github.com/hyprwm/Hyprland/commit/37d2840246af43a7141e2a036820fc3dec468f01
How to reproduce

Set scaling to 1.15
Somehow on my monitor hyprctl monitors shows scaling as 1.11 instead
Try moving cursor to the right or bottom edge of screen. The cursor is blocked by an invisible wall and can't reach the edge

Crash reports, logs, images, videos
No response

expected

oh wait point 3 is an issue

check with above

So point 3 issue is fixed, but there's some new problems. I can't move the cursor between my two monitors anymore with my current config. Also my two monitors are not aligned anymore.

hm, yeah, thats a problem. Maybe instead of automatically correcting the scale we should pop an error?
The thing is you're creating a very un-nice scale. Your logical size is fractional, and nothing likes that. Hyprland corrects that to the nearest, valid one

I would like to have the previous behavior. I use a calculated scale so that my two monitors can be perfectly aligned.
For context this is my config
monitor=HDMI-A-1,preferred,0x0,1.75 # 4k monitor
monitor=DP-2,preferred,2194x0,1.1669 # 1440p monitor. Calculation: 3840 / 1.75 = 2194; 1440 / (2160 / 1.75) = 1.1669

Is there some way to make it work like before instead of showing an error or autocorrecting it?

well if your scale would be perfectly aligned to produce integer logical sizes then no problems should arise

2560 / 1.1669 = 2193.85
1440 / 1.1669 = 1234.04

If we round it then the logical size should be 2194 * 1234
And it should be aligned with the 4k because
3840 / 1.75 = 2194.28
2160 / 1.75 = 1234.29

which rounds to 2194 * 1234 as well

and this rounding causes issues. Logical sizes should divide cleanly into ints.

because #3511 #2618 and many more

Can we just round the logical size to an integer and store it as an integer then? Not sure what it was stored as before

Can we just round the logical size to an integer and store it as an integer then? Not sure what it was stored as before

that's what we do. The problem is all the other apps physically cannot render correctly.

I'll add an option to override this, but it really should not be set.

I don't really mind what happens under the hood, but I would very much like it to:

let me have something other than 1.25, 1.5 etc
let me align the two monitors

Right now it doesn't even let me have 1.75 though

it's because you cannot have 1.75 on 4K as 3840 / 1.75 is not an integer.

align in what way? as in, same logical size?

I simply want the logical size to be 2194, but there seems to be no way I can make it that. I tried 1.7502278942570648 and that didn't work either. It's fractional but I can only put a floating point number

uhhh yeeaa there is also the fractional scale protocol which demands scales are (int / 360)

So I tried 1.92 on 4k and that still didn't work. And it autocorrected it back to 1.00. But it should be an integer because
3840 / 1.92 = 2000
2160 / 1.92 = 1125

Is it cuz the fractional scale protocal or why is this

But tbh, I can live with that config option and just use that in the future. I just feel like there should still be a better way to handle this cause it was working so perfectly before except that 1 pixel weird behavior sometimes

I also observe an issue with >1 scaling after update, which didn't exist before.
My config is:
monitor = DP-1,2560x1440@165,0x0,1.33

The apps that like VS Code, Telegram were scaling nicely, and the rendered image was sharp.
After update, I get an error message with the same config saying that 1.33 is bad and the nearest valid scale value can be 1.333333.
I updated the config to use 1.333333. However, now the apps like VS Code and Telegram are blurry.
The mitigation is to use
debug {
  disable_scale_checks = 1
}

and old 1.33 scale value.
With both 1.33 and 1.333333 hyprctl monitors returns the same output:
Monitor DP-1 (ID 0):
<EMAIL_ADDRESS>at 0x0
	description: Dell Inc. DELL S2721DGF D6ZSR83 (DP-1)
	make: Dell Inc.
	model: DELL S2721DGF
	serial: D6ZSR83
	active workspace: 1 (1)
	special workspace: 0 ()
	reserved: 0 42 0 0
	scale: 1.33
	transform: 0
	focused: yes
	dpmsStatus: 1
	vrr: 0
	activelyTearing: false

odd that 1.33 is sharp but 1.333333 is not. 1.33 creates 1924.8x1082.7, fwiw.

See the screenshots:
1.33

1.333333

is it a thing for all wayland apps or just select ones?

Firefox and kitty seem to work fine with both scale values. So, only the selected apps.

ah, fractional scaling protocol usage most likely then.

I nailed down the issue, but it's midnight. I hope I wont forget to fix it tomorrow.

can you add a nearestneighbor windowrule to your affected apps and tell me how it looks?

Using nearestneighbor windowrule VS Code looks veery sharp :open_mouth: with both 1.33 and 1.333333. I believe even better than before with 1.33. Thanks! Also, some interesting screen tearing-like effect start to happen during animation between float/tiling mode. Probably, it cannot decide where to put that extra 1 pixel you mentioned. On the other hand, that's a minor nuance.

Can you record it?

1.333333 with nearestneighbor (look at the return 0; statement):
https://github.com/hyprwm/Hyprland/assets/12598936/ef1a2d66-793a-4e72-831c-6c1edd9423ab
1.333333 without nearestneighbor:
https://github.com/hyprwm/Hyprland/assets/12598936/fa6a0f15-3ce0-4026-8b4f-07f700d7aa19

yeah, I think we should just use nearestneighbor on fractional-scale surfaces. At least when still. Maybe in motion we could temporairly revert to bilinear. I'll see tomorrow. Thanks

BTW, I have set nearestneighbor for 3 apps so far to fix the blurriness with 1.333333 scale. It turns out that all of them are using Electron. Probably, filtering Electron apps may be useful.

yeah sorry I'll try to get this sorted soon, xmas, new year, hyprlang, too many things going on at once and I have to work on my laptop as I am back home for the winter break, away from my desktop.

check with above

I want to add that on my pc (monitor resolution of 2560x1600 and scale set to 1.5) it correctly warned that it's not a clean divisor, but instead of setting the scale to the recommended value of 1.6, I set it to auto. Weirdly enough, with the value set to auto it still tries to set a scale of 1.5, which I thought was odd.
No issues if I set the scale explicitly to 1.6 or 1.33.

oh oopsie daisy

check with above

The above commit fixed the issue, thanks!

It seems that b5b025a1ed6cfa2b94ad0eff38a247223edd95bb has forced extra/telegram-desktop into nearest neighbor for some reason...

uhh it will if fractional scaling + using fs protocol + buffer misaligned + not animated. Does it look wrong, or what?

It looks very wrong, I guess it's using the fractional scale protocol but sends a scale of 2, not sure

screenshot?
with the fractional-scale protocol you're not allowed to use a scale of 2 per protocol.

I think this is the result of downscaling via nearest neighbor.

looks rough indeed.

@vaxerski would it be better to put debug:disable_scale_checks under something like misc? the debug category makes it sound like it's a hack that you're not supposed to use and only for developers (which perhaps you think it is, but i think it's a valid config option, at least for me). This may prevent confusion in the future

@vaxerski debug:disable_scale_checks was not added to the wiki btw

woops

fixed

@vaxerski would it be better to put debug:disable_scale_checks under something like misc? the debug category makes it sound like it's a hack that you're not supposed to use and only for developers (which perhaps you think it is, but i think it's a valid config option, at least for me). This may prevent confusion in the future

nononono it's there for a reason. It should not be used by users. Really.

nononono it's there for a reason. It should not be used by users. Really.

what exact issue does it cause? i've been using it for a week and it's working very smoothly, 0 bugs

just because you don't see any doesn't mean they aren't there.
I won't be searching for exact ticket numbers but:

pixel gaps between windows
blurry surfaces using fractional-scale protocol
pixel gaps on wallpaper apps
just to name three


Enable MockCrossrefProvider for test environments

See #Slack.
Approach
As per @ebezzi :
1. create a factory method for it
1. create MockCrossrefProvider() with an embedded validation logic
1. if deployment_stage == "test" return new MockCrossrefProvider()

@ebezzi : Given the portal redesign, is this still relevant?

Closing as per @ebezzi


curl: (22) The requested URL returned error: 404

I run using docker-compose and get an error.
runner_1  | curl: (22) The requested URL returned error: 404
runner_1  | Unrecognized command-line input arguments: 'token'. For usage refer to: .\config.cmd --help or ./config.sh --help
runner_1  |
runner_1  | --------------------------------------------------------------------------------
runner_1  | |        ____ _ _   _   _       _          _        _   _                      |
runner_1  | |       / ___(_) |_| | | |_   _| |__      / \   ___| |_(_) ___  _ __  ___      |
runner_1  | |      | |  _| | __| |_| | | | | '_ \    / _ \ / __| __| |/ _ \| '_ \/ __|     |
runner_1  | |      | |_| | | |_|  _  | |_| | |_) |  / ___ \ (__| |_| | (_) | | | \__ \     |
runner_1  | |       \____|_|\__|_| |_|\__,_|_.__/  /_/   \_\___|\__|_|\___/|_| |_|___/     |
runner_1  | |                                                                              |
runner_1  | |                       Self-hosted runner registration                        |
runner_1  | |                                                                              |
runner_1  | --------------------------------------------------------------------------------
runner_1  |
runner_1  | # Authentication
runner_1  |
runner_1  | Invalid configuration provided for token. Terminating unattended configuration.
runner_1  | .path=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
runner_1  | Starting Runner listener with startup type: service
runner_1  | Started listener process
runner_1  | Started running service
runner_1  | An error occurred: Not configured
runner_1  | Runner listener exited with error code 2
runner_1  | Runner listener exit with retryable error, re-launch runner in 5 secon```

cat .env
```RUNNER_REPOSITORY_URL=https://github.com/company-name/repo-name
# or RUNNER_ORGANIZATION_URL=https://github.com/your-organization
GITHUB_ACCESS_TOKEN=ghp_******```

This is how I run it with plain Docker. Just make sure to set the correct environment variables.
docker run -it --name my-runner \
    -e RUNNER_LABELS=selfhosted,ubuntu-docker \
    -e RUNNER_NAME=selfhosted-ubuntu-docker \
    -e GITHUB_ACCESS_TOKEN=«my-pat-with-sufficient-permissions-for-repo» \
    -e RUNNER_REPOSITORY_URL=https://github.com/«my-org»/«my-repo» \
    -v /var/run/docker.sock:/var/run/docker.sock \
    tcardonne/github-runner:ubuntu-20.04

You might get the 404 because the token does not give access to the given private repo.
How does your docker-compose look like?


scarb :fire:

There seems to be a problem, there is no Output whatsoever
use debug::PrintTrait;
'Hello, World!'.print();

does nothing, output is:
Output:
====================

====================

Seems like instead of running programs with scarb, you only compile and the output is always empty
pub fn scarb_build(file_path: &PathBuf) -> anyhow::Result<String> {
    let crate_path = prepare_crate_for_exercise(file_path);
    let config = scarb_config(crate_path);

    match compile(&config, false) {
        // Here it returns an empty string and that's why there is no output
        Ok(_) => Ok("".into()),
        Err(_) => anyhow::bail!("Couldn't build the exercise..."),
    }
}

Hi @beeinger - thanks for investigating, shramee is on it!
https://github.com/shramee/starklings-cairo1/pull/171 should solve it

The prints should be above it, output is the output of run function main(). If your function returns something, it'll be shown here. Runner just prints the messages.

Oh, makes sense!
Already tried that branch, thx, also have a look at this: https://github.com/shramee/starklings-cairo1/pull/171#issuecomment-1820013733


XRay traces not connected to log entries in Cloudwatch

Describe the bug
We've migrated from Java X-Ray SDK to AWS OTEL. And the thing we are facing now is the absence of log records while analyzing traces. This worked with X-Ray SDK:

However, with OTEL, the logs are not mapped (we use Fluentbit). The only difference we've found in the metrics/traces messages (compared to XRay SDK) is the absence of the following key:
...
Segments: [
 {
  ...
  "document": {
   ....
   "aws": {
     ...
     "cloudwatch_logs": {
         "log_group": "our log group"
     ....

The aws.cloudwatch_logs.log_group is missing when using OTEL. The documentation says it translates from aws.log.group.names (https://aws-otel.github.io/docs/getting-started/x-ray#otel-span-cw-logs-metadata-translation).
Steps to reproduce

Use EKS Cluster with OTEL collector (Sidecar: public.ecr.aws/aws-observability/aws-otel-collector:latest) and the following configuration:

    extensions:
      health_check:
    receivers:
      otlp:
        protocols:
          grpc:
            endpoint: <IP_ADDRESS>:4317
          http:
            endpoint: <IP_ADDRESS>:4318
      awsxray:
        endpoint: <IP_ADDRESS>:2000
        transport: udp
    processors:
      batch/traces:
        timeout: 1s
        send_batch_size: 50
      batch/metrics:
        timeout: 60s
    exporters:
      awsxray:
        region: eu-west-1
      awsemf:
        region: eu-west-1
    service:
      pipelines:
        traces:
          receivers: [otlp,awsxray]
          processors: [batch/traces]
          exporters: [awsxray]
        metrics:
          receivers: [otlp]
          processors: [batch/metrics]
          exporters: [awsemf]
      extensions: [health_check]

Use Fluentbit to send logs to CloudWatch
Use SpringBoot application v2.7.0 with OTEL agent (1.17.0-aws)

Call a REST Endpoint (the endpoint logs to console with Log4J2). The log is visible in Cloudwatch, the log entry contains AWS_XRAY_TRACE_ID.
What did you expect to see?
I see a log entry in the trace detail (as with the previous XRay SDK)
What did you see instead?
I see traces without log information, however, the log entries are visible in Log groups in Cloudwatch.
Additional context
We've tried configuration using environment properties (AWS_LOG_GROUP_NAMES/ARNS), in OTEL_RESOURCE_ATTRIBUTES, in span attributes, however, with no luck. Are we missing some kind of configuration (pod roles?) here?

Hi @mpetrik-profinit, thanks for raising this! The problem appears to be that the Java EKS resource detector does not record the aws.log.group.names attribute. However the EKS Plugin in X-Ray SDK does with this logic.
We will add a backlog task to add this functionality. We are also open to a PR to copy in this functionality to the EKS Detector!

We've tried configuration using environment properties (AWS_LOG_GROUP_NAMES/ARNS), in OTEL_RESOURCE_ATTRIBUTES, in span attributes, however, with no luck. Are we missing some kind of configuration (pod roles?) here?

Could you elaborate more? What value did you set AWS_LOG_GROUP_NAMES to? Did the value show up on the X-Ray trace?

OTEL_RESOURCE_ATTRIBUTES does not support arrays: https://github.com/open-telemetry/opentelemetry-specification/issues/2857
I think our best option is to update the resource detectors while the issue with the spec is not solved. I will be working in adding the same logic that is used in the xray sdk to all sdks that implement resource detectors.

Moved discussion to upstream https://github.com/open-telemetry/opentelemetry-java-contrib/issues/564

In the next release of the aws-otel-collector, v0.26.0 , we will add a couple of features that will make your life easier.

Support for setting the log groups using the OTEL_RESOURCE_ATTRIBUTES environment variable: https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/17503
Add support to setting log group properties in the awsxray configuration in the collector https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/16859

Unfortunately it is not possible to solve this issue in the resource detectors as there is no way to implement the log group detection in a stable and reproducible way.

Hi,
As I mentioned in https://github.com/open-telemetry/opentelemetry-java-contrib/issues/564 it is not possible to implement resource detectors that will reliably work across all environments, therefore that option will not be implemented.
However, collector v0.26.0 was released. It is now possible to set the log group property using the two mechanisms described above. https://github.com/aws-observability/aws-otel-collector/releases/tag/v0.26.0
Please use the OTEL_RESOURCE_ATTRIBUTES variable to set the aws.log.group.names resource attribute.
Please refer to this section:
https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/exporter/awsxrayexporter#traces-and-logs-correlation
I'm going to close the ticket since the original problem was addressed.


Multidimensional arrays as parameters.

Currently Swagger 2.0 support specs like that:
{
    "swagger": "2.0",
    "info": {
        "version": "0.0.0",
        "title": "Simple API"
    },
    "paths": {
        "/{foo}": {
            "parameters": [
                {
                    "name": "foo",
                    "in": "path",
                    "type": "array",
                    "items": {
                      "type": "array",
                      "items": {
                        "type": "string"
                      }
                    }
                }
            ],
            "get": {
                "responses": {
                    "200": {
                        "description": "OK"
                    }
                }
            }
        }
    }
}

By default collectionFormat is csv and this create problems with interpreting strings like 'a,b,c,d'.
Should it be interpreted like:
[
  ["a","b","c","d"]
]

Or maybe like this:
[
  ["a", "b"],
  ["c", "d"]
]

It should be forbidden to create nested arrays with same collectionFormat.
This change will prevent undefined behaviour situation and could be done in 2.0 as bugfix.
JSON Schema couldn't enforce such constraints, but it could be specified in spec.

Link to parent #565

Done in https://github.com/OAI/OpenAPI-Specification/pull/804

@fehguy I don't actually see this in the specification ... where do we forbid this (which would be the corresponding structure to what @IvanGoncharov posted):
parameters:
  - in: path
    name: foo
    schema:
      type: array
      items:
        type: array
        items:
          type: string
    example: 'a,b,c,d'
    required: true

we don't forbid this, we support it by allowing a schema object as the type for the parameter.

The original issue said

It should be forbidden to create nested arrays with same collectionFormat.

Now there isn't a collectionFormat anymore, but the style property, which only applies on top level.
So for style: simple (which is the default for path parameters) there is still no way of knowing whether /a,b,c,d would be parsed as [["a","b"],["c","d"]], as [["a","b","c","d"]] or as any other structure.
There doesn't even seem to be a way of using different styles for both levels of array (which could solve this potentially).
So I think the issue is still open.


Inconsistent Terraform Plan vs. State for mssql_database Resource

Current Behavior:
When running terraform plan, the output indicates that the mssql_database.database resource will be updated in-place. Specifically, it shows a change in the name attribute from "another_database_name" to "same_database_name". However, upon inspection of the Terraform state, it shows that the name attribute is already "same_database_name".
Expected Behavior:
The terraform plan output should reflect the actual state in the Terraform state file. If the name attribute is already set to "same_database_name" in the state, it should not be flagged for update.
Steps to Reproduce:
Ensure that the Terraform state is as described in the provided state snippet.
Execute terraform plan in the project directory.
Observe the plan output for the mssql_database.database resource.
Additional Information:
Terraform Version: 1.5.7
Provider: registry.terraform.io/pgssoft/mssql - v0.6.0
There are more than 20 databases in the same state file, and this behavior is happening for 4 of them.
Terraform plan:
# mssql_database.database will be updated in-place
  ~ resource "mssql_database" "database" {
        id        = "21"
      ~ name      = "another_database_name" -> "same_database_name"
        # (1 unchanged attribute hidden)
    }

Terraform state
{
  "mode": "managed",
  "type": "mssql_database",
  "name": "database",
  "provider": "provider[\"registry.terraform.io/pgssoft/mssql\"]",
  "instances": [
    {
      "schema_version": 0,
      "attributes": {
        "collation": "Latin1_General_CI_AS",
        "id": "21",
        "name": "same_database_name"
      },
      "sensitive_attributes": []
    }
  ]
}

I assume that the issue happens if there is a database with a certain ID (in this case id = 21) deleted and then a new database created. Then instead of "creating a new database with a new id", the old database will be "overridden".

For us this happens with RDS when it failover to another node in MultiAZ setup. Somehow during replication ID of databases is not the same in nodes. So only option atm is to failback to initial node where terraform apply was executed or change id in state.

Maybe it makes sense to use the database name as the primary key within the datastructure instead of the ID? I am not familiar enough with MSSQL to determine whether this is possible, but it seems to be a (hopefully) easy solution to the issue.


Page Publications

[x] add page and
[ ] add exemplary data and include in page

Here are 2 different mockups for that page. I personally prefer the 1st one because each paper/talk has its own text and that would make for me more sense to describe each paper/talk than writing about the categories in general, like based on your mockup.
One other thing: I couldn't read the 4th link you wrote on your mockup (next to HTML). Maybe you can help me :)

I agree, the first option make more sense and is more inviting to the read to click one of the links. Vote: Option 1
The 4th link is called "bibtex". Me personally, I really like it when I can easily can access the paper metadata in bibtex format. For inspiration, look here.
One minor design aspect: the icons seem to be quite small. Maybe it would an idea to increase their size and use them as anchors while they are overlapping the box' border. Just a quick idea. Don't take it too serious ;-)
Could you please implement your mockup. I'll take care about binding the content from a publications.yml file to it afterwards.
For the first month, I would show any publications nor talks, because we don't have any. As temporary solution I would simple write 1-2 sentences in normal text including a link to publications listed on the open_eGo page.
Later, when we have publications. I'll remove the text, fill the .yml file with content, and this should appear then...

Status: CSS not implemented yet


Set the type of data returned by the functions

When using the command "php artisan rest:quick-start" a file called "UserResource.php" will be generated, it does not have the return type in its functions and it sends me an error of the type:
{
    "message": "Declaration of App\\Rest\\Resources\\User::relations(Lomkit\\Rest\\Http\\Requests\\RestRequest $request) must be compatible with Lomkit\\Rest\\Http\\Resource::relations(Lomkit\\Rest\\Http\\Requests\\RestRequest $request): array",
    "exception": "Symfony\\Component\\ErrorHandler\\Error\\FatalError",
    "file": "/var/www/html/app/Rest/Resources/UserResource.php",
    "line": 35,
    "trace": []
}

Thanks @devcodemarcos 😄, merging !


feat(context): add executionCtx to Context

When using Module syntax, there is no FetchEvent, only ExecutionContext (FetchEvent implements ExecutionContext's functions as well, waitUntil and passThroughUntilException).
This PR adds c.executionCtx, which is set on both Service Worker syntax and Module syntax. c.event is only used on Service Worker syntax.

Hi @hansottowirtz !
You are right. It would be better to have c.executeCtx. And the naming is good.
I'll merge it!


Support for deprecated attributes

This PR implements #182.
See the implemented tests regarding how the feature behaves.

merging this so we can get things ready for the release


Issues with GluonTS library (MXNet Error)

I'm running into an error when running the DeepAREstimator from the GluonTS library. I have downloaded all of the necessary packages, and with using the mx.gpu() function, it recognizes that the GPU (Nvidia Cuda 11.2) exists on the system
Note, that I'm able to run the code fine on Colab without having to run specific commands like installing "libquadmath0" or the NCCL library (ver 2.8.4 for cuda 11.2), but when running on a docker image, this seems to be the case.
Here's the full error trace:
`0%|          | 0/1 [00:00<?, ?it/s]
learning rate from "lr_scheduler" has been overwritten by "learning_rate" in optimizer.
0%|          | 0/1 [00:03<?, ?it/s]
MXNetError                                Traceback (most recent call last)
 in
29       print("\nPatient {} - Amount of Days (Train): {}\n | Amount of Days (Valid): {}\n".format(p_id, train_days, valid_days))
30
---> 31       train1_output = estimator.train(training_data=training_data, validation_data=validation_data)
32     #print(agg_metrics)
/usr/local/lib/python3.6/dist-packages/gluonts/mx/model/estimator.py in train(self, training_data, validation_data, num_workers, num_prefetch, shuffle_buffer_length, cache_data, **kwargs)
205             num_prefetch=num_prefetch,
206             shuffle_buffer_length=shuffle_buffer_length,
--> 207             cache_data=cache_data,
208         ).predictor
/usr/local/lib/python3.6/dist-packages/gluonts/mx/model/estimator.py in train_model(self, training_data, validation_data, num_workers, num_prefetch, shuffle_buffer_length, cache_data)
177             net=training_network,
178             train_iter=training_data_loader,
--> 179             validation_iter=validation_data_loader,
180         )
181
/usr/local/lib/python3.6/dist-packages/gluonts/mx/trainer/_base.py in call(self, net, train_iter, validation_iter)
377                         epoch_no,
378                         train_iter,
--> 379                         num_batches_to_use=self.num_batches_per_epoch,
380                     )
381                     if is_validation_available:
/usr/local/lib/python3.6/dist-packages/gluonts/mx/trainer/_base.py in loop(epoch_no, batch_iter, num_batches_to_use, is_training)
308                                 batch_size = loss.shape[0]
309
--> 310                             if not np.isfinite(ndarray.sum(loss).asscalar()):
311                                 logger.warning(
312                                     "Batch [%d] of Epoch[%d] gave NaN loss and it will be ignored",
/usr/local/lib/python3.6/dist-packages/mxnet/ndarray/ndarray.py in asscalar(self)
2583             raise ValueError("The current array is not a scalar")
2584         if self.ndim == 1:
-> 2585             return self.asnumpy()[0]
2586         else:
2587             return self.asnumpy()[()]
/usr/local/lib/python3.6/dist-packages/mxnet/ndarray/ndarray.py in asnumpy(self)
2564             self.handle,
2565             data.ctypes.data_as(ctypes.c_void_p),
-> 2566             ctypes.c_size_t(data.size)))
2567         return data
2568
/usr/local/lib/python3.6/dist-packages/mxnet/base.py in check_call(ret)
244     """
245     if ret != 0:
--> 246         raise get_last_ffi_error()
247
248
MXNetError: Traceback (most recent call last):
File "../include/mshadow/././././cuda/tensor_gpu-inl.cuh", line 129
Name: Check failed: err == cudaSuccess (209 vs. 0) : MapPlanKernel ErrStr:no kernel image is available for execution on the device`

@jfrank94  Which docker image did you use? Could you share more details about how to reproduce the error?

@TristonC The docker image version is 19.3.12, but it's running through a Kubernetes cluster on Azure. In order to reproduce this error, you should download these packages first on a Jupyter notebook:
!sudo apt install libquadmath0
!sudo apt install libnccl2=2.8.4-1+cuda11.2 --allow-downgrades -y
!pip install mxnet-cu112
!pip install --upgrade gluonts
!pip install scipy
!pip install xlrd==1.2.0
Then import the mxnet library as well as these libraries under GluonTS:
from gluonts.dataset.common import ListDataset
from gluonts.model.deepar import DeepAREstimator
from gluonts.mx.trainer import Trainer
from gluonts.dataset.util import to_pandas
from gluonts.evaluation import Evaluator
from gluonts.evaluation.backtest import make_evaluation_predictions
from gluonts.support.util import get_download_path
from gluonts.mx.trainer import Trainer
from gluonts.model.predictor import Predictor
Then, you can import a dataset and convert it into a ListDataSet like so:
training_data = ListDataset(
[{"start": train.index[0], "target": train.value}],
freq = "1H"
)
Finally, initialize the DeepAREstimator and train it like so:
estimator = DeepAREstimator(freq="1D", prediction_length=28, trainer=Trainer(epochs=60, ctx=ctx,
num_batches_per_epoch=1), num_parallel_samples=1)
train1_output = estimator.train(training_data=training_data, validation_data=validation_data)
I hope this helps!

Yeah, I could not reproduce the error in my local container with GPU. I don't have access to Azure. @leezu Could you comment on this?

@TristonC Maybe it might help to have the folks who worked on the GluonTS library help as well? Thanks for your help so far, btw.

The trace goes down to MXNet. But sure, @szha Any GluonTS friend can help on this issue?

cc @lostella

looks like the same as https://github.com/awslabs/gluon-ts/issues/1571

@TristonC @szha Are there any updates as of yet?

GluonTS support should go in awslabs/gluon-ts#1571 as mxnet doesn't have knowledge of the downstream application. If there's any unexpected behavior in mxnet, please share how that's reproduced in a bug report here.

@szha Alright, no problem. Thanks for your help.


Not For Merge Yet. Should we use Fragments or Activities on the Nav drawer?

I want to hear what approach you thin we should use for the Nav Drawer, Look at help and About for now.

A poll is on 👍 slack channel.


Control Key + Mouse Wheel Zoom

Can you guys add a feature for us to zoom into the text by holding the control key and using the mouse wheel? This feature is very handy and is available in almost all web browsers.

Never mind. I found the option in the settings. Good job guys


Support the "simple" password_scheme

Old CouchDB's and Cloudant only support the "simple" password scheme.
It'd be great if this library could be used with those.
For instance, I was using it on BigBlueHat/clusers, but had to remove it to get that user creation form working on Cloudant.
Thanks!
:tophat:

Hey @BigBlueHat, I’m super sorry I missed your issues, I haven’t watched this repository by accident and only found it now. Is this still an issue for you? Can you explain what you mean with "simple" password scheme? I guess we would need to add an {scheme: 'simple'} option and then it would store something else than a derived key and salt? Happy to accept a PR for it :)

@BigBlueHat are you still interested in the feature? Otherwise I’d close it

@gr2m sorry I didn't see the earlier notification. I think it's fine to close. Cloudant is now completely Apache CouchDB 2.x and the older CouchDB's will be upgrading...one would think. If/when I desperately need this, I shall send a PR. :smile: Sorry I didn't need it badly enough earlier to do that. :wink:
Cheers!
:tophat:


Improve unit tests

Codecov Report

Merging #14 (d7e75c6) into main (d2af84c) will increase coverage by 12.90%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##             main      #14       +/-   ##
===========================================
+ Coverage   66.66%   79.57%   +12.90%
===========================================
  Files           2       10        +8
  Lines          42      235      +193
===========================================
+ Hits           28      187      +159
- Misses         12       37       +25
- Partials        2       11        +9

Impacted Files
Coverage Δ

internal/datasource/storage/dynamodb.go
100.00% <100.00%> (ø)

internal/datasource/storage/s3.go
100.00% <100.00%> (ø)

internal/email/delete.go
80.00% <100.00%> (ø)

internal/email/get.go
60.41% <100.00%> (ø)

internal/email/list.go
76.08% <100.00%> (ø)

internal/email/trash.go
100.00% <100.00%> (ø)

internal/email/untrash.go
100.00% <100.00%> (ø)

... and 5 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Changing click version dependency due to conflict with wandb

Changing the click dependency version to be the same as wandb's click version https://github.com/wandb/wandb/blob/0b8243f8f8b107ec18f4638cc5ab0abccd7315c2/requirements.txt#L1.
Wandb's requirements file states that 8.0 is broken, but I didn't not look into why this is.
Tested the following on python versions: 3.7, 3.8, 3.9
pip install ./sparsezoo ./sparseml ./deepsparse
pip install wandb

Closing for now, as this requires an update to the latest black.


chore: Remove area/config duplicate entry in labeler.yml

Missed this in https://github.com/distribution/distribution/pull/4256
charts/registry/config/**/* doesn't exist so I'm assuming this should be only configuration. Kept the top entry so everything is nicely sorted

we should consider sorting this file alphabetically; that may make it slightly less likely for duplicates to be added accidentally

You can see an example workflow to sort JSON and YAML files here https://github.com/cloudquery/cloudquery/blob/e7c97482779c6847c157a7e5dcf7666ca8b53985/.github/workflows/auto_sort_files.yml
You might want to use a different approach to trigger (we do it on a schedule, and you can chose to do it on each PR)

A validation step is probably ok (not always a fan of bots opening PRs), but sorting the file "once" to set a baseline would probably already be a good start.

@erezrokah mind sorting the file alphabetically? I think it's a reasonable suggestion, though might be funky hard to enforce


iOS: 3d Touch actions

Ideas: add feed, all site stories, search (jumps to search bar in all site stories).
Thanks to https://twitter.com/sciriously/status/662068522722312193

Useful to know (from https://twitter.com/sciriously/status/662103571966521344)
"I use 3D-T on TweetBot (search), camera (straight to video), iBooks (specific book), Swarm (check in), Pinterest (trending)"

Click for a demo movie.
You can try it yourself in the iOS Simulator via the SBShortcutMenuSimulator project.

Can we have "Add Site" automatically focus the text box?

Or better yet, have it start on the folder selector (this should be true of any opening of the Add Site dialog. When opening, just immediately press the folder selector).


Support for smart motorway signs.

Describe the feature
Support on the live map for smart Motorway Signs by London studios.
Similar usage to smart signs.
Additional Context
No response

I currently do not have access to this resource so I will not be able to implement this any time soon.


System.Reflections.Emit Causes Crash on iOS

Just as the title implies, I'm getting a crash when using IL2CPP in System.Reflections.Emit.]
IL2CPP:DynamicMethod::create_dynamic_method - System.Reflection.Emit is not supported.
  at System.Reflection.Emit.DynamicMethod.CreateDynMethod () [0x00000] in <filename unknown>:0
  at System.Reflection.Emit.DynamicMethod.CreateDelegate (System.Type delegateType) [0x00000] in <filename unknown>:0
  at Newtonsoft.Json.Utilities.DynamicReflectionDelegateFactory.CreateDefaultConstructor[T] (System.Type type) [0x00000] in <filename unknown>:0
  at Newtonsoft.Json.Serialization.DefaultContractResolver.ResolveContract (System.Type type) [0x00000] in <filename unknown>:0
  at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.Deserialize (Newtonsoft.Json.JsonReader reader, System.Type objectType, Boolean checkAdditionalContent) [0x00000] in <filename unknown>:0
  at Newtonsoft.Json.JsonConvert.DeserializeObject
IL2CPP:DynamicMethod::create_dynamic_method - System.Reflection.Emit is not supported.
  at System.Reflection.Emit.DynamicMethod.CreateDynMethod () [0x00000] in <filename unknown>:0
  at System.Reflection.Emit.DynamicMethod.CreateDelegate (System.Type delegateType) [0x00000] in <filename unknown>:0
  at Newtonsoft.Json.Utilities.DynamicReflectionDelegateFactory.CreateDefaultConstructor[T] (System.Type type) [0x00000] in <filename unknown>:0
  at Newtonsoft.Json.Serialization.DefaultContractResolver.ResolveContract (System.Type type) [0x00000] in <filename unknown>:0
  at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.Deserialize (Newtonsoft.Json.JsonReader reader, System.Type objectType, Boolean checkAdditionalContent) [0x00000] in <filename unknown>:0
  at Newtonsoft.Json.JsonConvert.DeserializeObject
(System.String value, System.Type type, Newtonsoft.Json.JsonSerializerSettings settings) [0x00000] in <filename unknown>:0
  at Newtonsoft.Json.JsonConvert.DeserializeObject[T]

Just looking for a workaround if possible.

Could you check that you're using is not official Json.NET but Json.Net.Unity3D? Because Json.Net.Unity3D is supposed to disable DynamicReflectionDelegateFactory for iOS, this callstack seems odd. And it would be better to show me your class to be serialized and code for this.

Try using the original JSON.NET for Unity (it's free now).
This one has been abandoned for a while.


reporting: restructure RRD handling for health graphs

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Is your feature request related to a problem? Please describe.
rrd.inc is an ancient integration and should likely  be removed (and for the most part it already was on the development version).
Describe the solution you like
Replace with flexible drop-in scripts and common boilerplate handling magic.
Describe alternatives you considered
N/A.
Additional context
N/A

implemented in https://github.com/opnsense/core/pull/7836


fix: 940 hide image toggle

Move state isDeepZoomSourceValid into config reducer, so both Graph and Menubar components can use that state
Extract selector shouldShowOpenseadragon to client/src/common/selectors.ts to be used by the components above
Create config selector file

https://github.com/chanzuckerberg/single-cell-explorer/assets/6309723/7d12238b-e172-46f4-9d94-4decc71f3490

@kaloster I think Seve doesn't have time to review this PR, so PTAL thank you 🙏 !!


String.raw does not get parsed correctly

Bug Report

🔎 Search Terms
String.raw backthicks
🕗 Version & Regression Information
Always existed.
⏯ Playground Link
Full Context.
Playground link with relevant code
isolated:
https://www.typescriptlang.org/dev/bug-workbench/?#code/MYewdgzgLgBANgSzAVwB4wLwwMpQE5IDmAdHgIYDuABgMQCEA9AEZIMQAWAUAKLYDCAQQAK3ACIB9ABIB5bABVxQgXMkYAJAG8Z8xcskMDDADpGGAX04QApgBMYAWgQOrMAEQQG2hUpUM1vQREJL11fVzc1OQEAJQBxbgVRAElovxCAOQEAWW5iACsIcFdOKk4DGCyATxhQGxdCECsIGHYrPCsgA
💻 Code

const linux = String.raw`#!/bin/sh
  HOST_PATH=$DIR/native-messaging-example-host
  ESCAPED_HOST_PATH=${HOST_PATH////\\/}

  sed -i -e "s/HOST_PATH/$ESCAPED_HOST_PATH/" "$TARGET_DIR/$HOST_NAME.json"

  # Set permissions for the manifest so that all users can read it.
  chmod o+r "$TARGET_DIR/$HOST_NAME.json"

  echo "Native messaging host $HOST_NAME has been installed."
  `
// My code goes here

🙁 Actual behavior
it interpretes ${HOST_PATH////\\/} while it should ignore that as this is a raw string it gets preserved exactly as is.
🙂 Expected behavior
simply highlight and regonize it as string no errors.

Oh this can be maybe ignored as the final result of that string will be interpreted the behavior is correct.


Migrate from Quickcheck to hedgehog

Both cardano-ledger and cardano-api use hedgehog generators rather than QuickCheck. We would be able to reuse and upstream more of our generator code if we switched to hedgehog as well.

https://github.com/IntersectMBO/cardano-api/issues/702


Remove uses of .device in cirq-pasqal in preparation for larger circuit.device deprecation

First devices removal as a part of #4744 .
The high level plan for deprecation is to:

pull all components of the library off of .device and onto ._device to keep existing behavior working.
Deprecate dependent functionality that relies on .device.
After all dependency deprecations are done, deprecate the .device property in cirq-core circuits.py.
After the deprecation cycle is over, delete .device and _device properties from circuit, abstractcircuit, frozencircuit etc.

This PR is tackles cirq-pasqal.

Hi @MichaelBroughton ! I'm completely outside of the loop here, so it would be great if you could clarify some things.
If I understand correctly, you plan on dissociating the device from the circuit. Does this imply that circuit creation will no longer be constrained by device limitations, or will there be another mechanism put in place to replace this? If so, will the behaviour be identical? If not, why do you want to get rid of this functionality?
Thanks in advance!

Hi @HGSilveri thanks for the quick response:

If I understand correctly, you plan on dissociating the device from the circuit. Does this imply that circuit creation will no longer be constrained by device limitations, or will there be another mechanism put in place to replace this?

That's right. There will be no new mechanism put in place to meaningfully "attach" a device to a circuit. This does not stop higher level objects like PasqalSampler (or the cirq_google engine emulator) being constructed with devices and using them for circuit validation.
Going forward we expect users to call device.validate_circuit instead of doing the "online style" of circuit construction.

If so, will the behaviour be identical? If not, why do you want to get rid of this functionality?

Looking through the library it was found that there could be a lot of inconsistent and weird behavior in library code that used circuits that might have had devices attached. Two good examples of this are:

In cirq-aqt the run_sample function here relies on the associated device to set the number of qubits used in the state vector simulation. Users relying on len(circuit.all_qubits()) or other functionality could wind up being very confused by this inconsistency when analyzing the returned bitstrings.
In cirq-core if we look at things like optimizers we have some that require gates and qubits to function and others that require circuits (i.e. here and here). If I needed to use both of these methods (or any pair like this) when building/optimizing my circuit I would be ignoring my device constraints for one and considering them with the other while building/optimizing my circuit. In order to ensure my circuit was indeed valid I would likely still wind up having to call device.validate_circuit when I was finished modifying the circuit anyway.

Moving users to just needing to call validate_circuit at their discretion is how we plan to fix these problems.
I copied this section out of the design doc just in case there's access issues.

Hi @MichaelBroughton , thanks for the thorough explanation! It's a bit of a shame to see this feature go, but I understand your motives.

I copied this section out of the design doc just in case there's access issues.

Thanks! I do not have access to those, indeed.


My findings about how Chromium disrespects privacy

Hello,
I still haven't tried ungoogled-chromium but I wanted to share with you my findings about how Chromium connects to Google hosts regardless of all attempts to stop these connections.
I have reported full details about that behavior to Chromium's bugzilla but the developers tend to merely admit it without considering privacy a priority.
Has that issue been fixed in ungoogled-chromium? Or is it something you may probably be interested to look into?

I don't know. Those in the linked bug report are tweaking settings at a pretty high level (i.e. settings exposed by the UI), and a lot of our modifications are based on Wireshark traffic and digging around in the source code (mostly the latter).
If you do find a leak, please report it.

tweaking settings at a pretty high level

Yes. That's why the bug report is titled "Privacy disrespected (by default)".
As you can see my test is also based on monitoring network connections. Not using wireshark but tcpdump.

If you do find a leak, please report it.

I have already done this - not in ungoogled-chromium but in chromium itself. I thought you might be interested to test the same in your tweaked version and check if it is fixed. Considering you already have a compiled version it will be much easier than waiting for me to learn how to compile your version, then test it etc. Of course - I will, but it may take a long time and if it is an actual leak, it may still not be fixed while you wait for me. Hence the whole report here. I hope that clarifies.

Domain substitution covers the domains that you reported in the bug report (1e100.net, google*.com, and many more not mentioned in the report. So any domain names hardcoded into the source code will not be able to function correctly unless obfuscated.
I also modified Chromium to log if the browser attempts to connect with the modified domains, so I can disable the unnecessary code and any other related services.
I am certainly not claiming to have blocked everything, but I believe that most, if not all unwarranted connections have been disabled.


fix(ocm): call OCM only for kubernetes-cluster entities

Description
This is a temporary fix to not call OCM on entity.page.overview mount for entities without isType: kubernetes-cluster.
Will need to sync with someone from dynamic plugins about context mounting, as if is supported only for cards.
Fixes
https://issues.redhat.com/browse/RHIDP-2438

@kadel @gashcrumb can you answer @dzemanov question here? Maybe @dzemanov can give a bit more context

Right now, if user specifies context mounting, it will be always mounted, if I understand correctly. However, it would be good to not add context when it is not needed.
For example, for ocm setup:
    janus-idp.backstage-plugin-ocm:
          appIcons:
            - name: ocmIcon
              importName: OcmIcon
          dynamicRoutes:
            - path: /ocm
              importName: OcmPage
              menuItem:
                icon: ocmIcon
                text: Clusters
          mountPoints:
            - mountPoint: entity.page.overview/context
              importName: ClusterContextProvider  # <-- provider always rendered for every `entity.page.overview`
            - mountPoint: entity.page.overview/cards
              importName: ClusterAvailableResourceCard
              config:
                layout:
                  gridColumnEnd:
                    lg: "span 4"
                    md: "span 6"
                    xs: "span 12"
                if:
                  allOf:
                  - isKind: resource
                  - isType: kubernetes-cluster
            - mountPoint: entity.page.overview/cards
              importName: ClusterInfoCard
              config:
                layout:
                  gridColumnEnd:
                    lg: "span 4"
                    md: "span 6"
                    xs: "span 12"
                if:
                  allOf:
                  - isKind: resource
                  - isType: kubernetes-cluster

ClusterAvailableResourceCard and ClusterInfoCard will be rendered only if if condition is met - for cluster resources. However, context is rendered always, no matter the entity. This means, that e.g when user clicks on user entity and overview page is rendered for it, we make requests to OCM, although they are not needed. Furthermore, we make invalid requests because context provider expects certain entity type.
If you try and add this to context provider and mount it, you can see it is rendered in every entity overview page:
  return (
    <ClusterContext.Provider value={value}>

The OCM API has been invoked to retrieve cluster information.

      {props.children}
    </ClusterContext.Provider>
  );

Maybe someone from frontend team can confirm, but I think it would be safe to add context only if condition is met in DynamicEntityTab.tsx.
Maybe some changes in code here: https://github.com/janus-idp/backstage-showcase/blob/main/packages/app/src/components/catalog/EntityPage/DynamicEntityTab.tsx#L54-L55


c$ gas currency e2e tests

Description
e2e tests for c$ gas currency
Tested
All the tests pass with a local geth.
Other changes
None
Related issues
https://github.com/celo-org/celo-monorepo-old/issues/3107
Backwards compatibility
Is backward compatible

Still usefull  to merge or close out?

Still usefull to merge or close out?

It's useful to merge.  Let's keep it open.

Still usefull to merge or close out?

It's useful to merge. Let's keep it open.

Is this still the case or should this be closed out?

Closing as this PR hasn't had any activity. Please reopen if it is still relevant!


Deal with subtyping/inheritance in capnp interfaces.

So, I just got a basic program running that responds to a GET / via the normal net/http package API, but talking to the sandstorm api server. woo! but it involved this hack:
https://github.com/zenhack/go.sandstorm/commit/1d5b98ec3b91867145d9c83552eb0b4a8df3d5c5
In the capnp schema, WebSession inherits from UiSession, but the generated go packages don't reflect this in a useful way.

For servers, I copy the methods into the _Server interface to avoid https://golang.org/issue/6977.  Cap'n Proto interfaces allow multiple inheritance, so you can quickly get into the diamond problem.
For clients, I don't have a better answer than the "hack" that you used.  Go doesn't have an equivalent idea about subtyping for structs.  While the generated code could instead use interfaces, this would introduce inefficiency, and would be at odds with the fact that capabilities are loosely typed anyway: you don't whether a capability actually implements an interface until you call the methods.  Think of the generated client types as a weak hint.
That said, I'm sorry this is clunky.  This just seemed like the least clunky thing in the design space while I was looking at it six months ago.  If you think of a better way to do it, I'd love to hear it!

I'll mull it over.

One thing that might make this a little nicer: we could add convience methods for the parent interfaces, to do the conversion. e.g. you'd have:
myWebsession.ToUiSession()
which would basically do what my patch above is doing. These would be defined only for the parent types declared in the schema. You're still doing the upcast manually, but it's a bit less clumsy, and it adds some typesafety, since you can't use it to make an invalid conversion.
Thoughts?

I like that idea.  One thought: what should happen if an interface inherits from multiple interfaces of the same name? Certainly not a showstopper, as capnpc-go has generally punted on this, but I'd like that to get better over time.

I think I'm going to mull over #46 for a bit, keeping this in mind as well. Any solution to #46 would have to account for this as well.

Acknowledged. golang/protobuf solves this better; you should take a look there for inspiration.

In the context of this bug, we could use the interface IDs to differentiate multiple parents of the same name.

Not sure I see how that would work? My brain immediately goes to something like websession.To(UiSession_Type), but the types don't work at all. You could tack the actual number on the end of the To_ method, but that kinda sucks from an API perspective, and to get something readable you'd have to use a name, which puts you back at square one.
It might improve ergonomics in general to generate more concise names where a lack of collisions permits it. For example, Sandstorm's web-session schema defines a Response struct in the scope of WebSession, but there's no other type in the schema called Response. Right now the type gets translated to WebSession_Response, so you end up with the very verbose and redundant websession.WebSession_Response. It might make things nicer if the code generator could work out that it won't cause collisions to chop of the WebSession_ and do so, so you'd just have websession.Response.
With that change in place, we could maybe shove package names on the front of colliding interface names in the To* methods, relying on the fact that this is unusual to make it not suck in the general case.

My idea was more to make ToUiSession789abcdef methods, but as I go to write it out, it sounds awful.


update the database used by the webapp

@dstndstn argues that the sqlite3 database currently used by the web-app is not optimal for a few reasons. Let's use this ticket to track the requisite updates as we prepare https://fastspecfit.desi.lbl.gov for public release.

Significant progress on this ticket was made in the webapp-fuji-v2 branch of migrating the database to postgres. However, loading the 1.4M Fuji targets barfs with an out-of-memory error:
% time python load.py
Read 1397479 rows from /global/cfs/cdirs/desi/spectro/fastspecfit/fuji/catalogs/fastspec-fuji.fits
Row 1024
Row 2048
Row 4096
Row 8192
Row 16384
Row 32768
Row 65536
Row 131072
Row 262144
Row 524288
Row 1048576
Bulk creating the database.
Traceback (most recent call last):
  File "/global/homes/i/ioannis/conda-envs/fastspecfit-webapp/lib/python3.10/site-packages/django/db/backends/utils.py", line 89, in _execute
    return self.cursor.execute(sql, params)
psycopg2.OperationalError: cannot allocate memory for output buffer

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/global/cfs/cdirs/desi/spectro/fastspecfit/webapp/py/fastspecfit/webapp/load.py", line 1201, in <module>
    main()
  File "/global/cfs/cdirs/desi/spectro/fastspecfit/webapp/py/fastspecfit/webapp/load.py", line 1198, in main
    Sample.objects.bulk_create(objs)
  File "/global/homes/i/ioannis/conda-envs/fastspecfit-webapp/lib/python3.10/site-packages/django/db/models/manager.py", line 85, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/global/homes/i/ioannis/conda-envs/fastspecfit-webapp/lib/python3.10/site-packages/django/db/models/query.py", line 815, in bulk_create
    returned_columns = self._batched_insert(
  File "/global/homes/i/ioannis/conda-envs/fastspecfit-webapp/lib/python3.10/site-packages/django/db/models/query.py", line 1816, in _batched_insert
    self._insert(
  File "/global/homes/i/ioannis/conda-envs/fastspecfit-webapp/lib/python3.10/site-packages/django/db/models/query.py", line 1790, in _insert
    return query.get_compiler(using=using).execute_sql(returning_fields)
  File "/global/homes/i/ioannis/conda-envs/fastspecfit-webapp/lib/python3.10/site-packages/django/db/models/sql/compiler.py", line 1657, in execute_sql
    cursor.execute(sql, params)
  File "/global/homes/i/ioannis/conda-envs/fastspecfit-webapp/lib/python3.10/site-packages/django/db/backends/utils.py", line 103, in execute
    return super().execute(sql, params)
  File "/global/homes/i/ioannis/conda-envs/fastspecfit-webapp/lib/python3.10/site-packages/django/db/backends/utils.py", line 67, in execute
    return self._execute_with_wrappers(
  File "/global/homes/i/ioannis/conda-envs/fastspecfit-webapp/lib/python3.10/site-packages/django/db/backends/utils.py", line 80, in _execute_with_wrappers
    return executor(sql, params, many, context)
  File "/global/homes/i/ioannis/conda-envs/fastspecfit-webapp/lib/python3.10/site-packages/django/db/backends/utils.py", line 84, in _execute
    with self.db.wrap_database_errors:
  File "/global/homes/i/ioannis/conda-envs/fastspecfit-webapp/lib/python3.10/site-packages/django/db/utils.py", line 91, in __exit__
    raise dj_exc_value.with_traceback(traceback) from exc_value
  File "/global/homes/i/ioannis/conda-envs/fastspecfit-webapp/lib/python3.10/site-packages/django/db/backends/utils.py", line 89, in _execute
    return self.cursor.execute(sql, params)
django.db.utils.OperationalError: cannot allocate memory for output buffer

The load also takes a significant amount of time (>2 hours in an interactive Perlmutter node). The problem appears to be Django rather than postgres, so I'm not sure where the memory barf is happening.


Cannot get 8.4 FPS on a RTX3090

My colleague wrote a loop which runs inference on this model as fast as possible. We only achieve ~2.5 FPS on an RTX3090, and inference on a single instance of the model uses about 8GB of GPU memory.
Could you provide some more context about the performance figure of 8.4 FPS reported in the paper? Did you do any TensorRT conversion, or use any code that is not in this repo? Did you use python/pytorch? Was there any parallelization? Any info that I could use to get from 2 FPS closer to 8 would be really helpful.

Hi @armbuster,
TensorRT is not used in our experiments. The way we get the speed is to sum up the runtime of all components in our model, with the time for input voxelization excluded. We don't count the time for voxelization because you can easily write a faster kernel to achieve this without storing all points in each grid to the DRAM (for example, using the dynamic voxelization implementation in CenterPoint official codebase). I guess the reason why you cannot see the 8.4FPS runtime is the slow voxelization.
Besides, the sparse convolution implementation used in this repo is a very slow one, it could be accelerated a lot. We are actually working on a new TorchSparse release that supports this new feature.
Best,
Haotian

Thanks Haotian for the context!


[Feature Request] Add a toggle for Mark Read Automatically

Version 6.1 (6106)

I think it might be a good idea to add a toggle for Mark Read Automatically.
Currently, an article would be automatically marked as read the second you click on it.
This might not be a desired a behavior if you are simply scanning through your feed to find something you want to read later since you have to mark an article back to unread manually.
I hope you guys could consider adding the following features for this:

Add a time delay: an article would only be marked as read after ... seconds
Add a toggle: users have full control and always manually decide if they want to Mark Read
Add a selection menu like the following:

Hi,
I agree that a good feature, I opened a pull request https://github.com/Ranchero-Software/NetNewsWire/pull/3749


Tensor flow for Docker on Mac M1 pro

would this wheel work on M1 pro Docker , tried but it does not import tensor flow and result in no module found error.
=> [internal] load build definition from HelloWorld.dockerfile                                                                                0.0s
=> => transferring dockerfile: 410B                                                                                0.0s
=> [internal] load .dockerignore                                                                                0.0s
=> => transferring context: 2B                                                                                0.0s
=> [internal] load metadata for docker.io/library/python:3.9                                                                                0.4s
=> [1/9] FROM docker.io/library/python:3.9@sha256:4c41af7a9e87a725aa0dd3857ceb51d0deea8923ceb362aa38eaa5e525a23a21                                                                                0.0s
=> [internal] load build context                                                                                0.0s
=> => transferring context: 68B                                                                                0.0s
=> CACHED [2/9] RUN python -m pip install --upgrade pip                                                                                0.0s
=> CACHED [3/9] RUN pip install tensorflow==2.6.0 -f https://tf.kmtea.eu/whl/stable.html                                                                                0.0s
=> CACHED [4/9] RUN pip install pandas                                                                                0.0s
=> CACHED [5/9] RUN pip install matplotlib                                                                                0.0s
=> CACHED [6/9] RUN pip install -U scikit-learn scipy matplotlib                                                                                0.0s
=> CACHED [7/9] RUN pip install nltk                                                                                0.0s
=> CACHED [8/9] COPY intent.json /                                                                                0.0s
=> CACHED [9/9] ADD TrainChatBot.py /                                                                                0.0s
=> exporting to image                                                                                0.0s
=> => exporting layers                                                                                0.0s
=> => writing image sha256:31d4858dc926f4663b4c0dbdf7c61ccff4ae26d7b449b913e96f600d3ce82d55                                                                                0.0s
(base) karthik@192 Documents % docker images
REPOSITORY               TAG       IMAGE ID       CREATED         SIZE
     a1b2dadd4492   8 minutes ago   1.49GB
     31d4858dc926   3 hours ago     3.17GB
     f0dfd254a745   3 hours ago     3.17GB
     2fa2d8bf0130   3 hours ago     3.15GB
     06d19ce991f9   5 hours ago     1.41GB
grafana/grafana-oss      latest    e44a96a6d890   8 days ago      263MB
docker/getting-started   latest    adfdb308d623   8 weeks ago     27.4MB
(base) karthik@192 Documents % docker run a1b2dadd4492
WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested
Traceback (most recent call last):
File "//./TrainChatBot.py", line 9, in
import tensorflow as tf
ModuleNotFoundError: No module named 'tensorflow'
(base) karthik@192 Documents %

Any updates on this?

WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested

Looks like you were pulling an x86 image instead of aarch64.
And I'm afraid that Apple M1 does not perfectly runs wheels built for Linux, and I would strongly advise you to try the official wheels for M1 (though I haven't tried personaly).


New list option --include-injected shows injected packages in main apps

The main reason for this pull request is to have the ability to see the injected packages without having to examine the underlying pipx_metadata.json file stored in each virtual environment. The name is a work in progress so I would like to get some suggestions about what to do with that.
If there is anything else that could be a cause for concern, please let me know. Thank you.

@cs01 I have implemented your requested changes. Please feel free to look at it again.

Looks good besides merge conflicts

Thanks for the update. I am out of town at the moment but will review it when I’m back next week.


New send message endpoints logic

Divided message sending logic into different methods for different media types or if we don't have it at all. It will improve code quality a lot.
Created auth decorator to remove a lot of repeated code and make it easier to use for contributors
Added sample of web UI favicon to have something
Chat setting panel in UI is now hidden if no there is no active chat and display chat data when we have active chat

max-complexity moved 10>6


missing runtime dependency ?

Contained scripts require non described gems in gemspec as runtime dependency, such like below.

bunny
rest-client
stomp

These gems should be added to sensu-plugins-rabbitmq.gemspec, I think.

Fixed in master


Search term should not be retained when closing and reopening Settings/keybindings editor

Steps to reproduce
Keyboard Shortcuts - Second Situation:

Open Keyboard Shortcuts / Settings editor and provide a filter
Close the editor and reopen it

Expected: Filter is not retained
Observed: Filter is retained

@bpasero What is the current expected UX behaviour in this case?

@sandy081 depends. we typically retain as much view state as possible when reopening a file (e.g. cursors, folding).
I would not expect that settings or keybindings editor preserve the state of the filter though.

-- Sorry for the English, I used Google Translator --
@sandy081 I do not think you got what I said !!
It is not to save what I researched in the field when I close the editor. (Must be kept as is)
Not to save anything !!
What it is to change: What I said is that it still saves when you open settings, search something, close the settings tab and reopen again.
When closing the editor it really should not bring anything that was previously searched.
Let me know if you get it now !! Or if Google Translator betrayed me again.

@Tekbr I think the summary/description of this issue aligns to what you mentioned in above comment.

-- Sorry for the English, I used Google Translator --
@sandy081 Uhmm .. I put a gif tried to explain ...

Note: If Settings or Key are open when Reopening the VSCode these can keep what was searched.

@Tekbr I think the summary/description of this issue aligns to what you mentioned in above comment.
Anyway, it gets a boost. I think translation confused me with what @bpasero  wanted to say.

Sorry for the misunderstanding. 😊 😊

Filters are part of the state we retain when settings/keybindings editor are closed and opened. Closing this as designed.

-- Sorry for the English, I used Google Translator --
@sandy081 I still see with a problem. For a comparison, Atom does not keep the information after we close the Settings tab. I hope that in the future it will be changed.
If I closed the tab I do not want to search for what was typed.

I see Atom does it that way. But we have this behaviour for a long time and it is good (not many complaints). Not sure if I would like to change the behaviour now which is not buggy.

@sandy081 All right ... Anyway thanks for the feedback. 👍


Encoding and MINI Thermal Printer model ZJ-5802LD

Hello,
First of all thank you for the plugin, it works perfectly with the model described in the title.
There is a problem when it comes to the encoding, though.
I'm using the printer to print some text in Catalan, and so I have tried to print the following string of characters both with "UTF-8" and "ISO-8859-1":
"à á è é í ï ò ó ú ü ç À Á È É Ì Ï Ò Ó Ú Ü Ç {br}"
Here are the results:
UTF-8:

ISO-8859-1:

This printer is like a copy (its external shape is exactly the same) of this model: Black Copper MINI Thermal Printer BC-P58B, but Chinese.
Is there a workaround or a way to print these characters correctly using this printer?
Thank you in advance.
If you need anyting else to look into this just say the word.

The following code is used to print with the said printer in an older software that we have, but the person that coded it is no longer around.
It is in Delphi:
function TPrintBluetooth.PreparaText(T: string): string;
begin
LOGI('TPrintBluetooth.PreparaText');
T := AjustarAmpleText(T);
Result := #27 + '@';                // Esc@ to start printing
Result := Result + #27 + 'R' + #7;  // Spanish characters
case FSetCaracters of
TscWPC1252 : Result := Result + #27 + 't' + #16; // wpc 1252 character set Spanish (Western Europe)

TscISO88591: Result := Result + #27 + 't' + #23; // ISO 8859-1 character set Spanish (Western Europe)

TscISO88592: Result := Result + #27 + 't' + #36; // ISO 8859-2 character set

else         Result := Result + #27 + 't' + #23; // ISO 8859-1

end;
Result := Result + T;               // The text to print
Result := Result + #13#10;          // Line break to make sure it prints the last line
end;

Here is the printer programmer manual (it is a bit difficult to understand):
58MMPrinter Programmer Manual-20150312.pdf

i think your printer has not set to the codepage you wanted to print, try trigger a printer self test, it should trigger a printing of current printer's detail showing which codepage been set.
If the current codepage doesn't support the character you want to print, you need to find the relevant printer's setting tool to set the codepage first. you may do so by start looking at the printer manufacturer's website download page.

I managed to get it working some time ago by doing what you are saying.
Thank you a lot for your answer anyway!
Salutations,
El lun., 17 dic. 2018 13:32, csleeds<EMAIL_ADDRESS>escribió:

i think your printer has not set to the codepage you wanted to print, try
trigger a printer self test, it should trigger a printing of current
printer's detail showing which codepage been set.
If the current codepage doesn't support the character you want to print,
you need to find the relevant printer's setting tool to set the codepage
first. you may do so by start looking at the printer manufacturer's website
download page.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/giorgiofellipe/cordova-plugin-datecs-printer/issues/78#issuecomment-447829654,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AilIwHCxIpeQLwo6oSUn7fBLvfGkLykyks5u547MgaJpZM4R-AHE
.


404 for pub authorized

This issue was originally filed by<EMAIL_ADDRESS>
What steps will reproduce the problem?

publish a package
authorize

I expect authorization allowed, but instead reach a 404 at: http://pub.dartlang.org/authorized
Despite the 404, authorization was successful and the package was published. Shiny!

This issue has been moved to dart-lang/pub#380.


api: add "infiniband" api extension

Signed-off-by: Christian Brauner<EMAIL_ADDRESS>

@stgraber, would be good if you could give this review-priority. If we see that there are more extensive changes needed I don't want to wait shortly before the release especially since I want to take a few days off. :)

You're missing an entry for the new device type in the "Device types" table in doc/containers.md.

On Tue, Dec 05, 2017 at 11:04:22PM -0800, Christian Brauner wrote:

brauner commented on this pull request.

	idx = strings.LastIndex(devName, ".")

	if idx == -1 {

		return fmt.Errorf("Invalid device name \"%s\"", devName)

	}

	relativeDestPath := devName[idx+1:]

	relativeDestPath = strings.Replace(relativeDestPath, "-", "/", -1)

	destPath := fmt.Sprintf("/%s", relativeDestPath)

	dummyDevice := types.Device{

		"path": destPath,

	}

	err = c.removeUnixDevice(fmt.Sprintf("infiniband.unix.%s", funName), dummyDevice)

	if err != nil {

		return err

	}

}

Right, took me a while to track down, the mix of the infiniband (global) and infinibandUnixDevices (local) maps makes it pretty hard to track down.

The commit message specifies in great detail how that is supposed to work and why it was done this way. :)

I wonder if we couldn't simplify that whole mess by just passing the per-interface file twice in this case.
So say you have ib0 and ib1 in your devices, you'd end up with unix.ib0.dev-infiniband-uverbs0 and unix.ib1.dev-infiniband-uverbs1 and then handle the situation in insertUnixDevice and removeUnixDevice so that they become no-op if another unix.*.PATH already exists.

How do you handle the case where you remove the device that first added that per-port device that is mounted into the container? Everytime you remove a unix device you do a setns() into the container to check whether the device is mounted? If it is leave it on-disk and mounted if it isn't wipe it?. If that's the case then you still have to do the removal logic that we have here since you need to identify any stale on-disk per-port devices each time you remove any infiniband device. But now it becomes less nice since you don't now the device of the device anymore that first created the per-port on-disk device since you wiped it from the database. It's also not a true representation of the per-port concept on-disk since you pretend it's per-port but you still treat it as per-fun.

You just look at /var/lib/lxd/devices/CONTAINER/, no need to look inside
the container. As /dev/uverbs0 will be listed as both
unix.ib0.dev-infiniband-uverbs0 and unix.ib1.dev-infiniband-uverbs0 in
there, you can trivially see that it still needs to be there, in which
case you remove the unix.ib0.dev-infiniband-uverbs0 file from
/var/lib/lxd/devices/CONTAINER but don't actually touch the container.
If ib1 is removed later on, the same check would only find one file for
the dev-infiniband-uverbs0 path which means it needs to be removed from
the container.
Same logic when adding it, if we find a file for it already under
another device name, then we don't do anything to the container.

This would also fix this case:
lxc config device add c1 kvm path=/dev/kvm
lxc config device add c1 kvm1 path=/dev/kvm
lxc config device remove c1 kvm
By effectively just dealing nicely with a user specifying the same device twice, first one wins and we don't remove until they're all gone.

I don't understand we always failed on this and I don't see why mounting the same device to the same path should be valid.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/lxc/lxd/pull/4067#discussion_r155159397

--
Stéphane

@stgraber, ready for another review.


chore: run poetry lock

update lock file

Let's update deps in one PR next time
Updating them one by one and then resolving conflicts in poetry.lock is not fun


Update Readme

The readme is pretty outdated... Would be nice to have this up to date!
—
Created via Raycast

Mention two seperate ways to use this:
Primary use case is CLI but can also use this within python.

https://github.com/marcderbauer/bloom
This could be used for reference. I'm very happy with how it looks 💯


Implementing layer-level normalization of gradients similar to neural_artistic_style?

When people are discussing style transfer projects, I have seen people mention that the  "weird muddy areas" they see on artwork produced by Neural-Style, are an issue which can be solved. Is this something that is possible for Neural-Style, seeing as Neural_Artistic_Style already solved this problem? How serious is this issue with Neural-Style?
The gradient normalization on neural_artistic_style being discussed in more detail: https://github.com/andersbll/neural_artistic_style/issues/39#issuecomment-223208827

Regarding the visual style, I perform layer-level normalization of gradients. In the VGG-net, the features in each layer may exist on different scales. By L1-normalizing the gradient signals, I get a more even contribution across the different layers. I suspect this is the secret sauce. :)

L1 normalization of gradients coming from content or style losses has been in neural-style for a long time; you can just add the flag -normalize_gradients.

Neural_artistic_style also seems to use scaling of gram matrices to
compensate for different sizes of content and style images. If I
understood the code correctly, it does not scale the size of the matrix,
but only the values in the matrix. I might give this a try as I have
already experimented with variations of the gram matrix. But this is not
likely to have a major impact on quality; to me it seems more like an
automatic adjustment of style weights depending on the image sizes.

@jcjohnson is there any samples of using -normalize_gradients with the commands? I'm getting just blurred results with the following command:
th neural_style.lua -style_image $HOME/sync/styles/style7.jpg -content_image $HOME/sync/iteration1/selfie2.jpg -output_image $HOME/sync/style7_sw150_sc15_norm.jpg -image_size 600 -backend cudnn -normalize_gradients


ZoomEye - Adding maximum number of items per page check in API

Issue: #333

Uggh, I didn't see your pull request and ended up raising one myself. https://github.com/projectdiscovery/uncover/pull/335

Uggh, I didn't see your pull request and ended up raising one myself. #335

You can choose a more suitable merged :-)

Closing as https://github.com/projectdiscovery/uncover/pull/335 is merged.


Angular 1.3 & Angular Bootstrap 0.13

Hi, I think the latest release of this plugin (0.6) specifies the bower dependencies as:
"angular": "^1.3.15"
"angular-bootstrap": "^0.12.1"

As far as I know, Angular Bootstrap didn't support Angular 1.3 until version 0.13, which was [recently released] (https://github.com/angular-ui/bootstrap/releases/tag/0.13.0).
This leads to warnings about potential code incompatibility. E.g.
angular-bootstrap#0.13.0 incompatible with ^0.12.1

Resort to using angular-bootstrap#~0.13 which resolved to angular-bootstrap#0.13.0
Code incompatibilities may occur.

Is there any plan to support angular-bootstrap 0.13 in the near future?
Thanks, and thanks also for the work going into this plugin!

Updated, thanks for letting me know. As far as I'm aware, this library has no issue with angular-bootstrap#0.13.0, and can be assumed to work with future versions of angular-bootstrap unless there is a breaking change for modals.


Enable import of full ART package

Is your feature request related to a problem? Please describe.
It would be nice to be able to import the full ART toolkit using import art
However, the import structure does not allow this:
import art
wrapper = art.classifiers.KerasClassifier(model=dummy_model)

Returns AttributeError: module 'art' has no attribute 'classifiers'
See:
https://colab.research.google.com/drive/1X3U0xm0uEEwWDM4njwwbl9aDK5Jk9OeO
I would be willing to submit a PR for this but wanted to check with the maintainers to see if there is a reason for this.

Hi @seanpmorgan Thank you for opening this issue. I don't think there is a specific reason why we don't support it so far. It would be great if you could make a PR for this. I'll assign this issue to you.


composer test results in at least 1 skipped test

Running composer test results with:
PHPUnit 4.8.36 by Sebastian Bergmann and contributors.
Warning:        The Xdebug extension is not loaded
                No code coverage will be generated.

...............................................................  63 / 405 ( 15%)
............................................................... 126 / 405 ( 31%)
............................................................... 189 / 405 ( 46%)
.............................S................................. 252 / 405 ( 62%)
............................................................... 315 / 405 ( 77%)
............................................................... 378 / 405 ( 93%)
...........................

Time: 6.17 seconds, Memory: 22.00MB

OK, but incomplete, skipped, or risky tests!
Tests: 405, Assertions: 501, Skipped: 1.

There is 1 skipped test in there. The skipped test should be either fixed or removed.

This was just the fluent test which is a suggested package, that's why the test was skipping. It works at expected.


API routes + Database for posts

Incl. creating data model + databases + linking with merkle tree builder

populate a db + common logic that populates frontend

The first iteration of nymjs is merged into main https://github.com/personaelabs/nym/commit/49c6841bcdba42f00ec064d63271bbc1e086c7cd.  Now integrating the API backend with nymjs!


Posting on Wordpress when using this plugin gives error message

Error: "Failed to connect to 2001:bc8:3890:a100::1: Network is unreachable" - Code: 0
on an otherwise blank white page. The URL of the page with the error message on it is http://www.polyinpictures.com/wp-admin/post.php and I got there from http://www.polyinpictures.com/wp-admin/post-new.php by clicking "Publish".
The post was posted to my Wordpress blog, though, and the toot was sent to my account on octodon.social.

Hi !
Your are not in the right repository, this is the google and firefox extension here.
Please try the new version of Mastodon Share for WordPress, some bugs fixed
I close this issue.


Reduce missing translation warnings

Several changes where made to the WAYF generation logic to reduce the amount of missing translation warnings.

Only load the IdP name for the current locale. This gets rid of the missing PT translation on our platform where PT is not an active language. This should solve similar issues for OpenConext users that use only one or two of the supported languages.
Translated the required translation id
Lowered the log level for this missing IdP name log entry.

See: https://www.pivotaltracker.com/story/show/176817180

Ah, the mighty JS test suite once again saves us from a regression! Will fix that right away.

Why was the required translation added? I don't see it being used in the changes, was it generating spam in the logs?

Yes, it logged a warning on every WAYF build.


Interactive learning navigation documentation

The current interactive learning navigation consists of the following options:
Do you want to stop?  (Use arrow keys)
   Continue
   Undo Last
   Fork
   Start Fresh
   Export & Quit

I couldn't find any documentation on any of these navigation options.
Some of these are intuitive (continue, undo last, export & quit).
But for others it is just not clear what choosing these options entails (Fork, Start Fresh).
From other issues here on GitHub I could gather what these options are supposed to do but what I wish for is an addition to the documentation where all navigation options are described.
For example:

Start Fresh: Starts a new conversation and story. The last story will still be saved.

This is still an issue

well, would you be willing to submit a PR for it?

@akelad I did not planned to. Doesn't mean it is not an issue worth improving. Don't get me wrong but this is your product and not a community project. You are responsible to develop it. I'm just reporting flaws.

well, it is somewhat a community project in that we allow contributors to contribute. I was simply asking you whether you wanted to make a contribution

@akelad Sorry for the misunderstanding. I thought you meant that either I'm resolving this issue or it is gonna be closed. Thanks for the clarification.


Import issue with pytorch

Works fine in the local environment but on deploying to aws lambda it gives an error in the zappa logs.
import torch isn't working.
 import torch
 File "/var/task/torch/__init__.py", line 56, in <module>
from torch._C import *
ModuleNotFoundError: No module named 'torch._C'

Has anyone figured this out? I have been stuck on this for a bit.

+1

@souravjamwal77 I have created an issue here https://github.com/zappa/Zappa/issues/1220  which is of similar nature.


Outdated link

Both of the (2) links on this page referring to the OSes "supported by Azure" point to an obsolete link. Is there a more up-to-date source for this information?

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

ID: c721f5ae-ece4-58ab-d00d-65856907c364
Version Independent ID: 7912f48e-25be-6efb-e770-c30404d84c58
Content: Support matrix for Hyper-V replication to Azure
Content Source: articles/site-recovery/hyper-v-azure-support-matrix.md
Service: site-recovery
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

@dcbrown16 Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

I removed the link. I can't find a more updated source. Should publish today.
#please-close

https://support.microsoft.com/en-us/help/2721672/microsoft-server-software-support-for-microsoft-azure-virtual-machines
@rayne-wiselman I later found this link that seems to be the correct current KB for Azure Windows support. Linux has a supported distro page in Azure somewhere.


Optimize user endpoints

Feature Request
Desired behaviour
Some endpoints are getting pokey and are doing too much in memory despite relatively low CPU use in general.
Take a look at converting functionality in
https://staging.dockstore.org/api/users/users/entries?count=10
https://staging.dockstore.org/api/users/user/extended
https://staging.dockstore.org/api/users/3/workflows
to better use the DB by using named queries instead of computing in memory.
┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Fix Versions: Dockstore 1.9
┆Sprint: Seabright Sprint 29 Dugong
┆Issue Number: DOCK-1153

For the first two endpoints, may also be able to use a technique similar to BioWorkflowDAO.java, where only the relevant columns are fetched, reducing the number of joins in the DB query.

#978 related, might want to look into storing commit SHA to see if it has changed before updating


Handle case when s3 config key is not a dict

This fixes a regression for the error case when a user
has an invalid CLI config file.  Examples include:
[default]
s3 = foo

[profile foo]
s3 =
signature_version = s3v4

Previously, we would check that the 's3' key was a dict type
before trying to access values.  This puts back the old
behavior.
Long term, we're going to need something that properly validates
the config keys and gives good error messages for these conditions.
Without this change you'd get this:
Traceback (most recent call last):
  File "aws-cli/awscli/clidriver.py", line 186, in main
    return command_table[parsed_args.command](remaining, parsed_args)
  File "aws-cli/awscli/clidriver.py", line 379, in __call__
    return command_table[parsed_args.operation](remaining, parsed_globals)
  File "aws-cli/awscli/clidriver.py", line 548, in __call__
    call_parameters, parsed_globals)
  File "aws-cli/awscli/clidriver.py", line 665, in invoke
    verify=parsed_globals.verify_ssl)
  File "botocore/botocore/session.py", line 814, in create_client
    client_config=config, api_version=api_version)
  File "botocore/botocore/client.py", line 62, in create_client
    verify, credentials, scoped_config, client_config)
  File "botocore/botocore/client.py", line 182, in _get_client_args
    new_config = Config(**config_kwargs)
  File "botocore/botocore/config.py", line 95, in __init__
    self._validate_s3_configuration(self.s3)
  File "botocore/botocore/config.py", line 132, in _validate_s3_configuration
    addressing_style = s3.get('addressing_style')
AttributeError: 'str' object has no attribute 'get'

cc @kyleknap @JordonPhillips

:ship:

:boat:


Hi

Hi
How to change Typeface this RingButton?
Thank you.

I'm sorry. It can't change the typeface.
But I think you can add this feature by youself. It isn't very difficult.
Also I'll help you, if you have any question. :)

Thank you!

Thank you for your guide.
But I can't add typeface for ringbutton because RingButton extends View
and View dosen't have typeface.


Types are not highlighted in FSI

Current master

I've got a simple fix for this - it's because structs aren't being mapped to a particular colour.

It works for me:

I'm just about to submit a PR.


Forked and maintaining - Updated to go mod and updated xtermjs => 1.17

I'm leaving this PR here as an option for anyone looking to use gotty. I'm maintaining it in at the current feature set while I've updated to use go mod and I've updated the xtermjs library version to the most current.
Thanks to yudai for making an excellent piece of software!

It also looks like this Fork[1] is actively maintained as well. I didn't find it because it wasn't forked proper from this repository and that's what I was looking at to try and find the actively maintained fork.
[1] https://github.com/sorenisanerd/gotty


[semantic-arc-opts] Convert exclusive accessed load [copy] that are n…

…ever consumed by a non-destroy value into a load_borrow.

@swift-ci smoke test

@swift-ci smoke test

Keep in mind that this is just creating the scaffolding for this optimization and verifies that the machinery works by implementing the simplest case (load [copy] of an in_guaranteed parameter). I will expand it over time.

@atrick You may want to take a look at this post commit.


Footer added

#230

@Anjalis14 please solve the conflict and then try to push

@YadavAkhileshh conflict has been solved and i pushed it