Update Sphinx with C++11 literals fix and add fmt::literals API docs

Sphinx 1.3.1 can't parse user-defined literals. This updates the doc build script to the (currently) latest state on the Sphinx master branch which has the fix.
As for the fmt::literals API docs, the examples do most of the explaining. They mirror the fmt::format and fmt::args examples right above them. The description text just points to the equivalent function. I really couldn't think of anything to write in the description that would be clearer than the example.

Thanks for another great PR. Nice that Sphinx has accepted your implementation of UDL support so quickly. I was going to explain how to use a specific version of Sphinx, but I see you've figured out this already.

It's a lucky coincidence that Sphinx's C++ domain is being actively developed right now, so the PR got merged right away.
Heads up: Looks like the Sphinx version on Travis wasn't updated, so the auto-uploaded docs look a bit weird.

Yeah, I'm looking into it right now.

Looks like pip does something crazy here: it installs version 1.3.1 of sphinx when installing dependencies for version 1.4a0.dev-20151014:
Downloading/unpacking sphinx>=1.3 (from sphinx-rtd-theme>=0.1,<2.0->Sphinx==1.4a0.dev-20151014)
  Downloading Sphinx-1.3.1.tar.gz (3.5Mb): 3.5Mb downloaded

(from https://travis-ci.org/cppformat/cppformat/jobs/85336329)
I'll try upgrading pip on Travis because the version used there is pretty old.

From the log you linked:

Installing collected packages: six, Jinja2, Pygments, docutils, snowballstemmer, babel, alabaster, sphinx-rtd-theme, Sphinx, MarkupSafe, pytz, sphinx

It installs both uppercase Sphinx and lowercase sphinx. Well, that pip v1.1 on Travis is a few years out of date.
Sorry about all this trouble. Between the macro stringification and this, It seems like every commit I make shakes out some crazy bug somewhere.

It installs both uppercase Sphinx and lowercase sphinx.

Good catch. This could be the problem.

Sorry about all this trouble. Between the macro stringification and this, It seems like every commit I make shakes out some crazy bug somewhere.

No problems, it's good to shake out the obscure bugs every now and then =. The documentation build script did have some issues which I'm fixing in the process.

Are you sure that this check_call is actually executed?
if LooseVersion(pip.__version__) < LooseVersion('1.5.4'):
    check_call(['pip', 'install', '--upgrade', 'pip'])

I don't see it in the log.
That pip.__version__ does not correspond to the pip in virtualenv, but to the interpreter currently running the script. Virtualenv installs its own internal pip version (1.1 on Travis). From the log, the system pip should be
Get:2 http://us.archive.ubuntu.com/ubuntu/ precise/universe python-pip all 1.0-1build1 [95.1 kB]

even older 1.0, so technically it should still be lower than 1.5.4 but maybe the pip.__version__ string is wonky?

You are right, it isn't. The pip.__version__ at that point is actually 6.0.7. I wonder where it comes from?

Anyway, pip is up-to-date now. Now sphinx-build breaks on sphinx-rtd-theme which is not used at all =):
pkg_resources.DistributionNotFound: sphinx-rtd-theme>=0.1,<2.0

Reported to https://github.com/sphinx-doc/sphinx/issues/2086.

The documentation build has been fixed. Two minor issues though:

fmt::literals::operator""_format doesn't have an anchor.
fmt::format() link is broken.

Missing anchors for _format and _a
As far as I can tell, it's an issue in Breathe related to the "Duplicate ID" warnings for overloaded functions. If I comment out the wchar_t overload of _format in format.h and request .. doxygenfunction:: operator""_format without parameters, the anchor shows up.
This issue affects UDLs because they only have the id_v2 in Sphinx, unlike older functions which also have id_v1. I'm not sure why this is, but I've tested it by adding a mock id_v1 to UDLs in the Sphinx source and it restores the anchor. However, I'm pretty sure the error is on Breathe's side since the number of ids shouldn't really matter.
I couldn't figure out exactly what's causing the issue in Breathe, but I'm fairly sure it's related to the function overload resolution. As I said, removing the wchar_t _format overload resolves the issue. I guess you're familiar with Breathe so maybe it's something obvious I'm missing. If not, let me know and I'll submit a proper issue report with example code to Breathe.
Broken fmt::format() link
It's pointing to the wrong functions because Sphinx can't handle overloads. Maybe the simplest solution is to remove the link entirely since fmt::format is actually right above it already. Same goes for _a and fmt::arg.

Right, I will look into the issue with UDLs in Breathe when time permits. For the second it's probably make sense to open an issue in Sphinx if they don't have one already. Anyway, I've opened #213 so that this is not forgotten as this PR has been closed.

I thought I was close before so I had another look at Breathe and I found the issue. I made a quick fix and it works, but it doesn't follow the factory pattern like the rest of the code so it looks like a bit of a hack. I'm not very good with factories. I'm not sure what would be the proper solution within that framework, so I'll leave it to you for a proper fix. The comments in the commit point to the cause of the issue: it's just that the options passed to create_factory_creator are never actually used.
It fixes the UDL issue and also resolves all those SEVERE: Duplicate ID warnings in the doc build log.

I can't say that I fully understand all the factory business in Breathe myself, so I opened a PR. Hopefully @michaeljones can guide us there.

And thanks for taking time to debug this and providing the fix, @dean0x7d.


Re-Ordering the Sections on the Getting Started Guide

This will close #409. I've updated the guide to lead with the beginner tracks.

Not sure if this is required, but I believe in a previous livestream you mentioned that you preferred the header text ("Code! Programming with p5.js") to link to the track page.

Ah, yes, I remember noticing this! I just tried to add it but I get an error with any of the following . . . any ideas? @fdoflorenzano do you happen to know what might be preventing me from making the header a link?
### [Code! Programming with p5.js](/tracks/code-programming-with-p5-js)

### Code! Programming with p5.js

Code! Programming with p5.js

The kebabCase function is being called with the contents of the header tag.
(This function is responsible for creating ids, i.e. code-programming-with-p5js which is used in the table of contents for linking to a section, like this)

The function is having trouble preparing an id for this section (code-programming with-p5js) because, it expects a string as its argument. However, it recieves the contents of the header, which happens to be the  tag.
If we remove the kebabCase call (line 65 above), suprisingly everything works fine. This is because, for Level 3 headings, the Table of Contents does not create an entry, so it technically does not require an ID field.

However, I do not know if this is an actual solution to the issue.

Hmmm, maybe I'll revisit this later, I also don't like how it looks with the color and underline. Perhaps it's not critical. @klinegareth if you want to take a quick look at this to make sure it's all good then merge go ahead! I can open a new separate issue to think about linking some of the titles (I had the same thought in a few other places on the site).


⚠ GitLab Server has degraded performance

In d4da08d, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 10684 ms

Resolved: GitLab Server performance has improved in a7e987e.


Generated code is wrong if Ok or Err are redefined

#[macro_use]
extern crate serde_derive;

struct Ok;
struct Err;

#[derive(Deserialize)]
struct S {
    x: ()
}

error: expected function, found `Ok`
 --> src/main.rs:7:10
  |
7 | #[derive(Deserialize)]
  |          ^^^^^^^^^^^
  |
note: defined here
 --> src/main.rs:4:1
  |
4 | struct Ok;
  | ^^^^^^^^^^

error: expected function, found `Err`
 --> src/main.rs:7:10
  |
7 | #[derive(Deserialize)]
  |          ^^^^^^^^^^^
  |
note: defined here
 --> src/main.rs:5:1
  |
5 | struct Err;
  | ^^^^^^^^^^^

This happens if someone does use hyper::* which includes a reexport of hyper::status::StatusCode::Ok :crying_cat_face:


read from zipstream with password

Hi,i want to read file from ZipInputstream.
This zip may be password protected or not,
and the password can also be any one of the known passwords.
I saw all the construction methods of ZipInputStream,but there is no suitable one.
What should I do?
Is there a method similar to ZipFile.isEncrypted and setPassword.
Thank you.

Below is the equivalent of ZipFile.isEncrypted:
ZipInputStream zipInputStream = new ZipInputStream(...);
LocalFileHeader localFileHeader = zipInputStream.getNextEntry();
localFileHeader.isEncrypted();

