Duplicate blank nodes in query results

Describe the bug
I have this query running against Stardog: https://s.zazuko.com/jD8b3Z
Assuming that Stardog is correct, I expect a matching result when querying in Oxigraph
However, Oxigraph returns all triples where they are constructed from blank nodes twice. Especially problematic in the case of anonymous syntax because that create doubles.
To Reproduce
Steps to reproduce the behavior:

docker image ghcr.io/oxigraph/oxigraph:0.4.0-alpha.6
Open the query UI and run the above
See 20 results instead of 15 coming from Stardog

It helps to used explicit BIND(BNODE()) instead of anonymous. They still come twice but can be deduplicated
And in general, I notice that other triples are duplicated too but only blanks a problem, of course

Thank you for the bug report! Sadly https://int.lindas.admin.ch/query returns me a 404 error stating that the lindas database does not exists.
To check an hypothesis: do you know if the SERVICE query is returning duplicated rows that Stardog would deduplicate but Oxigraph would not?

Uh yea, apparently we had some outage today. Lindas should be back online tomorrow.
Will check your hypothesis when it's running again

Thank you!

INT is still down but I changed that query to run against TEST. You can see the exact same result both using SERVICE (https://s.zazuko.com/3gAdZmU) and executer directly (https://s.zazuko.com/3gAdZmU)
This is what I get from when I do a SERVICE from oxigraph:
@prefix sh: <http://www.w3.org/ns/shacl#> .
@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .

_:webgrud-module-8-table-21targetPath <https://code.described.at/value> <https://webgrud.zazkoians.org/api/schema/table/metadata>, <https://webgrud.zazkoians.org/api/schema/table/metadata> ;
	<https://code.described.at/name> "targetPath", "targetPath" .

_:b_webgrud-module-8-table-21 <https://code.described.at/arguments> _:webgrud-module-8-table-21targetPath, _:webgrud-module-8-table-21targetPath ;
	<https://code.described.at/implementedBy> <https://kopflos.described.at/QueryShapeHandler>, <https://kopflos.described.at/QueryShapeHandler> ;
	<https://kopflos.described.at/queryParam> [
		<https://code.described.at/name> "nahrstoff" ;
	], [
		<https://code.described.at/name> "kultur" ;
	] ;
	<https://kopflos.described.at/method> "GET", "GET" .

<https://webgrud.zazkoians.org/api/table/webgrud-module-8-table-21> <https://kopflos.described.at/handler> _:b_webgrud-module-8-table-21, _:b_webgrud-module-8-table-21 ;
	sh:targetNode <https://webgrud.zazkoians.org/table/webgrud-module-8-table-21>, <https://webgrud.zazkoians.org/table/webgrud-module-8-table-21> ;
	a <https://kopflos.described.at/ResourceShape>, <https://kopflos.described.at/ResourceShape> .

<https://webgrud.zazkoians.org/table/webgrud-module-8-table-21> <https://webgrud.zazkoians.org/api/schema/cube> <https://agriculture.ld.admin.ch/agroscope/webgrud-module-8-table-21/1>, <https://agriculture.ld.admin.ch/agroscope/webgrud-module-8-table-21/1> ;
	<https://webgrud.zazkoians.org/api/schema/table/metadata> <https://webgrud.zazkoians.org/table/webgrud-module-8-table-21/metadata>, <https://webgrud.zazkoians.org/table/webgrud-module-8-table-21/metadata> ;
	a <https://webgrud.zazkoians.org/api/schema/Table>, <https://webgrud.zazkoians.org/api/schema/Table> .

On the other hand, I tried the same federated query from Jena and also the original, which used [ ] syntax for blank nodes and the results match those of oxigraph so maybe Stardog is doing some non-standard "optimisation" of that query which changes how the blank nodes are bound to more closely match the expectation...
Bottom line, the duplication is not a problem but having to BIND blank nodes is a nuisance. `unfortunately, that may be according to SPARQL spec.

A template can create an RDF graph containing blank nodes. The blank node labels are scoped to the template for each solution. If the same label occurs twice in a template, then there will be one blank node created for each query solution, but there will be different blank nodes for triples generated by different query solutions.

Looks like Stardog goes the extra mile not only to dedupe CONSTRUCTed triples but also collapse blanks which by spec should produce distinct nodes?

With the above in mind, I still observe some unnecessary duplicate results I cannot easily explain in other scenarios. Not only blank nodes.
Please try this query
PREFIX code: <https://code.described.at/>
PREFIX sh: <http://www.w3.org/ns/shacl#>
PREFIX kopflos: <https://kopflos.described.at/>
base <https://webgrud.lndo.site/>

DESCRIBE ?loader ?impl
WHERE {
  <api/table> sh:property/kopflos:handler ?handler .

  {
    ?handler kopflos:resourceLoader ?directLoader .
  }
  UNION
  {
    ?handler ^kopflos:handler/^sh:property/kopflos:api ?api .
    ?api kopflos:resourceLoader ?apiLoader .
  }

  BIND(COALESCE(?directLoader, ?shapeLoader, ?apiLoader, ?configLoader) as ?loader)

  ?loader code:implementedBy ?impl .
}

against this data
PREFIX schema: <http://schema.org/>
PREFIX sh: <http://www.w3.org/ns/shacl#>
PREFIX kopflos: <https://kopflos.described.at/>
PREFIX code: <https://code.described.at/>
base <https://webgrud.lndo.site/>

kopflos:DescribeLoader
  a code:EcmaScriptModule ;
  code:link<EMAIL_ADDRESS>.

<api/config>
  a kopflos:Config ;
  kopflos:api <> ;
.

<>
  a kopflos:Api ;
  kopflos:resourceLoader
    [
      code:implementedBy kopflos:DescribeLoader ;
    ] ;
.

<api/table>
  a kopflos:ResourceShape ;
  kopflos:api <> ;
  sh:targetClass <api/schema/Table> ;
  kopflos:queryShape <api/query-shape/Table> ;
  sh:property
    [
      sh:path <api/schema/Table/metadata> ;
      kopflos:handler
        [
          kopflos:method "GET" ;
          code:implementedBy
            [
              a code:EcmaScriptModule ;
              code:link <file:handlers/table.js#metadata> ;
            ] ;
        ] ;
    ] ;
.

If I CONSTRUCT the desired triples, I get 3 results but DESCRIBE doubles them. Is that related?

Thank you for investigating this!

Bottom line, the duplication is not a problem but having to BIND blank nodes is a nuisance. `unfortunately, that may be according to SPARQL spec.

Indeed, Oxigraph follows that and creates new blank node for each solution.

Looks like Stardog goes the extra mile not only to dedupe CONSTRUCTed triples but also collapse blanks which by spec should produce distinct nodes?

That's definitely possible (I am not very familiar with Stardog)

With the above in mind, I still observe some unnecessary duplicate results I cannot easily explain in other scenarios. Not only blank nodes.

I only get 3 results with the query executed on the data. Maybe you have a duplicated blank node in the data that leads to two values being bound to ?impl and so two "solutions" that would get described? Oxigraph is quite naive and if the same term is in multiple solutions it will be described multiple times (it's probably something that should be fixed).


Sync node labels and taints

This closes #5.

This is done – merging. :shipit: :shipit: :shipit:

This is done – merging. :shipit: :shipit: :shipit:


feat: DoH ability to use custom servers

Related Issues
Fixes :  #254

I've removed the quick fix for #253, tests are expected to fail until it's rebased with hyperglass 2.0.2

@thatmattlove as an open-source contributor, I wanted to express my concern about how this PR was merged. The minimum courtesy when someone contributes is to keep the author credited as a thank you for their contribution. On GitHub, you have the right to edit my PR by default. If you need to add something, feel free to do so and then merge it. Of course, you have the final say as you are the maintainer.
I think (and hope) that you did this for convenience, and I trust that next time it will be merged properly.
PS : Thanks for your OSS project ;)

Please refer to the Contributing Policy, namely:

Because I've been solo-maintaining and building hyperglass since around April 2019, I've become pretty particular about things that might seem trivial to someone just trying to help out. While I absolutely welcome development contributions, please don't be offended if pull requests are denied, or if I request things to be done a certain way.

I appreciate your willingness to contribute your time and effort to hyperglass, however, there were several reasons your PR was not merged:

I was already working on a solution after I saw #254, and didn't see your PR until those efforts were underway.
Your PR introduced a breaking change by removing the name field.
CI tests for your PR failed, though I have not looked into why.

In the future, if you plan to implement a fix for an issue you've raised, please indicate that intent in the issue so I know not to start working on it.

