Things to fix on lis-gene-search from user feedback

Here is a list of things I think should be fixed before we put lis-gene-search into production, from user input:

[x] Results stats: show total number and which ones in current page display: Like "Displaying 1-10 of 39 genes".
[x] geneFamilyIdentifier in URL isn't being picked up; presumably a mismatch with the expected query variable. This should fix the back-arrow-from-linkouts problem as well.
[x] #117

Results stats: show total number and which ones in current page display: Like "Displaying 1-10 of 39 genes".

This is something that I've wanted to do for a while with paginated results in general. @sammyjava Is there a way to request this information when sending a pathquery to Intermine?

geneFamilyIdentifier in URL isn't being picked up; presumably a mismatch with the expected query variable.

Is there an example for replicating this behavior? It was working when I implemented querystring parameters so this sounds like a regression.

geneFamilyIdentifier in URL isn't being picked up; presumably a mismatch with the expected query variable.

Is there an example for replicating this behavior? It was working when I implemented querystring parameters so this sounds like a regression.

We discussed it, I'll fix it and submit a PR for you to ponder. Should be trivial.

Results stats: show total number and which ones in current page display: Like "Displaying 1-10 of 39 genes".

This is something that I've wanted to do for a while with paginated results in general. @sammyjava Is there a way to request this information when sending a pathquery to Intermine?

Dang, I thought there was something in the JSON that gave the total results count and current page, but I was wrong, it just gives you the start value. That's very problematic. PathQuery doesn't support aggregate queries (like SELECT count(*) FROM...) so this makes sense that they don't reflect it in the metadata. One just doesn't know how many results there are, the query uses Postgres pagination blindly.
I may need to hack something in the mine server to allow us to get the query results count. Could be hairy. That's all we need since we know the page size and the start value.
curl "https://mines.legumeinfo.org/legumemine/service/query/results?query=<query+name%3D""+model%3D"genomic"+view%3D"Gene.primaryIdentifier+Gene.geneFamilyAssignments.geneFamily.primaryIdentifier"+longDescription%3D""+sortOrder%3D"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier+asc"><constraint+path%3D"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier"+op%3D"%3D"+value%3D"legfed_v1_0.L_001QTQ"%2F><%2Fquery>&format=json&start=1&size=20"
{
"modelName":"genomic",
"columnHeaders":["Gene > Identifier","Gene > Gene Family Assignments > Gene Family > Identifier"],
"rootClass":"Gene",
"start":1,
"views":["Gene.primaryIdentifier","Gene.geneFamilyAssignments.geneFamily.primaryIdentifier"],
"results":[
["aradu.V14167.gnm1.ann1.Aradu.02ZKU","legfed_v1_0.L_001QTQ"],
["aradu.V14167.gnm1.ann1.Aradu.04ERG","legfed_v1_0.L_001QTQ"],
["aradu.V14167.gnm1.ann1.Aradu.079IW","legfed_v1_0.L_001QTQ"],
["aradu.V14167.gnm1.ann1.Aradu.07IAJ","legfed_v1_0.L_001QTQ"],
["aradu.V14167.gnm1.ann1.Aradu.07LFN","legfed_v1_0.L_001QTQ"],
["aradu.V14167.gnm1.ann1.Aradu.09M2C","legfed_v1_0.L_001QTQ"],
["aradu.V14167.gnm1.ann1.Aradu.14QPB","legfed_v1_0.L_001QTQ"],
["aradu.V14167.gnm1.ann1.Aradu.18EDI","legfed_v1_0.L_001QTQ"],
["aradu.V14167.gnm1.ann1.Aradu.1E4GW","legfed_v1_0.L_001QTQ"],
["aradu.V14167.gnm1.ann1.Aradu.1MU0C","legfed_v1_0.L_001QTQ"],
["aradu.V14167.gnm1.ann1.Aradu.1YQ1J","legfed_v1_0.L_001QTQ"],
["aradu.V14167.gnm1.ann1.Aradu.26UU3","legfed_v1_0.L_001QTQ"],
["aradu.V14167.gnm1.ann1.Aradu.2851E","legfed_v1_0.L_001QTQ"],
["aradu.V14167.gnm1.ann1.Aradu.2J3WV","legfed_v1_0.L_001QTQ"],
["aradu.V14167.gnm1.ann1.Aradu.2L303","legfed_v1_0.L_001QTQ"],
["aradu.V14167.gnm1.ann1.Aradu.2RA92","legfed_v1_0.L_001QTQ"],
["aradu.V14167.gnm1.ann1.Aradu.2SD9I","legfed_v1_0.L_001QTQ"],
["aradu.V14167.gnm1.ann1.Aradu.2T9CD","legfed_v1_0.L_001QTQ"],
["aradu.V14167.gnm1.ann1.Aradu.2Z0LN","legfed_v1_0.L_001QTQ"],
["aradu.V14167.gnm1.ann1.Aradu.31UFB","legfed_v1_0.L_001QTQ"]
],
"executionTime":"2023.05.31 15:16::03",
"wasSuccessful":true,
"error":null,
"statusCode":200}

I had looked into this earlier and I think if you set format to jsonrows you get the info about the number of results (grokked by looking at the network traffic made by intermine when a query is done, it asks for this before asking again with format=json)

Hrm. I think that just gives you a different JSON syntax but it doesn't include extra metadata.
curl "https://mines.legumeinfo.org/legumemine/service/query/results?query=<query+name%3D""+model%3D"genomic"+view%3D"Gene.primaryIdentifier+Gene.geneFamilyAssignments.geneFamily.primaryIdentifier"+longDescription%3D""+sortOrder%3D"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier+asc"><constraint+path%3D"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier"+op%3D"%3D"+value%3D"legfed_v1_0.L_001QTQ"%2F><%2Fquery>&format=jsonrows&start=21&size=20"
{"modelName":"genomic","columnHeaders":["Gene > Identifier","Gene > Gene Family Assignments > Gene Family > Identifier"],"rootClass":"Gene","start":21,"views":["Gene.primaryIdentifier","Gene.geneFamilyAssignments.geneFamily.primaryIdentifier"],"results":[
[{"column":"Gene.primaryIdentifier","id":172012566,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.3246G","url":"/report.do?id=172012566"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}],
[{"column":"Gene.primaryIdentifier","id":172026066,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.34G7V","url":"/report.do?id=172026066"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}],
[{"column":"Gene.primaryIdentifier","id":172007486,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.3824G","url":"/report.do?id=172007486"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}],
[{"column":"Gene.primaryIdentifier","id":171989281,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.38T1L","url":"/report.do?id=171989281"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}],
[{"column":"Gene.primaryIdentifier","id":171966266,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.38TC7","url":"/report.do?id=171966266"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}],
[{"column":"Gene.primaryIdentifier","id":172034816,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.3D3JJ","url":"/report.do?id=172034816"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}],
[{"column":"Gene.primaryIdentifier","id":172048371,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.3E4ER","url":"/report.do?id=172048371"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}],
[{"column":"Gene.primaryIdentifier","id":172067901,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.3K5AX","url":"/report.do?id=172067901"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}],
[{"column":"Gene.primaryIdentifier","id":172061801,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.3L037","url":"/report.do?id=172061801"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}],
[{"column":"Gene.primaryIdentifier","id":172102401,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.3NF03","url":"/report.do?id=172102401"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}],
[{"column":"Gene.primaryIdentifier","id":172007146,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.3P8PP","url":"/report.do?id=172007146"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}],
[{"column":"Gene.primaryIdentifier","id":171990106,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.3T41N","url":"/report.do?id=171990106"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}],
[{"column":"Gene.primaryIdentifier","id":172086396,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.47T7Y","url":"/report.do?id=172086396"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}],
[{"column":"Gene.primaryIdentifier","id":172024596,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.4IV9Z","url":"/report.do?id=172024596"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}],
[{"column":"Gene.primaryIdentifier","id":172075271,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.4XS9W","url":"/report.do?id=172075271"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}],
[{"column":"Gene.primaryIdentifier","id":172006851,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.53MET","url":"/report.do?id=172006851"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}],
[{"column":"Gene.primaryIdentifier","id":171995781,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.56CQR","url":"/report.do?id=171995781"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}],
[{"column":"Gene.primaryIdentifier","id":172063656,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.5J4IV","url":"/report.do?id=172063656"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}],
[{"column":"Gene.primaryIdentifier","id":172110416,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.61PXE","url":"/report.do?id=172110416"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}],
[{"column":"Gene.primaryIdentifier","id":172061101,"class":"Gene","value":"aradu.V14167.gnm1.ann1.Aradu.65IXB","url":"/report.do?id=172061101"},{"column":"Gene.geneFamilyAssignments.geneFamily.primaryIdentifier","id":172293120,"class":"GeneFamily","value":"legfed_v1_0.L_001QTQ","url":"/report.do?id=172293120"}]
],"executionTime":"2023.05.31 15:44::13","wasSuccessful":true,"error":null,"statusCode":200}

There's also the jsonobjects format which the bluegenes visualizers use, but also without summary metadata:
{"modelName":"genomic",
"columnHeaders":["Gene > Id","Gene > Identifier","Gene > Gene Family Assignments > Id","Gene > Gene Family Assignments > Gene Family > Identifier"],
"rootClass":"Gene",
"start":21,
"views":["Gene.id","Gene.primaryIdentifier","Gene.geneFamilyAssignments.id","Gene.geneFamilyAssignments.geneFamily.primaryIdentifier"],
"results":[
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.WL45U","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172136767}],"class":"Gene","objectId":171954786},
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.2L303","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172169127}],"class":"Gene","objectId":171955271},
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.E2DKW","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172154896}],"class":"Gene","objectId":171955331},
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.D98LI","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172176054}],"class":"Gene","objectId":171957281},
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.S92GY","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172135246}],"class":"Gene","objectId":171959191},
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.KZG41","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172165072}],"class":"Gene","objectId":171959976},
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.M3RAY","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172139519}],"class":"Gene","objectId":171960161},
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.B87TS","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172130526}],"class":"Gene","objectId":171961341},
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.K6PB3","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172114318}],"class":"Gene","objectId":171962146},
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.079IW","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172160517}],"class":"Gene","objectId":171962826},
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.BJH2V","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172124983}],"class":"Gene","objectId":171962906},
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.C2DIQ","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172168594}],"class":"Gene","objectId":171965951},
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.38TC7","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172135138}],"class":"Gene","objectId":171966266},
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.V7TBU","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172150887}],"class":"Gene","objectId":171966561},
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.GXG9A","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172120134}],"class":"Gene","objectId":171968131},
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.YLF6X","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172123977}],"class":"Gene","objectId":171970601},
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.NF9J9","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172164675}],"class":"Gene","objectId":171971246},
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.W7F6J","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172151499}],"class":"Gene","objectId":171972101},
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.U9KEB","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172173003}],"class":"Gene","objectId":171973746},
{"primaryIdentifier":"aradu.V14167.gnm1.ann1.Aradu.EUY81","geneFamilyAssignments":[{"geneFamily":{"primaryIdentifier":"legfed_v1_0.L_001QTQ","class":"GeneFamily","objectId":172293120},"class":"GeneFamilyAssignment","objectId":172135610}],"class":"Gene","objectId":171973996}

