File operations on Windows with Python 3.13 give Permission denied

Steps to reproduce:

Take the server example from the tutorial and run it (I replaced '' with <IP_ADDRESS> for simplification but kept the rest).
Run the builtin ftp.exe, issue open localhost 2121, log in with either of the users defined in the server code. Issue dir, so far it works.
Issue get <any existing file>.

What happens:

the client says "550 Permission denied"
the server doesn't log anything helpful even with the debug log level, except clearly showing that it responded with 550 to RETR.

Same with STOR or APPE (though it looks like it responds "550 File exists" for STOR, even if it doesn't exist).

That means you can't open that specific file for reading. It's a problem of Windows permissions about that file / directory. You should fix it at system level, either by changing permissions for that specific user or changing the user.

That means you can't open that specific file for reading.

I can.

It's a problem of Windows permissions about that file / directory.

Nope.
Also, I now see how this could be seen as a stupid newbie question, especially as I forgot to specify that the same code works with the same files with Python 3.12 so I'm not surprised that it was quickly closed but please reconsider.

OK then please show your code and also the logs (or traceback, if any).

As I said the code is at https://pyftpdlib.readthedocs.io/en/latest/tutorial.html#a-base-ftp-server
The server logs:
INFO:pyftpdlib:concurrency model: async
INFO:pyftpdlib:masquerade (NAT) address: None
INFO:pyftpdlib:passive ports: None
DEBUG:pyftpdlib:poller: 'pyftpdlib.ioloop.Select'
DEBUG:pyftpdlib:authorizer: 'pyftpdlib.authorizers.DummyAuthorizer'
DEBUG:pyftpdlib:handler: 'pyftpdlib.handlers.type'
DEBUG:pyftpdlib:max connections: 256
DEBUG:pyftpdlib:max connections per ip: 5
DEBUG:pyftpdlib:timeout: 300
DEBUG:pyftpdlib:banner: 'pyftpdlib based FTP server ready.'
DEBUG:pyftpdlib:max login attempts: 3
INFO:pyftpdlib:>>> starting FTP server on :::2121, pid=2560 <<<
INFO:pyftpdlib:::1:51618-[] FTP session opened (connect)
DEBUG:pyftpdlib:::1:51618-[] -> 220 pyftpdlib based FTP server ready.
DEBUG:pyftpdlib:::1:51618-[] <- OPTS UTF8 ON
DEBUG:pyftpdlib:::1:51618-[] -> 530 Log in with USER and PASS first.
DEBUG:pyftpdlib:::1:51618-[] <- USER user
DEBUG:pyftpdlib:::1:51618-[] -> 331 Username ok, send password.
DEBUG:pyftpdlib:::1:51618-[user] <- PASS ******
DEBUG:pyftpdlib:::1:51618-[user] -> 230 Login successful.
INFO:pyftpdlib:::1:51618-[user] USER 'user' logged in.
DEBUG:pyftpdlib:::1:51618-[user] <- EPRT |2|::1|51619|
DEBUG:pyftpdlib:::1:51618-[user] -> 200 Active data connection established.
DEBUG:pyftpdlib:::1:51618-[user] <- RETR foo
DEBUG:pyftpdlib:::1:51618-[user] -> 550 Permission denied.
DEBUG:pyftpdlib:[debug] call: close() (<DTPHandler(id=1338375264240, addr='::1:51618', user='user')>)

I don't have a Windows box at the moment, but your error originates from here:
https://github.com/giampaolo/pyftpdlib/blob/53f3f66a969b605491573659c011322e6fab1839/pyftpdlib/handlers.py#L2422-L2427
What fails is the open() call, that's why I confirm that my suspicion is the path or the directory doesn't have enough permissions for your user. Try to add a raise statement at line 2425 and you should see the full file path + traceback.

  File "C:\Users\wRAR\AppData\Local\Programs\Python\Python313\Lib\site-packages\pyftpdlib\handlers.py", line 2427, in ftp_RETR
    fd = self.run_as_current_user(self.fs.open, file, 'rb')
  File "C:\Users\wRAR\AppData\Local\Programs\Python\Python313\Lib\site-packages\pyftpdlib\handlers.py", line 1971, in run_as_current_user
    return function(*args, **kwargs)
  File "C:\Users\wRAR\AppData\Local\Programs\Python\Python313\Lib\site-packages\pyftpdlib\filesystems.py", line 222, in open
    return open(filename, mode)
PermissionError: [Errno 13] Permission denied: 'C:\\Users\\wRAR\\Documents'

Looks like it tries to open a dir for reading.

pyftpdlib.filesystems.AbstractedFS("C:\\foo", None).ftpnorm("bar") returns "/bar" on 3.12 but "/" on 3.13, because os.path.isabs("/bar") is True on 3.12 but not on 3.13.

I'm gonna enable CI tests for python 3.13 and see if they turn red.

They do (this is good):

https://github.com/giampaolo/pyftpdlib/tree/py3-13-tests
https://github.com/giampaolo/pyftpdlib/actions/runs/11444456956/job/31839377779

This should now be fixed. Please confirm it worked if you have the chance.

Since this was a critical issue, I've just released pyftpdlib 2.0.1.


Calico manifest not working in the current walkthrough

Hi,
I'm trying to setup the master node according to this wakthrough: https://github.com/coreos/coreos-kubernetes/blob/master/Documentation/deploy-master.md
The problem is that I think that the Calico manifest yaml definition is wrong, because it contains a config map, a replica set and a daemon set...
I'm getting this error in my kubelet log:
Jan 05 11:00:35 coreos-kube-master-01.novalocal kubelet-wrapper[908]: E0105 10:00:35.335205     908 file_linux.go:107] can't process config file "/etc/kubernetes/manifests/calico.yaml": /etc/kubernetes/manifests/calico.yaml: read '# This ConfigMap is used to configure

.... the yaml definiton from the guide ...

Jan 05 11:00:35 coreos-kube-master-01.novalocal kubelet-wrapper[908]: ', but couldn't parse as pod(invalid pod: &api.ConfigMap{TypeMeta:unversioned.TypeMeta{Kind:"", APIVersion:""}, ObjectMeta:api.ObjectMeta{Name:"calico-config", GenerateName:"", Namespace:"kube-system", SelfLink:"", UID:"", ResourcceVersion:"", Generation:0, CreationTimestamp:unversioned.Time{Time:time.Time{sec:0, nsec:0, loc:(*time.Location)(nil)}}, DeletionTimestamp:(*unversioned.Time)(nil), DeletionGracePeriodSeconds:(*int64)(nil), Labels:map[string]string(nil), Annotations:map[string]string(nil), OwnerReferences:[]api.OwnerReference(nil), Finalizers:[]string(nil), ClusterName:""}, Data:map[string]string{"cni_network_conffig":"{\n    \"name\": \"calico\",\n    \"type\": \"flannel\",\n    \"delegate\": {\n      \"type\": \"calico\",\n      \"etcd_endpoints\": \"__ETCD_ENDPOINTS__\",\n      \"log_level\": \"info\",\n     \"policy\": {\n          \"type\": \"k8s\",\n          \"k8s_api_root\": \"https://__KUBERNETES_SERVICE_HOST__:__KUBERNETES_SERVICE_PORT__\",\n          \"k8s_auth_token\": \"__SERVICEACCOUNT_TOKEN__\"\n      },\n      \"kubernetes\": {\n          \"kubeconfig\": \"/etc/kubernetes/cni/net.d/__KUBECONFIG_FILENAME__\"\n      }\n    }\n}", "etcd_endpoints":"http://<IP_ADDRESS>:2379,http://<IP_ADDRESS>:2379"}}).

I'm running CoreOs 1185.5.0 and Kubernetes 1.5.1
Am I doing something wrong or is there an error in the guide? It seems to me that kubelet manifests should only be defined as pods, aren't they?
Thanks

@Rastusik yeah, this looks like a bug in the documentation.  The kubelet will only handle Pods from that directory, not ConfigMaps, etc.
I think the fix should be to kubectl apply that file rather than putting it in the kubelet config directory.  Looks like there is a PR open already to address that here: https://github.com/coreos/coreos-kubernetes/pull/799

I also ran into this and the fix is indeed to not copy these files to /etc/kubernetes but the kubectl create -f ... them.

@Rastusik I'll get the docs updated this week, the PR @caseydavenport referenced closed, but as stated the manifest should not live in the /etc/kubernetes/manifests/ directory and needs to be deployed with kubectl apply -f calico.yaml
Thanks for catching this.

i also ran into this issue. maybe it's also worth noting the difference between the manifest folder and kubectl apply, because that still isn't very clear to me.

@stephanlindauer The /etc/kubernetes/manifests/ path is for static pods see here for more info.  The short of it is Kubernetes will periodically scan that directory and deploy or update pods based on the manifests in there, which seems great for something like Calico, but it does not support deploying anything but pods, and we have a ConfigMap in the Calico manifest, so it fails to deploy anything in it.

@heschlie ah ok. understood. thanks for clearing this up. my use case is deploying a pod that is privately hosted on docker hub and therefore needs a secret to be created before. this forces me to curl everything as json to localhost on the master node once port 8080 is available. :/

The docs are not updated yet.


Remove duplicate results when activating multiple ImageViewer modules

Feature description
ImageViewer(name="One plot", img_globs=["my_amazing_plot.png"]),
ImageViewer(name="All Plots"),

It would be nice if "my_amazing_plot.png" would not appear twice on the dashboard page.
Proposed solution
Perhaps one one module is activated, it could check the output from the other modules and eliminate the duplicates?

Hmm. I see your point, but I think I would advocate for users to set specific globs for each ImageViewer module, or only use one with the default setting (all jpg/png/... files). Can you help explain why you chose your approach over one of those alternatives?

I often have many .pngs as output from postprocessing (>10 / job) and showing them all by default would result in a lot of clutter. So I select a few (2-3) as separate ImageViewer modules and add a general ImageViewer for the rest. Except it's not really the rest, it's the duplicates plus the rest :)
However this is not a big issue, I can definately work around it if the implementation is costly.

Another alternative would be to change the behavior of img_globs to allow a callable. The callable would act on filenames in the job directory as a filter (True means show, False means hide). This would make it easier to construct a custom set of images that excludes previously shown images.

Yes, sounds easier than setting up inter-module communication.


if results["IsErroredOnProcessing"]: TypeError: string indices must be integers

Error on run

I don't think returning a string was in the documentation, so I assume something is going wrong.  Can you add some more logging so you can see what it is returning and if their is an error message?  Then handle appropriately, whether that is exiting or retrying.


[WFCORE-3793]: Allow for the provisioning of SecurityRealm without a 'running' elytron

Jira: https://issues.jboss.org/browse/WFCORE-3793

