Fix link to react router upgrade page

Updated link in docs which to react-router upgrade page, current link is 404

Thanks @theghostbel


Adds memory-safety proofs for s2n_hmac functions

Resolved issues:
N/A.
Description of changes:
Adds CBMC proofs for the following functions in s2n_hmac module:

s2n_hmac_copy;
s2n_hmac_reset;

Call-outs:
N/A.
Testing:
N/A.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Overall these look great but would it be possible to move any asserts out of the conditional?

Thank you for the code review @adpaco-aws. Unfortunately not, these conditions can only be guaranteed upon success.

Overall these look great but would it be possible to move any asserts out of the conditional?

Thank you for the code review @adpaco-aws. Unfortunately not, these conditions can only be guaranteed upon success.


Create simple 'tutorial' environment for drop-in education classes

In order to promote and educate interested developers on egeria, there is a need to support a simple 'drop in' lab approach where a student can quickly have access to the assets they need & work throgh a workbook in relation to a particular egeria oriented scenario - perhaps around patient record assets, or maybe server configuration.
One approach is to provide the individual pieces they need - the egeria code, kafka, zookeeper, and let them work through using the code, doc, rest api using curl or postman.
However this is error prone - many moving parts to get in place before starting to learn egeria.
One possibility would be to use the helm charts we've already created. They are simple to install, but only if prereqs are met - access to a kubernetes environment, skills to use, security. We do benefit from re-use and high functionality
However kubernetes on a laptop can be problematic - for example the MacOS version currently results in significant CPU drain and battery impact - a problem for roaming users at a conference.
Creating a plain docker environment is attractive as it's a single environment to launch, but assembling a variety of components can be somewhat fragile
In between these two approaches is to use docker compose, or docker stacks -- this has far less capability than k8s, but is simple, and has much lower overhead in terms of requirements.
Further, rather than make use of docs in tools like Microsoft Word, we can use notebooks which include both text & python snippets to exercise the rest API. This too can be added to the combined environment

cc: @mandy-chessell

Current proposal is

Use docker-compose
Include jupyter notebook (lab) for supporting reading of text, launching python code against egeria
Include egeria, kafka, zookeeper

One created I will contact users to test - across Windows, Linux & MacOS

First version added to fork above

launches zookeeper, kafka, egeria, jupyter notebook lab
kafka, zookeeper use non standard ports to avoid clashes (+10000)

Still to do

pom
README
Agree final directory location
Decide way to add actual notebook content
Preconfiguration of egeria configuration

Added simple readme

If anyone would like to try this, start up the environment, but when using egeria, access it via http://egeria:8080 (not localhost), and for kafka use kafka:9082
With the notebook started you can click the ^ (up arrow) on the left hand side to upload a notebook file if you have one, or can create a new one.
Bear in mind that egeria comes up but without preconfiguration so you'll need to walk through the usual steps in the python notebook. Perfect for teaching admin ... though I'll look at flexible ways to get other assets preloaded as needed (notebooks or configuration)

I've posted my initial prototype notebook to https://gist.githubusercontent.com/planetf1/24b7c06dcb6c40b30a145f0b431200d3/raw/5c0fc3f324df2fc5d83ae4691970b82e86aab22c/EgeriaTutorial.ipynb
This will not work directly with the image here for two reasons

you'll need to change the ports & hostname
2 you'll need to initialize the egeria server (cohort etc)

but is provided as a starting template.
preloading of an egeria config is not in place.

See the following file for an example of configuring the server
This also proves basic connectivity though I've not yet checked kafka
https://gist.githubusercontent.com/planetf1/2e2b0a468bc0d32e3bf87d5b84d1a499/raw/fb46dcd3b12bf7e8845f7e7d8ff3a7d3a6ea879c/EgeriaAdmin.ipynb

I was able to run the image on windows in exactly the same way. NO changes, the notebook worked just fine, as did the sample Admin tutorial.
I didn't have docker installed, though I did previously, and Hyper-V (which docker requires) was enabled.
I did note that the download was large (though quick on my connection) with the expanded images after running the composition:
nigel@MEGA C:\Users\nigel
$ docker images
REPOSITORY                 TAG                 IMAGE ID            CREATED             SIZE
odpi/egeria                latest              5ceaeb9e0a6b        6 hours ago         208MB
bitnami/kafka              latest              a31b189faac7        9 hours ago         335MB
bitnami/zookeeper          latest              d40cd3e96ddf        14 hours ago        270MB
jupyter/minimal-notebook   latest              7233a81c81d8        9 days ago          2.67GB

In particular the jupyter notebook image is large
(though this is after being unpacked)
As a comparison, on my MacOS 'homebrew' install of jupyter, the size is 67 MB

See https://github.com/jupyter/docker-stacks/issues/775

Switched to lighter jupyter image, sufficient for demoing egeria - thanks for tip @CDaRip2U

Added 2 additional egeria services - and renamed original
So we now have

egeriacore
egeriadev
egeriadl

Docker compose does not support 'replicas' (unlike docker swarm, or effectively k8s) so the entire entry is replicated. switching to docker swarm from compose adds more complexity  which needs more evaluation
Note any existing notebooks need to be updated to use the new server names

Note that due to the way each container has it's own hostnames, the egeria servers are now at
http://egeriacore:8080 (also aliases to http://egeria:8080 for compatability with previous notebooks)
http://egeriadev:8080
http://eegeriadl:8080
as accessed from the notebook. The README.md has further info
cc: @mandy-chessell

any additional needs/bugs should be raised in new issue


[Mod Compatibility] [Enhancement] Painted Mystcraft Crystals

Not sure if possible, but very simple idea: Just like Painted Glowstone but with Mystcraft portal Crystals. The original texture is very ugly.

Wrong mod?

Good god. Sorry about that.


we should not extend the req object with the logger object but rather import it as a dependency when needed

https://github.com/betagouv/api-particulier-auth/blob/2124b782d041c3337d50f3ad7c82ceeb2d688342/api/server.js#L54

can you provide more information about th solution you think about ?


How to use Window.InnerWidth