maybe there are some other parameter diffs I didn't notice; here's the "copy as cURL" version of the request:
curl 'https://mines.legumeinfo.org/legumemine/service/query/results' \
  -H 'Accept-Language: en-US,en;q=0.9' \
  -H 'Cache-Control: no-cache' \
  -H 'Connection: keep-alive' \
  -H 'Cookie: visited=at-some-point; __utmc=189715127; _gid=GA1.2.1232678063.1685565217; JSESSIONID=2D28E948702F47FAFFE0CDD1CDB6F92C; __utmz=189715127.1685566328.72.20.utmcsr=jekyll.dev.lis.ncgr.org|utmccn=(referral)|utmcmd=referral|utmcct=/; __utma=189715127.1118889771.1674172994.1685566328.1685568692.73; _ga_MB7C521GCN=GS1.1.16855696<IP_ADDRESS>55696<IP_ADDRESS>; _ga=GA1.1.1118889771.1674172994; __utmt=1; __utmb=189715<IP_ADDRESS>5568692' \
  -H 'Origin: https://mines.legumeinfo.org' \
  -H 'Pragma: no-cache' \
  -H 'Referer: https://mines.legumeinfo.org/legumemine/results.do?trail=%257Cquery' \
  -H 'Sec-Fetch-Dest: empty' \
  -H 'Sec-Fetch-Mode: cors' \
  -H 'Sec-Fetch-Site: same-origin' \
  -H 'User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36' \
  -H 'accept: application/json;type=rows' \
  -H 'authorization: Token y1J8b88743i9e4kf172f' \
  -H 'content-type: application/x-www-form-urlencoded; charset=UTF-8' \
  -H 'sec-ch-ua: "Google Chrome";v="113", "Chromium";v="113", "Not-A.Brand";v="24"' \
  -H 'sec-ch-ua-mobile: ?0' \
  -H 'sec-ch-ua-platform: "macOS"' \
  --data-raw 'query=%3Cquery%20model%3D%22genomic%22%20view%3D%22Gene.ontologyAnnotations.ontologyTerm.identifier%20Gene.ontologyAnnotations.ontologyTerm.name%20Gene.secondaryIdentifier%20Gene.primaryIdentifier%20Gene.description%20Gene.id%22%20sortOrder%3D%22Gene.primaryIdentifier%20ASC%22%20name%3D%22goterm_genes%22%20%3E%3Cconstraint%20path%3D%22Gene.ontologyAnnotations.ontologyTerm.name%22%20op%3D%22%3D%22%20value%3D%22photosynthesis%22%20code%3D%22A%22%20%2F%3E%3C%2Fquery%3E&summaryPath=Gene.id&format=jsonrows' \
  --compressed

resulting in
{"uniqueValues":8544,"modelName":"genomic","columnHeaders":["item","count"],"rootClass":"Gene","start":0,"views":["Gene.ontologyAnnotations.ontologyTerm.identifier","Gene.ontologyAnnotations.ontologyTerm.name","Gene.secondaryIdentifier","Gene.primaryIdentifier","Gene.description","Gene.id"],"results":[
{"bucket":1,"average":472955824.79307116,"min":168893337,"max":811852611,"buckets":20,"count":820,"stdev":222138319},
{"bucket":2,"average":472955824.79307116,"min":168893337,"max":811852611,"buckets":20,"count":643,"stdev":222138319},
{"bucket":3,"average":472955824.79307116,"min":168893337,"max":811852611,"buckets":20,"count":591,"stdev":222138319},
{"bucket":4,"average":472955824.79307116,"min":168893337,"max":811852611,"buckets":20,"count":549,"stdev":222138319},
{"bucket":5,"average":472955824.79307116,"min":168893337,"max":811852611,"buckets":20,"count":438,"stdev":222138319},
{"bucket":6,"average":472955824.79307116,"min":168893337,"max":811852611,"buckets":20,"count":1175,"stdev":222138319},
{"bucket":7,"average":472955824.79307116,"min":168893337,"max":811852611,"buckets":20,"count":349,"stdev":222138319},
{"bucket":11,"average":472955824.79307116,"min":168893337,"max":811852611,"buckets":20,"count":159,"stdev":222138319},
{"bucket":12,"average":472955824.79307116,"min":168893337,"max":811852611,"buckets":20,"count":770,"stdev":222138319},
{"bucket":16,"average":472955824.79307116,"min":168893337,"max":811852611,"buckets":20,"count":459,"stdev":222138319},
{"bucket":17,"average":472955824.79307116,"min":168893337,"max":811852611,"buckets":20,"count":568,"stdev":222138319},
{"bucket":18,"average":472955824.79307116,"min":168893337,"max":811852611,"buckets":20,"count":750,"stdev":222138319},
{"bucket":19,"average":472955824.79307116,"min":168893337,"max":811852611,"buckets":20,"count":762,"stdev":222138319},
{"bucket":20,"average":472955824.79307116,"min":168893337,"max":811852611,"buckets":20,"count":511,"stdev":222138319}
],"executionTime":"2023.05.31 15:50::41","wasSuccessful":true,"error":null,"statusCode":200}

most of which is probably irrelevant but I think uniqueValues is the count you want

Yes, most excellent! The key is the summaryPath=Gene.id variable. This gives you an (undocumented) query summary. Here's a minimal working example; format can be "json", "jsonrows", or "jsonobjects". If you use start=100 the meaningless results list is empty as it tends to have about 14 elements.
curl 'https://mines.legumeinfo.org/legumemine/service/query/results' \
     -H 'Connection: keep-alive' \
     -H 'accept: application/json' \
     -H 'content-type: application/x-www-form-urlencoded; charset=UTF-8' \
     --data-raw 'query=%3Cquery+name%3D%22%22+model%3D%22genomic%22+view%3D%22Gene.id%22%20%3E%3Cconstraint+path%3D%22Gene.ontologyAnnotations.ontologyTerm.identifier%22+op%3D%22%3D%22+value%3D%22GO%3A0000015%22%2F%3E%3C%2Fquery%3E&summaryPath=Gene.id&start=100&format=json"'

It also works as a GET request:
https://mines.legumeinfo.org/legumemine/service/query/results?query=%3Cquery+name%3D%22%22+model%3D%22genomic%22+view%3D%22Gene.id%22+longDescription%3D%22%22+sortOrder%3D%22Gene.ontologyAnnotations.ontologyTerm.identifier+asc%22%3E%3Cconstraint+path%3D%22Gene.ontologyAnnotations.ontologyTerm.identifier%22+op%3D%22%3D%22+value%3D%22GO%3A0000015%22%2F%3E%3C%2Fquery%3E&summaryPath=Gene.id&start=100&format=json

great, thanks for figuring out the undocumented details

Feature request (being greedy).
"1-10 of 14 results"
Is it possible to give an option like "show 10 or 25 or 50 per page" in a drop down, a common feature seen in where pagination is available.

Feature request (being greedy).
"1-10 of 14 results"
Is it possible to give an option like "show 10 or 25 or 50 per page" in a drop down, a common feature seen in where pagination is available.

This is a feature request. Open a new issue if you want this functionality.

Bug.
Links( gene name and location) open the exact same page.

Links are implemented in Jekyll; this should be reported in the lis-jekyll repo.

All tasks are complete and last week's beta didn't reveal any bugs. Closing and moving into release phase.


gradle application plugin startScripts: add custom directory to classpath is almost impossible

My application has a directory structure like this:
app
   bin
   conf
   lib

