How is it possible to change/enhance the default configuration files for the standard Nginx ingress?

This is not a true issue but a question that aims to overcome a problem.
I am configuring my microservices/docker application (developed locally using docker and Envoy as reverse proxy) in MicroK8s. The applications are written in .net core 2.1 and do use IdentityServer4 as an OAuth2 provider (it is a docker service among the other services).
The services configured in my local environment (with Envoy as reverse proxy) are working fine; configured in MicroK8s they works, but the authentication give a problem when the auth service invoke (through a POST call) the original web application:
POST requests are not supported / 502 Bad Gateway / openresty/<IP_ADDRESS>
Searching on the internet i didn't find really anything specifically related to this (too much) generic error; the closer one was this post:
https://medium.com/@mshanak/solve-nginx-error-signin-oidc-502-bad-gateway-dotnet-core-and-identity-serve-bc27920b42d5
The author (Mahdi Shanak) suggest to modify the config files of nginx to increase the size of the proxy buffers. The files touched are "/etc/nginx/nginx.conf" and "/etc/nginx/conf.d/default.conf".
But here is the problem: if i want to try this solution i have to modify the files.... but i used the default ingress implementation (enabling the "ingress" module) and i don't see any easy way to change the default configuration files for nginx nor any documentation on how to do so.
Use the default modules of microk8s is very handy since i have a full environment running in few minutes, but i also need to go a little further in the configuration... Can you give me some advice on how i can proceed?
Thanks in advance.

Many of the nginx configuration can be added to the Ingress resource as annotations.
I don't think there is a need to actually change the nginx ingress controller.
Here's the link to all nginx ingress annotations.
https://kubernetes.github.io/ingress-nginx/user-guide/nginx-configuration/annotations/
Not particularly sure the actual mapping between the annotation and nginx conf though.

This might also help you configure nginx ingress.
https://github.com/dotnet/aspnetcore/issues/19293

Hi @balchua and thanks for the response.
After further search and tests, i found the following page:
https://www.b2-4ac.com/nginx-ingress-controller-errors/
(it is a confirm of the previous thread mentioned in my original comment, and the proof that some additional configuration is needed)
After that i added some annotation to the ingress, in particular the following:
nginx.ingress.kubernetes.io/proxy-buffering: "on"
nginx.ingress.kubernetes.io/proxy-buffer-size: 16k
nginx.ingress.kubernetes.io/proxy-body-size: 8m

They can overcome some of the common problems specified in the previous link and in particular some of my problems (in fact now i have different problems.... but not the same as before!!).
I think that i will continue searching for the correct configurations.
Still, i think that having to add multiple times the same annotations to different ingress definitions (and in a micro-service environment i can have a lot of different ingress) is still a lot of effort... being able to configure a standard behavior for the whole cluster seems to me something reasonably.
Thanks for your tips, especially for the advice about how to change the ConfigMap... i was still searching around the net the recommended way to do it in microk8s.

Different workloads will always require different configurations.
I think the way to configure nginx ingress controller is to edit the ConfigMap.  Give it a try and let us know how it goes. 👍

I will, thanks again!!

I can confirm that both approaches work:

By specifying the values as annotation of the ingress:
annotations:   nginx.ingress.kubernetes.io/rewrite-target: /$1   nginx.ingress.kubernetes.io/proxy-buffering: "on"   nginx.ingress.kubernetes.io/proxy-buffer-size: 16k   nginx.ingress.kubernetes.io/proxy-body-size: 8m
By creating a patch to the default nginx configmap:
`
apiVersion: v1
kind: ConfigMap
metadata:
name: nginx-load-balancer-microk8s-conf
namespace: ingress
data:
proxy-buffering: "on"
proxy-buffer-size: 16k
proxy-body-size: 8m
large-client-header-buffers: "4 16k"
use-forwarded-headers: "true"

The patch is applied with the command:
kubectl -n ingress patch configmap nginx-load-balancer-microk8s-conf --patch "$(cat nginx-config-map-patch.yaml)"
`
If there are no obvious drawbacks (performance issues for instance) I will use the second way, as it makes the yaml file cleaner and easier to maintain.
Thanks again for the help.


fix(type): allow type styles to inherit font-family

Part of #393, we need to support alternative font-family definitions for our type styles. The update to the preview example is to show that these styles should work in other languages.
Changelog
New
Changed

Type styles no longer include font-family in their definition
The type reset now has optional body/mono/base-font-size arguments

Removed

Good stuff!

Hey @joshblack... we talked about this already and about alternate display styles that might add a little more clarity so I know you're aware of it, but did you want to merge this for now anyway.
At larger sizes the languages just start to run into each other roller derby style.


groovy was not showing in Extensions market

Type: Bug
groovy was not support in vscode
VS Code version: Code 1.84.2 (1a5daa3a0231a0fbba4f14db7ec463cf99d7768e, 2023-11-09T10:51:52.184Z)
OS version: Windows_NT x64 10.0.19045
Modes:

Thank you for opening the issue. Sorry to hear you are experiencing the installation issue. It appears to be specific to VS Code editor and tools. It is best to reach out VS Code team at  https://github.com/microsoft/vscode/issues.
-VS Marketplace Team


Dynamically update Scylla targets

When adding / removing Scylla nodes from a cluster, one needs to manually update Prometheus config.
It would be much better to use dynamic information, maybe using one of Prometheus service discovery mechanisms or a new custom one.

This can only be solved by an external element like consul


[Windows] Always update agent configs on startup

When starting antrea-agent directly from helper script. The agent
configurations need to be updated every time before starting
the antrea-agent process because the k8s and antrea-controller
API server may be changed after the last startup.
Signed-off-by: Rui Cao<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@56f6ea9). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1847   +/-   ##
=======================================
  Coverage        ?   42.59%
=======================================
  Files           ?      111
  Lines           ?    13832
  Branches        ?        0
=======================================
  Hits            ?     5892
  Misses          ?     7441
  Partials        ?      499

Flag
Coverage Δ

unit-tests
42.59% <0.00%> (?)

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

/test-all

/test-windows-conformance-containerd

/test-windows-networkpolicy-containerd

/test-windows-containerd-conformance

/test-windows-containerd-networkpolicy

/test-windows-containerd-networkpolicy

/test-windows-containerd-networkpolicy

/test-windows-conformance

/test-windows-networkpolicy

/test-windows-containerd-conformance

/test-windows-containerd-networkpolicy

/test-windows-networkpolicy

/test-ipv6-only

/test-ipv6-all


Roll engine fb5b097e5c20..aea7071e2731 (2 commits)