I want to format my list so that when it is shown on mobile it will be full width.
I already tried this but without succes. Anybody know how to do this? Thanks in advance!!
{
"schema": "https://developer.microsoft.com/json-schemas/sp/view-formatting.schema.json",
"hideSelection": true,
"hideColumnHeader": true,
"hideListHeader": true,
"rowFormatter": {
"elmType": "a",
"attributes": {
"href": "=if([$URL] == '', '#', [$URL])",
"target": "=if([$NewTab] == true, '_blank', '')"
},
"style": {
"float": "left",
"width"<EMAIL_ADDRESS>118px, 95%)",
"clear": "=if(@rowIndex%3==0,'both','none')"

@thechriskent Do you know how I can apply this? Thanks!!!

 Hope that helps!


Invalid Foaled Date

When attempting to parse the foaled date for the horse associated with runner 1 in race 3 at Grafton on 2012-03-12, a ValueError (time data '-' does not match format '%d/%m/%Y') occurs in scrape_profile.

The relevant horse is https://www.punters.com.au/horses/Bonner_152493/


Swagger UI not adding Authorization header in MQTTnet.Server

Describe the bug
Swagger UI is not adding the Authorization key/value pair to header of REST API requests.
Which project is your bug related to?

MQTTnet.Server

To Reproduce
Steps to reproduce the behavior:

Run MQTTnet.Server-Portable-v3.0.12
Open Swagger UI endpoint in  browser
Click Authorize and enter some username/password
Try to execute some request
Curl is not adding Authorization to header

Expected behavior
That curl creates a request with Authorization to the header

@JanEggers I guess, this is a question for you.

nope MQTTnet.Server is preconfigured by @chkr1011 i dont know how he configured auth

I am not sure why this does not work. I tried some settings and they are also shown in the authorization menu but they still not working... If anyone knows why please let me know 😄

Changed the definition of securityScheme variable, property Scheme to lowercase "basic" and added the Reference property definition. Code snippet is part of Startup.cs.
var securityScheme = new OpenApiSecurityScheme
                {
                    Scheme = "basic",
                    Name = HeaderNames.Authorization,
                    Type = SecuritySchemeType.Http,
                    In = ParameterLocation.Header,
                    Reference = new OpenApiReference
                    {
                        Type = ReferenceType.SecurityScheme,
                        Id = "Swagger"
                    }
                };

c.AddSecurityDefinition("Swagger", securityScheme);

After that the authorization is included as part of header. Don't know the intended behavior though, authorization succeeds every time.

https://github.com/chkr1011/MQTTnet/blob/1a9460776b4d889aa78d0ea10978595f492435e3/Source/MQTTnet.Server/Web/Startup.cs#L152
looks broken to me. @chkr1011 what was your intention?

@JanEggers Yes this is intentional. When testing things locally there is no security "active". So that you must patch the Python file first by return false and checking the credentials based on your own requirements. Maybe it is a good idea to add a default password but if this is documented there is literally no password 😄

@githubjanson I added the snipped. Works perfectly 😄

Ok, great!

So...are you going to make the change?

Yes I will do.


[EC-51] Check on number of cell scan entries is incomplete

@tboinske-skyhook  - can you please take a look?


Unable to override CLI flag default for IgnoreDNSErrors from config file

While attempting to use dnsc yesterday morning, I configured the ignore_dns_errors
value in a local config.toml file and expected it to work. It did not.
Looking at the Getter method, this is the current logic:
func (c Config) IgnoreDNSErrors() bool {
  switch {

  // if not nil, a choice was made; use set choice, otherwise return default
  case c.cliConfig.IgnoreDNSErrors != nil:
    return *c.cliConfig.IgnoreDNSErrors
  case c.fileConfig.IgnoreDNSErrors != nil:
    return *c.fileConfig.IgnoreDNSErrors
  default:
    return defaultIgnoreDNSErrors
  }
}

We have the NewBaseConfig() constructor defined like so:
func NewBaseConfig() Config {

  // we have to explicitly initialize our IgnoreDNSErrors pointer field
  // to prevent "invalid memory address or nil pointer deference" panics
  return Config{
    cliConfig: configTemplate{
      IgnoreDNSErrors: new(bool),
    },
    fileConfig: configTemplate{
      IgnoreDNSErrors: new(bool),
    },
  }
}

which ensures that we will not have a nil pointer for either nested struct. To further complicate things, a default value is set via flag package
here:
  flag.BoolVar(c.cliConfig.IgnoreDNSErrors, "ignore-dns-errors", defaultIgnoreDNSErrors, ignoreDNSErrorsFlagHelp)
  flag.BoolVar(c.cliConfig.IgnoreDNSErrors, "ide", defaultIgnoreDNSErrors, ignoreDNSErrorsFlagHelp+" (shorthand)")

This causes the first check in the IgnoreDNSErrors() Getter method to always succeed (since c.cliConfig.IgnoreDNSErrors is not nil):
  case c.cliConfig.IgnoreDNSErrors != nil:
    return *c.cliConfig.IgnoreDNSErrors

This does not work (nil cannot be assigned to *bool):
  flag.BoolVar(c.cliConfig.IgnoreDNSErrors, "ignore-dns-errors", defaultIgnoreDNSErrors, ignoreDNSErrorsFlagHelp)
  flag.BoolVar(c.cliConfig.IgnoreDNSErrors, "ide", defaultIgnoreDNSErrors, ignoreDNSErrorsFlagHelp+" (shorthand)")

and this does not work (nil cannot be assigned to *bool):
  c.cliConfig.IgnoreDNSErrors = flag.Bool("ignore-dns-errors", nil, ignoreDNSErrorsFlagHelp)
  c.cliConfig.IgnoreDNSErrors = flag.Bool("ide", nil, ignoreDNSErrorsFlagHelp+" (shorthand)")

From what I can tell, it does not seem that you can have nil as a default value for flags using the standard library package.
I thought one workaround might be to make the default variable referenced by the flag package for the ignore-dns-errors flag name a pointer and perform pointer equality checks in the Getter method. I assumed that this would be the least invasive option, but after brief testing I still found that the compiler would (understandably) object to mixed types. This workaround didn't pan out.
Using the alexflint/go-arg package, I believe that you you can have nil flag values (no forced default value), so that may be one option to resolve this; this approach would be similar to the configuration setup that I used for my first Go project, atc0005/elbow.
References:

https://stackoverflow.com/questions/18429632/flag-package-in-go-do-i-have-to-always-set-default-value

Solved by the work on GH-71, GH-43.


Offer dynamic storage types in chargeback report

@miq-bot assign @gtanzillo
@miq-bot add_label enhancement, chargeback

@gtanzillo updated and added test 👍


[1726] Remove redundant MCB rollover code

Context
We have a new rollover service. This PR removes the redundant MCB code.
Checklist

[ ] Make sure all information from the Trello card is in here
[ ] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally

 Kill the MCB rollover code 🔪🩸⚰🪦

We can remove all of MCB now. This was the only function we were dependant on. I do everything via the console anyway cc. @gpeng


mio does not accept C++17 std::filesystem::path's for opening mapped files

A small improvement suggestion I have is that mio does not accept an object of type std::filesystem::path for mapped file paths:
const std::filesystem::path file_path = "...";
mio::mmap_source mapped_file(file_path); // Will not compile

The filesystem library was added since C++17.
Of course, you could just write the following right now and be all set:
const std::filesystem::path file_path = "...";
mio::mmap_source mapped_file(file_path.string());

But proper std::filesystem::path argument support would be nicer overall.
The README says that this is a C++11 library but moving forward and upgrading it to a C++17 library wouldn't hurt, would it?

I agree. If I am not mistaken this would also simplify support of unicode characters in the file path under Windows drastically.
Under C++17, a path can be created from an utf8-encoded string by using
std::filesystem::path myPath = std::filesystem::u8path(u8"äßçdé.txt").
All that is needed then is one single line:
CreateFileW(myPath.wstring().c_str(), ... // use unicode version of fct'CreateFileW' and pass a wide character string


Allow proper mapping of tenant hostnames to URLs

Opencast has some assumptions about how hosts are accessed in different
organizations. Especially the schema and port number are assumed to be
equal or in certain cases just the default protocol port number.
This changes the host mapping configuration to explicitly specify a URL.
Your pull request should…

[x] have a concise title
[x] close an accompanying issue if one exists
[x] be against the correct branch (features can only go into develop)
[x] include migration scripts and documentation, if appropriate
[x] pass automated tests
[x] have a clean commit history
[x] have proper commit messages (title and body) for all commits

@mtneug, can you add a release notes snippet for this?
See https://github.com/opencast/opencast/blob/develop/docs/guides/admin/docs/releasenotes/readme.txt


[Service Bus] Entity creation and deletion using @azure/arm-servicebus

Issue
https://github.com/Azure/azure-sdk-for-js/issues/1146
Description
Leveraging @azure/arm-servicebus to automate the creation and deletion of service bus entities.
Go through the following setup before executing any tests.
Setup
Register a new application in AAD

Follow the Docs and register a new application in the Azure Active Directory(in the azure-portal).
Note down the CLIENT_ID and TENANT_ID.
Create a SECRET(in the Certificates & Secrets tab) and note that down.

Assign owner role to the registered application

In the azure-portal, go to your service-bus app service and assign owner role to the registered application.
This can be done from Access control (IAM) tab(in the left-side-navbar of your service-bus-namespace in the azure-portal)
Doing this would allow the registered application manage the namespace, i.e., entity creation, deletion, etc.,

Update your environment variables

Populate the following variables in the .env.

ARM_SERVICEBUS_CLIENT_ID=""
ARM_SERVICEBUS_SECRET=""
ARM_SERVICEBUS_TENANT_ID=""
RESOURCE_GROUP=""
SERVICEBUS_NAMESPACE=""
AZURE_SUBSCRIPTION_ID=""

Note
RESOURCE_GROUP, SERVICEBUS_NAMESPACE and AZURE_SUBSCRIPTION_ID can be found at your respective servicebus-namespaces in the azure-portal.

https://github.com/Azure/azure-sdk-for-js/pull/1164#discussion_r255287513
@ramya-rao-a @AlexGhiondea
Added CLEAN_NAMESPACE env variable and used it in testUtils.
If this variable is undefined, then the deletion/creation thing won't happen.
In order to use the deletion/creation in testUtils, CLEAN_NAMESPACE must be populated by some value.
Any comments on this ?
Any suggestions on the name of the variable ? (CLEAN_NAMESPACE / CLEAR_ENTITIES ??)

Shouldn't we be updating renewlock.spec.ts file as well?

@ramya-rao-a
Yes, doing it now!

This great @HarshaNalluru!
I really like the fact that by default, the tests would run without re-creating the resources. Makes it easier for running local tests.


feat(kpi severity): Add severity for kpi

:tada: This PR is included in version 1.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Crash when passing undefined identifier GS [instance(...)] attribute

I moved all the CSM depth-pass shaders to a single file.
If only the VS and PS are in the file, and the GS is in a separate file, it works. Once I move the GS into the same file, Slang crashes.
To reproduce: Falcor, branch single-shader-file-csm, run the Shadows sample.

Looking into it now.
Note that Conor Lavelle contacted me about a similar issue where he got crashes in Slang because the file contained a line like [instance(CASCADE_COUNT)] on the GS entry point, but CASCADE_COUNT was only being defined for some program compiles and not others. A simple workaround was to conditionally define CASCADE_COUNT to one if it wasn't already defined.
Of course, Slang shouldn't be crashing in cases like that anyway, and we may need to place some more serious restrictions where entry-point attributes don't get validated unless the given entry point has actually been requested for compilation.

I've confirmed that this is the same underlying issue that Connor ran into.
I'm working for a fix for the crash in Slang, after which you get an "undefined identifier" error for the name you are passing to the [instance(...)] attribute on the GS. This behavior matches fxc and dxc in teh same case.
The right answer is to use the preprocessor to define the instance count macro inside the shader, if it hasn't been passed in from outside.

I will of course fix the Slang crash.


Add command to sync metadata to all supported modules

See https://github.com/silverstripe/silverstripe-framework/issues/8343

Script features

Github labels (see existing script)
Contributing.md file (see framework example)
Issue and pull request templates (see framework example)
LICENSE

I think it makes sense for these commands to live in cow

Now covered in https://github.com/silverstripe/cow/pull/114

This is merged now, we can add more as needed


Can JCL language id be changed to lower case "jcl"

Hi,
I'm writing an extension that contributes "JCL" snippets.
"contributes": {
		"snippets": [
			{
				"language": "jcl",
				"path": "./snippets/snippets.json"
			}
		]
	},

I've noticed that this extension defines JCL with language id "JCL".
Whilst a number of other VS code extensions use the id "jcl" ( lower case) .
The "VS Code" docs suggest it is a good idea to use "lower case" for langauge ids.
...

New identifier guidelines#
When defining a new language identifier, use the following guidelines:

Use the lowercased programming language name.

...

It'd be great if this extension could use a lower case id "jcl";
That way my snippets could work with multiple vs code extensions.
Thanks for considering this feature request.
Travis

The extension already has a lower case alias for the language, so your snippets should be picked up.
            {
                "id": "JCL",
                "aliases": [
                    "JCL",
                    "jcl"
                ],
                "extensions": [
                    ".jcl",
                    ".job",
                    ".cntl",
                    ".prc",
                    ".proc"
                ],
                "configuration": "./jcl.configuration.json"
            },

Hi,
Thanks for the quick reply.
I get this "unknown language 'jcl'" error when I open a JCL file:

My package json looks like:
 "contributes": {
        "snippets": [
            {
                "language": "jcl",
                "path": "./snippets/snippets.code-snippets"
            }
        ]
    }

So it looks like  "contributes.snippets" uses the "id" of the language rather than alias?
This page ( https://code.visualstudio.com/docs/languages/identifiers ) says;

"Note that casing matters for exact identifier matching ('Markdown' != 'markdown')"

Ideally, I agree it would be nice to change but then I end up breaking one that uses it in uppercase.
I might be wrong but I think you can add more than one language id separated by a comma.
 "contributes": {
        "snippets": [
            {
                "language": "jcl,JCL",
                "path": "./snippets/snippets.code-snippets"
            }
        ]
    }

Thanks


Ignore Email Addresses

In this PR, WebTrends was added to<EMAIL_ADDRESS>https://github.com/roryp/devopsdemo/pull/5/commits/3e499aaa1822214a0cbf562b6f29734d34c6be95#diff-04c6e90faac2675aa89e2176d2eec7d8R34

Updated the RegEx query to require https, http or ftp: https://github.com/brminnick/GitHubReadmeWebTrends/blob/e4ff2ba5d750100bb82cdabdbc6c0fbf0af350d9/GitHubReadmeWebTrends.Functions/Functions/VerifyWebTrendsFunction.cs#L26


Using  with useGoogleLogin() opening authentication popup twice on click of sign in button

Hello,
Using  with useGoogleLogin() opening authentication popup twice on click of sign in button. Please let me know if there is any workaround on it.
Regards,
Sumit Awasthi

You shouldn't use them together.


Node.js default example fails due to dependency errors despite running in a default Docker image

Describe the issue
id: outputs_nodejs
namespace: company.team

description: This flow will install the npm package in a Docker container, and use kestra's NodeJS Commands task to run the script.

tasks:
  - id: run_nodejs_commands
    type: io.kestra.plugin.scripts.node.Commands
    inputFiles:
      output_nodejs.js: |
        const requestify = require('requestify');
        const Kestra = require('@kestra-io/libs');

        function GetDockerImageDownloads(imageName){
          // Queries the Docker Hub API to get the number of downloads for a specific Docker image.
          var url = `https://hub.docker.com/v2/repositories/${imageName}/`
          console.log(url)
          requestify.get(url)
            .then(function(response) {
              result = JSON.parse(response.body);
              Kestra.outputs({"pull_count": result['pull_count']})
              return result['pull_count'];
            })
            .catch(function(error) {
              console.log(error);
            })
        }

        GetDockerImageDownloads("kestra/kestra")
    taskRunner:
      type: io.kestra.plugin.scripts.runner.docker.Docker
    containerImage: node:slim
    warningOnStdErr: false
    beforeCommands:
      - npm install requestify
      - npm install @kestra-io/libs
    commands:
      - node outputs_nodejs.js

  - id: log_downloads
    type: io.kestra.plugin.core.log.Log
    message: "Number of downloads: {{ outputs.run_nodejs_commands.vars.pull_count }}"

From https://kestra.io/docs/how-to-guides/javascript#variable-output
Environment

Kestra Version: develop

Related: https://github.com/kestra-io/libs/issues/9

we should check with the latest version of node.js

I tried this with 22.12.0 and 23.4.0 and both have the same error. Looking online, it seems that the ReferenceError: module is not defined could be down to the runtime environment expecting ES Modules but the file wants CommonJS. Could be related to the introduction of this to the package.json
https://github.com/kestra-io/libs/commit/c7e0e74cb3ad4f6b5b37106854119816545ab629#diff-32d0752e48631d28147a6d14525fd9e8bdc58f81823a5d476ded9633b3d7c220R21


numeric Alias: unmaskAsNumber

Hi!
it's more a feature request than a bug. unmaskAsNumber works as expected to reformat german number format to JS-Format.
With this option enabled unmask returns the value of the input field casted as number. This breaks the default behaviour of $('input[type="text"]').val() as this usually returns a string.
The "problem" I'm facing is that some libs are expecting a string (e.g. jQuery-form). Since they are doing some string operation on the fields - they crash (e.g. Uncaught TypeError: e(...).val(...).replace is not a function).
I think it shouldn't be inputmasks job to cast values, but I understand if you don't want to break current behaviour.
So I vote for a (weird sounding) option like "unmaskAsNumberString":
... (opts.unmaskAsNumberString ? processValue : Number(processValue) ) ) : processValue; ...
best regards and many thanks for your great work!
Arne

Sorry duplicate to #1288


App Settings Colon Delimiter Not Allowed

Hi,
In my appsettings.json file I have settings like this:
  "AzureStorageSettings": {
    "ConnectionString": "",
    "TableName": "Foo"
  }

I want to override the value of AzureStorageSettings:ConnectionString configuration with a secret stored in Azure Devops. In the past, I used to simply add -AzureStorageSettings:ConnectionString $(AzureStorageConnectionString) to the App settings section of the task and that worked perfectly.
However, with version 4, I get an error saying that colon character is not allowed. Is there a new way of achieving this or a way that I can espace the colon character? I tried everything that I can think of.

The solution is to use __ instead of : in App settings. The app settings get added as environment variables and Microsoft.Extensions.Configuration can handle __ for nesting configs.


Support for custom flags after API change

This issue was originally about implementing support for /mlp/ "flags", but it appears custom flags as a whole on 4chan have changed in the API, now being under "board_flags" rather than "troll_flags" and so the 4chan extension needs to be changed.

Should work after extension update. You'll have to reload the boards list to refresh configuration.


Added device ids (af00, af01) in fpga_devices

Signed-off-by: Sivaramakrishnan, VenkataramananX<EMAIL_ADDRESS>

Coverage remained the same at 65.911% when pulling a4f7b08180ffe7f92a69ecb298c029d5291772ae on vsivar1x/opae_io_fix into 08486768ac7ff88bd548604093c7039363f65c46 on master.


Jest crashes if snapshot folder gets removed

🐛 Bug Report
If you're using snapshots and remove the folder while jest is idle (Using it with the --watchwatchAll flag) the process crashes. Not sure if this is related to jest though.
To Reproduce
Use jest with the --watchAll  flag and remove the __snapshots__ folder after a test is done.
Expected behavior
It could just tell me that the __snapshots__ is not longer available and recreate it.
Link to repo
Here you go: jest-error-snapshots-folder-removed.
It fails here. I've used fs-extra but you can remove it manually too via your file-explorer.
envinfo
    OS: Windows 10 10.0.18362
    CPU: (16) x64 AMD Ryzen 7 1700X Eight-Core Processor
  Binaries:
    Node: 10.16.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.17.3 - C:\Users\root\AppData\Roaming\npm\yarn.CMD
    npm: 6.13.1 - C:\Program Files\nodejs\npm.CMD
  npmPackages:
    jest: ^24.9.0 => 24.9.0

Seems to work in properly in my Mac
   OS: macOS Mojave 10.14.6
  Binaries:
    Node: 10.16.0
    Yarn: 1.13.0
  npmPackages:
    jest: 24.9.0

Issue seems to be only in Windows. Ran it in a Windows 10 VM and got this
Watch Usage: Press w to show more.events.js:187
      throw er; // Unhandled 'error' event
      ^

Error: EPERM: operation not permitted, lstat 'C:\cygwin64\home\<user>\jest-error-snapsho
ts-folder-removed\test\__snapshots__\index.test.js.snap'
Emitted 'error' event on NodeWatcher instance at:
    at NodeWatcher.<anonymous> (C:\cygwin64\home\<user>\jest-error-snapshots-folder-remo
ved\node_modules\sane\src\node_watcher.js:291:16)
    at callback (C:\cygwin64\home\<user>\jest-error-snapshots-folder-removed\node_module
s\graceful-fs\polyfills.js:295:20)
    at FSReqCallback.oncomplete (fs.js:158:21) {
  errno: -4048,
  code: 'EPERM',
  syscall: 'lstat',
  path: 'C:\\cygwin64\\home\\<user>\\jest-error-snapshots-folder-removed\\test\\__snapsh
ots__\\index.test.js.snap'
}

@M4rk9696 Yep, that's exactly the error I get. Should've posted it below my question...


[BEAM-9496] Add to_dataframe and to_pcollection APIs.

This may be a more natural interface for mixing pandas-style code with Beam.
It does have the downside of being inefficient if dataframes are converted back to PCollections one at a time.
Builds on https://github.com/apache/beam/pull/10760

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Run Python PreCommit

@TheNeuralBit PTAL

Run Python PreCommit


Remove isapprox for intervals which has been moved to IntervalSets

It seems the tests failed due to an isapprox check. Since isapprox moved, perhaps we can also remove the test from DomainSets, tag a new release, and worry about the error being thrown elsewhere?


Add files via upload

@urimsft
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

The article folder partner-center-portal no longer exists in the main branch. I'll close this PR, but if you still want to make updates to a current article, feel free to open a new PR.


Set recognition model and vendor by metadata

Currently the selection of vendor and model is defined by assetDomain and assetPath.
Case 1: Photographer uploads images with category "Celebrity". We would like to use the info and send the images to the corresponding model.
Case 2: Files archived from Enterprise contain an editorial category. We might want to use sceCategory for setting the model
Case 3: To reduce cost it would be convenient if we could avoid sending agency images to recognition - !usageRights:agency
All cases can be solved by the ability to map queries to model and vendor.

Interesting idea... but could be a bit tricky to implement. Question is how far you want to go with the queries? Identical to everything that's possible in Elvis?
The tricky part is that the image recognition integration doesn't have knowledge of the Elvis query language (think about interpreting something like: "category:Celebrity AND (status:final OR licenseStartDate:[2018-05-01 TO 2018-12-31])". This is something Elasticsearch (Lucene) can interpret inside Elvis. I now see two options if we want to implement queries like these in this integration:
Option A: use the Elvis search API to perform a queries on the given asset. Downside of this option is that you'd have to perform separate queries for every mapping you have. This might put a heavy API load on Elvis.
Option B: Create logic in the integration to interpret the query and make the selection locally. This would be very nice, but is probably really complex to implement (you'd have to re-invent Lucene query parsing and searching...).
Both options are far from ideal. An another alternative might be to support less features in the query, for example a subset of the options you'd have in Lucene or just a single metadata=value match. Ideally I would opt for something where the integration can match the right model without too much additional logic and without extra API calls towards Elvis.

Keeping this integration focused on recognition is key. Perhaps combine two webhooks?
The first will react on asset_create and other events, have the logic required to set the model (model field per vendor?), and flag the record for being picked up using a boolean field cf_aiRecognize. A sample for doing this would be welcome.
The second webhook could then react if cf_aiRecognize is changed to true and use the model set by the first webhook to call this integration.


Merged two run commands into one in Dockerfile

Why? To save a layer?
@squaremo thoughts?

@rade Yes, i guess this is what Dockerfile's best practices also says.


package.json depends on package outside of npm

I'm working on build machine that doesn't allow access to github.com to ensure that all the code is in our local git repository, and npm repository.  When I tried to access this package using
npm install -g grommet-cli
I kept getting an error saying (and I'm paraphrasing)
Unable to download https://github.com/alansouzati/vorpal/tarball/EXIT_ON_PARSE

I tracked this down to the grommet-cli/package.json file containing a dependency for
vorpal which requires that file.
This is a request to publish the EXIT_ON_PARSE package to the normal npm repository so it will install normally without requesting a download from github.com.
Thank you,
Patrick

yeah I'm afraid I dont have a solution for this problem. We got a pull request open to add this feature to Vorpal (the tool behind our CLI) and the project seems to be "abandoned".
Here is the PR:
https://github.com/dthree/vorpal/pull/169
We will re-evaluate the usage of vorpal in the future. But for now, unfortunately we need to use that branch.

I understand.  Thanks you for the consideration and quick response to the issue.
Please feel free to re-open it if it's appropriate.


matcha-gtk-theme: 2022-06-07 -> 2022-11-15

Description of changes

matcha-gtk-theme: 2022-06-07 -> 2022-11-15
matcha-gtk-theme: replace duplicate files with symlinks
matcha-gtk-theme: add arguments to select color and theme variants

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@ofborg eval


QueryBuilder like condition without enclosing %?

we are using the querybuilder to build a simple like condition.
$names = self::find()->where(['user_id' => $user->id])->andWhere(['like','item_name','ADMIN_'])->select('item_name')->column();

the generated sql is:
SELECT `item_name` FROM `t_auth_assignment` WHERE (`user_id`=1) AND (`item_name` LIKE '%ADMIN\\_%')

in: https://github.com/yiisoft/yii2/blob/master/docs/guide/db-query-builder.md
i cant find an option how to choose about the enclosing.
e.g.
SELECT `item_name` FROM `t_auth_assignment` WHERE (`user_id`=1) AND (`item_name` LIKE 'ADMIN\\_%')

or:
SELECT `item_name` FROM `t_auth_assignment` WHERE (`user_id`=1) AND (`item_name` LIKE '%ADMIN\\_')

its very common that you dont want to use %search%
Is it undocumented or simply not implemented?

@qiangxue
$result = $this->find()->where(['like','slug','is%', false])->select(['slug'])->column();
Not working.
Please suugest.


Adds srun functionality

Uses pytest-xdist for parallel testing and srun for distribution.

:green_circle: balfrin/unit_test/summary.log

:green_circle: tsa/integration_test/spack_info_cosmo-dycore.log
:green_circle: tsa/integration_test/spack_info_cosmo.log
:green_circle: tsa/integration_test/spack_info_int2lm.log
:green_circle: tsa/integration_test/spack_spec_cosmo-dycore.log
:green_circle: tsa/integration_test/spack_spec_cosmo-dycore_+cuda_+gt1_+build_tests.log
:green_circle: tsa/integration_test/spack_spec_cosmo-dycore_~cuda_+gt1.log
:green_circle: tsa/integration_test/spack_spec_cosmo.log
:green_circle: tsa/integration_test/spack_spec_cosmo_cosmo_target=gpu_+serialize_+eccodes_+claw_+zlib_ng_+oasis.log
:green_circle: tsa/integration_test/spack_spec_cosmo_cosmo_target=gpu_~cppdycore.log
:green_circle: tsa/integration_test/spack_spec_cosmo_~eccodes.log
:green_circle: tsa/integration_test/spack_spec_int2lm.log
:green_circle: tsa/integration_test/spack_spec_int2lm_+eccodes_+parallel.log
:green_circle: tsa/integration_test/spack_spec_int2lm_~eccodes.log
:green_circle<EMAIL_ADDRESS>:green_circle<EMAIL_ADDRESS>:green_circle<EMAIL_ADDRESS>:green_circle<EMAIL_ADDRESS>:red_circle<EMAIL_ADDRESS>:yellow_circle<EMAIL_ADDRESS>AssertionError exception when releasing read lock
:red_circle<EMAIL_ADDRESS>:red_circle<EMAIL_ADDRESS>:red_circle<EMAIL_ADDRESS>:yellow_circle<EMAIL_ADDRESS>AssertionError exception when releasing read lock
:green_circle<EMAIL_ADDRESS>:green_circle<EMAIL_ADDRESS>:green_circle<EMAIL_ADDRESS>:green_circle<EMAIL_ADDRESS>:green_circle: tsa/unit_test/summary.log

:green_circle: daint/integration_test/spack_info_cosmo-dycore.log
:green_circle: daint/integration_test/spack_info_cosmo.log
:green_circle: daint/integration_test/spack_info_int2lm.log
:green_circle: daint/integration_test/spack_spec_cosmo-dycore.log
:green_circle: daint/integration_test/spack_spec_cosmo-dycore_+cuda_+gt1_+build_tests.log
:green_circle: daint/integration_test/spack_spec_cosmo-dycore_~cuda_+gt1.log
:green_circle: daint/integration_test/spack_spec_cosmo.log
:green_circle: daint/integration_test/spack_spec_cosmo_cosmo_target=gpu_+serialize_+eccodes_+claw_+zlib_ng_+oasis.log
:green_circle: daint/integration_test/spack_spec_cosmo_cosmo_target=gpu_~cppdycore.log
:green_circle: daint/integration_test/spack_spec_cosmo_~eccodes.log
:green_circle: daint/integration_test/spack_spec_int2lm.log
:green_circle: daint/integration_test/spack_spec_int2lm_+eccodes_+parallel.log
:green_circle: daint/integration_test/spack_spec_int2lm_~eccodes.log
:green_circle<EMAIL_ADDRESS>:green_circle<EMAIL_ADDRESS>:yellow_circle<EMAIL_ADDRESS>Timed out waiting for a write lock
:yellow_circle<EMAIL_ADDRESS>Timed out waiting for a write lock
:red_circle<EMAIL_ADDRESS>:red_circle<EMAIL_ADDRESS>:green_circle<EMAIL_ADDRESS>:green_circle<EMAIL_ADDRESS>:green_circle<EMAIL_ADDRESS>:red_circle<EMAIL_ADDRESS>:green_circle<EMAIL_ADDRESS>:yellow_circle<EMAIL_ADDRESS>Timed out waiting for a write lock
:green_circle<EMAIL_ADDRESS>:green_circle<EMAIL_ADDRESS>:green_circle: daint/unit_test/summary.log

:green_circle: tsa/integration_test/spack_info_cosmo-dycore.log
:green_circle: tsa/integration_test/spack_info_cosmo.log
:green_circle: tsa/integration_test/spack_info_int2lm.log
:green_circle: tsa/integration_test/spack_spec_cosmo-dycore.log
:green_circle: tsa/integration_test/spack_spec_cosmo-dycore_+cuda_+gt1_+build_tests.log
:green_circle: tsa/integration_test/spack_spec_cosmo-dycore_~cuda_+gt1.log
:green_circle: tsa/integration_test/spack_spec_cosmo.log
:green_circle: tsa/integration_test/spack_spec_cosmo_cosmo_target=gpu_+serialize_+eccodes_+claw_+zlib_ng_+oasis.log
:green_circle: tsa/integration_test/spack_spec_cosmo_cosmo_target=gpu_~cppdycore.log
:green_circle: tsa/integration_test/spack_spec_cosmo_~eccodes.log
:green_circle: tsa/integration_test/spack_spec_int2lm.log
:green_circle: tsa/integration_test/spack_spec_int2lm_+eccodes_+parallel.log
:green_circle: tsa/integration_test/spack_spec_int2lm_~eccodes.log
:yellow_circle<EMAIL_ADDRESS>AssertionError exception when releasing read lock
:red_circle<EMAIL_ADDRESS>:red_circle<EMAIL_ADDRESS>:red_circle<EMAIL_ADDRESS>:red_circle<EMAIL_ADDRESS>:red_circle<EMAIL_ADDRESS>:green_circle: tsa/unit_test/summary.log

launch jenkins int2lm daint

:green_circle: tsa/unit_test/summary.log

:green_circle: balfrin/unit_test/summary.log

:green_circle: daint/integration_test/spack_info_int2lm.log
:green_circle: daint/integration_test/spack_spec_int2lm.log
:green_circle: daint/integration_test/spack_spec_int2lm_+eccodes_+parallel.log
:green_circle: daint/integration_test/spack_spec_int2lm_~eccodes.log
:green_circle<EMAIL_ADDRESS>:green_circle<EMAIL_ADDRESS>:green_circle: daint/unit_test/summary.log

launch jenkins tsa daint int2lm

:green_circle: balfrin/unit_test/summary.log

:green_circle: tsa/integration_test/spack_info_int2lm.log
:green_circle: tsa/integration_test/spack_spec_int2lm.log
:green_circle: tsa/integration_test/spack_spec_int2lm_+eccodes_+parallel.log
:green_circle: tsa/integration_test/spack_spec_int2lm_~eccodes.log
:green_circle<EMAIL_ADDRESS>:green_circle<EMAIL_ADDRESS>:green_circle: tsa/unit_test/summary.log

:green_circle: daint/integration_test/spack_info_int2lm.log
:green_circle: daint/integration_test/spack_spec_int2lm.log
:green_circle: daint/integration_test/spack_spec_int2lm_+eccodes_+parallel.log
:green_circle: daint/integration_test/spack_spec_int2lm_~eccodes.log
:green_circle<EMAIL_ADDRESS>:green_circle<EMAIL_ADDRESS>:green_circle: daint/unit_test/summary.log

Did you check if the yellow-circle works?

Yes, here: https://github.com/C2SM/spack-c2sm/pull/591
And I discovered that using pytest-xdist without srun is apparently not such a good idea. It seems to me that spack is not completely thread-safe.


[L15] Add error messages to require statements in Flipper

There's more of these to address in the openzeppelin-audit branch or DanielVF/resolutionUpgrade but not sure how we'll proceed with merging that code. We'll have to remember it in the review for whatever gets merged.
Contract change checklist:

[x] Code reviewed by 2 reviewers. See template and example.
[x] Unit tests pass
[x] Slither tests pass with no warning
[ ] Echidna tests pass if PR includes changes to OUSD contract (not automated, run manually on local)

I just make a new row in the spreadsheet for the separate contract groups that a single issue applies to and mark them off one by one.
LGTM.

Changes look good, but we should get tests working correctly before merging.


feat: download pdf in attachment folder and expose a variable {{{pdfAttachment}}} in the template

#13


How to add Scrolling support for the fragments ?

Hi,
I tried to add the scrolling support to the fragments, but the view pager is not scolling the fragment i have even tried by adding the following event for the pager.
pager.setOnTouchListener(new View.OnTouchListener() {
    @Override
        public boolean onTouch(View v, MotionEvent event) {
            v.getParent().requestDisallowInterceptTouchEvent(true);
            return false;
        }
    });

But no luck, any leads will be helpful

Having a scrolling view inside another view that scrolls as well usually does not work out too well unless a few things are done beforehand.
Please give this a read for a possible pointer.

Closed, please reopen if your issue has not been resolved.


forwarding compile test succeeded when it should have failed

I'm getting the following on cargo t:
failures:

---- src\compile_test.rs - compile_test::ForwardWithoutCode (line 113) stdout ----
Test compiled successfully, but it's marked `compile_fail`.
---- src\compile_test.rs - compile_test::ForwardWithoutCode (line 98) stdout ----
Test compiled successfully, but it's marked `compile_fail`.

failures:
    src\compile_test.rs - compile_test::ForwardWithoutCode (line 113)
    src\compile_test.rs - compile_test::ForwardWithoutCode (line 98)

@cormacrelf does this make any sense to you? Did your more recent changes make this issue moot?

Ah, no, that's because diagnostic(code) is now optional regardless. Those tests were written with code being mandatory in mind. You can delete 'em

done. :)


[Enhancement] Manual start gradle check is required to merge anything

Describe the bug
Before PR's are allowed to be merged the OpenSearch repository enforces running a CI task, ./gradlew check, which runs full unit, integration, smoke, and varying cluster tests. This takes a very long time and consumes a lot of resources. To avoid becoming a bottleneck we introduced a manual start gradle check that a maintainer can kick off by adding a comment to a PR once a basic review of a PR has been made.  This is a manual step.
Expected behavior
No manual interaction from a maintainer other than approving/merging a PR.
Screenshots

Additional context

I'm not Peter, but no time like the present to work things out :)

@peterzhuamazon I'm OK pulling the trigger on this if you are, any concerns?

I do have concerns since this will autotrigger on EVERY COMMIT, not EVERY PR.


Adding sidebar issue

I am new to Laravel and and I am using ACL package.I have successfully installed it but I can add items in the sidebar by reading documentation.Can you please give me example from which I can easily understand that how to add items in the sidebar??Reply please...
By the way the package is really awesome.

Hi,
if you want to have the sidebar menu globally do this in your BaseController:
/**
 * Setup the layout used by the controller.
 *
 * @return void
 */
protected function setupLayout() {

	$sidebar = array(
		"Dashboard"           => array(
			'url'         => URL::route( 'index' ),
			'icon'        => '<i class="fa fa-users"></i>'
		), // ... and the rest of the items
	);

            // 'share' the variable over all the requested templates
	View::share( 'sidebar_items', $sidebar );
    }


"Scroller" doesn't have the icon in the toolbox

Describe the bug
"Scroller" doesn't have the icon in the toolbox
Steps to reproduce the behavior
1.Navigate to https://arcade.makecode.com/--skillmap#docs:/skillmap/space
2.Create a Space Adventure Game---->START
Actual behavior
"Scroller" doesn't have the icon in the toolbox

Additional context
1.OS: Windows(rs6)
2.arcade version: 1.6.24
3.Microsoft MakeCode version: 7.2.25

Fixed with https://github.com/microsoft/pxt-arcade/pull/4199


EarthCube Product Registry

I met with Stephen Richard from the EarthCube resource registry & USGIN to discuss potential synergy with EarthCube's efforts to catalog tools & tool metadata with regards to our #4. My dream would be to register our tools with the EarthCube registry and then create a pretty view using their backend or some RESTful frontend they would provide.
Regarding registering: we should probably do that anyway. Here is the EarthCube Tool Registry Form. Stephen was able to look at the seascapeR package and add it relatively easily. The resulting record can be viewed in the Suave tool browser interface, just search for "seascape". After another harvesting/processing step is complete the record will be viewable in the GEOCODES tool browser. This tool browser will soon be upgraded to something like the new GEOCODES interface (currently beta).
My quick notes about way this all works:

registry form goes to spreadsheet
data gets picked up and turned into JSON-LD (based on Schema.org?).
data is put into a triplestore database so SPARQL queries can be run
other harvesters (like google datasets) can pick up the data published as JSON-LD using schema.org standards.

In Summary

It would be ideal if we could leverage this existing infrastructure but a lot of work will need to be done.
Stephen's group may be able to provide us with more features as they continue development. So this is worth keeping an eye on.
At a minimum we should consider the existing tool metadata work here
we should register our tools in the EC Registry either way

Nice! Here's what I'm seeing on desktop in Suave tool browser interface:

That's a good idea to borrow from fields there:
seascapeR
seascapeR
Resource type: Software
The seascapeR package provides functions for fetching, analyzing and visualizing Seascapes, a global and regional dynamic sea water classification product derived from satellite imagery by Maria Kavanaugh (OSU). MBON Seascapes identifies spatially ex...
links: https://marinebon.org/seascapeR/
Type of Resource
Software
Name
seascapeR
Description
The seascapeR package provides functions for fetching, analyzing and visualizing Seascapes, a global and regional dynamic sea water classification product derived from satellite imagery by Maria Kavanaugh (OSU). MBON Seascapes identifies spatially explicit water masses with particular biogeochemical features using a model and satellite-derived measurements. Marine Biodiversity Observation Network (MBON) is a global initiative to integrate data to improve understanding of changes and connections between marine biodiversity and ecosystem functions. In particular, this package is meant to help nodes in the MarineBON.org network extract Seascape data, especially across NOAA Sanctuaries, for comparison with biological data from eDNA, sound, telemetry and other observational data to evaluate how dynamic water masses relate to ecosystem function.
Citation
Best, Ben, Herrera, Enrique Montes, and Kavanaugh, Maria, 2021, SeascapeR,
Responsible Party
creator, person, Ben Best,
creator, person, Enrique Montes Herrera
creator, person, Maria Kavanaugh,
Readable Page
https://marinebon.org/seascapeR/
Keywords
oceanography, sea water mass classification
Maturity/status
In production
License
MIT
Target Audience
Data Users, Scientists, Members of the Public
Expected Lifetime
Unknown
Usage
Unknown
Related Resources
Seascapes classification scheme,
Science Domain
Biological Oceanography, Chemical Oceanography, Environmental Science, Marine Geology
Funding Source
US National Oceanic and Atmospheric Administration (NOAA)
Programming Languages
R
Code Repository URL
https://github.com/marinebon/seascapeR
Input format
text/plain;application=OpenDAPv2-DDS
Runtime Environment
R studio
Installer Package
In R Studio remotes::install_github('marinebon/seascapeR')
Stewardship Name
Ben Best,
Stewardship Type
Person
Registry Contributor
Stephen M. Richard
ORCID of the Contributor
https://orcid.org/0000-0001-6041-5302
Email Address
<EMAIL_ADDRESS>Timestamp
Fri Apr 02 2021 13:11:29 GMT-0700 (Mountain Standard Time)

Also, there's  more on how to enable dataset discovery here:  Dataset  |  Search Central  |  Google Developers


Imagenet bugfix: validation dataset

In case of validation, indexes are going through the whole dataset and not the just over validation subset.

good catch!


Unable to get it working with relay modern

i believe i've set everything up according to the docs but the plugin doesn't seem to be working and i'm seeing an error in the vs code dev tools.
my .gqlconfig:
{
  schema: {
    files: 'graphql.schema.graphql',
  },
  query: {
    files: [
      {
        match: { include: '**/*.jsx' },
        parser: [ 'EmbeddedQueryParser', { startTag: 'graphql`', endTag: '`' } ]
      }
    ]
  }
}

i've tried with schemas in json as well as type declaration format. i've tried with and without relay: true.
after restarting the plugin, i get the following output
[Info  - 3:09:39 PM] NODE_PATH value is: /Users/brett/.config/yarn/global/node_modules
[Info  - 3:09:39 PM] Module '@playlyfe/gql' loaded from<EMAIL_ADDRESS>[vscode] File extensions registered:  [ 'json', 'jsx' ]
[Watch established (gqlSchemaFiles)]
	basePath: /Users/brett/projects/app-dealer-mc
	RelativePath: undefined
	Files: "graphql.schema.json"
[Watch established (gqlQueryFiles-0)]
	basePath: /Users/brett/projects/app-dealer-mc
	RelativePath: undefined
	Files: {
  "include": "**/*.jsx"
}

along with this error in the vs code dev tools
Trace: BAD uri lacks scheme, falling back to file-scheme.
	at /Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:114:656
	at new f (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:114:729)
	at new p (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:116:930)
	at Function.parse (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:116:279)
	at nullConverter (/Users/brett/.vscode/extensions/kumar-harsh.graphql-for-vscode-1.13.1/node_modules/vscode-languageclient/lib/protocolConverter.js:22:47)
	at Object.asUri (/Users/brett/.vscode/extensions/kumar-harsh.graphql-for-vscode-1.13.1/node_modules/vscode-languageclient/lib/protocolConverter.js:25:16)
	at LanguageClient.handleDiagnostics (/Users/brett/.vscode/extensions/kumar-harsh.graphql-for-vscode-1.13.1/node_modules/vscode-languageclient/lib/client.js:2119:29)
	at initialize.connection.initialize.then.connection.onDiagnostics.params (/Users/brett/.vscode/extensions/kumar-harsh.graphql-for-vscode-1.13.1/node_modules/vscode-languageclient/lib/client.js:1994:53)
	at handleNotification (/Users/brett/.vscode/extensions/kumar-harsh.graphql-for-vscode-1.13.1/node_modules/vscode-jsonrpc/lib/main.js:489:43)
	at processMessageQueue (/Users/brett/.vscode/extensions/kumar-harsh.graphql-for-vscode-1.13.1/node_modules/vscode-jsonrpc/lib/main.js:260:17)
	at Immediate.setImmediate [as _onImmediate] (/Users/brett/.vscode/extensions/kumar-harsh.graphql-for-vscode-1.13.1/node_modules/vscode-jsonrpc/lib/main.js:247:13)
	at runCallback (timers.js:696:18)
	at tryOnImmediate (timers.js:667:5)
	at processImmediate (timers.js:649:5)

syntax highlighting works and the GQL icon is white with a "GraphQL language server is running" tooltip on hover, but autocomplete doesn't work and i'm getting "unknown type" and "fragment ... is never used" errors on everything.

thanks in advance for any help!

Working on it. Should be out soon 🙂

nice, thanks! 🙏


The package's private functions can be called outside the package

Enhancement

Package feature in early version of IvorySQL is not supported now.


configure wallets

Support okx mobile wallet
Refactor wallet config

@taryune Now working with OKX mobile app as expected flow


[Feature Request] Schedule for adding or removing media

Description
Good morning,
As I state on the Jellyfin reddit (see here), this might be interesting for users like me who have limited storage coming in at its maximum to have a schedule for future additions and deletions (may also be interesting to integrate it into the current Jellyseerr system for user addition requests, so that they are informed at the temporal level of when the media will be available).
Desired Behavior
Added calendar system for adding and removing media by Jellyfin admin.
Additional Context
No response
Code of Conduct

[X] I agree to follow Overseerr's Code of Conduct

Jellyseerr does not really interact directly with the media itself though. A feature that's upcoming that might sort of interest you is local watchlist where you can have stuff added to watchlist to be requested later

Good morning,
Sorry, I may have expressed myself badly, but I was thinking of a calendar system, and that Jellyfin managers have write access to indicate when a next film or series will be released on the platform, or when a film will disappear, with dates (and if possible even times like a classic calendar). Users will have read access.
I know Jellyserr doesn't interact with live storage but it can be linked to the *arr stack, what I meant is like the watchlist system you are setting up, with the system calendar indicated previously there would also be the possibility of "programming" the stack *arr to recover a series or a film such day at such hour.
If this is feasible on your end, it could be an interesting addition for many worlds I think.


RuntimeError: wx2811_init failed with code -5 (mmap() failed)

I’m working on getting a new strip up and running and am testing it out with the included strandtest.py script. I’m getting the below error after installing and running the following command. This looks like a permission error, but I am using sudo, so any additional guidance is appreciated.
sudo python3.7 strandtest.py --clear
(Note that the error message is transcribed since I wasn’t able to copy and paste from my RPi).
Can't open /dev/mem: Permission denied
Traceback (most recent call last):
    File “/home/pi/rpi-ws281x-python/examples/strandtest.py”, line 97, in <module>
        strip.begin()
    File “/usr/local/lib/python3.7/dist-packages/rpi_ws281x/rpi_ws281x.py”, line 131, in begin
        raise RuntimeError(‘ws2811_init failed with code {0} ({1}}).format(resp, str_resp))
RuntimeError: wx2811_init failed with code -5 (mmap() failed)

Run it as root. It needs very low-level hardware access.

Closing because I'm not running into this anymore. I was running as root but have made other changes and I'm not sure how this was resolved.


What's new in 2.5 on 2.6

What's new button shows in 2.5 text and points to old Release Notes 2.5 page/
Tested on master-head (8c014ef)

Works fine on master-head (3ae6f43)


[Tablet] Small issues on users section

Preconditions:

IPAD Device 9,7'

Go to:  /admin/users/index
https://image.prntscr.com/image/tqlGZN7lQ6GDZSe83bEZow.png

@dawid-g Fixed

confirmed.


Unable to connect: "Failed to clear webhook"|"Connection reset by peer"

Hello, my bot is failing to start with error
Failed to clear webhook
akka.stream.StreamTcpException: The connection closed with error: Connection reset by peer

My guess is my connection to Telegram servers gets blocked.
Among your resent issues I saw your answer about proxy support. Is it still possible to access your refactored version (3.1.0-RC1)?

It could be that you have another instance of your bot already running e.g. in the background, an instance that didn't died even if you Ctrl-C it... it happens.
Make sure to run a single instance of your bot. Kill all the java processes.

You are probably right, but I've just tried running another bot which has never been launched before. The error reminds the same. So I still think, proxy will help me :)

You can pull the proxy logic yourself.
Just need to copy ScalajHttpClient.scala and ScalajHttpMarshalling and use it as you client.
Here's an example
https://github.com/mukel/telegrambot4s/blob/refactor/examples/src/ProxyBot.scala

I tried to copy two mentioned files, but no luck. These classes depend on changes in JsonMarshallers, RequestHandler, which I managed to repeat in my project as well. But there is one more thing I can't solve: ScalajHttpMarshalling is working with sealed trait InputFile, and I can't apply case operator on project private classes Path, Contents etc

I'm on vacation ATM (no laptop) but I could publish an RC2 with the recent patches and proxy support next week.
I've been working on a large refactor (ScalaJs support, circe-based serialization, proxy support via ScalajHttpClient, modular design...) You can try building from source the js branch (it uses mill instead of SBT).
Sorry for the inconvenience, feel free to open an issue or continue the discussion here, I'll galdly help you sort it out.

I can wait, I don't have urgent tasks :)
Btw, here is the bot I created using your library (just a dictionary of English words): https://github.com/Tieru/MacMillanTelegramBot . If you have time to look at that, any feedback is welcome.
I can also help you with maintaining your library. When you are back from your vacation you can delegate me simple tasks that you have in mind, but have no time to spend on them.
Please let this issue open until we try proxy. I guess many Russians will have problems with running their bots, so this topics might be a solution for them in the future.

I couldn't find any proxy support in 'js' branch. However I tried 'refactor' and now I can confirm that proxy does solve the issue! Thanks for your help!

Hello, @mukel . Could you please release a version that includes proxy? I have built a jar that includes those changes and it's fine for development, but deploy process to server or to another pc is a bit painful.
Probably I could help you somehow in order to save your time and prepare new releases

Closing this in favor of #63


add maxReadBatches and endRecordId to readShardStream handler

PR Description
Type of change

[x] New feature

Checklist

I have run format.sh under script
I have commented my code, particularly in hard-to-understand areas
New and existing unit tests pass locally with my changes

proto update https://github.com/hstreamdb/protocol/pull/81


Conditional header

If user is logged out, they see Sign up and Sign in button. Otherwise they see buttons to take then to their profile and to logout.
Logged in

Logged out

Hmm, something is apparently broken because this shouldn't actually be possible, accessing request.user should require decorating a view with @uses_session. My plan for this was to have it done with javascript actually, the reason is that 99% of the content on a PyPI page is not user specific, so if we keep the responses user agnostic then we canc ache the response once in varnish for all users, instead of once per user.

Heh, I bet it's because it's accessing it inside of a template, and that happens at the rendering layer.

accessing it inside of a template

That is true. And I don't think anything is broken yet. I forgot to add translations for the text in the buttons in the second commit, but fixed it in the third one.
I'm not sure how the js implementation would work - an ajax request perhaps, but I think that's just overcomplicating the client code.

The problem is, let's take the / view other than this little bit of the UI there is nothing in there that is user specific so if we can avoid adding user specific data to it we can use the same cached response for every user. This is important because PyPI gets roughly 100 million HTTP requests a day and anything we can cache at Fastly is a request that doesn't hit our backends which let us serve more traffic with less servers (and be faster at it too). So Warehouse has a goal where we don't add user specific content to pages unless that page needs it to function.
There are a few ways to handle this that doesn't cause us needing to cache every page for each user instead of just globally:

As you identified, use an AJAX request to fetch the current logged in user and render this little bit of UI based on that.
Have the login/logout view add another cookie that just states the logged in user's name. The web application will never use this cookie but a little bit of js can decide to render a logged in UI based on the existence of this cookie.
Use edge side includes which will allow us to cache the main project page, but drop a little tag into the response which will be replaced inside of varnish with the value of some other response. This is sort of like the {% include %} tag except it happens inside of varnish and varnish can cache the main response separately (and without being user specific) while caching the user information per user.

To be clear, the broken thing is that this PR works at all, because Warehouse should error when you attempt to access request.user unless you've decorated a view with the uses_session decorator.

Ah this makes sense. Thanks for explaining this.
I'll reimplement this using the second method.


Shaders not loading (Windows)

I always get the "cannot access to..." when trying to load a shader (.bin compiled file)
this is the code where i get the error:
bgfx::ShaderHandle loadShader(const char *FILENAME)
{
    const char* shaderPath = "???";

    switch(bgfx::getRendererType()) {
        case bgfx::RendererType::Noop:
        case bgfx::RendererType::Direct3D9:  shaderPath = "shaders/dx9/";   break;
        case bgfx::RendererType::Direct3D11:
        case bgfx::RendererType::Direct3D12: shaderPath = "shaders/dx11/";  break;
        case bgfx::RendererType::Gnm:        shaderPath = "shaders/pssl/";  break;
        case bgfx::RendererType::Metal:      shaderPath = "shaders/metal/"; break;
        case bgfx::RendererType::OpenGL:     shaderPath = "shaders/glsl/";  break;
        case bgfx::RendererType::OpenGLES:   shaderPath = "shaders/essl/";  break;
        case bgfx::RendererType::Vulkan:     shaderPath = "shaders/spirv/"; break;
    }

    size_t shaderLen = strlen(shaderPath);
    size_t fileLen = strlen(FILENAME);
    char *filePath = (char *)malloc(shaderLen + fileLen);
    memcpy(filePath, shaderPath, shaderLen);
    memcpy(&filePath[shaderLen], FILENAME, fileLen);

    FILE *file = fopen(FILENAME, "rb");
    fseek(file, 0, SEEK_END);
    long fileSize = ftell(file);
    fseek(file, 0, SEEK_SET);

    const bgfx::Memory *mem = bgfx::alloc(fileSize + 1);
    fread(mem->data, 1, fileSize, file);
    mem->data[mem->size - 1] = '\0';
    fclose(file);

    return bgfx::createShader(mem);
}

the error happens at this line:
fread(mem->data, 1, fileSize, file);

for testing i tried alot of other codes (from examples, tutorials etc..)
but NONE of them worked and always gave the "cannnot access to that memory" error
and for extra testing i built normal bgfx and tried to run examples but when i tried to switch from helloworld to cubes it crashed
i though this was my pc's error and i updated the drivers but that still did not work.
i use windows 11.
i also use c++ 17
*edit the original error is "access violation writing location" (this is from cmake debug)
and the error when i run it on gdb is "warning: C:\Users\REDACTED\bgfx\examples\common\bgfx_utils.cpp(75): Failed to load shaders/glsl/vs_cubes.bin."

https://bkaradzic.github.io/bgfx/examples.html#examples

https://bkaradzic.github.io/bgfx/examples.html#examples

it still happens though.
it just froze this time

If you already using VS o to build just run it thru VS. Also you should do debug build to get more information about issues.


feat: added the animation to the website with framer motion

style: added the animation to the website with framer motion
Related Issue
Closes: #764
Changes made 👷🏻♂under GSSOC 2023

Added animation to whole website using Framer motion replacing AOS.  (and the animation is set for once , not repeated every time person scrolls ,if any changes req. do inform)  : )

Thanks for the PR, I will review it soon ⚡

Hi @ManishaSwain8 , do take a look at this video , make sure your volume is up!
Love your work ⚡

Afternoon @tamalCodes ,
I went through your video but I didn't get the vanishing part,
https://github.com/tamalCodes/Milan/assets/96622693/a22b6bfb-afa6-4fac-bda1-b61629371df2
You want like this ? ? ?

That makes sense, sorry I should have been clear, can you please watch this video - I think that will clear out any confusion that exists.

@tamalCodes got it !
I will work on it , and for the extra image that I added, img was there in the code as you said for mobile view sorry about that I didn't put much attention to the code .
And the rest changes will be done  !

Afternoon,@tamalCodes
I did all the changes as per the requirement but I am not able to find the solution for the reverse animation in Framer Motion,
I went through the whole, documentation but did not find any useful article related to the issue .Though I know (reverse in transition property )but I have no idea how to trigger it while scroll.
Hope you understand ,You can assign this issue to someone else !  : |)

Afternoon,@tamalCodes I did all the changes as per the requirement but I am not able to find the solution for the reverse animation in Framer Motion, I went through the whole, documentation but did not find any useful article related to the issue .Though I know (reverse in transition property )but I have no idea how to trigger it while scroll. Hope you understand ,You can assign this issue to someone else ! : |)