But unfortunately, once the password is set via ZipInputStreams constructor, it cannot be changed again. I will add a feature to also set the password later via a ZipInputStream.setPassword(char[])` method. I will include this in the next release.

Feature added in v2.8.0 released today.


error with telegram

seems there is an issue with the telegram class ...
Traceback (most recent call last):
File "./hah.py", line 4, in
import httpx, json, telegram_send, argparse
File "/usr/local/lib/python3.8/site-packages/telegram_send/init.py", line 2, in
from .telegram_send import configure, send
File "/usr/local/lib/python3.8/site-packages/telegram_send/telegram_send.py", line 33, in
from telegram.constants import MAX_MESSAGE_LENGTH
ImportError: cannot import name 'MAX_MESSAGE_LENGTH' from 'telegram.constants' (/usr/local/lib/python3.8/site-packages/telegram/constants.py)

Related to https://github.com/rahiel/telegram-send/issues/115, I'll configure proper pyenv and requirements.txt instead of relying on "latest"

Fixed :)


Unable to view PDF From Amazon Service Url in AdvanceWebview.

Thanks!
Are you able to view other PDFs (from other sites)?
Can you share a URL to a PDF from Amazon (that does not work)?

@ocram  is this issue fixed?


Services now on Make.

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (b061aa0) 67.21% compared to head (063d252) 67.23%.

Files
Patch %
Lines

common/util/db.go
71.42%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #83      +/-   ##
==========================================
+ Coverage   67.21%   67.23%   +0.02%
==========================================
  Files          68       68
  Lines        2452     2454       +2
==========================================
+ Hits         1648     1650       +2
  Misses        763      763
  Partials       41       41

Flag
Coverage Δ

unittests
67.23% <87.50%> (+0.02%)
:arrow_up:

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

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


[Bug]: SQL Function does not handle multi-line statements

Terraform CLI Version
1.9.4
Terraform Provider Version
0.95.0
Terraform Configuration
resource "snowflake_function" "TEST_PARSER_SQL_FUNC" {
  database = local.databases[var.DATABASE]
  schema = snowflake_schema.TEST_PARSER.name
  name = "SQL_FUNC"

  arguments {
    name = "arg1"
    type = "number"
  }
  comment             = "Example for SQL language"
  return_type         = "VARCHAR(30)"
  null_input_behavior = "CALLED ON NULL INPUT"
  return_behavior     = "VOLATILE"
  statement           = <<-EOT
    select
      CONCAT('Result: ', arg1 + 1)
 EOT
}

Category
category:resource
Object type(s)
resource:function
Expected Behavior
The query should be able to handle carriage returns and should escape single quotes using a second single quote.
CREATE FUNCTION "DB_DPT_DEV_STG"."TEST_PARSER"."SQL_FUNC" (arg1 NUMBER) RETURNS VARCHAR VOLATILE COMMENT = 'Example for SQL language' AS 'select CONCAT(\'Result: \', arg1 + 1)'

Actual Behavior
request injects newlines
CREATE FUNCTION "DB_DPT_DEV_STG"."TEST_PARSER"."SQL_FUNC" (arg1 NUMBER) RETURNS VARCHAR VOLATILE COMMENT = 'Example for SQL language' AS 'select \n  CONCAT(\'Result: \', arg1 + 1)'

Compilation of SQL UDF failed: SQL compilation error: syntax error line 1 at position 1 unexpected 'select'. syntax error line 2 at position 9 unexpected ''Result: ''. syntax error line 2 at position 29 unexpected ')'.

Steps to Reproduce

Copy configuration
Replace database and schema
Terraform apply

How much impact is this issue causing?
High
Logs
No response
Additional Information
No response
Would you like to implement a fix?

[ ] Yeah, I'll take it 😎

Nevermind...pitcher's mistake. Turns out the semicolon at the end of the statement was the culprit!!


Update available banner stays though latest version installed already

Bug description:
Ferdi keeps showing the update available banner even though I am already on the latest version (excl. beta and nightly).
Steps to reproduce:

Update Ferdi to 5.6.2
See update banner still there
Click update & restart
See update banner still there

Expected behavior
No update banner
Screenshots

Environment:

Operating System: Mac Monterey (12.0.1)

Ferdi Version: 5.6.2

Server: No Account

Debug information:

{"host":{"platform":"darwin","release":"21.1.0","screens":[{"id":459106329,"bounds":{"x":0,"y":0,"width":3008,"height":1692},"workArea":{"x":0,"y":25,"width":3008,"height":1603},"accelerometerSupport":"unknown","monochrome":false,"colorDepth":24,"colorSpace":"{primaries:BT709, transfer:IEC61966_2_1, matrix:RGB, range:FULL}","depthPerComponent":8,"size":{"width":3008,"height":1692},"displayFrequency":59,"workAreaSize":{"width":3008,"height":1603},"scaleFactor":2,"rotation":0,"internal":false,"touchSupport":"unknown"},{"id":69734208,"bounds":{"x":-1440,"y":792,"width":1440,"height":900},"workArea":{"x":-1440,"y":817,"width":1440,"height":875},"accelerometerSupport":"unknown","monochrome":false,"colorDepth":30,"colorSpace":"{primaries:BT709, transfer:IEC61966_2_1_HDR, matrix:RGB, range:FULL}","depthPerComponent":10,"size":{"width":1440,"height":900},"displayFrequency":60,"workAreaSize":{"width":1440,"height":875},"scaleFactor":2,"rotation":0,"internal":true,"touchSupport":"unknown"}]},"ferdi":{"version":"5.6.2","electron":"13.3.0","installedRecipes":[{"id":"messenger","version":"1.1.0"},{"id":"msteams","version":"1.3.2"},{"id":"telegram","version":"3.2.1"},{"id":"whatsapp","version":"3.3.2"},{}],"devRecipes":[],"services":[{"id":"e984efe5-4a97-41ab-a514-f93e056de579","recipe":"whatsapp","isAttached":true,"isActive":true,"isEnabled":true,"isHibernating":false,"hasCrashed":false,"isDarkModeEnabled":true},{"id":"78eeca24-8e22-4d0d-9f3e-82540a1d6141","recipe":"telegram","isAttached":true,"isActive":false,"isEnabled":true,"isHibernating":false,"hasCrashed":false,"isDarkModeEnabled":true}],"messages":[{"type":"error","info":["error","Warning: Failed prop type: Invalid prop `actions.app` of type `Object` supplied to `_class2`, expected instance of `_class`.\n    in _class2 (created by inject-_class2-with-stores-actions)\n    in inject-_class2-with-stores-actions (created by RouterContext)\n    in RouterContext (created by Router)\n    in Router (created by _class)\n    in _class (created by inject-_class-with-stores-actions)\n    in inject-_class-with-stores-actions\n    in IntlProvider (created by _class)\n    in _class (created by inject-_class-with-stores)\n    in inject-_class-with-stores\n    in Provider"]},{"type":"error","info":["error","Warning: Failed prop type: Invalid prop `actions.user` of type `Object` supplied to `_class2`, expected instance of `_class`.\n    in _class2 (created by inject-_class2-with-stores-actions)\n    in inject-_class2-with-stores-actions (created by RouterContext)\n    in div (created by _class2)\n    in div (created by _class2)\n    in _class2 (created by _class2)\n    in ThemeProvider (created by _class2)\n    in _class2 (created by inject-_class2-with-stores-actions)\n    in inject-_class2-with-stores-actions (created by RouterContext)\n    in RouterContext (created by Router)\n    in Router (created by _class)\n    in _class (created by inject-_class-with-stores-actions)\n    in inject-_class-with-stores-actions\n    in IntlProvider (created by _class)\n    in _class (created by inject-_class-with-stores)\n    in inject-_class-with-stores\n    in Provider"]},{"type":"error","request":{"result":null,"wasExecuted":true,"method":"default"},"error":{},"response":{},"server":"https://api.getferdi.com"},{"type":"error","info":["error","Warning: Can't perform a React state update on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in %s.%s","the componentWillUnmount method","\n    in WithTheme(Jss(_class2)) (created by _class2)\n    in _class2 (created by WithTheme(_class2))\n    in WithTheme(_class2) (created by Jss(WithTheme(_class2)))\n    in Jss(WithTheme(_class2)) (created by _class2)"]},{"type":"error","info":["error","Warning: Can't perform a React state update on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in %s.%s","the componentWillUnmount method","\n    in WithTheme(_class2) (created by Jss(WithTheme(_class2)))\n    in Jss(WithTheme(_class2)) (created by _class2)"]},{"type":"error","request":{"result":null,"wasExecuted":true,"method":"getUserWorkspaces"},"error":{},"response":{},"server":"https://api.getferdi.com"},{"type":"error","info":["error","Warning: Failed prop type: Invalid prop `actions.service` of type `Object` supplied to `_class2`, expected instance of `_class`.\n    in _class2 (created by inject-_class2-with-stores-actions)\n    in inject-_class2-with-stores-actions (created by RouterContext)\n    in RouterContext (created by Router)\n    in Router (created by _class)\n    in _class (created by inject-_class-with-stores-actions)\n    in inject-_class-with-stores-actions\n    in IntlProvider (created by _class)\n    in _class (created by inject-_class-with-stores)\n    in inject-_class-with-stores\n    in Provider"]},{"type":"error","info":["error","Warning: Failed prop type: Invalid prop `actions.settings` of type `Object` supplied to `_class2`, expected instance of `_class`.\n    in _class2 (created by inject-_class2-with-stores-actions)\n    in inject-_class2-with-stores-actions (created by _class2)\n    in div (created by _class2)\n    in div (created by _class2)\n    in _class2 (created by Jss(_class2))\n    in Jss(_class2) (created by _class2)\n    in _class2 (created by Jss(_class2))\n    in Jss(_class2) (created by _class2)\n    in ThemeProvider (created by _class2)\n    in _class2 (created by inject-_class2-with-stores-actions)\n    in inject-_class2-with-stores-actions (created by RouterContext)\n    in RouterContext (created by Router)\n    in Router (created by _class)\n    in _class (created by inject-_class-with-stores-actions)\n    in inject-_class-with-stores-actions\n    in IntlProvider (created by _class)\n    in _class (created by inject-_class-with-stores)\n    in inject-_class-with-stores\n    in Provider"]},{"type":"error","request":{"result":null,"wasExecuted":true,"method":"features"},"error":{},"response":{},"server":"You are using Ferdi without a server"},{"type":"error","request":{"result":null,"wasExecuted":true,"method":"latest"},"error":{},"response":{},"server":"You are using Ferdi without a server"},{"type":"error","request":{"result":null,"wasExecuted":true,"method":"features"},"error":{},"response":{},"server":"You are using Ferdi without a server"},{"type":"error","request":{"result":null,"wasExecuted":true,"method":"latest"},"error":{},"response":{},"server":"You are using Ferdi without a server"}],"workspaces":[],"windowSettings":{"width":1506,"height":961,"x":141,"y":252,"displayBounds":{"x":0,"y":0,"width":3008,"height":1692},"isMaximized":false,"isFullScreen":false},"settings":{"autoLaunchInBackground":false,"runInBackground":true,"reloadAfterResume":true,"enableSystemTray":false,"startMinimized":true,"confirmOnQuit":false,"minimizeToSystemTray":false,"closeToSystemTray":false,"privateNotifications":true,"clipboardNotifications":true,"notifyTaskBarOnMessage":true,"showDisabledServices":true,"showMessageBadgeWhenMuted":true,"showDragArea":false,"enableSpellchecking":true,"spellcheckerLanguage":"en-us","darkMode":true,"locale":"en-US","fallbackLocale":"en-US","beta":false,"isAppMuted":false,"enableGPUAcceleration":true,"server":"You are using Ferdi without a server","predefinedTodoServer":"https://app.franztodos.com","autohideMenuBar":false,"lockingFeatureEnabled":false,"locked":false,"lockedPassword":"******","useTouchIdToUnlock":true,"scheduledDNDEnabled":false,"scheduledDNDStart":"17:00","scheduledDNDEnd":"09:00","hibernateOnStartup":true,"hibernationStrategy":300,"wakeUpStrategy":"300","inactivityLock":0,"automaticUpdates":true,"showServiceNavigationBar":false,"universalDarkMode":true,"userAgentPref":"","adaptableDarkMode":true,"accentColor":"#7367f0","serviceRibbonWidth":"68","iconSize":"20","sentry":false,"nightly":false,"navigationBarBehaviour":"never","searchEngine":"google","useVerticalStyle":false,"alwaysShowWorkspaces":false,"liftSingleInstanceLock":false,"serviceLimit":5,"hibernate":false,"todoServer":"isUsingCustomTodoService","customTodoServer":""},"features":{"isServiceProxyEnabled":true,"isWorkspaceEnabled":true,"isTodosEnabled":true,"isSettingsWSEnabled":false},"user":"82c1cf9d-ab58-4da2-b55e-aaa41d2142d8"}}

Happened for me, too.
Same issue described in #329.
Manually downloading and installing is my current work-around for updates.

@j7e yeah, I just realised I totally looked over the release of 5.6.3 and was convinced 5.6.2 was the latest stable release 🙈. Manual download did the trick indeed!


az graph query ingores underlying API error and returns incorrect results

The CLI command az graph query -q $query (in resource-graph extension) ignores the underlying error returned by the REST API and returns the incorrect data. While the API should not return "bad" data, the Azure CLI also should catch the error and display it. Instead, it simply displays the bad data.
The Azure CLI uses the rest API endpoint:
https://management.azure.com/providers/Microsoft.ResourceGraph/resources?api-version=2019-04-01
and includes the subscriptions and query in the body of the call. See execute_query() in https://github.com/Azure/azure-cli-extensions/blob/master/src/resource-graph/azext_resourcegraph/custom.py.
Extension name (the extension in question)
resource-graph
Description of issue (in as much detail as possible)
This issue came about due to a complicated query which lists all VMs and their associated NSGs (quite useful to see at a glance). The query works fine in the Azure ARM portal, but fails when used via the REST API. But when tested with the Azure CLI, it mysteriously returned results without any error. On further investigation, the results were incorrect, so it appears that the error is being ignored by the CLI extension.
To reproduce this issue, use the following within PowerShell (to make it easy to pass the large $query to AZ CLI):
$query = 'Resources
| where type =~ "microsoft.network/networkinterfaces"
| mv-expand ipConfigurations = properties.ipConfigurations
| extend ipCount = array_length(properties.ipConfigurations)
| extend privateIPType = tostring(ipConfigurations.properties["privateIPAllocationMethod"])
| extend privateIP = tostring(ipConfigurations.properties["privateIPAddress"])
| extend subnetId = tostring(ipConfigurations.properties.subnet["id"])
| extend publicIPid = tostring(ipConfigurations.properties["publicIPAddress"].id)
| extend nicId = tostring(id)
| join kind=leftouter (Resources
	| where type contains "publicIPAddresses" and isnotempty(properties.ipAddress)
	| extend publicIP = tostring(properties.ipAddress),
		publicIPid = tostring(id)) on publicIPid
| join kind=leftouter (Resources | where type == "microsoft.network/networksecuritygroups"
	| mv-expand nics = properties.networkInterfaces
	| extend nicId = tostring (nics.id),
		nicNSG = name,
		nicNSGgroup = resourceGroup  ) on nicId
| join kind=leftouter (Resources
	| where type == "microsoft.compute/virtualmachines" and isnotempty(properties.networkProfile.networkInterfaces)
	| extend vmName = name
	| extend vmSize = tostring(properties.hardwareProfile.vmSize)
	| extend osType = tostring(properties.storageProfile.osDisk.osType)
	| mv-expand nics = properties.networkProfile.networkInterfaces
	| extend nicId = tostring (nics.id) ) on nicId
| join kind=leftouter (Resources | where type == "microsoft.network/networksecuritygroups"
	| mv-expand subnets = properties.subnets
	| extend subnetId = tostring(subnets.id),
		vnetName = split(tostring(subnets.id),"/")[8],
		subnetName = split(tostring(subnets.id),"/")[10],
		subnetNSG = name,
		subnetNSGgroup = resourceGroup
	) on subnetId
| project subscriptionId, nicName=name, resourceGroup, vmName, vmSize, osType, vnetName, subnetName, nicNSG, subnetNSG, location, ipCount, privateIPType, privateIP, publicIP, tags, subnetId, nicId'

# The output does NOT match the prject statement and no error was displayed:
PS C:\> $r = az graph query -q $query
PS C:\> $r1[0]

extendedLocation :
id               : /subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/DataCatalogs-EastUS/providers/Microsoft.DataCatalog/catalogs/MyDC
identity         :
kind             :
location         : eastus
managedBy        :
name             : MyDC
plan             :
properties       : @{admins=System.Object[]; enableAutomaticBillingUnitAdjustment=False; provisioningState=Succeeded; sku=Standard; users=System.Object[]}
resourceGroup    : datacatalogs-eastus
sku              :
subscriptionId   : 00000000-0000-0000-1111-000000000000
tags             : @{CreatedBy=Joe; CreatedTime=11/21/2020 01:28:24}
tenantId         : 00000000-0000-0000-0000-000000000000
type             : microsoft.datacatalog/catalogs
zones            :

# Just to show that the same $query fails in PowerShell Search-AzGraph which uses the same API version:

PS C:\> Search-AzGraph -Query $query
Search-AzGraph : {
  "error": {
    "code": "BadRequest",
    "message": "Please provide below info when asking for support: timestamp = 2020-12-28T14:05:36.2981641Z, correlationId = c20308c2-a874-4fe1-a063-8a4f963a2378.",
    "details": [
      {
        "code": "InvalidQuery",
        "message": "Query is invalid. Please refer to the documentation for the Azure Resource Graph service and fix the error before retrying."
      },
      {
        "code": "TooManyMvExpandColumns",
        "message": "The query invokes mvexpand on 4 columns, and has exceeded the limit of 2. Please consider simplifying the query logic."
      },
      {
        "code": "TooManyJoins",
        "message": "The query uses 4 join operators, and has exceeded the limit of 3. Please consider simplfying the query logic."
      }
    ]
  }
}
At line:1 char:1
+ Search-AzGraph -Query $query
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : CloseError: (:) [Search-AzGraph], ErrorResponseException
    + FullyQualifiedErrorId : BadRequest,Microsoft.Azure.Commands.ResourceGraph.Cmdlets.SearchAzureRmGraph

PS C:\>

Version
# Version of AZ tool on Windows 10 Desktop:
PS C:\> az version
{
  "azure-cli": "2.15.1",
  "azure-cli-core": "2.15.1",
  "azure-cli-telemetry": "1.0.6",
  "extensions": {
    "resource-graph": "1.1.0"
  }
}

Additional Notes
Note that the query itself works perfectly well in the Azure ARM portal but fails when used via the API.
A separate bug https://github.com/Azure/azure-powershell/issues/13845 has been opened for this issue to address the separate  REST API quota issue. But this bug does not resolve the fact that the Azure CLI ignored the error.

resource-graph

As captured in another thread we have more limits in the ARG CLI. Please provide feedback in our uservoice channel so that we can prioritize most important features @ https://feedback.azure.com/forums/915958-azure-governance?category_id=345061

Since @chiragg4u addressed the throttling/limits aspect of the question, let me close this by addressing the incorrect results issue you're seeing when you use Azure CLI within PowerShell.
Background: From Azure CLI "Issues with PS" page, we see that "In order for a symbol to be received by Azure CLI, you will have to take both PowerShell and Command Prompt's parsing into consideration".
Incorrect Results Explanation: When you run az graph query -q $query - the CLI reads only the first line Resources and sends that to our service. This is why you see an incorrect result returned from your request to Azure Resource Graph when using the Azure CLI within PowerShell.
To fix this -- you can run the snippet below and you will retrieve the correct result.
$query | az graph query -q "@-"


Revert "CLOUD-45450 Cleanup CloudContext"

Reverts sequenceiq/cloudbreak#1001

This is a wrong direction that every method is abused with CloudStack

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1890/


v0.3.3 is incompatible with sphinx-rtd-theme 1.0.0

v0.3.3 is incompatible with sphinx-rtd-theme 1.0.0. Because of ReadTheDocs this is probably the most popular theme for sphinx.
The conflict is caused by:
    m2r2 0.3.3 depends on docutils>=0.19
    sphinx 5.1.1 depends on docutils<0.20 and >=0.14
    sphinx-rtd-theme 1.0.0 depends on docutils<0.18

That's not the only one, its even incompatible with myst_parser. Please consider downgrading to 0.18.0.

Use v0.3.2

Use v0.3.2

Of course, no other option for now.
Could you not incorporate some logic to handle using an older version of docutils?

You could pin down a version of m2r2 in your build config by m2r2==0.3.2

That's exactly what I do. All my projects are failing on the dependabot update.
Example: https://github.com/LizardByte/Sunshine/pull/327

There's nothing than can be done

Of course there is. It's quite easy to handle import logic... try/except/else/finally or if/else.
Untested, but this should do the trick.
import mistune
from docutils import io, nodes, statemachine, utils
from docutils.parsers import rst
from docutils.utils import column_width
from pkg_resources import get_distribution

try:
    ErrorString = io.error_string
except:
    from docutils.core import ErrorString

# use ErrorString in code below

Above code is based on this PR: https://github.com/CrossNox/m2r2/pull/49/files ... I don't know if there multiple PRs to handle the update of docutils to v0.19.
Since readthedocs is supporting python 2.7 still, it's unlikely they'll make v0.19 of docutils a requirement and probably instead stick with < 0.18.

I see that there were two new versions for 0.3.3. Is it possible to redact 0.3.3 from pypi in favor of the post2 version?
The original 0.3.3 is still installable under Python 2.7.


organization logo

it would be nice since it is an image library to have a logo rather than the github default organization, I will try to make points to be part of the organization

done! at least for now :)


Semantic-release

Generate Changelog automatically. This should be linked somehow with commit messages and lint
https://github.com/semantic-release/changelog

Pipeline should be something like this .github/workflows/publish.yml (the env var NPM_AUTH_TOKEN is already added in the repo):
name: CI
push:
  branches:
    - master
jobs:
  ci:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2
      - uses: actions/setup-node@v1
        with:
          node-version: 12
      - run: npm ci
      - run: npm run build
      - run: npm run lint
      - run: npm test
      - run: npm run publish
        env:
          NODE_AUTH_TOKEN: ${{ secrets.NPM_AUTH_TOKEN }}


Break long strings

One thing that clang-format does that is super useful is to split long strings into shorter ones at whitespace boundaries. In c++ this does not even require a concatenation operator. I am not sure how that works in rust.
It is very useful writing longer text (like error messages or usage help texts) in one string and later run clang-format and it aligns everything nicely. If the text needs editing, clang-format can be rerun and it will merge and split the string again to make it look nice again.

If I understand you correctly, then rustfmt already does this in a way. It doesn't break them into shorter strings, but breaks long literals into multiple lines. I just opened a PR with some tests for this functionality: https://github.com/nrc/rustfmt/pull/109.

@SirVer do you like the way rustfmt does this? Or do you have ideas to do it better?

No, that is exactly what I had in mind. I wanted to test that feature on a test program:
struct Error(String);

fn main() {
    let b = Ok(1);
    let e = match b {
        Ok(a) => a,
        Err(e) => Err(Error(format!("Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum: {}", err.description()))),
    };
}

but unfortunately on my Mac I am unable to run rustfmt at head:
$ rm -rf target
$ cargo build
   Compiling rustfmt v0.0.1 (file:///Users/sirver/Desktop/Programming/rust/rustfmt)
   Compiling rustc-serialize v0.3.14
   Compiling strings v0.0.1 (https://github.com/nrc/strings.rs.git#551331d0)
   Compiling toml v0.1.20
$ target/debug/rustfmt
dyld: Library not loaded: x86_64-apple-darwin/stage2/lib/rustlib/x86_64-apple-darwin/lib/librustc_driver-11582ce5.dylib
  Referenced from: /Users/sirver/Desktop/Programming/rust/rustfmt/target/debug/rustfmt
  Reason: image not found
zsh: trace trap (core dumped)  target/debug/rustfmt
$ rustc --version
rustc 1.2.0-nightly (8937ec100 2015-06-15)
$ git rev-parse HEAD
b74cdb7b533a1c9b37ed66e74ca5246887d9afef

Should I open a new bug report for that?

That looks like some kind of library path error. I've not seen anything like it. I expect it is a local error with your setup. Best path is to ask around on #rust-internals to see if anyone else has seen such an error. If it is a problem with rustfmt, file a bug report, but I suspect it is not.

The panic happens because of https://github.com/nrc/rustfmt/issues/86. Rustfmt runs on the same example with a non-tuple struct:
struct Error {
    x: String,
}

fn main() {
    let b = Ok(1);
    let e = match b {
        Ok(a) => a,
        Err(e) => Err(Error { x: format!("Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum: {}", err.description())}),
    };
}

The string literal isn't split correctly, though. I suspect this is because the literal is both inside a macro and a match, neither of which are formatted (yet!)

Without the crash I have the same behavior as you, i.e. the string is not reformatted. My OP also used a string in a format!, so that is why I did not see it fixed. That use case is rather frequent for me though, would be nice to see it fixed.

So, I'm pretty sure the only reason this isn't working is because the string is inside a macro. That is a big thing to fix, but possible, and fun. See #35. I think that will completely fix this, so closing this issue.

Makes sense. Thanks.


handle null params

fixes #2443

Gonna close my PR in favor of this one.

LGTM - only thing left is I think the build is broken.

Finally!
What is a planned release time?

@s13o
The next Spring Cloud release is scheduled for next week, but it can be delayed.


About how to use

I feel sorry to bother you. Would you mind to tell me how to use your WTConv in my model(I use Unet as my backbone).

No problem, if your backbone uses depth-wise convolutions you can switch them directly with WTConv (see wtconvnext.py line 103).
If your backbone uses only dense convolutions, you can replace them with separable convs, for example:
conv = nn.Conv2d(in_c, out_c, kernel_size=3)

can be replaced with:
conv_dw = nn.Conv2d(in_c, in_c, kernel_size=3, groups=3)  # depth-wise 3x3
conv_pw = nn.Conv2d(in_c, out_c, ketnel_size=1)  # point-wise 1x1

The results should be quite similar to the original network (however, it should be more efficient).
Now you can switch the depthwise conv with WTConv
conv_dw = WTConv(in_c, in_c, kernel_size=3, wt_levels=3)  # WTConv with 3x3 kernel and 3 levels
conv_pw = nn.Conv2d(in_c, out_c, ketnel_size=1)  # point-wise 1x1

I really appreciate your kind help.

I'm glad I can help :)

WTConv supports strides, you can use it.
So instead of
conv = nn.Conv2d(in_c, out_c, kernel_size=3, stride=2, padding=1)

You can use
conv = nn.Sequential(
    WTConv2d(in_c, in_c, kernel_size=3, stride=2, wt_levels=2),
    nn.Conv2d(in_c, out_c,, kernel_size=1)
)

(and modify wt_levels as you like)


move media to correct status and set progess_limit correctly

Ajax still needs to be fixed to remove it from the list (if we do this) when the status is changed.
I am also not sure why status == :completed does not work, so just changed to a string.

screw you timeout error. 🖕

Ajax still needs to be fixed to remove it from the list

Is this when it's in CW and you set it to 12/12, it goes to Completed? I vote against that since I also adjust the rating and content disappearing on you is 👎
Basically the way it works in V2 is 👌

uhhh that is the way it works in v2? If it is set to 12/12 it goes to completed, if you change status to completed, it goes to completed (and sets to 12/12). Both of these options remove it from the list (in v2).


Simple Colors Experiment Replication

Hi,
I've been trying to reproduce the results from the paper on the simple colors dataset. I'm able to get good segmentations when using the command given:
python train_mdl.py \     --coding_len_coeff=0.1 \     --use_abs_pos_kl=1.0 \     --batch-size=512 \     --seed=0 \     --dataset-path=./data/toy_data.npy \     --max-iters=30000

However, this uses the default skill length prior values of seg-num = 5, seg-len = 8. In the paper it's stated that LOVE should work even without this prior required by VTA but I tried setting both values to larger than the sequence length as suggested in the paper but couldn't get good segmentations:
python train_mdl.py \     --coding_len_coeff=0.1 \     --use_abs_pos_kl=1.0 \     --batch-size=512 \     --seed=0 \     --seg-num=24 \     --seg-len=24 \     --dataset-path=./data/toy_data.npy \     --max-iters=3000

Are there other hyperparameter changes I need to make to replicate the results from the paper?
Thanks!

Have you tried running with a different seed? I believe that this should indeed work with the larger values of sequence length and numbers of segments, and that perhaps this is just an unlucky seed.
We're currently swamped with ICML, but can take a closer look afterwards, if this continues to not work.


Import models and scenarios from generative search engine verifiability paper

We should import over the relevant models and scenarios from https://github.com/nelson-liu/evaluating-verifiability-in-generative-search-engines
cc @nelson-liu

I looked at the paper and it seems that the models are:

Bing Chat
NeevaAI
perplexity.ai
YouChat
Do we have API keys for those models?

For Bing Chat, it seems like there is no official repo or API. I found this repo though: https://github.com/acheong08/EdgeGPT/tree/master/docs

For NeevaAI, I did not find anything, and they announced that they have no API: https://help.neeva.com/hc/en-us/articles/4404882342803-Developer-API

For YouChat, I found this: https://github.com/ettiebot/chatapi/tree/main

Not planned.


analyzer crash: type 'MixinDeclarationImpl' is not a subtype of type 'ClassDeclaration'

Please review the below report for any information you do not wish to share and report to
https://github.com/dart-lang/sdk/issues/new
Exception from analysis server (running from VSCode / Dart Code)
What I was doing
Added prefer_mixin lint to analysis_options.yaml and restarted VSCode.
Versions

Flutter SDK 0.4.3-pre.1162
Visual Studio Code 1.27.1
Dart Code 2.18.0

Analyzer Info
The analyzer was launched using the arguments:
/home/yjbanov/code/flutter/flutter/bin/cache/dart-sdk/bin/snapshots/analysis_server.dart.snapshot
--client-id=Dart-Code.dart-code
--client-version=2.18.0

Exception
Exception while using a AvoidRenamingMethodParameters to visit a MethodDeclarationImpl in MixinDeclarationImpl in CompilationUnitImpl
type 'MixinDeclarationImpl' is not a subtype of type 'ClassDeclaration'
#0      _Visitor.visitMethodDeclaration (package:linter/src/rules/avoid_renaming_method_parameters.dart:69:22)
#1      MethodDeclarationImpl.accept (package:analyzer/src/dart/ast/ast.dart:7686:49)
#2      LinterVisitor._runSubscriptions (package:analyzer/src/lint/linter_visitor.dart:671:14)
#3      LinterVisitor.visitMethodDeclaration (package:analyzer/src/lint/linter_visitor.dart:419:5)
#4      MethodDeclarationImpl.accept (package:analyzer/src/dart/ast/ast.dart:7686:49)
#5      NodeListImpl.accept (package:analyzer/src/dart/ast/ast.dart:8304:20)
#6      MixinDeclarationImpl.visitChildren (package:analyzer/src/dart/ast/ast.dart:7919:13)
#7      RecursiveAstVisitor.visitMixinDeclaration (package:analyzer/dart/ast/visitor.dart:1026:10)
#8      LinterVisitor.visitMixinDeclaration (package:analyzer/src/lint/linter_visitor.dart:432:11)
#9      MixinDeclarationImpl.accept (package:analyzer/src/dart/ast/ast.dart:7910:49)
#10     NodeListImpl.accept (package:analyzer/src/dart/ast/ast.dart:8304:20)
#11     CompilationUnitImpl.visitChildren (package:analyzer/src/dart/ast/ast.dart:2437:21)
#12     RecursiveAstVisitor.visitCompilationUnit (package:analyzer/dart/ast/visitor.dart:714:10)
#13     LinterVisitor.visitCompilationUnit (package:analyzer/src/lint/linter_visitor.dart:120:11)
#14     CompilationUnitImpl.accept (package:analyzer/src/dart/ast/ast.dart:2430:49)
#15     LibraryAnalyzer._computeLints (package:analyzer/src/dart/analysis/library_analyzer.dart:269:10)
#16     LibraryAnalyzer._analyze.<anonymous closure>.<anonymous closure> (package:analyzer/src/dart/analysis/library_analyzer.dart:139:13)
#17     __InternalLinkedHashMap&_HashVMBase&MapMixin&_LinkedHashMapMixin.forEach (dart:collection/runtime/libcompact_hash.dart:370:8)
#18     LibraryAnalyzer._analyze.<anonymous closure> (package:analyzer/src/dart/analysis/library_analyzer.dart:138:17)
#19     _PerformanceTagImpl.makeCurrentWhile (package:analyzer/src/generated/utilities_general.dart:210:15)
#20     LibraryAnalyzer._analyze (package:analyzer/src/dart/analysis/library_analyzer.dart:137:37)
#21     LibraryAnalyzer.analyze.<anonymous closure> (package:analyzer/src/dart/analysis/library_analyzer.dart:78:14)
#22     _AsyncAwaitCompleter.start (dart:async/runtime/libasync_patch.dart:49:6)
#23     LibraryAnalyzer.analyze.<anonymous closure> (package:analyzer/src/dart/analysis/library_analyzer.dart:77:65)
#24     _PerformanceTagImpl.makeCurrentWhileAsync (package:analyzer/src/generated/utilities_general.dart:222:21)
#25     _asyncThenWrapperHelper.<anonymous closure> (dart:async/runtime/libasync_patch.dart:77:64)
#26     _rootRunUnary (dart:async/zone.dart:1132:38)
#27     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#28     _FutureListener.handleValue (dart:async/future_impl.dart:129:18)
#29     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:642:45)
#30     Future._propagateToListeners (dart:async/future_impl.dart:671:32)
#31     Future._addListener.<anonymous closure> (dart:async/future_impl.dart:351:9)
#32     _rootRun (dart:async/zone.dart:1124:13)
#33     _CustomZone.run (dart:async/zone.dart:1021:19)
#34     _CustomZone.runGuarded (dart:async/zone.dart:923:7)
#35     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:963:23)
#36     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#37     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#38     _Timer._runTimers (dart:isolate/runtime/libtimer_impl.dart:391:30)
#39     _Timer._handleMessage (dart:isolate/runtime/libtimer_impl.dart:416:5)
#40     _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:171:12)

Diagnostics requested after the error occurred are:
[
    {
        "name": "/home/yjbanov/code/flutter/flutter/packages/flutter",
        "explicitFileCount": 895,
        "implicitFileCount": -37,
        "workItemQueueLength": 503,
        "cacheEntryExceptions": []
    }
]

@stereotype441 @JekCharlsonYu @leafpetersen

The linter has been updated in SDK.
So, the fix should be in.
https://dart-review.googlesource.com/c/sdk/+/76565


fix: better handle canceled contexts in queries

Make sure we don't block forever writing the result.
Make sure we stop the query when the context is canceled.
Allow receiving one result after the context is canceled, the query could have been writing it and may not have checked if the context was canceled yet.

The bugfix looks correct, but the test feels flaky (if only because its a weird assertion).

I agree. I'll make it a little more flexible.

Ok, I gave it a flex of "10". We need to cancel within 10 results.


Implement n-best decoding without an LM.

The results of using 1-best and n-best decoding are given below:

The column in nbest indicates the size of n in n-best.

Detailed WERs are shown below:
# NO LM Rescoring
# 1-best
# [test-clean-no_rescore] %WER 7.19% [3781 / 52576, 503 ins, 318 del, 2960 sub ]
# [test-other-no_rescore] %WER 18.99% [9941 / 52343, 1129 ins, 930 del, 7882 sub ]
#
# [test-clean-no_rescore-50] %WER 7.20% [3785 / 52576, 606 ins, 279 del, 2900 sub ]
# [test-other-no_rescore-50] %WER 19.23% [10064 / 52343, 1418 ins, 801 del, 7845 sub ]

# [test-clean-no_rescore-100] %WER 7.18% [3774 / 52576, 604 ins, 279 del, 2891 sub ]
# [test-other-no_rescore-100] %WER 19.23% [10063 / 52343, 1420 ins, 800 del, 7843 sub ]

# [test-clean-no_rescore-200] %WER 7.18% [3777 / 52576, 604 ins, 279 del, 2894 sub ]
# [test-other-no_rescore-200] %WER 19.24% [10069 / 52343, 1411 ins, 810 del, 7848 sub ]

# [test-clean-no_rescore-300] %WER 7.19% [3778 / 52576, 604 ins, 279 del, 2895 sub ]
# [test-other-no_rescore-300] %WER 19.20% [10052 / 52343, 1409 ins, 805 del, 7838 sub ]

# [test-clean-no_rescore-500] %WER 7.19% [3778 / 52576, 604 ins, 279 del, 2895 sub ]
# [test-other-no_rescore-500] %WER 19.25% [10074 / 52343, 1420 ins, 809 del, 7845 sub ]

n-best decoding without an LM does not seem to outperform 1-best decoding when no LM is used.
Anyway, it shows how we can use k2 for n-best decoding.

Cool!
I suppose it's OK to merge even though it doesn't help, because  it might be useful as an example.


Create installs into fixed path

The install is not so flexible - as it includes version number (2.1.0) in the path - and hence becomes a moving target - if the path could be fixed it would be easier to configure bindmounts etc.
drwxr-xr-x 2 wso2user wso2  26 Mar 17 13:56 <IP_ADDRESS>
drwxr-xr-x 8 wso2user wso2 255 Mar 16 12:51 jdk-8u112
drwxrwxr-x 1 wso2user wso2  68 Mar 17 13:56 wso2am-2.1.0
wso2user@498c0937865d:/mnt$ pwd
/mnt
wso2user@498c0937865d:/mnt$

The same goes for all configuration - it's a bit of a hassle to have to bind-mount one and one file over the default ones in order to configure the container for your environment.

I'm not exactly clear on the problem here. Is it the 2.1.0 on the path /mnt/wso2am-2.1.0?

Yes - it would be cleaner if it was on a fixed path such as /app or /mnt/wso2am - having a symlink would fix it too - generally anything that will make it more like http://12factor.net to run

+1
I assume you're using WSO2 Puppet modules to configure the Docker images. If that is the case, the name of the symlink that is created at /mnt can be configured so that it wouldn't contain the product name.
Navigate to PUPPET_HOME/hieradata/dev/wso2/common.yaml file. There is a field with key wso2::carbon_home_symlink (around line 23 at the moment). Change this so that reflects something like the following.
wso2::carbon_home_symlink: "/mnt/%{::product_name}"

This should result in the product version agnostic sym link at /mnt/wso2am. :)

@imesh @lasanthaS Maybe we can make the above as the default pattern for sym link generation, seeing as the product version information could anyway be embedded in the Docker image tag? If this feels like a Docker only scenario, maybe we could have this be overridden at the platform level common data file?

+1 for the suggestion David!
All WSO2 product distributions contain the product version in their folder names. Up to now we have not changed that in deployments. It's true that it will be easier to manage volume mounts and Docker COPY commands if the product path does not contain the product version.
@chamilad +1 the proposal!


Keep debs of previous Pipewire versions in the PPA

I would like to suggest to keep the most recent, say, ~5 Pipewire releases in the PPA, so that it's easy to roll back an update in case of regressions and new bugs. For instance, I just ran into https://gitlab.freedesktop.org/pipewire/pipewire/-/issues/1975 which seems to have been introduced with v0.3.41 or v0.3.42. Unfortunately, though, I can't downgrade back to v0.3.40 since I can't find the corresponding deb files anywhere (neither in the PPA nor anywhere else).

duplicate of #32  Seems peoples have lots of problem finding old releases, I will mention it in Readme.md

Thanks so much! I had really tried everything, so I'm glad I wasn't the only one haha.
What could be the reason that apt-get doesn't find those old releases, though?

apt only find releases not superseeded one...
This behaviour is true for all distros... you need to manually download debs and install with dpkg -i


fix: reaction bounds of atp-reactions and two cpk-reactions

Main improvements in this PR:
Updated reaction bounds of 38 ATP-drivenr eactions, and also two reactions CPKS4a and b in the coelimycin biosynthesis.
Reconstructed model and checked that growth predictions are unchanged.
I hereby confirm that I have:

[x] Tested my code with all requirements for running the model
[x] Selected devel as a target branch (top left drop-down menu)
[x] If needed, asked first in the Gitter chat room about this PR

Closes #54


Beefy: Provide well-formed ValidatorSet

PR:

provides strict API for creating and working with a Beefy ValidatorSet,
disallows beefy-gadget operations (fail early) when ValidatorSet in not initialized (created).
makes some code efficiency improvements, removes some data cloning

Fixes https://github.com/paritytech/grandpa-bridge-gadget/issues/307

Needs companion PR in Polkadot.
Polkadot implements dummy BeefyApi (see here) with empty validator set which is not possible (not allowed) anymore with this PR.
What should we do for these dummy BeefyApi runtime api impl in Polkadot, Kusama, Westend? We either:

remove them -> implies runtime API change
or,
provide some other dummy validator set - some random key instead of empty set.

WDYT?

@tomusdrw can you please take a look?


Replace datatable

Hi!
I'm think of replacing datatable with bootgrid. I've used it in one of my projects and I'm quite satisfied with it.  What do you think @thekordy ? If you agree, I'll start doing it. I saw it once on an old todo-list that the removal of datatable was actually planned.

Hi
Actually I am interested, let's do it :+1:

Is it better?

In my opinion, yes. There are fewer possibilities, but more than enough.

https://github.com/thekordy/ticketit/commit/cb4711b12cd1ac099d28174fbd3766b89b2707b3
Here's the first approach. Still needs testing

I added text search functionality. It searches in subject, body and comments

Hi balping, the bootgrid is not loading data. what could the problem be. have tried to troubleshoot but have failed. Please help

Hi @kasirye please use the network tab of chrome dev tools and see if anything is wrong there

@balping there is nothing wrong... i see no errors.  I'll make a fresh install anyways and see what' wrong

Yes @balping, i have some tickets in the database

It looks like bootgrid's js file isn't loaded check, there's a reference to it in the page's source html. If not, maybe your blade files are not set correctly

Perfect @balping,  It was my blade master template. Thanks alot for your time. I appreciate


⚠ T:Riza Docs has degraded performance

In 168e0b3, T:Riza Docs (https://docs.triza.dev) experienced degraded performance:

HTTP code: 200
Response time: 4687 ms

Resolved: T:Riza Docs performance has improved in 49e6737.


Valid icon doesn't disappear when boolean fields are unchecked

The issue is this:
  nga.field('test', 'boolean').label('Test');

I check it on and the right valid green flag appears but when I uncheck it, it doesn't disappear.
Even the label still remains green.
I also added a validation (required: false) but it didn't worked.
Is there any work around?

Am I understanding correctly? You have a boolean field, and you want to give it the value 'undefined', not true or false. If this is your problem, it's related to #225.

No reply for two weeks, besides the handling of boolean fields has changed since #629. Closing.


Reporting a comment should result in an action after n reports.

@smn @miltontony Is this still something we need to work on?

@miltontony would know. Considering it's had zero activity since August I'd suggest closing it.

It was resolved in https://github.com/praekelt/molo.commenting/pull/10


add yq

adds the yq (github.com/mikefarah/yq/) yaml processor

I gave a shot at integrating it with renovate. please let me know if you prefer another approach

Thanks for the PR! One thing about the renovate is it will update the image whenever the GH repository mikefarah/qy release a new version. Will the apk repository http://dl-cdn.alpinelinux.org/alpine/edge/community gets the updated version as soon as the git repo create a new release? If not then the renovate updating image might break.

I see that yq release does include all the binary assets in all kind of formats, You can download the binary and put it inside /usr/local/bin like how I do kustomize etc. There's an extracted version https://github.com/mikefarah/yq/releases/download/v4.34.2/yq_linux_amd64 that you can just curl and rename it to yq without needing to use tar too.

I see your point. Besides, if i understand correctly, it looks the alpine package maintainer appears to only publish the latest version of yq with each release of alpine, and is not publishing all released versions of yq itself.
https://pkgs.alpinelinux.org/packages?name=yq&branch=v3.18&repo=community&arch=&maintainer=

Maybe do it like this might be better because we don't need to worry if the alpine repo is updated or not:
RUN curl -L -o /usr/local/bin/yq \
    https://github.com/mikefarah/yq/releases/download/${YQ_VERSION}/yq_linux_amd64 \
    && chmod +x /usr/local/bin/yq

also, if it seems ok to you, how would you feel about adding ca-certificates, openssh-client, and curl (well, just don't del curl)?
it would allow the container to clone a remote git manifest repo, update a manifest yaml with yq, and commit the change.
i can add it in another commit or this one if preferred

Maybe do it like this might be better because we don't need to worry if the alpine repo is updated or not:
RUN curl -L -o /usr/local/bin/yq \
    https://github.com/mikefarah/yq/releases/download/${YQ_VERSION}/yq_linux_amd64 \
    && chmod +x /usr/local/bin/yq

I did try this, and i'm not sure whether it's an issue with how that package is built vs the one in the alpine repo... but i found that the one in the alpine repo did not encounter certain errors that the prebuilt binaries on github did- even given the same version of yq.

also, if it seems ok to you, how would you feel about adding ca-certificates, openssh-client, and curl (well, just don't del curl)?
it would allow the container to clone a remote git manifest repo, update a manifest yaml with yq, and commit the change.
i can add it in another commit or this one if preferred

I'm not sure about adding openssh and ca-certificates. I don't think pushing a commit from a linting tool output is a good practice imho. It should be the linting tool telling you about something is wrong and you fix it manually based on the suggestion given or  even chose to ignore it sometimes. But that's just my opinion. Are u sure having them in the container is a good choice?

i understand, and I can separate it to another container.
The way I had imagined it, just using different ci stages. One stage for linting, then another stage to update k8s manifests / deploy config to the environments.
My goal or thought was to use the same image for all these stages- linting and k8s manifest updates, since all these operations share so much in common, and having just slightly different containers for each seemed unnecessary.

Okay I'll let this slide because there's maybe only a couple of people using this container anyway. :smile:

Fixes: https://github.com/budimanjojo/kubernetes-toolbox/issues/154


Enhancement: Centers the labels on settings page

Closes #1965. Centers the labels on settings page and also adds padding for better onPress accuracy.
Before:

After:

This fix can be further improved but for now it looks better than the current state IMO.

@borisyankov Made the changes.

Thanks @pulkonet for taking a look at this!
I don't think this is actually an improvement; it makes the two links on the main settings screen oddly prominent, when most users will want to look at either of them between 0 and 1 times (and mostly 0) in the whole time they use the app.
Instead, let's follow the pattern that I see looking at several other apps (including Google Maps, Google Play, and FB Messenger):

have an item "Terms & Privacy", which leads to another settings screen
on that screen, have each of these as regular items that look like any other settings items.

I'll go turn that into a quick issue.

Oh ha, in fact I already said that on #1965 last week. :) I'll follow up there.


Document and consolidate component decisions and history of components in Figma.

Using the template Ravina used for the Date Input:
Consolidate our other Component Decisions:

https://www.figma.com/file/p0eiLxdme1g93sIuggjipR/Colour-%26-typography-differences-GCDS-vs.-Canada.ca?type=design&node-id=0-1&mode=design&t=jQvvFQ2SJ2Bb0SjL-0
https://drive.google.com/drive/folders/142OxVWx-Q23OBSLu1FqUCoJ8_imhtyIU
https://docs.google.com/document/d/12-jUh2iDUOmPt0qiN58Xco_uUJQsa5yNgUl7DrmrYis/edit#heading=h.hm6izldno7xr

All should be consolidated into a single Figma page per component. This will help in the future if we need to reference our decisions or share externally our rationale.

Done: new Figma section created to house decisions, past versions and more.
https://www.figma.com/design/4wF8Kr9o2sGTQjaOpY7qcZ/Component-archive-%2B-decision-record?node-id=567-9908&t=mLVB5FFKbaiwZy4I-0


schema.switch support in version 0.7.0-rc.3

Here's the schema i'm using:
{
 "type": "object",
 "required": [
  "type"
 ],
 "properties": {
  "id": {
   "type": "integer"
  },
  "type": {
   "type": "string"
  }
 },
 "switch": [
  {
   "if": {
    "properties": {
     "type": {
      "constant": "site"
     }
    }
   },
   "then": {
    "properties": {
     "data": {
      "type": "object",
      "default": {},
      "properties": {
       "name": {
        "type": "string"
       }
      },
      "required": [
       "name"
      ]
     }
    }
   }
  },
  {
   "if": {
    "properties": {
     "type": {
      "constant": "page"
     }
    }
   },
   "then": {
    "properties": {
     "data": {
      "type": "object",
      "default": {},
      "properties": {
       "title": {
        "type": [
         "string",
         "null"
        ]
       },
       "template": {
        "type": "string"
       }
      },
      "required": [
       "template"
      ]
     }
    }
   }
  }
 ]
}

It validates all right with ajv and that kind of data:
{
 "type": "page",
 "data": {
  "title": "Home",
  "template": "templates/home.html"
 }
}

$afterValidate receives that json all right.
However, when inserting it using insertGraph method, it strips the data property.
I have to modify the schema and declare "properties.data" to keep objection from pruning that column.
I suppose the problem comes from the "switch" clause, while objection only checks what's in the schema "properties".
Isn't there any way to abstract that second-guessing from objection and let all knowledge about schema be given by ajv ?

Yes

Ha, that explains a lot of confusion i had. IMO pickJsonSchemaProperties should be false by default, but i won't complain.

I agree. The feature is just a historical remnant. I'll probably set the default to true or remove the feature alltogether at some point.

Well, actually, before changing anything, i think something's wrong with ajv or with json-schema spec about additionalProperties and switch keywords.

Did you reopen this by accident? Or is there something you wish to be changed in objection?

I'll probably set the default to false or remove the feature alltogether at some point.

so i thought this issue should stay open until that point ? Or not !

We should open a separate issue for that. This isn't very descriptive.


Question: Mixin compatibility

Does plugin can generate refmaps? I want to use it with mixins.

Hello! This plugin doesn’t currently generate refmaps. I would need to look into how that works since I haven’t looked into the internals of mixins very much, and I don’t really know what a refmap is or how to generate it to be honest. There was a pull request, #12 that attempted to add support for remapping mixins, but I did really like how it had to generate mappings manually every time. Since you are the second person to ask about this there seems to be demand for it, so I could maybe look into it and see if there is a way to create this refmap file with the data the plugin already has access to.

Thanks for answer. I just wanted to use mixins on paper using ignite launcher without remap=false


PPO2 does not work on mujoco env

Hi, everyone. I ran ppo2 on mujoco env with default parameters as follows:
python -m baselines.run --alg=ppo2 --env=Ant-v2 --num_timesteps=1e6
but the code is stuck on the following line:
https://github.com/openai/baselines/blob/1259f6ab25c6f7261e33c4c3b92df869188f9260/baselines/ppo2/ppo2.py#L157
and I didn't get any error or warning.
Actually, both Atari envs using ppo2  and any env using trpo-mpi work successfully.
I use python 3.5.2 virtual env on anaconda. And mujoco and tensorflow-gpu work well.
So if anyone could help me out here, I would be very happy. Thanks

Could you post the console output? Even if it doesn't show the error, it would still be helpful to see what's being printed out. How do you know it's stuck at that line?

@brett-daley  Thanks for your reply. I debugged the code step by step via Pycharm and found it's stuck at that line.
Ran python -m baselines.run --alg=ppo2 --env=Ant-v2 --num_timesteps=1e6 ， and the console output as follows:
/tmp/openai-2019-03-13-21-18-19-703742
Logging to /tmp/openai-2019-03-13-21-18-19-703742
env_type: mujoco
2019-03-13 21:18:23.796512: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 AVX512F FMA
2019-03-13 21:18:23.877188: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:964] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2019-03-13 21:18:23.877643: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1432] Found device 0 with properties:
name: GeForce GTX 1080 Ti major: 6 minor: 1 memoryClockRate(GHz): 1.582
pciBusID: 0000:65:00.0
totalMemory: 10.91GiB freeMemory: 7.81GiB
2019-03-13 21:18:23.877663: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1511] Adding visible gpu devices: 0
2019-03-13 21:18:24.145689: I tensorflow/core/common_runtime/gpu/gpu_device.cc:982] Device interconnect StreamExecutor with strength 1 edge matrix:
2019-03-13 21:18:24.145726: I tensorflow/core/common_runtime/gpu/gpu_device.cc:988]      0
2019-03-13 21:18:24.145734: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1001] 0:   N
2019-03-13 21:18:24.145940: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1115] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 7537 MB memory) -> physical GPU (device: 0, name: GeForce GTX 1080 Ti, pci bus id: 0000:65:00.0, compute capability: 6.1)
/home/drl/anaconda3/envs/mujoco150_new/lib/python3.5/site-packages/gym/envs/registration.py:14: PkgResourcesDeprecationWarning: Parameters to load are deprecated.  Call .resolve and .require separately.
result = entry_point.load(False)
WARN: gym.spaces.Box autodetected dtype as <class 'numpy.float32'>. Please provide explicit dtype.
WARN: gym.spaces.Box autodetected dtype as <class 'numpy.float32'>. Please provide explicit dtype.
Training ppo2 on mujoco:Ant-v2 with arguments
{'nminibatches': 32, 'gamma': 0.99, 'value_network': 'copy', 'network': 'mlp', 'cliprange': 0.2, 'lr': <function mujoco.. at 0x7f6a44f65b70>, 'ent_coef': 0.0, 'log_interval': 1, 'nsteps': 2048, 'lam': 0.95, 'noptepochs': 10}

Have you tried other environments like Hopper-v2 or other baseline algorithms? Do they also not work?
Since it's getting stuck on the train function, I wonder if it's a Tensorflow/GPU issue. Try export CUDA_VISIBLE_DEVICES='' in your command-line to force it to use the CPU and let us know if it runs.

@brett-daley  Other mujoco envs doesn't work.
I force to use the CPU via '', but the code also is stuck at the same line.
The console output:
/tmp/openai-2019-03-14-12-15-59-120520
Logging to /tmp/openai-2019-03-14-12-15-59-120520
env_type: mujoco
2019-03-14 12:16:03.907368: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 AVX512F FMA
2019-03-14 12:16:03.915597: E tensorflow/stream_executor/cuda/cuda_driver.cc:300] failed call to cuInit: CUDA_ERROR_NO_DEVICE: no CUDA-capable device is detected
2019-03-14 12:16:03.915644: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:163] retrieving CUDA diagnostic information for host: drl-pc
2019-03-14 12:16:03.915651: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:170] hostname: drl-pc
2019-03-14 12:16:03.915697: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:194] libcuda reported version is: 390.48.0
2019-03-14 12:16:03.915722: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:198] kernel reported version is: 390.48.0
2019-03-14 12:16:03.915728: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:305] kernel version seems to match DSO: 390.48.0
/home/drl/anaconda3/envs/mujoco150_new/lib/python3.5/site-packages/gym/envs/registration.py:14: PkgResourcesDeprecationWarning: Parameters to load are deprecated.  Call .resolve and .require separately.
result = entry_point.load(False)
WARN: gym.spaces.Box autodetected dtype as <class 'numpy.float32'>. Please provide explicit dtype.
WARN: gym.spaces.Box autodetected dtype as <class 'numpy.float32'>. Please provide explicit dtype.
Training ppo2 on mujoco:Hopper-v2 with arguments
{'gamma': 0.99, 'log_interval': 1, 'network': 'mlp', 'lr': <function mujoco.. at 0x7f3682665b70>, 'value_network': 'copy', 'cliprange': 0.2, 'lam': 0.95, 'noptepochs': 10, 'ent_coef': 0.0, 'nsteps': 2048, 'nminibatches': 32}
My TensorFlow version is 1.12.0, and Cuda 9.0. I don't know if there's something wrong with the python environment configuration. But it is weird. Because trpo-mpi works well, which means TensorFlow/GPU/mpi/mujoco are okay.

there was a bug in mpi_adam_optimizer that was trying to call a single-threaded session from within that session (via a tf.pyfunc). Should be fixed now.


subpath field is not removed when submitting workload from file

Pull request
subpath field is not ignored when submitting workload from file(both while creating & updating)
What this PR does / why we need it
Fixes #88
Describe testing done for PR

create workload with subPath
export the workload into a file
apply the workload again, subPath should not be deleted

Minor nit, when we merge this, we should change the commit message from [fix] to fix: to follow the standard.


Wrap response

#54
This adds the jikan_url and response headers to requests:
replaces _check_response with:
_wrap_response (implemented/overriden in jikan/aiojikan), which:
parses the json (sync/async specific)
calls (abstractjikan) _check_response (raises API error for failure)
calls (abstractjikan) _add_jikan_metadata (to add the headers and jikan url)
>>> bebop = jikanpy.Jikan().anime(1)
>>> bebop["jikan_url"]
'https://api.jikan.moe/v3/anime/1'
>>> bebop["headers"]
{'Server': 'nginx/1.14.0 (Ubuntu)', 'Date': 'Fri, 30 Aug 2019 06:07:02 GMT', 'Content-Type': 'application/json', 'Content-Length': '2313', 'Connection': 'keep-alive', 'Access-Control-Allow-Origin': '*', 'Access-Control-Allow-Methods': '*', 'ETag': '"b88bef423688b42bc628035d65e4b0ee"', 'X-Request-Hash': 'request:anime:d6092f2422f084452c84555f17c7ba047e6998d3', 'X-Request-Cached': '1', 'X-Request-Cache-Ttl': '43176', 'Content-Encoding': 'gzip', 'Vary': 'Accept-Encoding', 'X-Cache-Status': 'HIT'}

also:
updated the response type hints, to be requests.Response
and aiohttp.ClientResponse instead of Any
updated tests to match changes to _check_response
Couldnt find an logical way to split commits, so I just left it as this.
Let me know if anything should be changed, otherwise I can update the documentation.

Codecov Report

Merging #56 into master will decrease coverage by 0.28%.
The diff coverage is 98.63%.

@@            Coverage Diff             @@
##           master      #56      +/-   ##
==========================================
- Coverage     100%   99.71%   -0.29%
==========================================
  Files           6        6
  Lines         376      352      -24
==========================================
- Hits          376      351      -25
- Misses          0        1       +1

Impacted Files
Coverage Δ

jikanpy/aiojikan.py
100% <100%> (ø)
:arrow_up:

jikanpy/jikan.py
100% <100%> (ø)
:arrow_up:

jikanpy/abstractjikan.py
99.52% <90%> (-0.48%)
:arrow_down:

Continue to review full report at Codecov.

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

I'd think the codecov has gone down because of the other helper functions added, I can write tests for those tomorrow.

I just checked codecov and you just need one method to make sure that _wrap_response in abstractjikan.py raises NotImplementedError.

Thanks. Unless you have any changes, I think that's all I had planned for this PR. Not sure when I'll get to the PR for allowing someone to pass headers, but I'll create an issue for it now.

I haven't actually reviewed the code yet, I just saw the codecov stuff. I'll do it by tonight.

Here are the changes that black proposes when I run: black -t py37 jikanpy/*jikan*

diff --git a/jikanpy/abstractjikan.py b/jikanpy/abstractjikan.py
index 104453b..6bb850f 100644
--- a/jikanpy/abstractjikan.py
+++ b/jikanpy/abstractjikan.py
@@ -56 +56 @@ class AbstractJikan(ABC):
-        **kwargs: Union[int, Optional[str]]
+        **kwargs: Union[int, Optional[str]],
@@ -77 +77 @@ class AbstractJikan(ABC):
-        url: str
+        url: str,
@@ -81 +81,3 @@ class AbstractJikan(ABC):
-        response_dict["headers"] = dict(response.headers)  # converts from CIMultiDictProxy for aiohttp.ClientResponse
+        response_dict["headers"] = dict(
+            response.headers
+        )  # converts from CIMultiDictProxy for aiohttp.ClientResponse
@@ -86,4 +88 @@ class AbstractJikan(ABC):
-        self,
-        response: Any,
-        url: str,
-        **kwargs: Union[int, Optional[str]]
+        self, response: Any, url: str, **kwargs: Union[int, Optional[str]]
diff --git a/jikanpy/aiojikan.py b/jikanpy/aiojikan.py
index 491443e..58bcddc 100644
--- a/jikanpy/aiojikan.py
+++ b/jikanpy/aiojikan.py
@@ -26 +26 @@ class AioJikan(AbstractJikan):
-    async def _wrap_response( # type: ignore
+    async def _wrap_response(  # type: ignore
@@ -30 +30 @@ class AioJikan(AbstractJikan):
-        **kwargs: Union[int, Optional[str]]
+        **kwargs: Union[int, Optional[str]],
@@ -37 +37,3 @@ class AioJikan(AbstractJikan):
-        self._check_response(response_dict=json_response, response_status_code=response.status, **kwargs)
+        self._check_response(
+            response_dict=json_response, response_status_code=response.status, **kwargs
+        )
@@ -56 +58,3 @@ class AioJikan(AbstractJikan):
-        return await self._wrap_response(response, url, id=creator_id, endpoint=creator_type)
+        return await self._wrap_response(
+            response, url, id=creator_id, endpoint=creator_type
+        )
@@ -76 +80,3 @@ class AioJikan(AbstractJikan):
-        response: aiohttp.ClientResponse = await self.session.get(self.season_archive_url)
+        response: aiohttp.ClientResponse = await self.session.get(
+            self.season_archive_url
+        )
@@ -112 +118,3 @@ class AioJikan(AbstractJikan):
-        return await self._wrap_response(response, url, username=username, request=request)
+        return await self._wrap_response(
+            response, url, username=username, request=request
+        )
@@ -123 +131,3 @@ class AioJikan(AbstractJikan):
-        return await self._wrap_response(response, url, request=request, type=type, period=period)
+        return await self._wrap_response(
+            response, url, request=request, type=type, period=period
+        )
diff --git a/jikanpy/jikan.py b/jikanpy/jikan.py
index c1f481e..895a2db 100644
--- a/jikanpy/jikan.py
+++ b/jikanpy/jikan.py
@@ -14,4 +14 @@ class Jikan(AbstractJikan):
-        self,
-        response: requests.Response,
-        url: str,
-        **kwargs: Union[int, Optional[str]]
+        self, response: requests.Response, url: str, **kwargs: Union[int, Optional[str]]
@@ -27 +24,5 @@ class Jikan(AbstractJikan):
-        self._check_response(response_dict=json_response, response_status_code=response.status_code, **kwargs)
+        self._check_response(
+            response_dict=json_response,
+            response_status_code=response.status_code,
+            **kwargs,
+        )
@@ -111 +112,3 @@ class Jikan(AbstractJikan):
-        return self._wrap_response(response, url, request=request, type=type, period=period)
+        return self._wrap_response(
+            response, url, request=request, type=type, period=period
+        )

I'm not a fan of the hanging braces after return statements, I could modify the async return lines so that it gets saved to a variable (json_response?) and then return it, or we can leave it as this.

Included some tests for header keys since those are going be used more in v4
https://github.com/AWConant/jikanpy/issues/54#issuecomment-522727534

Believe that covers everything mentioned in the review, let me know if anything else should be changed.

I'm also okay with the hanging braces on return statements when formatting with black, but if you really don't like it, feel free to store the result into a variable and then return the variable instead.

Adding json_response to some of the aiojikan endpoints but not others which dont need them seems needlessly messy, so I'm find leaving it how black formats it.


Created request and response objects for sdk 2.x

Added

AuthContext
AuthDestination
AuthResponse
AuthType
CrossApp
PrefillInfo
AuthException

I have a personal preference of grouping the top-level definitions in a single file if they're small and closely related to each other. But that's just preference.

I'd like to keep it this way. To me it becomes easier in searching, referencing and more readable. Thanks for the suggestion.


Support for leaveFor

I've used js-spatial-navigation in another project that wasn't using react. Now I'm considering using it again for a react-based project, hence I ended up finding your library. One feature I'd probably miss is the ability to specify leaveFor in sections. Conceptually, that means that you'd need a way to specify the section ID in the component properties, instead of auto-generating them. Is this something you've considered?

I've had to do this in a project of mine.
// FocusableSection.js
getChildContext() {
  return { focusableSectionId: this.sectionId }
}
// ...
componentWillMount() {
  this.sectionId = this.props.id ? this.props.id : `section-${sectionsIds++}`
}
// ...
JsSpatialNavigation.add({
  leaveFor: this.props.leaveFor, // TODO: Add more options like this
  id: this.sectionId,
  selector: selector,
  enterTo: enterTo,
  defaultElement: defaultElement
})

Some bits may be missing but you get the idea :)

I have a PR open, did it via RenderProps and added a neighborLeft/neighborRight/neighborDown/neighborUp as well as a sectionId property which is needed. It all maps through to the sectionId and leaveFor: {neighborLeft, neighborRight, neighborDown, neighborUp} which gets passed to js-spatial-navigation
You can yarn add https://github.com/chwagssd/react-js-spatial-navigation.git if you like for now, until PR #11 gets merged by @dead


Add CRISPR repeat and spacer sequences to database

We want to include the sequences of CRISPR repeats and spacers in our database. Repeat sequences can be useful for comparing CRISPR systems between different species, since species with similar repeat sequences may have some shared CRISPR history. Spacer sequences are the link between viruses and bacteria - since spacers come from viruses or other foreign DNA, studying spacers can shed light on how bacteria and viruses interact.
One source of this data is the CRISPRdb database, which has detected CRISPR arrays in thousands of species of bacteria and archaea. We will start by adding their data to our database, and going forward we will continue to periodically update our database from theirs.

The function
Input:

List of CRISPR repeat sequences from CRISPRdb: http://crispr.i2bc.paris-saclay.fr/crispr/BLAST/DR/DRdatabase
List of CRISPR spacer sequences from CRISPRdb: http://crispr.i2bc.paris-saclay.fr/crispr/BLAST/Spacer/Spacerdatabase

The function should do the following:

Insert each repeat sequence into the Repeat table with an auto-generated Primary Key if it isn't already in the table.
Insert each spacer sequence into the Spacer table with an auto-generated Primary Key if it isn't already in the table.
Insert each accession number into the Organism table in the Accession column. The OrganismName column can remain blank for now - this is addressed in another issue.

Each pair of lines in the linked file contains the accession numbers of all the organisms that have a particular sequence followed by the sequence itself. For example, in this block the first line is a list of accession numbers with a postscript, the second is the repeat (GAATCTTCGAGATAGAATTGCAAG). The actual accession number is just the letters and the first block of numbers, i.e. NC_000854.
>NC_000854_2|NC_000854_4|NC_022521_3|NC_022521_6
GAATCTTCGAGATAGAATTGCAAG

Helpful information:
Find our SQLite database here. Its schema is not in its final form, but it has enough to get us started on rewiring our analysis scripts. Here's what it looks like (courtesy of @dbonsma):

Example data: https://github.com/goyalsid/phageParser/blob/master/data/DRdatabase.txt

Here's the CRISPRdb: http://crispr.i2bc.paris-saclay.fr/

Resolved in #105


SW-3966 Add planting site selector on withdrawal log

Missing: Remove “Destination” filter from Withdrawal History tab’s table filters to avoid conflict with top-of-page Planting Site selector.

Discussed with Constanza. I will make the changes on top of this as a separate PR, to be merged into this PR.

Discussed with Constanza. I will make the changes on top of this as a separate PR, to be merged into this PR.

Thanks Karthik!

Okay, my changes are in.
Since this was authored by @constanzauanini and me, @charlie-tf you may need to approve again 🙏

@charlie-tf @constanzauanini let's get this in? The changes are radical enough we don't want current QA on the older version.

Looks great! 👍

I was reviewing it! Thanks for the changes Karthik!! I can't approve because i'm the author haha

Looks great! 👍

I was reviewing it! Thanks for the changes Karthik!! I can't approve because i'm the author haha

Thanks Charlie and Constanza. Charlie approved it, we should be good. Looks like the QA plan has no direct reference to selecting a planting site, should need no changes. There's this bug that we can probably close: https://terraformation.atlassian.net/browse/SW-3998

Yes, that bug should be fixed with this changes

I will send out a revert on this PR since this may be an incorrect solution. Sorry for the wasted effort.
We need to follow up with Chudo and Alex again.


the learning_rate became nan in seq2seq model

I use the seq2seq provided by tensorflow, a very curious problem is that the learning_rate will become nan only when the length of the bucket is more than 40 with 4 layers.
The learning_rate should be decaied and more and more smaller, why it will became nan?
The init learning_rate is 0.01 and the learning_rate_decay_factor is 0.5
The training data is nist
The training is in a centos with four K40 together.

Closing due to inactivity. Feel free to open a new github issue if the problem still persists in recent versions.


Any demo?

Expected Behavior

Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Version used:
Environment name and version (e.g. Chrome 39, node.js 5.4):
Operating System and version (desktop or mobile):
Link to your project:

Hi there !
Would you like us to add a section about apps using iohook ? So you can have some real-world exampels ?

Closing this, added an Apps section in the README.


Known issue GTM with GA4

Hello again :-),
we have got many online shops and one shop has got a Google Tag Manager with Google Analytics 4 implementation.
After I denied the Cookies:
CookieConsent [AUTOCLEAR]: deleting cookie: '_ga_YZ791V8SYH' path: '/' domain: (2) ["xxx-xxx", ".xxx-xxx"] cookieconsent.js:900 CookieConsent [AUTOCLEAR]: deleting cookie: '_ga' path: '/' domain: (2) ["xxx-xxx", ". xxx-xxx"] cookieconsent.js:900 CookieConsent [SET_COOKIE]: cookie cc_cookie='{"level": ["necessary"]}' was set!
Now I reload the page and 1 of 2 Google Cookies are still available: _ga_YZ791V8SYH
This is only by Google Tag Manager with Google Analytics 4.
Thanks

I did see both cookies are deleted, but If I changed the page only this 1 cookie _ga_YZ791V8SYH is create again.

are you properly loading gtag.js using the required syntax of the cookieconsent? like this:
<script type="text/plain" data-cookiecategory="<your_category>" src="https://www.googletagmanager.com/gtag/js?id=GA_MEASUREMENT_ID" async></script>
<script type="text/plain" data-cookiecategory="<your_category>">
  window.dataLayer = window.dataLayer || [];
  function gtag(){window.dataLayer.push(arguments);}
  gtag('js', new Date());

  gtag('config', 'GA_MEASUREMENT_ID');
</script>

can you make sure there are no other scripts which might trigger ga to run?

does this behavior occur in all browsers (chrome, firefox etc) or on a specific browser only.

do you have any special triggers configured on GA which might cause the cookie to re-set after a specific event?

have you tried enabling the reload option?

if none of these help I will need to manually check the intersted domain/page.

I had same issue. I solved it by using 'update' parameter, not 'default' during onChange in gtag function.
gtag('consent', 'update', {
    'ad_storage': 'denied',
    'analytics_storage': 'denied'
});


Change information

Hey can you change like the values of hidevm_ahci? like the model name and other stuff. Because someone can just try to find exactly the same system information that you put in the hardener and use it to detect if you are using a vm

What prevents you from doing so in your script. It is EXAMPLE information if you don't know that.


How can I update vcpkg from a previous version?

Hi,
I downloaded vcpkg some time ago (version: Vcpkg package management program version 0.0.81-144d3718c4197b101c7d61ee6a258200371fb1ab) and now realize there is a newer version available (Vcpkg package management program version 0.0.91).
What is the correct way to start using the new version? Just download the latest vcpkg and build it with vcpkg.sln and then copy the exe to the root directory of my previous vcpkg tree?
Seems reasonable but just wanted to make sure.
Regards,
Juan Dent

Even simpler: just run .\bootstrap-vcpkg.bat :) Ideally, vcpkg update should mention this as well.

I ran .\bootstrap-vcpkg.bat but it does not change the version of vcpkg!

Sorry for not mentioning: you do need to run git pull first!

sorry for my ignorance -- where am I to run

git pull
?
If I run it on the command prompt I get:

C:\Users\Juan Dent\C++ Development\Source Components\vcpkg-master>git pull
There is no tracking information for the current branch.
Please specify which branch you want to merge with.
See git-pull(1) for details.

git pull <remote> <branch>

If you wish to set tracking information for this branch you can do so with:
git branch --set-upstream-to=origin/<branch> master

Regards,
Juan

Seeing that I had no tracking information for this branch I ran this:

git branch --set-upstream-to=origin/master

Then

git pull

Which got me the following:

error: Your local changes to the following files would be overwritten by merge:
ports/cppwinrt/CONTROL
ports/cppwinrt/portfile.cmake
Please commit your changes or stash them before you merge.
error: The following untracked working tree files would be overwritten by merge:
ports/date/CONTROL
ports/date/portfile.cmake
Please move or remove them before you merge.
Aborting
Updating 144d3718..374253cb

Should I commit the changes to ports/cppwinrt/? Should I delete ports/date/?
Regards,
Juan

If you don't have any changes you'd like to keep, you can simply git reset --hard origin/master to reset your current branch to match the current upstream master.
Otherwise, I'd:
git add ports/cppwinrt
git add ports/date
git commit -m "wip"
git pull

Then, git will indicate what (if any) it was unable to merge on its own and you can decide what changes to keep or discard.

got it thanks!!
Juan


[google_maps_flutter] Can not add a symbol to a polyline on google map

We can not a symbol to a polyline on google map in flutter like we did on below link
https://developers.google.com/maps/documentation/javascript/symbols#add_to_polyline

Treating this as proposal.

Treating this as proposal.

any update on this?

google_maps_flutter developments seem to be at an end point.
Many important feature, already present in javascript, are missing, and the evolution is on stale.
Is there an alternative in Flutter?


Dicom file location is not displayed correctly

I am on this example （https://rawgit.com/cornerstonejs/cornerstoneWADOImageLoader/master/examples/dicomfile/index.html）
Got what I need imageLoader Parameter value
But the final display is not correct and the image display is incomplete.
Hope to help solve
Thank you

Steps to reproduce? Expected behavior vs actual?

@dannyrb  hello
I dragged my local dicom file to Analyze here and got the parameters I needed, but the effect is not ideal.
（https://dicom.innolitics.com/ciods/mr-image）

such as

I'm still not sure I follow. How can I reproduce your issue? If I wanted to have the same result on my machine, what steps would I need to follow? What is the behavior? What is the (different) expected behavior?

This is my code

Sorry, because I am a newbie, I don’t know much about this piece.This is my code
test.zip

Sorry, because I am a newbie, I don’t know much about this piece.This is my code
test.zip


Resolve agent connect to docker socket issues

Signed-off-by: Peter Zhu<EMAIL_ADDRESS>Description
Resolve agent connect to docker socket issues.
The issue is due to the user ec2-user for AL2 and ubuntu for Ubuntu2004 not added to the docker group, thus have no access to the docker socket.
Got permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Get "http://%2Fvar%2Frun%2Fdocker.sock/v1.24/containers/json": dial unix /var/run/docker.sock: connect: permission denied

Issues Resolved
https://github.com/opensearch-project/opensearch-ci/issues/188
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.

Approved @peterzhuamazon assuming the AMI's are already built and releases public.
Thank you

Approved @peterzhuamazon assuming the AMI's are already built and releases public. Thank you

Yes, thanks.


Brunel 2.4: issues with colors of map items with minimum values

I used the US map  (states level) colored with range of "reds" colors based on value having non-negative reals.  The items with the actual value 0 (min possible value in my case) are shown in blue instead of the lowest possible color.   The states with absent info are shown in grey as it should be.

Ah, the perils of writing in multiple languages simultaneously. I used a Javascript shortcut to check for a null or undefined value, ignoring the fact that zero is also not truthy, and we didn't have a good check for that in our tests. The fix was in the Java generation code and I have posted a new version online.


Add FP8 custom op for NVIDIA Hopper GPUs

In this PR, we add one custom op to utilize the fp8 gemms provided by the XLA to better utilize NVIDIA Hopper GPUs. Users can then code-inject this custom op like:
transformer_layer_p.tr_fflayer_tpl.fflayer_tpl.linear_tpl.einsum_tpl = \
          pax_fiddle.Config(fp8_ops.Fp8EinsumOp)

The PR also provides unit tests and one util function var_fp8 to be used in paxml to deal with the fp8_params.
The related paxml change is in here. And for the preliminary perf, we can observe more than 20% improvement over bf16 on a 8xH100 node for the GPT5B model.
Reference:

FP8 basics.

Basics of QDQ process used in our custom op.

cc. @pjannaty @reedwm

Closing as this is resubmitted in another PRs.


Heatmap support

Coverage decreased (-1.8%) to 97.811% when pulling 938bbd2116c18098eeb7fb435a31ef78f9674dfe on JorgeMartinezG:hm into 45389ca8d2c04b7c5345d440c34185b4fade06d4 on boundlessgeo:master.


read handler

hai
you have done an excellent work.thanks for that
and i would like to know how to handle a read request.
actually this is my need
i have to activate my sensor only when a user request a read characteristic
how can i do that...?

Hi @iqbalpalemad,
Thanks for the positive feedback!
What board are you using? This might be possible with nRF51 based boards only.
I'm closing this as a duplicate request, #71 is already open to track this, please follow it for updates.


memory footprint large -- can we bring it down? --

Instruments is suggesting that we're at about 800 MB memory -- at least not crashing the app w/in ten minutes -- note that this went up above 1.3 GB within 15 minutes and that led to slowly responding buttons and then to crash at fifteen minutes
Starts at 25 MB (remains there for about a minute), then to 250 MB, then 385 MB by 1.5 minutes, 480 MB by 2 minutes, 700 MB by 2.5 minutes, 830 MB by 3 minutes (and also timing point for population to be loaded); stable around 830 MB from that point for fairly long; went down to 124 MB after Daily Progress update (took longer than 5 minutes to update; repeated to check, so both times); crash (probably memory) at fifteen minutes in on this round as well
Adding on that both crashes had lldb pointers to HealthManager with either fetchSamples or fetchSamplesOfType being implicated as part of the crash -- dispatch_group_leave also comes up -- so could be related to how we are sending things off for the queue --

Put into backlog to double check and fix


Need Help when using matter adapter in a docker environment

Dear iobroker community,
Currently I am struggling with making the iobroker matter adapter reachable from outside over IPv6.
Neither using a macvlan setup nor attaching the container to a bridge network let me beeing successful in terms of exposing the container by IPv6.
Has anyone a successful running config with Linux/docker/iobroker/matter?
Would be great to get some hints or docker compose files :)
Thank you!
Oliver

Hello @schluo, I have no docker environment, so I can't answer your question how to set it up.
I think such question can be easier discussed in forum e.g. https://forum.iobroker.net/topic/78133/matter-adapter-tester-für-alpha-phase-gesucht
Did you see this hint for Docker in this chapter? https://forum.iobroker.net/topic/78133/matter-adapter-tester-für-alpha-phase-gesucht


Add support for zips with extra data embedded at the front

What steps will reproduce the problem?
1.  Create zip file
2.  Append zip file to arbitrary data (with zip at end)
3.  Try to read zip using miniz

What is the expected output? What do you see instead?

I expect this to work since the zip format can be appended to anything that
still work (IE: self extracting zips are simply zips appended to binaries)

Please provide any additional information below.

If I change the line at about 3041 where you read the `cdir_ofs` to instead
calculate it based on previous knowledge, it works.  You can even calculate the
difference between the actual offset with the declared offset to detect how
many bytes were inserted before the zip.

  // This is the old line that reads the offset form the file
  cdir_ofs = MZ_READ_LE32(pBuf + MZ_ZIP_ECDH_CDIR_OFS_OFS);

  // If you replace it with this code, it will work for embedded zips.
  cdir_ofs = cur_file_ofs - cdir_size;

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 17 Nov 2014 at 6:32

I played around with minizip and zip files appended to executables.
I could make it work by running zip -A $myexecutable on it (zip being infozip, -A is to "adjust self-extracting executable archives". "The -A option tells zip to adjust the entry offsets stored in the archive to take into account this "preamble" data."


4.0: Add if an asset pack has eyesight data in the asset downloader

As a user, I want to know if the asset pack I'm downloading has eyesight coordinates in the config (for layouts, thumbnail generation, etc.)

Added!