https://github.com/flutter/engine/compare/fb5b097e5c20...aea7071e2731
git log fb5b097e5c20..aea7071e2731 --no-merges --oneline
aea7071e2 Roll src/third_party/skia e6c85755832b..e70c6b13dd55 (1 commits) (flutter/engine#6766)
1132d6c42 Roll src/third_party/skia 11407e56f277..e6c85755832b (2 commits) (flutter/engine#6765)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

@liyuqian : New roll has been created by https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Repo has passed aea7071e27312a22cd58951b9a4459f3b640891e; will open a new dry run.


False positives of missing_required_field when encountering empty transfer_type values in transfers.txt

Bug report
The GTFS Validator is generating false positives of the missing_required_field when it encounters empty transfers.txt#transfer_type field values.
Describe the bug
Given an empty value within the transfers.txt#transfer_type field (which is valid according to the spec), when the validator checks the file, then it will falsely raise a missing_required_field notice.
How we reproduce the bug
Example feed: Archive.zip
In this feed in the transfers.txt file, the following use case is shown:
from_stop_id,to_stop_id,transfer_type,min_transfer_time
1,2,,

This is a valid use case in that the transfer_type value is empty which indicates a "Recommended transfer point between routes".
Expected behaviour
The validator should not flag this as an error.
Observed behaviour
The GTFS Validator flagged the above-mentioned use case as an error.
Screenshots:
See relevant JSON output for the above-linked feed.
{
  "notices": [{
    "code": "missing_required_field",
    "severity": "ERROR",
    "totalNotices": 1,
    "sampleNotices": [{
      "filename": "transfers.txt",
      "csvRowNumber": 2.0,
      "fieldName": "transfer_type"
    }]
  },
...

Environment versions

validator version: v2, v3.0.1, v3.1.0
Java version:
OS versions:

Hello Evan,
Thanks for flagging this! This is super useful, having a reliable validation report is a top priority. 🙂
You are right, this field is considered required in the validator, so the notice is triggered for every row that has an empty value, regardless of whether the column is present or not.
👇 Source code from GtfsTransferSchema.java.
  @Required
  GtfsTransferType transferType();

This is another example of a slightly blurry definition of what "empty" means in the spec. We have a similar issue with timepoint, and we ended up differentiating between (a) an empty value but an existing column, and (b) an empty column.
I think we need to follow a similar logic here.

empty value but existing column: valid data

from_stop_id
to_stop_id
transfer_type
min_transfer_time

1
2

empty column: invalid data

from_stop_id
to_stop_id
min_transfer_time

1
2

cc @Cristhian-HA @bdferris @barbeau

@isabelle-dr @bdferris-v2 It seems that @Required annotation is used to validate both that the header exists (DefaultTableHeaderValidator.java) and that the field is non-empty (RowParser.java).
If I understand correctly, the goal here is to generate an error notice if the column header is missing, but no notice if the header is present and the field is empty?  Or is it conditional - the field should not be empty if min_transfer_time is non-empty?

@briandonahue the goal is to generate a notice if the file transfers.txt is present but the header transfer_type is not in the header of the file. No link with min_transfer_time here.
It looks like @Required in the schema file was not a good approach because it generates missing_required_field for every record that has an empty value in transfer_type.
This "0 or empty" semantic is often used for optional fields in the spec, I believe transfer_type is the only required field that has it. It's not great in terms of interpretation by a machine.

Could we use the MissingRequiredColumn notice for this?

Could we use the MissingRequiredColumn notice for this?

@isabelle-dr I believe so. I'll just add a validator that manually checks that the header exists, and throws that notice, and remove the @Required annotation.

@isabelle-dr on further review - I see that there is a GtfsTableContainer.TableStatus.INVALID_HEADERS status for Tables that is supposed to represent if columns are missing, which is tested here.  I think a better approach may be to add a new annotation, @RequiredColumn or @RequiredHeader? And potentially rename @Required to @RequiredField or @RequiredValue?
Then we would update the logic in AnyTableLoader.java to check both annotations and add the MissingRequiredColumn notice in either case.

Interesting. I think we also use @Required for files.
Brian is on holiday this week.
@aababilov, any wisdom to share?

I think we also use @Required for files.

You're correct, I missed that detail. Well, we could still do this and just add a @RequiredHeader annotation and leave the @Required functionality as-is.  I'd argue it might be clearer to have separate annotations for Fields, Headers, and Files but I could raise that as a separate issue if it seems useful...


Ergonomic way to configure Zeebe via chart values

As a Zeebe developer I'd like an easy way to configure Zeebe (for example via env variables) through chart values.
Right now this is not easily possible because setting extra env variables overrides the defaults.

closed via #14


VSCode can't open terminal by system software policy

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

VS Code Version: 1.58.0 (system)
OS Version: Windows 10 21H1 19043.1083

Steps to Reproduce:

Setup the Software Policy (in computer policy / Windows Settings / Security Settings)
Open terminal in vscode --> fail.

I don't sure it's a bug or not.
I wondered why Vscode open pty by the \\?\ prefixed path, and the document doesn't mention it.
Windows default Software policy only allow C:\WINDOWS but without \\?\C:\WINDOWS
So maybe the documents could be update to provide more information?

The \?\ is a UNC path https://en.wikipedia.org/wiki/Path_(computing)#Universal_Naming_Convention
It sounds like you're intentionally blocking what the terminal needs to do by setting up this policy? I'm not too familiar with software policies in Windows but can you opt out the VS Code install directory or something?


remove all references to Public Resets in comments

Codecov Report

Merging #1648 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1648      +/-   ##
==========================================
+ Coverage   87.33%   87.36%   +0.03%
==========================================
  Files          97       97
  Lines        6225     6225
==========================================
+ Hits         5436     5438       +2
+ Misses        563      561       -2
  Partials      226      226

Impacted Files
Coverage Δ

packet_unpacker.go
86.67% <ø> (ø)
:arrow_up:

session.go
74.06% <ø> (+0.31%)
:arrow_up:

Continue to review full report at Codecov.

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


Git - git.stage shortcut loses file list focus

Does this issue occur when all extensions are disabled?: No, it's related to the git extension

VS Code Version:

Version: 1.87.0 (Universal)
Commit: 019f4d1419fbc8219a181fab7892ebccf7ee29a2
Date: 2024-02-27T23:42:56.944Z (1 wk ago)
Electron: 27.3.2
ElectronBuildId: 26836302
Chromium: 118.0.5993.159
Node.js: 18.17.1
V8: <IP_ADDRESS>-electron.0

OS Version:
Darwin arm64 22.5.0

Steps to Reproduce:

Add this keybind in vscode:

    {
        "key": "ctrl+alt+m",
        "command": "git.stage",
        "when": "view.workbench.scm.visible"
    },

Have some unstaged files and focus on one

Click ctrl + option + m to stage your selected file

Previously the focus would go to the next item of the list, but now the focus go to the workbench view, which is pretty annoying when you want to do that multiple times in multiple files

I used this shortcut massively last year and a recent update (something like 1-2 moths ago broke it)

I would like to add that there is also git.stageAll and git.unstageAll which always asks for repo even if there is one in focus, would be nice to have them worked. In that case workflow would be like this:

focus source control view
stage all files or go through every file with up, down and space or enter to review and stage what's needed
unstage some files (if stageAll was used)
ctrl + enter to commit

right now i have to either use mouse or move focus back to "source control view" every time

Hey! Any updates on this? 😬
It's being a while and I would love to get this fluid behaviour as before.
I wish I had the knowledge to contribute as well. If anyone could provide me some directions I could try digging into it.
Thanks!

This is important to me too and deserves attention

Likewise I've found the loss of focus behaviour here infuriating - for something so small and easy to fix, this issue as been on and off in vscode for ages. Would love to see it properly addressed

@LeonardoRick, sorry for not getting back to you on this until now. When you invoke the command using the keybinding, do you have the file that you are staging opened in the editor? I am asking since invoking the git.stage command using a keybinding only works if the file is opened in the editor, since the command does not have access to the selected item in the list.

+1


Use keyboard to open method information

VSCode Version: 1.2.0-Insider
OS Version: Windows 10 Build 14332

How do I use the keyboard to open detailed method information?

Cheers,
Trevor Sullivan
Microsoft MVP: Cloud & Data Center Management
https://trevorsullivan.net
https://twitter.com/pcgeek86

Hitting ctrl+space toggles the information


layers: Migrate surface support check to core_validation

fix memory leak of pQueueFamilyIndexSupport
remove vestige of queue family support check in the swapchain layer

This is running on our test farm now. I'll merge it once it's done.


Removes spring-cloud-sleuth-zipkin-stream

Fixes #727

Codecov Report

Merging #756 into 2.0.x will decrease coverage by 0.13%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##              2.0.x     #756      +/-   ##
============================================
- Coverage     71.48%   71.35%   -0.14%
+ Complexity     1267     1232      -35
============================================
  Files           187      183       -4
  Lines          4885     4747     -138
  Branches        529      504      -25
============================================
- Hits           3492     3387     -105
+ Misses         1111     1089      -22
+ Partials        282      271      -11

Impacted Files
Coverage Δ
Complexity Δ

...ramework/cloud/sleuth/trace/SpanContextHolder.java
78.94% <0%> (-5.27%)
16% <0%> (-1%)

...k/cloud/sleuth/zipkin2/DefaultEndpointLocator.java
84.61% <0%> (+5.12%)
17% <0%> (+1%)
:arrow_up:

...oud/sleuth/stream/ServerPropertiesHostLocator.java
75% <0%> (+5.55%)
14% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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

Maybe we should update the docs with the info that we did this?


Fix spelling errors.

The lintian QA tool reported a spelling error for the Debian package build:

allowd -> allowed

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

Don't raise the barrier to contribution with a CLA: https://github.com/Unidata/cftime/pull/240
I'm not going to spend time on legal shenanigans to contribute trivial changes.
You can get the patch from the Debian package: https://salsa.debian.org/debian-gis-team/cftime


Firefox 63 WebExt DDNs

User story
As an extensions developer, I want to learn about the new WebExtension/Browser Extension features in Firefox 63, so I can use them in my own projects.
Acceptance criteria

[x] All DDNs complete.

[ ] Reviews request from engineers for all documentation work.

Tasks
Items for Irene

[x] Implement Google Chrome New Tab Page color properties [2 points]

[ ] NEW Implement theme experiments [2 points]

[ ] NEW Allow theming sidebars [1 point]

(smaller items, total 3.5 points)

[ ] webRequest.getSecurityInfo should not return localised values

[x] Add sameSite attribute (no_restriction, lax, strict) to browser.cookies API

[x] Implement support for the 'highlighted' API for multiselect tabs with tabs.query

[x] Expose multiselected status to "tabs.Tab.highlighted", and allow to change it via "browser.tabs.highlight()"

[x] Remove "Open in Sidebar" feature

[ ] NEW contextMenus do not support other protocols (ex. 'magnet:', 'acestream:', 'sop:*')

[x] NEW contextMenus.OnClickData should provide selector of the clicked element

[x] NEW Implement support for updateProperties.highlighted in browser.tabs.update()

[x] NEW Impossible to change multiselection state of tabs by setting "highlighted" attribute of tabs without focus change

[ ] NEW Allow extensions to access tab's "attention" flag

[ ] NEW Provide a management.install command for themes only

Items for Will

[x] Add an API to list installed search engines and execute searches using only those engines [1.5 points — already started, but it needs cleanup; see will's existing comments]

[x] NEW commands API is not treated as user input [1 point]

[x] NEW Implement support for the 'onHighlighted' API for multiselect tabs [1 point]

[x] NEW Implement "visible" parameter in menus.create and menus.update [1 point]

[x] NEW Add ability to set access key to context menu item [1 point]

(smaller items, total 3.5 points)

[x] Add speculative connection type for webrequest and proxy

[x] activeTab permission not granted for tab on which the contextmenu is used

[x] Choose browserAction text color among white and black, maximizing contrast

[x] webRequest, add a public documentation about "documentUrl" [Will did most of this, just needs BCD adding]

[x] Make accentcolor and textcolor optional

[x] tabs.onUpdated filtering uses isarticle, details uses isArticle.

[x] Allow to set browserAction badge text color

[x] TLS status API should return null for certain attributes

[x] topSites places support

New tab colors was much simpler than it appeared. I only had to update browser compatibility since the properties are already defined as a result of Chrome support for them.

Asked about this one: #1470516 as I'm not sure that it  needs anything but a release notes comment because none of the existing documentation says anything about localized strings.

Remaining DDNs:
1347207 should be done today.
1418602 asked what changes there are to document here. It isn't clear from reading the bug whether theme creators have to define something in particular to have this work.
1280370 should be done this afternoon.
1396684 This one may not be done until tomorrow (Friday) morning.
1369209 Another one that may not be done until tomorrow (Friday) morning.

1369209 - Some of the information in the bug description implies that the management.install method only applies to lightweight themes. It doesn't seem to make sense to document it if LWT is going away.

#1347207 - Removing this one from the issue as it is more extensive than a DDN. I will discuss the subject of experimental themes with the team and determine how ready it is for release, then come up with a plan to document it fully.


Monthly statistics reports - make dates in copy dynamic

Context
There are several hard-coded dates in the copy for the monthly statistics report. These need to be replaced with helper methods that return dates appropriate to the current date.
Changes proposed in this pull request

[x] Extract a MonthlyStatisticsPresenter to encapsulate any helper methods needed (and reduce the number of variables passed from controller to view template).
[x] Replace hard-coded dates in Introduction.
[x] Replace hard-coded dates in Preamble for section 4.
[ ] Replace hard-coded dates and counts in Footnotes. (I think this needs some alternate copy ready for next year, without the references to UCAS).

Guidance to review

Anything missing?

Link to Trello card
https://trello.com/c/wehZ0e0y/4189-monthly-report-make-dates-in-copy-dynamic
Things to check

[x] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[x] This code does not rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] Required environment variables have been updated added to the Azure KeyVault

Merging this PR and splitting out the concerns about footnotes to a separate trello card


Sigma settings are not working when using neo4j

Perhaps it is an error on my part but I use the same setup for gefx as I do for cypher and thus far it appears as though I cannot change the settings on a sigma instance.
I can loop through the edges/nodes in the graph and change their colours individually but I think this will cause a problem the larger the graph becomes.
Any help would be greatly appreciated.
Examples of non-working versions of what I have tried so far:
var s = new sigma({
    container: 'graph-container',
    type: 'canvas',
    settings: {
        defaultNodeColor: '#ec5148',
    }
});

sigma.neo4j.cypher(
    { url: 'http://localhost:7474', user: 'neo4j', password: 'admin },
    'MATCH (n:Node) OPTIONAL MATCH (n)-[r]->(m) RETURN n,r,m LIMIT 100',
    s,
    function() {
        s.refresh();
    }
);

sigma.neo4j.cypher(
    { url: 'http://localhost:7474', user: 'neo4j', password: 'admin },
    'MATCH (n:Node) OPTIONAL MATCH (n)-[r]->(m) RETURN n,r,m LIMIT 100',
    {
        container: 'graph-container',
        settings: {
            defaultNodeColor: "#F00"
        }
    },
    function(s) {
    }
);

If it helps anyone, I noticed line 82 of sigma.neo4j.cypher.js (within sigma.neo4j.cypher_parse) overrides the colour setting (and a few others) after the cypher query is executed. Commenting this line out has worked for me.


Inconsistent behavior from D2Reader.goTo()

I am trying to use the goTo function to navigate. However, it is inconsistent. Sometimes to goes to the correct location, other times it just says at the cover page. This is the code I am using:

Maybe I am missing something?

it seems like calling D2Reader.goTo on a button click is consistent

it seems like calling D2Reader.goTo on a button click is consistent

Please provide more details of what you mean by inconsistent .

I think this was the issue with using the navigate function in a useEffect. I moved it out into a separate function and its working fine now. closing


Fix: Enhance get orgServer logic

Fix: Enhance get orgServer logic
Bug: When trying to run the smoke tests by being in a user home workspace, tests fail
Root cause
When kubeconfig don't has specified context may get rootServerURL failed.
[knarra@knarra kcp-tests]$ ./bin/kcp-tests run all --dry-run | grep "Multi levels workspaces lifecycle should work" | ./bin/kcp-tests run -f -
INFO Sep 20 21:46:53.878: serverURL is
INFO Sep 20 21:46:54.891: Error running /bin/kubectl get workspace/~ --server= -o=jsonpath={.status.URL}:
Error from server (NotFound): clusterworkspaces.tenancy.kcp.dev "~" not found
INFO Sep 20 21:46:54.891: Getting home workspace server failed: "exit status 1"

[knarra@knarra kcp-tests]$ kubectl kcp ws .
Current workspace is "root:users:lw:ao:rh-sso-knarrakcp".

Fix Solution
Change get the root worksapce server by parse the workspace.kcp.dev/current.cluster.serverURL.
Local Test Record

$ ./bin/kcp-tests run all
$ ./bin/kcp-tests run smoke

$ ./bin/kcp-tests run all
started: (0/1/5) "[area/transparent-multi-cluster] Author:pewang-Critical-[Smoke][BYO] Validate creating, modifying and deleting a deployment from KCP get synced to the pcluster [Suite:kcp/smoke/parallel/minimal]"

started: (0/2/5) "[area/workspaces] Author:pewang-Medium-[Smoke] Multi levels workspaces lifecycle should work [Suite:kcp/smoke/parallel/minimal]"

started: (0/3/5) "[area/apiexports] Author:pewang-Critical-[Smoke] Verify APIBinding working with personal workspace [Suite:kcp/smoke/parallel/minimal]"

skip [/Users/wangpenghao/Upstream/kcp-tests/test/extended/syncer/syncer.go:23]: No pcluster kubeconfig set for the test scenario

skipped: (11.9s) 2022-09-26T02:13:24 "[area/transparent-multi-cluster] Author:pewang-Critical-[Smoke][BYO] Validate creating, modifying and deleting a deployment from KCP get synced to the pcluster [Suite:kcp/smoke/parallel/minimal]"

started: (0/4/5) "[area/common-version] Author:knarra-Medium-[Smoke] Checking kcp server version should display correctly [Suite:kcp/smoke/parallel/minimal]"

passed: (11.4s) 2022-09-26T02:13:36 "[area/common-version] Author:knarra-Medium-[Smoke] Checking kcp server version should display correctly [Suite:kcp/smoke/parallel/minimal]"

passed: (34.5s) 2022-09-26T02:13:47 "[area/apiexports] Author:pewang-Critical-[Smoke] Verify APIBinding working with personal workspace [Suite:kcp/smoke/parallel/minimal]"

passed: (1m33s) 2022-09-26T02:14:45 "[area/workspaces] Author:pewang-Medium-[Smoke] Multi levels workspaces lifecycle should work [Suite:kcp/smoke/parallel/minimal]"

started: (0/5/5) "[area/workspaces] Author:zxiao-Medium-[Serial] I can create context for a specific workspace and use it [Suite:kcp/smoke/serial]"

passed: (15.4s) 2022-09-26T02:15:00 "[area/workspaces] Author:zxiao-Medium-[Serial] I can create context for a specific workspace and use it [Suite:kcp/smoke/serial]"

4 pass, 1 skip (1m48s)

/lgtm


[DOCS] How to get your receipts for a tax return

Context
Meals of Gratitude put out an update telling reminding their community how to get receipts for their donations to file their tax return correctly. We should document this for users.
Who does this change impact?
Contributors to collectives.
How do users use it?
Users can get their receipts by

Going to 'Settings' > Payment receipts (preferred)
Viewing their debit transactions on their personal profile
Accessing their donations from individual collective pages

Docs Process

Overview

Changes to be made

[ ] Add 'Downloading your tax receipts' to 'financial contribtors' section
[ ] Add link to the above in FAQ.

### Estimate of effort

@kaylarep @BenJam Just touching base about @alanna's comment above. I'm not too versed in these tax processes. Have these been added to the OCF and OSC docs? If so could we make a plan to action and then I can ensure these are linked out to. 😊

https://app.gitbook.com/@opencollective/s/help/financial-contributors/receipts
Currently in progress, aiming to add more screenshots just clarifying what I can do $$ wise to show these receipts  😊

Sorted! https://docs.opencollective.com/help/financial-contributors/receipts Please let me know if anything else need to be added or changed :)


fix text above GetCL when concentration selected

On the Predict page, if you click the “concentration button” the sentence above the CL button at the bottom right of the page, “The selected % threshold to estimate…” is not present on both the French and English pages.

fixed


Converted MesheryNotification.js to functional

Notes for Reviewers
This PR fixes #8677
Signed commits

[x] Yes, I signed my commits.

please make sure to always sign-off your commits

please make sure to always sign-off your commits

I tried to sign the commit again, but i am unable to do it now. Let me know if it is required this time.
I'll definitely sign the commits next time.

this component becomes redundant with the new notification center updates .

more efforts here might go in vain .

more efforts here might go in vain .

I was assigned this task for the issue #8677 by @Rajdip019 . Will this be merged or not?

Can you check all the other components as well in this issue #8668 @aabidsofi19 . There could be other components as well which may become obsolete after new updates people are working on.

Thanks, @ishashankmittal 👏
Heads-up: there's a merge conflict to look into.

Hey @ishashankmittal
Let's discuss this as your agenda item for today's Meshery development meeting,
Please add it if it's not already there : https://docs.google.com/document/d/1Dg4Uz9rGTuGHmYZwIt47z1j5x3LmnpxD1M1SgypAkQ0/edit


Add tokens key to tokensUserProvider in access_tokens_with_lucid.stub

🔗 Linked issue

❓ Type of change

[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

Add tokens key to tokensUserProvider in access_tokens_with_lucid.stub to fix typescript build error.

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Nice catch!


Expose event for watch task completion

I'm using watch to run build commands when files are changed.
The process takes a while, so if I refresh the browser before it completes I still get back the old files.
This can be confusing because it keeps tricking me into thinking my changes didn't work.
I want to create a middleware for connect to delay requests while watch tasks are running. Looking at the code, it appears watch already has internal events that fire when tasks start and stop.
https://github.com/gruntjs/grunt-contrib-watch/blob/master/tasks/watch.js#L31-L46
I would like to expose these events by either passing them up through grunt, or exposing a way to specify callbacks in the watch config. I believe this functionality could be useful in other situations other then my own.
Is that a feature that could be easily added, or would be possible to merge if I wrote it myself?
Thanks!

why not incorporate livereload or something else that will refresh the page once the changes are done instead of doing it manually?

Thanks for the reply Eddie!
Some people have been using live reload, but it's not something we have set-up out of box for everyone.
The build process can take a while after files are saved. People not used to live reload still switch to the browser and expect to see their changes right away after the refresh.
We've been using the events in a forked version and it's helped reduce the confusion.
Let me know if you have any other concerns or things I can help with!

That sounds more like a training issue than a technical one. Just tell them the page will automatically reload once its ready to. Or do something to speed up the time between changes going in and the build finishing.


Keep custom editor theme when changing editor settings

Currently whenever the editor settings are changed in any way, the theme is reset to the default one and an editor restart is required to bring the custom theme back.

Thanks!

Cherry-picked for 3.4.

Cherry-picked for 3.3.1.


Switch to using 0.6 planner for server-side planning

has some issues but will send follow up fixes.


Doc: change .svg to .png.

There is a bug with .svg images - https://github.com/sphinx-doc/sphinx/issues/3266
If we use .svg then we also need to require Sphinx 1.5.1.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1623/ for details.

Intel CC build successful! See https://ci.trafficserver.apache.org/job/icc-github/56/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1518/ for details.

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/188/ for details.

@SolidWallOfCode  Why is this not merged?

I think I will update to require Sphinx 1.5.1 instead. That's in general availability.

Yeh, that's fine, I upgraded Sphinx long ago on the Docs build box.

Replaced by #1606.


Relax mvdr dtype constraint

To make MVDR work in the neural network distributed training. Address https://github.com/pytorch/audio/issues/1991

Can you run this function in some audio data for both cfloat and cdouble and compare the outputs? This way we will know if running the operator in cfloat can lead to diverging results in some cases due to numerical issues.

The stv_power solution of MVDR gets nan outputs if using torch.cfloat dtype. I need to debug it.
Using torch.cdouble:
ref_channel:  15.035907456985868
stv_evd:  16.563734673832553
stv_power:  17.820481909929907

Using torch.cfloat:
ref_channel:  15.040328979492188
stv_evd:  16.56372833251953
stv_power:  nan

The reason why the value becomes nan is the overflow of cfloat tensor.
a = torch.tensor([-2.7791e+12+2.3049e+11j], dtype=torch.cfloat)
b = torch.tensor([4.5982e+09+512.j], dtype=torch.cfloat)
c = torch.tensor([9.7214e+21], dtype=torch.cfloat)
print(a * b / c.real)

@fmassa @mthrok  Do you have any idea to address it?

Addressed the overflow issue by switching the division and multiplication:
a / c.real * b

The stv_power solution still downgrades a lot (from 17.82 to 8.32).
ref_channel:  15.040328979492188
stv_evd:  16.56372833251953
stv_power:  8.320789337158203

@ Emrys365 Have you had similar experience when using floating point dtype?

@Emrys365 Have you had similar experience when using floating point dtype?

Yes! I think this is a common observation in several works, e.g. Section 4.4 | Heymann et al; 2019 and Section 2.2 (4) | Zhang et al; 2021.
The higher precision is likely to improve the stability in operations such as matrix inverse (this is also involved in the power method).

After offline discussion with @Emrys365, a good workaround is allowing user to use either cfloat or cdouble for the input, and the MVDR module will implicitly convert the spectrogram to cdouble for computation, the output is cfloat so that users don't need to worry about manually converting the dtype.

The Si-SDR score looks fine now:
ref_channel:  15.035911560058594
stv_evd:  16.563732147216797
stv_power:  17.820480346679688

@nateanl So my understanding is that, the original issue about cdouble not supported in distributed training can be worked around by keeping the use of cdouble inside a module. Is that correct?

@mthrok Actually when I installed the PyTorch from the source, the cdouble tensor works in distributed training. So the current design is good in terms of functionality.
Here I just want to improve the usability of this module, by allowing user to use the default cfloat dtype and don't need to worry about the conversion. However, the test shows converting to cfloat fails to pass the gradcheck, which means the module is very sensitive to the precision of data.

To guarantee the MVDR module passes gradient check, we should keep the current design (i.e., force the users to use torch.cdouble for input tensor and let users decide whether to convert it to torch.cfloat for joint training).
In this way, users are aware that double precision is essential to MVDR module, and there's risk of losing gradient precision if they convert the output to floating point precision.


Is Inf time (fps) measued on 8 GPUs?

Is Inf time (fps) measued on 8 GPUs?

The fps that is benchmarked at the end of the training epoch is with multiple GPUs, but it normally contains complex post-processes for the results, you can use benchmark.py to get a more accurate inference speed.

Feel free to reopen the issue if there is any question


Finetune cli updates

added hyperparameters and validation file parameters to CLI
updated documentation to reflect said changes

Summary by CodeRabbit

New Features

Introduced the ability to pass hyperparameters and a validation file when creating a job from a file.

Documentation

Updated documentation to include examples of how to use these new options in the command line interface.

Bug Fixes

Added validation for hyperparameters and existence of the validation file.

its still not correct cause file_ids and filenames are mangeled i'm going to go with daaniyaan solution


Feature Request: terminal smaller than 80 columns when using -p argument

I like to "zoom in" on my terminal emulator to display/screenshot text art the largest possible setting. For still images, I can do this with the cat command, but .dur animations must be viewed in Durdraw, which cannot be viewed in a terminal narrower than Durdraw's interface. It would be neat if i could make the image larger when started in viewer mode, since the interface is never visible. Provided it's not more trouble than it's worth to implement, of course.
Thanks!

I hadn't considered super zoomed-in fonts. That's really cool!
This request makes a lot of sense to me. I would like for the whole editor to support less than 80 columns someday, but you're right, viewer mode should not have this restriction regardless.
I'll look at fixing this soon. Thank you for the request!

It should be fixed in the beta/dev build. Will be in 0.27.0 release. Thanks again!


Please provide an example with client.send using callback

Hello,
I'm new to node.js. I am able to connect and send netcat client commands using node-netcat.
What I've been struggling with is - I need to send a series of commands.
client.send (password)
(handle authentication)
client.send (command)
(handle result)
It would be incredibly helpful to see an example chaining a series of client.send to each other using callbacks.
I've tried a variety of ways, and I'm getting undefined in the callback, like it's not waiting for the result before proceeding.
I suspect it'd be a common usecase to chain client commands together, so it would be a huge help to provide an example.
Thank you!

Hi @pdbossman thanks for using this module, the last update was 7 years ago unfortunately no time to update it, happy to share an example of queuing cmds from a client to a server hope the example is clear enough
const { client } = require("node-netcat");
const lasync = require("lasync");

const c = client(5000);

c.on("open", () => {
        console.log("connect");
});

c.on("data", (data) => console.log(`receiving ${data}`));

c.start();

//
// queueing cmds and processing in sequence
//
// in each message I am using a ; as a delimiter
// for the message protocol between the server and the client
//

lasync.series(
        [
                (cb) => c.send("cmd 1;", cb),
                (cb) => c.send("cmd 2;", cb),
                (cb) => c.send("cmd 3;", cb),
        ],
        (error) => {
                if (error) {
                        console.error(error);
                        return null;
                }

                console.log("all messages sent");
        }
);


autocomplete box closes when typing ':'

In hint/xml-hint.js, when using namespaced elements such as text:p, the autocomplete box closes when ':' is typed. The box can be reopened with ctrl-space.
In autocomplete, the box should not closes when an input character is part of one of the suggestions.

Here a workaround:
hintOptions: {schemaInfo: schemaInfo, closeCharacters: /\ >/}

The value for closeCharacters in xml-hint.js could be set to this value as a default.

closeCharacters is currently not derived from the mode, so we can't set this in xml-hint.js, and for now setting it manually is the only option.

had a similar problem with # character
in CM6 I had to use:
validFor: /[a-gA-G]#?[a-zA-Z0-9]*/
see https://codemirror.net/docs/ref/#autocomplete.CompletionResult


Cannot publish to AMT sandbox, is HTTPS required?

I successfully deployed the tool in off-line version. Now I want to deploy it on AMT Sandbox but I get this error when running turkic publish:
turkic.api.CommunicationError: The value "http://localhost//?id=13" is not valid for the parameter ExternalURL.
This URL must use the HTTPS protocol..<PHONE_NUMBER>548 s)

In my config.py file I have this:
sandbox     = True # if true, put on workersandbox.mturk.com
localhost   = "http://localhost/"
...

Is it required to use HTTPS? I have not seen any reference to it in the instructions. If someone has run into the same problem I would appreciate a workaround solution if available or confirmation that HTTPS is needed.
Best,
Jeff

Hi @jeffshih
I'm facing issue while connecting with Amazon Mturk sandbox. My database connection and host connection are fine.
I'm new to Mturk, Can you guide me how to get these configuration
signature = "" # AWS secret access key ?
accesskey = "" # AWS access key ID ?
Thanks in advance

Hi @gsikheria
You can find these on the amazon mturk website
Just follow the steps till stage 4, Vatic will do the rest part.

Any solutions @jeffshih ? I am stuck at the same place, decided to try to https route but then am getting issues connecting to the appache server iteslf in the trukic status --verify

@PieterFabry  Nah, I can't solve this problem, so I first use a self assigned SSL certificate for testing and verification of the publishing procedure(You can easily google for some way to get a self-assigned SSL), then I buy an address and a SSL certificate for the official publishing.

Are there any updates on this issue?
I have the same problem where it returns the error,
"The value "http://localhost/?id=5" is not valid for the parameter ExternalURL. This URL must use the HTTPS protocol.."

I encounter same problem too, is there any solution?

Did anyone find a solution?


Build getting failed when tried to run the project.

Can you please update the project with latest libraries.
FAILURE: Build failed with an exception.

What went wrong:
The Android Gradle plugin supports only Kotlin Gradle plugin version 1.3.10 and higher.
The following dependencies do not satisfy the required version:
project ':barcode_scan' -> org.jetbrains.kotlin:kotlin-gradle-plugin:1.2.51

I fixed on this commit https://github.com/alfianlosari/flutter_qr_code_scanner_generator_sharing/pull/7

Pubspec.yaml:
update "qr_flutter: ^2.0.0"

Build.Gradle:
Compile version Updated to 28
Build version updated to 28


Actually update cache for all events that have impl for the UpdateCache trait.

Some cache events don't actually do anything currently, an example of this is the event BanAdd, where all it does is return Ok(()).
Related is #25

Looks like we're missing:

channel pins update
voice server update

Could be missing one or two!
For things like ban adds and integration updates I'm not sure there's much to meaningfully track, bans for example aren't given in GUILD_CREATEs over the gateway, so anything we could track is probably only partial.


update snapshots for 2.12

Description
update snapshots for 2.12
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.
[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@joshpalis let's hold off on approvals until CIs are passing. For 2.12 we should not have failing UT, and don't want to merge changes if it breaks infra builds


"Paste - and Keep Selections" with Multicursor

Is it possible to add "Paste - and Keep Multicurors" fearure?
Using "Paste - and Keep Selections" with Multicursor gives single selection.
So its not possble to continue work with multiselection.

works fine for me, it keeps selections and multicursors

Thanks for checking.
Does not work in PhpStrom 2021.2.2.
Will recheck in PhpStrom 2023 in 1 day.

PhpStorm 2023.1.2
Copied 4 lines selected via multi cursor
But after pasting, only 1 cursor was left.

I see, I was always copypasting to multicursors.
IntelliJ itself pastes it with 1 cursor, I made only a small modification to add the selection. This would require much more work.


UIL doesn't support scheme(protocol) by default

why im getting this?
E/ImageLoader﹕ UIL doesn't support scheme(protocol) by default [R.mipmap.ic_notification]. You should implement this support yourself (BaseImageDownloader.getStreamFromOtherSource(...))
java.lang.UnsupportedOperationException: UIL doesn't support scheme(protocol) by default [R.mipmap.ic_notification]. You should implement this support yourself (BaseImageDownloader.getStreamFromOtherSource(...))

Move mipmap/ic_notification to drawable/. We used image loader that's not support mipmap right now.


change popup logic to open immediately and retrieve message

as it is, it might cause popup blockers to block the popup because it's retrieving data, then opening the window

Also throwing a fit sometimes because of invalid code point when writing directly to output.


Deploying SharpShell Extensions - Restarting Explorer

Hi,
I added a reply / question to the CodeProject solution but wasn't sure where may be best. I am adding reference here as well:
https://www.codeproject.com/Articles/653780/NET-Shell-Extensions-Deploying-SharpShell-Servers?msg=5871061#xx5871061xx

Have a nice day. I want to report a problem. The program called Cybereason, which is used by one of our customers, blocks the sharpshell dll files we use in our project and removes them from memory at runtime. Since these dll files are defined as add-ons to windows explorer, windows explorer also crashes at that time. Only when we use ContexMenu, when we right click on any file, explorer.exe is being restarted. If we use IconOverlay, this reboot process repeats every second and the user becomes unable to use his/her computer. We couldn't figure out if there is a faulty method we are using or if there is another conflicting situation. Can you offer an idea or a solution suggestion on this issue?


Enable Dialogue concerning stock amounts

Test several design patterns for the following dialogue

[ ] If the guest has insufficient credits, a message is posted that says "We're sorry, you have insufficient noms please select a lower amount. Thank you!"
[ ] Notify guest if trying to add an item that is not available.

This can be a modal, error message in the window or whatever makes sense.

@komizutama Right now, when the user has insufficient credits to add an item, the plus button is replaced by a "Max allowed" text, and it is not possible to add the item, which means the error dialog would never be shown, unless we let the user click on the "Max allowed" text, which will only display the error dialog.
The same thing happens for unavailable items.
I understand it would be a good idea to clearly show the user the reason why they cannot add more of an item, but I'm not sure showing a dialog on click is the best solution.

Yeah definitely agree not Dialog/modal but also that ticket probably should have been cleaned up/de-priototized behind thia's other UI work since ember changes there inform best decision ... Apolou, I didn't see which issues you'd self assigned


Cordova 5.0.0

Missing changes

Could you compile successful and see a working map on Cordova 5.0.0 ?

And please submit to #test branch

Sorry, wasn't my intention to open this pull request, I was working with my friend to run the plugin on Cordova 5.0,  but we didn't test it well enough yet...
When we make sure that everything it's ok I would be happy to submit it to the test branch!
Thanks

allright!

@Nipher how tests go on?

Hi, I didn't test much, but I could compile successful and see a working map on Cordova 5.0.0,
We (me and @Phenome ) commented this line:
import org.apache.cordova.ScrollEvent;
And  we commented out the onEvent processing, that seemingly processes only scroll events...

Thanks @Nipher I'll test it day after tommorow and give feedback

Well done guys! @piernik let us know how it goes, I might have some time to test it as well during this week

Great! Thank you guys, let me know if I should do something else or if there is something wrong...
@Hirbod
Should I open a new pull request with the latest commit to the test branch?

I can confirm that Your version works well with cordova 5.0 - thank You vary much - great!!

@Nipher I have one problem.
After opening map default controllers (zoom in out and gps) are disapearing though they are still there (tapping on a blank space do proper action).

@wf9a5m75

I will test this later.

Works for me too, with the same little issue that @piernik pointed out. Thanks!

I've the same problem of @piernik too

Same issue here. Would the fix from @Phenome fixes all issues?

What went wrong:
Execution failed for task ':processDebugResources'.

com.android.ide.common.internal.LoggedErrorException: Failed to run command:
C:\Users\PANA\AppData\Local\Android\android-sdk\build-tools\21.1.2\aapt.
exe package -f --no-crunch -I C:\Users\PANA\AppData\Local\Android\android-sdk\pl
atforms\android-21\android.jar -M C:\tmp\HelloMap\platforms\android\build\interm
ediates\manifests\full\debug\AndroidManifest.xml -S C:\tmp\HelloMap\platforms\an
droid\build\intermediates\res\debug -A C:\tmp\HelloMap\platforms\android\build\i
ntermediates\assets\debug -m -J C:\tmp\HelloMap\platforms\android\build\generate
d\source\r\debug -F C:\tmp\HelloMap\platforms\android\build\intermediates\res\re
sources-debug.ap_ --debug-mode --custom-package com.example.myapp -0 apk --outpu
t-text-symbols C:\tmp\HelloMap\platforms\android\build\intermediates\symbols\deb
ug
Error Code:
1
Output:
C:\tmp\HelloMap\platforms\android\build\intermediates\res\debug\xml\conf
ig.xml:29: error: Error parsing XML: unbound prefix

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

BUILD FAILED
Total time: 18.634 secs
C:\tmp\HelloMap\platforms\android\cordova\node_modules\q\q.js:126
throw e;
^
Error code 1 for command: cmd with args: /s /c "C:\tmp\HelloMap\platforms\androi
d\gradlew cdvBuildDebug -b C:\tmp\HelloMap\platforms\android\build.gradle -Dorg.
gradle.daemon=true"
ERROR building one of the platforms: Error: C:\tmp\HelloMap\platforms\android\co
rdova\build.bat: Command failed with exit code 8
You may not have the required environment or OS to build this project
Error: C:\tmp\HelloMap\platforms\android\cordova\build.bat: Command failed with
exit code 8
at ChildProcess.whenDone (C:\Users\PANA\AppData\Roaming\npm\node_modules\cor
dova\node_modules\cordova-lib\src\cordova\superspawn.js:131:23)
at ChildProcess.emit (events.js:98:17)
at maybeClose (child_process.js:756:16)
at Process.ChildProcess._handle.onexit (child_process.js:823:5)
C:\tmp\HelloMap>
Line 29 is :
<vs:plugin name="plugin.google.maps">
        <param name="API_KEY_FOR_ANDROID" value="###MANUALLY REPLACED FOR COMMENT ##" />
        <param name="API_KEY_FOR_IOS" value="###MANUALLY REPLACED FOR COMMENT ##" />
    </vs:plugin>

The File at 'C:\tmp\HelloMap\platforms\android\build\intermediates\res\debug\xml' is ...
<?xml version='1.0' encoding='utf-8'?>
<widget id="com.example.myapp" version="0.0.1" xmlns="http://www.w3.org/ns/widgets" xmlns:cdv="http://cordova.apache.org/ns/1.0">
    <preference name="loglevel" value="DEBUG" />
    <feature name="Whitelist">
        <param name="android-package" value="org.apache.cordova.whitelist.WhitelistPlugin" />
        <param name="onload" value="true" />
    </feature>
    <feature name="HttpRequest">
        <param name="android-package" value="plugin.http.request.HttpRequest" />
    </feature>
    <feature name="GoogleMaps">
        <param name="android-package" value="plugin.google.maps.GoogleMaps" />
    </feature>
    <feature name="External">
        <param name="android-package" value="plugin.google.maps.External" />
    </feature>
    <feature name="Geocoder">
        <param name="android-package" value="plugin.google.maps.MyGeocoder" />
    </feature>
    <allow-intent href="market:*" />
    <name>HelloMap</name>
    <description>
        A sample Apache Cordova application that responds to the deviceready event.
    </description>
    <author<EMAIL_ADDRESS>href="http://cordova.io">
        Apache Cordova Team
    </author>
    <content src="index.html" />
    <vs:plugin name="plugin.google.maps">
        <param name="API_KEY_FOR_ANDROID" value="###MANUALLY REPLACED FOR COMMENT ##" />
        <param name="API_KEY_FOR_IOS" value="###MANUALLY REPLACED FOR COMMENT ##" />
    </vs:plugin>
    <access origin="*" />
    <allow-intent href="http://*/*" />
    <allow-intent href="https://*/*" />
    <allow-intent href="tel:*" />
    <allow-intent href="sms:*" />
    <allow-intent href="mailto:*" />
    <allow-intent href="geo:*" />
    <preference name="android-minSdkVersion" value="18" />
    <preference name="android-targetSdkVersion" value="21" />
</widget>
<!-- From: file:/C:/tmp/HelloMap/platforms/android/res/xml/config.xml -->

@wf9a5m75 I know that You are very bussy but can You give it a look and fix Your plugin to work with cordova 5.0? It would be great.

Hello Guys, Is there a plan to continue development of this plugin ? if not, can you guys suggest any other plugin which would allow a native map to be displayed. I already tried Angular Maps and its very slow.

If you want to use under cordova 5 you an check:  https://github.com/Nipher/phonegap-googlemaps-plugin this works for us.
Franklin Garzón
Lucha hasta alcanzar tus sueños
Date: Wed, 17 Jun 2015 06:21:17 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Subject: Re: [phonegap-googlemaps-plugin] Cordova 5.0.0 (#516)
Hello Guys, Is there a plan to continue development of this plugin ? if not, can you guys suggest any other plugin which would allow a native map to be displayed. I already tried Angular Maps and its very slow.
—
Reply to this email directly or view it on GitHub.

I am having #537 issue with @Nipher fork but was warking before with the master here.

This PR is not working for me:
/home/alexis/mafia/cordova/platforms/android/src/plugin/google/maps/MyPluginLayout.java:53: error: cannot find symbol
    this.root = (ViewGroup) webView.getParent();
                                   ^
  symbol:   method getParent()
  location: variable webView of type CordovaWebView
/home/alexis/mafia/cordova/platforms/android/src/plugin/google/maps/MyPluginLayout.java:261: error: cannot find symbol
        webView.requestFocus(View.FOCUS_DOWN);
               ^
  symbol:   method requestFocus(int)
  location: variable webView of type CordovaWebView
/home/alexis/mafia/cordova/platforms/android/src/plugin/google/maps/GoogleMaps.java:138: error: cannot find symbol
    root = (ViewGroup) webView.getParent();
                              ^
  symbol:   method getParent()
  location: variable webView of type CordovaWebView
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: /home/alexis/mafia/cordova/platforms/android/src/org/apache/cordova/file/AssetFilesystem.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
3 errors
:compileDebugJava FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':compileDebugJava'.
> Compilation failed; see the compiler error output for details.

@Nipher
I can comfirm: this version is not working. Tested with the current stable Cordova and Android release.

The master is up to date now! And works well with the new cordova!

:+1:


[vcpkg scripts] A few osx fixes

@m-kuhn any other additions necessary?

Sending it through the build pipeline now

Looks good, minimal additional changes are needed for iOS, either in here or as a follow up
https://github.com/Neumann-A/vcpkg/pull/9

@m-kuhn does the CMAKE_${lang}_OSX_DEPLOYMENT_TARGET_FLAG change work for you?

@m-kuhn does the CMAKE_${lang}_OSX_DEPLOYMENT_TARGET_FLAG change work for you?

Yes, this looks good.
Sidenote: it took me an extra run to figure out that a change to scripts/get_cmake_vars/CMakeLists.txt does not trigger a rebuild, so I had to fake a modification into scripts/cmake/vcpkg_configure_make.cmake to force rebuild.

@JackBoosY another baseline regression...

@JackBoosY another baseline regression...

Fixing in https://github.com/microsoft/vcpkg/pull/26783.

@Neumann-A Could you please resolve the conflicts?

/azp run

Hmm don't know what happened to jemalloc on osx. Has been green in https://github.com/microsoft/vcpkg/pull/26617/commits/184abe3c41315e6a353b0016eb8ebce38c372b92 and only thing I did was merge with master. Looks like a concurrency issue?

Hmm don't know what happened to jemalloc on osx. Has been green in 184abe3 and only thing I did was merge with master. Looks like a concurrency issue?

install: mkdir /Users/vagrant/Data/packages/jemalloc_x64-osx/Users/vagrant/Data/installed: File exists

This is a typical symptom of concurrency issues during make install on osx. We need a DISABLE_PARALLEL_INSTALL and/or a reliable mkdir wrapper for osx.

Should we wait for #27105 merge or just rerun the pipeline test?


Docs.rs won't build

For the past 2 releases, the docs on https://docs.rs/notan have failed to build. I tested the build locally on Ubuntu Unity 22.10, and they built fine. Build logs can be found here: 0.9.0, 0.8.0.

Ugh, I thought I fixed this making --all-features works again (https://github.com/Nazariglez/notan/commit/283422376933b297468a6713a632ea0b37adbf1a). I guess that shaderc is a no go for docs, so I will change how the doc generation works excluding it.
Thanks for the heads up @midnadimple !

Should be fixed now! thanks

np mam


[Bug] - Logout, reset to anonymous throws an error

Describe the bug
Passing true to the resetToAnonymous parameter of the logout function throws an error.
Expected behavior
The system should be able to logout and then create a new anonymous user immediately after.
Environment:

iOS deployment target: 13.0
RxFireAuth version: 1.1.0

This has been fixed in version 1.2.0, available now.


More details in Rigid Body add_force documentation

Specifies that add_force position is in local space among other details.
Documentation is now similar for add_force, add_central_force, apply_impulse, apply_central_impulse.

Thanks!

Cherry-picked for 3.2.2.


Fix style issues and improve update instructions.

Docs Build status updates of commit 76fc23f:
:clock10: Full build: the average full build time is 32 min(s) 2 sec(s), this is based on the last 107 build(s) for this repository.

Docs Build status updates of commit 76fc23f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-2.1.0/authenticate-azureps.md
:white_check_mark:Succeeded
View (azps-2.1.0)

docs-conceptual/azps-2.1.0/context-persistence.md
:white_check_mark:Succeeded
View (azps-2.1.0)

docs-conceptual/azps-2.1.0/install-az-ps.md
:white_check_mark:Succeeded
View (azps-2.1.0)

docs-conceptual/azps-2.1.0/migrate-az-1.0.0.md
:white_check_mark:Succeeded
View (azps-2.1.0)

docs-conceptual/azps-2.1.0/new-azureps-module-az.md
:white_check_mark:Succeeded
View (azps-2.1.0)

docs-conceptual/azps-2.1.0/overview.md
:white_check_mark:Succeeded
View (azps-2.1.0)

docs-conceptual/azps-2.1.0/uninstall-az-ps.md
:white_check_mark:Succeeded
View (azps-2.1.0)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix Style/RescueModifier offenses

From #752.

I'd rather use try here.

This is the only necessary change. The rest looks good!

:+1:

Closed by https://github.com/airblade/paper_trail/pull/762
Thanks Paul!


Use marshmallow to parse/validate input form

Consider using marshmallow to parse/validate input form in spawner.py.
This would separate input form handling from the spawner and should make code neater.

Consider using pydantic to leverage python types.


Cannot show .js outline after opening long simple .js file

Issue Type: Performance Issue
Open VS Code. Open workspace "lib-translit".
Open file TranslitTable.js. Show outline pane and wait for symbols to show.
Open file dict_en.js. Wait for symbol (there is only one) to show. (This is
an encoded CMUDICT dictionary, about 10000 lines long)
Open file Translitor.js.
Notice: the processor usage is high. Usually, after few minutes it shows "No symbols found in document 'Translitor.js'". It seems to be a bit random.
If it shows the symbols, open the file DomAnnotator.js.
Note: If dict_en.js is not opened, the symbols for all the files are shown quickly.
VS Code version: Code 1.28.0 (431ef9da3cf88a7e164f9d33bf62695e07c6c2a9, 2018-10-05T14:58:53.203Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4510U CPU @ 2.00GHz (4 x 2594)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
7.89GB (4.36GB free)

Process Argv

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
    1	    68	  8168	code main
    0	    47	  1656	   shared-process
    0	   119	  7836	   gpu-process
    0	    67	  8236	   window (Issue Reporter)
    0	    88	 11804	   window (Translitor.js - lib-translit (Workspace) - Visual Studio Code)
    0	    20	  3960	     watcherService
    0	    61	  8716	     extensionHost
   27	   653	 10436	       electron_node tsserver.js
    0	    58	  2028	         electron_node typingsInstaller.js typesMap.js
    0	    11	  9340	     electron-crash-reporter

Workspace Info
|  Window (Translitor.js - lib-translit (Workspace) - Visual Studio Code)
|    Folder (translit): 7 files
|      File types: js(6) css(1)
|      Conf files:;

Extensions: none

60677-outline-afler-long.zip

Ups, I reported the problem while my son was logged in into github.

Thanks for the example code. I've opened https://github.com/Microsoft/TypeScript/issues/27960 to track the root cause of this issue upstream


Are settings such as AT+CWAUTOCONN unconditionally written to flash?

Hi,
I'm just wanting to clarify the behavior for flash writes so I don't accidentally wear out the flash. AT+CWAUTOCONN says "The configuration changes will be saved in the NVS area."
Should I expect this command to write to flash every time it is executed, or does it only save the setting when it changes? The Zephyr driver writes this value each time the device reboots, so I will need to change the behavior if the write is unconditional

@campbellyoung Hi, thanks for your observation! All nvs write operations will not write to flash unconditionally.
ESP-IDF merged this feature on this PR: https://github.com/espressif/esp-idf/pull/3310, and ESP-AT got this after v<IP_ADDRESS> version.
Please know.

I'll update the esp-at documentation to make it clearer of flash write.

Great, thanks for the clarification! I thought that would be the case but it’s nice to have it confirmed. Appreciate the detailed response 🙂


Microk8s should expose the docker agent to allow for local builds

Hi all,
Microk8s should expose the docker agent to allow for local container builds using the daemon included inside the snap, I.E:
microk8s.docker build .
microk8s.docker login someregistry.catloverextreme.com

Etc.
Cheers,

Calvin

See also minikube doc here: https://kubernetes.io/docs/getting-started-guides/minikube/#reusing-the-docker-daemon

yes, I think a better option would be to have local docker client re-use
the docker daemon running from the snap
On Tue, May 29, 2018 at 7:13 AM Oliver Burkhalter<EMAIL_ADDRESS>wrote:

See also minikube doc here:
https://kubernetes.io/docs/getting-started-guides/minikube/#reusing-the-docker-daemon
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/juju-solutions/microk8s/issues/31#issuecomment-392740116,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAET1dEYsQfrnHnH-CeNc-I9-NfEgd2Dks5t3S1rgaJpZM4UQPj4
.

I agree with @marcoceppi   I would like to use docker client and docker-compose on my laptop while microk8s is running. There should be a way to expose the microk8s docker daemon to other users on the machine.. would make image development faster and simplify tooling to not have to use special commands.

@bkcsfi you should already be able to do that by pointing your docker client to the socket the docker daemon is using.
docker -H  unix://var/snap/microk8s/current/docker.sock

Can confirm this works for at least getting the Docker client talking to the internal daemon. Worth pointing out though that another slash is required
> docker -H unix:///var/snap/microk8s/current/docker.sock ps
root@other-k8s:/var/snap/microk8s/current# docker -H unix:///var/snap/microk8s/current/docker.sock ps
CONTAINER ID        IMAGE                         COMMAND                  CREATED             STATUS              PORTS               NAMES
fedf8c9eb0e9        6c0dbeaa8d20                  "/pod_nanny --config…"   6 minutes ago       Up 6 minutes                            k8s_eventer-nanny_heapster-v1.5.2-577898ddbf-m868v_kube-system_0e11103f-8a47-11e8-9183-42010a800002_1

whereas without the additional /:
> docker -H unix://var/snap/microk8s/current/docker.sock ps
Cannot connect to the Docker daemon at unix://var/snap/microk8s/current/docker.sock. Is the docker daemon running?

Additionally one can use export DOCKER_HOST=unix:///var/snap/microk8s/current/docker.sock to avoid the -H argument on each command

In the edge and 1.11/edge channels you will now find the microk8s.docker command. This addition came in with the registry addon (microk8s.enable registry) that start an insecure registry at localhost:32000. Have a look here: https://github.com/ubuntu/microk8s/blob/master/docs/registry.md

Feel free to reopen this issue if you find something missing.


docs(traverse): document soundness hole

Document the remaining (small) soundness hole in Traverse.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#7515  👈 (View in Graphite)
#7514
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Merge activity

Nov 27, 8:18 PM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


Add Closing Bracket on Example

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


[Roadmap][Features] These are features that could be useful for JSONMaestro

I've thought about some features for JSONMaestro a little bit

JSON Serialization/Deserialization
Able to use JSON5
Compliant with ECMA262
Able to parse partial and or faulty strings
logging utility
Convert datatypes (CSV, etc) to JSON
Able to fetch from URL/URI
API tooling
Able to lint JSON/JSONC, possible formatting
Generate random JSON for testing purposes/mocking
Generate a CMakeLists.txt file from a compile_commands.json
Generate .xls/xlsx from JSON
Able to split large JSON files into chunks
JSON Database tooling
Able to 'flatten' nested json
work with .jsonl format
works with jsonp format
numpy arrays
hash generation of json objects
works with tabbed data or TSV
gettext and dict support

How much of this is feasible without making the package too bloated?

So we can make library one package and then apps be seperate, what also probably be more reliable for the end user, as in that case if library doesnt package apps also wont package

Agreed. let me know what you think on these features.

JSON Serialization/Deserialization

Not sure about this one becuase its kindoff alredy implement
also can be considered as plain old data seriasilation

Able to use JSON5

would priritize this as it follows json theme
My main question here is "In House" implementation or use pyjson5?

Compliant with ECMA262

havent read the spec yet so could be useful

Able to parse partial and or faulty strings

would leave this perhaps a feature that user has to bring in via the patches as it could be quite unstable

logging utility

very usefull for cli users

Able to fetch from URL/URI

would leave for later since it will require rework of cli app

API tooling

hostable api?

Able to lint JSON/JSONC, possible formatting

theres prettier, eslint, etc... so is it really necessary?

Generate random JSON for testing purposes/mocking

will be good for users testing in offline environments

Generate a CMakeLists.txt file from a compile_commands.json

can be quite complex, as there can be compile_commands.json per cmake project

Generate .xls/xlsx from JSON

not sure how since its basically propriatery format

Able to split large JSON files into chunks

sure, noone like big github api response dumps (i dont know how much have worked around github api but it can have up to 5mb dumps)

Able to 'flatten' nested json

would leave it for later

work with .jsonl format

quite easy without validation

works with jsonp format

padded or pretty format?

numpy arrays

not sure havent used numpy

hash generation of json objects

good idea, like nested sha-sum file?


Copy tf_attrtype.h

In pre-built TensorFlow packages for Windows systems, include/tensorflow/c/tf_attrtype.h is missing. include/tensorflow/c/c_api.h needs this header file.

I signed it!

@sjamesr we may want to do this for all the other header files you added.

@gunan @sjamesr I have added more header files.

tensorflow/c/tf_attrtype.h
tensorflow/c/tf_datatype.h
tensorflow/c/tf_status.h
tensorflow/c/tf_tensor.h

Is this enough? Is tensorflow/c/c_api_experimental.h needed?

I do not think we need c_api_experimental. That header is meant to be unstable, and should not be dedpended on by end users.


Joining tinted-theming

Hi @ShiromMakkad @kkirss, I'm part of Tinted Theming, an org continuing the development of the base16 work Chris began initially. You can read more about the formation of the organization here: https://github.com/tinted-theming/home/issues/51
base16-jetbrains is currently in the organisation and I'm busy looking at updating it to contain styling for the full jetbrains IDE. Since this is essentially what you've done, would you be interested in moving this repo to the Tinted Theming organization, perhaps moving this repos commits into base16-jetbrains and you two continuing to maintain it?

This seems like a good idea for users. What do I need to do?

Great! I've added you and  @kkirss as members to the organisation in the jetbrains team (so you'll also have maintain access to the other repo). You can transfer this repository to the org (repo Settings -> Transfer).
As for base16-jetbrains vs base16-jetbrains-ide. What are your thoughts on how to go about the two repos? Merge them? Keep separate?

If you want to keep this repo under your name, we could also merge the repos into base16-jetbrains (bring the commits across), while you archive this one.

Yea, most jetbrains themes keep the editor and full ide theme as a single plugin, so I think it's best to move into base16-jetbrains as a single unified repo.
Let's archive this repo and move my commits into the base16-jetbrains repo. Couple questions:

When the commits get moved, will my name and history be preserved? Id like to keep that.
Will I have edit permissions on base16-jetbrains? Id like to be able to make branches and merge my prs.

Yes your name will be preserved on the commits. The “move” of the commits isn’t as much of a “move” as it is copying the commits. I’ll do it tomorrow on a temporary branch and once you’re happy with it I can make the main branch reflect the changes.
You should have maintainer access to base16-jetbains right now, but yes that will continue to be the case after I’ve copied your commits across.

I've pushed a new branch to base16-jetbrains which contains basd16-jetbrains commits with base16-jetbrains-ide commits put on top of it (so the commits follow chronologically). Have a look at the branch (titled new-main temporarily) and let me know if there is anything you want to change. Your commits have you marked as the original author and has me marked as someone who has editted the commit (since I'm the one who was doing the rebasing and resolving the conflicts): https://github.com/tinted-theming/base16-jetbrains/commits/new-main/

Not sure why Github is showing just my email instead of my profile on that commit. It's doing the same on this repo for only the latest commit. Maybe I didn't have my git username set properly? Any way to fix?

I've removed the image so your email isn't on the page. It's something you have to change in your github settings (the email that is put onto the commits). Any commits you've made in the past actually has your email currently attached to it and changing the settings only affects future commits.
Github doesn't show the emails, but if you checkout someone's repo and look at the author of the commits you more often than not see their real email address.

I haven't done it before, but I could try to replace your email with a different value if you want? But note: your email is currently on the commits on this repo at the moment.

I'm ok with my email being attached to every commit, but if you look at the git history of this repo you'll see the issue is here too. It's not linking to my Github profile and name properly. It says<EMAIL_ADDRESS>authored" instead of "ShiromMakkad authored" Looking into this rn.

My guess is that you made that commit from another computer or a computer which wasn't set up properly (maybe new computer?)
Your username is set as your email value and your email is set to<EMAIL_ADDRESS>.

I've adjusted your username and email on that specific commit: https://github.com/tinted-theming/base16-jetbrains/commits/new-main/

Some changes will need to be made to combine the two repos. Also, the Readme will need to be updated with new installation instructions. I can work on that.

Ok great, you can push changes to the new-main branch. We can replace main branch with that once it's good enough for users to look at.

One thing, I'm looking at: https://github.com/tinted-theming/schemes/blob/spec-0.11/base16/tomorrow-night-eighties.yaml and I'm not seeing slugs for any themes. Has that been deprecated? What will be filled in when I put {{scheme-slug}} in a template?

I've just generated the themes with base16-builder-go (the only builder that implements spec 0.11 at the moment) and pushed to new-main. I'll add a github action to run the builder on the repo every week so it's up to date with the latest schemes.

Just adding in case it's of interest to you, we have a tinted-theming channel on Matrix if you prefer DMs https://matrix.to/#/#tinted-theming:matrix.org

You mentioned that the readme needs to be updated and some other changes to combine the repos, which changes are you referring to? Things look quite similar on the new-main branch: https://github.com/tinted-theming/base16-jetbrains/tree/new-main

Since the "installation" instructions are the same on new-main I'm going point main branch there and complete the transition. Any other changes (to the readme or anything else) can happen afterwards.

https://github.com/tinted-theming/base16-jetbrains now reflects the update. I suggest putting your repo in archive mode with a deprecation notice pointing to https://github.com/tinted-theming/base16-jetbrains as the new maintained version. You're a maintainer on that repo, we can discuss things on issues there or on our Matrix channel. I'll close this github issue now.


Support "Processing" condition on SNR

ECOPROJECT-1198

just curious if SNR runs into same CI issues as NHC
/test 4.13-openshift-e2e

/retest

/retest


Allow Client to Delete Streams

If a client subscribes to a stream, the publisher may wish to cancel the stream without publishing any data. A DELETE should close any connected subscribers.

Had some discussion about this. Do you have any input as to which part in the stack should execute the ensure-DELETE pattern?

IIRC, this was a short-circuiting mechanism rather than an assurance. I can't remember the use case any more, though, so feel free to close unless you have one.


How to turn on subtitles

I see there is some code in place to have SubtitleTrack objects. Is there any way to turn these on?

There have two way to do it.
You can delete the "--no-sub-autodecete-file" in your VlcSettings, it will auto select sub file.
Or, you can use the Player.LoadMediaWithOptins(), use those options:
:sub-file=C:\sub.ass

The subtitles I wish to show are embedded in the video file. I wish, also, to turn them on and off while the video is playing.
I'm guessing that is not an option at the moment?

I don't know about it.
Maybe you can check the document of VLC.

I found some other apis about subtitles. It may help you to do it.
I'll add it later.

That would be great. Thank you @higankanshi !
My need for this is not immediate. In fact, I'm hoping to convince my bosses that it is unnecessary at all. But I wanted to get the conversation started.

You can notice that a update have been committed to dev branch.
New subtitle APIs in VlcMediaPlayer:
Property              VlcMediaPlayer.Subtitle
Property              VlcMediaPlayer.SubtitleCount
Property              VlcMediaPlayer.SubtitleDescription
Method                VlcMediaPlayer.SetSubtitleFile

Awesome! I'll check it out when I get a chance.


Add ConvertableKeep()

ConvertableKeep is not checking on keep.

Codecov Report

Merging #11 (d4d66a6) into master (ecd95e6) will increase coverage by 0.17%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
+ Coverage   81.81%   81.99%   +0.17%
==========================================
  Files          12       12
  Lines         407      411       +4
==========================================
+ Hits          333      337       +4
  Misses         43       43
  Partials       31       31

Impacted Files
Coverage Δ

currying.go
92.98% <100.00%> (+0.52%)
:arrow_up:

Continue to review full report at Codecov.

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


Does not work for DayZ servers

I'm currently updating it to a new major version. Apologies if you saw inconsistent changes along the way.

You might want to stick to version 0.0.* until 1.* is finished.


source = "tabbed-right" seems to conflict with annotations in Material

Describe the bug
There's really two bugs here. The first bug is that I've enabled pymdownx.tabbed but the tabs don't appear to be working when I set source="tabbed-right".
Secondly, if I have any annotations using Material -- using the tabbed option breaks them -- e.g.
Screenshots
Before tabs (here, I'm using source="console"):

After, with source = "tabbed-right":

System (please complete the following information):

Markdown Exec version: 1.2.0
Python version: 3.11
OS: Mac OS X

Hello, thanks for the report :slightly_smiling_face:
Could you share some Markdown that replicates the issue? Use 4 backticks to write the 3-backticks code blocks :wink:
````
```python exec="1" source="console"
code block
```
````

Yes! Will share in a moment.

@pawamoy

-   __Model code__

    ```python exec="true" source="tabbed-right"
    print(5 + 5)
    ```

    Defining a beta-bernoulli process in GenJAX.

-   __Inference code__

    ```python
    def sir(prng_key, gen_fn,
            obs, args, n_samples):
        pass
    ```

    This works for **any** generative function - not just the beta-bernoulli model defined on the left.

So this maybe breaks inside the div, so let me try outside too.

@pawamoy Sorry, here's a complete MWE example in Markdown.
# MWE

This is something (1).
{ .annotate }

1.  By design, blah blah blah.

-   __One block of code__

    ```python exec="true" source="tabbed-right"
    print(5 + 5)
    ```

-   __Another block of code__

    ```python exec="true" source="tabbed-right"
    print(5 + 5)
    ```

Unfortunately (or fortunately :grinning:) I cannot replicate the issue.
Given this markup:
## MWE

This is something (1).
{ .annotate }

1.  By design, blah blah blah.

-   __One block of code__

    ```python exec="true" source="tabbed-right"
    print(5 + 5)
    ```

-   __Another block of code__

    ```python exec="true" source="tabbed-right"
    print(5 + 5)
    ```

...and this MkDocs configuration:
markdown_extensions:
- md_in_html
- attr_list
- pymdownx.superfences
- pymdownx.tabbed

...with Material for MkDocs Insiders installed in a Python 3.10 environment, everything renders fine, with both the console and tabbed-right styles for including the source:

Can you check your configuration to make sure all the necessary Markdown extensions are loaded? Also, make sure to add a blank line after/before any
/
 containing Markdown contents.

Closing for now, let me know if you still have the issue!


Slow

With an HTML file of around 600 lines, the program never finishes.

I'm archiving this repo and so closing this.


feat: getcaip2 function to be synchronous

Closes #381

@DhruvJain1122 - can you please lint this and re-commit?

Please add test cases in BitcoinChainAdapter.test.ts and EthereumChainAdapter.test.ts to verify

Constructor with no chainId
Constructor with a valid testnet chainId
Constructor with an invalid chainId (or for a different chain)


Pithos

move from own file to python-packages.nix
upgrade from 1.0.1 to 1.1.1

If it's an application (not a library), it shouldn't be inside python-packages.nix

Very well.

The icons etc are missing it would be cool if you could install those too.

@lowfatcomputing, the only icon is the system tray one. It shows up fine in my tray.. does it not work for you?

@domenkozar, anything else that needs changing before merging?

Fixed

As for my top-row button-icons issue, it looks like someone will need to package gnome-icon-theme-symbolic at some point and add it as a dependency to pithos:
https://github.com/pithos/pithos/issues/137#issuecomment-59370450

@lowfatcomputing, could you try adding gnome3.adwaita-icon-theme to your systemPackages and see if that fixes it?

@obadz, That seemed to have worked. Thanks!

The GTK/Gnome stuff is kind of confusing. I don't know what the right way to tackle this better is..

@lowfatcomputing @obadz Instead of manually call wrapProgram you should use the wrapGAppsHook. I've already fixed this: 93adb22c6faaaa3100882409daf11877ea672885 (also a few other nitpicks, see current file in master branch)
Thanks for your contribution!

@lowfatcomputing Could you please open an issue regarding the missing gnome-icon-theme-symbolic?

@geerds, thanks.


Extra argument in call

There are extra argument 'fullscreen' in call when full screen was true or false

The full screen argument in
 shouldPresent( _ : UIViewController?, from: UIViewController?, fullscreen: Bool )
is required to switch between fullscreen and resized with blur views in the detailViewController wich is not accessible.


Empty lists when fetching for books to download

When I try to download a book from Sanoma, after inserting my credentials, the script doesn't show anything on the list. Is there a way to fix this or is it just the backend that has been patched?

something updated, tbh no idea what, tried a couple of random stuff and it seems to work with the latest update i've uploaded. Try it and let me know if there are still issues

So, I tried but it doesn't work. Is there any other way to get the ID of the book (like with hubscuola, where the script uses te id from the yrl and the session token)? Anyway, thanks for the effort :) these scripts are really useful

I've uploaded another fix, try again

Yes, now it works. Thanks for fixing it apparently only for me


Rails 4.2 compatability

Hi
Is anyone working on this right now?
Thanks,
Ed.

@uu59 How about this? Do you have a plan to update Rails dependency?

I think nothing problem to upgrade to 4.2.
But I have no plan to do that because feature enhancement is important than upgrading Rails for now.
Anyway, going Rails 4.2 is welcome :)

Great, thanks!


Ohio Scientific port

As written in README.md, a wide variety of OSI systems are supported. Screen drivers for 540B color graphics card and VT100 serial system are included. Perhaps I'll craft Hazeltine 1420 and/or DEC VT52 drivers later.

Excellent, thanks! The disk code looks like it was a marathon... I've left a couple of nitpicks, but other than that it looks good to go.
Is this the last 'major' 6502-based microcomputer that needs a port? If you can call the C64 version a port.

Merging; thank you very much. That's a lot of disk images!

Merging; thank you very much. That's a lot of disk images!

Haha, yes. And I even left out a serial system with 5.25" floppies. Also no 400 series with upgraded video to 540B from the 500 series, but with proper modification of the old 440 video board it could still use the ASCII keyboard.

Is this the last 'major' 6502-based microcomputer that needs a port? If you can call the C64 version a port.

Maybe the Rockwell AIM-65 with either the RM65 floppy board (pretty rare in the wild) or the AH5050 IEC interface (there's the 1541 again :smile:).


how to add subSection ?

how to add subSection ?

I'm sorry it's taken me so long to get back to you.
Could you explain what you mean by "subSection?" I'm asking because it's not really an official term in the EPUB specification, so I can better help you if I know exactly what you're asking for.
For example, this library has the ability to add "sections," which are saved as separate files in the resulting EPUB and added to the table of contents.
Thanks!

like this psiegman/epublib
 // Add Chapter 2
      TOCReference chapter2 = book.addSection( "Second Chapter",
        getResource("/book1/chapter2.html", "chapter2.html") );
// Add Chapter2, Section 1
      book.addSection(chapter2, "Chapter 2, section 1",
        getResource("/book1/chapter2_1.html", "chapter2_1.html"));

Closed by #58


Cannot make y-axis not starting at 0

Version
4.9.0
Steps to reproduce
Changing chart config to
yAxis: {
type: "value",
axisLabel: {
fontSize: 14,
fontFamily: '"Poppins", sans-serif',
},
axisLine: {
onZero: 0,
onZeroAxisIndex:0,
lineStyle: {
color: "#e9e9e9",
},
},
},
What is expected?
Chart y-axis not starting at 0
What is actually happening?
Chart y-axis still starting at 0

You can set yAxis.scale to true

Well, that was quick. thanks for pointing that out, i guess i had wrong settings before.


request.path doesn't include request.script_root when running under a subdirectory

I have an application which can be configured to run under a subdomain/domain or a subdirectory. This is controlled by setting APPLICATION_ROOT to a value other than /.
Some of my decorators use request.path to determine where to redirect the user after logging in. However, when running under a subdirectory, request.path does not include request.script_root and the user could get redirected somewhere they shouldn't. Using request.url creates full URLs when we should only need relative ones.
Based on docs it is intentional that request.path doesn't include request.script_root.
Ultimately because my application has a plugin interface and I don't want to deviate too much from Flask documentation, what is the optimal way to get this behavior to be consistent between deployment strategies?

Using url_for() doesn't seem optimal because certain target endpoints could require arguments to be passed in which might not work without parsing request.path anyway.
Combining request.script_root and request.path in every usage seems like an easy way to introduce bugs
Creating a BaseRequest subclass seems like the smartest solution but should there be a new request property (e.g. request.script_path)? Or should I replace the existing path and full_path?

The PR I have in preparation to fix this issue uses a BaseRequest subclass but I don't feel great about it:
https://github.com/CTFd/CTFd/pull/802/files#diff-1dec134679d6639679743763954061d5R28

I don't understand the bug you're reporting. Please provide a concrete, minimal example of the issue you're having when reporting a bug.

from flask import Flask, request
from werkzeug.serving import run_simple
from werkzeug.wsgi import DispatcherMiddleware

app = Flask(__name__)

@app.route('/hi')
def hello_world():
    return 'Hello, World!'

@app.before_request
def show_path():
    print request.script_root, request.path

app.config['APPLICATION_ROOT'] = '/test'

if __name__ == '__main__':
    application = DispatcherMiddleware(Flask('dummy_app'), {
        app.config['APPLICATION_ROOT']: app,
    })
    run_simple('localhost', 4000, application, use_reloader=True)

Here's a diluted test case based off of the Flask quickstart and here is the output.
❯ python example.py
 * Running on http://localhost:4000/ (Press CTRL+C to quit)
 * Restarting with stat
/test /hi
<IP_ADDRESS> - - [15/Dec/2018 18:41:55] "GET /test/hi HTTP/1.1" 200 -

Using APPLICATION_ROOT this server is running in a subdirectory but request.path is pointing to /hi when the true full path is /test/hi.
I understand that the docs are clear in saying that request.path doesn't return the full URL path but what is the best way to grab it that doesn't involve doing request.script_root + request.path everywhere? Would it just be to subclass the Request object and reprepand request.path with request.script_root?
For example:
class CorrectSubDirectoryRequest(Request):
    @cached_property
    def script_path(self):
        return self.script_root + self.path

I don't understand why you need to look at the script root. The path is the path within your application, which routing is performed on. The script root is outside your application, but is handled by url_for.

How would you restructure this decorator?
https://github.com/CTFd/CTFd/blob/master/CTFd/utils/decorators/__init__.py#L70
If I redirect purely based off request.path a user could end up redirected
to a different application.
If I use url_for I would need to reconstruct the necessary arguments to
each endpoint right? Is there a better way to do this than just appending
script root and path? At that point I might as well overload path with a
subclass.
On Sat, Dec 15, 2018 at 7:36 PM David Lord<EMAIL_ADDRESS>wrote:

I don't understand why you need to look at the script root. The path is
the path within your application, which routing is performed on. The script
root is outside your application, but is handled by url_for.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_pallets_flask_issues_3032-23issuecomment-2D447608483&d=DwMCaQ&c=slrrB7dE8n7gBJbeO0g-IQ&r=GrSEXVo9nswNldgE1q1G0g&m=i7Z6JAW7nC3cm1rAh3M1ShAsoSaNmvgCoIRzuuAM51Y&s=TX-7_GKdIvLxkIOVnwyhq9pzF-iGx1bxcOMo8vISI74&e=,
or mute the thread
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AAKJvZEtON1aW643noml5cIDXwdySNCXks5u5ZWAgaJpZM4ZU5fv&d=DwMCaQ&c=slrrB7dE8n7gBJbeO0g-IQ&r=GrSEXVo9nswNldgE1q1G0g&m=i7Z6JAW7nC3cm1rAh3M1ShAsoSaNmvgCoIRzuuAM51Y&s=QmqR7K2HdQWKr3rEa0AnzVKKLFCcjyqJt_2JBVzAGt4&e=
.

request.url instead of request.path


WARNING: erroneous pipeline: no element "kvssink"

I have cross compiled amazon kinesis cpp sdk for aarch64 system. the gstreamer pipeline failed to execute with some warnings.
gstreamer pipeline:
gst-launch-1.0 rtspsrc location="rtsp_url" short-header=TRUE ! rtph264depay ! video/x-h264, format=avc,alignment=au ! kvssink stream-name="my_stream_name" storage-size=512 access-key="my_access_key" secret-key="my_secrete_key" aws-region="ap-south-1"

error logs:
(gst-plugin-scanner:2467): GStreamer-WARNING **: 15:05:05.692: Failed to load plugin '/root/amazon-kinesis-video-streams-producer-sdk-cpp/build/libgstkvssink.so': /lib/libstdc++.so.6: version `GLIBCXX_3.4.26' not found (required by /root/amazon-kinesis-video-streams-producer-sdk-cpp/build/libgstkvssink.so)

(gst-plugin-scanner:2467): GStreamer-WARNING **: 15:05:05.714: Failed to load plugin '/root/amazon-kinesis-video-streams-producer-sdk-cpp/build/libKinesisVideoProducer.so': /lib/libstdc++.so.6: version `GLIBCXX_3.4.26' not found (required by /root/amazon-kinesis-video-streams-producer-sdk-cpp/build/libKinesisVideoProducer.so)

(gst-plugin-scanner:2469): GStreamer-WARNING **: 15:05:05.811: Failed to load plugin '/root/amazon-kinesis-video-streams-producer-sdk-cpp/build/libKinesisVideoProducer.so': /lib/libstdc++.so.6: version `GLIBCXX_3.4.26' not found (required by /root/amazon-kinesis-video-streams-producer-sdk-cpp/build/libKinesisVideoProducer.so)

(gst-plugin-scanner:2470): GStreamer-WARNING **: 15:05:05.934: Failed to load plugin '/usr/lib/gstreamer-1.0/libcurl.so': /lib/libc.so.6: version `GLIBC_2.28' not found (required by /usr/lib/gstreamer-1.0/libcurl.so)

(gst-plugin-scanner:2470): GStreamer-WARNING **: 15:05:05.943: Failed to load plugin '/usr/lib/gstreamer-1.0/libgstkvssink.so': /lib/libstdc++.so.6: version `GLIBCXX_3.4.26' not found (required by /usr/lib/gstreamer-1.0/libgstkvssink.so)

(gst-plugin-scanner:2470): GStreamer-WARNING **: 15:05:05.947: Failed to load plugin '/usr/lib/gstreamer-1.0/libKinesisVideoProducer.so': /lib/libstdc++.so.6: version `GLIBCXX_3.4.26' not found (required by /usr/lib/gstreamer-1.0/libKinesisVideoProducer.so)
WARNING: erroneous pipeline: no element "kvssink"

i set the below environment variable path as per my system.
export GST_PLUGIN_PATH=pwd/build
export LD_LIBRARY_PATH=pwd/open-source/local/lib
unable to understand what is going wrong?
any help would appreciated. let me know if need more info.

@gorleabhinav ,
It is possible libstdc++6 is not available on your machine? Can you try installing libstdc++6 and see if that helps?

Closing due to no response. Feel free to reopen if you have any updates.

the problem with the libstdc++6 library version, previously the system has libstdc++6.0.24 but i updated it with libstdc++6.0.27. now issue get resolved. Thanks.