Intresting, well in that case I think we should keep aos as it is and not implement these changes.
Thanks for your time, it means a lot when someone spends time brainstorming ⚡

Yup , sorry for inconvenience but I will keep on hunting for the solution specially in *Framer Motion * ! : |)


Cow: Watch hangallery.vn

Cow requests the watch of the watch_keyword hangallery\.vn. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

hangallery\.vn has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


make manpager.vim self-contained

As discussed in https://github.com/vim/vim/issues/2323#issuecomment-346626606
this new implementation does no longer depend on ftplugin/man.vim and thus is stabler than the ancient hack.

since you are the maintainer, you can simply send the complete file for inclusion to Bram. Then it will be included in one of the next runtime file updates.

I think this was included as of d23a823669d93fb2a570a039

Sorry, correct commit should have been 7254067ee970686cc3ff4a1effc


Write a DocStrings for _diff.py

Rewrite documental string for _diff.py and push them to the symbolical_dynamics branch.


"Unable to verify the JWS" exception with a valid signature and key

I'm trying to create a client for ory-am/hydra which exposes JWKs for verifying signatures.  I've been able to get code working using node.js, but the equivalent code is not working in PHP.  I've uploaded sample code at https://gist.github.com/nubs/55bea96f606a8c1de08c21cbdb9ae6b0.
The error I get is:
PHP Fatal error:  Uncaught exception 'InvalidArgumentException' with message 'Unable to verify the JWS.' in /home/anubis/jose/src/Verifier.php:140
Stack trace:
#0 /home/anubis/jose/src/Verifier.php(62): Jose\Verifier->verifySignatures(Object(Jose\Object\JWS), Object(Jose\Object\JWKSet), NULL, NULL)
#1 /home/anubis/jose/src/Loader.php(114): Jose\Verifier->verifyWithKeySet(Object(Jose\Object\JWS), Object(Jose\Object\JWKSet), NULL, NULL)
#2 /home/anubis/jose/src/Loader.php(49): Jose\Loader->loadAndVerifySignature('eyJhbGciOiJSUzI...', Object(Jose\Object\JWKSet), Array, NULL, NULL)
#3 /home/anubis/jose/test.php(17): Jose\Loader->loadAndVerifySignatureUsingKey('eyJhbGciOiJSUzI...', Object(Jose\Object\JWK), Array)
#4 {main}
  thrown in /home/anubis/jose/src/Verifier.php on line 140