Because I've been solo-maintaining and building hyperglass since around April 2019, I've become pretty particular about things that might seem trivial to someone just trying to help out. While I absolutely welcome development contributions, please don't be offended if pull requests are denied, or if I request things to be done a certain way.

Sure, I understand, and I've read it, but you should not say contributions are welcome when you say right after that you can deny them just like that. You can deny it as you are the maintainer if we can't agree on how it must be done, and that's okay, but you can just request changes when someone has started working on it.

I was already working on a solution after I saw https://github.com/thatmattlove/hyperglass/issues/254, and didn't see your PR until those efforts were underway.

No worries, it happens, but my PR was linked to the issue.

Your PR introduced a breaking change by removing the name field.

Yes, you are right.

CI tests for your PR failed, though I have not looked into why.

I've written in the next comment why; it's due to the "." bug. I've removed the temporary fix to get it merged easily. Rebasing on it is the only thing that was needed.


Webpa crash fix for an invalid curl request

Codecov Report

Merging #35 into wdmp-c_yocto will decrease coverage by 0.2%.
The diff coverage is 33.33%.

@@               Coverage Diff                @@
##           wdmp-c_yocto      #35      +/-   ##
================================================
- Coverage         91.57%   91.37%   -0.21%
================================================
  Files                 3        3
  Lines              1793     1797       +4
  Branches            184      185       +1
================================================
  Hits               1642     1642
- Misses               41       44       +3
- Partials            110      111       +1

Impacted Files
Coverage Δ

src/wdmp-c.c
80.64% <33.33%> (-2.14%)
:arrow_down:

Continue to review full report at Codecov.

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


Password authentication

I'd like to use Next.js Starter to use email & password-based authentication. It seems next-auth takes care of this now as it adds the /email/signin route.
Is it possible to add a way to supply a username and password validation function to next-auth (that I can connect to MongoDB inside of)?
Something else to consider is using bcrypt to generate password hashes. I don't need this now since I will not be doing user registration (only login), but I might need it later on for other projects.

This is now resolved in 1.8.0, which allows any authentication mechanism by defining a custom signIn() function which enables a POST based route to /auth/signin.
As this approach doesn't it prescribe the fields it can be used for username and password, two factor authentication or even token based SSO integration. An example of how to configure and use it provided in the examples folder.