Core - Full Integration Build 7291 outcome was UNKNOWN using a merge of 6fa88ffb7250e11049a8b4c1c844fce8a3f2748e
Summary: Canceled (Error while applying patch; cannot find commit 3b8330dde8c42ffbe1f9e9a410efdb74806d7a40 in the https://github.com/wildfly/wildfly-core.git repository, possible reason: refs/pull/3250/merge branch was updated and the commit selected for the ... Build time: 00:00:23

retest this please

Core - Full Integration Build 7293 outcome was UNKNOWN using a merge of 2e59026c2b08e616121991f71c1908e1b1f0fe66
Summary: Canceled (Error while applying patch; cannot find commit 16a03f5dcd4609466fabcba21d699af28ddefff1 in the https://github.com/wildfly/wildfly-core.git repository, possible reason: refs/pull/3250/merge branch was updated and the commit selected for the ... Build time: 00:00:14


Blurry Font Rendering

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.63.2 (Electron 13.5.2,  Chromium 91.0.4472.164)
OS Version: Windows 10 (NT x64, 10.0.19043)

Steps to Reproduce:

Just run VS Code.

I have seen this posted here before for previous versions (< 1.52). Font rendering is very blurry. So far, I've tried:

Disabling full screen optimizations;
"--disable-gpu --enable-use-zoom-for-dsf"
Adjusting ClearType.

Here are some examples using Source Code (Variable):

As I have seen posts dealing with this before, it is surprising that it has not yet been fixed. Also, I've seen an outrageous Microsoft "help" page saying scaling is "hit or miss" (!!!). As I need to perform extensive work using either VSCode or another similar software, I simply cannot play "hit or miss" with my eyes—and that is true for every user.
I appreciate if you can give any suggestions on how to improve this.

Solved by

Solved by using another font. Issue probably font-related?


Allow dynamic file names for generated SQLite

After the generation of SQLite files, I have to rename them prior to using them in my project, as it requires SQLite databases to follow a specific naming convention.  It would be ideal if I could specify a template string in the XML configuration file that would construct the name of the file dynamically based on the presence of supported tokens.  It's important that it supports multiple tokens in the template string.
Examples of replacements:
%h = 2-digit hour in 24-hour format
%H = 2-digit hour in 12-hour format
%a = AM or PM
%i = 2-digit minute
%s = 2-digit second
%d = 2-digit day of the month
%m = 2-digit month of the year
%Y = 4-digit year (e.g., 2005)
%y = 2-digit year (e.g., 05)
%f = Windows file time
%u = unix time
%g = random GUID
%t = prompt for user text

Partially completed in commit 74dccb158e9344706b6768034a394176a9d356c9


Netsh helper problems

Describe the bug
When installing from the latest MSI, netsh no longer works.
This blocks the 0.4.0 pre-release.
OS information
No response
Steps taken to reproduce bug
Download the MSI from a recent build artifact (e.g. from a recent scheduled build run)
Install the MSI.
Run "netsh".
C:\Temp>netsh

Invalid command name: (null)

Invalid command name: (null)

Then run the MSI again to uninstall.
Netsh now works again.
Expected behavior
netsh should work
Actual outcome
see repro steps
Additional details
No response

this is a dup of #1390


too much time spent "resolving" text

macOS Version(s) Used to Build
macOS 12 Monterey
Xcode Version(s)
Xcode 14
Description
Run FlowDemo under a profiler. Use the stress test.
Crash Logs, Screenshots or Other Attachments (if applicable)

Hopefully we can call resolve and cache the results.


posting right in JMAP for subaddressing

i am tackling the step 4 of this issue, that is, adding JMAP support for subaddressing
in order to allow subaddressing or not for a specific mailbox, there is the need for a p (posting) right, that can be granted to the identifier anyone

third Container startup failed for image opensearchproject/opensearch:2.14.0 error in a row
i don't think i have the right to retrigger the build, can i leave it to you @chibenwa ?


Avg shape function clarification

I was wondering if you have a tutorial or more documentation on how to use the utilities.avg_shape_spt() function. I'm confused by the required inputs, and I'm not sure what the output is.
Thanks!

Hi @jg9zk ,
The usage is not well-demonstrated in the tutorials. Thanks for bringing this to our attention. We will add a demonstration to the tutorials. Possibly the code related to that functionality needs to be rewritten a little bit for ease of use. (Note to self: Users typically won't store the coupling matrices between all cells in a large dataset, because this is too much data to store on disk. Users should be able to easily compute the coupling matrices between all cells in a small cluster, and use this as an argument to avg_shape_spt.)
What it returns is also not well documented in the docstring, so that needs to be fixed.
Essentially what it returns is a pair of

a graph, encoded as an adjacency matrix
a vector of scalar intensity values, which we call "confidence" values

The graph is supposed to represent an "average morphology" for the cluster, whose topology and branching structure are an amalgamation of the cells in the cluster, a topological idealization of the cells in this cluster.
There are several plotting tools which can plot such a graph using force-directed layout to position the nodes in a readable way, and color them by the confidence values. Networkx includes such a function. The confidence values are higher at nodes where the neurons in the cluster could be aligned well, and lower at nodes where the nodes could not be aligned well. So, the brightly colored parts of the visualization are where the "signal" is (representative aspects of the morphology across the cluster) and the dark parts are variability within the cluster (parts of different cell morphologies which are in conflict with each other.)


add google ads source

@AdamSulek please add a google ads source as we discussed

you can use the test account: https://developers.google.com/google-ads/api/docs/first-call/overview
note the credentials.json file should be in this format: https://dyvenia.github.io/viadot/howtos/config_file/

I think this won't be a priority for the upcoming months.


Missing options in search dialog

My search dialog has these options:

Suchen in (search in: document, project, solution etc.)

Externe Elemente einschließen (include external elements)
Sonstige Dateien einschließen (include other/misc. elements)

Dateitypen (file types)

The two options in the middle of my list are not explained. What are they good for? Why was the first unchecked and the second prechecked?
I have the latest stable version of VS 2019 16.8.

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

ID: 0e27881f-04bc-b7ba-e217-8360278a14f8
Version Independent ID: d31c8050-339a-688c-869e-ff7942fd3f74
Content: Find in Files - Visual Studio
Content Source: docs/ide/find-in-files.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@ygoe Thanks for your feedback. We are working on updating our documentation and should have it published very soon. Meanwhile, please see the Modernizing Find in Files blog post for more information about the Find in Files feature changes.

@ygoe Thanks for your feedback. We are working on updating our documentation and should have it published very soon. Meanwhile, please see the Modernizing Find in Files blog post for more information about the Find in Files feature changes.

Thanks for the info. And, oh well, those are bad descriptions in the UI.

When searching a solution, there are checkboxes to include miscellaneous files (files that you’ve opened but aren’t part of the solution) as well as external items (files like “windows.h” that you might reference but aren’t part of the solution).

Let me suggest the following alternatives that might avoid the user's way to the documentation altogether:

Include opened documents not part of the solution
Include referenced files not part of the solution

Because who knows what somebody was thinking of when writing "miscellaneous" and "external". It could be anything, really.

Thanks for the info. And, oh well, those are bad descriptions in the UI.

When searching a solution, there are checkboxes to include miscellaneous files (files that you’ve opened but aren’t part of the solution) as well as external items (files like “windows.h” that you might reference but aren’t part of the solution).

Let me suggest the following alternatives that might avoid the user's way to the documentation altogether:

Include opened documents not part of the solution
Include referenced files not part of the solution

Because who knows what somebody was thinking of when writing "miscellaneous" and "external". It could be anything, really.

@ygoe Thanks for your inquiry, and apologies for our delayed response. We've updated the page to include descriptions of the UI (user interface) and how each option might be useful to folks.
As to your ideas about how we could optimize the actual in-product UI, please consider using our Suggest a Feature option, where product issues and suggestions are triaged and acted upon.
Meanwhile, we'll close this issue, with our thanks to you for helping us improve the documentation.

Done: https://developercommunity.visualstudio.com/t/Change-search-dialog-texts-to-make-them/1491709


[Bug] DSHuds Issue Latest Version

Describe the bug
When updating to OpenGlasses-MC1.12.2-2.2-38 or greater, the light display and chunk borders from Dynamic Surroundings: HUDs are not visible anymore. It works with the previous OpenGlasses-MC1.12.2-2.2-25 though.
Minecraft:

OpenGlasses-MC1.12.2-2.2-38
forge-1.12.2-<IP_ADDRESS>38
DynamicSurroundingsHuds-1.12.2-<IP_ADDRESS>BETA

Additional context
I am using OptiFine as well. OptiFine settings did not appear to have any effect on this regression, though I have not yet tested whether issue occurs without OptiFine so it may be a contributing factor.

should be fixed with this version:
http://ci.pc-logix.com/job/OCGlasses/41/artifact/build/libs/OpenGlasses-MC1.12.2-2.2-41.jar
also updated commons lib to support a light version of thermal vision when fastrender is enabled
https://www.curseforge.com/minecraft/mc-mods/commons0815/files/2812359

Closing this as there wasn't any feedback, feel free to open a new bug report referencing to this if the issue persists for you or comes up again.


287 Upgrade to vert.x sql client 3.9.2

I think I've found a bug in the vertx client for MySQL.
Disabling the query cache solves the issue.
We have a query that has a Buffer as parameter. If the first time we run the query the parameter is null, the client will infer the type in DataTypeCodec#inferDataTypeByEncodingValue as DataType.NULL instead of DataType.BLOB. It seems that at that point the value is cached and therefore the encoding changes depending on when you run the query.
For now I've disabled the cache for MySQL so that we can upgrade.
@aguibert Would you be able to have a look at this, please?

I guess it's not surprising that there's a regression, since I know a bunch of work was done in that area in this release (partly as a result of me reporting a couple of bugs).

When https://github.com/hibernate/hibernate-reactive/pull/290 is merged, I'm going to enable the tests that will work with this version of the client.

I've created this issue on the vert.x sql client: https://github.com/eclipse-vertx/vertx-sql-client/issues/719

I'm merging this one. It's pretty simple and we can re-enable the cache for MySQL when they've fixed the bug

Done https://github.com/hibernate/hibernate-reactive/pull/293


Indicate focus group performance

Based on the "grade" fields of all students in a group, determine how the focus group is performing.
Split into good, OK, or poor.

Decided not to implement feature


PocketCast(s) changed app name

Slipped after 1.2.1 update.

Thank you for the contribution. It needed some fixes, so they were made in 54284cd336eaecc3fa0683a9a72e66c5abf36982. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


Add Bose Soundtouch component

Description:
Add support to Bose Soundtouch multi-rooms speakers (https://www.bose.com/en_us/products/speakers/wireless_speakers/soundtouch-10-wireless-system.html).
This PR add support for the following media supports:

DLNA
Radio
Spotify

I'm unable to test others (Google Music, Deezer, Apple Music, ...) because I don't have any accounts but I should be able to create trial accounts in order to implement theses providers in the future.
Multi-room is fully supported.
I'm not a native Python developer and I didn't know all quality software tools used by Home Assistant (Tox, Pylint, ...) so I tried to do my best but any feedback are welcome.
I have errors with Tox but I guess nothing related to this PR.
Related issue (if applicable): No
Pull request in home-assistant.github.io with documentation (if applicable):
I'll start working on the documentation after submitting this PR
Example entry for configuration.yaml (if applicable):
Defined devices
media_player:
  - platform: soundtouch
    host: <IP_ADDRESS>
    port: 8090
    name: Soundtouch Living Room
  - platform: soundtouch
    host: <IP_ADDRESS>
    port: 8090
    name: Soundtouch Kitchen
...

Play media
Soundtouch devices support 6 presets who can be used in order to change the selection
- service: media_player.play_media
  data:
    entity_id: media_player.soundtouch_living_room
    media_content_id: <1-6>
    media_content_type: PLAYLIST

Multi-room
In order to use multi-room ('Zone' in soundtouch vocabulary):
Create a zone and play on all other slaves
automation:
  - alias: Play everywhere
    trigger:
       ...
    action:
      - service: media_player.soundtouch_play_everywhere
        data:
          master: media_player.soundtouch_living_room

Creating a zone with one master and two slaves
- service: media_player.soundtouch_create_zone
  data:
    master: media_player.soundtouch_living_room
    slaves:
      - media_player.soundtouch_room
      - media_player.soundtouch_kitchen

Adding a slave to an existing zone
- service: media_player.soundtouch_add_zone_slave
  data:
    master: media_player.soundtouch_living_room
    slaves:
      - media_player.soundtouch_room

Removing a slave from a zone (removing all slaves delete the zone)
- service: media_player.soundtouch_remove_zone_slave
  data:
    master: media_player.soundtouch_living_room
    slaves:
      - media_player.soundtouch_room

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io => Not yet

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc. => Unit tests added so I didn't update this file

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Ok, I just discovered 2 new tools: flake8 and pep257 :)
Sorry for the spam ...
About the failings jobs in Travis, it seems there is a race condition in the test file test_device_sun_light_trigger
I'm able to reproduce locally about 10% of the time even on the dev branch (so without my PR) with the message:
Error inside async loop: Task exception was never retrieved

But I can't explain why I'm the only one to have this unit test error on my PR (lack of luck ? Am I missing something ?). Any help would be more than welcome.

I've addressed the device sun light trigger flakiness in PR #4247, we should not see it anymore.

I synced the dev branch but I still have the same issue: https://travis-ci.org/home-assistant/home-assistant/builds/173721401

Closing this to give some time for extracting the protocol into a lib as discussed in https://github.com/home-assistant/home-assistant/pull/4242#discussion_r87688807


CORS is not working

When setting SITE_DOMAIN to https://my.cousins-matter.tld and using a reverse proxy in front of cousins-matter (fronting lighttp), there is an error:
Forbidden (Origin checking failed -https://my.cousins-matter.tld does not match any trusted origins.): /members/login/
so we have to add the cors django application and the CORS setup in settings.py which is based on SITE_DOMAIN.

Actually, this is not based on SITE_DOMAIN, we'll use CORS_ALLOWED_ORIGINS


Memory leaks in gRPC versions 1.18 and 1.3

Hi All,
Does anyone know of any fixes for the following memory leaks seen in gRPC versions 1.18 and 1.3?
Note: we see these leaks when running valgrind on the process. There is steaming activity on the gRPC server in all of these leaks.
In gRPC version 1.18,
==19520== 9,838 (6,963 direct, 2,875 indirect) bytes in 1 blocks are definitely lost in loss record 1,220 of 1,254
==19520==    at 0x4EDA0F3: malloc (vg_replace_malloc.c:300)
==19520==    by 0xE904549: gpr_malloc (alloc.cc:57)
==19520==    by 0xE904764: gpr_malloc_aligned (alloc.cc:93)
==19520==    by 0xE8B8798: gpr_arena_alloc_maybe_init(unsigned int) (arena.cc:61)
==19520==    by 0xE8B873A: gpr_arena_create(unsigned int) (arena.cc:157)
==19520==    by 0xE8A9903: grpc_call_create(grpc_call_create_args const*, grpc_call**) (call.cc:332)
==19520==    by 0xE8755FA: accept_stream(void*, grpc_transport*, void const*) (server.cc:868)
==19520==    by 0xE81B46B: grpc_chttp2_parsing_accept_stream(grpc_chttp2_transport*, unsigned int) (chttp2_transport.cc:804)
==19520==    by 0xE805D95: init_header_frame_parser(grpc_chttp2_transport*, int) (parsing.cc:622)
==19520==    by 0xE805250: init_frame_parser(grpc_chttp2_transport*) (parsing.cc:292)
==19520==    by 0xE80469F: grpc_chttp2_perform_read(grpc_chttp2_transport*, grpc_slice) (parsing.cc:177)
==19520==    by 0xE822FF6: read_action_locked(void*, grpc_error*) (chttp2_transport.cc:2527)
==19520== 5,696 (5,120 direct, 576 indirect) bytes in 32 blocks are definitely lost in loss record 1,208 of 1,254
==19520==    at 0x4EDA0F3: malloc (vg_replace_malloc.c:300)
==19520==    by 0xE904549: gpr_malloc (alloc.cc:57)
==19520==    by 0xE8C5227: grpc_raw_compressed_byte_buffer_create (byte_buffer.cc:39)
==19520==    by 0xE8C51D0: grpc_raw_byte_buffer_create (byte_buffer.cc:30)
==19520==    by 0xE8B0445: process_data_after_md(batch_control*) (call.cc:1304)
==19520==    by 0xE8AF893: receiving_stream_ready(void*, grpc_error*) (call.cc:1329)
==19520==    by 0xE8AEACA: receiving_stream_ready_in_call_combiner(void*, grpc_error*) (call.cc:1341)
==19520==    by 0xE8EDE65: exec_ctx_run(grpc_closure*, grpc_error*) (exec_ctx.cc:40)
==19520==    by 0xE7DEB22: grpc_closure_run(char const*, int, grpc_closure*, grpc_error*) (closure.h:259)
==19520==    by 0xE7DC633: recv_message_ready(void*, grpc_error*) (message_size_filter.cc:202)
==19520==    by 0xE8EDE65: exec_ctx_run(grpc_closure*, grpc_error*) (exec_ctx.cc:40)
==19520==    by 0xE8EDCA1: grpc_core::ExecCtx::Flush() (exec_ctx.cc:147)
==29823== 948 (160 direct, 788 indirect) bytes in 1 blocks are definitely lost in loss record 1,097 of 1,233
==29823==    at 0x4EDA0F3: malloc (vg_replace_malloc.c:300)
==29823==    by 0xE904549: gpr_malloc (alloc.cc:57)
==29823==    by 0xE8D451B: grpc_resource_quota_create (resource_quota.cc:632)
==29823==    by 0xE8D0101: grpc_tcp_create(grpc_fd*, grpc_channel_args const*, char const*) (tcp_posix.cc:1073)
==29823==    by 0xE8CD686: on_read(void*, grpc_error*) (tcp_server_posix.cc:248)
==29823==    by 0xE8EDE65: exec_ctx_run(grpc_closure*, grpc_error*) (exec_ctx.cc:40)
==29823==    by 0xE8EDCA1: grpc_core::ExecCtx::Flush() (exec_ctx.cc:147)
==29823==    by 0xE8E433F: pollset_work(grpc_pollset*, grpc_pollset_worker**, long long) (ev_poll_posix.cc:1133)
==29823==    by 0xE8E1C4D: pollset_work(grpc_pollset*, grpc_pollset_worker**, long long) (ev_posix.cc:322)
==29823==    by 0xE8F9BED: grpc_pollset_work(grpc_pollset*, grpc_pollset_worker**, long long) (pollset.cc:48)
==29823==    by 0xE8BAAF7: cq_next(grpc_completion_queue*, gpr_timespec, void*) (completion_queue.cc:1039)
==29823==    by 0xE8B9812: grpc_completion_queue_next (completion_queue.cc:1115)

In gRPC version 1.3,
==67657== 4,807,025 (1,536 direct, 4,805,489 indirect) bytes in 12 blocks are definitely lost in loss record 1,012 of 1,012
==67657==    at 0x4EDA0F3: malloc (vg_replace_malloc.c:300)
==67657==    by 0xE8288DE: gpr_malloc (alloc.c:71)
==67657==    by 0xE80BD37: grpc_raw_compressed_byte_buffer_create (byte_buffer.c:50)
==67657==    by 0xE80BCE0: grpc_raw_byte_buffer_create (byte_buffer.c:42)
==67657==    by 0xE7C0F3F: process_data_after_md (call.c:1247)
==67657==    by 0xE7BF51A: receiving_stream_ready (call.c:1270)
==67657==    by 0xE822E91: exec_ctx_run (exec_ctx.c:102)
==67657==    by 0xE824934: grpc_closure_run (closure.c:132)
==67657==    by 0xE7855E8: recv_message_ready (message_size_filter.c:135)
==67657==    by 0xE822E91: exec_ctx_run (exec_ctx.c:102)
==67657==    by 0xE824934: grpc_closure_run (closure.c:132)
==67657==    by 0xE7D4751: null_then_run_closure (chttp2_transport.c:1079)
==67657== 591 (112 direct, 479 indirect) bytes in 1 blocks are definitely lost in loss record 924 of 1,012
==67657==    at 0x4EDA0F3: malloc (vg_replace_malloc.c:300)
==67657==    by 0xE8288DE: gpr_malloc (alloc.c:71)
==67657==    by 0xE7E361B: grpc_resource_quota_create (resource_quota.c:581)
==67657==    by 0xE7A164B: grpc_tcp_create (tcp_posix.c:590)
==67657==    by 0xE79CBAE: grpc_tcp_client_create_from_fd (tcp_client_posix.c:161)
==67657==    by 0xE79D031: tcp_client_connect_impl (tcp_client_posix.c:328)
==67657==    by 0xE79D601: grpc_tcp_client_connect (tcp_client_posix.c:385)
==67657==    by 0xE76D439: chttp2_connector_connect (chttp2_connector.c:205)
==67657==    by 0xE802944: grpc_connector_connect (connector.c:49)
==67657==    by 0xE7FED75: continue_connect_locked (subchannel.c:419)
==67657==    by 0xE7FDD44: maybe_start_connecting_locked (subchannel.c:501)
==67657==    by 0xE7FDA18: grpc_subchannel_notify_on_state_change (subchannel.c:552)
Thanks,
Mike

I'm afraid to say I don't have clue which PR fixed this problem. First of all, the latest version of gRPC, 1.30 doesn't have this problem? If it's fixed, you can bisect the version of gRPC started to address this problem and might be able to find the right PR.


[Git] Setup tab disappears after the first commit, even though there is no Lepiter knowledge base

Pull a new repo, set it up:

It will have some changes:

Once committed, Setup tab disappears

It is no longer possible to setup a Lepiter DB from the Git tool. Intended?

If we want to keep Setup tab we should flip GtGitRepositoryDatabaseSetupItem>>#isRequired to true.


core.js -VS- global pub/sub

I'm trying to understand core.js.

Every module has an init and destroy
Every module can listen and notify (i.e. a global pub/sub)
A module can be coupled to a DOM element.

Correct?
I could implement the same with a global pub/sub system and a single Mediator:

Let every module emit a created(name) event when it is defined.
The Mediator listens to created events and keeps a list of module names.
Mediator.start(xxxx): emits started xxxx(DOMElement) where xxxx is the module name.
Mediator.stop(xxxx): emits stopped xxxx where xxxx is the module name.

Should be the same functionality, only a different way to view it... right?

Hello @markmarijnissen , thanks for getting interested on Core.js
To your first question:
Yes, basically that's how core.js works
And to the second question:
I may say yes, they would work basically the same, but the idea of Core.js is that the Core component has access to extensions (aka jQuery), and that the modules should never talk to other extensions or other modules directly, for that you use the sandbox.
Also not everybody knows how to create a cool structure using pub/sub and a mediator, Core.js helps with that as well since those features are already built-in, you just have to code :)
I get what was your point, and if you know how to structure your files, and know how to implement a cool mediator + pub/sub, then you probably don't need Core.js
Otherwise it would help you a lot, principally to understand how to structure your app if you don't have much experience with Javascript.
What do you think?

Yes, I agree. It helps to have libraries that support in having a proper structure.
I am curious - do you have any experience building (big or small) apps using core.js?
As my own software development experience gains, I try to develop a "design philosophy" that helps me make good apps. I've noticed pro's and cons from all kinds of approaches (i.e. SpineJS, Backbone, AngularJS, plain old javascript, etc).
Currenty I'm a fan of a mediator-ish structure - where I have absolutely no requirements of the the code I am using. I could use any EventEmitter, Router, jQuery, etc. In my opinion, it is a huge advantage that you don't have to wrap your code in any framework code.
Then - the big benefit I discovered: A design philosophy should have room for "messy" code. It often takes me a few (2-3) iterations before I fully grasp the scope of the problem and I can write code. With my current "Mediator" philosophy, I independent, standalone modules that do one thing, and one thing well. The Mediators couple everything together. It's easy for me to throw away or replace Modules that are not working. For more complicated stuff, I can also write a big, ugly, coupled Mediator... and once I have fully implemented everything, I can refactor it into independent modules. (Which I often like to share on NPM/Bower).
The Core.js library does many things I realy enjoy. In particular, it facilitates a clean seperation of concern.
Still - how would you combine Core.js with a library like Famo.us? Or with Backbone? I am curious!

Cool, in fact the mediator pattern helps with a lot of stuff, and can be a standalone approach to develop small size and big size applications.
Unfortunately since core.js is too young, and I'm not working very often on personal projects (other then Core.js) I have not implemented it in any app... Also I don't know the "best" approach to combine a third party framework (like backbone) using Core.js, but I'd to something like that:
Core.extend('Backbone', {}); // $1

Core.register('tweet-list', function(sandbox) {
  function setListView() { // $2
    return sandbox.x('Backbone').View.extend({
      el: this.el // $3
    });
  }

  return {
    init: function() {
      this.View = setListView.apply(this); // $4

      // code, code, code
    }
  }
});

Core.start('tweet-list');

$1 First of all I extend Core's features using the extend method
$2 Inside the module, i created a simple function just to config the View using backbone
$3 Since the module name is a reference for an element in the DOM, I can reference it to the el of
the View configuration (isolation of the DOM)
$4 After init is triggered, I get a reference into this.View of the Constructor created by backbone, so I can use it in other methods.
Note: I use apply to call the setListView, since the this.el inside the setListView method would not be accessible.
After that is up to your imagination to continue coding.
Again, don't know if this would be the best approach, but if I would start from scratch, that would be something acceptable in my opinion. What do you think about this code? do you have a better idea of how to structure it?

@markmarijnissen any thoughts on what I said?
:)

Hey - yes I have some thoughts about it.
I don't see the added benefit of the "extensions". Your module is still tightly coupled to BackBone.

If sandbox.x('BackBone') does not exist, what are you going to do? Re-implement the entire BackBone API?
It's just too easy to get a reference to some external library/object - this makes it easy to abuse; i.e. people ignoring strict module seperating by adding "extentions"

I think key to proper encapsulated modules is to meet following conditions:

coupling should be kept to a minimum (i.e. only use BackBone View, not entire BackBone lib)
coupling should be in language of the module - i.e. it shouldn't be x(BackBoneView) but just x(CreateView). The implementation of CreateView is up to the developer.

This way, the interface of modules to other modules is three-fold

notify (emit events)
listen to events
call global functions

In a way, "global functions" are not even really needed. You could always do:
notify('createView',function callback(view) { // .... });

Although working with callbacks for calling synchronous methods might be cumbersome.

I see, so instead of extending the entire Backbone library, I'd extend only what I need from backbone (views for example)?
And even further, I could extend Core with a method created by my own that just returns an instance of a Backbone View? That looks like a better approach then the one I've showed you.
Just one question, I don't quite get the notify('createView', function() {});, did you really meant notify?

Interesting. I like the direction of this. Its better to have a simple well defined API structure to Core.js and have 'plugins' or 'helpers' that implement connecting that API to Backbone, React or whatever.
This needs to be thought about properly, and from a higher level.
I also agree with mark, sandbox.x('BackBone') should not be available, but I do not like the idea of global functions. Any interaction between modules must come via messages and messages only. Senders must not know anything about receivers or even if anything reacted to them. If a Sender must get info from outside, then it should be done view a Receiver message.
The thing is, at some point you are going to need to write some Backbone code. No matter how well its abstracted, this is what will do the work. So where would this go?

I've been thinking about this and following on from @allain pull request https://github.com/mauriciosoares/core.js/pull/30 I think we have come across something really important.
If we extend core.Start to accept an array of modules then we could effectively produce the kind of split thats required for loading above the fold items.
Core.start(['above-the-fold-a', above-the-fold-b']).then(function(result) {
  console.log('Above the fold modules loaded', result);
  Core.start('less-important-module');
});

What do you think?

I like it. Though if you are doing that it's tempting to overload start so
that no arguments means "all" and then deprecate startAll
On Jan 18, 2015 3:53 PM, "Jae Task"<EMAIL_ADDRESS>wrote:

I've been thinking about this and following on from @allain
https://github.com/allain pull request #30
https://github.com/mauriciosoares/core.js/pull/30 I think we have come
across something really important.
If we extend core.Start to accept an array of modules then we could
effectively produce the kind of split thats required for loading above the
fold items.
Core.start(['above-the-fold-a', above-the-fold-b']).then(function(result) {
console.log('Above the fold modules loaded', result);
Core.start('less-important-module');
});
What do you think?
—
Reply to this email directly or view it on GitHub
https://github.com/mauriciosoares/core.js/issues/25#issuecomment-70425360
.

Yeah, science bitch! (breaking bad quote),
I like your solution @jaetask, and we could indeed deprecate startAll like @allain said.
I'll definitely work on this.

I'll close this issue by now... since we changed our subjects here... and i've opened a new issue #32, if you guys are interested on suggesting some other things


[WIP] OpenStack adding Kyetone v3 support

OpenStack adding Kyetone v3 support

draft of what is needed for v3
WIP :

depends on fog 1.34, last needed fix was done by upstream
need to change it to support both v2 and v3

@blomquisg I guess adding selectbox with v2 and v3 in registering Provider will do the job?
@kbrock we need to sync up on what is planned for openstack tenants, then similar thing will be probably for keystone v3 domains

@miq-bot add_label wip

ok, so now it shoudl support both v2 and v3
Option to pick the API version has been added to UI
before:

after:

@Ladas, thanks for the screenshots!

@miq-bot remove_label wip


bug while creating widget

the screen kind of blinks when creating note widget, so can't create the widget at all. it doesn't happen when creating the library widget but when i click the plus sign, it blinks again.

Could you please provide a video? There isn't much info about the issue.

yes sorry I don't know how else to describe it. but here is a screen capture
the first part is when I try to create a "note" widget and the second part is when I try to create a "library" widget. I just press the home button to exit this

Thanks for the info. Can you tell me whether the issue happens with the app when you launch it or is it just with widgets?

the issue only persists when creating widget, the app works great

fixed in the new update thanks


Gpt-neo inference with Deepspeed: IndexError: Dimension out of range

Thanks for this useful repository. I was able to follow it to train a gtp-neo 2.7B model.
Inference on the model works well for me, using less than 8GB of Vram, so fits on consumer-level gpus, however, I'm not yet able to get the inference working with Deepspeed.
To be clear...
I am using the code from here:
https://github.com/Xirider/finetune-gpt2xl/blob/main/README.md#generate-text-with-a-gpt-neo-27-billion-parameters-model
And it works well, if I comment out this line:
deepspeed.init_inference(model, mp_size=1, dtype=torch.half, replace_method='auto')
If I retain the line, then the inference fails with this error message:
  File "/opt/conda/lib/python3.8/site-packages/deepspeed/ops/transformer/inference/transformer_inference.py", line 374, in forward
    output = DeepSpeedSelfAttentionFunction.apply(
  File "/opt/conda/lib/python3.8/site-packages/deepspeed/ops/transformer/inference/transformer_inference.py", line 312, in forward
    output, key_layer, value_layer, context_layer = selfAttention_fp()
  File "/opt/conda/lib/python3.8/site-packages/deepspeed/ops/transformer/inference/transformer_inference.py", line 270, in selfAttention_fp
    qkv_out = qkv_func(input,
IndexError: Dimension out of range (expected to be in range of [-2, 1], but got 2)

I'm actually a bit vague on whether Deepspeed actually should be used with inference for GTP-NEO, as far.
Huggingface says....
https://huggingface.co/transformers/main_classes/deepspeed.html

DeepSpeed ZeRO-2 is primarily used only for training, as its features are of no use to inference.

But, Microsoft has a guide which shows the usage of Deepspeed for inference with this model...
https://github.com/microsoft/DeepSpeed/blob/master/docs/_tutorials/inference-tutorial.md#end-to-end-gpt-neo-27b-inference

After further testing, I am finding that inference with Deepspeed DOES offer the promised ~2x speedup, and this code ...
https://github.com/Xirider/finetune-gpt2xl/blob/main/README.md#generate-text-with-a-gpt-neo-27-billion-parameters-model
DOES work, but only if I use the not-finetuned, original model, from Huggingface -- "EleutherAI/gpt-neo-2.7B"..
so:
model = GPTNeoForCausalLM.from_pretrained("EleutherAI/gpt-neo-2.7B").half().to("cuda")
tokenizer = AutoTokenizer.from_pretrained("EleutherAI/gpt-neo-2.7B")

works well.
But when I used my custom finetuned model, which I finetuned according to this guide... https://github.com/Xirider/finetune-gpt2xl/blob/main/README.md#finetune-gpt-neo-27-billion-parameters
....
I get the

error when I try inference.
Again, inference on my finetuned model, WITHOUT deepspeed, works well.
So in short it seems that something about the finetune process shown here produces a finetuned model that doesn't seem to work with Deepspeed inference, even through the original (not-fintuned) model can be shown to work will deepspeed inference.

I've found what seems to be a solution, that enables Deepspeed inference on my trained gpt-neo model.
The issue seems to be that the trained gpt-neo has this line in its config.json:
"use_cache": false,
I can see that the original model -- "EleutherAI/gpt-neo-2.7B", this value is set to false
https://huggingface.co/EleutherAI/gpt-neo-2.7B/blob/main/config.json#L77
So it seems that if, after training, I manually modify config.json, in the folder of my trained model, and set this value to true, then Deepspeed inference works as expected. I get about a 25% speedup on inference on an rtx 2070 super.
Closing this.

Thanks for the tip, it works!


A restarting Cascade client may freeze [investigated and solution proposed]

Sometimes, we write a bash script calling a cascade client a couple of times, for example,  to set up the application pools. The cascade_client command following another could freeze nondeterministically.
Thiago @tgarr identifies that the issue is caused by two lines in derecho code, which skipped creating a new connection for the coming-back external cascade client (cascade_client). He enforced recreation of the connection by commenting on the above two lines, and the phenomenon disappears.
[Explanation] In the current Derecho implementation, the external client leaves brutally without notifying the Derecho group member. In the above case, the Derecho group member is the cascade_server process. The group member will eventually find out when the P2P heartbeats fail and clean it up (with some annoying error messages we have wanted to remove). However, the external client comes can goes frequently because the bash calls a couple of consecutive cascade_client commands. Sometimes, before the cascade_server process detects the leaving of a previous external client, a new external client with the same node ID tries to connect, triggering the condition in the above two lines.
[Solution Proposal] I propose to introduce a graceful quitting procedure for a leaving external client. It not only solves the above issue but also gets rid of the annoying error messages. We should only report errors when ABNORMAL things happen.

Commenting out those two lines actually doesn't solve the issue completely. The client doesn't freeze any more and the server doesn't enter an inconsistent state, however the client gets libfabric error -16 (-FI_EBUSY) most of the time (at least using TCP, I haven't tried with RDMA), which means "Device or resource busy".
I agree that a "graceful exit" is the best solution. But can we do something about this -16 error in the client? Wait and try again maybe? It's a small thing but it can break simple setup scripts that use the command line client, which is not very nice for a 1.0 product release.

New update: I tested on Fractus with TCP and RDMA. I had the same issue with TCP (as in my local machine), however it works fine with RDMA. So the problem seems to be limited to TCP. The code path is the same, so maybe it's something in libfabric?

I guess that libfabric has an issue connecting to the same remote ip/port without cleaning everything up. I prefer the graceful exit approach instead of fixing something caused by a problematic workaround. What do you think, @tgarr ?

Agreed!


Run Javascript tests automatically in Github actions workflow

Pushing (or maybe creating a pull request) should start a minecraft server running PaperSpigot, run Javascript tests (runTests()-function) and exit with a zero or non-zero code, depending on the test result.

Was done, though vk-ts doesn't yet use this.


Crawler.stop() is undefined

Hi
I use the simplecrawler in Simple Mode. I can start it by calling Crawler.crawl(url).on(...
On fetchcomplete I push the processed results to a client app through a websocket.
Now if someone starts crawling a really large site and decides to abort it, I want to be able to stop the crawling process, but when I call Crawler.stop(), I get TypeError: undefined is not a function.
How can I stop the crawler completely?
Here is the code I use (it's Coffeescript)
Crawler = require "simplecrawler"

Crawler.crawl url
  .on "fetchcomplete", (item) ->
    # do something

  .on "complete", ->
    # do something else

# to test it I called Crawler.stop() after a timeout, but it seems to be undefined
setTimeout ( ->
  Crawler.stop()
  console.log 'stopped'
), 4000

Thanks in advance
Steve

Hi Steve —
In this case, Crawler (with a capital C) is the constructor for simplecrawler. Crawler.crawl returns an instance of Crawler, which has the method required. So, you can do this:
Crawler = require "simplecrawler"

myCrawler = Crawler.crawl url
  .on "fetchcomplete", (item) ->
    # do something

  .on "complete", ->
    # do something else

# to test it I called Crawler.stop() after a timeout, but it seems to be undefined
setTimeout ( ->
  myCrawler.stop()
  console.log 'stopped'
), 4000

Please let me know how that goes for you — and don't hesitate to ask if you require any further help. I'm pretty slow about following up issues but I'll do my best. :)

Perfect. Thanks.


Problems to run ASR baselines

Hello
I'm trying to run the ASR baselines you have provided. I have downloaded the datasets with scripts/download_datasets.sh in manifest/slue-voxceleb and manifest/slue-voxpopuli.
When running
bash baselines/asr/ft-w2v2-base.sh manifest/slue-voxceleb save/asr/w2v2-base-vc
I have the following error:
File "/home/brun/fairseq/fairseq_cli/hydra_train.py", line 27, in hydra_main
_hydra_main(cfg)
File "/home/brun/fairseq/fairseq_cli/hydra_train.py", line 56, in _hydra_main
distributed_utils.call_main(cfg, pre_main, **kwargs)
File "/home/brun/fairseq/fairseq/distributed/utils.py", line 369, in call_main
main(cfg, **kwargs)
File "/home/brun/fairseq/fairseq_cli/train.py", line 94, in main
model = task.build_model(cfg.model)
File "/home/brun/fairseq/fairseq/tasks/audio_finetuning.py", line 193, in build_model
model = super().build_model(model_cfg, from_checkpoint)
File "/home/brun/fairseq/fairseq/tasks/audio_pretraining.py", line 197, in build_model
model = super().build_model(model_cfg, from_checkpoint)
File "/home/brun/fairseq/fairseq/tasks/fairseq_task.py", line 335, in build_model
model = models.build_model(cfg, self, from_checkpoint)
File "/home/brun/fairseq/fairseq/models/init.py", line 106, in build_model
return model.build_model(cfg, task)
File "/home/brun/fairseq/fairseq/models/wav2vec/wav2vec2_asr.py", line 209, in build_model
w2v_encoder = Wav2VecEncoder(cfg, len(task.target_dictionary))
File "/home/brun/fairseq/fairseq/tasks/audio_finetuning.py", line 170, in target_dictionary
return self.state.target_dictionary
File "/home/brun/fairseq/fairseq/tasks/fairseq_task.py", line 41, in getattr
self._state[name] = self._factoriesname
File "/home/brun/fairseq/fairseq/tasks/audio_finetuning.py", line 123, in load_target_dictionary
return Dictionary.load(dict_path)
File "/home/brun/fairseq/fairseq/data/dictionary.py", line 226, in load
d.add_from_file(f)
File "/home/brun/fairseq/fairseq/data/dictionary.py", line 239, in add_from_file
raise fnfe
File "/home/brun/fairseq/fairseq/data/dictionary.py", line 236, in add_from_file
with open(PathManager.get_local_path(f), "r", encoding="utf-8") as fd:
FileNotFoundError: [Errno 2] No such file or directory: '/home/brun/slue-toolkit/manifest/slue-voxceleb/dict.ltr.txt'
Could you help me to solve this problem.
(Sorry if the issue is naive, I am completely new to speech processing)
Many thanks!

Hi @CBrun38,
I apologize for this.
Please run to generate the dictionary files:
for split in voxceleb voxpopuli; do
for label in ltr wrd; do
    python slue_toolkit/prepare/create_dict.py manifest/slue-${split}/fine-tune.${label} manifest/slue-${split}/dict.${label}.txt
done
done

Best,
Felix


Get-ADObject fails with error: "invalid enumeration context"

The previous screenshot was from a VPN connected system. This next screenshot was taken from a W365 cloud virtual host.

The invalid enumeration context is an error that cannot be avoided when using ADWS in very large domains, you will need to use splitsearch and lettersplitsearch to batch the enumerations to small parts.
The error occurs due to the server setting a limit on the enumeration session and terminates it after 30 minutes (hence the need to enumerate containers separately and split enumeration further by object names)


Unknow other variant for fragment

Tries to solve the issue.
Add the option fragments-other-variant to allow the generation of an Unkown "serde other" variant in the Enum generated by fragments.
The option can be set as macro attributes or by flag if using the graphql-client-cli.
As default the option is disabled.

I'll try to find time soon for a proper review. Thanks for the PR!

Hi @tomhoule, I have a doubt.
Handling the Enum types the lib adds Other(String) variant as default (no need to set any flags).
What about doing the same thing for the union/interface types making it the default behaviour.
This :point_up: would make it possible to remove the new flag --fragments-other-variant and simplify the code.

I have only a pretty distant memory of what the generated code looks like, but that sounds very reasonable.

@tomhoule I made a second commit which implements the feature as suggested by @damjack.
You can find:

in the first commit contains the changes proposed to handle the feature through a new option
in the second commit contains the changes proposed to handle the feature as default behavior, no options needed

In this way it is easy to see the two versions and decide which one to carry on

Hi @visd0m , I just had a look at both commits, and I think we should have the opt-in option at first (first commit). In general the commit looks good :+1: I'll cut a release now, so we can merge without worrying too much after that and have time to test.

Hi @visd0m , I just had a look at both commits, and I think we should have the opt-in option at first (first commit). In general the commit looks good 👍 I'll cut a release now, so we can merge without worrying too much after that and have time to test.

@tomhoule I reverted the last commit and restore the first changes.
Just wondering if after the merge we can think about another release 😬 ? (hoping to use this without pointing to master)

Yes, we can definitely cut another release as soon as I find some time for graphql-client again, hopefully soon. I'd probably take the opportunity to also change graphql_client_codegen to not depend on syn and quote anymore too.

@visd0m I changed the main branch for the repo from master to main and that automatically closed the PR. Please re-open the PR to the main branch, sorry for the unintended problem.


[Enhancement] Allow to hide photos from certain albums/folders in the timeline/photos view

Is your feature request related to a problem? Please describe.
As a user I have added photos from my DCIM folder and my Screenshot folder. Atm the screenshots show up in the timeline view. I do understand that there is a Blacklist feature, but this hides the Screenshot folder from the album view as well.
Describe the solution you'd like
I'd like to have an option to choose which albums/folders/sources to show in the timeline. This so that for example, the screenshots don't pollute my timeline view where I only want to see my photos.
So a similar solution that you've implemented for the Blacklist feature, but where you only blacklist certain albums from the timeline view. Because I still want to have access to the Screenshot folder in the albums view.
Describe alternatives you've considered
N/A
Additional context
Timeline view = The start view of the app (couldn't find a name for it in the app)

So, basically an option to either hide from both timeline and albums or just from timeline?

It depends on the users of course. But today the Blacklist is for both. So maybe have an option:
"Albums to show"
Maybe just show the different albums in thav view and have some checkbox-like UI component for Timeline and Albums. So the user could quickly uncheck the albums that they don't want to be shown for either view. Just a very quick suggestion.
Albums to the left in a column and then the column to the right having the ui components that allow you to show hide for eitehr view.

Today will be available as nightly


Customize size of right hand sidebar

It would be nice if the right hand sidebar width could be customized. I have a custom rich text field I'm working on that could benefit from some extra horizontal space. I'd be willing to wager that this would be broadly beneficial for puck users. I don't have a specific fix in mind, but some options I've thought about are as follows

Keep the side bar sized to a static value, but allow this static value to be overridden via props on the  component.
It looks like this value comes from here currently
Allow the user to click and drag on the left hand edge of the sidebar, dynamically growing or shrinking (kind of) similar to how the html  component works

Great request @kadencartwright
You've found the right value. The styles in the root Puck component are a bit clunky (and still inline 🐒)
I think the second option definitely sounds like a good idea.
The first option does too, although I'm not sure whether it would be better solved as part of #139, or separately.


Zoom in / Zoom out method scale amounts

Hello! Is it possible to have a parameter to the zoomIn and zoomOut methods so they can scale a certain amount? I dug through the code a little and currently it scales by this.props.zoomSpeed * some multiplier, but it would be great to be able to determine a zoom speed separately for these methods :)

@emilyzhong Thank you for opening this issue!
It is a good improvement, I'll try to get it as soon as possible 😃

@emilyzhong 0.4.0 has just been released, including a new zoomSpeed parameter for zoomIn and zoomOut :)

Thank you for such a speedy update ! 🎉


server.Server: cleanup of public functions

- What I did
deleted or unexposed public functions that were not used outside of server.Server
- How I did it
- How to verify it
- Description for the changelog

/test all

/retest


Cannot switch snapshots in published repo with API

Hello, I would like to thank you for a very nice and convenient tool for managing debian repositories. Currently I'm studying the REST API and unfortunately I can't get how to switch the snapshot for a published repository.
Assume that we have several snapshots, and the first of them was used for publishing:
isaev@wks-isaev:~/repo$ curl http://localhost:8079/api/snapshots | python -m json.tool
[
    {
        "CreatedAt": "2015-03-16T16:23:19.038895455Z",
        "Description": "Snapshot from local repo [mailpaas]",
        "Name": "snap1"
    },
    {
        "CreatedAt": "2015-03-16T16:27:25.400791874Z",
        "Description": "Snapshot from local repo [mailpaas]",
        "Name": "snap2"
    },
    {
        "CreatedAt": "2015-03-16T16:32:23.057045651Z",
        "Description": "Snapshot from local repo [mailpaas]",
        "Name": "snap3"
    }
]
isaev@wks-isaev:~/repo$ curl http://localhost:8079/api/publish  | python -m json.tool
[
    {
        "Architectures": [
            "amd64",
            "source"
        ],
        "Distribution": "utopic",
        "Label": "",
        "Origin": "",
        "Prefix": ".",
        "SourceKind": "snapshot",
        "Sources": [
            {
                "Component": "local",
                "Name": "snap1"
            }
        ],
        "Storage": ""
    }
]

I would like to switch the published repo to the most recent snapshot. But nothing happens:
isaev@wks-isaev:~/repo$ curl -X PUT -H 'Content-Type: application/json' --data '{"Sources": [{"Name": "snap3"}], "ForceOverwrite": true, "Signing": {"Batch": true, "Passphrase": "password"}}' http://localhost:8079/api/publish//utopic | python -m json.tool
{
    "Architectures": [
        "amd64",
        "source"
    ],
    "Distribution": "utopic",
    "Label": "",
    "Origin": "",
    "Prefix": ".",
    "SourceKind": "snapshot",
    "Sources": [
        {
            "Component": "local",
            "Name": "snap1"
        }
    ],
    "Storage": ""
}

Could you please clarify, why doesn't it switching? Thank you.

Documentation and example are wrong :(
"Sources" should be "Snapshots", and it should contain "Component" field, something like that in your case:
curl -X PUT -H 'Content-Type: application/json' --data '{"Snapshots": [{"Component": "local", "Name": "snap3"}], "ForceOverwrite": true, "Signing": {"Batch": true, "Passphrase": "password"}}' http://localhost:8079/api/publish//utopic

Thanks for noticing that!

Thanks for a prompt reply! Succeeded with your last command.


feature: Design tweaks [NP-534]

Related Content gets a top margin to make it start at the same height as the H1 title.
H2 gets a top margin after targeted content and callouts.
Notice banner template adjusted to account for p tags coming in with the content.

See Advice Collection sample page to check changes

LGTM!


API interface and python_ The class (RelationsQueryFilter) does not match

/api/entitiesQuery/count Count Entities by Query：Relations Query Filter
API interface and python_ The class (RelationsQueryFilter) does not match

/example_application_2.py
Through learning, I think they should be consistent, and the examples are consistent.


Add tooling support for module typed binding patterns

Description:
Module-level typed binding pattern support will be introduced via #26599 #28227 #27824. Need to add tooling support for list, record, and error binding patterns.

Closing the issue, since the fixes have been addressed.


Pizza list Added

Hi, @raniyaabdalla ! It looks to me like you submitted the completed project as a PR. That's only necessary if you found a mistake or some other change that needs to be made in the orignal.
Other than that, your code looks great! Hope you enjoyed the training!


Overhaul recipes to replace iron-stick with iron-chest and concrete

This resolves #106, though I did go a bit further than what was 'spitballed' there. Since we need to change the recipe anyway if we want to eliminate iron-stick, might as well take another look at the recipe balance at the same time.
Milestone 0.7.0 might become 1.0.0; it depends on how big this recipe rebalance ends up "feeling" if it gets bikeshedded.

In Factorio 2.0, iron-stick is an unlockable recipe, not automatically available at the start of the game. Adding iron-stick to every early technology that wants to use it in the resulting recipes feels hacky (though the base game does this; no less than 4 technologies unlock the iron-stick recipe now).
Crafting basic warehouse & storehouse has a relatively unchanged total iron cost, but iron-stick and some of the steel-plate in their recipes have been replaced with iron-chest.
Logistic variants are now intentionally more costly to craft, requiring multiples of the logistic chests they are to emulate (instead of only one), adding concrete, and increasing the steel-plate count.
Concrete also added to warehouse-logistics-research-1 prerequisites. Note that no migration is added to retroactively apply this change; players who have already unlocked this technology will keep it, but can't craft any more logistic buildings until they unlock Concrete.
My lore-based reason for adding concrete to logistic buildings is that they probably have some automated item handling inside that needs the smoother floor, and can't deal with the roughness of stone brick. In reality of course, the reason is wanting to bump the complexity and cost a little more in exchange for the convenience.

Would also possibly bump this to a v1.0.0 with these changes, as it's pretty significant

Would also possibly bump this to a v1.0.0 with these changes, as it's pretty significant

Wouldn't you know, I thought about that too 😸:

Milestone 0.7.0 might become 1.0.0; it depends on how big this recipe rebalance ends up "feeling" if it gets bikeshedded.

We should go directly from 0.6.0 to 1.0.0 IMO. There are just a few things left to finish up, and I wouldn't hold up a new release for #108; another manual release or two will be fine.
Next thing I work on for Warehousing will be this recipe overhaul. I like the advanced circuit idea!

Happy, you just have to resolve the conflict :)

I went ahead and resolved the conflicts :)

I went ahead and resolved the conflicts :)

Thanks, but merge commits inside feature branches are awful. Better to rebase, or manually merge on the CLI (like I'm about to).


[youtube] Fails extracting video of a "My Mix" playlist

[x] I've verified and I assure that I'm running youtube-dl 2017.03.24
[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

$ ./youtube-dl -v --dump-json --no-playlist "https://www.youtube.com/watch?v=MuAGGZNfUkU&list=RDMM"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'--dump-json', u'--no-playlist', u'https://www.youtube.com/watch?v=MuAGGZNfUkU&list=RDMM']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.03.24
[debug] Python version 2.7.10 - Darwin-16.4.0-x86_64-i386-64bit
[debug] exe versions: none
[debug] Proxy map: {}
WARNING: Falling back on generic information extractor.
{"display_id": "annotations_invideo?cap_hist=1\\u0026playlist_id=RDMM\\u0026video_id=MuAGGZNfUkU\\u0026client=1\\u0026ei=kM7UWNTbOsHf4gKRo7moBA", "extractor": "generic", "protocol": "https", "format": "0 - unknown", "requested_subtitles": null, "uploader": "www.youtube.com", "format_id": "0", "playlist_index": null, "playlist": null, "http_headers": {"Accept-Language": "en-us,en;q=0.5", "Accept-Encoding": "gzip, deflate", "Accept": "text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8", "User-Agent": "Mozilla/5.0 (X11; Linux x86_64; rv:10.0) Gecko/20150101 Firefox/47.0 (Chrome)", "Accept-Charset": "ISO-8859-1,utf-8;q=0.7,*;q=0.7", "Cookie": "YSC=8Nz98YaRkEs; VISITOR_INFO1_LIVE=abqg9qI7WNk; PREF=f1=50000000"}, "url": "https://www.youtube.com/annotations_invideo?cap_hist=1\\u0026playlist_id=RDMM\\u0026video_id=MuAGGZNfUkU\\u0026client=1\\u0026ei=kM7UWNTbOsHf4gKRo7moBA", "extractor_key": "Generic", "title": "Hans Zimmer - Time", "id": "annotations_invideo?cap_hist=1\\u0026playlist_id=RDMM\\u0026video_id=MuAGGZNfUkU\\u0026client=1\\u0026ei=kM7UWNTbOsHf4gKRo7moBA", "ext": "unknown_video", "webpage_url": "https://www.youtube.com/watch?v=MuAGGZNfUkU&list=RDMM", "_filename": "Hans Zimmer - Time-annotations_invideocap_hist=1_u0026playlist_id=RDMM_u0026video_id=MuAGGZNfUkU_u0026client=1_u0026ei=kM7UWNTbOsHf4gKRo7moBA.unknown_video", "fulltitle": "Hans Zimmer - Time", "age_limit": 0, "webpage_url_basename": "watch"}

Extracted (single URL) is https://www.youtube.com/annotations_invideo...
Works perfectly when trying with https://www.youtube.com/watch?v=MuAGGZNfUkU instead of https://www.youtube.com/watch?v=MuAGGZNfUkU&list=RDMM.

https://www.youtube.com/watch?v=MuAGGZNfUkU&list=RDMM is not a valid mix URL.


[23.11] nixos/network-interfaces: add warning if network-interfaces.target is used

With https://github.com/NixOS/nixpkgs/pull/272169 this is deperacted, we might aswell warn in advance.
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Adding warnings on things that work on a stable release goes against what I expect of a stable release.


Use multiple provider in structured rails app

Hi all,
i'm using this awesome gem for a bit and it's very usefull but i didn't expected how to have multiple component add in a page connected to the redux-store.
For example i need to have component A and B in a page so i use
<%= react_component('A', @resource_props, {prerender: false}) %>

and
<%= react_component('B', @resource_props, {prerender: false}) %>

to add them in my page. I need to have both of them connected to redux-store so i made two containers
export default class A extends React.Component {
  render() {
    return (
      <Provider store={store}>
        <Awidget></Awidget>
      </Provider>
    );
  }
}

export default class B extends React.Component {
  render() {
    return (
      <Provider store={store}>
        <Bwidget></Bwidget>
      </Provider>
    );
  }
}

and two component exported with the redux connect method. Everything works fine except that i need to create the store in my store.jsx file with
export const store = configureStore();

and import it in two container to have them both access the store but this prevent me to initialize my store with some props from Rails like in the react_on_rails helloworld example. How can i pass the store to both the Provider and initialize it with some props? I'm doing something wrong?
I hope I have been clear and thanks for the help

@alexfedoseev should this be feasible?
@Tamiyadd Can you setup your import of the store so that when you import it, it lazily loads.

@alexfedoseev and I discussed this. I'll be coming up with a solution soon.

For those following this issue, keep an eye on https://github.com/shakacode/react_on_rails/pull/215

I hope to have this done by the weekend.

This has shipped in 3.0.0.rc.1, @Tamiyadd


Restricted keywords as cli args

Language restricted keywords like "new" cannot be used as struct members but can be possible cli sub-commands or options.

Reopening:
Unintended behaviour when trying to parse subcommands that uses the transform.

then Unintended behaviour seems to be happening when dealing with sub commands. Closing this, see #6


Not working with operator mono italic font any longer

hi there,
is is my favourite theme.
unfortunately it doesn't seem to support the italic operator font for ligatures any longer.
any fixes or settings I need to consider?
thanks for help!

I've removed the italics because it's more readable for me. I don't use Operator Mono anymore.
You can restore that settings by adding the following to your preferences file inside "editor.tokenColorCustomizations":
"[Electron]": {
    "textMateRules": [
        {
            "scope": "entity.other.attribute-name",
            "settings": {
                "fontStyle": "italic"
            }
        },
        {
            "scope": "support.type, support.class",
            "settings": {
                "fontStyle": "italic"
            }
        }
    ]
}

Hope it helps you.

Amazing, thank you very much!
This is the best theme ever :-)
For others and future lookups, this is what I added to my Settings to make it work with Operator Mono Italic Fonts:
  "workbench.colorTheme": "Electron",   "editor.fontFamily": "Operator Mono",   "editor.fontWeight": "200",   "editor.fontSize": 12,   "editor.tabSize": 2,   "editor.fontLigatures": true,   "editor.tokenColorCustomizations": {     "[Electron]": {       "textMateRules": [         {           "scope": "entity.other.attribute-name",           "settings": {             "fontStyle": "italic"           }         },         {           "scope": "support.type, support.class",           "settings": {             "fontStyle": "italic"           }         }       ]     }   }

Perfect! Closing the issue.

I've removed the italics because it's more readable for me. I don't use Operator Mono anymore. You can restore that settings by adding the following to your preferences file inside "editor.tokenColorCustomizations":
"[Electron]": {
    "textMateRules": [
        {
            "scope": "entity.other.attribute-name",
            "settings": {
                "fontStyle": "italic"
            }
        },
        {
            "scope": "support.type, support.class",
            "settings": {
                "fontStyle": "italic"
            }
        }
    ]
}

Hope it helps you.

This is what I've been looking to customize my workspace thank you! I didn't know I could do this.


Add Img Thumbnails in img src elements to make it more intuitive and save time

img src selecting the folder displays the file inside it, for img display the image thumbnail to save time to look for it again in the folder.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

Now that I look at it again, I don't think this has to do with Emmet.
Reassigning to Martin.


DO NOT MERGE, uploading binaries for 0.5.1

cc @vincentqb
Signed-off-by: Eli Uriegas<EMAIL_ADDRESS>

Closing, 0.5.1 has been released


Introduce DIC

In order to improve code quality and make possibile to have test (without be crazy :smile: ) we are going to introduce DIC and translate all the PHPCI parts to services.
All parameters and services are defined in service.yml file in the application root, we will split this file in smaller elements after the platform migration to DIC is done.

[x] Introduce DIC
[x] Migrate web application to use DIC

[x] Convert Config to services
[x] Convert [Entity]Store to services
[x] Convert [Entity]Service to services
[x] Convert [Xxxx]Controller to services
[ ] Migrate logging (Monolog) to service
[ ] Migrate mail system (SwiftMail) to service

[x] Migrate CLI app to use DIC

[x] Convert [Xxxx]Commands to services
[x] Convert console app
[x] Convert daemonize app

[ ] Improve controllers' testing
[ ] Migrate plugin management system to services
[ ] Migrate builder system to service
[ ] Remove pimple to manage DIC in the plugin system
[ ] Testing, testing and testing again

This task is big and can introduce a large amount of regression but is needed to make the platform easy to maintains and evolve and I think we can't skip but I'm open to any other suggestion
cc/ @Adirelle @tvbeek @dancryer

Hi @mavimo - Are you still working on this PR? Looks like you got a really long way with it!

@dancryer I'm working on it, but is complex activity (involve all part of the application) and need a lot of test.. so any kind of support is really appreciated :smile:
I think is also a critical activity since can improve the capability to test, extend and evolve the platform, what do you think?


[GHSA-mm8v-wmqx-8h2j] An issue was discovered in the femanager extension before...

Updates

Affected products
CVSS
CWEs
Description
Severity
Source code location
Summary

Comments
Package context from the TYPO3 PSIRT.

Affected versions should be:
< 5.5.3
>= 6.0.0, < 6.3.4
>= 7.0.0, <= 7.0.1

Affected versions should be:

Pretty much like in PR #1677


RxMoya: Create Single directly instead of Observable.create().asSingle()

Hey there :)
I’ve looked through the code and noticed that for some reason the RxSwift implementation creates an Observable using Observable.create and then uses asSingle() to convert it, instead of just using Single.create to begin with, so pushed this quick commit.
There should be zero impact, as it's just an implementation detail.

Sorry about the incorrect branch here ;) It auto-selected and I missed it. @sunshinejr

          2 Warnings

:warning:
Any changes to library code should be reflected in the Changelog. Please consider adding a note there.

:warning:
The library files were changed, but the tests remained unmodified. Consider updating or adding to the tests to match the library changes.

  Generated by :no_entry_sign: Danger

All green. Do we want a Changelog entry for this ? @sunshinejr

Just an internal fix without impact, don't think it needs a Changelog entry. :)

Thanks a lot for contributing to Moya! I've invited you to join the Moya GitHub organization – no pressure to accept! If you'd like more information on what that means, check out our contributor guidelines and feel free to reach out with any questions.

That was my thought as well. Thanks !

Thanks for cleaning up my mistake @freak4pc 🙈

Haha, no worries. My pleasure @AndrewSB 💯


Create a proper "no location permission granted" notification.

Just fill in stuff here: GeoChecker.java#L98-L106

A comment about this , I am not sure if such a thing you wanted to achieve.


Update each widget struct doc page with supported builder XML sections (and pre-defined identifier strings)

Review and determine, while discussing with team, a reasonable and manageable way to revise the document page for the various widgets, to concisely, accurately and thoroughly specify the supported builder XML sections. In addition to concise descriptions of the supported section options, each predefined identifier string required for the XML should be specified.

The documentation comes from the gtk project, please report any missing documentation there https://gitlab.gnome.org/GNOME/gtk/. But note, a very broad issue like this is not really actionable. Instead, write down the list of widgets that you think should contain more documentation of how to use the widget in an xml file.

The documentation comes from the gtk project, please report any missing documentation there https://gitlab.gnome.org/GNOME/gtk/. But note, a very broad issue like this is not really actionable. Instead, write down the list of widgets that you think should contain more documentation of how to use the widget in an xml file.

Understood. I'll close this issue, and submit a revised edition of it which lists those Gtk widgets which insufficiently document their buildable XML section details.


Possibility to remove icon tray

I would like to have the possibility to remove icon tray from the taskbar.

You can move it into the "overflow" menu.

Thanks for reply.
I know I can move it to the overflow menu but i would like to remove it completely, jast having it in background without icons, like some other application do.
1101ks
Da: Jay
Inviato: domenica 30 agosto 2020 13:51
A: microsoft/PowerToys
Cc: 1101ks; Author
Oggetto: Re: [microsoft/PowerToys] Possibility to remove icon tray (#6237)
You can move it into the "overflow" menu.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Duplicate of https://github.com/microsoft/PowerToys/issues/765


Design Document of Applicative Kore

Reviewer checklist

[ ] Test coverage: stack test --coverage
[ ] Public API documentation: stack haddock
[ ] Style conformance: stylish-haskell

I'm going to merge this because it's approved and there has been no activity for two weeks.


Consider taking advantage of PHP-DI container compilation

According to the PHP-DI docs, setting definitions in the container directly via the set method prevents entries from being added to a compiled container.
Would you have any objections to reworking the container creation logic in order to:

Use the DI\ContainerBuilder class
Conditionally enable compilation based on some environment setting
Use array definitions instead of directly setting items on the container

Just for an idea of what this might look like:
https://github.com/ssnepenthe/Slim-Skeleton/commit/08981c32e451f839ecbe1fe7ee8f74f695fe3444

Definitely open to doing so. Would you like to raise a PR for it?


[DX]

The Vercel bot started acting out and keeps asking me to authorize some stuff:

We should probably contact them and ask them what's going on, or I'll look into it some more.

The Vercel bot started acting out and keeps asking me to authorize some stuff:

We should probably contact them and ask them what's going on, or I'll look into it some more.

Actually it's not acting rouge, it's happening because pull request are made from people who don't have access to this repo, you are the only one who does have access to this repo and it's deployment so it's letting you know that someone else is trying to deploy something without your permission.


Fix directional deepslate

Fix the renewableDeepslateFromSplash rule being slightly larger in the positive axis's.
Also fixes the bounding box being shorter due to casting negative values
This is untested!

Rounding types:

Current effect on splash bounds:

Thanks for the great visuals! Your latest commit (d03a478e1cc07f9973b6c4410cbde001d820ca5a) makes the converted area uneven again, being 1 larger in the Positive directions. It both ceils the high bound and includes the high bound of the loop.
In that version, a splash potion bounding box from -8.5 to -4.5, for example, will iterate from -9 -> -4 even though the block at -4 (which encompasses -4.0 to -3.0) isn't low enough to reach the -4.5 upper bound. I believe the version pre the latest commit is more correct (at 1648c8f2aa4737d49802b1275e87b43270dd13de).
But that still has some inconsistency. The position of the potion on impact is the location that it was at the beginning of the tick in which it would hit. This means that once you start having faster potion velocities (say drop from 30+ blocks), there may be only a 1 deep area converted instead of 2 deep etc. However, this is how regular splash potions work, so maybe it's OK. But those operate on entities, so a little inconsistency is expected.
It would be simple enough to use the actual raytraced collision point as the base of the calculation (just use hitResult.getPos() when hitResult.getType() == Block ) and would make the results much more repeatable, but it would also add inconsistency with where an entity would actually get hit by a potion.
The consistency with entities getting splashed doesn't matter though except for lore. I think the reliable conversion is more important. Also, this is also how splash water bottles work to extinguish fires, so it's what I think I'll implement.


WW-5251 Reinstate deleted interfaces with transparent compat

Partially reverts https://github.com/apache/struts/pull/670 to restore compatibility with deprecated interfaces
Refs WW-5251

@brianandle If you bundle these classes with your Struts application, it should restore compatibility.

Sorry, I merged wrong PR :( Should I revert it?

@lukaszlenart  I believe it's the correct one, this seems like a better approach vs the revert, but the ParameterAware still needs an additional change by @kusalk. Or you could fix the method signature (see comment)

Thanks @kusalk that other PR looks good to me. Commented there too


Add a configuration to define the unified console context path

Description
Today there's no easy way to change the unified console context path. It would be helpful especially to "expose" the console behind a Apache HTTPd proxy.
Motivation
It would be helpful especially to "expose" the console behind a Apache HTTPd proxy.

It would be helpful especially to "expose" the console behind a NGINX reverse proxy.

@ciukstar If you're using nginx, you can leverage the NGINX's 'rewrite' to do that.
For example
location ^~/your_context_path/ {
        rewrite ^/your_context_path/(.*)$ /$1 break;
        proxy_pass http://YOUR_DRUID_CONSOLE;
    }

@FrankChen021 Thank you for suggestion.
But it doesn't work for all API requests (XMLHttpRequest, I guess) like "/status", "/proxy", "/assets/kafka.png", "/assets/s3.png", etc. Of course I handle these cases one by one, but I am not sure that all were resolved.
There are errors at least for "#load-data" and "Druid Doctor" -> "Analyze Druid cluster".
Thanks again.

Oh, I get it, the front-end console is not aware of the context path.

Yes. It seems that Ajax requests are not aware of the context path. Also requests for static assets are not aware of the context path and may interfere with existing client applications (for instance, in my case, an Angular application which also publishes static resources at "/assets")


Customize error format (Serializer for Errors)

One issue I have had is that my team is responsible for integrating with some services that use a slightly different error response body. We would like to maintain a standard across all of our services and so having a way to easily define the format of the ErrorResponse would be amazing.

You might be happy to know version 2, being released within days, includes a concept for error serializers for customizing it exactly the way you want :)

Nice! Will this newest version be compatible with the same versions of laravel/lumen? Also anyway we can checkout the dev release prior to official release? I had made contributions in the past and I'd love to continue to help where I can! :)

Yeah, the plan is to keep the same version requirements. I started a new rewrite after the current dev branch was created, so that's not currently up to date. However, I can push it all up on a new branch if you want to take a look. It's not quite ready to be used in production yet though, but it's almost there! All feedback and help is definitely highly appreciated :)

That sounds great! Going to close this ticket unless you want to keep it open until release?

Made a new "v2" branch located here: https://github.com/flugger/laravel-responder/tree/v2
The documentation has also been rewritten (but it's not complete). Lemme know what you think!
And sure, let's keep it open until the version is released.

Awesome! Took a quick glimpse and it all looks great! Would you like me to do Pull Requests for feedback? or just comment back here?

Either is fine really! If you have certain things you want to discuss in more detail it might be easier to use another platform. Are you a member of the Larachat Slack channel by any chance?

Just join under ductiletoaster

Version 2 has been released and introduces the concept of error serializers.


az storage fs directory upload with login authentication fails with MSI aut not yet supported

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az storage fs directory upload
Errors:
The command failed with an unexpected error. Here is the traceback:
MSI auth not yet supported.
Traceback (most recent call last):
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/storage/azcopy/util.py", line 108, in login_auth_for_azcopy
    token_info = _unserialize_non_msi_token_payload(token_info)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/storage/azcopy/util.py", line 144, in _unserialize_non_msi_token_payload
    parsed_authority = urlparse(token_info['_authority'])
KeyError: '_authority'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/az/lib/python3.6/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 657, in execute
    raise ex
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 712, in _run_job
    return cmd_copy.exception_handler(ex)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/storage/__init__.py", line 371, in new_handler
    first(ex)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/storage/__init__.py", line 370, in new_handler
    raise ex
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 691, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/storage/operations/azcopy.py", line 74, in storage_fs_directory_copy
    azcopy = _azcopy_login_client(cmd)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/storage/operations/azcopy.py", line 125, in _azcopy_login_client
    return AzCopy(creds=login_auth_for_azcopy(cmd))
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/storage/azcopy/util.py", line 110, in login_auth_for_azcopy
    raise Exception('MSI auth not yet supported.')
Exception: MSI auth not yet supported.

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

authentication with az login --use-device-code using an AD account
az storage fs directory upload --file-system {} --account-name {} --auth-mode {} --destination-path {} --recursive --source {}

Expected Behavior
Directory to be uploaded successfully. It worked with version 2.27.2 of az cli on August 20th.
Environment Summary
Linux-4.9.0-13-amd64-x86_64-with-debian-9.13, Debian GNU/Linux 9 (stretch)
Python 3.6.10
Installer: DEB

azure-cli 2.30.0

Extensions:
storage-preview 0.7.4

Additional Context
target is an ADLS Gen 2 Data Lake

storage

Meet the same issue when calling "az storage blob directory upload"
azure cli version : 2.30.0
storage-preview version : 0.7.1
Error: WARNING: This command is implicitly deprecated because command group 'storage blob directory' is deprecated and will be removed in a future release. Use 'az storage fs directory' instead.
ERROR: The command failed with an unexpected error. Here is the traceback:
ERROR: MSI auth not yet supported.
Traceback (most recent call last):
File "C:\Users\shiswang.azure\cliextensions\storage-preview\azext_storage_preview\azcopy\util.py", line 81, in blob_client_auth_for_azcopy
token_info = _unserialize_non_msi_token_payload(token_info)
File "C:\Users\shiswang.azure\cliextensions\storage-preview\azext_storage_preview\azcopy\util.py", line 104, in _unserialize_non_msi_token_payload
parsed_authority = urlparse(token_info['_authority'])
KeyError: '_authority'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 231, in invoke
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 657, in execute
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 720, in run_jobs_serially
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 712, in run_job
File "C:\Users\shiswang.azure\cliextensions\storage-preview\azext_storage_preview_init.py", line 252, in new_handler
handler(ex)
File "C:\Users\shiswang.azure\cliextensions\storage-preview\azext_storage_preview_init.py", line 194, in handler
raise ex
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 691, in _run_job
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/init.py", line 328, in call
File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 121, in handler
File "C:\Users\shiswang.azure\cliextensions\storage-preview\azext_storage_preview\operations\azcopy.py", line 19, in storage_blob_upload
azcopy = _azcopy_blob_client(cmd, client)
File "C:\Users\shiswang.azure\cliextensions\storage-preview\azext_storage_preview\operations\azcopy.py", line 65, in _azcopy_blob_client
return AzCopy(creds=blob_client_auth_for_azcopy(cmd, client))
File "C:\Users\shiswang.azure\cliextensions\storage-preview\azext_storage_preview\azcopy\util.py", line 83, in blob_client_auth_for_azcopy
raise Exception('MSI auth not yet supported.')
Exception: MSI auth not yet supported.
To open an issue, please run: 'az feedback'

az storage fs directory upload is using azcopy which is integrated inside Azure CLI. After CLI core migrated to MSAL, MSAL won't return  refresh_token any more. This causes authentication failure with azcopy when users specify --auth-mode login. Users can work with --auth-mode key –account-key, --connection-string or --sas-token  or without any of them as a workaround.
We are working with MSAL team to see if they can add refresh_token in response and seek advice from Azcopy team if they can use MSAL cache directly.

Thank you for the suggestions, however authenticating with an AAD account is what I need, as in my context I currently don't have the other authentication options. ACLs are setup on my AAD account and I can't generate a SAS token, nor do have I access to an account key for the target storage.
In my case where I needed to transfer files from a remote server to the storage, I ended up having to download them to my desktop and upload them to the storage through Azure Storage Explorer :(

Sorry for the inconvenience. If you have to auth with AAD, can you try azcopy directly instead of cli's integrated azcopy?

Sorry for the inconvenience. If you have to auth with AAD, can you try azcopy directly instead of using cli's integrated azcopy?

Ran a quick test and it did work with direct azcopy, thanks for the suggestion :)

PS: For me it worked to provide the AZCOPY vars for SP client secret login, since the fs directory uses az copy internally this resolved the auth issue for me.


Integrity: Apply for Package Reservation in Nuget

Consider applying for the package reservation in Nuget.
The below were taken from here.
When reviewing any application for ID prefix reservation, the NuGet.org team will evaluate the application against the below criteria. Please note that not all criteria need to be met for a prefix to be reserved, but the application may be denied if there is not substantial evidence of the criteria being met (with an explanation given):

Does the package ID prefix properly and clearly identify the reservation owner?
Has the owner enabled 2FA for their NuGet.org account?
Is the package ID prefix something common that should not belong to any individual owner or organization?
Would not reserving the package ID prefix cause ambiguity, confusion, or other harm to the community?

When publishing packages to NuGet.org within your ID prefix reservation, the following best practices must be considered:

Are the identifying properties of the packages that match the package ID prefix clear and consistent (especially the package author)?
Do the packages have a license (using the license metadata element and NOT licenseUrl which is being deprecated)?
If the packages have an icon (using the iconUrl metadata element), are they also using the icon metadata element? It is not a requirement to remove the iconUrl but embedded icons must be used.

Prefixes are now reserved.


[1.16.5] Sound manager not starting properly

Mod Version:
DynamicSurroundings-1.16.5-<IP_ADDRESS>.jar
Forge Version:
36.2.8
Link to client log:
N/a
Link to crash log:
N/a
Description:
Ive noticed a issue that seems to be present with 170 mods that I use, im not sure if its cross-mod issue
but I have noticed the sound filtering system not working correctly if one were to ajust the sound play settings or if they are blocked/culled
I noticed that some of these issues are somewhat corrected manualy by playing the edities sound in the sound manamgent GUI. but even then it needs to be redone after every instance the game is restarted/re-opened
tihs mod means alot to me and it would be nice if some fix could be made. if I had to take a guess, it might have todo with whatever is being done to ajudst the sounds not loading up properly

I'm not sure if this is the same issue but in my modpack I lowered the volume of a few specific sounds (armor) and I have to open the GUI and click OK everytime I reopen the game in order for it to work correctly. The settings get saved but they do not work correctly until I open and close the GUI.

Seems related