Just trying to load the key works ($loader->load($signature)), but verifying the signature fails.
I've tried jumping through the code to track down why it didn't work, but I wasn't able to find anything.  Does anything look out of place with what I'm doing?
Thanks for the library!  As long as I can get this working, this will have worked out great for my project.

Thanks for reporting me that bug.
I am trying to find where it comes from as your signature is valid.
Stay tuned.

I found the bug and I am creating a patch to fix it.
Do you allow me to reuse the key and the JWT from your gist above for a test case to avoid futur code regression?

Yup - it's a temporary key and fake jwt, so go right ahead.  Thanks!

Tagged 5.0.1

Awesome - thanks.  I'll take a look!

It worked fine - thanks for the quick resolution!


Feature request (small) : Add PV size to log

To be able to calculate how much storage is/was consumed, it would be great to have the size in GB in the logs when the PV is created, should be very simple.

Usually data like this would be captured in an object representing the volume so that it could be more easily used programmatically rather than requiring someone to scrape log files, but it's not completely clear what you're trying to achieve. Can you elaborate on your use case?

I agree that this would be the best :=)
We have some customers (service providers) which have to check which pvc/pv where provisioned at which time and how long they were used to charge the  client for the disk usage. If there would be an output or trigger with this info it would be very easy. I. e output (maybe optional) in form of csv delimited format. Or the possibility to query Trident /etcd maybe ?
Hope this makes it a little bit mor clear.
Thx in advance.