Hello, I am trying to conduct a very simple test of the custom signIn() function. I've downloaded the example from github, run npm install in the 'example' directory, then npm run dev. Without modifying anything else in the example, I've uncommented the signIn() function, and altered it like so:
signIn: ({form, req}) => {
  return new Promise((resolve, reject) => {
    // Should validate credentials (e.g. hash password, compare 2FA token
    // etc) and return a valid user object from a database.
    /*
      return usersCollection.findOne({
      email: form.email
    }, (err, user) => {
      if (err) return reject(err)
      if (!user) return resolve(null)

      // Check credentials - e.g. compare bcrypt password hashes
      if (form.password === "test1234") {
        // If valid, return user object - e.g. { id, name, email }
        return resolve(user)
      } else {
        // If invalid, return null
        return resolve(null)
      }
    })
    */
    resolve({id:1234, name: 'Test', email<EMAIL_ADDRESS>  })
}

Now when I visit http://localhost:3000/auth/credentials and input anything into the "Email Address" and "Password" fields, I appear to login successfully. I see in the network tab that /auth/signin is being visited, followed by the callback.js page. I'm then returned to the index page but i am not signed in. The session object does not include a user object, and the "Manage Account" and "Sign out" links are not appearing.
What am I doing wrong?


Test/add tests for new classes

It includes #88 and #89. Should be merge after that.
Ensure that we test new classes vs old functions.

Thanks!


MSID: 96344 Version: 2 DOI: 10.1101/2<IP_ADDRESS>5454

MSID: 96344
Version: 2
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>5454
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=96344
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>5454

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/96344v2
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>5454
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/96344/v2/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

@acollings, just double checking we're OK to proceed with use of both solid and compelling here?

Hi @fred-atherden this looks good, thanks!


[Bug]: googlechrome installing a specific version fails

Prerequisites

[X] I have written a descriptive issue title.
[X] I have searched all issues/PRs to ensure it has not already been reported or fixed.
[X] I have verified that I am using the latest version of Scoop and corresponding bucket.

Package Name
googlechrome
Expected/Current Behaviour
I'd like to downgrade googlechrome in order to reproduce a problem with https://code.earthengine.google.com/
So, I looked at the commit history https://github.com/ScoopInstaller/Extras/commits/master/bucket/googlechrome.json and picked the version before the most recent and expect it to install.
But sadly it fails with this error message
> scoop install<EMAIL_ADDRESS>WARN  Given version (119.0.6045.124) does not match manifest (119.0.6045.160)
WARN  Attempting to generate manifest for 'googlechrome' (119.0.6045.124)
Autoupdating googlechrome
Searching hash for 119.0.6045.124_chrome_installer.exe in https://scoopinstaller.github.io/UpdateTracker/googlechrome/chrome.min.xml
Could not find hash in https://scoopinstaller.github.io/UpdateTracker/googlechrome/chrome.min.xml
Downloading 119.0.6045.124_chrome_installer.exe to compute hashes!
The remote server returned an error: (404) Not Found.
URL https://dl.google.com/release2/chrome/$match32_119.0.6045.124/119.0.6045.124_chrome_installer.exe#/dl.7z is not valid
Could not install<EMAIL_ADDRESS>Select-CurrentVersion : Cannot bind argument to parameter 'AppName' because it is an empty string.
At C:\Users\achimgaedke\scoop\apps\scoop\current\lib\core.ps1:239 char:54
+     return $null -ne (Select-CurrentVersion -AppName $app -Global:$gl ...
+                                                      ~~~~
    + CategoryInfo          : InvalidData: (:) [Select-CurrentVersion], ParameterBindingValidationException
    + FullyQualifiedErrorId : ParameterArgumentValidationErrorEmptyStringNotAllowed,Select-CurrentVersion

You cannot call a method on a null-valued expression.
At C:\Users\achimgaedke\scoop\apps\scoop\current\lib\manifest.ps1:36 char:5
+     $app = $app.TrimStart('/')
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
    + FullyQualifiedErrorId : InvokeMethodOnNull

Couldn't find manifest for ''.

Steps to Reproduce
PS> scoop install<EMAIL_ADDRESS>
Possible Solution
Sorry, I'm not very familiar with debugging powershell apps.
Scoop and Buckets Version
scoop --version
Current Scoop version:
v0.3.1 - Released at 2022-11-15

'extras' bucket:
dc76cf6cf (HEAD -> master, origin/master, origin/HEAD) sidekick-browser: Update to version <IP_ADDRESS>304-816141b

'main' bucket:
cfc1c8fcf (HEAD -> master, origin/master, origin/HEAD) red-view@20nov23: Fix hash (Closes #5288)

Scoop Config
scoop config

last_update                       aria2-enabled scoop_branch scoop_repo
-----------                       ------------- ------------ ----------
2023-11-21T11:52:44.5322355+13:00          True master       https://github.com/ScoopInstaller/Scoop

PowerShell Version
> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      5.1.22621.2506
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.22621.2506
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Additional Softwares
No response

This feature is not guaranteed to be 100% available

A known and documented issue.


Any plan to open source your Web Interface?

Try app.arukas.io yesterday, it's really easy to use.  Would you open source web ui? So people can use it on other devices like raspberry pi.

Hi @yangxuan8282, that's a good question!
We are considering making the Arukas Web UI open source.
But there are lots of other tasks to do.
I think it will take more than a year to open source the Web UI.

Great, wait for run it on rpi4 or rpi5.

@uzyexe hi, how did your implement Endpoint on app.arukas.io (select a prefix for domain name), like gogs.arukascloud.io, can I do this on my local docker ? It's really convenience. I guess it's something like SkyDNS, but not sure.

@yangxuan8282 It can not be done with Docker alone.
Arukas is Mesos and Marathon based docker container hosting.
Arukas Endpoint is a Marathon-LB based load barancer.
Design:
http://www.slideshare.net/sakura_pr/201608-mesos-studypublic-64721613/42
I wish for it to help you.

hi @uzyexe , thanks for reply. In second link, there is a demo: https://github.com/uzyexe/mesos-marathon-demo
If this demo can provide Endpoint and webui like Arukas?
And do you think this demo is overkill for a single raspberry pi 3( 4× ARM Cortex-A53, 1.2GHz; 1G RAM)
I just need Endpoint and webui, can I strip some service?

@yangxuan8282
For the Mesos/Marathon + marathon-lb, recommend a minimum of 8 GB of memory + swap space.
Mesos / marathon is suitable for requirements to host more than 10,000 containers.
However, it does not have a User-Like WebUI.
If the number of containers you host is less than 100 containers, I think that kubernetes (k8s) is suitable.
Kubernetes has an endpoint loadbalancer called a service router. There will also be WebUI.
Or you may consider using RancherOS.


Refactor FastAPI backend with router

Current
The backend entrypoint in /backend/service/listener.py is one large file
Requirement
Modularize the different API paths with a FastAPI router. A possible way to modularize for now

File/Folder creation, deletion, and renaming in one place
All other API paths in another place

Hello, I would like to ask if I could work on this issue.

Hi @malhotra5 , i happy to contribute, can you please assign me for #9

Hi @SalimNhiri, that you for contributing! I've assign this issue to you. Here is a guide for making contributions in case it helps.


Figure out how to deal with passwords between CERN and the tev machine

Looks like we need a key...

We need to do it in two days I think

When we want to run an scp from the CERN machines to the tev machines. We might be able to do this from inside lxplus, using it as a way-station.
When we want to run grid download commands into the LLP disk directly.

We have to implement i/o and command response, and make sure we can get to and from machines.

Make sure it can deal with the unkown question - that it isn't known to SSH and see how we can suppress that. Then we are done.


chore(parser): add --subservices tag

Context
Prowler allows to execute checks from a specific service using --service / --services tag
Description
This pr adds the --subservice / --subservices tag in Prowler. This feature allows you to run checks for a specific subservice(s)
License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

And we need to differentiate the service for each subservice.

Are we going to allow the use of --service and --subservice in the same execution? I think it's important just to check an specific subservice instead of all.

We are closing this issue. We are not sure about the subservice concept since it was crafted by us and maybe it should be replaced by the ResourceType. We will circle around this again in the future.
Thanks for your effort @pedrooot 👏


Bug

If u use thingamajig and wubium u get QnQDR idno if it was intended or its a bug im ganna report it

This is an intended feature. It was included in the Buff update


Use JsonSettings for date formatting and casing

This fixes https://github.com/NancyFx/Nancy/issues/1607
I wasn't sure how we wanted the settings from  JsonConvert.DefaultSettings() and JsonSettings to take priority so that might need to be revised a bit.
It'd be helpful if Json.IsJsonContentType() could be made public since all of the serializers have their own copy which is out of sync.

@xt0rted - quick question. Can you show some example code how this PR can be used / is helpful? I was going to create another issue and wasn't sure if this PR handles my use case / scenario already.

@PureKrome by default the json.net serializer doesn't use the same default settings as the built-in one, it's also not controllable via Nancy.Json.JsonSettings. This meant if you wanted camelCase you'd have to add your own json.net serializer and override the one in your IoC container of choice like so:
public class CamelCaseJsonSerializer : JsonSerializer
{
    public CamelCaseJsonSerializer()
    {
        ContractResolver = new CamelCasePropertyNamesContractResolver();
#if DEBUG
        Formatting = Formatting.Indented;
#endif
    }
}

// Registration with StructureMap
public class RepositoryRegistry : Registry
{
    public RepositoryRegistry()
    {
        For<JsonSerializer>()
            .Use<CamelCaseJsonSerializer>();
    }
}

That's not very SDHP.
With these changes camelCase is now the default (a breaking change) since that's what Nancy's default is and then it could be turned off simply by setting JsonSettings.RetainCasing to true. As @khellang pointed out in the original issue the setting name should be changed, but that was left for another PR.
You'd also able to use JsonSettings.ISO8601DateFormat to toggle between the ISO format and Microsoft's "\/Date(1333645844276-0600)\/" format.
The last thing this added was the ability to use json.net's JsonConvert.DefaultSettings() builder in the event you have an existing setup that's used outside of Nancy (such as in other middleware or 3rd party components). Then the JsonSettings would be applied as needed.
Aside from review this also needs a good rebasing.

This meant if you wanted camelCase you'd have to add your own json.net serializer and override the one in your IoC container

I'm doing the exact same code and it's very very annoying.

That's not very SDHP.

The last thing this added was the ability to use json.net's JsonConvert.DefaultSettings()

Er.. can you show some sample code, what this means and why this might be helpful, pwease?

@PureKrome I don't use this (yet) but here's a modified version of one of my OWIN Startup classes that should help show it in use.
public partial class Startup : RaygunStartup
{
    public override void SetupConfiguration(IAppBuilder app, RaygunSettings settings)
    {
        JsonConvert.DefaultSettings = () =>
            new JsonSerializerSettings
            {
                NullValueHandling = NullValueHandling.Ignore,
#if DEBUG
                Formatting = Formatting.Indented,
                TraceWriter = new DebugJsonLogger(), // uses System.Diagnostics.Debug
#else
                TraceWriter = new JsonLogger(), // uses SimpleLogging
#endif
            };

        app.UseRaygun(options => { /* ... */ });
        app.Use<CleanHeadersMiddleware>();
        app.Use<HeartbeatMiddleware>();
        app.Use<SafeCrawlingMiddleware>();
        app.Use<PlaceholderMiddleware>();
        app.UseSimpleAuthentication();
        app.UseNancy(options => { /* ... */ });
    }
}

Given this setup as long as everything creates their json.net serializer like so then the system-wide settings will be used as the starting point.
var settings = JsonConvert.DefaultSettings();
var serializer = JsonSerializer.Create(settings);

So given the following model in debug mode:
var model = new UserData
{
    Id = 1234,
    Name = "A Test",
    Description = null
};

You'll get the following json in response:
{
  "Id": 1234,
  "Name": "A Test"
}

Along with json.net's logging written out to one of the custom loggers:
Info - Started serializing UserQuery+UserData. Path ''.
Info - Finished serializing UserQuery+UserData. Path ''.
Verbose - Serialized JSON:
{
  "Id": 1234,
  "Name": "A Test"
}

The response from Nancy will be slightly different and that's because we're overriding some of the settings with those from JsonSettings. This means the response coming from Nancy would actually look like:
{
  "id": 1234,
  "name": "A Test"
}

Here you get camelCase which came from JsonSettings but you also get the indented formatting, description is left out since it's null, and logging written to your custom logger.

These changes are to help with consistent defaults and ease of configuring common settings. If you want to use a custom ContractResolver that say filters out all SuperSecretId properties from your models then you'll need to do some overriding, but for most things you can now use Nancy's JsonSettings and json.net's JsonConvert.DefaultSettings.
To get around the overriding we could do one of the following:

Skip applying Nancy's settings if JsonConvert.DefaultSettings is to be used
Make the DefaultSerializer helper function public static similar to the one in json.net and let the user do whatever they want if they don't like the defaults

I'll rebase & update once I know what direction we want to go.

Gotcha!
(and thanks for answering the q with code and explanation, etc).

Hi, could you please rebase this? kntxbai

@thecodejunkie  rebased. Hopefully I didn't miss anything.

@thecodejunkie I think the project I originally tested this on wasn't configured correctly. What I'm now seeing is even though JsonSerializer isn't registered with StructureMap the default constructors on JsonNetSerializer & JsonNetBodyDeserializer aren't being used.
If this is how all of the IoC containers are going to function then I don't think we can have JsonSerializer as a parameter unless we override its settings in each class. I'm also not sure why there's default constructors since they're not used anywhere (aside from the unit tests).
If I remove the constructors with the serializer parameter then everything works as intended and you can use Nancy's JsonSettings and Json.Net's JsonConvert.DefaultSettings to configure your serializers further. Is this the route we want to go?
Instead of making the Helper class public would it be better to rename Json.cs to something like JsonUtility and then make this instance of IsJsonType public? Nancy.Serialization.ServiceStack  has the same helper.


bqplot stuck on pip install/debian jessie 8

I'm trying to install bqplot on debian jessie 8 using the command sudo -H pip install bqplot. After collecting dependencies the installation process just gets stuck i.e. never complete the installation.
Collecting bqplot
    Downloading bqplot-0.9.0-py2.py3-none-any.whl (2.2MB)
       100% |████████████████████████████████| 2.3MB 67kB/s
Collecting numpy>=1.10.4 (from bqplot)
    Downloading numpy-1.12.1.zip (4.8MB)
       100% |████████████████████████████████| 4.8MB 33kB/s
Collecting pandas (from bqplot)
Downloading pandas-0.20.2.tar.gz (10.3MB)
       100% |████████████████████████████████| 10.3MB 15kB/s
Then it goes on forever, please advise.

Nevermind, it turns out it just take some really long time to finish the task.

Sorry for the late reply. I think that the process appeared to be hanging because of the npm build (bundling of the javascript).
This can be really really slow if you are building e.g. from a raspberry pi, and just a minute on a regular computer.

Thanks for reporting!


samply: 0.11.0 -> 0.12.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/mstange/samply/releases.
meta.description for samply is: A command line profiler for macOS and Linux
meta.homepage for samply is: https://github.com/mstange/samply
meta.changelog for samply is: https://github.com/mstange/samply/releases/tag/samply-v0.12.0
Updates performed

Rust version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.12.0 with grep in /nix/store/5fw8gyrply2ph86w7dd96rrdgfb1560n-samply-0.12.0
found 0.12.0 in filename of file in /nix/store/5fw8gyrply2ph86w7dd96rrdgfb1560n-samply-0.12.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
samply

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5fw8gyrply2ph86w7dd96rrdgfb1560n-samply-0.12.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A samply https://github.com/r-ryantm/nixpkgs/archive/4f683916db67d7ce882bd79109d275dbb2463f61.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/4f683916db67d7ce882bd79109d275dbb2463f61#samply

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5fw8gyrply2ph86w7dd96rrdgfb1560n-samply-0.12.0
ls -la /nix/store/5fw8gyrply2ph86w7dd96rrdgfb1560n-samply-0.12.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

samply

Maintainer pings
cc @figsoda for testing.

Result of nixpkgs-review pr 304429 run on x86_64-linux 1

1 package built:

samply


Missing '%matplotlib inline' on several notebooks

Informations

Qiskit version: 0.12.1
Python version: 3.7.4
Operating system: macOS Mojave v10.14.6

What is the current behavior?
Matplotlib figures do not get plotted unless local ipython configuration files contain the instruction to make matplotlib = "inline". The affected notebooks (inline required but not present) are:

[ ] ./qiskit/fundamentals/2_plotting_data_in_qiskit.ipynb
[ ] ./qiskit/advanced/terra/2_operators_overview.ipynb
[ ] ./qiskit/advanced/terra/3_advanced_circuit_visualization.ipynb
[x] ./qiskit/advanced/terra/4_transpiler_passes_and_passmanager.ipynb
[ ] ./qiskit/advanced/aer/1_aer_provider.ipynb
[x] ./qiskit/advanced/aer/2_device_noise_simulation.ipynb
[ ] ./qiskit/advanced/aer/3_building_noise_models.ipynb
[ ] ./qiskit/advanced/aer/4_custom_gate_noise.ipynb
[ ] ./qiskit/advanced/aer/6_extended_stabilizer_tutorial.ipynb
[ ] ./qiskit/advanced/ignis/1_calibrating_a_qubit.ipynb
[ ] ./qiskit/advanced/ignis/4_measurement_error_mitigation.ipynb
[ ] ./qiskit/advanced/ignis/5a_randomized_benchmarking.ipynb
[ ] ./qiskit/advanced/ignis/5b_interleaved_rb.ipynb
[ ] ./qiskit/advanced/ignis/5c_purity_rb.ipynb
[ ] ./qiskit/advanced/ignis/7_quantum_volume.ipynb
[ ] ./qiskit/advanced/aqua/artificial_intelligence/qsvm_classification.ipynb
[ ] ./qiskit/advanced/aqua/finance/simulation/credit_risk_analysis.ipynb
[ ] ./qiskit/advanced/aqua/optimization/vehicle_routing.ipynb

Steps to reproduce the problem
Reset the ipython configuration files if necessary and run any affected notebook.
What is the expected behavior?
Figures should show up when the notebooks are run.
Suggested solutions

[ ] Add the %matplotlib inline magic to the beginning of the affected notebooks.
[ ] Include a brief reference to this behavior in ./qiskit/fundamentals/2_plotting_data_in_qiskit.ipynb.
[ ] Include a note on how to set the ipython local configuration files (stackoverflow) in ./qiskit/fundamentals/2_plotting_data_in_qiskit.ipynb and/or ./qiskit/advanced/terra/3_advanced_circuit_visualization.ipynb.

It is technically not needed anymore on recent releases of Jupyter.

Great news! Thank you for the update. I am closing this issue then, feel free to reopen if necessary.


Expected Delta fails to low temp when BG is falling but not as fast as expected

For example, here a low temp basal was ended because Avg Delta > Expected Delta, even though Eventual BG was much lower than min set.

Think we can close this since #47 was merged


Statistic chooser does not register that the user already set the different parameters

When the user switches back to the statistic chooser from the statistic he created, all parameters (except the dates) are preset to the values he put in earlier. The problem is that the system behind the ui does not register that and tells the user he did not select anything if he tries to recreate his statistic with that settings. You can even reselect all of the already selected parameters which causes a lot of confusion.

Not recreatable in the current master branch. This used to be an issue but was fixed in Pullrequest #95. You probably got this error in a Branch not up-to-date with master

Not recreatable in the current master branch. This used to be an issue but was fixed in Pullrequest #95. You probably got this error in a Branch not up-to-date with master


Link to specific items in FAQ and Troubleshooting guide

At the moment one can not link to specific items (questions/topics) in FAQ or Troubleshooting guide:
https://keda.sh/docs/2.6/faq/
https://keda.sh/docs/2.6/troubleshooting/
Would be nice to add possibility to do that (ie. generate links for each item.
Use-Case
Access inidividual items in FAQ/Troubleshooting with links
Specification

[ ] Generate links for FAQ
[ ] Generate links for Troubleshooting guide

@chalin Is this something you can pick up as part of the rework, please?

@zroubalik - yes it can be. It would help if I knew the relative priority of issues -- see #727.

This issue has already been fixed.
PRs:

Feat: copy to clipboard for troubleshooting page
Feat: Copy to Clipboard for FAQ

I think we should close this one.
cc: @tomkerkhove

Yes indeed, thanks!


IOS Subscription problem without stack trace

Describe the issue
I have a problem, but stack trace is empty - It's hard to describe.
Work on subscription function in flutter.
I use the Subscription in my widget.
It's working properly on android. It's awesome! :)
But on IOS at the first build it's fine - subscription is okay yet. At next build cannot connect to backend. (I think build, handshake, connect, build widget and die but build, handshake, connect, disconnect, build widget sometimes working).
The problem only affects IOS.
I work on Django, Channels and Strawberry GraphQL.
To Reproduce

Open app
Navigate to Subscription widget
Go back
Navigate to Subscription widget - It's not working on IOS, cannot subscribe again. It should work on IOS too.
Stack trace is empty.

Expected behavior
Similar operation on Android and IOS.
Additional context
Flutter 3.10.5 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 796c8ef792 (4 weeks ago) • 2023-06-13 15:51:02 -0700
Engine • revision 45f6e00911
Tools • Dart 3.0.5 • DevTools 2.23.1
Getx - State manager. get: ^4.6.5)
Django 4.2.2 and 4.2.3
strawberry-graphql-django 0.9.5 and 0.10.6
Channels 4.0.0
Broker is RabbitMQ
Backend log:
`
| Fatal error: protocol.data_received() call failed.
| protocol: <carehare._protocol.Protocol object at 0xffff6edd6690>
| transport: <_SelectorSocketTransport fd=17 read=polling write=<idle, bufsize=0>>
| Traceback (most recent call last):
|   File "/usr/local/lib/python3.11/asyncio/selector_events.py", line 1009, in _read_ready__data_received
|     self._protocol.data_received(data)
|   File "/usr/local/lib/python3.11/site-packages/carehare/_protocol.py", line 118, in data_received
|     self._handle_buffer_frames()
|   File "/usr/local/lib/python3.11/site-packages/carehare/_protocol.py", line 170, in _handle_buffer_frames
|     self._handle_frame_or_crash(channel_id, frame)
|   File "/usr/local/lib/python3.11/site-packages/carehare/_protocol.py", line 184, in _handle_frame_or_crash
|     self.accept_frame(frame, channel_id)
|   File "/usr/local/lib/python3.11/site-packages/carehare/_protocol.py", line 212, in accept_frame
|     return self._accept_frame_after_handshake(frame, channel_id)
|            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|   File "/usr/local/lib/python3.11/functools.py", line 946, in _method
|     return method.get(obj, cls)(*args, **kwargs)
|            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|   File "/usr/local/lib/python3.11/site-packages/carehare/_protocol.py", line 220, in _accept_frame_after_handshake
|     channel.accept_frame(frame)
|   File "/usr/local/lib/python3.11/functools.py", line 946, in _method
|     return method.get(obj, cls)(*args, **kwargs)
|            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|   File "/usr/local/lib/python3.11/site-packages/carehare/_exchange_channel.py", line 52, in _
|     self._delivering.pop(tag).set_result(None)
| asyncio.exceptions.InvalidStateError: invalid state
`
Device / execution context
IPhone 14 Pro Max (16.4 IOS) (emulated) - Second subscription problem
Android SDK 33 - working properly

This is hard to reproduce any helps?

Okay, I try, so
My backends subscription is:
@strawberry.subscription
    async def inventory_count_header(self, info: Info, inventory_count_header_id: int) -> AsyncGenerator[
        InventoryCountHeaderWithInventoryItemsType, None]:
        ...

        ws = info.context.ws
        channel_layer = ws.channel_layer
        group_name = ...

        await channel_layer.group_add(group_name, ws.channel_name)

        await channel_layer.group_send(
            group_name,
            {
                'type': ...,
                'operation': ...,
                'id': ...,
            },
        )

        async for message in ws.channel_listen(..., groups=[group_name]):
            operation: Operation = message['operation']

            ...

            yield result

and on flutter:
...

class InventoryCountHeaderWidget extends InventoryCountHeaderTabWidget {
  const InventoryCountHeaderWidget({super.key, required this.inventoryCountHeaderId});
  final int inventoryCountHeaderId;

  @override
  Widget build(BuildContext context) {
    return GetBuilder<InventoryCountHeaderController>(builder: (controller) {
      return Subscription(
          options: SubscriptionOptions(
            document: controller.inventorySubscription(),
            variables: {'inventoryCountHeaderId': inventoryCountHeaderId},
          ),
          builder: (result) {
            Widget w;
            if (result.hasException) {
              w = ...;
            }
            else if(result.data?['inventoryCountHeader'] == null) {
              w = ...;
            }
            else if(result.data?['inventoryCountHeader'] != null && ...) {
                w = ...;
            }
            else {
              w = ...;
            }

            return w;
    });
  }
}

I use GetX Statemanager.

1. Navigate to InventoryCountHeaderWidget, it's working. I get data from subs.
2. I go back (Get.back())
3. Navigate again to InventoryCountHeaderWidget and it cannot reconnect to subs (I think InventoryCountHeaderWidget stay in memory maybe stay connected and reload not working properly)

Seems connection is alive, result is: QueryResult(source: QueryResultSource.loading, data: null, context: Context({}), exception: null, timestamp: 2023-07-13 09:21:49.702098) but nothing more. Its stucked - Can not work backend with current connection. -  I think its consequence from backend, but there are something in after first build on IOS.

I know it's not much, but for now I'm also groping in the dark without an error message.

https://drive.google.com/file/d/1lweiw8_yFtPqXVZX0O7ZbuYsO9l_Zhfg/view?usp=sharing

I close the ticket, because I changed the channel layer and now it's working with android and ios.
Strange, but I think it can be a solution.
Thank you for your time!

Can you tell what you mean by  channel layer?


feat(FX-4152): position account above inbox in mweb nav

The type of this PR is: Feat

This PR solves FX-4152
Description
Moves account before inbox on mobile web navbar

The ticket also mentioned placing the > chevrons in the menu on the right side but if I remember correctly we decided not to proceed with that due to overlapping issues with the x button of the menu bar right @dimatretyak ?
Yes, that's right


Material support

Any plans on making of materials export support?

Unfortunately not. I'm not as active with Three.js as I used to be, so I guess someone else has to carry the torch forward.  Sorry

Maybe we can. Can you at least point is where to look in your code and
where to start from?
On Mar 6, 2017 8:32 PM, "David Spohr"<EMAIL_ADDRESS>wrote:

Unfortunately not. I'm not as active with Three.js as I used to be, so I
guess someone else has to carry the torch forward. Sorry
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BlackDice/Cinema4D-Threejs-Exporter/issues/3#issuecomment-284487711,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHTX1davhXIaekvae55EyC485xNOCh0Kks5rjFFUgaJpZM4MSU-u
.

It's similar to all the other types of data.
In exporter.py try to get materials from base document with this function:
https://developers.maxon.net/docs/Cinema4DPythonSDK/html/modules/c4d.documents/BaseDocument/index.html?highlight=materials#BaseDocument.GetMaterials
Here is the reference for the C4D material:
https://developers.maxon.net/docs/Cinema4DPythonSDK/html/modules/c4d/C4DAtom/GeListNode/BaseList2D/BaseMaterial/Material/index.html?highlight=material
Not sure if it has all the data that would be needed.
Then you have to figure out which materials are active on the mesh, probably by tags on the active object.
https://developers.maxon.net/docs/Cinema4DPythonSDK/html/modules/c4d/C4DAtom/GeListNode/BaseList2D/BaseObject/index.html?highlight=object#BaseObject.GetTags
Then you have to figure out how to parse the data to json into values that three.js understands.
If you can make that happen, I can probably find some time to do the UI work to finish it off.


Update composer.json

This makes the installer fetch the latest version of Wordpress upon Bedrock installation.

x-ref: #297


Minor issue: ECMAScript regex literal flags can be unicode escapes but TS doesn't support this

Was playing around with regular expression literals in JavaScript, and noticed that TypeScript doesn't support unicode escape sequences in position of regex flags. Of course, this is a very minor issue, but TypeScript is supposed to be a syntactic superset of ECMAScript.
Regex flags production is:
 RegularExpressionFlags ::
          RegularExpressionFlags IdentifierPart

IdentifierPart production is:
IdentifierPart ::
          UnicodeIDContinue
          $
          _
          \UnicodeEscapeSequence
          <ZWNJ>
          <ZWJ>

Code
var z = /aa/\u0075;   // should be equivalent to '/aa/u'

Expected behavior:
Code is valid.
Actual behavior:
Code is non-compilable.

In general, we don't down-level compile regular expressions, but we should at least parse them.

@DanielRosenwasser This isn't a downlevel issue, even, this is a parse issue. Reading more of the spec than mentioned here, we _should parse it, but:
<IP_ADDRESS>Static Semantics: Early Errors
RegularExpressionFlags::RegularExpressionFlagsIdentifierPart
It is a Syntax Error if IdentifierPart contains a Unicode escape sequence.


Add a LSTM-CRF model at Conlll2003 Dataset

Description
Add the LSTM-CRF model for Conll2003 dataset at reproduction dir based on fastNLP lib, inspired by the paper https://arxiv.org/pdf/1508.01991.pdf
Main reason
Provide a new demo for how fastNLP can facilitate the development of the deep learning model. FYI:
https://github.com/hazelnutsgz/fastNLP/tree/hazelnutsgz-crf-lstm/reproduction/LSTM-CRF
Checklist  检查下面各项是否完成
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [$CATEGORY] (例如[bugfix]修复bug，[new]添加新功能，[test]修改测试，[rm]删除旧代码)
[x] Changes are complete (i.e. I finished coding on this PR)  修改完成才提PR
[x] All changes have test coverage  修改的部分顺利通过测试。对于fastnlp/fastnlp/的修改，测试代码必须提供在fastnlp/test/。
[x] Code is well-documented  注释写好，API文档会从注释中抽取
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change  修改导致例子或tutorial有变化，请找核心开发人员

Changes

Interactive jupyter notebook
Well-structured codebase for training & testing
A README file for the instruction

Mention:
@yhcc @xpqiu @FengZiYjun @2017alan

Is the util file missing here? When I run it, I am prompted that there is no load_data function.

Yes, we don't have load_data function. You may use an old version.


Can't use syllables  or ku suli words

I cant use syllables  or ku suli words

@gtbot2007, I couldn't find it documented anywhere, but for syllables you can capitalise the first letters to have them interpreted as syllables:
eg: sitelen SiTeLen
will display the word-glyph for sitelen followed by the 3 syllables in a cartouche, to show the difference. I think that addresses half of this issue...


The type 'Dynamic' is missing, is there any replacement?

Describe the bug
It seems that the type 'Dynamic' was removed in the new version.
Is there any reason, or is there any replacement?
It is quite useful when serialize the dynamic data to database.

Yes, the Dynamic type has been dropped starting from 3.0.0 because it created a consequent overhead when writing models, and especially when reading them. It severely impacted performance when using large objects and/or arrays.
We now instead prefer declaring objects, and implicitly using JsonElements/JsonNodes, thus relying on the native and extremely performant System.Text.Json serializer.
If need arises, I could however easily add a new Dynamics package with a couple of adjustments.
Does that answer your question?


where is backend code?

Sir，where is node.js code or C# .net backend code?
I am little confonused about coffeeViewModel to add userid to coffee to backend database!!!
could you please github your backend code,sir?

i just found it!
C# .net backend:
var claimsPrincipal = this.User as ClaimsPrincipal;
string sid = claimsPrincipal.FindFirst(ClaimTypes.NameIdentifier).Value;
item.UserId = sid;

https://azure.microsoft.com/zh-cn/documentation/articles/app-service-mobile-dotnet-backend-how-to-use-server-sdk/#user-info


use yaml format for permissions and users instead of json

Coverage decreased (-0.005%) to 96.359% when pulling 425f70a22163845f83701cd818da3ffe9476ae6a on feature/yaml-everywhere into 73bff4a75479de5500ec5fd56f172666563b10f8 on master.

Coverage decreased (-0.005%) to 96.359% when pulling a3d53433f8adb5631f870eda294601ba7e0d0aa0 on feature/yaml-everywhere into 770a1898a0f905e63863c63485a4b9c5eb993f35 on master.

Coverage decreased (-0.005%) to 96.359% when pulling cb55b7df7e138d4bdff12e4695255e157cdbf22d on feature/yaml-everywhere into 770a1898a0f905e63863c63485a4b9c5eb993f35 on master.

Coverage decreased (-0.08%) to 96.395% when pulling 42baad24f7b09f13d95ff4c420f681c12d5f7c2e on feature/yaml-everywhere into 770a1898a0f905e63863c63485a4b9c5eb993f35 on master.


[Documentation] Render project links in PyPI page

Type

[ ] Content inaccurate
[x] Content missing
[ ] Typo

URL

https://pypi.org/project/shocktube1dcalc/
Description

We may want the following links in the PyPI page

homepage (github project)
issue tracker
code
documentation

By tweaking pyproject managed by poetry we could complete this task https://python-poetry.org/docs/pyproject/

Will the document be generated in a typeset form?  Maybe by CI or something else?

Will the document be generated in a typeset form? Maybe by CI or something else?

I did not follow what you meant by "typeset form". Poetry will generate meta-data for the package. I am not sure which format poetry generated, and we could have a look later on.

Oh sorry I misunderstood what you meant for the [Documentation] tag in the subject line.  You are updating the information that will go to PyPI.
What I meant for typesetting is something like https://yyc.solvcon.net/en/latest/writing/2015/euler.html to be generated somewhere.  I think a solver of a specific problem needs pages to document the equations and visualization of the results and they should be typeset.  Do you have a plan for doing that?

I don't have such plan. However, it sounds a good idea to me!
Maybe correlates this old slack thread :)  https://solvcon.slack.com/archives/C030M15GZ/p1553560440024400


having issue compile a shell program and it is bug likely

gen_isr_tables.py: error: IRQ 155 (offset=0) exceeds the maximum of 149
the soc I am using has IRQ 155, stm32h723zg
how do I solve it?

In the file Kconfig.defconfig.stm32h723xx you need to change NUM_IRQS to 163


unable to resolve workspace folder

Type: Performance Issue
i am getting this pop up when in run my files.unable to resolve workspace folder
VS Code version: Code 1.79.2 (695af097c7bd098fbf017ce3ac85e09bbc5dda06, 2023-06-14T08:57:04.379Z)
OS version: Windows_NT x64 10.0.22621
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-9300H CPU @ 2.40GHz (8 x 2400)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
7.82GB (2.87GB free)

Process Argv
--crash-reporter-id 9777b91a-ca52-491a-89ee-2b02cbd86745

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
    1	   232	  8940	code main
    0	    75	  2536	ptyHost
    0	    27	  3280	   crashpad-handler
    0	    42	  6000	   utility-network-service
    0	    74	  7088	fileWatcher [1]
    0	   158	  7612	window [1] (client.py - Visual Studio Code)
    0	   155	  9492	   gpu-process
    1	    87	 14840	window [2] (Issue Reporter)
    0	   151	 14956	extensionHost [1]
    0	    30	 11456	     d:\anaconda\python.exe c:\Users\praka\.vscode\extensions\ms-python.isort-2022.8.0\bundled\tool\server.py
    0	    10	 11792	       console-window-host (Windows internal process)
    0	   135	 15512	     electron-nodejs ("C:\Users\praka\AppData\Local\Programs\Microsoft VS Code\Code.exe" --ms-enable-electron-run-as-node c:\Users\praka\.vscode\extensions\ms-python.vscode-pylance-2023.6.20\dist\server.bundle.js --cancellationReceive=file:5cd6c5c6cf5f380fde8ef0ac6e33cdbb9b3b98c5a0 --node-ipc --clientProcessId=14956)
    0	    94	 15768	shared-process

Workspace Info
;

Extensions (18)

Extension
Author (truncated)
Version

doxdocgen
csc
1.4.0

opencv-intellisense
did
0.0.4

better-cpp-syntax
jef
1.17.2

isort
ms-
2022.8.0

python
ms-
2023.10.1

vscode-pylance
ms-
2023.6.20

jupyter
ms-
2023.5.1001582324

jupyter-keymap
ms-
1.1.2

jupyter-renderers
ms-
1.0.15

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

cmake-tools
ms-
1.14.33

cpptools
ms-
1.16.0

cpptools-extension-pack
ms-
1.3.0

ruby
reb
0.28.1

cmake
twx
0.0.17

vscode-lldb
vad
1.9.2

vscode-ruby
win
0.28.0

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
2e4cg342:30602488
pyind779:30671433
89544117:30613380
pythonsymbol12:30671437
2i9eh265:30646982
showlangstatbar:30737416
vsctsb:30748421
pythonms35:30701012
03d35959:30757346
pythonfmttext:30731395
pythoncmv:30756943
fixshowwlkth:30730052
pythongtdpath:30769146
bgfeh915:30769767
dh2dc718:30770000

/gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Sync Package to Home Directory By Default

#1090  👈
main

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

Join @dongreenberg and the rest of your teammates on  Graphite


Commercial support active?

Hi,
I'm looking for commercial support and I emailed the address<EMAIL_ADDRESS>Haven't heard back after a couple of days. I just wanted to check that's still active & still the best way to get commercial support?
Thanks.

@dbruning please accept my apology, you now have a reply over there.


Tika 1.7 support

Hi, Any plans for adding Tika 1.7 support ?
Thanks,
Wazhar

Yes this was on my stack, thank you very much for remembering. My tests are all green, I deployed it. If you find any issues, I would appreciate if you can post it. Enjoy!


Change display name for issuer on the connection, credential, and offer objects

Is it possible to change the name used on the objects and notifications that winf up in StreerCred so they use the label BCGov-POC-Issuer  instead of identity-kit-agent?https://app.zenhub.com/files/219878732/42b0d59e-b074-4d37-8efe-e9dc818e3be8/download

@esune - can you please drive this?  Basically, it means changing the name of the schema, I think. I assume there is likely a configuration change to be done in concert in the IDK controller/agent.
Please make sure that in doing this you also update the Keycloak realm proof request to accept this schema in addition to the old one that this will replace, so existing credentials that have been issued from the IDK service will continue to work.
Thanks

I believe the name corresponds to the agent label, I have updated it to use BCGov-POC-Issuer. It may be necessary to establish a new connection for the name tyo update in the app, I am not sure how that works.
No proof-request configuration update is therefore required.

Ah...good to know.  Thanks.  That does make sense.
On Fri, Dec 13, 2019 at 10:16 AM Emiliano Suñé<EMAIL_ADDRESS>wrote:

I believe the name corresponds to the agent label, I have updated it to
use BCGov-POC-Issuer. It may be necessary to establish a new connection
for the name tyo update in the app, I am not sure how that works.
No proof-request configuration update is therefore required.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bcgov/identity-kit-poc/issues/93?email_source=notifications&email_token=AAHYRQXUYCIYEIEUL76UHJ3QYPGOHA5CNFSM4J2FU2Q2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEG2ZSZI#issuecomment-565549413,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHYRQWY264WOIOQYEZ7V53QYPGOHANCNFSM4J2FU2QQ
.


Make allowed ports for registerTemplate api configurable

ISSUE TYPE

Enhancement Request

COMPONENT NAME

API

CLOUDSTACK VERSION

4.11

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

Currently, Cloudstack restricts urls for registerTemplate to a small set of ports.  Would be nice to make this configurable to allow an admin to configure the allowed ports.  In my case, I have a server listening on 8081 which I do not have control over. I'd like to be able to configure Cloudstack to allow this port.
STEPS TO REPRODUCE

EXPECTED RESULTS

ACTUAL RESULTS

@csquire can you post here the screenshot of the error? Or, maybe the log entries with the exception? Or the error message  you received in the client side?

@rafaelweingartner Yeah, it returns a 403 and shows Forbidden in a browser. The behavior is intentional, it's not a bug per se.
Offending code is here:
https://github.com/apache/cloudstack/blob/master/plugins/integrations/prometheus/src/main/java/org/apache/cloudstack/metrics/PrometheusExporterServerImpl.java#L51-L59

I am confused now. Are you talking about the code that register templates or the code that handles Prometheus integration?

Sorry, last comment was posted in the wrong issue, I deleted it.
You just get an error back that your port is not allowed Only ports 80, 8080 and 443 are allowed
https://github.com/apache/cloudstack/blob/master/utils/src/main/java/com/cloud/utils/UriUtils.java#L267-L270

@csquire Will removing port checking be an appropriate solution for this or should there be some global settings where admin can set a list of allowed ports?
cc @rhtyd @nvazquez

It should be any port @shwstppr


Increment jenkins lib version to 1.5.2

Signed-off-by: Sayali Gaikawad<EMAIL_ADDRESS>Description
Increment jenkins lib version to 1.5.2. This version fixes the problem mentioned here: https://github.com/opensearch-project/spring-data-opensearch/pull/74#issuecomment-1370286856
PR with fix https://github.com/opensearch-project/opensearch-build-libraries/pull/102
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

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

@gaiksaya I've recreated the pre-release tag but Jenkins job didn't trigger this time ....

Hi @reta ,
I pushed one too. It failed too https://build.ci.opensearch.org/job/spring-data-opensearch-release/4/
Taking a look. Will delete the one I created.

Update:
The issue was the authorized GitHub user to read the draft release was not added to this repo. I did that and it went through. current we missed an env variable, which I am adding now to our jenkins env. 1hr tops ETA


Unexpected behavior when relaunching Enjoyhint

I created a function that gets called whenever I click a certain button to launch the Enjoyhint steps. However, I get weird behavior after the first click of the button.
I'm experiencing two types of behaviors:

If I click the 'X' on the top right corner, the next time I click the button to show the Enjoyhint steps, it starts the step right after the one before I clicked X, then moves on to the second over all step, and works fine thereafter.
If I go through all steps, the next time I click the button it does nothing. When I click it another time, it starts from the second step, and when I click next, it stays on that step and works fine thereafter.

Here's a shortened sample of the code I am trying to run:
$('#btn').click(function() {
//initialize instance
var enjoyhint_instance = new EnjoyHint({});
    //simple config.
    //Only one step - highlighting(with description) "New" button
    //hide EnjoyHint after a click on the button.
    var enjoyhint_script_steps = [
      {
          'click #a': 'random text a',
          showNext: true,
          showSkip: false
      },
      {
          'click #b': 'random text b',
          showNext: true,
          showSkip: false
      },
      {
          'click #c': 'random text c',
          skipButton: { className: "myNext", text: "Got it!" },
          showNext: false,
          showSkip: true
      }
    ];
    //set script config
    enjoyhint_instance.set(enjoyhint_script_steps);

    //run Enjoyhint script
    enjoyhint_instance.run();
});

Appreciate any help.

There's several event handlers that aren't re-registered properly, and values that don't get re-initialized in this version.  Take a look at my fork, I think I've fixed what's breaking for you.


Add a README to the examples/ directory

Explaining/introducing each example a little. Along with a structural test that checks they align.
Maybe there are also other directories that ought to have a README...

Just describing the examples in a README.md highlighted for me that some of the examples are weak sauce and should be replaced by better ones — i.e. #194.
The good news is that as soon as we do #324, we'll have that (ish), and we can close #194.


Changed the description to be compatible with Chromium Edge.

Changed the description from using classes for Legacy Edge  to using classes for Chromium Edge.

Docs Build status updates of commit b5d7e7c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/webdriver-chromium/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Hi @tatsuya-midorikawa - thanks for the contribution. I spoke to Engineering and we plan to make some updates to the docs by next week. I'll go ahead and close this PR for now because of those incoming updates.
Thanks again for the contribution!


NLD Translation

If you create blank report and edit after is not the maximum. If you can you can create report but with description. Thanks.


docs: updated Azure DevOps feature docs regarding installing Arcus packages in pipelines

Updated the Azure DevOps docs on how to install the Arcus packages inside your pipeline.
Previously, using -Force, this step could take up to 5 minutes.
Now, by registering the repository and using -AllowClobber step takes one minute.
Relates to #284

Thanks for changing this so quickly in these code-samples!
However, I think we should immediately close the related issue. In my humble opinion, we should have some docs which explicitly state that you can install the package this way

I have added a recommendation to use a Linux agent instead of a Windows agent in the text. Installing the package takes multiple minutes with Windows, but less than one minute for Linux.

I think @fgheysels meant, "we should not immediately close the issue" 😅 But Alex already took care of this by using 'Related to' instead. So, all is good. 👍

I think @fgheysels meant, "we should not immediately close the issue" 😅 But Alex already took care of this by using 'Related to' instead. So, all is good. 👍

I actually meant that we might need to add this somewhere in a more 'general' section of the docs as well ?

@spasal looks very good! As @fgheysels mentioned #285 (comment) it would be good to add the same code-sample to lines 21 to 23. Could you change it on lines lines 21 to 23 there as well to:
Set-PSRepository -Name PSGallery -InstallationPolicy Trusted
Install-Module -Name Arcus.Scripting.DevOps -Repository PSGallery -AllowClobber

Ahah, missed that one. Updated it, thanks for the suggestion!


fix: tpyo

PR Checklist

[ ] Addresses an existing open issue: fixes #000
[ ] That issue was marked as status: accepting prs
[x] Steps in CONTRIBUTING.md were taken

Overview

Oops you did it again!

@all-contributors please add @astorije for code.

🤖 Beep boop! This comment was added automatically by all-contributors-auto-action.
Not all contributions can be detected from Git & GitHub alone. Please comment any missing contribution types this bot missed.
...and of course, thank you for contributing! 💙


fix explorer's rpc override

something about the previous setup disabled the ?rpc= override, this fixes it and preserves the functionality.
Also removes unused extra types

actually Chris' changes may cover most of this


Syntax to break an expression over multiple lines?

The PRQL syntax currently usually (with the exception of lists?) treats newlines as terminating characters for statements. (And please do correct me if I'm wrong about this!)
For long expressions (and perhaps also statements?), it would be nice to have a syntax that would allow users to break that expression up over multiple lines.
This idea is motivated by the discussion in https://github.com/prql/prql/issues/788.
Is there a current way to do this? If not, would parentheses be the natural way to express this?

This is a good point.
I don't think we should need it that often — one of the benefits of PRQL should be to split up logic into modular pieces, represented by transforms on separate lines.
But it would be presumptuous to assume we can always do this, and #788 is a good case — even if the "split up into multiple filters" suggestion is reasonable, it doesn't cover the case of a long list of or-ed conditions.
Newlines have an important semantic meaning, so we've "used up" that syntax. So a couple of options:

Use \ to allow line breaks
Find the cases that force long lines and offer a solution to them (e.g. an any function to take a list of conditions and string them together with ors)

Any thoughts?

My comment around parentheses was wondering if you could do something like what Python does to "free up" the newline syntax inside of parentheses:

\ also seems like a decent solution to me

Yes, interesting. So we could have a rule that finishing the line on an operator (e.g. + / or / etc) causes a line continuation.
Are there any instances where we want to finish on an operator?
One concern is that while editing a line in the middle of a query, the error messages might be quite confusing:
from foo
derive bar = baz or<cursor>
aggregate (
  max fuz
)

...would this have a confusing error message saying "baz or aggregate is invalid"?

Yes, interesting. So we could have a rule that finishing the line on an operator (e.g. + / or / etc) causes a line continuation.

But that's maybe valid only inside a set of parens? (I'm not sure what the full Python parsing logic is.) Additionally, it might also be valid if the next line starts with an operator, which is that last Python example.

...would this have a confusing error message saying "baz or aggregate is invalid"?

"Unexpected newline"?

"Unexpected newline"?

The challenge is that it wouldn't be parsed as a newline, it would be parsed as derive bar = baz or aggregate (, unless I'm misunderstanding something?

But that's maybe valid only inside a set of parens?

Yes, this could be possible. It starts to get a bit complicated — copy something from inside parens to outside parens and it doesn't work any longer.
But python is good prior art, I think this should be OK.

Additionally, it might also be valid if the next line starts with an operator, which is that last Python example (inside parens).

Yes, good idea. We'll have to see whether pest supports this, I think it should be possible with positive predicates

Cool! And to be clear, I'm just offering my random thoughts as a fan in the peanut gallery. Appreciate all the work you've already done!

Cool! And to be clear, I'm just offering my random thoughts as a fan in the peanut gallery. Appreciate all the work you've already done!

Thoughts generally welcome, and in the specific case they're also great ideas @ajfriend !

I'd argue against trying to parse newlines after operators or inside parenthesis. I like \ far more, regardless of the fact that I find it quite ugly.
That's because newline (as a token) is already "used up" on higher levels of parsing, as @max-sixty said. Precedence goes roughly like:
newline, |
and, or
<, >, <=, >=
+, -
*, /
(), .., []

Simplified, tokens are first split by newlines and |, then by and and or, and so on.
Now, if we would want to ignore newlines in () or after +, we'd have to parse () and + higher in precedence list, which would add a lot of complexity.
But why can python support this? Python has two precedence lists: one for expressions and one for statements. Newline is a part of statement precedence list and it means "end of statement". And because statements cannot be part of expressions, and because ( indicates unfinished expression, Python can only use expression precedence list and resolve newline to mean "whitespace".
Back to PRQL, newlines can be part of expressions - actually the whole pipeline is an expression! That's why even in (), newlines must mean "pipe".
I came up with another idea: newlines meaning "pipe" make sense only in pipelines. Pipelines can be just top-level plain function calls, or function calls wrapped in (). But they cannot be function calls in []! Which means that without much hussle, I think we can make this work:
derive [
  some + long + expr +
    spans + multiple + lines
]

or
derive [some + long + expr +
    spans + multiple + lines]

I have a question, not about future multiline syntax, but about a potential workaround: perhaps in Prql 0.3.1, we can use currying to add arguments one at a time. Perhaps not.
This is not meant as a derailing feature request, only as a "can this workaround be made to work" query. If it can't no worries.
Alright, the workaround
# This function takes three arguments, so maybe you want to write it on multiple lines
func addthree<column> a b c -> s"a + b + c"

# 'arg x' takes a curried function and adds x to the end of its argument list.
# addthree a | arg b  -> addthree a b
# because newlines are equal to pipes, this allows building up a function's arguments over multiple lines
func arg myarg myfunc -> ( myfunc myarg )

from mytable
derive [
  my_thing = (  # a multiline expression
    addthree apples  # addthree apples _ _
    arg bananas  # addthree apples bananas _
    arg citrus  # `addthree apples banana citrus`, a.k.a. `apples + bananas + citrus`
  )
]

Compiling this with 0.3.1 gives an error (below) that I think I can summarize as "I expected a column expression, but I found a function with signature infer -> infer". Is it possible to write the code differently so it works in 0.3.1? If not, no worries.
# Linebreaks added. Character span not accurate.
Error {
  span: Some(span-chars-110-176),
  reason: Expected {
    who: Some("function std.derive, param `columns`"),
    expected: "type `column`",
    found: "type `func infer -> infer`"
  },
  help: None
}

Woah, a great workaround using currying! Maybe it was worth implementing :smile:
Your example should work and the fact that it doesn't is a bug. I'll look into it. There seems to be a problem with pipelines, because normal function calls do work (but defeats the whole purpose of the workaround):
func addthree<column> a b c -> s"{a} + {b} + {c}"

func arg myarg myfunc -> ( myfunc myarg )

from mytable
derive [
  my_thing = (
    arg "citrus" (arg "bananas" (addthree "apples"))
  )
]

Cancel that, even my example produces a wrong output.

Well, thank you very much for taking a look. PRQL is a beautiful thing. Have a good slide into the new year!

A thought about backslash \ for line continuations: it can interact badly with line comments.

In languages where \ at the end of a line indicates a line continuation ...
... and '#' comments all characters until the end of the line ...
... in those languages, you can't have line comments on multiline expressions.

For example, this is invalid bash:
tar \
  -x  # extract \
  -z  # gzipped \
  -f  # read from file \
  myfile.tar.gz

and so is this:
mycommand \
  | head --bytes=-2  # remove spurious CR LF from end \
  | tail +2  # remove header

Some potentional ways to allow comments AND line continuations, if that is important to you.
Contrived example featuring addthree. Find shows with free places, by adding up how many places each kind of ticket represents, and then comparing that to the theatre's capacity.
Flavour 1: 'backslash EOL' for line continuation. 'backslash space comment EOL' also for line continuation.  'EOL' for pipe.
 I prefer flavour 4 the most.
But as I said, backslash looks ugly (that's just my preference) and has to be adjusted when adding a new line at the end. It's like a trailing comma. And we cannot allow trailing backslashes...

I agree for Flavour 4! (I think the example is missing +s — is that right? Or I'm misunderstanding the example.)
Overall I'd be fine with this; it doesn't conflict with other syntax. I think we should try and reduce the amount it's needed — but sometimes it will be, as the example shows.
And agree with @aljazerzen on this point; specifically that it's not "composable", in the same way as a trailing comma isn't:

has to be adjusted when adding a new line at the end. It's like a trailing comma. And we cannot allow trailing backslashes...

If I can suggest something to the design — trailing spaces after the backslash should be ignored. In bash, \  does not count as a trailing backslash, which can be really confusing.

As discussed on Discord — are we broadly in agreement on this option?

Flavour 4: line continuations allow separate-line comments, not end-of-line comments,
from tickets_per_show
filter addthree \
    seats \
    # couches are double-sized
    couches * 2   \
    # standing people take less room
    standing / 2  \
    != max_capacity

Note:

(I think the example is missing +s — is that right? Or I'm misunderstanding the example.)

This does seem like the best option. Nevertheless, I'd prefer not add, for the reasons above.
I'm -0.5 on it.

How would you represent the a long expression @aljazerzen ? I agree the trailing backslashes are ugly! But is there a better way?
FWIW this is longer the case:

But they cannot be function calls in []! Which means that without much hassle, I think we can make this work:

Within lists, we could make this work:
from a
select [
  sum
  b,
  avg
  b
]

... and an idea: non-pipelining parenthesis!
from a
select ((
  sum
  b
))

... and an idea: non-pipelining parenthesis!

The abyss just stared back at me! (niche joke / reference!)

Within lists, we could make this work:

How about for a long expression with operators? I think that's the most likely long expression, rather than function calls. As above, we could allow ending a line with those (though it may create some odd error messages)?
I think that's possible. I'm not sure whether that's better than the trailing slashes.

I'm also OK saying "the current state isn't great. But the options we're looking at aren't good either, we're going to leave this open for a while and reflect, rather than implementing something bad", if we think that reflection might pay off.
Or "we're OK with X, but it's not a priority to implement for us, we'll accept a PR", which is an implicit filter without us getting in the way of anyone.
I think both of those are better than leaving something hanging for ages (as you pointed out on Discord / on the call!)

FWIW, my favourite is probably Flavour 1 and I'm not such a fan of Flavour 4 - it just seems harder to read to me (but I'm just speaking as a user, not an implementer).
Wouldn't Flavour 2 possibly address the "trailing comma" type issue as only the first line is different and that's special anyway. It wouldn't be my first choice as you can't tell on each line that there is more coming but to me it seems acceptable if it deals with the trailing commas issue. How would this flavour deal with commenting things out though?
Would the following work?
from tickets_per_show
filter addthree
    \ seats
    # \ couches * 2   # couches are double-sized
    \ couches * 3   # couches are triple-sized
    \ standing / 2  # standing people take less room
    \ != max_capacity

@max-sixty 's suggestion of "leave this open and accept a PR" seems good for now.

Would the following work?
from tickets_per_show
filter addthree
    \ seats
    # \ + couches * 2   # couches are double-sized
    \ + couches * 3   # couches are triple-sized
    \ + standing / 2  # standing people take less room
    \ != max_capacity

This actually might work! It makes the parser a bit ambiguous for one item, but that's OK I think.
Or are there things I'm missing?
(I'm going to take the executive decision of filling in the +, which I'm fairly convinced are missing in error and make the examples wrong, since no one corrected me above)

@max-sixty The reason there are no +s is because of the addthree function that was defined higher up (here).

snth is right. I felt that a long expression like seats + couches + standing risked a sidetrack into "a line ending in + is obviously incomplete" territory, hence why I made up addthree seats couches standing for the examples.