What I want to achieve is to have conf/ added to the classpath in the startup script(s). However - whatever I do gradle always adds lib/conf to the classpath:
CLASSPATH=$APP_HOME/lib/conf
I started with this one (no solution unless I put conf in lib/conf - which I don't want):
https://stackoverflow.com/questions/35545405/gradle-add-directory-to-classpath?rq=1
... which directs you to this gradle issue (it's a 'won't fix' and it also doesn't work):
https://issues.gradle.org/browse/GRADLE-2991
... and finally to more creative answers like patching the generated install scripts after they are generated:
https://stackoverflow.com/questions/10518603/adding-classpath-entries-using-gradles-application-plugin
I tried both using startScripts and createStartScripts... It always adds lib/ before conf.
Would it be possible to get a forceClassPath or something so that I don't have to patch the install scripts?
I'm using the latest gradle version 4.10.2 on macosx.

Unfortunately, we will not provide any classpath feature unless there is a wider pattern around this. You can use a custom template to fulfill the use case. The Javadocs provide an example.
If you have a clearer pattern/use case that we can look at with regards to why the conf can't be in lib and that it's a widespread usage. We can look at modelling this inside the application plugin.

You can use a custom template to fulfill the use case. The Javadocs provide an example.

I did not try, but by looking at the template the lib in the classpath is hardcoded. Is there any particular reason for forcing lib directory? For example, I prefer having a flat structure for my very small application (script + jar)

Agree with @neworld here. Hardcoding with no option to change doesn't seem reasonable.

Agree with @neworld here. Hardcoding with no option to change doesn't seem reasonable.

I have a slightly different issue that also relates to missing configuration options: I don't want to hide my start-scripts in a "bin" directory but rather have them in the toplevel. Modifying the template for that is quite a hassle.

I have a slightly different issue that also relates to missing configuration options: I don't want to hide my start-scripts in a "bin" directory but rather have them in the toplevel. Modifying the template for that is quite a hassle.

Why is this closed?

Why is this closed?

I'm also wondering why this is closed. This issue still persists.
There's now a way to change the executable directory (default bin), so why is lib still being hardcoded?


Adding config option CALICO_LIBNETWORK_NAMESPACE

Description
This PR makes the CALICO_LIBNETWORK_NAMESPACE option configurable. By default the libnetwork plugin always uses the "libnetwork" namespace for configuring workload endpoints.
Todos

[ ] Tests
[ ] Documentation
[ ] Release note

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


Make operator readiness probe pass without KubeVirt CR present

What this PR does / why we need it:
In case of no active KubeVirt installation, the certificates for the
operator-readiness probes are never generated. In this case, fall back
to a self signed certificate. As soon as an installation is present, use
the "official" certificate.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:
NONE

/cc @jean-edouard

/retest

/override ci/prow/images

/test pull-kubevirt-e2e-k8s-1.16


Better lazy_indexing - Fix #470

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #470

[ ] Tests for the changes have been added (for bug fixes / features)
[x] Documentation has been added / updated (for bug fixes / features)
[ ] HISTORY.rst has been updated (with summary of main changes)
[ ] bumpversion (minor / major / patch) has been called on this branch
[ ] Tags have been pushed (git push --tags)

What kind of change does this PR introduce?

The error in #470 was coming from all-nan slices failing in np.nanargmax. This modifies the _from_other_arg so that  all-nan slices are first filled with 0, then masked back on the result.

Not directly in #470, but that issue shows that lazy_indexing was not lazy.  Dask doesn't implement np.take_along_axis (yet) . I modified the whole function using the same code as in xr.DataArray.idxmax. We are pretty much doing the exact same as this function, but it's not yet released.

Pull Request Test Coverage Report for Build 1978

9 of 11 (81.82%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 77.267%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

xclim/indices/run_length.py
6
8
75.0%

Totals

Change from base Build 1975:
-0.02%

Covered Lines:
2658

Relevant Lines:
3440

💛  - Coveralls

I copied code from xarray for this, assuming they would have tested that part... Turns out there is a bug...

I believe this is fixed! (Collateral damage: pydata/xarray#4135)

Reverted back to the initial option for lazy_indexing. This adds the constraint that the data cannot be chunking along "time", so I added some chunking management for anuclim indices.

I think we've got it.  running ok on my end


Are docs available for older versions of wowchemy?

Prerequisites

[X] I have searched for duplicate or closed feature requests
[X] I am mindful of the project scope

Proposal
It would be nice to have access to documentation for older versions of wowchemy, as is the case with other docs pages (e.g. readthedocs).
Motivation and context
The API for wowchemy changes frequently, the current docs are not applicable to the version of wowchemy I am currently running, and upgrading wowchemy often breaks my website.

This is an independent community-driven open source and open-science project created by creators/researchers for creators/researcher. So the documentation is dependent on what the community contributes. As we are an independent community, not a big corporation those behind RStudio or Wordpress, please, if you find an opportunity for improvement, help support independent open source by contributing improvements :)
If you are interested in supporting independent open source and open science, you can find the contributor docs here: https://github.com/wowchemy/wowchemy-hugo-themes/blob/main/CONTRIBUTING.md
Some of the most popular docs pages which the community has contributed to can be found at https://github.com/wowchemy/wowchemy-hugo-themes/tree/main/docs/en which is versioned in Git, so you can travel back in time. We are in the process of migrating the rest of the docs pages to this folder.
Also, you should find documentation and changes in the release notes in GitHub Releases (or on the blog for older versions).
The documentation pages on the site, also try to highlight where possible how to perform an action with older and newer versions of the software.
The reason for a number of breaking changes in Wowchemy over the last few years is due to breaking changes in Hugo, which we obviously have to adopt if the software is to be compatible with new versions on Hugo. The Hugo community is one of the largest static site generator communities, so there tend to be a high velocity of improvements. Every site should be tied to a specific version of Wowchemy and Hugo so there is no risk of a site breaking when Hugo team release a new update. Wowchemy is now in a very stable phase where all the main user feedback has been addressed. You will find smaller GitHub projects will likely cause you more concern due to their smaller communities and even more rapid roll out of breaking changes due to them scaling up.
If you have any questions regarding Wowchemy or Hugo, just follow the links to ask a question and raise it in our Discord community (rather than raising questions as Github issues) and the large community will of course try to help you. To complement this, there is also a huge Hugo community on the Hugo forum which are happy to try to help with any Hugo issues.

Hi Geo! Thanks for taking the time to respond to this issue.
It makes sense that Wowchemy changes over time, whether it be due to functionality being added or due to changes to Hugo itself.
However, coming back to @scottgigante-immunai 's original question: is there a way to access the Wowchemy documentation website for different versions of Wowchemy?
You mention Wowchemy is open-source and that the community has contributed to the documentation website, but if the repository for the documentation was open source we could just go back in time ourselves and look at the documentation for Wowchemy v5.2.* without any issue.

Thanks @gcushen . As an example, if I was interested in creating a menu for my site that's built on Wowchemy v5.2, where could I find the equivalent page for https://wowchemy.com/docs/getting-started/get-started/#menu ? I don't see it in the docs/en directory you linked to.

Unfortunately the solution here (for anyone reading along at home) was to stop using Wowchemy. We instead migrated our website to Quarto.


LICENSE file

First of all, thanks for a nice software!
Could you add LICENSE file?
The following page might help for choosing a license.
Choosing an open source license doesn’t need to be scary - Choose a License
Thanks!

License add, thanks.

Thanks!


AttributeError: 'module' object has no attribute 'WebsocketApp'

AttributeError: 'module' object has no attribute 'WebsocketApp'
I have done pip uninstall websocket-client and the install websocket-client back, but still got the same error.. can someone help me with this error.? really need help

Hi,
with pushubllet v0.12.0 and websocket-client 1.0.1 again same AttributeError
I tried:

downgraded pushbullet (version 0.9x, 0.10x) and same error.
downgraded websocket-client version and still AttributeError

AttributeError: module 'websocket' has no attribute 'WebSocketApp' on GoogleColab Python 3.7


allow dependsOn plugin definition to be a callback

Describe the feature
There are modules that can register multiple runtime plugins based on configuration options or even on installed dependencies and/or Nuxt modules.
Right now, there is no way to configure dependsOn, it must be an static array.
An example can be found here: https://github.com/userquin/vuetify-nuxt-module. This module has a few runtime plugins that can be enabled via configuration:

vuetify-date when enabled and the corresponding @date-io/* dependency installed
vuetify-i18n when @nuxtjs/i18n module configured

I want to use this new feature to allow main Vuetify Nuxt runtime plugins to be parallel, registering secondary plugins or external plugin via custom virtual module using dependsOn callback:

https://github.com/userquin/vuetify-nuxt-module/blob/main/src/runtime/plugins/vuetify.ts
https://github.com/userquin/vuetify-nuxt-module/blob/main/src/runtime/plugins/vuetify-sync.ts

Additional information

[X] Would you be willing to help implement this feature?
[ ] Could this feature be implemented as a module?

Final checks

[X] Read the contribution guide.
[X] Check existing discussions and issues.

I think we'd prefer for the moment to leave open the option for dependsOn to be fully statically analysable at build-time to reduce runtime cost even further ...
For the feature you'd like to implement, you can do this without a callback, by generating a different plugin template and (within getContents) generate a different dependsOn array depending on what modules are installed, etc. You can do this with addPluginTemplate (or addTemplate) which allows you to generate a virtual plugin file.
Example: https://github.com/nuxt/nuxt/blob/642d4dc9de091009a18074bb9d6b162d35565532/packages/nuxt/src/components/templates.ts#L36-L71

Refactored all my plugins and also added both main plugins using addPluginTemplate, it is working, feel free to close this issue and the linked PR: https://github.com/userquin/vuetify-nuxt-module/pull/185

Having discussed it with the team, we think we'd prefer not to add this potential complexity, particularly in view of the possibility of static analysis of dependsOn.


Fail on empty source directories in buildSrc as well.

Context
These cause cache misses and slow down the build. We need to dogfood this issue so we are motivated to fix it.
Follow up from #14407
If you are reading this, please vote this up:
#2463
Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[ ] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:check

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

@bot-gradle test ready for merge

Sorry I don't understand what you said, please type @bot-gradle help to get help.

@bot-gradle test readyformerge

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test readyformerge

OK, I've already triggered ReadyForMerge build for you.

@jjohannes @wolfs I've updated the PR to use a script located in gradle/shared-with-buildSrc. What do you think?
@bot-gradle test readyformerge

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test readyformerge

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test readyformerge

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test readyformerge

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test readyForMerge

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test readyForMerge

OK, I've already triggered ReadyForMerge build for you.


Helm cannot fetch images from a local minikube repository

$ minikube version
minikube version: v0.19.0
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"6", GitVersion:"v1.6.3", GitCommit:"0480917b552be33e2dba47386e51decb1a211df6", GitTreeState:"clean", BuildDate:"2017-05-10T15:48:59Z", GoVersion:"go1.7.5", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"6", GitVersion:"v1.6.0", GitCommit:"fff5156092b56e6bd60fff75aad4dc9de6b6ef37", GitTreeState:"clean", BuildDate:"2017-05-09T23:22:45Z", GoVersion:"go1.7.3", Compiler:"gc", Platform:"linux/amd64"}
$ helm version
Client: &version.Version{SemVer:"v2.4.1", GitCommit:"46d9ea82e2c925186e1fc620a8320ce1314cbb02", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.4.1", GitCommit:"46d9ea82e2c925186e1fc620a8320ce1314cbb02", GitTreeState:"clean"}

I'm trying to run helm on minikube, and running into an issue where it isn't pulling my image.
I see my image in the minikube repository:
$ eval $(minikube docker-env)
$ docker images | grep alpine-flask
alpine-flask                                           v1                  2aa3b783340c        5 seconds ago       95.8MB

If I run the image via kubectl directly, it fires up just fine:
$ kubectl config set-context minikube
Context "minikube" set.
$ kubectl run test-alpine-flask --image=alpine-flask:v1
deployment "test-alpine-flask" created
$ kubectl get pods
NAME                                 READY     STATUS        RESTARTS   AGE
test-alpine-flask-2836657737-4074n   1/1       Running       0          3s

but if I try to run that same image via helm, it says that it can't find the image:
$ helm install flask
...
$ kubectl get pods
NAME                                 READY     STATUS         RESTARTS   AGE
alpine-flask-1265597736-2rtn2        0/1       ErrImagePull   0          11s
alpine-flask-1265597736-rxhjp        0/1       ErrImagePull   0          11s
test-alpine-flask-2836657737-4074n   1/1       Running        0          4m
$ kubectl describe pod alpine-flask-1265597736-2rtn2 | grep -A10 Events
Events:
  FirstSeen	LastSeen	Count	From			SubObjectPath			Type		Reason		Message
  ---------	--------	-----	----			-------------			--------	------		-------
  40s		40s		1	default-scheduler					Normal		Scheduled	Successfully assigned alpine-flask-1265597736-2rtn2 to minikube
  40s		24s		2	kubelet, minikube	spec.containers{alpine-flask}	Normal		Pulling		pulling image "alpine-flask:v1"
  38s		23s		2	kubelet, minikube	spec.containers{alpine-flask}	Warning		Failed		Failed to pull image "alpine-flask:v1": rpc error: code = 2 desc = Error: image library/alpine-flask not found
  38s		23s		2	kubelet, minikube					Warning		FailedSync	Error syncing pod, skipping: failed to "StartContainer" for "alpine-flask" with ErrImagePull: "rpc error: code = 2 desc = Error: image library/alpine-flask not found"

  38s	12s	2	kubelet, minikube	spec.containers{alpine-flask}	Normal	BackOff		Back-off pulling image "alpine-flask:v1"
  38s	12s	2	kubelet, minikube					Warning	FailedSync	Error syncing pod, skipping: failed to "StartContainer" for "alpine-flask" with ImagePullBackOff: "Back-off pulling image \"alpine-flask:v1\""

Any suggestions on how I can debug this further would be much appreciated. The issue seems similar to https://github.com/kubernetes/kubernetes/issues/24903, but I can run kubectl directly and have it pull my local image, it's only when I throw helm into the mix that my image is not found.

Helm doesn't do the image fetching. Only Kubernetes does. Helm merely pushes the manifest with the image reference.
To test, try using helm --debug --dry-run install.... and then manually load that manifest with kubectl create -f.

Chopping down my problem into the smallest possible example and then using helm install --debug --dry-run flask to produce a file to feed to kubectl helped figure this one out. Sleeping on it probably helped, too. :)
I needed to set imagePullPolicy: IfNotPresent instead of Always.
I get it into my head that by setting the policy to Always I can tag v1 over and over again and it will grab the newest version, but that's not the way to think about imagePullPolicy. I should also be less lazy by just incrementing my versions and using them, anyway, to be certain that the code that I want is the code that actually gets run.


consumer offset is not resuming from where it left.

Description
i have observed on start of application it does not resume where it left. below is my configuration
var consumerConfig = new Dictionary<string, object>
            {
                 { "bootstrap.servers", "list-of-brokers"},
                 { "client.id", "name-of-machine" },
                 { "auto.offset.reset","earliest"},
                 { "fetch.min.bytes", 1},
                 { "group.id", "some-group-id-i-don't-change" },
                 { "socket.keepalive.enable", true },
                 { "auto.commit.interval.ms", 5000 },
                 { "enable.auto.commit", true },
            };

in my code i assign topic to a consumer.(i get data on partition 0 only. )
Consumer.Assign(new List<TopicPartition> { new TopicPartition(topic, 0) });

and after that i start consuming message via
if (Consumer.Consume(out msg, TimeSpan.FromSeconds(10)))
{
     //process message
}

note : i don't if i am missing anything.
should i call Resume/ wait for OnPartitionsAssigned
public List<TopicPartitionError> Resume(IEnumerable<TopicPartition> partitions)
public event EventHandler<List<TopicPartition>> OnPartitionsAssigned

How to reproduce
it is intermittent issue. i have seen it 3-4 times in last 10 days.
Checklist
Please provide the following information:

[x] Confluent.Kafka nuget version: 0.11.5
[x] Apache Kafka version: 1.1.0
[x] Client configuration:
[x] Operating system: ubuntu
[ ] Provide logs (with "debug" : "..." as necessary in configuration)
[ ] Provide broker log excerpts
[ ] Critical issue

@mhowlett  thanks for info.
I will use subscribe and see if it helps.

@mhowlett i think issue is due to __consumer_offsets  truncation. i have not seen this with high throughput topics. we may close this.
@mhowlett thanks again.


Block labels in JS are faded out as "unused"

The block label is faded out as "unused", which doesn't seem right. Why would it be used or unused?
From #51264

We marked it as unused if the label is not referenced anywhere. Adding break foo somewhere in the program will cause it to become referenced.
Is this what you see?

Oh, that makes sense. Thanks


add raises statement sync pragma in to server.nim serverprivate.nim

This is duplicates https://github.com/status-im/nim-presto/pull/67


NIFI-9790: Fixed race condition in which SwappablePriorityQueue could…

… attempt to access the 0th element of an empty collection

Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Enables X functionality; fixes bug NIFI-YYYY.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically main)?

[ ] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

This is rather difficult to create a unit test for, as it's a threading bug. But it's pretty clear by looking at the code how the PR resolves the issue. Also set a breakpoint and ensured that we hit the newly introduced code path in a running instance and that it was handled as expected.


I can't set my own tag

In the 2.0.1 version, lines 71 and 72 have the following code:
Response response = chain.proceed(request);      List segmentList = ((Request)request.tag()).url().encodedPathSegments();
I took my url from the tag, but I set the url myself, So a ClassCastException occurs when used,I need to set up my own tag to cancel the request at any time through this tag

Can you share your code? @zzh1026


HTTPS Calls Fail for https://localhost, reason:self-signed certificate

Describe the bug
When setting a provider to https://localhost so I can hit endpoints for local development, promptfoo's HTTPS calls fail after 4 retries.
To Reproduce
description: 'Test eval'

prompts:
  - "{{testPrompt}}"

providers:
  - id: 'https://localhost:44800/chat'
    config:
      method: 'POST'
      headers:
        'Content-Type': 'application/json'
      body:
        message: '{{prompt}}'
      responseParser: 'json.output'
tests:
  - vars:
      testPrompt: Hello

Create a small API with an endpoint "/chat" and run it on https://localhost:44800 or any other port
Use the given config file as an example
Run npx promptfoo eval
Wait for it to complete
See error

Expected behavior
A successfull HTTP API call and response.
Actual behaviour
[FAIL] HTTP call error: Error: Request failed after 4 retries: request to https://localhost:44800/chat/ failed, reason: self-signed certificate
System information (please complete the following information):
Windows 11
Node v18.19.0 installed with NVM

Promptfoo version: 0.75.0

Additional context
I was using an ASP.NET Web API locally and sending it the prompt to test its responses.

Have you tried with "http" in the URL instead of https?
On Wed, Aug 7, 2024, 2:41 AM Muhammad Rafay Nadeem @.***>
wrote:

Describe the bug
When setting a provider to https://localhost:44800 so I can hit endpoints
for local development, promptfoo's HTTPS calls fail after 4 retries.
To Reproduce
description: 'Test eval'
prompts:

"{{testPrompt}}"
providers:
id: 'https://localhost:44800/chat'
config:
method: 'POST'
headers:
'Content-Type': 'application/json'
body:
message: '{{prompt}}'
responseParser: 'json.output'tests:
vars:
testPrompt: Hello

Create a small API with an endpoint "/chat" and run it on
localhost:44800 or any other port
Use the given config file as an example
Run npx promptfoo eval
Wait for it to complete
See error

Expected behavior
A successfull HTTP API call and response.
Screenshots
HTTP call error: Error: Request failed after 4 retries: request to
https://localhost:44800/chat/ failed, reason: self-signed certificate
System information (please complete the following information):
Windows 11
Node v18.19.0 installed with NVM

Promptfoo version: 0.75.0

Additional context
I was using an ASP.NET Web API locally and sending it the prompt to test
its responses.
—
Reply to this email directly, view it on GitHub
https://github.com/promptfoo/promptfoo/issues/1388, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACLYJQNODHV7IXMAP3LRBDZQHTT7AVCNFSM6AAAAABMEAYJTGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ2TGMBUGA4TKMQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Yes it works when it's http but not when it's https.

You can probably work around this by setting NODE_TLS_REJECT_UNAUTHORIZED = "0" (https://stackoverflow.com/q/52478069/219604)
#1396 will also solve

Landed in 0.76.0

I'm not sure about the security implications of removing the check but calls with HTTPS are working now. Thanks.


Jenkins dashboard is not accessible outside the k8 cluster in aws ec2

followed this documentation: https://www.jenkins.io/doc/book/installing/kubernetes/
Installed Jenkins with YAML files so only read that part of documentation.

Resolved.


Classic SKU not available with Container registry.

The documentation lists the available SKUs for the Azure Container Registry as ..
accepted values: Basic, Classic, Premium, Standard
Classic is not a SKU of Container Registry as seen by ...
https://docs.microsoft.com/en-us/azure/container-registry/container-registry-skus

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

ID: c608162e-cf40-9d53-da4a-51f51a0311ea
Version Independent ID: eccbb543-be68-7de2-d41d-abee55c81452
Content: az acr
Content Source: src/azure-cli/azure/cli/command_modules/acr/_help.py
Service: container-registry
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

acr

Thanks for reporting this. We will address this in the near future when roll out a new api-version.
Since #11487 was opened to track the same thing, hence I am closing this one.


Update Repo Link

Description
This PR updates the repo link across the codebase by replacing the old link github.com/privacy-scaling-explorations/bandada with the new link github.com/bandada-infra/bandada. The update is applied to all files and locations where the old repo link was referenced.
Related Issue
Closes #442
Does this introduce a breaking change?

[ ] Yes
[x] No

Hey @vplasencia, I have updated the other occurrences as well.

@gitpoap-bot @neel-ds

Hello @vplasencia,
Please check coverage in README
https://github.com/bandada-infra/bandada/blob/ee95cbc1e452a04960ad13d6ed8a255f2c377aa0/README.md?plain=1#L20-L22
Coveralls is not setup for new repo link. Thus, it's throwing invalid response data. Please do the needful!


Make use of connection pooling in most tests

This changes a lot more than "just" tests and needs a Jira imo


button to make analysis public

Button to make it public:

Error when can't delete:

When success:

Code looks good, but can you put the button next to the "Mapping file" button? Also, instead of "(Analysis is public)" can you just add "(public)" and "(private)" when it's not public?

Now, it looks like this:

Codecov Report

Merging #2407 into dev will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##              dev   #2407      +/-   ##
=========================================
+ Coverage   93.79%   93.8%   +<.01%
=========================================
  Files         161     161
  Lines       18827   18844      +17
=========================================
+ Hits        17659   17676      +17
  Misses       1168    1168

Impacted Files
Coverage Δ

...lers/analysis_handlers/tests/test_base_handlers.py
98.82% <100%> (+0.04%)
:arrow_up:

...ta_pet/handlers/analysis_handlers/base_handlers.py
98.87% <100%> (+0.2%)
:arrow_up:

Continue to review full report at Codecov.

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


Unable to connect with ssl set to false

After upgrading to 0.6.1 I'm not able to connect to a server with ssl set to false.
It seems that is not valid to sent sslopts when ssl is set to false.
After changing
def open(server, port, ssl, timeout, sslopts) do
    :eldap.open([to_charlist(server)], [{:port, port}, {:ssl, ssl}, {:sslopts, sslopts}, {:timeout, timeout}])
end

to
def open(server, port, ssl, timeout, sslopts) do
    :eldap.open([to_charlist(server)], [{:port, port}, {:ssl, ssl}, {:timeout, timeout}])
end

the connection was successful.

Thanks for reporting the issue, should now be fixed in 0.6.2. I've also added a test to make sure this doesn't slip through next time.

Ah, looks like I broke it. Apologies!


Tmds.Libc does dependency does not work with Xamarin.iOS

The dependency on Tmds.Libc introduced to solve #22 breaks building for Xamarin.iOS. This is because when compiling for Xamarin.iOS all referenced assemblies are needed for the linker even if those assemblies will not be used at runtime. However, Tmds.Libc does not deliver any assemblies for Xamarin.iOS.
Perhaps it's possible to use the needed DllImports directly in net-mdns instead of depending on Tmds.Libc.

Thanks for taking the time to report an issue; I really appreciate the feedback!
Sadly I'm not supporting Xamarin.iOS.  I've only read about Xamarian and have never played with it. If you want support for X-iOS then I need

to know to which CI platform (circleCI, travis, appveyor ...) supports it
a script to compile and run the unit tests

I'm happy to try to find a work-around if you can provide the CI information.

Thank you for putting all the work into this project!
Unfortunately getting CI running for Xamarin.iOS seems to be quite complicated. If I understand correctly this can only work if you have a MacOS build server.
However making net-mdns work with Xamarin.iOS should not be too complicated. I guess it was working before #22 was fixed. The only problem is that Tmds.Libc does not deliver an assembly for Xamarin.iOS. Although this assembly is not needed at runtime and will never be called the special nature of Xamarin.iOS build process makes it necessary that there is at least an assembly with dummy implementations.
The easiest solution would be to deliver a dummy assembly with Tmds.Libc, but I guess the maintainers of Tmds.Libc won't do that.
For now I will try the version before #22 was fixed or switch to Zeroconf Nuget package.

I am not an expert for nuget, but it might be sufficent to set the targetFramework for Tmds.Libc in the .nuspec file to "netcoreapp" (since there is no blacklisting capability for "xamarinios")?

I'm not associated with the Tmds.Libc project.  Perhaps, you should raise an issue there.

I was not talking about the nuspec configuration for Tmds.Lib but for your project. In this case net-mdns would not reference Tmds.Lib on platforms where it is not needed anyway.


supprot for changing imageOptim config

For now it still need to change ImageOptim config manually.
Can you support apply config with command? or some file input?
It will be handy if need to apply different config on sets of images.

Can you add some examples of what this would look like when using it? I don't know quite what you mean.
Thanks

For example, you have two set of image, one set have to optimized to quality set to lossy 90% and another to be lossy 50%. Different set of image will be use on different site.
Currently, when you are to set different quality, you have to open ImageOptim and change the config from setting panel. If we can have change quality setting in command line, we can have some script that optimize two set of image without changing the config with UI.
could be like: imageoptim -a -ImageOptimQuality 70 -d  ./path
Thanks

Oh I see, you're asking for options to be added to ImageOptim-CLI to edit the following from ImageOptim.app.

I don't think those are exposed by the ImageOptim binary, but I will revisit if they get added in the future.
Thanks a lot for this idea.

I don't think it's expose neither and I did not find a easy way to change that. Maybe we can replace the config file somewhere. Thank you for reading my request. ;)


refactor: Update reports codecs to match C# library's defined types

Compared C#'s defined responses to currently defined codecs. Fixed inconsistencies and got stuck on a weird bug. I don't think it's fixed, but IMO not worth worrying about since it's such a specific "special case". Will try to articulate how it happened in an issue.


ERROR: 2nd expression of procedural for-loop is not constant!

As the title error
I would like to translate the sv code of orv64 into verilog code and use yosys to synthesis .orv64
/root/yosys-uhdm-plugin-integration/UHDM-integration-tests/tests/orv64_uhdm/rtl/../rtl/orv64/orv64_ptw_core.sv:224: ERROR: 2nd expression of procedural for-loop is not constant! make: *** [Makefile:55: uhdm/yosys/test-ast] Error 1
the code is just like this

I think this is caused by insufficient uhdm support, so I can only manually split this for loop into a constant for loop and an if conditional statement
Here is a case
(almost 30MB, so I split it into two files and I think you should rename the orv64_1.zip to orv64_uhdm.z01 before unzip)
orv64_1.zip
orv64_uhdm.zip

Thanks for reporting this. I understand that you are calling make uhdm/yosys/test-ast with your test case? I'm seeing several errors in Surelog there:
[ERR:UH0718] /media/hdd/sv-tests/third_party/tools/yosys-uhdm-plugin-integration/UHDM-integration-tests/tests/orv64-581/rtl/../rtl/orv64/orv64_ptw_core.sv:200:15: Illegal lhs of type wire "excp_cause".

[ERR:UH0718] /media/hdd/sv-tests/third_party/tools/yosys-uhdm-plugin-integration/UHDM-integration-tests/tests/orv64-581/rtl/../rtl/orv64/orv64_ptw_core.sv:203:15: Illegal lhs of type wire "excp_cause".

[ERR:UH0718] /media/hdd/sv-tests/third_party/tools/yosys-uhdm-plugin-integration/UHDM-integration-tests/tests/orv64-581/rtl/../rtl/orv64/orv64_ptw_core.sv:206:15: Illegal lhs of type wire "excp_cause".

[ERR:UH0718] /media/hdd/sv-tests/third_party/tools/yosys-uhdm-plugin-integration/UHDM-integration-tests/tests/orv64-581/rtl/../rtl/orv64/orv64_ptw_core.sv:209:15: Illegal lhs of type wire "excp_cause".

[ERR:UH0718] /media/hdd/sv-tests/third_party/tools/yosys-uhdm-plugin-integration/UHDM-integration-tests/tests/orv64-581/rtl/../rtl/orv64/orv64_ptw_core.sv:212:15: Illegal lhs of type wire "excp_cause".

[ERR:UH0718] /media/hdd/sv-tests/third_party/tools/yosys-uhdm-plugin-integration/UHDM-integration-tests/tests/orv64-581/rtl/../rtl/orv64/orv64_tlb.sv:592:7: Illegal lhs of type wire "lru_mask".

[ERR:UH0718] /media/hdd/sv-tests/third_party/tools/yosys-uhdm-plugin-integration/UHDM-integration-tests/tests/orv64-581/rtl/../rtl/orv64/orv64_tlb.sv:594:7: Illegal lhs of type wire "way_enable".

Could you confirm if they are present in build/slpp_all/surelog.log?


Bug: Drag and Drop on desktop shortcut froze Files

Description
In a folder view, I dragged an mp4 file and dropped it on top of a link to mpc-hc64.exe in OneDrive ("C:\Users\Ben Moore\OneDrive\Software\MPC-HC.1.7.13.x64\mpc-hc64.exe") that was on the desktop. An icon was displayed on the desktop with a "Move" arrow at the top left. That icon froze and then Files wouldn't access the mp4 file again. I had chosen the "Experimental" feature of having Files run as the default file explorer. I restarted Windows Explorer using the Task Manager but that'd didn't help. I even reset the default file explorer. I finally had to restart Windows.
That process works fine with Windows File Explorer.
Additionally, although I have the option set to display know file types, they aren't displayed in the list of recent files.
I'm running <IP_ADDRESS> on Windows 10.0.19045.4291.
Steps To Reproduce
In a folder view, I dragged an mp4 file and dropped it on top of a link to mpc-hc64.exe in OneDrive ("C:\Users\Ben Moore\OneDrive\Software\MPC-HC.1.7.13.x64\mpc-hc64.exe") that was on the desktop. An icon was displayed on the desktop with a "Move" arrow at the top left. That icon froze and then Files wouldn't access the mp4 file again. I had chosen the "Experimental" feature of having Files run as the default file explorer. I restarted Windows Explorer using the Task Manager but that'd didn't help. I even reset the default file explorer. I finally had to restart Windows.
Requirements

e.g. Fix crash that occurs when...

Files Version
<IP_ADDRESS>
Windows Version
10.0.19045.4291
Log File
debug.log

Thanks for the report, Please update the steps to reproduce to be a clear step by step on how to reproduce the issue.

@fastoy Can you add the information request above

I can rephrase it but those are the steps:
1 In a folder view
2. Dragged an mp4 file from the folder and dropped it on top of a LINK on the desktop.  That link was to mpc-hc64.exe in OneDrive ("C:\Users\Ben Moore\OneDrive\Software\MPC-HC.1.7.13.x64\mpc-hc64.exe")
3. An icon was displayed on the desktop with a "Move" arrow at the top left. That icon froze and then Files wouldn't access the mp4 file again.
I had chosen the "Experimental" feature of having Files run as the default file explorer. I restarted Windows Explorer using the Task Manager but that'd didn't help. I even reset the default file explorer. I finally had to restart Windows.

Led to here from https://github.com/files-community/Files/issues/16120

appears to be duplicate of #6309

It doesn't look the same. This issue is about opening a file by dragging to a shortcut on the desktop to open the file in that program. The issue you link is about moving a file from the desktop to another folder in Files.


Wrap utils.js functions in a closure

Updated to fix the js tests.
Its passing on JDK11 on https://travis-ci.com/psiinon/zap-hud have restarted the JDK 8 tests

Those constants are used by other files, and CORS prevents other domains calling the functions


execute-job/wrong-commit-sha-on-pr/e060540f-32e0-458a-84f0-bfda86052c9f

PR for testing be564f557d1dc5bc126a49de7cc496a85f9cf2e6

/canonical/self-hosted-runners/run-workflows 92a3af83b63af8d423a2f2c104925ba7ffd08d0f


Remove build status from PR

Please add a meaningful description for your change here

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.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @aaltay @ibzib @kennknowles @lukecwik

Yes, I think it just shows the status of the tests in more structured form.
README.md seems to have too many status which doesn't seem to be useful from PR author point of view.
Anyways, I am open to linking .test-infra/jenkins/README.md instead.

Just noticed that we left the GHA status in the template, but really I have no idea what those even test.

I mean the badges in the templates that show the status on master branch, to help people know if their PR is unhealthy because the repo is unhealthy. We may as well remove them all, really.


Pause MC App CR when version is updated

Towards https://github.com/giantswarm/giantswarm/issues/15091

@tomahawk28 in short: config-controller takes care. Longer version here https://github.com/giantswarm/giantswarm/issues/14821#issuecomment-748000367

@tomahawk28 in short: config-controller takes care. Longer version here https://github.com/giantswarm/giantswarm/issues/14821#issuecomment-748000367


backgroundGeoLocation.isLocationEnabled returns always 1 even GPS turned off

Hi!
Using backgroundGeoLocation.isLocationEnabled to validate, if GPS is turned off or on, but it returns always 1.
Device is Huawei P7, Android 4.4.2. The latest Cordova and geolocation plugin

Can you post code?

plugins.insomnia.keepAwake(     function(){         backgroundGeoLocation.isLocationEnabled(function (response) {             if (response === 1) {                 backgroundGeoLocation.configure(setPosition, service.showError, service.gpsOptions);                 backgroundGeoLocation.start(function(){                     if (service.modalInstance !== null)                         service.modalInstance.close();                 });                 setStatus(gpsStatus.ON);             }else{                 backgroundGeoLocation.showLocationSettings();             }         });     },     function () {         backgroundGeoLocation.isLocationEnabled(function (response) {             if (response === 1) {                 backgroundGeoLocation.configure(setPosition, service.showError, service.gpsOptions);                 backgroundGeoLocation.start(function(){                     if (service.modalInstance !== null)                         service.modalInstance.close();                 });                 setStatus(gpsStatus.ON);             }else{                 backgroundGeoLocation.showLocationSettings();             }         });     } );

found the answer - isLocationEnabled() checks if the location services is enabled but I was expecting it to check if GPS is turned on.
So it's my mistake


Liquibase: error while connecting to postgresql: FATAL: invalid value for parameter "TimeZone": "Etc/UTC"

Description
An error occurs when trying to run the latest Waltz with a PostgreSQL 14.7: Unexpected error running Liquibase: Connection could not be created to jdbc:postgresql://postgresql-k8s-primary.test-charm-ovos.svc.cluster.local:5432/waltz with driver org.postgresql.Driver.  FATAL: invalid value for parameter "TimeZone": "Etc/UTC"
Full logs:
2023-06-19T06:55:34.848Z [waltz] >>> Update DB
2023-06-19T06:55:35.017Z [waltz] >>> Database is ready.
2023-06-19T06:55:36.801Z [waltz] ####################################################
2023-06-19T06:55:36.801Z [waltz] ##   _     _             _ _                      ##
2023-06-19T06:55:36.801Z [waltz] ##  | |   (_)           (_) |                     ##
2023-06-19T06:55:36.801Z [waltz] ##  | |    _  __ _ _   _ _| |__   __ _ ___  ___   ##
2023-06-19T06:55:36.801Z [waltz] ##  | |   | |/ _` | | | | | '_ \ / _` / __|/ _ \  ##
2023-06-19T06:55:36.801Z [waltz] ##  | |___| | (_| | |_| | | |_) | (_| \__ \  __/  ##
2023-06-19T06:55:36.801Z [waltz] ##  \_____/_|\__, |\__,_|_|_.__/ \__,_|___/\___|  ##
2023-06-19T06:55:36.801Z [waltz] ##              | |                               ##
2023-06-19T06:55:36.801Z [waltz] ##              |_|                               ##
2023-06-19T06:55:36.801Z [waltz] ##                                                ##
2023-06-19T06:55:36.801Z [waltz] ##  Get documentation at docs.liquibase.com       ##
2023-06-19T06:55:36.801Z [waltz] ##  Get certified courses at learn.liquibase.com  ##
2023-06-19T06:55:36.801Z [waltz] ##  Free schema change activity reports at        ##
2023-06-19T06:55:36.801Z [waltz] ##      https://hub.liquibase.com                 ##
2023-06-19T06:55:36.801Z [waltz] ##                                                ##
2023-06-19T06:55:36.801Z [waltz] ####################################################
2023-06-19T06:55:36.801Z [waltz] Starting Liquibase at 06:55:36 (version 4.5.0 #52 built at 2021-09-27 16:19+0000)
2023-06-19T06:55:36.805Z [waltz] Liquibase Version: 4.5.0
2023-06-19T06:55:36.805Z [waltz] Liquibase Community 4.5.0 by Datical
2023-06-19T06:55:37.228Z [waltz] Unexpected error running Liquibase: Connection could not be created to jdbc:postgresql://postgresql-k8s-primary.test-charm-ovos.svc.cluster.local:5432/waltz with driver org.postgresql.Driver.  FATAL: invalid value for parameter "TimeZone": "Etc/UTC"
2023-06-19T06:55:37.228Z [waltz]
2023-06-19T06:55:37.229Z [waltz] For more information, please use the --log-level flag
2023-06-19T06:55:37.292Z [waltz] >>> Database failed to update.
2023-06-19T06:55:37.293Z [pebble] Service "waltz" stopped unexpectedly with code 1

Waltz Version
ghcr.io/finos/waltz:latest
Steps to Reproduce
A kubernetes environment with a Juju controller bootstrapped. Microk8s is sufficient to observe this issue. This environment can easily be replicated by following the following guide (except the Charmcraft section): https://juju.is/docs/sdk/dev-setup
After obtaining a Kubernetes cluster with a Juju controller bootstrapped, run the following:
juju add-model waltz-model
juju deploy postgresql-k8s --trust --channel 14/stable
juju deploy finos-waltz-k8s --channel edge
juju relate finos-waltz-k8s:db postgresql-k8s:db
watch juju status

After all the charms become Active, curl the Waltz Unit IP:
curl WALTZ_UNIT_IP:8080
# curl: (7) Failed to connect to WALTZ_UNIT_IP port 8080: Connection refused

The command should fail. Looking at the Waltz pod logs, the error described above can be seen:
kubectl logs -n  waltz-model pod/finos-waltz-k8s-0 -c waltz

Expected Result
Liquibase should have run successfully, and the Waltz service should have started.
Actual Result
Waltz container crashes due to Liquibase error.

I had two postgresql versions, 14 and 15 installed. I configured and was using 15, but liquibase was using version 14. I just uninstalled version 14, started 15 and cleaned up brew on mac.


Why does the codebase use a “feature” pattern?

Checking at the guides of Quarkus, it seems like it uses the typical java pattern of JAXRS API -> Service - Repository
This code base seem to use “Features” instead of “Services”. Any particular reason why?

Hi, in this code i chose to call "feature" small portions of business rules, trying to give these classes some unique responsibility but nothing prevents the creation of a "service" layer that works as a facade for these "features".


Added MySpace strategy

Added support for MySpace OAuth.

Can you remove the version bump commit? I'll pull if you do.

Done.


job concurrent execute if recreate job very fast

make a job like

schedule  the job， server： A

start server B only start schedule， not init job

then stop server A，wait a minute， B start to schedule the job， then start server A, will see server A and B both running the same job。If server A wait a few seconds and create the job, will only one server running the job.
Full code is here

Still relevant, please reopen


[IOTDB-5627] Optimize Schedule related metrics and Update Performance Overview Dashboard

Add Local/Remote Schedule Metrics
Update Related Docs
Update the Performance Overview Dashboard


Inheirtance of env.spec

After talking through seeing this deployed for real, it is clear that there is an ambition to be able to define one spec for multiple architectures and environments. Aside from the fact that the env.spec currently produces just one env.manifest, there are specialisations which would be useful to be able to add on a per-platform basis.
Something like:
primary env.spec:
env:
 - python 2.7.*
 - iris 1.9.*

site specific env.spec:
extends: primary
env:
 - secret_sauce 3.14.*

platform site specific env.spec:
extends: site specific (these are just names, and will probably just reference git repos)
env:
 - my_platform_specific_package
features:
 - mkl

Of course, conda uses selectors for this purpose, but:

our specifications (currently) are on a per platform basis (i.e. there is 1 env.spec for 1 manifest)
we do not want a specific platform support team to necessarily have merge permissions on the global environment specification
for site specific software (which may have no public release) it is desirable to be able to define these in a non-public facing environment specification whilst sharing as much as possible the core environment

The wart here is that the labels are part of the environment definition and live alongside the env.spec.

these are just names, and will probably just reference git repos

Useful if they could also reference branches.


Remove flash component

Added FlashHelper
Added Flash/default.ctp  (with class handling similar to what's done in the app's element)
Removed FlashComponent as it's responsibilities are now taken over by the helper

Scope for improvement:

@AD7six suggested making the alerts dismissable, which is a matter of updating the attributes of the alert divs.

Updates done as discussed. The helper code is a bit uglyish but does the job.


Snakemake can't find sequencing files if --directory output_dir is used.

Problem Description
When running the default command as recommended in the readme. I get an error saying the long-read file cannot be found.
snakemake --snakefile rotary/rules/rotary.smk \
  --configfile myconfig.yaml \
  --directory output_dir \
  --conda-prefix conda_envs \
  --jobs 20 \
  --use-conda \
  --conda-frontend mamba \
  --rerun-incomplete \
  --reason \
  --printshellcmds 2>&1 | \
  tee rotary.log

Problem Solution (Bodge)
Replaced  --directory output_dir  with  --directory ./. Does it want me to put the input files in the output directory?

@LeeBergstrand Sorry, this is poorly documented, but the absolute paths to the input files are needed in the config file (I think this might be why you are running into trouble). Does that help?

@jmtsuji Thanks


Subtitle field for Accordion component

Is your feature request related to a problem? Please describe.
In our project, we're going to use accordions to make hide 15 forms in 15 Accordion.Item's. We're going to have to implement some custom functionality for this, because we want to be able to see a "short status"/summary of the inside state without opening each accordion.
Describe the solution you'd like
We had our UX'er (@AnnKinge) come up with a design (which we have currently implemented into our application), which looks something like this:

Additional context
As mentioned, we have (for now) implemented the entire Accordion-component as a clone of yours, and added the desired functionality, but that's also the only difference between ours and yours. If you were to add this functionality, we could go back to using EDS maintained code, which would be nice for us.

Instead of cloning the whole component, could you possibly do something like this? (I'm using style-tag here so you see whats going on but of course you can use class or styled-components override as well)

result:

That's clever, I thought about doing something like this, but I just assumed that Accordion.Header wanted a string input. I should've just tried!

No problem! I didn't test this thuroughly with large amounts of text etc. but I think this should work, with just some tweaking to conform with the design. I'll close this issue as resolved 👍


DEX UI not working after helm upgrade

Issue
Dex UI does not work when upgrading Epinio from a previous version. It needs to be recreated.
Steps to reproduce

Install Epinio on previous version to v1.7.0 (ie v1.6.2)
Upgrade Epinio via helm

Observe how the following error screen is displayed:

Workaround
Restart Dex using:
kubectl delete pod -n epinio -l [app.kubernetes.io/name=dex](http://app.kubernetes.io/name=dex)

Just adding this as a reference but we can fix this automatically restarting the pod if something in the secret changes: https://helm.sh/docs/howto/charts_tips_and_tricks/#automatically-roll-deployments
We need to check if that's feasible adding annotations to the Dex deployment.

Having a deeper look at this it's not possible to do it in an "easy" way. The annotation should be added in the Dex deployment subchart (not possible), and possibily it's better to let the user decide how and when restart the deployment of Dex.
I will add a note in the release, but I think it's fine to close this issue like this.


support for multical 66 c

A “request” for standard data #1 with 300 Baud and receiving data with 1200 Baud.
MSComm1.Settings = “300,E,7,2” MSComm1.Output = “/#1”
Delay (10) “Wait to clear output buffer” MSComm1.Settings = “1200,E,7,2”
Temp = MSComm1.Input

done in branch en61107


Create a function in cartodb extension that run some basic health checks

We can start by having a function that checks that all the extension funcion exist in the db at the end of the extension load and have the proper signatures

what about this?

Makes no sense as number and name of functions and the like is varying right now...


Build failing when building tests

[ 65%] Linking CXX executable ../../../out/TypeTest
CMakeFiles/TypeTest.dir/TypeTest.cpp.o: In function `TypeTest::testDataInterval()':
TypeTest.cpp:(.text+0x1ede): undefined reference to `bool QTest::qCompare<bool, int>(bool const&, int const&, char const*, char const*, char const*, int)'
TypeTest.cpp:(.text+0x1f68): undefined reference to `bool QTest::qCompare<bool, int>(bool const&, int const&, char const*, char const*, char const*, int)'
collect2: Error: ld returned 1

I need to do a pull-request again for my fork, should be fixed if I start a pull request with:
https://github.com/ReDucTor/boomerang/commit/7a3ae1e0d9f69479d0a381a7c4c442a75a84c412

@Fighter19   Which compiler ? Which Qt ?

nemerle, it's the linker.
On May 19, 2016 7:23 AM, "Artur K."<EMAIL_ADDRESS>wrote:

@Fighter19 https://github.com/Fighter19 Which compiler ? Which Qt ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/nemerle/boomerang/issues/61#issuecomment-220296952

@hakusdream  The linker is reporting the error, but the missing reference might be caused by bad compiler-qt version: Qt compiled for vs2013 used with vs2015

Mismatched header/binary.

Probably different versions of Qt, one which requires consistent types, the other did not.
It's just a template function that ensures types don't match for comparison and there is no implicit conversions happening, changing them to failing locations from ints to bools fixes the issue. (See earlier referenced commit)

@ReDucTor See #62.
Rebase your commits, also.

Actually this problem occured on ArchLinux with the current up-to-date libraries.

I'll try on Ubuntu though Travis-CI.
On May 19, 2016 10:33 AM, "Fighter19"<EMAIL_ADDRESS>wrote:

Actually this problem occured on ArchLinux with the current up-to-date
libraries.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/nemerle/boomerang/issues/61#issuecomment-220342744


Feature request: "into"

I'd like to be able to initialize a U256 like this:
foo(10.into());

Where foo is a function that takes a U256 as an input.
This operation is not currently supported. I am getting the following error:
help: the following other types implement trait `From<T>`:
  <Uint<256, 4> as From<revm_interpreter::revm_primitives::B256>>
  <Uint<256, 4> as From<primitive_types::U256>>
label: the trait `From<{integer}>` is not implemented for `Uint<256, 4>`

I was going down a bit of rabbit hole, since I (naively) thought that this should be straight forward to implement.
I ran into the wall of the problem of the collision with the autogenerated TryFrom implementation described here.
Imo it would be really neat if we could implement either From or TryFrom based on wether the maximum value of the input type fits into the given Uint variant.
i.e. Uint<256, 4> would implement From<u128> (with the autogenerated "Infallible" TryFrom implementation) whereas Uint<64, 1> would implement TryFrom<u128>. Thereby you would only allow people to use .into() when the conversion is safe and .try_into() when it is not.
I am not sure if this is possible at all in rust though.
I posted my first attempt to implement and the problem I ran into here. Basically it boils down to wether it is possible to implement a trait exclusively for "type variants" whose const generic value satisfy a specific condition. ( BITS >= 128 in the above example).


Hitting escape in a drawer saves the drawer, instead of canceling edits.

Bug Description
Hitting the escape key to close the drawer will still persist data inside the drawer. Since it works with the vue store, I am not sure if this is just ‘how it works’, or whether it would be possible to implement. I think it's possible since this is similar how editing blueprints works in Statamic (canceling, escape or clicking outside is the same):
https://user-images.githubusercontent.com/9193686/132004753-30f980c0-6022-4c50-86b9-7a7d0a0a30ae.mov
How to Reproduce
Edit something and see that it actually updated the values, instead of canceling them.
Extra Detail
https://user-images.githubusercontent.com/9193686/132004706-4db55e06-2907-4692-979c-a946df806e71.mov
Environment
Statamic 3.2.2 Pro
Laravel 8.58.0
PHP 8.0.6
edalzell/blade-directives 3.5
handmadeweb/buildamic dev-main

@sliver37 this was resolved recently wasn't it?


Connecting to Elsaticsearch

Have been trying to elasticsearch thru superset
I typed in SQLAlchemy URI as
elasticsearch+http://[username]:[password]@localhost:9200/
is this is the correct syntax because it tells the connection seems ok but the tables are not showing,what can be done
Also,what happens when we give the mapping in addition like
elasticsearch+http://[username]:[password]@localhost:9200/monthly

Hi,
Please post the output from:
curl -u username:password http://localhost:9200/_cat/indices

green  open .apm-agent-configuration PagMWp9kR4Sw0F2VPx4Zxw 1 0     0 0    283b    283b
yellow open monthlydate              KRMZ77s1Q3aq9OxOSg8rYA 1 1     0 0    283b    283b
green  open .reporting-2020.02.02    27fHmtIITIS4pj1mz_hzBw 1 0     1 0 407.7kb 407.7kb
green  open .kibana_1                si-Se2tKQ46authC3xy5XA 1 0     7 0  22.4kb  22.4kb
yellow open movies                   ArWlVhksQ9e2ISyuMJhqzg 1 1     0 0    283b    283b
green  open .kibana_task_manager_1   uIOLevlbTvK1JJjF5Boghg 1 0     2 0  12.5kb  12.5kb
yellow open apache_superset          YGQvuDmET4KxJt2xliIrxw 1 1     0 0    283b    283b
yellow open daily                    iYCHZLO2TUSJhnGfjZtLOw 1 1     0 0    283b    283b
yellow open monthggd                 WDp-DfRQTxm7MVbKhNyrkg 1 1     0 0    283b    283b
yellow open monthly                  LocrDyBrTKyOvOZFxgga2g 1 1 10102 0   1.6mb   1.6mb
yellow open monthly_table            9XBpKMDhQrmKY4P_xzZwMA 1 1     0 0    283b    283b
yellow open monthlydd                Pkyd32tmQB-FKGeF2M7tHQ 1 1     0 0    283b    283b
yellow open monthly_date             Fh5H1SzlTbOqmNH6R429Lw 1 1     0 0    283b    283b
yellow open table                    sTBFZmIqQAGbQZIs3wBpEw 1 1     0 0    283b    283b

This is probably because all your non internal indexes are on a yellow state.
You can check what elasticsearch will recognize has "tables" by issuing the following command:
POST /_sql?format=txt
{
"query": "SHOW TABLES'"
}
https://www.elastic.co/guide/en/elasticsearch/reference/current/sql-syntax-show-tables.html#sql-syntax-show-tables


Fix bbmap website link (issue#1402)

Minor documentation fix for nf-core/modules#6764 . The bbmap website link moved to a new location, links updated.
Does this even deserve note in CHANGELOG.md?

Created pull-request upstream as per @maxulysse instructions.
https://github.com/nf-core/modules/pull/6768


Update SC bins and npm deps

/cc @jdalton

What's involved with updating? Do you have to compile something?

Just needed to download the libs and package them. Released on NPM at 2.5.0


@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/sxs-feedstock#10.


feat(rn-changelog-generator): Prepend changelogs to the supplied file

Description
This updates the rn-changelog-generator to prepend the newly generated changelog to the supplied --changelog file instead of relying on redirecting the terminal output to a file. It also updates the rn-changelog-generator readme to point to the new @rnx-kit/rn-changelog-generator  package
Resolves https://github.com/facebook/react-native/issues/33011
Test plan
Run node lib/index.js --base v0.66.4 --compare v0.67.0-rc.0 --repo ../../../react-native/ --changelog ../../../react-native/CHANGELOG.md and ensure the file provided is prepended

@lunaleaps would love your input here as well, just to ensure this is the expected behaviour

The "Review" check is failing because of a missing change log file. Can you run yarn change to add one? 🙏

In terms of tackling the issue, this looks good to me! This work will remove one manual step in using the changelog generator. Thank you @gabrieldonadel for working on it!


Update resolver version in workspace

New rust version (1.72) uses a new dependency resolver. We have conflict depending the version of the crates, so we can overwrite for the whole workspace to the newest one.

Without it it didn't even compile in my local setup.


Add output writer for Graphviz DOT format

Add an output writer that outputs relationships in the Graphviz DOT format for generating a graph showing how different files tie together.
Resources for information on the file format include:

https://graphviz.org/doc/info/lang.html
https://en.wikipedia.org/wiki/DOT_(graph_description_language)

The graph "type" output should probably be digraph.

I'm not that far along on this but I'm working on the .DOT output.


Spell Checking

Fixed various spelling errors. The most notable being replacing the "OnlosedChatroom" event in IChatroomNetwork with "OnClosedChatroom"

That's a lot of spelling mistakes 😅 Thanks for this!


fixes down() method in migration

This PR is a quick fix in down() method in the migration.

Thanks!


Coroutine 'ClientSession.close' was never awaited

Looks like there is a bug:
/usr/local/lib/python3.6/site-packages/telepot/aio/api.py:41: RuntimeWarning: coroutine 'ClientSession.close' was never awaited

https://github.com/nickoala/telepot/blob/master/telepot/aio/api.py#L41

Oops. Will fix it in next release.

Just realized, aiohttp.ClientSession.close() has only become a coroutine a few months ago. I will also need to fix the aiohttp dependency.


Create documentation

Any work already started regarding the documentation? What you think of Vitepress?
I'm using it for one of my libraries and it very easy to deal with.

Source code: https://github.com/skarllot/Expressions/tree/main/docs
Published page: https://fgodoy.me/Expressions/

Nothing started yet, what kind of documentation are you thinking? Like auto-generated from XML comments, or just overall guides and quick reference for the most common methods and types?
I haven't used Vitepress myself but I like the documentation pages it generates.

I'm thinking on the second option

That sounds good! The README currently has that but it would definitely look nicer as a page. Can it be hosted on GH Pages easily?

Yes, can be automated using GitHub actions (example).

Nice! Now we just need a logo and a theme color. 😅

The hardest step 😞

What you think about this logo?

Other options:

AI-generated logos still have ways to go. 😅


rename internal "_compute" step to _nodes

The internal "_compute" step (previously "alias_compute") doesn't just set up
compute nodes but also controller, storage, and network nodes, so rename it
like we already did with its sub-steps in #320.
https://github.com/SUSE-Cloud/automation/pull/320

Closing in favour of #323.


Revert "Added support for removing duplicate security group references based on names"

Reverts gogoair/foremast#248

This didn't do what we expected. :/


How to change the positions of hazards and goal?

With the help of the document, I can create a custom environment using the Safety Gym engine, and set the number of hazards. But the initial positions of hazards and goal seem to be random. How can I set the initial positions of hazards with my
own data?

If you look in engine.py at the DEFAULT config dictionary, you will find things such as 'robot_locations' and 'walls_locations' which are by default empty list. If you wish to force the robot to spawn at a specific location or the walls to spawn at specific locations, set these values in your own config to be lists of (x,y) coordinates.
For example, if I wanted my robot to spawn at (0,0) and be surrounded by walls I could do the following:
config = {
  'robot_locations': [(0,0)],
  'walls_num': 8,
  'walls_locations': [(-1,-1), (-1,0), (-1,1), (0,-1), (0,1), (1,-1), (1,0), (1,1)],
}

env = Engine(config)


dynamicBullets is not refreshing correctly on breakpoint

[x] bug

Swiper 5.3.8
Swiper 6.0.4
samle codepen
https://codepen.io/smartfox87/pen/yLewmJK
Video Sample
https://take.ms/2Qv3f
Windows 10 Google Chrome last version

I found a solution
breakpoints: {
320: {
slidesPerView: 1,
slidesPerGroup: 1,
pagination: {
dynamicBullets: true,
dynamicMainBullets: 7,
},
},
430: {
slidesPerView: 2,
slidesPerGroup: 2,
pagination: {
dynamicBullets: false,
},
},
},
on: {
breakpoint: function (swiper) {
if (swiper.params.pagination.dynamicBullets) {
swiper.pagination.init()
} else {
setTimeout(function () {
swiper.pagination.el.style.width = '100%'
swiper.pagination.el.classList.remove('swiper-pagination-bullets-dynamic')
})
}
},
}

This problem doesn't seem to be restricted to dynamicBullets but also normal pagination. The number of bullets doesn't change when the number of slidesPerView & slidesPerGroup changes according to a breakpoint.

I have the same issue on v6.4.10

Same problem here.
I need to call swiper.pagination.update() after a resize because the size and position of the pagination is not calculated again.

Same Issue in Swiper 8.0.6
pagination properties dont change on resize, only on reload

Swiper 8.4.6, pagination bullets count is not updating on resize.

The issue is still present in Swiper 9 and 10. Example https://codesandbox.io/s/swiper-responsive-breakpoints-forked-swhpdq
Make sure the preview shows 3 slides and refresh the preview if needed.
example.txt
Then it shows 5 bullets instead of 3

I am have similar issue in swiper 10.2.0: breakpoints with pagination dont work. I cant disable dynamic bullets on breakpoint, if i resize screen. Only work after full SPA page reloading.

Issue is closed because of outdated/irrelevant/not actual/needed more information/inactivity.
If this issue is still actual and reproducible for latest version of Swiper, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


Add ninja to injective

Description
Added NINJA token from Injective

Checklist

Adding Assets

If adding a new asset, please ensure the following:

[x] Asset is registered to the Cosmos Chain Registry.
[x] Add asset to bottom of zone_assets.json.

The IBC channel referenced in path must be registered to the Chain Registry.
osmosis_unlisted defaults to true (until the respesentation and transferring of the new asset has been validated)

On-chain liquidity
For each new asset, please provide the plan for on-chain liquidity of the asset: (choose one)

[ ] Ready -- A liquidity pool has been created. The pool ID is: ______
[x] Soon -- A pool will be created. (See: Pool Setup Guilde.)

[x] (optional) A preview of the Osmosis Zone app with the new asset added is requested for creating the pool. (Supercharged Liquidity pools cannot be created via Osmosis Zone app)

[ ] StreamSwap -- The token is, or will be, going through a StreamSwap stream; thus, the token should be listed without requiring on-chain liquidity. A Pool ID will be provided in this PR following the stream's completion once the team has had a chance to create a pool using the earned funds.

@DocBadTouch NINJA needs to be added to the Cosmos Chain Registry
https://github.com/cosmos/chain-registry/blob/master/injective/assetlist.json

@sunnya97 Sorry about that, I put the PR for adding everything to Cosmos Chain Registry before this one and thought it would be merged by now since other ones after mine got merged, but I guess not. I will see if I can get that PR pushed through so this can pass the validation checks.


Proxy adding content-length and/or transfer-encoding: chunked header

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.

@angular/cli: 1.2.1
node: 6.11.1
os: darwin x64
@angular/animations: 4.3.0
@angular/common: 4.3.0
@angular/compiler: 4.3.0
@angular/core: 4.3.0
@angular/forms: 4.3.0
@angular/http: 4.3.0
@angular/platform-browser: 4.3.0
@angular/platform-browser-dynamic: 4.3.0
@angular/router: 4.3.0
@angular/cli: 1.2.1
@angular/compiler-cli: 4.3.0
@angular/language-service: 4.3.0
Repro steps.

Proxy a request to a backend server which uses the 'transfer-encoding: chunked' strategy
The log given by the failure.

[HPM] Error occurred while trying to proxy request /api/backend/content from localhost:4200 to http://localhost:4444 (HPE_UNEXPECTED_CONTENT_LENGTH) (https://nodejs.org/api/errors.html#errors_common_system_errors)
Desired functionality.

I think this is related to a bug in the express-http-proxy
https://github.com/villadora/express-http-proxy/pull/47
I want to know if the version of the express proxy used by angular-cli contains the fix above
Mention any other details that might be useful.

The CLI just passes on the proxy options to https://github.com/webpack/webpack, that would be a better place to open an issue like this one. To know the version of express-http-proxy installed in your project, you can do npm list express-http-proxy.