Oh. If you're just trying to determine when a PVC or PV was created, Kubernetes already tracks that natively. For example:

$ kubectl get pvc -n trident trident -o json | jq '.metadata.creationTimestamp'
"2018-10-26T19:43:57Z"
$ kubectl get pv trident -o json | jq '.metadata.creationTimestamp'
"2018-10-26T19:43:58Z"

Thanks for your answer, maybe my explanation was that too bad :
I can see the creation time and size as long as a PVC exists, right ?
But how to get the information when a developer creates a pvc and deletes that again after 2 hrs ?
That is the problem. We have to know when a pvc is created , which size and when deleted.
Maybe I have a missunderstanding but I believe the info of an PVC is only available while it exists, not when it has gone.

That is true, but you can use the standard k8s API to handle this too, by using the List API to receive notifications when PVCs are created and deleted. This is similar to how Trident itself works.
Here's an example of kubectl calling that API. While it was waiting on the call with watch=true, I created and deleted a PVC, and you can see that it received multiple notifications for that.

$ kubectl get pvc --watch-only -v=6
I0314 14:31:19.128358    5494 round_trippers.go:405] GET https://xx.xx.xx.xx:6443/api/v1/namespaces/default/persistentvolumeclaims?limit=500 200 OK in 16 milliseconds
I0314 14:31:19.131966    5494 round_trippers.go:405] GET https://xx.xx.xx.xx:6443/api/v1/namespaces/default/persistentvolumeclaims?resourceVersion=20091587&watch=true 200 OK in 1 milliseconds
NAME     STATUS    VOLUME   CAPACITY   ACCESS MODES   STORAGECLASS   AGE
my-pvc   Pending                                      default        0s
my-pvc   Pending                     default   1s
my-pvc   Pending                     default   2s
my-pvc   Pending   default-my-pvc-c6fb8   0           default   7s
my-pvc   Bound   default-my-pvc-c6fb8   1Gi   RWO   default   7s
my-pvc   Terminating   default-my-pvc-c6fb8   1Gi   RWO   default   21s
my-pvc   Terminating   default-my-pvc-c6fb8   1Gi   RWO   default   21s


New-GcsObject error message when trying to upload folder hard to understand

Trying to upload a folder:
New-GcsObject -bucket abucket -name aname -file path/to/a/dir
Error thrown says File not found, I would have expected something along the lines of New-GcsObject only supports file upload, path/to/a/dir is a folder

We have the ability to upload a folder with https://github.com/GoogleCloudPlatform/google-cloud-powershell/pull/329 so this won't error out anymore.


[Event Change Request] table 179 "Reversal Entry" - OnBeforeCheckFA

Hello can you please add a IsHandled Parameter to the OnBeforeCheckFA event?
    local procedure CheckFA(FALedgEntry: Record "FA Ledger Entry")
    var
        FA: Record "Fixed Asset";
        FADeprBook: Record "FA Depreciation Book";
        DeprCalc: Codeunit "Depreciation Calculation";
        IsHandled: Boolean;
    begin
        IsHandled := false;
        OnBeforeCheckFA(FALedgEntry, IsHandled);
        if IsHandled then
            exit;
        FA.Get(FALedgEntry."FA No.");
        CheckPostingDate(
          FALedgEntry."Posting Date", FALedgEntry.TableCaption(), FALedgEntry."Entry No.");

    [IntegrationEvent(true, false)]
    local procedure OnBeforeCheckFA(var FALedgerEntry: Record "FA Ledger Entry"; var IsHandled: Boolean)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10889.


feat(explorer): explorer deployment setup

Closed in favor of https://github.com/latticexyz/mud/pull/3244


Cleanup python script approach in 4788 deployment and custom-mev-flood funder

Switch to a python container or some other approach instead

Cleanup: https://github.com/kurtosis-tech/ethereum-package/commit/18da90bd3f8b6471457e613edc1e17ff01d2ae0a


Document interaction of human-readable rule names and error handling

The presence of a human-readable name for a rule (regardless of its content) can affect the kinds of errors that PEG.js produces. This is a subtle effect that is important for getting good parse errors, but it doesn't seem to be documented.
Here's a tiny grammar that parses sequences of lower-case letters separated by semicolons:
S = A (";" A)*
A = [a-z]

What happens if I mistakenly use an upper-case letter? If I try to parse a;B, this grammar emits the error:
Expected [a-z] but "B" found.

But if I assign a human-readable name for the first rule:
S "foo" = A (";" A)*
A = [a-z]

I get a very different error message:
Expected end of input but ";" found.

The parser seems to prefer "blaming" named rules over unnamed rules. This seems like a reasonable way to control error messages, and there's kind of an intuitive logic to it: unnamed rules are more likely to be some kind of internal implementation detail, and the library tries to avoid exposing them too much to the user.
But unless I'm missing something, this behavior isn't documented. (I found it by trial and error.) I'd be happy to draft some text to help document this if I can have some help with understanding exactly how this feature works.

@sampsyo You are right that this behavior is not documented, or rather documented quite vaguely:

A rule can also contain human-readable name that is used in error messages (in our example, only the integer rule has a human-readable name).

In fact, the whole error handling area is currently not documented much.

I'd be happy to draft some text to help document this if I can have some help with understanding exactly how this feature works.

Are you still interested in this? You pretty much already discovered how the feature works — low-level expressions (string literals, character classes, and .) produce expectations on match failures and a rule with human-readable name just overwrites these expectations with a one containing rule’s human-readable name.

Sure! I won't pretend that I understand it completely, but I can try putting together a few examples to demonstrate the effect. I'll open a PR.

Merged #415


Alias resolving issue on Windows

using latest webpack, ang pug-plugin
can't resolve alises usage like in example
try to use "extends ~Views/layouts/default.pug"
in "views/pages/home/index.pug"
got error:
ERROR in ./src/views/pages/home/index.pug Module build failed (from ./node_modules/@webdiscus/pug-loader/src/index.js): NonErrorEmittedError: (Emitted value instead of an instance of Error) [pug-loader] Pug compilation failed. PugLoaderException: [pug-loader] the file ~Views/layouts/default.pug can't be resolved in the pug template: D:\0html\pug_plugin_test2\src\views\pages\home\index.pug Error: Can't resolve '~Views/layouts/default.pug' in 'D:\0html\pug_plugin_test2\src\views\pages\home'     at D:\0html\pug_plugin_test2\src\views\pages\home\index.pug line 2     at processResult (D:\0html\pug_plugin_test2\node_modules\webpack\lib\NormalModule.js:755:12)     at D:\0html\pug_plugin_test2\node_modules\webpack\lib\NormalModule.js:860:5     at D:\0html\pug_plugin_test2\node_modules\loader-runner\lib\LoaderRunner.js:400:11     at D:\0html\pug_plugin_test2\node_modules\loader-runner\lib\LoaderRunner.js:252:18     at context.callback (D:\0html\pug_plugin_test2\node_modules\loader-runner\lib\LoaderRunner.js:124:13)     at D:\0html\pug_plugin_test2\node_modules\@webdiscus\pug-loader\src\index.js:270:21     at Object.compile (D:\0html\pug_plugin_test2\node_modules\@webdiscus\pug-loader\src\index.js:256:5)     at Object.module.exports (D:\0html\pug_plugin_test2\node_modules\@webdiscus\pug-loader\src\index.js:269:11)
it seems it try to find "pug for extending" in same directory.
if specify directly "extends ../../layouts/default.pug" it works.
instead of it, aliases for "~Images", "~Styles" work properly.
use Windows 10 x64.

Hello @JONMARSHALDEV,
thanks for the bug report.
The issue is fixed in the version  2.7.7.


Thoroughly test and document installation from source against custom MPI

The correct installation command is:
python -m pip install --no-binary parafields -v parafields

It should be discouraged to pass :all: to --no-binary because it will e.g. build CMake from source which tales a long time.

This is done and test in CI.


Change fences default to true

Initial checklist

[X] I read the support docs
[X] I read the contributing guide
[X] I agree to follow the code of conduct
[X] I searched issues and couldn’t find anything (or linked relevant results below)

Problem
Fenced code is (arguably) much better than indented code.

Indented code and lists interfere
Indented code does not work in MDX
Indented code uses 4 spaces, block quotes use a > and one space, lists use different “indents”, making markup source code impossible to align, so best to not even try
fenced code supports language flags, which are super useful
fenced code supports “meta” (arbitrary stuff), which is while not used much, useful in plugins

Historically fenced code was not supported everywhere. I think we’re now at a point where it’s basically everywhere. Hence this suggestion.
Solution
default to fences: true.
Alternatives
as is

Definitely agree with this one.


fix date format code in kat backend

Fixed in #1728, which incorporates kat-client v1.5.0.

Released in 0.74.1.


super-voxel size tuning, SVB block size tuning

We need to tune super-voxel size and SVB block size.

Better to post this here:
https://github.com/HPImaging/sv-mbirct/issues


F16 DED

Hello, my project is the F16 DED. I tried my way around programming. I also get the display, but as soon as I use it, the output throws everything into confusion. Is there a chance that you will help to get the Code right.

I will do my best.  Can you share your sketch?

My sketch."#define DCSBIOS_DEFAULT_SERIAL#include "DcsBios.h"#include <Arduino.h>#include <Wire.h>#include <U8g2lib.h>#include <U8x8lib.h>U8G2_SSD1309_128X64_NONAME0_F_HW_I2C u8g2(U8G2_R0, /* reset=/ 13);void onDedLine1Change(char newValue) {    /* your code here /    u8g2.setFont(u8g2_font_5x7_tf);    u8g2.drawStr(0,10,newValue);    u8g2.sendBuffer();        }DcsBios::StringBuffer<25> dedLine1Buffer(0x44fc, onDedLine1Change);void onDedLine2Change(char newValue) {    /* your code here /    u8g2.setFont(u8g2_font_5x7_tf);    u8g2.drawStr(0,20,newValue);    u8g2.sendBuffer();}DcsBios::StringBuffer<25> dedLine2Buffer(0x4516, onDedLine2Change);void onDedLine3Change(char newValue) {    /* your code here /    u8g2.setFont(u8g2_font_5x7_tf);    u8g2.drawStr(0,30,newValue);    u8g2.sendBuffer();}DcsBios::StringBuffer<25> dedLine3Buffer(0x4530, onDedLine3Change);void onDedLine4Change(char newValue) {    /* your code here /    u8g2.setFont(u8g2_font_5x7_tf);    u8g2.drawStr(0,40,newValue);    u8g2.sendBuffer();}DcsBios::StringBuffer<25> dedLine4Buffer(0x454a, onDedLine4Change);void onDedLine5Change(char newValue) {    /* your code here /    u8g2.setFont(u8g2_font_5x7_tf);    u8g2.drawStr(0,50,newValue);    u8g2.sendBuffer();}DcsBios::StringBuffer<25> dedLine5Buffer(0x4564, onDedLine5Change);void setup(void) {    u8g2.begin();   // Start OLED    u8g2.clear.Display();    DcsBios::setup();}void loop(void) {   DcsBios::loop();  }"Am 24.08.2021 21:31 schrieb Talbot @.**>:
I will do my best.  Can you share your sketch?
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android.

Hey. So looking at this the DCS bios usage looks right to me.  I suggest the following:
Remove the SendBuffer calls.  According to the documentation for your display controller, those are to be used with NextPage() for buffered display, which you are not using.  https://github.com/olikraus/u8g2/wiki/u8g2reference#drawstr
Simplify for testing.  Start with all code in setup, issue the clear, then try drawStr with known strings like "L2HERE".  Once that displays properly, move each line into the DCS callbacks but leave in the static strings.  If that works, Finally use the dcs newValue strings.
Let me know how that goes.

Hey, i tried your solution but no results. The first sketch shows me the results in the pictures. The last picture is when i push Comm1 button.Am 05.09.2021 17:22 schrieb Talbot @.***>:
Hey. So looking at this the DCS bios usage looks right to me.  I suggest the following:
Remove the SendBuffer calls.  According to the documentation for your display controller, those are to be used with NextPage() for buffered display, which you are not using.  https://github.com/olikraus/u8g2/wiki/u8g2reference#drawstr
Simplify for testing.  Start with all code in setup, issue the clear, then try drawStr with known strings like "L2HERE".  Once that displays properly, move each line into the DCS callbacks but leave in the static strings.  If that works, Finally use the dcs newValue strings.
Let me know how that goes.
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android.

I don't see the sketches or pictures.  Maybe easier to discuss on discord?
Puma [476th 76th vFS]#7227


Adds ES5 REST index DAO for conductor 2.0

With ES5 and later, there is a big push to use the REST client rather than the TCP transport client.
Not only that, but some services such as AWS ElasticSearch only support the HTTP REST client. This
is an attempt to add the ES 5 REST client for conductor.
This is me attempting to add this to Conductor 2.0. There is another PR to add this to Conductor 1.12.

Pull Request Test Coverage Report for Build 1971

120 of 290 (41.38%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 62.515%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

es5-persistence/src/main/java/com/netflix/conductor/elasticsearch/es5/ElasticSearchV5Module.java
9
10
90.0%

es5-persistence/src/main/java/com/netflix/conductor/dao/es5/index/ElasticSearchRestDAOV5.java
104
273
38.1%

Totals

Change from base Build 1967:
-0.5%

Covered Lines:
7148

Relevant Lines:
11434

💛  - Coveralls

Oy vey. Will probably work on improving the coverage later. But if anyone wants to take this and run with it. Please feel free. I'm no longer really working on conductor integration anymore.

@bitbier There's something in this change that's causing conductor-server to be built as version 1.12.4-SNAPSHOT instead of 2.1.0-SNAPSHOT .. It's not apparent to me why though.  I'll take a closer look at that and help out with getting some test coverage to get this merged.
I'll follow up to coordinate updating this PR with that work.

@bitbier please take a look at davidwadden/conductor#es-5-rest-module-2.0-refactor for my progress so far.
I started with changing around the integration test setup to DRY the same integration test used for ES Transport Client on Dynomite/Redis backend.  High-confidence integration tests were maybe most important to me.  I did need to add some async polling following indexing new documents in a couple places.
I changed the test setup on TestElasticSearchRestDAOV5.java to create and destroy the indices for every test method and added some coverage.
Looking for any feedback as I work through getting happy path coverage on the DAO itself.

@cyzhao Since ES transport client is deprecated in future releases : https://www.elastic.co/guide/en/elasticsearch/client/java-api/master/transport-client.html Is there any plans of merging this / completing this PR ?

@mudit3774 I've improved test coverage over at https://github.com/davidwadden/conductor/commits/es5-rest-client and was going to open a new PR since the original submitter hasn't gotten back to me.  It's not perfect yet but good enough to get a code review I think.  I didn't change production code too much, mostly backfilling tests;  both end-to-end and unit.

Merged #887 which extends from this.


recommend rustup over multirust

I think I'll hold this off until rust 1.8 release, because the arm example on the README won't work on stable until that release.


GDExtension method calls should not pass nullptr args

fix for a nasty crash that happens when passing a null variant to any gdextension function parameter expecting any kind of Object *.
the GDExtensionMethodBind  expects a pointer to an Object * but get_opaque_pointer returns a nullptr which it tries to dereference..
could also be fixed in godot-cpp with a nullptr check, but this seemed more in line with what the binding code expects
this or another fix should rlly be applied to 4.2, 4.1, nd 4.0
it can be easily cherrypicked for 4.2 but 4.1 and 4.0 have different binding code, ill make a PR for 4.1 when its apropriate
fixes https://github.com/godotengine/godot/issues/86478
fixes https://github.com/godotengine/godot-cpp/issues/1056

Hm. I'll do some more "historical" investigation when I have a chance, but if we've really been encoding null objects this way in 4.0, 4.1, and 4.2, I'd be reluctant to change this in Godot, because if it's been consistent, this sounds more like a godot-cpp bug.

pretty sure a bug in godot itsself,, i mean it doesnt even reach the gdextension dll,,
it seems like a clash between what the VariantInternal stuff used to do and what the GDExtensionMethodBind expects when making a validated call,, in 4.1 the crash happens in the extension dll instead
i guess the ptr to nullptr could be a ptr to the variant itsself like it is here:
https://github.com/godotengine/godot/blob/8fb744ac230a3896bf354e3a82760a94dccde024/core/extension/gdextension.cpp#L239
but only if a null variant is guaranteed  to be zeroed out (  ^ as long as ret_opaque points to valid memory it shouldnt really matter what it is bc its memory for a null return type, not getting read or written to, just maybe ^ )
and i cant look thru every file to check..
so this solution seemed pretty good

hmmm, the failed check is a cancelled ios build,,,

Sorry for taking so long to get back to this!
I didn't do any testing, but I skimmed the code from 4.0, 4.1 and 4.2, and it does appear that we've been encoding null Object * as nullptr in ptrcalls through all of those versions.
So, I'm personally against making this change in Godot. I think consistency with previous versions is important unless there is a clear advantage to changing the encoding, and I don't think there is in this case (in fact, I think the current encoding is slightly better because we don't have to dereference the Object ** to check for nullptr - it's just nullptr). While this change is unlikely to break anything, it's likely that all bindings other than godot-cpp are already doing a correct null check, and changing the encoding is going to make the extra check that they're doing redundant, but they probably won't remove it in order to retain compatibility with older Godot versions.
I think the crash should be fixed in godot-cpp instead

it does appear that we've been encoding null Object * as nullptr in ptrcalls through all of those versions.
So, I'm personally against making this change in Godot. I think consistency with previous versions is important unless there is a clear advantage to changing the encoding

i dont think this comment makes sense, this is a crash that happens when a validated call is made and a Nil value is encoded as an Object ** , a Nil value can only be passed to an Object * parameter, builtin types cannot be null  ,,  its been consistently broken in all of gdextension this has to be fixed in Every version ,, there is no consistency to follow , that doesnt mean it was intentional, and i dont think this is 'changing the encoding' i think its following it more closely

While this change is unlikely to break anything, it's likely that all bindings other than godot-cpp are already doing a correct null check, and changing the encoding is going to make the extra check that they're doing redundant, but they probably won't remove it in order to retain compatibility with older Godot versions.

it actually seems like D bindings Dont make this check,
https://github.com/godot-dlang/godot-dlang/blob/ff421674d9f775f9cc88e57a6ab625b172b97b4f/src/godot/api/wrap.d#L356
but godot-rust does,,, if im interpreting this correctly
https://github.com/godot-rust/gdext/blob/88d8ed0719eb70d2a5af370e9b0fa672acea032a/godot-core/src/obj/raw.rs#L464C12-L464C15
since its a validated call, i doubt a null check should be done ,, it would need to get added in a couple places, it seems unlikely that someone just forgot that, i was interpreting the GDExtensionObjectPtr * or Object ** as 'a non null pointer to a godot Object *' ,, like a reference, but is a C api, because all other builtin type pointers dont have a null check ,, none of the validated arg to pointer conversion stuff have a null check ,, but its not written out anywhere ,, just gotta ask whoever made this i guess ,,, vnen and/or reduz

@vnen i see your name and reduz in the commit log for gdextension validated ptrcall ,, do you have insight about this?
are GDExtensionObjectPtr *s meant to be nullable when making a validated call?
should this change be in godot-cpp instead?

Here's alternative PR https://github.com/godotengine/godot-cpp/pull/1405 which fixes the issue in godot-cpp rather than Godot itself.
I've also added discussion of this issue to the agenda of the next GDExtension meeting. (If anyone is interested in participating, please stop by #gdextension on chat.godotengine.org for details about the GDExtension meetings.)

We discussed this at the GDExtension meeting, and felt that it would be great if @Bromeon (or other maintainers of GDExtension bindings) could weigh in on the current encoding of Object *. It would also be nice to hear from @vnen or @reduz on what the original intention of the encoding was. Mainly, it's a question of if the current encoding is "incorrect" and should be fixed, or if the encoding is OK (since it has at least been consistent) and godot-cpp should be updated to correctly handle it.

In godot rust, null objects in argument calls are passed as pointers to null object pointers in all version of godot except 4.0. In 4.0 we pass them directly as null pointers.
However we also dont use null objects directly all that frequently (im unsure if we ever do actually) when calling gdextension methods since we have an outstanding bug where direct method calls only allow for non-null objects. The current workaround there is to use Object.call instead which means we then pass null variants instead of null objects. So it may be that we're doing it wrong and just haven't noticed because we actually dont ever do it.
I dont think it would really impact much if we end up needing to switch to encoding them as null pointers instead.

(Adding this comment just to differentiate my personal opinion from what I wrote in my previous comment, which was the result of that GDExtension meeting, and isn't the same as my personal view.)
As I've stated previously, it's my opinion that we don't really gain anything by changing the encoding as proposed in this PR: both the current encoding and the proposed encoding are capable of representing a null Object * just fine.
The only advantage to changing the encoding is theoretical "correctness", if the proposed encoding more closely matches the original intention for the encoding (however, it's not totally clear what the original intention was).
My personal vote is still for keeping the current encoding. Regardless of the intention, it's the encoding we've been using in all versions of Godot 4.x, and it's capable of representing what we need it to, so I see no compelling reason to change it.
But we certainly could improve the overall situation by documenting the encoding, so that it's clearer to the implementors of GDExtension bindings.

As I've stated previously, it's my opinion that we don't really gain anything by changing the encoding as proposed in this PR: both the current encoding and the proposed encoding are capable of representing a null Object * just fine.

My personal vote is still for keeping the current encoding. Regardless of the intention, it's the encoding we've been using in all versions of Godot 4.x, and it's capable of representing what we need it to, so I see no compelling reason to change it.

That is reasonable -- it would also prevent bindings from possibly needing to deal with different behavior across multiple API versions. As such, I'm with you in light of "don't fix what's not broken".

Sorry I though this was a fix, so adding to the discussion: Regarding the intention of the encoding, I think (if I am not mistaken) it was that it passed a pointer to a pointer for returning, and a single pointer for argument passing. The problem is that this caused a lot of issues so I think at some point it was changed?
I honestly would use what work best for you, since you are the consumers of the API at this point.

I checked #1405 and this looks like a better solution I guess, since each platform can handle it as preferred?

Superseded by PR https://github.com/godotengine/godot-cpp/pull/1405


Nested grid support

When nesting grids, the .container wrapper class causes a noticeable padding on elements, this means that if you have a nested containers, the margins on nearby columns do not line up.

@SevereOverfl0w I will test more on the nesting grids. thanks.

@SevereOverfl0w I have examples here. If there is any problem, please tell me.